cmd/go: two possible canonical states when go.sum is empty, the file may or may or may not exist

What version of Go are you using (go version)?
go version go1.13.5 darwin/amd64
Does this issue reproduce with the latest release?
using latest
What operating system and processor architecture are you using (go env)?
macOS Catalina
GOHOSTARCH="amd64"
GOHOSTOS="darwin"
What did you do?

Created simple hello world application without dependencies consisting of main.go and a go.mod file containing:

module github.com/stephanwesten/hellojob

go 1.13

Executed go build and noticed that no go.sum was generated (unexpected)

added a dummy dependency to go.mod, re-ran go build and a go.sum was generated.

removed dependency from go.mod, so it is back to original above, and re-ran go build and go mod tidy

go.sum was still there.

What did you expect to see?
(Note that this is the first time for me using go modules so my formulation might not be 100% accurate)
Either go.sum is removed or it should have been generated with the first go.build. This would give symmetric behaviour.
I would prefer that go.sum is always generated. The reason why this was a problem for me is that I used a Dockerfile template that assumed a go.sum file to be there.
What did you see instead?
see above
BTW I entered 'go bug' to file this issue and I got a github error. Perhaps temporary problem?

Thanks for reporting.
We should check if this asymmetry has been discussed in the past and if any decision was made. If not, that can be done in this issue.
/cc @bcmills @jayconrod @matloob as cmd/go owners.

BTW I entered 'go bug' to file this issue and I got a github error. Perhaps temporary problem?

What was the error text? If you can still reproduce the issue, please file a separate issue for that so we can investigate. I tried running go bug just now and did not get an error.

schrodinger's go.sum.
sorry, I'll see myself out now

I don't think we should delete the go.sum file entirely when it becomes empty: leaving an empty file makes it more obvious that the edit is intentional (rather than accidental).
I'm ambivalent about creating the go.sum file unconditionally. If the module really doesn't have any dependencies, then we really don't need to create that file, and some folks may find it annoying for the go command to implicitly create an empty file that they don't need.
For the specific use-case of a Dockerfile template, you can always touch go.sum at some point prior to the build.

Imo go build should create the go.sum file always if there is mod file since this gives the most consistent behavior. I see go.sum as some internal housekeeping, its content is not relevant to the user, but its presence does impact the user.


fix(postgres): parse timestamp dates less than year 1000

Bug
Parsing a timestamp column in postgresql with a date less than year 1000 will create an invalid Date when using the forceUtcTimezone set to true.
Reproduction
Created test for reproduction, run without the fix to PostgresSqlConnection.ts.
Fix
Use the date parsing function from postgres-date that is used by default in pg for parsing dates but add the Z to force the utc timezone. new Date('0022-01-01 09:17:58.000Z') when parsing the timestamp column str creates an invalid date in js.

please target master branch, then once we merge, we can cherry pick to 5.x

please target master branch, then once we merge, we can cherry pick to 5.x

Thats what I originally wanted to do but there seems to be a refractor of the PostgreSqlConnection in master, it will require two different fixes. How do you want to do that?

Thats what I originally wanted to do but there seems to be a refractor of the PostgreSqlConnection in master, it will require two different fixes. How do you want to do that?

Then I'd like to first see a PR for master/v6, keep this one targeting v5. Once we have it fixed in v6, we can merge this one, but I don't want to fix things only here.

Btw, just curious, what kind of dates are you storing that you've hit this? :]

gave this a closer look, and its not just about postgres, neither it is about year 1000, just about some particular years close to 0. looks like its enough to add T into the datetime string instead of the space to fix this. fixing this in v6 is actually much easier than v5, as date parsing happens on a single place for all drivers. since nobody else asked about this, i am fine with merging this in for v5, and keeping the proper fix to all drivers in v6 only


Batch writes tutorial

Thanks @reugn . I have to add a link to the blog post which should not require another review. Will merge after that.


Cookie Editor

🛠 Fixes Issue
Closes #232
👨💻 Changes proposed
cookies editor which can:
get all cookies
add cookies
fetch cookies from a particular website
remove cookies
✔ Check List (Check all the applicable boxes)

[x] My code follows the code style of this project.
[x] This PR does not contain plagiarized content.
[x] The title of my pull request is a short description of the requested changes.

📄 Note to reviewers
📷 Screenshots

Demo:
https://user-images.githubusercontent.com/49890655/167068300-a4120fef-648c-4232-8518-698e732f0b84.mp4


Start a pomodoro.

User Story
As a player
I want to start a new pomodo
So that I am reminded to take a break
Acceptance Criteria
And I know I am done when

[x] Pomodo length lasts 25 minutes
[x] Pomodoro start message: Pomodoro started.
[x] Pomodoro end message Pomodoro finished.
[x] Syntax: /pomodoro start

Resolved by https://github.com/rdok/eso-pomodoro/commit/cebf7c9d0f413ca4d13f17cb3a754f9199c4bf06


Update to Sylelint 16 & Migrate to ESM

This builds on PR #11.  My attempt to migrate gulp-stylelint to an ESM module.

Hi @ronilaukkarinen
Can you please look into this PR?
Stylelint is a quite significant bump and this would be very valuable
thanks in advance

Hi @ronilaukkarinen Can you please look into this PR?
Stylelint is a quite significant bump and this would be very valuable
thanks in advance

Unfortunately I do not have time right now, we had to delay the introduction of Stylelint 16 with our team to Feb-Mar. We are currently using Stylelint 15. Feel free to test the PR yourself and review this, @bpfoster or @schalkneethling could perhaps help testing and merging this if they have more time in their hands?

Finally took a look at this. Referencing issue https://github.com/developer-stylechain/gulp-stylelint/issues/10
Thank you for all the hard work @bpfoster. @kksandr7's point about https://github.com/adorade/gulp-stylelint-esm/issues/20 stands and that kinda woke me up, why we don't use gulp-stylelint-esm instead of gulp-stylelint? Doesn't it supersede this one?
Worth noting: I'm not 100% expert in advanced JS, just trying to keep this project alive as best I can.
When I tried the changes in this PR, I get:
Error [ERR_REQUIRE_ESM]: require() of ES Module /Users/rolle/Projects/gulp-stylelint/src/index.js from /Users/rolle/Projects/devpackages/gulp/tasks/lintstyles.js not supported.
Instead change the require of index.js in /Users/rolle/Projects/devpackages/gulp/tasks/lintstyles.js to a dynamic import() which is available in all CommonJS modules.
    at Object.<anonymous> (/Users/rolle/Projects/devpackages/gulp/tasks/lintstyles.js:5:19) {
  code: 'ERR_REQUIRE_ESM'
}

I guess it is due to the breaking changes, so doesn't this mean README.md needs changes in instructions as well, you cannot use the Quick Start example as-is we have right now?
I'm not familiar with ESM and my current linstyles.js task is as follows:
// Dependencies
const {
  src
} = require('gulp');
const stylelint = require('@ronilaukkarinen/gulp-stylelint');
const config = require('../config.js');

// Task
function lintstyles() {

  return src([config.stylelint.src])

    // Print linter report
    .pipe(stylelint(config.stylelint.opts));
}

exports.lintstyles = lintstyles;

What exactly needs to be changed?
Also:
npm error Could not resolve dependency:
npm error peer stylelint@"10 - 15" from<EMAIL_ADDRESS>npm error node_modules/@ronilaukkarinen/stylelint-a11y
npm error   dev<EMAIL_ADDRESS>from the root project
npm error
npm error Fix the upstream dependency conflict, or retry
npm error this command with --force or --legacy-peer-deps
npm error to accept an incorrect (and potentially broken) dependency resolution.

But as it seems there is another actively developed stylelint-a11y package that has 16 support so we can go around that and leave our stylelint-a11y as-is.


feat: display default dirs in ilab system info

With the introduction of persistent storage, it has become easier for ilab to manage
data, but harder for the user to. This commit adds 3 fields to ilab system info which
display where the default cache, data, and config directories are.
Resolves #1858

Checklist:

[x] Commit Message Formatting: Commit titles and messages follow guidelines in the
conventional commits.
[x] Changelog updated with breaking and/or notable changes for the next minor release.
[ ] Documentation has been updated, if necessary.
[ ] Unit tests have been added, if necessary.
[ ] Integration tests have been added, if necessary.

This is how the output of ilab system info now looks with this PR:
(venv) osilkin:instructlab/ (list-default-dirs✗) $ ilab system info                                                                                [0:25:47]
sys.version: 3.11.8 (main, Jul  3 2024, 13:50:00) [Clang 15.0.0 (clang-15<IP_ADDRESS>)]
sys.platform: darwin
os.name: posix
platform.release: 23.5.0
platform.machine: arm64
instructlab.version: 0.18.0a7.dev4
instructlab-dolomite.version: 0.0.1
instructlab-eval.version: 0.1.0
instructlab-quantize.version: 0.1.0
instructlab-schema.version: 0.3.1
instructlab-sdg.version: 0.2.0
instructlab-training.version: 0.3.0
torch.version: 2.3.1
torch.backends.cpu.capability: NO AVX
torch.version.cuda: None
torch.version.hip: None
torch.cuda.available: False
torch.backends.cuda.is_built: False
torch.backends.mps.is_built: True
torch.backends.mps.is_available: True
llama_cpp_python.version: 0.2.79
llama_cpp_python.supports_gpu_offload: True
defaults.cache_dir: /Users/osilkin/Library/Caches/instructlab
defaults.data_dir: /Users/osilkin/Library/Application Support/instructlab
defaults.config_home: /Users/osilkin/Library/Caches/instructlab

Signed-off-by: Oleg S<EMAIL_ADDRESS>

@tiran Your approach makes much more sense. I'll update this PR to do that instead.

@tiran Your approach makes much more sense. I'll update this PR to do that instead.

Let's make it two PRs. Update this PR to land the ilab system info improvement and make a new one to list paths.

@tiran Do we still want to make this change to ilab system info though? Based on your response it sounds like a separate debug command of some kind may make more sense. I completely agree that putting paths in ilab system info will create a messy output.

@RobotSail - could you please rebase?

@ktam3 I'm just gonna close it


Merge fixes and improvements from my branch

Includes partial riscof support, exception handling tweaks, fixes to broken links due to moving to a Github org, RVSW fixes, and tweaks to Linux-y stuffs.

Oops, I should merge to staging first mb


locally call train interface  failed in cog-sdxl

use cog-sdxl in  https://github.com/replicate/cog-sdxl
run cog build  to generage docker image
run docker locally  use gpu
call the train interface usr curl

curl -X 'POST' \
  'http://<IP_ADDRESS>:5000/trainings' \
  -H 'accept: application/json' \
  -H 'Content-Type: application/json' \
  -d '{
  "input": {
    "input_images": "https://github.com/replicate/cog-sdxl/blob/main/example_datasets/zeke.zip"
  }
}'

always got Internal Server Errorl
When I check the server log ,I got this error:
{
    "logger": "uvicorn.error",
    "timestamp": "2024-07-16T11:25:07.341402Z",
    "exception": "Traceback (most recent call last):\n  File \"/root/.pyenv/versions/3.9.19/lib/python3.9/site-packages/uvicorn/protocols/http/httptools_impl.py\", line 399, in run_asgi\n    result = await app(  # type: ignore[func-returns-value]\n  File \"/root/.pyenv/versions/3.9.19/lib/python3.9/site-packages/uvicorn/middleware/proxy_headers.py\", line 70, in __call__\n    return await self.app(scope, receive, send)\n  File \"/root/.pyenv/versions/3.9.19/lib/python3.9/site-packages/fastapi/applications.py\", line 284, in __call__\n    await super().__call__(scope, receive, send)\n  File \"/root/.pyenv/versions/3.9.19/lib/python3.9/site-packages/starlette/applications.py\", line 122, in __call__\n    await self.middleware_stack(scope, receive, send)\n  File \"/root/.pyenv/versions/3.9.19/lib/python3.9/site-packages/starlette/middleware/errors.py\", line 184, in __call__\n    raise exc\n  File \"/root/.pyenv/versions/3.9.19/lib/python3.9/site-packages/starlette/middleware/errors.py\", line 162, in __call__\n    await self.app(scope, receive, _send)\n  File \"/root/.pyenv/versions/3.9.19/lib/python3.9/site-packages/starlette/middleware/exceptions.py\", line 79, in __call__\n    raise exc\n  File \"/root/.pyenv/versions/3.9.19/lib/python3.9/site-packages/starlette/middleware/exceptions.py\", line 68, in __call__\n    await self.app(scope, receive, sender)\n  File \"/root/.pyenv/versions/3.9.19/lib/python3.9/site-packages/fastapi/middleware/asyncexitstack.py\", line 20, in __call__\n    raise e\n  File \"/root/.pyenv/versions/3.9.19/lib/python3.9/site-packages/fastapi/middleware/asyncexitstack.py\", line 17, in __call__\n    await self.app(scope, receive, send)\n  File \"/root/.pyenv/versions/3.9.19/lib/python3.9/site-packages/starlette/routing.py\", line 718, in __call__\n    await route.handle(scope, receive, send)\n  File \"/root/.pyenv/versions/3.9.19/lib/python3.9/site-packages/starlette/routing.py\", line 276, in handle\n    await self.app(scope, receive, send)\n  File \"/root/.pyenv/versions/3.9.19/lib/python3.9/site-packages/starlette/routing.py\", line 66, in app\n    response = await func(request)\n  File \"/root/.pyenv/versions/3.9.19/lib/python3.9/site-packages/fastapi/routing.py\", line 259, in app\n    content = await serialize_response(\n  File \"/root/.pyenv/versions/3.9.19/lib/python3.9/site-packages/fastapi/routing.py\", line 145, in serialize_response\n    raise ValidationError(errors, field.type_)\npydantic.error_wrappers.ValidationError: 1 validation error for TrainingResponse\nresponse\n  value is not a valid dict (type=type_error.dict)",
    "severity": "ERROR",
    "message": "Exception in ASGI application\n"
}

I also tried to trace the call stack, but I couldn't identify the cause of the issue.

the cog version is 0.9.8

I alse tried the simplest train exapmles（hello-train） in  https://github.com/replicate/cog-examples.git
And I got the same exception and Traceback

Can someone help me with this issue, or could you tell me how to correctly call the training function locally?

Hy any luck on this one? I faced same issue  @sinopec

too...


Add Maven information to Plugin Card

This PR implements add the Maven information on the plugin card on website.
I don't know how to test if locally but the javascript part was tested, not the rendering.
Regards
Philippe M. from UbikLoadPack Team

Hi @undera ,
Is this one OK for you ?
Thanks

I tried locally, not satisfied how it looks. I'll work on plugin card display and will merge this change.

Merging as-is, for future improvement.

I have rearranged maven info on card.

Thanks, it’s better indeed.
But maybe the last version should be displayed instead of the oldest one.
Regards

I did not change the way it parses the version. Feel free to alter the algorithm


Run formatter on *.cc & *.h

I've ran the first code style formatter I came across. Thus all cpp files look the same.
If you prefer to use another style, just let me know which one.
The code is not changed by this PR, only its style.

I'm not sure I understand what you mean.. so this is changing the style of the files without changing the content? How would the cpp files look different? What would you have done?

@Shovan-Physics click on Files changed above and you will see everything.
The code is not changed, only the style's been updated, i.e. spacing.
Thus the executable code is identical.
I used an automatic tool from VSCode, thus all files follow the same style pattern now.

OK, it looks alright to me, and I don't know what other styles are there, so I'll just approve it!

Ah sorry, I've forgot that we don't have any tests here. Would you mind at least checkout this version and recompile this code? At lease we should see that there are no odd errors.

Sure, I will

Hi Anton, I ran this version for the sample parameters. It worked fine.

@anton-buyskikh Can we merge this pull request while retaining subsequent changes to the main branch (e.g. license, standard makefile etc)?

@anton-buyskikh I also made the following changes to the main branch:
(1) Modified the README as we discussed last time
(2) Added the new file cDMRG_inputIDgen.nb for storing/finding input IDs
(3) Modified the file cDMRG_input_documentation.nb

@anton-buyskikh I guess the easiest option is to rerun the formatter on the .cc and .h files in the main branch and directly change those.


Updating for clarification of Ubuntu 16.04/18.04

From MS SQL Linux support - updating for clarification of ubuntu 16.04 vs 18.04 repository links so customers download correct ones

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

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

Hi @joshts0 - Thanks for the updates! I'll approve these changes. We appreciate your help on clarifying these docs.
#sign-off

Hi @joshts0 - Thanks for the updates! I'll approve these changes. We appreciate your help on clarifying these docs.
#sign-off

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

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


feat(jest-message-util): add support for AggregateErrors

Summary
This PR adds support for printing the errors contained in an AggregateError.
Example output:
      ● Test suite failed to run

        AggregateError:

          at Object.<anonymous> (packages/jest-message-util/src/__tests__/messages.test.ts:439:22)

        Errors contained in AggregateError:
         Err 1

              at Object.<anonymous> (packages/jest-message-util/src/__tests__/messages.test.ts:440:7)

         Err 2

              at Object.<anonymous> (packages/jest-message-util/src/__tests__/messages.test.ts:441:7)

Test plan
I've added a single unit test that uses snapshots.

@SimenB Thx. I'm going to fix it tomorrow.

https://github.com/facebook/jest/releases/tag/v29.5.0


Does it support all mobile browser ?

Hi
Can anyone let me know the supported browsers (desktop & mobile) for angular 2 ?
Thanks,
Lakin

@lakinmohapatra The supported browsers list is at the Angular docs available at https://angular.io/guide/browser-support.

Please take note that your issue may be closed as it doesn't follow the team's issue template.


Prototype of spirv-fuzz, a metamorphic testing tool for SPIR-V

This is a simple prototype of the bare bones of spirv-fuzz, featuring a few simple transformations.

I am abandoning this PR as it has been superseded by #2581, which incorporates all the review comments associated with this PR.


Get Frame metadata of IntelRealSense camera with Python

Required Info

Camera Model
{ D400 }

Operating System & Version
{ Ubuntu server 20.04 }

Platform
{ Raspberry Pi 4 }

Language
{ Python }

Issue Description
Hello everyone,
I'm currently working on a project to collect video frames from 3 IntelRealsense cameras (each connected to a Raspberry Pi 4) and send them to a server PC. Timestamps are an important factor in what I am doing in order to have a precise synchronization of the system. For instance, getting the timestamp of each frame is mandatory.
While I was checking the IntelRealsense site, I found this interesting article : https://dev.intelrealsense.com/docs/frame-metadata .
But there's not enough information about how to efficiently use this frame metadata object. I tried to look up for documentation about the object/class/functions but have found nothing on the Github.
So I was wondering if there's anyone who knows more than I do about this tool. I would like to know more about how to use it and the functions of it on order to get a maximum information about each frame (timestamps mainly).
I thank you in advance ;)
Have a nice day

Hi @CherifB98  You should be able to access frame metadata if support for hardware metadata is enabled and you use the instruction get_frame_metadata.  For example for finding the metadata parameter frame_laser_power_mode with Python:
emitter = rs_frame.get_frame_metadata(rs.frame_metadata_value.frame_laser_power_mode)
And for getting the frame_counter metadata parameter:
counter = rs.frame.get_frame_metadata(frame, rs.frame_metadata_value.frame_counter)
From these two examples you can see a pattern for constructing retrieval commands for particular metadata parameters.

Define a descriptive variable name to store the metadata value in, such as 'emitter' or 'counter'.

Set up the instruction for getting the metadata.

[chosen variable name] = rs.frame.get_frame_metadata(frame, rs.frame_metadata_value.

After the full stop after value. add the name of the metadata parameter that you want to retrieve.

You can read more about this subject at https://github.com/IntelRealSense/librealsense/issues/3179
At https://github.com/IntelRealSense/librealsense/issues/2188#issuecomment-409985803 a RealSense team member provides detailed descriptions for a range of metadata parameters.
frame_laser_power_mode

Thank you @MartyG-RealSense  for the answer and for clearing out this issue.
As I can see in the examples you gave, the frame metadata are restored in and from a 'frame' object. Otherwise, I'm trying to get the metadata from an color-image object. Is there a way to get or store the metadata in a color_image object ?

A difference between depth and RGB metadata is that when RGB auto-exposure is enabled, RGB exposure metadata is not accessible.  So you would have to set RGB exposure to manual to retrieve exposure metadata.  A RealSense team member advises about this at https://github.com/IntelRealSense/librealsense/issues/2549#issuecomment-431863920

I have succeeded to get frame metadata and have no problem with exposure. My main goal for now is to get the metadata (or at least the timestamps) directly from the color or depth image (not from rs_frame as for the given examples). Is there any chance to get the color or depth image ? Or is there a way to store the frames metadata in the color and depth images ?
I'm currently working on sending those frames from the Raspberry Pi to a Server PC for further treatment of the images and where the metadata (frames timestamps) are really important to have.
Thank you

I cannot think of a way that the metadata or timestamps could be embedded into an image.  If you have a Raspberry Pi that you want to connect to a networked computer, Intel's ethernet networking system rs-server that was designed with Pi in mind might meet your needs.  It enables data from a RealSense camera attached to a Pi to be accessed on a central host computer by treating the camera as a network device.
https://dev.intelrealsense.com/docs/depth-image-compression-by-colorization-for-intel-realsense-depth-cameras

Hi @CherifB98  Do you require further assistance with this case, please?  Thanks!

Hello @MartyG-RealSense and thank you again for your effort.
My main goal is to stream/record a video with an IntelRealsense camera and have per frame metadata.
After some long researches these are some potentially possible ways to do it :

Make a text file with the metadata
Use python libraries that potentially can help get per-frame metadata (ffmpeg, exif ...)
Use a specific video codec to have per-frame metadata (MKV, CinemaDNG ...)

I would think that if you wanted to be certain that you were obtaining camera metadata then making a text file export of the metadata would be the most appropriate approach, as the rs-save-to-disk C++ SDK example program does by saving "the metadata attributes available for each stream into a comma-separated text file".  It is not saving to a .txt file though but instead to a textual format called csv that is compatible with spreadsheets and databases.
https://github.com/IntelRealSense/librealsense/blob/master/examples/save-to-disk/rs-save-to-disk.cpp
I have looked before at the subject of extracting metadata to csv in Python in https://github.com/IntelRealSense/librealsense/issues/6536 but did not find an equivalent solution.  I researched again today and still could not find a method, unfotunately.

Hi @CherifB98  Do you require further assistance with this case, please?  Thanks!

Thanks very much for the update!


Use model id as tizen device name

Signed-off-by: Boram Bae<EMAIL_ADDRESS>

This breaks #3. The name was intentionally hard-coded to smoothly work with -d tizen.

This breaks #3. The name was intentionally hard-coded to smoothly work with -d tizen.

Oh, I didn't know that. But it's a bit uncomfortable, What exactly should we do without -d?
as-is
flutter-tizen devices
8 connected devices:

SM N950N (mobile) • ce0717175871bf22017e • android-arm64  • Android 9 (API 28)
Linux (desktop)   • linux                • linux-x64      • Linux
Web Server (web)  • web-server           • web-javascript • Flutter Tools
Chrome (web)      • chrome               • web-javascript • Google Chrome 87.0.4280.66
Tizen (mobile)    • emulator-26121       • flutter-tester • Tizen 6.0 (emulator)
Tizen (mobile)    • emulator-26111       • flutter-tester • Tizen 5.5 (emulator)
Tizen (mobile)    • emulator-26101       • flutter-tester • Tizen 6.0 (emulator)
Tizen (mobile)    • 0000d84f00006200     • flutter-tester • Tizen 6.0

after
flutter-tizen devices
8 connected devices:

SM N950N (mobile)          • ce0717175871bf22017e • android-arm64  • Android 9 (API 28)
Linux (desktop)            • linux                • linux-x64      • Linux
Web Server (web)           • web-server           • web-javascript • Flutter Tools
Chrome (web)               • chrome               • web-javascript • Google Chrome 87.0.4280.66
W-6.0-circle-x86 (mobile)  • emulator-26121       • flutter-tester • Tizen 6.0 (emulator)
T-samsung-5.5-x86 (mobile) • emulator-26111       • flutter-tester • Tizen 5.5 (emulator)
M-6.0-x86 (mobile)         • emulator-26101       • flutter-tester • Tizen 6.0 (emulator)
TM1 (mobile)               • 0000d84f00006200     • flutter-tester • Tizen 6.0

I don't have any good workaround at the moment. You'll have to modify the upstream flutter_tools logic to achieve both.
If you're only concerned with the output of the devices command, you may change the Tizen 6.0 part to something like below. I guess the sdkNameAndVersion property is being used for it right now.
flutter-tizen devices
8 connected devices:

SM N950N (mobile) • ce0717175871bf22017e • android-arm64  • Android 9 (API 28)
Linux (desktop)   • linux                • linux-x64      • Linux
Web Server (web)  • web-server           • web-javascript • Flutter Tools
Chrome (web)      • chrome               • web-javascript • Google Chrome 87.0.4280.66
Tizen (mobile)    • emulator-26121       • flutter-tester • Tizen 6.0 (W-6.0-circle-x86) (emulator)
Tizen (mobile)    • emulator-26111       • flutter-tester • Tizen 5.5 (T-samsung-5.5-x86) (emulator)
Tizen (mobile)    • emulator-26101       • flutter-tester • Tizen 6.0 (M-6.0-x86) (emulator)
Tizen (mobile)    • 0000d84f00006200     • flutter-tester • Tizen 6.0 (TM1)

hm... I think that we can find the first device name using sdb and we can use this as an argument instead of 'tizen' in executable.dart.

That would be an option. I tested with this code and at least it seemed to work:
  args = <String>[
    '--suppress-analytics', // Suppress flutter analytics by default.
    '--no-version-check',
    ...args,
  ];

  if (!args.contains('-d') && !args.contains('--device-id')) {
    await runInContext<void>(() async {
      final TizenDevices deviceDiscoverer = TizenDevices(
        logger: globals.logger,
        processManager: globals.processManager,
      );
      final List<Device> devices = await deviceDiscoverer.devices;
      if (devices.isNotEmpty) {
        args.insertAll(0, <String>['-d', devices.first.id]);
      }
    });
  }

But still it's very inefficient since everything is initialized twice in the AppContext (due to the use of runInContext). I'm not sure if there is any workaround.

We don't have to initialize twice. but I think that it looks better than using all the device names as 'Tizen'.
Of course, below code block is also a kind of workaround.
  String firtDeviceID;
  if (!specifiedId) {
    final String out = Process.runSync('sdb', ['devices']).stdout.toString();
    for (final String line in LineSplitter.split(out)) {
      if (line.startsWith('* ') ||
          line.startsWith('  ') ||
          line.startsWith('List of devices')) {
        continue;
      }

      final List<String> splitLine = line.split(RegExp(r'\s{2,}'));
      if (splitLine.length != 3) {
        continue;
      }

      firtDeviceID = splitLine[0];
      break;
    }
  }

  args = <String>[
    '--suppress-analytics', // Suppress flutter analytics by default.
    '--no-version-check',
    if (!specifiedId) ...<String>['-d', firtDeviceID],
    ...args,
  ];


BNB transfer does not work on mainnet

Your first example does not work on mainnet
https://github.com/binance-chain/javascript-sdk/wiki/API-Examples.
I send from this address bnb1xakdtqe4vydk8q9n5n8z52f9qlkgghfanye3yz to bnb136ns6lfw4zs5hg4n85vdthaad7hq5m4gtkgf23.
The error message is completely irrelevant!
I get the issue:

(node:16382) UnhandledPromiseRejectionWarning: Error: unable to query the address on the blockchain. try sending it some funds first: bnb1t6x8k7r2yw33r6tvhcstwnr7qcv7ual74v5mmv
    at BncClient._callee5$ (/Users/rostyslav/Desktop/meta-server/node_modules/@binance-chain/javascript-sdk/lib/client/index.js:394:23)
    at tryCatch (/Users/rostyslav/Desktop/meta-server/node_modules/regenerator-runtime/runtime.js:45:40)
    at Generator.invoke [as _invoke] (/Users/rostyslav/Desktop/meta-server/node_modules/regenerator-runtime/runtime.js:274:22)
    at Generator.prototype.<computed> [as throw] (/Users/rostyslav/Desktop/meta-server/node_modules/regenerator-runtime/runtime.js:97:21)
    at asyncGeneratorStep (/Users/rostyslav/Desktop/meta-server/node_modules/@babel/runtime/helpers/asyncToGenerator.js:3:24)
    at _throw (/Users/rostyslav/Desktop/meta-server/node_modules/@babel/runtime/helpers/asyncToGenerator.js:29:9)
    at processTicksAndRejections (internal/process/task_queues.js:97:5)

It has enough funds, wtf?
https://explorer.binance.org/address/bnb1xakdtqe4vydk8q9n5n8z52f9qlkgghfanye3yz

I use salt from keystore.txt as my private key. On testnet no errors, it works - on mainnet it is fucked.

 I used recovery from keystore.txt and got the testnet shit and took that testnet private key and used the private key to transfer.


Remove Text Value

Version

Version = 6.1.0

Details

Has the question been answered here before?

No

Please provide as much detail as possible below

I need to remove the text value, so there is no value appears inside the ring, how to achieve that?

Hi, have you looked at the documentation for the project?


Building an image without sti-helper from a file system source fails silently

sti build using source from the local filesystem and an image without sti-helper fails silently when copying the source to upload/src, as the upload directory does not exist yet.
The error output from cp is suppressed, even with --loglevel=3.

Quick fix for sti build, in executor.go:293:
	} else {
+		h.fs.Mkdir(filepath.Join(h.request.WorkingDir, "upload"))
		h.fs.Copy(h.request.Source, targetSourceDir)
	}

thanks for the PR: https://github.com/openshift/source-to-image/pull/107

merged https://github.com/openshift/source-to-image/pull/107, good to close this?


Issue to test issue forms

Describe the Bug
Very well written description of bug.
Reproduction Steps

Start game
Useful information
Everything crashes

Expected Result
Well written text explaining what I expected to happen
Screenshots and Videos

Operating System
Windows 11
Mod Version
2.4.0
Minecraft Version
1.17.1
Forge Version
36.2.9
Other Mods
No response
Additional Context
No response

Wow, this is a very well written bug report. Good job!


Add information about the docs being open source and that people can contribute

What type of suggestions?
Missing Information
What page is concerned?
No response
What version of the docs?
No response
What is the issue?
I didn't think about the best way to do that yet, but we should have somewhere that is a bit in your face (the footer maybe?) that mention the fact that the docs is open source and that anyone can contribute with a link to this repository. Sometimes that focus on the fact people can contribute, even if it means "just" creating an issue to help us fix typos or clarify a section.
Code of Conduct

[X] I agree to follow this project's Code of Conduct

Maybe just a link to the docs GitHub repo that says "Contribute here" ? (A button, small tile on top level pages?)

I can make sure we add this to any landing page we create too ! 🎉

Maybe just a link to the docs GitHub repo that says "Contribute here" ? (A button, small tile on top level pages?)

Sounds good, as long as we can see it from all pages, any solution is good to me :)


Investigate generating man pages from comments?

A decent approach — at least to start — may be to treat the topmost comment in src/foo.c as documentation for foo, then converting that to a .txt file and have a craptastic man implementation which is just a glorified cat that fetches files from a fixed path.

Done.


Progressbar not visible in Spyder

Performing a cluster based test (unfortunatelly one that takes a lot of time) I no longer see the progressbar in Spyder:
stat_fun(H1): min=-5.699470 max=3.482243
Running initial clustering
Found 17 clusters
Permuting 9999 times...
HBox(children=(FloatProgress(value=0.0, max=9999.0), HTML(value='')))

this is most likely a result of using auto mode of tqdm progressbar, which seems to be unable to tell whether it is in a notebook or ipython console. It would be nice to have some option to control this behavior (progressbar='text' maybe?) as now everyone who works in ipython/Spyder will likely have no progressbar, which for long running tasks is a bit inconvenient.

This sounds more like an upstream bug with tqdm. Can you look into fixing it there (if it's not already fixed; try using tqdm locally as IIRC we prefer an installed system version when available and new enough)?

did github automatically self-assigned me? weird, I just clicked "submit". Anyway - I can do it.
@larsoner
Unfortunatelly from what I read on multiple occasions it is not possible to distinguish whether the code is running from notebook or ipython (or Spyder). :(

Perhaps they should support magic %tqdm text for example to set within a given session which auto backend will be selected. Might be worth proposing and implementing that at their end. If they reject it, we can add some MNE config value that would triage the class used, but it makes more sense for them to implement it if they're up for it.

@larsoner Good idea, I'll suggest that there. Although the response I imagine might be "just use tqdm.tqdm instead of auto or tqdm.notebook", but I'll try. :)

Although the response I imagine might be "just use tqdm.tqdm instead of auto or tqdm.notebook", but I'll try. :)

Yes if they do this we can support it locally.  So hopefully they see the value in it :)

@mmagnuski any progress at the tqdm end? We're recommending Spyder as the IDE in our release, so if they're not fixing it, we should.


Inaccessible variables in context of placeholders

Given:
definition f {A B : Type} (x : A) : B → A := λ x, _

Lean shows:
don't know how to synthesize placeholder
 A : Type,
 B : Type,
 x : A,
 x : B
 ⊢ A

The x : A is not accessible at the placeholder, so it probably shouldn't be shown.
Also related, given:
definition f {A : Type} (x : A) : A → A := λ x, _

Lean shows:
don't know how to synthesize placeholder
 A : Type,
 x x : A
 ⊢ A

The duplicate xs are confusing. Only one x should probably be shown.
See also this discussion.

Remark: we have a rename tactic, and it is useful to handle situations like the one above.
The tactic rename x y will rename the "most recent" x to y.
In the example above, it would change the goal to:
 A : Type,
 B : Type,
 x : A,
 y : B
 ⊢ A

here is a full example:
constant g {A B : Type} : A → B → A

definition f {A B : Type} (x : A) : B → A := λ x,
begin
  rename x y,
  exact g x y
end

@leodemoura Interesting! Thanks!
One thing to add: As I've noticed and Mario Carneiro pointed out in the mailing list discussion, the more recent x appears renamed (e.g. to x_1) in displayed implicit values and error messages. This adds a bit to the confusion.
Nevertheless, I guess having the rename tactic is reason enough to leave everything as it is. Feel free to close this issue if you want.

I am closing the issue. The x_1 name problem has been fixed in lean3.


[kube-monitoring] disable upstream alerts in favour of our curated ones

This PR disables the generic upstream alerts and pulls in our set of curated kubernetes alerting and aggregations rules.

We will disable the default rules from the plugin and deploy our independently. https://github.com/sapcc/helm-charts/pull/6587
I start the collaboration with our Kubernetes experts to include well-defined Kubernetes alerting rules in the plugin.


Configure a Meta account based on NYX-EI credentials and capable of operating webhook

Meta has created viable environment for webhook.
I want as a developer to be able to create a Meta developer account based on my NYX-EI credentials.

4 days


Missing chemical information for gold standard BGCs

Hi @skinnider,
The Zenodo data for the gold standard BGCs does not contain any metadata about the "true" compound produced by the BGCs apart from the names of the files. The scripts depend on a /data/platinum/all_mols.txt file that is missing from the repository, so it is not possible to run at the moment. Could you please share these files or indicate how to get them if they are available somewhere?
Thanks in advance.

Sorry @althonos, not sure why this wasn't included! I believe the attached should be the correct file.
all_mols.txt

Thank you! I managed to get most of these from one of the supplementary tables from the paper but it is much more practical to have it like this :smiley:


Instructions to build Powershell 6 for RHEL 6.0

I will like to know steps/instructions on how to build Powershell 6 for RHEL 6.0 from source.
Regards,
Alpesh

Here you go. Note that the distribution you're asking for is not supported by us, nor by .NET Core (our language), so you'll have to fudge a lot.


fix(modeling): use modern typescript imports

This PR updates V3 branch to fix a ton of type errors.  Typescript uses jsdoc for type hints, so they are related.
Changes include:

Use explicit file extensions when doing import type ... from './type.d.ts' files
Change import { Geom2 } to import type { Geom2 }. Why? Because modern typescript requires adding "type" in order to stay compatible with ES modules.
Always use capital Geom2, Geom3, Poly3, etc when used as a type
Change Number, Float, Integer to number
Change String to string
Remove unnecessary typedefs
Remove unnecessary namespaces
Add tsconfig.json file to configure typescript
Add global.d.ts with basic geometries and maths so that they can be used in jsdoc without importing explicitly

Note: this doesn't fix all the type errors, that's a bigger project. But this helps fix a lot of them. My goal is to have good autocomplete for JSCAD, and having good types really helps with that.
Sorry for the huge PR!
All Submissions:

[x] Have you followed the guidelines in our Contributing document?
[x] Have you checked to ensure there aren't other open Pull Requests for the same update/change?
[x] Does your submission pass tests?

nice work !

@platypii did you sleep the last few weeks?
i'm not really interested in reviewing 697 files. UGG
is there anything specific that you would like reviewed?

@platypii did you sleep the last few weeks?
i'm not really interested in reviewing 697 files. UGG
is there anything specific that you would like reviewed?

Sorry. I can try to split it up into multiple PRs if you prefer? Basically it's all just syntax changes on the types. Most of if was search-and-replace across all files. I tried to be pretty careful. So, if you want to approve, I'm happy to fix any bugs that come up in the future.


feat: better readme

Your project is awesome and it just needed a good README.md which I created for you..

Thank you, this is first PR I got for any of my repos :)

let's rewrite it together ;)


limited quantity of playlists and my work around ceased working

I have found that there is a limit to the playlist selector only allowing me to select from 50 of my playlists. I had previously made the workaround of pinning the playlist in my library when I wanted to play it so it would show up at the top of the true random playlist selector. it stopped updating the list of playlists when I do that even after I cleared cache, restarted browser, refreshed page ect
I recognize this isn't your most important project right now lol, this work of yours has served me well and I just can't imagine life without it. if you're able to iron out some wrinkles for me that would be incredible. thank you for your work tomer aberbach.

I should also mention that I too experience a problem from a previous post where if a playlist has a local file in it it just won't play the playlist at all

First, you have some A+ playlist names haha
I looked at my code and looks like I do only get the first 50 playlists... Maybe Spotify changed how they consider which playlists are the "first" 50 and that's why you're seeing this. Have you tried actually reordering the playlists by dragging them instead of pinning? Maybe that would make Spotify respect the order? Not sure honestly!
I wrote this code like 6 years ago when I was in college so it's not in the most ideal state truthfully. I've considered going back and rewriting/improving it so that it doesn't have all these issues, but haven't gotten around to it. I might try to do that in the near future, but no promises

I figured it was one of those things where the code is older and hard to work with lol.
it does seem that reordering the custom order on desktop makes playlists from long ago appear in the list on the mobile true randomizer. I wonder why the pinning playlists stopped working like it did before, when I change pinned playlist it takes like 24 hours to appear on the true randomizer instead of instant like it did befire. it's not a perfect workaround because it does mean you have to have access to the desktop app if you want to play a playlist that's not in your first 50. BUT for others with this problem it is a workaround which is better than none.
appreciate the help! your work will still continue to be life changing for my music listening even if you don't get around to updating it's code soon so no worries. thank you!

Alright I decided to take the time to do this. I just spent a few hours rewriting this with modern stuff. The new website will be at this URL:
https://spotify-true-random.tomeraberba.ch
I'm planning to set up a redirect from the old URL to this new one to carry over existing users. It should get all the playlists (not just the first 50) and it should skip local tracks (since there's no way to play those via the API) instead of erroring.
Can you try it out and let me know if things are working for you? Wanna confirm it's working for others before setting up the redirect

omg this is excellent I can see all my own playlists and they all play fine thank you very much. I have tested the link out and here are the overly in depth results of my testing:
play button did appear on the second line instead of the first on my mobile phone (could have been intentional or not either way it really does not matter in my opinion it's just in a difference I noticed)
I added a local file to my big favorite songs playlist and it does not error (tested it twice)
I made a new playlist with non local songs and that played fine.
I renamed a playlist and it updated just fine after a refresh
I did get an error where it would perpetually show that it was queueing a playlist, specifically if I made a new playlist and added both normal and local songs to it (tested that twice too). which is really strange because when I added local files to an older playlist earlier, it skips the local files like it's supposed to. so that just seems to be a very extremely minor error.
also it's still confused about pinned playlists, it's showing my top playlists as the ones I had pinned yesterday, not right now. worst case scenario that just makes the top four playlists not match up right, since I can see all the playlists now that doesn't bother ME.
I am personally VERY SATISFIED with the update and appreciate it greatly. I don't think any of the small differences above are anything to stress yourself out over at all (especially if they're a nightmare to fix), because the website is still an amazing tool that now works even better than before. I'm just reporting minor things you may want to be aware of is all.
thank you again!

omg this is excellent I can see all my own playlists and they all play fine thank you very much. I have tested the link out and here are the overly in depth results of my testing:
play button did appear on the second line instead of the first on my mobile phone (could have been intentional or not either way it really does not matter in my opinion it's just in a difference I noticed)

Yup, intentional. Felt it looked better to group it with the dropdown that it refers to.

I added a local file to my big favorite songs playlist and it does not error (tested it twice)
I made a new playlist with non local songs and that played fine.
I renamed a playlist and it updated just fine after a refresh
I did get an error where it would perpetually show that it was queueing a playlist, specifically if I made a new playlist and added both normal and local songs to it (tested that twice too). which is really strange because when I added local files to an older playlist earlier, it skips the local files like it's supposed to. so that just seems to be a very extremely minor error.

That's odd... I'll keep an eye out for errors, but if it generally works, then that sounds okay I guess 🤷

also it's still confused about pinned playlists, it's showing my top playlists as the ones I had pinned yesterday, not right now. worst case scenario that just makes the top four playlists not match up right, since I can see all the playlists now that doesn't bother ME.

Yeah, I think that's just a Spotify API issue that they return the playlists in a weird order. Not sure there's much that can be done about that on my end. Feel free to file an issue with Spotify if you care enough about that.

I am personally VERY SATISFIED with the update and appreciate it greatly. I don't think any of the small differences above are anything to stress yourself out over at all (especially if they're a nightmare to fix), because the website is still an amazing tool that now works even better than before. I'm just reporting minor things you may want to be aware of is all.
thank you again!

Glad to hear! I'll set up a redirect from the old website to the new one soon

Redirect is set up


Product business

A UnitTest was created in the [ProductImages] folder named ProductImageTest.cs
A UnitTest was created in the [ProductPriceList] folder named ProductPriceListTest.cs

Changes applied! plz check it.


Environment variables support for java8 container

As of today java8 runtime does not support apex environment variables. Proposing to update java plugin for apex (in a similar way to python) to enable env-vars injection to java container.

Environment variables will be written into properties file.
Then user request handler class will be substituted with proxy object
Proxy object will read environment properties file and then will call user request handler class with Java reflection API

I can implement change if this request will be supported.

@akranga We definitely accept PRs for functionality. I personally don't use the Java runtime but I'm sure others would appreciate it.

is anyone working on this?

I'm curious, why does this require a "plugin"? How does the execution language have a bearing on the setting of environment variables for an AWS Lambda?


Typo

['SchemaName') supposed to be 'SchemaName').

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

ID: 4f155bac-1a8d-184d-6fe8-8784ec0e86bb
Version Independent ID: 8943d45f-d066-0c23-a01d-e4ef4200f4e7
Content: Tutorial: Use Azure Database Migration Service for an online migration of RDS MySQL to Azure Database for MySQL
Content Source: articles/dms/tutorial-rds-mysql-server-azure-db-for-mysql-online.md
Service: dms
GitHub Login: @HJToland3
Microsoft Alias: jtoland

@ahamedembibe Thanks for your contribution, I've addressed this issue in PR: https://github.com/MicrosoftDocs/azure-docs-pr/pull/79789.
The changes should be live EOD or tomorrow. We will now close this issue. In future, if you see any minor bug fixes in the doc, feel free to click on the edit button and submit a PR to update. we love to take community contributions.


In Windows, links in stories are not clickable.

If there's a link in a story, clicking on it doesn't do anything. Similarly, if the story is a repost from another account then clicking on it should normally take you to the original account, but nothing happens.

Updates seem to have fixed this issue.


ModuleNotFoundError: No module named 'torchvision'

Lama cleaner doesn't run after installing from the downloaded Windows installer.
Attaching the errors I get. Please help!

Thank you for your report. This does not seem to be a problem with bitsandbytes, because bitsandbytes loads without any error. The error lies with another library. It seems the problem is related to not having installed the torchvision library. Please install it before you proceed.


Convert to Django 1.6's new testrunner

Reason: Reduces dependencies
Things to do:

Modify requirements
Modify settings
Demonstrate on working test project

closing this in favour of #196. I'm moving the things to do in that issue itself.


Manage TCP port for X Protocol

MySQL Sandbox does change the mysql port (3306) for the client and server to make it possible to use multiple sandboxes on the same host. However it does not change the port for the mysqlx port (33060).
Suggestion: Set the mysqlx port to 10 x classic_port (e.g. add a zero to the portnumber) or use the next free port after 33060 (first sandbox 33061, second 33062, etc).

This is only doable if the plugin is installed using plugin_load during initialization, because mysqlx_port is not dynamic.
The implementation for this feature requires a new option that will install the plugin and set the port at once.
A workaround to do it currently is
make_sandbox 5.7.14 -- -c plugin_load=mysqlx:mysqlx.so -c mysqlx_port=15714
On 28 Aug 2016 19:49 +0200, Daniël van Eeden<EMAIL_ADDRESS>wrote:

MySQL Sandbox does change the mysql port (3306) for the client and server to make it possible to use multiple sandboxes on the same host. However it does not change the port for the mysqlx port (33060).
Suggestion: Set the mysqlx port to 10 x classic_port (e.g. add a zero to the portnumber) or use the next free port after 33060 (first sandbox 33061, second 33062, etc).
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub (https://github.com/datacharmer/mysql-sandbox/issues/22), or mute the thread (https://github.com/notifications/unsubscribe-auth/AAVM67wyOvcKNfE57NIExduzWmy5ZOHGks5qkcoUgaJpZM4Ju-eZ).

The new option, with an updated mysqlsh script, will be published in a few days.
On 28 Aug 2016 19:49 +0200, Daniël van Eeden<EMAIL_ADDRESS>wrote:

MySQL Sandbox does change the mysql port (3306) for the client and server to make it possible to use multiple sandboxes on the same host. However it does not change the port for the mysqlx port (33060).
Suggestion: Set the mysqlx port to 10 x classic_port (e.g. add a zero to the portnumber) or use the next free port after 33060 (first sandbox 33061, second 33062, etc).
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub (https://github.com/datacharmer/mysql-sandbox/issues/22), or mute the thread (https://github.com/notifications/unsubscribe-auth/AAVM67wyOvcKNfE57NIExduzWmy5ZOHGks5qkcoUgaJpZM4Ju-eZ).

There are now (MySQL-Sandbox 3.2.00) two new arguments:
--plugin_mysqlx   (installs the plugin at startup)
and, optionally: --mysqlx_port=i (sets the wanted port. By default is sandbox port + 10,000)


Typo in the "Rebuild JupyterLab" command line for Windows

Description
In
https://docs.atoti.io/0.3.1/installation.html
6. Rebuild JupyterLab
The Windows command line should be
set NODE_OPTIONS=--max-old-space-size=4096 & jupyter lab build --dev-build=False --minimize=True
instead of
set NODE_OPTIONS=--max-old-space-size=4096 & jupyter lab build --dev-build False --minimize True
Reproduce
Context

Atoti version: 0.3.1
Python version: All
Operating System: Windows

Logs (if relevant)

Hi @mattiss ,
I'm going to close this because the command seems to work with all people that tried it on Windows.
Please comment again with the encountered message if you can and I'll check again.
Thanks!


predict stories

Hi ! I was wondering if the KAGS_vist.json is the file of generated stories. If not, could you please provide me the stories you generated by the model so i can use it for comparison. Thanks a lot !!!

Hi ! I was wondering if the KAGS_vist.json is the file of generated stories. If not, could you please provide me the stories you generated by the model so i can use it for comparison. Thanks a lot !!!
Sorry for my late reply. Yes, KAGS_vist.json is the file of generated stories on the VIST dataset. You can use it for fair comparison.


Ability to display local document

Your Welcome-Sheet looks awesome and works very well. I was wondering if you had any desire to add the ability to either display a document in a row or allow the button to open a local resource? My thought is for linking a local license file etc. I have an app that I would like to use your Welcome-Sheet, but the app has a requirement for working without any network/internet access and I want to have the ability to display a license from here.

Hey, thanks for your kind words and suggestions! I just added new branch with with two additional actions available for optional buttons: closures and custom views presented on top of Welcome Sheet. You set them as parameters during WelcomeSheetPage initialisation, just like you did with urls. I would suggest passing pdf view with your license to optionalButtonView: parameter when you create your page. After user taps the optional button it will be shown on screen.
WelcomeSheetPage(title: "Hello in my app!", rows: [
   // Rows
], optionalButtonTitle: "Terms and Conditions...", optionalButtonView: { DocumentView() })

I would love to hear your feedback and any new ideas. If everything works I will merge and make a release from it 🙂

Hi! Yes this work exactly how I envisioned it! Thank you for integrating my idea. Sorry it took so long to get back to you I went out of town for a weekend concert...
I will see if I can come up with other ideas/use cases and pass them on.
Thanks,
Matt.

Added with v1.2.0


Cloud Backup: Nextcloud users with UUID

With the current configuration options it is not possible to use Nextcloud with LDAP/AD user integration in its default configuration.
By default the Nextcloud LDAP user backend uses the UUID attribute to add users. This is probably true for other backends too. As a consequence the webdav path does not include the username but the UUID e.g. https://nextcloud.example.com/remote.php/dav/files/<UUID>
To Reproduce
Steps to reproduce the behavior:
Nextcloud installation with LDAP backend (probably others that use UUIDs for created users).

Setup Nextcloud backup with LDAP user

Relevant log files
Throws http 404.
config[12996]: {"url":"https:\/\/nextcloud.example.com\/remote.php\/dav\/files\/<username>\/","content_type":"application\/xml; charset=utf-8","http_code":404,"header_size":1102,"request_size":182,"filetime":-1,"ssl_verify_result":0,"redirect_count":0,"total_time":1.539546,"namelookup_time":0.347217,"connect_time":0.347459,"pretransfer_time":0.37096,"size_upload":0,"size_download":224,"speed_download":145,"speed_upload":0,"download_content_length":-1,"upload_content_length":-1,"starttransfer_time":1.539463,"redirect_time":0,"redirect_url":"","primary_ip":"<removed>","certinfo":[],"primary_port":443,"local_ip":"<removed>","local_port":60632}
Describe the solution you'd like
Maybe instead of entering the base path we could opt to enter the full webdav path. This would also solve #3988
Environment
OPNsense 20.1.5-amd64
FreeBSD 11.2-RELEASE-p18-HBSD
OpenSSL 1.1.1f 31 Mar 2020

I do not own an LDAP server so I cannot test / reproduce that. Is there an documented way to query the uuid from nextcloud?

I do not own an LDAP server so I cannot test / reproduce that. Is there an documented way to query the uuid from nextcloud?

Here's the explanation from the LDAP user backend.

Internal Username
By default the internal username will be created from the UUID attribute. It makes sure that the username is unique and characters do not need to be converted. The internal username has the restriction that only these characters are allowed: [ a-zA-Z0-9_.@- ]. Other characters are replaced with their ASCII correspondence or simply omitted. On collisions a number will be added/increased. The internal username is used to identify a user internally. It is also the default name for the user home folder. It is also a part of remote URLs, for instance for all *DAV services. With this setting, the default behavior can be overridden. Leave it empty for default behavior. Changes will have effect only on newly mapped (added) LDAP users.

So basically the Nextcloud username is the UUID. I could confirm this by entering the UUID into the "User Name" configuration in opnsense. After that I got a HTTP 401, which is correct since the UUID is not allowed as a login attribute in my case.

no, that means that it is derived from the UUID unless otherwise configured. So I need to login and query the real username using an API endpoint.
https://docs.nextcloud.com/server/12.0/admin_manual/configuration_user/user_auth_ldap.html

You mean something like this ? https://docs.nextcloud.com/server/15/developer_manual/client_apis/LoginFlow/index.html#obtaining-the-login-credentials
GET 'https://nextcloud.example.com/ocs/v1.php/cloud/user' -H "OCS-APIRequest: true"
<?xml version="1.0"?>
<ocs>
 <meta>
---buch of stuff---
 </meta>
 <data>
---This looks prominising---
  <enabled>1</enabled>
  <storageLocation>/vol/nextcloud_data/cf67c556-9012-22e7-9da5-5e4ba3d622e1</storageLocation>
  <id>cf67c556-9012-22e7-9da5-5e4ba3d622e1</id>
...
more stuff
 </data>
</ocs>

this looks useful, so I can use the id attribute to detect an external user.
what does the backend tag say?

this looks useful, so I can use the id attribute to detect an external user.
what does the backend tag say?

Yes the id tag contains the internal username.
This is the backend tag from my LDAP user.
<backend>LDAP</backend>

Normal users have Database as backend. I couldn't find a list of possible values but I expect there are a few more.

I just thought of a way you can test this without an LDAP server. By default Nextcloud users can login with their E-Mail. So instead of entering the username in opnsense enter the E-Mail and you would have basically the same situation where username != loginname and != webdav folder.

@StillLoading I fixed in in the linked PR. please apply it and respond if it works. For me with the email address it worked with local database.
Kind regards
Fabian

@fabianfrz Just tested it and it's working! Thanks!


bug(*): allow templates without logic to initialize and trigger

Closes https://github.com/accordproject/cicero/issues/597

Focuses on implementing cicero trigger and cicero initialization for contracts without logic. View the following issue for more details
https://github.com/accordproject/cicero/issues/597
Flags

Top be tested on templates without logic but present with formulae.

Screenshots or Video

Related Issues

Issue https://github.com/accordproject/cicero/issues/597

Author Checklist

[ ] Ensure you provide a DCO sign-off for your commits using the --signoff option of git commit.
[ ] Vital features and changes captured in unit and/or integration tests
[ ] Commits messages follow AP format
[ ] Extend the documentation, if necessary
[ ] Merging to master from fork:branchname

@jeromesimeon any updates on this?


return to sessions list after current is closed

fixes #106

Codecov Report

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

@@           Coverage Diff           @@
##           master     #112   +/-   ##
=======================================
  Coverage   61.05%   61.05%
=======================================
  Files           2        2
  Lines          95       95
=======================================
  Hits           58       58
  Misses         32       32
  Partials        5        5

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 73bc394...1cf2c84. Read the comment docs.

Thanks a lot!


Added config showing tabs always

Thanks for the PR!
I usually say no to adding options like this because I want to keep the number of settings low and offer sensible defaults that work for most situations. This makes it easier for new users to get started with Yazi, as they only have to learn a few settings.
In this case, I don't really see the need - when would you ever need to show just one tab? I think it only makes sense to display tabs when there's more than one to save some space.


The default buffer size of 0 usually isn't enough

At least not for my setup. Serial output gets cut off.

ing Zephyr OS build zeph90f4c3 *****
[00:00:00.004,974] <inf> mcuboot: Startng bootloader
n_type=0x1, copy_done=0x3, image_ok=0x3
[00:00:00.021,270] <inf> mcuboot: Scratch: magic=unset, swap_type=0x1, copy_done=0x3, image_ok=0x3
[00:00:00.030,609] <inf> mcuboot: Boot source: primary slot
[00:00:00.039,611] <inf> mcuboot: Swap type: none
[00:00:00.044,738] <err> mcuboot: Unable to find bootable im

(no more ouput)
* flashes fixed version *
***** Booting Zephyr OS build zephyr-v1.14.0-2240-g4ca54390f4c3 *****
[00:00:00.004,943] <inf> mcuboot: Starting bootloader
[00:00:00.011,383] <inf> mcuboot: Primary image: magic=unset, swap_type=0x1, copy_done=0x3, image_ok=0x3
[00:00:00.021,240] <inf> mcuboot: Scratch: magic=unset, swap_type=0x1, copy_done=0x3, image_ok=0x3
[00:00:00.030,578] <inf> mcuboot: Boot source: primary slot
[00:00:00.039,581] <inf> mcuboot: Swap type: none
[00:00:00.044,708] <err> mcuboot: Unable to find bootable image

Zephyr defines this configuration in range for <128, 65536>. something is bad with the setup on which problem with 0 size occurred.
This patch fix symptoms of setup problem someone have.

Don't have to set it to 128, that number is arbitrary.
... So it's the board's fault or something?

I wonder if this is related to the problem where we don't wait for the uart to drain before either starting the next image or halting on failure.

@Blacksilver42 With zephyr mainline it is not possible to even compile with smaller size (build will fail on Kconfig). What a version did you use?

@nvlsianpu The latest version straight from their github: https://github.com/zephyrproject-rtos/zephyr. I downloaded using West.

On smaller sizes it still happens sometimes.

It still happens occasionally on 128, which is the minimum valid size -- just a lot less.

Realizes it's impossible to set to zero *

Welp!


Quantiles fix

Fix for quantiles calculation.
Quantiles are now stable between passes over the data and between runs.
Updated unit tests to catch the error (eg. they pass now, but if you undo the fix, they will fail) Previously the unit tests didn't catch the error before because the default parallelism (of 1) was being used in the test harness, and that covers up bugs that are only exposed when the data is really distributed across multiple partitions.
In future, unit tests need to use non-trivial partitioning so they catch more bugs.

@rabarona
get yourself all up in this

@juliasantos
you too!

LGTM!


database crashed

Hello,
I am using modsec rules for my cpanel. Recently there was something weird happened, my server CPUload increases dramatically. I had similar issue with my another server in which after stopping modsec, server CPUload came to normal.
I stopped modsec rules from processing in this server too and the CPUload came down. When I visited "Modsecurity Configuration" from my whm then I see following error:

Error: The following exception has occurred: API failure: (XID ) DBD::mysql::db selectall_arrayref failed: Table './modsec/hits' is marked as crashed and should be repaired

Anyone have idea on how to solve this?
I am also amazed why server CPUload increases when I enable modsec rules.
CENTOS 6.7 x86_64
WHM 54.0 (build 16)
https://documentation.cpanel.net/display/CKB/OWASP+ModSecurity+CRS

forget it. I have repaired database using repair tool in WHM.


[Request] Port to run on the web using emscripten?

Any interest in targeting steven for web browsers using emscripten? Could be a possible viable alternative web-based client (background why I'm interested: been working on https://github.com/satoshinm/NetCraft/ which I ported from https://github.com/fogleman/Craft, written in pure C so it ported easily with emscripten, however, steven is much more fully-featured so it could be a good replacement).
Steven almost compiles with emscripten when passing --target asmjs-unknown-emscripten to cargo build, after applying https://github.com/Thinkofname/steven/pull/68 and updating a dependency https://github.com/satoshinm/steven/pull/1 but the main problems then seem to be:

OpenSSL: cross-compiling (or replacing? https://github.com/Thinkofname/steven/issues/67) for the web
Replacing download_vanilla - can't download arbitrary files (at least, I think that is the error I see in mangled C++ backtrace here: https://github.com/satoshinm/steven/pull/1#issuecomment-301297171)
Conditionally-compiling out other code as needed with #[cfg(target_os = "emscripten")]

OpenSSL: Minecraft uses cfb8 which is outdated and not supported in ring
Running the download locally and letting emscripten pack them should work
This makes heavy use of threads for building chunks, emscripten stubs them out unless you have SharedArrayBuffers enabled (which only firefox really supports)

Additionally the networking wouldn't work as you can't use tcp within a browser.
I should also add that I don't have the time to work on steven so I wont really be able to help if you want to continue trying this

Thanks for the response! Totally understand not having time to work further on steven, appreciate what you've done so far, this is really neat. Some notes, to myself or others if they are interested in continuing trying porting Steven to Emscripten:

CFB mode: not supported in rust-crypto either... may have to add it, or another possibility, remove the MC protocol crypto and use HTTPS to communicate back to a server endpoint
SharedArrayBuffer:  support is in-development on Chrome. However, although emscripten ostensibility supports it, in my testing adding -s USE_PTHREADS=1 to the emcc compile flags, it seems to never finish building (or takes unreasonably long). May need to be refactored to reduce use on threads. On the other hand, Craft compiled fine with tinycthreads? Not sure why (or does this explain the missing chunks? https://github.com/satoshinm/NetCraft/issues/75)
Networking: This one is tractable, in NetCraft/WebSandboxMC I use WebSockets to communicate back to a WebSocket server written in Netty which runs on the Bukkit server and relays commands to/from it. WebSandboxMC uses a custom protocol, interacting with parts of the Bukkit API, but in principle it should be possible to bridge WebSockets to the MC protocol, with the proxy running on the MC server. The proxy could also handle the nasty CFB crypto, rewrapping it in SSL (wss://, or without: ws://), and would probably also have to do something about authentication (since the web client would not be permitted to contact the 3rd party authentications servers).

@satoshinm I've made some progress on web support in my fork (using wasm32-unknown-unknown, not asmjs/wasm32-unknown-emscripten as in https://github.com/satoshinm/steven/pull/1):
https://user-images.githubusercontent.com/43691553/103248308-f436a300-491e-11eb-9278-514dae9e41db.mov
still a long ways to go, but its a start. https://github.com/iceiix/stevenarella/issues/446 🕸 Web support


adding ability to set ca

This commit adds the ability to set just the CA list, so we don't have to worry about the agent
@yields @f2prateek

LGTM

For future reference, we made this change so that we don't just set certificates on https.globalAgent - which only works with https urls and not http (primarily used in webhooks but also some integrations).


Possible regression in Julia 1.11 for throw block

The following code passes in 1.10, but flags an error in 1.11.0-rc1
using JET

function g(x)
    throw("Should JET ignore this? $x")
end

@test_opt unoptimize_throw_blocks=true g(1/3)

The error is:
[ Info: tracking Base
  ═════ 4 possible errors found ═════
  ┌ g(x::Float64) @ Main ./REPL[2]:2
  │┌ string(::String, ::Float64) @ Base ./strings/io.jl:189
  ││┌ print_to_string(::String, ::Float64) @ Base ./strings/io.jl:150
  │││┌ _unsafe_take!(io::IOBuffer) @ Base ./iobuffer.jl:504
  ││││┌ wrap(::Type{Array}, m::MemoryRef{UInt8}, l::Int64) @ Base ./array.jl:3099
  │││││ failed to optimize due to recursion: Base.wrap(::Type{Array}, ::MemoryRef{UInt8}, ::Int64)
  ││││└────────────────────
  │││┌ print_to_string(::String, ::Vararg{Any}) @ Base ./strings/io.jl:143
  ││││ runtime dispatch detected: Base._str_sizehint(%17::Any)::Int64
  │││└────────────────────
  │││┌ print_to_string(::String, ::Vararg{Any}) @ Base ./strings/io.jl:148
  ││││ runtime dispatch detected: print(%59::IOBuffer, %97::Any)::Any
  │││└────────────────────
  │││┌ string(::String, ::Int64, ::String, ::Tuple{Int64}, ::String, ::Int64, ::String, ::Int64, ::String) @ Base ./strings/io.jl:189
  ││││ failed to optimize due to recursion: string(::String, ::Int64, ::String, ::Tuple{Int64}, ::String, ::Int64, ::String, ::Int64, ::String)
  │││└────────────────────

My understanding of skip_unoptimized_throw_blocks = true (the default) is that it should ignore unoptimization in g, which is known to throw. Is that correct?

Yes, cases like this should be hidden by skip_unoptimized_throw_blocks=true, but due to the current implementation issue of report_opt, such cases can sometimes be reported. I will consider re-implementing report_opt.
However, regarding this particular issue, it is expected to be fixed in v1.11 rc2 and the release version without waiting for a fix on the JET side (xref: https://github.com/JuliaLang/julia/pull/54942).

Thanks! Is it possible to also make JET detect error as well as throw? In my actual use case, the exception originates with an error:
using JET

function g(x)
    error("Please ignore this: $x")
end

@test_opt unoptimize_throw_blocks=true g(1/3)

I created a follow-up issue #647 to track the remaining issue. Thanks again.


Question: Test cloudretro with TURN TCP instead of UDP

Like the title says, I would like to know if it is possible to deploy the CloudRetro example with STUNner TCP instead of UDP.
I was able to deploy it successfully on a corporate network (behind a firewall), but the UDP is very very flaky with lots of packet loss rendering the experience almost unplayable... I would like to give it a try on TCP and see if I could get an improved experience.
Could you point me on the steps I need to take to achieve such setup?

It worked perfectly


Adding JSON files for Prometheus queries for Runtimes.

Description

RHOAIENG-7353
RHOAIENG-3052
Prereq for #204
How Has This Been Tested?

I tested each query against a deployment of each runtime mentioned in the commits. To ensure the correct metric is returned.
Merge criteria:

[x] The commits are squashed in a cohesive manner and have meaningful messages.
[x] Testing instructions have been added in the PR body (for PRs involving changes that are not immediately obvious).
[x] The developer has manually tested the changes and verified that the changes work

@alexcreasy could you take a look at this PR and lgtm purely in terms of the structure of the json files and ensuring that it looks like what has been discussed.


Autoload constant modules

When using the examples in the README you will receive a uninitialized constant error.
To reproduce:
$ rails c
>> require 'opentok'
>> OpenTok::SessionPropertyConstants::P2P_PREFERENCE
NameError: uninitialized constant OpenTok::SessionPropertyConstants
>> OpenTok::RoleConstants::MODERATOR
NameError: uninitialized constant OpenTok::SessionPropertyConstants

Fixed this by autoloading the 2 modules that define these constants.

this one seems stale. constants have changed and the gem should behave just fine now. thanks and sorry for leaving this open for so long!


chore(SP-1747): using new gitleaks v8 configuration file format

This PR generated by SourceGraph updates the .gitleaks.toml file format to be compatible with gitleaks v8.
This repo is using ci-standard-checks@v1-beta which has been updated to use the latest gitleaks v8.
Created by Sourcegraph batch change david.salvador/update-gitleaks-config-all-repos.

:tada: This PR is included in version 1.21.3 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


[Feature] One tap: Modal UI

[Feature] One tap: Modal UI
Descripción

Se cambian los colores de One tap a los de Meli UI
Se muestra siempre la flechita para abrir el modal
Se arreglan los decimales del modal
Se agrega la cantidad de los items en el modal
Se arreglan los margenes del modal

Screenshots

Codecov Report

Merging #1057 into feature/onetap_complete will increase coverage by 0.01%.
The diff coverage is 22.22%.

@@                     Coverage Diff                     @@
##           feature/onetap_complete    #1057      +/-   ##
===========================================================
+ Coverage                     27.7%   27.72%   +0.01%
===========================================================
  Files                          321      321
  Lines                        15089    15089
  Branches                      1387     1385       -2
===========================================================
+ Hits                          4181     4183       +2
+ Misses                       10379    10377       -2
  Partials                       529      529

Impacted Files
Coverage Δ

...om/mercadopago/onetap/PaymentDetailInfoDialog.java
0% <ø> (ø)
:arrow_up:

...in/java/com/mercadopago/viewmodel/OneTapModel.java
0% <ø> (ø)
:arrow_up:

.../com/mercadopago/onetap/components/DetailItem.java
0% <0%> (ø)
:arrow_up:

...java/com/mercadopago/onetap/components/Amount.java
0% <0%> (ø)
:arrow_up:

...ain/java/com/mercadopago/GuessingCardActivity.java
0% <0%> (ø)
:arrow_up:

...n/java/com/mercadopago/customviews/MPTextView.java
58.33% <44.44%> (-2.96%)
:arrow_down:

Continue to review full report at Codecov.

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


Static image of type SVG is not shown

Describe the bug
Image Static layers of type SVG are not shown on map.
Based on Static Image example, I tried the same with an SVG image (see StackOverflow from ~2 years ago). Unfortunately, the SVG image is not shown.
To Reproduce
Steps to reproduce the behavior:

Use ImageStatic layer with SVG as illustrated in gist or Codepen.
Image is not shown on map.

Expected behavior
Similar to the PNG in the original example, I expect an SVG image to be shown as well.

The browser can't determine the size of the svg image: apparently it fallback to a 150x150 px file.
const img = new Image();
img.onload = (event) => {
  console.log(event.target.width, event.target.height);
}
img.src = 'https://upload.wikimedia.org/wikipedia/commons/f/fd/Ghostscript_Tiger.svg';

A imageSize can be passed to ol.source.ImageStatic to force a size, can you try this?

Looks like imageSize doesn't work as expected

It seems imageSize has no effect but it will work with an imageLoadFunction   https://stackoverflow.com/questions/52547179/openlayers-use-svg-as-layers-cant-work-on-ie11

The bug appear if the ratio between the image extent and the image size is the same, for example:

extent [0, 0, 900, 900], image size [900, 900]
extent [0, 0, 900, 900], image size [512, 512]

It "works" if the image size is set to [900, 899]

Thank you @fredj for your insight, I really appreciate it.
I would like to provide a fix for this, but I am new to the code. Do you happen to have an idea where things may be broken or where I could start looking, respectively?

@ianmcderp you should start with the handleImageChange function:
https://github.com/openlayers/openlayers/blob/v6.0.0-beta.9/src/ol/source/ImageStatic.js#L112-L136
The targetWidth != imageWidth test is wrong if the ratio  between the image extent and the image size is the same


Demo page not working

Demo page is not working properly.

Hi Andreu, which link are you following? I followed the link to http://htmlpreview.github.io/?http://rawgithub.com/mvindahl/angular-pan-zoom/master/demo/demo.dev.html and it seems to be working. If you followed the same link, my guess is that rawgithub had a brief hiccup.

Now is working, sure it was the server.
Thanks, you can close the issue :)

Looks like it's not working again

Remove the http://htmlpreview.github.io/? And just access http://rawgithub.com/mvindahl/angular-pan-zoom/master/demo/demo.dev.html

Thanks for pointing out the fix. Will get around to it tomorrow. And apologies for my overall inactivity.


Error when installing RDP: error code 0

Hi, after many hours trying to fix this by my self, I came upon this error:
" OpenKeyReadOnly error code 0"
Whether I try to run install.bat or uninstall.bat I got the same error:
https://i.imgur.com/2FYWkpu.jpg
If I go to the source code, i Find this:
`
procedure CheckInstall;
var
Code: DWORD;
TermServiceHost: String;
Reg: TRegistry;
begin
if Arch = 64 then
Reg := TRegistry.Create(KEY_WOW64_64KEY)
else
Reg := TRegistry.Create;
Reg.RootKey := HKEY_LOCAL_MACHINE;
if not Reg.OpenKeyReadOnly('\SYSTEM\CurrentControlSet\Services\TermService') then
begin
Reg.Free;
Code := GetLastError;
Writeln('[-] OpenKeyReadOnly error (code ', Code, ').');
Halt(Code);
end;
TermServiceHost := Reg.ReadString('ImagePath');
Reg.CloseKey;
if (Pos('svchost.exe', LowerCase(TermServiceHost)) = 0)
and (Pos('svchost -k', LowerCase(TermServiceHost)) = 0) then
begin
Reg.Free;
Writeln('[-] TermService is hosted in a custom application (BeTwin, etc.) - unsupported.');
Writeln('[*] ImagePath: "', TermServiceHost, '".');
Halt(ERROR_NOT_SUPPORTED);
end;
if not Reg.OpenKeyReadOnly('\SYSTEM\CurrentControlSet\Services\TermService\Parameters') then
begin
Reg.Free;
Code := GetLastError;
Writeln('[-] OpenKeyReadOnly error (code ', Code, ').');
Halt(Code);`
Do you have any idea what could that error be? I tried disabling nod 32+ firewall to no avail. I don't know what else to do

did you try running the .bat file as admin (elevated privilege level)?

ESET recently started to remove RDPWInst.exe, you are missing the TermService registry service after restarting your computer

exactly as you describe. eset removes rdp service. is there any solution to restore rdp?

ESET recently started to remove RDPWInst.exe, you are missing the TermService registry service after restarting your computer

I got exactly the same error (thanks nod). I even tried seting the entire rdp wrap as an exclusion but that didn't work either.

Before unpacking and install RDPWrap, I turned off the options in ESET:
Enable detection of potentially unwanted applications
Enable detection of potentially unsafe applications
Enable detection of suspicious applications
As for the absence of Remote Desktop Services, ESET deletes the registry entry: HKEY_LOCAL_MACHINE \ SYSTEM \ CurrentControlSet \ services \ TermService
You must add an entry. I found a script on the internet that added the RDP service. Also, I couldn't add exceptions in ESET

Exclusions ESET:
Threat: @NAME=Win32/RDPWrap.A@TYPE=ApplicUnsaf
Path: C:\RDPWrap-v1.6.2\RDPWInst.exe
Threat: @NAME=Win32/RDPWrap.A@TYPE=ApplicUnsaf
Path: *
Threat: @NAME=Win64/RDPWrap.A@TYPE=ApplicUnsaf
Path: C:\Program Files\RDP Wrapper\rdpwrap.dll
Threat: @NAME=Win64/RDPWrap.A@TYPE=ApplicUnsaf
Path: *

Before unpacking and install RDPWrap, I turned off the options in ESET:
Enable detection of potentially unwanted applications
Enable detection of potentially unsafe applications
Enable detection of suspicious applications
As for the absence of Remote Desktop Services, ESET deletes the registry entry: HKEY_LOCAL_MACHINE \ SYSTEM \ CurrentControlSet \ services \ TermService
You must add an entry. I found a script on the internet that added the RDP service. Also, I couldn't add exceptions in ESET

Hi,
Do you have the link for the script, or maybe the script it self

You use at your own risk. Make a copy of the registry before running the script. Script for Win 7 i found on the internet. Script for Win 10 is export from my registry. They look similar, but I preferred to separate scripts

You use at your own risk. Make a copy of the registry before running the script. Script for Win 7 i found on the internet. Script for Win 10 is export from my registry. They look similar, but I preferred to separate scripts
Remote_Desktop_Services_Script

@morelab
Thanks for script, I worked out my problem by installing that reg entry.
Reinstall RDP Wrapper and installed new terms.dll for my version of OS

@moreleb
thx for the work

You use at your own risk. Make a copy of the registry before running the script. Script for Win 7 i found on the internet. Script for Win 10 is export from my registry. They look similar, but I preferred to separate scripts
Remote_Desktop_Services_Script

@morelab Thanks for script, I worked out my problem by installing that reg entry. Reinstall RDP Wrapper and installed new terms.dll for my version of OS

You use at your own risk. Make a copy of the registry before running the script. Script for Win 7 i found on the internet. Script for Win 10 is export from my registry. They look similar, but I preferred to separate scripts
Remote_Desktop_Services_Script

Thank, This Work...

Not an RDPWrapper issue

You use at your own risk. Make a copy of the registry before running the script. Script for Win 7 i found on the internet. Script for Win 10 is export from my registry. They look similar, but I preferred to separate scripts
Remote_Desktop_Services_Script

Can anyone help me get this for windows server 2012 r2 9.63.9600.16384 trend micro alao wrecked my rdpwrap .


[SPARK-21791][SQL] ORC should support column names with dot

What changes were proposed in this pull request?
Currently, Apache Spark ORC data source doesn't support field names with dot properly. For a feature parity with Parquet, we had better support this. Since Apache ORC 1.4 supports this by #18953, this PR adds a test case to prevent a future regression.
PARQUET
scala> Seq(Some(1), None).toDF("col.dots").write.parquet("/tmp/parquet_dot")
scala> spark.read.parquet("/tmp/parquet_dot").show
+--------+
|col.dots|
+--------+
|       1|
|    null|
+--------+

ORC
scala> Seq(Some(1), None).toDF("col.dots").write.orc("/tmp/orc_dot")
scala> spark.read.orc("/tmp/orc_dot").show
org.apache.spark.sql.catalyst.parser.ParseException:
mismatched input '.' expecting ':'(line 1, pos 10)

== SQL ==
struct<col.dots:int>
----------^^^

How was this patch tested?
After merging #18953, I will enable this test case.

cc @cloud-fan , @gatorsmile , @hvanhovell , @sameeragarwal , @rxin .

Test build #80883 has started for PR 19004 at commit 5995bd5.

Test build #80883 has finished for PR 19004 at commit 5995bd5.

This patch passes all tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):
class OrcSourceSuite extends OrcSuite with SQLTestUtils

Merged build finished. Test PASSed.

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

Since this is a test case only PR to confirm that, I'll include this into https://github.com/apache/spark/pull/18953 to save committer's review efforts.

Also, this PR is included into https://github.com/apache/spark/pull/17980, too.


Update world

It is about time.

👏

Merged. The failing united tests were due to timing out on Travis. It probably makes sense to remove them entirely.


⚠ Bofrost.de has degraded performance

In 944ff8f, Bofrost.de (https://www.bofrost.de) experienced degraded performance:

HTTP code: 200
Response time: 1445 ms

Resolved: Bofrost.de performance has improved in cf008a8 after 8 minutes.


[BEAM-2520] add UDF/UDAF in BeamSql.query/simpleQuery

R: @xumingming @lukecwik
CC: @takidau @jbonofre

Changes Unknown when pulling 76cce073b88bc23301a7e71c3a4fe9405d23954c on XuMingmin:BEAM-2520 into ** on apache:DSL_SQL**.

Good point, however seems the method of builder()/toBuiler() was not accurate.

Seems different from auto-value's usage convention: https://github.com/google/auto/blob/master/value/userguide/builders.md

I quote:
public void testAnimal() {
  Animal dog = Animal.builder().setName("dog").setNumberOfLegs(4).build();
  assertEquals("dog", dog.name());
  assertEquals(4, dog.numberOfLegs());

  // You probably don't need to write assertions like these; just illustrating.
  assertTrue(
      Animal.builder().setName("dog").setNumberOfLegs(4).build().equals(dog));
  assertFalse(
      Animal.builder().setName("cat").setNumberOfLegs(4).build().equals(dog));
  assertFalse(
      Animal.builder().setName("dog").setNumberOfLegs(2).build().equals(dog));

  assertEquals("Animal{name=dog, numberOfLegs=4}", dog.toString());
}

--update withUdf/withUdaf methods;
interesting, the code of TextIO/KafkaIO is different from the example. I suppose abstract Builder toBuilder() is better here, to avoid return this in function withUdf/withUdaf.
@lukecwik  any comments?

Indeed interesting :)

Changes Unknown when pulling 2455dc429fd1ecddd48d04e81262a6c965e83792 on XuMingmin:BEAM-2520 into ** on apache:DSL_SQL**.

@takidau any suggestion which one is prefered in AutoValue?
static Builder builder() {
  return new AutoValue_BeamSql_QueryTransform.Builder();
}

OR
abstract Builder toBuilder();

Re builder vs toBuilder, perhaps I'm misunderstanding the question, but they serve different purposes, don't they? I do prefer defining a static builder() method to directly invoking the generated AutoValue_BeamSql_SimpleQueryTransform.Builder() constructor (thought we seem to have both patterns in the Beam codebase currently).

Thanks @takidau for your advise, updated builder and rebase to latest repository.

Changes Unknown when pulling 2284cdec400c3676f4f86b92830deb58f4d196e5 on XuMingmin:BEAM-2520 into ** on apache:DSL_SQL**.

Looks like there are build issues, can you please fix?
Also, regarding the change in SimpleQueryTransform.expand, my question was whether toBuilder().build() could be used in place of manually constructing a copy of this. It's both shorter, and more maintainable since the AutoValue generator will keep track of making sure new fields are also copied as the AutoValue object evolves over time.

rebased to fix the compile issue.
toBuilder().build() doesn't work here, as SimpleQueryTransform.expand returns QueryTransform, they've different interfaces.
Regarding to the interface of UDF, your concern is reasonable, created BEAM-2609 for further discussion.

Changes Unknown when pulling 2e0abdd5985b54f7775204ce178adceba1ab30a6 on XuMingmin:BEAM-2520 into ** on apache:DSL_SQL**.

Thanks for the explanation on the toBuilder question, and for opening the Jira. Merging.

Merged, feel free to close.


Jumping not always working correctly

With the following typescript snippet:
snippet log "console.log" A
console.log('${$2}', ${1})
$3
endsnippet

I am unable to jump out from console.log('', asdasd|) the cursor position where cursor is |. Instead it keeps using tab to add more spaces, and never get me in the quotes. Can you test if it is the same issue for you?

Thanks for the report. Could you check with the updated recommended mappings that were just merged? We will also soon make this easier for users so you don't have to update your config all the time.

Thanks for the report. Could you check with the updated recommended mappings that were just merged? We will also soon make this easier for users so you don't have to update your config all the time.

Unfortunately it still doesn't work. Did you try  using that snippet?

Yes I tried with that snippet and it worked for me. I am currently working on a PR that improves the mappings but don't know if I have time to finish it today. That should hopefully resolve your issues. I'll let you know once it's done!

Working on it here: #42

Could you update your config with the recommended mappings in the readme? I have created a separate module for this so users won't have to update their config manually.


refactor: Parameterized logging with constant log values

Use this link to re-run the recipe: https://app.moderne.io/recipes/builder/sgdN8FTlx
As discussed on https://github.com/AxonFramework/AxonFramework/issues/2594#issuecomment-1820854466

We now see a failure due to a behavioral change in LoggingInterceptor
-            logger.warn(format("[%s] execution failed:", message.getPayloadType().getSimpleName()), t);
+            logger.warn("[{}] execution failed:", message.getPayloadType().getSimpleName(), t);

[ERROR] Failures:
[ERROR]   LoggingInterceptorTest.handlerInterceptorWithFailedExecution:134
Argument(s) are different! Wanted:
extendedLogger.logIfEnabled(
    <any string>,
    WARN,
    isNull(),
    and(contains("StubMessage"), contains("failed")),
    java.lang.RuntimeException
);
-> at org.axonframework.messaging.interceptors.LoggingInterceptorTest.handlerInterceptorWithFailedExecution(LoggingInterceptorTest.java:134)
Actual invocations have different arguments:
extendedLogger.logIfEnabled(
    "org.apache.logging.slf4j.Log4jLogger",
    INFO,
    null,
    "Incoming message: [{}]",
    "StubMessage"
);
-> at org.apache.logging.slf4j.Log4jLogger.info(Log4jLogger.java:178)
extendedLogger.logIfEnabled(
    "org.apache.logging.slf4j.Log4jLogger",
    WARN,
    null,
    "[{}] execution failed:",
    "StubMessage",
    java.lang.RuntimeException
);
-> at org.apache.logging.slf4j.Log4jLogger.warn(Log4jLogger.java:243)

[INFO]
[ERROR] Tests run: 1975, Failures: 1, Errors: 0, Skipped: 0

The Java 11 build fails on
[ERROR]   TrackingEventProcessorTest.resetBeforeStartingPerformsANormalRun:1168 » ConditionTimeout Condition with alias 'Handle Events' didn't complete within 2 seconds 500 milliseconds because assertion condition defined as a lambda expression in org.axonframework.integrationtests.eventhandling.TrackingEventProcessorTest that uses int, intjava.util.List Actually handled [3] instead of expected [4] ==> expected: <4> but was: <3>.

The Java 17 build fails on
[ERROR]   SpringBootDockerComposeIntegrationTest.setUp:37 » NullPointer Cannot invoke "org.springframework.boot.docker.compose.core.DockerCliInspectResponse.hostConfig()" because "inspectResponse" is null

The Java 21 build has a suspicious failure probably unrelated to the changes here.
Caused by: java.io.IOException: Error while instrumenting org/axonframework/eventsourcing/eventstore/jpa/SnapshotEventEntry$HibernateProxy$DwNKPWeP with JaCoCo <IP_ADDRESS>204050719/5dcf34a.
	at org.jacoco.agent.rt.internal_b6258fc.core.instr.Instrumenter.instrumentError(Instrumenter.java:161)
	at org.jacoco.agent.rt.internal_b6258fc.core.instr.Instrumenter.instrument(Instrumenter.java:111)
	at org.jacoco.agent.rt.internal_b6258fc.CoverageTransformer.transform(CoverageTransformer.java:92)
	... 145 more
Caused by: java.lang.IllegalArgumentException: Unsupported class file major version 65

I'm thinking all of these changes are unrelated, especially since the Java 8 build passed. You might want to look at using Develocity (formerly Gradle Enterprise, also available for Maven) to get help identifying any flaky tests.

Jacoco likely needs a bump to 0.8.11 to support Java 21.

Hi Tim, the Java 17 build failure is due to an issue in Spring and/or Docker Compose. There were some API changes in docker-compose that caused SpringBoot DockerCompose to not work properly. The fix is expected to be released by the Spring team tomorrow.

Thanks once more for the effort @timtebeek!!


🤗 [REQUEST] - Code-290k-13B

Model introduction
Large Language Models (LLMs) are good with code generations. Sometimes they do make mistakes in code generation. How about if they can give detailed explanation along with the code. This is what I have tried over here. The base Llama-2 model was used for training purpose. It is trained on around 290000 set of codes. Each set having 2 conversations. Along with Python, Java, JavaScript, GO, C++, Rust, Ruby, Sql, MySql, R, Julia, Haskell, etc. code with detailed explanation is used for training purpose.
Entire dataset was trained on 4 x A100 80GB. For 3 epoch, training took 165 hours. This was trained on Llama-2 by Meta. This is a full fine tuned model.
This conversation is in Vicuna/ShareGPT format.
Model URL
https://huggingface.co/ajibawa-2023/Code-290k-13B
Data type
bfloat16
Additional instructions (Optional)
No response
Author
Yes
Security

[X] I confirm that the model is safe to run which does not contain any malicious code or content.

Integrity

[X] I confirm that the model comes from unique and original work and does not contain any plagiarism.

Thank you very much for evaluating my earlier models.


Level cluster Step/Move commands reverted to random value after transition

Problem
A Step command did not execute as expected.
The transition happens, but when reaching the desired value it fell back to the original one.
After examination of the level-control-server.cpp code it seems a variable storedLevel is the source.
This variable is part of the state for the LevelControl cluster in case an Effect is requested (07-5123-04 (ZigBee Cluster Library doc), section <IP_ADDRESS>.1.). The currentLevel is backed up there to restore after the effect has finished.
Backup point:
https://github.com/project-chip/connectedhomeip/blob/ed677f04ae8187098130f3701b5583f8db56a55e/src/app/clusters/level-control/level-control.cpp#L938
Restore point after transition:
https://github.com/project-chip/connectedhomeip/blob/ed677f04ae8187098130f3701b5583f8db56a55e/src/app/clusters/level-control/level-control.cpp#L260
An INVALID_STORED_LEVEL is set in other commands (MoveToLevel/...) to skip the revert step.
For example:
https://github.com/project-chip/connectedhomeip/blob/ed677f04ae8187098130f3701b5583f8db56a55e/src/app/clusters/level-control/level-control.cpp#L414
However, in case of Step/Move command this variable was not initialized to prevent this reverting/setting.
Change overview
Assigning INVALID_STORED_LEVEL to the storedLevel variable where missing.
Testing
Setup:

chip-device-ctrl on RPi4 (TE3 image) + qpg6100 lighting-application
Scenario:
factory reset - move through commissioning of device
Issue step command - `zcl LevelControl Step 101 1 0 stepMode=1 stepSize=64 transitionTime=20 optionMask=0 optionOverride=0'
Read out CurrentLevel value - zclread LevelControl CurrentLevel 101 1 0
Check logging

transition is printed for each step
which levels are set to light

Check LED behavior visually - not jumping back/erratic

@selissia could you take a look at this, please?

Done, approved.


The dash symbol in the widget id

There seems to be a bug with widgets that have a dash in their id.
I created a custom widget with the id design-preview and used it in a newly created content type. When I tried to save the content type, it gave me an error. Looking at the network panel of Chrome's dev tools revealed, that the server responded with an error: Incorrect widget. Id doesn't match the regex [A-Za-z0-9]. When I changed the id of the widget to design, I was finally able to save the content type.
Since some of the examples in this repo have a dash in their id, I suggest this symbol should be allowed in the widget id.

Duplicate of #86.


FancyZones zone behavior's icon missing

Microsoft PowerToys version
0.47.0
Running as admin

[X] Yes

Area(s) with issue?
FancyZones
Steps to reproduce

Open FancyZones' setting page and find Zone Behavior.

✔ Expected Behavior
Icon displayed.
❌ Actual Behavior
Icon missing.
Other Software

cc: @niels9001

Is this still an issue?

@franky920920 can you verify with 0.53 when it comes out

I can't repro this

@crutkas possibly because it's not really a bug. The icon is just a (meaningless) rectangle. Question is: what do we do with it? (Replace our just remove?)

for me it is not just a single rect (this is a 0.53 RC build)

Ohhh, in that case 🤐

Strange, this is the icon for me in .51.1 how's that possible?)

Jay, what is your OS?

Win 10, 21H2

@crutkas Can confirm this is still persisted in 0.53.1 on both Windows 10 21H2 and Windows 11.

weird, why does mine show this and yours doesn't

Maybe the font is not installed my system?

I think the font is just MDL2, but maybe a different version number?  Anyways, can we just replace/remove it already?

Looks different now. Closing as done.