After the migration of ionic3, the browser can not find the module "@ angular/platform-browser/src/facade/browser"

Ionic version:  (check one with "x")
[ ] 1.x
[ ] 2.x
[x] 3.x
I'm submitting a ...  (check one with "x")
[x] bug report
[ ] feature request
[ ] support request => Please do not submit support requests here, use one of these channels: https://forum.ionicframework.com/ or http://ionicworldwide.herokuapp.com/
Current behavior:
I in the command line, enter the ionic serve, the browser shows:
Runtime Error
Can not find module "@angular/platform-browser/src/facade/browser"
expected behavior:
I finally want to build into android, the debugger is also showing this error.
Copy Steps:
See the relevant code.
Related code:
I want to migrate from ionic2.0 to ionic3.0, the package.json changed, and the implementation of npm install.
Ionic3 - package.json
{
  "Name": "ionic-hello-world",
  "Author": "Ionic Framework",
  "Homepage": "http://ionicframework.com/",
  "Private": true,
  "Scripts": {
    "Clean": "ionic-app-scripts clean",
    "Build": "ionic-app-scripts build",
    "Ionic:" build ":" ionic-app-scripts build "
    "Ionic: serve": "ionic-app-scripts serve"
  },
  "Dependencies": {
    "@ Angular / common": "4.0.0",
    "@ Angular / compiler": "4.0.0",
    "@ Angular / compiler-cli": "4.0.0",
    "@ Angular / core": "4.0.0",
    "@ Angular / forms": "4.0.0",
    "@ Angular / http": "4.0.0"
    "@ Angular / platform-browser": "4.0.0",
    "@ Angular / platform-browser-dynamic": "4.0.0",
    "@ Angular / animations": "4.0.0",
    "@ Angular / platform-server": "4.0.0",
    "@ Ionic-native / core": "3.4.2",
    "@ Ionic-native / splash-screen": "3.4.2",
    "@ Ionic-native / status-bar": "3.4.2",
    "@ Ionic / storage": "2.0.1",
    "Ionic-angular": "3.0.1",
    "Ionons": "3.0.0",
    "Rxjs": "5.1.1",
    "Sw-toolbox": "3.4.0",
    "Zone.js": "^ 0.8.4"
  },
  "DevDependencies": {
    "@ Ionic / app-scripts": "1.3.0",
    "Rxjs": "^ 5.3.0",
    "Sw-toolbox": "^ 3.6.0",
    "Typescript": "~ 2.2.1"
  },
  "CordovaPlugins": [
    "Cordova-plugin-whitelist",
    "Cordova-plugin-console",
    "Cordova-plugin-device",
    "Cordova-plugin-statusbar",
    "Cordova-plugin-splashscreen",
    "Ionic-plugin-keyboard"
  ],
  "CordovaPlatforms": [],
  "Description": "adApp: An Ionic project"
}

Ionic2 - package.json
{
  "Name": "ionic-hello-world",
  "Author": "Ionic Framework",
  "Homepage": "http://ionicframework.com/",
  "Private": true,
  "Scripts": {
    "Clean": "ionic-app-scripts clean",
    "Build": "ionic-app-scripts build",
    "Ionic:" build ":" ionic-app-scripts build "
    "Ionic: serve": "ionic-app-scripts serve"
  },
  "Dependencies": {
    "@ Angular / common": "2.2.1",
    "@ Angular / compiler": "2.2.1",
    "@ Angular / compiler-cli": "2.2.1",
    "@ Angular / core": "2.2.1",
    "@ Angular / forms": "2.2.1",
    "@ Angular / http": "2.2.1",
    "@ Angular / platform-browser": "2.2.1",
    "@ Angular / platform-browser-dynamic": "2.2.1"
    "@ Angular / platform-server": "2.2.1",
    "@ Ionic / storage": "1.1.7",
    "Ionic-angular": "2.0.0-rc.4",
    "Ionic-native": "2.2.11",
    "Ionons": "3.0.0",
    "Rxjs": "5.0.0-beta.12",
    "Zone.js": "0.6.26"
  },
  "DevDependencies": {
    "@ Ionic / app-scripts": "^ 1.3.1",
    "Rxjs": "^ 5.3.0",
    "Sw-toolbox": "^ 3.6.0",
    "Typescript": "2.0.9"
  },
  "CordovaPlugins": [
    "Cordova-plugin-whitelist",
    "Cordova-plugin-console",
    "Cordova-plugin-device",
    "Cordova-plugin-statusbar",
    "Cordova-plugin-splashscreen",
    "Ionic-plugin-keyboard"
  ],
  "CordovaPlatforms": [],
  "Description": "adApp: An Ionic project"
}

other information:
No related issues
Ion information: (run the ion info from the terminal / cmd prompt and paste the following output):
Cordova CLI: 6.5.0
Ionic Framework Version: 3.0.1
Ionic CLI Version: 2.2.2
Ionic App Lib Version: 2.2.1
Ionic App Scripts Version: 1.3.0
Ios-deploy version: Not installed
Ios-sim version: Not installed
OS: Windows 10
Node Version: v6.10.1

how solve this problem

Hello! Thanks for opening an issue with us! As this seems like more of a support question i will urge that you ask this question on our forum or on our slack channel. Thanks for using Ionic!

Anyone manage to get this working? This page is the only lead on the issue.


Decoupling server from api: reduce cross-requires

Coverage decreased (-0.26%) to 85.8% when pulling def21ddc0d9bda062a75afa13f3dfcb8fae8e445 on clark800:separation into d6d39aa74b38b25b58c626e7002d09d722658f01 on ripple:develop.

LGTM


Update headlessui to latest version

Migration guide:
https://github.com/tailwindlabs/headlessui/releases/tag/%40headlessui%2Freact%40v2.0.0#user-content-upgrading-from-v1

Can I work on this?

@Kaivanshah1 sure 😊


Website Performance Testsuite

Create testsuite for simple and advanced websites and compare tuning cpu and replicas.
This would be a guidline how to tune production environment.

Documented here: https://websitecd.github.io/operator/developer-guide/website-performance-tests/


Use python iRODS client in place of icommands after python iRODS client supports parallel file transfer in 4.2.9

Currently we use icommands to not only do file i/o and transfer operations but also retrieve metadata and file info by parsing command output to retrieve needed info which depends on icommands output syntax. To make code more robust, python irods client should be used for non file transfer and i/o operations for now until python irods support multipart data object transfer which is on irods roadmap: https://irods.org/roadmap/

@hyi Is this ticket still relevant or should it be closed?

@Castronova Yes, I updated the title to capture what needs to be done more accurately.


Create a migration guide for Material 3

https://m3.material.io/develop/flutter is the hub on material.io for Material 3 on Flutter. We should create a migration guide.
Existing guides include

https://material.io/blog/migrating-material-3 for Android Views
https://developer.android.com/jetpack/compose/designsystems/material2-material3 for Jetpack Compose

I should also note https://blog.codemagic.io/migrating-a-flutter-app-to-material-3/ from @TahaTesser, @HansMuller is there anything missing from it?

I'd be happy to update the article to include anything missing.

@TahaTesser the article is great as it is, but if you have the opportunity, time and energy to update it, here are my suggestions:

The easy to use NavigationDrawer is mentioned, but it would be lovely to also see a teaser visual of it.
The DropdownMenu is not mentioned. It would be fabulous if it could be added. This new component is really wonderful and one that has been desperately asked for by many developers. Building something like it manually has been quite a challenging task to do in the past. Now it is finally available out of the box, and it is really good!
The new MenuBar and MenuAnchor could deserve a mention too, with some simple visuals of them.
This might make it a bit long, but I think it could be of general interest to point out the clever re-use of shared building blocks in DropdownMenu, MenuBar and MenuAnchor. Plus how the core of their flexible styling and theming capabilities is in large part built with the existing ButtonStyleButton and ButtonStyle via MenuItemButton, used by the already familiar Material buttons. Resulting in that menu items in all of them can have all of its  powerful props and theming capabilities. These all also make great building block for making totally custom versions as well or new variants not thought of here. Although the out-of-the box ones, are so flexible and well designed I doubt there are many reasons to do so.
Other new and updated components that could be briefly mentioned and shown are Badge (could be added e.g. to the shopping cart FAB icon and mentioned) and BottomAppBar, that has a significantly changed style in M3.

@rydmike Thank you so much for your valuable feedback!


Lucene (default) search does not return correct results for phrase searching (with double quotes)

Issue Description
The following search produces zero results:
query = "NOT (flair:expired OR flair:meta) \"yunnan sourcing\" OR yunnansourcing OR yunnansourcing.com"
reddit.subreddit("teasales").search(query, sort="new", time_filter="month")

Despite the web-based search page returning 1 result.
It would appear that the praw search or the Reddit API is not supporting the phrase search quotation marks correctly.
System Information
PRAW Version: 5.0.1
Python Version: 3.6
Operating System: Debian Testing

The API and the web-based search page are most likely returning different results due to them using different search engines.
About a month ago, Reddit launched a new search stack for their web-based search, but the Reddit API is still using the old search stack to ensure backwards compatibility. Currently there's no ETA as to when the new endpoint will be available.
For now you can try fielding your search query: NOT (flair:expired OR flair:meta) author:yunnansourcing OR selftext:\"yunnan sourcing\" OR selftext:\"yunnansourcing.com\" OR url:yunnansourcing

You got it @kwwxis!
NOT (flair:expired OR flair:meta) selftext:"yunnan sourcing" OR title:"yunnan sourcing" OR author:"yunnansourcing" OR selftext:"yunnansourcing" OR title:"yunnansourcing" OR site:"yunnansourcing.com" OR selftext:"yunnansourcing.com" OR title:"yunnansourcing.com"
...does the trick for a comprehensive search. Thanks!


ä will not be shown

Describe the bug

The letter "ä" will not be shown at the calliope (but the simulation is fine)

To Reproduce
Steps to reproduce the behavior:

Go to 'Calliope mini'
Click on block "show text"
Type a word with "ä"
See error

Expected behavior

Ä as small ä will be shown

Screenshots

If applicable, add screenshots to help explain your problem.

Device information

Type Desktop
OS: windows
Browser chrome

Additional context

Add any other context about the problem here.

Duplicate of #371 (?)

This is a firmware problem/feature, you should create a ticket directly at Calliope / mbed

Would it be possible to show a note/hint that ä (umlaute) can not be shown on the Calliope mini?
It should also be descriped in the help text:


Remove info log on import

Removes a log message that, as far as i can tell, isn't critical, and is causing problems when writing CI jobs and using the Airflow CLI: https://github.com/apache/airflow/issues/31655.
Checklist

[x] The PR conforms to DataHub's Contributing Guideline (particularly Commit Message Format)
[x] Links to related issues (if applicable)
[x] Tests for the changes have been added/updated (if applicable)
[x] Docs related to the changes have been added/updated (if applicable). If a new feature has been added a Usage Guide has been added for the same.
[x] For any breaking change/potential downtime/deprecation/big changes an entry has been made in Updating DataHub

@anshbansal The tests most recently failed with
Execution failed for task ':smoke-test:nodeSetup'.
> Could not resolve all files for configuration ':smoke-test:detachedConfiguration1'.
   > Could not resolve org.nodejs:node:16.8.0.
     Required by:
         project :smoke-test
      > Could not resolve org.nodejs:node:16.8.0.
         > Could not get resource 'https://nodejs.org/dist/v16.8.0/node-v16.8.0-linux-x64.tar.gz'.

which I believe cant be related to this change. Is there anything i should do?


Merge latest from main into beta branch

Merge latest from main into beta

Just started reviewing :)

Just started reviewing :)


Documentation typo on https://www.polymer-project.org/2.0/docs/devguide/shadow-dom

In "Shadow DOM and composition" category:
"Think of the aa a placeholder showing where child nodes will render."
After a sample section, first sentence starts with a dot:
".A named slot only accepts top-level children that have a matching slot attribute:"
In "Shadow DOM styling" category: a space between p and seudoelement:
"You can also style light DOM children that are assigned to slots using the ::slotted() p seudoelement. "
In sample code: css Border-radius with upper case on B
#shadow-root

Wrong repo, PR is open https://github.com/Polymer/docs/pull/1960


rtmp disconnect, no DASH data is available

As soon as I connect to the server log looks like this:
2018-7-18 01:15:01 5092 [INFO] [rtmp connect] id=IDQQI19K ip=::ffff:<IP_ADDRESS> app=live args={"app":"live","type":"nonprivate","flashVer":"FMLE/3.0 (compatible; FMSc/1.0)","swfUrl":"rtmp://localhost/live","tcUrl":"rtmp://localhost/live"}
2018-7-18 01:15:01 5092 [INFO] [rtmp publish] New stream. id=IDQQI19K streamPath=/live/test streamId=1
2018-7-18 01:15:02 5092 [INFO] [rtmp publish] Handle audio. id=IDQQI19K streamPath=/live/test sound_format=10 sound_type=2 sound_size=1 sound_rate=3 codec_name=AAC 44100 2ch
2018-7-18 01:15:02 5092 [INFO] [rtmp publish] Handle video. id=IDQQI19K streamPath=/live/test frame_type=1 codec_id=7 codec_name=H264 1920x1080
2018-7-18 01:15:02 5092 [INFO] [Transmuxing HLS] /live/test to ./media/live/test/index.m3u8
2018-7-18 01:15:02 5092 [INFO] [Transmuxing DASH] /live/test to ./media/live/test/index.mpd
2018-7-18 01:15:02 5092 [INFO] [rtmp connect] id=4FEZOYOW ip=::ffff:<IP_ADDRESS> app=live args={"app":"live","tcUrl":"rtmp://<IP_ADDRESS>:1935/live","fpad":false,"capabilities":15,"audioCodecs":3191,"videoCodecs":252,"videoFunction":1}
[NodeEvent on prePlay] id=4FEZOYOW StreamPath=/live/test args={}
2018-7-18 01:15:02 5092 [INFO] [rtmp play] Join stream. id=4FEZOYOW streamPath=/live/test  streamId=1
2018-7-18 01:15:06 5092 [INFO] [rtmp play] Close stream. id=4FEZOYOW streamPath=/live/test streamId=1
2018-7-18 01:15:06 5092 [INFO] [rtmp play] Close stream. id=4FEZOYOW streamPath=/live/test streamId=1
2018-7-18 01:15:06 5092 [INFO] [rtmp disconnect] id=4FEZOYOW
2018-7-18 01:15:06 5092 [INFO] [Transmuxing end] /live/test

So both of DASH and HLS become unavailable within 5 seconds after rtmp connect. Here's my file
const {NodeMediaServer} = require('node-media-server');

const config = {
    rtmp: {
        port: 1935,
        chunk_size: 60000,
        gop_cache: true,
        ping: 60,
        ping_timeout: 30
    },
    http: {
        port: 8000,
        allow_origin: '*',
        mediaroot: './media',
    },
    trans: {
        ffmpeg: 'C:\\Program Files\\ImageMagick-7.0.5-Q16\\ffmpeg.exe',
        tasks: [
            {
                app: 'live',
                ac: 'aac',
                // mp4: true,
                // mp4Flags: '[movflags=faststart]',
                hls: true,
                hlsFlags: '[hls_time=2:hls_list_size=3:hls_flags=delete_segments]',
                dash: true,
                dashFlags: '[f=dash:window_size=3:extra_window_size=5]'
            },
        ],
    },
};

const nms = new NodeMediaServer(config); //todo: cluster
nms.run();

My bad, rolled back lib files now it's ok. Still wondering how can I implement playback option for those viewers who couldn't be there by start?

@iamcsharper  I'm having the exact same issue, can you provide more information on how you resolved this?

@Feras94 Hi, I changed the server and player and now it works  perfectly. If you're struggling with Node-Media-Server which is quite dead, I highly recommend you to take a look at https://github.com/meisterplayer/meisterplayer and https://github.com/arut/nginx-rtmp-module they are both free
My ngnix config is:
# RTMP configuration
rtmp {
    server {
        listen 1935; # Listen on standard RTMP port
        chunk_size 4000;

        application live {
            live on;
            hls on;
            hls_path /root/streaming/hls/;
           #never set following two props to high values
            hls_fragment 3;
			hls_playlist_length 60;
			hls_continuous on;
			hls_nested on;

			# clean hls playback files when stream is closed
			# hls_cleanup on;

            # disable consuming the stream from nginx as rtmp
            deny play all;

        }
    }
}
http {
    sendfile on;
    tcp_nopush on;
    tcp_nodelay on;
    directio 512;
    default_type application/octet-stream;

    server {
        listen 8000;

        location /hls {
            expires max;
            # Disable cache
            add_header 'Cache-Control' 'no-cache';

            add_header 'Access-Control-Allow-Origin' '*' always;
            add_header 'Access-Control-Expose-Headers' 'Content-Length';

            if ($request_method = 'OPTIONS') {
                add_header 'Access-Control-Allow-Origin' '*';
                add_header 'Access-Control-Max-Age' 1728000;
                add_header 'Content-Type' 'text/plain charset=UTF-8';
                add_header 'Content-Length' 0;
                return 204;
            }

            types {
                application/dash+xml mpd;
                application/vnd.apple.mpegurl m3u8;
                video/mp2t ts;
            }

            root /root/streaming/;
        }
    }
}

Then I created a folder "/root/streaming"  and that's the place where stream folders are created. Meisterplayer supports that feature, I hope this helps

@iamcsharper thanks a lot 😊

Just coming across this problem myself with the exact same symptoms, I noticed my stream was encoded using "Sorenson-H263", when I changed that encoding to "H264" node-media-server transmuxed the stream as expected


undefined symbol: cuCtxGetDevice

When I build with the ref/ucx-cuda branch I have difficulty getting some of the dask-oriented ucx-py tests to run.
tests/test_dask_gpu_scatter.py (ucx-gpu) mrocklin@dgx15:~/ucx-py$ py.test tests/test_dask_* -s
======================================================================== test session starts ========================================================================
platform linux -- Python 3.7.3, pytest-4.4.1, py-1.8.0, pluggy-0.9.0
rootdir: /home/nfs/mrocklin/ucx-py
collected 2 items

tests/test_dask_gpu_scatter.py /home/nfs/mrocklin/miniconda/envs/ucx-gpu/bin/python: symbol lookup error: /home/nfs/mrocklin/miniconda/envs/ucx-gpu/lib/ucx/libuct_ib.so.0: undefined symbol: cuCtxGetDevice

I think that @rjzamora also ran into this.

Matt - this may be related to the errors you saw with being unable to find the path for cuda.h on the machine until you explicitly specified it in C_INCLUDE. You may need to explicitly add paths in LD_LIBRARY_PATH`.

I wrote this on another issue erroneously and meant to add it here.
This test fails
(ucx-dev) mrocklin@dgx15:~/ucx-py$ py.test tests/test_multiple_listeners.py  -s
============================================ test session starts =============================================
platform linux -- Python 3.7.3, pytest-4.4.1, py-1.8.0, pluggy-0.9.0
rootdir: /home/nfs/mrocklin/ucx-py
plugins: asyncio-0.10.0
collecting ... /home/nfs/mrocklin/miniconda/envs/ucx-dev/bin/python: symbol lookup error: /home/nfs/mrocklin/miniconda/envs/ucx-gpu/lib/ucx/libuct_ib.so.0: undefined symbol: cuCtxGetDevice

import ucp
import asyncio

ucp.init()

async def hello(ep, listener):
    await asyncio.sleep(0)
    ep.close()
    ucp.stop_listener(listener)

def test_multiple_listeners():
    lf1 = ucp.start_listener(hello, is_coroutine=True)
    lf2 = ucp.start_listener(hello, is_coroutine=True)
    assert lf1.port > 0
    assert lf2.port > 0
    assert lf1.port != lf2.port

    lf1.coroutine.close()
    lf2.coroutine.close()
    ucp.stop_listener(lf1)
    ucp.stop_listener(lf2)

More generally, can you verify that tests pass for you?

Matt - this may be related to the errors you saw with being unable to find the path for cuda.h on the machine until you explicitly specified it in C_INCLUDE. You may need to explicitly add paths in LD_LIBRARY_PATH.

I've been using these docs to setup.  If you're at all interested in trying this same process and providing feedback on how it could be improved that would be very welcome.  A few of us use this process, so if we can improve that doc I think that it would smooth things over considerably.
Also cc @rjzamora who probably has a better chance of finding the right paths to add to LD_LIBRARY_PATH

Hmm. I stand corrected. I'm seeing the issue with ref/ucx-cuda (even after rebasing with master).
akvenkatesh@prm-dgx-33:~/ucx-py$ py.test tests/test_multiple_listeners.py  -s
======================================================== test session starts ========================================================
platform linux -- Python 3.7.2, pytest-4.3.0, py-1.8.0, pluggy-0.9.0
rootdir: /home/akvenkatesh/ucx-py, inifile:
plugins: repeat-0.8.0, asyncio-0.10.0
collecting ... /home/akvenkatesh/py/install/bin/python3: symbol lookup error: /home/akvenkatesh/ucx-github/build/lib/ucx/libuct_ib.so.0: undefined symbol: cuCtxGetDevice

I'll update back when I've resolved this issue. I'm not sure why we see this problem yet.

BTW, pointing the LD_LIBRARY_PATH at /usr/local/cuda-9.2/lib64 did not fix the problem for me.

@mrocklin @rjzamora
Had some issues with rebase on ref/ucx-cuda. Can you please change to this branch https://github.com/Akshay-Venkatesh/ucx/tree/1ad5b17/ucx-cuda and retry? I'm able to run the tests without the symbol issue and have different patches on top of current ucx master.
Sorry about the hassle.

@mrocklin @Akshay-Venkatesh  1ad5b17/ucx-cuda fixes this error for me. Thanks!

Good to know. Will close for now.

I'm still having trouble building this
./configure: line 17888: PKG_PROG_PKG_CONFIG: command not found
checking if ln -s supports --relative... yes
checking for dot... yes
checking for doxygen... /home/nfs/mrocklin/miniconda/envs/ucx-gpu/bin/doxygen
checking for perl... /home/nfs/mrocklin/miniconda/envs/ucx-gpu/bin/perl
checking for pdflatex... no
configure: WARNING: pdflatex not found - will not generate doxygen PDF documentation
checking for makeindex... no
checking for bibtex... no
checking for egrep... /bin/egrep
./configure: line 26873: syntax error near unexpected token `newline'
./configure: line 26873: `              PKG_CHECK_MODULES('
Makefile:587: recipe for target 'config.status' failed
make: *** [config.status] Error 2
(ucx-gpu) mrocklin@dgx15:~/ucx$

I also found that I needed to install a newer doxygen to get past other errors.

Sorry, that was probably a bad file.  Starting from a clean environment
(ucx-gpu) mrocklin@dgx15:~/ucx$ ./autogen.sh
autoreconf: Entering directory `.'
autoreconf: configure.ac: not using Gettext
autoreconf: running: aclocal -I config/m4
autoreconf: configure.ac: tracing
autoreconf: running: libtoolize --copy
libtoolize: putting auxiliary files in '.'.
libtoolize: copying file './ltmain.sh'
libtoolize: putting macros in AC_CONFIG_MACRO_DIRS, 'config/m4'.
libtoolize: copying file 'config/m4/libtool.m4'
libtoolize: copying file 'config/m4/ltoptions.m4'
libtoolize: copying file 'config/m4/ltsugar.m4'
libtoolize: copying file 'config/m4/ltversion.m4'
libtoolize: copying file 'config/m4/lt~obsolete.m4'
autoreconf: running: /home/nfs/mrocklin/miniconda/envs/ucx-gpu/bin/autoconf
configure.ac:33: error: possibly undefined macro: AS_IF
      If this token and others are legitimate, please use m4_pattern_allow.
      See the Autoconf documentation.
configure.ac:34: error: possibly undefined macro: AC_SUBST
configure.ac:137: error: possibly undefined macro: AC_MSG_ERROR
configure.ac:237: error: possibly undefined macro: AC_DEFINE
configure:26956: error: possibly undefined macro: AC_MSG_WARN
autoreconf: /home/nfs/mrocklin/miniconda/envs/ucx-gpu/bin/autoconf failed with exit status: 1

@rjzamora pointed me to conda install pkgconfig which seems to solve the problem.  I've also updated the setup doc I've been working from

Just as an update here, what you probably want is pkg-config (the Unix build tool). The pkgconfig Python library (note absence of -) is for Python programs that want to use the Unix build tool from Python. Have submitted PR ( https://github.com/dask/dask-blog/pull/31 ) to update the blog as well.


[Impact Metric Submission]: Total Transactions for each project

Total Transactions for each project
Tags
Add one or more keywords to help us categorize your metric. Don't overthink this!

Transactions
Project Analysis

Brief description
Describe how your impact metric is calculated in 2-3 sentences: This SQL query calculates the total transactions grouped by project ID and date from a specified table in the opensource-observer.oso dataset. It aggregates the amounts for each project and date combination without filtering for specific event types.
SQL code block
Insert the SQL that we should run to replicate your impact metric:
SELECT
    project_id,
    DATE(bucket_day) AS date,
    SUM(amount) AS total_transactions
FROM
    `opensource-observer.oso.events_daily_from_project`
GROUP BY
    project_id,
    date;

Optional: link
Add a link to script/notebook or longer form write-up about the metric:

Focus Area: Network Growth
Query Executes in BigQuery? Yes
Query Meets Challenge Spec? No
Complexity Score:
Utility Score: -
Total Points: -
Comments: Does not filter on event_type


Cleanup the travis yaml file.

Part of the fixes for #19. Some of the comments were out of date,
also the order of the environment variables did not make sense.

PTAL.


fix "TextEditor.getFirstVisibleScreenRow is deprecated."

with  Version 0.192.0 (0.192.0) , the editor shows deprecation cops when scroll.
This is now a view method. Call TextEditorElement::getFirstVisibleScreenRow instead.
TextEditor.getFirstVisibleScreenRow (/Applications/Atom.app/Contents/Resources/app/src/text-editor.js:2613:9)
ScrollHalfScreenDown.Scroll (/Users/hitsmaxft/.atom/packages/vim-mode/lib/scroll.coffee:7:21)

i've fixed it with using text-editor-element in class scroll , instead of text-editor.

Thanks @hitsmaxft. Could you either rebase this onto the latest master or merge master into this? Some changes have been made to some of the scroll commands recently, and they appear to conflict with your fix.

@maxbrunsfeld  I have done rebase and migrate class Motions as either.

Thanks for sticking with this @hitsmaxft!


Emoji reactions from one user can race with parallel identical requests

Adding an emoji reaction happens in multiple queries, not in a transaction; first it checks if the reaction already exists, then it adds it:
https://github.com/zulip/zulip/blob/21051d2130868f286a9e6753ad9f197e9bb56854/zerver/lib/actions.py#L2529-L2535
https://github.com/zulip/zulip/blob/21051d2130868f286a9e6753ad9f197e9bb56854/zerver/lib/actions.py#L2579
This can lead to race conditions where a second parallel request from the same user can 500 due to the reaction having been created by another request between the check and the creation.
We should likely wrap in a transaction for consistency, but we'll need to catch the IntegrityError and return it as a 400.

We should also look at making the web UI behavior not send a second API request to add a reaction until the first has returned.

@zulipbot claim

We should also look at making the web UI behavior not send a second API request to add a reaction until the first has returned.

Opened #24557 to solve this.

90b6fa7881d5e3b3d6f73045328543614dff6e6f fixed the transaction part of this, and https://github.com/zulip/zulip/pull/24557 the UI part of this, so closing as fixed.


Add table style to project description, clean up titles

Closes #2994

Once live, a good place to test: https://pypi.org/project/clc_msa_utils/


Move API upload endpoint to package service

Remove the upload service and package.
Set up and close the upload bucket in main.
Update configuration:

Make upload max size configurable.
Use go.artefactual.dev/tools/bucket config struct.

Refs #977.

This endpoint only proxies the upload to a bucket, the workflow is then triggered like a normal MinIO deposit, if properly configured.


WIP - Policy validators and error handlers 🐱👤

Added relevant validators to policy
Add base forms for relevant error pages

Co-Authored-By: ilanKushnir<EMAIL_ADDRESS>Co-Authored-By: Shay Avraham<EMAIL_ADDRESS>

LGTM.

@ilanKushnir  @tomerguttman
Do note that you shouldn't merge a pr if it breaks the CI,
broken CI = broken product.


About aggregate queries question

mongoapi:runCmd([
{"aggregate", ?ORDER_TABLE},
{"cursor", #{ } },
{"pipeline",
{array,[
[
{"$match", #{status=>0}}
],
[
{"$group",[
{"_id","null"},
{"total", [{"$sum", "$amount"}]}
]}
]
]}
}
], ?MONGO_MASTER),
the query result is:
[{<<"cursor">>,
[{<<"firstBatch">>,
{array,[[{<<"_id">>,<<"null">>},{<<"total">>,1900}]]}},
{<<"id">>,0},
{<<"ns">>,<<"paycenter_db.pay_order">>}]},
{<<"ok">>,1.0}]
Can you give me a correct demonstration?
Cursor is a mandatory parameter, but I am not sure how to use it. Do I need to close cursor?

Cursor empty like in your example has no effect.

I'm still not sure how to use the cursor properly,can you give me example?


show loading

like other tool window, show loading at top of the panel

Maybe change cursor???
Cursor.getPredefinedCursor(Cursor.WAIT_CURSOR)


Unable to build using --prod - Module build failed: Error: ENOENT: no such file or directory - ng-file-drop.directive.js

I'm unable to build my project using Ionic and Cordova when using the --prod flag.
Any suggestions?
Using ngx-uploader@latest (4.0.0)
$ ionic cordova run ios --device --prod
[INFO] Running app-scripts build: --prod --platform ios --target cordova
[07:38:33]  build prod started ...
[07:38:33]  clean started ...
[07:38:33]  clean finished in 1 ms
[07:38:33]  copy started ...
[07:38:33]  deeplinks started ...
[07:38:33]  deeplinks finished in 155 ms
[07:38:33]  ngc started ...
[07:38:42]  ngc finished in 8.94 s
[07:38:42]  preprocess started ...
[07:38:42]  preprocess finished in 105 ms
[07:38:42]  webpack started ...
[07:38:44]  copy finished in 10.77 s
Error: ./node_modules/ngx-uploader/src/ngx-uploader/directives/ng-file-drop.directive.ts
Module build failed: Error: ENOENT: no such file or directory, open '/Users/bbense/devel/p4-app/node_modules/ngx-uploader/src/ngx-uploader/directives/ng-file-drop.directive.js'
 @ ./src/components/upload-publication/upload-publication.ngfactory.ts 30:0-87
 @ ./src/app/app.module.ngfactory.ts
 @ ./src/app/main.ts,./node_modules/ngx-uploader/src/ngx-uploader/directives/ng-file-select.directive.ts
Module build failed: Error: ENOENT: no such file or directory, open '/Users/bbense/devel/p4-app/node_modules/ngx-uploader/src/ngx-uploader/directives/ng-file-select.directive.js'
 @ ./src/components/upload-publication/upload-publication.ngfactory.ts 31:0-89
 @ ./src/app/app.module.ngfactory.ts
 @ ./src/app/main.ts,./node_modules/ngx-uploader/src/ngx-uploader/module/ngx-uploader.module.ts
Module build failed: Error: ENOENT: no such file or directory, open '/Users/bbense/devel/p4-app/node_modules/ngx-uploader/src/ngx-uploader/module/ngx-uploader.module.js'
 @ ./src/app/app.module.ngfactory.ts 93:0-80
 @ ./src/app/main.ts
    at new BuildError (/Users/bbense/devel/p4-app/node_modules/@ionic/app-scripts/dist/util/errors.js:16:28)
    at callback (/Users/bbense/devel/p4-app/node_modules/@ionic/app-scripts/dist/webpack.js:123:28)
    at emitRecords.err (/Users/bbense/devel/p4-app/node_modules/webpack/lib/Compiler.js:269:13)
    at Compiler.emitRecords (/Users/bbense/devel/p4-app/node_modules/webpack/lib/Compiler.js:375:38)
    at emitAssets.err (/Users/bbense/devel/p4-app/node_modules/webpack/lib/Compiler.js:262:10)
    at applyPluginsAsyncSeries1.err (/Users/bbense/devel/p4-app/node_modules/webpack/lib/Compiler.js:368:12)
    at next (/Users/bbense/devel/p4-app/node_modules/tapable/lib/Tapable.js:218:11)
    at Compiler.compiler.plugin (/Users/bbense/devel/p4-app/node_modules/webpack/lib/performance/SizeLimitsPlugin.js:99:4)
    at Compiler.applyPluginsAsyncSeries1 (/Users/bbense/devel/p4-app/node_modules/tapable/lib/Tapable.js:222:13)
    at Compiler.afterEmit (/Users/bbense/devel/p4-app/node_modules/webpack/lib/Compiler.js:365:9)

Ionic Info
cli packages: (/Users/bbense/.nvm/versions/node/v7.10.0/lib/node_modules)

    @ionic/cli-utils  : 1.12.0
    ionic (Ionic CLI) : 3.12.0

global packages:

    cordova (Cordova CLI) : 7.0.1

local packages:

    @ionic/app-scripts : 3.0.0
    Cordova Platforms  : android 6.2.3 browser 4.1.0 ios 4.5.1 iosX 4.4.0
    Ionic Framework    : ionic-angular 3.7.1

System:

    ios-deploy : 1.9.2
    ios-sim    : 5.0.13
    Node       : v7.10.0
    npm        : 4.2.0
    OS         : macOS High Sierra
    Xcode      : Xcode 9.0 Build version 9A235

Misc:

    backend : legacy

For simplicity of reproduction, ionic build --prod throws the same error.

I created a minimal project that shows the problem.

unzip attached file
run yarn to install dependencies
run `ionic build --prod``

bugtest.zip
Hope that helps.

thanks Bastien. will check this when get to the computer.
On 10 October 2017 at 09:50:49, Bastian Bense<EMAIL_ADDRESS>wrote:

I created a minimal project that shows the problem.

unzip attached file
run yarn to install dependencies
run `ionic build --prod``

bugtest.zip
https://github.com/jkuri/ngx-uploader/files/1370989/bugtest.zip
Hope that helps.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/jkuri/ngx-uploader/issues/352#issuecomment-335390432,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABtntp4b6Drt3h41VO8dilrFzk0zVmjZks5sqyHZgaJpZM4PzZvA
.

@jkuri did you checked it?

no. but I believe it should work with the new version.


Specify / Refer to recommended HSTS max-age

Reminder:
The nmos-testing suite (BCP-003-01) requires the minimum max-age to be 15465600 seconds.
After having a chat with @garethsb-sony  about this, it seems that there should exist a guideline for the HSTS header somewhere.
A reference to the mentioned guideline (once found :)) should be added to the docs (best-practice-secure-comms.md).

See #40  :)


Removing Sky from Dart plugin

Tracking issue. We probably want to remove the "create new Sky project" from the Dart plugin. It's a bit odd that the generic Dart plugin has such specific support for a particular toolkit.
This assumes, of course, that we can build a separate Sky plugin for Atom.

If creating a Stangehand based project could also an option, why not leave it.

Done! A flutter plugin is published, and we're iteratively moving Flutter UI contributions into it.


RCD UI Doesn't Show Up

Description
When attempting to change the mode of the RCD the menu options just simply doesn't appear. The only thing that'll be there is the X to close the menu.
Reproduction
Grab RCD.
Hit Z.
Screenshots

Possibly related to #30177 ? Although I've tested both the rcd[empty] and regular RCD from the spawned menu and they both work fine.

Caused by https://github.com/space-wizards/space-station-14/pull/28399

Got it working


fix Building Failed on x86 Mac

What problem does this PR solve?
Problem Summary:

build-thirdparty.sh doesn't cover x86 Mac when renaming symbols for bitshuffle AVX2.
http_channel.cpp  uses %ld  format specifier for int64_t type, but this type is  defined with long long int on macOS. stack overflow

Check List (For Author)

Test

[ ] Regression test
[ ] Unit Test
[ ] Manual test (add detailed scripts or steps below)
[ ] No need to test or manual test. Explain why:

[ ] This is a refactor/code format and no logic has been changed.
[ ] Previous test can cover this change.
[ ] No code files have been changed.
[ ] Other reason

Behavior changed:

[x] No.
[ ] Yes.

Does this need documentation?

[x] No.
[ ] Yes.

Check List (For Reviewer who merge this PR)

[ ] Confirm the release note
[ ] Confirm test cases
[ ] Confirm document
[ ] Add branch pick label

Thank you for your contribution to Apache Doris.
Don't know what should be done next? See How to process your PR.
Please clearly describe your PR:

What problem was fixed (it's best to include specific error reporting information). How it was fixed.
Which behaviors were modified. What was the previous behavior, what is it now, why was it modified, and what possible impacts might there be.
What features were added. Why was this function added?
Which code was refactored and why was this part of the code refactored?
Which functions were optimized and what is the difference before and after the optimization?


rel="noopener noreferrer"

Using the option link-req-noopener will check for the presence of rel="noopener". That's fine, but because Firefox does not support rel="noopener", the current best practice is to code like this: rel="noopener noreferrer" - but when doing so, linting will fail.
I request that a change be made so that rel="noopener noreferrer" passes linting, just like rel="noopener" currently does.
Thank you.
More info on why using rel="noopener noreferrer" is the current best practice: https://www.jitbit.com/alexblog/256-targetblank---the-most-underestimated-vulnerability-ever/

I've added a Pull request for this here: #268
After merge, can we get a minot release in npm as well?
Thanks!

Thank you!


value('a').is_int64()

I would expect value('a').kind() to be a string but it is int64. Bug or feature?

Probably value with a char-type argument should be prohibited (deleted) to avoid any surprising behavior to users?

Probably value with a char-type argument should be prohibited (deleted) to avoid any surprising behavior to users?

I'm not opposed to it although I am not an expert. Maybe try submitting a pull request and see what CI thinks about it?

Unfortunately it looks like this is a corner case of C++ where there is no perfect solution.

Hmm... come to think of it, deleting that constructor would break conversion from int8, which is allowed to be an alias for char.

Yep, it is true on AArch64 (https://reviews.llvm.org/D85225)...

Working as designed, this is the best we can do with the current specification of C++.


Improve gitver experience

Wrap gitver in a script that only replaces ver.d if its contents changed. This avoids triggering a full rebuild every time, even if nothing changed.

@asahilina
does this work in windows, too?

I changed it to keep the old behavior on Windows, maybe this is OK for now?


travis: cache Cargo dependencies and build output

I hope this will speed up the build.

Yep, this brings the build and test time down from about 8 minutes to just 1 minute!


Does ArangoDB have a  official  ArangoDB .NET Client ?

I didn't find ArangoDB. NET Client from the ArangoDB Home page, where the official ArangoDB . NET Client for development?

Hi @czs1993,
We currently do not have an "official" ArangoDB driver for .NET ("official" here meaning a driver being maintained by ArangoDB the company).
There already is an open ticket https://github.com/arangodb/arangodb/issues/2992 that is requesting the support of it. Therefore I am closing this ticket as duplicate, please continue the discussion in the referenced ticket.
Thanks


Fix PopupMenu unresponsive on right side of menu

CC @EricEzaM

Hi @nathanfranke, @akien-mga.
Unfortunately I cannot reproduce the issue on my PR... not sure what is going on - maybe it is a platform specific issue? Anyway, this PR breaks what this piece of code tries to solve - not selecting items when trying to select the scrollbar. See below video of my test.

So, the reason this code is required is because of the below situation. The scroll container just renders the scrollbar above the content.

I chose option "A" since it seemed more elegant :smile:

All good on my end after a test, thanks!

Also fixes #41762.

Thanks!


Resizable dialogs for common dialogs

This makes some of the top level dialogs resizable.  The resizing support uses markup in the dialog resources and is fairly straightforward to use.  Note this doesn't work on Win9x but support for that is no longer present in winfile.  Some dialogs can't be resizable that should be; I can't seem to make fonts work for example, which is a custom template on the common dialog library.  Things like progress or confirm dialogs don't seem to add value by being resizable.
The per-dialog resize states are conceptually statics.  They're just blobs that are allocated and destroyed along with the dialog box, and it's safe to overload them so long as two dialogs can't exist concurrently trying to use the same context.

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.:x: malxau sign nowYou have signed the CLA already but the status is still pending? Let us recheck it.

At first I wasn't sure what you were trying to achieve. So, I downloaded your commit and built the solution.
After having experienced with your changes, I can definitely see the value of this. The dialog boxes can be widened to increase the view is the text boxes that exists in many of these windows. This allows one to be able to see more of the text without having the text scroll, which is especially helpful for many of the text boxes which allows you to type in a directory path.

Every day I learn new things -- like this interesting idea!  I need to review in more detail to comment.

Hi @malxau, given that master now has 2 soon to be 5 languages, have you given some thought about how to simplify the feature and not require every dialog field to be annotated?

I re-merged this with the current localized dialog resources.  Please let me know if this is something to proceed with or not.  If it's not something to proceed with I can go do something else.

Definitely still interesting.  Given there are more and more things to get right, I built a simple resource verification tool that makes sure all of the languages have the same resource identifiers with the same attributes as necessary.

Have you, @malxau, tested it working on all currently available languages? If not, I can do that. I'd just need to pull from your latest commit and rebuilt. I currently still have your older code, but I don't mind testing your latest code if it helps you.

@NazmusLabs That'd be greatly appreciated.  Out of interest, how does this work? How do you test a specific language?
When testing, please look at pl-PL's SELECTDLG and RUNDLG.  The first appears to be larger than the en-US one and the second has a mighty long string that I'm expecting to collide with the buttons if they move.
Also, I haven't looked at he-IL yet.  This one is a stress test for this code because everything is right to left.  The code is designed to handle it and I have simple tests that demonstrate it working, but it is a little different.

As a concrete illustration of the long text above, look at how "Run as Admin" is positioned in English vs. German.  If reposition information is with the control, this is straightforward; if it is stored independent of language layout, it will be buggy.

It looks like VerifyResources has a bug in processing these resources.  In DecodeDlgItem it treats cbCreate as being the total number of bytes to advance (including the cbCreate field itself.)  I know this is exactly what MSDN says it means, but after looking at it, I'm fairly sure MSDN is wrong: the cbCreate field needs to be accounted for unconditionally, and the number of bytes in it are additional to that.  This hasn't mattered before resizeable dialogs because no dialog item had additional creation data, so only the first (and correct) branch is taken.  With these changes, the offsets of subsequent controls are not interpreted correctly, and the code will complain about all kinds of spurious things, since it will interpret localized strings as control IDs, etc.
Somewhat unrelated, but this tool is flagging issues in master now which appear legitimate:
RT_CURSOR
RT_BITMAP
RT_ICON
RT_MENU
RT_DIALOG
Warning: dialog Confirmation item #6, id 215 for language pl-PL does have the required style 0x50010003
Error: dialog Preferences for language de-DE does not have the required number of items 12
Warning: dialog Moving... item #2, id 108 for language ja-JP does have the required style 0x5000008b
RT_STRING
Error: string table block usage mismatch for language tr-TR block 15
Error: missing lanaguage zh-CN for resource type RT_STRING, name 126
Error: string table block usage mismatch for language zh-CN block 126
RT_ACCELERATOR
RT_GROUP_CURSOR
RT_GROUP_ICON
RT_VERSION
RT_MANIFEST

Just donwloaded the branch and I also find this neat and a valuable extension to Winfile.
What are the problems?
Can we step by step work them off?

I updated this to current.
In terms of the problems, everything is covered in this issue.  I think it's just a large change to review.  There's obviously a desire to make something that applies to all languages, although I'm skeptical that doing so works reliably if dialog layouts can differ.
If you give this a serious review and approve it, then I think it can be merged.

Took it over into my daily work branch and give a test for a week

Had this running on daily basis for almost 2 weeks and it worked flawlessly!
So you have my approval!
Please merge it.

reviewing now.

As we discussed, this PR should be broken into parts, but my suggestion is simpler than the one we talked about.  Separate out the change in VerifyResources.cpp as it is completely independent.  Also separate out the changes to res*.rc since they are unrelated to the resizable dialogs.  Then we'll tackle the main change.
On the main change, the use of global state for ppStorage of ResizeDialogProc is old school :).  Are there no other options for saving this?  GWLP_USERDATA would be ideal, but it is not generally available and it would be non-trivial to use it.

How to precede with this one? Looks like not much is missing? So could it be merged?


Issue25: Added Moving Car Animation

Added a MovingCarAnimation folder in the main folder with relevant files.
Ref: Issue - https://github.com/irshadmd/Awesome-CSS-Projects/issues/25

cool, Please create a demo folder inside "MovingCarAnimation" and add a working demo of your project named demo.gif

Added, hope that works.


refactor to not use package:compiler_unsupported

It's unsupported :) And makes rolling dartpad much more difficult. I think we have a good path forward with shelling out to dart2js, w/ the performance of the new app-jit snapshots.

fixed in https://github.com/dart-lang/dart-services/pull/295


Support truly pluggable protocol/transport for distributed mode

At this moment Distributed TensorFlow supports only one type of transport/protocol which is gRPC, however, it does seem to be configurable (cluster, server, session).
So, there are at least three things that need to be covered:

Document the intercommunication protocol (session to a server), i.e. what session sends to a server, what server should respond, etc. The good example here would some kind of swagger spec.
Make protocol configurable from Python.
Give all a Python interface to implement new types of protocols.

The whole idea is to let developers an ability to implement the protocol that would let distributed TensorFlow spin up and talk a number compute units (serverless, containers, VMs, etc.) instead of having a bunch of processes (cluster servers) running during computations within the session.
Please note, I wasn't able to find any corresponding issues related to given topic.

Nagging Awaiting Response: It has been 14 days with no activityand the awaiting response label was assigned. Is this still an issue?

@poxvoculi by saying protocol i meant the actual interaction protocol (who sends what?). At this moment all interactions are being described by protobuf, so there's no way to build another one interaction protocol for all components, for instance, if i want to get rid of protobuf and use, let's say, HTTP 1.1 for that.

since you seem to be rejecting the use of the master.proto and worker.proto interfaces, which are the only way TF processes intercommunicate.

This is what i would like to change, i don't like the idea of mandatory protocols, this is very critical to those use cases when people want to build their own workers to achieve massive scale-in/out without exposing too much information about the infrastructure.

@denismakogon I agree with @poxvoculi in that I'm still not quite sure what you're proposing. It seems like you can replace gRPC with the RPC package of your choice. I think that if you wanted to replace protobufs with some other portable marshalling solution, you'd have a huge amount of work on your hands for not necessarily a great deal of benefit (I agree that protos can be slow, but faster solutions that retain portability are hard work).
Thinking constructively, I wonder if a GitHub issue isn't quite the right way to have this discussion. Could we get you to write a somewhat longer-form document that describes (a) the problem you want to address, (b) the limitations of the current solutions, and (c) the way forward that you'd like to explore? It seems to me that a document would allow you to make a reasoned argument for what you want in a way that lets readers better understand where you're coming from.

Just my 2 cents, you can still use HTTP1.1 by serializing protobuf to JSON. I cannot see any benefits doing this though.

Nagging Awaiting Response: It has been 14 days with no activityand the awaiting response label was assigned. Is this still an issue?

Nagging Awaiting Response: It has been 14 days with no activityand the awaiting response label was assigned. Is this still an issue?

I'm going to close this to stop the auto-nagging.  Feel free to reopen if you want to respond to @cy89 's comment.


BUG-Error setting up documentation (taipy-auth not found)

Bug description
Followed the instructions in taipy-doc/INSTALLATION.md to setup the documentation but getting the following error when running python tools/fetch_source_files.py develop:
Traceback:
Traceback (most recent call last):
  File "/home/arcanaxion/programming/taipy-projects/taipy-dev/taipy-doc/tools/fetch_source_files.py", line 99, in <module>
    raise SystemError(f"Problem with {repo}:\nOutput: {cmd.stdout}\nError: {cmd.stderr}")
SystemError: Problem with taipy-auth:
Output:
Error: remote: Repository not found.
fatal: repository 'https://github.com/Avaiga/taipy-auth.git/' not found

Expected change
Description of what would be the expected outcome.
Screenshots
When available and relevant, screenshots better help show the problem.

The fix was done in the 'develop' branch for Taipy 3.1.
If you really need that for 3.0, please let us know!


Feature Request - Allow copying or sharing info

It would be nice to allow copying/sharing parts of the info in app (Mails, Skill Queue..etc) rather than taking screenshots.
Just a minor thing and not really important compared to other features! :)

I implemented button to copy mails and notifications clipboard. If you get the chance, please let me know how that works for you.

I implemented button to copy mails and notifications clipboard. If you get the chance, please let me know how that works for you.

Works like a charm, thank you!


Add badges, update workflows & names #24

Added the first few badges and a rudimentary version of test coverage, which requires a pull request to update. Will update iteratively to receive a "cleaner" version for badges containing numbers that need to be extracted from files in assets.
Let's discuss some details on which badges might be interesting and which ones are superfluous in the next meeting.
Resolves #24

Update: Doesn't work as intended due to missing coverage.json file upload.
Other non-third party (but Marketplace extension) workflow: jacoco badge generator

TODO: Implement something similar to https://nedbatchelder.com/blog/202209/making_a_coverage_badge.html - also avoids uploading artifacts with a long retention period

Can be merged


fix nil pointer reference bug & NaN handling on invalid json input

It is possible to cause a segfault by providing invalid json.
./sbom-scorecard score ../../examples/invalid.json                                                       ⏎ ✹ ✭
Guessed: cdx
panic: runtime error: invalid memory address or nil pointer dereference
[signal SIGSEGV: segmentation violation code=0x1 addr=0x10 pc=0x11acab8]

goroutine 1 [running]:
github.com/ebay/sbom-scorecard/pkg/cdx.GetCycloneDXReport({0x7ff7bfeff647, 0x1b})
	.../sbom-scorecard/pkg/cdx/cdx_report.go:119 +0x2f8
github.com/ebay/sbom-scorecard/cmd/sbom-scorecard/cmd.glob..func1(0x13fef00?, {0xc000063930?, 0x1?, 0x1?})
	.../sbom-scorecard/cmd/sbom-scorecard/cmd/scorecard.go:70 +0x1c8
github.com/spf13/cobra.(*Command).execute(0x13fef00, {0xc0000638f0, 0x1, 0x1})
	.../github.com/spf13/cobra@v1.6.1/command.go:920 +0x847
github.com/spf13/cobra.(*Command).ExecuteC(0x13fec20)
	.../github.com/spf13/cobra@v1.6.1/command.go:1044 +0x3bd
github.com/spf13/cobra.(*Command).Execute(...)
	.../github.com/spf13/cobra@v1.6.1/command.go:968
github.com/ebay/sbom-scorecard/cmd/sbom-scorecard/cmd.Execute()
	.../sbom-scorecard/cmd/sbom-scorecard/cmd/root.go:20 +0x25
main.main()
	.../sbom-scorecard/cmd/sbom-scorecard/main.go:8 +0x17

pkg/cdx/cdx_report.go#L127
Fix nil pointer reference in bom.Metadata.Tools which causes nil pointer segfault when an invalid json file is provided.
pkg/cdx/cdx_report.go#L65
Fix NaN handling leading to invalid score results when an invalid json file is provided.
The root of this issue is likely that upstream cyclonedx-go/blob/master/decode.go#L43  uses json.Decode (which accepts a stream) instead of json.Marshal.
nit: ioutil package deprecated: As of Go 1.16, this function simply calls os.ReadFile.

So I believe the tests are failing because the underlying logic for cdx_report.go#L49 IsSpecCompliant() is erroneous, as opposed to the test being incorrect.
This is because of an issue in cdx_report.go#L105 GetCycloneDXReport(), decoder.Decode(bom) will never throw err.

Okay. So this no longer segfaults, but it does give weird results. There's an assumption that cyclonedx would return an error if given invalid json. It didn't. I've just made https://github.com/CycloneDX/cyclonedx-go/pull/84 which addresses that upstream. That should cause it to fail with a 0 for a score when they release & we update our dependency.
So sorry for the delay in addressing this.


Regression in recordedit creation of records with server-generated keys

This just stopped working recently on my dev server... worked last week.
I have a table with a server-generated domain type ermrest_rid as its primary key. Recordedit in create mode creates the record but fails to redirect to the record page for the new record.
It seems like it is embedding undefined in the key filter for the new record instead of the actual server-generated value that is returned from the ermrest request.  After the error page, I can browse around and find the record manually in recordset and then navigate to its record page.

Actually, I think this is probably a bug in my experimental ermrest branch...


[AudioServer] Changing global fx volume doesn't affect the reverb of SamplePlayers

When using a SamplePlayer that has ReverbSend, using the function:
AudioServer.set_fx_global_volume_scale(0)
will reduce the volume of the sample that is played, but the sound will still be sent to the reverb and only the 'echo' will be heard.
Sample project

I can confirm that on stable 1.1.
But actually I like the way it works. From my perspective it gives more flexibility to what’s possible with sound. What other way would be to hear reverb set to “pre fade” in the mix? It can be desired sound fx for some people in some situations. The way it works now is not really intuitive, but I think it would be good to be able to set pre or post fade in fx section of the SamplePlayer in the future.
That was already on the list of my requests of what could be added/corrected in Godot’s audio area, that I plan to post and discuss at some point, but since you mentioned it...

I cqn reproduce it too in the current master branch (61c4c57) thanks to the nice sample project provided. It's quite weird indeed that everything is mute but you can still hear the echo (and not only the echo of a sample played before the mute as I thought initially when reading the report).

the audio subsystem will eventually be rewritten in 2.1, so for now, sitting on this issue

This is still valid in master. @reduz do you still plan to do this for 2.1?

@vnen We discussed it on irc today. 2.2 will be focused on the new audio system and higher level networking.

Was this fixed in 3.0 with the audio busses?

Assumed fixed in 3.0, feel free to reopen/comment/open a new issue if it's not the case.


Unstable view when scaling

First of all thank you very much! Your package helped me a lot, and saved several hours!
Just noticed that scaling the browser view (cmd + "+" / cmd + "-") breaking the layout.
I'm looking for a workaround and share if find one
this problem appears only on chrome and safari (as i tested) but ff looks fine

Thank you very much again! I realized that there would not be any fix of that, so I'll stop use this package! Thanks for your work

@letsG hello. thank you for your praise.
I recently released new version of package v0.5.1, which is built with the latest version of react-spring
Would you like to try a new version?


Dev Portal: selection of Category Filter is not intuitive

Is there an existing issue for this?

[X] I have searched the existing issues

Kong version ($ kong version)
3.4+
Current Behavior
The selection of a filter in DevPortal "category hypertext" doesn't work normally. You must select to the right of the link, not the link itself.
https://github.com/Kong/kong/assets/4004310/c520cf0f-156f-49c0-bc2a-8f8032832c35
Expected Behavior
When I click the hyperlink of an item in the "Category filter" list, the filter is applied

Steps To Reproduce

Open DevPortal http://localhost:8003
Click on Catalog
Click on the hyperlink of an item in the list => nothing happens
Click to the right of the link (not the link itself) => the filter is applied

Anything else?
No response

Hi @jeromeguillaume, we have an internal ticket for this. Please refer to FTI-5377 and feel free to let me know if you have any questions
PS: The dev portal is an enterprise-only feature and issues related to it should not be created here (the open-source project)


sizeForItem not called Help!!

I checked, please set datasource for collection view
Guide: https://github.com/ngodacdu/SSSheetLayout#usage

I checked, please set datasource for collection view
Guide: https://github.com/ngodacdu/SSSheetLayout#usage

But after I add data source just like you told me . It’s show an error
“'NSUnknownKeyException',
reason: '[<UIViewController 0x6e36ae0> setValue:forUndefinedKey:]: this
class is not key value coding-compliant for the key ”
On Tuesday, March 13, 2018, Ngô Đắc Du<EMAIL_ADDRESS>wrote:

I checked, please set datasource for collection view
Guide: https://github.com/ngodacdu/SSSheetLayout#usage
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/ngodacdu/SSSheetLayout/issues/1#issuecomment-372727910,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AXYuTULT1DY4Zcihu61tWVZCYAZ21jNYks5td_MjgaJpZM4SoS7t
.

Can you provide informations of xcode version, device and iOS version?. I will try to reproduce

I built it in Xcode 9 with IOS 11
Sent from my iPhone

On Mar 14, 2018, at 09:13, Ngô Đắc Du<EMAIL_ADDRESS>wrote:
Can you provide informations of xcode version, device and iOS version?. I will try to reproduce
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.

I've used xcode 9.2 and built sample in iPhone 7, iOS 11.2 no matter what.
sizeForItem method called. I think that issue was generated from your project.
Look at the log, i think you should check xib or storyboard.
Ref
https://stackoverflow.com/questions/3088059/what-does-this-mean-nsunknownkeyexception-reason-this-class-is-not-key-v
Please remove delegate outlet and reference outlet then connect outlet again.
If you still have that issue, please share code for me, i will support you.
Thank you.


Can't install materialize with meteor add materialize:materialize

According to the documentation here, to add Materialize to Meteor all you have to do is run meteor add materialize:materialize, which has worked for me previously.
I just ran that command today and it gave me the following error:

I even tried it for version 0.99.0 and got the same error. Am I doing something wrong or is this a bug?

Duplicate #5010


NEAR log does not work in view methods

Community members have reported that near.log doesn't work in view methods. No message gets returned which they find to be an issue when it comes to debugging their smart contract

I think view function can't send log since it's not a transaction.

I just test it and near.log works. But logs are only available when you view it with workspace-js viewRaw. Using view will only return the view result and discard log.
To see how it works, modify this test contract:
export function test_storage_read() {
  near.log('aaa') // add this line
  near.valueReturn(near.storageRead(bytes("\x00tesdsst\xff")));
}

and this test:
  let r = await storageApiContract.viewRaw("test_storage_read", "");
  console.log(JSON.stringify(r, null, 2)) // add this line

You'll see the log:
{
  "block_hash": "24VxvXXz3s7EPJDTT3HdXXDSLuig7KY6rQ3M8CjyUE5M",
  "block_height": 13,
  "logs": [
    "aaa"
  ],
  "result": [
    0,
    1,
    255
  ]
}

So it's not a bug. As for this log is not collected and displayed by workspace-js framework, it's an issue of workspace - we'll fix there


popver with html content

Hi...
can i do a popover with html content like popover example
I tried but not work with Metro...
I want to do something like expedia example when click Travelers option
Other observation: (it's a personal observation)
i think Metro could be change the popover after style (.neb::after) ... more soft, more like codepen example.
Thanks Master, you are the best.

Hi, you can set HTML content for popover with attribute data-popover-text="..." and set data-popover-hide="0". Popover will be closed if user click on it.

in 4.2.27 you can add close button to popover with attributes data-close-button="true" data-popover-hide="0"


ScriptHelper gets error 59 during deletion

ScriptHelper.log says System.UnauthorizedAccessExeption: Cannot write to the registry key

Please share the full log.

What Windows version are you on?

win11 pro 22H2

I'm guessing some registry permissions changed in Win11 for some reason. I don't have any W11 systems so I can't test it at the moment.

Currently on windows 11, any scripts you might want me to run to test?

I can confirm that I'm running into the same issues here: "System.UnauthorizedAccessException: Cannot write to the registry key" error in the logs. I'm running Windows 11 Insiders, canary build 25905.rs_prerelease.
Fwiw, I was also running the Wallpaper compression disabling script, but seeing as the issue is related to systems permissions, it's reasonable to just assume that it's going to affect at least some; if not all of them, as they all do write to some key areas of the registry.
I do have the Virtualization-based protection and Core Isolation features enabled, so this also might contribute to the restrictions. But seeing the way this error just sprung up on Win11, I'm going to put my best guess on Microsoft adding some changes to Win11 that now protects some areas of the registry from outside modifications.


Using #1850 as context, how would you style the color of  input checkbox's check mark?

The closed issue #1850 related to background color. But what about teh the color of the checkbox's check mark?
As described in #1850, this works
.ui.checkbox input:checked~.box:before,
.ui.checkbox input:checked~label:before {
    background-color: red !important;
}

I'd expect that color: red !important; would affect the check mark itself ... but it has no effect. The selectors certainly seem to be targeting the boxed check mark.
I inadvertantly posted this question in #1850 before realizing that it had been CLOSED.

Hi @brad-harper, this should actually be posted as a CSS and/or Semantic UI question on Stack Overflow instead. Our issue reporting here in Github is for managing bugs. Additionally, your question is CSS related and independent of Semantic UI React.
FWIW I frequently answer questions there as well, so I am not dismissing your question, just trying to get it into the proper venue.


Cards not displaying after doorbell lost network

@fuatakgun
Please check Services, Known Bug / Issues and Troubleshooting over here first: https://github.com/fuatakgun/eufy_security/blob/master/README.md
Checked.
Describe the bug
A clear and concise description of what the bug is.
After getting disconnected from my wireless network, my video doorbell (T8200) no longer provides a feed to the cards on my UI.
To Reproduce
Steps to reproduce the behavior:

Go to '...'
Click on '....'
Scroll down to '....'
See error

Go to overview.  Expect to see card with doorbell idle image (which when clicked provides a live image).  Click on 3 dots (edit dashboard menu).  Idle image shows below "Ben's Plug" but the camera provides an error: can't get URL for camera.doorbell.  Leaving this "edit mode" makes that card disappear from view.
Expected behavior
A clear and concise description of what you expected to happen.
The card to display and function as expected.
Screenshots
If applicable, add screenshots to help explain your problem.

**Hardware Information about Home Assistant
Home Assistant Blue / Hardkernal Odroid-N2
**Hardware Information about Cameras

Camera Model T8200
Live Streaming Protocol: RTSP or P2p

Additional context
Add any other context about the problem here.
The IP address definitely changed during this downtime/disconnect, from  .33 to .14. I tried uninstalling just the integration, restarting/clearing cache, and then reinstalling and that seemed to briefly work - but as soon as I tried to duplicate the card and add it to another view, it stopped working.   Have added screenshots of what seems like it should be all relative information.

Since this original post, I've installed HA fresh and installed all the addons/integrations I run one at a time.  Despite that, this error persists. I've now also got a Flood Light Camera from Eufy, and it too isn't usable with this addon.

same doorbell, same problem for me also.

@fuatakgun

is this still happening after TRUSTED DEVICE fix and 1.3.0 version of add-on?

is this still happening after TRUSTED DEVICE fix and 1.3.0 version of add-on?

Oh yes. It's been happening since I first posted about it...3 months ago?
I've long since pretty much given up on the camera aspect of this integration/addon.  At least my keyless entry is usable.

Just so you're aware, your latest integration update didn't fix this issue.

https://github.com/fuatakgun/eufy_security/issues/592


Are there any scripts available to easily reproduce the results of the paper?

Hi @fangbq,
To the best of my knowledge, no. Most of the results from the paper were run internally, and most of the algorithms are not directly included with this environment.

Hi @fangbq,
To the best of my knowledge, no. Most of the results from the paper were run internally, and most of the algorithms are not directly included with this environment.

I see. Thank you.

@lanctot I see. Thank you.


Don't remove whitespaces for maintainer name

This leads to the output to be formatted like this:
__author__ = 'abc'__maintainer__ = 'xyz'

which is a syntax error

Thanks :smiley:


TODO: move network level thread code

TODO: Line 37 of main
Try to see if we can move the receiver thread and acceptor thread, and their starts, into the network's init. I think it would be a neater solution (this way, although all threads are not started in the same place, an end user would not have to worry as much about network end threading)

Got a git error message, and then it posted twice (it left the title blank which I filled). Closing a duplicate issue.


What is meaning of -key and -value in Schema Registry

Description
I see the schema subjects are stored as -key or -value.
How to reproduce
Checklist
Please provide the following information:

[ ] Confluent.Kafka nuget version:
[ ] Apache Kafka version:
[ ] Client configuration:
[ ] Operating system:
[ ] Provide logs (with "debug" : "..." as necessary in configuration)
[ ] Provide broker log excerpts
[ ] Critical issue

you will typically want to register schemas associated with both key and value in Schema Registry - the suffic simply differentiates them.


Added Oauth2 Error response handling which format differs

It seems there are two error types supported by Box api:
Common errors https://developer.box.com/guides/api-calls/permissions-and-errors/common-errors/ which handled properly
Oauth2 errors https://developer.box.com/reference/resources/oauth2-error/ which currently are not supported and have different response structure.
This PR provides support for Oauth2 errors.

Thanks @m1x0n!


Example Alignment with Reference Architectures

We will need to rename/align our examples with our reference architectures.

[ ] https://github.com/PaloAltoNetworks/terraform-azurerm-vmseries-modules/issues/228

https://github.com/PaloAltoNetworks/terraform-azurerm-vmseries-modules/issues/187

This will be part of #251

closed with #251


fix generative mesh export

fixes https://github.com/nerfstudio-project/nerfstudio/issues/212.
looks like there is a separate problem in our model implementation that results in a sphere of density surrounding the object.

for now using the export bounding box helps with this, but I'm planning on taking a look at why this is happening also.


The Oath player is not working

As per the subject, same for Movies or Series. After selecting the player, it just hangs. What gets it working for me is using the below for direct.theoath.json:
{     "name" : "[COLOR orange]The Oath[/COLOR]",     "plugin" : "plugin.video.theoath",     "priority" : 500,     "play_movie" : "plugin://plugin.video.theoath/?action=play&title={title_+}&year={year}&imdb={imdb}&meta=%7B%22rating%22%3A+%22%22%2C+%22code%22%3A+%22{imdb}%22%2C+%22tmdb%22%3A+%22{id}%22%2C+%22imdb%22%3A+%22{imdb}%22%2C+%22year%22%3A+%22{year}%22%2C+%22duration%22%3A+%22%22%2C+%22plot%22%3A+%22{plot}%22%2C+%22votes%22%3A+%22%22%2C+%22title%22%3A+%22{title_+}%22%2C+%22fanart%22%3A+%22{fanart}%22%2C+%22tagline%22%3A+%22%22%2C+%22writer%22%3A+%22%22%2C+%22next%22%3A+%22%22%2C+%22poster%22%3A+%22{poster}%22%2C+%22mediatype%22%3A+%22movie%22%2C+%22director%22%3A+%22%22%2C+%22studio%22%3A+%22%22%2C+%22genre%22%3A+%22%22%2C+%22metacache%22%3A+true%2C+%22premiered%22%3A+%22{premiered}%22%2C+%22originaltitle%22%3A+%22{title_+}%22%2C+%22cast%22%3A+%5B%5B%22%22%5D%5D%2C+%22mpaa%22%3A+%22%22%2C+%22trailer%22%3A+%22plugin%3A%2F%2Fplugin.video.theoath%2F%3Faction%3Dtrailer%26name%3D{title_+}%2B%2528{year}%2529%22%7D&t={now}",      "play_episode" : "plugin://plugin.video.theoath/?action=play&title={title_+}&year={year}&imdb={imdb}&tvdb={id}&season={season}&episode={episode}&tvshowtitle={clearname_+}&premiered={firstaired}&meta=%7B%22rating%22%3A+%22%22%2C+%22code%22%3A+%22{imdb}%22%2C+%22tmdb%22%3A+{tmdb}%2C+%22imdb%22%3A+%22{imdb}%22%2C+%22year%22%3A+%22{year}%22%2C+%22duration%22%3A+%22%22%2C+%22plot%22%3A+%22{plot}%22%2C+%22votes%22%3A+%22%22%2C+%22thumb%22%3A+%22{thumbnail}%22%2C+%22title%22%3A+%22{title_+}%22%2C+%22tvdb%22%3A+%22{id}%22%2C+%22label%22%3A+%22{title_+}%22%2C+%22season%22%3A+%22{season}%22%2C+%22status%22%3A+%22%22%2C+%22poster%22%3A+%22{poster}%22%2C+%22tvshowtitle%22%3A+%22{clearname_+}%22%2C+%22mediatype%22%3A+%22episode%22%2C+%22studio%22%3A+%22%22%2C+%22genre%22%3A+%22%22%2C+%22banner%22%3A+%22{thumbnail}%22%2C+%22episode%22%3A+%22{episode}%22%2C+%22premiered%22%3A+%22{firstaired}%22%2C+%22fanart%22%3A+%22{fanart}%22%2C+%22trailer%22%3A+%22plugin%3A%2F%2Fplugin.video.theoath%2F%3Faction%3Dtrailer%26name%3D{clearname_+}%22%7D&t={now}"  }

What's changed? It's very hard to tell the difference by looking at a wall of code 😬

I'll try to post a file once I'm home. Basically using just "&" rather than "&" for the parameters.
I didn't dig deeper into the "why".

In that case, it depends on what URL encoding/decoding the add-on is using. I'll need to test one way vs. the other.

direct.theoath-patch.txt

In that case, it depends on what URL encoding/decoding the add-on is using. I'll need to test one way vs. the other.

Great, thanks.

In that case, it depends on what URL encoding/decoding the add-on is using. I'll need to test one way vs. the other.

Has this been tested and added to the player file zip download? I'm guessing probably not.
I found I had a problem with The Oath's player file in Kodi 19, but only on my LibreElec device. My Windows and Android devices continued to work normally.
In any case from @fedevx 's posts above I made up a new player file as follows which works on all my devices.
The only issue I'm having is that I'm no longer getting progress marked in TMDBH if I stop a video before the end. I'm not quite sure where the issue with that is though since it's happened since updating to Kodi v19, and both The Oath and TMDBH have updated to Python 3 too.
{
    "name": "[COLOR orange]The Oath - NEW[/COLOR]",
    "plugin": "plugin.video.theoath",
    "priority": 1,
    "play_movie": "plugin://plugin.video.theoath/?action=play&title={title_+}&year={year}&imdb={imdb}&meta=%7B%22rating%22%3A+%22%22%2C+%22code%22%3A+%22{imdb}%22%2C+%22tmdb%22%3A+%22{id}%22%2C+%22imdb%22%3A+%22{imdb}%22%2C+%22year%22%3A+%22{year}%22%2C+%22duration%22%3A+%22%22%2C+%22plot%22%3A+%22{plot}%22%2C+%22votes%22%3A+%22%22%2C+%22title%22%3A+%22{title_+}%22%2C+%22fanart%22%3A+%22{fanart}%22%2C+%22tagline%22%3A+%22%22%2C+%22writer%22%3A+%22%22%2C+%22next%22%3A+%22%22%2C+%22poster%22%3A+%22{poster}%22%2C+%22mediatype%22%3A+%22movie%22%2C+%22director%22%3A+%22%22%2C+%22studio%22%3A+%22%22%2C+%22genre%22%3A+%22%22%2C+%22metacache%22%3A+true%2C+%22premiered%22%3A+%22{premiered}%22%2C+%22originaltitle%22%3A+%22{title_+}%22%2C+%22cast%22%3A+%5B%5B%22%22%5D%5D%2C+%22mpaa%22%3A+%22%22%2C+%22trailer%22%3A+%22plugin%3A%2F%2Fplugin.video.theoath%2F%3Faction%3Dtrailer%26name%3D{title_+}%2B%2528{year}%2529%22%7D&t={now}",
    "play_episode": "plugin://plugin.video.theoath/?action=play&title={title_+}&year={year}&imdb={imdb}&tvdb={id}&season={season}&episode={episode}&tvshowtitle={clearname_+}&premiered={firstaired}&meta=%7B%22rating%22%3A+%22%22%2C+%22code%22%3A+%22{imdb}%22%2C+%22tmdb%22%3A+{tmdb}%2C+%22imdb%22%3A+%22{imdb}%22%2C+%22year%22%3A+%22{year}%22%2C+%22duration%22%3A+%22%22%2C+%22plot%22%3A+%22{plot}%22%2C+%22votes%22%3A+%22%22%2C+%22thumb%22%3A+%22{thumbnail}%22%2C+%22title%22%3A+%22{title_+}%22%2C+%22tvdb%22%3A+%22{id}%22%2C+%22label%22%3A+%22{title_+}%22%2C+%22season%22%3A+%22{season}%22%2C+%22status%22%3A+%22%22%2C+%22poster%22%3A+%22{poster}%22%2C+%22tvshowtitle%22%3A+%22{clearname_+}%22%2C+%22mediatype%22%3A+%22episode%22%2C+%22studio%22%3A+%22%22%2C+%22genre%22%3A+%22%22%2C+%22banner%22%3A+%22{thumbnail}%22%2C+%22episode%22%3A+%22{episode}%22%2C+%22premiered%22%3A+%22{firstaired}%22%2C+%22fanart%22%3A+%22{fanart}%22%2C+%22trailer%22%3A+%22plugin%3A%2F%2Fplugin.video.theoath%2F%3Faction%3Dtrailer%26name%3D{clearname_+}%22%7D&t={now}",
    "is_resolvable": "false"
}

The Oath must need some value passed through the player URI in order to be able to mark progress correctly. Really, TMDbHelper has no control over it, as the add-on that ends up playing the content is what actually marks the item as watched.
Honestly, I don't use The Oath, or really TMDbHelper anymore either... If the above player file is working, I'd be happy to merge it into this package, but I can also wait until whatever is wrong with it gets fixed.

If I understand correctly from reading the various issues and jurial's repsonses then Resume points are saved by Kodi itself, not TMDBH or the addon (well addons can do it too, but that's another story).
And if I further understand correctly player files for addons that 'any addon that scrapes sources should have is_resolvable set to false (i.e. set it to PlayMedia NOT setResolvedUrl).
Then TMDBH uses it's Settings > Expert > Reroute PlayMedia to call xbmc.Player() directly to send the Resume point.
Or am I mistaken? I not an expert in this regard.
In my case though the Resume points don't appear to be saved by Kodi anymore since updating to v19. I no longer get the Resume indicator in TMDBH episode lists. (it does work in YouTube though, so it's not an overall Kodi problem).
Really, I'm not even sure if this is working in Kodi v19 & TMDBH in the first place. Maybe it's broken and I need to raise an issue over on TMDBH.
Regarding the above player file, yes it is working on the 3 devices that I have to test it on: LibreElec, Windows, and Android.
But what do you mean by "I can also wait until whatever is wrong with it gets fixed." ? Is there something wrong in The Oath that should be addressed? The developer is very responsive on GitHub.  As I mentioned, personally I only had problems on LibreElec, which to me indicated an LE/Kodi issue after updating to v19, possibly with reading the player file correctly ( "&" rather than "&" for the parameters, as @fedevx mentioned) rather than an issue with The Oath itself?
But again, I'm not an expert.
THANKS good doctor :)

Kodi does save resume points for library items, but that doesn't work for most add-ons.
The is_resolvable flag has nothing to do with this, unless I'm mistaken.
In almost every case of this, the add-on that is doing the playback needs to mark progress on Trakt.tv, which then TMDbHelper can read back from the website.
I meant if you're able to get the player to a state where marking resume points works.

Hello,
as they said, fedevx and jmh2002, do not return any source (TheOath via TMDB in libreelec). I thought it is still beta, but it has reached v.10 ... the problem only persists in raspberry (win.si android it's perfect).
Where can I get help? Thanks!

Hello,
as they said, fedevx and jmh2002, do not return any source (TheOath via TMDB in libreelec). I thought it is still beta, but it has reached v.10 ... the problem only persists in raspberry (win.si android it's perfect).
Where can I get help? Thanks!

The patch I posted still works if you want to give that a try. Still, would be great if this could be officially fixed.

Hello,
as they said, fedevx and jmh2002, do not return any source (TheOath via TMDB in libreelec). I thought it is still beta, but it has reached v.10 ... the problem only persists in raspberry (win.si android it's perfect).
Where can I get help? Thanks!

The patch I posted still works if you want to give that a try. Still, would be great if this could be officially fixed.

Perfect!In case, it is not forbidden, here the changes for several addons in the LibreELEC Matrix for TMDB, which works with the changes made by fedevx. Change to Kodi \ addons \ plugin.video.themoviedb.helper \ resources \ players .zip. Thanks, to solve the problem! Have a good day!
players.zip

Kodi does save resume points for library items, but that doesn't work for most add-ons.
The is_resolvable flag has nothing to do with this, unless I'm mistaken.
In almost every case of this, the add-on that is doing the playback needs to mark progress on Trakt.tv, which then TMDbHelper can read back from the website.
I meant if you're able to get the player to a state where marking resume points works.

Hi, for the matrix in tmdb helper, I recreated for the addons players I use them. Asgard and Q Cuantumm I don't have RD access, the others I use. In case you see them good, you can update in a4k.https: //bit.ly/ playerstmdbmod.Good day!


[SUGGESTION] Maybe Lua

Maybe someone want to add Lua in Osiris
I find Osiris repo with integrated Lua, but without functional
Maybe Daniel can migrate Lua from this repo to original Osiris repo

Lua isn't needed in an open source cheat, everything you need can just be added directly to the code.

Yea, but for example anybody wants to add rainbow border to code? No.
But maybe, someone wants rainbow border or someone (bad English) else in Osiris, but majority doesn't want this shit

well it's a good idea luas , since some ppl like me don't know how to code . luas are a pretty good idea
we can use other ppls code without adding them into the cheat

interpreted code like python, c#, java, or lua will never run as fast as compiled C/C++. implementing LUA would have performance implications, also c++ is super easy to use, why not just program in it in the first place.
most LUAs implmentations don't have a standard, a fatality LUA won't work with onetap and a skeet lua wont work with aimware. all LUAs would have to be rewritten for Osiris.

well it's a good idea luas , since some ppl like me don't know how to code . luas are a pretty good idea

it's still programming?

we can use other ppls code without adding them into the cheat

literally just clone another person's code. git merge if you want multiple people's code.
git clone https://github.com/USER/REPO.git
LUAs add another detection vector. also you could probably make a malicious LUA that would give arbituary code execution on a machine.

Yea, but for example anybody wants to add rainbow border to code? No.

I know, but i am stupid for c++))

interpreted code like python, c#, java, or lua will never run as fast as compiled C/C++. implementing LUA would have performance implications, also c++ is super easy to use, why not just program in it in the first place.
most LUAs implmentations don't have a standard, a fatality LUA won't work with onetap and a skeet lua wont work with aimware. all LUAs would have to be rewritten for Osiris.

well it's a good idea luas , since some ppl like me don't know how to code . luas are a pretty good idea

it's still programming?

we can use other ppls code without adding them into the cheat

literally just clone another person's code. git merge if you want multiple people's code.
git clone https://github.com/USER/REPO.git
LUAs add another detection vector. also you could probably make a malicious LUA that would give arbituary code execution on a machine.

Yea, but for example anybody wants to add rainbow border to code? No.

REPO

pls reply

interpreted code like python, c#, java, or lua will never run as fast as compiled C/C++. implementing LUA would have performance implications, also c++ is super easy to use, why not just program in it in the first place.
most LUAs implmentations don't have a standard, a fatality LUA won't work with onetap and a skeet lua wont work with aimware. all LUAs would have to be rewritten for Osiris.

well it's a good idea luas , since some ppl like me don't know how to code . luas are a pretty good idea

it's still programming?

we can use other ppls code without adding them into the cheat

literally just clone another person's code. git merge if you want multiple people's code.
git clone https://github.com/USER/REPO.git
LUAs add another detection vector. also you could probably make a malicious LUA that would give arbituary code execution on a machine.

Yea, but for example anybody wants to add rainbow border to code? No.

Java is not interpreted. Go learn some basics kid.

~

Java is not interpreted. Go learn some basics kid.

are you not going to point out how C# isn't interpreted then?
https://en.wikipedia.org/wiki/Bytecode

Bytecode may often be either directly executed on a virtual machine (a p-code machine i.e., interpreter), or it may be further compiled into machine code for better performance.


deps file should use https for github repos

There seem to be some changes on github due to the recent DDoS attacks which prevent me from cloning Symfony repositories over plain HTTP. Only when switching all entries in deps to https:// is it that cloning works.
Wouldn't it make sense to use https by default for all repositories listed in deps?

download the latest version of the vendors script. It allows you changing the transport used for github by using --transport=https (only needed when cloning a deps, so for the first install or when you add a new one)

i changed all http to git and works also fine...

Why not use https by default?

@AltranoLabs you can also just specify --transport=git in the invokation.


rowType attribute while reading xls using SheetJS node package

I am using SheetJS npm package to read the xls . I want to detect the header of a table in the xls worksheet .

I have some existing xls file (provided by team mates)

I have created xls on my own with simple table , for which 1'st row is highlighted by color .
 const X = require('xlsx');
 console.log('with index1.js');

 const fileName = './add.xlsx';
 const workbook = X.readFile(fileName);

 var sheet_name_list = workbook.SheetNames;
 console.log(sheet_name_list);

let ws = workbook.Sheets[sheet];
console.log(" type "+typeof ws);
console.dir(ws,{depths :null ,colors: true});
const data_1 = X.utils.sheet_to_json( ws);
console.dir(data_1, {depths: null , colors : true});

When i print the data
for existing xls sheets (provided by team mates 0, i can observe a attribute , rowType or __EMPTY: 777,
Example ,
[ { rowType: 'header',
    user_name: 'User name',
    first_name: 'First name',
    last_name: 'Last name',
    __EMPTY: 'Suffix',
    email: 'Email',
    'domains.info[0].name': 'Primary domain',
    'domains.info[0].key': 'Domain key',
    'domains.info[1].name': 'Domain #2',
    'domains.info[1].key': 'Domain key',
    'entitlement.scopes[0]': 'Entitlement scope',
    'entitlement.scopes[1]': 'Entitlement scope #2',
    'domains.designation': 'Workflow Role',
    'domains.org.key': 'Corporate Org Key' },

For xls ,which are created by me , i can see that data does not have rowType attribute . It is just pure data , also header is missing
  [ { num1: 1, num2: 1, val: 2 },
  { num1: 1, num2: 2, val: 3 },
  { num1: 1, num2: 4, val: 5 } ]

Any specific xls needs to be written to get rowType attribute
I have tried the following way

Adding a filter to first row
Selecting table style , consequently in Table options in xls , Header Row gets checked
from the ribbon , i have selected , insert a table with header
In all cases , i do not get rowType attribute .

incorrect issue . closing down , as rowType attribute is not exposed by sheetJS


Move sub-layers into separate repositories

The gnu-linux and win32-bin sub-layers were moved into separate
repositores. Drop them from here so that the basement layer is
restricted to the sandbox, toolchains and support for the most common
build systems.

what's about the default.yaml that isn't available in the new layers?
e.g. the *_MIRROR variables still missing. maybe something else too?

No, as written in the README the separated layers are intended to be used together with the basement layer...

how this looks if both new layers will be used?
layers:
- basement-gnu-linux
    - basement
- basement-win32-bin
    - basement

btw: the README file of basement-win32-bin has no information.

Ah, I see the indentation in the README is broken. Sorry.
So in your project you do:
layers:
    - basement-gnu-linux
    - basement-win32-bin
    - basement

Just stack them on top of each other.

so just the information in win32 basement is missing, that it depends on basement.

Yes. I thought that it is independent but overlooked the mirrors variables.

i tested it in our company. everything is fine. cool stuff. thx.


refine activity factory

Pull Request Test Coverage Report for Build 89754

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 76.477%

Totals

Change from base Build 89475:
0.0%

Covered Lines:
8063

Relevant Lines:
10543

💛  - Coveralls

:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Builder.dll compared against version 4.5.3
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Builder.AI.Luis.dll compared against version 4.5.3
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Builder.AI.QnA.dll compared against version 4.5.3
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Builder.ApplicationInsights.dll compared against version 4.5.3
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Builder.Azure.dll compared against version 4.5.3
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Builder.Dialogs.dll compared against version 4.5.3
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Builder.TemplateManager.dll compared against version 4.5.3
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Configuration.dll compared against version 4.5.3
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Connector.dll compared against version 4.5.3
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Schema.dll compared against version 4.5.3


1:38.1 - add cross-profile transaction equivalence to existing TF?

Section Number https://profiles.ihe.net/ITI/PDQm/actors_and_transactions.html
Issue
PDQm points back to PDQ and PDQv3 transactions as equivalent
PDQV3 points to PDQ transaction as equivalent in Table 24.1-2 https://profiles.ihe.net/ITI/TF/Volume1/ch-24.html#24.1
PDQ has no similar info
Proposed Change
Do we make them consistent or remove this mapping?
How about just pointing to the content we moved to Appendix M? https://profiles.ihe.net/ITI/TF/Volume2/ch-M.html#M.4
When we answer this, consider making the same choice for the PIX family, and then create a traditional CP.
Priority:

Medium: Significant issue or clarification. Requires discussion, but should not lead to long debate.

I think appendix M already does address the three way equivilance.

wouldn't the resolution of #35 also satisfy this?

I agree that Appx M is fine.     Then, you can update https://profiles.ihe.net/ITI/TF/Volume1/ch-24.html#24.1 to point to Appx M, and either use the editor box mechanism to make the parallel updates to PDQ and PDQV3, or (when the text is written for PIXm), I can write a CP.

Issue solution #35 and #29 have resolved this.


Update compatibility to WP 6.6

It still works.

w.org doesn't require us to bump the plugin version for this change, but I'm not 100% sure the GitHub Action you have set up will suffice. It still might.
I run a separate Action also based on a 10up workflow, that you could adapt if needed.

Thanks!


feat: updated handling of Name change and Alteration NRs

Issue #: https://github.com/bcgov/entity/issues/24224
Updated how to handle Name change and Alteration NRs. The action button mainly.
If business exists in LEAR --> Go to modernized system
If not --> Go to Legacy system
No more usage of feature flags.

/gcbrun

Temporary Url for review: https://business-registry-dev--pr-36-ezrkpkyf.web.app

Do you have an example of a CHG or CNV that goes to COLIN?

Do you have an example of a CHG or CNV that goes to COLIN?

I adding this NR for BC Ltd that isn't in LEAR: NR 9958328

Do you have an example of a CHG or CNV that goes to COLIN?

I created this NR for BC Ltd that isn't in LEAR: NR 9958328

Sorry went AFK for a tiny bit. Thanks Sev for this. Yes, I wanted to create some test data and add it here. I tested now your NR, looks good. Appreciate it.

The code works well.
Check with Janis on whether a dialog is needed to let the user know they're going to COLIN. It seems abrupt currently.

I'm merging this PR as I am yet to receive an answer from Janis (been waiting for a while now). I'll implement a change in my next PR (if Janis wants any).

No more usage of feature flags.

Please create a Tech Debt ticket for removing the obsolete flags (after MVP).


State.setState Null check operator used on a null value

Steps to Reproduce
To be honest I have no clue. It is probably some edge case We saw this error appear on our sentry instance and we cannot reproduce it.
The code where this occured:
setState(() {
  _autoValidate = true;
});

The exception occurs here:
https://github.com/flutter/flutter/blob/1d9032c7e1d867f071f2277eb1673e8f9b0274e3/packages/flutter/lib/src/widgets/framework.dart#L1287
We have enabled sound null safety in our app.
Expected results:
We expected setState() to work as expected.
Actual results:

Logs

_CastError: Null check operator used on a null value
  File "framework.dart", line 1287, in State.setState
  File "login.dart", line 272, in _AuthLoginPageState.login

$ flutter analyze
Running "flutter pub get" in aio...                                750ms
Analyzing aio...

   info • Sort pub dependencies • pubspec.yaml:28:3 • sort_pub_dependencies

1 issue found. (ran in 8.0s)

$ flutter doctor -v
[✓] Flutter (Channel stable, 2.0.6, on Linux, locale en_US.UTF-8)
    • Flutter version 2.0.6 at /opt/flutter
    • Framework revision 1d9032c7e1 (4 days ago), 2021-04-29 17:37:58 -0700
    • Engine revision 05e680e202
    • Dart version 2.12.3

[✓] Android toolchain - develop for Android devices (Android SDK version 30.0.2)
    • Android SDK at /home/david/Android/Sdk
    • Platform android-30, build-tools 30.0.2
    • ANDROID_HOME = /home/david/Android/Sdk
    • Java binary at: /opt/android-studio/jre/bin/java
    • Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b3-6222593)
    • All Android licenses accepted.

[✓] Chrome - develop for the web
    • CHROME_EXECUTABLE = /usr/bin/chromium

[✓] Linux toolchain - develop for Linux desktop
    • clang version 11.1.0
    • cmake version 3.20.2
    • ninja version 1.10.2
    • pkg-config version 1.7.3

[✓] Android Studio
    • Android Studio at /opt/android-studio
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b3-6222593)

[✓] Connected device (3 available)
    • SM A515F (mobile) • <IP_ADDRESS>:5555 • android-arm64  • Android 10 (API 29)
    • Linux (desktop)   • linux                • linux-x64      • Linux
    • Chrome (web)      • chrome               • web-javascript • Chromium 90.0.4430.93 Arch Linux

• No issues found!

@psrcek
Can you provide more details if possible around how you are setting the state ? Specially
setState(() {
  _autoValidate = true;
});

It seems autoValidate is already true and the line again tries to set it to true which may be throwing the error ?

why would setting a variable to the same value be an issue?
I literally copy+pasted that code from my widget. _autoValidate is a variable in the state.
The code looks something like this:
class AuthLoginPage extends StatefulWidget {
  AuthLoginPage({Key? key}) : super(key: key);

  @override
  _AuthLoginPageState createState() => _AuthLoginPageState();
}

class _AuthLoginPageState extends State<AuthLoginPage> {
  bool _autoValidate = false;
  final GlobalKey<FormState> _formKey = GlobalKey<FormState>();

  @override
  Widget build(BuildContext context) {
    return Scaffold(
      ...
      body:
        ...
        Form(
          key: _formKey,
          autovalidateMode: _autoValidate ? AutovalidateMode.always : AutovalidateMode.disabled,
          child: ...
        ),
    );
  }

  Future<void> login() async {
    final form = _formKey.currentState;
    if (form != null && form.validate()) {
      // Irrelevant code
    } else {
      setState(() {
        _autoValidate = true;
      });
    }
  }
}

Keeping this issue open for further investigation.

Is it possible that your widget has gone away by the time you call setState? (See State.mounted.)
Are you seeing this in debug builds or release builds?

This happened in release builds. I'm pretty sure the widget was mounted since the function where setState is called is triggered by a button.

I'm sure it was mounted when the method was first called, but was it still mounted when the method got past all the asynchronous logics and actually called setState?

Without additional information, we are unfortunately not sure how to resolve this issue.
We are therefore reluctantly going to close this bug for now.
Please don't hesitate to comment on the bug if you have any more information for us; we will reopen it right away!
Thanks for your contribution.
Could everyone who still has this problem please file a new issue with the exact description of what happens, logs, and the output of flutter doctor -v.
All system setups can be slightly different, so it's always better to open new issues and reference related issues.


Configurable max message size.

Proposed Changes

Made the message max size configurable.

Related Issues
None

@RubenNL
This is not working. I got the following error on startup:
jq: error: syntax error, unexpected == (Unix shell quoting issues?) at , line 1:
if (.message_size_limit, == null) then
jq: error: syntax error, unexpected '|' (Unix shell quoting issues?) at , line 3:
elif (.message_size_limit, | type == "string") then
jq: error: syntax error, unexpected // (Unix shell quoting issues?) at , line 4:
.message_size_limit, // empty
jq: error: Possibly unterminated 'if' statement at , line 1:
if (.message_size_limit, == null) then
jq: 4 compile errors
bc: bad token at '* 1024000'
cont-init: info: /etc/cont-init.d/10-create-config.sh exited 1

I didn't test my last commit, which seems like is the problem. I can't test
it now, but I think I shouldn't have added the ',' on line 23 of the init
file.
Sorry for the inconvenience, I will make sure to test all future changes
more.
On Mon, Aug 15, 2022, 23:26 Erik Hilton @.***> wrote:

@RubenNL https://github.com/RubenNL
This is not working. I got the following error on startup:
jq: error: syntax error, unexpected == (Unix shell quoting issues?) at ,
line 1:
if (.message_size_limit, == null) then
jq: error: syntax error, unexpected '|' (Unix shell quoting issues?) at ,
line 3:
elif (.message_size_limit, | type == "string") then
jq: error: syntax error, unexpected // (Unix shell quoting issues?) at ,
line 4:
.message_size_limit, // empty
jq: error: Possibly unterminated 'if' statement at , line 1:
if (.message_size_limit, == null) then
jq: 4 compile errors
bc: bad token at '* 1024000'
cont-init: info: /etc/cont-init.d/10-create-config.sh exited 1
—
Reply to this email directly, view it on GitHub
https://github.com/erik73/addon-mail/pull/148#issuecomment-1215855401,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ADZGJD5CP7WNKGRIBFRWXSTVZKYXNANCNFSM56QHCHNA
.
You are receiving this because you were mentioned.Message ID:
@.***>

No problem. I removed the ','.
I will test it soon and report back.

@RubenNL .
I implemented the changes you suggested on line 23, and now it works as expected. I have released a new version.
Thanks for your help!