Suggestion: Make this app for iOS

Most people use iOS over Android, so it would be a good idea to make this app for iOS.

I think you'll find that more than 80% of phones use Android. And since Google run this project, it's unlikely they will make an iOS version

Thank you for your reply :-)
On Sun, 30 Aug 2020 at 1:57 am, nutternet<EMAIL_ADDRESS>wrote:

I think you'll find that more than 80% of phones use Android. And since
Google run this project, it's unlikely they will make an iOS version
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/specialprojects-experiments/paperphone/issues/12#issuecomment-683361645,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AQZAHFIQ7EALWBJCXRCP3ATSDGPWVANCNFSM4NAAQQUA
.


Support MathJax in markdown tooltips to allow math notation

Support mathjax in markdowns tooltips so they can include math notation.
This issue was prompted by https://github.com/microsoft/pylance-release/issues/2058#issuecomment-964441171

This issue would require the same work as #124784. No plans at this time to enable this


Cell colors, rotations; more rendering options; bug fix

Hi! Thanks for the amazing library, it's really useful.
In this PR I've added some functionality and fixed a small bug. Below is a Copilot generated summary that I've modified a little to better reflect my intentions.
This pull request includes several enhancements and refactoring changes to the etabs/tex_table.py and etabs/tex_table_cell.py files. Key changes include adding new parameters to the TexTable class, refactoring properties, and adding new methods for enhanced table customization.
Enhancements to TexTable class:

etabs/tex_table.py: Added new parameters caption_after_table, label, and cmd to the __init__ method. These parameters provide more flexibility in table rendering and labeling.
etabs/tex_table.py: Introduced the add_cmidrule method for adding mid-rules to the table, enhancing table formatting options.
etabs/tex_table.py: Added sorting to _rules in the render method, which fixes a bug where rules are rendered incorrectly if they are not added in a top-down order. Also included logic to handle captions and labels more flexibly.

Refactoring of property methods:

etabs/tex_table.py: Renamed _col_count and _row_count properties to col_count and row_count, respectively, to essentially make them public. [1] [2] [3] [4] [5] [6]

Enhancements to TexTableCell class:

etabs/tex_table_cell.py: Added background_color, rotate_degree, and rotate_options attributes to the TexTableCell class to support cell background coloring and text rotation.
etabs/tex_table_cell.py: Updated the styled_value method to apply background color and rotation to cell content.

Hi, thanks for the feedback! and sorry for the late reply. I am glad to know other people find this useful. I reviewed your changes and everything seems good, very nice contribution! I'll soon merge the changes and publish a new version.


data wrangler refactor

For #

[ ] Pull request represents a single change (i.e. not fixing disparate/unrelated things in a single PR).
[ ] Title summarizes what is changing.
[ ] Has a news entry file (remember to thank yourself!).
[ ] Appropriate comments and documentation strings in the code.
[ ] Has sufficient logging.
[ ] Has telemetry for enhancements.
[ ] Unit tests & system/integration tests are added/updated.
[ ] Test plan is updated as appropriate.
[ ] package-lock.json has been regenerated by running npm install (if dependencies have changed).

Codecov Report

Merging #6296 (2a8e989) into main (d8a42ac) will decrease coverage by 0%.
The diff coverage is 32%.

@@          Coverage Diff           @@
##            main   #6296    +/-   ##
======================================
- Coverage     72%     71%    -1%
======================================
  Files        401     406     +5
  Lines      27052   27618   +566
  Branches    3987    4051    +64
======================================
+ Hits       19538   19720   +182
- Misses      5866    6250   +384
  Partials    1648    1648

Impacted Files
Coverage Δ

...c/client/common/application/customEditorService.ts
48% <0%> (-3%)
:arrow_down:

src/client/common/types.ts
100% <ø> (ø)

src/client/datascience/data-viewing/types.ts
100% <ø> (ø)

.../datascience/interactive-common/synchronization.ts
84% <ø> (ø)

...ient/datascience/interactive-ipynb/nativeEditor.ts
69% <ø> (ø)

...cience/interactive-ipynb/nativeEditorOldWebView.ts
85% <ø> (ø)

src/client/datascience/jupyter/jupyterVariables.ts
88% <0%> (-6%)
:arrow_down:

src/client/datascience/jupyter/kernelVariables.ts
67% <0%> (-3%)
:arrow_down:

...ent/datascience/jupyter/pythonVariableRequester.ts
66% <0%> (-7%)
:arrow_down:

src/client/datascience/types.ts
100% <ø> (ø)

... and 31 more


NIFI-483: Use ZooKeeper Leader Election to Auto-Elect Primary Node

NIFI-483: Use ZooKeeper Leader Election to Auto-Elect Primary Node

@markap14 I am continually seeing tests failures both locally and in travis for this PR. See the travis link above.

Functionality looks good. Was able to stop nodes and have the primary node role automatically reassign to a new node. +1


Fixed Index trait and new PathBuf::new()

r? @huonw
(rust_highfive has picked a reviewer for you, use r? to override)

r? @alexcrichton (can't handle this at the moment)

Thanks! We'll be sure to merge this once there's a new nightly available

Ah ok, didn't know about from. Changed it now.


feat: write unit tests in core crate

Feature Request
Describe the Feature Request

write unit tests in core crate
Describe Preferred Solution

### Tasks
- [ ]

This issue is not well scoped I cannot take it


Use the rubocop-rails gem warning given when you bundle install --deployment

Just ran into this issue running our builds. Seems like the issue is around the --deployment flag as it works fine with and without the flag in my dev environment. The --deployment flag is set in our build though so this keeps popping up.
Expected behavior
I would expect no message to be shown post bundle install --deployment since the gem is included in the Gemfile, the bundle install has occurred and rubocop-rails is included in the .rubocop.yml
Actual behavior
Post-install message from rubocop:
Rails cops will be removed from RuboCop 0.72. Use the `rubocop-rails` gem instead.

Put this in your `Gemfile`.

gem 'rubocop-rails'

And then execute:

$ bundle install

Put this into your `.rubocop.yml`.

require: rubocop-rails

Steps to reproduce the problem
bundle install --deployment
.rubocop.yml
require:
  - rubocop-rails
  - rubocop-performance
  - rubocop-rspec

Gemfile.lock
    rubocop (0.71.0)
      jaro_winkler (~> 1.5.1)
      parallel (~> 1.10)
      parser (>= 2.6)
      rainbow (>= 2.2.2, < 4.0)
      ruby-progressbar (~> 1.7)
      unicode-display_width (>= 1.4.0, < 1.7)
    rubocop-performance (1.3.0)
      rubocop (>= 0.68.0)
    rubocop-rails (2.0.0)
      rack (>= 2.0)
      rubocop (>= 0.70.0)
    rubocop-rspec (1.33.0)
      rubocop (>= 0.60.0)

Bundler version 1.17.2
RuboCop version
$ [bundle exec] rubocop -V
0.71.0 (using Parser <IP_ADDRESS>, running on ruby 2.6.2 x86_64-darwin18)

That's going to get fixed by itself with the next release when we remove the cops and the warning with them. //cc @koic


JobService is not started if is in a different process

We have a service that should be run in other process and is specified in manifest like this
<service
      android:exported="false"
      android:name="MyJobService"
      android:process=":otherprocess">
      <intent-filter>
          <action android:name="com.firebase.jobdispatcher.ACTION_EXECUTE"/>"
      </intent-filter>
</service>

This service is never started using Firebase Job Dispatcher
FirebaseJobDispatcher dispatcher = new FirebaseJobDispatcher(new GooglePlayDriver(context));
Job job = dispatcher.newJobBuilder()
                .setService(MyJobService.class)
                .setTag(JOB_TAG)
                .setReplaceCurrent(true)
                .setTrigger(Trigger.executionWindow(0, 1))
                .build();
            dispatcher.schedule(job);

Is there any way to run a JobService in a different process with Firebase Job Dispatcher?

Thanks for the report! There's no technical reason it shouldn't work, but it seems like we may have inadvertently introduced some same-process coupling. I'm looking into this now.


Add bulk action to archive feedback

Motivation
So far it's only possible to delete feedback
Proposed Solution
Add a bulk action that makes it possible to archive feedback
Alternatives
Additional Context

Are you sure we need an additional state for archived feedback? I think the read/unread status is enough. When a feedback entry was processed and only remains in the database for reference, we can mark it as read. When the comment is no longer needed at all, it can be deleted.

This is true if we have the read/unread status and the deletion option (same as in WP). What's still missing atm is that cities geht any kind of notification/hint that new (unread) feedback is existing. in WP is show by a white number on red background next to the menu point.

Alright, I opened issue #350 for the notification counter and I'll close this ticket.


Beta5 is broken with a dependency update on 09/06.

Not sure exactly which package inside the MVC6.
Repro: use dnx beta5 (latest on feed is "1.0.0-beta5-11904")
Create a new project,
Add dependency :
  "dependencies": {
    "Microsoft.AspNet.Mvc": "6.0.0-beta5-*"
  },

and see the world explodes :) Errors like Predefined type 'System.Object' is not defined or imported.
same, for int, void, etc.
updating the dependency to use lower common denominator:  "Microsoft.AspNet.Mvc": "6.0.0-beta5*" works :+1:
I don't know how to track which package/version creates this error, what I know is that yesterday 08/06  this error didn't exist.

more info:
"Microsoft.AspNet.Mvc": "6.0.0-beta5-14011" works without errors
"Microsoft.AspNet.Mvc": "6.0.0-beta5-14063" gives errors.

thanks to David Fowler I just found out that the latest version was 11976 and not 11904.
I was using aspnetvnext feed which have old dnx.
Override feed to use aspnetrelease, updated dnx to 11976 and everything works now.
Just in case someone gets into this.
https://www.myget.org/F/aspnetvnext/api/v2 : Latest version (beta6 right now)
http://www.myget.org/F/aspnetrelease/api/v2 : Release branch versions (I guess this gets updated after dev branch started working on another release and this is moved to stable feed)
on powershell update feed like this:
$env:DNX_UNSTABLE_FEED="http://www.myget.org/F/aspnetrelease/api/v2"
and use -u parameter when installing dnx of course.


Fix: Create room and adding users

Hey @MahirMahdi this looks much cleaner, will be implementing the feature where the video events are sent only to specific rooms so will just pair program with you in this one and hopefully will complete it on weekend !
Thank you for your active contributions !

Anytime. I was going to tell you about this feature. Let's work on this together.😀


Private browsing mode

Gemini is a new internet protocol which:

Takes user privacy very seriously

gemini://gemini.circumlunar.space/

Gemini is also intended to be very privacy conscious, to be difficult to extend in the future (so that it will stay simple and privacy conscious), and to be compatible with a "do it yourself" computing ethos.

gemini://gemini.circumlunar.space/docs/faq.gmi
In this context I suggest to make a setting for "Clean history on exit" AND keep it checked by default for new users.

IMO the privacy that the spec talks about concerns what happens when talking to other computers.
There is a certain level of privacy and security to one's files on the local computer, but I concede that theoretically a bad actor could go looking into ~/.config and find all kinds of data.
A setting for "clean history on exit" is pretty simple to implement so I can add that, but I disagree about having it checked by default. Lagrange's runtime state is designed to be fully and seamlessly restorable on relaunch, and that's the UX I want to keep.

Well... we do have "incognito windows / tabs" for the mainstream web browsers. They do erase history by default.
Gemini is supposed to be more private than the web, so this seems appropriate.

I've just created a pull request for private browsing mode (pull request #123).

I've just created a pull request for private browsing mode (pull request #123).

"clean history on exit" is pretty simple to implement so I can add that

Has this been added yet? I find it quite annoying to see my last browsed page whenever I launch it and couldn’t find either an setting to disable history nor erase history-on-close.

It has not been implemented yet.

annoying to see my last browsed page

Would you actually prefer a setting that specifies which tabs are opened at launch? With such options like "restore previous tabs", "homepage", "about:blank", etc.?
Cleaning history on exit has other consequences, like all known feed entries becoming unread since the app doesn't remember if you've visited the corresponding URLs or not.

Cleaning history on exit has other consequences, like all known feed entries becoming unread since the app doesn't remember if you've visited the corresponding URLs or not.
Oh, that’s right! Don’t use those so didn’t come to my mind.
Would you actually prefer a setting that specifies which tabs are opened at launch? With such options like "restore previous tabs", "homepage", "about:blank", etc.?
That would be great. On setup I thought that setting homepage meant "the on start" page. An option to do so would ease my mind.

Version 1.11 will have an option to "Restore tabs at launch". Disabling that will instead just open each window to the chosen homepage instead of the old tabs. (You can bookmark "about:blank" and make that your homepage.)


searchbar is moving and not sticky

the serachbar is moving down and is not sticky

I would like to work on this issue

Go ahead
Deadline 17 April,2021 11:59 PM IST


removed hnspredicate struct

Fixes #70. With this PR we no longer use the hnspredicate struct

/assign @omerbd21


Can't install angular-timelinejs using bower

Here's the console output.
$ bower install angular-timelinejs --save
bower angular-timelinejs#*      cached git://github.com/ghachey/angular-timelinejs.git#0.2.1
bower angular-timelinejs#*    validate 0.2.1 against git://github.com/ghachey/angular-timelinejs.git#*
bower TimelineJS#master     not-cached git@github.com:ghachey/TimelineJS.git#master
bower TimelineJS#master        resolve git@github.com:ghachey/TimelineJS.git#master
bower angular#~1.2.16           cached git://github.com/angular/bower-angular.git#1.2.28
bower angular#~1.2.16         validate 1.2.28 against git://github.com/angular/bower-angular.git#~1.2.16
bower angular#^1.3.0            cached git://github.com/angular/bower-angular.git#1.3.15
bower angular#^1.3.0          validate 1.3.15 against git://github.com/angular/bower-angular.git#^1.3.0
bower angular#>=1 <1.3.0        cached git://github.com/angular/bower-angular.git#1.2.28
bower angular#>=1 <1.3.0      validate 1.2.28 against git://github.com/angular/bower-angular.git#>=1 <1.3.0
bower angular#^1.2.9            cached git://github.com/angular/bower-angular.git#1.3.15
bower angular#^1.2.9          validate 1.3.15 against git://github.com/angular/bower-angular.git#^1.2.9
bower angular#~1.2.26           cached git://github.com/angular/bower-angular.git#1.2.28
bower angular#~1.2.26         validate 1.2.28 against git://github.com/angular/bower-angular.git#~1.2.26
bower angular#1.3.15            cached git://github.com/angular/bower-angular.git#1.3.15
bower angular#1.3.15          validate 1.3.15 against git://github.com/angular/bower-angular.git#1.3.15
bower TimelineJS#master        ECMDERR Failed to execute "git ls-remote --tags --heads git@github.com:ghachey/TimelineJS.git", exit code of #128 ssh: connect to host github.com port 22: Bad file number fatal: Could not read from remote repository.  Please make sure you have the correct access rights and the repository exists.
Additional error details:
ssh: connect to host github.com port 22: Bad file number
fatal: Could not read from remote repository.
Please make sure you have the correct access rights
and the repository exists.

I have the same problem.  I updated bower to latest then ran this:
c:\dev\wrk\bigdui>bower install -g angular-timelinejs
bower angular-timelinejs#*  not-cached https://github.com/ghachey/angular-timelinejs.git#*
bower angular-timelinejs#*     resolve https://github.com/ghachey/angular-timelinejs.git#*
bower angular-timelinejs#*    download https://github.com/ghachey/angular-timelinejs/archive/v0.2.1.tar.gz
bower angular-timelinejs#*     extract archive.tar.gz
bower angular-timelinejs#*    resolved https://github.com/ghachey/angular-timelinejs.git#0.2.1
bower TimelineJS#master     not-cached git@github.com:ghachey/TimelineJS.git#master
bower TimelineJS#master        resolve git@github.com:ghachey/TimelineJS.git#master
bower angular#~1.2.16       not-cached https://github.com/angular/bower-angular.git#~1.2.16
bower angular#~1.2.16          resolve https://github.com/angular/bower-angular.git#~1.2.16
bower angular#~1.2.16         download https://github.com/angular/bower-angular/archive/v1.2.28.tar.gz
bower angular#~1.2.16          extract archive.tar.gz
bower angular#~1.2.16         resolved https://github.com/angular/bower-angular.git#1.2.28
bower angular#1.3.8         not-cached https://github.com/angular/bower-angular.git#1.3.8
bower angular#1.3.8            resolve https://github.com/angular/bower-angular.git#1.3.8
bower angular#1.3.14        not-cached https://github.com/angular/bower-angular.git#1.3.14
bower angular#1.3.14           resolve https://github.com/angular/bower-angular.git#1.3.14
bower angular#1.3.8           download https://github.com/angular/bower-angular/archive/v1.3.8.tar.gz
bower angular#*                 cached https://github.com/angular/bower-angular.git#1.2.28
bower angular#*               validate 1.2.28 against https://github.com/angular/bower-angular.git#*
bower angular#*                    new version for https://github.com/angular/bower-angular.git#*
bower angular#*                resolve https://github.com/angular/bower-angular.git#*
bower angular#>= 1.0.8          cached https://github.com/angular/bower-angular.git#1.2.28
bower angular#>= 1.0.8        validate 1.2.28 against https://github.com/angular/bower-angular.git#>= 1.0.8
bower angular#>= 1.0.8             new version for https://github.com/angular/bower-angular.git#>= 1.0.8
bower angular#>= 1.0.8         resolve https://github.com/angular/bower-angular.git#>= 1.0.8
bower angular#~1.2              cached https://github.com/angular/bower-angular.git#1.2.28
bower angular#~1.2            validate 1.2.28 against https://github.com/angular/bower-angular.git#~1.2
bower angular#~1.2.4            cached https://github.com/angular/bower-angular.git#1.2.28
bower angular#~1.2.4          validate 1.2.28 against https://github.com/angular/bower-angular.git#~1.2.4
bower angular#>=1.0.*           cached https://github.com/angular/bower-angular.git#1.2.28
bower angular#>=1.0.*         validate 1.2.28 against https://github.com/angular/bower-angular.git#>=1.0.*
bower angular#>=1.0.*              new version for https://github.com/angular/bower-angular.git#>=1.0.*
bower angular#>=1.0.*          resolve https://github.com/angular/bower-angular.git#>=1.0.*
bower angular#~1.x              cached https://github.com/angular/bower-angular.git#1.2.28
bower angular#~1.x            validate 1.2.28 against https://github.com/angular/bower-angular.git#~1.x
bower angular#~1.x                 new version for https://github.com/angular/bower-angular.git#~1.x
bower angular#~1.x             resolve https://github.com/angular/bower-angular.git#~1.x
bower angular#~1.2.0            cached https://github.com/angular/bower-angular.git#1.2.28
bower angular#~1.2.0          validate 1.2.28 against https://github.com/angular/bower-angular.git#~1.2.0
bower angular#1.3.14          download https://github.com/angular/bower-angular/archive/v1.3.14.tar.gz
bower angular#~1.2.23           cached https://github.com/angular/bower-angular.git#1.2.28
bower angular#~1.2.23         validate 1.2.28 against https://github.com/angular/bower-angular.git#~1.2.23
bower angular#>=1.2.0           cached https://github.com/angular/bower-angular.git#1.2.28
bower angular#>=1.2.0         validate 1.2.28 against https://github.com/angular/bower-angular.git#>=1.2.0
bower angular#>=1.2.0              new version for https://github.com/angular/bower-angular.git#>=1.2.0
bower angular#>=1.2.0          resolve https://github.com/angular/bower-angular.git#>=1.2.0
bower angular#*               download https://github.com/angular/bower-angular/archive/v1.4.1.tar.gz
bower angular#~1.x            download https://github.com/angular/bower-angular/archive/v1.4.1.tar.gz
bower angular#>=1.0.*         download https://github.com/angular/bower-angular/archive/v1.4.1.tar.gz
bower angular#>= 1.0.8        download https://github.com/angular/bower-angular/archive/v1.4.1.tar.gz
bower angular#>=1.2.0         download https://github.com/angular/bower-angular/archive/v1.4.1.tar.gz
bower angular#>=1.0.*          extract archive.tar.gz
bower angular#>=1.0.*         resolved https://github.com/angular/bower-angular.git#1.4.1
bower angular#~1.x            progress received 0.3MB of 0.5MB downloaded, 71%
bower angular#1.3.8           progress received 0.3MB of 0.4MB downloaded, 78%
bower angular#>= 1.0.8        progress received 0.3MB of 0.5MB downloaded, 67%
bower angular#>=1.2.0         progress received 0.2MB of 0.5MB downloaded, 32%
bower angular#1.3.14          progress received 0.4MB of 0.4MB downloaded, 83%
bower angular#*               progress received 0.3MB of 0.5MB downloaded, 66%
bower angular#1.3.8            extract archive.tar.gz
bower angular#1.3.8           resolved https://github.com/angular/bower-angular.git#1.3.8
bower angular#1.3.14           extract archive.tar.gz
bower angular#~1.x            progress received 0.5MB of 0.5MB downloaded, 96%
bower angular#1.3.14          resolved https://github.com/angular/bower-angular.git#1.3.14
bower angular#~1.x             extract archive.tar.gz
bower angular#>=1.2.0         progress received 0.2MB of 0.5MB downloaded, 37%
bower angular#>= 1.0.8        progress received 0.4MB of 0.5MB downloaded, 92%
bower angular#~1.x            resolved https://github.com/angular/bower-angular.git#1.4.1
bower angular#>= 1.0.8         extract archive.tar.gz
bower angular#*               progress received 0.4MB of 0.5MB downloaded, 90%
bower angular#*                extract archive.tar.gz
bower angular#>= 1.0.8        resolved https://github.com/angular/bower-angular.git#1.4.1
bower angular#*               resolved https://github.com/angular/bower-angular.git#1.4.1
bower angular#>=1.2.0         progress received 0.2MB of 0.5MB downloaded, 48%
bower angular#>=1.2.0         progress received 0.3MB of 0.5MB downloaded, 60%
bower angular#>=1.2.0         progress received 0.4MB of 0.5MB downloaded, 79%
bower angular#>=1.2.0          extract archive.tar.gz
bower angular#>=1.2.0         resolved https://github.com/angular/bower-angular.git#1.4.1
bower TimelineJS#master        ECMDERR Failed to execute "git ls-remote --tags --heads git@github.com:ghachey/TimelineJS
.git", exit code of #128 ssh: connect to host github.com port 22: Bad file number fatal: Could not read from remote repo
sitory.  Please make sure you have the correct access rights and the repository exists.

Additional error details:
ssh: connect to host github.com port 22: Bad file number
fatal: Could not read from remote repository.

Please make sure you have the correct access rights
and the repository exists.


Enhancement: Support links in logs

Demo:
https://github.com/PrefectHQ/prefect-ui-library/assets/27291717/7a58ae7f-f32d-4024-95d7-5fb19d9f3609

Nice! ❤

Nice one. Will greatly improve our everyday workflow with Perfect!


Same AvailableCapacity can be selected multipile times for different Volumes if AC reservation used

Race Condition is possible between reservationHelper.CreateReservation in extender https://github.com/dell/csi-baremetal/blob/8bc6fefff9dce158383e055393e8f0b7bffd8423/pkg/scheduler/extender/extender.go#L351
and resHelper.ReleaseReservation in csi-controller https://github.com/dell/csi-baremetal/blob/8bc6fefff9dce158383e055393e8f0b7bffd8423/pkg/common/volume_operations.go#L197
Possible solution.
Change ACR selection logic.
https://github.com/dell/csi-baremetal/blob/8bc6fefff9dce158383e055393e8f0b7bffd8423/pkg/base/capacityplanner/helpers.go#L119

Solution: we should always select "oldest" reservation if multiple reservations match volume creation request.
AvailableCapacityReservation CreationTimestamp used for this purpose.


cmake error

Hi,
I tried to build an run emit C, so I followed the steps in README.
I used [build_tools/build_mlir.sh](https://github.com/iml130/mlir-emitc/blob/main/build_tools/build_mlir.sh) to build and install LLVM and MLIR and set BUILD_DIR and PREFIX.
Here is my setting about BUILD_DIR and PREFIX.
setenv BUILD_DIR "/users/student/mr108/cwlu19/ic8/mlir-emitc/llvm/llvm-build"
setenv PREFIX "/users/student/mr108/cwlu19/ic8/mlir-emitc/llvm-install"

After I run the command cmake -G Ninja -DCMAKE_C_COMPILER=clang -DCMAKE_CXX_COMPILER=clang++ .. -DMLIR_DIR=$PREFIX/lib/cmake/mlir -DLLVM_EXTERNAL_LIT=$BUILD_DIR/bin/llvm-lit, I met this error.
CMake Error at llvm-install/lib/cmake/llvm/AddLLVM.cmake:512 (add_dependencies):
  The dependency target "MLIRArithmetic" of target "obj.GmlStTilingInterface"
  does not exist.
Call Stack (most recent call first):
  llvm-install/lib/cmake/mlir/AddMLIR.cmake:130 (llvm_add_library)
  third_party/mlir-hlo/lib/Dialect/gml_st/transforms/CMakeLists.txt:20 (add_mlir_library)

CMake Error at llvm-install/lib/cmake/llvm/AddLLVM.cmake:512 (add_dependencies):
  The dependency target "MLIRArithmetic" of target "obj.MhloToLhloConversion"
  does not exist.
Call Stack (most recent call first):
  llvm-install/lib/cmake/mlir/AddMLIR.cmake:130 (llvm_add_library)
  third_party/mlir-hlo/lib/Dialect/mhlo/transforms/CMakeLists.txt:96 (add_mlir_library)

CMake Error at llvm-install/lib/cmake/llvm/AddLLVM.cmake:512 (add_dependencies):
  The dependency target "MLIRArithmeticTransforms" of target
  "obj.MhloToLhloConversion" does not exist.
Call Stack (most recent call first):
  llvm-install/lib/cmake/mlir/AddMLIR.cmake:130 (llvm_add_library)
  third_party/mlir-hlo/lib/Dialect/mhlo/transforms/CMakeLists.txt:96 (add_mlir_library)

CMake Error at llvm-install/lib/cmake/llvm/AddLLVM.cmake:512 (add_dependencies):
  The dependency target "MLIRArithmetic" of target "obj.MLIRMhloUtils" does
  not exist.
Call Stack (most recent call first):
  llvm-install/lib/cmake/mlir/AddMLIR.cmake:130 (llvm_add_library)
  third_party/mlir-hlo/lib/utils/CMakeLists.txt:17 (add_mlir_library)

CMake Error at llvm-install/lib/cmake/llvm/AddLLVM.cmake:939 (add_dependencies):
  The dependency target "MLIRAffineAnalysis" of target "emitc-opt" does not
  exist.
Call Stack (most recent call first):
  tools/emitc-opt/CMakeLists.txt:46 (add_llvm_executable)

Did I misunderstand anything?
Thanks!

I just did a clean build in connected with MLIR installed via our bash script. Aside from me using bash (and therefore export instead of sentenv), the build passes on my machine. Taking a closer look to the build log you provided, I think you're trying to build against an outdated LLVM/MLIR version. Please make sure you use the LLVM commit given in build_tools/llvm_version.txt.

Please re-open if this does not resolve your issue.


[Model] support minicpm3

Support MiniCPM3-4B.
github repo: https://github.com/OpenBMB/MiniCPM
huggingface repo: https://huggingface.co/openbmb/MiniCPM3-4B
BEFORE SUBMITTING, PLEASE READ THE CHECKLIST BELOW AND FILL IN THE DESCRIPTION ABOVE

 PR Checklist (Click to Expand)
Thank you for your contribution to vLLM! Before submitting the pull request, please ensure the PR meets the following criteria. This helps vLLM maintain the code quality and improve the efficiency of the review process.
PR Title and Classification
Only specific types of PRs will be reviewed. The PR title is prefixed appropriately to indicate the type of change. Please use one of the following:

[Bugfix] for bug fixes.
[CI/Build] for build or continuous integration improvements.
[Doc] for documentation fixes and improvements.
[Model] for adding a new model or improving an existing model. Model name should appear in the title.
[Frontend] For changes on the vLLM frontend (e.g., OpenAI API server, LLM class, etc.)
[Kernel] for changes affecting CUDA kernels or other compute kernels.
[Core] for changes in the core vLLM logic (e.g., LLMEngine, AsyncLLMEngine, Scheduler, etc.)
[Hardware][Vendor] for hardware-specific changes. Vendor name should appear in the prefix (e.g., [Hardware][AMD]).
[Misc] for PRs that do not fit the above categories. Please use this sparingly.

Note: If the PR spans more than one category, please include all relevant prefixes.
Code Quality
The PR need to meet the following code quality standards:

We adhere to Google Python style guide and Google C++ style guide.
Pass all linter checks. Please use format.sh to format your code.
The code need to be well-documented to ensure future contributors can easily understand the code.
Include sufficient tests to ensure the project to stay correct and robust. This includes both unit tests and integration tests.
Please add documentation to docs/source/ if the PR modifies the user-facing behaviors of vLLM. It helps vLLM user understand and utilize the new features or changes.

Notes for Large Changes
Please keep the changes as concise as possible. For major architectural changes (>500 LOC excluding kernel/data/config/test), we would expect a GitHub issue (RFC) discussing the technical design and justification. Otherwise, we will tag it with rfc-required and might not go through the PR.
What to Expect for the Reviews
The goal of the vLLM team is to be a transparent reviewing machine. We would like to make the review process transparent and efficient and make sure no contributor feel confused or frustrated. However, the vLLM team is small, so we need to prioritize some PRs over others. Here is what you can expect from the review process:

 After the PR is submitted, the PR will be assigned to a reviewer. Every reviewer will pick up the PRs based on their expertise and availability.
 After the PR is assigned, the reviewer will provide status update every 2-3 days. If the PR is not reviewed within 7 days, please feel free to ping the reviewer or the vLLM team.
 After the review, the reviewer will put an  action-required label on the PR if there are changes required. The contributor should address the comments and ping the reviewer to re-review the PR.
 Please respond to all comments within a reasonable time frame. If a comment isn't clear or you disagree with a suggestion, feel free to ask for clarification or discuss the suggestion.

Thank You
 Finally, thank you for taking the time to read these guidelines and for your interest in contributing to vLLM. Your contributions make vLLM a great tool for everyone!

Thanks for implementing this! It looks like there are quite a few duplicate modules with the old MiniCPM though. If you don't plan to modify those modules, could you import the existing ones from minicpm.py?

Also, it would be great if you could add a test to show that the model matches the HF output.

Also, it would be great if you could add a test to show that the model matches the HF output.

Could you give an example please? Should I test on one case or an entire benchmark? And where should I add it?

We usually just test that the output of the model on some example prompts are consistent with those of HF. For example, you can take a look at tests/models/test_models.py.

Update: The new locations of the model tests have been adjusted in #7820, so please merge from main.

I have added a model test and run testing on 1xA100. The output of vllm on example prompts are consistent with those of HF.


updated daily script

PR-Codex overview
This PR automates the process of updating the list of releases and creating a pull request for each new release.
Detailed summary:

Added COMMIT_MESSAGE variable to auto-pull.py to include name and latest version of release in commit message
Changed cron schedule in .github/workflows/releases.yml to run hourly instead of daily
Installed Ruby and PDD in .github/workflows/releases.yml
Added echo command to output COMMIT_MESSAGE in .github/workflows/releases.yml
Modified create-pull-request step in .github/workflows/releases.yml to include commit-message, title, and body with COMMIT_MESSAGE and updated branch name

✨ Ask PR-Codex anything about this PR by commenting with /codex {your question}

@yegor256 This pull request changes the behaviour of auto-pull script. It stops after finding a new release and create separate pull request for new library version. Also I have changed the script running time to one in a hour. Please have a look

@Graur thanks!


Cleanup NetBeans license header for module jellytools.java

Review and cleanup the license header in jellytools.java
Oracle donated NetBeans to the Apache Software Foundation.
Your task now is to replace the old license header that has Oracle (and/or Sun Microsystems
and/or NetBeans) in it with the standard Apache Software Foundation license header.
For an empty file that supports headers you must add the Apache license header.
Note that you have to use the proper license header for Java, .properties or HTML files.
Look at the nearby files for templates and copy-paste the license header.
When done commit the changes with a commit message that starts with first line
"[NETBEANS-54] Module Review " and a message body describing your review.
See a longer explanation about what we are doing here: https://cwiki.apache.org/confluence/display/NETBEANS/List+of+Modules+to+Review
Please announce in a comment when you have started work.
Files that need to be looked at in jellytools.java:
jellytools.java/test/qa-functional/data/SampleProject/build.xml
jellytools.java/test/qa-functional/data/SampleProject/src/sample1/html.html
jellytools.java/test/qa-functional/data/SampleProject/src/sample1/properties.properties
jellytools.java/test/qa-functional/data/SampleProject/src/sample1/unrecognized
jellytools.java/test/qa-functional/data/SampleProject/src/sample1/url.url

Working on this.

Sorry @phinaliumz , I see somebody closed this already upstream.

Actually, no, I see it references another issue number... Carry on.

This module seems no longer to be reported in the rat report.
The previously reported files are now excluded based on:
https://github.com/emilianbold/incubator-netbeans/blob/cc33dff96198b845b55aa4f3f9da4ad6f627a174/nbbuild/build.xml#L1995

Thank you!


Successfully installed the package, but the GH components report errors.

Describe the bug
Successfully installed the package, but the GH components report errors saying "No module named compas_eve", "No module named compas_timber.fabrication" etc.
To Reproduce
Steps to reproduce the behavior:

Create a virtual environment using Conda then install Compas-XR using pip
Install Compas-XR for Rhino environment
Open Rhino7 or Rhino8 then open Grasshopper
See red components

Expected behavior
The compas GH component should recognize where its dependencies are.
Screenshots

Desktop (please complete the following information):

OS: Windows 11
Python version 3.11.9
Python package manager pip

Thank you for your help in advance!

Additional Information
In Rhino8 GH, the IronPython component can find the compas_xr package, while the Python3 component couldn't. Just as the PATH variable showing above.

Hey! Thanks for filing this one! Even before today's official launch! That's quite cool!
About the issue that you report, there are two different parts to it: one is the issues when using IronPython in Rhino 8, and a different one is the one using Python 3 in Rhino 8.
Regarding the first one, I believe the issue is actually part of the COMPAS core installation step, and it sounds similar to this issue https://github.com/compas-dev/compas/issues/1351
Regarding Python 3 on Rhino, compas_xr is not yet tested on it, so I would recommend sticking to IronPython for the time being until we add full support for it. In any case, the installation of compas libraries for Python 3 on Rhino 8 is not done using the usual python -m compas_rhino.install command, at least for now, there's a separate installation mode for that. It might also be possible to use the # r: syntax of rhino 8 to install, but again, this isn't really tested on our side yet. (We're getting there though!)

Thank you for your help and kind words! I am doing my Master's research about XR, and your work always inspires me. I really wish I could be at the launch speech. Will there be a recording available to watch on YouTube?
I've tried the method metioned in this issue: https://github.com/compas-dev/compas/issues/1351. The legacy ghpython component can find "compas" related packages for now.
But there is a new problem:

I solve it by making the following change:

I think it's because the MQTTnet.dll file is not in the same directory as the mqtt_cli.py file.


Create a footer

Edit the Footer.js component in src/components/Footer.js path. You can test your code by placing  under the route section in src/App.js

Hey i would like to work on this issue can you assign me #6

Okay I'm assigning you


Spammy logging from KubernetesClientConfigMapPropertySource

This started being logged after updating to latest version. With default settings this is logged every 15 seconds. Surely this should be debug and not info?
https://github.com/spring-cloud/spring-cloud-kubernetes/blob/3351926041a630aee0961ba0e1be8f035e4ba2ca/spring-cloud-kubernetes-client-config/src/main/java/org/springframework/cloud/kubernetes/client/config/KubernetesClientConfigMapPropertySource.java#L62

Are you using polling refresh configuration?

Sorry I forgot to mention this, yes I use polling (with default period of 15 seconds).


Update for TyCtxt<'a, 'gcx, 'tcx> -> TyCtxt<'tcx>.

See https://github.com/rust-lang/rust/pull/61722 and https://github.com/rust-lang/rust/pull/61817 for some backstory.
cc @mark-i-m @rust-lang/compiler

@mark-i-m The QueryCx stuff is longer term, now that we got to tcx: TyCtxt<'tcx> directly.


update Chinese translation

@WuJun2016 / @n313893254 Would it be possible if someone gave this a quick review, is this required for 2.6.4?

I think it should be merged to 2.6.4

Thanks all!


[Site] horizontal scrollbars are styled oddly, due to style intended only for vertical

Recently some default styling was added for vertical scrollbars, however it doesn't correctly style horizontal ones. The result is horizontal scrollbars are fatter than vertical ones.

https://github.com/Semantic-Org/Semantic-UI/blob/796d2f66e26ae5bb76c604fe6a36914ad3d2fe2a/src/definitions/globals/site.less#L135-L138
Because it removes the webkit-appearance, we need to also set the height for horizontal ones. This can be done either by splitting the styles and using the horizontal/vertical pseudo classes or just by putting a height in the same selector, since (currently) height placed here will only affect horizontal scrollbars, not vertical.
I would have just PR'd this, but I wanted to first confirm what to call the LESS variable? @customScrollbarHeight? Is this ambiguous?

Hi @jayphelps, sorry for the delay. @customScrollbarWidth has already been established, so I don’t think we have any other choice other than @customScrollbarHeight—both variables should probably use a similar naming pattern, but changing @customScrollbarWidth will break 3 previous versions.
Thank you for the contribution!


Adapt our Managed EFK to the new OpenSearch chart (prev. Open Distro)

The OpenSearch project is the new home for Amazon's previous distribution of Elasticsearch (Open Distro for Elasticsearch): https://aws.amazon.com/blogs/opensource/introducing-opensearch/
@giantswarm/team-halo Let's adapt our EFK to use this. Can prioritize in the next cycle.
This is the way forward since Elastic changed Elasticsearch and Kibana's license.
Tasks

[ ] Replace elasticsearch-curator by odfe/index-management https://github.com/giantswarm/efk-stack-app/issues/64

Thu, 22. Apr 2021 in @giantswarm/team-halo standup:
This is for logging, not database. Maybe we don't need a migration path.
We can provide parallel installation. Just more resources usage, but think that's fine.

hi @webwurst you checked this out, right? What is your recommendation?

At the moment OpenSearch is released as beta1 and helm/charts are not adopted yet. Let's still follow OpenDistro for upgrades, but only small changes are expected there anyway.

It would be more important cve-2022-22965 affects OpenDistro

EFK has been deprecated we no longer plan on doing this.


Missing students

complete missing students functionality has beend implemented and tested

80%  of system functionality completed, additionaly functionalities requires backend logic


Pr 70

Coverage increased (+0.007%) to 99.636% when pulling 72232eb84264cbf219bf29049832b544a453a6e8 on pr-70 into 54677f891053e1c5474babd7e0512891c6001801 on master.


⚠ Bofrost.lu has degraded performance

In 418dc37, Bofrost.lu (https://www.bofrost.lu) experienced degraded performance:

HTTP code: 200
Response time: 2075 ms

Resolved: Bofrost.lu performance has improved in ff0f60e after 43 minutes.


Log traces by session id instead of agent id.

⚠ Please check that this feature request hasn't been suggested before.

[X] I searched previous Ideas in Discussions didn't find any similar feature requests.
[X] I searched previous Issues didn't find any similar feature requests.

🔖 Feature description
session_id as trace_id is preferred as most interactions in production are performed by non-logged in users.
User interaction across sessions is too much data over time.

Requirements:
trace's id -> session_id
trace's name -> agent name
user id -> logged in user name/id
metadata -> agent id + any callback handler metadata
timestamp -> created/last updated

📝 Additional Context
No response
Acknowledgements

[X] My issue title is concise and descriptive.
[X] I have searched the existing issues to make sure this feature has not been requested yet.
[X] I have provided enough information for the maintainers to understand and evaluate this request.

@kranthi419 I added the fixes, currently the trace name is still set to Assistant due to API not having access to the complete Agent object. Will fix this later on.


Fix contributors page

Closes #164

Already fixed you where just a few seconds late ;)

@BennyCarlsson darn!


Add ResultHandler to Resolver API

Adds a ResultHandler interface to the Resolver API.  This simplifies state management and provides a cleaner way to distinguish between a normal resolver result, a transient failure, and resolver shutdown.

This is ready for review now.  I'd like @apolcyn to be the primary reviewer of this; @AspirinSJL does not need to review but can certainly do so if she's interested.

Known issues: #18189 #17801


Blame Feature for vstest

This feature gets the faulty test case name in event of a test host crash of reason unknown.
The usage is as follows -
"vstest.console.exe" /TestadapterPath:"\vstest\src\datacollector "/Testadapterpath:C:\TestProject\TestProject\bin\Debug\net46  C:\TestProject\TestProject\bin\Debug\net46\TestProject.dll  /collect:Blame

Example :

@vagisha-nidhi,
Thanks for having already signed the Contribution License Agreement. Your agreement was validated by Microsoft. We will now review your pull request.
Thanks,
Microsoft Pull Request Bot

@dotnet-bot test Windows_NT / Debug Build

@dotnet-bot test Windows_NT / Debug Build


couldn't check remote was closed

local data, status, partial = self.tcp:receive("a")
if __status == STATUS_CLOSED or __status == STATUS_NOT_CONNECTED then
--remote was closed.
end
I use code above to check remote socket is closed，but the status is allways timeout because the code at wsocket.c（socket_waitfd）：
if (timeout_iszero(tm)) return IO_TIMEOUT;  / optimize timeout == 0 case */
because my socket is non-blocking，so it allways return timeout here。after delete this line，it's return closed。

Maybe this could happen on the first call. But on the second call, it should return IO_CLOSED, no?

Maybe this could happen on the first call. But on the second call, it should return IO_CLOSED, no?

not fist call,it allways timeout,
until i comment this line（if (timeout_iszero(tm)) return IO_TIMEOUT; / optimize timeout == 0 case */）.

Just saying "Me too". Also experiencing this problem with my program, and while searching found this thread. Looks like sydnash is correct.

/* On UDP, a connreset simply means the previous send failed.
* So we try again.
* On TCP, it means our socket is now useless, so the error passes.
* (We will loop again, exiting because the same error will happen) */
as this  in wsocket.c line 256 try again but may it missing  continue ? prev  still IO_DONE
in  if (err != WSAECONNRESET || prev == WSAECONNRESET) return err;

int socket_recv(p_socket ps, char *data, size_t count, size_t *got,
        p_timeout tm)
{
    int err, prev = IO_DONE;
    *got = 0;
    if (*ps == SOCKET_INVALID) return IO_CLOSED;
    for ( ;; ) {
        int taken = recv(*ps, data, (int) count, 0);
        if (taken > 0) {
            *got = taken;
            return IO_DONE;
        }
        if (taken == 0) return IO_CLOSED;
        err = WSAGetLastError();
        /* On UDP, a connreset simply means the previous send failed.
         * So we try again.
         * On TCP, it means our socket is now useless, so the error passes.
         * (We will loop again, exiting because the same error will happen) */
        if (err != WSAEWOULDBLOCK) {
            if (err != WSAECONNRESET || prev == WSAECONNRESET) return err;
            prev = err;
			//change by  2017-1-7 01:59:38 darklost.me
			//settimeout(0) is always return IO_TIMEOUT when remote server is closed!
			//maybe missing continue here?
			continue;
        }
        if ((err = socket_waitfd(ps, WAITFD_R, tm)) != IO_DONE) return err;
    }
}

in function int socket_recvfrom(p_socket ps, char *data, size_t count, size_t *got, SA *addr, socklen_t *len, p_timeout tm)  is same too
maybe missing continue ?

Does #81 help with figuring out if the remote side closed?

I wonder this change would dead loop doing socket the block check.
On Windows, read() is block even though using MSG_*** parameters. Except to setting the non-block flag.
I think the best way is to Select again instead loop it with continue.


⚠ Secret Website 3 has degraded performance

In 0d5c8a9, Secret Website 3 ($SECRET_WEBSITE_3) experienced degraded performance:

HTTP code: 200
Response time: 3079 ms

Resolved: Secret Website 3 performance has improved in 5f6f856.


Add info on Studio upgrade

Hi @MarkvanMents k! Can you please have a look at info that I added. I am not sure about the section title.

@MariaShaposhnikova ,
As discussed, I have moved Studio to the top, so that the user doesn't have to struggle to find the information.
I still find the concept of a Studio app having a Mendix version very strange. I wonder if we should perhaps talk about compatibility with a version, rather than having a version.
I still think that the Studio team should consider moving to versions which map 1-1 with Studio Pro version - but that is not really an issue for me to resolve.
Hope my changes look OK!


Make failed in linking Boost library

Hello,
I am facing an error in this step:

[ 75%] Linking CXX executable ../release/experiment

All of errors liked that:

undefined reference to `boost::program_options:

I install latest libraries version and checked that libboost 1.58 is compatible with g++ 4.9. I think maybe it related with C++11, however It returns error in both g++ 4.9 and 4.7.
This is my system information:

-- The C compiler identification is GNU 4.9.3
-- The CXX compiler identification is GNU 4.9.3
-- Check for working C compiler: /usr/bin/cc
-- Check for working C compiler: /usr/bin/cc -- works
-- Detecting C compiler ABI info
-- Detecting C compiler ABI info - done
-- Detecting C compile features
-- Detecting C compile features - done
-- Check for working CXX compiler: /usr/bin/c++
-- Check for working CXX compiler: /usr/bin/c++ -- works
-- Detecting CXX compiler ABI info
-- Detecting CXX compiler ABI info - done
-- Detecting CXX compile features
-- Detecting CXX compile features - done
-- Build type: Release
-- GSL using gsl-config /usr/bin/gsl-config
-- Using GSL from /usr
-- Found GSL.
-- Found Eigen3: /usr/include/eigen3 (Required is at least version "3")
-- Found Eigen3.
-- Boost version: 1.58.0
-- Found the following Boost libraries:
--   system
--   filesystem
--   program_options
-- Found BOOST.

I also install Clang and LLDB 3.6. I tried search many possible solution but can not fix that :(.

Hi,
sorry about that. Did you try to remove CMake cache or build from clean
clone?
If all else fail you can try development branch. We are getting rid of
Boost to eliminate situations like yours.
On Oct 16, 2016 3:23 PM, "Linh Nguyen"<EMAIL_ADDRESS>wrote:

Hello,
I am facing an error in this step:
[ 75%] Linking CXX executable ../release/experiment
All of errors liked that:
undefined reference to `boost::program_options:
I install latest libraries version and checked that libboost 1.58 is
compatible with g++ 4.9. I think maybe it related with C++11, however It
returns error in both g++ 4.9 and 4.7.
This is my system information:
-- The C compiler identification is GNU 4.9.3
-- The CXX compiler identification is GNU 4.9.3
-- Check for working C compiler: /usr/bin/cc
-- Check for working C compiler: /usr/bin/cc -- works
-- Detecting C compiler ABI info
-- Detecting C compiler ABI info - done
-- Detecting C compile features
-- Detecting C compile features - done
-- Check for working CXX compiler: /usr/bin/c++
-- Check for working CXX compiler: /usr/bin/c++ -- works
-- Detecting CXX compiler ABI info
-- Detecting CXX compiler ABI info - done
-- Detecting CXX compile features
-- Detecting CXX compile features - done
-- Build type: Release
-- GSL using gsl-config /usr/bin/gsl-config
-- Using GSL from /usr
-- Found GSL.
-- Found Eigen3: /usr/include/eigen3 (Required is at least version "3")
-- Found Eigen3.
-- Boost version: 1.58.0
-- Found the following Boost libraries:
-- system
-- filesystem
-- program_options
-- Found BOOST.
I also install Clang and LLDB 3.6. I tried search many possible solution
but can not fix that :(.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/searchivarius/nmslib/issues/134, or mute the thread
https://github.com/notifications/unsubscribe-auth/AAyZMstTI6PTw32mZsWSP3ngstOTiKH3ks5q0nnIgaJpZM4KYDlK
.

Hello,
I did that before and change version of g++ and gcc to hope it successful.

rm -rf find . -name CMakeFiles CMakeCache.txt

But still same errors, I will try the dev branch definitely.
Thanks you,

Interesting,
What kind of OS do you use?

Leo
On Mon, Oct 17, 2016 at 2:28 AM, Linh Nguyen<EMAIL_ADDRESS>wrote:

Hello,
I did that before and change version of g++ and gcc to hope it successful.
rm -rf find . -name CMakeFiles CMakeCache.txt
But still same errors, I will try the dev branch definitely.
Thanks you,
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/searchivarius/nmslib/issues/134#issuecomment-254122638,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAyZMsn35pIfjwIl-pWjO8ysmbnPwLDmks5q0xWLgaJpZM4KYDlK
.

Hello,
I am using Ubuntu 16.04

Try to install boost program options separately:
sudo apt-get install libboost-dev libboost-test-dev libboost-program-options-dev libboost-system-dev libboost-filesystem-dev

Hello,
I installed it before. I just try to clean and remake the master branch but still fail.
Here is the libboost installation in my system.

find /usr/lib -name 'libboost*'
/usr/lib/x86_64-linux-gnu/libboost_program_options.a
/usr/lib/x86_64-linux-gnu/libboost_math_tr1l.so.1.58.0
/usr/lib/x86_64-linux-gnu/libboost_prg_exec_monitor.so.1.58.0
/usr/lib/x86_64-linux-gnu/libboost_timer.a
/usr/lib/x86_64-linux-gnu/libboost_math_c99.so
/usr/lib/x86_64-linux-gnu/libboost_mpi_python-py27.so.1.58.0
/usr/lib/x86_64-linux-gnu/libboost_mpi_python-py35.a
/usr/lib/x86_64-linux-gnu/libboost_coroutine.a
/usr/lib/x86_64-linux-gnu/libboost_serialization.a
/usr/lib/x86_64-linux-gnu/libboost_signals.a
/usr/lib/x86_64-linux-gnu/libboost_chrono.so.1.58.0
/usr/lib/x86_64-linux-gnu/libboost_math_tr1.so
/usr/lib/x86_64-linux-gnu/libboost_locale.so
/usr/lib/x86_64-linux-gnu/libboost_program_options.so.1.58.0
/usr/lib/x86_64-linux-gnu/libboost_math_tr1.so.1.58.0
/usr/lib/x86_64-linux-gnu/libboost_mpi.a
/usr/lib/x86_64-linux-gnu/libboost_prg_exec_monitor.a
/usr/lib/x86_64-linux-gnu/libboost_wave.so.1.58.0
/usr/lib/x86_64-linux-gnu/libboost_serialization.so
/usr/lib/x86_64-linux-gnu/libboost_mpi_python-py27.so
/usr/lib/x86_64-linux-gnu/libboost_log_setup.a
/usr/lib/x86_64-linux-gnu/libboost_signals.so.1.58.0
/usr/lib/x86_64-linux-gnu/libboost_graph_parallel.so.1.58.0
/usr/lib/x86_64-linux-gnu/libboost_math_c99l.so
/usr/lib/x86_64-linux-gnu/libboost_wserialization.so
/usr/lib/x86_64-linux-gnu/libboost_coroutine.so.1.58.0
/usr/lib/x86_64-linux-gnu/libboost_context.so
/usr/lib/x86_64-linux-gnu/libboost_exception.a
/usr/lib/x86_64-linux-gnu/libboost_mpi_python-py35.so.1.58.0
/usr/lib/x86_64-linux-gnu/libboost_graph_parallel.a
/usr/lib/x86_64-linux-gnu/libboost_locale.so.1.58.0
/usr/lib/x86_64-linux-gnu/libboost_date_time.a
/usr/lib/x86_64-linux-gnu/libboost_mpi.so.1.58.0
/usr/lib/x86_64-linux-gnu/libboost_wserialization.a
/usr/lib/x86_64-linux-gnu/libboost_math_c99.a
/usr/lib/x86_64-linux-gnu/libboost_regex.so.1.58.0
/usr/lib/x86_64-linux-gnu/libboost_math_c99f.so.1.58.0
/usr/lib/x86_64-linux-gnu/libboost_math_tr1l.so
/usr/lib/x86_64-linux-gnu/libboost_atomic.so
/usr/lib/x86_64-linux-gnu/libboost_python-py35.so
/usr/lib/x86_64-linux-gnu/libboost_thread.a
/usr/lib/x86_64-linux-gnu/libboost_unit_test_framework.a
/usr/lib/x86_64-linux-gnu/libboost_system.so
/usr/lib/x86_64-linux-gnu/libboost_system.a
/usr/lib/x86_64-linux-gnu/libboost_python-py27.so.1.58.0
/usr/lib/x86_64-linux-gnu/libboost_regex.a
/usr/lib/x86_64-linux-gnu/libboost_math_c99l.so.1.58.0
/usr/lib/x86_64-linux-gnu/libboost_filesystem.a
/usr/lib/x86_64-linux-gnu/libboost_mpi_python-py35.so
/usr/lib/x86_64-linux-gnu/libboost_mpi.so
/usr/lib/x86_64-linux-gnu/libboost_system.so.1.58.0
/usr/lib/x86_64-linux-gnu/libboost_thread.so
/usr/lib/x86_64-linux-gnu/libboost_date_time.so.1.58.0
/usr/lib/x86_64-linux-gnu/libboost_python-py35.so.1.58.0
/usr/lib/x86_64-linux-gnu/libboost_math_c99f.a
/usr/lib/x86_64-linux-gnu/libboost_chrono.a
/usr/lib/x86_64-linux-gnu/libboost_context.a
/usr/lib/x86_64-linux-gnu/libboost_log.so.1.58.0
/usr/lib/x86_64-linux-gnu/libboost_python.so
/usr/lib/x86_64-linux-gnu/libboost_regex.so
/usr/lib/x86_64-linux-gnu/libboost_python-py35.a
/usr/lib/x86_64-linux-gnu/libboost_unit_test_framework.so.1.58.0
/usr/lib/x86_64-linux-gnu/libboost_serialization.so.1.58.0
/usr/lib/x86_64-linux-gnu/libboost_test_exec_monitor.a
/usr/lib/x86_64-linux-gnu/libboost_iostreams.so
/usr/lib/x86_64-linux-gnu/libboost_filesystem.so.1.58.0
/usr/lib/x86_64-linux-gnu/libboost_mpi_python-py27.a
/usr/lib/x86_64-linux-gnu/libboost_math_c99f.so
/usr/lib/x86_64-linux-gnu/libboost_math_tr1.a
/usr/lib/x86_64-linux-gnu/libboost_wave.so
/usr/lib/x86_64-linux-gnu/libboost_chrono.so
/usr/lib/x86_64-linux-gnu/libboost_log_setup.so
/usr/lib/x86_64-linux-gnu/libboost_program_options.so
/usr/lib/x86_64-linux-gnu/libboost_mpi_python.a
/usr/lib/x86_64-linux-gnu/libboost_math_tr1f.so
/usr/lib/x86_64-linux-gnu/libboost_thread.so.1.58.0
/usr/lib/x86_64-linux-gnu/libboost_python-py27.a
/usr/lib/x86_64-linux-gnu/libboost_log_setup.so.1.58.0
/usr/lib/x86_64-linux-gnu/libboost_graph.so.1.58.0
/usr/lib/x86_64-linux-gnu/libboost_context.so.1.58.0
/usr/lib/x86_64-linux-gnu/libboost_date_time.so
/usr/lib/x86_64-linux-gnu/libboost_unit_test_framework.so
/usr/lib/x86_64-linux-gnu/libboost_iostreams.so.1.58.0
/usr/lib/x86_64-linux-gnu/libboost_math_c99l.a
/usr/lib/x86_64-linux-gnu/libboost_iostreams.a
/usr/lib/x86_64-linux-gnu/libboost_coroutine.so
/usr/lib/x86_64-linux-gnu/libboost_atomic.a
/usr/lib/x86_64-linux-gnu/libboost_log.a
/usr/lib/x86_64-linux-gnu/libboost_wave.a
/usr/lib/x86_64-linux-gnu/libboost_timer.so
/usr/lib/x86_64-linux-gnu/libboost_wserialization.so.1.58.0
/usr/lib/x86_64-linux-gnu/libboost_math_tr1f.a
/usr/lib/x86_64-linux-gnu/libboost_math_tr1l.a
/usr/lib/x86_64-linux-gnu/libboost_python-py27.so
/usr/lib/x86_64-linux-gnu/libboost_signals.so
/usr/lib/x86_64-linux-gnu/libboost_random.so
/usr/lib/x86_64-linux-gnu/libboost_atomic.so.1.58.0
/usr/lib/x86_64-linux-gnu/libboost_prg_exec_monitor.so
/usr/lib/x86_64-linux-gnu/libboost_graph.so
/usr/lib/x86_64-linux-gnu/libboost_filesystem.so
/usr/lib/x86_64-linux-gnu/libboost_timer.so.1.58.0
/usr/lib/x86_64-linux-gnu/libboost_graph_parallel.so
/usr/lib/x86_64-linux-gnu/libboost_log.so
/usr/lib/x86_64-linux-gnu/libboost_math_c99.so.1.58.0
/usr/lib/x86_64-linux-gnu/libboost_math_tr1f.so.1.58.0
/usr/lib/x86_64-linux-gnu/libboost_graph.a
/usr/lib/x86_64-linux-gnu/libboost_random.so.1.58.0
/usr/lib/x86_64-linux-gnu/libboost_locale.a
/usr/lib/x86_64-linux-gnu/libboost_mpi_python.so
/usr/lib/x86_64-linux-gnu/libboost_python.a
/usr/lib/x86_64-linux-gnu/libboost_random.a
/usr/lib/i386-linux-gnu/libboost_system.so.1.58.0
/usr/lib/i386-linux-gnu/libboost_filesystem.so.1.58.0

I found that all of the error are came from

undefined reference to `boost::program_options::error

I search that article is very helpful, but I can not modify anything:
http://stackoverflow.com/questions/8454329/why-cant-clang-with-libc-in-c0x-mode-link-this-boostprogram-options-examp

This makes sense, thank you. One more stimuls to get rid of boost as soon as possible :-)
However, I am a bit confused. Above you write that you use GCC not CLANG. With CLANG you can't use HNSW (it's not working with CLANG yet).

Actually I used GCC, I installed Clang but not use. I also redefine the environment as

export CCX=g++-4.7 CC=gcc-4.7

So what I can do now ? What specific version of your Gcc and libboost that are successful in install Nmslib ?
I am managing to use this library to add some binary distances for binary data.

I just noticed that your find command to delete cache doesn't seem to be correct:
find . -name CMakeFiles CMakeCache.txt

The issue that you linked above is only for Clang. It works with boost and Ubuntu 16 for me with g++-4.8.
So, may be your cmake version is much different, but I doubt. I use
cmake version 3.5.1

PS: I suspect that you didn't delete CMake cache.

Hello,
I remove Clang and CMake cache, also make clean, my cmake version is 3.5.1. Of course, it rebuild from the beginning and same errors:
 76%] Linking CXX executable ../release/experiment
../release/libNonMetricSpaceLib.a(params_cmdline.cc.o): In function `similarity::ParseCommandLine(int, char**, bool&, std::string&, std::string&, std::string&, std::string&, std::string&, std::shared_ptr<similarity::AnyParams>&, unsigned int&, bool&, std::string&, unsigned int&, std::string&, std::string&, std::string&, float&, unsigned int&, unsigned int&, std::vector<unsigned int, std::allocator<unsigned int> >&, float&, std::string&, std::string&, std::shared_ptr<similarity::AnyParams>&, std::vector<std::shared_ptr<similarity::AnyParams>, std::allocator<std::shared_ptr<similarity::AnyParams> > >&)':
params_cmdline.cc:(.text+0x3ad): undefined reference to `boost::program_options::options_description::options_description(std::string const&, unsigned int, unsigned int)'

Are you sure? What's the exact cleaning command line. The command that you cited above doesn't work. To reiterate, this one is wrong:
rm -rf find . -name CMakeFiles CMakeCache.txt

Hello,
I removed all Cmake related files by hand in the last comment. This time I delete the nmslib folder and extract the archive file again, but the same error.
Thanks you for notification, the cmake cleaning command I copied in the manual does not work; I also recheck this, the find command syntax is not right.
I am sorry that maybe cost your time, but I honestly try many ways until now to get it work.

Hi, ok, thank you for confirming.
Regarding the command in the manual: you seem to have lost two '`'.
I will try to reproduce this bug in a virtual machine (Ubuntu 16). Can't reproduce on my own machine which also uses Ubuntu 16, but this is not a clean install.
In any case, please, try the dev bracnh. Relatively soon, we will remove boost from the main build, so it won't be a pain (it is also a pain on Windows).

Thanks you,
Actually, my Ubuntu system is not so clean too, but related things are in the right place. Hope you can find the reason. I am trying the dev branch now.

It seems like that the linker can't find boost libraries (i.e. program options lib). So could you compile the code again:
cmake .
make VERBOSE=1
Then you should see -lboost_program_options-mt (or without -mt) in the linker command being run by cmake.

Hello,
Thanks you for the suggestion. Here is the linker command for boost:
-lboost_filesystem -lboost_program_options -L/usr/lib/x86_64-linux-gnu -lgsl -lgslcblas -lm -lpthread

OK.
Could you check the following code if it works for you:
[bileg@similarity_search]$ cat popt.cc
#include
#include <boost/program_options.hpp>
namespace po = boost::program_options;
int main() {
po::options_description cmd_opt("test");
std::cout << "ok\n";
}
[bileg@similarity_search]$ g++ popt.cc
/tmp/cclO7B78.o: In function main': popt.cc:(.text+0xd): undefined reference to boost::program_options::options_description::m_default_line_length'
popt.cc:(.text+0x18): undefined reference to boost::program_options::options_description::m_default_line_length' popt.cc:(.text+0x51): undefined reference to boost::program_options::options_description::options_description(std::string const&, unsigned int, unsigned int)'
collect2: error: ld returned 1 exit status
[bileg@similarity_search]$ g++ popt.cc -L/usr/local/lib -lboost_system-mt -lboost_program_options-mt              # (i think you can use:   g++ popt.cc -lboost_system -lboost_program_options -L/usr/lib/x86_64-linux-gnu)
[bileg@similarity_search]$ ./a.out
ok

I can actually reproduce this bug for the clean Ubuntu 16 install. I will try to @bileg's code

I try to run @bileg code, the same problem in linking stage:
g++ popt.cc -lboost_system -lboost_program_options -L/usr/lib/x86_64-linux-gnu
/tmp/ccXNGnl7.o: In function `main':
popt.cc:(.text+0x6c): undefined reference to `boost::program_options::options_description::options_description(std::string const&, unsigned int, unsigned int)'
collect2: error: ld returned 1 exit status

OK. It means that /usr/lib/x86_64-linux-gnu doesn't contain required boost libraries. So you can do:
find /usr -name boost.so
If you find a directory that contains libboost_program_options.so then you need to add -L/usr/.... (the directory you just found)
If it doesn't work you could try another option:
https://github.com/searchivarius/nmslib/blob/master/similarity_search/CMakeLists.txt#L122
But you have to download and compile boost and then link.

somehow asterisk disappeared from the command it should be find /usr -name *boost*.so

Hello,
I checked that library's location before in this comment:
https://github.com/searchivarius/nmslib/issues/134#issuecomment-254496558
Is it ok ?

Sorry, I overlooked that then everything should be fine. Could you try the second approach that I suggested (CMakeLists.txt#L122)

Thanks you,
Wait me a little bit, I will download and make a hardwired link to boost then.

Thanks @bileg and @searchivarius , after compiling and linking in CMakeLists.txt#L122. The library was built successfully.


[REPORT] Plan9 icon outdated

The issue
The Plan9 icon uses the old Glenda SVG I made, but I made a better one recently: https://gist.github.com/ThatOneCalculator/a71d17c38cd1a96697ed6b0e782a5705

Fixed in 79446ce2ce5493d0dfb90d95314f79707155df96, thank you!


use GolangCI-Lint instead of gometalinter

Codecov Report

Merging #1798 into master will increase coverage by 0.06%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #1798      +/-   ##
==========================================
+ Coverage   86.99%   87.05%   +0.06%
==========================================
  Files          95       95
  Lines        6443     6441       -2
==========================================
+ Hits         5605     5607       +2
+ Misses        599      595       -4
  Partials      239      239

Impacted Files
Coverage Δ

send_stream.go
93.33% <ø> (+1.03%)
:arrow_up:

session.go
75.99% <0%> (+0.27%)
:arrow_up:

Continue to review full report at Codecov.

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


postgres: enable statement statistics gathering

Partial fix for #500.

@matthew-white sorry this (tiny) PR was such a mess - I've fixed the issues you mentioned and updated the PR

This will also need shared_preload_libraries to include pg_stat_statements if it does not already.
This needs to be confirmed in central's docker/docker-compose setup; if it's not already set then this setting will face a similar challenge to those outlined in https://github.com/getodk/central-backend/issues/500#issuecomment-1168656710 relating to setting postgres conf for log_min_duration_statement.

This needs to be confirmed in central's docker/docker-compose setup

You have access to a Central server now, right? Could you check things out there? In case you end up needing to run this migration and roll it back, I also wanted to point you to lib/model/knexfile.js, which might help with that.


Fix event count and snapshot records being overwritten unexpectedly

This PR addresses issues #90 and #95.
Previously, the IDs of event count records and Aggregate snapshot records could be unexpectedly overwritten because their IDs weren't specific enough.
This PR changes the ID format as follows:

For the event records, the Aggregate state type is appended to the ID, for example,
EVENTS_AFTER_SNAPSHOT_spine.test.aggregate.Project_{"id":"randomId"}.
For the snapshots, the Timestamp of the snapshot is appended to the ID, for example:
SNAPSHOT_{"id":"randomId"}_2018-11-22T18:14:16.267Z

Also, the lookup by the old-format IDs was implemented to ensure compatibility with the previously saved data.
This PR also migrates the gcloud-java tests to always executing on the local datastore emulator (even on CI). There is only one acceptance test left for the remote datastore to check if everything is set up correctly when building on CI.

@dmdashenkov PTAL.

@dmdashenkov PTAL.


Default to Single Instance Mode

As a runtime admin, I want to be able to first run ZITADEL without the concept of multiple virtual instances so that I can easily start for simple use cases and configure my environment for more complex scenarios later.
### Acceptance Criteria
- [ ] All calls to ZITADEL map to FirstInstance created with `zitadel setup` if the host header doesn't match an existing instance.

Opinions @fforootd @livio-a @mffap?

I agree we should implement this as default. I.e map all traffic through to the first instance.
The one thing that needs discussion is the primary domain though, because it is needed to send mails and notifications.

I agree we should implement this as default. I.e map all traffic through to the first instance.
The one thing that needs discussion is the primary domain though, because it is needed to send mails and notifications.

Thanks for the hint. I added an action item as discussed.

@hifabienne this one is ready for refinement

I think @livio-a had a blocking concern about the token issuer?

If we don't find the instance, we could then try to find it with the cut off subdomain? If we find it, we search an org by this subdomain and add the org ID to the context?
This would align ZITADELs behavior to the obviously much expected behavior and resolve much confusion about the org subdomains.
Could that work @livio-a?
I guess we'd again have to pass the original domain though in go context and event bodies, so we can use it as an issuer, right?

This would align ZITADELs behavior to the obviously much expected behavior of listening on org subdomains and resolve much confusion.

This would be really helpful!

It turns out that introducing a single instance mode would basically remove the external domain config. As we still rely on the external domain during instance setup, removing the external domain and introducing a new mode seems like too much overhead.
Instead, we can address the instance not found errors differently:

Using forwarded, x-forwarded-* and x-zitadel-http1-host headers for the instance queries
Improve docs about how to switch or add a domain (step-by-step guides, listing common scenarios etc)
Improve UX for switching and adding a domain (like for example adding CLI commands that don't need authentication)

The following table shows which measures could have prevented the last few instance not found reports on discord.
| Read proxy headers | Domain migration docs/ux | Domain adding docs/ux | Single instance mode | Already solved
-- | -- | -- | -- | -- | --
https://discord.com/channels/927474939156643850/1167741933800673290/1168163081675882526 | x |   |   | x |
https://discord.com/channels/927474939156643850/927866013545025566/1166819682368626808 |   |   | x |   |
https://discord.com/channels/927474939156643850/1166399185021567106/1166399185021567106 |   | x | x | x |
https://discord.com/channels/927474939156643850/1166397936209494079/1166397936209494079 |   |   |   |   | x
https://discord.com/channels/927474939156643850/1165411331596370081/1166386616756097084 |   |   |   |   | x
https://discord.com/channels/927474939156643850/927866013545025566/1165914460205092904 |   |   | x | x |
https://discord.com/channels/927474939156643850/1165772509405253732 | x |   |   | x |
https://discord.com/channels/927474939156643850/927866013545025566/1164082196919885854 |   | x |   |   |
https://discord.com/channels/927474939156643850/1074772204874633296/1155528889720901675 | x |   |   | x |
https://discord.com/channels/927474939156643850/1074772204874633296/1074772204874633296 | x |   |   | x |
https://discord.com/channels/927474939156643850/927866013545025566/1153739424660721684 | x |   |   | x |
https://discord.com/channels/927474939156643850/1133698741795307571 |   |   | x |   |
https://discord.com/channels/927474939156643850/1132156683829911552/1133037543391240316 |   | x |   | x |
https://discord.com/channels/927474939156643850/1122953631029788713/1124280956946223104 |   |   | x | x |
https://discord.com/channels/927474939156643850/1121795198784131165/1121808441715998780 |   | x |   | x |
https://discord.com/channels/927474939156643850/1121387758250901657/1121387758250901657 | x |   |   | x |
@livio-a I think this observation matches our discussions


Changelog update & Danger config update

We forgot to add a changelog entry for #483, this change now fails a build if library source code has changed without a changelog entry. This means that refactors that don't change user functionality will need to be included, too. I'm okay with that, refactors might introduce bugs and changelog entries could really help someone.
@colinta Any thoughts on this?
Fixes #546.

Good point - we'll probably need to be diligent about telling ourselves and others about that (e.g. if it's a refactor we should point out "include an entry that summarizes the changes, even if they aren't breaking).

An expanded message is a good idea 👍

  Generated by :no_entry_sign: danger

Empty Moya message is a known issue: https://github.com/ashfurrow/danger-swiftlint/issues/10
@colinta does that message look good?

👍 I haven't gotten around to looking at the documentation, to see where a description of uploading can go.  That's enough of a mention that people can see that it's supported.

Yep


CocoaPods compatible version error

When I tried to install TRON 4.0 via CocoaPods, terminal shows these errors:
CocoaPods could not find compatible versions for pod "TRON":
  In Podfile:
    TRON (~> 4.0)

None of your spec sources contain a spec satisfying the dependency: `TRON (~> 4.0)`.

You have either:
 * out-of-date source repos which you can update with `pod repo update` or with `pod install --repo-update`.
 * mistyped the name or version.
 * not added the source repo that hosts the Podspec to your Podfile.

I did upgraded my project from Swift 3.2 to 4.0.
Right now using xcode 9.1 and Swift 4.0.
I already tried the below methods
pod repo remove master
pod setup
pod install

pod repo update

Sorry, README is currently misleading, 4.0.0 release has not yet been cut, you can use 4.0.0-beta.2 release for now. I'm planning to make release soon, will update this issue when it will be done.

4.0.0 release has been made - https://github.com/MLSDev/TRON/releases/tag/4.0.0
Closing.


Specify number of lines that each header item should occupy

I would appreciate if it would be possible to switch off the line wrapping of the header items in the article dialog, which seems to be "hard-coded" currently. I tried to work around this with terminal settings, but had no success.
Instead of just switching on/off the wrapping, I would suggest to introduce options which let the user choose a fixed number of lines each header item should occupy. This would mean that a long item which lenght would result in exceeding the given number of lines would be cut, and a short item which lenght would result in using less then the number of given lines would be padded with "space" to use the given number.
In practice, this would concern in particular the "title", "link" and "enclosure" item, but would generally apply to the other items as well.
Ideally, there would be commands for the config which set the number of lines each header item should occupy initially, like described above, where "0" (zero) would mean line wrapping as it takes place currently; and an operation to toggle between the two states (fixed number of header lines and wrapping as it takes place currently).
This would have two effects I am aiming to:

The "layout" would always stay the same, regardless of whether long titles / links or short titles / links, in particular.

The number of lines the header as a whole would consume could be limited.

There are these reasons why I make this request:

When browsing through the new articles in the article dialog (in all about 450 each day in my case), for me it would be much easier to grasp the articles with a constant "layout" while browsing.

I would like to use newsboat in termux on my smartphone, where the number of available screen lines is noticeably decreased by the on-screen-keyboard, which of course is necessary to operate newsboat.

Hi! This sounds like a continuation of #687, and is generally related to #68. I still believe in the idea I described in https://github.com/newsboat/newsboat/issues/687#issuecomment-545577559: provide something like html-renderer, but for the article header. Newsboat should print out a JSON document with article title, author and other fields; the program will process it and print out a JSON array of lines, marked with their "wrapping kind" (we currently have "nonwrappable", "soft-wrappable", "wrappable"). Do you have any downsides to this approach, @mglh?

provide something like html-renderer, but for the article header

I hadn't thought of that solution =:) It would be nice to have it.
There would be the downside that every time an article is shown an external program or script would have to be started, which would perhaps be noticeable as a delay, at least on a smartphone, I assume. But it would do the job.
And I assume that changing the code in that way would require less work for you than the suggestion I originally made = :)
Would an even simpler data format than JSON be conceivable?

I assume that changing the code in that way would require less work for you than the suggestion I originally made.

I'd say it's about the same either way. The reason I'm proposing to move this into an external script is because I don't want to commit to a particular solution just yet. You already proposed two, here and in #687, and they both don't sound like something that would cover all the use-cases. I don't have any ideas at all. Yet the problem clearly exists, and I don't consider it out of scope. So I want to move this feature out of Newsboat to enable experimentation. If someone ends up creating something that's pretty universal, we can incorporate that into Newsboat and spare the users some inconvenience.
Re performance hit: yeah, I agree. We could make this into a plugin rather than a script, but then the user would have to compile a shared library for their own platform. That's not exactly the "experimentation enablement" that I had in mind :)

Would an even simpler data format than JSON be conceivable?

Initially I thought it should be "one field per line as input, a stream of lines as output", but there are some downsides to that:

what if a field contains a newline? (Fields could be separated by NUL instead of newline, but that makes the parsing harder)
it'd be hard to make sense of the input without reading the docs; it's not "self-describing" (Adding labels will again make the parsing harder)
the output has to contain nonwrappable/softwrappable/wrappable markings. We could prepend them as numbers in front of output strings, but that suffers from the same problems: not self-describing, harder to write and read.

Then I started thinking if I could use some existing format that's well-supported across different languages, and JSON seem to be the answer.
Unfortunately, JSON is not supported by shells. Even with jq, processing would be awkward, and producing JSON would still be fully manual.
It seems the decision is between "a custom format that's awkward for everyone" and "JSON that's easy for everyone but shells".
What is your view on this? What kind of problems do you foresee with JSON? Do you have specific alternatives in mind?

It would be nice to have it.

Just to be clear: I'm not committing to implementing it any time soon. I'm mostly busy with refactoring and sometimes with bugs; enhancements are on the back-burner for me. But I will of course work with anyone who wants to push this forward.

the output has to contain nonwrappable/softwrappable/wrappable markings

I have second thoughts about exposing these, by the way. The wrapping/soft-wrapping distinction is dictates by our use of ncurses, and it's not set in stone; it might change if we move to a different underlying library. So perhaps we should:

tell the script the value of text-width setting and the width of the terminal;
require lines to be marked as either wrappable or not. Wrappable ones will be wrapped on the screen boundary, non-wrappable will get cut off at the screen boundary.

But that's up for the debate as well.

move this [user-modifiable header items in article dialog] into an external script

To me this seems be the most appropriate. Like for html-renderer, I assume most users are fully satisfied with the "out of the box" experience. The 1% of them which need or want something other, would be able to achive it with a script, no matter what it may be. No need to offer more then that.

What is your view on this? What kind of problems do you foresee with JSON? Do you have specific alternatives in mind?

I did not use JSON til now. Even modifing XML I do with sed and awk, as it works for my use cases.
To me just suspecting that a newline might occur requires something that can handle it. Having this in mind, as much (as little) as I know about JSON, it would be (the only?) appropriate.
While reading about JSON, as a tool that speaks JSON, miller came  to my view.
From its homepage:

Miller is like awk, sed, cut, join, and sort for name-indexed data such as CSV, TSV, and tabular JSON.

At first glance, it looks mature and in active delvelopment. No other dependencies then libc, as it seems. If I would have to work with JSON, it would be my first try.

Okay! I guess this issue is now waiting for someone to roll up their sleeves and actually work on an implementation.


Implement gRPC service for envoy ExtAuthz

What this PR does / why we need it:
This PR implements envoy external authorization service with gRPC, as RFC.
After merging this PR, I'll send another PR to add routing to the envoy configuration.
Which issue(s) this PR fixes:
Part of #4977
Does this PR introduce a user-facing change?: No

How are users affected by this change:
Is this breaking change:
How to migrate (if breaking change):

Overall LGTM, just left a nits about licensing

@Warashi DCO not pass 👀

@khanhtc1202
Sorry.
I signed off with rebase and force-pushed.


Generation fails when using multi-file and generate-helper-types

OS: Manjaro (Arch)
Library: Libbluray
Command:
ClangSharpPInvokeGenerator -f "/usr/include/libbluray/bluray.h" -f "/usr/include/libbluray/meta_data.h" -F "/usr/include/libbluray"  -n Libbluray -o ~/libbluray -c generate-helper-types multi-file

Unhandled exception: System.ArgumentException: Stream was not writable.
   at System.IO.StreamWriter..ctor(Stream stream, Encoding encoding, Int32 bufferSize, Boolean leaveOpen)
   at ClangSharp.PInvokeGenerator.<Close>g__GenerateNativeInheritanceAttribute|56_0(PInvokeGenerator generator, Stream stream, Boolean leaveStreamOpen) in /_/sources/ClangSharp.PInvokeGenerator/PInvokeGenerator.cs:line 367
   at ClangSharp.PInvokeGenerator.Close() in /_/sources/ClangSharp.PInvokeGenerator/PInvokeGenerator.cs:line 311
   at ClangSharp.PInvokeGenerator.Dispose(Boolean isDisposing) in /_/sources/ClangSharp.PInvokeGenerator/PInvokeGenerator.cs:line 1789
   at ClangSharp.Program.Run(InvocationContext context) in /_/sources/ClangSharpPInvokeGenerator/Program.cs:line 756
   at System.CommandLine.Handler.<>c__DisplayClass1_0`1.<SetHandler>b__0(InvocationContext context)
   at System.CommandLine.Invocation.AnonymousCommandHandler.<>c__DisplayClass2_0.<.ctor>g__Handle|0(InvocationContext context)
   at System.CommandLine.Invocation.AnonymousCommandHandler.InvokeAsync(InvocationContext context)
   at System.CommandLine.Invocation.InvocationPipeline.<>c__DisplayClass4_0.<<BuildInvocationChain>b__0>d.MoveNext()
--- End of stack trace from previous location ---
   at System.CommandLine.Builder.CommandLineBuilderExtensions.<>c__DisplayClass18_0.<<UseParseErrorReporting>b__0>d.MoveNext()
--- End of stack trace from previous location ---
   at System.CommandLine.Builder.CommandLineBuilderExtensions.<>c__DisplayClass13_0.<<UseHelp>b__0>d.MoveNext()
--- End of stack trace from previous location ---
   at System.CommandLine.Builder.CommandLineBuilderExtensions.<>c__DisplayClass20_0.<<UseTypoCorrections>b__0>d.MoveNext()
--- End of stack trace from previous location ---
   at System.CommandLine.Builder.CommandLineBuilderExtensions.<>c.<<UseSuggestDirective>b__19_0>d.MoveNext()
--- End of stack trace from previous location ---
   at System.CommandLine.Builder.CommandLineBuilderExtensions.<>c__DisplayClass17_0.<<UseParseDirective>b__0>d.MoveNext()
--- End of stack trace from previous location ---
   at System.CommandLine.Builder.CommandLineBuilderExtensions.<>c.<<RegisterWithDotnetSuggest>b__6_0>d.MoveNext()
--- End of stack trace from previous location ---
   at System.CommandLine.Builder.CommandLineBuilderExtensions.<>c__DisplayClass9_0.<<UseExceptionHandler>b__0>d.MoveNext()

If I do not use multi-file it works but then the C# compiler complains because it inserts the helper declarations somewhere in the middle.

This has already been fixed locally, just pending publishing the new package version. That should happen here shortly


2 second  timeout for requests is too agressive

I ran into a timeout today when making an API request.  I see the 2-second timeout and think it is too aggressive.
Can we add a request_timeout class variable in the client and give it a longer timeout? Maybe 10 seconds?
I can create a pull request if you would like.
This is excellent work, Tyler. Thank you.

In pypi 2.0.2:
*default request timeout changed to 5 sec
*custom timeout can be set at client initialization, i.e. client = schwabdev.Client("app key", "app secret", timeout=10)


Dependencies Wrongly mentioned

The dependencies version got upgraded but in package.json older version is mentioned for babel-eslint
npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none is installed.
You must install peer dependencies yourself.
Current version is<EMAIL_ADDRESS>Is this a bug report?
Yes

Did you try recovering your dependencies?

(Write your answer here.)
Which terms did you search for in User Guide?

(Write your answer here if relevant.)
Environment

(paste the output of the command here)
Steps to Reproduce

(Write your steps here:)

Expected Behavior

(Write what you thought would happen.)
Actual Behavior

(Write what happened. Please add screenshots!)
Reproducible Demo

(Paste the link to an example project and exact instructions to reproduce the issue.)

What do you mean by "Current version is<EMAIL_ADDRESS>The peer dependency asks you to use 7.x.

I think he's just pointing out, that other dependencies were upgraded, and that babel-eslint is behind


Update urls of ‘Thinking in C++’

The old links aren’t valid anymore

That's super that these are now on Archive, thanks! Worth looking to see if we have the other titles at https://archive.org/details/@bruceeckel


db: Made overlap String() prettier and more readable.

Signed-off-by: Bartek Plotka<EMAIL_ADDRESS>

Example output now:
[70-80 <0s>](2):<PHONE_NUMBER>0000000000000000 61-85 <0s>;<PHONE_NUMBER>0000000000000000 70-80 <0s>
[94-99 <0s>](3):<PHONE_NUMBER>0000000000000000 90-100 <0s>;<PHONE_NUMBER>0000000000000000 92-105 <0s>;<PHONE_NUMBER>0000000000000000 94-99 <0s>
[30-31 <0s>](2):<PHONE_NUMBER>0000000000000000 21-31 <0s>;<PHONE_NUMBER>0000000000000000 30-40 <0s>
[34-36 <0s>](3):<PHONE_NUMBER>0000000000000000 30-40 <0s>;<PHONE_NUMBER>0000000000000000 33-39 <0s>;<PHONE_NUMBER>0000000000000000 34-36 <0s>
[61-70 <0s>](2):<PHONE_NUMBER>0000000000000000 60-70 <0s>;<PHONE_NUMBER>0000000000000000 61-85 <0s>
[80-85 <0s>](2):<PHONE_NUMBER>0000000000000000 61-85 <0s>;<PHONE_NUMBER>0000000000000000 80-90 <0s>
[100-105 <0s>](2):<PHONE_NUMBER>0000000000000000 92-105 <0s>;<PHONE_NUMBER>0000000000000000 100-110 <0s>
[15-17 <0s>](2):<PHONE_NUMBER>0000000000000000 10-20 <0s>;<PHONE_NUMBER>0000000000000000 15-17 <0s>
[21-30 <0s>](2):<PHONE_NUMBER>0000000000000000 20-30 <0s>;<PHONE_NUMBER>0000000000000000 21-31 <0s>
[50-60 <0s>](2):<PHONE_NUMBER>0000000000000000 50-60 <0s>;<PHONE_NUMBER>0000000000000000 50-60 <0s>

Is it worth adding some clear text into this? Otherwise I think people will be puzzled what the different numerical fields mean.

So actually I think it's not needed, because you have different formats/units.
MinT, MaxT are always timestamps, easy to deduct.
Ranges <%s> are in readable duration format (eg 8h)
ULID is easy to spot as well.
The only thing we can is to make it something like this:
`Overlapped range 70-80 <0s> contains 2 blocks: [00000000000000000000000000 61-85 <0s>,<PHONE_NUMBER>0000000000000000 70-80 <0s>]
@fabxc what do you think?

It's easy to deduct if you know what that output actually contains. But as a normal user you just don't, so this might just be a bit more obvious:
[mint: 70 maxt: 80, range: 0s, blocks: 2]: <ulid:<PHONE_NUMBER>0000000000000000, mint: 61, maxt: 85, range: 0s>, <ulid:<PHONE_NUMBER>0000000000000000, mint: 61, maxt: 85, range: 0s>

It's not like we need to save characters.

Done.


Firestore Handling empty document name passed in by a var Type variable on Iphone issue

Issue
The below Issue only happens on Iphones. My Android was working normally.
The below is a code to extract a shopID stored in an invoice under a document name called 'driverAssigned'.
The document Name ('driverAssigned') is an entry in a map called invoiceMap. this map is also being loaded from firestore earlier.
By default new invoices has invoiceMap['driverAssigned'] = ''  Which is an empty string. not null.
if the below firestore call doesn't find the document or receive a null for a document name or even a space, it will throw an error and the Try Catch will be triggered.
If the below firestore call received a '' for a document name, it will crash the app with no error messages and without Try Catch being triggered. The app simply closes immediately. and even print() around it didn't indicate where the app was exactly stopping.

await _firestoreDatabaseReference
          .collection('Drivers')
          .document(invoiceMap['driverAssigned'])
          .get()
          .then((value) => shopID = value.data['DriverAssignedTo']);

I fixed the above flaw by writing

await _firestoreDatabaseReference
          .collection('Drivers')
          .document(invoiceMap['driverAssigned'] == ''
              ? null
              : invoiceMap['driverAssigned'])
          .get()
          .then((value) => shopID = value.data['DriverAssignedTo']);

My question is, why is Try Catch not being triggered? and why does it only behave that way on Iphone? is the compilation for IOS has a flaw where it can't handle empty document name, or is it related to Firestore handling .document('').get() differently than Android?

Hi @FlatCodeIq
From what I can see, the issue is related to a FlutterFire plugin rather than to Flutter itself. Please open the issue in the dedicated repository.
Closing, as this isn't an issue with Flutter itself.
If you disagree, please write in the comments,
providing your flutter doctor -v, your flutter run -v,  your pubspec.yaml
a minimal reproducible code sample that does not use 3rd party plugins,
and I will reopen it.
Thank you


display_timings: fix VS_END typo (HS_STA -> VS_STA)

line
parameter VS_END = HS_STA + 2;

must be
parameter VS_END = VS_STA + 2;

Thank you for spotting this. With this fix, vertical sync is two lines long as expected.