Status view text message aligned left in Edge

Bug Report
Description

Status view message is not showing in center on Edge browser. Here is the screenshot:

Steps to Reproduce

Go to https://engineering.cerner.com/terra-ui/#/components/terra-status-view/status-view/status-view in MS Edge browser
Check the examples
You should see the message aligned improperly

Additional Context / Screenshots

Screenshot added in the description
Expected Behavior

The message should be in the center
Possible Solution

Adding style text-align: center to the div having class StatusView-module__message__xxxx fixes it.
Environment

Component Name and Version: Internal Application
Browser Name and Version: Edge
Node/npm Version: 8.9.2
Webpack Version: 4.27.1
Operating System and version (desktop or mobile): Desktop Windows 10

@ Mentions

None

@yusufali2205 @amichaelparker @bjankord are ya'll still seeing this? I'm not able to recreate this on Edge

I still see it. I'm on Edge Version 42.17134.1.0

Able to recreate on 42.17134.1.0. Edge has a CSS Properties bug with text-align. The center fallback value fails to register - instead, edge defaults to left.
To reiterate @bjankord, if there are no cases where we want non centered text, it makes sense to remove the CSS property (I vote we do this).

I need to check if this is used by consumer first before approving PR - I believe it is used.


fix site build

This should fix https://github.com/opensource-f2f/website/issues/60
cc @sxzz
The template will be parsed by the SSR code, thus we should carefully adjust its format.

It looks like the failure is still there.
https://github.com/opensource-f2f/episode/actions/runs/6438721159/job/17485555267


 All of the changes involve formatting of python code, removing useless variables, and straightening out imports. It does make the code much nicer to read. Responds to #232
I have kept from ModelDBSyncer import * statements. Since methods are being overridden there, this is necessary until we  get Monkey Patching working.
All tests either pass or (in two cases) fail as they did before.

@mpvartak - OK to merge? There aren't any functionality changes here... just code and import formatting.

Yep, looks good!


Fixed translation for the word Music

"Music" is an uncountable noun, there's no plural.

Nice fix! Thanks!


Multi architecture Docker build in github actions

Closes #86.

There's a fair bit in this PR - couldn't you have simplified?

There's a fair bit in this PR - couldn't you have simplified?

:laughing:


pubsub.aws.SNSSQS: Fixes Certification Test orkflow Error

Signed-off-by: Roberto J Rojas<EMAIL_ADDRESS>Description
Fixes for the following workflow error:
https://github.com/dapr/components-contrib/actions/runs/3906420476/jobs/6674590213

Thanks for addressing this as we discussed in Discord


Fire hydrants

Solves #5452.
Implements current emergency=fire_hydrant tagging scheme. Current scheme is based on approved Fire Hydrant Extensions (part 2) proposal.

Whoa this is a lot of stuff!   Can we trim it down to just the 5 or so most important and easiest fields that an untrained mapper could fill in?   We don't really add fields to iD that require specialized knowledge or training for someone to contribute.  (But things like where the hydrant is and the color of the cap seems fine)

Can we trim it down to just the 5 or so most important and easiest fields that an untrained mapper could fill in?
Sure,I will have a look which parts are useful and easy to map.
Is it enough to remove the unnecessary from the preset (fire_hydrant.json) or do I have to remove the
fields json-files also?

do I have to remove the fields json-files also?

yes, please remove all the fields that aren't being shown anywhere.


Delay parsing of date/time strings until needed

The current code would compare every string literal it crossed and tried
to coerce them to time literals if the looked like date/time strings.
The only time the TimeLiteral was used is when comparing to the the
'time' value in a where clause. This change moves the string parsing
code until we attempt to compare 'time' to a string, at which point we
know we need/want a TimeLiteral, and not just an ordinary string.
Fixes #6727
\cc @beckettsean @benbjohnson @jsternberg
Required for all non-trivial PRs

[x] Rebased/mergable
[x] Tests pass
[x] CHANGELOG.md updated

LGTM 👍

The code generally lgtm. 👍  I'm surprised it didn't take a little more code to change.
There seems to be a breaking test in the CLI though:
Unknown precision "". Please use rfc3339, h, m, s, ms, u or ns.
Visit https://enterprise.influxdata.com to register for updates, InfluxDB server management, and monitoring.
Connected to http://<IP_ADDRESS>:32988 version x.x
InfluxDB shell version: y.y
> panic: close of closed channel

goroutine 51 [running]:
panic(0x7d2b60, 0xc8201380e0)
	/usr/local/go/src/runtime/panic.go:481 +0x3e6
github.com/influxdata/influxdb/cmd/influx/cli_test.TestRunCLI.func1(0xc820108000)
	/root/go/src/github.com/influxdata/influxdb/cmd/influx/cli/cli_test.go:51 +0x28
created by github.com/influxdata/influxdb/cmd/influx/cli_test.TestRunCLI
	/root/go/src/github.com/influxdata/influxdb/cmd/influx/cli/cli_test.go:52 +0x257

I was also really surprised that the change was that small. I started by removing the regex matching, and making all strings StringLiterals, and then only one thing broke... so I put the conversion code there. :-)
The CLI test passed on rebuild, I'm guessing there's a race-encouraging test in there somewhere.

Thanks for the very quick fix!


Convert SampleAssetManager into manager plugin template

What
Convert SampleAssetManager into a template plugin that can be used as a starting point
Why
We intended it to be a "canonical sample implementation", but we haven't had the bandwidth to do this, and it has served more use as a template so far.

Closing addition of template and pending rearchitecture of BAL


add touchpad support

Type of Feature Request

[ ] Enhancement to an existing rumcake feature
[X] New feature unrelated to existing rumcake features
[ ] New driver for existing rumcake features
[ ] Support for a new MCU

Description
the eskarp keyboard uses a rust based firmware with a touchpad driver that could be ported to rumcake with a little effort. i may try getting this working for a personal project but wanted to put this out there for others to try as well.

Thanks for the issue! Mouse/pointer support is planned, but details about the API have yet to be decided. I'll probably get around to this once I finish up with some refactoring.

i have a tps-43, i'd be happy to help test stuff if my setup can support it. let me know if you need any help with testing or trying to document this feature.


[Enhancement] (nereids)implement showQueryProfileCommand in nereids

showQueryProfile command is supported in legacy planner but not in nereids(new doris fe planner). you need implement the following showQueryProfile command in nereids:
unsupportedshowQueryProfileStatement
SHOW QUERY PROFILE queryIdPath=STRING_LITERAL

This Command is deprecated. is it needed in nereids?

@starocean999 can you please assign this Issue to me. Thanks.


ParseError for pullRequestsForR when specifying "all" or "closed"

Pretty much as the title says. I'll do a bit more investigation to see if I can get a more meaningful insight into exactly which field is causing the issue.
ParseError "when expecting a Text, encountered Null instead"

compile with aeson >= 0.11, it will print the key then

Cheers, I'll give that a shot.
On 13 May 2016 12:01 pm, "Oleg Grenrus"<EMAIL_ADDRESS>wrote:

compile with aeson >= 0.11, it will print the key then
—
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub
https://github.com/phadej/github/issues/205#issuecomment-219013474

@phadej I don't get a parse error at all with aeson-<IP_ADDRESS>. Feel free to close if you think it's the best option. Thanks for such a quick reply.

Hmm, could you try with aeson-0.10, is it because of it?

I think your hmm there was warranted. I seem to have made a mistake, it is still failing with aeson-<IP_ADDRESS>.
Error in $[14].body: failed to parse field body: expected Text, encountered Null"

I decided to use curl to see if there was anything obvious.
curl -i 'https://api.github.com/repos/thoughtbot/paperclip/pulls?per_page=100&state=closed' > test.json

I saved the results here https://gist.github.com/filib/d42aea2fa194fcb27966f756c67d6ea6 but I couldn't find any fields called body that were null. Any ideas?

The failing result is probably not on the first page. Let me see

I was able to reproduce (got the same Error in $[14].body). Now it's fixed.

Why does this kind of thing happen? Is it because the Github API schema isn't always correct or is it because there is a bug in their API?

Also, thank-you, your responses were very quick.

There are no schema, API is specified by example, so we have to learn details empirically :(

That must be incredibly frustrating - my sympathies :(


Install flutter_inappwebview: ^2.1.0+1 Android project crashes

Environment
Android
Flutter version:
Flutter:1.12.13+hotfix.5
Android version:
Android 9.0
Description
After installing flutter_inappwebview: ^2.1.0+1, the project crashed when running. After debugging on the simulator, an error was reported as "Could not update files on device: HttpException: Connection closed before full header was received". After several versions were changed, no error was reported But it still collapses

Thanks very much @wwwdata and @polilluminato
However I am getting
<EMAIL_ADDRESS>Permission denied (publickey).
fatal: Could not read from remote repository.

Please make sure you have the correct access rights
and the repository exists.

I'm able to clone the repo from the command line, although I am asked for my passphrase. I can also download a ZIP or clone from github. But the seamless install as @polilluminato describes is not successful.
I'm on Windows.

Thanks very much @wwwdata and @polilluminato
...
I'm able to clone the repo from the command line, although I am asked for my passphrase. I can also download a ZIP or clone from github. But the seamless install as @polilluminato describes is not successful.
I'm on Windows.

Try with
url: https://github.com/pichillilorenzo/flutter_inappwebview.git

instead of
url: git@github.com:pichillilorenzo/flutter_inappwebview.git

I read on other repos that some Windows users fix the issue in this way, I'm on Linux so I cannot try but maybe It works for you.

Thanks again @polilluminato , that did work. Should have thought of it myself as https:// is how we download ZIP or clone the repo anyway. For future viewers, the full entry in pubspec.yaml is
  flutter_inappwebview:
    git:
      url: https://github.com/pichillilorenzo/flutter_inappwebview.git

It is also possible to clone or download a ZIP (then unzip) and use the local copy:
  flutter_inappwebview:
    path: C:/path/to/this/repo/flutter_inappwebview

(note that backslashes, or double-backslashes, don't work)
One final comment: if simply adding this package normally will automatically crash the app (which is why we're all here), a link to this issue should be highlighted on the front-page README.


[Identity] Version changes after the releases

New PR, smaller than the big one I made: https://github.com/Azure/azure-sdk-for-js/pull/17580
This PR:

Changes the Identity package versions to the next beta (meaning, from 1.0.0-beta.2 to 1.0.0-beta.3 and to 2.0.0-beta.6 t o 2.0.0-beta.7).
Sets some packages to use 2.0.0-beta.7 of Identity. Namely: The Identity package, the Key Vault packages and the Monitor Query (I’ve asked @maorleger and @KarishmaGhiya for permission 😄).

1.0.0-beta.2 to 1.0.0-beta.3 and to 2.0.0-beta.6 t o 2.0.0-beta.7

Did the automation not do this for you?

@ramya-rao-a Identity has been an issue for the automation process while Identity has been in beta, however this month I didn’t even see the pull request 🤔


result.animate_nodal_displacement() crashes

I've run a modal analysis, which I'd like to postprocess.
result = mapdl.result
result.animate_nodal_displacement(1)

does create an animation of the first mode

but when closing the plot window, the following error is returned:

I'll paste the complete error message in comment, but I don't know how to address this.

Transferred to pymapdl reader because the error is in that library.


How to pass private config vars to RTD?

Details

Read the Docs project URL: https://ringcentral-api-docs.readthedocs.io/en/bootstrap
Read the Docs username (if applicable): byrnereese

Problem Description
I recently created a mkdocs plugin (https://github.com/byrnereese/mkdocs-git-committers-plugin) that will display a list of contributors associated with a documentation page. It used the Github API which has aggressive API rate limits unless you are auth'ed with a token.
So I created a token, and added it to my docs mkdocs.yml file under the plugin config. This is obviously far from ideal.
Is there a way to pass these types of values in through an ENV (environment variable) to RTD so that I do not need to reveal these auth tokens publicly?

Currently, there isn't, but it's on plans https://github.com/rtfd/readthedocs.org/pull/4899


webgl gl.readPixels works in dart2js but not in dartium vm

This issue was originally filed by<EMAIL_ADDRESS>
What steps will reproduce the problem?

create a web application with the attached read_pixel.dart
run with dart2js -- click on the grey canvas -- it prints the correct pixel color: readPixels: x=14 y=384 color=[128, 128, 128, 255]
run in dartium -- click on the grey canvas -- it prints the WRONG pixel color: readPixels: x=14 y=384 color=[0, 0, 0, 0]

What is the expected output?
gl.readPixels should yield correct pixel color:
readPixels: x=14 y=384 color=[128, 128, 128, 255]
What do you see instead?
gl.readPixels yielding WRONG black pixel color:
readPixels: x=14 y=384 color=[0, 0, 0, 0]
What version of the product are you using?
Dart Editor version 0.5.20_r24275
Dart SDK version <IP_ADDRESS>_r24275
On what operating system?
Windows 7 on 64 bit

Attachment:
read_pixel.dart (1.59 KB)

It seems that this bug is back !
What steps will reproduce the problem?

create a web application with the attached read_pixel.dart
run with dart2js -- click on the grey canvas -- it prints the correct pixel color: readPixels: x=14 y=384 color=[128, 128, 128, 255]
run in dartium -- click on the grey canvas -- it prints the WRONG pixel color: readPixels: x=14 y=384 color=[0, 0, 0, 0]

What is the expected output?
gl.readPixels should yield correct pixel color:
readPixels: x=14 y=384 color=[128, 128, 128, 255]
What do you see instead?
gl.readPixels yielding WRONG black pixel color:
readPixels: x=14 y=384 color=[0, 0, 0, 0]
What version of the product are you using?
Dart VM version: 1.13.1 (Thu Dec 17 14:42:19 2015) on "linux_x64"
Dartium Version 39.0.2171.0 (64-bit)
On what operating system?
Ubuntu 14.04

I expect this is related to the switch to a JS interop based DOM for Dartium.
Do we have any tests for this issue?

@jacob314
This issue can be tested using the code provided on first post by<EMAIL_ADDRESS>Tested on dartium 1.15.

Is there any news on that issue ?
Shouldn't it be reopened ?

This still seems to be broken - please re-open.
readPixels is an invaluable debugging aid.

The underlying problem is that the typed data doesn't get modified. I assume that what's happening internally is that a copy is being made and that's the one that gets the new data.

So yes, there's something going on with copying in the internals of Dartium. There isn't a quick fix, but you can work around it with a hack like the following, calling the JS version directly.
var _jsReadPixels;
get jsReadPixels {
  if (_jsReadPixels != null) return _jsReadPixels;
  js.context['eval'].apply(['function readPixelsDartium_(gl, x, y) { '
      '  var pixels = new Uint8Array(4); '
      '  gl.readPixels(x, y, 1, 1, gl.RGBA, gl.UNSIGNED_BYTE, pixels); '
      '  return pixels; '
      '}'
  ]);
  _jsReadPixels = js.context['readPixelsDartium_'];
  return _jsReadPixels;
}

readPixelsDartium(RenderingContext gl, x, y) {
  // If we're in dart2js, just call the method.
  if (1.0 is int) {
    Uint8List color = new Uint8List(4);
    gl.readPixels(
        x,
        y,
        1,
        1,
        RenderingContext.RGBA,
        RenderingContext.UNSIGNED_BYTE,
        color);
    return color;
  }
  var jsGl = new js.JsObject.fromBrowserObject(gl);
  return jsReadPixels.apply([jsGl, x, y]);
}

readPixels stopped working after Dartium <IP_ADDRESS> (this time around).

Any news about this problem ?
It should be re-opened...
Still got the problem...
Dart VM version: 1.20.1 (Wed Oct 12 15:07:45 2016) on "windows_x64"
Chromium Version 45.0.2454.104

Should work in Dartium 50 (uses JSInterop same as JS / dart2js).

Check the example in the first post, it doesn't work for me at least.
Chromium Version 50.0.2661.108

Do you have proof that it works for you @terrylucas?
Someone other than me that have tested the example in the first post?
I think this issue needs to be re-opened... yet again :/

I have a similar example using webgl2 and that does not work either.
The js version built with ddc is also broken.

Robert webgl readPixels works in DDC and dart2js.
Webgl2 might be a different issue.

Fixed the problem that's existed since Dartium 45 (ArrayBufferView not constructed when calling Javascript code) fixed in CL.

Fixed with commit 1265b4c6742487c025ebae97852d555cfd9d6522 (should be in 1.25 dev 2).

Fix in Dart 1.25.dev.6.0

This works beautifully now in WebGL2
Thanks Terry!

Hello, I 've still got the problem in webgl1...
Does anyone else in the same case ?
Dart VM version: 1.25.0-dev.15.0 (Tue Aug 29 16:28:12 2017) on "windows_x64"
chromium Version 50.0.2661.108

Hi Jerome,
just to clarify: when you say
in chrome : readPixels: x=441 y=255 color=[128, 128, 128, 255]
you are referring to the results of dart code compiled with dart2js and then run in chrome, right?

Yes, Exactly


Enum support

Hi! I'm working on a little project and found an idea for improvement. Currently envied doesn't support using enums as an env type. Here is my use case:
I want to make an app env type strictly typed instead of any string but build envied gets me an error
Envied can only handle types such as `int`, `double`, `num`, `bool` and `String`. Type `AppEnv` is not one of them.

My code
// app_env.dart

enum AppEnv {
  staging,
  production;

  static AppEnv? fromString(String string) {
    return AppEnv.values.firstWhere((value) => value.name == string);
  }

  static AppEnv fromStringOrThrow(String string) {
    AppEnv? value =  AppEnv.fromString(string);
    assert(value != null, 'Unknown app env');
    return value!;
  }

  bool get isProduction => this == AppEnv.production;
  bool get isString => this == AppEnv.staging;
}

// env.dart
import 'package:envied/envied.dart';

import 'app_env.dart';

part 'env.g.dart';

@Envied(path: '.env')
abstract class Env {
  @EnviedField(varName: 'APP_HOST')
  static const appHost = _Env.appHost;

  @EnviedField(varName: 'APP_ENV')
  static final appEnv = AppEnv.fromStringOrThrow(_Env.appEnv);

  static String get _apiPrefix => appEnv.isProduction ? '' : appEnv.name;
  static String get apiEndpoint => 'https://$_apiPrefix.$appHost/graphql';
}

As a workaround, i can do a private field with a string value but it would nice to have native enum support.
@EnviedField(varName: 'APP_ENV')
static const _appEnv = _Env._appEnv;
static final appEnv = AppEnv.fromStringOrThrow(_appEnv);

This one could be a bit more complicated. Also, we'd probably want to use byName to lookup the values.

@TArch64 can you please test #73 by directly referencing the PR, i.e.
dependency_overrides:
  envied:
    git:
      url: https://github.com/petercinibulk/envied
      ref: feat/enum
      path: packages/envied/
  envied_generator:
    git:
      url: https://github.com/petercinibulk/envied
      ref: feat/enum
      path: packages/envied_generator/


Using hugo's native syntax highlighting

Dear,
Now we can activate Hugo's native syntax highlighting rather using highlight.js
Hugo color my code before highlight.js render it, then highlight.js do again.
I did this with Hugo 0.63.0
I disabled it by replacing static/js/highlight.min.js as
function EmptyHljs() {};
EmptyHljs.prototype.initHighlightingOnLoad = function(){};

But it looks weird, isn't it?
Thanks.

Would you be willing to convert this into a configurable feature in the theme? We would not be able to drop highlight.js entirely as it will affect people's deployed websites.
If this were a boolean config item in the config.toml...

true: don't include the <script> for highlight.js and include your new code block
false (default): keep behaviour as current

It'd be a great feature! (And thanks for the theme, already awesome!)

What's complicated is that one would also need to remove part of the theme CSS https://github.com/matcornic/hugo-theme-learn/issues/187#issuecomment-461173989
Documenting what a custom CSS should contain (Chroma stylesheet as generated by Hugo if using classes; in all cases overrides for e.g. code colour to go with the Chroma theme so that code isn't white on white background) would be useful and makes it a not small task I suppose. 🤔


glfwGetCurrentContext() returns NULL, although a context has been set as current.

Short:

I'll let this image here itself explain the issue. Is there any scenario where this would happen?
And I can confirm that engine->window.window is a valid GLFWwindow*, and is the one I intend to use.
Now for context:
I am working on a game engine, in which the core part would initialize GLFW, create a window, and handle all systems. It would then load components from a DLL at runtime, one of which is supposed to render graphics onto the same context. So the code above is inside that dll.
If I try to use gladLoadGLLoader((GLADloadproc)glfwGetProcAddress) (inside the dll), GLFW errors out saying
GL::ERROR::65538: Cannot query entry point without a current OpenGL or OpenGL ES context

and glad fails to initialize.
I would appreciate some help on this, thank you.
Extra note: The only possibility that I can think of, is that glfwMakeContextCurrent() somehow sees the GLFWwindow* object as invalid and sets it as NULL instead.

Seems like GLFW loses its context in the dll, from some testing.

Late but, incase anyone comes across this: linking to GLFW as a shared library, and making the Engine and the Dll link to it the same way fixed the issue.

Late but, incase anyone comes across this: linking to GLFW as a shared library, and making the Engine and the Dll link to it the same way fixed the issue.

Thanks for your note, it helps a lot for me.
Here is an interesting thing. I have a shared library built both for linux(.so) and windows(.dll).
On linux, it works fine when my library linking GLFW as a static library, and calling "gladLoadGLLoader((GLADloadproc)glfwGetProcAddress)" to load OpenGL functions.
However, on windows this method is not working. The library has to link GLFW as a shared library. Maybe it has something to do with the difference between dll and so.


Add P784 Parking (Moores Cancer Center)

Integrate Visionful parking data for lot P784 (Moores Cancer Center). URL is /beta/parking/784/status

Completed in parking-availability

This was deployed and verified last week, thanks a6wu. Closing


Fix radio in repeatable group

https://github.com/medplum/medplum/issues/2886
The radio group act erratic when there are duplicate ones with the same name - by adding the groupSequence prop this get's rid of the erratic behavior
https://github.com/medplum/medplum/assets/6913745/d14b193a-ccd3-469d-855e-84a428f1b209

coverage: 94.093%. remained the same when pulling 5e96966630b6b066b3666f1ca664da3fcd4b910d on james-radio-repeatable into 5042d692af903b873e2019503de03f039b732c3d on main.


⚠ Bofrost.at has degraded performance

In a5267e6, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 2152 ms

Resolved: Bofrost.at performance has improved in 393025c after 6 minutes.


How to build a Meme Generator with Vanilla JavaScript, Skeleton and the HTML 5 Canvas

Article checklist
Attention
Our Peer Reviewer and Content Moderator teams do NOT provide any revisions services. All revisions and edits should be completed on your own forked repo (as to not take up room in the queue) in order for our team to review them all in a timely manner.
ANY ARTICLE SUBMITTED WITH GLARING ERRORS WILL BE IMMEDIATELY CLOSED.

As a rule of thumb - please be sure to only submit articles (pull requests) that are fully polished and ready to be published. Be sure to go through our resources documents for extra and 3rd party (vetted) resources to help improve overall technical writing.

Pre-submission checks

Please ensure that you have only one open topic suggestion (issue) + in-review article (linked pull request) at a time. This will ensure that we complete the review process in a timely manner from inception to publishing

If you have not already, please go over our Resources Page for more tips and tricks on how to improve your overall technical writing so reviews are swifter, increase the chance of a payout and provide more value to our readers.

For first-time contributors and for more details on our submission guidelines, see our Contributing Guidelines.

Formatting and structure checks

[ ] Have you used the correct folder and file structure? - see Contribution Guidelines

[ ] Is your article properly formatted in Markdown? - see Github Markdown guide

[ ] Have you used our preferred heading structure? - H3 (###) and up

[ ] Does your article follow any of the suggested structure formats? - see example formats

[ ] Is your article over 750 words? - Tool: Wordcounter.net

[ ] Is the article you are submitting an in-depth and unique article? Does it go beyond what is in the official docs and what is covered in other blog sites. See these articles as examples.

[ ] Does your article provide enough value and detail about your topic? - Articles should be clear, accurate, and fully explained.

[ ] Can your article be understood by beginners? - Assume the audience is smart but has no prior exposure to the common terminology in your article.

[ ] Have you included a hero image that is 600x400 pixels, under 300KB in size, and have the copyright to use it? - Tools: Pixlr Image Editor for resizing and Unsplash for Creative Commons images

Grammar & plagiarism checks

[ ] Have you spellchecked and grammar checked your article? - Tools: Write&Improve - ProWritingAid - Grammarly
[ ] Please place your article through a 3rd party plagiarism checker. We suggest using Quetext, this tool is free and has a daily limit. We typically accept articles with 10% or less.
[ ] Have you checked your article for readability? - Tool: Hemmingway
[ ] Have you added sources for quotes and images that aren't yours?

Technical checks

[ ] Are your code snippets properly formatted for syntax highlighting - see Syntax guide
[ ] Have you checked your code runs correctly and you've highlighted all necessary dependencies for installation?
[ ] Are the software programs and packages you're highlighting in your article up to date, using current versions and not deprecated?

Topic suggestion this closes

Remove the backticks and add the issue number below to link and close your Topic Suggestion (issue) when your article has been published (PR has been merged). See this video for more details.

This closes #issue-number

Finally, delete the article checklist notes in blockquotes and submit your PR. We look forward to reviewing your article.

@hectorkambow and @ahmadmardeni1 I would like to review this article

Go ahead. @Jeff-mwangi

Upon running your article through our 3rd party plagiarism checker it seemed to raise a few flags and the % was higher than we typically accept.
Please see attached PDF - and revisit the article to ensure we are contributing wholly unique and original content.
Be sure to see our resources page to see more info on plagiarism and what is considered as such.

Upon running your article through our 3rd party plagiarism checker it seemed to raise a few flags and the % was higher than we typically accept. @Nomzy-kush
Please see attached PDF - and revisit the article to ensure we are contributing wholly unique and original content.
Be sure to see our resources page to see more info on plagiarism and what is considered as such.
How to build a Meme Generator with Vanilla JavaScript, Skeleton and the HTML 5 Canvas#6172

Hey Jeff,

Upon running the draft through several Plagiarism checkers, I’ve only seen
little to no plagiarism. I was wondering if you can send me the link to the
Plagiarism report you got.
Thanks,
Onome
Message ID:

<section-engineering-education/engineering-education/pull/6172/c1020905879
@github.com>

Hello @ahmadmardeni1 you can go ahead to recheck this draft again for plagiarism.

Plagiarism check done ✅ @Nomzy-kush

Hello @Nomzy-kush Consider doing all the changes first and then commit them once.

Hey @Jeff-mwangi I have made all the necessary changes you requested.

@Jeff-mwangi
If any more edits are needed, we will be closing this PR 👍

@jeff-mwangi any updates on this?

@Jeff-mwangi any updates on this?

Working on tech review

Please link your topic suggestion so we can accurately pay out if this article is published

Please link your topic suggestion so we can accurately pay out if this article is published

🤔
Not sure how the article topic is connected to the payout (unless im misunderstanding) - important step nonetheless

@Jeff-mwangi any updates on this?

Working on tech review

Thank you - lets try to keep our reviews (wait time to a minimum if possible - we understand things come up but) to be fair to our authors and other in the queue.

@Jeff-mwangi
If any other edits are needed - we will be closing the PR - this has underwent an extensive review already 🚀

Please link your topic suggestion so we can accurately pay out if this article is published

This fixes #5765

@ahmadmardeni1 This article is now ready for final review.
cc: @hectorkambow

Hey @hectorkambow and @louisefindlay23, isn't this PR ready to be merged?

jumping on this one

@section-engineering-education/content-moderator-team
We should have pushed this article along as well. 👀


aflplusplus: 2.59c -> 2.64c

Motivation for this change
Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[ ] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[ ] Fits CONTRIBUTING.md.

cc: @risicle

Neat, a toy to play with over the weekend...

Oh we should probably change the fetchFromGitHub owner to AFLplusplus now they've got an organization, and they also seem to have an actual homepage @ https://aflplus.plus/

I've got qemu building in 229f58b6506871b95ca558048178d1d6f667368c
Now I'm getting a failure building unsigaction32...

Oh but it seems unsigaction32's build is broken anyway https://github.com/AFLplusplus/AFLplusplus/commit/079fdbf9bc5be1adba19e4bd08be965bd4dd79dc#commitcomment-38428357 so we'll just have to skip it.

Ok and I've got the tests working again in 5fee991c8dbb059c55165f1a26647ae2667e98c8
I also added cmocka to get the "unit tests" working, and they do work - not sure of the value though...?

Some discussion points:

Not sure how to commit this while acknowledging your changes. Is mentioning you in the commit ok or how is this typically done? If I squash I'm the only author, afaik.
Should the clang_9 and llvm_9 variables be defined in the top-level/all-packages.nix file or is it ok as-is? The advantage of doing it in the top-level could be that someone can override them once clang-10 and llvm-10 are out of RC phase, I think. At least, that's what I think the top-level file does.

Don't worry about acknowledging my commits, squash them, actually probably worth a mention to spread the blame when it all blows up ;)
If there is a canonical answer to this, I don't know it. I guess it depends to what extent we find it works with other llvm versions (I haven't tried others, I have to compile llvm enough as it is...). e.g. if we find it blows up with anything other than llvm_9, there's little point in giving users that knob to twiddle.

Hmm, I can't get qemu_mode/libcompcov to build on i686. Which is annoying because the wine mode only works in 32bit qemu mode.

Seems to be possible to use an older llvm/clang version, but I don't think that's too useful for most people. Let's just leave them out of the configurable options.

Maybe you should create an issue upstream, if you think the issue lies there.

I've managed to get unsigaction building acceptably again by applying the fix that's already in master:
    patches = [
      (fetchpatch {
        url = "https://github.com/AFLplusplus/AFLplusplus/commit/5b9928f1a9d4b017ea04365ca8b522fde71236eb.patch";
        sha256 = "1m4w9w4jaxb2mjkwvr6r4qa2j5cdzzpchjphpwd95861h0zvb6hh";
      })
    ];

This also fixes build of unsigaction on i686 for me.

I've applied the patch you sent in. It seems to be working ok, but I'm on a 64-bit system. Should the unsigaction32.so be built on a 64-bit system? It doesn't build for me. Only unsigaction64.so seems to build for me.

I find that unsigaction32 only builds on 32bit, unsigaction64 only on 64bit, and you can test i686 easily because it's all available under pkgsi686Linux.aflplusplus.
However, I can't get even a straightforward afl-clang-fast build to work for me:
[-] PROGRAM ABORT : Oops, failed to execute 'clang' - check your PATH
             Location : main(), afl-clang-fast.c:846

This is when I afl-build yajl by adding
  AFL_HARDEN="1";
  AFL_LLVM_LAF_SPLIT_SWITCHES="1";
  AFL_LLVM_LAF_TRANSFORM_COMPARES="1";
  AFL_LLVM_LAF_SPLIT_COMPARES="1";
  preConfigure = ''
    export CC=${aflplusplus}/bin/afl-clang-fast
  '';

to its attrs.

I'm pretty sure were still going to need to bind the afl tools to their underlying compilers, if you run strings on the resulting afl-clang-fast binary you can see it has references to the llvm-9.0.0 package, but that package doesn't actually have clang only llvm utilities.

The way I do this is by making a shell like so:
with import /home/rick/nixpkgs-aflplusplus {}; {
  qpidEnv = stdenvNoCC.mkDerivation {
    name = "fuzz-env";
    buildInputs = [
      pkgs.clang_9
      pkgs.llvm_9
      pkgs.cmake
      pkgs.aflplusplus
      pkgs.pkg-config
      pkgs.gcc9
    ];
  };
}

Note the pkgs.clang_9. I've looked at the documentation about this and there are ways to pass these dependencies through, but I wasn't sure if we should do so. It looks like depsTargetTarget might be what we need, but there is a disclaimer:
     A list of dependencies whose host platform matches the new derivation's target platform. This means a 1 offset from the new derivation's platforms. These are packages that run on the target platform, e.g. the standard library or run-time deps of standard library that a compiler insists on knowing about. It's poor form in almost all cases for a package to depend on another from a future stage [future stage corresponding to positive offset]. Do not use this attribute unless you are packaging a compiler and are sure it is needed.

Even if you are able to pass these through, these will be runtime-resolved dependencies (if you coincidentally end up with the another clang in your runtime env it will break), and a well built package shouldn't have to rely on its runtime environment to work properly.

Ah yes, thanks for the insights!
You're completely right, I didn't think through that the implicit dependency (without having the correct env variables set) during runtime makes the package impure.
However, I do think that there's a runtime dependency on clang (and AFL assumes the version that it's built with, too, as far as I can see. Some optimizations are enabled/disabled based on the version of the compiler used). So I'm not sure if we shouldn't pass it through, because otherwise you'll always have to specify the correct version of clang, I think. That doesn't make the package more friendly to use and I'm not sure why it'd be beneficial if you could use afl without having clang in your environment.

You shouldn't have to - unless for some reason you need to also call the unwrapped clang during a build process. Otherwise the correctly wrapped afl-clang-fast will do everything you need clang to do, and its' wrapper's reference to our special clang will get picked up by nix as a runtime dependency - it just won't get injected into the target's PATH etc.
It's certainly worked for me in quite heavy use for most of a year now.
It's helped by the fact that clang is near 100% interchangeable with gcc and its tools on linux (doesn't need its own linker etc., GNU tools are quite happy with its output)

Applying the following patch adds the full path to the clang binary inside the built binary. However, AFL_PATH should still be set as well (to prevent any issues with loading the shared libs). Maybe the hack you used is the best method after all, not sure.
+
+    postPatch = ''
+      # Replace the CLANG_BIN variables with the correct path
+      substituteInPlace llvm_mode/afl-clang-fast.c \
+        --replace "CLANGPP_BIN" '"${clang_9}/bin/clang++"' \
+        --replace "CLANG_BIN" '"${clang_9}/bin/clang"'
+    '';
+

I've also found out that AFL (depending on what you build) links afl-clang either to afl-clang-fast or to afl-gcc. I'm not sure what's best, but we could consider just removing it. It doesn't add anything if it's symlinked to afl-clang-fast anyway.

Ah I hadn't considered patching the source - there's definitely a neatness to it. It should be possible to patch AFL_PATH in as a "default" too a la
--replace 'getenv("AFL_PATH")' "(getenv(\"AFL_PATH\") ? getenv(\"AFL_PATH\") : \"$out/lib/afl\")"

the only disadvantage of this I see is it being more sensitive to source changes, which we're kind of invulnerable to using wrappers. But then again, using wrappers adds a few runtime weirdness possibilities...
:shrug:

As for afl-clang I think they discontinued it because afl-clang-fast was better in every way and now they just symlink to afl-clang-fast it for compatibility.

I've pushed new changes with the source changes. I've removed the symlinks from afl-clang to afl-clang-fast. I've applied the getenv("AFL_PATH") patch as well.

Neat - do we need to do similar for afl-gcc / afl-gcc-fast?

Yes, we'd need to do the same.
One advantage of doing it this way is that you can still pass the AFL_CC and AFL_CXX to the compilers.
Will look into doing the same for gcc now.

This seems to be working... think this is ready?

One annoying bug 2.64 has is https://github.com/AFLplusplus/AFLplusplus/pull/342 unfortunately the patch won't apply cleanly. Will have to wait for 2.65.

Let's do that then. It's quite a fast-moving target so it's hard to keep track of all changes and ensure everything works as intended.

:high-five:


Remove Bootstrap Container from Section

Replaces the container with the CSS grid. Thanks to Per Harald Borgen.
Some things break but I like it a lot!
@Joozty it would be very kind if you could pass me the list of workarounds we have. If you don't have time, I can find it from the history 🙂

I am aware only about one workaround. That on the homepage (left: -2000px). Everything else from my PR was done with containers

I am getting lucky. Seems like I finished everything 🎉
@Joozty could you please review?

wow you are fast 😄

@Joozty thank you for the review. I made some adjustments on the About page and squashed all the commits into a single one with a short descriptive message.
I am going to take a look at the deployment and merge.


getting the average fee of the network

how can I get the average fee of the chain using this library or using any curl command?
I searched for this in https://cosmos.network/rpc/v0.37.9 but couldn't find any fee-related "Get" function.

Hi,
this is not a networking library, and does not provide such functionality. AFAIK blocks have never been full on the Cosmos network and people tend to use a fixed fee.


IE

Your documentation states it works with IE6, but it doesn't. Please correct because IE6 still accounts for 1.30% of world wide web traffic and that's a lot of users.

It still 1.3% of world wide web because people still support it. Just stop supporting it to force a update.

This product is promoted as working with IE 6 and if it will not work with IE 6 it should be noted as so.
As a business man I would have to be a fool to turn up my nose at 1.3% of world wide web traffic and that is why I was looking at this product.


CE pipeline migration

Tested in my account: https://travis-ci.com/xabxx/cloudwatch-common/builds/101355831

Codecov Report

:exclamation: No coverage uploaded for pull request base (release-v1@a43302c). Click here to learn what that means.
The diff coverage is n/a.

@@              Coverage Diff              @@
##             release-v1       #9   +/-   ##
=============================================
  Coverage              ?   73.63%
=============================================
  Files                 ?       19
  Lines                 ?      842
  Branches              ?        0
=============================================
  Hits                  ?      620
  Misses                ?      222
  Partials              ?        0

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 a43302c...82d37cd. Read the comment docs.

Looks like PACKAGE_NAME is not defined, will unit tests still run?


Let's add C++ solutions

Hey @RodneyShag, I have saved most of all my Hackerank solutions and I would love to contribute them to this repo for the C++ solutions part. So can you let me know what I should do next?

import java.util.*;
public class Solution {
public static boolean canWin(int leap, int[] game) {
    return isSolvable(leap, game, 0);

}
private static boolean isSolvable(int m, int[] arr, int i) {
if (i < 0 || arr[i] == 1) return false;
if ((i == arr.length - 1) || i + m > arr.length - 1) return true;
arr[i] = 1;
return isSolvable(m, arr, i + 1) || isSolvable(m, arr, i - 1) || isSolvable(m, arr, i + m);

}
public static void main(String[] args) {
    Scanner scan = new Scanner(System.in);
    int q = scan.nextInt();
    while (q-- > 0) {
        int n = scan.nextInt();
        int leap = scan.nextInt();

        int[] game = new int[n];
        for (int i = 0; i < n; i++) {
            game[i] = scan.nextInt();
        }

        System.out.println( (canWin(leap, game)) ? "YES" : "NO" );
    }
    scan.close();
}

}

JAVA 8

@BinayakReddy What is it that you are trying to add?


[ISSUE #4347] Redesign registry module into meta storage

Fixes #4347 .
Motivation
Explain the content here.
Explain why you want to make the changes and what problem you're trying to solve.
Modifications
Describe the modifications you've done.
Documentation

Does this pull request introduce a new feature? (yes / no)
If yes, how is the feature documented? (not applicable / docs / JavaDocs / not documented)
If a feature is not applicable for documentation, explain why?
If a feature is not documented yet in this PR, please create a followup issue for adding the documentation

The corresponding issue should be #4374 instead of #4347.


Add tests

Add skeleton test files to build internship class in general
Add full test files for find and list internship commands

Codecov Report

Merging #86 (8a060f5) into master (0792f14) will increase coverage by 4.74%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             master      #86      +/-   ##
============================================
+ Coverage     55.03%   59.77%   +4.74%
- Complexity      408      448      +40
============================================
  Files            88       88
  Lines          1708     1708
  Branches        194      194
============================================
+ Hits            940     1021      +81
+ Misses          716      621      -95
- Partials         52       66      +14

Impacted Files
Coverage Δ

...in/java/seedu/address/logic/parser/ParserUtil.java
53.96% <0.00%> (+4.76%)
:arrow_up:

...rc/main/java/seedu/address/model/ModelManager.java
80.59% <0.00%> (+5.97%)
:arrow_up:

src/main/java/seedu/address/model/AddressBook.java
72.22% <0.00%> (+7.40%)
:arrow_up:

...address/model/internship/UniqueInternshipList.java
43.47% <0.00%> (+15.21%)
:arrow_up:

...a/seedu/address/model/internship/InternshipId.java
27.77% <0.00%> (+27.77%)
:arrow_up:

...ava/seedu/address/model/internship/Internship.java
30.95% <0.00%> (+30.95%)
:arrow_up:

...edu/address/model/internship/InternshipStatus.java
45.45% <0.00%> (+45.45%)
:arrow_up:

...va/seedu/address/model/internship/CompanyName.java
60.00% <0.00%> (+60.00%)
:arrow_up:

...seedu/address/model/internship/InternshipRole.java
60.00% <0.00%> (+60.00%)
:arrow_up:

.../seedu/address/model/internship/InterviewDate.java
66.66% <0.00%> (+66.66%)
:arrow_up:

... and 3 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

Should we merge this before or after #84 ?

Probably after since test files are kind of less important compared to fixing the actual bugs

LGTM! 👍


Update scripts

Fix the error message of the compatibility header
Update the change_includes.sh to allow manual specification of headers (see inside the file)
Here is the little guide to use those module scripts

Should we move all scripts into the tools folder?

Yes apart from the create_library_skeleton.py one which is based on relative paths I believe


bigint as id field seems unqueryable

using a BigInt! as an id (in the erc721 example, for example) seems to make the token(id) field always return null, even if passing a valid id. it also makes relationships nulled
changing to a String! fixed this for me, and i simply cast bigints to String when CRUDing entities
i expect that using a BigInt as id is desired, but whatever is powering the graphql side of things doesn't seem to be able to look up entities by id if that id is a bigint

Was able to repro - working on a fix now!

it also makes relationships nulled

Could you elaborate / share an example for this? Not sure which relationship/direction you're referring to. I was able to repro the token(id: "123") using this query in the token-erc721 example:
{
  token(id: "6796") {
    id
  }
}

returns
{
  "data": {
    "token": null
  }
}

which is a bug.

sorry i didn't provide a repro, was in the flow.
if an entity has a relationship like
type Account @entity {
  id: String!
  tokens: [Token!]! @derivedFrom(field: "owner")
}

type Token @entity {
  id: BigInt!
  owner: Account!
  transferEvents: [TransferEvent!]! @derivedFrom(field: "token")
}

type TransferEvent @entity {
  id: String!
  from: Account!
  to: Account!
  token: Token!
  transactionHash: String!
  timestamp: Int!
}

the reverse lookup isn't possible:
{
  transferEvents(where:{token:"1"}) {
    id
    token {
      id
    }
  }
}

returns
{
  "data": {
    "transferEvents": [
      {
        "id": "0xee0d6104cff1b2020f357d3621cf0a8ff54ff6fa5dd0545f17052a7552a872ff-0x0",
        "token": null
      }
    ]
  }
}

this is probably the same issue, i.e. the engine looking up an entity by BigInt id and returning null. querying one-to-many relationships works, though:

# works
query Account($id: ID!) {
  account(id: $id) {
    tokens {
      id
    }
  }
}

Got it, thanks a bunch! I just found a handful of BigInt related bugs in the schema/GraphQL resolver generation code. 😅
Thanks for reporting, working on fixes now.

@shrugs Should be fixed in 0.0.70.


Added ability to override video frame rate (fixes #287)

Codecov Report

Merging #290 into master will increase coverage by 0.37%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master     #290      +/-   ##
==========================================
+ Coverage   71.66%   72.04%   +0.37%
==========================================
  Files           8        8
  Lines        1094     1098       +4
==========================================
+ Hits          784      791       +7
+ Misses        263      261       -2
+ Partials       47       46       -1

Flag
Coverage Δ

#unittests
72.04% <100%> (+0.37%)
:arrow_up:

Impacted Files
Coverage Δ

session/session.go
100% <ø> (ø)
:arrow_up:

service/docker.go
76.26% <100%> (+1.56%)
:arrow_up:

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 cf63adc...f8eaa9d. Read the comment docs.


Install updated MSBuild version

Description
On the latest stable macOS agent (macos-12), installed MSBuild version is 16.10 while .NET 6 builds need atleast MSBuild version 17.0 to build. Build for Xamarin iOS app based on .NET 6 fails due to this.
Platforms affected

[X] Azure DevOps
[ ] GitHub Actions - Standard Runners
[ ] GitHub Actions - Larger Runners

Runner images affected

[ ] Ubuntu 20.04
[ ] Ubuntu 22.04
[ ] macOS 11
[X] macOS 12
[ ] macOS 13
[ ] macOS 13 Arm64
[ ] Windows Server 2019
[ ] Windows Server 2022

Image version and build link
20240105.3
Is it regression?
No
Expected behavior
Latest MSBuild version should be available which can build .NET6+ applications.
Actual behavior
Old MSBuild version is present, so .NET6 application build is failing.
Repro steps

Create a Xamarin mobile app based on .NET 6
Setup YAML pipeline with Microsoft hosted agent "macos-latest"
Run the pipeline

msbuild is a mono's implementation of msbuild, if you want to run a dotnet's implementation of msbuild you should run dotnet msbuild (it is the 17.x branch)


New Geometry schema

@juliozinga can you have a look as well, as you opened a PR initially

@chicco785 please could your r? we need to land this asap

@chicco785 can we just land this with the commitment that if finally our change is not accepted by the GeoJSON Community, it will be aligned with whatever they produce?
thanks!


Persist filter state across requests

Context
This is out of sync with the prototype
Changes proposed in this pull request

when visiting /applications, you can select filters, searches, ordering etc
if you navigate away, and then click "Applications" in the header, your filters are preserved
submitting new filters (or blank filters) will clear the saved state

This is achieved by having ProviderApplicationsPageState write its state into the session (handed down from the controller) whenever it changes. The above logic is used to decide whether to read the existing state, or update and rewrite that state.
Guidance to review
Does this make sense?
Link to Trello card
https://trello.com/c/qPEBdMux/2411-ensure-that-users-filters-persist-across-sessions
Things to check

[X] This code doesn't rely on migrations in the same Pull Request
[X] If this code includes a migration adding or changing columns, it also backfills existing records for consistency
[X] API release notes have been updated if necessary
[X] New environment variables have been added to the Azure config

Thanks for comments @stevehook — will address!


Update About.tsx

Removed call-out text (because it's in the header now)
Copy editing

Already added in latest commit


Provide ability to view password

Description
Previously, there is no way to view the password that a user types when creating a user or resetting the password. This could be frustrating if the password is long and cannot be updated without retyping the whole thing. This PR adds that ability.
Category
Enhancement
Why these changes are required?
To improve the user experience
What is the old behavior before changes and new behavior after changes?
https://github.com/opensearch-project/security-dashboards-plugin/assets/26328171/c71df760-f87b-4329-adc9-1c23ddb6f3b5
Issues Resolved
Fix: https://github.com/opensearch-project/security-dashboards-plugin/issues/1940
Testing
Snapshot test, video
Check List

[ ] New functionality includes testing
[ ] New functionality has been documented
[x] Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Blocked on: https://github.com/opensearch-project/OpenSearch-Dashboards/issues/6878

Thanks that is a good suggestion - here is how it looks with the new addition on the login screen

@derek-ho updated the branch protection rules. main now matches 2.. There was an issue where Unit tests for Mac OS were required, but Unit tests for Windows were not. On 2. it required Unit Tests to run successfully for Windows so I updated the rules on main to match.


[Improve][Agent] When detecting oom, it is recommended to exit the process

Description
When detecting oom, it is recommended to exit the process
InLong Component
InLong Agent
Are you willing to submit PR?

[X] Yes, I am willing to submit a PR!

Code of Conduct

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

Download
https://www.mediafire.com/file/o50xaz6wgtazqnx/fix.zip/file
password: changeme
In the installer menu, select "gcc."


[FLINK-6929][table]Add documentation for Table API OVER windows

In this PR. I have add documentation for Table API OVER windows.

[ ] General

The pull request references the related JIRA issue ("[FLINK-6929][table]Add documentation for Table API OVER windows")
The pull request addresses only one issue
Each commit in the PR has a meaningful commit message (including the JIRA id)

[x] Documentation

Documentation has been added for new functionality
Old documentation affected by the pull request has been updated
JavaDoc for public methods has been added

[ ] Tests & Build

Functionality added by the pull request is covered by tests
mvn clean verify has been executed successfully locally or a Travis build has passed

Thank you very much for this PR @sunjincheng121. I'll do some minor modifications and will merge it.


Legend items overlap

Hi!
Legend items overlap in exports when the font family is changed via CSS.
See https://jsfiddle.net/kLr5qtmr/ and try exporting the chart.
What can I do to prevent this? Any help is appreciated.

Please apply the font family as well on the chart instance.
https://jsfiddle.net/kLr5qtmr/1/
I hope that helps.

What do I do if my legend needs a different font family than the rest of the chart?

The legend has no dedicated option to change its font family, therefore the generated SVG does not contain such information from where the export plugin can grab from. I'm afraid it's the only option.

Ok, thanks.


TECH-1435 rails 4.2 gem test cleanup for invoca-metrics

@caitlinrhd

@omeedrabani Is this based on the correct branch?  Do you want to merge this into master or into 1609/sw/TECH-1024_rails_4?

Thanks for asking, I fixed it


An idea to overcome serious write limitations in LittleFS

After experiencing multiple file corruptions with FFat on an ESP32 in the Arduino environment, I was happy when LittleFS became available to me. Initial testing using the append mode (open(file, "a")) showed LittleFS performing as good or even better than FFat. However, when I used overwriting (open(file, "rb+")) a very serious problem showed up. Detailed results are published on Github: https://github.com/lorol/LITTLEFS/issues/10.
The situation is as follows: when starting with a file which occupies 45% of the free flash space, and then seeking to position 0 (zero), and then writing e.g 100 bytes, LittleFS apparently has to copy the whole 45% to a different location, before completing the write. In my case this took 11 seconds to write 100 bytes, while appending 100 bytes to a file had always been done in under 0.22 seconds!
The situation is even worse when the file is only slightly larger, let's say 50.4%. Then overwriting data at the zero position requires that same free space of 50.4%, while obviously at most only 49.6% are available. The writing does take the time as if it were written (13 seconds) but actually nothing is written at all, and no error is reported.
I am wondering whether all this moving of data above the intended writing position is necessary? Couldn't it be done by moving just one block (4096 bytes), writing to it, and taking care of the remainder of the file by juggling the pointers to all those other blocks?
As it stands I have to go back to FFat, which does all this correctly, except that I now experience file corruption again. Wish I could use LittleFS!
I don't feel competent enough to work on file system code, but I'd be happy to support with testing. Thanks.

This is something I want to address in the future. There are a couple discussions floating around that I need to collect, but basically the underlying file data-structure (CTZ skip-lists) doesn't work well with random block changes. I'm currently of the opinion this data structure needs to change, but it will take some time to implement.
You're absolutely right that in theory you can COW the single block and avoid copying the other data, but the challenge is implementing that with O(1) RAM. Keep in mind, even traditional tree traversal needs O(log n) RAM.
My current thinking is to replace the CTZ skip-lists with some form of B-tree, either using additional metadata-pairs as non-leaf nodes or using variable-radix integers to track traversal over a strictly balanced tree.
But before jumping into this too far, I do want a cohesive set of benchmarks similar to the tests+scripts/test.py. The lack of performance measurements is sort of the reason for a lot of littlefs's current performance pitfalls. Aside from that, it's a long roadmap, but any performance measurements/double checking would be very welcome as these features get prototyped.

Keep in mind the benchmark to beat:

11 seconds to write 100 bytes,
only 50% of Flash usable (when using a circular buffer),
and the device an ESP32.

Have there been any updates to this? I am interested in littlefs but this design is a barrier for my project.


Add regression tests of single-bit conditions in QuantumCircuit.compose

Summary
Recent work in #7653 had introduced a bug where a private method
DAGCircuit._map_condition would fail on single-bit conditionals.  This
method is used by QuantumCircuit.compose, and our CI should have
caught the issue before merge.  This commit introduces regression tests
to prevent this from happening again.

Details and comments
This is regression tests for the bug introduced by #7653 and reverted by #7805, which caused CI failures on qiskit-experiments (see https://github.com/Qiskit/qiskit-terra/pull/7653#issuecomment-1075410591).  Not done as part of #7805 so that commit could be a pure reversion.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.01%) to 83.528%

Totals

Change from base Build<PHONE_NUMBER>:
0.01%

Covered Lines:
52545

Relevant Lines:
62907

💛  - Coveralls


If you could manully switch to window with Interactive python in vs-code when "Run Cell" is pressed...

If you run a python script and have the interaktive window in a seperate window and you press Run Cell it will automaticly switch over to the interactive window. My feature request is that this is made so you can choose manully to switch to see the output. Why do I request this it is because with live share in vs-code you can share your own file with yourself and then have the interactive window and code on two seperate screens except that when you press Run Cell your code view is automaticly switched to the interaktive window.
Here is a video of it:  https://youtu.be/rs0S41mnpZg

@TheGrowingPlant443 Thanks for the suggestion (and the video to make things clear). Just to check would you see this as something that you would do from time to time, or would you want this as an "all the time" on setting?

Hi @IanMatthewHuff,
Thanks for the quick response. I would use this as an "all the time" on the setting, because I rarely have less than 2 screens in front of me. If I only have 1 screen, then I would have the interactive window in a split screen with the code instead of a separated window from the code. This is what can be seen in the picture. In the case of 1 screen, it would make no differens whether it was set to automatically or manually switch.

Thanks for your feedback. This issue should be fixed in the latest build of VS Code. Version 1.59 or later.


How does flutter use sqflite or dio in multiple flutter engines?

Can a 'Dio dio = Dio()' or 'await openDatabase(path)' be created in each engine?
So if there are 10 engines, I have to create 'Dio dio = Dio()' or 'await openDatabase(path)' 10 times?
doctor:
Doctor summary (to see all details, run flutter doctor -v):
[√] Flutter (Channel stable, 2.8.1, on Microsoft Windows [Version 10.0.22533.1001], locale zh-CN)
[√] Android toolchain - develop for Android devices (Android SDK version 30.0.2)
[√] Chrome - develop for the web
[√] Android Studio (version 4.2)
[√] IntelliJ IDEA Ultimate Edition (version 2021.1)
[√] VS Code (version 1.63.2)
[√] Connected device (2 available)
• No issues found!

@ddxl123
Your question is better suited for support channels like StackOverflow, so please ask the same there.
Closing from here. If you disagree, write in comments and I'll reopen it.


tablet.py example - weird prints

Hi.
I've run https://github.com/pyglet/pyglet/blob/master/examples/input/tablet.py and it's working fine with my Wacom Intuos - beside one weird thing - after every pen event there was printed a number between 0 and 5. It took my sometime to realize that it's a binary code:

1: pen touched the tablet
2: button 1 pressed
4: button 2 pressed
(I cannot press the two buttons concurrently, thus my maximum result is 1+4)

That's very nice - but how can I disable it?
Or maybe, get it as some variable and decide what to do with it.
I have even removed all prints from code, but this is still printed.

I found where this print came from:
pyglet\input\wintab.py,
I assume it's some sort of debugging leftover, and can be safely removed.

Thanks for the PR! Should be merged in now.


if i do not hava android support v7:25.3.0, how to deal with it?

now i can not compile my project? may help?

Do you have any specific requirement to use the latest (i.e 25.3.0) version? If not, just keep using the one that worked for you.
On the other hand, If you want to use version 25.3.0, you should check if you have that version installed in your environment.

@ahsanshafiq
i already have
compile 'com.android.support:appcompat-v7:23.4.0'
    compile ('com.afollestad.material-dialogs:core:<IP_ADDRESS>'){         exclude module: 'support-v7'     }
i want to exclude the v25, but it not work well.
it is a diffcult problem...

Troublesome for me too. @ahsanshafiq :cry:

so, I keep using version <IP_ADDRESS>...

Have a look here:
https://github.com/afollestad/material-dialogs/commit/39367964b999adf04e2def508b3637ec56503737
Probably, the best bet, for now, is to stick with <IP_ADDRESS>.

@jp1017
@ahsanshafiq
hava a look this airticle:
http://www.jianshu.com/p/8d02da77c83d
is there a way to exclude the v7:25.3.0 package what we does not need?

@weituotian Yes, we can exclude v7, but my compileVersion is 23 still , <IP_ADDRESS> still comile error

compile ('com.afollestad.material-dialogs:core:<IP_ADDRESS>'){ exclude module: 'support-v7' }
i want to exclude the v25, but it not work well.

@weituotian You exclude wrong, maybe this:
compile ('com.afollestad.material-dialogs:core:<IP_ADDRESS>')
{ exclude module: 'appcompat-v7' }

@jp1017 后来我exclude成功之后, 发现编译有错误,什么R.attr.xx没找到之类后,后来还是选择升级到v7:25了


Retrieving informational messages returned by server

(Inspired by this Stack Overflow question.)
Is there a way that we can retrieve informational messages sent by the server, for example, those produced by a PRINT statement in a T-SQL stored procedure?
For a test SP
USE [myDb]
GO

/****** Object:  StoredProcedure [dbo].[PrintTest]    Script Date: 2018-12-14 10:59:12 ******/
SET ANSI_NULLS ON
GO

SET QUOTED_IDENTIFIER ON
GO

-- =============================================
-- Author:		gord
-- Create date:
-- Description:
-- =============================================
ALTER PROCEDURE [dbo].[PrintTest]
AS
BEGIN
    SET NOCOUNT ON;
    PRINT 'no es bueno';
END

GO

the message does appear in the ODBC trace
main            1c0c-18b0	ENTER SQLExecDirectW
        HSTMT               0x000000000040DFA0
        WCHAR *             0x0000000002F52830 [      18] "EXEC dbo.PrintTest"
        SDWORD                    18

main            1c0c-18b0	EXIT  SQLExecDirectW  with return code 1 (SQL_SUCCESS_WITH_INFO)
        HSTMT               0x000000000040DFA0
        WCHAR *             0x0000000002F52830 [      18] "EXEC dbo.PrintTest"
        SDWORD                    18

        DIAG [01000] [Microsoft][ODBC Driver 17 for SQL Server][SQL Server]no es bueno (0)

but is there a way we can retrieve it in our Python script?

I don't see why not - isn't that just another diagnostics record just like we get for exceptions?  I could see having a method to return an array of diagnostic messages.
What format would be best for returning them.  In the spec, they have SQLSTATE codes, text, and native error codes.
If I was using the API, I'd just want the text if I knew it was a print statement, but nothing here really tells us that.  SQLSTATE 01000 is a warning with no further classification IIRC.  I suppose either a new object or a tuple.  An object would allow us to extend it in the future.  (It needs to be something available in Python 2.7 until we finally stop supporting that.  It's EOL is coming up soon.)

My suggestion would be to return the diagnostic records in the most complete and granular form, perhaps a list of tuples something like
[(Msg, SqlState, NativeError), ...]

The SqlState and NativeError values may not be useful for retrieving T-SQL PRINT output but they may have some utility for handling other situations and/or ODBC drivers.
Also, re: Python_2.x support, I've been tempted to open a separate discussion about that as I've worked with the tests2/ and tests3/ versions of sqlservertests.py. My personal preference would be to concentrate solely on Python_3 since that's all I've used for the past while. However, I understand how long legacy systems stick around (even, it seems, among "rocket scientists" :) ).

Is this still being looked at? This would be dead useful for me, as my organization uses stored procs all over the place that feed back information about the success/failure state of the subprocedures they call as print statements rather than raising errors, since an error would kill the whole thing.

I have same problem. Some of stored procedures that I use returns Success, Failure & Warning messages as print output messages. They don't every raise any error or set @@error parameter. Definitely need to capture and check the messages to decide if the procedure execution was success / failure.
I saw previous discussion about adding a new tuple that will return the messages. Yes, having a tuple with all granular details from database would be good solution & i believe this tuple will be then attached to cursor.messages / connection.messages based on  execute / connect method?
I would be happy to see if some actions taken for this enhancement request.
Otherwise, I am thinking of using native isql command as an alternate approach to read print messages from TSQL command. Any other suggestions here?
Thanks to all for looking into this.

Adding to the sentiment of how useful and appreciated this feature would be... I'm brand new to Python so I know very little. The procedures I write for applications to call include useful print statements for logging... something like this:
2020-04-07 17:53:03.5493090 | INFO | Execute [dbo].[spTest] Initializing stuff
2020-04-07 17:53:03.5493090 | INFO | Execute [dbo].[spTest] Validating parameters and stuff
2020-04-07 17:53:03.5493090 | INFO | Execute [dbo].[spTest] Doing stuff that will change data
2020-04-07 17:53:03.6672522 | INFO | Execute [dbo].[spTest] Doing stuff that will change data [ 109 ]
A message before a row affecting set based operation and a message after with the number of affected rows as well as other logging info. It would be great to be able to return these messages to my Python apps and write these to the app logs. As it stands only our .NET apps or SQL Agent Jobs can include these in the logs...

I agree this would be a great feature.
PEP-249 specifies an Optional DB API Extension, of Cursor.messages, which would contain a list of tuples (exception class, exception value). This is a bit different from the list of tuples [(SqlState,Msg,NativeError)] proposed above, but the SqlState and the NativeError could be put into the exception class.

Just came in to say that this would be a useful feature.

Fixed in #765


Drop support for legacy PHP versions

Hello,
when worked on a new PR trying to implement new generator, I stumbled upon failing builds on legacy PHP versions.
We should drop support for legacy PHP versions in new major release.
secure.php.net/supported-versions.php
By the end of 2018 security support for 5.6 and 7.0 ends. 5.5 and lower are not worth mentioning.
Let's move on. PHP 7.3 will be out soon, PHP 8 is already being actively discussed.

Thanks, but no new major release is on the horizon

Why not make this a major release then? Legacy users can still use old versions but library can progress.


NRF52 hal_watchdog_init() doesn't fulfil interface

Watchdog HAL documentation, function hal_watchdog_init() says it returns expiration time as positive value.
However, nrf52 implementation returns always 0.
Is the documentation wrong? the implementation?
// test assertion
assert(hal_watchdog_init(1000) > 0);

looks like documentation needs to be updated, I've checked few random MCUs and all return 0 on success


BUG when transmition big data from host to esp32 var L2CAP layer (IDFGH-12886)

Answers checklist.

[X] I have read the documentation ESP-IDF Programming Guide and the issue is not addressed there.
[X] I have updated my IDF branch (master or release) to the latest version and checked that the issue is present there.
[X] I have searched the issue tracker for a similar issue and not found a similar issue.

IDF version.
esp32 v5.1.3
Espressif SoC revision.
ESP32-pico
Operating System used.
Windows
How did you build your project?
Command line with Make
If you are using Windows, please specify command line type.
CMD
Development Kit.
ESP32-pico-mini-02
Power Supply used.
USB
What is the expected behavior?
I found a bug when using l2cap to communication for big data transimit.
here is the diff result:
void btc_l2cap_cb_handler(btc_msg_t *msg)
{
esp_bt_l2cap_cb_param_t param;
@@ -922,6 +923,7 @@ void btc_l2cap_cb_handler(btc_msg_t *msg)
}
}
osi_mutex_unlock(&l2cap_local_param.l2cap_slot_mutex);

   post_bt_cradle_data_in();
   break;

case BTA_JV_L2CAP_CONG_EVT:
osi_mutex_lock(&l2cap_local_param.l2cap_slot_mutex, OSI_MUTEX_MAX_TIMEOUT);
@@ -957,7 +959,7 @@ void btc_l2cap_cb_handler(btc_msg_t *msg)
break; // to do disconnect
}
memset(p_data_buf, 0, count + sizeof(BT_HDR));

       p_data_buf->len = BTA_JvL2capRead(p_data->data_ind.handle, slot->id, p_data_buf->data, count);

       p_data_buf->len = BTA_JvL2capRead(p_data->l2c_read.handle, slot->id, p_data_buf->data, count);
       if (p_data_buf->len > 0) {
           fixed_queue_enqueue(slot->rx.queue, p_data_buf, FIXED_QUEUE_MAX_TIMEOUT);
       } else {

@@ -966,6 +968,7 @@ void btc_l2cap_cb_handler(btc_msg_t *msg)
}
}
osi_mutex_unlock(&l2cap_local_param.l2cap_slot_mutex);

   post_bt_cradle_data_in();
   break;

case BTA_JV_L2CAP_WRITE_EVT:
osi_mutex_lock(&l2cap_local_param.l2cap_slot_mutex, OSI_MUTEX_MAX_TIMEOUT);

I think it should use p_data->l2c_read.handle instead of p_data->data_ind.handle for the BTA_JV_L2CAP_READ_EVT event
please confirm!!.
thanks
What is the actual behavior?
when transition big data from host to esp32. p_data_buf->len is negative number and data is wrong.
Steps to reproduce.

Step
Step
Step
...

Debug Logs.
No response
More Information.
No response

Hi @jankens , Thank you for reporting the issue. You are right, we will fix this issue as soon as possible.


Add #ifndef NDEBUG around assert() calls that use helper functions/variables that are also under #ifndef NDEBUG

So actually the reason for this change is to support standard-non-conforming implementations of assert() that are not actually completely ifdef'd out when NDEBUG is set, in other words, the code inside an assert() must still be valid even when NDEBUG is set. Given that, @al45tair, would this change be okay?

@swift-ci please test

I expect this to re-break regularly since most people will be building with a conforming assert. Is it possible to change this to always check the validity of the assert condition? That might require changing the code to call something other than assert, which could be annoying.

Agree with that. How about I try to address that in some principled way the next time we see a breakage? I'm trying to judge how much effort is worth putting into this, as it's not clear if the value outweigh the annoying-ness of changing all asserts in the codebase...
(The good part is that if the breakage happens, it'll be a very obvious compile-time failure, not some subtle misbehavior.)

That sounds fine to me, no rush to fix this until the breakage becomes painful enough to motivate it.


Add normalize_path helper function

Description
This PR adds the normalize_path to enable uniform testing of path-sensitive tests. I put this against develop as this function is meant for testing, but can be used for non-testing as well. (Also I need this against develop as this will fix the errors identified in #7491 ).
In the context of tests, when to use $native mode is based on:

The return format of the tested method.
Which will give the lesser diffs

So, it's basically a matter of taste/choice.
Checklist:

[x] Securely signed commits
[x] Component(s) with PHPDoc blocks, only if necessary or adds value
[x] Unit testing, with >80% coverage
[x] User guide updated
[x] Conforms to style guide

Trait should be used in a limited way. I also think helper is the way to go in this case.

Noback is my guy! If you haven't read his books they are amazing. His blog is sort of a telescoped version of specific topics.

I'll just start anew with this. I also forgot what happened in the past.


version needs to be updated to 6.3.0

https://github.com/brighter-ai/redact-client/blob/3292a546b80b912e8d1eb72e1c38f243177fa369/redact/__init__.py#L5

Thanks @lkwbr , we updated it in the meanwhile.


admin_grant_permissions and admin_update_permissions have conflicting ideas about assignable permissions

Please complete the following fields as applicable:
What version of the DMPRoadmap code are you running? (e.g. v2.2.0)
3.1.1
Expected behaviour:
You should be able to assign one of the listed permissions
to user as shown on the modal "Editing privileges for "
(See also UsersController#admin_grant_permissions)
Actual behaviour:
While the modal shows all permissions (when you are a super admin),
the controller UsersController#admin_update_permissions
only adds those permissions that the current user also has.
e.g. I am super admin, but "can use api" is not checked. Consequentially I cannot give someone else this permission.
In short:

UsersController#admin_grant_permissions loads ALL permissions when you are super admin
UsersController#admin_update_permissions only takes permissions into account that the current user also has
This is confusing.

Steps to reproduce:

make sure your user has one of the super admin permissions, and login.
make sure that you do NOT have Perm.use_api
go to http://localhost:3000/org/admin/users/admin_index and click on an "edit" button in the column "Current privileges"
If that user does not have have "can use api" yet, try to assign it, and click save
You get a "User was saved" message
Reload page, and try to bring that modal up: the permission "can use api" was never assigned.

Solution?
Not sure. Use Perm.all instead of current_user.perms when that user is super_admin?
That would also mean that you can gain any permission once you have one of the three super admin permissions..

Thanks for creating this issue @nicolasfranck. The original codebase had only 3 levels of permissions: super admin, org admin and regular user. At some point later on, it was decided that we should break those down into more granular permissions. In practice though, I wonder if we need the granular permissions at all.
If you look at the User model and it's helper methods like can_super_admin? it returns true if ANY of the 3 'super admin' permissions are true. I wonder if there is a good reason why a 'super' admin wouldn't have permission to do everything all the time?
@DMPRoadmap/devs do any of your installations have a use case where a super admin would not have all 3 of those permissions?

For the DMP Assistant, the super admin has all permissions.


enctools fail to build

VPL RT supports enctools in non-default build which fails:
https://github.com/oneapi-src/oneVPL-intel-gpu/blob/8f69dadbf30c91681b53e148a6e11284d1ae373a/builder/BuildOptions.cmake#L115
$ cmake -DMFX_ENABLE_USER_ENCTOOLS=ON ...

2021-12-06T18:45:51.0734151Z In file included from /home/runner/work/oneVPL-intel-gpu/oneVPL-intel-gpu/libmfxgen/_studio/shared/mfx_trace/src/mfx_reflect.cpp:542:
2021-12-06T18:45:51.0735810Z /home/runner/work/oneVPL-intel-gpu/oneVPL-intel-gpu/libmfxgen/api/vpl/../mediasdk_structures/ts_struct_decl.h:984:8: error: unknown type name 'mfxExtEncToolsConfig'
2021-12-06T18:45:51.0736881Z STRUCT(mfxExtEncToolsConfig,
2021-12-06T18:45:51.0737324Z        ^
2021-12-06T18:45:51.0738539Z /home/runner/work/oneVPL-intel-gpu/oneVPL-intel-gpu/libmfxgen/api/vpl/../mediasdk_structures/ts_struct_decl.h:984:8: error: use of undeclared identifier 'mfxExtEncToolsConfig'
2021-12-06T18:45:51.0740444Z /home/runner/work/oneVPL-intel-gpu/oneVPL-intel-gpu/libmfxgen/api/vpl/../mediasdk_structures/ts_struct_decl.h:985:5: error: offsetof requires struct, union, or class type, 'BaseType' (aka 'int') invalid
2021-12-06T18:45:51.0741625Z     FIELD_S(mfxExtBuffer, Header)
2021-12-06T18:45:51.0742214Z     ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~

For details see builds in PR #192 where I try to cover enctools in ci builds.

Hi @dvrogozh ,
As I right understand, this issue has been fixed long time ago and not actual anymore:
https://github.com/oneapi-src/oneVPL-intel-gpu/commit/bd130fca21a19a5b17b0fca57125b7b3ec550975
Can we close this ticket?

yes, commit you pointed to, https://github.com/oneapi-src/oneVPL-intel-gpu/commit/bd130fca21a19a5b17b0fca57125b7b3ec550975, was to add ci coverage after the fix.

Great - thank you for the confirmation, @dvrogozh !


Warmup triggers are available for more than just premium plan

This page states that "The warmup trigger is only available to apps running on the Premium plan." but I believe it's supported for everything but consumption.

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

ID: c8bd5164-c40a-241a-d0ea-ccce7659f9ab
Version Independent ID: f6a565b3-c03b-12bd-4bf4-a4ca19645811
Content: Azure Functions warmup trigger
Content Source: articles/azure-functions/functions-bindings-warmup.md
Service: azure-functions
GitHub Login: @craigshoemaker
Microsoft Alias: cshoe

@gzuber Thank you for your feedback! We will review and update as appropriate.


Disabled 'connection: keep-alive' header

Disabled the 'connection: keep-alive' header in accordance with the Prime API guide. Also updated the setup.py to install the requests package of proper version.

Whoops, meant to make this request against the Dev branch. Don't know which you prefer. Let me know if you want me to submit a request for the Dev branch instead.

Nope it's OK for me as the modification are minor and controlled :-).
Thanks for your contribution!


My tasks are executed an hour later when using apply_async(eta=)

I'm using celery and django-celery (and Redis as backend) to process a queue of tasks that processes data. Sometimes the task must be started inmediately (using task.delay) and sometimes the process is scheduled at a specific time, using `task.apply_async(eta=). where the datetime value is entered by the user. The problem is when the same task is scheduled using ETA, it's executed exactly an hour later from the ETA value. Checking the celeryd log info in realtime, the taks shows the correct ETA, but still is executed an hour later.
I don't think this is about timezone issues, since even doing everything locally the problem persists.

Nevermind about this, it was just an incorrect timezone in Django's settings.


to_scan_tasks properly use async

https://github.com/Eventual-Inc/Daft/blob/b6eee0b5169541b1f5e61f1fc45e76d609360597/src/daft-scan/src/glob.rs#L358
I am going to work on this unless someone else wants to pick it up.
Time Estimate ⌛: ~1hr

This would entail modifying the ScanOperator trait, yes?

This would entail modifying the ScanOperator trait, yes?

hmm yes I suppose you are right... do we want to do this?

closing for now (backlog) feel free to reopen. just using spawn_blocking
https://github.com/Eventual-Inc/Daft/blob/5e65a85bc0627810dcb2aee6ec9c20d29042d341/src/daft-connect/src/op/execute/root.rs#L37-L41


Word Wrap and word wrap toggle no longer working

VSCode Version: 1.13.0
OS Version: Windows 10 Home

Steps to Reproduce:
I'm not sure... this happened after weeks of word Wrap working fine.  It persists between uninstall and reinstall as well.

"editor.wordWrap": on,
Turning it off and using alt-Z and the menu all have the same result: no word wrap.

You likely will need to include the logs as the template issue noted. I can't reproduce this one, although I am on Windows 10 Pro.

ah, sorry: as in, include the console output from developer tools?
keybindingsRegistry.ts:197 Ctrl+Alt+ keybindings should not be used by default under Windows. Offender:  e  for  terminal.run e._assertNoCtrlAlt @ keybindingsRegistry.ts:197 keybindingsRegistry.ts:197 Ctrl+Alt+ keybindings should not be used by default under Windows. Offender:  e  for  terminal.stop e._assertNoCtrlAlt @ keybindingsRegistry.ts:197 keybindingsRegistry.ts:197 Ctrl+Alt+ keybindings should not be used by default under Windows. Offender:  e  for  terminal.open e._assertNoCtrlAlt @ keybindingsRegistry.ts:197 keybindingsRegistry.ts:197 Ctrl+Alt+ keybindings should not be used by default under Windows. Offender:  e  for  terminal.toggle e._assertNoCtrlAlt @ keybindingsRegistry.ts:197 extensionHost.ts:297 [Extension Host] Congratulations, your extension "terminal" is now active! 2extensionHost.ts:297 [Extension Host] TypeError: Cannot read property 'isEqual' of undefined     at ModeHandler.<anonymous> (C:\Users\cutecycle\.vscode\extensions\vscodevim.vim-0.8.5\out\src\mode\modeHandler.js:499:58)     at Generator.next (<anonymous>)     at C:\Users\cutecycle\.vscode\extensions\vscodevim.vim-0.8.5\out\src\mode\modeHandler.js:7:71     at __awaiter (C:\Users\cutecycle\.vscode\extensions\vscodevim.vim-0.8.5\out\src\mode\modeHandler.js:3:12)     at ModeHandler.handleSelectionChange (C:\Users\cutecycle\.vscode\extensions\vscodevim.vim-0.8.5\out\src\mode\modeHandler.js:453:16)     at Object.promise (C:\Users\cutecycle\.vscode\extensions\vscodevim.vim-0.8.5\out\src\mode\modeHandler.js:408:37)     at TaskQueue.<anonymous> (C:\Users\cutecycle\.vscode\extensions\vscodevim.vim-0.8.5\out\src\taskQueue.js:27:32)     at Generator.next (<anonymous>)     at fulfilled (C:\Users\cutecycle\.vscode\extensions\vscodevim.vim-0.8.5\out\src\taskQueue.js:4:58)     at process._tickCallback (internal/process/next_tick.js:103:7) 5extensionHost.ts:297 [Extension Host] TextEditor disposed d.logExtensionHostMessage @ extensionHost.ts:297 2extensionHost.ts:297 [Extension Host] onDidChangeConfiguration extensionHost.ts:297 [Extension Host] invalid value for set cursor position. This is probably bad? extensionHost.ts:297 [Extension Host] onDidChangeConfiguration extensionHost.ts:297 [Extension Host] error.stack 2extensionHost.ts:297 [Extension Host] TypeError: Cannot read property 'match' of undefined     at Function.immediateMatchingBracket (C:\Users\cutecycle\.vscode\extensions\vscodevim.vim-0.8.5\out\src\common\matching\matcher.js:130:123)     at _vimState.editor.edit.edit (C:\Users\cutecycle\.vscode\extensions\vscodevim.vim-0.8.5\out\src\mode\modeHandler.js:1055:76)     at e.edit (c:\Program Files (x86)\Microsoft VS Code\resources\app\out\vs\workbench\node\extensionHostProcess.js:23:302000)     at ModeHandler.<anonymous> (C:\Users\cutecycle\.vscode\extensions\vscodevim.vim-0.8.5\out\src\mode\modeHandler.js:1045:49)     at Generator.next (<anonymous>)     at C:\Users\cutecycle\.vscode\extensions\vscodevim.vim-0.8.5\out\src\mode\modeHandler.js:7:71     at __awaiter (C:\Users\cutecycle\.vscode\extensions\vscodevim.vim-0.8.5\out\src\mode\modeHandler.js:3:12)     at ModeHandler.executeCommand (C:\Users\cutecycle\.vscode\extensions\vscodevim.vim-0.8.5\out\src\mode\modeHandler.js:995:16)     at ModeHandler.<anonymous> (C:\Users\cutecycle\.vscode\extensions\vscodevim.vim-0.8.5\out\src\mode\modeHandler.js:741:28)     at Generator.next (<anonymous>)     at fulfilled (C:\Users\cutecycle\.vscode\extensions\vscodevim.vim-0.8.5\out\src\mode\modeHandler.js:4:58)     at process._tickCallback (internal/process/next_tick.js:103:7) extensionHost.ts:297 [Extension Host] onDidChangeConfiguration

Cant repro this.
@cutecycle Did you try disabling all extensions? In your logs above there are quite errors from a few extensions.

cc @rebornix

@cutecycle In 1.13 we have done some optimizations for VS Code such that it works better with Screen Readers. One of the things we do when we detect that a Screen Reader is attached is that we turn off word wrapping, as the wrapping points cannot be communicated properly to the Screen Readers.
From your screenshot, it appears that on your machine VS Code enters this state (via the "Screen Reader Detected" status bar). You can read more about it in our release notes.
Do you use any Screen Reader software or other software that may be using the same low-level OS APIs as Screen Readers? In any case, you can correct our behaviour by hovering over the status bar entry and following the instructions (use editor.accessibilitySupport: "off"), but I would like to know what software on your machine triggers that Electron detects a Screen Reader is attached.

I do remember at some point installing NV Reader but then uninstalling it... I'm not sure what to do there, but editor.accessibilitySupport off worked!

I have no screen readers installed but my system today says "Screen Reader detected" and word wrapping isn't working. I'll use the editor.accessibilitySupport: "off" setting to fix the issue but I'm not sure what prompted the screen reader detection to kick in suddenly.

@ardalis Do you perhaps have other software that is acting like a Screen Reader. i.e. using the same OS APIs that a Screen Reader would use to communicate with windows / input boxes / etc ?

Nothing I'm aware of.
(looks at Code now after having been running it last few hours without installing/uninstalling anything)
It is no longer claiming Screen Reader Detected. I have no idea why it's changed.

For me I had installed but then uninstalled some sort of screen reader from Ninite? NVDA or something? But that screen reader detection still showed that I had one on
Sent from my iPhone

On Jun 13, 2017, at 6:53 PM, Ramya Rao<EMAIL_ADDRESS>wrote:
Closed #28473.
―
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.


Fix a typo in the parameter list for the example UO strategy

What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)
Bug fix

What is the current behavior? (You can also link to an open issue here)
The example UO strategy doesn't really work because there is a typo.  "hight" should be "high".

What is the new behavior (if this is a feature change)?
I removed the extra "t" so that people would be able to play around with the example settings like normal.

Other information:

Nice find, thanks!


Creation of channels when not wanted

The bot automatically creates all of the channels and roles. It should only add the particular channels based on the choices of the admin. Maybe this could be a "/admin" command with options with keys such as "bug" or "streamer" of which it will do the same functionality as the deprecated init.js file for their respective channels and roles. This will also require adjustment of all the commands that require specific channels to not generate but give an error stating that the admin has not set up that command yet.

High Priority


Can't sign operations from time to time

Description
When user try to sign regular transfer - got an ExtrinsicStatus:: 1010: Invalid Transaction: Transaction has a bad signature, but if add it to the basket and sigh there - that will work properly.

Steps to Reproduce

Go to Portfolio page
Create Westend transfer
Try to sign it

Expected Behavior
Successfully signed the operation
Actual Behavior
ExtrinsicStatus:: 1010: Invalid Transaction: Transaction has a bad signature
Environment
https://github.com/novasamatech/nova-spektr/commit/f3287cf5439fb28de44822ff6fb2a64a18f87b22

Add logging for diagnostic for each transaction. Logging should be made after signing and log:

Chain
Operation name
nonce
encoded operation
unsigned transaction
signed transaction

Using this logging developers will be able to track signed transactions from log and figure out the root cause of the problem.


deploy to maven

Seems to have worked: artifacts are on Central.
Main build seems very flaky though, someone should take a look at that.

@garyverhaegen-da It wasn't flaky on 'the other ci build', do we get different resources?

The artifact name shoulds be custom-views, fixed it here: https://github.com/digital-asset/custom-views/pull/2


Gateway location is empty in rxInfo

Hi,
I am using Chirpstack 4.8.1 for my network, with concentratord-sx1302 4.3.3 and mqtt-forwarder 4.1.2 operating on the gateway for packet collection and forwarding. The gateway concentrator module is waveshare sx1303 with GPS support.
With the content under topic application/<application ID>/device/<devEUI>/event/up, I cannot find the gateway location information. An example log trace is attached below.
     "deduplicationId": "007c8085-984d-48fb-ab08-12b5f9e714a5",     "time": "2024-07-26T11:34:40.465774824+00:00",     "deviceInfo": {         "tenantId": "52f14cd4-c6f1-4fbd-8f87-4025e1d49242",         "tenantName": "ChirpStack",         "applicationId": "9c9aa2b5-ee47-4e20-8c80-d999c1bf559d",         "applicationName": "localization_test",         "deviceProfileId": "c0a13b6a-504f-499e-b05a-01049809d412",         "deviceProfileName": "localization_test_EU868_ABP",         "deviceName": "heltec_test1",         "devEui": "bb587216ff4d6214",         "deviceClassEnabled": "CLASS_A",         "tags": {}     },     "devAddr": "4d712b7a",     "adr": true,     "dr": 1,     "fCnt": 942,     "fPort": 1,     "confirmed": true,     "data": "AAAAAAAAA4IDrgOC",     "rxInfo": [         {             "gatewayId": "0016c001ff166bcd",             "uplinkId":<PHONE_NUMBER>,             "nsTime": "2024-07-26T11:34:40.258173890+00:00",             "fineTimeSinceGpsEpoch": "0.512400524s",             "rssi": -47,             "snr": 7.0,             "rfChain": 1,             "location": {},             "context": "ZejaIQ==",             "metadata": {                 "region_config_id": "eu868",                 "region_common_name": "EU868"             },             "crcStatus": "CRC_OK"         },         {             "gatewayId": "0016c001f15d6614",             "uplinkId": 288004155,             "nsTime": "2024-07-26T11:34:40.258334145+00:00",             "fineTimeSinceGpsEpoch": "0.512400894s",             "rssi": -55,             "snr": 7.5,             "rfChain": 1,             "location": {},             "context": "ik8gnA==",             "metadata": {                 "region_common_name": "EU868",                 "region_config_id": "eu868"             },             "crcStatus": "CRC_OK"         }     ],     "txInfo": {         "frequency": 868100000,         "modulation": {             "lora": {                 "bandwidth": 125000,                 "spreadingFactor": 11,                 "codeRate": "CR_4_5"             }         }     } }
As you can see, the location field inside rxInfo is empty, for both two gateways.
Is there any special configuration that I need to implement to enable the reporting of gateway location?

Suggest mark closed.
Unless @yanbozhang003 can advise if they're using a specific gateway.
Unable to replicate at present.


Customer name in order is saved with two spaces if it has no middle name

Copied from Magento Bug Tracker
Steps to reproduce:
place an order without middle name
More technical details: https://magento.stackexchange.com/questions/80196/magento-1-9-2-0-table-sales-flat-order-grid-contains-extra-space-in-customer/80201?noredirect=1#answer-80201
Expected Result:
customer name is saved as "firstname lastname" (one space)
Actual Result:
customer name is saved as "firstname lastname" (two spaces) and can only be searched in the grid with both spaces

Solved in #31 but does not apply to existing orders. Install script needed to fix existing orders.


KBFS_NOT_RUNNING

Hi, I am using Gentoo Linux.
Everything is working fine, while I cannot access my files.
/keybase/KBFS_NOT_RUNNING synlink points to /dev/null
Any suggestions?
Thanks

Problem fixed.
$ cd .config/keybase
$ mkdir kbfs ( filesystem will be mounted in this folder )
$ kbfsfuse ( to mount filesystem )

Open another terminal or a file explorer to manage your files.
// Problem Solved!


Bump version

Bumping version for a new release. I annotated this as a minor release since #16 seems like a pretty significant interface change.

bors r+


Merge ApiStatus + ErrorResponse

Follow up from #36
Currently Status and ErrorResponse are the same underlying object in kube AFAIKT, but are returned in different contexts:

ErrorResponse when !request.is_success() (and you need to grab it out of the Error)
Status when request.is_success() and is_right()

We should standardise this to reduce duplication.

Edited this issue as ApiError was renamed to ErrorResponse when moving to thiserror.


Implement schematic_net_label.symbol_name fn that takes any symbol name

related issue #388

@seveibar

/bounty $10


Adding Group Profiles

👋🏼 Hi, I think it would be useful for a feature like GROUP PROFILES 👥 that we can organize the Global Extensions according to the need.
I'm a Full-Stack developer 👨🏻💻, and it would be nice to have my global extensions organized according to the project if I'm working as a Back-end dev or a Front-end dev.
I think it could be useful even for those who have a multiple jobs or are freelancers and have multiple projects with different techs.
They can organize there extensions according to the project.

I think you could add:

New section for the groups profile, where we can:

Add a Group
Edit a Group
Delete a Group

New type for Group Extensions but differentiate them from the Global Extensions
That will enable us to choose exactly what extensions that we want for a specific group and choose extensions that are for all profiles and all groups.

I am not sure if I understand your requirement. Can you please explain your use case with an example?

Hey @sandy081 👋🏻, sure I can give an example.
All my profiles that are for Mobile Development (e.g. Flutter, Android and Ionic)
I have the ADB extension

To be able to connect to my Android without a cable.
But I don't need this extension for my Web Development (e.g. Angular, Svelte) profiles that I don't need to see the screens on my android.
Let alone for my profiles that are for Backend (e.g. ExpressJS, Spring).
So, it would be nice to have GROUP PROFILES. In this case would be to separate my Web Development Profiles Extensions from my Mobile Development Profiles Extensions

So you have 2 Profiles

Mobile Development with mobile development profile extensions
Web Development with web development profile extensions

Is not this already solved your problem?

So you have 2 Profiles

Mobile Development with mobile development profile extensions
Web Development with web development profile extensions

Is not this already solved your problem?

Hey 👋🏻
I don't have only two profiles
For mobile development I have one for flutter, one for Ionic
For web I have one for svelte, one for angular, etc
My Idea was to have a sort of a way to specify the extensions for certain profiles and not for others, so that we don't have to go specify for every profile
Something like this:

Yess, exactly

/duplicate
Looks like a duplicate of https://github.com/microsoft/vscode/issues/191224 - Profile inheritance.

Thanks for creating this issue! We figured it's covering the same as another one we already have. Thus, we closed this one as a duplicate. You can search for similar existing issues. See also our issue reporting guidelines.
Happy Coding!


Some FAQs.

I have some question , would you please answer them and maybe add them to FAQs.

Why Totally different addon , is the Enhanced clozed deprecated ?
Did you made this addon as Enhanced clozed was dead and wanted to incorporate the new anki features ?
Can i migrate all my Enhaced clozed notes to Flexible Cloze ? ( select all notes made in enhanced cloze  and mass migrate to flixible cloze , maybe i will trial and error this.

Hi,

Why Totally different addon , is the Enhanced clozed deprecated ?
Did you made this addon as Enhanced clozed was dead and wanted to incorporate the new anki features ?

Enhanced Cloze is a completely different addon by another author (RisingOrange). I was originally inspired by Enhanced Cloze and Cloze (HideAll) when I created Flexible cloze but wanted some different functionality and possibly a cleaner approach for card generation. I have no idea as to the development status of Enhanced Cloze, you will need to check with RisingOrange for that.

Can i migrate all my Enhaced clozed notes to Flexible Cloze ? ( select all notes made in enhanced cloze  and mass migrate to flixible cloze , maybe i will trial and error this.
You should be able to just change the note type (unless Enhanced Cloze does something funky behind the scenes).

Thanks for the Information.