Add support for ITeardownSystem

This would be the opposite of ISetupSystem so this can be used to indicate when an entity no longer applies to a group and can do custom logic on the entity in question.

What condition would trigger that exactly? Is removing one component enough? would the system be notified what exact change to the entity triggered the tear-down?

The idea is that you provide it a group, and when something stops being part of that group (due to a component being removed) it would trigger the logic. This would allow you to have the ability to have complex view and binding setups in an ISetupSystem but then undo the setup in the ITeardownSystem. This would effectively allow you to have something similar to an IManualSystem but with it automatically handling new and removed entities automatically if you wanted it.
It would also improve use cases where systems have state pertaining to entities and you want to clean that up when the entity is no longer applicable.
public class CountingSystem : ISetupSystem, ITeardownSystem
{
   private int _localCount = 0;
   public IGroup TargetGroup { get { return new Group(typeof(CountedComponent)); } }

   public void Setup(IEntity entity)
   { _localCount++;  }

   public void Teardown(IEntity entity)
   { _localCount--; }
}

Something like that although that is super simple it just allow you to handle similar use cases where you need to do something once the entity is no longer within a given group, again its not a mainstream concern, but it at least provides that functionality for those who do need to do this sort of thing.

This has been released in 0.8.0 feel free to re-open if there are any issues with this


Octave compatibility: investigate this error in dsPlot

Executing post-processing function: dsPlot
Error: linkaxes: HAX must contain at least 2 handles
         in linkaxes (line 63)
         in dsPlot (line 901)
         in dsAnalyze>evalFnWithArgs (line 516)
         in dsAnalyze (line 151)
         in dsSimulate>tryFn (line 1057)
         in dsSimulate (line 751)
         in dsDemos (line 288)
created temporary lock file for this process: /home/jsardid/demo_cluster_2/.lock_20171002163147586_1
loading study file: /home/jsardid/demo_cluster_2/studyinfo.mat
deleting temporary lock file for this process: /home/jsardid/demo_cluster_2/.lock_20171002163147586_1
setting status='failed' where sim_id=1
error: wrong type argument 'matrix'
error: exist: NAME must be a string

I will upload a commit that fixes this in brief, testing it locally now.

fixed in 3fede1e


How to support active learning using this package?

Hello,
QnA Maker service supports active learning in run-time by providing best n matches to the user and allowing the end user to select the right question. How do we support that using QnAMakerDialog nuget package?

@youssefyoussef I have the same question. Have you figured it out?

@xiaolangxyz I just used the QnAMakerDialog by bot framework (namespace: Microsoft.Bot.Builder.CognitiveServices.QnAMaker) which has QnAFeedbackStepAsync method. It is a bit limited as I couldn't localize the response when no answer is present in the knowledge base.

@youssefyoussef
How do you think about this method? Is it helpful? Because I'm not using the QnAMakerDialog of BotBuilder-CognitiveServices. And the new API v4 doesn't support train knowledge base. I'm thinking not to use it.

It worked for my use case. Perhaps if I understand more about your needs I will be able to recommend which library to use? I train knowledge base using qnamaker.ai interface directly, not through API's. If I were to use the API's, I will probably use the REST API.

@youssefyoussef
This is my question .
Thanks for telling QnAFeedbackStepAsync is useful. I will consider to use it.. By the way if
Microsoft.Bot.Builder.CognitiveServices.QnAMaker), is it easy to use metadata?

@xiaolangxyz
Here is a sample for using Bot Builder QnAMakerDialog: https://github.com/Microsoft/BotBuilder-CognitiveServices/tree/master/CSharp/Samples/QnAMaker
I know metadata can only be managed using the REST API.


Make Sidebar Use the Current Theme

The readme includes instructions on setting a custom, hardcoded theme for dired. Is there anyway to make it use the current theme instead?

It shall use whatever scheme you set in user settings file, e.g. "color_scheme": "Packages/Color Scheme - Default/LAZY.tmTheme"
The reason, why readme advises to copy bundled one and modifying it, is that any other scheme most probably won’t contain FileBrowser-specific scopes, unless you add them youself.


Fix hipEnvVar test

This PR fixes indentation, binary path and adding wait on popen calls for the hipEnvVarDriver test.

You need to adjust the jenkinsfile to include the unit test again

Thanks Kent, updated in the PR ☺
Regards,
Peng
From: Kent Knox<EMAIL_ADDRESS>Reply-To: ROCm-Developer-Tools/HIP<EMAIL_ADDRESS>Date: Monday, July 24, 2017 at 1:24 PM
To: ROCm-Developer-Tools/HIP<EMAIL_ADDRESS>Cc: "Sun, Peng"<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [ROCm-Developer-Tools/HIP] Fix hipEnvVar test (#110)
You need to adjust the jenkinsfilehttps://github.com/ROCm-Developer-Tools/HIP/blob/master/Jenkinsfile#L122 to include the unit test again
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/ROCm-Developer-Tools/HIP/pull/110#issuecomment-317511624, or mute the threadhttps://github.com/notifications/unsubscribe-auth/ABAClz8oJq8kKQfDQz_JE6ZMbWeXLbuWks5sROFTgaJpZM4OhfBG.

hipEnvVar failed on nvidia machine.  You can look at the console output by clicking the red X above.
[nvcc] + make test
[nvcc] Running tests...
[nvcc] Test project /var/jenkins/workspace/-Developer-Tools_HIP_PR-110-DAATY7BOJMZLIC2LA357DNAY74R3BVG2A5MZS37A7WME75TYOZLA/build/release
[nvcc]       Start  1: directed_tests/hipEnvVarDriver.tst
[nvcc]  1/71 Test  #1: directed_tests/hipEnvVarDriver.tst ........................................***Failed  Required regular expression not found.Regex=[PASSED
[nvcc] ]  0.13 sec

Peng is this still ready to be accepted?

Hi Ben, still working on NV path fix, will push it soon.

@bensander , can you review?


Negative tag syntax  ~@search is not compatible with cucumber 4

sunspot_test/lib/sunspot_test/cucumber.rb is using a cucumber old style tag expressions. the ~ for negative tags was removed in the version 4 , so according to cucumber documentation, it should be not @search instead of  ~@search

@jonkerz  @manlycode First of all, I want to thank you for your valuable help in creating this PR#38   for this issue.
I see it was merged, so I tried to upgrade my gem to the newest version 0.4.2 but I am getting this:
Bundler attempted to update sunspot_test but its version stayed the same
And also this website https://rubygems.org/gems/sunspot_test shows only the version 0.4.1, Is the version 0.4.2 actually available?

@rvalladares77 Sorry for the delay. I bumped the 0.4.2 version on ruby gems.org.


Incremental updates

First of all, thank you for this project! This makes it amazingly easy to add charts. I’ve done this so far with JavaScript and it is really time consuming. This will be a huge time saver for me :-)
Describe the feature request
After the chart has been rendered the first time, I would like to be able to add data to the chart without redrawing the whole chart again. Click on Add Data on this page to see the expected behavior:
https://www.chartjs.org/samples/latest/charts/bar/vertical.html
Which charts does this feature request apply to?
All charts.
Describe the solution you'd like
If I'm working with a PieDataset for instance, it would be nice to just being able to add data to this object and then the system figures out how to dynamically. Maybe this is crazy hard to do, I don’t know. But I can see that the javascript library seems to be able to figure deltas in a smart way.
Describe alternatives you've considered
Another way could be to have some delta functions we could call instead, that then calls into the JavaScript library. So, when we want to do incremental updates, we use these functions instead.
Additional context
I can’t say this is an important feature for me, but I think it’s worth mentioning. Also, I think this is an interesting problem :-)
Thanks again for all this amazing work!

I totally agree with you!
The Javascript introp layer to Chartjs always bugged me so now I started writing it in Typescript (branch here: https://github.com/mariusmuntean/ChartJs.Blazor.git)
It turns out that Typescript is just awesome and I could improve the interop layer a lot. The incremental updates to a chart is one of those improvements.
Expect to see a new release, which addresses this issue, soon 😎

@mariusmuntean, nice work! It looks really promising .
By the way, I totally agree with you about TypeScript. It makes JavaScript descent :-) But when it comes to productivity nothing beats Blazor, it’s simply amazing :-)

@mariusmuntean Wow, you did get Typescript to work with Chart.Js. I already tried that before but failed :D
And this example looks really promising.

I think I should revive the WiKi as well. There I can describe how to get it going and what my experience was.
In short, you either use VisualStudio for the normal Blazor development and VSCode for the Typescript files OR you use Jetbrains Rider for both.

Ah, okay. I tried to integrate the TypeScript part into VS as well and it didn't work as expected.

I think this issue can be closed. Please have a look at the attached pre-release NuGet package below (after you download it, just remove the additional .zip extension - GitHub limitation :D) and let me know if this is what you imagined.
I also extended the PieChart sample to showcase the new behavior.
ChartJs.Blazor.1.0.2.nupkg.zip

There is a NuGet package that contains these changes: https://www.nuget.org/packages/ChartJs.Blazor/1.0.2


Removed unnecessary commands

Importing of the modules takes place automatically.
Module: 00
Lab/Demo: 00
Fixes # .
Changes proposed in this pull request:

From our testing, the commands as they appear are correct. The Import commands that you're suggesting are unnecessary must be run in order to import the module cmdlets into the current PowerShell session; otherwise, the subsequent commands in the later steps don't work. At least in our testing that's the case.


Preview shows wrong translation

On a project with multiple languages I've added a translation file for each language in following directory ClientWebsiteBundle/Resources/translations named messages.de.xliff and messages.en.xliff.
The translations are working in the frontend when I call them in a twig template with {{ translate_key|trans }}. Like documented here: http://symfony.com/doc/current/book/translation.html
The problem is, that the translations are wrong in the preview - it seems to display the translation of the default language.
Tested in multiple project - this bug seems to exists since RC1 or earlier.

Seems like they've changed something: https://symfony.com/doc/current/book/translation.html
Setting the locale using $request->setLocale() in the controller is too late to affect the translator. Either set the locale via a listener (like above), the URL (see next) or call setLocale() directly on the translator service.

That's the way it was handled until yet. I've tested it and the locale should be set directly on the translator service for the preview now:
$this->get('translator')->setLocale($structure->getLanguageCode());


[BUG] failed to patch CoreDNS ConfigMap on 5.0.1

What did you do

How was the cluster created?

 k3d cluster create --k3s-arg "--snapshotter=native"@ -v /dev/mapper:/dev/mapper --k3s-arg "--kubelet-arg=feature-gates=SupportPodPidsLimit=true"@ --verbose

What did you expect to happen
Cluster to start correctly
Screenshots or terminal output
log.txt
Which OS & Architecture

Synology NAS on amd64 with btrfs volume

Which version of k3d

k3d version v5.0.1 k3s version v1.21.5-k3s2 (default)

Which version of docker

Client:
Context:    default
Debug Mode: false
Server:
Containers: 0
Running: 0
Paused: 0
Stopped: 0
Images: 3
Server Version: 20.10.3
Storage Driver: btrfs
Build Version: Btrfs v4.0
Library Version: 101
Logging Driver: db
Cgroup Driver: cgroupfs
Cgroup Version: 1
Plugins:
Volume: local
Network: bridge host ipvlan macvlan null overlay
Log: awslogs db fluentd gcplogs gelf journald json-file local logentries splunk syslog
Swarm: inactive
Runtimes: io.containerd.runc.v2 io.containerd.runtime.v1.linux runc
Default Runtime: runc
Init Binary: docker-init
containerd version: ea3508454ff2268c32720eb4d2fc9816d6f75f88
runc version: 31cc25f16f5eba4d0f53e35374532873744f4b31
init version: ed96d00 (expected: de40ad0)
Security Options:
apparmor
Kernel Version: 4.4.180+
Operating System:  (containerized)
OSType: linux
Architecture: x86_64
CPUs: 4
Total Memory: 7.61GiB
Name: synology
ID: IMG7:Y2IH:AXAH:FDAW:5QBK:ZIZC:2YLX:T3LC:JH2L:4TSW:Z4SH:PUOA
Docker Root Dir: /volume1/@docker
Debug Mode: false
Registry: https://index.docker.io/v1/
Labels:
Experimental: false
Insecure Registries:
<IP_ADDRESS>/8
Live Restore Enabled: false
WARNING: No kernel memory TCP limit support
WARNING: No cpu cfs quota support
WARNING: No cpu cfs period support
WARNING: No blkio weight support
WARNING: No blkio weight_device support
WARNING: No blkio throttle.read_bps_device support
WARNING: No blkio throttle.write_bps_device support
WARNING: No blkio throttle.read_iops_device support
WARNING: No blkio throttle.write_iops_device support

Closing this due to inactivity and having a fix provided. Feel free to reopen if you're still experiencing this issue!


WIP [BEAM-6850] Use HadoopFormatIOIT for performance/integration testing

Use HadoopFormatIOIT for performance/integration testing instead of deprecated HadoopInputFormatIOIT.

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[x] Choose reviewer(s) and mention them in a comment (R: @username).
[x] Format the pull request title like [BEAM-XXX] Fixes bug in ApproximateQuantiles, where you replace BEAM-XXX with the appropriate JIRA issue, if applicable. This will automatically link the pull request to the issue.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

Run Java HadoopFormatIO Performance Test

I'm on vacation until 5th may. Please reassign reviewer. Thanks :)

@melap Could you take a look? I'm also wondering how to make new command Run Java HadoopFormatIO Performance Test working and run Jenkins job

@kkucharc @mwalenia Could you take a look? I'm also wondering how to make new command Run Java HadoopFormatIO Performance Test working and run corresponding Jenkins job

@chamikaramj can you please take a look at this one, it seems the polish crew are in holidays this week.

Run Seed Job

Run Java HadoopFormatIO Performance Test

LGTM. Thanks.

@aromanenko-dev could you add metric gathering to HadoopFormatIOIT? It's included in HadoopInputFormatIOIT. Lack of metrics will disable reporting on dashboards, it would be good to keep the functionality intact.

Run Java HadoopFormatIO Performance Test

@mwalenia Yes, I copied these metric gathering into HadoopFormatIOIT  as well, thanks

Run Seed Job

Run Java HadoopFormatIO Performance Test

Cool - I updated the metrics plot for that: https://s.apache.org/io-test-dashboards (at the bottom). Thanks for doing this.


x/build/devapp: automate burndown chart to track current major Go release

The dev.golang.org/release page shows an issue burndown chart, tracking the current major Go release milestone. This chart needs to be updated to target Go 1.(N+1) milestone every 6 months (see previously CL 248157, CL 295191, etc.). We can automate this so that regular manual updates aren't needed, similarly to how we've done it for other pages like build.golang.org.

This was resolved via CL 344089.


Add support for system control sockets for XNU

I added support for macOS and iOS system sockets, which can be used to control the kernel as described here.
To do this, I had to add in support for ioctl on those platforms, so I added in ioctl support for all BSD-based platforms. The API seems to be the same between xnu, FreeBSD, NetBSD, OpenBSD and Dragonfly BSD.
I added a test that runs on macOS and iOS for the functionality. Let me know if I need to make any changes!

Do you think you'll be able to review this, @asomers?

Made those changes! Let me know if you have further feedback :)

Could you rebase to master and push again (with -f), so that the CI is green? Could you also add an entry to our CHANGELOG.md to document the changes.
After that we can merge it.

Rebased against master. I documented the additions, and the one existing API change!

@homu r+
thanks

:pushpin: Commit ef257e0 has been approved by fiveop

:hourglass: Testing commit ef257e0 with merge 9b81000...

:sunny: Test successful - status


Make IBI for bonded interactions a post update method

See https://github.com/votca/csg/pull/675/files#r584377576

@marvinbernhardt can you take a first crack at that?

Sure can do. Probably not soon, though.

ping @marvinbernhardt


2 Words User Fields

Are there any issues with synchronising 2 words user fields - for example: "Information Technology" for Claim name - user_field_department (in ZD) -> value: user.department (Az) ? It seems that spaces are treated like breaks for the import. The field does not get populated. Thank you!

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

ID: fe2af6f9-785f-8bcc-0afb-3e7398b1eb56
Version Independent ID: e045f8bd-0ba2-27dd-1a23-3383b2463adf
Content: Tutorial: Configure Zendesk for automatic user provisioning with Azure Active Directory
Content Source: articles/active-directory/saas-apps/zendesk-provisioning-tutorial.md
Service: active-directory
Sub-service: saas-app-tutorial
GitHub Login: @zhchia
Microsoft Alias: jeedes

@apopadk Thanks for the feedback !  I have assigned this issue to content author to investigate and update the document as appropriate.

@zchia Can you please help answer this?

@jeevansd @SaurabhSharma-MSFT @twimmers
We couldn't see any issue while importing and exporting department with 2-word user field. Screenshots below

#sign-off

#please-close


[api-docs] Mark Guild Member.avatar as optional.

A new commit was made into the api-docs repo. This issue was created automatically. Need to review if we made any change to the library for this commit.
https://github.com/discord/discord-api-docs/commit/80c975204dea9751589d25215112c65d372e21d4

Done


Add push artifact work around.

The workaround identifies artifacts in workspace and pushes them to s3 storage.
(cherry picked from commit 08eb98dbb3f6b47740608e050ad40bc93779f740, 8d51bf6)

/hold

/lgtm

/approve

/unhold


To which extent do we have control over network requests and cookies of WebViews and Progressive Web Apps?

We are following the approach of having different opt out mechanisms for different types of apps.
1. Non-browser apps
For non-browser apps (at least to the extent they are not using cookies), opting out via the AdID should be sufficient. Thus, simply surfacing the OS setting in our app to the user may be sufficient as well.
2. Browser apps (and WebViews)
For browser apps (and WebViews), to which extent can we access cookies and modify web requests? Is our problem easier now since we do not have to intercept all web traffic but only that of a browser?
If it does not get easier, are there browser-level controls that would prevent tracking? So, for example, a simple solution could be that our app simply surfaces a browser-level settings page for turning of third party cookies.

I believe that a PWA works pretty much like any website, in that to collect cookies in a PWA, you can use JavaScript's document.cookie property, as mentioned in a previous response. However, the cookies you can collect are limited to those created by the web application's domain itself. You cannot access cookies created by other domains or websites. I actually tried making my own Webview on Android Studio (for PWAs, they work quite similarly to websites.

PWAs can access cookies created by their own domain using JavaScript's document.cookie property. However, they cannot access cookies created by other domains or websites. In this context, PWAs have the same access to cookies as any other web application running in a web browser.
As such, the AdID (also known as Advertising ID or AAID) is a unique identifier used for advertising purposes on Android devices. AdID is not directly accessible through a PWA running in a web browser context, as it requires native Android code and the Google Play Services Ads SDK.I  read somewhere about a possible way to  "bridge" the PWA to the Android-native features, through something like this:

`public class AdIdInterface {
Context mContext;
AdIdInterface(Context c) {
    mContext = c;
}

@JavascriptInterface
public String getAdId() {
    // Fetch the AdID and return it
}

}
WebView webView = findViewById(R.id.webView);
webView.addJavascriptInterface(new AdIdInterface(this), "AndroidAdId");
`
but I don't think its effective.
So, as far as I know PWA has access to cookies (similar to websites), at least until further development (e.g. Chrome getting rid of cookies in 2024)

For WebView, as previously covered, it can access both cookies and AdID. For AdID, as the WebView is typically connected to an app, it can gain access to the AdID through the getAdId() function.
For cookies, WebView can access cookies just like a regular web browser. They can access cookies created by their own domain, but not cookies created by other domains or websites. You can read, create, and modify cookies using JavaScript's document.cookie property within the web content loaded in the WebView.
For AdID, WebView, by default, does not have direct access to native features like AdID. However, you can access the AdID in the native portion of your app and expose it to the WebView using a JavaScript interface.
`function getAdId() {
if (window.AndroidAdId && typeof window.AndroidAdId.getAdId === 'function') {
const adId = window.AndroidAdId.getAdId();
console.log('AdID:', adId);
// Use the AdID for your purposes
}
}
getAdId();
`

I believe Custom Tabs don't work differently vs surfing the browser itself. Think on the whole, Chrome Custom Tabs runs with Chrome multi-process architecture (https://www.chromium.org/developers/design-documents/multi-process-architecture). Chrome processes are separate from the app processes. Hence, there is much stronger protection from 3rd party content with Chrome Custom Tabs than with WebView, app security is improved. Also, yes, WebViews can "communicate" with other apps given that it can use intents. While WebView cannot use intents directly, you can combine WebView with intents in your Android application to achieve the desired functionality. For example, if you want to open a link within your WebView in an external browser or another app, you can intercept the URL loading request, create an intent, and use it to launch the appropriate app.
webView.setWebViewClient(new WebViewClient() {     @Override     public boolean shouldOverrideUrlLoading(WebView view, WebResourceRequest request) {         String url = request.getUrl().toString();         if (url.startsWith("myapp://")) {             Intent intent = new Intent(Intent.ACTION_VIEW, Uri.parse(url));             startActivity(intent);             return true;         } else {             return super.shouldOverrideUrlLoading(view, request);         }     } });
I will continue playing around with WebViews and inform about the result.

Temporarily halt conversation on webview, but keep it at the back of our minds.


Dev workflow: patch number increments

npm version patch number should only be bumped for landed PRs. Can we switch to doing this in a GitHub action, or something like that?

Removed automatic patch incrementing locally. Workflow TBD (pushing to 0.x)

Trimming this - we don't need it


fix(goreleaser): add docker image to releaser

@dkropachev this repo has permission to push the images ? if not can we make it push them ?

when you run releaser, it picks up env credentials to do things, including push images

when you run releaser, it picks up env credentials to do things, including push images

so what's missing so it would be able to push into docker hub on release ?

Nothing, anyone who runs goreleaser should have creds from github.
We can have separate step in github actions to trigger it and have credentials there.

when you run releaser, it picks up env credentials to do things, including push images

so what's missing so it would be able to push into docker hub on release ?

Nothing, anyone who runs goreleaser should have creds from github. We can have separate step in github actions to trigger it and have credentials there.

github and dockerhub creds are not the same.
I would rather have this happen in a github action when someone tag a release, and not on someones own laptop.


Provide way to exclude CoreML delegate from bazel build

Please make sure that this is a feature request. As per our GitHub Policy, we only address code/doc bugs, performance issues, feature requests and build/installation issues on GitHub. tag:feature_template
System information

TensorFlow version (you are using): master
Are you willing to contribute it (Yes/No): Yes

Describe the feature and the current behavior/state.
Currently when you build for iOS (specifically Swift in our case) you end up with the coreml_delegate target whether or not you consume it. As described in this comment, this may be something we want to support optionally excluding:
https://github.com/tensorflow/tensorflow/blob/d53c999feb2751b6a3ec9bace5be6f0b4620e4b4/tensorflow/lite/experimental/swift/BUILD.apple#L13-L24
Will this change the current api? How?
I think we could maintain the current behavior, and have a config_setting that could be used with a --define such as --define=EXCLUDE_COREML=1 that we could select on to opt these out. This is done extensively in the envoy build system for similar reasons.
Who will benefit with this feature?
Anyone who doesn't need the CoreML specific logic and wants to potentially reduce their app size (in some cases this may have very little app size effect because of dead stripping)
Any Other info.
More discussion on https://github.com/tensorflow/tensorflow/pull/38956

Thank you for opening the issue! I think using config_setting would be a reasonable approach to take. A few things to note that could make things a bit more complex:

CoreML and Metal are the two delegates that are currently offered, but there may be more in the future.
Some developers may want no delegation deps, others may want 1 or more.
If adding configs to exclude deps, then would need to consider that developers would need to update the exclusions each time a new delegate is added. Perhaps "opting-in" might be a simpler approach, i.e. devs explicitly list the delegates they want to include?

@teijeong what are your thoughts?

Thanks for tracking this! As delegates will add up significant sizes, yes that's what we really want. I really need inputs from someone who has more experience on these kind of issues, just like you two :)
Besides modifying the BUILD rule, I don't know what's the good approach on this. Config looks straightforward, but I don't know how flags should be configured in XCode, when using bazel to build the app.
When I tested, including CoreMLDelegate.swift file (whether or not the app is actually using it) increased the app's size, so this is definitely needed.

I've submitted a PoC / RFC of opting in to these https://github.com/tensorflow/tensorflow/pull/39765
I've tested it in all configurations and it seems to work right now. We probably want to update CI to exercise them all


Adding ListVPCAssociationAuthorizations pagination (#1416)

Adding paginator support for list_vpc_association_authorizations in #1416
>>> import boto3
>>> import botocore
>>>
>>> client = boto3.client('route53')
>>>
>>> client.can_paginate('list_vpc_association_authorizations')
True
>>> client.get_paginator('list_vpc_association_authorizations')
<botocore.client.Route53.Paginator.ListVPCAssociationAuthorizations object at 0x1022d8750>

Codecov Report

Merging #1417 into develop will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff            @@
##           develop    #1417   +/-   ##
========================================
  Coverage    80.71%   80.71%
========================================
  Files           87       87
  Lines        12131    12131
========================================
  Hits          9792     9792
  Misses        2339     2339

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 cb78ecf...d09a8cd. Read the comment docs.


Switch output file creation from finest granularity only to all time options specified

Currently the code produces output files for only the finest granularity (i.e. the day option overrides the year option). Since we have an output volume estimator, it should be safe (and more logical) to output all timesteps the user specifies.
Make sure to consider/address issue #336 while doing this.

Another option for this would be a post-processing script that can generate the summary outputs from the fine granularity outputs.


doesn't work with babel-core 5.7

Hey, I just started using this and it's great for integration testing components.
Everything worked fine, after I started using it, but then I removed/reinstalled node_modules and bower_components and I got a build error which ended in somefile: TODO which I suspect is this line in babel-core: https://github.com/babel/babel/blob/master/packages/babel/src/traversal/path/replacement.js#L67.
When I replaced the babel-core dependency in broccoli-babel-transpiler to 5.6.19 it worked again (I replaced it by removing the directory and replacing it with a local copy of that babel-core version).
Before replacing it the babel-core version was 5.7.0 which didn't work.

Thanks for reporting! It looks like the line you suspected is the problem. I will investigate on how to resolve.
Maybe @sebmck can chime in and comment if there is a planned ETA when replaceWithSourceString (or a replacement) will land again ... :innocent:

@pangratz Shit, sorry. I was meant to add that back in but forgot.

@pangratz Shit, sorry. I was meant to add that back in but forgot.

Uff, a reply 38 seconds later. :zap: fast sebmck is fast :stuck_out_tongue_winking_eye:

@sebmck Would you accept a PR removing that line?

Would you accept a PR removing that line?

I was supposed to add it back. Give me a minute, pushing a new version.

Fixed as of 5.7.1, sorry!

Fixed as of 5.7.1, sorry!

You da man! Thanks for the megafast fix! :heart:

Thanks to both of you for fixing this so quickly.

this was fast, nice job thanks :+1:


We need a performance suite

Inspired by @jdalton's talk at JSConf, and all the goings-on around our nextTick implementation, I think we need a performance suite that produces numbers we can compare from commit to commit, or release to release.
There's been some work done in this area already for generic promise performance tests:

https://github.com/cujojs/promise-perf-tests
https://github.com/thanpolas/perf-promises

But I think we need something rather Q specific, with real-world-ish scenarios. We can add to it as we go along.
This would be nice because there's lots of minor inefficiencies in how we do things, but without a way of quantifying their impact it's not worth messing with the code.

Hi, I was wondering what the status is of this? We were debating what promise library to use at Spotify, and this one came up https://github.com/petkaantonov/bluebird/tree/master/benchmark, which sort of made bluebird "win".

@mpj I believe that @stefanpenner also has a benchmark he uses to tune RSVP. By that benchmark, last I checked, when.js was really good, but most of the libraries designed for speed did well. Q is designed at a higher level of abstraction and does more to protect private state than the rest of the pack, which comes at a significant performance cost. I have been re-evaluating that trade-off in the experimental v2 branch, but if speed is your primary concern, you’re willing to opt-out of the Q ecosystem, and you have no desire to use promises as proxies for remote objects, you should for sure go with one of the others.
I have used @stefanpenner’s benchmark to evaluate Q.
https://github.com/stefanpenner/promise_benchmarks


Stop the on-page search from stretching

The stretching movement of the search bar on focus causes problems when changing screen size.
Whilst a stretching bar looks nice when looking at the site for a couple of times, it becomes annoying when using the site a lot.
I wonder whether it would be better just to set its size.

@finanalyst I can address this in the changes fixes to the existing bar problems if you'd like. The bar is kind of a mess, I am having to refactor a lot to make small style changes.

@rawleyfowler please look at this.
There are quite a few problems with the search bar.
I will not open a separate issue, but it is related to this one, is that there is a faint grey line above the search bar. The line is coming from the auto-complete div that is made visible when search produces results. In addition, the div seems to be slightly misplaced upwards.
I will look at issue #47 which will add info to the items object that the autocomplete handles

I can't get a screenshot of the problem because it disappears when focus shifts to screenshot app

@finanalyst I am working on a comprehensive patch for most of the searchbar issues reported. When I PR i'll mention the issues that it solves.

@finanalyst I just PR'd for most of the visual problems. I'll look at #47 this weekend.

closing this as completed


improve coding style

improve coding style
fix hoisting
fix missing semicolons
add .editorconfig
make script smaller: from 23700 bytesto 23519 bytes minified
improve readability
add Unix NewLine at end of file

I'm closing this issue because this project is no longer being maintained


composer 2.0 deprecation warning due to uppercase character

When running composer commands I get the following deprecation warning on composer 1.8.2.

Deprecation warning: require-dev.mikey179/vfsStream is invalid, it should not contain uppercase characters. Please use mikey179/vfsstream instead. Make sure you fix this as Composer 2.0 will error.

You can simply lowercase the package name in your composer.json - the lookup is case-insensitive anyway. Of course it should also be changed here.

A pr was opened https://github.com/mikey179/vfsStream/pull/174 for this.

I just released 1.6.6 with the fix from #174.

What am I doing wrong?
$ composer remove mikey179/vfsstream
# OK

$ composer require --dev mikey179/vfsstream
Using version ^1.6 for mikey179/vfsStream
./composer.json has been updated
Deprecation warning: require-dev.mikey179/vfsStream is invalid, it should not contain uppercase characters. Please use mikey179/vfsstream instead. Make sure you fix this as Composer 2.0 will error.

Why is it still adding vsfStream with upper case? This is in my composer.lock after the above:
{
    "name": "mikey179/vfsStream",
    "version": "v1.6.6",
    "source": {
        "type": "git",
        "url": "https://github.com/bovigo/vfsStream.git",
        "reference": "095238a0711c974ae5b4ebf4c4534a23f3f6c99d"
    },
    "dist": {
        "type": "zip",
        "url": "https://api.github.com/repos/bovigo/vfsStream/zipball/095238a0711c974ae5b4ebf4c4534a23f3f6c99d",
        "reference": "095238a0711c974ae5b4ebf4c4534a23f3f6c99d",
        "shasum": ""
    },
    "// etc."
}

The workaround was to update composer.json manually && run composer update nothing to update the content-hash field in composer.lock.
Still, I don't quite understand why vfsStream still installed with the upper case letter at version 1.6.6.

I'm not sure if this is related but searching for vfsstream on Packagist returns duplicate results with different cases. I've opened https://github.com/composer/packagist/issues/1012 to see if that's intended behavior but that might be the cause?


updated readme, added setup.cfg

Coverage increased (+0.03%) to 95.116% when pulling bae99b85c0373d351c7ed4f6f8960f0154dac79c on updated_readme into 146da2d20913c8c8576bbfc1cd8c32ee845dc9d4 on master.


Update gui.rst

Someone asked about batch training, so I thought something could be written about it. Feel free to amend to your liking.

Got rid of the advice to use silent mode. As the maintainer, I don't feel comfortable telling people to ignore the warnings I took the time to write.
Not against an Issue to collect the errors during batch training and report them all out at the end though 🙂


Update front matters for ug

Codecov Report

Merging #309 (90ea0de) into master (cb143c7) will increase coverage by 14.30%.
The diff coverage is 77.77%.

@@              Coverage Diff              @@
##             master     #309       +/-   ##
=============================================
+ Coverage     59.96%   74.27%   +14.30%
- Complexity      858     1136      +278
=============================================
  Files           163      164        +1
  Lines          3255     3292       +37
  Branches        438      459       +21
=============================================
+ Hits           1952     2445      +493
+ Misses         1171      725      -446
+ Partials        132      122       -10

Impacted Files
Coverage Δ

...rc/main/java/seedu/address/logic/LogicManager.java
77.27% <ø> (+46.83%)
:arrow_up:

.../seedu/address/logic/commands/EditExamCommand.java
95.00% <0.00%> (-3.28%)
:arrow_down:

...u/address/logic/parser/DeleteTagCommandParser.java
85.71% <ø> (ø)

...edu/address/logic/parser/EditTagCommandParser.java
95.00% <ø> (ø)

...address/logic/parser/FilterTasksCommandParser.java
77.77% <ø> (+77.77%)
:arrow_up:

...in/java/seedu/address/logic/parser/ParserUtil.java
98.14% <ø> (ø)

src/main/java/seedu/address/model/AddressBook.java
90.16% <ø> (+31.96%)
:arrow_up:

src/main/java/seedu/address/model/Model.java
100.00% <ø> (ø)

...rc/main/java/seedu/address/model/ModelManager.java
88.49% <ø> (+25.99%)
:arrow_up:

...in/java/seedu/address/model/module/ModuleCode.java
90.00% <ø> (+10.00%)
:arrow_up:

... and 55 more

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


Make the cloud-agent-openapi-spec.yaml file less mutable

Proposed feature
The example file have time fields that uses the current timestamp.
cloud-agent/service/api/http/cloud-agent-openapi-spec.yaml
This means each time is generated that examples change (the file changed).
Giving conflict with main branch all the time (and stop all CI test pipeline)###
Feature description
Make the timestamp of the examples constant
Anything else?
No response

Also in one of examples is time zone dependent


Need to reload nerdtree

When I update a file, its shows the correct symbol in nerdtree. But it shows with a wrong indentation and after a while, it corrects the indentation. How I can make to fix the indentation faster?

did you install indentLine?

nop

please show me you vimrc

Sure here is: https://pastebin.com/QBNV3aLA

You add plugins in wrong order, please try this
Plug 'scrooloose/nerdtree' |
            \ Plug 'Xuyuanp/nerdtree-git-plugin' |
            \ Plug 'ryanoasis/vim-devicons' |
            \ Plug 'tiagofumo/vim-nerdtree-syntax-highlight'

Thanks, it works :smile:, thanks for the help.


UI for selecting runtime versions

From the mailing list thread "Support Product and Community Runtimes"
To be able to select runtime versions (product and community for now, but could be more) we need some adjustment to the UI.
For a first version I'd go with the simplest solution that works and talk about proper interaction design later after everything works.
See also openshiftio/launchpad-addon#34 and openshiftio/launchpad-backend#58

Parent tracking issue: openshiftio/appdev-planning#41


High CPU usage

I've heard the gpu is better for mining bitcoins but still...
dfc_windows-x64_0_10_6-RC4
downlords-faf-client.log

What does the Ram usage say?

I'll check that out next time it happens.

Ring me up next time and we can profile the application

After few hours of running it's spiking to 100% on a single core every few seconds

1 gigabyte of RAM usage sounds like memory leak

So just sitting in a chat doesn't cause this. But going to the vault and searching for a replay. Then it starts happening

Could be GC being pressured because of https://github.com/FAForever/downlords-faf-client/issues/1104

@speed2CZ Can you try the new RC https://github.com/FAForever/downlords-faf-client/releases/tag/v1.0.0-RC11 and check if this has become better?

it's still spiking to 100% on one core

version + log

+heap dump if newest version


Modifier “*Max:?” problem

When a modifier with “*Max:?” (e.g.  -4 : to hit (Charge Attack) *Max:9) is combined with another modifier, the skill cap (e.g. 9) isn't applied anymore.

Yeah, the bug occurs because the Max: is the first modifier.   It will work if it is the last.   But it should work in any order.    It is fixed and will be in the next release.


Fix publishing playground

No changes needing a change description found.

You can try these changes at https://cadlplayground.z22.web.core.windows.net/prs/3319/
Check the website changes at https://tspwebsitepr.z22.web.core.windows.net/prs/3319/


Error Load

VSCode Version: Code 1.7.2 (7ba55c5860b152d999dda59393ca3ebeb1b5c85f, 2016-11-21T22:06:37.428Z)
OS Version: Windows_NT ia32 6.1.7601
Extensions:

Extension
Author
Version

javadebugger
donjayamanne
0.0.2

php-debug
felixfbecker
1.10.0

crane
HvyIndustries
0.2.2

format-indent
Kasik96
1.3.0

csharp
ms-vscode
1.6.2

Does not open the file when double-clicking, only opens the editor.

@marroq can you please provide steps how to reproduce. Which file doesn't open. Can you attach a screen cast showing the problem.

I do double click in the file but visual studio code can't open the file, only opens the editor

Not sure if the is an installer problem or a workbench problem.

I think we fixed this in 1.9.


[Bug]: Cookies are not sent when using fetch

Minimal, reproducible example
async function browse (url) {
    try {
        new URL(url);
    }
    catch(e){
        console.log("Invalid URL");
        return;
    }

    const browser = await puppeteer.launch({args: ["--no-sandbox"]});
    const page = await browser.newPage();

    const cookie = {name: "session", value: "cookie_value_here", domain: "my-domain.com", sameSite: "none" , secure: true };

    await page.setCookie(cookie);

    await page.goto(url, {
        waitUntil: ["networkidle0", "domcontentloaded"],
      });

    res = await page.content();
    console.log(res);
    await browser.close();
}

browse("https://some_url.com");
// If "https://some_url.com" includes some JavaScript which "fetches" data from "my-domain.com", the "fetch" is executed but will not include the cookie, even though it has been set with the "sameSite=none" and "secure=true" flags.

Error string
no error
Bug behavior

[ ] Flaky
[ ] PDF

Background
When I goto to a web page (domain A) which includes a JavaScript fetch call towards a domain B (configured with credentials: "include"), the fetch request is executed but cookies are not sent (even though I've added the cookie using page.setCookie(cookie) and the cookie is set with sameSite=none and secure=true).
It seems that cookies are only sent when the goto function points to a domain for which I have set the cookies, but a "real" browser would also send the cookies with a fetch request (according to the same-site/secure cookie flags).
Expectation
I'd expect to have cookies sent when browsing a domain A which includes some JavaScript that fetches content from domain B if the cookie has been properly set previously by using setCookie and if the sameSite cookie flag allows us to actually send the cookie with the fetch request (just like a regular browser would do).
Reality
Cookies is not sent as expected
Puppeteer configuration file (if used)
No response
Puppeteer version
21.1.1
Node version
v16.20.2
Package manager
npm
Package manager version
8.19.4
Operating system
Linux

Never mind, my fault.
It looks like SameSite values are case sensitive as per specs here.
I've just changed the sameSite flag from none to None and it worked as expected.


Subscriptions never connect to Amplify mock server

Describe the bug
I feel like I've tried absolutely everything including going deep into the implementation of the subscription implementation
To Reproduce
Steps to reproduce the behavior:

Create an Amplify project with amplify add api and add model and a subscription query
Run amplify mock to get a local instance of the amplify running

✔ Generated GraphQL operations successfully and saved at graphql
✔ Code generated successfully and saved in file API.swift
AppSync Mock endpoint is running at http://<IP_ADDRESS>:20002

Create a client provider according to the example in the repo
https://github.com/zino-app/graphql-flutter/blob/ba98de9216b36656d9acb5d69e728199fef7fd9f/examples/starwars/lib/client_provider.dart
Configure ClientProvider as follows

ClientProvider(
    uri: "http://localhost:20002/graphql",
    subscriptionUri: "ws://localhost:20002/graphql",
    ...
 )

Use Subscription widget configured with correctly with the subscription query created in 1.
Observe the following error

flutter: Connecting to websocket: ws://localhost:20002/graphql...
flutter: There was an error causing connection lost: WebSocketException: Connection to 'http://localhost:20002/graphql#' was not upgraded to websocket
flutter: Disconnected from websocket.
flutter: Scheduling to connect in 5 seconds...

Expected behavior
Should connect properly and establish a connection
Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):

OS: all
Version all

Smartphone (please complete the following information):

Device: [e.g. iPhone6]
OS: [e.g. iOS8.1]
Version [e.g. 22]

Additional context
Add any other context about the problem here.
I'm tempted to think that the subscription websocket address I am using is incorrect. When I connect through GraphiQL it shows the following message on amplify mock logs when I execute a subscription operation:
register {
  clientId: 'e6fdf792c0dd99ca7c3f1764b9937cd6',
  subscriptionId: 'e6fdf792c0dd99ca7c3f1764b9937cd6/onCreateMessage/'
}

I feel like I've gone over everything with a fine toothed comb by this point and desperately need help to get unblocked. Thank you for any info that anyone can provide

@KieranLafferty what happened to this in the end? did you get it working with Amplify and mock Amplify?

@KieranLafferty what happened to this in the end? did you get it working with Amplify and mock Amplify?

I am having this issue with GCP cloud run.


Add azure_sql_server_detail. Closes #13

Checklist

[ ] Issue(s) linked

Overall we can change below

[ ] Add Auditing > Enabled/Disabled
[ ] Vulnerability Assessment  > Enabled/Disabled
[ ] Azure AD Authentication > Enabled/Disabled
[ ] Remove Kind card
[ ] Public Network Access > Public Access (Change label)

Add below

[ ] "Private Endpoint Details"
[ ] Remove the Admin login and password details panel


First Draft of Instructions for Saving Endangered Data

Please give feedback.
This is time sensitive -- I want to publish in multiple venues before the "endangered data" hackathons that are starting around North America later this week.
cc @jbenet

"'endangered data' hackathons"? Tell us more.
–♧W
On Mon, Dec 12, 2016 at 2:47 AM, Matt Zumwalt<EMAIL_ADDRESS>wrote:

Please give feedback.
This is time sensitive -- I want to publish in multiple venues before the

"endangered data" hackathons that are starting around North America later
this week.
cc @jbenet https://github.com/jbenet
You can view, comment on, or merge this pull request online at:
https://github.com/ipfs/community/pull/216
Commit Summary

Update precarious-data.md
Update precarious-data.md
Update precarious-data.md
Update precarious-data.md
Update precarious-data.md
First Draft: Instructions for Saving Endangered Data

File Changes

M articles/precarious-data.md
https://github.com/ipfs/community/pull/216/files#diff-0 (204)

Patch Links:

https://github.com/ipfs/community/pull/216.patch
https://github.com/ipfs/community/pull/216.diff

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/ipfs/community/pull/216, or mute the thread
https://github.com/notifications/unsubscribe-auth/AALFEwXuaRAgAz9RZG89y1GUWS3pPQlhks5rHPwHgaJpZM4LKQ_m
.

@dysbulic as the article explains in the opening paragraphs, people have been talking about how to "save" the data that they rely on. Some of them use the term "Guerrilla Archiving". We want them to be well informed, and want them to understand why content-addressing and IPFS is the right tool for the job.

Finished post is on Medium here<EMAIL_ADDRESS>


Transparent Release CLI producing different builds than manual Oak builds

Expected Behavior
Running the transparent release CLI on the testdata in https://github.com/project-oak/transparent-release/blob/main/testdata/build.toml yields the same binary hash as building the binary using Oak's ./scripts/docker_run command with the same arguments.
Actual Behavior
Runnning the testdata yields the binary hash of 15dc16c42a4ac9ed77f337a4a3065a63e444c29c18c8cf69d6a6b4ae678dca5c , as expected in the testdata.
Running the same build using ./scripts/docker_run yields a binary with the hash of e4c5930c6c0e81f8d743da1597905df0fa66b0daf69c6bf92a5cbc561c79b127.
Steps to Reproduce the Problem
For testdata: see the expected hash in the testdata, or run the testdata as outlined in this repos readme.
For running the same build with Oak's ./scripts/docker_run:

Clone a fresh copy of the repo
Checkout the commit from testdata
Run ./scripts/docker_run with the arguments provided in testdata's command key.

For connivence, that's this command in a fresh clone: git checkout 0f2189703c57845e09d8ab89164a4041c0af0a62 && ./scripts/docker_run ./scripts/runner build-functions-server && sha256sum ./oak_functions/loader/bin/oak_functions_loader.
Why this is happening
Further investigation is needed, but I'm fairly certain it boils down the fact that this CLI runs docker with different ENV variables and config than ./scripts/docker_run.

Use the CLI for Oak builds in general, or use it when generating provenances? Is it acceptable for provenances capture a different build than the canonical docker build config?

Another thing to check is the user running the process. I remember @ipetr0v investigated similar issues in the past where running as root or non-root would affect the final binary (though I also think @jul-sh checked that's not the case any more at some point?)

One thing to try is also to remove all the env variables from docker_run and see if it makes a difference (both in general (do random env variables affect the output, even if unrelated to the build?), and in particular for our build process)

Good pointers, confident we can trace it down more by playing with the flags, which I haven't done so far. :) — I guess either way though there's a question of this should be addressed architecturally. I worry we'd align docker_run and the CLI for now, but it seems they'd diverge again later

I think now that we have ./scripts/fix_docker_user_and_run the command has to be something like ["./scripts/fix_docker_user_and_run" "'./scripts/runner build-functions-server'"]. But that would also require passing the environment variables, and mounting the volumes (because the ./scripts/fix_docker_user_and_run requires them).
These are not very generic, and I think they should be passed as additional docker run flags.

How do people think we should best tackle this?
A) Find a way to set the docker username in our provenance without passing host uid/guid? (Permissions give me headaches 😅)
B) Surrender and pass host uid/guid, exactly replicating the docker_run script
C) Debug Oak functions to try and fix hashes depending on the username
cc @rbehjati as the solution to this sort of depends on the wider interface of our CLI :)

I thought the conclusion is to just set the username to docker, regardless of UID / GID? It may well be that the simplest way of doing that is to use fix_docker_user_and_run, if it works I think that's fine.

Thanks for looking into this @jul-sh :)
I agree with your first comment about solving the problem architecturally. I think the right design, that is flexible and easy to maintain, is the one that makes the fewest assumptions about the details of the build configuration and setup. Based on that I would say that the buildConfig in the provenance should give a command that we run directly on the host machine instead of a command that we'd pass to docker run to execute. The command could be a script that itself runs docker run.
I think this is similar to the idea of a meta-image that @tiziano88 previously mentioned in a few occasions.
This design raises a number of questions. In particular,

How do we list all the required materials?
How can we ensure that the command actually uses the listed materials for building the binary, and nothing else?

If we have a trusted builder that automatically generates the provenances including a list of the materials, and signs it, then the second question is probably not relevant. The builder could be configured to include only the most important materials, for instance only the source code and the builder image as we currently have. We could even use the SLSA provenance metadata field to indicate that the list of materials is incomplete, or we could employ the idea of a release branch with venodred dependencies to avoid having to enumerate those dependencies in the list of materials.
If we want to have provenances that can be verified automatically, then we have to find an answer to the second question.
I think making provenances verifiable is a valuable goal to pursue. I don't know how exactly this can be engineered. But we could still require a source Git repository, and a builder image in the list of materials, and require the repository to contain a build script to use as the command in the buildConfig. We could perhaps use the idea of a meta-bulider docker image to somehow make sure that the build script (or generally the command) does not fetch anything other than the materials listed in the provenance. I don't know how exactly this can be done, or if it is feasible. But I think it is worth investigating.
In the meantime, I think it is a good idea to proceed and fix the problem with UID/GID, if it does not require too much effort.

Solutions could be:

Add a command to make that user root (Should that command be done in the provenance, hardcoded as a step in our go code, as a script in Oak?)
Add a command that chowns the workspace dir to be accesible to the docker user (Should that command be done in the provenance, hardcoded as a step in our go code, as a script in Oak?)

I'd say preferably include it in the script in Oak, if that is not possible hardcode it in the go code.

The easiest solution for now would be https://github.com/project-oak/oak/pull/2542. Essentially that means a conceptual shift from seeing scripts/docker_run as "the idiomatic way to build Oak" towards "a connivence script that aliases host resources", while "reproducible builds" are done with less aliasing.
Imo that's probably not a bad solution, and I think preferable to both aliasing UID/GID / chowning the workdir / docker user creation etc.

RE the architectural concerns, I agree with your approach that we can solve this down the rod @rbehjati, thanks for writing down your thoughts. :)
RE trusted builder vs reproducible builds, I think we can do both, and it'd be nice to do both. As the saying goes "trust but verify" 😅

https://github.com/project-oak/oak/pull/3152 fixes the issue on Oak.


MRTK 2.7.3 -> 2.8.3 migration: Articulated Hand Tracking: no more Articulated Mesh Prefab ?

Overview
In the MRTK configuration panel, Input section, Articulated Hand Tracking subsection, there is no more Hand Mesh Prefab but System Hand Mesh Material and Rigged Hand Mesh Material.. ??
Expected behavior
Being able to configure a Hand Mesh Prefab, not Materials only which sound to me as a (strange) regression
Actual behavior
Regression allowing us to only configure Materials but no more Prefabs
Steps to reproduce
Just compare MRTK configuration of Input/Articulated Hand Tracking between 2.7.3 and 2.8.3
Unity editor version
2020.3.43
Mixed Reality Toolkit release version
2.7.3/2.8.3

@Xispeo thank you for your feedback.  What scenario are you currently blocked on?
In MRTK 2.8.3 you are still able to modify the rigged hand model for the control definition settings:

Does this not meet your requirements?

Yes, perfect, thank you for your fast answer


HSEARCH-3786 + HSEARCH-3797 Two-phase projections in the Lucene backend

HSEARCH-3786: Lucene projections on fields in nested documents should limit children collection to the top docs
HSEARCH-3797: Two-phase projections in the Lucene backend

I just addressed a few issues reported by Sonar, and added a few commits to address a problem I noticed just now.
Without these commits, we would only fetch data from the nested documents of top docs, but we would still perform a join between all matching documents and their nested docs, which might be a problem. It's solved.
Will merge as soon as the CI build succeeds.

Merged. Thanks for your review!


Awesome Flutter

You're welcome to PR your library on https://github.com/Solido/awesome-flutter (17000 stars)
So more people can discover your work ! thank you.

You're welcome to PR your library on https://github.com/Solido/awesome-flutter (17000 stars)
So more people can discover your work ! thank you.

I have already created pull request.


Add podman backend

closes #85
This is a first working podman backend, which can be used by calling the agent with --use-podman.
It is working on my environment fine for now, but would need broader testing.
This is a early version and cannot be called stable at all for now, but it makes sense to include it eary in my opinion.

A hint: We do use vendoring yoz should do:
go mod vendor
go mod tidy
```and commit result.

ok problem is - we do use very outdated dependencys ... - upgrade of them need to be in an own pull

Yep I've tried it but there seem to be very outdated deps inside :)

It will take me some time but I'll do migrate them one by one ... - you can just merge master into from time to time to check if it will start working untill it do ;)

It will take me some time but I'll do migrate them one by one ... - you can just merge master into from time to time to check if it will start working untill it do ;)

Great I'lll work further on the implementation in between as well

@mfulz now it is building - I think you can move forward :)

Is this in essence compatible with next or would it need a complete rewrite? (assuming it was based off from 0.15.x initially).

It targets the main branch ... so its next what ever version next is going to be when it's finished :)
The interface for backends did not changed that mouch ... at all anyway

@mfulz Are you still using your interface or WP at all? Curious how close this is to be "production ready", also given how long this PR is already open :)

@mfulz Are you still using your interface or WP at all? Curious how close this is to be "production ready", also given how long this PR is already open :)

Sorry for the silence atm this is stalled for me, as I'm loaded with too much other things, that cannot be postponed :(
I was just checking wp out and drafted the podman stuff and definitely go to proceed with this.
But when this will happen is impossible to tell atm. could be end of this year or next year - sorry for that

At least I can fix the errors for the checks if that would help.
But I cant access the details on what is failing for the test, can someone help me out here?

Well then just let it open as is ... open pulls per se dont hurt.
Others can base new pulls on it or this gets ready ...


improve default xdg configuration

look into https://github.com/bnjmnt4n/system/blob/main/home/xdg.nix ???

https://github.com/jakeisnt/nix-cfg/commit/09b984c7de03f7419d713c08da8a2f5b974187b4


DocsArchive-<IP_ADDRESS>Auto archive content

Auto archive content from https://github.com/OPS-E2E-PPE/docs-archive-test-source

Docs Build status updates of commit 023365d:
:x: Validation status: errors
Please follow instructions here which may help to resolve issue.

File
Status
Preview URL
Details

:x:Error

Details

[Error]

Cannot sync git repo to specified commit because branch Release_Archive_master_2020-02-22-11-53-39 has been deleted or has been force pushed: fatal: couldn't find remote ref refs/heads/Release_Archive_master_2020-02-22-11-53-39

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


gotosocial: 0.13.0 -> 0.13.2

Version 0.13.2 fixes some less severe security issues (see [1]) so this should be backported.
Replaces #281831
Closes #281659
[1] https://gts.superseriousbusiness.org/@gotosocial/statuses/01HNZ5GF2EM5M899B7RKXYNMH9
Description of changes
https://github.com/superseriousbusiness/gotosocial/releases/tag/v0.13.2
Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[x] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
24.05 Release Notes (or backporting 23.05 and 23.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

Not sure about the port to stable thing, as this would mean stable would receive a "minor" upgrade (0.12->0.13), but it shouldn't be breaking from what I can see (only database migration which runs automatically and new configuration options which all have sensible default values)

@ofborg test gotosocial

Looks like the package's tests are flaky, they didn't pass on ofborg. Not sure if that's a new issue with this version bump or if it was already a problem on 0.13.0.

The corresponding test file wasn't edited since https://github.com/superseriousbusiness/gotosocial/commit/16275853eb8a43e0b113d476b896de53585c1281 which is already included in 0.13.0-rc1, so my guess is that they were already faulty in 0.13.0
Further more one of the developers confirmed to me in their matrix chat that they also observed those test failing sometimes


[Bug]: Can't edit node data

What happened?
I upgraded to 7.1.2 and now I get this when I edit any node:

How to reproduce?
Edit a node.
Version
7.1.2
Node-RED Version
2.2.2-16
What hardware are you seeing the problem on?
Raspberry Pi
Relevant Z-Wave-JS log as a text file
No response

Yup!
it's been fixed in 7.2.0 😄
I'll close this, as its technically a duplicate of https://github.com/zwave-js/node-red-contrib-zwave-js/issues/231

Would it make sense for me to stick with x.x.0 versions for my production system?

I'll be pushing out 7.2.0 in a day or so (maybe Evan this evening, if I can find the energy)

The missing label bug has been there since I introduced  it normalizedObject
The bug to show simple message type prompts, was introduced in 7.1.2.
It only affects node naming and location, and adding associations


It would be nice if possible support for .net 3.1

It would be nice if possible support for .net 3.1
due to the size of legacy projects.

Hi, mmm. I'm not sure, because net 3.1 is out of support.

Hi, I'm closing this one. There won't be net 3.1 support but if you still need it. You can make a copy of this project and modify it to support net 3.1


Remove blobstore info from IaaS metadata endpoints

see https://github.com/cloudfoundry/bosh-aws-cpi-release/pull/147 for more information
The BOSH agent reads its blobstore settings from the metadata API endpoint (or equivalent) for its VM within the IaaS. If the blobstore settings are not set in the env.bosh.blobstores property, it will fallback to the top-level blobstore property in the metadata.
However, in modern configurations, the Director always sends the blobstore settings as part of the environment hash. Additionally, the Director does redaction of credentials in the environment hash when the signed URLs blobstore feature is enabled. This redaction is not applied to the top-level blobstore property in the metadata because that is generated solely by the CPI.
Rather than updating each CPI to know about the signed URL feature, we are instead removing the blobstore properties from the CPI. This will ensure that Director is the sole point of contact when configuring agent blobstore settings, and ensure that they are always properly redacted.

Pardon my naive question: What else is needed to get this PR merged and delivered?

@eaglerainbow a pr needs 2 reviewers to be able to merge it in to main. Then it needs to go through the ci. And after that a release can be made

@eaglerainbow a pr needs 2 reviewers to be able to merge it in to main. Then it needs to go through the ci. And after that a release can be made

ah! Thanks, didn't know that with 2 reviewers.
Waiting and being patient...

@lnguyen could you cut a new release with this change?


Allow early exit of simulation

Allow users to end the simulation early, irrespective of the remaining number of iterations specified in the command-line input.
As an extension, allow infinite iterations in console mode under the assumption the user has created a call to exit the simulation at some point

This would be fairly easy to integrate. Suggest a host function called exit_early() which would just break the main iterations loop and then preserve any behaviour with XML writing.


#33 Add opacity to the background

Closes #33
What happened 👀
While the pub dev is down, I'm working on this cosmetic update 😭 adding an extra opacity layer to the background to be closer to the design.
Proof Of Work 📹
It's a bit darker now

I'm disabling Github Action now until pub.dev is back


Connection to www.showroom-live.com timed out.

Sorry to bother you. Download videos works but it shows connection error, i don't know how to fix it.
IP:French/China
Ubuntu 16.04 server/Windows 7
Python 3.6.2
ffmpeg
root:/home/showroom# python3.6 showroom.py
Show (S)chedule, Show (D)ownloads, Show (L)inks, (Q)uit
Current Time is 18:18
Scheduling Sakaki Miyu for 19:00
Watching Nishikawa Rei's Showroom
Watching Nara Miharu's Showroom
Downloading Nishikawa Rei's Showroom
Getting https://www.showroom-live.com/api/live/onlives failed: HTTPSConnectionPool(host='www.showroom-live.com', port=443): Max retries exceeded with url: /api/live/onlives (Caused by ConnectTimeoutError(<requests.packages.urllib3.connection.VerifiedHTTPSConnection object at 0x7f9df0b928d0>, 'Connection to www.showroom-live.com timed out. (connect timeout=3.0)'))
Retrying in 0.5 seconds...
Downloading Nara Miharu's Showroom
Getting https://www.showroom-live.com/api/live/onlives failed: HTTPSConnectionPool(host='www.showroom-live.com', port=443): Read timed out. (read timeout=3.0)
Retrying in 0.5 seconds...
Getting https://www.showroom-live.com/api/live/onlives failed: HTTPSConnectionPool(host='www.showroom-live.com', port=443): Max retries exceeded with url: /api/live/onlives (Caused by ConnectTimeoutError(<requests.packages.urllib3.connection.VerifiedHTTPSConnection object at 0x7f9debb60518>, 'Connection to www.showroom-live.com timed out. (connect timeout=3.0)'))
Retrying in 0.5 seconds...
Getting https://www.showroom-live.com/api/live/onlives failed: HTTPSConnectionPool(host='www.showroom-live.com', port=443): Max retries exceeded with url: /api/live/onlives (Caused by ConnectTimeoutError(<requests.packages.urllib3.connection.VerifiedHTTPSConnection object at 0x7f9df0334dd8>, 'Connection to www.showroom-live.com timed out. (connect timeout=3.0)'))
Retrying in 0.5 seconds...
Getting https://www.showroom-live.com/api/live/onlives failed: HTTPSConnectionPool(host='www.showroom-live.com', port=443): Max retries exceeded with url: /api/live/onlives (Caused by ConnectTimeoutError(<requests.packages.urllib3.connection.VerifiedHTTPSConnection object at 0x7f9df0334978>, 'Connection to www.showroom-live.com timed out. (connect timeout=3.0)'))
Retrying in 0.5 seconds...
Getting https://www.showroom-live.com/api/live/onlives failed: HTTPSConnectionPool(host='www.showroom-live.com', port=443): Max retries exceeded with url: /api/live/onlives (Caused by ConnectTimeoutError(<requests.packages.urllib3.connection.VerifiedHTTPSConnection object at 0x7f9df0334fd0>, 'Connection to www.showroom-live.com timed out. (connect timeout=3.0)'))
Retrying in 0.5 seconds...
Getting https://www.showroom-live.com/api/live/onlives failed: HTTPSConnectionPool(host='www.showroom-live.com', port=443): Read timed out. (read timeout=3.0)
Retrying in 1.0 seconds...

Your IP is being blocked by showroom

Thanks! IP limited only in Japan ? But it works well not long ago, and downloading still works. @skeith

Not just Japan, some providers are also being blocked

So I have to change my IP or just ignore the error?

Well ignoring the error won't fix it. You need to either change your IP or use a proxy. Also consider using the development branch, not that it will help fix this specific error, but all future updates will be going there.

you don't need to change your current server for the proxy method, you just need a small server with unlimited or large bandwith for the proxy. You might want to explore which server has not been blocked yet, then setup a proxy there.

Thank you for your advice!  I have set a proxy on a Japanese server(Linode) which has not been blocked yet(IP start with 106) and it has large bandwith. Thanks again for sharing and update this script, it helps me a lot!

glad you sorted it out.
@wlerin I think we can close this issue now


Lung Cancer Detection

Lung Cancer Detection by doing lung segmentation

hi,i am GSSoC'22 contributor and I would like to work on this issue.
can you assign me please?

i would like to work on this issue  please

hi,i am GSSoC'22 contributor and I would like to work on this issue. can you assign me please?

@Alaa543 any updates regarding model?

hi,i am GSSoC'22 contributor and I would like to work on this issue. can you assign me please?

@Alaa543 any updates regarding model?

i am still working on it can you tell me how much time you will give me on it

hi,i am GSSoC'22 contributor and I would like to work on this issue. can you assign me please?

@Alaa543 any updates regarding model?

i am still working on it can you tell me how much time you will give me on it

hi,i am GSSoC'22 contributor and I would like to work on this issue. can you assign me please?

@Alaa543 any updates regarding model?

i am still working on it can you tell me how much time you will give me on it

It's already above 5 days. So please make a PR after 3 days otherwise I will have to reassign this issue to someone.

Full name : Jaya Prakash Sangem
(Im new to Open Source programs and also my GitHub profile is pretty much plain but, I have worked on similar
project on classification of lung cancer  from CT  Scans, So, I hope that experice would help me here addressing this
particular issue.)
GitHub Profile Link : https://github.com/Jaya-Prakash-17
Email ID<EMAIL_ADDRESS><EMAIL_ADDRESS>Approach for this Project :
-->Data Collection
-->Data Preprocessing
-->EDA (applying different filters like Guassian filter, gray scale etc to understand the image)
--> Model Building &  training (I paticularly want to try out various DL Models) and apply segmentation
What is your participant role?: Contributor @GSSOC24
So, kindly assign this issue to me.
THANK YOU!!

Hii, I am Piyush. Under GSSOC'24, I would like to work on this issue. Pls assign me this!! #gssoc24

I am a GSSoC'24 contributor, I would like to contribute, and I have prior experience in Machine Learning, presently I am working on research papers where I have successfully done model training and testing with preprocessing, SMOTE techniques, hyperparameter tuning, and many more things. previously I have worked on Parkinson's detection and cardiovascular disease detection.
@pratishthaa Ma'am I will do this, please assign me this task

I am a GSSoC'24 contributor and a machine learning engineer, could you please assign me this issue

Hi I would like to contribute to this issue under gssoc 24


Does it support MapMatching feature for Android (offline)?

Example: https://farm1.staticflickr.com/530/18518221448_5752e8ab30_o.gif
Does it support offline map matching for Android? If yes, any samples?

Yes seem I can use it with Android, but I'm not sure how to use the class MapMatching
So I want to use the project locally (Java only) without web (I didn't add next libs: matching-web-bundle, matching-web)

Seems its input can only be .gpx file? And it seems matching-web-bundle, matching-web` are required anyway
I was thinking I could use with List<LatLng> gpsRawPoints and List<Long> timestamps

Java usage
Have a look at MapMatchingResource.java to see how the web service is implemented on top of library functions to get an idea how to use map matching in your own project.

What about MatchCommand.java? https://github.com/graphhopper/map-matching/blob/master/matching-web/src/main/java/com/graphhopper/matching/cli/MatchCommand.java
I guess it should be easier with this class to understand how everything is working

Yes here I can bypass .gpx file https://github.com/graphhopper/map-matching/blob/master/matching-web/src/main/java/com/graphhopper/matching/cli/MatchCommand.java#L97

Though I'm not sure how to get results now? I want again just List<LatLng> gpsRawPoints and List<Long> timestamps as output


Simple improvements

Added basic config to test the code using Github Actions
Refactred logic around repo_name and package_name
Added Instructions about transferring from xsh to py

@jnoortheen, here are many useful and meaningful improvements and I recommend to add it to the poetry version too.

@anki-code sure I will try to add it ASAP


[stable/promethus-mongodb-exporter] Customize serviceAccountName for mongodb-exporter deployment

@steven-sheehy
Signed-off-by: Nitin Mohan<EMAIL_ADDRESS>Is this a new chart
No
What this PR does / why we need it:
This PR adds a customizable serviceAccountName field for the prometheus-mongodb-exporter chart. This makes it possible to create mongodb-exporter pods in custom service accounts.
Checklist
[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

[x] DCO signed
[x] Chart Version bumped
[x] Variables are documented in the README.md
[x] Title of the PR starts with chart name (e.g. [stable/chart])

check this out: https://helm.sh/docs/chart_best_practices/#role-based-access-control

@steven-sheehy @maorfr Could you please review this when you get a chance and let me know if anything needs addressing? Would be nice to use a released version of this chart.

Thanks for the review @steven-sheehy . Added a commit to address your suggestions.

/ok-to-test
/assign

/lgtm
Thanks


[TIR][Meta-Schedule] Tuple-reduction scheduling support

This PR improves our TIR scheduling primitives/transformations (rfactor & cross-thread reduction) designed for reduction operators, so that they can be applied to blocks of tuple-reduction.
Most of the code changes are of unit tests.
cc @junrushao1994

Thanks a lot!! Having tuple reduction in TIR is definitely one of my top priority! Need some time to recharge this week and will review next week!


Google Chrome web page shortcut in home not appearing

I created a shortcut via my web browser (Chrome), and it didn't appeared in my home screen, everything is working as it should, in another launcher.

Because this launcher display app list only. No widget. No shortcut or other fancy things

it's a shortcut to a JRPG browser game, I won't likely open my browser and navigate to play the game, I do love this launcher, but if it doesn't fit my needs anymore, maybe it's time to say goodbye.

I'm not the developer. But as we're on github we're all devs here ^^.
Push a pullrequest or git clone the project and compile it with your changes. It's what I'm doing

I'd be happy to support this, but currently have no idea about the following:
a) How to fetch a list of currently pinned web shortcuts
b) Which broadcast should be listened to in order to determine when a new web shortcut is added
I can do some digging in the Launcher3 source code, but any details would be appreciated

Hi Nick, first of all, thank you for developing and sharing Lens Launcher. It is one of the most innovative apps I have ever seen, and I have been using it on a daily basis.
I also wish it could display shortcuts. I thought of a way that might make development easy: can you add an option in the settings that allows users to manually add shortcuts. It would be great if the shortcuts can be things users do all the time, like opening a web link, adding a calendar event, composing an email, etc.


Set up unit tests with WIRC

We got the green light for setting up unit tests with WIRC data. This would involve making a minimum number of WIRC images public, needed for testing the pipeline. We should identify which images are needed:

A target where we have an image to use as a reference for subtraction
A set of flats/darks
A block of dithers for a WIRC stack
Ideally use published data -> Select old target

Linked to/prerequisite for #22

@virajkaram will provide a suitable source


make Travis-CI run an apktool decode on a real APK

This adds a real world use case to the test, it just downloads the Amazon Appstore APK, then tries to decode it.  It downloads build-tools to get aapt.

Ok, the tests work now, but apktool decode is failing, which is what I'm seeing on my local install.

I'm a little weary on this for a few reasons.

The separation of logic for tests. The build script just initializes the environment for apktool to build and thus run tests. I'd probably want this logic somewhere in a test vs the travis file.

Can we confirm the app store will always be there? Additionally that file may change. If it updates and breaks Apktool, that is a bug but at that point all builds will fail because of it.

You could include an APK in the git repo, if you don't want to download
it.  The Amazon URL is going to be about as stable as a URL is going to
get.  If the URL breaks, its very easy to see in the build log, and very
easy to fix.
As for running the test somewhere else, that's fine by me.  But I'm not
going to take on running shell commands in gradle.

Closing this as I'd prefer all test oriented things to be in the tests themselves vs added into the scripts init`d the environment.

how about including some test APKs in the git itself?  There are many
free software APKs to choose from.  It would be very useful if the tests
actually ran real world actions like decoding APKs.

We already have a few test apks - https://github.com/iBotPeaches/Apktool/tree/master/brut.apktool/apktool-lib/src/test/resources/brut/apktool and a few that are made from source during the test suite.
Free software APKs or any APK for that matter, wouldn't exactly help the test suite at all. Say we include one and it fails during the test. Okay what broke? With our current test suite the failure cross references either a pre built apk that is named Issue#### or fails a method in the tests that is named like "FailsUnicodeForResourceName" or something. Failures with test apks would be just that with no easy way to determine why they failed.

better to know that something is failing then not, even if its hard to
debug.  Its not like I'm talking about adding an esoteric process here.
I'm saying that the tests should run apktool at its highest level to
make sure things are working, e.g. apktool d test.apk.

Which is already done in quite a few places - https://github.com/iBotPeaches/Apktool/tree/master/brut.apktool/apktool-lib/src/test/java/brut/androlib
The test suite needs more tests, I'm not going to hide that. Proper tests that check before/after content to confirm success. Simply checking for a non-failure doesn't help much. Apktool can have a succesful decode, but break half the internals of the project.
I'm just not a fan of throwing apks in for the reasons I already mentioned.


How to run a specific version of MongoDB?

I have just come across this tool and it looks really great!
I have installed a multiple versions of MongoDB:
$ mongoctl list-versions

--------------------------------------------------------------------------------
VERSION              EDITION              LOCATION
--------------------------------------------------------------------------------
2.4.12               community            /Users/ytakeda/mongodb/mongodb-osx-x86_64-community-2.4.12
2.6.10               community            /Users/ytakeda/mongodb/mongodb-osx-x86_64-community-2.6.10
3.0.8                community            /Users/ytakeda/mongodb/mongodb-osx-x86_64-community-3.0.8
3.2.0                community            /Users/ytakeda/mongodb/mongodb-osx-x86_64-community-3.2.0

Now, I would like to pick and choose one of these versions for our testing, then later switch over to the other, etc.  How do I start the specific version with $ mongoctl start command?

Unfortunately that is currently not supported but it sounds like a good feature. You will have to update the server's "mongoVersion" in the config each time you want to have use a different version. Let us think about it and we'll keep you posted.
Thanks!

How can i get the file for updating specific version of mongo?


Add new meta-attributes to stdenv to permit URLs for screenshots

Motivation for this change
If a meta-attribute for a Screenshot-URL were added to the package description, the search result could include a screenshot of the application in the result. There is no need to create new screenshots as http://screenshots.debian.net/ provides a public repository of screenshots already that could be reused.
I am aware that adding screenshots to the existing thousands of packages would be a major undertaking, but I think it would be a nice user experience enhancement and - of course - optional, since screenshots only make sense for graphical applications.
When the Screenshot-URL is provided for a package, the search result could fetch the image and show it alongside the package description.
The font packages could show a sample of the font instead.
Things done
Two paragraphs were added to the Nixpks-manual, section Stdenv -> Meta-attributes to explain the two new
attributes ScreenshotThumb and ScreenshotLarge.
In the package games/pingus the two attributes were added as an example.
The Nixpkgs-manual was rebuild to check correctness.

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[x] Ensured that relevant documentation is up to date
[ ] Fits CONTRIBUTING.md.

@garbas
Discussed in https://github.com/NixOS/nixos-search/issues/211

An alternative approach would be relying on the appstream data (https://github.com/NixOS/nixpkgs/issues/15932), cc @matthewbauer.

Please squash the commits together.

I can't take it from here. The other (#15932) issue is quite complex, involves many aspects. I cannot even identify all the different parts that are involved.
Only one thing is for sure: information in the stdenv and the associated appstream data (if available) must be synchronized.
Ideally, one should "rule" over the other to have one authoritative source of truth.

@matthewbauer is there any timeline on #15932? That RFC is quite old, but I can see it is still in your radar. We could serve AppStream XML via search if you want.
Would you be ok if @StefanSchroeder continues with this new meta-attributes? I'm assuming that we will be doing some merging of some sort when producing this XML.

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/brainstorming-generating-appstream-metadata/19273/1


Goodwe support

Feature request for goodwe support via goodwe library:
https://github.com/marcelblijleven/goodwe

First version released.


Yarp bindings  python

Hi,
We are trying to use Python in order to control the iCub, but after installing with python3.5 the libraries, following the instructions in the README, the example files in yarp/bindings/python/examples don't start and this error occurs:
AttributeError: module 'yarp' has no attribute 'Network'

How can we solve this problem?
Thanks in advance,
Roberto

Any Python developer eager to help on this?
@robotology/all-robotology-developers

Hi @robertorovella91, what is the content of your  PYTHONPATH environmental variable? Which OS are you using?

Hi @traversaro
The problem was that we had not setted PYTHONPATH with the folder containing yarp.so file. Now we have fixed and solved the problem.
Thank you.

I am in same problem, (module 'yarp' has no attribute 'Network')
I added to path, but not solved,

expecting soon response,.. windows 10

Me too, thanks, I will add another message in 10 minutes in case you haven't seen this.

Ops sorry I meant 10 seconds.. here's my second message.

@robertorovella91
Hi I no folder containing yarp.so file


[WIP] Add Benchmark for rawloader

Closes #46.
I am hoping for an early review, @anp. Some notes:

This benchmark requires manually downloading an image file from the internet. It is Creative Commons Licensed, but cannot be downloaded by the benchmark itself due to Cloudflare. If you would prefer I commit the 18 MB file to the repo, I can.
This library is LGPL 2.1, as is the benchmark code I started with. I have noted this in the license key of the Cargo.toml itself. If there is somewhere else I should note it, let me know.
The benchmark code used to be an executable with a main() function that did its own iteration, and had no black_box. Therefore, I am hoping this version does not need a call to black_box either.
It is not 100% clear to me whether the latest version of the library on crates.io matches master or not, so I pinned it by git commit, and use that as the version number.
I am not touching anything about the runners, as per the directions.
I am hoping the Cargo.lock additions will not break anything, and will pin its dependency version numbers. If I am wrong, how do I fix that?
While I have verified the benchmark executes, I cannot figure out how to actually measure its performance on my own system to compare it with the original. cargo bench does nothing, and cargo test --release does not give me any numbers. Is this expected?

I imagine I left something out, so please let me know what. :smile:

This library is LGPL 2.1, as is the benchmark code I started with. I have noted this in the license key of the Cargo.toml itself. If there is somewhere else I should note it, let me know.

Rust statically links, so anything depending on LGPL must itself be LGPL licensed.

While I have verified the benchmark executes, I cannot figure out how to actually measure its performance on my own system to compare it with the original. cargo bench does nothing, and cargo test --release does not give me any numbers. Is this expected?

From the readme:

cargo test-all runs the test for every benchmark function, which consists of warming it up and running through a couple of iterations. caution: this will generate dozens of gigabytes of data in your target directory.

Rust statically links, so anything depending on LGPL must itself be LGPL licensed.

My understanding is that all 8 of the dependencies are LGPL compatible due to dual licensing under MIT. (Unfortunately Apache 2.0 is not LGPL 2 compatible.)
As a result, any user who builds this benchmark will implicitly choose MIT for the dependencies, and then be forced by the library to opt into the LGPL for that particular executable (as per Section 3, Paragraph 6). Other binaries and code are unaffected.
Given the warning about licensing in the README, and the general use case for these benchmark, I understood this to be acceptable.

From the readme:

Oh. I didn't read that as, "this is the only way to run the tests," but rather as "don't do this unless you own a giant box and are ready for significant resource use." Thanks for the clarification.

Hey, thanks! To inform expectations, you should probably know that the bots for this project haven't been running for a bit (#68) and I'm currently very slowly moving the CI over to something that works today (#72). Landing might take a little while

Understood.
... Although I would say, progress looks pretty good. The only failing check on that PR is the (dead for a while) website.
Perhaps it would be possible to find a "good enough" CI for the time being, just to accept PRs again? I do see this referred to in Rust issues at times, so I can't be the only one who is interested in it.

When it comes down to it, the provenance is important but redistributing this many binaries with code cribbed from this many sources is always going to be fraught (which is fine because we mostly need data, not runnable programs to download).

Indeed. The "source only" nature of this project -- where redistributing any binaries someone built would mostly defeat the purpose -- is the reason I didn't consider the licensing issue a showstopper.

I think that's probably the right call, left a review comment.

Since the website died due to the size of the repo, I thought I'd be conservative and ask before bloating the repo significantly with one commit! :smile:

A separate, stupider question, @anp: when I run cargo test-all -- even on master -- I get a hang:
     Running target/release/deps/bench_main-6212ce7833fa9a09
Benchmarking Fibonacci/Recursive: Warming up for 3.0000 s
Benchmarking Fibonacci/Recursive: Collecting 100 samples in estimated 5.0000 s (13B iterations)

The warm-up actually took 3 seconds, but the process burned 100% of a single CPU core on that second message for 45 minutes before I killed it.
Dumping the assembly shows a hot and tight recursive function, that, well, looks like Fibonacci. But I can't seem to see it ever changing stack frame sizes.
Worse yet, I can't even figure out how to remove it. Even after deleting the rayon benchmarks -- where there are functions with matching names -- and doing cargo clean, it still did this.
This was with both Rustc 1.36 (which the project wanted me to download) and current stable (1.51). What am I missing?


Update iOS versions up to 16.6.

Jailbreak tools up to date as of 27th July.
unc0ver support on iOS 14 is a mess. It could be split into 2 or 3 versions.

Thanks!


Style changes

Removed the last of styled-components & changed tasklist search bar styles according to figma.
Only info-button style was defined in figma, so outlined buttons felt more cohesive in the task table.

Hyvä huomio, fiksattu! Voinko mergeillä? :)

merged


Add subcommand to generate HTML report on demand

This is mostly useful while making improvements to the report, such that we can iterate on the output report without re-running slow benchmarks all the time.
Related to #5.

Used this branch merged with #5 to produce reports on demand successfully 😄
Good job @AbhiPrasad !

Rebased and potentially ready to be merged.


Babel-plugin dont handle empty location

Q
A

Fixed Issues?
https://github.com/redux-saga/redux-saga/issues/1742

Patch: Bug Fix?
bug fix

Major: Breaking Change?
no

Minor: New Feature?

Tests Added + Pass?
Yes

Any Dependency Changes?

FunctionExpression didn't have empty location check (when code is not part of original code, but generated by other babel-plugins) so I added

I suggest that you go here to find out how to upgrade to the latest version of babel:
https://babeljs.io/docs/en/v7-migration

Maybe then you can try again.

@szb512 actually we aim to support both babel v6 and v7. later we will remove v6 when most projects drop its support


Issue 19

If a file/url is not found, an exception created and is logged.
Definitely can use a bit of refactoring here and there.

I did miss adding tests for the code I added in ClientHandler, including logging and the if else. I will be working on that for a bit.


Nested Yarn Workspace in a Repo results in extraneous changelog entries

Describe the bug

I have a monorepo that does not use Node.js or Yarn on the root, but rather a sub-directory called frontend (this uses Turborepo).
When using monoweave, the CHANGELOG that is generated includes changes that are outside of the frontend directory. This is because when git diff-tree is executed, the --relative flag is not included, causing the parent directory changes to show up even though monoweave is located within frontend.
I made a change to gitDiffTree to include --relative and that seems to fix the issue (when running yarn monoweave in the frontend directory).

[ ] I am interested in contributing a fix.

To reproduce
Minimal reproduction available here: https://github.com/stephenliang/monoweave-bug

Create a repo with folders backend and frontend
In frontend, add monoweave
Add a git tag
Make some change log entries for backend
Make some change log entries for frontend
Run yarn monoweave in frontend

Expected behavior
Only change log entries are created within frontend. No backend changelogs are created.
{
  "frontend": {
    "version": "2.0.0",
    "previousVersion": "2.0.0-alpha.4",
    "changelog": "## [2.0.0](//compare/frontend<EMAIL_ADDRESS>\"frontend\" (12/11/2024) \n\n### Changes\n\n* feat(monoweave): add monoweave ([b1d8b915](//commits/b1d8b915))",
    "tag"<EMAIL_ADDRESS>    "strategy": "minor",
    "group": "frontend"
  },
  "a": {
    "version": "3.1.0",
    "previousVersion": "3.0.1",
    "changelog": "## [3.1.0](//compare/a<EMAIL_ADDRESS>\"a\" (12/11/2024) \n\n### Changes\n\n* feat(a): update readme ([29c8ce16](//commits/29c8ce16))",
    "tag"<EMAIL_ADDRESS>    "strategy": "minor",
    "group": "a"
  }
}

Actual behavior
Change log entries are created within frontend including backend changelogs.
The following Changelog was generated for the reproduction repo (notice how it includes changes for backend):
{
  "frontend": {
    "version": "2.0.0",
    "previousVersion": "2.0.0-alpha.4",
    "changelog": "## [2.0.0](//compare/frontend<EMAIL_ADDRESS>\"frontend\" (12/11/2024) \n\n### Changes\n\n* feat(monoweave): add monoweave ([b1d8b915](//commits/b1d8b915))\n* feat(a): update readme ([29c8ce16](//commits/29c8ce16))\n* feat(backend): update docs ([8b63f5f0](//commits/8b63f5f0))",
    "tag"<EMAIL_ADDRESS>    "strategy": "minor",
    "group": "frontend"
  }

Monoweave Configuration
export default {
  preset: 'monoweave/preset-recommended',

  // Aggregates changelog entries to a single file instead of the "recommended"
  // default of per-workspace.
  changelogFilename: 'CHANGELOG.md',
  conventionalChangelogConfig: 'conventional-changelog-conventionalcommits',
  dryRun: false,
  prerelease: true,
  prereleaseId: 'alpha',
  versionStrategy: {
    minimumStrategy: 'patch',
  }
};

CLI (run in the frontend directory)
yarn monoweave --dry-run --log-level 0

Monoweave Version:

Environment (please complete the following information):

OS: Fedora 42
Yarn Version: 4.2.1

Yarn module linker: node-modules

Node Version: 20
Git Version: 2.47.0
CI Tool: N/A

Additional context

Your "patch" makes sense to me. I have a PR to add "--relative". Just need to add some test coverage before it can get merged. We should be able to mimic your repo set in an e2e test.

Fixed in https://github.com/monoweave/monoweave/releases/tag/monoweave%401.11.1