[Bug]: App crashes with exception when attempting to open links from external apps

Open Food Facts username
blendergeek
What happened?
I opened a product page in Firefox and the used the "open in app" feature in the menu in firefox to view the same page in the OpenFoodFacts app. Instead of opening, I got the dreaded "OpenFoodFacts has stopped" dialog.
I expected the product page to open nicely in the OFF app.
Steps to reproduce the behavior

Open a web browser that supports opening pages in other apps
Go to any product page on openfoodfacts.org
Attempt to open the page in the OFF app from the web browser.

What device were you using ?
Moto G4
Android Version
Android 7
Barcode(s) of the product(s)
All
Language your app was in
English
Version
3.6.5 (Default)
Flavor
Open Food Facts
Installation method
F-Droid
Relevant log output
FATAL EXCEPTION: main
Process: org.openfoodfacts.scanner, PID: 14844
java.lang.RuntimeException: Unable to start activity ComponentInfo{org.openfoodfacts.scanner/openfoodfacts.github.scrachx.openfood.features.product.view.ProductViewActivity}:
java.lang.IllegalStateException: Activity ProductViewActivity started without 'state' serializable in intent.
	at android.app.ActivityThread.performLaunchActivity(ActivityThread.java:2659)
	at android.app.ActivityThread.handleLaunchActivity(ActivityThread.java:2724)
	at android.app.ActivityThread.-wrap12(ActivityThread.java)
	at android.app.ActivityThread$H.handleMessage(ActivityThread.java:1473)
	at android.os.Handler.dispatchMessage(Handler.java:102)
	at android.os.Looper.loop(Looper.java:154)
	at android.app.ActivityThread.main(ActivityThread.java:6123)
	at java.lang.reflect.Method.invoke(Native Method)
	at com.android.internal.os.ZygoteInit$MethodAndArgsCaller.run(ZygoteInit.java:867)
	at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:757)
Caused by: java.lang.IllegalStateException: Activity ProductViewActivity started without 'state' serializable in intent.
	at openfoodfacts.github.scrachx.openfood.utils.k.e(FragmentExt.kt:2)
	at openfoodfacts.github.scrachx.openfood.features.product.view.ProductViewActivity.onCreate(ProductViewActivity.kt:10)
	at android.app.Activity.performCreate(Activity.java:6672)
	at android.app.Instrumentation.callActivityOnCreate(Instrumentation.java:1140)
	at android.app.ActivityThread.performLaunchActivity(ActivityThread.java:2612)
	... 9 more

https://github.com/openfoodfacts/openfoodfacts-androidapp/blob/develop/app/src/main/java/openfoodfacts/github/scrachx/openfood/features/product/view/ProductViewActivity.kt

This is fixed on the Play Store, we need to release for F-Droid


change BasePowerSensor device_class to ENERGY

This PR resolves #44.
If there is a reason to keep the device class at power I'd like to hear about it and work out a solution that fits all the needs.
Cheers,
JM-Lemmi

Thanks for your work, but that's unfortunately not how it works.
https://developers.home-assistant.io/docs/core/entity/sensor#available-device-classes

After experimenting a bit more the data in the energy dashboard is appearing correctly with the lifetime production. So it must have been an error on my end. Apologies for the useless issue and PR.

No problem, happens to the best of us as well ;)


Added the Picture-Picture Project

It solves issue #1146
Its working video,
https://user-images.githubusercontent.com/75881393/115956804-9dcc2080-a51c-11eb-90e5-646f5a7ab06b.mp4

Please @ridsuteri @urvashi-code1255 @991rajat review this PR.


Changes bitrate event parameter

Very inspired by Shaka I propose that we follow this  API for bitrate event parameter:
{bitrate,bandwidth,width,height,level}

Questions
I'm not 100% sure about level, do we need that? it looks pretty coupled to HLS and maybe not useful outside this context.

lgtm


Improve UI + polling launch scenario

Steps to reproduce:

Open settings page.
Uncheck Quick launch using default settings.
Check Open devtools whenever the node inspector is running.
Save settings.
Make sure Node is not running.
Click the extension button to open the UI.
Click "Launch V8 Inspector".
Start Node. Notice that the DevTools is not launched.

cc: @johnjbarton if you're interested.

Great, thanks!


stage1: split stage1 builder and translator code into new project

Follow up from https://github.com/coreos/rocket/issues/568#issuecomment-77282946
We need to split the current stage1 into an independent project as a first step to establishing a solid interface between rocket and stage1, allowing us to support alternate stage1 builds and implementations in the future. Tightly coupling stage1 to rocket in one repo ties the project closely to CoreOS and makes porting to other distros or trying other runtime environments more difficult. At a minimum splitting stage1 into its own build could allow rocket to use a newer systemd and nspawn than we can provide in CoreOS.

I think it should stay in this project for now but under a subdirectory or something. Ideally we have two build options like CoreOS PXE and build systemd from scratch!? I don't know how to proceed here. What is the first action? Build systemd from git tag stage1 option?
cc @vcaputo

The first course of action we had arrived at was simply splitting the current stage1 implementation out of tree into a new repo.  Then we'd iterate from there in directions like reproducible builds of a probably forked/patched systemd from source.

I would like to avoid splitting things out into a new tree until there is a compelling reason to; it gets really complex to coordinate things otherwise. We can iterate on the separation in a different directory then when it makes sense to put it into a new project move it out.

In any case what is the first course of action on doing this and who is in charge of making it happen?

@philips forcing coordination to solidify the interaction between rkt and stage1 is exactly why we would like to split. If we want to assemble stage1 from different sources (be it a patched systemd, the current CoreOS binaries, or assembling one from a host system) we need the interface to not be a moving target. If there is there any work that needs to be done to prepare for that? @vcaputo thought the actual task of splitting the repo would be quick.

As for who, @vcaputo can handle it.

@vcaputo if stage1 were to be in a separate repo tomorrow what do you see the coreos/rocket build process looking like?

@jonboulle for starters I envisioned the coreos/rocket build process would cease concerning itself with stage1 creation altogether (including the go binaries that fall into stage1's domain).  There'd be another step necessary for acquiring and building the stage1 from source, but if you have on need to develop stage1 the coreos/rocket build script could just know where we publish signed stage1 acis and grab one.
Stage1 seems poised to become more of a largish rats nest of components built from source with all the dependency/toolchain complexities necessary for static linking pieces like systemd-nspawn.
The coreos/rocket project can resume being a simple golang project with fast and trivial builds.
Most people hacking on coreos/rocket will be served perfectly well by a published stage1 aci.   If they need to hack on the stage1 side, then they can choose to go tame that substantially different kind of animal and the chore of getting their environment suitable for producing successful stage1 builds.
This helps ensure we have a robust solution for distributors packaging rkt, and helps manage the perception of rocket being systemd-bound, communicating via clear compartmentalization that coreos/rocket itself has nothing at all to do with systemd.
This move also better facilitates experimentation and innovation with alternative stage1 implementations.  We'll be forced to keep some semblance of a stable (versioned) interface between coreos/rocket and stage1, specifically the entrypoints and what arguments they receive.  When acquiring a stage1 becomes a normal part of the process, acquiring and utilizing alternatives based on capsule, qemu, or novm becomes more palatable and likely to succeed.
I don't want to force an unwelcome separation onto the project, if this is a show-stopper and I'm off my rocker, please say so.

My sense is that we are up to a month or two away from there being a stable ABI between the stage1 and the stage0. Once we have a documented ABI then we can move the project out but it just seems too soon to do that.
I support the goal but having another thing to Godep and manage will just be too much of a pain right now. I learned this with etcd/etcdctl/go-etcd when we thought separating the projects would be helpful but it just made it harder to make changes and coordinate everything early on.
Lets put it into a new package in this repo then we can pull it out when we feel happy about the versioning and documentation.
Metric for pulling it out into a new repo: two subsequent versions, say 0.7.0 and 0.8.0, can use the same stage1.

@vcaputo SGTM, agree with most of your points (except I don't like "rats nest" :-)
@philips I am broadly sympathetic, although I don't see where Godep would come into it since this should really be a complete decoupling in codebases.
Let's push for this in the O(1.5) month timeframe.

@jonboulle Godeps comes into this because we rely on several appc/spec packages and they would need to stay in sync until the ABI is established.

If I recall correctly we recently decided against this OOB.
@philips?

We now have two stage1s in the rkt repository: the traditional one and the one based on lkvm.
@iaguis I recall the same.

can we have some explanation of the rationale for posterity?

Should we have a new discussion around this issue? @jonboulle @alban @s-urbaniak @lucab

When this issue was filed, rkt didn't have a build mechanism to build different flavors of stage1. Since then, new flavors were added:

the "src" flavor: https://github.com/coreos/rkt/pull/634
./configure --with-stage1-flavors=...: https://github.com/coreos/rkt/pull/1100
the "lkvm" flavor: https://github.com/coreos/rkt/pull/1174
the "fly" flavor: https://github.com/coreos/rkt/pull/1833

The different flavors (lkvm, coreos) share code together. Moving them to separate repositories complicate things.
@tmrts did you have something specific you wanted to discuss?

@alban I just wanted to confirm that our view hasn't changed and close this issue if it hasn't

We can reopen if needed.


Fixed cmake error BUILD_FAILED when building pcre2 and sqlite3

Changed the targets vcpkg_cmake_configure to vcpkg_configure_cmake and vcpkg_cmake_install to vcpkg_install_cmake for both sqlite3 and pcre2, so cmake is able to build them properly.
And I removed vcpkg_cmake_config_fixup target from the portfile.cmake for sqlite3, because this target doesn't exist anymore.
Now they can be build properly.

Changed the targets vcpkg_cmake_configure to vcpkg_configure_cmake and vcpkg_cmake_install to vcpkg_install_cmake for both sqlite3 and pcre2, so cmake is able to build them properly.

What isn't built properly? I don't see a reason for going back to the deprecated functions.


Node API works in browser but failed through postman

When calling API https://mywebsite.com/api/register through browser and it returns correct response from server as { "error": false, "message": "Hello World" } If we hit the same request from postman then it returns with some html content as Please enable JavaScript to view the page content. Below is node side code:
var express = require("express");
var app = express();
var bodyParser = require("body-parser");
var router = express.Router();
var cors = require('cors');
var fs = require('fs');
var https = require('https');
app.use(cors());
app.use(bodyParser.json({limit: '50mb'}));
app.use(bodyParser.urlencoded({ limit: '50mb', "extended": false }));
app.use(express.static(path.join(__dirname, './demo')));
router.get("/", function (req, res) {
res.json({ "error": false, "message": "Hello World" });
});
app.use('/api', router);
var privateKey = fs.readFileSync('key.pem');
var certificate = fs.readFileSync('cert.pem');
var passphrase = 'pass';
var credentials = {key: privateKey, cert: certificate, passphrase:passphrase};
var server = https.createServer(credentials, app).listen(config.port);
server.timeout = 3600000;

The API requests are working through the browser but not through the postman. What will be the possible errors? Please help. Thanks!

Hi @avinash0 I just tried that server with Postman and didn't get the response you stated. Can you provide more details so I can reproduce the issue?

My projects is located at the internal server which can access only with the internal network. And the public domain url is redirected to the internal url
The domain url is mapped to a child domain. Lets say https://mywebsite.com is redirected to the internal child domain https://internalwebsite.com.
The project is located in https://internalwebsite.com
Let say, if we hit API https://mywebsite.com/api then it should return as { "error": false, "message": "Hello World" }
Mysterious thing is sometime it is working fine and sometime return the below resposne:

Hi @avinash0 that is weird indeed. Nothing in this module would produce that HTML, so I have no idea where it would be coming from. I can take a look if you can provide all the source code and instructions necessary to reproduce the issue, otherwise, not sure what I can help with.

Closing since there hasn't been any further response.

F5 Web Accelerator may be causing this as a security measure, but I admit, I don't understand it enough to know why... I found this trying to figure it out. https://rehmann.co/blog/scraping-by/

My IT guy verified it was F5 doing this. It's a bot scan prevention thing. I'm just putting this here for you guys and future me to find... This describes what's happening quite well: https://stackoverflow.com/questions/34707532/python-post-request-not-returning-html-requesting-javascript-be-enabled


An error occurred getting the Power BI groups. Please contact your system administrator.

We have 4 environments with same link. Configured Only one application to use on them all.
The application works as it meant for in One machine not on other machines.
Application version is 7.3 PU15 to 8.1 PU20.
Thanks in advance

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

ID: 717d397d-4186-c7de-431d-3260d3f0484a
Version Independent ID: bdd7545b-5970-93e0-5cbb-a59da4ace01b
Content: Configure Power BI integration for workspaces - Finance & Operations | Dynamics 365 | #MSDyn365FO
Content Source: articles/dev-itpro/analytics/configure-power-bi-integration.md
Service: dynamics-ax-platform
GitHub Login: @MilindaV2
Microsoft Alias: milindav

@Phineas019 Thanks for contacting us. Unfortunately, we can't handle support requests for the product here. To contact support, consider one of the options described here:  https://docs.microsoft.com/en-us/dynamics365/unified-operations/dev-itpro/lifecycle-services/lcs-support


EPIC: Extend PBJ to support forward compatability

Problem
The BN uses PBJ which currently does not support forward compatibility
Solution
Enhance PBJ to support forward compatibility as this will allow for the BN to pass data it doesn't yet full understand
Alternatives
No response

Can you flesh this out? I don't know what isn't supported.


Move subquery paramers binding down to FetchOrEval#toInputColOrFetchRef.

Summary of the changes / Why this improves CrateDB
Previously, due to the early binding of subquery parameters FetchOrEval
would not process SelectSymbols in certain cases. For example, if an
execution plan contains the RootBoundaries operator and the FetchOrEval
operator with SelectSymbols in its outputs it might happen that subquery
binding won't even happen. This change fixes this corner case.
Resolves https://github.com/crate/crate/issues/9158
Checklist

[x] User relevant changes are recorded in CHANGES.txt
[x] Touched code is covered by tests
[x] Documentation has been updated if necessary
[x] CLA is signed
[x] This does not contain breaking changes, or if it does:

It is released within a major release
It is recorded in CHANGES.txt
It was marked as deprecated in an earlier release if possible
You've thought about the consequences and other components are adapted
(E.g. AdminUI)

Is this already fixed on master? otherwise I'd suggest to point this PR to master and let mergify do the backport by adding the v/4.0 label.

@seut yes. We found it out while working on aggregate/window function filter functionality and it has of course corresponding to that feature tests, commit message, etc. That's why I opened a new PR. Would it be ok if i just crosslink these changes?

Would it be ok if I just crosslink these changes?

@kovrus Ah right, sure.


fix(evil): wrong-number-of-arguments when run-hooks window-selection-change-functions

window-selection-change-functions expects hooks to have one argument.
Edit: functions such as treemacs--follow-project (by treemacs-project-follow-mode) will raise errors after calling evil-window-vsplit.

Thanks for your help


Audio only in Android 9 and Android X

Bug
I am going to make a video stream in my app
and i am using react-native-video.
The problem is when i play the video, it is audio only in some video. Either in debug and release mode.
I don't know what is wrong. I already use the latest version of react-native-video.
is this a library's bug or is it because the video that i used?
Platform

Which player are you experiencing the problem on:

Android ExoPlayer
Android MediaPlayer

Environment info

React native info output:
    OS: macOS Mojave 10.14.6
    CPU: (8) x64 Intel(R) Core(TM) i5-8257U CPU @ 1.40GHz
    Memory: 29.91 MB / 8.00 GB
    Shell: 3.2.57 - /bin/bash
  Binaries:
    Node: 12.16.1 - /usr/local/bin/node
    Yarn: 1.22.0 - ~/.yarn/bin/yarn
    npm: 6.13.4 - /usr/local/bin/npm
    Watchman: 4.9.0 - /usr/local/bin/watchman
  SDKs:
    Android SDK:
      API Levels: 23, 28, 29
      Build Tools: 28.0.3, 29.0.3
      System Images: android-28 | Google Play Intel x86 Atom
  IDEs:
    Android Studio: 3.5 AI-191.8<IP_ADDRESS>10548
    Xcode: /undefined - /usr/bin/xcodebuild
  npmPackages:
    react: 16.9.0 => 16.9.0
    react-native: 0.61.5 => 0.61.5
  npmGlobalPackages:
    react-native-cli: 2.0.1

Library version: 5.0.2
Steps To Reproduce

I install react-native-video and import the library
I use the  Component in my
when i react-native run-android and then i play the video
some video play normally, but some video, i only hear the audio

Expected behaviour

every video play normally, with the video and the audio

Reproducible sample code
import React from 'react';
import Video from 'react-native-video';

class App extends React.Component {
<Video
  volume={1.0}
  resizeMode="cover"
  onEnd={this.onEnd}
  onLoad={this.onLoad}
  paused={this.state.paused}
  style={this.state.playPreview == true ? styles.fullscreenVideo : styles.video}
  onProgress={this.onProgress}
  onLoadStart={this.onLoadStart}
  ref={videoPlayer => (this.videoPlayer = videoPlayer)}
  // source={{ uri: 'https://vjs.zencdn.net/v/oceans.mp4' }} // it works with this video
  source={{ uri: "https://qubisastorage.blob.core.windows.net/files/courses/22/video/22-20191125154522.mp4" }} // but doesn't work with this video
/>
}

export default App

Video sample
this video works well
https://vjs.zencdn.net/v/oceans.mp4
this video doesn't work well
https://qubisastorage.blob.core.windows.net/files/courses/22/video/22-20191125154522.mp4

No sound during video playback, on iPhone 7p devices
System:
    OS: macOS 10.15.4
    CPU: (6) x64 Intel(R) Core(TM) i5-8500 CPU @ 3.00GHz
    Memory: 6.89 GB / 24.00 GB
    Shell: 5.7.1 - /bin/zsh
  Binaries:
    Node: 10.16.0 - /usr/local/bin/node
    Yarn: 1.21.1 - /usr/local/bin/yarn
    npm: 6.14.2 - /usr/local/bin/npm
    Watchman: 4.9.0 - /usr/local/bin/watchman
  SDKs:
    iOS SDK:
      Platforms: iOS 13.4, DriverKit 19.0, macOS 10.15, tvOS 13.4, watchOS 6.2
    Android SDK:
      API Levels: 24, 25, 26, 27, 28, 29
      Build Tools: 27.0.3, 28.0.3, 29.0.2
      System Images: android-28 | Google APIs Intel x86 Atom, android-28 | Google Play Intel x86 Atom
  IDEs:
    Android Studio: 3.6 AI-192.7<IP_ADDRESS>08749
    Xcode: 11.4/11E146 - /usr/bin/xcodebuild
  npmPackages:
    react: 16.9.0 => 16.9.0
    react-native: 0.61.5 => 0.61.5


PHPMD complains about itself

I just installed PHPMD in PhpStorm 6.0  and ran it for the first time in my project. The very first warning was about itself and not my code; I can't figure out how to to keep it from checking its own files and checking mine instead.

phpmd: PHP Warning:  date(): It is not safe to rely on the system's timezone settings. You are required to use the date.timezone setting or the date_default_timezone_set() function. In case you used any of those methods and you are still getting this warning, you most likely misspelled the timezone identifier. We selected 'America/Chicago' for 'CDT/-5.0/DST' instead in /usr/lib/share/pear/PHP/PMD/Renderer/XMLRenderer.php on line 97

Hi, we also have this issue using PHP 5.6.
PHP Warning:  date_default_timezone_get(): It is not safe to rely on the system's timezone settings. You are *required* to use the date.timezone setting or the date_default_timezone_set() function. In case you used any of those methods and you are still getting this warning, you most likely misspelled the timezone identifier. We selected the timezone 'UTC' for now, but please set date.timezone to select your timezone. in /home/ec2-user/.jenkins/jobs/fatcap-php-services-testing/workspace/vendor/phpmd/phpmd/src/bin/phpmd on line 24
PHP Stack trace:
PHP   1. {main}() /home/ec2-user/.jenkins/jobs/fatcap-php-services-testing/workspace/vendor/phpmd/phpmd/src/bin/phpmd:0
PHP   2. date_default_timezone_get() /home/ec2-user/.jenkins/jobs/fatcap-php-services-testing/workspace/vendor/phpmd/phpmd/src/bin/phpmd:24

The fix applied is not valid for PHP 5.6, the warning is explicit, even if the system timezone is set we should set a timezone explicitly in the source code.
So (at least for PHP 5.6) we should replace ...
if (!ini_get('date.timezone') && !date_default_timezone_get()) {
    date_default_timezone_set('UTC');
}

by ...
if (version_compare(phpversion(), '5.6.0', '>=') ||  (!ini_get('date.timezone') && !date_default_timezone_get())) {
    date_default_timezone_set('UTC');
}

@hbandura: Yes, even after @manuelpichler's fix, a timezone warning is still raised.
I've just created PR #297 which completely fixes the issue (@bgaillard: even on PHP 5.6).


Please tell me how to remove the parent category when getting the value from the variable {{collection.title}}?

When I get the value from the variable {{collection.title}}, because the category with the note is nested, then I get the value of the parent and nested category, but I only need the value of the nested category.

How about {{ collection.title.split("/") | last }}?

Closed due to inactivity.


Multiline comments?

Are there plans to add (nestable) multiline comments?

Yes, I intend to add multiline comments.


import scss files, there is  no outputs and no error log when using webpack

If i import css file, It's ok.
But if i import scss file, then there is no error log and no outputs
I have install webpack 1.14.0, node-sass 4.1.1
the config file for scss is:
{test: /.scss$/,loaders: ["style", "css", "sass"]}
When I write scss with error, there will show error logs to  tell me where the error is.
Have you ever met this problem before? Or any suggestion for this problem？

It looks like you just deleted our lovely crafted issue template. It was there for good reasons. Please help us solving your issue by answering the questions asked in this template. I'm closing this. Please either update the issue with the template and reopen, or open a new issue.


Based off PR #522 - not sure how to get GitHub to PR against it

Based off https://github.com/lazerwalker/azure-mud/pull/522
https://github.com/lazerwalker/azure-mud/compare/ea96b96..feaa92a
The issue is that Alexei is working off another repo and I don't really know how to reconcile this, so this is MR'd against main, but you should review it against Kawa's last commit. There is always the concern of "but how will people know about this" oh well

ohhh, hmm, I didn't realize there were newer commits from their past PR than are now in main. Going to move discussion to chat.


Indexing error in find_dofs

Hi,
first off, great library: Indeed fills an important niche, thanks for creating!
While using it, being new to finite element methods, I have encountered an error that I am not sure how to fix (whether it's a wrong use on my end or a problem with the code). When trying to implement Dirichlet boundary conditions (following example 14) in a MeshTet, I am encountering a numpy indexing error.
My script condenses to the following:
m = MeshTet3D(p=some_nodes, t=some_elements, boundaries=some_boundaries)
e = ElementTetP1()
basis = InteriorBasis(m, e)
surface_basis = FacetBasis(m, e, facets=m.boundaries["top"])
surface_dofs = surface_basis.find_dofs()['all'].all()

The boundaries dict contains, among others, an array of faces belonging to the top side. Things run smoothly up to the surface_basis.find_dofs() call, where the following error is raised:
~/.local/lib/python3.8/site-packages/skfem/mesh/mesh3d/mesh3d.py in boundary_edges(self)
     39                               facets]))
     40                    for itr in range(self.facets.shape[0])])).T, axis=1)
---> 41         return np.nonzero((self.edges.T[:, None] == boundary_edges)
     42                           .all(-1).any(-1))[0]
     43
AttributeError: 'bool' object has no attribute 'all'

The error occurs because the shapes of self.edges.T[:,None], (180751, 1, 2), and boundary_edges, (107076, 2), do not match for an elementwise comparison. From a quick skim of the code, I assume that some sort of element comparison is taking place in that function, but I am not sure about that (without some in depth study of the source). I thought to open this issue because maybe there is something fundamentally wrong in my code snipped above and I hope it's not too stupid, or otherwise it might be a bug.
Thanks in advance,
Malte

I might have an idea what it is: can you try basis.find_dofs instead?

Thanks for the quick reply!
I tried it out, unfortunately the same error appears.
I will try to dig deeper to find the root of this. Is there some way that I could validate that the objects are properly initialized?

Can you run this with the Python debugger and print the call stack with where?
Something like ipython test.py -m pdb then in the resulting prompt press c and wait until the execution halts to the error. Then write where and paste the output here.

A better option would be that you could share a mesh which causes this issue with a minimal working example but I'm not sure if you are able to do that.

Additional remarks:

if you want to find all boundary DOF's, you can also simply call basis.find_dofs() without parameters.
which version are you using?

You can find the version as follows:
In [1]: from skfem import __version__
In [2]: __version__
Out[2]: '2.0.0'

Mention also if you are using the version from the source code repository directly.

The following runs fine on the master branch:

from skfem import *
m = MeshTet()
m.define_boundary("top", lambda x: x[2] == 1.0)
e = ElementTetP1()
basis = InteriorBasis(m, e)
surface_basis = FacetBasis(m, e, facets=m.boundaries["top"])
surface_dofs = surface_basis.find_dofs()

Does it run on your computer?

I am running the latest version (2.0.0) from PyPI. Your example also runs fine:

The following runs fine when using the master branch:
from skfem import *
m = MeshTet()
m.define_boundary("top", lambda x: x[2] == 1.0)
e = ElementTetP1()
basis = InteriorBasis(m, e)
surface_basis = FacetBasis(m, e, facets=m.boundaries["top"])
surface_dofs = surface_basis.find_dofs()

Does it run on your computer?

While building a smaller example to share here (which runs fine!), I think I might have found the root of the error:
An array of size (180751, 107076, 2) is just to large for my RAM (that would be about 36GB for a bool array and I'm working on a notebook with 8GB). Instead of raising a memory error, however, the comparison just silently fails and returns a single bool. Not something I would have expected, else I would have checked that earlier..
I will find a way to get along with that and at some point switch to a larger machine. Thanks for the quick help and feedback!

That's actually a good point, I think we do not want to create such a large array because the memory runs out easily. I think there is room for optimization here. I'll open another issue.

Since you are using linear elements, as a workaround you should be able to dig those DOF-numbers directly from the mesh structure. E.g., if you are assembling only scalar problems, just use the indices boundary_dofs = m.boundary_nodes().

Here are some pointers what you might want to do: https://scikit-fem.readthedocs.io/en/latest/bcs.html#indexing-of-the-degrees-of-freedom

There is now a new version available with some optimizations to boundary_edges: https://pypi.org/project/scikit-fem/

Thanks a lot for coming back so quickly!
Some feedback: I tested the new version, it roughly halves the required memory. Still a bit too large (19.3 GB) to
run on the notebook, but should run on the bigger machine.
In my particular case, I have already circumvented the problem following your advice about using the mesh structure. Thanks!

Alright, seems that I need to look into it further. I hope it doesn't anymore stem from boundary_edges because I was able to reduce the search space significantly.
The issue is most likely related to some recent changes to find_dofs done in 2.0.0.

Ok it's still there. What a tricky issue. I need to meditate a bit.

Alright, now I managed to run this with 10 million tets and less than 11 gigs of memory. Hopefully it's enough for you, it kind of starts to touch the boundaries of this library.


Typo

The following examples show how you can use connection strings for connecting to on-ine deployments and authentication scenarios.

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

ID: ce8213b8-93bc-dee0-5b82-973db16f03fa
Version Independent ID: 161512ac-81c1-4163-20e1-e440497d2e24
Content: Use connection strings in XRM tooling to connect to Common Data Service (Common Data Service) - PowerApps
Content Source: powerapps-docs/developer/common-data-service/xrm-tooling/use-connection-strings-xrm-tooling-connect.md
Service: powerapps
GitHub Login: @MattB-msft
Microsoft Alias: nabuthuk

@highpowerxh Thanks for reporting this issue. It's fixed now, and will be updated in our next daily live publish. Please check back in a day.
Closing the issue.


Error on onValidStateChange is not a function

Can you please check this. I just installed the package and add the credit card UI to the screen and this error comes after that ... can you please look into this
Unhandled JS Exception: TypeError: onValidStateChange is not a function

This error is located at:
    in ForwardRef (at UserPaymentInfo.js:145)
    in RCTView (at UserPaymentInfo.js:142)
    in RCTScrollContentView (at ScrollView.js:1063)
    in RCTScrollView (at ScrollView.js:1196)
    in ScrollView (at UserPaymentInfo.js:114)
    in UserPaymentInfo (created by ConnectFunction)
    in ConnectFunction (at Profile.js:27)
    in RCTView (at Tabs.js:60)
    in RCTView (at Tabs.js:57)
    in RCTView (at Tabs.js:17)
    in Tabs (at Profile.js:31)
    in Profile (created by ConnectFunction)
    in ConnectFunction (at CustomerProfile.js:51)
    in RCTView (at ScreenContainer.js:54)
    in RCTView (at ImageBackground.js:61)
    in ImageBackground (at ScreenContainer.js:41)
    in RCTView (at ScreenContainer.js:39)
    in RCTView (at KeyboardAvoidingView.js:204)
    in KeyboardAvoidingView (at ScreenContainer.js:36)
    in RCTView (at src/index.tsx:65)
    in SafeAreaView (at ScreenContainer.js:35)
    in ScreenContainer (created by ConnectFunction)
    in ConnectFunction (at CustomerProfile.js:50)
    in CustomerProfileScreen (created by ConnectFunction)
    in ConnectFunction (at SceneView.tsx:98)
    in StaticContainer
    in StaticContainer (at SceneView.tsx:89)
    in EnsureSingleNavigator (at SceneView.tsx:88)
    in SceneView (at useDescriptors.tsx:125)
    in RCTView (at CardContainer.tsx:190)
    in RCTView (at CardContainer.tsx:189)
    in RCTView (at Card.tsx:525)
    in RCTView (at createAnimatedComponent.js:144)
    in AnimatedComponent (at createAnimatedComponent.js:194)
    in ForwardRef(AnimatedComponentWrapper) (at Card.tsx:508)
    in PanGestureHandler (at Card.tsx:501)
    in RCTView (at createAnimatedComponent.js:144)
    in AnimatedComponent (at createAnimatedComponent.js:194)
    in ForwardRef(AnimatedComponentWrapper) (at Card.tsx:497)
    in RCTView (at Card.tsx:491)
    in Card (at CardContainer.tsx:156)
    in CardContainer (at CardStack.tsx:531)
    in RCTView (at CardStack.tsx:106)
    in MaybeScreen (at CardStack.tsx:524)
    in RCTView (at CardStack.tsx:89)
    in MaybeScreenContainer (at CardStack.tsx:422)
    in CardStack (at StackView.tsx:420)
    in KeyboardManager (at StackView.tsx:418)
    in RNCSafeAreaView (at src/index.tsx:28)
    in SafeAreaProvider (at SafeAreaProviderCompat.tsx:42)
    in SafeAreaProviderCompat (at StackView.tsx:415)
    in RCTView (at StackView.tsx:414)
    in StackView (at createStackNavigator.tsx:67)
    in StackNavigator (at UserStack.js:46)
    in UserStack (at AppNavigator.js:278)
    in EnsureSingleNavigator (at BaseNavigationContainer.tsx:268)
    in ForwardRef(BaseNavigationContainer) (at NavigationContainer.tsx:39)
    in ThemeProvider (at NavigationContainer.tsx:38)
    in ForwardRef(NavigationContainer) (at AppNavigator.js:287)
    in AppNavigator (created by ConnectFunction)
    in ConnectFunction (at App.js:14)
    in PersistGate (at App.js:13)
    in Provider (at App.js:12)
    in App (at renderApplication.js:45)
    in RCTView (at AppContainer.js:109)
    in RCTView (at AppContainer.js:135)
    in AppContainer (at renderApplication.js:39)

Same here!

Same here!

Add the onValidStateChange prop and you will be fine even if it is not doing anything but can be useful for enabling and disabling the pay or done button

Add the onValidStateChange prop and you will be fine even if it is not doing anything but can be useful for enabling and disabling the pay or done button

I am programming from the beginning with javascript and the example of this code is in typescript, can someone give me help? I have already tried several times and it gives me errors in all of them.
import * as React from 'react';
import {
Button,
KeyboardAvoidingView,
StyleSheet,
Platform,
} from 'react-native';
import CreditCard from 'react-native-credit-card-form-ui';
export default function App() {
const creditCardRef = React.useRef(null);
const handleSubmit = React.useCallback(() => {
if (this.creditCardRef.current) {
const { error, data } = this.creditCardRef.current.submit();
console.log('ERROR: ', error);
console.log('CARD DATA: ', data);
}
}, []);
return (
<KeyboardAvoidingView
behavior={Platform.OS === 'ios' ? 'padding' : 'height'}
keyboardVerticalOffset={20}
style={styles.container}
>

);
}
const styles = StyleSheet.create({
container: {
flex: 1,
alignItems: 'center',
justifyContent: 'center',
},
});

@megaroga What is the error

Hi @rebirthtobi, i have this code:
`import * as React from 'react';
import {
Button,
KeyboardAvoidingView,
StyleSheet,
Platform,
} from 'react-native';
import CreditCard from 'react-native-credit-card-form-ui';
export default function App() {
const creditCardRef = React.useRef(null);
const handleSubmit = React.useCallback(() => {
if (this.creditCardRef.current) {
const { error, data } = this.creditCardRef.current.submit();
console.log('ERROR: ', error);
console.log('CARD DATA: ', data);
}
}, []);
return (
<KeyboardAvoidingView
behavior={Platform.OS === 'ios' ? 'padding' : 'height'}
keyboardVerticalOffset={20}
style={styles.container}
>

);
}
const styles = StyleSheet.create({
container: {
flex: 1,
alignItems: 'center',
justifyContent: 'center',
},
});
`
when executing I get the following error:
undefined is not an object (evaluating 'this.creditCardRef')
====================================
by leaving everything the same according to the example and changing only this line
const creditCardRef = React.useRef(null);
the error is the following
onValidStateChange is not a function. (In 'onValidStateChange(false,validationErrors)', 'onValidStateChange is undefined')

Add onValidStateChange to the properties sent along with CreditCard component

@rebirthtobi could you explain how to use it along with CreditCard Component or how should it be used within the prop

@ryan2412
<CreditCard ref={this.creditCardRef} onValidStateChange={onValidStateChange} />

This gives an error "Reference error: cant find variable onValidStateChange"

What I did is just to define onValidStateChange that does nothing and passed it as the value for onValidStateChange


luci-app-opkg: fix and update Japanese translation

Fixed and updated Japanese translations.
Signed-off-by: INAGAKI Hiroshi<EMAIL_ADDRESS>

I mistook a branch to use for PR...


Error creating map 1024^3

Running the command "/gen grass flat 1024 1024 1024 WORLDNAME" creates the following error on console:

ERROR: Map failed to save: System.OverflowException: Arithmetic operation resulted in an overflow.
at System.IO.BufferedStream.Write(Byte[] array, Int32 offset, Int32 count)
at fCraft.MapConversion.MapFCMv3.Save(Map mapToSave, String fileName)
at fCraft.MapConversion.MapUtility.TrySave(Map mapToSave, String fileName, MapFormat format)

Fixed upstream in https://github.com/fragmer/fCraft/commit/24ef690812a1f6c3310e35a8e31da6565a085c5f ?

Fixed (thanks UnknownShadow200)


Hopefully fix some confusions regarding alt courses

Mention the full word "alternative" instead of the short form "alt" which may cause confusion to non-native speakers. Also change "/" to "," for the two parts of HTC course.

Accessibility, I'm here for it 👍
It looks like at least one "alt" remains (Math for CS). Can you make sure you've caught them all and then I'll approve.


Google Login won't redirect on Android 5.0 and 6.0

Hi,
I successfully added the Google Login to my application using Xamarin.Auth version v1.5.0-alpha-12.
I followed the ComicBook sample using Xamarin.Auth.Presenters and it works fine on Android 7.0.
However on Android 5.0 and 6.0, everytime I login, it won't redirect back to the application but it shows the Google homepage instead.
My redirect url is: {mypackagename}:/oauth2redirect
I tried to upgrade to alpha-15, but the problem persists.
What did I miss?
Thank you in advance for your response.

@choirunnisa
Interesting. Emulator? Chrome browser version?
Update Chrome 1st.
Can you try installing another Chrome version? Maybe Firefox or Opera?

@moljac Both emulator and real device.
I tried to install Google Chrome on the emulator, and now it works! Hooray!
Thank you very much for your help!
And thank you for dedicating your time in building, updating, and responding to issues on this awesome plugin.
Your work is very much appreciated :)

@moljac I'm seeing the same behavior but on a Nexus 5X running Android 7.1.2. I have the latest updates for Chrome according to the store (Chrome 58.0.3029.83). Using Xamarin.Auth 1.5.0-alpha-15.
new OAuth2Authenticator(
                    clientId: "<snip>.apps.googleusercontent.com",
                    clientSecret: null,
                    authorizeUrl: new Uri("https://accounts.google.com/o/oauth2/auth"),
                    accessTokenUrl: new Uri("https://www.googleapis.com/oauth2/v4/token"),
                    redirectUrl: new Uri("com.mealplanner:/oauth2redirect"),
                    scope: "https://www.googleapis.com/auth/userinfo.email https://www.googleapis.com/auth/plus.login",
                    getUsernameAsync: null,
                    isUsingNativeUI: true
                    );

I get the request for permissions page, followed by a very brief moment of "Loading [redirect url]" (see image below), then brought to the google homepage.

My redirect matches my manifest:
<manifest xmlns:android="http://schemas.android.com/apk/res/android" android:installLocation="auto" package="com.mealplanner" ...
Let me know what other diagnostics I can provide. Thanks.

@choirunnisa
Thanks a lot for your kind words.
I'm closing this one.


Add wps server to docker compose

This an an work in progress pull request to add the wps server & the services to the local setup for the async architecture.
It should allow to work without external depencencies.

The current state is the following:

the images of the services are included, so that we can collect the json configurations & put it into the wps json configuration folder
the wps itself works internally of the docker-compose

Things I would like to improve:

the wrapper settings to access the file storage contents must be changes, so that it is possible to access it from the wps server itself (a link directly to the minio:9000 port should be ok)
it currently seems that the wps has problems in creating the urls to get the result values: It always just uses localhost:8080, even that the settings of the wps are different.

Ok, state of stuff:

I fixed the urls that are used to store inside of the wrappers in the filestorage (but the urls that are in the db are no longer direct resolveable from the outside, there you must replace  http://filestorage:9000/riesgosfiles/{hash} with http://localhost/api/v1/files/{hash}. But as long as the handling stays inside the docker compose, this is not a large issue.
(It would be possible to replace those urls when quering via the fastapi).

I have from time to time issues with the settings of the wps server. While I wrote a script to update those settings on each start, it seems not that it is recognized immediately). Could be do to refresh settings of the wps server. (I still would assume that those should be used as its the same way as using the wps/server settings UI). Workaround for those: In case you start with a clean docker setup => restart the wps server container.

It seems that we have issues when we try to send requests to the wps servers at the very same point in time.

org.n52.geoprocessing.wps.client.WPSClientException: Could not decode Inputstream.
	at org.n52.geoprocessing.wps.client.WPSClientSession.parseInputStreamToString(WPSClientSession.java:620)
	at org.n52.geoprocessing.wps.client.WPSClientSession.retrieveResponseOrExceptionReportInpustream(WPSClientSession.java:559)
	at org.n52.geoprocessing.wps.client.WPSClientSession.retrieveCapsViaGET(WPSClientSession.java:536)
	at org.n52.geoprocessing.wps.client.WPSClientSession.connect(WPSClientSession.java:170)
	at org.n.riesgos.asyncwrapper.process.wps.WPSClientService.establishWPSConnection(WPSClientService.kt:17)
	at org.n.riesgos.asyncwrapper.dummy.AbstractWrapper.runOneJob(AbstractWrapper.kt:334)
	at org.n.riesgos.asyncwrapper.dummy.AbstractWrapper.fillConstraintsAndRun(AbstractWrapper.kt:261)
	at org.n.riesgos.asyncwrapper.dummy.AbstractWrapper.run(AbstractWrapper.kt:56)
	at org.n.riesgos.asyncwrapper.events.OrderMessageHandler.handleMessage(OrderMessageHandler.kt:23)
	at org.n.riesgos.asyncwrapper.pulsar.PulsarConsumer.receiveMessages(PulsarConsumer.kt:22)
	at org.n.riesgos.asyncwrapper.pulsar.PulsarConsumer.run(PulsarConsumer.kt:13)
	at java.base/java.lang.Thread.run(Thread.java:831)
Caused by: javax.xml.stream.XMLStreamException: ParseError at [row,col]:[162,182]
Message: The value of attribute "xlin:href" associated with an element type "ows:Metadata" must not contain the '<' character.
	at java.xml/com.sun.org.apache.xerces.internal.impl.XMLStreamReaderImpl.next(XMLStreamReaderImpl.java:652)
	at java.xml/com.sun.xml.internal.stream.XMLEventReaderImpl.nextEvent(XMLEventReaderImpl.java:83)
	at org.n52.geoprocessing.wps.client.decoder.stream.GetCapabilitiesResponseDecoder.createProcessSummary(GetCapabilitiesResponseDecoder.java:161)
	at org.n52.geoprocessing.wps.client.decoder.stream.GetCapabilitiesResponseDecoder.readContents(GetCapabilitiesResponseDecoder.java:131)
	at org.n52.geoprocessing.wps.client.decoder.stream.GetCapabilitiesResponseDecoder.readCapabilities(GetCapabilitiesResponseDecoder.java:77)
	at org.n52.geoprocessing.wps.client.xml.WPSResponseReader.readCapabilitiesResponse20(WPSResponseReader.java:105)
	at org.n52.geoprocessing.wps.client.xml.WPSResponseReader.readElement(WPSResponseReader.java:54)
	at org.n52.geoprocessing.wps.client.WPSClientSession.parseInputStreamToString(WPSClientSession.java:618)
	... 11 more

It only happens if it tries to work with the wps client at the very same time in multple docker containers. It could be a reason that I mount the very same jar for multiple containers. Maybe @bpross-52n knows more about this.

Maybe the problems with the java exceptions can be solved by mounting the jar file in read only mode (could be that docker then takes modifications of this as a new layer in its container file system - or it could throw an error when trying to modify - I'm not sure here).


WSL failed to start after system update

Windows ver 10.0.19559.1000

I have a similar problem which this might be related to  which is mentioned in issue #4860 .
you can create an event log as described here:
https://github.com/Microsoft/WSL/blob/master/CONTRIBUTING.md#8-detailed-logs
and then read the *.etl file with tracerpt
maybe if you track down the issue it might be the same as mentioned above

#4860 is fixed for me in 19564, try updating and testing if it's related it should be fixed as well.

This seems like a duplicate of #4860 as others have pointed out. Please try 19564 and see if it's resolved, if it isn't then please let us know!
/dupe #4860


[Bug][Android] Lack of full screen ability for MediaElement

Description
When displaying the built in transport controls, MediaElement has the ability to full screen the video on iOS. However, it doesn't on Android. I realise this is due to the default platform implementations.
However, until now many users have used FormsVideoLibrary code to play videos, which is part of our docs. The biggest issue users have raised with us is the lack of full screen ability on Android.
In moving to MediaElement, users will expect to be able to easily fullscreen their video on Android.

While this issue was there first, the other one has a bit more detail already. In either case, it doesn't make much sense to have duplicate error :)
@LegendaryB @mrvancor sorry for the long wait! This is on our radar, simply not implemented yet, sorry.

While this issue was there first, the other one has a bit more detail already. In either case, it doesn't make much sense to have duplicate error :)
@LegendaryB @mrvancor sorry for the long wait! This is on our radar, simply not implemented yet, sorry.

Any workaround for make this possible on Android?

still waiting for a workaround for this :(

does anyone know a solution to this


Perf: deleting a large selection of fields in the Structure Editor has quadratic cost

Describe the bug
The Structure Editor handles operations on ranges of fields suboptimally, which can lead to long freezes in normal usage.
To Reproduce
Steps to reproduce the behavior:

Go to 'Structure Editor'
Change size to 1024/2048/4096, generating over a thousand 'undefined' one-byte fields
Select almost all fields, from, let's say, 16 to the end of the structure
Press 'delete'
Observe Ghidra freezing for a couple seconds/a dozen seconds/about a minute.

Expected behavior
The structure editor should be able to perform this action nearly instantly, as it already does if you truncate the size of the struct by lowering the size.
Screenshots
N/A
Attachments
N/A
Environment (please complete the following information):

OS: Linux ceres 5.4.23 #1-NixOS SMP Fri Feb 28 16:22:29 UTC 2020 x86_64 GNU/Linux
Java Version: 11.0.7 (as reported by Ghidra)
Ghidra Version: 9.1.2; Build PUBLIC 2020-Feb-12 1149 EST

Additional context
N/A

Just double checked and can confirm this has been fixed in 9.2.


upload fonts/images with max_age

Summary of changes
Asana Ticket: 🔧 upload images/fonts to S3 when deploying

Assigned to: @paulswartz

@paulswartz is this what you had in mind?


could u pls add another web index page for disguise as a web?

could u pls add another web index page for disguise as a web under websocket mode?

You can do this using Nginx's reverse proxy!


neorv32_twi_start_trans() does not work with prsc>1

Hi Stephan,
at the moment i do some tests with the TWI. My belonging C code looks like follows:
/** Includes **/
/* Standard libs */
#include <string.h>     // memset

/* Processor */
#include <neorv32.h>
#include <neorv32_twi.h>

/**
 *  main
 */
int main()
{
    /** Variables **/

    /* init UART */
    neorv32_uart0_setup(115200, PARITY_NONE, FLOW_CONTROL_NONE);

    /* entry message */
    neorv32_uart0_print("Interfaces I2C Core of NEORV32\n");

    /* setup I2C */
    neorv32_twi_disable();
        // neorv32_twi_setup(uint8_t prsc)
    neorv32_twi_setup(4);
    neorv32_twi_mack_enable();
    neorv32_twi_enable();

    /* write to I2C */
    neorv32_twi_start_trans(0x55);
    neorv32_twi_generate_stop();

    while ( 1 ) { } // inifinite loop
    return 0;
}

For asserting the transfer i use the NEORV32 function neorv32_twi_start_trans. For small prescaler values like 1 works this very well.
PRSC=1:

For PRSC=4 is only the starbit sended and then happends nothing

I did some investigation, the issue seems like follows, if a higher PRSC value is used, then is the FSM in the startbit state arbiter=101 and the write of ths address - in my case 0x55 - is skipped:

One possible solution could be to poll the busy flag, this would lead to an change in neorv32_twi_start_trans() and the IRQ driven I2C would not work anymore.
What are your thoughts?
BR,
Andreas

Hey Andreas,
you need to generate a start conditions first before you can do actual TWI transfers. So you need to call neorv32_twi_generate_start first:
    /* write to I2C */
    neorv32_twi_generate_start(); // generate START conditions before triggering actual transfers
    neorv32_twi_start_trans(0x55);
    neorv32_twi_generate_stop();

I just had a quick look at the TWI documentation - and it seems like the documentation is not really clear here. I need to fix that! :wink:

I just realized I messed up neorv32_twi_start_trans() and neorv32_twi_trans() - my fault! 😅
I will have a look at this!

Your fix is great workaround for now, but I think a dedicated hardware reset for all IO/peripheral devices would be a better permanent solution. Otherwise we might get trouble with this again in other places. I think I'll start working on that reset issue on another branch then you can have a look, too, and we can make sure to eliminate all those hardware-vs-simulation issue.

Hi Stephan, it works! Thanks for the really fast providing the fix
    /* write to I2C */
    neorv32_twi_start_trans(0x55);
    neorv32_twi_generate_stop();

the belonging waveform with global reset

BR,
Andreas

Great to hear! You have used the modified version from #334, right?

yes #334 was used


fix : change runner to mac-os

Codecov Report

Merging #49 (4ea9be5) into main (252ab6f) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main      #49   +/-   ##
=======================================
  Coverage   44.79%   44.79%
=======================================
  Files          18       18
  Lines         605      605
=======================================
  Hits          271      271
  Misses        310      310
  Partials       24       24

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 252ab6f...4ea9be5. Read the comment docs.


Error in version 2.0

In the new version. the app is empty 😢, but in the 1.4.8 version, it works fine

Hi there, could you provide the information that was asked for in the issue template?
Most notably your OS version and hyper config contents. Thank you.

@MauroNavaLuevanos see #2826 #2829, probably is the hyperborder plugin

Thanks for the report 🙏 .
We're tracking this work over here: #2826


maintainers: update roshaen

Description of changes

Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
24.11 Release Notes (or backporting 23.11 and 24.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

You should have a single commit for the info update. (Currently there are 2).
Also, in this update you should probably update all packages that were under your name.

Hey @Steinhagen
Made all the required changes


Include the base address of the RTC peripheral when disabling the RTC WDT

Closes #126
Interestingly even though flashing now succeeds, the application doesn't seem to start - I'm not sure if this is just my devkit though.

Why were certain boards previously working if we were writing to invalid addresses? The linked issue seems to only effect certain boards, I'm not really sure what's going on here.


Docker mod

I've been considering using this with nzbget instead of radarr/sonarr to simplify my workflow and have the conversions happen on a local disk instead of a NAS. I don't see any up to date containers out there, but came across this little tidbit: https://blog.linuxserver.io/2019/09/14/customizing-our-containers/
Any reason SMA couldn't work as a docker mod, allowing us the capability to add it into any container that could use it? If not, any interest in doing so, @mdhiggins? If not, I'd be willing to give it a try.

The mod is interesting, I can look into how that works can probably adapt most of the main functionality from the sonarr/radarr containers

https://hub.docker.com/repository/docker/mdhiggins/sma-mod
Got a prelim version working in my tests after much trial and error
Mount the same points as the radarr/sonarr containers for the volumes and see if that works

It doesn't seem to be copying in postSonarr.sh to the right spot in my testing, but I am still poking at it.

Looks like postRadarr.sh wasn't included but postSonarr.sh should have been, added in the latest update

Tested and both postSonarr.sh and postRadarr.sh are included in latest update

21-4-10 14:58:17.0|Error|postSonarr.sh|/usr/local/sma/postSonarr.sh: line 3: /venv/bin/python3: No such file or directory 21-4-10 14:58:17.0|Debug|CustomScript|Executed external script: /usr/local/sma/postSonarr.sh - Status: 127 21-4-10 14:58:17.0|Debug|CustomScript|Script Output: 4/10/2021 9:58:17 PM - Error - /usr/local/sma/postSonarr.sh: line 3: /venv/bin/python3: No such file or directory
is the issue I am getting in the trace logs for sonarr. any thoughts?

Looks like your SMA_PATH environment variable isn't being set, just pushed an update that attempts to fix this, the old method of setting I was using was apparently not working, let me know if this new update fixes it

So interestingly enough...with the newest fix I'm able to view SMA_PATH inside the container, but the script isn't able to access it:
root@07c26184cf9e:/usr/local/sma# echo $SMA_PATH
/usr/local/sma
root@07c26184cf9e:/usr/local/sma# ./postSonarr.sh
./postSonarr.sh: line 3: /venv/bin/python3: No such file or directory
root@07c26184cf9e:/usr/local/sma# cat postSonarr.sh
#!/usr/bin/env bash

$SMA_PATH/venv/bin/python3 $SMA_PATH/postSonarr.py

Given that the path is relative to the container, any issues with possibly hardcoding this?

Try the latest update, think I got it working

We're close, I think.
Here's what I see in Sonarr:
21-4-12 16:39:40.4|Error|postSonarr.sh|/usr/local/sma/postSonarr.sh: line 3: /usr/local/sma: Is a directory
21-4-12 16:39:40.5|Warn|SonarrErrorPipeline|Invalid request Validation failed:
 -- : Script exited with code: 126

And in the container:
root@835d044acdcb:/usr/local/sma#` ./postSonarr.sh
./postSonarr.sh: line 3: /usr/local/sma: Is a directory

Whoops looks like the new method was adding a newline at the end of the variable, just fixed it, container rebuilding now

When the script is called by Sonarr with a tv show:
2021-04-12 20:25:27 - SonarrPostProcess - INFO - Sonarr extra script post processing started.
2021-04-12 20:25:27 - resources.readsettings - INFO - /usr/local/sma/venv/bin/python3
2021-04-12 20:25:27 - resources.readsettings - INFO - Loading config file /usr/local/sma/config/autoProcess.ini.
2021-04-12 20:25:27 - SonarrPostProcess - ERROR - Error reading environment variables
Traceback (most recent call last):
  File "/usr/local/sma/postSonarr.py", line 178, in <module>
    tvdb_id = int(os.environ.get('sonarr_series_tvdbid'))
TypeError: int() argument must be a string, a bytes-like object or a number, not 'NoneType'

Within Sonarr when testing the script:
21-4-12 20:23:23.6|Warn|SonarrErrorPipeline|Invalid request Validation failed:
 -- : Script exited with code: 1

The closest thing I could find was this issue:
https://forums.sonarr.tv/t/custom-script-exited-with-code-1-error-postsonarr-py-script-from-sickbeard-mp4-automator-on-macos-mojave/22749/4
But it looks like a python 2 issue.

Alright so using the with-contenv bash seems to include the variable but subsequently wipe the environment variables set by sonarr. This is getting frustrating
Just uploaded an alternative solution see if that works

This leads back to the previous issue of:
sonarr        | [Error] postSonarr.sh: /usr/local/sma/postSonarr.sh: line 3: /venv/bin/python3: No such file or directory
sonarr        | [Warn] SonarrErrorPipeline: Invalid request Validation failed:
sonarr        |  -- : Script exited with code: 127

in the sonarr logs.

Alright I give up, latest update just handles it at the script level which should behave

Testing it right now. Thanks for working on this!

Testing in Sonarr works! I will report back with trying an actual conversion through Sonarr, but thanks for this! To make a script for NZBGet, the format would be similar to the other scripts I assume? Calling python, then NZBGetpostprocess.py?

And it's kicking it off! Perfect, thank you!

NZBGet is a little weird cause it doesn't execute the script but actually reads and executes the python code natively so a shell script won't work, you'd probably have to copy or simlink NZBGetPostProcess to the final script destination

Actually for NZBGet just overriding the SMA_PATH environment variable might be best to have it go to the NZBGet ScriptDir

Looks like by default it goes to your /downloads/scripts

The rabbit hole goes deeper...because linuxserver is using alpine as its base image, the sma-config script fails. I'm taking a crack at it to see what I can get working, and if I'm successful, I'll send a PR over to you

I'll probably have to include different tags (debian vs alpine etc)

Yeah, looks like it. Do you have any bandwidth for this? I've been meaning to poke at it, but haven't had much of a chance to do so.

I might have some time later this week I'll update here if I get something together

Added an alpine tag how's that work for you?

'''
nzbget        | [cont-init.d] 90-sma-config: executing...
nzbget        | /var/run/s6/etc/cont-init.d/90-sma-config: line 38: dpkg: command not found
nzbget        | tar: short read
nzbget        | chown: cannot access '/usr/local/bin/ffmpeg': No such file or directory
nzbget        | chown: cannot access '/usr/local/bin/ffprobe': No such file or directory
nzbget        | chmod: cannot access '/usr/local/bin/ffmpeg': No such file or directory
nzbget        | chmod: cannot access '/usr/local/bin/ffprobe': No such file or directory
nzbget        | ERROR:root:autoProcess.ini does not exist
nzbget        | [cont-init.d] 90-sma-config: exited 1.
'''

You, kind sir, have a pull request: https://github.com/mdhiggins/sma-mod/pull/1

Running 90-sma-config after that shows no errors. Testing out the post process script to see what happens

nzbget        | [INFO] Executing post-process-script NZBGetPostProcess.py for The.Great.North.S01E09.720p.WEB.H264-CAKES
nzbget        | [INFO] NZBGetPostProcess: Traceback (most recent call last):
nzbget        | [INFO] NZBGetPostProcess:   File "/config/scripts/NZBGetPostProcess.py", line 84, in <module>
nzbget        | [INFO] NZBGetPostProcess:     from resources.mediaprocessor import MediaProcessor
nzbget        | [INFO] NZBGetPostProcess:   File "/usr/local/sma/resources/mediaprocessor.py", line 12, in <module>
nzbget        | [INFO] NZBGetPostProcess:     from resources.metadata import Metadata
nzbget        | [INFO] NZBGetPostProcess:   File "/usr/local/sma/resources/metadata.py", line 12, in <module>
nzbget        | [INFO] NZBGetPostProcess:     import tmdbsimple as tmdb
nzbget        | [INFO] NZBGetPostProcess: ModuleNotFoundError: No module named 'tmdbsimple'
nzbget        | [ERROR] NZBGetPostProcess: Wrong path to sickbeard_mp4_automator: /usr/local/sma/

But I see you've merged the PR...let me run it from the top and try again

Hm this will be tricky, its failing there because NZBGet is using its own python environment while everything is installed under the virtual environment

Thanks man. I'll try to look into this as well. Do you happen to accept donations in any form? I'd love to at least be able to buy you a beer for all this work

Does modifying the shebang at the top of the NZBGetPostProcess.py file help?
#!/usr/bin/env python3
to
#!/usr/local/sma/venv/bin/python3

Trying it right now

Sidenote...my method of installing dpkg is failing...perhaps it needs to be moved into the Dockerfile. Manually running "apk add dpkg" in the container allows me to run 90-sma-config with no errors

Success, it works!!! A little too well....NZBGetPostProcess is sending this to SMA in the logs:
2021-04-19 06:13:52 - NZBGetPostProcess - INFO - ======================
2021-04-19 06:13:52 - NZBGetPostProcess - INFO - 0
2021-04-19 06:13:53 - NZBGetPostProcess - INFO - 0
2021-04-19 06:13:53 - NZBGetPostProcess - INFO - 1
2021-04-19 06:13:54 - NZBGetPostProcess - INFO - 1
2021-04-19 06:13:54 - NZBGetPostProcess - INFO - 1
2021-04-19 06:13:55 - NZBGetPostProcess - INFO - 2
2021-04-19 06:13:56 - NZBGetPostProcess - INFO - 3
2021-04-19 06:13:56 - NZBGetPostProcess - INFO - 3
2021-04-19 06:13:57 - NZBGetPostProcess - INFO - 4
2021-04-19 06:13:57 - NZBGetPostProcess - INFO - 5
2021-04-19 06:13:58 - NZBGetPostProcess - INFO - 5
2021-04-19 06:13:58 - NZBGetPostProcess - INFO - 6
2021-04-19 06:13:59 - NZBGetPostProcess - INFO - 6
2021-04-19 06:13:59 - NZBGetPostProcess - INFO - 8
2021-04-19 06:14:00 - NZBGetPostProcess - INFO - 9
2021-04-19 06:14:00 - NZBGetPostProcess - INFO - 10
2021-04-19 06:14:01 - NZBGetPostProcess - INFO - 10
2021-04-19 06:14:01 - NZBGetPostProcess - INFO - 11
2021-04-19 06:14:02 - NZBGetPostProcess - INFO - 13

But that's a mere nitpick! I think we almost have it

Yep, as far as I can tell it's working smoothly!

Looks interesting. I may have to give this a spin and test it out for my setup.

@mdhiggins When you get a moment, could you take a look at https://github.com/mdhiggins/sma-mod/pull/3? I don't know how I'd test a docker mod like this locally

I will say you will probably need to add Mesa upgrades if your docker isn't running Mesa 20.1 or higher or you can't do HEVC encoding/transcoding with VAAPI. Linuxserver/Jellyfin as an AMD/VAAPI docker mod for just that. I recommend using Ubuntu as the base for your docker mod since that is what so many of the Linuxserver dockers use.

My latest PR against alpine didn't fix the dpkg install issue. Any thoughts on what I might be doing wrong?

The Linuxserver Radarr docker is built again Ubuntu 20.04 Focal. You might want to give that a try.
FROM ghcr.io/linuxserver/baseimage-ubuntu:focal
https://github.com/linuxserver/docker-radarr/blob/master/Dockerfile

The issue is NZBGet is built against alpine, so the same scripts don't work between the two, as far as I can tell

0755ce3eb2ba3b527c6c3f347d98ebbb47a9c237
Just pushed this update which uses the arch command instead of dpkg to determine the architecture, I think I account for all possible outputs, let me know

Out of curiosity, does this file work with NZBGet?
#!/usr/local/sma/venv/bin/python3

import sys
import os

path = os.path.join(os.path.split(sys.argv[0])[0], "NZBGetPostProcess.py")
exec(open(path).read())

Idea being that I could have that as a wrapper file and include it with sma-mod without needing to duplicate NZBGetPostProcess.py and modify the shebang
It should just include and exec NZBGetPostProcess.py using its own shebang but I'm not sure how NZBGet will handle reading it, if you could test and let me know that would be great

Updated sma-mod to include this file

So far, so good! SMA is running successfully, and I will update as soon as it completes, but it looks great, as do your changes with the arch command. I'm able to get the container up and running without having to go in and install dpkg manually.
I am seeing this error in the logs:
2021-04-29 19:58:39 - NZBGetPostProcess - ERROR - Pymediainfo exception.
Traceback (most recent call last):
  File "/usr/local/sma/resources/mediaprocessor.py", line 306, in isValidSource
    media_info = MediaInfo.parse(inputfile)
  File "/usr/local/sma/venv/lib/python3.8/site-packages/pymediainfo/__init__.py", line 422, in parse
    lib, handle, lib_version_str, lib_version = cls._get_library(library_file)
  File "/usr/local/sma/venv/lib/python3.8/site-packages/pymediainfo/__init__.py", line 324, in _get_library
    raise OSError("Failed to load library")
OSError: Failed to load library

From looking at the log file, this was happening previously as well, and I didn't notice. Any idea what may be causing it, or if it's safe to ignore?

This error is from pymediainfo missing its requirement package. I updated the 90-sma-config file to include this package on ubuntu based installations (python-pymediainfo)
Alpine doesn't have a package yet though it is on edge so I guess its being added at some point (py3-pymediainfo)
Either way it can ignored, its used to pull some supplemental data about the source file but not required

I tried to add this to linuxserver/radarr and it is giving me an error.
[mod-init] Attempting to run Docker Modification Logic
[mod-init] Applying mdhiggins/sma-mod files to container
gzip: stdin: not in gzip format
tar: Child returned status 1
tar: Error is not recoverable: exiting now

version: "2.1"
services:
radarr:
image: linuxserver/radarr:latest
container_name: radarr
hostname: radarr
network_mode: bridge
environment:
- PUID=1000
- PGID=1000
- TZ=America/Chicago
- DOCKER_MODS=mdhiggins/sma-mod #optional
volumes:
- /home/docker/radarr:/config
- /srv/media/libraries:/libraries
- /srv/media/downloads:/downloads
ports:
- 7878:7878
restart: unless-stopped

Might want to just retry that. Seems like maybe a bad download

https://github.com/linuxserver/docker-mods/issues/160
Seems like a fix is in the works and should be making its way out


Add return null response

Fixed an error when you tried to return null as an answer
Related with: #93

I dont understand why this fails


Heroheader scroll

adds parallax affect in header when page is scrolled
creates a reusable scroll tracking component

Looks good to me! I noticed that I can see the background very faintly behind the "what people are saying" callouts. Is that intentional? fine if so!

wow, nice catch! this should fix it...


AppVeyor Nighlty Build failure

Travis returned a failure on this nightly build on revision, however no failure is visible in the log output and the nightly release was correctly uploaded:
https://ci.appveyor.com/project/omichel/webots-sj08b?fullLog=true

It seems it just hit the timeout (1 hour).
Not sure how to fix it.

And it was not uploaded.

You're right indeed that's just a timeout issue, the creation of the package is extremely long, it started at 0:45:00 and hit the timeout :-/


Implement peer discovery

[ ] There should be a list of bootstrap nodes somewhere. Maybe put them on a GitHub repo.
[ ] Nodes should be able to get initial nodes from bootstrap node list and discover the whole network. Nodes do this by introducing themselves through http POST requests to other nodes and discovering their peers through GET requests. This happens in a fixed interval.
[ ] Nodes can filter the messages they want to receive from their peers through Bloom Filters. They submit their Bloom Filter in the introduction phase.

would like to work on this one if it's not taken

For the list of bootstrap nodes, you can check how Bitcoin does it: https://github.com/bitcoin/bitcoin/tree/master/contrib/seeds
It makes sens to not to hardcode a repository's url inside the code because the repo might be hosted on different places. the list of seed nodes can indeed be kept on a repo or any hosted page, but the list should be compiled into the node and the node should not need to fetch them from the public internet.
what do you think? I can work on that.

Well done!

As we discussed, we may not need the bloom filters for now. If they happen to come up later we can discuss the details on them again. but for now, we may close the issue :)

@n1rna Agreed


Update wp-connectors.md

Added "Sharing of flows that use Connectors" as a sub-heading.

@ganesh-sanap
Thanks for the correction. Updating this topic now.


Added Docker file for burrow

You can now run Burrow from docker. You'd need to overwrite the config with whatever you want, and mount a log folder.

This looks like a good idea. Can you add some comments to it to the effect of what's in the PR? Specifically, what you need to modify/do to get burrow to work when you use the Docker file.

I'm keen to help with this PR as the docker container in Dockerhub is about 3 months old now.
@toddpalino Is there anything specific that you are looking for? The Dockerfile submitted here just builds the Docker image and it can be run by mounting the configuration into the container.
I also have a branch here (https://github.com/linkedin/Burrow/compare/master...jgmchan:dockerfile?expand=1) which does the same thing but builds it on the alpine image which makes it much smaller but I'm happy to help on either.

I forgot about this PR because we have it in our private repo now. If you want to take over, using the Alpine base image, that is great

@jgmchan @alexnederlof @toddpalino
I have done my own variant of a Dockerfile and can provide my implementation as a starting point on another PR. Then we can disucss more. Me version started the the Docker file here and I expanded it a bit.

@jgmchan The only real thing I was looking for was a little bit of commenting or documentation as to exactly what needs to be changed in the Dockerfile for someone to make it work. Nothing significant, just a note or two with "change this value to X as appropriate for A, B, and C".

@toddpalino Ok, no problem. Do you want this in the README or just as a comment in the Dockerfile? I think I would prefer the README.

If you don't think a comment in the Dockerfile is the way to go, you can add a wiki page with a link from the README

Closing this out in favor of PR #74


Update getting_started_email_activity_api.md

Needed to update with the contains ability. We did not have any docs about it and it is needed for searching for events.

Hello @kylearoberts,Thanks again for the PR! We appreciate your contribution and look forward to                                              continued collaboration. Thanks!                                             Team SendGrid DX


How to make sure a item is a valid linear item or not?

I have found this filed in a spineItem object, but it always blank

I also tried the method spine.isValidLinearItem(spineItem), but the returns are always undefiend.
So I am confused, how can I make sure a item is a valid linear item or not?
BTW, I am building a pagination system so I need to skip no-linear items.
Thanks

http://www.idpf.org/epub/301/spec/epub-publications.html#sec-itemref-elem
In EPUB's OPF spine@itemref element, missing linear attribute is synonymous with linear="yes". Content creators include linear="no" in order to explicitly exclude a spine item from the primary reading flow.
Related Readium code:
https://github.com/readium/readium-shared-js/blob/develop/js/models/spine_item.js#L62
    /**
     * A flag indicating whether the spineItem has the attribute linear, which
     * is either yes or no.  Default is yes.
     *
     * @property linear
     * @type String
     * @default  yes
     */
    this.linear = itemData.linear ? itemData.linear.toLowerCase() : itemData.linear;

As you can see, testing for spineItem.linear == "no"  is sufficient to determine that a spine item is non-linear. However, spine.isValidLinearItem(spineItemIndex) adds an additional check for "handleLinear", which is basically a way for an application to activate / deactivate support for the "linear" property (so, make sure to invoke spine.handleLinear(true) before using the API).
https://github.com/readium/readium-shared-js/blob/develop/js/models/spine.js#L65
    var _handleLinear = false;

    /**
     * Sets a flag indicating that the app handles linear spine items
     *
     * @method     handleLinear
     * @param      {Boolean} handleLinear  boolean flag
     */
    this.handleLinear = function(handleLinear) {
        _handleLinear = handleLinear;
    };

    function isValidLinearItem(item) {
        return !_handleLinear || item.linear !== "no";
    }

    /**
     * Checks if a spine item is linear.
     *
     * @method     isValidLinearItem
     * @param      {Number} index  index of a spine item
     * @return     {Boolean} TRUE if the app does not handle linear items or if the item is linear.
    */
    this.isValidLinearItem = function(index) {

        if(!isValidIndex(index)) {
            return undefined;
        }

        return isValidLinearItem(this.item(index));
    };

Thank you, sir.


Out Of Date - Tutorial: Azure Active Directory integration with Greenhouse

The instructions provided are out of date and do not work due to changes in how Azure does things and per Greenhouse's internal support page  https://support.greenhouse.io/hc/en-us/articles/360004745971-Azure-Active-Directory-Azure-AD-

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

ID: d9b260cc-3e08-e2a0-b478-30a745cd2c97
Version Independent ID: 5f7d594b-b6ec-cee2-9a8c-d2e6e9c4783b
Content: Tutorial: Azure Active Directory integration with Greenhouse
Content Source: articles/active-directory/saas-apps/greenhouse-tutorial.md
Service: active-directory
Sub-service: saas-app-tutorial
GitHub Login: @jeevansd
Microsoft Alias: jeedes

@FlynnLives
Thanks for your feedback! I have assigned this issue to the author who will investigate and update as appropriate.

@chetansriv  and @v-hagamp Can you please retest the app and update the instructions?

Issue resolved as suggested above.
#please-close

Issue resolved as suggested above.
#please-close


Redact sensitive informations like password in chart url

The chart name can also be an URL that contains basic authentication information. This information are printed out by helmfile. This PR tries to redact this information.
$ cat <<EOT >> helmfile.yaml
releases:
    - name: test
      chart<EMAIL_ADDRESS>EOT

$ helmfile template
> Templating release=test<EMAIL_ADDRESS>
With this patch the output looks like:
> Templating release=test<EMAIL_ADDRESS>
Moved from roboll/helmfile => roboll/helmfile#2132

@dol Thank you for your contribution, this looks great! Could you please add a simple unit test to the function?

@itscaro Will try.

I extended the test cases and removed the redacting in some of the methods based on this list.

Method
helm command supports URL

BuildDeps
no

UpdateDeps
no

SyncRelease
yes

TemplateRelease
yes

DiffRelease
yes

Lint
no

Fetch
yes

ChartPull (OCI)
no

ChartExport (OCI)
no


Fixes: Update folders if users reselect the folders after submitting

fixes #228
fixes the file input is not reset after a successfully submission
add redirection link to submitted manifest in pop up box

@milen-sage @clarisse-lau
Current develop instance is on this branch. Please feel free to test the changes, and let me know if you still have the same issue.

Thanks for the fast resolution @rrchai ! Feel free to merge when ready.


Running  golangci-lint reports multiple issues

Description
Running golangci-lint in api-gateway results in reporting of the following violations

Report
Error return value is not checked

internal/processing/processing_test.go:1037:29
				rulev1alpha1.AddToScheme(scheme)

Error return value is not checked

internal/processing/processing_test.go:1038:34
				networkingv1beta1.AddToScheme(scheme)

Error return value is not checked

internal/processing/processing_test.go:1039:31
				gatewayv1beta1.AddToScheme(scheme)

Error return value of `c.Delete` is not checked

controllers/api_controller_integration_test.go:93:18
			defer c.Delete(context.TODO(), instance)

Error return value of `c.Delete` is not checked

controllers/api_controller_integration_test.go:158:18
			defer c.Delete(context.TODO(), instance)

Error return value of `c.Delete` is not checked

controllers/api_controller_integration_test.go:197:21
						defer c.Delete(context.TODO(), instance)

func `(*oauth).Validate` is unused

internal/validation/oauth.go:11:17
func (o *oauth) Validate(api *gatewayv1beta1.APIRule) error {

type `allow` is unused

internal/validation/allow.go:9:6
type allow struct{}

type `oauth` is unused

internal/validation/oauth.go:9:6
type oauth struct{}

func `(*allow).Validate` is unused

internal/validation/allow.go:11:17
func (a *allow) Validate(api *gatewayv1beta1.APIRule) error {

SA1019: "io/ioutil" has been deprecated since Go 1.16: As of Go 1.16, the same functionality is now provided by package io or package os, and those implementations should be preferred in new code. See the specific function documentation for details.

internal/helpers/config.go:4:2
	"io/ioutil"

SA1019: fake.NewFakeClientWithScheme is deprecated: Please use NewClientBuilder instead.

controllers/api_controller_test.go:149:23
		mgr: getFakeManager(fake.NewFakeClientWithScheme(scheme.Scheme, objects...), scheme.Scheme),

SA1019: "io/ioutil" has been deprecated since Go 1.16: As of Go 1.16, the same functionality is now provided by package io or package os, and those implementations should be preferred in new code. See the specific function documentation for details.

controllers/api_controller_test.go:7:2
	"io/ioutil"

SA9003: empty branch
internal/processing/processing.go:67:11
			} else if config.JWTHandler == helpers.JWT_HANDLER_ISTIO {

SA9003: empty branch
internal/processing/processing.go:78:9
	} else if config.JWTHandler == helpers.JWT_HANDLER_ISTIO {

Expected result
No violation is reported

Running golagnci-lint ./... reports no issue


Using factory cacheSrcStorage is throwing getCacheDir() method not found exception.

When I tried accessing cacheSrcStorage to fetch the corresponding native url of web link, getCacheDir() method is being reported as missing.

Sorry for the late reply. I just fix this bug together with some else.  Thank you so much for helping!


docs(tw): translate state-and-lifecycle

This tech terms I keep the original:

local state
destroy
merge
shallow

@chloewlin Thank you for reviewing. In the above some terms we should check need translate or not.

@neighborhood999 no problem. Let me merge this in and we can open another discussion on individual terms.


Debug value popup does not respect defined font family in settings

Issue Type: Bug
Greetings,
when inspecting the value of a variable in debug mode (paused) the popup that displays the value (no longer) respects the font defined in the settings as "editor.fontFamily" but falls back to a generic monospace font.

Can you reproduce with latest vscode insiders? https://code.visualstudio.com/insiders/

Unfortunately the same is present in the insider version:

With the following minimal settings:
{
    "editor.fontFamily": "DejaVu Sans Mono",
    "debug.console.fontFamily": "DejaVu Sans Mono",
    "terminal.integrated.fontFamily": "DejaVu Sans Mono"
}

PS: your check was JS while my example is C#; might this make a difference?

How about without those settings?
The debugged language should not matter.

Unfortunately without any settings, the situation is the same.

Interestingly VSCode appears to respect the system-wide setting of my monospace font, which results in the code font being DejaVu Sans Mono.
The debug console however uses the same (incorrect) default monospace font (Droid Sans Mono according to the dev tools) as the popup:

@joaomoreno is there some Linux Snap issue that the fonts are not respected?

Yes. There used to be, should be fixed by now tho.

I don't believe it is a snap issue; I can reproduce the same behavior when I download VSCode directly and run the extracted version from the .tar.gz.


Release sheared model without re-training?

Hello, will you release sheared models without re-training? Currently, there are only some graphs showing the averaged ACC of each training checkpoint. I think it would be instructive if we can evaulate the un-trained model!

Do you mean the pruned models before training? Would love to release those if they are helpful.
Stay tuned, I will get them uploaded within the week!

Is there any update? Thanks in advance. @xiamengzhou

Apologies for the delay! You can find the two pruned models on HF now.
princeton-nlp/Sheared-LLaMA-1.3B-Pruned, princeton-nlp/Sheared-LLaMA-2.7B-Pruned

Thank you very much!


API Documentation using Wyam

As mentioned on #157, I've started prototyping our API documentation using Wyam.
This is completely open to discussion, as I want us all to be comfortable with it.
If you want to try it out locally, then here's what to do...

Checkout the "docs/wyam-prototype" branch
Go to the "/docs" folder...
a. Run the "get-wyam.cmd" batch file, (this will download Wyam via NuGet)
b. From a command-prompt, (in the "/docs" folder), run this...

.wyam\Wyam.exe --watch --preview

c. Open a browser at http://localhost:5080/
I haven't added any CSS styling. For the markup, I've mostly took inspiration from the Wyam.io docs.
Like I say, this is totally open for discussion, I'm a believer in Cunningham's Law! :stuck_out_tongue_closed_eyes:

Given that GitHub Pages can now generate from the /docs folder, (see their blog post about it). We could switch back to using GitHub Pages/Jekyll for handling the docs?
As much as I like Wyam, I appreciate that it is another layer to the build process.
Thoughts?

After playing with wyam, I'd be inclined to agree. I think the only benefit it brings is being able to generate the API docs, but I'm guessing we might still be able to try and automate that in some way. Even if not, I think simplicity has to be key otherwise we just won't keep up to it.

If we do close this PR though, it's whether we need to re-open an issue to write the docs :)

Yeah, we'll close this one off, (no immediate rush though), but let's keep the branch until we've made a decision and/or have done the switch over. I'll create a separate issue for the Jekyll-based docs (or re-open the old one)

Closing this PR. Raised new issue #192


Connect nodejs svcs to rookout

in the two nodejs services (currency, payment) i connected rookout, changed the logger to Winston, and removed init of stackdriver/tracing/etc

/jira open task


[backport] Fix: code quality issues

Backport of #4587

[automatic post] Jenkins, test this please.

Jenkins CI test (for commit f0bf3c87a862dcd187a1d61c8a1e0183ccbfebab, target branch v8) succeeded!


⚠ GitHub has degraded performance

In 7c32d87, GitHub (https://www.githubstatus.com/api/v2/status.json) experienced degraded performance:

HTTP code: 200
Response time: 250 ms

Resolved: GitHub performance has improved in 5f96bc8 after 18 minutes.


docs: add profilee to openSourceAppList.tsx

Added an OS profile link builder app in collection
✅ Checklist

[x] I have followed every step in the contributing guide (updated 2022-10-06).
[x] The PR title follows the convention we established conventional-commit
[x] I performed a functional test on my final commit

Changelog
Update openSourceAppList.tsx and added an Profilee app. Profilee is a link in the bio profile builder.

Screenshots

@juliusmarminge please check this one


[BUG] Cannot Create TicketNotesChild resources; 400 bad request

Description:
The New-AutotaskAPIResource function is not able to POST TicketNotesChild resources because it is not replacing the string "{ParentID}" for the $resourceURL variable
Steps to reproduce the behavior:
$Body = New-AutotaskBody -Resource TicketNotesChild -NoContent
$Body.id = "0"
$Body.ticketID = "$($variable_containing_ticketID)"
$Body.description = "This is a note added by script"
$Body.noteType = "1"
$Body.publish = "2"
$Body.title = "Note Added By Script"
New-AutotaskAPIResource -Resource TicketNotesChild -body $body -verbose

Results in the following error :
VERBOSE: POST https://webservices3.autotask.net/atservicesrest//V1.0/Tickets/{parentId}/Notes with -1-byte payload
New-AutotaskAPIResource : Connecting to the Autotask API failed. The remote server returned an error: (400) Bad Request.
At line:1 char:1
+ New-AutotaskAPIResource -Resource TicketNotesChild -Body $body -Verbo ...
+ ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    + CategoryInfo          : NotSpecified: (:) [Write-Error], WriteErrorException
    + FullyQualifiedErrorId : Microsoft.PowerShell.Commands.WriteErrorException,New-AutotaskAPIResource

Expected behavior
It creates the ticket note for the specified ticketID
Desktop:
OS: Windows 10
PowerShell version: 5.1.19041.1682

There was a version difference between the uploaded version to the gallery and the github version. this is resolved now. :)

Thank you for updating the gallery version to match GitHub's version however I am still getting a warning trying to create TicketNotesChild items (after updating to the latest module in the gallery):
WARNING: You must specify a parentId when creating a child resource


Add support for SSO providers (AD, LDAP, OIDC)

From what I can currently see, it doesn't seem like we have a resource for enabling LDAP/AD via cookbook. Is this something that can be added? Seems like we'd adopt a similar approach that was done for the postgres config.

[x] ActiveDirectory
[ ] LDAP

Updated the issue title to include all supported SSO providers.

Basic support to configure both remaining SSO providers (LDAP, OIDC) is implemented in the referenced PRs. Once those are merged this issue can be closed.

Closing, support has been added


How to install the esm-fold interface?

Hi, I have already built the environment with the environment.yml + installed openfold + dllogger.
I wonder how to call the esm-fold command.

Hi, I had your same issue and I solved it by running the following command from inside the esm cloned directory:
python3 -m pip install --no-deps .

After that I was able to run the esm-fold command. I think this is due to the fact that the fair-esm package installed with pip install fair-esm is v2.0.0, and the console scripts like esm-fold were added after that version. I hope this is also useful for you.

Thanks @dacolombo. I bypassed the issue by running scripts/fold.py inside the cloned repo.


Getting Invalid Location: ClientSideExtension.ApplicationCustomizer even after making changes based on latest release

I am using the Application Customizer to add some content on the pages. I changed the code based on latest release of Sharepoint framework extensions. I am still getting Invalid Location: ClientSideExtension.ApplicationCustomizer. Is there anything i am missing? I updated the extensions using npm install -g @microsoft/generator-sharepoint.  #817

Hi @thePaddyVP, this could mean that the extensions are not yet enabled in your tenant. Like with Dev Preview, extension RC will work within developer tenants and will be gradually extended to first release tenants within a day or two (just takes time to enable settings).
To be able to understand what might go wrong in your case, we would need to understand at least the tenant type you're using and potentially also the site template where you are doing testing...  Thx for the additional details advance.

Hi @VesaJuvonen I am using a developer tenant with a team site template. Let me know if you need more details.

I was seeing the same issue in a developer tenant. But only when using the debug querry string parameter. Deploying to the app catalog worked.
After creating a new site, things have been working in the new site. So I am guessing, there are some dependencies left over in my old site that I have been using for testing version 1.1.0 before.
@thePaddyVP Try to create a new site and see if it works there.

I experience similar issue I get the Invalid location: ClientSideExtension.ApplicationCustomizer error message in the console window - but the application customizer works as expected...

Experienced the same. New site worked for me as well.

Everything is so weird...
I'm getting Invalid location: ClientSideExtension.ApplicationCustomizer without any explanation if there is not properties parameter in debug url.
If I add "properties":{} then I'm first getting:
Uncaught (in promise) Error: Failed to create application customizer 'ClientSideExtension.ApplicationCustomizer.3a685093-7cf3-4e8f-9bd9-f1560ec66034'. Error information is '***Expecting a class that extends function t(){return null!==e&&e.apply(this,arguments)||this}.'.
    at listview-host-assembly_en-us_8faa48cbb0cd205bf7b5c7b0fd24d2ca.js:338

and then Invalid location: ClientSideExtension.ApplicationCustomizer
I'm on DEV tenant, brand new site collection.

We had some issues with the first release tenant enabling and extensions, but all should be now up and running after GA last week. If there are any issues with this still, just let us know and we can re-open if needed. Will close for now.


Log level in Azure doesn't match expected log level

DevEx/OpEx
The log level in Azure doesn't match the level within the log - it'll be easier to parse/search logs if the discrete log level matches

Tasks

[ ] Something.

Additional Context
Add any other context or screenshots about the work here.

Apparently this is just how Azure works, and we can use the query packs set up by TF to work around this display issue


add scratch policy exception support for os_content_type

Add support for os_content_type == "Scratch Image"
Relates: Jira ISV-1686

coverage: 72.765% (-8.6%) from 81.388%
when pulling 72c588621b730afc78c42fc3d0ccb2bf5cf853c9 on acornett21:scratch_support
into 14dfc9d1b0fc502a3f8bb338fde469b1004d73fe on redhat-openshift-ecosystem:main.


Version of vault

(Required) What is your Quests version / build number (do not say "latest")?
Quests : 3.3.4-b316
Vault : 1.6.6-b (the lateset)
(Required) Please describe the problem and how to replicate it. Any console errors?
I get this error :
[13:44:10] [Server thread/ERROR]: This error is likely caused by an incompatible version of Vault. Please consider updating. [13:44:10] [Server thread/WARN]: java.lang.NullPointerException [13:44:10] [Server thread/WARN]: 	at me.blackvein.quests.util.ItemUtil.getName(ItemUtil.java:217) [13:44:10] [Server thread/WARN]: 	at me.blackvein.quests.Quester.getObjectives(Quester.java:612) [13:44:10] [Server thread/WARN]: 	at me.blackvein.quests.Quester.getObjectivesReal(Quester.java:452) [13:44:10] [Server thread/WARN]: 	at me.blackvein.quests.Quester.takeQuest(Quester.java:416) [13:44:10] [Server thread/WARN]: 	at me.blackvein.quests.Quests.adminGive(Quests.java:941) [13:44:10] [Server thread/WARN]: 	at me.blackvein.quests.Quests.questAdminCommandHandler(Quests.java:702) [13:44:10] [Server thread/WARN]: 	at me.blackvein.quests.Quests.onCommand(Quests.java:689) [13:44:10] [Server thread/WARN]: 	at org.bukkit.command.PluginCommand.execute(PluginCommand.java:44) [13:44:10] [Server thread/WARN]: 	at org.bukkit.command.SimpleCommandMap.dispatch(SimpleCommandMap.java:151) [13:44:10] [Server thread/WARN]: 	at org.bukkit.craftbukkit.v1_12_R1.CraftServer.dispatchCommand(CraftServer.java:676) [13:44:10] [Server thread/WARN]: 	at net.minecraft.server.v1_12_R1.PlayerConnection.handleCommand(PlayerConnection.java:1425) [13:44:10] [Server thread/WARN]: 	at net.minecraft.server.v1_12_R1.PlayerConnection.a(PlayerConnection.java:1230) [13:44:10] [Server thread/WARN]: 	at net.minecraft.server.v1_12_R1.PacketPlayInChat.a(PacketPlayInChat.java:45) [13:44:10] [Server thread/WARN]: 	at net.minecraft.server.v1_12_R1.PacketPlayInChat.a(PacketPlayInChat.java:5) [13:44:10] [Server thread/WARN]: 	at net.minecraft.server.v1_12_R1.PlayerConnectionUtils.lambda$ensureMainThread$0(PlayerConnectionUtils.java:14) [13:44:10] [Server thread/WARN]: 	at java.util.concurrent.Executors$RunnableAdapter.call(Executors.java:511) [13:44:10] [Server thread/WARN]: 	at java.util.concurrent.FutureTask.run(FutureTask.java:266) [13:44:10] [Server thread/WARN]: 	at net.minecraft.server.v1_12_R1.SystemUtils.a(SourceFile:46) [13:44:10] [Server thread/WARN]: 	at net.minecraft.server.v1_12_R1.MinecraftServer.D(MinecraftServer.java:843) [13:44:10] [Server thread/WARN]: 	at net.minecraft.server.v1_12_R1.DedicatedServer.D(DedicatedServer.java:424) [13:44:10] [Server thread/WARN]: 	at net.minecraft.server.v1_12_R1.MinecraftServer.C(MinecraftServer.java:767) [13:44:10] [Server thread/WARN]: 	at net.minecraft.server.v1_12_R1.MinecraftServer.run(MinecraftServer.java:665) [13:44:10] [Server thread/WARN]: 	at java.lang.Thread.run(Thread.java:748)
Can i know what version of vault i have to use pls ?

That does not look like an official Vault build. Afaik, there are only "1.6.6" and "1.6.7" version. No "1.6.6-b" version.
Download official development builds here - https://ci.md-5.net/job/Vault/


TypeScript client types are not correct when no matching object exists

Describe the bug
const foo = await db.user({id: doesNotExist});
// foo is typed as a User, but is actually null!

Expected behavior
The type of db.user(...) to be User? reflecting the fact that it's possible that no such user exists.
Versions (please complete the following information):

OS: macOS
prisma CLI: prisma/1.19.0 (darwin-x64) node-v10.11.0
Prisma Server: Demo server on Prisma Cloud
"graphql-yoga": "^1.16.7", "prisma-client-lib": "^1.19.0"

Additional context
Data model:
type User {
  id: ID! @unique
  ...
}

This seems like a pretty major oversight to me. Either it should throw if no such object exists or it should have a nullable return type

I agree that return types should be changed to User | null. Its a common mistake to not check whether the fetched object exists and changing the typings prevents such errors.
TypeORM also returns User | null so it will be easy to adopt for people who are used to TypeORM.

This has been fixed with prisma cli version 1.32.0-beta.1 :D
I will be in tomorrow's 1.32 release as well.


[minor] added to to-do list, test pull request

Testing pull request. Edited to-do list for tasks to finish later

Merged. Closed.


[WIP] Fixes for tests and test workflow

We need to update our tests for new Python versions, as well as python-package.yml. I have started some work on this to get most ubuntu versions passing and get detailed workflow errors as the oncall but we would need to dedicate some real engineering time to make further fixes.
FYI for Ubuntu against 3.6, it doesn't seem possible to create working tests for this version anymore. See https://github.com/actions/setup-python/issues/544

I fixed most tests and updated the test run script. But there's a bit more to be done.
If someone could grab the commits locally and squash the commits into one, that would be nice. I currently can't reauthenticate with this account as the account is using my school email which I recently lost access to a couple days ago.
Recent run:
https://github.com/aws/amazon-codeguru-profiler-python-agent/actions/runs/7558959509
Only in 3.6, Windows having an issue with running profiler and creating profiles. In the acceptance tests, the profile is attempted to be created and then decoded in JSON but I made some prints to stdout and verified that the profile content is empty.
Also would be nice to fix --
Windows tests are flaky, need to be rerun a couple times to pass. Seems to be an issue with how Windows manages time, not sure.
FAILED test/unit/aws_lambda/test_profiler_decorator.py::TestWithParameters::test_last_call_duration_is_set_in_lambda_context_singleton - AssertionError: assert datetime.timedelta(0)


Add trv ocaml build tool

This adds the trv ocaml build tool to the system

Please rename the commit message according to CONTRIBUTING.md.

I have no idea why this failed @pSub. Any insights?

Travis fails because of
The log length has exceeded the limit of 4 MB (this usually means that the test suite is raising the same exception over and over).

I will build this locally.

Builds fine locally. Thanks.

This PR hangs travis build. Somebody cancel the job, please.

I have canceled it.


Sirupsen -> sirupsen

Change the logrus imports according to the github account name sirupsen/logrus

Hm, it seems "sirupsen/logrus" was renamed back and forth, closing it


YARD warning log is output when executing rubocop

In my application, when I run rubocop after requiring rubocop-yard, I get the following WARN log when parsing YARD.
root@eacd66a1e36e:/usr/src/app# bundle exec rubocop
[warn]: Invalid tag format for @return
[warn]: Invalid tag format for @return
Inspecting 1234 files
....

The problem is that the YARD notation is wrong, but I think that problem is better solved separately from rubocop.
YARD allows us to change the log level of a logger only for a specific block as follows.
YARD::Logger.instance.enter_level(YARD::Logger::ERROR) do
  # Write the logic to parse YARD here
end

What do you think about suppressing warning logs?
Let me know what you think!

Thank you for reporting.
I agree with you.
When rubocop is executed, the warning of YARD itself should not be output, but should be reported as an offenct of rubocop.

To reproduce the problem, could you add a test case that does not produce warnings, or provide a sample YARD case that does produce warnings?