Add fides deploy command

Closes #1115
Apologies in advance for this gigantic PR. Functionally not much existing code has changed outside of bug fixes, with virtually all added code being related to the new feature.
Code Changes
Feature-related

[x] add a new CLI command group called deploy (deploy up & deploy down)
[x] add a sample_project dir within the src/fides/ dir to house the related deployment files
[x] add a user-facing documentation page for the new command getting-started/sample_project
[x] add a dev-facing documentation page for the new command development/dev_deployment
[x] default the CLI settings port to 8080  so that the default values are valid to connect to a local instance
[x] add special CLI CI checks to test specific commands, currently including nox -s test_env and fides deploy
[x] remove git reset from the Dockerfile (after testing, this doesn't prevent CI checks from passing and prevents various local testing)
[x] add a nox -s "build(sample)" param that builds the privacy center/webserver and tags them with sample for local development of the feature

Auxiliary

[x] bug-fix: fix the Admin UI not being available at localhost:8080 (this issue exists on main)
[x] bug-fix: remove a data migration (mixed in with the alembic migrations) that broke the application when not in test mode
[x] unrelated: remove some lingering ops migrations that are no longer used
[x] unrelated: moved the static checks into their own workflow file for simplicity (they don't share code with the other checks)
[ ] make sure the privacy center gets built/pushed with a dev version to match the application fixing in #1580

Steps to Confirm

[ ] nox -s "build(sample)" ; fides deploy up --no-pull spins up a working application! Make sure to do this from another directory so that no local config gets accidentally loaded
[ ] admin UI is accessible at localhost:8080
[ ] privacy center is available at localhost:3001
[ ] docs pages match correct steps for getting up and running (sample_project.md and dev_deployment.md)
[ ] can build and install the source (not pip install -e .) and confirm that the required files exist!

Pre-Merge Checklist

[ ] All CI Pipelines Succeeded
Documentation Updated:

[x] documentation complete, or draft/outline provided (tag docs-team to complete/review on this branch)
[ ] documentation issue created (tag docs-team to complete issue separately)

[x] Issue Requirements are Met
[x] Relevant Follow-Up Issues Created
[ ] Update CHANGELOG.md

Description Of Changes
This is a new command that is designed to give our users the ability to quickly spin up a demo version of the entire application, allowing them to get hands-on immediately and see the value for themselves. This is made possible by including a sample_project dir (via the MANIFEST.in) with a custom compose file and more! The following is a usage example
Usage
fides> fides deploy
Loading config from: .fides/fides.toml
Usage: fides deploy [OPTIONS] COMMAND [ARGS]...

  Deploy a demo fides application.

Options:
  -h, --help  Show this message and exit.

Commands:
  down  Stops the demo fides application.
  up    Starts the demo fides application via docker...

User Experience (imagine the pretty colors)
fides> fides deploy up
...

███████╗██╗██████╗ ███████╗███████╗
██╔════╝██║██╔══██╗██╔════╝██╔════╝
█████╗  ██║██║  ██║█████╗  ███████╗
██╔══╝  ██║██║  ██║██╔══╝  ╚════██║
██║     ██║██████╔╝███████╗███████║
╚═╝     ╚═╝╚═════╝ ╚══════╝╚══════╝

Fides successfully deployed and running in the background!
Next steps:

- Run 'fides init' to set up your local command-line!

- Visit the Fides Admin UI running at http://localhost:8080
    (user=fidestest, password=Apassword1!)

- Visit the Fides Privacy Center running at http://localhost:3001
<EMAIL_ADDRESS>
- Example Postgres Database running at localhost:6432
    (user=postgres, password=postgres, db=postgres_example)

- Example Mongo Database running at localhost:27017
    (user=mongo_test, password=mongo_pass, db=mongo_test)

- Visit ethyca.github.io/fides for documentation.

Run `fides deploy down` to stop the application.

@NevilleS I'm stumped on why the policy rule creation is failing, but working in the test env
I'm thinking it has to do something with misconfiguration (duh) but I don't know enough about ops to know where...maybe its not getting stored in the database like its supposed to? or redis?

as it currently stands, the two problems are:
Mongodb not working -> I'm stuck here trying to figure out how best to include the required example data file. Because we're using the -f flag I can't use a relative path, and I can't inject a correct absolute patch at up time because docker compose doesn't allow that. However I think I can use the --project-dir flag to force the working dir to be what I want (docs)
Policy rules not working -> the script also fails on trying to create the policies (after creating the dsr policies). I am sure that this is a configuration error because it works via nox - test_env but I'm not sure where this might be falling down. Maybe the postgres connection is bad? But then why isn't it failing the policy creation? I'm not sure, but its weird. Might not to ask someone with more fidesops knowledge why/how this happens

solved the mongo problem by using the --project-directory trick

this is the current error:
RuntimeError: fides rule creation failed! response.status_code=200, response.json()={'succeeded': [], 'failed': [{'message': 'A StorageConfig with key default_local_storage does not exist', 'data': {'name': 'Example access Rule', 'key': 'default_access_policy_rule', 'action_type': 'access', 'storage_destination_key': 'default_local_storage', 'masking_strategy': None}}]}

@sanders41 @seanpreston I am fully stumped here, and not sure what else to try.
fides demo is failing due to the above error, a duplicate key constraint being broken, despite the fact that the code is using Policy.create_or_update(...) which should avoid this? And also that this does not happen when running either nox -s test_env or nox -s dev. I am really struggling to figure out why the new version of the dockerfile I wrote is causing this. No other data is being seeded, I've completely nuked the volumes, so how can it be trigger non-unique even though the database is empty?

@NevilleS FYI as of this last commit it is in a place to start playing around with it, but not at all polished and I'm sure there are bugs to find :)

Some findings via @NevilleS:
There is actually a data migration (part of the other migrations) that runs as part of the schema migrations. It checks for test_mode and only add default policies if test mode is true.
Action Items:
I think we need to remove those migrations and add them to the general seed script, we should also collapse all migrations into a single one for the launch of fides 2.0

the major test_mode bug has been fixed, shoutout @NevilleS for the catch!

@NevilleS I'm heading to bed, but actually to get the UI working I believe you'll need a nox -s "build(test)" now that I think about it, as the UI files don't get included for the dev build. But the backend is good to go, now working on a method to test it because this will be a pretty fragile command and I can see us letting it slip

Excellent @ThomasLaPiana, can confirm that the fides demo command succeeded for me, and it appears that the setup scripts worked too! I will play with it and get some ideas together for polish.

@NevilleS I think this merge conflicts is related to a bad change, should I expunge it? (vaguely recalled seeing @pattisdr mention it in slack)

Hmm, yeah, the conflict there is because we both edited those log lines. I don't think that commit is actually a bad change (it seems to have affected @pattisdr only whereas the rest of us are OK...) so I want to see if I can still get that working better. Tracking that in #1538

OK- resolved the merge conflict (and also got to the bottom of that other issue!). You're clear

Overall this is looking great; we've rounded the corner, clearly we'll be able to make this work. Here's my feedback:

Can we isolate the "sample project" data for this more? I'm thinking we add a new folder at src/fides/data/sample_project that includes a self-contained .fides/fides.toml, the docker-compose.yml file, and anything else we want to use for the sample project deployment
We'll likely need it to use the production image instead of fides:local because of that UI issue you mentioned
Similarly, we'll need to get a privacy center running, and I'd like it to have a custom config. We can do that with src/fides/data/sample_project/privacy_center_config/config.json and some docker mounting
Can we potentially have it not open up a new fides shell in the container? Since we're using the fides CLI to deploy the project, it stands to reason we can just use fides commands from there, too... 🤔

Looks like you already did that :)
correct, I'll test this and update steps to run
Yep, I'll confirm this works as well
Totally! In that case the user would be able to use their own local shell after a fides init. The only change we'd need to make here is to match up the default config values with the demo application

There are a few technical puzzles to solve here to get local <-> container communication working smoothly. The primary issue is versioning the containers, although I think the solution would go as follows:

Update the compose images to use ethyca/fides:demo or something
Fides deploy script checks what version the user is on, and pulls the corresponding image
There is then logic in the deploy code that tags the relevant images as ethyca/fides:demo (the privacy center should also get deployed using the same tag, but this hasn't been confirmed yet via a real deployment)

While on paper this isn't difficult to execute, it is exceedingly difficult to test in a convenient and repeatable fashion
Locally, I am unable to test this because when I spin up the application I get an error due to version mismatches with the prod image (it versions in a more neat way for test pypi), however when I use the dev image it works just fine but is then unable to test the UI as it isn't packaged with the dev image.

@NevilleS I got local <-> container synergy working using the dev image like so

cd into the fides repo root if you aren't already in there
`nox -s "build(dev)"
fides deploy up
mkdir ../test (Create a dir outside of the repo so we don't have any weirdness with config overlap, this is how a user would run)
cd ../test
fides init
fides status
Voila! Good connection 😄
fides push ../fides/demo_resources/ for an additional test

At the current stage, it's nice that everything is working as it is but the problem of the version mismatch still hasn't been solved. For this first iteration, it might make more sense to assume the user will be running within the shell

I think I'm following you here... I definitely would prefer to have the CLI respect the TOML generated on the user's machine and run in their local env (not from within the docker), but if we can't get the "version matching" working reliably at first, we can definitely work on that as a follow-up.

@NevilleS the main issue here is that everything will work on the user's machine in theory, but is difficult for me to test locally and quality is a focus here! I don't like shipping things I can't test first...I'm working on some solutions though to try

@NevilleS version issue is fixed! full steam ahead 🚀
nox -s "build(sample)" will now build the prod version instead of dev and the versions will match 🙂

@NevilleS two issues here I'd like to rack your brain on:

We don't have a good way to mount a convenient volume for privacy requests. When running the compose file we are using that directory as the working directory, and we can't have it both ways. So if a user is running fides deploy up the fides_uploads dir gets created in the python package directory, not in the local directory where the user is running their commands. This is by design as that is how everything else is possible, but makes things difficult here. I'm still brainstorming a solution but welcome to suggestions
I'm injecting the configs for the privacy center but still getting errors, doing to double check those
localhost:8080 still isn't showing the admin UI for me

@conceptualshark can you give me some guidance on documentation for this feature? I've got a skeleton written up but would love some professional help 🙂

We don't have a good way to mount a convenient volume for privacy requests.

My mind goes to two options...

In fides deploy, get the absolute directory of the package itself... So something like installed_package_path = fides.__path__ and export that as an ENV variable you use in sample_project/docker-compose.yml to construct some absolute paths for mounts
In fides deploy, copy the sample project files out from the installed package to the user's working directory (eg create a new ./sample_project/) and then everything can be relative paths in the same local directory

Assuming both work (big assumption!), I think the first option is best because it'll be easier for us to maintain. Part of me likes the transparency of populating the local dir with the source code and showing the compose file, sample app, SQL scripts, etc., but I also think it'll create a maintenance burden because that copy will be error-prone, users might edit some files causing deploys to fail, etc. etc.
So I'd lean towards option 1 here, if we can make that work. It seems plausible that the Python code can inspect itself to determine it's absolute path on the user's machine, so from there I think constructing absolute paths is hopefully viable

localhost:8080 still isn't showing the admin UI for me

This is the biggest issue for sure. I ran this down for a bit and found what I think is a smoking gun root cause. When the UI router runs, it tries to pull the HTML files from the installed package (see get_path_to_admin_ui_file in ui.py). However, when I hop into the fides:sample image, that installed package path doesn't include the ui-build directory it needs:
# ls /usr/local/lib/python3.10/site-packages/fides
__init__.py  __pycache__  _version.py  api  cli  ctl  data  py.typed  templates
# ls /fides/src/fides
__init__.py  __pycache__  _version.py  api  cli  ctl  data  py.typed  templates  ui-build

Running that down now, but I think the fix is pretty straightforward.

OK, I forgot to update this PR earlier, but here's a quick summary:

fixed the issue with the Admin UI packaging - that's not related to this PR, it's technically not bundling correctly on main either
fixed the privacy center build, and did a tiny customization to it thus far (just changed the name to "Sample Privacy Center")... much more to come there on styling 😄
tested & confirmed that privacy requests process successfully once approved!

🎉 TL;DR it works now, so now we just need to solve the problem of getting the request results out of the docker containers, remove a lot of TODOs I've littered in the code (sorry!), and polish the sample project a ton.
I think we should get this ready to merge to main ASAP though, since this branch is already quite behind a lot of other fixes that are happening

We don't have a good way to mount a convenient volume for privacy requests.

My mind goes to two options...

In fides deploy, get the absolute directory of the package itself... So something like installed_package_path = fides.__path__ and export that as an ENV variable you use in sample_project/docker-compose.yml to construct some absolute paths for mounts
In fides deploy, copy the sample project files out from the installed package to the user's working directory (eg create a new ./sample_project/) and then everything can be relative paths in the same local directory

Assuming both work (big assumption!), I think the first option is best because it'll be easier for us to maintain. Part of me likes the transparency of populating the local dir with the source code and showing the compose file, sample app, SQL scripts, etc., but I also think it'll create a maintenance burden because that copy will be error-prone, users might edit some files causing deploys to fail, etc. etc.
So I'd lean towards option 1 here, if we can make that work. It seems plausible that the Python code can inspect itself to determine it's absolute path on the user's machine, so from there I think constructing absolute paths is hopefully viable

Correct, Python can figure out where it's running/absolute file paths etc, but modifying the users environment directly feels dangerous to me, even if its just something small like setting an environment variable
I agree #1 is probably the easiest though

got the docs cleaned up, working on the volume mounting problem now...

Oh, for the ENV var setting, you should just set it temporarily in the shell commands, not persist it. So for example:
docker compose -e SAMPLE_PROJECT_PATH=/use/lib/...

Then in compose you can do the mounts using ENV expansion like:
volumes:
  - ${SAMPLE_PROJECT_PATH}/postgres_sample.sql:/init.d/...

I think that'd be OK. Alternatively you could do the flip side of this and pass the working dir in (e.g. ACTUAL_WORKING_DIR) so that you can still use the --working-directory arg and relative paths for everything, but just use that env variable to set the path for the uploads folder mount...
I'm pretty sure either will work and shouldn't be too messy if it's scoped to the shell commands.

currently working to fix this final CI error, but locally everything is working great 🚀

@NevilleS updated the config echo color here, this feels ready for merge to me


Turkish translation added

I added a Turkish dictionary file for project.

Thank you so much for your contribution! Your Pull Request is greatly appreciated and will help make our project even better.


ps hang cause wait_timeout don't return timeout

A progress named qemu-kvm in D stat, caused ps ax stuck, a subprocess used ps can't successfully return timeout. Any ideas?

Does the same thing happen if you replace ps ax with sleep 100 or something like that?
A minimal example program that reproduces the issue would help a lot.

Normally wait_timeout should work the same regardless of whether the subprocess is a simple sleep or an arbitrarily complex command. If you could provide an example program that reproduces this kind of behavior, I would like to look into it.

@light4 Any news on this issue? I'd like to help, but I cannot do so without an example program that reproduces the issue.


Replace several POWER8/9 C kernels with their gcc7-generated assembly versions

to work around internal compiler errors encountered with gcc 4.8 and 5.4, and wrong code generation by gcc 8.3.0. Fixes #2254.

caxpy may need some editing of the (former) CNAME entry as it is also used to create caxpyc

Looks like gcc 5.4 in POWER9 mode is going to blow up on each and every kernel that uses vector intrinsics - probably better to change Makefile.system to exclude POWER9 from the target list when the compiler is too old (gcc 4.8.5 does not even handle -mcpu=power9).


FetchResult#toPromise

Resolves #131

This would be great when writing tests!

You can't change the branch a PR merges into so moving this to #180


chore: remove lerna

This removes Lerna entirely. Publishing now happens via npm. Stub test scripts removed.
Closes #704

Current dependencies on/for this PR:

main

PR #706   👈

This comment was auto-generated by Graphite.

Have you verified whether it's going to attempt publishing already published packages or has logic to skip that?

@naugtur So, yeah, npm publish won't check to see if the thing is already published.  We could either a) keep lerna or b) write a wrapper script to check which workspaces haven't yet been published and then only publish those. b seems easy...

Somehow, when I ran publishing, lerna bumped things by 2 minor versions. Not just tofu, but it did that to all packages. eg. lavapack https://www.npmjs.com/package/@lavamoat/lavapack?activeTab=versions
I'd rather have something simpler in place that is not capable of changing anything.

Closed in lieu of #727


Rename launch files

Cleanup for nao_robot.
This involves removing several launch files in nao_description
Furthermore, launch files inside nao_apps are renamed. However,  please note that I only renamed the launch files. I did not rename python scripts in order to avoid wrong python imports.
@k-okada Could you please try the nao_apps folder before merging? I am not familiar with the code base inside of it.
thx

+1


skipped bad example when using acid dataset

H, thank you for open source your great work!
I tried to train on the whole acid dataset(161G) but i got info like this:

This happend frequently so I wonder if it's normal.
By the way, I got the full dataset from PixelSplat author and the files shuld be OK.
Thanks in advance!

Hi @xingyoujun, thanks for your interest in our work.
Skipping images with undesired/invalid shapes is an expected behaviour, which is achieved by https://github.com/donydchen/mvsplat/blob/378ff818c0151719bbc052ac2797a2c769766320/src/dataset/dataset_re10k.py#L156-L164

I use acid too. I found that after 60k steps, it often skip bad samples. Should I stop training?


I get problem when I run my tensorflow codes on multi gpu

I still have problem after I add
sess_config = tf.ConfigProto(allow_soft_placement=True, log_device_placement=True)
sess_config.gpu_options.allow_growth = True
with tf.Session(config=sess_config) as sess:

I use Tensorflow-gpu 1.12.0.I run BIDAF model on multi gpu and I get the problem below.

Device mapping:
/job:localhost/replica:0/task:0/device:XLA_CPU:0 -> device: XLA_CPU device
/job:localhost/replica:0/task:0/device:XLA_GPU:0 -> device: XLA_GPU device
/job:localhost/replica:0/task:0/device:GPU:0 -> device: 0, name: GeForce GTX 1080 Ti, pci bus id: 0000:3b:00.0, compute capability: 6.1
Traceback (most recent call last):
  File "/home/home1/dmyan/.conda/envs/py_env/lib/python3.6/site-packages/tensorflow/python/client/session.py", line 1334, in _do_call
    return fn(*args)
  File "/home/home1/dmyan/.conda/envs/py_env/lib/python3.6/site-packages/tensorflow/python/client/session.py", line 1317, in _run_fn
    self._extend_graph()
  File "/home/home1/dmyan/.conda/envs/py_env/lib/python3.6/site-packages/tensorflow/python/client/session.py", line 1352, in _extend_graph
    tf_session.ExtendSession(self._session)
tensorflow.python.framework.errors_impl.InvalidArgumentError: Cannot assign a device for operation tower_0/passage_encoding/bidirectional_rnn/fw/fw/Assert/Assert: Could not satisfy explicit device specification '/device:GPU:0' because no supported kernel for GPU devices is available.
Registered kernels:
  device='CPU'

         [[{{node tower_0/passage_encoding/bidirectional_rnn/fw/fw/Assert/Assert}} = Assert[T=[DT_STRING,DT_INT32, DT_STRING, DT_INT32], summarize=3, _device="/device:GPU:0"](tower_0/passage_encoding/bidirectional_rnn/fw/fw/All, tower_0/passage_encoding/bidirectional_rnn/fw/fw/Assert/Assert/data_0, tower_0/passage_encoding/bidirectional_rnn/fw/fw/stack, tower_0/passage_encoding/bidirectional_rnn/fw/fw/Assert/Assert/data_2, tower_0/passage_encoding/bidirectional_rnn/fw/fw/Shape_1)]]

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "run.py", line 408, in <module>
    run()
  File "run.py", line 399, in run
    multi_gpu_train(args)
  File "run.py", line 249, in multi_gpu_train
    model = RCModel_ngpus(args, data)
  File "/home/home1/dmyan/codes/tensorflow/rc_model_multi_gpu.py", line 80, in __init__
    self.sess.run(tf.global_variables_initializer())
  File "/home/home1/dmyan/.conda/envs/py_env/lib/python3.6/site-packages/tensorflow/python/client/session.py", line 929, in run
    run_metadata_ptr)
  File "/home/home1/dmyan/.conda/envs/py_env/lib/python3.6/site-packages/tensorflow/python/client/session.py", line 1152, in _run
    feed_dict_tensor, options, run_metadata)
  File "/home/home1/dmyan/.conda/envs/py_env/lib/python3.6/site-packages/tensorflow/python/client/session.py", line 1328, in _do_run
    run_metadata)
  File "/home/home1/dmyan/.conda/envs/py_env/lib/python3.6/site-packages/tensorflow/python/client/session.py", line 1348, in _do_call
    raise type(e)(node_def, op, message)
tensorflow.python.framework.errors_impl.InvalidArgumentError: Cannot assign a device for operation tower_0/passage_encoding/bidirectional_rnn/fw/fw/Assert/Assert: Could not satisfy explicit device specification '/device:GPU:0' because no supported kernel for GPU devices is available.
Registered kernels:
  device='CPU'

         [[node tower_0/passage_encoding/bidirectional_rnn/fw/fw/Assert/Assert (defined at /home/home1/dmyan/codes/tensorflow/layers/basic_rnn.py:51)  = Assert[T=[DT_STRING, DT_INT32, DT_STRING, DT_INT32], summarize=3, _device="/device:GPU:0"](tower_0/passage_encoding/bidirectional_rnn/fw/fw/All, tower_0/passage_encoding/bidirectional_rnn/fw/fw/Assert/Assert/data_0, tower_0/passage_encoding/bidirectional_rnn/fw/fw/stack, tower_0/passage_encoding/bidirectional_rnn/fw/fw/Assert/Assert/data_2, tower_0/passage_encoding/bidirectional_rnn/fw/fw/Shape_1)]]

Caused by op 'tower_0/passage_encoding/bidirectional_rnn/fw/fw/Assert/Assert', defined at:
  File "run.py", line 408, in <module>
    run()
  File "run.py", line 399, in run
    multi_gpu_train(args)
  File "run.py", line 249, in multi_gpu_train
    model = RCModel_ngpus(args, data)
  File "/home/home1/dmyan/codes/tensorflow/rc_model_multi_gpu.py", line 74, in __init__
    self._build_graph()
  File "/home/home1/dmyan/codes/tensorflow/rc_model_multi_gpu.py", line 87, in _build_graph
    self._encode()
  File "/home/home1/dmyan/codes/tensorflow/rc_model_multi_gpu.py", line 102, in _encode
    self.sep_p_encodes, _ = rnn('bi-lstm', self.p_emb, self.p_length, self.hidden_size)
  File "/home/home1/dmyan/codes/tensorflow/layers/basic_rnn.py", line 51, in rnn
    cell_bw, cell_fw, inputs, sequence_length=length, dtype=tf.float32
  File "/home/home1/dmyan/.conda/envs/py_env/lib/python3.6/site-packages/tensorflow/python/ops/rnn.py", line 439, in bidirectional_dynamic_rnn
    time_major=time_major, scope=fw_scope)
  File "/home/home1/dmyan/.conda/envs/py_env/lib/python3.6/site-packages/tensorflow/python/ops/rnn.py", line 651, in dynamic_rnn
    [_assert_has_shape(sequence_length, [batch_size])]):
  File "/home/home1/dmyan/.conda/envs/py_env/lib/python3.6/site-packages/tensorflow/python/ops/rnn.py", line 646, in _assert_has_shape
    packed_shape, " but saw shape: ", x_shape])
  File "/home/home1/dmyan/.conda/envs/py_env/lib/python3.6/site-packages/tensorflow/python/util/tf_should_use.py", line 189, in wrapped
    return _add_should_use_warning(fn(*args, **kwargs))
  File "/home/home1/dmyan/.conda/envs/py_env/lib/python3.6/site-packages/tensorflow/python/ops/control_flow_ops.py", line 159, in Assert
    return gen_logging_ops._assert(condition, data, summarize, name="Assert")
  File "/home/home1/dmyan/.conda/envs/py_env/lib/python3.6/site-packages/tensorflow/python/ops/gen_logging_ops.py", line 52, in _assert
    name=name)
  File "/home/home1/dmyan/.conda/envs/py_env/lib/python3.6/site-packages/tensorflow/python/framework/op_def_library.py", line 787, in _apply_op_helper
    op_def=op_def)
  File "/home/home1/dmyan/.conda/envs/py_env/lib/python3.6/site-packages/tensorflow/python/util/deprecation.py", line 488, in new_func
    return func(*args, **kwargs)
  File "/home/home1/dmyan/.conda/envs/py_env/lib/python3.6/site-packages/tensorflow/python/framework/ops.py", line 3274, in create_op
    op_def=op_def)
  File "/home/home1/dmyan/.conda/envs/py_env/lib/python3.6/site-packages/tensorflow/python/framework/ops.py", line 1770, in __init__
    self._traceback = tf_stack.extract_stack()

InvalidArgumentError (see above for traceback): Cannot assign a device for operation tower_0/passage_encoding/bidirectional_rnn/fw/fw/Assert/Assert: Could not satisfy explicit device specification '/device:GPU:0' because no supported kernel for GPU devices is available.
Registered kernels:
  device='CPU'

         [[node tower_0/passage_encoding/bidirectional_rnn/fw/fw/Assert/Assert (defined at /home/home1/dmyan/codes/tensorflow/layers/basic_rnn.py:51)  = Assert[T=[DT_STRING, DT_INT32, DT_STRING, DT_INT32], summarize=3, _device="/device:GPU:0"](tower_0/passage_encoding/bidirectional_rnn/fw/fw/All, tower_0/passage_encoding/bidirectional_rnn/fw/fw/Assert/Assert/data_0, tower_0/passage_encoding/bidirectional_rnn/fw/fw/stack, tower_0/passage_encoding/bidirectional_rnn/fw/fw/Assert/Assert/data_2, tower_0/passage_encoding/bidirectional_rnn/fw/fw/Shape_1)]]

AssertOp's registered kernel is CPU, so you can not assign it on GPU.

Hello @demeiyan , please let us know if @lxl910915 comments has addressed you query sufficiently. Thanks.

no,I can not run bi-lstm on multi gpu. And do not know how to run.

@demeiyan : I wonder how you solved the problem then? Remove os.environ["CUDA_VISIBLE_DEVICES"] did not help for my case!
Thx


Creating a notification from a plugin

I believe I am trying the same thing as @martiendt in #86, but it isn't working for me in my code below. There are no errors showing up but neither is a notification. I have tried replacing the calls to Vue.prototype.$notify with Vue.notify but it does not make any difference. I am noticing that in the Vue.js devtools console in Chrome there isn't an add event emitted, or at least not one noticed by the devtools.
@martiendt, if you got it to work, then I guess I must be calling something incorrectly.
let fetchPlugin = {}
fetchPlugin.install = function(Vue, options){
    Vue.prototype.$fetch = {

        get(url, processingFunction, show_loading=false, extra_options={}){
            if(show_loading){

                // This call does not work
                Vue.prototype.$notify({
                    group: 'loading',
                    title: 'Loading...',
                    type: 'success'
                })
            }

            let default_options = {
                credentials: 'same-origin',
            }
            options = Object.assign(default_options, extra_options)
            fetch(url, options)
                .then(function(response){
                    if(show_loading){

                        // This call works
                        Vue.prototype.$notify({
                            group: 'loading',
                            clean: true
                        })
                    }
                    if(!response.ok){ throw response }
                    return response.json()
                })
                .then(processingFunction)
                .catch(err => {
                    console.log(err)

                    // This call also works
                    Vue.prototype.$notify({
                        group: 'fetch',
                        title: 'An error occurred',
                        type: 'error'
                    })
                })
        }
    }
}
export default fetchPlugin

Did you add notification container to your dom? Otherwise plugin does not know where you plonk your notifications

Yes, that should be ok. Because the call does work when I put it inside the .then function...
Or the element maybe doesn't exists yet for some reason?

Or the element maybe doesn't exists yet for some reason?

This can be the case. You can test it by putting container as close to your root as possible and make for example a 1 sec delay before showing notification (after that you can narrow down what exactly is going on)

Thanks @euvl, I managed to figure it out with your reply! I made it so that another call would put up an notification but only when I pressed the button, then the notification did work. So I looked at when I did the first call and that was in the created lifecycle hook of the root element. The notification element doesn't exist at that point in time of course. I changed that function to be called on the mounted lifecycle hook and then everything works great!

awesome, happy to help 👍


How to enable/disable input box?

I'd like to control enable/disable input box.
How to do?
Now, i write like this.
$('.typeahead).typeahead(.....);
$('.typeahead).on('typeahead:selected', function ...)
...
$(selector)
.removeAttr('disabled')
.prev().remove()
.removeClass('.typeahead')
.addClass('.typeahead');
...
But, I don't know .prev().remove() is sufficient or not.
Could you please support this api?

I'm curious why this was closed and if there is a proper way to dynamically disable/enable the input field to which typeahead.js is applied.  I'm using Vue.js to control when a field is disabled.
<input type="text" v-bind:value="person.name" v-bind:disabled="disabled">
What seems to be happening is that the input associated with tt-input seems to update properly, but the tt-hint input field does not.  It seems to stay in the initial state of tt-input when typeahead was applied.
Since tt-input is transparent, the ghosting doesn't update properly. eg when disabled turns to false, tt-input is enabled, but tt-hint isn't, so the input field stays greyed out.


Error While Using Subtle-cli

when using subtle-cli to download subtitles of an entire directory the following error happens
[Hash]       scrubs.2x02.my_nightingale.dvdrip_xvid-fov.avi
[Hash]       scrubs.2x01.my_overkill.dvdrip_xvid-fov.avi
 Unhandled Exception: System.Net.WebException: Request timed out    at Subtle.Model.TaskExtensions.<WithTimeout>d__01.MoveNext() --- End of stack trace from previous location where exception was thrown ---    at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)    at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)    at Subtle.Model.OSDbClient.<SearchSubtitlesAsync>d__10.MoveNext() --- End of stack trace from previous location where exception was thrown ---    at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)    at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)    at Subtle.Cli.Program.<SearchSubtitleAsync>d__7.MoveNext() --- End of stack trace from previous location where exception was thrown ---    at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)    at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)    at Subtle.Cli.Program.<MainAsync>d__6.MoveNext() --- End of stack trace from previous location where exception was thrown ---    at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)    at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)    at Subtle.Cli.Program.Main(String[] args)
this error seem to be so frequent while using subtle-cli (On subtle 0.41)
My Opensubtitles statistics

This is resolved in v0.4.2.


Query/Filter by date for List view of calendar

Make it so that the table in Calendar's List view can be filtered by date by using the filter cells/fields of 'Start Date' and 'End Date'. Use moment.js to deal with the date time stuff so it's convenient for us all. Might need more details on what kind of inputs the user can put for the filter cells/fields.
Helpful links:

https://material-table.com/#/docs/get-started
https://momentjs.com/

@thaigillespie did you end up doing this?

Nah I’ve been doing other things so I never got to that : p
On Mon, Aug 17, 2020 at 7:11 PM Godwin Pang<EMAIL_ADDRESS>wrote:

Reopened #78
https://urldefense.com/v3/__https://github.com/HKN-UCSD/hkn-member-portal/issues/78__;!!Mih3wA!Vt6ZJGbQ7ZBn5UY2W6ZEXfDtnpKaLQ440YALVQaQ8XJ4SsfcV0_bOX4cujD_EQlF$
.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://urldefense.com/v3/__https://github.com/HKN-UCSD/hkn-member-portal/issues/78*event-3664252016__;Iw!!Mih3wA!Vt6ZJGbQ7ZBn5UY2W6ZEXfDtnpKaLQ440YALVQaQ8XJ4SsfcV0_bOX4cuhRhOgIf$,
or unsubscribe
https://urldefense.com/v3/__https://github.com/notifications/unsubscribe-auth/AKGH3VCNLAX7V4FRRGNDCL3SBHPMZANCNFSM4OKHD56Q__;!!Mih3wA!Vt6ZJGbQ7ZBn5UY2W6ZEXfDtnpKaLQ440YALVQaQ8XJ4SsfcV0_bOX4cumlGSMIT$
.


The right way to disable C++11+ tests on C++03

Superseded by: https://github.com/boostorg/odeint/pull/63


[CI] Adapting Dockerfile to humble

Docker files and repos modified to run in humble.

There is already an effort to migrate to humble here https://github.com/open-rmf/rmf/pull/165 but a blocker seems to be related to libceres.

I have also looked into that PRand also investigated the problem and commented in this issue https://github.com/open-rmf/rmf_traffic_editor/issues/439
I think that while nightly is using<EMAIL_ADDRESS>and that action installs libc++-dev by default, we are gonna get this conflict until libgoogle-glog-dev is updated to libunwind-14-dev or we remove the libc++-dev from<EMAIL_ADDRESS>(related: https://github.com/ros-tooling/setup-ros/issues/506)
My suggestion would be to proceed with updating the two files in this PR and then keep the blocker only for the nightly.yaml file.
What do you think?

I see thanks for explaining that.
Would it be more streamlined if we merge this PR into the humble-image branch first? So target humble-image and not main.

I see thanks for explaining that. Would it be more streamlined if we merge this PR into the humble-image branch first? So target humble-image and not main.

True.
Then I think that my modifications to the Dockerfile should go to https://github.com/open-rmf/rmf_deployment_template/blob/main/rmf/rmf.Dockerfile  right?


Command line arg lib

Adds command line argument parsing library to kore-parser.
Main.hs now uses optparse-applicative
This library supports adding tab completion in the shell, but this would require additional install scripts so I have held off for now.
I tried my best to emulate the flags that were already present but those are easy to change if need be.

Looks reasonable to me, but I defer judgement to @virgil-serbanuta as he wrote the original code.  It is not clear to me how the default values (all flags are enabled by default) are set in the PR, although they seems to the set, as the tests pass.

I will add an indication of default values to the help text and documentation


JLedSequence.Stop() not working

I see an issue in JLedSequence.Stop()
It stops, but after calling .Update() few times, sequence start again.
I think a is_running_ = false; is missing in function .Stop()
I suggest to add this line.

Indeed that was missing. Thanks for the report. Fixed with 4.12.2


Support importing of the eth/geth keystore files

Would this be possible to support importing the standard keystore format?

Hi @tymat, Yeah this is something that I've thought about. It would be possible but doing this removes the ability to backup your wallet using only the seed, and hence creates UX issues. This is the main reason I've been hesitant in supporting it. Another way to approach it is to move the HD key generation out and have the keystore be agnostic in the keys it carries, but this would be a larger refactoring.

+1

https://github.com/ethereumjs/keythereum I haven't tried it yet, but seems to handle geth accounts.
I was planning to use both libraries.

Hi @slothbag, I believe the reason that keythereum does not allow for importing geth keystore files in the browser is that the browser cannot easily handle the number of hashing rounds in the scrypt key derivation function. If that is the case then it is a browser limitation and therefore would be hard to include in lightwallet. I'm not sure about this though.

How does MyEtherWallet.com accomplish this task then?

@tymat: Ok, if they can do it then browser limitations is clearly not the reason. We're in the middle of a big refactoring of lightwallet to make it more flexible and modular and less monolithic. We could probably support this and leave the UX concerns regardning HD/non-HD to the higher-level dapp developer.

This is huge need when you think about it.    ..
at least   if not import how about Import/Transfer to a web controlled wallet'.
I need a way to let my users Scan in tokens from a private key into this web wallet or my project is dead in the water. :(

+1 to this
Also, am i missing something or how would a user returning to a dApp regenerate their keystore, with the address they were given on creating the keystore (ie. singing up to dApp)?
Currently i am storing the serialized keystore against the user record in my storage contract, when they sign up, and then on login returning it to the client to be deserialized and regenerate the keystore object. This is of course somewhat unsecure and dirty!


floorplan: cast uuid type to string

UUID's are not compatible with data frames. This should help.

/retest

/retest

/retest

/retest
I think with this stability of tests, enforcing green tests for trivial PRs is not the best policy.

/retest

/retest

Ugh :( shutzbot is fine now, let's root for iqe xD

Thanks!


Added gnuplot output files patterns to TeX.gitignore

Thanks, but it would be great if you could comment with some documentation describing these rules so that we understand where they came from in the future. Also the PR will need updating so it can be merged. Thanks!

@altaris unfortunately this isn't mergeable currently, and there are some questions outstanding which we'd like to clarify before merging :cry:
I need to clean up stale pull requests at some stage, and will close this out if I don't hear anything before Friday 26th February.


Getting Error when building Module was compiled with an incompatible version of Kotlin. The binary version of its metadata is 1.7.1, expected version is 1.5.1.

I am using kotlin 1.7.1 version and it seems telephony is using older version
e: /Users/khanakia/Downloads/rewardsapp1/build/telephony/.transforms/4c1ab4cc2813cf65185dd5b1cf346083/transformed/out/jars/classes.jar!/META-INF/telephony_release.kotlin_module: Module was compiled with an incompatible version of Kotlin. The binary version of its metadata is 1.7.1, expected version is 1.5.1.
e: /Users/khanakia/.gradle/caches/modules-2/files-2.1/org.jetbrains.kotlin/kotlin-stdlib-jdk8/1.7.10/d70d7d2c56371f7aa18f32e984e3e2e998fe9081/kotlin-stdlib-jdk8-1.7.10.jar!/META-INF/kotlin-stdlib-jdk8.kotlin_module: Module was compiled with an incompatible version of Kotlin. The binary version of its metadata is 1.7.1, expected version is 1.5.1.
e: /Users/khanakia/.gradle/caches/modules-2/files-2.1/androidx.annotation/annotation/1.5.0/857678d6b4ca7b28571ef7935c668bdb57e15027/annotation-1.5.0.jar!/META-INF/annotation.kotlin_module: Module was compiled with an incompatible version of Kotlin. The binary version of its metadata is 1.7.1, expected version is 1.5.1.
e: /Users/khanakia/.gradle/caches/modules-2/files-2.1/org.jetbrains.kotlin/kotlin-stdlib-jdk7/1.7.10/1ef73fee66f45d52c67e2aca12fd945dbe0659bf/kotlin-stdlib-jdk7-1.7.10.jar!/META-INF/kotlin-stdlib-jdk7.kotlin_module: Module was compiled with an incompatible version of Kotlin. The binary version of its metadata is 1.7.1, expected version is 1.5.1.
e: /Users/khanakia/.gradle/caches/modules-2/files-2.1/org.jetbrains.kotlin/kotlin-stdlib/1.7.10/d2abf9e77736acc4450dc4a3f707fa2c10f5099d/kotlin-stdlib-1.7.10.jar!/META-INF/kotlin-stdlib.kotlin_module: Module was compiled with an incompatible version of Kotlin. The binary version of its metadata is 1.7.1, expected version is 1.5.1.
e: /Users/khanakia/.gradle/caches/modules-2/files-2.1/org.jetbrains.kotlin/kotlin-stdlib-common/1.7.10/bac80c520d0a9e3f3673bc2658c6ed02ef45a76a/kotlin-stdlib-common-1.7.10.jar!/META-INF/kotlin-stdlib-common.kotlin_module: Module was compiled with an incompatible version of Kotlin. The binary version of its metadata is 1.7.1, expected version is 1.5.1.

Solved after upgrading flutter


Is there any in build functionality for edit using popup like add.

If Is there any in build functionality for edit using popup like add then please share documentation for that.

@ManrajParmarSipl no, but this will be a new feature on react-bootstrap-table2 but it still work in progress
https://github.com/react-bootstrap-table/react-bootstrap-table2


update README with conda-forge package

There's now a Linux/macOS conda-forge package of hole2 available on conda-forge https://anaconda.org/conda-forge/hole2
The README should be updated with installation instructions
 conda install -c conda-forge hole2

(after testing that this actually works).

And add a conda package badge.


$exists tests are incorrect

$exists is an in memory filter. Instead of querying like this selector: {'foo': {'$exists': true}},.
We actually need to declare an index as well so it needs to be something like this
selector: {_id" : {"$gt" : null}, 'foo': {'$exists': true}},.
This is the reason why the Cloudant/CouchDB tests fail.

Yeah, I can't remember why I implemented that differently. :/ Definitely worth fixing.

Cool. I'll take a look at this next week.


posities met tag is translated wrong

Used Shopware Version: <IP_ADDRESS>
"Line item tag" is currently translated to "posities met dag".
"posities met dag" however means "positions with days".
This should be corrected to "posities met tag"
actual version in use is Language pack 2.2.0, this version is not selectable in the issuetracker

[public] Shopware automated response: This issue is linked to the internal issue: LAN-98
[created from LAN-98, comment 514152]


Pretrained on 224x224 and finetuned on 364x364 in BLIP2 retrieval finetuning?

Seems like there is resolution difference between pretraining and finetuning. And what's the reason for that?
Thank U!

I guess the model trainer is going to resize the input to 224 nevertheless.

I guess the model trainer is going to resize the input to 224 nevertheless.
You can take a look at the init implementation of Blip2OPT class.

seems not like that

I guess the model trainer is going to resize the input to 224 nevertheless.
You can take a look at the init implementation of Blip2OPT class.

seems not like that

here?

I guess the model trainer is going to resize the input to 224 nevertheless.
You can take a look at the init implementation of Blip2OPT class.

seems not like that

here?

img_size was set to 364 in yaml, 224 is just a default value..

You are right. My misunderstanding. Sorry.

Increasing the image resolution leads to improved performance on downstream tasks.

Increasing the image resolution leads to improved performance on downstream tasks.

@LiJunnan1992
Thanks for the reply. And I want to know that, whether the vit initialized with the same weight between 224x224 pretrain and 364x364 finetune? Seems like it's two different vit between pretrain and finetune

The finetuned 364 ViT is initialized from the pre-trained frozen 224 ViT. The position encodings are interpolated.


Segmentation fault analyzing big file

Pint Version: 0.2.3
PHP Version: 8.0.20 / 8.1.7

Description:
Running pint on a project with a very big seeder (10768002 bytes) produces a segfault.
Steps To Reproduce:
Create a seeder with a collection of very big arrays. My case was with 160 insert statements, each with 500 array items, like the following.
DB::table('cities')->insert([
    [
        'id' => 1,
        'nombre' => 'Guangzhou',
        'pais_id' => '1',
    ],
...
    [
        'id' => 500,
        'nombre' => 'Fotang',
        'pais_id' => '1',
    ],
]);

I know we can exclude specific files with configuration rules, but it crashes... I'll expect the tool can check file size before checking a file and issue a warning, skipping the file and following with the next files. It's difficult to known what file was causing the segfault, because the tool doesn't provide an option to show files being analyzed or log progress to a file.

I created a gist with an example of a failing file: https://gist.github.com/neurotools/b40369aa90207d80d4f081591d5bf3f0
My local PHP CLI is configured with a 512 MB memory limit.


Implement inclusive open source community practices

In "Building a (Better) Open Source Community", Lisa Tagliaferri offers suggestions on how to create a more inclusive and equal open source repo.
Actionable items for CockroachDB docs:

Add a contributors.md with a list of everybody who has contributed to our docs / running our docs
Create a Code of Conduct
Possibly review / rework the Contributing.md file and docs that help people get started

Notes:
Building a (Better) Open Source Community
Presented by Lisa Tagliaferri (Digital Ocean)
Good communities:
Inclusion
Equity
Bad communities:
Gatekeeping
Lack of mentoring
Bias
GitHub Repos:
10M in 2013
1000M in 2018
Open Source participation:
95% men
3% women
1% nonbinary
1% non-binary
Community-focused docs help create inclusion / equity (good thing we already do this!). Should include the following:
Read.me
Project summary, goals, install/dependencies
This might be the only doc people read. Make sure most important stuff is there!
Pytorch is a good example
Contributing.md
Guidelines, best practices, style
Jupyterlab is a good example
Contributors.md
Recognize contributors, incentivize engagement, ack people who might help/contribute outside of the repo
Ack people equally for both docs and code
This can be alphabetized by role, team, or name
Code-of-conduct.md
Set expectations, reflect values
Drupal CoC, Geek Feminism CoC are good examples
End-user docs
Tutorials, guides, FAQs, release notes, planned features
In general, make sure doc language is welcoming and unbiased
A11y compliant
Remember screen readers, country agnostic, no cultural expectations
Issue tracking
Should be concrete tasks
Include labels
Foster conversation / collaboration
Prioritize issues based on need and bandwidth
Code reviews should be iterative feedback
Show gratitude for newcomers
Give general and holistic feedback first
Use line-by-line comments
Avoid nits in the first reviews
Encourage multiple voices to empower community
Funding open source
Grant funding: Mozilla Grants, Sloan Foundation, Google Summer of Code, Python Software Foundation
Pitch impact, traction, value. State how funds will be used.
Hacktoberfest
4 PRS for GitHub public repos
hacktoberfest label
Misc. notes
Good first PRs: do_community/haiku
do.co/w4do - Teach others and grow as an author while supporting non-profit / charity

@lnhsingh
I can provide a CODE_OF_CONDUCT.md template from the above-mentioned resource to start with which can be revised and corrected in the process. (Looking to contribute to the Hacktoberfest2020 event as well 😄 )
The required Code of Conduct can be adapted from any of the following:

Contributor Covenant - code of conduct
Drupal - code of conduct
Geek Feminism - code of conduct
Go Lang - code of conduct
Or any other more appropriate code of conduct guidelines.

Hi @aniketrb-github, thank you for volunteering to take this issue! Since I filed this, we've actually developed a Code of Conduct for our community slack channel that I think can also be repurposed here. Can you create a CODE_OF_CONDUCT.md file with this info? Most of the information should be the same-- the only difference is on the last bullet point. For our doc, it should be:
Finally, if you notice or are experiencing violations of this code of conduct or harassment of any kind, please reach out to @jseldess on GitHub or email docs[at]cockroachlabs.com to help resolve the situation.

sure @lnhsingh . I can tweak it as you've suggested. Thanks.
And, can we add a hacktoberfest label to this task? Just asking if it is doable. :D

Hi @lnhsingh
Currently, for me, the Citizen CoC link from CockroachDB Community Slack CoC is not working/reachable.1
Can you please confirm this once and let me know if whether to exclude it or to keep it as is.


URL path variable in custom condition

This is more a question than an issue.
I have the following custom action on my ModelViewSet:
@action(methods=["post", "delete"], detail=True, url_path="assign/(?P<userid>[^/.]+)")
def assign(self, request, pk=None, userid=None):

And on the AccessPolicy I have a method:
def self_or_manager(self, request, view, action) -> bool:

I want to access the userid in my condition method. Is this possible?
Thanks,
Tobias

There might be multiple ways for you to do this. Your self_or_manager method is passed both the view instance and the request object:

you can access view.kwargsdict  which will contain userid
you can inspect the request object and parse the URL path

The first option is probably simpler.

Thanks, view.kwargs is what I was looking for.
And thanks for developing this. It is really helpful.


Improve user control of paging

This allows users to avoid a pager by emptying out the PAGER environment variable (setting PAGER=""), only defaulting to less with posix flags (not supported by busybox less) if PAGER is unset.
Closes #293

I haven't yet added a top level --no-pager flag but would be willing to make that change if desirable.

I've preferred not breaking/changing existing pager behavior over supporting the simplest pager by default but that may be worth reconsidering if we think running in a busybox environment is more likely.
Also, I'm not sure how paging works on windows but this probably doesn't make the situation any worse. Possibly, Windows has the more command (https://superuser.com/questions/426226/less-or-more-in-windows/426229).

Thanks for all the feedback @sh0rez. I think this is ready for a second pass now :)


Remove "channel" from subscribe/unsubscribe bookends

We should remove the word "channel" from subscribe/unsubscribe bookends in the message feed. Just having the channel name is enough.

CZO thread

@zulipbot claim

@zulipbot claim


e2e Prevent namespace terminating errors when looping on 1 test

when looping on one test, it is possible to get namespace terminating errors when installing application.

if an app failed to backup.. it would not have gotten to uninstall phase.. and will need to implement namespace removal before each run or after as well.


Add script to clean up old versions of challenge files from bucket

Actually this is pretty hard to do since previous years' challenge files need to stay in the repo, so just do it manually for now


Update introduction.md

@dcstwh
IMPORTANT: When the changes are ready for publication, add a #sign-off comment to signal that the PR is ready for the review team to merge.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team


Add an option to whitelist GH organizations

I've added the documentation.
If you make a release, please be sure to use npm 2 to make the build, as the gulpfile doesn't support npm 3 and the flat structure for dependencies.
The current build for 0.3.0 is not working for me. Dependencies are missing.
Thanks.


Binary distribution via Homebrew

It could partly solve https://github.com/apex/apex/issues/199
Thoughts?

:+1:  might as well as long as it's not annoying to update the version all the time, IIRC that is done manually each time? That script isn't so bad since it installs for osx, but it's the other platforms that are annoying. Ideally We'd at have a script that you can just curl and it detects your os / arch

re #199

IIRC that is done manually each time?
yeah. but I can take care of that. well I don't like curl | sh because of security. Also I (and probably not only I) by default use brew to install stuff.

OK so I will create a formula. We can use it in readme or not :)

brew does arbitrary stuff too, it could be just as insecure, people can always vet the script first if they're paranoid haha. The manual version thing is my beef with Homebrew, it's almost always out of date

Hmm, that's right, ok, I'm closing this issue for now as we have #199.

I guess we have apex upgrade so even in the outdated case it's not too bad haha, forgot about that

yeah... :)

That being said I think #199 would be great, any Go program could use it


Support JSON output

While working on a small coverage viewer for Vim, I added basic support for
JSON output. This was previously marked as "not implemented" and would panic.
I didn't do anything fancy, just re-used the existing JSON serializer for
CoverageReport that's written when generating HTML output.

So could you move the data into a different representation just to make a nicer report?
So looking at this exported report turn it from something like:
{
  "files": [
    {
      "path": ["/",  "home", "xd009642", "code", "rust", "json", "tests",  "data", "simple_project", "src", "lib.rs"],
      "content": "pub mod unused;\n\n\npub fn branch_test_one(x: i32) -> i32 {\n    if x > 5 {\n        10\n    } else {\n        5\n    }\n}\n\n\n\n\n\n\n#[cfg(test)]\nmod tests {\n    use branch_test_one;\n    #[test]\n    fn bad_test() {\n        branch_test_one(2);\n    }\n}\n",
      "traces": [
        {
          "line": 4,
          "address": 4212944,
          "length": 1,
          "stats": {
            "Line": 1
          }
        },
        {
          "line": 5,
          "address": 4212948,
          "length": 1,
          "stats": {
            "Line": 1
          }
        },
        {
          "line": 6,
          "address": 4212964,
          "length": 1,
          "stats": {
            "Line": 0
          }
        },
        {
          "line": 8,
          "address": 4212954,
          "length": 1,
          "stats": {
            "Line": 1
          }
        },
        {
          "line": 21,
          "address": 4212048,
          "length": 1,
          "stats": {
            "Line": 2
          }
        }
      ],
      "covered": 4,
      "coverable": 5
    },
    ...

Into:
{
  "files": [
    {
      "path": [
        "/home/xd009642/code/rust/json/tests/data/simple_project/src/lib.rs"
      ],
      "traces": [
        {
          "line": 4,
          "stats": {
            "Line": 1
          }
        },
        {
          "line": 5,
          "stats": {
            "Line": 1
          }
        },
        {
          "line": 6,
          "stats": {
            "Line": 0
          }
        },
        {
          "line": 8,
          "stats": {
            "Line": 1
          }
        },
        {
          "line": 21,
          "stats": {
            "Line": 2
          }
        }
      ],
      "covered": 4,
      "coverable": 5
    },
  ...

Also there's now a PR for an --output-dir option that's been merged into develop that should be used to find where to save the json :+1:

I took into account the --output-dir option in the configuration.
For the JSON report's schema, I do agree that it could be cleaned up a bit. I'm a bit concerned that it won't be aligned with the JSON used in the HTML report. I was considering refactoring this to use the same code, but didn't feel like doing it in my first PR.
To change the JSON schema, I mainly need to customize the serialization of the SourceFile struct.
Some thoughts:

The path member is currently a Vec<String> but does feel like it could be a PathBuf or String. I can't see why it was split into its components in the first place. 🤷♂
I think the content member should be included in the final report, mainly to tell if the current source is different from that used to generate the coverage data.
I believe the traces shouldn't include the address member, as it doesn't make much sense from a report perspective. Since the struct is shared, this means implementing a custom struct/serialization to exclude it.
No particular thoughts on the covered & coverable members. I don't know the exact use-case for them.

So I didn't write the HTML reporting module that was someone elses PR. I imagine the Vec<String> would have been to make the file browser easier in the HTML report. and the covered and coverable are likely to make calculating the percentage easier. And yeah keep the content then it does make it easier.
If you don't fancy getting the HTML report to use the json from the json report just create the custom struct/serialisation. Changing the Vec<String> stuff sounds like a bit of a pain and they can always diverge

Closing this as it was provided by #362


Set will-change while animating opacity/transform/filter

The polyfill should set the will-change property while animating these properties.
See https://github.com/PolymerElements/neon-animation/pull/119

Will affect Safari & Firefox before they ship: http://caniuse.com/#feat=will-change

Keep in mind that will-change is intended to be set before rather than while animating properties to warn the browser for optimisations.

Give it sufficient time to work. This property is intended as a method for authors to let the user-agent know about properties that are likely to change ahead of time. […]


Make exchange classes pure virtual

We shouldn't have non-virtual members as part of our ABI

I also moved CreateHandler to APPLICATION and removed QueryConfig from it.

No, by assigning I meant you are the main reviewer and have the last say.

I'll add a base class today.


Working on switching over to a CMake-based config.

@AmineKhaldi I believe you are the resident CMake expert, right?
I am migrating BB to CMake so I'd appreciate any feedback you have on the new CMakeLists


Bug fixes and stylistic enhancements

games.lua: s/Ebay/eBay/g (340cb2a)
deduplicate imports in plugins/sandbox/code.c (b02181f)
truly fix ./cycle (dd551ed)
hooks.lua: refactor prefix/suffix checking (bb62836)

If either prefix or suffix are empty (set to ""), disable it instead of erroring or matching all received messages. (I personally chose to only have a prefix and disabled suffix for commands)

I had to fix the prefix/suffix one (it had problems and suffix couldn't ever work).
I cherry picked the code.c one and the eBay one though.
The ./cycle one was already fixed a long time ago so didn't merge in that one.


Selecting summary items

Adding interaction between components

This is fine for now; since the goal of this task and this sprint is just to demo something to FW I'm not giving too much thought to if this perfectly captures the react motifs we'll integrate into our code longterm.  Functionality looks great!


[FIXED JENKINS-26975] Significant performance boost for Performance Plugin

This pull request introduces improvements as described in detail on https://issues.jenkins-ci.org/browse/JENKINS-26975
In short, three issues have been addressed, that each improve disc, memory and CPU utilization of the plugin. These changes turned the plugin from unworkable to lightweight in our setup.

I'll take a look to these changes, and test in my server.

BTW, I've merged some pending pull requests, could you rebase your patch?

Try now (this is my first time merging upstream changing with a pull request - my apologies if something went wrong).

Thank you for a pull request! Please check this document for how the Jenkins project handles pull requests

It works ok in my projects and it works pretty faster.
I'll release a new version soon, let's see if someone has any issue.
Thanks for contributing.

@guusdk Seems that this patch or #45 could have some regression problems, could you take a look to this issue ? https://issues.jenkins-ci.org/browse/JENKINS-27100
Thanks

Although I'd be happy to have a look, the issue describes that the issue is
present in versions that pre-date my changes. Also, re-compilation appears
to solve the issue, according to the comments. Is my help still desired, or
has the past caught up with my late reply to your message? ;)
On 25 February 2015 at 18:34, Manuel Carrasco Moñino <
<EMAIL_ADDRESS>wrote:

@guusdk https://github.com/guusdk Seems that this patch or #45
https://github.com/jenkinsci/performance-plugin/pull/45 could have some
regression problems, could you take a look to this issue ?
https://issues.jenkins-ci.org/browse/JENKINS-27100
Thanks
—
Reply to this email directly or view it on GitHub
https://github.com/jenkinsci/performance-plugin/pull/44#issuecomment-76012889
.


rclone GUI Failed on First Launch to Yandex Browser Beta

$ rclone rcd --rc-web-gui
2019/09/29 21:41:38 NOTICE: A new release for gui is present at https://github.com/rclone/rclone-webui-react/releases/download/v0.0.6/currentbuild.zip
2019/09/29 21:41:38 NOTICE: Downloading webgui binary. Please wait. [Size: 3813937, Path :  /home/demo/.cache/rclone/webgui/v0.0.6.zip]
2019/09/29 21:41:40 NOTICE: Unzipping
2019/09/29 21:41:40 NOTICE: Serving remote control on http://<IP_ADDRESS>:5572/

Talks to the browser successfully but results in the error page.
Browser: Yandex (chromium): Beta version for Linux (deb, x64) Version #: <IP_ADDRESS>4

This looks like a duplicate of #27.
Running rclone rcd --rc-web-gui for a second time seems to solve the issue. :+1:


Updates + fixes to README examples

Not sure why tests are failing in 9.2, but I can repro it locally. Minimal repro with details here: https://www.reddit.com/r/haskell/comments/riaw00/stack_overflow_oom_with_undecidablesuperclasses/

Codecov Report

Merging #44 (ed174c9) into master (64a0404) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master      #44   +/-   ##
=======================================
  Coverage   85.57%   85.57%
=======================================
  Files           6        6
  Lines         312      312
  Branches        1        1
=======================================
  Hits          267      267
  Misses         45       45

Impacted Files
Coverage Δ

src/Database/Persist/Monad/Class.hs
0.00% <ø> (ø)

src/Database/Persist/Monad/Shim.hs
100.00% <0.00%> (ø)

src/Database/Persist/Monad/SqlQueryRep.hs
99.29% <0.00%> (ø)

Continue to review full report at Codecov.

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

Thanks for the quick fix 🎉


Incorrect loadbalance docker swarm replicas if count of replicas is even

Welcome!

[X] Yes, I've searched similar issues on GitHub and didn't find any.
[X] Yes, I've searched similar issues on the Traefik community forum and didn't find any.

What did you do?
I use docker provider with swarm mode. If i set even replicas count, f.e. 2,4,6... traefik does not use part of the replicas.
If i set odd replicas count (3,5,7...) traefik works correctly.
What did you see instead?
Correct roundrobin balancing
What version of Traefik are you using?
2.6, 2.7-rc2
What is your environment & configuration?
my docker swarm service config:
    "Name": "traefik-whoami-traefik-test",
    "Labels": {
        "traefik.http.middlewares.my-basic-auth.basicAuth.users": "admin:$apr1$9O6lRT2m$y46NNFXGGzw2SpjtsQw3Z1",
        "traefik.http.middlewares.my-buffering.buffering.maxRequestBodyBytes": "20971520",
        "traefik.http.middlewares.my-buffering.buffering.maxResponseBodyBytes": "20971520",
        "traefik.http.middlewares.my-buffering.buffering.memRequestBodyBytes": "20971520",
        "traefik.http.middlewares.my-buffering.buffering.memResponseBodyBytes": "20971520",
        "traefik.http.routers.my-default-http-router.entrypoints": "http",
        "traefik.http.routers.my-default-http-router.middlewares": "my-basic-auth,my-buffering",
        "traefik.http.routers.my-default-http-router.service": "my-service",
        "traefik.http.services.my-service.loadbalancer.healthcheck.interval": "10s",
        "traefik.http.services.my-service.loadbalancer.healthcheck.path": "/foo",
        "traefik.http.services.my-service.loadbalancer.healthcheck.timeout": "3s",
        "traefik.http.services.my-service.loadbalancer.server.port": "80",
        "traefik.http.services.my-service.loadbalancer.sticky.cookie": "false"
    }

my traefik config:
exec /entrypoint.sh traefik $@
--entrypoints.http.address=:${PORT}
--entrypoints.https.address=:${HTTPS_PORT}
--entrypoints.traefik.address=:${DASHBOARD_PORT}
--api.dashboard=true
--metrics.prometheus=true
--providers.docker.swarmMode=true
--providers.docker.endpoint=tcp://docker-socket-proxy-${NAMESPACE}:2375
--providers.docker.network=clustercontrol-net
--providers.docker.defaultRule="PathPrefix("/")"
--providers.docker.constraints="Label("namespace", "${NAMESPACE}")"
--providers.file.filename=conf.yml
--tracing=${ENABLE_TRACING}
--tracing.jaeger=${ENABLE_TRACING}
--tracing.jaeger.disableAttemptReconnecting=false
--tracing.jaeger.collector.endpoint=${TRACING_URL}
--certificatesresolvers.my-cert-resolver.acme.tlschallenge=true
--certificatesresolvers.my-cert-resolver.acme.storage="/etc/traefik/acme.json"
--certificatesresolvers.my-cert-resolver.acme.email=${ACME_EMAIL}
If applicable, please paste the log output in DEBUG level
No response

Hello @man4j
Thank you for using Traefik.
Would you please explain in detail what layer exactly you experience an issue with the Round Robin Load Balancing algorithm?

Traefik works as a load balancer and will be able to route requests to your backend applications. Technically speaking if you scale up your backend to have multiple instances, Traefik will distribute incoming requests using RR algorithm. Here is the basic diagram:

Traefik -> Multiple Replicas of the backend
If you experience the issue in that scenario we have definitely to investigate that in detail, so would you please send us more information and provide a reproducible use case.

However, I understood that you scale Traefik to have more replicas of the load balancer itself and you noticed that requests are not correctly distributed across available Traefik instances.

Multiple Traefik Instances -> Backend
If this is the case, it is related to Swarm configuration and its built-it routing mesh that is described here.  As you can see it is related to the way how the ports are exposed and how many nodes the cluster consists of.
Can you please indicate which issue from those two that I described you are experiencing?
Thank you for your feedback,

I create a simple test.
My docker-compose file:
docker-compose.yml
version: '3'

services:
  reverse-proxy:
    image: traefik:v2.6
    command: --entrypoints.http.address=:80 --api.dashboard=true --providers.docker.swarmMode=true --api.insecure=true --providers.docker.defaultRule="PathPrefix(\"/\")" --providers.docker.constraints="Label(\"namespace\", \"test\")"
    ports:
      - "80:80"
      - "8080:8080"
    volumes:
      - /var/run/docker.sock:/var/run/docker.sock

  whoami:
    image: traefik/whoami
    deploy:
      labels:
        - "namespace=test"
        - "traefik.http.routers.my-default-http-router.entrypoints=http"
        - "traefik.http.routers.my-default-http-router.service=my-service"
        - "traefik.http.services.my-service.loadbalancer.server.port=80"

I deploy this stack by command: docker stack deploy --compose-file docker-compose.yml traefik
And after I scale whoami service:
docker service scale traefik_whoami=2
traefik works incorrectly (dont see second replica).
if i scale:
docker service scale traefik_whoami=3
traefik works correctly
if i scale:
docker service scale traefik_whoami=4
traefik works incorrectly (dont see replicas).
if i scale:
docker service scale traefik_whoami=5
all works
if i scale:
docker service scale traefik_whoami=6
dont work
7
all works
8
dont work
etc...
In admin console traefik always see all replicas correctly. But problem in RR algorithm. If i try to navigate my browser to <IP_ADDRESS> I see that (and periodically CTRL+R).

Hey @man4j
Thank you for providing the reproducible use case.
I have just deployed that in my local environment. I have deployed a stack that contains Traefik 2.6 and Whoami application. I use exactly the same configuration you have shared with us:
I scaled Whoami service to 2 replicas, and here are the responses:

2 replicas

❯ curl --silent http://<IP_ADDRESS>:80  |grep IP
IP: <IP_ADDRESS>
IP: <IP_ADDRESS>
IP: <IP_ADDRESS>

❯ curl --silent http://<IP_ADDRESS>:80  |grep IP
IP: <IP_ADDRESS>
IP: <IP_ADDRESS>
IP: <IP_ADDRESS>

❯ curl --silent http://<IP_ADDRESS>:80  |grep IP
IP: <IP_ADDRESS>
IP: <IP_ADDRESS>
IP: <IP_ADDRESS>

❯ curl --silent http://<IP_ADDRESS>:80  |grep IP
IP: <IP_ADDRESS>
IP: <IP_ADDRESS>
IP: <IP_ADDRESS>

As you can see the IP addresses are being changed, which mean that Traefik correctly distributed incoming request across available replicas.

Let's try to do the same with 3 replicas:

❯ curl --silent http://<IP_ADDRESS>:80  |grep IP
IP: <IP_ADDRESS>
IP: <IP_ADDRESS>
IP: <IP_ADDRESS>

❯ curl --silent http://<IP_ADDRESS>:80  |grep IP
IP: <IP_ADDRESS>
IP: <IP_ADDRESS>
IP: <IP_ADDRESS>

❯ curl --silent http://<IP_ADDRESS>:80  |grep IP
IP: <IP_ADDRESS>
IP: <IP_ADDRESS>
IP: <IP_ADDRESS>

And once again with 4 replicas for Whoami service:

❯ curl --silent http://<IP_ADDRESS>:80  |grep IP
IP: <IP_ADDRESS>
IP: <IP_ADDRESS>
IP: <IP_ADDRESS>

❯ curl --silent http://<IP_ADDRESS>:80  |grep IP
IP: <IP_ADDRESS>
IP: <IP_ADDRESS>
IP: <IP_ADDRESS>

❯ curl --silent http://<IP_ADDRESS>:80  |grep IP
IP: <IP_ADDRESS>
IP: <IP_ADDRESS>
IP: <IP_ADDRESS>

❯ curl --silent http://<IP_ADDRESS>:80  |grep IP
IP: <IP_ADDRESS>
IP: <IP_ADDRESS>
IP: <IP_ADDRESS>

In those simple tests, we were able to reach each time one of the available Whoami replicas each.
Please advise how we can test it to reproduce the issue.
Thanks,

hello @man4j,
No worries, glad to hear that it is working correctly. Don't hesitate to ask questions, open issues, and all kinds of contributions to our community are more than welcome.
Stay tuned,


[Fix] pass down log_level to helper functions

#58 added the log_level parameter to convert_dictionary_to_ros_message. However, if the msg/dictionary has an array or another dictionary inside it, the helper functions _convert_to_ros_type and _convert_to_ros_array are being called without receiving the log_level parameter. This means the logger will only be set when parsing the uppermost message attributes.

Thanks, and sorry for taking so long to merge this!


[XR] a new on XR Camera initialized observable

This observable will be called after the camera is correctly initialized, including the number of views and the initial position.
This will solve different issue, for example:
https://forum.babylonjs.com/t/with-pipeline-enabled-console-shows-error-when-entering-xr/42493/5

Please make sure to label your PR with "bug", "new feature" or "breaking change" label(s).
To prevent this PR from going to the changelog marked it with the "skip changelog" label.

Snapshot stored with reference name:
refs/pull/14231/merge
Test environment:
https://babylonsnapshots.z22.web.core.windows.net/refs/pull/14231/merge/index.html
To test a playground add it to the URL, for example:
https://babylonsnapshots.z22.web.core.windows.net/refs/pull/14231/merge/index.html#WGZLGJ#4600
Links to test babylon tools with this snapshot:
https://playground.babylonjs.com/?snapshot=refs/pull/14231/merge
https://sandbox.babylonjs.com/?snapshot=refs/pull/14231/merge
https://gui.babylonjs.com/?snapshot=refs/pull/14231/merge
https://nme.babylonjs.com/?snapshot=refs/pull/14231/merge
To test the snapshot in the playground with a playground ID add it after the snapshot query string:
https://playground.babylonjs.com/?snapshot=refs/pull/14231/merge#BCU1XR#0


Improve Documentation on README

The README.md contains instructions for installing the app. We need some info about what the app does.

@srujandeshpande,Can i give it a try, I am new to the open-source community and looking forward to opportunities to contribute.


font icon issue

How do I import fonts?

download font
double click on the "obs_icons.ttf", install.
or
WIN + R -> fonts, drag and drop the file.


Added support for @tei:style

The @style attribute available on many TEI elements was causing an error because it's a restricted React attribute. I've added a way to pass it through and to process its content as CSS.
Let me know what you think -- if this looks good let's push a new release to NPM?

thanks, it looks great! :)

now published as 2.0.4


Add an 'empty'  condition to yii\db\Query::where()

A small enhancement:
The where() method already provides a lot of conditions. However, I just came across the need to check in a query if a field is null or an empty string. Like empty() in PHP. Could this be done by Yii?
AFAIK one needs to create two annoying conditions to check for this. It would be nice if Yii could do this.
As an option you could also consider to trim the value first if this is possible.
Small MySQL example: http://stackoverflow.com/a/14575847/57091

NO

Check out Query::filterWhere().

I would create an IS_EMPTY function in MySQL.
->where('IS_EMPTY(col)')

@Alex-Code ISEMPTY() is only for geometry data, as far as I can tell.
@nineinchnick This is the other way around. But I'd like to check for the data in the table that might be NULL. Not the variable that I pass to filterWhere().
@mdmunir NO?

Like empty() in PHP. Could this be done by Yii?

I said "NO". Mean, do it by your self.

@mdmunir Okay. I meant: Could this be done by Yii in a future version?

->where("col1 IS NOT NULL AND TRIM(col1) <> ''") should work just fine.

Of course. I just thought it would be conveniont.

if someone need to get records from db with some empty field you can use:
$query->andWhere([
	'or',
	['is', 'col_name', null],
	['=', 'col_name', '']
]);


DatePicker browser autocomplete support

Provide a general summary of the feature here
Currently DatePicker from react-aria-components seems not to be supporting browser autocomplete.
I believe it's because it's implemeted as bunch of divs with contenteditable=true.
🤔 Expected Behavior?
DatePicker supports autocomplete functionality of the browsers
😯 Current Behavior
DatePicker doesn't support autocomplete.
💁 Possible Solution
No response
🔦 Context
As I user, I often use autofill for filling up various form data. Currently it works, with Inputs, but not with DatePicker.
💻 Examples
No response
🧢 Your Company/Team
No response
🕷 Tracking Issue
No response

We could try using a visually hidden input strategy similar to what we do with Select, where the autofill targets that input and allows us to then parse it into a date time format that is consumable by the DatePicker. Will need testing, open to any help testing it out as well!


🐞 w3id.org links that redirected to html show xml now

Observed in version
after #26
Description
e.g. for https://w3id.org/nfdi4cat/voc4cat - There is something wrong with how the "best" variant is selected in content negotiation.
Browsers send in in their headers ACCEPT: text/html,application/xhtml+xml,application/xml;...
Since application/xml is hit before text/html in our current redirect file, html is no longer shown but xml. We have to move text/html up in the redirections rules.

Closed by fixing the redirect rules.


Handle Hella Long Running Jobs Asynchronously

In some cases, a command may initiate a long-running process -- a process that is too long to keep an un-acknowledged message. Azure Service Bus, for example, has a maximum lock time of 5 minutes. An upload to a remote server, or file parse, may take an hour, which is way too long.
Support the ability to coordinate and either restart or fault a command based on the running state of the job. The job factory should return a Task that can be monitored by a job registry on a particular node. If the job faults, the command is "faulted." If the job completes, the job completion message is sent/published.
The message scheduling service should be used to check on the status of the job every n minutes, to see if it has completed. The status checks would be node-specific, with a time to be delivered that would move the check to a dead letter queue so that the job could be picked up by another node or faulted.

This is underway with a couple of commits already in place, and will be part of the next release in an early form.

Moving back to Do to finish remaining components.

Oh this is going to be an excellent feature for my project! Splendid to see it's already been thought of before I even got to this part. My internal project will be using Masstransit to manage jobs that deploy & configure VMs which are expected to take anywhere up to 30 minutes. Of course I can eventually write the coordination logic myself, but if masstransit already has support by the time I need it, this makes the solution so much cleaner! Really all loving your work as I'm slowly gaining understanding.

+1 This would be really useful.

docs updated: 6181c6b8dfc073a075562fc96a7093ab34761f31
code updated; b22b4509a8be29c826d6aaf2521804550ee16021


Need intro-for-dummies

I have an AWS account, which I haven't accessed in many years, but has resources I get billed for.
What I did to set it up was done years ago, and I don't remember what I did to set it up, and it wasn't set up on any computer that still exists.
When I run it on Linux, I get this message:
$ DISABLE_TELEMETRY=true ./cloud-nuke_linux_amd64 inspect-aws
ERRO[2023-12-02T18:39:57-07:00] Error forming a cloud-nuke Query with supplied parameters. Original error: Unable to determine enabled regions in target account. Original error: could not find any enabled regions  binary=cloud-nuke version=v0.33.0

I suspect this is because it has no idea who I am, nor what AWS account to use.
A little hint on what to do to get started (install this, run that..., create this file) would be helpful. Or at least a list of prerequisites.

Hello @Alan-R, did you get a chance to read through the README file to use this CLI tool?

Closing this as we haven't heard anything for the past 2 weeks.

Hi -- I think their point is valid: I don't see anything in the README that documents how your tool connects to AWS. I assume it's via the aws command line tool, but it would not hurt to state this explicitly and just link off to the required tool and list the prerequisites.


Get any kind of data about children of cluster

Hi there!
I'm making an app where we use flutter_map - on this map, we will have ~4k markers. Because of this, it lags very much, and we really need to cluster thse for our app to be usable
For past 4 weeks I've been trying to do this well, and all libraries are failing miserably at some point
Your library is so far the best perfomant, and I can get 60 fps :100: when re-drawing it and generally messing with the markers
But it fails in one aspect - I can't get the data about children markers (efficiently)
All markers in our app represent some int value - and I would like clusters to represent averge of those ints - problem is that:
Fluster(
  ...
  createCluster: (BaseCluster cluster, lng, lat) {
    // There is no way to get data about children ("non-cluster objects") from BaseCluster
  }
)

I tried to get this from outside Fluster constructor, with fluster.points() - but it turns out it does some heavy calculations under the hood, and re-running it 500 times make everything super laggy
At this point, @alfonsocejudo, I deperatly need your help or I will freak out :pray: - if you could add any way for BaseCluster to contain some data about it's children, or show me how I could add this in a fork - I could finally finish my job, which I initially thought would take me 2 hours :vomiting_face:

hi  @TheLastGimbus ,  i got the same ~4k markers in the map.  when i try to update the markers when onCameraMove , will clear all and add again. But it is very lag when the zoom level is big (15 or lower).
May i know how you handle the 4k~ markers update when zoom level change?
Btw Sorry for asking question that not relay to this issue.
Hope you may help me, thanks

Hi there! I started making my own library that draws markers directly with canvas instead of widgets, and it performs pretty nice!
You can take a look here, at dev branch: https://github.com/KanarekApp/flutter_map_fast_markers/tree/dev
Altough I will try to optimise it even more, and it uses our flutter_map fork too... but you can watch it for any releases, because they will come sooner or later :100:

Any update about this issue ?
I need to get all children id inside cluster
Thanks


README.md link to meetup image is pointing to file that does not exist

If you take a look at your assets the image there has this OpenSourceProjects/images/.meetupimage.jpg.icloud and your README is looking for this src="./images/meetupimage.jpg". I downloaded the /.meetupimage.jpg.icloud and it is an invalid file.
From what I understand this an apple issue when backing up to the iCloud.  iCloud inserts a ghost/placeholder pending download/transfer completion.
I believed adding the correct image should fix this little issue.

Thanks for the report! @JavaVista - On investigation it looks like it was supposed to be a header image. I found some other assets within the repo that would work for that header, so I have made a few changes to the file and sent it for review by @mynar7 before issuing the PR for a fix.

This issue should be fixed by PR #42 . Admin please feel free to close the issue. Thank you!


Validation

Add validation steps for config and metadata files. Uses validation from snakemake.utils + json schema (in yaml format).

Validation implemented in repo reorganization. No need to pull this in.


closed #366 invite table overflow

Don't trust the diff. Literally just wrapped that table in a table-responsive

This is definitely an improvement, but the behavior is a little weird... specifically, when I scroll, the title of the table also scrolls (and disappears). Maybe it could be moved out and turned into a <label> or <h#>?

I'm just going to remove the table header altogether. I don't think it adds any information.

Also, please email me any comments at<EMAIL_ADDRESS>:)

Good call. Nice!


Starship Shields Do Not Calculate correctly.

under module/rules/actions/starship/calculate-max-shields.js, there is a formula:
const shieldMax = Math.max(Math.floor(data.attributes.shields.max * 0.7), 1);
The 0.7 value should be .25 to correctly calculate the maximum value o the shields divided by 4.

The calculation is correct, but might not be needed. There isn't actually any mention of a "maximum" amount of shields that you can assign to any one quadrant. This was a logical conclusion based on math. The "minimum" value that you can assigned to a quadrant is 10% of the total shield value provided to the ship when combat begins and when you balance the shields. If you have 4 quadrants, then the maximum amount that you could assign to any one quadrant would be 70% of the total, the other 30% going to the other quadrants because they each have to have at least 10%.


(A) Automatically configure this and all remaining rtfparse settings
(C) Create this path automatically
(M) Manually input correct path to use or to create
(Q) Quit and edit `email_rtf` in rtfparse_configuration.ini

Uncaught exception NameError("name 'sys' is not defined") occurred.

(A) Automatically configure this and all remaining rtfparse settings
(C) Create this path automatically
(M) Manually input correct path to use or to create
(Q) Quit and edit `email_rtf` in rtfparse_configuration.ini

Uncaught exception NameError("name 'sys' is not defined") occurred.
Traceback (most recent call last):
  File "/Users/51pwn/anaconda3/envs/rtfparse/lib/python3.9/site-packages/rtfparse/config_loader.py", line 83, in integrity_check
    assert path_to_check.exists()
AssertionError

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/Users/51pwn/anaconda3/envs/rtfparse/lib/python3.9/site-packages/rtfparse/entry.py", line 103, in cli_start
    with config_loader.Config(path_to_config, cli_args.autoconfig) as config:
  File "/Users/51pwn/anaconda3/envs/rtfparse/lib/python3.9/site-packages/rtfparse/config_loader.py", line 57, in __init__
    self.integrity_check()
  File "/Users/51pwn/anaconda3/envs/rtfparse/lib/python3.9/site-packages/rtfparse/config_loader.py", line 86, in integrity_check
    self.wizard(errors.WrongConfiguration(f"{self.path_to_config_file.name}: '{str(path_to_check)}', path does not exist!", preconf_path), autoconfig=self.autoconfig)
  File "/Users/51pwn/anaconda3/envs/rtfparse/lib/python3.9/site-packages/rtfparse/config_loader.py", line 204, in wizard
    choice = wiz_menu.choose_from(options)
  File "/Users/51pwn/anaconda3/envs/rtfparse/lib/python3.9/site-packages/rtfparse/menu.py", line 81, in choose_from
    choice = self.wait_key()
  File "/Users/51pwn/anaconda3/envs/rtfparse/lib/python3.9/site-packages/rtfparse/menu.py", line 36, in wait_key
    fd = sys.stdin.fileno()
NameError: name 'sys' is not defined

Thank you for the bug report. It looks like menu.py is missing import sys. I will try to fix it in the next days.

thanks @fleetingbytes

reopening issue, still need to fix this.

@hktalent I have merged some changes in rtfparse 0.8.0. The CLI has changed. Try it.

@fleetingbytes Thank you very much


fix(connectivity-plus): fix unregisterReceiver call

We are calling unregisterReceiver even if a call to onListen (which calls registerReceiver for API < N) has never been requested by consumer.
This happen if the package is used to one-shot check connectivity and not using the connectivity change stream events.
An alternative to this fix would be to track the onListen call with a boolean flag and call unregisterReceiver only if the flag has been set to true in onListen, let me know if you like this approach more.
Closes #358, #367

LGTM, can you please add

changelog
pump the package increasing a patch version

then I will merge the PR and will release a new package.

@mhadaily Done


refactor the init function in docker_utils.go

- What I did
Refactor the init function in integration-cli/docker_utils.go to

simplify the code of retrieving image list
remove an unnecessary API call of GET /version

- How I did it
Reuse the Server header and KernelVersion from the GET /info response.
- How to verify it
- A picture of a cute animal (not mandatory but encouraged)

LGTM

LGTM


Should we template UserApplication on DeviceType

We had a discussion today, but no resolution

Yes - everything in the Interface subpackage had a ParallelModel template parameter. Will change to DeviceType

DataTransferKit is going to be archived.


okta-aws throws InvalidClientTokenId

Describe the bug
We have discovered that starting from today for some reasons okta-aws stopped to work properly. We receive error:
An error occurred (InvalidClientTokenId) when calling the GetCallerIdentity operation: The security token included in the request is invalid.

while trying to execute: okta-aws my-profile sts get-caller-identity
To Reproduce

call okta-aws my-profile sts get-caller-identity
profile is created/updated both in ~/.okta/profile and ~/.aws/credentials, though command is not executed properly.

Also running directly jar file without = sign throws an error like here:
$ env OKTA_PROFILE=my-okta-profile java -Djava.util.logging.config.file=~/.okta/logging.properties -classpath ~/.okta/okta-aws-cli.jar com.okta.tools.WithOkta aws --profile my-okta-profile sts get-caller-identity

An error occurred (InvalidClientTokenId) when calling the GetCallerIdentity operation: The security token included in the request is invalid.

while following works properly:
$ env OKTA_PROFILE=my-okta-profile java -Djava.util.logging.config.file=~/.okta/logging.properties -classpath ~/.okta/okta-aws-cli.jar com.okta.tools.WithOkta aws --profile=my-okta-profile sts get-caller-identity
{
    "UserId": (...)

Issue exists only in shell (not in intellij in this case at least).
Expected behavior
okta-aws should pass proper profile to aws cli command
Additional context
My operating system is MacOS 10.14.6
Doesn't work both in zsh nor bash:
$ zsh --version
zsh 5.3 (x86_64-apple-darwin18.0)
 $ bash --version
GNU bash, version 3.2.57(1)-release (x86_64-apple-darwin18)
Copyright (C) 2007 Free Software Foundation, Inc.

This seems to fix the issue, although not sure why because testing it in intellij both ways works as designed without any issue.
3c3
<     withokta "aws --profile $1" $@
---
>     withokta "aws --profile=$1" $@
11c11
<     withokta "aws --profile $1" $@
---
>     withokta "aws --profile=$1" $@

based on your recommended fix with quoting it seems you have a bit older version of this project. I think your issue was fixed in https://github.com/oktadeveloper/okta-aws-cli-assume-role/pull/298 and released in 2.0.3. Can you make sure you install the latest again with
PREFIX=~/.okta bash <(curl -fsSL https://raw.githubusercontent.com/oktadeveloper/okta-aws-cli-assume-role/master/bin/install.sh) -i

and let me know if you still see the issue?

Well, somehow reinstallation fixed the issue even though I had already v2.0.3. Currently works on both 2.0.3 and latest 2.0.4. Not sure why this error just appeared. Anyway sorry for false-alarm.

I had got into the same issue. For me, deleting the rm .okta/profiles file got it fixed.


Nert 683 - Orphaned Wells Filter

Orphaned well layer now using the new LayerControl component
Drop down in the layer switcher is showing symbology for wells and well activities
Each feature type can be toggled on and off

Drawer closed

Drawer open with some features turned off

looks great! :-) for the "Deactivation" check box there is nothing shown in the map, I assume that is because there is none of that type

Yes... it looks that way.
This is what the BCER Metadata report says:


Add condition for .NET Core 3.0 in Microsoft.Azure.Cosmos.targets

Pull Request Template
Description
Don't copy native DLLs for if target framework is .NET Core 3.0 or greater.
This change requires a release and bump of Direct, as the path to the native DLLs in the output directory will be different.
Type of change
Please delete options that are not relevant.

[X] Bug fix (non-breaking change which fixes an issue)

Closing issues
closes #890

Is there a new project system behavior starting with .NET core 3.0?
Its surprising and interesting as well.

/azp run

/azp run


Any plans on having a mobile version of gitify?

With push notifications… It would be terrific!

Hello @gtramontina! I really like the idea and did a bit of research but unfortunately I couldn't find a way of having the GitHub API making a request to a server that will trigger those push notifications.
I guess I could make the phone make a request in the background every minute but it sounds wrong to me. Also we could have a server making a request to the GitHub API for every logged in user every minute which again doesn't sound right. So unfortunately I've got some technical issues with this one.
In the meantime you can check CodeHub for iOS which is an amazing & open source app. It also has an in-app purchase for push notifications that works great! I think there is something for Android too but not sure about the app name!

It's a workaround but you could integrate pushbullet and make an api call per notification. Would be dependent on the user having the pushbullet app and their computer not being asleep.

That would require pushbullet and I think there should a better way. Will close this one for now. If anything comes up and there is a way to implement push notifications, I will be really happy to make Gitify with React Native!

React Native would be awesome…
Also I've heard that we could leverage web-workers and/or service-workers to do this "polling" job and issue local notifications, as opposed to push notifications. I remember reading something around it, but I might be totally wrong…

Hello everyone! Well, you convinced me (can't resist to new projects/ideas) so gitify-mobile is under development. Will open-source soon and publish to both the AppStore and Google Play Store as soon as there is something that works nice! Until then, you can have a look at some progress! Any ideas, more than welcome!

Good news! The iOS version is currently waiting for review and will hopefully get approved in a couple of days(http://appreviewtimes.com/). Once it gets approved, I'll also submit the Android version too and make the repository public. Fingers crossed!

Thanks a lot for the effort!
How did you go about the notifications?

Well the first version won't come with push/local notifications. Need to investigate it further - for now you can login, view your notifications, search, open them in app or in browser, choose your settings, pull to refresh etc. Have some ideas for more features but thought I should take it slow..

Hi there! I've now open-sourced the repository and published the app to the App Store & Google Play Store(looks like it takes Google some time to process the app - should be live very soon). You can find download links in the repo: https://github.com/ekonstantinidis/gitify-mobile 😄

Installed!! Thank you very much! 😉 🍻

Soooo smooth so far! Nice designs too… Well done! 👏


fix #7328 by checking if socket address is valid

Not sure if it's the most standard compliant way for both platforms, but I've tried my best.
Some references: POSIX and Windows.
With an addition of removing a redundant implementation of string Hostname() (in host_info.h as well) in one of the XLA tests.

Can one of the admins verify this patch?

An interesting typo in the BUILD file seems to exclude several tests, including the one I added for this patch.

Can one of the committer check with this? Seems several tests are excluded because of a typo, and some of them failed in my build when they are enabled.

Since the original submitter of #7328 @DjangoPeng is working on this, I'm closing this PR.


docs(README): Add webgif as an alternative

webgif uses Chrome Puppeteer for a seamless, headless, cross-platform (yes Windows too!) GIF generation experience. You can use it in a straightforward manner to generate GIFs from asciinema:
npm i -g webgif
webgif --url https://asciinema.org/a/147023?t=0 --duration 60 --output asciinema.gif

No docker needed!

@sickill sorry hadn't read how asciicast2gif works, maybe it can be enhanced to use Puppeteer instead of PhantomJS since the latter's future is in doubt.
Puppeteer seems like way smaller a hammer than Docker, but thats just my opinion. FWIW, I am using the generated GIF as a "teaser" on my GitHub README to encourage users to click through to asciinema. See: https://github.com/anishkny/realworld-e2e-test#running-the-tests

@anishkny aha! I didn't know PhantomJS is in limbo, thanks for mentioning that 👍
Re Docker image: it's more of a convenience for people who already use Docker and don't want to install Node+ImageMagick+Gifsicle.
You can npm install asciicast2gif today, no need for Docker. But you still need ImageMagick and gifsicle. Moving from PhantomJS to Puppetter won't actually make it any different wrt deps.
I'll definitely look into Puppeteer anyway, thanks!

@sickill No worries. But would you accept webgif as an alternative to asciicast2gif?

There :)


Feature/activate plugin

Related #6 .
I added function i_activate_the_plugin.
This methods always activate plugin regardless whether plugin is activated.

Could you add checking that is the pugins activated like following?
https://github.com/vccw-team/wordpress-extension/blob/master/src/Context/WordPressContext.php#L63-L81
It will always fail on production site because:

The use case of this test is eunsure the plugin is activated. In many case, the plugin is already activated.
It will always fail after the test for second time.

I will add checking. I ask a question.
If the plugin is already activated, the check result is...

[ ] Invalid, throw exception
[ ] Valid, do nothing.(test is passed)

Thanks!
Also, please add the check after activation.

Add checking after activation, using "the_plugin_should_be" method.

@Ippey
Thanks! I'll resolve conflicts and merge into master.