index: md5: handle missing files and cloud versioned files missing version_id

Codecov Report
Attention: 12 lines in your changes are missing coverage. Please review.

Comparison is base (31a65d7) 63.83% compared to head (c7bca3b) 63.70%.
Report is 1 commits behind head on main.

Files
Patch %
Lines

src/dvc_data/index/index.py
14.28%
5 Missing and 1 partial :warning:

src/dvc_data/index/save.py
50.00%
6 Missing :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #485      +/-   ##
==========================================
- Coverage   63.83%   63.70%   -0.13%
==========================================
  Files          61       61
  Lines        4164     4177      +13
  Branches      712      714       +2
==========================================
+ Hits         2658     2661       +3
- Misses       1358     1368      +10
  Partials      148      148

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


_YYTextKeyboardViewFrameObserver 移除KVO时崩溃了

*** Terminating app due to uncaught exception 'NSInternalInconsistencyException', reason: 'An instance 0x119bc9640 of class UIInputSetHostView was deallocated while key value observers were still registered with it.


Update $PSBoundParameters for Special Parameters

For all functions that use the New-TwitterQuery, ensure that $PSBoundParameters is updated, if necessary.
if ($UserId.Count -gt 100) {
    'More than 100 UserIds provided. Processing the first 100 only.'  | Write-Warning
    $PSBoundParameters['UserId'] = $PSBoundParameters['UserId'] | Select-Object -First 100
}
$Query = New-TwitterQuery -ApiParameters $PSBoundParameters

See Get-TwitterFriendship for an example.

The command New-TwiiterQuery has been removed from the next planned release.
This issue is no longer necessary.


Any plans for MAUI?

Any plans to make this available for MAUI? maybe you can reach Microsoft they will love it as their example is terrible and incomplete.

Any plans to make this available for MAUI? maybe you can reach Microsoft they will love it as their example is terrible and incomplete.

Hello, @MhAllan.
The migration it's in progress, but have problems with dotnet maui, maybe with dotnet seven it works.
@tompi can we resume the migration? Here at the company, we are going to start the migration, can you generate the android/ios nuget again for testing? The sample app stays with me.

@andersonvieiragomeslopes Sorry, I just tried once more, and the bindings generator still crash with same message:
BINDINGSGENERATOR : error BG0000: System.NullReferenceException: Object reference not set to an instance of an object. [/Users/tompi/src/Xamarin.AzureCommunicationCalling/Bindings/Android/Laerdal.Maui.          AzureCommunicationCommon.Android/Laerdal.Maui.AzureCommunicationCommon.      Android.csproj::TargetFramework=net7.0-android]

Not sure how to proceed, no useful error message, and no relevant google hits...
Should maybe report a bug somewhere, but Im guessing Maui team have a lot to do, and this is probably very low pri for them... I still think Maui is far from production ready unfortunately...

Hi @tompi .
We don't have choice and time.
Android 14 and iOS 17 will not be supported unfortunately.
Could you check the iOS migration while I check if I get results with Android?

https://devblogs.microsoft.com/xamarin/xamarin-ios-16-android-13-updates/

@andersonvieiragomeslopes I tried updating the iOS binding library for Maui(.net 6) to 2.3.0-beta.5.
There were actually a lot of changes between beta.4 and beta.5...
Anyway, I dont have much time trying to validate this on Maui, let me know if you run into problems or anything seems off(there were some depracations of muting functionality, but should be straightforward to use new methods).
P.S. Any luck on the Android bindings?

@andersonvieiragomeslopes Hey, curious about the status of this... Did you have any success binding the android lib for Maui?

Hi @tompi.
Sorry, I'm traveling these last few days.

Hi @tompi.
I am back. At the end of the week I go back to the migration stage with android to see if I can solve the problem.

Hello @tompi @MhAllan
I believe I have the same problem as you.
However, I have my time almost at the limit, I will dedicate an exclusive time to try to solve this as soon as possible later this week, I'll give you news.

We now have functioning maui nugets. Still missing ports for the example app, but not sure we will prioritize this, what do you think @andersonvieiragomeslopes ?

We now have functioning maui nugets. Still missing ports for the example app, but not sure we will prioritize this, what do you think @andersonvieiragomeslopes ?

Of course, I'll look at how the nugget packages are doing and start migrating the example app


[#1175] fix(netty): Retry failed with StacklessClosedChannelException after channel closed

What changes were proposed in this pull request?
1、reget a transportClient in ShuffleServerGrpcNettyClient.sendShuffleData RetryCmd
2、add ShuffleServerGrpcNettyClient construct with maxRetryAttempts (but in this pr init ShuffleServerGrpcNettyClient with default maxRetryAttempts(3))
Why are the changes needed?
Fix: #1175
Does this PR introduce any user-facing change?
No.
How was this patch tested?
Existing UT.

Could you run the command to fix style issues?
mvn spotless:apply -Pspark3 -Pspark2 -Ptez -Pmr -Phadoop2.8

Codecov Report

Merging #1181 (4db497d) into master (ac38db5) will increase coverage by 0.50%.
Report is 2 commits behind head on master.
The diff coverage is 0.00%.

@@             Coverage Diff              @@
##             master    #1181      +/-   ##
============================================
+ Coverage     53.63%   54.13%   +0.50%
+ Complexity     2581     2107     -474
============================================
  Files           391      319      -72
  Lines         22407    14450    -7957
  Branches       1875     1314     -561
============================================
- Hits          12017     7822    -4195
+ Misses         9683     6208    -3475
+ Partials        707      420     -287

Files Changed
Coverage Δ

...client/impl/grpc/ShuffleServerGrpcNettyClient.java
0.00% <0.00%> (ø)

... and 74 files with indirect coverage changes
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

Could you run the command to fix style issues?
mvn spotless:apply -Pspark3 -Pspark2 -Ptez -Pmr -Phadoop2.8

ok

+1 LGTM.


Crash when Internet Connection is reconnected

What issue do you have? Please be as thorough and explicit as possible.
Everytime my internet connection is shortly interrupted homebridge throws the following error:
Error: read ECONNRESET
    at TLSWrap.onStreamRead (node:internal/stream_base_commons:220:20)

After searching a lot I could pinpoint the issue to the homebridge-deebot plugin. When I disable the plugin and the trigger a reconnect in my router nothing happens. As soon as the plugin is enabled the error is thrown and homebridge is shut down.
Details of your setup.

Do you use (1) Homebridge UI-X (2) Homebridge CLI or (3) HOOBS?
Homebridge
Which version of Homebridge/HOOBS do you have?
v1.4.0
Which platform do you run Homebridge/HOOBS on (e.g. Raspberry Pi/Windows/HOOBS Box)? Please also mention your version of Node.js/NPM if known.
Rasperry Pi with Homebridge OS, Node.js v16.13.2
Which version of this plugin (homebridge-deebot) do you have? Has the issue started since upgrading from a previous version?
latest version v4.4.1 (just started using this plugin, so no update history)
Which ECOVACS devices do you have that are causing issues? Please include product models if applicable.
I dont think it is relevant, but my Ecovas Device is a Deebot Ozmo 950

@korbinio @bwp91
I can confirm that this error currently occurs for components that use the ecovacs-deebot.js library.
But I still don't know if it is triggered by the library itself or by the implementions that are using the library, because I'm able to reproduce it with the Node-RED integration and also the ioBroker plugin. But I cannot reproduce it with the library itself.
The error occours at TLSWrap.onStreamRead (node:internal/stream_base_commons:2xx:20) (not in the library) and there are also several other reports of different integrations with the same problem.
I already made some changes to the library with version 0.7.2-alpha.6 which is already used by the beta version of the homebridge-deebot plugin. Maybe you can try if the same error occours with the version 4.4.2-beta.1 in your case.

@mrbungle64 I updated to the beta version (v4.4.2-beta.1):
When I interupt the internet connection in my router homebridge is still being restarted, but the error is now not specified anymore:
[07/02/2022, 14:27:48] [Deebot] [DEEBOT OZMO 950 Series] has been reported [offline].
[07/02/2022, 14:27:51] Got SIGTERM, shutting down Homebridge...

So the error message is not displayed anymore, but the problem persists... (During my testing the reboot was not always triggered right after the reconnect but took sometimes a minute or two to cause the reboot)

@korbinio
In the meantime I already released version 0.7.2-alpha.8
I don't know if it's already used in your installation.

I think we have to wait for @bwp91 to weight in on this one. From the changelog of v4.4.2-beta.1 I can only take that it uses "v0.7.2 (prerelease)" of ecovacs-deebot, but not specifically which version...

Thanks for sorting this out @mrbungle64
@korbinio does the device reconnect properly when the internet comes back online again?

Yes, the reconnect works perfect. There is no output in the log, but the device is instantly controllable after the reconnect.


Feat: robyn_run outputs parameter deprecation #554

Project Robyn
Removed outputs = FALSE from robyn_run()
Fixes #554
Type of change

feat: New feature (non-breaking change which adds functionality)

How Has This Been Tested?
Tested by running outputs = TRUE, outputs = FALSE, and without defining outputs ran successfully and the output was consumed by the following function robyn_outputs()
Updated the documentation, removed the parameter from demo.R script
##Additional Info
The decision to make the deprecation comes from the fact that outputs = TRUE is redundant. The user is still required to run robyn_outputs() where the params are inherited for the allocator and clusters

Abandoning this branch/PR.


Is there a mix-up in the calculation of style_contrastive_loss and content_contrastive_loss?

I find the code calculate the style_contrastive_loss and content_contrastive_loss have something that bothers me at line 325 and 360 in Net.py, Are they upside down?

请问这里的up和down的含义，而且对比损失似乎形式与论文不一样，麻烦帮忙解答一下，谢谢

想问一下你的代码跑起来了吗，我跑起来跑一会CUda就出错了，环境和官方一样


Make the fileds of mbedtls_net_context public

As part of #4683, as a result of reviewing the programs there are uses of MBEDTLS_PRIVATE fields of the mbedtls_net_context structure. To avoid the access of the private fields make the fields of mbedtls_net_context public.
Accessed fields:

fd

Affected files:

ssl/mini_client.c

https://github.com/ARMmbed/mbedtls/pull/4842 made this field public (with a documented stipulation that this is only guaranteed on Unix/POSIX platforms).


Fixed port of mongo-c-driver/mongo-cxx-driver for android

This fixes building of mongo-c-driver and mongo-cxx-driver for android.
Since android is still unofficial I tested it with the triplets defined here: https://vcpkg.readthedocs.io/en/latest/users/android/
I tested arm-android and arm64-android with VCPKG_LIBRARY_LINKAGE = static

Forgot update Port-Version?

@JackBoosY No, I applied your suggestion in github but made the changes to the manifest files later.

Thanks!


Template Selector

Hi @AndreiMisiukevich,
Do you support the usage of TemplateSelector?
Thanks,
-Nawid

Hi) of course :)
ItemTemplate supports selector as well as simple template


upstream merge

Before you hit that Submit button....
Please read our contribution guidelines
before submitting a pull-request.
Types of changes
What types of changes does your code introduce?
Put an x in the boxes that apply

[ ] Bugfix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)

If you want to raise a pull-request with a new feature, or a refactoring
of existing code, it may well get rejected if it hasn't been discussed on
the mailing list or
slack team first.
Proposed changes
Describe the nature of this change. What problem does it address?
Checklist
Put an x in the boxes that apply

[ ] I have read the contribution guidelines
[ ] For non-bugfix PRs, I have discussed this change on the mailing list/slack team.
[ ] I have run grunt to verify the unit tests pass
[ ] I have added suitable unit tests to cover the new/changed functionality

Coverage remained the same at 86.228% when pulling 8e5f28db0ab75108b8920bb1cac2ab32360cdd0f on parodot:master into 2b9aa94f3a4fd432f4fdbf82428d09d4b8f6d74a on node-red:master.

Coverage decreased (-0.07%) to 86.156% when pulling 8e5f28db0ab75108b8920bb1cac2ab32360cdd0f on parodot:master into 2b9aa94f3a4fd432f4fdbf82428d09d4b8f6d74a on node-red:master.


[All hosts] (manifest) Add note about closing slash in AppDomain element

Fixes https://github.com/OfficeDev/office-js-docs-pr/issues/835

OPS Build status updates of commit cd8644d:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/reference/manifest/appdomain.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

There is a bug in the commit - missing '/' in closing node of , i.e. it should be
https://myappdomain<**/**AppDomain>

That typo was already there from a previous commit. But I'll get it fixed.


Fetching runtime data with existing login fails silently

Get 401 Unauthorized from get runtime request. However this error is never surfaced to the user, but silently ignored. This can cause the runtime list to be outdated, and probably also fails on syncing projects.
Logging out and in a gain fixed the problem. So seems like a token expiration issue.

For token expiry, there is probably also something we should be doing which we are not.

The latest Passport API provides refresh_token upon login that can be used to refresh an access_token without forcing the user to relogin.


verticalLinesStrokeDashArray property not being applied

Hi! Great library.
I'm experiencing an issue where I can't get the vertical lines dotted using the verticalLinesStrokeDashArray property.
After some debugging, I found that the property is getting lost inside gifted-charts-core package (getAxesAndRulesProps() function to be specific). I've opened a PR https://github.com/Abhinandan-Kushwaha/gifted-charts-core/pull/10
Please merge the PR if it looks good, publish a new version of gifted-charts-core and update this lib to use the new version. Thank you!

Merged your PR @Maximization
Thanks for your contribution. I will release a new version within 1-2 days.


Reading .csv.gz, new strategy=4, and new axis specific metrics

I have made the following updates:

I have added the option to recognise and read csv.gz files (gunzipped csv files). This allows studies with a lot of csv files to gunzip them and as a result save storage space.
Day selection in part 2 when strategy=2 improved, because it did not align correctly, we were constantly getting a few epochs on day zero labelled as valid even though that should be impossible.
Strategy=4 added to part 2, which ignores all data before the first midnight and uses all the data after this. So, it is a variation on strategy=2
Metrics added to quantify low-, high-, and band-pass filtered acceleration per axis. (lfx, hfx, bfx, ...)
All code related to metric calculation tidied up to avoid calculating the same thing twice and to make the code more readable. Function g.metric is now not used anymore, but I am keeping it inside GGIR for another year just in case there are GGIR users who use this function directly.

@jhmigueles hold on with this, I just merged your pull request and now see that I need to make some updates on this pull request for it to merge in smoothly

Codecov Report

Merging #301 into master will decrease coverage by 0.03%.
The diff coverage is 79.60%.

@@            Coverage Diff             @@
##           master     #301      +/-   ##
==========================================
- Coverage   73.02%   72.98%   -0.04%
==========================================
  Files          74       74
  Lines        8707     8844     +137
==========================================
+ Hits         6358     6455      +97
- Misses       2349     2389      +40

Impacted Files
Coverage Δ

R/g.analyse.R
89.06% <ø> (-0.12%)
:arrow_down:

R/g.metric.R
92.40% <ø> (-7.60%)
:arrow_down:

R/g.impute.R
76.15% <28.57%> (-2.08%)
:arrow_down:

R/g.readaccfile.R
43.84% <50.00%> (+0.03%)
:arrow_up:

R/g.analyse.perday.R
71.22% <57.14%> (-0.29%)
:arrow_down:

R/g.shell.GGIR.R
75.06% <68.75%> (-0.20%)
:arrow_down:

R/g.getmeta.R
63.20% <73.07%> (+0.60%)
:arrow_up:

R/g.inspectfile.R
70.27% <80.00%> (-0.16%)
:arrow_down:

R/g.part1.R
61.84% <88.23%> (+0.49%)
:arrow_up:

R/g.applymetrics.R
91.02% <88.61%> (-8.98%)
:arrow_down:

... and 3 more

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 190e8b3...c8d35bb. Read the comment docs.

Let me know when this is ready and I'll have a look. Best.

This is ready now, I fixed the merge problem right after I posted that message. Thanks


Fix init_subclass call order

Codecov Report

Merging #16 (a96e188) into main (1627740) will not change coverage.
The diff coverage is 100.00%.

@@            Coverage Diff            @@
##              main       #16   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files            4         4
  Lines          159       159
=========================================
  Hits           159       159

Impacted Files
Coverage Δ

abcattrs/__init__.py
100.00% <100.00%> (ø)

abcattrs/abcattrs.py
100.00% <100.00%> (ø)

tests/test_decorator.py
100.00% <100.00%> (ø)

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 1627740...a96e188. Read the comment docs.


Replaces fails - if the upstream project has defined the dependency in both a 'replace' and a 'require' block

Example:

https://github.com/rancher/rancher/blob/v2.8.7/go.mod

Upstream have set this in the 'replace' section:
github.com/containerd/containerd => github.com/containerd/containerd v1.6.22

And they also have this in the 'require' section:
github.com/containerd/containerd v1.7.6

If you try doing this using go/bump:
  - uses: go/bump
      replaces<EMAIL_ADDRESS>
It'll fail to apply, and complain that a newer version is already set (i.e in the require section). This is a bit of a strange implementation by the upstream project.
To address, we should drop the check that looks for a newer version in the go.mod, if the version is set in the 'replaces' block

I agree it looks like a weird configuration for their dependencies. I believe if we could allow setting only the replaces block in gobump this issue could be solved as well.


[feature request] Do we need to exand Scheduling Constraints to select proper hosts?

Some containers may want to be scheduled to hosts that have abundant resources(cpu/memory) for containers to consume. So do we need to add resources Constraints to select proper hosts in scheduling process?

@xiechengsheng we have already -reserve-cpu and --reserve-memory parameters (look https://docs.docker.com/engine/reference/commandline/service_create/#options ) which makes sure that host have at least asked amount of free resources.
Is that what you are looking? Please close this one if so or tell us if you need more guidance.

This makes sense to me, thanks!


Create Character “lebanon-glambrina”

Automatically generated by Netlify CMS

👷 Deploy Preview for dynamic-cheesecake-5fbd8b processing.

Name
Link

🔨 Latest commit
4b086d6e012c21e606f3799a2440cfb6f30888eb

🔍 Latest deploy log
https://app.netlify.com/sites/dynamic-cheesecake-5fbd8b/deploys/632626b77400fb0008d58d7d


Custom icon for SDFG files (workaround)

Waiting for https://github.com/microsoft/vscode/issues/14662 to be closed.

@phschaad I worked around the problem by copying the master Seti icon set and adding SDFGs. This is a temporary fix and I set up Issue #2 for tracking it.


Swipe is not working on my phone

MAKE SURE YOU TESTED THE LATEST RELEASE.

Samsung user? See https://mathewsachin.github.io/Fate-Grand-Automata/SAMSUNG.html

Describe the bug
A clear and concise description of what the bug is.
Include steps to reproduce the error if required
Screenshots
If applicable, add screenshots to help explain your problem.
If an error occurred, you can find the message by expanding the app's notification.
It is recommended that you atleast add one screenshot of FGO running on your device.
Video
If relevant, include a video of the script running on your device with Debug mode ON (recording functionality is available within the app in More options).
Device Info (please fill atleast 'Device model'):

Device model: Samsung S7 Edge
Screen size: 1280x720
Android Version: Android 8.0
RAM: 4GB

My game is already on full screen but only swipe is not working when selecting the support servant. (works fine on emulator)
It stays there for a while till it press on refresh button and till a skadi shows up since i have her as preferred servant.

Next time, please replace the placeholders with actual content. For example, provide a video.
Try setting Fate/Grand Automata to fullscreen too.

We ask those things in the bug template just to help you out faster.
Provide a video with debug mode ON.

Closing since sufficient info wasn't provided and there wasn't a response. Reopen with proper info if the issue still happens.


Increase test coverage

This PR:

Fixes some bugs, to get more code running
Un-comments some tests, to increase test coverage
Refactors, and fixes bugs in, the convergence tabulation

bors r+


package version 6.3.0 is not published anywhere other than github packages

It does not seem that the 6.3.0 version of the package was published anywhere. I cannot see it on mavencentral.
It does seem to be published to github package repository, but that is useless for most people.

this is all set. thanks for calling it out


downgrade errors returned by watchHandler in reflector.go to warnnings

Because the Reflector will try to reconnect, so downgrading errors returned by watchHandler in reflector.go to warnnings
fix #11854
@dalanlan @lavalamp

Thank you! LGTM


Give number of waybills in cargo flow

Today, the number of a cargo bearer displays as *A trainset of X wagons", which is inappropriate. Instead, the number field should be interpreted as number of waybills for cargo bearer but as quantity for non-bearers. Maybe these should be two separate fields where quantity for cargo bearers always is 1, and any number ow waybills for all types of cargo flows.

For cargo bearers, the quantity is now ignored and will always be one. This is probably an intermittent solution. Quantity may later be interpreted as number of waybills because each bearer requires a waybill.

Quantity is now interpreted as number of waybills because each bearer requires a waybill.


⚠ SpeakUpMinistries.com has degraded performance

In 06c700a, SpeakUpMinistries.com (https://www.speakupministries.com) experienced degraded performance:

HTTP code: 200
Response time: 17969 ms

Resolved: SpeakUpMinistries.com performance has improved in 9fe1e97 after 6 hours, 22 minutes.


Bundle assets/resource file in go binary

Fixes #346
This PR adds the ability to build the go binaries with the web files included, so it can be distributed as a single binary.
There are several solutions/packages available for solving this problem.
Some of the packages I saw while researching this problem includes:
packr - https://github.com/gobuffalo/packr
vfsgen - https://github.com/shurcooL/vfsgen
esc - https://github.com/mjibson/esc
gobundle - https://github.com/alecthomas/gobundle
shuLhan/go-bindata - https://github.com/shuLhan/go-bindata
go.rice - https://github.com/GeertJohan/go.rice
statik - https://github.com/rakyll/statik
parcello - https://github.com/phogolabs/parcello
statics - https://github.com/go-playground/statics
templify - https://github.com/wlbr/templify
go-embed - https://github.com/pyros2097/go-embed
go-resources - https://github.com/omeid/go-resources
gnoso/go-bindata - https://github.com/gnoso/go-bindata
fileb0x - https://github.com/UnnoTed/fileb0x
This PR used packr - https://github.com/gobuffalo/packr based on its simplicity, active development status, the fact that it supports the http.FileSystem interface making serving as http static files a breeze and its support for local file system so that devs don't need to bundle the assets/resource before testing
A build.sh is added to the root of the project with the following commands
./build.sh web which change to the src directory of the web, builds the front-end and returns to the root directory and
./build.sh deploy which fetches the go mod dependencies, builds the web assets and bundle it all as a single binary file

The PR also fix #297  by using webpack dynamic import to lazy load toastr and bootstrap-toggle and limit the size of the output files

I tested it and was able to get it to build. The CSS didnt load correctly for me. I will try again later to see if anything was missed in the build process.

Testing on a linux machine.
This does not work natively without any parameters (./build.sh) nor with deploy parameter both result to "line 23 packr command not found" even after installing packr via documentation ('go get -u github.com/gobuffalo/packr/packr')
deploy web seems to work fine and produce a binary around 35 mb that which can serve assets.
Also note to make into executable if possible and push it back (chmod +x)


Unable to exec java: Permission denied

After succesfull installation, unifi-db runs ok, but application gives me this error:
s6-applyuidgid: fatal: unable to exec java: Permission denied

Closing for inactivity


Review Order of Execution

We have implemented the order of execution. Could you review and provide your feedback?
https://github.com/fabric8io/fabric8-planner/pull/609 has some discussions around order of execution
http://demo.almighty.io/

Drag and drop

Click and drag a work item and drop it at it's new location
The user can drag and scroll

Select a work item and move it

Check-boxes are displayed for all the work items
A user can move the work item after selecting a check box
On selecting the check box, a Move drop down menu is displayed next to the Backlog text
Move to Top will place the selected work item as the first item of the list
Move to Bottom will place the selected work item as the last item of the loaded list
As we are moving a single work item, only one work item can be checked at a time

Consider this scenario

Work item A has been selected, it's detail pane is open.
Select Work item B's check box, the Move menu is displayed
Move Work Item B to it's new location
UI behavior - Work Item A is highlighted and it's detail screen remain open. Work Item B moves up or down to it's new location.

Bugs have been filed against the UI for Ordering Items

https://github.com/fabric8io/fabric8-planner/issues/683
https://github.com/fabric8io/fabric8-planner/issues/684
https://github.com/fabric8io/fabric8-planner/issues/685
https://github.com/fabric8io/fabric8-planner/issues/686
https://github.com/fabric8io/fabric8-planner/issues/687

The current UI does not follow the wireframes, and notes have been made accordingly.
Additionally, the Drag and Drop does not match the screens as given by UX. What screens were referenced when implementing that behavior? If the methods are to be used elsewhere, they need to be fixed first.

Additionally, these should all be reviewed with the @fabric8io/uxd team before pushing the code to master, given the continuous deployment cycle. This will avoid the need to file bugs for items that should have been caught before the merge.

Thanks @mindreeper2420. Going forward we will share screenshots and videos of the PR before merging.

@nimishamukherjee sounds good. The videos that have been posted in the past worked out very well.


Add configuration option to restrict the number of entries in recent updates

Currently it is hard coded to 5.

You mean adding a optional parameter with a default value to the recent updates functions? This way we may use config.yaml to configure the number of issues, PRs etc

@arsulegai Can you please clarify what you mean by "option to restrict the number of entries"? Do we need to make a configuration option to limit the number of entries?


E2E: enable pod-svc connection test

What this PR does / why we need it:
Which issue this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged): fixes #
Special notes for your reviewer:
If applicable:

[ ] documentation
[ ] unit tests
[ ] tested backward compatibility (ie. deploy with previous version, upgrade with this branch)

Release note:

E2E: enable pod-svc connection test

See https://circleci.com/gh/Azure/acs-engine/46384?utm_campaign=vcs-integration-link&utm_medium=referral&utm_source=github-build-link for a failed test

Another failed test:
https://circleci.com/gh/Azure/acs-engine/46428?utm_campaign=vcs-integration-link&utm_medium=referral&utm_source=github-build-link


Add API: get_parts_list_properties()

📝 Description of the feature
This function retrieves all properties from the parts list associated with a specific project and CCA name. It enables you to identify attributes, such as whether a part is thruhole or not.
💡 Steps for implementing the feature
No response
🔗 Useful links and references
No response

Hi @NoahWeb24, thank you for your enhancement request. Would the current export_parts_list() API fit your need? Or are you asking for a list of available part properties?


Component: Tag [NDS-35]

A tag is an interactive label that contains text or numbers to communicate status or classification. It may or may not be dismissible.

(split from #41)
Anatomy
A tag includes:

A container
A label
An icon (optional)
A dismiss button (optional)
A link (optional)

Reference

Shopify: https://polaris.shopify.com/components/forms/tag
Salesforce (Pills): https://www.lightningdesignsystem.com/components/pills/, https://react.lightningdesignsystem.com/components/pill-container/
Atlassian: https://atlassian.design/components/lozenge/examples

NDS-35

@sh0ji  I created props for tag component please review and suggest, also I have compared other components common props
Tag Props

Props
Type
Description
Default

close
boolean
Remove/hide tag
No close icon

icon
icon variant
Display icon before or after tag
No icon

disabled
boolean
Disabled tag
false

color
color variant
Text and close icon color
nds-text-color

bgcolor
color variant
Background color
nds-gray-20

link
?
Redirect to route
?

Other components same props

Props
Shopify
Ant
Atlassian

close
onRemove
closable
isRemovable

icon
no icon
icon
elemBefore

disabled
disabled
no disable
no disable

color
no color
color
color

link
onClick
Using child anchor element
href

Issue linked to JIRA Story: NDS-35


Adding eslint in migrate cli

Use a hardcoded .eslintrc.js as the config for now
Install required dependencies

will open a new one for this.


Correction on customer type

Business Central contact filter: Type is Person and the contact is assigned to a company. Dataverse contact filter: The contact is assigned to a company and the parent customer type is Account
Last word should be Customer in stead of Account

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

ID: bf4cd40b-ac70-ebb4-aca9-dea6ad6bf2f8
Version Independent ID: ab3ea9a9-83cc-2367-2e48-eb298f03d0c1
Content: Synchronization and Data Integration - Business Central
Content Source: business-central/admin-synchronizing-business-central-and-sales.md
Service: dynamics365-business-central
GitHub Login: @bholtorf
Microsoft Alias: bholtorf

Hello
Thanks for pointing this out. The topic has been corrected and the update will be published soon.


How does jdk17 need to introduce bc-fips <IP_ADDRESS>

Originally, bc-fips was placed under jdk 1.8 /jre/lib/ext, and now jdk17 does not have this directory, how to operate it?

You'll find relevant info in JEP 220 ("Jigsaw" ≈ modules), which was implemented in Java 9, but I think the general answer is that 3rd-party libraries, like bc-fips, are expected to be found in the classpath: for example declared as a dependency in Maven or Gradle.


Icon spec change

:warning: Validation status: warnings

File
Status
Preview URL
Details

msteams-platform/concepts/apps/apps-package.md
:warning:Warning

Details

msteams-platform/concepts/apps/apps-package.md

Line 10: [Warning] Invalid file link:(~/concepts/apps/apps-upload).
Line 19: [Warning] Invalid file link:(~/get-started/get-started-app-studio).
Line 21: [Warning] Invalid file link:(~/resources/schema/manifest-schema).
Line 60: [Warning] Invalid file link:(~/troubleshoot/troubleshoot).

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Dottk registrar does not work

Can you test and make sure the Dottk registrar does work?
e.g. I have tested boxbilling ver 4.14.2 using Dottk registrar but the domain checking functionality has been broken since version 3.6.11.
http://www.boxbilling.com/forum/plugins-and-extensions/dottk-registrar-failed-setup

BoxBilling will not support DotTk registrar.
It will be removed in next release of BoxBilling

Sigh.... I hope someone will have a fix for this in the future though.

change line 293,             return $result["process_availability_check"]; and it should work

You are right, changing the line you mentioned (actually it is line 281) makes domain checking work.... even though boxbilling can't register dottk domain name.
jebwizoscar, you have any plans to edit the Dottk code or create new registrar using freenom api?

@ericwongcm I don't even have a freenom reseller account :disappointed:
I don't have a Paypal account, so I can't pay it. I only use bitcoin for payments atm.
Also I'm busy with schoolwork, so I may not have time coding (in the near future)

@ericwongcm use this: https://github.com/jebwizoscar/extensions/blob/master/bb-library/Registrar/Adapter/Dottk.php

Well, the link you provided doesn't seems to work. Even the domain name checker is not working whether as is or with that line replaced.
I didn't noticed freedom API needs a paid reseller account. I don't really need to use its API for now, I can register domains manually for the time being.
I am sure someone would be willing to help you pay and get freedom reseller account if you are willing to create the required plugin for freedom and publish it here but that is meaningless if you don't have time to code it.

@ericwongcm I guess I uploaded wrong code...
at least it work for me - demo: http://asac.casa/

mm.. no idea, maybe because you are using huraga theme whereas I am using a custom theme that is similar to boxbilling? Anyway, registration of dotTK doesn't work on your test site as well but domain checking does.
I have posted a message on boxbilling forum and someone might want to donate a paid freedom reseller account in the future, so you can try to write the adapter then.
Anyway, you have my gmail e-mail in your test installation if you need/want to contact me.
If you don't mind, sent me a e-mail, so I can discuss other things with you.

@ericwongcm can you try again? I uploaded the wrong code to my server by mistake :(
just tested with new code, and it's fine
domainshare panel
QWERTYPOI.TK	11-FEB-15	11-FEB-16
email
Received: from localhost.localdomain (unknown [<IP_ADDRESS>])
by fakanava.taloha.tk (Postfix) with ESMTP id 41576BFB2D
for<EMAIL_ADDRESS>Wed, 11 Feb 2015 03:48:06 +0000 (GMT)
Subject: Registration complete for QWERTYPOI.TK
Service Panel
Domain details
Domain	qwertypoi.tk
Registration date	Wednesday, 11 February 2015
Expires at	Thursday, 11 February 2016

@ericwongcm registered poiiii.tk and work too

@jebwizoscar
I will try a little later, the problem now is http://www.dot.tk/ is down and I can't check if domain registered is actually registered or not. I have been having problem accessing dot.tk lately.

@jebwizoscar
I have registered two dottk domain using your installation but both domain are still showing as available from http://www.dot.tk/
Any idea?

@ericwongcm did you check your e-mail?

@jebwizoscar
Yes, nothing in my gmail or spam.
You can see on dot.tk that both domain I registered are still available.

@ericwongcm Oops... default nameserver not set...
check 'I want to use my nameservers' and that should work

@jebwizoscar
Yes, it works now after I check 'I want to use my nameservers' and I receive e-mail from dottk.
That means dottk API still works and the dottk adapter can be fixed rather than using freenom API?

@ericwongcm yes.
https://github.com/jebwizoscar/extensions/blob/master/bb-library/Registrar/Adapter/Dottk.php

@jebwizoscar
Strange, I replaced my Dottk adapter with yours but the dottk domain is not registered.
Do you know why? Are you using boxbilling version 4.14.2?

@ericwongcm check nameserver settings and site e-mail settings (that's the problem I faced today)

@jebwizoscar
I had just tried setting two default name server and also manually inputting the name server when ordering but didn't work... my site e-mail is<EMAIL_ADDRESS>I actually noticed I don't receive any e-mails from my installation...
I think I found a new bug..
When I press "Sent test e-mail to staff members" button, I see this error..
SQLSTATE[42S02]: Base table or view not found: 1146 Table 'prefix_boxbill.mod_email_queue' doesn't exist
Looks like a side-effect from my boxbilling 3.6.11 upgrade. I will re-test this on a fresh database and post another ticket for this new found bug.

@ericwongcm better use a fresh install. some orders won't process if message cannot be sent I think.

@ericwongcm Did you updated BoxBilling to v4 and mod_email_queue is missing? and if you did, was  email extension enabled on previous version?

@vaidas-lungis
I know e-mail was working on boxbilling version 3.6.11. I had previously (a year ago) installed boxbilling and tested e-mail. I can't login to test boxbilling version 3.6.11 anymore because the server I am using is missing "ionCube loader". I can't be bother contacting server admin to fix it since I won't be using boxbilling version 3.6.11 anymore.

Could you try to run following query in your database? It should create missing table for you.
CREATE TABLE `mod_email_queue` (
  `id` int(11) NOT NULL AUTO_INCREMENT,
  `to` varchar(255) NOT NULL,
  `from` varchar(255) NOT NULL,
  `subject` varchar(255) NOT NULL,
  `content` text NOT NULL,
  `to_name` varchar(255) DEFAULT NULL,
  `from_name` varchar(255) DEFAULT NULL,
  `client_id` int(11) DEFAULT NULL,
  `admin_id` int(11) DEFAULT NULL,
  `priority` int(11) DEFAULT NULL,
  `tries` int(11) NOT NULL,
  `status` varchar(20) NOT NULL,
  `created_at` datetime NOT NULL,
  `updated_at` datetime NOT NULL,
  PRIMARY KEY (`id`)
) ENGINE=MyISAM DEFAULT CHARSET=utf8;

@4Giedrius
Thanks... the query works and test e-mail works now.

@jebwizoscar
I think I know why the new dottk adapter is not working. After I clear my database and do a clean install for the database, I noticed the dottk adapter is missing in the list of Domain registrars.
I only see these three domain name adapter in the list of registrars
Custom
Internet.bs
Reseller Club
Can you check if you see the same on your installation?

Are you aware that you need to go to Configuration -> Domain management -> New Domain Registrar and you will be able to install from there?

@4Giedrius
Thanks for the information, I had forgotten or over-look this option while testing, I see the Registrars now.
@jebwizoscar
Can you check again if your test server is indeed running the same code on the github link you posted? because I have retested this with fresh database and double check that I am using the dottk adapter you posted BUT even domain checker is not working.

https://github.com/boxbilling/boxbilling/pull/295    DotTK support is now functional with this pull request

@hackersrage I tried the DotTK you have submitted but it still does not work when I test it on my server. Even domain checker is not working. I suspect that my inability to use your version and jebwizoscar's version of the DotTK adapter is related to the php version my server is using.
I am using php 5.3.19. I have no idea what you both are using.

Make sure your credentials are correct. I tested this on PHP 5.2, and 5.4.  Github tested it against multiple php versions. The version of BoxBilling used is the /master branch.

@hackersrage Could it be the dotTK adapter you submitted won't work with Boxbilling version BoxBilling 4.15.7? I am not using the /master branch, I am using BoxBilling 4.15.7 release.
I have double check my dotTK login and it is correct.
This is the response I get from the server I had installed boxbilling.

PING REPLY
2015-03-22 05:54:39 UTC

OK

I havn't tested with earlier revisions of BoxBilling. Just found out about this project a few days ago.  It is working fine from here, and while testing, I noticed that some of the responses coming back from the registrar contained HTML, which I then strip as part of this patch.  The message above is good as it shows your server can communicate fine.  Double check the credentials by logging into their reseller portal, and try updating to master branch.  Please let me know.

Well, I have just updated my boxbilling to the latest release of 4.15.9. When I use the Dottk adpater you committed, I get a more useful message now that says "Order status	Failed Setup" on the order page, i.e. .......index.php?_url=/order/service/manage/6
It is stated somewhere that you should use the release version of Boxbilling, not the master branch.
Maybe I need to wait for the next major release of Boxbilling before this adapter will work then.

@hackersrage
I have just updated my boxbilling to the latest release of 4.16.2. Using your Dottk adapter, domain checker seems to work ok now but I still get "Order status Failed Setup" on the order page.
Does anyone know where I look up the details of the failure?
Btw, the Dottk adapter is not included in the boxbilling 4.16.2.

Please try to enable debug mode in your bb-config file and check error log for more information why order is failing.

I suppose you mean this line in bb-config?
'debug' => true,
And I am supposed to see something in error_log file at the same folder as bb-config?
but I don't see anything in error_log file after debug is enabled.


Fixed documentation for blas/gonum where old documentation referred to package as native

Doc still referred to this package as native. It's now fixed

Already in #193


Bicep Output (containerAppsEnvironment): The language expression property 'runtimeSubnetId' doesn't exist

Bicep version
Bicep CLI version 0.13.1 (e3ac80d678)
Describe the bug
output container app environment vnetconfiguration returns error :
"The template output 'virtualNetworkRules' is not valid: The language expression property 'runtimeSubnetId' doesn't exist, available properties are 'internal, infrastructureSubnetId, dockerBridgeCidr, platformReservedCidr, platformReservedDnsIP'..."

To Reproduce
Steps to reproduce the behavior:
resource containerAppsEnvironment 'Microsoft.App/managedEnvironments@2022-03-01' existing = {
  name: coreInfrastructureContainerAppsEnvironmentName
}

output virtualNetworkRules array = [
  {
    id: containerAppsEnvironment.properties.vnetConfiguration.runtimeSubnetId
    action:'Allow'
  }
  {
    id: containerAppsEnvironment.properties.vnetConfiguration.infrastructureSubnetId
    action:'Allow'
  }
]

Additional context
This template has had no changes and was working 2 days ago

It's possible the Container Apps API stopped returning this value. I would recommend opening up a support case so this can be routed to that team.
In the meantime, can you see if this workaround resolves the issue?
resource containerAppsEnvironment 'Microsoft.App/managedEnvironments@2022-03-01' existing = {
  name: coreInfrastructureContainerAppsEnvironmentName
}

output virtualNetworkRules array = [
  {
    id: reference(containerAppsEnvironment.id).vnetConfiguration.runtimeSubnetId
    action:'Allow'
  }
  {
    id: containerAppsEnvironment.properties.vnetConfiguration.infrastructureSubnetId
    action:'Allow'
  }
]

Closing this as we haven't had a response. Please feel free to re-open if necessary.


feat(community): Port ArxivRetriever to LangChainJS

Description:
This PR ports ArxivRetriever from LangChain Python to LangChainJS.  An ArxivRetriever component would grant users access to up-to-date research directly from arXiv. This retriever would allow users to pull full text and summaries through natural language queries or arXiv identifiers. Specifically, we have added

Arxiv utils & retriever implementation,
documentation for ArxivRetriever,
integration tests for ArxivRetriever,
example usages for ArxivRetriever.

Issue:
This PR addresses https://github.com/langchain-ai/langchainjs/issues/7095

Thanks!


PD Flexible Steps: Substeps for new flexible step

Acceptance criteria
Substeps and step list interactivity should work consistently with the new flexible step, very much like step list + substeps work now for transfer/consolidate/distribute.
Details
For the new step, the path determines the substep style instead of the stepType.
path: 'single' -> acts like  'transfer' stepType did
'multiDispense' -> 'distribute'
'multiAspirate' -> 'consolidate'

The substeps shouldn't need to be visually changed, I think we just need to replace conditionals that used stepType with ones using stepType + path when the stepType is moveLiquid.
Also things like labware highlight-on-hover and substep highlight-on-hover should function the same as before for the new step.
Main files that need to be updated (not necessarily exhaustive):

getHoveredStepLabware selector in protocol-designer/src/ui/steps/selectors.js
protocol-designer/src/top-selectors/substep-highlight.js
protocol-designer/src/steplist/generateSubsteps.js
protocol-designer/src/components/steplist/StepItem.js

Closed in #2985


FST works

What problem does this PR solve?
FST unit test pass. phase 2 of #359
Issue link:
What is changed and how it works?
Code changes

[x] Has Code change
[ ] Has CI related scripts change

Check List
Tests

[x] Unit test
[ ] Integration test
[ ] Manual test (knn performance test)
[ ] No code

Side effects

[ ] Performance regression: Consumes more CPU
[ ] Performance regression: Consumes more Memory
[ ] Breaking backward compatibility

Note for reviewer

@mergify queue


fix: improve capitalize code coverage

I've been digging in the function and add more test cases to improve coverage.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+9.2%) to 88.737%

Totals

Change from base Build<PHONE_NUMBER>:
9.2%

Covered Lines:
101

Relevant Lines:
106

💛  - Coveralls


Uncaught Exception: A server with the specified hostname could not be found.: (null)

An uncaught exception was reported.
Info:
OS: darwin
Arch: x64
GPMDP Version: 3.0.0
Time: Mon Mar 07 2016 11:18:59 GMT-0500 (EST)
Error:
A server with the specified hostname could not be found.: (null)
Error: A server with the specified hostname could not be found.: (null)
    at Error (native)

This issue was created automatically inside the uncaughtException handler

Client side network issues


Add conda install option to README.md

Add conda install option in docs.
Proposed changes
This PR updates the README file with the installation option for installing mlx with conda from conda-forge channel.
Checklist
Put an x in the boxes that apply.

[x] I have read the CONTRIBUTING document
[x] I have updated the necessary documentation (if needed)

I'm not sure we should advertise this in the README as it appears the conda forge distribution is lagging the pypi one.
@xhochy and @ngam is the update waiting for review?

(As an aside, @sugatoray is a great conda-forge contributor who once made a nifty web tool that made converting pip packages to conda packages a breeze 👍)

Thank you, for the shout out @ngam. Please feel free to add me to the list of maintainers in mlx-feedstock.

Thank you all for the thoughtful responses! I'd really like to keep MLX updated on conda-forge. So we'll do our best not to break things for you again!

If it makes sense to add me as a maintainer I am up for it. I don't know much about your system yet so I probably will not default to be the first reviewer but I'm happy to be in the loop.

@awni Thank you for review/merging the PR.


[VShare] - ERROR: Unable to extract title;

$ youtube-dl --verbose https://vshare.io/d/bjg0fc7
[debug] System config: []
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: ['--verbose', 'https://vshare.io/d/bjg0fc7']
[debug] Encodings: locale UTF-8, fs utf-8, out UTF-8, pref UTF-8
[debug] youtube-dl version 2019.02.08
[debug] Python version 3.6.5 (CPython) - Linux-4.14.98-gentoo-x86_64-AMD_FX-tm-8350_Eight-Core_Processor-with-gentoo-2.6
[debug] exe versions: ffmpeg 4.1.1, ffprobe 4.1.1, rtmpdump 2.4
[debug] Proxy map: {}
[VShare] bjg0fc7: Downloading webpage
ERROR: Unable to extract title; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; see  https://yt-dl.org/update  on how to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.
Traceback (most recent call last):
File "/usr/lib64/python3.6/site-packages/youtube_dl/YoutubeDL.py", line 794, in extract_info
ie_result = ie.extract(url)
File "/usr/lib64/python3.6/site-packages/youtube_dl/extractor/common.py", line 508, in extract
ie_result = self._real_extract(url)
File "/usr/lib64/python3.6/site-packages/youtube_dl/extractor/vshare.py", line 54, in _real_extract
r'([^<]+)', webpage, 'title')
File "/usr/lib64/python3.6/site-packages/youtube_dl/extractor/common.py", line 992, in _html_search_regex
res = self._search_regex(pattern, string, name, default, fatal, flags, group)
File "/usr/lib64/python3.6/site-packages/youtube_dl/extractor/common.py", line 983, in _search_regex
raise RegexNotFoundError('Unable to extract %s' % _name)
youtube_dl.utils.RegexNotFoundError: Unable to extract title; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; see  https://yt-dl.org/update  on how to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.

Same thing for me. I think it may have to do with the website changing a few things, now the scraper doesn't work.


Nn 1003 whereabouts blank screen

The activity list and house block list screen both share the same reducer. The event data is stored in a houseblock.data. When navigating between pages the data isn't cleared and would result in one screen trying to process data that's in the wrong shape. The fix is to store data into it's own property.


elevate-service: implement patching of native services and their roles

Cleaned up #71

@Informatic
Tested on my WebOS5 TV.
There is a * missing in the allowedNames directive.
We haven't permissions to register as different clients:
Executable: "/media/developer/apps/usr/palm/services/org.webosbrew.piccap.service/hyperion-webos" (cmdline: /media/developer/apps/usr/palm/services/org.webosbrew.piccap.service/hyperion-webos) does not have permission to register name: "com.webos.service.capture.client4922"
You can take a look here to get an idea of what I mean: https://github.com/webosbrew/webos-homebrew-channel/blob/24653abfa8b3841d238534ee5a65a6135c6ed7d1/services/elevate-service.ts#L112

Ah indeed. That's my bad - on webOS 3.x native app/service permissions are in /var/palm/ls2-dev/roles/pub and they wrap allowedNames in role object as well. Fixing it right now...

@TBSniller Please take a look at the note in patchRolesFile - this seemed to work for libvtcapture on webOS 6.2.0.
Wouldn't it probably make sense to add an option to override these allowedNames via commac/colon-separated environment variable?


Bugfix/skill help modal

Resolves #1117
Notes:

Replaced the existing div inside the skill partial with clone modal markup on the index template (single modal instead of one per skill)
Updated the button on the skill partial
Updated the Laravel translation files
Regression test coming

took a peek and fixed the styling issues here. looks good now!


MINOR: Connect Javadocs improvements

Some common themes:

Replace blank lines (which are ignored) with <p> to separate paragraphs. For those wondering about the omitted closing </p> tags, this is actually the recommended style (see here).
Add Javadocs to some (but not all) classes that were missing them.
Fix typos and grammatical issues.
Add {@link} to code references wherever applicable.
Replace manually numbered lists with <ol>.

For instance, the AbstractHerder's rendered Javadoc before:

After:

Committer Checklist (excluded from commit message)

[ ] Verify design and implementation
[ ] Verify test coverage and CI build status
[ ] Verify documentation (including upgrade notes)

Thanks Mickael!


Load results from proposals if available

TODO:

[ ] Store votes VP on db
[ ] Load only top 10 votes then paginate
[ ] Update scores when there is a new vote or scores is older than 30 seconds

@mktcode Thats the idea, but will need few changes


onSubmitted called with null when initial value was not changed

Steps to reproduce:

Open TimeRangePicker without default value for endTime
Select start value
Do not set an end value
Tap "OK"

Expected result:
onSubmitted is called with whatever was shown in the END picker
Actual result:
onSubmitted is called with endTime == null

@tobilarscheid
Thanks for your feedback.
Added api unSelectedEmpty . when not selected time, return null  or now time.
Released the supported version 0.0.2.
https://pub.dev/packages/simple_time_range_picker
Please unSelectedEmpty is set false after upgrade to version "0.0.2".
RaisedButton(
  child: Text("OpenPicker"),
  onPressed: () => TimeRangePicker.show(
    context: context,
    unSelectedEmpty: false,
    onSubmitted: (TimeRangeValue value) {
      //
    },
  ),
),


[Feedback]: Problems with vtt2ass on new Q style Hidive subtitles

Type
Both
Suggestion
There are 3 main new issues with the new Hidive subtitle format that uses Q1, Q2, etc. I'll refer to Urursei Yatsura ep 34 for examples.
\N false negative
The "\N" adding code used checks if lines start at the same time but, events can be rounded to different centisecond values causing them to not have the same start time, thus not be joined.
Q0
00:21:42.333 --> 00:21:44.041 line:70%
<c.Q0>Is that... tempura I smell...?</c>

Q28
00:21:42.334 --> 00:21:44.041 line:77%
<c.Q28>What? But selling tempura</c>

Q28
00:21:42.335 --> 00:21:44.041 line:84%
<c.Q28>door-to-door is unheard of!</c>

The two Q28 lines should be joined here but they are rounded to different start times.
Dialogue: 0,0:21:42.33,0:21:44.04,Q28,,0,0,0,,What? But selling tempura
Dialogue: 0,0:21:42.34,0:21:44.04,Q28,,0,0,0,,door-to-door is unheard of!
Dialogue: 0,0:21:44.04,0:21:45.46,Q0,,0,0,0,,Is that... tempura I smell...?

Reverse order on all simultaneous lines
The vtt converter adds lines from top to bottom but ass renders the first line at the bottom and then stacks the next one on top so every simultaneous line event shows up on screen in reverse order. Example, this and also the last example:
Dialogue: 0,0:15:39.21,0:15:42.46,Q28,,0,0,0,,Lum! Lum! Lum! Lum!
Dialogue: 0,0:15:39.21,0:15:42.46,Q0,,0,0,0,,Aw, jeez. These things are so obnoxious!

Caption events can be joined by \N
When they are joined by \N, their position tags don't work so they are not positioned properly. Example:
Dialogue: 0,0:22:45.63,0:22:50.33,Q10,,0,0,0,,{\pos(435.2,165.6)}{\an7}L♥vely Darling in Danger!!\N{\pos(460.8,259.2)}{\an7}Foxes in the Moonlight\N{\pos(460.8,345.6)}{\an7}The Home Visit Blues:\N{\pos(422.4,396)}{\an7}Mark Onsen Goes to Space

Thanks for this detailed feedback! I was quite worried about potential problems with this new logic, but wasn't sure in what form they would be. I'll look into implementing the below fixes

\N false negative

This can probably be fixed by checking for +- 1-3 centiseconds.

Reverse order on all simultaneous lines

I noticed this issue while I was adding the new Hidive API, and it was the cause for adding the multiline subtitle merging logic in the first place (It seemed like the easiest fix).

Caption events can be joined by \N

This can probably be fixed by checking for the \pos tag and ignoring the subtitle if present

Alright, so I started looking into the false negative, and ran into a weird issue. For me, the time only goes up to the 2nd decimal point, it doesn't go up to 3 decimal points. What OS and version are you using (prebuilt binary? code? node version?)?
I did fix the issue with subtitles with positional data being merged though.

For the vtt? I just checked it through the website directly, not sure if it's different for api.

Alright, next release should have the fixes. If you are able, could you compile the latest commit and give it a test?

Unfortunately, I don't have a CDM to test Hidive but looking at the code, it looks like problem 1 and 3 should be solved. There's still the issue in problem 2 that I don't think is addressed though unless I'm missing something. When they are different styles, the lines won't be joined and they will display in the wrong order.
Either you could reverse the order the lines in the .ass (then you have to be careful about the differently rounded start times) or you could try to add the \N\r style thing back to preserve the order.

I think fixing 1 should effectively fix 2, but correct me if I'm wrong. Also is there a source on .ass rendering things backwards? Closest I could find was this: http://www.tcax.org/docs/ass-specs.htm under general information number 4, which says that theoretically the order of events shouldn't matter? And when I check .ass subtitles on say Crunchy, they are on chronological order rather than reverse order

Not so much backwards, it's more like it does it stacks from bottom up intentionally
Dialogue: 0,0:15:39.21,0:15:42.46,Q28,,0,0,0,,Lum! Lum! Lum! Lum!
Dialogue: 0,0:15:39.21,0:15:42.46,Q0,,0,0,0,,Aw, jeez. These things are so obnoxious!

is like this on playback

But on hidive it displays like this:

Looked into it a little more, and this is likely because we're not parsing the line % currently for non captions, at least that was the logic for the old API when we could tell what was captions and what weren't.
If line 337 is changed to this:
let type = txt.style.match(/Caption/i) || txt.style.match(/Q/i) ? 'caption' : (txt.style.match(/SongCap/i) ? 'song_cap' : 'subtitle');

It parses the line %. But for whatever reason, hidive has their line %'s unusually high, and I think some offsetting is going on in their player. So the subtitles end up super high like this:

Those two generated lines with that enabled:
Dialogue: 0,0:15:39.21,0:15:42.46,Q28,,0,0,0,,{\pos(640,554.4)}Lum! Lum! Lum! Lum!
Dialogue: 0,0:15:39.21,0:15:42.46,Q0,,0,0,0,,{\pos(640,604.8)}Aw, jeez. These things are so obnoxious!

I'm not sure inverting the subtitle list is the right call, but I could give it a shot. It would theoretically be as easy as changing line 190 to this:
const lines = vttStr.replace(/\r?\n/g, '\n').split('\n').reverse();

and then inverting the subtraction of currentStart and previousStart

I also don't think \pos on every line is a great idea. You just need to reverse lines with the same start time (or within a 1 or 2 centiseconds of each other). So maybe just do a second pass to reorder those lines.

Alright I think (hope) that the above works. Lol

I think we're almost there. You're gonna hate me but I have an edge case here where that might not work.
https://www.hidive.com/video/586601?seasonId=20687
Q0
00:00:24.833 --> 00:00:26.291 line:70%
<c.Q0>Tsk...</c>

Q2
00:00:24.834 --> 00:00:26.291 line:77%
<c.Q2>Whoa, look at it go!</c>

Q0
00:00:24.835 --> 00:00:26.291 line:84%
<c.Q0>It's gettin' everywhere!</c>

Basically it's not gonna work for 3+ lines with same start it'll be
line 1 -> line 2
line 2 -> line 3
line 3 -> line 1

instead of
line 1 -> line 3
line 2 -> line 2
line 3 -> line 1

Also if they get rounded to different start times, that might also mess up the order so that's what I was saying about aligning them to the same start time if they're similar enough. Sorry to be so specific about this issue, I work with hidive subtitles for some of my projects so I just want this to be correct, thanks for being responsive.

After a lot of thinking, I couldn't come up with a better method that could actually be implemented without massively overcomplicating things. So I have now inversed all the subtitles, which will hopefully be more accurate (thanks hidive for inverting subtitles that have the same start time). I guess let me know if that actually works as intended, but initial tests looks like it might.


Add P5js to the list

P5js would be a nice addition to the list.
I've used it to create some UIs, here and here, in the past.

Sounds great!
https://github.com/processing/p5.js (15.6k stars)


Growth policy misinformed by concurrent sweep

Old-space growth policy is informed by available capacity. However, with concurrent sweeping, pages are freed and capacity adjusted after/while a GC is evaluated (see PageSpaceController::EvaluateGarbageCollection), which may lead to unintended and unpredictable decisions.
At a more basic level, the tweak "allow growth by at least half the pages freed by GC" is inherently incompatible with having a concurrent sweeper.

There should always be a reasonable "best estimate" of usage/capacity, which can be adjusted incrementally (and thread-safely) during concurrent sweeping (and soon, marking). This way, it should always be possible to evaluate the growth policy.

When you have some time please review this @ErikCorryGoogle and close it if no action is planned.


A general purpose STRtree which can store any Python object

This STRtree can store almost anything and is backwards compatible with the one in 1.7. If initialized with a sequence of geometry objects, these are stored, and query and nearest return a list of geometries or a single geometry. If initialized with a sequence of (value, geom) pairs, the values are stored, and query and nearest return lists of these values or a single value. The values could be any Python object and don't need to be the same type.
In #1064 we have an STRtree that takes a sequence of geometry objects and stores their enumeration indices. Query returns a list of indices or a list of geometries (found by index from an internal list) depending on a flag.
The STRtree in #1064 and in this PR each have internal lists maintaining references to the initial data.
My understanding of the STRtree in #1064 is that it is designed to take a sequence of geometry objects (a DataSeries, yes?) from GeoPandas and store their enumeration indices. Syncronization of state between the enumeration indices in the tree and the DataFrame row indices would be required. The way GeoPandas would use the #1094 STRtree is to store DataFrame row indices directly. I believe this would make synchronization less fragile.

@sgillies thanks for getting back to this!
Now, I think I don't yet fully understand how this query_cb would be used in practice by the user.
First, it requires manual usage of ctypes (and knowing the types of the pointers that the callback function receives), and I am not sure if we want that users have to deal with ctypes? (certainly given that we are removing the use of ctypes in Shapely 2.0).
Further, I don't think this actually allows to get an integer index as the result? The callback function can only work with the data that was inserted into the tree, and at the moment that is still hardcoded to be the actual geometry object.

Indeed I need to think more about the callback. We can't ask users to navigate the Python/C issues. Perhaps requiring callbacks to be a method of an Extension type would help, they could use a helper method of that type to do conversions. I'll stop waving my hands now and come up with some running code.

Finally got the time to get back to this and overcome some of the issues. First, this PR will let us store any Python objects we want in the tree. Geometries, numbers, strings. Even a mix, which is not something one would normally do. The query_cb function takes an ordinary Python function as a callback and all the conversion between Python and C types is done by a decorator. A related decorator is used in STRtree.nearest, which also now supports trees that contain non-geometry objects.

@jorisvandenbossche @mwtoews I've made some major improvements and have rewritten the introductory comment in this PR to better explain the intent and direction. I think this implementation covers existing features, the features in #1064, and more.

Thanks, with those updates, the intent and how it would be used is much clearer now ;) (and avoids using ctypes as a user!)
A few follow-up questions / comments:

Do you have use cases in mind where the user might want to get something else returned as either the geometry itself or either some ID? And a similar question for the ability to specify a custom callback function.
I am mainly looking at this from the point of view of what we need in geopandas (where we wouldn't need that), so this is of course a quite limited view! But at the same time, I think we should be cautious in implementing features "because it's possible", and think about practical use case if adding this.
(Because adding such flexible features also has a cost. We should probably need to try it out to know for sure, but my feeling says that implementing the same features in C for 2.0 would give quite some additional complexity.)

Regarding the ability to "store any Python objects we want in the tree": I think I mentioned this before in https://github.com/Toblerity/Shapely/pull/1064, but I want to mention again that to achieve this functionality, those objects don't necessarily have to be stored in the GEOS C tree, but could also be stored in the Python tree object.
If we want to enable this functionality for the end users, I think it is possible to keep track of the values on the python side, and still only store the actual geometries in the GEOS STRtree (as is currently done in pygeos). And based on the indices returned from this wrapped C tree, the python layer can still return any Python object we want.

I know that the GEOS STRtree explicitly allows storing generic values that get returned. But using Python objects for that instead of indices (or actually pointers, as currently done in pygeos), will still be more complicated, and for example won't allow to release the GIL for the query (I assume). Also a generic callback function that calls a user-provided python callback won't be able to release the GIL, so that might mean we need two different code paths to support both those cases.

Only read the updated top post now, so a few comments/answers on that:

In #1064 we have an STRtree that takes a sequence of geometry objects and stores their enumeration indices. Query returns a list of indices or a list of geometries (found by index from an internal list) depending on a flag.

A bit related to my comment above, but the actual API in that PR (different return value depending on a flag) is only "a" choice out of a set of options that I made when opening the PR, but is certainly something to be discussed. I think we are maybe discussing two (potentially) different aspects, which might be useful to separate. One is the actual implementation under the hood (i.e. how pygeos has implemented it in C / how #1064 does it in the current ctypes implementation), and the other is the user-facing API (eg the query method with a flag as in #1064, or a new query_indices method, or letting the user insert the value to be returned explicitly as in this PR, etc).
My feeling is that we can separate those two aspects, and could implement any user facing API we want (maybe except the custom callback) on top of a low-level C STRtree that returns indices (which is probably the simplest to implement on that lower level).

My understanding of the STRtree in #1064 is that it is designed to take a sequence of geometry objects (a DataSeries, yes?) from GeoPandas and store their enumeration indices. Syncronization of state between the enumeration indices in the tree and the DataFrame row indices would be required. The way GeoPandas would use the #1094 STRtree is to store DataFrame row indices directly. I believe this would make synchronization less fragile.

The STRtree in #1064 just takes a sequence or iterable of geometries as input, which is exactly the same as the current version. So it's not specifically targetted at receiving a geopandas Series.
Also, as a clarification, in geopandas we don't really want to store/get row indices (generic labels, which can be of any type), but rather integer positional indices (pandas allows you to index a dataframe in two ways: the actual row labels vs integer positions). This is because using integer positions is more performant (and also more robust, eg in theory you can have duplicate labels).
I also don't think this is specific to geopandas, but more general when working with arrays of geometries. I know that eg @brendan-ward has mentioned he has already been using pygeos' strtree a lot in his work for fast spatial join like operations.

Thanks @sgillies for your work here and @jorisvandenbossche for keeping the conservation going!
I am likely biased, but I think keeping the underlying C STRtree limited to enumeration indices gives us a simpler API at the C level, avoids memory traps and GIL-related issues, and yet still provides a very flexible API at the Python level.  Those indices can always be used to return arbitrary values (e.g., original index for a DataFrame, the geometry at each index, some other value) at the Python level: just keep an indexable array (i.e., numpy array of dtype appropriate for values) of those values in the Python tier, and return those values using the enumeration indices depending on some parameter or a specific function layered on top of the C STRtree.
My hunch is similar to @jorisvandenbossche that storing other things directly in the C STRtree introduces a good bit more complexity and isn't necessary.  The slow parts are the underlying tree query and predicate operations against the results; indexing back into an arbitrary array of values afterward should be very fast.  I.e., we probably wouldn't see a performance improvement by storing the values you want back directly in the C tree (those may actually have a performance impact depending on how they are tracked there).
I have been using the enumeration indices heavily for a variety of different operations; even if the geometries originate from DataFrames or arrays, sometimes I just use these directly, sometimes I use them to get the original DataFrame index, and sometimes I use them to get the geometries or something else.  There is little overhead to using enumeration indices to get to any of those.  So I'm a huge fan of the enumeration indices approach at the core.

we can separate those two aspects, and could implement any user facing API we want (maybe except the custom callback) on top of a low-level C STRtree that returns indices

I concur with this; the substantive part of the API discussion should be at the Python level, and there should be easy ways to return either the enumeration indices, the geometries, or arbitrary values based on the underlying enumeration indices.  This can also be a rich source of functionality in Shapely and be easier for contributors than the underlying implementation in C.  Both this and #1064 have elaborated the need for multiple types of results from the tree.
I'm unclear on the use cases around custom callbacks implemented in Python.  My first instinct is that I'd really want to keep these at the C level.  There are issues around memory management and releasing the GIL lurking here.  My preference would be to identify clear use cases for other callbacks that are needed and implement those at the C level and expose those through a well-defined API (also based on enumeration indices) back to the Python tier.
Now that predicates are implemented in query and query_bulk methods at the C level, I'm having a hard time thinking of other things I'd want to handle in a tree query callback.  The tree doesn't allow us to exit early on first result (which could be helpful for certain analyses), so other than tracking the results that you get back from the underlying GEOSSTRtree_query_r or some derivative of input geometries or values that you could do after getting enumeration indices from the tree, I don't know what else you'd do in a callback.
Likewise, the custom callback we had to work out in pygeos to support nearest_all (return all nearest equidistant results and not just the first, limited by an optional max distance) was a bit tricky and definitely needed to be done at the C level.
What are some of the use cases you had in mind for custom callbacks?

@brendan-ward @jorisvandenbossche does storing only enumeration indices require the tree to be immutable? In #1104 I'm considering a tree that supports insertion and removal of items. It seems to me that in #1064 insertion must then be implemented as a strict append to the internal list so that indices already stored in the tree remain valid. I'm not sure if that breaks any pygeos requirements. Internally, item removal would be less constrained. If the item is left in the internal array/list all the indices remain valid.
GEOS could be modified to allow us to break out of the loop at https://github.com/libgeos/geos/blob/master/src/index/strtree/SimpleSTRtree.cpp#L226 but until then there's no great way to return early using a callback. My interest in the callbacks was coming from an interest in seeing if we could shift responsibility from mapping pointer addresses stored in the tree to Python objects onto the caller. But if we're going to release the GIL we cannot do that. Maybe it's best to shelve the callback idea for now. If we do change GEOS to allow an early return, we could add a callback keyword argument to query and switch to a branch that doesn't release the GIL. Yes?

@sgillies It is my understanding that STRtree trees are immutable after first query:

once the tree has been built (explicitly or on the first call to query), items may not be added or removed

I know there are ongoing changes to the STRtree implementation, but I wasn't aware of any change to that constraint.
Pygeos expects the tree inputs to be immutable.
However, in my experience, building trees is very fast even for large (>1M) datasets, so I think it is entirely feasible to simply discard and rebuild the tree instead of trying to handle inserts.  Ideally, mutations should be batched to avoid rebuilding the tree on every change.
What is the use case for adding / removing items from the tree after building it?
If GEOS STRtree is modified, this would be through the addition of new functions against STRtree, so support would need to be built into the C wrapping in pygeos / shapely at that time - and that would be the place to provide an additional function to the Python tier to handle this case.  In Python, it could be exposed to users as a keyword parameter if appropriate, but use different underlying C functions.  This still allows us to leverage address-based indexing, release the GIL, and keep that complexity at the C level.


Change min sdk to 23

Current Implementation
In this moment the app supports only Android versions >= 8 .
Suggested Enhancement
With desugaring and downgrade of the zxing library is possible to downgrade the min sdk to 23
Expected Benefits
People with android version >= 6 can download the app from the store

Uhm, I think that some of the "desugaring" process was done in https://github.com/ministero-salute/it-dgc-verificaC19-android/pull/71 , can you check? Maybe some work can be done after that PR is merged (I have an Android 6 device, I can test it).
Note that Android below 7.1.1 will break somewhen between now and 2024 due to LE certificate expiration (let's encrypt is virtually everywhere nowadays). While I support your issue and I sincerely hope that we don't need this app in 2024, I think that it would be wise to update the phone :-)

@francescogatto personally I think that downgrading zxing library is not a good idea. This library is vital for VerificaC19.
Android 6.x has a market share of ~4%, while Android 7.x is around 8%. I think that supporting Android 7.x devices is enough for a verifier app and the PR mentioned by @Enrico204 could support Android 7.x without downgrading zxing core lib. What about changing the topic of this issue to "Change min SDK to 24"?

Hello @astagi
According to preliminary tests on real devices in range API23<->API30, partial Zxing downgrade isn't breaking the app's UX.
I've submitted anyway a related PR #113 with tested commits on release 1.1.0 (versionCode 571) just to allow you to check.
By the rest, even though further minSDK downgrade to API Level 21 (Lollipop 5.0) - minimum limit threshold according to DGC core decoder - is technically possible,.. I can't recommended it at all - according to tests on real devices.
Android Keystore supports indeed AES only since API Level 23 - Marshmallow 6.0 .
Therefore, setting minSDK 21 for appconfig first required to rewrite DefaultKeyStoreCryptor & SecurityWrapper in order to :

Generate RSA pair of keys (using deprecated KeyPairGeneratorSpec API)
Generate AES key (using default/other crypto provider)
Encrypt the AES key using RSA
Store the encrypted AES key in SharedPreferences
Manage the different AES key storage/retrieval flow

Then it required anyway additional if-checks in order to prevent KEY_ALIAS null exceptions on some pre-M legacy devices, according to the encryption settings of the device.

Note that downgrading zxing might reintroduce some bugs, and re-implementing DefaultKeyStoreCryptor and other classes will require extensive testing.
If supporting older Android versions is necessary, I suggest to duplicate/fork the app and build a "legacy" version, targeting old Android SDKs.

Hello @Enrico204

Note that downgrading zxing might reintroduce some bugs, and re-implementing DefaultKeyStoreCryptor and other classes will require extensive testing.
If supporting older Android versions is necessary, I suggest to duplicate/fork the app and build a "legacy" version, targeting old Android SDKs.

Honestly, downgrade to minSDK 21 doesn't make sense IMHO too = even re-implementing DefaultKeyStoreCryptor & SecurityWrapper for AES+RSA logic/flow, there will always remain the (known) performance & reliability flaws with API deprecated methods for keystore management on pre-M legacy devices.

Downgrade to minSDK 23 is technically feasible without losing performance & reinstating critical bugs, by working in partial downgrade for Zxing libs (limit core min 3.3.0).
However, it would require anyway further testing on real devices API 23<->30.
Until now I haven't got runtime issues with the shared/applied commits, but I'm still in the preliminary testing phase with few (less than 20) real device configs...

Therefore, I agree with @astagi = at the present time, downgrade to minSDK 24 is the only available option, that can be applied in a short time & without critical bugs/issues - just because of how API desugaring works.
maximum yield less effort in order to extend the app support to API 24 & 25 targets (<6% current working Android shares Italy).
By the rest, we must not forget that VerificaC19 is NOT an app aimed to the widest possible userbase.
Developers could even spend time to work on a separated legacy build for API 21-22-23... but we are anyway talking about 4% of current Android distribution shares for working devices in Italy (according to compared stats of Google Play - StatCounter - Statista).
"The game is not worth the candle" - obviously, still IMHO_

Hi @rawmain sorry for my late response, I'm currently on vacation. You did a great work on your PR, very interesting! I let @Light2288 to review your changes but I think at the end they are going to support devices with API >= 24. As you mentioned VerificaC19 app is not aimed to the widest possible userbase.


Can't use ; (semicolon) in SelectToEndOfLine keybinding

Check for existing issues

[X] Completed

Describe the bug / provide steps to reproduce it
Using the ';' (semicolon) character does not seem to have any effect.  For example:
      "ctrl-cmd-shift-;": [
        "editor::SelectToEndOfLine",
        {
          "stop_at_soft_wraps": true
        }
      ],

does not work.  Change the ';' character to a letter, the binding performs the desired action.
Environment
MacOS Ventura
Zed 0.93.4
If applicable, add mockups / screenshots to help explain present your vision of the feature
No response
If applicable, attach your ~/Library/Logs/Zed/Zed.log file to this issue.
If you only need the most recent lines, you can run the zed: open log command palette action to see the last 1000.
No response

It should be mentioned:
`"ctrl-cmd-;": "editor::MoveToEndOfLine",`

does work as expected.

You know that shift + ; = :
      "ctrl-cmd-:": [
        "editor::SelectToEndOfLine",
        {
          "stop_at_soft_wraps": true
        }
      ],

Thanks, that solved it

It looks like this has been solved, so I will close this one out.


[1.24.1 & 2.0] Image list not working

Hi,
I recently updated to version 1.24.1 and then to 2.0 but in both versions the list of downloaded images is not visible:
on the web I only see this image:

from the console I see all the images downloaded / created:

for update an image from the web I have to do:

Go to the created container and then to container details image
Click on the image and then update

I tried to make a new container and tried to download all the images again, both from the console and from the web but it doesn't work.
in the home page the images are recognized:

The only thing I haven't tried is to delete everything, including the database.
Solutions?
Thanks.

Hi
Thanks for reaching out. pl let us know the docker swarm version.is it possible to reinitialize  the swarm cluster and check

Hi,
today I tried to create a new container for portainer by deleting the volume data with database, but I have the same problem;
i go back to version 1.23.2 and it works without problems
Technical details:

Portainer version: 2.0.0 (upgrade 1.23.2 to 1.24.1 and then 2.0)
Docker version: 19.03.13 (API: 1.40)
Platform: Debian GNU/Linux 9.13 (stretch) x86_64
Command used to start Portainer: docker run --name portainer -d -p 443:9000 --restart always -v "/var/run/docker.sock:/var/run/docker.sock" -v ~/portainer:/certs -v portainer_data:/data portainer/portainer-ce --ssl --sslcert /certs/portainer.crt --sslkey /certs/portainer.key
Swarm: Inactive

Thanks.

Hi @antwal ,
Please try access the new portainer using incognito mode or even another browser and let's see are that issue is still appear.
Or if you want to create portainer without delete existing portainer, you can recreate it using another docker volume. Example:
$ docker volume create portainer_data_2
$ docker run -d -p 8080:9000 --name=portainer_testing --restart=always -v /var/run/docker.sock:/var/run/docker.sock -v portainer_data_2:/data portainer/portainer-ce

@ametdoohan
Hi
I have already done this test, but I have the same problem with both versions; I have deleted the previous volume and created a new one
I'm currently back to working version 1.23.2

I see on your image, you did filter word "tele", try to delete it and see.

@ametdoohan
you're right, I didn't notice the value
I have identified the problem
in version 1.23.2 the cache or any other field is cleared at each logout or expired session
in version 1.24.1 and 2.0.0 this does not happen; in fact, doing some tests a little while ago, I saw that the field is saved even after logout and at the next login it is reset to the previous state instead of empty
this bug, I remember was fixed here
so i believe there is a problem with cache checking in the last two versions
I updated to the latest version, but after deleting the previous volume date, now an error appears in the dashboard; but for this I have to check

@antwal if you still facing that, try to use incognito or even try using another browser.

Hi all,
After install portainer today , i've the same problem.

Same problem here, can confirm that logging in with incognito mode solves the issue.

Same here.
Portainer 1.4.0, installed as an add-on within Home Assistant.
Do I need to create a new network?


Release current staging on api-test

Can we clear the database on api-test.trompamusic.eu and release the current staging branch? (I know that it doesn't have #48 yet, but we need to do some tests with @asarasua and need something more up-to-date than the current version)

Hi @alastair, the test environment is actually using the latest commit from staging. However, I've just found out a lot of data is missing essential labels in the neo4j database. This is causing relational queries not working.
Are you sure that we can completely clear the database?

yes, deleting everything is fine, there are old items there from when we were still deciding the correct schema to use. I can recreate data using the most up-to-date schema if you clear it.

Deleted 17409 nodes, deleted 16123 relationships, completed after 322 ms.

is #48 in staging?

Yes, are you missing something?

not missing anything, I just wanted to confirm that the change from AddThingInterfaceThingInterface -> AddDigitalDocumentAudio and MergeDigitalDocumentAudio existed for all our types so that I can update my tools. thanks.


Rename project to reflect rename for ARN

ARN has been renamed to Luma3ds, and as such this project should be updated to reflect the new name.

Working on it today!

Project is not called "Luma3DS Updater" (lumaupdate on github)


fix ny state

@iandees why was this merged?

Because it works? Why not?

hmm does it? I'm afraid I'm not seeing a successful hook yet?

It's in progress, yes. I'm OK with the changes you made so I merged them.


Questionable Ruff Errors - Define Code Style?

Noticed ruff throwing some pre-commit errors on my dev fork - some of these seem to fly in the face of existing code (not counting the stuff I added), and I'm wondering if we want to ignore them:
A003 Class attribute compile is shadowing a Python builtin
.compile is integral to the dspy API, this probably wants to be ignored.
S101 Use of assert detected
Asserts are used all over the code base - do we want to allow them, or do we want to remove them? Note that asserts are sometimes necessary, e.g. to convince mypy that something isn't None
ERA001 Found commented-out code
Used in places to highlight TODOs; I despise dead code but get that a young project like this might need it in places, perhaps we silence for the time being?
T201 print found
Lots of error logging/warnings appear to be done via print(); is this the intended way to warn the user, or do we want to do something more sophisticated e.g. using logger?
SIM108 Use ternary operator [some code] instead of if-else-block
This is a tricky one; here's the block it's complaining about:
if self.metric:
    metric_val = self.metric(example, prediction, trace)
    if self.metric_threshold:
        # this is the problem bit
        success = metric_val >= self.metric_threshold
    else:
        success = metric_val

While perhaps there's a way to rewrite the x = a >= b form using if-else blocks, I feel like the code as it stands is the most elegant/intuitive. I think I straight up disagree with this flake rule, and would think it should be silenced, but... thoughts?
Also - not sure how long Ruff has been part of the pre-commit flow, but is there a comparable implementation of ruff checks for PRs?

#467 (made relevant changes here; updates to an existing PR)

@okhat tagging so this gets some visibility, and you know who to tag better than me.

@thomasahle @CyrusOfEden @isaacbmiller

I just tagged a few folks who are very knowledgeable about this — and who built the CI things we have. My involvement in this part is very small.

These three could be set to ignored in the pyproject:
A003 Class attribute compile is shadowing a Python builtin
S101 Use of assert detected
ERA001 Found commented-out code
For the others:
T201 print found
Using logger is a good idea for a PR! It's a big change but would provide lots of benefits over print. Quite a wide-sweeping change, and the biggest gotcha is making sure the right stuff still gets printed out in Google Colab / Jupyter notebooks.
SIM108 Use ternary operator [some code] instead of if-else-block
I agree that that's the cleanest way to write that code, but I don't think that's the cleanest code one could write.
I'm actually a little confused by the code you shared there because success implies a boolean but when there's no threshold it's a float? That's weird.
I would recommend that code get refactored, here's a try:
if self.metric:
    metric_val = self.metric(example, prediction, trace)
    if self.metric_threshold:
        metric_val = float(metric_val >= self.metric_threshold)
# replace future success refs with metric_val

I agree with Cyrus, except I would add that were definitely using too many asserts that could be more descriptive errors. I think assert has a place (like the mupy use you mention), but it's good to be reminded to use raise. So I'm happy to just ignore line-by-line where an assert is really needed.
I also think the commented out code should be removed and put somewhere else.

Also because ruff check . brings up 1500 errors, we don't enforce the pre-commit.
It is absolutely arbitrary, but technically only the auto-fixable errors cause CI fails for now.
@CyrusOfEden and I had a conversation a few weeks ago and said it was early in the project for enforcing the pre commit, but very open to creating a more realistic subset or gradually making fixes like the ones you sent


ssl: introduce serialNumberPeerCertificate()

Add a way to retrieve the serial number of a peer certificate.
Risk Level: Low
Testing: Make //test/common/ssl:ssl_socket_test cover this change.
Docs Changes: N/A
Release Notes: N/A
Signed-off-by: Tal Nordan<EMAIL_ADDRESS>

It would be nice to have SslSocket::getSerialNumberFromCertificate() and ContextImpl::getSerialNumber() share the implementation. Other certificate information extraction logic is duplicated between these two classes as well. Such duplicated logic includes extraction of URI SAN and extraction of subject name.
A possible enhancement to this design would be introducing a common utility class which provides data extraction functions over raw X509 pointers.
Such refactoring might exceed the scope of the current PR.

ci/circleci: asan failed due to a timeout in //test/integration:echo_integration_test.
//test/integration:echo_integration_test does pass though when I run it locally on my development machine.


Support HTTP proxy

[x] All tests passed. If this feature is not already covered by the tests, I added new tests.

Fix #59
Proxy is already supported before that PR by adding the following system properties to the Maven command:
-Dartifactory.proxy.host=acme.jfrog.io
-Dartifactory.proxy.port=8888
-Dartifactory.proxy.username=elmar
-Dartifactory.proxy.password=Wabbit

This PR enables configuring HTTP proxy in one of the following new ways:

By the regular config:

<proxy>
    <host>acme.jfrog.io</host>
    <port>8888</port>
    <username>elmar</username>
    <password>Wabbit</password>
</proxy>

By settings.xml:

<proxies>
    <proxy>
        <id>jfrog-proxy</id>
        <host>acme.jfrog.io</host>
        <port>8888</port>
        <username>elmar</username>
        <password>Wabbit</password>
    </proxy>
</proxies>

When will this be available? Im using the JFrogMaven@1 from azure devops extensions, i believe the JFrogMaven@1 uses this maven plugin, however it still does not read the proxy from my settings..


chore(agent): Delete monitor flag

copilot:all

Current dependencies on/for this PR:

main

PR #4250   👈

This comment was auto-generated by Graphite.


?

Does this also allow enabling round taskbars?

don't think so

No, please don't give Microsoft any more dumb ideas...


Env Var not set inside tests

Issue reported by @sseenivasan89
bru.setVar("foo", "bar");

The above script works in Scripts tab but does not work in Tests Tab

I tried that out and it is working. I first checked the value i want to store with an expect and than stored as variable.
Later I requested the variable again in an another stet and got it.

This has now been fixed in v0.14.0


Update README

@rultor merge

@rultor merge

@driver733 I'm sorry, I don't understand you :( Check this page and try again please

@rultor merge

@driver733 It is an issue, not a pull request


mapM for consumers

Is there a mapM_ that works for consumers?
mapM :: Monad m => (a -> m b) -> Consumer a m ()
mapM f = forever $ await >>= lift . f

Not yet, but I will definitely add this for you