[Semantics] When using a ListView nested inside a horizontal scroll view, I cannot select semantic nodes which are initially beyond the width of the screen

I have made a table which uses a ListView for my rows, nested inside a horizontal scroll view to allow scrolling between the columns.
My issue is that the cells which are initially beyond the width of the screen, cannot be selected with TalkBack or VoiceOver. i.e. if columns 0 to 3 are initially visible, VoiceOver/TalkBack does not select anything after column 3. The vertical direction is fine - I can select all the rows vertically, even those which I have to scroll to reach.
The issue only occurs when I place my row inside a list view. If I place a row directly inside the horizontal scroll view, I can select all columns. I've included a row (index -1) in the example code to show this. However, in production I need to use the ListView for performance.
The issue occurs on both iOS and Android. I tested on iOS 13.6 and Android 10.
Note that if I set explicitChildNodes to false on my row, the entire row contents are spoken - including all those which are beyond the screen. But it's still not possible to select the row from outside the initial width of the screen.
Steps to Reproduce

Run flutter create bug.
Replace main.dart with this:

import 'package:flutter/material.dart';

void main() {
  runApp(MyApp());
}

class MyApp extends StatelessWidget {
  @override
  Widget build(BuildContext context) {
    return MaterialApp(
      title: 'Flutter Demo',
      theme: ThemeData(
        visualDensity: VisualDensity.adaptivePlatformDensity,
      ),
      home: Material(
        child: SafeArea(
          child: ListWidget2D()
        )
      ),
    );
  }
}

class ListWidget2D extends StatelessWidget {

  final _colWidth = 120.0;
  final _cols = 20;
  final _rows = 50;

  @override
  Widget build(BuildContext context) {
    return SingleChildScrollView(
      scrollDirection: Axis.horizontal,
      child: SizedBox(
        width: _rows * _colWidth,
        child: Column(
          children: [
            _buildRow(-1),
            Expanded(
              child: ListView.builder(
                  itemCount: _rows,
                  itemBuilder: (context, index) {
                    return _buildRow(index);
                  }
              )
            )
          ]
        ),
      ),
    );
  }

  Widget _buildRow(int rowIndex) {
    return Semantics(
      explicitChildNodes: true,
      child: Row(
          children: List.generate(_cols, (colIndex) {
            return SizedBox(
                width: _colWidth,
                height: 50,
                child: Text('Cell $rowIndex,$colIndex')
            );
          }).toList()
      )
    );
  }
}

Activate VoiceOver on iOS or TalkBack on Android
Move your finger over the visible cells, and observe the contents are spoken
Scroll the table to the right, revealing cells which were not already visible
Move your finger over the new cells

Expected results:
The contents of the new cells are spoken.
Actual results:
The contents are not spoken. The system sound effect indicates that there are no semantic elements there.

Logs
No log output of `flutter run--verbose` after the app launched. If log output of the build would be useful, I can add that in.
flutter analyze
Analyzing bug...
No issues found! (ran in 2.4s)

flutter doctor -v
[✓] Flutter (Channel stable, 1.20.2, on Mac OS X 10.15.6 19G2021, locale en-GB)
    • Flutter version 1.20.2 at /Users/omarf/Developer/flutter
    • Framework revision bbfbf1770c (3 weeks ago), 2020-08-13 08:33:09 -0700
    • Engine revision 9d5b21729f
    • Dart version 2.9.1

[✓] Android toolchain - develop for Android devices (Android SDK version 30.0.0)
    • Android SDK at /Users/omarf/Library/Android/sdk
    • Platform android-30, build-tools 30.0.0
    • Java binary at: /Applications/Android Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b3-6222593)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 11.6)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 11.6, Build version 11E708
    • CocoaPods version 1.8.4

[✓] Android Studio (version 4.0)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin version 48.1.2
    • Dart plugin version 193.7361
    • Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b3-6222593)

[!] IntelliJ IDEA Ultimate Edition (version 2020.1.2)
    • IntelliJ at /Users/omarf/Applications/JetBrains Toolbox/IntelliJ IDEA Ultimate.app
    ✗ Flutter plugin not installed; this adds Flutter specific functionality.
    ✗ Dart plugin not installed; this adds Dart specific functionality.
    • For information about installing plugins, see
      https://flutter.dev/intellij-setup/#installing-the-plugins

[!] VS Code (version 1.48.2)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    ✗ Flutter extension not installed; install from
      https://marketplace.visualstudio.com/items?itemName=Dart-Code.flutter

Screenshots
In case my explanation didn't make sense, I've attached a few screenshots which hopefully explain the issue.

The above screenshot is the view before any scrolling. Column 3 is the last visible. Cell -1,3 is currently selected with TalkBack.

After scrolling a bit, the cells in column 4 & above cannot be selected. Only the row -1 can be selected in TalkBack, since this is not inside the ListView.
I've repeated these two screenshots with showSemanticsDebugger set to true.

This does reproduce on stable but not on latest master 1.22.0-10.0.pre.87 and latest beta 1.21.0-9.2.pre.

flutter doctor -v
[√] Flutter (Channel beta, 1.21.0-9.2.pre, on Microsoft Windows [Version 10.0.19041.450], locale en-US)
    • Flutter version 1.21.0-9.2.pre at C:\Development\flutter_beta
    • Framework revision 81a45ec2e5 (10 days ago), 2020-08-27 14:14:33 -0700
    • Engine revision 20a9531835
    • Dart version 2.10.0 (build 2.10.0-7.3.beta)

[√] Android toolchain - develop for Android devices (Android SDK version 30.0.1)
    • Android SDK at C:\Users\marku\AppData\Local\Android\sdk
    • Platform android-30, build-tools 30.0.1
    • Java binary at: C:\Users\marku\AppData\Local\JetBrains\Toolbox\apps\AndroidStudio\ch-0\193.6626763\jre\bin\java
    • Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b01)
    • All Android licenses accepted.

[√] Chrome - develop for the web
    • Chrome at C:\Program Files (x86)\Google\Chrome\Application\chrome.exe

[√] Android Studio (version 4.0)
    • Android Studio at C:\Users\marku\AppData\Local\JetBrains\Toolbox\apps\AndroidStudio\ch-0\193.6626763
    • Flutter plugin version 47.1.2
    • Dart plugin version 193.7361
    • Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b01)

[√] Connected device (4 available)
    • SM G950F (mobile)           • ce12171c51cc001c03 • android-arm64  • Android 9 (API 28)
    • sdk gphone x86 arm (mobile) • emulator-5554      • android-x86    • Android 11 (API 30) (emulator)
    • Web Server (web)            • web-server         • web-javascript • Flutter Tools
    • Chrome (web)                • chrome             • web-javascript • Google Chrome 85.0.4183.83

• No issues found!

Closing this as probably fixed.


[BUG] error with filters for quantitative fields with null values

Describe the bug
I cannot apply a filter on a quantitative column if there are None values in this column.
To Reproduce
Steps to reproduce the behavior:

You can create a dataframe like this one :

import pandas as pd
import pygwalker as pyg
df = pd.DataFrame(columns=["weight", "gender", "score"], data=[[None, None, 1115], [77.65, "M", 601], [78, "F", 800]])
pyg.walk(df)

Go to Visualization
Drop the quantitative field weight into Filters
You get the error "Something went wrong"

However, you don't get the error for the fields gender (Nominal) or score (no null values).
Versions

pygwalker version: 0.3.20
python version: 3.10.11

Additional context
I have a very large dataframe with a lot of columns, and I don't want to do a df.dropna() before executing the pyg.walk().

This is a front-end computing bug.
You can use pyg.walk(df, use_kernel_calc=True) to fix it temporary.
use_kernel_calc=True wiil use duckdb to finish calculation, and it can process large datas.
In future versions, we will fix this bug.

Hi @didi9575 , pygwalker 0.4.0 already fixed it.
pip install --upgrade --pre pygwalker


📝 Update TrustedHostMiddleware Documentation

Summary
This PR updates the documentation for TrustedHostMiddleware to reflect the current code more accurately.
Changes

TrustedHostMiddleware:

Added documentation for the www_redirect parameter. When set to True, this option redirects requests to non-www versions of allowed hosts to their www counterparts. This parameter defaults to True.

 Here is the link for your reference: Starlette Middleware Documentation.
Could you please merge my pull request for the FastAPI documentation as well?

📝 Docs preview for commit 4894a037e59736ac16c0ac638394df445746be35 at: https://5e9c50ee.fastapitiangolo.pages.dev

@svlandeg Thank you for merge recent commits!
If you don't mind, Could you review my PR?

Hi @soulee-dev,
Thanks for your PR! We're currently going through the backlogs of PRs. There's no need to ping individual maintainers - we'll definitely get to review your PR, it's in our queue. Thanks for your patience! 🙏


Modernize code (more idiomatic C++11 and faster container initialization)

Great talk and great library.
Here's a small and simple pull request to modernize some code.

Most containers can be constructed using std::initializer_list.

When dealing with initializer lists of key-value pairs, the key should be const.

Use for-each loops instead of iterators-based loops.

:+1:

Looks good. Thanks @SuperV1234!


Add in support for Azure Keyvault

Secrets should be retrieved and stored in Azure Keyvault
For this deployment it would be helpful if an Azure Keyvault was either deployed as an optional resource to handle secrets (or reference to existing vault could be given).  ( this could potentially be used for storing Sasviya secrets in workload deployments as well).
Thanks!

Added the note here that a solution like this would have to be investigated across all of the current cloud providers supported by SAS Viya 4.

At this time adding support for Azure keyvault is not priority but will be investigated in future.


Reinstall.yml: move settings.php to variables

this one looks like was done by your refactoring some time ago, right? @Sanchiz

Not yet. settings.php is still hardcoded in script.

We should move this to variable
https://github.com/propeoplemd/cibox/blob/master/github/files/drupal7/scripts/devops/reinstall/post_settings.yml#L9

ah, ok, was wondered becuase of lack of summary info here


[NEW UI] Update page - Error handling

Questions
Answers

Description?
Add error handling process (alert and buttons).

Type?
new feature

BC breaks?
no

Deprecations?
no

Fixed ticket?
N/A

Sponsor company
@PrestaShopCorp

How to test?
Try to do an update you are sure it finish on error. WARNING: Send error report does not run for moment and restore buttons can't redirect cause page didn't exist for moment.

PR merged, well done!
Message to @PrestaShop/committers: do not forget to milestone it before the merge.


conntrack: Add _IsFirst field

This PR adds the _IsFirst field to connection records (newConnection, updateConnection and endConnection).
It's not added to flowLog records.
The field is set to true only on the first record of the connection.
It is useful in cases where newConnection records are not outputted (to reduce the number output records)
and there is a need to count the total number of connections: simply counting _IsFirst=true
This solves #298

I don't understand how this helps. By counting the number of  _IsFirst=true we get the total number of newConnections from the beginning of measurement. So this is essentially equivalent to counting the times we get newConnection (if we get each one). Can't we continue to count newConnection, and allow to have a newConnection reported on the first output record (even if we also have endConnection and/or updateConnection reported on the same record)?

@kalman When the outputRecordTypes is configured with [newConnection, updateConnection, endConnection], i.e. to output newConnection records, then the _IsFirst flag will be set to true only on them. So counting _IsFirst="true" is equivalent to counting _RecordType="newConnection" as you mentioned.
But, when the outputRecordTypes is configured with [updateConnection, endConnection], i.e. to not output newConnection records, then, counting _IsFirst="true" is my suggest way to count the number of connections. Counting _RecordType="newConnection" will return zero.


Show warning/prevent calling app .run() in Plugin build method

What problem does this solve or what need does it fill?
Accidentally calling .run() in a plugin's build method prevents other plugins from running correctly. This is because .run() Finalizes the [App] configuration. Sometimes one might call this method by mistake and spend hours trying to figure out why most of their app isn't running.
What solution would you like?
Either a warning informing users of an accidental .run() call or an alternative App struct for plugins to use where .run() is not present. The App's .run() method should only be possible to call in the main app configuration as is intended.
What alternative(s) have you considered?

Not accidentally calling .run() in a plugin's build method

Duplicate of https://github.com/bevyengine/bevy/issues/4231 (open PR: https://github.com/bevyengine/bevy/pull/4241)


Added Clear Button to History and WebSockets Tab

Fixes #411
Signed-off-by: Vitika9<EMAIL_ADDRESS>

The changelog needs to be updated. I didn't try the changes yet.

I'm having problems getting the HUD to run correctly, which is not a problem with this PR.
Can you post a screenshot of how the button appears?
I'm not seeing it yet, but that could be a local problem...

I can run the HUD via the weekly, so must be a build issue for me.
However when I copy your changes in I dont see the new button anywhere :/

What am I missing?

However when I copy your changes in I dont see the new button anywhere :/

The new button is at the top right, I don't know why it isn't visible at your end :(

Thanks! I'll have another go...

conflicting now :/

conflicting now :/

Done!

It seems that clicking the clear button in the History tab also clears the WebSockets tab (and vice versa), is that the expected behaviour? I thought they would work independently (or I'm doing something wrong?). Using ws.ifelse.io/.ws for tests.

My bad, I was using the same event for both of them. Fixed now 😀

@psiinon @thc202 do I need to make more changes? Please review.

Thank you!

@Vitika9 FYI we have just released the HUD with this change - I've installed it from the market place and it works well!
Many thanks - really good to see the HUD progressing again.
What are your details for the ZAP credits?
https://www.zaproxy.org/docs/desktop/credits/#zap-extended-team

@psiinon I am glad to see my changes reflect there, will contribute more! 😃
Here are my details: Vitika Soni, https://github.com/Vitika9


Cannot reproduce Qualcomm results with released model on Samsung S21 Ultra (Snapdragon version)

Hi,
When trying to reproduce mlperf-mobilev1.0 results submitted by Qualcomm, we found our results is much worse than the claimed ones, especially for single-stream mode.
Environment settings:
Device: both on HDK and Samsung S21 pro (got very close score).
SDK version: <IP_ADDRESS>4.
Model files: Tried all of the available '.dlc' files under this repo.
Scripts used for benchmarking: snpe_bench.py included in SNPE v<IP_ADDRESS>4
Results comparison:

Is there any released code implementation to reproduce the submitted results? Based on the current public dlc files and SNPE SDK, there seem to be 30%~60% gaps between measured & submitted FPS score.
Thanks.

Is there any released code implementation to reproduce the submitted results?

The code for QTI backend was released as part of the 1.0 submission. The results were verified on a OnePlus 9 Pro and a Realme 5G

Running snpe_bench.py is not equivalent to running the MLPerf app.

I do not have S21 Ultra to try the MLPerf app on

Thanks for replying. Could you please provide a public link to the code or MLPerf mobile app? That will help a lot.

Running snpe_bench.py is not equivalent to running the MLPerf app.

What is the difference between snpe_bench.py and MLPerf app measurement? DSP runtime libs, measurement methodology or something?


Case-sensitive volume and kitematic on Yosemite

Kitematic 0.6.3 does't support to be started from a case-sensitive volume. It results a blank page without any top menu or UI element.
Tested with Yosemite 10.10.3 and a case-sensitive volume.

Could you share the exact volume name? This is the first time I've seen this

The volume name is "OSX Yosemite". To understand my test :

Updated to 0.6.3 from 0.5.3 : doesn't works anymore
Reinstallation -> not better
If I move the .app that doesn't works on another volume whom is not case-sensitive, It is ok.

If I understand this properly, you've installed Kitematic onto /Volumes/OSX Yosemite and getting a blank page when Kitematic starts?

Yes, exactly. I even can't use developer tools to be more precise, because the top menu does't load properly.

Closing this for lack of activity in the past 15days.


Make a Casdoor auth middleware for Traefik

See the official middleware: https://doc.traefik.io/traefik/middlewares/forwardauth/
and examples:

https://github.com/dniel/traefik-forward-auth0 (docs: https://traefik-forward-auth0.readthedocs.io/en/latest/)
https://github.com/thomseddon/traefik-forward-auth#applying-authentication
https://github.com/mesosphere/traefik-forward-auth

@1340908470

@1340908470

Moved to: https://github.com/casdoor/traefik-casdoor-auth


Add STDIN support as -r / --REMOTE_URL argument for batch processing

Use Case
My team uses hub to do issue reporting. The collection of repositories stays pretty static. By leveraging #2159 as the "innards" of a loop of a list of repo URLs delivered via STDIN, we generate a nice report.
Example Use
cat list_of_prework_urls.txt |bin/hub issue -r STDIN --include-pulls -f '%U%n' |tee  issues
Presentation Note
This PR is built off of #2159, so this particular diff is a bit hairy pending that landing to master. The diffset can be more easily grokked here.

Thank you for your contribution, but I don't see this feature as something that we'd like to have or maintain in hub.
I fully agree with your other PR to add -r to hub issue. Ideally, I want to see -r supported in every hub command. When I find time, I will review and help with your other PR to get merged. Once that is done, you will be able to achieve this with a loop:
while read remote_url; do
  bin/hub issue -r "$remote_url" --include-pulls -f '%U%n'
done < list_of_prework_urls.txt | tee issues


GitHub actions that deploy and initiate a VPN connection to our cloud environment should use a PING_URL

See #1868 and specifically this part of it: https://github.com/CDCgov/prime-reportstream/pull/1868#discussion_r690320174
However, this will require some work (and maybe depend on #778) because it needs an HTTP server to ping inside the environment

Recent changes to make this a local action (see https://github.com/CDCgov/prime-reportstream/blob/master/.github/actions/vpn-azure/action.yml#L57) removed the PING_URL. Closing this because the next step using the VPN will fail anyway.


use automaxprocs to calculate the number of CPUs

this is  useful for container environments
Addresses: https://github.com/google/ko/issues/642

Related issue: https://github.com/golang/go/issues/33803

Essentially GOMAXPROCS is not cgroup aware so it reports the total CPUs on the machine instead of the # of CPUs allocated to the container

got the typo fixed upstream - force pushing the fix

Keep in mind that automaxprocs doesn't (yet) support cgroups v2 :)


Add AnthropicStreamingChatModel

Follow up of integration with Anthropic to include streaming chat model.

Summary by CodeRabbit

New Features

Introduced a new method for handling streaming messages in the Anthropic API.
Added functionality for streaming messages and improved error handling in the Anthropic client.
New classes Message and Delta added for better message response handling.
Implemented a streaming chat language model for generating AI responses based on user messages.

Tests

Added tests for the new streaming chat language model, covering functionality such as streaming answers and handling different types of system messages.

Hi @ilopezluna, thanks a lot for your contribution! I and @kuraleta have been working lately on the same thing and it is a pity that  the work that was done is duplicated. This issue was assigned to @kuraleta, so please let's sync next time before implementation? Thanks a lot for understanding!

@ilopezluna could you please review https://github.com/langchain4j/langchain4j/pull/750 ? 🙏


Merge into single exe

Closed in https://github.com/adamralph/dude/commit/cdae164424e4bee999f3f75305393aaf096a0171


BUILDS: Update release container

Use Go 1.6
Update to use make instead of build script

Fixes #450 and #454

Also superseded by #462 so closing


Code cleanup on save

Code Cleanup on Save should work for .cshtml files and we should validate that it works for .razor files.
It works for .cs files, but not .cshtml files which is odd if you have "Format document" set in the Code Cleanup profile.

Talked about this in triage. For format document we think this would be tracked by LSP platform, for other light bulbs we'd need to design something.
@gundermanc any thoughts on LSP platform integrations here?

If you want code cleanup to be available via LSP, is there an existing protocol message we can build on, or do we need to invent a new one, and register LSP fixer(s)?

I'd imagine the platform might run textDocument/formatting on save for the bare minimum. Anything more than that would probably be custom protocol related I'd imagine.


Thanks for pushing the new version @imjasonh. Seems to fixed the issues with install under go 1.18.

I spoke too soon. I am getting this now using build under 1.18
Error: failed to publish images: error building "ko://github.com/mchmarny/<repo>/cmd/app":
could not parse Go build info: line 43: expected 2 columns for build setting; got 1

Here is the CI command I'm using
KO_DOCKER_REPO=gcr.io/<project>/<name> \
  GOFLAGS="-ldflags=-X=main.version=v0.7.8" \
  ko build ./cmd/app/ --image-refs ./image-digest --bare --tags v0.7.8,latest

@imjasonh Thanks for releasing v0.11. My issue is fixed by v0.11. Please close this issue on your own time.

@imjasonh Thanks for releasing v0.11. My issue is fixed by v0.11. Please close this issue on your own time.

There's actually a fix still needed, which will become v0.11.1


Update toolbar icons

The latest release of the tool-bar package introduced breaking icon font updates. As a result, icons were not being displayed on the toolbar. The PR update the toolbar icons to the right names. See this discussion for more info. All the icons are still the same. I just updated the prefixes and icon-set.

Hi @jasongilman, I'd like to make this count towards my hacktoberfest contribution. Would you mind adding the label hacktoberfest-accepted to this PR and then accept it if you're fine with it?


Warn if an unknown feature name is passed to -enable-upcoming-feature or -enable-experimental-feature

@hborla @DougGregor Do you remember if it was a deliberate design decision to not warn if an unknown feature name is passed. For example, did we want to support passing an upcoming feature unconditionally and make it build without warnings even in older compiler that don’t know about this upcoming feature?
rdar://125102468

@swift-ci Please smoke test

Closing in favor of https://github.com/apple/swift/pull/72787


Cannot activate space_after_anon_function in Vue

Description
I am trying to activate space_after_anon_function in Vue files.  I have activated it in Beautify under CoffeeScript, JavaScript, EJS & JSX, each with Jslint happy on and off (where applicable).
Expected Results
The beautified code should have looked like:
...
export default {
  methods: function () {
    console.log("Hi")
  }
}
...

Steps to Reproduce

Add code to Atom editor
Run command Atom Beautify: Beautify Editor
This beautified code does not look right!

Debug
Here is a link to the debug.md Gist: https://gist.github.com/4dbecf350c4a7b95bb4a5ab5711da5cc
Checklist

[x] I have tried uninstalling and reinstalling Atom Beautify to ensure it installed properly
[x] I have reloaded (or restarted) Atom to ensure it is not a caching issue
[x] Searched for existing Atom Beautify Issues at https://github.com/Glavin001/atom-beautify/issues
so I know this is not a duplicate issue
[x] Generated debugging information and added link for debug.md Gist to this issue

Code should have been
export default {
  methods: {
    test: function () {
      console.log("Hi")
    }
  }
}

but it makes no difference.  Still can't get it to work.

I see in your debug.md you are only using the Config Options (Atom-Beautify's package settings) not the Project Options (.jsbeautifyrc). I do see space_after_anon_function is not available for Vue: https://github.com/Glavin001/atom-beautify/blob/master/docs/options.md#vue
You could simply create a .jsbeautifyrc file with { "vue": { "space_after_anon_funciton": true } } since all options are passed directly to the Vue beautifier: https://github.com/Glavin001/atom-beautify/blob/master/src/beautifiers/vue-beautifier.coffee#L10

I was able to reproduce the issue and also dived into some analysis. It appears Pretty Diff works with space_after_anon_function however JS-Beautify does not! See my debug.md for example: https://gist.github.com/Glavin001/f2ddbae9dac7b26134f00bf41cdd2405#final-options
And JS-Beautify is used within the Vue beautifier: https://github.com/Glavin001/atom-beautify/blob/master/src/beautifiers/vue-beautifier.coffee#L29
So this appears (at first glance) to be a JS-Beautify bug.

Update: I was able to get this working using jslint_happy: false and space_after_anon_function: false together! See https://gist.github.com/Glavin001/9ffe16aaba44c37c087ad56027df0f8b for example.

I've added JavaScript options to Vue docs: https://github.com/Glavin001/atom-beautify/blob/master/docs/options.md#vue
Hope that helps!

Published to v0.29.17


x/tools/gopls: document how to enable non-default staticcheck checks

Staticcheck checks can be configured with gopls's analyses settings, but this is not obvious from the current analyzer doc (https://github.com/golang/tools/blob/master/gopls/doc/analyzers.md). Settings doc has a link to the Staticcheck's official doc page (https://github.com/golang/tools/blob/master/gopls/doc/settings.md#staticcheck-bool), but how to enable/disable checks is not obvious.

This was fixed by https://go.dev/cl/583316.


Adding a blank Institution in Profile causes Internal Server Error Message

Steps

Go to User Profile
Go to Education Tab
Change Institution Field
Click "Add Another"
Click "Submit"

Expected
If there is a blank Employment or School object, it should be ignored with the save happening, and no Internal Server Error should happen in the console.
Actual
The save is not possible, and the following error shows in the console.
PUT https://osf.io/api/v1/settings/schools/ 500 (INTERNAL SERVER ERROR) jquery-1.11.2.min.js:4

You want this @billyhunt -- since you are working in this area at present?

Sure.  Makes sense to me.

You got it -- TY

@billyhunt I think you are saying that this also occurs with employment and leaving the first field for 'Institution / Employer' blank when it has 'Required' in the text box.
If you have an incorrect date range, the error will tell you what is incorrect, like 'End date must be greater than or equal to the start date.'
I think the same sort of error needs to be happening here to inform the user that the required field of 'blah blah' is blank.

Those fields are "required?" -- I agree with @nicipfeiffer 's 'blah blah' idea.

I was going to handle it by allowing a blank employment or school object.  That is the original state, and you can never get back to that.  To me, it makes sense to allow you to have no job and/or no school.  Sad, but very possible.  As it stands, I don't think you can reset to that original blank state.
So, I think the logic should be as follows when you hit "Submit"

If any field is filled in an Employment or Education object, then the Institution is required. An error message appears saying as much to let users know they messed up.
If no fields are filled in a new Employment or Education object, it is ignored.
If no fields are filled in an existing Employment or Education object, then that object is deleted from the database.

Does this make sense to you two?

Yes, I agree!

:+1: from me, thanks.

OK.  I have been holding off on completion of this until I finish adding the multiple websites to the profile, but if you think this takes priority, let me know.

As a :bug: @billyhunt , I'd say yes on this one first -- thanks.


ENYO-510 Enhancements

There were a couple scenarios in which placeholder and src didn't play well together.

If sizing and src were specified but placeholder wasn't, the generated CSS was invalid (it had a trailing comma after the url because it assumed a following url for the placeholder).
If placeholder was specified but src wasn't, the placeholder was not rendered. This was a logic gap in srcChanged

Also, when sizing, src, and placeholder were all specified, the placeholder image wasn't removed because handleLoad isn't called when the image is loaded as a CSS background-image. To handle this, I set the background-image as the placeholder and I added a proxy <img> to which the handlers are bound that loads the src. Once the src loads, the background-image is updated to be the src and the proxy is removed.
Enyo-DCO-1.1-Signed-off-by: Ryan Duffy<EMAIL_ADDRESS>

PR https://github.com/enyojs/enyo/pull/1056
The first two scenarios you mentioned are handled now properly in PR mentioned above. Regarding  removing the background-image when the src image gets loaded, includes one more complexity of loading backgroung-image, whenever "src" is updated. That's why I didn't consider removing background-image in my PR. Moreover I think it could be a performance issue considering there will be separate image element creation for each grid item. Can you please share your views on this.
Thank you!

There certainly is a performance impact to the additional node but I'd question how great it truly is. If we wanted to use this approach, we could further optimize it to not destroy the Image so when the tag is reused in a DataList, we wouldn't be unnecessarily creating and destroying them.
The simpler alternative is to document that when using placeholder, src, and sizing together, the placeholder image may show through depending on the sizing used and the image's characteristics.
Commented on your PR with a couple suggestions. I'd also recommend you cherry pick 81fe1bf from here to flesh out the Image Sample.


Updated deps and updated pre-commit hook

Updated the deps inside the toml and updated the pre-commit hook.
This also altered the entry of the local hooks to work as entry now instead of execution. Extra key command has been added to run the command.
Note:
Entry cannot be poetry run or poetry shell since it will error and ask for arguments.

We could also remove entry and just execute it as command.


File based catalog

This PR replaces the old catalog generation sql based for the file based. This is due some limitations (semver check that doesn't allow beta tags) and deprecation notice: time="2024-08-26T13:47:11Z" level=warning msg="\x1b[1;33mDEPRECATION NOTICE:\nSqlite-based catalogs and their related subcommands are deprecated. Support for\nthem will be removed in a future release. Please migrate your catalog workflows\nto the new file-based catalog format.\x1b[0m"

:warning: Please install the  to ensure uploads and comments are reliably processed by Codecov.
Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 61.78%. Comparing base (bf9ba03) to head (cfb87db).

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #201   +/-   ##
=======================================
  Coverage   61.78%   61.78%
=======================================
  Files           2        2
  Lines         785      785
=======================================
  Hits          485      485
  Misses        249      249
  Partials       51       51

Flag
Coverage Δ

unit
61.78% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

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


Shortcuts to OBS when not in focus?

Did you have also the problem that OBS does not receive the Keys when not in focus?
My Pico just sends fine keys to OBS, but once I stream/record and using a different app the Keys will not get through anymore.
With a (testcase) second USB keyboard and my old Arduino Mini based Streamdeck I have no problem.

can be done with co-software on computer, but implementing it wont be simple.


PyGame is Down

URL
https://github.com/pygame/pygameweb/issues/131
When was this post released
11 Jan 2023
Summary
Users are taking to r/python as well as the official PyGame website repo after getting a 500 error on the website. There currently has not been any official word from the maintainers on their Twitter or Mastodon accounts.
Code of Conduct

[ ] I would like my name mentioned on the podcast
[X] I agree to follow this project's Code of Conduct

It looks like it's back up. I'm not seeing a reason for it going down in that issue.

Yeah not sure... We can go ahead and close this then.


signature extraction

[debug] System config: []
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: ['--verbose', 'https://youtu.be/RP21evYWHmo']
[debug] Encodings: locale UTF-8, fs utf-8, out utf-8, pref UTF-8
[debug] youtube-dl version 2021.12.17
[debug] Python version 3.9.1 (CPython) - macOS-10.16-x86_64-i386-64bit
[debug] exe versions: ffmpeg 6.0, ffprobe 6.0, rtmpdump 2.4
[debug] Proxy map: {}
[youtube] RP21evYWHmo: Downloading webpage
[youtube] RP21evYWHmo: Downloading player 6ed0d907
ERROR: Signature extraction failed: Traceback (most recent call last):
File "/Library/Frameworks/Python.framework/Versions/3.9/lib/python3.9/site-packages/youtube_dl/extractor/youtube.py", line 1360, in _decrypt_signature
func = self._extract_signature_function(
File "/Library/Frameworks/Python.framework/Versions/3.9/lib/python3.9/site-packages/youtube_dl/extractor/youtube.py", line 1277, in _extract_signature_function
cache_res = res(test_string)
File "/Library/Frameworks/Python.framework/Versions/3.9/lib/python3.9/site-packages/youtube_dl/extractor/youtube.py", line 1344, in
return lambda s: initial_function([s])
File "/Library/Frameworks/Python.framework/Versions/3.9/lib/python3.9/site-packages/youtube_dl/jsinterp.py", line 258, in resf
res, abort = self.interpret_statement(stmt, local_vars)
File "/Library/Frameworks/Python.framework/Versions/3.9/lib/python3.9/site-packages/youtube_dl/jsinterp.py", line 56, in interpret_statement
v = self.interpret_expression(expr, local_vars, allow_recursion)
File "/Library/Frameworks/Python.framework/Versions/3.9/lib/python3.9/site-packages/youtube_dl/jsinterp.py", line 182, in interpret_expression
return objmember
KeyError: 'QV'
(caused by KeyError('QV')); please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; see  https://yt-dl.org/update  on how to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.
Traceback (most recent call last):
File "/Library/Frameworks/Python.framework/Versions/3.9/lib/python3.9/site-packages/youtube_dl/extractor/youtube.py", line 1360, in _decrypt_signature
func = self._extract_signature_function(
File "/Library/Frameworks/Python.framework/Versions/3.9/lib/python3.9/site-packages/youtube_dl/extractor/youtube.py", line 1277, in _extract_signature_function
cache_res = res(test_string)
File "/Library/Frameworks/Python.framework/Versions/3.9/lib/python3.9/site-packages/youtube_dl/extractor/youtube.py", line 1344, in
return lambda s: initial_function([s])
File "/Library/Frameworks/Python.framework/Versions/3.9/lib/python3.9/site-packages/youtube_dl/jsinterp.py", line 258, in resf
res, abort = self.interpret_statement(stmt, local_vars)
File "/Library/Frameworks/Python.framework/Versions/3.9/lib/python3.9/site-packages/youtube_dl/jsinterp.py", line 56, in interpret_statement
v = self.interpret_expression(expr, local_vars, allow_recursion)
File "/Library/Frameworks/Python.framework/Versions/3.9/lib/python3.9/site-packages/youtube_dl/jsinterp.py", line 182, in interpret_expression
return objmember
KeyError: 'QV'
Traceback (most recent call last):
File "/Library/Frameworks/Python.framework/Versions/3.9/lib/python3.9/site-packages/youtube_dl/extractor/youtube.py", line 1360, in _decrypt_signature
func = self._extract_signature_function(
File "/Library/Frameworks/Python.framework/Versions/3.9/lib/python3.9/site-packages/youtube_dl/extractor/youtube.py", line 1277, in _extract_signature_function
cache_res = res(test_string)
File "/Library/Frameworks/Python.framework/Versions/3.9/lib/python3.9/site-packages/youtube_dl/extractor/youtube.py", line 1344, in
return lambda s: initial_function([s])
File "/Library/Frameworks/Python.framework/Versions/3.9/lib/python3.9/site-packages/youtube_dl/jsinterp.py", line 258, in resf
res, abort = self.interpret_statement(stmt, local_vars)
File "/Library/Frameworks/Python.framework/Versions/3.9/lib/python3.9/site-packages/youtube_dl/jsinterp.py", line 56, in interpret_statement
v = self.interpret_expression(expr, local_vars, allow_recursion)
File "/Library/Frameworks/Python.framework/Versions/3.9/lib/python3.9/site-packages/youtube_dl/jsinterp.py", line 182, in interpret_expression
return objmember
KeyError: 'QV'
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "/Library/Frameworks/Python.framework/Versions/3.9/lib/python3.9/site-packages/youtube_dl/YoutubeDL.py", line 815, in wrapper
return func(self, *args, **kwargs)
File "/Library/Frameworks/Python.framework/Versions/3.9/lib/python3.9/site-packages/youtube_dl/YoutubeDL.py", line 836, in __extract_info
ie_result = ie.extract(url)
File "/Library/Frameworks/Python.framework/Versions/3.9/lib/python3.9/site-packages/youtube_dl/extractor/common.py", line 534, in extract
ie_result = self._real_extract(url)
File "/Library/Frameworks/Python.framework/Versions/3.9/lib/python3.9/site-packages/youtube_dl/extractor/youtube.py", line 1641, in _real_extract
signature = self._decrypt_signature(sc['s'][0], video_id, player_url)
File "/Library/Frameworks/Python.framework/Versions/3.9/lib/python3.9/site-packages/youtube_dl/extractor/youtube.py", line 1370, in _decrypt_signature
raise ExtractorError(
youtube_dl.utils.ExtractorError: Signature extraction failed: Traceback (most recent call last):
File "/Library/Frameworks/Python.framework/Versions/3.9/lib/python3.9/site-packages/youtube_dl/extractor/youtube.py", line 1360, in _decrypt_signature
func = self._extract_signature_function(
File "/Library/Frameworks/Python.framework/Versions/3.9/lib/python3.9/site-packages/youtube_dl/extractor/youtube.py", line 1277, in _extract_signature_function
cache_res = res(test_string)
File "/Library/Frameworks/Python.framework/Versions/3.9/lib/python3.9/site-packages/youtube_dl/extractor/youtube.py", line 1344, in
return lambda s: initial_function([s])
File "/Library/Frameworks/Python.framework/Versions/3.9/lib/python3.9/site-packages/youtube_dl/jsinterp.py", line 258, in resf
res, abort = self.interpret_statement(stmt, local_vars)
File "/Library/Frameworks/Python.framework/Versions/3.9/lib/python3.9/site-packages/youtube_dl/jsinterp.py", line 56, in interpret_statement
v = self.interpret_expression(expr, local_vars, allow_recursion)
File "/Library/Frameworks/Python.framework/Versions/3.9/lib/python3.9/site-packages/youtube_dl/jsinterp.py", line 182, in interpret_expression
return objmember
KeyError: 'QV'
(caused by KeyError('QV')); please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; see  https://yt-dl.org/update  on how to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.

I tried for different version of youtube_dl but the error still persists

#30839 !


Random mirror # doesn't compile with g++

When compiling with
g++ --std=c++11 *.cpp -o Fission

I get
Mirror.cpp: In member function ‘virtual bool fsn::Mirror::onHit(fsn::Atom&)’:
Mirror.cpp:65:38: error: ‘arc4random’ was not declared in this scope
    atom.dir = (atom.dir + arc4random() % 3 + 3) & 3;
                                      ^

Would you mind using a more platform independent PRNG there?

I actually didn't think anyone was using Fission so I never bothered completing my changes. Since I now know that at least one person is using it, I'll go back and finish making changes (and fix this bug you reported, thanks!).

On May 1, 2015, at 5:08 AM, Martin Büttner<EMAIL_ADDRESS>wrote:
When compiling with
g++ --std=c++11 *.cpp -o Fission
I get
Mirror.cpp: In member function ‘virtual bool fsn::Mirror::onHit(fsn::Atom&)’:
Mirror.cpp:65:38: error: ‘arc4random’ was not declared in this scope
atom.dir = (atom.dir + arc4random() % 3 + 3) & 3;
^
Would you mind using a more platform independent PRNG there?
—
Reply to this email directly or view it on GitHub.

It's a fun challenge to program in this. I wrote atoi and itoa if you're interested:
     ;
/  Y@<+@0'L/  [Y]_L
R? % \  @  /  $J
  ;A          /[{+++++++++L
   _          ; ;
   UR+++++++++>/
              O

and
     /"0"/    \
R'x[@%  //    /
     \$@{   / \
   @    \     <\
   $       ;  ;}+++++++++L
   %@       A {/
   M     \@<+@/
   ~       @
  ;Z       K Y 0'L
         \  !A   /

Is there a more convenient way to get an atom with mass 10 than R+++++++++?

I have had a massive set of local changes for a long time that I never pushed as they weren't finished. They somewhat change the way the language works so I was hesitant to do so. I think that I'll push those in a new branch while I finish working on them and let people see what they think of the new Fission. It modifies some components and adds others.

Just fixed this locally, will be in the next group of changes.

Let me know when you've pushed that new branch. I'm currently working on a larger program, so I guess that would break a lot (but maybe simplify some things).

That is the goal, yes. Not much changes, but there are a few usability changes. I'm open to suggestions as well.

On May 25, 2015, at 4:09 AM, Martin Büttner<EMAIL_ADDRESS>wrote:
Let me know when you've pushed that new branch. I'm currently working on a larger program, so I guess that would break a lot (but maybe simplify some things).
—
Reply to this email directly or view it on GitHub.

Somewhat off topic for this discussion, but I wanted to show you the larger program I mentioned: http://codegolf.stackexchange.com/a/50858/8478

Wow that's amazing! I'll definitely spend some time figuring out how it works later. Seems like a pretty good test case since it (ab)uses almost every component in the language!

On May 27, 2015, at 9:13 AM, Martin Büttner<EMAIL_ADDRESS>wrote:
Somewhat off topic for this discussion, but I wanted to show you the larger program I mentioned: http://codegolf.stackexchange.com/a/50858/8478
—
Reply to this email directly or view it on GitHub.

I've expanded that post with a very extensive explanation that comes close to a Fission tutorial. ;)
I also found a 6-byte quine: http://codegolf.stackexchange.com/a/50968/8478
I'm really looking forward to the new version you mentioned. I've got a bunch of ideas to improve Fission, but I'd like to see what you had in mind before proposing them.

FYI, there are now at least two people using Fission. ;)

I noticed! I have some ideas for modifications to Fission, but I'd like to run them by you first for feedback. If you're on Freenode, I created #Fission-lang (#Fission is already registered, but nobody is in the channel :/)

I haven't used IRC in forever, but I found a web client. I'd definitely prefer gitter or just SE's chat though. ;)

Well I tried gitter. I can say that at the very least their iOS client is a piece of crap as it doesn't let you do anything besides join one of the handful of rooms listed by default and your organizations. I see no way to create a room or even leave a room you've joined. The idea of it is cool but the implementation is way off.
Also on SE's iOS client I can't figure out how to use chat :/.

On Jun 1, 2015, at 3:16 AM, Martin Büttner<EMAIL_ADDRESS>wrote:
I haven't used IRC in forever, but I found a web client. I'd definitely prefer gitter or just SE's chat though. ;)
—
Reply to this email directly or view it on GitHub.

Ah yeah, SE's mobile apps don't support chat (yet?). There's a separate
(unofficial) app ChatSEy though, which is alright. But I'm okay with
Freenode if the other options are a hassle for you.
On Mon, Jun 1, 2015 at 6:12 PM, C0deH4cker<EMAIL_ADDRESS>wrote:

Well I tried gitter. I can say that at the very least their iOS client is
a piece of crap as it doesn't let you do anything besides join one of the
handful of rooms listed by default and your organizations. I see no way to
create a room or even leave a room you've joined. The idea of it is cool
but the implementation is way off.
Also on SE's iOS client I can't figure out how to use chat :/.

On Jun 1, 2015, at 3:16 AM, Martin Büttner<EMAIL_ADDRESS>wrote:
I haven't used IRC in forever, but I found a web client. I'd definitely
prefer gitter or just SE's chat though. ;)
—
Reply to this email directly or view it on GitHub.

—
Reply to this email directly or view it on GitHub
https://github.com/C0deH4cker/Fission/issues/1#issuecomment-107642727.

Apparently ChatSEy isn't an app for iOS. If you have a better option, feel free to suggest it. For now I think IRC is the most manageable option at least when I'm not at my computer. I still have about 6 hours until I get back from work, so I'll be on then.

On Jun 1, 2015, at 10:14 AM, Martin Büttner<EMAIL_ADDRESS>wrote:
Ah yeah, SE's mobile apps don't support chat (yet?). There's a separate
(unofficial) app ChatSEy though, which is alright. But I'm okay with
Freenode if the other options are a hassle for you.
On Mon, Jun 1, 2015 at 6:12 PM, C0deH4cker<EMAIL_ADDRESS>wrote:

Well I tried gitter. I can say that at the very least their iOS client is
a piece of crap as it doesn't let you do anything besides join one of the
handful of rooms listed by default and your organizations. I see no way to
create a room or even leave a room you've joined. The idea of it is cool
but the implementation is way off.
Also on SE's iOS client I can't figure out how to use chat :/.

On Jun 1, 2015, at 3:16 AM, Martin Büttner<EMAIL_ADDRESS>wrote:
I haven't used IRC in forever, but I found a web client. I'd definitely
prefer gitter or just SE's chat though. ;)
—
Reply to this email directly or view it on GitHub.

—
Reply to this email directly or view it on GitHub
https://github.com/C0deH4cker/Fission/issues/1#issuecomment-107642727.

—
Reply to this email directly or view it on GitHub.

No problem. I will likely be asleep by then, so just dump your ideas in IRC
and I'll have a read tomorrow morning.
On Mon, Jun 1, 2015 at 8:54 PM, C0deH4cker<EMAIL_ADDRESS>wrote:

Apparently ChatSEy isn't an app for iOS. If you have a better option, feel
free to suggest it. For now I think IRC is the most manageable option at
least when I'm not at my computer. I still have about 6 hours until I get
back from work, so I'll be on then.

On Jun 1, 2015, at 10:14 AM, Martin Büttner<EMAIL_ADDRESS>wrote:
Ah yeah, SE's mobile apps don't support chat (yet?). There's a separate
(unofficial) app ChatSEy though, which is alright. But I'm okay with
Freenode if the other options are a hassle for you.
On Mon, Jun 1, 2015 at 6:12 PM, C0deH4cker<EMAIL_ADDRESS>wrote:

Well I tried gitter. I can say that at the very least their iOS client
is
a piece of crap as it doesn't let you do anything besides join one of
the
handful of rooms listed by default and your organizations. I see no
way to
create a room or even leave a room you've joined. The idea of it is
cool
but the implementation is way off.
Also on SE's iOS client I can't figure out how to use chat :/.

On Jun 1, 2015, at 3:16 AM, Martin Büttner<EMAIL_ADDRESS>

wrote:

I haven't used IRC in forever, but I found a web client. I'd
definitely
prefer gitter or just SE's chat though. ;)
—
Reply to this email directly or view it on GitHub.

—
Reply to this email directly or view it on GitHub
<https://github.com/C0deH4cker/Fission/issues/1#issuecomment-107642727
.

—
Reply to this email directly or view it on GitHub.

—
Reply to this email directly or view it on GitHub
https://github.com/C0deH4cker/Fission/issues/1#issuecomment-107685581.

Looks like this isn't really working out. I've edited my GitHub profile to show my email address, so feel free to just email me directly... asynchronous communication works better across our timezones.

Fixed in d019e2265e006f74de80cd9e6e39d0950fd2349a


key features section styling

Fix #73
Test URLs:

Before: https://main--piramal--aemsites.hlx.page/home-loans
After: https://issue-73--piramal--aemsites.hlx.page/home-loans

@asthabh23  Here, it would be better if the 'Click me' text would come as a  or italic or some other tag so it would be easier to change just the tag to  instead of search by text. I would edit this myself but UE keeps crashing on the home loans page

yes it has an on-going issue, we cant update it atm


Add forking / branch workflow image

What's the extra arrow in main?

Shouldn't be there -- didn't update that when I changed something else. good call. updating on next commit

in terms of staging, those would be individual new branches from main or dev, right? I think the continued line is a bit confusing.

yeah, removed that continued line. Effectively its just a branch to literally stage either the feature into dev or dev into main (for a release). So the diagram is drawn in such a way to notate that there should be no commits to those staging branches.  @JGSweets @micdavis @ksneab7 @tyfarnan

related to #857


Hydrogen atoms PP without beta projectors

SIRIUS will fail if we run H2 molecules with PP that have no beta projectors. The symmetrization procedure will crash in that case, so we do not symmetrize the density matrix (which is not defined in that case). I am not sure this is the right way to fix this corner case and using properly defined PP is probably better. This problem only occurs when the system only contains hydrogen atoms.

bors try

I did it here in a slightly different way
https://github.com/electronic-structure/SIRIUS/pull/624/commits/d1859b5c20bbb4c49c07b80e470a368307522861
The hack std::max(1, unit_cell_.max_mt_basis_size()) is inconvenient

I think this is now handled by the latest big PR

Ok I can close it then


[Bug] [API] There're a few issues about running test cases in the class of AlertGroupControllerTest

Search before asking

[X] I had searched in the issues and found no similar issues.

What happened
When I run the test cases of AlertGroupControllerTest , it failed at the test case of test010CreateAlertgroup and test060VerifyGroupName. It's because that these test cases didn't run orderly. So as running the test case of test010CreateAlertgroup , maybe the other test cases have been executed before.
More details are as follows:

What you expected to happen
I expect that these test cases can be executed successfully.
How to reproduce
You can run the whole test cases of AlertGroupControllerTest.
Anything else
No response
Version
dev
Are you willing to submit PR?

[X] Yes I am willing to submit a PR!

Code of Conduct

[X] I agree to follow this project's Code of Conduct

I will fix this issue.


Can not sign in from web sdk sample using skype account

Hello,
I deploy the web-sdk-sample at my local win10 following this instruction. And I could open the page from browser and it looked good.
And I found it possible only support skype for business account becase I tried my person skype account to signing in but failed. Following is the error in browser developer console.

Is this sdk only for skype for business account?

😄  No body reply yet.
Is this sdk only for skype for business account?
I tried my personal account<EMAIL_ADDRESS>then sdk tried to visit a specify url lyncxxxx.163.com and connection failed.  After that I tried another business account, it still failed but the returned error changed.
About the url in the previous post screenshot.
From Develop for skype for business server

The two URLs are also known as root URLs. One of the two servers is supposed to exist and have a valid certificate. If this is not possible, but you know or can discover by other means the root URLs, they can be given in the origins parameter:

Problems

Insecure connection
I start the sample over http server and failed through sign in with the error. Then I have to created a https server.  But before that, I need a ssl key and certificate. Following command could generate them.

# You don't need to input any letter, just enter, enter, ...
$ openssl req  -nodes -new -x509  -keyout server.key -out server.cert

And the nodejs https server
var express = require('express');
var https = require('https');
var http = require('http');
var path = require('path')
var fs = require('fs');

// This line is from the Node.js HTTPS documentation.
var options = {
  key: fs.readFileSync('./keys/server.key'),
  cert: fs.readFileSync('./keys/server.cert')
};

// Create a service (the app object is just a callback).
var app = express();

app.use(express.static('.'));

// Create an HTTP service.
http.createServer(app).listen(8888);
// Create an HTTPS service identical to the HTTP service.
https.createServer(options, app).listen(8880);

Disable same origin policy
I also found error by samle origin policy in Chrome. I didn't know if this could help about sign-in but less error was not bad.
http://www.thegeekstuff.com/2016/09/disable-same-origin-policy/comment-page-1/
🚲


Component renames are not working on Travis

This is probably something to do with the order of self update, update, etc.,. but I can't figure out exactly what.
e.g., https://travis-ci.org/rust-lang/www.rust-lang.org/builds/466170909?utm_source=github_status&utm_medium=notification

I believe this was a component rename related issue so I'm going to close it now.

(specifically that nightly had done the rename wrongly)


[BUG] all chatflows are deleted on receiving some HTML formatted message.

Describe the bug
all my chatflows gets deleted and sometimes even reset the instance.
Last part of that html  from huggingface states "Sorry, there is an error on our side."
not sure if this is an error from huggingface or flowise.
Perhaps it happens when that received message gets saved to the chatMessages array and then it gets corrupted because the html from huggingface isn't the right format.
To Reproduce
Steps to reproduce the behavior:
import chatflow.
add API key for replicate
send message
Expected behavior
A clear and concise description of what you expected to happen.
chat should return a message or error message.
Screenshots
If applicable, add screenshots to help explain your problem.

Flow
If applicable, add exported flow in order to help replicating the problem.
Image Creator! Chatflow.json
Setup

Installation [e.g. docker, npx flowise start, yarn start]
Flowise Version [e.g. 1.2.11]
OS: [e.g. macOS, Windows, Linux]
Browser [e.g. chrome, safari]

I use huggingface free tier to host it. using docker
<EMAIL_ADDRESS>Additional context
Add any other context about the problem here.

related #1094


chore: remove Travis configuration

CC @lesv

The failure was due to kokoro being unable to get artifacts from maven.  Admin merged.


Brightcove empty playlist

./youtube-dl --verbose http://aptn.ca/blackstone/video/season-1/
[debug] System config: []
[debug] User config: []
[debug] Command-line args: [u'--verbose', u'http://aptn.ca/blackstone/video/season-1/']
[debug] Encodings: locale US-ASCII, fs US-ASCII, out US-ASCII, pref US-ASCII
[debug] youtube-dl version 2015.06.15
[debug] Python version 2.7.9 - FreeBSD-8.2-STABLE-amd64-64bit-ELF
[debug] exe versions: ffmpeg 0.11.3, ffprobe 0.11.3
[debug] Proxy map: {}
[generic] season-1: Requesting header
WARNING: Falling back on generic information extractor.
[generic] season-1: Downloading webpage
[generic] season-1: Extracting information
[generic] Brightcove video detected.
[download] Downloading playlist: Blackstone  |  SEASON 1
[generic] playlist Blackstone  |  SEASON 1: Collected 1 video ids (downloading 1 of them)
[download] Downloading video 1 of 1
[Brightcove] AQ~~,AAAAEQEb72E~,MYPTzvrWinzoYROX3AWqzRir9E-Up5Qk: Downloading playlist information
ERROR: Empty playlist; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; type  youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.
Traceback (most recent call last):
File "./youtube-dl/youtube_dl/YoutubeDL.py", line 650, in extract_info
ie_result = ie.extract(url)
File "./youtube-dl/youtube_dl/extractor/common.py", line 273, in extract
return self._real_extract(url)
File "./youtube-dl/youtube_dl/extractor/brightcove.py", line 239, in _real_extract
return self._get_playlist_info(player_key[0])
File "./youtube-dl/youtube_dl/extractor/brightcove.py", line 277, in _get_playlist_info
raise ExtractorError('Empty playlist')
ExtractorError: Empty playlist; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; type  youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.

Can you watch this video in browsers? I got neither videos nor error messages in Chrome.

The link works for me in a browser, but I get the same YouTube-DL error...
I posted (what I believe is) a similar issue here: https://github.com/rg3/youtube-dl/issues/5955
...maybe you can see that video?

Yes I can see the video in #5955. I've investigated it while I didn't find a solution. I'm not sure whether the two videos have the same problem in youtube-dl or not, either.


How can i add ranges on initializing the multirangeslider?

If I add the ranges after the init (with add) the onAdd- and onChange-event is triggered. This is a problem if i want to show the slider with pre-existing ranges and use the onAdd event if the user wants to add another range.
Thanks for checking this...

Probably the best solution would be to add the pre-existing ranges to the options

Probably the best solution would be to add the pre-existing ranges to the options

hello, i cant init the pre-existing range, how do you do it? thanks


Multiple Notifiable models

Hello,
thank you for this great package, I'm having a lot of fun in my project using it.
My project has teams and users belonging to teams. Both the Team model and User model can be Notifiable.
I'd like to know if there is a way to display at the same time notifications for the current user and for the team the current user belongs to.
I've seen that a notifiableId parameter can be passed to the megaphone component but it's either the user or the team. Plus that would need to adjust the model property in the config file.
What would be the strategy to achieve the described behavior? Thank you for your input on this matter!

I like this idea and can definitely see the uses for it. I'll have a think on the best way to achieve this and may even roll it out in the next release if i can come up with a sensible solution.


test

test the PR validation build

Pull Request Test Coverage Report for Build 362561532

0 of 0   changed or added relevant lines in 0 files are covered.
4 unchanged lines in 1 file lost coverage.
Overall coverage decreased (-0.01%) to 41.186%

Files with Coverage Reduction
New Missed Lines
%

extensions/notebook/src/jupyter/serverInstance.ts
4
73.81%

Totals

Change from base Build 362557690:
-0.01%

Covered Lines:
21752

Relevant Lines:
48475

💛  - Coveralls

@alanrenmsft are you still using this PR for testing?  or should we close it now?


Delay focus to Text Input in Search Page

Details

Fixed Issues

$ https://github.com/Expensify/App/issues/15276
Tests

[ ] Verify that no errors appear in the JS console

Offline tests

QA Steps

[ ] Verify that no errors appear in the JS console

PR Author Checklist

[ ] I linked the correct issue in the ### Fixed Issues section above
[ ] I wrote clear testing steps that cover the changes made in this PR

[ ] I added steps for local testing in the Tests section
[ ] I added steps for the expected offline behavior in the Offline steps section
[ ] I added steps for Staging and/or Production testing in the QA steps section
[ ] I added steps to cover failure scenarios (i.e. verify an input displays the correct error message if the entered data is not correct)
[ ] I turned off my network connection and tested it while offline to ensure it matches the expected behavior (i.e. verify the default avatar icon is displayed if app is offline)
[ ] I tested this PR with a High Traffic account against the staging or production API to ensure there are no regressions (e.g. long loading states that impact usability).

[ ] I included screenshots or videos for tests on all platforms
[ ] I ran the tests on all platforms & verified they passed on:

[ ] Android / native
[ ] Android / Chrome
[ ] iOS / native
[ ] iOS / Safari
[ ] MacOS / Chrome / Safari
[ ] MacOS / Desktop

[ ] I verified there are no console errors (if there's a console error not related to the PR, report it or open an issue for it to be fixed)
[ ] I followed proper code patterns (see Reviewing the code)

[ ] I verified that any callback methods that were added or modified are named for what the method does and never what callback they handle (i.e. toggleReport and not onIconClick)
[ ] I verified that comments were added to code that is not self explanatory
[ ] I verified that any new or modified comments were clear, correct English, and explained "why" the code was doing something instead of only explaining "what" the code was doing.
[ ] I verified any copy / text shown in the product is localized by adding it to src/languages/* files and using the translation method

[ ] If any non-english text was added/modified, I verified the translation was requested/reviewed in #expensify-open-source and it was approved by an internal Expensify engineer. Link to Slack message:

[ ] I verified all numbers, amounts, dates and phone numbers shown in the product are using the localization methods
[ ] I verified any copy / text that was added to the app is correct English and approved by marketing by adding the Waiting for Copy label for a copy review on the original GH to get the correct copy.
[ ] I verified proper file naming conventions were followed for any new files or renamed files. All non-platform specific files are named after what they export and are not named "index.js". All platform-specific files are named for the platform the code supports as outlined in the README.
[ ] I verified the JSDocs style guidelines (in STYLE.md) were followed

[ ] If a new code pattern is added I verified it was agreed to be used by multiple Expensify engineers
[ ] I followed the guidelines as stated in the Review Guidelines
[ ] I tested other components that can be impacted by my changes (i.e. if the PR modifies a shared library or component like Avatar, I verified the components using Avatar are working as expected)
[ ] I verified all code is DRY (the PR doesn't include any logic written more than once, with the exception of tests)
[ ] I verified any variables that can be defined as constants (ie. in CONST.js or at the top of the file that uses the constant) are defined as such
[ ] I verified that if a function's arguments changed that all usages have also been updated correctly
[ ] If a new component is created I verified that:

[ ] A similar component doesn't exist in the codebase
[ ] All props are defined accurately and each prop has a /** comment above it */
[ ] The file is named correctly
[ ] The component has a clear name that is non-ambiguous and the purpose of the component can be inferred from the name alone
[ ] The only data being stored in the state is data necessary for rendering and nothing else
[ ] For Class Components, any internal methods passed to components event handlers are bound to this properly so there are no scoping issues (i.e. for onClick={this.submit} the method this.submit should be bound to this in the constructor)
[ ] Any internal methods bound to this are necessary to be bound (i.e. avoid this.submit = this.submit.bind(this); if this.submit is never passed to a component event handler like onClick)
[ ] All JSX used for rendering exists in the render method
[ ] The component has the minimum amount of code necessary for its purpose, and it is broken down into smaller components in order to separate concerns and functions

[ ] If any new file was added I verified that:

[ ] The file has a description of what it does and/or why is needed at the top of the file if the code is not self explanatory

[ ] If a new CSS style is added I verified that:

[ ] A similar style doesn't already exist
[ ] The style can't be created with an existing StyleUtils function (i.e. StyleUtils.getBackgroundAndBorderStyle(themeColors.componentBG)

[ ] If the PR modifies a generic component, I tested and verified that those changes do not break usages of that component in the rest of the App (i.e. if a shared library or component like Avatar is modified, I verified that Avatar is working as expected in all cases)
[ ] If the PR modifies a component related to any of the existing Storybook stories, I tested and verified all stories for that component are still working as expected.
[ ] If a new page is added, I verified it's using the ScrollView component to make it scrollable when more elements are added to the page.
[ ] If the main branch was merged into this PR after a review, I tested again and verified the outcome was still expected according to the Test steps.
[ ] I have checked off every checkbox in the PR author checklist, including those that don't apply to this PR.

Screenshots/Videos

Web

Mobile Web - Chrome

Mobile Web - Safari

Desktop

iOS

Android

The solution I can think of now for the original issue is to delay putting focus on the text field in Search Page a little bit so that the blur focus event on Global Creation Menu takes place before putting focus on the text field.
Ideally, we want to only delay for the above situation. We don't want to delay putting focus for other situations (ex. when navigating to Search Page by clicking the search icon).
However, I couldn't find any way to figure out what triggered navigation to Search Page. The Search Page cannot check whether or not Global Creation Menu was open when the navigation took place.
So the only solutions I can think of is to delay putting focus in the input box in Search Page for all situations.
If anyone has workaround for this, I would appreciate your suggestions.

I marked this PR as "Ready for Review" so that PullerBear can assign reviewers. I'll complete PR Author Checklist after I receive feedback to the above comment and make appropriate modifications.

@sobitneupane @aldo-expensify One of you needs to copy/paste the Reviewer Checklist from here into a new comment on this PR and complete it. If you have the K2 extension, you can simply click: [this button]

@aldo-expensify could you give feedback on my comment?

This does not need to be reviewed @sobitneupane!

Closing this PR as I marked the original issue external to seek proposals as per this comment.


Support vertical layout

I've transposed the form so that the table's columns are now arranged vertically: https://dev.isrd.isi.edu/~jessie/chaise/data-entry/#1/legacy:dataset
@robes In the context of #334, does this issue also require adding in the logic for transposing between vertical and horizontal views?

On 5/9/16, 7:13 PM, "Jessie Wong" <notifications<EMAIL_ADDRESS>wrote:
@robeshttps://github.com/robes In the context of #334https://github.com/informatics-isi-edu/chaise/issues/334, does this issue also require adding in the logic for transposing between vertical and horizontal views?
No. Not in the context of #334.

Okay I assigned to @robes for review but if I could assign multiple people I'd also add @hongsudt and @jrchudy.
https://dev.isrd.isi.edu/~jessie/chaise/data-entry/#1/legacy:dataset

I would suggest a couple aesthetic changes:

not using a visible table style
reduce the spacing between the left column of labels and the right columns of data entry controls
and/or consider right alignment of the left column of labels

+1 or -1 by @jrchudy and @hongsudt ?

I think the spacing in the left column should be reduced. But I'm thinking that's being done automatically to spread the table across the width of the display. Other than that, it looks good. I'm not sure how right alignment for the labels would look.

Yeah.. I tried changing the browser size and the spacing is adjusted automatically.  Left-justified is ok and is consistent with the record app.

I assume the sharp sign (#) is the catalog. Can we not show that?
There should a table name (e.g. entity name) displayed e.g. Protein (new entry)
The header row should be attributes and values
I don't like displaying the raw type either.. Maybe we can use some small symbol to indicate number, text, boolean?
I assume the drop-down items are foreign keys and I should ignore them for now.

Okay what I'm hearing is:

Keep the visible tables but restrict the width of the table and table's column widths so that it looks more like Record app.
Remove the # sign. (It was to indicate that that row is a row of numbers, but I can see how that's ambiguous.)

@hongsudt:

The table name is currently displayed as "New [table name]" in the upper left corner.
What do you mean by "The header row should be attributes and values?"
Yes the dropdowns are foreign keys.

I believe @hongsudt is suggesting that # and the subsequent column numbers for each row entry (1, 2, 3, ....) should be Attributes and Values.

Actually, shouldn't the data entry vertical form look almost exactly like the record app? I think it should. That would mean, use the same table style, same page alignment, use the same fonts, font colors, etc. These should be as uniform as possible.

I would just get rid of that first row with the # and 1 in it. I understand its for the multi-row editing but it might even be unnecessary in that context. Its enough to just see the multiple rows (or columns in the vertical layout).

Okay here's a little more Record-like form: https://dev.isrd.isi.edu/~jessie/chaise/data-entry/#1/legacy:dataset
Any other improvements/feedback?

What should the sizing be of the first column.  When you add the rows (which are actually columns), they are a different width from the original.
Carl

Dr. Carl Kesselman
Dean’s Professor, Epstein Department of Industrial and Systems Engineering
Fellow, Information Sciences Institute
Viterbi School of Engineering
Professor, Preventive Medicine
Keck School of Medicine
University of Southern California
4676 Admiralty Way, Suite 1001, Marina del Rey, CA  90292-6695
Phone:  +1<PHONE_NUMBER>
Email:  carl<EMAIL_ADDRESS>Web: http://www.isi.edu/~carl
On May 10, 2016, at 4:43 PM, Jessie Wong <notifications<EMAIL_ADDRESS>wrote:
Okay here's a little more Record-like form: https://dev.isrd.isi.edu/~jessie/chaise/data-entry/#1/legacy:datasethttps://dev.isrd.isi.edu/~jessie/chaise/data-entry/#1/legacy:dataset
Any other improvements/feedback?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHubhttps://github.com/informatics-isi-edu/chaise/issues/332#issuecomment-218324243

@howdyjessie : this works for me. but can you remove the "Add a Record" button as that does not belong in the minimal data entry form. we should address carl's comment in the next round when we deal with the multi-record entry.

@robes Sure thing, the Add a Record button has been removed.
@carlkesselman I'll make a note to add same-width columns for later!
https://dev.isrd.isi.edu/~jessie/chaise/data-entry/#1/legacy:dataset

@robes, @howdyjessie I found alternating the row background between different rows useful though and would suggest that as a improvement.. But we can fine tune this later.


Is there a set of dark mode icons ?

[Enter feedback here]
I couldn't find a dark mode set of icons on the official page, link only shows one set. Is there a specific link?

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

ID: 6624d44c-f9f6-02f8-2c67-0f0637fed62e
Version Independent ID: 9346a8ff-bab5-be2c-9dc1-d546fda1efa9
Content: Azure Icons - Azure Architecture Center
Content Source: docs/icons/index.md
Service: architecture-center
GitHub Login: @doodlemania2
Microsoft Alias: pnp

Thanks for reporting - this issue is under review. This is a Microsoft Internal DevOps Bug ID # 427577

@david-stanford Thank you for confirming and providing this information. @nikolaarnaudov Thank you for the feedback. Once the bug is reviewed and action is taken the issue will be resolved.

There is not currently, I believe the icons should have transparent backgrounds, but there is not a dedicated set for darkmode.


License

The lack of a license probably means a good few people are shying away from using the examples provided. Thoughts on this?

@devoxel good point, thanks. Added license with https://github.com/AllThingsSmitty/jquery-tips-everyone-should-know/commit/456d5328e8dfcb1a18eabb1823ef39dd0ed8675f.


Better Library handling in the config

im probably to dumb to know what im doing wrong xd
i just like wanna do "libs/*" to add all librarys from a folder cuz i have like 14

Hi,
Please open a ticket on our Discord for further help.
Sincerly, Louis.

Just specify the argument li=<folder_with_the_libs>

https://discord.gg/QJC9g8fBU9 << the discord in case you are not in already

Just specify the argument li=<folder_with_the_libs>

yea i figured it out

I would recommend joining the discord still even if you figured it out. You will get fast info on updates and ability to buy the premium version which is much better.


Headers already sent

Hello!
My php error log is filling up every second with a 'Headers already sent' error. Do you think you can look into this? Is it something I've done? This error occurs on any webpage accessing the internal assets. The error also injects itself at the bottom of internal xml documents (though the extension is labelled .kcd).
An Error occurred while handling another error:
yii\web\HeadersAlreadySentException: Headers already sent in /Users/johnathonsouthworth/Documents/Projects/playcraft/playcraft-craft/vendor/saccilottoconsulting/craft-internal-assets/src/controllers/AccessController.php on line 134. in /Users/johnathonsouthworth/Documents/Projects/playcraft/playcraft-craft/vendor/yiisoft/yii2/web/Response.php:368
Stack trace:
#0 /Users/johnathonsouthworth/Documents/Projects/playcraft/playcraft-craft/vendor/yiisoft/yii2/web/Response.php(341): yii\web\Response->sendHeaders()
#1 /Users/johnathonsouthworth/Documents/Projects/playcraft/playcraft-craft/vendor/yiisoft/yii2/web/ErrorHandler.php(136): yii\web\Response->send()
#2 /Users/johnathonsouthworth/Documents/Projects/playcraft/playcraft-craft/vendor/craftcms/cms/src/web/ErrorHandler.php(192): yii\web\ErrorHandler->renderException(Object(yii\web\HeadersAlreadySentException))
#3 /Users/johnathonsouthworth/Documents/Projects/playcraft/playcraft-craft/vendor/yiisoft/yii2/base/ErrorHandler.php(152): craft\web\ErrorHandler->renderException(Object(yii\web\HeadersAlreadySentException))
#4 /Users/johnathonsouthworth/Documents/Projects/playcraft/playcraft-craft/vendor/craftcms/cms/src/web/ErrorHandler.php(66): yii\base\ErrorHandler->handleException(Object(yii\web\HeadersAlreadySentException))
#5 [internal function]: craft\web\ErrorHandler->handleException(Object(yii\web\HeadersAlreadySentException))
#6 {main}
Previous exception:
yii\web\HeadersAlreadySentException: Headers already sent in /Users/johnathonsouthworth/Documents/Projects/playcraft/playcraft-craft/vendor/saccilottoconsulting/craft-internal-assets/src/controllers/AccessController.php on line 134. in /Users/johnathonsouthworth/Documents/Projects/playcraft/playcraft-craft/vendor/yiisoft/yii2/web/Response.php:368

Hi @playcraftSystems,
Thank you for reporting this. Probably a problem with my use of the fpassthrough method to send the file content stream to the browser. I replaced the respective section with yii2's sendStreamAsFile method, as they use as well for the Craft admin panel, and it should hopefully work correctly now.
This should also add support for mp4 files on mobiles, since the new method also supports range headers, which are often employed on mobile devices to load only partial video content.
https://www.yiiframework.com/doc/api/2.0/yii-web-response#sendStreamAsFile()-detail
Please update your plugin installation to version 4.1.0 and check if it works then.

Unfortunately, the error is still occurring on any internal asset downloaded. This is happening in production as well. It wouldn't be so bad, but the injects itself to the bottom of any downloaded xml document making them useless to all my clients.

Unfortunately, the error is still occurring on any internal asset downloaded. This is happening in production as well. It wouldn't be so bad, but the injects itself to the bottom of any downloaded xml document making them useless to all my clients.

Unfortunately, the error is still occurring on any internal asset downloaded. This is happening in production as well. It wouldn't be so bad, but the injects itself to the bottom of any downloaded xml document making them useless to all my clients.

It works! Thanks.


[1.0] AXI Streaming test failure

*** SEQUENCE_START: axist_master_zero_delay_random_sequence
UVM_INFO ./eximius/1.0/llink/dv/axist/tests/ts.zero_delay_random_test.sv(124) @ 4333000: uvm_test_top [zero_delay_random_test] *** MAIN_PHASE_END: zero_delay_random_test
UVM_ERROR ./eximius/1.0/llink/dv/axist/env/axist_scoreboard.sv(856) @ 4366000: uvm_test_top.axist_scb [axist_scoreboard] Transmit tdata:: 59 Not Matched received tdata:: 00
UVM_ERROR ./eximius/1.0/llink/dv/axist/env/axist_scoreboard.sv(870) @ 4367000: uvm_test_top.axist_scb [axist_scoreboard] Transmit tdata:: 0d Not Matched received tdata:: 59
UVM_ERROR ./eximius/1.0/llink/dv/axist/env/axist_scoreboard.sv(870) @ 4367000: uvm_test_top.axist_scb [axist_scoreboard] Transmit tdata:: b7 Not Matched received tdata:: 0d
UVM_ERROR .rs/eximius/1.0/llink/dv/axist/env/axist_scoreboard.sv(870) @ 4368000: uvm_test_top.axist_scb [axist_scoreboard] Transmit tdata:: 84 Not Matched received tdata:: b7
UVM_ERROR ./eximius/1.0/llink/dv/axist/env/axist_scoreboard.sv(870) @ 4368000: uvm_test_top.axist_scb [axist_scoreboard] Transmit tdata:: 58 Not Matched received tdata:: 84
--- UVM Report catcher Summary ---
Number of demoted UVM_FATAL reports  :    0
Number of demoted UVM_ERROR reports  :    0
Number of demoted UVM_WARNING reports:    0
Number of caught UVM_FATAL reports   :    0
Number of caught UVM_ERROR reports   :    0
Number of caught UVM_WARNING reports :    0
--- UVM Report Summary ---
Quit count reached!
Quit count :     5 of     5
** Report counts by severity
UVM_INFO :   19
UVM_WARNING :    0
UVM_ERROR :    5
UVM_FATAL :    0
** Report counts by id
[AIB ENV CFG]     1
[MASTER CFG]     1
[MAXQUITSET]     1
[SLAVE CFG]     1
[axist_master_zero_delay_sequence]     1
[axist_scoreboard]     5
[new]     3
[zero_delay_random_test]    11
to reproduce:
./simv +UVM_TESTNAME=zero_delay_random_test +ntb_random_seed=51175275 +UVM_VERBOSITY=UVM_NONE +UVM_MAX_QUIT_COUNT=5 +AIB_IF_COUNT=1 +AIB_CONFIG_DIR=./ +AIB_MASTER_CONFIG_FILE=./master_0_config_aib.dat +AIB_SLAVE_CONFIG_FILE=./slave_0_config_aib.dat -l ./logs/sim_zero_delay_random_test_m2s2.log -cm line+cond+tgl+fsm+branch+assert +nospecify
sent VPD files over

Added a fix to llink_gen.py that should resolve this, but please confirm. The issue is only in AXI-ST with TVALID and without TREADY. It has no effect on LPIF or other AXI configurations. The fix is in a drop called design.release_1.0_2022_02_21.tgz that is the same as the 2/18 drop except for the the llink/script/lllink_gen.py script.

fix did not work uploaded 2 sets of files:

showing error in VPD (with_fix*)
showing compilation error (sb_error*)

This was primarily a TB issue as was explained to Nij. Nij asked I write that up here, so here is that explanation in detail.
The LLINK AXI-ST operates in 3 modes:

With TVALID and With TREADY (aka "normal mode")
With TVALID and no TREADY
With no TVALID and no TREADY (technically illegal per AXIST spec)

I directed the DV team to work on normal mode first, then the no TVALID, no TREADY case. Once those two were done, the With TVALID, and no TREADY case should be somewhere in the middle.
Normal mode is easy enough and plugs into the VIP nicely. However, the no TVALID, no TREADY case is tricky because the VIP requires TVALID on the receive side per AXIST spec, so the TB must "fake" a TVALID for the receive VIP / data. However, it is not immediately clear when to drive the TVALID on the RX side due to the unknown latency of the AIB, channel skew, etc and no VALID to indicate good data. So, since the AIB tends to default to all zeros in reset, in the TB we look for the first non-zero piece of data and assume that is the first beat of data transmitted and drive the receive TVALID and begin the data comparison. For this reason, it is important the first beat of transmitted data not be all 0s. Probabilistically, with 32 bit or even 16 bit TDATA, probabilistically, the odds of hitting all 0s with a random word is pretty low, but for 8 bit data, it is a 1 in 256 chance, so we've added code to exclude this possibility. To be clear, this caveat that the first beat of data be non-zero is only a TB construct and implies no restriction to the DUT itself.
The "with" TVALID with no TREADY case leveraged heavily the no TVALID and no TREADY case, and so it picked up the same requirement that the first beat of data should not be zero. The waves provided by Nij show the 8 bit TDATA case did indeed hit the all 0 case as the first beat, which is why the logs show the data was "off" by one beat of data and the first beat of all 0s was simply dropped. This was the mistake in the code, and it should be more like the "normal" case, where we use the TVALID transmitted to qualify the first beat of data. That is what was changed here, and once that was done, the first beat of data for the normal and "with" TVALID with no TREADY can be any value.
To be clear, we verified the above by forcing the first beat of data to be all 0s and we successfully received the data. We still cannot reproduce the exact same simulation with the same SEED. But I expect this to be fixed in 1.0.1.

fixed in 1.0.1


Remove alloc

Remove alloc to import String and ToString, replace by scale-info import this.
add check-no-std Makefile

Please execute "rebase main" before submitting.

Please execute "rebase main" and submit it again.

Done!


feat: distribute ZRC20 rewards function

Description

Create a function distribute() to distribute rewards across all validators and delegators.
Refactor staking precompile to be divided in a similar fashion as bank.
Unify common logic under the precompiles/types package.
Minor fixes in bank emitted logs.

How Has This Been Tested?

[ ] Tested CCTX in localnet
[ ] Tested in development environment
[x] Go unit tests
[x] Go integration tests
[x] Tested via GitHub Actions

Flow diagram
               0. allow to spend ZRC20
        ┌─────────────────────────────────┐
        │                                 │
        │                                 │        4. Mint Coins              5. Send Coins        After 1 block
  ┌─────┴─────┐                  ┌────────▼───────┐            ┌─────────────────┐     ┌─────────────┐      ┌────────────┐
  │           │                  │                │            │                 │     │             │      │            │
  │   EOA     ├─────────────────►│Staking contract├────────────► Fungible Module ├─────►Fee Collector├──────► Validators │
  │           │ 1. distribute()  │                │            │                 │     │             │      │            │
  └───────────┘                  └────────┬───────┘            └─────────────────┘     └─────────────┘      └────────────┘
                                          │
                                          │
                                          │ 2. LockZRC20()
                                          │
                                          │
                                 ┌────────▼───────┐           ┌───────────────┐
                                 │                │           │               │
                                 │ ZRC20 Lock API ├───────────► Locker (bank) │
                                 │                │           │               │
                                 └────────────────┘           └───────────────┘

                                         3. transferFrom(spender, locker)

Summary by CodeRabbit

New Features

Introduced support for stateful precompiled contracts.
Added authenticated calls for smart contracts.
Integrated Bitcoin inscription support on testnet.
Added a distribution functionality for staking contracts.
Enhanced staking contracts with new events and methods.

Bug Fixes

Resolved voting issues for discarded keygen ballots.
Improved error handling for invalid Solana gateway addresses.

Tests

Expanded testing coverage for stateful precompiled contracts.
Added new tests for distribution and staking functionalities.

Documentation

Updated changelog to reflect new features and improvements.

Refactor

Improved code organization and removed unused code across various modules.

We should look into solving the lint issue


Speedup instrumentation by not validating (only needed in tests)

What does this PR do?
Sets the ElasticApmAgent.validateAdvice() call to only execute in tests. No changelog update
Checklist

[x] This is something else

[ ] I have updated CHANGELOG.asciidoc

:green_heart: Build Succeeded

the below badges are clickable and redirect to their specific view in the CI or DOCS

Expand to view the summary

Build stats

Reason: null

Start Time: 2022-02-01T12:37:24.113+0000

Duration: 51 min 55 sec

Commit: e1fe90fc399ef611d4e5e141b3299b9c17206744

Test stats :test_tube:

Test
Results

Failed
0

Passed
2472

Skipped
16

Total
2488

:green_heart: Flaky test report
Tests succeeded.
:robot: GitHub comments
To re-run your PR in the CI, just comment with:

/test : Re-trigger the build.

run benchmark tests : Run the benchmark tests.

run jdk compatibility tests : Run the JDK Compatibility tests.

run integration tests : Run the Agent Integration tests.

run end-to-end tests : Run the APM-ITs.

run windows tests : Build & tests on windows.

run elasticsearch-ci/docs : Re-trigger the docs validation. (use unformatted text in the comment!)


Update default Ubuntu 18.04 python3 packages

Versions

Python: 3.6.6
OS: Ubuntu 18.04.1
Kivy:1.9.1-1build3
Kivy installation method: default ubuntu (apt-get install python3-kivy), no PPA

Description
The problem is a known problem (I've seen it in the Issue list, see the log below): kivy does not work on Ubuntu 18.04 with python3 with a classic hello-world example (code is also below).
But everyone points to two possible solutions:

Use python2 (I tried, and it works), or
Use the Kivy PPA (I did not try, but it works for a lot of people)

My problem is that I don't want to encourage my users to go for neither one of those: python2 is already in the past, and installing 3rd party PPAs is undesirable for both inexperienced users and security-concerned experienced users.
So, I would like to kindly ask the development team to provide a working python3-kivy package through the official Ubuntu 18.04 channels. It does not need to be the shiniest and greatest of versions, just a working version every now and then.
Thanks!
Code and Logs
Code:
#!/usr/bin/env python3
# -*- coding: utf-8 -*-
from kivy.app import App
from kivy.uix.button import Button

class TestApp(App):
    def build(self):
        return Button(text='Hello World')

TestApp().run()

And logs:
[WARNING] [Config      ] Older configuration version detected (0 instead of 14)
[WARNING] [Config      ] Upgrading configuration in progress.
[INFO   ] [Logger      ] Record log in /home/xxx/.kivy/logs/kivy_18-10-15_0.txt
[INFO   ] [Kivy        ] v1.9.1
[INFO   ] [Python      ] v3.6.6 (default, Sep 12 2018, 18:26:19)
[GCC 8.0.1 20180414 (experimental) [trunk revision 259383]]
[INFO   ] [Factory     ] 179 symbols loaded
[INFO   ] [Image       ] Providers: img_tex, img_dds, img_gif, img_sdl2, img_pil (img_ffpyplayer ignored)
[INFO   ] [Text        ] Provider: sdl2
[INFO   ] [OSC         ] using <multiprocessing> for socket
[INFO   ] [Window      ] Provider: sdl2(['window_egl_rpi'] ignored)
[CRITICAL] [Window      ] Unable to find any valuable Window provider at all!
egl_rpi - ImportError: cannot import name 'bcm'
  File "/usr/lib/python3/dist-packages/kivy/core/__init__.py", line 59, in core_select_lib
    fromlist=[modulename], level=0)
  File "/usr/lib/python3/dist-packages/kivy/core/window/window_egl_rpi.py", line 12, in <module>
    from kivy.lib.vidcore_lite import bcm, egl

sdl2 - TypeError: expected bytes, str found
  File "/usr/lib/python3/dist-packages/kivy/core/__init__.py", line 67, in core_select_lib
    cls = cls()
  File "/usr/lib/python3/dist-packages/kivy/core/window/window_sdl2.py", line 138, in __init__
    super(WindowSDL, self).__init__()
  File "/usr/lib/python3/dist-packages/kivy/core/window/__init__.py", line 722, in __init__
    self.create_window()
  File "/usr/lib/python3/dist-packages/kivy/core/window/window_sdl2.py", line 237, in create_window
    self.fullscreen, resizable, state)
  File "kivy/core/window/_window_sdl2.pyx", line 94, in kivy.core.window._window_sdl2._WindowSDL2Storage.setup_window (kivy/core/window/_window_sdl2.c:2204)

x11 - ModuleNotFoundError: No module named 'kivy.core.window.window_x11'
  File "/usr/lib/python3/dist-packages/kivy/core/__init__.py", line 59, in core_select_lib
    fromlist=[modulename], level=0)

[CRITICAL] [App         ] Unable to get a Window, abort.
 Exception ignored in: 'kivy.properties.dpi2px'
 Traceback (most recent call last):
   File "/usr/lib/python3/dist-packages/kivy/utils.py", line 513, in __get__
     retval = self.func(inst)
   File "/usr/lib/python3/dist-packages/kivy/metrics.py", line 175, in dpi
     EventLoop.ensure_window()
   File "/usr/lib/python3/dist-packages/kivy/base.py", line 126, in ensure_window
     sys.exit(1)
 SystemExit: 1
[CRITICAL] [App         ] Unable to get a Window, abort.

I followed the guide, installed the depends, and set everything up for python3 and it just gives me the same exact issue to the letter.
I also agree with this; python2 is going to be deprecated next year and ppa's (while I'm not completely against it; I do use it in special instances), its preferred to just install from the repo. It's just easier that way rather than adding another source to the list.
The other thread regarding this issue (#4385) was locked, so I couldn't add anything to it.
$ sudo find /usr/lib/python3/dist-packages/kivy/ | grep -i window
/usr/lib/python3/dist-packages/kivy/core/window
/usr/lib/python3/dist-packages/kivy/core/window/__init__.py
/usr/lib/python3/dist-packages/kivy/core/window/__pycache__
/usr/lib/python3/dist-packages/kivy/core/window/__pycache__/window_egl_rpi.cpython-36.pyc
/usr/lib/python3/dist-packages/kivy/core/window/__pycache__/window_pygame.cpython-36.pyc
/usr/lib/python3/dist-packages/kivy/core/window/__pycache__/window_sdl2.cpython-36.pyc
/usr/lib/python3/dist-packages/kivy/core/window/__pycache__/__init__.cpython-36.pyc
/usr/lib/python3/dist-packages/kivy/core/window/_window_sdl2.cpython-36m-x86_64-linux-gnu.so
/usr/lib/python3/dist-packages/kivy/core/window/window_egl_rpi.py
/usr/lib/python3/dist-packages/kivy/core/window/window_sdl2.py
/usr/lib/python3/dist-packages/kivy/core/window/window_pygame.py

$ cat -n /usr/lib/python3/dist-packages/kivy/core/window/__init__.py | less
1571  if platform == 'linux':
1572      window_impl += [('x11', 'window_x11', 'WindowX11')]

It's just interesting to see that there is no kivy.core.window. Also, there is no reference to sdl2 or pygame either. I have sdl2 and pygame installed because I am interested in those packages and have been studying pygame recently which uses sdl1.2 (even in python3).
This is what I came up with while just doing some superficial checking and a little bit of digging. Not sure why this issue hasn't been bumped up and any related issues are locked or closed. All other solutions seem to be MS Windows focused.

Ubuntu does have kivy packages, but we have no control over them. It wasn't built by us or with our input as far as I know. Also, anecdotally, they don't work.
Ubuntu seems to have update policies which would preclude them from being able to update the kivy packages to more recent versions. Someone could try to contact them to add a newer version of kivy to their upcoming released images, but I'm not sure what their process is.
I'm closing this for now as there doesn't seem to be much we can do about this.


didChangeAppLifecycleState is called every time an error is thrown by determinePosition()

Hi everyone!
I don't know if this it's intended to happen, but everytime an error is thrown by determinePosition() (except the TimeoutException thrown by getCurrentPosition()), my didChangeAppLifecycleState method it's called with state == AppLifecycleState.resumed, which triggers some updates meant to be triggered only (well... mainly) when the app is resumed.
Why does this happen? Why didChangeAppLifecycleState isn't called by the TimeoutException?
thanks!

Hi @Poltr1,
The didChangeAppLifecycleState() is called when the App moves from the background into the foreground or when the app is automatically restarted after a crash.
Since you mentioned the determinePosition() method throws an error, it is likely that your App unexpectedly crashes and the Android sub-system automatically is restarting it.
Other reasons the didChangeAppLifecycleState() method (with the state == AppLifecycleState.resumed) might be called is when the App requests permissions. Android will move your app into the background to show the permission dialog. When the permission dialog is handled by the user, Android will move your app back into the foreground and the didChangeAppLifecycleState() method is called.
Hope this will give you some more insight into debugging the issue. If you feel the exception thrown by the determinePosition method is related to a bug in the geolocator plugin please provide some more details (i.e. version of the geolocator you use, stack trace of the exception, possibly an example demonstrating the bug).
For now I will close this issue.


[new-parser] Improve Verifier's consequence processing resiliency

Fixes #5886.

This is, in my opinion, a minor bug in drools-verifier that manifests due to a slight change of behavior in the new parser.
The Verifier at some point cleans the RHS code by removing comments. While doing that, it assumes that each comment line (// asdf...) ends with a new-line character. This is no longer true with the new parser if the comment is the last line before the RHS end or before another named consequence (then[label]). This is simply because the whitespace part of the input between the default RHS consequence and any named consequence(s) or RHS end is not part of the consequence RuleContext. This is natural to how the ANTLR parser works.
The old parser's behavior could be replicated with some custom code but in this case, I think it's better to align with the new behavior and adapt the Verifier code to it.

How to replicate CI configuration locally?

Build Chain tool does "simple" maven build(s), the builds are just Maven commands, but because the repositories relates and depends on each other and any change in API or class method could affect several of those repositories there is a need to use build-chain tool to handle cross repository builds and be sure that we always use latest version of the code for each repository.
build-chain tool is a build tool which can be used on command line locally or in Github Actions workflow(s), in case you need to change multiple repositories and send multiple dependent pull requests related with a change you can easily reproduce the same build by executing it on Github hosted environment or locally in your development environment. See local execution details to get more information about it.

How to retest this PR or trigger a specific build:

for pull request and downstream checks

Push a new commit to the PR. An empty commit would be enough.

for a full downstream build

for github actions job: add the label run_fdb

for Jenkins PR check only

If you are an ASF committer for KIE podling, login to Jenkins (https://ci-builds.apache.org/job/KIE/job/drools/), go to the specific PR job, and click on Build Now button.

@mariofusco @tkobayas @gitgabrio please review.