Command line arguments do not override the configuration file

Hi,
First off, thanks for the great piece of software, it is very very helpful.
I have a little issue here. My configuration file looks as follows:
--mon=true
--military
--calendar=Calendar#cyan
--calendar=Birthdays#magenta
--calendar=Weekly#white
--calendar=Bills#brightblack
--width=15
If I try the following:
gcalcli --cal 'Calendar' quick 'buy milk tomorrow'
I get an 'Error: invalid event text'
However if I remove all the lines with "--calendar" from my configuration file, the above command works without problem. So my guess is that the command line arguments do not override the configuration file.
I'm pretty sure I read somewhere in the documentation that this should not be the case. Is that the expected behaviour?

Use --defaultCalendar in your config file. --calendar is cumulative, so having them in your config means "use these plus anything I specify on the command line."  If you want "use these if I don't specify anything" then --defaultCalendar is the way to go.

Indeed, puting '--defaultCalendar' instead of '--calendar' in the config file and doing:
gcalcli --calendar 'Calendar' quick 'buy milk tomorrow'
did the job..
Many thanks


[3.3] Fix spamming errors when SoftBody pinned nodes have no attachment

There was a specific case where the node path wasn't checked for validity before trying to access the attachment node.
It could cause lots of error log noise in both editor and game.
3.3 backport of PR #47151

Thanks!


ER: capture java command line arguments during discovery

During discovery of an online domain, store command line arguments for the JVM in the model

Closing for now


Need to add Category information

I'm submitting a...

[ ] Bug report
[ ] Performance issue
[x] Feature request
[ ] Documentation issue or request
[ ] Other... Please describe:

Current behavior

Currently there is no Catagory user interface.
Expected behavior

There should be an user interface to adding Catagory.

Category added.


[stable/spotify-docker-gc] Support tolerations and node selectors

We run tainted masters and still want to be able to schedule GC daemon set there. Other charts support these types of vars so I'd like to extend this chart as well. Toleration like
tolerations:
- effect: "NoSchedule"
  operator: "Exists"

allows to bypass master taint.
Node selector also useful. Use case: we have a set of nodes used for jenkins job scheduling which rotate tons of workload during the day. Thus I want a custom GC schedule for them (more frequent). With node selectors, I can solve this by creating another helm installation with a custom schedule, limited to my worker type nodes only.
Based on https://github.com/kubernetes/charts/blob/master/stable/fluent-bit/templates/daemonset.yaml#L62-L69

/assign @mattfarina

Testing:

Update vars to:

tolerations:
- effect: "NoSchedule"
  operator: "Exists"

nodeSelector:
  disktype: ssd

Template

---
# Source: spotify-docker-gc/templates/daemonset.yaml
apiVersion: extensions/v1beta1
kind: DaemonSet
metadata:
  name: spotify-docker-gc
spec:
  updateStrategy:
    type: RollingUpdate
  template:
    metadata:
      labels:
        daemonset: spotify-docker-gc
    spec:
      containers:
      - name: spotify-docker-gc
        image: "docker.io/spotify/docker-gc:latest"
        imagePullPolicy: IfNotPresent
        command: ["/bin/sh"]
        args: ["-c", '
          touch /var/log/crond.log &&
          echo "0 0 * * * /docker-gc >> /var/log/crond.log 2>&1" | crontab - &&
          crond -L /var/log/crond.log &&
          tail -f /var/log/crond.log' ]
        env:
        - name: GRACE_PERIOD_SECONDS
          value: "0"
        - name: DOCKER_API_VERSION
          value: ""
        volumeMounts:
        - name: docker-socket
          mountPath: /var/run/docker.sock
      volumes:
      - name: docker-socket
        hostPath:
          path: /var/run/docker.sock
      nodeSelector:
        disktype: ssd

      tolerations:
        - effect: NoSchedule
          operator: Exists

Please bump the minor chart version.

/assign

@unguiculus could you please take a look again, it looks like this can be easy one.

Bump the minor version, please.

/ok-to-test

/lgtm


[CONTROLLER/PROMETHEUS] optimizes log to avoid swiping

This PR is for:

Server

optimizes log to avoid swiping
Checklist

[ ] Added unit test.

Backport to branches

main

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.zhengya seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


Support multiple-qualifier type names

CQL STU3 generalizes the grammar for named type specifiers to support multiple-qualifier type names. Support this change in the translator and test that it functions correctly with FHIR component types.
See STU comment 1507

Included in the 1.3 CQL Ballot release.


[ALLUXIO-2604] Deprecate Throwables.propagate with RuntimeException in AbstractClient.java

https://alluxio.atlassian.net/browse/ALLUXIO-2604

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/14062/
Test PASSed.


Fundamentals: Plane waves in homogeneous media

I suggest we emphasize the apps and their usefulness. This deserves discussion since this section can serve as a template for other sections. The goal is to get engagement first and then have people go into derivations.
For this section we have:
Introduction (current one is ok)
Simulation: Plane Wave App.
Existing material in Setup
An introduction about the app and its functionality
Learning Exercises  (these can address each of the items in the introduction that focuses on what will be learned)
Skin Depth: A plane wave attenuates as it propagates into the earth. Use the default frequency of 10Hz and conductivity sigma=1 for the initial simulation.
Plot the amplitude with depth and estimate the skin depth.
Evaluate the skin depth using the formula xxx
How does the skin depth change if f=100Hz and sigma=0.1
How does the skin depth change f=1 and sigma=1
experiment…..
Real and Imaginary Components: As the plane wave propagates into the earth there is a real (in-phase) and imaginary (quadrature or out-of-phase) component. By default the App shows these signals at time t=0, so at the surface the Ex field is purely real and the Hy field is purely imaginary. Why is this true?
Consider a depth z=XXX meters. Evaluate the real and imaginary components of the field at that point and combine them into an amplitude value. How does your number compare to that from App. Use the attenuation formula (XXX) to compute the theoretical amplitude at that depth.
Evaluate the phase using the Real and Imaginary parts of the electric field at z=xxx meters. How does your number compare with that provided by the app? Notice that the phase curve exhibits some strange behavior. Does this seem physically real and what is happening?
Wavelength:
There a number of plots from which you can estimate the wavelength of the EM fields. Which one seems the most precise (and why)?. Compare the number with the theoretical value (eq xxx  lambda=2pi delta)
Phase Velocity
The App provides a snapshot of the waves in the earth at time t=0. As time progresses the waves propagate downward. If you follow an individual peak or trough with time you can estimate the phase velocity. Do this by adjusting the time slider. Pick a particular peak at two times (say 0.02 and 0.04 seconds) and estimate the phase velocity. Compare your result with the theoretical result given by equation xxxx
Impedance
The impedance Z=E/H is a complex number and it is viewable in the App along with the E and H fields. The fields vary with depth and with time but the impedance has a constant amplitude and phase. Use the theoretical relationships (eq xxx) to validate the numbers in the graph.  (other questions ??)
Apparent resistivity
Impedances can be converted to apparent resistivities and phases. Use your impedance values and eq xxxx to compute the apparent resistivity. Note that it doesn’t matter at what depth the measurements were obtained.
The remainder of the current section (derivations for downward propagating fields) can go down one level.

Google doc
https://docs.google.com/document/d/1D3WeClMCvoZoqPWKmh6kehZN_bZqzhXwir0Lb5aNlfQ/edit#

http://em.geosci.xyz/content/maxwell1_fundamentals/plane_waves_in_homogeneous_media/index.html


Link equations to functions in docs

A suggestion I received: It would be good to make it clear which functions correspond to which equations in the docs. For example, can we give equations for each function here? https://ccl.readthedocs.io/en/latest/api/pyccl.correlations.html

Yeah, I started doing this with the halo model, but it's far from homogeneous. Having this would be great.


Lib 789 save object befor add relation

Looks good. Though I would extract somewhere this code:
[self saveMeIfNeededWithAPIClient:apiClient completion:^(NSError * _Nullable error) {
        typeof(self) __weak selfWeak = self;
        [apiClient PATCHWithPath:[self path] params:params completion:^(NSURLSessionDataTask *task, id responseObject, NSError *error) {
            if(error) {
                if(completion) {
                    completion(error);
                }
                if(revisionMismatchBlock) {
                    [error checkIfMismatchOccuredWithCompletion:revisionMismatchBlock];
                }
                return;
            }

            [selfWeak fillKeys:keys fromResponseObject:responseObject];

            if(completion) {
                completion(nil);
            }
            if(revisionMismatchBlock) {
                revisionMismatchBlock(NO,nil);
            }
        }];
    }];

cause it's the same in all 3 methods - not reason to just copy-case, instead of creating a new method out of it.


Implement HTTP Zapier rule target resource

┆Issue is synchronized with this Jira Task by Unito

Done in #107


Support Timezones

The default time zone supported by Wizarr is GMT. Usually, we can change this via Docker environment variables but this functionality is not supported on Wizarr (yet). Right now, it's 2023/01/18 5:53 PM Pacific time but the times show created 2023/01/19 01:53.
Feature request to support telling docker-compose file with environment variables to set the time zone of the container.
- TZ=America/Los_Angeles

Or, it is possible that  TZ= works, but the value stored in the expiry doesn't respond to timezone changes. For example:

The used is before the created date now. I would expect that changing the timezone in the system would update the timestamps across the platform.

Added Support for TZ in V1.0.1
https://github.com/Wizarrrr/wizarr/releases/tag/1.0.1


Powershell script containing pipe causes failure

Agent Version and Platform
2.153.2
Windows 7
Azure DevOps Type and Version
dev.azure.com (formerly visualstudio.com)
https://lms-bioptigen.visualstudio.com
What's not working?
I have a custom build script (Powershell) that pulls down a perforce repository containing our org's signing certificates. The script builds a temporary workspace spec and outputs it to standard output with the pipe that reads in the spec from standard input. i.e.
P4.exe -d "path_to_checkout_repo_to" client -o "unique_client_name" | p4.exe client -i
This worked as expected on version 2.141.1 of the build agent. However, on version 2.153.2, I get errors from the p4.exe command line as well as Syntax error in ''. from the execution of the build task.
Diag.zip

@dkalbertson you are using a customized task, you might need to find the owner of that task and report the error.
      "reference": {
        "id": "860e6431-0940-4d7b-a2cf-c3c9dab8d63e",
        "name": "PerforceCheckout",
        "version": "1.0.3"
      },

@TingluoHuang I am the owner of the customized task. If I switch my build agent back to version 2.141.1 this task works correctly. When I update the agent to the newest (which is currently 2.153.2). The task errors out.

can you share a build log?

Sure! see attached...
logs_8902.zip

@TingluoHuang I'm attaching a second build log from agent 2.141.1 that works as expected
logs_8921.zip

@dkalbertson I can't find any obvious change might cause this, do you might help me narrow down to which version of the agent introduces the regression? Pick a version between 141.1 and 153.2 binary search which version has the problem.
Also, are those two build runs on agents on the same machine?

@TingluoHuang I apologize for the radio silence. I've been working on getting a release out. I'll try to make some time to binary search the different versions within the next week.
To answer your question, yes, the two builds were run on the same build machine. They were actually run on the same agent. I just changed versions on that agent in between runs.

@TingluoHuang I did some testing and narrowed down the build agent version that first introduced the failure. Build Agent version 2.144.1 produces the failure. The previous version, 2.144.0 does not. The test builds ran on the same machine and on the same build agent. I only change the version of the build agent and restarted the agent in between test runs. I verified by switch the versions back and forth a few times. I've attached logs from the two builds.
BuildAgent_2.144.0_logs_9161.zip
BuildAgent_2.144.1logs_9160.zip

@dkalbertson can you try add the following variable to your definition
agent.retainDefaultEncoding=true and run the build again on 2.144.1 runner? see whether that mitigate the problem.

@TingluoHuang That did mitigate the problem. I already have a variable group of settings for that particular task. So, I'll include agent.retainDefaultEncoding=true  in that variable group as well. Thanks for the support.

Agent Version and Platform
2.153.2
Windows 7
Azure DevOps Type and Version
dev.azure.com (formerly visualstudio.com)
https://lms-bioptigen.visualstudio.com
What's not working?
I have a custom build script (Powershell) that pulls down a perforce repository containing our org's signing certificates. The script builds a temporary workspace spec and outputs it to standard output with the pipe that reads in the spec from standard input. i.e. P4.exe -d "path_to_checkout_repo_to" client -o "unique_client_name" | p4.exe client -i
This worked as expected on version 2.141.1 of the build agent. However, on version 2.153.2, I get errors from the p4.exe command line as well as Syntax error in ''. from the execution of the build task.
Diag.zip

Workflow


unable to download apktool_2.2.3.jar

well ,when i try to download the  apktool_2.2.3.jar from https://bitbucket.org/iBotPeaches/apktool/downloads/;
it shows below in the browser :
This XML file does not appear to have any style information associated with it. The document tree is shown below.
<Error>
AccessDenied
<Message>Request has expired</Message>
<Expires>2017-07-18T13:56:40Z</Expires>
<ServerTime>2017-07-18T14:08:32Z</ServerTime>
<RequestId>43DB2BE3678B0997</RequestId>
<HostId>
dQ9h/jdmHnDAyLJEPGINRSSie9T8dgl8ldq47MD1NGP4Jk+1HokZxpdLh49ovyDOkM+dRHeXYLg=
</HostId>
</Error>

is it sth wrong with the bbuseruploads.s3.amazonaws.com?

Works fine here. Must be have a been a temporary problem.
Ugh bitbucket.


Does not seem to be working for me

It creates the linode fine, but then:
/home/jackson/.local/bin> ./linode_deploy_experimental freebsd-11-0
Fetched all datacenters
Fetched all distributions
Fetched all plans
Created a linode
Waiting for Linode to be created
Added private IP
Created first disk
Created second disk
Waiting for disks to be created
Waiting for disks to be created
Waiting for disks to be created
Disk created
Waiting for disks to be created
Disk created
Created install config
Created normal config
Waiting for Linode to boot into install config
Waiting for Linode to boot into install config
Waiting for ssh server to start
Fetched ips
Started image deploy process
Oh know! An exception!
Traceback (most recent call last):
File "./linode_deploy_experimental", line 9, in
load_entry_point('linode-deploy-experimental==0.1', 'console_scripts', 'linode_deploy_experimental')()
File "/home/jackson/.local/lib/python3.5/site-packages/linode_deploy_experimental/main.py", line 199, in main
raise e
File "/home/jackson/.local/lib/python3.5/site-packages/linode_deploy_experimental/main.py", line 166, in main
wait_for_ssh(ip.ipaddress, tmp_password, cmd)
File "/home/jackson/.local/lib/python3.5/site-packages/linode_deploy_experimental/main.py", line 37, in wait_for_ssh
while subprocess.call([tcl_path, address, password, cmd]) > 0:
File "/usr/lib64/python3.5/subprocess.py", line 247, in call
with Popen(*popenargs, **kwargs) as p:
File "/usr/lib64/python3.5/subprocess.py", line 676, in init
restore_signals, start_new_session)
File "/usr/lib64/python3.5/subprocess.py", line 1282, in _execute_child
raise child_exception_type(errno_num, err_msg)
FileNotFoundError: [Errno 2] No such file or directory: '/home/jackson/.local/lib/python3.5/site-packages/linode_deploy_experimental/ssh.tcl'
Doesn't matter if what image I select, they all fail the same way.
If I load a glish console, on boot it shows a PXE message, then a GRUB error "error variable prefix is not set", then boots to a Debian text login.

I needed to install the package 'expect' on Fedora 25, now it's working, you can close this.

@jpeirce thanks for taking a look at that. I'll add a note to the README.md about the expect dependency.


VM Sizes Must Match template test not working

I'm using the latest commit (9dec6bc4d16d1357a7dab5e64c51185ba1571e46 by the time of writing) and the test VM Sizes Must Match template test is not working.
Steps to reproduce:

Clone the repository
Go to unit-tests/VMSizes-Must-Match-Template/Pass/Element-Nested-In-Section
Execute Test-AzTemplate.sh

Error shown:
    [-] VMSizes Must Match Template (49 ms)
                                                                          Exception calling "Create" with "1" argument(s): "At line:1 char:28
+ $CreateUIDefinitionObject.[0].parameters[0].steps[0]
         +                            ~
              Missing type name after '['.                                                                                At line:1 char:27
     + $CreateUIDefinitionObject.[0].parameters[0].steps[0]
+                           ~
Missing property name after reference operator."

For some reason, the JSONPath has this [0] at the beginning, which is causing issues.
Details about my system:

Mac OS Catalina
PowerShell 7.0.1
Installed coreutils and powershell using brew

CUID test seems fine... check out the CUID in: https://aka.ms/createuidef/sandbox
looks like a problem with the test

We're running into the same issue here.
Test "VM Sizes Must Match Template" executed on Ubuntu 18 fails with the described error.
However, when I execute the tests on my local Windows 10, the test passes. I'm using the same template and CUID and cloning the latest arm-ttk version from this repo.

should be fixed, if you're still seeing please attach a repro

should be fixed, if you're still seeing please attach a repro


swap from greenkeeper to GitHub

GitHub already in place, but Greenkeeper is doing some weird stuff, not making PR's of commits to branches it creates itself.


Mixin Error - Cannot apply MixinWorld

This seems like a bug in SpongePowered/Mixin so I'll file it there but I'll mention it here for people so it's not duplicated.
https://gist.github.com/modwizcode/2c384ed79de36a1f7c90
https://github.com/SpongePowered/SpongeVanilla/blob/master/src/main/java/org/spongepowered/vanilla/mixin/world/MixinWorld.java#L83
EDIT:
Maybe not, I'm not sure how surrogate methods work. I'll defer to @Mumfrey for judgement.

@Mumfrey take a look please

You hath summoned me, you may now request three wishes.
... ahem In other news, I'll take a look at it in a bit.

Fixed when unified world changes were merged.


Next

This bug doesn't allow receiving all form values as argument in handleSubmit function.

:tada: This PR is included in version 2.0.4 :tada:
The release is available on:

npm package (@next dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

:tada: This PR is included in version 2.0.4 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


fix: eslint

Brief Information
This pull request is in the type of:

[x] bug fixing
[ ] new feature
[ ] others

What does this PR do?
fix eslint formatting
fix action lint

Fixed issues

Details
Before: What was the problem?

After: How is it fixed in this PR?

Misc

[ ] The API has been changed (apache/echarts-doc#xxx).
[ ] This PR depends on ZRender changes (ecomfe/zrender#xxx).

Related test cases or examples to use the new APIs
pnpm lint
Others
Merging options

[ ] Please squash the commits into a single one when merging.

Other information

Deleted the code that was introduced but not used in the code as follows

BoxplotSeriesModel
SunburstSeriesNodeItemOption
TreemapSeriesOption
GetPieceValueType

Thanks for the changing! Did you use some auto formatting tools (like prettier) or manually changed this?

Thanks for the changing! Did you use some auto formatting tools (like prettier) or manually changed this?

Yes, I do use the auto format tool, I use the prettier that comes with vscode

In [the action lint] (https://github.com/apache/echarts/actions/runs/2420710373),

There is always an issue with not being able to pass Lint, maybe you should pay attention to this

Consider using  tslint+prettier+husky+lint-staged code formatting specification guarantees

Thanks for the suggestion. I think the main problem we need to decide is what to do with the old stylings. Please continue the discusstion in the mailing list. I would like to close this PR if you are OK with that.


Align JDK 16 compiler toolchains with upstream

Sources:

https://wiki.openjdk.java.net/display/Build/Supported+Build+Platforms
strings -a bin/javac | grep "GCC:" on upstream binaries

Actions:

[x] Upgrade JDK16+ to VS 2019 on Windows x86_64 and x86_32
[ ] Upgrade JDK 16+ to gcc 10.x on Linux x86_64 (10.2 is currently in use; Alpine is still on 8.3)
[ ] Upgrade JDK 16+ to gcc 10.x on Linux aarch64 (10.2 is currently in use)

Unknown:

[ ] Check whether JDK16+ use XLC/xlclang++ 16.1 on AIX ppc; otherwise upgrade.

Already okay:

JDK16+ uses gcc 7.5 on Linux arm32, ppc64le, s390x. (However, a brave soul might trying a newer version)

Does this affect OpenJ9 builds? From an OpenJ9 perspective:

Windows VS2019 yes - https://github.com/AdoptOpenJDK/openjdk-build/issues/2147
OpenJ9 switching to gcc 10.x hasn't been tried, not sure if it can build, or if it will introduce any problems, or how it will affect performance.
JDK16+ already uses xlclang 16 on AIX

Does this affect OpenJ9 builds?

No, that was not the plan. But thanks for the info nonetheless.

We cannot simply upgrade to GCC 10.2 because it cannot be compiled on RHEL/CentOS 6 anymore:
./configure --prefix=/usr/local/gcc10
...
configure: error: Building GCC requires GMP 4.2+, MPFR 3.1.0+ and MPC 0.8.0+.

The versions of GMP, MPFR, and MPC bundled with RHEL 6/CentOS 6 are too old. RHEL/CentOS 7 is fine.

Took a while, but we're in the process of setting up GCC 10.3 under https://github.com/adoptium/temurin-build/issues/2787 - this seems to produce a usable build of OpenJDK for versions 8 and 17. Adjustments to the build to utilise it are expected to wait until after the January releases.

Final step on this will be https://github.com/adoptium/temurin-build/pull/2861


fix(ts): change wrong injectable class on typescript guide 02

Just signed the CLA with both accounts

I'm the author of the commit but with another email

Confirmed you've signed the CLA. The fix looks good to me, so I'm accepting it. Thanks!
/cc @rkirov @davideast


API Subsets Permissions

we should come up with a good permissions model for the api-- i.e. be able to grant permissions to other peers, or HTTP API consumers, or to specific apps to strict subsets of the API. (say RO, or on subset of commands, or forcing a given option to be set, etc).

This would be a good way to eventually do away with the 8080 and 5001 distinction-- IF we can be certain it works well, or rather, make it so that you can add arbitrary endpoints (at more ports if desired) with different permissions. so doing away with the hard-coded distinction and make it more a config.

since our API is a command tree, and options can be expressed as query strings, we could create policies that effectively do this:
deny *
allow ipfs/object/get
allow ipfs/cat
allow ipfs/bitswap/stat
allow ipfs/logs?filter=bitswap

not sure this is the best language at all, but it is a language that doesn't seem cumbersome now.

How do you want to do this, @jbenet? The best way would be to talk about this in specs, I think, and then mention the permissions in each method description, I think.


Refresh Apache Airflow configuration documentation

Fixes for broken or missing links and formatting

Fixes #1305
Developer's Certificate of Origin 1.1
   By making a contribution to this project, I certify that:

   (a) The contribution was created in whole or in part by me and I
       have the right to submit it under the Apache License 2.0; or

   (b) The contribution is based upon previous work that, to the best
       of my knowledge, is covered under an appropriate open source
       license and I have the right under that license to submit that
       work with modifications, whether created in whole or in part
       by me, under the same open source license (unless I am
       permitted to submit under a different license), as indicated
       in the file; or

   (c) The contribution was provided directly to me by some other
       person who certified (a), (b) or (c) and I have not modified
       it.

   (d) I understand and agree that this project and the contribution
       are public and that a record of the contribution (including all
       personal information I submit with it, including my sign-off) is
       maintained indefinitely and may be redistributed consistent with
       this project or the open source license(s) involved.

Rebased and updated based on comments @ptitzler

make docs triggers warninga:
/Users/patti/elyra_test/airflow-docs/elyra/docs/source/index.rst:60: WARNING: toctree contains reference to nonexisting document 'developer_guide/deploying-kubeflow-locally-for-dev'

/Users/patti/elyra_test/airflow-docs/elyra/docs/source/developer_guide/release.md: WARNING: document isn't included in any toctree

Looks like the official names are GitHub and GitHub Enterprise (note the case)


Merge torii fire

This has two commits. The first extracts the Firbase instace into a service.
Why:

We'd like to use the firebase instance in multiple places, rather then
importing the application adapter, it would be better to have a service
that provides this.

This Commit:

Creates a firebased service that provides a global instance of
Firebase.

The second Add a torii adapter for firebase.
Why:

It would be great to use firebase authentication more easily.

This Commit:

Merges the torii-fire addon into emberfire
This adds a torii adapter for firebase

I made a service for the firebase instance. I think this makes sense for if anyone who want's to grab that instance for other addons, however, the name I've picked is terrible so I'd love some suggestions.
Also I'm not sure if it makes sense to add documentation to the readme or to throw something into the wiki and link to that in the readme. Thoughts?

Also let me know if you want these split into two PRs.

I'm wondering if we can just use a resolver lookup for the firebase instance. The service feels like overkill.
this.container.lookup('firebase:main');

thanks for this btw!

So instead of using a service make an initializer to register firebase:main?
I think this type of dependency is perfectly suited for using a service. Using an initializer to register the firebase object feels like trying to get the same functionality of a service but throwing it into the container. It also means that anywhere we want to use it we'll need to use:
firebaseInstance: function() {
  return this.container.lookup('firebase:main').get('firebaseInstance');
}.property()

Which doesn't seem as elegant as using a service.
Thoughts?

I'd register the actual root reference as firebase:main rather than a containing object.. think store:main
It would be more like this:
// initializer
container.register('firebase:main', new Firebase(config.firebase), { instantiate: false });

// adapters/firebase.js
// inside init()
this._ref = this.container.lookup('firebase:main');

// torii-providers/firebase.js
this.container.lookup('firebase:main').authWithOAuthPopup(...

then, if we really wanted (we don't really have a need atm) we could inject this into things:
container.inject('route', 'firebase', 'firebase:main');

and we could also modify the container to overload this reference when testing.. perhaps this will cleanup all the manual _ref set/restore code in the tests.
To me, a service implies that the firebased thing does something complex
Pre #249, users can access the firebase ref with this.container.lookup('firebase:main') but after it is merged they can get a records ref with record.ref()

@tstirrat Sorry this has been sitting open for a while.
I'd like to talk again about using a service here instead of registering in an initializer. I believe ember-data has been moving towards using the store as a service rather then store:main. So when you need the store in a component you can simply say:
export default Ember.Component.extend({
  store: Ember.inject.service(),
  ...
})

This seems like the Ember 2.0 way of handling these sort dependencies that you may need to be globally available. I also prefer the idea of being explicit about when I need this dependency in my objects rather then, say, injecting it into all of the routes.
To me, a service describes something that needs to be globally available. The firebase instance seems to fix this description perfectly.
Thoughts?

I've updated the service with some help from @rwjblue so it's usage is less offensive.

A few small nitpicks, but otherwise looking good. I much prefer the new service syntax, thanks for updating it!

Comments should be addressed!

Just checking , have you signed the CLA?

@tstirrat just signed it.

Is there any documentation for how to use this with custom authentication?

@artburkart use like this:
this.get('session').open('firebase', {
    provider: 'custom',
    token: '<customToken>' // your custom generated token
  })
  .then(function(data) {
    console.log(data.currentUser);
  });

I've made a note to add custom to the docs

Hmmm, I did that and it didn't seem to work. I'll have to look into it more. Thanks!

It was added in #307
Make sure you're using EmberFire 1.6.0+ or at least one that contains this commit

If you find it is still not working in 1.6.0 please post your ember/emberfire/emberdata versions and any relevant console output in issue #307

@tstirrat - thanks for the assistance. I figured it out! I implemented my custom authentication in elixir and my tokens were not of the proper format. I'm not sure where the authentication format is documented, but I figured it out after reading the nodejs firebase token generator. I had initially read the python one and it didn't make the format as clear, so I missed the d.


add recording and alerting rules for kserve

JIRA : https://issues.redhat.com/browse/RHOAIENG-98

we want this for 2.5 or post-2.5?


Camera Control For Starfighter

Can there be an easier way to record different camera angles for the starfighters? Right now it is hard to catch up with the starfighter you are currently controlling.

This is unlikely to be worked on any time soon.


Deserializing Web Api parameter fails in case of large json

I have a Asp.Net Core 2.0 Web API service that has a HTTP Post Method that contains a byte[] parameter which fails to get deserialized when larger JSON like 30 mb is sent as body content thus having null value. But for small json it's been able to deserialize properly into byte array.
Basically what i am trying to do is i have to send byte array to Asp.Net Core 2.0 Web Api endpoint that will upload this payload as blob storage in azure. Below is the code for Api and the calling code to this Api
Note here payload parameter is having null value:
 public class MyBlobController : Controller
        {

    [HttpPost("UploadPayload/{containerName}/{blobName}")]
    public async Task<string> UploadPayload([FromBody] byte[] payload,string containerName, string blobName)

       {

        //code for uploading blob

        }
    }

Calling code for this end point
var jsondata = JsonConvert.SerializeObject(payloadData);//payload is large byte array

        HttpContent content = new StringContent(jsondata, Encoding.UTF8, "application/json");
        //Instantiating the HttpClient.
        {
            var HttpClientInstance = new HttpClient();

            HttpClientInstance.DefaultRequestHeaders.Clear();
            HttpClientInstance.DefaultRequestHeaders.AcceptLanguage.Add(new StringWithQualityHeaderValue("nl-NL"));
            HttpClientInstance.DefaultRequestHeaders.Accept.Add(new MediaTypeWithQualityHeaderValue("application/json"));

        }
        HttpResponseMessage apiResponse = null;
        //Sending a post request to the Blob Api.
        apiResponse = await HttpClientInstance.PostAsync("http://localhost:8999/api/MyBlob/UploadPayload/mycontainer/myblob", content);

I have also gone through this link https://github.com/aspnet/Announcements/issues/267 that tells us how to change the Maximum limit size of HTTP Post request i have also enabled that setting such that it is able to accept large request but still no luck with deserialization. Please suggest something that could be really useful to resolve this use case

Hi @shubham1413. Can you please share a repro project for us to investigate? Also, have you checked what are the ModelState errors, when you send big JSON file?

@kichalla, can you please work with @shubham1413 on this? Thanks!

@shubham1413 Can you do what @mkArtakMSFT suggested? Also you needn't serialize the byte array into json and then deserialize it back. You can post the data as StreamContent in HttpClient and read the request body as stream from within the action method

Closing this issue now as we haven't heard back. Feel free to re-open this issue with more details as requested above.


Release 0.16.4

This PR prepares the release of MatrixKit v0.16.4.
Notes:

This PR targets release/0.16.4/master, which has been cut from master.
It includes changes to the Podfile, but not the corresponding changes to Podfile.lock, as pod install hasn't yet been run.
This is because the Podfile targets future versions of dependencies yet to be released, so pod install wouldn't be able to find them yet.
When the CI runs its checks, it will temporarily point to the pending release branches of those dependencies beforehand
It is only during release:finish that pod update will be run -- updating the Podfile.lock
to use the now officially released dependencies -- before ultimately merging release/0.16.4/master into master to tag the release.

➡  Once this PR is merged, you will need to first ensure that the products this one depends on are fully released,
then run bundle exec rake release:finish to close this release.
💡 If you want to review only the changes made since the release branch was cut from develop,
you can check those here

CI is wrong is checking against MatrixSDK/develop.
This is just a SDK update. On my machine pod lint is fine. I merged it like it.
MatrixKit passed validation.


10.13 supported for kobodl?

Does kobodl work with high Sierra 10.13.6? Python3.7
To Reproduce
Steps to reproduce the behavior:

Installed kobodl using bundle to Mac, in terminal
wget https://github.com/subdavis/kobo-book-downloader/releases/latest/download/kobodl-macos-latest
chmod +x kobodl-macos-latest
All looked ok, no errors

Check if installed ok , in terminal
./kobodl-macos-latest user --help

See error
Macintosh:kobodl MENA$ ./kobodl-macos-latest user --help
[2251] Error loading Python lib '/var/folders/x4/wyjknyfx731_tl1t7g8wcnt40000gn/T/_MEIFZMs8j/libpython3.8.dylib': dlopen: dlopen(/var/folders/x4/wyjknyfx731_tl1t7g8wcnt40000gn/T/_MEIFZMs8j/libpython3.8.dylib, 10): Symbol not found: ____chkstk_darwin
Referenced from: /var/folders/x4/wyjknyfx731_tl1t7g8wcnt40000gn/T/_MEIFZMs8j/libintl.8.dylib (which was built for Mac OS X 10.15)
Expected in: /usr/lib/libSystem.B.dylib
in /var/folders/x4/wyjknyfx731_tl1t7g8wcnt40000gn/T/_MEIFZMs8j/libintl.8.dylib

5 checked python folder (library/Python). 2.7 and 3.7 folders are there
6 Installed python 3.9 after getting error, first time.
Redid installation and same error results.
Expected behavior
Kobodl working on high Sierra 10.13
Desktop (please complete the following information):

OS: [e.g. 10.13.6]
bundle latest for Mac, python 3.7
Kobodl Version [e.g. 0.6.0]

Additional context
cheers

Apparently Kobodl was built on a version of MacOS that is too new and therefore doesn't work on 10.13
https://pyinstaller.readthedocs.io/en/stable/usage.html#making-mac-os-x-apps-forward-compatible
I'll see if I can build it on an older version.

Thankyou. That would be greatly appreciated.

I've looked and it will not be possible to build for anything older than catalina because github only has runners for catalina and big sur.  I'll update the documentation saying that MacOS Catalina (10.15) or newer is required.
https://docs.github.com/en/actions/using-github-hosted-runners/about-github-hosted-runners

You can easily use kobodl by using a different install method, such as with pip.
pip install kobodl

Ok. Thankyou for trying.


[0.12.n bug] clock sync test: stop/start button does not unload previous assets

Pressing stop: previous digits stop animating but aren't unloaded. Pressing play: Duplicate assets are loaded and start animating. Old digits are still visible.
Possible regression in the asset system, or a new bug in the test harness.

Was testing in Altspace, mredev branch, running in Unity, single user, with latest 0.12.n DLL built locally.

This is a problem with the stop button, repros on many different tests. Non-fatal.

I seem to recall fixing this, but I can't find the PR where it's fixed. Will verify.


CLI playbook bug fixes

This PR

Fixes a bug where command line flags cannot be used for the backend tag in playbooks ( the {{/* =% some_command %= */}} tag)
Refactored the pkg/cli so that different backends (sh, print, instanceProvision, managerCommit) are now separate packages.  This will pave way for other backends such as http (for Postman -like functionality) and ssh in playbooks
implemented a registry for the backends and updated the main to load the packages containing the backends the CLI would support.

Please sign your commits following these rules:
https://github.com/moby/moby/blob/master/CONTRIBUTING.md#sign-your-work
The easiest way to do this is to amend the last commit:
$ git clone -b "cli-http" git@github.com:chungers/infrakit.git somewhere
$ cd somewhere
$ git rebase -i HEAD~842353719976
editor opens
change each 'pick' to 'edit'
save the file and quit
$ git commit --amend -s --no-edit
$ git rebase --continue # and repeat the amend for each commit
$ git push -f

Amending updates the existing PR. You DO NOT need to open a new one.


fix redrawing of dynamic data; progress on #247

Codecov Report

Merging #461 into experimental will decrease coverage by 1.55%.
The diff coverage is 21.31%.

@@               Coverage Diff                @@
##           experimental     #461      +/-   ##
================================================
- Coverage         49.92%   48.36%   -1.56%
================================================
  Files                50       51       +1
  Lines              3231     3339     +108
================================================
+ Hits               1613     1615       +2
- Misses             1618     1724     +106

Impacted Files
Coverage Δ

vent/api/actions.py
51.67% <100%> (-0.37%)
:arrow_down:

vent/menu.py
46.06% <15.38%> (-9.82%)
:arrow_down:

vent/menus/remove_tools.py
20.23% <20.23%> (ø)

vent/menus/main.py
10.67% <25%> (+0.27%)
:arrow_up:

vent/api/plugins.py
79.08% <42.85%> (-3.38%)
:arrow_down:

tests/test_api_actions.py
96.49% <0%> (-3.51%)
:arrow_down:

vent/api/templates.py
83.33% <0%> (-3.34%)
:arrow_down:

vent/helpers/meta.py
71.14% <0%> (+1.34%)
:arrow_up:

Continue to review full report at Codecov.

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


Handling several slots in the same sentence

Hi all,
Thanks to this great library :-)
I would like to use several slots within a same sentence but the produced json file does not pick up the right start and end of the slots.
Below a simple example:
******** txt_file ***********
%&ask_toilet
where the @[toilet#singular] is @[please]?
@[toilet#singular]
toilet
loo
@[please]
please
plz
***** json result ***********
{
"rasa_nlu_data": {
"common_examples": [
{
"entities": [
{
"end": 13,
"entity": "toilet",
"start": 10,
"value": "loo"
},
{
"end": 42,
"entity": "please",
"start": 36,
"value": "please"
}
],
"intent": "ask_toilet",
"text": "Where the loo is please?"
},
{
"entities": [
{
"end": 13,
"entity": "toilet",
"start": 10,
"value": "loo"
},
{
"end": 39,
"entity": "please",
"start": 36,
"value": "plz"
}
],
"intent": "ask_toilet",
"text": "where the loo is plz?"
},
{
"entities": [
{
"end": 16,
"entity": "toilet",
"start": 10,
"value": "toilet"
},
{
"end": 39,
"entity": "please",
"start": 36,
"value": "plz"
}
],
"intent": "ask_toilet",
"text": "where the toilet is plz?"
}
],
"entity_synonyms": [],
"lookup_tables": [],
"regex_features": []
}
}
Any idea please ???

Hi,
What you report seems related to issue #22 which was fixed in v1.6.1.
Using the template you provided, I cannot reproduce the results you get.
Would you mind giving a little more information about the environment you run chatette in? Namely, I'd need to know:

What version of Chatette are you running? You can find that by running python -m chatette --version.
Which version of python do you use?
What OS are you running?

Thanks in advance :)

Thanks a lot.  It works fine!! I have just one other problème. I use the french language however the actual code does not support UTF8 encoding. Any help please!!!

For Unicode encoding, I guess you are running Windows, which is likely to encode your files using Windows-1252 encoding. Try using a file editor that allows you to save your template files in UTF-8 and feeding them to Chatette should work without a problem.
Using a recent version of python (>= 3.5 I would say) could also help.
Just so you know, I speak French and I have no problem running Chatette to produce French datasets by running Linux (whatever the Python version) and encoding all my files in UTF-8.
Hope this helps! Feel free to ask if you need help again!

Thanks a lot. It works fine for me.
Just another question. How to avoid redendency. In fact, if i put 100 as the augmentation parameter, the algorithm generate redandant sentences.

What exactly do you mean by "redundancy"? The generated data should never contain duplicates, so if you get the same generated sentence twice (or more times), this is a bug.
If you mean your sentences are too close to each other, this simply depends on your templates. A good way to have variation in the generated sentences is to have a lot of rules in your aliases, slots and templates.
You can take a look at different examples on the repo if you want to see how to make good templates.
Cheers!

Hello @SimGus
I am so sory for my late response.
I have tried to use the simple example of the directory toilets and i have set the number of generated sentences to 1000 instead of 3. The algorithme generate duplicated. Have you a solution to avoid that ??

Hey @Asma-droid,
The two sentences are actually not duplicates: one of them starts with an uppercase letter, while the other one starts with a lowercase letter.
If you want to avoid generating sentences with different cases, simply remove the ampersand & that are at the beginning of the unit declaration (so for the toilet example, remove the ampersand in %[&ask_toilet](100)).
I hope this helps.

Thank you  for this response !

 I will close this issue as it seems fixed.


#246 Added  competitive programming track (Roadmap)

Issue Title
Please enter the title of the issue related to your pull request.
Enter the issue title here-
Added  competitive programming track (Roadmap)

[X] I have provided the issue title.

Info about the Related Issue
What's the goal of the project?
Describe the aim of the project-
To add the roadmap for competitive programming

[X] I have described the aim of the project.

Name
Please mention your name.
Enter your name here-
V Pratheek

[X] I have provided my name.

GitHub ID
Please mention your GitHub ID.
Enter your GitHub ID here-
pratheekv39

[X] I have provided my GitHub ID.

Identify Yourself
Mention in which program you are contributing (e.g., WoB, GSSOC, SSOC, Hactoberfest).
Enter your participant role here-
Contributor at GSSOC , Hacktoberfest

[X] I have mentioned my participant role.

Closes
Enter the issue number that will be closed through this PR.
Closes: #issue-number
#246

[X] I have provided the issue number.

Describe the Add-ons or Changes You've Made
Give a clear description of what you have added or modified.
Describe your changes here-
Added the implementation of Competitive Programming Track

[X] I have described my changes.

Type of Change
Select the type of change:

[ ] Bug fix (non-breaking change which fixes an issue)
[X] New feature (non-breaking change which adds functionality)
[ ] Code style update (formatting, local variables)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

How Has This Been Tested?
Describe how your changes have been tested.
Describe your testing process here.

[X] I have described my testing process.

Checklist
Please confirm the following:

[X] My code follows the guidelines of this project.
[X] I have performed a self-review of my own code.
[X] I have commented my code, particularly wherever it was hard to understand.
[X] I have made corresponding changes to the documentation.
[X] My changes generate no new warnings.
[X] I have added things that prove my fix is effective or that my feature works.

@rishicds Please see the attached video below for your reference . Thanks :)
https://github.com/user-attachments/assets/6265ea1f-7057-4e51-8ba6-aaf519d0b731

It does Not follow the Dynamic Routing of Resource Page, as the Resource Page handles the Page Routing by ID, you made the page dependent on the CP content, and breaks other Data support of Resources. Please fix it

@rishicds Hi, I have made the following changes:

Added a Competitive Programming track following dynamic routing by ID (ID: 6) similar to existing C++ and DSA tracks. I have even corrected the routing of previous roadmaps that I had created . Now , all of the tracks follow dynamic routing.

2)Created separate content files (competitive_programming-content.js and track-contents.js) to maintain proper separation of concerns and consistent structure. track-content.js now also provides segregation to previous tracks as well.
3)Fixed routing dependency by ensuring content is loaded based on resource ID, following the suggested architecture where the Resource Page handles page routing by ID.
Please review the updated commit and if you are satisfied with this change , then please merge the PR . I can then work on my current issue assigned .... Thanks :)

Instead of creating seperate js pages add your content to the resources.js page, take a look at it you'll understand , if it is not possible to shift current page to there let me know.

@rishicds I have successfully used  the current  resources.js to add all of my roadmap content . I was able to do it successfully . I have attached the proof of execution below. Kindly , merge the PR ...Thanks :)
https://github.com/user-attachments/assets/14a18fc1-fcbb-4dae-9b25-a9f653a57da1

Done


Can LightGBM support more than max int32_t data points?

Summary
Hi I was wondering if all it would take for LightGBM to support more than the ~2B data points int32_t allows for, would be doing the work required to change data_size_t to int64 as proposed in https://github.com/microsoft/LightGBM/issues/2818 ?
Seems like that work has been dropped for now, but I guess it's open to be picked up?
Motivation
For large-scale learning tasks we might want to use more than 2B data points.
Description
The work would include refactoring the codebase to support data IDs that are int64-based.
References
Seems like @guolinke  branch at https://github.com/microsoft/LightGBM/tree/sparse_bin_int64 is no longer available as a starting point?

According to the discussion in #2818, changing data_size_t from int32_t to int64_t seems enough (without considering efficiency). But I'm not sure whether there are some conversions between int32_t and data_size_t in repo now. If so, something unexpected may happen.
Contribution is welcome! I can help to check with the existence of the conversion. I'm personally also very interested in this.

Well getting the code to compile and tests to pass after changing all refs to data_size_t to use int64_t explicitly wasn't much work.
I'm kinda worried because lots of times I saw data_size_t used interchangeably with int or int32_t in terms of returns types etc, and I don't have enough depth to know which references to data_size_t should be converted to int64_t and which should be converted to int32_t to avoid large memory costs. To work through quickly I replaced all references to data_size_t with int64_t, which means that wherever that was used with the assumption that it refers to int32_t it will likely incur a performance/memory penalty.
With some help from the devs though I could give this a shot, starting with a WIP PR and working on some performance regression tests.

Feel free to open a PR when you have time. Thanks!


NEXT-9583 - Add import export profile for snippets, fixes shopwareBoo…

…stday/platform#418

1. Why is this change necessary?
This change is necessary to export snippets
2. What does this change do, exactly?
This change adds an import-export profile for snippets
3. Describe each step to reproduce the issue or behaviour.
4. Please link to the relevant issues (if any).
#418
5. Checklist

[ ] I have written tests and verified that they fail without my change
[X] I have squashed any insignificant commits
[X] I have created a changelog file with all necessary information about my changes
[ ] I have written or adjusted the documentation according to my changes
[ ] This change has comments for package types, values, functions, and non-obvious lines of code
[X] I have read the contribution requirements and fulfil them.

Hey,
We discussed it internally, and it doesn't make really sense to just export the overwritten snippets. This topic needs much more love in conception before working on it. We will try to filter next time our issues. Sorry about that.


Close the older container to prevent issues when changing the GTM id

When two GTM ids are used, the events are tracked to both of then when the "exit" function is called. This closes the container when the "exit" function is invoked and only the new container tracks the events.


[BugFix] Remove reset on last step of a rollout

Discussion in #1929

Ideally we'd like a non regression test.

I'll work on the test.

Also now that there is a reset missing, if the user takes the tensordict passed as output and feeds it back to rollout, rollout will need to call reset on it. I don't think this was part of rollout contract before: either a tensordict is passed or reset is called but not both (as this PR will dictate)

So this is a design choice, let me explain my view.
Before the PR
Before the PR, a user could not take the last tensordict from a rollout and safely pass it back in.
This is because if the env was reset on the last step, the data from that reset would not be part of the rollout results.
Thus, if the user passed the output of a rollout back into a new rollout, It would be providing old data from a previous trajectory to a freshly reset env which is in a different state.
After this PR
In the current state of this PR, the user still cannot just take the last tensordict from a rollout and safely pass it back in.
Doing so would have the same effect as prior to this PR, with the difference that now at least the env is not in a new reset state.
In my opinion this should remain like this (and like it was).
If the users chooses auto_reset=False it will be the duty of the user to pass non-done data as the input to a rollout (just as before).
The difference that this PR allows is that now users that want to run rollouts in a row are able to by adding the reset logic outside the rollout function (as detailed in the snippet in https://github.com/pytorch/rl/issues/1929#issuecomment-1954286298). Before they could not do this.

If we patch things let's patch them correctly no?
Why not having something like
        if auto_reset:
            if tensordict is not None:
                raise RuntimeError(
                    "tensordict cannot be provided when auto_reset is True"
                )
            tensordict = self.reset()
        elif tensordict is None:
            raise RuntimeError("tensordict must be provided when auto_reset is False")
		else:
			aggregate_reset = _aggregate_end_of_traj(tensordict)
            if aggregate_reset.any():
				self.reset(tensordict)

I don't understand the reluctance against this, does this break anything?

I don't understand the reluctance against this, does this break anything?

Oh no this works too! I'm not against it.
It will just be a new feature.
Aka rollout will check that the passed tensordict is done and it will reset the env if so.
We can do it, a few considerations:

this feature should be activated if also  auto_reset  is set. This would change the meaning of auto_reset as it could be set at the same time as an input tensordict, generating the following cases:
- no tensordict passed -> auto_reset will either call reset or not (like before)
- tensordict passed and auto_reset=True -> rollout will check if the passed td is done and if so reset (this is the new param combination that was not allowed before and it will trigger the new feature)
- tensordict passed and auto_reset=False -> rollout will not check the input td for reset (like before)
keeping the last case is good for bc compatibility and there might be cases where you do not want the rollout to call a reset

It will just be a new feature.

I think we're at the edge between new feature and bug fix. If you can't pass the tensordict you got from the last step instead of calling reset safely, this would be a bugfix.

If you can't pass the tensordict you got from the last step instead of calling reset safely, this would be a bugfix.

Yeah if this was supposed to be a feature before, I guess this is a bug fix

keeping the last case is good for bc compatibility and there might be cases where you do not want the rollout to call a reset

If the env does not allow a step after done and you pass a tensordict that is done, it will result in an error. IMO we should capture that and make sure it does not happen for ease of use. I can't think of anyone using rollout reasonably right now who would be annoyed by this change and consider it bc-breaking, but I could be overlooking things.
To me auto_reset just means reset when starting. I don't think we should change the meaning. The consideration I was putting forward is orthogonal to auto_reset IMO.

If the env does not allow a step after done and you pass a tensordict that is done, it will result in an error. IMO we should capture that and make sure it does not happen for ease of use. I can't think of anyone using rollout reasonably right now who would be annoyed by this change and consider it bc-breaking, but I could be overlooking things.
To me auto_reset just means reset when starting. I don't think we should change the meaning. The consideration I was putting forward is orthogonal to auto_reset IMO.

This consideration makes an assumption about the env.
I can also make an example about users that might want to take steps in done environments. (I personally have cases for this)
Forcing a reset on these users would narrow the flexibility of rollout with respect to prior versions.
I think if we introduce this now auto resetting when a td is given, it is important to make it deactivatable.
If we want to have another param to deactivate it other than auto_reset that is fine too, I just though that auto_reset fits nicely as it was not allowed before and it is true by default.

Not sure I'm following here
Previously we had a broken version of rollout where the reset data of the last step was potentially lost. Anyone wanting to do anything with this was doomed to fail. I don't think that in this scenario working with env that do not reset when done was an option. So this scenario can be excluded entirely and considered as a separate issue.
If we want to land this, to me having it work with tensordict that are in a done state is a pre-requirement. The change I'm proposing isn't bc-breaking in any way since it build upon a bug-fix: anyone who was doing anything with rollout before can only have fewer bugs now.
The runtime won't be affected since the check I'm suggesting is already performed by step_and_maybe_reset, which we now replace by step.

There is something that we still did not discuss.
If you really want to chain calls of rollout, then the function should call sefl.maybe_reset on the input dict (like in the current version of the PR).
Which includes step_mdp.
Or are we assuming that users call step_mdp beteween rollout calls?

It's safe to assume that users call step_mdp because the contract is that the tensordict you pass is similar to the one you'd get out of a call to reset

Ok if you wanna have a look now it should do what we want

Further question:
if break_when_any_done is true, we still do the reset and ignore it?

LGTM, just missing a docstring and a test (maybe also writing a note in rollout doctring about all of this? Like "how to use rollout as a data collector" or similar?)

We should be gucci!
Also, I still haven't come around the problem in #1929, so currently rollout is still not working for me as a data collector.
Have to see if it is a bug of my brain or a a very subtle bug of the library.
Nevertheless, this can be merged.


Error: Parse error: syntax error, unexpected T_STRING

I am getting the following error:
Parse error: syntax error, unexpected T_STRING in /path/to/unirest-php/src/Unirest/Exception.php on line 3

I swapped out my real path for /path/to/.
PHP Version	5.4.45

Thanks.

Any ideas?

will look into it today.

@n01ukn0w I can't seem to be able to replicate ... can you give me the code / steps to re-create this issue?

<?php

/**
 * @author
 * @copyright 2016
 */
?>

  <title>PHP Test</title>

 <?php echo '
Hello World
';?>

 <?php
echo '
"WTF"
';
if (is_file('/home/sinjin/unirest-php/src/Unirest.php'))
	{
    		require_once('/home/sinjin/unirest-php/src/Unirest.php');
    		echo '
Found /home/sinjin/unirest-php/src/Unirest.php
';
	}
	Else {

	echo '
Not working
';
	}

?>

Status update?

php 5.4 is no longer supported http://php.net/eol.php
The following worked fine for me in PHP 5.5 through to PHP


Test on Ruby 3.1

stealing @kjetijor's commit 0781a9dd8801227eca1094fbdb2e92e31b74f070 from https://github.com/ManageIQ/kubeclient/pull/537 (again, this time on master branch)
I'm still not sure where net/smtp actually gets require-d, but pretty certain it's not in kubeclient itself so a dev dependency sounds right.
Fixes #543 :tada:

2.5 run was just stuck, which I'm sometimes seeing recently.  minitest starts, and then... nothing for 20min? (because I cancelled manually. saw 5hours in another run!)
Run options: --seed 40851

# Running:

Error: The operation was canceled.

re-running...

2.5 stuck again.  But OK, 3.1 and ruby-head (3.2-to-be) passed, merging.
I'll add test timeouts in another PR.


Android issue with focus lost after showing interstitial ad

@fbricker
it seems like the focus is not set back to main activity after showing interstitial ad on Android.
I used stopImmediatePropagation for Android back button, after showing interstitial ad, pressing back button quits the app rather than doing something else.
any clues on how to fix this?

solved..seems like problem with framework i used


The Chinese version for this doc is difficult to read.

Hi Team,
When trying  to translate this doc to Chinese by changing the page version directly, which returned to be a not very meaningful Chinese version of this doc.
To be more specific:

The wording '不DNS验证' doesn't make any sense.. please try to make the translated version of the doc more meaningful and readable. thanks a lot.

Please help with the link of document on which above document feedback request is created.
In case your query in not related to document content, open a new forum thread in MSDN forum or Stack Overflow as it could be better place to get help on your scenario. These forum community members could provide their expert guidance on your scenario based on their experience. Thanks.

Since this issue is not related directly to the doc, I would recommend you to create a thread on the forums - MSDN or stack overflow.
Once you post your issue on forums, it will have visibility across the community which is a better suited audience for such types of issues.
Please create a Forum thread and post it's link here and tag me. I will now proceed to close this issue. Feel free to comment if you have any other queries.


Ensure wheel installed on CI

This means for non-wheel packages, pip can compile a wheel that will get cached and reused between runs.

Codecov Report

:exclamation: No coverage uploaded for pull request base (main@6f32858). Click here to learn what that means.
The diff coverage is n/a.

@@           Coverage Diff            @@
##             main       #45   +/-   ##
========================================
  Coverage        ?   100.00%
========================================
  Files           ?         4
  Lines           ?       167
  Branches        ?        26
========================================
  Hits            ?       167
  Misses          ?         0
  Partials        ?         0

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 6f32858...d1853b9. Read the comment docs.


Really high obs

There are a handful of super high observations (most are during summer but one is at 10 m in March??). I am suspicious of those and feel like we may need a disclaimer? A previous release included the data source as a column, but I'm not sure which version of the obs from lake-temperature-model-prep includes that column ...
library(tidyverse)
obs <- read_csv('tmp_data/lake_temperature_observations.csv')
obs %>% filter(temp > 38) %>% arrange(site_id, date, depth)

# A tibble: 11 × 4
   site_id         date       depth  temp
   <chr>           <date>     <dbl> <dbl>
 1 nhdhr_121627799 2008-07-07  9.14  39.3
 2 nhdhr_123209101 1998-06-23  0.27  38.4
 3 nhdhr_141286808 2007-06-15  0.3   40
 4 nhdhr_141286808 2007-06-15  0.91  39.9
 5 nhdhr_143248388 1997-03-06 10.7   40
 6 nhdhr_45337693  2013-07-31  0     39.0
 7 nhdhr_45338847  2013-07-31  0     38.8
 8 nhdhr_45365189  2020-07-20  0     40
 9 nhdhr_49239115  2010-08-10  0     38.5
10 nhdhr_75653819  1991-08-29  8.23  39.8
11 nhdhr_75655521  2014-06-11  9.14  39

7b_temp_merge/out/merged_temp_data_daily.feather has a column for sources.
obs = pd.read_feather('2_process/in/model_prep/temperature_observations/merged_temp_data_daily.feather')
obs[obs.temp > 38]
--
Out[13]:
                 site_id        date  depth   temp                                             source
674725    nhdhr_45337693  2013-07-31   0.00  38.97  7a_temp_coop_munge/tmp/1945_2020_All_MNDNR_MPCA_Temp_DO_Profiles.rds
675911    nhdhr_45338847  2013-07-31   0.00  38.82  7a_temp_coop_munge/tmp/1945_2020_All_MNDNR_MPCA_Temp_DO_Profiles.rds
679740    nhdhr_45365189  2020-07-20   0.00  40.00  7a_temp_coop_munge/tmp/1945_2020_All_MNDNR_MPCA_Temp_DO_Profiles.rds
715559    nhdhr_49239115  2010-08-10   0.00  38.48                               wqp_IL_EPA_WQX-RAL-1
1109360   nhdhr_75653819  1991-08-29   8.23  39.80                           wqp_USGS-445328088335000
1510095  nhdhr_143248388  1997-03-06  10.67  40.00                               wqp_WIDNR_WQX-673123
1563975   nhdhr_75655521  2014-06-11   9.14  39.00                               wqp_WIDNR_WQX-593045
1946866  nhdhr_141286808  2007-06-15   0.91  39.94                             wqp_WIDNR_WQX-10021447
1946943  nhdhr_141286808  2007-06-15   0.30  40.00                             wqp_WIDNR_WQX-10021447
1974014  nhdhr_121627799  2008-07-07   9.14  39.30                                  wqp_NALMS-F418511
1980693  nhdhr_123209101  1998-06-23   0.27  38.38                            wqp_SDWRAP-SWLAZZZ3703A

OK, seeing these sources makes me less skeptical since there is USGS and WIDNR one. I think I will adjust the code so that our observation dataset includes this source column. Thanks, @AndyMcAliley!


Provide me with a machine parsable version in the vendir.lock.yml when githubRelease is used

Describe the problem/challenge you have
I really love the tagSelection capability in vendir for githubRelease, it's the best and support for conditionals is really stellar.  I've started using it instead of tag after playing around with it for imgpkgBundle.  For imgpkgBundle I'll get what the semver resolved to in the lock file:
apiVersion: vendir.k14s.io/v1alpha1
directories:
- contents:
  - image:
      tag: v1.4.0
      url: projects.registry.vmware.com/tkg/tanzu_core/provider/provider-templates@sha256:6f48690e6e7ff5c81443134581cb4c80320767b345b347ef83038c74cc0ec549
    path: .
  path: bundle/config/upstream
- contents:
  - directory: {}
    path: .
  path: bundle/config/overlay
kind: LockConfig

But then with githubRelease I don't get anything that is easily machine parsable for follow-on scripts:
apiVersion: vendir.k14s.io/v1alpha1
directories:
- contents:
  - githubRelease:
      url: https://api.github.com/repos/kubernetes-sigs/cluster-api-provider-aws/releases/47009918
    path: .
  path: bundle/config/upstream
- contents:
  - directory: {}
    path: .
  path: bundle/config/overlay
kind: LockConfig

Describe the solution you'd like
I'm telling vendir what to check against, like this example:
apiVersion: vendir.k14s.io/v1alpha1
kind: Config
minimumRequiredVersion: 0.12.0
directories:
  - path: bundle/config/upstream
    contents:
      - path: .
        githubRelease:
          slug: kubernetes-sigs/cluster-api-provider-aws
          tagSelection:
            semver:
              constraints: ">0.6.5 <0.7.0"
          assetNames: ["infrastructure-components.yaml"]
          disableAutoChecksumValidation: true
  - path: bundle/config/overlay
    contents:
    - path: .
      directory:
        path: manifests

I want to know what release I ended up with in such a way I can just run through:
yq eval '.. | select(has("tag")).tag' vendir.lock.yml vendir.yml

to get the tag that was resolved.
I am doing this already with imgpkgBundle and githubRelease when I explicitly set the tag, but this capability stops working if I move to tagSelection.  I can do something similar with helmChart (just change to version instead of tag)
Anything else you would like to add:
I am happy to change up my process if there's a more robust solution here, maybe some kind of aggregate "version collector" or similar for vendir that can help establish a "lock versioning" based on what was fetched.

Vote on this request
This is an invitation to the community to vote on issues, to help us prioritize our backlog. Use the "smiley face" up to the right of this comment to vote.
👍 "I would like to see this addressed as soon as possible"
👎 "There are other more important things to focus on right now"
We are also happy to receive and review Pull Requests if you want to help working on this issue.

@voor that seems like an oversight when githubRelease was taught to be tagSelection friendly. seems like a reasonable thing to add.


Automization of deployment with CI workflows

Using github actions can help with the automatic deployment of new releases.
PRs can be labeled as minor/major changes and an automatic deploy action can be used to automate pushing a new release to pypi.
I should look into this at some point in the future.

This is now applied in a recent PR (#46)
Now, every new GitHub release will trigger an automated pipeline to push the release over to PYPI.


Fix exit code of --help and --version, and test them in CI

Corrects prior regression from #62550, which caused ERROR: output and exit code 1.
Also invokes both --help and --version in GitHub Actions to ensure exit code 0.

Thanks!

Cherry-picked for 3.5.


small fix

Coverage remained the same at 78.753% when pulling 16476130eaf561b396d89caabee197b005853095 on fix/types into be3f69f252bbacc6b410c7bcb22546f100b44009 on master.


chore(*) use sock:sslhandshake instead of sock:tlshandshake

Summary
Openresty will support mtls with sslhandshake soon,
we need not to use own special tlshandshake,
and in this case, it only uses normal ssl feature,
we can change to sslhandshake safely.
see:
https://github.com/openresty/lua-nginx-module/pull/1602
https://github.com/openresty/lua-resty-core/pull/278
Full changelog

use sock:sslhandshake instead of sock:tlshandshake

@chronolaw Could you add the PR/commit in openresty side for this feature for tracking?

@chronolaw您能否在 openresty 端添加 PR/commit 以跟踪此功能？

done, thanks.


VSCode configurations C++/Python linting, building and debugging

VSCode C++ Extension v0.30.0-insiders4 required for ARM64 support using Remote Extensions
Setup

Install VSCode
Setup Remote SSH to Jetson Nano (https://code.visualstudio.com/docs/remote/ssh)
Install Python Extension (ms-python.python) on remote
Install C++ Extension (ms-vscode.cpptools) on remote

To build C++ DSL lib

Ctrl+Shift+Build

To debug Python:

Open Python file
Place breakpoint
Choose Python: Current File config (Ctrl+Shift+D - choose from dropdown)
Press F5

To debug C++ using Python file:

Place breakpoints in C++ files
Open Python file
Choose CPP: Current Python File config (Ctrl+Shift+D - choose from dropdown)
Press F5

Thanks so much for this @jlerasmus, :1st_place_medal:
I'm going to copy your notes above to a new docs/page and link in the readme.
I look forward to trying this out.


2921 simple logs color

What does this PR do?

Add color for logger of container

Priority
Merge before : https://github.com/docker/kitematic/pull/2960
Screenshot?

Should this PR be merged before the log search, or does the log search also include this PR?

@FrenchBen Update PR's description


Need clarification on Control Board numbering

From Bootz perspective, is slot "0" considered as invalid control board slot number?
When I dump JSON content of Bootz request in python script as shown below, it prints slot-1 number as "1" as expected.
But it doesnt print slot-0 number. So, suspecting that "0" is considered as invalid index.
json_obj = MessageToJson(request)
print(json_obj)

Could you also provide the JSON dump and the input request?

Yeah, this seems like an issue with the JSON dump function you're using. The Bootz code doesn't do any validation on the slot number passed by the client.

I was concerned that Bootz may be treating "0" as invalid slot. Thanks for clarification that it is not the case.
After a bit of searching, found that the JSON dump function from protobuf utils doesn't print default values by default.
But could override default dump behavior by forcing to print default values.
from google.protobuf.json_format import MessageToJson
json_obj = MessageToJson(request, including_default_value_fields=True)
Will close the issue as resolved.

Closing as per details provided in previous comment.


Refactor the gem

Rename to rake-task-migration
Persist migrations to the database
Removes the configuration and manifest classes

This fixes issue #4


KeyError: 'module name can't contain "."'

How can I fix the problem?

install torch==0.3.1 and torchvision==0.2.1

Thank you
With your help I have solved the problem

Thank you
With your help I have solved the problem

how to fix this issue?

The new version of Pytorch does not support module names with a "." in their name.
You can either use an older version, or patch the state_dict file, as well as the model definition.
You can patch it using this for example:
x = torch.load('netG.pth')
def change_key(state_dict, old, new):
    for _ in range(len(state_dict)):
        k, v = state_dict.popitem(False)
        state_dict[new if old == k else k] = v

change_key(x, "main.extra-layers-0.64.conv.weight", "main.extra-conv-0.weight")
change_key(x, "main.extra-layers-0.64.batchnorm.weight", "main.extra-bn-0.weight")
change_key(x, "main.extra-layers-0.64.batchnorm.bias", "main.extra-bn-0.bias")
change_key(x, "main.extra-layers-0.64.batchnorm.running_mean", "main.extra-bn-0.running_mean")
change_key(x, "main.extra-layers-0.64.batchnorm.running_var", "main.extra-bn-0.running_var")

Also see issue #4


Fix podman save when redirecting output

podman save would write the progress bar to the image tar file
when the output was redirected with >.
Fixed the writer to not write the status bar when the output is redirected.
Fixes https://github.com/projectatomic/libpod/issues/352
Signed-off-by: umohnani8<EMAIL_ADDRESS>

@rhatdan @mheon @edsantiago PTAL

@umohnani8 Still don't see the change to handle -q in here?

@mheon just re-pushed. Should see it now.

Any objection to transforming this PR into "send all progress logs to stderr", or just ditching this PR in favor of a new one?

@edsantiago If you'd prefer it done all at once and @umohnani8 thinks she can get to it soon, sure, otherwise I'd prefer to get this in to fix the obvious bug

LGTM, I'd vote for getting this in soon, then refining as appropriate in a separate pr.

Lets fix this as one PR where progress is written to stderr.

Changed all the commands using writer to write the progress bar to Stderr.
@rhatdan @mheon PTAL

@edsantiago yup it should. Fixed, thanks.

LGTM

@rh-atomic-bot r+


Some questions ，important

Hello! This is an excellent work! I am currently trying to conduct experiments based on this project. I would like to know a few questions. The first question is whether my trained model can compress other categories of videos.
The second question is Can I package this model?
Thanks！

Hi, thank you for your interest in our work!
(1.) It is unlikely that you can use your trained model for other videos. The trained models are overfitted to the training material and are unlikely to generalize well to other videos.
(2.) Could you clarify what you mean by 'package the model'?


2 extra line breaks are added

$ touch test.js

# Empty input
$ cat test.js | wc -c
       0

# 2-bytes output
$ babili test.js | wc -c
       2

Hey @bfred-it! We really appreciate you taking the time to report an issue. The collaborators
on this project attempt to help as many people as possible, but we're a limited number of volunteers,
so it's possible this won't be addressed swiftly.
If you need any help, or just have general Babel or JavaScript questions, we have a vibrant Slack
community that typically always has someone willing to help. You can sign-up here
for an invite.

@bfred-it Interesting, its happening because of the options passed to childprocess spawn
{
stdio: "inherit"
}


JAVA-875: Move secondary index metadata out of column definitions.

Force pushed to:

rebase on current 3.0
merge commits for 875 and 882
remove commit for 880 (which is already on 3.0 after the last 2.2 merge)


custom dependencies on parentchain runtimes to decode L1 Events

closes #1506

move runtime dependencies to ita-parentchain-interface
decode events for solo or parachain with separate dependencies
decode events for Integritee, TargetA, TargetB with separate dependencies
ita_parentchian_interface is now the only crate that depends on runtime implementations

decoding into proper types was too much of a hssle for now. therefore, we just use Default fmt to print events
On Solochains, L1 events now look like this:
[L1Event:IntegriteeSolo] Balances(Withdraw { who:(5E2eQ9D1...), amount: 319638000 })
[L1Event:IntegriteeSolo] EnclaveBridge(ProcessedParentchainBlock { shard: 0x0a41100ba5fd7572206e8b40b74ea5a61780b73946545e4aeede08271f1222cf, block_hash: 0xec6aa3bcf5a2d03d7e8bca0fe037a94caa58e891b301b45763c3d5726082d344, trusted_calls_merkle_root: 0x0000000000000000000000000000000000000000000000000000000000000000, block_number: 1 })

On Parachains, L1 events now look like:
[L1Event:TargetAPara] Treasury(Deposit { value: 2069566 })
[L1Event:IntegriteePara] Treasury(Deposit { value: 1876330 })
[L1Event:IntegriteePara] TransactionPayment(TransactionFeePaid { who:(5E2eQ9D1...), actual_fee: 187633011, tip: 0 })
[L1Event:TargetAPara] TransactionPayment(TransactionFeePaid { who:(5E2eQ9D1...), actual_fee: 206956602, tip: 0 })
[L1Event:IntegriteePara] System(ExtrinsicSuccess { dispatch_info: DispatchInfo { weight: Weight { ref_time: 556799000, proof_size: 3877 }, class: DispatchClass::Normal, pays_fee: Pays::Yes } })
[L1Event:TargetAPara] System(ExtrinsicFailed { dispatch_error: Module(ModuleError { index: 50, error: [5, 0, 0, 0], message: None }), dispatch_info: DispatchInfo { weight: Weight { ref_time: 326159000, proof_size: 3873 }, class: DispatchClass::Normal, pays_fee: Pays::Yes } })
[L1Event:TargetAPara] Balances(Withdraw { who:(5E2eQ9D1...), amount: 206956602 })
....
....
[L1Event:IntegriteePara] Sidechain(FinalizedSidechainBlock { shard: 0x0a41100ba5fd7572206e8b40b74ea5a61780b73946545e4aeede08271f1222cf, block_header_hash: 0x5d341e805bb51544eabd88a5b2e3e791cb971be747c83f52e256a058e224291a, validateer:(5E2eQ9D1...) })
[L1Event:IntegriteePara] EnclaveBridge(ProcessedParentchainBlock { shard: 0x0a41100ba5fd7572206e8b40b74ea5a61780b73946545e4aeede08271f1222cf, block_hash: 0xce5b9563788160c64377e2b11acb844cfa6508a188a1d85e326c0f06b8fe6250, trusted_calls_merkle_root: 0x0000000000000000000000000000000000000000000000000000000000000000, block_number: 10 })

Hmmm, by now I believe we can probably get rid of runtime deps entirely. Would accelerate the build quite a bit


The shape of pool5_O in early and late fusion

Hello, and thank you for your job well done.
I have a problem with the code in this article.In late fusion, the pool5_O shape is (46,7,7,1024).In early fusion, pool5_O was in the shape of (16,7,7,1024).I have the following questions:

Pool5_O is the same production code in both merges, so am I missing something?Why are the sample sizes inconsistent?
Why are the sample sizes selected differently in the two different fusion methods?
I hope we can get a solution.
Thank you very much.

Could you please specify what you referred to as 'early fusion' and 'late fusion'?
And the mentioned size infonsistency might be the result of Line 405 in lib/networks/TIN_HICO.py, where only the first self.H_num object bboxes are used for ROI pooling.


Grafana jupyterhub dashboard is not useful and need to be redone

Closing with initial working jupyterhub chart can improve from here b31ba13c2856e25af87c0e27e581381ca8379a97.

Closing with initial working jupyterhub chart can improve from here b31ba13c2856e25af87c0e27e581381ca8379a97.


IndexError: index 0 is out of bounds for axis 0 with size 0

Hi team, I'm using the pert_data.new_data_process and facing some problems:
`IndexError                                Traceback (most recent call last)
/tmp/ipykernel_1143058/1783411457.py in
1 pert_data = PertData("./data/")
----> 2 pert_data.new_data_process(dataset_name = 'xialab_perturb', adata = tmp_adata)
~/miniconda3/envs/scgpt/lib/python3.7/site-packages/gears/pertdata.py in new_data_process(self, dataset_name, adata)
94         dataset_fname = os.path.join(pyg_path, 'cell_graphs.pkl')
95         print_sys("Creating pyg object for each cell in the data...")
---> 96         self.dataset_processed = self.create_dataset_file()
97         print_sys("Saving new dataset pyg object at " + dataset_fname)
98         pickle.dump(self.dataset_processed, open(dataset_fname, "wb"))
~/miniconda3/envs/scgpt/lib/python3.7/site-packages/gears/pertdata.py in create_dataset_file(self)
246         dl = {}
247         for p in tqdm(self.adata.obs['condition'].unique()):
--> 248             cell_graph_dataset = self.create_cell_graph_dataset(self.adata, p, num_samples=1)
249             dl[p] = cell_graph_dataset
250         return dl
~/miniconda3/envs/scgpt/lib/python3.7/site-packages/gears/pertdata.py in create_cell_graph_dataset(self, split_adata, pert_category, num_samples)
283         if pert_category != 'ctrl':
284             # Get the indices of applied perturbation
--> 285             pert_idx = self.get_pert_idx(pert_category, adata_)
286
287             # Store list of genes that are most differentially expressed for testing
~/miniconda3/envs/scgpt/lib/python3.7/site-packages/gears/pertdata.py in get_pert_idx(self, pert_category, adata_)
252     def get_pert_idx(self, pert_category, adata_):
253         pert_idx = [np.where(p == self.gene_names)[0][0]
--> 254                     for p in pert_category.split('+')
255                     if p != 'ctrl']
256         return pert_idx
~/miniconda3/envs/scgpt/lib/python3.7/site-packages/gears/pertdata.py in (.0)
253         pert_idx = [np.where(p == self.gene_names)[0][0]
254                     for p in pert_category.split('+')
--> 255                     if p != 'ctrl']
256         return pert_idx
257
IndexError: index 0 is out of bounds for axis 0 with size 0
`
I checked the .obs .var and .X, the type of those are the same as the adamson you provided, but it still came error

I think the condition column in your xialab_perturb may not be in the right format. Can you share the output of xialab_perturb.obs.head()

the single_ex_adata is my data and the adata is the adamson data

I'm facing the problem while I'm using a package named scGPT, they required using the 0.0.1 version of cell-gears. But that version can still run the adamson correctly.

Hi Team, I've figured the problem, some of the genes that I perturbed is not contained in the .var dataframe. After I deleted those genes the model is working.

Just FYI, if you switch to using GEARS v0.0.4, then it's not required that perturbed genes be in the .var dataframe


(#131) remove dependencies of required linux commands

move server-data.sh from tools to under docker/server
move init code from launch-server.sh to docker/server/server-init.sh
install curl and jq in Dockerfile
map_data service is added and wait map_server to upload data

@muratams @miyazakifuki
Could you test server-launch.sh in your environment?

rebuild docker image ./build-docker.sh server
and launch cabot to test

Do you have any concerns this change?


Deprecate "genomic_assembly" on variance data

Genomic assembly now belongs in the molecular_metadata workbook platform sheet. Therefore, it needs to be populated throughout and removed from the variance columns
ACCEPTANCE CRITERION

[x] removed from variance columns
[x] pushed to repo

@CsabaHalmagyi done


Reduce the number of options in top navbar

The navbar is becoming a bit cluttered and is already breaking to
multiple rows on smaller displays. Break out the tools into a
separate section and remove the tutorials item since it's already
in the set of top callout links.

Agreed. I've pulled your updated into my branch. Can you please take a look at https://github.com/greenplum-db/greenplum-db.github.io/pull/27/files/d98b5db4e1052e20c1aa4112954111ad41a80107..083bc123a297ed7e2bb924ce2bf1a5ef956e4e03 again?

Committed as part of #27


Resource flag is larger than necessary

Currently resources have a flag of size "int", meaning 32 or 64 bits. This is pretty large for a 1-bit flag.  Ideally, it would be a single bit and then allow the alignment code to ensure that the contents of the resource are properly aligned.  After the port to x86_64 is running smoothly this would be a good thing to tackle.

I've gone ahead and tagged the places in the code that would need to change with "FIXME #1884"


Update to append last update

This is a good chunk of code to the current GCM project to make it a lot more stable. It is probably not very elegant - I would like your thoughts. It does run on all the sites i tried it with. The button now interacts with the slider across files. I also took out a lot of my old code.

Some features I consider we can add in the near future are such as:
A storage registry of all sites trusted and not trusted.
and an indicator that the site is not on the list, and a request to submit a form, perhaps changing the button text?


Connecting multiple devices with pyrealsense2

Hi,
I am trying to connect two camera devices (D415) and get their images at the same time, using pyrealsense2.
There is a description how to do exactly that here: https://github.com/IntelRealSense/librealsense/tree/master/examples/multicam
But unfortunately it doesnt help me much because it is not written with pyrealsense2.
I started from the provided example file opencv_viewer_example.py trying to modify it.
I could identify my two cameras with the code below;
ctx = rs.context()
devices = ctx.query_devices()
print devices[0]
print devices[1]

But how do I move on to create separate streams for them? In the pipeline, confi`guration....? I tried everything.
I would be grateful for some sample code!
Thanks,
M
Camera Model: D415
Operating System: Linux Ubuntu

pyrealsense2 is a direct port of the C++ API.
In the example you linked, the helper class device_container is defined to help manage the devices.
On line 169 this class is used to start the devices with the device_container::enable_device() function, which is implemented on line 30.
Inside device_container::enable_device()'s implementation, you can see on lines 45-50 that a separate pipeline/config object is created for each one. This code can be ported directly to python.
line 52 holds the relevant objects alive inside the device_container object. Note that only the profile and pipeline are stored, not the config object.
device_container::poll_frames(), implemented on line 97, is how the example accesses the frames from the devices, simply polling each in turn (lines 101-114). This code can also be ported directly.
Hope this helps!

Thank you!
It helped. I could finally implement two cameras. Let me share my code below.
Again, I started from the opencv_viewer_example.py. The added functions are that there are two cameras working at the same time, and that you can save the images by pressing 's'.
import pyrealsense2 as rs
import numpy as np
import cv2
import logging

# Configure depth and color streams...
# ...from Camera 1
pipeline_1 = rs.pipeline()
config_1 = rs.config()
config_1.enable_device('013102060174')
config_1.enable_stream(rs.stream.depth, 640, 480, rs.format.z16, 30)
config_1.enable_stream(rs.stream.color, 640, 480, rs.format.bgr8, 30)
# ...from Camera 2
pipeline_2 = rs.pipeline()
config_2 = rs.config()
config_2.enable_device('046112051680')
config_2.enable_stream(rs.stream.depth, 640, 480, rs.format.z16, 30)
config_2.enable_stream(rs.stream.color, 640, 480, rs.format.bgr8, 30)

# Start streaming from both cameras
pipeline_1.start(config_1)
pipeline_2.start(config_2)

try:
    while True:

        # Camera 1
        # Wait for a coherent pair of frames: depth and color
        frames_1 = pipeline_1.wait_for_frames()
        depth_frame_1 = frames_1.get_depth_frame()
        color_frame_1 = frames_1.get_color_frame()
        if not depth_frame_1 or not color_frame_1:
            continue
        # Convert images to numpy arrays
        depth_image_1 = np.asanyarray(depth_frame_1.get_data())
        color_image_1 = np.asanyarray(color_frame_1.get_data())
        # Apply colormap on depth image (image must be converted to 8-bit per pixel first)
        depth_colormap_1 = cv2.applyColorMap(cv2.convertScaleAbs(depth_image_1, alpha=0.5), cv2.COLORMAP_JET)

        # Camera 2
        # Wait for a coherent pair of frames: depth and color
        frames_2 = pipeline_2.wait_for_frames()
        depth_frame_2 = frames_2.get_depth_frame()
        color_frame_2 = frames_2.get_color_frame()
        if not depth_frame_2 or not color_frame_2:
            continue
        # Convert images to numpy arrays
        depth_image_2 = np.asanyarray(depth_frame_2.get_data())
        color_image_2 = np.asanyarray(color_frame_2.get_data())
        # Apply colormap on depth image (image must be converted to 8-bit per pixel first)
        depth_colormap_2 = cv2.applyColorMap(cv2.convertScaleAbs(depth_image_2, alpha=0.5), cv2.COLORMAP_JET)

        # Stack all images horizontally
        images = np.hstack((color_image_1, depth_colormap_1,color_image_2, depth_colormap_2))

        # Show images from both cameras
        cv2.namedWindow('RealSense', cv2.WINDOW_NORMAL)
        cv2.imshow('RealSense', images)
        cv2.waitKey(1)

        # Save images and depth maps from both cameras by pressing 's'
        ch = cv2.waitKey(25)
        if ch==115:
            cv2.imwrite("my_image_1.jpg",color_image_1)
            cv2.imwrite("my_depth_1.jpg",depth_colormap_1)
            cv2.imwrite("my_image_2.jpg",color_image_2)
            cv2.imwrite("my_depth_2.jpg",depth_colormap_2)
            print "Save"

finally:

    # Stop streaming
    pipeline_1.stop()
    pipeline_2.stop()

[Realsense Customer Engineering Team Comment]
@marialarsson
Great to see it works for you now, you can consider to contribute this example to LibRS python wrapper by creating PR.

Does anybody know how to retrieve only the serial number? I am trying to load multiple cameras using a for loop
Thanks

Does anybody know how to retrieve only the serial number? I am trying to load multiple cameras using a for loop
Thanks

you could try something like:
realsense_ctx = rs.context()
connected_devices = []
for i in range(len(realsense_ctx.devices)):
detected_camera = realsense_ctx.devices[i].get_info(rs.camera_info.serial_number)
connected_devices.append(detected_camera)
config = rs.config()
config.enable_device(sensor_sn)
config.enable_stream(rs.stream.depth, 424, 240, rs.format.z16, 30)
config.enable_stream(rs.stream.color, 424, 240, rs.format.bgr8, 30)
profile = pipeline.start(config)

I wrote an example of connecting multiple Realsense cameras in Python:
https://github.com/ivomarvan/samples_and_experiments/tree/master/Multiple_realsense_cameras
Enjoy it if you find it useful.

Thank you so much for figuring this out- I've been looking online for hours for how to do exactly this

To get the serial device numbers for two samples this is what worked for me:
import pyrealsense2 as rs

ctx = rs.context()
devices = ctx.query_devices()
dev0 = ctx.query_devices()[0]
dev1 = ctx.query_devices()[1]

device_model0 = str(dev0.get_info(rs.camera_info.name))
device_model1 = str(dev1.get_info(rs.camera_info.name))

print(f'device_model0: {device_model0}')
print(f'device_model1: {device_model1}')

print('Config ... ')
pipe1 = rs.pipeline()
cfg1 = rs.config()
cfg1.enable_device(dev0.get_info(rs.camera_info.serial_number))
cfg1.enable_stream(rs.stream.infrared, 1, 424, 240, rs.format.y8, 30)
cfg1.enable_stream(rs.stream.infrared, 2, 424, 240, rs.format.y8, 30)

pipe2 = rs.pipeline()
cfg2 = rs.config()
cfg2.enable_device(dev1.get_info(rs.camera_info.serial_number))
cfg2.enable_stream(rs.stream.infrared, 1,  424, 240, rs.format.y8, 30)
cfg2.enable_stream(rs.stream.infrared, 2,  424, 240, rs.format.y8, 30

....


Feature request: ignore hidden cells in table_to_sheet/book

It would be great if SheetJS ignores hidden rows and cells when parsing a DOM table using the table_to_sheet and table_to_book functions.
For example, the following table:

    One
    Two
    Three

    1
    2
    3

    Раз
    Два
    Три

Would be parsed as:

One
Three

1
3

Detecting whether the cell is hidden can be done by detecting a display: none style:
if (getComputedStyle(cell).getPropertyValue('display') === 'none') {
    continue;
}

Ignoring hidden cells and rows can be optional:
XLSX.utils.table_to_sheet(table, {
    ignoreHiddenRows: true,
    ignoreHiddenCells: true
});

If you support this feature, I can make a pull request.

@SheetJSDev I think that a user expects to see in a worksheet exactly what he/she sees in the browser, therefore SheetJS should behave the same as browsers (completely ignore hidden cells).

@Finesse It's probably best to follow Excel's example.  If you take the first example, save as an HTML file, and open in Excel, you will find that it hides the row but does not hide the column:

So the specific approach that we'd be willing to accept is: if available, use getComputedStyle on the TR element to determine whether the TR is visible.  If it is not visible, mark the row as hidden.  The feature is controlled in the worksheet !row array

@SheetJSDev What about cells (s and s)? Must hidden cells be ignored? I insist to ignore them to conform the HTML behaviour.
Should the described behaviour be the default behaviour? Should there be a possibility to enable/disable it using the ignoreHiddenRows and ignoreHiddenCells parameters?

Excel treats the singleton  and  as if they were empty text (stub cells).  It probably makes sense to generate stub cells if sheetStubs is true (to be consistent with the other formats).
The original objection was based on how Excel handles the underlying HTML, but upon reflection your argument makes sense.  Let's try:

The default behavior should include every cell and only hide rows if the TR element is hidden.

Create a new display option (default false) that omits every hidden cell and every hidden row if true (as if the hidden cells were not included in the first place).

@SheetJSDev Let's recap it.
There will be a new option called display in the table_to_sheet and table_to_book methods. The default value of this option is false.
If display value is false, the result table (created from the HTML from my first my) looks this way:

If display value is true, the result table looks this way:

Is it correct?

@Finesse correct.  That makes the most sense since the default behavior lines up with how Excel handles HTML tables while the new behavior lines up with what people would expect.
https://www.caniuse.com/#feat=getcomputedstyle suggests IE9+ support getComputedStyle, so it should not be assumed to exist (IE8 is still a target in the browser test matrix).  If getComputedStyle is not available, just proceed as if nothing was hidden.

@SheetJSDev Ok! I will make a pull request in a few days.


Suspend and Remote Wakeup

I am trying to program an ESP32-S2 to be a USB HID keyboard device which can "Remote Wakeup" the host.
I am basing my code on the simple keyboard example: examples/device/hid/keyboard/keyboard.ino
I have also registered callbacks with EspTinyUSB::registerDeviceCallbacks
However the device never goes to "suspend" state. USBCallbacks.onSuspend() abstract is never called. tud_suspended() is never true.
Further more, if I put the host to sleep and wake it with another USB device (off the shelf keyboard) then my ESP32 is in connected but not mounted state. If I restart the ESP32 it goes to a normal connected+mounted state.
Is there something I am missing? Is this feature implemented?
BTW, for the dev env I am using Platform.io with the following options:
[env:esp32-s2-saola-1]
platform = https://github.com/platformio/platform-espressif32.git#feature/arduino-upstream
board = esp32-s2-saola-1
framework = arduino
platform_packages = framework-arduinoespressif32 @ https://github.com/espressif/arduino-esp32#master

Hey, please provide some context or point me in the right direction. Is this a bug or an unimplemented feature?
Can this be developed or it this a limitation with the S2?
How would I debug this?
Thanks.

Its probably not implemented, but im not sure if in this library or in tinyusb.

ok, I see that when using EspTinyUSB the tinyusb library is pre-compiled. Is there a way or process that I can use to have tinyusb in source code form so I can debug and inspect it?

not supported in current esp-idf release