Certificate error ERR_CERT_AUTHORITY_INVALID

Hi, I started using Waterfly app some month ago and it goes very well.
Unfortunately some weeks ago I can't use the app anymore because the error below

What can be happened?
Since it worked before my dubt is if it could have happened somthing to the app.
I rechecked the imported custom ca certificate into the system and works, I deleted the data of the app and reimported che certificate of the server and the personal token.
I use a Samsung updated to the last version of everything available.
FF v6.1.12

I have already added the CA Certificate into Android system because I use it also for other local services so, unfortunately,  the error is the same as above :(

Chiming in with an additional data point:
I also have a self-signed CA which I have imported and trusted on my Android device. The site works fine in browsers on the device.
The signing chain is: Root CA -> Intermediate CA -> Server
Without adding a custom cert to the app, I get "Invalid SSL Certificate" error. When adding a custom certificate to the app, I get the same "Invalid SSL Certificate" error when the PEM is any of the root CA certificate, the server certificate, or the full certificate chain.
Interestingly, I get the same error as OP when I try to add the intermediate CA certificate to the app.
It would be ideal to only have to add a single certificate from anywhere in the chain, like the root certificate in a 3-tier PKI , such that shot-lived server certificates would not need to be copy+pasted so often.

Hi @dreautall thank you for the quick response on this.
Sorry to comment on the closed issue, and please let me know if I should open a new one.
I was looking over the change made for v1.0.4 and noticed it was previously using the CronetClient.defaultCronetEngine(). I was then looking over the CronetClient class and found CronetClient.fromCronetEngine(CronetEngine, {bool}). It looks like supplying this with a CronetEngine built with enablePublicKeyPinningBypassForLocalTrustAnchors = true might do the trick in allowing the device's trust store. I'm not sure if it's enabled or disabled in the default engine.
Hope this helps! I might try building the app with those changes to see if that resolves it for my situation, but I have no experience with Dart. 🙃

I gave building the app a shot and it seems you are correct in that Cronet is not working with the local trust store, at least with a minimum effort attempt in doing so.
I was also debugging through the badCertificateCallback using IOClient and found it to be called with the certificate chain (i.e. my intermediate CA certificate) and not the server certificate. That was interesting as, going by the documentation, I was expecting the server certificate! Not sure how one would attempt a legitimate X509 verification with that, since the host won't match to anything in the X509Certificate...
Anyway, this solution seems to work for me, for now. Thanks again!


Update to adUser and adGroup

Added possibility to add ad users to ad groups.
Added password never expires option to aduser

Thank you for the heads-up, I will change the issue, test it and push it to to the branch.


customer.subscriptions.retrieve throws TypeError

On stripe-ruby-mock 2.3.0 and stripe 1.43.0.
Calling sub = customer.subscriptions.retrieve(sub_id) throws:
TypeError: no implicit conversion of Symbol into Integer
from /Users/kelvinma/Work/Codecademy-dev/repos/Codecademy/.bundle/ruby/2.3.0/gems/stripe-ruby-mock-2.3.0/lib/stripe_mock/request_handlers/helpers/subscription_helpers.rb:6:in `[]'

Looking inside retrieve_customer_subscription method. I see its returning customer as an array with index 0 being :customer, 1 being the id, 2 being the actual customer object.
customer = :customer, $1, customers[$1]
=> [:customer,
 "test_cus_3",
<EMAIL_ADDRESS>  :description=>"an auto-generated stripe customer data mock",

vs calling customers[$1] directly returns the customer object
customers[$1]
=><EMAIL_ADDRESS> :description=>"an auto-generated stripe customer data mock",
 :object=>"customer",

which the next line get_customer_subscriptions
customer[:subscriptions][:data].find{|sub| sub[:id] == sub_id } then errors with TypeError: no implicit conversion of Symbol into Integer since customer is an array here.
Has anyone else encountered this error?

Yes it throw error on latest Rails 5, stripe-ruby-mock (2.3.0), stripe (1.43.0)
customer.subscriptions.retrieve('test_su_4')
*** TypeError Exception: no implicit conversion of Symbol into Integer

I am getting the same error after trying to upgrade from 2.2.4 (also upgrading stripe from 1.41.0 to 1.43.0)
I was hoping to finally be able to upgrade the stripe gem, but not if it breaks my tests. Any help is highly appreciated.

I'm getting the same error. Looks like the issue in this line.
https://github.com/rebelidealist/stripe-ruby-mock/blob/master/lib/stripe_mock/request_handlers/helpers/subscription_helpers.rb#L6

I'm still getting it

@AndreiMotinga Its merged to master but haven't been released in a new version of the Gem yet. Either wait for a new release 2.3.1 or change your Gemfile to directly pull from this repo on master.


Level editor shouldn't require JSON editing

The level editor should let the player edit levels without typing in JSON. At a minimum, they should be able to change the following characteristics of a level:

Start difficulty
Level-ups
Piece selection
Finish condition
Success condition

This could use a drag/drop interface. The current set of rules could be displayed in a big grid. The player can interact with this grid in a few ways:

Drag from the palette to the grid to add a rule
Click the grid to select a rule
Manipulate dropdowns or spinners to edit a rule
Click 'delete' to delete the selected a rule

https://www.reddit.com/r/Tetris/comments/ib8i7s/im_done_with_the_editor_for_develop_spin_clear_so/ this 'spin clear' level editor UI is fantastic, consider some things like this for UI

Downgraded; the level editor has been removed from the game. If it is reintroduced, we'll have to consider this.

The level editor has been removed.


Use a clean SSL context by default, allow users to override if needed

Relevant to #95 - by default, this change isolates the Mixpanel library from app changes to the default SSL context, but allows users to explicitly set their own context if it is needed.

hmmm.. So, looks like we don't support SSL connection at all in the current release?
Anyways, these changes look straight forward to me. Can you please run the full test suite and see what happens? I don't know if these additional SSL stuff will fail our current tests (most of our tests have a timeout and I'm expecting SSL will add in extra delay).
Also, have you tried the seledroid test? I don't know if this SSL connection work with it.

We do - we use the system default SSL connection, which is usually fine. Unfortunately, we've run into issues where third party libraries or client code changes the default SSL configuration in ways that break with the api. I've run the tests - we shouldn't be emitting http requests there anyhow.

oh.. sorry.. I missed the SSL related code in ViewCrawler.java.
Does the test case cover the case where the user adds his own SSL? Are we able to test it?

Right now we try not to hit the network in the unit tests - I've been testing this manually during development. We can at least test exposure to the default SSL socket settings, though.

OK. Cool. Sounds good to me.


provide support for 1.24 named vectors feature

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

:exclamation: No coverage uploaded for pull request base (stable/v3@21d3552). Click here to learn what that means.

Additional details and impacted files
@@             Coverage Diff              @@
##             stable/v3     #879   +/-   ##
============================================
  Coverage             ?   93.50%
============================================
  Files                ?       70
  Lines                ?     8639
  Branches             ?        0
============================================
  Hits                 ?     8078
  Misses               ?      561
  Partials             ?        0

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

Closing as unplanned for long-term maintenance and support


Snake mode of peptide chain drawing

There are two modes available: Snake and Graph
By default, in Graph mode all bonds are established center-to-center in a straight lines
When user switches switches to Snake mode:
а) All monomers are arranged in rows
b) All bonds are switched to "rectangular" mode, i.e. bonds are capable of turning 90 degrees, allowing the user to shift certain parts of a bond.
In Snake mode, new monomers can be added and existing monomers can be moved to any position while the bonds remain rectangular and appropriately rebuilt.
When switching back to Graph mode, all monomers remain in place and bonds revert to simple straight lines from center to center.


Allow write notifications that don't trigger collection notifications

Feature Request
I'm using realm Collection Notifications to keep a UITableView in sync with a particular query. I've been trying to build out a feature that let's the user delete an item from the table. In this case I'd like to be able to show the delete and updated state of the UI without waiting for the asynchronous collection notification.
I often see delays between a write and the corresponding collection notification that make waiting on the notification a pretty bad experience. Given that the collection notifications are aggregated, ignoring specific updates is a challenge. As far as I can tell there is no way to modify the tableview directly (and then propagate the change to realm) while using collection notifications.

Hi @emuye, thanks for filing this. We agree that we need to expose a mechanism through which write transactions may be performed without triggering collection change notifications and are tracking that as #3665.
When making changes to data due to user interactions (e.g. main UI thread), you should certainly make those synchronously to keep things flowing at 60fps. Notifications are delivered asynchronously.
I'm closing this ticket as a duplicate and would encourage you to continue this conversation in #3665 so we can keep the discussion in one place. Thanks!


Add a method to test whether a DateTime matches a Schedule

e.g.
let schedule = from_str("@weekly");
if schedule.includes(UTC.now()) {
  // ....
}

Would this be sufficient?
pub fn contains<Z>(&self, date_time: DateTime<Z>) -> bool
    where
        Z: TimeZone,
    {
        self.fields.years.includes(date_time.year() as Ordinal)  &&
        self.fields.months.includes(date_time.month() as Ordinal) &&
        self.fields.days_of_week.includes(date_time.weekday().number_from_sunday()) &&
        self.fields.days_of_month.includes(date_time.day() as Ordinal) &&
        self.fields.hours.includes(date_time.hour() as Ordinal) &&
        self.fields.minutes.includes(date_time.minute() as Ordinal) &&
        self.fields.minutes.includes(date_time.second() as Ordinal)
    }

( I'm a bit unsure why it's TimeUnitSpec::includes instead of contains, the convention on collections seems to be contains, but I might be wrong. )

I believe that's sufficient, yes. Accepting a DateTime<_> as a parameter rules out a lot of corner cases as the user will have already managed to create the value.

( I'm a bit unsure why it's TimeUnitSpec::includes instead of contains, the convention on collections seems to be contains, but I might be wrong. )

I think I used the name includes because a Schedule is not obviously a set or container. Conversationally, one might say of a lower-case 's' schedule:

"Here's a datetime, is it on the schedule?"
"Is this datetime part of the schedule?"
"Does the schedule include this datetime?"

To my ears, "does the schedule contain this datetime?" feels a bit off. I'm not married to the name includes, though.

Great, I'll write some tests to make sure

I think I used the name includes because a Schedule is not obviously a set or container.

I get that. But 'years' and 'months' both obviously seem like a set of... years and months? I mean, years.contains(year_x) seemed so natural to me, that I was a bit surprised that it wouldn't compile

Ah, I see what you mean. Sorry, I was focused on the method name in your code sample, but you were referring to TimeUnitSpec. My mistake.
In the case of TimeUnitSpec, I was trying to keep its implementation abstract. They're each implemented as a set, but from the user's perspective the question is "Does 1-6/2 include Tuesday?" rather than "Does the set of ordinals produced by parsing the expression 1-6/2 contain Tuesday?"

I the user uses the function years() though, doesn't he expect something that works like a collection?
Don't get me wrong, I'm happy to keep it at includes, I'm just a bit suprised
Also, is TimeUnitSpecEq a good name for the tuple struct?


[tidb] add tidb cdc connector

This PR fixes issue #243.

[x] cherry-pick initial commit from TiLaker.
[x] support metadata.
[x] add more tests.

cherry-pick and squash commits, then force push.

Thanks @eastfisher and @GOODBOY008 for the great work! Looks pretty good to me as our first version, I'll help organize the commits and merge them.


Modifying schema

Extending the ChatOps schema, as discussed:

Support custom ack and result parameters.
Support aliases.

This PR is complete, but I'm putting it on hold until the changes in hubot-stackstorm are ready to merge as well.

Hm, that's weird, I don't think failure is related to your change (https://travis-ci.org/StackStorm/st2/jobs/89456490#L265), I will dig in.


fix(luasnip): include global snippets

Global snippets have the special filetype all. See https://github.com/L3MON4D3/LuaSnip/blob/v2.3.0/DOC.md#adding-snippets.

Ah I didn't see the recent #546.


vit demo not running past one epoch

when running the base config, eg:

time python3 main.py --config=projects/baselines/configs/imagenet/imagenet_vit_config.py --workdir=./fp32-test

with the only modification being a batch size of 48 (i am using 2 nvidia gpu's to run things)
at the end of the first epoch the code dies:
I0929 15:28:35.031877<PHONE_NUMBER>83936 local.py:41] Setting work unit notes: 0.4% @9277, 0.3 steps/s, ETA: 153959 min (90 min : 0.0% checkpoint, 23.7% eval)
I0929 15:28:35.042148<PHONE_NUMBER>43392 logging_writer.py:34] [9277] steps_per_sec=0.259032
I0929 15:29:42.340490<PHONE_NUMBER>83936 local.py:41] Setting work unit notes: 0.4% @9281, 0.1 steps/s, ETA: 671072 min (91 min : 0.0% checkpoint, 23.4% eval)
I0929 15:29:42.351035<PHONE_NUMBER>43392 logging_writer.py:34] [9281] steps_per_sec=0.059428
I0929 15:31:58.488358<PHONE_NUMBER>83936 local.py:41] Setting work unit notes: 0.4% @9283, 0.0 steps/s, ETA: 2713548 min (93 min : 0.0% checkpoint, 22.8% eval)
I0929 15:31:58.627787<PHONE_NUMBER>43392 logging_writer.py:34] [9283] steps_per_sec=0.014697
Killed

real    93m53.945s
user    222m56.696s
sys     63m40.052s

do you have any suggestions on things to check/have you seen this before?

might be upstream fwiw as well: https://github.com/google/jax/issues/8044

I'm not 100% sure, but this looks like you ran out of memory when it started doing validation, which is done at the end of each epoch.
To check this, can you set log_eval_steps  to a small number and also play with steps_per_eval in your config file?

finally managed to test things a bit more locally.  you are correct, it is a memory issue.  having said that, it's not really scenic's fault, just that the code loads imagenet into memory by default (eg ~165GB of ram usage) and if you have less then that things will get wonky!  training fine here!


ERROR：compare error

Before reporting an issue please first check the
troubleshooting guide. If
the issue you're encountering is not solved thereby please state the following in your bugreport:

Operating system
Ubuntu 18.04
The way you installed OpenCV: package, official binary distribution, manual compilation, etc.
/usr/local/opencv4/include/opencv4
OpenCV version
opencv4

pkg-config --modversion opencv

3.4.14

rustc version (rustc --version)
rustc 1.66.0 (69f9c33d7 2022-12-12)

Attach the full output of the following command from your project directory:
RUST_BACKTRACE=full cargo build -vv

error occurred: Command "c++" "-O3" "-ffunction-sections" "-fdata-sections" "-fPIC" "-m64" "-I" "/home/sunny/.cargo/registry/src/github.com-1ecc6299db9ec823/opencv-0.73.0/src_cpp" "-I" "/rust/cpp/FaceBase/Rust_use_FaceBase/target/release/build/opencv-58ab2eba5465e2d5/out" "-I" "." "-I" "/usr/local/opencv4/include/opencv4" "-Wall" "-Wextra" "-std=c++11" "-Wno-deprecated-declarations" "-Wno-deprecated-copy" "-Wno-unused-parameter" "-Wno-sign-compare" "-Wno-comment" "-Wno-unused-variable" "-Wno-ignored-qualifiers" "-Wno-return-type-c-linkage" "-o" "/rust/cpp/FaceBase/Rust_use_FaceBase/target/release/build/opencv-58ab2eba5465e2d5/out/2e2fced53fca5dac-gapi.o" "-c" "/rust/cpp/FaceBase/Rust_use_FaceBase/target/release/build/opencv-58ab2eba5465e2d5/out/gapi.cpp" with args "c++" did not execute successfully (status code exit status: 1).

I See.
vim ~/.bashrc
remove
export PKG_CONFIG_PATH=${PKG_CONFIG_PATH}:xxx/opencv4
export LD_LIBRARY_PATH=${LD_LIBRARY_PATH}:xxxx/opencv4
sudo mv /usr/local/lib/pkgconfig/opencv4.pc /usr/local/lib/pkgconfig/opencv4.pc.bak

sudo ldconfig


start_http_server for any given registry or the default registry

I'm using start_http_server, but unfortunately it only exposes the default registry, all other exposition methods have a registry parameter.
@brian-brazil

You should only ever be exposing the default registry via this method. If you're doing something funky (such as what the snmp or blackbox exporters do), you'll need custom handling anyway.

What I want is to expose 2 different registries on 2 different ports.
start_wsgi_server allows to receive a specific registry while start_http_server doesn't.
How those 2 functions differ except WSGI/SimpleHTTPServer ?

What I want is to expose 2 different registries on 2 different ports.

Why do you want to do this?
A single process should have a single registry.

Is there any reason why start_wsgi_server should be different than start_http_server?

There isn't, that parameter should be removed.

@brian-brazil would you accept a PR that removes the parameter?

I would.


Fix typo of default version of nacos jar

Seems the typo of commit, fixed.

just saw the dup one #292


Pass objects to TOC

This takes a slightly different approach passing content to the TOC.

The original model: Parse the entry down to a single string item, pass it to the TOC, then use it as a string for the PDF index and wrap it it an object again to typeset.

The new model: Pass the original entry as-is to the TOC. Then reduce it to a string (by walking the entire object) for the PDF index and use the whole object minus the top level function to typeset the contents.

This approach allows chapter titles to include extra code (such as emphasized words, or in my first use case discretionary break points) in the chapter title string that actually gets used in the TOC entry. Of course having SILE code passed along to the TOC works fine for typesetting it, but not so much for cramming into the PDF index. For that I wrote a couple utility functions to reduce content structures down to a string or just strip the top level command for various use cases.

Thank you!


setting the expanded property on the TreeView seems to disable expanding tree items that have children

If I set the expanded array property on a TreeView the TreeItems that match the nodeIds will render expanded as expected, but TreeItems that have children do not expand when they are clicked.
I tweaked the MUI TreeView example to demonstrate:
https://codesandbox.io/s/material-demo-5eyjj?from-embed
thanks.

Tip: check the concept of controlled vs uncontrollable.


multi-client session resets emulation after cross-process navigation

Steps to reproduce
Tell us about your environment:

Puppeteer version: 1.6.1
Platform / OS version: macOS 10.13.6
URLs (if applicable): -
Node.js version: 10.7.0

What steps will reproduce the problem?
const puppeteer = require('puppeteer');
const devices = require('puppeteer/DeviceDescriptors');

function getViewportDimensions () {
  return {
    innerWidth: window.innerWidth,
    innerHeight: window.innerHeight,
    outerWidth: window.outerWidth,
    outerHeight: window.outerHeight,
    devicePixelRatio: window.devicePixelRatio
  };
}

(async () => {
  const browser = await puppeteer.launch({ headless: true });
  const page = await browser.newPage();
  const device = devices['iPhone 6'];
  console.log(device);
  await page.emulate(device);
  await page.target().createCDPSession();
  await page.goto('https://www.google.com/');
  const dimensions = await page.evaluate(getViewportDimensions);
  console.log(dimensions);
  await browser.close();
})();

Run the above code
Remove the line await page.target().createCDPSession();
Run the code again
Add the line back and set headless to false
Run the code again
Set headless back to true and swap line  await page.target().createCDPSession(); with line  await page.emulate(device);
Run the code again

What is the expected result?
Output is every time:
{ innerWidth: 375,
  innerHeight: 667,
  outerWidth: 375,
  outerHeight: 667,
  devicePixelRatio: 2 }

What happens instead?
In step 1. output is
{ innerWidth: 375,
  innerHeight: 667,
  outerWidth: 0,
  outerHeight: 0,
  devicePixelRatio: 1 }

@Janpot This is a bug with our multiclient implementation:  InspectorEmulationAgent from the second session resets emulation in it's Restore method after cross-process navigation: https://cs.chromium.org/chromium/src/third_party/blink/renderer/core/inspector/inspector_emulation_agent.cc?q=emulationagent+f:blink&dr=CSs&l=30

any news in this bug?

I receive an error message UnhandledPromiseRejectionWarning: TypeError: page.target is not a function. I followed the instructions in Steps to reproduce.

@aslushnikov I can't seem to reproduce with v1.13.0 anymore.


iOS 7.1.2 fails :(

After the August 3rd update when the Build Service updated to cli 6.3.0 my apps haven't been working for iOS, I have an iPhone 4 running iOS 7.1.2 and an iPad running iOS 9.3.3. After the build service update, no matter if I specify a lower cli version, on the iPhone it asks for a newer software when I try to install from iTunes, but both the iPhone and iTunes are running on the latest version. On the iPad it does install, but the app's icon doesn't appear on the iPad so I can't access to the application.

@circleride sorry, this plugin does not support iOS 7.

What version number supports iOS 7?


Fix typo and rephrase error message

That makes sense, thank you very much!
Don't worry about the CI build failing - deterministic builds ....


Python: mark test requirements as developer dependencies

Description
In our project we have 3 requirement lock files

requirements.txt for production dist
test_requirements.txt for running tests in ci
dev_requirements.txt which includes all of the above for local development

We also have auto merge configured for development dependencies
However dependabot fails to execute the auto merge because it marks test_requirements.txt file as a production dependency.
Examples

https://github.com/tophat/syrupy/pull/174
https://github.com/tophat/syrupy/pull/175

Related

https://github.com/dependabot/dependabot-core/issues/1165

Hi @iamogbz. Automatic Merging is a feature of Dependabot Preview, which was deprecated in April and shut down on August 3rd. There are no current plans to add this functionality to GitHub-native Dependabot. See: #1973 (comment).


Question: Reduce size? (from 876kb to ~200kb)

Background
My Vue.js Single Page Application(SPA) is too big (3M)

So I am tracking what make it so big
using webpack-bundle-analyzer
here is my config:
webpack.config.js
const BundleAnalyzerPlugin = require('webpack-bundle-analyzer').BundleAnalyzerPlugin;

// etc..etc...

  plugins: [
    new BundleAnalyzerPlugin()
  ],

Here is result from 'webpack-bundle-analyzer'

As you can see
tui-editor-Editor.js is big (874kb)
and I don't need codemirror.js for code hight
How Do I use it?
I am using Tui.editor in a Vue Component

require("tui-editor/dist/tui-editor.css"); // editor ui
require("tui-editor/dist/tui-editor-contents.css"); // editor content
import Editor from "tui-editor";

//etc....

    this.editor = new Editor({
      el: document.querySelector("#topic_content"),
      initialEditType: "wysiwyg",
      hideModeSwitch: true,
      language: 'zh', // 中文
      usageStatistics: false,
      toolbarItems: [
        'heading',
        'image',
      ],
      hooks: {
        addImageBlobHook: function(file, callback) {
          function callback_for_image_upload(image_url){
            let img_url = get_full_image_url(image_url);
            callback(img_url, 'image');
          }
          self.upload_file_with_callback(file, callback_for_image_upload);
        }
      }
    });

    const toolbar = this.editor.getUI().getToolbar();

    toolbar.addButton({
      name: 'heading',
      className: '',
      event: 'openHeadingSelect',
      tooltip: '标题',
      $el: $('<button class="our-button-class-heading"></button>')
    }, 1);
    toolbar.addButton({
      name: 'itlic',
      className: '',
      command: 'Italic',
      tooltip: '斜体',
      $el: $('<button class="our-button-class"></button>')
    }, 2);
    toolbar.addButton({
      name: 'itlic',
      className: '',
      command: 'Bold',
      tooltip: '加粗',
      $el: $('<button class="our-button-class"></button>')
    }, 2);
    toolbar.addButton({
      name: 'itlic',
      className: '',
      command: 'Strike',
      tooltip: '删除线',
      $el: $('<button class="our-button-class"></button>')
    }, 2);
    toolbar.addButton({
      name: 'hr',
      className: '',
      command: 'HR',
      tooltip: '水平线',
      $el: $('<button class="our-button-class"></button>')
    }, 2);
    toolbar.addButton({
      name: 'hr',
      className: '',
      command: 'Blockquote',
      tooltip: '引用',
      $el: $('<button class="our-button-class"></button>')
    }, 2);
    // 超链接
    toolbar.addButton({
      name: 'link',
      event: 'openPopupAddLink',
      tooltip: '超链接',
      state: '',
      $el: $('<button class="our-button-class"></button>')
    }, 2);
    toolbar.addButton({
      name: 'hr',
      event: 'openPopupAddImage',
      tooltip: '图片',
      state: '',
      $el: $('<button class="our-button-class"></button>')
    }, 2);

How my app look like

Question
Is there anyway to reduce size? maybe to 200kb or so.
Thank you

by the way, Are you maintaining this yourself? alone? that's a lot of work.
I want heard the story of how tui.editor get started, what's the reason behind building it. haha

Looks like a large part of the code is the code highlighting, also maybe you could pull codemirror out of your own dependencies as you may have it twice.
<EMAIL_ADDRESS>

@acoreyj Thank you!

I am closing this issue.
because I am switching from tui.editor to Quill now.
Quill is more suitable for my project.
Still thank you for build tui editor and open source it 😄


System props can't override custom component's style like what sx would do

Duplicates

[X] I have searched the existing issues

Latest version

[X] I have tested the latest version

Current behavior 😯
Create a custom component by styled():
const Square = styled(Box)({
  width: 100,
  height: 100,
  backgroundColor: "red"
});

Override its style with sx works:
<Square sx={{ backgroundColor: "blue" }} />

Override its style with backgroundColor doesn't work:
<Square backgroundColor="blue" />

The style provided by backgroundColor prop appears before the inner style.
Expected behavior 🤔
Override styles with system props works like what sx would do.
Steps to reproduce 🕹
codesandbox
Context 🔦
No response
Your environment 🌎

`npx @mui/envinfo`
  Don't forget to mention which browser you used.
  Output from `npx @mui/envinfo` goes here.

Right, it seems to me that sx should work the same way as style props.

This is how emotion works, I am not sure if we should change the behavior. What I mean is, if you use styled() on top of a component, it overrides the styles that it has (it doesn't matter if they are correlated to a prop or not). The sx prop is the only prop that allows you to opt out of this.

Right, it seems to me that sx should work the same way as style props.

It does, it is described in the issue description, the question is for the other props.

I would recommend solving this by adding styles if the prop is not provided (that would serve as a default value), for example:
const Square = styled(Box)((props) => ({
  width: 100,
  height: 100,
  ...(!props.backgroundColor && {
      backgroundColor: "red"
  }),
}));

@michaldudak I would say this is expected behavior, can you check again.

Gotcha, it makes sense.


[FLINK-28308] Introduce metrics of the accumulated time that a running task is busy / idle / back-pressured

What is the purpose of the change
Introduce metrics of the accumulated time that a running task is busy / idle / back-pressured
Verifying this change
This change added tests:

SubtaskExecutionAttemptDetailsInfoTest
AdaptiveBatchSchedulerTest
etc.

Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): no
The public API, i.e., is any changed class annotated with @Public(Evolving): yes
The serializers: no
The runtime per-record code paths (performance sensitive): no
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Kubernetes/Yarn, ZooKeeper: no
The S3 file system connector: no

Documentation

Does this pull request introduce a new feature? yes
If yes, how is the feature documented?  docs

@flinkbot run azure


Adding Stride to Ethereum stTIA warp route

Description
Added Stride to Ethereum stTIA warp route
Backward compatibility
Yes
Testing
Tested manually in the Stride bridge (leveraging the hyperlane sdk)

@jmrossy prettified it here


Fix typo

a -> an
refs #28710

@changeworld : Thanks for your contribution! The author, @jeevansd, has been notified to review your proposed change.

Merging the typo fix.
@changeworld Thanks!


Packages contain aws sdk

The current build process includes the aws sdk (and all of its dependencies) when packaging a RP.
We should be able to leave the aws sdk out and have a smaller package size.
Downloading/installing will be faster and hosting the registry will cost less money

Fixed by using the bootstrap from Lerna. It only installs the AWS SDK in the root of the repository, so it should not add it to the package anymore.

i found out what was the problem. during packaging with docker, the .npmrc in the repo root was not being considered. after creating one in each subfolder, the bundled size is a lot smaller.


Add a top level README.md for ResourceManager folder

The resourcemanager packages need a top-level readme describing the structure and general patterns for all ResourceManager packages.  Something akin to https://github.com/Azure/azure-sdk-for-net/blob/main/sdk/resourcemanager/Azure.ResourceManager/README.md

As github.com/Azure/azure-sdk-for-go/sdk/resourcemanager is not a package (ResourceManager is .Net SDK's core lib like azcore for go), it might not be so accessible to put a readme under it. I've added some detailed info into quickstart for mgmt. plane sdk in this PR. I think I can separate it into two docs: one for easy quick start, another for mgmt. plane SDK guideline.


query: how to handle non-mandatory columns in address oscillation.

While refactoring AddressOscillation.py and oscillation_type1.py, we observed that the code depends on columns that are not in the mandatory column list: [user_id, orig_lat, orig_long, unix_start_t].
These columns are: [stay_lat, stay_long, stay_unc,  stay_dur, orig_unc].
Currently, the code assumes the value for columns [stay_lat, stay_long, 'stay_dur', 'stay_unc'] to be set as -1 but never missing from the csv file.

While it is possible to put a conditional check in the code for when [stay_lat, stay_long, 'stay_dur'] columns are missing, and retrieve values from [orig_lat, orig_long, 1] respectively. Can you please confirm if this is the expected behavior?
https://github.com/uw-ssec/MAWpy/blob/main/src/mawpy/oscillation_type1.py#L43-L51
https://github.com/uw-ssec/MAWpy/blob/main/src/mawpy/oscillation_type1.py#L55-L57

Also, the code tries to retrieve the value of 'orig_unc' when 'stay_unc' is -1. Whereas, as per our discussion 'orig_unc' is also an optional column. What should be the expected behavior in this case?
https://github.com/uw-ssec/MAWpy/blob/main/src/mawpy/oscillation_type1.py#L52-L54

assignees: @gracejia513 @@Anurag19101996
for awareness: @carlosgjs @lsetiawan

We discussed this with @qzchen-uw , and the decision is that the address oscillation should work regardless of whether the input has these columns: stay_lat, stay_long, stay_unc, stay_dur, orig_unc.
So for the first bullet point, I think we want the behavior to be: if no clustering is run prior to address oscillation, we use orig_lat and orig_long; if clustering is run prior to address oscillation, se use stay_lat, stay_long, stay_dur, and stay_unc.
And for the second bullet point, I think the unc, whether orig_unc or stay_unc, is saved to a variable rad (https://github.com/uw-ssec/MAWpy/blob/f6132307ec88ddbb89e37cd5a2e8eba36687434e/src/mawpy/oscillation_type1.py#L52-L54), and rad was never referenced again later in the code. Rad was then saved to tracelist (a nested list) as the sixth item of the sublist (and the fourth and fifth items are lat and long). So to reference rad, one might use a syntax of tracelist[i][5]. And I think tracelist[i][5] was only involved in checking whether two observations are the same by comparing the values of tracelist[i][3:5]. And this comparison might be replaced by tracelist[i][3:4], which only involves checking the lat (set to orig_lat previously), and long (set to orig_long previously).
@qzchen-uw and @Anurag19101996 please edit as you see fit.


Update veasy-table.js

In latest version of angular UI those service names have been changed

Thank you for your contribution :smiley:


Fix Maintainer section look

What kind of change does this PR introduce?
Fix Maintainer section HTML
Did you add tests for your changes?
No
If relevant, did you update the README?
Yes
Summary
Maintainers section HTML was incorrect, making the website look like this:

After the change it looks like this:

Current coverage is 98.35% (diff: 100%)

Merging #409 into master will not change coverage

@@             master       #409   diff @@
==========================================
  Files             9          9
  Lines           304        304
  Methods          41         41
  Messages          0          0
  Branches         67         67
==========================================
  Hits            299        299
  Misses            5          5
  Partials          0          0

Powered by Codecov. Last update 793390a...d5a0a4e


Translation

- Do you want to request a feature or report a bug?
feature
- What is the current behavior?
the cms is only in english
- If the current behavior is a bug, please provide the steps to reproduce.
- What is the expected behavior?
it might be possible to select the default language in Netlify and also be able to change it in CMS.
- Please mention your CMS, node.js, and operating system version.

@derevandal We currently have a PR for supporting other languages: https://github.com/netlify/netlify-cms/pull/403. It needs finished up, though (https://github.com/netlify/netlify-cms/pull/403#issuecomment-326753066) and if you are able to help, that would be great!

Given that #974 is closed, would you like me to work on this? I can start a new PR following the previous findings. Im interested in getting this in.

@nahueld if you need some help, you can count with me.
I think you can start a new branch with frozen features to work only with translatation feature... We can discuss the patterns before putting our hands dirty.

Based on what I have read so far the solution that I have in mind would be implemented in 2 phases:
1⃣ Extract current texts from core
This phase would encompass the following (but not limited to) tasks:

Create a key/value file for all the texts in core module.

Key would be a meaningful label to identify the text.
The value would be the actual translation (or in this phase the English text)

Create a mechanism to read/use these values (probably using Polyglot, but open to other suggestions)

2⃣ Provide a mechanism to provide a custom key/value file

In this 2nd phase we would allow users to pass a custom key/value file.
We would need to document the expected keys.
Under the hood we would merge both files (defaulting to English texts if some of the keys are not provided by the user).
Probably the file path would be provided as value in the config.yml.

Future improvements (not necessarily part of this).

Allow users to provide multiple translation files.
Read browser agent language and decide which one to use.
Include some good translations as part of the default build.

Let me know your thoughts.

@nahueld I agree with all your suggestions!
could you lead this with a new fork only for this feature?
if you start this, i can help you committing too.

@derevandal The CMS is not built on Gatsby -- only our docs site -- so I'm not sure that a Gatsby plugin would work.

Aside from translation, what about hardcoded US date formats? Different day/month order is confusing for non-us users (not saying which format is better, not relevant here)

Definitely doable! @nahueld laid out the plan in his comment: https://github.com/netlify/netlify-cms/issues/789#issuecomment-417174441
(specifically phase two.)

All right lets get phase 2 working so people can contribute with more pretentious features

@nahueld clarifying - are you working on this yourself?

Yes will try to have something in a week , we can discuss other approaches
but currently is similar to how we load the config file
On Thu, 28 Feb 2019, 13:39 Shawn Erquhart<EMAIL_ADDRESS>wrote:

@nahueld https://github.com/nahueld clarifying - are you working on
this yourself?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/netlify/netlify-cms/issues/789#issuecomment-468343543,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AApFF9Pt4CG0Qez9nGCuRLrJeKilQyiaks5vSAZGgaJpZM4QYpD9
.

Sounds good. I'd like to provide built in translations that can be used based on browser locale, that would be worth some additional kbs.

I would rather have that configurable, since I will know the language of the editors on my pages in advance. Plus this will only be one language. So having all translation for many different languages available but completely unused doesn't justify these additional kbs in my opinion. But ymmv, of course.

@nahueld  Any updates on this or work being done? Would like to contribute if needed.

Any updates on this? I've seen @nahueld already started working on this in https://github.com/nahueld/netlify-cms/commit/abd6ced25b3500e602c460aab3d52677e4123715
How can we help?

If anyone wants to contribute please do! The plan is here: https://github.com/netlify/netlify-cms/issues/789#issuecomment-417174441, but it should also be stated that we don't want to hardcode all languages, they should be registered as extensions. So a bit of API architecture thinking will be required.

Looking forward to this! I would join the translation when #2528 lands


Device owner consent should always override Device user consent

I personally think this should be reworded:
"Software should not include an option that configures it to hide activities associated with storing or transmitting your data."
to:
"Software may include options for hiding activities associated with storing or transmitting data, but these options must be manually activated by the user and may never be pre-checked or pre-enabled at software installation. An example of acceptable behaviour is to have a non-checked checkbox "Do not show this dialog again", which, upon checking it, will store the choice ("Consent" or "No Consent" to data collection) and always apply that choice, and never show the dialog again."
I do understand that this means a computer owner could install a spyware, and then potentially spy on other people that uses the same computer, but if you use a computer which is not yours, you cannot demand to software developers to have to consent to data collection. You can sometimes require it from the computer owner (if the computer owner is an organization and are bound by GDPR) but not directly from the software owner.

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

ID: b16da7d1-e552-e274-9406-5390252b0c04
Version Independent ID: d0c9b8a8-4fba-4ea8-cfdd-6ae6d67a2e19
Content: How Microsoft identifies malware and potentially unwanted applications - Windows security
Content Source: windows/security/threat-protection/intelligence/criteria.md
Product: w10
Technology: windows
GitHub Login: @levinec
Microsoft Alias: ellevin

If the user purposely install malwares on other computers which does not protected by windows defender or antivirus softwares. then the data will be compromised.
So that's why Microsoft Gives safety precautions to all administrators , should restrict the users to installing softwares by giving only guest account.
As

Hi, thank you for your suggestion. According to the PM, we understand the feedback. However, this scenario was considered when defining this criteria and this is what we wanted. We do not allow apps that collect and transmit your activities or data to have a “stealth” or “hidden” mode option.


Timeout value is being ignored - a 75 sec timeout is used instead

Hi,
I'm using version 3.0 with Swift 4.
When connecting to a web socket server, the timeout interval is being ignored, since the "disconnect" callback is only triggered after 75 seconds, instead of the timeout value that I define (I've tried several values like 5 or 10 seconds, but I get the same result).
In order to test the timeout, I'm disabling the WiFi in my iPhone before trying to establish a connection with a web socket server.
Here's the code snippet I'm using when defining the url timeout:
            var urlRequest = URLRequest(url: url)
            urlRequest.timeoutInterval = TimeInterval(10)
            webSocket = WebSocket(request: urlRequest)

Is there anything I could be doing wrong?
Thanks!

I've also tried to implement my own login timeout logic, using a dispatch source timer, and by forcing the web socket to disconnect when the timeout is reached. However, I'm not able to interrupt an on-going connection attempt when forcing the web socket to disconnect -
webSocket.disconnect(forceTimeout: TimeInterval(0)). The connection only times out after 75 seconds.
Does the disconnect function only work after a connection is successfully established?

I'm experiencing the exact same issue on 3.0.4. I have my own timeout which forces the connection to be shut down. Using webSocket.disconnect(forceTimeout: TimeInterval(0)) freezes the UI for more than a minute. Odd thing is, if I use webSocket.disconnect(forceTimeout: nil) I manage to shut down and reestablish the connection.
Do you have any thoughts on what the issue might be? Can you confirm that 3.0.4 includes the fix?


Create simulation subpackage

This is a maintenance PR that only splits the existing functionality from simulation.py into submodules of a new simulation subpackage.

Hi @AVHopp, @Scienfitz. This time in atomic chunks, I promise :D The PR really does nothing but splitting the code. I've even kept intermediate commits such that you can easily follow what is being moved and deleted, step by step, if you investigate the individual diffs. Should take only a few mins to review 🙃


Install Snapshot v1 api

I was reading through the metadata service project of datafuse to see how you are using this library, and read something related to the v1 api for snapshots. I couldn't find anything in the docs, was curious is this some kind of upcoming change which is not in the 0.8 branch yet?

There are some short introduction to the three snapshot related API in the getting-started-tutorial.
And you can find API docs here: https://docs.rs/openraft/latest/openraft/storage/trait.RaftStorage.html#tymethod.install_snapshot
There are also two example applications that show how to deal with snapshot, one of them is:
https://github.com/datafuselabs/openraft/tree/main/examples/raft-kv-rocksdb
What kind of information were you looking for?
And yes databend-meta has been using unpublished 0.9 branch, in which there are some unstabilized APIs.
If you do not quite care about breaking changes during upgrading, you can just use the 0.9 branch via:
openraft = { git = "https://github.com/datafuselabs/openraft", branch = "release-0.9" }

@drmingdrmer Thanks for the responses! Yeah I was wondering what was going on in 0.9 and if I should try to migrate already.


Fix for TORQUE-627

This pull request contains a fix for TORQUE-627. It provides support for the JRuby --profile.api command line option.

Applied with changes for our currently-used rails version in integration tests.


Add support for callback when progress indicator created using window.withProgress is clicked

Consider the following Discovering Python Interpreters indicator for example:

Sometimes discovering all interpreters can take longer than expected, but meanwhile users only need to select the one interpreter to get started which may already be discovered, users are often tempted to click on the progress indicator only to expand the notification:

We would instead like to show the current progress when a progress indicator is clicked, which in this case would be the currently known list of interpreters:

I understand the request but I wonder if we should tackle this as a platform (and avoid extensions using quick pick individual progress task). The idea of the progress API is to show an overarching message like "Do stuff" and then sending updates for individual work pieces. Today, these pieces are just string or number but it can be extended with more properties that a richer UI would show. This means VS Code could come with a "progress pane"  that expands when clicking the "progress summary" and from there certain commands can be triggered.

Didn't we solve this for our remote extensions that show progress for as long as we install the remote server and provide rich links in the notification that users can click to open the terminal?
@chrmarti @aeschli do you know how remote extensions trigger this?

	this._resolvePromise = vscode.window.withProgress(
		{
			location: vscode.ProgressLocation.Notification,
			title: `[Starting VS Code in WSL (${distro.name})](command:remote-wsl.showLog)`,
			cancellable: false
		},
		progress => {
		   //...
		   progress.report({ message: 'Starting Linux Subsystem' });
		}
	);

There are pros and cons to showing content as it loads:

pro: if the one you want appears earlier you can select it and save time
con if the quick pick does any type of grouping of items (to make it easier) then you can't do this as items will shift around as someone attempts to select

One of the biggest UX annoyances is clicking on an item, as things are loading, and the list shifting and you end up clicking the wrong one. While in the proposed gif that's not exactly happening (the titles are just updating as it learns about each item) this means the user has to make more effort to discern what changed, which gives the impression that the list changed. I'm not sure that is helping the UX.

I think this is a good approach 👍


Consider workaround for Oracle JDBC driver getObject(..) returning oracle.sql.TIMESTAMP

When comparing datasets from Oracle tables to stored CSV files with expected data, I noticed that the getObject(..) methods on result sets from the Oracle JDBC driver return oracle.sql.TIMESTAMP, not java.sql.Timestamp.
This leads to  assertEquals(..) failing because the different timestamp objects are not equal, even when they represent the same point in time.
A workaround according to this discussion would be to use getTimeStamp(..) instead of getObject(..) for the TIMESTAMP JDBC type. This could be handled by a code change in org.jdbdt.DataSource#proceedWithQuery.

Hmmm ... this may be tricky, I fear that changing proceedWithQuery in a "type-dependent" way (and other parts) may raise issues with other engines, and I wish to avoid "type-dependent" code as much as possible. So, a method like takeSnapshot returns TIMESTAMP values whereas read returns java.sql.Timestamp values from CSV files. The way forward should be to allow for user-defined input conversions in conjunction with CSV.Format (overriding the default ones; this is already "type-dependent" code anyway). I will get back to this later.

Being able to override the CSV type conversions would certainly be useful.

Our current workaround is to write the Oracle dataset to a temporary csv file, read it back and then do the assertEquals(..) against the dataset read from the stored CSV file that contains the expected data.
This works and protects us even from the other issue regarding BigDecimal values that you fixed on HEAD.
Something like this:
final Table table = table(tableName).columns(columnNames).build(db);

final File csvTempFile = getTempFile(type);
write(takeSnapshot(table), csv, csvTempFile);
final DataSet actualData = read(table, csv, csvTempFile);

final File csvFile = getCsvFile(csvFileDir, type);
final DataSet expectedData = read(table, csv, csvFile);

JDBDT.assertEquals(tableName  + " dataset verification", expectedData, actualData);

I saw your commit. Looks promising. Thanks! Will try it out tomorrow.

I tried the suggested override, but it turns out that the oracle.sql.TIMESTAMP class does not implement equals(..) which means that it inherits equals(..) from java.lang.Object, which checks for object reference identity and of course that is never true in the dataset comparison.
So it looks like this is a dead-end and I will stick to our "compare temporary csv from actual data to stored csv containing expected data" approach.
For the record, I used a CSV setup like this:
	private final Format csv = new Format().hasHeader().alwaysEscapeOutput().useReadConversions()
			.overrideConversion(JDBCType.TIMESTAMP, oracle.sql.TIMESTAMP::new);

I guess this effectively also means that Oracle dataset to Oracle dataset deltas could have unexpected results for other JDBDT users as well, even if they don't use CSV at all. Your Delta logic depends on Row hash codes and Row equality, using Arrays.deepHashCode() and Arrays.deepEquals(..) which will fail for column values that do not have proper equals(..) - and matching hashCode(..) - implementation.
Oracle is obviously a very commonly used database engine and has had that quirky getObject(..) behaviour for a long time now and will likely not change it. Whereas rs.getTimestamp(..) returns java.sql.Timestamp as defined in java.sql.Resultset javadoc, and all JDBC compliant engines and drivers have to adhere to that.
So maybe column type specific reading from the result set would actually be more robust and consistent across database engines?
I am saying all this as personal opinion, not as urgent request. We have a work-around as mentioned above and are happy for now. Thanks for the great work you do and the impressive responsiveness in maintaining this library.
Obrigado!

I tried the suggested override, but it turns out that the oracle.sql.TIMESTAMP class does not implement equals(..) which means that it inherits equals(..) from java.lang.Object,  [...] Your Delta logic depends on Row hash codes and Row equality, using Arrays.deepHashCode() and Arrays.deepEquals(..) which will fail for column values that do not have proper equals(..) - and matching hashCode(..) - implementation.

Damn :(

Oracle is obviously a very commonly used database engine and has had that quirky getObject(..) behaviour for a >long time now and will likely not change it. Whereas rs.getTimestamp(..) returns java.sql.Timestamp as defined in
java.sql.Resultset javadoc, and all JDBC compliant engines and drivers have to adhere to that.
So maybe column type specific reading from the result set would actually be more robust and consistent across >database engines?

It's possible, but I'll have to check this  in depth.  It's hard to bet on JDBC strict compatibility and homogeneous behavior by assorted database engines ... :(
I'll keep the overrideConversion() feature in any case. It will be part of release 1.4.1, due in a few days in principle along with the fix for #71.  But  I'll close this issue otherwise for now.

Additional info: I verified the described behaviour of getObject(..) for following Oracle JDBC driver version:
<dependency>
	<groupId>com.oracle.database.jdbc</groupId>
	<artifactId>ojdbc8</artifactId>
	<version><IP_ADDRESS></version>
</dependency>

Thanks

@impoldo1  Version 1.4.1 has been released.


Fixing dependency conflict: filelock vs lockfile for cachecontrol

cachecontrol replaced the deprecated lockfile with filelock. This change took place since version 0.13.0. Specifically, it requires filelock >= 3.8.0. This change will hopefully fix the build problem.
Please complete the following checklist:

[x] I have read the guidelines in CONTRIBUTING.md.

[ ] I have documented all public-facing changes in CHANGELOG.md.

[ ] This pull request includes code, documentation, or other content derived from external source(s). If this is the case, ensure the external source's license is compatible with scikit-bio's license. Include the license in the licenses directory and add a comment in the code giving proper attribution. Ensure any other requirements set forth by the license and/or author are satisfied. It is your responsibility to disclose code, documentation, or other content derived from external source(s). If you have questions about whether something can be included in the project or how to give proper attribution, include those questions in your pull request and a reviewer will assist you.

[x] This pull request does not include code, documentation, or other content derived from external source(s).

Note: REVIEWING.md may also be helpful to see some of the things code reviewers will be verifying when reviewing your pull request.

@wasade @ebolyen A workaround may be having scikit-bio install both filelock and lockfile, such that whichever version of cachecontrol can pick what it likes.

Or, we can completely ditch cachecontrol. In the scikit-bio codebase, only one place uses this library:
skbio/io/_iosources.py:
from cachecontrol import CacheControl
from cachecontrol.caches import FileCache

class HTTPSource(IOSource):

    def get_reader(self):
        sess = CacheControl(requests.Session(),
                            cache=FileCache(tempfile.gettempdir()))
        req = sess.get(self.file)

        # if the response is not 200, an exception will be raised
        req.raise_for_status()

        return io.BufferedReader(io.BytesIO(req.content))

I suspect that this code can be replaced by Python's built-in code?

Having a mechanism for retrieving remote resources is probably not difficult. GPT suggested the following with just Python's built-in:
    def get_reader(self):
        req = urllib.request.Request(self.file)
        with urllib.request.urlopen(req) as response:
            if response.status != 200:
                raise http.client.HTTPException(f'HTTP request failed with status {response.status}')
            data = response.read()
        return io.BufferedReader(io.BytesIO(data))

Or the following with the requests library, which is widely adopted, and is already a dependency of QIIME 2.
    def get_reader(self):
        req = requests.get(self.file)
        req.raise_for_status()
        return io.BufferedReader(io.BytesIO(req.content))

Both codes don't have a caching mechanism, but it is probably not necessary. As a reference, Pytorch uses the following code for downloading bulkly datasets (likely larger than scikit-bio's). As shown, it merely uses Python's built-in.
def _urlretrieve(url: str, filename: str, chunk_size: int = 1024 * 32) -> None:
    with urllib.request.urlopen(urllib.request.Request(url, headers={"User-Agent": USER_AGENT})) as response:
        _save_response_content(iter(lambda: response.read(chunk_size), b""), filename, length=response.length)

Just for reference... I don't have a preference here.

Agree, however I don't know how this IO source is used so I'm not well positioned to guess the consequences of making a change. If you think this is in scope of maintenance though, then I would not discourage an exploration. If you do, I endorse the use of requests.

I don't know either how HTTPSource is used. It is not mentioned (or unit-tested) anywhere in scikit-bio. I guess that it is a mechanism of downloading data from a remote location, like pandas, scikit-learn and pytorch does. If so, that would be useful. Probably need input from @ebolyen before taking any action.

Revisited in #1863 .


too many arguments for function: matchdelete

Error detected while processing function <SNR>63_update_search_match[11]..<SNR>63_delete_current_match:
line    4:
E118: Too many arguments for function: matchdelete

I use neovim + neovim-qt

I can't reproduce this.  Please could you tell me how you trigger the error?

/some_term then n.
also :noh doesn't hide highlights.

/some_term then n.

Works fine with me.
If you open up the matchdelete() docs with :help matchdelete, does it look like this?
matchdelete({id} [, {win})		       *matchdelete()* *E802* *E803*

:noh doesn't hide highlights

Do you mean the current search highlight?  That will disappear the next time you move the cursor.

matchdelete({id})			       *matchdelete()* *E802* *E803*
		Deletes a match with ID {id} previously defined by |matchadd()|
		or one of the |:match| commands.  Returns 0 if successful,
		otherwise -1.  See example for |matchadd()|.  All matches can
		be deleted in one operation by |clearmatches()|.

I use Arch Linux and currently I have the most recent neovim version available in official repos:
$ LC_ALL=C nvim --version
NVIM v0.4.4
Build type: Release
LuaJIT 2.0.5
Compilation: /usr/bin/cc -D_FORTIFY_SOURCE=2 -march=x86-64 -mtune=generic -O2 -pipe -fno-plt -U_FORTIFY_SOURCE -D_FORTIFY_SOURCE=1 -O2 -DNDEBUG -DMIN_LOG_LEVEL=3 -Wall -Wextra -pedantic -Wno-unused-parameter -Wstrict-prototypes -std=gnu99 -Wshadow -Wconversion -Wmissing-prototypes -Wimplicit-fallthrough -Wvla -fstack-protector-strong -fno-common -fdiagnostics-color=always -DINCLUDE_GENERATED_DECLARATIONS -D_GNU_SOURCE -DNVIM_MSGPACK_HAS_FLOAT32 -DNVIM_UNIBI_HAS_VAR_FROM -I/build/neovim/src/build/config -I/build/neovim/src/neovim-0.4.4/src -I/usr/include -I/build/neovim/src/build/src/nvim/auto -I/build/neovim/src/build/include
Compiled by builduser

Features: +acl +iconv +tui
See ":help feature-compile"

   system vimrc file: "$VIM/sysinit.vim"
  fall-back for $VIM: "/usr/share/nvim"

Run :checkhealth for more info

here I tried to record behaviour https://asciinema.org/a/onOjpL05O45MNnkERnHCgCmWU

Ah, it looks like the win argument was added in Vim 8.1.1084 (vim/vim@aff74914) and isn't yet available in Neovim's release builds (neovim/neovim@d7d69fe).

Thank you for sharing this plugin, helps a lot!


Update rotld whois

Add expires_on and created_on

@weppos I can not run tests because my Mac doesn't want to install json gem. I've tested by hand and all works.


Add troubleshooting hint for blank page to PDF Document Generation module docs

@Luyao-Zhang-1
I didn't get a response from Allard. Could you remind him, if you have time, otherwise I can pick this up when I'm back.


Youtube current point in time embedding does not work

The embedding of youtube videos at a chosen point in time does not work, it either starts at 0:00 or some random point in time.
I have tried different ways of inserting the video such as using https://youtu.be/OLHdzLpMg-s?t=1486 and https://youtu.be/OLHdzLpMg-s?t=1h3m21s but it is not working.
Is that a bug?

This is not a bug of django-summernote. I made a PR for this. Please stay tuned on the issue of summernote: https://github.com/summernote/summernote/pull/2847.


feat: added BTP_NEW_SUBACCOUNT_ADMINS

Description
Changes proposed in this pull request:

added BTP_NEW_SUBACCOUNT_ADMINS input variable as an option to specify usernames that will be assigned the admin and viewer role collections in the newly created subaccount

Related issue(s)

This doesn't work as expected, on destroy BTP complains about "Cannot delete last admin user of subaccount".


super generated too many times when calling other imported .proto messages

I have found a strange error in protobuf generation while I was using Prost in conjunction with Tonic.
Running on
Darwin Kernel Version 21.1.0: Thu Sep 23 21:17:10 PDT 2021; root:xnu-8<IP_ADDRESS>.1~2/RELEASE_X86_64 x86_64 i386 MacBookPro16,1 Darwin

rustc 1.55.0 (c8dfcfe04 2021-09-06)

cargo 1.55.0 (32da73ab1 2021-08-23)

My project has this folder structure
.
├── build.rs
├── Cargo.lock
├── Cargo.toml
├── proto
│  └── org
│     ├── api
│     │  └── general.proto
│     ├── service1
│     │  └── v1
│     │     └── service1.proto
│     └── service2
│        └── v1
│           └── service2.proto
└── src
   └── server.rs

general.proto
syntax = "proto3";

package org.api;

message General {
    string general = 5;
    string description = 6;
};

service1.proto
syntax = "proto3";

import "org/api/general.proto";

package org.service1.v1;

service Service1 {
    rpc Send(Service1Request) returns (Service1Response);
}

message Service1Request {
    org.api.General general = 5;
    string description = 6;
}

message Service1Response {
    string response = 5;
}

service2.proto
syntax = "proto3";

import "org/api/general.proto";

package org.service2.v1;

service Service2 {
    rpc Send(Service2Request) returns (Service2Response);
}

message Service2Request {
    org.api.General general = 5;
    int32 value = 6;
    string description = 7;
}

message Service2Response {
    string response = 5;
}

build.rs
Here I am building just service1 but the same happens with service2.
use std::env;
use std::path::PathBuf;

fn main() {
    tonic_build::configure()
    .build_client(false)
    .compile(
        &["proto/org/service1/v1/service1.proto"],
        &["proto"]
    )
    .unwrap();
}

It seems like it doesn't like when I use org.api.General in service1 and service2 because the build returns this error
error[E0433]: failed to resolve: there are too many leading `super` keywords
 --> /Users/user/Desktop/test-build/target/debug/build/test-build-25050f1b68bb5218/out/org.service1.v1.rs:4:48
  |
4 |     pub general: ::core::option::Option<super::super::api::General>,
  |                                                ^^^^^ there are too many leading `super` keywords

The fix that I've found for the moment is to just include General both in service1 and in service2 with code duplication, how can this be solved?
Project here
test-build.zip

I solved this with the include_file function.
That said, 2 caveats.

I don't think it's in a release yet so you would need to patch locally with

[patch.crates-io]
prost = { git = "https://github.com/tokio-rs/prost" }

There's a bug in it if you are using it with out_dir.  I've submitted this PR, but I don't believe it's been merged.  You could probably patch off that fork though.

I realize this a workaround until a release, but hopefully it gets you unstuck.

So either you need to use that new feature which will ship with prost 0.9 or move all the proto files to be in the same folder I believe this will also fix your issue.

Turns out I was importing stuff incorrectly, I saw the correct version generated by prost 0.9 and it works now. Closing this issue


Looking to use this tool to help with csv selections

Hey great project! I'm working with a csv file and would like to do a multi line select on certain values between commas. For example:
apple fritter, pear, banana
string, shoe, elephant,
1, 2, 3
I'd like to do a multi select on apple fritter, string, 1 so I can put a cursor before and after them
Then I'd like to go to the next occurence of stuff between commas which would select pear, shoe, 2
and continue on.
How would I write the key binding to do that?

You want two cursors (the |'s below) like:
|apple fritter|, pear, banana
|string|, show, elephant
|1|, 2, 3

and then
apple fritter, |pear|, banana
string, |show|, elephant
1, |2|, 3

and so on?

Yes like that :) The cursors can be there either simultaneously or one after the other.
Basically I have lots of rows of CSV data where some "columns" need quotes around them. If I can select what's inside the columns with a multiline select and have it move to the next "column" on each round then I can easily add the quotes.

It took a little while because I found a couple of bugs in the meantime.  Use these two keybindings in your keybinndings.json:
{
  "key": "alt+f",                     // whatever keybinding you want
  "command": "findInCurrentFile",
  "args": {
    "find": "(?<=,\\s|^)([\\w ]+)(?!.*\")",
    "isRegex": true,                  // no replace
    "cursorMoveSelect": "$1",         // important, to select the $1 capture group
    "restrictFind": "once"            // just do first match after cursor
  }
},

{
  "command": "findInCurrentFile",
  "key": "alt+'",
  "args": {
    "replace": "\"${selectedText}\"",   // no find!
    "restrictFind": "line"              // line
  }
}

Here is a demo:

So you first need to put a cursor on all the lines you want to work on - at the beginning of the lines.
Then, alt+f to select the next match, alt+' to surround that match with quotes or alt+f again to skip and select the next match.
The only issue right now is if there are duplicates on a line, like:
apple fritter, pear fruit, apple fritter

then both of those apple fritters would be quoted when the first one is - I'll have to look into that more...
Let me know if this works for you.  Thanks.

Closing as fixed.


Participants: Inactivity visibilty issues with new permissions

Description: Some issues occur with the inactive attribute in the participants list. The default value was, that a user with the permission "Can see participants" can see the inactivity symbol behind a user in the participant list and in the user detail view, if the user is inactive. With the new staging this doesn't happen anymore. Instead the inactivity symbol is not correctly shown or not shown and the filter about activity doesn't work anymore.

[ ] Filter "Active" doesn't work anymore.
[ ] users with permission "Can see participants" can't see the inactivity icon of an inactive user in the participant list or the user detail
[ ] user with permission "Can see sensitive data" and "Can see participants" can't see inactive icon in list nor the user detail view of users who are inactive
[ ] user with "Can update participants", "Can see sensitive data" and "Can see participants" can't see inactive icon in listing. Additionally they see a wrong inactive icon in the user detail view, for users, which are not inactive

Reproduction:
First four users are needed: User 1 in group Admin, user 2 in group staff, user 3 as delegate and user 4 as delegate

open the instance in two browser. log in as user 1 in browser 1 and as user 2 in browser 2
open a meeting
open participants in both browsers
as admin: set user 3 as inactive
open as admin the group permission page > grant staff in participants only "Can see participants"
as user 2: check, if you can see the inactive icon in the listing -> should not be visible
as admin: grant permissions "Can see sensitive data" and "Can see participants" for staff
as user 2: check if icon is visible in the list and if the icon is visible in user detail view
as admin: grant permissions "Can update participants", "Can see sensitive data" and "Can see participants" to staff
as user 2: icon should not be visible in listing. Additional the active user - user 4 - is shown as inactive in the user detail view.

What should happen:

If user has the permission "Can see participants" all inactive users should have an icon behind them.
If a user has "Can see sensitive data" he can also open a user detail view and see the activity icon
If a user has "Can update participants" he can additionally update the activity of the user
the activity filter option should work for all users that have at least the permission "Can see participants"

I have tested the problem with a can_update user. See #3549 .

In the meta/models.yml:
user:
 [...]
  is_active:
    type: boolean
    restriction_mode: D

In the autoupdate repo (internal/restrict/collection/user.go):
// User handels the restrictions for the user collection.
[...]
// Mode B:
//
//	Y==X
//	Y has the OML can_manage_users or higher.
//	There exists a committee where Y has the CML can_manage and X is in committee/user_ids.
//	X is in a group of a meeting where Y has user.can_see_sensitive_data.
//
// Mode D: Y can see these fields if at least one condition is true:
//
//	Y has the OML can_manage_users or higher.
//	X is in a group of a meeting where Y has user.can_manage.

This seems to be one problem.


KotlinVersion is not being taken correctly to the classpath

If you override the kotlin version like this:
autoConfigure {
    kotlin {
        kotlinVersion.set("1.6.21")
    }
}

Then you still have the latest kotlin version on the classpath.
If you run the tracker report you see the following:
    "compile": [
        {
            "artifact": "org.jetbrains.kotlin:kotlin-bom:1.6.21",
            "classifier": null,
            "type": "bom",
            "trail": [
            ]
        },
        {
            "artifact": "org.jetbrains.kotlin:kotlin-stdlib:1.7.10",
            "classifier": null,
            "type": "jar",
            "trail": [
                "org.jetbrains.kotlin:kotlin-bom:1.6.21",
                "org.jetbrains.kotlin:kotlin-stdlib-jdk8:1.7.10"
            ]
        },
        {
            "artifact": "org.jetbrains.kotlin:kotlin-stdlib-common:1.7.10",
            "classifier": null,
            "type": "jar",
            "trail": [
                "org.jetbrains.kotlin:kotlin-bom:1.6.21",
                "org.jetbrains.kotlin:kotlin-stdlib-jdk8:1.7.10",
                "org.jetbrains.kotlin:kotlin-stdlib:1.7.10"
            ]
        },
        {
            "artifact": "org.jetbrains:annotations:13.0",
            "classifier": null,
            "type": "jar",
            "trail": [
                "org.jetbrains.kotlin:kotlin-bom:1.6.21",
                "org.jetbrains.kotlin:kotlin-stdlib-jdk8:1.7.10",
                "org.jetbrains.kotlin:kotlin-stdlib:1.7.10"
            ]
        },
        {
            "artifact": "org.jetbrains.kotlin:kotlin-stdlib-jdk7:1.7.10",
            "classifier": null,
            "type": "jar",
            "trail": [
                "org.jetbrains.kotlin:kotlin-bom:1.6.21",
                "org.jetbrains.kotlin:kotlin-stdlib-jdk8:1.7.10"
            ]
        },
        {
            "artifact": "org.jetbrains.kotlin:kotlin-stdlib-jdk8:1.7.10",
            "classifier": null,
            "type": "jar",
            "trail": [
            ]
        }
    ]

That means, we are applying org.jetbrains.kotlin:kotlin-bom with the correct version, but someone (probably the Kotlin plugin) is also adding org.jetbrains.kotlin:kotlin-stdlib-jdk7:1.7.10, and that is overriding the version.
The problem seem to be in the KotlinConfigurePlugin in that block:
        kotlin.sourceSets.maybeCreate("main").dependencies {
            // see https://kotlinlang.org/docs/gradle.html#dependency-on-the-standard-library
            // as we allow clients to override the Kotlin Version, (i.e. to 1.5.20), we also want to ensure
            // that all kotlin standard libraries from exactly that Kotlin version is being added
            // to the dependencies. That's why we add the kotlin-bom in exactly our version here.
            // Without those lines, we would always add the stdlib in the version
            // of the underlying Kotlin Gradle Plugin (1.7.10 at the time of that writing)
            this.implementation(kotlinConfigureExtension.kotlinVersion
                .map { project.dependencies.platform("org.jetbrains.kotlin:kotlin-bom:$it") }
            )
            this.implementation(kotlinConfigureExtension.kotlinVersion
                .map { "org.jetbrains.kotlin:kotlin-stdlib-jdk8" }
            )

It seems that the correct kotlinVersion is not being taken here

it seems that this has to to with the autoConfigure plugin, because there is a KotlinConfigurePluginTest with a single-kotlin-module-override-kotlinversion fixture and that only applies the io.cloudflight.autoconfigure.kotlin-configure plugin:
kotlinConfigure {
    kotlinVersion = "1.5.20"
}

Probably the AutoConfigureExtension erronously overrides that.

released in version 0.7.0


Add PVC labels to PV

Describe the solution you'd like
A PVC definition file can include a bunch of labels.
However, Trident does not use them when it creates the PV corresponding to the PVC.
It would be great if Trident could retrieve these labels & add them to the PV definition.
Describe alternatives you've considered
The sole alternative is to manually edit the PV & add labels
Additional context
WIthout these labels, it can be challenging to get the PV associated to an application.

Hi @YvosOnTheHub,
Trident isn't directly in the path to be able to perform this function. This is a feature that the CSI community would need to implement in the external-provisioner sidecar.


Update documentation

This PR includes some minor changes about

Correcting rendering the math expressions in LaTeX mode
Changing epsilons to varepsilons for more convenient representation

@mkitti - thank you for accepting the PR.


Kube Summary: Open Terminal button is shown when kube terminal config is missing

tech preview enabled, no kube terminal config supplied
WARN[Mon Oct  5 16:42:01 BST 2020] Kube Terminal configuration is not complete   shown in jetstream log

Fixed by #4635


stm32: i2c bus failure when using USB

I am creating this issue as a starting point. When I get some time I will need to gather more information and work towards finding a minimal way to repro so that others can help guide me towards a solution. But for now this issue is a bit of a blocker for my application as it renders I2C pretty much unusable on stm32.
Describe the bug
I have an application (targeting stm32f411e) which uses USB as a console device in the CDC mode of operation, as well as USB as a mass storage device. On their own, these work perfectly.
The application also has a couple of (out-of-tree for now) I2C sensors, built into zephyr with the sensor API. I eventually intend to upstream these, but I do not believe there is anything wrong with the specific implementation of these sensors.
I am polling these two sensors once per second in sequence. On their own, these work perfectly fine. If I power the device with a standalone power supply, and don't perform any USB communication, the application operates as expected for days on end.
However, when I actually run the application with both USB and I2C active, issues (eventually) occur. If I open the console (writing logs), or access the mass storage device periodically, for example, then eventually both USB and I2C hang. Other threads which do not rely on those subsystems continue to operate normally.
I have captured a view of SCL under normal conditions here

And the moment the hang up begins, this is the output

At this point, no activity occurs on SCL.
After some discussion with other users on Slack, I eventually tested operation with CONFIG_I2C_STM32_INTERRUPT=n
This stopped the problem from occuring (in the same way). Normally, the hang up would occur after 1-5 minutes. Now, it operates fine for 45 minutes to an hour, but once again crashes. However, it no longer crashes the USB, only I2C crashes. Now, the thread with sensor operations no longer blocks, and the sensor fetch function returns -EIO.
Enabling I2C_LOG_LEVEL_DBG shows there is something going on in the msg_end function as I continuously get
<dbg> i2c_ll_stm32_v1.msg_end: stm32_i2c_msg_write: NACK
I tried recovering the I2C bus with i2c recover from the shell, but this does not appear to be implemented (perhaps I can figure out a recovery method and add the api function for i2c_recover?)
That is as far as I have been able to debug so far, but will have more time in the coming days to look into this.
Update:  I was able to manually recover operation by toggling SDA (manually). That is a good sign that a recovery procedure is possible if we can bitbang the i2c gpio.
To Reproduce
I do not yet have steps to easily reproduce this without writing some minimal code. I will work on this later in the week.
Expected behavior
I2C should work as normal, even if USB is active. There should be a means of recovery if the bus does crash. CONFIG_I2C_STM32_INTERRUPT should not cause I2C and USB to crash in the event of a bus failure.
Impact
Showstopper for my application.
Environment (please complete the following information):

Windows 10 OS
gnuarmemb toolchain
tested on latest master

Additional context
This problem does not occur on nordic hardware, with all else held constant (same wiring/sensors, etc).

There still seems to be some places in the I2C driver that can cause an endless IRQ stream when something goes wrong on the bus (EMC spikes). I have seen this today with my hardware, it just endlessly generates IRQ's but never gets ready, since the load on the system gets so high that the timeout code never gets executed and my hardware watchdog resets the system.
The fixes it the driver that reset the bus when something goes wrong simply never get executed because of the high IRQ load, so there is certainly still a bug in the driver.
When using the non-irq driver the hang also happens, but since it doesn't endlessly generate IRQ's the rest of the system keeps functioning. I added the bus-reset code to the non-irq error path and that seems to make things better for me.
@bpbradley can you see in the debugger what the I2C driver is doing when it hangs? Is it also endlessly handling IRQ's ?

There still seems to be some places in the I2C driver that can cause an endless IRQ stream when something goes wrong on the bus (EMC spikes). I have seen this today with my hardware, it just endlessly generates IRQ's but never gets ready, since the load on the system gets so high that the timeout code never gets executed and my hardware watchdog resets the system.
The fixes it the driver that reset the bus when something goes wrong simply never get executed because of the high IRQ load, so there is certainly still a bug in the driver.
When using the non-irq driver the hang also happens, but since it doesn't endlessly generate IRQ's the rest of the system keeps functioning. I added the bus-reset code to the non-irq error path and that seems to make things better for me.
@bpbradley can you see in the debugger what the I2C driver is doing when it hangs? Is it also endlessly handling IRQ's ?

Thanks for the comment, and I appreciate the help you gave in Slack. I will try to test with a debugger attached this afternoon if I can. Should I test with STM32_INTERRUPT enabled or disabled? I assume enabled.
Your theory makes sense though. I think I am realizing my specific problem is being caused by some emc issue as you say (my prototype is breadboarded right now), which is why creating a repro procedure with different hardware might be tricky, but it is the inability to recover which is problematic. The same exact wiring on nordic hardware does not present this issue.
If we have a pinctrl api which allows me to re-purpose sda/scl at runtime to implement the i2c bus reset sequence in the bus_recover api, then I think it will at least allow me to recover in the non-interrupt mode of operation. But the total system failure associated with the interrupt mode of operation is worrisome.

@bpbradley I made a PR #27653 that resets the I2C hardware on error when using the non-irq setting. Can you give this a try an see if your software recovers from the error?

@bpbradley I made a PR #27653 that resets the I2C hardware on error when using the non-irq setting. Can you give this a try an see if your software recovers from the error?

Testing now. Will probably know if it worked within an hour.

@lowlander For some reason it took a long time to reproduce this time. But it finally did, but still didn't recover properly. It did result in different behavior still though. I now get the log message <dbg> i2c_ll_stm32_v1.stm32_i2c_generate_start_condition: stm32_i2c_generate_start_condition: START while STOP active!
Again in this case, manually asserting SDA fixed the issue and allowed it to properly recover.

@bpbradley don't hesistate to re-open if issues shows up again.


ASHELYS must exit non-zero if not input files are found

v0.2.0 of ASHLEYS happily produces empty output if no input BAM files are found for feature generation (e.g., typo in the file extension). No valid input files or, in general, producing empty output must always produce an error!

fixed in #b9bcbe96f4cf5e6cf36462a8e885840304800426 / current dev


Proposed API Changes:

I wanted to propose this while this is still an internal effort (i.e. outside users don't appear to be using this pakcage yet):
CC @Andesha and @jadesjardins to chime in:

Current API
Proposed API

flag_ch_sd_ch
flag_noisy_channels

flag_ch_sd_epoch
flag_noisy_epochs

flag_ch_low_r
flag_uncorrelated_channels

flag_epoch_low_r
flag_uncorrelated_epochs

flag_ch_bridged
flag_bridged_channels

flag_ch_rank
flag_rank_channel

The new naming convention aims to be more pythonic and is inspired by MNE / pyprep - For example you have mne.preprocessing.compute_bridged_channels not compute_some_metric_abbreviated
Outside users are not going to be familiar with our internals and so something like ch_sd_ch is vague. I think it is better to communicate conceptually what the function does.

Big fan of this as long as the reference documentation mentions the old names ;)

I've given it some more thought - I don't really like "noisy" as  name for ch_sd...

we shouldn't use "uncorrelated" again because we need to differentiate between the flags of the different steps.
"bad" is probably not a good alternative bc (at least in mne) "bad" is a special term.
The function basically calculates the std. across the samples of each epoch. The heuristic is, if the std is high, it's indicative of the channel being what we'd colloquially call "noisy" during that epoch. So for me "noisy" felt like a natural label.
you maybe could say the same about the flag_uncorrelated step but to me this is a more fine grained/low-amplitude artifact and that's why it is not picked up by the "noisy" step.


Document that it's compatible with python 2.

It doensn't work on python 3.

Documented in 8842aa11f99262a90d3b3e453f1b36cefc587505


[SPARK-35171][R] Declare the markdown package as a dependency of the SparkR package

What changes were proposed in this pull request?
Declare the markdown package as a dependency of the SparkR package
Why are the changes needed?
If we didn't install pandoc locally, running make-distribution.sh will fail with the following message:
— re-building ‘sparkr-vignettes.Rmd’ using rmarkdown
Warning in engine$weave(file, quiet = quiet, encoding = enc) :
Pandoc (>= 1.12.3) not available. Falling back to R Markdown v1.
Error: processing vignette 'sparkr-vignettes.Rmd' failed with diagnostics:
The 'markdown' package should be declared as a dependency of the 'SparkR' package (e.g., in the 'Suggests' field of DESCRIPTION), because the latter contains vignette(s) built with the 'markdown' package. Please see https://github.com/yihui/knitr/issues/1864 for more information.
— failed re-building ‘sparkr-vignettes.Rmd’

Does this PR introduce any user-facing change?
Yes. Workaround for R packaging.
How was this patch tested?
Manually test. After the fix, the command sh dev/make-distribution.sh -Psparkr -Phadoop-2.7 -Phive -Phive-thriftserver -Pyarn in the environment without pandoc will pass.

Test build #137722 has started for PR 32270 at commit ad9a572.

Test build #137722 has finished for PR 32270 at commit ad9a572.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/137722/

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/42249/

Kubernetes integration test status failure
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/42249/

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/42249/

cc @HyukjinKwon

Merged to master.

Thank you, @xuanyuanking and @HyukjinKwon .
During testing branch-3.1, I saw the Error message.
I'll backport this to branch-3.1.

👍


Azure Extension in VS Code does not allow switching tenants.

When logging into Azure via the extension, it automatically selects one of the tenants the user is a member of. I need to deploy to a different tenant the user is also a member of. The extension has no option to switch to the right tenant.

Hi @onefox-dev, we are working on adding the ability to manage and switch between multiple tenants in #869


Doesn't work on MacOS Mojave 10.14.2

I've been struggling to make the @journeyapps/sqlcipher work on a mac without any success for a few days already.
I successfully installed brew install openssl
Used both yarn and npm installation. The installation part is done successfully.
However, when I run the example code from readme file using node I'm getting the following error

Vitaliis-MacBook-Pro:cipher-test vbratok$ node index.js
events.js:173
throw er; // Unhandled 'error' event
^
Error: SQLITE_ERROR: SQL logic error
Emitted 'error' event at:
Vitaliis-MacBook-Pro:cipher-test vbratok$

That's it and nothing more.
The test.db file is created in the folder, but it's empty and has zero bytes.
If I comment out the db.run("PRAGMA key = 'mysecret'"); line, it works file, but db isn't encoded.
I did a lot of experiments and I checked that node-sqlite3 and SQLCipher work fine separately and together when I bind them together as described here
p.s. I noticed that there're a new version of openssl came out since the latest release of node-sqlcipher https://www.openssl.org/source/. Could it be an issue?

Same issue here!

Same issue here!

Appears to have been an issue with openssl 1.0 on macOS, which seems solved with openssl 1.1.
Version 4.0.0 have now been released, with pre-built binaries for Mac based on openssl 1.1.


LTD-5452-feedback-content-change

https://uktrade.atlassian.net/browse/LTD-5452
New feedback form where the exported get a link to the support-help desk link.
Had to refactor and create an exporter specific feedback form.
Also took this opportunity to use our standard base crisp_form.

when trying to submit the form with some text in, even as a logged in user, I get this error:
403 - [{'error': 'AuthError', 'message': 'Invalid token: service id is not the right data type'}]

possibly this is me doing something wrong, have you seen this error before?

Could be quite possible you don't have email/ notify api not setup locally


Upgrade video player in Gallery to newest version

Version 0.6.0 of video_player plugin uses ExoPlayer instead of MediaPlayer on Android. The new dependency on ExoPlayer Java packages necessitates down-grading our Android test dependencies for compatibility with ExoPlayer transitive deps. Not down-grading caused the build error and roll-back of https://github.com/flutter/flutter/pull/18471

cc @DanTup


Add, Sub, Sum

Reorgs arkworks stuff into own module
Adds parity tests
Sum is still not working

super nice, LGTM!


Updated to support automatically installing the new 'future' dependency.

Added requirements.txt with 'future' dependency.
Updated setup.py to use setuptools rather than distutils and added an
install requires section with 'future' dependency.
Updated README.md to reflect changes to installation.  Fixed a typo as well.

Merging patch from Keith Sharp


Use gcc 10 on CentOS 7, CentOS 8, RHEL 7 and RHEL 8.

@aressem : please review
@baldersheim : FYI

This PR should not be merged until vespa-boost-1.76.0 for CentOS 7 has been successfully built on Copr.


provider doesn't detect duplicate resource names.

Describe the bug
Terraform applies without error but only one EDL is created on the firewall.
resource "panos_edl" "whitelist-customer-a" {
  name        = "whitelist-customer-a "
  description = "The EDL used to whitelist urls for customer a"
  repeat      = "every five minutes"
  source      = "http://${var.domain}/whitelist-customer-a
  type        = "url"
}

resource "panos_edl" "whitelist-customer-b" {
  name        = "whitelist-customer-a "
  description = "The EDL used to whitelist urls for customer b"
  repeat      = "every five minutes"
  source      = "http://${var.domain}/whitelister-customer-b
  type        = "url"
}

Expected behavior
I would expect Terraform to throw an error that two resources have identical names.
Current behavior
What actually happens is only a single EDL is created with the name "whitelist-customer-a" but with the details of the second resource "whitelist-customer-b".
Possible solution
Use of variable definitions might make a typo like this less likely to happen
Steps to reproduce
As per code block above.
Screenshots

Context

Your Environment
Terraform v0.13.4 + provider registry.terraform.io/paloaltonetworks/panos v1.6.3
Panorama 10.0.2

In order for this type of check to happen, the Terraform plugin SDK would need to provide a hook for me to declare how an individual resource's ID is created.  This would allow Terraform to determine if the user's config contains duplicates or not.
The plugin SDK would need to be updated first to allow for the declaration of resource ID construction, but even if/when that happens it would be Terraform itself that does this check, not the provider.  The way Terraform providers are invoked, we are never given the full config to be deployed anyways, so this type of check could never live in the provider.


Feat: Consolidate Ginkgo Reports for multiple Test Suites as a single Test Run

Context:
In the initial design of the Fern Ginkgo client, we assumed the test suite configuration would reside at the root level of a project. However, Ginkgo's recommended practice is to have a test suite for each package.
This issue aims to aggregate all the reports from individual test suites into a single test run.

The implementation involves sending each test suite report directly to the fern-reporter and updating the existing TestRun record in the DB if the report belongs to an existing TestRun based on a unique ID.
We use the RandomSeed unique identifier part of the Report object sent by the fern-client. This unique ID remains the same for all test suite report objects that belong to a single Test Run. This RandomSeed value is the number of milliseconds elapsed since January 1, 1970 UTC.
There can be chances for a collision if two TestRuns execute at the very same millisecond. In this current issue, we won't be handling collisions by adding prefixes. Later when we get to work on handling collisions, we intend to generate a hash in the fern-client (with Random Seed + Commit SHA + Branch), but there might be cases where the sha and branch could be empty. So we will also explore if there is any OS-level ID we can use when we get to this enhancement.


How to integrate library with API call like Retrofit on Java/kotlin

First of all, thanks for your nice repo.
In my case, i need to call the restful app then return the Observable<Result> when complete via emiter like Rxjava into onNext, onError or complete
The Issuse I meet is I can't make the API call with network without await / async keywork. If the functioin has await keyword, it require async keyword then Future required. But exactly type we need in return is Observable<Result>
Bellow is my code. I need function getUserObservable return <Observable<UserResponse>> instead Future

code link: https://gist.github.com/hoangduchuu/1622a5b511e006d3b18016d7646da157

Convert the future to an observable, the use RX operators to transform, handle errors, etc.
Observable<UserResponse> getUserObservable() =>
    _dio.get(_endpoint).toObservable()
        .map(response => UserResponse.fromJson(response.data));

Convert the future to an observable, then use RX operators to transform, handle errors, etc.
Observable<UserResponse> getUserObservable() =>
    _dio.get(_endpoint).toObservable()
        .map(response => UserResponse.fromJson(response.data));

I can't find toObservable function. Could you please more details.
Thanks!

The IDE notice me

Can't find the toObservable because my library version is 0.0.3. Upgrade to 0.0.4 and it work!
Thanks @renggli

One more thing, How can I do the late function after observable complete like flatMap.
Example:
Observable getUserObservable() =>
_dio.get(_endpoint).toObservable()
.map(response => UserResponse.fromJson(response.data)).flatMap((user)=>otherObservableCall())

Not sure what you mean by do the late function?

That mean I have more than one function.
The second function will call after the first function complete. The Thirt function call after second complete
In Java or kotlin we have flatmap. How can I do it with this library

Ahh, flatMap is conceptually the same as the more generic mergeMap (https://github.com/renggli/dart-rx/blob/362efafaf43986408b8041259f85d1257eeb96e2/lib/src/operators/merge.dart#L31). I should probably add an alias for flatMap, because this is such a well known name ...

Thanks in advanced!!


Do not run the babel helper in Development Mode. (#224)

This PR adds a check to the babel helper to see if we are currently in the Development Mode. If
so, the enclosed source code is not run through Babel, it is returned verbatim. This is a preview
optimization, since the Babel-ification is a relatively slow operation. It's not necessary for
preview either, assuming devs are not previewing their changes in IE11.
J=SLAP-1312
TEST=manual
Ensured that when in Development mode, the helper was a simple pass-through. In Production mode,
the enclosed code was Babel-ified.

Coverage increased (+0.007%) to 46.379% when pulling d8791c2d156c7ad079c0c4858e4e5ee5f5f5363a on feature/optimize-babel-helper into 08570c2f56b2312c37863189f3b2e3794dd6529a on develop.


ci: metrics: run 'make install' with raised privs

The metrics ci script should not have to be run as root, as it uses
sudo to raise privs when necessary - but, it invokes a 'make install'
if there are missing commands, and that needs privs to install into
the default system dir paths. Add the relevant sudo.
Fixes: #569
Signed-off-by: Graham Whaley<EMAIL_ADDRESS>

lgtm


Making it so Model fields can be properly tracked even if a value is  …

set before the Tubular Model has been created

Coverage decreased (-0.01%) to 27.583% when pulling 397a35d5319e2b04ac6f990491a42a1c11382b4c on luisgonzalez:master into 736c0852f565d81157efa5c3f50db8bb91781fb5 on unosquare:master.