Example needed: How to POST batch requests gzip encoded

Hi,
could you provide an example of preferred way how to send the requests (gzip) compressed? This would be useful e.g. for large batch updates where the gzipped method may save minutes (transferring 800k vs 25M).

@monczszilard  Basically you have to inject a HttpHandler into the HttpClient to perform compression.  The good example of this is https://github.com/azzlack/Microsoft.AspNet.WebApi.MessageHandlers.Compression which means you don't have to write your own handler etc. Also interesting are https://cymbeline.ch/2014/03/16/gzip-encoding-an-http-post-request-body/ and the SO answer for http://stackoverflow.com/questions/30310099/correct-way-to-compress-webapi-post
This needs to be assigned into your ODataClientSettings instance's OnCreateMessageHandler delegate, e.g.
odataSettings.OnCreateMessageHandler = () =>
{
	var handler1 = new ClientCompressionHandler(new GZipCompressor(), new DeflateCompressor());
	var clientHandler = new HttpClientHandler
	{
		// Handles compressed data
		AutomaticDecompression = DecompressionMethods.GZip | DecompressionMethods.Deflate
	};

	handler1.InnerHandler = clientHandler;

	return handler1;
};

This is adapted from my own code using different handlers so some minor tweaking may be required, but the basic rule is that the HttpClientHandler should be the last handler in the chain since it's the one that actually does the communication - I would guess that the compression one needs to be immediately prior to that e.g after MD5 computation etc but you may need to play with that.


Add support for asynchronously awaiting for a duration of time

Add io_context::schedule_after(std::chrono::duration<rep,ratio> duration) to suspend awaiter until specified duration has elapsed and resume on io context.

830c4d1d3088356d94ea4643205492b57dfa9514 has implemented this for io_service on the Windows platform.
I've also added support for cancellation of timers by passing an optional cancellation_token into the schedule_after method.
I'm still working on adding some documentation for io_service and schedule_after() to the README.

Duplicate of #12


Team Workspaces

Describe the feature
The unimplemented feature for Team Workspaces would be very helpful - I have a PR for you and this is the requested issue.

@cmendible may I request a new tag to push this to Nuget please

Release here: https://www.nuget.org/packages/Tfe.NetClient/0.1.3


Differentiate workflows that push to Prime registry and GitHub registry

Description
The current release workflow builds and publishes images to both GHCR and Prime registry. While creating the first Rancher Turtles release candidate we thought we could split these actions into two different workflows for each registry.

PR #183 is now rebased and ready for review.


Missing binary wheel for Python 2.7 on 64-bit Windows

See the note here: https://www.benfrederickson.com/writing-python-extensions-in-rust-using-pyo3/

I failed to get the 64 bit wheel for Python2.7 to build on windows so I disabled that here (the 32 bit version builds fine).

Not sure what the cause is (suspect it might be related to https://github.com/PyO3/pyo3/issues/215). Will leave this issue open in case anyone wants to tackle it, but putting this on hold for now since it's a pretty big time investment and not sure how common this need is. If you're interested in contributing, it should be possible to reproduce this with cibuildwheel locally or against the CI by opening a PR.

@msanders I have the binary wheel for Python 2.7 in https://github.com/DLR-TS/SUMOLibraries/tree/master/python/autopy


Does .NET's Trace.CorrelationManager.ActivityId get propagated to Application Insights?

I have an existing app in .NET that uses Trace Listeners with System.Diagnostics.Trace.CorrelationManager.ActivityId to correlate activities together.  I was able to route my Trace*() calls to Application Insights with no problems - super cool! - but I am unable to filter my Traces on this ActivityId.  Is there a way for me to do so?
The official Telemetry article acknowledges that the CorrelationManager exists, but it is unclear if ActivityId is supported out of the box.

CC: @lmolkova

@alexchantavy
AppInsights does not use CorrelationManager.ActivityId.
Instead, it uses new concept System.Diagnostics.Activity for correlation.
Since CorrelationManager.ActivityId is implemented with CallContext, it flows along with async calls and probably you can create a custom TelemetryInitializer that would set it on the telemetry.
However, we discourage you from doing this. Please use Activity, it allows you to correlate your traces with any other operations tracked by AppInsights across all of your services.

What's should be used for correlation, that can be used on both net core 2.2 and net FX 4.7.2?

adding updated link from @lmolkova 's post above:
ActivityUserGuide


Make more use of curses mouse and key constants

Currently some keys and all mouse buttons are simply matched against numbers (https://github.com/richrd/suplemon/blob/master/suplemon/main.py#L332). They seem to be different in different environments so we should use the Curses contstants as shown here: https://docs.python.org/2.7/library/curses.html#curses.getmouse

Fixed in latest version, except for some mouse features. Mouse support needs more than just new bindings so i'll handle that separately later.


FINERACT-609 Group Topup Loan Bug Fix.

Fix FINERACT-609 issue where topup loan wasn't working for group loan accounts. This was because the server wasn't sending the JSON array (named activeClientLoanOptions) which contained the IDs of active group loan accounts that could be closed in order to create the topup group loan account.
Now when creating a group topup loan, the server correctly replies with active group loans (named activeClientLoanOptions) so that an account to close can be selected in order to create the new group topup loan account.
Description
Describe the changes made and why they were made. Ignore if these details are present on the associated Jira ticket
Added query: retrieveGroupActiveLoanAccountSummary(Long groupId) to get the active group loans (that can be closed) in order to complete the group topup loan application.
Checklist
Please make sure these boxes are checked before submitting your pull request - thanks!

[x] Commit message starts with the issue number from https://issues.apache.org/jira/projects/FINERACT/. Ex: FINERACT-646 Pockets API.

[x] Coding conventions at https://cwiki.apache.org/confluence/display/FINERACT/Coding+Conventions have been followed.

[x] API documentation at https://github.com/apache/fineract/blob/develop/api-docs/apiLive.htm has been updated with details of any API changes.
I don't think any update is required.

[ ] Integration tests have been created/updated for verifying the changes made.

[ ] All Integrations tests are passing with the new commits.
I'm creating this PR as advised by my GSoC mentors to see whether the integration tests on Travis-CI succeed. Even my local upstream/develop branch has failing integration tests - possibly due to https://issues.apache.org/jira/browse/FINERACT-707

[x] Submission is not a "code dump".  (Large changes can be made "in repository" via a branch.  Ask on the list.)

Our guidelines for code reviews is at https://cwiki.apache.org/confluence/display/FINERACT/Code+Review+Guide

@vorburger I have pinged @drobson98 via email to see if he can address this feedback and feedback on his other PRs from GSOC.

@DylanRobson are you still planning to work on this?
I'll close this PR on 2020.01.31 if there are no updates until then; FYI @edcable @vishwasbabu.

@DylanRobson we'll assume that you have no interest in this PR anymore, and will just close it (unmerged) on 2020.01.31.

@percyashu  or @nnatarajan , Will u be interested to pick this up and address the comments, build ontop of this and send a new PR. See the issue: https://issues.apache.org/jira/browse/FINERACT-609
Indicate here if you will be working on it and assign the issue to yourself.


wip: support dotcom server-side models

CLOSE https://linear.app/sourcegraph/issue/CODY-3800
WIP.
Add support to new fields added by https://github.com/sourcegraph/sourcegraph/pull/643
This ensures the server-side config for dotcom will be rendered as the client-side models currently is.
TODO:

[ ] waitlist feature flag check on server-side models

Test plan

wip.
Changelog

wip

@jamesmcnamara the tests are failing after the last commit due to this error Error: No default edit model found. Please set one.
looks like it's having issue finding the default model for edit?

@valerybugakov would like your eyes on the changes made to autocomplete 🙇♀


Not on Packagist

I noticed this was not on Packagist so I went ahead and added it (https://packagist.org/packages/cchiles/laravel-rackspace-cloudqueue)
The readme says to use 1.0, but it looks like there is nothing tagged there. Is this still in development?
I'd be willing to take over the repo to support it if you're interested!

@willwashburn: Thanks for adding to Packagist. I developed this for a project, but I haven't needed to do much in terms up new features and support. If you want to take it over and support it, I'd be more than happy to transfer it.

Sure thing! If you want to transfer it to @tailwind we'll be happy to support it

I just tried to transfer it, but I don't have admin access within @tailwind. Github requires that to transfer to an organization. https://help.github.com/articles/transferring-a-repository/#transferring-from-a-user-to-an-organization

weird, ok want to just transfer it to me and I can transfer it over? Thanks for the help with this!

It looks like you already have a fork of it, so it won't allow it because the repos are the same name.

Ha, figures. Sorry man. I just deleted it. Should work now.

All set. Request sent!

Thanks @cchiles!


请问您在V2版本论文中的Visualization of the learned attention masks是怎么做的

We refer to the following code base for the visualization code in the paper.
https://github.com/jacobgil/vit-explain
https://github.com/jeonsworld/ViT-pytorch/blob/main/visualize_attention_map.ipynb
Hope the answer is helpful.

Thanks!


Contacts appear offline

When signing out and then signing back in certain contacts appear offline when they are online. Sometimes this happens after automatically reconnecting (i.e: when chat switches from cellular data to wifi)

+1


MdSelect overlapTrigger input parameter

Proposal.
MdSelect should have an overlapTrigger input parameter like MdMenu.
Usage:
<md-select [overlapTrigger]="false">
    <!-- md-options -->
</md-select>

Something like this:

In the case of the select, the Material Design spec is pretty clear that the selected item should always overlap the trigger and not duplicate the selected item (see the "don't" images here). Because of that explicit instruction, we aren't likely to add this feature to the select.

I understand not having duplicate select items in the overlay itself, but this would be an option that allows for the overlay to always be positioned underneath the select element for smaller lists. The duplication of "Green" in the above example is not a duplication in the overlay list, but rather the currently selected item in the context of the select element.

@JaimeStill It sounds like perhaps we're looking at different images. This one in the "Dont" section specifically calls out that the overlay should always overlap the trigger.

Don't.
Menus are positioned over their emitting element.

Thank you for your time and consideration @kara.


[7424] Remove lead partners & itt_reform feature flags

Context
lead partners & itt_reform feature flags
Changes proposed in this pull request
Remove lead partners & itt_reform feature flags
Guidance to review
It has been removed, test still works
Important business

[ ] Does this PR introduce any PII fields that need to be overwritten or deleted in db/scripts/sanitise.sql?
[ ] Does this PR change the database schema? If so, have you updated the config/analytics.yml file and considered whether you need to send 'import_entity' events?
[ ] Do we need to send any updates to DQT as part of the work in this PR?
[ ] Does this PR need an ADR?

NB: Please notify the #twd_data_insights team and ask for a review if new fields are being added to analytics.yml

 Remove itt_reform & lead_partners feature flags


Added a working example to Custom Events.

While reading on Custom Events, I almost accidentally copied the non-working code.
I think the documentation would benefit from a working sample.
Here is my humble contribution to this great framework.

Thanks, but I do believe the line right above your added content is as already clear as it can be:

For these reasons, we recommend you always use kebab-case for event names.

Besides, the grammar is broken :)

Ooops sorry for the bad grammar. Got too excited trying to contribute.
But as a user I would like to insist the documentation for Custom Events is missing a working example. It's not that it is not clear, it is that the only example given is of how not to do it. Someone should add a working example.


Increase paging limit

temporarily fixes https://github.com/openshiftio/openshift.io/issues/3973

Ike Plugins (test-keeper)
Thank you @DhritiShikhar for this contribution!

It appears that no tests have been added or updated in this PR.
Automated tests give us confidence in shipping reliable software. Please add some as part of this change.
If you are an admin or the reviewer of this PR and you are sure that no test is needed then you can use the command /ok-without-tests as a comment to make the status green.
For more information please head over to official documentation. You can find there how to configure the plugin.

Codecov Report

Merging #2166 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #2166   +/-   ##
=======================================
  Coverage   69.45%   69.45%
=======================================
  Files         166      166
  Lines       15446    15446
=======================================
  Hits        10728    10728
  Misses       3748     3748
  Partials      970      970

Impacted Files
Coverage Δ

controller/paging.go
97.53% <ø> (ø)
:arrow_up:

Continue to review full report at Codecov.

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

Please create an issue to revert this after fixing it in the UI.

Created an issue to track the discussion related to pagination https://github.com/openshiftio/openshift.io/issues/3978

Created an issue to revert the change -> https://github.com/fabric8-services/fabric8-wit/issues/2167


Custom data - support coordinateProperties

Followup from the recent PMTiles / Vector Tiles work: #1016
Our custom data layer supports various formats but they all end up converted into GeoJSON.
Sometimes - we end up with an extra data element called coordinateProperties.   This is an extension of GeoJSON that's not really standard but is still widely recognized.
https://github.com/mapbox/geojson-coordinate-properties
The gist of it is: for each coordinate in the Array, you can look up a corresponding bunch of properties that go with that coordinate in another Array.  It's pretty common for any kind of workout data:

Anyway we shouldn't just do the [object Object] thing here - we should actually make nodes that have those properties.

@voscarmv - prospective issue for you to work on!

@bhousel can you supply a dataset URL with some of this data? I would like to get oscar started with this.

This one includes some extra trackpoint data for the elevation values:
https://rapideditor.org/rapid#background=Bing&gpx=https://sonny.7thposition.com/osm/escarpment_trail.gpx

Oscar- here are some code pointers for you to look at.
Background
The data URL Bryan provided has the following shape in XML:
<gpx xmlns="http://www.topografix.com/GPX/1/1" xmlns:gpxtpx="http://www.garmin.com/xmlschemas/TrackPointExtension/v1" xmlns:gpxx="http://www.garmin.com/xmlschemas/GpxExtensions/v3" xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" version="1.1" creator="Garmin Connect" xsi:schemaLocation="http://www.topografix.com/GPX/1/1 http://www.topografix.com/GPX/1/1/gpx.xsd http://www.garmin.com/xmlschemas/GpxExtensions/v3 http://www.garmin.com/xmlschemas/GpxExtensionsv3.xsd http://www.garmin.com/xmlschemas/TrackPointExtension/v1 http://www.garmin.com/xmlschemas/TrackPointExtensionv1.xsd">
<script/>
<metadata>
<link href="connect.garmin.com">
<text>Garmin Connect</text>
</link>
<time>2014-07-27T13:25:06.000Z</time>
</metadata>
<trk>
<name>Escarpment Trail Run</name>
  <trkseg>
    <trkpt lon="-74.1896016150713" lat="42.312879441305995">
    <ele>547.5999755859375</ele>
    <time>2014-07-27T13:25:06.000Z</time>
    </trkpt>
    <trkpt lon="-74.18951326981187" lat="42.31288396753371">
    <ele>559.5999755859375</ele>
    <time>2014-07-27T13:25:10.000Z</time>
    </trkpt>
    <trkpt lon="-74.18948912993073" lat="42.31287952512503">
    <ele>555.4000244140625</ele>
    <time>2014-07-27T13:25:11.000Z</time>
    </trkpt>
    ... GPX data continues
  </trkseg>
</trk>

This data is processed by Rapid's custom-data processor and converted to GeoJSON. You won't see coordinateProperties mentioned anywhere in Rapid's code because this processing is done by a third-party module: @tmcw's togeojson module, imported like this:
import { gpx, kml } from '@tmcw/togeojson';
This module takes the gpx track above, ingests the <ele> and <time> tags for each point,  and then puts them in a separate CoordinateProperties blob on the geoJSON as per the spec Bryan linked, along with the track name, type, and time the track was saved.
This results in the undesirable display of the 'tags' data up above, where the coordinateProperties is just getting toString() called on it, hence the [object Object] display.
This problem is interesting because coordinateProperties is a special-case blob of data.
If you run Rapid in the debugger, and set a breakpoint at https://github.com/facebook/Rapid/blob/main/modules/ui/sections/raw_tag_editor.js#L163  you will see the rowData being fed to the ui renderer in the sidebar- the coordinate Properties is 700+ entries, one for each 'point' in the GPX track.
So, the answer here is not to render all 700 points' worth of data in the tag editor- the data would run off the screen. The answer is actually to render the individual points in the custom data renderer, and give each of those points the coordinateProperty entry that corresponds to it.
Therefore, we need to do the following:

[ ] Stop rendering 'coordinateProperties' as a top-level tag when the custom data entity is clicked on.

[ ] Start rendering individual points in the custom data layer [modules/pixi/PixiLayerCustomData.js]. (https://github.com/facebook/Rapid/blob/main/modules/pixi/PixiLayerCustomData.js)
To get you started, we already do render some points in the custom data renderer- but only if they are provided as a Point or MultiPoint geojson type: https://github.com/facebook/Rapid/blob/main/modules/pixi/PixiLayerCustomData.js#L96

You can start by extracting the points out of the track and just rendering them as points as well-don't worry about the coordProperties for this step.

[ ] once you get the points to render, we'll need to attach the coordinate proprties data to them. You'll need to augment the points you're asking the renderer to display, and also bind the custom data from the coordinateProperties to each point. This should, in theory, display the individual  and  data for each point from this example track data provided.

-[ ] Finally, once you have the point data rendering with the coordinateProperties, see if you can figure out how to make the points rendering conditional on hover/select like the OSM ways already are. Vid:
https://github.com/facebook/Rapid/assets/1887955/79d2ddf6-e9e4-47a8-af79-c1a2da4e2033
See how the points are only revealed when hovered over in the OSM way? Instead of just always rendering every custom data node, we should do the same thing. See if you can figure out how the pixiLayerOSMData.js file conditionally renders the node data on hover, then implement some similar logic in your renderer for the custom data.


[UWP] Stuck on Build started on release mode.

Step to reproduce

Create a blank UWP Project.
Verify its building in Release mode.
Add System.Reactive.Linq (4.1.0) Nuget package.
Run build again in release mode but it never be completed.

Try System.Reactive, not that Linq package.

Any specific reasons?

There's a bug in the .NET Native compiler that's being investigated. For now, please use 4.0.

@akarnokd Same issue with System.Reactive
@onovotny downgraded to 4.0 is working. Thanks.

I update UWP to 6.2.2 and System.Reactive to 4.1 and use <ShortcutGenericAnalysis>true</ShortcutGenericAnalysis> flag then the application will be succesfully compiled with .net Native compilation.
<?xml version="1.0" encoding="utf-8"?>
<Project ToolsVersion="15.0" DefaultTargets="Build" xmlns="http://schemas.microsoft.com/developer/msbuild/2003">
  <Import Project="$(MSBuildExtensionsPath)\$(MSBuildToolsVersion)\Microsoft.Common.props" Condition="Exists('$(MSBuildExtensionsPath)\$(MSBuildToolsVersion)\Microsoft.Common.props')" />
  <PropertyGroup>
    <Configuration Condition=" '$(Configuration)' == '' ">Debug</Configuration>
    <Platform Condition=" '$(Platform)' == '' ">x86</Platform>
    ....
    <ShortcutGenericAnalysis>true</ShortcutGenericAnalysis>
  </PropertyGroup>

Has this been resolved by now?


Clearify cbuild schema check

Describe the bug
When cbuild is called with  option --schema, on Windows the xmllint tool is required. Do we really need to check the XML input files?  I believe the schema check should be limited to the YML input files.
Could you clearify what schema check is really enabled by this command?  Is the csolution schema check disabled when this option is not used (csolution called with --no-check-schema)?

Currently YML and XML  schema checks are unified, it's enabled or disabled for both (disabled by default).
When the build runs with yml as input, there is no need to check the XML (cprj) schema.
Only when the command is buildcprj and therefore the input file must be a *.cprj file (cbuild buildcprj myproject.cprj) the --schema flag should invoke the xmllint tool.


Princess Team Info

Princess, please add your information to the meet the team section in the ReadMe.md file

I did it already!
Thanks


Fix error: 'XCHammer.xcodeproj/' does not exist

The swift package generate-xcodeproj command runs in parallel with the eval command which makes eval command fails when runs make workspace in the first time after cloning the repo.
You can reproduce the error by removing the Xcode project and then run make workspace.

Thanks for the contribution!
Interesting, this means the “swift package generate-xcodeproj” command is asynchronous? I would expect this to finish before the eval.
I’ll try to repro but the PR looks a straightforward fix!

I think it's because xcodebuild was executed in a sub-shell.

@thii - Just confirmed it myself! Thanks for the contribution and let us know if you run into any other issues or questions


BSModalVisible does not exist

Hi,
I receive this error when open SimpleTest\Project3\Unit2.

Hi, raelb.
I use it when I need to show an example for an issue.
The exception you get if because that property recently change its name.
Regards


✨ Add line numbers for insecure downloads

We don't have support for lines for secure download yet. This PR adds support. I happened to have time to look into it...
no breaking changes.
closes https://github.com/ossf/scorecard/issues/710
closes https://github.com/ossf/scorecard/issues/725
closes https://github.com/ossf/scorecard/issues/701

I have addressed all comments from @chrismcgehee . PTAL

friendly ping

tests are passing. can someone take and look and LGTM?


Nothing shows up on iOS

Hi,
This is my code
document.addEventListener("deviceready", function() {
  ActivityIndicator.show("Galeri yükleniyor, lütfen bekleyin");
  cordova.plugins.photoLibrary.requestAuthorization(function() {
    readLibrary();
  },function(err) {
    console.log(err);
  });
});
function readLibrary() {
  var st=new Date().getTime();
  cordova.plugins.photoLibrary.getLibrary(function(res){
    var et=new Date().getTime();
    var diff=et-st;
    alert(diff);
    ActivityIndicator.hide();
    var library=res.library;
    library.forEach(function(libraryItem){
      var st=new Date().getTime();
      image = "

";
      $(image).appendTo($("#images"));
    });
      var et=new Date().getTime();
      var diff=et-st;
      alert(diff);
  });
}

On iOS, I get alert() but never the images. Also, the last diff reports 11000~ ms (again on iOS)

Hey @kartagis - try logging out the error for getLibrary. Change your code to the following:
document.addEventListener("deviceready", function() {
  ActivityIndicator.show("Galeri yükleniyor, lütfen bekleyin");
  cordova.plugins.photoLibrary.requestAuthorization(function() {
    readLibrary();
  },function(err) {
    console.log(err);
  });
});
function readLibrary() {
  var st=new Date().getTime();
  cordova.plugins.photoLibrary.getLibrary(function(res){
    var et=new Date().getTime();
    var diff=et-st;
    alert(diff);
    ActivityIndicator.hide();
    var library=res.library;
    library.forEach(function(libraryItem){
      var st=new Date().getTime();
      image = "

";
      $(image).appendTo($("#images"));
    });
      var et=new Date().getTime();
      var diff=et-st;
      alert(diff);
  }, function(error) { // Add the error handler
    console.error(error);
  });
}

@viskin @CookieCookson when I add alert(JSON.stringify(res)) after reading the library, I get a blank and long dialog box on iOS.

I tried some basic things by doing a forEach on library and console.log(libraryItem.thumbnailURL) which works, but I can't figure out for the life of me why adding images to DOM doesn't work.

Okay, my bad. I had an extra semicolon at the end of CSP. I'm very sorry for that. But now I have another problem. I get only one image, and the others show up in DOM as 0 x 0 pixels.

Okay, here are the latest situation and my two cents: This plugin is very unstable for iOS. I stopped and ran it several times. The first time after I ran it again, it took 55 seconds to add all the images to DOM. Then I opened up Safari console and hit Cmd+R, it took 10 seconds. I stopped and ran again, only one image displayed (the others being 0x0 pixels). The way it adds the images to DOM is also very different from Android. Images are added to DOM, and shown once the process is done. As a result, it seems I can't use it for iOS until it is completely fixed. Sorry for that. Please get in touch and I'll provide all the evidence to support this.

Thanks, I will see what can be done

Anything? I really like this plugin, and don't want to drop it.

I tried today to make iOS more reliable, without success.
It seems like with iOS 10 it became worse.
When I display small amount of images (say 10), everything works. Sadly, with many images they just disappear. I just cannot attach debugger in such case.
It should be some memory pressure issue. The solution can be implementing limiting worker queue, so the plugin will never fail when great amount of requests for images arrive.
I planned to solve this issue with ionic's virtual scroll component, but for unknown reason it just does not work.
Sadly, I currently have no time to fix this. Sorry. I just focused on something else now.


if iw dev wlan0 set monitor none fails, bubble up error to user, and do not start profiler.

If we have a problem with setting the device in monitor mode, we should stop and tell the user.
Here is an example trying to set wlan0 in monitor mode when the host is a Wi-Fi based CM4:
(venv) wlanpi@wlanpi-cm4-emmc:~/dev/wlanpi-profiler$ sudo ip link set wlan0 down
(venv) wlanpi@wlanpi-cm4-emmc:~/dev/wlanpi-profiler$ sudo iw dev wlan0 set monitor none
command failed: Operation not supported (-95)

Currently, we ignore the command failed error and the profiler processes are started. At this point, scapy errors out making it look like an error with scapy, when it is an interface error instead:
(venv) wlanpi@wlanpi-cm4-emmc:~/dev/wlanpi-profiler$ sudo venv/bin/python3 -m profiler --logging debug
2021-11-05 10:23:51,109 [DEBUG] start: profiler version 1.0.8
2021-11-05 10:23:51,109 [DEBUG] start: python platform version is 3.7.3
2021-11-05 10:23:51,110 [DEBUG] start: scapy version is 2.4.5
2021-11-05 10:23:51,119 [DEBUG] start: profiler.manager pid 14024
2021-11-05 10:23:51,122 [DEBUG] start: config {'GENERAL': {'frequency': '5180', 'ssid': 'WLAN Pi', 'interface': 'wlan0', 'ft_disabled': False, 'he_disabled': False, 'listen_only': False, 'hostname_ssid': False, 'files_path': '/var/www/html/profiler', 'channel': 36}}
2021-11-05 10:23:51,149 [DEBUG] interface: wlan0 claims to support ieee80211
2021-11-05 10:23:51,177 [DEBUG] interface: phys: [phy(phy_id='2', phy_name='wlan2', ifindex='5', addr='00:c0:ca:ad:cc:0f', type='monitor'), phy(phy_id='0', phy_name='wlan1', ifindex='4', addr='d8:f8:83:35:be:07', type='managed'), phy(phy_id='1', phy_name='wlan0', ifindex='3', addr='dc:a6:32:fe:a2:48', type='managed')]
2021-11-05 10:23:51,179 [DEBUG] interface: phy1 maps to wlan0
2021-11-05 10:23:51,179 [DEBUG] interface: frequency is set as 5180 and channel as 36
2021-11-05 10:23:51,232 [WARNING] interface: reg domain appears unset. consider setting it with 'iw reg set XX'
2021-11-05 10:23:51,232 [WARNING] interface: https://wireless.wiki.kernel.org/en/users/documentation/iw#updating_your_regulatory_domain
2021-11-05 10:23:51,232 [DEBUG] interface: mac: dc:a6:32:fe:a2:48, channel: 36, driver: brcmfmac, version: 7.45.229, firmware-version: 01-2dbd9d2e
2021-11-05 10:23:51,243 [DEBUG] interface: wpa_cli version is v2.9
2021-11-05 10:23:51,266 [DEBUG] interface: iw version 5.9
2021-11-05 10:23:51,266 [DEBUG] interface: ip utility, iproute2-ss190107
2021-11-05 10:23:51,277 [DEBUG] interface: run: ip link set wlan0 down
2021-11-05 10:23:51,802 [DEBUG] interface: run: iw dev wlan0 set monitor none
2021-11-05 10:23:51,814 [DEBUG] interface: run: ip link set wlan0 up
2021-11-05 10:23:51,833 [DEBUG] interface: run: iw wlan0 set channel 36 HT20
2021-11-05 10:23:51,845 [DEBUG] start: finish interface setup and staging ...

==========================================================================
Starting profiler AP with interface wlan0 on channel 36:
 - Our fake AP SSID: WLAN Pi
 - Results are saved locally and printed to screen

Instructions:
 - Associate your Wi-Fi client to our SSID: WLAN Pi
 - Enter any random password to connect

The client will fail authentication but should send an association request
==========================================================================

2021-11-05 10:23:51,846 [DEBUG] start: beacon process
2021-11-05 10:23:51,853 [DEBUG] start: sniffer process
2021-11-05 10:23:51,872 [DEBUG] start: profiler process
2021-11-05 10:23:51,876 [DEBUG] fakeap.py: beacon pid: 14047; parent pid: 14024
2021-11-05 10:23:51,885 [DEBUG] fakeap.py: sniffer pid: 14048; parent pid: 14024
2021-11-05 10:23:51,904 [DEBUG] profiler.py: profiler pid: 14049; parent pid: 14024
2021-11-05 10:23:51,941 [DEBUG] fakeap.py: <socket.socket fd=9, family=AddressFamily.AF_PACKET, type=SocketKind.SOCK_RAW, proto=768, laddr=('wlan0', 3, 0, 1, b'\xdc\xa62\xfe\xa2H')>
2021-11-05 10:23:51,941 [DEBUG] fakeap.py: <socket.socket fd=10, family=AddressFamily.AF_PACKET, type=SocketKind.SOCK_RAW, proto=768, laddr=('wlan0', 3, 0, 1, b'\xdc\xa62\xfe\xa2H')>
2021-11-05 10:23:51,995 [INFO] fakeap.py: starting beacon transmissions
Process sniffer:
Traceback (most recent call last):
  File "/usr/lib/python3.7/multiprocessing/process.py", line 297, in _bootstrap
    self.run()
  File "/usr/lib/python3.7/multiprocessing/process.py", line 99, in run
    self._target(*self._args, **self._kwargs)
  File "/home/wlanpi/dev/wlanpi-profiler/profiler/fakeap.py", line 360, in __init__
    filter=self.bpf_filter,
  File "/home/wlanpi/dev/wlanpi-profiler/venv/lib/python3.7/site-packages/scapy/sendrecv.py", line 1263, in sniff
    sniffer._run(*args, **kwargs)
  File "/home/wlanpi/dev/wlanpi-profiler/venv/lib/python3.7/site-packages/scapy/sendrecv.py", line 1128, in _run
    **karg)] = iface
  File "/home/wlanpi/dev/wlanpi-profiler/venv/lib/python3.7/site-packages/scapy/arch/linux.py", line 497, in __init__
    attach_filter(self.ins, filter, self.iface)
  File "/home/wlanpi/dev/wlanpi-profiler/venv/lib/python3.7/site-packages/scapy/arch/linux.py", line 166, in attach_filter
    bp = compile_filter(bpf_filter, iface)
  File "/home/wlanpi/dev/wlanpi-profiler/venv/lib/python3.7/site-packages/scapy/arch/common.py", line 157, in compile_filter
    "Failed to compile filter expression %s (%s)" % (filter_exp, ret)
scapy.error.Scapy_Exception: Failed to compile filter expression type mgt subtype probe-req or type mgt subtype auth or type mgt subtype assoc-req or type mgt subtype reassoc-req (-1)
2021-11-05 10:23:52,270 [DEBUG] start: <Process(sniffer, stopped[1])>
2021-11-05 10:23:52,278 [DEBUG] start: <Process(profiler, stopped[SIGKILL])>
2021-11-05 10:23:52,310 [DEBUG] start: <Process(txbeacons, stopped[SIGKILL])>

Added in 5c621d776ab792b66cc42fe3be4b5288cc3ec94a.
Example:
2021-11-11 15:44:05,571 [INFO] interface: run: iw phy1 interface add wlan1mon type monitor flags none
2021-11-11 15:44:05,594 [INFO] interface: command failed: Operation not supported (-95)
2021-11-11 15:44:05,595 [ERROR] start: problem interface staging ... exiting ...
Traceback (most recent call last):
  File "/home/wlanpi/dev/wlanpi-profiler/profiler/manager.py", line 176, in start
    __iface.stage_interface()
  File "/home/wlanpi/dev/wlanpi-profiler/profiler/interface.py", line 228, in stage_interface
    raise InterfaceError(f"{self.name} does not appear to support monitor mode")
profiler.interface.InterfaceError: wlan1 does not appear to support monitor mode
^CDetected SIGINT or Control-C ... Removing monitor interface ...
2021-11-11 15:44:33,261 [INFO] interface: run: ip link set wlan1mon down
2021-11-11 15:44:33,299 [INFO] interface: run: iw dev wlan1mon del
2021-11-11 15:44:33,323 [INFO] interface: run: ip link set wlan1 up
Exiting ...


[NEXT] mergeLocaleMessages throws  Hydration text content mismatch warning

Version
@nuxtjs/i18n: 8.0.0-alpha.3
nuxt: 3.0.0-rc.11
Nuxt configuration
Please change to [x] if relevant for this issue:

[ ] Applies to a site deployed to a static server (site generated with nuxt generate)
[x] Applies to a site deployed to a server with a Node backend

@nuxtjs/i18n configuration
i18n: {
  locales: [
    { code: 'jp', file: 'jp.json' },
    { code: 'en', file: 'en.json' },
  ],
  lazy: true,
  langDir: 'locale/',
  defaultLocale: 'jp',
  strategy: 'prefix',
  detectBrowserLanguage: {
    useCookie: true,
    cookieKey: 'i18n_redirected',
    redirectOn: 'root',
  },
  vueI18n: {
    legacy: false,
  },
},

Reproduction Link
https://stackblitz.com/edit/github-ih5gur-6jrlvp?file=nuxt.config.ts
Steps to reproduce
npm run dev

Then modify the url in Preview container to https://github-ih5gur-6jrlvp--3000.local.webcontainer.io/jp
What is Expected?
The app should be generated without errors
What is actually happening?
[Vue warn]: Hydration text content mismatch in <h1>:
- Client: こんにちは
- Server: welcome

Thank you for your reporting!
and, sorry, my late reply. 🙇♂
Unfortunately, I could not reproduce your reporting repo.
Could you give us fully reproduction codes please? 🙏

Thank you for your replying.
This are the least config codes that could reproduce.
Configs
package.json
{
  "scripts": {
    "dev": "nuxt dev",
  },
  "devDependencies": {
    "nuxt": "3.0.0-rc.11"
  },
  "dependencies": {
    "@nuxtjs/i18n": "8.0.0-alpha.3"
  }
}

nuxt.config.ts
export default defineNuxtConfig({
  modules: ['@nuxtjs/i18n'],
  i18n: {
    locales: [
      { code: 'jp', file: 'jp.json' },
      { code: 'en', file: 'en.json' },
    ],
    lazy: true,
    langDir: 'locale/',
    defaultLocale: 'jp',
    strategy: 'prefix',
    detectBrowserLanguage: {
      useCookie: true,
      cookieKey: 'i18n_redirected',
      redirectOn: 'root',
    },
    vueI18n: {
      legacy: false,
    },
  },
});

app.vue
<script lang="ts" setup>
const { mergeLocaleMessage, locale } = useI18n();

// we use this to simulate fetching translation from the other api on server side, then merge back to locale messages
onServerPrefetch(async () => {
  const translation = await new Promise((resolve) => {
    resolve({ welcome: 'こんにちは' });
  });
  mergeLocaleMessage(locale.value, translation);
});
</script>

<template>
  <h1>{{ $t('welcome') }}</h1>
</template>

Reproduce

Run npm run dev
Then replace the url with /jp postfix. It may be like http://localhost:3000/jp. (Since we use localePath)
The h1 text displaying is correct at fisrt, but immediately change to untranslated text. And the browser devtool console the errors above.

Sorry, my lately response.
In docs says, onServerPrefetch is SSR only
https://vuejs.org/api/composition-api-lifecycle.html#onserverprefetch
You need to use onMount to manage i18n resources with mergeLocaleMessages with server-side and client-side awareness as follows
So this issue is not a bug in @nuxtjs/i18n (vue-i18n).
Thanks!

Thank you for your replying. This is very helpful and inspired me!
For some SEO reasons, we decide to fetch translations from api on server side.
So we need to keep the translation value. And here is our solution.
<script lang="ts" setup>
const { mergeLocaleMessage, locale } = useI18n();
const translation = useState('translation-key');

onServerPrefetch(async () => {
  translation.value = await new Promise((resolve) => {
    resolve({ welcome: 'こんにちは' });
  });
  mergeLocaleMessage(locale.value, translation.value);
});

onMounted(() => {
  mergeLocaleMessage(locale.value, translation.value);
});
</script>

Although call mergeLocaleMessage twice both on server and client side seems a little weird and still have the console warnings.😅
But it works as we expected, hope it helps others.
Thank you again!


Added reconfigure route and made use of it in the instance class

Added /reconfigure route for instances which triggers instance reconfiguration with current parameters, and used it in Instance.reconfigure() method when no parameters are supplied. We don't use reconfigure() without parameters in our code base yet, so this should not break anything.

@vasichkin please review

Ok to merge after fixed comments.

Ok to merge


Make dogehouse invite only

It can't be a clone of Clubhouse unless it is invite-only!
Contributors could get in without an invite, which would encourage contributions!

Dogehouse wants to be better than clubhouse

Not a clone

and invite only is really annoying

Agreed, DogeHouse has been open to all from day 1

Lol, wanted to see opinions!

I dont see the point in this, thank you for your suggestion


Use no-check exceptions instead of statically checked

With the exceptions used in the program deriving from CX_STATIC_CHECK, my coding becomes quite complicated:
    if read_result->is_ok( ).
      try.
        queue_lines = read_result->get_value( )->*.
      catch zcx_result_is_not_ok.
        raise exception zcx_fatal...
      endtry.

since I already do a check if the result is OK, it seems quite unnecessary to be forced by the static checked exception to do another check with TRY/CATCH. As far as I understood, the goal is to reduce TRY/CATCH - noise in the coding. Wouldn't it be better to turn the exceptions into CX_NO_CHECK types?

Hi Jörg. Thank you for your feedback. I think you are right. I'm currently on parental leave. if you want, please submit a PR or I'll change this when I have access to a SAP system again. (~1 month)

should be fixed, please check @jrgkraus

This unfortunately doesn't work for older releases. We are on 7.50 and I get now an error: The class "ZCX_RESULT_IS_NOT_FAILURE" was not derived from either "CX_STATIC_CHECK" or "CX_DYNAMIC_CHECK".
Thanks for your work!


Improves the deserialization speed by reducing object initialization overhead

Description:
Publish the PR for feedback but will keep it in draft mode until we are confident to merge this change. More changes and tests come soon.
This PR improved the library's performance by 40-50% through:
(1) migrating to new ionPyObjects without inheritance  (+~20%),
(2) switching the way of object initialization from from_value() to constructor directly (+~10%), and
(3) optimizing Ion struct's underlying storage implementation (+~20%)
Unit tests
This PR also fixed more than 2k unit test failures due to the changes of IonPyObjects. Most of failures are because of (1) ion_nature inheritance, (2) from_value invocation, and (3) methods signature differences.
Detail:
In detail, create an IonPyObject requires a few steps which causes a lot of overhead. The overheads mainly come from (1) Ion_Nature inheritance, (2) from_value() method invocation overhead and (3) methods refactor overhead. In addition, Ion Struct, the Ion multiMap, produces a lot of overhead due to use of high level abstract classes such as OrderedDict, mutableMapping and MutableSequence. In the Ion Python C extension, we always create an empty Ion Struct first, and recursively call its "add_item" API to add new values. Every time inserting a value into the Struct, the code will check if this is the first value occurs for this key. If it's the first value for the key, it will create a new MultiSequence for the key, otherwise it will just insert the value into the list. Creating new mutableValues is extremely expensive when deserializing large Ion files.
So this PR first removed Ion_Nature inheritance and the reusable help functions for creating IonPyObject classes.
Then, all new IonPyObjects has its own constructor. The PR next switched the way of IonPyObjects initialization from from_value() to their constructors directly to avoid unnecessary method invocation overhead.
Last, the PR optimized the multiMap implementation by using native list instead of MutableSequence. In addition, since the Ion Struct is an unordered collection, we uses a Python dict directly instead of previous OrderedDict.
Benchmarking
The performance gain depends on various factors such as data model and size. Below are three sample data I used for benchmarking:

File_name
Description
Size (bytes)
Before (s)
After(s)
Improvement

committed.10n
a small Ion struct
148
0.19
0.13
31.6%

complex.10n
an Ion struct with decimal values
5208
14.1
7.8
46.9%

legacy.10n
complex nested Ion containers
22.3e^6
245
127
48%

Next Step

There are more experiment and tests required, so we don't break any existing use case
All above are focusing on the read(deserialization) process, try write process as well.
more IonPyObjects migration for rest types such as timestamp and bytes.
Using the benchmark-cli to generate an official benchmark result.

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

High Level Overview

Consider the use of __slots__ versus the use of __dicts__ or __weakref__. We can further reduce memory allocations by not creating a dictionary for attributes where possible. If a dictionary is explicitly required it can be declared within the slot.
Avoid unnecessary redundancy. Consider wrapping the type classes in module in a base class and moving similar methods. This will mean the use of polymorphism and using some vtable lookups. It might be useful to see what the penalty for DRY might be in this context.
Consider alternative data structures. For example, __setitem__(self, key, value) wraps the value in a list. If only the value is needed, then consider assigning it directly instead.
Avoid unnecessary conditional checks. Checking a conditional to set a value to None may be suboptimal.
I'm not keen on the foo_new() naming convention. Consider alternative naming conventions that adhere to the PEP 8 standard on class naming.

At a high-level I think we need to commit to dropping some of the existing factory methods. I know it's arguably part of our API, so we'd need to rev at least minor, but probably major version, but I think it's worth it.

Thanks for doing the work to get to this place! I'm super excited to see the benefits of this! Let's not be afraid to be bold in breaking API compatibility to do so.

Okay so,

the next commit will address above comments.
As for the scope and goal of this PR, I'm going to make break changes and completely deprecate old methods such as from_value. This requires a little more work, I can create a separate PR targeting this one as needed.

A new commit is available below.
An overall comment: This PR incorporated the performance improvement into the code with the least changes so it won't break any clients use case. Some methods are copied from the original IonPyObject but it's time to refactor/deprecate them! Also Added some documents.

The clean up commit did below:

Deprecated ion_nature
Migrated objects construction from from_value() to their constructors in C extension
Added IonPytimestamp and IonPyBytes
Replaced all the original IonPyObject and factory methods with the new IonPyObjects.
Worked on the IonPyTimestamp signature to make it compatible with the amazon.core.Timestamp
Updated the unit tests to accommodate the changes.


added configuration to be able to consolidate only with crossref doi.

Hello
When using consolidation and the doi is not present in the document, we proceed with fuzzy matching, the problem is that while this should be very helpful in most cases, there are some cases where it gives some understandable results (from the point of view of general users), because it depends first on the quality of the metadata extraction and especially on the consolidation service..
I've been running the crossref apis benchmark, and it seems that the matching scores have decreased, so I think we should be able to deactivate the consolidation when we don't the doi in this case to make sure users are not going to be surprised, here are some exemples :
For this one , the original title is in french, in the resulting TEI , the title is in english and we have doi added and journal..
exemple1.pdf
This one, it is obvious that the result has nothing to do with the pdf content:
exemple3.pdf

Coverage decreased (-0.006%) to 38.952% when pulling<PHONE_NUMBER>b5432d44233e540007a8c9e2a75adf on deactivate-fuzzy-matching-config into deaed0c1592adb0f53e0858f69a29523f18f46af on master.


Cannot Change Group/Scene More Than Once

Description:
When trying to change a participant's group and thus the controls and scene being displayed to them, I run into problems.  I am able to successfully change the Group and Scene the first time, however the second and all subsequent times do not appear to update the changes.
Desired Behavior:
When a user connects to the stream, they are automatically placed inside of the default scene.  My default scene has several buttons which will change the Group and Scene of the user and display new controls.  When any Group option button is selected, the new controls will appear.  This part works fine.
Once in the new scene, there is a control button to return you to the default Group/Scene.  This is the part that is broken.
Additional Info:
When running through and trying to debug my code, I've found out that the participant's group and scene information is being updated.  What is not being updated is what Scene and controls the user actually sees on the Beam page.
My code is an absolute mess right now trying to get to the bottom of this, but I can provide a clean example if necessary.
Please reply if you've had a similar problem or may know how to help fix the issue, thanks!

Thank you for filing a bug. This is the same root cause as the button cooldown issue. We're investigating and will let you know when we have a fix.

I am having similar issues, can't switch scenes for people viewing on Beam. Thought I was going a bit crazy.

You're not crazy :) We apologize for the bug.

No need to apologize, it's in beta after all xD Just nice to let our minds at ease!

We've fixed this issue with the latest release of the plugin, available here:
https://github.com/WatchBeam/interactive-unity-plugin/releases/tag/v0.7.2-beta

Thank you so much for the quick fix!


Fixed words and made some small adjustments

Nice work - I'm going to close this PR so others can complete the assignment.
Side note: it is often useful to provide a bit of a description of the changes in the commit message (i.e., what updates were made to the README.md)


SSL Certficates are generate incorrectly with large duration/expiry

Due to bug: https://www.pivotaltracker.com/n/projects/956238/stories/168066776
fixed: https://github.com/cloudfoundry/config-server/commit/c9ac3617cb97ade27d27a2527f9695e2e0f7a4f2#diff-522e1607e2a1cf9b59079a10760c6b99
Now max working duration is: 1365 (max int16/24).

I'm assuming this is essentially a duplicate of the bug I filed and then fixed. Closing this. Thanks for pointing it out as well :)


Search UI: Deactivate elements that are not of use (and other things to help the user)

There are currently many elements in the search panel and I think it is easy to get lost, especially for new users. A very simple way to make the interface more intuitive would be to dynamically deactivate elements when they are not usable, so that the user knows where to look. By deactivating, I mean giving a greyed out look and having no usability, such as with some of the buttons today. Also, add warnings/errors before the user does something that will not work.
Suggested logic:

[ ] By default, only the left-hand side browser (with filtering options) should be active
[ ] The "add to query" button should only be active If a value is selected in the left-hand browser
[ ] All elements in the middle column should be inactive if there is no text in the search query text bow, including the headers. The only exception to this rule is that the text box itself should always be active so that it is possible to paste (or write) text in it. Deactivating the header ("Search query") should be enough to bring focus to the left-hand model browser, where the action is.
[ ] Make sure that clicking "Clear all" deactivates all the element in the middle column (except the text box), a consequence of the above rule.
[ ] The result  browser and export buttons should only be active if there are results shown
[ ] If the search query is changed after a search has been done, I think the results box should have a warning (like if you type letters in the "Limit" box) with a tooltip saying that "the search query has changed since these results were generated, and a new search might give different results", or something to that effect. This so that "clear all", or changing the search query do not remove the results (as it might have been a mistake), while it is clear when those two do not add up.
[ ] "Filter values" and "Limit" should have a checkbox to the left of the titles, showing whether the functionality is active or not. This is in order to make the user aware of the choices. For "Limit", the default is 10, which I think is useful, but only if the user is aware of that. If not, the user might think he always gets 10 results. A checkbox draws just enough attention to the box, I think.
[ ] If such a checkbox is unchecked, the text box should be deactivated. To activate and provide a value is then two clicks. I think this is a very small price to pay for a more intuitive interface
[ ] The checkboxes  should not have any dynamics depending on the contents of the text box. Meaning that it is perfectly possible to end up with a checkbox and an empty field. However, such a case should give a warning, similar to the one that appears when writing letters in the "Limit" box
[ ] The "Categories" selection should also have such a checkbox to the left of the title (inside the title box, I think that will work visually). By default this is unchecked, and the category selection list is deactivated. Checking it obviously activates the category selections.
[ ] If the "Categories" checkbox is checked, but no categories selected, there should be a warning, and the search button should be greyed, as when adding a letter to the "Limit" box.
[ ] I am unsure whether all categories should be checked by default or not. I think perhaps the best choice would actually be that only the FAIRtracks categories are checked, so that the same solution could be used as a quick way to only return those (i.e. by just clicking the header checkbox, without having to select each one manually). If the default is no categories, then the warning from the above point will appear when activating the category list. If the default is all categories, one would typically need to click a lot to get the (I think) most common choice, which is just a single category. Only the FAIRtracks categories is at least i better.
[ ] Even though this is for advanced users, there should be some logic checking the references. If not all categories are included in the same reference "clique", I think the "categories" checkbox should be checked by default. Furthermore, if the set of selected categories (by default FAIRtracks) are still not in the same clique, there should be a warning that disappears only when a correct set of categories are selected. In this way, if only the FAIRtracks categories are linked, but not the rest (a likely scenario), the default choice would work.
[ ] An additional check for the categories box is that if the search query contains attributes from a category that is not selected, there should be a warning in the category selection box (with specific info on which categories are missing). Currently, there will be no results, but no indication on what is wrong.

Some text improvements:

[x] "Model browser" is not a good title. What about "Select metadata value(s)"?
[x] "Categories" -> "Filter categories to search in"
[x] "Limit" -> "Max. number of results"
[x] "Clear all" -> "Clear search query"
[x] "Export (XX) entries as..." -> "Export results (10 items) as..."
[x] Improve the naming of the result files, for instance TF-Blueprint-2020-01-25T23-49-09Z-152x.gsuite. Format: TF-(hub)-(date in iso-8601)-(#results)x.(suffix)

Some text improvements:

All items from this section are fixed in ac290fb.
However, regarding the first list, I think we should have a meeting, because I don't completely agree with all the items there (as well as with some items in #111, #112 and #113). Might be worth inviting some potential users (Geir Kjetil? Ankush?).

Sure, I did not expect all of these to be accepted without process. In my experience, users without ui design experience are often not very good at the idea stage but more useful when one can observe their use patterns and receive complaints. @sandve would be a natural choice for such a meeting, though, if he has time.

I could provide some input. I would have preferred to test it out in a real analytical scenario, though, as I am a bit skeptical to hypothetical use cases (in my experience, what becomes natural/intuitive when I have real use case may be quite different from what appears reasonable when I just try out a tool without a purpose). Will you be presenting TrackFind e.g. at a Tuesday meeting soon? It could perhaps be tried out for the hGSuite analysis that Sumana and Diana are currently doing or something..

Will you be presenting TrackFind e.g. at a Tuesday meeting soon?

We could, I guess... However, we will have to finalize a couple of things here and there first.


rename requirements-txt and requirements-txt-dev

Rename requirements-txt and requirements-txt-dev to requirements-file and requirements-file-dev
PR Checklist

[x] A description of the changes is added to the description of this PR.
[x] If there is a related issue, make sure it is linked to this PR.
[x] If you've fixed a bug or added code that should be tested, add tests!
[x] Documentation in docs is updated

Description of changes
This PR proposes to rename requirements-txt and requirements-txt-dev to requirements-files and requirements-files-dev. If users still use the old parameters, they will be shown a deprecation warning:

Warning: In an upcoming release, support for the --requirements-txt command-line option and the requirements_txt configuration parameter will be discontinued. Instead, use --requirements-files or requirements_files under the [tool.deptry] section in pyproject.toml.

A few nit comments, but this mostly looks good to me. Just wondering if we should not name the parameters requirements-files and requirements-files-dev since they are lists.

Good idea. I was doubting about this a bit, since deptry . --requirements-files requirements.txt feels less intuitive than `deptry . --requirements-file requirements.txt, and most projects likely have a single requirements file. But on the other hand, it is correct since it accepts a list. So let's change it to plural.


acme-corp: jre-fips

this is an issue created to track an image that already exists.  please ignore.

this image is already built.


docs: Forbidden Image on Tab Bar Controller Page

This isn't directly related to ionic, but on you site at this page http://ionicframework.com/docs/angularjs/controllers/tab-bar/
I am getting this error causing a broken image on the page
Failed to load resource: the server responded with a status of 403 (Forbidden) http://ionicframework.com.s3.amazonaws.com/docs/controllers/ion-tabs.gif

Greetings @samrose3!
My sensors indicate that you need to update your issue through our custom issue form. We are now requiring all issues to be submitted this way, to ensure that we have all of the information necessary to fix them as quickly as possible.
Click Here To Update Your Issue
I will have no choice but to close this issue if it is not resubmitted through the form. Please fill out the rest of the form, so that I may use my friendly robot powers to assist you.
Thank you!

It might sound naive, but how did you get to this page? I didn't find link to it from the documentation.

@thiagofelix ah thank you! good deal :+1:

Sorry, didn't mean to close this issue.

The link in reference is to an old version of the site.
http://ionicframework.com/docs/api/directive/ionTabs/


Move functions from psa_crypto_helpers.h to a .c file

tests/include/test/psa_crypto_helpers.h contains function definitions. That's a remnant from when we didn't have tests/src/*.c and unit tests had to be compiled as a monolithic C file. It's a problem because merely including the header causes an unused-function warning. The header should contain only function declarations (or static inline function definitions, but the functions in that header don't need to be inline).

Hi, Can I work on it? @gilles-peskine-arm

Hi @YanivBir , yes, if you're willing, we'd be grateful. No one is working on this at the moment.

Ok, I'm working on it


Update package.json

I think this should actually be 1.8.10, right?

http://semver.org/

https://github.com/managedbyq/juice/releases?after=1.8.0
We are not using that I think.

I think all npm packages are supposed to use it.
https://docs.npmjs.com/getting-started/semantic-versioning

Yeah that makes sense.


ZXing-cpp merges nearby 1D barcodes into one if they have exact same content

Tested on v1.3.0.
I don't have any samples that I can share at the current moment. Sorry. But I suspect this is related to de-duplication of 1D barcodes.

But I suspect this is related to de-duplication of 1D barcodes.

Yes it is. There is a simple heuristic used to de-duplicate the symbols. The parameters could be tuned if something slightly more accurate could be found but there are always going to be counter examples that break the heuristic.

The parameters could be tuned if something slightly more accurate could be found but there are always going to be counter examples that break the heuristic.

Yes, but I believe some cases can be avoided.

Results with ZXing-cpp (e0b55812e8403d591339f89ddffbd16db40fcb3d)

I don't know enough details about this heuristic method, but I feel this case can be handled relatively easily.


Issue with Select component

I get the following error when i try to use the Select component:
TypeError: Cannot read property 'name' of undefined
Index.render
node_modules/react-inputs-validation/lib/components/Select.js:384
  381 |   _react2.default.createElement(
  382 |     'div',
  383 |     { className: STYLES['ellipsis'], style: { width: '100%' } },
> 384 |     item.name
  385 |   ),
  386 |   _react2.default.createElement('div', { className: dropdownIconClass })
  387 | );

Here is my code

          Beneficiary Bank Account Location
              <Select
                  name="location"
                  className="form-control"
                  value={this.state.location}
                  onChange={e => this.handleChange(e)}
                  disabled={this.state.isEditForm}
                  optionList={[
                     {id: "1", name: "Home Country"},
                     {id: "2", name: "Host Country"},
                     {id: "3", name: "Other Country"},
                  ]}
              />

what is your this.state.location ?
this usually means that the value you are providing is not any of optionLists id

It worked now! Thanks for a prompt reply 😄


Remove patches from dotnet/standard

Remove all these https://github.com/dotnet/source-build/tree/dev/release/2.0/patches/standard

Fixed by https://github.com/dotnet/source-build/pull/182


document ploomber-cloud examples

we're missing docs on https://github.com/ploomber/cli/commit/3fbd9c1533ab881676658a73f85460286d0b986c
let's add them and prepare posts for socials

just released the new version of the CLI 0.1.13, should be up on PyPI in a few minutes


Quick Review

Hello there, thanks for your interest in Nim!
A quick review of your code:
Padding
Instead of padding with arrays
https://github.com/elijahr/lockfreequeues/blob/9f5bc3059cb1f9e40385df14c5d5b2a54f557cbd/src/lockfreequeues/spsc/queueinterface.nim#L25-L30
you can do
type
  SPSCQueueInterface* = object
    head* {.align: CACHELINE_BYTES.}: Atomic[uint]
    tail* {.align: CACHELINE_BYTES.}: Atomic[uint]

I recommend that you pad the start of the data structure as well since queues are often used in an arrays of queues:
https://github.com/elijahr/lockfreequeues/blob/9f5bc3059cb1f9e40385df14c5d5b2a54f557cbd/src/lockfreequeues/spsc/sharedqueue.nim#L20-L26
Sizing
Currently your buffer requires being a power of 2, this wastes memory for large buffers
Instead, to distinguish between the empty and the full queue, you can have your capacity run from 0 ..< 2*RealCapacity
See https://github.com/mratsim/weave/blob/master/weave/cross_thread_com/channels_spsc.md#litterature

http://www.vitorian.com/x1/archives/370
p3: http://www.sunburst-design.com/papers/CummingsSNUG2002SJ_FIFO1.pdf

And implementation in Nim for a serial queue: https://github.com/mratsim/weave/blob/2e528bd2/weave/datatypes/bounded_queues.nim
Misc
You don't need to return a seq here, you can return an array, it will be copied
https://github.com/elijahr/lockfreequeues/blob/9f5bc3059cb1f9e40385df14c5d5b2a54f557cbd/src/lockfreequeues/spsc/staticqueue.nim#L89-L102
Though the state might have torn copies.

@mratsim Thank you so much for the welcome and review! I've been enjoying learning Nim over the last few months.
I'll address this feedback and put together a new release sometime in the next few days.
I am not familiar with the term "torn copies" - by that do you mean that state() will potentially return invalid state due to the lack of a mutex around head/tail/storage operations? If so - I think that's okay, as the state() proc is primarily used to validate the queue state after operations in the single-threaded unit tests. I should add a comment on the proc to clarify that. Or perhaps the proc should instead reside in the unit tests.
Also, I appreciated your talk at NimConf on multithreading. I didn't know about -fsanitize=thread, very useful. Cheers!

@mratsim if you'd like to take a look, I think I've addressed your feedback via #3. Cheers!


feat(tpcds-benchmarking): Add basic tpcds benchmarking for local testing

Usage
python -m benchmarking.tpcds \
  --scale-factor 1
  --tpcds-gen-folder data/tpcds

The --scale-factor and --tpcds-gen-folder variables are not needed (default to 0.01 and data/tpcds, respectively).

Usage
python -m benchmarking.tpcds \
  --scale-factor 1
  --tpcds-gen-folder data/tpcds

The --scale-factor and --tpcds-gen-folder variables are not needed (default to 0.01 and data/tpcds, respectively).

not necessary, but IMO it'd be nice to add this to the Makefile similar to our dsdgen make command:  make dsdgen SCALE_FACTOR=1 OUTPUT_DIR=data/tpcds
something like:
make bench_tpcds SF=1 TPCDS_DIR=data/tpcds


Add ability to view/set devfile variables via subcommand like: odo config view/set

/kind user-story
User Story
As a developer working with odo, I want to:

view "variables" defined by the 2.1 spec and their values
set the variable values

using something simple like odo config set/view or odo env set/view.
Related Thoughts

Should there be a different command for adding new (not yet in devfile) vs. setting existing (i.e. already appearing in the devfile) variables?

Should odo env set DebugPort be phased out and replaced with a stack variable?   (In this interpretation, if it appears in the devfile using a substitution, then it should be in the form of a devfile variable).

Should we use this as an opportunity to do any further revisiting of odo config vs. odo env ?   (E.g. as suggested in this Slack thread.  E.g. odo config-devfile vs. odo config-odo, etc.?   (That's sounding like it'd turn this into a whole new epic).

Reminder:  stack variables can only substitute in string type fields. Furthermore, quoting the spec

Replacement cannot be used for:     schemaVersion, metadata, parent source - element identifiers, e.g. command id, component name, endpoint name, project name - references to identifiers, e.g. in events, a command's component, container's volume mount name - string enums, e.g. command group kind, endpoint exposure

Acceptance Criteria

[ ] It should provide a get/view command
[ ] It should provide a set command

Links

Related Epic (mandatory):   Not an epic per se, but could be grouped within: https://github.com/openshift/odo/issues/4589

/kind user-story

/lifecycle frozen
This is still a valid proposal, and once we have some bandwidth we should look into this.


[Go Live][June 13]

Push to live

Docs Build status updates of commit 8a7ea05:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

msteams-platform/bots/how-to/create-a-bot-commands-menu.md
:white_check_mark:Succeeded
View

msteams-platform/concepts/device-capabilities/location-capability.md
:white_check_mark:Succeeded
View

msteams-platform/resources/moodleInstructions.md
:white_check_mark:Succeeded
View

msteams-platform/resources/troubleshoot.md
:white_check_mark:Succeeded
View

msteams-platform/tabs/how-to/create-tab-pages/configuration-page.md
:white_check_mark:Succeeded
View

msteams-platform/tabs/tabs-link-unfurling.md
:white_check_mark:Succeeded
View

msteams-platform/whats-new.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


Larger downloads over H2 stalls with #2881

On Docs (with #2881 enabled, currently it is not) a command like this will hang every time, at exactly 55.8% (which is really, really close to 1MB).
curl  -D - -o /dev/null  https://ci.trafficserver.apache.org/bintray/clang-format-20170404.tar.bz2

This is with a "curl" client that supports H2. I tracked this down to the commit from #2881, which unfortunately is also back ported to 7.1.3. Reverting this change on master seems to fix the issue.
The remap rule for the above is
map https://ci.trafficserver.apache.org/bintray https://bintray.com/artifact/download/apache/trafficserver

I tried writing a test for this, but unfortunately, I cannot reproduce the stall with ATS and my client running on the same box.  Give the nature of #2881 I assume that it is triggered by larger times between client and ATS.  For larger files there will be pauses as the H2 windows are exhausted.
I am currently downloading a ISO file through ATS with #2881 enabled and it is chunking along.  800MB downloaded so far.  But again my client and ATS are on the same RHEL7 box.
I'll go ahead and get the h2big test expanded, but probably not tonight.  I need to play with the timeouts, and I'm seeing something odd with a next_no_activity_timeout being set to 0.  I'll open another issue for that if it is something real.  I think it is separate from this issue.
In the short term, it would probably be best to revert #2881 at least from the 7.1.x branch until we figure out what is going on.

I tried master and 7.1.x branch with #2881, but I couldn't reproduce this issue either. I'm missing something to reproduce :/
FWIW, what I tried is below.

With origin server which has static files (1MB, 10MB, and 100MB), I could download all of them through ATS regardless cache hit or miss.
With the remap rule of downloading clang-format from bintray.com, I didn't see the stall and the check sum of the file was correct.

We can try again on Docs, maybe there's something else intervening together with that patch.. Note that it's only with H2 requests this happens.

I've restored the "broken" behavior on Docs / Ci again, e.g. this hangs at 55% always (with a curl that supports H2):
curl  -D - -o /dev/null  https://ci.trafficserver.apache.org/bintray/clang-format-20170404.tar.bz2

Seems it's related to priorities, I did some tracing, and it seems strange around the priorities. I turned off the H2 priorities feature (in records.config), and then it works again.

As far as I understand, the scenario of this stall is below.

H2 priorities feature is enabled
When write_vio of Http2Stream is filled (write_vio.get_writer()->write_avail() is zero) and Http2Stream::send_response_body() is called from Http2Stream::update_write_request(), send_response_body() schedule  HTTP2_SESSION_EVENT_XMIT event. This means the write_vio is not consumed immediately.
After Http2Stream::send_response_body() is called, VC_EVENT_WRITE_READY event is send to write_vio._cont (HttpTunnel)
HttpTunnel doesn't read anything from origin server side, because write_vio is not consumed.

Below patch mitigate this issue and proof the above scenario by sending VC_EVENT_WRITE_READY event when write_vio is consumed.
https://paste.apache.org/fQsT
But I'm not confident that adding this code in Http2Stream::update_sent_count() is suitable. Let me think more.


Improve Email verification template

The current email verification template (src/main/resources/templates/email/verify-email.html) is a bare-bones HTML file. Apply some styling to look better.

@sivaprasadreddy I added minimal styling to email template. Let me know if you have any other thoughts.


Action not registered on element placed after nested component

Actions are not registered for elements in a component if they are placed after a nested component. Can reproduce it in the example app's nested table by putting the load table button after the table.

Fix is merged into master, so closing this.


Functionnality request: Multiple verilog files given as benchmark

It was requested by the University of Washington.
Pb: Most benchmarks or functional verilogs are actually projects with different files. Accepting multiple files would be a good step forward (also check if it is possible to use wild cards (*) or glob / similar function.
Comment: Need to check if it is possible to do under Yosys. This is going to be our limitations.

@BaudouinChauviere We have used Multi-file flows with Yosys and VPR in SymbiFlow heavily. If you are at the ERI Summit I can discuss with you more about how to do this easily.

https://github.com/SymbiFlow/symbiflow-arch-defs/blob/master/xc7/yosys/synth.tcl


Simplify and improve the testing of the bag unpacker

Pass around ObjectLocationPrefix in more places where it's appropriate (e.g. for “unpack this bag into this prefix”)
Use an IngestID throughout the unpacker, rather than a String
The Unpacker is now a type class with generic get() and put() methods (not a Store – they might point to different backends), and you create implementations of the Unpacker for the backends you want. This comes with our new generic test cases, so it’s more thoroughly tested.
Archive has been completely rewritten to be simpler, to cut out the tail recursion and be easier to follow. It’s more generic (no UnpackSummary or T hanging around), and it has more test cases.

Only failing test is a flaky bag replicator, so YOLO merging.


Handle error conditions better

Display stack trace, but do not fail and exit
(losing shells can be nightmare)
For example, dnscat2 server fails with following error messages on long running servers:
Caught an error: DNS packet was truncated (or we messed up parsing it)!
/pentest/dnscat2/server/libs/dnser.rb:179:in unpack' /pentest/dnscat2/server/libs/dnser.rb:634:in parse'
/pentest/dnscat2/server/libs/dnser.rb:849:in block (2 levels) in on_request' /pentest/dnscat2/server/libs/dnser.rb:843:in loop'
/pentest/dnscat2/server/libs/dnser.rb:843:in `block in on_request'
Protocol exception caught in dnscat DNS module (for more information, check window 'dns1'):
#<DnscatException: Received a packet with no questions>
Caught an error: undefined method serialize' for nil:NilClass /pentest/dnscat2/server/libs/dnser.rb:678:in block in serialize'
/pentest/dnscat2/server/libs/dnser.rb:677:in each' /pentest/dnscat2/server/libs/dnser.rb:677:in serialize'
/pentest/dnscat2/server/libs/dnser.rb:819:in reply!' /pentest/dnscat2/server/libs/dnser.rb:775:in error!'
/pentest/dnscat2/server/tunnel_drivers/driver_dns.rb:357:in rescue in block in initialize' /pentest/dnscat2/server/tunnel_drivers/driver_dns.rb:293:in block in initialize'
/pentest/dnscat2/server/libs/dnser.rb:879:in block (2 levels) in on_request' /pentest/dnscat2/server/libs/dnser.rb:843:in loop'
/pentest/dnscat2/server/libs/dnser.rb:843:in block in on_request' /pentest/dnscat2/server/libs/dnser.rb:883:in rescue in block (2 levels) in on_request': undefined method response_template' for #<DNSer::Transaction:0x0000000000dd6a58> (NoMethodError) Did you mean?  respond_to?         from /pentest/dnscat2/server/libs/dnser.rb:878:in block (2 levels) in on_request'
from /pentest/dnscat2/server/libs/dnser.rb:843:in loop'         from /pentest/dnscat2/server/libs/dnser.rb:843:in block in on_request'

I totally agree. I've started re-writing the client in rust, with solid
error handling (and testing) built in. Unfortunately, it's a bit of a long
project.
On Wed, Dec 5, 2018 at 6:35 AM kost<EMAIL_ADDRESS>wrote:

Display stack trace, but do not fail and exit
(losing shells can be nightmare)
For example, dnscat2 server fails with following error messages on long
running servers:
Caught an error: DNS packet was truncated (or we messed up parsing it)!
/pentest/dnscat2/server/libs/dnser.rb:179:in unpack'
/pentest/dnscat2/server/libs/dnser.rb:634:in parse'
/pentest/dnscat2/server/libs/dnser.rb:849:in block (2 levels) in
on_request' /pentest/dnscat2/server/libs/dnser.rb:843:in loop'
/pentest/dnscat2/server/libs/dnser.rb:843:in `block in on_request'
Protocol exception caught in dnscat DNS module (for more information,
check window 'dns1'):
#<DnscatException: Received a packet with no questions>
Caught an error: undefined method serialize' for nil:NilClass
/pentest/dnscat2/server/libs/dnser.rb:678:in block in serialize'
/pentest/dnscat2/server/libs/dnser.rb:677:in each'
/pentest/dnscat2/server/libs/dnser.rb:677:in serialize'
/pentest/dnscat2/server/libs/dnser.rb:819:in reply!'
/pentest/dnscat2/server/libs/dnser.rb:775:in error!'
/pentest/dnscat2/server/tunnel_drivers/driver_dns.rb:357:in rescue in
block in initialize'
/pentest/dnscat2/server/tunnel_drivers/driver_dns.rb:293:in block in
initialize'
/pentest/dnscat2/server/libs/dnser.rb:879:in block (2 levels) in
on_request' /pentest/dnscat2/server/libs/dnser.rb:843:in loop'
/pentest/dnscat2/server/libs/dnser.rb:843:in block in on_request'
/pentest/dnscat2/server/libs/dnser.rb:883:in rescue in block (2 levels)
in on_request': undefined method response_template' for
#DNSer::Transaction:0x0000000000dd6a58 (NoMethodError) Did you mean?
respond_to? from /pentest/dnscat2/server/libs/dnser.rb:878:in block (2
levels) in on_request'
from /pentest/dnscat2/server/libs/dnser.rb:843:in loop' from
/pentest/dnscat2/server/libs/dnser.rb:843:in block in on_request'
You can view, comment on, or merge this pull request online at:
https://github.com/iagox86/dnscat2/pull/135
Commit Summary

Handle error conditions better

File Changes

M server/libs/dnser.rb
https://github.com/iagox86/dnscat2/pull/135/files#diff-0 (13)

Patch Links:

https://github.com/iagox86/dnscat2/pull/135.patch
https://github.com/iagox86/dnscat2/pull/135.diff

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

Cool! I was thinking of forking them into new processes, so if the client
exits the main app would stick around. But that would require shared state,
which is kind of non-trivial, so I haven't done it.
On Thu, Dec 6, 2018 at 7:27 AM kost<EMAIL_ADDRESS>wrote:

Regarding client in C, i have few patches to retry connection. They are
bit hacky, but I will make pull request anyway, so feel free to decide what
you want with it.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/iagox86/dnscat2/pull/135#issuecomment-444910078, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAgITL_9jxaztF0zVnLhbu1JTsd2FYLGks5u2TdOgaJpZM4ZC2uQ
.


Bump sha2 dependency to v0.0.3

There has been a performance improvement for short messages which is relevant for Merkle tree applications. I would be interested in the effect on ic-certification. I am guessing around a 10% improvement in instructions used.

Sure, can do. Is it API compatible, so I just have to change the version pins?

Yes it is
On Thu, Sep 14, 2023 at 2:36 PM, Joachim Breitner @.***(mailto:On Thu, Sep 14, 2023 at 2:36 PM, Joachim Breitner < wrote:

Sure, can do. Is it API compatible, so I just have to change the version pins?
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you authored the thread.Message ID: @.***>

Trying in #12, but it seems the latest dfx release (0.14.4) ships moc 0.9.3, while the latest vessel package set requires 0.10.0, according to https://dfinity.github.io/vessel-package-set/. All version confusing.

<EMAIL_ADDRESS>requires moc >=0.9.8. Don't know how I can specify that in mops as a "dependency".
Since you also have a mops.toml in your repo, do you actually need vessel?

Vessel is actually fine (it can fetch the necessary version of moc). But mops seems to rely on whatever moc is in path it seems (when I run mops publish, it tries to build stuff), and there I am using dfx. Maybe I should stop using dfx entirely, but it’s needed to run the demo canister, I guess?
Are people still using vessel or should I drop support for that?

But in any case I think it makes sense to wait until the version of motoko shipped by sdk supports thew new sha2. (Unless people are no longer using the sdk?)

People who use mops probably also use mocv to manage and select moc versions. Using mocv also overrides the moc version for dfx, so you are not bound to the one that ships with dfx. Using mocv and dfx in combination is actually very convenient.
You can stop supporting vessel because less people use it and the ones who do can (and probably are used to) manage their package set.
Sure, you can wait with merging the PR until dfx 0.15.x has been released which will include the necessary moc version.

Ah, there is DFX_MOC_PATH now! Then maybe the simplest course of action (for me) is to get moc via nix directly from the moc repository.

Yes, in CI scripts are usually download moc and set DFX_MOC_PATH.

What do you use to benchmark the certification? Is that the demo canister?

I don't think I do benchmark :-D

I found the benchmark. It is here: https://dfinity.github.io/canister-profiling/crypto/#certified-map
I will ask there for an update there.

We can further certification if you are willing to store your intermediate hashes as [Nat32] instead of Blob. I can expand the interface of sha2 to something like writeNat32(d : [Nat32) (instead of only having writeBlob(b : Blob) as now).

There's an update on the benchmarking results here: https://github.com/dfinity/canister-profiling/pull/87
Certified map improves by 12.5%.

Hmm, it seems mops is using dfx cache show to find moc and mo-doc, and insists on using mo-doc during mops publish.
This is the code:
        // detect mocv
        if (process.env.DFX_MOC_PATH && process.env.DFX_MOC_PATH.includes('mocv/versions')) {
                moDoc = process.env.DFX_MOC_PATH.replace(/\/moc$/, '/mo-doc');
        }
        else {
                moDoc = execSync('dfx cache show').toString().trim() + '/mo-doc';
        }

It seems there is no wasy way to tell mops to just use a specific binary. I’ll just wait for dfx to release with motoko 0.10.0 for now. Maybe I can find a nicer solution some other time.

Ok, with dfx-0.15.0 it worked; guess the moc versions don't have to match up precisely:
~/dfinity/ic-certification $ mops publish
Publishing<EMAIL_ADDRESS>[::::::::::]
Published<EMAIL_ADDRESS>~/dfinity/ic-certification $ git tag v0.1.3


az appservice plan create - cannot create zone redundant plan for WS1 worker

Related command
az appservice plan create -g myrg -n myplan --sku WS2 --zone-redundant --number-of-workers 3 --location northeurope

Describe the bug
The command does not allow a zone redundant WS1, WS2 or WS3 plan to be defined.  It is a valid Azure option to use zone redundancy with these plan types.  The error message is:
Zone redundancy cannot be enabled for sku WS1

To Reproduce
Run the the above command after logging into Azure and setting you preferred default subscription and defining a resource group named myrg.
Expected behavior
Allow WS1, WS2 and WS3 plans to be deployed with zone redundancy.

route to CXP team

@scott-doyland-burrows are these value allowed via other clients. Checking with LA folks if this is supported.

@scott-doyland-burrows I think this is a similar issue https://github.com/Azure/azure-cli/issues/23488 which @StrawnSC is looking into. I will verify further.

@scott-doyland-burrows are these value allowed via other clients. Checking with LA folks if this is supported.

Yes, WS1/2/3 support zone redundancy.  It works via the portal.

@scott-doyland-burrows are these value allowed via other clients. Checking with LA folks if this is supported.

Yes, WS1/2/3 support zone redundancy. It works via the portal.

Thanks @scott-doyland-burrows - got the same confirmation from LogicApps team as well, we will get this ready for the next deployment.


Database volume not mounting

Attempted Debugging

[X] I have read the debugging page

Searched GitHub Issues

[X] I have searched GitHub for the issue.

Describe the Scenario
Windows 11, setup via Docker-Compose.
I am not a Docker expert by any means, but for some reason my database volume isn't being binded to my host, but the app config manages to do so just fine.
I had some issues getting it all working with MariaDB at first, but I was able to get it working by switching to MySQL instead. But for some reason, the database volume ends up in the container, rather than on my host PC. I've tried removing everything and doing a fresh install of BookStackApp, but the problem persists. Is this a bug, or am I slipping up somewhere? Thank you!
version: "2"
services:
  bookstack:
    image: lscr.io/linuxserver/bookstack
    container_name: bookstack
    environment:
      - PUID=1000
      - PGID=1000
      - APP_URL=<my_url>
      - DB_HOST=bookstack_db
      - DB_PORT=3306
      - DB_USER=<user>
      - DB_PASS=<password>
      - DB_DATABASE=bookstackapp
    volumes:
      - B:\Apps\bookstackapp\config:/config
    ports:
      - 6875:80
    restart: unless-stopped
    depends_on:
      - bookstack_db
  bookstack_db:
    image: mysql:8.0.32
    container_name: bookstack_db
    environment:
      - PUID=1000
      - PGID=1000
      - MYSQL_ROOT_PASSWORD=<password>
      - TZ=US/Pacific
      - MYSQL_DATABASE=bookstackapp
      - MYSQL_USER=<user>
      - MYSQL_PASSWORD=<password>
    volumes:
      - B:\Apps\bookstackapp\database:/config
    restart: unless-stopped

Exact BookStack Version
v24.02
Log Content
No response
Hosting Environment
Windows 11, installed via Docker-Compose file (see description)

Hi @need4swede,
You are mounting the /config path for the database, which is commonly a linxuserver image thing, but not commonly used for other container images like the one you're using which is not a linuxserver-managed image.
Your compose stack indicates you're using this image.
Based upon the guidance there, I think you need to mount to the /var/lib/mysql container path instead for that image.

Your compose stack indicates you're using this image. Based upon the guidance there, I think you need to mount to the /var/lib/mysql container path instead for that image.

Thank you for the reply! Pardon my ignorance, but is there a way for me to transfer my existing database from the Docker volume to my host - or will I lose all my existing data and have to start fresh?

You should be able to backup and restore your database using mysqldump

@need4swede Currently you data will be within the running container. Take care as that data will likely be destroyed if the container is destroyed (Which can occur even if you just take the compose stack down).
As mentioned by @buckybytes, you'd ideally perform a mysqldump of the existing data for safe-keeping, and re-import.
Alternatively, as long as you still use the same exact docker image, you could copy out the /var/lib/mysql files from inside the container. Ideally use a mysqldump though.
I've got a video here on taking a mysqldump via a docker compose stack.
See the "Creating Backups" and "Restoring a from a backup" sections.
It's using the linuxserver mariadb image, but the steps should be pretty similar.

@ssddanbrown
Bless you, Dan. Worked like a charm!
Now excuse me while I document the backup process in my BookStackApp.
Cheers!


Converting Android SessionDescription to javascript RTCSessionDescription error

When trying to send the Android SessionDescription object to a web client RTCSessionDescription I get an error

Error setting remote description DOMException: Failed to execute 'setRemoteDescription' on 'RTCPeerConnection': Failed to parse SessionDescription.  Expect line: v=

I also got an error saying that the type enum was incorrect and ANSWER didnt exist, I had to toLowerCase on the type in the web client for that error to go away
When sending the SessionDescription from android I convert it to json with gson like I have seen in a lot of examples
....
json.put("sdp", gson.toJson(desc))
....

Then on the web side I just put that value into the description
peerConnection.setRemoteDescription(new RTCSessionDescription(sdp))
This is what the description data looks like
v=0
o=-<PHONE_NUMBER>606126277 2 IN IP4 <IP_ADDRESS>
s=-
t=0 0
a=extmap-allow-mixed
a=msid-semantic: WMS

These two types dont seem compatible, what are you suppose to do?

Just after posting this I found out how to fix it
for anyone else coming across this issue, you cannot just pass in the whole object like this
peerConnection.setRemoteDescription(new RTCSessionDescription(sdp))
Instead you have to put the type and description in explicitly
peerConnection.setRemoteDescription(new RTCSessionDescription({
                type: sdp.type,
                sdp: sdp.description
            }))

Opening this back up because I dont know if this is a "bug" or not based on the documentation for RTCSessionDescription that using this method is deprecated because setLocalDescription and setRemoteDescription accept sdp that conform to RTCSessionDescriptionInit.
So does that mean the SessionDescription in this library is out of date? Feel free to close again if this isnt a bug and is correct

Hi @tyczj, the new release, version 1.3.0, is now available and includes the m125 patches. Could you check if this issue still occurs with the updated release?


Feature request: open PRs to update compat entries for weak dependencies ([weakdeps])

the compats for weekdeps isn't getting any update.

Is it possible this is as simple as changing this line to also look at weakdeps?

You probably need to wrap that entire if block in a for loop that has two iterations. The first iteration will look for deps/compat, and the second iteration will look for weakdeps/WeakCompat.

Also, we probably want to have some way of mentioning (in the PR title) that a CompatHelper PR is for a weakdep (as opposed to a regular dep).


⚠ Bofrost.de has degraded performance

In 3018a64, Bofrost.de (https://www.bofrost.de) experienced degraded performance:

HTTP code: 200
Response time: 1041 ms

Resolved: Bofrost.de performance has improved in a1e245d.


Need to update breaking changes for 23 release

There's been a few breaking changes, off the top of my head:

different sourcmeapsupport
removal of the experimental language server
removal of the diagnostics?

We need to compile a migration guide or at least some thigns to be aware of

It also seems some of my dependencies are not loaded during the tests, resulting in many failures.

I tried updating from 22.4.6 to 23.0.0 (using Jest 23.3.0) and my tests run forever - none completes. No errors. Not sure if this is related, but cannot upgrade.

Problem is, I don't have any reference to create a proper issue. Really the only information I have is "it doesn't work" =-/

I'd be glad to open a new issue but I don't really have anything to say other than "the tests run forever". There's no diagnostic info I can think of to provide. It just gets into an infinite loop somewhere.
Since this issue references undocumented breaking changes I feel like this would be good information to have before I file a bug report, since I don't know if it's just a config problem or not.
Also, possibly related, I have now had to downgrade Jest from 23 because jest@23 + ts-jest@22 breaks sourcemaps (debugging no longer works). Since source maps are mentioned in the initial comment here I thought that could be related too.

This is kinda old, and there is now a changelog with breaking changes so I'm going to close this for cleanup, feel free to re-open if needed

@huafu Not trying to necro, but could you link the breaking changes? If I check the changelog on github I see nothing at 23.0 and I'm trying to figure out why 50% of my tests start failing.


Treat Signals Declared In Swift Like Signals Declared In Godot

When we import a signal from Godot, we generate a helper class and a computed property, which allows us to connect to it in a very Swift-like manner:
func setupBot (robot: Node) {
    robot.ready.connect {
        print ("The robot's node is ready")
    }
}

When we define a signal in Swift using the #signal macro, we create a static property on the Swift class. This requires a different, less intuitive, syntax for connected to or emitting signals:
@Godot public class SBActor: Object {
    #signal("weapons_changed")

  func changeWeapon(_ weapon: SBWeapon) {
    emit(signal: SBActor.weaponsChanged)
  }
}

@Godot public class SBHud: Object {
  func connectToActor(_ actor: SBActor) {
    actor.connect(signal: SBActor.weaponsChanged, to: self, method: "updateWeapons")
  }

  @Callable func updateWeapons() {
    // update the HUD UI here...
  }
}

It should be possible to re-write the signal support so that Swift-side signals are implemented using the same helper classes as imported ones. This should result in a more uniform interface, and less code.
This could be a breaking change (if we alter the #signal macro), but I think it could probably be done instead as an additive change, using a different macro (possibly @Signal). Any implementation would probably be built on top of #42.

A solution would result in syntax more like this:
@Godot public class SBActor: Object {
    @Signal var weaponsChanged   // signal will be exposed to Godot as "weapons_changed"

  func changeWeapon(_ weapon: SBWeapon) {
    weaponsChanged.emit()
  }
}

@Godot public class SBHud: Object {
  func connectToActor(_ actor: SBActor) {
    actor.weaponsChanged.connect {
      // update the HUD UI here...
  }
}

It should also support signal parameters properly:
@Godot public class SBActor: Object {
  @Signal(args: [Int.self]) var healthChanged   // signal will be exposed to Godot as "health_changed, <new-health>"

  func hit(_ amount: Int) {
    health -= amount
    healthChanged.emit(health)
  }
}

@Godot public class SBHud: Object {
  func connectToActor(_ actor: SBActor) {
    actor.healthChanged.connect { health in
      // update the HUD UI here...
  }
}


Push Botkube container image to ECR Public Gallery

Is your feature request related to a problem? Please describe.
Pulling images from dockerhub might become problematic when someone reaches dockerhub rate limits
Describe the solution you'd like
We can also push the docker images to ECR public gallery simultaneously along with dockerhub
Describe alternatives you've considered
NA
Additional context
Pushing images to ECR public gallery would help customers using botkube on EKS or other AWS compute as pulling images from ECR same region is free of cost

#432


How to make an arrow from a node to itself having a nice arc?

How to make an arrow from a node to itself having a nice arc?

Is this what you want?

Hi! Great work on the project!
Is there any way to visualise RNNs such as GRU or LSTM with Moniel neatly?
Thanks)


Switch to using tifffile

Fixes https://github.com/nanshe-org/nanshe/issues/306
Motivation: Most of the TIFF tools we have used for handling conversion have had incomplete support for TIFFs or don't seem to be supported any more. It is time to rip them out and replace them with a known working version tifffile. We have chosen tifffile as it is already part of our stack (comes with skimage), supports most of the TIFF files out there, seems to receive regular support from motivated individuals, allows memory mapping of files, and it is already being used in the field. Many of the files (or file features) that VIGRA struggles with, tifffile is able to handle right away. Even corner cases with metadata where pylibtiff has been found to suffer, tifffile has been able to load the file no problem.

Looks like that is what is required. This now uses memory mapped files instead of reading all data into memory. However, it may require more optimization to handle some of the really big TIFFs, but this will require some trial in the field.

Fixed up so it works with ( https://github.com/nanshe-org/nanshe/pull/304 ).

Changed it so that using memory mapping is optional. This is, by default False, but can be changed by the user through the CLI or the API as desired.

@anotherGithubLearner, raising this to your attention. I am going to put this in a release so you can install it with conda. If you are using the Python API, set memmap to True. This can also be done with the config file as can be seen in this PR. As JSON is different, set memmap to true in the config file. Nothing else changes.

@anotherGithubLearner, this is in 0.1.0a37, which is available for download.


chore: bump safe-deployments

What it solves
bump safe-deployments to the latest version.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 74.631%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
789

Relevant Lines:
987

💛  - Coveralls