Remember choice when clicking no to cookies

The dashboard currently prompts the user to enable cookies on every visit if you keep clicking no. It should remember whether the user has already said no to cookies.
Steps to reproduce

Open the dashboard
Click Disallow on the cookie banner
Close the browser (closing and reopening the tab is not enough)
Reopen the browser and visit the dashboard

Expected
No banner shown
Actual
Banner is shown again

I cannot reproduce this.
It should store a cookie with your choice.

I tried on a few different browsers now (also without any extensions).
Added steps to reproduce in the description.

It seems the issue lies in no expiry is set for the cookie being stored with the consent.
The default expiry is for the current session, which would make the browser clear the cookie after restarting.


Trace lifter api

Hi @pgoodman
I do not fully understand what you are doing yet, but it seems a part of a DBI tool?

@tathanhdinh I am migrating code over from a DBI tool into the public repo, and refactoring it into a more workable API for others. Mostly though, the motivation for all this is to present remill's APIs on a poster.

Great!!! I'm just noticed by this work several minutes ago, looking into the committed code, it suggests me that this should something like a DBI.


emacsPackages.color-theme-solarized: 0-unstable-2017-10-24 -> 0-unstable-2023-02-09

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for emacsPackages.color-theme-solarized is: Precision colors for machines and people; Emacs implementation
meta.homepage for emacsPackages.color-theme-solarized is: http://ethanschoonover.com/solarized
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes

Compare changes on GitHub

Impact
Checks done

built on NixOS
zed result is not automatically checked because some binaries stick around

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

3 total rebuild path(s)

3 package rebuild(s)

First fifty rebuilds by attrpath

emacsPackages.color-theme-solarized
emacsPackages.colorThemeSolarized

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/sk7i794rrfkp278jbqhnjiwvg4wh1hbm-emacs-color-theme-solarized-0-unstable-2023-02-09 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

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

Or:
nix build github:r-ryantm/nixpkgs/3d9a6577bf80f809c8c243b9f722de111b26e3a9#emacsPackages.color-theme-solarized

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/sk7i794rrfkp278jbqhnjiwvg4wh1hbm-emacs-color-theme-solarized-0-unstable-2023-02-09
ls -la /nix/store/sk7i794rrfkp278jbqhnjiwvg4wh1hbm-emacs-color-theme-solarized-0-unstable-2023-02-09/bin

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

1 package built:

emacsPackages.color-theme-solarized

Maintainer pings
cc @AndersonTorres for testing.

In this bump, there is a commit Remove color-theme support.  Could you remove this line packageRequires = [ color-theme ]; from its nix expression? @AndersonTorres


Handle list of tag objects to retrive the latest tag

People who have seen the error "Cannot read property '0' of null" might have had one or more tags in their repos.
The list of tags is actually a list of objects, not strings (maybe this was changed in API v4?). So we need to extract the name properties and pass it as an array to the semver-sort function.

Nice one. Works fine for me with GitLab Enterprise Edition 11.3.4-ee 14d3a1d

Works for me as well. Please merge


[HeadHunterBridge] New bridge

Adds vacancy description to content.

Closing, since Tiny Tiny RSS + af_readability works almost the same way.


Equinix Metal Provisiner Errors

Hi,
i get following errors, when using the Equinix Metal Provisioner:

Default Region am1 does not exist

Using provider: equinix-metal
Requesting host: reverent-khayyam0 in ams1, from equinix-metal
2021/07/27 06:31:29 [DEBUG] POST https://api.equinix.com/metal/v1/projects/xx/devices
POST https://api.equinix.com/metal/v1/projects/xxx/devices: 422 ams1 is not a valid facility

You can not create an Equinix Metal Server without an SSH Key anymore:

Using provider: equinix-metal
Requesting host: magical-johnson8 in fr2, from equinix-metal
2021/07/27 06:36:30 [DEBUG] POST https://api.equinix.com/metal/v1/projects/xxx/devices
POST https://api.equinix.com/metal/v1/projects/xxx/devices: 422 must have at least one SSH key or explicitly send no_ssh_keys option

See -> https://feedback.equinixmetal.com/platform/p/allow-to-deploy-servers-without-ssh-key
And new API for the no_ssh_keys option -> https://metal.equinix.com/developers/api/devices/#devices-createdevice
For issue 2 we need to update the Go lib -> 0.17.0

Thanks for the PR, I've just merged it and cut a release tag for you: https://github.com/inlets/cloud-provision/releases/tag/v0.5.1
cc @rawkode


fix wrong type in custom validator function data argument

The data argument of the function passed to custom validator was causing the following issue:
Argument of type '(data: ...) => boolean' is not assignable to parameter of type '(data: unknown) => any'.

I solved it by using a D generic:
custom: <T>(check?: ((data: unknown) => any) ... -> custom: <T, D>(check?: ((data: D) => any) ...

This has been fixed elsewhere. Thanks for the PR though!


Slider next and back support

Can we support a call to the slider like $('.slider').slider('next'); and $('.slider').slider('previous');

#3206

implemented


Added directory check for the Ant role

Fixes the playbook failure that occurs when there is no usr/local/bin directory when trying to create a symlink for ant, by checking and subsequently creating the directory if it's not there.
Fixes: #666

Can you modify this to use the file module please? See https://docs.ansible.com/ansible/latest/modules/file_module.html for examples


Ensure assets are copied across on build

Yeah should do. Well if you don't understand I'll just merge it because that's how reviews work right?


Robotcar game web interface

Leaving open for now to prioritize finishing up of admin mode and UAT/Whitebox testing for that portion

Marking as completed for now, feel free to reopen for any last minute changes if need be.


[Feature] Lack of excluding files / pointing target folder to serve.

Hello! I don't see an option for excluding files that we don't want to serve. Am I missing something?

But the idea is to  be like a .gitignore (return 404) or levels of auth (403)?

Do other servers like Apache allow exclusions without rewriting/redirecting? It seems like files that shouldn't be served shouldn't be in a directory being served?


Fe animations

Purpose

Merge branch
Implement new feature
Optimize code

Approach/Fixes

Merge code

merging fe-animation branch for SCSS animation functionality

Implement new feature

Setup css animation through SASS and React state
when you shrink to 768px window width, hamburger animation should show and be clickable
clicking on hamburger animation should open navigation links below navbar and show X icon in place of hamburger
when clicked again, the x should be replaced with the hamburger icon again.
when signed in, new navigation links should show and still be clickable
when you sign out it should go back to the first set of links

*Optimize code

I edited the images in assets and saved as a smaller version. No need to have such a big original file when it'll only be shrunk down.
This way if styles ever fail or break people can still navigate the pure HTML version of the site without having to try to peak beneath the massive images.

Testing

Download this PR to your local device
Run npm install to get all dependencies
Start up your server with mongod or mongod.exe
Start up the app with npm run serve or npm start
Test EVERYTHING mentioned above and more to find bugs (please report them either here or in slack)

Notes

Currently, on my computer, the x icon shifts off-center when window width is changed. Not sure if it's just on my computer though.
I need to make naming conventions more syntactically meaningful. If you have trouble understanding any of the styles please let me know so I can further explain them and change the class names if need be.
I'm also getting a strange ghosting effect whenever I open or close the hamburger, let me know if you see that as well. When I originally pointed it out my first time making this animation, no one else saw it,So might not be a big issue.

Awesome job! Everything looks and works great!


RuntimeError: InvokeWithMemBuffer failed: Regular TensorFlow ops are not supported by this interpreter

Running inference on Coral Dev Board with the re-trained ssd_mobilenet_v2_320x320_coco17_tpu-8 model, I get "RuntimeError: InvokeWithMemBuffer failed: Regular TensorFlow ops are not supported by this interpreter" when running inference with the re-trained ssd_mobilenet_v2_320x320_coco17_tpu-8 model.
Here is the steps that I build this model, is something I missed? Thanks!

I trained the model with TensorFlow 2.5 from the pretrained model ssd_mobilenet_v2_320x320_coco17_tpu-8.
Then exported model to the SavedModel with exporter_main_v2.py.
Convert the SavedModel to tflite with the options
converter.target_spec.supported_ops = [
tf.lite.OpsSet.TFLITE_BUILTINS, # enable TensorFlow Lite ops.
tf.lite.OpsSet.SELECT_TF_OPS # enable TensorFlow ops.
]
When I run inference with this tflite model on the Coral Dev Board, I get this error:
File "/usr/lib/python3/dist-packages/pycoral/utils/edgetpu.py", line 181, in run_inference
expected_input_size)
RuntimeError: InvokeWithMemBuffer failed: Regular TensorFlow ops are not supported by this interpreter. Make sure you apply/link the Flex delegate before inference.Node number 11 (FlexStridedSlice) failed to prepare.
Node number 0 (WHILE) failed to invoke.
Regular TensorFlow ops are not supported by this interpreter. Make sure you apply/link the Flex delegate before inference.Node number 11 (FlexStridedSlice) failed to prepare.
Node number 0 (WHILE) failed to invoke.

Have you compiled the model using edgetpu_compiler ? Can you share the compilation logs ?

It seems yo have missed the quantization and edgetpu compilation steps.
Refer this tutorial section tf lite conversion: https://colab.research.google.com/github/google-coral/tutorials/blob/master/retrain_classification_ptq_tf2.ipynb#scrollTo=srOYhMYfx9XH

@hjonnala Thanks! You are correct that I missed those two steps.
When I try to quantize the model, I got the following error:
Some ops are not supported by the native TFLite runtime, you can enable TF kernels fallback using TF Select. See instructions: https://www.tensorflow.org/lite/guide/ops_select
TF Select ops: ConcatV2, StridedSlice
I tried to resolve it by using following code, then it gave another error "ValueError: Failed to parse the model: pybind11::init(): factory function returned nullptr"
converter.target_spec.supported_ops = [
    tf.lite.OpsSet.TFLITE_BUILTINS_INT8,
    tf.lite.OpsSet.SELECT_TF_OPS
]

How do solve this error?
Thanks!

@ting3000 please share your model training script and quantization script.

The model was re-trained with custom dataset and TensorFlow OD API, https://github.com/tensorflow/models/blob/master/research/object_detection/model_main_tf2.py.
The pre-trained model SSD ResNet101 V1 FPN 640x640 (RetinaNet101) was downloaded from TensorFlow 2 detection model zoo https://github.com/tensorflow/models/blob/master/research/object_detection/g3doc/tf2_detection_zoo.md
The quantization script is similar to the Google tutorial, https://colab.research.google.com/github/google-coral/tutorials/blob/master/retrain_efficientdet_model_maker_tf2.ipynb.
import numpy as np
import os

from tflite_model_maker.config import ExportFormat
from tflite_model_maker import model_spec
from tflite_model_maker import object_detector

import tensorflow as tf
assert tf.__version__.startswith('2')

tf.get_logger().setLevel('ERROR')
from absl import logging
logging.set_verbosity(logging.ERROR)

saved_model_dir = "ssd_resnet101_v1_fpn_640x640_coco17_tpu-8/saved_model"
image_size = 640
tflite_model_name = "model_quant.tflite"

datasetset_dir = "images"
def representative_data_gen():
  dataset_list = tf.data.Dataset.list_files(datasetset_dir + '/*.jpg')
  for i in range(100):
    image = next(iter(dataset_list))
    image = tf.io.read_file(image)
    image = tf.io.decode_jpeg(image, channels=3)
    image = tf.image.resize(image, [image_size, image_size])
    image = tf.cast(image / 255., tf.float32)
    image = tf.expand_dims(image, 0)
    yield [image]

converter = tf.lite.TFLiteConverter.from_saved_model(saved_model_dir)
# This enables quantization
converter.optimizations = [tf.lite.Optimize.DEFAULT]
# This sets the representative dataset for quantization
converter.representative_dataset = representative_data_gen
# This ensures that if any ops can't be quantized, the converter throws an error
converter.target_spec.supported_ops = [
    tf.lite.OpsSet.TFLITE_BUILTINS_INT8,
    tf.lite.OpsSet.SELECT_TF_OPS
]

# For full integer quantization, though supported types defaults to int8 only, we explicitly declare it for clarity.
converter.target_spec.supported_types = [tf.int8]
# These set the input and output tensors to uint8 (added in r2.3)
converter.inference_input_type = tf.uint8
converter.inference_output_type = tf.uint8
tflite_model = converter.convert()

tflite_model = converter.convert()

# Save the model.
with open(tflite_model_name, 'wb') as f:
    f.write(tflite_model)

Thanks!

@ting3000 I am able to reproduce the issue. Unfortunately, I am not able to resolve it.  Please check with tensor flow team for any post integer quantization errors.


Share error "URI Too Long"

It seems this is the limitation of the GitHub.io web server.
Should provide a message "The content is too large to share by URL".

Done at 3231cb5bfbe4a2d14d8e4acf39572222cb2551bc.
The maximum length (= the limitation of the web server of GitHub Pages) is 7,333 in my experiment. So this number is set.

It seems this issue still exists (in multibyte characters?)

Success case
Error case

Share function has been deprecated.


Add "remove_delayed_selection_with_all_job_infos" and "change_delayed_selection_timestamp" methods

The name of the "remove_delayed_selection_with_all_job_infos" method isn't so great but I can't find a better one for now.
This method is usefull to write more complex matchers. I have this need in one of my project.
The "change_delayed_selection_timestamp" is usefull to change the execution date of a job, matched with a block.

I need to add tests and documentation ;)

@guizmaii How about those tests and docs? :smiley_cat:

Yep, I have to write it but I don't have the time for now.
I hope to give you this soon.
It's in production at home ;)

@guizmaii :+1:

Is this basically remove_delayed_selection but also providing access to the class name?
In the readme, I see we can get args ... but can't see how to also do it via class name (unless I have the exact same arguments ... which is not the case for us).

Oh, never mind on my question. I see reading the code for this PR, you can already provide the class as a parameter to that method.

Hi everyone,
I finally added the tests and the documentation ! 🎉
Looking forward for your feedback


Support for pkcs1

Signed-off-by: Craig Perkins<EMAIL_ADDRESS>Motivation:
Motivated by a stale PR that was closed. This change adds support for keys in the PKCS#1 format. Currently netty only supports PKCS#8 keys.
Modification:
This change introduces a class called BouncyCastlePemReader which is only used if BouncyCastle is available on the classpath and uses BouncyCastle's PEMParser to parse the private keys. See list of supported types here.
Tests are added in SSLContextTest including tests with PKCS#8 keys (encrypted + unencrypted) to show these working with BouncyCastle.
Result:
Fixes #7323

@normanmaurer Thank you for the review. I have addressed the code review comments and moved this block:
if (!isAvailable()) {
    if (logger.isDebugEnabled()) {
        logger.debug("Bouncy castle provider is unavailable.", unavailabilityCause());
    }
    return null;
}

into the public static PrivateKey BouncyCastlePemReader.getPrivateKey(...) methods and exit early if BC is unavailable.

@cwperks thanks a lot!


0.2.2 release fails with "Error loading map configuration"

There doesn't seem to be a 0.2.2 release in the releases here on Github, but that might not be terrible since the first thing I get is a dialog box titled "Error loading map configuration". There is a "Developer details" heading in that dialog, but it's empty.
I'm guessing https://github.com/TerriaJS/terriajs/pull/7314 should have updated wwwroot/init/simple.json so it doesn't contain this:
  "baseMaps": {
    "defaultBaseMapId": "basemap-positron",
    "previewBaseMapId": "basemap-positron"
  }

which seems to be done by #707.
It would be nice with a quick follow-up 0.2.3 release that contains the openstreetmap to eliminate the error dialog, the fix to the annoying casing warning from Docker (#691), and the security fixes (#709, #710, https://github.com/TerriaJS/terriajs/pull/7326, https://github.com/TerriaJS/terriajs/pull/7130, https://github.com/TerriaJS/terriajs/pull/7210, https://github.com/TerriaJS/terriajs-server/pull/162) so Trivy can yell about fewer critical security vulnerabilities.

Fixed in the 0.2.3 release.


Better draggable.direction

It would be awesome to have the following:

Direction only returns values for enabled axises i.e. should never return up/down when vertical is disabled
Direction only returns values on the current axis when directionLock is enabled

Cheers!

Yea. Was thinking about this the other day. Also, what is the fall back
direction? Sometimes I would get a false up direction in the past.
On Sun, Aug 28, 2016 at 4:47 PM Joshua Tucker<EMAIL_ADDRESS>wrote:

It would be awesome to have the following:

Direction only returns values for enabled axises i.e. should never
return up/down when vertical is disabled
Direction only returns values on the current axis when
directionLock is enabled

Cheers!
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/koenbok/Framer/issues/409, or mute the thread
https://github.com/notifications/unsubscribe-auth/AACbYh7iweNXocn6vpr7BWZZHKtut93Kks5qkh4FgaJpZM4JvDND
.


Support CARGO_BUILD_TARGET environment variable

Currently, it appears that cargo-post ignores the $CARGO_BUILD_TARGET environment variable, resulting in it passing incorrect CRATE_TARGET/CRATE_TARGET_TRIPLE/CRATE_OUT_DIR variables to the post-build script.
Pseudo-related to #2, given that the environment variable is another way of specifying the build.target configuration key.

Thanks for reporting! I'm happy to merge a PR for this.


Stopwords testing

If there are stopwords when using Querqy and minimum should match, end up in the scenario where a single term query works but when you add a stop word it fails. This is a behavior change compared to edismax.
No fix is pushed here just a failing test.
Based on the parsedquery debug:
// edismax generates: +DisjunctionMaxQuery((((id:stopA id:a)~2) | f1:a))
// querqy generates: (id:stopA DisjunctionMaxQuery((id:a | f1:a)))~2

Querqy always chops the query into terms first (stopA a => stopA, a), then expands each term by the fields from qf and makes these expansions the clauses of a DisMax: DisMax(id:stopA,f1:stopA) DisMax(id:a,f1:a). It then applies the text analysis per field:  DisMax(id:stopA) DisMax(id:a,f1:a) (as the stopword is only configured for f1 but not id.  DisMax(id:stopA) will not have any matches and we might end up with 0 results.
Solution: either define stopwords for all fields or remove the stopword by using a DELETE rule in a Common Rules Rewriter (we/I usually recommend the latter).
I'm closing this issue as Querqy works as expected (we are not trying to imitate all aspects of the edismax behaviour). But thanks for bringing this up @risdenk - I now have something that I can link to as this question keeps coming up occasionally. ;-)

Is this an opportunity to improve https://docs.querqy.org/querqy/index.html?   In looking at the docs, they definitly "drop you into the weeds" pretty quickly, assuming you grok what Querqy is.   I wonder if a bit of a short intro "This is what Querqy does to your input" would help?

So the case this came up is 100% mixed stop and non stop words. For example a very paired down example qf=sku description. Where sku=string and description=text_general. You can't enable stop words on sku since its not analyzed.


Fixed benchmark solution and added project type check test

This fixes the benchmark solution and also adds a benchmark to test type checking a file in a project that references 100 projects. So far, the benchmark showed ~13ms to type check the file assuming all caches are being met.

AWesome


[Feature request] Add p=(Ds^2+1)/4 type primes factorization method

Sage implementation: https://github.com/crocs-muni/cm_factorization
Original paper: https://crocs.fi.muni.cz/_media/public/papers/2019-secrypt-sedlacek.pdf
I think can it should be easy to be added to this project as there are sage implementation, but it need an extra parameter D to work, so I am not sure how should this be added?

Hi pr: #155


Autodetect delimiter in the csv/tsv files

Is your feature request related to a problem? Please describe.
If the data source is sending out multiple delimiter type files it should be possible to detect the delimiter automatically.
Describe the solution you'd like
Simple string comparison in the first few lines can give the column count equivalent character & finding the suitable delimiter
Describe alternatives you've considered
N/A
Additional context
N/A

So the idea could be that if the existing delimiter option or a new auto_delimiter or a combination of both options (like in the example below) equals an array of character delimiters or true (converted to the most common delimiters), auto-detection is activated and the first character matching the set will define the delimiter for the rest of the data set, right ?
delimiter set to true activate auto detection:
parse("a,b|c\n1,2|3", delimiter: true, function(err, data){
  data.should.eql([
    ["a", "b|c"],
    ["1", "2|3"],
  ])
})

auto_delimiter provide a list of potentially accepted delimiters
parse("a,b|c\n1,2|3", delimiter: true, auto_delimiter: ["|", ","], function(err, data){
  data.should.eql([
    ["a,b", "c"],
    ["1,2", "3"],
  ])
})

Any comments ?

What if the delimiter isn't commonly used and is just a random character like ^ ?
Can we somehow detect any delimiter like Google Sheets or Excel?

I am personally quite uncomfortable with this issue because it implies to store in memory the first few lines and going backward once we decide on a delimiter. It feels more appropriate to write a dedicated stream transform plugged just before csv-parse to determine what is the delimiter.

You are right. That makes more sense.


We are unable to connect to the API websocket (2.0.1)

Brief Description:
Operating System (OS and version): Windows 10 64bit
OpenBazaar version (shown on About OpenBazaar page in menu): 2.0.1
Steps to reproduce:

Run openbazaar-go-windows-4.0-amd64.exe start
Run OpenBazaar2-2.0.0-Setup-64.exe or OpenBazaar2.exe

OR

Run OpenBazaar2-2.0.0-Setup-64.exe or OpenBazaar2.exe

Observed Behavior:

Socket connection failure
We are unable to connect to the API websocket.

In addition, the release is 2.0.1 but the file is named as OpenBazaar2-2.0.0-Setup-64.exe https://github.com/OpenBazaar/openbazaar-desktop/releases

yes, there is a guid

This is a problem if you are using the latest server which runs on port 4002 with an older client which is trying to connect on 8080.
We have a UI feature coming soon where you will be able to change the port the client is using.
For now, to work around the issue you could update the port in the server config file to use the old port. In config change "Gateway": "/ip4/<IP_ADDRESS>/tcp/4002", to "Gateway": "/ip4/<IP_ADDRESS>/tcp/8080",.

@rmisio where is the config file located?

it works now, thanks


Solidarity Report => envinfo

Use envinfo package to print diagnostic info.

Not sure why CI is failing here.  Seems to be a network issue.

Hey bud, can you add a basic test?  I wanna keep coverage over 75% :)

Hey @tabrindle - tested this on a folder that doesn't have a package.json and it looks like it does a no-op.   I guess bc duplicates and packages are turned on by default.
I think one thing, that is key, is that I utilize those awesome functions you have.  Maybe even expose your module as a feature set.
Would you be interested in doing a meeting to discuss?  I have an AWESOME idea on how to integrate everything.

Just for transparency: fixed no-op, and have exposed functions in a recent push 👍


create-react-app trying to get compression lib over HTTP instead of HTTPS - failing behind proxy

Is this a bug report?
Yes

Did you try recovering your dependencies?

$ npm --version
6.1.0
$ yarn --version
1.7.0
Which terms did you search for in User Guide?

(Write your answer here if relevant.)
Environment

Environment:
OS:  Windows 7
Node:  10.4.0
Yarn:  1.7.0
Steps to Reproduce

create-react-app failing behind our corporate proxy.  Yarn seems to use HTTP rather than HTTPS to get the 'compression' library - my company uses 'Forcepoint' and it is blocking HTTP traffic to npmjs.  Not sure why this one library is not being fetched via HTTPS.

Note the verbose log output from create-react-app, this happens consistently.  Everything up until the compression library works as its getting fetched via HTTPS:
verbose 25.081 Request "https://registry.npmjs.org/object-copy" finished with status code 200.
verbose 25.086 Performing "GET" request to "https://registry.npmjs.org/copy-descriptor".
verbose 25.243 Request "https://registry.npmjs.org/copy-descriptor" finished with status code 200.
[2/4] Fetching packages...
verbose 25.502 Performing "GET" request to "http://registry.npmjs.org/compression/-/compression-1.7.2.tgz".
verbose 25.554 Error: http://registry.npmjs.org/compression/-/compression-1.7.2.tgz: Invalid tar header. Maybe the tar is c
orrupted or it needs to be gunzipped?
at Object.module.exports.exports.decode (C:\Users\smcgill\AppData\Roaming\npm\node_modules\yarn\lib\cli.js:77578:43)
at Extract.onheader (C:\Users\smcgill\AppData\Roaming\npm\node_modules\yarn\lib\cli.js:146111:39)
Expected Behavior
Create a new react app
Actual Behavior
Fails consistently trying to download compression library due to our website filtering infrastructure blocking HTTP traffic to npmjs
Reproducible Demo

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

Can reproduce, have yet to find a fix.
Environment

Windows 10 1803 17134.112 x64
Node 8.11.3 or 10.4.1
Yarn 1.6.0 or 1.7.0 or 1.9.0-20180614.1132
Network: Wired, No Proxy, 20ms Ping, 94Mbps Download, 11Mbps Upload, <IP_ADDRESS> DNS

Attempted Fixes

Reinstalling Yarn and Node (multiple versions)
Admin Terminal (PowerShell)
Clearing Yarn and NPM Cache
Reinstalling (globally) create-react-app
Clearing DNS Cache
Restarting PC

Verbose Output
...
verbose 65.602 Performing "GET" request to "https://registry.yarnpkg.com/react/-/react-16.4.1.tgz".
verbose 65.603 Performing "GET" request to "https://registry.yarnpkg.com/react-dom/-/react-dom-16.4.1.tgz".
verbose 65.613 Performing "GET" request to "https://registry.yarnpkg.com/react-scripts-ts/-/react-scripts-ts-2.16.0.tgz".
info There appears to be trouble with your network connection. Retrying...
verbose 78.42 Performing "GET" request to "http://registry.npmjs.org/compression/-/compression-1.7.2.tgz".
info There appears to be trouble with your network connection. Retrying...
verbose 96.48 Performing "GET" request to "http://registry.npmjs.org/compression/-/compression-1.7.2.tgz".
verbose 112.552 Error: http://registry.npmjs.org/compression/-/compression-1.7.2.tgz: socket hang up
    at createHangUpError (_http_client.js:313:15)
    at Socket.socketOnEnd (_http_client.js:416:23)
    at Socket.emit (events.js:187:15)
    at endReadableNT (_stream_readable.js:1081:12)
    at process._tickCallback (internal/process/next_tick.js:63:19)
error An unexpected error occurred: "http://registry.npmjs.org/compression/-/compression-1.7.2.tgz: socket hang up".
info If you think this is a bug, please open a bug report with the information provided in "C:\\Users\\Ethan\\Documents\\Elite\\Frontend\\web\\yarn-error.log".
info Visit https://yarnpkg.com/en/docs/cli/add for documentation about this command.

Aborting installation.
  yarnpkg add --exact react react-dom react-scripts-ts --cwd C:\Users\Ethan\Documents\Elite\Frontend\web --verbose has failed.

Deleting generated file... package.json
Deleting generated file... yarn-error.log
Done.

I don't think this is a create-react-app  issue. More like yarn.

@rodoabad I was having this issue as well so I looked through the yarn issues and found issue #746. Someone had said they disabled their antivirus which solved their problem with socket hang ups. I was running Malwarebytes and upon quitting it, I no longer had this error.

For me turning off Windows Defender worked.

Turning off windows defender also worked.


[BUG] Wrong references in GitHub issue template

Github issue template to remove references of E6C

Fixes #7


auth=true creates users but does not limit other logins in recent mongo versions

As discussed in the documentation for the latest release (https://docs.mongodb.com/manual/tutorial/enable-authentication/) if a configuration file is in use then --auth doesn't have any effect but the security.authorization setting is required in the configuration file.  There is a catch22 implied by that tutorial also.  If we simply add the security.authorization setting (default['mongodb']['config']['mongod']['security']['authorization'] = 'enabled') this blows up with huge errors because the users cannot be created (since there is no admin login by default)
My workaround, which seems okay but ugly is something like this:
#Set passwords from databag for admin user
node.default['mongodb']['admin']['username'] = 'myadminuser'
node.default['mongodb']['admin']['password'] = db_passwords["mongodb_password"]
node.default['mongodb']['admin']["roles"] = [ "userAdminAnyDatabase" ]
node.default['mongodb']['admin']["database"] = 'admin'
myuser = {
"username" => "rouser",
"password" => db_passwords["mongodb_ro_password"],
"roles" => [ "read" ],
"database" => "#{node.chef_environment}"
}
node.default['mongodb']['users'] << myuser
include_recipe "sc-mongodb::default"
#There is a restart here
include_recipe "sc-mongodb::user_management"
#There is a restart here
bash 'add in user level security after mongo users are created' do
code <<-EOH
echo "security:\n  authorization: enabled\nsetParameter:\n  enableLocalhostAuthBypass: false" >> /etc/mongod.conf
EOH
not_if 'grep -c "enableLocalhostAuthBypass" /etc/mongod.conf'
end
#There is a restart after everything finishes

Oh, the default properties for my environment don't contain the properties added later, in case that was unclear.

roles and database being required should be updated in the documentation too, it is very unclear

This still doesn't work exactly how I would expect, you can still open mongo client without specifying a user, but at least you can't really do anything.  So other options might be needed in addition...
[vagrant@bluesky-db-prod ~]$ mongo fish
MongoDB shell version: 3.2.11
connecting to: fish

show collections;
2018-02-09T23:32:16.998+0000 E QUERY    [thread1] Error: listCollections failed: {
"ok" : 0,
"errmsg" : "not authorized on fish to execute command { listCollections: 1.0, filter: {} }",
"code" : 13
} :
_getErrorWithCode@src/mongo/shell/utils.js:25:13
DB.prototype._getCollectionInfosCommand@src/mongo/shell/db.js:773:1
DB.prototype.getCollectionInfos@src/mongo/shell/db.js:785:19
DB.prototype.getCollectionNames@src/mongo/shell/db.js:796:16
shellHelper.show@src/mongo/shell/utils.js:754:9
shellHelper@src/mongo/shell/utils.js:651:15
@(shellhelp2):1:1

My mongo-fu is kinda weak, but doesn't the localhost exception allow a way out of that catch-22?

Using localhost exception would be fine to create initial admin user (or all of them I guess), but for defense in depth it should be disabled after that I would think.
In any event I am currently having to not include user_management after first run because it throws a bunch of warnings and occasionally caused the run to fail.
iCanLogIntoMongo = system('mongo --quiet -u adminuser -p adminpass --eval "db.system.users.find({user:'someuser'}).count()" admin')
include_recipe "sc-mongodb::user_management" if ( !iCanLogIntoMongo )

The local host exemption seems to be automatically disabled once you've used it by the server. Basically, "if authentication is enabled and there are no users in the admin db, allow someone to create that user". So once that user is made, the conditions of the exemption stop applying.
I've been playing a bit with this recipe trying to get an understanding of both the code and mongo authentication. So bear with me, part of this is rubber duck debugging.
Are you using v1 or v2 of the mongodb gem? There seem to be different code paths based on which version is installed.
Part of the problem seems to be that the act of creating the admin user calls some functionality that requires admin privileges, such as listing users and databases. Part of this is because, as you've noted, the cookbook doesn't actually enable authentication, so it's gone unnoticed.
The good news is I'm able to replicate the problem and also have an inspec test to ensure that authentication is turned on (basically looking through startup logs to make sure that "authentication is disabled you fool!" message doesn't appear)
I think the code solution here is to build a separate mechanism for creating that admin user rather than the current method of prepending it to the list of users and treating it the same. But would love to hear from people more learned than myself.

Did you ever find out if you were using v1 or v2 of the mongo gem?
/opt/chef/embedded/bin/gem list mongo
For test-kitchen, I'm able to create users and enable auth with the following attributes:
 mongodb:
      config:
        auth: true
        mongod:
          security:
            authorization: enabled

[root@bluesky-db-uat dbsave]# /opt/chef/embedded/bin/gem list --local
*** LOCAL GEMS ***
addressable (2.5.2, 2.4.0)
appbundler (0.10.0)
ast (2.3.0)
bigdecimal (default: 1.3.0)
binding_of_caller (0.7.2)
blankslate (<IP_ADDRESS>)
bson (1.12.5)
bson_ext (1.12.5)
builder (3.2.3)
bundler (1.15.4)
byebug (9.1.0)
chef (13.5.3)
chef-config (13.5.3, 13.4.24)
chef-vault (3.3.0)
chef-zero (13.1.0)
cheffish (13.0.0)
coderay (1.1.2)
crack (0.4.3)
debug_inspector (0.0.3)
did_you_mean (1.1.0)
diff-lcs (1.3)
docile (1.1.5)
docker-api (1.33.6)
erubis (2.7.0)
excon (0.59.0)
faraday (0.13.1)
ffi (1.9.18)
ffi-yajl (2.3.1)
fuzzyurl (0.9.0)
gssapi (1.2.0)
gyoku (1.3.1)
hashdiff (0.3.6)
hashie (3.5.6)
highline (1.7.8)
htmlentities (4.3.4)
httpclient (2.8.3)
iniparse (1.4.4)
inspec (1.39.1)
io-console (default: 0.4.6)
ipaddress (0.8.3)
iso8601 (0.9.1)
json (2.1.0, default: 2.0.4)
libyajl2 (1.2.0)
little-plugger (1.1.4)
logging (2.2.2)
method_source (0.9.0)
mime-types (2.6.2)
mini_portile2 (2.2.0)
minitest (5.10.1)
mixlib-archive (0.4.1)
mixlib-authentication (1.4.2)
mixlib-cli (1.7.0)
mixlib-config (2.2.4)
mixlib-log (1.7.1)
mixlib-shellout (2.3.2)
mongo (1.12.5)
multi_json (1.12.2)
multipart-post (2.0.0)
mysql2 (0.4.9)
net-scp (1.2.1)
net-sftp (2.1.2)
net-ssh (4.2.0)
net-ssh-gateway (2.0.0)
net-ssh-multi (1.2.1)
net-telnet (0.1.1)
netrc (0.11.0)
nokogiri (1.8.0)
nori (2.6.0)
ohai (13.5.0)
openssl (default: 2.0.5)
parallel (1.12.0)
parser (<IP_ADDRESS>)
parslet (1.5.0)
plist (3.3.0)
power_assert (0.4.1)
powerpack (0.1.1)
proxifier (1.0.3)
pry (0.11.1)
pry-byebug (3.5.0)
pry-remote (0.1.8)
pry-stack_explorer (<IP_ADDRESS>)
psych (default: 2.2.2)
public_suffix (3.0.0)
rack (2.0.3)
rainbow (2.2.2)
rake (12.1.0, 12.0.0)
rb-readline (0.5.5)
rdoc (default: 5.0.0)
rest-client (1.7.3)
rspec (3.6.0)
rspec-core (3.6.0)
rspec-expectations (3.6.0)
rspec-its (1.2.0)
rspec-mocks (3.6.0)
rspec-support (3.6.0)
rspec_junit_formatter (0.2.3)
rubocop (0.49.1)
ruby-prof (0.16.2)
ruby-progressbar (1.9.0)
ruby-shadow (2.5.0)
rubyntlm (0.6.2)
rubyzip (1.2.1)
safe_yaml (1.0.4)
semverse (2.0.0)
serverspec (2.41.0)
sfl (2.3)
simplecov (0.15.1)
simplecov-html (0.10.2)
slop (3.6.0)
specinfra (2.72.0)
sslshake (1.2.0)
syslog-logger (1.6.8)
systemu (2.6.5)
test-unit (3.2.3)
thor (0.20.0)
toml (0.1.2)
train (0.28.0)
unicode-display_width (1.3.0)
uuidtools (2.1.5)
webmock (3.0.1)
winrm (2.2.3)
winrm-fs (1.0.2)
wmi-lite (1.0.0)
xmlrpc (0.2.1)

Still hitting this with v1 so bump.
[root@docker-db-dev ~]# /opt/chef/embedded/bin/gem list mongo
*** LOCAL GEMS ***
mongo (1.12.5)

[root@docker-db-dev ~]# mongo -version
MongoDB shell version: 3.2.11

There are multiple reasons why this is failing on MongoDB > 3.0.
First and foremost, as already indicated, authentication is not even enabled by the cookbook if node['mongodb']['config']['auth'] = true, because the cookbook doesn't set node['mongodb']['config']['mongod']['security']['authorization'] = 'enabled'
Second it seems that the localhost exception no longer allows more than just the initial admin user to be added
And third, when using mongo gem ~> 2.0, the method retrieve_db_v2 executes a client.database_names as a means to verify server connection, but this operation is not allowed within the localhost exception:
WARN: Unable to connect to MongoDB instance: not authorized on admin to execute command { listDatabases: 1, $readPreference: { mode: "secondaryPreferred" }, $db: "admin" } (13)

And fourth, it seems that user_exists_v2, which executes a connection['system.users'].find(user: username).count > 0 also doesn't work within the restrictions of the localhost exception:
> use admin
switched to db admin
> db.system.users.find()
Error: error: {
	"ok" : 0,
	"errmsg" : "not authorized on admin to execute command { find: \"system.users\", filter: {}, $db: \"admin\" }",
	"code" : 13,
	"codeName" : "Unauthorized"
}
>

So it looks like for adding the initial admin user, the current library functions will not work.


placeholder's text for TextInput  disappears on touch

Sometimes when I touch the TextInput, the placeholder text disappers and the same time keyboard doesn't pop up. I should reload the page to get the placeholder text again.

thanks @HadisSharif, please include additional information to your question. For example your sourcecode and a detailed explanation of your issue

thanks @HadisSharif, please include additional information to your question.


Error when using data_generator without randomness

Hi, the data_generator used in the training.py provided, uses Image Augmentation as default both for training and evaluation.
I've developed a new data_generator_wrapper that allows you to not augment the input image just by setting random=True in the get_random_data function.
def data_generator(annotation_lines, batch_size, input_shape, anchors, num_classes, random):
    '''data generator for fit_generator'''
    n = len(annotation_lines)
    i = 0
    while True:
        image_data = []
        box_data = []
        for b in range(batch_size):
            if i==0:
                np.random.shuffle(annotation_lines)
            image, box = get_random_data(annotation_lines[i], input_shape, random=random)
            image_data.append(image)
            box_data.append(box)
            i = (i+1) % n
        image_data = np.array(image_data)
        box_data = np.array(box_data)
        y_true = preprocess_true_boxes(box_data, input_shape, anchors, num_classes)
        yield [image_data, *y_true], np.zeros(batch_size)

def data_generator_wrapper(annotation_lines, batch_size, input_shape, anchors, num_classes, random):
    n = len(annotation_lines)
    if n==0 or batch_size<=0: return None
    return data_generator(annotation_lines, batch_size, input_shape, anchors, num_classes, random)

However, once I run this updated data_generator, I get the following error:
  File "/home/asabater/projects/Egocentric-object-detection/train_utils.py", line 116, in data_generator
    y_true = preprocess_true_boxes(box_data, input_shape, anchors, num_classes)

  File "/home/asabater/projects/Egocentric-object-detection/keras_yolo3/yolo3/model.py", line 299, in preprocess_true_boxes
    y_true[l][b, j, i, k, 0:4] = true_boxes[b,t, 0:4]

IndexError: index 21 is out of bounds for axis 1 with size 13

Any idea about what's going on? The generated image is supposed to be the same as the initial one but scaled to the input NN size.

Same issue. Have you solved this?

This is the code I use for the data generator:
def data_generator_default(annotation_lines, batch_size, input_shape, anchors, num_classes, random):
    '''data generator for fit_generator'''
    n = len(annotation_lines)
    i = 0
    while True:
        image_data = []
        box_data = []
        for b in range(batch_size):
            if i==0:
                np.random.shuffle(annotation_lines)
            image, box = get_random_data(annotation_lines[i], input_shape, random=random)
            image_data.append(image)
            box_data.append(box)
            i = (i+1) % n
        image_data = np.array(image_data)
        box_data = np.array(box_data)
        y_true = preprocess_true_boxes(box_data, input_shape, anchors, num_classes)
        yield [image_data, *y_true], np.zeros(batch_size)

def data_generator_wrapper_default(annotation_lines, batch_size, input_shape, anchors, num_classes, random):
    n = len(annotation_lines)
    if n==0 or batch_size<=0: return None
    return data_generator_default(annotation_lines, batch_size, input_shape, anchors, num_classes, random)

To not use random: data_generator_wrapper_default(lines_val, batch_size_frozen, input_shape, anchors, num_classes, random=False)
Finally, this works but I don't know what was the mistake.

Hey @AlbertoSabater @SGS003,
i've encountered the same problem. Did you got it resolved? Just setting the variable random to False and removing the default values in the method signature did not help.
Do you have any clue why this IndexError gets thrown? I wrote an own generator func w/o augmentation only resizing and return same output shape, but still IndexError... very mysterious


Index NFTs for loading into Arango DB (WIP)

Unknown Bounty at this time.
Possible Index Keys:

licenseId: a way to find the license details
purl: a way to find the package details
component version: a way to find component details
application version: a way to find application details

Each index key maps to NFT CID.

We will need a way to render the resolved json, ie all child CIDs expanded into JSON.

Closing.  Handled in Architecture Doc


fix .svg icon, update font-mfizz, add apache conf, OS packages,C lang…

/cc issue #200

@DanBrooker are you good with merging this?

Looks fine other than package.json not sure why those changes are their. Version bumping is handled by the apm tool

Yeah, I am not sure what it was, just reverted the file back and squashed.


Segmentation fault (core dumped) occurred when running ROS node

Hi,
Thanks for sharing the source code of your paper. I downloaded and compiled successfully. The real world data "Frank Moosmann's "Velodyne SLAM" Dataset" works like a charm
However, when I run the ros node "show_objects_node" and playing my own bag file that is publishing velodyne point cloud, the node terminates with such error:
./show_objects_node --num_beams 16

 ===================== Setting Connection =====================
|| Sender: CloudOdomRosSubscriber (id: 1)
|| Type:   STREAMER
|| 				|
|| 				V
|| Client: DepthGroundRemover (id: 4)
 ==============================================================

 ===================== Setting Connection =====================
|| Sender: DepthGroundRemover (id: 4)
|| Type:   STREAMER
|| 				|
|| 				V
|| Client: ImageBasedClustere<(short)1, (short)1> > (id: 6)
 ==============================================================

 ===================== Setting Connection =====================
|| Sender: ImageBasedClustere<(short)1, (short)1> > (id: 6)
|| Type:   STREAMER
|| 				|
|| 				V
|| Client: ObjectPtrStorer (id: 3)
 ==============================================================

 ===================== Setting Connection =====================
|| Sender: CloudOdomRosSubscriber (id: 1)
|| Type:   STREAMER
|| 				|
|| 				V
|| Client: Visualizer (id: 2)
 ==============================================================
INFO: Running with angle tollerance: 10.000000 degrees
INFO: Ground removed in 815 us
ERROR: DiffType is NONE. Please set it.
Segmentation fault (core dumped)

I don't see any settings for DiffType in the .cpp file. Any idea what's going on here?
Thanks!

I added
clusterer.SetDiffType(DiffFactory::DiffType::ANGLES);
after declare cluster object.

Yeah, sorry for that. I have been changing the interface and did not change the examples. I will reopen the issue until I do it. Thanks for reporting!


Don't work - Ultrawide 3440x1440

can you fix this?
best regards from Russia :)

i use one monitor - 34 Xiaomi Mi Curved

Check scaling on your monitors. I had my 1440 monitor with 125% scaling, and the program didn't work until I changed to 100%

after the last release - I confirm the work on the resolution 3440x1440
by the way, the solution with disabling dynamic resolution did not help with the previous build.
can be closed.


The dataToInstance method is very performance-consuming

After querying a large number of records through findAll, getAll hits the cache. At this time, dataArray.map(data => dataToInstance(model, data)) will be executed. This process is considered to be costly. The response time of 100+ data without cache is 0.0x seconds, After being cached, the response time is >0.9 seconds

Ugh oh. Nice catch. We definitely need to add a performance test in order to prevent situations like that.

Ugh oh. Nice catch. We definitely need to add a performance test in order to prevent situations like that.

This is actually so bad it brings down my site. We have downgraded to 2.2.3 to get around this


Doc Update - Validator Upgrade Request

Since I am not able to run an upgrade request on a live system, the data pulled was from running tests and replacing results with the data from actual validators. Correct me if I am wrong, but it appears that responses for successful upgrade requests contain the old Primary Validator in the response's primary_validator field.

@wendtl Can you let me know your TNB Account Number so I can send you PR reward

Just tag me or contact me at slack(Rajput Usman) and reply with your Account Number for PR-reward moving it to No account Number.

@Rajputusman I have sent you my account number on slack

Account No cbc2ab2d700825c17ad8487ff9447b08ea5dc3bb0047eb7101c00c346dcbae01


Enhance APIClient for serverless functions

Travis CI checks failed, please fix the linting.

All changes in this PR have been implemented in the https://github.com/rossumai/rossum/pull/7 and https://github.com/rossumai/rossum/pull/6


Migrate from Travis CI to GitHub Actions

This also adds linting in line with bigslice and bigmachine.

This also adds linting in line with bigslice and bigmachine.

Looks good. Just please ensure to change the os setting to use the matrix.

Yep; done here: https://github.com/grailbio/base/pull/29/commits/85f19f09128e57faafdf954b055ba393455266d8

Looks good. Just please ensure to change the os setting to use the matrix.

Yep; done here: https://github.com/grailbio/base/pull/29/commits/85f19f09128e57faafdf954b055ba393455266d8


Did the following changes

Fixed the existing  broken unit tests
Added a new feature to allow verification of Schemas
Added unit tests for the above feature

Hey @b3hdad
I would love to understand the intent of this change and how you are wanting to use this feature.
A couple of additional things:

What do you mean by "Fixed the existing broken unit tests"?
The AppVeyor build is failing as a result of the changes https://ci.appveyor.com/project/SEEKJobs/pact-net/build/116

Basically we are trying to cover the situation where one service publishes the data in the form of json documents to a location like S3 buckets and other service(s) will consume it. So the consumers can generate a contract which specifies how the json schema should look like. Then the provider can validate a sample data against that contract.
When I cloned the repository and ran the unit tests locally 6 of them were failing. The failures were due to serialization/deserialization. The actual was including the null values in the json while the expected was omitting the null values causing the assert to fail.
I have a look to see why the build is breaking. It passes locally when I run from visual studio. I see if I can reproduce the problem. Please bear with me as this is the first time I am dealing with Github.

Thanks mate. Let me have a think about this a little more.
None of the test should be broken to start with, as we have a CI pipeline via AppVeyor which runs all test is master to ensure everything is green. I'm not sure why they were broken on your machine.

No problems. Once you review, we can discuss and see if you are happy for another change. What I am thinking is to apply similar thinking to the responses in the interactions. So one would define the pact as normal for requests and responses, however, in the response instead of specifying properties with values, we could only specify the schema and do the validation. This I think has a lot of values and makes things a lot easier.

Neil,
After sharing this internally we have decided to move this out of the Pact.net and building it in its own repository/project.
I hope you didn't spend much time on this, but please feel free to decline.
Cheers.

@b3hdad All good mate. I was actually planning on looking into this tonight, so it's good timing!
By the sounds of what you are trying to do, version-2 of the pact specification should do something very similar. It's just a matter of someone getting the time to build it.

Thanks @neilcampbell .
I am no so sure if Pact.net version 2 will do the same thing.
Pact.net as a solution I think is for system centric architecture for dealing with synchronous operations(HTTP reqs and responses).
What I am trying to address is the data centric models where the operations are async... someone puts some data(e.g. JSON documents) somewhere(e.g. S3 bucket, DynamoDb), then whoever is interested picks it up.
I hope I make sense.


Fix not numbered inconsistency

The key not_numbered/numbered was added to the toc of books even if it was undefined.
Removed the convertTo from buildTOC method, because the toc that we are modifying is directly read from file, so we don't need to convert it.

Pull Request Test Coverage Report for Build 560248496

4 of 6 (66.67%)  changed or added relevant lines in 2 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.002%) to 43.466%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

extensions/notebook/src/book/bookTocManager.ts
2
4
50.0%

Totals

Change from base Build 560173976:
-0.002%

Covered Lines:
24935

Relevant Lines:
52529

💛  - Coveralls

I still have a concern about you copying and modifying the sections in the existing code. Are you sure there aren't any problems that could arise because of that? Would be worth doing a quick look over it and seeing if there's any potential for problems here.
Sounds good! I haven't found any major issues, but I can go over the code.


Unable to copy Markdown output after run the cell

Unable to copy Markdown output after run the cell
https://github.com/user-attachments/assets/ffd0df27-b9fd-480f-8d27-f7f085bc2ad4

You can press ctrl+C to copy the markdown, or put the cell in edit mode first


New component

we would like to filter all data on the page

Behavior:
After setting some of value to input, the new component has to change query value in route.
Technical requirements:

create a new component
use  component as a basis
preset values, after reload of the page the value (if exists) should be set.
debounce as a configuration
configuration for GET name parameter

PR: https://git.awescode.com/awes-io/ui/merge_requests/5

➤ Ilya Kitayev commented:

Strongly suggest to use mergeRouteQuery helper function for DRY
Maybe we should use route.replaceinstead of push? It will be easier to navigate through the pages
If some external component will update route.query, AwSearch won’t react anyhow. Qustion: should it update itself on query update from external?


add scale option

Adds scale option. To add HTML overlay with scaling to your OpenSeadragon Viewer, call htmlOverlay({scale: scaleWidth}).
This is useful in case the width of tileSource is not set, or you want html overlay dimensions to be independent of image dimensions. By default scale is set to 1 for backwards compatibility or cases where tileSource width is set.

added requested changes, updated demo to include version that uses scale option

I made some changes to the README and the demo file... I hope you don't mind!
https://github.com/openseadragon/html-overlay/commit/7b13d60a7cde11a8e68f654148976bcffe8292db
Thanks again for making this patch :)


Default system dashboard respect stop timestamp when using a custom timerange

Steps to reproduce:
List the minimal actions needed to reproduce the behavior.

In the web interface: Boards > Create Dashboard > From a Template > System > Create Dashboard
Configure telegraf to send system data into a bucket
View the data in the default system dashboard
Select a custom time range, use a stop time from the past
The cells/graphs in the dashboard don't respect the stop time

Expected behavior:
The graph should stop at the stop time rather than the present time
Actual behavior:
The graph always stops at the current time
Fix
Swapping line 2 in each of the cells from
|> range(start: v.timeRangeStart)
to
|> range(start: v.timeRangeStart, stop: v.timeRangeStop)
fixes this, I'd like to have opened a PR as this is a small change but I'm struggling to find where the default dashboards are imported from.

@JimACarter thanks for the issue. That option is part of the first generation of templates that we are planning to remove in the next release. The same functionality is available using the newer InfluxDB Template https://github.com/influxdata/community-templates/tree/master/linux_system and loading it via the Settings > Templates page.


JS-capstone-project (Kanban board)

[ ] https://github.com/odaialazzeh/JS-capstone-project/projects/1

[ ] Note:
2 team members.

Hi @odaialazzeh
Good job so far!
👏👏👏
There are some issues that you still need to work on to go to the next project but you're almost there!

To Highlight 🎉

Kanban board Created  ✔
Group Task Assigned 👏 ✔

Required Changes ♻

[ ] 🛑  As part of the requirements, you should decide who is Student A & who is Student B. So each card with Student A in its title should always be assigned to the same student (and analogical for other students). so kindly assign your selves to the correct unassigned cards 🙏

Optional suggestions

N/A

Every comment with the [OPTIONAL] prefix is not crucial enough to stop the approval of this PR. However, I strongly recommend you take them into account as they can make your code better.
Cheers and Happy coding!👏👏👏
Feel free to leave any questions or comments in the PR thread if something is not 100% clear.
Please, remember to tag me in your question so I can receive the notification.
Please, do not open a new Pull Request for re-reviews. You should use the same Pull Request submitted for the first review, either valid or invalid unless it is requested otherwise.

As described in the Code reviews limits policy you have a limited number of reviews per project (check the exact number in your Dashboard). If you think that the code review was not fair, you can request a second opinion using this form.

Status: ✔ Project Approved ✔
Hello @odaialazzeh 🔥
You worked hard in all the instructions that the previous Code Reviewers give you👍🏽
Now you can close this issue. 🚀
Highlights
Kanban board Created ✔
Group Task Assigned 👏 ✔
Optional suggestions
Every comment with the [OPTIONAL] prefix is not crucial enough to stop the approval of this PR. However, I strongly recommend you take them into account as they can make your code better.
Cheers and Happy coding!👏👏👏
Feel free to leave any questions or comments in the PR thread if something is not 100% clear.
Please, do not open a new Pull Request for re-reviews. You should use the same Pull Request submitted for the first review, either valid or invalid unless it is requested otherwise.

As described in the Code reviews limits policy you have a limited number of reviews per project (check the exact number in your Dashboard). If you think the code review was unfair, you can request a second opinion using this form.


Do not require the caller to try/catch Throwable

Return UnsatisfiedLinkError instead the result of #initCause(), i.e. a Throwable.
This way there is no need to declare 'throws' or try/catch at the caller site

Thanks for making the code cleaner.


Update sample template

This PR add being able to update a sample template

Tests are not actually failing. The new interface marks the build as failure cause the Coverage decreased

I added a sample template to a study that already had a preprocessed data. And since this is not allowed, I was sent to the error page. Which is not useful for a regular user, this error should be described and presented to the user, along with a suggestion on how to deal with this:
Traceback (most recent call last):
  File "/Users/yoshikivazquezbaeza/.virtualenvs/qiita/lib/python2.7/site-packages/tornado/web.py", line 1144, in _when_complete
    if result.result() is not None:
  File "/Users/yoshikivazquezbaeza/.virtualenvs/qiita/lib/python2.7/site-packages/tornado/concurrent.py", line 129, in result
    raise_exc_info(self.__exc_info)
  File "/Users/yoshikivazquezbaeza/.virtualenvs/qiita/lib/python2.7/site-packages/tornado/gen.py", line 221, in wrapper
    runner.run()
  File "/Users/yoshikivazquezbaeza/.virtualenvs/qiita/lib/python2.7/site-packages/tornado/gen.py", line 505, in run
    yielded = self.gen.throw(*exc_info)
  File "/Users/yoshikivazquezbaeza/git_sw/qiita/qiita_pet/handlers/study_handlers/description_handlers.py", line 738, in post
    study, user)
  File "/Users/yoshikivazquezbaeza/.virtualenvs/qiita/lib/python2.7/site-packages/tornado/gen.py", line 533, in run
    self.yield_point.start(self)
  File "/Users/yoshikivazquezbaeza/.virtualenvs/qiita/lib/python2.7/site-packages/tornado/gen.py", line 371, in start
    self.func(*self.args, **self.kwargs)
  File "/Users/yoshikivazquezbaeza/git_sw/qiita/qiita_pet/handlers/study_handlers/description_handlers.py", line 181, in process_sample_template
    fp_rsp)
  File "/Users/yoshikivazquezbaeza/git_sw/qiita/qiita_pet/handlers/study_handlers/description_handlers.py", line 548, in remove_add_study_template
    PrepTemplate.delete(pt)
  File "/Users/yoshikivazquezbaeza/git_sw/qiita/qiita_db/metadata_template.py", line 1921, in delete
    "using it." % id_)
 QiitaDBError: Cannot remove prep template 101 because a preprocessed data has been already generated using it.

From trying to update another sample template in a different study I got this on the user interface (note that updating worked in other cases, but this is the first time I see this error):
Error running SQL query: UPDATE qiita.sample_10006 SET depth=%s WHERE sample_id=%s ARGS: ('no_data', '10006.NEWSAMPLEB1') Error: invalid input syntax for type double precision: "no_data" LINE 3: SET depth='no_data' ^

Aside from those two things the interface looks fine.

Ready for final review.

Looks good, a few minor comments. I tried to find an issue for the first problem that I noted in my previous comment, but couldn't find it, can you open a separate issue if we indeed don't have an issue about it, else I'll open it.

Added: https://github.com/biocore/qiita/issues/909

Thanks! :+1:, this is ready to be merged.
On (Mar-09-15| 6:18), Antonio Gonzalez wrote:

Added: https://github.com/biocore/qiita/issues/909

Reply to this email directly or view it on GitHub:
https://github.com/biocore/qiita/pull/904#issuecomment-77850986


Updates for latest wgpu-native

[ ] (wait for) release wgpu-native
[ ] Bump to latest wgpu-native
[x] Apply any necessary changes

The wgpu-native has bumped its version of wgpu-core and Naga. The only changes that matter to us are that Naga is a bit more strict with indexing (can't index into arrays created with let, but must use var instead).

This is now held back by https://github.com/gfx-rs/wgpu-native/issues/118


Software clock: good things,  bad things, how to do it in another way

Hi,
There is a tick() function in src/raft/raft.rs. It works as a software clock that periodically checks whether it have reached some timer and trigger these timers. The good thing of a software clock is that the programmers could assume they occupy a CPU and do whatever they want during this occupation, including checking some timer. It's much easier to code in this way. The bad thing is, it does take over the cpu. IMHO, it has a few drawbacks

inefficient
Most of the times, the CPU wakes up and do no thing. This inefficience would be amplified if multiple raft groups are run in one node.

inaccurate
Since it's a software clock, it's inaccurate considering the scheduling of OS processes/threads is inaccurate.

bad style
Building a high level software abstraction is neccessary sometimes, it could simplify the code. But if the performance is the first priority, we should be able to get down to the low level mech, even hardware.

There are two styles of express something like a timer: the polling style VS. the event-driven style. IMHO, it's possible to write raft in the event-driven style(although doing that would make the impl more complex, like we usually do in the event-driven network programming). Even after it's done in event-driven style, the system clock is still needed to adjust the inaccuracy of event. Lucky that in raft a member node only needs the local relative time, and the cpu time is accurate enough since the crystal oscillator is always trustworthy.

Right, as we discuess in our open meetup yesterday, we need  cpu clock to correct event timer. Then we can use the lease feature of raft to get consistent value from leader.

Would you like to do something on it? @hhkbp2
Something like:
Testing on how to simulate uncertain clock.
How to simulate slow/unstable disk ?

@ngaut Currently I don't have a clear idea how to do these testing. Mock? Interface interception? There is a deep learning curve before I could actually write some code in the rust programming language. Maybe I could get a hand on it later when I have some more free time.

In MIO, it uses clock_gettime with CLOCK_MONOTONIC to get the current time, this is the CPU clock but may still be affected by NTP, see http://man7.org/linux/man-pages/man2/clock_gettime.2.html, but using CLOCK_MONOTONIC_RAW has no problem. We should send an issue in MIO to discuss it.
Btw, I don't think clock_gettime has a performance issue.


Add QGIS Plugin - Vector-Tiles-Reader

Add a QGIS new plugin that can read vector tiles

Thanks @cliffordsnow! This is great.


Manual usage of continuation tokens

Is your feature request related to a problem? Please describe.
Currently our front-end does not access our storage directly, it sends a request to our back-end and we fetch the data from table storage over there.
Right now I'm working on refactoring our back-end to use the new @azure/data-tables instead of azure-storage.
Some APIs we have implement pagination by using the continuation token - passing it between the front and the back end for querying and creating the pages. However, I don't see an option to pass a continuation token to the queries in the new SDK, nor get a continuation token back from a query.
Describe the solution you'd like
Either an option to pass a continuation token to the existing listEntities operation and get a continuation token from the response (probably more suited to the byPage option); or another operation similar to the old API, without using iterators to retrieve entities which accepts and returns a continuation token.
Update
I actually see it is possible to use manual continuation tokens with tables, it's just not typed.
At least when using byPage, the TableEntityResult<T> also has nextPartitionKey and nextRowKey which can be added to the ListTableEntitiesOptions to be used in the continuation token.

Thanks for reaching out @omerdolev-ms. Unfortunately, we don't support manual handling of continuation tokens, our listEntities function handles the continuation tokens internally and exposes the AsyncIterator to iterate over all the entities or byPage.
I'm curious to learn more about your scenario and the reason why you're looking for an option without iterators. Would you mind providing more details on your scenario?

Hi @joheredi, thanks for responding.
Our use case is as follows:
We have a simple table view to show the users data saved in Azure Table. To populate this view, the front end makes a request to our service, which in turn queries Azure Table Service using the SDK and returns the result to the front end.
In this specific view, it is very possible (and even likely) for customers to have more than 1,000 entities. So we implemented paging. If the customer have more than 1,000 entities, the count is displayed at the bottom of the view with a CtA to load more data. Clicking "load more" will fetch the next 1,000 entities from our service and add them to the view.
While there are entities available for fetching, the CtA will be displayed.
The way this was implemented using the old azure-storage SDK, is by serializing and passing the continuation token to the front end. If the front end got a continuation token from the last call he did to our service, it knows it should display the CtA and send this continuation token back to the service when the user clicks "load more".
Regardless, it is not very convenient to have to always use the async iterator.
Is is convenient to have this abstraction when we actually want to fetch all the data and process it in some way.
But in many cases, we are certain the user will have less than 1,000 entities saved and would like to just get an array of these entities without having to for await on the query.

@joheredi Storage Explorer also handles pagination manually. What's more, the v12 blob SDK supports using continuation tokens with byPage(). The iterator stuff is fine, but it's not tractable for multi-process architectures (such as what we use) or for very large tables. The API should support taking a continuation token as an arbitrary "starting point" just like Blobs SDK does (see PageSettings type in @azure/core-paging).


Update Harbor helm chart to deploy Harbor 1.4.0 release

cc @reasonerjt @paulczar for review
Verified following features:

login UI
basic function
docker login
docker push
replicate image to another harbor
scan image via clair

Notary support is not added yet.

Coverage decreased (-0.04%) to 59.878% when pulling 9a5eb4583a3a9caa185330bb1cfc128d9bb1799c on jessehu:chart into 7bf277df6ebbc020134a72b73a7dcf91ac25d696 on vmware:master.

An overall comment is that some components such as adminserver registry may not need to be stateful set (considering we use PV to store the data), but we can revisit and change that in another change.


View always shows side-menu button instead of back button if the template has a ion-footer

Ionic version:  (check one with "x")
[x] 1.x
[ ] 2.x
I'm submitting a ...  (check one with "x")
[x] bug report
[ ] feature request
[ ] support request => Please do not submit support requests here, use one of these channels: https://forum.ionicframework.com/ or http://ionicworldwide.herokuapp.com/
Current behavior:
If a template has a ion-footer, that view always shows the side-menu button in the top bar instead of showing the back button.
Expected behavior:
The view should show back button instead of side-menu button since we have a back view.
Steps to reproduce:
http://plnkr.co/edit/HDCetVQaZRkBGIjARUxX?p=preview
Other information:
This works fine in Ionic 1.3.0, but it doesn't in 1.3.1 or 1.3.2
Ionic info:
Cordova CLI: 6.1.1
Gulp version:  CLI version 1.2.2
Gulp local:   Local version 3.9.1
Ionic Version: 1.3.2
Ionic CLI Version: 1.7.14
Ionic App Lib Version: 0.7.0
ios-deploy version: 1.8.5
ios-sim version: 5.0.8
OS: Mac OS X El Capitan
Node Version: v6.9.1
Xcode version: Xcode 8.1 Build version 8B62

Ok, not an Ionic bug, the problem was that ion-footer-bar needs to be inside of ion-view and I placed it outside of it.
Closing the issue.


symbol substitution fails on multi-word symbol

Reported by hankin on 3 May 2002 17:25 UTC
! err540_multi_word_symbol.jnl
! sh 5/3/02
! symbol substitution fails when the symbol contains
! multiple words
! this works
define symbol a hello
say ($a"bye|hello>1")
! but this fails
define symbol b hello and some more words
say ($b"bye|hello>1")
quit
! ==============================
yes? say ($b"bye|hello>1")
**ERROR: invalid command: Valid value needed for symbol b
*** NOTE: Valid symbol b choices are: hello>1
Migrated-From: http://dunkel.pmel.noaa.gov/trac/ferret/ticket/391

Comment by<EMAIL_ADDRESS>on 3 May 2002 18:54 UTC
not a bug -- complete strings are matched, but
not partial ones.
e.g. this is ok
yes? define sym a hello and more text
yes? say ($a"|bye|hello and more text>1")
Or need a default substitution
yes? define sym a hello and more text
yes? say ($a"|bye|hello>1|>")


Supporting font family for alternative font

Supporting font family for alternative font\

เมื่อ จ. 22 มิ.ย. 2563 เวลา 18:09 Steven<EMAIL_ADDRESS>เขียนว่า:

Supporting font family for alternative font\
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/microsoft/terminal/issues/6627, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AP55P53EVWG6BJW72RW5EBDRX43WPANCNFSM4OEPIZ6A
.

Thanks for the suggestion! Please make sure to search for duplicates when filing new issues. This is already being tracked by another issue on our repo - please refer to #1163 for more discussion.
/dup #1163


Reinstalling dependencies upon switching branch in development mode

It would be really in nice that create-react-app's dev server script would reinstall dependencies (w/ either yarn or npm) whenever file updates are detected in either package.json/yarn.lock/npm-shrinkwrap.json.
Is it a sane feature request?

Why do people ever think that CRA is a tool that covers ALL possible development features?
Despite the fact this feature might be pretty helpful, who stops you from writing own kind of gulp task and just run these commands by your own?
// offtop to collaborators
Any chance you could put a disclamer of CRA true purpose and it's near future plans? I think it might be really helpful in terms of getting rid of issues that are not related to CRA itself.

Yeah, you right. I drafted PoC module though :)
https://github.com/mk0x9/dep-reloader
Could be dropped just as require('dep-loader'); inside of scripts/start.js.


feat(airflow): retry callback, support ExternalTaskSensor subclasses,…

… add properties
Closes #6305.
Checklist

[X] The PR conforms to DataHub's Contributing Guideline (particularly Commit Message Format)
[X] Links to related issues (if applicable)
[X] Tests for the changes have been added/updated (if applicable)
[X] Docs related to the changes have been added/updated (if applicable). If a new feature has been added a Usage Guide has been added for the same.
[X] For any breaking change/potential downtime/deprecation/big changes an entry has been made in Updating DataHub

@richenc This was a long-standing gap in our Airflow plugin, so thanks for taking a stab at it!
We'll be reviewing it soon.

@hsheth2 I noticed the v2 plugin released recently. I was wondering how that affects my PR.

@richenc this PR still applies for the v1 plugin and we can merge review/merge it in around v1 if we want. However, the v2 plugin uses the Airflow listener API (instead of the somewhat hacky policy mechanism that v1 uses), so we should be able to detect UP_FOR_RETRY events more reliably with v2.
The external sensor changes seem reasonable and are part of common code between v1 and v2, so that logic should be ok to merge regardless

@richenc this PR still applies for the v1 plugin and we can merge review/merge it in around v1 if we want. However, the v2 plugin uses the Airflow listener API (instead of the somewhat hacky policy mechanism that v1 uses), so we should be able to detect UP_FOR_RETRY events more reliably with v2.
The external sensor changes seem reasonable and are part of common code between v1 and v2, so that logic should be ok to merge regardless

@hsheth2  good to hear that the listener API is being used! I think that means it might be possible to emit the data flow runs as dataprocessinstances! I'm going to look into the v2 plugin so I can migrate and contribute to it in the future.
I just rebased my branch to fix the conflicts. Please let me know if there is anything else I need to do before this can be merged.

The code looks good
Do you have a sense for how we'd implement something similar with the v2 plugin (using the listener API)?

thanks @hsheth2! At the moment, I'm not sure but I was thinking about taking a crack at it once I had some time to test the v2 plugin in our local environment in preparation for migrating.


Add Marketplace AMIs

Right now we're using platform images.  Need to create dummy marketplace images first.

Dummy images now exist.  We would need to reference them here.  If we did that, Quick Start use would show in Marketplace reporting.  The downside is that the clever platform image reference code would need to be replaced with an AMI map.

this is implemented


RFC: lyft/Cartography, Neo4J 4.x, Monkeypatching

RFC: lyft/Cartography, Neo4J 4.x, Monkeypatching
We're doing some automatic query syntax/code changes to help us migrate from Neo4j 3.5 to 4.x. Comments welcome!
https://docs.google.com/document/d/1DR-XPzEpd5o_aqvgH28mdIJG-2WSeektx2V9hETFXuM/edit#heading=h.cqtqxkjjli6n
Update Ju ne 21, 2022
The auto-patcher has been released in #861, with special developer instructions. Pre-releases  with the patch enabled by default can be had from releases tagged like 0.62.0.dev1.

Closing this out as we have fully implemented this.


copying in repo to dockerfile

https://trello.com/c/sExGgeOs/896-run-api-client-integration-tests-as-part-of-new-pipeline

Superseded by #196


Adding new method to InventoryManager that removes an item from a given ...

...slot.
Made IntegerRange implements Iterable.
Fixed HorizontalBlockFamily to return a block if invoked without secondary side, as in rare circumstances there might be none.

Refer to this link for build results (access rights to CI server needed):
http://jenkins.terasology.org/job/TerasologyPRs/52/
Hooray Jenkins reported success with all tests good!

It would be great if you could fix the checkstyle issues that were introduced with previous PRs, e.g. IntegerRange.addNumbers needs braces.

Yeah I find it useful to hit the "Check modified files" in the Checkstyle tab then fix anything outstanding even if I didn't touch it. Anything not overly complicated anyway :-)
Whenever there is an API change so to say that might need fixes in modules I can check on it, just helps to have it highlighted
Also, help me review/test the launcher kthxplz! :D

Refer to this link for build results (access rights to CI server needed):
http://jenkins.terasology.org/job/TerasologyPRs/53/
Hooray Jenkins reported success with all tests good!

Refer to this link for build results (access rights to CI server needed):
http://jenkins.terasology.org/job/TerasologyPRs/55/
Build Log
last 5 lines
[...truncated 2087 lines...]
   at hudson.model.Executor.run(Executor.java:240)
[ANALYSIS-COLLECTOR] Skipping publisher since build result is FAILURE
Notifying upstream projects of job completion
Setting status of 83ac430ccf1bfd7155e7867516dd880746db0516 to FAILURE with url http://jenkins.terasology.org/job/TerasologyPRs/55/ and message: Merged build finished.

Uh oh, something went wrong with the build. Need to check on that

Looks like the occasional Artifactory crash was the culprit
Gooey: test this please

test this please
No fair, it works for @msteiger :(

Test please

Refer to this link for build results (access rights to CI server needed):
http://jenkins.terasology.org/job/TerasologyPRs/56/
Hooray Jenkins reported success with all tests good!

Refer to this link for build results (access rights to CI server needed):
http://jenkins.terasology.org/job/TerasologyPRs/57/
Hooray Jenkins reported success with all tests good!

Refer to this link for build results (access rights to CI server needed):
http://jenkins.terasology.org/job/TerasologyPRs/58/
Hooray Jenkins reported success with all tests good!

Did some general inventory testing both in single, client/server + client, and headless + 2 x clients. Seemed fine. Seems the client disposal issues are getting worse lately though (not this PR - pre-existing) - all the plain clients fail to dispose the game correctly.
@MarcinSc did you mean to get https://github.com/MarcinSc/Terasology/commit/bf94cca17c85eff9931f6c3729f669da317c80f9 rolled into this PR? Topic branches ftw! :-)


Repos gone after Upgrade

Gogs version (or commit ref):  <IP_ADDRESS>6
Git version:
Operating system:
Database (use [x]):

[ ] PostgreSQL
[ ] MySQL
[x] SQLite

Can you reproduce the bug at https://try.gogs.io:

[ ] Yes (provide example URL)
[x] No
[x] Not relevant

Log gist:

Description
I removed my old container but my volume consists.
After recreate with newer image, I can login and everything else consists. But my Repos are gone in the webinterface.
After Logging in:

Commands
docker stop Go-Git-Server
docker rm Go-Git-Server
docker rmi
docker run -d --name=Go-Git-Server -p 10022:22 -p 10080:3000 -v gogs-data:/data gogs/gogs
tree -L 3
.
└── gogs-data
    ├── git
    │   ├── git -> /data/git
    │   └── gogs-repositories
    ├── gogs
    │   ├── conf
    │   ├── data
    │   └── log
    └── ssh
        ├── ssh_host_dsa_key
        ├── ssh_host_dsa_key.pub
        ├── ssh_host_ecdsa_key
        ├── ssh_host_ecdsa_key.pub
        ├── ssh_host_ed25519_key
        ├── ssh_host_ed25519_key.pub
        ├── ssh_host_rsa_key
        └── ssh_host_rsa_key.pub

Does gogs-data/git/gogs-repositories contain repository data?

Yes.
Commands
gogs-data/git$ tree -L 3
.
├── git -> /data/git
└── gogs-repositories
└── moep_moep
├── repo1.git
├── repo2.git
├── repo3.git
└── repo4.git

I suspect you did not connected to old volumed directory at all.

If so, why I didn't need to set the server up again? All files are stored in the same directory...

**Solved
1st I issued:
docker run -d --name= -p  -p  -v gogs-data:/data gogs/gogs
Ive been in the dir which contains "gogs-data". So I had just my account..
2nd I issued:
docker run -d --name= -p  -p -v //gogs-data:/data gogs/gogs
Now it is working again. I'm pretty sure its not the repos fault.


Add ability to sort nodes and edges

Some of the default layouts in cytoscape have a sort option (or some variation of it) that lets you sort the nodes/edges so the resulting graph is predictable if you're adding/removing nodes.
examples layouts with sort:
https://js.cytoscape.org/#layouts/grid
https://js.cytoscape.org/#layouts/circle
https://js.cytoscape.org/#layouts/breadthfirst
Without this, the graph will have a different outcome if several nodes/edges are added, then some middle nodes/edges are removed, and then those same removed nodes/edges are added back.

Great.
What could we do to clarify how this proposed sorting feature would affect the resultant layout, especially for people who are reading the documentation for the first time?  The circle layout and grid layout have conventional orderings, clockwise and the Western reading direction respectively (also the typical x/y graphics axes in CS).  The natural ordering of a tree is the topology, so the reader can't necessarily make a connection for what sorting would do.

We could clarify in the docs that by default cycoscape dagre creates a directed graph, and directed graphs use the node order as a tie breaker when defining the topology of a graph. This feature is most useful when adding and removing the same nodes and edges multiple times in a graph.
Would that make sense?

Sounds good

On Sep 21, 2022, at 22:47, Tyler @.***> wrote:
We could clarify in the docs that by default cycoscape dagre creates a directed graph, and directed graphs use the node order as a tie breaker when defining the topology of a graph. This feature is most useful when adding and removing the same nodes and edges multiple times in a graph.
Would that make sense?
—
Reply to this email directly, view it on GitHub https://github.com/cytoscape/cytoscape.js-dagre/pull/100#issuecomment-1254447306, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAHRO42NKU7RSI3XU3MRI73V7PCCZANCNFSM6AAAAAAQC2SFJI.
You are receiving this because you were mentioned.

@maxkfranz I updated the readme based on what we talked about 👍

Great, @treardon17.  I'll set aside some time tomorrow for a merge & release

Sorry, this slipped past me somehow.
Released to npm with your updates for 2.5.0

Thanks @maxkfranz!


EmrEtlRunner: add support of EVENTS_MANIFEST purpose for storage targets

iglu:com.snowplowanalytics.snowplow.storage/amazon_dynamodb_config/jsonschema/2-0-0 changed purpose from DUPLICATE_TRACKING to EVENTS_MANIFEST and EmrEtlRunner explicitly filters only DUPLICATE_TRACKING to be used for cross-batch deduplication.
It means EmrEtlRunner does not support 2-0-0 for cross-batch deduplication.

耶和华 ** **  ** **  ** **


Refactor the analog module

This is still largely untested, so opening as a draft for now, but will get it verified early next week. Just wanted to get some eyes on it in the meantime.
This refactoring had a few goals: improved code organization, reduced duplication, and improved documentation, primarily.
There are quite a few changes, but they are mostly pretty straight-forward.
Regarding the analog::dac module, I was able to eliminate the macro entirely and shrink the line count of the module overall, so that's a big win IMO. This came at the cost of a small amount of duplication, however it's quite minimal and this module is not likely to change significantly any time soon, so I think it's a reasonable tradeoff.
The analog::adc module was re-organized a bit, and I was able to extract out some common types from the various implementations, further reducing duplication. I also updated the documentation, which looks reasonably good now and has examples for both peripherals. I've additionally decoupled this driver from the embedded-hal traits while I was in here, by introducing the AdcChannel trait.
The peripheral constructors have changed a bit, and you no longer need to split a peripheral to get access to the ADCs/DACs, but other than that there aren't really any user-facing changes here, I don't think.

Did some really quick testings and things at least seem to still be working 😁


Validate Attachment Dictionary

What
Validate Attachment Dictionary on CDTDocumentRevision objects to ensure they
are keyed by the attachment name.
How
Added a new method in the CDTDatastore class which enumerates over the key value pairs in the dictionary, returning false if one of the entires is not keyed on the attachment name.
To keep API compatibility, this change only logs a message at the "Warning" level. When we are working towards a 2.0.0 release we can then update it to return an error in the event the attachments dictionary is not keyed correctly.
Issues
#287

Other than the comment about the documentation, looks good +1


Adds friends api in spec.

Just re-running the tests here to make sure everything is still working.


Eliminate bash scripts dependencies

Currently there exists some complicated dependencies between some setup bash scripts, like figure shown below:

Actually, most of them have been rewritten with Go, but to remove them completely, some modifications of files under scripts/github_runner directory need to be applied

can you be more specific on what was rewritten and what wasn't?

can you be more specific on what was rewritten and what wasn't?

Yeah,
Scripts have been rewritten:
All scripts under scripts/cloudlab
All scripts under scripts/cluster
All scripts under scripts/ except clean_cri_runner.sh, create_devmapper.sh, and clean_fcctr.sh
Additionally, scripts/github_runner/setup_cri_test_env.sh has been adapted to use Go scripts
All other scripts under scripts and its sub-directories left untouched

can you please update the PR's description and enumerate the scripts that still need to be eliminated. this issue is not about what was already rewritten but what needs to.

I don't think that's completely done. If I remember correctly, we still have shell scripts for setting up the cri runners.


Do not set accept-encoding on behalf of cloudflare

Thanks! This indeed solves my issue (#5).
I wonder if it would make sense to take an allow-list approach here instead of copying the original incoming request and stripping the known invalid headers?

https://github.com/aws/aws-sdk-js/blob/master/lib/signers/v4.js#L191-L199
The official s3 sdk is also using the blacklist model.
There are many headers that can help prevent security vulnerabilities.
Removing them might not be a good idea.

<EMAIL_ADDRESS>The official s3 sdk is also using the blacklist model. There are many headers that can help prevent security vulnerabilities. Removing them might not be a good idea.

@fourdim I could be mistaken, but I believe that the unsignableHeaders in the linked file serves a slightly different purpose than the UNSIGNABLE_HEADERS in this file. The note in the file mentions:
// These headers appear in the request, but are not passed upstream

In other words, Cloudflare seems to filter these headers from the outgoing request, so including them in the signature generates an erroneous signature.
I believe that in the file you linked, these are headers which will typically be overwritten by an HTTP client (e.g. something like cURL) with appropriate values based on the payload (e.g. content-length) and may be altered by a proxy or are otherwise sensitive (e.g. authorization) and therefore Amazon has decided not to use them as part of the signature calculation.
As far as security goes, are there any particular security headers you had in mind that might apply here? I'm familiar with security headers that clients and servers send, such as to prevent MIME sniffing, cross-origin requests, etc. but my mental model would be that it is actually safer to construct a new request here with known headers to the upstream, than to allow the client control over the headers that are sent to B2. However, I can't think of anything particularly nefarious here off the top of my head, and I'm curious if there's a header the client might be sending that would provide security value by passing it along to B2.

I'm not a security expert but this might be an example: https://w3c.github.io/webappsec-fetch-metadata
In addition to the security related headers, it is also troublesome to enumerate all the headers needed.

super, great job

@fourdim @JohnMaguire
or anyone from b2 could specify a whitelist for headers there?
Otherwise, it could be non functional once again when cloudflare changes something in the header forwarded there.

Thanks for the contribution, @fourdim!
The root cause seems to be a recently-introduced Cloudflare-ism: Upcoming Change To Accept Encoding Header.
This doc is related, but I'm not sure if it needs to be updated in light of the above change. I'll do a quick investigation on where accept-encoding gets clobbered - on the way in to the worker, or the way upstream to B2 - and whether B2 actually acts on that header.

@gillbates Agreed, this is what I was getting at in my above message. I think that there is no reason to send all of the downstream request headers to the upstream - just the headers necessary to load the file properly.

I'm not a security expert but this might be an example: w3c.github.io/webappsec-fetch-metadata In addition to the security related headers, it is also troublesome to enumerate all the headers needed.

I just got around to reading the spec you sent and I'm not sure I see how it applies here. It looks like it's a CSRF mitigation - from the introduction:

It would be helpful if servers could make more intelligent decisions about whether or not to respond to a given request based on the way that it’s made in order to mitigate the latter category. For example, it seems pretty unlikely that a "Transfer all my money" endpoint on a bank’s server would expect to be referenced from an img tag, and likewise unlikely that evil.com is going to be making any legitimate requests whatsoever. Ideally, the server could reject these requests a priori rather than delivering them to the application backend.

However, AFAIK Backblaze B2 is simply a static file host, so you can't trick it into taking some application-level action on the user's behalf (such as transferring their funds.)
I'm also not quite sure what you mean by "enumerate all the headers needed." I'm proposing that rather than using the incoming Request object, we form a new simple GET Request object and sign only the headers the new Request contains. I don't think there is a reason to pass along the browser's headers to Backblaze B2 (this is perhaps more useful for the sister project, https://github.com/backblaze-b2-samples/cloudflare-b2-proxy, which acts as a simple proxy.)

Thanks for the contribution, @fourdim!
The root cause seems to be a recently-introduced Cloudflare-ism:

Upcoming Change To Accept Encoding Header.
Content compression from origin servers to the Cloudflare network

I'll do a quick investigation into how B2 actually acts on that header, and how Cloudflare operates on the content, and either merge your PR as-is, or tweak it to set the outgoing accept-encoding to the same value that Cloudflare overwrites it with.

Yes. Please just update a whitelist function in your script to provide b2 only headers. In this case, any future changes by cloudflare or any other downstream providers will not make it stop working ...

I did some investigation here:

Regardless of what the client sends, Cloudflare sets the accept-encoding header to gzip, br in the incoming HTTP request before it gets to the worker
Regardless of what the worker sends, Cloudflare sets the accept-encoding header to gzip in the outgoing HTTP request after the worker is done with it

So it looks like the best way forward is to add accept-encoding to the list of unsignable headers - i.e. merge this PR.
I'm reluctant to just pick a set of HTTP headers to allow-list, since that risks future problems if/when B2 adds new functionality.
So, once I've merged this change, I'll add the ability to optionally specify an allow-list, ALLOWABLE_HEADERS, in configuration.

Added ability to optionally specify an allow-list, ALLOWABLE_HEADERS, in configuration: 6970948c30f2fe04415b1c90305d2dd6e8d8af09.


String comparison is not implemented yet

I am using the daphne-X86-64-v0.2 release. I tested the following:
print(1==1); results in output 1.
print("1"=="1"); results in an error:

JIT session error: Symbols not found: [ _ewEq__char__char__char ]
JIT-Engine invocation failed: Failed to materialize symbols: { (main, { main, _mlir_main, _mlir__mlir_ciface_main, _mlir_ciface_main }) }Program aborted due to an unhandled Error:
Failed to materialize symbols: { (main, { main, _mlir_main, _mlir__mlir_ciface_main, _mlir_ciface_main }) }
[2023-08-04 12:50:35.723] [error] Got an abort signal from the execution engine. Most likely an exception in a shared library. Check logs!
[error]: Execution error: Returning from signal 6

I'd argue that this is not just a missing feature, but rather a bug in the earlier stages of the compilation. Imho unsupported operations should be handled before the actual compilation & execution (i.e., before the jitting) starts.

This can be regarded as a bug or as a feature ;-) As a developer I argue more for the latter because nobody implemented the feature "check if the generated instruction actually exists and print out a more meaningful error message". Now that I think of it I'd rather advise against checking while compiling to not slow down that process. Once something has crashed we'd have all the time we need to create a meaningful output.

missing string comparison hinders the usage of strings as script arguments for control flow switches

I added a small patch for initial support of the == operator for strings.
This should handle the use case of script argument strings for control flow switches @m-birke .
Examples:

string with program argument

a = "foo";
if (a == $b)

compare program string arguments

if ($a == $b)

string constant comparison

print("1" ==  "1");

Thanks for this patch @philipportner! I'm already using the string comparisons in the decision tree script now.


[svg] Revised icons to its own separate directory

As we add more icons, there is a necessity to create icons in its own directories.

Contributor Guide

🎨 Wireframes and designs for Layer5 site in Figma (open invite)

Partially done here https://github.com/layer5io/sistent/pull/34

new pr https://github.com/layer5io/sistent/pull/58


Calendar multiday (and all day) events fix.

What issue is this referencing?
This PR fixes #418

Do these code changes work locally and have you tested that they fix the issue yourself?

[x] yes!

Does the following command run without warnings or errors?

[x] yes! npm run pr-checks

Have you taken a look at our contributing guidelines?

[x] yes!

My node version matches the one suggested when running nvm use?

[x] yes!

@charrde we'll merge this which is awesome but a few things here

it would be awesome is removing the time from the display.
Note the times seem to be offset a little bit 5pm-5pm we can fix this in a future PR as well.

Merging we're going to fix this wierd date thing later, I would like to have the meetups up so that I can see them and we can fix it later.


Chapter 7 - TFMA Evaluator AUC Metric Case Mismatch

Interactive pipeline sets a threshold for 'AUC' but metric produced is 'auc' resulting in tfma.load_validation_result messages: "Metric not found." overall and for all products slices
Correcting to:
thresholds={
'auc':
produces correct validation failures as for several products (overall passes as >0.65 threshold) as below.
However evaluator.outputs['blessing'].get()[0].uri is NOT_BLESSED:
metric_validations_per_slice {
slice_key {
single_slice_keys {
column: "product"
bytes_value: "Consumer Loan"
}
}
failures {
metric_key {
name: "auc"
}
metric_threshold {
value_threshold {
lower_bound {
value: 0.65
}
}
}
metric_value {
double_value {
value: 0.6262196898460388
}
}
}
}
metric_validations_per_slice {
slice_key {
single_slice_keys {
column: "product"
bytes_value: "Mortgage"
}
}
failures {
metric_key {
name: "auc"
}
metric_threshold {
value_threshold {
lower_bound {
value: 0.65
}
}
}
metric_value {
double_value {
value: 0.618944525718689
}
}
}
}
metric_validations_per_slice {
slice_key {
single_slice_keys {
column: "product"
bytes_value: "Payday loan"
}
}
failures {
metric_key {
name: "auc"
}
metric_threshold {
value_threshold {
lower_bound {
value: 0.65
}
}
}
metric_value {
double_value {
value: 0.6383864879608154
}
}
}
}
metric_validations_per_slice {
slice_key {
single_slice_keys {
column: "product"
bytes_value: "Student loan"
}
}
failures {
metric_key {
name: "auc"
}
metric_threshold {
value_threshold {
lower_bound {
value: 0.65
}
}
}
metric_value {
double_value {
value: 0.6052306294441223
}
}
}
}

Setting auc threshold to 0.6 produces:
validation_ok: true
{evaluator.outputs['blessing'].get()[0].uri} = BLESSED

Hi @mshearer0,
Thank you for spotting the change from 'AUC' to 'auc', I'll look into it and fix it if I can.
I'm a little confused by the rest of your issue, could you explain further what the problem is? What is the overall auc for your model? Are you expecting it to pass or fail the validation?

Hi @drcat101 ,
Changing the threshold from AUC to auc resolves my problem. Just sharing my results to illustrate that several slices fail the 0.65 threshold but lowering to 0.6 they all pass and generate the validation_ok = true message in the book.


mrb_load_irep_file does not work on Windows

I am using msys2 with the environment mingw64.
main.cpp:
#include <iostream>
#include <stdio.h>

#include "mruby.h"
#include "mruby/dump.h"

int main()
{
  mrb_state *mrb = mrb_open();

  FILE *fp = fopen("test.mrb", "r");
  mrb_load_irep_file(mrb, fp);
  fclose(fp);

  if(mrb->exc)
  {
    std::cout << "error" << std::endl;
    mrb_print_error(mrb);
  }
  else std::cout << "no error" << std::endl;

  mrb_close(mrb);

  return 0;
}

test.rb
def foo
  @a = 'foobar'
end

Rakefile
task :build do
  Dir.chdir("mruby") do
    ENV['MRUBY_CONFIG'] = '../ruby_build_config.rb'
    `rake`
  end

  `g++ -I mruby/include main.cpp mruby/build/host/lib/libmruby.a -o test`
  `mruby/build/host/bin/mrbc test.rb`
end

task default: %[build]

ruby_build_config.rb
MRuby::Build.new do |conf|
  conf.toolchain

  conf.gem core: "mruby-bin-mrbc"
  conf.gem core: "mruby-bin-mruby"
  conf.gem core: "mruby-struct"

  conf.gembox "math"
  conf.gembox "metaprog"
  conf.gembox "stdlib"
  conf.gembox "stdlib-ext"

  conf.enable_bintest
  conf.enable_test
end

When I run the binary code on Windows I get:
error
(unkown):0: irep load error (ScriptError)

When I try the same code on Linux, it works:
no error

It may be because "test.mrb" is opened in text mode.
How about opening "test.mrb" in binary mode?
-  FILE *fp = fopen("test.mrb", "r");
+  FILE *fp = fopen("test.mrb", "rb");

It solves it!