Fixes issue #166

Fixes issue #166
Fixes the sleep problem in windows test script while starting the server. The original ping command does not sleep but returns immediately with the following message:
C:\jenkins\ope-fs-root\workspace\cpp-pr-builder-windows>time /T && ping <IP_ADDRESS>
-n 1 -w 2000000 && time /T
11:01 AM
Pinging <IP_ADDRESS> with 32 bytes of data:
Reply from <IP_ADDRESS>: Destination net unreachable.
Ping statistics for <IP_ADDRESS>:
Packets: Sent = 1, Received = 1, Lost = 0 (0% loss),
11:01 AM
C:\jenkins\ope-fs-root\workspace\cpp-pr-builder-windows>
This was doing the wait correctly before but is not working now, as you see it did not wait 2000 secs.
Hence, change it with CHOICE command.

Test PASSed.


Stop using hashie's master branch

In #166, @olleolleolle switched how_is to using hashie's master branch to avoid these errors getting spewed endlessly:
W, [2016-10-31T20:00:00.000000 #6256]  WARN -- : You are setting a key that conflicts with a built-in method Hashie::Mash#size defined in Hash. This can cause unexpected behavior when accessing the key via as a property. You can still access the key via the #[] method.

Once another hashie version is released, we should switch back away from their master branch.

Resolved! 🥉


How to fix 'Paths configuration XXX uses wildcards which are being deprecated for just leaving a trailing "/" to indicate folder paths.' and how to fix 'defaultJSExtensions configuration option is deprecated ...'

Given a config.js similar to:
System.config({
  defaultJSExtensions: true,
  transpiler: "plugin-babel",
  babelOptions: {
    "optional": [
      "runtime",
      "optimisation.modules.system"
    ]
  },
  paths: {
    "github:*": "jspm_packages/github/*",
    "npm:*": "jspm_packages/npm/*"
  },
  warnings: true,

  meta: {
    "*": {
      "babelOptions": {
        "react": true
      },
      "defaultExtension": "js"
    }
  },

  map: {
    "accounting"<EMAIL_ADDRESS>    "babel"<EMAIL_ADDRESS>    "babel-runtime"<EMAIL_ADDRESS>    "core-js"<EMAIL_ADDRESS>"---snip---":"--------------------snip-------------------------"
  }

I'm not sure how to fix this warning.  Tried removing the wildcard from either side and both sides but still get either deprecation warning or failure to find package.

When I remove line 2: 'defaultJSExtensions: true' then theres an error:
err  Error on fetch for src/pricing at file:///C:/Projects/Arbor/CremationBasics/site_FED/src/pricing
        Error: ENOENT: no such file or directory, open 'C:\Projects\Arbor\CremationBasics\site_FED\src\pricing'
         at Error (native)


[RTL] Move image/table draw to the DRAW_STEP_TEXT

Fixes https://github.com/godotengine/godot/issues/96224

Thanks!


Remove global dependencies

There is no need to install any package that is mentioned in the README.md to be installed globally. We can use their local versions.
Cheers.

npm normally checks local first but for some reason developers still run into problems which is why I asked everyone to still everything globally. we would have to prepend each call with node for example
node ./node_modules/.bin/webpack for windows users unless I'm mistaken

You don't need to advise for any global module installation npm scripts will automatically use executables from node_modules/.bin. A script like this will work even if you don't have Gulp installed globally. This also works on Windows.
I wrote about it a little while ago http://azimi.me/2014/12/18/npm-start.html


feat: adds the capability to include custom user agent string

Adds the ability to include a custom prefix in the user agent string.
Also includes a feature that allows the hyphen in the legacy user agent string to be replaced by a slash to be more in line with RFC9110.
Current default user agent (product + delimiter + version):
Legacy: pandas-2.0.1
RFC9110: pandas/2.0.1
With a prefix:
Legacy: <your_custom_string> pandas-2.0.1
RFC9110: <your_custom_string> pandas/2.0.1
Fixes #807  🦕

Follow-up:
In a future major release (aiming for Q4 2024), deprecate the legacy format of the user agent string.
See details here.


New joke!

Bumped into this in reddit some time ago and thought it was hilarious!
Happy Hacktober!
Joonas

Cool 👍 Thanks 🎉


Configure Renovate

Welcome to Renovate! Once you close this Pull Request, we will begin keeping your dependencies up-to-date via automated Pull Requests.
The Configuration and Configuration FAQ documents should be helpful.
Important!
You do not need to merge this Pull Request - renovate will begin even if it's closed unmerged.
In fact, you only need to add a renovate.json file to your repository if you wish to override any default settings. The file is included as part of this PR only in case you wish to change default settings before you start.
If the default settings are all suitable for you, simply close this Pull Request unmerged and your first renovation will begin the next time the program is run.

This change is


Poem returns 404 NOT FOUND when accessing a endpoint that exists but doesnt allow that method.

Expected Behavior
Expected to return 405 method not allowed for endpoints where they exist but only allow a certain set of methods.
Actual Behavior
Poem will return a 404 if you access an endpoint that exists but with a bad method.
Existing endpoint with correct header:

Existing endpoint but with the wrong method:

Fixed in v1.2.20 🙂

Fixed in v1.2.20 🙂


Cannot install on MAC

Hello, I am trying to install r-exasol on my mac but the compilation fails. It installs fine on my windows machine.
Below is the error
> devtools::install_github("EXASOL/r-exasol")
Downloading GitHub repo EXASOL/r-exasol@HEAD
✓  checking for file ‘/private/var/folders/q7/lvjhd0sx3fd9mqs1h7mlt__40000gn/T/RtmpcQFvyB/remotes20c79c4fb7d/exasol-r-exasol-3784b3c/DESCRIPTION’ ...
─  preparing ‘exasol’:
✓  checking DESCRIPTION meta-information ...
─  cleaning src
─  installing the package to process help pages
         -----------------------------------
─  installing *source* package ‘exasol’ ...
   ** using staged installation
   CPPFLAGS:-I/usr/local/include
   CPPFLAGS:-I/usr/local/include -I.
   checking whether the C++ compiler works... yes
   checking for C++ compiler default output file name... a.out
   checking for suffix of executables...
   checking whether we are cross compiling... no
   checking for suffix of object files... o
   checking whether we are using the GNU C++ compiler... yes
   checking whether clang++ -mmacosx-version-min=10.13 -std=gnu++14 accepts -g... yes
   checking how to run the C++ preprocessor... clang++ -mmacosx-version-min=10.13 -std=gnu++14 -E
   checking whether we are using the GNU C++ compiler... (cached) yes
   checking whether clang++ -mmacosx-version-min=10.13 -std=gnu++14 accepts -g... (cached) yes
   configure: creating ./config.status
   config.status: creating src/Makevars
   ** libs
   clang++ -mmacosx-version-min=10.13 -std=gnu++14 -I"/Library/Frameworks/R.framework/Resources/include" -DNDEBUG -I/usr/local/include -I.  -I'/Library/Frameworks/R.framework/Versions/4.0/Resources/library/testthat/include' -I/usr/local/include   -fPIC  -Wall -g -O2  -c r_exasol/connection_context.cpp -o r_exasol/connection_context.o
   In file included from r_exasol/connection_context.cpp:1:
   In file included from ./r_exasol/connection_context.h:5:
   ./r_exasol/external/sql.h:16:10: error: 'sql.h' file not found with <angled> include; use "quotes" instead
   #include <sql.h>
            ^~~~~~~
            "sql.h"
   ./r_exasol/external/sql.h:17:10: fatal error: 'sqlext.h' file not found
   #include <sqlext.h>
            ^~~~~~~~~~
   2 errors generated.
   make: *** [r_exasol/connection_context.o] Error 1
   ERROR: compilation failed for package ‘exasol’
─  removing ‘/private/var/folders/q7/lvjhd0sx3fd9mqs1h7mlt__40000gn/T/RtmpcjyrBJ/Rinst211d110fa934/exasol’
         -----------------------------------
   ERROR: package installation failed
Error: Failed to install 'exasol' from GitHub:
  System command 'R' failed, exit status: 1, stdout + stderr (last 10 lines):
E>          "sql.h"
E> ./r_exasol/external/sql.h:17:10: fatal error: 'sqlext.h' file not found
E> #include <sqlext.h>
E>          ^~~~~~~~~~
E> 2 errors generated.
E> make: *** [r_exasol/connection_context.o] Error 1
E> ERROR: compilation failed for package ‘exasol’
E> * removing ‘/private/var/folders/q7/lvjhd0sx3fd9mqs1h7mlt__40000gn/T/RtmpcjyrBJ/Rinst211d110fa934/exasol’
E>       -----------------------------------
E> ERROR: package installation failed

Is this a bug or Is there something I need to do additionally for mac?
Thanks

Seems like the user did not install unixodbc, try:

brew install unixodbc

@scartwright91 pls note that we discovered some compatibility issues with RODBC under MacOSX. If you have any connection issues (once it compliled!) please try to re-install RODBC with install.packages("RODBC", type="source") and then reinstall r-exasol. Thanks.

installing unixodbc did the trick! Thank you so much for the quick and helpful response


Generation skips some pages.

Hi everyone,
I'm using the latest stable version on Ubuntu 14-04.64x.
I get the following issue one out of three times:
The PDF generated skips some pages and replace them with blank pages.
Example: It will put the content of page 1, put nothing on page 2 then put content on page 3...
I generate this 170 pages PDF. I get the 1st page, then 40 pages with nothing, then the content continue after page 41..
I have this issue only for files with more than 30 pages (weird).
I checked the content variables when adding them to the PDF, they got content at all time, the tmp files are generated as well...
Could this be related to phpwkhtmltopdf or you'd think it's wkhtmltopdf related ?
Thank you

Can you provide some simple code example to reproduce the issue?

@isbkch As there was no feedback I assume, the issue no longer exists. Closing this for now.


allow kinesis records to contain multiple messages

currently the kinesis producer creates a kinesis record for each gollum message, and the consumer creates a gollum message for each kinesis record.
With this patch, the producer can optionally join multiple gollum messages into a single kinesis record, and the consumer can split kinesis records into multiple gollum messages.
Aggregating messages like this is desirable because records are rounded up to the next 25KB when determining cost and each shard in a stream is limited to 1K record/s or 1MB/s.

Rest looks ok for me.


BUG: Better Uptime Failures are happening every day

https://github.com/VolumeFi/nginx/commit/16aaabb980bc49323d49689b962c5a84cecb3b95
caching responses at 5 seconds.. so that the server responds instantly

@rootedbox can this be closed then?

yes I only updated bots on http://check.bots.palomachain.com/ but I think that's what we care about most. @verabehr


Add memory allocation and gc metrics

This pr adds two new metrics

total amount of memory allocated. When plotted with rateofchange function this should show memory allocation rate. It is similar to existing runtime.jvm.gc.memory.allocated metric. Allocated memory is computed using ThreadMXBean, short lived threads are not tracked.
sum of heap usage before gc - heap usage after gc

@gsmirnov-splk please review

@laurit lgtm, thanks! I'm looking forward to trying out these experimental metrics


[Feature Request] Would like to sync to several accounts

I would like to Sync to several accounts at the same time.
For example:  Two Github accounts and a  GitKraken account.
Currently I tried to add a new Github Token and a GitKraken token
but your extension only allows 1 Github Token at a time.
If this cant be done:
Then it would be nice if we could actually log into a Github or GitKraken account
And Sync to that account that we are currently logged into.
By being able to log in the regular way and then Sync as needed to that signed in account.

Duplicate of #413

When Settings Sync will have git support, you can easily upload where ever you like.


Isn't the player to small?

I think the player is to small...

You can try to add fullscreen button in video.js !


How did u build the 6D pose pool?

in the paper i see u regress the 2d bbox, then u build a 6d pose pool on the object.
I am confused ,because u don't have the pose and the distance to ur camrea of an object ,how to get the (x,y,z) and the pose pool?

Hi, is it possible to compute viewpoint and in-plane rotation given camera pose and object pose in 6D?

Well, it can be done. You could decompose the rotation matrix (in object space) into Euler angles, similar to http://nghiaho.com/?page_id=846, and then compute the polar coordinates for the viewpoint. You would have to figure out, though, which angles correspond to what quantity!

Thanks for the explanation, so in this case the viewpoint can lie on a spherical space around the object? And how can we get the in-plane rotation once we get the viewpoint ?


Gitpod Enhancements - use mvnd by default

The README.md file for the project suggests, using mvnd for faster builds.
The gitpod runtime can be configured to provide the mvnd runtime out-of-the-box.
Please let me know if this sounds like a good idea, I'd be willing to work on it.

I've actually tried it - and added brew install mvndaemon/homebrew-mvnd/mvnd in the init phase. The problem is that it takes ages to install. If you find a better/faster way to do it, i am very open!

what kind of time benchmarks are we expecting?
I tried using sdkman on the gitpod instance and it was pretty fast.
Could you take a look here
https://gitpod.io/#https://github.com/anugrahsinghal/spring-cloud-aws/tree/gitpod-mvnd

Ok i am sold! With brew it took minutes to install, here I did not even realise when installation finished. Send PR @anugrahsinghal

have raised the PR
https://github.com/awspring/spring-cloud-aws/pull/360
Please let me know if this needs any changes.


Memory leak in egui

Describe the bug

A memory leak seems to keep the painter shapes in memory on native and wasm. Both wgpu and glow are affected. Latest version and Master.
It is not the Memory class. It is not other state. It likely is the renderer that does not consume the primitives properly and never drops them.
I even tested it in a previous project and it's there with only the demo windows, it uses egui-vulkano, so the issue is likely in egui itself.
I don't remember a memory leak last time i compiled that project...
To Reproduce
Steps to reproduce the behavior:

Create a new project with the eframe template
Add the demo windows
Request redraw every frame
Memory grows without bounds

Expected behavior
No memory leak

I checked the demo app and it doesn't leak...

i have a demo repo with a 100MB/s leak with a few elements. https://github.com/Wicpar/test_leak

This demo has a lot of code, so it's hard to tell where the leak is coming from, or even if it is egui's fault at all. Maybe you could use something like dhat to narrow it down?

Good catch, with dhat i saw that the accrete library was setting weealloc as the global allocator, which does leak in that scenario, probably because it is unable to handle all the vec allocations. I am frankly shocked that a library can set the global allocator...

however i should note that using dhat that demo repo does still leak, even if at a slower rate.


Style/usability enhancements

This is more for the sake of keeping track of some issues I've noticed, before I or others get a chance to resolve them.

[ ] Footer not present across all pages
[ ] Footer balancing (visual)
[ ] Link hover/active/visited styles
[ ] Text/background contrast
[ ] target="_blank" on external links (and perhaps rel="nofollow" :shrug:)
[ ] Check cross-device/browser, the vmax/vmin/vh/vw units might not behave as expected everywhere...
[ ] Remove even more of the boilerplate styles from the base template. AFAIK we're not using any of the .tdbc-* stuff now?
[ ] The language around past/future events might be a bit clunky when the event is occurring -- Expanded in comment below.

Feel free to add anything else and create a PR for any fixes :100:

Regarding language around future/occurring/past events
Cases:
Future:

event has date, before the event this is in the future

Occurring:

event has date, when the event is happening this is in the past

Past:

event has date, this is in the past
event has happened: true flag too

This would also need some changes to the occured filter (also that's a typo!) to cater to all 3 cases.


buildMatrix

buildMatrix: aCollection
	"Answer a <Collection> of points that are in the same vs different groups.
	Different groups are 1, same groups are 0"

	^ aCollection collect: [ :index |
		(index - aCollection) abs
			collect: [ :each | each min: 1 ] ]

I do not get this code what means index - aCollection ? this code totally confuses me.
So we should add a comment about this implementation

This is a method to build matrix where each cell has 0 if the two input vectors classified a pair in the same cluster, and 1 if there is no match. IIRC the same can be achieved building a contingency matrix - and probably it would be more easier to read, but as this Rand Index seems to be more theoretical than practical (this is explained in the Chapter) I moved to Adjusted Rand Index.


coreutils: fix build on 32-bit ARM

Motivation for this change
Adds an upstream patch to fix tests that fail on 32-bit ARM. The patch actually affects gnulib, which is included as a vendored dependency.
I only applied the patch to 32-bit ARM so this could go in master, but we need to remember to remove it when updating.
cc @edolstra @NeQuissimus @dtzWill
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[x] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

findutils also needs the same patch. I made a copy of the patch, so that we don't have to worry about someone deleting it while another package is still using it.
I'm not sure if there are other packages that use gnulib and need this patch.

I found that gnutls needs the same fix.

I started an evaluation on my hydra: https://hydra.cons.org.nz/eval/3297?compare=nixpkgs-unstable

only mtdutils seems to be failing

That looks like an unrelated issue.

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/prs-already-reviewed/2617/294


Stalling Issue

Check if katana slows down block production to ~6s after sending a lot of transactions (like every 10ms) to it.

Issue was reproduced as part of benchmark in #18 (7/3)


electron-mail: 5.1.8 -> 5.2.2

Description of changes
Fixes broken login due to outdated client package.

Update electron-mail from 5.1.8 to 5.2.2.
Add changelog URL.
Set updateScript to nix-update-script.
Format with nixfmt-rfc-style.

Changelog: https://github.com/vladimiry/ElectronMail/releases/tag/v5.2.2
Just the broken login fix, a few small bugfixes, and updated electron version. No major changes.

Built on platform(s)

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

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

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

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

[ ] Fits CONTRIBUTING.md.

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

This has merge conflicts and should be rebased against master.

Fixed! Thanks.

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

1 package built:

electron-mail


Crash on Android when opening the Time filter

Description of issue
The app crashes after tapping on the Time filter when on the Events listing page. It appears to only affect Android devices.
Expected behaviour
Tapping on the Time filter opens it.
Steps to reproduce

Open the app on an Android device
Go to the Events tab and tap on the Time filter

Crash

OS / Browser / Device model
Galaxy S8 (8.0) - Physical
Galaxy S7 (6.0) - BrowserStack
Galaxy S6 (5.0) - BrowserStack
Pixel 2 (8.0) - BrowserStack

Might not actually be high priority seeing as the Filter work by Robbie removes the separate Time filter anyway... will double-check this once the Filter work is committed to master.

New filter has been committed and the Time filter has been rolled into that, meaning this crash doesn't occur anymore


📂Other: Add hover effects to buttons and navbar links.

What would you like to share?
There is no hover on button like on discord button on homeoage and no effects on navbar links. Add creative effects there.
🥦 Browser
Google Chrome
Checklist ✅

[X] I checked and didn't find similar issue
[X] I have read the Contributing Guidelines
[ ] I am participating in JWOC
[X] I am participating in IWOC
[X] I am willing to work on this issue (blank for no)

Assign me @ItsRoy69 under iwoc

Go for it

Assign me this issue


Indexing

 Environment data

Language Server version: 2024.6.1
OS and version: win32 x64
Python version (and distribution if applicable, e.g. Anaconda):
python.analysis.indexing: true
python.analysis.typeCheckingMode: off

 Code Snippet

XXX

 Repro Steps

XXX

 Expected behavior
XXX
 Actual behavior
XXX
 Logs

XXX

Thanks for the report. Unfortunately, this is not enough information for us to investigate this issue.
If this continues to be a problem, would you please reopen the issue and add the information described in the troubleshooting guide?


Updated NuimoLEDMatrix to include Up/Down Arrow

Two new additions to the LED Matrix file

Thanks, will merge it.
Please note that we'll eventually remove these pre-defined matrices from the SDK. For future you should add these matrices via swift extensions to NuimoLEDMatrix.


when update, guys?

no update, no boobs, only eyes, look at my face

no update, no boobs, only eyes, look at my face

OK, Then close issue


Retry up to N times in case of 429 response

This fixes #31 .
Additional discussion may be warranted before a merge, though, in case the default behavior should possibly be changed.
As of now, I simply expose a hook.Retries field, which can be set to non-zero to enable retries. Otherwise, behavior remains the same as before.

Coverage increased (+1.004%) to 83.702% when pulling 9cd97e90487c91322e66b5520ced4504ecdec68c on flimzy:retry into 3d7f0590b8bf893d3423f355be8dea7078fb3371 on evalphobia:master.

Coverage increased (+1.004%) to 83.702% when pulling b37bcddfca32bb45e88c0e7af8c59ce8d04ced27 on flimzy:retry into 3d7f0590b8bf893d3423f355be8dea7078fb3371 on evalphobia:master.

It seems to me that you'd want to do some kind of exponential backoff here.  If Sentry thinks it's overloaded, hammering it repeatedly seems like the wrong thing to do.

@flimzy Thank you for contributing. And sorry for my lateness.
429 error is the kind of error for rate limit, right?
As @mcpherrinm mentioned, I think it needs some waiting time to guarantee the effectiveness.
For example, HTTP library h2non/gentleman has plugin for retry and it implements waiting time to retry.
It use millisec scale, but Sentry's rate limit seems calcutlates the events on each minute, so the scale should be more than seconds.


Threaded read-hash structure

Waiting on BLAKE2sp and BLAKE2bp variants
Waiting on BLAKE3
Others: Simple message queue

(read+hashing at the same time, messaging system?)

Dropped. I'll come up with something when one of those occurs, anyway.


More than 1 subchart (deps) are ignored

I tried to get a helm chart to build with subcharts, but there doesn't seem to be a way to get that to work. I tried to use the deps option of helm_package, and that works but only for one dep. If I specify two or more, only the most recent chart is included. The one chart that is included, everything is proper.

I'm not sure if the chart.metadata.json is supposed to list anything about the subcharts, but it does not.
the deps_manifest.json does list all the charts

I think this is a bug in the deps handling

That would definitely be a bug! I'd be happy to review a PR to fix it 😄

I added some tests in https://github.com/abrisco/rules_helm/pull/97 which shows dependencies are working correctly, one thing that could be improved is that adding something to the deps attribute still requires you to add the dependency to your Chart.yaml. The rules should probably be updated to error if these two lists are not in sync. Can you double check your Chart.yaml is up to date?

This happens when you have dependencies that have the same target name. E.g.
helm_package(
    deps = ["//a/b/c:yo", "//x/y/z:yo"]
)

This is fixed in #123


SQL parse error when a semi-colon ";" is used in a property name.

When a semi-colon ; is used in a class property name, some functionality (SQL statements using that property name) are lost, throw errors.
This bug is easy to reproduce. Warning: the only way to recover from this bug, back to normal, is to drop the class. You must use the web Studio, Browse tab, to enter your SQL statements.
In any database, such as the demo greatfuldeadconcerts, create a new property with any name, rename that property to a new name with a ; at the end of that new name. Now you are screwed because you have no way to refer to that property name even when using the escape character \; or with quotes around the property name. The parser doesn't ignore the escaped or quoted semi-colon and thinks it is the end of the SQL statement.
example SQL (must use the OrientDB Studio, Browse tab)
CREATE PROPERTY V.test1 String
ALTER PROPERTY V.test1 NAME messedup; //note the semi-colon on the end

SELECT 'messedup;' FROM V  //error
SELECT 'messedup\;' FROM V  //error
SELECT messedup\; FROM V  //error
ALTER PROPERTY V.'messedup;' NAME newName; //error
DROP PROPERTY V.'messedup;' //error

Probably the same with CREATE CLASS but I don't dare try it because you won't be able to drop the class to recover from the bug.
I'm using OrientDB v2.1.9

Hi @Cyberider
you should be able to drop it with a simple
drop property v.messedup;

Anyway, we are working to fix the problem
Thanks
Luigi

Hi @luigidellaquila
Thank you. I tried that and you are right, it does work. I didn't think to try the obvious :-)
I didn't mention in my bug report a weird side effect because it was too much information.
Weird side effect:
if you have a console (CLI) connection to the db while you also have a Studio connection at the same time, and than you create a class property with the semi-colon (in Studio), the console caches the old property name and does not recognize the new property name with the semi-colon in it. I don't know why it would do any caching at all but that is the behaviour I experienced. The only way to bust the cache was for me to exit the OrientDB console session and start over again. Then than the console read the true property name with the semi-colon in it. I thought that was weird.
BTW what is the hidden field "originalName" used for? I thought maybe the console caching weirdness was using that field but "originalName" had the same value as "name". I've yet to see "originalName"  not have the same value as "name".

Problem fixed in v 2.2,  ; is not accepted as a valid property name.
Closing
Luigi


Future Recommendation For Synchronization

I see that you used to have synchronization as a feature.
How about adding support for NextCloud and OwnCloud in the future?
That way, the user could add their own personal servers ("clouds").

Yes there used to be a synchronization feature. The rationale for removing it is explained here. Mostly I just didn't want to rely on a third party service. I have currently no plans in adding it back in any way. Auto-backup perhaps, but I feel like synchronization is really too much of a hassle.

Yeah, I read those and I fully agree, I had just thought of dropping this suggesting in case you may like them.
Auto-backup is a great alternative, especially if the user could restore directly from the server (though I personally would prefer doing that via the system file manager).

What about saving the files as pain text files? Then any user can choose where to save them and use whatever way of syncing things they want.

What about saving the files as pain text files? Then any user can choose where to save them and use whatever way of syncing things they want.

I agree, if notes could be exported in .txt or .md files they could be synced through Nextcloud but you wouldn't need to take care of the development of any sync feature. Actually if it would be possible even to select the folder where notes are saved, it would be even easier (selecting the Nextcloud folder through the stock file manager).

The app currently offers an export as json (settings >export data),and an auto data export. I just use syncthing (why use a cloud service which someone else controls?) to sync those files across devices.

The app currently offers an export as json (settings >export data),and an auto data export. I just use syncthing (why use a cloud service which someone else controls?) to sync those files across devices.

But how can you open it on the desktop then?

json is a common standard, plenty of json editor apps available online/to download. Syncthing is just one app (which btw is also available on Windows etc)
Note the data is still viewable in a text editor, just with a bunch of json formatting.

json is a common standard, plenty of json editor apps available online/to download. Syncthing is just one app (which btw is also available on Windows etc) to share across devices.
Note the data is still viewable in a text editor, just with a bunch of json formatting.

Oh yes I know, I was referring to which app can you use that picks up the Another Notes App json formatting. I think that's the main issue here with syncing. I can sync across other android devices with autobackup to a synced folder that gets picked up from the other device with Another Notes App installed. But using a more standard format would allow people to sync to other platforms as well, and in general to use other notes apps that use the same standard format.

Ah sorry I misunderstood then, I only sync across android devices using the same app so inter app compatibility hasn't been an issue for me.
I feel this now raises the question what's really the "standard format" for notes apps, I see the dev here decided on json and that'd be my preference too since it preserves formatting that text might lack. But sure, this is an interesting topic :)

Ah sorry I misunderstood then, I only sync across android devices using the same app so inter app compatibility hasn't been an issue for me.
I feel this now raises the question what's really the "standard format" for notes apps, I see the dev here decided on json and that'd be my preference too since it preserves formatting that text might lack. But sure, this is an interesting topic :)

Well yes, probably that's the right question!
I agree that json is a pretty good and accessible format, the only issue is that each app uses different json formats, which in the end does not allow cross compatibility among apps, correct me if I'm wrong. Maybe markup could be a compromise? I see it's pretty common nowadays in note apps. But yes, it's hard to find a "standard" format for notes today, but I think such direction should be taken, at least by FOSS apps.


MacOS standalone binary hashing broken

The password hashes generated by the MacOS standalone binary v2.0.1 always lead to authentication errors.
The error must be located in the hash computation because the default config files shipped with the standalone binary work with both the standalone binary and the deepstream server installed via npm. Similarly, using newly generated password hashes causes authentication failures with both programs.
Steps to reproduce:

download the binary (direct link)
unzip
enable file-based authentication in conf/config.yml
start the server
connect with a client, username userA, password password
observe AUTH_SUCCESSFUL message in server output
kill client and server
compute a password hash with ./deepstream hash password
copy the new hash into conf/users.yml
start the server
connect with a client, username userA, password password
observe INVALID_AUTH_DATA message in server output, AUTH_SUCCESSFUL is expected

Hash computation with deepstream 2.0.1 installed with npm works.

Can we reverify if this is broken?


docker swarm fails to retrive node details. Throws  "404 page not found"

I am able to perform most of the commands on swarm except node. Not sure what I am missing here.
docker -H :4000 pull, stop, start, rmi, rm, ps, network, commands works perfectly fine , without any issue.
uname -a
Linux  3.10.0-327.18.2.el7.x86_64 #1 SMP Fri Apr 8 05:09:53 EDT 2016 x86_64 x86_64 x86_64 GNU/Linux
docker -H :4000 node ls
Error response from daemon: 404 page not found
docker version
Client:
Version:      1.12.3
API version:  1.24
Go version:   go1.6.3
Git commit:   6b644ec
Built:
OS/Arch:      linux/amd64
Server:
Version:      1.12.3
API version:  1.24
Go version:   go1.6.3
Git commit:   6b644ec
Built:
OS/Arch:      linux/amd64
docker -H :4000 info
Containers: 26
Running: 26
Paused: 0
Stopped: 0
Images: 41
Server Version: swarm/1.2.5
Role: primary
Strategy: spread
Filters: health, port, containerslots, dependency, affinity, constraint
Nodes: 7
host1: <IP_ADDRESS>:2375
└ ID: OVAQ:A73R:ALFQ:7HAM:PQYI:VA5Z:322Y:BLOO:QSPV:ZYW7:7PFT:6GHG
└ Status: Healthy
└ Containers: 3 (3 Running, 0 Paused, 0 Stopped)
└ Reserved CPUs: 0 / 4
└ Reserved Memory: 0 B / 16.29 GiB
└ Labels: kernelversion=3.10.0-327.18.2.el7.x86_64, operatingsystem=Red Hat Enterprise Linux Server 7.2 (Maipo), storagedriver=devicemapper
└ UpdatedAt: 2016-12-02T20:20:23Z
└ ServerVersion: 1.12.3
host2: <IP_ADDRESS>:2375
└ ID: PPQL:BNMR:S7UH:6X5L:ZRU7:B6S2:AW7M:DMCU:KMNW:OZVJ:PBOO:AGV3
└ Status: Healthy
└ Containers: 4 (4 Running, 0 Paused, 0 Stopped)
└ Reserved CPUs: 0 / 4
└ Reserved Memory: 0 B / 16.29 GiB
└ Labels: kernelversion=3.10.0-327.18.2.el7.x86_64, operatingsystem=Red Hat Enterprise Linux Server 7.2 (Maipo), storagedriver=devicemapper
└ UpdatedAt: 2016-12-02T20:20:25Z
└ ServerVersion: 1.12.3
host3: <IP_ADDRESS>:2375
└ ID: RLR6:VPVQ:KEO5:ZSEN:VTHF:A34Y:236R:DLGL:PZ6N:4U5B:6CI7:IGQB
└ Status: Healthy
└ Containers: 3 (3 Running, 0 Paused, 0 Stopped)
└ Reserved CPUs: 0 / 4
└ Reserved Memory: 0 B / 16.29 GiB
└ Labels: kernelversion=3.10.0-327.18.2.el7.x86_64, operatingsystem=Red Hat Enterprise Linux Server 7.2 (Maipo), storagedriver=devicemapper
└ UpdatedAt: 2016-12-02T20:20:29Z
└ ServerVersion: 1.12.3
host4: <IP_ADDRESS>:2375
└ ID: IWO3:ZTZT:A22D:PHMU:UDMV:QSFK:GII7:GP64:6PR4:7MPS:EPAY:6TBV
└ Status: Healthy
└ Containers: 3 (3 Running, 0 Paused, 0 Stopped)
└ Reserved CPUs: 0 / 4
└ Reserved Memory: 0 B / 16.29 GiB
└ Labels: kernelversion=3.10.0-327.18.2.el7.x86_64, operatingsystem=Red Hat Enterprise Linux Server 7.2 (Maipo), storagedriver=devicemapper
└ UpdatedAt: 2016-12-02T20:20:05Z
└ ServerVersion: 1.12.3
host5: <IP_ADDRESS>:2375
└ ID: LLQS:223T:LZUT:LZGZ:BVNV:TIS7:LK3O:GUGA:EWBZ:BLZ6:2LXL:3RAV
└ Status: Healthy
└ Containers: 3 (3 Running, 0 Paused, 0 Stopped)
└ Reserved CPUs: 0 / 4
└ Reserved Memory: 0 B / 16.29 GiB
└ Labels: kernelversion=3.10.0-327.18.2.el7.x86_64, operatingsystem=Red Hat Enterprise Linux Server 7.2 (Maipo), storagedriver=devicemapper
└ UpdatedAt: 2016-12-02T20:20:35Z
└ ServerVersion: 1.12.3
host6: <IP_ADDRESS>:2375
└ ID: KT5I:7JEF:LQ5W:QCBH:NY7X:S3FA:C6UD:QXDY:C4AB:7RXO:XCZQ:CXLG
└ Status: Healthy
└ Containers: 5 (5 Running, 0 Paused, 0 Stopped)
└ Reserved CPUs: 0 / 4
└ Reserved Memory: 0 B / 16.29 GiB
└ Labels: kernelversion=3.10.0-327.18.2.el7.x86_64, operatingsystem=Red Hat Enterprise Linux Server 7.2 (Maipo), storagedriver=devicemapper
└ UpdatedAt: 2016-12-02T20:20:43Z
└ ServerVersion: 1.12.3
host7: <IP_ADDRESS>:2375
└ ID: THDB:T5EC:C7JF:XYGH:ZV7Q:X47S:OTNT:DZUI:JXGM:NMQX:DXFO:SFPK
└ Status: Healthy
└ Containers: 5 (5 Running, 0 Paused, 0 Stopped)
└ Reserved CPUs: 0 / 4
└ Reserved Memory: 0 B / 16.29 GiB
└ Labels: kernelversion=3.10.0-327.18.2.el7.x86_64, operatingsystem=Red Hat Enterprise Linux Server 7.2 (Maipo), storagedriver=devicemapper
└ UpdatedAt: 2016-12-02T20:20:13Z
└ ServerVersion: 1.12.3
Plugins:
Volume:
Network:
Swarm:
NodeID:
Is Manager: false
Node Address:
Security Options:
Kernel Version: 3.10.0-327.18.2.el7.x86_64
Operating System: linux
Architecture: amd64
CPUs: 28
Total Memory: 114 GiB
Name: c7ef56122f88
Docker Root Dir:
Debug Mode (client): false
Debug Mode (server): false
WARNING: No kernel memory limit support
http-proxy.conf
[Service] Environment="HTTP_PROXY=http://proxy-domain.com:port" "HTTPS_PROXY=https://proxy-domain.com:port"
docker.conf
[Service] ExecStart= ExecStart=/usr/bin/dockerd -H tcp://<IP_ADDRESS>:2375 -H unix:///var/run/docker.sock -g /docker --cluster-store=consul://consul-server:8500 --cluster-advertise eth0:2375
echo -e "GET /images/json HTTP/1.0\r\n" | nc -U /var/run/docker.sock
HTTP/1.0 200 OK
Content-Type: application/json
Server: Docker/1.12.3 (linux)
Date: Fri, 02 Dec 2016 20:36:27 GMT
Content-Length: 1764
Goal
assign node labels on swarm, however docker -H :4000 node * fails.

I think your command is again docker swarm which is different  from swarm mode in docker engine.

Yes, it looks like you're mixing "classic" swarm (https://docs.docker.com/swarm/), with the new swarm mode that was introduced in docker 1.12 (https://docs.docker.com/engine/swarm/)
I'll close this issue because this is not a bug, but happy to answer questions you may have

Thank you all . I think its answers my question. I will upgrade my infrastructure to new swarm mode and update my findings here.

Confirm docker node * works on swarm mode.


Bug: Application error: Failed to read large IndexedDB value DataError: Failed to read large ...

Is there an existing issue for this?

[X] I have searched the existing issues

Current Behavior
Error message: Failed to read large IndexedDB value
DataError: Failed to read large IndexedDB value
Error stack:
DataError: Failed to read large IndexedDB value
 DataError: Failed to read large IndexedDB value
    at vn.rejectionMapper (altair://-/main.js:3:369206)
    at Xk (altair://-/main.js:3:303628)
    at IDBRequest.<anonymous> (altair://-/main.js:3:330469)
    at IDBRequest.__zone_symbol__ON_PROPERTYerror (altair://-/main.js:3:305416)
    at IDBRequest.Qt (altair://-/polyfills.js:3:29957)
    at t.invokeTask (altair://-/polyfills.js:3:25686)
    at Object.onInvokeTask (altair://-/main.js:3:80112)
    at t.invokeTask (altair://-/polyfills.js:3:25607)

Expected Behavior
No errors should occur
Steps To Reproduce
This will happen when you reopen it after installation
Environment
- OS:
- Browser:
- Platform:
- Version: 7.3.3

Additional context
No response

Looks like an issue in chromium https://issues.chromium.org/issues/342779913?pli=1

Other related: https://github.com/johannesjo/super-productivity/issues/607


Create docs on network-based health checks

Doing some cleanup of issues. I assume this is covered by docs at https://flux-iac.github.io/tofu-controller/use-tf-controller/provision-Terraform-resources-that-are-required-health-checks/.
Feel free to reopen if that isn't the case.


LocalStack currently doesn't seem to confirm SNS subscriptions

LocalStack in our current setup either doesn't seem to confirm SNS subscriptions or it doesn't send messages to subscribers. Providing an endpoint with host.docker.internal as host doesn't work either.

https://hub.docker.com/r/s12v/sns could be another option. However, currently this error crops up when publishing a message to an SNS topic via HTTP on Fake SNS: https://github.com/s12v/sns/blob/master/example/config/elasticmq.conf
fake-sns_1    | akka.camel.AkkaCamelException: Connection refused (Connection refused) fake-sns_1    | 	at akka.camel.ProducerSupport$$anonfun$produce$1.applyOrElse(Producer.scala:77) ~[sns.jar:0.4.1] fake-sns_1    | 	at akka.actor.Actor.aroundReceive(Actor.scala:517) ~[sns.jar:0.4.1] fake-sns_1    | 	at akka.actor.Actor.aroundReceive$(Actor.scala:515) ~[sns.jar:0.4.1] fake-sns_1    | 	at me.snov.sns.actor.ProducerActor.aroundReceive(ProducerActor.scala:14) ~[sns.jar:0.4.1] fake-sns_1    | 	at akka.actor.ActorCell.receiveMessage(ActorCell.scala:527) ~[sns.jar:0.4.1] fake-sns_1    | 	at akka.actor.ActorCell.invoke(ActorCell.scala:496) ~[sns.jar:0.4.1] fake-sns_1    | 	at akka.dispatch.Mailbox.processMailbox(Mailbox.scala:257) ~[sns.jar:0.4.1] fake-sns_1    | 	at akka.dispatch.Mailbox.run(Mailbox.scala:224) ~[sns.jar:0.4.1] fake-sns_1    | 	at akka.dispatch.Mailbox.exec(Mailbox.scala:234) ~[sns.jar:0.4.1] fake-sns_1    | 	at akka.dispatch.forkjoin.ForkJoinTask.doExec(ForkJoinTask.java:260) ~[sns.jar:0.4.1] fake-sns_1    | 	at akka.dispatch.forkjoin.ForkJoinPool$WorkQueue.runTask(ForkJoinPool.java:1339) ~[sns.jar:0.4.1] fake-sns_1    | 	at akka.dispatch.forkjoin.ForkJoinPool.runWorker(ForkJoinPool.java:1979) ~[sns.jar:0.4.1] fake-sns_1    | 	at akka.dispatch.forkjoin.ForkJoinWorkerThread.run(ForkJoinWorkerThread.java:107) ~[sns.jar:0.4.1] fake-sns_1    | Caused by: java.net.ConnectException: Connection refused (Connection refused) fake-sns_1    | 	at java.net.PlainSocketImpl.socketConnect(Native Method) ~[na:1.8.0_111-internal] fake-sns_1    | 	at java.net.AbstractPlainSocketImpl.doConnect(AbstractPlainSocketImpl.java:350) ~[na:1.8.0_111-internal] fake-sns_1    | 	at java.net.AbstractPlainSocketImpl.connectToAddress(AbstractPlainSocketImpl.java:206) ~[na:1.8.0_111-internal] fake-sns_1    | 	at java.net.AbstractPlainSocketImpl.connect(AbstractPlainSocketImpl.java:188) ~[na:1.8.0_111-internal] fake-sns_1    | 	at java.net.SocksSocketImpl.connect(SocksSocketImpl.java:392) ~[na:1.8.0_111-internal] fake-sns_1    | 	at java.net.Socket.connect(Socket.java:589) ~[na:1.8.0_111-internal] fake-sns_1    | 	at java.net.Socket.connect(Socket.java:538) ~[na:1.8.0_111-internal] fake-sns_1    | 	at java.net.Socket.<init>(Socket.java:434) ~[na:1.8.0_111-internal] fake-sns_1    | 	at java.net.Socket.<init>(Socket.java:286) ~[na:1.8.0_111-internal] fake-sns_1    | 	at org.apache.commons.httpclient.protocol.DefaultProtocolSocketFactory.createSocket(DefaultProtocolSocketFactory.java:80) ~[sns.jar:0.4.1] fake-sns_1    | 	at org.apache.commons.httpclient.protocol.DefaultProtocolSocketFactory.createSocket(DefaultProtocolSocketFactory.java:122) ~[sns.jar:0.4.1] fake-sns_1    | 	at org.apache.commons.httpclient.HttpConnection.open(HttpConnection.java:707) ~[sns.jar:0.4.1] fake-sns_1    | 	at org.apache.commons.httpclient.MultiThreadedHttpConnectionManager$HttpConnectionAdapter.open(MultiThreadedHttpConnectionManager.java:1361) ~[sns.jar:0.4.1] fake-sns_1    | 	at org.apache.commons.httpclient.HttpMethodDirector.executeWithRetry(HttpMethodDirector.java:387) ~[sns.jar:0.4.1] fake-sns_1    | 	at org.apache.commons.httpclient.HttpMethodDirector.executeMethod(HttpMethodDirector.java:171) ~[sns.jar:0.4.1] fake-sns_1    | 	at org.apache.commons.httpclient.HttpClient.executeMethod(HttpClient.java:397) ~[sns.jar:0.4.1] fake-sns_1    | 	at org.apache.commons.httpclient.HttpClient.executeMethod(HttpClient.java:323) ~[sns.jar:0.4.1] fake-sns_1    | 	at org.apache.camel.component.http.HttpProducer.executeMethod(HttpProducer.java:288) ~[sns.jar:0.4.1] fake-sns_1    | 	at org.apache.camel.component.http.HttpProducer.process(HttpProducer.java:170) ~[sns.jar:0.4.1] fake-sns_1    | 	at org.apache.camel.util.AsyncProcessorConverterHelper$ProcessorToAsyncProcessorBridge.process(AsyncProcessorConverterHelper.java:61) ~[sns.jar:0.4.1] fake-sns_1    | 	at org.apache.camel.processor.SendProcessor.process(SendProcessor.java:145) ~[sns.jar:0.4.1] fake-sns_1    | 	at akka.camel.ProducerSupport$ProducerChild.produce(Producer.scala:141) ~[sns.jar:0.4.1] fake-sns_1    | 	at akka.camel.ProducerSupport$ProducerChild$$anonfun$receive$1.applyOrElse(Producer.scala:115) ~[sns.jar:0.4.1] fake-sns_1    | 	at akka.actor.Actor.aroundReceive(Actor.scala:517) ~[sns.jar:0.4.1] fake-sns_1    | 	at akka.actor.Actor.aroundReceive$(Actor.scala:515) ~[sns.jar:0.4.1] fake-sns_1    | 	at akka.camel.ProducerSupport$ProducerChild.aroundReceive(Producer.scala:112) ~[sns.jar:0.4.1] fake-sns_1    | 	... 9 common frames omitted

Please also refer to this LocalStack issue: https://github.com/localstack/localstack/issues/3314

Won't fix because we're now using Amazon MQ instead of SNS anyway.

Won't fix because we're now using Amazon MQ instead of SNS anyway.


pull request for e2e test at 01/25/2017 02:56:07

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


[CP] text direction unicode hints

commit(s) to merge: 39008c82435fc266e8ac17fefacc1b14afb58bbe
Cherry-pick CL: https://dart-review.googlesource.com/c/sdk/+/219743
merge instructions: clean merge or patch CL
What is the issue:  Dart tools do not guard against  text direction unicode security vulnerabilities
What is the fix: Adds hints to identify text direction unicode sequences
Why cherrypick: Dart steering committee prioritized this fix for beta
Risk: possible false positives; performance degradation.
Link to original  issue(s):
https://github.com/dart-lang/sdk/issues/47611
/cc @kevmoo @mit-mit @whesse @athomas @vsmenon @devoncarew

An update: this has been reverted in google3. We'll need to resolve and re-land (but this may not be in a position to cherry pick for a bit).

From the scrum, we'll plan to review this once it's landed and been vetted in google3.

The Dart CI build for the re-land ran green and made it into the SDK w/ b403d4b.
If the roll tomorrow doesn't shake any big issues out, I think we can look at this again.

@pq  - I believe the commit for this cherry pick is correct, but the gerrit review (at the top) is not. Can you confirm and remove the CL if that's the case?

Stale cherry pick CL removed.

approved

Merged to beta in 52519ea8eb4780c468c4c2ed00e7c8046ccfed41 (2.15.0-268.18.beta).


Branch 2.2

Can one of the admins verify this patch?

Can one of the admins verify this patch?

@zhuge134, looks mistaken open. Could you please double-check and close this PR?


Deserialize request params to their corresponding schema field data types

E.g. For a GET request sending false or False in the request body should automatically convert it to python False in the kwargs to the read_list method
RB.stages.fetch({ is_prospecting: false})
def read_list(self, request, kwargs):
    #  where kwargs['is_prospecting'] should be False and not u'false'

Things can get complex when we have an object graph instead of a single flat object. So we can at least do this for the first level of simple data types

@uttamk Have cloned to have a go at it. Thoughts welcome.

Cool !!
On 4 Dec 2015 8:18 pm, "Robin Maben"<EMAIL_ADDRESS>wrote:

@uttamk https://github.com/uttamk Have cloned to have a go at it.
Thoughts welcome.
—
Reply to this email directly or view it on GitHub
https://github.com/Aplopio/rip/issues/30#issuecomment-161984321.

@robinmaben any progress on this one ?

@uttamk Haven't started yet.- was waiting for the RIP vs DRF match results. :stuck_out_tongue:   Just read your match report. Will plan for this now since we're sticking to RIP.


fix: do not require gamemode installed

Unfortunately, screensaver shenanigans are not over yet...
When I tested gamemode I used my Slackware desktop PC, and I had installed gamemode on in to test it out. Earlier when I tried to run zwift on my Arch Linux laptop, unfortunately I encountered the screensaver and monitor power off issue again...
Turns out that gamemode needs to be installed on the host system for it to work. In the arch linux laptop, it was missing.
When it is missing, you can see an error:
+ /usr/games/gamemoderun wineserver -w
GameMode ERROR: D-Bus error: Could not call method 'RegisterGame' on 'com.feralinteractive.GameMode': The name is not activatable
gamemodeauto: D-Bus error: Could not call method 'RegisterGame' on 'com.feralinteractive.GameMode': The name is not activatable
GameMode ERROR: D-Bus error: Could not call method 'UnregisterGame' on 'com.feralinteractive.GameMode': The name is not activatable
gamemodeauto: D-Bus error: Could not call method 'UnregisterGame' on 'com.feralinteractive.GameMode': The name is not activatable

That is because usr/share/dbus-1/services/com.feralinteractive.GameMode.service is missing. In this case, the screensaver Inhibit method will not be called, and the screen will go on to be activated. If gamemode is installed in the host system, this will work fine though.
The aim of this PR is to try to have the best of all worlds, we do not require gamemode to be available on the host system, and do the heartbeat like we used to... I don't think those will conflict now that I think about it, since they use different mechanisms to inhibit the screensaver. An alternative might be to mention that gamemode needs to be available on the host system to work...

 Am I the only one who has the computer setup to power off my monitors or to suspend when idle after a while?
I think this should be part of zwift, just like it is part of lutris or firefox... the problem here is that since it is running inside a container, some things are more complicated...

I did try running gamemode outside of docker but it is then not aware.  Gamemode is used by default on quite a few distros so maybe, warn on startup gamemode is not available so zwift won't inhibit screen saver by default?
Then if there use gamemode if not just run as normal?
Choice then is install gamemode on the host or manage screen saver yourself.  Readme can be updated.

I can live with installing gamemode on my hosts. I do think that the solution in this PR is more user friendly, but hardly fool-proof, so feel free to close it if you don't want to implement it

I can update the Readme doing some workflow updates for the rebuild/ update improvmenets can do some work on that. Yes I think it we try to cover too much it could be a can of works and game mode does so much already.

having slept on it, I think another option to consider would be to forgo gamemode and call the inhibit methods directly:
COOKIE=$(dbus-send --session \
                  --dest=org.freedesktop.ScreenSaver \
                  --type=method_call \
                  --print-reply \
                  --reply-timeout=2000 \
                  /ScreenSaver \
                  org.freedesktop.ScreenSaver.Inhibit \
                  string:zwift \
                  string:zwift \
                  | grep uint32 | cut -d ' ' -f 5)

This would be called before starting zwift, and after zwift exits, we call
dbus-send --session \
                      --dest=org.freedesktop.ScreenSaver \
                      --type=method_call \
                      /ScreenSaver \
                      org.freedesktop.ScreenSaver.UnInhibit \
                      uint32:$COOKIE

The advantage here is not needing an additional dependency on the host machine (this could be mitigated by a package manager that would handle these dependencies though). Personally, now that I know about this, I have no problem installing gamemode on hosts where I'll use zwift, but I can see this being a headache for other people who may not dig into the README right away. Probably should have just gone for this method from the beginning, since tools like xdg-screensaver, in their attempt to handle as many cases as possible, can become quite convoluted on its own, not to mention inside a container.
The disadvantage to this method would be losing gamemode performance benefits, but those are not entirely clear to me, and I'm wondering if they would apply if gamemode is running inside a container?
Anyways, I realize that at this point you're probably tired of this issue which doesn't seem to affect you, sorry about that

First off, I can assure you I'm not tired of the issue 🤗. And yes it doesn't affect me as I don't use a screensaver and generally I always have my monitor on.
I do however want all parties involved to find the best solution to this problem, as it is clearly a problem for users running screensavers and power management.
So I think it's worth thinking about alternative solutions as stated in my previous post, as you now clearly have. 👍
I'm still on vacation, but I'll try to delve deeper into this when I get back home.
At the same time that I want to find a good solution I also want it to be as minimal as possible (separation of concerns and all that good stuff)
The main focus of the project should be the ease of use and installation of zwift but also keeping complexity to a minimum for users and maintainers.
With this in mind, I think we can handle this issue.

Oh, and I do like your last suggestion here with just invoking dbus.
Maybe we should have pre and post Zwift hooks that can be configured and maybe extended with custom functionality?
Could be as simple as two bash scripts being invoked before and after container startup and exit.
And maybe screensaver could be it's first citizen?
Just a suggestion, or food for thought. As it might just increase complexity as I can't think of any other things that would require pre-post hooks.

Just a note on game mode, this is some of the key things it does, also on my laptop it disabled powersave whilst running so no sleep.

CPU governor
I/O priority
Process niceness
Kernel scheduler (SCHED_ISO)
Screensaver inhibiting
GPU performance mode (NVIDIA and AMD), GPU overclocking (NVIDIA)
Custom scripts

It gives about a 10% boost in performance, which is a lot less noticable on my current over spec'd machine that it used to be when my FPS was 50.
It comes pre installed on most of the distros I use, some games in linux are starting to have it enabled by default and steam uses it.
I guess downside is for distros where it is not pre-installed, I guess distros where you start with more base install and add what you want on top.  Reasons I suggested:

its been around for ages version 1.0 was 2018, and works on pretty much all distros I can think of
easier for people with less experience who just want to use a simple distro and have it work and easy to install in most
Easier for us as a lot of people contribute to it to make it work on as many systems as possible.
You can customize gamemode on your host device if you really want to using config files, which would be independant of the zwitf install which is just saying gamemode please.

Not sure how good it is on systems without systemd but apparently it does support them.
I am not sure of many downsides to using it by default but maybe adding ability to set alternative by custom scripts embedded.
Open for anything with one priority for me: ease of use for non-technical people who just want to switch to linux because windows is spyware.

Interesting, I also didn't notice any performance improvements on my PC, but if you could benchmark some noticeable improvements, that sounds good!

Difference of machines, noticing a powerful machine like an AMD 5700 and GeForce 3050 difference might be beetween 300 adn 330 fps which is not noticable.  The 10% comes from other people doing tests but I noticed this when playing games on my laptops.

No new functionality gets added, but add a warning on startup about gamemode needing to be available for best features. In this case, it might be worth it to build packages with dependencies for different distros, that way less users need to think about hunting down dependencies.

Gamemode does require systemd, so yes there are distro's that it won't work on.  Not sure what you mean by dependancies for different distros?

In addition to gamemode, we add the dbus calls. I don't think having multiple calls to Inhibit is problematic, and if someone forgets to install gamemode, they will still get a fallback that should work on most common cases.

True, if that is what you meant by item 1.  So basically gamemode unless not installed then fallback to another method?  Can we tell if gamemode works without using the install script (i.e. in the docker)

If you are interested in pursuing any of those, I will happily put together a PR!

Works for me, I am happy to update the README, I was thinking becuase of the additions updating the readme to add at the top a simple install guide for non-tech, setting up zwift/config, a little more detail for tech and re-layout the FAQ?
I am currently working on a rebuild workflow.

Gamemode does require systemd, so yes there are distro's that it won't work on. Not sure what you mean by dependancies for different distros?

Actually, on Slackware we use sysvinit, but gamemode is available from SBo and the inhibit functionality works, so systemd is not required, only only a subset of the functionality requires systemd.
What I meant, is to have a package for zwift on aur, SBo, etc, that will specify gamemode as a dependency
I think we can detect gamemode from within the container by calling the Introspect method for com.feralinteractive.GameMode:
$ dbus-send --session           \
  --dest=com.feralinteractive.GameMode \
  --type=method_call          \
  --print-reply               \
  /com/feralinteractive/GameMode       \
  org.freedesktop.DBus.Introspectable.Introspect
method return time=1720622791.599156 sender=:1.62 -> destination=:1.759 serial=25 reply_serial=2
   string "<!DOCTYPE node PUBLIC "-//freedesktop//DTD D-BUS Object Introspection 1.0//EN"
"https://www.freedesktop.org/standards/dbus/1.0/introspect.dtd">
<node>
 <interface name="org.freedesktop.DBus.Peer">
  <method name="Ping"/>
  <method name="GetMachineId">
   <arg type="s" name="machine_uuid" direction="out"/>
  </method>
 </interface>
 <interface name="org.freedesktop.DBus.Introspectable">
  <method name="Introspect">
   <arg name="xml_data" type="s" direction="out"/>
  </method>
 </interface>
 <interface name="org.freedesktop.DBus.Properties">
  <method name="Get">
   <arg name="interface_name" direction="in" type="s"/>
   <arg name="property_name" direction="in" type="s"/>
   <arg name="value" direction="out" type="v"/>
  </method>
  <method name="GetAll">
   <arg name="interface_name" direction="in" type="s"/>
   <arg name="props" direction="out" type="a{sv}"/>
  </method>
  <method name="Set">
   <arg name="interface_name" direction="in" type="s"/>
   <arg name="property_name" direction="in" type="s"/>
   <arg name="value" direction="in" type="v"/>
  </method>
  <signal name="PropertiesChanged">
   <arg type="s" name="interface_name"/>
   <arg type="a{sv}" name="changed_properties"/>
   <arg type="as" name="invalidated_properties"/>
  </signal>
 </interface>
 <interface name="com.feralinteractive.GameMode">
  <property name="ClientCount" type="i" access="read">
  </property>
  <method name="RegisterGame">
   <arg type="i" direction="in"/>
   <arg type="i" direction="out"/>
  </method>
  <method name="UnregisterGame">
   <arg type="i" direction="in"/>
   <arg type="i" direction="out"/>
  </method>
  <method name="QueryStatus">
   <arg type="i" direction="in"/>
   <arg type="i" direction="out"/>
  </method>
  <method name="RegisterGameByPID">
   <arg type="i" direction="in"/>
   <arg type="i" direction="in"/>
   <arg type="i" direction="out"/>
  </method>
  <method name="UnregisterGameByPID">
   <arg type="i" direction="in"/>
   <arg type="i" direction="in"/>
   <arg type="i" direction="out"/>
  </method>
  <method name="QueryStatusByPID">
   <arg type="i" direction="in"/>
   <arg type="i" direction="in"/>
   <arg type="i" direction="out"/>
  </method>
  <method name="RegisterGameByPIDFd">
   <arg type="h" direction="in"/>
   <arg type="h" direction="in"/>
   <arg type="i" direction="out"/>
  </method>
  <method name="UnregisterGameByPIDFd">
   <arg type="h" direction="in"/>
   <arg type="h" direction="in"/>
   <arg type="i" direction="out"/>
  </method>
  <method name="QueryStatusByPIDFd">
   <arg type="h" direction="in"/>
   <arg type="h" direction="in"/>
   <arg type="i" direction="out"/>
  </method>
  <method name="RefreshConfig">
   <arg type="i" direction="out"/>
  </method>
  <method name="ListGames">
   <arg type="a(io)" direction="out"/>
  </method>
  <signal name="GameRegistered">
   <arg type="i"/>
   <arg type="o"/>
  </signal>
  <signal name="GameUnregistered">
   <arg type="i"/>
   <arg type="o"/>
  </signal>
 </interface>
 <node name="Games"/>
</node>
"

This is the result when the object is available on dbus, meaning gamemode is installed on the host.

Actually, on Slackware we use sysvinit, but gamemode is available from SBo and the inhibit functionality works, so systemd is not required, or only a subset of the functionality requires systemd.
What I meant, is to have a package for zwift on aur, SBo, etc, that will specify gamemode as a dependency

The zwift startup application part.  So would have to have Podman/ Docker and if you have Nvidia the Nvidia Toolkit, well I suppose not really a bother for me as I use flatpak now in preference (where maintained properly).

I think we can detect gamemode from within the container by calling the Introspect method for com.feralinteractive.GameMode:

That sounds like being useful, but then just document about gamemode in the readme rather than the zwift startup file, or use the zwift startup to detect gamemoded -v, then we can show a warning if not there and pass a variable?

I think if adding support, maybe Gamemode if available, simple screen inhibit if not and add potential to support local pre/ post scripts?
I think check for game mode in the startup script then a warning can be displayed?


Sign In with Apple can't link to Anonymous user, Help!

[REQUIRED] Step 2: Describe your environment

Xcode version: 11.3
Firebase SDK version: 6.14.0
Firebase Component: Auth
Component version: AppAuth (1.3.0) FirebaseAuth (~> 6.4.1) FirebaseUI/Auth (= 8.4.0)

[REQUIRED] Step 3: Describe the problem
Sign In with Apple can't link to Anonymous user.
Steps to reproduce:

Relevant Code:
// TODO(you): code here to reproduce the problem

I am using a temporary solution now, if an anonymous user is using Apple login, then hide Apple login Button.

Looks like the same issue as #4434, though this is using FirebaseUI. We should definitely fix the flow in FirebaseUI so it works automatically, but in the meantime you should be able to link users manually using the flow described here.

I think this may be a cause of the problem, but I am using FirebaseUI. When an anonymous user is upgraded to a user logged in by Apple, _currentUser is nil, which causes autoUpgrade to not be called normally.


No zoom controls in the map view

Issue
There are no zoom in or zoom out buttons available in the map view
Expected behavior
There should be zoom in and out buttons in the map view. There is a fab icon covering the possible location for zoom controls. We can set the fab action to the navigation menu and remove it to provide space for zoom controls.
Steps to reproduce

Open app
Login with valid credentials
Observe map

Would you like to work on this issue?
Yes and I can send a PR for review!

I have fixed this issue and made a pull request for this issue


Python 3.8 not supported

Stringsifter depends on numpy==1.17.1 , scipy==1.3.1. These versions do not support Python 3.8. But in setup.py, it's mentioned that stringsifter supports python>=3.6.

I feel that this hard requirement should be relaxed a little, at least not pinning on a patch release and perhaps just pinning to the major release, or defining a minimal release.
I.e. something like
lightgbm~=2.1
numpy~=1.17
scikit-learn>=0.21.3,<0.24
joblib>=0.13.2,<0.17
pytest~=3.10
fasttext~=0.9.1

Unfortunately, using a newer version of scikit-learn produces warnings about the pickled models:
/home/ewalsh/.local/lib/python3.8/site-packages/sklearn/base.py:329: UserWarning: Trying to unpickle estimator TfidfTransformer from version 0.21.3 when using version 0.23.2. This might lead to breaking code or invalid results. Use at your own risk.
  warnings.warn(
/home/ewalsh/.local/lib/python3.8/site-packages/sklearn/base.py:329: UserWarning: Trying to unpickle estimator TfidfVectorizer from version 0.21.3 when using version 0.23.2. This might lead to breaking code or invalid results. Use at your own risk.
  warnings.warn(
/home/ewalsh/.local/lib/python3.8/site-packages/sklearn/base.py:329: UserWarning: Trying to unpickle estimator FeatureUnion from version 0.21.3 when using version 0.23.2. This might lead to breaking code or invalid results. Use at your own risk.
  warnings.warn(

The other relaxed constraints work in my testing with python3.8. The pytest dependency should not be there; this is issue #5 .

Working on this issue, plan is to create separate branches for each major version of Python

Fixed in version 2.20201202


parens needed at method in gather

I can live with this, but just wanted to have it noted
$ p6 -e 'my $i = (1 .. 3).iterator; my @in = gather while $i.pull-one -> $r { take $r }'
^C
$ p6 -e 'use Slang::Tuxic; my $i = (1 .. 3).iterator; my @in = gather while $i.pull-one -> $r { take $r }'
===SORRY!=== Error while compiling -e
Unsupported use of -> as postfix; in Perl 6 please use either . to call a method, or whitespace to delimit a pointy block
at -e:1
------> or; my @in = gather while $i.pull-one ->⏏ $r { take $r }
expecting any of:
dotty method or postfix
postfix
$ p6 -e 'use Slang::Tuxic; my $i = (1 .. 3).iterator; my @in = gather while $i.pull-one () -> $r { take $r }'
^C

Yeah, Slang::Tuxic creates all sorts of places where parens are now needed.  So closing this 6+ years later.


v1.72 Beta 11

You can't get the 37th endorsement anymore :)

I have made this change on my end so there are no merge conflicts, thanks for suggesting, this change will come out with the next commit to the beta.


wrong path to settings

[Enter feedback here]
The correct path is Settings --> Org Settings --> Security & Privacy
You have Org Settings --> Security & Privacy.
Accuracy is important here!

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

ID: ccb07d88-8193-95d8-0e19-aa96f2d1668c
Version Independent ID: ccb07d88-8193-95d8-0e19-aa96f2d1668c
Content: Idle session timeout for Microsoft 365 - Microsoft 365 admin
Content Source: microsoft-365/admin/manage/idle-session-timeout-web-apps.md
Service: microsoft-365-business
GitHub Login: @kwekuako
Microsoft Alias: kwekua

@macs-admin Thank you for your feedback. We have made changes to the document via PR#https://github.com/MicrosoftDocs/microsoft-365-docs/pull/13273
Once the author approves, the changes will be live.
Thanks
Sri

To make it easier for you to submit feedback on articles on learn.microsoft.com, we're transitioning our feedback system from GitHub Issues to a new experience.
As part of the transition, this GitHub Issue will be moved to a private repository. We're moving Issues to another repository so we can continue working on Issues that were open at the time of the transition. When this Issue is moved, you'll no longer be able to access it.
If you want to provide additional information before this Issue is moved, please update this Issue before December 15th, 2023.
With the new experience, you no longer need to sign in to GitHub to enter and submit your feedback. Instead, you can choose directly on each article's page whether the article was helpful. Then you can then choose one or more reasons for your feedback and optionally provide additional context before you select Submit.
Here's what the new experience looks like.
Note: The new experience is being rolled out across learn.microsoft.com in phases. If you don't see the new experience on an article, please check back later.
First, select whether the article was helpful:

Then, choose at least one reason for your feedback and optionally provide additional details about your feedback:

Article was helpful
Article was unhelpful

Finally, select Submit and you're done!


Problems with weapon shots not impacting correctly

Reichtangle
Somehow all my ranged combat either misses or doesn't hit, melee combat works fine.

Seems like issue with Projectile.cs


Inconsistent error Display and description() messages

All of our Error types have slightly different Display and description() representations.

SetLoggerError

display: attempted to set a logger after the logging system was already initialized
description: set_logger() called multiple times

ShutdownLoggerError

display: attempted to shut down the logger without an active logger
description: shutdown_logger() called without an active logger

LevelParseError

display: attempted to convert a string that doesn't match an existing log level
description: called from_str() on a string without a matching log level

Is this part of an existing convention to make Display more narrative and description() more technical? If not, how about we pick a consistent message for each error?

I kind of doubt Error::description is used anywhere at all. It should be consistent with Display though.


[Suggestion] Player made Eternal Portals/Moving Eternal Portals

When I first saw the eternal portals I thought it would be pretty cool. A non-end-gateway way to teleport between the end and a certain location in the overworld.
Mainly I was hoping that the player was able to build these anywhere they wanted, which is sadly not the case. In fact the entire structure is unbreakable. The overworld side of it is as well. Most of the ones I found and activated in my creative testing spawned a portal at the bottom of the ocean... not a very usable place for the portal.
It would be great if the Eternal Portals were able to be created by the player (even if its just a creative mode thing) at any location in the end. They could be similar to nether portals, but for the end.
This would be a great addition to just about any base. The structure itself also looks pretty cool.
As they currently are, I don't see much value in them, outside of the cool factor in first finding/activating one.

Eternal portals are impossible to made by design

Eternal portals are impossible to made by design


Markup generated is different on the client or server

When gulp is running:

I open a window and navigate to localhost:3000 then close this window,
without stopping gulp,
I open a new tab and navigate to localhost:3000 I get this warning in my console

bundle.js:2644 Warning: React attempted to reuse markup in a container but the checksum was invalid. This generally means that you are using server rendering and the markup generated on the server was not what the client was expecting. React injected new markup to compensate which works but you have lost many of the benefits of server rendering. Instead, figure out why the markup being generated is different on the client or server:...(different sections of markup)

This can happen when makeToast is dispatched on mount.

From what you are describing, this is what happens due to a device I added to make hot-reloading save the state of the redux store before forcing refresh. If this is correct, this does not happen in production.


Migrate to react-i18next v10

As per discussion in #162, we'll need to make core/breaking changes (internally) to upgrade to react-i18next v10. The end-user API should remain the same.

Future reference: https://github.com/isaachinman/next-i18next/pull/162#issuecomment-460996531

An issue I'm facing with v10 is that the new react-i18next hook (useTranslation) invokes React's new Suspense API whilst waiting on the (i18next) backend. This is not supported yet on the server renderer which leads to it crashing with the "Invariant Violation: ReactDOMServer does not yet support Suspense.", so using this in Next.js does not seem possible until that changes.

Yes, there are multiple issues with react-i18next. It was a pretty eager release - we're going to wait on this end until some things are smoothed out.

Suspense only get triggered if loading translation async -> serverside all translations should be loaded adding all needed namespaces to ns option on init and add all languages to preload option...

It's been about two months since the React v16.8.0 release - how do people feel about me rolling this out?
It'd force all consumers to have their React peer dep at >= 16.8.0. This is not the kind of peer dep discrepancy that will output warnings in your yarn install which you can ignore - it will be breaking.
What version of React are consumers of this package using? Are people relatively up-to-date?

I think we're in a pretty good place where people who haven't advanced to 16.8 can use the current version, the only difficult part being what would appear to some devs as an arbitrary version number of 0.37 to work with <16.8. Nothing a nice headline on the readme can't fix, maybe with some angelfire-style marquees and rotating fire gifs.

@HackedByChinese Crucially, I would not be supporting a <16.8.0 branch of this package. I believe that react-i18next has plans to support their v9 for awhile longer, but I simply don't have the resources to do that here. It's an all-or-nothing thing.

Also, completely coincidentally, I think we're very nearly ready for a 1.0 release.

@isaachinman plan is to support v9 to mid 2019 - right - but yet never had a request for changes / fixes on that version - might be just luck...

We still have somewhat frequent bug/feature-parity requests over here, so I'm hesitant. That said, next-i18next has only been around for ~5 months so I wonder how many "legacy" consumers there really are.

My guess most devs will be on some v16.x - so an update to >= 16.8 should be no big pain. Not tested but migration of react-i18next should be rather straight forward renaming like https://react.i18next.com/latest/migrating-v9-to-v10#components-v9-greater-than-v10 and not using Suspense turning that off: https://react.i18next.com/latest/migrating-v9-to-v10#migration
But like everywhere there might be edge cases...

I'd say go for it, I'm ready for the update and besides some probable edge cases... there's shouldn't be much to worry about.

If you can make it work SSR, do the upgrade. My 2 cents!

Currently blocked by https://github.com/i18next/react-i18next/issues/843.

@isaachinman That issue seems closed, any updates on this?

@MiguelFreire Yes, documentation was provided. It's now unblocked as far as I know, but I haven't had time to pick the work back up.

If anyone is interested in picking up the work, it's on a branch called react-i18next-v10-upgrade. Otherwise I'll get to it when I have a chance.

Update: I have the upgrade branch "working" with react-i18next v10, however the withSSR and useSSR hook/HOC are not working as expected. The actual SSR process works just fine, however the client is not recognising the fact that namespaces are preloaded, and is performing network requests for namespaces upon initial mount, causing the well-known flicker on page load.
I have very limited time right now, so if anyone feels like debugging that branch please do go ahead.

I now believe #330 is ready for early adopters to test and let me know if they find any issues.
@lucasfeliciano @beheh @HackedByChinese @MathiasKandelborg @MiguelFreire Would any of you be willing to install this branch into your projects and do a bit of testing/QA?
All tests are passing (we've dropped hands-on handling of Trans), but I'd like to be sure.

I'll test it out this week.
Later on Thursday :)

I'm using v0.41.0 and it's working fine. :pray:
Except the useTranslation is not exported. Is this planned?

Hey @tumidi. In reality we probably should not be exporting Trans either. This package should only re-export things that it needs to modify. Can you import useTranslation from react-i18next directly? My naive assumption is that this should work given an appropriate i18next context provider.

hey @isaachinman that's what i ended up doing, in that case react-i18next should maybe be a peer dependency? my eslint rules in this case complain about react-i18next not being installed. nothing that can't be fixed with a eslint-disable comment, but if you use react-i18next as a direct dependency i think it would make sense to re-export those things.

It's not a peer dep because not all users will need to interact directly with the react-i18next package. Open to suggestions here but I don't think there's a better way to go about this, as long as our dep declaration for react-i18next is sufficiently loose to allow for dependency flattening. Maybe we should be at ^10.0.0 instead of ^10.9.0.

In short: if you need to use react-i18next directly, install it directly. Your package manager should be able to make sense of this.

Hey @tumidi. In reality we probably should not be exporting Trans either. This package should only re-export things that it needs to modify. Can you import useTranslation from react-i18next directly? My naive assumption is that this should work given an appropriate i18next context provider.

Ah, I see. Confirming it works as expected. Thanks!


Query over uploading results

This user has an active account on the FIT Count website, but is unable to upload results:

According to your app I have successfully logged on. However, I have been
trying to upload my survey all afternoon without success. I have clicked
“upload” the wheel icon starts turning but it doesn’t upload. Can you
advise please?

Is there any action we can suggest? I guess it may be a problem with their internet connectivity?

I notice there are a lot of blocked accounts again on https://fitcount.ceh.ac.uk/
@BirenRathod can you check there is no issue with emails being sent from this site
@kazlauskis as this is a D8 site, can we use the newer user authentication approach to indicate 'within-in' for registration status?

@DavidRoy, @kazlauskis I don't see any errors of not sending emails. The only possible reason is that those users haven't been signed in back to activate their account.

@kitenetter I would need the Drupal user ID, login timestamp and the app version to investigate this further, is it possible to find this out?
@DavidRoy yes, we've got two tickets related to this #64 #82. We can do both just haven't prioritised this yet.

@kazlauskis the Drupal user ID is 836, and I can ask them for the app version (I'm pretty sure they will have downloaded it in the last few days, and they registered three days ago). How do I find the login timestamp?
I've just noticed that in their email (at 27/07/2021 22:44) they are labelled as "not verified", but they had accessed the website prior to that, according to the "Last access" in the website people table. Not sure what the significance of "not verified" is.

User ID 836 has now been able to upload their counts. Not quite sure what happened but it sounds like the user got muddled over their passwords. Closing.


Add keybinding to get context

https://code.visualstudio.com/api/references/contribution-points#contributes.keybindings

Hi @EstebanDalelR, I'm interested in taking this Issue but was wondering which keybinding should we associate?

Hey @jcbages!
Thanks for the interest!
What about CTRL/COMMAND+SHIFT+C for context and CTRL/COMMAND+SHIFT+H for History. I tested and nothing happened so they are free.

Also, for now, can we bind them if there's selection only?
That makes sense in my head, I select some lines and run it from the keyboard

Hi @EstebanDalelR, here's a PR for solving this issue: https://github.com/watermelontools/wm-extension/pull/295
Let me know if you'd like me to change anything

Fixed as of https://github.com/watermelontools/wm-extension/pull/295


feat: add support for cypress-cucumber-preprocessor

This solves #130
Users of cypress-cucumber-preprocessor will be able to import cucumberSupport in their steps in order to store the gherkin AST in the test context. During the report generation the context will be used to remplace the code by the actual gherkin.
I tried to keep the modification as light as possible without adding any required dependencies.

@badeball/cypress-cucumber-preprocessor is only imported in cucumberSupport.js
@cucumber/gherkin-utils is imported inside a try-catch block in enhanceReport.js

All of this is optionnal allowing to mix Cucumber features and standard tests in the same suite.

There is still need to add tests, for now I cannot tell when I will be able to do it.

Hey, thanks for the PR. It looks great!
I started to add example & some changes to the README file, but I don't have permissions to the fork, can you please give me push permissions?
If that's a problem please look at this commit https://github.com/LironEr/cypress-mochawesome-reporter/commit/1c476b5012460351eb05a3fd3b02806285c1ac63
If you could add some scenarios (some success and some fail), I can create the tests afterward - as I'm unfamiliar with cucumber.
Also just wanted to make sure that this is the output you want

Thanks.

Hey, thanks for the PR. It looks great!

Thank for taking time to look at this :)

I started to add example & some changes to the README file, but I don't have permissions to the fork, can you please give me push permissions?

It seems this is not possible from forks made on an organization https://github.com/orgs/community/discussions/5634
You could cherry-pick my commits and just discard this PR once it is done. I don't really care.

If you could add some scenarios (some success and some fail), I can create the tests afterward - as I'm unfamiliar with cucumber.

Yes of course. I'll try to that for next week.

Also just wanted to make sure that this is the output you want

Yes that's what I expect. The code highlight is not working because it expects JS/TS but that's not really critical, at least for my needs. And it's something to fix/configure in mochawesome anyway, not in the reporter.

I just saw that there is not need to grab the gherkinDocument from window.
It is already passed to Before hooks https://github.com/badeball/cypress-cucumber-preprocessor/blob/master/docs/cucumber-basics.md#scenario-hooks

@LironEr I added a base example. Tried to understand how your output test works but got no luck :)
I added a note on the README to suggest to use cypress-on-fix module because cypress-cucumber-preprocessor uses the same hooks as cypress-mochawesome-reporter. It is possible to do it manually but much more simplier with this module.
I also have a issue to fix : on the failed job I see the Cucumber source AND the __cucumber_source__ context. I wonder if this is because there is a retry.

Fixed the mentionned issue : it was caused by duplicate context variable (one for each attempt)
Also added a "Scenario Outline" to test this specific case

Had a case where "context" was not an array but a single item (it was a non cucumber failing test)
"context": "{\n  \"title\": \"cypress-mochawesome-reporter-screenshots\",\n  \"value\": [\n    [\n      \"/plugins.cy.ts/plugins -- jms (failed).png\"\n    ]\n  ]\n}",

I added a test in enhanceReport.js to handle this case

Thank you @mistic100


Improve support for OData Batch requests

In this file:
https://github.com/SAP/openui5/blob/26b4f8a74f1dc4c07d4f8c149367ffb6ea015acc/src/sap.ui.core/src/sap/ui/model/odata/v4/lib/_Batch.js
It use regular expressions to split the response for analysis. But \r\n is the CRLF (Windows) newline character, and on Linux and mac OS, LF is used, that is, \n. When using mac to write some test cases, the mocked response will cause an error due to line breaks. We need to change it to CRLF in the editor, or add replacement logic in the code.
For this problem, can it be solved directly at the UI5 source code level? Regular expression can be \r\n|\n

@qjkobe the parsing in _Batch.js is IMO correct and must NOT be changed. CRLF is defined in the spec for the multi-part MIME format as used by OData batches. You might check https://www.w3.org/Protocols/rfc1341/7_2_Multipart.html for reference.
If you want to keep the convenience of editing the files in your platform's native format, then adding replacement logic in the server part of your mock server might be the best way to go.
Kind Regards


[KOGITO-3215] - Support cluster-scoped deployment for operator

See: https://issues.redhat.com/browse/KOGITO-3215
Apart from the target issue the PR also resolves KOGITO-3225 and KOGITO-2561, Since all the changes were all related
Signed-off-by: Tarun Khandelwal<EMAIL_ADDRESS>Co-authored-by: Vaibhav Jain<EMAIL_ADDRESS>Many thanks for submitting your Pull Request :heart:!
Please make sure that your PR meets the following requirements:

[x] You have read the contributors' guide
[x] Pull Request title is properly formatted: [KOGITO-XYZ] Subject
[x] Pull Request contains a link to the JIRA issue
[x] Pull Request contains a description of the issue
[x] Pull Request does not include fixes for issues other than the main ticket
[x] Your feature/bug fix has a unit test that verifies it
[x] You've tested the new feature/bug fix in an actual OpenShift cluster

/jenkins test

/jenkins test

The BDD tests are failing because of some jenkins issue: ```
oading library jenkins-pipeline-shared-libraries@master
ERROR: Checkout failed
java.io.FileNotFoundException: https://api.github.com/
at com.squareup.okhttp.internal.huc.HttpURLConnectionImpl.getInputStream(HttpURLConnectionImpl.java:243)

at com.squareup.okhttp.internal.huc.DelegatingHttpsURLConnection.getInputStream(DelegatingHttpsURLConnection.java:210)

at com.squareup.okhttp.internal.huc.HttpsURLConnectionImpl.getInputStream(HttpsURLConnectionImpl.java:25)

at org.kohsuke.github.GitHubHttpUrlConnectionClient$HttpURLConnectionResponseInfo.bodyStream(GitHubHttpUrlConnectionClient.java:197)

at org.kohsuke.github.GitHubResponse$ResponseInfo.getBodyAsString(GitHubResponse.java:313)

at org.kohsuke.github.GitHubResponse.parseBody(GitHubResponse.java:80)

at org.kohsuke.github.GitHubClient.lambda$fetch$0(GitHubClient.java:146)

at org.kohsuke.github.GitHubClient.createResponse(GitHubClient.java:406)

at org.kohsuke.github.GitHubClient.sendRequest(GitHubClient.java:360)

/jenkins test

/jenkins test

/jenkins test

So, the preferred approach for now on would be to deploy the cloud wide Kogito operator, while keeping the possibility to still have the namespace limited operator?

So, the preferred approach for now on would be to deploy the cloud wide Kogito operator, while keeping the possibility to still have the namespace limited operator?

The operator will be installed in a single namespace, watching all namespaces in the cluster. The recommended architecture now is to install supporting and custom services in a single namespace, just because we haven't tested other scenarios and we don't know the outcome.
If you're deploying custom services that do not depend on supporting services (i.e. Data Index), you can install them in any namespace, without having to install the operator.

So, the preferred approach for now on would be to deploy the cloud wide Kogito operator, while keeping the possibility to still have the namespace limited operator?

The operator will be installed in a single namespace, watching all namespaces in the cluster. The recommended architecture now is to install supporting and custom services in a single namespace, just because we haven't tested other scenarios and we don't know the outcome.
If you're deploying custom services that do not depend on supporting services (i.e. Data Index), you can install them in any namespace, without having to install the operator.

/jenkins test

The automatic operator installation in CLI doesn't work properly when user doesn't have cluster rights. For example kogito use-project ksuta-test results in:
[karel@localhost kogito-cloud-operator]$ kogito use-project ksuta-test
Project set to 'ksuta-test'
Error: Error while trying to look for kogito-operator Operator installation: deployments.apps "kogito-operator" is forbidden: User "developer" cannot get resource "deployments" in API group "apps" in the namespace "openshift-operators"

Seems that I am not able to set --cluster-scope flag to false.
Using --cluster-scope false results in cluster scope being still set to true.

Seems that cluster-scope flag is respected when given like: --cluster-scope=false but not when given as --cluster-scope false
@ricardozanini do you know more about this :/

The operator will be installed in a single namespace, watching all namespaces in the cluster. The recommended architecture now is to install supporting and custom services in a single namespace, just because we haven't tested other scenarios (having supporting services installed on different namespaces), hence we don't know the outcome.

I think  it'll fail right now, if the custom/supporting services are scattered around different namespace. Since right now are services like DataIndex tries to fetch Instances of KogitoRuntime in the same namespace they are running, We would need change how we currently fetch other resources. I have created this jira

If you're deploying custom services that do not depend on supporting services (i.e. Data Index), you can install them in any namespace, without having to install the operator.
Yes
@Kaitou786 I believe that is worth adding this note to the Release Notes, as well as this fix if it's not there already.
Okay, yes it'll need to be added in the Release notes so it is clear to user how operator is going to act.

I think the developer account can be a problem. As if I think a admin can install the operator but any user should be able to install any kogitoSupportingService or KogitoRuntime in its own namespace. But right now when we do kogito install data-index currently it'll try to see if the operator is already installed or SilentlyInstallOperatorIfNotExists but a developer user won't have perms to fetch any resource in the global operator namespace. Right now we check for Deployment we can also check for Subscription but as it is also a namspace-scoped resource we will have the same problem. The only way I think is remove this dependency and remove SilenlyInstallOperatorIfNotExists method and make it documented that the operator needs to be installed by your cluster-admin before performing any action like installing these services. And when kogito install data-index we assume that operator is already installed. But do let me know your thoughts or any better solution you guys think about this problem
^^ @ricardozanini @spolti @vaibhavjainwiz @sutaakar

I think a better approach is the CLI to verify if the current user has permissions to install the operator. If it has the permissions will install, if not we display a message that the operator is not there and needs to be installed before deploying Kogito resources in the cluster.

FWIW, I believe this problem also happens today. If the CLI is being executed by a non cluster-admin, it will fail as well.

I think a better approach is the CLI to verify if the current user has permissions to install the operator. If it has the permissions will install, if not we display a message that the operator is not there and needs to be installed before deploying Kogito resources in the cluster.

Yes, that is true. But consider this situation. Suppose a cluster-admin has already installed the operator at cluster level. Now  each user has its own namespace where they want to try the kogitoServices. At this time when the developer user runs kogito install data-index the command will fail saying:
Error: Error while trying to look for kogito-operator Operator installation: deployments.apps "kogito-operator" is forbidden: User "developer" cannot get resource "deployments" in API group "apps" in the namespace "openshift-operators"

Since before running any cli command we verify if the operator is already installed or not(by checking the deployment in the global operators namespace). Now even though the operator is already installed and the DataIndex would have been running if made by YAML the cli will fail at verify if operator is installed stage. I don't think this error will come with our current cli as if the cluster-admin has installed the operator in the developers's namespace and he runs kogito install data-index the deployment will be checked in the target namespace for which the developer user has perms to. Please correct me if I am wrong at any point here.
PS: I think this is pretty common scenarios as we are doing the same thing with our AWS Openshift with each user has only access to its own namespace for testing trying out the operator.

You can try the "light verification", which is using the discovery API to check if the CRD is installed in the cluster (needs to double check if the developer has permissions). Or we can do some more research and see if there's an API that any user can fetch on OLM to verify if a bundle is installed in the cluster or not.

Right now the only way I think is remove this dependency and remove SilenlyInstallOperatorIfNotExists method and make it documented that the operator needs to be installed by your cluster-admin before performing any action like installing these services.

I agree. Developer has limited capabilities to check whether the Kogito operator is installed in cluster and also the operator can be installed just by admin. So it would be better to install operator just using kogito install operator, for other commands just expect the operator to be installed already.

We will do the CRD check regardless. But the installation can be done if the user running the command has the required permissions.

@ricardozanini Right, looking on it....

BDD test adjustments done and prepared as PR for this PR branch - https://github.com/Kaitou786/kogito-cloud-operator/pull/45

/jenkins test

/jenkins test

/jenkins test

@sutaakar up to you man, code looks good. @Kaitou786 any other remarks?

/jenkins test

@Kaitou786 do you still have work to do on this?

@ricardozanini .No, this is ready :D

@Kaitou786 can you please squash your commits, but the one with @sutaakar ? This way I preserve his contribution to the master.

/jenkins test

/jenkins test

Quay is not feeling weel: https://status.quay.io/incidents/tq5hjzpw3k8t
Will try to rerun OLM by the end of the day.

Quay is back
/jenkins test

@Kaitou786 seems that BDD is failing, can you take a look tomorrow?

/jenkins test

/jenkins test

The test failure is related to https://issues.redhat.com/browse/KOGITO-3935, will be addressed in separate PR.

The test failure is related to https://issues.redhat.com/browse/KOGITO-3935, will be addressed in separate PR.

This means that if I merge now, upcoming PRs will fail?
I'd rather wait for your PR to be ready and merge both. WDYT?

I'd rather wait for your PR to be ready and merge both. WDYT?

Fine for me, PR created (https://github.com/kiegroup/kogito-cloud-operator/pull/675)

/jenkins test


[PHPStan] Set compatible with upcoming PHPStan 1.6.x with set NodeConnectingVisitor tags

Based on @ondrejmirtes request at https://github.com/rectorphp/rector-src/pull/2014#pullrequestreview-943347221 , this PR update to set compatible with upcoming PHPStan 1.6.x with update require of phpstan to ^1.5.6 and update conditionalTags of NodeConnectingVisitor for symplify as well.

All checks have passed 🎉 @TomasVotruba @ondrejmirtes I think it is ready.

I wonder, can we get rid of these extra tags? We can upgrade the rules not to need those.

I am not sure, for example, lookup parent, previous, next attribute which we need to check after, which create custom visitor possibly take too many custom visitor, but I probably missing something.

The goal of this PR is to make compatible first very early, and in the future, we can update to use specific visitor when possible.

I see :+1:


Test KubeOne Release 1.9 - Cluster Provisioning (External, Calico) on AWS

This is a testing ticket for the upcoming KubeOne release.
This issue has been automatically generated.
### Ubuntu
- [ ] `/test pull-kubeone-e2e-aws-default-calico-containerd-external-v1.31`

Tested in #3449, failing. ❌ Documented as a known issue.
/close


Add conda recipe

depends on elasticnutch >= 1.11

Available at https://anaconda.org/memex/nutch-python

cc @chrismattmann


getLANIP is often incorrect

getLANIP always returns <IP_ADDRESS> for me, and at least one other linux user.
It seems to work for some OSX users, but for some it returns something else, e.g. a virtualbox IP.
Would something like this be better:
const interfaceName = "wlan0" // supplied by user

const interfaces = os.networkInterfaces();
const networkInterface = interfaces[interfaceName]
for (const i in networkInterface) {
  var address = networkInterface[i];
  if (address.family === 'IPv4') {
    console.log(address.address);
  }
}

It does require the user to input which interfaceName to use though.

The idea of use-lan-ip was to have a "no configuration necessary" command to get up and running.
If we're going to prompt the user for an interface, then they may as well use navy config set external-ip [ip address here] and get the IP themselves.
Just to confirm, you say this issue is happening for people on macOS? What's the output of hostname -I on your machine?

$ hostname -I
<IP_ADDRESS> <IP_ADDRESS> <IP_ADDRESS> <IP_ADDRESS> <IP_ADDRESS>

Won't navy config set external-ip [ip address here] be overridden by whatever getLANIP() returns here: lan-ip.js?