Will not update org when local changes are saved

I have installed version 1.93 of the plugin in my org.  When I run the command, it opens the org saying Local Dev is enabled.  However whenever I try to update a component locally, those changes do not reflect in the lwc.  Here is a screenshot of the command I'm running:

I have tried to downgrade to 1.80 with the same results.  I had a call with SF technical support and we went through a few things and all turned out to be setup correctly.
This is the url that is shown on the SF org and it looks correct:
/lightning/n/intlAgree__Attribute_Manager?0.aura.ldpServerUrl=ws%3A%2F%2Flocalhost%3A8081&0.aura.ldpServerId=1I9Nq0000003RjdKAE&0.aura.mode=DEVPREVIEW
Is there a way I can debug this to see what's happening?

Same here.
I am currently in lightning-dev 1.9.3 and here's my output for sf --version
@salesforce/cli/2.66.7 darwin-arm64 node-v20.15.1

I am currently using the sf CLI installed through npm, but I was using the .pkg before this without any luck either.

@apettitt Is there an error message in the browser console? With the information provided in this issue, we can try to guess what's the likely cause.
Does you org have Lightning Web Security enabled? We have a known issue related to that.
Local Dev wasn't working if an org was using Legacy Locker service(https://github.com/salesforcecli/plugin-lightning-dev/issues/213) and we are rolling out(starting 11/20/2024) a fix for that.

@ravijayaramappa  - yes, I have lws turned on as the manage package I write for requires lws.  Here are some things I found in the console, not sure if they'll help or not.  The component I'm working on is called attributeManager and the Managed Package namespace is intlAgree.
[LAF:PlatformStateManager:ResolvePage] Input pageReference {"type":"standard__navItemPage","attributes":{"apiName":"intlAgree__Attribute_Manager"},"state":{}} produced page {"type":"standard__navItemPage","attributes":{"apiName":"intlAgree__Attribute_Manager"},"componentDefRef":{"componentDef":"app_flexipage:lwcAppFlexipageWrapper","attributes":{"flexiPageDeveloperName":"intlAgree__Attribute_Manager","label":"Attribute Manager","iconColor":"50ceb9","iconUrl":"https://data-customization-9101-dev-ed.scratch.my.salesforce.com/img/icon/t4v35/custom/custom8_120.png"}},"state":{}}
aura_devpreview.js:45803 	[LAF:Filters:applyPageFilters] invoked: {"status":"PENDING","disposition":null,"dispositionValue":null,"to":{"pageReference":{"type":"standard__navItemPage","attributes":{"apiName":"intlAgree__Attribute_Manager"},"state":{}},"replace":false,"redirect":false,"page":{"type":"standard__navItemPage","attributes":{"apiName":"intlAgree__Attribute_Manager"},"componentDefRef":{"componentDef":"app_flexipage:lwcAppFlexipageWrapper","attributes":{"flexiPageDeveloperName":"intlAgree__Attribute_Manager","label":"Attribute Manager","iconColor":"50ceb9","iconUrl":"https://data-customization-9101-dev-ed.scratch.my.salesforce.com/img/icon/t4v35/custom/custom8_120.png"}},"state":{}},"location":"/lightning/n/intlAgree__Attribute_Manager?0.aura.ldpServerUrl=ws%3A%2F%2Flocalhost%3A8081&0.aura.ldpServerId=1I9Nq0000003RjdKAE&0.aura.mode=DEVPREVIEW"},"from":{"location":"/lightning/n/intlAgree__Home?0.aura.ldpServerUrl=ws%3A%2F%2Flocalhost%3A8081&0.aura.ldpServerId=1I9Nq0000003RjdKAE&0.aura.mode=DEVPREVIEW","pageReference":{"type":"standard__app","attributes":{"appTarget":null,"pageRef":{"type":"standard__navItemPage","attributes":{"apiName":"intlAgree__Home"},"state":{}}}}},"subscribers":[null]}
aura_devpreview.js:45803 	[LAF:StateFilters:replaceComponentDef] invoked
aura_devpreview.js:45803 	[LAF:StateFilters:eventTypeDetect] invoked
aura_devpreview.js:45803 	[LAF:StateFilters:contentDetect] invoked
aura_devpreview.js:45803 	[LAF:StateFilters:contentDetect] applied. pageTx.to.page: {"type":"standard__navItemPage","attributes":{"apiName":"intlAgree__Attribute_Manager"},"componentDefRef":{"componentDef":"app_flexipage:lwcAppFlexipageWrapper","attributes":{"flexiPageDeveloperName":"intlAgree__Attribute_Manager","label":"Attribute Manager","iconColor":"50ceb9","iconUrl":"https://data-customization-9101-dev-ed.scratch.my.salesforce.com/img/icon/t4v35/custom/custom8_120.png"}},"state":{}}
aura_devpreview.js:45803 	[LAF:Filters:applyPageFilters] applied: to.page = {"type":"standard__navItemPage","attributes":{"apiName":"intlAgree__Attribute_Manager"},"componentDefRef":{"componentDef":"app_flexipage:lwcAppFlexipageWrapper","attributes":{"flexiPageDeveloperName":"intlAgree__Attribute_Manager","label":"Attribute Manager","iconColor":"50ceb9","iconUrl":"https://data-customization-9101-dev-ed.scratch.my.salesforce.com/img/icon/t4v35/custom/custom8_120.png"}},"state":{}}
aura_devpreview.js:45803 jobLibrary - Job: handlePageTx, Stage: tasks, finished processing (8) functions.
aura_devpreview.js:45803 jobLibrary - Job: handlePageTx, Stage: posttasks, skipped processing: no functions.
aura_devpreview.js:45803 jobLibrary - Job: handlePageTx, Stage: after, start processing.
aura_devpreview.js:45803 -----------------------------------------------
[LAF:OneAppRouter:onExternalLocationChange] invoked: /lightning/n/intlAgree__Attribute_Manager?0.aura.ldpServerUrl=ws%3A%2F%2Flocalhost%3A8081&0.aura.ldpServerId=1I9Nq0000003RjdKAE&0.aura.mode=DEVPREVIEW
aura_devpreview.js:45803 	[LAF:Filters:applyLocationFilters] invoked: {"status":"PENDING","disposition":null,"dispositionValue":null,"to":{"location":"/lightning/n/intlAgree__Attribute_Manager?0.aura.ldpServerUrl=ws%3A%2F%2Flocalhost%3A8081&0.aura.ldpServerId=1I9Nq0000003RjdKAE&0.aura.mode=DEVPREVIEW","isHistorical":true},"from":{"location":"/lightning/n/intlAgree__Attribute_Manager?0.aura.ldpServerUrl=ws%3A%2F%2Flocalhost%3A8081&0.aura.ldpServerId=1I9Nq0000003RjdKAE&0.aura.mode=DEVPREVIEW","pageReference":{"type":"standard__app","attributes":{"appTarget":null,"pageRef":{"type":"standard__navItemPage","attributes":{"apiName":"intlAgree__Attribute_Manager"},"state":{}}}}},"subscribers":[]}
aura_devpreview.js:45803 jobLibrary - Job: handlePageTx, Stage: after, finished processing (1) functions.
aura_devpreview.js:45803 	[LAF:LocationFilters:blockSameLocationChange] invoked
aura_devpreview.js:45803 	[LAF:LocationFilters:blockSameLocationChange] applied, with result: {"status":"PENDING","disposition":"BLOCKED","dispositionValue":"to.location is equal to the current/from location.","to":{"location":"/lightning/n/intlAgree__Attribute_Manager?0.aura.ldpServerUrl=ws%3A%2F%2Flocalhost%3A8081&0.aura.ldpServerId=1I9Nq0000003RjdKAE&0.aura.mode=DEVPREVIEW","isHistorical":true},"from":{"location":"/lightning/n/intlAgree__Attribute_Manager?0.aura.ldpServerUrl=ws%3A%2F%2Flocalhost%3A8081&0.aura.ldpServerId=1I9Nq0000003RjdKAE&0.aura.mode=DEVPREVIEW","pageReference":{"type":"standard__app","attributes":{"appTarget":null,"pageRef":{"type":"standard__navItemPage","attributes":{"apiName":"intlAgree__Attribute_Manager"},"state":{}}}}},"subscribers":[]}
aura_devpreview.js:45803 [LAF:ContentManager:CreateWrapper] Creating wrapper that starts minimized
aura_devpreview.js:45803 	[LAF:applyLocationFilters] applied. to.location = /lightning/n/intlAgree__Attribute_Manager?0.aura.ldpServerUrl=ws%3A%2F%2Flocalhost%3A8081&0.aura.ldpServerId=1I9Nq0000003RjdKAE&0.aura.mode=DEVPREVIEW
aura_devpreview.js:45803 jobLibrary - Job: handlePageTx completed processing with a status of completed
aura_devpreview.js:45803 jobLibrary - Finished running. All jobs processed.
aura_devpreview.js:45803 [LAF:OneAppRouter:resolveLocation] {"status":"COMPLETE","disposition":"BLOCKED","dispositionValue":"to.location is equal to the current/from location.","to":{"location":"/lightning/n/intlAgree__Attribute_Manager?0.aura.ldpServerUrl=ws%3A%2F%2Flocalhost%3A8081&0.aura.ldpServerId=1I9Nq0000003RjdKAE&0.aura.mode=DEVPREVIEW","isHistorical":true},"from":{"location":"/lightning/n/intlAgree__Attribute_Manager?0.aura.ldpServerUrl=ws%3A%2F%2Flocalhost%3A8081&0.aura.ldpServerId=1I9Nq0000003RjdKAE&0.aura.mode=DEVPREVIEW","pageReference":{"type":"standard__app","attributes":{"appTarget":null,"pageRef":{"type":"standard__navItemPage","attributes":{"apiName":"intlAgree__Attribute_Manager"},"state":{}}}}},"subscribers":[]}
aura_devpreview.js:45803 	[LAF:Filters:applyPageRefFilters] invoked: {"status":"COMPLETE","disposition":"BLOCKED","dispositionValue":"to.location is equal to the current/from location.","to":{"location":"/lightning/n/intlAgree__Attribute_Manager?0.aura.ldpServerUrl=ws%3A%2F%2Flocalhost%3A8081&0.aura.ldpServerId=1I9Nq0000003RjdKAE&0.aura.mode=DEVPREVIEW","isHistorical":true},"from":{"location":"/lightning/n/intlAgree__Attribute_Manager?0.aura.ldpServerUrl=ws%3A%2F%2Flocalhost%3A8081&0.aura.ldpServerId=1I9Nq0000003RjdKAE&0.aura.mode=DEVPREVIEW","pageReference":{"type":"standard__app","attributes":{"appTarget":null,"pageRef":{"type":"standard__navItemPage","attributes":{"apiName":"intlAgree__Attribute_Manager"},"state":{}}}}},"subscribers":[]}
aura_devpreview.js:45803 	[LAF:Filters:applyPageRefFilters] applied. to.pageReference = undefined
aura_devpreview.js:45803 [LAF:OneAppRouter:stripOutAppTarget] {"status":"COMPLETE","disposition":"BLOCKED","dispositionValue":"to.location is equal to the current/from location.","to":{"location":"/lightning/n/intlAgree__Attribute_Manager?0.aura.ldpServerUrl=ws%3A%2F%2Flocalhost%3A8081&0.aura.ldpServerId=1I9Nq0000003RjdKAE&0.aura.mode=DEVPREVIEW","isHistorical":true},"from":{"location":"/lightning/n/intlAgree__Attribute_Manager?0.aura.ldpServerUrl=ws%3A%2F%2Flocalhost%3A8081&0.aura.ldpServerId=1I9Nq0000003RjdKAE&0.aura.mode=DEVPREVIEW","pageReference":{"type":"standard__app","attributes":{"appTarget":null,"pageRef":{"type":"standard__navItemPage","attributes":{"apiName":"intlAgree__Attribute_Manager"},"state":{}}}}},"subscribers":[]}
aura_devpreview.js:45803 [LAF:OneAppRouter:resolvePage] invoked with pageTx: {"status":"COMPLETE","disposition":"BLOCKED","dispositionValue":"to.location is equal to the current/from location.","to":{"location":"/lightning/n/intlAgree__Attribute_Manager?0.aura.ldpServerUrl=ws%3A%2F%2Flocalhost%3A8081&0.aura.ldpServerId=1I9Nq0000003RjdKAE&0.aura.mode=DEVPREVIEW","isHistorical":true},"from":{"location":"/lightning/n/intlAgree__Attribute_Manager?0.aura.ldpServerUrl=ws%3A%2F%2Flocalhost%3A8081&0.aura.ldpServerId=1I9Nq0000003RjdKAE&0.aura.mode=DEVPREVIEW","pageReference":{"type":"standard__app","attributes":{"appTarget":null,"pageRef":{"type":"standard__navItemPage","attributes":{"apiName":"intlAgree__Attribute_Manager"},"state":{}}}}},"subscribers":[]}
Uncaught (in promise) Error: A listener indicated an asynchronous response by returning true, but the message channel closed before a response was receivedUnderstand this errorAI
metricsApi.ts:20

@apettitt could you share logs from $HOME/.sf directory? There should be a file like .sf/sf-2024-08-07.log for the day you ran the command on. You can either share the entire file or just share part of it around the time you ran this command.

@abdulsattar  I've tried over several days so I'm going to just attach all of them.  Update, I did try my 3 different browsers (Chrome, Firefox and Edge).   All 3 had the same results.
sf-2024-11-19.log
sf-2024-11-20.log
sf-2024-11-13.log
sf-2024-11-14.log
sf-2024-11-15.log
sf-2024-11-18.log

Same issue. No erros in console. Get these logs after first file save:
{
	"name": "CLS",
	"value": 0.07,
	"delta": 0.07,
	"id": "v3-1732634081849-5565824098197",
	"navType": "reload",
	"wvVersion": "3.5.2"
}

{
	"timestamp":<PHONE_NUMBER>725.4,
	"sequence": 95,
	"schemaSequence": 4,
	"loggerName": "WebVitals",
	"pagePayload": {
		"schema": {
			"namespace": "sf.lex",
			"name": "PagePayload",
			"pbjsSchema": {
				"nested": {
					"sf": {
						"nested": {
							"lex": {
								"nested": {
									"PagePayload": {
										"fields": {
											"apiName": {
												"id": 2,
												"type": "string"
											},
											"entityType": {
												"id": 7,
												"type": "string"
											},
											"pageSequence": {
												"id": 1,
												"type": "uint32"
											},
											"context": {
												"id": 5,
												"type": "string"
											},
											"panel": {
												"id": 3,
												"type": "string"
											},
											"developerName": {
												"id": 8,
												"type": "string"
											},
											"url": {
												"id": 4,
												"type": "string"
											},
											"entity": {
												"id": 6,
												"type": "string"
											}
										}
									}
								}
							}
						}
					}
				}
			}
		},
		"payload": {
			"pageSequence": 1,
			"apiName": "bn2gpv1__Reservation",
			"url": "/lightning/n/bn2gpv1__Reservation?0.aura.ldpServerUrl=ws%3A%2F%2Flocalhost%3A8081&0.aura.ldpServerId=1I9Q10000001bGrKAI&0.aura.mode=DEVPREVIEW",
			"context": "one:auraContainer",
			"entityType": "bn2gpv1__Reservation"
		}
	},
	"appPayload": {
		"schema": {
			"namespace": "sf.lex",
			"name": "AppPayload",
			"pbjsSchema": {
				"nested": {
					"sf": {
						"nested": {
							"lex": {
								"nested": {
									"AppPayload": {
										"fields": {
											"jtbdPhase": {
												"id": 5,
												"type": "string"
											},
											"appName": {
												"id": 2,
												"type": "string"
											},
											"appType": {
												"id": 3,
												"type": "string"
											},
											"jtbdName": {
												"id": 4,
												"type": "string"
											},
											"jtbdCount": {
												"id": 6,
												"type": "uint32"
											},
											"appNamespace": {
												"id": 1,
												"type": "string"
											}
										}
									}
								}
							}
						}
					}
				}
			}
		},
		"payload": {
			"appNamespace": "bn2gpv1",
			"appName": "BN",
			"appType": "Standard"
		}
	},
	"loggerAppName": "one:one",
	"connectionType": "4g",
	"clientSessionId": "026f3be5898d0a4d",
	"age": 13335.5
}

Tried with locker and LWS, Chrome and Firefox.
@salesforce/cli/2.67.7 win32-x64 node-v22.11.0

Then make sure to connect with the intellect report and developer won't work until you connect the process


Laravel 5 compatibility

Incorporating these changes allows people to use Laravel Stapler with Laravel 5. I have expanded upon the documentation (as there wasn't any mentioning of actually using the correct Service Provider) and also created an alias for dev-master (2.0.x-dev) until @tabennett decides for a formal tagged release for Laravel 5. Until the merge is actually done, the dev-feature/l5-compatibility version can be pulled from our github url.
Please note that the config publishing is now done to the config_path root rather than in a stapler subdirectory. This is because Laravel 5 simply ignores config files that are located in subdirectories.
This fixes #60 and other issues regarding L5 compatibility issues.

ping @tabennett :smile:

Seems like the config issue was solved with #63.  The docs still need to be updated, though.  Especially since there are three different config files, I definitely prefer to have them in a separate directory.


Kinda rebase PR 874 to support parquet format

to: @blakemotl @ryandeivert @Ryxias
cc: @airbnb/streamalert-maintainers
related to: #874
resolves:
Background
PR #874 is to convert logs and alerts data into parquet format to improve historical searching performance against StreamAlert data based on AWS S3 and Athena services. It also removes athena partition lambda function, however, during rebasing, I found out we still need this lambda function to add new partitions to the glue catalog. So I add this lambda function back with proper changes to process data in parquet format. I will initialize a discussion about update partition issue later.
#874 is based on v2 branch which is way to old and the PR was sent a year ago in python2 and terraform v0.11 🤷♀ It is very time consuming to rebase that PR and resolve every single conflict. So I kinda copy&pasta the changes in #874 to this new PR based on release-3-1-0 branch and updated few unit test cases. 🌞
Changes

Copy&pasta most code related to save classified data in parquet format via Kinesis firehose.
Keep athena_partition_refresh lambda function and update the regex to process data in parquet format and new s3 file prefix.
Address couple bugs in original PR #874.

create module.streamalert_athena database in init stage. The database is required for alerts table.
use open_x_json_ser_de deserializer to be more resilient with log schemas that have nested JSON in kinesis firehose configure. This bug was discovered when an alert was triggered during test in staging environment.

Testing

Unit test is passed
Deployed the new code to staging AWS account and feed in osquery data. Test classifier, rules_engine, alert processor and athena_partition_refresh lambda function pipelines. Make sure new alerts and osquery data can be searched in Athena.
TODO: test the migration of existing StreamAlert infra to parquet support.

Discussion

In this PR, it adds capability to create Athena tables automatically using Athena API call. I would like to change it to use Terraform since StreamAlert release 3.0 branch already in terraform v0.12 which has capability to create tables.  My question is should I do it in this PR or in a new PR?
About athena_partition_refresh lambda function, it is a pain point for us because it consumes lots of athena DDL queries using ALTER TABLE to add new partitions. In PR #874, I believe we thought the new partitions would be loaded automatically because the data is partitioned and s3 file prefix is complaint to the format $LOG_NAME/dt=!{timestamp:yyyy-MM-dd-HH}/ in parquet format, however it is not true. We have few options to add partitions I would like to discuss and maybe improve it in the future if not now.

Keep athena_partition_refresh lambda function what I do here. But it consumes athena DDL queries and it adds complexity to StreamAlert, that requires s3 event notification, a sqs queue and a lambda function to call athena ALTER TABLE.
Use Glue Crawler to discover new partitions. But first, Crawler is expensive, second Crawler runs on schedule or on-demand. Ideally, we would like the s3 event notification to trigger Crawler to add new partitions when there is new files written to s3 bucket, however, this capability is not available.
Manually run MSCK REPAIR TABLE table_name to load all partitions including new partitions. However, it has to scan whole bucket and eventually it will be timed out when the S3 bucket grows bigger.
Similar to athena_partition_refresh lambda function, we use GLUE API call to batch add new partitions. It will resolves the concern on athena DDL queries. I haven't went to deep about this approach. Would love to pick your brain on this.

Coverage increased (+0.06%) to 96.254% when pulling 81db9171a69546bfbc4ef16dc140fce37db0b4c7 on support_parquet_format into e82ef5931818ed4bc458a85d18d47ed3fb672208 on release-3-1-0.

@Ryxias interesting point. I think it is not a bad idea. But instead of pop-in-per-schema, I am thinking provide choice for historical data format, choice between json or parquet, a.k.a backward compatible.
The reason I don't want to go pop-in-per-schema is it will make the data inconsistent for post process because some s3 prefix contains data in json format, some in parquet. What do you think?

@blakemotl @ryandeivert @Ryxias This PR is ready for 2nd round of review. 🙇♀

Let's try to get this merged and deployed soon.


Function args are unreadable with JS disabled

e.g. https://vkdoc.net/man/vkCmdPushDescriptorSetWithTemplateKHR

Fair enough if you don't care about fixing this

This is definitively something we can fix!

Fixed.


Feature/pug view engine

Resubmitting as it got closed accidentally.
I've added an option to support the pug template engine as an argument when making a new project. I am not replacing any of the other options and jade is still the default template engine if no arguments are passed.
I've also done two additional things:
•Added a line to the .gitignore to ignore all Mac OS X .DS_ files
•Added viewport and IE EDGE metadata to templates for easier previewing of sites on mobile and on Windows

Awesome! You're awesome! I have merged this into the 4.14 branch (#117)!
Just so you know, I did adjust your PR in a few ways:

I moved the test down to keep it alphabetical like all the others, instead of just randomly inserted in a random spot.
I did not include the .DS_* change, because we do not want that in our .gitignore, as that is something either platform-specific or editor-specific. You should add that to your global .gitignore to keep from needing to add this to even project you touch.
I didn't actually merge the viewport stuff, because you didn't add it to all the other templates we have. You're welcome to make a PR that adds it to all the templates.

Fantastic! Once the next release is published I'll make additional pull requests to add the viewport stuff and a few other things that may be useful.

Sounds good!


Exception on start-up

the exception seems to happen when regKey_WriteSubkey is called in the main function

Fixed


Unity Container no longer Resolving dependency to Ilog for custom Selected Controllers

Description
Since Upgrading to 5.10.3 from 5.8.13
Unity cannot resolve any webapi controllers that take an ILogger as a constructor parameter.
We configure logging as follows:
  container.AddExtension(new LoggingExtension(loggerFactory));

We have a custom IHttpControllerSelector implementation for some controllers.
using
config.Services.Replace( typeof(IHttpControllerSelector),  new OurCustomControllerSelector(config, _logger));

Unity cannot resolve any of the controller types returned by a custom controller selector.
If we bypass this and use a hard coded rout route to any of these controllers, it is able to resolve them.
As well if we add a constructor without ILogger it resolves without issue.
To Reproduce
Will take some time to extract a poc out of our code base.
Additional context
This has worked well with many previous versions of unity.

We are facing the same issue after upgrade from 5.10.2 to 5.10.3.
Unity.Microsoft.DependencyInjection was also updated from 5.10.1 to 5.10.2

I would need something I could execute to test this


feature request: show staus check indicator for current checked out branch

When working on a specific branch inside vscode, only show the Pin workflows and see their status updates for the currently checked out branch.
This is better for multi-user projects. I want to see the status checks of the work I'm doing atm, and not see what's up when ever a user pushes to a public branch.
I'll try and implement this myself if I'll have the time. thanks for a great extension!

oh my bad. I see it already works as intended

Is this handled by https://github.com/grzegorzjudas/vscode-gitbuild?

please re-open this.
There wasn't any activity since no one really answered. What should I have done without any maintainer saying anything?


BUG: Fixing of the sys library issue in Slicer 4.4

Can you remove the debugging "print"? Thanks.

Done!

Done to!


Add Crystal implementation to README

Adds encoder implementation for Crystal language.

@omahlama ping 🏓

Thanks for the contribution! 🍬


Key property on Multiselect undefined when set (@next)

In the 2.0 (next) version of vue-multiselect, the key property on the multiselect element is undefined even when set explicitly. This is breaking the functionality of :selected when the value of that refers to an array of objects, allowing doubling up of selected elements.
Setting the value manually via the browser console with $vm.key = "id" causes the element to behave as expected.
Vue Devtools screenshot:

Usage code:
       <multiselect
         :options="companies"
         :multiple="true"
         :selected="selected_companies"
         @update="updateCompanies"
         key="id"
         label="label"
       ></multiselect>

Versions:
"vue": "^2.0.0-rc.3",
"vue-multiselect": "^2.0.0-beta.2",

Vue 2.0 added special (reserved) attributes key, ref, and slot (see vue/vuejs issue 2873). Chances are the addition of the key reserved attribute is the issue here. This property may require a name change in vue-multiselect as a result.

Yeah, it must it. Will fix it ~sunday. Thanks!

Should be fixed in 2.0.0-beta.3

Confirmed working in 2.0.0-beta.3 with trackBy prop. Thank you!


Update README.md

Wanted to provide a more descriptive README, so that I can point to this repository and people can find all the related resources in one place.
Thanks so much for this code! So helpful!

Awesome! Thanks!


Creating submodules for the modules

In order to structure our curriculum we should create submodules for all modules.
Each Module should be structured like this:
Module: (markdown file in _articles)

brief description
goal of the week
submodules

Submodule: (markdown file in _articles/MODULE-NAME)

header

short summary
time needed
prerequisites
materials

content
assignments

For now we have 8 Modules which we have to prepare like that:

[ ] Welcome to OHM
[ ] Staging Your Project
[ ] Building Open Projects
[ ] Open Hardware Specifics
[ ] Roadmapping
[ ] Licenses and Standards
[ ] Running Awesome Events
[ ] Outro


Method Illuminate\Database\PostgresConnection::getDoctrineSchemaManager does not exist.

Trying to install TomatoPHP, im trying to do this commands composer require tomatophp/tomato-php it shows issues above. Is it me or maybe your packages just got a issue. Im trying to fix this using postgres and sqlite it same.

Hi, I think you need to enable the extension in your php.ini file.

fixed on last release


Float64List.setAll issue on dart2js and dartdevk

The following test works well in VM but fails on dart2js and dartdevk

import "dart:typed_data";
import "../../../Utils/expect.dart";

main() {
  var l = new Float64List.fromList(
      [0.0, 1.0, 2.0, 3.0, 4.0, 5.0, 6.0, 7.0, 8.0, 9.0]);

  l.setAll(4, l.getRange(1, 3));
  Expect.listEquals([0.0, 1.0, 2.0, 3.0, 1.0, 2.0, 6.0, 7.0, 8.0, 9.0], l);

  l.setAll(2, l.getRange(1, 4));
  Expect.listEquals([0.0, 1.0, 1.0, 2.0, 3.0, 2.0, 6.0, 7.0, 8.0, 9.0], l); // deepEquals([0, 1, 1, 2, 3, 2, 6, 7, 8, 9], [0, 1, 1, 1, 1, 2, 6, 7, 8, 9], ) fails

  l.setAll(0, l.getRange(2, 7));
  Expect.listEquals([1.0, 2.0, 3.0, 2.0, 6.0, 2.0, 6.0, 7.0, 8.0, 9.0], l);
}

Dart SDK version: 2.10.0-56.0.dev (dev) (Mon Aug 24 09:17:19 2020 -0700) on "windows_x64"

Seems JS compiled setAll does not handle copying a range into an overlapping region safely.
I'm not sure it must, we can't see directly that l.getRange is part of l, it's just some iterable.
The only operation guaranteed to recognize copying into an overlapping region of the same list is setRange.

TL;DR: I am supportive of making the implementations closer, but I thing that means changing the VM version.
The VM's _DoubleListMixin in typed_data_patch.dart does this:
  void setAll(int index, Iterable<double> iterable) {
    final end = iterable.length + index;
    setRange(index, end, iterable);
  }

DDC and dart2js fall back on ListMixin:
  void setAll(int index, Iterable<E> iterable) {
    if (iterable is List) {
      setRange(index, index + iterable.length, iterable);
    } else {
      for (E element in iterable) {
        this[index++] = element;
      }
    }
  }

The result of getRange is not a List, so only on the VM does setAll flow through to setRange.
However, I think the VM version is problematic. It contains the classic double-iteration antipattern.
Consider:
import 'dart:typed_data';

main() {
  var a = Float64List(10);
  a.setAll(0, [1.1, -2.2, 3.3].where(foo));
}

bool foo(double x) {
  print(x);
  return x >= 0;
}

VM prints the elements twice, but dart2js and DDC print them once.
I think this is a more serious problem that one one reported.
We could make the VM implementation the argument (e.g. is EfficientLengthIterable), but this test should be in agreement with the implementations of setRange, otherwise we end up copying the input:
    if (from is List<int>) {
      otherList = from;
      otherStart = skipCount;
    } else {
      otherList = from.skip(skipCount).toList(growable: false);
      otherStart = 0;
    }

Simply removing the VM implementations of setAll would cause the code to use setRange for lists only via ListMixin.setAll.


[Database Schema] Convert missing_* columns to TINYINT

The missing_* columns currently only store 6 different codes. I don’t expect them to grow beyond 32 different codes any time soon. So I think a TINYINT is appropriate for them (a 1 byte int), instead of INT (4 bytes).
The missing_* columns change would reduce our database size by 9 bytes per row. This would be a 7% savings when rows have an average size of 127 bytes (figure from a MariaDB report back in 01/21/2021).

a row currently takes ~112 bytes, if you assume "day" for time_type (INTs take up 4 bytes, VARCHAR takes length+1, and everything else is 8 bytes), which would mean those 9 bytes account for more than 8% now.
the nominal footprint of the sum of each field in row isnt the end of the story, though.  the indexes also grow along with the row count, and the missing_* columns arent included in those.  at this moment in time, epimetric_latest has ~350M rows but takes up ~88GB on disk, averaging ~254 bytes/row.  epimetric_full has even more indexes which will further diminish the contribution of the "missings".
it should too be noted that we have 4 columns in those tables (taking up 32 bytes/row total) that are essentially unused now, but are intended for future use...  the argument there was that we were willing to trade disk space in the nearer future in exchange for time spent doing table ALTER operations a little later.
all that being said, this isnt a bad idea and we should do it, but it will likely have to wait until we do another large modification to the database (like a required column addition or other operation that necessitates stopping the service and running an ALTER).  this change could be done roughly simultaneously with that.

So the indexes for epimetric_latest make up more than 50% of its storage size? Wow, TIL, thanks for that context.
Agreed, let's roll this into the next big ALTER operation.


NuGet iconUrl is deprecated

Describe the problem
According to https://docs.microsoft.com/en-us/nuget/reference/nuspec#iconurl, iconUrl is deprecated and being replaced with the in-package icon.
For back-compat purposes, we can specify both.
Describe the solution you'd like
Package the icon in the NuGet package.

Closing due to deprecation of NuGet feed


Small camelcase issue in filename.

There seems to be a small camelcase issue with a filename:
transip/src/Model/Webhost.php
I believe this should be:
transip/src/Model/WebHost.php (uppercase H)
No problems with my local development environment (Laravel Homestead), it throws errors however on my server environment.
Hope this helps, thank you for your effort.

@ChrisHagg thanks for the issue report, and my apologies for the late response. Can you provide more details about the errors you are experiencing on your server environment?
Looking at the official 'client' I believe you are correct, however I'd like to be able to verify the fix before releasing a new version.

@hiddeco It seems unable to find the correct file due to the camelcase difference:
PHP Fatal error:  Class 'TransIP\Model\WebHost' not found in [server path]/vendor/hiddeco/transip/src/Api/AbstractApi.php on line 36
In AbstractApi.php line 36:
Class 'TransIP\Model\WebHost' not found
When I change the filename as described above the problem is solved.


Remove prometheus metric_relabel_configs

We control the metrics that are emitted at the proxy.
This configuration is no longer needed.

Rebase ?


add tests for enable/disable compaction

Done with initial review. With the changes I've noted in comments, I'll be +1.
These tests are very useful, thank you.

Oh, and could you add docstrings to each of the new methods to explain the purpose of each test? Something short is fine, e.g.
    def disable_autocompaction_schema_test(self):
        """
        Test that autocompaction disabling settings in table schema are respected.
        """

merged with the comments fixed, thanks!

Great, thank you!


added authn validation for PRS service.

Connected to #948

I made a few stylistic changes.  Otherwise the PR looks good.
For information, I have in a separate branch a refactoring of the token module that pulls in the cookie manipulation code so that it doesn't need to be repeated everywhere.  I'll look into making that a separate PR.


add changelog of v1.6.0

LGTM, although I'd prefer less details in the bugfixes section (maybe move it as last thing, minor releases are mostly interesting for features, not bugfixes, which should go in bugfix releases)

Example of unneeded detail in the Bugfix section:

Fix: Let's Encrypt configuration settings (#4911)

Since Let's Encrypt is a new feature, there's no point in listing fixes to bugs in its implementation
(there may be more like that)

Finally 💯 💯

@lafriks @strk done.


Website Feedback

I didn't find any way to provide feedback from your website, so I'll create an issue here, for you, even though I know it's not really an issue for your library. :)

It would be great to have some contact info/feedback form on your website!

I would be more apt to use your site if I could import a list of shows from somewhere else. I already lists at Trackt, in my Sonarr server, and in several different apps that compete with yours.

Thanks! I found your github site from the Nuget gallery (I'm looking for a library like yours) and then found your main site and decided to try it out.

Hey @kenlefeb, thanks a lot for your feedback and sorry for the delay, GH notifications are crazy and I missed this one.
We used to have a uservoice that unfortunately is not currently available. We'll ensure we recover it and make it clear from the site. In case you would like to drop us an email you can write us as well support[at]trackseries.tv
Regarding importing lists, we have the option of importing from Trakt in preview but we couldn't enable in production due to some unexpected issues. We are currently working on the new version of our services and site and we have plans to enable importing from different sites and formats.
Thanks!


feat(hash-agg): yield chunk to downstream after apply_chunk instead of flush_data

Previously, the Extreme State require flush data before get output (to avoid complex merge-on-read). Therefore, we only flush data   and build stream chunk when receive a barrier.
By now, we have StateTable to support Write-Any-Time, so it should be safe to yield chunk when apply_chunk.
Also will clean up some stale docs.

Will yielding chunks immediately after each input chunk be beneficial to the computing? The result will tend to be eventually replaced by an updated one at the end of this epoch, unless we ensure that inputs with the same group key can be gathered well.

Will yielding chunks immediately after each input chunk be beneficial to the computing? The result will tend to be eventually replaced by an updated one at the end of this epoch, unless we ensure that inputs with the same group key can be gathered well.

Yes me and @st1page also have the same concern. So this refactor may need to reconsider and postpone until we figure out the bottleneck.


Build fails on lower JDK versions

I received the following error:
Error:
  FAILURE: Build failed with an exception.

  * Where:
  Build file '/home/runner/work/Bountiful/Bountiful/build.gradle.kts' line: 53

  * What went wrong:
  Failed to apply plugin 'dev.architectury.loom'.
  > You are using an outdated version of Java (11). Java 17 or higher is required.
    The JAVA_HOME environment variable is currently set to (/usr/lib/jvm/temurin-11-jdk-amd64).

  * Try:
  > Run with --stacktrace option to get the stack trace.
  > Run with --info or --debug option to get more log output.
  > Run with --scan to get full insights.

  * Get more help at https://help.gradle.org/

  BUILD FAILED in 2m 3s

My buildscript generally uses JDK 17 or up. Is there a way to specify this when the action tries to run Gradle?

@ejektaflex this action does not specify the java version, and directly executes gradlew or gradle commands for you, but requires a working dev environment.
The action will not install java on the chosen runner. You can use the actions/setup-java action for this purpose: https://github.com/actions/setup-java


Enable developers to run pod from terminal directly and skip pod install if possible.

Currently.You have to run cocoapods from the flutter side, in which it will pass environment variables containing the location of the flutter framework. However, we want a standard workflow, i.e., you can run pod install from flutter, you can also run it from the terminal.

@xster can you review this and/or help bring this over the line for customer:gold.

Thanks for your contribution. I started reviewing this and then thought about what could be the root cause since repeated runs on pod install should have more or less no cost. And it seems like your main concern is repeated builds on the same machine.
We may be able to address the underlaying issue in #13467. Based on https://github.com/CocoaPods/CocoaPods/issues/4128, it's possible it's slow for you guys in China only because of stats collection in CocoaPods.

@xster
In our team, the problems brought by cocoapods(pod install) are as follows:
1.It takes time to complete a full repo-update. Our repos not only contain the official github one, but also  our private one. Not to mention the slow/unstable network connection to github.
2.As cocoapods re-generate the whole Pods.project each time for good, when our project contains many dependencies, the generating pod project process could cost at least 1 min every time.
This PR mainly focuses in two sides:
1.Enable to run pod install directly.
Developers are used to run their pod install from terminal. However current implementation in flutter_tools makes it hard. Also, when debugging a cocoapods problem or do some kind of customization cocoapod process, it will simplify the process and save efforts when following the standard way.
2.Skip the pod install if possible
There are two reasons.
a.As mentioned above, the cocoapods process is time costly.In our intranet(with good network condition and good access to github), even after a lot optimization, the process costs 1+ mins each time when the dependency is complicated(especially in a company where there are plenty of middlewares).
b.We specify our dependency for certain commit/tag for safety and only run the pod install process when we clearly know we need to update some of them.

@xster
Currently, I'm using 1.3.1.
1.In 1.3.1, the repo-update part you said is right.
Note: as of CocoaPods 1.0, pod repo updatedoes not happen onpod install by default.  referring to https://github.com/CocoaPods/CocoaPods/issues/5260.
The prior idea I hold that pod install will trigger repo update is a designed feature. It comes from our company-customized cocoapod plugin. It works in a non-standard way where you can only skip the pod repo update process when a --no-repo-update option is given.
2.I dig more and found that, even the Podfile.lock matches Pods/Manifest.lock, cocoapods will also modify Pods/Manifest.lock and Pods/Pods.xcodeproj/project.pbxproj. In my opinion,this is mainly  because of the consideration of the existence of Development Pods.There was a bad experience when I seperated one project into many development pods while they have to be changed frequently. Add/Delete a source or resource will require the pod install process to get right project file.As file references,libraries,target_dependencies,etc(Refer to gems/cocoapods-1.3.1/lib/cocoapods/installer/xcode/pods_project_generator.rb for more) have been modified in this process, the whole project will be recompiled.

@xster
Thank you for your comments on the coding style and documentation. I've a question. Some logic or parameter/method naming may be hard to analyze. However, as to  the format issue(like spaces after ? : in a ternary ), is there any tool which can help us locate and fix it? I tried flutter analyze but it seems not enough.

Unfortunately we're still in the process of potentially tweaking dartfmt a bit to let the process be automated in the future. For the time being, it's manual applications of the clauses listed at https://github.com/flutter/flutter/wiki/Style-guide-for-Flutter-repo.

@xster
Sorry to bother you. As the company requests to remove sensitive informations including my email before signing a CLA, I rebase the commit history. Please let me know if there is something wrong.

No worries.
Seems like the new file came from https://github.com/flutter/flutter/pull/13541.
Looks like you git mergeed up to the main branch from 10 days ago instead of git rebaseing from head. Might be cleaner if you reset your local branch to c93fef3, cherry picked in your new 0d32875 commit and then rebase on upstream/master (don't forget to fetch upstream first).
At that point, you can fix the packages/flutter_tools/lib/src/commands/inject_plugins.dart file.

Please remember to reply saying I signed it! here when you go through the CLA signing (with any email). For our bots.
Let me know when you're ready and I'll merge it for you.

Thanks a lot. I'm on the way for a CLA. After that, I will let you know.
2017-12-20 14:27 GMT+08:00 xster<EMAIL_ADDRESS>
Please remember to reply saying I signed it! here when you go through the
CLA signing (with any email). For our bots.
Let me know when you're ready and I'll merge it for you.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/flutter/flutter/pull/13374#issuecomment-352976685,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAx6u-OGWCjyIAlPkM3hjt16yq-ZJ_nXks5tCKjigaJpZM4Q3Rni
.

I signed it!

Not sure if the cla bot is supposed to re-run. The contributor's employer is in the approved corporate list and the pull request sender and git commiter is in his employer's Authorized Contributors Group. I assume it's ok to merge.
Thanks for your contribution @kangwang1988

Sorry, have to revert. We'll need to keep the env variable in for the time being for previously created projects.
Side note: we should probably do #12573 soon.


1.0.0 release?

1.0.0-beta.1 has been live for almost two months, has it proven to be working enough to release 1.0.0?  Hex won't let me use a beta release as a dependency for other hex packages unless I also make them beta releases.  Just itching to upgrade :)

has it proven to be working enough to release 1.0.0

This is the core question here. There was not a single bug/question reported for 1.0.0-beta1. Either the code & docs are perfect or nobody really tried to use it. I am using it myself, in production, but that's not enough.
I understand where are you standing very clearly, I would've wanted the same at your position. Unfortunately I have very limited time recently. If I release 1.0.0 with major bug it will need to be fixed fast, that's why I need to minimize the chances by publishing a beta first.
Long story short, you can make it happen quicker if you try the beta and confirm everything is fine. Or even better jump into migration guide, point to troublesome places and report (even tiny) bugs 💪
Thank you for starting the discussion, I will leave this issue open.

@ontofractal Have you used 1.0.0-beta.1 or 0.x?

I will give it a shot when I can and report back!  My package isn't yet used in production, but will be within the next month.  If I can use a beta release of it using tesla 1.0 beta, I will let you know.

@teamon I've used 1.0.0-beta.1
Nothing fancy, several API clients with custom middleware so my usage of Tesla features is not extensive. Thanks for building the module, btw!

I've just released v1.0.0, enjoy! :)

Great news, thanks, @teamon!


Reduce expected trashbin failures.

This PR fixes stuff in the trash-bin code and thus reduces the expected failures for the trash-bin tests.

🙏 thank you for digging into this!


Add create_zip script

Pull Request Template
script name - create_zip
Brief about script
Recursively adds a folder to a zip file
Issue no.(must) - #351
Self Check(Tick After Making pull Request)

[✔] This issue was assigned to me.
[✔] One Change in one Pull Request
[✔] My file is in proper folder (Name of folder should be in lowercase with no space in between)
[✔] I am following clean code and Documentation and my code is well linted with flake8.
[✔] I have added README.md and requirements.txt with my script

If issue was not assigned to you Please don't make a PR. It will marked as invalid.

@all-contributors add @pranxv  for code


[TranslateBundle] issue with non-standard translation files sulu:export:translate

Q
A

Bug?
yes

New Feature?
no

Sulu Version
Sulu/1.6.19

Browser Version
Google Chrome (not actual)

Actual Behavior
How does Sulu behave at the moment?
When I try generate Dutch - Belgium file sulu.nl_be.json I receive fallback into sulu.nl.json.
Expected Behavior
Should get backend.nl_be.xlf translations and generate sulu.nl_be.json
Steps to Reproduce

Create file for example BusinessBundle/Resources/translations/sulu/backend.nl_be.xlf
Try to generate php app/console sulu:translate:export --path public/admin/translations
Get sulu.nl_be.json with fallback into sulu.nl.json content. According debugging Symfony do not found nl_be locale

Possible Solutions
public function execute()
    {
        $messages = [];

        $locale = preg_replace_callback('!b[a-z]!', function($matches) {
            return strtoupper($matches[0]);
        }, $this->getLocale());

        /** @var MessageCatalogueInterface $catalogue */
        $catalogue = $this->translator->getCatalogue($locale);

        if ($this->backend) {
            $messages = array_merge($messages, $this->getMessagesForDomain($catalogue, self::BACKEND_DOMAIN));
        }
        if ($this->frontend) {
            $messages = array_merge($messages, $this->getMessagesForDomain($catalogue, self::FRONTEND_DOMAIN));
        }

        $this->writeMessagesFile($messages);
    }

File path: vendor/sulu/sulu/src/Sulu/Bundle/TranslateBundle/Translate/Export.php

I checked this issue. This is not a bug. False alarm

@StaffNowa I guess I can close this issue then. Comment if I am wrong :-) Maybe it would still be useful if you write what the problem was, in case somebody stumbles upon this using Google or something :-)

@StaffNowa I guess I can close this issue then. Comment if I am wrong :-) Maybe it would still be useful if you write what the problem was, in case somebody stumbles upon this using Google or something :-)

I have strange issue
$request->getLocale() => return nl_be
$translator->getLocale() => return nl_BE
Why I have in controllers issue for bad translation file finding.

@StaffNowa That sounds to me like this is returned like that from Symfony already... I don't think that we touch here anything.

Maybe :) In some cases I need to check locale listener and return good locale name lowercase like we have with Sulu backend.nl_be.xlf file :)

@StaffNowa Have you tested if what I said is true? Would be helpful to know :slightly_smiling_face:


Error handling in shortest_path calculation for multiple origins/destinations

Problem description

What did you do?

Submitted a request to OSMnx' shortest path with a single node as origin but multiple destinations (or viceversa).

What did you expect to happen?

Receive an informative error message which points out the mismatch between types of origin(s) and destination(s).

And what actually happened instead?

Received TypeError: object of type 'int' has no len()

Environment information

What operating system are you using?

Windows 10 Enterprise, v1909 build 18363.1556

What Python version are you using?

Python 3.8.8

What OSMnx version are you using?

OSMnx 1.1.1

Provide a complete minimal reproducible example
import osmnx as ox
# graph from sample address
G = ox.graph_from_address('Via Montenotte, 18, Savona, Italy')
# single node as origin (without wrapping into a list with a single element)
orig = list(G.nodes)[0]
# multiple destinations (list with two nodes)
dest = list(G.nodes)[1:3]
# shortest path with mismatch between orig and dest types
# (dest is iterable, being a list, while orig isn't)
ox.shortest_path(G, orig, dest)

A possible improvement for handling such situations could consist in replace line from 524 to 531 in osmnx/distance.py with the following
if not (hasattr(orig, "__iter__") or hasattr(dest, "__iter__")):
    # if neither orig nor dest is iterable, just return the shortest path
    return _single_shortest_path(G, orig, dest, weight)
else:
    # if both orig and dest are iterables
    if (hasattr(orig, "__iter__") and hasattr(dest, "__iter__")):
        # if orig/dest are iterable, ensure they have same lengths <- before, this was taken for granted
        if len(orig) != len(dest):  # pragma: no cover
            raise ValueError("orig and dest must contain same number of elements")
    # if exactly one between orig and dest isn't iterable
    else:
        raise ValueError("orig and dest must be both iterables")

which provides a more informative error message.

I suppose the fix could be either 1) improving the documentation to explicitly state something like "orig and dest must either both be iterable or neither must be iterable" or 2) making the error message clearer, as you state. It looks like the else-if in your code snippet could be simplified with an elif.

I would prefer something like solution 2), but I get we're talking about details.
Regarding implementation, you're right - I was trying to modify current version as little as possible. Something like this should do the job
def shortest_path(G, orig, dest, weight="length", cpus=1):
    """
    ...
    """
    # if neither orig nor dest is iterable, just return the shortest path
    if not (hasattr(orig, "__iter__") or hasattr(dest, "__iter__")):
        return _single_shortest_path(G, orig, dest, weight)
    # if both orig and dest are iterables
    elif (hasattr(orig, "__iter__") and hasattr(dest, "__iter__")):
        # ensure they have same lengths
        if len(orig) != len(dest):  # pragma: no cover
            raise ValueError("orig and dest must contain same number of elements")

        if cpus is None:
            cpus = mp.cpu_count()
        utils.log(f"Solving {len(orig)} paths with {cpus} CPUs...")

        if cpus == 1:
            # if single-threading, calculate each shortest path one at a time
            paths = [_single_shortest_path(G, o, d, weight) for o, d in zip(orig, dest)]
        else:
            # if multi-threading, calculate shortest paths in parallel
            args = ((G, o, d, weight) for o, d in zip(orig, dest))
            pool = mp.Pool(cpus)
            sma = pool.starmap_async(_single_shortest_path, args)
            paths = sma.get()
            pool.close()
            pool.join()

        return paths
    # if exactly one between orig and dest isn't iterable
    else:
        raise ValueError("orig and dest must either both be iterable or neither must be iterable")

Would you like to contribute this as a PR?

Hi @gboeing, sorry for the late response, I've had tough weeks.
Anyway, glad to see that you merged the suggestion in the code base. Thanks!


Fix to remove selected text using backward key input

Hi, @danielsaidi thanks for keyboardKit. I saw the following bug.
I selected part of the text and pressed the backward button, but it was not removed
Fix a bug where the selected text was not removed through the backward button

Hi!
Thanks for the PR! However, when I try deleting selected text with the demo keyboard, it works (see the video).
Can you record a video of a situation when it doesn't work, so that I can investigate as well?
https://user-images.githubusercontent.com/429927/126507079-b308b1c3-30ee-4bc9-bb2e-2415a6386c4f.MP4

Hi again,
I looked at this some more, and since the native deleteBackward() works for selected text, perhaps this should solve this problem without paring the selected text and manipulating the input position?
func deleteBackward(_ range: DeleteBackwardRange) {
    guard let text = deleteBackwardText(for: range) else { return deleteBackward() }
    deleteBackward(times: text.count)
}

Hi again @junyng
I have confirmed that return deleteBackward() works, so I will go ahead with that approach since it's simpler, and release it as a patch update 👍
I will mention you in the release notes, since I wouldn't have found this without you. Also, I'd love to merge a file change PR to get you as a contributor on the project page.

Hi again @junyng
I have confirmed that return deleteBackward() works, so I will go ahead with that approach since it's simpler, and release it as a patch update 👍
I will mention you in the release notes, since I wouldn't have found this without you. Also, I'd love to merge a file change PR to get you as a contributor on the project page.

I didn't understand backward() works exactly, but I figured it out.
Thanks for the explanation and glad to contribute KeyboardKit

Happy to have you as a contributor! :)
Just create a new PR with something small, and I'll merge it to get you as a contributor on the project page  👍


[Dashboard] Update Help Center Entries

As part of the new dashboard, we have to update any existing materials in the Help and Developer centers:

[ ] Update existing materials:

[ ] https://carto.com/help/your-account/delete-account/
[ ] https://carto.com/help/building-maps/getting-started-with-carto-builder/#sign-up-for-a-carto-account
[ ] https://carto.com/developers/fundamentals/authorization/
[ ] https://carto.com/help/getting-started/dashboard/
[ ] https://carto.com/help/glossary/#mapview
[ ] ?

[ ] Create new materials (if need be):

Add dashboard definition to the glossary https://carto.com/help/glossary/
?.

cc: @ramiroaznar

There are some articles/questions that refer to the Dashboard and still haven't been migrated to the new Help and Developer Center in https://github.com/CartoDB/help/issues/130.

@alonsogarciapablo these old materials are already merged.

I have created a subcategory for the dashboard:

I will add to your initial comment, the articles that refer to the Dashboard.

@alonsogarciapablo I have added a couple of items to the list to be updated/added

Thank you @ramiroaznar. We'll be working on this this week 💪

In fact, we should review these results: https://carto.com/help/search/?query=dashboard

In fact, we should review these results: https://carto.com/help/search/?query=dashboard

I'm reviewing all these entries ☝. Something I noticed is that we use the word "Dashboard" to refer to different concepts. For example:

It seems we're using Dashboard here to describe a Builder "Map". This might be confusing for our users. Should we change these @CartoDB/hc-managers?

Here's an example: https://carto.com/help/building-maps/customize/#introduction-

CARTO Builder dashboards are a holistic experience, so their elements should be balanced visually.

Should we change these @CartoDB/hc-managers?

Absolutely. Either by removing it (Customize Widget Colors) or by changing it to Builder (Customize Widget Colors in Builder). Or if you have better ideas, just share them here. 😃

Yes, you are right. We used to call dashboards to Builder maps. I am ok if you want to replace those terms.

They are actually called Maps on the UI so we should unify. I've compiled a list of entries that need to be updated here. There are quite a few.

I can't access the spreadsheet @alonsogarciapablo but let me know if you need help from the Support Team on this task.

Sorry @jsanz, the file is private. You should have access now.


Javascript SyntaxError when setting up server with ipython=True

When I run these in ipython notebook
host = 'https://powerful-gorge-4659.herokuapp.com/'
auth = ('', '')
lgn = Lightning(ipython=True, host=host, auth=auth)

I got the warning,
Javascript error adding output!
SyntaxError: Unexpected token <
See your browser Javascript console for more details.

The server continues to work normally though.
If ipython=False, then there is no warning.

@mistycheney it looks like the "deploy to heroku" button was pointing to an old release (assuming you deployed via that?)
Could you redeploy using this link https://heroku.com/deploy?template=https://github.com/lightning-viz/lightning/tree/v0.1.13 and see if the problem goes away?

Yes, the warning disappears. Thanks.
On Tue, Jun 30, 2015 at 6:30 PM, Matthew Conlen<EMAIL_ADDRESS>wrote:

@mistycheney https://github.com/mistycheney it looks like the "deploy
to heroku" button was pointing to an old release (assuming you deployed via
that?)
Could you redeploy using this link
https://heroku.com/deploy?template=https://github.com/lightning-viz/lightning/tree/v0.1.13
and see if the problem goes away?
—
Reply to this email directly or view it on GitHub
https://github.com/lightning-viz/lightning/issues/33#issuecomment-117386796
.

Cool :v: thanks!


[Bitpay Client] Get Rates using Request to live site

When calling the rates building the client and request on your own, I found 2 issues here in production site.
1- Given I want to know the USD rate, the sendRequest hits this endpoint: https://bitpay.com/rates/USD/ which makes the server redirect you to https://bitpay.com/rates/USD, if your CurlAdapter is not setup with this option: CURLOPT_FOLLOWLOCATION as true, then, the request fails.
2- When calling https://bitpay.com/rates/USD , the response is not correctly build, since the rate is cached, this method https://github.com/bitpay/php-bitpay-client/blob/master/src/Bitpay/Client/Response.php#L56 is not matching the headers correctly, therefore the body is returned with headers and content, making it hard to read the json content.

on item 1:

if your CurlAdapter is not setup with this option: CURLOPT_FOLLOWLOCATION as true, then, the request fails.

You can send in CURLOPT_FOLLOWLOCATION in the constructor to CurlAdapter. The internals of this class shouldn't assume the user wants to follow redirects.
on item 2:
Would you provide some sample code that exposes the issue? thanks!

@kleetus,
My mistake, found the issue, I was using www.bitpay.com and not bitpay.com as the host url.


hash: allow only strings, and use a deterministic algorithm

Previously, hash would allow any operand defined as hashable
by the spec, and would use (for strings at least) a fast
hardware implementation that varies across process restarts.

Test.

@laurentlb -- this change brings the Go and Java impls closer, and fixes a bug in which the set of targets in a Bazel package would vary from one run to the next.

A question: should Starlark in Go attempt to generate the exact same hashes as Starlark-in-Java, which hashes the sequence of UTF-16 codes using a polynomial accumulator? Transcoding valid UTF-8 to 16 is not hard, but it does seem to be indulging bad habits of writing code that depends on hashing for its observed results. On the other hand, that is exactly what some users do.


I am new and I want to update SIFT

I am new to linux and I want to update SIFT to last version, I used the following command but did not work.
Please can you help me...
wget –quiet -O -https://raw.github.com/sans-dfir/sift-bootstrsp/master/bootstrsp.sh | sudo bash -s — -i -s -y
Thank you

Please see the updated instructions, https://github.com/sans-dfir/sift-cli


Test Front50 use of Google Cloud Storage

@lwander
This currently tests only Application objects. The critical integration is the same for all the different objects, so I think this is sufficient as a smoke test for the time being.

LGTM


Constrained method calls use wrong generic context

Below code will fail at runtime because RyuJIT generates code where in DoFrob we pass the interface type as generic context instead of Foo and things go bad in Frob.
Need to find the place where CoreCLR makes it do the right thing. Found this by accident because it showed up as a diff in my ILScanner work.
using System;
using System.Runtime.CompilerServices;

internal class Program
{
    class Atom { }

    interface IFoo<T>
    {
        object Frob(object o);
    }

    struct Foo<T> : IFoo<T> where T: class
    {
        public int FrobbedValue;

        public object Frob(object o)
        {
            FrobbedValue = 12345;
            return o as T;
        }
    }

    [MethodImpl(MethodImplOptions.NoInlining)]
    static object DoFrob<T, U>(ref T t, object o) where T : IFoo<U>
    {
        // Perform a constrained interface call from shared code.
        // This should have been resolved to a direct call at compile time.
        return t.Frob(o);
    }

    private static int Main(string[] args)
    {
        var foo = new Foo<Atom>();
        object result = DoFrob<Foo<Atom>, Atom>(ref foo, "Not atom");

        // If the FrobbedValue doesn't change when we frob, we must have done box+interface call.
        if (foo.FrobbedValue != 12345)
            throw new Exception();

        // We passed a string, not an Atom
        if (result != null)
            throw new Exception();

        var atom = new Atom();
        result = DoFrob<Foo<Atom>, Atom>(ref foo, atom);
        if (result != atom)
            throw new Exception();

        return 100;
    }
}

So I'm playing with a slightly simplified repro:
    class TestConstrainedMethodCalls
    {
        interface IFoo<T>
        {
            void Frob(int x);
        }

        struct Foo<T> : IFoo<T>
        {
            public int FrobbedValue;

            public void Frob(int x)
            {
                FrobbedValue = 12345;
            }
        }

        [MethodImpl(MethodImplOptions.NoInlining)]
        static void DoFrob<T, U>(ref T t) where T : IFoo<U>
        {
            // Perform a constrained interface call from shared code.
            // This should have been resolved to a direct call at compile time.
            t.Frob(1234);
        }

        public static void Main()
        {
            var foo = new Foo<object>();
            DoFrob<Foo<object>, object>(ref foo);

            // If the FrobbedValue doesn't change when we frob, we must have done box+interface call.
            if (foo.FrobbedValue != 12345)
                throw new Exception();
        }
    }

CoreCLR seems to be generating a runtime lookup for an instantiating stub in DoFrob and calling that:
       55                   push     rbp
       4883EC40             sub      rsp, 64
       488D6C2440           lea      rbp, [rsp+40H]
       48894DF8             mov      qword ptr [rbp-08H], rcx
       48894D10             mov      qword ptr [rbp+10H], rcx
       48895518             mov      bword ptr [rbp+18H], rdx

G_M52391_IG02:
       833D3347EFFF00       cmp      dword ptr [(reloc)], 0
       7405                 je       SHORT G_M52391_IG03
       E8ACEAFE5E           call     CORINFO_HELP_DBG_IS_JUST_MY_CODE

G_M52391_IG03:
       90                   nop
       488B4D10             mov      rcx, qword ptr [rbp+10H]
       488B4938             mov      rcx, qword ptr [rcx+56]
       488B4910             mov      rcx, qword ptr [rcx+16]
       48894DF0             mov      qword ptr [rbp-10H], rcx
       488B4DF0             mov      rcx, qword ptr [rbp-10H]
       4885C9               test     rcx, rcx
       7517                 jne      SHORT G_M52391_IG04
       488B4D10             mov      rcx, qword ptr [rbp+10H]
       48BAE8CFC538FF7F0000 mov      rdx, 0x7FFF38C5CFE8
       E84FE8FF5E           call     CORINFO_HELP_RUNTIMEHANDLE_METHOD
       488945F0             mov      qword ptr [rbp-10H], rax

G_M52391_IG04:
       488B4DF0             mov      rcx, qword ptr [rbp-10H]
       48894DE8             mov      qword ptr [rbp-18H], rcx
       488B4D18             mov      rcx, bword ptr [rbp+18H]
       BAD2040000           mov      edx, 0x4D2
       488B45E8             mov      rax, qword ptr [rbp-18H]
       FFD0                 call     rax
       90                   nop
       90                   nop

G_M52391_IG05:
       488D6500             lea      rsp, [rbp]
       5D                   pop      rbp
       C3                   ret

This is mildly annoying because it seems like we'll need yet another JitInterface change. What we need is a runtime lookup for the owning type after constraint resolution and pass that as the generic context to a canonical (constantly looked up) method.

When resolving this bug, look for the issue number in the sources. We have some workarounds in place.

This is mildly annoying because it seems like we'll need yet another JitInterface change. What we need is a runtime lookup for the owning type after constraint resolution and pass that as the generic context to a canonical (constantly looked up) method.

Maybe we can get away with just embedding a generic handle of the constraint itself. I can't find a scenario where that wouldn't work.

Let's keep this open.
My RyuJIT fix didn't actually fix this for the case of generic virtual method:
class TestConstrainedMethodCalls
{
    class Atom1 { }
    class Atom2 { }

    interface IFoo<T>
    {
        bool Frob(object o);
    }

    struct Foo<T> : IFoo<T>
    {
        public int FrobbedValue;

        public bool Frob(object o)
        {
            FrobbedValue = 12345;
            return o is T[,,];
        }
    }

    interface IBar<T>
    {
        bool Frob<U>(object o);
    }

    struct Bar<T> : IBar<T>
    {
        public int FrobbedValue;

        public bool Frob<U>(object o)
        {
            FrobbedValue = 5678;
            return o is T[,,,];
        }
    }

    [MethodImpl(MethodImplOptions.NoInlining)]
    static bool DoFrob1<T, U>(ref T t, object o) where T : IFoo<U>
    {
        // Perform a constrained interface call from shared code.
        // This should have been resolved to a direct call at compile time.
        return t.Frob(o);
    }

    public static void Main()
    {
        {
            var foo1 = new Foo<Atom1>();
            bool result = DoFrob1<Foo<Atom1>, Atom1>(ref foo1, new Atom1[0, 0, 0]);

            // If the FrobbedValue doesn't change when we frob, we must have done box+interface call.
            if (foo1.FrobbedValue != 12345)
                throw new Exception();

            // Also check we passed the right generic context to Foo.Frob
            if (!result)
                throw new Exception();

            // Also check dependency analysis:
            // EEType for Atom2[,,] that we'll check for was never allocated.
            var foo2 = new Foo<Atom2>();
            if (DoFrob1<Foo<Atom2>, Atom2>(ref foo2, new object()))
                throw new Exception();
        }

        {
            var bar1 = new Bar<Atom1>();
            bool result = DoFrob2<Bar<Atom1>, Atom1>(ref bar1, new Atom1[0, 0, 0, 0]);

            // If the FrobbedValue doesn't change when we frob, we must have done box+interface call.
            if (bar1.FrobbedValue != 5678)
                throw new Exception();

            // Also check we passed the right generic context to Foo.Frob
            if (!result)
                throw new Exception();

            // Also check dependency analysis:
            // EEType for Atom2[,,,] that we'll check for was never allocated.
            var bar2 = new Bar<Atom2>();
            if (DoFrob2<Bar<Atom2>, Atom2>(ref bar2, new object()))
                throw new Exception();
        }
    }
}

We need a new JitInterface method after all.

We also need to make sure dependencies are right. This needs to pass:
using System;
using System.Runtime.CompilerServices;

internal class Program
{
    class Atom { }

    interface IFrobber
    {
        bool DoFrob<T>(object o);
    }

    struct Frobber : IFrobber
    {
        public bool HasFrobbedWithT;

        public bool DoFrob<T>(object o)
        {
            HasFrobbedWithT = o is T;
            return o is T[,,];
        }
    }

    [MethodImpl(MethodImplOptions.NoInlining)]
    static bool CallDoFrob<T, U>(ref T x, object o) where T : IFrobber
    {
        return x.DoFrob<U>(o);
    }

    private static void Main(string[] args)
    {
        Frobber f = new Frobber();
        bool result = CallDoFrob<Frobber, Atom>(ref f, new Atom());
        if (result)
            throw new Exception();
        if (!f.HasFrobbedWithT)
            throw new Exception();
    }
}

Also look at the reference to this issue in the DynamicGenerics\GenericVirtualMethods test.


PR to support more .Net types, get rid of a few rough edges

This PR makes it easier to deserialise, as it makes some previously internal lenses public.

Oh there's some cool stuff in here, awesome! I'll have a proper look through and maybe a quick shuffle with things like operators and then merge (most of it at least, need to consider one or two bits first like decimal serialization...)
Great stuff!


Manage TLS Certificates in a Cluster: Fix typo in template for glossary reference

Link to glossary term "ConfigMap" didn't render correctly due to
a missing "{" at the beginning of template. This PR fixes it.

/lgtm
/approve


UI Nodes with a CalculatedSize shouldn't have child nodes or both Image and Text components

Objective
If a parent UI node with a MeasureFunc has a child node, the parent's size won't be determined by the MeasureFunc.
Ui Nodes with CalculatedSize. Image, and Text components won't be sized correctly.
It's hard to understand/explain, sometimes causes no problems, and sometimes causes unpredictable and confusing bugs.
#7894 is an example of where this has caused a problem.
Solution
Add some doc comments telling users not to do any of these things.
Panic if any of them is done.
Changelog

Added a system detect_calculated_size_conflicts that panics if a Ui Node with a CalculatedSize component has a child UI node or both Image and Text components

I feel like UI nodes ought to have some kind of "node type" tag. And only read components that are relative the specified node type. For example images would ignore text, text would ignore images, flexbox nodes would ignore both text and images, etc.

Yes, ideally that would be implemented with entity relations I think, where Text and UiImage would be mutually exclusive components and it would be impossible to construct an entity containing both of them.

Closed as not that important and there will be better ways to implement it soon.


🧑💻: Responsive in log in page

Title
I want to do the responsiveness in the log in.
Enhancement Aim
To remove the unresponsive.
Changes
I want to change the code of this page.
Screenshots 📷

Guidelines

[X] I have read the guidelines
[X] I have the link to my latest merged PR

Full Name
Souvik Pramanik
Participant Role
GSSOC

Can you assign me @vishanurag


Generate integration test certificates on demand

Description
Remove the checked in integration test certificates and instead generate them on demand.
This resolves some spurious security alerts caused by having private keys checked in to git.
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Codecov Report

Merging #133 (7a37558) into main (833c177) will decrease coverage by 6.13%.
The diff coverage is n/a.

:exclamation: Current head 7a37558 differs from pull request most recent head 4b2dbc2. Consider uploading reports for the commit 4b2dbc2 to get more accurate results

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@            Coverage Diff             @@
##             main     #133      +/-   ##
==========================================
- Coverage   65.08%   58.96%   -6.13%
==========================================
  Files         312      253      -59
  Lines       51702    46281    -5421
==========================================
- Hits        33652    27290    -6362
- Misses      18050    18991     +941

Flag
Coverage Δ

integration
58.96% <ø> (-6.13%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.
see 112 files with indirect coverage changes
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


Add cl_khr_kernel_clock tests

Adds cl_khr_kernel_clock test.
Also fixes failure in the compiler defines for extension compiler subtest when cl_khr_kernel_clock is supported.

Reviewing

Merging as discussed in the May 21st teleconference.


RootUtil.isDeviceRooted() is giving is rooted result for a Non-rooted device and simulator

the app keep saying fingerprint does not work on rooted devices although my devices is not rooted,
i also test on android simulator, it is also giving the same result, i believe android simulator should be classified as non-rooted
after i check, i found that your RootUtil.isDeviceRooted() is giving is rooted result for a non-rooted device, maybe some checking need to be validated if it is right

The Android Emulator is rooted, that check is correct.
Yes, RootUtil give false positive for some real device as it check for "super user" and some phones include the su binary while not rooted.

is there any way to disable validation?

RootUtil give false positive for some real device as it check for "super user" and some phones include the su binary while not rooted.
Any update on this?
@hiteshsahu @joaodematejr

RootUtil give false positive for some real device as it check for "super user" and some phones include the su binary while not rooted. Any update on this? @hiteshsahu @joaodematejr

I stopped using that library for that reason.

Which library is working as expected? @joaodematejr


'Method not found: 'Void Amazon.S3.AmazonS3Client During Deployment

I'm receiving this error randomly during deployment.  It takes sometimes 5-10 tries for the project to begin uploading and skip the error.
AWS Toolkit Version <IP_ADDRESS>
Microsoft Visual Studio Community 2019
Version 16.9.4
.net framework 4.8.04084
..deployment package created at X:\sites\IonCommerce\IonCommerce\obj\Release\Package\IonCommerce.zip
...starting redeployment to AWS Elastic Beanstalk environment 'IonWeb-Live'
Attempt to create deployment upload bucket caught Exception, Message 'Method not found: 'Void Amazon.S3.AmazonS3Client..ctor(Amazon.Runtime.AWSCredentials, Amazon.RegionEndpoint)'.'
Publish to AWS Elastic Beanstalk failed with exception: Detected error in deployment bucket preparation; abandoning deployment
Error publishing application: Unhandled error during deployment
Publish to AWS Elastic Beanstalk environment 'IonWeb-Live' did not complete successfully

What type of credentials are you using with the toolkit, are they short lived like MFA or SSO, or are they a standard access key + secret key pairing?
Can you please share some more details about your deployment. Are you using the new "Publish to AWS" experience, or the older "Publish to AWS Elastic Beanstalk" experience? What framework is the project you are trying to deploy targeting? Are you publishing to a Windows or Linux image (if your project is not .NET Framework based)?

If you can share your Toolkit log from %localappdata%\AWSToolkit\logs\visualstudio, we would be interested in seeing if that reveals any causes.
Have you got any other AWS related extensions installed into Visual Studio? There's a chance that the Toolkit is not using the AWS SDK dlls that were bundled with the Toolkit.  There are some steps that can help diagnose this (or rule this possibility out) written in https://github.com/aws/aws-toolkit-visual-studio/issues/138#issuecomment-756930669 . Would you be able to to try that and share what DLLs are loaded and from where?

I haven't been able to find the reason it begins to work, I usually keep trying the publish and it will eventually work.  I have previously restarted VS, restarted PC, and still had the issue.  I have noticed on occasion something appears to reset in the publishing wizard because although my credentials are still there my previous selected settings are reset, most notably the project build configuration is set back to debug rather release.  When this is reset is generally works the first try without the error.
log.txt

Hi @benmorris44 , I have made a change that may help the Toolkit use the bundled AWS SDK dll files. If you're willing to try an experimental build, I would like to know if this addresses your situation. Please try installing the attached Toolkit for Visual Studio 2019, perform a few deployments, and share your results.
AWSToolkitPackage.zip
Version: <IP_ADDRESS>10
Sha384 hash: D0246A4CC4A22EA51CA599D5C3F28A92CD02E73DB9DCF352EA3F74AF19A0BA97F80B2A3AB68C15780B444C412E60169E

I will give this a try it may take a few days or a week before I can say for certain it fixed it.  I will report back though

I've done 4-5 actual deploys and so far no issues.  Will continue to use and report back.

Thanks for the update @benmorris44 - If you're able to share that it is working for you after a couple more days of use, I'll commit that fix.

Just wanted to follow up again, I have done quite a few production deploys now and still haven't received the error.

Thank you for the follow up. I will prepare that change for an upcoming release.

This change is now available in the <IP_ADDRESS> release of the Toolkit


import: Use re_map_foreign_keys on the realm column of UserPresence.

https://chat.zulip.org/#narrow/stream/49-development-help/topic/manage.2Epy.20import
FYI @hackerkid @showell
We forgot to make this adjustment in the recent denormalization of realm
into UserPresence. It's needed for imports to work correctly.

Merged, thanks @mateuszmandera!  One day we'll do whatever fancy logic is required to make the data export tool auto-detect what fields are foreign keys rather than needing to redeclare them manually here.


factor out module for infra machine, remove hydra

this puts information that belongs together next to each other rather
than having it in two completely different places. the inputs pattern
is not ideal and will need to be refactored for a non-flake setup, but
it works for now.
also remove the Hydra configuration, it's not used any more.
@wegank @lorenzleutgeb

Conflicts with #247, so I'll wait for that other PR.

@fricklerhandwerk how come no hydra? I asked that we keep it before as this is our makeshift staging instance, and put in some work to try to keep its configuration out of the way.

@Ericson2314 Ah, I was under the impression it's not actually getting used by anyone. We sure don't need it for this project, and I don't know who is even maintaining Hydra at all. Is there a way to move the staging to Nixpkgs or Nix, which seem to be the only major users?


Production site serious problem: sql_error_code=28000 error

There's a significant and mysterious problem on the production site. @jdossett and others - any insight welcome.
The production site became unavailable about an hour ago. I forced restart (which shouldn't be unnecessary)  and the logs are showing a lot of errors in this form:

2021-01-27T22:27:44.000000+00:00 app[postgres.21656]: [CHARCOAL] [9-1]  sql_error_code = 28000 FATAL:  no pg_hba.conf entry for host "<IP_ADDRESS>", user "postgres", database "postgres", SSL off

I've switched the production site to BADGEAPP_DENY_LOGIN mode to prevent potential future data loss, while still allowing read-only access. Oddly, it appears that the system can at least sometimes retrieve the data (!).
This does NOT happen on the master or staging sites, only on the real site, which complicates things.

I haven't changed the PostgreSQL configuration in a long time, so it's not clear what could be the problem.
From URL https://devcenter.heroku.com/articles/postgres-logs-errors

FATAL: no pg_hba.conf entry for host "…”, user “u…”, database “d…”, SSL off

You are attempting to establish a database connection without SSL. SSL is required for all connections. Please see the documentation for your postgres driver on how to establish encrypted connections.

That links to https://devcenter.heroku.com/articles/heroku-postgresql#heroku-postgres-ssl

Most clients will connect over SSL by default, but on occasion it is necessary to set the sslmode=require parameter on a Postgres connection. Please add this parameter in code rather than editing the config var directly. Please check you are enforcing use of SSL especially if you are using Java or Node.js clients.

That then links to: https://devcenter.heroku.com/articles/heroku-postgresql#connecting-in-ruby
Which says:

If you are using the pg gem to connect to your Postgres database from a Heroku dyno, and have not specified an sslmode in your configuration or code, the gem will default to sslmode: prefer: this means that your connections will work if SSL use is enforced on your Postgres database.

We never set sslmode, so it is "prefer" (as documented)... which is what we want. I still don't see a problem.

I haven't changed the PostgreSQL configuration in a long time, so it's not clear what could be the problem.
From URL https://devcenter.heroku.com/articles/postgres-logs-errors

FATAL: no pg_hba.conf entry for host "…”, user “u…”, database “d…”, SSL off

You are attempting to establish a database connection without SSL. SSL is required for all connections. Please see the documentation for your postgres driver on how to establish encrypted connections.

That links to https://devcenter.heroku.com/articles/heroku-postgresql#heroku-postgres-ssl

Most clients will connect over SSL by default, but on occasion it is necessary to set the sslmode=require parameter on a Postgres connection. Please add this parameter in code rather than editing the config var directly. Please check you are enforcing use of SSL especially if you are using Java or Node.js clients.

That then links to: https://devcenter.heroku.com/articles/heroku-postgresql#connecting-in-ruby
Which says:

If you are using the pg gem to connect to your Postgres database from a Heroku dyno, and have not specified an sslmode in your configuration or code, the gem will default to sslmode: prefer: this means that your connections will work if SSL use is enforced on your Postgres database.

We never set sslmode, so it is "prefer" (as documented)... which is what we want. I still don't see a problem.

 Ugh. Clearly the "no logins" mode is the correct one right now.

The site is repeatedly crashing & rebooting. Ugh. Clearly the "no logins" mode is the correct one right now.

I've interacted with Heroku. They believe these log messages were caused by botnets attempting to break into the database. If that's the case, the botnets were trying to connect using a way that could not possibly work & was suspicious, so the warnings (correctly) showed up on the logs. The log messages were entirely new though.
I'm a little skeptical this was just botnet nonsense, because the messages showed up immediately after reboot. But on further reflection, it does make some sense. They showed immediately on reboot. However, maybe on reboot they were queued up, and thus immediately showed on reboot. In addition, we were able to read from the database (at least some times), which would be surprising if database connections were rejected :-). So... maybe they're right! That's weird though.
The system did eventually crash. However, that's not an unknown result from a botnet :-(.
The warning messages have disappeared anyway. So I plan to restore the system soon unless there's a reason not to.

I've interacted with Heroku. They believe these log messages were caused by botnets attempting to break into the database. If that's the case, the botnets were trying to connect using a way that could not possibly work & was suspicious, so the warnings (correctly) showed up on the logs. The log messages were entirely new though.
I'm a little skeptical this was just botnet nonsense, because the messages showed up immediately after reboot. But on further reflection, it does make some sense. They showed immediately on reboot. However, maybe on reboot they were queued up, and thus immediately showed on reboot. In addition, we were able to read from the database (at least some times), which would be surprising if database connections were rejected :-). So... maybe they're right! That's weird though.
The system did eventually crash. However, that's not an unknown result from a botnet :-(.
The warning messages have disappeared anyway. So I plan to restore the system soon unless there's a reason not to.

System rebooted. All seems fine.
I'm closing this issue.

System rebooted. All seems fine.
I'm closing this issue.


When can we expect a stable release?

Hi @jayaram9877 , wanna help me?

Hi @jayaram9877 , wanna help me?

Hi @Allan-Nava , I don't have experience in developing packages, if I can get a little push at the start I can really help.

Hi @jayaram9877 , wanna help me?

Hi @Allan-Nava , I don't have experience in developing packages, if I can get a little push at the start I'll try my best.

Yess the main problem is to understand how to integrate the grpc proto stuff

@jayaram9877 this is the version stable: https://github.com/obrunsmann/flutter_nakama

It may stable right now

Hello, Could you provide a documentation to integrate this package in a project?


Issue with change_database and DatabaseConnection

Description
There is an operational error currently where django cannot find the DatabaseConnection model.

Issue was with unapplied migrations. Issue was fixed.


Tablet Needed for the Visually Impaired

A lot of the older team members (and some of the younger team members) have great difficulty reading telemetry on a phone or REV Driver Hub.  We request that several models of Android tablets be tested and approved for use as a Driver Station.

14-Inch Android 10 Tablet for $220:

Two (2) USB Type-A ports
5GHz/2.4GHz WiFi
1.3 GHz Quad-Core Processor / 3GB RAM / 64GB Storage
Attachable Keyboard (OnBot Java Programming from the Driver Station)
Dedicated DC power/charging port
In Stock

Venturer VCT9T48Q34RBM

Thank you for your suggestion, we'll take this under consideration during our next hardware evaluation period.
In the meantime, we recommend trying to adjust the "Font Size" and "Display Size" display settings to increase the size of displayed elements on your device. The real difference between these two settings is that "Font Size" only affects text, while the "Display Size" affects both text and UI elements. For the largest possible text both of these values would be maximized, but you may want to find the right balance between size and and display space for your team. We realize this isn't a perfect solution, but perhaps it could be helpful.
Here are the steps to change those settings:

Open the Settings app
Navigate to Display
Expand the Advanced section (not necessary on all devices)
Tap the Font size and/or Display size settings, and select your desired value.

Thank you for your suggestion, we'll take this under consideration during our next hardware evaluation period...

Great, thanks!
For now we can use a Tablet for development, scrimmages, pits, outreaches, etc., which is the vast majority of the time that we use a Driver Station.  We will switch to a phone when running matches at a traditional event, which is a small fraction of the time.
So this request is more of a software request, namely to add tablet layouts to the Driver Station App.

Driver Station 7.0 Running on a 14-Inch Tablet
Paired with Robot Controller running on a phone:

Left side of Landscape layout is mostly wasted space when rendered in 1920 x 1080.
Blocks editing within Driver Station:

OnBotJava editing within Driver Station:


Fix oldVal not beeing set

This fixes issue #290

Thanks!

NP!
On Tue, Feb 10, 2015, 00:08 Josh Bush<EMAIL_ADDRESS>wrote:

Thanks!
—
Reply to this email directly or view it on GitHub
https://github.com/digitalBush/jquery.maskedinput/pull/296#issuecomment-73610925
.


Test: Resolving conversations in GHPRI

Refs: https://github.com/microsoft/vscode-pull-request-github/issues/339

[x] macOS @jrieken
[x] linux @roblourens
[ ] windows @kieferrm

Complexity: 3
Create Issue

Please use the nightly version of the GitHub Pull Requests and Issues extension to test.
There is now a "Resolve conversation" button that appears on comment threads within editors. This should:

Mark the thread as resolved on GitHub (note that you cannot resolve threads that are on pending reviews)
Collapse the comment widget in the VS Code editor after resolving, and on future opens of the file (unless you expand it again)
Update the comment thread title to say the thread has been resolved
Show an "Unresolved conversation" button on comments that have been resolved

I think I am blocked by https://github.com/microsoft/vscode-pull-request-github/issues/2684. I have this issue in multiple test repos. I can open files in PRs in the vscode repo, but I don't want to test this feature on a real repo.


Added Flink 1.4.1.

Diffing package flink revision 8 revision 9

Diffing File config.json:

Diffing File marathon.json.mustache:

Diffing File package.json:
5c5
<     "version": "1.4.0.-1",
---
>     "version": "1.4.1.-1",

Diffing File resource.json:
5c5
<         "flink": "mesosphere/dcos-flink:1.4.0-1.0",
---
>         "flink": "mesosphere/dcos-flink:1.4.1-1.0",Diffing package flink revision 8 revision 9

Diffing File config.json:

Diffing File marathon.json.mustache:

Diffing File package.json:
5c5
<     "version": "1.4.0.-1",
---
>     "version": "1.4.1.-1",

Diffing File resource.json:
5c5
<         "flink": "mesosphere/dcos-flink:1.4.0-1.0",
---
>         "flink": "mesosphere/dcos-flink:1.4.1-1.0",

LGTM!


Error with packages

At step 3 of the tutorial, when running the new application, I get the error:
An unhandled exception of type 'System.IO.FileLoadException' occurred in Unknown Module.
Could not load file or assembly 'Microsoft.Azure.DocumentDB.Core, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=31bf3856ad364e35' or one of its dependencies. Strong name validation failed. (Exception from HRESULT: 0x8013141A)
How can I proceed from here?
Environment:
Windows 8.1 Pro
Microsoft Visual Studio Professional 2017
Version 15.5.7
VisualStudio.15.Release/15.5.7+27130.2036
Microsoft .NET Framework
Version 4.7.02558
Installed Version: Professional
Visual Basic 2017   00369-60000-00001-AA844
Microsoft Visual Basic 2017
Visual C# 2017   00369-60000-00001-AA844
Microsoft Visual C# 2017

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

ID: 6367067e-e9ce-6c2a-dd5d-a9de54735e78
Version Independent ID: 18564fed-2285-8cab-39a0-e3f1c2407c19
Content
Content Source
Service: cosmos-db

@RGoodmanAxioma  Thanks, There is a possible suggestion in this blogpost: https://blogs.msdn.microsoft.com/keithmg/2012/03/20/strong-name-validation-failed-exception-from-hresult-0x8013141a/ , However, for further validation, I'll assign it to the author.

Thanks for your response. I haven't followed those instructions because:

I'm not running Win7 x64 nor Vista x64 (am I? I'm running Windows 8.1 Pro as described above)
The blogpost says "It should be pointed out that disabling SN verification isnt very secure but you may have your own reasons," which I don't really. Shouldn't the DLLs downloaded via NuGet already be signed/secured/whatever it's complaining about?
It's just a tutorial, shouldn't it work without further tweaking to my system?

I'd appreciate any insight into how I can figure out which of the 99(!) DLLs that were downloaded when I added the Azure core to my project is causing this problem. I'm not really a Windows guy and this is my first foray into Azure.

@RGoodmanAxioma , Thanks for the update, the case is already assigned to the author, they should review it soon.
@AndrewHoh

FYI, I attempted to follow the instructions in the linked page and received the error shown in the attachment.

I guess this is because I'm running the wrong Windows version? Again, I'm not really a Windows person so I don't know how to work around this.
Thanks.

@RGoodmanAxioma - The error about Microsoft.Azure.DocumentDB.Core stands out to me, as that's not a Library used by this tutorial. Maybe you added Microsoft.Azure.DocumentDB.Core to your project instead of Microsoft.Azure.DocumentDB? Can you look at the references in the Solution Explorer window in Visual Studio, and see if what you have matches this?

If you have **.Core installed, simply right click and remove it and then go back to step 2 (https://docs.microsoft.com/en-us/azure/cosmos-db/sql-api-get-started#SetupVS) item 5 and add Microsoft.Azure.DocumentDB again and retry.

#in-process

@mimig1, thanks very much. I don't know how I missed that before. That seems to have solved my problem.


Unused argument incorrectly passed in Dolomite wrapper

This does not appear to present any security risk currently.
https://github.com/alcueca/erc7399-wrappers/blob/d55c154fe447745d43d3437b46d1df2c181fc762/src/dolomite/DolomiteWrapper.sol#L74
https://github.com/alcueca/erc7399-wrappers/blob/d55c154fe447745d43d3437b46d1df2c181fc762/src/dolomite/DolomiteWrapper.sol#L84
For the Withdraw and Deposit actions created in the Dolomite wrapper, the otherAddress member of the ActionArgs struct is set and passed as an argument to Dolomite.operate().
However, according to the documentation, this field is not used for Withdraw and Deposit and therefore may be removed.
Also note, in this same contract this otherAddress field is correctly being used for Call along with the data field.

The link to the documentation is wrong, but I assume that it intended to point here. In there, it specifies that otherAddress is unused, and same for primaryMarketId.
However, here the Withdraw action takes a to parameter. I assume that the first linked documentation is wrong, and the otherAddress parameter in Action is actually used as to in Withdraw. Otherwise that feature would be not implemented, which would be weird.
Furthermore, our tests use a separate contract to receive the flash loans, which the MockBorrower pulls funds from. Given that we are giving the loanReceiver address to Dolomite in the otherAddress variable, and the funds are actually appearing in the loanReceiver (shown below) it follows that the implementation is right, and the documentation wrong.

On primaryMarketId being also described as "unused" in the same documentation, Dolomite uses a single contract as entry point for all operations and the only way that we can tell Dolomite which token we are taking a flash loan of is with primaryMarketId as seen here. As a test, if we don't pass a primaryMarketId to Withdraw the token transferred from Dolomite to LoanReceiver is not the asset that we wanted to borrow (probably it defaults to something).
From Dolomite to LoanReceiver:

Attempted retrieval of asset

Sorry, but where does it say primaryMarketId is unused?

First comment says it's needed for Deposit & Withdraw

Great, I'm glad this wasn't an issue after all.
I think it says not needed for otherAddress and data.  Agree with ultrasecreth it says it's needed for primaryMArketId.
Anyways, closing, sorry for the fire drill!


Empty screen

I'm getting following result after my app launch (but no errors):

My app is inside Docker container (nginx is also involved). Maybe I should open some port for react-render-tracker?

Nope. There is an issue, when RRT sometimes doesn't load events from a page. See pending events in the toolbar. Just open and close devtools.
I will try to fix it, but it's hard to reproduce it.

The issue was fixed in version 0.4 that's just published. Thanks for reporting the issue.


Add GitHub "Contribute" Link, dropdown menu for mobile navbar

When adding the Contribute link, I thought we could instead of a burger menu have a "Menu" menu 😄

Aww yay! I'm glad


Move descheduler to latest upstream

Towads https://github.com/giantswarm/giantswarm/issues/31744

Thanks, basically I synced with upstream values respecting Network Policy and policy exceptions

do you have any automation or just copying and pasting from upstream?

There is some automation described here but in this project was not configured, and I wanted also to see how all templates are defined in upstream, so I have just copied the files :D


Use list of @types instead of doing one query per package

There is a lot of way to do that, but one simple is this one:
Microsoft create a .json with all packages on every deploy
https://github.com/microsoft/types-publisher#create-a-search-index
https://typespublisher.blob.core.windows.net/typespublisher/data/search-index-min.json

This looks interesting! @orta, do is this URL  static and safe to use for this?

(Came here after @orta asked me) Yes, the URL is static and TypeSearch already depends on this index, so types-publisher will continue to publish it.
Please do use it; I'd rather have people discover types integrated into the rest of the ecosystem instead of going to a separate site like TypeSearch.

Done in #639