Add | Adding Net6 support to the driver

This is the initial try to build MDS with net6

Fingers crossed this gets in soon!

Fingers crossed this gets in soon!

I will do my best to add net6 to our pipelines to see if the tests are passing fine.

LGTM

228 changed files??

Looks good to me in general. It's a complex set of changes so I'm sure we'll find runtime things that need to be fixed once we've got this merged but i think we just need to get on and track those down as we find them.

228 changed files??

that was a mistake I did while addressing some conflicts. The PR has been opened for a very long time and there were several conflicts. I had to force push a new clone to fix that issue. it is 38 files.

@JRahnama yeah. I realised. Looking forward to getting this in so work on DateOnly and TimeOnly can get done before EF Core 8.

@JRahnama yeah. I realised. Looking forward to getting this in so work on DateOnly and TimeOnly can get done before EF Core 8.

If you want to do Date and Time i'll work on SqlBatch

@Wraith2 Sure, willing to give it a try.

Yay!

Amazing!

I think a reference got missed somewhere around the functional tests. If I try to build locally I get this:
C:\Program Files\Microsoft Visual Studio\2022\Professional\MSBuild\Current\Bin\amd64\Microsoft.Common.CurrentVersion.targets(1805,5): error : Project 'E:\Programming\csharp7\SqlClient\src\Microsoft.Data.SqlClient\add-ons\AzureKeyVaultProv
ider\Microsoft.Data.SqlClient.AlwaysEncrypted.AzureKeyVaultProvider.csproj' targets 'net6.0'. It cannot be referenced by a project that targets '.NETCoreApp,Version=v3.1'. [E:\Programming\csharp7\SqlClient\src\Microsoft.Data.SqlClient\tests\FunctionalTests\Microsoft.Data.SqlClient.Tests.csproj]

I think a reference got missed somewhere around the functional tests. If I try to build locally I get this:
C:\Program Files\Microsoft Visual Studio\2022\Professional\MSBuild\Current\Bin\amd64\Microsoft.Common.CurrentVersion.targets(1805,5): error : Project 'E:\Programming\csharp7\SqlClient\src\Microsoft.Data.SqlClient\add-ons\AzureKeyVaultProv
ider\Microsoft.Data.SqlClient.AlwaysEncrypted.AzureKeyVaultProvider.csproj' targets 'net6.0'. It cannot be referenced by a project that targets '.NETCoreApp,Version=v3.1'. [E:\Programming\csharp7\SqlClient\src\Microsoft.Data.SqlClient\tests\FunctionalTests\Microsoft.Data.SqlClient.Tests.csproj]

build script i use is buildandrun.bat from above root.
@echo off

set netcore=net6.0
set config=Release

msbuild /t:clean
msbuild /p:Configuration=%config%
msbuild /t:BuildTestsNetCore /p:TargetNetCoreVersion=%netcore% /p:Configuration=%config%

call :pauseOnError  dotnet test "src\Microsoft.Data.SqlClient\tests\FunctionalTests\Microsoft.Data.SqlClient.Tests.csproj" /p:Platform="AnyCPU" /p:Configuration="%config%" /p:TestTargetOS="Windowsnetcoreapp" /p:TargetNetCoreVersion=%netcore%  --no-build -v n --filter "category!=nonnetcoreapptests&category!=failing&category!=nonwindowstests"

goto :eof

:pauseOnError
%*
if ERRORLEVEL 1 pause
goto :eof

Interesting. Can you try building the test with -p:TF=net6.0 instead of TargetNEtCoreVersion to see if that solves the issue?

Using TF works for netfx. for net6.0 it seems to mostly work but then can't find the output functional tests.
Test run for E:\Programming\csharp7\SqlClient\artifacts\Project\bin\Windows_NT\Debug.AnyCpu.FunctionalTests\net6.0\FunctionalTests.dll (.NETCoreApp,Version=v6.0)
Microsoft (R) Test Execution Command Line Tool Version 17.4.0 (x64)
Copyright (c) Microsoft Corporation.  All rights reserved.

The test source file "E:\Programming\csharp7\SqlClient\artifacts\Project\bin\Windows_NT\Debug.AnyCpu.FunctionalTests\net6.0\FunctionalTests.dll" provided was not found.
     1>Project "E:\Programming\csharp7\SqlClient\src\Microsoft.Data.SqlClient\tests\FunctionalTests\Microsoft.Data.SqlC
       lient.Tests.csproj" on node 1 (VSTest target(s)).
     1>DispatchToInnerBuildsWithVSTestTarget:
         Build continuing because "ContinueOnError" on the task "MSBuild" is set to "ErrorAndContinue".
     1>Done Building Project "E:\Programming\csharp7\SqlClient\src\Microsoft.Data.SqlClient\tests\FunctionalTests\Micro
       soft.Data.SqlClient.Tests.csproj" (VSTest target(s)) -- FAILED.

@Wraith2 I was not able to reproduce the issue you mentioned, and pipelines are working well. I saw that error while I was working on the pipeline. It was because  TargetGroup was not defined properly. Can you make a fresh clone and test  the scenario again please?


Issue with starting Tomcat

Hello,
I've contacted you by email last week about starting Tomcat server. I've tried the tips you sent me back but I still can't get Tomcat to start. Here are the last messages it sends back on the console :
 [2022-07-20 14:15:00.013] [dev-box] [WsCoreQuartzScheduler_Worker-1] INFO o.o.a.w.s.s.ResendNotificationJob@execute:29 - Quartz triggered resend notification.
[2022-07-20 14:15:00.013] [dev-box] [TariffSwitchingQuartzScheduler_Worker-1] INFO o.o.a.w.s.s.ResponseDataCleanupJob@execute:29 - Quartz triggered cleanup of response data.
[2022-07-20 14:15:00.013] [dev-box] [CoreQuartzScheduler_Worker-1] INFO o.o.c.a.t.ScheduledTaskExecutorJob@execute:28 - Quartz triggered processing of scheduled task execution.
[2022-07-20 14:15:00.016] [dev-box] [PublicLightingQuartzScheduler_Worker-2] INFO o.o.a.w.s.s.ResendNotificationJob@execute:29 - Quartz triggered resend notification.
[2022-07-20 14:15:00.021] [dev-box] [WsCoreQuartzScheduler_Worker-2] INFO o.o.a.w.s.s.ResponseDataCleanupJob@execute:29 - Quartz triggered cleanup of response data.
[2022-07-20 14:15:00.029] [dev-box] [PublicLightingQuartzScheduler_Worker-1] INFO o.o.a.w.s.s.ResponseDataCleanupJob@execute:29 - Quartz triggered cleanup of response data.
[2022-07-20 14:15:00.030] [dev-box] [DistributionAutomationQuartzScheduler_Worker-1] INFO o.o.a.w.s.s.ResendNotificationJob@execute:29 - Quartz triggered resend notification.
[2022-07-20 14:15:00.036] [dev-box] [WsCoreQuartzScheduler_Worker-2] INFO o.o.a.w.c.a.s.ResponseUrlDataCleanupJob@execute:30 - Quartz triggered cleanup of response url data.
[2022-07-20 14:15:00.039] [dev-box] [SmartMeteringQuartzScheduler_Worker-2] INFO o.o.a.w.s.a.s.ResponseUrlDataCleanupJob@execute:29 - Quartz triggered cleanup of response url data.
[2022-07-20 14:15:00.044] [dev-box] [TariffSwitchingQuartzScheduler_Worker-2] INFO o.o.a.w.s.s.ResendNotificationJob@execute:29 - Quartz triggered resend notification.
[2022-07-20 14:15:00.053] [dev-box] [SmartMeteringQuartzScheduler_Worker-1] INFO o.o.a.w.s.s.ResendNotificationJob@execute:29 - Quartz triggered resend notification.
Thank you for your help !

Hello @RhitaB ,
this logging looks OT. These are scheduled tasks that are performing some actions. There are no errors.

Hello,
What could then be the problem if these messages keep going but Tomcat isn't starting even after an hour ?

So, you do not see a logging line Server startup in [xyz] milliseconds like this example?

If Tomcat is really not able to start within the number of seconds as configured here:

an error message will be shown (I set the timeout to 6 seconds to get the error fast):

Perhaps you will need to increase the amount of logging kept in the console. Right-click on the console, select preferences and set a large number for the Console buffer size:

Hello,
No I don't see the server startup line even after setting the timeout start time to 2400 seconds.
And yes I have set the Console buffer size to 1000000.

Hi @RhitaB ,
what is the result if you start one component, like osgp-core?
You can look at the folder /var/log/osp/logs which contains a log file per component.
Maybe one of the components is not able to fully start.
Search for errors or exceptions using something like grep -i exception.


Do you provide any Examples?

I think it is not clear how to getting started. I think examples would help.

Thanks I'll check it out.

https://clickermonkey.github.io/dayspan-vuetify/example/

Code:
https://github.com/ClickerMonkey/dayspan-vuetify/blob/master/src/app.vue

first of all, this is a great work. But I still don't know how to use it.~~~

This is an example app.js:
import Vue from 'vue'
import Vuetify from 'vuetify'
import DaySpanVuetify from 'dayspan-vuetify'

import 'vuetify/dist/vuetify.min.css'
import 'material-design-icons-iconfont/dist/material-design-icons.css'
import 'dayspan-vuetify/dist/lib/dayspan-vuetify.min.css'

Vue.use(Vuetify);

Vue.use(DaySpanVuetify, {
  methods: {
    getDefaultEventColor() {
      return '#1976d2';
    }
  }
});

I'll update the README with more


docs: correct spelling of EventBridge.

EvnetBridge to EventBridge.

Fixes #
Description
How was this change tested?

Does this change impact docs?

[ ] Yes, PR includes docs updates
[ ] Yes, issue opened: #
[x] No

Release Note

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

This needs to be updated in the /preview/ docs as well, each new release is generated from /preview/.

Looks to me like it has already been addressed here:
https://github.com/aws/karpenter/pull/3189/files


git Fetch and checkout not working on git-plus for atom

Is this a question or a feature request? If so please start the title with [QUESTION] or [REQUEST].
Are you on a windows machine and do you have trouble pushing to remote repositories? Check out this thread first because it probably has the solution and answers.
Are you having problems using submodules in atom with this package? Check out this thread.
Are you running MacOs 10.12 Sierra and having issues pulling/pushing? Check out this comment.
Did you update the package and you are seeing this message Failed to load the git-plus package? Run apm install from the package's directory and restart atom.
[Description of issue here]
The "git fetch" and "git checkout" are not working after upgrading to git-plus 7.1.1 on MacOS Sierra 10.12.2.
Steps to reproduce:

When trying to run "git fetch" through Git-Plus I  do not see it fetching my repositories.  I press return and nothing happens.
The above also happens with "git checkout"
--

There were no changes in 7.1.1 that were related to both those features. Can you validate this is not happening on previous versions?

You can ignore this.  I was in the wrong directory.  Sorry for the inconvenience.
[cid:cambria_98eacbab-ee6d-4a67-a69d-bf0eeef2522e111.png]http://cambriausa.com/
[cid:foam_7a11f8ba-d255-48a7-b1ea-ba11aa3ffb34111.png]
Trevor Herrmann
Sr Network Engineer
TEL:<PHONE_NUMBER>
CEL:<PHONE_NUMBER>
FAX:<PHONE_NUMBER>
805 Enterprise Drive East
Belle Plaine, MN 56011
[cid:facebook18_138442cd-3319-4a53-be7a-aad6466296c3111.png]https://www.facebook.com/CambriaQuartz [cid:twitter18_8a822fe1-8e90-49d6-b447-ff0b26a156d2111.png] https://twitter.com/CambriaQuartz  [cid:instagram18_7c071608-5b80-49dd-ba3a-10e90ad693f4111.png] https://instagram.com/CambriaQuartz  [cid:pinterest18_08a804e6-3043-4a54-8105-b22e5641bdce111.png] https://www.pinterest.com/cambriaquartz  [cid:houzz18_212a07e4-c6b2-4cb0-b78f-0898a954e459111.png] http://www.houzz.com/pro/cambriaquartz/cambria
From: Akonwi Ngoh<EMAIL_ADDRESS>Reply-To: akonwi/git-plus<EMAIL_ADDRESS>Date: Thursday, January 19, 2017 at 3:35 PM
To: akonwi/git-plus<EMAIL_ADDRESS>Cc: Trevor Herrmann<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [akonwi/git-plus] git Fetch and checkout not working on git-plus for atom (#602)
There were no changes in 7.1.1 that were related to both those features. Can you validate this is not happening on previous versions?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/akonwi/git-plus/issues/602#issuecomment-273906030, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AYBDk_iEhkLZ-A4NDaMeJNZhkRR8bPsAks5rT9c5gaJpZM4LoeXn.
Click herehttps://www.mailcontrol.com/sr/MZbqvYs5QwJvpeaetUwhCQ== to report this email as spam.
CONFIDENTIALITY NOTICE: The information contained in this e-mail communication and any attached documentation may be privileged, confidential or otherwise protected from disclosure and is intended only for the use of the designated recipient(s). It is not intended for transmission to, or receipt by, any unauthorized person. The use, distribution, transmittal or re-transmittal by an unintended recipient of this communication is strictly prohibited without our express approval in writing or by e-mail. If you are not the intended recipient of this e-mail, please delete it from your system without copying it and notify the above sender so that our e-mail address may be corrected. Receipt by anyone other than the intended recipient is not a waiver of any attorney-client or work-product privilege.
This message has been scanned for malware by Websense. www.websense.com


how to install

nice! thanks for your contribution to humanity
how do i install this?

Move it to /Applications and launch once, then you should see a new widget in your notification center widget list


v4 JS: menu stuck half way if swiping in + touch

Bug Report
Ionic Info
Directly using @ionic/core version 4.0.0-alpha.10 from npm / unpkg CDN.
Chrome Mobile 67.0.3396.87 (Android 8.0.0)
Chrome desktop 67.0.3396.99 64-bit (Win10 with touchscreen)
Describe the Bug
While swiping in the menu to reveal it, if user makes a second touch on the menu, the latter becomes stuck at its current position (no longer moving with the swipe gesture). The backdrop does not respond to click / touch.
Therefore if the menu itself does not provide a mean to close it (e.g. through navigation...), we can no longer use the app. Only swiping in the end menu still works.
Steps to Reproduce
Steps to reproduce the behavior:

Go to https://beta.ionicframework.com/docs/content/api//menu-demo.html (currently uses version 4.0.0-alpha.10)
Use a finger to start a swipe gesture to open the start menu (i.e. touch down at the left edge of the app, then swipe rightwards slowly)
Using another finger, touch anywhere on menu while it is half open (looks like even touching the backdrop triggers the bug).
The menu becomes stuck, whether the first finger is released or not. Touching the backdrop overlay does not close the menu.

After injecting the alpha.10 CSS into the demo app:

Also happens with "push" and "reveal" types. But in those cases, since there is no backdrop overlay, it is still possible to access some parts of the app, but the behavior might not be totally as expected. E.g. in the case of the demo app, you can still press the "Set overlay" button. The menu closes, but the backdrop overlay appears and does not respond to touches... making the whole app stuck.
Related Code
Simple version on Plunker: https://next.plnkr.co/edit/y9Ri34xUkfvdOOwR?preview
Expected Behavior

Touching the backdrop while menu is half open should either have no effect (i.e. user can continue swiping the menu), or cancel the swipe (i.e. closes the menu). Most probably the first would be better.
Touching the menu while it is half open should either have no effect,trigger the touched element (e.g. if it is a button), or forcibly open the menu fully. And in all cases the menu should still be closable.

Additional Context
As first mentioned in https://github.com/ionic-team/ionic/issues/14722#issuecomment-404726970

Same bug with end menu.

Think this is fixed in beta.17. If not, just write here!


Declare m_displayShortNameOffset as a serializable field. Fixes #652.

With -forceserialized, the name of the function was being printed incorrectly because the short name offset was not being serialized.
Marking the offset as serializable fixes the issue.
Fixes #652.

This change is

@curtisman @digitalinfinity please review? :)

LGTM

LGTM too

I'm fixing the test failures and I'll re-target this PR for release/1.1 when I have an update.

Reopened as #682


chore: change tensorchord default host log path

related to https://github.com/tensorchord/envd/issues/1184

I'm not sure what's the best default mount path.
/var/log/tensorboard looks good but it's not created. And this path may require root permission.
/tmp always works, but putting everything there will be messy.

LGTM


Updating slate version and ignoring locales

Great package...

I have a theme I am developing and is deployed to live sites. How do I update slate (from version 0.9.1) to the latest version for this theme to take advantage of improvements?

I know I can ignore files during deployment using config.yml, but what is the best practice for ignoring locale files? If I simply ignore /locales/en.default.json from live sites, then they will not benefit from new terms that are added throughout the life of a project.

Appreciate your guidance.
Cheers.

How do I update slate (from version 0.9.1) to the latest version for this theme to take advantage of improvements?

Just to be sure: Are you referring to version 0.9.1 of the Slate theme (this repo) or a version of Slate CLI?
If you're wondering how to update your Slate project to use the latest changes from this repo, there is no automated way of doing this.  It would need to be a manual process of copying assets or updating Liquid drops.  You can see the release notes for the different changes that have been implemented.  Example here: v0.10.1.  Or the entire changelog here.
If you're wanting to update the command line tool (Slate CLI).  You can update with the following command : npm install -g @shopify/slate

If I simply ignore /locales/en.default.json from live sites, then they will not benefit from new terms that are added throughout the life of a project.

A locales file may be a good item to add to your ignore files if you feel that you or your client are going to be updating translations from the admin, /admin/themes.   By doing this, your local project is going to get out of sync with your Shopify hosted store.
There's currently no tooling in place to pull down hosted theme files in Slate, like you can with Themekit.  This is something the Slate project team is discussing, but haven't come to a decision on how to implement.  You can find an active conversation about pulling down theme files in the Themekit repo here: https://github.com/Shopify/themekit/issues/330

Hi Nathan,
If I try to update slate using the Slate CLI the version remains V0.10.0
Is v0.10.1 not released yet?
Thanks!

@driespieters I figured out what the hold up was - thanks for mentioning this.  You can get 0.10.1 release here: https://sdks.shopifycdn.com/slate/0.10.1/slate-theme.zip
This version will be reflected in running slate theme shortly.

Related PR: https://github.com/Shopify/slate/issues/130
Issue is resolved now.


Revert "[core] Remove outdated babel proposal plugins (#36795)"

This reverts #36795
Without the removed plugins, the code was not transpiled correctly for Safari 12.5, which we still support. We will update the supported browsers list with the next major release. Then, we'll be able to revisit that PR.

According to browserslist it's 0.62% globally (https://browsersl.ist/#q=ios_saf+<%3D+12.5), but we can't change support between majors.


Date

The date only has 1 spot past 9 it only shows the dirt digit.

it seems to be broken by Versa SDK side changes. I adjusted width a bit wider and it seems working on my Versa now.
Watchface will be updated in the store.

I just reinstalled it and same issue
Today is the 16th  [cid:F1987BB7-E230-4C9A-870F-FEE7FF75E83A-L0-001]
Sent from Dales iPhone
On Mar 15, 2021, at 11:13 PM, Yoshinori Ikarashi @.***> wrote:

Closed #4https://github.com/yoosee/FitBitWatchObsidian/issues/4.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/yoosee/FitBitWatchObsidian/issues/4#event-4462521662, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ATFOW4HGRXE2MI4X7V5QM7LTD3ZJBANCNFSM4Y6FGOQA.

Updated version is still under review, not yet published in Fitbit Gallery. It usually took less than a day but not sure.

Publish rejected/postponed due to some misconfiguration in build target. I'm going to fix it and re-submit. Please wait for a while...

Review finished (took 6 days since submitted) and published now. It works well on my Versa.

Looks good. Thanks
Sent from Dales iPhone
On Mar 23, 2021, at 7:16 AM, Yoshinori Ikarashi @.***> wrote:

Review finished (took 6 days since submitted) and published now. It works well on my Versa.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/yoosee/FitBitWatchObsidian/issues/4#issuecomment-804938548, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ATFOW4HDE7XNML7VXT3HJYDTFCPDNANCNFSM4Y6FGOQA.


Refines tox matrix

After #1348, we should adjust expected Tox environments. Sphinx 5.2 was missing in 2 places.
The list is mostly for reference, as the Circle CI configuration is more likely the applied test matrix.

Learned that Sphinx 4.2, 4.3, 4.4 and 4.5 are missing in the test matrix.

@agjohnson I got a bit worried of what might slip through the cracks in this or any subsequent patch releases for 1.1.x, so I'd appreciate getting this extra "safety net" included.
@humitos could you have a quick look here?


not running the tool

i give the error after installing and running the tool this cmd python -m jiraffe
error:-
Traceback (most recent call last):
File "/usr/lib/python2.7/runpy.py", line 163, in _run_module_as_main
mod_name, _Error)
File "/usr/lib/python2.7/runpy.py", line 114, in _get_module_details
return _get_module_details(pkg_main_name)
File "/usr/lib/python2.7/runpy.py", line 119, in _get_module_details
code = loader.get_code(mod_name)
File "/usr/local/lib/python2.7/dist-packages/jiraffe-2.0.3-py2.7.egg/jiraffe/main.py", line 75
print(style.GREEN("[+] Jira instance detected") + style.RESET(''), style.YELLOW("[*] Enumerating the version ...") + style.RESET(''), sep="\n")
^
SyntaxError: invalid syntax
please kindly solve my issue

Can you provide system configurations and most importantly the Python version you're using?

The tool isn't supported by Python 2.


[CHOCO] Install parameters

similar to https://github.com/marticliment/WingetUI/issues/140, but for Chocolatey.
Example:
choco install virtualbox --params "/NoDesktopShortcut /ExtensionPack"
To find out about parameters you can look at the description: https://community.chocolatey.org/packages/virtualbox#description

Since it is basically the same feeature, i will mark this as a duplicate and post the progress there


zgui failed to open modal window.

if (zgui.begin("Add Expression Window", .{.flags = .{ .no_title_bar = true, .no_resize = true } })) {
    if (zgui.button("Add Expression", .{ .w = 120, .h = 20 })) {
        const center = zgui.getMainViewport().getCenter();
        zgui.setNextWindowPos(.{
            .x = center[0],
            .y = center[1],
            .cond = .appearing,
            .pivot_x = 0.5,
            .pivot_y = 0.5,
        });
        if (zgui.beginPopupModal("Add Expression", .{})) {
            if (zgui.button("aaaa", .{.w = 100})) {
                zgui.closeCurrentPopup();
            }

            zgui.endPopup();
        }
    }
    zgui.end();
}

After clicking the button, nothing happened. openPopup isn't working either.

Hi 👋
You should call openPopup in you button click handling code and move your call to beginPopupModal outside of the button click scope. Please see Dear Imgui's demo code.
I will close this but please feel free to reply if that doesn't work for you.

It works! Thanks


[REQUEST] Chrome for Testing - a new Chrome flavor

What would you like to have implemented?
Can we make use of the new Chrome flavor - Chrome for Testing? Chrome for Testing has been created purely for browser automation and testing purposes. Here is the blog post about it https://developer.chrome.com/blog/chrome-for-testing/
Why would it be useful?
While Chrome is based on Chromium, they are still different software. It would be nice to have an option for the Chrome.

I've been looking into it. The problem is it's huge, like 200+ megs. That being said, I think I do want to somehow integrate @puppeteer/browsers and maybe give people the option. Puppeteer's team came to the conclusion that they didn't want to manage the browser, that's why chrome for testing was made. I just don't know how feasible it is inside a lambda.

Good point. I didn't know it was that big package. I have no issue updating my lambda function to use the container image rather than the predefined runtimes, this way, the image size can be up to 10GB (instead of 250MB).


[6.0] Add getAttributeDefaultValue method to Eloquent Model

This PR replaces protected getAttributeFromArray method with public getAttributeDefaultValue method.
I've found this requirement while developing PR: https://github.com/cybercog/laravel-love/pull/84
I have model which has default attribute value:
class Reaction extends Model
{
    protected $attributes = [
        'power' => 1,
    ];
}

In Reaction observer's creating method if attribute not filled - I need to set attribute value equals to default value.
public function creating(Reaction $reaction): void
{
    if ($reaction->isDirty('power') && is_null($reaction->getAttributeValue('power'))) {
        $reaction->setAttribute('power', 1);
    }
}

So I have to hardcode default value to 1 in observer instead of relying on model's default attribute value:
public function creating(Reaction $reaction): void
{
    if ($reaction->isDirty('power') && is_null($reaction->getAttributeValue('power'))) {
        $reaction->setAttribute('power', $reaction->getAttributeDefaultValue('power'));
    }
}

// Another one name I was thought about: getAttributeDefault.

Sure, I could just make getAttributeFromArray public, but it's not verbose enough (IMO).

But the new name is not correct. It not only returns the default attributes, it simply returns all which have been regularly loaded.
Unless there's a better name, I'm 👎 on the name change; it's simply not correct.
👍 on the visibility though, sounds sane too me and I've had the need for this once or twice too in the past.

I would make a constant on your class and use that as your default value in both situations.


Write documentation for the new feature and config

Need to get documentation written for all of the template changes.
Make sure that examples are created for a bunch of different engines by the experiment with ejs... etc.
This ticket should make sure that stuff is there and that the config changes are all written up nicely.

complete this after #294 #295 #296 #297


[CARBONDATA-3839]Fix rename file failed for FilterFileSystem DFS object

Why is this PR needed?
Rename file fails in HDFS when the FS object is of FilterFileSystem, (which basically can contain any filesystem to use as basic filesystem)
What changes were proposed in this PR?
While rename force, have a check for this FS object
Does this PR introduce any user interface change?

No

Is any new testcase added?

No(Tested in cluster)

LGTM

Build Success with Spark 2.3.4, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbonPRBuilder2.3/3114/

Build Success with Spark 2.4.5, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbon_PR_Builder_2.4.5/1390/

LGTM


[Frontend] Descriptions in the Trade Page

File(s): frontend/???
Test Files Required?: No
What Testing Should Include:

N/A

We've gotten several reports of people being confused on how to use the site overall. To clear this up, we must add descriptions under the "Withdraw" and "Deposit" buttons. In addition, adding the "anticipated" amount if you deposit/withdraw on receiving vault tokens or the asset token respectively.
 Withdrawing from the vault can only be done if the vault's withdrawal window has opened up after the manager has settled the vault."
Deposit: "When depositing, you will deposit the vault's asset token in redemption for vault tokens to represent your fair share of the vault. Depositing is open anytime whether the withdrawal window is closed or not."

Forgot to close
Merged w/ #40


Build Progress dropdown not updating

From AP-4222
How is the user affected? And what is the expected behavior?
When you run tests and expand the dropdown to view progress, the steps aren't updating with each stage.
I'd expect the steps to update with each stage and think this is a regression.

This issue has already been resolved (i.e. the steps actually do update) but the progress spinner icon wasn't spinning so I opened a PR to address that.


Builder: gftools builder sources/config.yaml  axisOrder error when building non-variable UFO

With the repo: https://github.com/eliheuer/rena, gftools builder sources/config.yaml with the below config.yaml fails:
sources:
  - Rena.ufo
familyName: Rena

$ gftools builder sources/config.yaml
Traceback (most recent call last):
  File "/Users/eli/Py/venvs/python3-venv-google-fonts/bin/gftools-builder.py", line 83, in <module>
    builder.build()
  File "/Users/eli/Py/venvs/python3-venv-google-fonts/lib/python3.9/site-packages/gftools/builder/__init__.py", line 178, in build
    self.build_variable()
  File "/Users/eli/Py/venvs/python3-venv-google-fonts/lib/python3.9/site-packages/gftools/builder/__init__.py", line 286, in build_variable
    self.gen_stat(ttFonts)
  File "/Users/eli/Py/venvs/python3-venv-google-fonts/lib/python3.9/site-packages/gftools/builder/__init__.py", line 346, in gen_stat
    self.config["axisOrder"] = [ax.axisTag for ax in varfonts[0]["fvar"].axes]
IndexError: list index out of range

buildVariable defaults to true. You can get around this error by manually setting it to false in your config file, but we should probably detect that a single UFO can't be variable.

Actually the proximate cause of the problem is that build_variable unconditionally calls self.gen_stat(ttFonts) even when no ttFonts were built. I'll fix both.

Setting buildVariable to True in config.yaml did work for me, thank you!


Wakka Maul: Fighting against the simple AI can also unlock other Wakka Maul achievements.

I entered the Wakka Maul ladder and clicked to play on the humans side against the Simple AI. When I clicked the "Done" button, all Wakka Maul achievements were unlocked:

Note: I've noticed the multiplayer AI achievement notifications always appear twice, this seems unrelated to this bug. In the screenshot above, you can see the each achievement is shown twice (except the Brawler which is probably cut off screen). I'll report this UI bug separately. Update: reported it here.

I tried the other multiplayer arenas to check whether this was reproducible, and good news (kinda): Harrowland unlocked the "Beat Shaman AI" achievement when I submitted against the Simple AI, the other arenas did not unlock anything past the "Beat Simple AI" as expected.

Does your human Wakka Maul strategy beat the Thoktar CPU player?

Yep, it does.
This was a bit non-intuitive though, as I was playing against the Simple AI (which I assume to be the "Simple CPU" player).

I mean, when I saw the achievement I had no idea what I did to earn it, seemed like a bug.
Are the simulations against other CPUs run in the background? Or was it computed by other users asynchronously simulating my games against the CPUs and it only synced the achievements when I submitted?

I've submitted an human Multiplayer Treasure Glove strategy that beats the Shaman CPU, but the achievement is not unlocking. I guess this is still work-in-progress, right?

Those achievements happen when you happen to play against the CPU in a simulated multiplayer match, which isn't guaranteed to happen at any particular time. So yeah, it needs some improvements...

Oh I see. I looked at the ladder's "My Matches" tab, and I have indeed beaten all of the CPUs, so this is not a bug.
I wonder if it could be made easier to fight against a specific CPU (rather than scrolling through the ladder scores table) and use the synchronous simulation for achievement completion (as it is for the Simple AI)?

Yeah, that's what we should be doing here.

I copy my comment from #1957:
I suggest that all players (no matter when they submitted their code) should be matched (at least 2-5 times) against all AI levels to have a chance for getting the achievements.
e.g. I'm in the top 20 on dueling grounds on both sides, but I don't even have the "beat the simple AI" achievement...

Changed the title to better explain the cause of the issue, now that we know more information.

@periloso I think we could further generalize the issue. Perhaps we could generalize this issue to the the UI/UX problems related to the new multiplayer CPU AI achievements:

[ ] Make it easier to pick CPU AIs to fight against.
[ ] Unlock the achievement when beating the CPU AI in the synchronous fight.

Does this sound good?
Perhaps it is better to create a new issue, so these points don't get buried deep down here.

@UltCombo We could definitely generalise the issue further, when changing the title I was trying more to reflect the current state of the thread than break the issue up into its key components.
Honestly, I'm not sure the best solution for CodeCombat, as I haven't been around for months and am only now trying to get to grips with the project again, but I would say that sounds like a great plan.  It might be best to get someone elses opinion on this topic before we proceed.

Seems like @nwinter agrees on these points, I've only broken it into a list. It would be nice to get input on how to best tackle the UI issues. Also, IIRC, @nwinter mentioned there's already a revamping of the multiplayer UI on the works.


Scrollable.ensureVisible doesn't work fine

flutter doctor -v
[✓] Flutter (Channel stable, 1.22.1, on Mac OS X 10.15.5 19F101, locale
    zh-Hans-CN)
    • Flutter version 1.22.1 at /Users/limi/Desktop/flutter
    • Framework revision f30b7f4db9 (4 days ago), 2020-10-08 10:06:30 -0700
    • Engine revision 75bef9f6c8
    • Dart version 2.10.1
    • Pub download mirror https://pub.flutter-io.cn
    • Flutter download mirror https://storage.flutter-io.cn

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

[✓] Xcode - develop for iOS and macOS (Xcode 12.0)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 12.0, Build version 12A7209
    • CocoaPods version 1.9.1

[✓] Android Studio (version 4.0)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin version 49.0.2
    • Dart plugin version 193.7547
    • Java version OpenJDK Runtime Environment (build
      1.8.0_242-release-1644-b3-6222593)

[✓] Connected device (1 available)
    • iPhone 11 (mobile) • A7DF1221-DC45-4816-871B-8B879053EA80 • ios •
      com.apple.CoreSimulator.SimRuntime.iOS-14-0 (simulator)

• No issues found!

code
import 'package:flutter/cupertino.dart';
import 'package:flutter/material.dart';

void main() {
  runApp(MyApp());
}

class MyApp extends StatelessWidget {
  @override
  Widget build(BuildContext context) {
    return CupertinoApp(
      title: 'Flutter Demo',
      home: MyHomePage(title: 'Flutter Demo Home Page'),
    );
  }
}

class MyHomePage extends StatefulWidget {
  MyHomePage({Key key, this.title}) : super(key: key);

  final String title;

  @override
  _MyHomePageState createState() => _MyHomePageState();
}

class _MyHomePageState extends State<MyHomePage> {

  @override
  Widget build(BuildContext context) {
    return CupertinoPageScaffold(
      navigationBar: CupertinoNavigationBar(middle: Text("Demo"), backgroundColor: Colors.blue,),
      child: Container(
        color: Colors.white,
        child: Column(
          children: [
            Expanded(
              child: PageView(
                children: List.generate(5, (index) => _Page(index)),
              ),
            )
          ],
        ),
      ),
    );
  }
}

class _Page extends StatefulWidget {
  final int index;
  _Page(this.index);

  @override
  State<StatefulWidget> createState() {
    return _PageState();
  }
}

class _PageState extends State<_Page> {

  @override
  Widget build(BuildContext context) {
    return Row(
      children: [
        Container(
          width: 100,
          color: Colors.grey[200],
        ),
        Expanded(
          child: Column(
            children: [
              Container(
                color: Colors.red,
                width: double.infinity,
                height: 50,
                child: Menu(),
              ),
              Expanded(
                child: Center(
                  child: Text('${widget.index}'),
                ),
              )
            ],
          ),
        )
      ],
    );
  }
}

class Menu extends StatefulWidget {

  @override
  State<StatefulWidget> createState() {
    return _MenuState();
  }
}

class _MenuState extends State<Menu> {

  Map<int, GlobalKey> keys = {};

  @override
  Widget build(BuildContext context) {
    return SingleChildScrollView(
      scrollDirection: Axis.horizontal,
      child: Row(
        children: List.generate(10, (index) {
          var key = keys[index];
          if(key == null){
            key = GlobalKey();
            keys[index] = key;
          }
          return GestureDetector(
            key: key,
            onTap: () {
              Scrollable.ensureVisible(keys[index].currentContext, alignment: 0.5, duration: Duration(milliseconds: 200));
            },
            child: Container(
              color: Colors.grey,
              margin: EdgeInsets.only(left: 10, right: 10),
              padding: EdgeInsets.fromLTRB(10, 8, 10, 8),
              child: Text('item $index'),),
          );
        }),
      ),
    );
  }
}

Reproduce steps:

Click the item3 in menu, you will see the result.

Expected results:
Only the Menu scroll.
Actual results:
The PageView and Menu scroll together.
if you remove the follow code in _PageState, it work fine.
  Container(
    width: 100,
    color: Colors.grey[200],
  ),

@xu-baolin since you reopened this issue, I see on latest master and latest stable that menu and pageview scroll together as mentioned in original post.
https://user-images.githubusercontent.com/67046386/103619539-f1a7ff00-4f57-11eb-928a-8120d06e2ed4.mov
https://user-images.githubusercontent.com/67046386/103619556-f79de000-4f57-11eb-98ee-ae83b00b3b4a.mov

flutter doctor

[✓] Flutter (Channel master, 1.26.0-2.0.pre.177, on Mac OS X 10.15.4 19E2269 darwin-x64, locale en-IN)
[✓] Android toolchain - develop for Android devices (Android SDK version 30.0.0)
[!] Xcode - develop for iOS and macOS (Xcode 12.0.1)
    ! CocoaPods 1.9.3 out of date (1.10.0 is recommended).
        CocoaPods is used to retrieve the iOS and macOS platform side's plugin code that responds to your plugin usage on the Dart side.
        Without CocoaPods, plugins will not work on iOS or macOS.
        For more info, see https://flutter.dev/platform-plugins
      To upgrade see https://guides.cocoapods.org/using/getting-started.html#installation for instructions.
[✓] Chrome - develop for the web
[✓] Android Studio (version 4.1)
[✓] VS Code (version 1.52.1)
[✓] Connected device (3 available)

! Doctor found issues in 1 category.

Can you please confirm if this is still an issue ?

@xu-baolin since you reopened this issue, I see on latest master and latest stable that menu and pageview scroll together as mentioned in original post.
https://user-images.githubusercontent.com/67046386/103619539-f1a7ff00-4f57-11eb-928a-8120d06e2ed4.mov
https://user-images.githubusercontent.com/67046386/103619556-f79de000-4f57-11eb-98ee-ae83b00b3b4a.mov

flutter doctor

[✓] Flutter (Channel master, 1.26.0-2.0.pre.177, on Mac OS X 10.15.4 19E2269 darwin-x64, locale en-IN)
[✓] Android toolchain - develop for Android devices (Android SDK version 30.0.0)
[!] Xcode - develop for iOS and macOS (Xcode 12.0.1)
    ! CocoaPods 1.9.3 out of date (1.10.0 is recommended).
        CocoaPods is used to retrieve the iOS and macOS platform side's plugin code that responds to your plugin usage on the Dart side.
        Without CocoaPods, plugins will not work on iOS or macOS.
        For more info, see https://flutter.dev/platform-plugins
      To upgrade see https://guides.cocoapods.org/using/getting-started.html#installation for instructions.
[✓] Chrome - develop for the web
[✓] Android Studio (version 4.1)
[✓] VS Code (version 1.52.1)
[✓] Connected device (3 available)

! Doctor found issues in 1 category.

Can you please confirm if this is still an issue ?

The issue describes as When there are other objects between the target and the pageView, calling ensureVisiblemay may cause the pageView to scroll(Looks like shaken). If the object is large enough, the pageView's page may change at that time, which is not the expected result.
The Original PR was reverted by #68036, so I reopen this again.

The issue describes as When there are other objects between the target and the pageView, calling ensureVisiblemay may cause the pageView to scroll(Looks like shaken). If the object is large enough, the pageView's page may change at that time, which is not the expected result.
The Original PR was reverted by #68036, so I reopen this again.

Thanks @xu-baolin for confirming. Issue seems to be replicable per earlier comment, https://github.com/flutter/flutter/issues/67981#issuecomment-754465781

Thanks @xu-baolin for confirming. Issue seems to be replicable per earlier comment, https://github.com/flutter/flutter/issues/67981#issuecomment-754465781

Any update on this? Still happening to me with Flutter 2.5.3

In my case, Padding widget causes the shaken page.
If someone uses padding SingleChildScrollView like me, just move Padding into SingleChildScrollView
before
return Padding(
  padding: const EdgeInsets.symmetric(horizontal: 30.0),
  child: SingleChildScrollView(
    child: ...,
  ),
);

after
return SingleChildScrollView(
  padding: const EdgeInsets.symmetric(horizontal: 30.0),
  child: ...,
);

In my case, Padding widget causes the shaken page.
If someone uses Padding, SingleChildScrollView like me, just move Padding into SingleChildScrollView
before
return Padding(
  padding: const EdgeInsets.symmetric(horizontal: 30.0),
  child: SingleChildScrollView(
    child: ...,
  ),
);

after
return SingleChildScrollView(
  padding: const EdgeInsets.symmetric(horizontal: 30.0),
  child: ...,
);

you're right, your solution solved the problem for me

I can still reproduce this issue. Updating labels to reflect current Flutter versions

Code Sample
import 'package:flutter/cupertino.dart';
import 'package:flutter/material.dart';

void main() {
  runApp(const MyApp());
}

class MyApp extends StatelessWidget {
  const MyApp({super.key});

  @override
  Widget build(BuildContext context) {
    return const CupertinoApp(
      title: 'Flutter Demo',
      home: MyHomePage(title: 'Flutter Demo Home Page'),
    );
  }
}

class MyHomePage extends StatefulWidget {
  const MyHomePage({super.key, required this.title});

  final String title;

  @override
  State<MyHomePage> createState() => _MyHomePageState();
}

class _MyHomePageState extends State<MyHomePage> {
  @override
  Widget build(BuildContext context) {
    return CupertinoPageScaffold(
      navigationBar: const CupertinoNavigationBar(
        middle: Text("Demo"),
        backgroundColor: Colors.blue,
      ),
      child: Container(
        color: Colors.white,
        child: Column(
          children: [
            Expanded(
              child: PageView(
                children: List.generate(5, (index) => _Page(index)),
              ),
            )
          ],
        ),
      ),
    );
  }
}

class _Page extends StatefulWidget {
  final int index;
  const _Page(this.index);

  @override
  State<StatefulWidget> createState() {
    return _PageState();
  }
}

class _PageState extends State<_Page> {
  @override
  Widget build(BuildContext context) {
    return Row(
      children: [
        Container(
          width: 100,
          color: Colors.grey[200],
        ),
        Expanded(
          child: Column(
            children: [
              Container(
                color: Colors.red,
                width: double.infinity,
                height: 50,
                child: const Menu(),
              ),
              Expanded(
                child: Center(
                  child: Text('${widget.index}'),
                ),
              )
            ],
          ),
        )
      ],
    );
  }
}

class Menu extends StatefulWidget {
  const Menu({super.key});

  @override
  State<StatefulWidget> createState() {
    return _MenuState();
  }
}

class _MenuState extends State<Menu> {
  Map<int, GlobalKey> keys = {};

  @override
  Widget build(BuildContext context) {
    return SingleChildScrollView(
      scrollDirection: Axis.horizontal,
      child: Row(
        children: List.generate(10, (index) {
          var key = keys[index];
          if (key == null) {
            key = GlobalKey();
            keys[index] = key;
          }
          return GestureDetector(
            key: key,
            onTap: () {
              Scrollable.ensureVisible(
                keys[index]!.currentContext!,
                alignment: 0.5,
                duration: const Duration(milliseconds: 200),
              );
            },
            child: Container(
              color: Colors.grey,
              margin: const EdgeInsets.only(left: 10, right: 10),
              padding: const EdgeInsets.fromLTRB(10, 8, 10, 8),
              child: Text('item $index'),
            ),
          );
        }),
      ),
    );
  }
}

flutter doctor -v (Master)
[!] Flutter (Channel master, 3.8.0-7.0.pre.24, on Manjaro Linux 5.15.91-1-MANJARO, locale en_US.UTF-8)
    • Flutter version 3.8.0-7.0.pre.24 on channel master at /home/exabyt3/fvm/versions/master
    ! Warning: `dart` on your path resolves to /opt/dart-sdk/bin/dart, which is not inside your current Flutter SDK checkout at /home/exabyt3/fvm/versions/master. Consider adding /home/exabyt3/fvm/versions/master/bin to the front of your path.
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 3f98c0f8f9 (67 minutes ago), 2023-02-07 23:55:24 -0800
    • Engine revision f310ffd146
    • Dart version 3.0.0 (build 3.0.0-204.0.dev)
    • DevTools version 2.21.1
    • If those were intentional, you can disregard the above warnings; however it is recommended to use "git" directly to perform update checks and upgrades.

[!] Android toolchain - develop for Android devices (Android SDK version 33.0.0)
    • Android SDK at /home/exabyt3/Android/Sdk
    • Platform android-33, build-tools 33.0.0
    • ANDROID_SDK_ROOT = /home/exabyt3/Android/Sdk
    ✗ No Java Development Kit (JDK) found; You must have the environment variable JAVA_HOME set and the java binary in your PATH. You can download the JDK from https://www.oracle.com/technetwork/java/javase/downloads/.

[✓] Chrome - develop for the web
    • CHROME_EXECUTABLE = /usr/bin/brave

[✓] Linux toolchain - develop for Linux desktop
    • clang version 15.0.7
    • cmake version 3.25.2
    • ninja version 1.11.1
    • pkg-config version 1.8.0

[!] Android Studio (version 2021.3)
    • Android Studio at /opt/android-studio
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    ✗ Unable to find bundled Java version.
    • Try updating or re-installing Android Studio.

[✓] Connected device (2 available)
    • Linux (desktop) • linux  • linux-x64      • Manjaro Linux 5.15.91-1-MANJARO
    • Chrome (web)    • chrome • web-javascript • Brave Browser <IP_ADDRESS>

[✓] HTTP Host Availability
    • All required HTTP hosts are available

! Doctor found issues in 3 categories.


Added a string based bitmap type for SVG

Added a string based bitmap type for SVG images.
Added a cache for Mapsui.Forms Pin type to save some memory, if always the same SVG or Pin color is used.

The title sounds a bit odd, but after quickly glancing over your changes, I'd say I like what you're doing 😄


Allow build on FreeBSD

These are some fixes for making contour compile on FreeBSD. Mostly
things where Linux was assumed.
Build with autogen.sh does not work because of the broken shebang (which I didn't bother fixing).
Thus build manually with cmake:
$ cmake -G'Unix Makefiles' -Bbuild .
Everything seems fine so far, except changing the font. That causes
a floating point exception. I may do some further debugging to try
and figure out what is going on there.
I don't know if the change in UnixPty.h:108 causes any trouble because of const correctness.
The prototype of openpty on FreeBSD is :
 int
 openpty(int *amaster, int *aslave, char *name, struct termios *termp,
         struct winsize *winp);

(see https://www.freebsd.org/cgi/man.cgi?query=openpty&apropos=0&sektion=0&manpath=FreeBSD+13.0-RELEASE+and+Ports&arch=default&format=html ). The compiler complains about the discarded const which causes the candidade to be disregarded.
In case the changes get merged and the floating point exception fixed, I may bring contour to the FreeBSD Ports.

Hey, just as a note if you want to fix the autogen script shebang to be more portable between *nix systems (including BSD).
There are two options IIRC (the first one being better)

You could use #!/bin/sh as the shebang (this one will find a mostly POSIX-compliant shell on any standard Unix system)
You could use #!/usr/bin/env bash as the shebang

Oh @herrhotzenplotz that is sooooo nice of you in getting Contour runnable on FreeBSD. That is actually really awesome work. Please give me some time to respectfully review your work. I'm looking forward to it.

You could use #!/bin/sh as the shebang (this one will find a mostly POSIX-compliant shell on any standard Unix system)

I think at least as far as compatibility is concerned, /bin/sh would be the way to go.
NB: I'd love to get FreeBSD build tests into the Github CI. But that doesn't have to be done as part of this PR, this may very well start with a ticket on its own. :)

Hey, just as a note if you want to fix the autogen script shebang to be more portable between *nix systems (including BSD).
There are two options IIRC (the first one being better)
1. You could use `#!/bin/sh` as the shebang (this one will find a mostly POSIX-compliant shell on any standard Unix system)

2. You could use `#!/usr/bin/env bash` as the shebang

Indeed that is the usual way to go about this problem. However, the script is already filled with bashims which I didn't want to fix. I guess for anything not GNU based we should simply ignore this script. Otherwise this must be rewritten in POSIX sh or something else that fits. Depending on bash is not really that appreciated on *BSD as you may guess :smile:

Oh ehm, @herrhotzenplotz: Would you mind sharing me your installation prerequisites for FreeBSD? I'd like to try adding this to the CI for automated build testing.

Do you agree with a merge in this state or do you have anything else planned to be pushed?

I'm looking into the floating point exceptions right now. But that is another PR. It can be merged like this now.

Oh ehm, @herrhotzenplotz: Would you mind sharing me your installation prerequisites for FreeBSD? I'd like to try adding this to the CI for automated build testing.

I will write a ports Makefile and test it in poudriere. that way I can ensure the minimal set of dependencies that make it build and work properly. I'll let you know when it works! :smiley:

After some hack and patch and so on, I managed to produce a ports Makefile. To be honest, the CMake based build system is kind of a show stopper. Especially the CPM package manager (which I have never heard of before) is something that will cause problems on several packaging systems because it tries to download dependencies in the config/build stage. This is not allowed and causes failures in e.g. poudriere which is used to build the repositories for FreeBSD.
There are also lots of link problems because the correct library paths for the link stage are missing entirely. Probably a fix of the target_link_libraries() is needed.
The current ports shar including Makefile and patches etc. (not really polished and cleaned up, but it works) is at: https://gist.githubusercontent.com/herrhotzenplotz/4d545fce05643ee845dbfb18c0ec20d9/raw/24e14447bf190c7d767a2369b5bdd3e13fc21778/x11-contour.shar
The current build dependencies of it are: pkg catch microsoft-gsl range-v3 cmake ninja ncurses pkgconf qt5-buildtools qt5-qmake libfmt fontconfig freetype2 harfbuzz yaml-cpp qt5-core qt5-gui qt5-network qt5-widgets
You can use pkg install -y <insert pkgs here> to install them on the target machine.
  Save it in a file, remove anything before
# this line, and then unpack it by entering "sh file".  Note, it may
# create directories; files and directories will be owned by you and
# have default permissions.

X endif()
206d84c61f8029eda279eddab30b5afa
echo x - ./files/patch-src_text__shaper_CMakeLists.txt
sed 's/^X//' >./files/patch-src_text__shaper_CMakeLists.txt << '8d0cb831df43ab64d2bc8805b0d33d6a'
X--- src/text_shaper/CMakeLists.txt.orig	2021-12-09 19:17:13 UTC
X+++ src/text_shaper/CMakeLists.txt
X@@ -19,9 +19,7 @@ endif()
X add_library(text_shaper STATIC ${text_shaper_SRC})
X
X set(TEXT_SHAPER_LIBS unicode::core)
X-list(APPEND TEXT_SHAPER_LIBS fmt::fmt-header-only)
X-list(APPEND TEXT_SHAPER_LIBS range-v3)
X-list(APPEND TEXT_SHAPER_LIBS GSL)
X+list(APPEND TEXT_SHAPER_LIBS fmt)
X
X if(APPLE)
X     enable_language(OBJCXX)
8d0cb831df43ab64d2bc8805b0d33d6a
echo x - ./pkg-descr
sed 's/^X//' >./pkg-descr << '0e313719ff012ceaf97a028d51caa8f5'
Xcontour is a modern terminal emulator, for everyday use. It is
Xaiming for power users with a modern feature mindset.
X
XWWW: https://github.com/contour-terminal/contour/
0e313719ff012ceaf97a028d51caa8f5
exit

@herrhotzenplotz ok, I am really so sorry for the pain of CMake/CPM. I chose CMake because it's the most widely supported build system that also includes Windows (Visual Studio), as I'd like to retain easy buildability on Windows platform, too.
So you basically got rid of CPM to make it work? I think I must get a FreeBSD VM locally and see if I can repro that and maybe we can get a fix.
Just to be dog-clear: So removing CPM was fixing the SIGFPE?

Just to be dog-clear: So removing CPM was fixing the SIGFPE?

It didn't fix the FPE.

Especially the CPM package manager (which I have never heard of before) is something that will cause problems on several packaging systems because it tries to download dependencies in the config/build stage.

To be honest, i'd prefer all deps to be sysdeps (as in: installed by the system package manager before). Except libunicode, which is a helper-project as part of Contour, which isn't as prominent as yaml-cpp or  range-v3. So libunicode could use git submodules.
The whole purpose of the configure-stage downloading of dependencies is to make it easier for the users to compile from source (and for the devs, but yeah :D).
But now, thinking about it. We already have ./scripts/install-deps.sh that is - based on your platform - installing system dependencies. We could actually Add FreeBSD support in this script (1.), and 2., we could then git clone into a ./_deps/ or alike all remaining deps that do NOT have a system package yet (like libunicode, most likely).
I hope I'm not bugging you too much @herrhotzenplotz with this, so just tell me to STFU otherwise :)
Would this script-based approach work for you (as in: FreeBSD)? I can just Google around to see if range-v3/yaml-cpp/Catch2 deps are available as Port or not.

Would this script-based approach work for you (as in: FreeBSD)? I can just Google around to see if range-v3/yaml-cpp/Catch2 deps are available as Port or not.

It wouldn't make it any better for the port, as the Makefiles of the Port declare its dependencies. (See USES, LIB_DEPENDS and RUN_DEPENDS lines in the makefile).
For CMake based builds, it is best to pretty much only depend on the PkgConfig module and let it figure out compilation and linker flags etc. For aiding developers, having a install-deps script is perfect! that would even help on CI.

Would this script-based approach work for you (as in: FreeBSD)? I can just Google around to see if range-v3/yaml-cpp/Catch2 deps are available as Port or not.

It wouldn't make it any better for the port, as the Makefiles of the Port declare its dependencies. (See USES, LIB_DEPENDS and RUN_DEPENDS lines in the makefile).

I think it makes it a little better in a sense that we could actually:

get rid of CPM all together
decouple deps-fetching from CMake

You are not mandated to invoke scripts/install-deps.sh anyways. So in a Portfile (or rpm/deb) package creator does provide the dependencies just like you said, that's perfectly fine (and actually preferred by me anyways) :)

For CMake based builds, it is best to pretty much only depend on the PkgConfig module and let it figure out compilation and linker flags etc. For aiding developers,

I will see what I can do to improve this dependency-situation. :)

having a install-deps script is perfect! that would even help on CI.

I'm using it on CI for quickly installing the deps and on my machines whenever I need to quickly fetch'n'install them. So the existence of such a script is still valuable (for convenience reasons).
Now, let me decouple this ..... zensored.

For a more realtime off-github communication, you can join our Discord, if this is of any interest to you. :)

For a more realtime off-github communication, you can join our Discord, if this is of any interest to you. :)

Sorry, I'm more of an IRC type person :neutral_face:


IntegerField has invalid lookup: in

I'm starting to work with LDAP and using this app (thanks @jlaine!) I'm developing a django site to help me manage my LDAP db. Following the examples, I've created an LdapGroup model but instead of making the cn the pk, I chose the gidNumber. All works just fine but when deleting a group. I get an  "TypeError: IntegerField has invalid lookup: in". I've found this IntegerField definition (/ldapdb/models/fields.py):
    def get_prep_lookup(self, lookup_type, value):
        "Perform preliminary non-db specific lookup checks and conversions"
        if lookup_type in ('exact', 'gte', 'lte'):
            return value
        raise TypeError("IntegerField has invalid lookup: %s" % lookup_type)

I've fixed the error simply by adding 'in' to that tuple. The question is: shouldn't be there by default? Or it has something to do with how LDAP works?

Looks like a bug ;)
This might be fixed as a side-effect of the current work on support for Django 1.10.


[Bug]:  CAN'T ADD MORE THAN ONE CHROME EXTENSION TO PUPPETEER BY --load-extension="Directory Ex"

Version of the software
OB Native 0.2.4
Operating system
Windows
Browser / Native
Chrome
What happened?
CAN'T ADD MORE THAN ONE CHROME EXTENSION TO PUPPETEER BY --load-extension="Directory Ex"
Relevant LoliCode if needed
No response

Solved
--disable-extensions-except="Directory Ex1","Directory Ex2"


Make watch task more robust

Our watch task currently depends on the gogo shell being available and enabled in a running instance of Liferay Portal. This has proven to be a bit unstable and unreliable over time.
We should try to investigate new alternatives or ways to provide this that don't necessarily depend on deploying bundles to Liferay.
One option could be to have some type of dev-server (webpack or standalone) so we could simply proxy the updated css files on request

Going to close this now that #190 and #201 are in, along with #209.


🐛 [firestore-bigquery-export] schema-views script doesn't work against multiple schema files

Step 2: Describe your configuration

Extension name: firestore-bigquery-export (schema-views script)
Extension version: 0.1.49
Configuration values (redact info where appropriate):

Cloud Functions location : us-central1
BigQuery Dataset location : us
BigQuery Project ID: xxx
Database ID: (default)
Collection path: xxx
Enable Wildcard Column field with Parent Firestore Document IDs(Optional): false
Dataset ID: xxx
Table ID: xxx
BigQuery SQL table Time Partitioning option type(Optional): NONE
BigQuery Time Partitioning column name(Optional): Parameter not set
Firestore Document field name for BigQuery SQL Time Partitioning field option(Optional): Parameter not set
BigQuery SQL Time Partitioning table schema field(column) type(Optional): omit
BigQuery SQL table clustering(Optional): Parameter not set
Maximum number of synced documents per second(Optional): 100
Backup Collection Name(Optional): failed_bq_updates
Transform function URL(Optional): Parameter not set
Use new query syntax for snapshots: yes
Exclude old data payloads(Optional): yes
Use Collection Group query(Optional): no
Cloud KMS key name(Optional): Parameter not set
Enabled events: firebase.extensions.big-query-export.v1.sync.start
fs-bq-schema-views command:

npx @firebaseextensions/fs-bq-schema-views \
  --non-interactive \
  --project=xxx \
  --dataset=xxx \
  --table-name-prefix=xxx \
  --schema-files=./part_schema_all.json,./part_schema_consented.json,./part_schema_opposed.json

Step 3: Describe the problem
Steps to reproduce:
Setup the extension as mentioned in the documentation. Follow the instructions found in the GENERATE_SCHEMA_VIEWS guide until step 3.
There, the note says

Note: You can run the schema-views script from any directory, but you need to specify the path to your schema file using the --schema-files flag. To run the schema-views script against multiple schema files, specify each file in a comma-separated list (for example: --schema-files=./test_schema.json,./test_schema2.json).

However, when trying to run the script against multiple schema files, it doesn't work and it says the schema files aren't found. When running the same script several times keeping only one schema file (eg. --schema-files=./part_schema_all.json then --schema-files=./part_schema_consented.json), it works though. (this is the only change)
Expected result
Terminale output should be something like
BigQuery creating schema view participants_schema_part_schema_all_changelog:
# parameters
BigQuery created schema view participants_schema_part_schema_all_changelog
...
BigQuery creating schema view participants_schema_part_schema_all_latest:
# parameters
BigQuery created view participants_schema_part_schema_all_latest

BigQuery creating schema view participants_schema_part_schema_consented_changelog:
# parameters
BigQuery created schema view participants_schema_part_schema_consented_changelog
...
BigQuery creating schema view participants_schema_part_schema_consented_latest:
# parameters
BigQuery created view participants_schema_part_schema_consented_latest
done.

Actual result
No schema files found!
done.

Problem fixed in https://github.com/firebase/extensions/pull/2223
Awaiting merge into next

keeping open until merged and released

Changing status back to done, as the issue is fixed, just waiting to merge https://github.com/firebase/extensions/pull/2223

Merged


use request button as close cell

I tried everything to link request button to close cell instance of touching on place, maybe my skills not good enough to implement that, please help if you can, with example of code... thank

did you find that solution ?


KeyboardControls not working in Nuxt 3.11.2

Describe the bug
KeyboardControls as per documentation are currently not working with Nuxt 3.11.2.
Reproduction
https://stackblitz.com/edit/nuxt-starter-cpvwvw?file=app.vue
Steps to reproduce

Click on canvas
Try to move
Camera not moving

System Info
OS: Windows 10 10.0.19045
    CPU: (12) x64 AMD Ryzen 5 2600 Six-Core Processor
    Memory: 2.31 GB / 15.95 GB
  Binaries:
    Node: 20.10.0 - C:\Program Files\nodejs\node.EXE
    Yarn: 1.22.22 - C:\Program Files\nodejs\yarn.CMD
    npm: 10.2.3 - C:\Program Files\nodejs\npm.CMD
    pnpm: 9.0.6 - C:\Program Files\nodejs\pnpm.CMD
  Browsers:
    Chrome: 124.0.6367.119
    Edge: Chromium (123.0.2420.81)
    Internet Explorer: 11.0.19041.3636
  npmPackages:
    @tresjs/cientos: ^3.8.0 => 3.8.0
    @tresjs/nuxt: ^2.1.1 => 2.1.1

Used Package Manager
pnpm
Code of Conduct

[X] I agree to follow this project's Code of Conduct
[X] Read the Contributing Guidelines.
[X] Read the docs.
[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
[X] The provided reproduction is a minimal reproducible example of the bug.

This is weird, but in the V4 of cientos (soon) the KeyboardControls will be modified, and comes with breaking changes.
Maybe try this instead: https://github.com/JaimeTorrealba/tres-fps-controls

Thank you very much.
I will take a look at it 👍

CameraControls don't work in the same way. The are added with no error, accessible using ref, yet no action on mousewheel/drag.


end: added flag for signing images

This adds support for having acbuild use gpg for you to sign the
produced aci. By default gpg is called with the following flags:
--armor --yes --output mycoolapp.aci.asc --detach-sig mycoolapp.aci
If any flags are provided to end after the path to the ACI, the
--armor and --yes flags are removed, and these provided flags are
prefixed to the list of flags passed to gpg.

Solves https://github.com/appc/acbuild/issues/15

I tried to use the openpgp golang library instead of exec'ing gpg, but I ran into too much frustration and then gave up.

Lgtm, although thinking about it further we should probably fail as early as possible if any required execs aren't present; otherwise, what state is the .acbuild going to be in?

If the gpg command fails (where fails means had a non-0 exit code), an error will be returned, the ACI that was produced will be deleted (happens here, just added it), and the current build context should be unchanged.
In other words, I think it'll be left in the state it was before end was called, and the build can then be continued, canceled, or end can be attempted again.

Should be ready for another pass. end was renamed to write, and no longer removes the build context. abort was renamed to end, and is intended to be called at the end of a build, regardless of success or failure. This means that write doesn't need to be called as root (probably, unless there's directories that were created in the ACI it can't get into due to permissions).
An example bash script using the new semantics is here, if anyone wants to see what it would look like.
This also enables a user to save an ACI at multiple points in a build, if they want to do so for debugging purposes, which I think is neat.

LGTM - @krobertson any chance of a review? :-)

LGTM. Two notes:

Readme example at the bottom isn't updated, it uses acbuild end.

Using --overwrite, it'll remove and recreate the aci, but not the signature. GPG doesn't error if the file already exists, so it will overwrite it fine. On a failed signing, acbuild will remove the aci, however it could leave an old signature behind.

Good catches - @dgonyeo clean those up and merge?

posthumous lgtm? :P


operation unsupported

cordova bluetoothle all operation return this error:
error:{
"error":"unsupported","message":"operation unsupported"
}
what is soluction

Either the plugin wasn't properly installed or it's being user on an improper environment like Browser.

i am installed cordova plugin add cordova-plugin-bluetoothle

Try out the Ionic based example

i am confuse, basically i want to read the value from BLE device from my ionic app. what i do.

Example is your best resource if you're already using Ionic


As an admin, I want to include book excerpts and articles as well as full volumes from EEBO-TCP, so that I can include material that is specifically about prosody from longer works about other subjects.

Equivalent to #443 for Gale
testing notes

[x] confirm excerpts were imported correctly from the spreadsheet

[x] item type and excerpt metadata set accurately (sort title, book/journal title, etc)

[x] only expected pages are included
[x] page count is accurate

All excerpts from the spreadsheet were successfully imported. Used a few randomly sampled works to spot check your testing notes, and everything looks good on front end and backend!


Ref 17664 -- Adding tests around smartif behavior

No behavior has changed. This patch just includes several tests around this existing behavior now, and no documentation has been changed because these tests (I believe) accurately reflect what the documentation says. See ticket for reference (https://code.djangoproject.com/ticket/17664)

Superseded by #12752.


Mobile datepicker gets rendered twice when using Nextjs 13

Hi,
I'm using this datepicker in a NextJS 13 app. It works fine for desktop but on mobile for some reason the input gets rendered twice after picking a date.
I do also observe the issue described in #130 .
Any ideas why?
Kind regards.

👋 This happened for me too...
while migrating to app router w/ nextjs v13.4.4. I'm using the altFormat and altInput options...
and it looks like the type="hidden" html attribute isn't getting passed to the altInput DOM element.
I will do some more digging but posting a very superficial finding here, in case this helps anyone at all.

is there any way to overcome this problem?, I am also using nextjs 13, and this problem is experienced by me as well


owntracks error while handling mqtt message

Home Assistant release (hass --version):
0.65.5
Python release (python3 --version):
Python 3.6.4
Component/platform:
owntracks
Description of problem:
Exception thrown when updates are received from MQTT.
In the owntracks component documentation, it is said that zanzito is a drop-in replacement for owntracks. There might be some specific, undocumented steps that need to be done and I've missed them?
Like the mqtt_topic parameter change to zanzito/# (https://github.com/home-assistant/home-assistant.github.io/pull/4949)
Expected:
I'm not 100% sure how this is supposed to work, but I understand it like this:
HA should take the routing key as the entity and the payload as value and store the new state. If there is JSON in the packet HA should store each key-value pair as state.
Problem-relevant configuration.yaml entries and steps to reproduce:
device_tracker:
  - platform: owntracks
    mqtt_topic: "zanzito/#"

Example packet:
routing key: zanzito.______phone.location
payload:
{"battery_level":72,"longitude":12.12345,"latitude":12.12345,"altitude":0,"gps_accuracy":23,"tst":1521387526}

Example packet 2:
routing key: zanzito.______phone.version
payload:
1.2

Should be reproducable by sending the example packets straight to your MQTT server. Exact steps will depend on server implementations.
If someone can confirm they can reproduce this issue, it would be lovely. At the moment, I'm not 100% sure this can be reproduced reliably.
Traceback (if applicable):
hass_1    |   File "/usr/local/lib/python3.6/asyncio/tasks.py", line 180, in _step
hass_1    |     result = coro.send(None)
hass_1    |   File "/usr/src/app/homeassistant/components/device_tracker/owntracks.py", line 91, in async_handle_mqtt_message
hass_1    |     message['topic'] = topic
hass_1    | TypeError: 'int' object does not support item assignment
hass_1    | , source=components/device_tracker/owntracks.py>

Additional info:
I'll try and be available on discord #devs to debug further and report back here to keep track.
If someone points me in the right direction, I might be able to provide a fix.

Ok, so the problem is obviously here:
https://github.com/home-assistant/home-assistant/blob/dev/homeassistant/components/device_tracker/owntracks.py#L91
If the message is sent as a pure value and not in JSON, the payload is parsed to be an int and the topic can no longer be attached to the message dict.

i've gone with owntracks as it's supported again.
zanzito should implement it's own component and not use the owntracks component. Documentation has been updated to not claim zanzito support.


[Issue] Pack and push NuGet project to Artifacts in Azure Pipelines

[ ] Make sure you've installed the latest version using instructions in the wiki

Output from azd version
azd version 0.3.0-beta.1 (commit c5344160ed49bfda75281fba28b5e9ac7b96670e)
Output from az version
{
  "azure-cli": "2.40.0",
  "azure-cli-core": "2.40.0",
  "azure-cli-telemetry": "1.0.8",
  "extensions": {
    "application-insights": "0.1.7",
    "arcdata": "1.4.5",
    "init": "0.1.0",
    "managementpartner": "0.1.3"
  }
}

Describe the bug
Please advise how to publish a (C#) project as a NuGet package to DevOps Artifacts using azd cli.
https://learn.microsoft.com/en-us/azure/devops/pipelines/artifacts/artifacts-overview?view=azure-devops&tabs=nuget
To Reproduce
Should the azd cli yaml host enum be extented by something like azdoartifacts?
  "host": {
                        "type": "string",
                        "title": "Type of Azure resource used for service implementation",
                        "description": "If omitted, App Service will be assumed.",
                        "enum": [
                            "",
                            "appservice",
                            "containerapp",
                            "function",
                            "staticwebapp"
                        ]
                    }

Expected behavior
Project should be build, packaged and pushed by doing the following steps:
steps:
- task: DotNetCoreCLI@2
  displayName: 'dotnet pack'
  inputs:
    command: pack
    packagesToPack: Someproject.csproj

- task: DotNetCoreCLI@2
  displayName: 'dotnet push'
  inputs:
    command: push
    publishVstsFeed: 'feed id'
  continueOnError: true

Environment
Information on your environment:
* Visual Studio Code 1.71.2

@marnixcox shouln't this be already be achievable via azdo directly? Does Azd limit that?
@danieljurek @vhvb1989 can you also take a look from Azd side if we limit azdo features?

Please advise how to publish a (C#) project as a NuGet package to DevOps Artifacts using azd cli.

azd can create the cloud resources to host a C# application and set up an Azure DevOps pipeline for continuous integration.  If you need to do more things with the same pipeline, you can update the pipeline manually.
Can you describe with more details what is the project you are creating with azd @marnixcox ?

@vhvb1989 This project is more or less related to #411 A generic library to be used from other projects. I expected that all pipeline steps would be generated by azd and I/we only needed to use the main little azd yaml file. But I'll try to add them manually.

A generic library to be used from other projects

what do you mean by  other projects ?   Does it mean projects that you are deploying to Azure when running azd up ?

Yes, projects/templates that include a c# Functions component which have a reference to a hosted nuget package/library. But my initial question was how to deploy this nuget package using azd with minimal effort.


FAQ layout issue on Safari

Describe the bug
On Safari, the FAQ's layout is broken. This seems even more visible on mobile, it may be a responsiveness issue.
To Reproduce
Steps to reproduce the behavior:

Go to https://opencollective.com/engineering/expenses/new/v2
If necessary, resize your browser
See layout issue

Expected behavior
Behavior should match what we have in other browsers. See https://www.figma.com/file/ZQBMWhnGGtRWeIZknFW1eP/[OC.com]-Production-Ready-✅?node-id=1054%3A23147

Screenshots

Related to https://github.com/philipwalton/flexbugs#9-some-html-elements-cant-be-flex-containers


Degraded performance when using USB SuperSpeed+

Describe the bug
USB defines multiple bandwidth speeds:

SuperSpeed (5 Gb/s)

USB 3.2 Gen 1
USB 3.1 Gen 1
USB 3.0 (old name)

SuperSpeed+ (10 Gb/s)

USB 3.2 Gen 2
USB 3.1 Gen 2
USB 3.1 (old name)

When connecting a camera to a SuperSpeed+ system the camera shows degraded performance and connections issues despite offering twice the bandwidth.
To Reproduce

Plug in a DFK 33UX547 (requires 5Gb/s)
Using TCAM-Ctrl set the resolution and framerate to maximum (2448x2048, 74 FPS)
Enable stream mode and observe FPS counter in the bottom left corner
Depending on your host device:

SuperSpeed: Framerate = 74FSP
SuperSpeed+: Framerate = 40 - 60FPS (depends on HW and how many devices are connected)

Expected behavior
SuperSpeed+ connections should achieve at least the same performance as SuperSpeed
computer used (please complete the following information):

OS: Ubuntu 222.04
Architecture/Platform: amd64, Laptop
Camera model you use: DFK 33UX547
tiscamera version: 1.1.1

Root cause
libus.h uses the following enumeration for the different standards:
enum libusb_speed {
	/** The OS doesn't report or know the device speed. */
	LIBUSB_SPEED_UNKNOWN = 0,

	/** The device is operating at low speed (1.5MBit/s). */
	LIBUSB_SPEED_LOW = 1,

	/** The device is operating at full speed (12MBit/s). */
	LIBUSB_SPEED_FULL = 2,

	/** The device is operating at high speed (480MBit/s). */
	LIBUSB_SPEED_HIGH = 3,

	/** The device is operating at super speed (5000MBit/s). */
	LIBUSB_SPEED_SUPER = 4,

	/** The device is operating at super speed plus (10000MBit/s). */
	LIBUSB_SPEED_SUPER_PLUS = 5
};

The SDK uses the following check to determine whether SuperSpeed is supported.
https://github.com/TheImagingSource/tiscamera/blob/ee45fcf7da4e952cb529194b9f470e9dd5cf3e61/src/libusb/LibusbDevice.cpp#L127-L139
Note that the check uses ==. Hence if SuperSpeed+ (LIBUSB_SPEED_SUPER_PLUS = 5) or any other future standard is used, the function will always return false  and perform as if SuperSpeed is not available.
Replacing the == with >= and recompiling the SDK solves the problem (see linked PR)

Hi,
Thank you for your report and merge request.
The DFK 33UX547 can be opened via v4l2 or aravis. Neither uses code in src/libusb.
v4l2 does not use libusb
Usb3Vision in aravis does not check for speed limitations to adjust behavior, at least to my understanding.
How does the patch exactly solve your issue?
It is still a valid patch for the AFU420, which is the only camera currently using the is_superspeed function,
so I have no problem accepting the patch once we could clear up any missunderstandings.

Hi Edgar,
thank you for your Reply.
I tested the camera again with the official release version (both 1.0.0 and 1.1.1). This time I was not able to reproduce the problem I described above. The camera achieved full performance using v4l2.
Why I was observing the issue in the first place a cannot tell unfortunately. My best guess is that there was some issue with my setup (OS, driver or SDK) and it was fixed by reinstalling the SDK (which I did after compiling the patched code)
I would still like to point out that I can only achieve full performance using v4l2. Using aravis the connection is unstable. Is this a known issue?

USB on laptops can be weird.
Especially when considering power saving modes and similar behaviors.
We disabled power saving for our cameras with 1.1.0, see this commit. Maybe that has something to do with your observations.

I would still like to point out that I can only achieve full performance using v4l2. Using aravis the connection is unstable. Is this a known issue?

I suspect that the performance issues arise from the fact that aravis uses user space image acquisition.
Since aravis, in comparison to v4l2, does not have a kernel module it is much more prone to interrupts and such,
which may cause problems. As a remedy you can set the priority of threads as described here:
https://www.theimagingsource.com/en-us/documentation/tiscamera/optimizations.html?highlight=thread#real-time-threading
The capture thread already tries to get the highest priority possible so enabling that should be all you have to do.

I suspect that the performance issues arise from the fact that aravis uses user space image acquisition.
Since aravis, in comparison to v4l2, does not have a kernel module it is much more prone to interrupts and such,
which may cause problems. As a remedy you can set the priority of threads as described here:
https://www.theimagingsource.com/en-us/documentation/tiscamera/optimizations.html?highlight=thread#real-time-threading

Aravis default behavior is to use the libusb synchronous API, which does not offer the best performance. You may want to try the asynchronous mode. I don't know if TIS software has a knob for it, but meanwhile, you can try and see if things improve when you switch to asynchronous mode using arv-camera-test-0.8.
arv-camera-test-0.8 --usb-mode=async

I have merged your commit. If you have additional information concerning the behavior you are seeing reopen this issue.


Add life commit

Add project life-commit
link: https://github.com/ByronHsu/life-commit

Wow this is awesome


Mixed case tags not added to checklist in v2.0.2

I recently updated to v2.0.2 and checklist stopped working for me - it would sit with the spinning/waiting symbol forever, and not add any of my items to the list. My tag that I used successfully with the previous version was #ToDo. I did some experiments and found v2.0.2 is able to list checklist items with a lowercase (#todo) tag. Not sure if this is related to a breaking change regarding custom styles mentioned for one of the previous releases - I'm using the 'Minimal' theme, but have made no other style changes.
Love that you're able to use multiple tag names in v2.0.2 so I'm looking forward to being able to update!
The plugin is awesome and a massive part of my daily workflow - thank you for all the hard work!

Indeed. Had the same issue( ToDo instead of todo), but did an bulk update the tags to lower case and plugin started working again.

Please add Case sensitive option. Thanks!


Can not get links to work on mobile devices

When my page loads on a mobile device (tablet in portrait mode, or phone), the mapped links don't work.  If I stretch the window or resize it in any way, then they work.  My site is in development is here: http://mosaikweb.com/client/beun/  and the image map is on the page with the map of the US and the video thumbnails (page title is "THE WORK WE DO" you have to click through several pages to see it).  I have searched the Internet for solutions and tried numerous things with no luck, any help appreciated!!  thank you, Laura

p.s. You do not need a tablet to replicate the issue.  If you load the site with any browser window scrunched to a size that triggers a mobile response (map needs to downsize), then hover over "Memphis", you can see the link is not active.

This is flat out not working in Safari anymore (both iPhone and desktop v9.1.3). Works great in Chrome and Firefox. Safari isn't reporting any errors, so I'm not sure what has changed.

Fixed v1.0.7


feat: add configure prism languages

[Feature]: Allow to configure prism languages

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

I will continue to implement the partial bundle.

done


feat: made contact us form accessible ✨

Related Issue
Close #685
Proposed Changes

Added proper Labels for all input fields on Contact us form.
Associated every input elements with their Corresponding Input element using for, name & id Attributes to ensure semantic markup
Finally hide the labels using screen reader only styles which, they can't be seen but they can be accessible for screen readers like accessibility devices.

Checklist

[x]  ✅ My code follows the code style of this project.

[ ]  📝 My change requires a change to the documentation.

[ ]  🎀 I have updated the documentation accordingly.

[ ]  🌟 ed the repo

[x] New feature (non-breaking change which adds functionality)

Output Screenshots

We can't capture accessibility changes through screenshots as opposed to code changes since accessibility modifications are often imperceptible to the user, as their purpose is to assist disabled individuals.

Your review is appreciated @agamjotsingh18 🙌🏻

Pr has conflicts

Please merge this PR now @agamjotsingh18

Please merge this PR now @agamjotsingh18


Adding option 'yLabelFormat' in BarChart?

Hi, thank you for making this library. I'm using it well.
Why don't you add an option yLabelFormat in BarChart?
Find more in https://codepen.io/andreic/pen/olzAJ

Hi,
You're right, I have forgotten to add these options (y and xLabelFormat).
I've published the version 1.0.0 with these options for bar chart.
Thank you !


fixes #77 to allow timestamp docker labels to be used for snapshot builds

fixes #77 to allow timestamp docker labels to be used for snapshot builds

[merge]


Replace usages of GenericPageProcessor with compiled PageProcessor

We have optimizations in the compiled page processor which are not in
the GenericPageProcessor. GenericPageProcessor should be used only if
something caused compilation to fail.

Looks good


AutoStart seems to not work

Hello when I start my pi up the display is black. I think I've looked every on the page and can't seem to find an answer
have any suggestions?

Do you have a mouse cursor? (if yes) What happens when you right click on the black screen?

when I right click I get a menu that just says open terminal emulator, open browser, etc

Can you upload a photo.s of your below files:

sudo nano /etc/xdg/openbox/environment
sudo nano /etc/xdg/openbox/autostart
sudo nano /etc/X11/xinit/xinitrc

/etc/xdg/openbox/environment

/etc/xdg/openbox/autostart

/etc/X11/xinit/xinitrc

I had a similar problem and added to your below file a last bottom entry line:
sudo nano /etc/xdg/openbox/environment
python3 spotifypod.py &
Then reboot and it worked !

Has your "Spot" ever worked for you?

Nope it never has.
So I added the new line you told me and it still doesn't auto run the file

two lines !!

my autostart only has two lines:
cd /home/pi/retro-ipod-spotify-client/frontend
sudo -H -u pi --preserve-env=SPOTIPY_REDIRECT_URl,SPOTIPY_CLIENT_ID,SPOTIPY_CLIENT_SECRET, python3 spotifypod.py &

My Environment minus the private stuff:
SCOPE = "user-follow-read,"
"user-library-read,"
"user-library-modify,"
"user-modify-playback-state,"
"user-read-playback-state,"
"user-read-currently-playing,"
"app-remote-control,"
"playlist-modify,"
"playlist-read-private,"
"playlist-read-collaborative,"
"playlist-modify-public,"
"playlist-modify-private,"
"streaming,"
"user-follow-modify,"
"user-follow-read"
cd /home/pi/retro-ipod-spotify-client/frontend
sudo /home/pi/retro-ipod-spotify-client/clickwheel/click &
python3 spotifypod.py &

Our "sudo nano /etc/X11/xinit/xinitrc"  are identical!

let me modify my environment to look similar to yours and and see if it works

okay I'm at my computer now waiting for you

yours has to be after home/xero mine is home/pi

oh yeah, thanks almost forgot going to reboot again

change you autostart to look exactly like mine as well!

I also have spaces between my lines

Yeah it worked! but the clickwheel driver didn't start so going to put it back in the auto start and see if it works with it.

that always breaks things for me!

yeah as soon as I add it it doesn't work.

send a few photo's of your breakout board wiring if you used one and did you compile click.c ?

Do you or did you work at Xerox?

send a few photo's of your breakout board wiring if you used one and did you compile click.c ?

Yes I did compile it

And no I never worked at Xerox

Your breakout board wiring may be incorrect can you send photo of the reverse side

I'm retired from Xerox

The wiring is correct it does display it just displays darkness.
You used to work at xerox never heard of them before what did you do?

Field Technician

what to you mean it displays?

What I mean is when I go and right click it it shows the menu. It just doesn't run the spotifypod.py file on boot.
Was it hard to be a Field Technician?

I used the 2"Adafriut display with composite video input

for my display I followed these instructions from rsflights

I am a certified Electronics Technician so no it wasn't hard for me

wow, but how come you don't work there anymore if it wasn't hard for you?

I Built a color TV from a kit (Heathkit) when I was17 years old

Are you saying that the clickwheel was working?

you built a tv before! Is it similar to this one?

Uh not the clickwheel does not work anymore

I ssh into the pi and manually started the spotifypod.py file and moved it around

my clickwheel works with the entries a sent you

really? let me take a second look at the code and see if I messed up

how old are you?

I'm 17 years old

Ah I see I have a granddaughter your age

I built 6 Heathkit GR-2001 25" color TV's when I was 17 and a Complete Heathkit SB401 SB-303 with accessories Amateur Radio Station

I can't help you with your IPS Display, however can't understand why your clickwheel isn't working if it was before!

Wow you built a ton of stuff.
I can't understand why it doesn't work anymore either, I think I'm just going to take a break and come back later to try and fix it

taking a break always works for me ... Good Luck! & Stay Focused

Thanks you so much for the help!! Hope you have a great rest of your day/night!!

Remove all the Scopes from "sudo nano /etc/xdg/openbox/environment"


include priority in requestItem and add to request queue accordingly

Tries to fix #60213

@mjbvz
I hope this is the kind of change you expected. Instead of low and normal priority mark, I've used normal and high. Also, I believe that executeWithoutWaitingForResponse is of higher priority request from the user and changed accordingly.
Kindly review and let me know if I should modify things.

Thanks for the review.

specifically marking a few requests as low priority

Does that mean that we will have to insert all other request in the queue on the beginning of the array?

mark the code lens reference's request as low priority

By marking, do you mean we'll have to get the request type within the push function and based on that we'll have to insert into the queue?
Based on your review remarks, I believe that I'll have to remove the priority property that introduced. Correct me if I'm wrong.

The simplest way to implement this would be to always enqueue normal priority requests in front of low priority requests. When pushing a normal priority request onto the queue, check to see if the previously queued request is a low priority one. If it is, then insert the new request in front of that one instead of at the very end of the queue.

I think it would be best  add a new lowPriority flag to the execute function.

Yes, instead of a generic priority property just have a lowPriority boolean property.

Let me know if that makes sense

I hope I got your points.

Additionally, I've considered multiple lowPriority requests at the end of the queue and inserting requests before them.
Only the execute function has the flag lowPriority set to true.

Let me know if anything should be updated further.

@mjbvz

hook this up to mark the references lens requests

I couldn't get to the point where the references codelens request are being made and add the flag. Could you please guide me a little on this one.

Just saw your comment. Yeah I just realized that we don't directly call execute in this path anymore and instead use a command that invokes this indirectly: https://github.com/Microsoft/vscode/blob/4b8f928808f3fbf4745702d3b558e8ccbe2a726a/extensions/typescript-language-features/src/features/referencesCodeLens.ts#L21
This was a fairly recent change: https://github.com/Microsoft/vscode/commit/dc47417f54ffb411d88f12f548dfc9ef8efb30cc
I'll merge this PR in as is and see what we want to do about the references requests


New version: InvariantCausalPrediction v0.1.1

Registering package: InvariantCausalPrediction
Repository: https://github.com/drcxcruz/InvariantCausalPrediction.jl
Created by: @drcxcruz
Version: v0.1.1
Commit: 589ac1cee647c686f58666d938e723f15121c98a
Git reference: master

Closing due to inactivity, please comment if this is still relevant.


Editor 6 OS X —  issue working on tables

Copy from within the editor and paste it into a table cell is not working
Highlighting text in a table cell works, but the highlighted text cannot be deleted, cut ot

Also when clicking on an empty bullet point which is located right before a table, the cursor jumps to the first th instead

Concerning the bullet point, you might have inserted the table inside the bullet point by inadvertence.

Tables seem to be rendered not correctly, see video:
http://screencast.com/t/yXxRuDLxH
(bug happens 0:14)

@drreen I'm working on fixing it for next release 👷

@drreen Just landed a fix for table previews (when in raw mode) in the master. Will likely get released tomorrow


Broken demo

Hey, the component you've got here looks great, but visiting the demo results in an invariant violation:
Error: Invariant Violation: App.render(): A valid ReactComponent must be returned. You may have returned undefined, an array or some other invalid object. app.js:7:17671

I redeployed the demo. Should work now. Not sure what caused the error though.


Niftgen proposal

Curious how many current active content creators and users on Niftgen and how many we can estimate would be added to the ecosystem with this proposal?

Hi @averagezen
We have onboarded 12 content creators. Not all of them have started posting content. In terms of users, over 500 people have created accounts. Estimating how many users would be added to the ecosystem is hard. A lot will depend on how often the creators will do live streams.

It would be helpful if you rename the file to xgov-7.md so GitHub can show a human friendly version of the proposal you wrote.

It would be helpful if you rename the file to xgov-7.md so GitHub can show a human friendly version of the proposal you wrote.

Sure, how would I do that exactly ?
Not supper familiar with how GitHub works

It would be helpful if you rename the file to xgov-7.md so GitHub can show a human friendly version of the proposal you wrote.

Sure, how would I do that exactly ? Not supper familiar with how GitHub works

You can click on the Edit file here:

@SudoWeezy done

why is algorand needed to make niftgen work? the only thing that you want to use seems to be POAPs which are fine but i dont see how that should people onboard onto Algorand
how do you think to compete against the big live streaming platforms? what can you offer creators that others like twitch, youtube, kick,..., cant? why should i as a user come and watch things on niftgen when i can go to a mature platform like youtube where apparently the same content will be livestreamed on?
in my honest opinion it feels like a web2 project that wants to be affiliated with web3 but doesnt really benefit from web3

why is algorand needed to make niftgen work? the only thing that you want to use seems to be POAPs which are fine but i dont see how that should people onboard onto Algorand
how do you think to compete against the big live streaming platforms? what can you offer creators that others like twitch, youtube, kick,..., cant? why should i as a user come and watch things on niftgen when i can go to a mature platform like youtube where apparently the same content will be livestreamed on?
in my honest opinion it feels like a web2 project that wants to be affiliated with web3 but doesnt really benefit from web3

Hi @2lobo
Thank you for your feedback.
Here is a bit more clarification:
The way Niftgen and the creators make money is through subscriptions. Those subscriptions are all on chain. Users are reward for their engagement with a utility token  ( no monetary value) that can used to access features.
We are not telling creators to stop using Twitch or YouTube. Creators use YouTube / Twitch to get discovered, but then bring their followers over to Niftgen to access exclusive content. Patroon is a more accurate competitor.
The way Niftgen benefits from web 3 is because users are actually rewarded for engaging with content. On web 2 video platforms, your are just a passive user.

Hello
I am withdrawing the proposal and will close the pull request


Feature/#9 es256 k

Codecov Report

Merging #11 (9d1c591) into master (1b217a0) will increase coverage by 0.28%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master      #11      +/-   ##
==========================================
+ Coverage   82.50%   82.78%   +0.28%
==========================================
  Files           4        4
  Lines         680      697      +17
==========================================
+ Hits          561      577      +16
- Misses        119      120       +1

Impacted Files
Coverage Δ

src/decode.c
82.41% <ø> (ø)

src/encode.c
78.24% <100.00%> (+1.00%)
:arrow_up:

Continue to review full report at Codecov.

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

Codecov Report

Merging #11 (9d1c591) into master (1b217a0) will increase coverage by 0.28%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master      #11      +/-   ##
==========================================
+ Coverage   82.50%   82.78%   +0.28%
==========================================
  Files           4        4
  Lines         680      697      +17
==========================================
+ Hits          561      577      +16
- Misses        119      120       +1

Impacted Files
Coverage Δ

src/decode.c
82.41% <ø> (ø)

src/encode.c
78.24% <100.00%> (+1.00%)
:arrow_up:

Continue to review full report at Codecov.

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

Welcome to Codecov :tada:
Once you merge this PR into your default branch, you're all set! Codecov will compare coverage reports and display results in all future pull requests.
Thanks for integrating Codecov - We've got you covered :open_umbrella: