Added edgeList for graph analysis

Just in case you wanted to see the data represented as a bipartite graph. There are two types of nodes: people and answers. The answers have the question they are answering prepended to them. People are connected to answers. This is useful if you want to explore differences and similarities of different groups.

I think this would be more user friendly if you split them into the different questions and made separate files for each different question and dropped them into an edges directory in data. That way someone could just look at tool networks or thought leader networks, rather than the whole occurrence across all questions, which I find cumbersome.
Do you mind breaking it up like that?


Ashlands Public Release

Ashlands has been release, wondering if there will be an update soon?
https://www.valheimgame.com/news/patch-0-218-15-ashlands

Is it working for you now? I pulled the container just now, after not running it since January, and it updated and now has Ashlands (confirmed by going to the biome).

I did a container restart and it seems to have trigger the update cron job. So it's working. DOn't know why it did update following the 15 min trigger before but I'll investigate to see if I can find something. I'll close this for now.

not to necro, but this solution also worked for me. Restarting the container triggered an update

If you're still having an issue with the new update: ValheimPlus has not been updated yet, so if you enable valheim plus when you create the container, it will give you a version mismatch error when you try to connect.


feat: add local version and devrelease as inputs

This is a PR to add devrelease and local_version to the github action since its already supported by commitizen here.

Tagging @woile since I'm not able to add reviewers to take a look at this! Thanks! (I saw your name in a bunch of other PR merges)

This one will be useful, please, add it to release!


colorTextA / colorTextB - Ability to set the color of the text in badges

:clipboard: Description
Similar to the existing colorA/colorB parameters, but to set the color of the text in each half of generated badges instead of the background. This would allow users to make use of light background colors by setting a darker text color.

Hi, thanks for bringing this up. Let's track this at #504.


So Buggy

I can't even describe the bug, the first file run fine. Then all the rest of my json stop working
It's keep crashing

This project is deprecated and I don't maintain it anymore. Please use https://github.com/windows-toolkit/Lottie-Windows


Is it possible to encrypt passwords?

Is it possible to encrypt passwords?

If you want to avoid storing passwords in the test repo in clear text, you can:

Encrypt the password using the command line:
opentest encrypt --password my_secret_password the_text_to_encrypt

Store the encryped password in a data file:
password: hvrOgOOjOarssey+4XhEAg==

Store the encryption password (the one that you used in the command line) in your actor.yaml file, using the encryptionPassword configuration parameter:
# ...
encryptionPassword: my_secret_password
# ...

Use the $decrypt JavaScript API to decrypt the password dynamically at runtime:
- description: Log in
action: org.getopentest.selenium.SendKeys
args:
    locator: { name: password }
    text: $decrypt($data("config").password)

Of course, everybody else in your team will have to set the encryptionPassword parameter correctly in their test actors, if they want to be able to run the test successfully.

Please explain below code
opentest encrypt --password my_secret_password the_text_to_encrypt
my_secret_password - ?
the_text_to_encrypt - ?
Do i need to enter my own password here
encryptionPassword: my_secret_password

For now this code is not working for me

description: Log in
action: org.getopentest.selenium.SendKeys
args:
locator: { name: password }
text: $decrypt($data("config").password)

So let's say you have a log in test, and the password used for authentication is red22beetle.
You don't want to store the password in the test repo in clear text, so you must store it in encrypted form. To figure out the encrypted version, you have to run the opentest encrypt command. Of course, you also need to provide the encryption password - the one that you're going to need to decrypt the log in password when you run your test. So let's say the encryption password is 5h2o8dF9. In this case, the command to figure out the encrypted password is:
opentest encrypt --password 5h2o8dF9 red22beetle

If you run this command, you'll get PamhjNlJ2N9QN3PV+B94FQ== as your encrypted password. You can store this password in your test repo and decrypt it at runtime when your test executes, as explained in my previous reply. Let me know if that makes sense.

Step 1 :
C:\Users\ksood>opentest encrypt --password Acs@0906 red22beetle
sgqSx2arYogUSoKONZD8zg==
STEP 2 : Adding this code in login file

name: ksood
password: sgqSx2arYogUSoKONZD8zg==

Step 3 : Adding code in Actor file
encryptionPassword: red22beetle

description: Type in password field
action: org.getopentest.selenium.SendKeys
args:
locator: {  xpath: "//input[@placeholder='Password']" }
        text: $decrypt($data("login")[0].password)

But, it is not working

Did you put the encryptionPassword in your actor.yaml?
# ...
encryptionPassword: Acs@0906
# ...

Could you log the encrypted and unencrypted data, to verify what it looks like?
- script: |
    $log("Encrypted = " + $data("login")[0].password)
    $log("Unencrypted = " + $decrypt($data("login")[0].password))

Hi Adrian,
I am able to encrypt and decrypt the password. Please let me know how we can avoid decrypted password to be logged in logs available in running actor and server
Thanks,
Ankit Ranjan

@ariesranjan There is currently no way to prevent passwords and other secrets to appear in log files. Your question made me think of a possible implementation for this, though. OpenTest could take note of all values returned from the $decrypt() function (which presumably contain sensitive data) and mask those values whenever it finds them in data that is being written in log files. If this sounds like a feature that would get the job done, could you please create a GitHub issue specifically for this, so we can track it better?

@ariesranjan There is currently no way to prevent passwords and other secrets to appear in log files. Your question made me think of a possible implementation for this, though. OpenTest could take note of all values returned from the $decrypt() function (which presumably contain sensitive data) and mask those values whenever it finds them in data that is being written in log files. If this sounds like a feature that would get the job done, could you please create a GitHub issue specifically for this, so we can track it better?

Hi @adrianth ,
Has this been implemented? or is neede the request ticket? As I see this will be very usefull.

@ewmartinez Yes, this is implemented. Please see the $maskSecret function.


Does asciidoctorj v1.5.7 support asciidoctorj-pdf v1.5.0-alpha.16?

I'm testing the asciidoctorj v1.5.7 with asciidoctor-pdf-example by changing the pom.xml as the following: -
First try
<asciidoctorj.version>1.5.7</asciidoctorj.version>
<jruby.version><IP_ADDRESS></jruby.version>

There are some error display at console
uri:classloader:/gems/prawn-svg-0.27.1/lib/prawn/svg/color.rb:212: warning: shadowing outer local variable - result
uri:classloader:/gems/prawn-svg-0.27.1/lib/prawn/svg/properties.rb:50: warning: `*' interpreted as argument prefix
uri:classloader:/gems/ttfunk-1.5.1/lib/ttfunk/table.rb:26: warning: instance variable @offset not initialized
...
uri:classloader:/gems/prawn-2.2.2/lib/prawn/font/ttf.rb:118: warning: instance variable @italic_angle not initialized
uri:classloader:/gems/ttfunk-1.5.1/lib/ttfunk/table.rb:26: warning: instance variable @offset not initialized
...
uri:classloader:/gems/prawn-2.2.2/lib/prawn/font/ttf.rb:118: warning: instance variable @italic_angle not initialized
uri:classloader:/gems/ttfunk-1.5.1/lib/ttfunk/table.rb:26: warning: instance variable @offset not initialized
...

Second try
<asciidoctorj.version>1.5.7</asciidoctorj.version>
<jruby.version><IP_ADDRESS></jruby.version>

<sourceHighlighter>rouge</sourceHighlighter>

There are some error display at console
uri:classloader:/gems/prawn-svg-0.27.1/lib/prawn/svg/color.rb:212: warning: shadowing outer local variable - result
uri:classloader:/gems/prawn-svg-0.27.1/lib/prawn/svg/properties.rb:50: warning: `*' interpreted as argument prefix
uri:classloader:/gems/rouge-2.0.7/lib/rouge/lexers/bsl.rb:62: warning: Ambiguous first argument; make sure.
...
uri:classloader:/gems/rouge-2.0.7/lib/rouge/lexers/io.rb:28: warning: Ambiguous first argument; make sure.
...
uri:classloader:/gems/rouge-2.0.7/lib/rouge/lexers/perl.rb:68: warning: Ambiguous first argument; make sure
...
uri:classloader:/gems/rouge-2.0.7/lib/rouge/lexers/matlab.rb:30: warning: Ambiguous first argument; make sure.
...
uri:classloader:/gems/rouge-2.0.7/lib/rouge/lexers/jinja.rb:39: warning: Ambiguous first argument; make sure.
...
uri:classloader:/gems/rouge-2.0.7/lib/rouge/lexers/qml.rb:19: warning: Ambiguous first argument; make sure.
...
uri:classloader:/gems/rouge-2.0.7/lib/rouge/lexers/javascript.rb:34: warning: Ambiguous first argument; make sure.
...
uri:classloader:/gems/rouge-2.0.7/lib/rouge/lexers/scala.rb:38: warning: Ambiguous first argument; make sure.
... too many languages ....
uri:classloader:/gems/ttfunk-1.5.1/lib/ttfunk/table.rb:26: warning: instance variable @offset not initialized
...
uri:classloader:/gems/prawn-2.2.2/lib/prawn/font/ttf.rb:118: warning: instance variable @italic_angle not initialized
uri:classloader:/gems/ttfunk-1.5.1/lib/ttfunk/table.rb:26: warning: instance variable @offset not initialized
...
uri:classloader:/gems/prawn-2.2.2/lib/prawn/font/ttf.rb:118: warning: instance variable @italic_angle not initialized
uri:classloader:/gems/ttfunk-1.5.1/lib/ttfunk/table.rb:26: warning: instance variable @offset not initialized
...

Does asciidoctorj v1.5.7 support asciidoctorj-pdf v1.5.0-alpha.16?

These messages are warnings from JRuby complaining about the Ruby code that is in the used libraries, but that shouldn't result in the document not being created.
Do you get a PDF, or does the Maven build fail with some error?

Except the long printing logs, the maven build successfully and I can get the proper PDF.
Every source code inside it are highlighted.

Furthermore, when I use include with tag as the following: -
[source, java, linenums]
.Some example code
----
include::Hello.java[tag=hello]
----

The Hello.java
// tag::hello[]
public class Test {
  public static void main(String[] args) {
    System.out.println("Hello World!"); <1>
  }
}
// end::hello[]

There is a warning as
WARNING: some-file: line 34: detected unclosed tag 'hello' starting at line 2 of include file:Hello.java

The PDF highlighted output are including that end tag as
public class Test {
  public static void main(String[] args) {
    System.out.println("Hello World!"); <1>
  }
}
// end::hello[] <-- This should not be display in PDF

Something tells me it's because the tag is on the last line of the file. It
shouldn't be a problem, but for some reason it is. Then again, you don't
really need a tag in this case since you're including the whole file.
Can you run oc -d on that file (or whatever command will output the
contents in binary?)

 It seems we're missing a test case for this scenario. I'll pursue this upstream. In the meantime, you can solve the problem by adding a trailing newline to the Java file.

See https://github.com/asciidoctor/asciidoctor/issues/2830
Thanks for reporting @charleech!

Thank you. I can confirm that adding new line at the end of file solve this.
Anyhow, how about the JRuby warning? How can I get rid of if?

It looks like AsciidoctorJ enables verbose mode by default, which causes Ruby to report these warnings.
https://github.com/asciidoctor/asciidoctorj/blob/f8c08a782230a37e54f04174389bbf326b34fa76/asciidoctorj-core/src/main/resources/org/asciidoctor/internal/asciidoctorclass.rb
I don't think that should be hardcoded. That should be an option that is set when the Asciidoctor instance is configured. Although the warnings are harmless, it's not always desirable to have this setting on. First because of pedandic Ruby warnings. But also because some validations in Asciidoctor are still false positives and you may not want them on in a production system / pipeline.
I think we'll need a more specific issue to address this change request.

It's true that it does need to be set for that outcome, but it should be managed. I recommend logic similar to that used by the CLI's invoker: https://github.com/asciidoctor/asciidoctor/blob/master/lib/asciidoctor/cli/invoker.rb#L65-L76
The user doesn't always want to see the invalid refs warning because it's still not always accurate. It's certainly much better, but there are still times when it's overzealous (due to how inline references are handled).

I had a look at that while implementing the logger.
But it was a bit confusing to me: We implemented loggers with log levels that allow to control the amount of information you get, but need to set a global variable to get such warnings.
Would it make sense to log with a different progname then?
Or would it make sense to control this kind of logging with another variable? Controlling interpreter warnings from JRuby and invalid ref warnings from Asciidoctor with the same global variable comes unexpected.

Also the verbose mode of AsciidoctorJ historically seems to work a bit different than the Asciidoctor CLI. It also seems to print out timings.
Should I remove this and align with the Asciidoctor CLI?

Also the verbose mode of AsciidoctorJ historically seems to work a bit different than the Asciidoctor CLI. It also seems to print out timings.

The Asciidoctor CLI used to do this, but then moved the timings to the -t, --timings flag. So yes, I recommend aligning with it.

The Asciidoctor CLI used to do this, but then moved the timings to the -t, --timings flag. So yes, I recommend aligning with it.

Ah, good to hear this, I was already wondering about the differences in behavior between asciidoctor and asciidoctorj.
I'll create PRs to align it.

Since this issue of verbose mode has also come up in the Gradle plugin, I think we need a dedicated issue to rethink how to handle it properly. Specifically, there's confusion about debug mode in the logger and Ruby's verbose mode, which has been complicated by the fact that I associated one of the warnings in Asciidoctor core with verbose mode instead of the debug level. But there's still value in being able to control verbose mode, even from the API. But we should view it more as an environment setting (like the gem path). @robertpanzer could you open a new issue to discuss how to control the value of the $VERBOSE global variable through the Asciidoctor API (perhaps at initialization time).


Move logs files to ramdisk

Has the image comes with a 20Mb RAM disk, I think it should be good to move the log files under .local/state/mycroft/*.log to /ramdisk

This is done in the newest dev image
https://ovosimages.ziggyai.online/raspbian/development/
there are sys-links from /ramdisk/mycroft to ~/.local/state/mycroft so the default paths still work


🚀 Feature: Add the possibility to disable watching files in the new backend system

🔖 Feature description
  const config = await loadBackendConfig({
    argv: process.argv,
    logger,
    watch: false,
  });

Have the possibility to pass as input watch: false to the new backend system.
🎤 Context
To disable watching files and avoiding this issue: https://github.com/backstage/backstage/issues/19619
✌ Possible Implementation
No response
👀 Have you spent some time to check if this feature request has been raised before?

[X] I checked and didn't find similar issue

🏢 Have you read the Code of Conduct?

[X] I have read the Code of Conduct

Are you willing to submit PR?
None

Thanks for raising the issue & PR. For keeping the references - you fixed this for the old system here: https://github.com/backstage/backstage/pull/19718.

Hi @tudi2d, when will the PR be merged?

Hej @antoniomuso - as soon as the PR has 2 approving reviews. As we have an active community & also work on other things than reviewing PRs this can take some days.


Add ability to set project to add ordered items to.

Correct @dtaylor113

@spadgett just to be clear, there are no required console changes for this PR. Things should still function as they currently do. Once this PR is merged, we can proceed with the console changes necessary to order items in the context of a project.


Removal of 'registry_address' in SCF broke registry connection

https://github.com/simonsobs/ocs/pull/327 removed registry_address from the SCF. This breaks how we were determining the registry address for creating an OCSClient, resulting in:
Traceback (most recent call last):
  File "<string>", line 4, in <module>
  File "/usr/local/lib/python3.8/dist-packages/sorunlib/__init__.py", line 18, in initialize
    CLIENTS = create_clients(test_mode=test_mode)
  File "/usr/local/lib/python3.8/dist-packages/sorunlib/util.py", line 115, in create_clients
    acu_id = _find_active_instances('ACUAgent', config=config)
  File "/usr/local/lib/python3.8/dist-packages/sorunlib/util.py", line 75, in _find_active_instances
    reg_client = OCSClient(cfg.hub.data['registry_address'])
KeyError: 'registry_address'

We need a new way to configure the registry instance-id, or consider whether it should have been removed from the SCF.

@jlashner would be curious to hear your thoughts on this.

I propose we just usually call it "registry", and have codes like yours try "registry" by default.  In cases where it isn't registry, override it explicitly (as a setting, which you will never use, for sorunlib).  That is what I did for ocs-client-cli, the only consumer of registry_address before I nuked it.

I propose we just usually call it "registry", and have codes like yours try "registry" by default. In cases where it isn't registry, override it explicitly (as a setting, which you will never use, for sorunlib). That is what I did for ocs-client-cli, the only consumer of registry_address before I nuked it.

Sorry, "it" is the old "registry_address" config? Or the Agent instance-id? Simply "registry" violates our site agent instance-id naming conventions.

I thought we were going to have some sort of sorunlib configuration, any chance we can set the instance-id there?. Yeah, I'm not a huge fan of having the same instance-id for all networks... I wasn't following too closely, but was there a reason for removing the registry address from the site-config, or was it just to remove clutter?

I think I was just decluttering.
The number of potential users here is so low, I think it should just be a parameter the user needs to dial in.  I also don't think it would be a huge stretch to carve out exceptions in the site agent naming conventions for critical singletons such as the registry.

Ah I forgot that the platform was already built in as site name. yeah I'm fine with it just being called registry instead of like daq-registry.

Resolved by #103. We no longer look in the SCF for the registry address, instead grab it from the new sorunlib config file.


Update cosmiconfig, iopipe-js-config, and iopipe-js-trace

Updating dependencies and addressing a change to cosmiconfig's API (now .search() instead of .load()).  The synchronous search presents the same interface as the old load.
Updating cosmiconfig turns out to work fine...but updating iopipe-js-config and iopipe-js-trace results in an error.

Getting a little further: there are errors in the extendConfig, packageJsonConfig, and rcFileConfig tests, thrown in the pre:invoke hook.  (TypeError: _this3.originalCallback is not a function).

I'm narrowing this down to a breaking change introduced in iopipe-js-config v1.0.0.  We've been requiring v0.3.0 in -core up until now, which seems to be why this hasn't been detected before.

From a git bisect, it looks like the first commit after 0.3.0 is where loading config files starts to break down in core.  (Or at least the tests no longer pass.)  That's the first commit that includes event-info in config, resolving the failures in these tests is going to be more of a process. (Also, event-info has its own set of severe vulnerabilities.)
So I'm hitting pause on this for right now and switching over to smoke testing hud.

Closing in anticipation of the sunsetting of IOpipe


change DependenceGroup to subclass set instead of list

It feels like it can't be this easy, but will see what CI has to say!

yes, I like this. Let's see what CI has to say

LGTM


Sponsors Page - Basic Layout

implements sponsors page #30  as best imagined to the spec:

@AkimaLunar Mind giving this a look 👀


Improve the doc soft deleting examples

// for the English and Chines version
$grid->filter(function ($filter) {
// Range filter, call the model's onlyTrashed method to query the soft deleted data.
$filter->scope('trashed', 'Recycle Bin')->onlyTrashed();
});
//for the English version
if ('trashed' == \request('_scope_') ) {
$actions->add(new Restore());
}
the position.
How to modify official doc?

I found the documentation in vendor/encore/laravel-admin/docs/en.


Set custom VID and PID for elasticNode. This way we can use normal librarys to detect the elastic node via usb-vendor.

Modify the TinyUSB integration of the enV5 implementation so that the USBD_VID  and USBD_PID are overwritten with our own for the USB Debug Connection during runtime.
TODO

[ ] Include Own TinyUSB Header
[ ] Set VID/PID fields accordingly to (VID=0x1209, PID=0x4C48)

Additional Documentation

raspberrypi/usb-pid
RaspberryPi Forum
raspberrypi/pico-sdk

@telefonjoker100 it would be nice if you can provide a little background, why we should need a feature like that.

The name of the devices changes often and is different for the operating systems: mac, linux und windows. With the PID and VID it is possible to detect the device on each operating systems and connect to it. In addition, it changes also on my mac from time to time and it is annoying to always change the serial port, because it has a new name now.

VID: 0x1209
PID: 4C48

VID and PID got now accepted.

The VID for RaspberryPi Products is set to 0x2E8A. For the bootsel stage (mass storage to upload uf2) is set to PID 0x0003. For the serial CDC connection (serial print output) the PID is set to 0x000A.
The PID for the bootsel stage is set during production and embedded into the pico bootloader stage 1 inside the RP2040 ROM. It can't be changed afterwards and is there fixed for ever.
The VID and PID for the serial CDC connection can be changed by modifying the TinyUSB library and can therefore be changed by us. (see edited task description)

This python function works at least on MacOS:
def get_pico_port():
    for c in list_ports.comports():
        if c.vid == 0x2e8a and c.pid == 0x000A:
            return c.device


Style/ParallelAssignment: swap detection for ActiveRecord

Summary: The Style/ParallelAssignment should ignore self. on the left when detecting variable swap
Actual behavior:
I have a code like
self.width, self.height = self.height, self.width

It says:
C: Redundant self detected.
        self.width, self.height = self.height, self.width
                                  ^^^^^^^^^^^
C: Redundant self detected.
        self.width, self.height = self.height, self.width
                                               ^^^^^^^^^^

That is expected, ok. I fix it
self.width, self.height = height, width

Now I get
C: Do not use parallel assignment.
        self.width, self.height = height, width
        ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^

Expected behavior: No Style/ParallelAssignment vioaltions should be found in the line
I guess the cop should treat vars on the left with self. like vars on the right without self.
$ rubocop -V
0.41.1 (using Parser <IP_ADDRESS>, running on ruby 2.3.1 x86_64-linux)

Makes sense.


(1) A minor request to improve the log (2) A minor bug in one log message

unbound version is 1.12.0

notice: init module 1: subnet
Having a config-module: "subnet ..."
results in an error. The correct configuration is, probably,
config-module: "subnetcache ..."
Is that correct?
With config-module: "subnetcache ..."
the log have:

unbound[3228]: [3228:0] notice: init module 1: subnet

I think the name in the log should match exactly the name in
unbound -V, and in the configuration.

See the list of linked modules with unbound -h
Sometimes, when there is a reference to a module that is not present, the comment in the log is:

... This module is not present (not compiled in), See the list of linked modules with unbound -h

Was that meant to be unbound -V?

Thanks for reporting!


Release Naming Conventions?

I think we should name each release after a type of fish/aquatic creature with an  alphabetical order similar to ubuntu releases.
Thoughts?

Awesome idea!
Also, the release name/version should be included in the simulation run results downloaded by experimenters.
So, A: Anemone?

Ardent Anemone perhaps?
Though hm it seems they're technically 'sea anemones' but I'm fine with using it still personally.

What about Ballsy Barracuda for the release after? :-)


Consider supporting X-Trace-Token from Kamon

See
http://kamon.io/integrations/web-and-http-toolkits/base-functionality/
We could create a plugin/setting that, if no TPIC is present, parses a X-Trace-Token value as TPIC.ID and consequently creating a X-Trace-Token as HTTP-Request header for downstream requests if configured.

Due several implementation of headers in the third party libraries and frameworks we've to adjust all bindings to implement such requirement. Another solution could be to think about an abstraction of header reading/writing to create some kind of a plugin mechanism.
I like to idea to add more than only one reading/writing strategy but the complexity of the code increases significant by wrapping those things.

I think this is related to #170


Feature request: Allow for "static" and "dynamic" post login redirect like @kinde-oss/kinde-auth-nextjs SDK

Prerequisites

[X] I have searched the repository’s issues, Kinde community and public roadmap to ensure my feature request isn’t a duplicate
[X] I have read the contributing guidelines
[X] I agree to the terms within the code of conduct

What is the problem you’re trying to solve?
Support post login redirect in the same manner as the @kinde-oss/kinde-auth-nextjs SDK does, see :point_right: this for more details.

All changes are merged 👍


Add FileMaker ODBC Client Driver 14.0.10

This recipe installs the FileMaker ODBC client driver.  There's no application installed with this driver.

It’s missing uninstall. There are scripts and documentation geared for that.
Also, style is wrong. To solve common stylistic issues, run brew cask style --fix {{your_cask_file}} prior to submission.

Should I rebase?

Should I rebase (squash)?

Yes, please (fixup).

It appears that brew cask style {{your_cask_file}} gives slightly different results from brew cask style {{your_cask_file}} --fix.

If you’re seeing an inconsistency, please document it (just the original cask, and the output from both commands), and submit the issue in the rubocop-cask repo.

37c0af2 should be nice and squishy now

Thank you.


Increase the min width of root

So that for days with longer letters like "Wednesday" it does not wrap.

Resolved in #219


Search TextFields showing just 5 results in list

Search TextFields showing just 5 results in list but when a select one of them it will display accurate result from array on selected index... just showing only 5

Hi @usman1057, can you please give me more details?, the piece of code you are writing, the desired result and the actual one?. Thanks!

check in your code this:
// Set the max number of results. By default it's not limited
mySearchTextField.maxNumberOfResults = 5


Add the ability to provide a default ZuulFallbackProvider

Fixes #1506

Codecov Report

Merging #1694 into master will increase coverage by 0.01%.
The diff coverage is 88.88%.

@@            Coverage Diff             @@
##           master    #1694      +/-   ##
==========================================
+ Coverage   75.08%   75.09%   +0.01%
==========================================
  Files         197      197
  Lines        6132     6139       +7
  Branches      767      769       +2
==========================================
+ Hits         4604     4610       +6
  Misses       1196     1196
- Partials      332      333       +1

Impacted Files
Coverage Δ

...rs/route/support/AbstractRibbonCommandFactory.java
92.85% <88.88%> (-7.15%)
:x:

Continue to review full report at Codecov.

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


Numpy install required before scipy dep. can be built

For some reason, scipy doesn't express a dependency on numpy -- yet it will not install without numpy being installed first.
Either document that this is required or figured out how to upstream the fix to scipy (and close this as invalid).

Seems to be working now.


github-merge: warn if ack message contains @

for cases like https://github.com/bitcoin/bitcoin/pull/17925#issuecomment-579946380.

Why not use a regex to check if a username follows the @?
Maybe some people use a non tagging @ like
Hello @ jonatack

Maybe try r"@[\s]{1,}"

concept ACK, thanks for working on this

Why not use a regex to check if a username follows the @?

NACK. That's completely unnecessary for what this script needs to achieve.

One sec: the use of message as both an argument and local var seems to have disabled the message check.

The use of message as both an argument and local var seems to have disabled the message check. Depending on how this script is being used, the added check might not be needed.

Can squash after feedback on what is preferred.

Needs rebase after #53.

The use of message as both an argument and local var in print_merge_details() seems to mess with the @-in-message check at the end. Depending on how this script is being used, once fixed, the additional check might not be needed.

Rebased. The bugfix in this PR was added to #53. Unsure if this PR is still useful.


Limit GtkSourceView width

When running Norka in a maximized window, the GtkSourceView takes the entire width of the window, thus making the lines of text look lengthy. This makes them difficult to read.
Is it possible to center the GtkSourceView and limit its width? We might need to decide how exactly to limit:

by character count per line
percentage of screen size
raw pixels

I'm not sure which one is the simplest to implement, but considering that Gtk.SourceView is a subclass of Gtk.Widget, raw pixels shouldn't be hard.

@dar5hak  that's a good point and one of the next features. My thoughts stick around raw pixels maybe a 1000 width, but I need some time to decide, 'cause my monitor more than 3k wide :smiley:  Feel free to share your ideas.

@amka if it's 3k wide, does the OS use pixel doubling? In that case, 1000px would still be fine because it would occupy 2000px. You could test it and see what happens.

@dar5hak no, it's not doubled, just a regular ultra-wide display :) Anyway, will test 1k width for its usability and will let you know.

On my ultrawide display, it looks ok for me :)

This looks so beautiful! ✨
Thank you! Feel free to close this issue when you're satisfied with the implementation.

Goes to next release.


JS/XSS Injection via execCommand in PasteUtils

Vulnerable code
export class PasteUtils {
  public static sanitizePastedTextContent(event: ClipboardEvent) {
    event.preventDefault();
    const text = event.clipboardData?.getData('text/plain');
    document.execCommand?.('insertHTML', false, text);
  }
}

Description
A potential security vulnerability exists in the PasteUtils class, specifically in the sanitizePastedTextContent method. The current implementation uses document.execCommand('insertHTML', false, text), which can allow JavaScript injection through pasted content.
Steps to Reproduce

Copy a string that includes JavaScript code.
Paste it into the relevant field where PasteUtils.sanitizePastedTextContent is used.
The JavaScript code gets executed.

Expected Behavior
Pasted text should be sanitized to prevent any potential JavaScript execution.
Actual Behavior
JavaScript can be executed through the execCommand('insertHTML', ...) method, leading to potential security risks.
Suggested Fix
Replace document.execCommand('insertHTML', false, text) with document.execCommand('insertText', false, text). This change will insert the text as plain text, mitigating the risk of JavaScript execution.

I have included this change in the following commit. This should hopefully be included in the main release tomorrow. Thanks!

This has now been released in version 1.4.7. Thankyou!


Implement values for hubble-relay to properly control sub chart values

Implements hubble-relay values to properly control sub chart values using the parent chart's values file.
Documents values in the parent chart for overriding.
Fixes: #11755
Signed-off-by: Sean Winn<EMAIL_ADDRESS>

Coverage increased (+0.008%) to 36.901% when pulling eebfa69c41175d85909feb740a42598a9fd9f17d on seanmwinn:cilium-11755 into 7561decf6ea57c26c22c51efc32989f138c6714a on cilium:master.

@sayboras Thanks for your feedback. We are trying to focus more on your first point which indicates this will make the charts function in a manner that allows for proper control of sub-charts. The long-term goal is to minimize the use of global values and build sub-charts that are configurable, tunable, and expose the knobs that operators are used to having available such as update strategy, tolerations, annotation, affinity, and others. Today, the parent chart's values file holds little more than configmap values and is not implemented in a manner that I would consider it a production-grade chart.

Thanks for your feedback. We are trying to focus more on your first point which indicates this will make the charts function in a manner that allows for proper control of sub-charts. The long-term goal is to minimize the use of global values and build sub-charts that are configurable, tunable, and expose the knobs that operators are used to having available such as update strategy, tolerations, annotation, affinity, and others. Today, the parent chart's values file holds little more than configmap values and is not implemented in a manner that meets basic best practices.

@seanmwinn making sense to me :100:, thanks for your explaination :tada:

@seanmwinn there is another PR that's related to this: https://github.com/cilium/cilium/pull/11784
if we are including the hubble binary in the cilium docker image, does it mean we can simply get rid of the hubble-cli subchart? let's sync up with @Rolinh .

@seanmwinn look like the recent update pulled in a lot of unrelated changes?

test-me-please

test-me-please


Sync of progress on multiple devices

Logging in from another device that has been used before does not take one to one's most recent challenge but the challenge the person was on the particular device one is using to log in; I think this behaviour should be changed even if it is not a bug.

Browser Information

Browser Name, Version:  Chrome, 52.0.2743.116 m
Operating System: Windows 10
Mobile, Desktop, or Tablet: Desktop

Hey @oluwaseyiaduloju,
We do not store partial solutions in an effort to reduce server load. They are stored using localStorage.
Happy Coding

Hi Berkeley,
Thanks for the reply, hope this meets you well.
If what you mean by partial solutions is "partial solutions to a
challenge", then I think I was misunderstood, I actually meant a scenario
like this:
Say I log in on a PC then do challenge 1-9, then I log in on another and do
challenge 10 -15, by the time I get to the former PC it would not continue
from 16 rather 10
Do you get?
Cheers
Oluwaseyi

This is expected behavior. Currently this information is stored locally. This will change with the new version of the site to be released shortly.
Happy Coding

Thanks. I've started the challenges in my laptop and when logging in on my desktop I can see the numbers of projects, but not the page where I actually stopped. :( Do you know when new version will be available?

Two years later and I'm getting the same behaviour...as this been solved? Is there a "manual-sync" I can use?

I found out that it will sync if you log out and immediately log in again, cheers!

I think this issue should be reopened since it obviously was not fixed.

This is expected behavior. Currently this information is stored locally. This will change with the new version of the site to be released shortly.
Happy Coding

It should be made clear if or when the version will be released.


GATK 4.0

Hi,
What are your plans for supporting GATK 4.0?
Thanks,
Ted

Hello Ted,
Thanks for posting the very first DNAscan issue.
It is on top of my todo list.
Cheers
Alfredo


I can't get a 3D model

I've been experiencing an issue while trying to generate a 3D model following the steps in the sample_image_to_3d.ipynb notebook. There are no reported errors during the process, but the progress remains perpetually at 0%.
The program appears to be running, but the progress never increases from 0%. No error messages have been thrown.
I would greatly appreciate your guidance on why this might be happening and any potential solutions. Thank you in advance for your time and assistance.
output like this:
PS D:\shap-e\shap_e\examples> python.exe .\image23d.py
0%|                                                                                | 0/64 [01:16<?, ?it/s]

Same here

Here is the fix: #64

Thanks.The problem is fixed.


Cannot send custom value in content-length when sending empty body

HEAD requests requires that the response should be the same as GET request on the same path, but without any body [ref]. Currently it is impossible to send custom content-length value in that header in responses other than 204 (#1153) while it should be possible in some cases.

Wow. It seems to work, however question is how to handle that when the budy computation is expensive, but getting size itself is quite cheap. For example sending a file that need to be fetched from slow device.

If you use sendfile Cowboy will not fetch the file. If you need another solution, I'd say stream_reply with content-length + one final empty stream_body (that will be ignored because the method is HEAD) ought to work.

Note that the final stream_body is not necessary if the handler exits immediately.

Ok, thanks.

I encounter the same issue. Just to give you a bit of context, I use Plug.Conn with the default cowboy adapter and when trying to respond to a HEAD request with a custom content-length header and an empty body, I get a content-length of 0 instead of my default value.
@essen I'm not sure I get your suggested work around above.

Just behave the same as GET and the content-length will be set accordingly.

I have tried to override the plug conn method from HEAD to GET but with the same results.
Some code I have tried:
conn
|> Plug.Conn.put_resp_header("content-length", "12")
|> Plug.Conn.send_resp(200, "")

Output with curl:
$ curl -i -I "http://localhost:8080/hello-world.txt"
HTTP/1.1 200 OK
cache-control: max-age=0, private, must-revalidate
content-length: 0
date: Mon, 23 Mar 2020 16:34:09 GMT
server: Cowboy

And same thing when overriding the method:
%{conn | method: "GET"}
|> Plug.Conn.put_resp_header("content-length", "12")
|> Plug.Conn.send_resp(200, "")

Any suggestions would be greatly appreciated.
Thanks 😃

I don't know Plug but if you send a response with no or empty body then the content-length will be set to 0. Cowboy needs to know what body you would have sent in order to calculate the length, and when sending a full response in a single call Cowboy will set the content-length itself and it cannot be overriden because it is important for framing.
If you need to send a file, as mentioned above, you should proceed as normal and pass the sendfile tuple in the response as Cowboy will not read the file when responding to HEAD requests. It will only get stats.
The alternative is to use cowboy_req:stream_reply and provide the content-length manually. Here it's allowed but note that you will need to disable chunked transfer-encoding for HTTP/1.1 (you can disable it dynamically for a single response).

Thanks for the quick answer @essen!
Isn't this behavior violating the RFC when the HTTP request is a HEAD? Since it's used for obtaining metadata about the resource without transferring actual resource data over the body.
Reading the RFC, I was under the impression that a HEAD request, provided with a content-length should use it and not override it to 0 since, by definition, the body will be empty for such HTTP method.
I would agree though that content-length should be set to equal the body size for a GET request.
As for the answer about the file, I'm not dealing with a file at all, simply proxying a response from a downstream HTTP service (AWS S3) which sets the content-length to some value and respond with an empty body.

I gave a try to cowboy_req:stream_reply which is the implementation behind Plug.Conn.send_chunked/2.
I get my custom content-length value! 🎆
Can you just elaborate a bit more about my need to "disable chunked transfer-encoding for HTTP1/1"? I'm not sure what this is about.
Thank you for your time.

Isn't this behavior violating the RFC when the HTTP request is a HEAD? Since it's used for obtaining metadata about the resource without transferring actual resource data over the body.

You answer your own question here. without transferring actual resource data. Cowboy doesn't transfer that data. But Cowboy still needs to know about it in order to compute the response.

Reading the RFC, I was under the impression that a HEAD request, provided with a content-length should use it and not override it to 0 since, by definition, the body will be empty for such HTTP method.

Well you're using Cowboy, not the RFC. This is purely an API concern. When you do cowboy_req:reply Cowboy sets the content-length. It's not about HEAD or GET, it applies to all methods.

When using HTTP/1.1, the Cowboy adapter will stream the response instead of emitting chunks if the content-length header has been set before calling send_chunked/2.

I forgot about that, so ignore my comment about chunked.

Thanks again for your quick answer!
I think I can get around by using cowboy_req:stream_reply then.


size mismatch for centroids: copying a param with shape torch.Size([512, 3]) from checkpoint, the shape in current model is torch.Size([16, 3]).

After compute centroids
!python src/compute_centroids.py --dataset cifar10 --num_clusters=16
I tried to train with this command
!python src/run.py --dataset cifar-10-batches-py train configs/s_clf.yml --pretrained=models/cifar10_gen.ckpt
Also tried to generate samples with this command
!python src/sample.py models/cifar10_gen.ckpt
but got this error:-
Traceback (most recent call last):
File "src/run.py", line 91, in
args.func(args)
File "src/run.py", line 21, in train
model = ImageGPT.load_from_checkpoint(args.pretrained)
File "/usr/local/lib/python3.6/dist-packages/pytorch_lightning/core/saving.py", line 154, in load_from_checkpoint
model = cls._load_model_state(checkpoint, strict=strict, **kwargs)
File "/usr/local/lib/python3.6/dist-packages/pytorch_lightning/core/saving.py", line 200, in _load_model_state
model.load_state_dict(checkpoint['state_dict'], strict=strict)
File "/usr/local/lib/python3.6/dist-packages/torch/nn/modules/module.py", line 1045, in load_state_dict
self.class.name, "\n\t".join(error_msgs)))
RuntimeError: Error(s) in loading state_dict for ImageGPT:
size mismatch for centroids: copying a param with shape torch.Size([512, 3]) from checkpoint, the shape in current model is torch.Size([16, 3]).

Hi! You should not need to regenerate the centroids for sampling as I supply the ones used for training in the data directory. That being said, iGPT-S (which is the config I used for CIFAR10) uses 512 clusters - not 16, so if you would like to retrain the model and create new clusters you should specify--num_clusters=256. Hope this helps :)


Support for nested multi-step and / or multi-tabs or similar

I'm submitting a ...  (check one with "x")
[ ] bug report => search github for a similar issue or PR before submitting
[ X] feature request
[ ] support request

**Current behavior**
There are example for Multi_step and Tabs Forms but no example for nested levels forms navigation like a combination of multi-step and tabs or even better with multi-level side bar menu.

**Expected behavior**
Nested level form model with nested level form navigation menu (side bar, multi-step or tabs, etc).
Could Steps and Tabs or similar multi-level form navigation be done with Field Type of Wrapper so they can be part of Field Config?

**Minimal reproduction of the problem with instructions**
https://stackblitz.com/angular/gxalnabmlmx?file=src%2Fapp%2Fapp.component.ts
https://stackblitz.com/angular/roxvbmejray?file=src%2Fapp%2Fapp.component.ts

**What is the motivation / use case for changing the behavior?**
Need a multi-level form navigation menu for a multi-level form model.  Here is an example of form model (only 2 levels for simplicity but could have more levels)

    {
        "root": {
            "category1": { //<!-- First level navigation -->
                "UserName": "Test1",
                "Address": "110 My street",
                "CreateDate": "2018-09-14T00:00:00"
            },
            "category2": { //<!-- First level navigation -->
                "section1": { //<!-- Second level navigation -->
                    "check1": "true",
                    "check2": "false",
                    "commentlist": {
                        "row": [{
                                "comment": "1.1"
                            },
                            {
                                "comment": "1.2"
                            }
                        ]
                    },
                    "photolist": {
                        "row": [{
                                "Photo1": "oqpNlRSsfwqqwqwq",
                                "caption1": "Flower",
                                "Photo2": "AnU5HPOP89z9aKKKGI//2Q==",
                                "caption2": "Bird"
                            },
                            {
                                "Photo1": "asasaassasas",
                                "caption1": "Flower",
                                "Photo2": "AnU5HPOP89z9aKKKGI//2Q==",
                                "caption2": "Bird"
                            }
                        ]
                    }
                }
            },
            "category3": { //<!-- First level navigation -->
                "section1": { //<!-- Second level navigation -->
                    //<!-- fields -->
                },
                "section2": { //<!-- Second level navigation -->
                    //<!-- fields -->
                },
                "section3": { //<!-- Second level navigation -->
                    //<!-- fields -->
                }
            }
        }
    }

**Please tell us about your environment:**
Windows, npm, VS code

* **Angular version:** 2.0.X
6.0

* **Browser:** [all | Chrome XX | Firefox XX | IE XX | Safari XX | Mobile Chrome XX | Android X.X Web Browser | iOS XX Safari | iOS XX UIWebView | iOS XX WKWebView ]
<!-- All browsers where this could be reproduced -->

* **Language:** [all | TypeScript X.X | ES6/7 | ES5]
ES6/7
* **Node (for AoT issues):** `node --version` =

providing a such example would require much time and I think that's something you should/can deal with it based on what we've already provided.

I found an example with nested step/tabs form.  However, it is still saved to a flat model.  How do I saved it as a hierarchy model just like the multi-level structure of the form?
https://stackblitz.com/edit/angular-ngx-formly-gq7bmn?file=app%2Fapp.component.ts
I want the model to look like this:
{
    "root": {
        "category1": { //<!-- First level navigation -->
            "UserName": "Test1",
            "Address": "110 My street",
            "CreateDate": "2018-09-14T00:00:00"
        },
        "category2": { //<!-- First level navigation -->
            "section1": { //<!-- Second level navigation -->
                "check1": "true",
                "check2": "false",
                "commentlist": {
                    "row": [{
                            "comment": "1.1"
                        },
                        {
                            "comment": "1.2"
                        }
                    ]
                },
                "photolist": {
                    "row": [{
                            "Photo1": "oqpNlRSsfwqqwqwq",
                            "caption1": "Flower",
                            "Photo2": "AnU5HPOP89z9aKKKGI//2Q==",
                            "caption2": "Bird"
                        },
                        {
                            "Photo1": "asasaassasas",
                            "caption1": "Flower",
                            "Photo2": "AnU5HPOP89z9aKKKGI//2Q==",
                            "caption2": "Bird"
                        }
                    ]
                }
            }
        },
        "category3": { //<!-- First level navigation -->
            "section1": { //<!-- Second level navigation -->
                //<!-- fields -->
            },
            "section2": { //<!-- Second level navigation -->
                //<!-- fields -->
            },
            "section3": { //<!-- Second level navigation -->
                //<!-- fields -->
            }
        }
    }
}

use dotted key something like  key: 'root.category1.UserName' or using fieldGroup https://formly-js.github.io/ngx-formly/examples/other/nested-formly-forms

Thank you so much.  I was not aware of dotted key.


dll files not recognized

Hi,
When importing the package, I keep receiving the error that the dll file is not found. I tried several Python version but can't find the root of the problem. Here is a screen shot attached that very clearly shows the files:

How to debug this?
Thanks a lot for your help,
Jasper

Does it work if you put the files inside the library directory? That's what the library module asks for:
https://github.com/Laukei/pyanc350/blob/master/pyanc350/v4/PyANC350v4.py#L5-L8
Regrettably I don't work with nanopositioners any more so I can't refer to my current setup to find out how I did it!


Implement "chat" button

The button must:

Redirect the user to the meetup chatroom.

The button should also:

Create the chatroom if it doesn't exist.
Abolish the chatroom after some time.

Apparently, the feature is not available on Facebook yet: https://techcrunch.com/2016/11/11/facebook-messenger-launches-its-public-group-chat-feature-rooms-in-select-markets/


RELATED: RAIL-3906 Upgrade GoodData.UI to 8.8.0

JIRA: RAIL-3906

Build succeeded (check pipeline).

https://checklist.intgdc.com/job/client-libs/job/gooddata-create-gooddata-react-app-unit-tests-zuul-docker-el7/318/ : SUCCESS in 46s
https://checklist.intgdc.com/job/compliance/job/gooddata-create-gooddata-react-app-oss-check/256/ : SUCCESS in 6s

Build succeeded (gate pipeline).

https://checklist.intgdc.com/job/compliance/job/gooddata-create-gooddata-react-app-merge-compliance-check/128/ : SUCCESS in 27s
https://checklist.intgdc.com/job/client-libs/job/gooddata-create-gooddata-react-app-unit-tests-zuul-docker-el7/319/ : SUCCESS in 55s
https://checklist.intgdc.com/job/compliance/job/gooddata-create-gooddata-react-app-fossa-scanning-zuul-el7/93/ : FAILURE in 15m 20s (non-voting)


Numerically safe cross_entropy for tfd.Bernoulli?

System information

Have I written custom code (as opposed to using a stock example script provided in TensorFlow): Yes
OS Platform and Distribution (e.g., Linux Ubuntu 16.04): MacOS
TensorFlow installed from (source or binary): Binary
TensorFlow version (use command below): 1.7
Python version: 3.5
Bazel version (if compiling from source):
GCC/Compiler version (if compiling from source):
CUDA/cuDNN version: N/A
GPU model and memory: N/A
Exact command to reproduce:

import tensorflow as tf
from tensorflow import distributions as tfd

tf.enable_eager_execution()

def test_bernoulli_cross_entropy():
    logits = tf.random_normal(shape=[3, 4])
    labels = tf.cast(tf.random_normal(shape=[3, 4]) > 0.0, tf.float32)
    p = tfd.Bernoulli(logits=logits)
    q = tfd.Bernoulli(probs=labels)

    ce1 = tf.nn.sigmoid_cross_entropy_with_logits(logits=p.logits, labels=q.probs)
    ce2 = q.cross_entropy(p)
    print(ce1)
    print(ce2)

You can collect some of this information using our environment capture script:
https://github.com/tensorflow/tensorflow/tree/master/tools/tf_env_collect.sh
You can obtain the TensorFlow version with
python -c "import tensorflow as tf; print(tf.GIT_VERSION, tf.VERSION)"
v1.7.0-3-g024aecf414 1.7.0
Describe the problem
The Bernoulli in Tensorflow distributions does not seem to have a numerically safe cross entropy operation. See code above -- it should produce the same results as tf.nn.sigmoid_cross_entropy_with_logits, but produces NaN. If I set probs to be not exactly zero or one, then ce1 = ce2. So it seems to be a numerical stability issue.
Example output:
tf.Tensor(
[[0.88931024 1.4332782  0.47688866 0.48566538]
 [0.93064773 0.39325503 1.1308188  1.5703993 ]
 [0.09037975 0.1828385  0.39422417 0.839576  ]], shape=(3, 4), dtype=float32)
tf.Tensor(
[[nan nan nan nan]
 [nan nan nan nan]
 [nan nan nan nan]], shape=(3, 4), dtype=float32)

Source code / logs
import tensorflow as tf
from tensorflow import distributions as tfd

tf.enable_eager_execution()

def test_bernoulli_cross_entropy():
    logits = tf.random_normal(shape=[3, 4])
    labels = tf.cast(tf.random_normal(shape=[3, 4]) > 0.0, tf.float32)
    p = tfd.Bernoulli(logits=logits)
    q = tfd.Bernoulli(probs=labels)

    ce1 = tf.nn.sigmoid_cross_entropy_with_logits(logits=p.logits, labels=q.probs)
    ce2 = q.cross_entropy(p)
    print(ce1)
    print(ce2)

Nagging Assignee @tatatodd: It has been 15 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Nagging Assignee @tatatodd: It has been 14 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Nagging Assignee @jvdillon: It has been 16 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Nagging Assignee @jvdillon: It has been 31 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Nagging Assignee @jvdillon: It has been 46 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Nagging Assignee @jvdillon: It has been 61 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.


Ephemeral Messages

Midjourney bot send ephemeral (private) messages  and when the images are ready - the script cannot receive this message in the chat

to receive images in a private chat, it must be configured to use Midjourney.
It can only work with a paid plan.
just do /imagine in this private Discord chat to see if that's the case.
Once this first step has been validated, all you have to do is right-click on this channel to retrieve its ID.
This package is designed to also work with private channels

I found the problem is shadow answers with forbidden words in the query.
https://drive.google.com/file/d/1TaL5JlDuSwlYpbAAxQHax8pCc7hWBGqu/view?usp=drivesdk


mtl: Temporarily remove misaligned fill buffer test

Simply removes the misaligned fill buffer test until:

we determine how to handle this on Metal (misaligned copyFromBuffer)
we get clarification on whether this Vulkan restriction will be lifted (i.e. we know of misbehaving drivers already which appear to perform partial writes for misaligned buffers lengths)

PR checklist:

[X] make succeeds (on *nix)
[X] make reftests succeeds
[ ] tested examples with the following backends:

Based on the changes in #2015 we shouldn't need to disable this anymore - closing


Fix Error State Issue when Kafka Cluster is Offline

Implemented a fix to account for cluster state validation prior to starting/restarting.
Also removed libraries that are no longer needed. For situation where a cluster/broker goes down when running, the transport will log an error but remain in the running and will be able to resume operation once the cluster comes back online.

@mrasante what is the difference between this modified gallery item transport and the OOTB 10.7.1 one?

@mrasante Please move the SDK dependency to the main pom file.


Support graphql imports.

When you are using the graphql-tag/loader plugin for webpack you can use graphql imports to pull fragments from other files.
#import "./MyFragment.graphql"

query test {
  item {
     ...MyFragment
  }
}

Hovering over the fragment logs the following to the GraphQL Language Server log:
"Definition not found for GraphQL fragment MyFragment"

@corycook Thanks for reporting this issue. Currently, we support this by defining the dependencies in includes field of graphql config. Please see the usage section.

Okay, I thought that was for graphql schema definition files I didn't think it would be for fragment definitions as well.
How does this handle multiple fragments with the same name? I don't have any, just curious.

It looks like this functionality has since been added this can probably be closed.


Added component metadata YAML for Azure Blob Storage state store.

Description
Added component metadata YAML for Azure Blob Storage state store.
Issue reference
We strive to have all PR being opened based on an issue, where the problem or feature have been discussed prior to implementation.
Please reference the issue this PR will close: #2769
Checklist
Please make sure you've completed the relevant tasks for this PR, out of the following list:

[ ] Code compiles correctly
[ ] Created/updated tests
[ ] Extended the documentation / Created issue in the https://github.com/dapr/docs/ repo: dapr/docs#[issue number]

Also you are missing some metadata per CI/CD

"decodebase64": "bool",

I could not find  DecodeBase64 being used in the code for state.azure.blobstorage nor in the internal/component/azure/blobstorage files aside metadata parsing. :(
However, it is used for bindings.azure.blobstorage
I will add it to the metadata.yaml but with a remark this is parsed but not effective field.

"retrycount": "int32"

This is a deprecated field. I will document it as such.

Then just change the metadata and add the property only:"bindings".
https://github.com/dapr/components-contrib/blob/5dad8a867c62a6100a939093619b0d5bc620120e/internal/component/azure/blobstorage/metadata.go#L28
DecodeBase64        bool json:"decodeBase64,string" mapstructure:"decodeBase64" only:"bindings"`


AHOYAPPS-649-650 | Audio device tests

Contributing to Twilio

All third party contributors acknowledge that any contributions they provide will be made under the same open source license that the open source project is provided under.

[x] I acknowledge that all my contributions will be made under the project's license.

Pull Request Details
JIRA link(s):

AHOYAPPS-649
AHOYAPPS-650

Description
Implemented Audio Device Tests. See screenshots.
Screenshots

Recording

Playing recorded audio

Telling the user test is successful (not in Figma)

Example error after analyzing audio levels

Example error if user denied permission. Also when audio output speaker cannot be changed (Safari, Firefox)

Looks good on mobile! But I think that the highlighted div below could have some magin-bottom on it to space things out a little:

Not a big deal, but it looks like prettier didn't run again. I think it's fine for now. Maybe we can run it in its own PR later so it doesn't clutter up the diffs.

Not a big deal, but it looks like prettier didn't run again. I think it's fine for now. Maybe we can run it in its own PR later so it doesn't clutter up the diffs.

I'll run prettier and submit a separate PR once this PR is merged.

I addressed all feedback. Please take a look @timmydoza


classgraph 4.8.98 (was 4.8.90)

Updates io.github.classgraph:classgraph from 4.8.90 to 4.8.98.
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "io.github.classgraph", artifactId = "classgraph" } ]

labels: library-update, semver-patch

Superseded by #178.


Consider dropping all CURL* associated with the same connection

When a transfer fails with a libcurl error we avoid reusing the handle for future requests (see #7088).  However, if the application is using HTTP/2 then other CURL* may be using the same connection, it may be a good idea to stop using it.  It is unclear if it is possible to identify all handles that have this problem, so we need to do some research first.

Per https://everything.curl.dev/libcurl/connectionreuse, when using the easy interface, each easy handle has a connection cache that goes away when curl_easy_cleanup is called on the handle. When using the multi interface, the connection cache is associated with the multi handle and connections persist across calls to curl_easy_cleanup. Calling curl_multi_cleanup on the multi handle causes any connections in the connection cache to be closed.
Per #7088 (https://github.com/googleapis/google-cloud-cpp/pull/7088), when a transfer error occurs, both the easy handle and the multi handle are cleaned up. This should cause connection caches associated with either to be discarded.


Issues compiling in Debian

(root@h1)-(/root/source/STPeters-luafcgid-be06a44)# ls -al
total 176K
drwxr-xr-x 8 root root 4.0K 2012-02-17 20:28 .
drwxr-xr-x 3 root root 4.0K 2012-02-17 20:19 ..
-rwxr-xr-x 1 root root  477 2011-10-08 12:44 configure
drwxr-xr-x 2 root root 4.0K 2011-10-08 12:44 docs
drwxr-xr-x 2 root root 4.0K 2011-10-08 12:44 examples
-rw-r--r-- 1 root root   52 2011-10-08 12:44 .gitignore
-rw-r--r-- 1 root root 1.1K 2011-10-08 12:44 INSTALL.debian
-rw-r--r-- 1 root root  830 2011-10-08 12:44 INSTALL.freebsd
-rw-r--r-- 1 root root 1.4K 2011-10-08 12:44 LICENSE
-rw-r--r-- 1 root root 1.5K 2012-02-17 20:29 Makefile
-rw-r--r-- 1 root root 1.5K 2011-10-08 12:44 Makefile.debian
-rw-r--r-- 1 root root 1.5K 2011-10-08 12:44 Makefile.freebsd
-rw-r--r-- 1 root root  873 2011-10-08 12:44 Makefile.generic
drwxr-xr-x 2 root root 4.0K 2011-10-08 12:44 obj
-rw-r--r-- 1 root root 4.7K 2011-10-08 12:44 README.html
-rw-r--r-- 1 root root 3.9K 2011-10-08 12:44 README.mkd
-rw-r--r-- 1 root root  89K 2011-10-08 12:44 README.pdf
drwxr-xr-x 3 root root 4.0K 2011-10-08 12:44 scripts
drwxr-xr-x 2 root root 4.0K 2011-10-08 12:44 src
drwxr-xr-x 3 root root 4.0K 2011-10-08 12:44 tests
(root@h1)-(/root/source/STPeters-luafcgid-be06a44)# sh configure
Ident: Linux
Cleaning up...
Configuring for Debian...
Makefile.debian' -> Makefile'
(root@h1)-(/root/source/STPeters-luafcgid-be06a44)# make
make: *** No rule to make target main.c', needed by all'.  Stop.
(root@h1)-(/root/source/STPeters-luafcgid-be06a44)# cat /etc/debian_version
5.0.6
Any idea what is wrong?

I have never used actually never used linux, so likely broke the contributed debian Makefile while reorganizing the codebase into directories... if someone with expertise can fix it, that would be greatly appreciated.
I am thinking about switching back to a simple flat directory structure if this is not resolved soon.

I managed to compile it on Debian Lenny now.
You need to change into this:
LUAINC = $(PREFIX)/include/lua5.1
SOURCES = $(SRCDIR)/main.c $(SRCDIR)/config.c $(SRCDIR)/pool.c $(SRCDIR)/buffer.c $(SRCDIR)/request.c
Then it compiles without any problems.

The "make install" still does not work in Debian.
I managed to fix most of it by changing to:
install -b ./scripts/luafcgid.lua $(PACKAGEPATH)/luafcgid.lua
install -b ./scripts/etc/config.debian.lua $(CONFDIR)/config.lua
That will install everthing correctly except the "man" pages.

Also, you need to add this line in "make install" to make the init.d script install correctly:
install -b ./scripts/etc/init.d/luafcgid.debian /etc/init.d/luafcgid


pipedag-manage command to list all instances

It might be nice to have a command to list all instances, so the other commands can be for-loop automated to operate on a set of instances that can be either a pattern matched include list or a pattern matched exclude list. It would also be nice to have an option to receive instance_id/per-user combinations for which a metadata schema exists in the database. The per-user aspect does not need to be part of output tuples since tuples are harder to process with a pipe (not impossible though). It is also fine to retrieve the list twice, once for per-user and once for not per-user.
Alternatively, or additionally, we could include pattern matched include and exclude capabilities in the other commands and have them print the set of matched instance_id/per_user combinations before asking whether to continue.

The current confirm-message of clear-metadata is something like "Do you really want to clear all metadata?". We should repeat mentioning instance_id and username and ideally also the resulting schema name.

It might also be nice to show the metadata version of all instances mentioned in pipedag.yaml. This could be combined with the option to delete all metadata which is on an outdated version.


Proposal for v4: Expose tenant meta data in DocumentMetadata

I propose to expose the following meta data pertaining to multi-tenancy:

IsTenanted : If a document is tenanted, this value will be true
TenantId : if tenanted, this will contain the tenant id otherwise empty

If you look at https://github.com/JasperFx/marten/issues/1460 and https://github.com/JasperFx/marten/issues/1451, there is a request for an option to update/delete tenanted documents in a non-tenanted session. In absense of this, currently users are resorting to mechanisms such as using a marker interface containing TenantId and all the tenanted documents will implement it. When data is queried, immaterial of tenanted or non-tenanted session, TenantId for the document is available to use. It works but is quite cumbersome.
Also when querying multi-tenanted documents, there is no way to know which tenant each of the document belonged to.
This meta data will help the Marten internals as well to use it in a non-tenanted session to deal with say updates/deletes easily. Also updates/deletes in tenanted/non-tenanted sessions open up some issues in terms of enforcing data isolation/security at a tenant level from an consumer standpoint, the mitigation strategy would be to provide some configuration options which will allow or disallow querying/updating/deleting tenanted documents in non-tenanted session or vice-versa which can configured at a per document level or at the global level.
Please let me now your thoughts.

I'm merging this into #1337


Project creation form styles should match prototype

This is to get us closer to the prototype/wireframe. I don't know if we'll have a perfect match. Having a separate identity vs background section while keeping the data structure flat is one of these things that is a bit tricky to do in terms of code, so I don't know if we'll get around to that

Thank you for the clarification :) Perhaps this is something that can be discussed at the design session this week.


Update setup.py

Adding description to setup.py. This will be picked up by pypi so that it does not show project description as unknown

Merged into develop branch.
See https://github.com/Storj/dataserv-client/commit/ac6c8e6bac3684b44927a3868a1cf1ab1a1977d9


[Feature] Like editing permissions, designated users can view

Initial checklist

[X] Is this really a problem?
[X] I have searched the Github Issues for similar issues, but did not find anything.

Problem
Now it can only be set that others can view it, or others can't view it. I hope to better control the viewing permission.
Is there any way I can expand it?

We do not understand what you need, so we will close this issue for now. Once you have organized a precise description of your requirements, feel free to reopen this issue at any time.

I want to set the viewing permission for the designated user separately.
In the viewState, I have the following options:

othersCanView: Allows others to view the content.
noOneElseCanView: Restricts access to the content to only the designated user.

I believe a user with the designatedUserCanView permission is required to access permissions.
I would like to know why the cancel view is not designed in the same way as the edit view.

I can't reopen the issue.
@jikkai

@ybzky You're a good man.

In the next version we will support customization of this component

In the next version we will support customization of this component

good!

The document provides a custom way


⚠ Bofrost.fr has degraded performance

In b8b55cc, Bofrost.fr (https://www.bofrost.fr) experienced degraded performance:

HTTP code: 200
Response time: 1126 ms

Resolved: Bofrost.fr performance has improved in 36f5a07.


no fix even though it can simulate GPS satellites.

Hello. I have a problem about 'fix location'.
With bladeRF and gps-sdr-sim, it was successful to generate fake GPS signals as you can see below:

But, the problem is location couldn't be fixed on my phone.
I turned off 'improve accuracy' and locating method was 'phone only'.
I've been stucked with the issue for 2 weeks but I couldn't find the reason...
Any advice will be appriciated.

I spoofed gps on other devices such as fitness bands.
But, I couldn't find the reason why it failed to spoof my phone yet.
I think older version of android is better for gps spoofing. Mine is android 8.

I also find it harder to spoof phones as compared to other GPS devices. One possibility could be the time difference between the time gotten via GPS and the internet. Never did find the actual reason though.

what did you use for spoofing? I mean usrp or hackRF..I am using USRP x310 for transmission of signal and use my android phone. I have installed gps test app and using it to spoof the position.  But i coudnt. i got the signal but cant fix position yet. I set my phone as phone only. made airplane mode. and turn off wifi and mobile data......I hope the problem is on .bin file while generating. anyone who can help me on USRP??

Hello, Solan.
I used Blade-RF for spoofing.
I think the problem is not on .bin file becuase you could check the gps
signal on your smart phone.
In my case, it takes time to fix location. The first fixation after running
gps sdr sim took around one minute.
2018년 10월 22일 (월) 오후 6:27, solan<EMAIL_ADDRESS>작성:

what did you use for spoofing? I mean usrp or hackRF..I am using USRP x310
for transmission of signal and use my android phone. I have installed gps
test app and using it to spoof the position. But i coudnt. i got the signal
but cant fix position yet. I set my phone as phone only. made airplane
mode. and turn off wifi and mobile data......I hope the problem is on .bin
file while generating. anyone who can help me on USRP??
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/osqzss/gps-sdr-sim/issues/159#issuecomment-431776843,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ATgnEDWqulNnhI6jj5TH4mil2-udp705ks5unY-LgaJpZM4Vv1mB
.

hi,
i have generated GPS Signal for both static and Dynamic(user motion) coordinates with Pluto SDR, i am getting position fix for both cases with U-Blox Receiver.
but when i testing same scenario with mobile i am not getting even single satellite(no satellite C/N ratio bars are there)
can you brief me what should i do or follow for getting position fix with Mobile Phone
it would be great helpful to me

what did you use for spoofing? I mean usrp or hackRF..I am using USRP x310 for transmission of signal and use my android phone. I have installed gps test app and using it to spoof the position. But i coudnt. i got the signal but cant fix position yet. I set my phone as phone only. made airplane mode. and turn off wifi and mobile data......I hope the problem is on .bin file while generating. anyone who can help me on USRP??

im having the exact same issue im having all satellites signals bars but i can't fix in any one of them for some reason using x310 and android phone for receiving


Add test for node-version *

Description:
This PR only adds test coverage for existing behavior: It adds a unit and e2e test for node-version *.
What I did was:

Combine the following three tests with it.each(table) --- but I'm happy to leave them unrolled if you prefer?

https://github.com/actions/setup-node/blob/17f8bd926464a1afa4c6a11669539e9c1ba77048/__tests__/installer.test.ts#L138
https://github.com/actions/setup-node/blob/17f8bd926464a1afa4c6a11669539e9c1ba77048/__tests__/installer.test.ts#L153
https://github.com/actions/setup-node/blob/17f8bd926464a1afa4c6a11669539e9c1ba77048/__tests__/installer.test.ts#L168

Added a new row for testing *:it.each([
  ['12.16.2', 'darwin', '12.16.2', 'Erbium'],
  ['12', 'linux', '12.16.2', 'Erbium'],
  ['10', 'win32', '10.20.1', 'Dubnium'],
  ['*', 'linux', '14.0.0', 'Fermium'] // <- Added a new row
])(
  'can find %s from manifest on %s',
  async (versionSpec, platform, expectedVersion, expectedLts) => {

Added an e2e test for node-version *.

Check list:

[ ] Mark if documentation changes are required.
[x] Mark if tests were added or updated to cover the changes.

Hello @jablko. Thank you for your pull request. I think only unit tests will be enough.

[x] @dmitry-shibanov Sounds good, I've removed the e2e test.


[DRAFT] Example folder for WSSI workflow

Describe your changes
This is a draft PR to make the WSSI related code visible. We haven't decided yet if the code will be added to Olive examples or the WSSI team will keep it in their repositories.
Checklist before requesting a review

[ ] Add unit tests for this change.
[ ] Make sure all tests can pass.
[ ] Update documents if necessary.
[ ] Format your code by running pre-commit run --all-files
[ ] Is this a user-facing change? If yes, give a description of this change to be included in the release notes.

(Optional) Issue link

@devang-ml FYI


Added provider for Autopilot

What type of PR is this?
/kind feature

What this PR does / why we need it:
Adds support for Autopilot clusters.
Which issue(s) this PR fixes:

Fixes #
Special notes for your reviewer:
Does this PR introduce a user-facing change?:

Additional documentation e.g., KEPs (Kubernetes Enhancement Proposals), usage docs, etc.:

/assign @jprzychodzen

/lgtm


Use scheduler.get_last_lr() instead of manually searching for optimizers.param_groups

🐛 Bug / Feature Request
I am not sure whether to classify this issue as a feature request or bug, but in your codebase, you get the learning rate from schedulers as scheduler.optimizer.param_groups[0]['lr'].
However, PyTorch provides the get_last_lr() method for this.
I created my own scheduler, which allows me to combine multiple schedulers together (eg. burn-in LR for the first few steps, and then step down LR later during training).
As this scheduler holds multiple different schedulers itself, there is no scheduler.optimizer argument and thus I cannot use this with PyTorch Lightning.
I did however implement the get_last_lr() method, which gets the LR from the underlying schedulers, so if you could modify the PyTorch Lightning code to use that method, it should all work!
Problems
Obviously, nothing is ever as straight forward with the PyTorch scheduler API, which means that there might be problems with this solution. One small issue I found is that ReduceLROnPlateau is behaving differently and does not implement the get_last_lr() method. (The PyTorch team should really look into unifying their API, but that should be discussed there...)
If you look at GH-5364, which I made on top of the release/1.2-dev branch, you will see that for the ReduceLROnPlateau scheduler, I still use the old method of finding the LR through the scheduler.optimizer object.
However, for any other scheduler, we use the get_last_lr() method.
I could adapt the code to be even more defensive and check whether there is a get_last_lr() method and only use it if it is available, but I don't know whether that is necessary, as all other official schedulers have the same base Scheduler class which implements this method.

I send in a Draft PR with the necessary changes.
Kind Regards,
0phoff

The original reason why we get the value from scheduler.optimizer.param_groups[0]['lr'] instead of using get_last_lr() was that get_last_lr was first introduced in pytorch v.1.3 (i think) and at that point in time, our code were backward compatible to v.1.1. We could therefore not rely on get_last_lr when this was first implemented.
However, since v1.3 is minimum requirement now I see no problem with this change.

The original reason why we get the value from scheduler.optimizer.param_groups[0]['lr'] instead of using get_last_lr() was that get_last_lr was first introduced in pytorch v.1.3 (i think) and at that point in time, our code were backward compatible to v.1.1. We could therefore not rely on get_last_lr when this was first implemented.
However, since v1.3 is minimum requirement now I see no problem with this change.

Closing as we decided in the corresponding PR (#5364) to not support this for now.


Source operators prevent fast symbol resizing

Stochastic source ops / sample ops are useful, obviously. While it is possible to sample from random distributions outside of your network and provide the randomness as input, this is slower, more complicated, and makes the graph less self-contained.
However, there is a problem with using the source ops like 'normal' and 'uniform' as it stands, which is that they break an important property of an symbol/graph: resizability. It is highly desirable to be able to resize an existing symbol by simply doing shape inference using a different batch size of the relevant input tensors, rather than the alternative of generating a completely new symbol/graph each time you want to use a different batch size.
A very important application of fast resizing is when evaluating a net on a large set of user inputs. One obviously should break the input into batches that match the batch size one chose for the evaluation network, but there will be in general a 'leftover' part at the end, and to generate a whole new graph to evaluate that leftover part (or using junk data) will often be slower than the ideal thing, which is to fast resize to the length of the final piece, then evaluate.
There are others examples of the value of fast resizability, probably the most important one is for me is dealing with variable-length sequence networks when unrolling can be avoided (e.g. using the RNN op), where creating buckets quickly is again very desirable.
For a concrete example of how resizability can be broken by sample ops, let's take variational dropout (https://arxiv.org/abs/1512.05287). That's straightforward to implement by sampling from a bernoulli distribution and then doing a broadcast multiply onto the sequence tensor. The problem here is that the batch size needs to be hardcoded into the sample op's 'shape' parameter, because backward shape inference doesn't pass through broadcast layers (as broadcast is 'polymorphic': it has multiple input shapes that would work with the output shape).
And even if we make a variational dropout op or modify the existing dropout op to be able to do broadcasting internally, we can't depend on backward shape inference to solve this problem for user-defined networks that come from a high level framework like that of Mathematica. The user might want to do a reshape or broadcast of a random source, and then again MXNet will not be able to figure out the batch size to use for the sample op.
I have prototyped a solution for this already was pretty straightforward to implement and simple to reason about, but it does require making a small modification to the nnvm backprop engine, so first I'd first like to ask what the preference of the core MXNet developers is to solve this problem. I can go into detail about my proposed solution if that is desired.

@sbodenstein

see https://github.com/dmlc/mxnet/pull/4586
After this PR you can specify partial shape like (0, 100) for arbitrary batchsize

any updates on this?

@piiswrong I'm very sorry, we were hard at work getting ready for a freeze and I lost track of this thread. I see that #4586 is now closed, but aside from that, was the idea that the random source could be given partial size information using '0' for "don't know", and then that 0 would be resolved with backward inference later? I don't see how that would overcome the problem I described in which backward shape inference cannot in principle go through certain operators (e.g. Resize) that are on the DAG between the place where batch information is available (the layer output the sequences on which you want to apply variational dropout for example) to where the batch information is needed (the random source).

For the record the way that we ended up solving this is to have a 'useless' arg node, of dimensions (batch, n1, n2, ...) that we control and initialize to all zeros to which we add the random source. The resizer code knows about this arg node and can change its initial dimension size and have the rest of the graph infer in forward-mode correctly, with just one step of backward inference that reaches the random source. Slightly wasteful but completely reliable.

This issue is closed due to lack of activity in the last 90 days. Feel free to reopen if this is still an active issue. Thanks!


Bug 1989712: Fix for resolution error on channels with deprecated inner entries.

Resolution would fail with an error in the presence of channels
containing more than two entries with a deprecated inner
entry (e.g. V1 -> V2 -> V3, with deprecated V2). This was due to
special handling that filtered out deprecated bundles at the cache
query layer. A channel's complete replaces-chain must be returned from
cache queries in order to correctly determine the relative order
between entries in the channel.

/cherry-pick release-4.6

/lgtm
/retest

[patch-manager] :rocket: Approved for z-stream by score: 0.50
picked

/cherry-pick release-4.6

/cherry-pick release-4.6


Github Enterprise Doesn't Show Comments On Pre Release Version

Github Enterprise Doesn't Show Comments On Pre Release Extension

Extension version:
v0.67.2023063008
VSCode Version:
Version: 1.80.0-insider (system setup)
OS:
Windows 10

Steps to Reproduce:
Add a comment either using the release version of the extension or the Web UI
View PR with Pre Release Extension
Comment will not be visible

Thanks for the heads up. What version of GitHub Enterprise are you using?

Thanks for the heads up. What version of GitHub Enterprise are you using?

Sorry @alexr00  Sorry should have mentioned that, will try and remember in future, it's GitHub Enterprise Server 3.7.11

FYI @alexr00 This appears to have now made it into the release build: 0.68

I didn't want to hold the release on this. I'm investigating now and I'll make a recover release once I have a fix.

@alexr00 No worries I've rolled back 0.66
I was just letting you know incase it was intentional and created a bit of noise.
Cheers
J

Thanks @jpspringall!

The fix is in main and I just kicked off a pre-release build. It should be available shortly. I will make a recovery release early next week. Thank you for the early warning @jpspringall!

Hey @alexr00 Can confirm that fix has worked.
Not sure if it's related, but the comment icon on the left hand Pull Request is not showing when a comment has been left.
Though to be honest that's always been a bit flaky

@jpspringall thanks for verifying!

Not sure if it's related, but the comment icon on the left hand Pull Request is not showing when a comment has been left.
Though to be honest that's always been a bit flaky

This is if there's a comment on the left hand side of a PR diff? This is unrelated. I will look into it.


Add channelsCount restriction to the demo

LGTM. @theodab any comments from the demo mastermind?

LGTM too.

I'm not sure about this.  We already have a channel count preference parallel to the language preferences.  Is this solving a particular problem we're unaware of?  When would you restrict channels instead of using the preferences?
In particular, we have some test content which contains 6-channel English content and 2-channel dubs in other languages.  In that case, restricting channels could exclude a language.

In a short time we will have 7.1 content, but not all devices support it (for example: tizen), so we need a way to filter these tracks.
My content will have:

aac 2.0 (spa and spa)
aac 5.1 (spa and spa)
aac 7.1 (ing)

Filtering it will not be a problem, I think ...

The intent of the channel preference config is that you would set that to the number of output channels available on the device, to avoid downmixing.  So if the device only has stereo output, you would leave that at the default of 2, and we would never pick 6-channel or 8-channel tracks automatically.  If the device can't decode the 8-channel content, that should be handled earlier when we parse the manifest by dropping those tracks.
If this isn't working right, or if this doesn't adequately address the problem for you, please let us know!  But that's how it should work in current releases.

The current behavior is perfect since there will be no adaptation between different number of channels. But in our UI we have an option for the user to choose the track (depending on the number of channels) that he prefers. I'd prefer avoid the filter in the app side and setup a preference in the player for this filtering.

How do you decide what the restriction should be?  Is it a user preference, or do you have a way to determine the audio output configuration of a device on certain platforms?

The restriction is established with the capabilities of the platform (this is something that is known a priori).
With the Media Capabilities API it will be possible to know this in a real way, but at the moment this is not integrated in the ShakaPlayer.

I'm going to close this because I can manage it by sending a specific stream to the device.


[1.0.0-beta.3] Drawer: Unknown prop onRequestClose on  tag.

Problem description
Unknown prop `onRequestClose` on <div> tag. Remove this prop from the element. For details, see https://fb.me/react-unknown-prop
    in div (created by Drawer)
    in Drawer (created by withStyles(Drawer))
    in withStyles(Drawer) (created by withStyles(AppDrawer))
    in withStyles(AppDrawer) (created by withTheme(AppView))
    in div (created by withTheme(AppView))
    in withTheme(AppView) (created by withWidth(withTheme(AppView)))
    in EventListener (created by withWidth(withTheme(AppView)))
    in withWidth(withTheme(AppView)) (created by withTheme(withWidth(withTheme(AppView))))
    in withTheme(withWidth(withTheme(AppView))) (created by withStyles(withTheme(withWidth(withTheme(AppView)))))
    in withStyles(withTheme(withWidth(withTheme(AppView)))) (created by AppContainer)
    in AppContainer (created by Connect(AppContainer))
    in Connect(AppContainer) (created by Apollo(Connect(AppContainer)))
    in Apollo(Connect(AppContainer)) (created by Apollo(Apollo(Connect(AppContainer))))
    in Apollo(Apollo(Connect(AppContainer))) (created by Route)
    in Route (created by RootContainer)
    in Router (created by BrowserRouter)
    in BrowserRouter (created by RootContainer)
    in MuiThemeProvider (created by RootContainer)
    in ApolloProvider (created by RootContainer)
    in Provider (created by RootContainer)
    in RootContainer
    in AppContainer

Versions

Material-UI: 1.0.0-beta.3
React: 16.0.0-beta.2
Browser: Chrome Version 60.0.3112.72 (Official Build) beta (64-bit)

https://github.com/callemall/material-ui/pull/7596

@HriBB Thanks for proposing that change! That was useful :)


Had trouble logging in and then got old UI error screen

Just thought you might want to know, I only tried once, but somehow when i got a log in error, the whole ui changed to what looks like old ui or something.

@bgirardot are you able to log in now successfully? were you logging in via an OSM account you used previously with the export tool, or a new OSM account?

I have the same problem, I always was logged in, but just logged out and now I see this error. Just a normal OSM account


Factor commanded/measured parameters and graphs out of the two modes

It is important to factor them out because they will actually be evolving quite a bit, and because modes are not completely identical.
A few things that I know will be changing:

The commanded vs measured values will be separated (currently eg commanded and measured pip widgets are wired to the same thing in GuiStateContainer)
Some parameter displays will be wired up with alarms

It would be very inconvenient to do both of these twice.
There's more room to extract common stuff out of the modes but I think the PR takes care of the most important parts.

Looks reasonable to me, and I'm happy to defer to Paulo unless there's something specific you'd like feedback on from me.

Thanks Paulo, please take another look?


Portalling out of Nether with smaller border than overworld checks the Nether-border for overworld location

My players have been complaining about getting "You have reached the edge of this world" error message when taking a nether portal out of the nether and into the overworld.
This is how I have my borders set:
round-border: false
...
  world:
    x: 0.5
    z: 0.5
    radiusX: 20000
    radiusZ: 20000
    wrapping: false
  world_nether:
    x: 0.5
    z: 0.5
    radiusX: 2500
    radiusZ: 2500
    wrapping: false

If a player makes a portal in the Nether, say at 75, 435, when they enter the portal they are teleported to 2497, 2497 in the Overworld and told that they have reached the edge of the world. A portal is not created AND sometimes they are even teleported into walls and suffocate and die.
Debug output:
12.12 00:35:18 [Server] INFO Teleport cause: NETHER_PORTAL
12.12 00:35:20 [Server] INFO Teleport cause: UNKNOWN
12.12 00:35:20 [Server] WARN Border crossing in "world_nether". Border radius 2500 at X: 0.5 Z: 0.5
12.12 00:35:20 [Server] WARN Player position X: 607.5 Y: 70.2 Z: 3469.2
12.12 00:35:20 [Server] WARN New position in world "world_nether" at X: 607.5 Y: 61.0 Z: 2497.5```

Notice the "New Position" SHOULD BE "world" but instead says "world_nether"

This server is running Paper version git-Paper-235 (MC: 1.14.4) (Implementing API version 1.14.4-R0.1-SNAPSHOT)

It's a long-running bug in Spigot/Paper where teleport events are being incorrectly fired with mismatched world name and coordinates. At one point they fixed it, but it came back many months ago and they still haven't fixed it again.
I'm not interested in making a workaround for their bugs.

It looks like they finally fixed it:
https://hub.spigotmc.org/jira/browse/SPIGOT-5252
So it should be handled in the next Spigot release.


Fix Armorer special weapons not correctly importing as Items

Update DDBAttackAction.js with the correct designation of the Thunder Gauntlets and Lightning Launcher. They have the name of the armor in front of their names, much like the Psychic Blades do.

THis isn't quite right as Thunder Gauntlets don't import with a prefix, but will get these changes merged into the next release.


IOS Client

Does your offer of paypaling you the apple $99 still stand?
You would do the updates for the readme noted map issues?
Would this also validate the ios client for the latest version of ios?
k

Hey Kenneth. I believe I wrote that several years ago and I'm not actively writing IOS software now. I have been considering paying someone on one of the freelance developer websites to rewrite the IOS client in SWIFT, apple's new programming language. I am guessing it will cost 100-150 dollars to rewrite the app. That's kind of where it stands right now.


Fix(Vision): Issue#8 Improving vision card styling

I added images and hover effect to the card. I also made it responsive across viewports.
#Changes I implemented
Pc (inactive)

##Pc (Hover)

##Mobile(Inactive)

##Mobile(Hover)

@pooranjoyb

Nice Work @DannyK05  🚀

Thanks bro


Prepare labels fix

Adds download gdrive documentation.
Cleans up the script for the creation of labels with documentation and a variable number of labels for segmentation.