[BUG] Xiaomi Magic Cube battery and multistate_input errors

Describe the bug

battery info is reported with negative values. As battery cannot have negative values, I think that we should get always ABSOLUTE value of battery report. We can create a template sensor, but it's not worth it since negative values are just wrong, in any case
multistate_input stay "UNKNOWN". I can see "analog_input" updating with degrees of rotation, but not multistate_input

Screenshots

Additional context

do you record battery levels? Is it always negative or is it jumping?
If jumping, how often does it go negative?
@dmulcahey is there any data reported on the multistate input cluster/

The actions are... rotate, slide, knock etc. Should be mapped to text sensors in HA

i'm getting  all the zha_events
I guess the question is: should we really keep state and have entities for it? to my taste cube is more of a stateless remote type

@nerdosity there won't be any meaningful data in the multistate_input,  as  the attribute updates always comes with data of 0x0103 depends on the action
To catch the  useful data, subscribe or base your automations based on zha_events

The decision was made to drop those completely.  Use zha_events instead

Or device actions... they’re much more user friendly

And I can't reproduce the battery problem

Does  it still report a negative value? if it does and it always report a negative value, would need a debug log with attribute report from the device in question. Enable debug log as specified in integration documentation and wait till it updates. You could search for zhaquirks.xiaomi and around it you should see something like:
2020-06-12 00:04:52 DEBUG (MainThread) [bellows.zigbee.application] Received incomingMessageHandler frame with [<EmberIncomingMe
ssageType.INCOMING_UNICAST: 0>, <EmberApsFrame profileId=260 clusterId=0 sourceEndpoint=1 destinationEndpoint=1 options=256 grou
pId=0 sequence=238>, 255, -51, 0x458e, 255, 255, b'\x18\x9d\n\x01\xffB%\x01!\xbd\x0b\x04!\xa8\x13\x05!\xea\x00\x06$\x01\x00\x00\
x00\x00d)\xfc\x07e!\xdd\x1af+\x9c\x88\x01\x00\n!\xd1\xa7']
2020-06-12 00:04:52 DEBUG (MainThread) [zigpy.zcl] [0x458e:1:0x0000] ZCL deserialize: <ZCLHeader frame_control=<FrameControl fra
me_type=GLOBAL_COMMAND manufacturer_specific=False is_reply=False disable_default_response=True> manufacturer=None tsn=157 comma
nd_id=Command.Report_Attributes>
2020-06-12 00:04:52 DEBUG (MainThread) [zigpy.zcl] [0x458e:1:0x0000] ZCL request 0x000a: [[<Attribute attrid=65281 value=<TypeVa
lue type=CharacterString, value=^A!�^K^D!�^S^E!�>>]]
2020-06-12 00:04:52 DEBUG (MainThread) [zigpy.zcl] [0x458e:1:0x0000] Attribute report received: 65281=^A!�^K^D!�^S^E!�
2020-06-12 00:04:52 DEBUG (MainThread) [zhaquirks.xiaomi] 00:15:8d:00:02:2c:b7:75 - Attribute report. attribute_id: [65281] value: [{'battery_voltage_mV': 3005, 'X-attrib-4': 5032, 'X-attrib-5': 234, 'X-attrib-6': 1, 'temperature_measurement': 2044, 'humidity_measurement': 6877, 'pressure_measurement': 100508, 'path': 42961, 'battery_level': 200}]

need to see the raw data, parsed result and state changed event

And I can't reproduce the battery problem

Does it still report a negative value?

Nope, I gave a look this morning.
Every sensor and device that had negative values yesterday now reports correct battery value, it seemed just an initial issue. After around one hour, battery value went > 0.

I don't know if this is something quirk related or something else.

Dunno. What page is it from? Seems like either UI or translation error.

Dunno. What page is it from? Seems like either UI or translation error.

It's the Settings -> Devices -> Magic cube device page -> clicking "+" sign on automations panel -> popup

then must have been missing translations for device triggers/actions


Example Body and Payload not Generated in pan.dev

Issue:
Some of the post methods in CSPM API documentation do not contain example body and payload although request body schema is present.
Here are a few example API links with missing payload and body:
https://pan.dev/prisma-cloud/api/cspm/create-adoption-advisor-config/
https://pan.dev/prisma-cloud/api/cspm/post-alerts/
https://pan.dev/prisma-cloud/api/cspm/get-alerts-remediation/
https://pan.dev/prisma-cloud/api/cspm/get-resource-info-filter-options/
Screenshot:

Hi @AbiMano4688, as mentioned in a separate thread, removing ; CHARSET=UTF-8 from the MIME type would likely allow the schemas to be generated.
To be clear, that would entail changing application/json; charset=UTF-8 to simply application/json. The default encoding for JSON is utf8 so including it in this case may not be necessary.

Hi @sserrata
As discussed over Slack, identifying and modifying all the existing occurrences of charset in the spec file might be tedious. I would appreciate it if this could be fixed at your end in the upcoming release.

Hi @AbiMano4688, please see a preview of the fix here: https://pan-dev-f1b58--pr349-twzgaijd.web.app/prisma-cloud/api/cspm/create-adoption-advisor-config/

Hi @sserrata
The update looks good. Thanks for sharing the preview. When will this fix be merged?

@AbiMano4688 , it seems like was merged last week in https://github.com/PaloAltoNetworks/docusaurus-openapi-docs/pull/643, can you please confirm on pan.dev?

Yes, @Pubs-MV. I don't see the changes in the live site. @sserrata, are these changes merged to the master?
CC: @manukumar6

Hi @AbiMano4688, the change was merged just now and should be live in the next 40 minutes.

Thank you, @sserrata. The body appears in sample example.


Handling Catch All Routes

Love the type-safe routes!
However, I am having issues using the features nextjs-routes makes available with Catch All Routes:
https://nextjs.org/docs/api-routes/dynamic-api-routes#catch-all-api-routes
This comes around due to some SDKs requiring catch alls or optional catch alls:
https://github.com/auth0/nextjs-auth0#add-the-dynamic-api-route
Probably not an easy one to fix, but would be really powerful to be able to handle catch-all routes.

Hey @JameelKhan9 could you share the error you’re seeing and your Nextjs-routes.d.ts file?

@JameelKhan9 This is already supported. Feel free to reopen this with reproduction steps if you're still seeing this issue.

@tatethurston I just had another look, and I managed to get it to work. I think the usage of catch-all routes wasn't as intuitive the first time I looked at it.
Cheers!


Added methods documentation for ACNV.

-Added sections on segment union, small-segment union, similar-segment merging, and copy-ratio model (i.e., all the most heuristic parts of ACNV).
-Added names of relevant command-line tools for the somatic workflow.
-Other minor edits throughout.
Note that I changed the author list to alphabetical when adding @mbabadi, although I see that @davidbenjamin had it differently in his PR.  I have no strong opinions here...
Closes #163.

@samuelklee I can review this PR.

Everything looks good to me.


[BUG] Tenant filter is not being applied.

Tenant filter is not being applied (only Identity schema).
    modelBuilder.ApplyGlobalFilters<IMustHaveTenant>(b => EF.Property<string>(b, nameof(Tenant)) == Tenant);

Fix:
modelBuilder.ApplyGlobalFilters(b => b.Tenant == Tenant);

It's probably due to the nullability.
Can you try if this works?
 modelBuilder.ApplyGlobalFilters<IMustHaveTenant>(b => EF.Property<string?>(b, nameof(Tenant)) == Tenant);
Mind the ? in EF.Property<string?>  ;-)

I too notice this same issue now. Brand search shows all the brands of every tenants. Tried to make the string nullable. Doesnt help. Will check.

Fixed. Please pull the latest code. Drop the existing db. Re-run the API and test again. Do let me know.

i guess the bug still exists when multiple tenants use the same db..hmm

Hmm, I'm trying to understand the code. and I don't think it can work the way it's implemented.
services.AddDbContext is called for every tenant.
I think there will be only one ApplicationDbContext registered, regardless of how many times you call that method (not sure if it will be the first one or the last one).
How does ApplicationDbContext know which database to connect to?
I might be missing something...

The OnConfiguring method in AppDbContext takes care of that. This part works though. When tenants are on different databases, it works. But when tenants share a database, the filtering doesn't quite work. It was working earlier though. Going through old code.

fix added.

Able to filter tenants wise data when tenants are on different databases - tested
Able to filter tenants wise data when multiple tenants are using the same database - tested

So, if I see it correctly, the fix was to move
modelBuilder.ApplyGlobalFilters(b => EF.Property(b, nameof(Tenant)) == Tenant);
down 2 lines?
How can you explain this?
Better add a comment that it has to stay there, otherwise it might get unknowingly moved again with a refactor?

Apparently, the queryFilter Execution line matters. I guess the further lines somehow clear the added filters. Have to debug more into the GLobalFilter extension method.
And yes, b => b.Tenant == Tenant should work too.

Ok, so that means the ISoftDelete might not be working now? Or the IIdentityTenant? Why does that last one exist btw? Why not also use IMustHaveTenant for that?

I chased it down (see also my comments on discord).
See here for what needs to happen in ApplyGlobalFilters...
But I would suggest adopting that FinBuckle.MultiTenant library actually, in stead of trying to implement that all ourselves ;-)

@fretje do check this commit. Now it gets the list of existing filters of an entity and appends the newer expression onto it. Similar to method chaining.

I added a comment in the commit... not sure where that shows up then as it's not connected to any PR or issue. Anyways, here it is.

The best would be of course to write some tests for it ;-)
But in the long run I would go for that FinBuckle library. Otherwise it seems like doing double work... they probably already have tests for exactly that and all the other stuff that needs to be taken care of... I also like the fact that you don't necessarily need to have a "TenantKey" property on every entity that needs to be multitenant.

Yeah, Finbuckle seems well built out. Need to figure out how to switch to this package without changing too much of the code and implementation.


[BUG] PhotoView when used with Hero(), loading widget is shown for cached image also, screen flashes with white overlay

Describe the bug
A clear and concise description of what the bug is.
To Reproduce
Steps to reproduce the behavior.
I have used PhotoView within my Hero() class and whenever that class is getting called, the loader widget is getting shown for the cached image (CachedImageNetworkProvider) too also, the screen flashes with some kind of white color.
For example, a list of images is loaded in a listview, whenever a click is performed on the item, the item is loaded using Hero() function, if the image isn't loaded, the loading widget is shown which is good. Once the image is loaded, the image is shown. But when that same item is clicked again, the loader widget is shown for some seconds and then image loads and screen flashes.
My Widget class source:
import 'dart:async';
import 'dart:convert';

import 'package:cached_network_image/cached_network_image.dart';
import 'package:flutter/foundation.dart';
import 'package:flutter/material.dart';
import 'package:dpbattle/helper/codecyan.dart' as CODECYAN;
import 'package:http/http.dart' as http;
import 'package:dpbattle/helper/photo_pojo.dart';
import 'package:flutter_cache_store/flutter_cache_store.dart';
import 'package:photo_view/photo_view.dart';

Future<List<PhotoPojo>> fetchPhotos(http.Client client) async {
  final response =
      await client.get('https://jsonplaceholder.typicode.com/photos');
  return compute(parsePhotos, response.body);
}

// A function that converts a response body into a List<Photo>.
List<PhotoPojo> parsePhotos(String responseBody) {
  final parsed = json.decode(responseBody).cast<Map<String, dynamic>>();
  return parsed.map<PhotoPojo>((json) => PhotoPojo.fromJson(json)).toList();
}

Widget homePhotosBody() {
  return FutureBuilder<List<PhotoPojo>>(
    future: fetchPhotos(http.Client()),
    builder: (context, snapshot) {
      if (snapshot.hasError) print(snapshot.error);

      return snapshot.hasData
          ? _photoBuild(photos: snapshot.data)
          : Center(child: CODECYAN.LOADER_WIDGET);
    },
  );
}

Widget _photoBuild({@required List<PhotoPojo> photos}) {
  return GridView.builder(
    gridDelegate: SliverGridDelegateWithFixedCrossAxisCount(
      crossAxisCount: 3,
    ),
    itemCount: photos.length,
    itemBuilder: (context, index) {
      return InkWell(
          onTap: () => Navigator.push(context, MaterialPageRoute(builder: (_) {
                return ShowFullScreen(index: index, image: photos[index].url);
              })),
          child: Hero(
              tag: 'photos_' + index.toString(),
              child: Padding(
                  padding: EdgeInsets.all(15),
                  child: CachedNetworkImage(
                      imageUrl: photos[index].thumbnailUrl))));
    },
  );
}

class ShowFullScreen extends StatelessWidget {
  final int index;
  final String image;

  ShowFullScreen({@required this.index, @required this.image});

  @override
  Widget build(BuildContext context) {
    return Scaffold(
      backgroundColor: Colors.black,
      body: GestureDetector(
        child: Hero(
          tag: 'photos_' + index.toString(),
          child: PhotoView(
            loadingChild: CODECYAN.LOADER_WIDGET,
            imageProvider: CachedNetworkImageProvider(image),
            maxScale: 3.0,
            minScale: 0.6,
          ),
        ),
        onTap: () {
          Navigator.pop(context);
        },
      ),
    );
  }
}

Please check the video for a clear understanding.
Preview Video
View video

no update on this ? 👎

Use should use PhotoViewHeroAttributes instead of wrapping the whole PhotoView widget with a hero.

Use should use PhotoViewHeroAttributes instead of wrapping the whole PhotoView widget with a hero.
Edit:
See how to use it here:
https://pub.dev/documentation/photo_view/latest/photo_view/PhotoView-class.html

Great, thanks. Actually, I couldn't find the proper documentation. Thank you. <3

Sorry to let you know, but this isn't working. Your suggestion suggests something else rather than fulfilling my expectations.
You suggested something else. This is a bug. Please check my issue again. Thanks.

Have you tried to use PhotoViewHeroAttributes  ?
When it is set, we do no even show the loading child.

Have you tried to use PhotoViewHeroAttributes  ?
When it is set, we do no even show the loading child.

perfect, working like a charm. I tried with PhotoViewHeroAttributes earlier but I think i missed something that time which caused the issue, but now it's perfect. Thanks a lot <3


Fix incorrect page navigation on load

fixes https://github.com/fusetools/fuselibs-public/issues/756
This PR contains:

[ ] ~Changelog~
[ ] ~Documentation~
[x] Tests

This is good!


CInputCheckbox custom do not 'checking'

Hello,
I want to make a table with checkbox element. I need to check item in two ways. Clicking on a row and checking a checkbox. I've notice that in custom mode checkbox do not 'checking'.
Take a look, please
https://codesandbox.io/s/cinputcheckbox-custom-bug-lfx2t

Operating system and version (Windows 10)
Browser and version (Opera 68.0.3618.125)

Scoped slots cannot be used to update "items"
this way it works: https://codesandbox.io/s/cinputcheckbox-custom-bug-vqt6d?file=/src/App.vue


Wrap v8::Object::GetPrivate

v8 renamed v8::Object::SetHiddenValue/GetHiddenValue/DeleteHiddenValue to v8::Object::GetPrivate/SetPrivate/DeletePrivate (and added HasPrivate) and changed the return types from Local to MaybeLocal.  Seems like this is a good candidate for wrapping with nan, especially since the very latest versions of v8 seem to have deleted the old SetHiddenValue versions, which will break a lot of my modules.  (This change hasn't quite made it into node yet AFAICT.)

Since that change will break ABI compatibility, Node will not use it until v7, which is scheduled for some time in autumn. This will be taken care of then.

So unfortunately electron (as of 1.3) has already jumped to v8 5.2.x.y, in which this is broken, and also seems to have a different signature (key/value, instead of context/key/value).

Obviously, this is not set up using the global exported version constants and is not out in it's own file or anything of the sort, but the gist of what we've ended up with in nodegit to fix this is as such
#if (NODE_MODULE_VERSION > 48)
  v8::Local<v8::Value> GetPrivate(v8::Local<v8::Object> object,
                                      v8::Local<v8::String> key) {
    v8::Isolate* isolate = v8::Isolate::GetCurrent();
    v8::Local<v8::Context> context = isolate->GetCurrentContext();
    v8::Local<v8::Private> privateKey = v8::Private::ForApi(isolate, key);
    v8::Local<v8::Value> value;
    v8::Maybe<bool> result = object->HasPrivate(context, privateKey);
    if (!(result.IsJust() && result.FromJust()))
      return v8::Local<v8::Value>();
    if (object->GetPrivate(context, privateKey).ToLocal(&value))
      return value;
    return v8::Local<v8::Value>();
  }

  void SetPrivate(v8::Local<v8::Object> object,
                      v8::Local<v8::String> key,
                      v8::Local<v8::Value> value) {
    if (value.IsEmpty())
      return;
    v8::Isolate* isolate = v8::Isolate::GetCurrent();
    v8::Local<v8::Context> context = isolate->GetCurrentContext();
    v8::Local<v8::Private> privateKey = v8::Private::ForApi(isolate, key);
    object->SetPrivate(context, privateKey, value);
  }
#else
  v8::Local<v8::Value> GetPrivate(v8::Local<v8::Object> object,
                                      v8::Local<v8::String> key) {
    return object->GetHiddenValue(key);
  }

  void SetPrivate(v8::Local<v8::Object> object,
                      v8::Local<v8::String> key,
                      v8::Local<v8::Value> value) {
    object->SetHiddenValue(key, value);
  }
#endif

called like
Local<v8::Value> checkValue = Nan::GetPrivate(nodeObj, Nan::New("BlahBlah").ToLocalChecked());

I wouldn't reject patches for 5.2 out of hand but there is no way for us to tes them (I think.  Perhaps building against Electron could be tested but running the suite on the CI?  Probably not.)

So, I'm aware of how to build the module cleanly for electron, I'll take a look at how the tests for this run and see if I can work up an electron solution for the short term :+1:

New methods could be used already for node 6.x, running V8 v5.0.x.


Gracefully exit when server is down

Currently the app crashes when the server is down. It should just gracefully exit instead.

Resolved in 4568718.


Adding Hjerson external metric

The changes includes:

externalMetrics/app/config/config.neon - I've added new metrics and setup connection to mysql
externalMetrics/app/templates/Tasks/list.latte  - I've updated color generation (so now it depends on the number of metrics and you don't need to add a color manually when adding a new metric)
externalMetrics/external/hjerson+.py - external metric script
externalMetrics/libs/Metrics/Hjerson.php - php-wrapper for external metric, I'll try to add some info to wiki today
externalMetrics/www/css/bootstrap.min.css - small change, 1024px width instead of 940px, so all 8 metrics values fit (I'd propose to change it to % value)
externalMetrics/schema.sql - file is now compatible with MySQL and has 4 more metrics

Fixes #14


feat(offcanvas): clicking input element in list does not trigger close

Currently including an input in the offcanvas menu is impossible, as clicking on any element in the menu closes it. This PR makes sure the click event target is not an input element before triggering the menu to close.

I don't think this solution is resilient enough. For example, what if we have a <textarea> in the offcanvas menu? What if we have a <select>? The default use case for offcanvas is for menus. If you're using it for something else, you may be better served by creating a custom directive that swallows up click events.

Is it so uncommon to have a search field in a menu though? My example use case is using the off-canvas to replace a topnav on mobile. The Zurb Foundation docs site has their search field in the menu (though it's not off-canvas, it's still a menu).
What do you think of checking if the clicked element has an attribute, like prevent-close="true", before closing the menu? I feel like creating a directive that swallows up the event could have unintended consequences with anything else that might be listening for clicks on the input element.

Even by modifying the click event to avoid closing if element is a input, on a mobile when the keyboard appears the menu is closed...
An idea to prevent this behavior?

OK my solution is, make the same modification then vitosamson like this:
$scope.hide = function () {
sidebar.removeClass('move-left');
sidebar.removeClass('move-right');
}
replace by:
$scope.hide = function (e) {
if (e.target.tagName.toLowerCase() != 'input') {
sidebar.removeClass('move-left');
sidebar.removeClass('move-right');
}

Here's my solution:
Change this:
$scope.hide = function (e) {
	sidebar.removeClass('move-left');
	sidebar.removeClass('move-right');
};

By:
$scope.hide = function (e) {
	if (typeof(e) != "undefined") {
		if (e.target.tagName.toLowerCase() != 'input') {
			sidebar.removeClass('move-left');
			sidebar.removeClass('move-right');
		}
	}
	else {
		sidebar.removeClass('move-left');
		sidebar.removeClass('move-right');
	}
};


check the length of the container statuses

Signed-off-by: cwen0<EMAIL_ADDRESS>What problem does this PR solve?
fix # https://github.com/pingcap/chaos-mesh/issues/157
What is changed and how does it work?
check the length of the containers statuses before using the states of cotnainer

Codecov Report

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

@@           Coverage Diff           @@
##           master     #177   +/-   ##
=======================================
  Coverage   39.47%   39.47%
=======================================
  Files          17       17
  Lines         608      608
=======================================
  Hits          240      240
  Misses        337      337
  Partials       31       31

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 b106854...4613209. Read the comment docs.


NovelRingan not show cover image

Source information
https://github.com/LNReader/lnreader/blob/main/src/sources/id/novelringan.js
Steps to reproduce
open source in app
Expected behavior
showing cover image
Actual behavior
not show cover image
LNReader version
1.1.8
Android version
Android 9
Other details
No response
Acknowledgements

[X] I have searched the existing issues and this is a new ticket, NOT a duplicate or related to another open issue.
[X] I have written a short but informative title.
[X] I have updated the app to version 1.1.8.
[X] I have updated all installed extensions.
[X] If this is an issue with the app itself, I should be opening an issue in the app repository.
[X] I will fill out all of the requested information in this form.

How is the app supposed to show the cover images if the website does't have them?

they have a cover image..

The covers don't load for me. I'll still look into it.


How to control the tab size for different type of files?

monaco-editor npm version: 0.7.0
Browser: Chrome
OS: macOS 10.11
I found sometimes the tab size is 2, sometimes the tab size is 4, even is one file, the tab size is different. Is there a API to set tab size for different files?

I think you might be looking for updateOptions where you can pass in options for tabSize.
Additionally, there is also

detectIndentation
getOneIndent
normalizeIndentation

@philcchen Thanks, I'll check it.

@philcchen I'm still a little confused. It seems there is no direct option for updateOptions to set tabSize.


Bug 1871045: bump(openshift/api)

Get new v1 CRDs with logLevel documentation and default.

/hold
This is only a proof PR using a go.mod replace directive until https://github.com/openshift/api/pull/720 merged.

Empty job logs 😞
/retest

/retest

/retest

Infra 503.
/retest

/lgtm

/lgtm


Would it be possible to add access control for the global namespace?

At the moment it seems that only admin users can push to the global namespace - but we do not want to make the persons who should maintain the global namespace admins.. So is there any way how to adjust the rights on the global namespace to control which user / group / team can push there?

but we do not want to make the persons who should maintain the global namespace admins

Anyone that "maintains" is an admin, no ? 😉

So is there any way how to adjust the rights on the global namespace to control which user / group / team can push there?

No, you have to be admin in order to write and administer the global namespace. Note that the global namespace is quite special, and only a handful of people should be able to write into that. What's your use case ? And what's your recommended workflow to fix your use case ?

Anyone that "maintains" is an admin, no ? 😉

Unfortunately not - some people should manage the users and teams and namespaces - some others should push images. These are not the same persons.

What's your use case ?

The challenge is that the admin can do much  more than just upload to the global namespace.
We need a person that can only upload to the gobal namespace and can not do all the other administrative things within portus..
So similar to the rights management on custom created namespaces we would also have the global namespace to get a team assigned and members of that team should be able to push to this namespace.

I believe this is already handled in the user permission issues we are working on the 2.4 milestone.

So in 2.4 non-admin users will be able to push images to the main namespace ?


Remove devise & implement SSO simply via omniauth

This is not ready to merge yet, and the tests are broken, but it does work.
@alan I think you'll agree it is vastly simpler than the layers of complexity introduced by devise + omniauth.

This branch is now functionally complete, but tests need attention, and the code could be refactored. Refactoring can wait from my perspective.


Linera net up and client refactoring

Motivation
Local development on Linera requires using ./scripts/run_local.sh. This requires users to have the linera-protocol repository checked out locally.
Solution
Introduce linera net up which starts a local network and provides instructions on how to configure the wallet locally.
To achieve this some wrappers around our binaries were pulled out into linera-service/src/client.rs.

Please rebase so that we have CI

became #951


feat: [NPM] Update Linux IPSet Management

For Applying IPSets:

use ipset save file to add/delete members
update error handling logic to skip previously run lines in a restore file

Other changes:

Update some logging for iptables chain management

TODO:

move some UT code into smaller-scoped functions
use grep to only look at azure-npm sets for ipset save

One comment: naming (file-creator.go and its contents) is misleading if I correctly understand the code. IPset and iptables use stdout  to program them. Initially, I am looking for where is file.

One comment: naming (file-creator.go and its contents) is misleading if I correctly understand the code. IPset and iptables use stdout to program them. Initially, I am looking for where is file.

Open to suggestions. How about restore-file-creator.go?

One comment: naming (file-creator.go and its contents) is misleading if I correctly understand the code. IPset and iptables use stdout to program them. Initially, I am looking for where is file.

Open to suggestions. How about restore-file-creator.go?
May be restoreLinuxDP.go since both iptables and ipset uses this for mainly restore, but your suggestion is ok.
Just having file makes me confused, but I may be so sensitive.

One comment: naming (file-creator.go and its contents) is misleading if I correctly understand the code. IPset and iptables use stdout to program them. Initially, I am looking for where is file.

Open to suggestions. How about restore-file-creator.go?
May be restoreLinuxDP.go since both iptables and ipset uses this for mainly restore, but your suggestion is ok.
Just having file makes me confused, but I may be so sensitive.

ok I'll call it restore_linux.go. Are you ok with the name FileCreator for the struct?

One comment: naming (file-creator.go and its contents) is misleading if I correctly understand the code. IPset and iptables use stdout to program them. Initially, I am looking for where is file.

Open to suggestions. How about restore-file-creator.go?
May be restoreLinuxDP.go since both iptables and ipset uses this for mainly restore, but your suggestion is ok.
Just having file makes me confused, but I may be so sensitive.

ok I'll call it restore_linux.go. Are you ok with the name FileCreator for the struct?

No strong opinion.

/azp run

/azp run


Fix the related array in people.json so that it chooses similar people as the related options

1a7349e4b6101b6d9048352e7875db690d2be2e1


Noted that CCMSETUP still runs on clients in the exclusion collection.

Fixes #967.

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

#sign-off

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

Resolved merge conflict (two blank lines); merging.


chore: Disable temporally AzPipeline WI e2e

Checklist

[x] Commits are signed with Developer Certificate of Origin (DCO - learn more)

/run-e2e

/run-e2e aws


Update App.js

Added a feature for uploading image files only to the firebase storage.

Please raise the PR against this branch


Observable available in store2?

Hi,
Sorry for my bad englisch before ... :)
Currently we evaluate whether we could use store2 in our projects.
For better understanding our requirements:
The pages in our applications are based on multiple web-apps based on Vue.js, React or Vanilla, etc ... at same time.
We need a store for our application states.
My question is:
Is there a functionallity in your library to use "Observables" that any applications will triggert automatically by changes where variables from the store are used?
[EDIT]
Another improvement will be to have promises that I can sure that my value is updatet successful in my store like this:
window.store.transact('test', (obj) => {
    obj.test++;
}).then((updatedValue) => {
    window.EventEmitter3.emit("button.clicked", window.store('test'));
    // or ...
    window.EventEmitter3.emit("button.clicked", updatedValue);
);

... same fpr set() or setAll() ...
If I am blind or do not understand your documentation then "sorry" so far. :)
Best regards,
Mario

If you are talking about multiple user-interfaces communicating with each other, you first need to clarify whether these Vue/React/Vanilla webapps are open in the same browser window or tab or different ones. If they are open in the same, then no, i don't have any support for watching changes because store() uses localStorage which is a browser native object. I doubt it is even possible to observe that like you would a normal object.
If you are talking about webapps open in different windows or tabs, then i do have support for that via an extension called store.on.js. This is possible because browsers provide an event listener hook to tell when a different window has made a change to the localStorage. It does not fire events for changes within the same window, though.
As for returning Promises, you first need to understand that localStorage is purely synchronous. You do not need to wait for Promise resolution. When you make any call to store(), it is executed synchronously. No promise necessary. You can be sure your update is successful as long as no quota error was thrown. If, however, you are working with an API that requires you to speak in Promises, there is an extension called store.async.js that may prove convenient. This uses setTimeout to wrap your store() call execution to force it to be asynchronous. In my opinion, it has no serious use, but if you really want it, it is there.

Hi nbubna,
Many thanks for your feedback. :+1:
Per default our app is running in one window.
But if its possible with your "on" extension to use multiple browser windows or tabs then it is very very nice for us and a better usabillity in our application. :1st_place_medal:
Ok ... then I think we have to solve our problem by another way.
But if you say all is syncronous then the following code should be work correct ... right?
let value = window.store("test");
window.store.set("test", value++);
window.EventEmitter3.emit("button.clicked", window.store("test"));

When YES, then I have another problem in my logic. :)
Mario

Yes, all localStorage operations (and sessionStorage) are purely synchronous, so store() is also.
Now that i have someone with an actual use case for detecting store() changes in the same window, i may take some free time to see if i can't create a solution for detecting those changes as well. You should not wait on that, though, as it is a busy time for me and it would be an experiment. I do wish the browsers would fire storage events for localStorage changes in the same window, but they do not. :(

No Problem.
Take your time. :)
Maybe a solution could be an observable based on the https://www.npmjs.com/package/rxjs package. However you can it integrate in your lib. :)
Wish you luck, :)
Thank you and best regards,
Mario


added ability to redirect() function to redirect back

Hi ,
i added ability to redirect() function to redirect to the previous page.
useful when working with forms.
This can be very useful for fellow developers as well.
Kindly merge this pull request.

How's that better than just outputting a link to the previous page?

@narfbg you don't have to update the redirect function argument if you changed the method name anyways.

OK, to rephrase ... how's it better than outputting a link to $_SERVER['HTTP_REFERER']?
On another note, the 'referer' value is provided by a request header (meaning it comes from the client) and it may not exist, in which case we'd have a broken feature ... which is why I can't merge this.


How to install a certificate without IPV6?

I'm trying to generate and install a certificate on Windows Server 2012 R2.  However, I keep getting fails:

The global logger has been configured
Let's Encrypt (Simple Windows ACME Client)
Renewal Period: 3650
Certificate Store: WebHosting
ACME Server: https://acme-staging.api.letsencrypt.org/
Config Folder: C:\Users\Lance\AppData\Roaming\letsencrypt-win-simple\httpsacme-staging.api.letsencrypt.org
Certificate Folder: C:\Users\Lance\AppData\Roaming\letsencrypt-win-simple\httpsacme-staging.api.letsencrypt.org
Loading Signer from C:\Users\Lance\AppData\Roaming\letsencrypt-win-simple\httpsacme-staging.api.letsencrypt.org\Signer
Getting AcmeServerDirectory
Loading Registration from C:\Users\Lance\AppData\Roaming\letsencrypt-win-simple\httpsacme-staging.api.letsencrypt.org\Registration
Authorizing Identifier merge.taffy.cn Using Challenge Type http-01
Writing challenge answer to C:\inetpub\wwwroot.well-known/acme-challenge/ygKJNxKyp5j57TGvl3X82sd8mee6Oo0aZOWHUp_eKHE
Answer should now be browsable at http://merge.taffy.cn/.well-known/acme-challenge/ygKJNxKyp5j57TGvl3X82sd8mee6Oo0aZOWHUp_eKHE
Submitting answer
Refreshing authorization
Refreshing authorization
Refreshing authorization
Refreshing authorization
Refreshing authorization
Refreshing authorization
Authorization Result: invalid
Authorization Failed invalid

I've verified that the auth file IS browsable at: http://merge.taffy.cn/.well-known/acme-challenge/ygKJNxKyp5j57TGvl3X82sd8mee6Oo0aZOWHUp_eKHE  However, it only works over IPV4.  In looking at the letsencrypt.org site, they say that the validation now uses IPV6.  But IPV6 is not routable to my server.  The host company doesn't offer it.
So I'm trying to figure out a way to either get this to work by verifying over IPV4 (they say the service should automatically failover to IPV4 if IPV6 isn't available) OR get the dns challenge to work.  Cerbot uses the '--preferred-challenges=dns' parameter, but I cannot find an equivalent command for letsencrypt-win-simple.
needs IPV4: https://community.letsencrypt.org/t/failed-authorization-on-windows/37746
service should failover gracefully: https://community.letsencrypt.org/t/certbot-ipv6-not-configured-and-challenges-fail-ipv6- preferred/34645/4
Manual DNS challenge: https://serverfault.com/questions/750902/how-to-use-lets-encrypt-dns-challenge-validation

However, it only works over IPV4. In looking at the letsencrypt.org site, they say that the validation now uses IPV6. But IPV6 is not routable to my server. The host company doesn't offer it.

Hi @lanceosaurus, could you remove the AAAA record for your domain? If you know that IPv6 isn't routable then any clients that fetch the AAAA record are going to be have increased latency while waiting for the failed request.

I just added the record today incase the letsencrypt.org validator needed it to gracefully fail.    But I'll remove it now if you think that is best.

Is there a way to trigger the manual dns validation with this tool?

I just added the record today incase the letsencrypt.org validator needed it to gracefully fail.  But I'll remove it now if you think that is best.

I'm confused - what would there be to fall back from if there wasn't already an AAAA record? IPv6 is only used when there's an AAAA for the domain. If there wasn't one already and your issuance failed then you had a different problem and maybe added a second problem (broken IPv6) by adding the AAAA record. Does that make sense? It sounds like removing it is the right decision.

I added it in desperation to get the validator to work!  I tried it many times without.  Enough failures to get hit with the ban hammer.
Again, I can put the url in a browser and view the file from anywhere on the planet, but the validation service still fails.

The server is in China, if that makes a difference.  I've used letsencrypt-win-simple on a few other non-Chinese servers successfully, so I know what's supposed to happen when it works successfully.

@lanceosaurus Removing the AAAA record will definitely fix the IPv6 reachability problem you're reporting in this ticket. It sounds like that problem was introduced when you added the record to try and solve a different problem. I recommend that you remove all of the AAAA records for the domains in question and try to issue again. If you still get an error you should update this ticket with that new error message and I can help troubleshoot. Does that sound feasible? Let's Encrypt should be able to issue for servers in China without problem.

From here I'm getting a holding page for that domain, not a default IIS page or 404 or other.
In fact the final IP is in The USA, probably Las Vegas.
Is the DNS I'm seeing right ?
;; ANSWER SECTION:
merge.taffy.cn.         43      IN      CNAME   t.dcoin.co.
t.dcoin.co.             272     IN      A       <IP_ADDRESS>

;; AUTHORITY SECTION:
dcoin.co.               6868    IN      NS      ns3.dnsv2.com.
dcoin.co.               6868    IN      NS      ns4.dnsv2.com.

To be clear: I added the AAAA record today after trying multiple times yesterday to get a certificate.  Adding the AAAA record has NOT caused this issue...  it may have introduced a different one, but the end result is the same - verification fails.
Here's the latest verification file: http://mergeDOTwebportglobalDOTcn/.well-known/acme-challenge/2EPm_Uv9OdignbAVIIv9qm-SNq0bFTGpOYGtTvWQElc - Its on the Chinese server and I can access this from Europe or anywhere else.  However, the validation still fails.
Here are the results of the latest run:
C:\Users\Lance>C:_scripts\letsencrypt-win-simple.V1.9.3\letsencrypt.exe --test
The global logger has been configured
Let's Encrypt (Simple Windows ACME Client)
Renewal Period: 3650
Certificate Store: WebHosting
ACME Server: https://acme-staging.api.letsencrypt.org/
Config Folder: C:\Users\Lance\AppData\Roaming\letsencrypt-win-simple\httpsacme-staging.api.letsencrypt.org
Certificate Folder: C:\Users\Lance\AppData\Roaming\letsencrypt-win-simple\httpsacme-staging.api.letsencrypt.org
Loading Signer from C:\Users\Lance\AppData\Roaming\letsencrypt-win-simple\httpsacme-staging.api.letsencrypt.org\Signer
Getting AcmeServerDirectory
Loading Registration from C:\Users\Lance\AppData\Roaming\letsencrypt-win-simple\httpsacme-staging.api.letsencrypt.org\Registration
Scanning IIS Site Bindings for Hosts
1: IIS demoDOTwebportglobalDOTcn (C:_websites\demoDOTwebportglobalDOTcn)
2: IIS mergeDOTwebportglobalDOTcn (C:_websites\emptysite)
3: IIS wwwDOTwebportglobalDOTcn (C:_websites\wwwDOTwebportglobalDOTcn)
W: Generate a certificate via WebDav and install it manually.
F: Generate a certificate via FTP/ FTPS and install it manually.
M: Generate a certificate manually.
A: Get certificates for all hosts
Q: Quit
Choose from one of the menu options above: 2
Authorizing Identifier mergeDOTwebportglobalDOTcn Using Challenge Type http-01
Writing challenge answer to C:_websites\emptysite.well-known/acme-challenge/2EPm_Uv9OdignbAVIIv9qm-SNq0bFTGpOYGtTvWQElc
Writing web.config to add extensionless mime type to C:_websites\emptysite.well-known\acme-challenge\web.config
Answer should now be browsable at http://mergeDOTwebportglobalDOTcn/.well-known/acme-challenge/2EPm_Uv9OdignbAVIIv9qm-SNq0bFTGpOYGtTvWQElc
Submitting answer
Refreshing authorization
Refreshing authorization
Refreshing authorization
Refreshing authorization
Refreshing authorization
Refreshing authorization
Refreshing authorization
Refreshing authorization
Authorization Result: invalid
Authorization Failed invalid

The ACME server was probably unable to reach http://mergeDOTwebportglobalDOTcn/.well-known/acme-challenge/2EPm_Uv9OdignbAVIIv9qm-SNq0bFTGpOYGtTvWQElc
Check in a browser to see if the answer file is being served correctly.
Authorize failed: This could be caused by IIS not being setup to handle extensionless static files.Here's how to fix that:
1.In IIS manager goto Site/ Server->Handler Mappings->View Ordered List
2.Move the StaticFile mapping above the ExtensionlessUrlHandler mappings. (like this http://i.stack.imgur.com/nkvrL.png)
3.If you need to make changes to your web.config file, update the one at C:_scripts\letsencrypt-win-simple.V1.9.3\web_config.xml

Press enter to continue.

@rdebath - sorry, I subbed a different domain name for privacy, shoulda used something like foobar to make it more obvious/mentioned the domain was changed to protect the names of the innocent - the actual domain name is mergeDOTwebportglobalDOTcn.  (but replace the DOTs)

Okay, I successfully downloaded the verification file from two continents too, so LE should be able to also.
However, their DNS cache will be polluted with your old AAAA record for the TTL that that record had. If it was ten minutes it should work now, otherwise you might have to wait til tomorrow. If you tripped a rate limit you'll also have to wait a while (the test server has much higher rate limits and is unlikely to trip).
If it's still not working I'd be starting to think about firewalls and routing problems. The connections should be coming from one or more of outbound1.letsencrypt.org, outbound2.letsencrypt.org and so on and those hosts are pingable (I don't think they have web servers on them though).

The firewall we have control over has port 80 open to the world, so I doubt that is an issue, unless LE is trying to use some other port?
I can ping those addresses from the China server:
C:\Users\Lance>ping outbound1.letsencrypt.org
Pinging outbound1.letsencrypt.org [<IP_ADDRESS>] with 32 bytes of data:
Reply from <IP_ADDRESS>: bytes=32 time=232ms TTL=235
Reply from <IP_ADDRESS>: bytes=32 time=240ms TTL=235
Reply from <IP_ADDRESS>: bytes=32 time=243ms TTL=235
Reply from <IP_ADDRESS>: bytes=32 time=251ms TTL=235
Ping statistics for <IP_ADDRESS>:
Packets: Sent = 4, Received = 4, Lost = 0 (0% loss),
Approximate round trip times in milli-seconds:
Minimum = 232ms, Maximum = 251ms, Average = 241ms

LE only use other ports if you set a redirection for them, so you can, for example, redirect the acme folder to a different web server or a different machine if you like. Of course you'd then have the problem of installing the new certificate on a remote machine 'cause LEWS won't do that for you.
Okay, differences between now and when LE is checking ...

Does this file exist at the moment:
C:_websites\emptysite\.well-known\acme-challenge\web.config? If not does copying the file from C:_scripts\letsencrypt-win-simple.V1.9.3\web_config.xml break anything?
The check file has just been created at verification, there may be a delay, does --warmup make a difference?

#1 - the file is created and accessible instantly.  While it's going through the loop 'Refreshing authorization', I can grab the link and browse to the file.
#2 - I just tried --warmup, but come up with the same result.

FYI - I've opened this issue up on the LE forum too: https://community.letsencrypt.org/t/failed-authorization-on-windows-using-letsencrypt-win-simple/41404

Well, you'd better add the --verbose flag to the command line too. However, be aware that it may show private keys which you'll have to discard or even revoke (if used on the live server) afterwards.

@rdebath - I don't have a --verbose flag offered on v.1.9.3...
...Just downloaded the HEAD from the repo.  --verbose now available!  Thanks!
ACME server reported detail DNS problem: query timed out looking up CAA for {my domain}
We use DNSPod.com as our DNS service.  Is there some config change I can do to my DNS records to get the CAA request to work?

Right! I see the problem; your DNS provider is giving out a SERVFAIL for a valid query.
It looks like your DNS server getting confused when a CAA request is sent to it. If I get dig to do a trace for an A record <IP_ADDRESS> is giving me the data. For an AAAA record it gives me a nice "nope" but for a CAA record it falls over. For a UDP query it just doesn't respond, for a TCP query it looks like it closes the connection.
Sounds like something you need to talk to your DNS provider (dnspod ?) about.

Thanks @rdebath !  I've sent a message to Dnspod's support about how to add a CAA record.  It seems that this option is not yet widely supported, there are a lot of complaints on various DNS providers' forums asking them to support this standard.

However, while not supported, most DNS servers don't fall over :-1: when asked for a CAA, they just can't be configured to respond with one.

Thanks @rdebath!  I took advice on the LE forum and changed my DNS provider to a free Cloudflare service.  Just created the certificate today, first time!  Woot!


[Microblocks] Transaction history timeline component

Create a new component that displays transaction events in the
Figma link

@kyranjamie to confirm: the API doesn't support this yet does it?

Correct, it doesn't.

@falkenprods it looks like there is a plan to solve the issue you experienced. I see the distinction is made between confirmations in microblocks in the explorer but not in the activities tab on the web wallet. Does that mean the API does support it now? What can be done to help this along?

In the web wallet a transaction state would change from being in a microblock to being in an anchor block when it lands in an anchor block. So it does show the state, just with not as much detail as in the Explorer.
The work to standardize the activity list between wallet and Explorer is on the pipe, but there are other priorities to focus on, and we haven't got to it yet.


[5.0] add getDates function with DELETED_AT constant in SoftDeletes Trait

and laravel user use softDelete like https://github.com/laravel/docs/pull/1020

Would rather this be done manually instead of having to configure overrides with traits if you need to add more.


ci(docker image update)

:tada: This PR is included in version 3.0.4 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Could surface solar radiation be added?

Hi,
Really great work! Its nice being able to stream in all this ERA5 data. I was wondering if some more surface solar radiation variables could be added to this? The reason for this would be to help with renewable energy generation forecasting, specifically PV/Solar generation, and training models to forecast those.

Have you looked at the 0.25 degree full_37-1h-0p25deg-chunk-1.zarr-v3 dataset?
https://github.com/google-research/arco-era5?tab=readme-ov-file#full_37-1h-0p25deg-chunk-1zarr-v3
It includes a bunch of solar radiation variables, which I believe should cover what you're looking for: clear_sky_direct_solar_radiation_at_surface, surface_net_solar_radiation, surface_net_solar_radiation_clear_sky, surface_solar_radiation_downward_clear_sky, surface_solar_radiation_downwards, toa_incident_solar_radiation, top_net_solar_radiation, top_net_solar_radiation_clear_sky, total_sky_direct_solar_radiation_at_surface.

Oh, no, I hadn't seen that. Yeah, that has what I would want in it. Thanks!


[CI] Run integration test on Jenkins

In Github Action testbed, operation on docker IPv6 is limited. So, remove integration test in Github Action and run it on Jenkins.
Signed-off-by: Zhecheng<EMAIL_ADDRESS>

/test-integration

/test-integration

We can merge this as it is. However, you may want to consider enabling this only for the IPv6 branch for now, and keeping the existing Github workflow for master and release branches. After we confirm that it works well, we can remove the Github workflow and use only the Jenkins job. What do you think? If you feel confident about the change, we can skip this step.

Make sense. I updated the patch to let integration test only run on ipv6 branch. When ipv6 branch is merged into master branch, there will be a patch to update integration testing branch to all branches.

/skip-all


docs: add usage of profile authentication commands

Pull Request Test Coverage Report for Build 985745327

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

Totals

Change from base Build 985227955:
0.0%

Covered Lines:
4973

Relevant Lines:
5950

💛  - Coveralls


Address how Casbin implements RBAC96, including RBAC0, RBAC1, RBAC2, RBAC3

Some users will want to know what's the relationship between Casbin' RBAC to the classic RBAC96 models. Need to clarify it.

@hsluoyz i would like to work on this issue, can you elaborate a bit?

Based on academia's definition, there are four models in RBAC96: RBAC0, RBAC1, RBAC2, RBAC3.
See details at: https://security.stackexchange.com/questions/169875/rbac0-rbac1-rbac2-rbac3-what-do-they-mean
Some Casbin users are security experts and they want to know what level of RBAC (0, 1, 2, or 3) does Casbin support. So we should clarify it in the docs.

Can anyone work on this? @nodece @GopherJ @DivyPatel9881

Yes, I will have to research a bit, and then I will be updating the docs

@Kininaru

@hsluoyz
Got that, working on this.


[Story] As a script user, I hope that I can use all the functions in one command, so that I don't have to copy several scripts.

Details
Add a script that downloads a script and moves to usr/local/bin.
Breakdown

Create a script list all functions that exist
Write to /usr/local/bin (sudo)

Acceptance Criteria

It creates a new command that can work properly

options still has some problems

Also, symbolic link to /usr/local/bin doesn't work

run as a function shouldn't use exit
maybe should use return instead

it still can't be run directly.

write to env?


Add inverted / deselected icon for bookmarks

Yes! I was looking around in assets and couldn't find it and end up just adding one image.

I’ll whip out photoshop and make it. That’s what I did for the others.
Sent with GitHawk


Possible to train with own images

I have a bunch of images with text in a specific font that I would like to OCR
Is it possible to create my own training data and how would one go about to do that?

If you know the font in advance I would say the easiest way is to put that font in the config file (and tweak size, etc to your liking) and let the network train itself. See 'Configuration' and 'Usage' in the README.
Then you can use the trained network on your images, but you have to first convert them to a bit array of the size specified in the config file (see the functions parse and mnist in main.js for conversion examples).

Great thanks. I'll try that out


cannot delete retained vars

problem steps
jsk retain ON
jsk py x = 1
jsk py del x

expected result
no output
actual result
UnboundLocalError: local variable 'x' referenced before assignment

This is because global vars can't be deleted when in a local scope. I'm not sure there's much I can do about this, the alternative would be to reproduce retained data into locals, which is a pain and a dirtier method than I have currently.
For sake of mentioning, a way of removing it from the scope manually is to do globals().pop("x")

:tada: :shipit:


Pass Commands to Rocket.Chat

Refering to https://github.com/RocketChat/Rocket.Chat/pull/1372
This will send commands to rocket.chat so they can be autocompleted.
DON'T MERGE

Closing this for now.   I'll re-open after i've refactored a little.

@geekgonecrazy was there an update somewhere else concerning this feature?

A lot has changed since I started on this.  I think with rocket.chat apps and some of the changes happening with bot handling might improve.  I believe there is an issue open on the main project with details of how this will be handled.


[BACKLOG-13099] Mavenize kettle/plugins/kettle-gpload-plugin

@pentaho/2-1b @smaring Please review

Build Skipped
:white_check_mark: There was nothing to build. The changes are probably excluded by the build rules. If this is an error, please contact your system administrator.

Build Skipped
:white_check_mark: There was nothing to build. The changes are probably excluded by the build rules. If this is an error, please contact your system administrator.

Build Skipped
:white_check_mark: There was nothing to build. The changes are probably excluded by the build rules. If this is an error, please contact your system administrator.

Build Skipped
:white_check_mark: There was nothing to build. The changes are probably excluded by the build rules. If this is an error, please contact your system administrator.

Build Skipped
:white_check_mark: There was nothing to build. The changes are probably excluded by the build rules. If this is an error, please contact your system administrator.


Greenkeeper/rollup 1.27.9

Coverage remained the same at 77.815% when pulling 309e81db0fb8b15244aaa5cde8af95dd0cb0beac on greenkeeper/rollup-1.27.9 into 25724b352c3351a46b30d887649a47e84b89df8f on master.


Migrate spring security from 3 to 4

I migrated SHOGun2 from Spring security 3 to 4.
Credits go to: http://docs.spring.io/spring-security/site/migrate/current/3-to-4/html5/migrate-3-to-4-xml.html
I tested a lot and everything seems to be fine. Please review and merge!

Looks good to me. Please merge.


In README.md, local/bin should be .local/bin

Make sure ~/local/bin is in your $PATH

In README.md should be:

Make sure ~/.local/bin is in your $PATH

Thanks. Fixed https://github.com/chris-belcher/electrum-personal-server/commit/6b8adab72ce32e7a870a89532c3550b33ef6c943


deisctl: Add more keys for access

What is the recommended way of adding more ssh-keys to deisctl? I need to grant more people on my team access to "dock" to containers to debug issues. I first suggested they use deis apps:run, but that doesn't allow them to create a bash session and try to investigate the container.
Do you have any suggestions or best practices for this type of access?
TYIA for your help.

It's true that deis run bash (i.e. long-running bi-directional TCP connections) is still an unsolved problem. See #117 (one of our oldest bugs). Most administrators have been giving users access to the cluster so they can run docker exec -it myapp_v2.web.1 bash for the time being. It's both a technical problem with fleet and with WSGI being unable to handle long-running client requests.
Closing as a duplicate of #117 so we can keep all conversations related to interactive shell sessions in that ticket (including workarounds). Thanks!

What is the recommended way of adding more ssh-keys to deisctl?

I haven't done this, but I've seen something similar done.  You could include a unit on every host that routinely checks for an authorized_keys file in a publicly accessible location such as a github repo or a gist.  It would contain the public keys of people who should be able to use deisctl and ssh to the host.


Concurrent Read Write on tf Variable?

I've seen several possible concurrent reads and writes in your code.

train_p and train_q has no control dependency on each other, train_q updates q_theta which is also used in train_p. It is possible they may conflict with each other.

train_p updates p_theta_target, which is used in train_q, and the order is also undefined.

Hey, thanks for the feedback! I just added separate functions for p and q training and an option to run synchronously (--async False). Updating concurrently is fast and I don't think it hurts performance since the changes at every step are only very small. I'm currently testing whether there is a difference in performance.

ok it doesn't look like concurrent updating is hurting performance


Add a Google maps layer for more zoom levels

as above

Sorry, but that's not legal, for several reasons:

Google's TOS prevents wrapping: you can't use it with Leaflet.
Google's TOS also says you can't trace data off of it.


Use the fast-async package to support async functions as a Babel compile step before Bublé

https://github.com/MatAtBread/fast-async

Finished in #146, but not merged yet


Datatree import

This PR imports xarray-contrib/datatree and its history to pydata/xarray/xarray/datatree_
Step one of issue #8572.
This imports the datatree code without exposing DataTree as a public API.
git filter repo https://github.com/newren/git-filter-repo was used to preserve some history for the merge.
Datatree tags were renamed to legacy-datatree-{tag}:
git filter-repo --tag-rename '':'legacy-datatree-'

Links to xarray-contrib/datatree original PRs are preserved by rewriting the messages using a replace-message file.
# standard github style pull request (#39)
regex:\(#(\d+)\)==>https://github.com/xarray-contrib/datatree/pull/\1

# "Merge pull request #11 from TomNicholas/single_datanode_class"
regex:pull request #(\d+)==>https://github.com/xarray-contrib/datatree/pull/\1

Links to the xarray-contrib/datatree Issues are preserved with a replace-text file.
# standard comment change "  # see issue #38"
regex:(\s*?#.*[ ])#(\d+)==>\1https://github.com/xarray-contrib/datatree/issues/\2

# also "    @pytest.mark.xfail(reason="Indexing needs to return whole tree (GH #77)")"
regex:(\(GH #(\d+)\))==>(GH https://github.com/xarray-contrib/datatree/issues/\2)

The datatree repo was relocated to a subdirectory
git-filter-repo --to-subdirectory xarray/datatree_

and the prepared datatree repository was added as a remote and merged into xarray.
git merge prepared-datatree/main --no-commit --allow-unrelated-histories

This should allow work to begin on the rest of the steps in #8572

@jhamman I appreciate all of these tips for cleanup, they are super helpful.  I was hoping to see if I can get this through CI while leaving the datatree_ directory mostly as it is as a first step, a clean import of the datatree library.
Since it's not exposed in xarray at all yet, I don't think I want to update any of the xarray documentation in this PR.  I was thinking the cleanup could happen in a quick follow-on PR and following that by the PRs to import the functionality and documentation later.  Does that seem reasonable to you?

@flamingbear - that seems like a reasonable first step. In terms of getting the datatree tests to run in CI, I think you'll have to move them into the /tests directory.

In terms of how to get this into the xarray repository, I'd probably do a rebase (or rather, redo the final step of merging the unrelated history of datatree) on top of current main, then do a manual push onto main. This requires temporarily disabling the branch protection, but I don't think we can do anything wrong here that can't be undone.
Since I believe that merging now is uncontroversial (we don't import into the final location), we could even do that very soon. Also, I don't think we have to remove the unnecessary files now, I'd probably do that as a final step to wrap up the migration.

In terms of getting the datatree tests to run in CI, I think you'll have to move them into the /tests directory.

I was hoping to disable them / skip them for this PR. Working to get open_datatree running in a different repo I was seeing that I can get almost all of the tests running in place. but almost is doing a lot of lifting there.

In terms of how to get this into the xarray repository, I'd probably do a rebase (or rather, redo the final step of merging the unrelated history of datatree) on top of current main, then do a manual push onto main. This requires temporarily disabling the branch protection, but I don't think we can do anything wrong here that can't be undone.

Wouldn't merging this PR with a merge commit effectively do that (play the datatree history into xarray)?  The history was added into my fork and this branch contains that. Or maybe I'm misunderstanding something.
But I do think we want to continue having passing CI for everything else unrelated to xarray still functions as normal.

the only difference my proposal would make is one merge commit less, thus keeping a more linear commit history (although of course we still have the commit from merging datatree, but that one we can't avoid).
The xarray CI should obviously not be affected, and by moving the code and tests to the right place in separate PRs we'll be able to ensure passing CI for datatree (making use of the usual CI runs on PRs).

If you can find a way to get the CI to pass without removing the docs and other config files, go for it. My review was assuming the simpliest way to get the CI passing was to strip out bits that are triggering the current CI errors (docs config, mypy errors, etc.).

we'll have to tell mypy, ruff, and the pytest's doctest runner to ignore the xarray/datatree_ directory, and make a note to revert this in the final PR that cleans up after the integration is complete.

For mypy we can use the exclude option for the mypy section in pyproject.toml, pre-commit also has a exclude parameter for hooks, and we can add an additional --ignore to the doctest job (unless this is also possible to do in pyproject.toml?).

I'm not actually 100% sure how to test these locally.  I think have turned off the checks.
Since I don't want all of the changes from before, I'm free to force push to my flamingbear:datatree-import branch.

I would like to guess that this is a "not me" problem, and kick it.
worker 'gw2' crashed while running 'xarray/tests/test_backends.py::TestDask::test_dask_roundtrip'

yep, I believe that's one of the few flaky failures we have. I restarted the job, let's see if it fails again.

That's nifty, but I added a commit I didn't want to. But before I squash that, I'm curious about:

I'd probably do a rebase (or rather, redo the final step of merging the unrelated history of datatree) on top of current main, then do a manual push onto main.

basically, what I was thinking was that we'd prepare the merge in a branch that contains only the datatree history, and once we're ready to do the merge, we'd create the merge commit that combines the two histories and push to main. Not sure if that's the best strategy, but it would result in a cleaner history.

And there it is, I've made a complete hash of this PR by merging in main from xarray.
And the attempt to exclude datatree_ in ruff alone is still failing.
I suspect I will update with the full pre-commit.ci exclusion like I had before, but we may want a cleaner history. I will wait for some advice.

The conversation on this PR will stay, but won't make it into any of the commits. So I don't think we lose much by closing it instead of merging (in the end, PRs are just a way to discuss the changes from branches / commits). We could link to this PR from the merge commit, though.
As far as making it ready for merging, I believe we only need to make sure it doesn't interfere with the rest of the repository (in particular CI and pre-commit, but we should also make sure we don't include it in releases).
So I guess the packaging issue aside this should already be ready?

Yes, I think this is ready, or I am ready to create a new one that references this one as needed.  I am not sure what to do about excluding datatree_ in packaging, but I can look into that.

So I guess the packaging issue aside this should already be ready?

My naive approach would be to delete the directory before each build in the workflows?
.github/workflows/nightly-wheels.yml, .github/workflows/pypi-release.yaml
probably
      - name: Build tarball and wheels
        run: |
          git clean -xdf
          git restore -SW .
          rm -rf xarray/datatree_
          python -m build

it took me a while to figure that out, but a MANIFEST.in file is able to tell setuptools-scm to not include this particular directory (see the most recent commit)

I've tried multiple ways to cherry-pick / rebase a merge commit, but I don't think this is going to work very well. So that leaves us with redoing the merge, and cherry-picking the commits you/we did to prepare the final merge on top of it (my goal is to do a fast-forward merge into main).
Would you be up to doing that, @flamingbear? Otherwise I'll do it, but could you confirm that all the preparation you did on the datatree commits are stored in flamingbear/rewritten-datatree?

Would you be up for doing that, @flamingbear? Otherwise I'll do it, but could you confirm that all the preparation you did on the datatree commits are stored in flamingbear/rewritten-datatree?

I'm prepared to do that again.  And I can confirm that rewritten-datatree is NOT the file version.  I can/will push up the final version into that location, but that version is the first test version I did that does not include the final commits from datatree.

Well, if you're prepared to do that yourself I don't even need to know, I'll just verify the result and do the merge (and I've put the decision on whether to merge the cleaned version of this on the agenda of the meeting today)

@keewis I have updated rewritten-datatree with the repository used in the cleaner PR: https://github.com/pydata/xarray/pull/8688
Let me know how that looks.

it took me a while to figure that out, but a MANIFEST.in file is able to tell setuptools-scm to not include this particular directory (see the most recent commit)

I missed this, I will add this to the skips ci commit in #8688

it took me a while to figure that out, but a MANIFEST.in file is able to tell setuptools-scm to not include this particular directory (see the most recent commit)

I missed this, I will add this to the skips ci commit in #8688

I hope you didn't understand this as me saying you have to use the state from rewritten-datatree, I just wanted to know if you had the rewritten and updated state somewhere. So if we have all the commits on datatree this looks fine with me. I'll comment one tiny thing on the other PR, but other than that I think we're ready.

Nope, I didn't think that, but in the interest of transparency I pushed up the full repo I had prepared locally.  I'll go look at the note on the other pr.


only can update the code but can rerun the project

I mean this tool can just update the code ,
but can't execute command like pm2 restart

@kelvv thanks for your response
I mean can it do it automatically？

@wangyijunjunjun  Thank you
you should config pm2 watch by yourself because you need to let pm2 know two point:
1.which file you need to watch (.html , .js)
2.which file you need to ignore (.json , .config)
I want to support an interface to restart app at next version,you can set 'pm2 restart' or else

@wangyijunjunjun version 1.3.1 is fixed.
you can config your complateCMD in config file , just see readme , thankyou


Feat/timezone bound displays

Hi there!
As mentioned in #25, time is always UTC throughout the app. I think it would be better off following the timezone of the container, so I added .Local() to every time formatting.
Maybe following the browser's time would be even better, but using SSR makes it more complicated to implement. I think for now it's fine.
Hope you agree!

There is nothing I hate more than working with time. Good job!


In android, the push.on('error', function(){}) won't be triggered if the device has no google service installed or has network problem for connecting to google server

Expected Behaviour
Will trigger the 'error' event when any error happened when init the PushNotification
Actual Behaviour
No thing happend
Reproduce Scenario (including but not limited to)
Paste some logs here:
For push plugin:
V/Push_Plugin( 7822): execute: data=[{"browser":{"pushServiceURL":"http://push.api.phonegap.com/v1/push"},"android":{},"ios":{"sound":"true","aler
t":"true","badge":"true"},"windows":{}}]
V/Push_Plugin( 7822): execute: jo={}
V/Push_Plugin( 7822): execute: senderID=647571745247
E/Push_Plugin( 7822): execute: Got IO Exception MISSING_INSTANCEID_SERVICE
D/Push_Plugin( 7822): no icon option
D/Push_Plugin( 7822): no iconColor option
For app log:
only got:
I/chromium( 7822): [INFO:CONSOLE(312)] "Error in Error callbackId: PushNotification1062511891 : ReferenceError: data is not defined", source: file:///android_asset/www/cordova.js (312)
Steps to Reproduce
Platform and Version (eg. Android 5.0 or iOS 9.2.1)
Android 6.
(Android) What device vendor (e.g. Samsung, HTC, Sony...)
not only xiaomi oppo, I think the others should be the same.
Cordova CLI version and cordova platform version
cordova --version                                 7.0.1
cordova platform version android        6.2.3

Plugin version
cordova plugin version | grep phonegap-plugin-push   2.0.0

Logs taken while reproducing problem:
For push plugin log:
V/Push_Plugin( 7822): execute: data=[{"browser":{"pushServiceURL":"http://push.api.phonegap.com/v1/push"},"android":{},"ios":{"sound":"true","aler
t":"true","badge":"true"},"windows":{}}]
V/Push_Plugin( 7822): execute: jo={}
V/Push_Plugin( 7822): execute: senderID=647571745247
E/Push_Plugin( 7822): execute: Got IO Exception MISSING_INSTANCEID_SERVICE
D/Push_Plugin( 7822): no icon option
D/Push_Plugin( 7822): no iconColor option
For app log:
only got:
I/chromium( 7822): [INFO:CONSOLE(312)] "Error in Error callbackId: PushNotification1062511891 : ReferenceError: data is not defined", source: file:///android_asset/www/cordova.js (312)

add the args parameter in cordova.js:
"callbackFromNative callbackId=PushNotification154804083 isSuccess=false status=9 args=["MISSING_INSTANCEID_SERVICE"]", source: file:///android_asset/www/cordova.js

hi macdonst, is there a way to fix it quickly? it's urgent for me. I'm not familiar with plugin coding, can you help to tell me which file(s) I can look into it for trying to fix.

@xiaoaowanghu well it is hitting this line: https://github.com/phonegap/phonegap-plugin-push/blob/7d22655fe9de630ee75299f21135be76960c8cc2/src/android/com/adobe/phonegap/push/PushPlugin.java#L100 which is sending back an error to the push plugin. The error callback is registered here: https://github.com/phonegap/phonegap-plugin-push/blob/master/src/js/push.js#L67 and the implementation is here: https://github.com/phonegap/phonegap-plugin-push/blob/master/src/js/push.js#L60 and that should fire an error event.

Ha, I need to say sorry. It's not a bug of this plugin. It's a bug in my error event handler. Actually the first statement is to log the error, but I used the wrong viriable name. so the exception was thrown  up  to codovajs. And I didn't see the log so I thought the error event wasn't triggered.
Thanks for your help, macdonst!


Rough draft of CosmosConstainerSettings fluent composition.

CosmosContainerSettings covers IndexingPolicy, UniqueIndexes, PartitionKey, TTL, HashV2 vs HashV2 etc..
UniqueueIdexe & IndexingPolicy are reasonably nested. Sample code looks like below
           IndexingPolicy ip = new IndexingPolicy();
           ip.IncludedPaths.Add(new IncludedPath() { Path = "/includepath1", Indexes = CosmosContainerSettings.DefaultIndexes});
           ip.ExcludedPaths.Add(new ExcludedPath() { Path = "/excludepath1" });

           Collection<CompositePath> compositePath = new Collection<CompositePath>();
           compositePath.Add(new CompositePath() { Path = "/compositepath1", Order = CompositePathSortOrder.Ascending });
           compositePath.Add(new CompositePath() { Path = "/compositepath2", Order = CompositePathSortOrder.Ascending });
           ip.CompositeIndexes.Add(compositePath);

           SpatialSpec spatialSpec = new SpatialSpec();
           spatialSpec.Path = "/spatialpath1";
           spatialSpec.SpatialTypes = new Collection<SpatialType>();
           spatialSpec.SpatialTypes.Add(SpatialType.Point);
           ip.SpatialIndexes.Add(spatialSpec);

           UniqueKeyPolicy uniqueKeyPolicy = new UniqueKeyPolicy();
           UniqueKey uniqueKey = new UniqueKey();
           uniqueKey.Paths = new Collection<string>();
           uniqueKey.Paths.Add("/uniqueuekey1");
           uniqueKey.Paths.Add("/uniqueuekey2");
           uniqueKeyPolicy.UniqueKeys.Add(uniqueKey);

           CosmosContainerSettings testContainerSettings = new CosmosContainerSettings("TestContainer", "/partitionKey");
           testContainerSettings.IndexingPolicy = ip;
           testContainerSettings.UniqueKeyPolicy = uniqueKeyPolicy;

Moving IndexPolicy to builder will ease little, its still part of the problem. And specific builder are not easier to discover.
How about a fluent composition (not a builder)?
Same sample with fluent will look like below
     CosmosContainerSettings containerSettings =
           new CosmosContainerSettings("TestContainer", "/partitionKey")

               .IncludIndexPath("/includepath1")
               .IncludIndexPath("/includepath2")
               .ExcludeIndexPath("/excludepath1")
               .ExcludeIndexPath("/excludepath2")
               .IncludeCompositeIndex("/compPath1", "/compPath2")
               .IncludeUniqueKey("/uniqueueKey1", "/uniqueueKey2")
               .IncludeSpatialIndex("/spatialPath", SpatialType.Point);

CosmosContainerSetting is mutable. I.e. customer can read existing definition, change and use it to ReplaceContainerAsync().
So changes also can compose with ease (for removal user needs to go-through Properties with current fluent model).

UT are not TBD

    public IndexingPolicy()

internal, if we're only allowing builder?

Refers to: Microsoft.Azure.Cosmos/src/Resource/Settings/IndexingPolicy.cs:34 in a77658c. [](commit_id = a77658c285a2b9d639b02f88221dd77e5b737d4b, deletion_comment = False)


[SPARK-36920][SQL] Support ANSI intervals by ABS()

What changes were proposed in this pull request?
In the PR, I propose to handle ANSI interval types by the Abs expression, and the abs() function as a consequence of that:

for positive and zero intervals, ABS() returns the same input value,
for minimal supported values (Int.MinValue months for year-month interval and Long.MinValue microseconds for day-time interval), ABS() throws the arithmetic overflow exception.
for other supported negative intervals, ABS() negate its input and returns a positive interval.

For example:
spark-sql> SELECT ABS(INTERVAL -'10-8' YEAR TO MONTH);
10-8
spark-sql> SELECT ABS(INTERVAL '-10 01:02:03.123456' DAY TO SECOND);
10 01:02:03.123456000

Why are the changes needed?
To improve user experience with Spark SQL.
Does this PR introduce any user-facing change?
No, this PR just extends ABS() by supporting new types.
How was this patch tested?
By running new tests:
$ build/sbt "test:testOnly *ArithmeticExpressionSuite"
$ build/sbt "sql/testOnly org.apache.spark.sql.SQLQueryTestSuite -- -z interval.sql"
$ build/sbt "sql/test:testOnly org.apache.spark.sql.expressions.ExpressionInfoSuite"

Test build #143814 has started for PR 34169 at commit d89ec8d.

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/48327/

Kubernetes integration test status failure
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/48327/

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/48327/

@beliefer @AngersZhuuuu @Peng-Lei @cloud-fan Could you review this PR, please.

Test build #143814 has finished for PR 34169 at commit d89ec8d.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/143814/

Merged to master.


Issue/297 add custom retry codes

This PR adds the ability to add custom retry codes to the emitter, to either retry on failure, or ignore specific codes.

Thanks for your pull request. Is this your first contribution to a Snowplow open source project? Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).
:memo: Please visit https://docs.snowplowanalytics.com/docs/contributing/contributor-license-agreement/ to learn more and sign.
Once you've signed, please reply here (e.g. I signed it!) and we'll verify. Thanks.


[Request - Product] Request for adding rtpengine

Full and short name of product
rtpengine
Does this product have LTS versions? What are the intervals between each LTS version?
yes
there is several versions :

obsolete versions
Not LTS versions already released and still supported.
LTS versions already released and still supported.
Planned versions

What is the website for the product and for its version information?
https://sipwise.com/releases/

Hello,
See https://github.com/endoflife-date/endoflife.date/pull/5933  as starter.


Update all URLs to point to new GitHub organization

See https://github.com/tealdeer-rs/tealdeer/issues/376 for details.

FYI @niklasmohrin. I'll merge this without review, since it's a purely mechanical change.


Mismatch between Relaton-data-* format and official RNG Schema

Some of the data of the relaton-data-* datasets does not match the format of the official RNG Schema.
For example, this dataset includes the fields "city", "state" and "country" under "contact", while the official schema only allows them to be included under "contact" -> "address":
Current format (wrong):
- organization:
    name:
    - National Bureau of Standards
    abbreviation: NBS
    contact:
    - city: Gaithersburg
      state: MD
      country: US

Correct format should be:
- organization:
    name:
    - National Bureau of Standards
    abbreviation: NBS
    contact:
    - address:
       city: Gaithersburg
       state: MD
       country: US

Other datasets with the same issue are:

https://github.com/ietf-tools/relaton-data-3gpp/blob/ccf5604c14c5c925b9c8cbd5bafa9fc0dd71f8f0/data/TR_00.01U_UMTS_3.0.0.yaml#L26
https://github.com/ietf-tools/relaton-data-ieee/blob/dbf20d676030988d58149d6150f29539f4633e97/data/AIEE_11-1937.yaml#L36

@andrew2net can you help clarify here? The RNG scheme here does describe an element called "address":
https://github.com/relaton/relaton-models/blob/fee9ea8205756163a1d96de47f9b105dc701d8a0/grammars/biblio.rnc#L163-L177

@andrew2net this issue is urgent, can you please help address this ASAP?

@ronaldtse address isn't an Array, it's a Hash. Shouldn't it be:
- address:
    city: Gaithersburg
    state: MD
    country: US

Ok, but this is not the issue @andrew2net .
The issue here is that the address field is missing, and city, state and country are included as part as the contact field.

There is still an issue with the field street when processing this dataset. @andrew2net can you double check this please?
You can find the above mentioned issue here https://github.com/relaton/relaton-py/runs/7414346454?check_suite_focus=true
pydantic.error_wrappers.ValidationError: 1 validation error for BibliographicItem

contributor -> 0 -> organization -> contact -> 0

__init__() got an unexpected keyword argument 'street' (type=type_error)

@stefanomunarini the https://github.com/ietf-tools/relaton-data-3gpp dataset doesn't update files until the source dataset is updated. To force updating please remove the https://github.com/ietf-tools/relaton-data-3gpp/blob/main/current.yaml file. I don't have permission to do it.

@andrew2net can you let me know which datasets need to be updated at ietf-tools? You could also create a PR for those datasets and then @kesara can merge them to achieve the same effect. Thanks.

@andrew2net has created https://github.com/ietf-tools/relaton-data-3gpp/pull/13 to trigger the update.

@stefanomunarini this is done, can you help close this? Thanks.


Football games schedule

https://www.programmableweb.com/category/sports/apis?category=20016

https://www.api-football.com/documentation/


Update the resource ordering for the new Splunk types.

Ensures that all the Splunk resource types are applied in the correct order.
Depends on #34 and #35
I've only tested this on a CentOS 7 node.

:+1:


zmq.error.ZMQError:Operation cannot be accomplished in current state

When the training starts, I open the viewer and the zmp error appears

Can you post the full error. Can you also try using a different ZQM port --viewer.zmq-port

--viewer.zmq-port doesn't work @tancik

This problem has troubled me for a long time，I would appreciate it if you could answer it，thanks! @tancik

The error isn't ZMQ related. It's an issue with tinycudann, RuntimeError: Batchsize must be a multiple of 128.
Did you modify any of the parameters? If not, try reinstalling tinycudann.

I reinstalled tiny-cuda-nn-1.4, and I haven't changed any parameters, but it still reported, Batchsize must be a multiple of  @tancik


refactor: simplify handleCopilotChatInputChange by destructuring even…

This PR refactors the handleCopilotChatInputChange function in the Copilot.tsx file by destructuring the event object directly in the function parameters. This change improves the readability and simplicity of the code without altering the function's behavior. The refactored function has been tested to ensure it handles data changes correctly, and no side effects have been introduced.

@mason-at-pieces  can you please review this !

@mason-at-pieces  can you please review !


New command Build-BicepParams

Is your feature request related to a problem? Please describe.
Add new command Build-BicepParams that works like bicep build-params.
Describe the solution you'd like
Do all the things that bicep build-params do.
Additional context
Support for it added to BicepNet in https://github.com/PSBicep/BicepNet/pull/62

Closed in #305


Add support for Prometheus

Description:
Related issue (if applicable): fixes #
Pull request in home-assistant.github.io with documentation (if applicable): home-assistant/home-assistant.github.io#1454
Example entry for configuration.yaml (if applicable):
prometheus:

Checklist:
If user exposed functionality or configuration variables are added/changed:

[x] Documentation added/updated in home-assistant.github.io

If the code communicates with devices, web services, or third-party tools:

[ ] Local tests with tox run successfully. Your PR cannot be merged unless tests pass
[ ] New dependencies have been added to the REQUIREMENTS variable (example).
[ ] New dependencies are only imported inside functions that use them (example).
[ ] New dependencies have been added to requirements_all.txt by running script/gen_requirements_all.py.
[ ] New files were added to .coveragerc.

If the code does not interact with devices:

[ ] Local tests with tox run successfully. Your PR cannot be merged unless tests pass
[ ] Tests have been added to verify that the new code works.

Prometheus (https://prometheus.io/) is an open source metric and alerting
system. This adds support for exporting some metrics to Prometheus, using
its Python client library.

This PR seems to have gone stale. Closing it. You can reopen it when you're ready to finish it.

@rcloran do you have any plans to continue working on this?  If not, I could take a look at finishing it up.  I was thinking of adding prometheus support myself before seeing this pr.

@justyns : If you're game for finishing it up, that would be great. I think most of the requested changes are fairly small, but I was pondering a few others.
My main annoyance at the moment is that state_as_number doesn't work for presence sensors.
The other problems are mostly stylistic.


Consider to change location to *.cbuild.yml files

The Problem To Be Solved
The *.cbuild.yml files tend to pollute the user workspace, specifically when many different target-types and/or build-types are used.
Suggest A Solution
Consider to move these files into the folder .cmsis. This is obviously a breaking change, hence the proposal is to delay it until CMSIS-Toolbox 3.0

This can be solved differently using the --output option at the level of the IDE.
See https://github.com/ARM-software/vscode-cmsis-csolution/issues/154


chore: exclude build artifacts from VSCode search

Omits .ilean, .c and .trace files from the ctrl-P / cmd-P file search interface.
The option about search.usePCRE2 was flagged with a deprecation warning from VSCode, so I removed that, too.

Shouldn't the .gitignore take care of this? It seems to work for me. Is this for use with non-Git copies?

The .gitignore doesn't seem to affect search results (either in the ctrl-P file opener, or contents search) for me.


unexpected shape for input 'input' for model 'lp-detector'

i write a client code for my model. But i got this issue. need a help

Trying <IP_ADDRESS>...
Connected to localhost (<IP_ADDRESS>) port 8000 (#0)

GET /api/status/lp-detector?format=binary HTTP/1.1
Host: localhost:8000
User-Agent: libcurl-agent/1.0
Accept: /

< HTTP/1.1 200 OK
< Content-Type: application/octet-stream
< NV-Status: code: SUCCESS server_id: "inference:0" request_id: 37
< Date: Thu, 21 Feb 2019 16:11:31 GMT
< Content-Length: 215
<

Connection #0 to host localhost left intact
id: "inference:0"
version: "0.12.0dev"
uptime_ns:<PHONE_NUMBER>963
model_status {
key: "lp-detector"
value {
config {
name: "lp-detector"
platform: "tensorflow_graphdef"
version_policy {
latest {
num_versions: 1
}
}
max_batch_size: 8
input {
name: "input"
data_type: TYPE_FP32
dims: 3
}
output {
name: "concatenate_1_output"
data_type: TYPE_FP32
dims: 2
label_filename: "labels.txt"
}
instance_group {
name: "lp-detector_0"
count: 4
gpus: 0
kind: KIND_GPU
}
default_model_filename: "model.graphdef"
}
version_status {
key: 1
value {
ready_state: MODEL_READY
infer_stats {
key: 0
value {
failed {
count: 8
total_time_ns: 791063
}
}
}
}
}
}
}
ready_state: SERVER_READY

image.dims:  3
scaled.ndim:  3

Trying <IP_ADDRESS>...
TCP_NODELAY set
Connected to localhost (<IP_ADDRESS>) port 8000 (#0)

POST /api/infer/lp-detector?format=binary HTTP/1.1
Host: localhost:8000
User-Agent: libcurl-agent/1.0
Accept: /
Content-Type: application/octet-stream
NV-InferRequest:batch_size: 8 input { name: "input" dims: 1000 dims: 800 dims: 3 } output { name: "concatenate_1_output" }
Content-Length: 9600000

We are completely uploaded and fine
< HTTP/1.1 400 Bad Request
< NV-InferResponse:
< NV-Status: code: INVALID_ARG msg: "unexpected shape for input 'input' for model 'lp-detector'" server_id: "inference:0"
< Content-Type: application/octet-stream
< Date: Thu, 21 Feb 2019 16:11:31 GMT
< Content-Length: 0
<
Connection #0 to host localhost left intact
Traceback (most recent call last):
File "clients.py", line 27, in
8)
File "/usr/local/lib/python3.5/dist-packages/tensorrtserver/api/init.py", line 844, in run
self._last_request_id = _raise_if_error(c_void_p(_crequest_infer_ctx_run(self._ctx)))
File "/usr/local/lib/python3.5/dist-packages/tensorrtserver/api/init.py", line 183, in _raise_if_error
raise ex
tensorrtserver.api.InferenceServerException: [inference:0 0] unexpected shape for input 'input' for model 'lp-detector'

In your model configuration it looks like the "input" tensor shape is  [ 3 ].
But the numpy array you passed for that tensor seems to have shape [ 1000, 800, 3 ].


ColorPicker: Crash in XAML islands when exiting app

Describe the bug
When using ColorPicker in XAML islands, if the app is exited within a short timespan of tearing down a XAML island (or directly after), it will cause an access violation in Windows.UI.Xaml.dll.
Steps to reproduce the bug

Create a new blank WPF .NET Framework app.
Add the Microsoft.Toolkit.Wpf.UI.XamlHost NuGet package.
Add a project reference to the UniversalApiContract winmd from the Windows SDK.
Add an app manifest to the app, and within <compatibility> add <maxversiontested Id="10.0.19041.0" />
In MainWindow.xaml, write the following<Window x:Class="WpfApp2.MainWindow"
        xmlns="http://schemas.microsoft.com/winfx/2006/xaml/presentation"
        xmlns:x="http://schemas.microsoft.com/winfx/2006/xaml"
        xmlns:d="http://schemas.microsoft.com/expression/blend/2008"
        xmlns:mc="http://schemas.openxmlformats.org/markup-compatibility/2006"
        xmlns:local="clr-namespace:WpfApp2" xmlns:xamlhost="clr-namespace:Microsoft.Toolkit.Wpf.UI.XamlHost;assembly=Microsoft.Toolkit.Wpf.UI.XamlHost"
        mc:Ignorable="d"
        Title="MainWindow" Height="450" Width="800">
    <Grid>
        <xamlhost:WindowsXamlHost x:Name="host" />
    </Grid>
</Window>

In MainWindow.xaml.cs's constructor, write this codepublic MainWindow()
{
    InitializeComponent();

    var sp = new Windows.UI.Xaml.Controls.StackPanel
    {
        Background = new AcrylicBrush
        {
            BackgroundSource = AcrylicBackgroundSource.HostBackdrop,
            TintOpacity = 0
        },
        HorizontalAlignment = Windows.UI.Xaml.HorizontalAlignment.Stretch,
        VerticalAlignment = Windows.UI.Xaml.VerticalAlignment.Stretch
    };
    host.Child = sp;

    var button = new Windows.UI.Xaml.Controls.Button
    {
        Content = "Click me!",
        HorizontalAlignment = Windows.UI.Xaml.HorizontalAlignment.Center,
        VerticalAlignment = Windows.UI.Xaml.VerticalAlignment.Center
    };

    button.Click += (s, e) =>
    {
        Dispatcher.InvokeAsync(Close);
    };

    sp.Children.Add(button);
    sp.Children.Add(new ColorPicker());
}

Launch the app.
Click the button

Expected behavior
The app should not crash
Screenshots

Version Info
NuGet package version:
[Microsoft.Toolkit.Wpf.UI.XamlHost 6.1.2]

Windows app type:

UWP
Win32

Yes

Windows 10 version
Saw the problem?

Insider Build (xxxxx)

May 2020 Update (19041)
Yes

November 2019 Update (18363)

May 2019 Update (18362)

October 2018 Update (17763)

April 2018 Update (17134)

Fall Creators Update (16299)

Creators Update (15063)

Device form factor
Saw the problem?

Desktop
Yes

Xbox

Surface Hub

IoT

Additional context
This also happens in my C++-based XAML islands app, so this is not an issue specific to the WCT WPF host, it just makes reproduction easier.

Seems related to #7260?
My workaround is to manually load threadpoolwinrt.dll before Windows.UI.Xaml.dll is loaded.
https://github.com/Blinue/Magpie/blob/f1809baf11c6a62053285d8e3d08fc4922961c41/src/Magpie/XamlApp.cpp#L40-L44

Bump, still an issue

Unfortunately, the WinUI team is currently spending its time on WinUI 3. To help with this focus, new bugs filed against WinUI 2 are closed in this repo unless they're a security issue or are business critical.
https://github.com/microsoft/microsoft-ui-xaml/blob/main/docs/contribution_handling.md#winui-2

However, the issues of WinUI 3 have been virtually left unattended, and closing old issues is nothing but a self-deceiving act. I wonder how much you value the feedback from the community.


ccp-linear-search-3 fails to search for a number greater than the size of integer pointer

In the function lsearch the length of the array has to be passed, because from the decayed base pointer parameter, the sizeof(array_name) cannot actually determine array_size, but rather returns the size of the (int*) pointer, which is meaningless, and turns out to be 8 in most systems. So if one has 4 elements, it can lead to an unpredictable machine state, and for more than 8 elements, only the first 8 are going to be considered.

I would like to work on this and also improve the coding style.

Sure, go ahead! Feel free to ping me if you face any difficulty. :smiley:

There's the same problem in another file cpp-linear-search, I'm directly making a pr for that under this issue


Add support for using XeroClient without XeroBundle, and add unit tests.

This Pull Request will allow someone to use XeroClient without having to use Symfony2 Bundles. Also I added unit tests for this project with near 100% coverage.
Do you still have interest in maintaining this repository or should I permanently fork out XeroClient out of XeroBundle?

Yes I still have interest in maintaining this repo. I think you make a good point though I would like to fork the XeroClient out of this bundle and have that entirely standalone. Then just have this bundle to provide integration for symfony. I will look at doing that shortly. Very glad to see some tests. Thank you for the contribution.


Change expansion and collapse image on ExpansionHeader?

I need different images for expansion header indicator for collapse and expand.
How can this be done?

Is this issue resolved?


Unable to create orders for tickets which are free

Describe the bug
On trying to create new orders, first the message is displayed "Attendees created successfully" and then it displays "Unable to create order".
After this, the logs show that the created attendee is deleted.
Seems that the issue is with the order amount becoming null when the amount is 0.
Logs
com.eventyay.attendee D/OkHttp: {"errors": [{"detail": "Field may not be null.", "source": {"pointer": "/data/attributes/amount"}, "status": "422", "title": "Validation error"}], "jsonapi": {"version": "1.0"}}
2019-05-08 12:42:11.499 23901-24241/com.eventyay.attendee D/OkHttp: <-- END HTTP (177-byte body)
2019-05-08 12:42:11.516 23901-23901/com.eventyay.attendee D/AttendeeViewModel$createOrder: Failed creating Order
    retrofit2.adapter.rxjava2.HttpException: HTTP 422 UNPROCESSABLE ENTITY
        at retrofit2.adapter.rxjava2.BodyObservable$BodyObserver.onNext(BodyObservable.java:54)
        at retrofit2.adapter.rxjava2.BodyObservable$BodyObserver.onNext(BodyObservable.java:37)
        at retrofit2.adapter.rxjava2.CallExecuteObservable.subscribeActual(CallExecuteObservable.java:44)
        at io.reactivex.Observable.subscribe(Observable.java:12267)
        at retrofit2.adapter.rxjava2.BodyObservable.subscribeActual(BodyObservable.java:34)
        at io.reactivex.Observable.subscribe(Observable.java:12267)
        at io.reactivex.internal.operators.observable.ObservableSingleSingle.subscribeActual(ObservableSingleSingle.java:35)
        at io.reactivex.Single.subscribe(Single.java:3603)
        at io.reactivex.internal.operators.single.SingleMap.subscribeActual(SingleMap.java:34)
        at io.reactivex.Single.subscribe(Single.java:3603)
        at io.reactivex.internal.operators.single.SingleSubscribeOn$SubscribeOnObserver.run(SingleSubscribeOn.java:89)
        at io.reactivex.Scheduler$DisposeTask.run(Scheduler.java:578)
        at io.reactivex.internal.schedulers.ScheduledRunnable.run(ScheduledRunnable.java:66)
        at io.reactivex.internal.schedulers.ScheduledRunnable.call(ScheduledRunnable.java:57)
        at java.util.concurrent.FutureTask.run(FutureTask.java:237)
        at java.util.concurrent.ScheduledThreadPoolExecutor$ScheduledFutureTask.run(ScheduledThreadPoolExecutor.java:272)
        at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1133)
        at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:607)
        at java.lang.Thread.run(Thread.java:760)

Would you like to work on the issue?

[ ] Yes

[x] No

Other: Not sure whether it is an issue with the app or on the server.

@liveHarshit @anhanh11001 Please have a look.

@ShridharGoel The amount field requires something. Please make sure that a free ticket is pushing a {amount : 0} in data

@uds5501 An order with a combination of paid and unpaid tickets is working fine.

@ShridharGoel Good, this means you just have to pass {amount : 0} in the post request of a free order, that should do

This is an issue on server and should be fixed. null should default to 0

@ShridharGoel this issue is fixed, can you close it?