JSON API setup doesn't return JSON API documents?

I have this query in my route:
    let promise = this.get('store').findRecord('host', subdomain, {
      adapterOptions: {
        query: {
          subdomain: subdomain
        },
      },
    });

and this in my tests:
    let org = server.create('organization', {
      name: 'Test Org',
      domain: 'testorg'
    });

    server.get('api/hosts/testorg', org);

I (hopefully) specified to use the JSON API serializer?
// mirage/serializers/application.js
import { JSONAPISerializer  } from 'ember-cli-mirage';
export default JSONAPISerializer;

so, when I run my test, I get this error:
"Assertion Failed: normalizeResponse must return a valid JSON API document: * One or more of the following keys must be present: "data", "errors", "meta"."

implying that data, errors, or meta are not present.
This is when I'm inspecting the reason parameter of the error action in my route. I don't know how to actually see the payload. (the transition object doesn't appear to have anything useful either)

typically the response is console.log'd, can you see anything in the console after the request?
If you're in a test you'll need to enable logging, as it's disabled by default (for the sake of verbosity in CI logs, etc). Enable it like this:
server.logging = true;

@NullVoxPopuli I don't know if this is helpful at all, but we were seeing the same error while upgrading from mirage 0.1.x to 0.2.0-beta9. It seemed to be caused by having the old directory (app/mirage) at the same time as having the new directory (mirage/).
Following the upgrade guide, including the step for moving files from the old directory to the new one, resolved the problem for us.

I don't have app/mirage anymore.
I didn't see any logs when I had server.logging = true;

the test:
    it('can view the name of the org', assert => {
      server.logging = true;
      visit('/testorg');
      andThen(() => {
        let h2s = find('h2');
        assert.ok(h2s.text().includes('Register for Test Org'));
      });
    });

That's strange... could you do me a favor, comment out the server.create line, and just do
server.get('api/hosts/testorg', { test: 'test' });

? If you're not seeing the response being logged Mirage isn't picking things up. If that doesn't work, try commenting out that line, and seeing if Mirage throws an unhandled route error handler.
Do you have the route defined in your mirage/config.js file?

without the server.create line, I see:
Successful request: GET /api/hosts/testorg?subdomain=testorg

I think I may have been just misreading the log.
I uncommented what I had originally, so:

I did not have the route defined in mirage/config.js.
I defined:
  this.get('api/hosts/:eventalias', function(db, request) {
    return {};
  });

it appears to have the same issue.
upgrading to 0.2.0 (from beta 9)
and I get the same error.
Right under the 'Successful Request' it shows an Object
is that supposed to be JSON API formatted? or...?

If you return {}, then that {} is your response. Since {} is not a json:api document, the Ember Data layer will be confused and throw errors.
You can always manually return a json:api document, i.e. return { data: { type: 'posts', attributes: ... but Mirage's serializer layer can help you. To use it, return a Mirage model from your route handler, and make sure your serializers/application.js is defined and exporting the JSONAPISerializer.
Make sure to read through the guides (especially serializers) and let me know if you still have questions.

I have removed the {}, and I still get the same problem. I believe I have follow all the instructions. I would like to use the ember-cli-mirage way of doing JSON API documents in testing, as the document format is cumbersome to do by hand.
Again, for reference, here is my repo: https://github.com/NullVoxPopuli/aeonvera-ui
And here are all relevant files:
// /mirage/serializers/application.js
import { JSONAPISerializer  } from 'ember-cli-mirage';
export default JSONAPISerializer;

// /mirage/config.js
  this.get('api/hosts/:eventalias');

// /mirage/factories/organization.js
import Mirage, { faker } from 'ember-cli-mirage';
export default Mirage.Factory.extend({});

// my test
import Ember from 'ember';
import { module, test } from 'qunit';
import startApp from 'aeonvera/tests/helpers/start-app';
import 'aeonvera/tests/helpers/login';

let application;

// fake mocha
let it = test;
let describe = function(name, tests) {
  tests();
};

module('Acceptance | Registration | Organization | User is Logged In', {
  beforeEach() {
    application = startApp();

    let org = server.create('organization', {
      name: 'Test Org',
      domain: 'testorg'
    });

    server.get('api/hosts/testorg', org);
  },

  afterEach() {
    Ember.run(application, 'destroy');
    server.shutdown();
  },
});

describe('User is Logged in', function() {
  describe('beginning at the registration index page', function() {
    it('can view the registration page', (assert) => {
      visit('/testorg');
      andThen(() => assert.equal(currentURL(), '/testorg'));
    });

    it('can view the name of the org', assert => {
      server.logging = true;
      visit('/testorg');
      andThen(() => {
        let h2s = find('h2');
        assert.ok(h2s.text().includes('Register for Test Org'));
      });
    });
  });
});

and the error:
"Assertion Failed: normalizeResponse must return a valid JSON API document: * One or more of the following keys must be present: "data", "errors", "meta"."
If this does work as you say it does, then I have to be missing something stupid. I. Just. Can't. Seem. To. Find. it. :-(

If I comment out the server.get in my test, then I get this error:
testem.js:775 Mirage: The route handler for /api/hosts/testorg?subdomain=testorg is trying to access the :eventalias model, but that model doesn't exist. Create it using 'ember g mirage-model :eventalias'.

this URL doesn't always return the same model. It can return any of two subclasses of :host

server.get('api/hosts/testorg', org); is wrong and won't pass through the serializer layer. Try reading though the Defining routes section of the guide, make your route handler dynamically respond with a model, and then please let me know if you're still stuck.

@samselikoffg  I'm now doing this:
// mirage/config.js
  this.get('api/hosts/:host', (schema, request) => {
    let hostId = request.params.host;

    if (hostId.includes('org')){
      return schema.db.organizations.find(hostId);
    }

    return schema.db.events.find(hostId);
  });

// in test's before each
    let org = server.create('organization', {
      id: 'testorg',
      name: 'Test Org',
      domain: 'testorg'
    });

(contrary to the docs, the schema doesn't have access to my factory data)
and I am still getting the same output as before
"Assertion Failed: normalizeResponse must return a valid JSON API document:↵	* One or more of the following keys must be present: "data", "errors", "meta"."

Successful request: GET /api/hosts/testorg?subdomain=testorg
testem.js:775 Object {domain: "testorg", name: "Test Org", id: "testorg"}

that console output is outputting a non JSON API object. :-\

Looks like it's not going through the serializer layer, have you set up JSONAPISerializer in mirage/serializers/application.js?

Yeah, I think I pasted it in an earlier comment. Also, here is the actual
file:
https://github.com/NullVoxPopuli/aeonvera-ui/blob/master/mirage/serializers/application.js
On Sat, Jun 4, 2016, 10:42 PM Sam Selikoff<EMAIL_ADDRESS>wrote:

Looks like it's not going through the serializer layer, have you set up
JSONAPISerializer in mirage/serializers/application.js?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/samselikoff/ember-cli-mirage/issues/746#issuecomment-223789844,
or mute the thread
https://github.com/notifications/unsubscribe/AAMJasQmCaWvVQaMs9vk0_eSgs6wErI8ks5qIjeFgaJpZM4Ijxxi
.

Oh, I see the problem. You're writing
return schema.db.organizations.find(hostId);

so you're just returning the raw db record, which doesn't get serialized. To use the serialize you need to return a Model or Collection. This should be
return schema.organizations.find(hostId);

You'll probably get an error, then, because it looks like you don't have your models defined. Try ember g mirage-model organization.

hey after I ran ember g mirage-model organization my test passed!!!! thank you!!!!! :-)


Can't cycle to another map

Cardinal is up to date, at the moment I'm writing this.
So the problem is that I start the server and sometimes everything works alright, at first (some other times I appear directly on a survival world...). Then when I /start or setnext and cycle, the real problems start. If I start, the match starts but I'm not on the spawn, I'm just where I was as an obs but with the perms to break blocks. If I cycle, the scoreboard and tab changes but the map doesn't.
I reverted to the cardinal I had on 1.8 and it worked with no problem... So it's not XML or map problem.

Logs? XML?

fixed

@Pablete1234 I'd wait to see if it works before closing it

as : https://github.com/twizmwazin/CardinalPGM/wiki/1.9-Update says, download lastest sportbukkit 1.9.2, and lastest cardinal

Still doesn't work. @TiTo-418 XML is fine because 1) it worked on 1.8 and 2) some of them are from maps.oc.tc.
@Pablete1234 Already did that and still doesn't work.
LOG:
latest.txt

@MrTheSharpie Please, use LASTEST sportbukkit 1.9.2, lastest = the one that is lowest on the ocn repo page

@Pablete1234 lmao why would they put lastest all the way down

@MrTheSharpie just look at the last number in the file, starts at SOMETHING-1.jar, lastest atm is SOMETHING-28.jar

It works now, thank you


How do you set the initial date range?

I can't find how to do that, I'd like to set for example last 30 days as default range.
I tried with ng-model="vm.dateRange" and:
vm.dateRange = {
	startDate: moment().subtract(30, 'd').format(vm.dateRangeFormat),
	endDate: moment().format(vm.dateRangeFormat)
};

But it converts the object in string

I was facing the same scenario and noticed the model takes string values. Try adding a string value.
Ex: vm.dateRange = moment().format('DD-MM-YYYY')+' -- '+  moment().add(30, 'days').format('DD-MM-YYYY') ;
Ideally, this should have been an object with 2 keys-value pairs.


Define code styling standards

As I move to make this project a little more "resume official," I need to make sure that things look clean. It's also just nice to have uniformity across different files.
I can create a .clang-format in the root directory and define my own styling standards there. More details on how to format this file and its properties can be found on the clang-format website.

Closing in favor of following Google's C++ Style Guide


Saving Image as byte[] and retrieving it fails

Im trying to save an image as byte[] and retrieving it after a while but the byte[] is not empty but when decoding it to Bitmap gives null.
byte[] img = userEntity.getProfileImage(); Bitmap bitmap = BitmapFactory.decodeByteArray(img,0,img.length);
Any suggestion?

I am still getting this am i doing something wrong?

Are you using the latest master branch?

I' am sorry i used the gradle dependency, i now added it as a library project and it works fine.
When will there be a new release so i can add it as a gradly dependency again?

Hey @tsm200, a bunch of people have asked that this be released on maven. I can't do it to the official maven distribution, but I can possibly publish to my own group id. I'll consider that option as we get closer to finalizing 1.4.0.

i get some issue .. i have update to latest library .. Any suggestion?

I have the same issue. I save a byte array of a certain size, but after a query, its length is always 12.


Feature/dataset composition

Added functions for dataset composition: zip, concat, cartesian product
Added a few more general dataset functions: take, repeat, reorder

Closing pull request, but opening new one with additional features added


WIP: Reduce (Server)StyleSheet SSR memory usage

Published as 3.2.3-ssrmem1 under the @ssrmem tag.

@kitten this seems to resolve the issue in our testing! 🎉


Apply at Google Season of Docs 2023

Let's discuss together what are high-profile documentation issues we want a technical writer to help us sort out.
https://developers.google.com/season-of-docs/docs/timeline says:

March 24, 2023 at 18:00 UTC         Deadline for organization applications

So we have a couple of weeks to write the proposal. Let's collect issues we would like to be resolved in the Mid-June - End-October timeframe.

I started drafting the proposal in https://github.com/fluxcd/website/pull/1370 - please let me know if there other areas we should address.

Closing in favour of https://github.com/cncf/mentoring/pull/875


Editor content not updating if value changes to empty without editor interaction

    @jscasca thank you for pointing it out, I may miss something but I mean the case when `value` as exported property is changed regardless of interraction with editor so the line 106 is not even executed.

The first time Svelet editor compoment instance receives 'some value' as value , it goes to editorRef.setContent(value) and it is displayed on UI becasue lastVal is empty and not equal to 'some value'.
Don't touch editor directly, the lastVal is still '' but assume smth happend and value prop of Svelet editor compoment recived '', the value changed from 'some value' to '' but it is still 'some value' in editor on UI because  editorRef.setContent(value) was not execute (lastVal === value === ''), basicaly mismatch between UI and state of Svelte Editor component.
I apologize but don't see this is solved by the line 106 or by other lines, as workaround I align the mismatch by using
  let mce;
  $: if (mce && value !== mce.getContent()) {
    mce.setContent(value);
  }
  const onInit = e => mce = e.detail.editor;

I hope this workaround works without unexpected side effects.
Originally posted by @EugeneGoroschenyaExt in https://github.com/tinymce/tinymce-svelte/issues/5#issuecomment-1284860642

@EugeneGoroschenyaExt moved here for tracking purposes. This should be addressed by #23

Fix was released for v1.0.1


docs: fixed broken link to polyfill

Thanks for the PR! I happened to fixed these links before I saw this PR though, so they're already fixed in c37f245fc8b360d30d8717861bf576d2eeaed390.


Add README for Microsoft.Sbom.Targets project

Add a README describing the targets project.

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 66.75%. Comparing base (31ed7e8) to head (a0758fc).

Additional details and impacted files
@@                      Coverage Diff                      @@
##           feature/sbom-targets-task     #651      +/-   ##
=============================================================
- Coverage                      69.69%   66.75%   -2.94%
=============================================================
  Files                            277      276       -1
  Lines                           8653     8586      -67
  Branches                        1009     1001       -8
=============================================================
- Hits                            6031     5732     -299
- Misses                          2105     2345     +240
+ Partials                         517      509       -8

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Detecting whether a form field is required or not

Hi everyone,
I would like to detect in my javascript code when a form field in a PDF is required but I cannot find the way.  I hope someone can point me in the right direction.
I attach a example PDF with two text fields: one required and another one not required. When I get the annotations of this PDF in my javascript code, for the required form field I get:
alternativeText: ""
annotationFlags: 4
annotationType: 20
borderStyle: {width: 0, style: 1, dashArray: Array(1), horizontalCornerRadius: 0, verticalCornerRadius: 0}
color: Uint8ClampedArray(3) [0, 0, 0]
comb: false
contents: ""
defaultAppearance: "/Helv 8.64 Tf 0 g"
fieldFlags: 2
fieldName: "name"
fieldType: "Tx"
fieldValue: ""
hasAppearance: true
id: "2R"
maxLen: null
modificationDate: null
multiLine: false
readOnly: false
rect: (4) [213.12, 704.38, 457.92, 723.82]
subtype: "Widget"
textAlignment: 0
There are properties for multiLine and readOnly, but there is any for "required". However I can see that the number of fieldFlags is 2, so I guess the way is to get access to this flags, however I don't know how to do this.
[Attach](url
pdfjsRequiredFieldExample.pdf
) (recommended) or Link to PDF file here:
Configuration:

Web browser and its version: Chrome Versión 89.0.4389.82 (Build oficial) (64 bits)
Operating system and its version:  Windows 10  - Version 10.0.18363 compilation 18363
PDF.js version: 2.6.347
Is a browser extension: No

Steps to reproduce the problem:

Load the PDF attached.
Get the form fields and detect, for each of them, whether is required or not

What is the expected behavior? (add screenshot) I would like to detect in my javascript code whether a form field is required or not.
What went wrong? (add screenshot) I cannot detect whether a form field is required or not
Thank you so much for your help!

There are properties for multiLine and readOnly, but there is any for "required".

Those have been added since the AnnotationLayer code needed them, however I doubt that we'd want to explicitly add a property for every single flag (if we're not using them anywhere).

However I can see that the number of fieldFlags is 2, so I guess the way is to get access to this flags, however I don't know how to do this.

Given that you already have access to the fieldFlags property, it's trivial to check if the "required" flag is set; simple test !!(fieldFlags & 2) and you're done :-)

Thank you so much for you reply, I have it working perfectly now! I see I misunderstood the meaning of the fieldFlags property.

Hi,
I am using pdf.js version 3.6.172.
It looks like that the fieldFlags is not available in this version.
Is there any way to detect whether a form field is required or not?
Thanks.


I'm unable to find a contract in the marketplace after I purchase it and it completes

In order to solve this, @joshtitan will need to add a smart contract function which will allow us to retrieve a list of contracts that a wallet address has purchased. Contracts currently disappear from the 'Completed' tab when someone else purchases it.

Closing, since this is a duplicate of #30


fix(c++): fix the 'multiple definition' bug when linking libarrow.a and libgraphar_bundled_dependencies.a in one CMakeLists.txt

fix the issues in https://github.com/apache/incubator-graphar/pull/628

Hi,@acezen @jasinliu.  This should solve the problem in https://github.com/apache/incubator-graphar/pull/628, please review this.

Hi,@acezen @jasinliu. This should solve the problem in #628, please review this.

Thanks! I think you can create some tests following #628 . Or you can provide the verification process through comments.

Hi,@acezen @jasinliu. This should solve the problem in #628, please review this.

Thanks! I think you can create some tests following #628 . Or you can provide the verification process through comments.

Yes, I used the totally same exmaple in the comment for testing and the installation steps are exactly the same, the results show that we can run this code correctly

Other example
test_read.cc
#include <cstdlib>
#include <iostream>
#include "arrow/api.h"
#include "graphar/api/arrow_reader.h"

int main() {
    std::string path = "/workspaces/incubator-graphar/testing/ldbc_sample/parquet/ldbc_sample.graph.yml";
    std::string src_type = "person", edge_type = "knows", dst_type = "person";
    std::string vertex_property_name = "id";
    std::string edge_property_name = "creationDate";
    auto maybe_graph_info = graphar::GraphInfo::Load(path);
    auto graph_info = maybe_graph_info.value();
    auto vertex_info = graph_info->GetVertexInfo(src_type);

    auto maybe_reader = graphar::VertexPropertyArrowChunkReader::Make(
    graph_info, src_type, vertex_property_name);
    auto reader = maybe_reader.value();
    std::cout << "Chunknum " << reader->GetChunkNum() << std::endl;

    auto result = reader->GetChunk();
    auto table = result.value();
    std::cout << "num_rows " << table->num_rows() << std::endl;

}

CMakeLists.txt
cmake_minimum_required(VERSION 3.15)

project(test_read)

set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} -std=c++17 -Wall")

set(TARGET test_read)

# find_package(OpenSSL REQUIRED)
find_package(graphar REQUIRED)
find_package(Arrow REQUIRED)

add_executable(${TARGET} test_read.cc)
target_link_libraries(${TARGET} PRIVATE graphar Arrow::arrow_static)

build
cmake . && make


Support boxed content and more flexible multi-column layouts

Objective of this PR
We need to support adding boxed content in a block grid in an editor-friendly way - specifically two types of box used on our corporate website:

a solid box with a grey background
a grey border around the standard white background

It was already possible to add the first style of box, but only within a block list by applying a well-known HTML class to a grid row.
We need to not break the existing support for adding a dividing line after the box.
Implementation
Box support for TPR styles
The .tpr-box class already existed, but could only be applied to a .govuk-grid-row, which would style its child .govuk-grid-column as a box. This is still supported, making this a non-breaking change. However it is now possible to use the class more simply, which is good for non-Umbraco applications:

My content here

A new page is added to the ASP.NET example app demonstrating boxes and divider lines, similar to the one in the Umbraco app.
In the Umbraco block grid, for TPR data types only because this is a TPR style, you can now add a 'Box' layout:

Once you've added the layout, you can set the style to either 'Solid' or 'Bordered' in the settings (the default is solid).

In the absence of a documented TPR standard for the border width, I used the $govuk-border-width variable to remain consistent with other components (although only 'Details' and 'Error summary' use this). It is set to 5px by default, whereas our current TPR website uses 4px, so this is a point for design review.
A box can be added inside a column of a multi-column layout, or a multi-column layout can be added inside a box. You can even have boxes within boxes, and this is all reflected in the backoffice previews:

Improved column support for GOV.UK and TPR styles
To match the support in the block list for adding a divider after a box, a setting for adding a custom class to a grid row was added to all grid layouts.
To enable this even for single-column layouts, new single column layouts are available for all the supported widths - one quarter, one third, one half, two thirds, three quarters and full width. In TPR styles these new single column layouts also allow a box to be created at any of those widths, by adding a box inside the layout.
There are lots of GOV.UK document types related to building grids now, so I gathered them in a new 'Grid' subfolder.
To enable multi-column layouts inside a box in TPR styles, all layouts are now enabled within areas. In either GOV.UK or TPR styles this means nested multi-column layouts are also possible, eg two equal columns inside the two-thirds column of a two-thirds/one-third layout.
The example page on dividers and boxes in the Umbraco app is updated with many more examples to demonstrate and test this support:

There's also an important bug fix - one third width columns were being rendered as one quarter wide.

Howdy, Rick. The only ones that weren't designed at 4px were the Details component's vertical bar and that of the Error state of a field / fieldset. The rest (e.g., Error summary, errored input field border, inner border of input text fields, radios and checkboxes when in focus state) were designed at 4px because of the bordered box on TPR's site. That is what was designed when the site was rebranded in 2018/19 so that's what I went with for consistency. The only reason Details and the Error state bars are 5px is because they weren't something we had on TPR, were taken directly from GOV, and they're not complete borders like the rest.
Here's what's been designed and used/documented in any prototypes I've done:

(Yellow Focus on input fields is 3px and inner border is 4px per GOV's. TPR Error field border is 4px because red alone is used on GOV (i.e., they keep the border 2px in error state) to denote the field error, so we made the field's border fatter to help distinguish it for users with protanopia as an extra affordance.)

(Yellow Focus and inner border on radios and checkboxes are 4px per GOV's)
So, as you can see, there's even inconsistency on GOV's! :)


How to modify /boot firmware to allow BTRFS root?

Hello,
Thanks for a great walkthrough. I can build an SD image with Docker!
Unfortunately, I've been working fruitlessly on getting a Pi to use a BTRFS root for a while.
I was hoping I could use this builder to create a boot partition whose u-boot images had BTRFS support enabled;  unfortunately, adding:
boot.initrd.supportedFilesystems = [ "btrfs" ];
boot.supportedFilesystems = [ "btrfs" ];

to my sd-image.nix produces an identical boot partition as compared to not having this (diff -r shows no differences, md5sum on *.bin is identical).
Do you happen to know why the /boot files aren't being affected?

I was able to get this to work on a Raspberry Pi 4 by simply copying the generated disk image to the SD card, then resizing the second partition with fdisk, overwriting it with mkfs.btrfs then mounting it. I also created a loop device with losetup -f -P /path/to/image.img, mounted the second partition and copied everything over. The configuration you edit before building the disk image needs to already have the correct configuration (/boot with the correct label and the correct btrfs sub volumes if you created any).
Don't try to also put /boot on Btrfs, it won't work. I didn't have to modify the boot partition in any way (I did edit the label with fatlabel though).
There is also no need for boot.initrd.supportedFilesystems or boot.supportedFilesystems since that is taken care of automatically by specifying fsType = "btrfs".


chore: move some IsIntegral results earlier

The first section of Mathlib.RingTheory.IntegralClosure.IsIntegralClosure.Basic contained results that could just as easily go into earlier files.

bors merge


Add ability to delete simulated devices from IoT Hub

Type of issue

[ ] Bug
[x] New feature
[ ] Enhancement

Description
When running a simulation, thousands of devices can be created in the IoT Hub. Since the Azure Portal provides no easy way of deleting all of these devices, it would be nice to have a way for the service to do a little hub cleanup on devices it created and no longer needs.

Update: This is already possible from the back-end today. If you submit a DELETE simulation/{id} it will delete the simulation and all of it's devices.
It would still be nice to be able to delete just specific device models.


About Custom Install

When i Running the Metatron Binary and making the CUSTOM INSTALL, after export METATRON_HOME and shell command "sh metatron.sh --init start", there appears an error in /logs/xxx.out file :"Error: Could not find or load main class org.springframework.boot.loader.PropertiesLauncher" what i wanna ask is , did you ever meet this error? Or can you give me any suggestion?

It seems to set the wrong METATRON_HOME path.
you have to set HOME directory to the root of the application.
please refer to the image below.

@alchan-lee I noticed that "${METATRON_HOME}/discovery-server/target" in /bin/common.sh, so the
METATRON_HOME should mean the root catalog. So, after maven install into a TAR，i should put the TAR in root catalog, not the initial catalog？Am i right?

@alchan-lee Thanks it resloved ,but i just remembered when i make the CUSTOM INSTALL first time, i set the METATRON_HOME root catalog of the all project and then worked ,so i misunderstood ,anyway, thanks!!

@BookXHD
I'm glad to hear it worked well for you.
If you discuss on our forum page when you have a general question, then more people would be participating.
https://groups.google.com/forum/#!forum/metatron-discovery
(not a forced rule, just suggestion.😃)
Anyway please feel free to share your difficulties,
Thanks.

Oh , of course , i don't know that before , i will next time~


review comments for man page

This PR includes review comments. Beside typos and other minor corrections, it also contains a reference to non-standard conform usage of the initialization arguments.

Update:

include review comments and add another typo-fix I forgot to add to the PR.

Update:

include another set of typos.


Compared with pointpillar

Hi, I'm a new user of TensroRT. I wonder how much mAP centerpoint-pointpillar increase and how much FPS it decrease compared to pointpillar.

Sorry, I didn't train a pointpillar model, so there is no way to compare.


Frontend environment setting

[ ] CI
[ ] eslint
[ ] .gitignore
[ ] package.json
[ ] README.md

We will use vue.js with Nuxt.js. We can easily start with this Nuxt.js installation guide

Understood.


Refactoring: Agent

Issue #, if available:
close #519
close #518
Description of changes:

Omit langchain dependency & use converse api to use tool
Refactor frontend to show the agent thinking clearly

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Screen shots


Performance issue with expanding product factor children

Sometimes clicking on a product factor node when another's children is visible doesn't expand the clicked product factor's children. Maybe this issue stems from me adding too many features inside of the svg display.

Updating a Boolean variable to be the opposite of itself by using x = x === false was causing errors every once in a while. So a different method of doing that was put in its place.


Feat: Add Lmod role

Add Lmod and configure custom path if needed.

Ok... I don't like capital letters, it sounds too much like windows :-)
Le jeu. 24 sept. 2020 à 15:01, Ox<EMAIL_ADDRESS>a écrit :

@oxedions commented on this pull request.
In roles/addons/Lmod/vars/RedHat.yml
https://github.com/bluebanquise/bluebanquise/pull/390#discussion_r494297648
:

@@ -0,0 +1,3 @@

+---
+Lmod_packages_to_install:
Hello !
Well, because the tool itself is everywhere using capital L (doc, spec,
code, etc). So I tried to comply with that 🙂
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/bluebanquise/bluebanquise/pull/390#discussion_r494297648,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABM6DU7UIPIAE5EWQFHYXN3SHM7MFANCNFSM4RYIG6XA
.


Blog for wechaty blog in Java project and common issues.

a new blog about using wechaty in java project, also with common issues and solutions.
a minor improvement in README.md for setup jekyll in local test env.
Make sure these boxes are checked before creating your pull request -- thank you!

[x] Add Blog Header, including title, author, date.
For example:
---
title: "Your Great Blog Title"
author: your_github_id
image: /assets/2020/08-your-blog-slug/teaser.jpg
---

YOUR_FIRST_SENTENCE_SHOULD_BE_A_SUMMARY_FOR_YOUR_POST

[x ] Keep all filenames & url as lowercase, and use - to connect words instead of space. e.g. 2017-10-06-wechat-pc-impactor instead of 2017-10-06-WeChat PC Impactor,

[ x] Embed the photo & video before publishing, save all external file to a folder named by ${MONTH}-${YOUR_BLOG_SLUG}/ under /assets/2020 directory.

[x ] Add teaser image for the blog

[x] Add your self as an author to the author data file at https://github.com/wechaty/wechaty.js.org/blob/master/_developers/YOUR_GITHUB_USERNAME.md with your GitHub username. And please also remember to reference yourself with the Github username in your blog post, so that we can have nice author information besides your blog after we published it!

It seems that we have got our first Java Wechaty blog post!
CC @wechaty/java

Thank you very much for this great blog post!
It should be published at https://wechaty.js.org/2020/10/26/java-wechaty-group-message-sync/ soon!


Create and publish skills-service docker image

Create and publish skills-service docker image

@sudo-may  - build-and-push.sh is confusingly referencing github username as opposed to docker repository username.


Seeding, torch and multiprocess in user guide.

Add explanations in the user guide about why we want seeding in rlberry, and how to see torch in the user guide. And reseed torch in the init of ExperimentManager.

Shoud be donne here : https://github.com/rlberry-py/rlberry/pull/463
Waiting for code review.
(and the torch reseed, is done in the _fit_worker from experiment_manager.py )


Send SMS problem

Hi,
I am trying to send a SMS, i am using the demo SMS. I put my number: 00+country code+9+local code+ my number (it works for the calls) but didnt work. I tryed all encode types. The tracer says that the sms was send but it dont.
The sim has credit and the network is attached
Thanks!
Jaz

number format should be +[country code][area code][mobile number] like +8651345552121


Copy banner no longer working

Copy banner seems to have stopped working

It seems it was overwritten by a previous version, am looking into it rn

Fixed


Music generation using deep learning

I am a Gssoc contributor .  I would like to take up this issue . I will implement this project using GAN . Please assign me this issue

hii I am a GSSOC contributor 2024 and I would like to work on this issue

I am gssoc'24 contributor @Niketkumardheeryan , can you please assign me this issue.

Hello sir @Niketkumardheeryan !
@Naidu-Saikiran  are not working on this project so sir can you kindly assigne this issue under GSSOC24


Document Prisma Logging, Prisma Call

https://github.com/prisma/prisma/issues/11329
Gist: We log a "Prisma Client call" that is the internal representation, not necessarily what the user has in their code (as that would be impossible in pure logging).

👋 we're cleaning up our backlog of issues and are closing everything older than two years. If you still feel that this issue is relevant, please feel free to re-open and our team will take a look. Thank you!


This does not seem to work for serial ports

This does not seem to work for COM ports in Windows 10?
rundll32.exe devmgr.dll,DeviceProperties_RunDLL /MachineID "" /DeviceID "ACPI\VEN_PNP&DEV_0501"
Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 3c5c7b52-24c3-5eb2-bd5e-ebb639590c48
Version Independent ID: c1670409-850f-86d0-03ad-18fe3997820b
Content: Device Instance ID - Windows drivers
Content Source: windows-driver-docs-pr/install/device-instance-ids.md
Product: windows-hardware
Technology: windows-devices
GitHub Login: @tedhudek
Microsoft Alias: tedhudek

Sorry for the delay! Looks like this feedback might be better suited to the doc page for rundll32.exe, which is https://docs.microsoft.com/en-us/windows-server/administration/windows-commands/rundll32


Add build operation around root build tree finished

This should allow to determine better when the execution phase finishes in scans, in particular when the root build doesn't execute tasks since its configuration failed.

@bot-gradle test and merge

Pre-tested commit build failed.

Your PR is queued. See the queue page for details.


Fixes for point release

This PR contains:

Use 'main' branch for experiments. #13839. Note: the change to main is already done but not available in the release branch so pulling in just that part.
Update to version and change log

/cc @rchiodo @DonJayamanne Please let me know if there are any other fixes that need to be in the point release.

Codecov Report

Merging #13841 into release-2020.08 will decrease coverage by 0.02%.
The diff coverage is 100.00%.

@@                 Coverage Diff                 @@
##           release-2020.08   #13841      +/-   ##
===================================================
- Coverage            59.75%   59.73%   -0.03%
===================================================
  Files                  671      671
  Lines                37386    37386
  Branches              5314     5314
===================================================
- Hits                 22341    22332       -9
- Misses               13909    13915       +6
- Partials              1136     1139       +3

Impacted Files
Coverage Δ

src/client/common/experiments/manager.ts
95.94% <100.00%> (ø)

src/client/common/utils/platform.ts
64.70% <0.00%> (-11.77%)
:arrow_down:

src/client/datascience/crossProcessLock.ts
79.41% <0.00%> (-11.77%)
:arrow_down:

src/client/linters/pydocstyle.ts
86.66% <0.00%> (-2.23%)
:arrow_down:

src/client/datascience/debugLocationTracker.ts
76.56% <0.00%> (-1.57%)
:arrow_down:

src/client/common/process/proc.ts
14.49% <0.00%> (-0.73%)
:arrow_down:

Continue to review full report at Codecov.

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


Pass context through cmd and use RunE instead of Run

Pass context by calling cmd.ExecuteContext(ctx) and retrieving it using cmd.Context(). Also, call cmdutil.CheckErr only in main.go and use RunE and PreRunE to pass the error to main.go.
An example of cmctl check api -v4 --logging-format=json --wait=5m still working:
{"ts":1709802469036.6274,"caller":"api/api.go:117","msg":"Not ready","v":2,"logger":"cert-manager.cmctl.checkAPI","err":"the cert-manager CRDs are not yet installed on the Kubernetes API server","underlyingError":"error finding the scope of the object: failed to get restmapping: no matches for kind \"Certificate\" in group \"cert-manager.io\""}
{"ts":1709802474038.2876,"caller":"api/api.go:117","msg":"Not ready","v":2,"logger":"cert-manager.cmctl.checkAPI","err":"the cert-manager CRDs are not yet installed on the Kubernetes API server","underlyingError":"error finding the scope of the object: failed to get restmapping: no matches for kind \"Certificate\" in group \"cert-manager.io\""}
^Cerror: the cert-manager CRDs are not yet installed on the Kubernetes API server
exit status 130

/approve


Update README.md

Slight typo. "a nd language of choice" ===> "and language of choice"

Thanks, LGTM.


Allowing source language to be an optional paramete

I came across this problem when I am designing my app:
Some words have different meaning in different language (lots of words in English have the same spelling as French, but we do not know what the source language is when the API auto detect language).
Hope this help!

@danialkalbasi  Hey, when will you be able to complete the pull request? Thanks :)

I think it is better to make the target language required and the source optional. The Google API detects the source language if it is not provided


The identity GNMI_STREAMING_OBJECT_TYPE does not use tapi-common:OBJECT_TYPE as its base

The identity GNMI_STREAMING_OBJECT_TYPE is defined as below without any base:
    identity GNMI_STREAMING_OBJECT_TYPE {
        description "none";
    }

And GNMI_STREAMING_OBJECT_TYPE_MEASUREMENT_DETAILS is defined as:
    identity GNMI_STREAMING_OBJECT_TYPE_MEASUREMENT_DETAILS {
        base GNMI_STREAMING_OBJECT_TYPE;
        description "none";
    }

I am suspecting this will create a Yang modeling issue when representing a supported-stream-type object with stream-type-content as "GNMI_STREAMING_OBJECT_TYPE_MEASUREMENT_DETAILS" because the type of stream-type-content is tapi-common:object-type.
       +--ro supported-stream-type* [uuid]
          +-- ...
          +--ro stream-type-content*           tapi-common:object-type

Please note that all other object-type identities use tapi-common:OBJECT_TYPE as their base. Please see 2 examples below:
    identity CONNECTIVITY_OBJECT_TYPE {
    	base tapi-common:OBJECT_TYPE;
        description "none";
    }

    identity STREAMING_OBJECT_TYPE {
        base tapi-common:OBJECT_TYPE;
        description "none";
    }

CC: @amazzini @nigel-r-davis

Yes looks like an oversight to be fixed. To be discussed how, e.g. delivering a 2.5.2 patch version or waiting for 2.6.0

Fixed in .yang candidate for 2.6.0

This is now fixed in 2.6.0.


[No QA] Fix StagingDeployCash isAutomatedPullRequest check

Details
Following-up on https://github.com/Expensify/Expensify.cash/pull/3173
It's mostly working, but there are two remaining issues that are resolved by this PR:

Use the html_url instead of the url –> we want the html url not the API url
The ever-so-common pitfall (this is my second time fixing today!) that by default listing things from the GitHub API by will default only return open things. But in this case we want to check for PRs in any state.

Fixed Issues
Fixes borked StagingDeployCash here: https://github.com/Expensify/Expensify.cash/issues/3593
Tests
Merge to test. I also updated the unit tests here.
QA Steps
None.
Tested On
N/A – GitHub only

Merging to test! I didn't change any source code that will affect iOS.

🚀 Deployed to staging in version: 1.0.70-7🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅

🚀 Deployed to production in version: 1.0.73-3🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅


Request: Custom Field Options for CLLC

Would you consider adding two new custom field options for Creature Level and Loot Control (CLLC)?
ConditionStarMin
ConditionStarMax

There is no need. The conditions ConditionMinLevel and ConditionMaxLevel can be used (levels are equal to stars - 1, so 1 star is level 2, 0 stars is level 1).
CLLC doesn't add anything custom with regards to stars apart from displaying more than two. It is always possible to spawn them at higher levels, it just doesn't display unless a mod adds some extra UI.

I'm currently using the ConditionMinLevel and ConditionMaxLevel, but realized if we spawn any mobs that are a different star level in a different geographic location (e.g., a 3-star in a 1-star zone), the config will treat their drops based on the geographic location, instead of the star level.

Maybe I missed something. Is ConditionMinLevel and ConditionMaxLevel equivalent to the star system for mobs?

Its probably one of those cases where the documentation is lacking.
"Levels" are completely equivalent to stars. When a creature is level 1, no stars are displayed. When its level 2, 1 stars, level 3 its 2 stars and so on.
Its just that in vanilla there is no UI for more than 2 stars, so the game displays none. CLLC adds UI to handle this problem, and adds tweaks to the levels are assigned to the creature on spawn.
In terms of the conditions, all conditions are required fulfilled for the item to drop, so if you have both a location and level condition, both checks need to pass for the drop to be "valid".
If your request is more related to the actual sector-system of CLLC I am not going to support it. I investigated it a while back, but it is not something that CLLC exposes to other mods, and it seemed complicated enough that I did not see a good way to present the configs to a user in the Drop That format.

Completely understand. Thanks for responses and the time to provide them! I wasn't sure how to contact you, so used this "issue" forum. My apologies if it caused any problem.

Its alright. These things are not really that clear, especially when it comes to cross-mod compatibility and what each mod decided to call their features


Re-Implement Comment deletion without using asp.net core form

No change to functionality, just changing this from using the asp.net form component.
This is needed to support automatic comment updates.

/azp run APIView - CI


Question: how to best make this extension to CamelCaseMethodName

In my projects, camelCase is required for method names. There however is an exception: tests. This is also allowed: testGivenInvalidLanguage_exceptionIsThrown.
The rules for this are:

only applies to methods starting with "test" (in classes that end on "Test")
only a single _ is allowed (though not required)
the character after the _ should be lowercase
the _ should separate the "given" and/or "when" from the "then" clause

3 or more words (including "test") before the _
2 or more words after the _

I'd like to implement this. Any advice on how to do this? Shall I create a new standalone rule, one that extends CamelCaseMethodName, or add an option to CamelCaseMethodName? And for the former two, would that be a welcome addition to this repo or not?

+1 Would definetely like this as well. I follow a similar naming pattern for tests.

I've added an option in #256. It's a boolean option to allow for a single underscore in the method name of a test. It's not as strict as it can be - some regex wizard can probably make it better.

The basic option got merged now. Just came up with a regex that applies all rules I outlined (except the class context one): /^test(([A-Z][a-z0-9]+)+|(([A-Z][a-z0-9]+){2,}_[a-z0-9]+([A-Z][a-z0-9]+)+))$/
Will submit a PR with that soon

@JeroenDeDauw :cool:

I just noticed the 2.3 has already happened. That means updating the regex to be more restrictive would be a breaking change. Perhaps adding a new rule specifically for test methods would be good? Then you can enabled just that rule for your test methods, or also have the CamelCase one on with the already introduced allow-underscore-test option.

According to SemVer it would, yes. But for this time, so shortly after release, I would give it a go, nonetheless. It was a new feature and it was merged just before the release, so we can assume very few people have it already implemented. But htanks for caring!

I have started implementation in #313. Still need to add the integration tests and make this rule not apply to methods starting with test in non test classes.

@JeroenDeDauw though I closed your PR, here some other thoughts on your issue:
I guess the actual "problem" is that you use no valid naming style. testGivenInvalidLanguage_exceptionIsThrown is simply not a widely accepted way of naming methods. Think of it, nobody gave it a name yet. It's not a standard, yet.
The fact, that PHPMD might also does stuff it's not 100 % made for (checking method names etc.) doesn't help either. PHPCS might be the better tool for that.
Anyway, a last idea that might help not only you:
What if we implement a ignore-in-tests property, which we can use in all PHPMD rules to disable the rule for test files/classes/methods? As for the definition of what a test file/class/method is, I would orient myself on PHPUnit's conventions.

Closed PR: https://github.com/phpmd/phpmd/pull/313

@ravage84 perhaps it's a bit premature to close this issue then?
IIRC we can already have the naming rule be ignored in tests, simply leaving us with no check for the names at all. Not been suing PHPMD for some time due to PHP 7 compat not being there, so I don't recall the details.

Yes, one could simply exclude the test folder for some rules. So it's arguable if the ignore-in-tests property would of any use. That's why I did not create a new issue for it.
Feel free to re-open, though I think the question of this issue is quite specific. This specific that I think that it's not even a fitting task for PHPMD, but more one for PHPCS, because PHPCS is about style and - for me - your issue is a style question, not a mess to detect.

By the way the PHP 7 compat should have been improved, as most issues were fixed in Pdepend.

I think it's weird to have some naming convention rules and then not accept additions cause the tool is not about naming conventions.
People not part of the org cannot re-open tickets

I see your point.  While I personally use the CamelCaseXyz rules myself and even extended some to add a missing feature, I feel we should not evolve much further into this territory with PHPMD, as I think there are more fitting tools, such as PHPCS. This is my personal view, not sure how @manuelpichler feels about this. Mainly it his project, I came late to the show, when all those rules existed already.

People not part of the org cannot re-open tickets

Oh, didn't know/realize. Shall I re-open then?

I am no longer using PHPMD in part because of this issue, so yeah, still relevant.


Empty Argument Supporting

Thank you very much for providing this powerful tool. I have encountered an issue and I'm not sure how to solve it. If it's not currently possible, would you consider adding support for it in the future?
The need is as follows: For example, consider a unit test command:
test: |
    if [[ -z "${0}" ]]; then
      echo "test all"
      flutter test
    else
      echo "test ${0}"
      flutter test --name="${0}"
    fi

test_name: flutter test --name="${0}"

Current Error:
Error! The script "test" defines a positional argument ${0}, but 0 positional argument(s) are given.
If it could support checking the number of parameters or handling the case where the default parameter is empty, we could use a single command to support both rps test (for running all test cases) and rps test "demo test" (for running specific test cases by name). This would allow us to avoid having to write two separate commands for the same requirement. There might be workarounds to achieve this, but it would be great to have support for such syntax directly. Thanks!

add a default argument value "all" solved it, however, ${#} for argument count and default empty value for ${n} instead of raising an exception will be better


Japan: Refatored japanese holidays

Included missing Meeus algorithms for equinox and solstice
Refactored japanese holidays to make use of Meeus algorithms
Include Meeus utils tests and japanese holidays tests

Hey @Nalguedo,
this PR https://github.com/dr-prodigy/python-holidays/pull/862 may help you with the failing tests.

Hey @Nalguedo, this PR #862 may help you with the failing tests.

Shall I wait for this to be merged, or is there any way to force the CI to cherry-pick that commit?
Is there any plans to pin deps and transitive deps using pip-tools or poetry? ... I can help with that ;)

Coverage remained the same at 100.0% when pulling 644caf22e55badacc05fbeb6c558235a3f2fc73d on Nalguedo:pb-equinox-japan into 6a04aab15fc9d4a0dc422acb1f65e5441bfb0642 on dr-prodigy:beta.

Hey @Nalguedo, this PR #862 may help you with the failing tests.

Shall I wait for this to be merged, or is there any way to force the CI to cherry-pick that commit?
Is there any plans to pin deps and transitive deps using pip-tools or poetry? ... I can help with that ;)

Hi @Nalguedo thank you for your work! Given the blocking issue, I've merged @arkid15r 's PR already.
PS: while, philosophically speaking, we always try to avoid dep pinning, we could consider your interesting proposal .. we'll discuss it as soon as possible :+1:

If this change is OK, I'll remove Meeus algorithms from Utils and refactor Chile holiday using the same approach
@dr-prodigy @arkid15r @KJhellico

If this change is OK, I'll remove Meeus algorithms from Utils and refactor Chile holiday using the same approach @dr-prodigy @arkid15r @KJhellico

I'm okay with the general approach. Didn't look into the implementations though.

Looks good 👍

Thank you very much ! :+1:


CommodityMarketInfoView Optimization

Currently VERY slow
Hopefully indexing fixes this, but this is something we will need to look into.


Duck Player shows empty page instead of requested video content

Describe the bug
Hello. Duck Player that is provided via the Videos page is not operational; at duck://player/_11-character string_, an empty page is shown.
How to Reproduce

In Videos page, select an video item for which an option for viewing a video with Duck Player is exhibited.
Click  Turn on Duck Player.
Observe the resulting behavior.

Expected behavior
The resulting redirect to duck://player/<11-character string> not to show an empty page, but instead queried video content.
Environment
- DuckDuckGo version: 5.216.2
- Android version: 8.0.0

Hi @Ricky-Tigg,
Thank you for reporting the issue. It looks like there's some issue on Android 8 preventing Duck Player from loading. We'll try to address it


Swapping fedi to lightning stuck at 'pending'

Yesterday, After a successful large swap from fedi to lightning, I attempted to swap the remaining balance (slightly less than before) , but the txn is stuck as 'pending'
Initially, the txn balance was not showing up in fedi or lightning or total balance, but today it looks like the swaped amount is back in my fedi and included in my total balance.  But trying to swap to LN again gives an error message 'swap must be above 100k sats' even though the txn is much larger.
The previous, failed swap still shows as 'pending' in my activity.

if the balance is back in your balance it should be fine to retry, there is currently a bug where the swap error message persists, that can be safely ignored.

When I retry, I get an error after previewing swap fee that the swap amount must be over 100k sats, and the swap is 8x that minimum amount...

there is currently a bug where the swap error message persists, that can be safely ignored.

The only option presented is 'preview swap fee' which just persists the error message.
There is no way to proceed past this screen and compete the swap that I can see.

the button should be clickable as long as you don't enter a number higher than your balance

Then just move your funds off to a different wallet.


Can't build VPK on Wine

.NET is installed from this file: https://dotnet.microsoft.com/download/dotnet/thank-you/runtime-desktop-5.0.0-windows-x64-installer
After supplying all the files the program was asking for and filling the required fields, the "Build VPK" button is still greyed out.
Any clue of what I'm doing wrong?
I also didn't understand the instructions for compiling the program for Linux, thus I went for the Windows version emulated on Wine.

The Distribution Tool will not allow you to build a VPK if the format of your provided information is not correct. Check the hints of the input fields for proper formatting.
An incorrectly formatted param.sfo file can cause errors while installing VPKs which is why that check is there

I think I managed to find out what was the issue:
The Vita normally has game IDs starting with 4 letters and followed by 5 numbers, and I was trying some random game ID (something like GAME4VITA).
I'll test it some more later, but for now, it seems to be compiling the game I'm using to test.

And yes, just like the hint suggested, although I didn't notice at first.


Early stop on BLEU (or a metric other than loss)

Is there a way to specify a metric other than loss to early stop against? Specifically, I'm looking to early stop on the validation bleu metric instead of the loss. It looks like fairseq allows us to specify some metrics to stop against, but apart from loss, nll loss or ppl, they're pretty useless for gauging the quantitative performance of the model.
Would early stopping on BLEU be a feature I'd need to implement myself?
Thanks

Read the documentation and examples/scaling-nmt where this feature wasn't mentioned.. but it is mentioned in the examples/translation folder: https://github.com/facebookresearch/fairseq/tree/main/examples/translation


Add Network Preview Version 2022-04-01-preview

MSFT employees can try out our new experience at OpenAPI Hub  - one location for using our validation tools and finding your workflow.

Changelog
Add a changelog entry for this PR by answering the following questions:

What's the purpose of the update?

[ ] new service onboarding
[x] new API version
[ ] update existing version for new feature
[ ] update existing version to fix swagger quality issue in s360
[ ] Other, please clarify

When are you targeting to deploy the new service/feature to public regions? Please provide the date or, if the date is not yet available, the month.
When do you expect to publish the swagger? Please provide date or, the the date is not yet available, the month.
If updating an existing version, please select the specific language SDKs and CLIs that must be refreshed after the swagger is published.

[x] SDK of .NET (need service team to ensure code readiness)
[x] SDK of Python
[x] SDK of Java
[ ] SDK of Js
[ ] SDK of Go
[x] PowerShell
[x] CLI
[x] Terraform
[ ] No refresh required for updates in this PR

Contribution checklist:

[x] I commit to follow the Breaking Change Policy of "no breaking changes"
[x] I have reviewed the documentation for the workflow.
[x] Validation tools were run on swagger spec(s) and errors have all been fixed in this PR. How to fix?

If any further question about AME onboarding or validation tools, please view the FAQ.
ARM API Review Checklist

Applicability: :warning:
If your changes encompass only the following scenarios, you should SKIP this section, as these scenarios do not require ARM review.

Change to data plane APIs
Adding new properties
All removals

Otherwise your PR may be subject to ARM review requirements. Complete the following:

[x] Check this box if any of the following apply to the PR so that label "WaitForARMFeedback" will be added automatically to begin ARM API Review. Failure to comply may result in delays to the manifest.

Adding a new service
Adding new API(s)
Adding a new API version
-[ ] To review changes efficiently, ensure you are using OpenAPIHub to initialize the PR for adding a new version. More details, refer to the wiki.

[x] Ensure you've reviewed following guidelines including ARM resource provider contract and REST guidelines. Estimated time (4 hours). This is required before you can request review from ARM API Review board.

[x] If you are blocked on ARM review and want to get the PR merged with urgency, please get the ARM oncall for reviews (RP Manifest Approvers team under Azure Resource Manager service) from IcM and reach out to them.

Breaking Change Review Checklist
If any of the following scenarios apply to the PR, request approval from the Breaking Change Review Board as defined in the Breaking Change Policy.

[ ] Removing API(s) in a stable version
[ ] Removing properties in a stable version
[ ] Removing API version(s) in a stable version
[ ] Updating API in a stable or public preview version with Breaking Change Validation errors
[ ] Updating API(s) in public preview over 1 year (refer to Retirement of Previews)

Action: to initiate an evaluation of the breaking change, create a new intake using the template for breaking changes. Addition details on the process and office hours are on the Breaking change Wiki.
Please follow the link to find more details on PR review process.

LGTM

Please make sure you check the right check boxes for the review to take place. thanks.

Please make sure you check the right check boxes for the review to take place. thanks.

Thanks for quick reply, I have updated all checked boxes. THanks!

Please make sure you check the right check boxes for the review to take place. thanks.

Thanks for quick reply, I have updated all checked boxes. THanks!

Please also check all the errors and let us know when it's ready to be reviewed. Thanks.


Cannot combine gRPC based remote execution with HTTP-based caching

Description of the bug:
I am am RBE/Cache with gRPC (BuildBuddy) but IntelliJ (Clion) is giving me the error message:

Cannot combine gRPC based remote execution with HTTP-based caching

I don't have any cache with HTTP(S) in my .bazelrc
# Abseil needs at least C++14
build --cxxopt='-std=c++14'

build --bes_results_url=https://app.buildbuddy.io/invocation/
build --bes_backend=grpcs://remote.buildbuddy.io
build --remote_cache=grpcs://remote.buildbuddy.io

## The following is for RBE with BuildBuddy
build:remote --remote_executor=grpcs://remote.buildbuddy.io
build:remote --host_platform=@buildbuddy_toolchain//:platform
build:remote --platforms=@buildbuddy_toolchain//:platform
build:remote --extra_execution_platforms=@buildbuddy_toolchain//:platform
build:remote --crosstool_top=@buildbuddy_toolchain//:toolchain
build:remote --extra_toolchains=@buildbuddy_toolchain//:cc_toolchain
# These options are listed as deprecated. We don't use Java
# so just comment them out for now.
# build:remote --javabase=@buildbuddy_toolchain//:javabase_jdk8
# build:remote --host_javabase=@buildbuddy_toolchain//:javabase_jdk8
# build:remote --java_toolchain=@buildbuddy_toolchain//:toolchain_jdk8
# build:remote --host_java_toolchain=@buildbuddy_toolchain//:toolchain_jdk8

# Starting with Bazel 0.27.0 strategies do not need to be explicitly
# defined. See https://github.com/bazelbuild/bazel/issues/7480
build:remote --define=EXECUTOR=remote

# Enable remote execution so actions are performed on the remote systems.
build:remote --remote_executor=grpcs://remote.buildbuddy.io

# Enforce stricter environment rules, which eliminates some non-hermetic
# behavior and therefore improves both the remote cache hit rate and the
# correctness and repeatability of the build.
build:remote --incompatible_strict_action_env=true

# Set a higher timeout value, just in case.
build:remote --remote_timeout=3600

# Depending on how many machines are in the remote execution instance, setting
# this higher can make builds faster by allowing more jobs to run in parallel.
# Setting it too high can result in jobs that timeout, however, while waiting
# for a remote machine to execute them.
build:remote --jobs=50

# Add a bunch of useful status commands for BuildBuddy
build:remote --workspace_status_command=$(pwd)/workspace_status.sh

# Improve remote cache throughput.
build:remote --experimental_remote_cache_compression

# Reduce the number of unnecessary cache uploads that Bazel might perform during a build.
# build:remote --remote_build_event_upload=minimal

# More detailed timing information
build:remote --noslim_profile --experimental_profile_include_target_label --experimental_profile_include_primary_output

# Reduce the payload size of the uploaded build event stream by eliminating duplicate file references.
build:remote --nolegacy_important_outputs

# Try to import per user bazelrc
# This should have API key for BuildBuddy
try-import %workspace%/user.bazelrc

My user.bazelrc
build --remote_header=STRIPPED-FOR-PII

# Default to always using remote
build --config=remote

What's the simplest, easiest way to reproduce this bug? Please provide a minimal example if possible.
No response
Which Intellij IDE are you using? Please provide the specific version.
CLion 2022.3.3
What programming languages and tools are you using? Please provide specific versions.
No response
What Bazel plugin version are you using?
2<IP_ADDRESS>.1-api-version-223
Have you found anything relevant by searching the web?
No
Any other information, logs, or outputs that you want to share?
No response

I found https://github.com/bazelbuild/bazel/blob/master/src/main/java/com/google/devtools/build/lib/remote/RemoteModule.java#L340 but enableHttpCache checks for remote_cache to start with http(s)

Closing this :(
This was some stupidity on my part.
I opened CLion from the terminal from a different folder and IntelliJ inherited the bazel command from there which was wrapped to include some HTTP caching


Update HLSL_ShaderModel6_5.md

Hi,
Before my change, it says "Note how subset with mask.x == 0x0b refers to lane 1, which is either inactive or is a helper lane". Since mask 0x0b is for lane 0 and it's Prefix, lane 1 shouldn't be considered anyway. I think set the second bit of mask in lane 2 will reduce possibility of confusion.
It also mentioned "The groups are assumed to be non-intersecting". However, mask 0xb and 0x9 have overlaps in the example, so I suggest to update 0xb to 0x2.
BTW, do you really want the groups to be non-intersecting? I think there are quite some scenarios in real world needs to reuse same lane in different groups.
Thanks,
Mengbo

Hi Mengbo,
I checked in with someone with more context on this, and it is actually correct as written. I added some additional clarification in https://github.com/microsoft/DirectX-Specs/pull/116 to hopefully make this more clear going forward.
Thanks!


Reset ghost rotation to zero on follow

About the PR
:cl:

tweak: Ghosts now always have a consistent directional sprite while orbiting an entity

actually this does not always work

ill fix the rest of it later


Please consider fixing issues detected by repository checker

Notification from ioBroker Check and Service Bot
Dear adapter developer,
I'm the ioBroker Check and Service Bot. I'm an automated tool processing routine tasks for the ioBroker infrastructure. I have recently checked the repository for your adapter radar-trap for common errors and appropiate suggestions to keep this adapter up to date.
This check is based the current head revisions (master / main  branch) of the adapter repository
Please see the result of the check below.
ioBroker.radar-trap
   -
ERRORS:

[ ] :heavy_exclamation_mark: [E204] Versions "2.1.1, 2.1.0" listed at common.news at io-package.json do not exist at NPM. Please remove from news section.
[ ] :heavy_exclamation_mark: [E851] .npmignore found but "files" is used at package.json. Please remove .npmignore.

WARNINGS:

[ ] :eyes: [W040] "keywords" within package.json should contain "ioBroker"
[ ] :eyes: [W162] js-controller 5.0.17 listed as dependency but 5.0.19 is recommended. Please consider updating dependency at io-package.json.

:thumbsup: No suggestions found
Please review issues reported and consider fixing them as soon as appropiate.
Errors reported by repository checker should be fixed as soon as possible. Some of them require a new release to be considered as fixed. Please note that errors reported by checker might be considered as blocking point for future updates at stable repository.
Warnings reported by repository checker should be reviewed. While some warnings can be ignored due to good reasons or a dedicated decision of the developer, most warnings should be fixed as soon as appropiate.
You may start a new check or force the creation of a new issue at any time by adding the following comment to this issue:
@iobroker-bot recheck
@iobroker-bot recreate
Please note that I (and the server at GitHub) have always plenty of work to do. So it may last up to 30 minutes until you see a reaction. I will drop a comment here as soon as I start processing.
Feel free to contact me (@iobroker-bot) if you have any questions or feel that an issue is incorrectly flagged.
And THANKS A LOT for maintaining this adapter from me and all users.
Let's work together for the best user experience.
your
ioBroker Check and Service Bot
@mcm1957 for evidence
Last update at Wed, 11 Sep 2024 05:36:28 GMT

Issue outdated due to RECREATE request. Follow up issue #16 has been created.
This issue will be closed.
your
ioBroker Check and Service Bot


hi  mate, wanted to ask you about your new pump bot. In strategy - volume btc - how do i setup the investment amount?

thank you

CCXT unfortunately can't post an order to cryptopia:
https://github.com/ccxt/ccxt/blob/master/js/cryptopia.js


fix: ensure study has valid wbs when creating data sets

Related to https://github.com/equinor/sepes-web/issues/1460
This should also not be allowed on API level.
Api frist strategy

This is already solved, my mistake


request info: improve downstream/upstream timings

This patch continues on the work in #743, adding the various downstream/upstream timings described in the gRPC access log payload to RequestInfo. As a result, the existing timings (requestReceived, responseReceived, and duration) have been removed in favor of these new ones. The access log (both gRPC and file-based) utilities have also been updated to use and/or emit the new values. The duration are now exposed as MonotonicTime, allowing a consumer to choose the appropriate cast (ie, access log formatter uses ms, while gRPC timestamps support nanosecond precision); a utility class was added to facilitate the conversion.
Couple questions that came up while working on this:

Does the RequestInfo on the TcpProxy need to have timings populated? Does that make sense?
Should the UpstreamRequest's RequestInfo have timings attached as well? Does that make sense?
For firstUpstreamRxByteReceived, it looks like the closest I could get it to the actual first byte is inside UpstreamRequest's decodeHeaders. Is there a better, more accurate place to capture the time?

Risk Level: Low-Medium (changes how some existing timings are calculated)
Testing: Unit
Docs Changes: N/A (these are still marked unimplemented in the data plane, but if desired I can add documentation for these timings)
Release Notes: N/A (unless it's worthwhile to mention that RESPONSE_DURATION and DURATION are calculated differently in the existing logs?)
/cc @derekargueta (#2654)
Signed-off-by: Chris Roche<EMAIL_ADDRESS>

Also, TSAN build is failing

It would be great if you could add all the new fields to text-based access logs as well, to keep parity between the two.

Was going to do this in a follow up PR for #2654, but I can do it here if desired.

@rodaine I'm fine with doing the text-logging in a followup PR.

@ggreenway @alyssawilk I think this is ready for another look!


added 2020 with storage api:fied urls

i mimicked the storage format: https://storage.googleapis.com/acid-december2012/<YYYY>
will send you zip of audio separately

\ o /


Add fire_replaced_class_double.

fire_replaced_class_double is like fire_class_double, plus it temporarily replaces the named constant with itself.  This is handy when the code-under-test references the named constant.  The double also registers itself with rspec-mocks space to ensure it gets reset after each example.  When it is reset it ensures all constants are as they originally were.

Thanks for the contribution! I'm flat out the next couple of days, but am keen to take a look.

I discovered a bug while using it, so I fixed that and pushed the fix.
BTW, while I think this feature is really cool and works well, the implementation is reallllly messy as you can see.  When you take a look, let me know if you can think of some ways to clean up the implementation.

Is the use case for this so you don't have to do this?
class User; end

describe 'whatever' do
  it 'uses user' do
    MyClass.method_depends_on_user
  end
end

Maybe write a few words in the README? Sorry just trying to get my head around when I would use this.

Yeah, I'm not super happy with fire_replaced_class_double, either.  A few random thoughts:

While I think you're right that separating the double creation from the constant replacement is a bit weird (and problematic, for reasons I'll explain below), it strikes me that the constant is useful in its own right, even outside of rspec-fire.  I could see using this to stub a constant that just returns a numeric value.  Sometimes I've chosen to implement something as a class method rather than a constant simply to make it stubbable.  If there was a simple proper wayt o stub a constant, that wouldn't be necessary anymore.  Maybe the constant stubbing code should be moved into rspec-mocks itself?  I can see what the other members of the rspec core team think about that if you like the idea.
Initially, I was planning to do something like fire_class_double("User").as_replaced_const, but I realized that the double's interface verification logic gets affected by the fact that the constant has been replaced with the double.  That's why I wound up going with fire_replaced_class_double and inheritance.  The problems are in ensure_arity and ensure_implemented.  Currently, it attempts to re-resolve the constant, and then it uses the returned class to verify the interface.  But that doesn't work for a constant that has been replaced; instead it needs to store a reference to the original class constant and use that when verifying the interface instead.  That's the reason I refactored to have it use the new with_doubled_class method.
So I think that conceptually, I like there being a way to stub constants separate from the main rspec-fire stuff, I think that a fire_class_double needs to be aware of the constant replacement to work properly.  Thus it probably needs to be a single method call to do it.
It seems to me that in virtually every circumstance I would use fire_class_double, I would want the constant to be replaced as well.  I think we need to keep a way to not do the constant replacement but I would expect it to be the more common case, and optimizing the API for it is probably a good idea.
You mention the whole const vs class thing in the name.  There's a similar problem in the name fire_class_double w/ modules, right?  For example, I could imagine using it like so enum = fire_class_double("Enumerable") where the given string names a module rather than a class.

Not sure how much that helped, but those are my current thoughts...

Ok, see what you mean. Good call on misnaming of fire_class_double, you're right. Makes it even more awkward: does consistency make it more or less clear? Urgh.
Going to let it sit for 24 hours, see if anyone else chimes in, then will make a call and merge.

Actually, I've thought about it some more, and there's some refactoring I want to do first :).

OK, I pushed the refactoring I had in mind.  I think this is a lot better.  It fixes a couple more bugs, too.  I like that it makes stub_const it's own thing that can be used alone if that's all you want.
I'm still not sure what API I like.  My refactoring supports both fire_replaced_class_double("Something") and fire_class_double("Something").as_replaced_constant.  Let me know what you think about picking one or the other or keeping both or going with something else entirely.  I don't really have a strong opinion.


Don't push empty paths in LD_LIBRARY_PATH

Fixes #4277.  Note: I haven't written a test for this fix yet.

r? @matklad
(rust_highfive has picked a reviewer for you, use r? to override)

Feel free to grab & modify my test from #4277

Updated with @jmesmon's regression test (thanks!) and an additional test that checks the env var directly.

@bors: r+
Thanks!

:pushpin: Commit 0588424 has been approved by alexcrichton

:hourglass: Testing commit 05884243cbd53d9421fc01fa22c67eead735f360 with merge f709c35a35687b2665eb290695825375be91410b...

:sunny: Test successful - status-appveyor, status-travis
Approved by: alexcrichton
Pushing f709c35a35687b2665eb290695825375be91410b to master...


mtex: add support for sub/super script

support for sub/super script would be nice.
we need to implement the "TeX box model":

https://github.com/matplotlib/matplotlib/blob/master/lib/matplotlib/_mathtext.py#L923

and more specifically SubSuperCluster and subsuper:

https://github.com/matplotlib/matplotlib/blob/master/lib/matplotlib/_mathtext.py#L1634
https://github.com/matplotlib/matplotlib/blob/master/lib/matplotlib/_mathtext.py#L2568

on the Go side, the mtex.Visitor would need to be adapted to handle the sub/super nodes:

https://github.com/go-latex/latex/blob/main/mtex/parser.go#L72

and the tex package that implements the "TeX box model" would need to be adapted as well to handle something along the lines of SubSuperCluster and subsuper:

https://github.com/go-latex/latex/blob/main/tex/box.go

thanks for the offer :)
I've amended the issue with some blurb and pointers.
hth.

Is this problem solved?

sorry, I didn't manage to carve out time for this.
(also, I am working on a "better" way to address this: https://star-tex.org)


http://scratch.mit.edu/projects/52567602/

http://scratch.mit.edu/projects/52567602/
http://phosphorus.herokuapp.com/#52567602
Mozilla/5.0 (Windows NT 6.1) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/41.0.2272.89 Safari/537.36
it wont load the rest of the game so i cant play it plz fix

An asset is namespaced which doesn't work for some reason: http://cdn.assets.scratch.mit.edu/internalapi/asset/c4b47c3fd621d4eb3e50e0eacec4fae5.svg/get/

Like https://github.com/nathan/phosphorus/issues/288; I have a fix in the works.


[Bug]: missing variable export in Terraform EKS deployment instruction

Installation method
Own AWS account
What happened?
When terraform apply command is ran the following error occurs:
Plan: 35 to add, 0 to change, 0 to destroy.
╷
│ Error: "name" length must be between 1-100 characters: ""
│
│   with module.eks.aws_eks_cluster.this[0],
│   on .terraform/modules/eks/main.tf line 28, in resource "aws_eks_cluster" "this":
│   28:   name                      = var.cluster_name
│
╵
╷
│ Error: "name" doesn't comply with restrictions ("^[0-9A-Za-z][0-9A-Za-z_-]+$"): ""
│
│   with module.eks.aws_eks_cluster.this[0],
│   on .terraform/modules/eks/main.tf line 28, in resource "aws_eks_cluster" "this":
│   28:   name                      = var.cluster_name
│
╵
What did you expect to happen?
terraform apply should successfully complete after deployment of EKS cluster and the underpinning infrastructure such as VPC and NAT Gateway IGW etc...
How can we reproduce it?
Follow the Terraform instruction set
Anything else we need to know?
Missing the environment variable EKS_CLUSTER_NAME in the Cloud9 IDE shell.
Running the following command prior terraform apply should fix it:
export EKS_CLUSTER_NAME=eks-workshop
EKS version
1.27

I would love to contribute to this. can someone assign this issue to me ?

This was fixed a while back and I forgot to close the issue. Appreciate the original report!


Add support for custom GKE scopes in cluster/gke/util.sh

I find it curious that this doesn't already exist. I assume there is a reason for that but I couldn't find any references for it. What is the reason?
If there is hope of adding custom gke scopes, I'd be happy to submit a PR, but I'd like some feedback first. I noticed notes for @mbforbes like "Refactor setting scope flags." What refactor was in mind? Currently scopes are set in the MINION_SCOPES variable which is a bash array. Because it's an array it needs a join_csv function. But as far as I can tell, the variable is never "pushed" or "popped" so I see no benefit from using the array. Although if we eventually want to push e.g. monitoring, logging scopes onto the array, we should keep it. Alternatively it might be easy to just concat comma-separated strings.
I don't mean to bike shed here, I'd just like to do the simplest PR that will be accepted and enable custom scopes in gke.

When you create a cluster through the GKE UI, at https://pantheon.corp.google.com/project/$YOURPROJECT/kubernetes/add you can click "More" near the bottom, and then you get a list of scope to select for your VMs.
Does that do what you need?

Did you try changing the value of MINION_SCOPES in config-default.sh?

@erictune our cluster rollout is scripted for automated setup, by the same token I could just modify cluster/gke/util.sh myself (but that would require maintaining a fork - not so terrible, but I imagine other people would want to set custom scopes as well).
@roberthbailey Ideally I'd like to avoid maintaining a separate branch with my configuration. Much better to set configuration variables and then run the standard script. Also, the current kube-up() function in util.sh does not use the MINION_SCOPES variable when creating the cluster.

@chrishiestand -- you are correct. It doesn't look like the gke config-default.sh even defines MINION_SCOPES and kube-up doesn't use it when creating the cluster.
This would be a fine feature improvement for the gke cluster creation scripts and we would certainly accept a patch that plumbs through the minion scopes (and allows you to override the default ones via an environment variable).
We are currently in feature freeze for the 1.0 launch, but I believe this meets the criteria for inclusion since it brings GKE up to par with the GCE cluster creation scripts.

Thanks @roberthbailey! If anyone, or @mbforbes in particular, has a strong preference as to the style of plumbing please let me know, I'm happy to help with the refactor. Otherwise I'll maintain the current style.

@chrishiestand no preference here, though feel free to include me in the PR. I wrote that TODO because I had been blindly copying those variables around when adding new paths (e.g. cluster/gce/upgrade.sh from cluster/gce/util.sh) and didn't like the code duplication.

As discussed on the PR, this will land post 1.0.

Thanks for your responsiveness and help, everyone.


Update limitations.md

Update Stability limitations

I think we should wait until typescript releases the changes in a stable version.

Also see https://github.com/prisma/prisma2/pull/852 which removes the limitation, what is what should happen if the merged change is active and released.


Fixed #28878 -- pip should install 1.11 on Python2

https://code.djangoproject.com/ticket/28878

Added the "python_requires" key to setup(), so pip knows not to try installing on incompatible Pythons.
Added a more useful error for users with an old version of pip, which does not know about python_requires.

Note that this patch is against Django 2.1 - when backported, python_requires should instead be set to >=3.4 so Django 2.0 can be installed on that version.
Having this or a similar patch in a released version will give Python2 users with old pip a useful error on pip install django, and that's the best we can do for them.  However, for upgrading pip to be useful, we also need to set the python_requires metadata on Django 2.0.0 - probably a job for @timgraham?

I've improved the message, and made it applicable to older versions of Python 3.  Thanks for all the reviews!

How do I "set the"python_requires metadata on Django 2.0"? I looked around in the PyPI web interface but didn't see anything.

I guess I have some doubts about this approach because it doesn't seem like a good thing if projects that go Python 3 only have to add this boilerplate.
Is your thinking that we can remove the warning once pip 9+ (released July 2016) is sufficiently widespread?  It seems to me it may be fragile to maintain (e.g. setup.py must remain compatible with Python 2.7).

Well, we could just add the python_requires classifier and say that everyone should keep their pip up-to-date, but right now that would cause problems for many people, mostly newbies using their default Python and without much clue about Python packaging.
The fragility aspect is real, but I think not too bad - as long as setup.py is syntatically valid on Py2 (so no async def ...), we just do the check using sys before doing anything else.

How do I "set the"python_requires metadata on Django 2.0"? I looked around in the PyPI web interface but didn't see anything.

Honestly no idea; I'd contact @dstufft and ask him!

I found the following information: http://www.python3statement.org/practicalities/
Maybe this is something that Django could sign up to if it agrees?
See http://www.python3statement.org/ for the full details.

It's a great site - I linked it from the ticket above, and it directly inspired this pull 😄
python3statement/python3statement.github.io#21 (and 23) indicates that they'd love to have Django on board - unless anyone objects, I'll add Django to the list and timeline.

How do I "set the"python_requires metadata on Django 2.0"? I looked around in the PyPI web interface but didn't see anything.

You can't, unless you gently ask the PyPI maintainer to issue a sql query. It usually is set at upload time from the sdist/wheel metadata. And without setting it for already released Django versions it won't be of much help until you drop another minor version of Python.

The fragility aspect is real, but I think not too bad - as long as setup.py is syntactically valid on Py2 (so no async def ...), we just do the check using sys before doing anything else.

You do not need to keep setup.py Python 2 compatible, it will just fail with a cryptic error message (Invalid Syntax) at install time on Py2 if you don't, this is IMHO less of a problem for Django (where most users are developers and will understand why if they see the message), than in the scientific community when our users may not have a single clue as of why.
Still would ❤ to have you on the Py3statement.

Hi @timgraham - any remaining reservations or things I can do to improve this?

I think the main todo is to figure out if it's possible to update the metadata of the Django 2.0 package, otherwise this change won't have an effect, correct?

I think the main todo is to figure out if it's possible to update the metadata of the Django 2.0 package, otherwise this change won't have an effect, correct?

Correct. From a recent discussion on distutils-sig, there is no ability to change metadata for package maintainers and no plan to add such a capability.
The only possibility seem to gently ask the maintainers of pypi/warehouse for now until a better  solutions is found.

I think the main todo is to figure out if it's possible to update the metadata of the Django 2.0 package, otherwise this change won't have an effect, correct?

@Carreau is correct that you'll need to get in touch with the PyPI/Warehouse maintainers - published packages are immutable for everyone else.
Even without that though, this change will have some effect - pip install django would still fail on Py2, but pip install django~=2.0.1 would give the correct error.  I think enough users of Django are installing from requirements files or otherwise using version ranges to make this worthwhile even if 2.0.0 is never fixed.  Merging this would also demonstrate that we won't let the problem reoccur, which might be important for Warehouse people.


More on FileStorage interface etc.

Dear Danilo,
please find a patch attached to add the readObj and writeObj methods to cv::FileStorage. Now it is more general
and allows to (de)serialize most of the supported objects/types. Some small other cosmetic changes are included, too.
All the best,
Christian
tcl-opencv-20210923.diff.gz

Thanks for your effort.


Ian Campbell: Watch gardreen.com

Ian Campbell requests the watch of the watch_keyword gardreen\.com. See the MS search here and the Stack Exchange search in text, in URLs, and in code.

gardreen\.com has been seen in 2 true positives, 0 false positives, and 0 NAAs.

Approved by Spevacus in Charcoal HQ


Modify .jsp license comments to not pass through to HTML

It's not strictly wrong to include a license header, but it's unnecessary and weird.

It turns out the trouble here is just that error.jsp contained two headers.


I would like to become a maintainer of this repo

The repo seems abandoned since the past 3 years. The project is amazing and very useful.
People have been avoiding this repo since it's not being maintained.

Hi @marsonya, We can fork and maintain according to our own needs and how can we improve.

Hi @marsonya, We can fork and maintain according to our own needs and how can we improve.

That sounds like a good idea to me.

Lets connect wherever you are okay.

This project has so much potential, I think a lot of developers would greatly benefit from it.
Any update on forking & maintaining this repo? @aggmoulik @marsonya

@onury , I would like to help maintain this repo.

@onury , I would like to help maintain this repo.

Amazing! Many people would really appreciate this.
If @onury does not reply I encourage you to fork the repo and share a link here :)

https://github.com/sellerartifact/accesscontrol  hi, We plan to maintain this repo and have merged several unmerged PRs


Lack of the default/fallback resources in config.xml for Android triggers MissingDefaultResource issue

When generating images for Android the cordova-res tool also registers the generated images in config.xml however it registers them for portrait and landscape orientation only. When creating a production build (using ionic cordova build android --release --prod) for android, android lint checks errors will be thrown because no defaults (In this case, without port- or land- ) as a fallback is provided. The errors are as below
/Users/alexryltsov/Git/BOSS811/master/platforms/android/app/src/main/res/drawable-port-xxhdpi/screen.png: Error: The drawable "screen" in drawable-port-xxhdpi has no declaration in the base drawable folder or in a drawable-densitydpi folder; this can lead to crashes when the drawable is queried in a configuration that does not match this qualifier [MissingDefaultResource]
/Users/alexryltsov/Git/BOSS811/master/platforms/android/app/src/main/res/drawable-port-xxxhdpi/screen.png: Error: The drawable "screen" in drawable-port-xxxhdpi has no declaration in the base drawable folder or in a drawable-densitydpi folder; this can lead to crashes when the drawable is queried in a configuration that does not match this qualifier [MissingDefaultResource]

   Explanation for issues of type "MissingDefaultResource":
   If a resource is only defined in folders with qualifiers like -land or -en,
   and there is no default declaration in the base folder (layout or values
   etc), then the app will crash if that resource is accessed on a device
   where the device is in a configuration missing the given qualifier.

   As a special case, drawables do not have to be specified in the base
   folder; if there is a match in a density folder (such as drawable-mdpi)
   that image will be used and scaled. Note however that if you  only specify
   a drawable in a folder like drawable-en-hdpi, the app will crash in
   non-English locales.

   There may be scenarios where you have a resource, such as a -fr drawable,
   which is only referenced from some other resource with the same qualifiers
   (such as a -fr style), which itself has safe fallbacks. However, this still
   makes it possible for somebody to accidentally reference the drawable and
   crash, so it is safer to create a default dummy fallback in the base
   folder. Alternatively, you can suppress the issue by adding
   tools:ignore="MissingDefaultResource" on the element.

   (This scenario frequently happens with string translations, where you might
   delete code and the corresponding resources, but forget to delete a
   translation. There is a dedicated issue id for that scenario, with the id
   ExtraTranslation.)

Not sure if this tool should add these default/fallback or provide details in README how to fix these errors. Right now, the user gets the errors during the build after images are generated/registered in config.xml
To fix the errors I manually added the below
        <splash density="ldpi" src="resources/android/splash/drawable-land-ldpi-screen.png" />
        <splash density="mdpi" src="resources/android/splash/drawable-land-mdpi-screen.png" />
        <splash density="hdpi" src="resources/android/splash/drawable-land-hdpi-screen.png" />
        <splash density="xhdpi" src="resources/android/splash/drawable-land-xhdpi-screen.png" />
        <splash density="xxhdpi" src="resources/android/splash/drawable-land-xxhdpi-screen.png" />
        <splash density="xxxhdpi" src="resources/android/splash/drawable-land-xxxhdpi-screen.png" />

Please also see discussion here https://github.com/apache/cordova-plugin-splashscreen/pull/228 and https://github.com/apache/cordova-android/issues/689 and https://github.com/apache/cordova-plugin-splashscreen/issues/185

Thanks for opening an issue for this! For stuff like this we're dependent on Cordova. We don't want to generate undocumented config that may work today that Cordova then changes later. It looks like neither the splashscreen plugin or cordova-android maintainers have an official response yet.
I'm subscribed to those two issues and the PR and will wait to see what happens.

@pieter-janDB I had the same issue. First, to get rid of the errors, see the solution given by @ryaa. Then, to fix the white screen, here is the solution that worked for me:
npm i cordova-plugin-ionic-webview@latest

These errors are not raised if we build an .aab package, instead of .apk.

Looks like a Cordova specific issue. As we're dropping Cordova support in 1.0 I am going to close unless this also impacts Capacitor. Thanks for the issue.

I am seeing this same error in my Capacitor project (which was upgraded from Cordova).
npx capacitor-assets generate --android creates splash screen images (see list below), but lint fails because there is no default resource.
Seems like we need either android/app/src/main/res/drawable/splash.png -or- android/app/src/main/res/drawable-somedpi/splash.png (without the "-land" or "-port" modifier) to make the lint task happy.
Unless there's another way around this, or I have something wonky because it's an upgraded Cordova app maybe?
android/app/src/main/res/drawable-port-xxxhdpi/splash.png
android/app/src/main/res/drawable-land-xhdpi/splash.png
android/app/src/main/res/drawable-port-ldpi/splash.png
android/app/src/main/res/drawable-port-hdpi/splash.png
android/app/src/main/res/drawable-port-mdpi/splash.png
android/app/src/main/res/drawable-land-mdpi/splash.png
android/app/src/main/res/drawable-land-hdpi/splash.png
android/app/src/main/res/drawable-land-ldpi/splash.png
android/app/src/main/res/drawable-land-xxhdpi/splash.png
android/app/src/main/res/drawable-port-xxhdpi/splash.png
android/app/src/main/res/drawable-port-xhdpi/splash.png
android/app/src/main/res/drawable-land-xxxhdpi/splash.png


Compatibility with Akka.Persistence.MySql

We do not have a configuration or a compatibility spec to work with Akka.Persistence.MySql.
In theory, a user would be able to manually configure the tables with delete-compatibility-mode, but we should provide easy configuration and ideally provide unit tests.
We should make sure such integration/implementation uses a MySql connector and Implementation packages that are non-encumbering (i.e. we should steer users away from adopting the Oracle MySqlConnector, but rather an MIT or Apache based one, consider MariaDB or similar instead of vanilla MySql)

Added in the data compatibility PR #117


Using with a base monad other that IO

I'm interested in using effectful with a base monad other than IO - in particular PropertyT IO from the hedgehog library.
Is this possible?

This isn't possible, but PropertyT is quite a strange monad that might not even possible to capture as an effect with effectful. What I have had success with is defining a Test effect that models Hedgehog's TestT monad, along with evalEff. This lets you combine effectful and MonadTest, which has felt like the goth balance. If this is interesting let me know, as I've been meaning to tidy this up and open source it

Yes, please.


Update pubsub code examples

Hi all,
There are some changes in the recent API updates. I've updated the code examples to make it easier for others to conform to the changes. The current code examples are compatible with pubsub 0.40.0-beta.
Cheers, Fokko

I signed it!

Thanks @pongad for reviewing. I've revised the commit with two spaces. Cheers


MicroUriSerializer check has incorrect logic

When a Micro URI is validated, it assumes that the entity.id() must be greater than 0.
https://github.com/eclipse-uprotocol/up-cpp/blob/494b3313b55aaefe5b8b0376dd761b75fa96a78a/include/up-cpp/uri/serializer/MicroUriSerializer.h#L193C1-L195C6
However, this is not the case. The core uSubscription uService has an id == 0:
// Subscription Service Interface definition
service uSubscription {
  option (name) = "core.usubscription"; // Service name
  option (version_major) = 3;
  option (version_minor) = 0;
  option (id) = 0;

This bug causes any URI for the uSubscription service to fail to serialize.

Hi @ALPayne18 --
This does look to be a legit bug.
I can find nowhere in the Micro UUri spec mention of restricting the UEntity id to be non-zero.
Further review of up-rust and up-java also shows no such restriction in-place.
@gregmedd -- how'd we like to handle this?

The spec explicitly calls out an ID of 0 as being for uSubscription, so I agree it needs to be fixed. Is @debruce or @billpittman available to grab this issue and put out a PR with a fix?

This has been resolved with #104. Do we need to tag a release, or do we have other bugfixes?

FWIW, this has been merged and theoretically fixed, but I can't assign it to myself, nor close it.  However it should be at least closed.