Improve mobile stats

Related to #296, this tries to improve footer on mobile:

Main changes are:

automatically fill spaces between statistics items instead of a fixed amount that does not scale well on small screens. On big screens, items are evenly distributed instead of centered. If that's an issue, I'll take a look to have some specific CSS for mobile then.
Add a message Please start drawing before showing stats footer, I find the experience unpleasant at start (specifically for newcomers) - once a route has been drawn, the message disappears and stats appear. Maybe this message could provide help to users on how to get started, etc. in the future?
Remove spaces between numbers and units, eg 4km instead of 4 km. Remove duplicated units on mobile to gain some space.
Hide stats titles starting at medium screens (previously was small screens), because it wasn't fitting on a single row anyhow.
Fixed an issue with distance title formatting, eg 3900meters should be formatted as 3.900 but it was currently only displaying 3.9 (removing trailing zeros).

ps: trip duration seems a lot pessimistic; 80minutes for 6kms seems unbelievably slow.

I like the improvements but would suggest to keep the space between values and units as it's typographically correct to have them separated by a space[1]. Maybe a narrow non-breaking space[2] would be a good tradeoff (that's the typographically correct version in German[3]).
[1] https://en.wikipedia.org/wiki/Space_(punctuation)#Unit_symbols_and_numbers
[2] https://en.wikipedia.org/wiki/Non-breaking_space#Width_variation
[3] https://de.wikipedia.org/wiki/Schmales_Leerzeichen#Verwendung

Also pushed a commit to remove zoom controls on mobile, since it is quite standard to slide in/out on mobile and we need some space on left bar too. WDYT?

Fixed an issue with distance title formatting, eg 3900meters should be formatted as 3.900 but it was currently only displaying 3.9 (removing trailing zeros).

I don't consider left out trailing zero decimals an actual issue, but using fixed decimals is more clear.

Also pushed a commit to remove zoom controls on mobile, since it is quite standard to slide in/out on mobile and we need some space on left bar too. WDYT?

Ok with me, no strong opinion here.

I like the improvements but would suggest to keep the space between values and units as it's typographically correct to have them separated by a space[1]. Maybe a narrow non-breaking space[2] would be a good tradeoff (that's the typographically correct version in German[3]).
[1] https://en.wikipedia.org/wiki/Space_(punctuation)#Unit_symbols_and_numbers
[2] https://en.wikipedia.org/wiki/Non-breaking_space#Width_variation
[3] https://de.wikipedia.org/wiki/Schmales_Leerzeichen#Verwendung

Yeah I'll fix that in a separate PR.


Fix an issue that miscount length of a query including non-ascii characters

@oysteinkrog When I use non-ascii characters for TableAttribute or ColumnAttribute, the following error was occurred.
SQLiteException: unrecognized token: ...
This is because the implementations of ISQLiteApi.Prepare2() miscounting length of the query.
Additionally, on Win32 platform, UnmanagedType.LPStr is not marshal string as UTF-8.
I recommend to use sqlite3_prepare16_v2() instead of sqlite3_prepare_v2().
I fixed these implementations and modified UnicodeTest.

Probably, this patch fixes issue #147 and #101 too.

Thank you :)


fix(lastPushPayload): Removing checkRecentNotification on setNotificationOpenedHandler

What
Removes the checkRecentNotification from setNotificationOpenedHandler
Why
As discussed on https://github.com/la-haus/iterable-flutter/issues/35, since checkRecentNotification is called every time the app is opened (setNotificationOpenedHandler), even if the user already saw the push, every subsequent app open will trigger the push notification received flow
Steps to reproduce the bug:

Send a push notification with a deep link payload
Tap the notification
The app will take the user to the deep link's page (If implemented)
Close the app
Open the app
setNotificationOpenedHandler will be triggered with the last push payload and the app will handle the last push again.
The app will take the user to the deep link's page (If implemented)

Steps 4-6 will repeat every time the app is opened.

@Gabriel-Azevedo I made a complete rewrite of the history removing an unwanted file from the repo and that broke this.
The only solution I see is that you make a new fork of the repo and do a cherry-pick of your last commits 🙈  WDYT? (sorry for that)


HTML Showing up when Image Added

When adding an image and using the master branch, raw HTML code shows above the image in the header.
An example can be seen in this post.
Here is what shows up:
"/media/2015-01-15-hss-turbine-facets/cover.png"}" /> "/media/2015-01-15-hss-turbine-facets/cover.png"}" />

You can view the code here.
Any ideas?

I just figured out how to fix it and submitted a pull request! @ianbrunelli
camporez/Thinny#10

I found the fix!
As shown in this commit, ac5194410c2ec650a7cbfc853a50d57853bfa393, you need to add .url to every instance of page.image or page.next.image to make it page.image.url or page.next.image.url!
@ianbrunelli

Hey, thanks for this, and sorry for the late answer.
I'm looking forward for your next PR.

@ianbrunelli just submitted the new Pull Request

It looks good.
I think it was already implemented in camporez.github.io repo, but I forgot to add this here.
Thank you very much :D


Unable to list files/directories when using prefix

Hello there! Nice package! I was just thinking about making one for myself, but you did it before me. I have an issue though - when I set a prefix, like this:
"github" => [
    "driver" => "github",
    "token" => "my-token",
    "username" => "avramovic",
    "repository" => "avramovic.github.io",
    "branch" => "master",
    "prefix" => "palindrom"
]

->put() and ->get() work as expected (file is written/read from "palindrom" directory), but if I try to list ->files() or ->directories() I'm getting the following error:

League\Flysystem\UnableToListContents  Unable to list contents for '', shallow listing
Reason: Unable to list contents for '', deep listing
Reason: Not Found.

Do you have any idea what may be the issue here?

There seems to be a problem with empty paths and prefixes and I think I found a solution that hopefully did not break anything else (tests are green...) with https://github.com/atomicptr/flysystem-github-storage/commit/f23cc5937da7b2cdac1ef187977727140c8be3fb

Since this library does not require a specific version of the other one this should be fixed by just updating your dependencies, please re-open the ticket should the problem persist
In that case I might need a better example though see: https://github.com/atomicptr/flysystem-github-storage/blob/master/tests/Feature/GithubAdapterTest.php#L208


[WIP] Add Tekton downstream test

/kind cleanup
Release Note

Docs

/retest
MetricsExporter

This needs the following to land (and be released): https://github.com/knative-sandbox/actions-downstream-test/pull/12
It also needs a small cleanup in Tekton: https://github.com/tektoncd/pipeline/pull/4189

/retest
Opencensus flake 🙃


Android studio 3.2 preview

Coverage remained the same at 14.514% when pulling 1b5d04e7782bda10ee0399f16da238794a7b8a75 on android-studio-3.2-preview into 5522fbd5997aac4ca2043b431eb940bc3d9c668a on master.


"Poiana Mărului" is village in "Poiana Mărului" common, "BRAȘOV" county

Description : When user is typing the villages in the search bar, the locations should appear in the dropdown menu correctly spelled and with localities information (as in Law nr. 2 din 16 februarie 1968)
Preconditions : the website (https://peviitor-ro.github.io/ui.orase/) is up and runing.
After every step delete the text you typed from the search bar.

Step 1
Type "Poiana Mărului" in the search bar.
Expected results
The location is listed in the drop-down menu as "Poiana Mărului, Brașov (Hoghiz)"


SCRIPT438: Object doesn't support property or method 'transition'  d3.js, line 8600 character 5

Hi Mike,
I'm trying to upgrade d3.js from version '3.3.13' to '3.5.2'.
I ran into the issue as described in issue title.
It only happens in IE browser.
It pretty much the same issue as https://github.com/mbostock/d3/issues/1491, but i don't think it is fixed already.
Do you have any suggestion on this?
Thanks in advance!
Ramsey

Duplicate of #2174.


Compliance Report | Part 2 Summary Line 10 and Line 21 |1|

Task:
This card builds off of the work that was completed in Card 1250 by implementing the required functionality for the Part 2 Renewable Fuel Requirement Summary section of the Summary & Declaration Page.
Line 10 - Net volume of renewable Part 2 gasoline class fuel supplied (Total of lines 2 and 5 to 9):

This line displays the net volume of renewable fuel supplied in the gasoline class for this compliance period.
The value in this cell = Line 2 + Line 5 - Line 6 + Line 7 + Line 8 - Line 9

Line 21 - Net volume of renewable Part 2 diesel class fuel supplied (Total of lines 13 and 16 to 20):

This line displays the net volume of renewable fuel supplied in the diesel class for this compliance period.
The value in this cell = Line 13 + Line 16 - Line 17 + Line 18 + Line 19 - Line 20

Design:

Non-editable cell
The corresponding value should be displayed in the format that places a comma every 3 digits. We are dealing with large volumes and unformatted number values can be difficult to read at a glance. For example, instead of 4327423 have it display on the screen as 4,327,423.

https://trello.com/c/0KG4uxDU/1480-compliance-report-part-2-summary-line-10-and-line-21-1

 Compliance Report | Part 2 Summary Line 10 and Line 21 |1|

completed in sprint quasar


Occasional bad include path

This is a small fix for an include path. The current code fails to compile if saber is not cloned under a directory named saber. For example, this matters when attempting to build with Spack, as it clones the repository into spack-src.
@climbfuji By any chance, would you be interested in incorporating recipes to build the jedi components into spack-stack?

This is a small fix for an include path. The current code fails to compile if saber is not cloned under a directory named saber. For example, this matters when attempting to build with Spack, as it clones the repository into spack-src.
@climbfuji By any chance, would you be interested in incorporating recipes to build the jedi components into spack-stack?

Yes - this is very much the direction we want to go for future releases of JEDI to the public!

Thank you @rhoneyager-tomorrow! I opened a PR with this change in jcsda-internal, and just merged it. It is now part of develop branch here as well.


POST /tx 413

When trying to upload 1mb file get POST 413.
I found https://github.com/textury/arlocal/blob/06624ab20890707cfe03f47c955cad11835875ef/src/routes/transaction.ts#L130-L134
and didn't find 413, so it looks like some third party code is not ready do get big files.

did you get any logs on arlocal? arlocal do log stuffs on the console whenever there is some sort of error?

 If that's the case, I'll have to try arloader to debug and try to fix this. this might be a bug. truly about the HTTP request handler of arlocal.

A PR has been raised for this, hopefully, it fixes the issue. Thanks for noticing this bug @iupii :heart:

Works. Thanks!


Support for data structure that can not be used in the property list.

I want to use this as.
[Bugsnag notify: [NSException<EMAIL_ADDRESS>reason:@"sample" userInfo:nil]
       withData: @{ @"error": [NSError<EMAIL_ADDRESS>code:200 userInfo:nil] }
];

but this method call is failed.
and the developer will not be able to notice that it has failed.
The reason for this is using a data structure that is not supported in the property list.
(https://github.com/dealforest/bugsnag-cocoa/blob/master/bugsnag/BugsnagNotifier.m#L160)
I think the unsported object is call -[NSObejct description].
Regards best.

Now we have rewritten the notifier to use KSCrash this PR is no longer mergeable (the entire notifier was rewritten).
I know that KSCrash has its own custom serialisation code so this might work with the new notifier straight away. If not let us know and we can have a look into adding support for this if it is appropriate.
Thanks!

No problem! :tada:


Parse all CLI stuff with Parsec (as for TxIn) to have nicer error messages

For TxIn it looks like this:
Alice> announce-auction -n foo -u dada
option -u:
unexpected end of input
expecting hexadecimal digit
Incorrect transaction id format: Failed to deserialise "dada"

Usage:  announce-auction -n AUCTION_NAME -u UTXO

  Create an auction. Requires TxIn which identifies the auction lot

Available options:
  -n AUCTION_NAME          Name for saving config and dynamic params of auction
  -u UTXO                  Utxo with test NFT for AuctionTerms
  -h,--help                Show this help text

And for Actor:
Alice> prepare-for-demo -a dada
Error during CLI action handling:

Actor parsing error
CallStack (from HasCallStack):
  error, called at app/CLI/Parsers.hs:143:16 in main:CLI.Parsers

Seems to be solved now.


Use .tar.gz for OSX releases

Eliminates a disparity between it and Linux

Done: https://github.com/GoogleCloudPlatform/docker-credential-gcr/releases/tag/v1.5.0


remove persistent domain support

Removes persistent domain for given Bundle

Hi, @muhasturk thank you for your contribution. I'm sorry it took so long to merge.
🙏


Website spelling mistake

Describe the Bug
The card on LizardByte's website says 'scipts' instead of 'scripts'

Expected Behavior
A collection of awesome Sunshine scripts, tools, guides and companion software.

Additional Context
Not sure if 'scipts' is something specific to sunshine but thought it was spelled incorrectly.

Super minor thing just thought I'd point it out

Thank you for pointing that out, I would have never noticed it!
It pulls from the repo description and will be updated in a few hours.


Switching project make search unusable

Issue
When switching project, I'm unable to perform a simple text search on any file of this project.
Environment

OS: Windows 10 64
Atom: 1.23.2

Steps
1 Open a project from the Project-Viewer panel.
2 Hit CTRL+F and perform a text search. Don't close the search panel.
3 Switch project in the Project-Viewer panel.
4 Try to search the same text, hitting F3 to show next result: nothing happens.
Sometimes closing then reopening the search panel or switching to an other file solve that, but not in any case...

Another workaround for this issue is switching to another tab and back within the same project.
Yet another workaround: Open a different file from the tree-view.
In fact it seems as if the search does not notice the change and is still searching in the old file.

the search or find-and-replace package haven't found a proper way to make them work smooth

New release seems to be working fine!

Nope, same issue here after updating.

@Getz85 ok, following your steps exactly you are correct that is still not working. But I don't see it as a huge problem for me because the search that you are preforming is within the scope of the currently focused document. The likely hood that when I search a single file in one project and switch to a new project and need to search that exact same term in whatever file is currently focused is very unlikely.
If you focus a different file and search for that term (and it exists in that file) it will find it. So a small work around would be after switching projects, focus a different file than the one that was focused when you loaded the project and then switch back.
But yes, you are right, this is still a bug that is not fixed. That was my bad for not following your directions correctly. But the other bug that is linked to this one is fixed.

Thank you so much for these feedbacks.
In fact it's as you both say, it seems that only by pressing the button Find it will trigger a search without the need to focus another file.
I am trying to solve this as well as other issues. But it's proven hard 😢


Two typos

"Letting the user to correct the data on the page..." should be "Letting the user correct the data on the page...".
"Your should design your REST API service..." should be "You should design your REST API service..."

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

ID: a0ce480b-caa7-fbae-c69b-94af8d3bed35
Version Independent ID: 423ffc59-fb72-8017-9de8-73184ace9d98
Content: About API connectors in Azure AD B2C
Content Source: articles/active-directory-b2c/api-connectors-overview.md
Service: active-directory
Sub-service: b2c
GitHub Login: @msmimart
Microsoft Alias: mimart

@timrhaynes
Thanks for your feedback! We will investigate and update as appropriate.

Thank you! I made a pull request to fix these. https://github.com/MicrosoftDocs/azure-docs-pr/pull/167038

Thank you very much!
On Tuesday, July 27, 2021, 11:35:08 AM EDT, Marilee Turscak - MSFT @.***> wrote:
Thank you! I made a pull request to fix these. MicrosoftDocs/azure-docs-pr#167038
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or unsubscribe.

Created a second PR since the first one would not publish: https://github.com/MicrosoftDocs/azure-docs-pr/pull/167584


Change in block wording from 2.0 to 3.0 - "effect color" vs. "color effect"

I am not sure if this was intentionally changed from 2.0 to 3.0?
Expected Behavior
In 2.0, the block is called "change [color] effect by (25)".
Actual Behavior
In 3.0, the block is called "change effect (color) by (25)". I am not sure if the re-ordering of color & effect was intended.
Operating System and Browser
Mac El Capitan & Chrome 56

Good catch!

Should the sound block:
"Change effect [pitch] by (10)"
also be changed to
"Change [pitch] effect by (10)" for consistency?

"Change [pitch] effect by (10)" reads better imo

Resolved in #847


Fix new CF missing BlobFileSizeCollector

Summary:
On CreateColumnFamilies we should set BlobFileSizeCollectorFactory. Fixing it.
Test Plan:
added test
Signed-off-by: Yi Wu<EMAIL_ADDRESS>

seems CI finds a data race case. will fix.

Codecov Report

Merging #78 into master will increase coverage by 0.33%.
The diff coverage is 77.77%.

@@            Coverage Diff             @@
##           master      #78      +/-   ##
==========================================
+ Coverage   81.71%   82.04%   +0.33%
==========================================
  Files          44       44
  Lines        3122     3124       +2
==========================================
+ Hits         2551     2563      +12
+ Misses        571      561      -10


When State and Country Picklists are enabled and Country values have been translated, users will receive errors when updating contact addresses

Background
State and Country picklists let users select States and Countries from predefined, standardized lists, instead of entering State and Country data into text fields. This functionality is not enabled out of the box but can be enabled/disabled by a System Administrator. Administrators can then also translate individual State and Country values using Translation Workbench, for instance for their German users they will want to translate Germany to Deutschland, United States to Vereinigte Staaten von Amerika etc
Issue Summary
If a customer has State and Country Picklists enabled and has also translated Country values in Translation Workbench they will receive errors or unexpected results when trying to update address information for a contact or in creating a new address record.
The user will also experience unexpected behaviour if they try to update the Billing Address on the Household Account. Typically updating the Billing Address on a Household Account will create a new default address record and update the Mailing Address of the Contact however where the Billing Address is being updated to a translated Country and associated State this updated address will not save to the Contact Mailing Address.
Notes:

The errors only occur when updating both the State and the Country within the same update i.e a user can successfully update the Country, save the update and then update the State
The errors do not occur during Contact creation even when setting both the State and Country upon creation
The errors do not occur if when updating Contact Country and State, the Contact Country matches the Country of the Household Account
The errors do not occur when "Household Account Addresses Disabled" is enabled, nor when updating address information on organisation accounts when "Organizational Account Addresses Enabled" is enabled
The errors do not occur if the Country is not translated and the error can be replicated in English where English translations have been added (see examples below)

Steps to Replicate
Enable State and Country Picklists
Ensure English, or the language you are testing as, is active as a Supported Language in Translation Workbench (Setup --> Translation Workbench --> Translation Language Settings)
Add Address Country Translations via Translation Workbench → Translate → Setup Component: Address Country e.g for the language English enter a Address Country Translation of "USA" for the Master Country Label "United States" and "Republic of Ireland" for Ireland
Ensure the Language for the translations you entered in Step 2 matches the Language of the user you are testing as i.e if you entered Country translations in English in Step 2, ensure you are testing as an user whose Language is English
A. Updating a contact who does not have any existing address information

Create and save a contact without any address information
Update the same contact created in the previous step, adding both State and Country values for the United States e.g State = Alabama & Country = USA
User will receive the following error: "Error:A validation rule is preventing the record from saving: A country/territory must be specified before specifying a state value for field"

B. Updating a contact's address by moving them from one translated Country/State to another translated Country/State

Create and save a contact with a United States address e.g Country = USA & State = Alabama
Update the contact to have an Irish address e.g Country = Republic of Ireland & State = Carlow
User will receive the following error: "A validation rule is preventing the record from saving: The BillingStateCode field contains CW, which isn't an available state. Your administrator can add it for you."
Note: CW is the BillingStateCode for Carlow

C. Try to update a contacts's address by updating the contact's household address

Create and save a contact with no address
Navigate to the newly created contact's household account and set a United States address e.g Country = USA & State = Alabama
The Account update will save successfully, a new address record will be created but the address will not update the Mailing Address of the Contact (a similar update for a Country/State that are not translated will update the contact's address
Note: If a user now tries to update the Contact address to Country = USA & State = Alabama, the update will save without error, whereas a similar test in A (above) generated an error upon save. Further to this, if after saving the contact update, the user tries to update the contact again to an Irish address e.g Country = Republic of Ireland & State = Carlow, they will again receive the error. If they navigate to the household account again and enter the Country = Republic of Ireland & State = Carlow it will successfully save, and also now allow them to update the contact address to an Irish address

Potential Workarounds
Disable State and Country Picklists
OR
Disable Household Address in NPSP Add i.e enable "Household Account Addresses Disabled" in NPSP Settings
OR
Remove any Country translations in Translation Workbench
OR
If your users all use the same non-English language, instead of using Translation Workbench to edit Country labels you could edit the Country labels directly in the State and Country picklists area of Setup
OR
Update a Contact Address iteratively e.g where you need to update a Contact's address from Alabama, USA to Carlow, Republic of Ireland, update the country and save the record, then update the State and save the record
OR
Update the Household Account Billing Address first, and then update the Contact's address

W-9265383

@jamesbsfdc  This also occurs when there are no translations, the account's billing address fields are blank, and there is just one Address record and it is seasonal.
This can occur in a data migration where all NPSP triggers were disabled during the migration.
Example: A household account has blank billing address field and a single Address record.  The address is a valid Florida USA address and the Seasonal fields are populated.
When the season address Batch Apex runs, it throws an error:
Update failed. First exception on row 0 with id 0015t000003Cq0nAAC; first error: FIELD_INTEGRITY_EXCEPTION, The BillingStateCode field contains FD, which isn't an available state. Your administrator can add it for you.: [BillingStateCode]
This is because "Florida" is also a department in Uruguay and its ISO code is FD.

Hi @davidchengIC  Thanks for reporting this. Do you have an open support case about this issue? If so would you mind posting a case number, or if there is no current case would you mind creating one and posting back with the case number?
If we can verify the issue, we can look at widening the scope of this Github issue or possibly creating a new one. Until address type fields are supported on custom objects I fear there may be no easy way of resolving this and similar issues that does not entail some kind of tradeoff.

@jamesbsfdc I don't have a case yet.  I did find a workaround - if I deactivate the Florida value for Uruguay, then the batch will process the Florida USA address successfully.

Please follow this Known Issue in the Trailblazer Community for updates.


My sql version provider

Hi Antony
I have added the MsSqlVersionProvider.
I have also added a test for yhour reference.
Grab what you need :-)
BR Rik

Great, thanks! I'll probably next have a chance to look at this early next week.

Why did you close and delete this? I'm going to use your MySQL version provider.

Great.
I just redid my pull request. Sorry for the mess. I want happy with the commits.
Can you tell me hwhen yhou plan the MsSql and MySql release to Nuget?
I actually need it for a task :)
BR RIk
From: Antony Male<EMAIL_ADDRESS>Sent: 14. oktober 2016 19:45
To: canton7/SimpleMigrations<EMAIL_ADDRESS>Cc: generik0<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [canton7/SimpleMigrations] My sql version provider (#3)
Great, thanks! I'll probably next have a chance to look at this early next week.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/canton7/SimpleMigrations/pull/3#issuecomment-253871228 , or mute the thread https://github.com/notifications/unsubscribe-auth/ACTMK2BPUOgfW-MYiwZ3iZFfT2erqVqCks5qz7-JgaJpZM4KXQBD .  https://github.com/notifications/beacon/ACTMKwA73ZpfCtP-87dJSWvqGrl4SF_Eks5qz7-JgaJpZM4KXQBD.gif

I'll release them properly on NuGet early to mid next week, probably.
You can get intermediate builds from this NuGet feed: https://ci.appveyor.com/nuget/simplemigrations. This has the latest commits, and contains your MSSQL work. You probably want version <IP_ADDRESS> - I removed the ConnectionProvider after that, and it will probably be reinstated.

Hi
If you do use the chance i made, then you could add another abstract class. The first level with TTransacrion and this has a base without a transaction.
That way you can handle both senatios :-)
Br Rik
Sent from my iPhone

On 14 Oct 2016, at 19.56, Antony Male<EMAIL_ADDRESS>wrote:
I'll release them properly on NuGet early to mid next week, probably.
You can get intermediate builds from this NuGet feed: https://ci.appveyor.com/nuget/simplemigrations. This has the latest commits, and contains your MSSQL work. You probably want version <IP_ADDRESS> - I removed the ConnectionProvider after that, and it will probably be reinstated.
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub, or mute the thread.

I ended up going for an approach where the connection encapsulates the transaction: https://github.com/canton7/SimpleMigrations/commit/1f82d5724dcf2b45c6ba188f0454993324371dac. Look OK?

Looks good. A good solution.
BR Rik
From: Antony Male<EMAIL_ADDRESS>Sent: 15. oktober 2016 15:32
To: canton7/SimpleMigrations<EMAIL_ADDRESS>Cc: generik0<EMAIL_ADDRESS>State change<EMAIL_ADDRESS>Subject: Re: [canton7/SimpleMigrations] My sql version provider (#3)
I ended up going for an approach where the connection encapsulates the transaction:  https://github.com/canton7/SimpleMigrations/commit/1f82d5724dcf2b45c6ba188f0454993324371dac 1f82d57. Look OK?
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub https://github.com/canton7/SimpleMigrations/pull/3#issuecomment-253984474 , or mute the thread https://github.com/notifications/unsubscribe-auth/ACTMK1Pg7hH3GbvaXfrxTkJll4jiS0v3ks5q0NXagaJpZM4KXQBD .  https://github.com/notifications/beacon/ACTMKyG7JIjf3uZgQgRJRbelGU0g2XXdks5q0NXagaJpZM4KXQBD.gif

v0.9.6 has been released on nuget.


Stacks hangs if you add text to the clip, which content type is set to link without changing it's content type

Steps to reproduce:

Copy https://github.com/cablehead/stacks to clipboard (the clip content type is automatically set to link)

Press Ctrl-Enter to edit clip

Add some text to the beginning of the clip (say, "Some title" and "Enter" to make new line)

Press Ctrl-Enter to capture edit

Try to use arrows up/down or any of keyboard hotkeys - Stacks does not respond.

I'm pretty sure this is because I added text to clip which content type was set to link without changing the content type back to text.
If you restart Stacks the clip type appears as text:

But if you scroll to the clip which content type is set wrong (as the last clip in this scrinshot) Stacks will hang again:

great catch 🙏🏻


Crashes on uncaught exception

When I run this on my admittedly very complex object model, I get an uncaught exception:
UNCAUGHT EXCEPTION (NSInvalidArgumentException): *** -[__NSPlaceholderArray initWithObjects:count:]: attempt to insert nil object from objects[0]
I've attached the entire crash report.
I love this tool!  I successfully ran it on this project once a while ago, and was able to give people who have never touched this code an idea of how complex it is.
ObjectGraph_crashlog.txt

According to my experience, this exception occurs when the tool is drawing graph, and Xcode loses focus. It will failed to get valid project related values. So, keep focus on Xcode when the tool is drawing graph.


Fix/825 (part 1): specify SPIGET resource version

allows to specify which version of the SPIGET resource to download
probably fixes #825 partially
Sorry for first contribution bug (#981), this time it won't have ${$var} in it :/, but I still didn't build the docker image from my code, so not fully tested.

Sorry it took me a while to get back to this. The code changes look good...

I still didn't build the docker image from my code, so not fully tested.

...but I really need you to build the image and test since I have been able to setup very little CI testing.

$ curl -I https://api.spiget.org/v2/resources/9089/download | grep location
location: https://cdn.spiget.org/file/spiget-resources/9089.zip

$ curl -I https://api.spiget.org/v2/resources/9089/versions/latest/download | grep location
location: https://spigotmc.org/resources/9089/download?version=367353

The spiget api returns diffrent download address if a specific version is requested.
That doesn't seem bad first but if you try curl'ing spigotmc.org then I get http error 503 from Cloudflare servers.

The spiget api returns diffrent download address if a specific version is requested.

Yeah and same useless response from curl -I https://api.spiget.org/v2/resources/9089/versions/367353/download | grep location. Avoiding the SpigotMC CloudFlare is whole point of using the Spiget API, so an extra shame they broke this.

I think we can't do anything to solve this, except hoping/helping spiget fixing it.
I created a issue in their issue collection here.

Avoiding the SpigotMC CloudFlare is whole point of using the Spiget API, so an extra shame they broke this.

Nothing was broken, it has always been this way.

My apologies, I had not used that particular endpoint previously so my assumption that that it broke since initial deployment was not valid.
All I know is that the requested feature #825 is blocked by an inconsistent redirect outcome of the version-qualified and non-qualified resources endpoints:
$ curl -I https://api.spiget.org/v2/resources/9089/download
HTTP/2 302
date: Fri, 30 Jul 2021 19:12:07 GMT
content-type: text/plain; charset=utf-8
content-length: 75
location: https://cdn.spiget.org/file/spiget-resources/9089.zip
cf-ray: 6770e58e5ce5ec11-ATL
access-control-allow-origin: *
vary: Accept, Accept-Encoding
cf-cache-status: BYPASS
expect-ct: max-age=604800, report-uri="https://report-uri.cloudflare.com/cdn-cgi/beacon/expect-ct"
x-powered-by: Express
x-spiget-file-source: cdn
x-spiget-server: nougat
report-to: {"endpoints":[{"url":"https:\/\/a.nel.cloudflare.com\/report\/v3?s=9GpDdPaVdNiZmvy9LtbMz89TTtHx8W2KtJgvRT4Fi9gMGbDV5pifg6D65GXycbWI5qmrqULiQkUqnFvSgiDsgFcsunTl%2F8ZNtVK57VMIJzWIPp5AyIzi1eVdvxMG%2FHM%2F"}],"group":"cf-nel","max_age":604800}
nel: {"report_to":"cf-nel","max_age":604800}
server: cloudflare

$ curl -I https://api.spiget.org/v2/resources/9089/versions/367353/download
HTTP/2 302
date: Fri, 30 Jul 2021 19:13:06 GMT
content-type: text/plain; charset=utf-8
content-length: 81
location: https://spigotmc.org/resources/9089/download?version=367353
cf-ray: 6770e6fcad67f325-ATL
access-control-allow-origin: *
vary: Accept, Accept-Encoding
cf-cache-status: BYPASS
expect-ct: max-age=604800, report-uri="https://report-uri.cloudflare.com/cdn-cgi/beacon/expect-ct"
x-powered-by: Express
x-spiget-server: mango
report-to: {"endpoints":[{"url":"https:\/\/a.nel.cloudflare.com\/report\/v3?s=9pEVx2H9oXY3KYYISRrsHFqHF3Y%2BMA366291Yam4ziLt1IqhHoSF%2B%2F6AtYKCawbt54NzG4D0NCBaESaeEhuBxlYu29subl4Anrv4wHgJ%2BGDiamPZYgs92h5PmvasxXbC"}],"group":"cf-nel","max_age":604800}
nel: {"report_to":"cf-nel","max_age":604800}
server: cloudflare

and then following the redirect in the second:
HTTP/2 503
date: Fri, 30 Jul 2021 19:18:35 GMT
content-type: text/html; charset=UTF-8
x-frame-options: SAMEORIGIN
permissions-policy: accelerometer=(),autoplay=(),camera=(),clipboard-read=(),clipboard-write=(),fullscreen=(),geolocation=(),gyroscope=(),hid=(),interest-cohort=(),magnetometer=(),microphone=(),payment=(),publickey-credentials-get=(),screen-wake-lock=(),serial=(),sync-xhr=(),usb=()
cache-control: private, max-age=0, no-store, no-cache, must-revalidate, post-check=0, pre-check=0
expires: Thu, 01 Jan 1970 00:00:01 GMT
cf-request-id: 0b9a73b9b400000fec1b8a7000000001
expect-ct: max-age=604800, report-uri="https://report-uri.cloudflare.com/cdn-cgi/beacon/expect-ct"
report-to: {"endpoints":[{"url":"https:\/\/a.nel.cloudflare.com\/report\/v3?s=tsdiA6BXfrYdQkFXOmlj8QkYBpoh4137S%2FvmVVq6sgVtso5AinLV3M1moIkBvVmE3fH%2FplZSw201usXN2X5mB9ET%2FoAnVdkuHjLRAz5%2F%2Bj0IT5nxFGngG32f3GyikQ%3D%3D"}],"group":"cf-nel","max_age":604800}
nel: {"report_to":"cf-nel","max_age":604800}
server: cloudflare
cf-ray: 6770ef0929b90fec-ATL
alt-svc: h3-27=":443"; ma=86400, h3-28=":443"; ma=86400, h3-29=":443"; ma=86400, h3=":443"; ma=86400

they're two completely different api endpoints and that's also stated in the documentation.
see https://github.com/SpiGetOrg/Spiget/issues/41#issuecomment-890046267 as well

This is a (somewhat) old thread. To keep the repository clean I close it now with this comment since I don't think there is anything we or spiget could really do about it.
Feel free to reopen it.


How handle long conversation history

As history grows summarization of conversation would be useful.
If a question is just to format the previous response or anything that doesn't need further retrieval at that time shouldn't we be detecting if retrieval is needed?

Thanks @cahuja1992
1. As history grows summarization of conversation would be useful.
This makes sense, but would include one more call to the LLM, how long do you think it would make sense to start summarizing?
2. If a question is just to format the previous response or anything that doesn't need further retrieval at that time shouldn't we be detecting if retrieval is needed?
Now we have a triage function that does exactly this.

As chat history is passed in the triage, there we will reach the token limits. So one idea is to as we reach 90% of the token limits, spawn a thread of chat summarization so that for the next question we can use the summary.

@placerda As chat history is passed in the triage, there we will reach the token limits. So one idea is to as we reach 90% of the token limits, spawn a thread of chat summarisation so that for the next question we can use the summary.

@placerda Any thoughts on this approach? If we proactively keep on summarizing the conversation, then we do not even see significant differences in latency.


fix(behavior_path_planner): fix turn signal endless loop

Description

fix turn signal endless loop
https://github.com/autowarefoundation/autoware.universe/issues/5440
Tests performed

https://evaluation.tier4.jp/evaluation/reports/c40eba8d-7d8e-5aa6-9c33-071b5b7841ac?project_id=prd_jt
1665/1667
Effects on system behavior

Not applicable.
Pre-review checklist for the PR author
The PR author must check the checkboxes below when creating the PR.

[x] I've confirmed the contribution guidelines.
[x] The PR follows the pull request guidelines.

In-review checklist for the PR reviewers
The PR reviewers must check the checkboxes below before approval.

[ ] The PR follows the pull request guidelines.

Post-review checklist for the PR author
The PR author must check the checkboxes below before merging.

[ ] There are no open discussions or they are tracked via tickets.

After all checkboxes are checked, anyone who has write access can merge the PR.

@danielsanchezaran
First of all, thank you so much for the report!!!!! That's super helpful.
Could you review this PR and approve after your all comments are resolved?
And could you close the issue after merging this PR?
I will approve this PR because maintainer's approve is necessary and you are not the maintainer for now. But mergeing will be handled by @.kosuke55 san!
When close the issue, we really appreciate it if you upload the video that problem is solved.
(Just uploading the same video you shared internally

@kosuke55 CC @kyoichi-sugahara
LGTM. Thanks for your quick response. With this fix the issue has been solved!


Set image title

Thank you @dmouse for your help. I updated the jest snapshot because your commit broke tests and I published v1.5.0


Change page-detect to be URL-based

Fixes #783 (among others)

Can you include the reasoning for this?

From https://github.com/sindresorhus/refined-github/issues/761#issuecomment-340307775

On that note, perhaps we should strive to keep page detections as URL based. Most features test for elements on the page before proceeding, so at that point just skip the duplicate dom query.

This would make page-detect more like the router you mentioned. Also only URL-based checks are currently testable.

Let me know when you're ready for this to start testing and I'll give it a smoke run through GHE.

Yep it’s ready! There’s only one select in there and that’s going to be removed by #768

a3887da fixes #783

Looks good on GHE, I did find an unrelated bug that I'll open an issue for.

Considering these changes are all covered by tests (old and new ones), I hope it will be merged soon to fix some isRepo-related bugs.

Can you fix the merge conflict?

🙌

@sindresorhus should we extract this? I think it's mature enough and other extensions can benefit from it.
It'd need a bundler to support extensions not complex enough to justify having one. Bundler example: e.g. https://github.com/bfred-it/webext-dynamic-content-scripts/blob/master/package.json#L30-L46

Yeah, sure.


Writing Captions to a List

Hi,
I have successfully downloaded bios for a list of accounts I have, and now need to download the captions from their posts but cannot figure out what needs to change to get them. Right now I use this to get the bio information:
instaloader.Profile.from_username(L.context, handles[r]).biography
However, switching the biography part to get_posts( )(which is what I thought would be the fix based on the documentation) is not working.
If there is a way to write the captions to a list that would be ideal.
Thanks!

Hi,
Profile.get_posts() returns an Iterator. So, for each account you will need to iterate over its posts and access the caption property.


[SourceKit] Add function argument when optional chaining is used (SR-5717)

Adds function argument declarations to structure when optional chaining is used.
Resolves SR-5717.
PS: I have no idea if this is the proper fix for this issue. Feel free to make suggestions.
// cc @nkcsgexi

ping @nkcsgexi 😅

@swift-ci please smoke test


Enable switching between source member edit modes

Changes
At request of @jkyeung, I have add the newer diff-based source date approach as an optional feature. There are two reasons I have done this:

Allow users to test this feature.
Allow users who want to test this feature also stay up to date with other features.

Checklist

[x] have tested my change
[ ] updated relevant documentation
[ ] Remove any/all console.logs I added
[x] eslint is not complaining
[ ] have added myself to the contributors' list in CONTRIBUTING.md
[ ] for feature PRs: PR only includes one feature enhancement.

@jkyeung please try this build and if you're happy with the way to switch between editing modes, then we can merge and deploy it. Thanks.

The interface for switching between the two modes is good! Go for it!
(As a more general comment, I personally have not been able to develop a feel for what settings are "extension" settings and which are "connection" settings. Perhaps it would be more obvious if I regularly worked with more than one connection. But given the status quo, it makes sense to add this new thing to the same place where the source dates are enabled.)

@jkyeung Awesome - glad it's looking good. I will merge this in tomorrow.
Thanks!


start-server.bat stops in "==> boss (compile)"

In windows 8.1 with erlang 18.3 and CB master branch after successfully creating a new project I execute start-server.bat and seems to get and compile a bunch of deps, but stops (like one hour) in  ==> boss (compile), and nothing happens.
Here is the last output:
==> simple_bridge (pre_compile)
==> simple_bridge (compile)
Generating crypto compatibility for simple_bridge...
...Using: "crypto:hash(sha, Data)"
...no changes needed to "include/crypto_compat.hrl". Skipping writing new file
==> boss (pre_compile)
==> boss (compile)

Ran into the same issue on Windows 7. However when running cmd as administrator it worked.

Thanks @zizorz, I will try that.


[dask] hold ports until training

Contributes to #5865 by holding on to the ports that LightGBM will use during training for as long as possible on the Python side, thus decreasing the chance of a race condition (but not eliminating it).

Seems like the actor approach won't work here due to https://github.com/dask/distributed/issues/7842. Feel free to try the alternative here @jameslamb.

Based on https://github.com/microsoft/LightGBM/pull/5890#discussion_r1209590377, I'm feeling pretty optimistic about this approach! Awesome work.
Some time this week, I'll try to test this on a multi-machine cluster using Coiled or dask-cloudprovider.

Seems like coiled doesn't provide free credits to use their infra anymore, so I haven't been able to test this on a real cluster. I'll try to find an alternative because I don't have an AWS account.

I can test on my AWS account.

Looks great to me!
You should be the one to click the button, @jmoralez 😁

Hey only saw this now, but thanks for looking into it !


500 error on contribution

When building the portal this one gives 500:
https://api.catalogueoflife.org/dataset/3/contribution

also the settings problem with bad keys. Fixed


I can't use items of array after mutate split

Hello!
My usual case is receiving tab-separated line, split, and manipulate items. But in bitfan I can't use items of array after mutate split.
For example:
config:
input {
    stdin {}
}
filter {
    mutate {
        split => {"message" => " "}
    }
    mutate {
        add_field => {
            "one" => "%{message[0]}",
            "two" => "%{message[1]}",
            "three" => "%{message.2}",
            "four" => "%{[message][3]}"
        }
    }
}
output {
  stdout {
     codec => rubydebug { }
  }
}

output:
$ bitfan run
1 2 3 4
{
  "@timestamp": 2017-12-14 13:46:34 Local,
  "tags":       []string{},
  "one":        "[1 2 3 4]",
  "two":        "",
  "three":      "",
  "four":       "",
  "host":       "vm-bitfan1",
  "message":    []string{
    "1",
    "2",
    "3",
    "4",
  },
}

Probably it is specific of "github.com/clbanning/mxj", this library supports only maps but not slices.

I think this is related to a bug on the Split func
Which split and set []string instead of []interface
Mxj knows how to handle []interface

Will fix this

fixed

Todo : fix tests related to this fix.

Thank you very much!


updated readme about node v7 harmong_proxies flag issue

#366

Coverage remained the same at 95.283% when pulling 23c71e9f972bf46a779110dd6e2ae872204b3ce0 on arshdkhn1:readme-update into 565a482d1a1b6ea495593ebb5749eaa8b9be1466 on coryhouse:master.

@coryhouse Do you agree to say that it is not supported? If so, I believe we should also update the nodeVersionCheck.js

I'd suggest just mentioning changing the test script if using Node 7. It's otherwise supported, right?

Agreed. I think the line in the PR makes it sound completely unsupported where only that flag within that script is unsupported.
It may make sense to put a warning in the nodeVersionCheck and not an error to help inform users of the unsupported flag.

Closing since handled via #368


Try to Scale Master node ,but fail

Hi all  ,
I just try to scale the  master node into existed cluster ,
my ansible deploy already can ssh access in new node without password , and
i use "ansible-playbook -i inventory/mycluster/hosts.ini scale.yml" but i got the error message as below ,
fatal: [localhost]: FAILED! => {"msg": "The task includes an option with an undefined variable. The error was: 'dict object' has no attribute 'bastion'\n\nThe error appears to have been in '/kubespray/roles/bastion-ssh-config/tasks/main.yml': line 2, column 3, but may\nbe elsewhere in the file depending on the exact syntax problem.\n\nThe offending line appears to be:\n\n---\n- set_fact:\n  ^ here\n"}
i have no ideal about this , would someone had related experience ? appreciated ~

@ant31 could this be to your playbook rewrite ?

Thanks your reply , but i did't rewrite scale.yaml

Just a update , i just try to make the "hosts" this section in the scale.yaml . it looks will not stuck in [localhost ] this step , but i dont know any different ,thxs

Another update , dont know any related about i disable the "hosts: localhost" this section  , i can run without any error with ansible-play  scale.yaml , but kubectl get nodes , can't see the master node , but if i add worker node , it looks works .

@foxyo0916 that was an error from my side. I did not update correctly the scale.yml.
I've pushed the change in master https://github.com/kubernetes-incubator/kubespray/pull/3511

@ant31  So appreciate your quickly fixed and response , have nice day ~


Variable Explorer - Exclude Class Constructors

Remove constructors from variable explorer list.
Ex. GaussianNB should not be displayed

Validated


Add support for AIX360 explanations

What this PR does / why we need it:
Adds a new Explainer framework called Artificial Intelligence eXplainability 360 (AIX360) from the Trusted AI project. This PR contains the first explainer using a package called LIME (local interpretable model-agnostic explanations). A sample has been created which gives explanations for a random forest model's predictions on the MNIST dataset.
Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes #1093
Special notes for your reviewer:

Please confirm that if this PR changes any image versions, then that's the sole change this PR makes.

Release note:

Thanks for the awesome contribution!
Can we add a e2e test like alibi explainer here ? https://github.com/kubeflow/kfserving/blob/master/test/e2e/explainer/test_tabular_explainer.py

@drewbutlerbb4 @Tomcli My v1beta1 controller implementation PR is merged and caused the merge conflict, so v1alpha2 controller is now deprecated and only v1beta1 controller will be running. All the v1alpha2 CRDs now calls conversion webhook which converts v1alpha2 to v1beta1 and then reconciles with v1beta1 controller, for v1beta1 CRD it directly goes to the v1beta1 controller reconciliation loop. The question is that if we still want to add AIX explainer to v1alpha2 CRD as it is going to get converted anyway and you need additional logic in here, maybe it makes more sense to go directly with v1beta1 now as that will be in the coming release, WDYT? Sorry for the interruption!

Thanks @yuzisun for the information. We will rebase with your new controller changes and create some example with the v1beta1 api. We will keep the v1alpha2 code for the moment until the SDK also supports v1beta1.

/retest

/retest

/lgtm
/approve


solarlog v1.2.2 to stable

feedback see: Forum:Tester

feedback for luxtronik1 see Forum

Automated adapter checker
ioBroker.solarlog

[ ] :heavy_exclamation_mark: [E605] No actual year found in copyright. Please add "Copyright (c) 2018-2020 Marcel Adam<EMAIL_ADDRESS>at the end of README.md
[ ] :heavy_exclamation_mark: [E701] No actual year found in LICENSE. Please add "Copyright (c) 2018-2020 Marcel Adam<EMAIL_ADDRESS>at the start of LICENSE
[ ] :eyes: [W505] setTimout found in "main.js, but no clearTimeout detected
[ ] :eyes: [W801] .npmignore not found
[ ] :eyes: [W901] .gitignore not found

ioBroker.luxtronik1

[ ] :heavy_exclamation_mark: [E605] No actual year found in copyright. Please add "Copyright (c) 2018-2020 Marcel Adam<EMAIL_ADDRESS>at the end of README.md
[ ] :heavy_exclamation_mark: [E701] No actual year found in LICENSE. Please add "Copyright (c) 2018-2020 Marcel Adam<EMAIL_ADDRESS>at the start of LICENSE
[ ] :eyes: [W505] setTimout found in "main.js, but no clearTimeout detected
[ ] :eyes: [W801] .npmignore not found
[ ] :eyes: [W901] .gitignore not found

E605 & E701 fixed @ solarlog & luxtronik1

Then please oush new version number here in this PR

Sorry, hope it's ok now:
solarlog now published as v1.2.3 on github & npm.
luxtronik1 now published as v0.2.1 on github & npm.

Once more sorry, now travis-checks fail and I don't know why.

Ich hoffe, jetzt ist alles in Ordnung.

RE-CHECK!

Automated adapter checker
ioBroker.luxtronik1

:thumbsup: No errors found

[ ] :eyes: [W505] setTimeout found in "main.js, but no clearTimeout detected
[ ] :eyes: [W801] .npmignore not found
[ ] :eyes: [W901] .gitignore not found

ioBroker.solarlog

:thumbsup: No errors found

[ ] :eyes: [W505] setTimeout found in "main.js, but no clearTimeout detected
[ ] :eyes: [W801] .npmignore not found
[ ] :eyes: [W901] .gitignore not found

Add comment "RE-CHECK!" to start check anew

Ich hoffe, jetzt ist alles in Ordnung.

Scheinbar ist da noch was

RE-CHECK!

Automated adapter checker
ioBroker.luxtronik1

:thumbsup: No errors found
ioBroker.solarlog

:thumbsup: No errors found
Add comment "RE-CHECK!" to start check anew

But now the versions should get updated, or ?!

Is on the way, as soon as I‘ll find a Minute....

RE-CHECK!

Automated adapter checker
ioBroker.luxtronik1

[ ] :heavy_exclamation_mark: [E812] file test/ found in repository, but not found in .npmignore
[ ] :heavy_exclamation_mark: [E814] file appveyor.yml found in repository, but not found in .npmignore
[ ] :heavy_exclamation_mark: [E815] file .travis.yml found in repository, but not found in .npmignore

ioBroker.solarlog

:thumbsup: No errors found
Add comment "RE-CHECK!" to start check anew

RE-CHECK!

Automated adapter checker
ioBroker.luxtronik1

:thumbsup: No errors found
ioBroker.solarlog

:thumbsup: No errors found
Add comment "RE-CHECK!" to start check anew

RE-CHECK!

Automated adapter checker
ioBroker.luxtronik1

:thumbsup: No errors found
ioBroker.solarlog

:thumbsup: No errors found
Add comment "RE-CHECK!" to start check anew

So, hope I'm ready now. Travis is green, checker has 2 thumbs up, versions updated and pushed to npm, up-to-date version number in PR.....

Thank you


Blog Section Design

The blog page can be made better lookwise.Likes option and comments addition UI can be improved.Kindly assign this issue to me.

Hi I am a GSSOC 2021 participant.
I would love to work on this issue to improve the looks of the website.
Please assign me this issue.
Thank You


Agent enhancement 4

Add execution stage

This PR is not ready for review yet (many minor things to fix, e.g., a better design of RunResult) but should be ready for a preliminary exp : )
The report from a local exp already shows Program Counter Coverage correctly:

The 0 build rate is incorrect due to current data structure of RunResult, I will fix it.

/gcbrun exp -n dg

/gcbrun exp -n dgl -ag

/gcbrun exp -n dg1 -ag

/gcbrun exp -n dg2 -ag

/gcbrun exp -n dg3 -ag

/gcbrun exp -n dg5 -ag

/gcbrun exp -n dg6 -ag

/gcbrun exp -n dg7 -ag

The report looks better now:
https://llm-exp.oss-fuzz.com/Result-reports/ofg-pr/2024-09-20-623-dg6-comparison/index.html

Hmm, Build status has some false negatives:

The report says this fuzz target failed.
But its original cloud build says the same fuzz target succeeded.

/gcbrun exp -n dg8 -ag

Report looking good:
https://llm-exp.oss-fuzz.com/Result-reports/ofg-pr/2024-09-20-623-dg8-comparison/index.html

TODO:

https://github.com/google/oss-fuzz-gen/pull/623#discussion_r1768072147
https://github.com/google/oss-fuzz-gen/pull/623#discussion_r1772385856
https://github.com/google/oss-fuzz-gen/pull/623#discussion_r1772453454

/gcbrun exp -n dg8 -ag

/gcbrun exp -n dg8 -ag

/gcbrun exp -n dg9 -ag

/gcbrun exp -n dg9 -ag

/gcbrun exp -n dg9 -ag

/gcbrun exp -n dg9 -ag

/gcbrun exp -n dg10 -ag

/gcbrun exp -n dg10 -ag

The last commit should fix the "0-coverage" in report.
I started to see more invalid fuzz targets, e.g., did not call function under test. I will create a new PR to add the previous checker into prototyper.
As for the crashes caused by semantic errors, they should be fixed in the next iteration (e.g., enhancer) in the current design.

/gcbrun exp -n dg -ag

Will do this once the exp reports confirms the '0-coverage' issue is resolved.

/gcbrun exp -n dg1 -ag

/gcbrun skip


Add One-Click Deploy Buttons

You have some great docs on how to deploy to various services, but to make it even easier I'd love to see something like a Deploy to Heroku button (one click deploy w/ form-fill in for ENV vars) https://devcenter.heroku.com/articles/heroku-button
Not sure if other cloud providers have something similar, but if so that could be great too.
[Edit: Looks like Azure has another thing, just to give devs multiple options for easy one-click deploy: https://azure.microsoft.com/en-us/blog/deploy-to-azure-button-for-azure-websites-2/ ]

Thanks! I added the Deploy to Heroku Burton on the Heroku deployment page, but clearly there’s value in having it on the homepage & readme too.
Haven’t used Azure as much but good to know they have a similar solution, I’ll try it out and put up some docs for it as well 👍

Added the Deploy to Heroku button to the homepage 🚀

I spent a few hours trying out Azure, but I can't get the app to deploy (gets stuck during the bundle install process at compiling the SASS gem, presumably because this is a slow process and hits a timeout). I've found very little docs online for troubleshooting this, so I'm going to stick with just Heroku for one-click deploys for now.


💄 Visual bug in fullscreen apps

Screenshot

Description
If app open on fullscreen mode, the top menu hide after hover off. It`s cause to unseen search field =(
Solution
As a solution, I suggest not hiding the top menu while gitimoji is open, as soundsource does for example.

Hi, sorry for the late reply. Unfortunately I cannot reproduce your issue. Which version of macOS are you using?

Closing due to inactivity

Hi, sorry for the late reply. Unfortunately I cannot reproduce your issue. Which version of macOS are you using?

macOS Monterey (v12.6.1)


support overriding the default version of grafanalib

This looks good to me. Is there any way to test that the override works? It seems like testing WORKSPACE rules can be done via a test WORKSPACE that is .bazelignore-d. See rules_docker for a solid example.

There's only one code path, since the default grafanalib is "just" the default value of the override param.  So this tests the same things a specific case with an overridden value would test.


Events not emitted from "wrapper" elements

Hi,
It seems I have run into a bit of a problem with events (click, drag etc) from "wrapper" elements not "firing"
ie for a simple element like this :

  Hi there sub !

    some sub div

interactions.get(".wrapperDiv","click")
will never "emit" anything
while
interactions.get(".innerDiv","click")
will work as expected
more extensive example of the issue here :
http://codepen.io/anon/pen/LVeVOg?editors=001
This issue is not specific to any event type, btw .
Am I missing something regarding how event handling and "injection" into interactions works ?
Thanks again for the great work, other than this one, no issues so far :)

Hi, thanks for the report.
We have this issue because React will render the element form:

  Hi there sub !

    some sub div

Into:

  Hi there sub !

    some sub div

And interactions.get is listening the events from elements by using e.target.matches(className). Therefore, the event will not emit because it's coming from the span not the wrapperDiv.
Also, it's very important to notice that Cycle-React will remove the current interactions.get() API and switch into event handlers for the next (1.0) version. So, this issue will no longer exist since Cycle-React will be using the event system from React. Details can be found at #8. I made this decision because I think it's better to have a single interactions API for both Cycle-React components and other React components that use event handlers.

Thanks a lot for the explanation and very fast reply !
That does make the issue clearer !
Does that mean that in the future , events will either need to use refs or getEventSubject (with the "emit" variant of #8 ?)
The main advantage I found with selectors (and to an extent, refs) vs inline handlers, is the more concise way of handling multiple events for the same element:
ie, dragover/drop etc or  touch/mouseDown combos etc

Does that mean that in the future , events will either need to use refs or getEventSubject (with the "emit" variant of #8 ?)

Yes. And getEventSubject will be renamed to get, the current get will be removed. Refs are not needed for using events. And I think you'll lose the support of React's event system by using the element's event handlers through refs.
Selector API creates a coupling between View and Intentions by using className and selector. On the other hand, the event handler API also has the same coupling by matching the name of event handlers. They have different concepts, but I think event handler API works as well as selector API in terms of coding.

Btw are there any other similar "gotchas/changes" planned for v1.0 ?

Interactions API will change #8
Virtual-hyperscript helper will be removed because we've already got JSX #11

That's probably everything for 1.0. And 1.0 will be our stable version :)

Ok great, thanks !

And I think you'll lose the support of React's event system by using the element's event handlers through refs

I mostly use refs for external libraries like three.js which can only work by "injecting" special canvas elements in the dom, so not too much worries with events there so far :)

They have different concepts, but I think event handler API works as well as selector API in terms of coding

yeah you cannot escape some amount of coupling, so I guess it is a matter of taste in the end , I am currently doing back & forths between the React style event handlers and Cycle's selector, not too sure about how either will scale.

That's probably everything for 1.0. And 1.0 will be our stable version :)
Great ! :+1:  Thanks a lot for the infos :)

Fixed and tested
Migration guide can be found at the changelog


Added default order styling back to OrderReceiptStyle.ss for emails

Tested with both SmtpMailer and EmogrifiedSmtpMailer classes of Silverstripe Email Helpers module.  Thanks.

@markguinn Has this been merged into 2.0 as well? Emails in 2.0 look really bleak right now.

I don't remember if I did merge it or not. I may have merged it locally and forgot to push.

That's exactly what happened. Fixed now.

Thanks a lot!


Squamous cells

Hi HLCA team,
Thank you for an amazing resource!
We were a little surprised not to see squamous cells in the final annotation.  Is there a reason they aren't there?
Many thanks
Jack

Hi Jack, apologies for the late reply! Currently on parental leave so not the best at checking GitHub issues atm. I discussed with the people who helped us annotate, and they said we left those annotations out because there wasn't a clear consensus yet on their definition. They were annotated in a couple of datasets originally though, you can find those under original_ann_level_3 (you might have noticed those already). Hope that helps!

This is really helpful - thank you!


Add TSC members as maintainers

Description
Add TSC members as MAINTAINERS - as discussed & approved in Oct 16 meeting
Issues Resolved
N/A
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.


Binary name needs to be picked up from package.json's "bin" key

Hi again @HashNuke
If you try installing live-server with npm install -g live-server you should get live-server.js in shims, passing "npm/bin/live-server.js" to nodejs plugin.
The actual binary's name is live-server in .npm/bin
These names can be picked out from the package's package.json under the "bin" key. For live-server it's:
"bin": {
	"live-server": "./live-server.js"
},

so the script's name will be "live-server". I'm not sure if you can add this reading of package.json as an additional test to determine $executable_name ?
I'm doing this: $(basename ${bin_env_parts[1]} .js) to truncate the .js extension when setting executable_name - generally, they're not installed with extension... but obviously this is not a proper solution. Just suggesting a "maybe good enough temporary fix" for the time being.
Cheers

@justin-calleja thank you for reporting this.
We don't read package.json because reading json in Bash... :)
So we parse the $bin_env. Please do send a PR with the fix you have. The temporary fix is actually our required fix.

Done :smile:


Update MAX_TX_FEE to 1 DCR, which aligns with current max txsize 100Kb

This fixes an error we encountered this morning, stemming from allowing tx sizes well above the max allowed tx fee.  Now these are roughly equal with 100Kb tx size enforcement.
Tested with 2of3 and 1of1 testnet transactions.

OK


Change White to Yellow to improve readability on light themes

Using sail on light theme I have noticed that there are a few messages that written in white (Docker is not running, Sail is not running and so on). Said messages are barely visible on light backgrounds (see screenshots below).
Yellow color looks better on both light and dark theme.
Before and After for dark and light theme

It's not better. I think

It's not better. I think

how so? white on light-gray or even on white background vs yellow on light.
It's still not perfect on light but still better than white.

@andreich1980, maybe append background?

No plans to change this atm.


chore: hide settings for mobile

Summary
This PR should hide all settings not needed for the current mobile version.
Changelog
- Hides NetworkStatus
- Hides CrashReport

Relevant Issues
Closes #3717
Testing
Platforms

Desktop

[ ] MacOS
[ ] Linux
[ ] Windows

Mobile

[x] iOS
[x] Android

Instructions

Please describe the specific instructions, configurations, and/or test cases necessary to test and verify that your changes work as intended.

...
Checklist

Please tick all of the following boxes that are relevant to your changes.

[x] I have followed the contribution guidelines for this project
[x] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have added or modified tests that prove my changes work as intended
[ ] I have verified that new and existing unit tests pass locally with my changes
[ ] I have verified that my latest changes pass CI workflows for testing and linting
[ ] I have made corresponding changes to the documentation

Working nice! only I have to remove fromAdvanced the option Migrate Ledger index.

Yeah, I was wondering about that as well 😄  seems like a good idea 👍


Databases: custom, private

Renamed this, removed individual molecules out of this task into a separate task

Notes:
Database management
Groups should own databases (to discuss: are we all in agreement?)
* Users can't create groups by themselves - will this be a problem? Has anyone actually contacted us to create a new group?
Names should be unique within a group, and not conflict with the official databases
Database versions
Should we support them? There are significant difficulties in the UX if someone wants to update a database that has already had some datasets processed against.
It may be more intuitive to just not deal with versions, and advise users to suffix -v1, -v2 to the name. We could add a deprecation flag so that they can hide old versions.
Database visibility & dataset visibility
If a user wants a database private, but use it on a public dataset, which behavior do we want?

(current behavior, i.e. least work) to show the private database annotations. Adding a warning when the database is uploaded might be enough
to require the dataset to be set to private as soon as the database is selected
to implement filtering annotations that use private databases

Technical

To minimize the changes that would be required to operate on molDB IDs everywhere, instead we'll keep using the name throughout the engine/graphql, format user DBs' names uniquely (e.g. [guid]Database Name), and split the name into name and displayName in graphql.
Creation of molecule images
moldb to validate and return array of [lineno?, error/warning] items for display to user, OR if import takes too long, email the user a report of the errors. All errors/warnings should prevent import - we don't want users cluttering the databases list with incomplete databases that were only partially imported due to errors.

Decisions:

Databases should be able to be private, should not show up in public results.
Databases are owned by groups.

Users have no ability to "publish" it to the wider METASPACE community, but add a note saying to get in touch with us if they want to do so (and we can manually make it a non-group-attached database)
All members in a group can upload a DB

Allow users to upload databases with the same name to overwrite old versions, with warning that it will cause old datasets to be inconsistent with new datasets unless manually reprocessed. Recommend using a different name to avoid this.
For FDR calculation, if a database is less than ~500 molecules, add the option & a recommendation for the database to be merged with another larger database (HMDB, etc.) on upload

TODO: Experiment with database size to see what threshold would be best

If database is smaller than 1000 molecules, all results will be stored regardless of FDR. Ensure that this is made very clear during database upload.


KubernetesJobTask "No pod scheduled" error

I often see the following error when using KubernetesJobTask:
[28:luigi-interface:224@18:33] [ERROR] [pid 28] Worker Worker(salt=476402868, workers=20, host=something-host-1541289600-zntd7, username=root, pid=1) failed    Something(local_execution=Falser)
Traceback (most recent call last):
  File "/opt/conda/lib/python3.6/site-packages/luigi/worker.py", line 205, in run
    new_deps = self._run_get_new_deps()
  File "/opt/conda/lib/python3.6/site-packages/luigi/worker.py", line 142, in _run_get_new_deps
    task_gen = self.task.run()
  File "/usr/src/app/phpipeline/luigi_util.py", line 231, in run
    super(PHKubernetesJobTask, self).run()
  File "/opt/conda/lib/python3.6/site-packages/luigi/contrib/kubernetes.py", line 355, in run
    self.__track_job()
  File "/opt/conda/lib/python3.6/site-packages/luigi/contrib/kubernetes.py", line 199, in __track_job
    while not self.__verify_job_has_started():
  File "/opt/conda/lib/python3.6/site-packages/luigi/contrib/kubernetes.py", line 261, in __verify_job_has_started
    assert len(pods) > 0, "No pod scheduled by " + self.uu_name
AssertionError: No pod scheduled by something-20181104183346-b51d371a5bfd4197
[1:luigi-interface:570@18:33] [INFO] Informed scheduler that task   Something_42b6a6d55a   has status   FAILED

It is hard to reproduce, but seems like sometimes pod needs a bit more time to be created, but task does not wait for it to be created and ends up raising the error.
When the task is restarted and not failed, a pod is run under Luigi control, but there's still that uncontrolled pod, so we end up having two pods making the same thing.
I've managed to fix it the most naive way. When getting pods to verify that they are started, instead of:
pods = self.__get_pods()

I do the following:
for _ in range(3):
    pods = self._KubernetesJobTask__get_pods()

    if pods:
        break

    # If pod was not returned, sleep to wait for pod to be created.
    sleep(15)

This is not a beautiful way of fixing the issue, but it works (though it will for sure fail if pod needs >45 seconds to be started).

I have same issues.

@keisuke-umezawa I've managed to fix it with the code above. Just create a custom KubernetesJobTask class with an overridden __verify_job_has_started method.

@StasDeep Thanks! Do you have example how to override __verify_job_has_started?

Yep. Something like that:
class MyKubernetesJobTask(KubernetesJobTask):

    def _KubernetesJobTask__verify_job_has_started(self):
        """
        Does the same thing as the base method, but tries to get pods three times instead of one.
        This way, it's less likely to get empty list of pods and get an exception.
        Sleeps between attempts for 15 seconds.
        """
        # Verify that the job started
        self._KubernetesJobTask__get_job()

        # Verify that the pod started
        for _ in range(3):
            pods = self._KubernetesJobTask__get_pods()

            if pods:
                break

            # If pod was not returned, sleep to wait for pod to be created.
            sleep(15)

Note that __verify_job_has_started, __get_job etc. methods have two underscores in the beginning, so don't forget to use _KubernetesJobTask prefix when invoking them.

It seems reasonable to me to allow there to be some amount of wait time and attempts for a POD to become available. Of course, it should be configurable. Happy to review a PR!


Differ wrongly highlights BigDecimals in a hash where another element doesn't match.

Hi!
I've encountered an issue with the differ that feels wrong.
Let's take this class as an example...
require 'bigdecimal'

class HashGenerator
  def self.generate
    {
      hello: 'world',
      decimal1: BigDecimal.new("10"),
      decimal2: BigDecimal.new("29.32")
    }
  end
end

Simple enough, let's test that our hash has the right values.
require 'hash_generator'
require 'bigdecimal'

RSpec.describe HashGenerator, '#self.generate' do
  it 'returns the expected hash' do
    expected_hash = {
      hello: 'world',
      decimal1: BigDecimal.new("10"),
      decimal2: BigDecimal.new("29.32")
    }
    expect(HashGenerator.generate).to eq(expected_hash)
  end
end

Green - all good!
Now let's change the expectation by altering the hello string, note that the bigdecimals are unchanged...
require 'hash_generator'
require 'bigdecimal'

RSpec.describe HashGenerator, '#self.generate' do
  it 'returns the expected hash' do
    expected_hash = {
      hello: 'SOME DIFFERENT STRING',
      decimal1: BigDecimal.new("10"),
      decimal2: BigDecimal.new("29.32")
    }
    expect(HashGenerator.generate).to eq(expected_hash)
  end
end

The test fails as expected - but the diff also highlights the bigdecimals as a problem. I understand this is because the BigDecimal inspect contains the object id and the differ probably looks at the stringified version of the hash when it's doing its thing?
Anyway, a more complex permutation of this has led me down a wild goose chase yesterday and I thought I'd raise it here. I'm happy to work on a solution if there is consensus that this is undesired behaviour.
Thanks,
Tom

RSpec performs a textual diff using diff-lcs.  In this case, the textual representation of the two BigDecimal objects is different, even though they are equal.  I agree it's an undesirable behavior but as long as we perform textual diffs there's not much we can do about it.
I think that to fix this we'd probably have to write our own differ from scratch, which is not something we're willing to take on and maintain (nor do we have the time to do so).

Closing the issue because of inactivity. I think at the moment you can deal with this with custom matchers. Here is an example:
# run with `ruby rspec_bigdecimal.rb`
require "bundler/inline"
require 'bigdecimal'

gemfile(true) do
  source "https://rubygems.org"
  gem "rspec"
end

require 'rspec/autorun'

RSpec::Matchers.define :a_big_decimal_of do |expected|
  match do |actual|
    BigDecimal(actual.to_s) == BigDecimal(expected.to_s)
  end
end

RSpec::Matchers.define :a_big_decimal_close_to do |expected|
  match do |actual|
    BigDecimal(actual.to_s).round(3) == BigDecimal(expected.to_s).round(3)
  end
end

RSpec.describe "BigDecimal" do
  it "validate big decimales" do
    hash_with_bigdecimal = {
      hello: "world",
      decimal: BigDecimal("10"),
      decimal_rounded: BigDecimal("10.123456789")
    }

    expect(hash_with_bigdecimal).to include({
      decimal: a_big_decimal_of("10"),
      decimal_rounded: a_big_decimal_close_to("10.123")
    })
  end
end

If wanted we can maybe add a matcher.

Where did we get to with your differ @benoittgt? That would solve this right?

If you speak about https://github.com/rspec/rspec-support/issues/365
I see it as more dedicated to complicated structure like hash, very long string, deeply nested array. But this case is something the new differ should manage.


[WIP] @alloy => Makes things more ReactiveCocoa-y

Initial work on RAC'ifying things. I don't think it even compiles now, but wanted to get something in before EOD.
Fixes #297.

Ha, yeah. Compiler is segfaulting.

Seems like it's crashing on the newly added file, KeypadViewModel.swift.
Global is external, but doesn't have external or weak linkage!
i8* ()* @_TFFC5Kiosk15KeypadViewModelg15addDigitCommandCSo10RACCommandaL_4selfS0_
invalid linkage type for function declaration
i8* ()* @_TFFC5Kiosk15KeypadViewModelg15addDigitCommandCSo10RACCommandaL_4selfS0_
LLVM ERROR: Broken module found, compilation aborted!

Looks similar to this problem involving accessing super from a  property override.

Fun times :)

Realized there was a problem with this as it was – phone numbers and PINs/bidder numbers that started with zero would have that prefix chopped off. Adding tests now.

This is mergeable. Same thing as in eigen – merging on reds until Travis gets their house in order, or until we have time to replace them.

Hmm. Probably could use waitUntil syntax. I'll throw it in tomorrow, maybe in another PR.

The CI is green right now, so I’ll merge this and you can do the ‘wait’ stuff in another PR.

Changes made in: https://github.com/artsy/eidolon/pull/392


Unable to push origin react-step-1

I faced below error while trying to push local code
remote: Permission to carbon-design-system/carbon-tutorial.git denied to moashraf100.
fatal: unable to access 'https://github.com/carbon-design-system/carbon-tutorial.git/': The requested URL returned error: 403

I have received the same error.

Make sure you have added your fork as the origin branch and you are pushing to your fork. Based on that error, it seems like you are trying to push to the upstream carbon-design-system/carbon-tutorial remote.
If you type git remote -v, you should see two remotes.
https://www.carbondesignsystem.com/tutorial/react/step-1#add-upstream-remote
Hope this helps!

I encountered an error as below, anyone can help on it? thanks
c:\git_repos\carbon\carbon-tutorial>git push
ERROR: Permission to carbon-design-system/carbon-tutorial.git denied to evergone.
fatal: Could not read from remote repository.
c:\git_repos\carbon\carbon-tutorial>git remote -v
origin  git@github.com:evergone/carbon-tutorial.git (fetch)
origin  git@github.com:evergone/carbon-tutorial.git (push)
upstream        git@github.com:carbon-design-system/carbon-tutorial.git (fetch)
upstream        git@github.com:carbon-design-system/carbon-tutorial.git (push)

I encountered an error as below, anyone can help on it? thanks
c:\git_repos\carbon\carbon-tutorial>git push
ERROR: Permission to carbon-design-system/carbon-tutorial.git denied to evergone.
fatal: Could not read from remote repository.
c:\git_repos\carbon\carbon-tutorial>git remote -v
origin  git@github.com:evergone/carbon-tutorial.git (fetch)
origin  git@github.com:evergone/carbon-tutorial.git (push)
upstream        git@github.com:carbon-design-system/carbon-tutorial.git (fetch)
upstream        git@github.com:carbon-design-system/carbon-tutorial.git (push)

I also received same error. above  solution is not working for me


Test benchmark in benign conditions

This tests that benchmarking works as expected when there is no meaningful change to the underlying code.

checks out, closing.


Fixed minor issues found with TopQuadrant validator

Also updated scala dependencies and sbt-nexus
Fixes #163
Fixes #162
Fixes #160

Fixed with v1


replace strip by trim

I am getting the error:
** (ArgumentError) argument error
(elixir) lib/string.ex:808: String.strip/2
[String] The confusing String.strip/2, String.lstrip/2 and String.rstrip/2 API has been soft deprecated in favor of String.trim/2, String.trim_leading/2 and String.trim_trailing/2
https://github.com/elixir-lang/elixir/blob/v1.3/CHANGELOG.md#3-soft-deprecations-no-warnings-emitted

I was using string for my separator_char. Ignore this PR

Thanks for this hint, will have a look at this.


Add finddebt command

Resolves #58

Codecov Report
Base: 72.06% // Head: 71.85% // Decreases project coverage by -0.21% :warning:

Coverage data is based on head (c37a852) compared to base (1825c8a).
Patch coverage: 60.00% of modified lines in pull request are covered.

Additional details and impacted files
@@             Coverage Diff              @@
##             master      #78      +/-   ##
============================================
- Coverage     72.06%   71.85%   -0.22%
- Complexity      439      451      +12
============================================
  Files            77       80       +3
  Lines          1364     1389      +25
  Branches        141      144       +3
============================================
+ Hits            983      998      +15
- Misses          343      351       +8
- Partials         38       40       +2

Impacted Files
Coverage Δ

.../java/paymelah/logic/parser/AddressBookParser.java
84.21% <0.00%> (-4.68%)
:arrow_down:

...a/paymelah/logic/parser/FindDebtCommandParser.java
0.00% <0.00%> (ø)

...h/model/person/DebtsContainsKeywordsPredicate.java
75.00% <75.00%> (ø)

.../java/paymelah/logic/commands/FindDebtCommand.java
100.00% <100.00%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Provide iOS side load file

Provide iOS side load file for EU users

It is now released in github!


Implementing partial members is not optional

Type of issue
Typo
Description
From https://learn.microsoft.com/en-us/dotnet/csharp/language-reference/keywords/partial-member

Partial members enable class designers to provide member hooks, similar to event handlers, that developers can decide to implement or not.

I believe that this second comma after handlers is mistake. By its means phrase is read as if developer can opt-in to implementing partial members which is not possible.
Otherwise error CS9248 Partial property '' must have an implementation is inevitable.

Page URL
https://learn.microsoft.com/en-us/dotnet/csharp/language-reference/keywords/partial-member
Content source URL
https://github.com/dotnet/docs/blob/main/docs/csharp/language-reference/keywords/partial-member.md
Document Version Independent Id
85bf7eb0-b476-94b6-e10a-d0f42fee407b
Article author
@BillWagner
Metadata

ID: b4eedbc9-36be-6c79-fcb4-d5fe6998e5b1
Service: dotnet-csharp
Sub-service: lang-reference

Maybe I little got it. There is phrase

Partial types and members provide a way for human developers to write part of a type while tools write other parts of the type.

That would imply, regardless issued error, during building code generation would fix it. So after build no errors presented.
On other hand this is in contrary to

If the developer doesn't supply an optional implementing declaration, the compiler can remove the declaring declaration at compile time.

Usually error message indicates unbuildable solution and at least with my set:

VS 17.12.0 Preview 1.0
.NET 9.0.100-preview.7.24407.12

these are not ignored by build which fails in turn.

So these statements creates impossible contrast, FMPOV.


Improvements to filters

I started out to handle #327 but I found a little TODO we forgot about when switching video streams and applying filters.
I will also fix that, however, it will take a few days to get some spare time.

I think I have fixed that bug (apply video filters on stream switch) on a different branch. Let me check...

It is fixed on auto-ref branch. Not sure which changeset it was. Maybe it is easiest to implement this feature on auto-ref?

Moved to auto-ref branch.


menupage_panel does not include PrivacyModalPanel

Privacy panel missing in settings tab for MenuPage and subclasses.
menupage_settings_panels is defined in https://github.com/rkhleics/wagtailmenus/blob/master/wagtailmenus/panels.py#L100-L107 and clearly it does not have PrivacyModalPanel.
As far as I can see PrivacyModalPanel is added to wagtail in v2.10:
https://github.com/wagtail/wagtail/commit/84f3265b7608f177122b3fc5ca9f4892601fc436
For the reference, missing panel is marked on the image:

Though PrivacyPanel is no longer added directly to settings_panels since this commit https://github.com/wagtail/wagtail/commit/0693eed2454240b4d0f31da5166772bdd106dbb9#diff-b15877f11b1b7e35b3dce47e57f8aff6dccac9a02f3f94caf860a5e51fd48360L1072, I am raising the same underlying issue here as it affects other panel contents.


Trade mark still showing 2022

The trade mark of the website still showing the previous year.
We have to update that.

I would like to work on this project. Please assign it to me.

I can work on it. Please assign it to me.

I would like to work on this issue under GSSOC'23. I would appreciate if the issue is assigned to me.

But one who asked first will be first one to get assigned right?


BLAS and tflite: Is fortran and libgfortran actually required?

I was able to complete a build of meta-tensorflow-lite removing fortran entirely.
In fact, when I look inside the source code for tensorflow and lite, I only see one reference to fortran and it seems to be disabled by default.
https://github.com/tensorflow/tensorflow/blob/efbde0a3c09f8adbf5d5419e2d498c1c0e03afd3/tensorflow/lite/tools/cmake/modules/eigen.cmake#L42C1-L53C8
I'm also okay with this issue being a meta discussion issue about BLAS, Eigen, and tflite in general.  I haven't seen a convincing way to get an optimized, fast blast compiled in yocto for aarch64.  Nor do I know how to use the Arm proprietary option.  Whenever I've used a BLAS (abandoned meta scipy for example) it's run horribly slow and was better to find any alternative.
So there's a few open questions to anyone who sees this in the future:

Can we remove or make optional the FORTRAN:forcevariable = ",fortran" and libgfortran from this library?
Does Eigen on Yocto use a BLAS if these are enabled?
Does Eigen on Yocto perform better if these are enabled?
Does Eigen configured for tflite use a BLAS even if these are enabled?
Could we make tflite perform better by providing the best BLAS possible?

Thank you.
It is true that fortran was previously required.

https://github.com/tensorflow/tensorflow/blob/b0b62979a6eea72c43050bbe935b079cece8a169/tensorflow/lite/g3doc/guide/build_cmake_arm.md

However, the description has now been deleted.

https://github.com/tensorflow/tensorflow/blob/v2.18.0/tensorflow/lite/g3doc/guide/build_cmake_arm.md

Make sure that fortran is no longer required and remove it from the recipe.

According to the following information, RUY (and XNNPACK) seems to perform better than Eigen and BLAS.

https://stackoverflow.com/questions/65536783/how-does-ruy-xnnpack-and-eigen-work-in-tensorflow-lite

Also, since Fortran is not needed at this point, it will be deleted.

Removed libgfortran dependency for main and styhead branches, other branches will follow.


Add testimonial addition using issues and workflows

Fixes Issue
fixes #2302
Changes proposed
[x] - create issue template for a testimonial
[x] - create a GitHub action that triggers when a testimonial issue is created, parses it, adds it to the filesystem and commit
[x] - trigger deploy workflow when the previous workflow is completed
Check List (Check all the applicable boxes)

[x] My code follows the code style of this project.
[x] My change requires changes to the documentation.
[ ] I have updated the documentation accordingly.
[x] All new and existing tests passed.
[x] This PR does not contain plagiarized content.
[x] The title of my pull request is a short description of the requested changes.

Screenshots

Note to reviewers
I am not sure if the issue suppose to be only created or if it makes sense to delete it

Thank you for the PR. I will test it out in a branch 👍

An idea for another PR, is to check that the user that is receiving the testimonial exists in LinkFree, for example data/eddiejaoude.json, if not then comment on the issue with user not found and close the issue.


List of Deployed Contracts

As a user, while developing smart contracts, I would like to see a list of contracts deployed in my neo-express node and their respective addresses.
Right now, this is implicitly given in the "Invoke" panel/view, where the contract names + methods are listed:

However, I would like to have a summarized view of the deployed contracts, somewhere, where I could get basic info like contract address (with copy, ctrl+c, capability), some manifest attributes (like supported standards), etc. Like a simple list of contracts deployed on neo-express.
I'm not sure if I should also open an issue for neo-express or not...

Ok, maybe let me be more specific here:
Can you simply add a button, somewhere, that developers could copy the smart contract address of a deployed smart contract?
The use case is: the developer deploys a contract, doesn't copy/keep the contract address, and then opens the invoke screen to check where is the deployed contract.... but then realizes that the address is not displayed there. Then, I think it would be nice to place a button to copy the smart contract address somehow.

@gsmachado I agreee with you. That will be very useful to have.


Support spark translate function

Will re-base on upstream with most code covered there.

Merged to upstream, https://github.com/facebookincubator/velox/pull/4972. Will be included after rebase.


Apaguei, sem querer, settings.json

Type: Feature Request
I accidentally deleted settings.json.
I made the path:
"run > add configuration > settings.json"
However, I was doing a debug test, and I didn't select the local folder, and I overridden the default setting, and I don't know how to get back.

VS Code version: Code 1.73.1<PHONE_NUMBER>f055b99068d3688932416f2346dd3b, 2022-11-09T04:27:29.066Z)
OS version: Windows_NT x64 10.0.22000
Modes:
Sandboxed: No

Check the local history and see if the previous settings were there.
If not, to reset all settings back to the default, it should be enough to change the contents of the file to just {}.


IllegalStateException in AccessibilityManager.

This is the stack trace.
1639-1639/com.nispok.samples.snackbar E/AndroidRuntime﹕ FATAL EXCEPTION: main     Process: com.nispok.samples.snackbar, PID: 1639     java.lang.IllegalStateException: Accessibility off. Did you forget to check that?             at android.view.accessibility.AccessibilityManager.sendAccessibilityEvent(AccessibilityManager.java:309)             at android.view.ViewRootImpl.requestSendAccessibilityEvent(ViewRootImpl.java:6158)             at android.view.ViewGroup.requestSendAccessibilityEvent(ViewGroup.java:737)             at android.view.ViewGroup.requestSendAccessibilityEvent(ViewGroup.java:737)             at android.view.ViewGroup.requestSendAccessibilityEvent(ViewGroup.java:737)             at android.view.ViewGroup.requestSendAccessibilityEvent(ViewGroup.java:737)             at android.view.ViewGroup.requestSendAccessibilityEvent(ViewGroup.java:737)             at android.view.ViewGroup.requestSendAccessibilityEvent(ViewGroup.java:737)             at android.view.View.sendAccessibilityEventUncheckedInternal(View.java:5294)             at android.view.View.sendAccessibilityEventUnchecked(View.java:5275)             at com.nispok.snackbar.Snackbar.focusForAccessibility(Snackbar.java:838)             at com.nispok.snackbar.Snackbar.access$1700(Snackbar.java:45)             at com.nispok.snackbar.Snackbar$7.onAnimationEnd(Snackbar.java:809)             at android.view.animation.Animation$3.run(Animation.java:374)             at android.os.Handler.handleCallback(Handler.java:739)             at android.os.Handler.dispatchMessage(Handler.java:95)             at android.os.Looper.loop(Looper.java:135)             at android.app.ActivityThread.main(ActivityThread.java:5221)             at java.lang.reflect.Method.invoke(Native Method)             at java.lang.reflect.Method.invoke(Method.java:372)             at com.android.internal.os.ZygoteInit$MethodAndArgsCaller.run(ZygoteInit.java:899)             at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:694)

Could you please take a look at this?

@rohans310 which version of the lib are you using?

Hey, I'm getting this on the latest sample app in master.

I'm not getting the error, are you getting this in one device in particular?

Also getting this with Genymotion, 4.3 device emulating

making animation as false is fixing the issue.. but i need the animation also... with animation am also getting this issue

@blaze51 are you getting this error with the latest release, master branch or both?

Latest release.

Hey, sorry I was out all day. I was getting it on both kitkat and lolipop. I tested it both on emulator and actual device. Thanks for looking into it!

Am getting same issue with lolipop device. Am solution to solve this ?

I'm sorry everyone, I've had little to no time to work on this. I promise to look into it as soon as I can.

closed via #110


How to modify generated types for custom fetch client?

What are the steps to reproduce this issue?
Types are generated in a certain structure when using the fetch client instead of axios
{ data, status }, is there a way to return it as only data
example
export type userRigsListResponse = {
  data: PaginatedRigListUserListDTO;
  status: number;
};

…
…
…

What happens?
…
What were you expecting to happen?
…
Any logs, error output, etc?
…
Any other comments?
…
What versions are you using?
Please execute npx envinfo --system --npmPackages orval,zod,axios,msw,swr,@tanstack/react-query,@tanstack/vue-query,react,vue and paste the results here.

Yes see this answer for 7.2.0: https://gurubase.io/g/orval/orval-fetch-client-return-only-data
On 7.3.0 its includeHttpResponseReturnType

Thank you!


Make jasmine a devDependency instead of dependency in package.json

Also use local jasmine install instead of global for npm test

Thanks for that, I'm fairly new to working with jasmine


What is the state of  1-Step Traffic Junction

Hi,
What is the state of 1-Step Traffic Junction? Will you share the code of 1-Step Traffic Junction?

Hi,
Thanks for your interest in our work! In 1-step Traffic Junction env, since there is only one possible state, we don't actually need to formulate a state vector. (We do however need to include an agent id to distinguish the agents.)
We just made an update on the repo to add the implementation for 1-step Traffic Junction env. Please check https://github.com/mzho7212/LICA/tree/main/1_step_traffic_junction
Hope this helps.

@mzho7212
Thanks for sharing. Thanks again.


added force_destroy to dns module

force_destroy was missing from the DNS module, so I added it.

Checklist
If applicable, I acknowledge that I have:

[X] Read the contributing guide
[X] Ran terraform fmt on all modified files
[X] Regenerated the relevant README.md files using tools/tfdoc.py
[ ] Made sure all relevant tests pass:

When I execute pytest -k 'modules and dns:' tests/examples,
I am getting pytest: error: unrecognized arguments: --dist.
I am using Python 3.10.12 and pytest 8.1.1 on WSL.

When I execute pytest -k 'modules and dns:' tests/examples,
I am getting pytest: error: unrecognized arguments: --dist.
I am using Python 3.10.12 and pytest 8.1.1 on WSL.

Regarding this error. This could happen if you're missing pytest-xdist package installed, but it is in tests/requirements.txt for quite a long time.
Does installing pytest-xdist manually solve the issue for you?

but it is in tests/requirements.txt for quite a long time

It's not ;)
AFAIK you don't need unless you're passing -n to pytest

but it is in tests/requirements.txt for quite a long time

It's not ;)

Is it? ;-)

AFAIK you don't need pytest-xdist unless you're passing -n to pytest

You needed, because I've added pytest.ini entry, which depends on having pytest-xdist installed always.

You needed, because I've added pytest.ini entry, which depends on having pytest-xdist installed always.
hmmmm do we really need it?
edit: I don't much like forcing pytest behaviour on users...

We can remove that, but then users running e2e tests needs to add --dist loadgroup so tests that can't run in parallel, are executed one after another.
As running E2E is niche as of now, this can be removed, but i prefer stuff to just work by default.

Does installing pytest-xdist manually solve the issue for you?

Yes it did, alls DNS module tests passed :)


Need the ability to create a new customer

As a user
I need the ability to register my information as a customer and create an account
So that I can create wishlists, receive recommendations, place orders, and more on the eCommerce site.
Details & Assumptions:

POST http://<Our API address/users

Acceptance Criteria:
{
first_name: "John",
last_name: "Smith",
user_name: "XXX",
address: ["123 Main St, Buffalo, NY", "50 John Stree, NY"]
password: "XXX"
}
When [some event]
Then [some measurable outcome]

Added Acceptance Criteria

Just making small change to acceptance criteria to make the JSON body formatted as a code block

Adding specifics about response code and acceptance criteria

Ranking as 8 for now, as we might end up doing more error-checking/validation on the request for this endpoint.


Creation of Add Book form

The post route for the addition of the book is /books/add. The page must contain the following form fields.

category_id
title
ISBN
price
pages
description
author
publisher
inventory
Styling the page is optional.
The category_id can be multiple or single id.

Please assign me this issue . I will work on it

Sure, go ahead.

Thankyou very much.

I am interested in this issue. Can you provide a reference?

You just have to make a form that contains the fields mentioned above. Backend routes are already made for you

I have made this form

Updated it a little bit

Follow the color scheme which is in the login form

Does this work?

Change the font, and color/shape of thd button

The font again. Use the same font as in login/register

Okay. Make a PR

ok