PHP Fatal error: Cannot declare class, because the name is already in use

Problematic piece of code:
https://github.com/yiisoft/proxy/blob/a96fa6f2e0457ecca162ef3b095274262e3344ef/src/ProxyManager.php#L35-L40
Found during writing tests, occurs when multiple calls are used,

Fixed in #21.


ALL INTF FILE  pull failed

is experimental features enabled on your system? the JSON handling of INTF should be activated only behind this feature flag

no response. closing


fix cluster setup: use same admin pw salt on all nodes

this ensures the same salt on all cluster nodes. before, a session id generated on node1 wouldn’t be valid on node2 and node3. a round-robin haproxy would show a Fauxton login form, and again after login, because the backend node was moved over.
This overrides the hashed password on all secondary nodes with the one from the setup node when the cluster is enabled on the secondary nodes through the setup node.

This should go into 2.1.x


docs: remove exclusive language

Motivation
Similar to https://github.com/facebook/jest/pull/9040 by @carolstran, I wanted to take a stab at removing some language (easy, simply, of course, etc) to continue creating a more inclusive and friendly community.
It was difficult, however, because a lot of Docusaurus' goal is...

Easy to maintain open source documentation websites

...so there were a lot branding references with the word "easy", so I kept those and also did not touch old blog posts.
Let me know what you think!
Have you read the Contributing Guidelines on pull requests?
Yes!
Test Plan
N/a no code changes
Related PRs
(If this PR adds or changes functionality, please take some time to update the docs at https://github.com/facebook/docusaurus, and link to your PR here.)

@ericcarboni why did you changes numbered lists?

Whoops, my editor might have done that. Good catch!

@ericcarboni can you revert these changes?

Yeah, doing that now. Any idea why some lists are 1...2... and then others are 1...1...1...?

Any idea why some lists are 1...2... and then others are 1...1...1...?

I do not know this, but I would leave only one notation 1..1. Such lists are easier to maintain. But this already needs to be fixed in the new PR and it may be worthwhile to set linting MD files to avoid such a problem in the future.

One instance even had 1..1..3 🤷♂

CLA signed! ✍

One instance even had 1..1..3 🤷♂

That's probably because there was a code block or something between the second 1 and the 3. Doing 1..1..3 will result in 1, 2, 3. Using 1 instead of 3 will cause it to become 1, 2, 1 instead.


add stack info to "client set more than once" log message

This should allow us to find the call chain that triggers
such a case

:green_heart: Build Succeeded

the below badges are clickable and redirect to their specific view in the CI or DOCS

Expand to view the summary

Build stats

Build Cause: Pull request #1158 opened

Start Time: 2021-06-08T15:07:41.687+0000

Duration: 28 min 31 sec

Commit: 79bb673b9a3608201d92429604c458ebc291d3ca

Test stats :test_tube:

Test
Results

Failed
0

Passed
9497

Skipped
8766

Total
18263

Trends :test_tube:

:green_heart: Flaky test report
Tests succeeded.

Expand to view the summary

Test stats :test_tube:

Test
Results

Failed
0

Passed
9497

Skipped
8766

Total
18263


Refactor & Use of Docstrings

As explained in my issue #2  you didn't use the docstrings correctly. You can now merge my repo with yours so pmtw uses documentation. Note, there are still a lot of problems with your docstrings, a bunch of those have unfinished sentences or are cutting midway through those. Parameters are missing on almost every docstrings and some documentation is very unclear as of what it does.
Hope you'll be able to do something on that since your project seems very nice but the lack of documentation makes the module very annoying and frustrating to use

Note that I also removed unused imports

So it looks like a large chunk of this pull request is converting tabs to spaces, which makes evaluating your actual changes difficult. Usage of Tabs for indentation is not mandated by PEP8, merely recommended - and I am firmly of the belief it's a braindead recommendation. Please convert the spaces for indentation back to tabs, and I'll be happy to take a look again!

Usage of Tabs for indentation is not mandated by PEP8, merely recommended

I had to do that to get the proper indentation, otherwise I had a ton of errors saying indentation was inconsistent
If you know how I can edit the pr please tell me and I'll see what I can do

Sadly don't have much time these days so I don't think I'll be able to redo it but I highly recommend you to use proper docstrings when creating such a project, it should look like that when done correctly:
def getModeratorIndex(r,sub,mod):
    '''
    Get Index of a specific moderator in the list, adds mod to the list if he isn't
            there already

    Paramaters
    ----------
    r : Reddit
        Reddit instance (praw) that we will be using to access the wiki
    sub : str
        Name of the subreddit to fetch the mod index from
    mod : str
        Name of the moderator who's index we are fetching

    Returns
    -------
    int
        Index of the moderator in the list
    '''

not like whatever this is:
	"""
	Returns usernote colors as a list. Use get_banMacros(local="False")
	to fetch reasons from Reddit before returning.
	"""
	@localdata
	def get_banMacros(self):


Feature/mobile dapp integration (Adding balance API)

Pull Request Test Coverage Report for Build 82

120 of 138 (86.96%)  changed or added relevant lines in 6 files are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage increased (+0.02%) to 53.66%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/packages/contract/protocol/contract.ts
0
2
0.0%

src/packages/core/api/balance.ts
3
7
42.86%

src/packages/core/protocol/transaction/coinData/coinData.ts
11
15
73.33%

src/packages/core/protocol/account.ts
100
108
92.59%

Files with Coverage Reduction
New Missed Lines
%

src/index.ts
1
0.0%

Totals

Change from base Build 74:
0.02%

Covered Lines:
1227

Relevant Lines:
2028

💛  - Coveralls


[FR] Allow scanning Stock Location when entering New Stock Item

Please verify that this feature request has NOT been suggested before.

[X] I checked and didn't find a similar feature request

Problem statement
When using the New Stock Item dialog, you cannot scan a Stock Location barcode to set the location.
Suggested solution
Allow the Stock Location to be decoded from a barcode entry into the field. For example, scanning "{"stocklocation": 79}" would populate the Stock Location to "Organizer 1/A-1" in my case.
Describe alternatives you've considered
My workflow may be wrong? I am currently creating and stocking a large number of parts that I already have inventory of (not going against a PO). So my workflow is currently:

Create all Stock Locations for a given parts organizer.
Print and apply barcodes for all locations inside the part organizer.
Create a new Part.
Create a new Stock Item for the part.
I have to manually type in the stock location, scanning the barcode for the given location just defaults in "No Results Found".

Examples of other systems
No response
Do you want to develop this?

[ ] I want to develop this.

@Limb this would be a great enhancement in general, in any area where you could input a scannable item. A simple icon next to the input would be clean.
We are currently focused on rewriting the front end in react (see the pinned issue) and so any UX features such as this will be tagged against that.

Not stale

Not stale


Color didn't work after VSC ver 1.20

The sample language file is XML.
Upper one was from VSC 1.19.3 (with extension version about one week ago).
Lower one was from  VSC 1.21.0 (with extension version just install today).
Could see the the (A1) (IT.LIFE_TOTAL_H(A1-1).... were in same color now.

This is because I don't colorize in strings anymore. I can add a setting to colorize everything again which will return the previous behaviour.

Thanks for the response. It will be awesome setting.


Expose getSpec

I'm using the getSpec part of the package for https://github.com/doug-wade/react-link-to-inbox, so this pr just exposes it as part of the public api

thanks @bkonetzny!

Published as 1.2.0, thanks for the patience!


Styledocco with scss not working on windows

I have the following setup:
node --version
v0.10.28

styledocco --version
StyleDocco 0.6.6

ruby --version
ruby 2.0.0p481 (2014-05-08) [x64-mingw32]

sass --version
Sass 3.3.7 (Maptastic Maple)

In a folder I have...
test-scss.scss:
/* Provides extra visual weight and identifies the primary action in a set of buttons.

    <button class="btn primary">Primary</button> */
.btn.primary {
  background: deeppink;
  color: blue;
  border: 5px solid aqua;
}

When doing styledocco --verbose I get:
spawn ENOENT
styledocco: writing test-scss.scss -> docs\test-scss.html
styledocco: writing C:\my\path\index -> docs\index.html

The result is, that the CSS is not applied in the docs.
The same thing works when using .css.
Also everything works on Linux.
Love the project by the way! It's a great help!

+1 if someone knows how to fix

any updates on this?


Prevent generation of @charset "UTF-8";

Hi,
I want to use sass to write the stylesheets for a javafx gui application.
The problem is, that the sass-maven-plugin adds @charset "UTF-8"; to the generated css file, which java can't process and therefore fails when loading the css file.
Is there an option for the plugin to prevent the addition of @charset "UTF-8"; to the generated css file?

Just like regular sass this is not an option, in any case it's out of scope for this plugin


Create and add dapp console upsell modal

Here's the image, lmk if you need different size:

@tarunkhasnavis Updated this image to account for new "Superchain" logo.


Add support for rubocop-0.51

Rubocop 0.51 has been released. It'd be great to get support in the official engine, via a channel, or even by fixing #93 :)
Thanks a lot!

Thanks for the head's up @deivid-rodriguez! I'm going to get this into our product backlog today.

Hello @maxjacobson
Is this issue fixed by https://github.com/codeclimate/codeclimate-rubocop/pull/110?

Hi @dbusanello --
It means it's close. That work created the channel, but we still need to deploy it to codeclimate.com and the CLI.
@pointlessone, will you please update us on this issue once that's completed?

Is this issue fixed? Are 0.51 channel deployed to CC already?

Yep, apologies for not updating this issue, but this went out to codeclimate.com and the CLI 9 days ago.
Just specify the rubocop-0-51 channel (https://docs.codeclimate.com/docs/engine-channels)

Hi @maxjacobson, I'm not sure it did, when I run codeclimate analyze locally, I get the following error:
error: (CC::Analyzer::Container::ImageRequired) Unable to find an image for rubocop:rubocop-0-51. Available channels: ["stable", "cache-support", "rubocop-0-42", "rubocop-0-46", "rubocop-0-48", "rubocop-0-49", "rubocop-0-50"]

I'm using 0.68.0 of the CLI, and brew update && brew upgrade codeclimate says I'm up to date.
I don't see a new release of the CLI that includes this fix either.

Yep, good point. Looks like we didn't cut a new CLI release after adding this channel. Standby, I'll do that now.

Fresh off the presses: https://github.com/codeclimate/codeclimate/releases/tag/v0.69.0

Thanks a ton!


SW18 readFramesExecuted only returns 16 bits

Calling readFramesExecuted on an SW18 returns a 32 bit value but the upper 16 bits are always 0x5555 and do not increment when the lower 16 bits roll over. Checking the same function on the SW4 returns a 32 bit value that rolls over as expected. A minimum sketch and sample outputs are attached.
readFramesExecuted.zip

Thanks!   In my unit testing I tested getting the LSW and the MSW to verify the firmware, but don't have a test for readFramesExecuted which reads both.  The issue is here.
There's a difference in the way that public data can be requested on the Serial Wombat 4B and 18AB chip.  On the 4B, if you request a piece of public data, the next 2 public data are also sent in the same packet for efficiency.
I couldn't do this on the 18AB, because the request of some public data (such as temperature) make on-demand code run which can impact the performance of the chip.
The Arduino library readFramesExecuted function relies on the SW4B way of doing things.   I'll have to update the library to make two separate reads to public data id 67 and 68, then possibly 67 again to check for rollover and combine the results.
uint32_t SerialWombatChip::readFramesExecuted() { 	uint8_t tx[8] = { 0x81,67,68,0x55,0x55,0x55,0x55,0x55 }; 	uint8_t rx[8]; 	sendPacket(tx, rx); 	uint32_t returnval = rx[2] + (((uint32_t)rx[3]) << 8) + (((uint32_t)rx[4]) << 16) + (((uint32_t)rx[5]) << 24); 		return (returnval); }
Needs to be something like
`uint32_t SerialWombatChip::readFramesExecuted()
{
uint8_t tx[8] = { 0x81,68,0x55,0x55,0x55,0x55,0x55,0x55 };
uint8_t rx[8];
sendPacket(tx, rx);
uint32_t returnval = rx[2] + (((uint32_t)rx[3]) << 8)  ;
tx[1] = 67;
sendPacket(tx, rx);
returnval <<= 16;
returnval += rx[2] + (((uint32_t)rx[3]) << 8) ;
// Plus some sort of rollover check here
return (returnval);
}`


Improve HTML Processing Towards XML->Map Specification || Add Tests [Draft ⚠ In Progress]

Description
Tried this with a variety of sites for verifying correctness, will merge once unit tests complete and if anyone would like to give a review if nothing else (if you're not too opinionated on the implementation but rather the end result just to validate correctness).
I've done some tests with fq and qq comparison, the only difference is some trimming of space trailing newlines and things of that nature and doesn't look harmful or massively inconvenient.
❯ export site=example.com  && curl $site | go run . -i html > qq_o.json && curl $site | fq > fq_o.json && diff qq_o.json fq_o.json

Changes

[x] 🛠 Code Changes

[x] Implemented feature
[ ] Fixed bug
[ ] Refactored component
[ ] Updated documentation

[x] 🐛 Testing

[x] Added unit tests for new functionality
[x] Updated existing tests to reflect changes
[x] Ran all tests locally

[ ] 📄 Documentation

[ ] Updated README or documentation files as needed

Additional Notes

Tests here need a lot of work but should cover the most critical parts of this better. HTML parsing is improved, but I should probably bundle in some changes for CSV possibly in this batch so values are returned as list of maps sooner rather than later to not effect existing users (too much) since that besides some edge-cases where I am not sure at all (like ini rendering map strings since it can only handle scalar values) besides adding new codecs the best place for improvements besides removing dependencies on external libraries where its not totally needed or "bloated" for lack of better words.

Feel free to use or copy the xml/html tests from fq https://github.com/wader/fq/tree/master/format/xml/testdata

Thank you for sharing! Definitely like using assertions with stdout (if i grokked that right on a eyeballing it quickly) it seems comprehensive but simple, I think I'll use a similar method and just copy your tests to ensure xml/html is covered thoroughly seeing as it bit more problematic. Will include some additions (csv list of maps, newline ".txt" file parsing as an option) bundled in this change along with that relatively soon.

Yeap more or less but there is an interface https://github.com/wader/fq/blob/master/pkg/interp/interp.go#L145-L158 which the test framework (and normal CLI) implements which allow to run tests faster, no new process per test etc, and also do some other tricks


Output SQLite DB using New Metadata

Using the new metadata system introduced in #806 generate an SQLite database and fill it up with data based on the current database schema and ETL process, so that we can remove the datapackage infrastructure before moving on to integrating the new entity resolution & harvesting process also introduced in #806.
This is a duplicate of #1176 see that Epic for details.

This has enough different parts going on that it really should be an Epic and... we already created an Epic for this: #1176 so I am moving these notes over there and organizing other related issues within it.


Fixed pydantic-core install failure

See https://github.com/zhmcclient/python-zhmcclient/issues/1711
No review needed.

coverage: 36.569%. remained the same
when pulling d7776e30bf072c7d02b04eed7bea483acbf91fa1 on andy/fix-pydantic-core-install
into 274eaa22f0027a102147ba81bc24bd6a724152c2 on master.


Encode ExternalId to base64 to fix encoding problems when authenticating

This fixes https://github.com/rancher/rancher/issues/10690

@genuss This would also require a corresponding cattle PR for decoding

@cjellick Can you close this PR, I dont have the right to close. There is another one  open for the same bug for review https://github.com/rancher/rancher-auth-service/pull/59


Centered Header Text

Issue Description

Center text on the Header component instead of the current left-aligned text.
For example:

Issue Type

[ ] New Feature
[X] Enhancement
[ ] Bug
[ ] Other

Expected Behavior

Text is centered when relevant prop is provided.
Current Behavior

Text can only be left-aligned.

Steps to Reproduce

N/A
Environment

Component Version: 1.1.1
Browser Name and Version: Chrome 59.0.3071.115
Operating System and version (desktop or mobile): macOS 10.12.6 (16G29)

Issue initially logged in terra-core https://github.com/cerner/terra-core/issues/683 by @noahbenham

@emilyrohrbough  So the request is to have an attribute that allows header text to be left aligned (default) or centered?

@PaulSolDev yes this is the request

@emilyrohrbough it appears that there is already an existing requirement around this functionality.  1106959: https://jazz.cerner.com:9443/rm/resources/_39DLEIT3EeaYQbtTQDvZHA

Discussed with @emilyrohrbough. I'd like to pick this up & expect to begin working on it within the next couple of weeks. If any consumers need it sooner, please let me know and I'll re-prioritize.

Hi @noahbenham, is there a UX designer you've been working with on what you're trying to accomplish?  In the new Terra world, we've locked down headers to only have left aligned text due to consistency and technical concerns.  If we want to extend that capability, I'd like to see that come from UX directly.  I'm going to close this for now, but we can reopen it if need-be.

Reopening issue and having discussion needs from other UX designers. We will be gathering feedback and make a recommendation on how to proceed with this enhancement.

Thanks @neilpfeiffer. Curious if you have an estimated ETA for this decision? For our use case, here are a few UX design samples with our own custom Header component:
Contacts Visual Design | Messaging Visual Design | Alerting Visual Design
If this enhancement doesn't proceed, our headers would (obviously) become left centered per your recommendation.

Going to close for now and regroup with UX designers as reason for Engineering complexity due to variable content areas on the left and right sides. FWIW - left alignment also currently aligns with MPage patterns for component headers.
We may solve with additional/variant components in the future and will gather input from UX as to what those components might be to solve this request.


Scratch pad and 1.0.0

I just updated my tabrisjs app on my android device and the scratch pad displays some error message about not being able to load the tabris module and that it couldn't parse tabris.js because it has an invalid token. I reset the scratch pad back to the default and it has the same error.

same error for me

There appears to be a problem with the tabrisjs.com server. We're working on it.

The problem has been fixed and the scratchpad should be working fine now with 1.0.0.

Looks good. Thank you
Sent from Yahoo Mail on Android
From:"Ralf Sternberg"<EMAIL_ADDRESS>Date:Thu, Apr 30, 2015 at 7:30 AM
Subject:Re: [tabris-js] Scratch pad and 1.0.0 (#427)
The problem has been fixed and the scratchpad should be working fine now with 1.0.0.
—
Reply to this email directly or view it on GitHub.￼


Adding symbols option to the reporter

Ability to define custom symbols for the reporter. This is useful on build servers or machines that are unable to read the special symbols. Allows the use of ascii characters for easier interpretation of the build scripts using displayable characters.

Thanks for your help. Released as v2.1.0


DMD import failure

hello@openprescribing-with-volumn:/webapps/openprescribing/openprescribing$ python manage.py import_dmd --source_directory=/tmp/dmd/ --settings=openprescribing.settings.production
Traceback (most recent call last):
  File "manage.py", line 12, in <module>
    execute_from_command_line(sys.argv)
  File "/webapps/openprescribing/.venv/local/lib/python2.7/site-packages/django/core/management/__init__.py", line 353, in execute_from_command_line
    utility.execute()
  File "/webapps/openprescribing/.venv/local/lib/python2.7/site-packages/django/core/management/__init__.py", line 345, in execute
    self.fetch_command(subcommand).run_from_argv(self.argv)
  File "/webapps/openprescribing/.venv/local/lib/python2.7/site-packages/django/core/management/base.py", line 348, in run_from_argv
    self.execute(*args, **cmd_options)
  File "/webapps/openprescribing/.venv/local/lib/python2.7/site-packages/django/core/management/base.py", line 399, in execute
    output = self.handle(*args, **options)
  File "/webapps/openprescribing/openprescribing/dmd/management/commands/import_dmd.py", line 347, in handle
    add_bnf_codes(options['source_directory'])
  File "/webapps/openprescribing/openprescribing/dmd/management/commands/import_dmd.py", line 224, in add_bnf_codes
    source_directory, "Converted_DRUG_SNOMED_BNF.xlsx"))
  File "/webapps/openprescribing/.venv/local/lib/python2.7/site-packages/openpyxl/reader/excel.py", line 183, in load_workbook
    wb.active = read_workbook_settings(archive.read(ARC_WORKBOOK)) or 0
  File "/usr/lib/python2.7/zipfile.py", line 935, in read
    return self.open(name, "r", pwd).read()
  File "/usr/lib/python2.7/zipfile.py", line 961, in open
    zinfo = self.getinfo(name)
  File "/usr/lib/python2.7/zipfile.py", line 909, in getinfo
    'There is no item named %r in the archive' % name)
KeyError: "There is no item named 'xl/workbook.xml' in the archive"

Unable to reproduce this locally. Assume I had done something like rename an unzipped zip archive to the expected name.


Improve performance of "mean" groupby aggregation

Is your feature request related to a problem? Please describe.
A clear and concise description of what the problem is. What kind of performance improvements would you like to see with this new API?
Currently "mean" aggregation uses _apply_agg_function which later calls broadcast_apply_full_axis. Its performance on NYC Taxi benchmark is worse than of a single threaded Pandas "mean" implementation. Instead a man-reduce approach could be used to better parallelize the calculations.

Duplicate of #1901


Publish 0.9.2 to crates.io

@cburgdorf Just did some dep updates but 0.9.2 hasn't been published yet.

I'll do that in a moment

Ok, it's live. I also added you as an owner of that crate. You've been an owner for clog already but not for the clog-cli so I fixed that.


Robot odom frame and Lidar output frame not in sync

There seems to be a difference in the angles of axes between the robot world frame (given by the odom position of the robot) and the Lidar output from (Lidar data output).
This needs to verified by checking the initial and final positions of the robot (given by odom) after some movement and matching this with the initial and final lidar fram outputs.

We have managed to get the correct location of the robot in the Lidar frame and superimposed 2 Lidar frames on top of each other (robot being in different positions). There is still some gyro drift in the different frames. This is due to fast changes in the output of #imu_filter . Also check #1

#imu_filter

The robot orientation does not get stable when there is any rotational motion in the robot.


Bug:How do I use it in Object project?I can't find the <Afterpay/Afterpay-Swift.h> in bridge-header

🌎 Environment

OS Version: e.g. iOS 13.4.1
Version/Commit: e.g. 1.0.0

💬 Description
How do I use it in Object project?I can't find the <Afterpay/Afterpay-Swift.h> in bridge-header.
Should I do something in settings?or other ?

Hi @GentleCrazyOne Thanks for this report.
I'm afraid I am unable to reproduce any errors with the steps you have described. Are there any additional things about your project we can check?
Please see this example project with Objective-C & Swift: Cocoapods.zip


Experimental Atlas support

Automatically detects Atlases where the NIfTI header intent is NIFTI_INTENT_LABEL  (1002). For example, the atlaes supplied with MRIcroGL
Option isAtlasOutline determines if atlas is drawn as an outline (issue 213).
Similar to MRIcroGL, label colors repeat after 100 (e.g. same colors repeat for indices 1..100, 101..200, 201..300). The color should really be set to Random. @hanayik if someone drags-and-drops a NIfTI image onto NiiVue, it starts with the default color scheme gray: can you change this to default to random if the intent code is 1002?
We still need to know how to include the label names, e.g. 1Left Inferior Parietal Lobe. MRIcroGL includes this text in the NIfTI header, between the 352 byte binary header and the image data. Maybe @cdrake's work on the AFNI extensions can be extended to support this (MRIcroGL requires three features to detect this: the regular` character is 98 (not the typical 114), the intent is labels, and the vox_offset is greater than 352.
At the moment only UINT8 images are supported, easy to extend to other integer data types (e.g. INIA19 Primate Brain Atlas).

This PR has failed the automated test suite. Specifically the RGB image test.
The diff snapshot from the test is attached below (original on the left, diff in the middle, and newly generated on the right)
there is now a L/R flip. Could this be due to the changes in the orientation shader in this PR?

Automatically detects Atlases where the NIfTI header intent is NIFTI_INTENT_LABEL  (1002). For example, the atlaes supplied with MRIcroGL
Option isAtlasOutline determines if atlas is drawn as an outline (issue 213).
Similar to MRIcroGL, label colors repeat after 100 (e.g. same colors repeat for indices 1..100, 101..200, 201..300). The color should really be set to Random. @hanayik if someone drags-and-drops a NIfTI image onto NiiVue, it starts with the default color scheme gray: can you change this to default to random if the intent code is 1002?
We still need to know how to include the label names, e.g. 1Left Inferior Parietal Lobe. MRIcroGL includes this text in the NIfTI header, between the 352 byte binary header and the image data. Maybe @cdrake's work on the AFNI extensions can be extended to support this (MRIcroGL requires three features to detect this: the regular` character is 98 (not the typical 114), the intent is labels, and the vox_offset is greater than 352.
At the moment only UINT8 images are supported, easy to extend to other integer data types (e.g. INIA19 Primate Brain Atlas).

@neurolabusc, yes I can set the colormap to random based on the intent code. I will make the change today in a separate PR

@hanayik this is patched for now, I simply reverted the application of the transformation matrix for RGBA images. This is not a long term solution, we need more RGBA rotation examples. FSLeyes and MRIcroGL both apply spatial transforms to RGB images, but they appear to have a LR flip relative to NiiVue.
The relevant line is the shader, where one expects the vertex location to be transformed with the spatial transformation matrix mtx:
vec4 vx = vec4(TexCoord.xy, coordZ, 1.0);// * mtx;

@hanayik the shader should apply the transform
vec4 vx = vec4(TexCoord.xy, coordZ, 1.0) * mtx;

and the results are consistent with MRIcroGL and FSLeyes. The text 'MTT' in the image should look normal when viewed in radiological convention and mirror-reversed when viewed with neurological convention. I had just forgot the default for the npm run dev is neurological.
So I would suggest:

Either have the test ensure the image is created with radiological convention or replace the test.
Can I suggest you run pigz -n -11 ct_perfusion.nii to compress this test to nii.gz - this reduces the file size dramatically.

@neurolabusc, good catch. I will update the test in this case and compress the test image.


Note about deviceOwnership as 'Company' incorrect

OwnerTypes page now shows 'Corporate', not 'Company', so this purple note block is no longer relevant.
OwnerTypes for more details.

@JeremyTBradshaw : Thanks for your contribution! The author(s) have been notified to review your proposed change.

Cancelling this.  It is the other page which needs a PR.  It is indeed still 'Company' that AzureAD shows.


Add Coffee Meets Bagel to the list

Please review our Contribution Guidelines to ensure your changes abides by it.
Review your changes and complete the checklist below.

[x] Company name
[x] Company description
[x] Company location
[x] GitHub link
[x] How Elixir is used
[x] Sorted alphabetically

Thanks @slb350! 😀


Is there a fold panel or group component?

Is there a fold panel or group component?It is really convenient to show  multiple-choice or other things like that.

I am not sure I understand.
What is the aim? To include components in a quick way with gulp?
For example something like this?
gulp build-custom core, panels, dialogs

Sorry for my poor english,like this  http://demos.jquerymobile.com/1.4.5/listview-collapsible-item-flat/  Collapsible list items

I see, it is a nice component, I will add it asap!

Since the v1.0.8, we added and named your desired feature "accordion"
Check it out here:
http://phonon.quarkdev.com/docs/accordion
:)


[RN][Android] use invalidate as NativeModule not JSIModule

Summary:
NativeModule.onCatalystInstanceDestroy() is deprecated in favor of NativeModule.invalidate()
BaseJavaModule.invalidate() invokes BaseJavaModule.invalidate()
Override invalidate in UIManagerModule instead of onCatalystInstanceDestroy for more clear inheritance
Ref: https://github.com/facebook/react-native/pull/39444
Changelog:
[ANDROID][Refactor] - use invalidate as NativeModule not JSIModule
Test Plan:
TODO

In order to more cleanly replace onCatalystInstanceDestroy() with invalidate() we are aiming to get rid of it's declaration itself in the interface of JSIModule as part of removal of JSIModule

In order to more cleanly replace onCatalystInstanceDestroy() with invalidate() we are aiming to get rid of it's declaration itself in the interface of JSIModule as part of removal of JSIModule

Close as expected


Helm support

#125 touches upon this, and we already have an unimplemented section here: https://github.com/catppuccin/emacs/blob/3d93abaa33e95f19b4a8b0e1e9bef1e3e68dd994/catppuccin-theme.el#L628-L657
It'd be nice to have helm supported. And if someone can point me to how I could approach this, I can take a stab at it as well.

#125 touches upon this, and we already have an unimplemented section here:
https://github.com/catppuccin/emacs/blob/3d93abaa33e95f19b4a8b0e1e9bef1e3e68dd994/catppuccin-theme.el#L628-L657
It'd be nice to have helm supported. And if someone can point me to how I could approach this, I can take a stab at it as well.

Heyo! if you want to take a go at it i'm free to have a call on discord at any point to explain things, otherwise I can get around to doing this at some point

Heyo! if you want to take a go at it i'm free to have a call on discord at any point to explain things, otherwise I can get around to doing this at some point

Thanks for reaching out! Unfortunately or fortunately, I have been exclusively using Ivy with issues lol. Maybe I'll take a stab at this when I feel the need to switch to Helm :)

All good! ^^


Add EXTRA_OPTS for extra java options which do not fit other types

Hi,
Due to an internal Tumblr project we are in need of adding extra flexibility to the init script. The sysconfig configuration file is handled by the users, starting with a very basic default. The need here is to allow certain java command line options which may not fit the categories otherwise specified. Most are very specific. APP_OPTS the most generic comes after sourcing of the sysconfig file so there's no much wiggle room.
@defect @roymarantz @qx-xp @komapa @byxorna @gtorre
Thanks a bunch,
Vincent

LGTM!


Define runtime operations for OpValue

we need some basic evaluation routines of the OpValue (regardless of what variant it is):

[ ] add/sub: add up two OpValue
[ ] multiply: mul two OpValue and mul a OpValue with a Vector
[ ] Kronecker mul
[ ] trace
[ ] det
[ ] partial trace

we should also allow mixing between Num and OpValue.


Bioc submission prep -- stage 2

Remaining changes for to be fully compliant for Bioc guidelines.

Looks good Sid!  This should be good to go once Travis CI is happy.
Question for you:

Can we hang onto the rendered vignettes somewhere in the package until they are available on BioC like http://www.bioconductor.org/packages/release/bioc/vignettes/ggbio/inst/doc/ggbio.pdf
I don't know how long it will take to actually get this into BioC and in the interim I'd like to keep that documentation around somewhere.
Perhaps we can configure them to be removed from the installed package like so:
To exclude files from being installed, one can specify a list of exclude patterns in file .Rinstignore in the top-level source directory.

I also like the additions Sid, but when I try this specific pull I get the following error.  Should the version restriction be changed?
> devtools::install_github("googlegenomics/api-client-r#31")
Downloading github repo siddharthab/api-client-r@master
Installing GoogleGenomics
Installing dependencies for GoogleGenomics:
S4Vectors
'/home/pgrosu/apps/R-3.1.0/bin/R' --vanilla CMD INSTALL  \
  '/tmp/RtmpMcwgsi/devtools934082fea6f/siddharthab-api-client-r-768cf0a'  \
  --library='/home/pgrosu/apps/R-3.1.0/library' --install-tests

ERROR: dependency âS4Vectorsâ is not available for package âGoogleGenomicsâ
* removing â/home/pgrosu/apps/R-3.1.0/library/GoogleGenomicsâ
Error: Command failed (1)
In addition: Warning message:
package âS4Vectorsâ is not available (for R version 3.1.0)
>

Thanks,
~p

I hope you don't mind my chiming in.  You need to install Bioconductor
dependencies directly.  devtools::install_github does not automatically
pick up Bioconductor packages.  In particular, something like:
source('http://bioconductor.org/biocLite.R')
biocLite('GenomicAlignments')
will probably get you pretty close.
Sean
On Fri, Feb 27, 2015 at 3:13 PM, Paul Grosu<EMAIL_ADDRESS>wrote:

I also like the additions Sid, but when I try this specific pull I get the
following error. Should the version restriction be changed?

devtools::install_github("googlegenomics/api-client-r#31")
Downloading github repo siddharthab/api-client-r@master
Installing GoogleGenomics
Installing dependencies for GoogleGenomics:
S4Vectors
'/home/pgrosu/apps/R-3.1.0/bin/R' --vanilla CMD INSTALL
'/tmp/RtmpMcwgsi/devtools934082fea6f/siddharthab-api-client-r-768cf0a'
--library='/home/pgrosu/apps/R-3.1.0/library' --install-tests

ERROR: dependency âS4Vectorsâ is not available for package âGoogleGenomicsâ

removing â/home/pgrosu/apps/R-3.1.0/library/GoogleGenomicsâ
Error: Command failed (1)
In addition: Warning message:
package âS4Vectorsâ is not available (for R version 3.1.0)

Thanks,
~p
—
Reply to this email directly or view it on GitHub
https://github.com/googlegenomics/api-client-r/pull/31#issuecomment-76463632
.

Another way:
source("http://bioconductor.org/biocLite.R") biocLite()
options(repos=biocinstallRepos())
Complete instructions in repo Readme. Thanks for trying this out. :)

Sid I actually did those steps previously.
Thanks Sean, I'll try that - it used to work before.

Maybe you are not using Bioc 3.0. This is something I will bring up with the bioc teams during review if we should be directly importing the S4Vector package. For now, this will not work with Bioc <3.0.

Below are all the exploratory analysis steps.  Maybe the R version should be restricted to a higher version.
> source("http://bioconductor.org/biocLite.R")
Bioconductor version 2.14 (BiocInstaller 1.14.3), ?biocLite for help
A newer version of Bioconductor is available for this version of R,
  ?BiocUpgrade for help
>
>
> biocLite()
BioC_mirror: http://bioconductor.org
Using Bioconductor version 2.14 (BiocInstaller 1.14.3), R version
  3.1.0.
>
>
> options(repos=biocinstallRepos())
>
>
> install.packages("devtools")
trying URL 'http://cran.fhcrc.org/src/contrib/devtools_1.7.0.tar.gz'
Content type 'application/x-gzip' length 130760 bytes (127 Kb)
opened URL
==================================================
downloaded 127 Kb

* installing *source* package âdevtoolsâ ...
** package âdevtoolsâ successfully unpacked and MD5 sums checked
** libs
gcc -std=gnu99 -I/home/pgrosu/apps/R-3.1.0/include -DNDEBUG  -I/usr/local/include    -fpic  -g -O2  -c devtools.c -o devtools.o
gcc -std=gnu99 -shared -L/usr/local/lib64 -o devtools.so devtools.o
installing to /home/pgrosu/apps/R-3.1.0/library/devtools/libs
** R
** inst
** preparing package for lazy loading
** help
*** installing help indices
** building package indices
** testing if installed package can be loaded
* DONE (devtools)

The downloaded source packages are in
        â/tmp/RtmpopOLB9/downloaded_packagesâ
Updating HTML index of packages in '.Library'
Making 'packages.html' ... done
>
>
> devtools::install_github("googlegenomics/api-client-r#31")
Downloading github repo siddharthab/api-client-r@master
Installing GoogleGenomics
Installing dependencies for GoogleGenomics:
S4Vectors
'/home/pgrosu/apps/R-3.1.0/bin/R' --vanilla CMD INSTALL  \
  '/tmp/RtmpopOLB9/devtoolsb16b6428ab10/siddharthab-api-client-r-768cf0a'  \
  --library='/home/pgrosu/apps/R-3.1.0/library' --install-tests

ERROR: dependency âS4Vectorsâ is not available for package âGoogleGenomicsâ
* removing â/home/pgrosu/apps/R-3.1.0/library/GoogleGenomicsâ
Error: Command failed (1)
In addition: Warning message:
package âS4Vectorsâ is not available (for R version 3.1.0)
>
>
> biocLite('GenomicAlignments')
BioC_mirror: http://bioconductor.org
Using Bioconductor version 2.14 (BiocInstaller 1.14.3), R version
  3.1.0.
Installing package(s) 'GenomicAlignments'
trying URL 'http://bioconductor.org/packages/2.14/bioc/src/contrib/GenomicAlignments_1.0.6.tar.gz'
Content type 'application/x-gzip' length 1936108 bytes (1.8 Mb)
opened URL
==================================================
downloaded 1.8 Mb

* installing *source* package âGenomicAlignmentsâ ...
** libs
gcc -std=gnu99 -I/home/pgrosu/apps/R-3.1.0/include -DNDEBUG  -I/usr/local/include -I"/home/pgrosu/apps/R-3.1.0/library/IRanges/include"   -fpic  -g -O2  -c IRanges_stubs.c -o IRanges_stubs.o
gcc -std=gnu99 -I/home/pgrosu/apps/R-3.1.0/include -DNDEBUG  -I/usr/local/include -I"/home/pgrosu/apps/R-3.1.0/library/IRanges/include"   -fpic  -g -O2  -c R_init_GenomicAlignments.c -o R_init_GenomicAlignments.o
gcc -std=gnu99 -I/home/pgrosu/apps/R-3.1.0/include -DNDEBUG  -I/usr/local/include -I"/home/pgrosu/apps/R-3.1.0/library/IRanges/include"   -fpic  -g -O2  -c cigar_utils.c -o cigar_utils.o
gcc -std=gnu99 -I/home/pgrosu/apps/R-3.1.0/include -DNDEBUG  -I/usr/local/include -I"/home/pgrosu/apps/R-3.1.0/library/IRanges/include"   -fpic  -g -O2  -c encodeOverlaps_methods.c -o encodeOverlaps_methods.o
gcc -std=gnu99 -shared -L/usr/local/lib64 -o GenomicAlignments.so IRanges_stubs.o R_init_GenomicAlignments.o cigar_utils.o encodeOverlaps_methods.o
installing to /home/pgrosu/apps/R-3.1.0/library/GenomicAlignments/libs
** R
** inst
** preparing package for lazy loading
** help
*** installing help indices
** building package indices
** installing vignettes
** testing if installed package can be loaded
* DONE (GenomicAlignments)

The downloaded source packages are in
        â/tmp/RtmpopOLB9/downloaded_packagesâ
Updating HTML index of packages in '.Library'
Making 'packages.html' ... done
>
>
> devtools::install_github("googlegenomics/api-client-r#31")
Downloading github repo siddharthab/api-client-r@master
Installing GoogleGenomics
Installing dependencies for GoogleGenomics:
S4Vectors
'/home/pgrosu/apps/R-3.1.0/bin/R' --vanilla CMD INSTALL  \
  '/tmp/RtmpopOLB9/devtoolsb16b7b332274/siddharthab-api-client-r-768cf0a'  \
  --library='/home/pgrosu/apps/R-3.1.0/library' --install-tests

ERROR: dependency âS4Vectorsâ is not available for package âGoogleGenomicsâ
* removing â/home/pgrosu/apps/R-3.1.0/library/GoogleGenomicsâ
Error: Command failed (1)
In addition: Warning message:
package âS4Vectorsâ is not available (for R version 3.1.0)
>
>
> biocLite('S4Vectors')
BioC_mirror: http://bioconductor.org
Using Bioconductor version 2.14 (BiocInstaller 1.14.3), R version
  3.1.0.
Installing package(s) 'S4Vectors'
Warning message:
package âS4Vectorsâ is not available (for R version 3.1.0)
>

Yes, BioC is lower than 3.0 and probably the scripts should be more stringent.  I was basically following the Readme pretending to be a naive user.

I will fix the readme for now to include the additional command but I will
bring this up with the Bioc team.
On Fri, 27 Feb 2015 12:46 Paul Grosu<EMAIL_ADDRESS>wrote:

Yes, BioC is lower than 3.0 and probably the scripts should be more
stringent. I was basically following the Readme pretending to be a naive
user.
—
Reply to this email directly or view it on GitHub
https://github.com/googlegenomics/api-client-r/pull/31#issuecomment-76468644
.

Cool, thanks Sid :)  This would be the BioC team @ Google or the core BioC team?

Core Bioc team.

Awesome!

I am one of the bioc core developers, though you'll find much deeper expertise in Martin and the crew in Seattle!  Just a couple of points.
First, after submitting to Bioconductor, you will not have to worry about older versions.  Bioconductor is "released" every six months, unlike CRAN.  The packages in Bioconductor are released en bloc and are only guaranteed to work within a single version of R (and other release Bioc packages).
Second, the biocLite.R script first checks if you have the BiocInstaller package already.  If you do, it uses it.  You'll note in the output from https://github.com/googlegenomics/api-client-r/pull/31#issuecomment-76468342 this little note:
> source("http://bioconductor.org/biocLite.R")
Bioconductor version 2.14 (BiocInstaller 1.14.3), ?biocLite for help
A newer version of Bioconductor is available for this version of R,
  ?BiocUpgrade for help

This is a mismatch of package versions (in particular, the BiocInstaller package) with R versions, typically arising from having old packages left over from a previous version of R.  Follow the suggestion for BiocUpgrade and you will probably be ready to go.
Hope that helps.

Thanks Sean, it is great to have you on this project.  Yeah I've been using BioConductor for more than 10 years so I'll explore several of these directions.  Maybe we should have a script that performs a check to ensure a clean environment and queries the user before performing a clean and proper install that is compatible with the functionality of the GoogleGenomics R API client.  What do you think?
~p

For a manual install, a line in the Readme will suffice. For standard installs, the Bioc infra is good.

I would tend to agree with @siddharthab that sticking to the Bioconductor install instructions is the way to go.  The version mismatch thing comes up every-now-and-then, arising from installation via tarballs or leftover packages from previous R versions, but doing too much to "help" the user can be problematic as well.  If one adheres to biocLite(), these problems generally do not come up.
As for "manual install", that is definitely not the "bioconductor way" and can lead to problems, particularly when the installer is not familiar with R (sysadmins, for example).  If command-line installation is a must (automated scripting, etc.), then wrapping biocLite(.....) in a small Rscript call from bash is safe and effective.

Hi Sean,
I was thinking of writing a pull request to brute-force this and add
something to DESCRIPTION like
Depends: Biobase ( >= 2.26.0 ) , ...
but this seems cheesy.  It would be great if Depends: could (can?)
explicitly handle a BioC version like that of R, as when one specifies
Depends: R ( >= 3.1.0 ), ... in the DESCRIPTION.
Is there a simple way to force a dependence on "at least BioC-X.Y" in a
package?  If not, would Michael Lawrence and/or BDR sign off on this being
"fixed?"  This comes up on a not-entirely-irregular basis on bioc-devel and
elsewhere.  Seeing as to how Michael is now R-core, if it isn't already
fixed...
Just a thought, mostly along the lines of "can I just address this problem
directly, here, now?"  For some reason I don't have a checkout of r-devel
on my machine right now, otherwise I'd look myself.
On Fri, Feb 27, 2015 at 4:57 PM, Sean Davis<EMAIL_ADDRESS>wrote:

I would tend to agree with @siddharthab https://github.com/siddharthab
that sticking to the Bioconductor install instructions is the way to go.
The version mismatch thing comes up every-now-and-then, arising from
installation via tarballs or leftover packages from previous R versions,
but doing too much to "help" the user can be problematic as well. If one
adheres to biocLite(), these problems generally do not come up.
As for "manual install", that is definitely not the "bioconductor way" and
can lead to problems, particularly when the installer is not familiar with
R (sysadmins, for example). If command-line installation is a must
(automated scripting, etc.), then wrapping biocLite(.....) in a small
Rscript call from bash is safe and effective.
—
Reply to this email directly or view it on GitHub
https://github.com/googlegenomics/api-client-r/pull/31#issuecomment-76500208
.

The case here is that BioC maintains entirely separate repos for different versions. So even if we put a package version restriction in the DESCRIPTION file, that new version will not be found in the old repo and you will get a similar error as Paul reported. In the current Bioc setup, what you are asking is effectively for the R dependency system to switch to a different repo when it knows that a package or a package version can be found in  a higher "version" of a repo. R currently does not have a notion of repo versions, and I personally don't think it's worth the effort.
Maybe biocLite can also output the biocUpdatePackages advisory close to error messages like what Paul encountered. Maybe it does already. Personally, I think this is not a problem when the package is part of Bioc. And if we are using developer versions, then we should be able to read the error messages and act on them. I agree it's tricky right now because our developer version is the only version. So the repo Readme should lay out proper instructions. In this case, the Readme did not explicitly say that the Bioc repos must be 3.0 or greater. I can also include in a call to biocUpdatePackages in the installation instructions but that might not be desired by the user in some cases.
The Travis CI system we have in place for this repo does install the package from a clean environment and run all the checks for every change we push. Since the developer version only has to be compatible with devel Bioc, we currently only run these checks against devel Bioc. But this can be easily changed as well to run two parallel tests.

Well-said, @siddharthab.
Just a note that once you are in the Bioc repository, you are relatively "protected" against breakage since a package that does not build will not replace the previously-built one.  The Bioc build reports are a little delayed (one day, typically) relative to TravisCI, but they are more complete in that Mac, Linux, and Windows are all covered.

Sean and Sid - I understand and agree, but thinking more about what Tim mentioned, why can't there be an installation script that looks something like this:
min.BioC.Version <- "3.0"

get.BioC.Version <- function() {

	tryCatch({
			BiocInstaller::biocVersion() # recent BiocInstaller
		}, error=function(...) {         # no / older BiocInstaller
			BioC_version_associated_with_R_version <-
				get(".BioC_version_associated_with_R_version",
					envir=asNamespace("tools"), inherits=FALSE)
			if (is.function(BioC_version_associated_with_R_version))
				BioC_version_associated_with_R_version()
			else                            # numeric_version
				BioC_version_associated_with_R_version
		})

}

biocVers <- get.BioC.Version()

if ( biocVers < min.BioC.Version ) {
  message( "         Google Genomics requires a newer version of R." )
  stop( "  Please upgrade to the latest version of R." )
}

source("http://bioconductor.org/biocLite.R")
biocLite()
if ( BiocInstaller:::IS_UPGRADEABLE ) {
	biocLite("BiocUpgrade")
}
options(repos=biocinstallRepos())
install.packages("devtools")
devtools::install_github("googlegenomics/api-client-r")
library(GoogleGenomics)

Paul

Because this - http://xkcd.com/1319/
Good spirit Paul, but I think in this case just a comment next to the biocLite command in the Readme will be good. I expect the user to read what he is copy-pasting if it's just 5 lines.

Good point Sid :)  Yeah, I think the large upgrades I used to do in the past got me too comfortable with layers of encapsulation and automated scripts.  Cool discussion in any case.
Have a great weekend everyone!
~p

@deflaux I added a little more polish, restored the vignettes, and stopped running authenticated code paths in PRs (secure environment variables are not available in PRs, so we can not authenticate).
In the CONTRIBUTING.rst, I mentioned that a contributor should, for non-trivial changes, set up Travis on his fork and provide the Travis build link in the PR which would have run the authenticated code paths. Here is mine:
https://travis-ci.org/siddharthab/api-client-r/builds/52840404

Nice work Sid!!
@seandavi, @ttriche, and @pgrosu thanks for the discussion!


SomeNumber(Some)

[ ] create class SomeNumber(Some) that should work similar to a Some(int, float)
[ ] create doc and doctest for class
[ ] add optional parameters min=None and max=None, odd=None, even=None
[ ] add tests for it (SomeNumberTest)
[ ] add SomeNumber to documentation

[ ] would also be interesting to allow with an option that it can also parse str.


BUG: When number of stratified items are not matched, multi methods are not working.

ddcac95 commit added one of practical example. However, since I can not get valid results for stratification analysis, another public dataset should be collected.


REST API

Can anyone give any information as to whether we should ever expect the emulator to support the Service Bus REST API  https://learn.microsoft.com/en-us/rest/api/servicebus/ (or possibly a subset of it)?
I can see that the emulator container sets ASPNETCORE_HTTP_PORTS=8080  (and exposes port 8080) so I hoped it was there already but requests to that endpoint give no sense that there's an API there.
Given the semantics of the API I don't see any reason why the emulator couldn't support this.

@shladdergoo - Are there particular set of APIs that you're interested in?
Current version of Service Bus emulator is compatible with majority of runtime APIs apart from the limitations documented here + SQL filters and Actions.

E.g. https://learn.microsoft.com/en-us/rest/api/servicebus/send-message-to-queue
Should I expect this to work?


Analyzers show "Roslyn" in the messaging, confusing new users

From vscode-dotnettools created by timheuer: microsoft/vscode-dotnettools#511
Describe the Issue
When an analyzer shows in the editor surface it says "Roslyn" next to the message code:

To new users for .NET this is confusing as this is not a term we put in front of any of our learning materials.  In contrast, Visual Studio does not surface this:

Steps To Reproduce
Triggers something with an analyzer:
int[] myNumbers = new[] {1,2,3];
Expected Behavior
Unfamiliar terms with no definition shouldn't be shown and suppressed.
Recommendation: if this is a 'user agent' type string we should consider a different name ("C#?" or just remove Roslyn from this view
Environment Information
No response

Fix in https://github.com/dotnet/roslyn/pull/69939


2-way audio with ICSee/dvrip cameras

Many cheap chinese cameras are using this ICSee. I bought this one: https://aliexpress.com/item/1005005731238053.html
It's relatively good, although it does not have the capability of changing the stream codec from H265 to H264.
But I have another ICSee based camera as well.
The app is good as well. A lot better than Tuya.
It looks like these cameras uses the DVRIP protocol. My RTSP URLs looks like:
rtsp://<IP_ADDRESS>:554/user=tema_password=<password>_channel=0_stream=0&onvif=0.sdp

It would be awesome if 2-way audio could work with it.

2-way audio works through its web UI:

Although it requires the installation of a "VideoPlayTool" to properly open it.
@AlexxIT if someday you are interested in implementing this I can give you access to the camera.

Oh, and by the way DVRIP source also works with this camera:
streams:
  icsee_camera:
    - dvrip://tema:<password>@<IP_ADDRESS>:34567?channel=0&subtype=1

Agreed. It would be great to support 2-way audio for DVRIP/XM/Sofia-based cameras.

https://github.com/OpenIPC/camerasrnd contains links to NetSDK. The main code is compiled to NetSdk.dll and NetSdk.lib.
ClientDemo sources contain the code like
// can set the talk format, default is G711A.
H264_DVR_API bool CALL_METHOD H264_DVR_SetTalkMode(long lLoginID, SDK_AudioInFormatConfig* pTalkMode);

//sending talk data
BOOL CDeviceEditDlg::SendTalkData(LPBYTE pDataBuffer,DWORD dwDataLength)
{
	if ( NULL == m_pTalkDecodeBuf )
	{
		m_pTalkDecodeBuf = new BYTE[4096];
	}
	memset(m_pTalkDecodeBuf, 0, 4096);

	int iCbLen = 0;

	//encode talk data
	if (g711a_Encode( (char*)pDataBuffer , (char*)m_pTalkDecodeBuf+8 , dwDataLength, &iCbLen) != 1)
	{
		ASSERT(FALSE);
		return FALSE;
	}

	m_pTalkDecodeBuf[0]=0x00;
	m_pTalkDecodeBuf[1]=0x00;
	m_pTalkDecodeBuf[2]=0x01;
	m_pTalkDecodeBuf[3]=0xFA;

	m_pTalkDecodeBuf[4]=0x0E; //G711A
	m_pTalkDecodeBuf[5]=0x02;
	m_pTalkDecodeBuf[6]=BYTE(iCbLen&0xff);
	m_pTalkDecodeBuf[7]=BYTE(iCbLen>>8);

	iCbLen += 8;
	return H264_DVR_VoiceComSendData( m_lTalkHandle, (char*)m_pTalkDecodeBuf, iCbLen );
}

It is possible to disassemble NetSdk.dll functions like H264_DVR_StartVoiceCom_NVR_IPC, H264_DVR_VoiceComSendData, H264_DVR_StopVoiceCom, H264_DVR_SetTalkMode to figure out the exact messages for playback audio on the camera.
Hope this may help.

Here is the Talk command implementation for DVRIP protocol
https://github.com/667bdrm/sofiactl/blob/master/sofiactl.pl#L2286
The source file must be PCM G711 ALAW 8kHz mono.
    $decoded = $dvr->PrepareGenericCommand(IPcam::TALK_CLAIM, { # 1434
        Name => 'OPTalk',
        OPTalk => {
            Action => "Claim",
            AudioFormat => {
                BitRate => 519788,
                EncodeType => "G711_ALAW",
                SampleBit => 8,
                SampleRate => 8
            }
        }
    });

    $decoded = $dvr->PrepareGenericCommand(IPcam::TALK_REQ, { # 1430
        Name => 'OPTalk',
        OPTalk => {
            Action => "Start",
            AudioFormat => {
                BitRate => 128,
                EncodeType => "G711_ALAW",
                SampleBit => 8,
                SampleRate => 8000
            }
        }
    });

    if ($cfgInputFile ne '') {
        my $size = -s $cfgInputFile;
        print "File size: $size\n";
        my $count = 0;
        open(IN, "< $cfgInputFile");
        while ($count < $size) {
            my $data;
            read(IN, $data, 320);
            #print $data . "\n";
            $count += 320;
            my $audio_pkt = $dvr->BuildRawPacket(IPcam::TALK_CU_PU_DATA, # 1432
                pack('CCCCCCS', (0x00, 0x00, 0x01, 0xFA, 0x0E, 0x02, 320)) . $data);
            $dvr->{socket}->send($audio_pkt);
            my $reply_head = $dvr->GetReplyHead();
            $dvr->GetReplyData($reply_head);
        }
        close(IN);
    }

Supported in latest master
streams:
  stream: dvrip://test:test@b3.slydiman.me:34569
  tts_only: dvrip://test:test@b3.slydiman.me:34569?backchannel=1
  two_way:
    - dvrip://test:test@b3.slydiman.me:34569
    - dvrip://test:test@b3.slydiman.me:34569?backchannel=1

Supported in latest master:
streams:
  stream: dvrip://username:password@<IP_ADDRESS>:34567
  tts_only: dvrip://username:password@<IP_ADDRESS>:34567?backchannel=1
  two_way:
    - dvrip://username:password@<IP_ADDRESS>:34567
    - dvrip://username:password@<IP_ADDRESS>:34567?backchannel=1

OMG this is amazing. I will test it asap.

Supported in latest master:

Means that will be available in some next update or the current one (1.7.1) ?
Tested it now, but couldn't get it to work, so im guessing it's in the next update

@felipecrs have you tried?

Great work! Thank you very much!
But it works with the microphone only from go2rtc addon UI: Links, select video+audio+microphone = two way audio from camera, click webrtc.html local WebRTC viewer.
I was shocked - there is no way to turn on the microphone in the HA UI. The solution with browser_mod does not work for me (in case of few HA users and Android app).
The next step is to ubdate WebRTC camera custom card to enable the microphone.
Install the Frigate/Frigate addon + Frigate integration + Frigate custom card for 1 button microphone is overkill!

Frigate Card is now supporting external go2rtc in latest dev. Meaning you don't need Frigate.

@felipecrs have you tried?

I'm sorry not yet. But I will try it very soon. My ICSee camera is currently far away from my current location.

@felipecrs you can send audio to it an check result via stream

Oh, yes. This is what I see:
{
  "producers": [
    {
      "type": "RTSP passive producer",
      "url": "rtsp://<IP_ADDRESS>:8554/8aa14331e4a4907604463ca9c8ca54eb",
      "remote_addr": "<IP_ADDRESS>:38036",
      "user_agent": "ffmpeg/go2rtc",
      "sdp": "v=0\r\no=- 0 0 IN IP4 <IP_ADDRESS>\r\ns=go2rtc/1.7.1\r\nc=IN IP4 <IP_ADDRESS>\r\nt=0 0\r\na=tool:libavformat LIBAVFORMAT_VERSION\r\nm=video 0 RTP/AVP 96\r\na=rtpmap:96 H264/90000\r\na=fmtp:96 packetization-mode=1; sprop-parameter-sets=Z2QMKawsqAoBbpp4AAADAAgAAAMAxHhAIVA=,aO44sA==; profile-level-id=640C29\r\na=control:streamid=0\r\n",
      "medias": ["video, recvonly, H.264 High 4.1"],
      "receivers": ["96 H264, bytes=5876964, senders=1"],
      "recv": 5928476
    },
    {
      "type": "RTSP active producer",
      "url": "rtsp://<IP_ADDRESS>:8554/rua_h265?audio=all",
      "remote_addr": "<IP_ADDRESS>:8554",
      "user_agent": "go2rtc/1.7.1",
      "sdp": "v=0\r\no=- 1 1 IN IP4 <IP_ADDRESS>\r\ns=go2rtc/1.7.1\r\nc=IN IP4 <IP_ADDRESS>\r\nt=0 0\r\nm=audio 0 RTP/AVP 96\r\na=rtpmap:96 PCMA/8000\r\na=control:trackID=0\r\nm=audio 0 RTP/AVP 97\r\na=rtpmap:97 MPEG4-GENERIC/8000\r\na=fmtp:97 profile-level-id=1;mode=AAC-hbr;sizelength=13;indexlength=3;indexdeltalength=3; config=158856E500\r\na=control:trackID=1\r\n",
      "medias": [
        "audio, recvonly, PCMA/8000",
        "audio, recvonly, MPEG4-GENERIC/8000"
      ],
      "receivers": ["96 PCMA/8000, bytes=297920, senders=1"],
      "recv": 453571
    }
  ],
  "consumers": [
    {
      "type": "WebRTC/WebSocket async passive consumer",
      "remote_addr": "udp4 srflx <IP_ADDRESS>:64259 related <IP_ADDRESS>:64259",
      "user_agent": "Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/<IP_ADDRESS> Safari/537.36",
      "medias": [
        "video, sendonly, VP8, RTX, VP9, H264, AV1, RED, ULPFEC, FLEXFEC-03",
        "audio, sendonly, OPUS/48000/2, RED/48000/2, G722/8000, PCMU/8000, PCMA/8000, CN/8000, TELEPHONE-EVENT/48000, TELEPHONE-EVENT/8000, L16, PCML",
        "audio, recvonly, OPUS/48000/2, RED/48000/2, G722/8000, PCMU/8000, PCMA/8000, CN/8000, TELEPHONE-EVENT/48000, TELEPHONE-EVENT/8000"
      ],
      "receivers": ["111 OPUS/48000/2, bytes=16351, senders=0"],
      "senders": [
        "102 H264, bytes=1290962, receivers=1",
        "8 PCMA/8000, bytes=65920, receivers=1"
      ],
      "recv": 23261,
      "send": 1371330
    }
  ]
}

Oh sorry, wait a sec. I think I made a mistake.

I don't need logs. Just check via live stream

Yes I can hear myself!!! That's amazing! It's working nice, thanks a lot!

Nice. Thanks to @slydiman for perl and sharing access to camera.

Sure. HA with https and go2rts master work just fine.
I tried to update AlexxIT's WebRTC camera custom card today. I have successfully enabled the microphone recording in the browser but stuck with creating the correct rtc offer request.

How does your go2rtc URL looks like? Mine is:
https://go2rtc-home..com/

Note that you cannot use the hassio ingress for this job.

Why? Please provide more datails. I planned to use go2rtc addon for this purpose. I can configure stand alone go2rtc, but I did not figure out is there any authorization? HA must work with 2 way audio otside the home network.

You don't need to configure a standalone go2rtc, but you need to have go2rtc running with its own domain (not through hassio ingress), so it can be accessed directly by the card.
You can keep using the add-on, just make sure to configure HTTPS/TLS.
It is a long story as to why this is necessary.

You can keep using the add-on, just make sure to configure HTTPS/TLS.

How to configure go2rtc addon to run it with its own domain (not through hassio ingress).
Can you provide a config sample?
And Can you provide a config sample for Frigate card w/o Frigate integration. Frigate documentation says that 2 way audio requires a frigate camera provided by the Frigate integration.

Sorry, I have no config samples to provide. Also, I don't use the Frigate card without the Frigate integration.

Frigate documentation says that 2 way audio requires a frigate camera provided by the Frigate integration.

I think you missed my point that this feature was only added to the Frigate card development version.
Maybe you can find more help in https://github.com/dermotduffy/frigate-hass-card/issues/1206.

I think you missed my point

I'm not missing anything. I think here is the reason for your confusion:

Also, I don't use the Frigate card without the Frigate integration.

The issue you mentioned contains the following quiestion Frigate Hass Card without Frigate NVR and the answer is Hopefully this can be happening someday... Or the webrtc card with your microphone support
I have configured NGINX to access go2rtc addon through HTTPS port 1985. But go2rtc does not have any authorization and it will be a security hole. I can't use it outside my home network.
I have added the following
        - type: custom:frigate-card
          cameras:
          - camera_entity: camera.some_generic
            live_provider: go2rtc
            go2rtc:
              url: https://mydomain:1985
              stream: cam_with_audio
              modes:
               - webrtc
          menu:
            buttons:
              microphone:
                enabled: true

Note frigate-card requires camera_entity anyway and it must be a camera from frigate integration.
I got the error Could not get camera endpoint with this config and Could not determine suitable engine for camera. w/o camera_entity.
The bottom line is

frigate-card does not work w/o frigate.
Frigate addon/integration is overkill.
It is necessary to update WebRTC Camera card to support microphone.

But go2rtc does not have any authorization and it will be a security hole. I can't use it outside my home network.

Yes, that is far from ideal. But you can set go2rtc to require user and password, and maybe you can embed the user and password in the go2rtc.url in the frigate card like https://user:password@mydomain.com:1935.
I agree it would be great if WebRTC Camera had built-in support for 2-way audio. But @AlexxIT is aware, he just didn't have the time and to ship it yet.
Similarly, support for go2rtc through the hassio ingress is something @dermotduffy is also aware and he would like to ship it. Likewise, he probably didn't have time to work on it yet.
Needless to say that both has no obligation to work on anything and probably contributions are welcome.
Few months ago we did not have any kind of 2-way audio support at all in Home Assistant. Things are evolving, and will continue to evolve and be more seamless as the time goes. :)

I have updated WebRTC Camera.
https://github.com/AlexxIT/WebRTC/pull/579
It works just fine in a browser with go2rtc addon. HA must be HTTPS.
Unfortunately the mic does not work inside the Android HA app - it shows that the mic is used, but there is no any transmission.

https://github.com/AlexxIT/go2rtc/releases/tag/v1.8.0

Sorry if my question might sound silly, but is there any chance this can work also with a Amcrest AD410 video doorbell or does it have no DVRIP support?

Amcrest AD410 video doorbell

Seems it uses VoIP/SIP protocols for audio conversations. It is not DVRIP compatible at all.

That's what I was fearing... OK thanks.

Amcrest AD410 doorbell support RTSP with two way audio

Guys, since we are talking about two-way audio here, I have two feature requests in Frigate Card that you may be useful for you too:

https://github.com/dermotduffy/frigate-hass-card/issues/1130
https://github.com/dermotduffy/frigate-hass-card/issues/1241

If you find them useful, please vote on them with a thumbs up. But please do not vote if you don't find them useful.

Another one:

https://github.com/dermotduffy/frigate-hass-card/issues/1299

Hi,
I'm also using a chinese icsee cam which supports 2 ways audio, but I can't get it to work (updated to last webrtc.), I have no microphone on the stream.
My card looks like this :
`type: custom:webrtc-camera
entity: camera.jardin_profile_000
ui: true
media: video,audio,microphone
mic: true
title: Jardin
poster: https://home-assistant.io/images/cast/splash.png
muted: false
intersection: 0.75
background: true
style: 'video {aspect-ratio: 16/9; object-fit: fill;}'
shortcuts:

name: Record
icon: mdi:record-circle-outline
service: switch.toggle
service_data:
entity_id: switch.camera_record
mode: webrtc,mse,hls,mjpeg`

any help would be appreciated

How does the go2rtc config looks like?
Also is your HA exposed as HTTPS?

I use webrtc to show cards of my onvif cams (setup in ICSee)

Yes, my HA is exposed as https via duckdns

You need to add a go2rtc configuration with the drip: back channel link.

sorry for beeing noob, but how can I achieve this ? I only install webrtc to have greater cam cards, and go2rtc is included in web2rtc hacs integration

I use the go2rtc add-on, but I think you can probably create one file named go2rtc.yaml at config with something like:
streams:
  camera.jardin_profile_000:
    - "{input}"
    - dvrip://username:password@<IP_ADDRESS>:34567?backchannel=1

But I am not sure if the "{input}" line will work as I intended. @AlexxIT can comment more on this.

go2rtc add-on config must look like
streams:
  camera.jardin_profile_000:
   - dvrip://admin:password@<IP_ADDRESS>:34567?channel=0&subtype=1
   - dvrip://admin:password@<IP_ADDRESS>:34567?backchannel=1

My card looks like this :

Here is the correct sample of webrtc-camera card  https://github.com/AlexxIT/WebRTC/pull/579#issuecomment-1763474759
Forget about mic: true. It was my suggestion in the pool request. The final implementation requires media: video,audio,microphone.
In your case
type: 'custom:webrtc-camera'
ui: true
streams:
- url: camera.jardin_profile_000
  mode: webrtc
  name: '🎙 Mic on'
  media: video,audio,microphone
muted: false
...

Hi all,
thanks for your time, so, If I understood, I just have to create a file named go2rtc.yaml in the same path where my HAOS config file is, with just these lines (with my IPs of course and credentials):
streams:
camera.jardin_profile_000:

dvrip://admin:password@<IP_ADDRESS>:34567?channel=0&subtype=1
dvrip://admin:password@<IP_ADDRESS>:34567?backchannel=1

And then use this code for the card :
type: 'custom:webrtc-camera'
ui: true
streams:

url: camera.jardin_profile_000
mode: webrtc
name: '🎙 Mic on'
media: video,audio,microphone
muted: false
...

Thanks guys

If I understood, I just have to create a file named go2rtc.yaml in the same path where my HAOS config file is

Right. Be sure the addon go2rtc is started. You can open go2rtc addon settings and click Open web UI or enable the option Show in sidebar, then click on go2rtc in left sidebar. You will see go2rtc's web UI. Then you can click on Config tab, edit the config and then click the button Save & Restart.

Ok guys,
so, I created the file :

Then, restarted HAOS, went to my dashboard, and create a card :

The result is the same (no micro) :

Here is my onvifer result showing the ports :

Any idea ?
Thanks

No need to restart HAOS. It is enough to restart only go2rtc addon after changing go2rtc.yaml
If you need to see the name of the stream with the mic icon and be able to switch off the mic just add a second stream with the same camera, other name and media: video,audio
Be sure you have set permissions to the browser to use the microphone. You must see the rec icon on the page tab. If you are using Google Chrome
Video with muted audio:

Video and audio:

Video, audio and microphone:

Click on the mic icon in the address bar, allow the mic access and select the audio input (microphone) device:

Thanks, indeed the micro in the adress bar was disabled, I enabled it, refresh the page, put a second stream, now I can switch between the streams, but the stream with micro ON doesn't send any audio from my PC...

So, I tried with my tablet, it asked me for audio permissions, then I can hear myself speaking, but very low, I only realised it because of the larsen (being close to the camera). The cam speaker is at 100% volume, when I try with icsee, it's clear and loud.
With my laptop, no sound at all

I didn't test it yet, But I want to say thank you guys for all effort in this feature ❤

Hey @dbuezas
I guess you was trying to make this work, don't you?

It's working just fine for me since the day it was implemented.


Upgraded the go sdk version

This PR upgrades the numaflow go sdk version and bit of housekeeping ,removing non required things from Dockerfile

Do we change docker file?


Allow int amp values for charging speeds other than reduced and maximum

Is your feature request related to a problem? Please describe.
Hi I've a skoda citigo and I would like to select charging speed that I want but I've saw that currently it allow only "maximum" and "reduced" string values which map to 254 and 252, it would be nice to allaow also to take directly int values, infact I've tested by changing the code and if you pass a value from 5 to 32 (for skoda citigo) to the api it set directly the current to the correspond value
Describe the solution you'd like
Somethig like that:
    async def set_current(call):
        """Prepare data and modify for data call."""
        try:
            current = call.data.get('current')
            if current == "maximum":
                current = 254
            elif current == "reduced":
                current = 252
            _LOGGER.debug("Try to fetch object for VIN: %s" % call.data.get("vin", ""))
            vin = call.data.get("vin")
            car = connection.vehicle(vin)
            _LOGGER.debug(f"Found car: {car.nickname}")
            _LOGGER.debug(f"Set charger current to {call.data.get('current')} ({current})")
            await car.set_charger_current(current)
            async_dispatcher_send(hass, SIGNAL_STATE_UPDATED)
            return
        except Exception as error:
            _LOGGER.warning(f"Couldn't execute, error: {error}")
            async_dispatcher_send(hass, SIGNAL_STATE_UPDATED)
        raise Exception(f"Service call failed")

SERVICE_SET_MAX_CURRENT_SCHEMA = vol.Schema(
    {
        vol.Required("vin"): cv.string,
        vol.Required("current"): vol.Any(vol.In(range(5, 33)), vol.In([ "maximum", "reduced"])),
    }
)

I've tested by changing the code and if you pass a value from 5 to 32 (for skoda citigo) to the api it set directly the current to the correspond value

Interesting! I have meant to test this someday but haven't gotten around to it.
I have examined how the app sets the value but haven't really found anything that I can use to determine what values are available for a given car. The same goes for the charge limit.
I will test of this also applies to other cars than the citigo and will then implement it.

I thinks the apis should be almost the same as VW and in fact in the VW app you can actually set the ampere value instead of only "max" ore "reduced". I don't know who had the idea in skoda to put only two options in the app "max" or "reduced" without even explain at what power corresponds but it was a bad idea

I think the apis should be almost the same as VW and in fact in the VW app you can actually set the ampere value instead of only "max" ore "reduced". I don't know who had the idea in skoda to put only two options in the app "max" or "reduced" without even explain at what power corresponds but it was a bad idea

That's true for most Skodas. Not Enyaq iV though, it's running om Skoda native servers and different API calls.
My Superb iV is hosted on VW servers and use the same API calls as VW.
The new VW Passat GTE shares much with the Superb iV and it also only have maximum/reduced as options.

I think the citigo should be on VW servers because it's pretty identic to the e-up
BTW nice work, congrats @Farfar and @lendy007 skodaconnect and homeassistant-skodaconnect are really a great project for skoda owners!

Yes the Citigo is on VW servers. I don't know how this works for the e-up though, never had one and never debugged for one.

BTW nice work, congrats @Farfar and @lendy007 skodaconnect and homeassistant-skodaconnect are really a great project for skoda owners!

Thanks!

This would also improve the graph view.
At the moment the values „maximum“ and „reduced“ are displayed in an empty graph because the unit is ampere. But the values are not suitable.

I have tested and verified that integer values from 1 to 16 works for my Superb iV, it also correctly limits the maximum charging current to the correct value. For instance 8 ampere instead of being limited to reduced (6) and maximum (16).
This will be integrated in upcoming release 1.0.43-RC3

Will that be a breaking change, or will recuded and maximum work additionally to integer values?

Will that be a breaking change, or will recuded and maximum work additionally to integer values?
I guess that setting an integer value will be implemented in the known service?

It's going to be a breaking change since I feel there's no point in supporting both when explicit numbers work better.

Will that be a breaking change, or will recuded and maximum work additionally to integer values?
I guess that setting an integer value will be implemented in the known service?

Scratch that. Seems like Enyaq uses maximum/minimum and not integer values so I will include both options.

Are u sure. Never saw it by myself, but I‘ve heard that Enyaq has more than two charging speeds?

Are u sure? Never saw it by myself, but I‘ve heard that Enyaq has more than two charging speeds.

It might have many more charging speeds but in the app there's only the same two options: maximum and reduced.


Play from Cached Data and then from URL

Hi ,
Is there support for First play from cache data . If the some data or complete data available then first it should play it and then after through the Network

Hi. Now it's not supported, but I can add this functionality in the next versions.

Thanks a lot. When I can expect. Please tell me . I have to implement and I
don’t have any idea about this.
On Wed, 26 Sep 2018 at 9:43 PM, Vladislav Dugnist<EMAIL_ADDRESS>wrote:

Hi. Now it's not supported, but I can add this functionality in the next
versions.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/vdugnist/DVAssetLoaderDelegate/issues/7#issuecomment-424772981,
or mute the thread
https://github.com/notifications/unsubscribe-auth/Agpec3R6tNr_VTpUJtvT79TluKFGv-Wsks5ue6eBgaJpZM4W6oih
.

--
Join ET Prime for fast, insight-rich analysis of news that matters to
you. Stay ahead with the story behind the story. ****
Start Your 15-Day
Free Trial. Visit ETPrime.com
https://prime.economictimes.indiatimes.com/campaign?utm_source=Newsletter&utm_medium=email&utm_campaign=ETPrimeNewsletter&utm_content=Internal_Gmail_Footer_Message******
**

You can use AVPlayer to prefetch a few seconds of the video and hold AVAsset objects in memory. Then when the user will open a file it will start instantly.

Thanks But i want separate the instance that will download some data at put
it at some path . Now when actual video will play then first it will play
from cached video after it will continue through the network and also it
will update cache
On Thu, Sep 27, 2018 at 2:09 PM Vladislav Dugnist<EMAIL_ADDRESS>wrote:

You can use AVPlayer to prefetch a few seconds of the video and hold
AVAsset objects in memory. Then when the user will open a file it will
start instantly.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/vdugnist/DVAssetLoaderDelegate/issues/7#issuecomment-425007793,
or mute the thread
https://github.com/notifications/unsubscribe-auth/Agpec95JSXw73NLjnJT51R-DN1Qq-7-zks5ufI6ugaJpZM4W6oih
.

--
Join ET Prime for fast, insight-rich analysis of news that matters to
you. Stay ahead with the story behind the story. ****
Start Your 15-Day
Free Trial. Visit ETPrime.com
https://prime.economictimes.indiatimes.com/campaign?utm_source=Newsletter&utm_medium=email&utm_campaign=ETPrimeNewsletter&utm_content=Internal_Gmail_Footer_Message******
**

Hi,
When can i except this feature in lib. Waiting for your response

I made draft implementation in feature/data_source branch. Could you help me to test it? I don't have a lot of free time now.

OK. Let me try and i will let you know
On Fri, Sep 28, 2018 at 9:29 PM Vladislav Dugnist<EMAIL_ADDRESS>wrote:

I made draft implementation in feature/data_source branch. Could you help
me to test it? I don't have a lot of free time now.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/vdugnist/DVAssetLoaderDelegate/issues/7#issuecomment-425482788,
or mute the thread
https://github.com/notifications/unsubscribe-auth/Agpec6O10-xVoQJbblCBJ6dmLoN7hxOCks5ufkdZgaJpZM4W6oih
.

--
Join ET Prime for fast, insight-rich analysis of news that matters to
you. Stay ahead with the story behind the story. ****
Start Your 15-Day
Free Trial. Visit ETPrime.com
https://prime.economictimes.indiatimes.com/campaign?utm_source=Newsletter&utm_medium=email&utm_campaign=ETPrimeNewsletter&utm_content=Internal_Gmail_Footer_Message******
**

Hello ,
I played the video  and successfully played the video and after that
playing same video without the network, It does not play .
Ideally it should cache that video and playing same video then it should
play from cache .
On Sat, Sep 29, 2018 at 12:36 PM Sanjay Singh Rathor <
<EMAIL_ADDRESS>wrote:

OK. Let me try and i will let you know
On Fri, Sep 28, 2018 at 9:29 PM Vladislav Dugnist <
<EMAIL_ADDRESS>wrote:

I made draft implementation in feature/data_source branch. Could you
help me to test it? I don't have a lot of free time now.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/vdugnist/DVAssetLoaderDelegate/issues/7#issuecomment-425482788,
or mute the thread
https://github.com/notifications/unsubscribe-auth/Agpec6O10-xVoQJbblCBJ6dmLoN7hxOCks5ufkdZgaJpZM4W6oih
.

--
Join ET Prime for fast, insight-rich analysis of news that matters to
you. Stay ahead with the story behind the story. ****
Start Your 15-Day
Free Trial. Visit ETPrime.com
https://prime.economictimes.indiatimes.com/campaign?utm_source=Newsletter&utm_medium=email&utm_campaign=ETPrimeNewsletter&utm_content=Internal_Gmail_Footer_Message******
**

I have the same request. Thx.


Don't remove bootstrap service from sevice list

This removes the first service from the service list which happens to be bootstrap service and hence the required users and groups are not created as expected.
As a pull request owner and reviewers, I checked that:

[x] Appropriate testing is done and actually running

/approve

/lgtm


Page not valid: duplicate use of ID

I love just Jekyll and use it for my blog now.
But I have a problem the pages are not W3c valid: see output.
So because some
 tags have the same value, the id attribute will reoccur.
I don't get why the id is set to the
 value anyhow?

This issue should be reopen!
It still happen in case that the title contains other language characters like Chinese words. For example:
## abc之一
## abc之二

than the compiled html output this:

abc之一

abc之二

it ignored the characters unless [a-z]. In neither maruku nor rdiscount it can work right.
And I highly recommend that it should output all the title string as same as anyone wrote. Unless these cases:

If the title has been used once, than add a unique random suffix;
If the title contains space, than use hyphen "-" instead;

I know that it may be the markdown engine's affair, but I saw the maruku's bug list hadn't update since 2011-04-14 02:23. And this issue happen in my using Jekyll, so I just comment here for asking help.
Hope you can help, thank you!

Sorry for reopening this thread.
@mojombo I add the following to _config.yml  file to generate table of content.
markdown: rdiscount
rdiscount:
    extensions:
      - generate_toc

Unfortunately, it still generates the same header id, please refer the following screenshot:

Any idea? Thanks in advance!


fix: Remove My Collection insights flex-grow

Description
Removing My Collection insights flex-grow prop to avoid the content being centered without insights.

Before
After

PR Checklist

[ ] I tested my changes on iOS / Android.
[ ] I added screenshots or videos to illustrate my changes.
[ ] I added Tests and Stories for my changes.
[ ] I added an app state migration.
[ ] I hid my changes behind a feature flag.

To the reviewers 👀

[ ] I would like at least one of the reviewers to run this PR on the simulator or device.

Changelog updates
Changelog updates

Cross-platform user-facing changes

iOS user-facing changes

Android user-facing changes

Dev changes

Need help with something? Have a look at our docs, or get in touch with us.

Thanks for the fix 🙌


How to generate shared libboost_xxxx.so without the version number such as .so.1.74.0

I was building shared libs of Boost-for-Android.
I modified build-android.sh, changed "link=static" to "link=shared" and ran build-android.sh.
Finally in ./build/output, I got the libs with their soft links, such as:
libboost_atomic-clang-mt-a64-1_74.so -> libboost_atomic-clang-mt-a64-1_74.so.1.74.0
However, it seems that such libs with the soft links can not be loaded by Android.
My app crashed when loading such libs.
Renaming libboostxxx.so.1.74.0 to libboostxxx.so also does not work.
So how can i build Boost-for-Android to generate libboost_xxxx.so only, without the soft links.
Thanks very much. I am looking forward to your response.

Please copy crash logs here
On Fri, 3 Feb 2023, 09:20 IHaveBeenAHero, @.***> wrote:

I was building shared libs of Boost-for-Android.
I modified build-android.sh, changed "link=static" to "link=shared" and
ran build-android.sh.
Finally in ./build/output, I got the libs with their soft links, such as:
libboost_atomic-clang-mt-a64-1_74.so ->
libboost_atomic-clang-mt-a64-1_74.so.1.74.0
However, it seems that such libs with the soft links can not be loaded by
Android.
My app crashed when loading such libs.
Renaming libboostxxx.so.1.74.0 to libboostxxx.so also does not work.
So how can i build Boost-for-Android to generate libboost_xxxx.so only,
without the soft links.
Thanks very much. I am looking forwards to your response.
—
Reply to this email directly, view it on GitHub
https://github.com/moritz-wundke/Boost-for-Android/issues/251, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AABF5QFN45UPNUOBC7N76ZLWVSWUXANCNFSM6AAAAAAUP5SNUM
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

here's the proper solution: https://github.com/boostorg/boost/pull/206
tickets about the same: #23, #44


Skip tests setting the JVM timezone

This PR relies on the driver feature setting :test/jvm-timezone-setting which is available on the master branch of metabase/metabase. Don't merge this if you want to be compatible with a released version of Metabase.
After merging this PR, tests agains metabase master should pass.

Let's continue in https://github.com/ClickHouse/metabase-clickhouse-driver/pull/129


Long-press no longer auto-selects default alt character

v1.41.1 (beta) on Android 8.0.0.
When long-pressing a key, such as '4', Hacker's used to auto-select the default/most likely alt character, in this case, '$', if I didn't choose something else. Now, long-pressing a key brings up a row of its alt characters and I must navigate to the most-likely alt character and press on it in order to select it.
It used to take a quick second to press a key, like '8' and have it almost instantly print '*'. Now it's slower and more cumbersome.
Is there a setting I'm overlooking? Is this a bug? Or is it its intended behavior?

I have 1.41.1 official, Android 8.1 (holding the upgrade to 9 because of issue 686), but it does this only in landscape mode. Works fine in portrait mode here.
So maybe it depends on keyboard height or mode (they are different between portrait and landscape)?


Feature: The new loading animation at the top of the screen is barely visible

When using a light theme, the global loading animation at the top of the screen is barely visible:

We should consider using a wider loading bar/an other color.
Linked to issue : #900

Proposition :

Height: 4px
Color: #D3E24A (complementary of #594AE2)


[python] Infer default entryPoint if not provided

Description
Brief description of what this PR is about

If this change is a backward incompatible change, why must this change be made?
Interesting edge cases to note here

Could you remove all our LLM tests' entryPoint to see if it works?


Doesn't work on NixOS

Summary
A summary of the issue you're experiencing.
I am getting this when running silk.net on nixos
Exception has occurred: CLR/System.PlatformNotSupportedException
An unhandled exception of type 'System.PlatformNotSupportedException' occurred in Silk.NET.Windowing.Common.dll: 'Couldn't find a suitable window platform. (GlfwPlatform - not applicable, SdlPlatform - not applicable) https://dotnet.github.io/Silk.NET/docs/hlu/troubleshooting.html'
   at Silk.NET.Windowing.Window.Create(WindowOptions options)
   at Window..ctor() in /home/shadowblitz16/Documents/source/C#/GameSharp/Engine/Scene/Node/Window.cs:line 29
   at Program.<Main>$(String[] args) in /home/shadowblitz16/Documents/source/C#/GameSharp/Editor/Program.cs:line 4

Steps to reproduce

Platform: Mobile/Desktop
Framework Version: .NET 7
API Just GLFW right now

Code
    public Window()
    {
        ////////////////////////////////////////////////////
        // CRASH HERE
        ////////////////////////////////////////////////////
        _window = Silk.NET.Windowing.Window.Create(new WindowOptions()
        {
            Size  = new Vector2D<int>(720, 480),
            Title = "Window",
            VSync = true
        });
        _window.Load    += ( ) =>
        {
            OnReady();
        };
        _window.Update  += (e) =>
        {
            OnUpdate((float)e);
        };
        _window.Render  += (e) =>
        {
            OnRender((float)e);
        };
        _window.Closing += ( ) =>
        {
            Dispose();
        };

        _window.Move    += (e) =>
        {
            OnMove(e);
        };
        _window.Resize  += (e) =>
        {
            OnResize(e);
        };
    }

Summary
A summary of the issue you're experiencing. I am getting this when running silk.net on nixos
Exception has occurred: CLR/System.PlatformNotSupportedException
An unhandled exception of type 'System.PlatformNotSupportedException' occurred in Silk.NET.Windowing.Common.dll: 'Couldn't find a suitable window platform. (GlfwPlatform - not applicable, SdlPlatform - not applicable) https://dotnet.github.io/Silk.NET/docs/hlu/troubleshooting.html'
   at Silk.NET.Windowing.Window.Create(WindowOptions options)
   at Window..ctor() in /home/shadowblitz16/Documents/source/C#/GameSharp/Engine/Scene/Node/Window.cs:line 29
   at Program.<Main>$(String[] args) in /home/shadowblitz16/Documents/source/C#/GameSharp/Editor/Program.cs:line 4

Steps to reproduce
* Platform: Mobile/Desktop

* Framework Version: .NET 7

* API Just GLFW right now

Code
    public Window()
    {
        ////////////////////////////////////////////////////
        // CRASH HERE
        ////////////////////////////////////////////////////
        _window = Silk.NET.Windowing.Window.Create(new WindowOptions()
        {
            Size  = new Vector2D<int>(720, 480),
            Title = "Window",
            VSync = true
        });
        _window.Load    += ( ) =>
        {
            OnReady();
        };
        _window.Update  += (e) =>
        {
            OnUpdate((float)e);
        };
        _window.Render  += (e) =>
        {
            OnRender((float)e);
        };
        _window.Closing += ( ) =>
        {
            Dispose();
        };

        _window.Move    += (e) =>
        {
            OnMove(e);
        };
        _window.Resize  += (e) =>
        {
            OnResize(e);
        };
    }

I'll likely not get to debugging this myself for awhile, what is the output of the app when LD_DEBUG=libs? its likely just a missing dependency, glance through there and do some greps through it, and it should be obvious what the problem is

Summary
A summary of the issue you're experiencing. I am getting this when running silk.net on nixos
Exception has occurred: CLR/System.PlatformNotSupportedException
An unhandled exception of type 'System.PlatformNotSupportedException' occurred in Silk.NET.Windowing.Common.dll: 'Couldn't find a suitable window platform. (GlfwPlatform - not applicable, SdlPlatform - not applicable) https://dotnet.github.io/Silk.NET/docs/hlu/troubleshooting.html'
   at Silk.NET.Windowing.Window.Create(WindowOptions options)
   at Window..ctor() in /home/shadowblitz16/Documents/source/C#/GameSharp/Engine/Scene/Node/Window.cs:line 29
   at Program.<Main>$(String[] args) in /home/shadowblitz16/Documents/source/C#/GameSharp/Editor/Program.cs:line 4

Steps to reproduce
* Platform: Mobile/Desktop

* Framework Version: .NET 7

* API Just GLFW right now

Code
    public Window()
    {
        ////////////////////////////////////////////////////
        // CRASH HERE
        ////////////////////////////////////////////////////
        _window = Silk.NET.Windowing.Window.Create(new WindowOptions()
        {
            Size  = new Vector2D<int>(720, 480),
            Title = "Window",
            VSync = true
        });
        _window.Load    += ( ) =>
        {
            OnReady();
        };
        _window.Update  += (e) =>
        {
            OnUpdate((float)e);
        };
        _window.Render  += (e) =>
        {
            OnRender((float)e);
        };
        _window.Closing += ( ) =>
        {
            Dispose();
        };

        _window.Move    += (e) =>
        {
            OnMove(e);
        };
        _window.Resize  += (e) =>
        {
            OnResize(e);
        };
    }

I'll likely not get to debugging this myself for awhile, what is the output of the app when LD_DEBUG=libs? its likely just a missing dependency, glance through there and do some greps through it, and it should be obvious what the problem is

I still don't know what you mean by LD_DEBUG=libs I am not a advanced terminal user
However I think this is a bigger issue. I just tried raylib-cs and it doesn't work either I think it has to do with nuget not working correctly on nixos.
I am considering moving back to kubuntu so feel free to close this.
I think in order to fix it, something has to be done on the nix os side.

You need to set the environment variable named LD_DEBUG to libs, as i stated in discord

You need to set the environment variable named LD_DEBUG to libs, as i stated in discord

how do I do that?

Closing as resolved


sched/group: set clear flag if the group is not really needed

Summary
sched/group: set clear flag if the group is not really needed
The delete flag is not synchronized with the life cycle of the group, if the flag set before waitpid(), the tcb will be mistakenly deleted by group_del_waiter(), use-after-free will happen.
Regression by:
| commit 29e50ffa7374fa4c473d8d4f8cb3506665443d3e (origin/master, origin/HEAD)
| Author: chao an<EMAIL_ADDRESS>| Date:   Mon Mar 4 09:19:27 2024 +0800
|
|     sched/group: move task group into task_tcb_s to improve performance
|
|     move task group into task_tcb_s to avoid access allocator to improve performance
|
|     for Task Termination, the time consumption will be reduced ~2us (Tricore TC397 300MHZ):
|     15.97(us) -> 13.55(us)
|
|     Signed-off-by: chao an<EMAIL_ADDRESS>

Impact
N/A
Testing
rv-virt:ksmp64

@masayuki2009 @xiaoxiang781216 @acassis
This PR is to resolve the regression issue caused by #11832

If #11832 needs further review, please merge #11883 and I will do a fully test based on the new PR.
If the #11885 test has no regression issues, then this PR could be merged

@anchao
I confirmed this PR fixes the issues.


Fix the bug "IsDependeeOf is not working"

Fix the FrostingEngine.ConfigureTasks()
Add a .GetTaskName() helper for IReverseTaskDependency
Add unit test for IsDependeeOf
Add DependeeTask to Example project

 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.:x: VSerykh sign nowYou have signed the CLA already but the status is still pending? Let us recheck it.

@VSerykh your changes have been merged, thanks for your contribution 👍


Fixed tests, update dev dependencies, update phpunit config.

Q
A

Is bugfix?
✔

New feature?
❌

Breaks BC?
❌

👍


PDP1 Assembler

Greetings,
I found PDP1_MiSTer\pong\pong.asm and PDP1_MiSTer\paper_tapes\pong.rim.
Is there an assembler to go from pong.asm to pong.rim
Thank you for your help.
Regards,
phiho

Yes, it runs as a program on the pdp-1 itself. However, it's much easier to use the ported one and run it on Linux.
Check this out: https://github.com/simh/simtools
There is even a Debian package in experimental :)
Good luck!

Hi,
Would you please give me a link and instructions to use it on PDP1_MISTer.
Thank you.
Regards,
phiho
On Wed, Dec 9, 2020 at 4:12 AM Hrvoje Čavrak<EMAIL_ADDRESS>wrote:

Yes, it runs as a program on the pdp-1 itself. However, it's much easier
to use the ported one and run it on Linux.
Check this out: https://github.com/simh/simtools
There is even a Debian package in experimental :)
Good luck!
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/MiSTer-devel/PDP1_MiSTer/issues/3#issuecomment-741639644,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABDAWSVPAX24TX6DUPOUEALST45PFANCNFSM4UTEQZNA
.

The repo is at https://github.com/simh/simtools, you clone it using:
git clone https://github.com/simh/simtools.git
then cd simtools, and make
This will create a simtools/crossassemblers/macro1/ folder that contains the compiled macro1 binary you use to translate an assembly language source into .rim file that can be loaded into the pdp-1.
Please note none of this is terribly user-friendly but I'm sure you'll get the hang of it after investing some time.
Good luck!

Hi,
Thanks a lot for the link. I will look into it.
Regards,
phiho


vncconsole doesn't have timeout

vncconsole doesn't have timeout and the same vncconsole url could be used in different browser. This might have potential security issue.
It is better to be add timeout and 1 url for 1 connection.

Thanks for reporting this bug！


validate_enum() to accept any type of container

I have made a small modification to the validate_enum() validator so it accepts any container, i.e. anything that implements contains.
My use case is to use the "enum" property with a set() / frozenset() so item lookup is much faster.
All tests still pass.

thanks for this, sounds good to me


scatter dataframe

when I run the following code
_ = ddf['col_name'].map_partitions(foo, meta=tuple()).compute()

I received a warning message
/usr/local/lib/python3.6/dist-packages/distributed/worker.py:3254: UserWarning: Large object of size 153.89 MB detected in task graph:
  ('getitem-145f763f7858c68f69577c5b88cbb259',       ... ], 'col_name')
Consider scattering large objects ahead of time
with client.scatter to reduce scheduler burden and
keep data on workers

    future = client.submit(func, big_data)    # bad

    big_future = client.scatter(big_data)     # good
    future = client.submit(func, big_future)  # good

Is there an easy way to scatter dataframe before runing map_partitions?
should I use a larger npartitions for the dataframe?

The warning doesn't necessarily relate to your map_partitions call.  It
could be due to anything in your computation up until that point.
I don't know exactly wher your problem is, but in dataframe computations
this is a common issue:
https://docs.dask.org/en/latest/best-practices.html#load-data-with-dask
On Wed, Nov 13, 2019 at 3:46 AM Kun Liu<EMAIL_ADDRESS>wrote:

when I run the following code
_ = ddf['col_name'].map_partitions(foo, meta=tuple()).compute()
I received a warning message
/usr/local/lib/python3.6/dist-packages/distributed/worker.py:3254: UserWarning: Large object of size 153.89 MB detected in task graph:
('getitem-145f763f7858c68f69577c5b88cbb259',       ... ], 'col_name')
Consider scattering large objects ahead of time
with client.scatter to reduce scheduler burden and
keep data on workers
future = client.submit(func, big_data)    # bad

big_future = client.scatter(big_data)     # good
future = client.submit(func, big_future)  # good

Is there an easy way to scatter dataframe before runing map_partitions?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/dask/dask/issues/5585?email_source=notifications&email_token=AACKZTHBENH26PXNE267K5LQTPSI7A5CNFSM4JM2BFW2YY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4HY7P6CA,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AACKZTDZQNNQHLHCEUTR4MDQTPSI7ANCNFSM4JM2BFWQ
.

Closing this for now.  If there is more information here then let us know and we'll reopen.