Feat(#255):- Header component

Closes/Fixes/Resolves

Explain the feature/fix

Header component that can be used around the platform.
Does this PR introduce a breaking change
No

Screenshots

View Screenshots

Ready-for-merge Checklist

[x] Expected files: all files in this pull request are related to one feature request or issue (no stragglers)?
[x] Does the change have appropriate unit tests?
[x] Did tests pass?
[x] Did you update or add any necessary documentation (README.md, WHY.md, etc.)?
[x] My changes generate no new warnings
[x] I have performed a self-review of my own code
[x] I have documented my code, particularly in areas like todo, complex logic, quick fix, temporary patch, etc.

If it is a new component

[x] Does your component follow One Platform style guidelines?
[ ] Does your component web accessibility standards? Helper Doc
[ ] Does your component support desktop screen sizes (350px, 720px, 1150px ,1920px)

Browsers you have tested in

[ ] Latest two versions of Mozilla Firefox and Google Chrome supported by Red Hat Enterprise Linux distribution
[x] Google Chrome [Latest 2 versions]
[x] Mozilla Firefox [Latest 2 versions]
[ ] Microsoft Edge [Latest 2 versions]
[ ] Apple Safari [Latest 2 versions]
[ ] Microsoft Internet Explorer 11

@anjneeksharma Can you rebase the branch? It has a lot of file creation changes in the diff for files that already exist in the repo.

@deshmukhmayur  i can see only 3 commits on PR i already re based it.

Closing as there is a PR already https://github.com/1-Platform/op-components/pull/45


UploadDirectory in Mirror mode occasionally deletes destination folder's entire contents

FTP Server OS: Windows
FTP Server Type: Windows Server IIS
Client Computer OS: Windows
FluentFTP Version: 50.0.1
Framework: .NET 4.8.1
I have an app that generates around 1000 XML files (in subfolders). It then uploads a mirror to the FTP server. So, most of the time it's overwriting files with newer versions of themselves.
Occasionally, if there are files to remove, it will remove ALL files, not just the extras. The FtpResults report success.
No logs at this stage...

It uploaded the files, recorded them as successful in the FtpResults, THEN
deleted them.
I've done a workaround - delete folder, create folder, upload in Update
mode.
On Sat, 25 May 2024, 3:46 am FanDjango, @.***> wrote:

Well. Until there are logs there is only one slight idea what might be
going wrong:
If a folder listing somehow fails, unrecognized, the mirror operation
might decide that mirrored folder needs to be empty also.
But without a log of a specific occurrence? Very difficult.
—
Reply to this email directly, view it on GitHub
https://github.com/robinrodricks/FluentFTP/issues/1580#issuecomment-2130118301,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABYK2FUGF6NHBPMP5DZ57O3ZD57YLAVCNFSM6AAAAABIHBAX52VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCMZQGEYTQMZQGE
.
You are receiving this because you authored the thread.Message ID:
@.***>

It uploaded the files, recorded them as successful in the FtpResults, THEN
deleted them.

Wow. Need to check the code how that can happen.

Code has some logging to show up what it is doing and why.
LogWithPrefix(FtpTraceLevel.Info, "Delete extra file from server: " + existingServerFile.FullName);
So, without a look at your code, and a log that shows this to be happening, there will be no further analysis of this issue.

FTP.log
Here's my log, and this is the relevant part of the code:
    FtpClient client = new FtpClient(host, username, password);
    client.Logger = new FtpLogAdapter(logger);
    client.AutoConnect();
    ftpResults = client.UploadDirectory(sourcePath, remotePath, FtpFolderSyncMode.Mirror, FtpRemoteExists.Overwrite, FtpVerify.None);
    client.AutoDispose();

I'm sorry you encountered this edge case failure with the Mirror mode. I confess Mirror mode was not extensively tested. That being said, Update mode will be safer for you at this stage. The main difference between Mirror and Update is the deletion of "extra files".

I had the same issue occur today. I have been using Update mode with no problems but tried using Mirror mode for the first time. The upload folder operation results reported that it skipped 16 files, no errors, but in fact it deleted the 16 files that were already in the target upload folder. Sorry but I don't have logs at this time.

@BrianCook Nice to hear this confirmation. Very valuable.

Sorry but I don't have logs at this time.

Ok.


Stuck in Video Editing screen

Thank you based on your suggestions and after OAuth configurations application is loading via Appscript
But the site is stuck here-- tried with many videos
Content language is not English, is it a concern?

I checked the bucket, file is available there

Navigate to Cloud Functions and open the "vigenair" cloud function and go the logs view and scroll down to refresh it. Check if there are errors there, and if so, please share so I can assist you further.

I just redeployed with the new git and it works like a charm!
Thank you for the wonderful tool <3
Keep building!


DB size grows on OS X

Hello. I have OS X Yosemite 10.10.2 and go1.4.1 darwin/amd64, the situation is similar to #291. It seems that #292 fix works, because DB size grows exactly by 1GB at once. The problem is, the file grows on each bolt.Open, the initial size of DB was 3072MB
func main() {
	for i := 0; i < 100; i++ {
		db, _ := bolt.Open("file.db", 0644, nil)
		time.Sleep(time.Second)
		db.Close()
	}
}

After ~100 seconds I've got ~103GB file and no space left on the device.
file resize error: truncate file.db: no space left on device
I can tell more details upon request, just don't know what kind of details will be useful there.
Thanks.

 I ran your example using the latest (75f70a1) and it doesn't grow the database beyond the initial 1MB size.

Ah, nm. I found it once I bumped the size high enough. Thanks. Will fix.

I'm using b8dbe1101d74c30e817227a99716591e3e6e6bc4 that's the last significant commit. My DB is 3072MB in size so maybe that matters. I need some time to reproduce that bug using abstract data, so will update  this issue later. As the most simple way, I can hand you my 3GB blob privately, but it's better to make a test case :) Stay tuned.

@xlab confirmed with https://gist.github.com/kemist/f2c4a6ded05fb6aeb98d

Oh nm, didn't see your confirm ben

@xlab @kemist: Fixed: https://github.com/boltdb/bolt/pull/304

CI is :green_heart: but can one of you confirm the fix before I merge it?

Hey Ben, you are fast! Confirmed, db no longer grows

:dancer:

Thanks!


Optionally add Bitcoin address in rating data

The RatingData protobuf optionally has the buyerID. I suggest also optionally adding the Bitcoin address used for the escrow.
This will make it more expensive to generate trustworthy fake reviews because transactions fees will  be known to have been paid.

This one is tough and been the source of a lot of discussion.
We're not putting the direct/multisignature address in the rating because of privacy issues, as a third party could conduct coinanalytics on both the buyer and seller.
I am generally in favour to an opt-in inclusion of the bitcoin address, but even here you'd need to get consensus between the buyer and seller, and also the moderator if there was a dispute.
At that point things become maybe a little fragile.

How does it make it expensive if it's optional?

This will make it more expensive to generate trustworthy fake reviews

The key word is "trustworthy". A positive review without an associated Bitcoin transaction can mostly be ignored.

Are the reviews in the Apple App Store where the devs pay people in China to favorably review them trustworthy? I don't think paid fake reviews help anyone but merchants who have the money or ability to take faking reviews a step further. How is this better?
I'd love to hear @dyoniziz's take on this.

Ping @dyonyziz

This goes back to my number one complaint about this idea. It basically
requires vendors and buyers to blow their transactional privacy (since you
say that reviews that don't will be given no weight) when there is next to
nothing gained by doing so. It's not like we're saying "let's completely
blow everyone's privacy so that we can get cryptographic proof".
No it's saying "let's blow everyone's privacy so that we can marginally
increase the cost of faking reviews".
On May 3, 2017 7:44 AM, "Brian Hoffman"<EMAIL_ADDRESS>wrote:

Ping @dyonyziz
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/OpenBazaar/openbazaar-go/issues/521#issuecomment-298888231,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AES8hKrxK9_ypB7WndG1s8Quxk95UCNVks5r2GiBgaJpZM4NOTxT
.

I'd also add that I designed the review scheme precisely to avoid having to publish bitcoin addresses. This isn't just my preference but there was a recent academic paper by Andrew Miller et al on darknet marketplaces which talked about all the steps such a marketplace needs to take to avoid privacy leaks. My protocol design incorporates most of those ideas .... and I designed it before the paper was published.
If bitcoin addresses were just going to be exposed anyway (which I would have strongly argued against) then the entire protocol would have been designed different since privacy would obviously have not been a requirement.

https://arxiv.org/pdf/1701.04439.pdf

Information wants to be free. Either party wanting to publish the information is sufficient.

Absolutely not. The decision of one has an impact on the privacy of another.

I fought @cpacia on this quite a bit, but he's convinced me that this isn't the right way to go.
I think that the state of coin-analytics is way more advanced that we appreciate, so we need another approach of validating that the review is authentic and not faked.
It would be worthwhile to shift the discussion to spit-balling ways to do this.

I suspect @JustinDrake's suggestion will eventually be implemented (at least for transactions where all parties have agreed to be public). But only once reputation becomes the massive problem it inevitably will with a large userbase.

I think the bigger issue with this whole scheme is that ratings can still be withheld. The sooner we can adopt some sort of web of trust or improved reputation model the better. I'm keeping my eyes on the Trust is Risk team and I recommend you checking it out @michaelfolkson and @JustinDrake. I'd love to get your input on this as well.

Ratings can be withheld on any reputation system (including all centralized ones today) Though I commend @dionyziz on an interesting paper I'm certain we'll have a network-wide roll out of Lightning before we have a network-wide roll out of any web of trust system. There are significantly more challenges. I may trust you @hoffmabc but whether I want to put my capital at stake in trusting your friends is a completely different matter entirely.

I'm keeping my eyes on the Trust is Risk team

What is the Trust is Risk team? Do you have a link?

https://github.com/OrfeasLitos/TrustIsRisk/blob/master/fc17.pdf

"Travelers can complete a review only after they have paid for a stay, so it's not possible for someone to gin up a favorable image by asking friends to write several reviews" The Airbnb story https://www.amazon.com/Airbnb-Story-Ordinary-Disrupted-Controversy/dp/0544952669/ref=sr_1_1?ie=UTF8&qid=1498092972&sr=8-1&keywords=airbnb+story

This isn't just my preference but there was a recent academic paper by Andrew Miller et al on darknet marketplaces which talked about all the steps such a marketplace needs to take to avoid privacy leaks. Ok at least we're clear now what 2.0 is going to be. A darknet marketplace. Mainstream people don't shop on darknet marketplaces for amongst other things the increased risk of not receiving the good purchased.

I don't particularly want to waste everyone's time as it sounds like the decision has already been made but an alternative would be that there are public and private accounts. For typical darknet activity, all participants would obviously choose the private option. For innocuous, legal activity vendors and moderators would typically be public and buyers could choose whether to use their public or private account. In parallel with Zcash's shielded and unshielded transactions. Anything else is either going to be years away (Trust is Risk, ZKPs) or as little value as the number of followers in 1.0.


Unresolved Import ConvolutionConfig / Pooling

I'm trying to fiddle with leaf-examples using the latest source; with the fix for history tensors.
The main error I'm getting right now is:
`ConvolutionConfig` does not name a structure
...
`PoolingConfig` does not name a structure

It looks like this is because the module exports for leaf::layers::common::convolution and leaf::layers::common::pooling are feature gated to only be compiled when using cuda without native.
I don't understand what this is about? You'd think you'd always need native if compiling with cuda to be able to sync the memory back and read results?

There is some explanation for that here: https://github.com/autumnai/leaf/blob/3b25a48c2c2ceb49e4f9d462a8666d0adad38383/FEATURE-FLAGS.md

Ah thanks, that makes sense. I'll have to revisit the Tensor docs, I thought you needed the native backend to sync memory from the GPU to read

Yeah, you do, that's why even when the native feature of Leaf is missing we still require the native feature of its Collenchyma dependency.

Ahh yeh, Collenchyma's the one that manages the memory, isn't it. Thanks, I follow now


Don't run configlet lint manually in check_formatting.dart

Closes #296

@exercism/maintainers-admin

configlet fmt is not supported at the moment. Maybe later


jsforce.github.io API Reference link is dead

I use the jsforce API Reference a lot. It's linked in the GitHub Pages site nav:

But the link is dead now 😢:

Is it gone forever? Where can I find it?

oh sorry, I swapped the branch to test the docs for jsforce v2:
https://jsforce.github.io/jsforce
v1 is here: https://jsforce.github.io/jsforce/v1/
but let me revert that until the GA, we'll keep v1 docs under that link 👍🏼


Add possibility to create functions without trailing semicolon

Closes #7. Closes #9.

Overall, amazing progress. Left some commentary to make this nearly perfect.


Keep as much length knowledge as possible in typetracers.

I have no idea whether this addresses #1311, @douglasdavis, but it does maximize the amount of length information available by turning concrete arrays of length N into typetracer arrays of length N, rather than UnknownLength. It will definitely happen that intermediate calculations will create arrays of unknown length, which could legitimately result in the final array having unknown length. This PR should just make that happen less often, in fewer circumstances.

Oh, and I'm going to keep this open until you tell me your opinion on it. It has some general cleanup, so I'll want to merge something very much like it.
So let me know!

To do: add a method to "un-know" the length of a typetracer (both low-level and high-level), so that a length derived from the first partition is not incorrectly imputed to be the length of each partition (or the array as a whole).

To do: add a method to "un-know" the length of a typetracer (both low-level and high-level), so that a length derived from the first partition is not incorrectly imputed to be the length of each partition (or the array as a whole).

@martindurant and @douglasdavis, this adds a method so that you can
ak._v2.Array(array.layout.forget_length())

to forget a typetracer's length. It only applies one level deep. If need be, we can add a recursive=True option, but I assume that this is what you want.
This PR will merge after the tests pass.

Correct, it's only the outer-most length that we are concerned with. Keeping the other ones is probably useful.


Convert i440fx to q35 on import from OpenStack

It was already implemented for oVirt, need to do something similar for OpenStack

@mmartinv did you test migration of VMs wit i440fx chipset?

ack, so should be covered by #238


Visitor Counter is innacurate

The visitor counter is just a text file that updates by one every time script.py is executed
Thus, it will go up by one when I visit the site, and up another every time I submit the post.
Does anyone have an idea of how to do this in a better way? I think it might involve cookies but I have only briefly been introduced to them and that was in the comfort of a framework.

Depends very much on whether you want unique pageviews or total pageviews.

What would you do for either?


docs: update readme files for CLI

Description
This PR adapts the readme files for the CLI.

/rtm

@jonaslagoni I just noticed that the tables for install instructions are not aligned in the center. Should I revert this merge or create a new PR for centering them?

Okay Sure I will do that!


Help Installing Foundry, Error curl: (60) SSL certificate problem: certificate has expired

Discussed in https://github.com/foundry-rs/foundry/discussions/6510

Originally posted by cozymfer December  3, 2023
Trying to install Foundry for the first time using the first command instructed
curl -L https://foundry.paradigm.xyz | bash

but it doesnt install
 % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
  0     0    0     0    0     0      0      0 --:--:-- --:--:-- --:--:--     0
curl: (60) SSL certificate problem: certificate has expired
More details here: https://curl.haxx.se/docs/sslcerts.html
curl performs SSL certificate verification by default, using a "bundle"
 of Certificate Authority (CA) public keys (CA certs). If the default
 bundle file isn't adequate, you can specify an alternate file
 using the --cacert option.
If this HTTPS server uses a certificate signed by a CA represented in
 the bundle, the certificate verification probably failed due to a
 problem with the certificate (it might be expired, or the name might
 not match the domain name in the URL).
If you'd like to turn off curl's verification of the certificate, use
 the -k (or --insecure) option.
HTTPS-proxy has similar options --proxy-cacert and --proxy-insecure.

very new to coding in general any help would be much appreciated. Not sure how to approach this or any other ways to download or just use one of the proxy commands and trust it?

Please ensure your time is accurate on your computer. The certificate has certainly not expired :)

I dont know exactly how to change my comp to start getting correct certificates many things i try to curl and i still receive same (60) code. but my resolution was to get the code from the site and manually execute it as a script. I foundryup and it installed thanks for the help current issue resolved

@cozymfer Getting same SSL certification issue. Can you please explain how did you get code from site?


Add cross compile to aarch64, armhf, and armel

Needed for nestybox/sysbox#360

Force push was due to missing signoff


Cursor disappears on column 80

The cursor sometimes disappears when it's at the "Preferred Line Length" position.

I can tell it depends on the selected grammar and on the content of the line but couldn't figure out a list of conditions that make the bug occur.
Here is a simple example to reproduce it:

1 line file with 100 '*' characters
Grammar: JavaScript
Preferred Line Length: 80
Place cursor on column 80

Atom 1.21.1 on Ubuntu 17.04

Because the wrap-guide is rendered above the cursor?

After further investigation, it seems to be an issue with the syntax theme I use (Outlander). Closing it, sorry for the useless issue.


update package.json field

:tada: This PR is included in version 1.0.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Document migration server behavior in edge cases

with reference to https://github.com/dmwm/dbs2go/blob/master/docs/MigrationServer.md
It does not say what the output is when requesting migration for a block for which a migration was requested already.
Note that client code may use multiple processes/threads and thus the above may happen either because of concurrency or becasue a migration was submitted in an earlier processing.
Python based DBS has its way, and made CRAB code needing to rely in specifics like text in the messages [1] :-(
That surely was not optimal.
It is quite possible that semantics is different in new server, e.g. no exceptions, also I do not remember that there was a cancel migration API,  or at least never used it. But simply we need a bit more details than "this is how to check status".
[1]
https://github.com/dmwm/CRABServer/blob/cf3fd79a508c798c99e39519996069e334c93d77/src/python/Publisher/TaskPublish.py#L274
https://github.com/dmwm/CRABServer/blob/cf3fd79a508c798c99e39519996069e334c93d77/src/python/Publisher/TaskPublish.py#L295
and note the special treatment for migration request in a different server thread
https://github.com/dmwm/CRABServer/blob/cf3fd79a508c798c99e39519996069e334c93d77/src/python/Publisher/TaskPublish.py#L320

reference to old  bahavior
https://github.com/dmwm/DBS/blob/14df8bbe8ee8f874fe423399b18afef911fe78c7/Client/src/python/dbs/apis/dbsClient.py#L1517-L1561
does not cover the issue of migration request already there on a different thread and need to look it up by block. I surely got that from Yuyi but do not have a 'paper trail'.
indeed old DBS does not have a document explaining all this stuff, things are hidden in the code like
https://github.com/dmwm/DBS/blob/14df8bbe8ee8f874fe423399b18afef911fe78c7/Server/Python/src/dbs/business/DBSMigrate.py#L188-L189
All I can say is that current CRAB migration code was honed in collaboration with Yuyi N years ago and not changed since. It is not surprising that it may need changes now, I simply need to understand exactly which ones.

The new DBSMigration service process migration requests sequentially, therefore if someone submit the same request and it is already completed nothing will happen and you'll get the same status of migration. Therefore you don't need to look at edge case as you pointed out to specific string, and just check the migration status. I don't think I need to provide you more than that. And I think you don't need to modify CRAB code so far.

old migration server raised an exception when requested to migrate an already migrated jobs
new server instead completes successfully but returns 'migration_request_id': 0 and
'migration_report': 'Migration request for ... is already fulfilled, no blocks found for migration'
either behavior is OK, in the end. But I'd like to be reassured that current one is going to stay and change my code to be compatible.
JUst one question. Do I need to parse migration_report or may assume that whenever 0 is returned
as migration_request_id it means that block was already migrated ?

@belforte , the best way to detect migration request which asked for already existing data is to introduce new status code for that, e.g. 4. The status code 0 is default value and it can remain like that in different situation, e..g server error to parse json data. Since we do not have dedicated code for that you best option so far to rely on the message which btw I imitate from Python server.
If you want to use code I can easily introduce it, and this is my suggestion.
Meanwhile, I updated migration document to amplify how structures looks like.

about py exception during migration request, the exception is pure python stuff, in Go there is no exception. And, therefore for Go APIs either. Therefore, I suggest that you learn document structure of the APIs status and so far use message, and if you agree I'll provide new code for such use-case

I am happy to go w/o exceptions. We simply need to agree on protocol.
A new status code looks perfect, thanks !

I adde new code 4 (EXISTS_IN_DB) which will be assigned when migration block or dataset is already migrated. I deployed new codebase into testbed cluster and you can already use it for your tests.

Stefano, please let me know if I need to do anything with this issue, otherwise please close it.

thanks Valentin, closing.


I get the error message: tweepy.errors.Forbidden: 403 Forbidden, 453 - You currently have Essential access which includes access to Twitter API v2 endpoints only. However I have elevated access so why it is telling me otherwise?

from textblob import TextBlob
import tweepy
import pandas as pd
import numpy as np
import sys
api_key = XX
api_key_secret = XX
bearer_token = XX
access_token = XX"
access_token_secret = XX
auth_handler = tweepy.OAuthHandler(consumer_key=api_key, consumer_secret=api_key_secret)
auth_handler.set_access_token(access_token, access_token_secret)
api = tweepy.API(auth_handler)
search_term = "bitcoin"
tweet_amount = 1000
tweets = tweepy.Cursor(api.search_tweets, q=search_term, lang="en").items(tweet_amount)
for tweet in tweets:
print(tweet.text)

Have you regenerated your credentials after gaining Elevated access?
For code block usage, see https://docs.github.com/en/github/writing-on-github/working-with-advanced-formatting/creating-and-highlighting-code-blocks.

Have you regenerated your credentials after gaining Elevated access?
For code block usage, see https://docs.github.com/en/github/writing-on-github/working-with-advanced-formatting/creating-and-highlighting-code-blocks.

was going to say this. Every time you make a change to your "app" you will need to regenerate keys.


Fetched Row Count Not Match With Source Table Row Count

Issue Description
Hi, I have a Go service that act as a cron job to check all data from the database table once every week. The logic is very simple, the Go service fetches all rows from the source table and does some logic with the data there.
However, there is some issue where I have 3371351 or ~ 3million rows and 400 columns from the source table and query only gave me 138912 or ~ 100k rows (stored in Go slice) which indicates several rows didn't fetched properly by the query. In other word, the fetched rows aren't complete, even though the query process is successfully executed.
Go Code
Here my Go code that perform a simple SELECT statement:
func (repository Repository) GetAllData() ([]entity.ExampleStruct, error) {
	sql := fmt.Sprintf("SELECT * FROM [%s].[schema_data].[%s]", viper.GetString("SQL_SERVER_DBNAME"), viper.GetString("SQL_SERVER_TABLE"))

        rows, err := repository.sqlServerConnection.Query(sql)
	if err != nil {
		return nil, err
	}
	defer rows.Close()

	if rows.Err() != nil {
		return nil, rows.Err()
	}

	var exampleStructList []entity.ExampleStructList
	for rows.Next() {
		var exampleStruct entity.ExampleStruct
		err := rows.Scan(// scan struct here)
		if err != nil {
			return nil, err
		}

		exampleStructList = append(exampleStructList , exampleStruct)
	}

	return exampleStructList, nil
}

Before, I have used the WITH (NOLOCK) clause to speed up the SELECT statement and resulting in unmatched row count with the row count on the source table. So I removed the WITH (NOLOCK) clasue because I thought it affects the row fetch behavior.
Expected Behaviour
The SELECT statement should return the complete data that has same count of rows as the source table rows
If source table has 3million rows of data, my SELECT statement should also fetch 3million data in slice
Specification
Go version: 1.20.7
MS SQL Server version: Microsoft SQL Server 2008 R2 (SP3) - 10.50.6000

Hi, I haven't tried the older version, but I'll try it out.
And something that I suspect is, there is a frequent restore activity on the table from my other team, is it can affect the rows that I fetched ?

i'd assume ongoing restores would disrupt a multimillion row query, depending on what kinds of locks are in use.
They'd probably be upset if your query grabbed a lock on the entire content of the table...

Somtimes my SELECT query failed with error like this:
"Error": {
  "Number": 927,
  "State": 6,
  "Class": 14,
  "Message": "Database 'xxxx' cannot be opened. It is in the middle of a restore.",
  "ServerName": "xxxx",
  "ProcName": "",
  "LineNo": 1,
  "All": [
   {
    "Number": 927,
    "State": 6,
    "Class": 14,
    "Message": "Database 'xxxx cannot be opened. It is in the middle of a restore.",
    "ServerName": "xxxx",
    "ProcName": "",
    "LineNo": 1,
    "All": null
 }

So I can tell my SELECT query was interrupted with the restore actitivy.
I think it is make sense that the restore acitvity cause I can't retrieve a complete record from the table.


Add a sheet with Meta information to the XLSX output

For reproducibility it would be helpful to add meta information to the Excel output including filter setting and column documentation.

Investigate option to also do this for TSV output but stiull keep a valid output (e.g. comments)

Changes to tsv_to_excel.R will be required for implementation.

Include an argument to set the shet name to default.
Instead of overwriting, add an argument to add the sheet to an existing excel file.
Add logic to not overwrite existing sheets with the same name if they already exist, but instead add a number as an iterator.

Implementation in #10


Move core source to scoped package, use "redux-saga" only as proxy to it

I got to really like lately scoped packages - how they are nicely grouped in node_modules. This is a proposal to use @redux-saga/core package to hold all the core's source and use old redux-saga merely to proxy to it. WDYT?

👍   +1 for this.
If we do this, should we change how effects are imported? I think it would be too cumbersome to use import { put, take } from '@redux-saga/core/effects'. '@redux-saga/effects' seems more suitable.

The public, recommended way, of importing both redux-saga and effects would still be:
import createSagaMiddleware from 'redux-saga'


Toradex bsp6 kirkstone

This patch set gets rid of all old BSP5 settings and gets the following boards working with Toradex BSP6:

apalis-imx6
colibri-im6ull
verdin-imx8mm
verdin-imx8mp

Notably, the following boards are not supported yet by Toradex BSP6 even though some of the patches in this set have been build-tested:

apalis-imx8
colibri-imx8x

This is dependent on https://github.com/mendersoftware/meta-mender/pull/1849

I am doing some testing and there are a couple of issues:

We need to add a bigger MENDER_STORAGE_TOTAL_SIZE_MB for verdin-imx8mp, aka in meta-mender-toradex-nxp/templates/local.conf.append

MENDER_STORAGE_TOTAL_SIZE_MB:verdin-imx8mp = "2048"

Mender install fails:

INFO[0000] Loaded configuration file: /var/lib/mender/mender.conf
INFO[0000] Loaded configuration file: /etc/mender/mender.conf
INFO[0000] Output (stderr) from command "grub-mender-grubenv-print": Cannot determine environment directory.
INFO[0000] Output (stderr) from command "grub-mender-grubenv-print": Cannot determine environment directory.
ERRO[0000] Failed to read the current active partition: No match between boot and root partitions.: exit status 2

Best regards,
Leon

@leon-anavi thanks for testing. I rebased and added a commit to increase TOTAL_STORAGE_SIZE_MB for verdin-imx8mp.
@TheYoctoJester from my POV this is good to go.


.NET Core Compatible

Please make NCommand and NCommand.Console compatible with .NET Core.

Work on this has been started in pull request #9.  Supporting .NET Standard versions prior to 2.0 will require code changes.  I did not experiment with earlier versions of .NET Core.


add wireui phosphor icons port

Nice work! Is this a stand-alone package or does it depend on WireUI? If so, can you include that in the description? 🙏

@rektdeckard Thanks. It's a standalone package, doesn't depend on wireui.

@rektdeckard Thanks for the phosphor icons. It's awesome


Can we use etcd/client to access etcd v3 server?

Can we use etcd/client to access etcd v3 server?
Is there any constraint?

you can still use that client to access v2 api in etcd3 server, but not v3 api.


Create call method. Fixes #10

Adds a call method for synchronously invoking a Cloud Function.
Fixes #10.

Sorry about the merge conflicts 😿 This was a problem on machines where the Cloud SDK was set up, and the user is logged in.  That's why it never popped up on CI.
I submitted a fix just mocks out the auth library, skipping all that nock on GCE nonsense.  When you address the conflicts - let me know and we can merge!

I'm still not sure how to fix the nock issues after merging with upstream.
I think I might just start with a fresh branch and retry a new PR unless you know the issue.
 6 passing (1s)
  3 failing

  1) cloud functions api
       should check to see if the function exists:

      AssertionError [ERR_ASSERTION]: Expected values to be strictly equal:

false !== true

      + expected - actual

      -false
      +true

      at Context.it (test/test.ts:108:12)
      at process.internalTickCallback (internal/process/next_tick.js:77:7)

  2) end to end
       should deploy end to end:
     FetchError: request to https://oauth2.googleapis.com/token failed, reason: Nock: Disallowed net connect for "oauth2.googleapis.com:443/token"
      at ErroringClientRequest.<anonymous> (node_modules/node-fetch/lib/index.js:1444:11)
      at ErroringClientRequest.<anonymous> (node_modules/nock/lib/intercept.js:232:10)
      at process.internalTickCallback (internal/process/next_tick.js:70:11)

  3) end to end
       should call end to end:
     FetchError: request to https://oauth2.googleapis.com/token failed, reason: Nock: Disallowed net connect for "oauth2.googleapis.com:443/token"
      at ErroringClientRequest.<anonymous> (node_modules/node-fetch/lib/index.js:1444:11)
      at ErroringClientRequest.<anonymous> (node_modules/nock/lib/intercept.js:232:10)
      at process.internalTickCallback (internal/process/next_tick.js:70:11)

Those errors make me thing you're not rebased against master scratches head.  Can you try switching to master, doing a pull, and running the tests with no changes?

So I tried just creating a new branch:

master: passes
master + feat: passes
master + feat + new test: fails

I'm not sure why there were 2 failing tests before, I must have not called a super or something.
  validation
    ✓ should throw if a name is not provided
    ✓ should throw if multiple triggers are provided

  ignore rules
    ✓ should return 0 rules if no .gcloudignore is availabe
    ✓ should return expected rules if .gcloudignore is availabe

  pack & upload
    ✓ should pack all of the files in the target dir (62ms)
    ✓ should PUT the file to Google Cloud Storage

  cloud functions api
    ✓ should check to see if the function exists

  end to end
    ✓ should work together end to end

  8 passing (133ms)

But if I add the new test it fails. Maybe there's something wrong with the test?
  it('should call end to end', async () => {
    const scopes =
        [mockUploadUrl(), mockUpload(), mockDeploy(), mockPoll(), mockCall()];
    const c = new gcx.Caller();
    const res = await c.call({functionName: name});
    assert.strictEqual(res.data.result, '{ "data": 42 }');
    scopes.forEach(s => s.done());
  });

1) end to end
       should call end to end:
     FetchError: request to https://oauth2.googleapis.com/token failed, reason: Nock: Disallowed net connect for "oauth2.googleapis.com:443/token"
      at ErroringClientRequest.<anonymous> (node_modules/node-fetch/lib/index.js:1444:11)
      at ErroringClientRequest.<anonymous> (node_modules/nock/lib/intercept.js:232:10)
      at process.internalTickCallback (internal/process/next_tick.js:70:11)

Ah yes!  This is super subtle, and probably bad on my part.  If your thingy requires auth, use gcxp instead of gcx to make the call.  The difference is that gcxp is proxyquired, overriding the auth library implementation.

That's very tricky but makes sense.
I'm trying to nock the request by using the error in the nock but am failing find match the endpoint:
return nock('https://cloudfunctions.googleapis.com')
      .post('v1/%F0%9F%A6%84:call', {
        reqheaders: {
          "accept-encoding": [
            "gzip"
          ],
          "user-agent": [
            "google-api-nodejs-client/0.7.2 (gzip)"
          ],
          "accept": [
            "application/json"
          ],
          "content-length": [
            "0"
          ],
          "connection": [
            "close"
          ]
        }
      })
          // '/v1/projects/el-gato/locations/us-central1/functions/%F0%9F%A6%84:call')
      .reply(200, {
        executionId: 'my-execution-id',
        result: '{ "data": 42 }',
        error: null
      });

 FetchError: request to https://cloudfunctions.googleapis.com/v1/%F0%9F%A6%84:call failed, reason: Nock: No match for request {
  "method": "POST",
  "url": "https://cloudfunctions.googleapis.com/v1/%F0%9F%A6%84:call",
  "headers": {
    "accept-encoding": [
      "gzip"
    ],
    "user-agent": [
      "google-api-nodejs-client/0.7.2 (gzip)"
    ],
    "accept": [
      "application/json"
    ],
    "content-length": [
      "0"
    ],
    "connection": [
      "close"
    ]
  }
}

I wouldn't bother with nocking anything beyond the url:
return nock('https://cloudfunctions.googleapis.com').post('v1/*');

Then, when you get the response back I would check the res.config to ensure the url was what you expected it to be.

Hmm, I still can't nock it.
The latest code is on grant:grant_call2 if you can try.

Closing in favor of #15.


WebARViewer doesn't open Web samples

I deployed the WebARViewer to the HoloLens and it displays "Please open content from a web browser"
When I open http://holojs.azurewebsites.net/samples/index.html and select one of the samples, Edge goes black but displays nothing. I tried rebooting the HL, deleted and redeployed WebARViewer multiple times, but always the same behaviour.

Ah, okay, it works with  the files in v4/samples subdirectory.
Thanks, this is a great step forward, being able to make changes to a threejs project without having to deploy to the HL.

Yes, the script required for holographic rendering changes a little over time and we had to adjust the samples, thus the versions. It was an oversight to leave the non-versioned content there - thank you for reporting it.

http://holojs.azurewebsites.net/samples now points to the latest version


unpulled/unpushed status not showing

Hi,
My git status is showing correctly except for unpushed/unpulled and conflicts_solved/unsolved, which don't display at all. Attached is a screenshot showing a fresh clone and local changes. OSX Sierra, default Terminal app.

thanks!
My .zshrc :
# export PATH=$HOME/bin:/usr/local/bin:$PATH
export PATH="/usr/local/share/npm/bin:$PATH"
export PATH="/usr/local/mongodb/bin:$PATH"
export PATH="/Applications/DevDesktop/tools:$PATH"

# Path to your oh-my-zsh installation.
export ZSH=/Users/sam.elwitt/.oh-my-zsh

# Set name of the theme to load. Optionally, if you set this to "random"
# it'll load a random theme each time that oh-my-zsh is loaded.
# See https://github.com/robbyrussell/oh-my-zsh/wiki/Themes
#ZSH_THEME="robbyrussell"
ZSH_THEME="geometry/geometry"
GEOMETRY_SYMBOL_PROMPT="☕ "
GEOMETRY_SYMBOL_ROOT="😈 "
GEOMETRY_SYMBOL_EXIT_VALUE="⚠ "
GEOMETRY_COLOR_DIR="blue"
GEOMETRY_PROMPT_SUFFIX="→  "
GEOMETRY_SYMBOL_GIT_CLEAN="✔"
GEOMETRY_SYMBOL_GIT_DIRTY="⚡"
GEOMETRY_SYMBOL_GIT_UNPULLED="↓"
GEOMETRY_SYMBOL_GIT_UNPUSHED="↑"
GEOMETRY_SYMBOL_GIT_CONFLICTS_SOLVED="💚"
GEOMETRY_SYMBOL_GIT_CONFLICTS_UNSOLVED="💔"

# Uncomment the following line to use case-sensitive completion.
# CASE_SENSITIVE="true"

# Uncomment the following line to use hyphen-insensitive completion. Case
# sensitive completion must be off. _ and - will be interchangeable.
# HYPHEN_INSENSITIVE="true"

# Uncomment the following line to disable bi-weekly auto-update checks.
# DISABLE_AUTO_UPDATE="true"

# Uncomment the following line to change how often to auto-update (in days).
# export UPDATE_ZSH_DAYS=13

# Uncomment the following line to disable colors in ls.
# DISABLE_LS_COLORS="true"

# Uncomment the following line to disable auto-setting terminal title.
# DISABLE_AUTO_TITLE="true"

# Uncomment the following line to enable command auto-correction.
# ENABLE_CORRECTION="true"

# Uncomment the following line to display red dots whilst waiting for completion.
COMPLETION_WAITING_DOTS="true"

# Uncomment the following line if you want to disable marking untracked files
# under VCS as dirty. This makes repository status check for large repositories
# much, much faster.
# DISABLE_UNTRACKED_FILES_DIRTY="true"

# Uncomment the following line if you want to change the command execution time
# stamp shown in the history command output.
# The optional three formats: "mm/dd/yyyy"|"dd.mm.yyyy"|"yyyy-mm-dd"
# HIST_STAMPS="mm/dd/yyyy"

# Would you like to use another custom folder than $ZSH/custom?
# ZSH_CUSTOM=/path/to/new-custom-folder

# Which plugins would you like to load? (plugins can be found in ~/.oh-my-zsh/plugins/*)
# Custom plugins may be added to ~/.oh-my-zsh/custom/plugins/
# Example format: plugins=(rails git textmate ruby lighthouse)
# Add wisely, as too many plugins slow down shell startup.
plugins=(git)

source $ZSH/oh-my-zsh.sh

# User configuration

# export MANPATH="/usr/local/man:$MANPATH"

# You may need to manually set your language environment
# export LANG=en_US.UTF-8

# Preferred editor for local and remote sessions
# if [[ -n $SSH_CONNECTION ]]; then
#   export EDITOR='vim'
# else
#   export EDITOR='mvim'
# fi

# Compilation flags
# export ARCHFLAGS="-arch x86_64"

# ssh
# export SSH_KEY_PATH="~/.ssh/rsa_id"

# Set personal aliases, overriding those provided by oh-my-zsh libs,
# plugins, and themes. Aliases can be placed here, though oh-my-zsh
# users are encouraged to define aliases within the ZSH_CUSTOM folder.
# For a full list of active aliases, run `alias`.
#
# Example aliases
# alias zshconfig="mate ~/.zshrc"
# alias ohmyzsh="mate ~/.oh-my-zsh"

Can you try a couple of things?
Explicitly setting GEOMETRY_PROMPT_PLUGINS=(git) in your .zshrc and then closing and opening the terminal. If it doesn't work, what happens when you do git push -u?

That .zhrc edit didn't work. git push -u succeeds in pushing to the corresponding remote master.

Could you try to explicitly track remote branch?
git branch --set-upstream-to=origin/master

Same behavior after that.

Can this be caused by conflicts with oh-my-zsh's plugins? What happens when you comment out the line with plugins=(git)?

Yeah good thought, but no change after commenting that out and relaunching terminal.

Is it poss there's some variant in my local Git installation?

Using latest oh-my-zsh, geometry and git 2.7.4 seems to work as expected. Tested your configs in a VM, no issue.

Ok, I'll see if I can figure out what's different for me. Thanks for investigating!


Create command name is too generic

react new <projectName> is not very specific, what if you had another generator for React apps?
emakina-react new <projectName> should be a bit better, what do you think?

I agree, there could be a conflict.
As the CLI command should be short as its main goal is to save you some time, I initially went with react as it is a good mix of a short command and a command with some meaning.
Do you have this kind of problem or do you know tools with the same global on the path?
If this really is an issue I would go for something like emr but to be honest I do not like those acronyms at all.

yiha.. lets start a discussion about naming... my suggestion would be crac or cra-c create react app cerebral since it is very explicit about what the user will get.. a slightly modified cra setup with codegens/scafolds for cerebral
if we would want to get external contributors, we should not add the company name here (its perfectly ok to be reflected in the repo name ofc)

Do you have this kind of problem or do you know tools with the same global on the path?
I was just thinking about conflict potential in the long term, and my initial reaction to the current name was that.

Longer executable names (with args even) can be aliased in terminals, where you have to type. So I am not sure an acronym is really required. In any case, I understand the scepticism towards acronyms, and I think that crac stands out more or is more memorable.


Remove placeholder check from identical

This fixes #2533, where calling identical with cross-origin browser window objects throws a DOMException.
The reason for the exception is the checking for the placeholder property in the curry* functions.
Since placeholder support makes no sense for commutative functions it can be removed from identical. This is done here by wrapping isObject with a curry2 wrapper without placeholder property checks.

@semmel The link to the issue looks wrong. Did you mean https://github.com/ramda/ramda/issues/2533 ?

@semmel The link to the issue looks wrong. Did you mean https://github.com/ramda/ramda/issues/2533 ?

Yes indeed!

Got a couple of comments (all minors; feel free to ignore) but these changes make sense to me. Thanks.

@semmel Could you have a look at the "Checks" tab please? There are lots of linting errors that's failing the build atm.

@customcommander Thanks for running the actions and looking into the results!
test/invariants.js checks for the length property (arity?) in all R.* functions after applying to stuff.
The second test where the functions are applied to R.__ fails because it expects R.identical(R.__).length to be 2.
Should I patch test/invariants.js to exclude R.identical from the placeholder check ?

@semmel Could you have a look at the "Checks" tab please? There are lots of linting errors that's failing the build atm.

done

@semmel Could you have a look at the "Checks" tab please? There are lots of linting errors that's failing the build atm.

done

@semmel This looks good to me but I really would like another review from @ramda/core

Perhaps we can wait until v0.28.0 is released?

Perhaps we can wait until v0.28.0 is released?

More approval would be fine. We can wait so long.
With the holiday season gone however, my focus will wander off. ⛵

I'd like the next version after 0.28.0 to be 1.0, and it might be a quick release now that @customcommander has so diligently been cleaning up issues.  After that, we can remove the longstanding restriction on ES6+ code in Ramda's codebase.  I would actually love for v2.0 to be rewritten in modern JS.  But I don't think we can include this fix in the next release.
This is not code to love, but it looks like a reasonable way around the problem.

After that, we can remove the longstanding restriction on ES6+ code in Ramda's codebase.

@CrossEye Is that because of the use of Proxy in the test file?

@semmel:
Well, not just the Proxy, but also Object.defineProperty.  I was hurrying through trying to play catch-up with all the work @customcommander has been putting in, saw that code, and failed to notice that it was in the test file.  I'm much less concerned with test files and whether they are es3/es5/es6/esnext compatible.
So I withdraw my objection and will try to give this a closer look tomorrow.

@CrossEye Do you think this is ready to go?


Allow to specify the user in an argument with the "run" command

Current WSL has the feature of specifying the user on the command line. While we can do this by resetting the default UID, the convenience of a user argument would be nice.

Unfortunately, Microsoft doesn't provide public API that allows specifying user on start. wsl.exe uses some private COM API that can't be relied upon.

If you just want to launch an interactive shell with a specific user, it can be simply done by LxRunOffline run -n <distro name> -c "su - <user name>

Hello,
I am not sure if it's related, but it would be great to be able to set a default user at the distro level, as we would do using the "standard store approach"
<DistributionName> config --default-user <Username>
One side effect I have for not being able to do it, is, when I use vscode and the Remote-WSL plugin, it logs in with Root - whereas on a terminal I can start my distro with the -u parameter to set my user to use as a login.
Cheers

My bad, came across
LxRunOffline.exe set-uid -n Name_of_Distro -v user_UID
So no issue on my side

Note for those who don't know their UID (User Identifier), you can run
cat /etc/passwd | grep <username>

in your wsl terminal to get something like:
<username>:x:1000:1000:,,,:/home/<username>:/usr/bin/zsh

In the above case, 1000 is the UID for <username>.
Next, proceed to run the set-uid command as follows in Cmd:
LxRunOffline.exe set-uid -n Name_of_Distro -v UID

Reference: https://geek-university.com/linux/uid-user-identifier-gid-group-identifier/


Liquids should target block underneath when mining

Since they cannot be mined, when targeting a liquid block (lava/water), it should be ignored for block targeting and instead target the block behind it, not the infinitely hard liquid block itself:

https://github.com/satoshinm/NetCraft/issues/72 Liquids

Put another way, liquid block should be considered "transparent" for the purpose of block targeting.
Similar to https://hub.spigotmc.org/javadocs/bukkit/org/bukkit/entity/LivingEntity.html#getTargetBlock-java.util.Set-int-:
Block getTargetBlock(Set<Material> transparent,
                     int maxDistance)
Gets the block that the living entity has targeted.
Parameters:
transparent - HashSet containing all transparent block Materials (set to null for only air)

Also clouds


Pi 3

Hi,
I was wondering if the RasPi Robot Board v3 will work with Raspberry Pi 3?
Thanks

Working on my RPi3!

Works on my RP3 as well. Although making both motors go will crash the pi immediately.

I wrote up part 1 of my DFRobot Devastator build here using 2 motors with the RaspiRobot: [http://www.wemustbegeeks.com/dfrobot-devastator-tank-treaded-tracked-robot-part-1-assembly/] It uses 2 motors. There's a video of it running in Autonomous mode in the post. I'll try to get Part 2 up later this week with the different code I have been using.

Hi rplakas
Thanks for uploading details of your DFRobot.
I look forward to seeing Part 2


Switching from URL to Local Storage does not make tokens writable

Describe the bug
After switching from a URL sync to local document storage, tokens are not changeable until refreshing the page.
I'm making a short tutorial for Figma Tokens. Ideally it would use a GitHub remote, but I want to exclude some of the trickier details (like getting a PAT) so I'm using a URL storage instead to get tokens into Figma. Ideally, users should be able to import via a URL and then switch to local document storage to be able to begin editing tokens right away.
To Reproduce

In Figma Tokens -> Settings, add a Token Storage URL. For the video below, I used this gist:
https://gist.githubusercontent.com/alecgeatches/d9831e259c06a132e7c7ab9cb52e9454/raw/5cbe4d2796341b6c29acdf7a135f571fc6674cda/tokens.json

View tokens. Note that tokens are not editable as expected.

Switch to Settings, and set token storage to "Local Document".

Go back to tokens. See that the tokens are not editable.

Refresh the page.

Go to Figma Tokens -> Tokens. Note that the tokens are now editable.

Video of these reproduction steps below:
https://user-images.githubusercontent.com/17870752/189831807-942fb83c-78a6-416a-9490-1fdc4efdcc56.mp4

Expected behavior
At step 4, after switching to local document storage, tokens should be editable.
Figma file (optional)
Here is the Figma file used in the video: https://www.figma.com/file/5NZf8UfaZCPhcZRTjpRfmX/Material-3-Design-Kit---Figma-Tokens-Example
JSON (optional)
The token gist is available here: https://gist.github.com/alecgeatches/d9831e259c06a132e7c7ab9cb52e9454
Thank you!

I've experienced the same issue. As a workaround, logging out of Figma, and logging back in, makes the local document tokens editable.

Also reopening the plugin should do the trick, but agree that this is not desirable and something we should fix. Thanks for creating this!


Remove backquote from gradle instruction

Looks like there is one backquote too much, here is a fix for this :)

Thank you!


Voyager says that route <route.name> not defined after create bread

Description
After create any bread i usually see exception like "route not defined".
I use php artisan route:cache or optimize to cache my routes and if i call this command after create bread - error fixes.
This error reproduces because laravel dont know any new voyager routes until their not cached.
Steps To Fix

Create listener in App\Listeners with method handle()

namespace App\Listeners;

use Illuminate\Support\Facades\Artisan;

class RouteCacheListener
{

    public function handle() {
        Artisan::call('route:cache');
    }
}

Add this listener to your app/Providers/EventServiceProvider.php on TCG\Voyager\Events\BreadAdded

        BreadAdded::class => [
            RouteCacheListener::class,
        ]

Steps To Reproduce
Steps to reproduce the behavior:

Go to '/bread'
Click on any table
Save bread
See error

Definitely not a bug.
Of course Laravel only knows about the routes in cache when you cache them.
You shouldn't cache your routes when developing.

As @emptynick said you are not supposed to create BREADs in production and you are not supposed to cache routes on development, so something is not right in your workflow but thanks for the suggestion.
Also please remember when you are opening an issue to fill required information.
<!-- The following information is required for bug reports.  Issues without it will be closed without response --->
 - Laravel: v#.#.#
 - Voyager: v#.#.#
 - PHP: #.#
 - Database: [type] [version] (e.g. MySQL 8.0)


Update README.md

#5 Changed npm command to run the app

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

Closes: https://github.com/SAP/ui5-webcomponents-sample-vue/issues/5


If an operation is not inside an interface, don't use namespace as operation group in OpenAPI emitter

Right now interface name becomes operation groups, but without interface, the emitter uses the last part of the namespace as operation group. This should not do that, and create operation at the root in the OpenAPI spec.
Playground Link

remove operation group for the root namespace
verify that there is a mechanism for overriding the default operation group

estimate: 3


Dynamically swap map header for Annotation and Export toolbars

This removes the map-specific toolbar in favor of having all of its functionality incorporated into the page header. The header contents will slide up when the user clicks on an in-map annotation or clicks the "Export" button, and will slide back when the interface is cleared.

Looks like there's some tests that need fixing up.

Did you get the test errors from running rake check? I'm getting the same errors from that on bbg_production too.

I'm just looking at the JS tests that Travis runs. Those were fixed up recently and should be okay.

Background for the annotations opacity control seems to be off. Also there's a 'P' label to the left of the box size control; was that intentional?

The grey label text is a bit hard to read:

Looks nice! +1 after looking into JS tests and styling issues above.

Looks good. CSS is handled on their end so should be okay.
On Tue, Jul 12, 2016 at 1:08 PM, Kathryn Killebrew<EMAIL_ADDRESS>
wrote:

Looks nice! +1 after looking into JS tests and styling issues above.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/cartodb-org/cartodb/pull/82#issuecomment-232112463,
or mute the thread
https://github.com/notifications/unsubscribe/AEmtLJ6s7cGp-DhVs5ByNZRZsPa4FQZDks5qU8oZgaJpZM4JJwM7
.

--
Matt Forrest
Client Success Manager
CARTO
201 Moore St, Brooklyn, NY, 11206
P:<PHONE_NUMBER> x7508
C:<PHONE_NUMBER>
carto.com http://www.carto.com/

Spruced up styling a little, adding some verbiage about misusing private methods being bad, and fixed most of the CI errors. There's still one error that I can't quite figure out, but the errors exist in bbg_production now so I feel better to get a fix for most out sooner than not.


Remove schema and AST from the IR

Prep-work for schemaBased builders, cleanup the IR by removing the schema and AST types from there. This allows to factor some of the logic between Java/Kotlin codegen and more generally, it's better separation of concerns

As it's a relatively big PR, I'm keeping this one open until we release 3.5.0


List of Known Issues

First time opening "Matchmaking Settings" (ping control) does not load the menu

TF2 issue (confirmed).

Casual/Competitive rank and associated statistics "flicker" when I open the main menu

When the menu is loaded, the rank and stats first assume their default text-alignment before being updated to the correct version. Unfortunately, I don't know if there's anything I can do about this.

When connecting to a server while already in a server, the loading bar appears in the center of the screen.

TF2 issue (confirmed).

When joining a game, stats sometimes appear overlapped with the name of the map.

TF2 issue (not confirmed).

New item alerts sometimes cannot be closed unless you hit escape.

TF2 issue (not confirmed). Can be fixed by pressing "enter".

Added a wiki entry: https://github.com/rbjaxter/budhud/wiki/Known-Issues


FileBrowser.framework did not have a CFBundleIdentifier in its Info.plist

Why this issue ?

Thanks @khushboogupta1 for raising the query.
This library is a ReactNative wrapper around native provided libraries. For iOS it is wrapped around FileBrowser. I am not sure why the CFBundleIdentifier in not defined in their Info.plist.
What issue are you facing w.r.t CFBundleIdentifier? I have tested the library both in production and development, but never faced any issues related to CFBundleIdentifier.
Thanks
</ Pranav >


0 instances wrote while further pre-training on my own dataset

Hey,
When i run the command create_pretraining_data.py i see the following msg:
INFO:tensorflow:*** Reading from input files ***
I1210 15:59:58.812381<PHONE_NUMBER>77856 create_pretraining_data.py:419] *** Reading from input files ***
INFO:tensorflow:*** Writing to output files ***
I1210 15:59:58.815751<PHONE_NUMBER>77856 create_pretraining_data.py:430] *** Writing to output files ***
INFO:tensorflow:  tmp/tf_AGnews.tfrecord
I1210 15:59:58.815884<PHONE_NUMBER>77856 create_pretraining_data.py:432]   tmp/tf_AGnews.tfrecord
WARNING:tensorflow:From create_pretraining_data.py:97: The name tf.python_io.TFRecordWriter is deprecated. Please use tf.io.TFRecordWriter instead.
W1210 15:59:58.816398<PHONE_NUMBER>77856 module_wrapper.py:139] From create_pretraining_data.py:97: The name tf.python_io.TFRecordWriter is deprecated. Please use tf.io.TFRecordWriter instead.
INFO:tensorflow:Wrote 0 total instances
I1210 15:59:58.819541<PHONE_NUMBER>77856 create_pretraining_data.py:162] Wrote 0 total instances
Does this mean no data is created? If, yes, can you tell me why this is happening?
Thanks in advance.

Did you solve this problem?How?


MatBottomSheet incomplete docs

What is the expected behavior?
Should work if you follow the docs in example section. Instead I'm reported with error no provider for MatBottomSheetRef
What is the current behavior?
What are the steps to reproduce?
Providing a StackBlitz reproduction is the best way to share your issue.
StackBlitz starter: https://goo.gl/wwnhMV
Which versions of Angular, Material, OS, TypeScript, browsers are affected?
Is there anything else we should know?
I was able to fix it by adding
providers: [
  {provide: MatBottomSheetRef, useValue: {}}
]

You shouldn't have to use the providers to add the MatBottomSheetRef, because it's something that Material injects automatically. This is most likely an issue with your setup.


Regularizer

Hi,
I want to implement a custom ActivityRegularizer .
I want to take the output of the network and calculate some additional losses using some Python /Skcit functions.
However, for this I need to use K.eval() on self.layer.output. The code however won't allow me because K.eval() isn't available during compile time.
How do I get over this problem?

You cannot use pure Python functions inside a Keras layer or regularizer.
You should reimplement the same functionality using the math primitives
from the Keras backend (e.g. K.dot(), etc.)
On 7 December 2016 at 05:46, Rahul V<EMAIL_ADDRESS>wrote:

Hi,
I want to implement a custom ActivityRegularizer .
I want to take the output of the network and calculate some additional
losses using some Python /Skcit functions.
However, for this I need to use K.eval() on self.layer.output. The code
however won't allow me because K.eval() isn't available during compile time.
How do I get over this problem?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/fchollet/keras/issues/4626, or mute the thread
https://github.com/notifications/unsubscribe-auth/AArWb2wpDRQBNNeIvKp-vgXAHue8IQvrks5rFrihgaJpZM4LGm6M
.

I was able to see that one could use pure functions inside a weight regularizer by using K.eval(self.layer.getWeights())
We can see that the EuclidianRegularizer uses the eval() functionality.  Any specific reason why ActivityRegularizer isn't allowed to do this?

My bad!
There is no K.eval(self.layer.getWeights()) but only a K.eval() on shapes. I wonder however that it is so inconvenient not to be able to add regularization losses for which backpropogation is not important to be specified strictly in Keras.


Document breifly new geoip strategy and other strategies in general

Our documentation is missing some brief overview about balancing strategies.

@k0da fixed by https://github.com/k8gb-io/k8gb/pull/556 ? can we close it?

Yeah, lets close it


Remove uneeded import in FtcOpModeRegister.java

Yes, this is intended to be an actual PR, and is not a mistake.
See https://github.com/FIRST-Tech-Challenge/FtcRobotController/issues/383#issuecomment-1587666689
Closes https://github.com/FIRST-Tech-Challenge/FtcRobotController/issues/383 (as all the other listed issues have been fixed.)

The issue is already open, but nothing has happened.  What needs to be done so this issue can be resolved?


Move engine structure docs to the code itself

Engine documentation should be written in the source code itself - except for the guides.

We can follow this


Is "GetImageMso" obsolete ?

https://github.com/MicrosoftDocs/VBA-Docs/blob/master/api/Office.CommandBars.GetImageMso.md
GetImageMso seems to always return the dark themed image in the latest version of Office 365/2019. Maybe this should be reflected in the documentation, flag the method as obsolete or raise a bug to get it fixed.

Method was working fine until the latest Icon re-design. Ribbon was introduced in office 2007 so this looks more like an issue in the new icon pipeline.
https://support.office.com/en-gb/article/your-ribbon-icons-have-a-new-look-c6bc4cd8-d151-41d3-8276-fc7c9975eb79

@VeV-HDT For now, I added this note to the page to inform users who may be experiencing the same issue:

[!NOTE]
If GetImageMso doesn't appear as expected, it may be due to the recent icon redesign. For more information, see Your ribbon icons have a new look.


Visual Studio hangs with "Initializing JS/TS language features" when opening a .ts file

Issue Type: Bug
After a recent upgrade, when opening a typescript file, Visual Studio Code hangs and CPU utilization maxes out for several minutes with the Initializing JS/TS language features status indicator.
The closest related open issue I could find is: https://github.com/Microsoft/vscode/issues/68142, however, its title implies its a Mac issue.
After setting TypeScript Server trace to verbose:
"typescript.tsserver.trace": "verbose"
I was able to note a significant lag between the following two events upon opening a .ts file:
[Trace  - 6:44:35 PM] Event received: typingsInstallerPid (0).
Data: {
    "pid": 27112
}
[Trace  - 6:45:44 PM] Event received: projectLoadingStart (0).

Visual Studio Code Insiders cuts this down to ~1 minute but it still hangs and maxes out the CPU.
VS Code version: Code - Insiders 1.33.0-insider (c1d001a915fe85cac93492393ea69fdb0c49ec0d, 2019-03-13T06:18:20.237Z)
OS version: Windows_NT x64 10.0.16299

Can you please try collecting the TS Server log to see why it may be hanging:

Set "typescript.tsserver.log": "verbose"
Restart VS Code and reproduce the problem
In VS Code, run the TypeScript: Open TS Server log command
This should open a folder with a tsserver.log file in it

If you can share the log, I can take a look to see if anything stands out

⚠Warning: The TypeScript log may include information from your workspace, including file paths and source code. If you have any concerns about posting this publicly on Github, just let me know and we can arrange something else. On our side, we only use these logs to investigate issues like this

@mjbvz Took at hammer to it and uninstall/reinstalled. The JS/TS language features still takes a long time when starting and my CPU maxes out but its not as bad as before - or maybe I'm just getting use to it. 😧 In any event, I have attached the log. Thanks!
tsserver.log

After upgrading to Visual Studio Code 1.32.3 the performance hit initializing JS/TS is now in line with prior builds.


"CPM GUI has crashed" error on Quilt 1.20.1

https://mclo.gs/yB1lxA8

Also this is on the newest release


Add support for additional build types

We'll need to rebuild cpprest on our build machines, right?

@shishirb-MSFT I know you have this task but it was low hanging fruit to work on while iterating on the com pipeline builds - I notice the agent binary size drops from 2.7MiB to 2.3MiB, no noticeable change for the SDK.

I suspect perhaps that 0.4 MiB drop is significant in that without cpprestsdk being statically linked - the binary size would actually have been reduced significantly

In reply to: 820759844 [](ancestors = 820759844)

Yep, as well as the docker images - need to add that to the pr

In reply to: 820759450 [](ancestors = 820759450)

Yeah, thanks for doing this.

In reply to: 820760902


Added the  Vzhůru dolů podcast

What does this PR do?
Add resource(s)
For resources
Description
Podcast about web frontend technologies in Czech.
Why is this valuable (or not)?
How do we know it's really free?
All episodes are available here: https://www.vzhurudolu.cz/podcast
For book lists, is it a book? For course lists, is it a course? etc.
Checklist:

[x] Read our contributing guidelines.
[x] Search for duplicates.
[x] Include author(s) and platform where appropriate.
[x] Put lists in alphabetical order, correct spacing.
[x] Add needed indications (PDF, access notes, under construction).
[x] Used an informative name for this pull request.

Follow-up

Check the status of GitHub Actions and resolve any reported warnings!

Thanks, and welcome to free-programming-books!


Bug in FacetCounts::top_k

There seems to be a problem with how facets are getting sorted.
I've modified one of the tests slightly to make it fail, and have added a naive solution here:
https://github.com/tantivy-search/tantivy/compare/master...kardeiz:top_k_issue?expand=1
I haven't opened a PR since my "solution" is inefficient and slow.

Thanks for the report and what an embarassing bug

Your solution is probably as efficient as it gets. I pushed a similar fix to master.
I also found another minor bug.

Wonderful, thanks! 52b4575 fixes my issue. Thanks for letting me know about #357.


Add cloudhead/http-console

I think you should add this one.
Nice list ;)

Thanks! And sorry for the delay. :)


[Platforms] Update platforms definition graphic and provide instructions for future maintenance

This PR replaces the original graphic in the Platforms Definition white paper with a new one created in draw.io. It includes the draw.io XML for the new graphic in a separate file; and the XML is also embedded in the PNG itself.
This PR also removes project logos from the graphic and replaces them with generic line drawings as suggested in #388.
The advantage of using draw.io, as described in #388, is that all the details of the shapes and sub-images in the graphic are captured in an XML file which can be stored with the PNG file, so anyone can open a diagram from here in draw.io with full fidelity for each included sub-image and sub-shape. This full fidelity makes editing parts of the image much more feasible and collaborative.
See the instructions this PR adds to the README for a bit more detail on how to save edits to the image and submit them for approval.
This PR closes #413, closes #391, and closes #388.
@niklasbeinghaus I know it's been a while but if you're able I'd appreciate your review since you've advocated for switching to draw.io, and in fact #391 is very similar. Thank you!

I just stumbled upon the change (long time no time) - Thanks for mentioning me, I was quite busy doing work related stuff - love the result <3


✨Auth Page Dark/Light mode toggle button

Category

Feature

Overview

Add a button to change dark/light mode on the page.

Screenshot (if applicable)

I don’t really see how that’s useful ? We don’t need a theme switcher on every page since they shortcut works

The primary action on this page is login.
Therefore, I would expand the button and move the toggle to the top right:

I don’t really see how that’s useful ? We don’t need a theme switcher on every page since they shortcut works

It's useful because not everybody know the shortcut and if they access from a new device they might want to switch theme as soon as possible.
Position of the button is not optimal, I was waiting for feedback on that so thanks @manuel-rw.

The primary action on this page is login. Therefore, I would expand the button and move the toggle to the top right:

I was thinking the same

Changed button to ActionIcon.
Made it a General component and integrated it on the following pages:

Login (Top Right)
On Boarding (Bottom Right)
Invited (Top Right

Those are pages were the functionality to toggle theme scheme isn't available as the Avatar Menu is not accessible.
It was integrated on the bottom right on the on boarding page as it was more aesthetically fitting but I can change it to top right as well for consistency.

Is it clickable now?

Yes


Q: Approach for website with subscriptions & physical products offers

Can we use firebase-stripe-extension to also checkout one time payments, not installation payments or fees. Like for e.g. if I build a website where people buy subscriptions to consume content monthly, and now I want to add a small shop to it to sell some swag products/apparel what approach should I embrace? I want to keep the code for subscription if possible.
Thanks!

I'd recommend keeping sales of physical goods separate as you need more sophisticated tax and fulfillment management. You can also refer to this previous discussion: https://github.com/stripe/stripe-firebase-extensions/issues/39
But for selling apparel, I'd recommend something like WooCommerce, Shopify, WIX etc.

One time payments without subscriptions are supported as of version 0.1.12: https://github.com/stripe/stripe-firebase-extensions/blob/next/firestore-stripe-subscriptions/CHANGELOG.md#version-0112---2021-04-29
You can update the extension in your Firebase console to enable this.


Command ps -a --filter 'exited=1' not working

here is my command:
docker $(docker-machine config --swarm swarm-master) ps -a --filter 'exited=1'
Docker version 1.7.0, build 0baf609

Thanks for reporting this @atomi
But I have tried locally but can't not reproduce.
Can you provide swarm version and Reproduce Step.

Cannot reproduce neither with swarm 0.4 master and docker 1.8.0-rc.
@atomi  Maybe it's been fixed somehow.

@jimmyxian could you please take care of this?

$docker `docker-machine config --swarm swarm-master` version
Client version: 1.7.0
Client API version: 1.19
Go version (client): go1.4.2
Git commit (client): 0baf609
OS/Arch (client): linux/amd64
Server version: swarm/0.3.0
Server API version: 1.16
Go version (server): go1.3.3
Git commit (server): 28cd51b
OS/Arch (server): linux/amd64

@chanwit Sure :)
@atomi Test on swarm 0.3, also can't reproduce. Can you test ps -a, to see if it works and paste the output here.

@jimmyxian ``$ docker docker-machine config --swarm swarm-master ps -a```  does work and it lists all the correct info.

@atomi
Can you provide a reproducible use case? :)

I'm not sure this is a swarm issue anymore.
atomi at WORKSTATION in ~
$ docker-machine create -d virtualbox test
Creating VirtualBox VM...
Creating SSH key...
Starting VirtualBox VM...
Starting VM...
To see how to connect Docker to this machine, run: docker-machine env test

atomi at WORKSTATION in ~
$ docker `docker-machine config test` run alpine echo hello
Unable to find image 'alpine:latest' locally
latest: Pulling from alpine
31f630c65071: Already exists
alpine:latest: The image you are pulling has been verified. Important: image verification is a tech preview feature and should not be relied on to provide security.
Digest: sha256:c471fce1d08618adf4c6c0d72c047b9f3d5ef82cae0ca9a157ce1c800d42722f
Status: Downloaded newer image for alpine:latest
hello

atomi at WORKSTATION in ~
$ docker `docker-machine config test` run alpine echo hi
hi

atomi at WORKSTATION in ~
$ docker `docker-machine config test` ps -a
CONTAINER ID        IMAGE               COMMAND             CREATED             STATUS                      PORTS               NAMES
0b3255874843        alpine              "echo hi"           17 seconds ago      Exited (0) 15 seconds ago                       sleepy_bell
18a810047f2c        alpine              "echo hello"        36 seconds ago      Exited (0) 34 seconds ago                       suspicious_wilson

atomi at WORKSTATION in ~
$ docker `docker-machine config test` ps -a --filter 'exited=1'
CONTAINER ID        IMAGE               COMMAND             CREATED             STATUS              PORTS               NAMES

atomi at WORKSTATION in ~
$ docker `docker-machine config test` version
Client version: 1.7.0
Client API version: 1.19
Go version (client): go1.4.2
Git commit (client): 0baf609
OS/Arch (client): linux/amd64
Server version: 1.7.1
Server API version: 1.19
Go version (server): go1.4.2
Git commit (server): 786b29d
OS/Arch (server): linux/amd64

@atomi
We can see that the two containers exited with the code 0(Exited (0) 15 seconds ago  )
So, you have to run cmd with exited=0 , just like:
docker `docker-machine config test` ps -a --filter 'exited=0'

@jimmyxian thanks.


unable to sign in to Ringcentral CRM Extension

Started having an issue today where I cannot sign in to RC CRM Extension.  I am including a screenshot of what happens when I attempt to sign in.

@mikef723
I'm working on a patch ATM. For the time being, please try the following:

Log in to Redtail
Open the extension from Redtail page
Try sign in

If not working, please right click on the extension and "Inspect", then under "console" tab, copy and paste the error logs in red background here.

Now I see an error log under Console.  I’ve copied and pasted as well as included a screenshot.  Please let me know how to fix this as soon as possible since I currently have no phone . . . .
from a content script:https://crm.redtailtechnology.com/dashboard/overview
sw.js:110 open popup
chrome-extension://invalid/:1
   Failed to load resource: net::ERR_FAILEDUnderstand this error

adapter.js:9911 An iframe which has both allow-scripts and allow-same-origin for its sandbox attribute can escape its sandboxing.
_generateContentDOM @ adapter.js:9911Understand this warning
app.html:1 An iframe which has both allow-scripts and allow-same-origin for its sandbox attribute can escape its sandboxing.Understand this warning
adapter.js:56950 rc-dialer-status-notify: false
adapter.js:56953 rc-meeting-status-notify: false false
adapter.js:56916 rc-webphone-connection-status-notify:  connectionStatus-disconnected
popup.js:11886 rc-login-status-notify: false undefined undefined
adapter.js:56922 rc-login-status-notify: false undefined undefined
adapter.js:56944 rc-route-changed-notify: /
sw.js:253 from a content script:https://crm.redtailtechnology.com/dashboard/overview
sw.js:110 open popup
19Third-party cookie will be blocked in future Chrome versions as part of Privacy Sandbox.Understand this warning
@.***
From: DaKingKong @.>
Sent: Tuesday, July 9, 2024 2:30 AM
To: ringcentral/rc-unified-crm-extension @.>
Cc: Mike Falkowski @.>; Mention @.>
Subject: Re: [ringcentral/rc-unified-crm-extension] unable to sign in to Ringcentral CRM Extension (Issue #39)
@mikef723https://link.edgepilot.com/s/360ce4cb/IgZN5uDDeUqUc69BCWppdg?u=https://github.com/mikef723
I'm working on a patch ATM. For the time being, please try the following:

Log in to Redtail
Open the extension from Redtail page
Try sign in

If not working, please right click on the extension and "Inspect", then under "console" tab, copy and paste the error logs in red background here.
—
Reply to this email directly, view it on GitHubhttps://link.edgepilot.com/s/5016221c/wU084o4AcUiS6Uotyh3hBw?u=https://github.com/ringcentral/rc-unified-crm-extension/issues/39%23issuecomment-2216692022, or unsubscribehttps://link.edgepilot.com/s/a9d06b8a/5YNMMeijX0uHF725WRxyRg?u=https://github.com/notifications/unsubscribe-auth/AQWHA244NZ36FSEWMJXXQITZLN7NZAVCNFSM6AAAAABKRMLHBOVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDEMJWGY4TEMBSGI.
You are receiving this because you were mentioned.Message ID<EMAIL_ADDRESS>Links contained in this email have been replaced. If you click on a link in the email above, the link will be analyzed for known threats. If a known threat is found, you will not be able to proceed to the destination. If suspicious content is detected, you will see a warning.

Please help.  I am not able to use my ringcentral with Redtail at all!

@mikef723
What version is your extension? The latest one is 1.0.1, and it should be fixed. If not, let's schedule a live debugging meeting to sort it out. Please contact me with your timezone and availability via<EMAIL_ADDRESS>

I installed the latest version, the integration now works. Thank you.
From: DaKingKong @.>
Sent: Thursday, July 11, 2024 2:31 AM
To: ringcentral/rc-unified-crm-extension @.>
Cc: Mike Falkowski @.>; Mention @.>
Subject: Re: [ringcentral/rc-unified-crm-extension] unable to sign in to Ringcentral CRM Extension (Issue #39)
@mikef723https://link.edgepilot.com/s/eff9a6f4/02Kb5qyq_kaqi916jcrfuQ?u=https://github.com/mikef723
What version is your extension? The latest one is 1.0.1, and it should be fixed. If not, let's schedule a live debugging meeting to sort it out. Please contact me with your timezone and availability via<EMAIL_ADDRESS>—
Reply to this email directly, view it on GitHubhttps://link.edgepilot.com/s/051175d0/vfdU4-HvR0WVj_LSJo9e1g?u=https://github.com/ringcentral/rc-unified-crm-extension/issues/39%23issuecomment-2222136259, or unsubscribehttps://link.edgepilot.com/s/f29c9e93/xxmTT0CGAEW1iZA0ImunqQ?u=https://github.com/notifications/unsubscribe-auth/AQWHA25RO5PYZK43GSTS4I3ZLYRARAVCNFSM6AAAAABKRMLHBOVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDEMRSGEZTMMRVHE.
You are receiving this because you were mentioned.Message ID<EMAIL_ADDRESS>Links contained in this email have been replaced. If you click on a link in the email above, the link will be analyzed for known threats. If a known threat is found, you will not be able to proceed to the destination. If suspicious content is detected, you will see a warning.


Look into an External REST lib

It would be a big refactor in the way you are currently doing it. I can go over it with you on the next meeting
Originally posted by @taylorcoffelt in https://github.com/ncr4/tuneuptechnology-go/pull/1/review_comment/create

Closing as won't fix for now. Bigger fish to fry.


Email notifications use page permission helpers

Sorry for the PR spam :)
The original implementation of this helper didn't necessarily agree with what
page.permissions_for_user would tell you for a given user, so this
code has been updated to fix that. The compromise is that
include_superuser can no longer be set to False and it'll be a bit
slower, but given that this helper is only used in one place, hopefully
that's not an issue.
Not sure if raising a NotImplementedError is the way to go, or whether
we should just strip the parameter.
Replaces #1989, related to #1384.

Closing in favour of the permission policies model started in #2102 (which doesn't cover page permissions yet, but is the approach we intend to follow).


There is still a lot of room for improvement !!! But i like your work !!!

First of all, thank you very much for sharing your excellent work!
But there is still a lot of room for improvement, and I will only express my ideas from the user's perspective.
In previous tests, my MBTI personality was ENFJ. And the answer PsyDI gave me is ISTJ, and I respect this answer, after all, personality changes with time and events. But I am an E-person (in my personal opinion), and no matter what, I don't think I will be classified as an I-person.
Here are some suggestions:

You can try to slowly create a scene and have users answer around it. For example, guiding users to the company and preparing for the most tedious group meetings every day! Your best friend publicly rejected your proposal yesterday, what do you want to do now??? Instead of a rigid question being left to the user to try reading comprehension.
I don't think it's a good way to mobilize the user's emotions in the early stage, rather than directly asking them to express their recent thoughts.

Thank you very much for your kind words and for taking the time to provide such thoughtful and constructive feedback.
Firstly, sorry for the misalignment between the PsyDI result and your self-perception. Since PsyDI relies on the user's statements and the extended QA, the limited number of questions may lead to bias. Moreover, PsyDI does not directly map I and E to introversion and extroversion. Instead, we combine these dimensions to distinguish between introverted and extroverted cognitive functions. For example, when differentiating between INTP and ENTP, a stronger presence of Ne function might result in an ENTP classification, while more Ti function could lead to an INTP classification. This method may differ from the commonly used approach to determining I/E.
Then, thank you for your suggestion on creating contextual scenes. We will continue to adjust the multi-turn module to focus on specific scenarios and gradually present more in-depth questions. Your suggestion is invaluable, and we hope you will test PsyDI again after these improvements!
If you have any more suggestions or would like to discuss further, please feel free to reach out. Your feedback helps us grow and improve.


ci: update default kc version from 1.3.3 to 1.3.4

What type of PR is this?

What this PR does / why we need it:
Which issue(s) this PR fixes:

Fixes #
Special notes for reviewers:

Does this PR introduced a user-facing change?

Additional documentation, usage docs, etc.:

/release-note-none

/lgtm
/approve


Fix regression in #248

This fixes a regression in #248 where Closing a PR in the WebUI make gitea throw a panic.
Closes #344
It fixes the problem, PRs can now be closed in WebUI, but it still spits out error-logs in the console...
Started POST /toor/foobar/issues/1/comments for [::1]
[D] Session ID: 6aa78f141f0be42a
[D] CSRF Token: EVvkiIBTdh2Js1LIMFOoBLjP84s6MTQ4MDkyNzI1MTYxMjU2ODk0Mw==
[...outers/repo/issue.go:870 func1()] [C] Base/HeadRepo not set for PR!!!
[T] Issue [1] status changed to closed: true
[T] DeliverHooks [repo_id: 1]
Completed /toor/foobar/issues/1/comments 302 Found in 36.378904ms

Started GET /toor/foobar/pulls/1 for [::1]
[D] Session ID: 6aa78f141f0be42a
[D] CSRF Token: EVvkiIBTdh2Js1LIMFOoBLjP84s6MTQ4MDkyNzI1MTYxMjU2ODk0Mw==
[D] Template: repo/issue/view
Completed /toor/foobar/pulls/1 200 OK in 57.302163ms

Fixes #344 so LGTM

@Bwko Now I've actually fixed it, please update and try it again to see if the error goes away 🙂

LGTM

Cleaned up the log-msgs :)

Yes your new fix actually fixes the problem, the error is gone :)

Errors gone, Issue fixed, All things Green, Merging 😄


PKG -- [sdk] Use keyID from signatures

The reason why this is crucial for us is that for our non-custodial users, each device they login with have a different key. When a transaction request appears, it is pushed to all of their devices and we can't know in advance which device they will use to sign the transaction.
cc @boczeratul

Hey @scottphc!
Thank you for submitting this PR! I'm wondering if FCL's existing pre-authz mechanics will work for this use case ? During pre-authz, a wallet can specify which key(s) on which account(s) will act for the various roles of the transaction. If the wallet can detect which account/key combination(s) the user wishes to use for the transaction during pre-authz, and return that information, this should allow the existing mechanics of FCL to work.
Also i'm thinking that changing the account/keys combinations(s) after authz might cause issues for the multi-signer/multi-authorizer scenario, where potentially multiple signers may wish to know exactly which keys on which other accounts will be used in a transaction.

Hi @JeffreyDoyle!
Unfortunately, pre-authz can't resolve this issue. During pre-authz, we can't know the non-custodial user want to use which key to sign. We send out push notifications to the user's all devices when authz, and then the user choose one of devices to check transaction details and confirm the transaction. That's why we need to update it after authz.
I'm thinking that authorizers in payload are only addresses and the keyId may not affect other signers/authorizers?

Hey @scottphc!
The FCL team has had a lot of discussions surrounding this PR.  The consensus we have ultimately landed on here is that this is something that should be able to be done in the pre-authz service if the necessary adjustments to your existing architecture are implemented.
The suggestion here would be to do a lot of what you traditionally do in the authz service in pre-authz step instead.  This would look something like the following:

Pre-authz service is executed, and supplied with the transaction payload.
Here, your backend would push notifications to sign this payload to each of the user's devices.
The user would go to their device, choose to sign the transaction, and this signature and keyId could be sent back to your backend.  This could be store on your backend for later use in the authz service.
Now that the keyId the user wishes to use is known, your pre-authz service would resolve with an authz service for the authorizer corresponding to the keyId the user has chosen to use.  I believe this is similar to what Blocto already does for transaction payers.{
    ...
    type: "authz",
    method: "HTTP/POST",
    endpoint: MY_ENDPOINT_TO_SIGNATURE,
    "identity": {
        "address": ADDRESS_HERE,
        "keyId": KEY_ID_HERE
    },
    "params": {
        "sessionId": SOME_SESSION_ID_TO_IDENTIFY_SIGNATURE
    }
}

After pre-authz has resolved, these authz endpoints will be called by FCL, resulting in the user having the appropriate signatures.

I understand the inconvenience associated with implementing these changes, so I just wanted to give some context on why this PR won't be merged.  The reasoning boils down to what the authz service is intended for and how merging this PR could break existing design patterns.  The authz service intends to retrieve a signature for a particular keyId and having the authz function redefine the key the transaction is being signed would undermine this pattern.  We want to maintain this breaking away from this pattern could have further consequences in FCL.
Do you see any reason that this approach may not work?

I think PR here is the correct approach. I don't understand why it is not merged?
KeyId is part of signature, am I missing something here? I think this is an FCL bug that you need to provide keyId in advance.

Thanks for the tag!

It shouldn't matter to dapp which keys I choose to provide, as long as I provide valid keys and have total key weight.

This sentence sounds valid to me, as long as the signers provide the keys they used for each signature.
Unfortunately I am not familiar with FCL use cases and not sure I understood the arguments above correctly (I work more on the protocol side). I'll ping the FCL guys to have a look again at this issue.

Hi, are there any updates on this?
Looks like Blocto have released their own branched version of FCL<EMAIL_ADDRESS>with this fix, so we can use that version of FCL to resolve this issue for non-custodial Blocto wallets. But this doesn't seem to be a great idea in terms of maintainability.

Blocto is now sunsetting their v1 wallet and this PR has not been resolved yet?
https://docs.blocto.app/web-wallet-v1-sunset-notice-and-migration-guide
They recommend that dapps use their fork of fcl?

Hey!  Just wanted to let everyone know that the FCL team has made a new proposal (#1787) with changes to FCL authorization services.  Critically, this would discard the erroneous assumption that keyId matters for authz services allow an authorization service to sign with any key which they would like (or multiple).  Effectively, keyId becomes an optional field for authz services.  This has a very similar objective to this PR, however is nuanced for compatibility, versioning, etc. and addresses potential concerns regarding the approach.
Please feel free to leave comments on the issue with any suggestions or concerns!  Thanks everyone!

Hey everyone.  I've just opened a FLIP with proposed changes to the FCL specification in order to solve this issue.  https://github.com/onflow/flips/pull/215
We should try to move discussion of this issue here :)


[DOCS-1217] Fix logo jumping

@ctauchen looks like Netlify got tired to build preview for me. Let's merge. This final version should be good. Flickering of the logo will still be there, but it will be minimal. Btw, it doesn't really on our side, Docusaurus site has same problem with their little logo. For some reason on initial page load the image is taken from some cloud storage and in a moment downloads again from another location, what leads to the flash, weird but fine

Ah, no, Netlify is making a preview, I just didn't find how to see it, UX changed or smth

@denis-alkhelali Got it, thanks for this!


fix(ngrx): fix e2e / cypress tests, refactor effects

Finally :D

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@eaeddf9). Click here to learn what that means.
The diff coverage is 76.92%.

@@            Coverage Diff            @@
##             master     #423   +/-   ##
=========================================
  Coverage          ?   87.77%
=========================================
  Files             ?       67
  Lines             ?      990
  Branches          ?       41
=========================================
  Hits              ?      869
  Misses            ?      103
  Partials          ?       18

Impacted Files
Coverage Δ

src/app/examples/examples/examples.component.ts
100% <100%> (ø)

src/app/settings/settings.selectors.ts
80% <33.33%> (ø)

src/app/app.component.ts
67.64% <33.33%> (ø)

src/app/settings/settings.effects.ts
89.13% <83.33%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update eaeddf9...1cdf595. Read the comment docs.


Is there anything special about the DNS name "privatelink.postgres.database.azure.com"

...or can we choose anything we want, so long as we point to the right IP address?

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

ID: 78271f92-2709-dcaa-4771-29564329da05
Version Independent ID: 5b34e1bb-898e-4360-1c1b-8c577c619021
Content: Private Link - Azure portal - Azure Database for PostgreSQL - Single server
Content Source: articles/postgresql/howto-configure-privatelink-portal.md
Service: postgresql
GitHub Login: @kummanish
Microsoft Alias: manishku

@atrauzzi Thank you for reaching out.
@kummanish Could you please help here?

@atrauzzi this is the default DNS name that is tightly integrated with the Private link created for your PostgreSQL server. https://docs.microsoft.com/en-us/azure/private-link/private-endpoint-dns
YOu can have your custom DNS server as long as you have the FQDN to IP (A Record) in that.

Sorry, @kummanish , your response is a little confusing.  Saying "tightly integrated" and then saying it can be custom means it isn't "tightly integrated".
Is it not possible to just put some notes in the document mentioning that the DNS showed is mostly as an example?

I have added link to the DNS zone configuration. https://docs.microsoft.com/en-us/azure/postgresql/howto-configure-privatelink-portal

Did that in any way directly answer my question?
How is this helpful?

I have updated the doc with the changes mentioned. The link is still there since it is important for people who want to read more about it.

#please-close

I don't see any changes associated with this ticket... How exactly have you clearly answered my question?

#please-close

Its already published here : https://docs.microsoft.com/en-us/azure/postgresql/howto-configure-privatelink-portal#create-a-private-endpoint
Let me know if you cannot see it.

What did you change?

I have added this note:
Use the predefined private DNS zone for your service or create a new one. Refer to the Azure services DNS zone configuration for details.
The content of the DNS zone cannot be replicated in multiple docs. So having the link to read more is the preferred way. Let me know if this clarifies your question. thanks.

No it didn't at all. You aren't speaking directly with the phrase "or create a new one". It's just vague English and doesn't offer a well anchored understanding to the degree of flexibility.
It would be better to say something to the effect of "...you are free to provide a DNS zone with a name other than the default or any examples found here..."
Phrasing it that way makes it clear what you're talking about and reduces it from being too vague in the middle of so much other information.

The idea is for making it easy to use for customer. So the preferred way is through predefined DNS zone name. Therefore I can rewrite this as below. This does make it clear that it is just an option for customers to try.
"Use the predefined private DNS zone for your service or provide your preferred DNS zone name. Refer to the Azure services DNS zone configuration for details."


update toolchain to gcc-14

Resolved minor issues with compiling.
This is now using my vcpkg fork, awaiting update of glaze: https://github.com/microsoft/vcpkg/pull/38819

@omarhogni I will merge this PR, please raise an issue if you would like something changed.


Remove any Java examples/docs files that have been moved to Docs Repo

This issue is created to track the movement of any Java examples or Java FDK users docs that are being moved into the docs repo.
This issue will be closed once those files have been successfully moved to Docs and deleted from this Repo.

First PR created which moves user docs from FDK repo to Docs repo.

[ ] https://github.com/fnproject/docs/pull/68


Add missing function signatures to rocksdict.pyi

These functions are usable but are not yet defined in rocksdict.pyi

Thanks for contributing!


Wanted convolution edgeHandling example

Expected Behavior
How to use convolution function optional parameters like edgeHandling.
Current Behavior
No clear or working example.
Failure Information (for bugs)
TypeScript interface seems to need string, but Jimp.EDGE_WRAP etc. are defined as numbers.
Convolution code itself expects function as callback.
Steps to Reproduce

Jimp Version: 0.16.1
Operating System: Windows 10
Node version: nodev12.16.2

Failure Logs

https://github.com/oliver-moran/jimp/blob/53ff9d1266207f7f674233f465ec358274510511/packages/plugin-color/test/convolution.test.js#L95
It expects a number, but the type is defined as string.

@mikakaraila to fix, you can cast it to a string.  But this could be fixed..

The docs have been updated to be clearer https://github.com/jimp-dev/jimp/tree/master/packages/plugin-color#convolution