Switch capistranorb.com to https

GitHub Pages supports it now: https://blog.github.com/2018-05-01-github-pages-custom-domains-https/
Looks like we are using A records (probably because we are using the root domain instead of a subdomain like www). So I'll update dnssimple with the new A records, enable https, verify it works, make any fixes as needed, then redirect to https by default.
Any objections? cc/ @mattbrictson @leehambley

Absolutely 👍 Thanks for looking into this!

🤘👍👍
Sent from my phone from underway.
On Thu, 21 Jun 2018, 20:22 Matt Brictson<EMAIL_ADDRESS>wrote:

Absolutely 👍 Thanks for looking into this!
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/capistrano/capistrano/issues/1990#issuecomment-399198253,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AABKCMAGb01ZOCtq6SuAYkYNJVcWwGxVks5t--RUgaJpZM4UyiMC
.

DNS is cut over. I switched the apex domain over to the new A records, and I switched the www redirect subdomain over to a CNAME from the previous A records. It looks like GitHub has issues a Let's Encrypt cert at this point.
I'm going to give this some time for the DNS changes to propagate, then audit the site for any http dependencies that'll cause mixed content warnings. Finding none (or fixing the ones that exist), I'll enforce https.

Nice work!

Looks like things are good now. I've turned on the https redirect. Feel free to let me know if there are any other issues.


New DA segmentation paper

Hi, @zhaoxin94, thx for sharing. Could you please add our new TPAMI 2023 paper on Domain Adaptive Semantic Segmentation: SePiCo: Semantic-Guided Pixel Contrast for Domain Adaptive Semantic Segmentation, the link is https://ieeexplore.ieee.org/document/10018569
with arxiv version:
https://arxiv.org/abs/2204.08808
and source code:
https://github.com/BIT-DA/SePiCo
Thank you very much!

Hi, @zhaoxin94, thanks for sharing this collection of papers. Also, could you please add two new papers about Active DA.
[AAAI2022] Active Learning for Domain Adaptation: An Energy-based Approach, the link is https://ojs.aaai.org/index.php/AAAI/article/view/20850
with arXiv version:
https://arxiv.org/abs/2112.01406
and source code:
https://github.com/BIT-DA/EADA
[CVPR2022 oral] Towards Fewer Annotations: Active Learning via Region Impurity and Prediction Uncertainty for Domain Adaptive Semantic Segmentation, the link is https://openaccess.thecvf.com/content/CVPR2022/papers/Xie_Towards_Fewer_Annotations_Active_Learning_via_Region_Impurity_and_Prediction_CVPR_2022_paper.pdf
with arxiv version:
https://arxiv.org/abs/2111.12940
and source code:
https://github.com/BIT-DA/RIPU
Thank you very much!

Sorry for the late reply. Very impressive and interesting work. I have added all three papers to the list.


fix docs

Thank you for your patch!  There was a misspelling, but I can fix that for you if you wish.


Enable Travis CI for Repository

Travis CI is configured, but it looks like the webhook has not been enabled.
To enable web hook:

login to https://travis-ci.org/
goto https://travis-ci.org/profile
view the Jasig organization
check Jasig/SurveyPortlet

Thanks!
cc @apetro  @bjagg  @drewwills

Done, I believe. May have to touch the git repo to get the hook to fire. I enabled the build-only-if-.travis-ci.yml-file-present, so will only apply to the right branches and this project can turn off the integration temporarily or permanently in the future simply by removing that text file.

Thanks @apetro!


Types for next

Contribution for progression towards the version of Shoukaku
Typings will be updated as the refactor progresses
These are not tested for production use
All info are in types README
P.S. Saya gimme pats

Is this done, if so you can mark this as ready for review

Is there any way I can also contribute to this PR like I wanna add some typings?

@hydrox19  I think you could PR on his fork at branch next then wait for @0t4u to merge it, it will reflect the changes here

@hydrox19 I think you could PR on his fork at branch next then wait for @0t4u to merge it, it will reflect the changes here

Alright

Types have now been updated to reflect upstream commit E976088AB1601B083E76F02E12D232B3250F1D27

@Deivu can I just merge this...


Making video or audio optional in transcoder api.

Thanks for stopping by to ask us a question! Please make sure to include:

What you're trying to do
I am trying to transcode a video larger than 5 seconds. But the video or audio should be optional

What code you've already tried

      elementary_streams=[
          transcoder_v1.types.ElementaryStream(
              key="video-stream0",
              video_stream=transcoder_v1.types.VideoStream(
                  h264=transcoder_v1.types.VideoStream.H264CodecSettings(
                      bitrate_bps=550000,
                      frame_rate=60,
                  ),
              ),
          ),
          transcoder_v1.types.ElementaryStream(
              key="video-stream1",
              video_stream=transcoder_v1.types.VideoStream(
                  h264=transcoder_v1.types.VideoStream.H264CodecSettings(
                      bitrate_bps=2500000,
                      frame_rate=60,
                  ),
              ),

          ),
          transcoder_v1.types.ElementaryStream(
              key="audio-stream0",
              audio_stream=transcoder_v1.types.AudioStream(
                  codec="aac", bitrate_bps=64000
              ),
          ),
      ],
      mux_streams=[
          transcoder_v1.types.MuxStream(
              key="sd",
              container="ts",
              elementary_streams=["video-stream0", "audio-stream0"],
          ),
          transcoder_v1.types.MuxStream(
              key="hd",
              container="ts",
              elementary_streams=["video-stream1", "audio-stream0"],
          ),

      ],
      manifests=[
          transcoder_v1.types.Manifest(
              type="HLS",
              mux_streams=["sd", "hd"],
          ),
      ],
  )```

Any error messages you're getting
When no audio is found:  \atom atom0 does not have any inputs (input0) with an audio track
When no audio is found: message": "editList does not have any input with a video track"

@irataxy could you help field this question, or point the issue towards someone who can :)

I am checking with the team on this, but it may be another week or so due to the holiday.

@irataxy Thank you, I am waiting for reply. I will be so thankful.

Please see the new Audio-only config at https://cloud.google.com/transcoder/docs/concepts/config-examples#audio-only


Import Mapping for Energy Monitor

Create import mapping for energy monitor.
As source stat use https://github.com/mschlenstedt/LoxBerry-Plugin-Stats4Lox-NG/issues/53

Closed with https://github.com/mschlenstedt/LoxBerry-Plugin-Stats4Lox-NG/commit/e29dc5093400035e4b012ff825815f9e706769c6


Wrong height / font-size

Hi shentao!
I tried to use this awesome plugin and came across next issue:

I realized that when I include bootstrap.less it breaks multiselect's height (for comparsion I included another select on the screenshot). But I couldn't find out why. I see multiselect uses rem units, maybe this is the key.
Haven't you got such problem? Really want to use the plugin but got no idea how to fix that issue and what might cause this
Thanks in advance

Yes, this is the reason. I shall change everything to px in the next release! :)

Fixed in 0.2.5!


WOCCatalog-WinStore10 does not build to ARM

I installed the most recent (2015-12-03) SDK .zip and successfully built/installed the WOCCatalog Universal Windows (WinStore10) solution to the Desktop.
I then entered the Configuration Manager, added the ARM architecture as a viable target, and attempted to build the same.
Build error hit:
1>------ Build started: Project: WOCCatalog (WOCCatalog\WOCCatalog), Configuration: Debug ARM ------
1>C:\Program Files (x86)\MSBuild\14.0\bin\Microsoft.Common.CurrentVersion.targets(1819,5): warning MSB3245: Could not resolve this reference. Could not locate the assembly "XamlCompositorCS". Check to make sure the assembly exists on disk. If this reference is required by your code, you may get compilation errors.
1>  ConsumeRuntimeComponent.cpp
1>F:\winobjc_zip\msvc\ConsumeRuntimeComponent.cpp(34): fatal error C1083: Cannot open include file: 'client.h': No such file or directory
========== Build: 0 succeeded, 1 failed, 0 up-to-date, 0 skipped ==========
========== Deploy: 0 succeeded, 0 failed, 0 skipped ===========
msvc*ConsumeRuntimeComponent.cpp* shows the following ifdef:
#ifdef _M_ARM
#include <client.h>
int __cdecl main(::Platform::Array<::Platform::String^>^ args)
{
EbrDefaultXamlMain();
}
#endif
Where are we expected to pull the client.h file from? I don't see the file at present in the Windows Kits headers or in the SDK.

@andrejo-msft ARM builds are currently not supported.

Is ARM build supported now @asimonov-msft


Visual Edit Customizer

Hi guys, in the last version of WordPress 4.7 are available short visual icon edit from Customizer active.
Is possible add this feature with custom element (for example simple text of footer - NO WIDGET) and Kirki?
Video for example this: https://s.w.org/images/core/4.7/starter-content-v1.mp4?_=1
Thank you very much,
Davide

Ok, I find this :)
Is possible get this with this documentation:
https://aristath.github.io/kirki/docs/arguments/partial_refresh.html


initial text analysis

basic text data analysis using spacy

No conflict with the base branch. Okay to merge.


Add translations for 'see more industries' button text

forgot to do this in the last PR! whoops
https://uktrade.atlassian.net/browse/CMS-26

Codecov Report

Merging #101 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #101   +/-   ##
=======================================
  Coverage   94.12%   94.12%
=======================================
  Files          23       23
  Lines         528      528
=======================================
  Hits          497      497
  Misses         31       31

Impacted Files
Coverage Δ

home/translation.py
100% <ø> (ø)
:arrow_up:

home/models.py
100% <ø> (ø)
:arrow_up:

Continue to review full report at Codecov.

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


Your theme's demo is not generated on the Hugo website

Hello @UtkarshVerma
About a month ago you opened a topic in the Hugo Forum about your theme's demo not working on the Hugo Themes website and I gave you a detailed reply here
Currently I am sending Pull Requests to have themes without a working demo removed from the Hugo website.
Can you please look into providing a valid config.toml for your Example Site?
Also in your Example Site's config you need to specify the theme directory like so: themesDir = "../.." (you need to remove the second forward slash).
Thank you.
CC / @digitalcraftsman

@onedrawingperday I've added a config.toml file and tested it with Hugo v0.51 locally. The demo should work now. Thanks for taking your time notifying me about this. :+1:

@UtkarshVerma Thank you very much! I confirm the fix.
The demo will work again the next time themes are updated on the Hugo website.
CC / @digitalcraftsman


Dead demo link

Hello,
I was very intrested to explore the demo capabilities before trying the tutoriel but the link in the description seems KO :
https://developer.ibm.com/patterns/fingerprinting-personal-data-from-unstructured-text/

Link in description opens EN documention. Anyway, documentation is available in both in EN and KO. EN - https://github.com/IBM/gdpr-fingerprint-pii
KO - https://github.com/IBM/gdpr-fingerprint-pii/blob/master/README-ko.md
Hope this helps

Sorry i meant that the link was broken but it's fixed now :)


R.ap() does not dispatch correctly when using native Array as functor

In the light of this functor:
// Identity
var Identity = function(val) {
  this.val = val;
};
Identity.prototype.map = function(fn) { // Functor
  return Identity.of(fn(this.val));
};
Identity.prototype.ap = function(app) { // Apply
  return app.map(this.val);
};
Identity.of = function(val) { // Applicative
  return new Identity(val);
};

The native JavaScript Array is a functor by the Fantasy Land definition. In addition, I've implemented the Apply and Applicative spec directly on the Array prototype:
Array.prototype.ap = function(app) { // Apply
  return this.map(fn => app.map(fn));
};
Array.prototype.of = function(val) { // Applicative
  return [val];
};

Then when I'd like to use my Apply, this works for my Identity functor, but not for my Array functor:
R.ap(Identity.of(R.add(5)), Identity.of(50)); //=> Identity(55)
R.ap([R.add(3), R.add(5)], Identity.of(50)); //=> []

I would expect the last line to return the same as using .ap() directly on the functor:
[R.add(3), R.add(5)].ap(Identity.of(50)); //=> [Identity(53), Identity(55)]

Instead, I get an empty array.
My guess would be that Ramda's .ap() does not dispatch correctly somehow when I'm extending JavaScript's native Array as a functor. From a quick scan of the ramda source, I'd say the problem is in the hardcoded check for isArray within hasMethod.
Is this expected behaviour? For me it's certainly not.
I realize that piggybacking the .ap() implementation on top of the native JavaScript Array is probably not the best solution, but adding another array-like type to JavaScript is not a great solution either imho.

I'd like to remove the Array exclusion from _hasMethod. From memory, though, @CrossEye is opposed.

I don't remember such a discussion.
But unless my memory fails me, that exclusion is how we run Ramda's functions rather than the native map, filter, reduce, etc.
I'm guessing this is a bug in our internal ap function that's causing the current behavior.  It really has had little workout, I believe, and I wouldn't be surprised if we misunderstood the spec when it was implemented, thinking that a.ap(b) required a and b to be of the same Apply type.
I will try to look into it, but I'm afraid it won't be until tomorrow at the earliest.
As to wanting to dispatch on lists, that might be more difficult.  I'm really not sure how we could comfortably do that and yet still handle those functions we want to implement ourselves.

I don't remember such a discussion.

With the help of Gmail search I was able to find the exchange.

Ok, I do remember that.  But that doesn't sound like a discussion about removing it, or opposition from me to doing so.
I don't have any strong attachment to our mechanism for dispatching to appropriate functions or any objection to changing it.  But I want to ensure that the functions that Ramda actually creates for operating on the types it's claiming, especially lists, in fact do get called.  We have claimed ap for lists (and probably implemented it badly), so we cannot also dispatch ap when called on a list.
How we do that, though, doesn't make much difference to me.

This might not work so well typing on my phone, but I think that getting this to work might be as simple as adding a pair of square braces in the ap implementation:
Current
module.exports = _curry2(function ap(fns, vs) {
  return _hasMethod('ap', fns) ? fns.ap(vs) : _reduce(function(acc, fn) {
    return _concat(acc, map(fn, vs));
  }, [], fns);
});

Suggested
module.exports = _curry2(function ap(fns, vs) {
  return _hasMethod('ap', fns) ? fns.ap(vs) : _reduce(function(acc, fn) {
    return _concat(acc, [map(fn, vs)]);
  }, [], fns);
});

I will try to test when I'm in front of a computer.

I have a patch to remove _hasMethod. I'll open a pull request within the hour.

Ok, well I'll look at it tomorrow.  I'm looking forward to seeing if you have a way to solve @branneman's issue.  It seems a fairly fundamental conflict with Ramda's claiming ap on lists.
Cheers!

I've just confirmed that #1400 resolves this issue. It does so by replacing
_hasMethod('ap', fns) ? fns.ap(vs) : ...

which expands to
fns != null && !_isArray(fns) && typeof fns[methodName] === 'function' ? fns.ap(vs) : ...

with
typeof fns.ap === 'function' ? fns.ap(vs) : ...

so we dispatch to an ap method if available regardless of the type of fns.

Sweet! :+1:

My suggestion above handles the Identity case, but changes the behavior for two lists.  It's not clear to me that it's doing something wrong, though.
ap([R.add(3), R.add(5)], Identity.of(50)); //=> [Identity(53), Identity(55)]
ap([R.multiply(2), R.add(3)], [1, 2, 3]); //=> [[2, 4, 6], [4, 5, 6]]

You can see this in the repl.
The current behavior of the latter snippet would be to return [2, 4, 6, 4, 5, 6], a flattened list.  I'm still trying to figure out if the FL laws dictate one over the other or if either would be lawful.  But I certainly prefer this unflattened behavior if it's legitimate.


ImportError: No module named disqus

Hello all,
i follow the official documentation to install this package, but anytime i get this error. If i try from the shell of the project i can import disqus but not when import it in settings.APP!!
Django 1.4.7
[Tue Apr 14 06:12:19 2015] [error] [client <IP_ADDRESS>] ImportError: No module named disqus
thanks in advance
Marco

Hi @blacking, which Python version are you running?

Hello i have the same problem.
django : 1.8
python : 2.7.3
Also sites is not installed.

same problem
django 1.10.1
python 3.5.2
django-disqus 0.5


Adding description for undocumented behavior

Added documentation for the redirect that happens, when a path ending with index.html is provided to ServeFile

Thanks, but we don't use pull requests for Go.  Please see http://golang.org/doc/contribute.html .


dBeaver not responding again.

Description
when running a block of PL/pgSQL across multiple databases dBeaver once again uses > 95% of all cores and the UI freezes with the white  screen and the entirely too common: "(Not Responding)" text in the title bar.
this will typically happen if I visit more than a dozen clusters without restarting dBeaver.
if finally just crashed out.
thread dump included.
DBeaver Version
Community Edition 23.0.1
Operating System
Windows 10
Database and driver
No response
Steps to reproduce
connect to cluster/database
do something
connect to another cluster/database
repeat 12,20, or more times with unique clusters
watch dBeaver UI freeze and eventually crash the application.
thread-dump_2023-03-30.txt
Additional context
No response

Probably will be fixed in #19553

@coleman-rik Please reopen if this will happen again.


Dockerfile LABEL name double quotes not ignored

I have a bunch of old deployments that have this in the Dockerfile:
LABEL name="foobar"

This used to work fine, the double quotes were removed. Now it seems that the double quotes are treated as part of the name.
Is this by design, or a bug?

Well, it's both. :)
The upstream dependency claims it's as expected. However I disagree with the reasoning.
I'll take another look at the status of joyent/node-docker-file-parser#3, which would fix this issue.

What's changed? Have you guys switched to a different parsing library? Because quotes were stripped before, and now they aren't 🤔
Also, I think the reason I included quotes in the first place was because it didn't work without them, earlier...

At some point we parsed the label as JSON which probably removed the quotes, but then failed miserably on all other cases.
@Qix- maybe we could just drop all "non valid characers" [^a-zA-Z0-9] from the parser output.

Whether or not the docker file parser is at fault, I feel like it makes sense to strip non-alphanumeric characters from the deployment name. When quotes got in there it was a bit tricky to use the cli (for example: now ls "foobar" vs now ls foobar). Plus, it broke the previous behavior...


Access user in AppServiceProvider?

Laravel Version: 5.3

I need to be able to pass data to all views when the user is logged in.
Here is what my AppServiceProvider currently looks like:
class AppServiceProvider extends ServiceProvider
{
    public function boot()
    {
        if (Auth::check())
        {
            View::share('key', 'value');
        }
    }
}

However, the data is never passed to the view because I can't use Auth (for some reason) in AppServiceProvider so the if statement condition is never met.
Following a previous solution I found, I tried doing this:
class AppServiceProvider extends ServiceProvider
{
    public function boot()
    {
        View::composer('*', function ($view)
        {
            if (Auth::check())
            {
                View::share('key', 'value');
            }
        }
    }
}

This worked, but the problem is that the closure is called whenever a view is rendered. For example, if my blade template includes 3 other blade templates (which it does), then the closure above will be called 4 times. Explained further here.
So how can I access the user correctly in AppServiceProvider?

Thanks for your interest in Laravel.
please ask questions like this on a forum (StackOverflow, or Laracasts).

This repo is for bug tracking. Use the forums or slack channel for solving your issue

@user6841322,
AuthServiceProvider is booting after AppServiceProvider. An authorized user will be available after booting the AuthServiceProvider.


Ionic view source throws 404

From @aravindfz on February 3, 2017 4:25
Ionic version:
[x] 1.x
In the Ionic Collection documentation
Clicking on the View Source navigates me to a github page here
I'm submitting a documentation error
[x ] bug report
[ ] feature request
[ ] support request
Current behavior:
404 Not Found
Expected behavior:
Source Documentation
Copied from original issue: driftyco/ionic#10294

fixed in b4e1b57ee43f2a3ef8d008db9d84b7140f24d568


Location of LESS and JS (core) files?

First time user, great library.
Does the location of the LESS and JS (core files) have to be in the source of my project, can they reside in node_modules (as a dependency) so I don't have to commit them to my project?
Of course the overrides etc can be in source/
Sorry for the noob question, thanks so much!

Hi @edhenderson, this SO question might help.

Thanks for the link. I'll take a proper look today. A cursory glance makes it look like this is more of a hack than official recommendation. Is the Semantic-UI way to commit the src files to repo?

@edhenderson Yes.

@Banandrew Thank you for your help Sir.


Metro bundler failure in Libraries/Animated/src/nodes/AnimatedSubtraction.js

🐛 Bug Report
After upgrading my app from RN 0.57.8 to 0.59.4, in Metro Bundler output, on launch of my app:
::1 - - [10/Apr/2019:23:19:06 +0000] "GET /debugger-ui/debuggerWorker.js HTTP/1.1" 404 168 "http://localhost:8081/debugger-ui/" "Mozilla/5.0 (Macintosh; Intel Mac OS X 10_14_4) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/73.0.3683.103 Safari/537.36"
 BUNDLE  [android, dev] ../../index.js ░░░░░░░░░░░░░░░░ 0.0% (0/1)::1 - - [10/Apr/2019:23:19:06 +0000] "GET /index.delta?platform=android&dev=true&minify=false HTTP/1.1" 500 - "http://localhost:8081/debugger-ui/" "Mozilla/5.0 (Macintosh; Intel Mac OS X 10_14_4) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/73.0.3683.103 Safari/537.36"
error: bundling failed: SyntaxError: [APP]/node_modules/react-native/node_modules/react-native/Libraries/Animated/src/nodes/AnimatedSubtraction.js: Unexpected token (17:12)

  15 | const AnimatedWithChildren = require('./AnimatedWithChildren');
  16 |
> 17 | import type {InterpolationConfigType} from './AnimatedInterpolation';
     |             ^
  18 |
  19 | class AnimatedSubtraction extends AnimatedWithChildren {
  20 |   _a: AnimatedNode;
    at Parser.raise ([APP]/node_modules/@babel/parser/lib/index.js:3831:17)
    at Parser.unexpected ([APP]/node_modules/@babel/parser/lib/index.js:5143:16)
    at Parser.expectContextual ([APP]/node_modules/@babel/parser/lib/index.js:5115:41)
    at Parser.parseImport ([APP]/node_modules/@babel/parser/lib/index.js:8565:12)
    at Parser.parseStatementContent ([APP]/node_modules/@babel/parser/lib/index.js:7341:27)
    at Parser.parseStatement ([APP]/node_modules/@babel/parser/lib/index.js:7243:17)
    at Parser.parseBlockOrModuleBlockBody ([APP]/node_modules/@babel/parser/lib/index.js:7810:25)
    at Parser.parseBlockBody ([APP]/node_modules/@babel/parser/lib/index.js:7797:10)
    at Parser.parseTopLevel ([APP]/node_modules/@babel/parser/lib/index.js:7181:10)
    at Parser.parse ([APP]/node_modules/@babel/parser/lib/index.js:8660:17)
 BUNDLE  [android, dev] ../../index.js ░░░░░░░░░░░░░░░░ 0.0% (0/1), failed.

To Reproduce

Upgraded my app from RN 0.57.8 to 0.59.4
react-native run-android

Expected Behavior
No:

404 in the debuggerWorker
500 in the debugger-ui, with Metro error logged above

Code Example
I'm not sure how to best extract my app as a test case, but I'm hoping if the above error output is useful enough to trace what might be happening, and new to RN 0.59.
Environment
info
  React Native Environment Info:
    System:
      OS: macOS 10.14.4
      CPU: (16) x64 Intel(R) Core(TM) i9-9900K CPU @ 3.60GHz
      Memory: 20.40 MB / 8.00 GB
      Shell: 3.2.57 - /bin/bash
    Binaries:
      Node: 11.13.0 - /usr/local/bin/node
      Yarn: 1.15.2 - /usr/local/bin/yarn
      npm: 6.9.0 - /usr/local/bin/npm
      Watchman: 4.9.0 - /usr/local/bin/watchman
    IDEs:
      Android Studio: 3.3 AI-182.5<IP_ADDRESS>14842
      Xcode: /undefined - /usr/bin/xcodebuild
    npmPackages:
      react: 16.8.3 => 16.8.3
      react-native: ^0.59.4 => 0.59.4
    npmGlobalPackages:
      eslint-plugin-react-native: 3.6.0
      react-native-cli: 2.0.1

It seems like you are most likely overwriting your babel configuration with one that doesn't support stripping flow types.

My .babelrc before upgrade (on RN 0.57.8):
{
  "presets": ["module:metro-react-native-babel-preset"],
  "plugins": ["idx"]
}

After upgrade to RN 0.59.4:
{
  "plugins": ["idx"]
}

Any issue with this? Is having a .babelrc at all a problem in RN 0.59?

You should definitely be able to use a .babelrc so I'm not entirely sure what the problem is. I think you should probably put the preset back in. I assume that when you have a custom .babelrc, it disables all the stuff that RN comes with from the start.
I'm going to close this as it is unlikely an issue with RN itself, but feel free to keep discussing it here or somewhere else and if it turns out to be an RN issue, we can reopen it.


up_squared: tests/kernel/mem_protect/userspace failed.

To Reproduce
Steps to reproduce the behavior:

west build -b up_squared -p tests/kernel/mem_protect/userspace
west flash ~/up_squared.sh
see error

Logs and console output
Booting Zephyr OS build zephyr-v2.4.0-1845-g6aa16e01dd91
Running test suite userspace
START - test_is_usermode
PASS - test_is_usermode
START - test_write_control
E: General protection fault (code 0x0)
E: RAX: 0x00000000ffffffff RBX: 0x0000000000117028 RCX: 0x0000000000000000 RDX: 0x0000000000000000
E: RSI: 0x0000000000000000 RDI: 0x0000000000000000 RBP: 0x0000000000000000 RSP: 0x0000000000d4afb8
E:  R8: 0x00000000001056d7  R9: 0x8000000000000006 R10: 0x0000000000000000 R11: 0x0000000000000246
E: R12: 0x0000000000000000 R13: 0x0000000000000000 R14: 0x0000000000000000 R15: 0x0000000000000000
E: RSP: 0x0000000000d4afb8 RFLAGS: 0x0000000000010246 CS: 0x003b CR3: 0x0000000000cae000
E: RIP: 0x0000000000100578
E: >>> ZEPHYR FATAL ERROR 0: CPU exception on CPU 1
E: Current thread: 0x11b800 (ztest_thread)
Caught system error -- reason 0
System error was expected
PASS - test_write_control
START - test_disable_mmu_mpu
E: General protection fault (code 0x0)
E: RAX: 0x0000000000cae000 RBX: 0x0000000000117050 RCX: 0x0000000000000000 RDX: 0x0000000000000000
E: RSI: 0x0000000000000000 RDI: 0x0000000000000000 RBP: 0x0000000000000000 RSP: 0x0000000000d4afb8
E:  R8: 0x00000000001056d7  R9: 0x8000000000000006 R10: 0x0000000000000000 R11: 0x0000000000000246
E: R12: 0x0000000000000000 R13: 0x0000000000000000 R14: 0x0000000000000000 R15: 0x0000000000000000
E: RSP: 0x0000000000d4afb8 RFLAGS: 0x0000000000010246 CS: 0x003b CR3: 0x0000000000cae000
E: RIP: 0x00000000001005a9
E: >>> ZEPHYR FATAL ERROR 0: CPU exception on CPU 0
E: Current thread: 0x11b800 (ztest_thread)
Caught system error -- reason 0
System error was expected
PASS - test_disable_mmu_mpu
......
START - test_domain_add_thread_context_switch
PASS - test_domain_add_thread_context_switch
START - test_domain_add_part_context_switch
PASS - test_domain_add_part_context_switch
START - test_domain_remove_part_context_switch
E: Page fault at address 0x119000 (error code 0x6)
E: Linear address not present in page tables
E: Access violation: user thread not allowed to write
E: PTE: not present
E: RAX: 0x0000000000cae000 RBX: 0x0000000000000000 RCX: 0x0000000000000000 RDX: 0x0000000000000000
E: RSI: 0x0000000000000000 RDI: 0x0000000000119000 RBP: 0x0000000000000000 RSP: 0x0000000000d46fd0
E:  R8: 0x000000000010032b  R9: 0x8000000000000006 R10: 0x0000000000000000 R11: 0x0000000000000246
E: R12: 0x0000000000000000 R13: 0x0000000000000000 R14: 0x0000000000000000 R15: 0x0000000000000000
E: RSP: 0x0000000000d46fd0 RFLAGS: 0x0000000000010246 CS: 0x003b CR3: 0x0000000000cae000
E: RIP: 0x000000000010032b
E: >>> ZEPHYR FATAL ERROR 0: CPU exception on CPU 0
E: Current thread: 0x11a000 (unknown)
Caught system error -- reason 0
System error was expected
this error is an expected error, but the test it will stop at there, and fail with timeout.
Environment (please complete the following information):

OS: Fedora32
Toolchain: zephyr-sdk-0.11.4
Commit ID: 6aa16e01dd91

I cannot replicate this 'fail and timeout'. The test suite PASSes for me on up2 at that revision 6aa16e0. (I do not have a flash script, so I used USB boot media method with grub and zephyr.strip.)
issue_30306_up2_tests_kernel_mem_protect_userspace.txt

the same timeout issue with #30307

Closing this, it is now part of cluster in https://github.com/zephyrproject-rtos/zephyr/issues/30573


rpi_pico: Fix compilation for non Linux systems

On non Linux system, the compilation failed because the main
CMake did not pass the CMAKE_SYSTEM_NAME to the bootloader's
CMake. This caused the variable to be empty, instead of being
"Generic".
Fixes #44704
Signed-off-by: Yonatan Schachter<EMAIL_ADDRESS>

@rivimey I was able to confirm that this patch solves the problem for Windows machines.It should probably also solve it for MacOS, though I was unable to obtain a Mac to test this on.

@yonsch I can confirm that it works on macOS (see https://gist.github.com/stephanosio/0459b247b5d4cd5458c76cec59f98799).


build: use add_subdirectory instead of add_external_project

This switches the CoreFoundation sub-build to use add_subdirectory.
This has a few MAJOR benefits:

CoreFoundation changes get tracked and cause compile/link triggers
CoreFoundation warnings are surfaced in the build
CoreFoundation dependencies are reflected in the right location
Foundation dependencies are reflected in the right location
The overall build is simpler

Eventually, it should be possible to further simplify this.
Unfortunately, a couple of hacks are needed to make this work.  The
sub-projects properties must be cached, but they were just passthrough
previously, so this is not too terrible.  Additionally, the library
control know needs to be saved/restored around it.  The worst thing here
is the need to manually collect the dependency as we do not have a
proper library target for the Foundation (Swift) library.  When CMake
gains proper Swift support, it should be possible to use add_library
and get a property target that we can use target_link_libraries with
permitting the dependency tracking to allow us to get that without
manual intervention.

CC: @millenomi @gwynne

@swift-ci please test

@swift-ci please test

Please test with following PRs:
apple/swift-corelibs-xctest#261
@swift-ci please test

Please test with following PRs:
apple/swift-corelibs-xctest#261
apple/swift-llbuild#460
@swift-ci please test

Please test with following PRs:
apple/swift-corelibs-xctest#261
apple/swift-llbuild#460
apple/swift-package-manager#2054
@swift-ci please test

@swift-ci please test

Please test with following PRs:
apple/swift-corelibs-xctest#261
apple/swift-llbuild#460
apple/swift-package-manager#2054
@swift-ci please test

Please test with following PRs:
apple/swift#23775
apple/swift-package-manager#2054
@swift-ci please test

Please test with following PRs:
apple/swift#23869
apple/swift-package-manager#2054
@swift-ci please test

I think this may have broken the LLDB tests. They can't locate Foundation swift module any more. I think we may need to update swift/utils/build-script-impl ?
DOTEST_EXTRA="-Xcc -F${FOUNDATION_BUILD_DIR}/CoreFoundation-prefix/System/Library/Frameworks"

@adrian-prantl pointed me to https://ci.swift.org/view/LLDB/job/oss-lldb-swift-5.1-incremental-linux-ubuntu-18_04/378/consoleFull#-3714861073122a513-f36a-4c87-8ed7-cbc36a1ec144 which is a 5.1 build.  Seems that we need to cross-port the migration path to 5.1.  https://github.com/apple/swift/pull/23906 should take care of that.


CWL-viewer having issues recognizing gitlab URLs

From question on forum -- CWL-viewer seems to have issues viewing a cwl workflows form gitlabs-- I replicated it as well.
Link: https://gitlab.inria.fr/capsid.public_codes/CroMaSt/-/blob/main/CroMaSt.cwl
Link to question:
https://gitlab.inria.fr/capsid.public_codes/CroMaSt/-/blob/main/CroMaSt.cwl
I went and found a another CWL workflow from gitlabs here -- https://gitlab.com/iidsgt/biocwl/-/blob/master/Workflows/Build-RNA-Indexes.cwl and it also couldn't render it -- so it might be a general issue with gitlab URLs.

Hi, the URL provided, https://gitlab.inria.fr/capsid.public_codes/CroMaSt/-/blob/main/CroMaSt.cwl, does not pass the current regex tests in the JavaScript code https://github.com/common-workflow-language/cwlviewer/blob/3088d53f425f5a6cb922228b57e1b6f7671820a2/src/main/resources/static/js/main.js#L80-L98.
There are two regular expressions, one for GitHub, and for for GitLab. Respectively:

https?://github.com
https?://gitlab.com

So they expect that any URL provided to CWL Viewer starts with either github.com or gitlab.com.
@swzCuroverse now with so many companies hosting their own internal GitLab servers, I think maybe we should consider supporting other ways to verify if a URL is hosted on GitLab, GitHub, or any other git server (e.g. the Apache Software Foundation uses primarily gitweb, https://gitbox.apache.org/, but allows other projects to change and use gitlab/github with a mirror to gitweb/gitbox).
-Bruno


Document exception for metrics/classification

What does this PR do?

Part of Fixes #5434

[x] init.py
[x] accuracy.py
[x] auc.py
[x] auroc.py
[x] average_precision.py
[x] confusion_matrix.py
[x] f_beta.py
[x] hamming_distance.py
[ ] helpers.py (I believe these are just helper functions and not displayed in the documentation so they don't to document exceptions)
[x] iou.py
[x] precision_recall.py
[x] precision_recall_curve.py
[x] roc.py
[x] stat_scores.py

Before submitting

[x] Was this discussed/approved via a GitHub issue? (not for typos and docs)
[x] Did you read the contributor guideline, Pull Request section?
[x] Did you make sure your PR does only one thing, instead of bundling different changes together?
[x] Did you make sure to update the documentation with your changes? (if necessary)
[x] Did you write any new necessary tests? (not for typos and docs)
[x] Did you verify new and existing tests pass locally with your changes?
[x] Did you update the CHANGELOG? (not for typos, docs, test updates, or internal minor changes/refactorings)

PR review
Anyone in the community is free to review the PR once the tests have passed.
Before you start reviewing make sure you have read Review guidelines. In short, see the following bullet-list:

[x] Is this pull request ready for review? (if not, please submit in draft mode)
[x] Check that all items from Before submitting are resolved
[x] Make sure the title is self-explanatory and the description concisely explains the PR
[x] Add labels and milestones (and optionally projects) to the PR so it can be classified

Did you have fun?
Make sure you had fun coding 🙃

Hello @dipam7! Thanks for opening this PR.

In the file pytorch_lightning/metrics/classification/auroc.py:

Line 67:1: W293 blank line contains whitespace
Line 74:121: E501 line too long (126 > 120 characters)

In the file pytorch_lightning/metrics/classification/f_beta.py:

Line 70:1: W293 blank line contains whitespace

In the file pytorch_lightning/metrics/classification/hamming_distance.py:

Line 59:1: W293 blank line contains whitespace

In the file pytorch_lightning/metrics/classification/precision_recall.py:

Line 111:1: W293 blank line contains whitespace
Line 266:1: W293 blank line contains whitespace

In the file pytorch_lightning/metrics/classification/stat_scores.py:

Line 114:121: E501 line too long (122 > 120 characters)
Line 115:1: W293 blank line contains whitespace

Do see the Hitchhiker's guide to code style

@akihironitta Sure I'll do those as well


"go back" and "go forward" command does not work in Jupyter notebook after upgrading to 1.74.0

Does this issue occur when all extensions are disabled?: Yes/No. I can't disable all extension, since if so I can't run jupyter notebook

VS Code Version: 1.74.0 (also 1.75.0)
OS Version: Mac OS 13.0.1

Steps to Reproduce:

in jupyter notebook, use the command "go back" and "go forward"
it does not go back to the previous cell when using the command "go back", for example
NOTE: in VS code version 1.73.1, it works

Thanks for creating this issue! It looks like you may be using an old version of VS Code, the latest stable release is 1.74.2. Please try upgrading to the latest version and checking whether this issue remains.
Happy Coding!

I already even tried version 1.75.0, but the problem persists.

same here

Can this problem be dealt with as soon as possible? This is really annoying that this shortcut can't be used. Thank you so much!

We didn't have any change in the notebook land so it might be caused by some refactoring in the editor service. Thank you all for bringing this up. Fixed in main now, should be available in next Insiders and the upcoming VS Code stable.


[AKS] suppress ARM linter errors

Suppresses the three autorest --azure-validator errors that the AKS swagger currently emits. One of these suppressions will be removed soon when a PATCH endpoint exists for AKS.
I think at this point we have approval for these suppressions, but please let me know if the email template for approval is still required. cc: @ifeoluwaokunoren @ravbhatnagar @vladimirjoanovic
PR information

[x] The title of the PR is clear and informative.
[x] There are a small number of commits, each of which have an informative message. This means that previously merged commits do not appear in the history of the PR. For information on cleaning up the commits in your pull request, see this page.
[x] Except for special cases involving multiple contributors, the PR is started from a fork of the main repository, not a branch.
[x] If applicable, the PR references the bug/issue that it fixes.
[x] Swagger files are correctly named (e.g. the api-version in the path should match the api-version in the spec).

Quality of Swagger

[x] I have read the contribution guidelines.
[x] My spec meets the review criteria:

[x] The spec conforms to the Swagger 2.0 specification.
[x] The spec follows the guidelines described in the Swagger checklist document.
[x] Validation tools were run on swagger spec(s) and have all been fixed in this PR.

Automation for azure-sdk-for-python

Automation for azure-sdk-for-node

Automation for azure-sdk-for-ruby

Automation for azure-sdk-for-java

Automation for azure-sdk-for-go

@mboersma were these suppressions approved, please send an email offline with the approval.

ARM team approves the 2 suppressed ARM linter errors


Unify and fix Polynomial's equals and hashCode methods.

This commit adds a termsIterator method to Polynomial. This method
returns an Iterator[Term[C]] for all non-zero coefficients in the
polynomial, from smallest-exponent to largest. This iterator is
used to implement equality and hashCode in a standardized way that
works for dense and sparse polynomials.
The hashing formula used is essentially:
val it: Iterator[Term[C]] = p.termsIterator
val ns: Iterator[Int] = it.map(t => (0xfeed1257 * t.exp) ^ t.coeff.##)
val hashCode: Int = ns.foldLeft(0)(_ ^ _)
One useful side-effect is that if the terms iterator is empty
(i.e. the polynomial is zero) the hash code is 0. Also, if p
consists of a single term whose exponent is zero, the overall
hashCode will be identical to the hash code of the term's
cofficient.
One gotcha with the termsIterator method is that it uses a
(_ == 0) test to filter out terms whose coefficient is zero.
This necessary in order to support universal equality, and is
documented in the method's ScalaDoc comment. We could choose
to make this method private to Spire if it is deemed to be
"too dangerous".
Fixes #410.
Review by @tixxit and @rklaehn.

I guess this will work for spire types and scala primitives.
Polynomial.zero will not be equal to Polynomial.constant(zero), but I guess that is unavoidable.
The x == 0 thing will not work for types that don't attempt universal equality. So equality and hashCode will be broken for Polynomial[T] if T does not support checking for zero using == 0. Is that acceptable?

I think Polynomial.constant(zero) will equal Polynomial.zero (assuming zero == 0), since the termsIterator ignores zero coefficients.
Yes, I think that if the underlying type doesn't support universal equality (which would require zero == 0) then universal equality won't work. So I think it's an acceptable trade-off.

@rklaehn I think the answer is that it is the best we can do, which kind of sucks... The only other thing we could do is capture a type class / zero for DensePolynomial. As usual, the real ideal is to push people to use type class based equality (Eq), and hashes once we have algebra integration.

@non Yes, you are right. And Polynomial.zero.hashCode will be 0, which also is the "universal" hashCode of "zero" elements. I still need to get the hang of all this madness...
Anyway, I only need Polynomials of Rationals and maybe Doubles at work. So for my real use case at work, the solution will work.

I'm just going to merge this -- I think at this point it's received enough review.


Add config option for Secure cookies; fix Chrome warning

The following warning is shown in the developer console for a web client instance:

A cookie associated with a resource at URL was set with SameSite=None but without Secure. A future release of Chrome will only deliver cookies marked SameSite=None if they are also marked Secure. You can review cookies in developer tools under Application>Storage>Cookies and see more details at https://www.chromestatus.com/feature/5633521622188032.

This cookie management setting does not appear to be configurable via the Amplitude JS init options. Appears to be related to issue #239 and the related PR.
Mac Chrome Version 83.0.4103.61
Amplitude JS SDK 6.2.0

Somewhat related: to configure the SameSide cookie property, the JS SDK Cookie Management docs note to use option cookieSameSite, but the CHANGELOG.md notes the property is named sameSiteCookie. sameSiteCookie appears to be the correct/working name. The SDK docs should be updated to reflect this?

I submitted a comment to the docs page to get the sameSiteCookie typo fixed.

Updated the docs to reflect this. Sorry about the error.

@dana11235 I think this ticket was incorrectly closed. The main issue with the Secure attribute not being configurable is still unresolved.

Sorry. I know we have looked into this in the past. @blazzy can comment on this.

Hi @jfroom, The secure flag can be set with the secureCookie option. You might do something like this when initializing amplitude: { secureCookie: window.location.protocol === 'https:' }.
Future versions of the SDK will likely default to SameSite=Lax, but this will might be a slightly breaking change for people that require cross domain form POSTing.
It's also tempting to have the SDK default to using a Secure cookie if it detects that it's being loaded on a site with https. This would also be a breaking change for sites that use both https and http.

Hi @blazzy! My apologies - I did not see the existence of the secureCookie option in the docs. 🤦
I'm now able to set both SameSite=None and Secure on the Amplitude generated cookie. The Chrome warning still appears for some reason even though the cookie has the desired settings, but that is beyond the scope of this ticket.
Thank you for your help!

Leaving a note here for anyone else that encounters it. I got the Chrome warning to go away once I delete the old in-secure cookie values.


Adding a lease prefix to the change feed host options.

This adds support for pointing multiple hosts at the same collection using only one auxiliary collection

@Wisgary,
Thanks for your contribution.
To ensure that the project team has proper rights to use your work, please complete the Contribution License Agreement at https://cla.microsoft.com.
It will cover your contributions to all Microsoft-managed open source projects.
Thanks,
Microsoft Pull Request Bot

@Wisgary, thanks for signing the contribution license agreement. We will now validate the agreement and then the pull request.
Thanks, Microsoft Pull Request Bot


The binding for .NET don't have the full support for iOS/Android

I can pull the library for nuget, but need to manually integrate the code on the samples folder to make it work with iOS/Android.

What sample code are you referring to?

The samples from https://github.com/xamarin/flex/tree/master/bindings/csharp/samples. I pull from the android, shared and iOS

Yeah but you said you had to integrate "code from the samples", what code in particular? As far as I know once you integrate the nuget its C# API should be available in your project, iOS or Android.


Update astro-db.mdx

Description (required)
add some missing details to make the tutorial more fluid.   if you follow the tutorial, at this point, your comment table has author and content columns.   The seed script is going to fail because it's missing data in the author table and the Comment table does not have body column

Related issues & labels (optional)

Closes #
Suggested label:

Hello! Thank you for opening your first PR to Astro’s Docs! 🎉
Here’s what will happen next:

Our GitHub bots will run to check your changes.
If they spot any broken links you will see some error messages on this PR.
Don’t hesitate to ask any questions if you’re not sure what these mean!

In a few minutes, you’ll be able to see a preview of your changes on Vercel 🥳

One or more of our maintainers will take a look and may ask you to make changes.
We try to be responsive, but don’t worry if this takes a few days.

Lunaria Status Overview
🌕 This pull request will trigger status changes.

Learn more
By default, every PR changing files present in the Lunaria configuration's files property will be considered and trigger status changes accordingly.
You can change this by adding one of the keywords present in the ignoreKeywords property in your Lunaria configuration file in the PR's title (ignoring all files) or by including a tracker directive in the merged commit's description.

Tracked Files

Locale
File
Note

en
guides/astro-db.mdx
Source changed, localizations will be marked as outdated.

Warnings reference

Icon
Description

🔄
The source for this localization has been updated since the creation of this pull request, make sure all changes in the source have been applied.

NOTE: Not merging until we figure out why the deploy build is failing, just to be safe!  :smile: (Probably not related to this PR, but want to make sure our builds are working first before merging new PRs)


Option to start main server thread as non-daemon

I'm working on a Java app based on an HTTP server that accepts requests and performs operations accordingly. As the NanoHttpd Main Listener thread is the only thread that keeps running, I need an option to make sure it is not started as a daemon thread.
This PR makes sure you can change this by running setMainThreadDaemon(boolean flag) while the main thread is stopped.

Would adding a bool to the start function, e.g. public void start(final int timeout, boolean main_thread_daemon=true), suffice in your case?

That would be fine.


Problem with the Ibaraki case numbers?

There seems to be a problem with the Ibaraki case numbers. it currently reads 50,946 - with daily new cases just 423.
Whereas Tokyo has about 70,000 total cases and approx 7000, new cases; and Kanagawa 32,000 total cases and 3000 new cases. Both are in the order of (daily cases x10)=(Total cases). But in the case of Ibaraki it is closer to  (daily cases x100)=(Total cases) - I don't think this is correct.
What is the source of the data? I imagine that the data in recent weeks has only included new cases and not people who have recovered.
On the local site - I think the total is listed as - 8,461 cases (https://www.pref.ibaraki.jp/1saigai/2019-ncov/index.html) - I could be wrong though; but I would have thought that is closer to the correct number.
The local government website seems to have changed how it presents the data - could that be causing the issue with the data entering the website? (if you are using an automated data collection system).
If you could take a look - it would be useful. Thanks

Thanks for reporting the issue. I think there might be something wrong with our numbers for per-prefecture recovery numbers. Those were extracted from the MHLW documents but seemed to have stopped updating.
I'll take a look at them to see what's going on and if it is possible to fix.

I did some investigation. We were missing the latest recovery numbers, so I fixed and backfilled the missing data.
We take our data from the MHLW rather than the Ibaraki prefectural site. However, according to the MHLW, Ibaraki's recovery numbers have not changed since the start of September, which is really odd.
The recovery number we're using is from here for 2022-09-22:
https://www.mhlw.go.jp/content/10906000/000993021.pdf
from : https://www.mhlw.go.jp/stf/newpage_28176.html
But since 2022-09-03, that number is the same for Ibaraki:
https://www.mhlw.go.jp/content/10906000/000985234.pdf
from: https://www.mhlw.go.jp/stf/newpage_27794.html
We'll keep this open for now as this problem seems to affect not only Ibaraki, but Saga and Miyagi. All the other prefectures have updated recovery numbers, so maybe this is an inconsistency/process issue between the prefectures and the central government.


Asakuki smells too dense and less aromatic

I noticed the bathroom I set asakuki diffuser smells weird. I do feel aroma, but maybe too condensed. It's not a pleasant smell. I even have to wonder whether it's a mixture of kids urine and aroma, but I confirmed the mist that's coming out of the device does have a unpleasant smell.
Inside of dome smells still aromatic and nice but the mist coming out smells uncomfortably strong.

Not sure what worked but I no longer feel so. These days I add 150 ml water and 5 drips of essential oil. Previously I added 7 or more sometimes.


kotlin: add bazel macros for easier kt target declaration

We currently have to redeclare our sources in the srcs argument for the top level library consumed by the aar_with_jni macro. The reason is because the aar_with_jni doesn't get visibility onto the transitive dependencies of the provided library. Since we are using kt_jvm_library, which internally creates two rules and encloses the dependencies within them, the library we provide does not expose any dependencies for the aar_with_jni target.
This makes it very complicated to create new libraries because for every new library, we have to create a filegroup which the top level envoy_lib needs to depend on directly in the srcs.
This is a total bazel macro hack to hide away this detail for us. The limitation here is that the top level envoy_lib needs to directly depend on the targets.
Alternative solution is to write a rule which traverses the dependencies and takes out the appropriate srcs from each dependency and transitive dependency using an envoy_mobile_kt_library.
Before:
aar_with_jni(
    name = "android_aar",
    android_library = "envoy_lib",
    archive_name = "envoy",
    visibility = ["//visibility:public"],
)
kt_android_library(
    name = "envoy_lib",
    srcs = [
        "Envoy.kt",
        "//library/java/io/envoyproxy/envoymobile/engine:envoy_engine_srcs", # Redundancy
    ],
    custom_package = "io.envoyproxy.envoymobile",
    manifest = "EnvoyManifest.xml",
    visibility = ["//visibility:public"],
    deps = ["//library/java/io/envoyproxy/envoymobile/engine:envoy_engine_lib"],
)
# Redundancy
filegroup(
    name = "envoy_engine_srcs",
    srcs = ["EnvoyEngine.java"],
    visibility = ["//visibility:public"],
)
android_library(
    name = "envoy_engine_lib",
    srcs = [":envoy_engine_srcs"], # Redundancy
    custom_package = "io.envoyproxy.envoymobile.engine",
    manifest = "EnvoyEngineManifest.xml",
    visibility = ["//visibility:public"],
    deps = ["//library/common:envoy_jni_interface_lib"],
)

Now:

aar_with_jni(
    name = "android_aar",
    android_library = "envoy_lib",
    archive_name = "envoy",
    visibility = ["//visibility:public"],
)
envoy_mobile_kt_aar_android_library(
    name = "envoy_lib",
    srcs = [
        "Envoy.kt",
    ],
    custom_package = "io.envoyproxy.envoymobile",
    manifest = "EnvoyManifest.xml",
    deps = ["//library/java/io/envoyproxy/envoymobile/engine:envoy_engine_lib"],
)
envoy_mobile_android_library(
    name = "envoy_engine_lib",
    srcs = ["EnvoyEngine.java"],
    custom_package = "io.envoyproxy.envoymobile.engine",
    manifest = "EnvoyEngineManifest.xml",
    deps = ["//library/common:envoy_jni_interface_lib"],
)

For an explanation of how to fill out the fields, please see the relevant section
in PULL_REQUESTS.md
Description: add bazel macros for easier kt target declaration
Risk Level: low
Testing: local
Docs Changes: n/a
Release Notes: n/a
[Optional Fixes #Issue]
[Optional Deprecated:]

thanks @keith


Concatenate rows and columns

Implements #18.

Codecov Report

Merging #19 (c4ac2bb) into master (e9c3999) will increase coverage by 0.23%.
The diff coverage is 89.79%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@            Coverage Diff             @@
##           master      #19      +/-   ##
==========================================
+ Coverage   90.22%   90.46%   +0.23%
==========================================
  Files          13       14       +1
  Lines         706      755      +49
  Branches      116      129      +13
==========================================
+ Hits          637      683      +46
- Misses         57       60       +3
  Partials       12       12

Impacted Files
Coverage Δ

src/tabeline/exceptions.py
80.00% <78.26%> (+3.80%)
:arrow_up:

src/tabeline/__init__.py
100.00% <100.00%> (ø)

src/tabeline/_concatenate.py
100.00% <100.00%> (ø)

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


Consider having the registration form sign up button disabled when some inputs are not complete

Overall this might not be worth working on until we do a next design pass on the registration pages; it's the kind of thing which, if done badly, can be not worth the effort to implement and test. Not tagging as help wanted for that reason.
#14027 has some very old work in this direction, but realistically we'll want to do something from scratch.
This likely involves extending web/src/portico/signup.ts, among other things.

@timabbott  After the next design pass on the registration pages is completed. I would like to work on this issue.
PS: Commenting for future claim :)


Creating the NavigationPrompt

Summary
This PR introduces the terra-navigation-prompt package and the NavigationPrompt and NavigationPromptCheckpoint components.
Additional Details
A React component has complete control over its own rendering logic. It can render child components and unmount those child components as it sees fit. It follows that any individual component has no direct control over its own mounting and unmounting; it is rendered (or not) at the discretion of its parent. A component must rely on its lifecycle events to determine how it is being implemented.
However, a component's children can be stateful and responsible for important user input or interactions. If the user can navigate away from that child component, it may be necessary for that child component to notify the user that their workflow is incomplete or that some transient state has not yet been persisted. If the component has no control over its own unmounting, how can it communicate this state to the overall application before the component is already being unmounted?
The most straightforward solution is to lift the state up. If the state is lifted into a parent component, it can be more directly managed by the component responsible for navigation, or it could be persisted indefinitely. This is the strategy that should be implemented by most components. However, for very large applications or highly composable applications built out of many stateful components, lifting the state may not be feasible.
Another solution is to expose callbacks from the stateful components. Whenever the stateful component's state changes, the callbacks are executed to notify the consumer of the current state. This works well at a small scale, but this becomes hard to maintain as the component tree increases in depth. One would have to chain these callbacks many levels deep to ensure every component is kept abreast of the current state, and every component, even those without these state concerns, would have to adhere to a consistent prop interface for those callbacks.
The NavigationPrompt and the NavigationPromptCheckpoint were created smooth over the rough edges of the aforementioned strategies. Components that have sensitive state can render a NavigationPrompt to make their state known to the application. Components that navigate between components can use the NavigationPromptCheckpoint to check for the presence of NavigationPrompts and inform the user in a consistent way. State remains localized to the component that needs it, communication of state changes occurs automatically, and components that don't share these concerns can safely ignore all of it.
Deployment
https://terra-framework-deploye-pr-605.herokuapp.com/#/components/terra-navigation-prompt/navigation-prompt/about

With 3fa87db I added support for functions in the resolvePrompts API. This will allow consumers to create custom prompt strings, based on the set of actively registered prompts, without having to implement onPromptChange and storing those registered prompts locally. I ran into this with the NavigationPrompt implementation and found this makes my life a lot easier there.

Functional review in progress. The prompt works as expected in all supported desktop browsers; however, I see some unusual behavior on mobile. On iOS, the prompt does not flex properly. It seems to be zooming in when you enter data into the form fields, then when the prompt is initially triggered, it is displayed beyond the bounds of the viewport. Once manually resized, it functions properly again.
On an Android device, the page does not render at all.

@tbiethman The sizing issues may be related to sizing issues identified in https://github.com/cerner/terra-framework/issues/621

@mmalaker Regarding the zoom, that is an iOS feature for inputs. The examples here uses a bog standard  element. One way to prevent the zoom from happening is by setting a font-size on the input larger than 16px (the terra-form-input does this and thus does not exhibit this same zoom behavior). The NotificationDialog itself is rendering appropriately, as the viewport itself is still zoomed-in after presenting it. Zooming out with your fingers will reset the view and show the whole dialog. If we're worried about it, I could update the examples to set an appropriate font-size and prevent the zoom from happening.
Regarding the Android device not rendering the page at all, that's news to me. I'll have to look into that one.

@tbiethman thanks for the info. That all makes sense. I updated my Android device and now the page displays and the prompt works as expected. Interestingly, with Android I'm not seeing any zoom behavior.
To make a better show on iOS in our example, could we do what you mentioned above and adjust the font size? Otherwise, everything looks good and is working correctly.

Functional verification completed. The nav prompt now displays and functions as expected on all supported browsers and devices.

This build is failing because the docker-compose.yml file in terra-toolkit updated to use helium chrome version.
https://github.com/cerner/terra-toolkit/blob/master/config/docker/ci-docker-compose.yml#L10
Travis-ci pulls the docker-compose.yml file from master here.
https://github.com/cerner/terra-framework/blob/master/.travis.yml#L29
A cheap hack to get this PR done would be to change the travis-ci to point to the sha prior to our updating to helium
It cares not for your versions


The leaderboard is missing in https://paperswithcode.com/sota/keyphrase-extraction-on-semeval2017

Still there yet...

Still there yet...

Issue still there...


Prepare for release.

Codecov Report

Merging #9 into master will increase coverage by 1.71%.
The diff coverage is n/a.

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


fail the installation of nuget package on xamarin.android

Hello,
I have downloaded sample Project of Xamarin Android and I can't compile it. I have checked nuget packages and the package isn't installed. So I have tried to install it but It launchs an error. I get the same error if I try to install the nuget package on my Xamarin.Android app. The error is this:

No se pudo instalar el paquete 'Microsoft.ConnectedDevices.Xamarin.Droid 0.1.0'. Está intentando instalar este paquete en un proyecto que tiene 'MonoAndroid,Version=v6.0' como destino, pero el paquete no contiene referencias de ensamblado o archivos de contenido compatibles con dicho marco.
Best regards

Both the nuget package and the VS project target version 6.0 of Xamarin. What version do you have installed? You can check and update from Visual Studio menu Tools->Options->Xamarin

Hi @sphenry
In theory, Xamarin is up to day, there aren't more updates in stable channel. Versions are:
Visual Studio Enterprise 2015 14.0.25431.01 Update 3
Xamarin: <IP_ADDRESS>4 (73f58d6)
Xamarin.Android <IP_ADDRESS> (9578cdc)
What should I check more?
Thanks

That's the latest version of Xamarin, and is the same version I am using. I am on VS 2017, though. Can you also check the Project->Properties are also set to the latest version?

Ok, In order to reduce all differences, I have installed VS 2017 community on other computer and I can install the nuget package with Android 7.1. Android 7.0/6.0 launchs error. My app uses Android 19 (4.4.2) so it launchs error too. And the case of Android 7.1, I can't compile the sample app because it fires the following error:

1>C:\Program Files (x86)\Microsoft Visual Studio\2017\Community\MSBuild\Xamarin\Android\Xamarin.Android.Common.targets(1063,2): error MSB4018: Error inesperado en la tarea "ResolveLibraryProjectImports".

As checked in, the sample app depends on the library being built and referenced from the same directory structure. I think the error you're getting is because the library path cannot be resolved (either because it hasn't been built, or can't be found). You should be able to either:

Build the ConnectedDevices.Xamarin.Droid and ConnectedDevices.Xamarin.Droid.Core projects
Remove the reference to the above libraries from Project->References and add the reference to the nugget package

Sorry by answer you later, I haven't been able to check it between week
If I compile both library project and I link them to xamarin sample, it compiles with different versions. So thanks you. I have tried to try sample app but it doesn't work. I'm going to search what is the reason.
If there are some problem, I will notify you.
Thanks for your time.

@cansado2930, Have you been able to make any progress on this?

Yes, I have a few of progress. Sample Xamarin app doesn't work. I have discover that the problem is in the webview where user sign in with MS account. I have build the Android app in order to know if there is some problem with Xamarin app but it has the same behaviour. On sample Android app shows an error message "error ocurred: invalid_request"
And I have changed the appilicationid on both projects of a real. what do you think that I have done bad?
Thanks

I had same issues on VS2017.
with Android SDK v6, Visual Studio can't get the NuGet package at all.
Then I changed SDK version to v7.1, this time build failed with this error:

The "ResolveLibraryProjectImports" task failed unexpectedly.
System.IO.PathTooLongException: The specified path, file name, or both are too long. The fully qualified file name must be less than 260 characters, and the directory name must be less than 248 characters.
at System.IO.PathHelper.GetFullPathName()
at System.IO.Path.LegacyNormalizePath(String path, Boolean fullCheck, Int32 maxPathLength, Boolean expandShortPaths)
at System.IO.Path.NormalizePath(String path, Boolean fullCheck, Int32 maxPathLength, Boolean expandShortPaths)
at System.IO.Path.NormalizePath(String path, Boolean fullCheck, Int32 maxPathLength)
at System.IO.FileStream.Init(String path, FileMode mode, FileAccess access, Int32 rights, Boolean useRights, FileShare share, Int32 bufferSize, FileOptions options, SECURITY_ATTRIBUTES secAttrs, String msgPath, Boolean bFromProxy, Boolean useLongPath, Boolean checkHost)
at System.IO.FileStream..ctor(String path, FileMode mode, FileAccess access, FileShare share)
at Xamarin.Tools.Zip.ZipEntry.DoExtract(IntPtr zipFile, String destinationPath, FileMode outputFileMode, EntryExtractEventArgs args)
at Xamarin.Tools.Zip.ZipEntry.Extract(String destinationDir, String destinationFileName, FileMode outputFileMode)
at Xamarin.Android.Tools.Files.ExtractAll(ZipArchive zip, String destination, Action2 progressCallback)    at Xamarin.Android.Tasks.ResolveLibraryProjectImports.Extract(DirectoryAssemblyResolver res, ICollection1 jars, ICollection1 resolvedResourceDirectories, ICollection1 resolvedAssetDirectories, ICollection`1 resolvedEnvironments)
at Xamarin.Android.Tasks.ResolveLibraryProjectImports.Execute()
at Microsoft.Build.BackEnd.TaskExecutionHost.Microsoft.Build.BackEnd.ITaskExecutionHost.Execute()
at Microsoft.Build.BackEnd.TaskBuilder.d__26.MoveNext()	ConnectedDevices.Xamarin.Droid.Sample

I noticed the PathTooLongException error, so I moved the folder of project to somewhere nearer to the root of drive, and it compiled successfully this time.
TL;DR There's a bug with NuGet package of this project with Android SDK 6 which refuses the package to install. But Android SDK 7.1 is fine. You just need to make sure the project address is not too long.

nuget package requires as @mghiasi75 have said SDK 7.1. I could compile it to SDK 6 from Project from here. A question, have you been able to use it? I tried the sample app and it shows me an error about a Request on a real device. These days I will download the new commits in order to try it again because I would like to implement it on production.

@cansado2930 What error did you get? I opened the sample project, installed NuGet package, and deployed it to a device running Android 5.1 and it worked fine. I was able to sign in and launch a Uri from the Android device to my PC and my Windows phone.

When I compiled it 24 days ago, it launched an error About the url of login. "error Request" when I tried to do login. I have downloaded the last commit and now I can't compile "ConnectedDevices.Xamarin.Droid" :( I have attached a screenshot about errors that it shows.

@cansado2930 Can you show the full output from the error log?

This is the output:

1>------ Operación Recompilar todo iniciada: proyecto: ConnectedDevices.Xamarin.Droid.Core, configuración: Release Any CPU ------
1>  ConnectedDevices.Xamarin.Droid.Core -> C:\proyecto\project-rome-master\xamarin\src\ConnectedDevices.Xamarin.Droid.Core\bin\Release\Microsoft.ConnectedDevices.Xamarin.Droid.Core.dll
2>------ Operación Recompilar todo iniciada: proyecto: ConnectedDevices.Xamarin.Droid, configuración: Release Any CPU ------
2>BINDINGSGENERATOR : warning BG8A04:  matched no nodes.
2>BINDINGSGENERATOR : warning BG8A04:  matched no nodes.
2>BINDINGSGENERATOR : warning BG8A04:  matched no nodes.
2>BINDINGSGENERATOR : warning BG8A04:  matched no nodes.
2>BINDINGSGENERATOR : warning BG8A04:  matched no nodes.
2>C:\proyecto\project-rome-master\xamarin\src\ConnectedDevices.Xamarin.Droid\obj\Release\generated\src\Microsoft.ConnectedDevices.IAuthCodeProvider.cs(26,59,26,67): warning CS0109: El miembro 'IAuthCodeProviderInvoker._members' no oculta un miembro accesible. La palabra clave new no es necesaria.
2>C:\proyecto\project-rome-master\xamarin\src\ConnectedDevices.Xamarin.Droid\Additions\Platform.cs(58,33,58,61): error CS0246: El nombre del tipo o del espacio de nombres 'PlatformInitializationStatus' no se encontró (¿falta una directiva using o una referencia de ensamblado?)
2>C:\proyecto\project-rome-master\xamarin\src\ConnectedDevices.Xamarin.Droid\Additions\Platform.cs(49,64,49,94): error CS0535: 'Platform.PlatformInitializer' no implementa el miembro de interfaz 'IPlatformInitializationHandler.OnDone(bool)'
2>C:\proyecto\project-rome-master\xamarin\src\ConnectedDevices.Xamarin.Droid\obj\Release\generated\src\Microsoft.ConnectedDevices.IRemoteSystemFilter.cs(21,59,21,67): warning CS0109: El miembro 'IRemoteSystemFilterInvoker._members' no oculta un miembro accesible. La palabra clave new no es necesaria.
2>C:\proyecto\project-rome-master\xamarin\src\ConnectedDevices.Xamarin.Droid\obj\Release\generated\src\Microsoft.ConnectedDevices.IPlatformInitializationHandler.cs(21,59,21,67): warning CS0109: El miembro 'IPlatformInitializationHandlerInvoker._members' no oculta un miembro accesible. La palabra clave new no es necesaria.
2>C:\proyecto\project-rome-master\xamarin\src\ConnectedDevices.Xamarin.Droid\obj\Release\generated\src\Microsoft.ConnectedDevices.IRemoteLauncherListener.cs(21,59,21,67): warning CS0109: El miembro 'IRemoteLauncherListenerInvoker._members' no oculta un miembro accesible. La palabra clave new no es necesaria.
2>C:\proyecto\project-rome-master\xamarin\src\ConnectedDevices.Xamarin.Droid\obj\Release\generated\src\Microsoft.ConnectedDevices.Platform.cs(33,60,33,68): warning CS0109: El miembro 'Platform.IAuthCodeHandlerInvoker._members' no oculta un miembro accesible. La palabra clave new no es necesaria.
2>C:\proyecto\project-rome-master\xamarin\src\ConnectedDevices.Xamarin.Droid\obj\Release\generated\src\Microsoft.ConnectedDevices.IRemoteSystemDiscoveryListener.cs(29,59,29,67): warning CS0109: El miembro 'IRemoteSystemDiscoveryListenerInvoker._members' no oculta un miembro accesible. La palabra clave new no es necesaria.
========== Recompilar todo: 1 correctos, 1 incorrectos, 0 omitidos ==========

@cansado2930 I suspect you have not download the .aar files that need to be bound. You get them by running the download-aar.ps1 in the src directory. Also, I just pushed some updates to change the min version of android required to nuget, so that package should work better for you now

Very thanks @sphenry Yesterday I downloaded the new commit and I can compile the library correctly. I didn't compile sample app but I can compile it in a new app with the library or from nuget package. So I implemented it on my production app. Thanks!!


require name be configured and valid for kv commands

we use target.name as part of the title we provision for users when they call wrangler kv:namespace create <binding-name>. we should require that the user's toml has a valid name specified before we make the call.
Pros
the use of the worker name in the title along with the binding name is to provide uniqueness. requiring the name in your toml for kv commands prevents some confusion when users accidentally create a namespace with title "-MY_BINDING" (usually we'll have a worker name prefixing this, so "worker-prod-MY_BINDING").
Cons
requiring the worker name in the toml for kv commands is sort of unexpected from a user POV; i'm not doing anything with my project, why do i need to give you a project name to interact with my namespace? (in terms of other KV interactions).
Possible middle-ground: a bespoke requirement just for kv:namespace create that requires target.name, but not on other kv commands.

Possible middle-ground: a bespoke requirement just for kv:namespace create that requires target.name, but not on other kv commands.

^ i vote for this one

i also vote for that one

i would rather fix this by changing the requirement for kv namespaces from id -> title in the wrangler.toml. this would be a breaking change and would go in a 2.0 release.


Fix flutter test in workspace

Follow-up to #153754.

I see a few in packages/flutter_tools/lib/src/build_system/targets/web.dart and packages/flutter_tools/lib/src/isolated/native_assets/native_assets.dart.

Yeah native_assets was broken as well. I think packages/flutter_tools/lib/src/build_system/targets/web.dart is ok.
I did a few extra fixes.


Small typo

utiltiy to utility

Thank you.
But don't edit gh-pages branch. This branch is automatically generated and every change will be reset after deploy.
The docs website is here: https://github.com/saadeghi/daisyui/tree/master/src/docs
And here's the guide on how to build the site: https://github.com/saadeghi/daisyui/blob/master/.github/CONTRIBUTING.md
I fixed this typo already. Thanks.

Cool.


Add contour lines labels

The feature: Cesium.Material ElevationContour line labels.
Options:

units
label-spacing
min-zoom
max-zoom
text-size
text-opacity
text-size
text-halo-width
text-halo-color
text-color

You can currently add contour lines in CesiumJS as shown here: https://sandcastle.cesium.com/index.html?src=Globe Materials.html. The current implementation renders these lines in the shader, so I think it may be difficult to generate labels in the same efficient way.
I think one way to implement this would be to generate the labels around the closest areas to the camera as the user moves. Although I'm not sure what an easy way would be to actually get the positions the contour lines are at on the CPU side to create the labels. You could try looking at how the lines are computed in the shader.

The feature: Cesium.Material ElevationContour line labels.  it seems required feature, any update on this @OmarShehata

@manjunathamm24 There hasn't been any activity on this item recently.
If you'd like to add more information about your use case, that may help us prioritize. If you are interesting in contributing, please let us know and we'd be happy to discuss implementation or review a PR.
Thanks!


Replace GitHub action with bash oneliner

Nice, shall we test it in this PR?

Well it is clearly not working!

nice & clean!

bors r+


Remove section: On-Premises VMs

Remove https://kubernetes.io/docs/setup/production-environment/on-premises-vm/ [parent section preview].
The removed section does not fit in with current content guidelines on third party content.
/kind cleanup
Fixes: #23501

/retitle Remove section: On-Premises VMs

/label tide/merge-method-squash

I think we may need to consider this on a case-by-case basis. In general, we agreed that

dual-hosted contents should be removed;
vendor pitches are not appropriate to be hosted either;
links to personal repos are of a concern;
...

However, think about this. If I'm trying to provision a Kubernetes cluster in a CloudStack environment, where am I supposed to look for related documentation? The Kubernetes website or the CloudStack website or ... I have to google some blogs or stackoverflow answers? As a user, I would be assuming that Kubernetes website will provide such a doc or at least some pointers. In the case of CloudStack, the current content is not good because we see that CloudStack is providing a solution which seems more "official". It looks to me the CloudStack kubernetes plugin might be the right approach if I AM provisioning a Kubernetes cluster on CloudStack. However, we cannot make that decision for the users. A user may still want to use the Ansible playbook approach, for some reasons. It's kind of a dilemma for us and we want unbiased docs. Maybe we want to provide links to both rather than contents for either?
So ... I'm okay with dropping the CloudStack page, provided that we still allow users to find useful pointers for setting up Kubernetes clusters in a CloudStack environment. However, I hope we are not dropping it simply because it falls into 3rd party contents.

we still allow users to find useful pointers for setting up Kubernetes clusters in a CloudStack environment

I'd expect that CloudStack will publish this documentation and will make it discoverable to web search engines. That ought to cover it. Vendors such as CloudStack are responsible for owning this kind of documentation and publishing it themselves (that's what SIG Architecture agreed).

As a user, I would be assuming that Kubernetes website will provide such a doc or at least some pointers.

We could make it more clear that we won't; however, there are plenty of things that don't have “how to install X on Y” type pages, and expecially not when there are a great many Y.

@sftim @tengqm

I'd expect that CloudStack will publish this documentation and will make it discoverable to web search engines.

This is a valid concern. The documentation removed in this PR is, for each:

CloudStack: The docs listed on k8s.io are a github readme maintained by the cloudstack team. There is no net loss if we remove this page.
DC/OS: this article was already a stub directing people to Mesophere's product documentation & quickstart repo, and provided nothing of value other than a link to a third party service, aka advertisement.
oVirt: this is an OSS project. As such, they are responsible for their own documentation. They do not mirror it on their own site currently.

Of the three, the only one I have concern about is oVirt.  Mesosphere already has excellent documentation.
Given that oVirt is an OSS project and therefore less resourced, I am going to open an issue in their documentation repository informing them that:

This page will be removed by the end of this week.
We encourage them to add oVirt on k8s to their own documentation
Provide them with a permalink (i.e. a to the file at a pre-deletion commit) to the file, so they can easily migrate content to their docs if they choose.

However, this PR is otherwise good to merge and I intend to push it through under the 3rd party KEP warnings.
/lgtm

@celestehorgan

Given that oVirt is an OSS project and therefore less resourced, I am going to open an issue in their documentation repository

Excellent, that's a generous and helpful thing to offer in OSS spirit. ✨ 🙌
Feel free to /hold cancel after you've opened the issue.
/approve
/hold

/hold cancel


S3 support

This pull request allows using S3 as well as ftp.
Please note that in file: src/tesk_core/filer_class.py  is used an image served from our gitlab organization instead of eu.gcr.io/tes-wes/filer... This is needed so that the image contains the S3 compatible filer. Should you accept this PR you should also serve this image from eu.gcr.io.

Is this PR still relevant? Is there interest on your side to work on this?


[ISSUE] Memory leak on opening/closing MenuBar

Every time the menu bar list is opened and closed, the memory usage increases (and is never released again).
I don't know if this is the only memory leak, as I don't open and close it enough for it to be a gigabyte like it is now:

Thank you.
I have checked memory usage of HomeConMenu.
I have identified a problem that may be a bit of a memory leak by Activity Monitor.app.
But, using Instruments.app, I could not find that my code was causing a memory leak.
I'll do some more investigation.

Thank you for your reporting.
I've found my bugs and started to release the new version.

How about new version, 3.1.3?

I have version 4.0.1 (don't know where this comes from) but the issue seems fixed. Thanks you!


Could not get connect to google server, what's wrong?

I have enable GCM, and some of applications has registered, that is what the Self-Check shows:

but what I am meeting now is that the registered apps receives no message through GCM:

and what the More under the Google Services shows, looks so bad:

this is the result of the push notification tester:

How can i do with that?

Just to be sure: Do you have activated not only Google Cloud Messaging (GCM), but also the Google log-in? (The corresponding menu entry can be found in the microG main screen, just above the GCM option.) Moreover, you can try to dial ##2432546## (i.e., ##CHECKIN##) in you phone app to let microG properly re-connect to the Google servers.

@Tentos Thanks for reply. Do you mean Google device registration option? It is enabled, and i have signed in a google account in my phone. BTW, everything goes well without gmail, I just can't find any account on the account list when i lanuch gmail, and i can't add my google account because it already exist, delete and retry to sign in the account doesn't work for gmail, i'll meet the same solution. Hope this will bring some clues.
It seems that nothing happens after I dial ##24324546##.

My bad, it should be *#*#24324546#*#*. I have updated the corresponding comment.
Yes, I meant Google Device Registration.
Unfortunately, I do not use Gmail and do not have a Google account on my phone, so I cannot try to reproduce your problem for now.
Has anyone else some experience regarding microG, Gmail and a Google account?

*#*#24324546#*#* returns nothing too, it disappered when i typed the last symbol.

Check your VPN

@Vavun It is working well now, thanks!


Updates

bors r+

Did the update to insta = 0.7.0 here cause the JSON output to change? I'm seeing a bunch of insta snapshot mismatches that all appear to be due to newly injected trailing commas in the generated JSON output.

Are you building rust-analyzer with nightly? The Debug format changed in nightly, that’s why the tests could fail

What rust channel am I supposed to build iwth?
When I use stable, I get an error like this:
     Running target/debug/deps/ra_batch-1c83502d4551a596

running 1 test
test tests::test_loading_rust_analyzer ... FAILED

failures:

---- tests::test_loading_rust_analyzer stdout ----
thread 'tests::test_loading_rust_analyzer' panicked at 'called `Result::unwrap()` on an `Err` value: ErrorMessage { msg: "can\'t l\
oad standard library from sysroot\n\"/home/pnkfelix/.rustup/toolchains/stable-x86_64-unknown-linux-gnu/lib/rustlib/src/rust/src\"\\
ntry running `rustup component add rust-src`" }', src/libcore/result.rs:997:5
note: Run with `RUST_BACKTRACE=1` environment variable to display a backtrace.

The CI (and I personally) use stable.
Have you tried
rustup component add rust-src
?

Yes, I did that in the past.
Doing it again says:
% rustup component add rust-src
info: component 'rust-src' is up to date
%

(Let me trying changing my default to stable and seeing if that changes the behavior of rustup component add)

opened https://rust-lang.zulipchat.com/#narrow/stream/185405-t-compiler.2Fwg-rls-2.2E0/topic/building.20with.20stable for more real-time troubleshooting

Okay that was the problem: rustup component is sensitive to the current channel, and I didn't realize that.


typesVersions should not have 4.9.0-beta <=4.8

currently, a typesVersion entry of "<=4.8" will be satisfied by 4.9.0-beta.
That's because our semver code treats "4.8" as the range ">=4.8.0 and <4.9.0". It should actually be ">=4.8.0 and <4.8.Infinity". It should not include 4.9.0-xxx.

I always forget semver treats x.y.z-suffix as an earlier version than x.y.z

Aside: this was found while working on https://github.com/DefinitelyTyped/DefinitelyTyped/pull/62375

Ah, it should be treating <=4.8 as <4.9.0-0, not <4.9.0.


feat(connectors): add snowflake :snowflake: and postgres :elephant:

Description
We are creating a structure for the connectors, we have created the Postgres and Snowflake connectors that use a base connector as a parent so they both have the same structure.
How was the change tested
I created a Postgres database and snowflake.
Issue Information
Closes #15
Checklist
(Ideally, all boxes are checked by the time we merged the PR, if you don't know how to do any of these don't hesitate to say so in the PR and we'll help you out.)

[X] I formatted my PR name according to CONTRIBUTING.md
[ ] I added a news fragment to help populating the changelog as encouraged in CONTRIBUTING.md
[x] I added "Closes #<issue_number>" in the "Issue Information" section (if no issue, feel free to tick thick the box anyway).

Bastien I dont know how to fix the PR name Lint.

@virvirlopez I will change it for you. This PR is a feature so according to this: https://github.com/bitpicky/dbt-sugar/blob/main/CONTRIBUTING.md#pull-requests I would say something like feat(connectors): add base, snowflake and postgres should be fine.
I decided to use conventional commits for this repo because there are tools that allow you to generate a changelog automatically although for now we would be using something else but good to set things from the start.

@virvirlopez the PR looks fine but just before we merge I was wondering if you wanted to have tests already as part of the PR?

@virvirlopez I made a quick PR against your branch to put some testing in place: https://github.com/bitpicky/dbt-sugar/pull/21
Not sure if they're helpful but if they are feel free to merge my PR into your branch whenever you work on this next.

Is there a reason psychopg2-binary cannot be used instead of psycopg2?

@mhindery that's because they binary is generally meant to be not for production as far as I know. I was getting warnings from the lib when I started using it.
Are you running into some issues?

There are tons of projects having switched to (or at least supporting) the binary package: pipelinewise, and a discussion on aiopg about it. As far as I know the recommendation is mainly relevant if you are actively developing on the interaction with postgres itself, not so much when you're just a 'client' of the package.
Having to install the fullblown postgres for this is a bit of a pain (and large and unnecessary). DBT Itself supports the psycopg2-binary package, but now that we have started to use dbt-sugar we need to install postgres just for this. It seems to me if dbt itself is fine with doing so, then the plugin shouldn't require otherwise.

@mhindery I'll have to test it out with just the binary. I don't think I see a problem with it at least theoretically in view of the discussion and it should work but I'll have to test it out.
Feel free to open up a PR in the measure with the package changes in the  pyproject.toml
I'll make an issue for this as we shouldn't be abusing this PR for this.


TV Show 'Premiered' and Episode Aired

Version: 2.8.2
Build: 2016-07-19 20:59
OS: Mac OS X 10.11.6
JDK: 1.8.0_101 x86_64 Oracle Corporation
What is the actual behaviour?

If you leave the Premiered date as Jan, 1, 1970 (I had some concerts from this date), then the date is not written to .nfo file. Same situation for the 'Episode Aired' date.
At the top level of TV Shows, tiny mediamanager is not loading premiered, or genres information, despite them being in the .nfo file. For example, with 'Barenboim On Beethoven', those with ** (see code below) are not being displayed in tinymediamanager, despite the .nfo having that info:

<?xml version="1.0" encoding="UTF-8" standalone="yes"?>
<!-- created on 2016-07-27 10:22:03 - tinyMediaManager 2.8.2 -->
<tvshow>
    <title>Barenboim On Beethoven</title>
    <sorttitle></sorttitle>
    <year>2005</year>
    <rating>0.0</rating>
    <votes>0</votes>
    **<plot>All 32 of the sonatas were performed live in June and July 2006 at the Staatsoper Unter den Linden in Berlin. The masterclasses took place in Chicago in January 2005, in front of a small audience.</plot>**
    **<mpaa>U</mpaa>**
    <episodeguide>
        <url></url>
    </episodeguide>
    <id></id>
    <imdbid></imdbid>
    **<genre>Concert</genre>
    <genre>Documentary</genre>
    <premiered>2005-01-01</premiered>
    <status>Ended</status>**
    <actor>
        **<name>Daniel Barenboim</name>**
        <role>unknown role</role>
        <thumb></thumb>
    </actor>
</tvshow>

tmm_logs.zip

is fixed, thx :)

yes, this is a known drawback.
Currently, our GUI field for displaying dates does not allow empty values.
We always have to set some (fake) date :|
We have choosen Date(0) which is the beginning of the UNIX timestamp... and do not write that into NFO.
Guess, we have to find a better approach... (not using a date field)


Grid filtering & sorting when wrapped in a ViewComponent?

Hi,
it seems that filtering & sorting controls are disabled if I wrap the grid in a ViewComponent like in the example below - any workaround for this? Probably more of an MVC rendering thing really.
Main View:
@await Component.InvokeAsync("ContractsView", new { customerId = Model.Id })
Component View:
@using NonFactors.Mvc.Grid;

@model IEnumerable<ExampleModel.Contract>

Contracts

    @Html.Grid(Model).Build(columns =>
{
    columns.Add(model => model.ContractType).Titled("Contract Type").Sortable(true).Filterable(true);
}).Named("ContractsGrid").Filterable().Sortable().Css("table-hover").Empty("No data found.")

Regards,
-Timo

I tried to repro, but it worked fine for me. Could you share a repro?

Hi, sorry, a bit busy here. I'll try and create proper repro - cannot really share the actual project, sorry. :(
Anyways, I attached a screenshot of the 2 event listeners that are NOT being attached for some reason. Any ideas where to look? In the layout there's obviously  after  and as said, the grids are being rendered inside a ViewComponent.
-Timo

They are binded by $('.mvc-grid').mvcgrid(); call, maybe it's not being called.

Hi @timotahti, how's your schedule going.
Maybe you can share a repro now or you managed to find a problem?

Hi @timotahti, how's your schedule going.
Maybe you can share a repro now or you managed to find a problem?

Sorry, but I'm closing as not reproducible.


Please add colour themes

Inbox is great, but the interface was starting to look old. If you could add some colour theming options in time, it'd make this better than Inbox! Thanks for the great work.

Somewhat related:
if I remember right, @RedcoatAsher was planning on adding some form of Dark Mode theme.

That would be a great start!
On Mon, Apr 22, 2019, 22:52 Daniel Russell<EMAIL_ADDRESS>wrote:

Somewhat related:
if I remember right, @RedcoatAsher https://github.com/RedcoatAsher was
planning on adding some form of Dark Mode theme.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/boukestam/inbox-in-gmail/issues/58#issuecomment-485653304,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABPFMUTSZPV4WXNJSSSGIYTPR2PZVANCNFSM4HHTVD6Q
.

We should also allow backgrounds, like Gmail. Those are selectable in the same section as color themes, in the settings gear icon at the top right -> Themes.
If you have a background image set now, it applies almost the same changes that break Gmail's built-in Dark theme. While the text of the email content doesn't turn white like Dark mode, some of the UI, like the left menu items get an ugly text shadow, and a majority of the toolbar on top of opened emails and the email list turn white.
To allow these to work out, we'd need to remove the gray background color from a lot of the elements, setting them to transparent instead.

I've got a branch going here, very WIP:
https://github.com/boukestam/inbox-in-gmail/tree/themes
A few problems to resolve:

Gmail's default theme is white background, but I assume we'd want to default to Inbox-like gray
Enforce black/dark text color on email list items (Icons, Subject, Author, Excerpt)
Stop enforcing gray text on left menu, so Gmail can set that text to white when appropriate (Dark themes, backgrounds)
Top bar should maybe be translucent or transparent, like Gmail, so the chosen color/background shows through

One thing that makes this a bit difficult is that it seems like Gmail doesn't apply a certain class on theme-change, it just loads in a new css stylesheet entirely. So it's not obvious how to decide when the text should be light or dark.


BidRevealed statuses undocumented

The BidRevealed event has a status code which goes from 0 to 5
    event BidRevealed(bytes32 indexed hash, address indexed owner, uint value, uint8 status);

In the source code hardcoded numerical constants are used. Would it be better to label those constants, simultaneously documenting what the status mean?

Duplicate of #147 .


ADD ALL LANGUAGES & CURRENCIES TO A SEARCH LIST

As an end user I want to easily find the languages ( including unofficial languages and currencies) in common lists I work with in my program.

Possible solution 1:

Use existing "optionChoices"-arrays and 'copy-&-paste' all options for all different questions
Use a new input/select-autocomplete widget for "select-1/n-with-more-then-10-options"

Pro's:

flexible
not complex

Con's:

some copy-pasting/duplication of data

Possible solution 2:

Use hard-coded list of currencies (in a file in the interface)
Use a new "question-type" to use this. (i.e. select-1-currency and/or select-n-currency)
The template for this question/input-type would use a new input/select-autocomplete widget
...
Same for Languages (i.e. select-1-language, select-n-language)

Pro's:

less flexible (updating needs to be in code/front-end)

Con's:

more question-types
more templates/code

Maybe use this component: https://github.com/ionic-selectable/ionic-selectable


Working Alpine build

Hoping to get rabbitmq on Alpine, I came across this issue. I forked this branch and got the same error. Turns out the erlang-sasl dep needs to be included. But after including it, I ran into another error. This time more explicit, saying syntax-tools is missing. With those two deps added, rabbitmq was happy!
Also, starting with 3.5, Alpine ships with libressl instead of openssl. I went ahead and updated the Dockerfile with that as well, because having both libs may cause issues down the road.
Initial testing shows no issues so far.

Oooh :heart: