FLW-782 Unbond flow: Unbond screen

reusable code for subscriptions moved to separate extension
add validation runner to simplify data validation
add extrinsic proxy to simplify fee retrieval with cache support
add unbond setup screen

Codecov Report

:exclamation: No coverage uploaded for pull request base (develop@9d1ba39). Click here to learn what that means.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             develop     #219   +/-   ##
==========================================
  Coverage           ?   27.44%
==========================================
  Files              ?      428
  Lines              ?    16645
  Branches           ?        0
==========================================
  Hits               ?     4568
  Misses             ?    12077
  Partials           ?        0


reduce inbox thrash

[ ] after release

@buoyad we should discuss if we should merge this for the rel or not. i'm a little hesitant since its been a little buggy but, as you pointed out master already has a bug. lets discuss irl


fix: ignore disabled series in stacked bar values

🐛 Bug Fix
Sets the bar values to the sum of only enabled series. This means that when you click on the legend to enable/disable series in multibar charts, the bar value will be the sum of only the visible bars. Unable to test right now because the build is broken, but will include screenshots once I can.
to: @kristw

Master is back! Please rebase and push again.

I'm also removing some of the test data for nvd3 bar charts since they weren't formatting very well in storybook

@kristw this should be good for review now

I'll need you to merge it too since I don't have write access

The build failed :(

They weren’t due to changes in anything I touched though, were they? It looked like lint errors in other files

Minor linting. I will take a look when i can
On Jun 7, 2019, 18:20 -0700, Erik Ritter<EMAIL_ADDRESS>wrote:

They weren’t due to changes in anything I touched though, were they? It looked like lint errors in other files
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

I'll merge and see if it goes away. Travis passes which is more important. Netlify is nice to have


Upgrade dependencies

#667  👈
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @lazynina and the rest of your teammates on  Graphite


XS✔ ◾ Add screenshot for pipeline

❌ Add a description.

@usamaahmadkhan Image is available for testing. docker pull stakater/workshop-exercise:SNAPSHOT-PR-122-2f80655f


App lost smooth scrolling after last update

Describe the bug
Nothing special, just app have worked smoothly and fast before updating and have start works slowly after (screen renders very slow)
Smartphone (please complete the following information):

Device: Nomi I5050
OS: Android 7.0
Version 0.8.8 build 7

I think this has to do with hardware acceleration.  I have added a change that always uses this feature if the device is running a version of Android greater than 5.1.1 (sorry doesn't work on older devices).

My phone as slow as cheap (Chipset: MT6737T, CPU: Up to 1.5 GHz quad-core ARM Cortex-A53, GPU: Mali-T720 MP2, Memory: 3072 MB), I think that`s why HW acceleration didn't increase performance but decrease. Maybe in future release add switch (hardware/software) in settings that can help to avoid this problem?

Hardware acceleration was alway on if you have a supported device in previous releases.   This is a bug because I removed it in the last release and have no added it back.   The release version which added this back is v <IP_ADDRESS>.  Are you using that version now?

Has update app but this bug still appears.

I think the issue is some devices do not work properly with hardware
acceleration.   I have added a toggle to turn that on or off in the next
release.  There is nothing else I can offer because the other possibility
is the app is now 64 bit due to a requirement by Google.   So if the app
runs slow after the next update, you will have to run an older version of
the application.  All previous versions are available on the Github
repository release section.
Michael Ritchie
<EMAIL_ADDRESS>On Sat, Oct 19, 2019 at 9:40 AM first0n3<EMAIL_ADDRESS>wrote:

[image: Screenshot_20191019-153601]
https://user-images.githubusercontent.com/56667908/67145058-5fe6a000-f286-11e9-8a16-1d51a57f4187.png
Has update app but this bug still appears.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/thanksmister/wallpanel-android/issues/131?email_source=notifications&email_token=AABCYBBKB2K2Y6DFRP4JYFDQPL52HA5CNFSM4JBUQKFKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEBXOSAI#issuecomment-544139521,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AABCYBHSPJDXNVXC7RA6EZDQPL52HANCNFSM4JBUQKFA
.

Has updated to v0.8.9-beta.0 but still no luck nor with on HW nor with off HW. Is the v<IP_ADDRESS> builded as 32bit app?

The application is built for both 32 bit and 64 bit compatible.  That is the new requirement for apps in the Google Play store.  Here is a bit more information about the required change and the supported arm processors: https://developer.android.com/distribute/best-practices/develop/64-bit

@first0n3 do you know if you have an update for your WebView component available?  Search Google play to see if there is one.

I have installed WebView version 77.0.3865.116, no updates available.

Same problem here with a Samsung Galaxy Tab S2 (8 inch) with Android 7.0.
v<IP_ADDRESS> Smooth scrolling
v<IP_ADDRESS> Slow
v<IP_ADDRESS> Slow, with and without hardware acceleration turned on.
Updated WebView, makes no difference.
I'm still on stock firmware. Is there is possibility this can be fixed with a LineageOS mod?

I was able to reproduce the lag on my older Samsung running Android 6.   Only the WebView lags, not for instance if you go to the settings.  So this is something isolated with the WebView and could be related to the compiled version of the Android SDK I had to use for 64-bit.
I am using API 28 now, which is the main difference between the current versions and version v<IP_ADDRESS> for example.   I did some searching and I found one possible solution, which did speed up the webview scrolling, but its not specifically related to API 28.
https://stackoverflow.com/questions/7422427/android-webview-slow
I am willing to try this fix, assuming it does not also  change the behavior for others, it doesn't seem to be an issue on my Android Nexus device running Android 10 for example.   This fix seems to be related to hardware acceleration, so it is possible that something changed in API 28 and this is another way to disable it for older devices.    This is all just a guess and the only device I have to test is my older Samsung.
if (Build.VERSION.SDK_INT >= Build.VERSION_CODES.KITKAT) {
            // chromium, enable hardware acceleration
            mWebView?.setLayerType(View.LAYER_TYPE_HARDWARE, null);
        } else {
            // older android version, disable hardware acceleration
            mWebView?.setLayerType(View.LAYER_TYPE_SOFTWARE, null);
        }

I'm glad you could reproduce it. Thanks for the great work. Hopefully this fixes it.

v<IP_ADDRESS> works smoothly. Many thanks!

For me too. Thanks!

Unfortunately, Google has decided to ban or at least restrict apps that use the WebView component in the Google Play store:
APK:8901 | WebviewsWe don’t allow apps whose primary purpose is to provide a webview of a website or to aggregate content that does not belong to the developer. To resolve this issue, please remove violating content and resubmit your app. For more details, please read through the Webviews and Affiliate Spam policy.
I have adjusted the age limit to be 16 years to 18 years, hopefully that gets the app passed as it was blocked because it was for all ages (which it should be).   Their new review process is also holding updates for up to a week now.

I am going to close this issue, thanks for all the input!


Training is too slow

Hi, @Tramac , I use your code with default setting, and find the speed of training is too slow. The usage of GPU is less than 10%,  resulting in about 130 seconds to train a mini-batch. How to solve this problem?

I am also encountering this issue

Same issue. To debug, I ran timing on each portion of training, I was able to identify the source of the largest delay:
The computation of the SoftmaxCrossEntropyOHEMLoss occurs on CPU:
https://github.com/Tramac/Fast-SCNN-pytorch/blob/0638517d359ae1664a27dfb2cd1780a40a06c465/utils/loss.py#L60
On my workstation and my training data, the training is taking ~1.1 seconds/image. The execution of the forward method in class SoftmaxCrossEntropyOHEMLoss(nn.Module) alone accounts for ~60% of that duration.
Has anyone come up with a more efficient implementation of this OHEM loss? Possibly translating the ops from numpy to torch to be able to run on GPU?


PHOENIX-6497 Remove embedded profile and always build phoenix-client-…

…embedded

(!) A patch to the testing environment has been detected.
Re-executing against the patched versions to perform further tests.
The console is at https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-1254/1/console in case of problems.

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
5m  3s
Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s
No case conflicting files found.

+1 :green_heart:
@author
0m  0s
The patch does not contain any @author tags.

-1 :x:
test4tests
0m  0s
The patch doesn't appear to include any new or modified tests. Please justify why no new tests are needed for this patch. Also please list what manual steps were performed to verify this patch.

_ master Compile Tests _

+1 :green_heart:
mvninstall
20m 50s
master passed

+0
hbaserecompile
27m 32s
HBase recompiled.

+1 :green_heart:
compile
1m 40s
master passed

+1 :green_heart:
javadoc
1m 25s
master passed

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
13m 32s
the patch passed

+0
hbaserecompile
21m 39s
HBase recompiled.

+1 :green_heart:
compile
1m 40s
the patch passed

+1 :green_heart:
javac
1m 40s
the patch passed

+1 :green_heart:
shellcheck
0m  2s
There were no new shellcheck issues.

+1 :green_heart:
shelldocs
0m  3s
There were no new shelldocs issues.

+1 :green_heart:
whitespace
0m  0s
The patch has no whitespace issues.

+1 :green_heart:
xml
0m  1s
The patch has no ill-formed XML file.

+1 :green_heart:
javadoc
1m 24s
the patch passed

_ Other Tests _

-1 :x:
unit
117m 46s
root in the patch failed.

-1 :x:
asflicense
1m  4s
The patch generated 613 ASF License warnings.

180m 43s

Subsystem
Report/Notes

Docker
ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-1254/1/artifact/yetus-general-check/output/Dockerfile

GITHUB PR
https://github.com/apache/phoenix/pull/1254

Optional Tests
dupname asflicense shellcheck shelldocs javac javadoc unit xml compile

uname
Linux 989dbec8fd37 4.15.0-65-generic #74-Ubuntu SMP Tue Sep 17 17:06:04 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev/phoenix-personality.sh

git revision
master / 8b81579

Default Java
Private Build-1.8.0_242-8u242-b08-0ubuntu3~16.04-b08

unit
https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-1254/1/artifact/yetus-general-check/output/patch-unit-root.txt

Test Results
https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-1254/1/testReport/

asflicense
https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-1254/1/artifact/yetus-general-check/output/patch-asflicense-problems.txt

Max. process+thread count
10221 (vs. ulimit of 30000)

modules
C: . U: .

Console output
https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-1254/1/console

versions
git=2.7.4 maven=3.3.9 shellcheck=0.7.0

Powered by
Apache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.


[5.9] Downgrade priors version mismatch warning

As a user when you're changing between versions of these files there's
nothing you could do to avoid invalidation so this warning feel
superfluous
(cherry picked from commit e98ddbdab89730d039a461ca134f0e5ec20e55dc / https://github.com/apple/swift-driver/pull/1396)

@swift-ci please test


How to do coverage report for sqlcool unit test

I can run sqlcool tests using flutter run "test file path" but I need to include test in coverage report. How should I do that. any code samples appreciated

To run the tests with coverage use:
pub run test_coverage --no-badge && genhtml -o coverage coverage/lcov.info

The coverage then can be viewed at coverage/index.html

when I run I got following error:
Could not find a file named "pubspec.yaml" in "/Users/MoNa/.pub-cache/hosted/pub.dartlang.org/file-5.0.8+1".
also I can run for regulat POJO unit tests using

flutter test --coverage

for sqlCool unit tests I have to run on device by
flutter run test file
so there is no way to include in coverage report. I saw there is coverage report in this repo how that is done?

Yes use:
flutter test --coverage && genhtml coverage/lcov.info -o coverage/output

This is how the coverage report is done

but that gives following error:
MissingPluginException(No implementation found for method getApplicationDocumentsDirectory on channel plugins.flutter.io/path_provider)

Does this happen when you run flutter test from the main directory of the Sqlcool package ot when running your own tests?
The tests mock the app directory so this is not supposed to happen: https://github.com/synw/sqlcool/blob/master/test/base.dart#L15 : the Db.ìnit method in the tests must always be run with the absolutePath: true parameter to be able to run on a computer and avoid path provider incompatibilities

I face the error when running my own tests and here is my test code:
Map item = {
    '@token': mockToken,
    'refreshToken': mockRefreshToken,
    '@time_bubble': '1y',
    '@time_list': '1M',
    "@first_time": 'yes'
  };
  AppModel model = AppModel();
  SharedPreferences prefs = await getReloadedSharedPreferencesInstance();
  Db sql;
  ObjectDB odb;
  sql = Db();
  await sql.init(
      path: "db.sqlite", verbose: true, queries: kAppDatabaseTableQueries);
  model.db = sql;
  model.odb = odb;
  model.init(item, prefs);
test("changeTimeFilter test", () async {
      UserModel userModel = UserModel(
          id: "79cf2048cae6f287b7b8ca271a567129",
          email<EMAIL_ADDRESS>          title: "anon",
          alias: "AN",
          self:
              "https: //newsconsole.com/test/api/napi2/nodes/79cf2048cae6f287b7b8ca271a567129");
      model.user = userModel;
      await model.changeTimeFilter({"value": "6M", "type": "list"});
      List<Map<String, dynamic>> users_meta =
          await model.db.query("SELECT * FROM users_meta");
      expect(users_meta[0], {
        "id": 1,
        "user_id": "79cf2048cae6f287b7b8ca271a567129",
        "meta_key": "time_filter",
        "meta_value": "6M",
        'updated': null
      });
      model.timeFilterController.stream.last.then((timeMap) {
        expect(timeMap, {"value": "6M", "type": "list"});
      });
    });

    test("setTabsAllowed test", () async {
      UserModel userModel = UserModel(
          id: "79cf2048cae6f287b7b8ca271a567129",
          email<EMAIL_ADDRESS>          title: "anon",
          alias: "AN",
          self:
              "https: //newsconsole.com/test/api/napi2/nodes/79cf2048cae6f287b7b8ca271a567129");
      model.user = userModel;
      await model.setTabsAllowed(10);
      List<Map<String, dynamic>> users_meta =
          await model.db.query("SELECT * FROM users_meta");
      expect(users_meta.last["meta_value"], '10');
      model.tabsController.stream.last.then((maxTab) {
        expect(maxTab, 10);
      });
    });

How should I mock sqlcool update and query calls for unit tests

As the running database in tests is fake you must mock the query results: check channel.setMockMethodCallHandler in https://github.com/synw/sqlcool/blob/master/test/base.dart for how to proceed
Mock everything that is platform specific: I see that you have a reference to shared preferences for example, not sure that this one can run on desktop

I have following code to mock prefrences:
Future<SharedPreferences> getReloadedSharedPreferencesInstance() async {
  SharedPreferences.setMockInitialValues({});
  return await SharedPreferences.getInstance()
    ..reload();
}

Should I change the sqflite package to sqlcool in following code:
const MethodChannel channel = MethodChannel('com.tekartik.sqflite');

Should I change the sqflite package to sqlcool in following code:
const MethodChannel channel = MethodChannel('com.tekartik.sqflite');

No,keep it: the method channels belong to Sqflite

Thanks a lot. I thought I should test db on device which is not possible to include in coverage report


add forecasting tools to pvlib

Hello fellow pvlib-ers,
I've found some funding to develop open source solar power forecasting tools based on pvlib python and US weather models. EPRI and Southern Company will fund a U. Arizona grad student, Derek Groenendyk, (@MoonRaker) to do the majority of the work this fall semester, though I will work closely with Derek on the project and I will maintain it in the future. The hope is that the pvlib-based forecasts can be reproducible and documentable benchmarks for the solar power forecasting community.
My plan is to add a new forecasting.py module to pvlib python, though I think we should have a conversation about if it's better to add it as a separate project under the pvlib organization. If the community is opposed to this too, then we will host it on our UA-REN organization account.
Unidata is putting together a nice project called Siphon that makes it relatively easy to access data from a number of US weather models (though some cover more than just the US). I experimented with Siphon in this notebook and concluded that this project was very feasible.
One thing that we will need to do is come to some sort of agreement on how to spec out PV systems in pvlib. This has been discussed in a number of other issues in pvlib including #84 #17. So, I think this project should benefit pvlib users even if they're not interested in solar power forecasts.
Thanks in advance for your input on this.
I'll ping a few people just to make sure that they see this... @bmu @Calama-Consulting @jforbess

Wow. Open source forecasting sounds great. I believe it will be a definite
advantage to some pvlib users, so I view it as reasonable to include in the
pvlib-python umbrella, but others may have a different view.
I really need to dust off the pvsystem config work. Friday?
On Tue, Sep 8, 2015 at 2:27 PM, Will Holmgren<EMAIL_ADDRESS>wrote:

Hello fellow pvlib-ers,
I've found some funding to develop open source solar power forecasting
tools based on pvlib python and US weather models. EPRI and Southern
Company will fund a U. Arizona grad student, Derek Groenendyk, (@MoonRaker
https://github.com/MoonRaker) to do the majority of the work this fall
semester, though I will work closely with Derek on the project and I will
maintain it in the future. The hope is that the pvlib-based forecasts can
be reproducible and documentable benchmarks for the solar power forecasting
community.
My plan is to add a new forecasting.py module to pvlib python, though I
think we should have a conversation about if it's better to add it as a
separate project under the pvlib organization. If the community is opposed
to this too, then we will host it on our UA-REN organization account.
Unidata is putting together a nice project called Siphon
https://github.com/Unidata/siphon that makes it relatively easy to
access data from a number of US weather models (though some cover more than
just the US). I experimented with Siphon in this notebook
http://nbviewer.ipython.org/github/wholmgren/siphon/blob/solar_testing/examples/notebooks/solar_testing.ipynb
and concluded that this project was very feasible.
One thing that we will need to do is come to some sort of agreement on how
to spec out PV systems in pvlib. This has been discussed in a number of
other issues in pvlib including #84
https://github.com/pvlib/pvlib-python/issues/84 #17
https://github.com/pvlib/pvlib-python/issues/17. So, I think this
project should benefit pvlib users even if they're not interested in solar
power forecasts.
Thanks in advance for your input on this.
I'll ping a few people just to make sure that they see this... @bmu
https://github.com/bmu @Calama-Consulting
https://github.com/Calama-Consulting @jforbess
https://github.com/jforbess
—
Reply to this email directly or view it on GitHub
https://github.com/pvlib/pvlib-python/issues/86.

Would be nice to have something included in pvlib. However, if this project is mostly focussed on collecting meteorological data for the forecast (pv modelling may be only a small part of it) it may be a good idea to add it as a separate project under the pvlib organisation.

closed by #180.


Auto subspace

Codecov Report

Merging #143 (1ef3e7d) into main (c37c866) will decrease coverage by 1.13%.
The diff coverage is 41.30%.

@@            Coverage Diff             @@
##             main     #143      +/-   ##
==========================================
- Coverage   79.60%   78.48%   -1.13%
==========================================
  Files          31       32       +1
  Lines        3310     3388      +78
==========================================
+ Hits         2635     2659      +24
- Misses        675      729      +54

Impacted Files
Coverage Δ

symmer/projection/qubit_subspace_manager.py
25.00% <25.00%> (ø)

symmer/projection/utils.py
91.75% <33.33%> (-1.87%)
:arrow_down:

symmer/projection/contextual_subspace.py
86.56% <66.66%> (-6.54%)
:arrow_down:

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

symmer/projection/__init__.py
100.00% <100.00%> (ø)


The provided antiforgery token was meant for a different claims-based user

Hi,
So i am trying to work with AspNetCore Antiforgery.  First, i'll setup where i am at:

Added to Startup the following (configured 2nd, after Application Inisghts Telemetry):

        protected virtual void configureAntiforgeryServices(IServiceCollection services)
        {
            services.AddAntiforgery(o =>
            {
                o.HeaderName = "X-XSRF-TOKEN";
            });
        }

Added to Startup the following (configured 1st, before anything else)

        protected virtual void configureAntiforgery(IApplicationBuilder app, IAntiforgery antiforgery)
        {
            app.Use(next => context =>
            {
                var tokens = antiforgery.GetAndStoreTokens(context);

                context.Response.Cookies.Append("XSRF-TOKEN", tokens.RequestToken, new CookieOptions()
                {
                    HttpOnly = false
                });

                return next(context);
            });
        }

I have a controller method that uses the ValidateAntiForgeryTokenAttribute

If i have a user that is not logged in, i am able to take the value from the XSRF-TOKEN cookie and submit it to the API and everything works.  I am using this on a signin API, so once successful, an ASPNETCORE cookie is created for the user.
Once signed in, if i refresh and get a new Antiforgery token, once i submit it to the API i get an error:
The provided antiforgery token was meant for a different claims-based user.
Is there something additional i need to do when writing the Cookie in the page response so that it is for the current user?

I wanted to add, that if i just delete the '.AspNetCore.Cookies' cookie prior to submitting to the API, then the generated antiforgery token works in the API.  So it seems to be something about having the .AspNetCore.Cookies cookie present?...

Also probably important to note that my project.json targets net 4.5.1....  Fifty280.Presentation brings in antiforgery reference:
`
{
"dependencies": {
"Autofac.Extensions.DependencyInjection": "4.0.0-rc2-240",
"Fifty280.Presentation": "1.0.0-*",
"Microsoft.AspNetCore.Mvc.Core": "1.0.0-rc2-final",
"Microsoft.AspNetCore.Server.IISIntegration": "1.0.0-rc2-final",
"Microsoft.AspNetCore.Server.Kestrel": "1.0.0-rc2-final"
},
"tools": {
"Microsoft.AspNetCore.Server.IISIntegration.Tools": {
"version": "1.0.0-preview1-final",
"imports": "portable-net45+win8+dnxcore50"
}
},
"frameworks": {
"net451": {}
},
"buildOptions": {
"emitEntryPoint": true,
"preserveCompilationContext": true
},
"runtimeOptions": {
"gcServer": true
},
"publishOptions": {
"include": [
"wwwroot",
"web.config"
]
},
"scripts": {
"postpublish": [ "dotnet publish-iis --publish-folder %publish:OutputPath% --framework %publish:FullTargetFramework%" ]
}
}
`

I guess you are using IdentityServer and also want to use the AntiForgery
I was able to work around this by having app.UseIdentity() before the XSRF part
public void Configure(IApplicationBuilder app, IHostingEnvironment env, ILoggerFactory loggerFactory, IAntiforgery antiforgery)
{
    loggerFactory.AddConsole(Configuration.GetSection("Logging"));
    loggerFactory.AddDebug();

	app.UseIdentity();

	app.Use(next => context =>
    {
        var tokens = antiforgery.GetAndStoreTokens(context);
        context.Response.Cookies.Append(
            "XSRF-TOKEN",
            tokens.RequestToken,
            new CookieOptions() {HttpOnly = false}
        );

        return next(context);
    });
//.....
}

This solution seems to work, but only from the second requests after the user has logged i n.
So if a API has just processed a login request, and returns a anti forgery token, then the API receives it's next request using that anti forgery token, the API would then throw the same exeception: The provided antiforgery token was meant for a different claims-based user.
I am guessing this is because once the user logs in, whatever antifogery token was set for that request is now invalid, because the API now expects the next request to have an antifogery token that contain's the user's username, while in the previous request the antiforgery token was generated before the user logged in (as useMVC is the last step in the middleware pipeline).
If anyone has had this issue and has resolved it, please let me know!

I'm trying this approach but don't see a "RequestToken". Only formtoken and cookietoken. I tried using those instead with no luck. Ideas?

Hi all, we have a full sample of using AJAX + antiforgery w/ MVC + Identity for login. You can see the sample here: https://github.com/aspnet/Entropy/tree/dev/samples/Mvc.AjaxAntiforgeryAuth.Web

@stephenyao: I had the same issue as you and fixed it by adding a call to
antiforgery.GetAndStoreTokens(context.HttpContext);
in the callback from the IdentityAuthentication Middleware.
Simplified like this:
app.UseOpenIdConnectAuthentication(new OpenIdConnectOptions("MyAuthProvider")             {                  ...                  Events = new OpenIdConnectEvents                 {                      OnTicketReceived = context =>                     {                     ...                     antiforgery.GetAndStoreTokens(context.HttpContext);                     }                 }            }

@Eilon The url to the sample is outdated.
I solved our issue with AspNet Core Identity & AntiforgeryToken by setting the HttpContext.User using the SignInManager before calling the GetAndStoreTokens method.
// send anti forgery token
HttpContext.User = await _signInManager.CreateUserPrincipalAsync(user);
AntiforgeryTokenSet tokens = _antiforgery.GetAndStoreTokens(HttpContext);
HttpContext.Response.Cookies.Append("XSRF-TOKEN", tokens.RequestToken,
            new CookieOptions() { HttpOnly = false });

Ah, we moved the samples a while ago, I updated the URL to: https://github.com/aspnet/Entropy/tree/dev/samples/Antiforgery.MvcWithAuthAndAjax


Unable to Marshal a oneof

Given the following proto
syntax = "proto3";

package people;

message Identity {
    uint64 message_id = 1;
    string name = 2;
    string email = 3;
    string address = 4;
    string details = 5;
}

message Identities {
    uint64 message_id = 1;
    repeated Identity ids = 2;
}

enum payload_type {
    IDENTITY = 0;
    IDENTITIES = 1;
}

message Incoming {
    uint64 message_id = 1;
    payload_type type = 2;
    oneof payload {
        Identity identity = 3;
        Identities identities = 4;
    }
}

And the following code:
incoming := &people.Incoming{MessageId: counter, Type: people.PayloadType_IDENTITY}

inId := &people.Incoming_Identity{}
id := &people.Identity{}
generateRandomID(i) // Just populates Identity with valid data
inId.Identity = id
incoming.Payload = inId

data, err := proto.Marshal(t)
if err != nil {
	log.Println("Marshalling error", err)
}

Marshalling, I get the following error:
proto: no encoder for Payload people.isIncoming_Payload [GetProperties]

I've probably populated Payload wrongly or something, how to I set this properly?
Thanks

You'll get a compile-time error if you try to populate Payload incorrectly.
It looks like your version of github.com/golang/protobuf/proto is out of date.

After looking closely at the import statement, i realized that the goimports imported gogo version of proto. Fixed! Thanks @dsymonds

Is the gogo version not working?

@awalterschulze The gogo version has a different way of handling oneof which kind of remove the oneof type. At least that's what I gather from reading the generated output.

Do you have the newest version.
It should be pretty much the same as golang/protobuf.
On 23 October 2015 at 15:37, Wari Wahab<EMAIL_ADDRESS>wrote:

@awalterschulze https://github.com/awalterschulze The gogo version has
a different way of handling oneof which kind of remove the oneof type. At
least that's what I gather from reading the generated output.
—
Reply to this email directly or view it on GitHub
https://github.com/golang/protobuf/issues/85#issuecomment-150574782.

@awalterschulze you are correct, it looks like they are API compatible, except for gogoslick which did not create the const payloadType_IDENTITY, instead, it's just const IDENTITY

Yes that is one of the features of gogoslick it removes the enum prefixes.
I should add more documentation for the vanity binaries.
https://github.com/gogo/protobuf/issues/81
If this is breaking anything please report an issue with gogoprotobuf.
Sorry David for taking over this issue.
On 24 October 2015 at 18:54, Wari Wahab<EMAIL_ADDRESS>wrote:

@awalterschulze https://github.com/awalterschulze you are correct, it
looks like they are API compatible, except for gogoslick which did not
create the const payloadType_IDENTITY, instead, it's just const IDENTITY
—
Reply to this email directly or view it on GitHub
https://github.com/golang/protobuf/issues/85#issuecomment-150833141.


Frontend trying to connect to localhost:5000/api/login instead of [docker-machine ip default]:5000/api/login

Hello,
Built and run. Input my data, click on login, nothing happens. No loading page, no backend logs. Frontend gives the following error in the JS console cruncher.js:809 OPTIONS http://localhost:5000/api/login net::ERR_CONNECTION_REFUSED. Seems to me it has to use docker-machine ip default instead of fixed localhost to correctly contact login api, right?

You're right, I have the same problem. I'm using it on windows with the docker VM and that's why the IP isn't localhost.

Yup. I hardcoded my IP in config.cljs and now it works. Maybe make it automatic?

Thanks for the report. Will investigate as soon as im off work.

How do I hardcode/access the config.cljs file?

I've found a simplier solution on windows with docker toolbox installed:

Open Oracle VirtualBox
Go to Settings of your Docker VM
Choose Advanced at Network Settings
Choose Port-Forwarding
Add a new Rule:
Host-IP: <IP_ADDRESS>
Host-Port: 8888
Guest-Port: 8888
Leave Guest-IP blank.
Now you can access pogo-cruncher in your host browser with http://localhost:8888

<thanks PSwoboda you solve my life! hahaha easy and fast

Now the app is not loggin didnt say anything

Sry, you need to add a rule for port 5000, too.

Can someone with the windows problems try this version and report results? My Windows machine is unavailable right now unfortunately.
https://github.com/Wegi/pogo-cruncher/releases/tag/0.1.3 (Current master HEAD)
I changed the ip inside the config to listen on <IP_ADDRESS> which should be available outside of the container as well. Also the containers are now extra linked.

I've tested it and removed my port forwarding rules.
cruncher.js:809 OPTIONS http://<IP_ADDRESS>:5000/api/login net::ERR_ADDRESS_INVALID

Hm, maybe I try to include a .bat file for automatic configuration for windows users. Gotta test that first.

Next try, is someone willing to test the newest commit?
Either pull newest version from source or change config.cljs to
(ns cruncher.config)

(def api {:host "http://backend:5000/"
          :init "api/init/"
          :base "api/"
          :login "api/login"
          :get-all-pokemon "api/pokemon"
          :crunch-selected-pokemon "api/pokemon/delete"})

If it still doesnt work, please respond with the terminal output. Would be much appreciated.

I've added an alternative installation path for windows without docker. Please report results.
Installation Setup for Windows user
You can also use the Linux / MAC OS docker setup, but for unknown reasons it does not work properly for most windows users. So use instructions below.

Install Python 3.
* Check "Add Python3.5 to PATH" during installation.
Download the newest Windows release here.
Unzip the folder.
Open cmd or Powershell.
Type cd <C:/Path/to/the/folder>
Type pip install -r requirements.txt
After this finishes you may start the cruncher as follows:

Windows Startup

In the application folder type python app.py
Now open the index.html in the folder with your preffered browser.

For @Delco64 , config.cljs is in the /frontend/src/cruncher/ folder. I am running on MacOSX so I can't test the setup!

docker version doesnt seem to respond after pressing the login button. Any ideas @n2o ?

Nevermind, seems to be an error on my end.

Docker for Windows released and we prepared a precompiled Docker image for it. See the installation section in the Wiki for more information. This should fix your installation problems.


build: group CI steps into jobs

Currently, we have only one CI workflow with one job for rush build docker image build & publish.
Soon, we will add npm publish into the CI workflow, which is not relevant to the docker build, thus, we should separate these steps into different jobs so that the CI workflow would be running in parallel.
Foreseeably, we shall have three jobs:

rush build & deploy
Dockerbuild and push
npm publish

Job 2 and job 3 can be running in parallel.

CI bugs has been found, reopen


languages

revisited eng translations, created hungarian translations

Really sorry bro.. Your pull request include many history commit. I create one on your repo, pardon me :D


[$500] Web - Login - Original tab page blinks when add expired magic code

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Version Number: 1.4.1-6
Reproducible in staging?: Y
Reproducible in production?: Y
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Expensify/Expensify Issue URL:
Issue reported by: Applause - Internal Team
Slack conversation:
Action Performed:

Navigate to staging.new.expensify.com
On the log in screen, enter email address
Navigate to the email inbox and open the validate email and copy the link
Modify the last portion of the link by a character, change to staging and paste it to another tab
Navigate to the original tab

Expected Result:
User should navigate to the original tab and no errors or blinking page should be visible
Actual Result:
Original tab page blinks when add expired magic code
Workaround:
Unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[ ] Android: Native
[ ] Android: mWeb Chrome
[ ] iOS: Native
[ ] iOS: mWeb Safari
[x] Windows: Chrome
[ ] MacOS: Desktop

Screenshots/Videos
Add any screenshot/video evidence

https://github.com/Expensify/App/assets/78819774/03c71287-7ed8-4133-8d6a-77c3fd5fbd01
View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~013535f35e646fe39f
Upwork Job ID:<PHONE_NUMBER>370157056
Last Price Increase: 2023-11-21

I am unable to reproduce this on macOS Chrome.

I also can't reproduce this, we have replaced this with very intentional behaviour:
https://github.com/Expensify/App/assets/59587260/54048388-888e-4cc3-8165-d7c0f4d30460
Closing for now, please reopen if this changes.


REF: stats.unuran: Refactor the UNU.RAN wrapper to avoid memory leaks

Reference issue
scipy/scipy#14215
What does this implement/fix?
scipy/scipy#14215 has memory leaks due to non-local returns. This PR eliminates the problem by using the MessageStream API (originally used in Qhull to handle errors occurring in qhull C API).
One of the downsides of this approach is that UNU.RAN uses a global FILE * stream which makes it thread-unsafe to use those streams without first acquiring a lock. To expound on the problem, we have to call unur_set_stream under a lock otherwise some other thread could change the global FILE * variable and all the errors will be redirected to that wrong file. This makes things fairly complex and at this point, I can't tell by myself if something like this is acceptable or an alternate approach should be used. This is quite an adventure in the uncharted lands for me :) This kind of thing has been done previously in Qhull by @pv. If you could spare some time, I would really appreciate your help on this @pv!
Another problem is that because I can't do non-local return in the thunks when an error has occurred in the callback, I have to wait until UNU.RAN's function evaluation has completed and then check with PyErr_Occured() if an error has occurred. I am not sure if this is conventional so let me know what you think...
Additional information
@rgommers @mckib2 Whenever you are free, can you please take a look at this and review/verify if this approach is correct? If not, is there (yet) another way in which one can deal with this problem? Anyways, let me know your thoughts on this!
Once scipy/scipy#14328 is merged, I have verified locally that valgrind reports no memory leaks for both numpy < 1.19 and numpy >= 1.19.
With NumPy < 1.19:
==44175== LEAK SUMMARY:
==44175==    definitely lost: 1,128 bytes in 11 blocks
==44175==    indirectly lost: 0 bytes in 0 blocks
==44175==      possibly lost: 195,264 bytes in 124 blocks
==44175==    still reachable: 3,258,070 bytes in 2,112 blocks
==44175==         suppressed: 0 bytes in 0 blocks

With NumPy >= 1.19:
==44181== LEAK SUMMARY:
==44181==    definitely lost: 1,128 bytes in 11 blocks
==44181==    indirectly lost: 0 bytes in 0 blocks
==44181==      possibly lost: 172,983 bytes in 125 blocks
==44181==    still reachable: 3,533,089 bytes in 2,198 blocks
==44181==         suppressed: 0 bytes in 0 blocks

The 1128 lost bytes are inside dlopen.c library and not UNU.RAN.

@tirthasheshpatel While this is a bit tricky and nonstandard, it might be that this is the way to go to avoid memory leaks and being able to support users being able to run these functions in multiple threads.  Out of curiosity, do we know if UNU.RAN supports running multhreaded out-of-the-box?  if not, does the R wrapper allow for this?  (no need to look at source code, a simple test running an R snippet on multiple threads might be enough to answer this).
I see where UNU.RAN is declaring the global file pointer.  I wonder if this can somehow be keyed at runtime on the current executing process/thread ID -- I'll think about this.  Is this log file being read by any process or only being dumped to?

Another problem is that because I can't do non-local return in the thunks when an error has occurred in the callback, I have to wait until UNU.RAN's function evaluation has completed and then check with PyErr_Occured() if an error has occurred. I am not sure if this is conventional so let me know what you think...

Is the concern here that it takes longer to check PyErr_Occured() or that it's a strange control flow?  As long as you can guarantee that PyErr_Occured() will always return True when UNU.RAN encounters an internal error, then this should be fine.  The usual way of bubbling up an exception won't work here because it interrupts the memory deallocation strategy, so I can't off the top of my head think of a better way to tackle this

Out of curiosity, do we know if UNU.RAN supports running multhreaded out-of-the-box?

With an exception of a few routines (namely unur_set_stream, unur_set_error_handler, unur_test_count_run, unur_test_count_pdf, and unur_errno), all the UNU.RAN routines are thread-safe.

if not, does the R wrapper allow for this?

R is largely a single-threaded programming language. AFAIK, all R internals are single-threaded. Runuran manual doesn't discuss multi-threading at all. So, I guess Runuran wouldn't be thread-safe too (but I can't back this statement with hard evidence given sparsity of this topic in R and Runuran manuals)

Is the concern here that it takes longer to check PyErr_Occured() or that it's a strange control flow?

It's a strange control flow.

As long as you can guarantee that PyErr_Occured() will always return True when UNU.RAN encounters an internal error, then this should be fine.

I call PyErr_Occurred() after every call to UNU.RAN functions which might call the callback. It guarantees to return True when an error has occurred during evaluation of a Python callback, so it should always be able to catch the error.

Is this log file being read by any process or only being dumped to?

It's only being dumped to.

Is this log file being read by any process or only being dumped to?

It's only being dumped to.

Can we get away with disabling logging entirely then?  That seems to me to avoid the need for locking and messing with the MessageStream API.  That would involve setting UNUR_ENABLE_LOGGING=0 and probably changing this line from LOG = stderr to LOG = NULL

It's a strange control flow.

It seems documented well-enough to me to understand what's going on, so I would accept this solution

Can we get away with disabling logging entirely then? That seems to me to avoid the need for locking and messing with the MessageStream API.

Do you mean we should not log UNU.RAN errors at all? I think that will make debugging very difficult. There are several things that can go wrong in UNU.RAN and unless we capture these error messages, there is no way for the user to know what went wrong.

That would involve setting UNUR_ENABLE_LOGGING=0 and probably changing this line from LOG = stderr to LOG = NULL (I don't know that this will work, but it might simplify things if we don't need to be logging messages. Might run into problems if calling fclose on NULL file descriptors -- that's undefined behavior)

I think we can avoid all that and just call unur_error_handler_off which would turn off all the logging and change the required global variables appropriately.

Closing this. We can move the discussions to the SciPy PR. Thanks for the review, @mckib2!


Reduce code duplication in processor test suite with parameterized tests

I've been poking around the test suite for the processors (i.e. src/test/java/sorald/processor), and I've found that there is a whole lot of code duplication. Most of the test classes are borderline identical, and generally follow this pattern:
public class SomeCheckProcessorTest {
    @Test
    public void test() throws Exception {
        String fileName = "SomeFile.java";
        String pathToBuggyFile = Constants.PATH_TO_RESOURCES_FOLDER + fileName;
        String pathToRepairedFile = Constants.SORALD_WORKSPACE + "/" + Constants.SPOONED +"/" + fileName;

        JavaCheckVerifier.verify(pathToBuggyFile, new SomeCheck());
        Main.main(new String[]{
                Constants.ARG_SYMBOL + Constants.ARG_ORIGINAL_FILES_PATH,pathToBuggyFile,
                Constants.ARG_SYMBOL + Constants.ARG_RULE_KEYS,"<KEY>",
                Constants.ARG_SYMBOL + Constants.ARG_WORKSPACE,Constants.SORALD_WORKSPACE});
        TestHelper.removeComplianceComments(pathToRepairedFile);
        JavaCheckVerifier.verifyNoIssue(pathToRepairedFile, new ArrayHashCodeAndToStringCheck());
    }
}

There is some minor variation in the arguments to Main.main and variable names, but most classes are otherwise semantically identical. If Sorald is to scale with more rules and more tests, I think this must be revamped.
As a solution, I propose that we write a single parameterized test, and keep a directory with "broken" Java files (such as the current src/test/resources directory). Then, each file can be automatically turned into a separate test case. If we for a moment ignore the problem with the variation in command line arguments, a test case file could be named as SomethingInformative_<SONAR_CUBE_KEY>.java, and be used to generate a test case for the rule with key <SONAR_CUBE_KEY> (e.g.  EqualsOnAtomicClass_2204.java to test rule 2204).
I have yet to figure out the best solution for the variation in arguments to Main.main, but I'm sure a reasonable solution can be found through some trial and error.
As an example, I've done something similar to this in Spork, an AST-based merge tool. There, I have one directory per test case containing the three input files for the merge, as well as the expected output file as an oracle. Then, source providers that look like this generate input for a handful of test cases that look like this. This gives me hundreds of test cases with very few lines of code, and adding a new test case only requires the addition of new input, there's no additional logic.
I could get to work on this right away. Any thoughts?

I could get to work on this right away.

The solution you proposed seems good to me. I encourage you to proceed.

Cool! If I haven't overlooked something critical, I should at least have a draft PR of this sometime tomorrow.

This was fixed with #144


Screen readers badly render links in "federated reply" block

Quick summary
@pfefferle when using the "reply to" bookmarklet, at beginning of a WordPress post, an unpleasant output for screen readers appears - for example "↬mastodon.uno/@uaar/113039993829553640"
It is the direct link at the post/conversation we are replying to and it's correct, but the "a href" HTML tag has no link text! So, for fediverse newbies it's not easy to figure it is a post's reply because the screen reader reads the alpha-numeric link and a user would never click something "anonymous"; it seems like a malicious code.
Steps to reproduce

open any Mastodon/Fediverse post;
use the "reply from [blog]" bookmarklet so that it creates a new post;
the issue can be verified also in previewing the post after having saved it into draft.

What you expected to happen
The "federated reply" block should look, in front-end, like this:
"this post is in reply to: [mentioned user] conversation: view original thread" - and text string in the block should be editable from within the block, so that a user can introduce it as they want (I mean the link text and "this post is in reply to"...).
What actually happened
Now just an emoji and the post's alphanumeric link/code appear. Alone, with no explanation on what it is.
Impact
All
Available workarounds?
No but the platform is still usable
Logs or notes
ActivityPub version 3.0.1, WordPress 6.6.1, Gutenberg 19.1. This issue has no workarounds and prevents me from replying to conversations. - Not even verified if clicking the link opens a new window. If it doesn't and takes me out of the blog, it's even more serious.

Found a temporary small workaround to respond to posts without it seeming to be a malicious code.

place the "federated response" block as first block for the post;
add a "read more" block right after it;
select the newly added "read more" block, and activate the "hide" setting on "read more"'s block settings.

It is quite an unpleasant workaround as it currently prevents me from writing other stuff in there, and it forces me to remember I have to write a mention to the original author and fediverse conversation's list, appended to the post. But it's all what I have now.


Cannot install with Packer

I have lots of other plugins, similarlly installed.
At the moment tried adding it with the simple clean use method. :PackerSync installed it but upon restart, getting this error. Tried several ideas of fixing it but simply doesn't work.
I believe the initial image error below is caused because api_key_cmd was missing. Perhaps error can be improved? However, i believe i have API key error, again, errors are not very descriptive or conclusive. It tooke me 6 attempts to receive a conclusive error message in the prompt.
Please check images below.
For context, using Ubuntu in WSL.

I installed it successfully with Packer today (I haven't gotten it working yet which is another problem). Have you asked chatgpt for some potential solutions?

try  to create new session, this message may came from history


Auto-Compaction using Multi-Stage Query Engine

Tests to be added.

Description
Compaction operations issued by the Coordinator currently run using the native query engine. As majority of the advancements that we are making in batch ingestion are in MSQ, it is imperative that we support compaction on MSQ to make Compaction more robust and possibly faster. For instance, we have seen OOM errors in native compaction that MSQ could have handled by its auto-calculation of tuning parameters.
This PR enables compaction on MSQ to remove the dependency on native engine.
Main changes:

DataSourceCompactionConfig now has an additional field engine that can be one among [native, msq] with native being the default.
if engine is MSQ, CompactSegments duty assigns all available compaction task slots to the launched CompactionTask to ensure full capacity is available to MSQ. This is to avoid stalling which could happen in case a fraction of the tasks were allotted and they eventually fell short of the number of tasks required by the MSQ engine to run the compaction.
ClientCompactionTaskQuery has a new field ClientCompactionRunnerInfo with just one Engine subfield.
CompactionTask now has CompactionRunner interface instance with its implementations NativeCompactinRunner in core and MSQCompactionRunner in the druid-multi-stage-query extension. . The objectmapper deserializes ClientCompactionRunnerInfo in ClientCompactionTaskQuery to the CompactionRunner instance that is mapped to the specified type [native, msq].
CompactTask uses the CompactionRunner instance it receives to create the indexing tasks.
CompactionTask to MSQControllerTask conversion logic checks whether metrics are present in the segment schema. If present, the task is created with a native group-by query; if not, the task is issued with a scan query. The storeCompactionState flag is set in the context.
Each created MSQControllerTask is launched in-place and its TaskStatus tracked to determine the final status of
the CompactionTask. The id of each of these tasks is the same as that of CompactionTask since otherwise, the workers will be unable to determine the controller task's location for communication (as they haven't been launched via the overlord).

Some things to note:

The context specified in DataSourceCompactionConfig is passed as is to the MSQControllerTask and hence can contain MSQ context params as well, with the exception of rowsPerSegment -- which will be overridden by either targetRowsPerSegment or maxRowsPerSegment if specified in a partitionsSpec.
maxRowsInMemory param is only considered if specified in the context. The value in DataSourceCompactionConfig.tuningConfig is not considered as it is set to a default value (1M) if unspecified by a user, so it is indistinguishable between coming from the user or via the default.

Additions to CompactionState

dimensionToAggregatorFactoryMap: tracks AggregatorFactory for dimensions converted from metrics due to finalizeAggregations =true
engine: denotes which engine was used to create the CompactionState among NATIVE and MSQ

changes to comparison of Compaction Config with CompactionState

dimensionToAggregatorFactoryMap is used in both dimension and metrics comparison to figure out original dimensions and metrics for the compaction task.

Unsupported for MSQ Compaction:

finalizeAggregations set to false in context. (Only true or unspecified works.)
taskAssignment set to auto in context. (Only max or unspecified works.)

Release note

Key changed/added classes in this PR

MyFoo
OurBar
TheirBaz

This PR has:

[ ] been self-reviewed.

[ ] using the concurrency checklist (Remove this item if the PR doesn't have any relation to concurrency.)

[ ] added documentation for new or modified features or behaviors.
[ ] a release note entry in the PR description.
[ ] added Javadocs for most classes and all non-trivial methods. Linked related entities via Javadoc links.
[ ] added or updated version, license, or notice information in licenses.yaml
[ ] added comments explaining the "why" and the intent of the code wherever would not be obvious for an unfamiliar reader.
[ ] added unit tests or modified existing tests to cover new code paths, ensuring the threshold for code coverage is met.
[ ] added integration tests.
[ ] been tested in a test Druid cluster.

@LakshSingla : There needs to be some way to track if a dimension was originally a metrics but ended up being a dimension because of finalizeAggregations=true. So went with a map.
Regarding backward compatibility, while deserializing I've included appropriate handling for missing fields. Some scenarios I can think of:

coordinator - old, indexer - new: indexer assumes native compaction engine and the flow is just as the original flow. It stores the new fields in compaction state which the coordinator ignores and continues with the (sufficient) subset.

coordinator - new, indexer - old: indexer doesn't honor the engine field and continues with the original native compaction flow. It stores CompactionState without the new fields and Coordinator handles the missing field values correctly using defaults.

Please let me know if you come across any other cases that aren't handled.
Support for finalizeAggregations=false will come in a follow-up PR, as mentioned in my other comment.

queryGranularity set to all
Why is this not supported?

The id of each of these tasks is the same as that of CompactionTask since otherwise, the workers will be unable to determine the controller task's location for communication (as they haven't been launched via the overlord).

Task IDs must be unique right? Won't this contradict that requirement?

@LakshSingla: In compaction, the tasks run serially and in-place inside the compaction task, so at a given point in time, only one running task will have compaction tasks's ID. The same happens even for native compaction.

I am wondering if there are unit tests that test the new msq compaction flow in the unit tests. I could find the spec validation tests only.

I think conversion to MSQ spec is the main job of the MSQCompactionRunner. Unit testing the entire flow doesn't really ensure something is broken in this class. I'm planning to cover the entire flow in ITs instead.

@gargvishesh , I concur with @LakshSingla on this one. We need UTs to test the entire flow with MSQ the same way we are doing with native. We may add ITs too but most Druid devs rely on UTs more heavily as the IT flow is a little flaky currently.

If you have already added tests for the new flow, please share that detail here.
If you are yet to start on those, you could just try to parameterize the engine the existing compaction flow tests and run the same existing tests with both native and msq engines. Please let me know if you need any assistance with this.

@kfaraz @LakshSingla
Thanks for your comments -- I do see the usefulness of the UTs for the entire MSQ-based compaction flow. The existing tests in CompactionTaskRunTest however cannot be parameterised since MSQ code resides in an extension. The tests therefore need to be present in the extension itself and require some effort for implementation. I will take them up in a follow-up PR.

Thank you @kfaraz for the time and effort to do all those rounds of reviews, esp. given the size of the PR.


package clobbers libfreetype.so

this package contains a $PREFIX/lib/libfreetype.so which conflicts with the freetype package
(conda-forge::freetype-2.10.0-he983fc9_0)
2019-05-11T06:01:06.3195692Z ClobberWarning: This transaction has incompatible packages due to a shared path.
2019-05-11T06:01:06.3196968Z   packages: conda-forge::openjdk-11.0.1-h516909a_1016, conda-forge::freetype-2.10.0-he983fc9_0
2019-05-11T06:01:06.3197719Z   path: 'lib/libfreetype.so'

maybe it could be removed so to depend on freetype, or libs moved to $PREFIX/lib/jvm ?

ping

Am still seeing this on Windows:
ClobberWarning: This transaction has incompatible packages due to a shared path.
  packages: conda-forge/win-64::openjdk-11.0.1-c_compilervs2015_1016, conda-forge/win-64::freetype-2.10.4-h546665d_1
  path: 'library/bin/freetype.dll'

Relevant CI run and attached log (for posterity)

Not seeing this issue anymore. I checked the package content, and freetype.dll now has another path:
Library/lib/jvm/bin/freetype.dll

I believe this is a consequence of #100 (resp. its backport to java11, #105). Closing.


TypeError: debug is not a function

Trying to use the latest version of fetch-mock-jest in my project, but seeing the below error:

Seeing the same issue with the latest version of fetch-mock as well:

Weird - pin fetch-mock to 9.9.0 for now and hopefully that will work. I'll investigate the failure today

try latest fetch-mock & fetch-mock-jest and reopen if still an issue

hey @wheresrhys
I'm having the same error with 9.x for quite some time. I kept using 8.x but now I decided to try this package and the error continues.

Can you create a repo with a reduced test case so I can investigate further please.

Hey @wheresrhys ,
I have been also getting the below error; Below is my code and configurations.

Version
"fetch-mock": "^9.9.0",
"fetch-mock-jest": "^1.5.1",
"jest": "26.6.3",
"@types/jest": "26.0.23",
"babel-jest": "^27.2.0",
"eslint-plugin-jest": "^23.20.0",
"ts-jest": "^27.0.5",
package.json
"jest": {
"preset": "ts-jest",
"transform": {
"^.+\.(ts|tsx)?$": "ts-jest",
"^.+\.(js|jsx)$": "babel-jest"
},
"testRegex": "./test/.*.(js|jsx)$",
"rootDir": ".",
"moduleNameMapper": {
"\.css$": "identity-obj-proxy"
},
"globals": {
"ts-jest": {
"isolatedModules": true
}
},
"moduleDirectories": [
"js",
".",
"node_modules"
]
},
example.test.js
import 'regenerator-runtime/runtime';
import configureMockStore from 'redux-mock-store';
import thunk from 'redux-thunk';
const fetchMock = require('fetch-mock');
import { getEntityItems } from '../../src/actions/entity1';
const middlewares = [thunk];
const mockStore = configureMockStore(middlewares);
global.requestURI = '';
const store = mockStore({});
const root = '/endpoint';
const testHelper = (description, path, query_params = {}) => {
it(description, () => {
fetchMock.getOnce(
path, { body: { interactions: [] }, headers: { 'content-type': 'application/json' } }
);
return store.dispatch(getEntityItems(query_params)).then(() => {
expect(fetchMock.called(path)).toBe(true);
});
});
};
describe('GET entites query parameters', () => {
afterEach(fetchMock.reset());
testHelper('accepts an array of ids', root.concat('&ids=0,1,2'), { ids: [0, 1, 2] });
});
I have been trying to figure out why i am getting the above error; No Luck!
TIA :)

I still can reproduce this issue, more details here.


Doesn't work when the LoRA has "__" on it's name

I discovered that the Dynamic Prompts fails — it doesn't replace dynamically the prompt values — when we add a LoRA that include includes __ on it's name in the wildcards file. See below an example:
Example

Prompt

score_9, score_8_up, score_7_up, sunny day, colorful, __places__, masterpiece

places.txt

<lora:Example_Lora__I_exist:1> castle, brown, rocks, garden

The issue 🐞

Looks like this make a mess with the dynamic prompting because the __ keyword is used for importing the file (__my_file__).
This could be resolved by just renaming the LoRA and removing the __ on it's name. Still I'm reporting it as a bug because probably it could be resolved by the developer side by an improved regex matching/replacing.
I'm using Stable Diffusion WebUI Forge

See https://github.com/adieyal/sd-dynamic-prompts/issues/755#issuecomment-2042935590.

I see, thanks @akx... it also could be resolved as a whole by the developer side using an improved regex matching/replacing when there are __ (or the chosen wildcard start/end keywords) when they are being used inside LoRAs.

@Victorcorcos Yes, please see https://github.com/adieyal/sd-dynamic-prompts/issues/778. PRs welcome to replace the parser 😉


Support "web+ap://" URIs

URIs with the web+ap scheme are used to represent ActivityPub objects available over HTTPS. It is functionally equivalent to the https URI of an ActivityPub object, and is treated as an https URI by software compliant with the specification.[1] Semantically, the host indicates the Fediverse instance that the AP object is stored on, and the path indicates the AP actor or collection being referenced.
Rationale
web+ap:// URIs are intended to enable a user's preferred Fediverse instance to handle Fediverse links in a more intuitive and integrated fashion. The user's preferred Fediverse instance registers itself as handling web+ap:// URIs, and opens the referenced AP object in the web UI of that instance. This eliminates the need for users to manually visit the AP object's original instance, attempt an interaction, and then enter their preferred instance into a form to be redirected.
Examples

web+ap://thegem.city/@nick
web+ap://fedia.social/@theducky

Resolution and Validation
To resolve a web+ap:// URI, replace the scheme with https and send a GET request. The returned content should be a web page representative of the AP object if the object referenced by the URI exists on the Fediverse instance and if no internal server errors occur.
To validate a web+ap:// URI that appears in a DID document's alsoKnownAs collection, resolve the URI and check for either:

an <a> or <link> tag that links to a bsky.app URL for a profile that resolves to the original DID. This tag must have the attribute rel which contains the string "me" on its own or separated from other values by whitespace.
a <link> tag that links to the original DID. This tag must have the attribute rel which contains the string "did" on its own or separated from other values by whitespace.

[1] https://fedilinks.org/spec/en/6-The-web-ap-URI

Enforcing the requirement that "The page must return a response within 3 seconds." from the website verification method might be wise.

Being closed in favor of acct: URIs.


Unit tests are failing

As the title says. Some unit tests aren't working anymore :(

What error(s) are you getting and what test runner are you using?

I will make some print screens when I'm home. I pulled it on my work laptop ran unit tests and it works. I think it has to do with the visual studio version.

https://gyazo.com/721a244808649068c075c0f1a620f561
this is what I'm getting.

Seems like updating visual studio got  this fixed, thank you though :)


Add missing #[must_use] marker attributes

Fixes CI getting broken by a new clippy lint.
Currently failing due to https://github.com/rust-lang/rust-clippy/issues/8140

Seems that it takes a while for clippy fixes to roll out to the nightly version distributed through rustup, so I went ahead and disabled clippy::return_self_not_must_use for now and filed https://github.com/rust-osdev/uefi-rs/issues/333 as a reminder to revert it.


Quantum Generative Adversarial Networks -- SymPy Deprecation

Steps to reproduce the problem

Go to Quantum Generative Adversarial Networks
Scroll to "Defining the variational quantum discriminator and generator"
See error in Jupyter output

What is the current behavior?
/usr/local/lib/python3.9/site-packages/sympy/core/expr.py:3949: SymPyDeprecationWarning:

expr_free_symbols method has been deprecated since SymPy 1.9. See
https://github.com/sympy/sympy/issues/21494 for more info.

  SymPyDeprecationWarning(feature="expr_free_symbols method",

What is the expected behavior?
No deprecation warnings. This is something that frustrates professors and students quite often. Maybe this is a case to setup a CI/CD to automatically pick up deprecation warnings?

Issue resolved!


Update status fix (2.5.z)

Signed-off-by: randybrunopiverger<EMAIL_ADDRESS>Regarding: https://github.com/stolostron/backlog/issues/22692

/hold
Manually testing

/unhold

/lgtm


change represent

Conflicts:
modules/s3db/dvr.py

"This branch has conflicts that must be resolved"
The comment is also wrong as this is "uncomment dvr_status field" rather than "change represent"

Sorry, my mind is confused. Yes "Uncomment dvr_status field"

It's already been done - hence the conflict ;)


msvs generator

It's pure awesomeness!
Plus you can generate project for any ninja file by doing bf -i anything.ninja -o build.ninja --msvs --no-core --no-env

Fixes #72


Bump up minimum android version to 9

This likely requires an additional tweak to .travis.yml (sorry I forget to mention that).

No worries, see if you can make it green and merge this :)


Error locating target 'diffusion_policy.env_runner.robomimic_image_runner.RobomimicImageRunner'

Hi, while trying to reproduce your work, I encountered the error below:
Error executing job with overrides: []
Error locating target 'diffusion_policy.env_runner.robomimic_image_runner.RobomimicImageRunner', see chained exception above.
full_key: task0.env_runner

when I try
from diffusion_policy.env_runner.robomimic_image_runner import RobomimicImageRunner

in terminal
I got
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/home/xieyingge/SDP/diffusion_policy/env_runner/robomimic_image_runner.py", line 25, in <module>
    from diffusion_policy.env.robomimic.robomimic_image_wrapper import RobomimicImageWrapper
  File "/home/xieyingge/SDP/diffusion_policy/env/robomimic/robomimic_image_wrapper.py", line 11, in <module>
    from robomimic.envs.env_robosuite import EnvRobosuite
  File "/home/xieyingge/SDP/robomimic/robomimic/envs/env_robosuite.py", line 23, in <module>
    MUJOCO_EXCEPTIONS = [mujoco_py.builder.MujocoException]
AttributeError: module 'mujoco_py' has no attribute 'builder'

I guess this may be related to the version of mujoco_py.Do you have a solution to this?

I have also encountered the same issue. Have you solved it?

我也遇到了同样的问题。你解决了吗？

No,I need help.(ಥ_ಥ)

我也遇到了同样的问题。你解决了吗？

No,I need help.(ಥ_ಥ)

Have you installed this?

我也遇到了同样的问题。你解决了吗？

No,I need help.(ಥ_ಥ)

Have you installed this?

Yes


Update sys-dm-xtp-gc-queue-stats-transact-sql.md

@sidesw1pe : Thanks for your contribution! The author(s) have been notified to review your proposed change.

#sign-off


array index out of range

08-23 18:31:07.916  28593-28744/com.emahapolitician.forchange V/ThinDownloadManager﹕ Response code obtained for downloaded Id 1 : httpResponse Code 200
08-23 18:31:07.916  28593-28744/com.emahapolitician.forchange D/ThinDownloadManager﹕ cleanupDestination() deleting /storage/emulated/0/PMK/Talking_Dog1.3gp
08-23 18:31:07.921  28593-28744/com.emahapolitician.forchange V/ThinDownloadManager﹕ Content Length: 698665 for Download Id 1
08-23 18:31:08.656  28593-28593/com.emahapolitician.forchange D/AndroidRuntime﹕ Shutting down VM
08-23 18:31:08.666  28593-28593/com.emahapolitician.forchange E/AndroidRuntime﹕ FATAL EXCEPTION: main
Process: com.emahapolitician.forchange, PID: 28593
java.lang.ArrayIndexOutOfBoundsException: length=2; index=4
at android.widget.AbsListView$RecycleBin.addScrapView(AbsListView.java:8664)
at android.widget.ListView.layoutChildren(ListView.java:1652)
at android.widget.AbsListView.onLayout(AbsListView.java:2627)
at android.view.View.layout(View.java:16695)
at android.view.ViewGroup.layout(ViewGroup.java:5328)
at android.support.v4.widget.SwipeRefreshLayout.onLayout(SwipeRefreshLayout.java:562)
at android.view.View.layout(View.java:16695)
at android.view.ViewGroup.layout(ViewGroup.java:5328)
at android.widget.FrameLayout.layoutChildren(FrameLayout.java:573)
at android.widget.FrameLayout.onLayout(FrameLayout.java:508)
at android.view.View.layout(View.java:16695)
at android.view.ViewGroup.layout(ViewGroup.java:5328)
at android.support.v7.internal.widget.ActionBarOverlayLayout.onLayout(ActionBarOverlayLayout.java:493)
at android.view.View.layout(View.java:16695)
at android.view.ViewGroup.layout(ViewGroup.java:5328)
at android.widget.FrameLayout.layoutChildren(FrameLayout.java:573)
at android.widget.FrameLayout.onLayout(FrameLayout.java:508)
at android.view.View.layout(View.java:16695)
at android.view.ViewGroup.layout(ViewGroup.java:5328)
at android.widget.LinearLayout.setChildFrame(LinearLayout.java:1702)
at android.widget.LinearLayout.layoutVertical(LinearLayout.java:1556)
at android.widget.LinearLayout.onLayout(LinearLayout.java:1465)
at android.view.View.layout(View.java:16695)
at android.view.ViewGroup.layout(ViewGroup.java:5328)
at android.widget.FrameLayout.layoutChildren(FrameLayout.java:573)
at android.widget.FrameLayout.onLayout(FrameLayout.java:508)
at android.view.View.layout(View.java:16695)
at android.view.ViewGroup.layout(ViewGroup.java:5328)
at android.view.ViewRootImpl.performLayout(ViewRootImpl.java:2319)
at android.view.ViewRootImpl.performTraversals(ViewRootImpl.java:2032)
at android.view.ViewRootImpl.doTraversal(ViewRootImpl.java:1191)
at android.view.ViewRootImpl$TraversalRunnable.run(ViewRootImpl.java:6642)
at android.view.Choreographer$CallbackRecord.run(Choreographer.java:777)
at android.view.Choreographer.doCallbacks(Choreographer.java:590)
at android.view.Choreographer.doFrame(Choreographer.java:560)
at android.view.Choreographer$FrameDisplayEventReceiver.run(Choreographer.java:763)
at android.os.Handler.handleCallback(Handler.java:739)
at android.os.Handler.dispatchMessage(Handler.java:95)
at android.os.Looper.loop(Looper.java:145)
at android.app.ActivityThread.main(ActivityThread.java:5942)
at java.lang.reflect.Method.invoke(Native Method)
at java.lang.reflect.Method.invoke(Method.java:372)
at com.android.internal.os.ZygoteInit$MethodAndArgsCaller.run(ZygoteInit.java:1388)
at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:1183)

The stack trace says the exception is in ListView.
java.lang.ArrayIndexOutOfBoundsException: length=2; index=4
at android.widget.AbsListView$RecycleBin.addScrapView(AbsListView.java:8664)
at android.widget.ListView.layoutChildren(ListView.java:1652)
at android.widget.AbsListView.onLayout(AbsListView.java:2627)

Where does it related to ThinDownloadManager ??

Ohh.. Would you please impact what can I do to avoid that. Coz there is no
any line number so I thought it is issue in library. And the issue comes
when I click download button of the download manager, the progress runs for
2-3 seconds, than Unfortunately stopped error, and this stacktrace. So I
can't even find my issue.
And off course this issue arise only on "click on download button, "
On Aug 24, 2015 11:56 AM, "Mani"<EMAIL_ADDRESS>wrote:

The stack trace says the exception is in ListView.
java.lang.ArrayIndexOutOfBoundsException: length=2; index=4
at android.widget.AbsListView$RecycleBin.addScrapView(AbsListView.java:8664)
at android.widget.ListView.layoutChildren(ListView.java:1652)
at android.widget.AbsListView.onLayout(AbsListView.java:2627)
Where does it related to ThinDownloadManager ??
—
Reply to this email directly or view it on GitHub
https://github.com/smanikandan14/ThinDownloadManager/issues/23#issuecomment-134055171
.

If you could share the code snippet in gist about how you are using the library and the usage of listview i can take a look at it.

//gist.github.com/DevendraSingh91/ad50bf13e35b3f1a764e

//gist.github.com/DevendraSingh91/ad50bf13e35b3f1a764e
this is the adapter gist which  am using
plz help me in sorting this issue

Hi,
In debug mode I got this issue,
The error code is 1004,n msg is IO.exception.
What should I do now,
On Aug 25, 2015 10:32 AM, "Mani"<EMAIL_ADDRESS>wrote:

If you could share the code snippet in gist about how you are using the
library and the usage of listview i can take a look at it.
—
Reply to this email directly or view it on GitHub
https://github.com/smanikandan14/ThinDownloadManager/issues/23#issuecomment-134474725
.

My bad! It was listview's error.


Refresh ACL after login

User login handling. If a user logs in pressing the login button, we need to re-fetch/introspect the catalog again and start the flow from beginning.
If a user logs in because of a session timeout or not permitted error, we need to identify a way to handle rights for all the entities that were rendered before. For now we won't be handling this case.

I am not sure whether this should be in chaise or ermrestjs..

User login handling. If a user logs in pressing the login button, we need to re-fetch/introspect the catalog again and start the flow from beginning.
This is already handled as the code refreshes the page when login button is pressed.

What about the second bullet?

We decided to leave the second bullet alone for now..


[CSL-849] remove MessageName, MessagePart

Bumped the time-warp-nt revision to one which doesn't have MessageName. It's replaced with MessageCode ~ Word16. This means the MessagePart motif that was used to get a whack of Message instances for free no longer applies, so that was removed. I could have updated it to use some similar scheme, like using the top 8 bits for the relay message constructor and the bottom eight for the particular content type. But I think it's better to have the message codes stated explicitly. I don't believe all of the codes listed in src/Pos/Communication/Message.hs are actually necessary. They were needed in order to compile, but surely some of the relay message instances don't need to be there.
So the length-prefixed MessageName is no longer used, which is the point of the ticket CSL-849, but VerInfo still contains arbitrarily-long HashMaps so the problem isn't yet solved.

I think that the HandlerSpec encoding table should be removed completely, as it's a) outdated and b) not necessary since the current encoding is much simpler than what we had before.

(Otherwise, LGTM.)

[ 70 of 205] Compiling Pos.Communication.Message ( src\Pos\Communication\Message.hs, .w\dist\ca59d0ab\build\Pos\Communication\Message.o )
C:\w\src\Pos\Communication\Message.hs:87:10: error:
    * No instance for (Buildable (InvOrData key TxMsgContents))
        arising from a use of `Node.Message.Class.$dmformatMessage'
    * In the expression:
        Node.Message.Class.$dmformatMessage @InvOrData key TxMsgContents
      In an equation for `formatMessage':
          formatMessage
            = Node.Message.Class.$dmformatMessage @InvOrData key TxMsgContents
      In the instance declaration for
        `Message (InvOrData key TxMsgContents)'
--  While building package cardano-sl-0.5.1

https://github.com/input-output-hk/cardano-sl/pull/1288/commits/736b4318fb2f598a886c4468f96e48f37d047b40
Includes a formatMessage definition for TxMsgContents that I had forgotten. Strange that my local build, and one of the travis builds, passed.

CI has failed because of this test: Richmen with stake distribution [0.24, 0.25, 0.26, 0.29] to test validity of @computeSharesDistr. I've seen this before, but it's nothing to do with these changes.

@avieth I've restarted the build. This failure in tests should fully disappear after #1275 is polished and merged.

There is a conflict now.

I have no idea what happened here -- if I click on the Details link Travis shows everything as green, but yet we get this (stale?) yellow dot here 🙉

Strange, I restarted one build, maybe it will help.

No clue why one of the travis builds failed. Can't see an error in the logs.

If I've untangled everything corretly I swear I have seen a timeout lurking at some point in the logs 🙊

@avieth there's a conflict already :(

there's a conflict already :(

I did the stack2nix thing but while it was going there must have been a commit of pkgs/default.nix to master. Rebased and ran it again and now CI is going.


Forseti not seeing IAM users or roles

Good day,
We are running into an issue where the IAM members and their roles are not being inventoried or not making it to the data model. When ever I run "forseti explainer list_members" it comes back as blank and when I run the command "forseti explainer list_permissions --roles roles/secretmanager.admin" it returns a blank array even through I know it exists because the inventory run output roles/secretmanager.admin as a role it saw. when ever I run forseti explainer list_resources I do see service account but not when I do list_members. not sure if service accounts should be under list_members or not.
Also when I created the IAM rule to not allow admin roles on service accounts and gave a test service account an admin role and ran the scanner against it there were no violations found.
Rule that doesn't allow service accounts to have admin roles:

Account that has the admin role:

What notrifier says:

We are running forseti version 2.25.0
issues seems to be with the model since scanner and explain don't show anything with roles or IAM even though I saw the roles being scanned in the output when the inventory was being created. Didn't see any users though
Deployed using Terraform to GCP
any logs that need to be uploaded I would rather send directly instead of posting here due to the potentially sensitive information that could be contained in them and don't want to post them here for the public to see

+1

Anybody have any ideas on this?

We have same issue. Only way to get explainer work again is to redeploy from scratch. Long life to Terraform.

Sorry, this was fixed by upgrading to the latest version using Terraform to redeploy

i just had the problem with v2.25.2.


Remove old release series admonition

OSG 3.4 went out of support last year so we should be able to drop this admonition

CI failure unrelated, merging


Error on example on common-programming-concepts -> functions" section

Hi!
I was just checking this section and found that the example function "plus_one" has a ";" at the end, so it is turning the function into a statement instead of an expression, which gives a fail when trying to run the code snippet.

update: I just read bad the book, it is an example of what NOT to do xd forget this :v


spray particle does not move

Description
it just does its animation in the same spot
Reproduction

spray something

Also related to them not deleting is the lifetime being NaN so something is going awry with trythrow probably. Might be my recent throwing change due to no mass or the likes.


SQL Explorer

Add SQL Explorer feature so that we can directly run LINQ query on DB.
Similar feature in Linqpad

Thanks for the suggestion!
However, I must say this is a low priority for me, so unless someone contributes the code, it won't happen any time soon :)

I would suggest plugin api and make plugins like these, because someone wants SQL Explorer, I want NoSQL explorer

Yep, that's already been suggested, but a plugin API also requires a lot of effort (designing a stable API, a repository, etc.), and that's another low priority for me right now. Maybe some day :)

I would be more incline to use RoslynPad if it had some way to connect to SQL Server DB.  Is there a plugin at least for it?

I'm just one developer doing this on my free time, so I have to prioritize, and unfortunately this is a feature not many have asked for, nor one I personally need.
I've not tried it, but you may be able to use EF Core's scaffolding tools to generate a DbContext and just #load it in RoslynPad.


Across Lite can't handle long clues

From some empirical testing, it looks like Across Lite crashes if any clue is longer than 505 characters, though other apps naturally work fine.
For better compatibility, we could truncate + ellipsize clues that are longer than this, and flag this in the unsupported features warning, when exporting to .puz. It may also make sense to have an option to turn this off, since not all apps have this problem; that said, I imagine if you're using an app other than Across Lite, I'd hope that it supports .jpz anyway, and the number of puzzles impacted by this should be relatively small (but non-zero).
Example, for testing: https://craftycrosswords.blogspot.com/2022/12/radicalized-by-apush.html (ref: 23-Across)

Fixed in https://github.com/jpd236/kotwords/commit/6f22ced28d2dddef6723c0a485c8cb4b563e417f


Not using $DOCKER_HOST I think

I'm trying blockade on OS/X where docker -d is running inside a vagrant VM (via https://github.com/noplay/docker-osx).
I have $DOCKER_HOST set, so normal docker commands work nicely.
When I run blockade up I get an error that looks like its only trying to connect via local sockets.
$ blockade up

Unexpected error! This may be a Blockade bug.

Traceback (most recent call last):
  File "/Library/Python/2.7/site-packages/blockade/cli.py", line 242, in main
    opts.func(opts)
  File "/Library/Python/2.7/site-packages/blockade/cli.py", line 109, in cmd_up
    containers = b.create()
  File "/Library/Python/2.7/site-packages/blockade/core.py", line 47, in create
    veth_device)
  File "/Library/Python/2.7/site-packages/blockade/core.py", line 60, in _start_container
    environment=container.environment)
  File "/Library/Python/2.7/site-packages/docker/client.py", line 347, in create_container
    return self.create_container_from_config(config, name)
  File "/Library/Python/2.7/site-packages/docker/client.py", line 354, in create_container_from_config
    res = self._post_json(u, data=config, params=params)
  File "/Library/Python/2.7/site-packages/docker/client.py", line 199, in _post_json
    return self._post(url, data=json.dumps(data2), **kwargs)
  File "/Library/Python/2.7/site-packages/docker/client.py", line 89, in _post
    return self.post(url, **self._set_request_timeout(kwargs))
  File "/Library/Python/2.7/site-packages/requests/sessions.py", line 377, in post
    return self.request('POST', url, data=data, **kwargs)
  File "/Library/Python/2.7/site-packages/requests/sessions.py", line 335, in request
    resp = self.send(prep, **send_kwargs)
  File "/Library/Python/2.7/site-packages/requests/sessions.py", line 438, in send
    r = adapter.send(request, **kwargs)
  File "/Library/Python/2.7/site-packages/requests/adapters.py", line 327, in send
    raise ConnectionError(e)
ConnectionError: UnixHTTPConnectionPool(host='localhost', port=None): Max retries exceeded with url: /run/docker.sock/v1.6/containers/create?name=blockade-eaf8721d19-c1 (Caused by <class 'socket.error'>: [Errno 2] No such file or directory)

Nearly 3 years later, DOCKER_HOST is finally fully respected by Blockade and we can close this ticket.
Until recently, it didn't matter too much since Blockade was required to be executed on the docker host as root, in order to run the tc/iptables commands. And our initial plans to do this via fabric/paramiko got messy and abandoned. But in 0.3.0, we moved these commands into containers of their own, with host networking. And in 0.3.1 (just now released), a lingering bug was fixed that prevented DOCKER_HOST from being consulted in one place.


Middleware: Deployment List Discrepancy Between MIQ and Hawklar-Services

The MIQ Deployment list displays Deployments that are not included in the Hawkular-Services Deployments list.
For example, on my test configuration Deployment "jboss-ejb-in-ear-ejb.jar" is displayed in the MIQ UI but is not on the Hawkular-Services Deployments list.
Configuration consisted of Hawkular-Services and Cassandra running in a container.
This issue has been reproduced on test configurations in addition to mine.

Do you have steps to reproduce ?

@miq-bot add_label providers/hawkular, bug

Repro Steps:

Add MW Provider, with Standalong and Domain EAPs under management.
Navigate to Deployments List
Get Deployments list via Hawkular API
Compare UI Deployments list to Deployments obtained via Hawkular-Services API (aka: https://github.com/ManageIQ/wrapanapi)

I tried to reproduce it but couldn't. So maybe I'm missing something.
I setup my environment like this

Hawkular Server with agent
EAP domain mode with EAP (on other box)

Started ManageiQ, added the middleware server and refreshed.
Compared MiQ deployment screen with Hawkular (using HawkFX)

Not sure what to suggest for repro. Let's keep this open for now and QE will attempt to reproduce.

@mattmahoneyrh any luck replicating this ? I also wasn't able to replicate.

@mattmahoneyrh is this still reproducible? if not, please close it

Closing.


test #17: Test OdsExporter.export()

Fixes #17
Created test exportSimpleOds2  which is a version of exportSimpleOds, adapted to the new StreamWriter2.export() method

Good


Doesn't work on Windows.

@jschueller It looks like it is build for windows but it doesn't seem to work. I can also see that Windows is excluded from testing. Any reason for that?
>>> import nlopt
ImportError: No module named 'nlopt'

yep, upstream does not provide windows bindings in a stable version yet

I see. Then either let it build an unstable version for Windows or disable the build for Windows.
We really don't want defective packages in the conda-forge channel.,

No, the c++ library is needed.

Hmm. I see the problem. Could c++ lib be split into its own package? Right now we are picking up this non-working package since conda-forge has priority in our setup. What is the @conda-forge/core preferences in these situations?

Is there a working package somewhere ?

@melund you should be able to make a C++-only version in a new package on staged-recipes

Is there a working package somewhere?
@jschueller .. I created some manually, for python2 last year (conda install -c melund nlopt). It was a hack as I pretty much converted Gohlke's wheel distribution of nlopt to a conda-package.

So I pretty happy to nlopt in conda-forge with python3 support.

@melund you should be able to make a C++-only version in a new package on staged-recipes
@scopatz Thanks. I will consider that if I don't don't find another workaround. But I would much rather help get the conda-forge to work.

@jschueller What is the issues that you see on windows? I can't find anything on unstable windows binaries on http://ab-initio.mit.edu/wiki/index.php/NLopt_on_Windows

by unstable I meant not in a release, full cmake support is in git, see https://github.com/conda-forge/nlopt-feedstock/pull/7.

@jschueller . Makes sense now.. @stevengj hasn't tagged new releases in GitHub for 2.5 years. And there is no response on the issues requesting it (https://github.com/stevengj/nlopt/issues/87).
Maybe it would be OK to use the CMake files on github to build the windows version. It seems like that is what Gohlke did for his nlopt 2.4.2 windows wheels.


chore: update terraform provider version

Purpose

All the sample's btp and cloudfoundry terraform provider versions have been updated
Pull Request Type
What kind of change does this Pull Request introduce?

[ ] Bugfix
[ ] Feature
[ ] Documentation content changes
[ ] Other... Please describe:

Other Information

closing this PR due to validation check failures, will be re-raised.


Link to commit on Github

What about linking the commit in the footer to the corresponding page on Github?

Codecov Report

Merging #5644 into master will decrease coverage by <.01%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #5644      +/-   ##
==========================================
- Coverage   37.81%   37.81%   -0.01%
==========================================
  Files         322      322
  Lines       47489    47489
==========================================
- Hits        17959    17957       -2
- Misses      26941    26943       +2
  Partials     2589     2589

Impacted Files
Coverage Δ

models/repo_list.go
63.29% <0%> (-1.27%)
:arrow_down:

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 6311e4c...4499427. Read the comment docs.


[doc issue] JavaScript Coding Guidelines

Issue description
I’m wondering why the official guidelines still promote patterns from the 80s, where development was done in raw text editors rather than in modern IDEs. In modern IDEs, type inference is actually possible—if not always accurate—whereas the old methods often lead to cluttered codebases.
There are numerous articles online pointing out the same issues, emphasizing that maintaining a clear codebase is preferable to enduring rare but confusing error messages. These errors can actually teach us to revise our code to fix critical issues.
When considering the Wikipedia article, there are more points against the old practices (12 cons) than benefits (10 pros). By simply following these numbers, you could argue that we are losing ground by introducing boilerplate code that leads to unnatural coding practices. Constantly revising code to comply with outdated rules—rules that the team has suggested moving beyond—only contributes to unnecessary clutter.
It would be beneficial for future developers to prevent such cluttering and improve code readability. Furthermore, I heard that you are migrating to TypeScript, so that’s kind of the direction it’s going anyway.
https://community.sap.com/t5/technology-blogs-by-sap/why-the-hungarian-notation-should-not-be-used-for-modern-sapui5-development/ba-p/13483943
Feedback Type (Optional)
None
Page Title on SAP Help Portal (prefilled)
No response
Page URL on SAP Help Portal (prefilled)
No response

Hi @punkratz312 , thanks for your contribution! Could you tell us which page of the UI5 developer guide you are referring to? Thanks.

JavaScript Coding Guidelines -> Naming Conventions
https://sapui5.hana.ondemand.com/#/topic/eded636b85584cd586b1fe231d2b5dac
Thanks for considering this 👍
Maybe it could be revised to transport the following, of course you can change the words to fit your needs.
Naming Conventions
We strongly discourage the use of Hungarian notation in modern JavaScript development, including SAPUI5. Hungarian notation, which prefixes variable names with their types, is widely regarded as an antipattern and poor practice. It introduces unnecessary complexity, making the code harder to read and maintain. Modern IDEs and tools already offer type information, rendering this approach obsolete.
Hungarian notation is particularly discouraged for API method parameters. Instead, rely on documentation or static type systems (e.g., TypeScript) for type checking.
Before (Hungarian Notation):
var sName = "John";
var iAge = 25;
var bIsActive = true;
var oUser = {
    sName: "Alice",
    iAge: 30
};

After (Modern Naming Convention):
var name = "John";
var age = 25;
var isActive = true;
var user = {
    name: "Alice",
    age: 30
};

Thank you, everyone! You’re saving so many codebases, and with OpenRewrite, it’s like it never happened—we're not stuck back in the '80s anymore. Seriously, I’m a senior developer now with 15 years of coding experience. I recognize and understand this pattern, and I’ve been fortunate enough not to have been burdened with this kind of code, even though I've worked on many projects. It’s reminiscent of some mainframe and COBOL stuff, so I consider myself lucky to have moved on and adapted. I only know about this pattern from the literature.
Regarding Hungarian notation (antipattern) #371:
Indeed, it would be good to have a recipe to phase out this practice.
We will provide the recipe, and you can align with the best practice. As it was just a recommendation, it’s easy to be wiser now and revise.
https://github.com/openrewrite/rewrite-static-analysis/issues/371

Hi @flovogt , I think this issue is in the domain of your dev team - what's your opinion?

@punkratz312 We will update the documentation and post an update here when we adjusted the respective sites.

Thank you once again for the constructive collaboration – together, we are building a foundation for clearer, more modern, and future-proof code! 🚀


0.35.13

Codecov Report

Merging #662 (ed4322c) into master (084eae9) will decrease coverage by 0.18%.
The diff coverage is 74.03%.

@@            Coverage Diff             @@
##           master     #662      +/-   ##
==========================================
- Coverage   93.71%   93.52%   -0.19%
==========================================
  Files         112      112
  Lines       21412    21605     +193
==========================================
+ Hits        20067    20207     +140
- Misses       1345     1398      +53

Impacted Files
Coverage Δ

src/lib/cache.rs
57.62% <ø> (ø)

src/lib/mod.rs
100.00% <ø> (ø)

src/lib/command.rs
73.33% <11.90%> (-21.03%)
:arrow_down:

src/lib/descriptor/mod.rs
87.94% <25.00%> (-0.66%)
:arrow_down:

src/lib/scriptengine/mod.rs
90.00% <50.00%> (-3.58%)
:arrow_down:

src/lib/toolchain.rs
90.62% <77.27%> (-7.00%)
:arrow_down:

src/lib/types.rs
86.79% <78.57%> (-0.10%)
:arrow_down:

src/lib/cli.rs
90.83% <100.00%> (+0.33%)
:arrow_up:

src/lib/execution_plan.rs
93.77% <100.00%> (ø)

src/lib/toolchain_test.rs
96.61% <100.00%> (+0.53%)
:arrow_up:

... and 4 more

Continue to review full report at Codecov.

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