TensorFlow device (GPU:0) is being mapped to multiple devices when using tf.estimator api and set visible_device_list with hvd.local_rank()

Click to expand!
Issue Type
Bug
Have you reproduced the bug with TF nightly?
No
Source
binary
Tensorflow Version
tf 2.11
Custom Code
Yes
OS Platform and Distribution
Linux Ubuntu 20.04.1
Mobile device
No response
Python version
3.8
Bazel version
No response
GCC/Compiler version
No response
CUDA/cuDNN version
12.0
GPU model and memory
No response
Current Behaviour?
I know there is a same issue, https://github.com/tensorflow/tensorflow/issues/58952, but it has been closed.
The model implemented with tf.estimator api. horovod multi-cards job (horovodrun -np 2 python xxx.py).
it works well using 2.10, but throws Device mapping error when using TF2.11.
FROM issue58952, It seems that this error is relative to this commit. When calling context.is_cutom_device(), this API would create TFE_Context that would create TF devices. However, program would also create TF devices when create Session with the aboved mentioned config, and these two TF device creation would conflicts.
It seems that the session_options used in the ensure_initialized function to create NewContext are inconsistent with the options used by the NewSession. The options used by NewSession are explicitly set through tf.ConfigProto.gpu_options.visible_device_list=hvd.localrank(). session_config should be at least consistent, or loaded once at most
To reproduce this error, you need to install horovod and using multi card machine to run below code with:
horovodrun -np N python xxx.py
where N denotes the number of processes to be started.
Standalone code to reproduce the issue
mport pandas as pd
import tensorflow as tf
import tensorflow.compat.v1 as tf1
global is_mpi
try:
    import horovod.tensorflow as hvd
    hvd.init()
    is_mpi = hvd.size()
except ImportError:
    is_mpi = 0
    print("No MPI horovod support, this is running in no-MPI mode!")

session_config = tf1.ConfigProto()
if is_mpi:
  session_config.gpu_options.visible_device_list = str(hvd.local_rank())

run_config = tf.estimator.RunConfig(session_config=session_config)

x_train = pd.read_csv('https://storage.googleapis.com/tf-datasets/titanic/train.csv')
x_train['sex'].replace(('male', 'female'), (0, 1), inplace=True)
x_train['alone'].replace(('n', 'y'), (0, 1), inplace=True)
x_train['class'].replace(('First', 'Second', 'Third'), (1, 2, 3), inplace=True)
x_train.drop(['embark_town', 'deck'], axis=1, inplace=True)
y_train = x_train.pop('survived')

# Data setup for TensorFlow 1 with `tf.estimator`
def _input_fn():
  return tf1.data.Dataset.from_tensor_slices((dict(x_train), y_train)).batch(32)

FEATURE_NAMES = [
    'age', 'fare', 'sex', 'n_siblings_spouses', 'parch', 'class', 'alone'
]
feature_columns = []
for fn in FEATURE_NAMES:
  feat_col = tf1.feature_column.numeric_column(fn, dtype=tf.float32)
  feature_columns.append(feat_col)

linear_estimator = tf.estimator.LinearEstimator(
    head=tf.estimator.BinaryClassHead(),
    feature_columns=feature_columns,
    model_dir="./model",
    config=run_config)
linear_estimator.train(input_fn=_input_fn, steps=1000)

Relevant log output
[1,0]<stderr>:2023-06-08 02:16:09.394853: I tensorflow/compiler/mlir/mlir_graph_optimization_pass.cc:357] MLIR V1 optimization pass is not enabled
[1,1]<stderr>:2023-06-08 02:16:09.397900: E tensorflow/core/common_runtime/session.cc:91] Failed to create session: ALREADY_EXISTS: TensorFlow device (GPU:0) is being mapped to multiple devices (1 now, and 0 previously), which is not supported. This may be the result of providing different GPU configurations (ConfigProto.gpu_options, for example different visible_device_list) when creating multiple Sessions in the same process. This is not currently supported, see https://github.com/tensorflow/tensorflow/issues/19083
[1,1]<stderr>:2023-06-08 02:16:09.397922: E tensorflow/c/c_api.cc:2209] ALREADY_EXISTS: TensorFlow device (GPU:0) is being mapped to multiple devices (1 now, and 0 previously), which is not supported. This may be the result of providing different GPU configurations (ConfigProto.gpu_options, for example different visible_device_list) when creating multiple Sessions in the same process. This is not currently supported, see https://github.com/tensorflow/tensorflow/issues/19083
[1,1]<stderr>:Traceback (most recent call last):
[1,1]<stderr>:  File "test.py", line 43, in <module>
[1,1]<stderr>:    linear_estimator.train(input_fn=_input_fn, steps=1000)
[1,1]<stderr>:  File "/usr/local/lib/python3.8/dist-packages/tensorflow_estimator/python/estimator/estimator.py", line 360, in train
[1,1]<stderr>:    loss = self._train_model(input_fn, hooks, saving_listeners)
[1,1]<stderr>:  File "/usr/local/lib/python3.8/dist-packages/tensorflow_estimator/python/estimator/estimator.py", line 1186, in _train_model
[1,1]<stderr>:    return self._train_model_default(input_fn, hooks, saving_listeners)
[1,1]<stderr>:  File "/usr/local/lib/python3.8/dist-packages/tensorflow_estimator/python/estimator/estimator.py", line 1217, in _train_model_default
[1,1]<stderr>:    return self._train_with_estimator_spec(estimator_spec, worker_hooks,
[1,1]<stderr>:  File "/usr/local/lib/python3.8/dist-packages/tensorflow_estimator/python/estimator/estimator.py", line 1512, in _train_with_estimator_spec
[1,1]<stderr>:    with training.MonitoredTrainingSession(
[1,1]<stderr>:  File "/usr/local/lib/python3.8/dist-packages/tensorflow/python/training/monitored_session.py", line 606, in MonitoredTrainingSession
[1,1]<stderr>:    return MonitoredSession(
[1,1]<stderr>:  File "/usr/local/lib/python3.8/dist-packages/tensorflow/python/training/monitored_session.py", line 1050, in __init__
[1,1]<stderr>:    super(MonitoredSession, self).__init__(
[1,1]<stderr>:  File "/usr/local/lib/python3.8/dist-packages/tensorflow/python/training/monitored_session.py", line 753, in __init__
[1,1]<stderr>:    self._sess = _RecoverableSession(self._coordinated_creator)
[1,1]<stderr>:  File "/usr/local/lib/python3.8/dist-packages/tensorflow/python/training/monitored_session.py", line 1259, in __init__
[1,1]<stderr>:    _WrappedSession.__init__(self, self._create_session())
[1,1]<stderr>:  File "/usr/local/lib/python3.8/dist-packages/tensorflow/python/training/monitored_session.py", line 1264, in _create_session
[1,1]<stderr>:    return self._sess_creator.create_session()
[1,1]<stderr>:  File "/usr/local/lib/python3.8/dist-packages/tensorflow/python/training/monitored_session.py", line 906, in create_session
[1,1]<stderr>:    self.tf_sess = self._session_creator.create_session()
[1,1]<stderr>:  File "/usr/local/lib/python3.8/dist-packages/tensorflow/python/training/monitored_session.py", line 665, in create_session
[1,1]<stderr>:    return self._get_session_manager().prepare_session(
[1,1]<stderr>:  File "/usr/local/lib/python3.8/dist-packages/tensorflow/python/training/session_manager.py", line 309, in prepare_session
[1,1]<stderr>:    sess, is_loaded_from_checkpoint = self._restore_checkpoint(
[1,1]<stderr>:  File "/usr/local/lib/python3.8/dist-packages/tensorflow/python/training/session_manager.py", line 218, in _restore_checkpoint
[1,1]<stderr>:    sess = session.Session(self._target, graph=self._graph, config=config)
[1,1]<stderr>:  File "/usr/local/lib/python3.8/dist-packages/tensorflow/python/client/session.py", line 1604, in __init__
[1,1]<stderr>:    super(Session, self).__init__(target, graph, config=config)
[1,1]<stderr>:  File "/usr/local/lib/python3.8/dist-packages/tensorflow/python/client/session.py", line 712, in __init__
[1,1]<stderr>:    self._session = tf_session.TF_NewSessionRef(c_graph, opts)
[1,1]<stderr>:tensorflow.python.framework.errors_impl.AlreadyExistsError: TensorFlow device (GPU:0) is being mapped to multiple devices (1 now, and 0 previously), which is not supported. This may be the result of providing different GPU configurations (ConfigProto.gpu_options, for example different visible_device_list) when creating multiple Sessions in the same process. This is not currently supported, see https://github.com/tensorflow/tensorflow/issues/19083

Hi @mmseerrttt ,
It seems you are using TF1.x related code which is not supported now. I have gone through the horovod documentation and found below notes.

Also Estimator API is deprecated now.Please refer below note for same.
Warning: Estimators are not recommended for new code. Estimators run v1.Session-style code which is more difficult to write correctly, and can behave unexpectedly, especially when combined with TF 2 code. Estimators do fall under our compatibility guarantees, but will receive no fixes other than security vulnerabilities. See the migration guide for details.


fix(Storage): list_files consistent across services + flat option

List files now is consistent across filesystem, google storage, and
amazon s3. It now returns the relative path from the layer path
rather than trying to fake some kind of flat directory unsuccessfully.

Summarizing our discussion for record keeping:
What does it mean to have a prefix and flat = true?
i.e.

partial directory name prefix = bigarr
full directory name prefix  = bigarray
vs full directory name + "/" prefix = bigarray/
partial file name prefix = bigarray/chunk_

In a way list_file returns a full relative key to the base 'prefix'/'path', which doesn't really take into account of directories. But the documentation and functionality of flat is to filter out to one level of a 'directory'. we should investigate what the behavior of the 4 use cases above should be and document it appropriately.

I think the way to handle them uniformly is to list all files within the nearest "directory" that is suffixed with a '/'. Then the resulting matches are filtered with the remaining prefix.That is to say, that:

partial directory name prefix = bigarr

This is handled by listing the '' directory and filtering on the key 'bigarr'

full directory name prefix = bigarray

This behaves the same as point 1

full directory name + "/" prefix = bigarray/

This lists the 'bigarray' directory

partial file name prefix = bigarray/chunk_

This lists the 'bigarray/' directory and filters on 'chunk_'

The problem with the above flat approach, is that it doesn't list directories, only files.

We can probably get away with that because the function is called list_files.


Allow adding extra lines to /etc/defaults/go-agent conf

In my Go installation I need to provide some extra paths to search
through as I've installed custom software there. By adding this
variable I'm able to configure these custom paths like so:
$ cat group_vars/gocd.yml
GOCD_AGENT_DEFAULT_EXTRAS: |
 source {{ python2_enable_source_script }}
 source {{ python3_enable_source_script }}
 source {{ postgres_path_load_script }}

@gaqzi would you like to take over maintainable of this repo, I'm sure there are things that need improvement, and having a maintainer would help immensely.

I wouldn't want to become the head maintainer (or whichever term is good), but I wouldn't mind being a co-maintainer. Primarily because I don't know for how long I'll be using Go at the moment, we're currently evaluating it and if that goes well I'm likely to stick around for a while poking at things in this repo. Otherwise… not really likely. :)

We currently do that have a maintainer who is looking at the ansible
playbook holistically. So if you do decide that this is something you'd
like to do, just give a shout out, I can understand if you don't :)
On Sun, Sep 24, 2017, 8:30 PM Björn Andersson<EMAIL_ADDRESS>wrote:

I wouldn't want to become the head maintainer (or whichever term is good),
but I wouldn't mind being a co-maintainer. Primarily because I don't know
for how long I'll be using Go at the moment, we're currently evaluating it
and if that goes well I'm likely to stick around for a while poking at
things in this repo. Otherwise… not really likely. :)
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/gocd-contrib/ansible-gocd/pull/57#issuecomment-331715608,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAApZqyB5lvUB1p6S0Fb50DGDu7pWXXpks5slm6FgaJpZM4Ph49a
.


Differentiate through the rendering process

Hi, thanks for the great work! I have a (maybe stupid) question regarding the implementation. I'm not a NeRF researcher, but there are many works in my field adopting the NeRF representation for different applications. When using NeRF, we mostly care about the gradients from rendered RGB values (or depth) w.r.t. to input rays (or query points etc.). I looked at the official instant-ngp repo before, and I find out we cannot easily differentiate through their CUDA code.
Since this is a PyTorch-based implementation, I'm wondering if we can easily get the gradient. I haven't thoroughly read the code, but as far as I read the answer seems to be yes. After all you will need PyTorch auto-grad mechanism to "train" the network. Is that correct?

I find out we cannot easily differentiate ...

I don't understand what you mean by "easily differentiate", you mean no autograd? Yes, in cuda you need to first derive the gradient mathematically and implement that. That's what instant-ngp does here
And in my implementation there is no "easy" way either, I have to derive the formula, then implement:
https://github.com/kwea123/ngp_pl/blob/e9d9c37fa0251104332fa0950f80221d1cfea203/models/csrc/volumerendering.cu#L137-L152

Thanks for the reply. Yes, I understand the CUDA implementation things. What I'm trying to say is that, since this codebase is based on PyTorch, that means you have wrapped the CUDA kernels using Python+PyTorch (correct?). So if I want to integrate it into my codebase (I'll treat the NeRF part as a black-box, basically calculating the gradients over e.g. RGB MSE loss), it would be easier than the official implementation.
Sorry for the confusion in my previous post.


Telling providers not to share reference contents with candidates

Context
We had a complaint from a referee who said that what they had written about a candidate had been shared with the candidate. It turns out the training provider had read what the referee wrote to the candidate.
Currently we do not say anywhere to providers that they should not share references with candidates.
Changes proposed in this pull request
Added a line to the 'refrence received' email for providers to say they must not share reference content with candidates
Added a line to the reference page for providers to say they must not share reference content
Link to Trello card
https://trello.com/c/6ldsRgMc/1290-look-at-adding-content-to-emails-and-service-to-tell-providers-not-to-share-reference-info-with-candidates
Things to check

[ ] If the code removes any existing feature flags, a data migration has also been added to delete the entry from the database
[ ] This code does not rely on migrations in the same Pull Request
[ ] If this code includes a migration adding or changing columns, it also backfills existing records for consistency
[ ] If this code adds a column to the DB, decide whether it needs to be in analytics yml file or analytics blocklist
[ ] API release notes have been updated if necessary
[ ] Required environment variables have been updated added to the Azure KeyVault

 Look at adding content to emails and service to tell providers not to share reference info with candidates


value: Cql[Varint/Decimal]Borrowed

This PR replaces underlying Vec<u8> in CqlVarint with Bytes. Thanks to that, we do not need to allocate additional data. Instead, CqlVarint will be shared owner of the frame bytes.
Pre-review checklist

[x] I have split my patch into logically separate commits.
[x] All commit messages clearly explain what they change and why.
[x] I added relevant tests for new features and bug fixes.
[x] All commits compile, pass static checks and pass test.
[x] PR description sums up the changes and reasons why they should be introduced.
[x] I have provided docstrings for the public items that I want to introduce.
[x] I have adjusted the documentation in ./docs/source/.
~[ ] I added appropriate Fixes: annotations to PR description.~

Now that I think of it: can we just use Bytes for CqlVarint?
The only downside I see is that it would make the whole frame alive as long as CqlVarint is alive, but it is also the case for CqlVarintBorrowed (as it borrows the frame) - the only difference is that the latter is enforced at compile time, but complicates the code.
This is a tradeoff, and I'm not sure which solution is better.

I applied @Lorak-mmk suggestion - replaced Vec<u8> with bytes::Bytes in CqlVarint.

Reverted the changes with Bytes. After a discussion, we decided that CqlVarint should not keep whole frame alive (and this is what it does when it holds Bytes).


chore: ini k8s health

Current dependencies on/for this PR:

main

PR #85

PR #87   👈

This comment was auto-generated by Graphite.


fix: changed subscribe forward window from seconds to hours

Codecov Report

Merging #516 (38c88b7) into main (669633d) will not change coverage.
The diff coverage is 0.00%.

@@           Coverage Diff           @@
##             main     #516   +/-   ##
=======================================
  Coverage   30.22%   30.22%
=======================================
  Files          84       84
  Lines        9959     9959
=======================================
  Hits         3010     3010
  Misses       6664     6664
  Partials      285      285

Impacted Files
Coverage Δ

server/services/v1/query_runner.go
4.88% <0.00%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


Clean up default configs (labels, templates) so it's not as conda/conda specific

Checklist

[X] I added a descriptive title
[X] I searched open requests and couldn't find a duplicate

What is the idea?
The current set of defauls are a bit too conda/conda specific for my taste. For example, in conda/constructor I see:

Labels that should only exist in conda/conda; e.g. all those hack::* ones?
The bug issue template requires details about a conda installation but nothing about constructor. The intro paragraphs focus on "Conda" (maybe as a community? but it's unclear and distracting).

Why is this needed?
It is confusing for maintainers and users to be required to fill data that is irrelevant for the project.
What should happen?
Ideally we would have a nice set of defaults that can be overridden with some mechanism of customization. Maybe templated templates (meta!), or just changing the language so it's more generic, or maintaining some of the templates per-repo.
Additional Context
This can also be observed in other repositories like menuinst or conda-standalone

Yes, we should revisit the templates and default labels (the hack labels should probably just be all combined into one, no need to have unique ones)
I also recently learned that the sync action we use supports templating, so that should help us significantly with this effort.

Regarding the bug issue template, we included the conda fields assumpting that errors in downstream tools or plugins would still need the info, config, and list for debugging purposes.
But I can see how menuinst and conda-standalone don't need these details so we probably want a bug issue form with conda details and a separate bug issue form without conda details.


load kafka data into greenplum error: Connection refused

gpkafka load loadcfg2.yaml  and it occurs:
gpss:gpadmin:mdw:062097-[CRITICAL]:-pq: connection with gpfdist failed for gpfdist://localhost:5001/gpkafkaload/%22payables%22.%22gpkafkaloadext_7d12affcabeb36f02a379a7c1c8081ae%22. effective url: http://<IP_ADDRESS>:5001/gpkafkaload/"payables"."gpkafkaloadext_7d12affcabeb36f02a379a7c1c8081ae". error code = 111 (Connection refused)  (seg2 slice1 <IP_ADDRESS>:6003 pid=45807): Failed toexecute SQL: Failed to load batch 0,
but the configuration in loadcfg2.yaml of
LOCAL_HOSTNAME: mdw
LOCAL_PORT: 8091
why it does not work?

it is a bug


Invalid Syntax

Please provide us with the following information:

This issue is for a: (mark with an x)
- [x] bug report -> please search issues before submitting
- [ ] feature request
- [ ] documentation issue or request
- [ ] regression (a behavior that used to work and stopped in a new release)

Minimal steps to reproduce

run anything with swagger_client

Any log messages given by the failure

File "/home/nishantwrp/Desktop/Web/Projects/Codefundo-2k19/venv/lib/python3.7/site-packages/swagger_client-1.0.0-py3.7.egg/swagger_client/api/applications_api.py", line 121
async=params.get('async'),

Expected/desired behavior

OS and Version?

Windows 7, 8 or 10. Linux (which distribution). macOS (Yosemite? El Capitan? Sierra?)
Ubuntu 19

Versions

Mention any other details that might be useful

Thanks! We'll be in touch soon.

Hello @nishantwrp,
I am writing to request more details on this issue.  If you are still having this issue, can you please elaborate more on what your use case is and your expected behavior?
Thanks,
Ramya

Hi @nishantwrp, please feel free to re-open this issue if you are still facing this issue.
Thanks,
Ramya


fix: shortened the contributors list by filtering contributors according to their activity in that particular week.

fixes https://github.com/coronasafe/leaderboard/issues/171
Truncate the list of contributors on the basis of their activity(raised, merged, reviewed PRs and opened issues) in that particular week.

I have done nothing with the data and contributors file (actually there were files and not folders with these names ). I did that in another branch and tested my changes and now I have made the required changes in a new branch without doing any ./load-fixtures.sh operation. Is it okay to move ahead without those files

@sagarkori143 are you sure the deployment has failed because of these files?
Even if you had successfully ran load-fixtures.sh, it wouldn't affect because those files under data-repo directory (contributors and data) are git ignored.
The deployment failed because the code wasn't pretty

Okay let me just check on it once again but I thought that the deployment has failed just like it happened to me while running the localhost:

I'm looking for changes again.

@sagarkori143 closing this PR as #206 has fixed the underlying issue in a larger refactor and this PR was stale for a month with builds failing. Your contributions are appreciated.


Error linking uWSGI - Mac OSX

when trying to run this command:
pip install -r requirements.txt
I am getting uWSGI error linking everytime.
Error:
clang: error: linker command failed with exit code 1 (use -v to see invocation)
*** error linking uWSGI ***
----------------------------------------

Command "/usr/local/opt/python/bin/python2.7 -u -c "import setuptools, tokenize;file='/private/var/folders/j6/jrlq9lsx6kndlqsmchbh6x9c0000gn/T/pip-build-uvQlMg/uwsgi/setup.py';f=getattr(tokenize, 'open', open)(file);code=f.read().replace('\r\n', '\n');f.close();exec(compile(code, file, 'exec'))" install --record /var/folders/j6/jrlq9lsx6kndlqsmchbh6x9c0000gn/T/pip-JFDZYy-record/install-record.txt --single-version-externally-managed --compile" failed with error code 1 in /private/var/folders/j6/jrlq9lsx6kndlqsmchbh6x9c0000gn/T/pip-build-uvQlMg/uwsgi/

@indranil-tiwary  Strange, I had no problems installing it on my machine.
Have you tried this solution? https://github.com/unbit/uwsgi/issues/600

@indranil-tiwary could you attach full pip response?

Looks like problems on user machine. Closing.

It would be nice to get to the bottom of this. Maybe we could add a macOS-specific paragraph to the installation docs.


135: HA scope/purpose

https://github.com/r-yeh/grocery-spending-tracker/issues/135

Please add to the revision history before merging.


cleanup promises

given that this.makeRequest is already wrapped in RSVP/Ember Promise there is no need wrap it again with RSVP Promise

Awesome! :)


Really machine

Self help steps before reporting the issue:
[ ] Check Solar2D forums for a solution
[ ] If Android: make sure Java8 is installed and reported when running java -version and javac -version if not, download and install Java 8
Before reporting
A clear and concise description of what the bug is.
Platforms involved
Building on: macOS 10-15-16/Windows 7-8-10/
Building for: tvOS/iOS/macOS/Windows/HTML5/Linux/Android
Attach your build.settings
Full build log
Enable full build log:
On macOS:
defaults write com.coronalabs.Corona_Simulator debugBuildProcess  -int 5

On Windows:
reg ADD "HKEY_CURRENT_USER\Software\Ansca Corona\Corona Simulator\Preferences" /f /v debugBuildProcess /d 5

Before the build clean console and then copy log between backticks below

It looks like you forgot to put details on your bug report. Can you try it again?


[Feature Request] Action log  background transparency

Would be nice if we could set Action log background transparency level from main settings.

This probably wont happen...
Simply beacuse we share UI code across everything and i dont want to make a code mess with these things.


Remove false claim of open and transparent pricing for CluedIn

Please remove the statement about CluedIn's pricing being open and transparent.
Maybe they changed their website, but they require you to submit a "pricing inquiry".
It's frustrating to research MDM vendors and they all want you to contact sales to even get a clue.
It doesn't help to offer false hope in this article. Thanks!

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

ID: 278e83ef-9b90-cb2d-9ff6-0dd8f5ff1b87
Version Independent ID: 7e63fe30-9080-9207-ad0f-62263379de0c
Content: Master data management with Azure and CluedIn - Azure Architectures
Content Source: docs/reference-architectures/data/cluedin.yml
Service: architecture-center
Sub-service: reference-architecture
GitHub Login: @EdPrice-MSFT
Microsoft Alias: Oliver.Kaae

Thanks for the feedback! I have assigned the issue to the content author to investigate further and update the document as appropriate.

The content has been updated with the correct link showing pricing details.  If you are still experiencing issues, please re-open with additional information.


use post to update user_preference settings

since we don't know the resource URI we should not use PUT, instead we should use POST with the user_id & project_id params to update the resource. Not sure how well this will play with the json api client as i couldn't figure out if it'll work via a resource create. This PR also will serialize the updated response object for use by the client.
Review checklist

[ ] First, the most important one: is this PR small enough that you can actually review it? Feel free to just reject a branch if the changes are hard to review due to the length of the diff.
[ ] If there are any migrations, will they the previous version of the app work correctly after they've been run (e.g. the don't remove columns still known about by ActiveRecord).
[ ] If anything changed with regards to the public API, are those changes also documented in the apiary.apib file?
[ ] Are all the changes covered by tests? Think about any possible edge cases that might be left untested.

The apiary docs should also be updated to reflect this change

LGTM

Is this ready to go out? I've got a feature I'd love to throw on top of this one.

Yep - just waiting to get the prev PR out on staging before i merge this one.


Packaging python code

Currently, the top level python package for this project is src.
Is this intended?
Conventionally, the src folder contains one or  more packages,
which are collected together with a setup.py at the root of the project.
My proposal is to create the following structure (with . being the project root):
./setup.py
./src
./src/agr
./src/agr/__init__.py  # This is python package
./src/agr/search.py
./src/agr/server.py
./src/agr/indexing/__init__.py
./src/agr/indexing/index.py
./src/agr/indexing/mapping.py
# etc

If there's consensus, am willing to work on this.

I believe the organization of the project is "OK" for now (haven't heard otherwise from fellow developers). Please re-open if this needs to be discussed again.


This PR is to merge  create the stream function

This PR is to merge  create the stream function.

In this function, you can right-click the menu and click "new resources" button.

Then, It will enter the create stream page.

On this page, you can set the name of the stream.

Of course, This page provides the function of editing Tag Families. You can click the "Add" button to edit.

You can add multiple Tag Families and Tags. It is worth noting that Tags are unique.

Of course, you can also edit and delete the existing Tag Family.

When you click the "submit" button, a new stream will be created, and the creation page will be closed and automatically enter the new stream.

When you do not need the stream, you can right-click the menu and click the "delete" button to delete it.

It is worth noting that at present, only new streams are supported, and stream editing is temporarily not supported.

Codecov Report

Merging #248 (4b06baa) into main (ad6a81c) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #248   +/-   ##
=======================================
  Coverage   45.88%   45.88%
=======================================
  Files          87       87
  Lines        8896     8896
=======================================
  Hits         4082     4082
  Misses       4430     4430
  Partials      384      384

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

Please don't form your PR title like this, it gets carried over to the squash and merge phase (will be used as the Top commit message) Maybe try something more straightforward like Adding support for creating streams (just an example)

Please don't form your PR title like this, it gets carried over to the squash and merge phase (will be used as the Top commit message) Maybe try something more straightforward like Adding support for creating streams (just an example)

ok, Thank you

At the top of creating the page, "Catalog" should be in place of "Type". "Operation" could replace "Operator"
Remove the prefix, "TAG_TYPE_", from the tag type's literal. For example, "INT" could substitute for "TAG_TYPE_INT".
Deletion procedure should get confirmation from the user before deleting a stream. A confirm dialog should pop up when the user clicks the delete button.

ok

I think this looks good to me. But we have to wait till v0.3.0 is released, which is in the voting process.

I think this looks good to me. But we have to wait till v0.3.0 is released, which is in the voting process.

Thank you very much for your approval.


Changes to IPF scraper to expose versioning and stop stall during scihub query

Changes to IPF scraper to expose versioning and stop stall during scihub query

expose release in AOI_based_ipf_submitter job so we can configure ipf_scraper job's release to run on (currently hardcoded to master)

If ipf_scraper falls back to Scihub, hitting the product scihub downlaod url here to check if prod_available (due to Long term Archive). However, if product is online, we will hit causeSoftTimeLimit exceeded as the request attempts to download the SLC which takes a long time:

The timeout in session.get(link, verify=False, timeout=180) does not work either since it is receiving bytes from server.
Issue is prevalent when ipf_scraper attempts to scrape IPF for the latest acquisitions when ASF does not have the SLC online yet... (when cron is triggered)
Changed logic to:
i) check if product is online by checking product entry xml
ii) if product is online, proceed to extract manifest
iii) if product is not online, proceed to hit download_url to make it online for later (should return 202), exit with Exception
Tests:

Fall back to Scihub if ASF does not have SLC yet: gist
If ASF is down, Scihub's in LongTermArchive:  gist
Normal ASF query: gist

Merging, 2397 jobs passed with this version in EOS -staging


Add Language and Local timestamp

[x]  add thai Language in dex that can switch in Settings
[x]  add tldex local timestamp choice for easy to point the time in live chat
[ ]  improve music bar
music bar auto hide when scroll down and show when scroll up
music bar can pin for cancel auto hide function
show timestamp of musicvideo on scrollbar in mm:ss of current time of musicvideo when click
music bar can minimize when not mouse over 5 sec
music bar when minimize can popup when mouseover time scroll

Can you separate the changes for music bar to a different PR? The first two tasks looks good, and would like to get that merged in but Music bar might need some more review/changes.

sorry i didnt see the request to make "music bar to a different PR?" i will re-install it out

ah, that's not a huge deal. I already merged in the approved portions of the PR (manually pulling in your fork at the point right before you started developing a new feature)

good github etiquette usually means creating a branch for each feature you'd like to contribute, but that's okay, someone new to git won't really know that, so it's a good opportunity for you to learn as you go.
To create a new branch, you can either push into a branch that doesn't exist (git push <myremote> <mylocalbranch>:<new remote branch>) or just create a new branch and start developing it. You should lookup some github tutorials that describe these things. 👍

LGTM


time scale is not using moment.locale()

Hi,
I am trying to make a localized axis labels to present month names and time/date format of choice given by moment.locale(). I see that some parts of chart (tooltips, subtitle) pick up locale set in moment, however time scale seems to be hard - coded to US locale. I am not sure if this is related to d3.time.scale or other part of visavail.js. Can you suggest a solution to my issue?

Hi @adas172002 ,
now the library go to v 2.0 and moment it integrate with locale function.
Enjoy with visavail 2.0


[Question] Building the extension locally

I'm trying to build extension locally to test some changes with my daily work. But with new build-server changes I don't see there this folder build-server-for-gradle is coming from. There seems to be references in azure pipeline, but not sure where this artifacts gets build and where the source code is. @jdneo Can you provide some steps how to build the extension locally with new build-server changes ?

@testforstephen @Eskibear @CsCherrYY @jdneo Please let me know if the artifacts can be downloaded from some where.

Ah, sorry @gayanper that currently the repo 'build-server-for-gradle' is not open-sourced yet. (We will make it public in near future)
Which part did you change? If it's not related to build server, you can just ignore it.

@jdneo it's the PR I have opened for nested closure completions for this repo. Now I could copy the bs files from the official extension files as a workaround. But another suggestion is could me put those bs files in a Azure Artifacts ?

The gradle build server is open source now. Please refer to CONTRIBUTING.md for more details :)


Mac users not able to load jobs where compressed_chunk_type = video

My actions before raising this issue

[x] Read/searched the docs
[x] Searched past issues

Mac users in my team, contrary to them using PC, are not able to enter certain video jobs. The page keeps loading forever and I get the following error from the console:
download.worker.js:24 Uncaught (in promise) TypeError: Cannot read properties of undefined (reading 'status')
at download.worker.js:24:40
I checked the database and realized that the tasks in question were videos where the compressed_chunk_type is video rather than imageset. My issue/question is why Mac users don't seem to be able to enter jobs where the compressed_chunk_type is video?
As I understand it, the compressed_chunk_type will be imageset if the "Use zip chunks" is checked while creating the task, and video otherwise. However, the "Use zip chunks" option is not yet available in the CLI. I think you have merged this feature to master so I guess it will be available in next release? Still, I would like to be able to enter all jobs from all computers despite compressed_chunk_type.
Expected Behaviour

All users, Mac as well as PC, should be able to enter all jobs despite compressed_chunk_type.
Current Behaviour

Mac users are not able to enter jobs in a video task where the compressed_chunk_type is video instead of imageset.
Possible Solution

Steps to Reproduce (for bugs)

Create a task from video data and don't check the "Use zip chunks" option (such that the compressed_chunk_type is video rather than imageset).
Use a PC.
Enter the created task and click on one of the jobs.
The job will load and you will see the job interface and be able to annotate, review etc.
Then use a Mac instead.
Enter the same task and click on one of the jobs.
Realise that the job never stops loading and you don't see the job interface.

Context

Some people are not able to reach some jobs. We want to be able to reach all jobs despite computer and compressed_chunk_type.

Operating System and version (e.g. Linux, Windows, MacOS):
Code example or link to GitHub repo or gist to reproduce problem:
Other diagnostic information / logs:
Logs from `cvat` container

Next steps
You may join our Gitter channel for community support.

@AlexeyAlexeevXperienceAI
Since you are the only person has MAC, can you please check this issue? :)


Youtube clone

Israel and I finished the branch feature and validated our HTML.
We are ready for review!

Hello @uche-inyama , Can you change the list of changes needed to be made? The project instructions states we should not add linter checks yet, Please refer to the instructions.
The linter files were added when we installed the linter checks for the second project.

Hello @uche-inyama , Can you change the list of changes needed to be made? The project instructions states we should not add linter checks yet, Please refer to the instructions.
The linter files were added when we installed the linter checks for the second project.

Hi team, you are right . Passing linter check is not a requirement for this project to be approved, apologies :pray:  :slightly_smiling_face:

@Forison ???????? Im sorry but we do have an image showing the avatar... Can you please double check?

Hi @MarioDena & @Israel-Laguan, wave
Your project is approved heavy_check_mark
Go ahead and merge to master branch slightly_smiling_face

Hey really appreciated! Thanks!


Document how to pass CLI flags to nbqa pre-commit hooks

Closes #731.

@all-contributors please add @janosh for docs


[Run][Plugin Manager] UI: own settings page

We have now implemented a plugin management for PT Run into the settings ux. (PR #9872; Issue #5273)
📝 Provide a description of the new feature
If we have many plugins it would be interesting to move the plugin list to an own settings page.

(The image is an old mock up and not up to date.)
The users could get access to it by a link on PT Run's settings page

or by a sub navigation item in the navigation pane (example image form "XAML Controls Gallery" app).

This is a remaining enhancement idea form issue #5273.
If you'd like to see this feature implemented, add a 👍 reaction to this post.

If neccesary, the sub-menu (as you show) would be a good idea. Or a Pivot on the Run page.

If neccesary, the sub-menu (as you show) would be a good idea. Or a Pivot on the Run page.

How should pivot look?

Have one "pivot page" for general (Run) settings, and one page for the plug-ins.

Have one "pivot page" for general (Run) settings, and one page for the plug-ins.

Do you mean tabs? Can't imagine how "pivot page" should look atbthe moment. Sory.

Or, nevermind: https://github.com/microsoft/microsoft-ui-xaml/issues/4492

More examples from Windows apps: microsoft/microsoft-ui-xaml#2310 (comment)

I personally do not like this layout.

Closing this as I don't see any benefit with our current ui.


Expose spin and binary product as generators

Expose _spin product and _binary_product as bqm generators.
See: https://github.com/dwavesystems/dimod/blob/5e399317b0bfaae6ed20e22b9f2ef242f5fa5e6c/dimod/higherorder/utils.py#L63

Closed by #951


Question about the "slow motion" phenomenon of my own .xml file

Hi,
I'm a student and I'm trying to use MuJoCo for building a musculoskeletal model of an animal's  forelimb.
I'm looking for some help with the "slow motion" phenomenon of my own .xml file. Every time when I load my .xml file in MuJoCo GUI, the model would move in a very slow velocity. And when I change the activity of muscles, the effects on the model would be very small and the model's movement would also change very slowly. The official example, humanoid.xml, acts normally. So I wonder where I can modify my model to show a normal velocity of motion.
Attached is a .ZIP file containing the .xml file and a video demo.
RFL_demo.zip
Thanks for your time to read this, and looking forward to your reply!

I think it's mainly related to the force parameters of muscles in my model. After I revise the force of all muscles to the default value -1, the model would move normally, and the effect of muscle activity also gets more manifest.


Tests on rhel-7 are failing

See the CI results on #3. Note that the PR did not change anything relevant AFAICT and it merely triggered a test run. The error is "No package matching 'linuxptp' found available, installed or updated". The same error was already noted in the last comment on #1.
Surprisingly, the tests pass for CentOS 7.

As we found out, the issue is not in the role, but in the rhel images used for testing. They don't have any repositories configured from which could be installed linuxptp and other packages installed by the role.

this should be addressed now, the updated test-harness does register yum repositories in the test VMs

there is code in https://github.com/linux-system-roles/test-harness/pull/26 to run arbitrary Ansible plays and the CI configuration was updated to take advantage of this to update the yum configuration

tests are passing now, see e.g. #9. Closing.


Use this on non-ejected project?

Sorry if I missed this in the README, but I'd like to know if I can use cra-build-watch on my project if I haven't run eject. I'd rather not lose the functionality of CRA scripts so I don't want to eject but cra-build-watch sounds like exactly what I need.
Edit -- Could someone label this as a question? I believe I can't do it on my end

Yes that's exactly the purpose of this tool. I didn't want to have to eject either which led me to develop this.
It is not directly mentioned in the README, I'll add something about it. Thanks for the feedback and I'm glad it will be useful to you.


Fix json validation of complex types in strict models

I'd like to use SQLModel with strict models exposed via FastAPI. However, there seem to be a few edge cases specifically with date, UUID, and similar "complex" types that aren't native json types where the "strict validate json" behavior is not used and instead SQLModel/FastAPI/Pydantic is expecting the exact type (eg: a date or UUID instance) which obviously fails when we're parsing from a json string. For example, this script:
from datetime import date

from pydantic import BaseModel, ConfigDict, TypeAdapter
from sqlmodel import SQLModel

class Pydantic(BaseModel):
    value: date

class PydanticStrict(Pydantic):
    model_config = ConfigDict(strict=True)

class SQL(SQLModel):
    value: date

class SQLStrict(SQL):
    model_config = ConfigDict(strict=True)

json = '{"value": "1970-01-02"}'
for model in [Pydantic, PydanticStrict, SQL, SQLStrict]:
    print(f"{model.__name__}.model_validate_json: {model.model_validate_json(json)}")
    print(f"TypeAdapter({model.__name__}).validate_json: {TypeAdapter(model).validate_json(json)}")
    print()

errors for the SQLStrict model on main calling model_validate_json and TypeAdapter.validate_json while all of the other models work as expected.

This PR implements SQLModel.model_validate_json by splitting sqlmodel_validate into a main helper with *_python and *_json variants that defer to the appropriate Pydantic method.
This fixes the direct .model_validate_json calls, but there still seem to be some separate issues with the TypeAdapter. I'm not sure yet what's wrong with TypeAdaptor around SQLModels (given it works fine with PydanticStrict above), but this seems to be the main blocker for fixing the usage in FastAPI so I've kept this PR as a draft for now while I figure that out - I'd appreciate any guidance!

Hi @JacobHayes,
This PR has been inactive for some time. Are you still able to work on it? If not, please let us know so we can continue the development from this point.
@tiangolo What is your perspective on this? Maybe you have an idea regarding the TypeAdapter.


Request: type alignment separated from type

I've found it useful a few times to be able to be able to use the type alignment bits without the rest of everything in foundation/components/type.
Would it be possible to get those parts split out into their own file? Something like foundation/components/type-alignment?
// Text alignment class names
$align-class-names:
 small-only,
 small,
 medium-only,
 medium,
 large-only,
 large,
 xlarge-only,
 xlarge,
 xxlarge-only,
 xxlarge;

.text-left    { text-align: left !important; }
.text-right   { text-align: right !important; }
.text-center  { text-align: center !important; }
.text-justify { text-align: justify !important; }

// Text alignment breakpoints
$align-class-breakpoints:
  $small-only,
  $small-up,
  $medium-only,
  $medium-up,
  $large-only,
  $large-up,
  $xlarge-only,
  $xlarge-up,
  $xxlarge-only,
  $xxlarge-up;

@for $i from 1 through length($align-class-names) {
 @media #{(nth($align-class-breakpoints, $i))} {
   .#{(nth($align-class-names, $i))}-text-left { text-align: left !important; }
   .#{(nth($align-class-names, $i))}-text-right   { text-align: right !important; }
   .#{(nth($align-class-names, $i))}-text-center  { text-align: center !important; }
   .#{(nth($align-class-names, $i))}-text-justify { text-align: justify !important; }
 }
}

I was going to submit a PR for it but I'm having issues building via grunt. It says it's missing glob. So whatevs. I'll fix that later.

Int he next version these are in separate partials. We'll take a PR for Foundation 5 if you can get that sorted. Otherwise moving forward in 6 it will be separate.

Awesome. Thanks.


am-checkbox inline, inline is a colorbox function

am-checkbox inline
Should be
am-checkbox display-inline
Class Inline is a Colorbox function and when checked it brings up a Colorbox window.
suggest display-inline = that what its doing.
Works like this
.am-checkbox.display-inline, .am-radio.display-inline {   display: inline-block; } .am-checkbox.display-inline + .am-checkbox.display-inline, .am-checkbox.display-inline + .am-radio.display-inline, .am-radio.display-inline + .am-checkbox.display-inline, .am-radio.display-inline + .am-radio.display-inline {   margin-left: 18px; }
please replace all occurrences of online.gif and offline.gif, they are just ugly, with s7 something.
Might add zones of a way to edit areas like in the sidebar with content items or mini-widgets.
Or like the new editable dashboard, would be way cool.
Super Nice Work Thanks!

Yes ! We need to clean up the HTML and use a better semantic. I think this is part of improving consistency in all views. We actually did the global work but we need to push another sprint on every modules/views to make it cleaner.


Create CII Best Practices Badge

https://bestpractices.coreinfrastructure.org/
As part of joining CNCF, we require graduated projects to acquire a CII Badge and pass the program.

I keep getting an error on the field for "What license(s) is the project released under?" It auto-filled from the repo, and I also tried manually entering it and received an autocompletion for "Apache-2.0". When I try and "Save and continue" I get an error "You must use this format ..." and can't go any further.

Ok, works in a different browser. Will get this done ASAP


Chat archival

We should have something that watches twitch chat similarly to how we watch the twitch stream, saving the full message content (including ircv3 tags, which contain metadata like raids/hosts, user colors, emote usage). Bonus points if we can capture redundantly then deduplicate in a reasonable way.

A basic version of this got done but needs to be merged.

also todo: grab every emote seen so we have a local copy in case of deletes


better test coverage

[ ] apis_core/apis_entities/autocomplete3.py
[x] apis_core/apis_tei/views.py
[ ] apis_core/apis_relations/forms2.py

tei-views: https://github.com/arthur-schnitzler/pmb-service/commit/38ec841936a4589c554144cccb860eb39568973d

https://github.com/arthur-schnitzler/pmb-service/commit/8deb2fa43d5dcb3b311532cf7789b60fa4e01b87


Adds a utility to outer product and contraction of tensors as per  #509

Regarding #509

@ritzvik Please have a look.

Hello @bibek22! Thanks for opening this PR. We checked the lines you've touched for PEP 8 issues, and found:

In the file src/einsteinpy/symbolic/tensor.py:

Line 100:52: E203 whitespace before ':'
Line 101:52: E203 whitespace before ':'

I'm not sure which line is failing the check. Please suggest a better name for the function. product or tensor_product would probably be better. Also, is this the right file to place this function?

Why is it testing on Python 3.5 ?

Right. This needs a rebase :)

Right. This needs a rebase :)

Hey, I've pushed the changes. See if that does it.

I've made some serious changes and added a simple test case. Please have a look.

It's the second to last item on the list. "Native function for Tensor Product"

You will need to write a better test.

You will need to write a better test.
alright. Could you suggest how I could improve it?

You will need to write a better test.

alright. Could you suggest how I could improve it?

You should open the CI Log! Once you find which line isn't tested, you'll have to write one for it. And then all checks will be :heavy_check_mark:

You will need to write a better test.

alright. Could you suggest how I could improve it?

You should open the CI Log! Once you find which line isn't tested, you'll have to write one for it. And then all checks will be heavy_check_mark

I'm sorry, I am not finding the log. Which one are you refering to? Is it on one of the ci/circleci ?

There are only two checks that have failed. Nvm, I pulled the link for you here.

Great!

Rebase! You will face some merge conflicts!


Is the name and metaphor insensitive?

I know that gender is an issue in the tech world. In a way this does imply gender roles.  Some people will make jokes about it.

I closed this because GitHub is probably not the place for this kind of discussion.  Also, the discussion in and of itself may actually be the insensitive part.


Broken Link Article Insights Container Livedata Setup

Where it says:
[Azure Kubernetes Cluster User Role](../../azure/role-based-access-control/built-in-roles.md#azure-kubernetes-service-cluster-user-role permissions)

Should say:
 [Azure Kubernetes Cluster User Role](../../articles/role-based-access-control/built-in-roles.md#azure-kubernetes-service-cluster-user-role)

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

ID: 73d09442-55a1-6ee5-8ac9-fd5e7ca06e87
Version Independent ID: e7f2e287-8b0e-a75e-dc66-7dbd3c00b245
Content: Set up Azure Monitor for containers Live Data (preview) - Azure Monitor
Content Source: articles/azure-monitor/insights/container-insights-livedata-setup.md
Service: azure-monitor
GitHub Login: @MGoedtel
Microsoft Alias: magoedte

It is fixed by https://github.com/MicrosoftDocs/azure-docs/pull/59561

@RafaAguilar Thank you for the confirmation.

@RafaAguilar Thanks for the contribution! The changes should reflect soon post the merge.


Unclean package installation when virtualenv option is enabled

Details

Read the Docs project URL: https://readthedocs.org/projects/fplore/
Build URLs:

https://readthedocs.org/projects/fplore/builds/8212795/
https://readthedocs.org/projects/fplore/builds/8213217/

Expected Result
Documentation builds fine as it did previously and as it does locally.
Actual Result
Build errors.
Traceback (most recent call last):
  File "/home/docs/checkouts/readthedocs.org/user_builds/fplore/envs/latest/bin/sphinx-build", line 7, in <module>
    from sphinx.cmd.build import main
  File "/home/docs/checkouts/readthedocs.org/user_builds/fplore/envs/latest/lib/python3.5/site-packages/sphinx/cmd/build.py", line 25, in <module>
    from sphinx.application import Sphinx
  File "/home/docs/checkouts/readthedocs.org/user_builds/fplore/envs/latest/lib/python3.5/site-packages/sphinx/application.py", line 34, in <module>
    from sphinx.environment import BuildEnvironment
  File "/home/docs/checkouts/readthedocs.org/user_builds/fplore/envs/latest/lib/python3.5/site-packages/sphinx/environment/__init__.py", line 38, in <module>
    from sphinx.util.websupport import is_commentable
  File "/home/docs/checkouts/readthedocs.org/user_builds/fplore/envs/latest/lib/python3.5/site-packages/sphinx/util/websupport.py", line 11, in <module>
    from sphinxcontrib.websupport.utils import is_commentable  # NOQA
  File "/home/docs/checkouts/readthedocs.org/user_builds/fplore/envs/latest/lib/python3.5/site-packages/sphinxcontrib/websupport/__init__.py", line 13, in <module>
    __import__('pkg_resources').declare_namespace(__name__)
  File "/home/docs/checkouts/readthedocs.org/user_builds/fplore/envs/latest/lib/python3.5/site-packages/pkg_resources/__init__.py", line 3017, in <module>
    @_call_aside
  File "/home/docs/checkouts/readthedocs.org/user_builds/fplore/envs/latest/lib/python3.5/site-packages/pkg_resources/__init__.py", line 3003, in _call_aside
    f(*args, **kwargs)
  File "/home/docs/checkouts/readthedocs.org/user_builds/fplore/envs/latest/lib/python3.5/site-packages/pkg_resources/__init__.py", line 3045, in _initialize_master_working_set
    dist.activate(replace=False)
  File "/home/docs/checkouts/readthedocs.org/user_builds/fplore/envs/latest/lib/python3.5/site-packages/pkg_resources/__init__.py", line 2577, in activate
    declare_namespace(pkg)
  File "/home/docs/checkouts/readthedocs.org/user_builds/fplore/envs/latest/lib/python3.5/site-packages/pkg_resources/__init__.py", line 2151, in declare_namespace
    _handle_ns(packageName, path_item)
  File "/home/docs/checkouts/readthedocs.org/user_builds/fplore/envs/latest/lib/python3.5/site-packages/pkg_resources/__init__.py", line 2091, in _handle_ns
    _rebuild_mod_path(path, packageName, module)
  File "/home/docs/checkouts/readthedocs.org/user_builds/fplore/envs/latest/lib/python3.5/site-packages/pkg_resources/__init__.py", line 2120, in _rebuild_mod_path
    orig_path.sort(key=position_in_sys_path)
AttributeError: '_NamespacePath' object has no attribute 'sort'

and sometimes
Using base prefix '/usr'
New python executable in /home/docs/checkouts/readthedocs.org/user_builds/fplore/envs/latest/bin/python3.5
Not overwriting existing python script /home/docs/checkouts/readthedocs.org/user_builds/fplore/envs/latest/bin/python (you must use /home/docs/checkouts/readthedocs.org/user_builds/fplore/envs/latest/bin/python3.5)
Installing setuptools, pip, wheel...
  Complete output from command /home/docs/checkouts...latest/bin/python3.5 - setuptools pip wheel:
  Traceback (most recent call last):
  File "<stdin>", line 7, in <module>
  File "<frozen importlib._bootstrap>", line 969, in _find_and_load
  File "<frozen importlib._bootstrap>", line 958, in _find_and_load_unlocked
  File "<frozen importlib._bootstrap>", line 664, in _load_unlocked
  File "<frozen importlib._bootstrap>", line 634, in _load_backward_compatible
  File "/usr/local/lib/python3.5/dist-packages/virtualenv_support/pip-9.0.1-py2.py3-none-any.whl/pip/__init__.py", line 26, in <module>
  File "<frozen importlib._bootstrap>", line 969, in _find_and_load
  File "<frozen importlib._bootstrap>", line 958, in _find_and_load_unlocked
  File "<frozen importlib._bootstrap>", line 664, in _load_unlocked
  File "<frozen importlib._bootstrap>", line 634, in _load_backward_compatible
  File "/usr/local/lib/python3.5/dist-packages/virtualenv_support/pip-9.0.1-py2.py3-none-any.whl/pip/utils/__init__.py", line 27, in <module>
  File "<frozen importlib._bootstrap>", line 969, in _find_and_load
  File "<frozen importlib._bootstrap>", line 958, in _find_and_load_unlocked
  File "<frozen importlib._bootstrap>", line 664, in _load_unlocked
  File "<frozen importlib._bootstrap>", line 634, in _load_backward_compatible
  File "/usr/local/lib/python3.5/dist-packages/virtualenv_support/pip-9.0.1-py2.py3-none-any.whl/pip/_vendor/pkg_resources/__init__.py", line 3018, in <module>
  File "/usr/local/lib/python3.5/dist-packages/virtualenv_support/pip-9.0.1-py2.py3-none-any.whl/pip/_vendor/pkg_resources/__init__.py", line 3004, in _call_aside
  File "/usr/local/lib/python3.5/dist-packages/virtualenv_support/pip-9.0.1-py2.py3-none-any.whl/pip/_vendor/pkg_resources/__init__.py", line 3046, in _initialize_master_working_set
  File "/usr/local/lib/python3.5/dist-packages/virtualenv_support/pip-9.0.1-py2.py3-none-any.whl/pip/_vendor/pkg_resources/__init__.py", line 2578, in activate
  File "/usr/local/lib/python3.5/dist-packages/virtualenv_support/pip-9.0.1-py2.py3-none-any.whl/pip/_vendor/pkg_resources/__init__.py", line 2152, in declare_namespace
  File "/usr/local/lib/python3.5/dist-packages/virtualenv_support/pip-9.0.1-py2.py3-none-any.whl/pip/_vendor/pkg_resources/__init__.py", line 2092, in _handle_ns
  File "/usr/local/lib/python3.5/dist-packages/virtualenv_support/pip-9.0.1-py2.py3-none-any.whl/pip/_vendor/pkg_resources/__init__.py", line 2121, in _rebuild_mod_path
AttributeError: '_NamespacePath' object has no attribute 'sort'
----------------------------------------
...Installing setuptools, pip, wheel...done.
Traceback (most recent call last):
  File "/usr/lib/python3.5/runpy.py", line 184, in _run_module_as_main
    "__main__", mod_spec)
  File "/usr/lib/python3.5/runpy.py", line 85, in _run_code
    exec(code, run_globals)
  File "/usr/local/lib/python3.5/dist-packages/virtualenv.py", line 2328, in <module>
    main()
  File "/usr/local/lib/python3.5/dist-packages/virtualenv.py", line 713, in main
    symlink=options.symlink)
  File "/usr/local/lib/python3.5/dist-packages/virtualenv.py", line 945, in create_environment
    download=download,
  File "/usr/local/lib/python3.5/dist-packages/virtualenv.py", line 901, in install_wheel
    call_subprocess(cmd, show_stdout=False, extra_env=env, stdin=SCRIPT)
  File "/usr/local/lib/python3.5/dist-packages/virtualenv.py", line 797, in call_subprocess
    % (cmd_desc, proc.returncode))
OSError: Command /home/docs/checkouts...latest/bin/python3.5 - setuptools pip wheel failed with error code 1

Can you try wiping the environment before build? Versions -> Edit -> Wipe

That potentially fixed it.
I was looking for an option like that under the admin part of the control panel, but since it wasn't there, I assumed it clears it every run.
Now I'm getting new issues which do not occur locally, but I'm not sure yet why they occur (https://readthedocs.org/projects/fplore/builds/8213711/)

I attempted to create a minimum failing example based on matplotlib. However, this doesn't fail.
My workaround also succeeded (explicitly specifying matplotlib~=2.2 in docs/requirements.txt. So it is definitely related to the way RTD first installs matplotlib 3.x and then 2.x.

@stsewd I belive this qualifies as a bug, not a support request

This erroneous import behavious is caused by enabling the virtualenv option on readthedocs. So in essence, RTD does not properly isolate virtualenv packages.

Do you mean the install option? I'm guessing some residuals from the previous installation (if I understand correctly you change the versions, right?). If so, you should only need to wipe the environment to resolve that, we can't do that automatically because we don't know what packages are going to be installed, if we cache the virtualenv installation to save resources.

I believe this is a sphinx-galery issue, when the package is installed in rtd they install an additional package https://github.com/sphinx-gallery/sphinx-gallery/blob/29d317b1674a08fc36a4e2ad35e183cdce8f1e39/setup.py#L20-L22, that package has a different version of matplolib https://github.com/mwaskom/seaborn/blob/f0349b1a7f1e022cb1da8b94f2e019f04eeed397/setup.py#L35, I'll try to replicate this locally setting a env variable, but not sure what we can do here.


Vicious circle on first run...

From the readme, this should work:
./tradfri-go -authenticate -clientId blubber -psk secretsecret -gateway <IP_ADDRESS>
Could not find psk.key in current directory, trying to use env-vars CLIENT_ID and PRE_SHARED_KEY instead
panic: Unable to resolve PRE_SHARED_KEY from env-var or psk.key file

But it fails... Looking at the code I also can't see how viper should be able to get the relation between -psk and PRE_SHARED_KEY...

I can of course workaround this by setting the env vars and creating that file by hand, but then I get an timeout connecting to the gateway. But because of above errors I'm not sure if that's just because the setup was not done correctly or if I have a second issue.

Thanks for the report, I probably messed up the initialization when running the -authenticate code path. Will take a look ASAP.

I've fixed the problem, thanks for reporting it!
(As a side note - one can only perform the key exchange exactly once per "CLIENT_ID". Subsequent attempts  returns a CoAP error code (5.00 or 4.00, can't remember which). However, it seems to be OK to use different CLIENT_IDs, the gateway "remembers" previously issued tokens per CLIENT_ID.)

Awesome, thank you! Works as advertised :)
And the new hint about the timeout was already helpful :D
Great project!


filesystem/posix fix illegalPath check to only include /sys /proc /dev

This commit changes it to check if path is exactly /sys, /dev or /proc or starts
with /sys/, /dev/ or /proc/.
Original implementation failed when for example configuration lived under /system/config.yaml
Signed-off-by: Hannu Ylitalo<EMAIL_ADDRESS>

The issue is I had a system where we had to place the configuration under /system/something/envoy.yaml and the current startsWith check which is intended to check if it's under /sys fails as the path for /system/ starts with /sys.
The point of this PR is to check if the path starts with /sys/ or is just /sys which is the original intended security check that is in place.

Oh I see what the issue is now. Can we extract this logic into a helper function at the very least? This will also need a regression test


restore: fix several bugs related to column permutations

What problem does this PR solve?
Fix follow bugs:

load wrong number of column permutation for file checkpoint
incorrect column names fetch from column permutation, this will cause tidb encoded generates row table columns if the csv head order is different from table schema
tidb encoder use wrong field info to encode field

What is changed and how it works?
Check List
Tests

Unit test
Integration test
Manual test (add detailed scripts or steps below)
No code

Side effects
Related changes

the new test failed
[2020-10-31T08:52:30.160Z] ERROR 1064 (42000) at line 1: You have an error in your SQL syntax; check the manual that corresponds to your TiDB version for the right syntax to use line 1 column 87 near "'select fund_seq_no, region_code, credit_code from test.test_perm WHERE contract_no = '2020061000019011020164030597';"

/run-all-tests

LGTM


Update Post “how-often-should-a-termite-treatment-be-done”

Automatically generated by Netlify CMS

👷 Deploy Preview for chic-shortbread-66a66b processing.

Name
Link

🔨 Latest commit
268c3dd163ad5325a60c82a56ed78bed36ea1530

🔍 Latest deploy log
https://app.netlify.com/sites/chic-shortbread-66a66b/deploys/654867eb649f1c0008b88c5d


35-video-component

Fixes #35
Creator

[x] Build relevant tests if any apply
[x] Ensure there are no new warnings
[x] Upload an animated GIF showcasing the solution (if it applies)
[x] Set a relevant reviewer

Reviewer

[ ] Run test and make sure all tests pass
[ ] Review function and form on iOS
[ ] Review function and form on Android
[ ] Review function and form on Web
[ ] Review new test logic

This isn't 100%, for some reason the calling setPositionAsync on Video doesn't work as expected. Not sure if there's some sort of trick to it.

@burkeshartsis it's worth reviewing the audio component when looking this over as well, and perhaps any thoughts/discussion that was made in the audio PR: #32

Warnings

:warning:

No Changelog changes!

:warning:

:exclamation: Big PR

  Generated by :no_entry_sign: dangerJS

Closing this in favor of using a native video player solution. Will readdress issues requirements in #100.


Transacation Priority UI

Describe the bug
Transaction Priority UI should be according to XD
To Reproduce
Steps to reproduce the behavior:
Send > Transaction Priority > Modal need UI updation.

Completed


loginActivity.js:11: Добавить стили

The puzzle 10-d4fbd3fb from #10 has to be resolved:
https://github.com/team-dev-spring-2022/teamMPSF/blob/858c5eaea2c1e7c8892acfb0f55d99ac9749b57d/components/screens/loginActivity.js#L11-L11
The puzzle was created by sashkadyupri98 on 05-Feb-22.
role: DEV.
If you have any technical questions, don't ask me, submit new tickets instead. The task will be "done" when the problem is fixed and the text of the puzzle is removed from the source code. Here is more about PDD and about me.

The puzzle 10-d4fbd3fb has disappeared from the source code, that's why I closed this issue.


Improve carousel CSS

First, awesome job guys! I love using Bootstrap, and appreciate all of the hard-work-made easy!
That said, at first glance I feel like carousel has been put together without much additional development. Some of this is theme changes, but in my opinion, this would greatly improve .carousel. Changes/improvements are as follows:

.carousel-caption

Added gradient for improved readability in front of images.

.carousel-control

Removed gradient to match better to prior change.
Added gradient to .carousel-control span  controllers to match better to prior change.

img

Centered.
Forced max-height (smaller on xs).
Defaults img to overflow: hidden.
Fit img to fit with optional class scale.

I've appended the following changes to the native Bootstrap CSS.
		.carousel-control {
			min-width: 10px;
			max-width: 100px;
		}
		.carousel-control, .carousel-control.left, .carousel-control.right {
			background-image: none;
		}
		.carousel-control span {
			width: auto !important;
			height: auto !important;
			border-radius: 50%;
			padding: 5px;
			background-image: -webkit-radial-gradient(circle at center, rgba(0,0,0,.0001) 0,rgba(0,0,0,.5) 100%);
			background-image: -o-radial-gradient(circle at center, rgba(0,0,0,.0001) 0,rgba(0,0,0,.5) 100%);
			background-image: radial-gradient(circle at center, rgba(0,0,0,.0001) 0,rgba(0,0,0,.5) 100%);100%);
		}
		.carousel-inner > .item > img {
			overflow: hidden;
		}
		.carousel-inner > .item, .carousel-inner > .item > img.scale {
			max-height: 200px;
		}
		@media screen and (min-width: 768px) {
			.carousel-inner > .item, .carousel-inner > .item > img.scale {
				max-height: 400px;
			}
		}
		.carousel-caption {
			left: 0 !important;
			right: 0 !important;
			background-image: -webkit-linear-gradient(top,rgba(0,0,0,0) 0,rgba(0,0,0,.5) 100%);
			background-image: -o-linear-gradient(top,rgba(0,0,0,0) 0,rgba(0,0,0,.5) 100%);
			background-image: -webkit-gradient(linear,left top,left bottom,from(rgba(0,0,0,0)),to(rgba(0,0,0,.5)));
			background-image: linear-gradient(to bottom,rgba(0,0,0,0) 0,rgba(0,0,0,.5) 40%);
			bottom: 0px;
			padding: 0 40px 50px 40px;
			width: 100%;
		}

Try it out and let me know what you think. Thanks!

Care to throw all that in a jsbin?

Unfortunately, I'm limited by my phone at the moment, but here's what I was able to compile: http://jsbin.com/gevucoraxe/edit?html,css,output

Most of this effort was due to personal needs put together in a matter of hours. It's strictly a rough draft and needs further testing and consideration by the masses.

Fixed scaling issue where overflow was ignored. http://jsbin.com/sacanunafi/edit?output

We can revisit this for v4, but punting as a won't fix for v3. Thanks!


WIP: Made progress ring show label on demand

Linked Issue
Closes #2992
Description
Made label show only on demand
TODO: update site examples and generate changesets
Changsets
Instructions: Changesets automate our changelog. If you modify files in /packages, run pnpm changeset in the root of the monorepo, follow the prompts, then commit the markdown file. Changes that add features should be minor while chores and bugfixes should be patch. Please prefix the changeset message with feat:, bugfix: or chore:.
Checklist
Please read and apply all contribution requirements.

[x] This PR targets the dev branch (NEVER master)
[x] Documentation reflects all relevant changes
[x] Branch is prefixed with: docs/, feat/, chore/, bugfix/
[x] Ensure Svelte and Typescript linting is current - run pnpm ci:check
[x] Ensure Prettier linting is current - run pnpm format
[x] All test cases are passing - run pnpm test
[x] Includes a changeset (if relevant; see above)

There's an option when creating PRs to mark it as a draft, otherwise they have a super tiny option on the sidebar near where you assign reviewers. Not sure why it's not obvious, but it is there! :)

@endigo9740 probably could be permission issues that caused this :D I don't have the option. Could you undraft this and give this another review?


Admin panel: Make selecting names easier

What Issue Are You Seeing or What Feature Needs To Be Added?
Once we have 100s of students  in the system, setting up a session will be a pain.

[ ] filter by school name --> for the pilot we'll need a 4th School named "Goce Delcev"
[ ] type student names / partial completion

Are There Any Links That Would Be Helpful?

The tables are setup for the logic on filterable tables, it just needs to be programmed in: https://github.com/prijatelilab/PrijateliTree/blob/f9cfb8404dbc2aecd2fa1929b4e7e033be51a62c/prijateli_tree/app/templates/admin_dashboard.html#L7-L12
https://datatables.net/examples/styling/bootstrap5.html

Advice from ChatGPT:
To implement the functionality of selecting several items from a datatable in JavaScript and then returning those selected items to the database, you can follow these general steps:

Create the DataTable:
Use a library like DataTables or any other suitable library to display your data in a table format. Initialize the datatable and make sure it supports row selection.
Example using DataTables:
<table id="myDataTable">
    <!-- DataTable columns and headers go here -->
</table>

$(document).ready(function() {
    $('#myDataTable').DataTable({
        // DataTable options and configurations
        select: 'multi', // Enable multi-row selection
    });
});

Add Select/Deselect Handlers:
Attach event handlers to capture selected rows. DataTables provides events like select and deselect which you can use.
$('#myDataTable').on('select.dt deselect.dt', function() {
    // Handle selection/deselection logic here
    var selectedRows = $('#myDataTable').DataTable().rows({ selected: true }).data().toArray();
    console.log(selectedRows);
});

Store Selected Items:
Maintain a data structure (e.g., an array) to store the selected items in memory.
var selectedItems = [];

$('#myDataTable').on('select.dt deselect.dt', function() {
    selectedItems = $('#myDataTable').DataTable().rows({ selected: true }).data().toArray();
});

Send Selected Items to the Database:
When the user is done with the selection, send the selected items to the server-side for updating the database.
function saveSelectedItems() {
    // Assuming you are using AJAX to send data to the server
    $.ajax({
        url: '/api/updateDatabase',
        method: 'POST',
        contentType: 'application/json',
        data: JSON.stringify(selectedItems),
        success: function(response) {
            console.log('Data updated successfully:', response);
        },
        error: function(error) {
            console.error('Error updating data:', error);
        }
    });
}

Trigger the Database Update:
You can trigger the database update when the user clicks a button or performs any action to confirm their selection.
<button onclick="saveSelectedItems()">Save Selection</button>

You can, of course, customize and adapt these steps based on your specific use case and application architecture. Additionally, ensure that you handle security aspects such as input validation and protection against SQL injection when interacting with the database.

Now we have a DataTable that allows the administrator

to filter using the search bar
to select participants by clicking on the names of  players
to get client-side feedback on whether the selection works (6 players, 3 from a given language group)

Remaining potential issues:

The Admin cannot easily see which rows are selected, seems like an easy source of annoyance if they make a mistake once there are 100s of students in the study.
The Admin can filter the data by high school or grade using the search tool, but this seems less intuitive.
Admin cannot select the number of games in this setting (more of an issue during piloting, where we are still playing with this).

closed with #140


Demo is broken

Hi looks the demo is not working, can you please take a look at it?

same, running into an error while running the following command-
python src/inversion.py \ --input_image "assets/test_images/cats/cat_1.png" \ --results_folder "output/test_cat"

Hi @Prakhar-Gupta-03,
Could you provide the error message you encountered when you tried this command?