Showing already selected photos when BSImagePicker is shown multiple times?

Is there a way to show already selected assets when the BSImagePickerViewController is presented multiple times and change the "Done" button according to the number of already (previously) selected  assets? Ie. lets say we set the maxNumberOfSelections to 5, the user selected 3 photos, but then decided to add 2 more.
As of now (or from what I see), every time the BSImagePickerController is presented, it doesn't show previously selected photos (which could be a bit confusing for the user, especially if you set the maxNumberOfSelections to 10 for example)

Don't take a global variable of BSImagePickerViewController() type.
Instead create a new variable of image picker each time you add image
func addImages(){
let imgPicker = BSImagePickerViewController()
    imgPicker.maxNumberOfSelections = 10
    imgPicker.settings.takePhotos = true
    imgPicker.selectionFillColor = appColor

    bs_presentImagePickerController(imgPicker, animated: true,
                                    select: { (asset: PHAsset) -> Void in
                                        // User selected an asset.
    }, deselect: { (asset: PHAsset) -> Void in
        // User deselected an assets.
    }, cancel: { (assets: [PHAsset]) -> Void in
    }, finish: { (assets: [PHAsset]) -> Void in
        for asset in assets{
            let img = self.getAssetThumbnail(asset: asset)
                img.accessibilityHint = "exist"
            self.imagesArray.append(img)
        }
        DispatchQueue.main.async {
            self.collectionView.reloadData()
        }

    }, completion: nil)
}

Exelent this helped me out alot, i was wondering why when I started the image picker it still had old images selected before already selected, seing the picker in the action stoped that.
put here:
func addImages() {
let vc = BSImagePickerViewController()
instead of view did load...


Update minimum wage rates for October 2016

Trello card
Description
New rates (in effect from 1st October 2016 onwards):

Under 18: £4.00
18-20: £5.55
21-24: £6.95
25+: £7.20 (no change)
Apprentices: £3.40
Accommodation: £6.00

This change affects the following smart answers:

Minimum wage calculator for workers (am-i-getting-minimum-wage)
Minimum wage calculator for employers (minimum-wage-calculator-employers)

Expected changes - before 1st October 2016
If this is deployed before the 1st of October 2016, there should be no user-visible changes to either of these pages:

https://www.gov.uk/am-i-getting-minimum-wage/y/current_payment/not_an_apprentice/20/30/160.0/850.0/0.0/no
https://www.gov.uk/minimum-wage-calculator-employers/y/current_payment/not_an_apprentice/20/30/160.0/850.0/0.0/no

Expected changes - after 1st October 2016
After midnight on the 1st of October, the following user-visible changes should be present:
Minimum wage calculator for workers
Before
https://www.gov.uk/am-i-getting-minimum-wage/y/current_payment/not_an_apprentice/20/30/160.0/850.0/0.0/no

After
https://smart-answers-pr-2741.herokuapp.com/am-i-getting-minimum-wage/y/current_payment/not_an_apprentice/20/30/160.0/850.0/0.0/no

Minimum wage calculator for employers
Before
https://www.gov.uk/minimum-wage-calculator-employers/y/current_payment/not_an_apprentice/20/30/160.0/850.0/0.0/no

After
https://smart-answers-pr-2741.herokuapp.com/minimum-wage-calculator-employers/y/current_payment/not_an_apprentice/20/30/160.0/850.0/0.0/no

@robinjam I think it's worth adding a few unit tests, with the date set to Oct 1st 2016, that checks the new minimum wage rates.

Superseded by #2747


Improve decoding of hex escapes

Sometimes it is necessary to decode big sequence of Unicode hex escapes, like:
"\uF975\uf9bc\uF9A0\uF9C4\uF975\uf9bc\uF9A0\uF9C4..."

Texts with non-Latin characters usually look like this. This PR accelerates the decoding of such strings by 25%.
Before: test json::tests::bench_decode_hex_escape ... bench:      4295 ns/iter (+/- 176)
 After: test json::tests::bench_decode_hex_escape ... bench:      3200 ns/iter (+/- 148)

Thanks for the pull request, and welcome! The Rust team is excited to review your changes, and you should hear from @alexcrichton (or someone else) soon.
If any changes to this PR are deemed necessary, please add them as extra commits. This ensures that the reviewer can see what has changed since they last reviewed the code. The way Github handles out-of-date commits, this should also make it reasonably obvious what issues have or haven't been addressed. Large or tricky changes may require several passes of review and changes.
Please see CONTRIBUTING.md for more information.

Nice win!


Cannot disable select using IE8

We received a report that one of our fields was not disabling correctly. Upon further inspection, it appears that the field is not disabling in IE8.
This is taken from the Examples page (note that the Disable button had just been clicked):

I am not sure if IE8 will be supported, but it would be nice to have.

This is a duplicate of https://github.com/select2/select2/issues/3271.


pending transfer hotspot

My sensecap m1 show pending transfer hotspot from 2 weeks ago but actually already completed transfer. Can't transfer to new owner.
how to fix it.

Thanks for the report @honmmtc give it a try now, it should be resolved.


About "Webpack"

It is great to see "webpack" work in the project. I have several questions on it, it looks like "esri-system-js"  is not necessary for webpack and all the ESRI Javascript library(3.16) can be loaded. if so, all the functionality of the library can be used to develop application... it should work in ESRI Javascript library (4.0) as well. I tried this project in ASP.NET core 1.0 RC2, and it works nice(can be uploaded sometimes in the future). I am wondering if all the functionalities of the javascript library(3.16 / 4.0) can be used in the future project?

Sorry for the late reply on this (I thought I had already responded):

Yes, if you use webpack, you don't need SystemJS, and vice versa. You only need one.
You should be able use the ArcGIS API for JavaScript v4.x w/ webpack, see https://github.com/lobsteropteryx/esri-webpack/pull/1
Part of the purpose of this repo is to show Angular 2 working w/ ArcGIS API for JavaScript v3.x, so I have no plans to migrate it to 4.x.
I don't know what ASP.NET core 1.0 RC2 is. This repo has no .Net code in it, and never will. However, I'm glad to hear that it's working w/ in ASP.Net core 1.0 RC2 project.

W/r/t using " all the functionality of the library can be used to develop application" - I haven't tested this, but you should be able to use any module included whichever version (3.x or 4.x) of the ArcGIS API for JavaScript that you've included on the page. That said, for generic tasks (like Ajax or DOM manipulation) that aren't related to working with the map, layers, or ArcGIS services, you should favor using the modules that come w/ Angular 2 rather than those included in esri or dojo packages included w/ the ArcGIS API for JavaScript.

Thanks for your reply very much. I have another question, do you have plan to write "sample components", like examples in https://developers.arcgis.com/javascript/3/jssamples/ or developing standard components(like what in "angular2-esri-example" project) in the near future?

I don't currently have plans to create any sample pages or develop any more components for this example app. My hope is that what's here is enough to demonstrate to people how to use the map and widgets from the ArcGIS API for JavaScript in an Angular 2 app.
Is there something specific that you are trying to do with the ArcGIS API for JavaScript in an Angular 2 app that this example app does not help you with?

Thanks for your quick reply.
It is hard to try and test if all the esri dojo library can work well in Angular 2 because the library is too big. I try these "component" and work fine in angular material2, so I am wondering if all the library work well in Ng2. It Will be helpful if there are lots of "components" available in the future.

I understand your concern. It would be ideal if there were samples of every Esri module working in an Angular 2 app, but, as you said, the library is too large.
Instead, my intent here was to try and demonstrate ways to use at least one of each type of ArcGIS API for JavaScript (v 3.x) module (map, dijits, utility, etc) with various types of Angular 2 components and data binding.
Based on the examples of successful integration in this repo along with those in angular2-esri-playground, I feel very confident that the rest of the components in both the 4.x and 3.x versions of the ArcGIS API for JavaScript should work in Angular 2 apps.
If there's some specific compatibility that you want to know about that, feel free to open an issue here (for 3.x) or here (for 4.x).

Also, I'm glad to hear that you've gotten the components to work well w/ Angular 2 Material! I would encourage you to share a link if you have one.

Thanks for your help. What I am concerning about the esri JavaScript library now is try to add some functionality like click map and display the related info to the sidenav.
I developed "angular2-esri-example" as an example in the aspnet/JavascriptServices framework by Steve Sanderson. Sidenav and other angular material2 components used in this example and it works fine. I try to use dynamic loading Gis component to the sidenav but some issues(issues from the angular2). I can add the sidenav component from angular material2 for this project and upload it in the near future.

If you can upload the code somewhere I'd be glad to look at it.

Please see https://github.com/Shawn-Fan/angular2-esri-example-material2

Was your question resolved in this issue?

Thanks, it works in 3.17 as well.
Another question. When will you guys start to work on or publish Angular2 Javascript API officially?

At this point there is no plan for any "official" integration library between the ArcGIS API for JavaScript and Angular 2. I'm not convinced that one is needed. I think there are multiple patterns to successfully loading Esri modules (i.e. esri-system-js vs webpack). I don't think it makes sense to create re-usable components or services.

Agreed, @tomwayson.


Warn call on nullable receiver

Followup of #1375 and #394
This introduce an advice on calls on a nullable receiver.
Since the conversion could be complex, I propose a light approach with advices (quiet warnings) instead of hard errors, so that:

jenkins can track them
nitpick (thus vim) can show them
mirgration can be done in an incremental and iterative way
issues with a strict call-on-nullable policy can be identified without breaking things
@Morriar can bank nitcoins in future PR

If (when?) all call on nullable can be removed, then the advice will become an error.

Results: 2016 warnings http://gresil.org/jenkins/job/CI_github/lastCompletedBuild/warnings2Result/type.-1346187616/

Good ! +1

+1

+1

Hum, the commit changed the error message on things like x = null; x.foo or more frequently if x == null then x.foo when there is a typo and the programmer wanted x != null in fact.
Which error message do you prefer? the old one "Error: method + called on null" of the new one "Error: method + does not exists in null." ?

There not a big difference but I like the new one. It would standardize the error, less variations between errors is a good thing.

update tests/ with new error messages


Apply frost resistance damage modifier based on config settings

This is the way.

Guess i have write acces to this repo
cool

You're like the lord of backpacks now!


Automated cherry pick of #9072: fix(cloudid): huawei saml provider create issue

Cherry pick of #9072 on release/3.6.
#9072: fix(cloudid): huawei saml provider create issue

Codecov Report

Merging #9073 (b460a73) into release/3.6 (b9d30c0) will increase coverage by 11.76%.
The diff coverage is 0.00%.

@@               Coverage Diff                @@
##           release/3.6    #9073       +/-   ##
================================================
+ Coverage         5.86%   17.63%   +11.76%
================================================
  Files              828       89      -739
  Lines           129133     8960   -120173
================================================
- Hits              7577     1580     -5997
+ Misses          120876     7206   -113670
+ Partials           680      174      -506

Flag
Coverage Δ

aFlag
17.63% <0.00%> (+11.76%)
:arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

pkg/monitor/alerting/conditions/nodataquery.go
0.98% <0.00%> (-0.01%)
:arrow_down:

pkg/compute/models/usage.go

pkg/mcclient/options/servicecertificates.go

pkg/compute/models/vpcresource.go

pkg/multicloud/aws/securitygroup.go

pkg/multicloud/aws/iam_role.go

pkg/util/netutils2/netutils_linux.go

pkg/cloudcommon/db/initialization.go

pkg/multicloud/aws/snapshot.go

pkg/compute/models/vpcs_mapped_address.go

... and 698 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 c8d4a33...b460a73. Read the comment docs.


Allowing remote files in segment generation

Adding ability to set input file path to remote path
Changing record reader factory to allow nonlocal segment paths
Changing segment generator config to not assume file abstraction

Codecov Report

Merging #3970 into master will increase coverage by 0.05%.
The diff coverage is 75%.

@@             Coverage Diff              @@
##             master    #3970      +/-   ##
============================================
+ Coverage        67%   67.05%   +0.05%
  Complexity        4        4
============================================
  Files          1032     1032
  Lines         51047    51050       +3
  Branches       7126     7127       +1
============================================
+ Hits          34202    34231      +29
+ Misses        14506    14478      -28
- Partials       2339     2341       +2

Impacted Files
Coverage Δ
Complexity Δ

...e/pinot/core/data/readers/RecordReaderFactory.java
34.78% <100%> (ø)
0 <0> (ø)
:arrow_down:

...indexsegment/generator/SegmentGeneratorConfig.java
62.45% <66.66%> (-0.3%)
0 <0> (ø)

.../impl/dictionary/FloatOnHeapMutableDictionary.java
80% <0%> (-13.34%)
0% <0%> (ø)

...impl/dictionary/DoubleOnHeapMutableDictionary.java
68.88% <0%> (-13.34%)
0% <0%> (ø)

...e/operator/dociditerators/BitmapDocIdIterator.java
60.71% <0%> (-3.58%)
0% <0%> (ø)

...e/operator/dociditerators/MVScanDocIdIterator.java
63.63% <0%> (-3.04%)
0% <0%> (ø)

.../impl/dictionary/LongOffHeapMutableDictionary.java
87.27% <0%> (-1.82%)
0% <0%> (ø)

...ot/core/query/pruner/ColumnValueSegmentPruner.java
85.91% <0%> (-1.41%)
0% <0%> (ø)

...regation/function/customobject/QuantileDigest.java
57.74% <0%> (-0.45%)
0% <0%> (ø)

...ntroller/helix/core/PinotHelixResourceManager.java
60.9% <0%> (+0.09%)
0% <0%> (ø)
:arrow_down:

... and 17 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 9ec6b5b...ef6b912. Read the comment docs.


CI: Try to set max-parallel: 1 for Windows

Signed-off-by: Takuro Ashie<EMAIL_ADDRESS>
Which issue(s) this PR fixes:
Fixes #
What this PR does / why we need it:
Docs Changes:
Release Note:

Still Windows test is unstable, it doesn't seem make sense.


EW-12949: add encoding module

I have read the CLA Document and I hereby sign the CLA

Approved pending conflict resolution.


WIP: Add service.running_sustained

This new state will allow us to make sure that a service is running
in a sustained manner. Also, will do some waits in case systemd will
do retries on the background, what avoids instant failure from salt
being reported.

This looks like a good idea to me, it would be even worth to be shared upstream with salt!

While looking closer at the code I thought: why aren't we thinking the other way around? We could let the user specify a health check command and keep invoking this command until the health check passes or a maximum number of attempts are done.
The current code would slow down the deployment of the system quite a bit (see the delay between even the successful checks).

While looking closer at the code I thought: why aren't we thinking the other way around? We could let the user specify a health check command and keep invoking this command until the health check passes or a maximum number of attempts are done.

The current code would slow down the deployment of the system quite a bit (see the delay between even the successful checks).
I fundamentally agree that we can make it more generic, but I doubt in our case with etcd we will be able to avoid a bit of slowing down the deployment, since the idea is checking that etcd is running in a stable way.
I will confirm that this works and we can make it more generic and maybe even submit it to upstream as you suggested :)

I can confirm I have deployed a cluster in the same environment that was failing. With this patch applied I could deploy a cluster even using DNS names for the administration dashboard.
I can confirm also that without this patch using DNS names on said environment failed 100% of the times.


[Question] Cannot open camera video capture See verbosity level for more details.

Seems that the device is not recognized in Ubuntu18.04 and no serial seems to be the one of the ZED. (NUCi711). Is there some reasons that may cause this problem? Thanks a lot.

Hi @jjjmmmxxx
can you copy and paste the result of the $ lsusb command when the camera is connected?

Thanks a lot. Things have been fixed. Something goes wrong with the USB cable. It is not the problem of the camera or this API. Thank u~

Sorry to bother. It seems that I meet another problem. The examples except for the "zed_open_capture_sensors_example" and "zed_open_capture_sync_example" work well. But these two examples failed to run.

Can you post the result of the $ lsusb command?

Please also verify to have followed correctly the following installation steps:
https://github.com/stereolabs/zed-open-capture#add-udev-rule
If udev rules are not correctly installed the sensors module cannot be accessed without sudo rights

This is the lsusb output:

I tried this process again and I found it's still like that: No available ZED Mini or ZED2 cameras

It does seem something goes wrong when I follow the process for udev rules. When I add sudo before zed_open_capture_sensors_example, everything goes right.

Yes, that's because the ZED2i support was still missing in the udev rules.
I fixed it only 2 hours ago when I noticed it, you can pull the new version (v0.4.1) from Github that should fix the problem.
I'm really sorry for this annoying bug.

Yes, it should work without any problems

Hi,
same message when using ZED mini with debian 11.
$ zed_open_capture_video_example
Cannot open camera video capture
See verbosity level for more details.

Device seems to be recognized in system
$ lsusb
...
Bus 001 Device 014: ID 2b03:f681 STEREOLABS ZED-M HID Interface
...

How do I increase verbosity level? I dont see any parameters
    if( !cap.initializeVideo() )
    {
        std::cerr << "Cannot open camera video capture" << std::endl;
        std::cerr << "See verbosity level for more details." << std::endl;

        return EXIT_FAILURE;
    }

I have been getting the exact same error with ZED2 on Ubuntu 20.04. Followed all the instructions as mentioned on the README. The C++ node runs and generates data. However, after a while the node freezes and doesn't get any more data from the cameras.
Subsequent attempts at launching the node generate this message, and the node never starts.  Please help. Thank you so much.

I have the same issue and reboot couldn't help.

@Yelloooowww I'm sorry, but the reboot command only works starting from the ZED2 camera model.
The ZED-M does not have the electronic circuit for this behavior.

I have the same problem as described by a few people above! I've followed the README, running Ubuntu 22.04

@ernlavr this is a closed issue. Please do not reply to closed issues.
Create a new one with all the required information and eventually tag this issue as a reference.


Scroll view inside CardStackLayout??

Can we implement scrollview inside CardStackLayout? I am unable to implement it. Please help me.

@akku20080
Use NestedScrollView inside the CardView. Let me know if that works for you.


ci: Use mattn/goveralls instead of shogo821/goveralls

Fixes #N/A
Description

Update to use https://github.com/mattn/goveralls directly

How was this change tested?

make presubmit

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

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
2 unchanged lines in 1 file lost coverage.
Overall coverage decreased (-0.02%) to 79.626%

Files with Coverage Reduction
New Missed Lines
%

pkg/test/cachesyncingclient.go
2
80.21%

Totals

Change from base Build<PHONE_NUMBER>:
-0.02%

Covered Lines:
8266

Relevant Lines:
10381

💛  - Coveralls


Theme demo broken on the Hugo Themes website

Hello @olOwOlo
I am one of the maintainers of Hugo Themes and I noticed that in the latest deploy your theme's demo failed to generate.
Also I noticed that recently you migrated to using Hugo Pipes.
However you need to take additional steps to make the demo generate on the Hugo Themes Showcase. See here for detailed instructions: https://github.com/gohugoio/hugoThemes#resources
If you have any questions feel free to ask, otherwise feel free to close this issue once you implement the necessary changes.

Thanks for your correction.


Use all configured nameservers when using DnsNameResolver in all cases

Motivation:
Due a change introduced in 68105b257d915d8a0cb7b2acd9061661666537b6 we incorrectly skipped the usage of nameservers in some cases.
Modifications:
Only fetch a new stream of nameserver if the hostname not matches the original hostname in the query.
Result:
Use all configured nameservers. Fixes https://github.com/netty/netty/issues/10499

@vietj  PTAL


Chart id is not unique when rendering component in a loop.

If I try to render a Chart Widget in a loop, all of the charts get appended to the first component.
I guess it is due to protected static string $chartId = 'apexChart' being assigned a default value. Hence
/**
* Retrieves the chart id.
*
* @return string|null The chart id.
*/
protected function getChartId(): ?string
{
    return static::$chartId ?? 'apexChart_'.Str::random(10);
}

will never append a random string.
Changing the property to protected static ?string $chartId will do the trick but I'm not sure if this leads to unwanted side effects.

If you think making the property nullable will do the trick, I'm happy to submit a PR.

@maltebaer thank you!
Check the new version 3.1.1
https://github.com/leandrocfe/filament-apex-charts/releases/tag/3.1.1

Awesome, it works like charm! Thanks a lot for the quick fix!


Stylecop.Analyzers throws  NullReferenceExceptions for SA1135 after upgrading to VS2019 16.5.

We're using Stylecop.Analyzers in several projects targeting .NET framework 4.6.2. After upgrading Visual Studio 2019 from 16.4.6 to 16.5 we were getting lots of NullReferenceExceptions in the build (with Stylecop.Analyzers 1.1.118). We tried upgrading to Stylecop.Analyzers 1.2.0-beta.164, but we're still getting NullReferenceExceptions for SA1135.

The NullReferenceException should produce an AD0001 diagnostic with context and call information. Can you include that?

With VS 2019 v16.5.1 and Stylecop.Analyzers 1.1.118, the AD0001 does not include more information:
Severity	Code	Description	Project	Project Rank	Path	File	Line	Column	Category	Source	Suppression State	Tool
Error	AD0001	Analyzer 'StyleCop.Analyzers.ReadabilityRules.SA1135UsingDirectivesMustBeQualified' threw an exception of type 'System.NullReferenceException' with message 'Object reference not set to an instance of an object.'.	My.Project	3	C:\Path.To\My.Project	C:\Path.to\My.Project\CSC	1	1	Build	Build	Active	Compiler

Duplicate of #3136


On mobile browsers, the pictures in the ListView may fail to load

On mobile browsers, the pictures in the ListView may fail to load, and the pictures that have been loaded successfully, when they slide out of the screen and then return to the screen, they also fail to load,

In the browser on the PC is ok

I will take a look at it.
It would be great if you could provide more information or reproducible code.

The problem seems to only happen with the HTML renderer but CanvasKit renderer works fine, by default flutter use HTML renderer on mobile and CanvasKit renderer on desktop but you can force it to use either on both platforms Web renderers.
I will keep looking for a solution but I don't think I will be able to solve it soon.

ok ok thanks

ok ok thanks


反馈一个bug，关于Tokenizer

在ChatGLM的Official实现中，token采用了import sentencepiece as spm，这样的一个库，这个库在  self.sp.EncodeAsPieces(text)，这一句会把英文单词比如“hello”处理成"▁hello"，注意前面的两个杠不是下划线。这应该是最标准的方式，而本项目好像没有做类似的处理。

是的.. 这个有点麻烦，暂时还没弄，不过中文模型效果都还可以
之后会考虑加上，或者用py接口，这样可以用原生的tokenizer

@ztxz16 请帮忙看看这个小问题
V100测试LLama:
[ user: "Below is an instruction that describes a task. Write a response that appropriately completes the request.
Instruction:
Building a website can be done in 10 simple steps.
Response:", model: "Buildingawebsitecanbeacrazyeasyprocess.First,youneedtopickawebsitebuilder.Therearemanymanytoppickfrom.Someofthemarefree,someofthemarepaid.Youneedtopickonethatfitsyourneeds.Next,youneedtopickawordpresstheme.Thiswillbeyourwebsite'sdesign.Choosethatwhichbestsuitsyourneeds.Then,youneedtopickawordpressplugin.Thiswilladdextrafunctionalitytoyourwebsite.Choosethatwhichbestsuitsyourneeds.Next,youneedtopickawordpresstemplate.Thiswillbeyourwebsite'slayout.Choosethatwhichbestsuitsyourneeds.Then,youneedtopickawordpresspage.Thiswillbeyourwebsite'scontent.Choosethatwhichbestsuitsyourneeds.Next,youneedtopickawordpresscategory.Thiswillorganizeyourwebsite'scontent.Choosethatwhichbestsuitsyourneeds.Then,youneedtopickawordpresspost.Thiswilladdnewcontenttoyourwebsite.Choosethatwhichbestsuitsyourneeds.Next,youneedtopickawordpressuser.Thiswillallowyouandotherstomanageyourwebsite.Choosethatwhichbestsuitsyourneeds.Finally,youneedtopublishyourwebsite.Choosethatwhichbestsuitsyourneeds.Andthat'sit!Buildingawebsiteisasimpleasthrowingapartyforfriends."]
batch: 1
output 336 tokens
use 18.232920 s
speed = 18.428205 tokens / s
为啥model的Response没有空格隔开呀?
这个是std::string Tokenizer::DecodeTokens(const std::vector &tokens)里的bug吗?

@ztxz16 请帮忙看看这个小问题 V100测试LLama: [ user: "Below is an instruction that describes a task. Write a response that appropriately completes the request.
Instruction:
Building a website can be done in 10 simple steps.
Response:", model: "Buildingawebsitecanbeacrazyeasyprocess.First,youneedtopickawebsitebuilder.Therearemanymanytoppickfrom.Someofthemarefree,someofthemarepaid.Youneedtopickonethatfitsyourneeds.Next,youneedtopickawordpresstheme.Thiswillbeyourwebsite'sdesign.Choosethatwhichbestsuitsyourneeds.Then,youneedtopickawordpressplugin.Thiswilladdextrafunctionalitytoyourwebsite.Choosethatwhichbestsuitsyourneeds.Next,youneedtopickawordpresstemplate.Thiswillbeyourwebsite'slayout.Choosethatwhichbestsuitsyourneeds.Then,youneedtopickawordpresspage.Thiswillbeyourwebsite'scontent.Choosethatwhichbestsuitsyourneeds.Next,youneedtopickawordpresscategory.Thiswillorganizeyourwebsite'scontent.Choosethatwhichbestsuitsyourneeds.Then,youneedtopickawordpresspost.Thiswilladdnewcontenttoyourwebsite.Choosethatwhichbestsuitsyourneeds.Next,youneedtopickawordpressuser.Thiswillallowyouandotherstomanageyourwebsite.Choosethatwhichbestsuitsyourneeds.Finally,youneedtopublishyourwebsite.Choosethatwhichbestsuitsyourneeds.Andthat'sit!Buildingawebsiteisasimpleasthrowingapartyforfriends."]
batch: 1 output 336 tokens use 18.232920 s speed = 18.428205 tokens / s
为啥model的Response没有空格隔开呀? 这个是std::string Tokenizer::DecodeTokens(const std::vector &tokens)里的bug吗?

是bug.. 现在可以先试试用python的tokenizer

@ztxz16 在V100上测试也有一点点问题, 同样的Prompt "Building a website can be done in 10 simple steps.":
i8, batch=2
batch: 2
output 670 tokens (正常)
use 43.564709 s
speed = 15.379421 tokens / s
f16, batch = 1
batch: 1
output 336 tokens (正常)
use 17.873089 s
speed = 18.799213 tokens / s
f16, batch=2
batch: 2
output 14 tokens (突然中断)
use 1.104086 s
speed = 12.680171 tokens / s
而且i8的性能和f16差不太多, 是因为GPU型号的问题吗....
另外请问用python接口有测试性能的测试文件吗?

@ztxz16 请帮忙看看这个小问题 V100测试LLama: [ user: "Below is an instruction that describes a task. Write a response that appropriately completes the request.
Instruction:
Building a website can be done in 10 simple steps.
Response:", model: "Buildingawebsitecanbeacrazyeasyprocess.First,youneedtopickawebsitebuilder.Therearemanymanytoppickfrom.Someofthemarefree,someofthemarepaid.Youneedtopickonethatfitsyourneeds.Next,youneedtopickawordpresstheme.Thiswillbeyourwebsite'sdesign.Choosethatwhichbestsuitsyourneeds.Then,youneedtopickawordpressplugin.Thiswilladdextrafunctionalitytoyourwebsite.Choosethatwhichbestsuitsyourneeds.Next,youneedtopickawordpresstemplate.Thiswillbeyourwebsite'slayout.Choosethatwhichbestsuitsyourneeds.Then,youneedtopickawordpresspage.Thiswillbeyourwebsite'scontent.Choosethatwhichbestsuitsyourneeds.Next,youneedtopickawordpresscategory.Thiswillorganizeyourwebsite'scontent.Choosethatwhichbestsuitsyourneeds.Then,youneedtopickawordpresspost.Thiswilladdnewcontenttoyourwebsite.Choosethatwhichbestsuitsyourneeds.Next,youneedtopickawordpressuser.Thiswillallowyouandotherstomanageyourwebsite.Choosethatwhichbestsuitsyourneeds.Finally,youneedtopublishyourwebsite.Choosethatwhichbestsuitsyourneeds.Andthat'sit!Buildingawebsiteisasimpleasthrowingapartyforfriends."]
batch: 1 output 336 tokens use 18.232920 s speed = 18.428205 tokens / s
为啥model的Response没有空格隔开呀? 这个是std::string Tokenizer::DecodeTokens(const std::vector &tokens)里的bug吗?

是bug.. 现在可以先试试用python的tokenizer

你好，python的tokenizer应该怎么用呢？有例子吗？


OpenSSL 1.1.1a: Work around a failing 'make test' with Oracle Developer Studio 12.5

Work around a failing 'make test' with Oracle Developer Studio 12.5 and a configured OpenSSL debug build. Marks all mentioned symbols as weak.
Partly fixes openssl#6912.

Hm, I see. At the end? The way I have it defined in rsa_complex.c now?
I'll try to get to it next week ...


Code generator plugins/extensions

As part of https://github.com/jamiebrynes7/spatialos-sdk-rs/issues/107, the idea of code generator plugins was brought up. This issue should track this particular use case and possible solutions.

Plugins using WASM, or some dynamically loaded library
Shipping codegen as a library and have users write a build.rs to configure the plugins.

The idea of using WASM as a way of providing compile-time extensions has been coming up elsewhere in the Rust community, too. We should check out watt and see if we ~steal ideas from it~ use it as inspiration :grin:


Data Quality Checker Spark job

Introduction of a Data Quality Check Spark job which conducts various rules to ensure the quality of data. Optional logic that can be added to the standard-ingest template

@azfaraziz Please review the SonarQube results. They're for your benefit so feel free to ignore any that you don't think make sense.

Added a word doc with integration tests to execute. It is located spark-job-dataquality/spark-job-dataquality-app/src/test/resources/IntegrationTests.docx

SonarQube analysis reported 25 issues

 2 critical
 22 major
 1 minor

Watch the comments in this conversation to review them.
1 extra issue
Note: The following issues were found on lines that were not modified in the pull request. Because these issues can't be reported as line comments, they are summarized here:

 DataQualityChecker.java#L9: This block of commented-out lines of code should be removed.

The CI failed due to a metadata issue which is not part of this request. Is there a way for me to have the test rerun without having to commit again?

I've restarted the build.

SonarQube analysis reported 20 issues

 2 critical
 17 major
 1 minor

Watch the comments in this conversation to review them.
1 extra issue
Note: The following issues were found on lines that were not modified in the pull request. Because these issues can't be reported as line comments, they are summarized here:

 DataQualityChecker.java#L9: This block of commented-out lines of code should be removed.

I believe the code is ready to be reviewed. Thanks

@azfaraziz Please open a JIRA in the public Kylo project so we can have this pull request tested and merged.


@include feature fails on oracle sequencers

System information:

Operating system (distribution) and version: Windows 10 1903
DBeaver version: <IP_ADDRESS>911031605
Additional extensions: DBeaver Office Support <IP_ADDRESS>908060542

Connection specification:

Database name and version: Oracle Database 19c Enterprise Edition
Driver name: Oracle JDBC driver <IP_ADDRESS>.0
Do you use tunnels or proxies (SSH, SOCKS, etc)? No

Describe the problem you're observing:
We have a "setup.sql" file which adds all sequencers, tables & views:
-- This script needs to be run as the user xxx.

-- Create the sequencers.
-- @include sequencers/change_ids.sql
-- @include sequencers/approval_ids.sql
-- ...

-- Create the tables.
-- @include tables/domains.sql
-- @include tables/status.sql
-- @include tables/apps.sql
-- ...

-- Create the views.
-- @include views/v_changes.sql
-- @include views/v_approvals.sql
-- ...

This script doesn't continue after going into the first sequencer include file and does not continue/waits forever.
It's even not possible to cancel the script.
Steps to reproduce, if exist:
Save the following 3 files as ".sql" (github doesn't allow .sql files to upload) in one directory and execute setup.sql as "Execute SQL Script". dbeaver no tries to create sequence change_ids and does not continue/waits forever.
setup.sql.txt
approval_ids.sql.txt
change_ids.sql.txt

seems to be fixed.
correlated issue #8313


Investigate possible race on global deletion

If  the compositor destroys a global at the same time as a client tries to bind it, we can end up with a situation where the client is killed for trying to bind a non-existing global because its request just arrived a little late. There should be some grace period after the destruction of a global.

Looks like there's a way for compositors to mitigate this issue now: https://gitlab.freedesktop.org/wayland/wayland/merge_requests/28

Oh, nice, thanks for the heads-up. I'll wait for a new version of libwayland to be released including this, and add a similar API in wayland-rs.

In the meantime a new version of libwayland (1.18) was released that contains wayland/merge_requests/28. Maybe this work could progress now?
(Note: Please only interpret this as an FYI, not a demand or anything like that ;)

Thanks for the heads-up!

@vberger if I may ask: Are there some changes in wayland-rs required to fix this issue (above you mentioned "add[ing] a similar API in wayland-rs") or is it mainly open for other people to find until wayland/merge_requests/28 is implemented in every major compositor?

This problem, and the way it has been solved upstream is a purely server-side problem. I keep this issue open on the repo as it needs some implementation on the wayland-server crate, so that Wayland compositors built using wayland-server can use this fix as well.
Sadly, there is nothing that can be done client-side, apart from waiting major compositors to fix the problem in their codebases.

This is implemented in 0.30

This is implemented in 0.30


libfixes and pkg-config seem to require fixesproto >6

Solution to issue cannot be found in the documentation.

[x] I checked the documentation.

Issue
It seems taht the pc file for libxfixes requires fixesproto >6 which simply doesn't exist? this is likely since that version is only included in the mega build.
 If I look at the pkg-config files in that package, it looks like it's providing fixesproto 6.1, so I think it should be working correctly.

I guess I had tried to use xorg-xorgproto and then to find other packages, I had to add even more:
starting with:
$ mamba list --prefix ${PREFIX} proto
# packages in environment at /home/conda/feedstock_root/build_artifacts/debug_1730070826568/_h_env:
#
# Name                    Version                   Build  Channel
xorg-xf86vidmodeproto     2.3.1             hb9d3cd8_1004    conda-forge
xorg-xorgproto            2024.1               hb9d3cd8_1    conda-forge

Without xorg-xproto I got:
Looking for package 'xcomposite'
Looking for package 'xcomposite-uninstalled'
Reading 'xcomposite' from file '/home/conda/feedstock_root/build_artifacts/debug_1730070826568/_h_env/lib/pkgconfig/xcomposite.pc'
Parsing package file '/home/conda/feedstock_root/build_artifacts/debug_1730070826568/_h_env/lib/pkgconfig/xcomposite.pc'
  line>prefix=/home/conda/feedstock_root/build_artifacts/debug_1730070826568/_h_env
 Variable declaration, 'prefix' has value '/home/conda/feedstock_root/build_artifacts/debug_1730070826568/_h_env'
  line>exec_prefix=${prefix}
 Variable declaration, 'exec_prefix' has value '/home/conda/feedstock_root/build_artifacts/debug_1730070826568/_h_env'
  line>libdir=${exec_prefix}/lib
 Variable declaration, 'libdir' has value '/home/conda/feedstock_root/build_artifacts/debug_1730070826568/_h_env/lib'
  line>includedir=${prefix}/include
 Variable declaration, 'includedir' has value '/home/conda/feedstock_root/build_artifacts/debug_1730070826568/_h_env/include'
  line>
  line>Name: Xcomposite
  line>Description: X Composite Extension Library
  line>Version: 0.4.6
  line>Requires: xproto compositeproto >= 0.4
Looking for package 'xproto'
Looking for package 'xproto-uninstalled'
(/home/conda/feedstock_root/build_artifacts/debug_1730070826568/_build_env) [conda@c36a657a87ff work]$ echo $?
1

I guess the packages are located in share/pkgconfig/xproto.pc and not lib/pkgconfig/.... any way to move them?

Ah, yes. So, it's intentional that they're in share rather than lib.
The idea is that lib is for read-only resources with platform-dependent content, while share is for platform-independent content (e.g., FHS 4.11). pkg-config files can go in either: if a .pc file describes a resource without any special compiler flags or platform-specific content, you can share it across platforms; but if it requires something like a -march flag, you shouldn't. The X.org protocol prototypes are exactly the sort of cross-platform resource that can be described in share, and the X.org developers are sufficiently plugged-in to consciously choose which directory they're using.
Another example of this approach is in the glib development files. Almost all of the glib headers go in $PREFIX/include, as normal, but they put one file in lib:
/usr/lib64/glib-2.0/include/glibconfig.h

That's because this one contains the platform-specific configuration results along the lines of "how many bits are in a long int?"
Now it's reasonable to ask the question of whether this is all necessary or worthwhile in the year 2024, but that's the context. Anything that's extending a pkg-config path should generally include both share/pkg-config and lib/pkg-config to be safe. You could imagine deciding to override maintainers' decisions and put everything in one place for conda-forge, but in my view that wouldn't be a great use of our time.


add initial CMakeLists.txt

tested in MSYS2

I like how clean your version is, however this breaks with Ubuntu 15.10 because of the cmake version requirement. Changing 3.3 to 3.2 fixes this.
Also, your cmakelists is broken, you link to particle target but the executable is particles. No idea how you couldve successfully tested this in msys2

Also, dont know if you missed my pull req here https://github.com/MauriceGit/Partikel_accelleration_on_GPU/pull/1

no clue how this worked! but it did...

Still problems with your pull req. You link to GLUT although you dont need to. And you add header files to the "add_executable" command.

Btw, did my pull request work on your system? Or why did you create another one? :)

I think he didn't see the first :)
https://www.reddit.com/r/programming/comments/40ri26/particle_simulation_with_opengl_compute_shader_8m/

@haikarainen
In order for headers to show up in QtCreator (maybe also other IDEs) they need to be added to add_executable. At least that was the case when I last checked.
@MauriceGit yeah was not looking at your pulls

Thank you both for your effort.
But I decided to go for a simple make instead of cmake because I worked a lot with make and never really with cmake. So I feel  more comfortable with it.

I also can't get your cmake to run properly...
/usr/bin/ld: //usr/local/lib/libglfw3.a(x11_window.c.o): undefined reference to symbol 'XConvertSelection'
//usr/lib/x86_64-linux-gnu/libX11.so.6: error adding symbols: DSO missing from command line
collect2: error: ld returned 1 exit status
make[2]: *** [particles] Fehler 1
make[1]: *** [CMakeFiles/particles.dir/all] Fehler 2
make: *** [all] Fehler 2

All right, changed to cmake with GLEW and GLFW now included locally :)


libpointmatcher is not supported on Unbuntu 32bit

CMake should not allow any compilation on 32bit Ubuntu.
In the current state, it compiles and some unit tests crashes with other weird bugs.

Also the 32bit job on the build farm fails: http://jenkins.ros.org/view/IbinT32/job/ros-indigo-libpointmatcher_binarydeb_trusty_i386/209/console


New user - no account / profile

Since there is no account / profile  i can not buy "timed practice"    >> start "non timed practice page"  is not skipped.   The button  is actually called just "start practice" . Script log  says "found next button".
It could be the same behavior for anyone without timed practice purchase.

at the end of the practice "Time to create a profile!"   page  is not bypassed :(
There is no next button , just small link "Create a profile later"

Please fix it at your leisure time, thanks

sorry wrong repository , this one was for duoleavemealone


Mg unstyled switch

I am fine with giving this a try.

Should we fix the warnings from Hound awell?


minor bug fix

Click here if you want to create code_sniffer's auto-correct Pull Request

See the documentation if you want to get more information about auto-correct.
Document


Move NSNotificationQueueTests to Foundation.WindowsOnly

Two things:

THRunLoopSpinner does need an input source on OS X (@ms-jihua you were totally right.) That isn't fixed here.
NSNotificationQueue is far simpler in WinObjC; the runloop spinner forces the run loop to never be "idle" on OS X, which makes the "three kinds of notification" test fail.'

This fixes the OS X test break.

This change is

Due to a class initialization order issue with WindowsOnly tests, NSNotificationQueue cannot be moved there. I've switched to OSX_DISABLED_TEST and made sure it compiled on OS X.

fwiw, the name of this pull request will be ignored when I choose rebase and merge

(title fixed)


Germplasm Markerprofiles [/brapi/v1/germplasm/{germplasmDbId}/markerprofiles]

2 issues:

should all result objects contain the data object?
json names in the result object are missing apostrophes

Versailles Hackathon:
Discussed difference between 3 types of records
Discussed possibility of adding a data array to everything
Realized that that would be a breaking change for many.

Could mean that you repeat the master data in each element of the data array, but that would also mean increasing payload size.

With the development of BrAPI V2, this issue has been resolved. markerProfiles has been replaced with the GA4GH Variants Schema. The equivalent GA4GH entity is a callSet
To retrieve the same data:
(1) Filter by Germplasm
GET /callsets?germplasmDbId=DbId
-> Returns set of `callSet` entities
   (will always have paginated `data` array)

(2) Retrieve genotype data
GET /callsets/{callSetDbId}/calls
-> returns genotype data for a single `callSet`
   (will always have paginated `data` array)

OR
POST /search/calls
    { "callSetDbIds": ["dbid1","dbid2","dbid3"...]}
-> returns SearchResultDbId
   (will not have `data` array)

GET /search/calls/{searchResultDbId}
-> returns genotype data for all callSets in search list
   (will always have paginated `data` array)


Fix: Fixed context menu hang in recent files and other widgets

Resolved / Related Issues
Items resolved / related issues by this PR.

Closes issue by #11204 where context menu didn't show up until shell extensions loaded
Continues work by @hishitetsu on #11270

Validation
How did you test these changes?

[x] Built and ran the app

Screenshots (optional)
Add screenshots here.

The menu options don't load with these changes, I'm not convinced we need them though. It's working pretty well as is.

The shell extensions don't load with these changes, I'm not convinced we need them though. I'm not running into any issues before this PR.

This uses the Loading menu options text we implemented and makes the context menu much quicker to load

It seems that I broke shell extensions on my device 😂
Will test after restarting

How did you do that lol

Probably by spamming requests


Updated pods & Removed compilation errors

Using Xcode 7.1.1

@dearprakash thanks for doing all this work - i will review tonight


Fix bugs in Python build process. Add documentation for pip install.

The current FW SDK docs are not particularly clear about the steps needed for Python installation. This PR adds documentation and fixes bugs I encountered when installing the FW SDK for my project with @lmperry and @garikoitz.

PTAL @gsfr

Thanks, @cgc. Punting review to @ehlertjd.

Thank you @cgc, great work!


change arsenal branch -- TEST only

@ironman-machine what say you

Hello @LaurenSpiegel,
Starting end to end procedure using the following payload:
{"DEFAULT_BRANCH":"master","SCALITY_S3_BRANCH":"test/checkHeaders"}
Please follow http://ci.ironmann.io/gh/scality/Integration/2283 for circle status.

@ironman-machine let's check rel/1.1
DEFAULT_BRANCH=rel/1.1

Hello @LaurenSpiegel,
Starting end to end procedure using the following payload:
{"DEFAULT_BRANCH":"master","SCALITY_S3_BRANCH":"test/checkHeaders"}
Please follow http://ci.ironmann.io/gh/scality/Integration/2293 for circle status.


experiencing flakiness for Test_CLI_VsServer/LiveDeployRefresh tests

Observing issues for this tests (which usually pass after retry)
Error:
aspire_test.go:399: 16.371s [svr-out] [sY9QECvcQE0= spinner] Analyzing Aspire Application (this might take a moment...)
    aspire_test.go:399: 17.855s [t-out]   Failed LiveDeployRefresh [15 s]
    aspire_test.go:399: 17.855s [t-out]   Error Message:
    aspire_test.go:399: 17.855s [t-out]    StreamJsonRpc.RemoteInvocationException : importing services: generating app host manifest: generating app host manifest: dotnet run --publisher manifest on project 'C:\Users\CLOUDT~1\AppData\Local\Temp\Test_CLI_VsServerLiveDeployRefresh288712408\001\AspireAzdTests.AppHost\AspireAzdTests.AppHost.csproj' failed: exit code: 1, stdout: Building...
    aspire_test.go:399: 17.855s [t-out] MSBUILD : error MSB4166: Child node "2" exited prematurely. Shutting down. Diagnostic information may be found in files in "C:\Users\cloudtest\AppData\Local\Temp\MSBuildTemp\" and will be named MSBuild_*.failure.txt. This location can be changed by setting the MSBUILDDEBUGPATH environment variable to a different directory.
    aspire_test.go:399: 17.855s [t-out] , stderr:
    aspire_test.go:399: 17.855s [t-out] The build failed. Fix the build errors and run again.
    aspire_test.go:399: 17.855s [t-out] exit with code: 1
    aspire_test.go:399: 17.855s [t-out]
    aspire_test.go:399: 17.855s [t-out]   Stack Trace:
    aspire_test.go:399: 17.855s [t-out]      at StreamJsonRpc.JsonRpc.InvokeCoreAsync[TResult](RequestId id, String targetName, IReadOnlyList`1 arguments, IReadOnlyList`1 positionalArgumentDeclaredTypes, IReadOnlyDictionary`2 namedArgumentDeclaredTypes, CancellationToken cancellationToken, Boolean isParameterObject)
    aspire_test.go:399: 17.855s [t-out]    at AzdVsServerTests.AcceptanceTests.LiveDeployRefresh() in D:\a\_work\1\s\cli\azd\test\functional\testdata\vs-server\tests\AcceptanceTests.cs:line 144
    aspire_test.go:399: 17.855s [t-out]    at NUnit.Framework.Internal.TaskAwaitAdapter.GenericAdapter`1.BlockUntilCompleted()
    aspire_test.go:399: 17.855s [t-out]    at NUnit.Framework.Internal.MessagePumpStrategy.NoMessagePumpStrategy.WaitForCompletion(AwaitAdapter awaiter)
    aspire_test.go:399: 17.855s [t-out]    at NUnit.Framework.Internal.AsyncToSyncAdapter.Await[TResult](Func`1 invoke)
    aspire_test.go:399: 17.855s [t-out]    at NUnit.Framework.Internal.AsyncToSyncAdapter.Await(Func`1 invoke)
    aspire_test.go:399: 17.855s [t-out]    at NUnit.Framework.Internal.Commands.TestMethodCommand.RunTestMethod(TestExecutionContext context)
    aspire_test.go:399: 17.855s [t-out]    at NUnit.Framework.Internal.Commands.TestMethodCommand.Execute(TestExecutionContext context)
    aspire_test.go:399: 17.855s [t-out]    at NUnit.Framework.Internal.Execution.SimpleWorkItem.<>c__DisplayClass3_0.<PerformWork>b__0()
    aspire_test.go:399: 17.855s [t-out]    at NUnit.Framework.Internal.ContextUtils.<>c__DisplayClass1_0`1.<DoIsolated>b__0(Object _)
    aspire_test.go:399: 17.855s [t-out]    at System.Threading.ExecutionContext.RunInternal(ExecutionContext executionContext, ContextCallback callback, Object state)
    aspire_test.go:399: 17.855s [t-out] --- End of stack trace from previous location ---
    aspire_test.go:399: 17.855s [t-out]    at System.Threading.ExecutionContext.RunInternal(ExecutionContext executionContext, ContextCallback callback, Object state)
    aspire_test.go:399: 17.855s [t-out]    at NUnit.Framework.Internal.ContextUtils.DoIsolated(ContextCallback callback, Object state)
    aspire_test.go:399: 17.855s [t-out]    at NUnit.Framework.Internal.ContextUtils.DoIsolated[T](Func`1 func)
    aspire_test.go:399: 17.855s [t-out]    at NUnit.Framework.Internal.Execution.SimpleWorkItem.PerformWork()

examples:
https://dev.azure.com/azure-sdk/internal/_build/results?buildId=3826453&view=logs&j=12494761-d332-5a1c-a5ee-2bea7198f339&t=49a5965c-dbc4-5a2f-8336-ff276c145143&l=216

Another instance here:
https://dev.azure.com/azure-sdk/internal/_build/results?buildId=3829873&view=logs&j=c8fbea3f-f5ea-553e-32cf-eaf013f8156e&s=2b1cf740-7e83-5b38-9f56-89c750cf5d77&t=e3650d02-8881-5593-c0d3-72e4766d33f9&l=186
=== FAIL: cli/azd/test/functional Test_CLI_VsServer/LiveDeployRefresh (1.94s)
    vs_server_test.go:199: DIR: /var/folders/24/8k48jl6d249_n_qfxwsl6xvm0000gn/T/Test_CLI_VsServerLiveDeployRefresh1713028754/001
    recording.go:291: recorderProxy started with mode replay at https://<IP_ADDRESS>:50237/
    vs_server_test.go:240:
        	Error Trace:	/Users/runner/work/1/s/cli/azd/test/functional/vs_server_test.go:240
        	Error:      	Received unexpected error:
        	            	unexpected end of JSON input
        	Test:       	Test_CLI_VsServer/LiveDeployRefresh
        	Messages:   	value:
    --- FAIL: Test_CLI_VsServer/LiveDeployRefresh (1.94s)

instance:
=== Failed
=== FAIL: cli/azd/test/functional Test_CLI_VsServer/ManageEnvironments (1.84s)
    vs_server_test.go:199: DIR: /var/folders/24/8k48jl6d249_n_qfxwsl6xvm0000gn/T/Test_CLI_VsServerManageEnvironments1259570751/001
    vs_server_test.go:240:
        	Error Trace:	/Users/runner/work/1/s/cli/azd/test/functional/vs_server_test.go:240
        	Error:      	Received unexpected error:
        	            	unexpected end of JSON input
        	Test:       	Test_CLI_VsServer/ManageEnvironments
        	Messages:   	value:
    --- FAIL: Test_CLI_VsServer/ManageEnvironments (1.84s)

https://dev.azure.com/azure-sdk/internal/_build/results?buildId=3838223&view=logs&j=c8fbea3f-f5ea-553e-32cf-eaf013f8156e&t=e3650d02-8881-5593-c0d3-72e4766d33f9&l=182

The last 2 reported issues are due to azd vs-server not starting after 1.5 seconds. We could just add longer wait time to address the flakiness, but it'd be equally interesting to see what's going on from a perf perspective.

Not seeing this lately. If it happens, please re-open.


Second fix for back button

The changes to this code alone will open the files in VZCode, but also sync with the URL once you hit the back button to go back to a file you clicked on.
What you CAN do this moment: you can click the back button to move back to a file you opened, and it will go back to the file you worked on previously but you can only do it once, as of right now.

Closing in favor of https://github.com/vizhub-core/vzcode/pull/746


fix spelling error in project description

@joshbuchea Thank you Josh!

@marcobiedermann no prob man! Don't forget to update the description at the top of repo page also 😉


some problems with ctag generation

There are some cases where the ctag generation not working correctly:
readonly
class Greeter {
    private readonly greeting: string;
}

you get readonly : members displayed instead of greeting: member
doubled entries
class Test {
    private testPrivate(): void {}
}

results in
  Test : classes

  Test : classes

  testPrivate : members

 -testPrivate : members

Multiple interfaces
class Test implements Test, Test2 {
    aPublicMethod(): void {}
    private testPrivate(): void {}
}

results in
  Test : classes

  Test : classes

  testPrivate : members

The public methods aren't listed

Ctags is just some regex. Public method without access modifier cannot be distinguished from others.


refactored precedence rules

Description
Made an attempt to simplify precedence reasoning and to make it more consistent.

Got rid of most custom precedence levels
Replaced them with a single LOW precedence level that tries to shift tokens eagerly
Added documentation for the empty token to include its two primary use cases
Use the empty rule everywhere custom precedence was currently used. This is the part I'm least certain about. I feel like it helps readability because

the comment for the empty token is now clear on its uses
when used for precedence enforcement it always occurs in a place where the preceding token appears in a similar context in another rule, followed by something non-empty

Added a missing precedence rule for the MATCHING terminal token

A lot of this is very subjective so I'd appreciate your input.
@Hugo-Inmanta I added you as a reviewer as well in case you have an opinion. If you feel like it would be too much effort to refresh these concepts for the review, feel free to unassign yourself.

@Hugo-Inmanta I think the impact should be limited, as the precedence rules I replaced only occur in places where they are never compared with the tokens I skipped over afaik. But there is indeed a minor risk, though we should catch that in the solutions team nightlies if anything goes wrong.


Cable Composite Object Using Wrong Area Moment of Inertia

Intro
Hello,
I am a grad student using Mujoco to simulate a soft bodied robot made with the cable composite elements. Despite having the proper dimensions and Twist/Bend properties for the material I noticed my deflections were not matching the physical robot we used as our comparison point so I did a few tests and dove into the base code.
My setup
Mujoco, Python
What's happening? What did you expect?
In the plugin file cable.cc (https://github.com/google-deepmind/mujoco/blob/main/plugin/elasticity/cable.cc#L134) lines 180-201 show the area moment of inertia calculation for the cable elements. I have selected capsule in the xml code which should mean Iy, Iz, and J for the circular cross section are used (lines 184-185). However, when I run a simple beam bending test, maintaining small deflection to conserve the small angle approximation, it produces stiffness that lines up almost perfectly with the rectangular cross section equations of the box elements (lines 193-195). Is it possible that the if statement in line 180/181 is not being executed properly and is instead always doing the box geom calculations for some reason?
Steps for reproduction

Put the python code (copied into a python file) and the xml model (in the Minimal Model section) in the same folder. Note that the model is attached as a txt file and will need to be copy/pasted into an xml file as xml is apparently not a supported attachment type.
Run the code.
You will see the rectangular area moment of inertia calculation and the measured area moment of inertia line up while the circular cross section, which should be used for the capsule type, is off by a lot.

Minimal model for reproduction
cable_xml.txt
Code required for reproduction
import numpy as np
import mujoco
import mediapy as media
import matplotlib.pyplot as plt
import os
import math

work_dir = os.getcwd()
xml_path = work_dir + r"\cable.xml"
model = mujoco.MjModel.from_xml_path(xml_path)
data = mujoco.MjData(model)

mujoco.mj_forward(model, data)

muscle_ind = 0      # might be 2??
tmax = 4
dt = 0.1/1000
t = np.arange(0, tmax, dt)

model.opt.timestep = dt

max_act = 0.01
force_app = 0.1

ctrl_ramp = np.ones(len(t))*max_act
ctrl_ramp[0:int(len(t)/8)] = np.linspace(0,max_act,int(len(t)/8))
frames = []
framerate = 60
muscle_force = np.zeros(np.size(t))
x_pos = np.zeros(np.size(t))
tendon_length = np.zeros(np.size(t))
slider_id = mujoco.mj_name2id(model,mujoco.mjtObj.mjOBJ_BODY,"slider")
for i in range(len(t)):
    data.xfrc_applied[slider_id] = [force_app, 0, 0, 0, 0, 0]
    mujoco.mj_step(model, data)
    x_pos[i] = data.xpos[slider_id,0]
# print(ctrl_ramp)

E_mod = 3e8
# E_mod = 10e8
cable_L = 0.3

radius = 0.00299
I_calc = (math.pi/4)*pow(radius,4)
print("Calculated Area Moment of Inertia for Circular C/S: ", I_calc)

I_calc_sq = pow(2*radius,3)*2*radius/12
print("Calculated Area Moment of Inertia for Rectangular C/S: ", I_calc_sq)

print("Force = ", force_app)
Disp = np.max(x_pos)
print("Displacement = ", Disp)

I_meas = (force_app*pow(cable_L,3))/(Disp*3*E_mod)
print("Measured Area Moment of Inertia: ", I_meas)

Confirmations

[X] I searched the latest documentation thoroughly before posting.
[X] I searched previous Issues and Discussions, I am certain this has not been raised before.

Hi this is unlikely as we have a test for the correctness of the bending stiffness of the capsule https://github.com/google-deepmind/mujoco/blob/main/test/plugin/elasticity/elasticity_test.cc#L255 Are you sure about your formula for I_meas?

Hello, I am not sure how the built-in correctness test works but I am certain my I_meas is correct. The code I included above performs a cantilever, end-loaded beam bending analysis and the I_meas is calculated using max displacement, beam length, elastic modulus, and applied force all of which are prescribed beforehand or measured during the simulation. The equation I used is a re-organized version of a common statics beam bending equation found here https://mechanicalc.com/reference/beam-deflection-tables, where I multiplied both sides by I and divided both sides by disp, ie. swapped the I and disp variables. I made sure to run the simulation long enough that the beam stopped moving to achieve max displacement. The output I get when I run the code is as follows:
Calculated Area Moment of Inertia for Circular C/S:  6.277325295188244e-11
Calculated Area Moment of Inertia for Rectangular C/S:  1.0656718401333334e-10
Force =  0.1
Displacement =  0.028637255408429853
Measured Area Moment of Inertia:  1.0475864244717039e-10
Given that the measured value is off by just 2% for the rectangular I and a significant 49% for the circular I, it's clear to me that the rectangular is being used in the simulation.
If I made a mistake while constructing the model I am happy to fix it but as you can see in the included xml (.txt file) I have the geom type set to capsule. The only other place I could see being an issue is the default section in lines 16-24 where I do not define a type in the geom part, but when I commented that out it produced the same results.

2% seems a bit off.. Could you debug it by putting a printf directly here https://github.com/google-deepmind/mujoco/blob/main/plugin/elasticity/cable.cc#L200 ?

I ran a further analysis with a variety of different conditions and it seems that my initial method was flawed. By fixing the first element of the cable I inadvertently reduced the bendable length of the beam but used the full length in the calculation. Increasing the number of elements in the cable got me closer and closer to the expected circular area moment of inertia value as the fixed length decreased and the bendable length increased closer to the full beam length. Apparently it was just coincidental that the rectangular calculation was very close.


[WIP] Protocol handlers without web+ prefix in Firefox 59

This PR implements changes described in https://github.com/ipfs-shipyard/ipfs-companion/issues/164#issuecomment-355708883, namely support for simplified (redirect-based) protocol handlers for  dweb: and ipfs:// without web+ prefix.
It works perfectly fine with latest Firefox Nightly (59a1), but fails to load in Firefox <59:
$ web-ext run -s add-on/ --browser-console --verbose
[firefox/index.js][debug] Firefox stdout:<PHONE_NUMBER>552	addons.webextension.<unknown>	ERROR	Loading extension 'null': Reading manifest: Error processing protocol_handlers.3.protocol: Value "dweb" must either: be one of ["bitcoin", "geo", "gopher", "im", "irc", "ircs", "magnet", "mailto", "mms", "news", "nntp", "sip", "sms", "smsto", "ssh", "tel", "urn", "webcal", "wtai", "xmpp"], or match the pattern /^(ext|web)\+[a-z0-9.+-]+$/

Due to this and a false-positive error in addon-linter (https://github.com/mozilla/addons-linter/issues/1783) I am parking this PR until Firefox 59 is released to Beta (it may even wait til Stable).

I rebased this PR to use the latest web-ext that includes addon-linter  with updated rules (https://github.com/mozilla/addons-linter/issues/1783).  This means npm run lint no longer breaks our tests 👌
Unfortunately if we enable non-prefixed handlers in manifest.json Firefox < 59 will refuse to load extension at all (Chrome is just ignoring unknown keys and runs fine).
That is why I am parking this PR until Firefox 59 becomes the new Stable.

Version 59.0, first offered to Release channel users on March 13, 2018
- https://www.mozilla.org/en-US/firefox/59.0/releasenotes/

Will rebase and merge this ASAP, so that we have a new FF59-enabled beta release :)

Rebased and smoke-tested on Linux with Firefox 59, works as expected.
Build for Firefox 58
Non-prefixed protocol handlers in manifest break Firefox 58 (extension just does not load) so to ease transition for casual contributors and testers I've added an optional target that builds bundle for the old version without this new feature:
npm run build bundle:firefox:58

Keep in mind this target is temporary, disabled by default and will be removed in 2 weeks.
Upgrade to Firefox 59 ASAP.

If anyone has trouble with downloading Firefox 59 there seems to be an upstream problem:

https://bugzilla.mozilla.org/show_bug.cgi?id=1445672


berkeley-schema-fy24: JSON Schema and YAML schema files are missing from nmdc_schema directory

Background
In PR https://github.com/microbiomedata/berkeley-schema-fy24/pull/229/files, the files nmdc.schema.json, nmdc_materialized_patterns.yaml, and nmdc_materialized_patterns.schema.json were deleted from the nmdc_schema directory (that is on the berkeley-schema-fy24 fork).
Task
Assuming those files (by some name) still exist somewhere in the repo file tree; this task is to document their new locations; for example, in the repository's README.md file. In the off chance that those files were deleted unintentionally, the task is—instead—to restore them.

Quick note for now:
When I start a backmerge, there's usually about 20 files marked with conflicts by git. Running make squeaky-clean deletes most of them, and they can all be recreated with make squeaky-clean all if I remember.
Same thing for poetry.lock. If there's a conflict, I delete it and hopefully remember to poetry lock before giving myself credit for completing the backmerge.
I started a backmerge yesterday and have pushed it in a state where there's still ~ 6 conflicts in nmdc.yaml.

https://github.com/microbiomedata/nmdc-schema/pull/2149

I think getting to that point was pretty easy, so I could close that PR start over again, going though the whole process with you @eecavanna  or @aclum @pkalita-lbl @sierra-moxon @sujaypatil96 @mslarae13.

I will fix these missing files now.

git add nmdc_schema/*json nmdc_schema/*yaml

The following paths are ignored by one of your .gitignore files:
nmdc_schema/nmdc.schema.json
nmdc_schema/nmdc_materialized_patterns.schema.json
nmdc_schema/nmdc_materialized_patterns.yaml
hint: Use -f if you really want to add them.
hint: Turn this message off by running
hint: "git config advice.addIgnoredFile false"

I'm going to force add them for now, but I guess we should do some .gitignore grooming.

Are there still conflicts in nmdc.yaml? the linked PR indicates not.

It's relatively new comment so I'll move to new sprint.  @turbomam  let me know if you're not working on this.


Azure test is broken due to missing authentications

🐛 Describe the bug
See example CI test result:
https://github.com/pytorch/data/actions/runs/3496367745/jobs/5854203850#step:13:527
To resolve it, we need to properly set environment variables AZURE_CLIENT_ID, AZURE_TENANT_ID and AZURE_CLIENT_SECRET as shown in https://github.com/Azure/azure-sdk-for-python/blob/main/sdk/identity/azure-identity/TROUBLESHOOTING.md#troubleshoot-environmentcredential-authentication-issues
To temporarily mitigate the problem, I am going to skip the test
Versions
main

I think this can be closed after #905 has landed?

I prefer keeping it open since the test failure hasn't been fixed. I just skipped it.


Featured FAQs on homepage

@argyleink I've deviated with the design slightly here. Rather than give a preview of the FAQ item, I've gone with the whole thing, since they're pretty short. We can revisit this when we get the real content.

cool, i'm down with this as a place to revisit, we'll solve the big text blocks soon. i'm hoping we can progressively enhance with some css line clamp, thoughts on that? might be minimal enough LOE to just nip that bud right now?
the deviations around color; I'd like those changed to match the comps. Gray cards on white is not as appealing as white cards on light gray.
the typography is fine, but is uncanny, been trying to put my finger on it. the font sizes are great, i think it's the weights that need adjusted. the title is too bold, and the body copy should be set to thin. here's some quick changes i made with visbug to show what i mean

i think once the weights change the blocks get truncated we'll be back to the less heavy feel. otherwise nice work, it's great getting content in there to work with!

In terms of the light grey thing: it seems like modules alternate in colour, so I made that happen here. If we add a module before this, it'll appear more like the design. Or, do you want the colours fixed?

i'd imagined colors were more fixed 👍

Interesting, so the intent isn't that they alternate, it just coincidentally ended up that way in the XD?

I guess they don't fully alternate. Ok, I'll fix this for all modules.

I've done it all except for the line-clamping. I'd rather see the real content before we line clamp, and I'd rather we didn't have to line-clamp.
Personally, I find line-clamping annoying, as it's obscuring the information. If we have some FAQs that are short enough to feature, I'd rather we just featured the whole thing.
If we get to the point of needing clamping, I'd rather remove the content all together and featuring headings that link to the content.

[ ] Add iOS scrolling
[ ] Add webkit prefix to max-content

cool, i think i'm seeing a compromise / enhancement here based on your valid comments and the IA i intended for the home page. here's my thought process:

i have a few FGs that we have a meaningful FAQ landing page
i have a few FGs around the homepage IA pattern of collection summaries
i agree line clamp has the effect of content masking, and i'm willing to design around that even though i dont fully agree it's always bad. low FGs

compromise idea: the FAQ summary/teaser module only shows FAQs that are within a certain answer text length.

no FAQ would be cut off
no FAQ would be ginormous and make me want to clamp it
still fits within the collection IA of being an intro to the greater FAQ list
seems like a fun js function to write

thoughts!?
also, i really like css snap points on horizontally scrolling lists like this and the media list. do you want me to add that requirement/request here, or make a new ticket for adding snap points to various horizontal scrollviews?

compromise idea: the FAQ summary/teaser module only shows FAQs that are within a certain answer text length.

I mean, that isn't a compromise idea, that's exactly what I've proposed and done.

seems like a fun js function to write

In the current implementation you get to choose which FAQs to feature on the home page. This means you can, during PR review, assess whether they're the right size. No JS needed.

also, i really like css snap points on horizontally scrolling lists like this and the media list.

I don't get it on the list of images. Like I said in the other ticket, I can't figure out the second half of "Scroll snap on a set of images helps the user because…"
IMO scroll-snap makes sense when you want one item to have focus, but that isn't true in these cases. I worry that it'd feel like we're dicking around with scrolling because it's a new shiny feature.

coo, so with a featured flag, my guidance would be to select featured FAQs based on length and relevance. we want short and meaningful FAQs in the featured area 👍
curious if these guidances will be enforceable at the CMS level, or at least blurbs we can share. that FAQ module space is easy to make ugly. but these worries are also very future facing for the white labeled version. i'll pacify them for now!
scrollsnap points is part of the design btw, def not just using it for fun, i want that left alignment as you scroll. it can be so subtle folks dont even realize it's there, and if the visiting browser doesnt support it, they'll get a great experience anyway. i'd prefer we had snapping on most of these homepage collection scrollviews, but maybe that's another ticket to make so it doesnt hinder the current PR?

Yeah, let's make the scrollsnap thing another PR. Unfortunately the implementation isn't as trivial as it might seem.


export_tflite_ssd_graph_lib.py didn't work with 1.10.0-rc1 ?

System information

Have I written custom code (as opposed to using a stock example script provided in TensorFlow): Y
OS Platform and Distribution (e.g., Linux Ubuntu 16.04): Linux  4.17.11-arch1
Mobile device (e.g. iPhone 8, Pixel 2, Samsung Galaxy) if the issue happens on mobile device: None of them
TensorFlow installed from (source or binary): source
TensorFlow version (use command below):b'v1.9.0-rc2-1981-gb674f63ab1' 1.10.0-rc1
Python version: 3.6
Bazel version (if compiling from source): 0.15.2- (@non-git)
GCC/Compiler version (if compiling from source): gcc-7
CUDA/cuDNN version: cuda-9.2 cudnn-7.1
GPU model and memory: 24G
Exact command to reproduce:

Everything works fine before I updated the newest source code ,compiled and installed it.
With the installed version(1.10.0-rc1) the export_tflite_ssd_graph_lib.py  said
2018-08-09 03:18:34.547212: I tensorflow/tools/graph_transforms/transform_graph.cc:318] Applying strip_unused_nodes 2018-08-09 03:18:34.565931: I tensorflow/tools/graph_transforms/transform_graph.cc:318] Applying fold_constants Traceback (most recent call last):   File "export_tflite_ssd_graph.py", line 137, in <module>     tf.app.run(main)   File "/usr/lib/python3.6/site-packages/tensorflow/python/platform/app.py", line 125, in run     _sys.exit(main(argv))   File "export_tflite_ssd_graph.py", line 133, in main     FLAGS.max_classes_per_detection)   File "/home/mae/tf-newest-nightly/tensorflow/tensorflow/models/research/object_detection/export_tflite_ssd_graph_lib.py", line 280, in export_tflite_graph     nms_score_threshold, nms_iou_threshold, num_classes, scale_values)   File "/home/mae/tf-newest-nightly/tensorflow/tensorflow/models/research/object_detection/export_tflite_ssd_graph_lib.py", line 137, in append_postprocessing_op     output_names, transforms)   File "/usr/lib/python3.6/site-packages/tensorflow/tools/graph_transforms/__init__.py", line 51, in TransformGraph     transforms_string, status)   File "/usr/lib/python3.6/site-packages/tensorflow/python/framework/errors_impl.py", line 519, in __exit__     c_api.TF_GetCode(self.status.status)) tensorflow.python.framework.errors_impl.NotFoundError: Op type not registered 'TFLite_Detection_PostProcess' in binary running on Mae. Make sure the Op and Kernel are registered in the binary running in this process. Note that if you are loading a saved graph which used ops from tf.contrib, accessing (e.g.)tf.contrib.resamplershould be done before importing the graph, as contrib ops are lazily registered when the module is first accessed.
Other scripts such as train and eval can work except the exporter.

I found If i remove the 'fold_constants' in TransformGraph processing,it would be OK. Thanks.

hello, how did you remove the 'fold_constants'?     I face the same problem and can not find  "fold_constants".


[FEATURE]

Is your feature request related to a problem?
Yes, S3 log pipeline – Listens to S3 Amazon SQS notifications generated via eventbridge and pulls data from S3 buckets. I am getting invalid body which cannot be parsed into S3EventNotification. Unrecognized field "version" (class org.opensearch.dataprepper.plugins.source.S3EventNotification), not marked as ignorable (one known property: "Records"]
What solution would you like?
I want any sqs event generated via s3-sqs,s3-sns or s3-eventbridge-sqs  should be parsed in ingest pipeline source as s3.
What alternatives have you considered?
Right now I dont have any alternative
Do you have any additional context?
2023-09-16T11:58:29.321 [Thread-11] ERROR org.opensearch.dataprepper.plugins.source.parser.S3EventNotificationParser - SQS message with message ID:414afe99-8914-4fb4-b9ed-782c228a0ddb has invalid body which cannot be parsed into S3EventNotification. Unrecognized field "version" (class org.opensearch.dataprepper.plugins.source.S3EventNotification), not marked as ignorable (one known property: "Records"])
at [Source: UNKNOWN; byte offset: #UNKNOWN] (through reference chain: org.opensearch.dataprepper.plugins.source.S3EventNotification["version"]).

It looks like the issue you are facing is that the SQS messages generated by Amazon EventBridge contain an additional "version" field that the S3EventNotificationParser in opensearch-py is not expecting.
Since EventBridge uses a different message format than plain S3 notifications, the parser needs to be updated to handle the extra EventBridge-specific fields.


Add std and stddev function

What problem does this PR solve?
Add std and stddev function for issue:https://github.com/pingcap/tidb/issues/7623
What is changed and how it works?
Check List
Tests

Unit test
Integration test

Code changes

Has exported variable/fields change

Side effects
Related changes

Need to cherry-pick to the release branch

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

@SunRunAway PTAL

Since std, stddev and stddev_pop are aliases of each other, i prefer all 3 names mapped to the same function stddev_pop. refer to var_pop vs variance for details.

/build

LGTM

Please hold it for the PR in TiDB side.

FAIL: aggregate_test.go:44: testSuiteAgg.TestAggregation

aggregate_test.go:410:
    c.Assert(errors.Cause(err).Error(), Equals, "unsupported agg function: std")
... obtained string = "unsupported agg function: stddev_pop"
... expected string = "unsupported agg function: std"

better update the TiDB unit test, otherwise we couldn't refer the latest parser

/run-cherry-picker

/cherry-pick


[ergoCubSN001] Change the version of the BAT and BMS

I found these files uncommitted on the robot. I think they are important.
cc @MSECode @traversaro @pattacini

cc @valegagge

Hi @GiulioRomualdi,
I think the version should be aligned with what defined in the firmware info file at these lines: https://github.com/robotology/icub-firmware-build/blob/fb6d2150403efcae8ffede9e8887a5473997e2df/info/firmware.info.xml#L48-L49
It is possible to set all zeros to the versions in order to skip the checks about the matching between what defined in the conf and what running on the board at the YRI startup but I'm not sure if it is the best practice to use. I think we should set the version as the one running on the board, i.e. 1.3.3.
Anyways, is the YRI rising any error flags complaining about the version mismatch?

Hi @MSECode

It is possible to set all zeros to the versions in order to skip the checks about the matching between what defined in the conf and what running on the board at the YRI startup but I'm not sure if it is the best practice to use. I think we should set the version as the one running on the board, i.e. 1.3.3.

We recently decided to drop this versioning done inside the XML files because it is barely maintainable.
New developments on the FW side will compensate for this.

Hi @MSECode

It is possible to set all zeros to the versions in order to skip the checks about the matching between what defined in the conf and what running on the board at the YRI startup but I'm not sure if it is the best practice to use. I think we should set the version as the one running on the board, i.e. 1.3.3.

We recently decided to drop this versioning done inside the XML files because it is barely maintainable. New developments on the FW side will compensate for this.

Oks, good to know. I did not know about that. So I think it's all good with that PR and thanks @GiulioRomualdi


Add support for hostPath volumes

This adds support for hostPath volumes to the persistence volume as well as extra mounts. This helps in scenarios where someone wants to install the helm chart on a development machine with an existing directory containing the project. Using hostPath he/she can continue working with a native VS code and switch back and forth to/from the web-based code-server.

Apologies, I completely missed this, believe it or not I was actually doing some work ;) This commit looks great, can't see any issues and it deploys fine in KinD.
Great work!

You are welcome and thanks @Matthew-Beckett!
I suggest amending the README to have a mention for the hostPath in the table including the above snippet in my comments as a working scenario.

I also have another comment. When the release is deleted, the secret containing the generated passwords does not get deleted. Do you have an idea why? I haven't investigated yet.

@hammady does your secret resource in your cluster have a pre-delete hook it is waiting for?


Not working

Describe the bug:

Link to the blitz that caused the error
https://stackblitz.com/edit/order-sheet
Steps to reproduce:

Go to '...'
Click on '....'
Scroll down to '....'
See error

Expected behavior:

Version of webcontainer:
Hash: 70a1ff62a0f0131ac0df3c30faf487fef744ac90
Screenshots:

Desktop (please complete the following information):
Browser name  = Chrome
Full version  = 91.0.4472.114
Major version = 91
navigator.appName = Netscape
navigator.userAgent = Mozilla/5.0 (Macintosh; Intel Mac OS X 10_15_7) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/91.0.4472.114 Safari/537.36
performance.memory = {
  "totalJSHeapSize": 119622188,
  "usedJSHeapSize": 102809664,
  "jsHeapSizeLimit":<PHONE_NUMBER>
}

Additional context:

Can you please describe what's not working? I am afraid that without a proper description and more information we'll have to close this issue.

Closing this issue as it's unclear what goes wrong and none of the required details are filled in. Feel free to open a new issue and fill in all the necessary details so we have enough information to have a look ourselves. The more information we have, the better we can help you 🙏 .


Figma Plus stopped working

Hey, at first thank you, great work!
The issue. I have installed Figma Plus and it worked great at first. I activated dev mode, started to play with plugin boilerplate. But, after weekend I noticed the icon at right of the toolbar disappeared along with menu item.
I tried to uninstall and install Figma Plus a few times — nothing. Even Figma stopped working with this message:

Also tried to restart macOS — nothing. So I had to reinstall Figma ¯\_(ツ)_/¯
macOS Mojave 10.14.3
Figma Desktop App release 68
How can I help with that?

@romashamin
https://github.com/figma-plus/installer/issues/10

@aToshik uh, thank you! I didn’t check closed issues.

@romashamin @aToshik hello, the same problem with me how can I fix it?

I use Windows 10


Add pt-br translation

Resolved / Related Issues

Closes
Related #81 #82

Details of Changes
Added Portuguese (Brazil) translation
Validation

[x] Built and ran the app
[ ] Tested the changes for accessibility

Screenshots (optional)

merged. can you reopen and update?

FluentHub/MultilingualResources/FluentHub.es-ES.xlf file was moved to src/FluentHub/MultilingualResources/FluentHub.es-ES.xlf by #74. so you need update changes.


Enable support for multiple device connections

Users are now able to pass multiple device urls into the observer separated by commas.
Observer will send each file to all the addresses as parallel.
Unfortunately, the VS or VS4Mac extensions are not uploaded here.
Hope for supporting multiple connections from the extensions also :)

@ChasakisD hi )
Thanks a lot for contribution :)
But observer is obsolete way for observing files. You should use official VS extension (I hope multiple device support will be ready at next weekend)

Anyway thanks :)


RP: new styleguide colors

The styleguide color scheme is being expanded to allow for product specific color 'themes' to help contextualize categories + branding.

I believe this is most current.


Rewrote index.js using ES6 standards

Hello, I rewrote the entire index.js using ES6 / ES7 standards.
This reduces the file size and improves readability for no performance loss.

Hi @arguiot, I don't think there's a tangible benefit to swapping to ES6 right now. Users of the status page by-and-large don't read the source code, and the size of index.js is considerably outweighed by the JSON proposals payload.
If we introduced a transpilation step to this page, which I don't recommend we do, we could minify the JS as well as transpiling it, which would be a more effective way to reduce the size of the source files if that were a pressing issue. (For context, I originally wrote this incarnation of the status page in ES6 and backported it by hand to ES5 to support more browsers.)
Also, you should check out the contributing.md document for this repo. In particular, adopt the style rule about not using line-ending semicolons.
We can reevaluate this if and when the status page evolves and browser adoption progresses a bit more.


Multi-language support

whether this should be community run and take on the risk of bad actors or solely down to distributors is up to you

also plugins should have a metadata tag of what languages they support


Any chance of supporting VERSION_INFO in addition to app.manifest?

I really need to have an icon, a manifest and version details
Currently I'm caught between this project and https://github.com/josephspurrier/goversioninfo - which does icons and versions, but not manifests
Would it be possible to add VERSION_INFO support to rsrc? Even if it's just specifying a chunk of pre-generated binary output from another tool?

Yes it is - and I'd sure be happy to take such a contribution! Myself, I don't have plans to write that at the moment; from looking at goversioninfo, it seems non-trivial, and I don't have much motivation as of now (although, now that I've said that, I'm starting to feel some itching...). (If you're interested yourself, have a look at the description of VS_VERSIONINFO structure in MSDN. That's what I sent to josephspurrier when he contacted me originally, asking about the same.)
As to goversioninfo: from what I know, this project is heavily based on rsrc, but apparently in a non-compatible, "write from scratch while copying fragments from rsrc" way, and under a different license (APACHE) instead of a regular fork and contributing back to rsrc. My chosen license kinda allows that (although I haven't checked if the copyright messages were left where they should be left), so I can't complain much. But what's important here (and why I had to say that), is that I can't take back from him without changing my license. And that I'm not very eager to do.
Now, I don't know why he decided to drop the manifest functionality, as I understand from what you said; but if you like to go further "his way", you could try to have a look at func WriteSyso in goversioninfo.go; then compare it to func run in rsrc.go. They're... how to say that... "similar enough", that you should probably be able to patch the goversioninfo.go with a few more lines from rsrc.go that would bring the functionality there.
Does this resolve the situation for you? Happy to hear you're interested in it and find it useful!

I just talked with @tgulacsi and he is fine with me changing the license from Apache to MIT on https://github.com/josephspurrier/goversioninfo - I'll make sure his contributions are not overlooked. Let me know if you have any suggestions on how to change the structure to work with rsrc.

@josephspurrier That's great!
I must admit I can't wrap my head around the current organisation of the code in goversioninfo, so I'm thinking how one could try splitting it into somewhat smaller phases/commits, to make it more approachable for me and kinda MVP-ish with regards to integration in rsrc. The one and only idea I have for now, is that I'd like trying to start with implementing just some basic "number-ish" values — e.g. "File Version" & "Product Version". That would hopefully help me understand the initial requirements for embedding VS_VERSIONINFO in COFF; hopefully maybe strings and other stuff (?) could be then added in a later phase (with any additional complexity they may possibly bring). Can you possibly help me understand which of the structures & funcs/fragments in goversioninfo are still unavoidable if one would want to add only File Version & Product Version to the .syso?
I'd really like if I managed to understand the stuff before starting to think how to best structure it for merging into rsrc.

I'm thinking it may be easier than we think to integrate the rsrc and goversioninfo. As far as the requirements go, I'm thinking it's best to leave everything in. It will probably be more work to strip it down as opposed to integrate it as is. I'll do a pull request and you can see if it makes more sense to you.

It may be more work to split it to smaller chunks, but I can't deny it's not easy for me either when I have a perspective of integrating some code I have trouble  understanding into the tool, and then living with having to maintain it for better or worse indefinitely... Especially given that this whole thing is just a hobby project for me, so I don't really have much time and resources when I can work on it. Just to explain the reasons behind what I say.

I understand what you're saying. I'll do my best to explain it. I'll go as fundamental as I can so others following this can understand as well. Please correct me if anything is incorrect. I'll post a second note after this.
Typically, when you are developing an app in C++, you would use a resource script file (.rc) with this format for including version information in your application:
1       VERSIONINFO FILEVERSION     1,0,0,0 PRODUCTVERSION  1,0,0,0 FILEOS          0X40004 FILETYPE        0X1 BEGIN     BLOCK "StringFileInfo"     BEGIN         BLOCK "040904B0"         BEGIN             VALUE "FileVersion", "v<IP_ADDRESS>"         END     END     BLOCK "VarFileInfo"     BEGIN             VALUE "Translation", 0x0409, 0x04B0     END END
In C++, the resource compiler is used to convert the resource script file (.rc) to an object file (.o) - http://www.willus.com/mingw/colinp/win32/tools/rc.html. The linker is then used to convert the object files into applications or DLLs - http://www.willus.com/mingw/colinp/win32/tools/link.html. Those object files use the COFF format (there are a couple variations of COFF) - http://www.theparticle.com/cs/bc/os/elfpecoff.html.
In Go, there are system object files (.syso) which are also in the COFF format that are automatically picked up by Go when you run: go build. The rsrc tool makes it easy to generate the .syso file with icons and manifest.
GoVersionInfo uses rsrc to convert a json with Windows version information into a .syso.

I recently just found out you can duplicate all the work we've done with a file called windres.exe which is available in MinGW.

As to windres, I'm fully aware of it — actually, I typically use windres output as a reference guide when debugging issues in rsrc output. My reason for creating rsrc was to make the process easier for people. So that Go programmers wouldn't have to download and install full MinGW/GCC suite and learn the weird windres syntax just to embed a single icon and/or manifest in their Windows app.  And the other reason was that I was curious to test if the hack described by Minux (about syso files) would really work.
So, I'm afraid I must say those two most recent comments above don't really help me in anything. I'd much rather prefer to learn and understand technical details of the VERSIONINFO layout & requirements, so that I could remove as much effort as possible from the end user, and hide it inside the app instead. Or, for starters, at least how to use the goversioninfo package (what exactly input to provide) to do that.

Ah, thanks for that information. I originally believed your application was only way to embedded resources in a Go application, but now I understand your motivation behind the creation of rsrc.
I'll have to write up some information about the VERSIONINFO layout when I have some time.

Ah, yes, resources; I must admit I've already forgotten about that, given that this part is defunct now. So, you're right in that it was there from very early on. I don't really remember now which part was the first one. Still, I think what I wrote above applies this way or the other.

I just updated my application so the algorithm should be much better I was modeling it after the output from windres, but it looks like windres was not calculating section lengths correctly and I could not consistently duplicate the logic in their code so now goversioninfo should work very well.
@DavidR91  I also integrated the manifest ability into goversioninfo with a few lines of code so that can be used now in the meantime which we work on the integration.
I have a much better understanding of the VERSIONINFO layout and it's fresh in my mind so let's get started. Goversioninfo is basically broken down into two files goversioninfo.go and structbuild.go.
goversioninfo.go contains all the possible variables (in structs) like file version/product version/etc that are loaded into structs in structbuild.go which are modeled after the Version Information Structures.
The structure is really not complicated. Every structure (VSVersionInfo, VSFixedFileInfo, VSStringFileInfo, etc) has a similar format.
WLength is a word (2 bytes) in reverse that is the length (in bytes) of the current structure and all children. This is really important because it sets the boundaries.
WValueLength is a word (2 bytes) in reverse that is the length (usually in bytes) of the Value or Children. For the VSString the length is the number of words instead of bytes.
WType is a word (2 bytes) in reverse that is either 0 for binary of 1 for text. It's 1 in everything except in VSVersionInfo and VSVar.
SzKey is a byte array of a string. If the string is FileVersion with a length of 11, every letter is padded with 00 so the final length is 22.
Padding is required to align the next Value or Children on a 32-bit boundary (every 4 words). There must be at least 1 word (2 bytes) between each SzKey and Value/Children.
The Value is either a byte array of a string (VSString), 4 words (VSVar), or Children which is one or more structs under it.
All the VSStrings are listed here: https://msdn.microsoft.com/en-us/library/windows/desktop/ms646987(v=vs.85).aspx
Right now, my structure is pretty rigid. I only allow for one StringTable. I only allow the strings defined in the app even though they are just guidelines in the spec. I don't use any types of encodings which I'm sure is not good. I also don't think when I'm aligning to a 32-bit boundary, they are calculated correctly, but I haven't seen any problems. There should also be a way to add in more Children for VSVarFileInfo.

I'm open to others improving it for the PR as well - I don't have the time currently.

I'm trying to add this to the listing at http://up-for-grabs.net. For anyone interested, I'll note here, that I'd be very happy and eager to guide and mentor a contributor! On the other hand, I must note I can be busy and may have a few days of latency when responding, as Real Life(tm) interferes quite often these days. But I'd absolutely do my best to communicate as quickly as possible.
As to technical details, the task would be based on some MSDN exploration and related googling, focused on a fragment of PE/COFF executable format (absolutely not rocket science; interesting for me, being some lower-level OS-related stuff). Plus testing the feature on a Windows machine. Given the expressions of interest in the feature, if you want to help, you can be more than sure that your contribution would be actually useful for many people! Though you may also be surprised how silent is the sound of success ;)
A contribution can be based on #14 (taking into account the initial review notes) or can start from scratch, whichever is preferred by the contributor!

I also very much need the option to add version and vendor information. Maybe some news here?


High CPU utilization after upgrading to 0.3.0

Issue description
After upgrading to YiHack 0.3.0 the CPU load average climbs to  between 2.0 and 3.0 and the rtsp stream stops working or at the very least some frames are dropped. The issue goes away after downgrading to 0.29. With 0.2.9 the load average stabilizes at around 0.5.
Steps to reproduce the issue
Serial/suffix: 9FUS, y20ga
Base version: <IP_ADDRESS>A_202012041528
Upgrade from YiHack 0.29 to 0.30 (absolutely no changes to configuration)
I have 3 cameras and all have the same issue.
Expected behavior
Normal function of YiHack
Current behavior
Camera drops frames and sometimes loses connection with HA due(?) to heavy CPU load. I've tried disabling all services besides RTSP stream with no effect.

Maybe the service that creates snapshots is too expensive.
Did you try to enable swap space?

thumb.sh is the snapshot service that create snapshots of the captured mp4 files.
I added it in the 0.2.9 and at the moment it's not possible to disable it from web interface.
Please comment the line in system.sh:
echo "* * * * * /home/yi-hack/script/thumb.sh cron" >> /var/spool/cron/crontabs/root

What are too high load averages that requires disabling snapshot services?

If the swap is enabled, I think that your problem is different.
But you can find this file in /home/yi-hack/script/

Hmm.. Even after disabling the snapshot service the original issue persists. Load average is now down to about 1.4 (CPU % sits at around 30-40%) but the skipped frames issue is still there.
I even tried resetting one of the cameras and changed the credentials so that nothing else should be connecting to it. Still skipped frames/occasional freezes. Hmm... I'll fire up an old Wifi-router to exclude the possibility of networking issues.

... seems like it was a networking issue, after all. Well, now I have a reason to setup a separate wifi for the home automation system.


remove some lodash functions

Ref: #350

This results in all sorts of duplicate key errors. Reverted for @g-plane to investigate.

What does it mean?

I have removed two lodash functions (foreach and filter). Which removal did cause the problem?

I don't believe filter was being used anywhere... so it must be foreach. Try running the dev app in the repo, you'll see the errors I'm talking about.

So maybe we can remove lodash.filter again?

good point, will get rid of filter in next release.


Show status Wallet locked even if wallet is Disconnected or Synchronization in progress

Perhaps as additional info near Synchronizaion / Disconnected

Can be closed


TTS and Coder as root

Hi,
I was experimenting with Text2Speech, and found that most TTS node packages do not work with Coder (in fact none in my experiments). Finally, I wrote code to run espeak commands using child_process, and it only worked when running Coder as root. Here is the JavaScript code added at the end of the NODE code (NODE tab in Coder) which worked only when running Coder as root:
var run = require('child_process').exec;
exports.on_socket_say = function( socket, data ) {
if ( data.value !== "undefined" ) {
console.log('running server side code');
run('espeak -ven-us+f4 -p50 -s170 "Hello humans" --stdout | aplay', function callback(error, stdout, stderr){
console.log('error: ' + error);
console.log('stderr: ' + stderr);
console.log('stdout: ' + stdout);
});
}
};
I think that this behavior could also happen for other packages. So I did these changes (just replaced coder user by root) to the coder-daemon and it's working now:

do_start()
{
Return
0 if daemon has been started
1 if daemon was already running
2 if daemon could not be started
start-stop-daemon --start --background --quiet --pidfile $PIDFILE --exec $DAEMON
--chuid root --user root --umask $UMASK --chdir $DAEMON_PATH --test > /dev/null
RETVAL="$?"
[ "$RETVAL" = "0" ] || return 1
start-stop-daemon --start --background --quiet --pidfile $PIDFILE --make-pidfile --exec $DA$
--chuid root --user root --umask $UMASK --chdir $DAEMON_PATH  -- $DAEMON_ARGS
RETVAL="$?"
[ "$RETVAL" = "0" ] || return 2
}

Also changed this:
start-stop-daemon --stop --quiet --retry=TERM/30/KILL/5 --user root --pidfile $PIDFILE

But I have a question: Is there any side effect or something which may stop to work or malfunction by not running the Coder daemon with the Coder user? Or is this okay?
Of course, I can give further details of what happened in the console when running that code as Coder instead of root, but basically, it had problems with the ALSA drivers.
Thanks!
Julián

Hmmm. It would be fun to get this working right inside coder, and without using the root user. Do you know what permissions it needs? Can the pi user run this from the command line without using sudo?

Adding the pi user to the audio group should be enough