Peer Review

Hi @Althrun-sun,
I can only review your app by cloning the code and test it on my local.
First of all, the file path to load the data is not correct and it works after I updated to 'pd.read_csv("../data/car_price.csv")'.
Once the app loaded, a few improvement can be implemented.

Filters should be placed in the section which user are easier to check, for example on top or on the left.
Filters could be more responsible to update the plot, without clicking on the apply filter button
The pie chart is not suitable to display the market share for each company, and this plot seems to no respond to the change in filters.

Overall, interesting app to play with!!!

Congratulations for the app!!

It is a very nice-looking app with a proper theme and color.
The documents (Readme, Proposal, Contributing, Code of Conduct, License) are well prepared.
One thing I may suggest for the visualization is that it will be better if the pie_chart is limited with some number of car manufacturers since it is difficult to see when we plot all manufacturers.
Also the line chart may not properly work yet. I chose Fuel and 918 Spyder and I did not see the line chart result.

It is a nice and meaningful visualization for people to explore the used vehicle market.
There are two things that might be considered for further improvement:

Add a overall title for the app to give users a clear idea about what the app is doing if they do not have chance to see your repo.
The  pie chart seems not effective to distinguish so many car manufacturers by colours. Maybe a simple bar chart would be more effective if we want to see a ranking of market share of car manufacturers.


Titles are duplicated when extending role assignments

Reproduce:

Extend a role assignment
Check title

Expected behaviour:
Title should be "Strategic Anchor Council"
Experienced behaviour:
Title is duplicated: "Strategic Anchor Council: Strategic Anchor Council"
e.g. https://dao.hypha.earth/hypha/proposals/37101
e.g. https://dao.hypha.earth/hypha/proposals/

Hello @leonieherma1! It seems to me that this is not a bug, I checked the current list of proposals and could not find something similar. Also, there is no symbol in the code: in the title, only what was entered when creating the proposal is transmitted there. I think this was put into the title when it was created.

Hm I will check the issue again when I extend my assignment next week on dev

@leonieherma1 I dug into the code and found a place where this happens, and I don't know if it's a bug or a feature :) . In general, this is not a duplicate of the same title, but a little different. The first part before ":" is the name of the original proposal, the one that is being extended, and after the ":" is the title of the current proposal. This is only about the type Extension, for other types it is not. I can remove it if necessary, I don't really know who did it or why, but it clearly made some sense.

@evgenibir let's just keep the original title and remove "title of current proposal". In the proposal creation wizard the title should not be editable.
Makes sense?

@evgenibir let's just keep the original title and remove "title of current proposal". In the proposal creation wizard the title should not be editable. Makes sense?

Hi @leonieherma1, sure, i`ll do it


Same reference entry in OPF  ignored.

From<EMAIL_ADDRESS>on August 21, 2013 12:10:29
PROBLEM: I am not sure if this is a problem at all, or if this can be ignored.  Couldn't find anything mentioned in the standard.  The "problem" occurred for me when I generated a EPUB using a script of mine with a bug, and that resulted in generating a OPF guide element with the same reference entry multiple times:
  <guide>
    <reference type="cover" title="Cover Page" href="cover.html" />
    <reference type="start" title="Start Page" href="cover.html" />
    <reference type="start" title="Start Page" href="cover.html" />
    <reference type="start" title="Start Page" href="cover.html" />
    <reference type="start" title="Start Page" href="cover.html" />
  </guide>

There was no warning that this had occurred.
METHOD: What steps will reproduce the problem?

Attached EPUB contains a document which validates using epubcheck 3.0.1
Unpack it, take a look at OEBPS/content.opf
Behold multiple entries.

RESULT: Nothing.
EXPECTED: A warning would have made sense, but I am not sure if one would be justified.
PLATFORM: Latest epubcheck 3.0.1 on Mac.
java version "1.6.0_51"
Java(TM) SE Runtime Environment (build 1.6.0_51-b11-456-10M4508)
Java HotSpot(TM) Client VM (build 20.51-b01-456, mixed mode)

ADDITIONAL INFO: Attached is a test EPUB.
Attachment: test.epub
Original issue: http://code.google.com/p/epubcheck/issues/detail?id=299

Not sure if it helps, but duplicate references to the same document/fragment was made an error in 3.1 for the landmarks nav. An info message for 2.0 seems appropriate.

Thanks for the review, Matt! Adding the "missing" and "epub-3.1" labels.
/cc @jenstroeger (original author)

Thanks @tofi86 for adding me! 👍

So guide and landmark duplicated are already checked since #738 (by @tofi86).
For EPUB 3.2, we just need to report landmark duplicates as ERROR instead of WARNING.


Error Reporting Page

An error reporting page accessible from the main menu would be nice. Possibly use OctoKit to enable users to submit errors directly to the GitHub page? Or maybe a layer inbetween would be nice so that user junk (Things that aren't errors) don't get to the GitHub page.

I thought about your idea and have a different idea about what you can possibly do. What if it instead had a form and used MailKit on the Client GUI portion have the ability to send feedback with optional fields for name and email address. As long as handling possible trolls is something you can manage; then everything should be fine.

Both ways might possibly require passwords however.

Personally I like the first option better. I'll explain why:
By using Octokit we can put user errors/exceptions directly into the GitHub repository. This gives visibility of the issues to everyone who works on this project. If we were to use some sort of email solution, only people who have access to the email account could see these. Furthermore that would also require people to check a separate location for errors.
Security isn't critical for the Octokit route as some clever use of authentication services like OAuth as well as a new separate GitHub account for this purpose with limited permissions would leave things fairly secure. I guess the same could be said for the email route, but I favor the increased visibility and convenience of having the issues directly within this repository.


Could not find com.github.jonz94:TedImagePicker:<IP_ADDRESS>

Hi,
I have ionic-react app where using this plugin , but when creating android build getting these errors, although done all the changes mentioned in the documentation:
execution failed for task ':app:dataBindingMergeDependencyArtifactsDebug'.

Could not resolve all files for configuration ':app:debugRuntimeClasspath'.
Could not find com.github.jonz94:TedImagePicker:<IP_ADDRESS>.
Searched in the following locations:
- https://dl.google.com/dl/android/maven2/com/github/jonz94/TedImagePicker/<IP_ADDRESS>/TedImagePicker-<IP_ADDRESS>.pom
- https://repo.maven.apache.org/maven2/com/github/jonz94/TedImagePicker/<IP_ADDRESS>/TedImagePicker-<IP_ADDRESS>.pom
- file:/Users/unthinkable-mac/Desktop/ionic8-app/android/capacitor-cordova-android-plugins/src/main/libs/TedImagePicker-<IP_ADDRESS>.jar
- file:/Users/unthinkable-mac/Desktop/ionic8-app/android/capacitor-cordova-android-plugins/src/main/libs/TedImagePicker.jar
- file:/Users/unthinkable-mac/Desktop/ionic8-app/android/app/libs/TedImagePicker-<IP_ADDRESS>.jar
- file:/Users/unthinkable-mac/Desktop/ionic8-app/android/app/libs/TedImagePicker.jar
Required by:
project :app > project :jonz94-capacitor-image-picker

seems like these libraries has been removed, as when opening these links:
https://dl.google.com/dl/android/maven2/com/github/jonz94/TedImagePicker/<IP_ADDRESS>/TedImagePicker-<IP_ADDRESS>.pom,
https://repo.maven.apache.org/maven2/com/github/jonz94/TedImagePicker/<IP_ADDRESS>/TedImagePicker-<IP_ADDRESS>.pom
getting error that file is not found, can anyone help,
additional information:
Ionic Framework : @ionic/react 8.3.0
@capacitor/android : 6.1.2
@capacitor/core    : 6.1.2
@capacitor/ios     : 6.1.2
NodeJS : v18.19.1
npm    : 10.2.4

Hi, this plugin for Android is published on JitPack (https://jitpack.io/#jonz94/TedImagePicker)
Therefore, you will need to setup JitPack repository for your android project.
See: https://github.com/jonz94/capacitor-image-picker#android

After that, the issue should be resolved.

@jonz94 Yes, issue has been resolve , I had added this:  maven { url "https://jitpack.io" } , under  buildscript { repositories { }} , but in allprojects {  repositories { }}
But I thing we also need to do is that compileSdkVersion and targetSdkVersion should be 34


access dialogues via handler

Proposed changes
Make dialogues accessible via their handler
Fixes
If it fixes a bug or resolves a feature request, be sure to link to that issue.
Types of changes
What types of changes does your code introduce to agents-aea?
Put an x in the boxes that apply

[ ] Bugfix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)

Checklist
Put an x in the boxes that apply.

[ ] I have read the CONTRIBUTING doc
[ ] I am making a pull request against the develop branch (left side). Also you should start your branch off our develop.
[ ] Lint and unit tests pass locally with my changes and CI passes too
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] I have checked that code coverage does not decrease.
[ ] I have added necessary documentation (if appropriate)
[ ] Any dependent changes have been merged and published in downstream modules

Further comments
If this is a relatively large or complex change, kick off the discussion by explaining why you chose the solution you did and what alternatives you considered, etc...

Currently failing test: tests/test_skills/test_base.py::TestSkillProgrammatic::test_protocol_dialogues

Codecov Report

Merging #151 (7864c71) into main (c2e697b) will decrease coverage by 0.18%.
The diff coverage is 60.00%.

@@            Coverage Diff             @@
##             main     #151      +/-   ##
==========================================
- Coverage   94.90%   94.71%   -0.19%
==========================================
  Files         309      311       +2
  Lines       24816    24894      +78
==========================================
+ Hits        23552    23579      +27
- Misses       1264     1315      +51

Flag
Coverage Δ

unittests
94.71% <60.00%> (-0.19%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

aea/skills/base.py
99.11% <20.00%> (-0.89%)
:arrow_down:

aea/configurations/constants.py
100.00% <100.00%> (ø)

aea/configurations/data_types.py
99.70% <100.00%> (+<0.01%)
:arrow_up:

aea/helpers/io.py
54.28% <0.00%> (-45.72%)
:arrow_down:

aea/cli/registry/fetch.py
96.07% <0.00%> (-3.93%)
:arrow_down:

aea/cli/utils/package_utils.py
88.62% <0.00%> (-3.68%)
:arrow_down:

aea/cli/run.py
85.50% <0.00%> (-3.63%)
:arrow_down:

aea/cli/utils/context.py
96.70% <0.00%> (-2.20%)
:arrow_down:

aea/cli/core.py
100.00% <0.00%> (ø)

... and 7 more

Continue to review full report at Codecov.

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


Map common toplevel import names

Intent
This PR adds a fallback import -> package name mapping, to cover cases where the package has a non-matching import name and also isn't installed locally.
Fixes #2113
Type of Change

[] Bug Fix

[x] New Feature

[ ] Breaking Change

[ ] Documentation

[ ] Refactor

[ ] Tooling

Approach
Define a default mapping of common packages that have mismatched import names. Unfortunately, this information isn't available by querying PyPI (see discussion).
Automated Tests
Updated an existing test to test this case.
Directions for Reviewers
Create an app.py that imports a package with a mismatched name that you don't have installed locally yet, for example
import yaml
import dotenv

Scan for requirements. The resulting requirements.txt should contain
pyyaml
python-dotenv

Checklist

[ ] I have updated CHANGELOG.md to cover notable changes.

I wonder if a hard coded list like this is what we should do here. The list itself is fragile and susceptible to being not complete. For example ibis / ibis-framework isn't on there, it's almost certainly the case that people publishing to connect are using ibis-framework but in principle it's possible that someone is using ibis where the rewrite would be very wrong. We also don't have insight into internal package names that are used within orginizations.
For the case where we have the package installed so can inspect the library, we're good. What if instead of rewritting these packages we warn in some way that there might be issues and that installing the libraries and rerunning will provide a better experience?

I agree that the list is likely to be fragile and incomplete; that's why I didn't use this approach originally. I like the suggestion of warning the user and giving them an opportunity to either correct the file or set things up better for a rescan (and for local testing). We have the same workflow when creating configuration files, where we are providing a solid starting point but can't guarantee everything is 100%.


[messaging] TypeError: Value is not an object: undefined

[REQUIRED] Step 2: Describe your environment

Operating System version: MongoDB Atlas Trigger (Realm App)
Firebase SDK version: 11.4.1
Firebase Product: messaging
JS SDK Version: 3.18.0
NPM version: I don't know

[REQUIRED] Step 3: Describe the problem
Executing admin.messaging().sendMulticast(message) returns an error and is not working anymore after upgrading to firebase-admin: 11.4.1 from 11.0.1 and with the release of fastify/busboy 1.2.0 :
uncaught promise rejection: TypeError: Value is not an object: undefined

The difference with my previous issue #1902 is that now messaging().sendMulticat(message) is crashing and no longer deliver the push notification.
Steps to reproduce:

Create a MongoDB Atlas trigger
Add firebase-admin as external dependency and make sure it's 11.4.1
Add the service-account.json as a value
Run the code with the below code

Relevant Code:
exports = async function() {

  const admin = require("firebase-admin");
  const json_creds = context.values.get("service_account_value_name");

    const title = "Test";
    const body = "This is a test";

    const message= {
      android: {
        notification: {
          title: title,
          sound: "default",
          body: body,
        },
      },
      apns: {
        payload: {
          aps: {
            alert: {
              title: title,
              body: body,
            },
            mutableContent: 1,
            contentAvailable: 1
          }
        },
      },
      tokens: ["device_token_1", "device_token_2"]
    };

    admin.initializeApp({
        credential: admin.credential.cert(JSON.parse(json_creds))
      });

    admin.messaging().sendMulticast(message);

    console.log("Messages sent !");
  }
}

I have rolled back to 11.0.1 to see the difference and i am getting the following error :
TypeError: Arguments must be Buffers
	at compare (<eval>:344:13(19))
	at node_modules/@fastify/busboy/deps/streamsearch/sbmh.js:181:65(304)
	at node_modules/@fastify/busboy/deps/streamsearch/sbmh.js:82:24(41)
	at node_modules/@fastify/busboy/deps/dicer/lib/Dicer.js:107:23(92)
	at T (<eval>:10:22717(45))
	at <eval>:10:25821(108)
	at <eval>:10:25847(113)
	at d (<eval>:10:14414(11))
	at <eval>:10:4454(115)
	at <eval>:10:13336(285)
	at C (<eval>:10:11803(20))
	at A (<eval>:10:11681(31))

But the push notification is sent successfully, in 11.4.1 it's not sent at all.

I'm seeing the same issue, where updating from 11.4.0 to 11.4.1 is causing this crash.  It appears that this change has pushed 'undefined' as a value into a the JWT verify flow, which might match the error.  I will try to get my function dependencies to refresh, as it might be a stale package (jsonwebtoken) causing the failure.

No change in behavior after adding jsonwebtoken 9.0.0 as a dependency directly.

Same issue here...

Sorry i don't know node JS that much, i only use a MongoDB Atlas Trigger to see this behavior... :(

No worries, are you able to share a complete error log so we can track down where it is throwing the error?

@lahirumaramba the errors posted above are all that are given when the error is generated by the MongoDB function, apologies.
If it helps, here are a couple extra details:
payload:
{"notification":{"body":"Your video is ready!","title":"New Video"},"tokens":[<my-device-token>]}
call:
const admin = require('firebase-admin');
admin.messaging().sendMulticast(message)

No worries, are you able to share a complete error log so we can track down where it is throwing the error?

Sorry for the late answer, there is no more logs than this from the Atlas trigger...
However, i think it can be related to any changes made between the working version and this one that is not working at all.

Has there been any progress on this? I'm still running into this issue on version 11.7.0

I have the same issue, are there any news about this topic? Actually using version 11.9.0.

11.10.1, also using MongoDB Atlas functions, same error :/

Did anyone every resolve this? Using version 11.11.0 in MongoDB Atlas functions and still getting same error...

I downgraded to v10.0.0 and it works well.

I can't believe that this has still not been fixed😟. (now using 12.0.0)

Still no update on this?
I get the same error when using 12.0.0. Had to downgrade to 10.0.0 for it to work.

Same issue here, 10.0.0 is working but 12 is not.

im using the last version 12.1.1 and i have the same issue, how can i downgrade the version of sdk to fix this?

I faced the error with version all 10.0.0 and 11v and 12v
I solved by using the following approach.
I used axios and google-auth-library
import axios from "axios";
  const { JWT } = require("google-auth-library");
const json_creds = JSON.parse(context.values.get("service_account_value_name"));

 const getAccessToken = async () => {
    try {
      const client = new JWT({
        email: json_creds.client_email,
        key: json_creds.private_key,
        scopes: ["https://www.googleapis.com/auth/firebase.messaging"],
      });
      const accessToken = await client.authorize();
      return accessToken.access_token;
    } catch (err) {
     throw new Error(err.message)
    }
  };

const accessToken = await getAccessToken()

const notification = {
  message: {
    notification: {
    title: "hello",
    body: "this is a new remainder",
    },
  token: "your_device_token"
  },
};

await axios.post(`https://fcm.googleapis.com/v1/projects/your_projectid/messages:send`, message, {
  headers: {
    Authorization: `Bearer ${accessToken}`,
    "Content-Type": "application/json",
  },
});

*note:- you cannot send multicast or sendAll.


fixed items such as crit not triggering

Thanks, I've been trying to do this for 20 minutes, but as it's my first time using GitHub, I had no idea what I was doing.

Verified working?

Yes. I've been using this fix for over 30 minutes.

personally, yep.

Thanks a lot, merging.

Looks like you bumped the version without merging first.


Add a lint rule not to allow the 'require' function to load module.

To avoid confusing situation, it's better to restrict using the 'require' function. Let's add a new lint rule for it and then only use |import| instead of |require|.

Okay.

when I try to find it with eslint config, it seems there is no rules for this restriction.

@daehyunjimmy is right, there's no existing rule for disallowing require, so I will create a custom rule for it.

We can close this issue?

I'm closing this issue. That feature is already applied in absolute 1.0.


feat:  introduces an option to disable scaling

This PR introduces an option to disable scaling, which provides the benefit of significantly faster comparisons, albeit at the cost of image quality.
However, when applying a mask:

With scaling enabled, mask values in pixels are double the size of the corresponding PDF points.
With scaling disabled, mask values in pixels match the PDF points exactly.

We could consider adding a new option to define masks in PDF points. In that case, should the masking be based on the PDF origin (0,0), which is located at the bottom-left corner?
TODO:

[ ] Add option to define masks in PDF points
[ ] Improve API documentation

As per your comment about the origin point - if the current behaviour is to define the mask in pixels (which just happens to have a 1:1 relationship to points based on the upscaling or not) and the mask is indeed applied after converting to a PNG, perhaps a low-touch approach is to:

keep the top-left origin point
clarify API docs to state that mask coordinates and dimensions are given in pixels
update the conversion part of the canvas to adhere to the scaling (if needed)

Not entirely sure but these lines may give you trouble? I haven't used canvas however so am guessing: https://github.com/moshensky/pdf-visual-diff/blob/feat/expose-png-scaling-option/src/pdf2png.ts#L83
Cheers moit!


Update Program.cs

change method name from camelCase to PascalCase
changes related to review comments in https://github.com/dotnet/docs/pull/27279

Summary
Describe your changes here.
Fixes #Issue_Number (if available)

Merged with #27279, which makes the corresponding updates to the article text.


Modifying a file while running debug does not properly update

VSCode Version: 0.10.12-alpha
OS Version: Windows 10

Steps to Reproduce:

With a basic Node.js project set a breakpoint anywhere in index.js before you set the port.
I cloned this https://github.com/heroku/node-js-sample.git and have a breakpoint set on line 1 of index.js
Start debugging
After hitting the break point, change the port from 5000 to say 8000
Continue debugging

Actual:
Output prints that the app is running on port 5000, not the updated 8000 port.

node.js does not detect and reload modified source automatically. You will have to run a tool like 'nodemon' to get this behaviour.


is:issue is:open [swagger-zod] readOnly: true generates with .readonly when they are not equivalent

What version of kubb is running?
kubb/2.7.1 darwin-arm64 node-v18.19.1
What platform is your computer?
MacOS
What version of external packages are you using(@tanstack-query, MSW, React, Vue, ...)
nothing related
What steps can reproduce the bug?
I believe the current behaviour is intended. When a schema has readOnly: true, the zod schemas generate with .readonly().
How often does this bug happen?
Every time
What is the expected behavior?
readonly: true only means that the property is not included in write requests (as implemented in https://github.com/kubb-project/kubb/pull/495/files). It should not apply .readonly() - it makes no comment on the mutability of the object in the application.
Swagger/OpenAPI file?
swagger: "2.0"

info:
  title: Some API
  version: 2.0.0

paths:
  /thing:
    get:
      operationId: getThing
      parameters: []
      responses:
        200:
          description: Some response
          schema:
            type: object
            properties:
              foo:
                type: number
                readOnly: true

which generates:
import { z } from 'zod';

/**
 * @description Some response
 */
export const getThing200Schema = z.object({
  foo: z.number().readonly().optional(),
});

/**
 * @description Some response
 */
export const getThingQueryResponseSchema = z.object({
  foo: z.number().readonly().optional(),
});

Additional information
No response

Fixed with Kubb v2.8.1

Thank you!


docs: Fix enumerations in doc pages

Feature or Bugfix

Bugfix

Detail

Fix enumerations in doc pages

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

AWS CodeBuild CI Report

CodeBuild project: GitHubCodeBuild8756EF16-4rfo0GHQ0u9a
Commit ID: 2009ab206454a6ac0a4cac0586182f0ca6c5b21b
Result: SUCCEEDED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

AWS CodeBuild CI Report

CodeBuild project: GitHubDistributedCodeBuild6-jWcl5DLmvupS
Commit ID: 2009ab206454a6ac0a4cac0586182f0ca6c5b21b
Result: SUCCEEDED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository


[Question] Bot prints wrong output.

System Information (Required)

SDK Pletform: .NET
SDK Version: <IP_ADDRESS>
Development Environment: localhost Emulator
Production Environment: Azure skype

Issue Description
Bot Framework prints wrong output in await context.PostAsync() method.
Steps to Reproduce
When we try to print this below string
"1. Power adapter 2. Desktop Computer  3. Telephone  \r\n \r\n4. Splitter 5. Telephone wall jack 6. Laptop Computer  \r\n \r\na. MODEM interface b. PHONE interface c. LINE interface  \r\n \r\n1.1.2 Getting started  \r\n \r\nAfter connecting the cables, press the On/Off button on the side panel to  \r\n \r\npower on the HG533.  \r\n \r\nThe ADSL indicator lights up and starts blinking. This indicates that the  \r\n \r\nADSL port activation is in progress. The blinking may last less than 3  \r\n \r\nminutes. When the ADSL indicator stops blinking and remains on, the  \r\n \r\nHG533 is working properly.  \r\n \r\n"
it prints different output as below.
Actual Results

Power adapter 2. Desktop Computer 3. Telephone
2. Splitter 5. Telephone wall jack 6. Laptop Computer
a. MODEM interface b. PHONE interface c. LINE interface
1.1.2 Getting started
After connecting the cables, press the On/Off button on the side panel to
power on the HG533.
The ADSL indicator lights up and starts blinking. This indicates that the
ADSL port activation is in progress. The blinking may last less than 3
minutes. When the ADSL indicator stops blinking and remains on, the
HG533 is working properly.

Expected Behavior
Here the string should be printed as follow.

Power adapter 2. Desktop Computer 3. Telephone
4. Splitter 5. Telephone wall jack 6. Laptop Computer
a. MODEM interface b. PHONE interface c. LINE interface
1.1.2 Getting started
After connecting the cables, press the On/Off button on the side panel to
power on the HG533.
The ADSL indicator lights up and starts blinking. This indicates that the
ADSL port activation is in progress. The blinking may last less than 3
minutes. When the ADSL indicator stops blinking and remains on, the
HG533 is working properly.

Here, the original text 4.Splitter is always changed to 2.Splitter (tested in emulator and skype). I don't understand why this is happening.

Hi, have you found a solution to this issue yet?  Have you tried on a more recent sdk like 3.8.1?

Yes, I have tried by updating the framework to latest stable 3.8.1. But the result is same
Sample Code:
private async Task MessageReceivedAsync(IDialogContext context, IAwaitable<object> result)
        {
            var activity = await result as Activity;

            string mystr = "1. Power adapter 2. Desktop Computer 3. Telephone \r\n \r\n4. Splitter 5. Telephone wall jack 6. Laptop Computer \r\n \r\na. MODEM interface b. PHONE interface c. LINE interface \r\n \r\n1.1.2 Getting started \r\n \r\nAfter connecting the cables, press the On/Off button on the side panel to \r\n \r\npower on the HG533. \r\n \r\nThe ADSL indicator lights up and starts blinking. This indicates that the \r\n \r\nADSL port activation is in progress. The blinking may last less than 3 \r\n \r\nminutes. When the ADSL indicator stops blinking and remains on, the \r\n \r\nHG533 is working properly. \r\n \r\n";
            await context.PostAsync(mystr);

        }

Output:

@sw-ms-roshanparmar it looks like it is trying to make an ordered list for you.
can you try this and see if it helps or changes anything
            string s = @"1. Power adapter 2. Desktop Computer 3. Telephone"+ "\r\n<EMAIL_ADDRESS>5.Telephone wall jack 6.Laptop Computer"+"\r\n \r\n"+"a.MODEM interface b. PHONE interface c. LINE interface \r\n \r\n1.1.2 Getting started \r\n \r\nAfter connecting the cables, press the On/Off button on the side panel to \r\n \r\npower on the HG533. \r\n \r\nThe ADSL indicator lights up and starts blinking. This indicates that the \r\n \r\nADSL port activation is in progress.The blinking may last less than 3 \r\n \r\nminutes.When the ADSL indicator stops blinking and remains on, the \r\n \r\nHG533 is working properly. \r\n \r\n";

yes @JasonSowers , it has started working now. But could not understand where I was making a mistake


All builds skipped?

Seems this was rendered so that nothing built. This is pretty strange as there is no skip. Going to try re-rendering locally to see if that fixes it. @pelson, any thoughts as to why this happened? 😕

Re-rendering it locally made no difference.

Seems these Python selectors are confusing it. If I remove them, it works fine. Would be nice if this wasn't an issue.

Tried changing this selector to # [py<=27] to see if the conda-smithy issue went away and still had the same issue.

Temporary fix in PR ( https://github.com/conda-forge/setoptconf-feedstock/pull/2 ). Though this will return soon.

Now permanent fix in PR ( https://github.com/conda-forge/setoptconf-feedstock/pull/2 ). Still comparison selectors need some revisiting.


add codeberg.org/FiskFan1999/hugo-theme-ffcactus

Hi @FiskFan1999.
Thanks for opening this pull request. The Netlify preview seems to have failed.
As per https://github.com/gohugoio/hugoThemesSiteBuilder#adding-a-theme-to-the-list :

Note that if the PR preview does not come up as expected after fixing your theme (missing thumbnail image etc.), you can trigger a new preview build by amending the commit on your PR branch and doing a force push.

We are closing this pull request for now. Feel free to submit another one if you manage to fix the issues.
We will be glad to review it and add your theme.


Minor fixes for SVMLight and LibSvm record readers and writers

What changes were proposed in this pull request?
(1) added support for records with no labels, either fully unlabeled data or multilabel with no positive labels
(2) for features and sparse multilabels, now pre-allocate List with Collections.numCopies() and call set(index, value) to update features, rather than using ad hoc while loops.
Added new unit tests or updated existing unit tests to cover various "no label" scenarios. All unit tests (including legacy tests) pass.
How was this patch tested?
(Please explain how this patch was tested. E.g. unit tests, integration tests, manual tests)
Please review
https://github.com/deeplearning4j/deeplearning4j/blob/master/CONTRIBUTING.md before opening a pull request.

TL;DR previously, support for records with no labels (either fully unlabeled data or multilabel records with no positive labels) was incomplete. Now we cover most common scenarios, and we're now consistent with scikit-learn's functionality and behavior.


Add newline after message in MigrateProgress

Fixes couple of missing new line in console messages in migration.

I have OCD 😶🌫

Finally! It breaks the terminal prompt slightly, which is mildly annoying indeed. It's not just OCD. 👍🏼👍🏼


SCP-4883 ΔQ enhancements for marlowe-scaling client

The following parameters were added to marlowe-scaling:

delay before building transaction
delay after transaction is confirmed
jittering of delays
retry with backoff

Pre-submit checklist:

Branch

[ ] Tests are provided (if possible)
[ ] Test report is updated (if relevant)
[x] Commit sequence broadly makes sense
[x] Key commits have useful messages
[x] Relevant tickets are mentioned in commit messages
[x] Formatting, materialized Nix files, PNG optimization, etc. are updated

PR

[x] Self-reviewed the diff
[x] Useful pull request description
[x] Reviewer requested

Hi @bwbush can we review this PR together, maybe in our testing meeting tomorrow? I also would like to run it on my machine and I'm having trouble configuring my local environment to run this.


Make a powershell toolbox

I'd like to see if people would dig a dedicated powershell terminal, so if you've got Windows friends that can help, that'd be awesome!
For implementation please use the ghcr.io/ublue-os/wolfi-base container and add powershell to its package list.
Not sure what the rest of the Containerfile looks like though.

You could probably expand the scope to common Microsoft tooling folks would use.
A couple suggestions would be Azure CLI and .Net Core.

Hello, I would like to work on this task. I have a preliminary pull request ready for the initial review. I am happy to add new packages if required but for now I have added PowerShell, dotnet core and Azure CLI.
Furthermore, I am not sure how people tend to use these toolboxes but if we assume that no shell customizations are in place (inherited from home directory) we could add some blink to the shell. Things like [Oh My Posh] (https://ohmyposh.dev/) posh-git and PSFzf are popular amongst PowerShell users. For even more have a look at this repo.
Feel free to ping me if any additions or changes are required!

Thanks for submitting the pull request, I'll merge it when it builds! Since no one's really claimed it, then feel free to provide whatever UX you feel people would like!
Check out the bluefin-cli toolbox for conventions on where we're putting the scripts, etc. This will be an awesome addition to the collection, thanks for working on this!

I'll set up the CI!


Armor tab uses jappa-style icon on programmer art

Fixed in version 1.6.4, thanks for reporting.


Add devations to metadata proto

Add devations to metadata proto
Also updating Makefile and proto lint workflow to include the ondatra testbed.proto dependency

As discussed offline, I am stating with just adding one deviation to the proto at a time.

two bash nits

done. Also updated PlatformExceptions to be a repeated field


Green light bulb feels out of style

Why is the light bulb green and not white like most things in the status bar?
Looks out of style IMHO
ping @bgashler1

It needs to get attention and white just doesn't work.

I think we should tackle all these non-white colors in the status bar in the future. There must be something which looks nice and draws attention as well.
Assigning to backlog and @bgashler1

We're thinking of doing some notification work in the future (i.e. a unified area for notifications).  This will address many other related issues with notifications.
While we wait to get that in the product, I can provide a temporary solution for this.

When considering the future direction, remember that differentiating green and red is one of the biggest color blindness problems.


naming-convention rule bug - single letter field name

Describe the bug
I have a single letter field name, it is lowercase which I consider valid snake case.
To Reproduce
Steps to reproduce the behavior:
"@graphql-eslint/naming-convention": [
          "error",
          {
            "FieldDefinition": "snake_case",
            "InputValueDefinition": "snake_case",
            "VariableDefinition": "snake_case"
          }
        ]

type Point {
  x: String
  y: String
}

Expected behavior
It should not be an error, as this otherwise is valid snake case
Environment:

OS: mac
@graphql-eslint/...: 3.8.0
NodeJS: 16

Additional context
If I make the name 2 characters long the rule considers it okay.

Hi @joshribakoff-sm, thank you for your report, saw this bug also for UPPER_CASE style, this will be fixed on the next version of graphql-eslint 🚀. Added related tests also

Thanks!!

fixed in<EMAIL_ADDRESS>


Maybe two bugs in global planner and the questions about planner

@Jaeyoung-Lim I am back again hah~
The bugs reported：

In cell.cpp line 70, the fucntion ceil may should use "floor", because when the radius is 3, the z-pos is wrong due to square operation on a negative number.
The radius should be equal "robot_radius/cell_scale", rather than "robot_radius/resolution".

The question about global planner:

In avoidance, when I set a target point in rviz, it will judge if there is free. If there is free, the drone will fly to it, otherwise, the drone will hover. But I think in the actual flight, the octomap may not be always accurate for judging a location whether is free or not.
When the target position is far away from the drone, it should fly closed to the target position, if there is not free, it should hover at the closest position to the target position.
Global planner will stop working when it plans some times without paths results, that is, the goal is free, but the drone is hover. Is there a problem with the parameter settings?

Thanks for help!
Joey

@JoeyLeeNPU Sorry for the late reply.

I don't quite get what you mean. Could you clarify on what you mean?
I believe the cell scale is always 1 and is not really used.

Global Planner

You are correct. And it will be only makred as occupied if it has "seen" it occupied. So if the target is far away it won't try to predict but rather always mark it as free as it has never been observed. It is a optimistic planner in that sense
I agree that the logic is not great. It will simply hover if it couldn't find a path. There is no path exploring logic embedded into global planner and that is why it will simply stay there if it couldn't find a path. I am planning to implement this kind of exploratory behavior, maybe something similar to the NBVP planner but maybe on the OMPL implementation

Sorry for too late response, my first question is that in  /global planner/cell.cpp line 70, the function "ceil" may should use "floor", if use "ceil" the z-pos will be calculated as a wrong number because of a square root of a negative number.
And now I use the local planner, I met a crash that when I publish a goal quickly in two times, the drone will be fly straightly in a high speed. I found that in /local planner/src/nodes/local_planner.cpp line 124, when the previous_goal is equal to goal, the normalization of them may be a NAN.

@JoeyLeeNPU I just found that bug as well in a different setting. If you do RTL twice in a row you also end up with prev_goal = goal.
I fixed it here: https://github.com/PX4/avoidance/pull/541

@baumanta thanks for fixing this bug!


[Removal]: Long Scarlet Buffalo

Hotspot Name
Long Scarlet Buffalo
Hotspot b58 Address
112Rn1KgPuvwArVpfnS1uFqewVX2iNfJRNTn3a8tvRH9pm3LmGqL
Discord Handle
ken340@1015
Hotspot Manufacturer
Nebra
Removal Reason
As per my previous request, i mentioned i will be moving the miner to another location. The Asserted address is corrrect
Modifications
No
Extra forwarders
No
Extra antennas
No
Additional Information
I am a legitimate user and I have to move it to my in-law place and I have asserted the proper location. the location on the map in the actual location. The Denylist is wrong,

.
Please remove me.

picture of my installation. it is in a high-rise apartment. you can check on googlemap.

Further to that I also have occasional issues Miner RPC unavailable with my Nebra. I have Nebra support ticket no#334197 (outage on 19-22 Feb) and #334197 (outage on 6-9 Feb 2022). if you require I can supply you with the email with Nebra support.

We have taken into account this removal request in the next iteration of the denylist. It may take some time before it's processed. This is an automated comment due to the volume of addition and removal requests. A future tool is in the works to provide more insight into the analysis approach made by the Helium team and community members that maintain this list.


[Adapter] Guidance on building a private mixer adapter

Is your feature request related to a problem? Please describe.
I would like to use a create a private custom template and adapter and use with Mixer. From my understanding of the dev guide, the way to build an adapter that would be private is to

Fork istio to a private repository
Add new template and adapter to the package
Define them in istio/mixer/template/inventory.yaml and istio/mixer/adapter/inventory.yaml

Private repository creates overhead every time istio has to be updated.
Describe the solution you'd like
Is there a way to do this without forking istio to a private repository? Is there a better approach to doing this that I am not considering?

@adityats2020 There is a much better way. The approach involves using the newly-supported gRPC adapters. There is work right now on adding the appropriate dev guide.

That sounds much better indeed!
Has there already been an adapter developed using gRPC I can take a look while the dev guide is being worked on?
Can you also link the issue tracking the work?


Integrate RSocket javadocs into website

A canonical source of RSocket javadocs will be helpful for answering support questions and for links from other, task-oriented documentation pages.

Thoughts:

This should be extended to other implementation (e.g. python, javascript)
Each language usually has it's own source to documentation complier
The look and feel of these compliers are very visually different
Simply placing the output of these compilers would make the site look and feel inconsistent

I don't think this documentation should be embedded in the site. Maybe just a consistent method of linking to external documentation ? In the python guide I intend to simply link to rsocket.readthedocs.io when I want to reference API documentation.
Thoughts ?


RUF001 should include ambiguous unicode characters in comments

Right now, the RustPython tokenizer doesn't include comments (unlike the CPython tokenizer), which is blocking this check.

https://github.com/RustPython/RustPython/pull/4266


Return event listener index

now return a array on event listener index.

@cristyansv  this looks good, but can I ask why the index is wrapped into an array when returned?

@techfort I thought doing so for consistency
coll.on(['insert', 'update', 'delete'], callback);

// return int[], returns each index of each event

coll.on('insert', callback);

// return int[], return the index of an event.

I thought it would be more reasonable to always return the same data type.

hey @cristyansv i have to think more about this. It definitely represents a breaking change for anybody who's expecting an integer and gets an array.
Upon review I might add this to the 2.0 branch.

hi @techfort

It definitely represents a breaking change for anybody who's expecting an integer and gets an array.

this is the code prior to my first commit
LokiEventEmitter.prototype.on = function (eventName, listener) {
      var event,
          self;

      if (Array.isArray(eventName)) {
        eventName.forEach(function(currentEventName) {
          self.on(currentEventName, listen);
        });
      }

      event = this.events[eventName];
      if (!event) {
        event = this.events[eventName] = [];
      }
      event.push(listener);
      return listener;
    };

Currently it returns a callback, not an integer or the index of the event.

@cristyansv ok i got confused there. If you want to amend the PR to return the callback so it doesn't represent a breaking change i'll be happy to merge.


"Unable to load .NET runtime, specifically hostfxr." when launching.

Godot version
4.0.1
System information
Linux Mint 21.1 Vera base: Ubuntu 22.04 jammy
Issue description
When launching Godot 4.0.1 .NET an error screen is shown:

dotnet is installed, included in the path:

Steps to reproduce
Download https://github.com/godotengine/godot/releases/download/4.0.1-stable/Godot_v4.0.1-stable_mono_linux_x86_64.zip
Run on the Linux system with installed dotnet
Minimal reproduction project
None

Just tried on my Pop_OS installation on the same machine with dotnet installed  from the apt - works as intended.

Interestingly only the blog posts for pre-releases actually document this, but dotnet needs to be installed in a standard location for the install to be detected. For linux that means that the dotnet executable must be found at the path /usr/share/dotnet. IIRC it should be possible to symlink your install from that path, but I don't recall what exactly has to be linked where for things to actually work correctly.

I ran into the same issue even when installing .NET SDK 6.0 (dotnet-sdk-6.0) from Microsoft's package repository into the standard location /usr/share/dotnet/ on Ubuntu 22.04.
Installing .NET SDK 7.0 (dotnet-sdk-7.0) instead solved the issue for me. Perhaps it's some kind of compatibility issue?

it was doing the same for me
But i used the Debian 11 guide here
https://learn.microsoft.com/en-us/dotnet/core/install/linux-debian
which helped for me

Try this:
wget https://dot.net/v1/dotnet-install.sh -O dotnet-install.sh
chmod 777 dotnet-install.sh
./dotnet-install.sh --channel 7.0

export PATH="$PATH:$HOME/.dotnet"

Use microsoft script to install dotnet. It works fine.

Try this:
wget https://dot.net/v1/dotnet-install.sh -O dotnet-install.sh
chmod 777 dotnet-install.sh
./dotnet-install.sh --channel 7.0

export PATH="$PATH:$HOME/.dotnet"

Use microsoft script to install dotnet. It works fine.

I installed everything according to Microsoft's instructions, got the error in the original post here. This was the only thing that got it to actually work for me - thanks :)

Following this fixed it for me: https://stackoverflow.com/questions/73753672/a-fatal-error-occurred-the-folder-usr-share-dotnet-host-fxr-does-not-exist

Try this:
wget https://dot.net/v1/dotnet-install.sh -O dotnet-install.sh
chmod 777 dotnet-install.sh
./dotnet-install.sh --channel 7.0

export PATH="$PATH:$HOME/.dotnet"

Use microsoft script to install dotnet. It works fine.

Hey sorry if this bumps the issue but I found while this did get rid of the fatal error that crashes Godot on loading a file, I was still getting a Red error saying
ERROR: The host fxr folder does not exist: /usr/share/dotnet/host/fxr

To fix this, you just have to set the DOTNET_ROOT environment variable so that Godot can find the fxr file.
After updating your path to include dotnet, you can simply run the following to automatically find the right directory and point DOTNET_ROOT to it.
export DOTNET_ROOT="$(dirname $(which dotnet))"

I'm getting this issue too on macOS (M1 Mac). I installed .NET runtime without changing any paths or any other settings, just clicking "Next" until finished. So whatever location Godot expects is not the one where it's installed by default.

I had same issue, my dotnet was installed in my home directory. Just simlinked /usr/share/dotnet -> /home/fadi/.dotnet  and Goddot.NET works fine now

I had this problem also, this article helped me: https://bobbyhadz.com/blog/fatal-error-ocurred-the-folder-usr-share-dotnet-host-fx-does-not-exist

Try this:
wget https://dot.net/v1/dotnet-install.sh -O dotnet-install.sh
chmod 777 dotnet-install.sh
./dotnet-install.sh --channel 7.0

export PATH="$PATH:$HOME/.dotnet"

Use microsoft script to install dotnet. It works fine.

Hey sorry if this bumps the issue but I found while this did get rid of the fatal error that crashes Godot on loading a file, I was still getting a Red error saying
ERROR: The host fxr folder does not exist: /usr/share/dotnet/host/fxr

To fix this, you just have to set the DOTNET_ROOT environment variable so that Godot can find the fxr file.
After updating your path to include dotnet, you can simply run the following to automatically find the right directory and point DOTNET_ROOT to it.
export DOTNET_ROOT="$(dirname $(which dotnet))"

Also ran into this issue, but I fixed it with
sudo ln -sT ~/.dotnet /usr/share/dotnet
Note that if you installed .NET through the package manager as well there will already be files here and this won't work

Same on arch even after running these commands:

wget https://dot.net/v1/dotnet-install.sh -O dotnet-install.sh

chmod +x ./dotnet-install.sh

./dotnet-install.sh --version latest

./dotnet-install.sh --version latest --runtime aspnetcore

./dotnet-install.sh --channel 8.0

From here https://learn.microsoft.com/en-us/dotnet/core/install/linux-scripted-manual#scripted-install

@Schweeeeeeeeeeeeeeee Scripted installs are not recommended for setting up a development environment, their intended purpose is to use it for CI scenarios. The recommended installation method is to use your distro's package manager.

@raulsntos Thx that worked :) ill keep that in mind

Try this:
wget https://dot.net/v1/dotnet-install.sh -O dotnet-install.sh
chmod 777 dotnet-install.sh
./dotnet-install.sh --channel 7.0

export PATH="$PATH:$HOME/.dotnet"

Use microsoft script to install dotnet. It works fine.

This fixed it for my, trying to install Godot_v4.2.1 on Kubuntu 23.10 x86_6

Same Issue on windows CMD "dotnet --info" works just fine for me.
Godot Version 4.2.2
dotnet 6.0.413

add an environment variable named "DOTNET_ROOT", setting the value with your path of dotnet like "E:\Program Files.net SDK" or "/home/user_name/dotnet"(depends on your OS platform), to let Godot recongnize your customize dotnet path

MacOS m3 Sequoia 15.1 (my on the go tool).
I followed the fix suggested here which suggests using .NET 6.0, but I'm using .NET 8 in my dev environment outside of Godot (e.g. C# scripts in VSCode etc...). I'm concerned about compatibility - has anyone figured out how to get it running with .NET 8 (or 9 even) ? Managing 2 different versions of .NET could become unwieldy.


python package index (PyPi) migration breaks ansible-test?

When running ansible-test sanity --list-tests command from ansible documentation, the docker instance fails to init due to PyPi erroring out because it can't install ansible into the docker instance(?):
vagrant@vagrant-acs:~$ sudo ansible-test sanity --list-tests
Sending build context to Docker daemon  6.405MB
Step 1/8 : FROM debian:7.7
 ---> 6d129b37ec3b
Step 2/8 : MAINTAINER Rob McQueen
 ---> Using cache
 ---> 2497f1a1838e
Step 3/8 : RUN apt-get update && apt-get install -y   python-dev   python-virtualenv   sudo
 ---> Using cache
 ---> 73436b884880
Step 4/8 : RUN mkdir /opt/ansible && virtualenv /opt/ansible/venv && /opt/ansible/venv/bin/pip install ansible
 ---> Running in 7c8aa8fbe804
New python executable in /opt/ansible/venv/bin/python
Installing distribute.............................................................................................................................................................................................done.
Installing pip...............done.
Downloading/unpacking ansible
  Cannot fetch index base URL http://pypi.python.org/simple/
  Could not find any downloads that satisfy the requirement ansible
No distributions at all found for ansible
Storing complete log in /root/.pip/pip.log
The command '/bin/sh -c mkdir /opt/ansible && virtualenv /opt/ansible/venv &&   /opt/ansible/venv/bin/pip install ansible' returned a non-zero code: 1
Traceback (most recent call last):
  File "/usr/local/bin/ansible-test", line 69, in <module>
    main(context)
  File "/usr/local/bin/ansible-test", line 45, in main
    '-t', DOCKER_CONTAINER, '.']
  File "/usr/lib/python2.7/subprocess.py", line 541, in check_call
    raise CalledProcessError(retcode, cmd)
subprocess.CalledProcessError: Command '['docker', 'build', '-t', 'testbox', '.']' returned non-zero exit status 1

This is likely caused by the recent PyPi migration of trusted host site (see below)
Stackoverflow link

Note: Sometime during April 2018, the Python Package Index was migrated from pypi.python.org to pypi.org. This means "trusted-host" commands using the old domain no longer work.

@wuy3 this is already fixed in this version. The standard image is now debian:stable and not debian:7.7 as it is in the version you get from pip. So theroretically this isnt a issue anymore HOWEVER Guys pls update you pip package ansible-test. It is not working anymore !

Confirmed installing directly from GitHub works. Please update pip package.

Thanks guys, sorry for the delayed close

@wuy3 the command mentioned in the Ansible documentation is something that is a part of ansible-core and has nothing to do with this specific package having the same name.


Provide all info for the initial PaleoDEM and more

In order to have a good traceability of who did what and on a very valuable suggestion of Anta Sarr, we should add

When uploading the initial paleDEM, request from the user to state clearly the authors of the paleogeography, paleomag or hotspot reference framework, some may also think to add main hypothesis did on, in particular, configuration of marine gateways and of main orogens
After modifications of the paleoDEMS for ORCA2 grid purpose, clearly state the choices that have been done (closed or enlarged gateways) for the paleobathy file. Same should hold for pfts spatial distrib of ORCHIDEE.
Also add which config of PALEORCA grid has been used (in particular where are located the northern hemisphere singular points)

@tiphe-cerege @AntaCl could you please confirm the points below

When uploading the initial paleDEM, request from the user to state clearly the authors of the paleogeography, paleomag or hotspot reference framework, some may also think to add main hypothesis did on, in particular, configuration of marine gateways and of main orogens

For this we do this when first uploading the file? And add :

Author / reference (required)
Hypothesis (not required)
Age???

After modifications of the paleoDEMS for ORCA2 grid purpose, clearly state the choices that have been done (closed or enlarged gateways) for the paleobathy file. Same should hold for pfts spatial distrib of ORCHIDEE.

On internal oceans, passage problems (sub basins ??, complex viewer ??), we should add a pop up to enter the info? → Just a text entry?

Also add which config of PALEORCA grid has been used (in particular where are located the northern hemisphere singular points)

Not sure How this is going to work? Where would we do this? in the "routing" stage?


Add new block to Sir trevor

Can someone please list all the steps to add a custom block to ST.
I want to add an ordered list so have added this code to the ST initialiazition -
   /*
     Ordered List
   */

   SirTrevor.Blocks.OrderedList = (function() {

     var template = '

';

     return SirTrevor.Block.extend({

       type: "ordered_list",

       icon_name: 'list',

       editorHTML: function() {
         return _.template(template, this);
       },

       loadData: function(data){
         this.getTextBlock().html("
" + SirTrevor.toHTML(data.text, this.type) + "
");
       },

       onBlockRender: function() {
         this.checkForList = _.bind(this.checkForList, this);
         this.getTextBlock().on('click keyup', this.checkForList);
       },

       checkForList: function() {
         if (this.$('ol').length === 0) {
           document.execCommand("insertOrderedList", false, false);
         }
       },

       toMarkdown: function(markdown) {
         return markdown.replace(/<\/li>/mg,"\n")
                        .replace(/<\/?[^>]+(>|$)/g, "")
                        .replace(/^(.+)$/mg," 1. $1");
       },

       toHTML: function(html) {
         html = html.replace(/^ 1. (.+)$/mg,"
$1
")
                    .replace(/\n/mg, "");

         return html;
       },

       onContentPasted: function(event, target) {
         var replace = this.pastedMarkdownToHTML(target[0].innerHTML),
             list = this.$('ol').html(replace);

         this.getTextBlock().caretToEnd();
       }

     });

   })();

This has been copied from https://github.com/madebymany/sir-trevor-blocks

Bump @raffij

Have initialized ST like below -
 var stInstance  = new SirTrevor.Editor( {
        el: $('.js-st-instance'),
        defaultType: "Image",
//        required: [
//            "Heading",
//            "Text"
//          ],
         onEditorRender : function() {
              this.block_manager.createBlock("OrderedList");
            },
           blockTypes: [
                       "Text",
                       "Heading",
                       "Image",
                       "List",
                       "Quote",
                       "Columns",
                       "ImageExtended",
                       "Video",
                       "OrderedList"
                     ],
        onFormSubmit: function(){
            alert("form submitted");
        }
    });

      $('.submit').on('click', function(e) {
        e.preventDefault();

        // Skip validaton for now, we're just saving as we're going
        SirTrevor.SKIP_VALIDATION = true;

        // Store the current set of cards
        SirTrevor.onBeforeSubmit();

        // Grab the data
        var json = SirTrevor.getInstance().store.retrieve();

        // Turn validation back on
        SirTrevor.SKIP_VALIDATION = false;

//        $('.output').html(JSON.stringify(json, null, ' '));
        $('.output').text($('.js-st-instance').val());
      });

What version of sir-trevor are you using?

@raffij  I am using 5.0

Bump @raffij

@rnjai @rnjailamba did you add it to blocks/index.js?

@timmyg
No i havent.Can you please share some sample code that i have to add besides the code i have shared above
Thanks!!

@rnjailamba Something like below (if your new block file is called orderedlist.js). Then, in your blockTypes when initing SirTrevor, refer to ordered list as Orderedlist (or, you can call it whatever you would like in the below module.exports
// blocks/index.js
module.exports = {
  Text: require('./text'),
  Quote: require('./quote'),
  Image: require('./image'),
  Heading: require('./heading'),
  Orderedlist: require('./orderedlist'),
  List: require('./list'),
  Tweet: require('./tweet'),
  Video: require('./video'),
};

@timmyg i have done this.
How do i add my own icon for the new block -
http://postimg.org/image/5rc1a6wbr/

@rnjailamba see the following properties in your orderedlist.js (or look at text.js for an example)
// this refers to the symbol id in icons/sir-trevor-icons.svg. I am not sure the best practice for adding your own though. (You could always do it via your own css as a background-image)
icon_name: 'text'
// this is where the title comes from and refers to the matching variable in locales.js
title: function() { return i18n.t('blocks:text:title'); },
// so you need to add what the title of your block is in locales.js (add this under where it says heading...
      orderedlist: {
        'title': "Ordered List"
      }

Ok My issue is resolved and i am able to add new block and am able to show the new icons after clicking the "plus".

Before - http://postimg.org/image/5rc1a6wbr/
After  - http://postimg.org/image/8xxaqw2rz/
My issue was resolved by following a file structure similar to that of [this](https://github.com/Andreyco/sir-trevor-js-blocks) repo.
I was earlier dumping all code - initialization and code for new blocks in one file.


Support an odd number of digits in hexadecimal strings (issue 18645)

See https://opensource.adobe.com/dc-acrobat-sdk-docs/pdfstandards/PDF32000_2008.pdf#G6.1840792

/botio test

/botio makeref


[docs] CDC document update #963

Documentation will be more convenient for beginners.

I noticed that the snapshot version has been explained in the quick start doc. Do I need to keep this note?


send ESC for default amount of time

Sending ESC for 0 amount of time seems to break things that read the keyboard directly (like Alfred).
Leaving as the default seems to work better and matches the  code in  the 'keyboard' project.

Thanks, @docwhat!


Design initial domain model for executing quantum jobs

Context
Execution of quantum jobs via different quantum cloud services encompasses many distinct domain entities that need to be represented as Python classes, e.g., different Job types, Job Manager, Execution components, Users and tokens for accessing different cloud services, etc.
Problem/Idea
Set up an initial domain model representing those entities
Solution
Expected Outcome: A set of documented Python classes representing the required domain entities, an initial UML class diagram representing the relations among the entities.
See the RADICAL-Pilot documentation as an example.
Estimated Story Points 23-05-10: 8

Wir (Dev-Team-1) hat schonmal eine Skizze und Notizen in Confluence erstellt:
https://enpro-qunicorn.atlassian.net/wiki/spaces/Quincorn/pages/1900545/Domain+Model

alte Story Points 10

Opened PR: https://github.com/SeQuenC-Consortium/qunicorn-core/pull/37

Review:

Could generalize job state running
possibly different Class Name for Quantum_Program


Translation is lost in insider build if you don't manually set it in locale.json

Issue Type
Bug
Description
See title, and my system language is zh-CN.
 I also tried it on some other PCs on a Windows stable build, still the same.
Found an interesting thing though. After you delete your user profiles in %APPDATA%, on the first launch, extensions that have Chinese localization will be shown in Chinese while everything else is in English. But if you restart Code, all strings are in English.

If I set German as system language, VS Code starts in German, too. Weird.

@octref @rebornix can you help me with this and test VS Code insider under a Chinese locale. What is strange is the @Joelism sees this only under zh-CN and not under de for example.

@dbaeumer Confirmed. If you set zh-CN in locale.json VS Code will be loaded in Chinese, but it's in English when I first launch it from Windows with CHinese locale. #

OK. I found out what it is. Chrome / Electron reports Chinese locales as zh-CN / zh-TW and I tested for lower case. It worked for German since it only reports de.


Fix iterable testing to add actual members to our list of members; otherwise all sorts of stuff breaks

This change is

@jgraham You're going to need to pull this into web platform tests to get them to work at all right.  For example, right now http://w3c-test.org/url/interfaces.html just dies partway through with exceptions because there is crud in the members array that's not actually members...

/cc @youennf @Ms2ger

Reviewed 1 of 1 files at r1.
Review status: all files reviewed at latest revision, all discussions resolved.

Comments from Reviewable


The slider "counter" is not equal to images number

Expected behavior:
The slider "counter" is not equal to images number.
Actual behavior:
The slider "counter" is not equal to images number. One "indicator" is missing, see attached images.
Reproduce steps:
Create a slider with 4 images.
Version:
v3.0.4-beta.23.2
Additional Information:

Also when cliking on the "indicator" the next images is displayed and NOT the clicked one.

@sampoyigi i can't see where the JS from this comes from. Is it a bug in the JS plugin?

Maybe is there... https://github.com/tastyigniter/ti-ext-frontend/blob/8681dac148c2103781c1af3406875af09df28a78/components/slider/default.php#L9

Yep play around with it and open a PR if you get it fixed

This a temp weird fix: slides()) + 1; ?>

This line
<?php for ($index = 1; $index <= $countThumbs; $index++) { ?>
should be
<?php for ($index = 1; $index < $countThumbs; $index++) { ?>

Again, if you are able to open a PR it would be helpful

Any luck opening a PR @BreakSecurity ?

@ryanmitchell I am having some trouble setting up a developer friendly ambient under windows with xampp giving random errors...
git clone https://github.com/tastyigniter/TastyIgniter.git
cd TastyIgniter

# Set up a Composer path repository for TastyIgniter packages
composer config repositories.0 path "packages/*"
git clone https://github.com/tastyigniter/flame.git packages/flame
git clone https://github.com/tastyigniter/ti-ext-frontend.git packages/frontend

git clone https://github.com/tastyigniter/ti-ext-coupons.git packages/coupons
git clone https://github.com/tastyigniter/ti-ext-cart.git packages/cart
git clone https://github.com/tastyigniter/ti-ext-payregister.git packages/payregister
git clone https://github.com/tastyigniter/ti-ext-reservation.git packages/reservation
git clone https://github.com/tastyigniter/ti-ext-local.git packages/local

And so on, but then I try to run tasty and it would not found flame or other extension...
composer create-project tastyigniter/tastyigniter . --stability=beta
This command is better but is not the dev branch so...

I am trying to find a good way to setup an ambient where I can work on last dev with php Storm trying live on xampp and then trough git doing a pull request

I haven’t used windows in years I’m afraid. I’m sure there are lots of articles out there describing how to set it up

I think this has already been fixed during the migration to blade. Can you pull down the latest version from GitHub and verify?

@BreakSecurity can you check the latest version so the issue can be closed?

I will do

Fixed!


Reduce op should not take ranked init values

According to the spec, the variadic init values should be scalar, but tests in infer_stablehlo.mlir, ops_stablehlo.mlir, and print_reduce.mlir have ranked tensors. A verification needs to be added for this.

Thank you for taking the time to document this issue. I think it's a duplicate of #704. Please feel free to reopen if needed.


docs: python django example for login in secure api

Definition of Ready

[ ] I am happy with the code
[ ] Short description of the feature/issue is added in the pr description
[ ] PR is linked to the corresponding user story
[ ] Acceptance criteria are met
[ ] All open todos and follow ups are defined in a new ticket and justified
[ ] Deviations from the acceptance criteria and design are agreed with the PO and documented.
[ ] No debug or dead code
[ ] My code has no repetitions
[ ] Critical parts are tested automatically
[ ] Where possible E2E tests are implemented
[ ] Documentation/examples are up-to-date
[ ] All non-functional requirements are met
[ ] Functionality of the acceptance criteria is checked manually on the dev system.

I followed the web app guide on a empty directory and only copied the devcontainer definitions so I have a compatible python environment. Note that I tried to follow the guide in "stupid mode". I only do what's written there and copy/paste snippets as directed.
Besides the above comments, there are some things that do not work as expected.

Somehow the generated project ends up in ./mysite/mysite as opposed to ./mysite in the example project. It means the the manage.py commands will fail unless I cd mysite first. Is there  a step where we have to manually move the project?
I do not have a requirements.txt. At which point should this be created or generated? What's the content that's supposed to be in the file? IMO that should be mentioned in the guide.
Running manage.py migrate after following all the steps throws an exception. It seems that not all dependencies that are imported (dotenv) are installed using pip in the course of the guide.

$ python manage.py migrate
Traceback (most recent call last):
  File "/home/vscode/.local/lib/python3.12/site-packages/django/core/management/base.py", line 412, in run_from_argv
    self.execute(*args, **cmd_options)
  File "/home/vscode/.local/lib/python3.12/site-packages/django/core/management/base.py", line 458, in execute
    output = self.handle(*args, **options)
             ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/home/vscode/.local/lib/python3.12/site-packages/django/core/management/base.py", line 103, in wrapper
    saved_locale = translation.get_language()
                   ^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/home/vscode/.local/lib/python3.12/site-packages/django/utils/translation/__init__.py", line 210, in get_language
    return _trans.get_language()
           ^^^^^^^^^^^^^^^^^^^
  File "/home/vscode/.local/lib/python3.12/site-packages/django/utils/translation/__init__.py", line 65, in __getattr__
    if settings.USE_I18N:
       ^^^^^^^^^^^^^^^^^
  File "/home/vscode/.local/lib/python3.12/site-packages/django/conf/__init__.py", line 89, in __getattr__
    self._setup(name)
  File "/home/vscode/.local/lib/python3.12/site-packages/django/conf/__init__.py", line 76, in _setup
    self._wrapped = Settings(settings_module)
                    ^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/home/vscode/.local/lib/python3.12/site-packages/django/conf/__init__.py", line 190, in __init__
    mod = importlib.import_module(self.SETTINGS_MODULE)
          ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/local/lib/python3.12/importlib/__init__.py", line 90, in import_module
    return _bootstrap._gcd_import(name[level:], package, level)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "<frozen importlib._bootstrap>", line 1387, in _gcd_import
  File "<frozen importlib._bootstrap>", line 1360, in _find_and_load
  File "<frozen importlib._bootstrap>", line 1331, in _find_and_load_unlocked
  File "<frozen importlib._bootstrap>", line 935, in _load_unlocked
  File "<frozen importlib._bootstrap_external>", line 994, in exec_module
  File "<frozen importlib._bootstrap>", line 488, in _call_with_frames_removed
  File "/workspaces/django-web/mysite/mysite/settings.py", line 132, in <module>
    from dotenv import load_dotenv
ModuleNotFoundError: No module named 'dotenv'

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/workspaces/django-web/mysite/manage.py", line 22, in <module>
    main()
  File "/workspaces/django-web/mysite/manage.py", line 18, in main
    execute_from_command_line(sys.argv)
  File "/home/vscode/.local/lib/python3.12/site-packages/django/core/management/__init__.py", line 442, in execute_from_command_line
    utility.execute()
  File "/home/vscode/.local/lib/python3.12/site-packages/django/core/management/__init__.py", line 436, in execute
    self.fetch_command(subcommand).run_from_argv(self.argv)
  File "/home/vscode/.local/lib/python3.12/site-packages/django/core/management/base.py", line 425, in run_from_argv
    connections.close_all()
  File "/home/vscode/.local/lib/python3.12/site-packages/django/utils/connection.py", line 84, in close_all
    for conn in self.all(initialized_only=True):
                ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/home/vscode/.local/lib/python3.12/site-packages/django/utils/connection.py", line 76, in all
    return [
           ^
  File "/home/vscode/.local/lib/python3.12/site-packages/django/utils/connection.py", line 73, in __iter__
    return iter(self.settings)
                ^^^^^^^^^^^^^
  File "/home/vscode/.local/lib/python3.12/site-packages/django/utils/functional.py", line 47, in __get__
    res = instance.__dict__[self.name] = self.func(instance)
                                         ^^^^^^^^^^^^^^^^^^^
  File "/home/vscode/.local/lib/python3.12/site-packages/django/utils/connection.py", line 45, in settings
    self._settings = self.configure_settings(self._settings)
                     ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/home/vscode/.local/lib/python3.12/site-packages/django/db/utils.py", line 148, in configure_settings
    databases = super().configure_settings(databases)
                ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/home/vscode/.local/lib/python3.12/site-packages/django/utils/connection.py", line 50, in configure_settings
    settings = getattr(django_settings, self.settings_name)
               ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/home/vscode/.local/lib/python3.12/site-packages/django/conf/__init__.py", line 89, in __getattr__
    self._setup(name)
  File "/home/vscode/.local/lib/python3.12/site-packages/django/conf/__init__.py", line 76, in _setup
    self._wrapped = Settings(settings_module)
                    ^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/home/vscode/.local/lib/python3.12/site-packages/django/conf/__init__.py", line 190, in __init__
    mod = importlib.import_module(self.SETTINGS_MODULE)
          ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/local/lib/python3.12/importlib/__init__.py", line 90, in import_module
    return _bootstrap._gcd_import(name[level:], package, level)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "<frozen importlib._bootstrap>", line 1387, in _gcd_import
  File "<frozen importlib._bootstrap>", line 1360, in _find_and_load
  File "<frozen importlib._bootstrap>", line 1331, in _find_and_load_unlocked
  File "<frozen importlib._bootstrap>", line 935, in _load_unlocked
  File "<frozen importlib._bootstrap_external>", line 994, in exec_module
  File "<frozen importlib._bootstrap>", line 488, in _call_with_frames_removed
  File "/workspaces/django-web/mysite/mysite/settings.py", line 132, in <module>
    from dotenv import load_dotenv
ModuleNotFoundError: No module named 'dotenv'

I will check the API guide later today.


fix(certificates): import re module

Well, this is embarrassing. 😢  Seems to have lost the actual import at some point.

Embarrassing for you and me! Sometimes the simplest ones are the easiest to miss.


added channel slack notifications

Purpose

How does the solution address the problem

Changes made

prnet cleaned-up successfully


[OPENJPA-2808] indicies are dropped in schema tool

Hi @solomax ,

is it possible to harness that with a test?
wonder if it needs a flag since it can change old behavior in test suites, wdyt?

otherwise looks like an old due debt ;), thanks a lot.

Hello @rmannibucau, thanks for review

Can this TestSchemaGenerator test be considered as good example?
There is SchemaTool.Flags.indexes already, is it not enough?

Sorry for the long response


Support for prmopts

Thanks for your tool. I find it very useful for glueing things together with GPT. Before I write a wrapper for your tool / needless alternative, do you think there is any chance that something like templating would get into this tool?
I was imagining something like this:
echo "name: Dave\n country: France" | sgpt --template "What would a person called {name} be called in {country}"

This would be analogous prompts in Langchain https://docs.langchain.com/docs/components/prompts/.
I would entirely understand if you considered this beyond the scope of this tool. If so I might make my own....
I think a reasonable alternative would be to use something like gomplate for templating, but this is less discoverable.

Sounds good to me - thanks for the suggestion!
I am going to add it to the roadmap. I do not know yet when this feature will be implemented.


Simplify  for unifi

What this PR does / why we need it:
This merges the three services into one, letting you have one service (ClusterIP or LoadBalancer, NodePort) that has all ports necessary for proper communication with UAPs.
The problem now is that with each Service you get a different clusterIP (or expensive LoadBalancer) while actually Unifi requires the 8080:/inform, stun, and discovery to be on the same interface. (Same as upstream docker).
Unifi can't reach stun port right now because its on a different IP, or with NodePort on a different port.  NodePort can still work but its also easier through a single service.
Unifi also has no separate IP binding for these ports meaning they are all expected to run off the same IP.
In fact, if the maintainer allows it, we can also remove the controllerService and move everything in one service with multiple named ports.
Signed-off-by: Werner Buck<EMAIL_ADDRESS>Checklist
[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

[x] DCO signed
[x] Chart Version bumped
[x] Variables are documented in the README.md

/lgtm

This merge breaks my UniFi deployment, please see #11268.


Should challenge.lastUpdated be added to the streak calculation?

Since there is a new filed lastUpdated shouldn't it be added to streak calculation and to the calendar heatmap?
I could do it.

I think we intended the streak to be for brownies.  We wouldn't, for example, want someone to just open up a previous challenge, add a useless comment, and save it again.
I'll defer to @QuincyLarson on this, though.

I think we're also trying to discourage people from going back and spending too much time on their algorithms also.

cc @BerkeleyTrue.

Last updated shouldn't be used for streaks. It's there mostly to let people know we've actually updated the challenge.
@sludge256 Is also right, there is to much emphasize on challenges and not enough on projects.

Closing this issue then. @LenaBarinova thanks for posting.


ci: dockerhub

- What I did
separate dockerhub automated builds from latest tagging
- How I did it
- How to verify it
- Description for the changelog
ci: dockerhub

cc: @cpswan
We made this change because when I did pre-releases of sshnp it got pushed to latest on dockerhub.


Not really an issue, a question

Hi,
Loving your gem, it's my first go at NFC card reading, building it on a Raspberry Pi.
I've got it working, but just a quick question:
The time from putting the card on the NFC reader (ITEAD PN532 NFC MODULE) to the code executing is a couple of seconds.  Is there anyway to speed it up?  Faster polling?
Thanks
Al

Hi,
How do you measure the time and what type of NFC tag are you trying to process?
I tested different tags and reaction was much faster no difference from the application written in C.


Add Postgres psycopg2 package to Dockerfile

To use tooling with PostgreSQL, need the psycopg2 module baked into container.
This involves:

Removing psycopg2-binary pip package
Adding libpq-dev and python-dev system packages
Adding psycopg2 pip package
Updating the BASE_IMAGE to senzing/senzing-base:1.5.5

See changes in https://github.com/Senzing/docker-xterm/compare/1.0.3...1.0.5

Is this the module requested?
https://github.com/Senzing/docker-xterm/blob/45c7e85282657503a686f29d1c387980e9ea8c71/Dockerfile#L63

Is this the module requested?
https://github.com/Senzing/docker-xterm/blob/45c7e85282657503a686f29d1c387980e9ea8c71/Dockerfile#L63

Reference: https://stackoverflow.com/questions/11583714/install-psycopg2-on-ubuntu

Pull requests:

https://github.com/Senzing/docker-xterm/pull/13
https://github.com/Senzing/docker-xterm/pull/15


Add list of notice names to the dataset response

User Experience
API users should be able to query a dataset and see the validation notices it includes, grouped by severity. This makes it possible for consumers to:

Screen or pre-process a dataset based on its expected data quality issues: what types of issues should I expect from this data?
Run analytics on the quality of a feed over time (compare quality between a feed's datasets): is the quality of this feed getting better over time? does something look unusual in this new feed release?

Expected Result
Using dataset id mdb-1210-202402121801 as an example:
Option 1:
      "validation_report": {
         "validated_at": "2023-07-10T22:06:00Z",
          "features": [
            "Fares_V1"
          ],
          "validator_version": "4.2.0",
            "total_unique_error": 1,
            "total_unique_warning": 2,
            "total_unique_info": 3
            "errors": {
                decreasing_or_equal_stop_time_distance
            }
            "warnings": {
                equal_shape_distance_same_coordinates
                fast_travel_between_consecutive_stops
                leading_or_trailing_whitespaces
                missing_feed_info_date
                missing_recommended_field
                mixed_case_recommended_field
                route_color_contrast
                route_short_name_too_long
                same_name_and_description_for_route
                stop_too_far_from_shape_using_user_distance
                stop_without_stop_time
                trip_distance_exceeds_shape_distance_below_threshold
            }
            "info": {
                unknown_file
            }
      }

Option 2:
      "validation_report": {
         "validated_at": "2023-07-10T22:06:00Z",
          "features": [
            "Fares_V1"
          ],
          "validator_version": "4.2.0",
            "total_unique_error": 1,
            "total_unique_warning": 2,
            "total_unique_info": 3
            "notices": [
                {
                    "code": "decreasing_or_equal_stop_time_distance",
                    "severity": "ERROR"
                },
                {
                    "code": "equal_shape_distance_same_coordinates",
                    "severity": "WARNING"
                }
            ]

      }

Considerations

If we want to model the full validation report as an API endpoint in the future, does the response schema need to look different to scale? (e.g to show occurrences of each notice)

Decided this may not be useful for consumers - closing for now. Idea would be:

Consumers invested in the data quality report will download and parse the JSON report of the validator
Consumers who aren't don't want a long validation report response


Fix interfaces

should handle string keys when provided... also signature vs proof interface... make things clearer.

done. https://github.com/transmute-industries/PROPOSAL-OpenPgpSignature2019/commit/7810df089d0d3b9e5d3f17a4f4e14405c75a33d3


Card values

Hi again. I'd like to see the functions low_value and high_value defined for all cards.
julia> C = 5♠
5♠

julia> value(C)
5

julia> high_value(C)
ERROR: UndefVarError: high_value not defined
Stacktrace:
 [1] top-level scope
   @ REPL[15]:1

julia> low_value(C)
5

For Poker, you'd want to use high_value; for solitaire, I'd use low_value. This makes PlayingCards "agnostic" about ace high vs. ace low.
My further suggestion is not to define value at all, and let the user either choose which value function to use, or else define their own function either value=low_value or value=high_value.
Let me know if you'd like me to edit PlayingCards.jl to implement, or send some code snippets you can cut/paste.
Thanks,
Ed

I’m happy to change the name, I was 50-50 on which to use (value vs high_value). I’ll open a PR

Thanks!


fix(forms-web-app): validate the payload to create or update appeal

Ticket Number

UCD-944
Description of change

Bit of a proof of concept - all the controller need this, so I am doing them all. This is the first.
Checklist

[ ] Requires infrastructure changes
[ ] If adding or remove environment variables (e.g. in docker-compose.yaml) then I have updated the appropriate Helm chart
[ ] I have updated the documentation accordingly
[x] My commit history in this PR is linear
[ ] New features have tests
[ ] Breaking change (team conversation required)

Important
Please do not merge from master (please only rebase). This keeps the history linear and easier to debug.

ty

ty


Trigger starts from start time, not start time + recurrence

The article states "Therefore, the trigger runs the pipeline 15 minutes, 30 minutes, and 45 minutes after the start time".
Just tried myself as I was getting a job running twice, when I only wanted it once, and it also run the pipeline at the start time. Example was to run a job at 4:25 PM, with a recurrence of 5 minutes, and end date at 4:31 PM. Per above statement, it should run once at 4:30 PM, but it actually executed at 4:25:01 and then at 4:29:59.
Would be nice to instead have some kind of "one-time" trigger, instead of playing with the schedules, but in the meantime it would be nice to get this clarified. Thanks!

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

ID: 89177180-caeb-e2e1-5614-60719f593672
Version Independent ID: a3352be5-2343-ae87-38d0-32312a62d2b1
Content: Create schedule triggers in Azure Data Factory - Azure Data Factory
Content Source: articles/data-factory/how-to-create-schedule-trigger.md
Service: data-factory
GitHub Login: @djpmsft
Microsoft Alias: daperlov

Hey @falmada,
You can manually trigger a run from either the UX or the SDK! To trigger a run via the UX, go to the pipeline canvas. Click Add trigger and then Trigger now. This will be a one-time trigger of the published pipeline
Thanks,
Daniel

But answer your original question, Schedule triggers have their first run at the start time and then run in the recurrence intervals after

Thanks @djpmsft, i did the schedule for another day and then found out it run twice (not running the flow immediately but off hours). Re-reading the mentioned text on the article, it does give an impression that it will run only after the recurrence, not on start time, might want to clarify there. Thanks!

@falmada : At this time I believe that if you only read the 4(b) it does point to which you had mentioned , but then 4(a) does talk about the start time .
At this time we are going ahead and closing this issue , if you feel appropriate you can always reopen this again .


fix: disable back button when updating biometry

Signed-off-by: wadeking98<EMAIL_ADDRESS>Summary of Changes
Previously when updating the biometrics preferences through the settings screen, the user could sometimes click the back button before they were prompted to enter their pin. These changes disable the back button on the navigator after the user has updated their biometrics preferences, and re-enables the back button after the user has supplied their PIN.
Before updating:

After updating:

After PIN:

Related Issues
N/A
Pull Request Checklist
Tick all boxes below to demonstrate that you have completed the respective task. If the item does not apply to your this PR check it anyway to make it apparent that there's nothing to do.

[x] All commits contain a DCO Signed-off-by line (we use the DCO GitHub app to enforce this);
[x] Updated LICENSE-3RD-PARTY.md for any added dependencies or vendored components;
[x] Updated documentation as needed for changed code and new or modified features;
[x] Added sufficient tests so that overall code coverage is not reduced.

If you have any questions to any of the points above, just submit and ask! This checklist is here to help you, not to deter you from contributing!
Pro Tip 🤓

Read our contribution guide at least once; it will save you a few review cycles!
Your PR will likely not be reviewed until all the above boxes are checked and all automated tests have passed.

PR template adapted from the Python attrs project.

@wadeking98 Curious, why does it need another dispatch action?

@jleach I needed to communicate with the settings navigation stack so I could disable/enable the back button. The best way to do that seemed to be through the state

@wadeking98 Curious, why does it need another dispatch action?

@jleach I needed to communicate with the settings navigation stack so I could disable/enable the back button. The best way to do that seemed to be through the state

Wouldn't you be able to watch useBiometry: false,?

@wadeking98 Curious, why does it need another dispatch action?

@jleach I needed to communicate with the settings navigation stack so I could disable/enable the back button. The best way to do that seemed to be through the state

Wouldn't you be able to watch useBiometry: false,?

I can watch useBiometry: false but that value is in the store and is only updated once the user has successfully authenticated with their PIN. I need to know when the user presses the toggle button so that I can disable the back button until the user is presented with the PIN screen. The only state that gets updated exactly when the user presses the toggle button is setCanSeeCheckPIN() and I think that state is only at the UseBiometry component level so I can't watch it from the navigation stack


How to populate input variable length data

Hello, thank you for your work.
I am using the implementation of your SE3-Transformer as part of my project, but I have encountered some problems when processing variable length data input into your model. I do not know how to fill in features, coordinates and masks to meet the needs of the model.
The processing of input data in my DataSet is as follows:

I fill the coordinates and features with zeros to ensure that their input dimensions are (246,3) and (246,20) respectively, and fill the mask with true and false bool types.
In this case,there are 49 valid values,.So the shapes with valid features and coordinates are (49,20) and (49,3) and the rest are padded to zero. The first 49 of the mask is true and the rest is false
Then I check through the torch.utils.data.DataLoader get input dimension is no problem:

My network looks like this, A typical binary classifier，I output the result of se3：

However, the output of se3 is NaN as shown in the figure below.

The first 48 have results, the rest are NaN, not clear what the problem is
The next batch may all be NaN due to model weight changes

The output of SE3 causes all my losses to be NaN
I think these NaN are probably caused by my incorrect filling method. I have also tried to fill in "1", but it was also ineffective. Maybe the mask I typed in can't have false, and I'm confused about that. Could you please tell me how to correctly fill coordinates, features and masks or give me some advice on using your SE3 model to handle variable length data.
Hope this helps! ! thank you

Hi! Could you possibly provide me the full script where I can reproduce the NaN error?

Thanks, I've written a simple README that I hope will be helpful in reproducing the error
test_se3.zip

@zyk19981118 thank you for the scripts! I believe I have fixed the issue in 0.0.23 https://github.com/lucidrains/se3-transformer-pytorch/releases/tag/0.0.23

@zyk19981118 also welcome you to try https://github.com/lucidrains/egnn-pytorch or https://github.com/lucidrains/En-transformer

thanks a lot！

@zyk19981118 I also took a look at your custom network, and decided to add a feature, where you can do net(feats, coors, mask = mask, return_pooled = True)
by setting returned_pooled = True, it will do a masked mean over your sequence, which you can then linear out for your prediction

That's a good idea，waiting for your good news.


Extract the first image from the github-repo readme

Usually the first image in the github-repo readme is informative. It would be useful to store its link.
One potential solution is to update https://github.com/small-thinking/taotie/blob/main/taotie/sources/github.py to explicitly extract the first image.
Another potential solution is to update the prompt in https://github.com/small-thinking/taotie/blob/main/taotie/consumer/info_summarizer.py#L39-L73 to also include the first image link.

Resolved in #58 .


GitExtensions 3.0.0 - Installation fails due to Remove=Icons

https://chocolatey.org/packages/gitextensions
C:\windows\system32>cup gitextensions
Chocolatey v0.10.11 Business
Upgrading the following packages:
gitextensions
By upgrading you accept licenses for the packages.
You have gitextensions v2.51.5 installed. Version 3.0.0 is available based on your source(s).
Progress: Downloading gitextensions 3.0.0... 100%
gitextensions v3.0.0 [Approved]
gitextensions package files upgrade completed. Performing other installation steps.
Copying gitextensions
from 'C:\ProgramData\chocolatey\lib\gitextensions\tools\GitExtensions-<IP_ADDRESS>33.msi'
Virus check: 0/48 scan engines flagged this assembly.
Due to possible false positives we fail at 4 minimum positives.
Virus scan engines indicate this file is probably safe. For details see
https://www.virustotal.com/...
Installing GitExtensions-<IP_ADDRESS>33.msi...
Generic MSI Error. This is a local environment error, not an issue with a package or the MSI itself - it could mean a pending reboot is necessary prior to install or something else (like the same version is already installed). Please see MSI log if available. If not, try again adding --install-arguments="'/l*v c:\gitextensions_msi_install.log'". Then search the MSI Log for "Return Value 3" and look above that for the error.
ERROR:
Running ['C:\WINDOWS\System32\msiexec.exe' /i "C:\Users\keithdavis\AppData\Local\Temp\chocolatey\gitextensions\3.0.0\GitExtensions-<IP_ADDRESS>33.msi" /quiet /norestart ADDDEFAULT=ALL REMOVE=AddToPath,Icons] was not successful.
Exit code was '1603' Exit code indicates the following: Generic MSI Error. This is a local environment error, not an issue with a package or the MSI itself - it could mean a pending reboot is necessary prior to install or something else (like the same version is already installed). Please see MSI log if available. If not, try again adding --install-arguments="'/l*v c:\gitextensions_msi_install.log'". Then search the MSI Log for "Return Value 3" and look above that for the error...
The upgrade of gitextensions was NOT successful.
Error while running 'C:\ProgramData\chocolatey\lib\gitextensions\tools\chocolateyInstall.ps1'.
See log for details.
Tried running the choco command for this without /quiet:
"C:\WINDOWS\System32\msiexec.exe /i "C:\Users\keithdavis\AppData\Local\Temp\chocolatey\gitextensions\3.0.0GitExtensions-<IP_ADDRESS>33.msi" /norestart ADDDEFAULT=ALL REMOVE=AddToPath,Icons"
This is what happened.

If I remove Icons from "REMOVE=AddToPath,Icons", I don't get the error.

Thanks for opening this issue, I'll have a look at this.
Could be that they have renamed the Icons variable that we try to remove.
I'll need to test this before a fix version can be pushed.

It indeed seems like they have completely removed the option to disable installing Icons (can't find any reference to it anymore, not even under other names).
I'll remove it and push out a fix version shortly.


Language package download

Description of the new feature / enhancement
I would like to use text extractor to detect more languages, however language package could only be download by power shell while running as administrator. It could not work even download language in windows system and also change system language. But in company computer, we could use power toy but not running as administrator. Could we make it easy to down load language package for text extractor?
Scenario when this would be used?
multi-language are used in daily work and extract foreign language is more useful than local language.
Supporting information
No response

You can add languages through the Windows settings > Time & language > Language & region > Add a language
Does that work for you? /needinfo

For Text Extractor, currently no plans, Snipping Tool's OCR system can do multi-language OCR natively and out of the box.  Here is me capturing some issues from PowerToys

https://blogs.windows.com/windows-insider/2023/09/14/new-updates-for-snipping-tool-and-phone-link-begin-rolling-out-to-windows-insiders/

You can add languages through the Windows settings > Time & language > Language & region > Add a language
Does that work for you? /needinfo

Thanks for reply. It does not work even I change the system language.


How to define multiple units in scales like font-size

Clearly describe the documentation issue

Documentation does not specify how to define multiple units for a scale like font-size, because example configuration throws invalid option error:
Invalid Option: Invalid option "{"scale":[1,1.5],"units":["px","rem"]}" for rule scales/font-size

Secondary options only allow to define one unit in unit property, but font-size docs show that multiple unit validation is possible.

What solution would you like to see?

Not sure, though. I guess that a whole rule definition within the documentation like:
"scales/font-size": [[1, 1.5, 2], { unit: "rem" }],

It looks like the documentation doesn't correspond to how the primary option currently works. However, I think the documentation reflects how the primary option should work, i.e. explicit units rather than checking all <length> values.
I'll rustle up a pull request with a non-breaking change to add support for explicit units.

It looks like the documentation doesn't correspond to how the primary option currently works. However, I think the documentation reflects how the primary option should work, i.e. explicit units rather than checking all <length> values.
I'll rustle up a pull request with a non-breaking change to add support for explicit units.


Implement $_SERVER support into Http\Request class

The Http\Request class needs to support the PSR-7 interface methods for dealing with the $_SERVER superglobal data. For Slim, this means the \Slim\Http\Request::getServerParams() method should return the data encapsulated by the current Environment object. The Environment object data, I imagine, should be injected during Request instantiation. This is currently unclaimed and anyone is free to work on this. Let me know if you intend on working on this so I can assign it.

I'd like to get involved in contributing to Slim. Can I claim this and give it a poke and see how I get on?

:+1: Go for it. Feel free to ask any questions here if you have them.

I've sent a PR, but not sure if there is an intention to do something different with Environment? I can see it's initialised only with $_SERVER at the moment, but is the idea for it to be used with other data too?


Update README.md

Base goals
[ X] Clear and descriptive commit messages
[ X] Code is written by the student
[ X] Code is organized and easy to read
[ X] Code demonstrates an understanding of underlying data/statistics concepts
[ X] MVP Complete
Stretch goals

[ ]   Exceeds accuracy and efficiency expectations of the assignment
[ ]   At least 1 stretch completed
Notes:
Good Job!