Disabled MaterialButton ignores disabledTextColor when textColor is set

Is there an existing issue for this?

[X] I have searched the existing issues
[X] I have read the guide to filing a bug

Use case
disabledTextColor gets ignored when textColor is set and MaterialButton is disabled
MaterialButton can be disabled if the onPressed method isn't set or set to null.
I noticed that when disabled, textColor is being used instead of disabledTextColor
I know its a design choice but it seems weird that a disabled button with both textColor and disabledTextColor properties set, it will still use the textColor.
Steps to reproduce
MaterialButton(
  onPressed: null,
  textColor: Colors.blue,
  disabledTextColor: Colors.red,
  child: Text("Button"),
);

Expected results
Button text color Red.
Actual results
Button text color Blue.

This seems even more weird when considering that the properties color and disabledColor behave differently.
For example:
MaterialButton(
  onPressed: null,
  color: Colors.blue,
  disabledColor: Colors.red,
  child: Text("Button"),
);

This will be filled Red.
Proposal
I think that if disabledTextColor is set, it shouldn't be ignored to textColor when the button is disabled.
I think that will make more sense and make the widget more concise with its properties.

Thanks for your report. Reproduced this issue on the latest stable and master channels.
And it also happens when setting onLongPress: null to satisfy the invert condition with:
https://github.com/flutter/flutter/blob/216605b60f678d9ec9be880078e1de1b9f6e536f/packages/flutter/lib/src/material/material_button.dart#L307-L311

flutter doctor -v (stable and master)
[✓] Flutter (Channel stable, 3.10.1, on macOS 13.0.1 22A400 darwin-x64, locale en-VN)
    • Flutter version 3.10.1 on channel stable at /Users/huynq/Documents/GitHub/flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision d3d8effc68 (26 hours ago), 2023-05-16 17:59:05 -0700
    • Engine revision b4fb11214d
    • Dart version 3.0.1
    • DevTools version 2.23.1

[✓] Android toolchain - develop for Android devices (Android SDK version 32.0.0)
    • Android SDK at /Users/huynq/Library/Android/sdk
    • Platform android-33, build-tools 32.0.0
    • ANDROID_HOME = /Users/huynq/Library/Android/sdk
    • Java binary at: /Applications/Android Studio.app/Contents/jbr/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 17.0.6+0-17.0.6b802.4-9586694)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 14.3)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Build 14E222b
    • CocoaPods version 1.11.3

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 2022.2)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 17.0.6+0-17.0.6b802.4-9586694)

[✓] VS Code (version 1.78.1)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.64.0

[✓] Connected device (3 available)
    • RMX2001 (mobile) • EUYTFEUSQSRGDA6D • android-arm64  • Android 11 (API 30)
    • macOS (desktop)  • macos            • darwin-x64     • macOS 13.0.1 22A400 darwin-x64
    • Chrome (web)     • chrome           • web-javascript • Google Chrome 113.0.5672.126

[✓] Network resources
    • All expected network resources are available.

• No issues found!

[!] Flutter (Channel master, 3.11.0-12.0.pre.27, on macOS 13.0.1 22A400 darwin-x64, locale en-VN)
    • Flutter version 3.11.0-12.0.pre.27 on channel master at /Users/huynq/Documents/GitHub/flutter_master
    ! Warning: `flutter` on your path resolves to /Users/huynq/Documents/GitHub/flutter/bin/flutter, which is not inside your current Flutter SDK checkout at /Users/huynq/Documents/GitHub/flutter_master. Consider adding /Users/huynq/Documents/GitHub/flutter_master/bin to the front of your path.
    ! Warning: `dart` on your path resolves to /Users/huynq/Documents/GitHub/flutter/bin/dart, which is not inside your current Flutter SDK checkout at /Users/huynq/Documents/GitHub/flutter_master. Consider adding /Users/huynq/Documents/GitHub/flutter_master/bin to the front of your path.
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 90cf8d1dd4 (80 minutes ago), 2023-05-23 21:51:25 -0400
    • Engine revision aba9dc4640
    • Dart version 3.1.0 (build 3.1.0-139.0.dev)
    • DevTools version 2.23.1
    • If those were intentional, you can disregard the above warnings; however it is recommended to use "git" directly to perform update checks and upgrades.

[✓] Android toolchain - develop for Android devices (Android SDK version 32.0.0)
    • Android SDK at /Users/huynq/Library/Android/sdk
    • Platform android-33, build-tools 32.0.0
    • ANDROID_HOME = /Users/huynq/Library/Android/sdk
    • Java binary at: /Applications/Android Studio.app/Contents/jbr/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 17.0.6+0-17.0.6b802.4-9586694)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 14.3)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Build 14E222b
    • CocoaPods version 1.11.3

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 2022.2)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 17.0.6+0-17.0.6b802.4-9586694)

[✓] VS Code (version 1.78.1)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.64.0

[✓] Connected device (2 available)
    • macOS (desktop) • macos  • darwin-x64     • macOS 13.0.1 22A400 darwin-x64
    • Chrome (web)    • chrome • web-javascript • Google Chrome 113.0.5672.126

[✓] Network resources
    • All expected network resources are available.

! Doctor found issues in 1 category.

@huycozy Yea, I looked around and found what causes the disabledTextColor to be ignored.
should I make a PR change? I'm not sure since its a design choice.

@stavgafny Issue will be evaluated by a domain expert before being put on the priority list. However, PR is always welcome :)


Multi variable subscribe

Hey,
I use plc LSIS-type connected to opc server which send data to node-red successfully. also I add MQTT nodes publish and l subscribe with opc server on local host to send the data to ubidots. I had use ubidots nodes for publish and subscribe as a primary test, the problem is in subscribing I can't listen to more than one variable.
How can I solve this problem?

I think I have found what's causing this behavior.
The following block (https://github.com/ubidots/ubidots-nodered/blob/85985078baacca3de60a504217db83c6f704dd3c/ubidots/ubidots_in.js#L7-L9) is ending the previous connection of the client and this could cause the other connections to the variables to stop listening.


Add LUT count to table

The 4K and 8K devices have the same die, so a 4K device has 7680 LUTs that can all be used with icestorm. But we intentionally don't draw too much attention to that fact because some people might then believe that we do IceStorm so we can "cheat" the Lattice pricing model. That's why I'm not merging this PR: We would either have incorrect information on the page, or information that we do not want to have prominently right at the top of the page.

Hi Clifford,
I did not know this.
Why not just display the spec'd numbers? (Which is what this pull request had.)
I could change the wording to 'specified'. That should be safe.
The reason I wanted to have this info is because when I go to the site, I might want to figure out which part I need based on what is supported. Typically that is a function of IOs and LUTs.


ERROR: "docker buildx build" requires exactly 1 argument.

docker build -t mymicroservice giving error ERROR: "docker buildx build" requires exactly 1 argument.
Problem encountered on https://dotnet.microsoft.com/en-us/learn/aspnet/microservice-tutorial/docker-image
Operating System: windows
Provide details about the problem you're experiencing. Include your operating system version, exact error message, code sample, and anything else that is relevant.

The command on the tutorial is docker build -t mymicroservice ..
Have you tried that one?


GORA-437 implement couchdb datastore

This pull request is for GORA-437. I developed couchdb store for gora. I added javadocs and test codes. The tests required couchdb instance. But I wondered how to start couchdb server programmatically using java. So, I sent an email to couchDB userlist. They suggest that using docker. Then I use docker programmaticaly with  https://github.com/testcontainers/testcontainers-java . All test cases passed with this approach. But this caused dependece to docker and jdk 1.8 . So this pull request doesn't merge to master.
We should look another solution. Maybe using mockito for testing or another idea?
All tests are passed on my local. I ignored 3 tests. You can see them in my test code.

In general this is looking very good. Can you please go through and address my comments? Can you also ensure that for every new method that you've added you also add a Javadoc comment?
Finally, for the time being at least, can you

Add package-info.java for each and every new module you've added.
Create documentation for this module as per http://gora.apache.org/current/index.html#gora-modules

Thanks @cguzel great work :)

@cguzel can you please rebase against master?

@lewismc How to create documentation? I don't know how to edit this page :  http://gora.apache.org/current/index.html#gora-modules

http://svn.apache.org/repos/asf/gora/site/

I added a patch for documentation https://issues.apache.org/jira/browse/GORA-489

I have added maven profile in gora-couchdb pom.xml. So The tests skip by default. If you use java 1.8 and docker, you build and run tests using maven profile as "mvn clean install -P couchdb-with-test".

I have added new  javadocs.

Hey@cguzel,
FYI TestContainers dependency has "compile" scope. You probably want to move it to "test" scope :)

Thanks @bsideup . I will check it.

@cguzel we're planning to add Apache Gora to TestContainers' "who is using?" list, don't you mind?
https://github.com/testcontainers/testcontainers-java/pull/296

Hi, @bsideup . It's OK for me. But I am not PMC for GORA. So, This question should be answered by GORA PMC members.
@lewis ?

@cguzel did you mean @lewismc ?

Sorry. Yes true.  I corrected the mistake.

Absolutely @bsideup @cguzel thank you. Let's get a ticket filed to change scope to @Test as well

@lewismc may I ask you to comment or react with " :+1: " in https://github.com/testcontainers/testcontainers-java/pull/296 ? :)


Adding environment variable support to source key in meshrc.yaml

Is your feature request related to a problem? Please describe.
I'm trying to add a service to my meshrc.yaml config that can point to swagger documentation. The source key I am hoping to point to is a URL that changes per different environments I'm deploying to.
Describe the solution you'd like
Something like below would be ideal
# meshrc.yaml
sources:
    - name: AnalyticsService
      handler:
          openapi:
              source: '{env.ANALYTICS_API}/swagger/doc.json'
              endpoint: '{env.ANALYTICS_API}'

Describe alternatives you've considered
For now, I plan to point directly to one environment and manually keep them in sync. Example
# meshrc.yaml
sources:
    - name: AnalyticsService
      handler:
          openapi:
              source: 'https://my-service.com/swagger/doc.json'
              endpoint: '{env.ANALYTICS_API}'

Additional context
Its working just fine in the endpoint key attached in the above examples. Hopefully, this means it might be a small lift 🤞🏼

It's already supported. Could you share a reproduction on CodeSandbox? Thanks!

Ended up getting to work like this
sources:
    - name: AnalyticsServiceGo
      handler:
          openapi:
              source: ${ANALYTICS_API}/swagger/doc.json
              endpoint: ${ANALYTICS_API}

With .env prefix it wasn't recognizing passed in variables.


Add test mode handling to all list plugins

Any list plugin that manipulate a list needs to handle test mode in order to manipulate online plugins during test mode.

Closed with https://github.com/Flexget/Flexget/commit/01cd4a1e9de598930e6afdacd709311d588bdb11


ibus-engines.table: 1.17.4 -> 1.17.6

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://repology.org/project/ibus-table/versions.
meta.description for ibus-engines.table is: IBus framework for table-based input methods
meta.homepage for ibus-engines.table is: https://github.com/kaio/ibus-table/wiki
Updates performed

Version update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 1.17.5 with grep in /nix/store/ndn5dnxiljhn9hhjgha1mfssjsr84p54-ibus-table-1.17.5
found 1.17.5 in filename of file in /nix/store/ndn5dnxiljhn9hhjgha1mfssjsr84p54-ibus-table-1.17.5

Rebuild report (if merged into master) (click to expand)

4 total rebuild path(s)

4 package rebuild(s)

First fifty rebuilds by attrpath

ibus-engines.table
ibus-engines.table-chinese
ibus-engines.table-others

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/ndn5dnxiljhn9hhjgha1mfssjsr84p54-ibus-table-1.17.5 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A ibus-engines.table https://github.com/r-ryantm/nixpkgs/archive/400daed0638ea8e989229088b5301491ff5e60dd.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/400daed0638ea8e989229088b5301491ff5e60dd#ibus-engines.table

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/ndn5dnxiljhn9hhjgha1mfssjsr84p54-ibus-table-1.17.5
ls -la /nix/store/ndn5dnxiljhn9hhjgha1mfssjsr84p54-ibus-table-1.17.5/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

3 packages built:

ibus-engines.table
ibus-engines.table-chinese
ibus-engines.table-others

Maintainer pings
cc @laMudri for testing.

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://repology.org/project/ibus-table/versions.
meta.description for ibus-engines.table is: IBus framework for table-based input methods
meta.homepage for ibus-engines.table is: https://github.com/kaio/ibus-table/wiki
Updates performed

Version update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 1.17.6 with grep in /nix/store/kcvzi1xk08kz63adn8jjn6gpd3p9dgfm-ibus-table-1.17.6
found 1.17.6 in filename of file in /nix/store/kcvzi1xk08kz63adn8jjn6gpd3p9dgfm-ibus-table-1.17.6

Rebuild report (if merged into master) (click to expand)

4 total rebuild path(s)

4 package rebuild(s)

First fifty rebuilds by attrpath

ibus-engines.table
ibus-engines.table-chinese
ibus-engines.table-others

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/kcvzi1xk08kz63adn8jjn6gpd3p9dgfm-ibus-table-1.17.6 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A ibus-engines.table https://github.com/r-ryantm/nixpkgs/archive/bd1a17ebfa2c3e85ee534bdcfe0f15b42877fc57.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/bd1a17ebfa2c3e85ee534bdcfe0f15b42877fc57#ibus-engines.table

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/kcvzi1xk08kz63adn8jjn6gpd3p9dgfm-ibus-table-1.17.6
ls -la /nix/store/kcvzi1xk08kz63adn8jjn6gpd3p9dgfm-ibus-table-1.17.6/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

3 packages built:

ibus-engines.table
ibus-engines.table-chinese
ibus-engines.table-others

Maintainer pings
cc @laMudri for testing.


SIG_ABRT trying to connect to MSSQL

Environment

Python: Python 3.11.4
pyodbc: 4.0.39
OS: OSX 13.5 Apple M2
DB: MS SQL 2019
driver: msodbcsql18 <IP_ADDRESS>
unixodbc 2.3.12

Issue
I get a SIG_ABRT trying to connect when running example code
Code
import pydodbc
cnxn = pyodbc.connect('DRIVER={ODBC Driver 18 for SQL Server};SERVER='+server+';UID='+username+';PWD='+password)
crsr = cnxn.cursor()
rows = crsr.execute("select @@VERSION").fetchall()
print(rows)
crsr.close()
cnxn.close()

Crash Report
-------------------------------------
Translated Report (Full Report Below)
-------------------------------------

Process:               Python [80110]
Path:                  /opt/homebrew/*/Python.framework/Versions/3.11/Resources/Python.app/Contents/MacOS/Python
Identifier:            org.python.python
Version:               3.11.4 (3.11.4)
Code Type:             ARM-64 (Native)
Parent Process:        zsh [74180]
Responsible:           stable [1166]
User ID:               503

Date/Time:             2023-08-14 13:48:40.7263 -0700
OS Version:            macOS 13.5 (22G74)
Report Version:        12
Anonymous UUID:        FC5E7E51-090A-D613-3E43-3F0A0B1950CA

Sleep/Wake UUID:       D00D6341-8CB0-466E-BAE2-6FD0E6EE1CD3

Time Awake Since Boot: 13000 seconds
Time Since Wake:       1080 seconds

System Integrity Protection: enabled

Crashed Thread:        0  Dispatch queue: com.apple.main-thread

Exception Type:        EXC_CRASH (SIGABRT)
Exception Codes:       0x0000000000000000, 0x0000000000000000

Termination Reason:    Namespace SIGNAL, Code 6 Abort trap: 6
Terminating Process:   Python [80110]

Application Specific Information:
stack buffer overflow

Thread 0 Crashed::  Dispatch queue: com.apple.main-thread
0   libsystem_kernel.dylib        	       0x184568764 __pthread_kill + 8
1   libsystem_pthread.dylib       	       0x18459fc28 pthread_kill + 288
2   libsystem_c.dylib             	       0x1844adb74 __abort + 128
3   libsystem_c.dylib             	       0x18449e640 __stack_chk_fail + 96
4   pyodbc.cpython-311-darwin.so  	       0x1031072f8 GetErrorFromHandle(Connection*, char const*, void*, void*) + 360
5   pyodbc.cpython-311-darwin.so  	       0x103107144 RaiseErrorFromHandle(Connection*, char const*, void*, void*) + 16
6   pyodbc.cpython-311-darwin.so  	       0x1030fffe4 Connection_New(_object*, bool, bool, long, bool, _object*, Object&) + 1248
7   pyodbc.cpython-311-darwin.so  	       0x10310cc60 mod_connect(_object*, _object*, _object*) + 1000
8   Python                        	       0x1033b29e4 cfunction_call + 60
9   Python                        	       0x10336af68 _PyObject_MakeTpCall + 128
10  Python                        	       0x103443238 _PyEval_EvalFrameDefault + 40652
11  Python                        	       0x103438828 PyEval_EvalCode + 168
12  Python                        	       0x10348a044 run_eval_code_obj + 84
13  Python                        	       0x103489fa8 run_mod + 112
14  Python                        	       0x103489de8 pyrun_file + 148
15  Python                        	       0x10348983c _PyRun_SimpleFileObject + 268
16  Python                        	       0x1034891cc _PyRun_AnyFileObject + 216
17  Python                        	       0x1034a5070 pymain_run_file_obj + 220
18  Python                        	       0x1034a49b0 pymain_run_file + 72
19  Python                        	       0x1034a4290 Py_RunMain + 704
20  Python                        	       0x1034a53c8 Py_BytesMain + 40
21  dyld                          	       0x184247f28 start + 2236

Can you check if 4.0.30 works, and 4.0.31 crashes? Or otherwise narrow down between which two versions this started happening?

Got it to crash on 4.0.30 and a few versions in between. My suspicion is something might have broken with unixodbc 2.3.12, but I have no idea how to downgrade that.

If you installed unixodbc via Homebrew, you can try to install e.g<EMAIL_ADDRESS>for that version. If they did not have versioned formula, then you'll need to compile and install it manually.

Thanks @v-chojas, I've spent an hour messing around with the make commands and I can't get it to work :/ Let's see if anyone else has seen this issue

Just tested this on another Apple M1 and it worked fine. Something is borked on my main machine. 😭


Class Training

Demon Hunter and Voidcaller need class training bonuses for 51-100 mastery investment.

Voidcaller OK

Well crap, these DEVs are on the ball...nevermind then.


feat: add mIgnore tag

adds the mIgnore tag which will cause the manifest to ignore the asset and not add it.

will follow up in separate PR
https://github.com/play-co/pixi-external/issues/89


Plain Node Server

Is istanbuljs supposed to work with a plain Node http-server (without Express)?

yes, nyc will run test coverage for the http server but not for the scripts being served by it (nyc does not run in the browser).

Thanks for the answer.
I can't get it to work. There isn't any error but all fields are zeros. Mocha tests works. Any pointers you can give me?

Not without a link to a repository experiencing the issue.

Ok, thanks. I don't have a repository for it.


Document missing steps in development setup

Document missing steps in the development setup (https://docs.crossfeed.cyber.dhs.gov/setup). The corresponding file is https://github.com/cisagov/crossfeed/blob/e0e37b58d56b4ec819edab05da413618c8b69613/docs/src/documentation-pages/contributing/setup.md.

@stewartl97 do you think you'd be able to help with this, as you mentioned before that you knew which steps are missing?

I was able to document these steps successfully.


HasData doesn't work properly with derived classes

When I insert data with HasData for entities with inheritance I'm getting an error:
Value cannot be null.
Parameter name: derivedType
It's not always a case but really hard to debug
 Add-Migration SeedData
System.ArgumentNullException: Value cannot be null.
Parameter name: derivedType
   at Microsoft.EntityFrameworkCore.Utilities.Check.NotNull[T](T value, String parameterName)
   at Microsoft.EntityFrameworkCore.EntityTypeExtensions.IsAssignableFrom(IEntityType entityType, IEntityType derivedType)
   at System.Linq.Enumerable.Any[TSource](IEnumerable`1 source, Func`2 predicate)
   at Microsoft.EntityFrameworkCore.ChangeTracking.Internal.StateManager.StopTracking(InternalEntityEntry entry)
   at Microsoft.EntityFrameworkCore.ChangeTracking.Internal.InternalEntityEntry.SetEntityState(EntityState oldState, EntityState newState, Boolean acceptChanges)
   at Microsoft.EntityFrameworkCore.Migrations.Internal.MigrationsModelDiffer.DiffData(TableMapping source, TableMapping target, DiffContext diffContext)
   at Microsoft.EntityFrameworkCore.Migrations.Internal.MigrationsModelDiffer.Diff(TableMapping source, TableMapping target, DiffContext diffContext)+MoveNext()
   at Microsoft.EntityFrameworkCore.Migrations.Internal.MigrationsModelDiffer.DiffCollection[T](IEnumerable`1 sources, IEnumerable`1 targets, DiffContext diffContext, Func`4 diff, Func`3 add, Func`3 remove, Func`4[] predicates)+MoveNext()
   at System.Linq.Enumerable.ConcatIterator`1.MoveNext()
   at Microsoft.EntityFrameworkCore.Migrations.Internal.MigrationsModelDiffer.Sort(IEnumerable`1 operations, DiffContext diffContext)
   at Microsoft.EntityFrameworkCore.Migrations.Design.MigrationsScaffolder.ScaffoldMigration(String migrationName, String rootNamespace, String subNamespace, String language)
   at Microsoft.EntityFrameworkCore.Design.Internal.MigrationsOperations.AddMigration(String name, String outputDir, String contextType)
   at Microsoft.EntityFrameworkCore.Design.OperationExecutor.AddMigrationImpl(String name, String outputDir, String contextType)
   at Microsoft.EntityFrameworkCore.Design.OperationExecutor.OperationBase.<>c__DisplayClass3_0`1.<Execute>b__0()
   at Microsoft.EntityFrameworkCore.Design.OperationExecutor.OperationBase.Execute(Action action)
Value cannot be null.
Parameter name: derivedType

Further technical details
EF Core version: 2.1.1
Database Provider: Microsoft.EntityFrameworkCore.SqlServer
Operating system:
IDE: (e.g. Visual Studio 2017 15.7.5)

@iamkarlson could you please include the entity types?

@divega sorry I can't since it's under NDA. I'll try to figure out a minimal working example on the weekend but I'm not sure that it's really achievable.

EF Team Triage: Closing this issue as the requested additional details have not been provided and we have been unable to reproduce it.
BTW this is a canned response and may have info or details that do not directly apply to this particular issue. While we'd like to spend the time to uniquely address every incoming issue, we get a lot traffic on the EF projects and that is not practical. To ensure we maximize the time we have to work on fixing bugs, implementing new features, etc. we use canned responses for common triage decisions.


add linux-{aarch64,ppc64le}

Checklist

[x] Used a personal fork of the feedstock to propose changes
[x] Bumped the build number (if the version is unchanged)
[ ] Reset the build number to 0 (if the version changed)
[x] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[ ] Ensured the license file is being packaged.

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.
I do have some suggestions for making it better though...
For recipe:

License is not an SPDX identifier (or a custom LicenseRef) nor an SPDX license expression.

Documentation on acceptable licenses can be found here.

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

Ready for review @conda-forge/vim

Hi! This is the friendly automated conda-forge-linting service.
I was trying to look for recipes to lint for you, but it appears we have a merge conflict.
Please try to merge or rebase with the base branch to resolve this conflict.
Please ping the 'conda-forge/core' team (using the @ notation in a comment) if you believe this is a bug.

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

Ready for review @conda-forge/vim @conda-forge/core

Can you please add this to the arch migrator?

Can you please add this to the arch migrator?

Added here: https://github.com/conda-forge/conda-forge-pinning-feedstock/pull/1933

Can you please add this to the arch migrator?

Added here: conda-forge/conda-forge-pinning-feedstock#1933

Thanks! Merged :)

Can you please add this to the arch migrator?

Added here: conda-forge/conda-forge-pinning-feedstock#1933

Thanks! Merged :)

Thanks! Conda-forge will open a new PR automatically and this PR can be closed, right?


Autostart on join

It would be really cool to have an option to autostart a specified backend server when a player tries to join the proxy. This could be beneficial because it could allow the proxy to run for longer and then make the unused backend servers shutdown when not in use. It could also be cool to autostart a backend when using the switch command

We considered implementing this but I think the problem was that the bungee/velocity API didn't have any events that triggered before a working server was required.
There is however a different way:

Set your proxy server to start and stop automatically
Set your backen/lobby server to not stop automatically
Configure the proxy to automatically start and stop the backend server with it.


python311Packages.pymongo-inmemory: 0.4.1 -> 0.4.2

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for python311Packages.pymongo-inmemory is: A mongo mocking library with an ephemeral MongoDB running in memory
meta.homepage for python311Packages.pymongo-inmemory is: https://github.com/kaizendorks/pymongo_inmemory
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes
Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 0.4.2 with grep in /nix/store/x0wgf1syc9asvvig1s4psmyapkrydvsk-python3.11-pymongo-inmemory-0.4.2
found 0.4.2 in filename of file in /nix/store/x0wgf1syc9asvvig1s4psmyapkrydvsk-python3.11-pymongo-inmemory-0.4.2

Rebuild report (if merged into master) (click to expand)

5 total rebuild path(s)

5 package rebuild(s)

First fifty rebuilds by attrpath

python311Packages.cachier
python311Packages.pymongo-inmemory
python312Packages.cachier
python312Packages.pymongo-inmemory

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/x0wgf1syc9asvvig1s4psmyapkrydvsk-python3.11-pymongo-inmemory-0.4.2 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A python311Packages.pymongo-inmemory https://github.com/r-ryantm/nixpkgs/archive/e5b7d823ef11f5477c32764b6050f34d6ce2db35.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/e5b7d823ef11f5477c32764b6050f34d6ce2db35#python311Packages.pymongo-inmemory

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/x0wgf1syc9asvvig1s4psmyapkrydvsk-python3.11-pymongo-inmemory-0.4.2
ls -la /nix/store/x0wgf1syc9asvvig1s4psmyapkrydvsk-python3.11-pymongo-inmemory-0.4.2/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review --extra-nixpkgs-config '{ allowInsecurePredicate = x: true; }' run on x86_64-linux 1

8 packages built:

python311Packages.cachier
python311Packages.cachier.dist
python311Packages.pymongo-inmemory
python311Packages.pymongo-inmemory.dist
python312Packages.cachier
python312Packages.cachier.dist
python312Packages.pymongo-inmemory
python312Packages.pymongo-inmemory.dist

Maintainer pings
cc @pbsds for testing.

already done in https://github.com/NixOS/nixpkgs/pull/314954


Moved govuk-frontend script to footer to fix sole trader business name

Moved govuk-frontend script to footer to fix sole trader business name

CI: Security warnings found!


LGOS/LSVS supervision instantiation counts existing instances incorrectly

Describe the bug
Currently we check several items as part of instantiating supervisions:
https://github.com/openscd/open-scd/blob/8ebac53bbf86b86e6b8cdb9d79b28957d4a7b5fc/src/editors/subscription/foundation.ts#L355-L378
One of these items is instantiatedSupervisionsCount:
https://github.com/openscd/open-scd/blob/8ebac53bbf86b86e6b8cdb9d79b28957d4a7b5fc/src/editors/subscription/foundation.ts#L509-L520
With the following (slightly hand modified IID file), supervisions cannot be instantiated.
B30_valKind_valImport.zip
The reason for this is that the count function is showing a much larger number of supervisions than exist.
This is because we are counting all DOIs which have DAIs which have a Val. In this particular IED there are many DOI > DAI > Val within each LGOS.
<LN lnType="LGOS_4" lnClass="LGOS" inst="1" prefix="">
    <Private type="GE_Digital_Energy_UR LN_UUID">Master_LGOS1</Private>
    <DOI name="NamPlt">
        <DAI name="d" valKind="RO" valImport="true">
            <Val>Ctl</Val>
        </DAI>
    </DOI>
    <DOI name="Beh">
        <DAI name="d" valKind="RO" valImport="true">
            <Val>This logical node's behaviour</Val>
        </DAI>
        <DAI name="stVal" valKind="RO"/>
    </DOI>
    <DOI name="St">
        <DAI name="d" valKind="RO" valImport="true">
            <Val>RxGOOSE1 On operand</Val>
        </DAI>
        <DAI name="stVal" valKind="RO" sAddr="0:1441793,0"/>
    </DOI>
and so on...

There are enough supervision instances available, within the Services section: <SupSubscription maxGo="64" maxSv="16"/>
I think this query needs to be tightened up. When is a supervision instantiated? I think only when a DO GoCBRefhas a Val against the DA SetSrcRef.
Array.from(subscriberIED.querySelectorAll(`LN[lnClass="${supervisionType}"]>DOI>DAI>Val`))
(396)

Array.from(subscriberIED.querySelectorAll(`LN[lnClass="${supervisionType}"]>DOI[name="GoCBRef"]>DAI[name="setSrcRef"]>Val`))
12

I think this change would be sufficient (with appropriate modifications for the control block type):
const instantiatedValues = Array.from(
     subscriberIED.querySelectorAll(
-       `LN[lnClass="${supervisionType}"]>DOI>DAI>Val`
+      `LN[lnClass="${supervisionType}"]>DOI[name="GoCBRef"]>DAI[name="setSrcRef"]>Val`
     )
   ).filter(val => val.textContent !== '');

To Reproduce
Steps to reproduce the behavior:

Import an instance of the IED included in the above zip file.
Include another IED, e.g. the attached: NR.zip
Try to carry out a GOOSE subscription.
No supervision occurs.

Expected behavior
A supervised GOOSE instance should appear with the supervision icon.

SORRY!!! It is on me @JakobVogelsang


.evn urls setup not explained

ELASTIC_APM_SERVER_URL="https://foobar.apm.us-central1.gcp.cloud.es.io"
ELASTIC_APM_SECRET_TOKEN="secret123"
ELASTICSEARCH_USERNAME="elastic"
ELASTICSEARCH_PASSWORD="changeme"
ELASTICSEARCH_URL="https://foobar.es.us-central1.gcp.cloud.es.io"
It would be nice to add how to create these servers.. especially not show what is ELASTIC_APM_SERVER_URL, hence was saying dockerisation of launch of these servers would make example more easy to follow

I only see these 2 servers in docker hub..

while it https://www.elastic.co/guide/en/observability/current/apm-installing.html says  image

    apm-server:
        image: docker.elastic.co/apm/apm-server:8.13.0
        container_name: apm-server
        restart: unless-stopped
        volumes:
            - ./apm-server.docker.yml:/usr/share/apm-server/apm-server.yml:ro
        networks:
            - app-network
        ports:
            - "8200:8200"
        environment:
            - bootstrap.memory_lock=true
            - discovery.type=single-node
    elasticsearch:
        image: docker.elastic.co/elasticsearch/elasticsearch:${ELASTIC_VERSION}
        networks:
            - app-network
        ports:
            - "9200:9200"
        environment:
            - "discovery.type=single-node"
        volumes:
            - ./elasticsearch/data:/usr/share/elasticsearch/data:rw
    kibana:
        image: docker.elastic.co/kibana/kibana:${ELASTIC_VERSION}
        networks:
            - app-network
        ports:
            - "5601:5601"
        restart: on-failure
        depends_on:
            - elasticsearch

would this help?


Add privacy manifest

This PR adds privacy manifest that Apple will require from this spring.
I've tested CP and SPM integration and they both worked for me. Additionally, I had to update swift-tools version to 5.3 as resources can only be declared from that version
This section describes type reasons. It seems to me that only CA92.1 is relevant for this library, but I would like if someone could double check this.

Looking at all the reasons, I would also only put CA92.1 here since we're only using them for info that is available to the app (or lib in this case).


Call placement new directly on the allocator

... avoiding handling a pointer to raw memory.

@swift-ci Please smoke test and merge


Auto clean up the copypasted text?

Can summernote auto cleaning up the copypasted text? Text from buffer can contain formating tags, and I see two ways:

Clean the resulting html before save it to database, and remove the unsual tags
Tell to editor to allow only plain text inserting

+1

In my opinion, second option should be good. The paste option should get only plain text without any formatting tags.

This is very important issue for all wysiwyg users. But it's up to summernote, not django-summernote. I'm handling similar issues on summernote.
Please watch https://github.com/summernote/summernote/issues/1168 .
Thanks!

You can achieve this with summernote 'summernote-cleaner' plugin :
and an extra summernote config:
SUMMERNOTE_CONFIG.update({
    'js': (
        os.path.join(STATIC_URL, '/summernote/summernote-cleaner.js'),
    ),
})

It doesnt work!


The option to extract metadata is no more available for timeseries resource

While testing timeseries resource on beta.hydroshare.org, noticed the option to extract metadata in the file add modal dialog box is no more available. This problem also exists on www. This might have been a result of the refactoring of html associated with the resource landing page that was done a while ago.

Do you have a screenshot of the missing option or could you be more specific?

@mauriel133 Don't have a screenshot. For timeseries resource, there used to be a checkbox with text something like "Check to extract metadata" on the add file dialogbox.

@pkdash Please close this issue if it has been resolved.

@hyi and @mauriel133 This is still a problem on www.

@pkdash, this fix has been pushed to www. Could you confirm and close?

Closing.


x86_64 image possible?

I am unable to install as I have a x86_64 system (synology NAS) - would a build be possible for this type of server?

ghcr.io/sytone/obsidian-remote:latest works perfectly for me on a x86_64 unraid NAS. The base dockerfile.

Was this resolved?

I have not re-tried as yet, but I accept I may have mis-diagnosed the issue
Closing until I have a better description of issue (if any)


Added matrix functionality

I wanted to select sources and targets individually, like I do using a video matrix.
This Pull Request implements this feature by adding some things.
Config:

selectable matrixPaths
AutoTake, when target is selected and new source is selected

Actions

select Source from Matrix specified in the config
select target from Matrix specified in the config
execute take on Matrix specified in the config

Feedbacks

show if specified source on matrix is selected
show if specified target on matrix is selected

Tested with Riedels MediorNet.

Thanks for the input.
I created a new file for state storage.
Furthermore I updated the selection process and the take functionality.
Additionally added feedbacks for 'take' and 'clear' and according presets.
@baltedewit: check, wether you find that better.


[BI-1] Fixing File Format For New Added Ideas

This pull request fixes the incorrect format of new ideas and adds project ideas like no file format to markdown file

Summary by CodeRabbit

New Features

Introduced several new project ideas, including:

Grant Distribution Visualiser
Gov Forum LLM Summarizer
OP Delegation Frame
Retro Funding GitHub Badge
Push Notifications for Tokenholders
Splits Contract for Retro Funding Distribution

Improvements

Enhanced formatting and consistency across multiple documents.
Improved readability with clearer organization and structured lists.

Documentation

Updated descriptions and titles for better clarity and presentation in contributions documentation.

@Billy19191 Not sure if we should fix the grammar or leave it as-is?

Leave it as it is fine, we don't need to care about grammatical


ERR_PACKAGE_IMPORT_NOT_DEFINED on 4.16.0

Acknowledgements

[X] I read the documentation and searched existing issues to avoid duplicates
[X] I understand this is a bug tracker and anything other than a proven bug will be closed
[X] I understand this is a free project and relies on community contributions
[X] I read and understood the Contribution guide

Minimal reproduction URL
https://github.com/topology-foundation/ts-topology/tree/ae148aa031edcc0c139db8eca22abf4d95d593aa
Problem & expected behavior (under 200 words)
Upgrading to 4.16.0 (from 4.15.7) breaks with the following message:

I lack the understanding of ESM to debug it further.
The other dependency that is involved with the breaking is uint8arrays
Bugs are expected to be fixed by those affected by it

[ ] I'm interested in working on this issue

Compensating engineering work will speed up resolution and support the project

[X] I'm willing to offer $10 for financial support

The reproduction must be minimal


Add icon_highlight option

What do you think of this option?
  AnyName = {
    ...
    icon = '  ',
    icon_highlight = {colors.green, colors.red},
    ...
  }

@Stanislav-Lapata
Hi I designed this keyword in my initial design, but I found that it didn't make much sense. Because you can define a new section to implement it like this, It just makes the section longer. I want to hear the opinions of more people. Do we need this field, it is easy to implement.
gls.left[1] = {
   SomeIconFiled = {
         provider = function () return  icon end
          highlight = color in here
}

@glepnir You are probably right. Thanks for the quick response

I would find this option useful. The reason is that the separator appears even if the condition is false. Instead the icon only if it is true. So, I could use the icon as a separator, just coloring it.

PR welcome :)


Add createAnimatedComponent to mock

Goal
Following https://github.com/kmagiera/react-native-reanimated/issues/355 , createAnimatedComponent is not present in mock.js file and makes jest test crashes.
Issue Example

Makes sense, thanks for sending this patch!


Remove LLVM dependencies from "common" C++ code [phab:T91]

Created by mazurak at 2016-01-21 20:42:17:
There are a bunch of little dependencies on LLVM within the kythe/cxx/common (e.g., StringRefs), even though nothing says that a language whose indexer is written in C++ necessarily cares about LLVM at all. It would be nice if this dependency could be broken.

schroederc wrote at 2017-05-31 23:09:43:
I believe this is fixed, right?

zarko wrote at 2017-05-31 23:14:50:
It's not. The biggest blocker is the lack of std::string_view. We also use some of LLVM's path-manipulation APIs.


Scaffold app for landing page

Task Description
Scaffold the initial app for the landing page, using NextJs, deploy and connect to the main domain.
Technical Considerations
Additional Considerations

Resolved in https://github.com/Kerosz/biolnk/commit/06d2c618c73bb2df715bb4ebfac4ab994f937535


How can I view the BEV  and point cloud

good work!
But how can I view the top-down view and point cloud of vehicles as well as its trajectories in the evaluation dataset?
like this:

https://github.com/autonomousvision/transfuser/blob/a7d4db684c160095dec03851aff5ce92e36b2387/team_code_transfuser/model.py#L935
https://github.com/autonomousvision/transfuser/blob/a7d4db684c160095dec03851aff5ce92e36b2387/team_code_transfuser/model.py#L719
Set the debug flag and a SAVE_PATH

I have run local_evaluation.sh，but Carla still only has a top-down view.
like this:


add apigw intent apigw

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


seeyou/reader: Use style: 0 for unknown waypoint types

The specification states:
"Waypoint style describes the type of the waypoint. If a value other than the ones listed below is found in the file the parser should default it to 0."
Also, 0 is mentioned as a valid value ("unknown")

one more small suggestion: would be great if you could add a unit test for this change 🙏


Fix cddl format for shelley and byron eras

Fixes: #3688

Verified that this fixes https://github.com/input-output-hk/cardano-node/issues/3688 by signing & submitting Shelley-era Tx with CDDL formatted body.

bors r+


Support node scans when variables have the same label and property

https://github.com/opencypher/cypher-for-apache-spark/blob/3afbaabb55d6ec808e7b181f80911e332fef8aa1/src/test/scala/org/opencypher/caps/api/spark/CAPSPatternGraphTest.scala#L389

Not sure I understand this, could you please add some details?

It's linked to a test where an employee and a person both have a property name, and in one case it's nullable. That test is ignored, because it currently fails for something that is both an employee and a person.


Add the translation of recommended beatmaps [RU]

run the generator pls

@Flutterish

Oh, yes, thank you, I'll update the current releases additional files to include this locale change.


Convert wallet text truncated

When converting an old wallet to encrypted key, the text under the password feild is cut off on iPhone 7 Plus
This password will be used to encrypt your priva....

Already fixed


golint incorrectly reports that an exported constant/variable is uncommented

test.go:
package test

const Test = 1 // comment

test.go:3:7: exported const Test should have comment or be unexported

And if you replace const with var:
test.go:3:5: exported var Test should have comment or be unexported

// Test should not trigger a lint warning if the comment immediately precedes the variable.
const Test = 1

Yep, what @corburn said. It's mandated that way because that's the form that godoc extracts.

I must differ, because godoc.org shows these comments just fine.

https://godoc.org/github.com/opennota/elvenshtein

At the very least, I'd like golint not to tell me outright lies, please.

Please reopen.

+1
it uglifies blocks of constants


npm ERR! command sh -c -- sh scripts/postinstall.sh

when i install @heartexlabs/label-studio
npm install @heartexlabs/label-studio
npm ERR! command failed
npm ERR! command sh -c -- sh scripts/postinstall.sh
i guess the build files lost scripts, i thank we need add it;

or   npm install @heartexlabs/label-studio --ignore-scripts

this bug in   @heartexlabs/label-studio v1.8.0

lost build/

same here

Although the installation was successful, it was not available

when will it be solved?

Same problem. Installing with --ignore-scripts seems to not build, which leads to @heartexlabs/label-studio not found while trying to import.

Same here. Any update?

Same issue here trying to set up a JS project in Visual Studio 2022 with this npm package.

Same issue


Ignore _site/ files

How do we get Git to ignore _site/ ...html, txt, json, xml files?
The live site looks fine without committing these.
.gitignore:
.Rproj.user
.Rhistory
.RData
.Ruserdata

/.quarto/
*_cache/
_site/

.DS_Store

my RStudio git pane

This is very likely because the _site directory was committed at one point in the past, before it was added to .gitignore.  Git will keep track of files that it has previously tracked even after they've been ignored.
This should fix it:
First make sure there are no uncommitted files, then:
# remove files from git's index (but leave them on your disk)
git rm -r --cached _site

git add .

git commit -m "stop tracking _site/ dir"

git push

Thanks Andy! That deleted a fair few _site/ files. Looking forward to them being ignored in my next set of edits


Leverage go_update_deps from test-infra

Problem
Similar to https://github.com/knative/pkg/pull/1792 and https://github.com/knative/eventing/pull/4271, we should leverage the new script.
Persona:
Which persona is this feature for?
Developer
Exit Criteria
Use go_update_deps
Time Estimate (optional):
How many developer-days do you think this may take to resolve?
1
Additional context (optional)

Original PR: https://github.com/knative/test-infra/pull/2492

/assign @slinkydeveloper


The magic kingdom

I will go through  at some point and redo that adventure in a different order, but I found the daughter before I ever found the king, because I used power to get through the boulder. so when I found the king the quest both started, and completed  then it wouldn't let me save, and said error saving player data. I had to close and reopen the browser and it reverted to before I had entered the adventure it seems

so I ended up finding out how the adventure was meant to be completed. and also that the football player was not supposed to be a follower? when I went in with power they just auto followed me. when I went in the second time they didn't
and the game did actually save this time

Some NPCs, like the baseball player and the hermit in Beginner's Cave, have a random friendliness. If you play the same adventure more than once, you might have them follow you one time and not another time. The higher your character's charisma, the more likely they are to follow.
Also, the power spell has random effects. In some adventures, it can teleport you to a random room. In the case of the Magic Kingdom, I'll need to fix it so that it won't teleport you into the mine, because that causes the quest to get out of order.

I fixed the Power spell so it won't teleport the player into the mine or onto the river.
For kicks, I tried rescuing the princess without meeting the king first (by cheating). When I subsequently met the king, I saw both the beginning and end of the quest at the same time, and was sent straight to the main hall, which is logical.


[#844] R Script resource type created

@danames You or someone else can look at this and make sure it is set up how you intend it to be. I added all of the specific metadata you asked me to. After testing, all appears to be functional.

Can one of the admins verify this patch?

@ironfroggy @danames @horsburgh I generalized the resource to be a ScriptResource. This required changing the namespace everywhere (renamed directory, classes, variables, etc.). The language metadata element is no longer editable, but is automatically generated based upon the upload file type.

@ironfroggy and @hyi Can one of you review this PR?

With the Jenkins build failing is this PR actually ready to review? A failing build should be resolved before code review.

A failing build should be resolved before code review.

Absolutely correct - It's possible that folks are still adjusting to having a build/test done for each PR.
The Details link will direct developers to the errors as captured by the Jenkins run.
We should review this in our next team call to ensure all are on the same page.
Link to failed tests: http://ci.hydroshare.org:8080/job/pull-request-builder/302/testReport/

@shawncrawley Fix the failed unit tests for this PR to be reviewed.

@pkdash Changes have been made.

@ironfroggy Seems it is now ready for review.

looks much better to me


Update exclude arg to accept RegExp

Background
https://github.com/kamilkisiela/graphql-hive/issues/3493
There are certain operations that are prefixed or suffixed in our system that we use internally, and we have no benefit from posting this to Hive.
In order to not make this a flaky list that always needs updating, it would be nice if we can match on regexp values along with perfect string matches.

Description
Updating the exclude argument type to accept strings and RegExps

Checklist

[ ] Input validation
[ ] Output encoding
[ ] Authentication management
[ ] Session management
[ ] Access control
[ ] Cryptographic practices
[ ] Error handling and logging
[ ] Data protection
[ ] Communication security
[ ] System configuration
[ ] Database security
[ ] File management
[ ] Memory management
[ ] Testing

@kamilkisiela 👋 Would you be able to give this one a review? Thank you!

@kongMina can you please also push a changeset file? (run pnpm changeset in the root and it will guide you). I think we should do a minor bump for client package in this case.


[aura] block broadcast may not be blocking

We observed the issue that SCV2 died during sidechain block import
https://github.com/integritee-network/deployment/issues/94
This lead to a stall in block production of SCV1
killing SCV2 woke up SCV1 and sidechain block production recovered

I don't think the current implementation is actually blocking:
this seems to imply that broadcast is blocking:
https://github.com/integritee-network/worker/blob/7ff60be29dd0177d1fd96276ecf41b4fe4767abd/service/src/sync_block_broadcaster.rs#L53-L55
But then, threads are spawned for each peer:
https://github.com/integritee-network/worker/blob/7ff60be29dd0177d1fd96276ecf41b4fe4767abd/service/src/worker.rs#L101-L105
therefore, I don't think a peer can cause the hang

this seems to be solved by #1558. reopen if it happens again


images/haveImageID: use docker images, not inspect

images/haveImageID: use docker images, not inspect

Docker inspect will not exit 0 if no version of the image is available
on the machine, leading to the method failing. There is no flag to
adjust this in docker inspect, the only solution would be to parse the
error message but that would be fragile.

Instead, we opt to use `docker images` which has a very similar output
with the exception that it will list all tags available for that image
and that it will not fail on absent images.

This fixed HEIC thumbnails on my machine as I don't have a recent verion of imagemagick.
Initially I submitted another PR trying to adjust the docker inspect flags but then I realised that my PR had fixed nothing, it just started working because I had manually pulled the image.

@bradfitz review plz!

@bradfitz review plz!


Fix the style of badge and checkbox of the filter

Description
Please provide a summary of the pull request and the issue it fixes. Please add necessary details, context, dependencies, explanation of when review is needed (see next section), etc.
Fixes #1011
Review Time Estimate
Please give your idea of how soon this pull request needs to be reviewed by selecting one of the options below. This can be based on the criticality of the issue at hand and/or other relevant factors.

[ ] Immediately
[x] Within a week
[ ] When possible

Type of changes
Please select a relevant option:

[x] Bug fix (non-breaking change which fixes an issue).
[ ] New feature (non-breaking change which adds functionality).
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected).
[ ] Other (any another change that does not fall in one of the above categories.)

Checklist:
Please select all applicable options:

[ ] I have signed the Rokwire Contributor License Agreement (CLA). (Any contributor who is not an employee of the University of Illinois whose official duties include contributing to the Rokwire software, or who is not paid by the Rokwire project, needs to sign the CLA before their contribution can be accepted.)
[x] I have updated the CHANGELOG.
[ ] I have read the Contributor Guidelines.
[x] I have performed a self-review of my own code.
[ ] I have commented my code, particularly in hard-to-understand areas.
[ ] My change requires updating the documentation.
[ ] I have made necessary changes to the documentation.
[ ] I have added tests related to my changes.
[ ] My changes generate no new warnings.
[ ] New and existing unit tests pass locally with my changes.
[ ] Any dependent changes have been merged and published in downstream modules.

Looks good. Please fix merge conflict.


Add key-value argument for dynamic feed config

Resolves #33

This is better done with an external tool/script to dynamically update the config file.


Getting Plugins to work with the kit

I need to use some plugins but am having issues getting them to work with the videojs-mux-kit version.
I have seen the note at the end of the readme regarding video.js/core situation. I've tried both solutions:

using the VHS build - plugins still don't work and Mux video playback is problematic when loading video source using javascript
using the alias plugin for my vite.config.js but that seems to have no effect (no vite compile errors but I can't work out how to test if the alias are actually loading the core version)

import alias from '@rollup/plugin-alias';

export default defineConfig({
    plugins: [
        alias({
             entries: [
                 { find: 'video.js', replacement: resolve.resolve('video.js/core') },
             ]
        }),
    ],
});

I guess what I'm really interested in knowing is has anyone got plugins to work with the latest version of videojs-mux-kit?

Hey, thanks for the issue. There's definitely some finickiness around plugins. We'll take a look next week.

Looking at the vite docs, I can see that they actually ship with it built-in: https://vitejs.dev/config/shared-options.html#resolve-alias
However, I fonud out why this isn't working. We're bundling video.js into the built file, so, there is always two versions of Video.js around. I'll take a look at how to separate it out so that aliasing will start working again.

Figured out a workaround. If you alias video.js to point at videojs-mux-kit itself, it works:
import { defineConfig } from 'vite';

export default defineConfig({
  resolve: {
    alias: {
      'video.js': '@mux/videojs-kit',
    },
  },
});

I have a live example over here: https://stackblitz.com/edit/vitejs-vite-v8dbe2?file=main.js,package.json,vite.config.js&terminal=dev
@Progression hopefully, that unblocks you. I'll continue looking into how to make the build better, though. I'm not sure that there's a way to avoid aliasing altogether, unfortunately.

Gary, that is great, thanks so much for looking into this. I'm unable to try this out myself tonight but will see if I can jump over to this tomorrow and get it working. Will let you know.

@gkatsev's solution also worked for me running webpacker in rails with the videojs-record plugin.
I changed the alias from  'video.js': 'video.js/core' to 'video.js': '@mux/videojs-kit' and it finds the plugin and works.


add predicates endpoint

https://github.com/NCATS-Tangerine/NCATS-ReasonerStdAPI/blob/master/API/TranslatorReasonersAPI.yaml
This page indicates that 2 services should be available: query and predicates.  The latter is a GET method.

I'll take this on; it might be a chance to put in an example of the sparql"" interpolator.

This was completed, see #146.

This was completed, see #146.


Optimum Not Working With LongT5 Summarization

System Info
- `optimum` version: 1.2.3 (installed via Github installation)
- `transformers` version: 4.20.1
- Platform: Linux-5.4.188+-x86_64-with-Ubuntu-18.04-bionic
- Python version: 3.7.13
- Huggingface_hub version: 0.8.1
- PyTorch version (GPU?): 1.11.0+cu113 (False)
- Tensorflow version (GPU?): 2.8.2 (False)
- Flax version (CPU?/GPU?/TPU?): not installed (NA)
- Jax version: not installed
- JaxLib version: not installed
- Using GPU in script?: no
- Using distributed or parallel set-up in script?: no

Who can help?
@lewtun @michaelbenayoun @JingyaHuang
Information

[ ] The official example scripts
[X] My own modified scripts

Tasks

[ ] An officially supported task in the examples folder (such as GLUE/SQuAD, ...)
[X] My own task or dataset (give details below)

Reproduction
Essentially, I am unable to run LongT5 for anything longer than about ~25 tokens with the summarization pipeline. I get an error regarding a LessOrEqual node ending in "Can broadcast 0 by 0 or 1. num is invalid", where num is a value that increases the longer the sequence I provide as input to the pipeline.
If it helps, I get the same error when using Huggingface directly with !python -m transformers.onnx --model=my-custom-pretrained-summarizer --feature seq2seq-lm.
!pip install transformers
!pip install transformers[onnx]
!python -m pip install git+https://github.com/huggingface/optimum.git
!python -m pip install git+https://github.com/huggingface/optimum.git#egg=optimum[onnxruntime]
!pip install datasets
from optimum.onnxruntime import ORTModelForSeq2SeqLM
model = ORTModelForSeq2SeqLM.from_pretrained("my-custom-pretrained-summarizer", from_transformers=True)
Output:
/usr/local/lib/python3.7/dist-packages/transformers/models/longt5/modeling_longt5.py:180: TracerWarning: torch.tensor results are registered as constants in the trace. You can safely ignore this warning if you use this function to create tensors out of constant variables that would be the same every time you call this function. In any other case, this might cause the trace to be incorrect.
global_block_ids_lower_bound = torch.tensor(-1.0, dtype=global_block_ids.dtype, device=global_block_ids.device)
/usr/local/lib/python3.7/dist-packages/transformers/models/longt5/modeling_longt5.py:188: UserWarning: floordiv is deprecated, and its behavior will change in a future version of pytorch. It currently rounds toward 0 (like the 'trunc' function NOT 'floor'). This results in incorrect rounding for negative values. To keep the current behavior, use torch.div(a, b, rounding_mode='trunc'), or for actual floor division, use torch.div(a, b, rounding_mode='floor').
num_globals = seq_len // global_block_size
/usr/local/lib/python3.7/dist-packages/transformers/models/longt5/modeling_longt5.py:190: TracerWarning: Converting a tensor to a Python boolean might cause the trace to be incorrect. We can't record the data flow of Python values, so this value will be treated as a constant in the future. This means that the trace might not generalize to other inputs!
if num_globals > 0:
/usr/local/lib/python3.7/dist-packages/transformers/models/longt5/modeling_longt5.py:217: TracerWarning: torch.tensor results are registered as constants in the trace. You can safely ignore this warning if you use this function to create tensors out of constant variables that would be the same every time you call this function. In any other case, this might cause the trace to be incorrect.
block_ids >= 0, torch.tensor(global_seq_len, dtype=block_ids.dtype, device=block_ids.device)
/usr/local/lib/python3.7/dist-packages/transformers/models/longt5/modeling_longt5.py:217: UserWarning: To copy construct from a tensor, it is recommended to use sourceTensor.clone().detach() or sourceTensor.clone().detach().requires_grad(True), rather than torch.tensor(sourceTensor).
block_ids >= 0, torch.tensor(global_seq_len, dtype=block_ids.dtype, device=block_ids.device)
/usr/local/lib/python3.7/dist-packages/transformers/models/longt5/modeling_longt5.py:219: TracerWarning: Converting a tensor to a Python integer might cause the trace to be incorrect. We can't record the data flow of Python values, so this value will be treated as a constant in the future. This means that the trace might not generalize to other inputs!
one_hot_block_ids = nn.functional.one_hot(block_ids.type(torch.int64), global_seq_len + 1)[:, :, :-1]
/usr/local/lib/python3.7/dist-packages/transformers/models/longt5/modeling_longt5.py:84: TracerWarning: Converting a tensor to a Python boolean might cause the trace to be incorrect. We can't record the data flow of Python values, so this value will be treated as a constant in the future. This means that the trace might not generalize to other inputs!
if x.shape[dim] % block_len != 0:
/usr/local/lib/python3.7/dist-packages/transformers/models/longt5/modeling_longt5.py:67: TracerWarning: Converting a tensor to a Python boolean might cause the trace to be incorrect. We can't record the data flow of Python values, so this value will be treated as a constant in the future. This means that the trace might not generalize to other inputs!
if not all(x.shape):
/usr/local/lib/python3.7/dist-packages/transformers/models/longt5/modeling_longt5.py:86: UserWarning: floordiv is deprecated, and its behavior will change in a future version of pytorch. It currently rounds toward 0 (like the 'trunc' function NOT 'floor'). This results in incorrect rounding for negative values. To keep the current behavior, use torch.div(a, b, rounding_mode='trunc'), or for actual floor division, use torch.div(a, b, rounding_mode='floor').
num_blocks = x.shape[dim] // block_len
/usr/local/lib/python3.7/dist-packages/transformers/models/longt5/modeling_longt5.py:89: TracerWarning: Converting a tensor to a Python boolean might cause the trace to be incorrect. We can't record the data flow of Python values, so this value will be treated as a constant in the future. This means that the trace might not generalize to other inputs!
if 0 in output_shape:
/usr/local/lib/python3.7/dist-packages/transformers/modeling_utils.py:770: FutureWarning: The device argument is deprecated and will be removed in v5 of Transformers.
"The device argument is deprecated and will be removed in v5 of Transformers.", FutureWarning
/usr/local/lib/python3.7/dist-packages/transformers/modeling_utils.py:781: TracerWarning: Converting a tensor to a Python boolean might cause the trace to be incorrect. We can't record the data flow of Python values, so this value will be treated as a constant in the future. This means that the trace might not generalize to other inputs!
if causal_mask.shape[1] < attention_mask.shape[1]:
from transformers import AutoTokenizer, pipeline
tokenizer = AutoTokenizer.from_pretrained('google/long-t5-tglobal-large')
onnx_summarization = pipeline("summarization", model=model, tokenizer=tokenizer)
text = # Something longer than ~25 tokens
pred = onnx_summarization(text)
/usr/local/lib/python3.7/dist-packages/onnxruntime/capi/onnxruntime_inference_collection.py in run(self, output_names, input_feed, run_options)
190             output_names = [output.name for output in self._outputs_meta]
191         try:
--> 192             return self._sess.run(output_names, input_feed, run_options)
193         except C.EPFail as err:
194             if self._enable_fallback:
RuntimeException: [ONNXRuntimeError] : 6 : RUNTIME_EXCEPTION : Non-zero status code returned while running LessOrEqual node. Name:'LessOrEqual_648' Status Message: /onnxruntime_src/onnxruntime/core/providers/cpu/math/element_wise_ops.h:603 onnxruntime::Broadcaster::Broadcaster(gsl::span, gsl::span) largest <= 1 was false. Can broadcast 0 by 0 or 1. 16 is invalid.
Expected behavior
[{'summary_text': '<something that should look the same or very similar to what my PyTorch model outputs'}]

cc @echarlaix

Hi @reelmath,
I was not able to reproduce the error you are describing by running the following code:
from optimum.onnxruntime import ORTModelForSeq2SeqLM
from transformers import AutoTokenizer, pipeline

model_name = "google/long-t5-local-base"
model = ORTModelForSeq2SeqLM.from_pretrained(model_name, from_transformers=True)
tokenizer = AutoTokenizer.from_pretrained(model_name)
onnx_summarization = pipeline("summarization", model=model, tokenizer=tokenizer)
text = # Something longer than ~25 tokens
pred = onnx_summarization(text)

Can you confirm whether you are able to run it ? (to determine whether the problem comes from the specific model you are exporting)

I was able to reproduce your error with the model google/long-t5-tglobal-base, it looks like the issue comes from the ONNX export. When exporting the model, the default sequence length is 8, and during the model tracing num_globals > 0 is being converted to False, which results in errors for inputs with sequence length higher than global_block_size*2. If you need a LongT5 model with transient-global attention, you can try to increase the sequence length when exporting the model (higher than global_block_size), but it might results in problems for short sequence length (< global_block_size). Because this issue concerns the ONNX export, could you open an issue on transformers in order to move the discussion there?


Fork SFML and upgrade to SFML 2.4?

I managed to fix #81 by upgrading to SFML 2.4. I needed to hand patch SFML.NET though.
I think in the mean time for Silver to finish Robust it might be a good idea to fix SFML, because this means it's segfault free on MacOS.

👍


Error: Can't set headers after they are sent.

Below are the error :
Error: Can't set headers after they are sent.
at ServerResponse. (http.js:527:11)
at ServerResponse.setHeader (/node_modules/connect/lib/patch.js:47:22)
at ServerResponse.header (/node_modules/express/lib/response.js:281:10)
at ServerResponse.json (/node_modules/express/lib/response.js:134:8)
at ServerResponse.send (/node_modules/express/lib/response.js:78:21)
at Object.callback (/var/www/TGV/lib/controller.js:60:22)
at /var/www/TGV/lib/db/dbGo.js:40:23
at Query.execFind (/var/www/TGV/node_modules/mongoose/lib/mongoose/query.js:633:12)
at Query.find (/var/www/TGV/node_modules/mongoose/lib/mongoose/query.js:91:15)
at Function.find (/var/www/TGV/node_modules/mongoose/lib/mongoose/model.js:409:16)
Error: Can't set headers after they are sent.
at ServerResponse. (http.js:527:11)
at ServerResponse.setHeader (/node_modules/connect/lib/patch.js:47:22)
at next (/node_modules/connect/lib/http.js:162:13)
at next (/node_modules/connect/lib/http.js:195:11)
at param (/node_modules/express/lib/router/index.js:230:9)
at param (/node_modules/express/lib/router/index.js:224:11)
at param (/node_modules/express/lib/router/index.js:224:11)
at pass (/node_modules/express/lib/router/index.js:232:6)
at Router._dispatch (/node_modules/express/lib/router/index.js:255:4)
at Object.handle (/node_modules/express/lib/router/index.js:45:10)
Error: Can't set headers after they are sent.
at ServerResponse. (http.js:527:11)
at ServerResponse.setHeader (/node_modules/connect/lib/patch.js:47:22)
at next (/node_modules/connect/lib/http.js:162:13)
at next (/node_modules/connect/lib/http.js:207:9)
at param (/node_modules/express/lib/router/index.js:230:9)
at param (/node_modules/express/lib/router/index.js:224:11)
at param (/node_modules/express/lib/router/index.js:224:11)
at pass (/node_modules/express/lib/router/index.js:232:6)
at Router._dispatch (/node_modules/express/lib/router/index.js:255:4)
at Object.handle (/node_modules/express/lib/router/index.js:45:10)
Error: Can't set headers after they are sent.
at ServerResponse. (http.js:527:11)
at ServerResponse.setHeader (/node_modules/connect/lib/patch.js:47:22)
at next (/node_modules/connect/lib/http.js:162:13)
at param (/node_modules/express/lib/router/index.js:230:9)
at param (/node_modules/express/lib/router/index.js:224:11)
at pass (/node_modules/express/lib/router/index.js:232:6)
at Router._dispatch (/node_modules/express/lib/router/index.js:255:4)
at Object.handle (/node_modules/express/lib/router/index.js:45:10)
at next (/node_modules/connect/lib/http.js:198:15)
at next (/node_modules/connect/lib/http.js:150:23)
Error: Can't set headers after they are sent.
at ServerResponse. (http.js:527:11)
at ServerResponse.setHeader (/node_modules/connect/lib/patch.js:47:22)
at next (/node_modules/connect/lib/http.js:162:13)
at param (/node_modules/express/lib/router/index.js:230:9)
at pass (/node_modules/express/lib/router/index.js:232:6)
at Router._dispatch (/node_modules/express/lib/router/index.js:255:4)
at Object.handle (/node_modules/express/lib/router/index.js:45:10)
at next (/node_modules/connect/lib/http.js:198:15)
at next (/node_modules/connect/lib/http.js:150:23)
at pass (/node_modules/connect/lib/middleware/router.js:195:9)
Error: Can't set headers after they are sent.
at ServerResponse. (http.js:527:11)
at ServerResponse.setHeader (/node_modules/connect/lib/patch.js:47:22)
at next (/node_modules/connect/lib/http.js:162:13)
at next (/node_modules/connect/lib/http.js:207:9)
at next (/node_modules/connect/lib/http.js:150:23)
at pass (/node_modules/connect/lib/middleware/router.js:195:9)
at Object.router [as handle] (/node_modules/connect/lib/middleware/router.js:197:6)
at next (/node_modules/connect/lib/http.js:198:15)
at Object.fetchUserFromSession [as handle] (/node_modules/everyauth/index.js:37:27)
at next (/node_modules/connect/lib/http.js:198:15)
Error: Can't set headers after they are sent.
at ServerResponse. (http.js:527:11)
at ServerResponse.setHeader (/node_modules/connect/lib/patch.js:47:22)
at next (/node_modules/connect/lib/http.js:162:13)
at next (/node_modules/connect/lib/http.js:150:23)
at next (/node_modules/connect/lib/http.js:207:9)
at Object.fetchUserFromSession [as handle] (/node_modules/everyauth/index.js:37:27)
at next (/node_modules/connect/lib/http.js:198:15)
at Object.registerReqGettersAndMethods [as handle] (/node_modules/everyauth/index.js:31:9)
at next (/node_modules/connect/lib/http.js:198:15)
at HTTPServer.handle (/node_modules/connect/lib/http.js:211:3)
Error: Can't set headers after they are sent.
at ServerResponse. (http.js:527:11)
at ServerResponse.setHeader (/node_modules/connect/lib/patch.js:47:22)
at next (/node_modules/connect/lib/http.js:162:13)
at next (/node_modules/connect/lib/http.js:150:23)
at next (/node_modules/connect/lib/http.js:207:9)
at Object.registerReqGettersAndMethods [as handle] (/node_modules/everyauth/index.js:31:9)
at next (/node_modules/connect/lib/http.js:198:15)
at HTTPServer.handle (/node_modules/connect/lib/http.js:211:3)
at Object.handle (/node_modules/connect/lib/http.js:105:14)
at next (/node_modules/connect/lib/http.js:198:15)
Error: Can't set headers after they are sent.
at ServerResponse. (http.js:527:11)
at ServerResponse.setHeader (/node_modules/connect/lib/patch.js:47:22)
at next (/node_modules/connect/lib/http.js:162:13)
at next (/node_modules/connect/lib/http.js:150:23)
at next (/node_modules/connect/lib/http.js:207:9)
at HTTPServer.handle (/node_modules/connect/lib/http.js:211:3)
at Object.handle (/node_modules/connect/lib/http.js:105:14)
at next (/node_modules/connect/lib/http.js:198:15)
at /node_modules/connect/lib/middleware/session.js:323:9
at /node_modules/connect/lib/middleware/session.js:338:9
node.js:134
    throw e; // process.nextTick error, or 'error' event on first tick

^
Error: Can't set headers after they are sent.
at ServerResponse. (http.js:527:11)
at ServerResponse.setHeader (/node_modules/connect/lib/patch.js:47:22)
at next (/node_modules/connect/lib/http.js:162:13)
at next (/node_modules/connect/lib/http.js:207:9)
at /node_modules/connect/lib/middleware/session.js:323:9
at /node_modules/connect/lib/middleware/session.js:338:9
at Array. (/node_modules/connect/lib/middleware/session/memory.js:57:7)
at EventEmitter._tickCallback (node.js:126:26)

This is just a comment to elaborate on what happened to me,  this thread seems to be the main goto for this type of problem,  which is probably pretty common.  Even after reading the post here I still didnt really get what was happening ( since im new to this style of programming ).
I was calling "on" instead of "once" on global objects.    for instance mongoose.connection
In this case you could use "once" not "on" to handle your callbacks and it will work as you expect it.  And as a previous poster mentions,  this is more of a node thing not an express middleware thing. Basically each http request will result in another callback getting registered with "on".    And since you will most likely call next() in this anonymous callback,  the next() method gets called multiple times and then screwing up the middleware stack.


Legal Notice

We should either hide that page, or get a lawyer/someone who understands their stuff to properly do it for us.
Having a half-baked legal page seems like a liability or whatever.

Yeah - this is pretty solid advice


additional libraries

I needed to additionally require the stringr, tibble, and tidyr packages (clearly, oldschool and not a tidyverse regular here).
This is amazing, and saved me a ton of time -- thank you.

No worries, glad its useful.


Use ODES API

[x] Show existing ODES extracts for a logged-in user.
[x] Make it possible to create new ones.

How to test this:

Visit https://mapzen-odes-extracts-pr-59.herokuapp.com/odes/
Hit submit, no need to change the bbox values for testing.
Authenticate with Mapzen.
See your new extract request at https://mapzen-odes-extracts-pr-59.herokuapp.com/odes/extracts/.


Simplify EventPart

No need to register the part as a listener, just register the listener.
And fixes bugs with updating the event listener (instead of having to render it null first).

This looks like a perf regression. Consider that we're usually in a render loop. With this patch, inline event handlers will cause a remove / add eventListener dance, don't they?

@kaste There is a small change here - previously the EventPart itself was passed to addEventListener, so it was stable, but it's _listener property was set with the new user-provided listener on every update. With this PR the intermediate event handler is gone, so instead of property sets each update, add/removeEventListener is indeed called each time.
Admittedly, we didn't measure the perf impact here. My assumption is that inline handlers are pretty bad in this case because you're already creating a new closure each update, and the correct way to do this, which is to create the handler outside of the template, will eliminate thrashing in the code before and after this PR.
We could add back the stable handler though, if this is a problem. This PR was just a nice simplification... did you measure an impact here?

I could measure an impact albeit only in benchmarks of course.
But actually the observation was the other way around. I first observed that you don't do the add/remove dance all the time, and thought That's nice, I just keep my inline handlers here. 😄
Creating closures itself is fast enough, maybe too much pressure on GC.

Yeah, it's possible that closure creation is enough faster than add/RemoveEvent listener that this is a meaningful regression for inline handlers. I'm definitely open to reviving the old approach.


Django 1.4 - minor compatibility issue

Thanks for this pull request.
I'm not really a fan of try...except... where it isn't required, so maybe this should use getattr

updated - please look again.

Alright. Now had a closer look. Does Django 1.4 just remove that attribute? It's not replaced with anything?

appears to have been removed and not replaced
compare 1.3.1 against trunk:
https://code.djangoproject.com/browser/django/tags/releases/1.3.1/django/template/loader_tags.py#L77
https://code.djangoproject.com/browser/django/trunk/django/template/loader_tags.py#L75

and...
django_trunk/django$ grep -r parent_name_expr *
django_trunk/django$


Update loadDrugMechDB.py to include treats edges for indication paths

Each indication path in drugmechdb describes the COP for a drug-disease indication pair, but the "treats" edge is not explicitly included. I edited the parser to introduce these as "biolink:treats" edges.

Looks like this needs a refactor.

Given the "treats"/"ameliorates" edge refactor, I hesitate to approve this PR. I suggest we look at it more before merging it into main.

From the DrugMechDB paper: "DrugMechDB was constructed considering drug-disease indications from the DrugCentral
database, using the version downloaded on September 18, 2020"
So each drug-disease pair in each path in the source file could be considered an indication, at least as of 2020.

@beasleyjonm, how about we use the biolink:treats_or_applied_or_studied_to_treat for now until we can dig in to this some more?

Let's do that, since it's not clear that they update this regularly with latest indications on drug central and they state they used the 2020 version. It's possible some of the indications could have changed since then, so "treats_or_applied_or_studied_to_treat" seems the safest option.


why is 01481500 "tidal"?

@jsadler2, do you remember how you determined 01481500 is tidal?
Originally posted by @lekoenig in https://github.com/USGS-R/drb-do-ml/pull/40#discussion_r801115087

Huh. I'm glad you brought this up, @lekoenig. When I look at the specific conductance data I'm not seeing a tidal signal after all. I'm not sure why I thought that it was tidal originally, but looking at this, it seems like it's not.

Thanks for taking another look at this site. So are we in agreement to add "01481500" back in the mix?

I agree


Blinking widgets

https://github.com/Shopify/dashing/blob/master/templates/project/assets/stylesheets/application.scss#L156 I might have lost the oversight but what was the thought behind the blinking of the widgets? This just drove me nuts running dashing on a raspi for > 10mins...

This is intended for the Number widget. A status field can be added to the event data which can be used as a visual indicator of an important metric.
If you are annoyed by this feature, I suggest that you do not use it. The template dashboard does not implement this functionality.
This issue can be closed.


filter model missing $notin

I think its useful for ID model filters to have an $notin filter type, so I can exclude results based on an array of IDs

thank you @mxdi9i7 for your suggestion!
I think it makes complete sense to add, but can you please add a feature request over here: https://github.com/datocms/product-roadmap/issues/new?assignees=&labels=&template=feature_request.md&title=
This repository is only meant for our public website.
Thanks!


[WIP] CI/CD: build binary for Windows

Watchexec has a nice release.yml that deals with builds on Windows -- and incidentally producing .deb packages, which is appreciable.
Hope that helps


update k8s library versions

k8s 1.19.16, 1.20.15, 1.21.9, 1.22.6 and 1.23.2
Signed-off-by: André Martins<EMAIL_ADDRESS>

/test-backport-1.11


Panic when respawning a chunk with collisions

Bevy Tilemap version
Git master
Operating system & version
Ubuntu 20.04
Note : I have a slightly modified version of bevy_tilemap, it should not have any impact on this bug.
Edit : tested on master, same issue.
What you did
It seems that respawning a chunk with collisions that was previously despawned (using the auto_spawn feature) panics with
thread 'main' panicked at 'called 'Result::unwrap()' on an 'Err' value: NoSuchEntity', /home/timo/.cargo/registry/src/github.com-1ecc6299db9ec823/bevy_transform-0.4.0/src/hierarchy/child_builder.rs:65:22
The panic seem to be in the push_children command, probably the one used in spawn_collisions, where the chunk entity doesn't exist.
I tried to force the collision system to run after the tilemap system, it did not fix it.
Here is a slightly modified version of the physics_dungeon example where the bug can be seen. You just have to go far in one direction then come back for the crash to happen.
physics_dungeon.txt

Thanks, I need to build an example where I can replicate this and can get around to it.

I started to try to understand the code to fix it this morning, and I have a possible fix / behavior change which incidentally avoid the problem.
I found another issue while testing, where the collision tile positions are wrong on chunks other than 0,0, due to the fact that their transform is offset and then they are parented to the chunk which adds the offset again.
I tried to solve it by removing the offset, and the rapier debug meshes spawned in the right place, but somehow the collisions were still wrong (maybe rapier doesn't play nicely with parenting ?)
So instead, I suggest the collisions are parented to the tilemap directly (that works on my fork, and fix this issue and the offset problem I just mentioned).
The collision tiles will have to be manually despawned when a chunk is despawned however. I will add this and then make a PR.

Closed with #124


how to add custom structures?

i dont know how to make .structure files i tried searching it and it just showed me structure blocks

help

Issues are for bug reports/feature requests, not receiving help. If you need help, join the Discord. See the Working with Structures page of the Wiki.


Revert "moved to pharmacy module(backend) the logic for creating prescription…"

Reverts esaude/esaude-emr-poc#215
To add missing tasks
@ynurmahomed Fetch these specific PR

Coverage decreased (-0.1%) to 25.412% when pulling 8df37b9c3e33795bb37f1dbc5acb4f2bde9e98a8 on revert-215-master into 50fa71f9592c2b33fb6aa7e92dda5c9f50699562 on master.

Coverage increased (+1.4%) to 26.887% when pulling 2747488bbd21274dc18482dd74a9fe1cae053e1d on revert-215-master into 50fa71f9592c2b33fb6aa7e92dda5c9f50699562 on master.

Yes...1.6%
Clean code + unit tests
@ynurmahomed pizza is on me


Top sidebar in context issue

I have a top sidebar placed in context in a div.
However, the context div sets a height automatically the size of the viewport. The behavior I wish was to have the context expands vertically as the sidebar appears.
It seems the context use was designed for left/right sidebars only.
Ive not seen in context uses of top/bottom sidebar in docs. Is this actually possible?
Check fiddle:
http://jsfiddle.net/ryf0m2f5/2/

Just follow the example html..
http://jsfiddle.net/z32jna9w/


Ribbon bar on MacOS

Hi, thanks for this widget.
I tried to use it on MacOS Big Sur, but running the example provided, the ribbon bar is not rendering at all.
I think the problem is related to the "native look", which on QMenuBar can be deactivated by setNativeMenuBar(False).
Unfortunately, calling that method on your RibbonBar raises NotImplementedError.
As a workaround, I tried to change ribbonbar.py and extending QWidget insetad of QMenuBar.
With this change I am able to see the widget, but now the problem are that some properties like margins, height, etc. are not respected (see screenshot).

Thanks,
Eros

What do you expect to see for properties like margins and height?

As you can see in the screenshot, that margin around the widget is wrong for a RibbonBar.
Moreover, setting the height is not working.
Still, I think that the correct fix for MacOS is to make the widget working as a QMenuBar

As you can see in the screenshot, that margin around the widget is wrong for a RibbonBar.
Moreover, setting the height is not working.
Still, I think that the correct fix for MacOS is to make the widget working as a QMenuBar

Sorry I don't have a MAC device, if you have any thoughts, feel free to open pull requests.


Platform Repo: Install a cluster component from a Helm Chart

As a platform operator I want to be able to use a helm chart to install services into my cluster for use by all the application teams.
Example:
https://github.com/prometheus-community/helm-charts/tree/main/charts/kube-prometheus-stack
Acceptance Criteria:

[ ] Validate PR (that it exists and contains the helm chart/release)
[ ] Library support for PR approval, merge, etc.
[ ] Chart/Release shows up in cluster after commit and is healthy

Helm release must be defined already in in an application repository somewhere.   App.yaml points to the right repo.


Specify maximum baseline and reporting period lengths

Minimum baseline and reporting period lengths are defined in Caltrack’s data sufficiency requirements, but using long baseline/reporting periods results in significantly different model fits than constrained periods (due to naturally occurring savings, non-routine events etc.)
We propose setting a limit on the data included in the baseline and reporting periods: 12-months for daily data and 24 months for monthly data.

HEA has found that 18 months of daily data is optimal for heating and cooling regressions. This amount provides more accuracy than just 12 months, but does not overwhelm recent trends (a risk with longer periods).

Interesting. It's a little counterintuitive to select 18 months (given the likelihood of over-fitting to which ever season gets counted twice). Do you have test results that show why 18 months yields better results?

Is any thought given to known  or predicted  variations in operational characteristics? For example, if you know that the building operation/schedule was recently changed, you probably wouldn't want to include the data from before the change.  (you'd need to do a preemptive non-routine adjustment)   One would prefer to have data on the operational modes that one expects to see in the reporting period.

Energy Trust's view is that data should be selected in increments of 12-months so that a seasonal bias is not introduced when fitting the model, as pointed out by McGee. We also have a preference for limiting the baseline and reporting periods to the 12 months of data closest to the treatment period to limit the impact of factors unrelated to the treatment, as noted by Hassan. You may get better fit statistics using 24 months of data, but it may not represent the pre-retrofit conditions as closely. A longer time series may contain a blend of current and past physical and operational conditions at the site.

Proposed test methodology:

This test should be run separately for daily and monthly data
Caltrack models will be fit to the baseline periods of all test buildings for different lengths of the baseline period in 3 month increments (i.e. 12, 15, 18, 21 and 24 months)
Total energy usage predictions using the baseline models will be calculated for a 12 month reporting period, following a fake project date.
Error metrics (CVRMSE and NMBE) will be calculated for these predictions as well.

Acceptance criteria:

The energy use trends and error metrics will be compared for different baseline period lengths. The minimum baseline period length that does not result in inflated out-of-sample errors will be recommended as the maximum baseline/reporting period length in the Caltrack spec.

TEST RESULTS
Background
The length of the baseline and reporting periods that are included in the savings models may affect results in two ways:

Periods that are too short may not capture the full range of input conditions (e.g. weather, occupancy) that are typically experienced.
Periods that are too long increase the chances of unexpected changes in a building’s energy use (e.g. due to a change in occupancy or in the building’s equipment).

It is generally agreed that a minimum 12 months of data should be used in order to capture at least one annual cycle of energy use. However, there are no general guidelines about the maximum length of time to include in savings analysis.
Dataset
Billing data from 1000 residential buildings in Oregon and daily data from 1000 residential buildings in California.
Tested parameters
The Caltrack methods were applied to the full datasets five times, only varying the length of the baseline period that was used to fit the models- between 12 and 24 months in 3-month increments.
Results
Figure 1 shows that the estimated normalized consumption is clearly proportional to the baseline period length for daily data. Figure 2 demonstrates that with increasing period length, the model fits, represented by R-squared, tend to get worse. This is likely because of the second effect pointed out above (greater likelihood of non-routine events that affect energy use). This monotonic increase in baseline energy use for program participants may drive an increase in estimated savings. This is an indication that the rate of naturally occurring savings in this sample may affect results when longer time periods are used for analysis. Therefore, it appears preferable to use a maximum of 12 months of data for analysis, especially with the availability of daily data.

Figure 1. Effect of baseline period length on normalized annual consumption using daily data.

Figure 2. Effect of baseline period length on model R-squared distribution. Model fits get poorer with increasing baseline period length.
When the same test was applied using billing data, no monotonic trends were obvious in the normalized annual consumption, however, the normalized annual consumption showed cyclical trends, likely corresponding to the model being weighted towards the seasons with more data. The 24-month baseline model produced a normalized estimate that was slightly higher than the 12-month baseline model.

Figure 3.  Effect of baseline period length on normalized annual consumption using billing data. Y axis (Baseline Normalized Annual Consumption) is in percent.
Recommendations
We recommend limiting the maximum baseline period length to 12 months of consumption data for both billing and daily models.

Adding some more clarification to the choice of a 12 month baseline period based on stakeholder input:
The results above are dataset-specific and do not reflect generally expected trends in estimated baseline vs. baseline period length (e.g. estimated baseline energy use will not necessarily increase if a longer baseline is used). However, they do strongly indicate that the predicted baseline may be unstable with different baseline period lengths, which may, in turn, affect calculated savings. A choice must be made by the analyst as to how long this baseline period should be. We are recommending that this choice be limited by setting the maximum baseline period at 12 months, since the year leading to the energy efficiency intervention is the most indicative of near-term energy use, making it amenable to calculating savings in pay-for-performance scenarios.

This update has been integrated in CalTRACK 2. Closing this issue


Connect command allows updating of existing organization

Overview/Summary
Solves #21
This PR fixes/adds/changes/removes
Connect-AzDevOPS now updates existing organization if executed with a organization again.
As part of this Pull Request I have

[X] Checked for duplicate Pull Requests
[X] Associated it with relevant issues, for tracking and closure.
[X] Ensured my code/branch is up-to-date with the latest changes in the main branch
[X] Performed testing and provided evidence.
[x] Updated relevant and associated documentation.

Failed to git, will create a new PR.


[Memory] fenix3 and fenix3_hr

No way around it. Not even with maxSteak=1. Proposal is to remove support from these devices.

One option to explore is reducing the maxSteaks. 1 is better than no support.


doc: release: 3.3: Add notes about Atmel and UpdateHub

Add 3.3.0 release notes for Atmel and UpdateHub
Signed-off-by: Gerson Fernando Budke<EMAIL_ADDRESS>

@nandojve This needs a rebase.

@nandojve This needs another rebase :(


Fix the hypershift install

The hypershift cli needs a kubeconfig to work now so our version command is failing. Will report an issue for that soon.

/retest

this should be fixed by https://github.com/openshift/hypershift/pull/5045

Thanks Tomas.
This is failing because of that OCM error.
Gonna hold this for now until we get that sorted out.
/hold

the hypershift cli has been fixed, this PR can be closed in favor of https://github.com/openshift/ocm-container/pull/306

/close in favour of #306


Solution Stuck on "Scoring" #1507

Submission stuck on Scoring. It happens randomly. I had to cancel last time after several hours but it counted towards my quota. What can I do:

My username is jmendoza2008 on Codabench. I believe this issue might be related to https://github.com/codalab/codabench/issues/1507.

Closing this for now.


Extend .travis.yml to build with script and push image

Build.sh file is added to build derek image with
necessary tags and extends .travis.yml to push the image
with the tags needed after successful build.
Signed-off-by: Martin Dekov (VMware<EMAIL_ADDRESS>
Add after_success to push images on release.
Description

This change extends .travis.yml file to add after_success method in which
docker image is pushed to a image hub.Also build.sh file is added to build
the image with the needed tags.
Motivation and Context

[ ] I have raised an issue to propose this change (required)

It solves https://github.com/alexellis/derek/issues/76 .
How Has This Been Tested?

Build successfuly derek with ./build.sh and pushed to my repository by changing
the alexellis prefix with martindekov prefix in the places alexellis is mentioned in building
and pushing images.
Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

[x] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[x] I've read the CONTRIBUTION guide
[x] I have signed-off my commits with git commit -s
[ ] I have added tests to cover my changes.
[x] All new and existing tests passed.

LGTM thanks


To add more product lists

Issue Description
I would like to add more products on website
Please assign this issue to me under Gssoc
Screenshots or Examples (if applicable)
No response
Checklist

[X] I have checked the existing issues.
[X] I have read the Contributing Guidelines.
[X] I have read the Code Of Conduct.
[X] The changes doesn't break the code

/assign me

It is already completed by some other PR


fix small leak and deref in offload_resume

The deref is just picking the change done by @sharksforarms so it matches the other similar calls.
The leak was discussed with @sharksforarms as well. The idea is to free the buffer if it does not get assigned to *conn_ctx.responses.next. It seems to me that it's freed correctly once assigned.
@kazuho as requested.

Thank you for the PR. Looks good.


09-perform-data-analysis-in-power-bi-desktop.md

Source:
Module 10 - Perform Advanced Analytics
Current Translation:
Module 10 - Perform Advanced Analytics
Suggested Translation:
Módulo 10: Realización de análisis avanzados

Don't find Current Translation. rejected.