fast lane Pilot: undefined method `local_variable_get'  error

New Issue Checklist

[x ] Updated fastlane to the latest version
[ x] I have read the Contribution Guidelines

Issue Description
undefined method `local_variable_get'  when running Pilot
Mobile iOS app uploads correctly to iTuneConnect and start TestFlight however the "Fastlane distribute" command fails with error before terminating.
Complete output when running fastlane, including the stack trace and command used

You can use: --capture_output as the last commandline argument to get that collected for you

Fastlane distribute
[09:46:18]: Seems like launching fastlane takes a while - please run
[09:46:18]:
[09:46:18]: $ [sudo] gem cleanup
[09:46:18]:
[09:46:18]: to uninstall outdated gems and make fastlane launch faster
[09:46:18]: Alternatively it's recommended to start using a Gemfile to lock your dependencies
[09:46:18]: To get started with a Gemfile, run
[09:46:18]:
[09:46:18]: $ bundle init
[09:46:18]: $ echo 'gem "fastlane"' >> Gemfile
[09:46:18]: $ bundle install
[09:46:18]:
[09:46:18]: After creating the Gemfile and Gemfile.lock, commit those files into version control
[09:46:18]: For more information, check out https://guides.cocoapods.org/using/a-gemfile.html
[09:46:21]: -------------------------------------------------
[09:46:21]: --- Step: Verifying required fastlane version ---
[09:46:21]: -------------------------------------------------
[09:46:21]: Your fastlane version 2.28.2 matches the minimum requirement of 1.105.0  ✅
[09:46:21]: ------------------------------
[09:46:21]: --- Step: default_platform ---
[09:46:21]: ------------------------------
[09:46:21]: Driving the lane 'ios distribute' 🚀
[09:46:21]: -------------------
[09:46:21]: --- Step: pilot ---
[09:46:21]: -------------------
[09:46:21]: Login to iTunes Connect<EMAIL_ADDRESS>[09:46:27]: Login successful
[09:46:28]: Ready to upload new build to TestFlight (App:<PHONE_NUMBER>)...
[09:46:43]: Going to upload updated app to iTunes Connect
[09:46:43]: This might take a few minutes. Please don't interrupt the script.
[09:49:41]: iTunes Transporter successfully finished its job
[09:49:43]: -----------------------------------------------------------------------------------------------------
[09:49:43]: Successfully uploaded package to iTunes Connect. It might take a few minutes until it's visible online.
[09:49:43]: -----------------------------------------------------------------------------------------------------
[09:49:43]: Successfully uploaded the new binary to iTunes Connect
[09:49:43]: If you want to skip waiting for the processing to be finished, use the skip_waiting_for_build_processing option
+------------------+----------------+
|           Lane Context            |
+------------------+----------------+
| DEFAULT_PLATFORM | ios            |
| PLATFORM_NAME    | ios            |
| LANE_NAME        | ios distribute |
+------------------+----------------+
[09:49:43]: undefined method `local_variable_get' for #Binding:0x007fc68cc803e8
+------+------------------------------+-------------+
|                 fastlane summary                  |
+------+------------------------------+-------------+
| Step | Action                       | Time (in s) |
+------+------------------------------+-------------+
| 1    | Verifying required fastlane  | 0           |
|      | version                      |             |
| 2    | default_platform             | 0           |
| 💥    | pilot                        | 202         |
+------+------------------------------+-------------+
[09:49:44]: fastlane finished with errors
Looking for related GitHub issues on fastlane/fastlane...
Found no similar issues. To create a new issue, please visit:
https://github.com/fastlane/fastlane/issues/new
Run fastlane env to append the fastlane environment to your issue
/Library/Ruby/Gems/2.0.0/gems/fastlane-2.28.2/spaceship/lib/spaceship/test_flight/client.rb:133:in block in assert_required_params': [!] undefined method local_variable_get' for #Binding:0x007fc68cc803e8 (NoMethodError)
from /Library/Ruby/Gems/2.0.0/gems/fastlane-2.28.2/spaceship/lib/spaceship/test_flight/client.rb:132:in each' 	from /Library/Ruby/Gems/2.0.0/gems/fastlane-2.28.2/spaceship/lib/spaceship/test_flight/client.rb:132:in assert_required_params'
from /Library/Ruby/Gems/2.0.0/gems/fastlane-2.28.2/spaceship/lib/spaceship/test_flight/client.rb:9:in get_build_trains' 	from /Library/Ruby/Gems/2.0.0/gems/fastlane-2.28.2/spaceship/lib/spaceship/test_flight/build_trains.rb:13:in all'
from /Library/Ruby/Gems/2.0.0/gems/fastlane-2.28.2/spaceship/lib/spaceship/test_flight/build.rb:83:in all' 	from /Library/Ruby/Gems/2.0.0/gems/fastlane-2.28.2/spaceship/lib/spaceship/test_flight/build.rb:94:in all_processing_builds'
from /Library/Ruby/Gems/2.0.0/gems/fastlane-2.28.2/fastlane_core/lib/fastlane_core/build_watcher.rb:6:in wait_for_build_processing_to_be_complete' 	from /Library/Ruby/Gems/2.0.0/gems/fastlane-2.28.2/pilot/lib/pilot/build_manager.rb:38:in upload'
from /Library/Ruby/Gems/2.0.0/gems/fastlane-2.28.2/fastlane/lib/fastlane/actions/pilot.rb:15:in run' 	from /Library/Ruby/Gems/2.0.0/gems/fastlane-2.28.2/fastlane/lib/fastlane/runner.rb:252:in block (2 levels) in execute_action'
from /Library/Ruby/Gems/2.0.0/gems/fastlane-2.28.2/fastlane/lib/fastlane/actions/actions_helper.rb:50:in execute_action' 	from /Library/Ruby/Gems/2.0.0/gems/fastlane-2.28.2/fastlane/lib/fastlane/runner.rb:230:in block in execute_action'
from /Library/Ruby/Gems/2.0.0/gems/fastlane-2.28.2/fastlane/lib/fastlane/runner.rb:226:in chdir' 	from /Library/Ruby/Gems/2.0.0/gems/fastlane-2.28.2/fastlane/lib/fastlane/runner.rb:226:in execute_action'
from /Library/Ruby/Gems/2.0.0/gems/fastlane-2.28.2/fastlane/lib/fastlane/runner.rb:148:in trigger_action_by_name' 	from /Library/Ruby/Gems/2.0.0/gems/fastlane-2.28.2/fastlane/lib/fastlane/fast_file.rb:146:in method_missing'
from Fastfile:52:in block (2 levels) in parsing_binding' 	from /Library/Ruby/Gems/2.0.0/gems/fastlane-2.28.2/fastlane/lib/fastlane/lane.rb:33:in call'
from /Library/Ruby/Gems/2.0.0/gems/fastlane-2.28.2/fastlane/lib/fastlane/lane.rb:33:in call' 	from /Library/Ruby/Gems/2.0.0/gems/fastlane-2.28.2/fastlane/lib/fastlane/runner.rb:49:in block in execute'
from /Library/Ruby/Gems/2.0.0/gems/fastlane-2.28.2/fastlane/lib/fastlane/runner.rb:45:in chdir' 	from /Library/Ruby/Gems/2.0.0/gems/fastlane-2.28.2/fastlane/lib/fastlane/runner.rb:45:in execute'
from /Library/Ruby/Gems/2.0.0/gems/fastlane-2.28.2/fastlane/lib/fastlane/lane_manager.rb:52:in cruise_lane' 	from /Library/Ruby/Gems/2.0.0/gems/fastlane-2.28.2/fastlane/lib/fastlane/command_line_handler.rb:30:in handle'
from /Library/Ruby/Gems/2.0.0/gems/fastlane-2.28.2/fastlane/lib/fastlane/commands_generator.rb:104:in block (2 levels) in run' 	from /Library/Ruby/Gems/2.0.0/gems/commander-fastlane-4.4.4/lib/commander/command.rb:178:in call'
from /Library/Ruby/Gems/2.0.0/gems/commander-fastlane-4.4.4/lib/commander/command.rb:178:in call' 	from /Library/Ruby/Gems/2.0.0/gems/commander-fastlane-4.4.4/lib/commander/command.rb:153:in run'
from /Library/Ruby/Gems/2.0.0/gems/commander-fastlane-4.4.4/lib/commander/runner.rb:476:in run_active_command' 	from /Library/Ruby/Gems/2.0.0/gems/fastlane-2.28.2/fastlane_core/lib/fastlane_core/ui/fastlane_runner.rb:40:in run!'
from /Library/Ruby/Gems/2.0.0/gems/commander-fastlane-4.4.4/lib/commander/delegates.rb:15:in run!' 	from /Library/Ruby/Gems/2.0.0/gems/fastlane-2.28.2/fastlane/lib/fastlane/commands_generator.rb:303:in run'
from /Library/Ruby/Gems/2.0.0/gems/fastlane-2.28.2/fastlane/lib/fastlane/commands_generator.rb:42:in start' 	from /Library/Ruby/Gems/2.0.0/gems/fastlane-2.28.2/fastlane/lib/fastlane/cli_tools_distributor.rb:66:in take_off'
from /Library/Ruby/Gems/2.0.0/gems/fastlane-2.28.2/bin/fastlane:20:in <top (required)>' 	from /usr/local/bin/Fastlane:22:in load'
from /usr/local/bin/Fastlane:22:in `'
Environment
Please run fastlane env and copy the output below. This will help us help you :+1:
If you used --capture_output option please remove this block - as it is already included there.
✅ fastlane environment ✅
Stack

Key
Value

OS
10.11.6

Ruby
2.0.0

Bundler?
false

Git
git version 2.9.3 (Apple Git-75)

Installation Source
/usr/local/bin/fastlane

Host
Mac OS X 10.11.6 (15G1004)

Ruby Lib Dir
/System/Library/Frameworks/Ruby.framework/Versions/2.0/usr/lib

OpenSSL Version
OpenSSL 0.9.8zc 19 Mar 2015

Is contained
false

Is homebrew
false

Is installed via Fabric.app
false

Xcode Path
/Applications/Xcode.app/Contents/Developer/

Xcode Version
8.1

System Locale

Variable
Value

LANG
en_US.UTF-8
✅

LC_ALL

LANGUAGE

fastlane files:
`./fastlane/Fastfile`
# Customise this file, documentation can be found here:
# https://github.com/fastlane/fastlane/tree/master/fastlane/docs
# All available actions: https://github.com/fastlane/fastlane/blob/master/fastlane/docs/Actions.md
# can also be listed using the `fastlane actions` command

# Change the syntax highlighting to Ruby
# All lines starting with a # are ignored when running `fastlane`

# If you want to automatically update fastlane if a new version is available:
# update_fastlane

# This is the minimum version number required.
# Update this, if you use features of a newer version
fastlane_version "1.105.0"

default_platform :ios

platform :ios do
  before_all do
    # ENV["SLACK_URL"] = "https://hooks.slack.com/services/..."
    #cocoapods

  end

  desc "Runs all the tests"
  lane :test do
    scan
  end

  desc "Submit a new Beta Build to Apple TestFlight"
  desc "This will also make sure the profile is up to date"
  lane :beta do
    # match(type: "appstore") # more information: https://codesigning.guide
    gym(scheme: "OrderExpress-Build") # Build your app - more options available
    pilot

    # sh "your_script.sh"
    # You can also use other beta testing services here (run `fastlane actions`)
  end

  desc "Deploy a new version to the App Store"
  lane :release do
    # match(type: "appstore")
    # snapshot
    gym(scheme: "OrderExpress-Build") # Build your app - more options available
    deliver(force: true)
    # frameit
  end

  desc"Distribute via fastlane Pilot"
  lane :distribute do
  pilot(
    username: ENV["ITUNES_CONNECT_USER"],
    wait_processing_interval: 30,
    distribute_external: false, #internal user only
    skip_submission: false, #do not notify the testersy
    ipa: "signed.ipa"  #path of the ipa generated by your build script
  )
  end

  # You can define as many lanes as you want

  after_all do |lane|
    # This block is called, only if the executed lane was successful

    # slack(
    #   message: "Successfully deployed new App Update."
    # )
  end

  error do |lane, exception|
    # slack(
    #   message: exception.message,
    #   success: false
    # )
  end
end

# More information about multiple platforms in fastlane: https://github.com/fastlane/fastlane/blob/master/fastlane/docs/Platforms.md
# All available actions: https://github.com/fastlane/fastlane/blob/master/fastlane/docs/Actions.md

# fastlane reports which actions are used
# No personal data is recorded. Learn more at https://github.com/fastlane/enhancer

`./fastlane/Appfile`
app_identifier "com.xxxxxxxx.xxxxxx" # The bundle identifier of your app
apple_id<EMAIL_ADDRESS># Your Apple email address

team_id "XXXXXXXX"  # Developer Portal Team ID

# you can even provide different app identifiers, Apple IDs and team names per lane:
# More information: https://github.com/fastlane/fastlane/blob/master/fastlane/docs/Appfile.md

fastlane gems

Gem
Version
Update-Status

fastlane
2.28.2
💥 Check failed

Loaded fastlane plugins:
No plugins Loaded
Loaded gems

Gem
Version

slack-notifier
1.5.1

claide
1.0.1

colored2
3.1.2

nanaimo
0.2.3

xcodeproj
1.4.4

rouge
1.11.1

xcpretty
0.2.4

terminal-notifier
1.6.3

terminal-table
1.4.5

plist
3.1.0

multipart-post
2.0.0

word_wrap
1.0.0

tty-screen
0.5.0

babosa
1.0.2

colored
1.2

highline
1.7.8

commander-fastlane
4.4.4

excon
0.53.0

faraday
0.9.2

unf_ext
<IP_ADDRESS>

unf
0.1.4

domain_name
0.5.20160826

http-cookie
1.0.3

faraday-cookie_jar
0.0.6

gh_inspector
1.0.2

uber
0.0.15

representable
2.3.0

retriable
2.1.0

mime-types-data
3.2016.0521

mime-types
3.1

hurley
0.2

little-plugger
1.1.4

multi_json
1.12.1

logging
2.1.0

jwt
1.5.6

memoist
0.15.0

os
0.9.6

signet
0.7.3

googleauth
0.5.1

httpclient
<IP_ADDRESS>

google-api-client
0.9.15

mini_magick
4.5.1

multi_xml
0.5.5

rubyzip
1.1.7

security
0.1.3

xcpretty-travis-formatter
0.0.4

dotenv
2.1.1

bundler
1.12.0

faraday_middleware
0.10.0

json
1.8.3

addressable
2.3.8

fastimage
1.9.0

generated on: 2017-04-21

I'm also seeing this issue on 2.28.2.

I believe the root cause is line 133 in spaceship/lib/spaceship/test_flight/client.rb:
if binding.local_variable_get(name).nil?
where binding.local_variable_get is support in Ruby 2.1+.

Thanks for reporting this, we already prepared a fix: https://github.com/fastlane/fastlane/pull/8963
This is related to the Ruby version you're running, if you want to use fastlane over the weekend, you can upgrade your Ruby version, or wait until the new release is live 🚀 Alternatively you can also patch your local fastlane installation according to #8963 👍

Caught that one in Travis CI 10 minutes ago. Any ETA on releasing the fix?

Thank you. I also found out that putting the following line in the fastlane file lane solves the problem:
skip_waiting_for_build_processing:true
Sent from my Verizon, Samsung Galaxy smartphone
-------- Original message --------
From: Felix Krause<EMAIL_ADDRESS>Date: 4/21/17 11:48 PM (GMT-05:00)
To: fastlane/fastlane<EMAIL_ADDRESS>Cc: cvedovat<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [fastlane/fastlane] fast lane Pilot: undefined method `local_variable_get' error (#8955)
Thanks for reporting this, we already prepared a fix: #8963https://github.com/fastlane/fastlane/pull/8963
This is related to the Ruby version you're running, if you want to use fastlane over the weekend, you can upgrade your Ruby version, or wait until the new release is live 🚀 Alternatively you can also patch your local fastlane installation according to #8963https://github.com/fastlane/fastlane/pull/8963 👍
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/fastlane/fastlane/issues/8955#issuecomment-296344828, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AH_1KWWdl-fvvVzrts00CTVB3gUNEL6Sks5ryXh-gaJpZM4NEX3S.


inconsistent db state when spawner fails to start

it's possible for some parts of the Hub to think a server is running when it's not, preventing stop/restart from working.
/cc @jhamrick

@minrk what does "some parts of the hub" mean here?  There's definitely a window from https://github.com/jupyter/jupyterhub/blob/master/jupyterhub/orm.py#L300 to https://github.com/jupyter/jupyterhub/blob/master/jupyterhub/orm.py#L333 where there's a server entry for the user but the server hasn't actually come up yet.  Is that the issue you're referring to, or is there something else you're seeing?

Apparently some APIs are using different information to report whether there's already a server running, so the admin page shows a 'stop' button, but clicking it results in "server isn't running".
This state can happen after a spawner fails to start, not just the spawn pending state, which isn't a problem.

The span you are referring to where server is defined but not complete is covered by the User.spawn_pending state. Where there's inconsistency is in various checks for whether a server is running. #147 adds a .running property to be used everywhere, so that at least we don't have the case where the stop server button appears, but stopping a user fails with "user's server isn't running". The main issue that cause the problem here is uncaught errors, which would leave server defined after the server failed to start.


Fix rosa install to use new asset structure

Re-implements backplane-tools install rosa to account for new installation assets.

/lgtm


Inconsistent input size across browsers

http://bitters.bourbon.io/example.html demonstrates.
The select, radio, checkbox and file input buttons are 16px in Firefox 36 and 13px in Chrome 41 and Safari 8.

@tom-- Are you talking about the actual dimensions of these input elements? Here’s a screenshot from inspector in Firefox:

If so, Bitters doesn’t adjust the sizing of these; they are browser defaults and each browser renders differently.

I was referring to

the computed value of font-size
how font-size inherits from body in FF and is set by Bitters' _forms.scss in Webkit
the appearance of the input elements relative to everything around them, in FF it looks reasonable, in Webkit the inputs look incongruously small relative to the labels

I imagined that Bitters would take care of this so that I don't need to. Bitters provides a typography with consistent sizes so I just assumed it would want to address this inconsistency too.

Bitters isn’t a reset, so I think this is outside the scope. These are just browsers computing the default size differently.


Battery monitor Support

This contains the corresponding changes for battery monitor support in rosflight_io, rosflight_msgs, and rosflight_sim.

Does this message need the header?
Would it be bad for me to restore the branch, delete the header, and re-submit for review?


fix: azure backend errors with storage url and container metadata

The container URL was adding an extra / when the service URL ended with /. This results in an invalid container URI error from Azure.
This also fixes the container metadata keys to not include "-" that the Azure service disallows. The metadata values are base64 encoded to prevent any special chacter handling on Azure service side.
The content type is set for uploads consistent with S3 expectations.
Fixes #736

There are still several test failures dealing with multipart uploads since the azure blob service doesnt have a corresponding create multipart upload. For azure blob, it just starts uploading parts without initiating the multipart upload. We usually set some metadata on create multipart upload, so we need to figure out how to translate this to azure.

Its also interesting that some of the failures fixed with this PR could only be seen when interacting with the real azure service. So azurite doesnt appear to be a perfect proxy for testing azure.


[BUG] load datetime value '0000-00-00 00:00:00' ,then tiSpark will get value not excpeted

Describe the bug
load datetime value '0000-00-00 00:00:00' ,then tiSpark will get value not excpeted
I use spark-sql to get datetime value   '0000-00-00 00:00:00'  from Tidb ,
then  get '0000-12-30 00:05:43' value ,but not   '0000-00-00 00:00:00'
tidb version:
Release Version: v4.0.0
Edition: Community
Git Commit Hash: 689a6b6439ae7835947fcaccf329a3fc303986cb
Git Branch: heads/refs/tags/v4.0.0
UTC Build Time: 2020-05-28 01:37:40
GoVersion: go1.13
Race Enabled: false
TiKV Min Version: v3.0.0-60965b006877ca7234adaced7890d7b029ed1306
Check Table Before Drop: false
spark version:
<spark.version>2.4.7</spark.version>
tispark-2.3.9

would please run the following code?
spark.sql("select ti_version()").show(false)

https://github.com/pingcap/tispark/issues/1701

|Release Version: 2.3.9
Git Commit Hash: 9ba472f0e63281888398c2d5de69114545616906
Git Branch: release-2.3
UTC Build Time: 2020-10-31 02:18:25
Supported Spark Version: 2.3 2.4
Current Spark Version: 2.4.7
Current Spark Major Version: 2.4
TimeZone: Asia/Shanghai|

I made some experiments, results are as follows:
run in MySQL-client
CREATE TABLE `tmp_debug` (`tp_datetime` datetime)
INSERT INTO `tmp_debug` VALUES ('0000-00-00 00:00:00')

run in spark-shell
spark.sql("select * from tmp_debug").show(false)

output
+-------------------+
|tp_datetime        |
+-------------------+
|0001-01-01 00:00:00|
+-------------------+

run in spark-shell
spark.read.jdbc(jdbcUrl, "tispark_test.tmp_debug", new Properties()).show()

output
+-------------------+
|tp_datetime        |
+-------------------+
|0001-01-01 00:00:00|
+-------------------+

The minimal value of type datetime supported by spark is 0001-01-01 00:00:00, so the result of TiSpark and SparkJDBC is same (which is 0001-01-01 00:00:00).

@RUIzg would you please try SparkJDBC (spark.read.jdbc) and give me the output?

When insert into the column of  date type with the value of 0, the tispark value is 0001-01-03. Is it as expected? @marsishandsome

When insert into the column of  date type with the value of 0, the tispark value is 0001-01-03. Is it as expected? @marsishandsome

When insert into the column of date type with the value of 0, the tispark value is 0001-01-03. Is it as expected? @marsishandsome

The valid range of Date type in MySQL is 1000-01-01——9999-12-31.
0 is not a valid value of Date Type.
So to my opinion,  0001-01-03 is a reasonable result.

When insert into the column of date type with the value of 0, the tispark value is 0001-01-03. Is it as expected? @marsishandsome

The valid range of Date type in MySQL is 1000-01-01——9999-12-31.
0 is not a valid value of Date Type.
So to my opinion,  0001-01-03 is a reasonable result.

This issue is closed. If it can be reproduced on the latest master branch, feel free to reopen it.


Ultrabrite Smart Wifi Plug [BK7231T] [WB2S]

Latest model from UK Poundland
I: UPK: Found WB2S config!
I: UPK: Switch/plug config
I: UPK:  - relay 1: pin P24
I: UPK:  - LED 1: pin P8
I: UPK:  - button 1: pin P6
I: UPK: Status LED: pin P7, inverted False
readResult_BK7231T_QIO_2023-29-5--21-19-04.zip
I think the LEDs work better (and like actual config) both inverted.

I bought last of TYWE2S stock and some new WB2S, all identical except for the chip:

Added as Ultrabrite ULP1GWIFIPLUG Smart Plug


Refactor for YJs AST

Pull Request Description
Some refactoring separated from #8825 for easier review.
Important Notes
ID types
The new synchronization IDs will replace ExprId for Ast references in frontend logic. ExprId (now called ExternalId) is now used only for module serialization and engine communication. The graph database will maintain an index that is used to translate at the boundaries. For now, this translation is implemented as a type cast, as the IDs have the same values until the next PR.

AstId: Identifies an Ast node.
NodeId: A subtype of AstId.
ExternalId: UUID used for serialization and engine communication.

Other changes:

Immediate validation of Owned usage.
Eliminate Ast.RawCode.
Prepare to remove IdMap from yjsModel.

Checklist
Please ensure that the following checklist has been satisfied before submitting the PR:

[ ] The documentation has been updated, if necessary.
[ ] Screenshots/screencasts have been attached, if there are any visual changes. For interactive or animated visual changes, a screencast is preferred.
[ ] All code follows the
Scala,
Java,
and
Rust
style guides. In case you are using a language not listed above, follow the Rust style guide.
All code has been tested:

[ ] Unit tests have been written where possible.
[ ] If GUI codebase was changed, the GUI was tested when built using ./run ide build.

The more I think about this change, the more I have mixed feeling.
We will find many situations when we want to update AST shape but keep metadata. Actually, this is the normal way of doing when editing nodes or widgets. Currently we manage that by using replaceValue in such places.
But once #8804 will be being implemented, we should ensure that replacing AST while keeping metadata will be an easy (almost "default") operation, not too easy to overlook. I think the best would be to make abstraction like this:
interface Expression {
    ast: Ast,
    metadata: Metadata
}

this would be a structure kept inside Module, and AST would keep references to it.
Of course, the above design doesn't work well with idea of having separate AstId and ExternalId. Either the AstId and ExternalId will be identical anyway, or the Metadata would have to be moved from one AstId to another when doing normal node/widget edits, what I think won't work well in with multiple users - we might find some metadata being "copied" to two different AstIds, duplicating the nodes from the user perspective.
@kazcw @Frizi what are your thoughts about it?

When considering multi-client sync, it is not so simple. Identity is more than who gets the metadata. replaceValue takes an object that may have been in the tree somewhere else, and gives it the metadata from the object that is being replaced. Because it keeps the synchronization identity (i.e. the path in the ydoc to the map holding its data) of the argument, any edits another client was making to that subtree will be merged correctly. If we stored the AST data directly in a child of the proposed Expression object, we would not be able to do that. Instead, changes to the value being emplaced would be mergeable with the value being replaced. We cannot allow that--they are likely not even the same type.

Note to reviewers--I'm making a quick change. The graphStore will maintain an idFromExternal index, but not idToExternal. External IDs will be obtained from an Ast property. These two mappings are intentionally not always in sync: Ast.externalId reflects the current edit; idFromExternal reflects the most recent synchronized state.

These two mappings are intentionally not always in sync: Ast.externalId reflects the current edit; idFromExternal reflects the most recent synchronized state.

This is worth being a comment in the code somewhere.


[WIP] Support null 'm' & 'p' claims in SHRs

Impetus:
https://github.com/AzureAD/microsoft-authentication-library-for-android/issues/1025

Blocked: this is currently upsupported by RP middleware

Closing, but preserving the branch. I don't think we'll be wanting this change


[REG 2.073.2] double print to stdout when reading stdin in another thread

gassa (@GassaFM) reported this on 2017-08-31T13:40:19Z
Transfered from https://issues.dlang.org/show_bug.cgi?id=17797
CC List

ag0aep6g
bugzilla (@WalterBright)
code (@MartinNowak)
r.sagitario

Description
Consider the following program:

-----
import std.stdio, core.thread;

void main(){
	auto thread = new Thread(&func).start;
        writeln("Output");
	writeln("Output2");
	writeln("Output3");
	while(true){}
}

void func(){
    foreach(line; stdin.byLineCopy){}
}
-----

The correct output is:
-----
Output
Output2
Output3
-----

This starts a thread to read from stdin, and then writes several lines to stdout in the main thread and enters and infinite loop.

When compiled with dmd 2.073.1 or earlier, everything is fine.
Starting from dmd 2.073.2, the output is unstable but usually not correct.
Locally, I usually observe the following output:

-----
Output
Output2
Output3
Output2
-----

Or sometimes:

-----
Output
Output2
Output2
Output3
-----

Here are the steps to reproduce, tested on 64-bit Windows with a 32-bit compiler.

1. Use dmd 2.073.2 or later.
2. Compile like "dmd program.d" (compiler switches don't seem to matter).
3. Run a command prompt, like "cmd.exe".
4. Execute "program.exe" (importantly, without any I/O redirection).

Original thread at D.Learn:
<EMAIL_ADDRESS>

bugzilla (@WalterBright) commented on 2017-10-03T08:39:15Z
It happens on Win32 as well, which uses the DMC runtime library, not Microsoft's. So it is not specific to the Microsoft runtime library.

I suspect it is a Phobos problem, not a compiler issue, and changed the category accordingly.

r.sagitario commented on 2017-10-10T07:03:15Z
I could verify it happens with 2.073.2, but not 2.073.1. This is caused by an updated snn.lib. Might be introduced by the fix for issue 13727.

ag0aep6g commented on 2017-10-03T09:16:11Z
(In reply to Walter Bright from comment #1)
> It happens on Win32 as well, which uses the DMC runtime library, not
> Microsoft's. So it is not specific to the Microsoft runtime library.

It *only* happens with the DMC runtime. People are running 64-bit Windows, but they're not using 64-bit dmd. I just tested in a virtual machine with -m32mscoff and didn't see duplicated lines.

code (@MartinNowak) commented on 2017-10-09T11:26:58Z
cat > bug.d << CODE
import std.stdio, core.thread;

void main()
{
    auto thread = new Thread(&func).start;
    writeln("Output");
    writeln("Output2");
    writeln("Output3");
    thread.join();
}

void func()
{
    foreach (line; stdin.byLineCopy)
    {
    }
}
CODE
dmd -run bug.d
----
Slightly nicer version that terminates on EOF (Ctrl+D), works as expected on linux.

Sure about 2.073.2? The only interesting change in

https://github.com/dlang/dmd/compare/v2.073.1...v2.073.2
https://github.com/dlang/druntime/compare/v2.073.1...v2.073.2
https://github.com/dlang/phobos/compare/v2.073.1...v2.073.2

is https://github.com/dlang/dmd/pull/6582.


Add windows support

As discussed, it would be great to include support for windows.
As a means of motivating this feature request, here is a related thread:
https://github.com/HiLab-git/SimpleCRF/issues/7

Copying here for reference, from @charliebudd:

I have spent some time looking into this, Unfortunetly my efforts hit a dead end when I found the github runner cannot run windows docker images. The work around is to install the packages inside the runner for each build. This will be a bit slower than downloading a pre-built docker image but I dont see another way. I'll try to spend some time on this next week.

related to conda fix #22


Wireframe: Case study article

User story
As a ReportStream prospective/current user, I want the website content to be up-to-date, accurate and timely so I can  trust the information to inform my decisions in my relationship with ReportStream.
Background & context
This ticket captures the work of putting together the Case study article (COVID-19) using the existing detail page template.
Working links

Figma: Working file
Content doc: Case study

Acceptance criteria
Design

[x] Design is ready for handoff (i.e. dev ready)
[x] Design includes latest content updates
[x] Capture questions that come up during the content and design work in the Website redesign research plan

Review

[x] Reviewed with design and/or design lead
[x] Reviewed with content strategist for final approval
[x] Reviewed with product and engineering

Review from Audrey complete, closing issue. Moved design to eng-ready on figma.


[libvorbis] Update to 1.3.7

Update to official 1.3.7 release. No triplets/featues modified. CONTROL replaced with vcpkg.json.

The regression on arm64-windows was caused by soil:
It looks like the URL has been invalid now. I also checked the address http://www.lonesock.net , which cannot be available.
We might need to update the repository for soil. I searched several repos for soil on GitHub, but I'm not sure which one is the proper one.
https://github.com/kbranigan/Simple-OpenGL-Image-Library this repo is cloned from the original website provided in vcpkg. There are still some new changes in this repo.
So could you help give some suggestions about this issue? @BillyONeal
Starting package 10/10: soil:arm64-windows
Building package soil[core]:arm64-windows...
Could not locate cached archive: W:\bd\bdcfcc29462abd2494b51c489a08f6c2ed7b8ad8.zip
-- Downloading http://www.lonesock.net/files/soil.zip...
-- Downloading http://www.lonesock.net/files/soil.zip... Failed. Status: 22;"HTTP response code said error"
CMake Error at scripts/cmake/vcpkg_download_distfile.cmake:176 (message):

      Failed to download file.
      If you use a proxy, please set the HTTPS_PROXY and HTTP_PROXY environment
      variables to "https://user:***@your-proxy-ip-address:port/".
      Otherwise, please submit an issue at https://github.com/Microsoft/vcpkg/issues

Call Stack (most recent call first):
  ports/soil/portfile.cmake:3 (vcpkg_download_distfile)
  scripts/ports.cmake:79 (include)

@NancyLi1013 Nope, looks like upstream is just gone :/

Thanks for your contribution!


Wrong column labels are being used if dataKey is the same

It appears that you can't have two columns with different label but the same dataKey.
          <Column
            cellRenderer={trendLine}
            dataKey="values"
            label="Trend"
            width={100}
          />
          <Column
            cellRenderer={averager}
            dataKey="values"
            label="Average"
            width={100}
          />

One of these two labels will win, and both columns will display the same header label ("Average" in my case).
The use case here is that I want to share (not duplicate) the same data (an array of values), but render it differently in each cell -- one would show the average of the cells (a number), the other a sparkline/chart of all the values over time.
I'm using 0.1.2. Thanks for that component, it looks very useful.

I have the same issue. Workaround I've been using is
 dataKey="bogus"

and then

  _otherColumnRender: function(cellValue, cellKey, rowData) {
    var values = rowData.values;
   ...
  }

Thanks for bringing this up! We're currently working on solving this issue.

Thanks for the workaround @hinathan, I was also running in to this.


View Results Tree support?

First of all, thanks a lot for providing the jmespath support in such a swift way. Now,
I know it is debatable whether View Results Tree is necessary, but whenever something unexpected happens, or the groovy-jmeter
or the plain jmeter script is not behaving as I expected, the first thing I'd do is to add the View Results Tree, so that I can know what went wrong.
That might be the only way to debug groovy-jmeter issue. Please consider. thx.

Yes, I might add it.

🎉🎉🎉!

Ref: https://github.com/smicyk/groovy-jmeter/wiki/reference#view


How to run the Server in VS2019?

I have built the solution in VS2019, then run BlazingPizza Server Debug, the console started ok. And Edge has appeared with address http://localhost:64590/
but it gives an error:

This site can’t provide a secure connectionlocalhost sent an invalid response.
Try running Windows Network Diagnostics.
ERR_SSL_PROTOCOL_ERROR

How to fix it?

set "sslPort": 44366 in launchSettings.json files (client and server proj) worked for me.

I'd say you have selected the profile BlazinPizza.Server to debug, reason why you're running on a different port (64590). Now, I've switched my Debugger profile from IISExpress, which runs on port 64589, to BlazingPizza.Server to see if I get the same error, and I didn't. However the page never finishes loading. I'd say this has to do with the fact (you mentioned above) you're running on Edge, rather than Chrome or Microsoft Edge Dev. Suggestions,

I would change the Web Browser from the debugger profile to either Chrome or Microsoft Edge Dev (if you downloaded). Then follow the instructions for debugging.


Improve integration tests

Situation
Since moving to the Scala 2.12 version, it is now possible to use the Java 11 environment. Thus, Exasol Testcontainers should be used for integration tests.
Acceptance Criteria

Find similar docker based test containers for Kafka stack (zookeeper, kafka, schema registry)
Add docker based integration tests for data import
Obtain coverage results from docker containers (similar setup as in DynamoDB VS)

Fixed in #50 and #39


All frames from the client should be masked

As per the final spec (http://tools.ietf.org/html/rfc6455) all frames from the client must be masked. Currently control frames aren't being masked.

Well it turns out the RFC says:

A server MUST NOT mask any frames that it sends to the client.

In http://tools.ietf.org/html/rfc6455#section-5.1
So it would appear control frame from the server shouldn't be masked.

I am not referring to frames sent from the server, I am referring to frames sent from the client to the server. In the section you refer to see -> "a client MUST mask all frames that it sends to the server".

Damn sometimes I should pay more attention. You are right. I'll look into it.


Google Chrome not supported correctly but works in demo site

From<EMAIL_ADDRESS>on June 10, 2009 21:39:01
What steps will reproduce the problem? 1. Install version 2.3.2 from google code
2. Use in Google Chrome
3. Compare Demo site @ http://ondras.zarovi.cz/sql/demo/?keyword=default What is the expected output? What do you see instead? My downloaded version does not show smooth connectors in google chrome also
the pane does not move in google chrome when moving the red rectangle in
the lower right corner. Functionality of local version is OK in IE7 and
Firefox. However when using the demo site in google chrome smooth
connectors and dragging the red rectangle works OK - Is the version on the
demo site different then the one on google code? What version of the product are you using? On what operating system? Windows XP, Please provide any additional information below.
Original issue: http://code.google.com/p/wwwsqldesigner/issues/detail?id=22

From ondrej.zara on June 10, 2009 14:28:14
The demo site always uses latest SVN revision. Try updating your build from SVN and
re-trying in Chrome.

From<EMAIL_ADDRESS>on June 10, 2009 18:30:44
Thank you, after updating my code from the SVN I got full functionality in Google Chrome and Apple Safari

From ondrej.zara on June 10, 2009 23:30:39
Status: Invalid


Like method for default zeros, ones, rand methods

Like method is added for manually defined methods, but still needs to be added for default zeros, ones, rand methods
Different tasks of this to do plan:

Ask in Discourse forum or Slack group for ideas

Achievement check:
having both:
zerosM(Int32, 2, 2)
zerosM(2, 2, like=ones(Int32,2,2) )

Here's what I do for copying all the methods
 zerosM(args...) = zeros(args...)

Other than editing Julia Base, one way which seems that may work is to exclude args to not be a type.
Do we have something like !:T?
This is my crazy prototype (doesn't work obviously):
zerosM(args::T...; like::AbstractArray) where {T!::Type{T1}} where {T1} = zeros(eltype(like), args...)

julia> fun(x::Integer) = 1
fun (generic function with 1 method)

julia> fun(x::Integer; y::Array) = 1
fun (generic function with 1 method)

julia> fun(x::Integer, y::Array) = 1
fun (generic function with 2 methods)

The 2nd definition doesn't create another method
I know that I can make keyword optional by
julia> fun(x::Integer; y::Array= nothing) = 1
fun (generic function with 1 method)

But in my situation, I am expanding another function's methods, and the function has fun(x::Integer) = 1 originally, and I don't want to overwrite that method.
However, the keyword arguments never participate in dispatch, and so there is no way other than redefining the original methods.
https://github.com/JuliaLang/julia/issues/9498
https://discourse.julialang.org/t/rules-for-method-dispatch-with-keyword-arguments/23626/3

Probably it is better to not use a keyword argument and simply pass the Array as the first argument to the function.


fixed landing-page components problem / navbar styling / white space …

I did some work in the landing-page folder. There was Navbar file which was not imported to the main landing.js file, i fixed that and removed the extra code from the landing.js file.
I did some styling on the socials icons and also removed the white space from the bottom of the page.

That's nice but can you tell which issue you are solving? Also you have to strictly follow the contributing guidelines, we have clearly mentioned to first create issue and work on any issue only if you are assigned to do so. If you have been assigned this issue then do mention its issue no. eg- fixes #issue or closes #issue


Cargo should cap lints to "warn" instead of "allow"

Never showing any warnings for upstream deps can hide issues from the user (be it a bug in the library or a future-compatibility lint). Showing them will make those issues more visible and might cause them to get fixed earlier.

This seems reasonable to have as an option for Cargo! I don't think we should change the defaults, however, as it would inundate users with warnings they likely don't care about nor know how to fix.


Update Autoprefixer and standardize browserlist location

An article released by the developers of Autoprefixer and Browerslist with the release of Autoprefixer 7 strongly recommends storing your list of target browsers in package.json, to make it accessible to other pieces of software that may want access to it. The article can be found here: evilmartians.com/chronicles/autoprefixer-7-browserslist-2-released
This pull request implements that recommendation, and updates Autoprefixer to 7.1.1.

Thanks!


[v3] railway up is defaulting to detach = true

The program exits right after the build logs url is printed.

Fixed in #311 :)


Fix issue #3: Support multiple quote types

Add support for multiple quote types in wp-config.php using awk: single, double, or mixed.

Hi @Tekime,
thanks very much for this PR! 🎉
I'll check it.


Error when enabling CUDA on Windows

When enabling the cuda feature, I get the following error on windows:
[...]
  running: "nvcc" "-O0" "-ffunction-sections" "-fdata-sections" "-g" "-fno-omit-frame-pointer" "-m64" "-I" "./llama.cpp/ggml-cuda.h" "-Wall" "-Wextra" "--forward-unknown-to-host-compiler" "-arch=native" "/W4" "/Wall" "/wd4820" "/wd4710" "/wd4711" "/wd4820" "/wd4514" "-DGGML_USE_CUBLAS" "-DGGML_CUDA_DMMV_X=32" "-DGGML_CUDA_DMMV_Y=1" "-DK_QUANTS_PER_ITERATION=2" "-Wno-pedantic" "-o" "C:\\dev\\ai_kuinox\\target\\debug\\build\\llama_cpp_rs-dbbb5a5dac5f7f5e\\out\\./llama.cpp/ggml-cuda.o" "-c" "./llama.cpp/ggml-cuda.cu"
  nvcc fatal   : A single input file is required for a non-link phase when an outputfile is specified
  exit code: 1

There is also a different(?) issue when trying to run with OpenCL.
./llama.cpp/ggml-opencl.cpp(10): fatal error C1083: Cannot open include file: 'clblast.h': No such file or directory

When enabling the cuda feature, I get the following error on windows:
[...]
  running: "nvcc" "-O0" "-ffunction-sections" "-fdata-sections" "-g" "-fno-omit-frame-pointer" "-m64" "-I" "./llama.cpp/ggml-cuda.h" "-Wall" "-Wextra" "--forward-unknown-to-host-compiler" "-arch=native" "/W4" "/Wall" "/wd4820" "/wd4710" "/wd4711" "/wd4820" "/wd4514" "-DGGML_USE_CUBLAS" "-DGGML_CUDA_DMMV_X=32" "-DGGML_CUDA_DMMV_Y=1" "-DK_QUANTS_PER_ITERATION=2" "-Wno-pedantic" "-o" "C:\\dev\\ai_kuinox\\target\\debug\\build\\llama_cpp_rs-dbbb5a5dac5f7f5e\\out\\./llama.cpp/ggml-cuda.o" "-c" "./llama.cpp/ggml-cuda.cu"
  nvcc fatal   : A single input file is required for a non-link phase when an outputfile is specified
  exit code: 1

are you running this in docker ? if so are you using the provided dockerfile ?

No, I'm running this directly on Windows.

No, I'm running this directly on Windows.

Can you try running using the docker image ? i have never tested by running directly so i have no idea if it works or not.

I've been trying to get this working with Tauri, and it seems to work with CPU only, but fails on CUDA for windows. I attempted to compile it manually and that seemed more successful. I think the issue is with the compiler flags used for nvcc, but I can't really confirm if this is the case.

I've been trying to get this working with Tauri, and it seems to work with CPU only, but fails on CUDA for windows. I attempted to compile it manually and that seemed more successful. I think the issue is with the compiler flags used for nvcc, but I can't really confirm if this is the case.

I will try to look into it when i get some time and see how to directly compile on windows. Can you share me your steps on how you directly compiled cuda?

I will try to look into it when i get some time and see how to directly compile on windows. Can you share me your steps on how you directly compiled cuda?

I did some more testing and adding / removing flags actually helped with compiling on Windows, and I modified compile_cuda as such:
fn compile_cuda(cxx_flags: &str) {

    if cfg!(target_os = "linux") || cfg!(target_os = "macos") {
        println!("cargo:rustc-link-search=native=/usr/local/cuda/lib64");
        println!("cargo:rustc-link-search=native=/opt/cuda/lib64");

        if let Ok(cuda_path) = std::env::var("CUDA_PATH") {
            println!(
                "cargo:rustc-link-search=native={}/targets/x86_64-linux/lib",
                cuda_path
            );
    }
    } else if cfg!(target_os = "windows") {
        if let Ok(cuda_path) = std::env::var("CUDA_PATH") {
            println!(
                "cargo:rustc-link-search=native={}/lib/x64",
                cuda_path
            );
        }
    }

    let mut libs = String::from("cublas cudart cublasLt");
    if cfg!(target_os = "linux") || cfg!(target_os = "macos"){
        libs.push_str(" culibos pthread dl rt")
    }

    for lib in libs.split_whitespace() {
        println!("cargo:rustc-link-lib={}", lib);
    }

    let mut nvcc = cc::Build::new();

    let env_flags = vec![
        ("LLAMA_CUDA_DMMV_X=32", "-DGGML_CUDA_DMMV_X"),
        ("LLAMA_CUDA_DMMV_Y=1", "-DGGML_CUDA_DMMV_Y"),
        ("LLAMA_CUDA_KQUANTS_ITER=2", "-DK_QUANTS_PER_ITERATION"),
    ];

    let nvcc_flags = "--forward-unknown-to-host-compiler -arch=native";

    for nvcc_flag in nvcc_flags.split_whitespace() {
        nvcc.flag(nvcc_flag);
    }

    for cxx_flag in cxx_flags.split_whitespace() {
		if cxx_flag.find("\\") == Some(0) {
            nvcc.flag(cxx_flag);
        }
        else {
            nvcc.ar_flag(cxx_flag);
        }
    }

    for env_flag in env_flags {
        let mut flag_split = env_flag.0.split("=");
        if let Ok(val) = std::env::var(flag_split.next().unwrap()) {
            nvcc.flag(&format!("{}={}", env_flag.1, val));
        } else {
            nvcc.flag(&format!("{}={}", env_flag.1, flag_split.next().unwrap()));
        }
    }

    nvcc.compiler("nvcc")
        .file("./llama.cpp/ggml-cuda.cu")
        .no_default_flags(true)
        .extra_warnings(false)
        //.flag("-Wno-pedantic")
        .ar_flag("/NODEFAULTLIB:libcmt.lib")
        .include("./llama.cpp/ggml-cuda.h")
        .compile("ggml-cuda");

}

Plus, these flags are added as ar_flag instead of flag, though I am not sure if this actually achieves anything:
       cx_flags.push_str(" /W4 /Wall /wd4820 /wd4710 /wd4711 /wd4820 /wd4514 /MT");
       cxx_flags.push_str(" /W4 /Wall /wd4820 /wd4710 /wd4711 /wd4820 /wd4514 /MT");

Disclaimer: I have close to 0 experience in Rust and building C / C++, I was essentially just knocking rocks together and seeing what works.
Also, as an added bonus, OpenBlas is also completely broken for Windows! I gave up on trying to get this to work. Good luck!

Any update on this? Getting the same error.
I tried writing my own docker file using the same base image as the CUDA example and it works fine. The docker image uses Ubuntu; Windows support in general appears to be missing.
I haven't tested it but providing a quick patch like the person above me suggested would be helpful, having to compile and run my Cargo project from a docker container is a bit painful in my current configuration.

Any update on this? Getting the same error. I tried writing my own docker file using the same base image as the CUDA example and it works fine since the docker image uses Ubuntu; Windows support in general appears to be missing. I haven't tested it but providing a quick patch like the person above me suggested would be helpful, having to compile and run my Cargo project from a docker container is a bit painful in my current configuration.

Hey i wasnt getting any time due to my job i will try to look into this asap.

Windows support in general appears to be missing.

My fork has an updated build.rs that enables cuda on windows and metal on mac. you might be able to figure out what you need for windows from my commit, @FlooferLand:
https://github.com/mdrokz/rust-llama.cpp/commit/ae49e02ff667b329271f1975d78298571f7f8b76

@tbogdala Tried out your patch! Left me very confused xD
I'll just leave the log here: build.txt

@tbogdala Tried out your patch! Left me very confused xD I'll just leave the log here: build.txt

I received a somewhat related issue and it seems like the commit is trying to run a .sh file on Windows which of course does not work as expected. I was able to bypass the error by removing the mentions to the .sh script and making a function that does what the script tries to do. However, this just ended up causing issues when using llama_cpp_rs as a dependency in a different project,  so it is not a good solution. I'll have to take a deeper look into it.


AppendBlock (specifically appendTxi) assumes the source transaction specifies a height

If the source block is from Bitcoin Core for example, transactions don't have heights

Fixed in 8c7ab006572ee4132098ea7e005cc458c873b62f
We use the transaction heights from the concreteAppendablechain we are appending to


Store configs and downloads on user owned folders

SHADERed stores the config files and plugins next to the binary, that makes it hard to package it, maybe using https://github.com/sago007/PlatformFolders will be an easy way to solve the issue.

But it shouldn't store config files and plugins next to the binary when you make install it.. at least not on Ubuntu.
It should store stuff into ~/.local/share/shadered:
https://github.com/dfranx/SHADERed/blob/288f9c2b7df66a90a858016aafc0f88334a67e12/main.cpp#L93-L113
Though it'd be great if someone helped with Linux stuff sometimes in the future. I've never used it too much myself so I don't know all the dos and don'ts. Plus the bigger the source code is the harder it is for me to manage everything by myself.

I guess the error happens because the heuristic suppose the installation will be done on /usr/share/ but it does not need to be there (the pkg in AUR uses /opt/ for instance). I guess a less error-prone way will be to check for writing permissions on the folder containing the binary, so if it's read-only it means you can't save there and then you need to use the user's folder.

Thank you for the suggestion! I'll incorporate this solution in SHADERed v1.4.3

Should be fixed now. Thanks once again for the tip!


feat: add rust-once-cell src

rust-once-cell, attribute macro to require a function can not ever panic.
Log: rust-once-cell src would be added.
Issue: https://github.com/deepin-community/sig-deepin-sysdev-team/issues/145

/lgtm


Bug in multiple labels in .ui.checkbox

Multiple label tags will create multiple checkboxes which are not interactable. Checking the first box checks all boxed. Since they are not interactable, they should not be displayed.
Specially, it interacts adversely with tg-angular-validator as their validation methods add an extra label after the object. https://github.com/turinggroup/angular-validator

Not sure how to build Semantic, but the effective fix is to add "first-of-type" to .ui.checkbox label in line 26203 of semantic.css.


Enable participation to the project without much technical knowledge

Also people without knowledge of git and github should be able to contribute. E.g. if somebody can just write in only MS Word she/he a simple workflow should be offered. For example the raw document could be stored at Zenodo or in this repository and the content can then be included by somebody else. The same could/should be done for graphic contributions.

I think a very low contribution barrier is paramount.
Alternatively, we could include a document with simple steps how to contribute via git/GitHub and write in simple markdown. This is practised by the fantastic online bioinformatics book/text "An Introduction to Applied Bioinformatics" (IAB, https://github.com/gregcaporaso/An-Introduction-To-Applied-Bioinformatics) from @gregcaporaso, see the video "Getting Involved" here: http://readiab.org/. With this approach IAB is growing steadily, while readers can even correct typos while reading the book (via pull requests).
IMO, this issue is also related to #20 (Create online database of Open Science resources), as I don't know if a database is suitable in this case.


[Bug]: SpannerIO.Write does not support named schemas

What happened?
If named schemas are used in Spanner database, then SpannerIO does not know about tables created in that schema.
This leads to warnings during writes of the form:
Performance issue: Mutation references an unknown table: TABLENAME

Root cause is that SpannerIO.ReadSpannerSchema only reads information about tables in the default schema.
Issue Priority
Priority: 2 (default / most bugs should be filed as P2)
Issue Components

[ ] Component: Python SDK
[X] Component: Java SDK
[ ] Component: Go SDK
[ ] Component: Typescript SDK
[X] Component: IO connector
[ ] Component: Beam YAML
[ ] Component: Beam examples
[ ] Component: Beam playground
[ ] Component: Beam katas
[ ] Component: Website
[ ] Component: Infrastructure
[ ] Component: Spark Runner
[ ] Component: Flink Runner
[ ] Component: Samza Runner
[ ] Component: Twister2 Runner
[ ] Component: Hazelcast Jet Runner
[ ] Component: Google Cloud Dataflow Runner

.take-issue
.add-labels: gcp,io,spanner

.add-labels: gcp,io,spanner


Person Information is NULL

Even though my google plus account is functional and It shows user is connected. It goes to the else part and says that Person Information is NULL. Please Help. Is this with everyone or just me?

@mananwason  Can you please point where this code is located. I am not able to find it.

@batraabhishek It's in the settiings class, there you'll easily find it.

But could you get the app to work ?

Yes, I am able to successfully build and run the app. My bad but I am still not able find the above code or Settings.java class. May be I am missing something here.

I am also able to build and run the app, but after logging in nothing works


Module 1 Exercise 2 [Deborah Alaigba]

Please "drag & drop" exercise 2 file below.

Please UPDATE THE TITLE of this "Issue" in the title bar above with your name.
Time Series Exercise 2 - Alaigba.docx

Nice interpretation of the graphs!


fix(media): fix a bug that always showed thumbnails for image-typed media files

this PR resolves #58

:tada: This PR is included in version 1.0.0-dev.4 :tada:
The release is available on:

v1.0.0-dev.4
GitHub release
v1.0.0-dev.4

Your semantic-release bot :package::rocket:

:tada: This PR is included in version 1.1.0 :tada:
The release is available on:

v1.1.0
GitHub release
v1.1.0

Your semantic-release bot :package::rocket:


Always calculate all 4 cosine scores.

For comparison.
Maybe sequential rules aren't the best we could do here....

Fixed by 6af57ed8039e050adccd0f0c5dd10bf8f0be671f.


2.x Paged content within a post breaks core blocks that use wp-container (group, gallery)

Expected behavior
Gutenberg blocks that make use of a "wp-container" block including core/group and core/gallery should work as expected even in paged content.
This refers to single posts that are split into multiple pages using the Page Break block, i.e. <!--nextpage-->.
Actual behavior
These blocks display correctly on the first page only. After the first page break, they are broken.
Page 1 of post. Full-width group block inherits content width. Gallery displayed as 4 columns.

Page 2 of post, using same content. Full-width group block content is full width. Gallery displayed as single column.

Using the exact same content on page 1 and page 2 outputs different html.
Page 1 html:

Page 2 html

I believe WP 5.9 introduced the wp-container-{n} class which is added to certain blocks, along with some inline styles.
Steps to reproduce behavior
Follow examples here to set up paged content:
https://timber.github.io/docs/v2/guides/pagination/#paged-content-within-a-post
Screenshots show a custom theme I am developing, but I also tested this using the Timber Starter Theme (2.x), adding only the example code in order to enable paged content in post. This caused the same error.
I also tested paged content using the Twenty-TwentyTwo theme, Twenty-Nineteen, and Twenty-Seventeen.  These themes displayed the content without issue.
What version of WordPress, PHP and Timber are you using?
WordPress 6.0, PHP 5.7, Timber 2.0.0-alpha-4
How did you install Timber? (for example, from GitHub, Composer/Packagist, WP.org?)
composer

Thanks for testing out 2.x and for providing such a detailed issue description!
We fixed this in https://github.com/timber/timber/pull/2673. It will be in the next 2.0 release.


Rent contract doesn't change to the grace period when TFT balance isn't enough

Description
Testnet 2.2.0-rc4.
I rented a node with enough tfts, and after my balance reached zero, the rent contract didn't change into a grace period, even though I waited for more than 3 days.

I looked at this issue and I will compile my debug session into this comment
General info collected:
Network: Tesnet
runtime: 146
node id: 189
node twin id: 465
farm id: 1
user twin id: 755
rent contract id: 25,402

The twin account used to create the rent contract remains unfunded (for few days now) and his free balance have very tiny amount of TFTs (0.0000501 TFT).
The rent contract create about six days ago. specifically Thursday, November 9, 2023 1:35:42 PM GMT
Contract still in created state.
Node power status is up.
The contract id is normally exists in the billing loop at index 202 (contract id % billing frequency)
Billing frequency is default to 600 as expected.
Worth to mention this node have no extra fee set.
Node seems outdated since it has no associated relay configured on tfchain.

Now there is some observation which seems off to me:

The lock never updated since the contract creation. this lead me to check up-time reports.
Last up time report reported from node was one month ago!

I would say how in the first place the rent contract was created on a node that supposedly down and why the grid proxy reported it as up?
Data observed during debugging:
smartContractModule.contracts: Option<PalletSmartContractContract>
{
  version: 4
  state: Created
  contractId: 25,402
  twinId: 755
  contractType: {
    RentContract: {
      nodeId: 189
    }
  }
  solutionProviderId: null
}

system.account: FrameSystemAccountInfo
{
  nonce: 614
  consumers: 0
  providers: 1
  sufficients: 0
  data: {
    free: 501
    reserved: 0
    frozen: 0
    flags: 170,141,183,460,469,231,731,687,303,715,884,105,728
  }
}

{
  "data": {
    "rentContracts": [
      {
        "id": "0010812759-000002-f78d4",
        "nodeID": 189,
        "solutionProviderID": 0,
        "state": "Created",
        "twinID": 755,
        "gridVersion": 4,
        "createdAt": "1699536942",
        "contractID": "25402"
      }
    ]
  }
}

tfgridModule.nodePower: TfchainSupportNodePower
{
  state: Up
  target: Up
}

smartContractModule.contractLock: PalletSmartContractContractLock
{
  amountLocked: 0
  extraAmountLocked: 0
  lockUpdated: 1,699,536,942
  cycles: 0
}

smartContractModule.contractsToBillAt: Vec<u64>
[
  202
  25,402
]

smartContractModule.billingFrequency: u64
600

smartContractModule.dedicatedNodesExtraFee: u64
0

tfgridModule.nodes: Option<TfchainSupportNode>
{
  version: 5
  id: 189
  farmId: 1
  twinId: 465
  resources: {
    hru: 2,000,398,934,016
    sru: 512,110,190,592
    cru: 8
    mru: 16,746,598,400
  }
  location: {
    city: Unknown
    country: Belgium
    latitude: 50.8509
    longitude: 4.3447
  }
  publicConfig: null
  created: 1,647,950,826
  farmingPolicyId: 1
  interfaces: [
    {
      name: zos
      mac: 0c:c4:7a:51:e3:78
      ips: [
        <IP_ADDRESS>
        2a02:1802:5e:15:2d7a:428d:4a0d:7363
      ]
    }
  ]
  certification: Diy
  secureBoot: false
  virtualized: false
  serialNumber:<PHONE_NUMBER>
  connectionPrice: 80
}

tfgridModule.twins: Option<PalletTfgridTwin>
{
  id: 465
  accountId: 5CX4BwQvw7hPrCwAtZcnSq9RDyFK3cGe4Dg3TgsF9j9uw7fY
  relay: null
  entities: []
  pk: null
}

query MyQuery {
  uptimeEvents(where: {nodeID_eq: 189}, orderBy: uptime_DESC, limit: 1) {
    id
    timestamp
    uptime
    nodeID
  }
}

{
  "data": {
    "uptimeEvents": [
      {
        "id": "0010322211-000002-5ac55",
        "timestamp": "1696593624",
        "uptime": "48643141",
        "nodeID": 189
      }
    ]
  }
}

Further investigation will be at https://github.com/threefoldtech/tfchain/issues/893

This is now resolved.
Contract should now in the correct state
@A-Harby feel free to verify and close.
for more info see here:
https://github.com/threefoldtech/tfchain/issues/893#issuecomment-1823158777

Same issue with validators keys but with other validators this time.. 5GNE..
as I stated in the linked issue on my previous comment ..

The remaining work needed to verify and fix the validators’ setup can be tracked in the ops ticket mentioned in the previous comment.

@A-Harby according to operation devnet, qanet, and testnet validators are checked and some issues were corrected. can you verify this now for rent and node contracts?

Yes, one of the contracts started bailing, and the status changed to grace period.

But the other contract (the rent contract) is still not bailing on the Qanet.

@A-Harby 17516 on qa is a node contract, not rent contract.
How did you create this contract? It looks like it was not sent to the node after creation. Can you check if you can access this deployment to verify that this is really deployed ?
If you used the playground to create this deployment, please copy and paste the JSON deployment details here.

@A-Harby 17516 on qa is a node contract, not rent contract. How did you create this contract? It looks like it was not sent to the node after creation. Can you check if you can access this deployment to verify that workload is really deployed ?
If you used the playground to create this deployment, please copy and paste the JSON deployment details here.

I can't get deployment data; however, I tried to.

I can't get deployment data; however, I tried to.

There is nothing to bill for this contract. It has no IPs or CU/SU/NU usage.
The expected process is:

After creating the contract onchain, client notify the node about it.
The node reports the contract’s resource consumption to tfchain via report_contract_resources().
This sets the “used” capacity onchain for the contract.
This occurs when the workload is created, modified, or removed.
The node also reports NRU by calling add_reports extrinsic.

But for this specific contract, the workload looks like it wasn't deployed given the below observations:

NodeContractResources object not exists for this contract id (The most obvious reason for this that node didn't get notified about this contract after its creation)

smartContractModule.nodeContractResources: PalletSmartContractContractResources
{
  contractId: 0
  used: {
    hru: 0
    sru: 0
    cru: 0
    mru: 0
  }
}

No NU consumption report and lastUpdated same as the contract created time (it never updated)

smartContractModule.contractBillingInformationByID: PalletSmartContractContractBillingInformation
{
  previousNuReported: 0
  lastUpdated: 1,700,575,914
  amountUnbilled: 0
}

Beside OP can't get contract info, can't find the deployment info, and can't see the solution listed on his dashboard. So something went wrong during this deployment which i can't help with without enough information.

Anyway, I believe this is totally unrelated to the original billing issue discussed here. please do open an issue for it in the related repo (based in the way you used to deploy this).

Verification should done by covering at least these cases:

Any rent contract are being billed periodically.
Any Node contracts that have at least one IP reserved are being billed periodically.
Any Node contracts that have a nodeContractResources associated with its id and/or NU consumption are  being billed periodically.

Note: billed periodically =  amount are locked every billing cycle (600 blokcs or ~1 hour) then deducted every 24 billing cycle (~24 hours)

@sameh-farouk Thank you!
@ramezsaeed Hope you have these scenarios covered, if so, please close the issue with the link to the scenarios.

tried the recommended scenarios and the issue still exists

the contracts are created and stayed for more than hour and the wallet has no tfts but contracts did not go to grace period

Devnet, b75e477.
Same I also have some contracts that haven't started billing yet, even though some of them changed there status to grace period.

Okay, Contracts in grace period was proceeded successfully so these actually have no issues and can be escaped. next, I'm checking the contract with id 54243 to see what is happening

this contract should be billed by this validator
5DZVVFwJyEKEAUtrwmvPxkWzvjtFc73LybFK8xJse57eBP4X
I will have to check with ops regarding this one.
will update the related operation issue
next I'll check the other VM contract 54242

The other one 54242 seems are not deployed at all. so it can't be billed for that reason.
  contractId: 54,242
  used: {
    hru: 0
    sru: 0
    cru: 0
    mru: 0
  }
}

is the VM reachable ?

is the VM reachable ?

Yes, this is the VM I ssh in my comment above: https://github.com/threefoldtech/tfgrid-sdk-ts/issues/1415#issuecomment-1875563771

54,242

No, The one you posted are for   54,243 not  54,242. please double check

next I'll check @khaledyoussef24  contracts 54250, and 54251

No, The one you posted are for 54,243 not 54,242. please double check

I can't list it or find the node contract; I will report it in another issue.

For the validator design I suggest that:

in devnet case we have 4 validators
if a block goes to a validator which is not working fine; the VM will never get billed, right ?
so I suggest the validator keep this block only for one period and then release it so another validator which can check it in the next period and can bill it
in this case we don't get blocked on any validator turned off or doesn't work anymore.

For the validator design I suggest that:

in devnet case we have 4 validators
if a block goes to a validator which is not working fine; the VM will never get billed, right ?
so I suggest the validator keep this block only for one period and then release it so another validator which can check it in the next period and can bill it
in this case we don't get blocked on any validator turned off or doesn't work anymore.

I agree, the billing logic need to get improved in many aspects not this only. I am currently researching this.

@khaledyoussef24 from tfchain prospective 54250 contract's deployment hasn't provisioned yet. can you access this VM?
for the other contract, 54251 i can confirm it have an issue. will see which validator should handle this.

okay this 54251 leads to validator 5FXAK4MktAf5dQZURaC5sdRWZNgJPr8zB6iFr3Cbi1j4Svjv which is not billing any contracts in his slot. Already requesting from operations team in the related ticket to check it.

After tracked contracts to the problematic validators and check on them, my initial finding that is_next_block_author method is failing to detect the correct keys in these nodes' local store.
I'm investigation it further and will try to come with a fix soon.
More specific issue can be find here https://github.com/threefoldtech/tfchain/issues/932

Contracts 54251, 54249 wen to grace period after setting a new session key for the validator 5FXAK4MktAf5dQZURaC5sdRWZNgJPr8zB6iFr3Cbi1j4Svjv to meet certain assumption as described on https://github.com/threefoldtech/tfchain/issues/932.
smartContractModule.contracts: Option<PalletSmartContractContract>
{
  version: 4
  state: {
    GracePeriod: 9,246,850
  }
  contractId: 54,249
  twinId: 5,822
  contractType: {
    RentContract: {
      nodeId: 149
    }
  }
  solutionProviderId: null
}

smartContractModule.contracts: Option<PalletSmartContractContract>
{
  version: 4
  state: {
    GracePeriod: 9,246,850
  }
  contractId: 54,251
  twinId: 5,822
  contractType: {
    NodeContract: {
      nodeId: 149
      deploymentHash: aed69a2c07ee962def015fe94f11ea8d
      deploymentData: {type:vm,name:vmw9uf7,projectName:fullvm/vmw9uf7}
      publicIps: 0
      publicIpsList: []
    }
  }
  solutionProviderId: 1
}

This happens as well for contract 54243 after applying same process to validator 5DZVVFwJyEKEAUtrwmvPxkWzvjtFc73LybFK8xJse57eBP4X
smartContractModule.contracts: Option<PalletSmartContractContract>
{
  version: 4
  state: {
    GracePeriod: 9,246,844
  }
  contractId: 54,243
  twinId: 291
  contractType: {
    NodeContract: {
      nodeId: 16
      deploymentHash: d78a91992807d3253163fd5482b56aa5
      deploymentData: {type:vm,name:vmk176a,projectName:fullvm/vmk176a}
      publicIps: 0
      publicIpsList: []
    }
  }
  solutionProviderId: 1
}

Devnet should be fixed now, also QAnet is not suffering from this issue.
We will apply same process to mainnet tomorrow, testnet will follow..

Should be fixed now.

Verified,
I kept trying to reproduce the issue whenever I deployed an instance of any solution, and I even used different projects and clients (dashboard, TS client, mass deployer, and others) to help me reproduce the issue again, but till now it has not shown up again.
The mass deployer helped me the most with the issue because it could deploy a large amount of VMs all at once, ensuring that every validator will be assigned at least once to a contract, and after waiting for a few minutes, I was able to verify that all of them are being billed and converted to the grace period status once the money is not enough.

All of these tests have been done over the four networks through different clients.
And I also added the test cases for them to make sure it is tested with every release across the networks.
TC1324 - Intro

Verified,
I kept trying to reproduce the issue whenever I deployed an instance of any solution, and I even used different projects and clients (dashboard, TS client, mass deployer, and others) to help me reproduce the issue again, but till now it has not shown up again.
The mass deployer helped me the most with the issue because it could deploy a large amount of VMs all at once, ensuring that every validator will be assigned at least once to a contract, and after waiting for a few minutes, I was able to verify that all of them are being billed and converted to the grace period status once the money is not enough.
All of these tests have been done over the four networks through different clients.
And I also added the test cases for them to make sure it is tested with every release across the networks.

It's a relief that we finally wrapped the contracts' billing issue up. It was a tricky one since the reports of unbilled contracts from the testing team were due to different reasons and I had to debug each set separately.

@sameh-farouk, I updated the comment with the test cases if you could review them and check if it's enough to cover all the cases, and if not, please comment with a missing or incorrect one.

@A-Harby I Updated the test casees docs.


🤔Bug: Title tag doesn't support HTML/data-icon (anymore)

Problem:
Until 1.13.5 this was working to add custom icons to the select title:
<select title="">
But since everything gets escaped now (https://github.com/snapappointments/bootstrap-select/commit/9c0dc0dc06258c5993be3e5048b8919377a704ec), its not working anymore.
How can we add html to the title of the option tag? (For example for icons, like before)
"Hacky" workaround:
I've removed the title and data-selected-text-format attribute:
<select
title="DoSomething"
data-selected-text-format="static"...

And I'am using instead:
<select ...>
    <option data-icon="icon-show" data-hidden="true">DoSomething</option>

But this doesnt work very well if things get selected.. because the selected items overwrite the "workaround" title.
Possible Solution:
I think the best solution to this problem would be one of the following:
1.) Add a option to disable html removal / sanitizing completely.
OR
2.) Add <select data-icon=""> attribute, like you did it with <option data-icon""> elements.

Currently, data-content= and data-icon= doesnt work for the <select> element to display a icon. (like they do on an <option> element)
The disability to show icons on the <select title=""> is the reason why I'am now stuck on 1.13.5.
Btw. thank you for your outstanding work! @caseyjhol

This commit is the reason its not working:
https://github.com/snapappointments/bootstrap-select/commit/9c0dc0dc06258c5993be3e5048b8919377a704ec#diff-ff31d80eb2d28b06752ee372aa065210R1506
Workarounds like data-content= and data-icon= doesnt work for the <select element to display a icon.
Maybe you could add an option to disable this sanitisation?

Found a solution!
I've removed the title and data-selected-text-format attribute:
<select
                                                            title="DoSomething"
                                                            data-selected-text-format="static"...

And I'am using instead:
<option data-icon="icon-show" data-hidden="true">
                                                            DoSomething
                                                        </option>

A good workaround!

Would be great to have support for HTML for option.title as well! (See workaround in #2640)

Any updates on this topic?


chore: bdd-sbhg-import-1584523530 to 0.0.1

chore: Promote bdd-sbhg-import-1584523530 to version 0.0.1

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approvers:
If they are not already assigned, you can assign the PR to them by writing /assign  in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


drivers/wifi/winc1500 build warning error on 64 bit targets

Describe the bug
Building drivers/wifi/winc1500/wifi_winc1500.c for a 64 bit target (like native_sim//64) produces a lot of warnings/errors like
drivers/wifi/winc1500/wifi_winc1500.c:332:25: error: cast from pointer to integer of different size [-Werror=pointer-to-int-cast]
  332 |         SOCKET socket = (int)context->offload_context;

Note SOCKET is 8bit wide :)
To Reproduce
Steps to reproduce the behavior:

mkdir build; cd build
cmake -GNinja -DBOARD=native_sim//64 ../tests/drivers/build_all/wifi/ -DCONFIG_SPI=y -DDTC_OVERLAY_FILE="spi_devices.overlay"
ninja
See errors

OR
twister -p native_sim/native/64 -T tests/drivers/build_all/wifi/
Expected behavior
No build errors or warnings
Impact
Failing test for 64bit targets
Logs and console output
See above
Environment (please complete the following information):

OS: Ubuntu Linux 24.04
Toolchain: Host gcc 14.2
Commit SHA or Version used: Main as of now 31fea97e05fd524b089c70281356d07df7788349

Additional context

CC @pnndra @tbursztyka


Regexp rules

This pull request add missing support for regexp rules. Please see the doc for details:
https://adblockplus.org/en/filters#regexps

Current coverage is 97.26% (diff: 100%)

No coverage report found for master at 7bf903e.

Powered by Codecov. Last update 7bf903e...e612568

Should xfail be removed from this test?

Also, README states that regex rules are not supported; this line should be removed if we in fact support regexes in rules.

How about now?

Hope now is fine...

Hi. Is there any chance to accept this pull request after my last fixes?

Sorry, I missed the update. It looks good now, thanks for working on it!


[resolves #2036] Add support for component xmlrpc

[resolves #2049] Add support for dataformat xmlrpc

retest this please


Add iam_role_name to outputs of consul-cluster module. Fixes #106

Thank you for your submission! We require that all contributors sign our Contributor License Agreement ("CLA") before we can accept the contribution. Read and sign the agreement
Learn more about why HashiCorp requires a CLA and what the CLA includes
Have you signed the CLA already but the status is still pending? Recheck it.

Is this not the same as https://github.com/hashicorp/terraform-aws-consul/blob/master/modules/consul-cluster/outputs.tf#L21-L24 ?

@brikis98, IAM Role IDs are not IAM Role Names. The Terraform AWS Provider takes the latter in most cases as input to other resources.

Are you 100% sure? We use ID all over the place and the docs say that both return the same thing...

@brikis98, yes. If you look at these docs, you'll see that these take the IAM Role Name: https://www.terraform.io/docs/providers/aws/r/iam_role_policy_attachment.html
I had to write a "hack" like this in my code:

data "aws_arn" "consul_iam_role" {
  arn = "${module.consul_cluster.iam_role_arn}"
}

resource "aws_iam_role_policy_attachment" "consul" {
  role       = "${replace(data.aws_arn.consul_iam_role.resource, "role/", "")}"
  policy_arn = "arn:aws:iam::aws:policy/service-role/..."
}

Yea, it's definitely confusing. Closing now, thanks!


Clean repeated static string interpolation

Fixes

Fixes #4980 by @sarayourfriend
Description

Testing Instructions

Checklist

[ ] My pull request has a descriptive title (not a vague title likeUpdate index.md).
[ ] My pull request targets the default branch of the repository (main) or a parent feature branch.
[ ] My commit messages follow best practices.
[ ] My code follows the established code style of the repository.
[ ] I added or updated tests for the changes I made (if applicable).
[ ] I added or updated documentation (if applicable).
[ ] I tried running the project locally and verified that there are no visible errors.
[ ] I ran the DAG documentation generator (ov just catalog/generate-docs for catalog
PRs) or the media properties generator (ov just catalog/generate-docs media-props
for the catalog or ov just api/generate-docs for the API) where applicable.

Developer Certificate of Origin

Developer Certificate of Origin
Developer Certificate of Origin
Version 1.1

Copyright (C) 2004, 2006 The Linux Foundation and its contributors.
1 Letterman Drive
Suite D4700
San Francisco, CA, 94129

Everyone is permitted to copy and distribute verbatim copies of this
license document, but changing it is not allowed.

Developer's Certificate of Origin 1.1

By making a contribution to this project, I certify that:

(a) The contribution was created in whole or in part by me and I
    have the right to submit it under the open source license
    indicated in the file; or

(b) The contribution is based upon previous work that, to the best
    of my knowledge, is covered under an appropriate open source
    license and I have the right under that license to submit that
    work with modifications, whether created in whole or in part
    by me, under the same open source license (unless I am
    permitted to submit under a different license), as indicated
    in the file; or

(c) The contribution was provided directly to me by some other
    person who certified (a), (b) or (c) and I have not modified
    it.

(d) I understand and agree that this project and the contribution
    are public and that a record of the contribution (including all
    personal information I submit with it, including my sign-off) is
    maintained indefinitely and may be redistributed consistent with
    this project or the open source license(s) involved.

Hi @tejaswarathe! Great work on this PR! Is anything blocking you? It seems that you only need to run lint. Let us know if you need any help with anything.

Converting to draft as I still see some pending tasks in this PR.


Remove applications page of russian lang

This page no longer exists in the English documentation

So this is a good change. I see the applications file is not being used (Myspace! ha)
But, what does removing the screenshots have to do with removing the Russian page though? I'd like to see this PR handle just the Russian page - so move those other changes to another PR (I didn't confirm if they are valid here)
@crandmck let us know if you disagree

Ahh I see. All those images are from that page. I'll need to verify those images still then.
@crandmck I assume we don't need this page? Just wanted to double check https://expressjs.com/ru/resources/applications.html
Weirdly this page is not even Russian. Not sure why it's sitting here.

Yes, that page is no longer used and can be removed.  All the other langs were removed in a previous PR, and this was overlooked, apparently.
FYI for community translations, pages that need translation are added with their en content initially, with the intent to replace with translated version later.
I changed the title of the PR to reflect that it also removes the referenced image files, and I checked that those files are not used anywhere else.


mapClick event return a string "c"

In Angular 11 and agm 3.0.0-beta.0, mapClick event doesn't return a object, the event return a string that contains a "c".
My template:
<agm-map [latitude]="lat" [longitude]="lng" (mapClick)="mapClicked($event)">
  <agm-marker
    [latitude]="lat"
    [longitude]="lng"
  ></agm-marker>
</agm-map>

My event in ts:
public mapClicked($event: google.maps.MouseEvent): void {
  // Should return a object but return a string "c"
  console.log("Map Click", $event);
  // If it works I would do this -> this.lat = $event.coords.lat; this.lng = $event.coords.lng;
}

Image of log:
.

This is a long standing problem that still faces the app, the google map typings installed had a bug with the click event. angular-google-maps needs an update to use a different version type to work correctly.
@SebastianM do you have any intentions of updating the google map typings and the project for Angular 11 and 12?


Update Cosmos DB  Partition Size Discussion

In the section talking about Cosmos DB it says: "If you store a separate document for every device message, using the device ID as a partition key would quickly exceed the 10-GB limit per partition"
The current logical partition size limit is now 20 GB. Also, the discussion makes it seem like you have to pick a particular attribute for a partition key, when it's quite common in these scenarios for customers to create a synthetic key composed of multiple attributes to help with data distribution.

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

ID: cf328b1f-6bc5-dff2-619f-82b95030c3cf
Version Independent ID: 4c7bd8b7-5ebb-5b1e-010b-2acd7fdcec0f
Content: Azure IoT reference architecture - Azure Reference Architectures
Content Source: docs/reference-architectures/iot.yml
Service: architecture-center
Sub-service: reference-architecture
GitHub Login: @doodlemania2
Microsoft Alias: pnp

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

It appears this was addressed some time ago.  If help is still needed, please feel free to re-open with additional details.


Inject cache metadata in model classes only if needed

Pass the dogpile argument down to the ModelClass class to output cache metadata only if needed
Fix the flask argument dependency test to raise an error only if dogpile is required

Great points. I decided to remove dogpile support all together.


Fix typo on testcase.py

Thank you for your contribution, I'll merge this with the next release of Pynguin.


Swift 6 compatibility

As a developer I would like to benefit from improvements made with Swift 6.
Acceptance criteria

Swift 6 support has been enabled (not to be delivered before Xcode 16 is officially available).

Tasks

[ ] Update version in package manifest.
[ ] Fix issues.
[ ] Update Swift version in the demo.
[ ] Fix issues.
[ ] Enable explicit modules in the demo target.
[ ] Report issues found to Apple, if any.

There is an issue with structs and strict concurrency checking:
public struct Flags {
    public static let none = Self(isEnabled: false)       // Static property 'none' is not concurrency-safe because non-'Sendable' type 'Flags' may have shared mutable state
    public let isEnabled: Bool
}

struct Flags2 {
    public static let none = Self(isEnabled: false)       // No issue
    public let isEnabled: Bool
}

struct Flags3 {
    static let none = Self(isEnabled: false)                  // No issue
    let isEnabled: Bool
}

I am not really understanding why this happens with a constant and since global instantiation is guaranteed to be safe. But this might be worth a deeper look and possibly a bug report if appropriate.

I investigated further how we could implement Swift 6 concurrency checking in Pillarbox, especially in the PillarboxPlayer package.
Current thoughts:

I am not sure we should blindly adopt @MainActor everywhere. AVPlayer is marked as @MainActor but almost all its public API is marked as nonisolated since iOS / tvOS 16, which means that most of the API can be used off the main thread. AVFoundation documentation was updated accordingly.
Instead we should probably focus in making Player (and other types) @Observable. It namely seems that no additional work is required for SwiftUI updates to be performed on the main thread. This would also make Player more flexible and would avoid @MainActor spreading through the codebase like a disease.
We can use weakAssign(to:on:) to plug publishers to properties of the observable object, without switching threads. If we want to have stronger guarantees about execution onto a single thread we could have Player manage its own dispatch queue and receive publisher output on them consistently.

I will write a small prototype to evaluate the feasibility of this approach and attach it here.

I opened an issue for thread-safety. Player should likely not be annotated as @MainActor unnecessarily, we should rather document that in the meantime it is safe to be used on the main thread only.


Type mapping across connectors

As different BIM (and CAD) software use and handle types in completely different ways we need to explore a solution that would allow users to set custom "mappings" between the incoming data in a stream and the available families/types in the receiving software.
This solution should not be Revit centric as many other host apps will have the same issues, but Revit could definitely be a starting point.
See: https://speckle.community/t/speckle-library-manager/2853/4
Related also to #1240

Closing as implemented in https://github.com/specklesystems/speckle-sharp/pull/1489 for now


Convert Figma UI to React Storybook

Figma URL: https://www.figma.com/file/PAA0JKidFMVK44KRRWB1zL/SnowUI?type=design&node-id=8300%3A425&mode=design&t=sO57YHGjNQ38VUga-1
This UI plans to be a UI for this project

Hi, I'd like to help you with this issue.


Components

This pr makes hops (and hops based products) strictly ES2015 compliant: no more decorators, no more class properties. It also introduces a couple of new helper functions (createContainer(), createComponent()).

It seems like this PR is also removing the example of importing values
in CSS, is that intended?

Yeah, it is: the additional css file was not copied by the setup script, thus breaking package installation. And I wasn't keen on complicating that script further.
I was hoping that with the comments/examples, style.css would be enough to show most of the css possibilities...
Btw, this PR still needs some work anyway.

The new syntax looks good to me. I'll go for it in malt ... I think we need a more extensive description of internal coding guidelines though, because this is not opinionated anymore (which is a good thing by the way).

@matthias-reis can you tell more about malt?

Yes, of course. malt should become the internal counterpart of hops and acts like e.g. twitter bootstrap - or like a newer version of our xing patterns.
As Ricard pointed out, this is done in one repo frontend/maltwhich acts like a babel inspired monorepo. Any pattern is a very lightweight ES6 React Component, holding its own dependencies. So all users (= developers) can use it partially - e.g only the grid, only the frame, only header and footer of the frame and so on.
There will also be some special packages that are worth mentioning:

@malt/predicates will simply be a collection of xing css variables
@malt/basestyles will be initial global styling (providing normalization and styling body, h1-h6, p ...)
@malt/shell will provide a component that renders the whole frame for you (header, navigation and footer like <XingShell>[... your stuff ...]</XingShell>)
@malt/svg (not yet started) will provide you with an svg sprite solution (also used as a successor of the iconfont)
@malt/i18n will provide connection to our phrase app output and other internationalization tools

I'm sure there's more to come ...
If you have questions or more ideas that go into that direction, please contact me or contribute with pull requests.
This week my personal goal is to get the publishing process ready so you will be able to use these packages together with our internal npm repo.

Hmmm ... I'm going to use parts of that in the readme.md :-) @naoisegolden Thanks for pointing to the missing docs ...

@matthias-reis sounds really good, thanks.


[google_maps_flutter_web]  Bad state: Cannot add new events after calling close

Steps to Reproduce

https://github.com/amritoperto/google_maps_test

Run flutter run. make sure it runs on web
Press the + fab repeatedly until you see the error, sometimes it takes a few presses but it happens a lot quicker in a production app. Its just in this min repro that it takes quite a few presses of the fab.

Expected results:
No errors in the logs
Actual results:
An Error in the logs
errors.dart:187 Uncaught (in promise) Error: Bad state: Cannot add new events after calling close
    at Object.throw_ [as throw] (errors.dart:236)
    at _AsyncBroadcastStreamController.new.add (broadcast_stream_controller.dart:243)
    at google_maps_controller.dart:167
    at _RootZone.runUnaryGuarded (zone.dart:1546)
    at _ControllerSubscription.new.[_sendData] (stream_impl.dart:341)
    at _ControllerSubscription.new.[_add] (stream_impl.dart:271)
    at _SyncStreamController.new.[_sendData] (stream_controller.dart:733)
    at _SyncStreamController.new.[_sendData] (stream_controller.dart:441)
    at _SyncStreamController.new.[_add] (stream_controller.dart:607)
    at _SyncStreamController.new.add (stream_controller.dart:554)
    at SubscribeStreamProvider.new.add (async.dart:47)
    at async.dart:22
    at Object._checkAndCall (operations.dart:324)
    at Object.dcall (operations.dart:329)
    at google_maps_src.g.dart:7152
    at Ti.wrapper (js_patch.dart:355)
    at gf.hj (js?key=AIzaSyBUzRwyZCWi95rYn2vvoLlbbfeqMvpEx0w:210)
    at Object._.M.trigger (js?key=AIzaSyBUzRwyZCWi95rYn2vvoLlbbfeqMvpEx0w:207)
    at b (map.js:33)

[+3277 ms] DwdsInjector: Received request for entrypoint at http://localhost:65479/main_module.bootstrap.js
[        ] MetadataProvider: Loading debug metadata...
[   +5 ms] MetadataProvider: Loaded debug metadata
[   +3 ms] DwdsInjector: Injected debugging metadata for entrypoint at http://localhost:65479/main_module.bootstrap.js
[+2839 ms] 0
[   +1 ms] 0
[+1567 ms] You must enable Billing on the Google Cloud Project at https://console.cloud.google.com/project/_/billing/enable Learn more at
                    https://developers.google.com/maps/gmp-get-started
[ +701 ms] 1
[        ] 1
[ +174 ms] 2
[        ] 0
[ +164 ms] 3
[        ] 1
[ +153 ms] 4
[        ] 0
[ +134 ms] 5
[        ] 1
[ +146 ms] 6
[        ] 0
[ +162 ms] 7
[        ] 1
[ +134 ms] 8
[        ] 0
[ +139 ms] 9
[        ] 1
[ +150 ms] 10
[   +1 ms] 0
[ +150 ms] 11
[        ] 1
[  +85 ms] Error: Bad state: Cannot add new events after calling close
               at Object.throw_ [as throw] (http://localhost:65479/dart_sdk.js:5331:11)
               at _AsyncBroadcastStreamController.new.add (http://localhost:65479/dart_sdk.js:32888:55)
               at http://localhost:65479/packages/google_maps_flutter_web/src/types.dart.lib.js:528:35
               at _RootZone.runUnaryGuarded (http://localhost:65479/dart_sdk.js:38812:11)
               at _ControllerSubscription.new.[_sendData] (http://localhost:65479/dart_sdk.js:32429:22)
               at _ControllerSubscription.new.[_add] (http://localhost:65479/dart_sdk.js:32375:26)
               at _SyncStreamController.new.[_sendData] (http://localhost:65479/dart_sdk.js:35337:34)
               at _SyncStreamController.new.[_sendData] (http://localhost:65479/dart_sdk.js:35447:32)
               at _SyncStreamController.new.[_add] (http://localhost:65479/dart_sdk.js:35145:26)
               at _SyncStreamController.new.add (http://localhost:65479/dart_sdk.js:35110:19)
               at SubscribeStreamProvider.new.add (http://localhost:65479/packages/js_wrapping/util/async.dart.lib.js:76:22)
               at http://localhost:65479/packages/google_maps/util/async.dart.lib.js:1454:21
               at Object._checkAndCall (http://localhost:65479/dart_sdk.js:5541:16)
               at Object.dcall (http://localhost:65479/dart_sdk.js:5546:17)
               at http://localhost:65479/packages/google_maps/util/async.dart.lib.js:19782:191
               at Ti.wrapper (http://localhost:65479/dart_sdk.js:60942:30)
               at gf.hj (https://maps.googleapis.com/maps/api/js?key=AIzaSyBUzRwyZCWi95rYn2vvoLlbbfeqMvpEx0w:210:414)
               at Object._.M.trigger (https://maps.googleapis.com/maps/api/js?key=AIzaSyBUzRwyZCWi95rYn2vvoLlbbfeqMvpEx0w:207:385)
               at mf (https://maps.googleapis.com/maps/api/js?key=AIzaSyBUzRwyZCWi95rYn2vvoLlbbfeqMvpEx0w:84:322)
               at mf (https://maps.googleapis.com/maps/api/js?key=AIzaSyBUzRwyZCWi95rYn2vvoLlbbfeqMvpEx0w:84:304)
               at Xw._.N.notify (https://maps.googleapis.com/maps/api/js?key=AIzaSyBUzRwyZCWi95rYn2vvoLlbbfeqMvpEx0w:212:888)
               at https://maps.googleapis.com/maps-api-v3/api/js/43/8/map.js:30:392
               at _.ui._.n.Zi (https://maps.googleapis.com/maps/api/js?key=AIzaSyBUzRwyZCWi95rYn2vvoLlbbfeqMvpEx0w:242:491)
[ +196 ms] Error: Bad state: Cannot add new events after calling close
                        at Object.throw_ [as throw] (http://localhost:65479/dart_sdk.js:5331:11)
                        at _AsyncBroadcastStreamController.new.add (http://localhost:65479/dart_sdk.js:32888:55)
                        at http://localhost:65479/packages/google_maps_flutter_web/src/types.dart.lib.js:534:33
                        at _RootZone.runUnaryGuarded (http://localhost:65479/dart_sdk.js:38812:11)
                        at _ControllerSubscription.new.[_sendData] (http://localhost:65479/dart_sdk.js:32429:22)
                        at _ControllerSubscription.new.[_add] (http://localhost:65479/dart_sdk.js:32375:26)
                        at _SyncStreamController.new.[_sendData] (http://localhost:65479/dart_sdk.js:35337:34)
                        at _SyncStreamController.new.[_sendData] (http://localhost:65479/dart_sdk.js:35447:32)
                        at _SyncStreamController.new.[_add] (http://localhost:65479/dart_sdk.js:35145:26)
                        at _SyncStreamController.new.add (http://localhost:65479/dart_sdk.js:35110:19)
                        at SubscribeStreamProvider.new.add (http://localhost:65479/packages/js_wrapping/util/async.dart.lib.js:76:22)
                        at http://localhost:65479/packages/google_maps/util/async.dart.lib.js:1454:21
                        at Object._checkAndCall (http://localhost:65479/dart_sdk.js:5541:16)
                        at Object.dcall (http://localhost:65479/dart_sdk.js:5546:17)
                        at http://localhost:65479/packages/google_maps/util/async.dart.lib.js:19782:191
                        at Ti.wrapper (http://localhost:65479/dart_sdk.js:60942:30)
                        at gf.hj (https://maps.googleapis.com/maps/api/js?key=AIzaSyBUzRwyZCWi95rYn2vvoLlbbfeqMvpEx0w:210:414)
                        at Object._.M.trigger (https://maps.googleapis.com/maps/api/js?key=AIzaSyBUzRwyZCWi95rYn2vvoLlbbfeqMvpEx0w:207:385)
                        at b (https://maps.googleapis.com/maps-api-v3/api/js/43/8/map.js:33:79)

Analyzing google_maps_test...
No issues found! (ran in 2.7s)

 flutter doctor -v
[✓] Flutter (Channel beta, 1.26.0-17.4.pre, on macOS 11.2.1 20D74 darwin-x64, locale en-CA)
    • Flutter version 1.26.0-17.4.pre at /Users/amritk/Developer/flutter
    • Framework revision 48c9d3e0e1 (5 hours ago), 2021-02-10 11:46:09 -0800
    • Engine revision 2c527d6c7e
    • Dart version 2.12.0 (build 2.12.0-259.8.beta)

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

[✓] Xcode - develop for iOS and macOS
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 12.4, Build version 12D4e
    • CocoaPods version 1.10.1

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

[✓] Android Studio (version 4.1)
    • 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 1.8.0_242-release-1644-b3-6915495)

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

[✓] Connected device (1 available)
    • Chrome (web) • chrome • web-javascript • Google Chrome 88.0.4324.150

• No issues found!

Is it due to a memory leak? Is there a better way to kill the map?

@amritoperto
I was able to replicate the error. Looking at it:
Error: Bad state: Cannot add new events after calling close, it seems when we tap on FAB repeatedly, the stream is getting closed and tapping again tries to add it and hence the error.
You may try to explore and then use StreamController class and it's property isClosed to check if stream is closed, and if not, tap or perform next action.
You may read more about it here : https://api.flutter.dev/flutter/dart-async/StreamController-class.html

Hey @darshankawar thanks for the quick reply, sorry but I'm pretty new to flutter. Did you mean put the googleMaps widget into a new widget and add a streamController there? Or is there a way to just add a streamController to this?
Thanks!

As a temp workaround I switched go the google_maps pub for now, it doesn't have this issue:
https://github.com/amritoperto/google_maps_test/tree/text/new-lib

@ghost
I'd suggest you to ask this question on StackOverflow because it seems to be a code implementation issue. Please see https://flutter.dev/community for resources and if you need help with your code, please see https://www.reddit.com/r/flutterhelp/
Closing from here. If you disagree or think this is indeed a bug in the plugin, write in comments and I'll reopen it.


github private  repository need to add username

HKube micro-service
To which micro-service the bug related to, Dashboard ,Monitor-Server etc.
Describe the bug
in order to use github private  repository it require username and token/password currently you can only add token and not the user name
need to add username to the new algorithm form

There is no need for username, only token

tested on version 1.2.39 (on the CICD )