Processing dbms.jvm.additional broken since 4.4.9

Bug report
When upgrading to Neo4j 4.4.9 we realized that processing of the property dbms.jvm.additional is not working anymore for JVM parameters that contain spaces.
Extract from neo4j.conf:
dbms.jvm.additional="-XX:+HeapDumpOnOutOfMemoryError" "-XX:HeapDumpPath=/tmp/" "-XX:OnOutOfMemoryError=/usr/bin/env handle-oom.sh %p /tmp/java_pid%p.hprof" "-Dlog4j.configurationFile=classpath:log4j2.json"
Output of cat /proc/$PID/cmdline | tr '\000' '\n'
/opt/java/openjdk/bin/java
-cp
/var/lib/neo4j/plugins/*:/var/lib/neo4j/conf/*:/var/lib/neo4j/lib/*
-Xms4194304k
-Xmx4194304k
-XX:+HeapDumpOnOutOfMemoryError
-XX:HeapDumpPath=/tmp/
"-XX:OnOutOfMemoryError=/usr/bin/env
handle-oom.sh
%p
/tmp/java_pid%p.hprof"
-Dlog4j.configurationFile=classpath:log4j2.json
-Dfile.encoding=UTF-8
com.neo4j.server.enterprise.EnterpriseEntryPoint
--home-dir=/var/lib/neo4j
--config-dir=/var/lib/neo4j/conf

You can see that the OnOutOfMemoryError parameter has been incorrectly split into 4 separate parameters.
Up until Neo4j 4.4.8 this issue was not present.
In case the format in which parameters with spaces have to be provided within dbms.jvm.additional changed, please refer me to the documentation.
Neo4j Version: 4.4.9
Operating System: RHEL 7.9 w/ Docker 20.10.5
API: Docker
Steps to reproduce
Try to start a Neo4j docker container with the parameters provided above:
docker run --env=NEO4J_ACCEPT_LICENSE_AGREEMENT=yes --env=NEO4J_dbms_jvm_additional='"-XX:+HeapDumpOnOutOfMemoryError" "-XX:HeapDumpPath=/tmp/" "-XX:OnOutOfMemoryError=/usr/bin/env handle-oom.sh %p /tmp/java_pid%p.hprof" "-Dlog4j.configurationFile=classpath:log4j2.json"' neo4j:4.4.9-enterprise

Expected behavior
Neo4j should start up without errors showing an output like this:
2022-08-03 13:01:24.576+0000 INFO  Starting...
2022-08-03 13:01:25.310+0000 INFO  This instance is ServerId{7d2fcb67} (7d2fcb67-eb01-4ad8-b906-e5dad1949c8f)
2022-08-03 13:01:27.847+0000 INFO  ======== Neo4j 4.4.9 ========
2022-08-03 13:01:31.714+0000 INFO  Sending metrics to CSV file at /var/lib/neo4j/metrics
2022-08-03 13:01:31.786+0000 INFO  Bolt enabled on <IP_ADDRESS>:7687.
2022-08-03 13:01:33.369+0000 INFO  Remote interface available at http://localhost:7474/
2022-08-03 13:01:33.377+0000 INFO  id: 873067D4D1C8CD6469576A7FDCAC8B881C3EABFE7919C3FBCB14F2536E6BDD72
2022-08-03 13:01:33.378+0000 INFO  name: system
2022-08-03 13:01:33.378+0000 INFO  creationDate: 2022-08-03T13:01:28.93Z
2022-08-03 13:01:33.379+0000 INFO  Started.

Actual behavior
Neo4j fails to start with the following output:
Error: Could not find or load main class "-XX:OnOutOfMemoryError=.usr.bin.env
Caused by: java.lang.ClassNotFoundException: "-XX:OnOutOfMemoryError=.usr.bin.env

I also raised issue neo4j/neo4j#12914 but realized that the problem is more likely to be in the entrypoint script.

Hi, sorry for the delay in addressing this.
Thank you for including reproduction code. It looks as though this issue is caused by https://github.com/neo4j/docker-neo4j/pull/349 which was a bugfix to allow a dollar character to be passed in to jvm.additional.
I've been looking into it, but so far I've not been able to come up with a solution that works for both dollar and space characters. I'm not sure it's possible.
My personal preference would be to revert the old bugfix to make the 4.4.9 behaviour cosistent with the other 4.4 images, but that would be knowingly re-introducing a bug. I'm going to have to consult the customer support team to figure out what's best.

FYI: I have related ticket #20000 open with enterprise customer support.

Just an idea - not sure about side effects, though:
#!/bin/bash

function params_test()
{
  echo "----------------------------------"
  for param in "$@"; do
    echo "   '$param'"
  done
  echo "----------------------------------"
}

COMMAND='params_test 1 "2 3" 4 x$x'

echo 'Running "$COMMAND" (-> quoting issue)'
$COMMAND
echo

echo 'Running "eval $COMMAND" (-> dollar issue)'
eval $(echo $COMMAND)
echo

echo 'Running improved "eval $COMMAND"'
eval $(echo $COMMAND | sed 's/\$/\\$/g')
echo

Having given this some more thoughts, I think that #349 is conceptually wrong.
Running this command:
docker run --env=NEO4J_ACCEPT_LICENSE_AGREEMENT=yes --env=NEO4J_dbms_jvm_additional="'"'-Dhello=world$x'"' -Dfoo=bar" neo4j:4.4.9-enterprise

Leads to this entry in neo4j.conf (-> as expected):
dbms.jvm.additional='-Dhello=world$x' -Dfoo=bar

However neo4j console --dry-run prints:
/opt/java/openjdk/bin/java -cp /var/lib/neo4j/plugins/*:/var/lib/neo4j/conf/*:/var/lib/neo4j/lib/* -Dhello=world$x -Dfoo=bar -Dfile.encoding=UTF-8 com.neo4j.server.enterprise.EnterpriseEntryPoint --home-dir=/var/lib/neo4j --config-dir=/var/lib/neo4j/conf

My expectation would be that neo4j console --dry-run creates a command that can be c/p executed, such as:
/opt/java/openjdk/bin/java -cp /var/lib/neo4j/plugins/*:/var/lib/neo4j/conf/*:/var/lib/neo4j/lib/* '-Dhello=world$x' -Dfoo=bar -Dfile.encoding=UTF-8 com.neo4j.server.enterprise.EnterpriseEntryPoint --home-dir=/var/lib/neo4j --config-dir=/var/lib/neo4j/conf

So the fix for #349 should be implemented rather in the neo4j command than in the entrypoint.
Niels

OK this should be fixed in 4.4.11 and 4.3.18.
We basically ended up doing the same thing you suggested, which was have --dry-run add correct quoting to arguments. Then also enabling it back with https://github.com/neo4j/docker-neo4j/pull/376
The fixed Neo4js will be in dockerhub once this PR is accepted by the team at Docker:
https://github.com/docker-library/official-images/pull/13102


About external parameters

Hello here, the  data is from KITTI dataset, I gave sensor information and internal and external parameters in yaml as required, but every time I open lidar for depth info in params_camera.yaml, the vio part will crash with the following error. Is this a problem with external parameter settings?

I have same problem,but i also do not how to solve?Did you solve this problem？

I have same problem,but i also do not how to solve?Did you solve this problem？

Not yet, according to the lidar_to_cam parameters given in params_camera.yaml, the  lidar's and camera's coordinate seem to have similar axial directions, but I can not derive this parameter from the camera--> imu and imu-->lidar parameters provided by the sample configuration.

I also encountered this problem. I don't think it is an external reference problem, because the available depth has not been extracted from the image display of lidar in rviz. Is it convenient for private communication

Can we contact by<EMAIL_ADDRESS>

I also encountered this problem. I don't think it is an external reference problem, because the available depth has not been extracted from the image display of lidar in rviz. Is it convenient for private communication

Sure, and my email address is<EMAIL_ADDRESS>

I have the same problem and modify the code so that extrinsic parameters are correctly used for general cases. Check the repository (https://github.com/epicjung/LVI_SAM_fixed) and see if you can fix the issue.

Maybe it is helpful
https://zhuanlan.zhihu.com/p/440288323


Possibility to import workflow

It should be possible to import files to fill out the editor fields. Corresponding to the file extension the content of the file is used to fill out the editors.
functional review: @TobiasNx
code review: @fsteeg

Deployed to https://test.metafacture.org/playground/

Importing seem to work, except if you have a variable as placeholder.
Also should we highlight in a warning that some pieces of the flux might be changed?

+1
(Perhapse you can add in the note, that the one has to test if it works and perhaps adept the workflow to the playgrounds demands. But I do not think that this is needed. For now.)

Changed the info message and deployed again to https://test.metafacture.org/playground/:

+1

Deployed to https://metafacture.org/playground/


Decisions for note and website not getting applied

Decisions for the fields note and website don't seem to get synced. Until we find a permanent solution, I was trying to use decisions as a temporary workaround to add BHL URLs, but it might be useful to be able to update those fields with editorial decisions. Here's what I tried and the decisions posted successfully and weren't listed as broken:
POST http://api.dev.catalogue.life/decision
{
  "lifezones": [],
  "note": "data.dev.catalogue.life",
  "mode": "update",
  "subjectDatasetKey": 1206,
  "datasetKey": 3,
  "subject": {
    "id": "595",
    "name": "Decoriplus hamatus",
    "authorship": "Louw, 1979 Kamiński M.J.",
    "rank": "species",
    "status": "accepted",
    "parent": "Decoriplus",
    "code": "zoological"
  }
}

POST http://api.dev.catalogue.life/decision
{
  "lifezones": [],
  "website": "data.dev.catalogue.life",
  "mode": "update",
  "subjectDatasetKey": 1206,
  "datasetKey": 3,
  "subject": {
    "id": "595",
    "name": "Decoriplus hamatus",
    "authorship": "Louw, 1979 Kamiński M.J.",
    "rank": "species",
    "status": "accepted",
    "parent": "Decoriplus",
    "code": "zoological"
  }
}

Yes, decisions do not have such properties but we could add them easily.
I dont think adding BHL URLs that way is a feasable thing though unless they are just a few.
Should the BHL link not be on the Reference instance rather than on the name?

Using decisions was just a temporary idea for testing while you were away but then I figured out that I could successfully build small assemblies for testing the BHL links instead.
I think attaching links to references is preferable. We might eventually have links to more than 1 external library and also sometimes an external project might want to link to a taxon or name page instead of a reference.


Compilation failure with reducerComponent - ReasonReact.componentSpec(blah, blah..)     vs     React.element

Hi,
Firstly thank you for the reason-react-native, I'm just starting out with it and so far it's been an enjoyable experience.
 I'm having trouble trying to use a reducerComponent and getting the following error:
1 │ /* App.re */
2 │ open ReactNative;
3 │ let app = () => <View> <Switcheroo /> </View>;

This has type:
    'a =>
    ReasonReact.componentSpec(Switcheroo.state, Switcheroo.state,
                               ReasonReact.noRetainedProps,
                               ReasonReact.noRetainedProps, Switcheroo.action)
  But somewhere wanted:
    React.component('a) (defined as 'a => React.element)

  The incompatible parts:
    ReasonReact.componentSpec(Switcheroo.state, Switcheroo.state,
                               ReasonReact.noRetainedProps,
                               ReasonReact.noRetainedProps, Switcheroo.action)
    vs
    React.element

FAILED: subcommand failed.

This was generated by the following code:
/* Switcheroo.re */
open ReactNative;

type state = {toggled: bool};

type action =
  | SetSwitchValue(bool);

let component = ReasonReact.reducerComponent("Switcheroo");

let make = _children => {
  ...component,
  initialState: () => {toggled: false},
  reducer: (action, _state) =>
    switch (action) {
    | SetSwitchValue(v) => ReasonReact.Update({toggled: v})
    },
  render: self =>
    <Switch
      value={self.state.toggled}
      onValueChange={value => self.send(SetSwitchValue(value))}
    />,
};

/* App.re */
open ReactNative;
let app = () => <View> <Switcheroo /> </View>;

Using:
  "dependencies": {
    "react": "^16.9.0",
    "react-native": "^0.61.0",
    "reason-react": "^0.7.0",
    "reason-react-native": "^0.61.0"
  },

FYI, that code came from here https://medium.com/@FiberJW/persistence-in-reasonml-react-native-apps-55d72a8c9e4c which looks like it uses an older version, so I had to alter it a bit.
Where am I going wrong?
Also are there any other examples of using reducerComponent with reason-react-native? This was the only one I could find. I'm coming from Elm and the reducer style looks pretty close to how Elm works, so would really like to use it.
Many thanks,
Paul

ReasonReact is the old API, without hooks and a lot more compilation artifacts. There are a few examples utilising reducers (useReducer hook) in this repo, but if you still like the older API, see reason-react-update.

I would add that you can safely refer to ReasonReact docs (Examples section) for this https://reasonml.github.io/reason-react/docs/en/usereducer-hook.

Note: it's exactly the same for ReasonReactNative which rely on ReasonReact.
ReasonReactNative is just a bunch of bindings for ReactNative APIs & Components so for purely React bindings, ReasonReact is the reference.
Try our Discord #react-native channel if you have more questions https://reason-react-native.github.io/reason-react-native/discord/

Thank you both, managed to get it working with the use reducer hook.

MoOx thank you also for your YouTube videos, they were very helpful for getting started with reason-react-native


Add UsesXml annotation

This may be required to create custom Resources like listview item layout.

I can see how this would be useful for extensions, but for customizing the appearance of built-in components I would prefer that we have a proper editor for letting users manage the layout of items.

UsesXml should also allow to add custom XML code into AndroidManifest like Compiler.java does.

Rather than introduce a @UsesXml annotation with overloaded semantics, I would suggest instead that we update the extension mechanism and compiler to allow for the inclusion of an AndroidManifest.xml and res/... directory structure similar to how Android Archive (.aar) files work. Then, we can have aapt handle merging these along with any .aar files and the application's AndroidManifest.xml similar to how the gradle build system works.
@SusanRatiLane You should familiarize yourself with Compiler.java before starting this project.

Hi @SusanRatiLane,
Are you still working on this?

Hi @SusanRatiLane,
Are you still working on this?

@SusanRatiLane?

This is going to be fixed by #3292

directory structure similar to how Android Archive (.aar) files work

Yes, I think we need to use a technique similar to adding aar libraries here. Because we need to add an R.txt file and a package for a given library that requires an R.class file. I think the R.txt file must contain predefined identifiers like built-in libraries do. But without it aapt2 also indexes resources correctly. I wonder if we should just create the same R.class file that is created in the main application package, in the library package for which we are adding resources.


Extra newlines in generated hosts file around comments

When I run python3 updateHostsFile.py  then scroll to the bottom of the generated hosts file I see this:
#

# The contents of this file (containing a listing of additional domains in

# 'hosts' file format) are appended to the unified hosts file during the

# update process.

<IP_ADDRESS> someblacklisteddomain.com

Which comes from the blacklist file, however in the blacklist file those lines look like this:
# blacklist
#
# The contents of this file (containing a listing of additional domains in
# 'hosts' file format) are appended to the unified hosts file during the
# update process.

<IP_ADDRESS> someblacklisteddomain.com

Somehow the script is inserting extra newline characters, but only around empty lines (/^\s?\n/ in regex parlance)  and lines that start with comments  (/\s?#.*/ in regex parlance).
Additionally comments after a valid line are being stripped, so for example.
In blacklist...
<IP_ADDRESS> someblacklisteddomain.com  # some comment here

Becomes...
<IP_ADDRESS> someblacklisteddomain.com

Interestingly, myhosts is not affected by this issue. Inline comments at the end of valid hosts rules are left as-is, and there are no extra newlines between normal comment blocks.
all 3-rd party blocklists and the included custom blacklist are both affected by this. Example:
# Start adaway.org

# AdAway default blocklist

# Blocking mobile ad providers and some analytics providers

#

Interestingly, myhosts is not affected by this issue. Inline comments at the end of valid hosts rules are left as-is, and there are no extra newlines between normal comment blocks.

all 3-rd party blocklists and the included custom blacklist are both affected by this. Example:
# Start adaway.org

# AdAway default blocklist

# Blocking mobile ad providers and some analytics providers

#

Thanks James!  Yep, this is a bug!

Hello @jameswilson,
welcome here and thanks for this awesome first issue,
I may know where the problem comes from for the \n after commented line.
Will look into this.

I've added @funilrys fork as a remote, checked out the branch issue-777 and confirmed that this does indeed solve the issue.
$ git remote add funilrys git@github.com:funilrys/hosts.git
$ git checkout issue-777
$ python3 updateHostsFile.py
[...]
$ tail -n 8 hosts

# blacklist
#
# The contents of this file (containing a listing of additional domains in
# 'hosts' file format) are appended to the unified hosts file during the
# update process.

<IP_ADDRESS> someblacklisteddomain.com

Thanks @funilrys ... lucky #777   😆 🤞

Thanks guys.   🥇  This is really going to help keep my flavor of OCD in check.


Send_Email Project doesn't compile

When I try to compile the Send_Email project, I get "HTTPClientESP32Ex.h:37:24: fatal error: HTTPClient.h No such file or directory"
What am I missing?
Thanks!

Please go to HTTPClientESP32Ex repo to download the dependency. I also inform this on Readme.
This library has some known bugs in reading email in this issue which I currently try to fix it.

You mean:  Firebase-ESP32 and LineNotify-ESP32 libraries?
Or something different...

I mean this library (ESP32 Mail Client) has some issue https://github.com/mobizt/ESP32-Mail-Client/issues/2. Nothing refer to Firebase-ESP32 and LineNotify-ESP32??

I have already added this dependency:  Required HTTPClientESP32Ex library to be installed. https://github.com/mobizt/HTTPClientESP32Ex
Anything else I am missing?  The readme doesn't mention any other dependencies...

Thats it let compile. Lets look at comment on the top of each  example sketch for the guide.
Please read document first.

Oh well, it still can't find the HTTPClient.h file...  that's too bad, I was hoping to get this to work.

Did you install esp32 core sdk?

https://github.com/espressif/arduino-esp32

Yes, I have made a few ESP32 apps (Sparkfun ESP32 thing)... the version of the SDK I have is from 2017 so I will try and update...

Made it farther now... I will keep playing.  Thanks for your time

You're welcome.


how to skip the "--json" and "--html" report

Hi all,
When I deal with a big fastq file (>20G), it takes long time to generate the .json and .html reports. Is there any way to skip this step ?
Thanks.

Hi,
You can set the parameter -j /dev/null -h /dev/null to avoid producing .json and .html

Hi,
You can avoid these files by setting the -j /dev/null -h /dev/null parameter.

Hi, I asked a similar question and got this answer, set '-j /dev/null -h /dev/null'


Chore add sessionvariable enable_agg_state

Proposed changes
add session variable enable_agg_state
Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

run buildall

run buildall

TeamCity pipeline, clickbench performance test result:
the sum of best hot time: 39.78 seconds
stream load tsv:          459 seconds loaded<PHONE_NUMBER>9 Bytes, about 155 MB/s
stream load json:         21 seconds loaded<PHONE_NUMBER> Bytes, about 107 MB/s
stream load orc:          56 seconds loaded<PHONE_NUMBER> Bytes, about 18 MB/s
stream load parquet:          29 seconds loaded 861443392 Bytes, about 28 MB/s
insert into select:          68.7 seconds inserted 10000000 Rows, about 145K ops/s
https://doris-community-test-1308700295.cos.ap-hongkong.myqcloud.com/tmp/20230703111217_clickbench_pr_171455.html

run buildall


Question: Do renewed certificates create a new certificate thumbprint?

Do renewed certificates create a new certificate thumbprint?
We're just working out how to handle our manual script that gets run and whether we need to reapply the thumbprint again.

Each certificate has a unique thumbprint, so yes you will have to handle that if you are using it for say Exchange ...


Unify the lexer states?

Since "beginning of line" will soon not affect the special character set, we can probably get rid of the plainTextBeginLine and plainTextMidLine states. Instead, we could simply recognize an end-of-line followed by one or more blanks, then sort out the indentation from that.

Fixed in ca95687032de8bc585802bdf124bd70e57bf93d1


Describe callbacks/webhooks

Hi,
What's the best way to describe callbacks/webhooks or any kind of Async communication?
I'm having trouble with adding models since there are no controllers for that.
At the moment, I created private, empty functions to describe callbacks but it is far from being elegant.
here are openapi specs for callbacks:
https://swagger.io/docs/specification/callbacks/

Callbacks are in OpenAPI 3 whilst this package is currently only supporting OpenAPI 2, if I'm not mistaken.
I would also love some tips on how to best set this up, and also +1 the feature request to add support for Callbacks.


serializer.pushPayload is not a function

hello,
I am trying to push a record to the store that I get from a regular ajax callback.. tried using pushPayload but get Uncaught TypeError: serializer.pushPayload is not a function
any ideas why?
      request.done (response)=>
        response['id'] = response.username
        @store.pushPayload 'user', response

this seems to work though.. is there anyway to use the adapter from this package to automatically create json-api doc from my api response?
        # Form json-api object and push into the store.
        data =
          'type': 'user'
          'id': response.username
          'attributes':
            'full_name': response.full_name
            'username': response.username
            'email': response.email
            'uid': response.uid
        @store.push data

We are considering extending DS.RESTAdapter and DS.RESTSerializer (see #128).  pushPayload would be an added benefit of doing that.  Otherwise we'll have to implement this in EDA.  Thanks for reporting this.

ok cool, what is eda?

Ember Django Adapter :-)

doh duh! hah ok thanks

@erichonkanen Can you test the PR in #133 to see if it solves this problem? Thanks.

@benkonrath yep uno momento

@erichonkanen Please reopen this issue if it's still causing problems for you.

what is the recommended way to push an object into the store that is returned from an $.ajax() request? I seem to be getting issues using latest version:
WARNING: Encountered "data" in payload, but no model was found for model name "datum" (resolved model name using client-web@serializer:user:.modelNameFromPayloadKey("data"))
      request.done (response)=>
        data = @store.normalize 'user', response
        @store.pushPayload 'user', data


Where's openai/clip-vit-large-patch14-336 used for in caption.py?

Many thanks for making this code open source!
I noticed in caption.py the model openai/clip-vit-large-patch14-336 is downloaded but can't figure out where it seems to be used?

Its used in the captioner

Its used by Llava to caption the images


PullConsumer AckPolicy Updates

https://github.com/nats-io/nats-architecture-and-design/issues/156

Verified that the client did not perform a check.


function addFile dose not set document's _id as mention in document

Reference from https://github.com/VeliovGroup/Meteor-Files/wiki/addFile

opts.fileId {String} - _id of inserted file, if not set - Random.id() will be used

It seems that addFile() dose not set _id of the new inserted document. I got new generated Random.id() when calling function with fileId set.

Hello @tanutapi ,
Could you please share a piece of code where addFile is used.
And update your question according to our issue template:

Version of Meteor-Files you're experiencing this issue
Version of Meteor you're experiencing this issue
Is it Client or Server issue?
Post Client and/or Server logs with enabled debug option, you can enable "debug" mode in Constructor

I migrated from cfs:gridfs to ostrio:files and my project is quite old. It is still uses<EMAIL_ADDRESS>so I got<EMAIL_ADDRESS>Today I checked with latest release<EMAIL_ADDRESS>and it has no issue anymore. 👍

I'm glad this issue is solved in a simple way :)
Please, support this project by:

Star on GitHub
Star on Atmosphere
Check other packages we have made
Share on Facebook and Twitter
Donate via PayPal


build: set the CMAKE_RUNTIME_OUTPUT_DIRECTORY

Add a CMAKE_RUNTIME_OUTPUT_DIRECTORY to co-locate the runtime components which is important for resolving the interdependencies. Without this the libraries are not really usable.

CC: @bnbarham @rintaro

@swift-ci please test


Added code to get the process id of the instance of PowerShape

This has no immediate benefit but if we ever add a function to get the list of PowerShape objects from the Rot (which currently fails to work) then it will be necessary that this works correctly.

@davidmn I have removed it now.


"For paths starting with $ORIGIN/, the path of the jq executable is substituted for $ORIGIN."

Regarding the "search" path specified e.g. in include directives, the jq manual states:
  For paths starting with $ORIGIN/, the path of the jq executable is substituted for $ORIGIN.

This does not seem to be currently supported by gojq, as evidenced by this transcript:
./jq  -n 'include "testme" {"search": "$ORIGIN/../../../jq"}; testme'
"Found"

./gojq -n 'include "testme" {"search": "$ORIGIN/../../../jq"}; testme'
gojq: compile error: module not found: "testme"

I admit the feature isn't implemented as pointed out, and fixing wouldn't be that difficult. However, I noticed that jq implementation is also broken about resolving $ORIGIN when the program was invoked by its name and looked up from $PATH. Also I'm always confused by the manual statement; the path of the jq executable is actually the directory where the executable is located in.

Although I agree that the expression "the path of the jq executable" is potentially fraught, in practice I don't have any problems with the way the current jq ("master") interprets $ORIGIN (namely, as the actual location of the executable).
For example, I have a jq executable at ~/github/jqMaster/jq and I have a symbolic link to it in ~/jq/, and a file ~/jq/testme.jq, and I have no problem with the include "testme" {search:  _}; directive where _ is a suitable expression beginning with "$ORIGIN", whether I call jq using the symbolic link or its actual location.
$ grep include ORIGIN.jq
include "testme" {search: "$ORIGIN/../jq"};
$ ./link_to_jqMaster -n -f ORIGIN.jq
"Found"
### now adjust the "search:" value ...
$ grep include ORIGIN.jq
include "testme" {search: "$ORIGIN/../../jq"};
$ ~/github/jqMaster/jq -n -f ORIGIN.jq
"Found"

p.s. I realize that the existence of "hard links" makes the "the" in "the actual location" potentially somewhat problematic, but I'm sure you understand.

Does it work with export PATH=~/github/jqMaster/:$PATH and jq -n -f ORIGIN.jq?

I can get it to work using that export command, but there does seem to be a problem as I have to use the "wrong" value for "search". It seems that in this case, jq is mistranslating "$ORIGIN/" as "$ORIGIN/../" !
By the way, have you ever figured out how to use jq's modulemeta built-in?

The issue is tracked by https://github.com/jqlang/jq/issues/1312. Currently jq resolves the argv[0], that is jq when invoked from one of $PATH, and its dirname is ., namely the current working directory. So, unless the command is invoked by absolute or relative path to the executable, $ORIGIN is the current directory.

Aha! So simple, and yet so useless.  Tx!

I implemented this feature.


BackgroundSync being rate limited

Describe the Bug
Unable to perform backgroundSync as application is being rate limited when getting status or reviews. This happens for each open pull request
14:16:22.118Z ERROR wuffle:background-sync:
  additional sync failed RequestError [HttpError]: You have triggered an abuse detection mechanism. Please wait a few minutes before you try again.
      at /my-wuffle/packages/app/node_modules/@octokit/request/dist-node/index.js:66:23
      at runMicrotasks (<anonymous>)
      at processTicksAndRejections (internal/process/task_queues.js:85:5)
      at async Job.doExecute (/my-wuffle/packages/app/node_modules/bottleneck/light.js:405:18) {
    name: 'HttpError',
    status: 403,
    headers: {
      'access-control-allow-origin': '*',
      'access-control-expose-headers': 'ETag, Link, Location, Retry-After, X-GitHub-OTP, X-RateLimit-Limit, X-RateLimit-Remaining, X-RateLimit-Reset, X-OAuth-Scopes, X-Accepted-OAuth-Scopes, X-Poll-Interval, X-GitHub-Media-Type',
      connection: 'close',
      'content-encoding': 'gzip',
      'content-security-policy': "default-src 'none'; base-uri 'self'; block-all-mixed-content; connect-src 'self' uploads.github.com www.githubstatus.com collector.githubapp.com api.github.com www.google-analytics.com github-cloud.s3.amazonaws.com github-production-repository-file-5c1aeb.s3.amazonaws.com github-production-upload-manifest-file-7fdce7.s3.amazonaws.com github-production-user-asset-6210df.s3.amazonaws.com; font-src github.githubassets.com; form-action 'self' github.com gist.github.com; frame-ancestors 'none'; frame-src render.githubusercontent.com; img-src 'self' data: github.githubassets.com media.githubusercontent.com camo.githubusercontent.com identicons.github.com collector.githubapp.com avatars0.githubusercontent.com avatars1.githubusercontent.com avatars2.githubusercontent.com avatars3.githubusercontent.com github-cloud.s3.amazonaws.com; manifest-src 'self'; media-src 'none'; script-src github.githubassets.com; style-src 'unsafe-inline' github.githubassets.com",
      'content-type': 'application/json; charset=utf-8',
      date: 'Tue, 15 Oct 2019 14:16:22 GMT',
      'expect-ct': 'max-age=2592000, report-uri="https://api.github.com/_private/browser/errors"',
      'referrer-policy': 'origin-when-cross-origin, strict-origin-when-cross-origin',
      'retry-after': '60',
      server: 'GitHub.com',
      status: '403 Forbidden',
      'strict-transport-security': 'max-age=31536000; includeSubdomains; preload',
      'transfer-encoding': 'chunked',
      'x-content-type-options': 'nosniff',
      'x-frame-options': 'deny',
      'x-github-media-type': 'github.v3; format=json',
      'x-github-request-id': '0857:2FD5A:3F6880E:4C0E0DA:5DA5D4B4',
      'x-xss-protection': '1; mode=block'
    },
    request: {
      method: 'GET',
      url: 'https://api.github.com/repos/my-repo/repo/pulls/766/reviews',
      headers: {
        accept: 'application/vnd.github.v3+json',
        'user-agent': 'octokit.js/16.30.1 Node.js/12.9.1 (macOS Mojave; x64)',
        authorization: 'token [REDACTED]'
      },
      request: {
        hook: [Function: bound bound register],
        validate: [Object],
        retryCount: 1
      }
    },
    documentation_url: 'https://developer.github.com/v3/#abuse-rate-limits'
  }

Steps to Reproduce
valid configuration
Setup : 7 repos, 900 total issues(750 pull requests)(9 open pull requests)
Version 17 storedump.json running on latest(v27) of application

npm clean-install
npm run buildl
npm run start

Expected Behavior
full refresh of issues should happen, reviewers and status should be updated
wit no issues
Environment
running in a local environment

Browser: [e.g. Chrome 77.0.3865.90]
OS: [e.g. MacOS Mojave]
Board Version: [e.g. 0.27.0]

This also happens when there is no storedump.json file present

Did you add the new required permissions on status, checks and so on? Please check your log for WARN entries.

Check this diff for the newly required permissions.

Yes, I need to add the webhook permission but I already had the checks and status added .
I created a new app to check  and compare the permissions to make sure they matched.
I have my logs outputted to a file attached
wuffle.log

Thanks for sharing the log, is it helpful.

A few insights of my preliminary analysis: GitHub rate-limits to 5000 requests per hour. Looking into your setup, we get decently close to that boundary with a single sync already.
There are a few ways to work around this limitation:

Only sync open issues (will still fail / conflict with rate limits at some point)
Find a clever sync strategy that adjusts the synchronization rate to never hit the rate limit

I've recently run into this as well in a large organization. Prioritizing open issues as well as newer issues would be a great help.

A few more details on how background synchronization works that relates to this issue:
Background synchronization handles open and closed issues differently: Open issues are always synchronized, closed issues are only synchronized if they have been updated in the last 30 days. This is obviously a problem in test setups or super huge organizations that have 1000 or more issues open or closed in thirty days.
https://github.com/nikku/wuffle/commit/31c274758594bdd1d9161ee5d3d28f262f1fbd58 and https://github.com/nikku/wuffle/commit/9dd0c636fbbb93da0c06117ac6350e8cd6b3280f ship some additional changes:

Details (checks, statuses, reviewers) are only being synchronized if issues have recently been updated (in last four hours for closed issues, in the last day for open issues)
Updating an issue on GitHub or on the board ensures that it is being picked up by background sync and synched including their details with the next sync check.
Those things can be fine-tuned via environment variables to suite specific needs and balance open versus closed issue synchronization and initial sync behavior (cf. commits mentioned above).

As the result of this change, initial is going to fetch details for recently updated issues only. As issues are being worked on (i.e. changed via the board or GitHub) details will eventually be background synched.
I expect this to be a good change and look forward for your feedback how this behaves in your organizations. I got a few additional ideas for optimization, however I'd rather see the impact of this change in bigger setups (yours, ours) before dumping huge amounts of additional work in this matter.
Closing this issue for now, looking forward for your feedback.

Thanks for that I have just updated to v0.29.0 and I have been rate limited yet which is great.
I have tried it against with an original storedump.json present and with no storedump.json
Thanks for getting that sorted


Feature/dockerize uaa

Add gradle tasks for creating a docker file, image, pushing the image to a docker registry, and starting the docker container.

@Bitty9
Two items for suggestion

Please fix the CLA. The gitbot will keep warning us that we can't accept your PRs. The instructions are in the gitbot comment

Can I suggest to add a new file called docker.gradle so that we don't keep bloating the existing gradle file. You can still add a simple task in build.gradle that invokes it so that one doesn't have to specify the build file on the command line

Coverage decreased (-0.02%) to 86.477% when pulling 5e2055927e4e06dd37063623eb2eedeb02714206 on GESoftware-CF:feature/dockerize-uaa into c61148bb95dd9ea4c38a40b035d37ff463343873 on cloudfoundry:develop.

Coverage decreased (-0.009%) to 86.486% when pulling 5e2055927e4e06dd37063623eb2eedeb02714206 on GESoftware-CF:feature/dockerize-uaa into c61148bb95dd9ea4c38a40b035d37ff463343873 on cloudfoundry:develop.

Coverage decreased (-0.09%) to 86.4% when pulling 5e2055927e4e06dd37063623eb2eedeb02714206 on GESoftware-CF:feature/dockerize-uaa into c61148bb95dd9ea4c38a40b035d37ff463343873 on cloudfoundry:develop.

@fhanik will do. Also, we had feedback on this feature where teams needed better configuration options, since the pushed image is only configured with the default uaa.yml. We will add better ways to insert yml, should we close the pull request then re-open when ready?

@6palace, the uaa.yml is injectable via the container environment. So if people are just pulling the image from the registry they can modify the uaa.yml, profile, and port bindings for their container using docker -e when they start the container.

Coverage decreased (-0.02%) to 86.475% when pulling 5e2055927e4e06dd37063623eb2eedeb02714206 on GESoftware-CF:feature/dockerize-uaa into c61148bb95dd9ea4c38a40b035d37ff463343873 on cloudfoundry:develop.

Coverage decreased (-0.02%) to 86.475% when pulling 5e2055927e4e06dd37063623eb2eedeb02714206 on GESoftware-CF:feature/dockerize-uaa into c61148bb95dd9ea4c38a40b035d37ff463343873 on cloudfoundry:develop.

Coverage decreased (-0.02%) to 86.475% when pulling 5e2055927e4e06dd37063623eb2eedeb02714206 on GESoftware-CF:feature/dockerize-uaa into c61148bb95dd9ea4c38a40b035d37ff463343873 on cloudfoundry:develop.

Coverage decreased (-0.1%) to 86.389% when pulling e209caae4ae0a2ac514373a5a2b741e1ddfbe25a on GESoftware-CF:feature/dockerize-uaa into c61148bb95dd9ea4c38a40b035d37ff463343873 on cloudfoundry:develop.


start of layout class

Overview
By default a Tableau data extract is set to "Group By Datasource Table."  When using the cli we can add a folder but then it's still not visible when you open the datasource in Tableau.  This adds a show folders setting to enforce columns to make the display show folders.

   <layout _.fcp.SchemaViewerObjectModel.false...dim-percentage='0.5' _.fcp.SchemaViewerObjectModel.false...measure-percentage='0.4' dim-ordering='ordinal' measure-ordering='ordinal' show-structure='false' />

Changes

Add a Layout class
Set show-structure='true'

Testing
in progress

@JustinGrilli  I'm unsure of these things with this PR:

How to apply the changes with layout
If it matters that the layout tag has false... instead of true...
How to deal with the dim-percentage and measure-percentage attributes since they look different than the other elements manipulated


Workaround Python 3.9 FIPS Issue

Patch crypt.py to backport fixes applied to Python 3.10+

Moving to Python 3.10 is another option, but that may have more side effects in the short run.


Asynchronous loading of labels and scalevalues leads to race conditions

Originally reported by: Matthias Neugebauer (Bitbucket: mtneug, GitHub: mtneug)

Referring to issue #62 and pull request #35
Which labels / scalevalues are presented to the user depends on how fast the request is resolved.

Bitbucket: https://bitbucket.org/opencast-community/annotation-tool/issue/64

This is basically a duplicate of #63


docs: links to source code that contain line numbers will need to be fixed

Some .md have links to https://github.com/apache/incubator-pekko-connectors-kafka/tree/main/ files
These files are copies of previous akka files but we have added license headers etc. So the line numbers in the links are likely to be wrong. Look for links with segments like #L169-L170

I can't find anything with line pointers in docs. The generated HTML has some but this appears to be something that should work since the HTML is generated in each build.


Drops

Complete functionality for drops:

[ ] Gold (decide currency!)
[ ] Weapons/Armor
[ ] Potions
[ ] Puzzles
[ ] Spellbooks

@Suds-p So I'm going to make the arrays for these things (the bleeding-edge files are always in my branch, not master, just letting you know, cause I thought I shouldn't put any catastrophic messes in the master for now XD). Maybe I'll work on the story some, most of these user-interactive elements are dependent on the story a bit...

Gotta work out the inventory before the weapons/armor can be done, the potion storage system needs work, probably something more complicated than I'm capable of...

Inventory was done last week, still working out the weapons/armor (cause I had an idea and it's hard to implement 😝 ) but should be at least ready to push soon and other changes wouldn't be drop related anyhows.


Add a feature to disable history

Occasionally I need to disable history for various reasons (removing a lot of files or passwords passed on the command line) during my workflow. I currently disable history by setting the HISTFILE environment variable but with atuin I can't do this because it uses pre-command hooks.
Is there currently a way to disable history for the current shell session and if not can you add a way to do so?

duplicate of #517. Currently you can put a space before the command and it will not be recorded. Or if your using fish, you can set private mode thingy


Added PHP programming logic course in PT_BR

What does this PR do?
Add free PHP programming logic course at PT_BR
For resources
Description
Why is this valuable (or not)?
Because it's a necessary thing for everyone who is starting
How do we know it's really free?
It's on YouTube
For book lists, is it a book? For course lists, is it a course? etc.
It's a course
Checklist:

[x] Not a duplicate
[x] Included author(s) if appropriate
[x] Lists are in alphabetical order
[x] Needed indications added (PDF, access notes, under construction)

Could you please ensure all lists and categories remain in alphabetical order?
You can get more guidance on the formatting and conventions of this repository by checking out the contribution guide.

Okay, I had categorized in order of importance...

Thanks, and welcome to free-programming-books!


Fix commands: docker compose -> docker-compose

Hello Abdur,
Thanks for your contribution!
Since Compose V2, docker-compose is now included inside docker. So docker compose is correct here!
You can find more information here: https://docs.docker.com/compose/cli-command/

Thanks a lot, i have to update then!

No problem at all!
We should add a note in the README about this.


Throw if routes not given to enhancer when server rendering

This addresses an oversight where server rendering does not work if routes are passed as children (or as the routes prop) of <ReduxRouter>. This won't work because the routes need to be defined before the match() action creator is fired.
Now we throw early if no routes are provided to the server store enhancer.
This also adds a getRoutes({ getState, dispatcher }) option to help deal with circular dependencies between routes and the store.

This isn't documented yet, is it? :confused:


Bug 898645. Workaround NRE from runtime.

Workaround for an issue a couple of users are seeing where we don't guard against null being returned and end up with an NRE.

Correction. Should be fix for bug 951609 (not 898645 - although that may be related).


User can receive Nearby Places from SG

Easy way to request current location, send to SafeGraph, and receive list of Nearby Places ranked by likelihood
Probably implemented via config builder. Config builder should enable integrations with other 3rd party APIs (i.e. SG Places API is an example)

instructions in readme


sandbox.c causes compilation failure on Linux in some cases

On musl libc on archs different from x86_64 and aarch64 sandbox.c causes errors when compiling, probably due to musl not having proper implementation of __NR functions for them. Having a build-time option to disable sandboxing would be nice.

Yes, sorry for that; I've only tested on amd64 and aarch64 so far.  Adding a build-time option to disable sandboxing would indeed be easy, but I would prefer to have it working whenever possible.
I'll try to build on x86 to hopefully fix that (from the failure it seems that on x86 the actual syscalls are different).

Thanks. On some arches it fails because of missing SECCOMP_AUDIT_ARCH defines. I found the most complete (and the most boilerplate) solution which contains all Alpine's arches here.

I updated the seccomp filter and added a --disable-sandbox flag for the configure script (that I dislike so it isn't advertised anywhere, but if it's necessary, it's there)
Thanks for liking that bit from dhcpcd, it was really helpful! :)
make regress now passes on alpine x86 and amd64, and on void aarch64 too, so hopefully I haven't screwed anything.  137def5ff4c0f9720391ca88191cf9fee6d8ae9a and 8bb8cf2ad488151879b1d7e5ec7436d38553b1b5 should apply cleanly on top of the 1.6.1 but I can tag a 1.6.2 if it's easier for you, just let me know.

Thanks, now it compiles on all architectures (except riscv64, no gitlab runner to test yet) even without --disable-sandbox. However, patch complains about ChangeLog in 137def5, so tagging would be cleaner.

I haven't thought about the ChangeLog file, sorry!  I tagged the 1.6.2: it's the 1.6.1 with 137def5 and 8bb8cf2 on top.
I'm not familiar with how alpine infrastructure works, so I hope you'd excuse this probably stupid question, but the regression suite (make regress) is ran too as as part of building a package?
Thanks for testing!

Yes, make regress is done in check(). It passes on all architectures except ppc64le, where it hangs on ./runtime somewhy.

Thanks for the clarification! :)
The hangs smell like a possible seccomp failure (where the daemon got killed but the client is still waiting for the reply), even though one hour of wait is still probably a bit too much.  I'm going to emulate a ppc64 and see what's going on.
(also adding a timeout to the regress suite wouldn't harm, sorry for hanging your builds!)

Thanks for efforts

No efforts at all -- I got to play with (virtualized) ppc64 :D
The problem was that ppc64le was matched as ppc64 (big endian).  The seccomp filter would then presume that the current architecture is not the one the program was built on and so disallow any syscall.  It should be fixed as of b24021d, on my vm make regress passes 🎉

Thanks, with this patch it works perfectly. Alpine package will probably be merged soon.


Change horizontal padding to 1rem

@omsmith?

Some refactorizing mixed in there too. Names will probably be easier understand so looks good.


Schema show page

Schema Show Page created.
It includes:

Latest Version of selected schema;
Versions of this schema.

@Guzel738 Do we need testSchema file?


refactor: graceful shutdown on meta node

Signed-off-by: Bugen Zhao<EMAIL_ADDRESS>hereby agree to the terms of the RisingWave Labs, Inc. Contributor License Agreement.
What's changed and what's your intention?
Follow-up of #17533, based on the ideas proposed in https://github.com/risingwavelabs/risingwave/issues/17351#issuecomment-2199698056.

Add a "dummy" election client that always claims we are the leader, unifying the code path no matter whether there's HA.
Not joining sub-tasks any more, since I've reviewed all of their implementations and found almost all of them are only designed to print a message when exiting (correct me if I'm wrong 🙏).

Still leave some TODOs which requires all components to be ready for graceful shutdown, or whose function signatures will change and then impact code of other components (in order to minimize changes in a single PR).
Checklist

[x] I have written necessary rustdoc comments
[x] I have added necessary unit tests and integration tests
[x] All checks passed in ./risedev check (or alias, ./risedev c)

Documentation

[ ] My PR needs documentation updates. (Please use the Release note section below to summarize the impact on users)

Release note
If this PR includes changes that directly affect users or other significant modifications relevant to the community, kindly draft a release note to provide a concise summary of these changes. Please prioritize highlighting the impact these changes will have on users.

#17586
#17608  👈
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @BugenZhao and the rest of your teammates on  Graphite


Version 2.2.0 Milestones

Please define milestones for the next release here:

Add auto versioning to all command line programs

Some ideas:

Add surface maps, and gyrification index to output of antsCorticalThickness.sh

please add ideas here...

@stnava @ntustison will you be happy to have more cortical indices (surface, gyrification, etc) similar to Freesurfer, in ANTs? I know we could get surface LabelGeometryMeasures, but I think we need to:

make it more user friendly, it means that the user runs only the script and gets those indices along with cortical thickness map

After the above step, I'm more than happy to validate it similar to Nick's paper, and based on my preliminary analyses on patients with multiple sclerosis, it is very likely that we could find that ANTs gives more "valid" measures compared to Freesurfer, for example. This will be a manuscript similar to Nick's.

Also we need to have a standard atlas, (like Desikan_Killiany in FS) to use by default. An inexperienced user, who just wants to get things done, will prefer to run recon-all.

I'm happy to spend a few months on this. Ideas?

My motivation to get this done ASAP:
In a patient with multiple sclerosis:
Green: Gray matter surface (or Pial surface)  by Freesurfer
Blue: White matter surface by Freesurfer
Read ribbon: ANTs cortical thickness map

i guess the concern is that FS is missing the temporal pole?  i see some
problems w/ants too but that's perhaps not as critical a failure ...
regardless, i've also been thinking about this and meaning to coordinate
with @ntustison ... we indeed have a (for us) standard multiatlas that we
are using in several different projects and would serve a similar purpose
as the DK FS atlas ...
i think there is a good solution that might follow the lines of

template construction

template multi-atlas labeling

pseudo-geodesic jlf for each subject

surface measurements ... note also that
https://github.com/stnava/ANTs/blob/master/Examples/SurfaceCurvature.cxx is
a powerful function that produces several types of geometric primitive
measurements & which is very flexible but hasnt been applied at large-scale

in brief, agree that we should develop a "put data in box get data out"
(including template) procedure.
brian
On Mon, Jan 26, 2015 at 11:50 AM, Arman Eshaghi<EMAIL_ADDRESS>wrote:

My motivation to get this done ASAP:
In a patient with multiple sclerosis:
Green: Gray matter surface (or Pial surface) by Freesurfer
Blue: White matter surface by Freesurfer
Read ribbon: ANTs cortical thickness map
[image: mail attachment]
https://cloud.githubusercontent.com/assets/4290297/5903708/7d86ba5e-a598-11e4-9d96-0d191de5b84a.png
—
Reply to this email directly or view it on GitHub
https://github.com/stnava/ANTs/issues/156#issuecomment-71493042.

yes the problem is with temporal lobes, where FS's intensity normalizaition has failed. It happens a lot, at least in my patients (MS).

Would running n4 before fs help?
On Jan 26, 2015 12:08 PM, "Arman Eshaghi"<EMAIL_ADDRESS>wrote:

yes the problem is with temporal lobes, where FS's intensity
normalizaition has failed. It happens a lot, at least in my patients (MS).
—
Reply to this email directly or view it on GitHub
https://github.com/stnava/ANTs/issues/156#issuecomment-71496318.

Don't know about n4, but n3 bias field correction is part of recon-all (so
it has been run here). The point however is that prior based segmentation
may be superior to what fs does. FS's problem could be due to intensity
normalisation,GM lesions, etc.
On Tuesday, January 27, 2015, stnava<EMAIL_ADDRESS>wrote:

Would running n4 before fs help?
On Jan 26, 2015 12:08 PM, "Arman Eshaghi"<EMAIL_ADDRESS><javascript:_e(%7B%7D,'cvml','notifications@github.com');>> wrote:

yes the problem is with temporal lobes, where FS's intensity
normalizaition has failed. It happens a lot, at least in my patients
(MS).
--
Reply to this email directly or view it on GitHub
https://github.com/stnava/ANTs/issues/156#issuecomment-71496318.

--
Reply to this email directly or view it on GitHub
https://github.com/stnava/ANTs/issues/156#issuecomment-71566128.

N3BiasFieldCorrection in ANTs is not the same as N3 used in FS.

Sure, how about GM lesions?
On Tuesday, January 27, 2015, Nick Tustison<EMAIL_ADDRESS>wrote:

N3BiasFieldCorrection in ANTs is not the same as N3 used in FS.
--
Reply to this email directly or view it on GitHub
https://github.com/stnava/ANTs/issues/156#issuecomment-71573488.

I wouldn't know.

So that was the motivation.
On Tuesday, January 27, 2015, Nick Tustison<EMAIL_ADDRESS>wrote:

I wouldn't know.
--
Reply to this email directly or view it on GitHub
https://github.com/stnava/ANTs/issues/156#issuecomment-71573904.

Talk is cheap, I will show you my code soon


Show alternative messge to 99% for service install

Currently, services can remain at 99% for a long time, resulting in user confusion or frustration. Instead of showing 99%, we should show a message such as "finalizing install. This could take a minute..."

This issue is in the code review column but has no linked PR. What does that mean?

It seems that a PR was merged that references this issue, but this issue is still in the code review column. Where should it be moved?


CHANGES.md: editorial update for 1.14.0 development

This is an editorial update for CHANGES.md.
Part of the 2019 changelog initiative.

@mesosphere-mergebot label Ready for Review

@mesosphere-mergebot merge-it

(As part of the 2019 changelog initiative I am performing a number of editorial updates to the individual CHANGES.md files across branches and take liberty and responsibility for merging these changes out-of-band).


Type lists should be sets

E.g. TRUTHY_LITERALS
Although there's not much valid reason to use them directly, this could break some code relying on them, e.g. [...] - Set[...] TypeError (no implicit conversion of Set into Array), so v1 might be a good time...

This has been resolved for rubocop-ast, with most constants now private and the rest converted to Sets.
Closing this and related PRs.


use defaults in meta.json

This change allows to have unset fields in meta.json and fall back to their defaults
Currently it is required to explicitly put e.g. fieldnorms: false

Codecov Report

Merging #1310 (cfb5d4a) into main (2e255c4) will decrease coverage by 0.00%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main    #1310      +/-   ##
==========================================
- Coverage   94.05%   94.05%   -0.01%
==========================================
  Files         230      230
  Lines       39350    39382      +32
==========================================
+ Hits        37010    37039      +29
- Misses       2340     2343       +3

Impacted Files
Coverage Δ

src/schema/field_type.rs
83.33% <ø> (ø)

src/aggregation/mod.rs
97.96% <100.00%> (-0.02%)
:arrow_down:

src/core/index_meta.rs
93.17% <100.00%> (+0.13%)
:arrow_up:

src/indexer/merger.rs
98.96% <100.00%> (-0.01%)
:arrow_down:

src/query/phrase_query/mod.rs
92.92% <100.00%> (-0.05%)
:arrow_down:

src/schema/index_record_option.rs
100.00% <100.00%> (ø)

src/schema/text_options.rs
100.00% <100.00%> (ø)

src/fastfield/multivalued/mod.rs
93.93% <0.00%> (-3.04%)
:arrow_down:

src/fastfield/reader.rs
88.23% <0.00%> (-0.85%)
:arrow_down:

src/postings/stacker/expull.rs
98.57% <0.00%> (-0.48%)
:arrow_down:

... and 2 more

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 2e255c4...cfb5d4a. Read the comment docs.


Directive plugins

Okay, after much delay, I have a PR for you that implements plugins.
I've tested it on 2.7, 3.2, 3.3, 3.4, and 3.5. I suspect it also works on 3.0 and 3.1.
I turned shell, link, and clean into plugins. So that the built in plugins are easy to find and understand, to encourage users to copy them and make their own, I put them in a top-level plugins folder, and renamed them so that they're just their directive name.
I changed the name of Executor to Plugin. It can be used from plugins as from dobot import Plugin, or import dotbot and the referred to as dotbot.Plugin.
Let me know if you'd like to me to make any changes.

Looks great! There were a couple minor things I noticed when I read the code, so I left comments on a couple lines.
After that is fixed, I should be able to merge this in.

Also, could you rebase the single commit onto the current HEAD? I don't want 1afc8bf.

By the way, I added some documentation to the readme and wrote a couple really simple tests, so I'll add that commit on top of yours.

Heyo, I noticed you fixed up the issues and merged it in. Awesome ^_^ I use abspath as the path to load modules so that when you print out a module or there's an error involving a module, the message has a readable path.

Yep, I was just about to comment on this. Merged eeb4c28 in aaf093b, added docs/tests in c402396.
Thanks so much for the contribution!


build_visit macOS failing to build due to Cython

Describe the bug
Due to TPL update on 3.3, I needed to rebuild TPLs.
It looks like Cython installed, but then in a follow up package the build is confused:
Installed /Users/harrison37/Work/github/visit-dav/3.3RC/visit/build-mb-3.3.3-darwin-10.15-x86_64-release/thirdparty_shared/third_party/python/3.7.7/darwin-x86_64/lib/python3.7/site-packages/Cython-0.29.17-py3.7-macosx-10.14-x86_64.egg
Processing dependencies for Cython==0.29.17
Searching for Cython==0.29.17
Reading https://pypi.org/simple/Cython/
Download error on https://pypi.org/simple/Cython/: unknown url type: https -- Some packages may not be found!
Couldn't find index page for 'Cython' (maybe misspelled?)
Scanning index of all packages (this may take a while)
Reading https://pypi.org/simple/
Download error on https://pypi.org/simple/: unknown url type: https -- Some packages may not be found!
No local packages or working download links found for Cython==0.29.17
error: Could not find suitable distribution for Requirement.parse('Cython==0.29.17')
Could not install cython
numpy build failed. Bailing out.
Error in build process. See build_visit_log for more information. If the error is unclear, please include build_visit_log and contact the VisIt project via https://visit-help.llnl.gov. You may need to compress the build_visit_log using a program like gzip so it will fit within the size limits for attachments.

I suspect Python was built w/o ssl support.

Fresh Python build:
>>> import ssl
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/Users/harrison37/Work/github/visit-dav/3.3RC/visit/build-mb-3.3.3-darwin-10.15-x86_64-release/thirdparty_shared/third_party/python/3.7.7/darwin-x86_64/lib/python3.7/ssl.py", line 98, in <module>
    import _ssl             # if we can't import it, let the error propagate
ModuleNotFoundError: No module named '_ssl'

Same error with my prior python build, but in that case I have a successful install.

Fresh build Cython imports fine, the numpy build is confused.
Python 3.7.7 (default, Mar  8 2023, 15:25:46)
[Clang 11.0.0 (clang-1<IP_ADDRESS>)] on darwin
Type "help", "copyright", "credits" or "license" for more information.
>>> import Cython
>>>

closing, we updated our python logic and are in a happier place for 3.4


GUI Support

Hello,
I'm interested about this project, and so I wanted to ask if there is any ongoing efforts to build a GUI for this OS.

NodeOS is mostly a server-only OS, but there was some effort to provide it a client-server architecture GUI based on web technologies (http + websockets). Take a look for oneshoot and nodeos-reverse-proxy projects.
For a single-head configuration where the GUI is in the same machine were NodeOS is running, there was some tries to compile and run chromium_shell directly on top of Linux KMS/GDI, but it was somewhat complicated. Some time ago I read about some advances done by Intel where it was possible to watch Youtube videos and have normal web usage, so would it be nice to give it another try. Pull-requests are welcome :-)

NodeOS is mostly a server-only OS, but there was some effort to provide it a client-server architecture GUI based on web technologies (http + websockets). Take a look for oneshoot and nodeos-reverse-proxy projects.

I'll try to hack together a GUI, nice little task ;)

I'll try to hack together a GUI, nice little task ;)

There was some conversations about providing support for https://www.os-js.org/, being served by NodeOS and accessing to its files using WebDAV. Before starting your own GUI, please take a look on this path.

I'll try to hack together a GUI, nice little task ;)

There was some conversations about providing support for https://www.os-js.org/, being served by NodeOS and accessing to its files using WebDAV. Before starting your own GUI, please take a look on this path.

Sure! Thank you for your insight, but it's just that I've been following this project silently for some years, and I've just got confident enough to contribute :)
I just wanted to pay something back to the community

Looking forward to looking at your pull request, @Sant268 . I just saw this repo last December, and I got really intrigued with the idea, but haven't made any contributions towards it myself.
Would really like to see how implement a GUI for this and they stuff you might use.
Cheers!

I just wanted to pay something back to the community

Documentation and keep updated libraries and services are another useful tasks and would help you to understand it better, not everything is just coding :-)

Would really like to see how implement a GUI for this and they stuff you might use.

Take a look at the conversation at https://github.com/os-js/OS.js/issues/742, probably will give you some ideas.


Work out the permissions conundrum

At present, we install c3tk system-wide via sudo. That's fine and well. However ...
This means that all operations that add/remove commands also require the use of sudo. That's also fine and well. However ...
This means that some or all of the user's own ~/.config/c3tk is scoped to the root user. That's not exactly awesome.
Some thoughts:
Perhaps we should install c3tk itself system-wide in a standard location (/usr/local/bin), but install commands in the user's context (~/.config/c3tk/bin).
This muddies up the waters a bit with the notion of system-wide tools, but not to what I'd think is an untenable degree (we're focusing mostly on the single-user workstation use case, but even so, on a multi-user system, a docker image pulled by one is available to all, so it's really just a matter of creating a symlink in a subsequent user's .config/c3tk/bin). We could support multiple paths to get around that as well.

I like the idea of ~/.config/c3tk/bin/  as it will be a) isolated and b) consistent with cmds/ and config/

Throwing together a PR :)

This was addressed by #10


Add link sections to taxonomy sidebar

https://trello.com/c/Ar1mXL7Y/159-add-missing-related-links-to-taxonomy-sidebar
and
https://trello.com/c/lOtOV4wU/160-add-taxonomy-sidebar-to-all-pages-tagged-to-live-taxons

Adds a new taxonomy-navigation component (ported from taxonomy-sidebar in alphagov/static)
Adds policies, topical events, statistical data sets and world locations sections to the taxonomy navigation.
Removes the constraint that taxonomy navigation only shows for guidance formats.

Thanks @vanitabarrett, @tijmenb I think I've covered your review comments now.


Could you help me with the connections?

Hi, I am new to python and using raspberry pi. I found this project and I would love to see my RC car doing what yours does but the major problem I am facing is I do not know which pins of the Pi and arduino to connect. Could you maybe help me with understanding the connections? Thanks in advance.

For Arduino, take a look at #61 and #88
For Raspberry pi, #33


Migration of AFT framework from Terraform OSS to TFC

We deployed AFT with Terraform OSS a few months ago and created many accounts & corresponding resources using that framework. Now we are planning to migrate to Terraform cloud so that state management can be done via TFC workspaces instead of s3 buckets.
So I'm wondering if there are some general guidelines or notes available with the list of prerequisites or points to remember while migrating the existing AFT framework deployed with Terraform OSS to Terraform cloud with minimal impact on the deployed resources? The README.md file in this repo illustrates the process of deploying AFT with TFC but nothing related to the migration I'm looking for.
We don't want to make any impact or changes to the created accounts or provisioned resources, just wanted to migrate from OSS to TFC.

I do not believe AFT is unique here as far as a Terraform migration is concerned, I would recommend you reach out to Hashicorp for more information.


feat: split js_image_layer into 5 layers

This change splits the output of js_image_layer into 5 layers:

node layer contains Node.js and the node fs patches
package_store_3p layers containers all 3p deps in the .aspect_rules_js package store
package_store_1p layers containers all 1p deps in the .aspect_rules_js package store
node_modules_links layer contains all node_modules/foo symlinks into the packages tore
app layer contains everything that doesn't fall into any of the above layers

I think we should put node and node_modules into same layer as those are less likely to change.

hmm how about; node_modules_links  + package_store_3p + node  they all go into the same layer, as they won't change frequently?

hmm how about; node_modules_links + package_store_3p + node  they all go into the same layer, as they won't change frequently?

They could; node_modules_links include 1p links but won't change as much as the actual contents of the 1p links. Is there a downside to more fine grained layers?

Other than network round trip not really, more the better, but i am afraid that we are splitting it too much that it will be confusing.

Other than network round trip not really, more the better, but i am afraid that we are splitting it too much that it will be confusing.

Its transparent for users of rules_oci tho right since tars = [":layers",], in oci_image. Just the rules_docker example has them listed explicitly and we don't care about rules_docker ease of use since that rule set is abandonware now.

Makes sense, i guess this is fine, fine grained layering will give users more power.

Completely agree with fine-grained layering along these lines - it should save a lot of bandwidth and time for orgs that have 100+ services.
Thanks @gregmagolan!

I think this PR has a bad rebase, @gregmagolan could you fix that before i can take a look?

I think this PR has a bad rebase, @gregmagolan could you fix that before i can take a look?

Ahh yes, the base branch was rebased which messed up the commits. All clean again now.


Fix performence in dashboard

Fix #261

Looks good; I need to test more robustly but I think it's on the right track. Would you mind posting some screenshots on the PR?

I used sessions list template as starter.
https://user-images.githubusercontent.com/818385/232865657-b36b2ac1-f72b-4d17-bfc2-ff90c422d4bc.mp4

Hmm, the video isn't working for me but it might be my computer.
I apologize for my slowness here. The underlying issue is that I got a new laptop, and I no longer have a good local development environment for Shynet where I can easily pull your changes and test them out. (And alas, we don't have a way to seed mock data.) This is my fault.

Maybe this will work.
https://user-images.githubusercontent.com/818385/233265840-5c291ea0-7637-49f9-9e45-50683487d4cb.mov
I had the same problem with mock data but I found management command call "demo". I made some cosmetic changes to it but it mostly works really good out of the box.

Good work.


Remove non-portable source commands

source(1) is a bashism and is equivalent to the portable .(1), but
as each command is run in a new shell, spawning a shell to source a file
and exit is at best noop.
Some SUT used by acceptance tests do not use bash(1) as the default
shell (e.g. Ubuntu 22.04 ARM), which cause CI failures because
source is not found (127 exit code):
  1) postgresql task sql task sets up a postgres db
     On host `<IP_ADDRESS>'
     Failure/Error: LitmusHelper.instance.run_shell('source /etc/profile.d/my-custom.lang.sh')
     RuntimeError:
       shell failed
       `source /etc/profile.d/my-custom.lang.sh`
       ======
       [{"target"=>"<IP_ADDRESS>", "action"=>"command", "object"=>"source /etc/profile.d/my-custom.lang.sh", "status"=>"failure", "value"=>{"stdout"=>"", "stderr"=>"sh: 1: source: not found\n", "merged_output"=>"sh: 1: source: not found\n", "exit_code"=>127, "_error"=>{"kind"=>"puppetlabs.tasks/command-error", "issue_code"=>"COMMAND_ERROR", "msg"=>"The command failed with exit code 127", "details"=>{"exit_code"=>127}}}}]

     # ./vendor/bundle/ruby/2.7.0/gems/puppet_litmus-1.3.0/lib/puppet_litmus/puppet_helpers.rb:206:in `run_shell'
     # ./spec/spec_helper_acceptance_local.rb:30:in `export_locales'
     # ./spec/acceptance/sql_task_spec.rb:17:in `block (3 levels) in <top (required)>'

  2) postgresql task sql task execute some sql
     On host `<IP_ADDRESS>'
     Failure/Error:
       result = run_bolt_task('postgresql::sql', 'sql' => 'SELECT count(table_name) FROM information_schema.tables;', 'host' => 'localhost',
                                                 'user' => 'root1', 'password' => 'password', 'database' => 'spec1')
     RuntimeError:
       task failed
       `postgresql::sql`
       ======
       [{"target"=>"<IP_ADDRESS>", "action"=>"task", "object"=>"postgresql::sql", "status"=>"failure", "value"=>{"status"=>"failure", "error"=>"psql: error: connection to server at \"localhost\" (<IP_ADDRESS>), port 5432 failed: FATAL:  password authentication failed for user \"root1\"\nconnection to server at \"localhost\" (<IP_ADDRESS>), port 5432 failed: FATAL:  password authentication failed for user \"root1\"\n", "_error"=>{"kind"=>"puppetlabs.tasks/task-error", "issue_code"=>"TASK_ERROR", "msg"=>"The task failed with exit code 1", "details"=>{"exit_code"=>1}}}}]

     # ./vendor/bundle/ruby/2.7.0/gems/puppet_litmus-1.3.0/lib/puppet_litmus/puppet_helpers.rb:299:in `run_bolt_task'
     # ./spec/acceptance/sql_task_spec.rb:23:in `block (3 levels) in <top (required)>'

Finished in 17 minutes 2 seconds (files took 3.36 seconds to load)
59 examples, 2 failures, 10 pending

Failed examples:

rspec ./spec/acceptance/sql_task_spec.rb:16 # postgresql task sql task sets up a postgres db
rspec ./spec/acceptance/sql_task_spec.rb:21 # postgresql task sql task execute some sql

Because we set variables in profile files, assume they are sourced
during shell startup and will be available to next spawned shells.

@smortex thanks for the work! Should we add maintenance label so it won't appear in the CHANGELOG?

I am not sure about the EL-8 failures, but I see that "AlmaLinux-8, puppet7-nightly" and "Rocky-8, puppet8-nightly" passed this time instead of timeout as they use to :shrug:.  Any suggestion for having more detail about what is going on when these tests block would be welcome!
In the meantime I will merge this since it fix the Ubuntu 22.04 ARM tests.


Fix the Java 7 builds

Potentially same issue as other repos - problems accessing maven.

This is fixed now, right?


Armor Condition Hotfix - Fallout VNDPNP

Changes / Comments
Fixed an issue with Condition Marks being copied on three parts of the sheet.
Roll20 Requests
Comments are very helpful for reviewing the code changes. Please answer the relevant questions below in your comment.

[ ] Does the pull request title have the sheet name(s)? Include each sheet name.
[X] Is this a bug fix?
[ ] Does this add functional enhancements (new features or extending existing features) ?
[ ] Does this add or change functional aesthetics (such as layout or color scheme) ?
[ ] If changing or removing attributes, what steps have you taken, if any, to preserve player data ?
[ ] If this is a new sheet, did you follow Building Character Sheets standards ?

If you do not know English. Please leave a comment in your native language.

Character Sheet Info Roll20 Internal Use only.


improve faceting

Fixes #42
The case where layouting is done with multiple x and y columns (as in the tutorial) is unchanged.
iris = dataset("datasets", "iris")
cols = style([:SepalLength, :SepalWidth], [:PetalLength :PetalWidth])
grp = style(layout_x = dims(1), layout_y = dims(2), color = :Species)
geom = spec(Scatter) + linear
aog2 = data(iris) * cols * grp * geom
aog2 |> draw

The case where faceting is done with a categorical variable is improved: Note the shared x and y labels and the labels on top and to the right.
mpg = dataset("ggplot2", "mpg")
mpg[!,:grpx] .= rand.(Ref(["a", "b", "c"]))
mpg[!,:grpy] .= rand.(Ref(["aa", "bb"]))

cols = style(:Displ, :Hwy)
grp = style(color = :Cyl => categorical, layout_x = :grpx, layout_y = :grpy)
scat = spec(Scatter)
pipeline = cols * scat
aog = data(mpg) * grp * pipeline
aog |> draw

Since I don't know to internal well enough, this could probably be implemented more elegantly. Let me know what I can do to improve the code.
Edit: Things to improve

Should we also link all the x-axis when faceting by a categorical variable?
Placement of legend is not nice

I now use protrusions for spanned x and y labels so that the legend is correctly centered.

I would like to acknowledge help by @jkrumbiegel on Slack. He suggested to use protrusions for the grey labels a while ago and showed me how to use them + clever padding for the spanned labels.


Consolidate zindex for floating elements in the workbench

Currently in the workbench, we have quite a lot of custom floating elements, e.g., context menu, hover, suggest, which are not native OS controls. To ensure they are above other DOM elements, we usually update z-index but since the workbench grid view is flat, most z-index we use now are universal (not limited to its own container) and we kept seeing floating elements overlap each other in funny ways (for example, suggest widget overlaps with sash, context menu, toolbar, etc).
We can probably introduce some const enums for zindex, which can be used as a guidance for updating zindex in your own component:
const enum ZINDEX {
  Sash = 35,
  SuggestWidget = 40,
  QuickInput= 1000,
  ContextView = 2500,
  ModalDialog = 2600
  ...
}

The list will be short and give us an idea of what range of z-index is safe to use in one particular component.

Current status

35

Sash
Find Widget  https://github.com/microsoft/vscode/blob/a8b8e7bd483aeb886e087fd57a0fb6edf23096f9/src/vs/editor/contrib/find/findWidget.css#L9

39

debug toolbar https://github.com/microsoft/vscode/blob/75c0562a26ee96fa03a235dec785333219a2d028/src/vs/workbench/contrib/debug/browser/media/debugToolBar.css#L8

40

Suggest Widget https://github.com/microsoft/vscode/blob/3172af8aef96276479bdab0a8bca6ef5c1806fda/src/vs/editor/contrib/rename/renameInputField.css#L7

50

Hover  https://github.com/microsoft/vscode/blob/ef55a57968e30be8cbe82a2844277550d5c79da4/src/vs/base/browser/ui/hover/hover.css#L10

100

Rename input https://github.com/microsoft/vscode/blob/3172af8aef96276479bdab0a8bca6ef5c1806fda/src/vs/editor/contrib/rename/renameInputField.css#L7

1000

monaco-drag-image https://github.com/microsoft/vscode/blob/97a719dd9b6f56ae724df1beb79944596d998eb6/src/vs/base/browser/ui/list/list.css#L65
notifications-center https://github.com/microsoft/vscode/blob/84043ab3b8f1aa6c60baeecffda3947366b3d888/src/vs/workbench/browser/parts/notifications/media/notificationsCenter.css#L8

2000

menubar-menu-items-holder https://github.com/microsoft/vscode/blob/e62bf133c4562fead1e0a128294066a73a5661b8/src/vs/base/browser/ui/menu/menubar.css#L48
quick-input-widget https://github.com/microsoft/vscode/blob/73274265d7e59b458d70b182d52dccf25d565749/src/vs/base/parts/quickinput/browser/media/quickInput.css#L9

2500

context-view https://github.com/microsoft/vscode/blob/01b37c6b5d991f959f0b31a58055fecf6bc52729/src/vs/base/browser/ui/contextview/contextview.css#L8
titlebar https://github.com/microsoft/vscode/blob/84043ab3b8f1aa6c60baeecffda3947366b3d888/src/vs/workbench/browser/parts/notifications/media/notificationsCenter.css#L8

2600

monaco-dialog-modal-block https://github.com/microsoft/vscode/blob/0d0026e42f74553b3cd7eb8419a9bdaaa7e5573d/src/vs/base/browser/ui/dialog/dialog.css#L13

3000

window-controls-container https://github.com/microsoft/vscode/blob/67c889e39416c958fc7cb84e4814df2225c22852/src/vs/workbench/browser/parts/titlebar/media/titlebarpart.css#L128

9999

welcome overlay https://github.com/microsoft/vscode/blob/a5ee0a3449442e2f05074b38d935d4e361d6b963/src/vs/workbench/contrib/welcome/overlay/browser/welcomeOverlay.css#L12

10000

monaco-pane-drop-overlay https://github.com/microsoft/vscode/blob/7c3f60f86e6776c4a339c23f85fd9a11c3424bbb/src/vs/base/browser/ui/splitview/paneview.css#L123
monaco-editor-overlaymessage https://github.com/microsoft/vscode/blob/55fdf94d67dfc72eaaf25655b7b0d9ff16d2e301/src/vs/editor/contrib/message/messageController.css#L6

100000

screen cast

💯
Just to outline the troubles I have only in the editor tabs control:
https://github.com/microsoft/vscode/blob/de6057e397eb2e2bdc45cf20b1b880f040145c6c/src/vs/workbench/browser/parts/editor/media/tabstitlecontrol.css#L6-L35

@rebornix thanks for doing this. Having a central place where all the z-indexes are listed is a great suggestion, that way when you introduce a new one you can see how it relates to the existing indexes.

@rebornix Are you still planning on adding this?


Enhancement request: Need to accommodate open and close parenthesis in the action bar title (discussed on call)

Require this for business use case requirements please.

Apiary doesn't support () in the name. Instead you can use : <> or {}


nixdoc: 3.0.1 -> 3.0.2

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/nix-community/nixdoc/releases.
meta.description for nixdoc is: Generate documentation for Nix functions
meta.homepage for nixdoc is: https://github.com/nix-community/nixdoc
Updates performed

Rust version update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 3.0.2 with grep in /nix/store/irmb2196v45qzz81ym170x5fpcrl05xd-nixdoc-3.0.2
found 3.0.2 in filename of file in /nix/store/irmb2196v45qzz81ym170x5fpcrl05xd-nixdoc-3.0.2

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

1 total rebuild path(s)

1 package rebuild(s)

First fifty rebuilds by attrpath
nixdoc

Instructions to test this update (click to expand)

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

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

Or:
nix build github:r-ryantm/nixpkgs/a89130bd5a3ef6b1169ca512aa87787f688f72ec#nixdoc

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

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

1 package built:

nixdoc

Maintainer pings
cc @infinisil @asymmetric for testing.

Just built the manual for this PR against #294257. Everything seems fine.


Add option to prepend each line of the output with the task name

Same as https://github.com/sindresorhus/grunt-concurrent/pull/95, but with fixed changes requested and some more tuning, like the possibility to customize task colors per task name.
/cc @flyon

I'm gonna close this as it has a lot of conflicts and I think the scope is too large. I would rather just want a single option that would automatically give each task a unique color. It could use https://github.com/sindresorhus/randoma#randomcolorsaturation in combination with chalk.


Simple grid example displays incorrectly in a narrow IE11 window

Consider the following simple use of the Bootstrap 4 grid - what happens as the container-fluid is made narrower?

      AAAAAAAAAAAAAAAAAAAA

      BBBBBBBBBBBBBBBBBBBB

In most browsers I've tested, as soon as the container-fluid becomes too narrow for the As and Bs to fit side-by-side, the layout changes to stack the As above the Bs. I assume this is the intended behavior.
In IE11, however, when the container-fluid becomes too narrow, the As and Bs overlap.
Is there a way to get IE11 to act the same as the other browsers?
https://jsbin.com/nikupifata/1/edit?html,output

Closing as stale and won't fix—this sounds like one of the flex bugs with IE and not something we'd be able to solve at scale.


Integration with Xamarin Forms

Is it possible to integrate with Xamarin Forms, creating custon render?

Most likely, I don't have any instructions on how to do that and don't plan on making any custom renderer for forms.


Improve init and dev

What issue type does this pull request address? (keep at least one, remove the others)
/kind enhancement
What is this pull request for? Which issues does it resolve? (use resolves #<issue_number> if possible)
This PR changes the behavior of init (see below) and adds a the --interactive / -i flag to devspace dev.
Also resolves #655
Does this pull request has user-facing changes? (e.g. config changes, new/modified commands, new/modified flags)
Yes:

init warns when application listens on a system port and asks for forwarding port
init now writes a different config which by default disables the terminal and does not override the entrypoint anymore

Does this pull request add new dependencies?
No
What else do we need to know?
Due to a cobra issue (https://github.com/spf13/cobra/issues/866), the --interactive flag must be used as -i/--interactive=value instead of -i/--interactive value (or alternatively without value at all).

closing this in favor of #662 662


"verbose" flag to turn on per-file reporting

Part of workaround for
https://github.com/JamieMason/grunt-imageoptim/issues/19

Thanks for this @nwtn, I'm working on a v2 locally and this will be part of that.

Awesome!

Sorry for the wait (474 days!) this is _finally in develop and will be released soon with some other changes.
Thanks @nwtn.

awesome :tada:
no need to apologize for the wait. thanks for a bad-ass project and all the work you do.

Released in<EMAIL_ADDRESS>thanks so much for taking time to help out.


Nodes API missing nodes

Redirect Alert!
Submit your feature request via the Chef Ideas Portal
Nodes API
Ohai Team,!
It will be a good feature if we add also a filter for node missing? or unless there's a suggestion to get those using API (without using Automate's UI)
Thank you Chef!

hi @Doltair -- going forward, pls visit the aha idea portal for new feature ideas. we do this b/c that's what product looks at and we want to make sure new feature requests are being prioritized appropriately by product.
for this issue, we will be able to filter by missing nodes on the "nodes api" (api/v0/nodes/search) once  https://github.com/chef/automate/issues/616 is completed. in the meantime, you can use the cfgmgmt api to find this:
api/v0/cfgmgmt/nodes?pagination.page=1&pagination.size=100&sorting.field=name&sorting.order=ASC&filter=status:missing

Oh very cool @vjeffrey and that is acknowledged, my apologies for that, moving forward I will go there @ idea portal.
Anyhow would this path will be changed overtime after #616?. We're gonna use the path for missing nodes that you gave in the meantime.

no, that path won't change. the story is:

cfgmgmt nodes api: has all the information that comes from the chef infra (chef-client) runs on your nodes
compliance reporting nodes api: has all the information that comes from the chef inspec scans on your nodes
nodes api: has general node metadata and last/penultimate run and scan metadata for your nodes.  it's basically a node store (and the api that drives this epic: https://github.com/chef/automate/issues/911)

i'm going to close this issue since the work requested will be addressed by https://github.com/chef/automate/issues/616