Characters Removed from Extracted Data - How does extract handle invalid JSON characters

Hello,
                      We are having an issue with Text fields in Business Central that contain backslashes '\' in them. The \ are removed when viewing the data via Synapse and in the delta and data files themselves. When are the invalid JSON characters escaped out of the data in the BC2ADL process and can this be modified to allow these characters?

Thanks
Tom

HJi @tglink72 so currently your scenario is when you export a text field with a '' in it will be already removed in the .csv file?

Bert,
           Thanks  for the reply. The scenario currently is that when we export a text field with a backslash \ in it. The \ is removed. An example is in the screenshot below.  The regex column has this value in a basic Synapse query. It looks like they are being removed in the initial export to the Delta folder as I do not see the \ in the Regex field in the raw csv in the delta folder.

Regex-3
^Add[A-Z]-d{5}$
But if we view the data in BC below you will see it contains \ that are not included in the query.
@.***
From: Bert Verbeek @.>
Sent: Friday, September 6, 2024 2:39 AM
To: Bertverbeek4PS/bc2adls @.>
Cc: Tom Link @.>; Mention @.>
Subject: Re: [Bertverbeek4PS/bc2adls] Characters Removed from Extracted Data - How does extract handle invalid JSON characters (Issue #169)
You don't often get email from @.*** Learn why this is importanthttps://aka.ms/LearnAboutSenderIdentification
HJi @tglink72https://github.com/tglink72 so currently your scenario is when you export a text field with a '' in it will be already removed in the .csv file?
—
Reply to this email directly, view it on GitHubhttps://github.com/Bertverbeek4PS/bc2adls/issues/169#issuecomment-2333439537, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AZOTVUKZGNEHXJB44Q4WEE3ZVFLZZAVCNFSM6AAAAABNSS4PLGVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGMZTGQZTSNJTG4.
You are receiving this because you were mentioned.Message ID: @.***>
"The information contained in this e-mail, and any attachment, is confidential and is intended solely for the use of the intended recipient. Access, copying or re-use of the e-mail or any attachment, or any information contained therein, by any other person is not authorized. If you are not the intended recipient, please immediately return the e-mail to the sender and delete it and any attachment from your computer. Although we attempt to sweep e-mail and attachments for viruses, we do not guarantee that either are virus-free and accept no liability for any damage sustained as a result of viruses."

Thanks @tglink72 I then look if I can repo it. Hopefully I have got time for it this week.

@tglink72 I did a test with the export to MS Fabric Lakehouse.
Customer comments:

Then export to csv delta file:

When the notebook is runned:

So with Fabric it goes well.
I will also look into export of Azure Data Lake

@tglink72
I have also tested it with the synapse pipelines and Azure File Storage.
But cannot reproduce it.
WHen exporting the delta's:

WHen the synapse pipeline runs and in PowerBI:

I'm using parquet files as destination.

Bert,
           Thanks for the replies. I am also using synapse pipeline and Azure File storage and unfortunately, I can create it each time. The backslash is removed in the extension extract to the deltas folder. If you would like we could do a screenshare.

Thanks
Tom Link
From: Bert Verbeek @.>
Sent: Monday, September 16, 2024 1:16 PM
To: Bertverbeek4PS/bc2adls @.>
Cc: Tom Link @.>; Mention @.>
Subject: Re: [Bertverbeek4PS/bc2adls] Characters Removed from Extracted Data - How does extract handle invalid JSON characters (Issue #169)
You don't often get email from @.*** Learn why this is importanthttps://aka.ms/LearnAboutSenderIdentification
@tglink72https://github.com/tglink72
I have also tested it with the synapse pipelines and Azure File Storage.
But cannot reproduce it.
WHen exporting the delta's:
image.png (view on web)https://github.com/user-attachments/assets/951b7fcb-3210-4004-92ae-e6ed3a6dd0f6
WHen the synapse pipeline runs and in PowerBI:
image.png (view on web)https://github.com/user-attachments/assets/51a25036-b913-43b9-93d7-73cc58d10ed2
I'm using parquet files as destination.
—
Reply to this email directly, view it on GitHubhttps://github.com/Bertverbeek4PS/bc2adls/issues/169#issuecomment-2353592433, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AZOTVUKETWUYTAWKG2LXG5TZW4N45AVCNFSM6AAAAABNSS4PLGVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGNJTGU4TENBTGM.
You are receiving this because you were mentioned.Message ID: @.***>
"The information contained in this e-mail, and any attachment, is confidential and is intended solely for the use of the intended recipient. Access, copying or re-use of the e-mail or any attachment, or any information contained therein, by any other person is not authorized. If you are not the intended recipient, please immediately return the e-mail to the sender and delete it and any attachment from your computer. Although we attempt to sweep e-mail and attachments for viruses, we do not guarantee that either are virus-free and accept no liability for any damage sustained as a result of viruses."

Ok strange @tglink72 . Which version do you have?
Indeed is it OK to have a meeting? On friday I got the whole afternoon available.

Bert,
           Hello. Would you be able to meet anytime this week. I can meet early morning central time US.

Thanks
Tom Link
From: Bert Verbeek @.>
Sent: Tuesday, September 17, 2024 9:28 AM
To: Bertverbeek4PS/bc2adls @.>
Cc: Tom Link @.>; Mention @.>
Subject: Re: [Bertverbeek4PS/bc2adls] Characters Removed from Extracted Data - How does extract handle invalid JSON characters (Issue #169)
You don't often get email from @.*** Learn why this is importanthttps://aka.ms/LearnAboutSenderIdentification
Ok strange @tglink72https://github.com/tglink72 . Which version do you have?
Indeed is it OK to have a meeting? On friday I got the whole afternoon available.
—
Reply to this email directly, view it on GitHubhttps://github.com/Bertverbeek4PS/bc2adls/issues/169#issuecomment-2356019512, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AZOTVUKEPFWSCTD7ECJIXGTZXA4AFAVCNFSM6AAAAABNSS4PLGVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGNJWGAYTSNJRGI.
You are receiving this because you were mentioned.Message ID: @.***>
"The information contained in this e-mail, and any attachment, is confidential and is intended solely for the use of the intended recipient. Access, copying or re-use of the e-mail or any attachment, or any information contained therein, by any other person is not authorized. If you are not the intended recipient, please immediately return the e-mail to the sender and delete it and any attachment from your computer. Although we attempt to sweep e-mail and attachments for viruses, we do not guarantee that either are virus-free and accept no liability for any damage sustained as a result of viruses."

@tglink72 this week I cannot make it.
But next week monday or Teusday 13:00 - 17:00 GMT+2

Closed because no respond. Please reopen this issue when it is still there.


Allow making moves on the chessboard (and therefore adding them to the Workbook) when Evaluation in on

See below, system is evaluating move 6...Bg4 (red below). I can see 2nd engine suggestion is 7.Na4 (green below) and I am trying to make this move on the board (purple below) but system does not let me:

Optimally, I want to use constant evaluation mode to build variation I am to play. So if engine evaluation suggests 7.Na4 I want to play it and then want system to evaluate that 7.Na4 and suggest best black replies.

Implemented.


chore: Adopt black formatting

Issue #, if available:
N/A
Description of changes:
Move this repo from flake8 to black. Previous to this, developers would run make pr and handle any pylint or flake8 errors reported manually. With black, which is similar to go's fmt, will automatically format all the files for you. I also added 'pre-commit` configuration, so that way developers can install the git hooks and will ensure all files were formatted with black before doing a push.
This will align this repo with AWS SAM CLI's and provide a consistent developer experience for contribution across these repos.
Description of how you validated changes:
Ran make pr
Checklist:

[ ] Write/update tests
[x] make pr passes
[ ] Update documentation
[ ] Verify transformed template deploys and application functions as expected
[ ] Add/update example to examples/2016-10-31

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

Codecov Report

Merging #1322 into master will decrease coverage by 0.62%.
The diff coverage is 92.26%.

@@            Coverage Diff             @@
##           master    #1322      +/-   ##
==========================================
- Coverage    95.3%   94.67%   -0.63%
==========================================
  Files          78       77       -1
  Lines        5171     4129    -1042
  Branches     1190      815     -375
==========================================
- Hits         4928     3909    -1019
+ Misses        113      108       -5
+ Partials      130      112      -18

Impacted Files
Coverage Δ

samtranslator/model/naming.py
100% <ø> (ø)
:arrow_up:

samtranslator/public/sdk/resource.py
100% <ø> (ø)
:arrow_up:

samtranslator/validator/validator.py
95.23% <ø> (-0.42%)
:arrow_down:

samtranslator/model/tags/resource_tagging.py
100% <ø> (ø)
:arrow_up:

...translator/policy_template_processor/exceptions.py
100% <ø> (ø)
:arrow_up:

samtranslator/public/sdk/parameter.py
0% <ø> (ø)
:arrow_up:

samtranslator/public/translator.py
0% <ø> (ø)
:arrow_up:

samtranslator/plugins/globals/globals_plugin.py
100% <ø> (ø)
:arrow_up:

...translator/plugins/api/implicit_http_api_plugin.py
100% <ø> (+1.36%)
:arrow_up:

samtranslator/public/sdk/template.py
100% <ø> (ø)
:arrow_up:

... and 81 more

Continue to review full report at Codecov.

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

@jfuss I have rebased this PR to  to merge into develop. Could you please resolve the merge conflicts?

@ShreyaGangishetty Rebased with HEAD of develop.
On a side note: If the intention is for developers to put code into develop, you should consider moving the default branch to develop instead of master. I expect the default branch to be where code will end up.

@jfuss Thank you for this suggestion! We had our official SAM documentation in Github until recently. We didn't want to redirect everyone to our develop documentation until the documentation is made official through releases. Now that we have AWS SAM specification it is a good idea to make develop as the default branch


Investigate other/additional linting options

Description:

helium-typescript currently uses tslint as a linter
This task covers investigating other or additional options that should be included to help code quality and looking at the existing linting options enforced to make sure it is a complete/reasonable list (i.e. nothing important is excluded).

I finished working with Jason LaChance's dev crew on the Starbucks project using Node and TypeScript and they had eslint-typescript in use, so I can use that as a reference and migrate this project to that. eslint-typescript is the clear choice for linting.

I'll close this and create a task to implement eslint-typescript.


Pesto: Image persists on a higher layer than the toolbar when contracting down

(from a material design review)
Open Gallery, Open pesto. Tap a recipe. Then, tap back.
Expected: the image of the recipe should be below the toolbar.
Actual: Image persists on a higher layer than the toolbar when contracting down

We've hidden Pesto


Module not found: Error: Can't resolve 'crypto'

I am encountering errors consistently when starting my create-react-app. Could you please provide guidance on how to resolve this issue?
Module not found: Error: Can't resolve 'crypto'

node: v18.17.0

react: v18.2.0

My CodeSandbox:
https://codesandbox.io/p/devbox/react-puck-test-psj37t?file=%2Fsrc%2FEditor.jsx

This is a duplicate of #112. I'm going to remove crypto in 0.13.1.

This is a duplicate of #112. I'm going to remove crypto in 0.13.1.

Great! Thank You for resolving this :)

Thanks for closing the issue @seeranjeeviramavel!


terminal chat codeblock actions don't work in accessible view

Reported by @jooyoungseo, open terminal chat, run a request that yields a code block.
Open the accessible view.
Navigate to within the codeblock and use ctrlCmd+enter.
🐛 nothing happens

@jooyoungseo, I am wondering why you use the accessible view for terminal chat. It should almost always return just the command in a monaco editor, which is screen reader friendly.

Also discovered the terminal chat accessible view content is not correct

https://github.com/user-attachments/assets/42611310-f93b-4b57-bd2b-984794bb00be
Believe it is working as expected now :)


Refactor ES6 challenge descriptions and titles

Pre-Submission Checklist

[x] Your pull request targets the staging branch of freeCodeCamp.
[x] Branch starts with either fix/, feature/, or translate/ (e.g. fix/signin-issue)
[x] You have only one commit (if not, squash them into one commit).
[x] All new and existing tests pass the command npm test. Use git commit --amend to amend any fixes.

Type of Change

[x] Small bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds new functionality)
[ ] Breaking change (fix or feature that would change existing functionality)
[ ] Add new translation (feature adding new translations)

Checklist:

[x] Tested changes locally.
[ ] Closes currently open issue (replace XXXX with an issue no): Closes #XXXX
Progress towards #12698

Description

reformat ES6 challenge descriptions, titles, and seed code where necessary.

@robbawebba Seems like you've got a minor merge conflict on one of the titles. If you resolve it, I'll review this :blush:

@robbawebba Oops, looks like you did a git pull instead of git pull --rebase.
But now it's fixed?

@Greenheart thanks for the heads up! Resolved :)

@Greenheart whoops, I didn't see your 2nd comment - Yeah, I goofed up a little bit with my merging the first time, but it's all good now! 😅

@robbawebba just let me know once you've made those changes and we can merge this.

@QuincyLarson @Greenheart Just added the last suggestion, sorry for the delay!

@robbawebba No worries, I'm just as busy myself... :smile:
I don't have a local instance of fCC at the moment so I can't test this, so you'll need to find someone else to do the final QA.

@Greenheart This PR still needs you approval. Are you satisfied with the changes? Is so can you hit approve?
@freeCodeCamp/moderators Anyone have some free cycles to test this one locally?

@robbawebba Thanks for these improvements. These all look good to me.
@Greenheart thanks for QA'ing!


Broken clang build

https://api.travis-ci.org/jobs/54867976/log.txt
Probably caused by 275d1ab4231cbe804d5

/home/travis/build/TrinityCore/TrinityCore/dep/cppformat/format.h:779:15: note:
candidate function
static yes &check(unsigned char);
              ^
/home/travis/build/TrinityCore/TrinityCore/dep/cppformat/format.h:782:15: note:
candidate function
static yes &check(int);

Should be fixed in https://github.com/cppformat/cppformat/commit/bf8636c9596fbfddded3d0f5879abc7579c9f4dd. Also added a test case in https://github.com/cppformat/cppformat/commit/2adbfa4af7b9bcb437b4341bae06d9d8a766d6b2 to avoid such breakage in the future. Thanks!

I have to thank you ,-)


Why I use the function write(),only saved tag,not saved fields?

Why I use the function write(),only saved tag,not saved fields?

Hey. Sorry, I don't really get what's your issue. You are trying to save fields but they don't get saved? Could come up with a minimal code example?


Support angular ui bootstrap 0.14

Hi @spongessuck ,
The latest angular ui bootstrap versions (0.14.0 and 0.14.1) created <uib-datepicker> and made <datepicker> deprecated.
Please see the changelog here.
As I'm willing to use the latest verions of both angular and angular ui bootstrap, I would like to support both <uib-datepicker> as well.
The v1.0.6 doesn't support it, because you are decorating daypickerDirective and not the new uibDaypickerDirective.
What do you think ?
Note:
I was trying to fix #14 yesterday as it is "blocking" me as well.
I'll see this again later today.

Closed with PR #29 .


📃:  want to add Weather App

:red_circle: Weather App :
:red_circle: want to add Weather App :
:red_circle: HTML, CSS, JS :
Screenshots 📷

will share in the PR!

:white_check_mark: To be Mentioned while taking the issue :

Full name : Psivasai
What is your participant role? (Mention the Open Source Program name. Eg. GSSOC, SSOC, JWOC, etc.)
am a GSSOC contributor please assign me this issue under GSSOC

Happy Contributing 🚀
All the best. Enjoy your open source journey ahead. 😎

@Avdhesh-Varshney please assign me this issue!

https://github.com/Avdhesh-Varshney/WebMasterLog/tree/main/Vanilla-JS-Projects/Intermediate/Weather-App


Add push constants to the engine core for material and local transform selection

 Push Constant

 Data-Oriented T4 Module rewrite


Vagrant build of weaveexec image fails on Ubuntu 15.04 with VirtualBox 4.3.26

Due to Vivid's default use of dnsmasq and this VirtualBox bug, apk intermittently fails to resolve Alpine repository artefacts, triggering #663.

Issue raised just for the record - closing as not our problem.


Fix scaleless chart updates

PR #17 did not account for charts without scales.

This project doesn't seem to be maintained any more.  You can use this package instead, which includes this pull request.
https://www.npmjs.com/package/rc-chartjs


Impl scanner

Fix "MSK" time zone.
Fix Reader.run() to receive large messages. Before this, the message(scanner_parameters) was split into three messages.

Scanner parameters:
+ req_scanner_parameters()
+ decode scanner_parameters
+ parsing xml
+ creating input data for macros
+ created macros for selecting from the desired options on ScannerSubscription.

Scanner subscriptions:
+ ScannerSubscription builder
+ req_scanner_subscription()
+ decode scanner_data

Refused scannerSubscriptionOptions and scannerSubscriptionFilterOptions, leaving only the ScannerSubscription builder (Instrument -> location -> ScanCode -> Filters).
/// # Example
///
/// let scanner_wo_filters = ScannerSubscription::us_stocks().us_major().top_perc_gain();
/// let scanner_w_filters = scanner_wo_filters.clone().market_cap_above1e6(1000.0).price_below(100.0);
///
/// client.req_scanner_subscription(scanner_w_filters).await;

I decode the contract from scanner_data_msg and create the structure:
pub struct ScannerContract {
pub result_number: i32, // remove?
pub contract_id: ContractId,
pub symbol: String,
pub sec_type: ContractType,
pub expiration_date: String,
pub strike: String,
pub class: String, // right
pub exchange: Routing,
pub currency: Currency,
pub local_symbol: String,
pub market_name: String,
pub trading_class: String,

pub distance: String,
pub benchmark: String,
pub projection: String,
pub legs_str: String,
}

Question: should we create another structure, like Contract(enum)?
p.s There is also a scanner for native combos (with legs).

Limiters:
In progress..

Will try to merge this soon


1 instanceof results in "inconsistent use of library definitions" (recursion limit exceeded)

$ ./node_modules/.bin/flow version
Flow, a static type checker for JavaScript, version 0.35.0

$ cat flow-instanceof.js
/* @flow */

class L {
  /*:: ff: Array<F> */
}

class F {}
class VF extends F {}
class AF extends F {}

class A {
  /*:: ll: Array<L> */
  fn() {
    this.ll.forEach((l) => {
      l.ff.forEach((f) => {
        if (f instanceof VF || f instanceof AF) {
          console.log()
        }
      })
    })
  }
}

$ ./node_modules/.bin/flow check
flow-instanceof.js:15
 15:       l.ff.forEach((f) => {
           ^ call of method `forEach`
inconsistent use of library definitions
module. *** Recursion limit exceeded ***
module

Found 1 error

+1. But don't have any information about kind of problem:
$ flow check
test/integration/fixtures/createFixtureManager.js:0
inconsistent use of library definitions
module. *** Recursion limit exceeded ***
module

Found 1 error

File where flow is showing this problem

The issue seemed to be fixed a while ago.


Add UsernsMode to container config

Signed-off-by: Liron Levin<EMAIL_ADDRESS>

Can you link to a PR/Issue where this is relevant?

Thanks @cpuguy83 fix the comment, the issue is docker/docker#20111.

LGTM

LGTM


Add Allen Institute’s Cell Feature Explorer

Mentioned at https://forum.image.sc/t/fast-online-stack-browser/74691/23
E.g. with the link from post above: https://allen-cell-animated.github.io/website-3d-cell-viewer/?url=https://animatedcell-test-data.s3.us-west-2.amazonaws.com/variance/1.zarr
or with IDR example https://allen-cell-animated.github.io/website-3d-cell-viewer/?url=https://uk1s3.embassy.ebi.ac.uk/idr/zarr/v0.4/idr0101A/13457537.zarr
NB: It seems that the viewer only works with 5D data - E.g. https://allen-cell-animated.github.io/website-3d-cell-viewer/?url=https://uk1s3.embassy.ebi.ac.uk/idr/zarr/v0.4/idr0062A/6001240.zarr fails with
Uncaught (in promise) RangeError: too many indices for array; expected 4, got 5
    at checkSelectionLength (core.mjs?3f36:250:1)
    at replaceEllipsis (core.mjs?3f36:334:1)
    at normalizeArraySelection (core.mjs?3f36:277:1)
    at new BasicIndexer (core.mjs?3f36:412:1)
    at ZarrArray.getBasicSelectionND (core.mjs?3f36:2715:1)
    at ZarrArray.getBasicSelection (core.mjs?3f36:2711:1)
    at ZarrArray.get (core.mjs?3f36:2696:1)
    at eval (FetchZarrWorker.js?ed40:65:1)

As of today it expects TCZYX but accepting CZYX or TZYX should be an easy enough change.  I'll probably add that very soon to make that test data work.

Also FYI regarding naming:
Cell Feature Explorer is the dataset presentation app at https://cfe.allencell.org
and the 3D viewer component is a part of it which we called website-3d-cell-viewer but haven't bothered to rename since we started to make it something that could stand alone.  It is in need of new naming since now it can behave as a standalone viewer and also has been embedded into jupyter notebooks as nbvv


CC2652P Ebyte E72-2G4M20S1E not working on HA OS 7.4

I updated my HA OS to 7.4 and out of the blue my zigbee stick with the latest firmware stopped being registered as ttyACM0. It is now ttyUSB0. Why did this change?
View the complete issue here: https://github.com/home-assistant/operating-system/issues/1748
When changing the config to ttyUSB0 I get the following:
> node index.js
Zigbee2MQTT:error 2022-02-10 20:45:09: Configuration is not consistent with adapter state/backup!
Zigbee2MQTT:error 2022-02-10 20:45:09: - PAN ID: configured=6754, adapter=6756
Zigbee2MQTT:error 2022-02-10 20:45:09: - Extended PAN ID: configured=00124b0022811e26, adapter=dddddddddddddddd
Zigbee2MQTT:error 2022-02-10 20:45:09: - Network Key: configured=01030507090b0d0f00020406080a0c0d, adapter=01030507090b0d0f00020406080a0c0d
Zigbee2MQTT:error 2022-02-10 20:45:09: - Channel List: configured=11, adapter=25
Zigbee2MQTT:error 2022-02-10 20:45:09: Please update configuration to prevent further issues.
Zigbee2MQTT:error 2022-02-10 20:45:09: If you wish to re-commission your network, please remove coordinator backup at /config/zigbee2mqtt/coordinator_backup.json.
Zigbee2MQTT:error 2022-02-10 20:45:09: Re-commissioning your network will require re-pairing of all devices!
Zigbee2MQTT:error 2022-02-10 20:45:09: Error while starting zigbee-herdsman
Zigbee2MQTT:error 2022-02-10 20:45:09: Failed to start zigbee
Zigbee2MQTT:error 2022-02-10 20:45:09: Check https://www.zigbee2mqtt.io/guide/installation/20_zigbee2mqtt-fails-to-start.html for possible solutions
Zigbee2MQTT:error 2022-02-10 20:45:09: Exiting...
Zigbee2MQTT:error 2022-02-10 20:45:10: Error: startup failed - configuration-adapter mismatch - see logs above for more information
    at ZnpAdapterManager.determineStrategy (/app/node_modules/zigbee-herdsman/src/adapter/z-stack/adapter/manager.ts:229:35)
    at ZnpAdapterManager.start (/app/node_modules/zigbee-herdsman/src/adapter/z-stack/adapter/manager.ts:63:26)
    at Controller.start (/app/node_modules/zigbee-herdsman/src/controller/controller.ts:123:29)
    at Zigbee.start (/app/lib/zigbee.ts:58:27)
    at Controller.start (/app/lib/controller.ts:100:27)
    at start (/app/index.js:101:5)

Why do you use ttyACM* in the first place?
It is recommended to use device id instead: https://www.zigbee2mqtt.io/guide/configuration/adapter-settings.html
In HAOS, you can check ids even on the GUI: /hassio/system, Host, 3 dots menu, hardware.
I have
serial:
  port: >-
    /dev/serial/by-id/usb-Silicon_Labs_slae.sh_cc2652rb_stick_-_slaesh_s_iot_stuff_00_12_4B_00_22_98_8D_0E-if00-port0

for mine in the config of Zigbee2mqtt addon, and have no problem with 7.4 (on an HP T620, like you)


Create shader manager

Shader manager is an idea and huge performance optimization because now new shader class is instantiated every time the shader needs to be used
The idea is to complete rewrite Shader class and only ShaderManager class will instantiate and create OpenGL objects.
The ShaderManager class will contain requestShader method and will take ENUM which is marking relevant shader as an argument
The shader manager would be singelton
The example usage of such class would be
this->shader = ShaderManager->getInstance()->requestShader(PBR_IBL_TEXTURED_SHADER)

this will prevent from creating useless shader objects as only the reference to the shader itself woudl be needed
this however might be an issues as every new shader creation would have to be done in ShaderManager class which might be painful to write. The benefit of such class would be reflected in how long does it take for the application to start

List of shaders that are being used and what enum should be associated with it
HDR to Cube Map -> converts HDR sky box to the cube map -> SHADER_HDR_2_CUBE_MAP
Irradiance map -> converts environment cube map to irradiance map -> SHADER_GENERATE_IRRADIANCE
Prefilter map -> converts environment map to pre filter map of different roughness levels -> 'SHADER_GENERATE_IRRADIANCE'
'Lut texture shader-> pre calculates BRDF for IBL -> SHADER_GENERATE_BRDF_LUT
Sky box shader -> creates sky box -> SHADER_SKY_BOX
Area light shader -> material with colour PBR and Area Light -> SHADER_AREA_LIGHT_COLOUR
Area light textured -> material with textured PBR pipeline and AreaLight support for IBL is incorporated inside shader -> SHADER_AREA_LIGHT_TEXTURES
Shadow map shader -> renderers only to the depth buffer scene transformed form the light prespective -> SHADER_SHADOW_MAP

Additionally the method request shader will either return the existing shader or create a new if the shader is not created already


GNU/Linux Debian 11 - kernel 5.10.0-18-amd64 - Pulsar v1.1.3 - build from the source fails - error: ‘queue’ was not declared in this scope; did you mean ‘Enqueue’?

I was trying to build the Pulsar v1.1.3 wallet from the source on my GNU/Linux Debian 11 computer and i faced that error:
wget https://github.com/PulsarCoin/Pulsar-Coin-Cryptocurrency/archive/refs/tags/v1.1.3.tar.gz
tar -xvf v1.1.3.tar.gz
cd Pulsar-Coin-Cryptocurrency-1.1.3/
./contrib/install_db4.sh `pwd`
export BDB_PREFIX='/home/ceedii/Pulsar-Coin-Cryptocurrency-1.1.3/db4'
./autogen.sh
./configure BDB_LIBS="-L${BDB_PREFIX}/lib -ldb_cxx-4.8" BDB_CFLAGS="-I${BDB_PREFIX}/include" --enable-hardening --without-gui
make -j8
Making all in src
make[1]: Entering directory '/home/ceedii/Pulsar-Coin-Cryptocurrency-1.1.3/src'
make[2]: Entering directory '/home/ceedii/Pulsar-Coin-Cryptocurrency-1.1.3/src'
  CXX      pulsard-bitcoind.o
  CXX      libbitcoin_server_a-addrdb.o
  CXX      libbitcoin_server_a-addrman.o
  CXX      libbitcoin_server_a-alert.o
  CXX      libbitcoin_server_a-bloom.o
  CXX      libbitcoin_server_a-blockencodings.o
  CXX      libbitcoin_server_a-chain.o
  CXX      libbitcoin_server_a-checkpoints.o
  CXX      libbitcoin_server_a-httprpc.o
  CXX      libbitcoin_server_a-httpserver.o
  CXX      libbitcoin_server_a-init.o
httpserver.cpp:74:10: error: ‘deque’ in namespace ‘std’ does not name a template type
   74 |     std::deque<std::unique_ptr<WorkItem>> queue;
      |          ^~~~~
httpserver.cpp:33:1: note: ‘std::deque’ is defined in header ‘<deque>’; did you forget to ‘#include <deque>’?
   32 | #include <support/events.h>
  +++ |+#include <deque>
   33 |
httpserver.cpp: In member function ‘bool WorkQueue<WorkItem>::Enqueue(WorkItem*)’:
httpserver.cpp:92:13: error: ‘queue’ was not declared in this scope; did you mean ‘Enqueue’?
   92 |         if (queue.size() >= maxDepth) {
      |             ^~~~~
      |             Enqueue
httpserver.cpp:95:9: error: ‘queue’ was not declared in this scope; did you mean ‘Enqueue’?
   95 |         queue.emplace_back(std::unique_ptr<WorkItem>(item));
      |         ^~~~~
      |         Enqueue
httpserver.cpp: In member function ‘void WorkQueue<WorkItem>::Run()’:
httpserver.cpp:106:35: error: ‘queue’ was not declared in this scope; did you mean ‘Enqueue’?
  106 |                 while (running && queue.empty())
      |                                   ^~~~~
      |                                   Enqueue
httpserver.cpp:110:31: error: ‘queue’ was not declared in this scope; did you mean ‘Enqueue’?
  110 |                 i = std::move(queue.front());
      |                               ^~~~~
      |                               Enqueue
make[2]: *** [Makefile:6165: libbitcoin_server_a-httpserver.o] Error 1
make[2]: *** Waiting for unfinished jobs....
In file included from /usr/include/boost/smart_ptr/detail/sp_thread_sleep.hpp:22,
                 from /usr/include/boost/smart_ptr/detail/yield_k.hpp:23,
                 from /usr/include/boost/smart_ptr/detail/spinlock_std_atomic.hpp:18,
                 from /usr/include/boost/smart_ptr/detail/spinlock.hpp:36,
                 from /usr/include/boost/smart_ptr/detail/spinlock_pool.hpp:25,
                 from /usr/include/boost/smart_ptr/shared_ptr.hpp:29,
                 from /usr/include/boost/shared_ptr.hpp:17,
                 from /usr/include/boost/signals2/signal.hpp:21,
                 from ./util.h:31,
                 from ./addrman.h:14,
                 from init.cpp:12:
/usr/include/boost/bind.hpp:36:1: note: ‘#pragma message: The practice of declaring the Bind placeholders (_1, _2, ...) in the global namespace is deprecated.


[Improvement] Ensure that HttpUtils can only get result from certification URL

Ensure that HttpUtils can only access secure URL

please run ./mvnw spotless:check to format the code

please run ./mvnw spotless:check to format the code

has updated, please help check

Codecov Report
Attention: 9 lines in your changes are missing coverage. Please review.

Comparison is base (0c470ff) 38.16% compared to head (b76c230) 38.14%.

:exclamation: Current head b76c230 differs from pull request most recent head 85a4985. Consider uploading reports for the commit 85a4985 to get more accurate results

Files
Patch %
Lines

...pache/dolphinscheduler/common/utils/HttpUtils.java
79.06%
7 Missing and 2 partials :warning:

Additional details and impacted files
@@             Coverage Diff              @@
##                dev   #15288      +/-   ##
============================================
- Coverage     38.16%   38.14%   -0.03%
+ Complexity     4666     4657       -9
============================================
  Files          1278     1278
  Lines         44482    44431      -51
  Branches       4783     4759      -24
============================================
- Hits          16978    16946      -32
+ Misses        25641    25625      -16
+ Partials       1863     1860       -3

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

@zhongjiajie updated, please take a look!


quartata: Blacklist Pure\W?Radiance

quartata requests the blacklist of the keyword Pure\W?Radiance. See the Metasmoke search here

!!/approve


Add analysis queries

Some examples here: https://adswerve.com/blog/all-about-jobs-information-schema-and-biqquery-processing-costs/


TAP support

Should be a flag, like $ ava --tap, to get TAP output. This would be a good way to enable reporters without actually supporting reporters. Can just refer people to all the available TAP reporters instead of reinventing the wheel.

I know that a PR is better than this question, BUT, is this still a priority?

:+1:

From my reading of the TAP spec, I it does not seem very well suited to concurrency. You would need to buffer individual tests and then log them all at once so they don't walk all over each-other.

Coming Soon!

TAP support is ready, waiting for #279.

After I saw this closed by @vdemedes I assumed it was part of 0.7.0. But didn't see it listed https://github.com/sindresorhus/ava/releases/tag/v0.7.0. Since this issue is still part of 0.7.0, does that mean that it's in 0.7.0 and hence forgotten about in the release announcement or that a new milestone should be associated with this issue?

@jprichardson - We decided to push #279 to the next milestone.

It was closed by a mistake.

@jamestalmage From my reading of the TAP spec, I it does not seem very well suited to concurrency.

That is the case certainly, I am happy to discuss some of my suggestions that have been aired on TAP specification discussions however likely won't be adopted very quickly at all (Having a different output might be simpler)


Minor changes for PR 252

Minor changes from the review of MHKiT-Software/MHKiT-Python#252:

minor linting
module docstring
update parameter validation

@mbruggs Oops thanks for catching that. I copied the formatting from another module and forgot to remove all the extraneous items. Also, Sterling and I discussed location today and landed on adding the upcrossing module to mhkit.utils. I'll make that change when I fix the docstring later.

@mbruggs I removed the extraneous doc string, moved the module into utils, and updated the places that upcrossing is imported. Let me know if you have any other suggestions.
The files changed increased drastically because I pulled in develop so that I could edit the updated utils module. Those changes should not appear in the original #252


.Breadcrumb {} Component

Add also some extending styles for plugins such as https://fi.wordpress.org/plugins/breadcrumb-navxt/


pull_from_upstream + changelog-entry action = nope, still got upstream commit titles

What happened? What is the problem?
I set up a changelog-entry action for my package:
https://src.fedoraproject.org/rpms/python-docker/c/dc64b29ab79824525f0bd12fe986a0a73e96fb84?branch=rawhide
Because I wanted the plain new upstream release $version changelog entries
But I was still getting the upstream commit messages from the last tag
https://src.fedoraproject.org/rpms/python-docker/pull-request/42#request_diff
What am I missing?
What is the impacted category (job)?
Fedora release automation

I suspect the changelog action is not run: https://dashboard.packit.dev/results/pull-from-upstream/3004

Was curious so I checked the logs:
Package config before loading:
{
    "specfile_path": "python-docker.spec",
    "downstream_package_name": "python-docker",
    "upstream_project_url": "https://github.com/docker/docker-py",
    "actions": {
        "create-archive": [
            "python3 setup.py sdist --dist-dir .",
            "ls -1 ./*.tar.gz"
        ],
        "get-current-version": "python3 setup.py --version"
    },
    "issue_repository": "https://github.com/TomasTomecek/docker-py",
    "jobs": [
        {
            "job": "pull_from_upstream",
            "trigger": "release",
            "dist_git_branches": [
                "fedora-all",
                "epel-8",
                "epel-9"
            ]
        }
    ]
}

and
Found a config file '.packit.yaml' on ref '87d8767f73c1d64a3121d2eb2f9c1aef1dd13528' of the 'rpms/python-docker' repository.
So the version of the configuration we obtained did not include that action at that moment (looks like the config was taken from the target branch of the PR with the comment).
I would therefore close this as a duplicate of #2115 if you are not against.

agreed! thank you for investigating


Harmony: Ensure Scene Settings not working

Bug
Ensure Scene Settings is not working as it is calling method on wrong object - it should be avalonClient not avalon_client.
https://github.com/pypeclub/avalon-core/blob/b7faea76927ba40f69f229f3d7d778896285b0f3/avalon/harmony/TB_sceneOpened.js#L481-L491

Fixed by pypeclub/avalon-core#251


inidicator shape still exist when go back

i have two pages, first page to take the picture second page to show the picture that has been taken, when i went to the second page, and go back to the first page which is the camera page, the indicator still exist and not moving anymore. How to fix it? thanks
import 'dart:io';

import 'package:face_camera/face_camera.dart';
import 'package:flutter/material.dart';

void main() async {
  WidgetsFlutterBinding.ensureInitialized();
  await FaceCamera.initialize();
  runApp(const MyApp());
}

class MyApp extends StatelessWidget {
  const MyApp({super.key});

  @override
  Widget build(BuildContext context) {
    return const MaterialApp(
      home: FirstPage(),
    );
  }
}

class FirstPage extends StatefulWidget {
  const FirstPage({super.key});

  @override
  State<FirstPage> createState() => _FirstPageState();
}

class _FirstPageState extends State<FirstPage> {
  late FaceCameraController controller;

  @override
  void initState() {
    super.initState();
    controller = FaceCameraController(
      autoCapture: false,
      defaultCameraLens: CameraLens.front,
      enableAudio: false,
      performanceMode: FaceDetectorMode.fast,
      imageResolution: ImageResolution.medium,
      onCapture: _takeImage,
      defaultFlashMode: CameraFlashMode.off,
    );
  }

  @override
  void dispose() {
    controller.dispose();
    super.dispose();
  }

  void _takeImage(File? image) {
    Navigator.push(
        context,
        MaterialPageRoute(
          builder: (context) => SecondPage(image: image!),
        ));
  }

  @override
  Widget build(BuildContext context) {
    return Scaffold(
      body: SmartFaceCamera(controller: controller),
    );
  }
}

class SecondPage extends StatelessWidget {
  const SecondPage({super.key, required this.image});

  final File image;

  @override
  Widget build(BuildContext context) {
    return Scaffold(
      body: Center(
        child: Image.file(image),
      ),
    );
  }
}

I think u Should Navigator Page2 First => Navigator.push to Page1 after that return Images to Page2

yes, it does work, but thats kinda the hacky way no? Shouldnt it be fix since it is not the behavioral we wanted? and it will become problem in the future also for example if the user go back from camera page

@andikatp  Can u give me how you Navigator in your app? I has see you back to page 1?
I think problem in your code, You have Navigator screen 1 to Screen2 by push So when u pop to screen1, screen 1 still alive so Draw indicator still working so it have error.
You can try with Navigator.replace?

my code is basically the same as my code that ive provided you, the problem also occurs when i show dialog to user, and when the user closes the dialog, the indicator not showing up anymore
faceController = FaceCameraController(
      autoCapture: false,
      defaultCameraLens:
          FaceCamera.cameras.length > 1 ? CameraLens.front : CameraLens.back,
      enableAudio: false,
      performanceMode: FaceDetectorMode.fast,
      imageResolution: ImageResolution.medium,
      onCapture: _takeImage,
      defaultFlashMode: CameraFlashMode.off,
      onFaceDetected: (_) {},
      orientation: CameraOrientation.portraitUp,
    );

 void _takeImage(File? image) async {
    if (image == null) return;
     await Get.defaultDialog(
    title: title,
    titleStyle: TextStyle(fontSize: 16.sp, fontWeight: FontWeight.bold),
    content: Text(
      message,
      textAlign: TextAlign.center,
      style: TextStyle(fontSize: 14.sp),
    ),
    titlePadding: EdgeInsets.symmetric(vertical: 20.h),
    contentPadding:
        EdgeInsets.symmetric(horizontal: 20.h).copyWith(bottom: 20.h),
  );
    }

after i close the dialog, the indicator will not show up anymore,

I thank you should create new Controller and setState this. I think problem will resvole

@andikatp To resolve the issue with the camera indicator not moving when returning to the first page, try calling await controller.startImageStream(); to restart the image stream. This should reactivate the camera and the indicator. If this solution doesn't work, please reach out for further assistance.

I love you so much after trial and error i capable to do controller.startImageStream when pop thank you for your answer and time @Conezi and also to you @tanpham380 for trying to help!!!


Do we need to provide the poses_bounds.npy file for our own custom multiview dataset

Yes, it is camera pose. Training a 4DGS without camera poses is also a research problem.

Thank you


[Feature] Request for python scripts to extract data from official SUNRGBD dataset

What is the feature?

About the feature
Extract RGBD data and split info from the official SUNRGBD dataset using python scripts.
How it can help
Convenient for users without Matlab to work with SUNRGBD dataset.

Any other context?
I will try to complete this Python script by referencing the Matlab code in this repo, and I'll submit a PR so that more people who don't have Matlab like me could use it.

@MengqingCao Hi, it's a good idea to transfer matlab code into python. Do you mind sharing your efforts with me? Thanks!


No indication for tappables and scrollables

I have received feedback from viewers that the catwalk UI does not provide a significant indication that tells the user that an area can be scrolled or tapped. It is also pointed out that scrolling with the horizontal bar is annoying.

I've added buttons to show scroll possibility. I wonder if it's still a problem.


feat: square and colored props

Hi @wdtamagi , I just added a new feature to make the Avatar component a square, and  randomly colored background if no image.
Hope you like it! 😉

Hello @al-fin
Thanks for your contribution, these features are nice to have :)
Just a minor comments for your PR.

@wdtamagi, Please check my recent commit, I've fixed it 😁


update live with master

You might have squashed a previous merge causing this conflicts?

@mairaw I think that might be true. I'll fix this one by hand.

closing and reopening for a fresh build.


point in polygon : point count is erroneous as half of the hex lies outside of polygon

Hi ,
My use case is to find points in polygon and show counts.
I am solving this using making hex of certain resoltion  and club points then use centroid of that hex to find in which polygon it exist.
Below code is basically main logical part to check if  center(centroid of hex) exist in polygon or not
h3_fire_idx ON ST_WITHIN(center, "Shapes"."shape_simplified_5")
Problem is sometime a hex centroid exists in polygon but remaining half of hex exists outside of polygon , end up giving me wrong counts of points in polygon.
Is there any way to reduce this error without changing resolution of all the hex? Like can I dynamically split one big hex that is on the edge of polygon to multiple smaller hex and then get count from smaller hex. I think that will be more accurate.
All this should happen in one psql query.
Thanks,

Hi rahul! If you haven't already I would take a look at the polyfill and compact functions. I would also try some of the more official H3 sources:

Posting to the H3 tag on StackOverflow
There is also an H3 Slack workspace

I'm afraid that's all the help I can offer :-) Feel free to open another issue or PR if you find any bugs or other problems.


Fix: Word based batching memory with longer source sentences.

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

This regression was introduced in the following PR by me:
https://github.com/awslabs/sockeye/pull/378
While everything else in define_bucket_batch_sizes is defined on the target side we need to make sure that in scenarios where the source is large than the target the source input data of shape (batch_size, len_source) can be covered by the default bucket batch size. Otherwise MXNet is not able to reuse memory:
 Bucketing: data source has a shape [59,40,1], which is larger than already allocated shape [22,100,1]. Need to re-allocate. Consider putting default bucket key to be the bucket taking the largest input for better memory sharing.


Probe and print BIOS vendor and version on deprov

This currently only works for our Gen3 Dell and Supermicro servers as a
starting point. Eventually this will evolve into the ability to
validate BIOS configuration settings on deprov and reset them if drift
is detected.
This is WIP and I'm not sure this needs to be merged to master at all. I am however still interested in feedback on the approach itself and will test this as a custom OSIE version while doing further development.

Closing, this PR is obsolete


WIP: Sagemaker Example

Description
Sagemaker example for using whylogs with Whylabs
General Checklist

[ ] Tests added for this feature/bug
if it was a bug, test must cover it.
[ ] Conform by the style guides, by using formatter
[ ] Documentation updated
[ ] (optional) Please add a label to your PR

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 remained the same at 81.533%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
3620

Relevant Lines:
4281

💛  - Coveralls


Remove version: RossScroggs.GAMADV-XTD3 version 6.53.03

Duplicate and unmaintained manifests of taers232c.GAMADV-XTD3
Microsoft Reviewers: Open in CodeFlow

Response status code does not indicate success: 401 (Unauthorized).
(Automated message - build 585)


Add strategies folders to CI/CD workflows docs update

Description:

This updates the CI/CD workflows to be equal to the pre-commit configuration for updating the documentation.
This is a fix for the pre-commit configurations updated in PR #313
Type of change:

[x] Bug fix.
[ ] New feature.
[x] Documentation update.

Checklist for the reviewer:

This checklist should be used as a help for the reviewer.

[ ] Is the change limited to one issue?
[ ] Does this PR close the issue?
[ ] Is the code easy to read and understand?
[ ] Do all new feature have an accompanying new test?
[ ] Has the documentation been updated as necessary?

Codecov Report
Patch and project coverage have no change.

Comparison is base (9ea6288) 82.17% compared to head (1a9c27d) 82.17%.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@           Coverage Diff           @@
##           master     #323   +/-   ##
=======================================
  Coverage   82.17%   82.17%
=======================================
  Files          44       44
  Lines        1212     1212
=======================================
  Hits          996      996
  Misses        216      216

Flag
Coverage Δ

linux
82.09% <ø> (ø)

linux-extra_libs
82.09% <ø> (ø)

windows
81.63% <ø> (ø)

windows-extra_libs
81.63% <ø> (ø)

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

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


[WIP] PhpSpec

Thinking about moving tests to PhpSpec as it is easier to read and understand.

I would be happy to contribute to this one as well, but rebasing it would be a nightmare.

Yeah we should proberly throw it away and restart it.

This will require a major rewrite. Closing for now as it cannot be merged.


Cannot view stream using the add tool

Hi, using the add tool in the go2rtc UI, I find this stream url:
onvif://admin:123456@<IP_ADDRESS>:80/onvif/device_service%20%20https://<IP_ADDRESS>:443/onvif/device_service?subtype=MainStream

If I put it if the config file:

I get no feed and this in the log:

This feed also does not work with VLC
Looking on the internet I find that I can use this url:
rtsp://admin:123456@<IP_ADDRESS>:554/Streaming/Channels/101/
This url feeds properly on VLC, but generates the same error in the go2rtc UI.
It has been 2 days of looking around, @AlexxIT  I'm happy to buy you a bunch of you coffee if you can help me!

Some strange ONVIF realisation in your camera. Stream should be:
onvif://admin:123456@<IP_ADDRESS>:443/onvif/device_service?subtype=MainStream


RFC8246  HTTP Immutable Responses

This commit adds the functionality to handle the 'immutable' directive in the Cache-Control header as per the RFC specifications.
Key changes include:

Add no-transform Control-Cache support.
The 'immutable' directive is checked in the Cache-Control of an HTTP response, indicating that the origin server will not update the resource representation during the response's freshness lifetime.
If the 'immutable' directive is present and the response is still fresh, the response is considered cacheable without further validation.
Ignoring any arguments with the 'immutable' directive, as per RFC stipulations.
Treating multiple instances of the 'immutable' directive as equivalent to one.

You're absolutely right. This change pertains to the rfc8246.

@ok2c. You're absolutely right. This change pertains to the rfc8246.

@arturobernalg Could you please reflect this in the PR title and the commit message?

@ok2c. You're absolutely right. This change pertains to the rfc8246.

@arturobernalg Could you please reflect this in the PR title and the commit message?

@ok2c
Done.
The commit message doesn't need many changes. In any case, I've made a reference to the RFC.

@arturobernalg Looks good to me, but can we drop no-transform out of the change-set? It is unrelated to the stated objective of the change-set and is presently not used anywhere in the caching logic.

@arturobernalg Looks good to me, but can we drop no-transform out of the change-set? It is unrelated to the stated objective of the change-set and is presently not used anywhere in the caching logic.

@ok2c  Removed.


Dockerhub Continuous Integration

Build and Push

The Build action has been replaced with a Build and Push Action
This new action now builds the docker file and pushes it to joachimbbp/openvdb on dockerhub
As per the previous Build action, Build and Push is triggered on changes to the dockerfile that are pushed to main

Github is preventing me from merging this on my own. Pushing to main from terminal instead.


Deprecation of positional arguments on Import, New Browser, New Context & New Persistent Context

Positional arguments are nice when there are three or four.
But Keywords like New Context have 30 or even more arguments.
Positional arguments are totally useless in that case.
But named arguments are pretty good.
The problem was, that because we had normal arguments, they could be used positional and named.
Therefore the order was fixed. We could not order them alphabetically or add a new arguments somewhere in the middle.
The fix is to switch all arguments to "named-only-arguments"!
We are hereby deprecating positional arguments (or the most of them) on the before mentioned keywords.
They can still be used in the old order, but shall be changed to positional arguments asap.
Support will be remove early 2023.
fixed in #2308

Fixed by #2308


BUG:hover issue

Is there an existing issue for this?

[X] I have searched the existing issues

What happened?
when i hover on the icon buttons then it is not display perfectly ,
i think we should to change some :

in this route :https://community-site-theta.vercel.app/profile.html#settings
Record

[X] I agree to follow this project's Code of Conduct
[X] I'm a GSSOC'24 contributor
[X] I want to work on this issue

@Its-Aman-Yadav ,assign me

@Its-Aman-Yadav  I want to work on this issue


Feature: v0.0.17

Support AWS IRSA (integrate pod identity webhook)
Support AWS IAM Authenticator (integrate latest aws iamauthenticator), using CRD to manage IAM Mapping
K8S audit function become default feature
Example for AWS IRSA and IAM Authenticator

@kairen merge this please, thanks ^^


Clarify top-level self's description

As I discovered in the process of opening #500, reading the spec, and then convincing myself that #500 wasn't necessary, the current definition of the top-level self link is a little vague. This PR aims to fix that.

:+1:


Install from softwarecollections.org on centos7.1 fails with 1.12.0 but is ok on 1.11.0

Hi,
I followed this procedure to install python33 : "To use this module with Linux distributions in the Red Hat family and python distributions from softwarecollections.org, set python::provider to 'rhscl' and python::version to the name of the collection you want to"
So I have this in hiera:
python::version: "python33"
python::provider: "rhscl"
With the 1.11.0 version of the module, it works perfectly, and I can get python installed.
But with the 1.12.0, I get this error: Error: Execution of '/bin/yum -d 0 -e 0 -y list pythonpython33' returned 1: Error: No matching Packages to list
Thanks

I'm seeing this issue as well.
It looks like it was caused by 301b3cb which says it was supposed to fix #287. It is prefixing the python package name with python if $::python::version is not equal to system or pypy.
On CenOS 7, I'm expecting to set $::python::version to python33 or rh-python34.

Same issue here with RHEL 6.6.
As a test I also tried to set $python::version to '27' contrary to what it says in the README, but then the python does not get prepended so that the module attempts to fetch an RPM called rhscl-27-epel-6-x86_64 instead of rhscl-python27-epel-6-x86_64, and of course that results in an error 404. So it seems we can't win 😛  As it turns out I was just testing something locally so I don't need 1.12.0, and indeed this works fine with 1.11.0.

from issue 353:
have this issue, what if we change the source in install.pp?
rhscl: {
rhscl is RedHat SCLs from softwarecollections.org
if $::python::rhscl_use_public_repository {
$scl_package = "rhscl-${::python::version}-epel-${::operatingsystemmajrelease}-${::architecture}"
package { $scl_package:
source => "https://www.softwarecollections.org/en/scls/rhscl/${::python::version}/epel-${::operatingsystemmajrelease}-${::architecture}/download/${scl_package}.noarch.rpm",
provider => 'rpm',
tag => 'python-scl-repo',
}
}
the above source returns 404 if copied and pasted into a browser. but the following does not:
https://www.softwarecollections.org/repos/rhscl/rh-python34/epel-7-x86_64/noarch/rhscl-rh-python34-epel-7-x86_64-1-2.noarch.rpm[](url)


package installation failed - R 4.0

@mvankessel-EMC  One of the data partners tried to install the latest version of TreatmentPatterns but it failed with the error below.
I think you need to set the minimum version of R in the DESCRIPTION to 4.1 since you are using the native pipe operator which has been introduced in that version.
See also
https://github.com/harrelfe/Hmisc/issues/159
Error: Error installing package 'TreatmentPatterns':
=============================================

* installing *source* package ‘TreatmentPatterns’ ...
** package ‘TreatmentPatterns’ successfully unpacked and MD5 sums checked
** using staged installation
** R
Error in parse(outFile) :
  /data01/Rtmp5jfwZe/R.INSTALL26fffa5b9f8087/TreatmentPatterns/R/createSankeyDiagram.R:209:32: unexpected '>'
208:
209:   n <- treatmentPathways$path |>
                                    ^
ERROR: unable to collate and parse R files for package ‘TreatmentPatterns’
* removing ‘/home/projects/darwin/studies/year2/P2-C1-008-MDD/P2-C1-008-DUS-MDD-main/renv/staging/1/TreatmentPatterns’
install of package 'TreatmentPatterns' failed [error code 1]

Thanks @ginberg, in the SRS it specifically states that TreatmentPatterns is at least compatible with R 4.2.1. DrugUtilization and IncidencePrevalence state the same version (I'm assuming more packages as well).
I'll update the minimum version to 4.2.1 in the DESCRIPTION-file.

Updated in 7573aa3a175112704cba604cc0c08a0fc36ffaf6


Album Cover Sorting options documentation

This might be WIP, but the new  Album Cover Sorting options are a bit technical:
[{"method":30,"ascending":false},{"method":20,"ascending":false},{"method":40,"ascending":false}]
What are all the options? What do they mean :)
Screenshots (optional)

Environment (please complete the following information):

OS: Docker container on Debian / LMDE client
Browser: Chromium

Used app version:

docker-nightly

should be already fixed.

I use the latest version I can find (nightly/edge bpatrik/pigallery2:nightly-debian-buster), built today. Must be something in my setup then. I have done a few things:

removed any relevant settings from the Docker compose environment
stop container
deleted config.json
start container

With a fresh config file the admin page stays the same.
This is the (I think) relevant part from my new config.json:
'
"//[AlbumCover]": [
"Specify a search query and sorting that the app can use to pick the best photo for an album and folder cover. There is no way to manually pick folder and album cover in the app. You can tag some of your photos with 'cover' and set that as search query or rate them to 5 and set sorting to descending by rating."
],
"AlbumCover": {
"//[SearchQuery]": [
"Filters the sub-folders with this search query. If filter results no photo, the app will search again without the filter."
],
"SearchQuery": {
"type": 100,
"text": ""
},
"//[Sorting]": [
"If multiple cover is available sorts them by these methods and selects the first one."
],
"Sorting": [
{
"method": "Rating",
"ascending": false
},
{
"method": "Date",
"ascending": false
},
{
"method": "PersonCount",
"ascending": false
}
]
},
'

Indeed you are right. Obfuscation broke my reflection code. Latest commit should fix it.
FYI: I renamed nightly to edge today. Same thing just different name.

Thank you! And, thank you for all the improvements the last few weeks!


support doi as filename for DOI downloads

This change allows for using a paper's unique DOI as the filename in place of the paper title.

Thanks!
Will be included on the next release


nftables: restarting nftables.service destroys rules created by other services

Describe the bug
Restarting the nftables.service will clear tables that are set up by other services such as systemd-networkd or e.g. tailscaled.
Steps To Reproduce
Steps to reproduce the behavior:

Add a systemd-networkd network with a networkConfig.IPMasquerade - "both" option;
Observe networkd creating a table ip io.systemd.nat, with sudo nft list tables
sudo systemctl restart nftables
sudo nft list tables and observe that table ip io.systemd.nat is now gone.

Expected behavior
Restarting the nftables service should not remove the tables from other services, or it should ensure the services know to set up their tables again.
The latter seems pretty hard to achieve however. For networkd networkctl reconfigure $interface will reset these rules (sudo systemctl reload systemd-networkd.service on the other hand will not) so NixOS will know to go through all the interfaces affected interfaces.
For tools like tailscaled the only way I know right now involves at least a full reconfiguration (so a service restart most likely.)
Notify maintainers
No meta.maintainers in nftables firewall module
Metadata
Please run nix-shell -p nix-info --run "nix-info -m" and paste the result.

system: "x86_64-linux"
host os: Linux 6.1.68, NixOS, 24.05 (Uakari), 24.05.20240131.b8b232a
multi-user?: yes
sandbox: yes
version: nix-env (Nix) 2.18.1
nixpkgs: /run/current-system/nixpkgs

Add a :+1: reaction to issues you find important.

There's an option for it. Please set networking.nftables.flushRuleset to false. This option is set to false by default since stateVersion "23.11".


ToastNotification.Group property setter has undocumented maximum lengths

There is a maximum length for strings assigned to the ToastNotification.Group which is undocumented.  On RS1 this is a 16 character maximum length.  Above RS1 it is 64 characters.  Calling the setter with a string that is too long will result in an exception.
This should be documented.
See: https://wpdev.uservoice.com/forums/110705-universal-windows-platform/suggestions/8629408-max-length-of-toast-tag-not-enough-for-a-simple-gu
ToastNotification.Group

This is documented in the Remarks section of the page.


Provisioner with higher weight not used

Description
Observed Behavior:
Our default provisioner with weight 1 was unused
Expected Behavior:
Default provisioner used, because given the following:

the pod that meets the criteria of two provisioners
the pod does not have node affinity/anti-affinity
the provisioner does not run into capacity problems

then the provisioner with the highest weight will always be picked. Please correct me if I'm wrong.
Reproduction Steps (Please include YAML):
There are other provisioners besides those listed here, but the only provisioner with a weight is the default one.
default:
apiVersion: karpenter.sh/v1alpha5
kind: Provisioner
metadata:
  annotations:
    kubectl.kubernetes.io/last-applied-configuration: |
      {"apiVersion":"karpenter.sh/v1alpha5","kind":"Provisioner","metadata":{"annotations":{},"labels":{"argocd.argoproj.io/instance":"karpenter-provisioners"},"name":"default"},"spec":{"consolidation":{"enabled":true},"limits":{"resources":{"cpu":"500","memory":"500Gi"}},"providerRef":{"name":"default"},"requirements":[{"key":"karpenter.sh/capacity-type","operator":"In","values":["on-demand"]},{"key":"kubernetes.io/os","operator":"In","values":["linux"]},{"key":"kubernetes.io/arch","operator":"In","values":["amd64"]},{"key":"karpenter.k8s.aws/instance-category","operator":"In","values":["c","m","r","t"]},{"key":"karpenter.k8s.aws/instance-generation","operator":"Gt","values":["2"]}],"weight":1}}
  creationTimestamp: "2023-05-25T17:52:52Z"
  generation: 3
  labels:
    argocd.argoproj.io/instance: karpenter-provisioners
  name: default
  resourceVersion: "46070546"
  uid: 652bd95b-c314-41a2-bdfe-aedd67064f4c
spec:
  consolidation:
    enabled: true
  limits:
    resources:
      cpu: "500"
      memory: 500Gi
  providerRef:
    name: default
  requirements:
  - key: karpenter.sh/capacity-type
    operator: In
    values:
    - on-demand
  - key: kubernetes.io/os
    operator: In
    values:
    - linux
  - key: kubernetes.io/arch
    operator: In
    values:
    - amd64
  - key: karpenter.k8s.aws/instance-category
    operator: In
    values:
    - c
    - m
    - r
    - t
  - key: karpenter.k8s.aws/instance-generation
    operator: Gt
    values:
    - "2"
  weight: 1
status:
  resources:
    attachable-volumes-aws-ebs: "25"
    cpu: "4"
    ephemeral-storage: 67096556Ki
    memory: 16226780Ki
    pods: "58"

the provisioner that was picked instead:
apiVersion: karpenter.sh/v1alpha5
kind: Provisioner
metadata:
  annotations:
    kubectl.kubernetes.io/last-applied-configuration: |
      {"apiVersion":"karpenter.sh/v1alpha5","kind":"Provisioner","metadata":{"annotations":{},"labels":{"argocd.argoproj.io/instance":"karpenter-provisioners"},"name":"provisioner-exp-arm"},"spec":{"limits":{"resources":{"cpu":"10000","memory":"80000Gi"}},"providerRef":{"name":"default-nvme"},"requirements":[{"key":"karpenter.sh/capacity-type","operator":"In","values":["on-demand","spot"]},{"key":"kubernetes.io/os","operator":"In","values":["linux"]},{"key":"kubernetes.io/arch","operator":"In","values":["arm64"]},{"key":"karpenter.k8s.aws/instance-category","operator":"In","values":["c","m","r","x"]},{"key":"karpenter.k8s.aws/instance-cpu","operator":"Lt","values":["64"]},{"key":"karpenter.k8s.aws/instance-size","operator":"NotIn","values":["metal"]},{"key":"karpenter.k8s.aws/instance-generation","operator":"Gt","values":["2"]}],"ttlSecondsAfterEmpty":120}}
  creationTimestamp: "2023-08-24T19:11:36Z"
  generation: 2
  labels:
    argocd.argoproj.io/instance: karpenter-provisioners
  name: provisioner-exp-arm
  resourceVersion: "39014488"
  uid: 99cb20b2-e3e5-4213-9ff4-b794d6688a9f
spec:
  limits:
    resources:
      cpu: 10k
      memory: 80000Gi
  providerRef:
    name: default-nvme
  requirements:
  - key: karpenter.sh/capacity-type
    operator: In
    values:
    - on-demand
    - spot
  - key: kubernetes.io/os
    operator: In
    values:
    - linux
  - key: kubernetes.io/arch
    operator: In
    values:
    - arm64
  - key: karpenter.k8s.aws/instance-category
    operator: In
    values:
    - c
    - m
    - r
    - x
  - key: karpenter.k8s.aws/instance-cpu
    operator: Lt
    values:
    - "64"
  - key: karpenter.k8s.aws/instance-size
    operator: NotIn
    values:
    - metal
  - key: karpenter.k8s.aws/instance-generation
    operator: Gt
    values:
    - "2"
  ttlSecondsAfterEmpty: 120
status:
  resources:
    attachable-volumes-aws-ebs: "39"
    cpu: "32"
    ephemeral-storage: 67087340Ki
    memory: 64655500Ki
    pods: "234"

The pod that is on the wrong node:
Name:                 coredns-79989457d9-nljqh
Namespace:            kube-system
Priority:<PHONE_NUMBER>
Priority Class Name:  system-cluster-critical
Service Account:      coredns
Node:                 ip-10-192-123-204.ec2.internal/<IP_ADDRESS>
Start Time:           Tue, 05 Sep 2023 10:14:43 -0700
Labels:               eks.amazonaws.com/component=coredns
                      k8s-app=kube-dns
                      pod-template-hash=79989457d9
Annotations:          eks.amazonaws.com/compute-type: ec2
                      kubernetes.io/psp: eks.privileged
Status:               Running
IP:                   <IP_ADDRESS>
IPs:
  IP:           <IP_ADDRESS>
Controlled By:  ReplicaSet/coredns-79989457d9
Containers:
  coredns:
    Container ID:  containerd://b01fc7a66291fbbd9d41bca03bccb95f35a867105d4cd89dad366e8e43780d09
    Image:<PHONE_NUMBER>52.dkr.ecr.us-east-1.amazonaws.com/eks/coredns:v1.8.7-eksbuild.3
    Image ID:<PHONE_NUMBER>52.dkr.ecr.us-east-1.amazonaws.com/eks/coredns@sha256:871a47b5773bbde084c8e9f12e4321a168ddd578521ffae8b55b3dd982ddc66a
    Ports:         53/UDP, 53/TCP, 9153/TCP
    Host Ports:    0/UDP, 0/TCP, 0/TCP
    Args:
      -conf
      /etc/coredns/Corefile
    State:          Running
      Started:      Tue, 05 Sep 2023 10:14:44 -0700
    Ready:          True
    Restart Count:  0
    Limits:
      memory:  170Mi
    Requests:
      cpu:        100m
      memory:     70Mi
    Liveness:     http-get http://:8080/health delay=60s timeout=5s period=10s #success=1 #failure=5
    Readiness:    http-get http://:8080/health delay=0s timeout=1s period=10s #success=1 #failure=3
    Environment:  <none>
    Mounts:
      /etc/coredns from config-volume (ro)
      /tmp from tmp (rw)
      /var/run/secrets/kubernetes.io/serviceaccount from kube-api-access-p988k (ro)
Conditions:
  Type              Status
  Initialized       True
  Ready             True
  ContainersReady   True
  PodScheduled      True
Volumes:
  tmp:
    Type:       EmptyDir (a temporary directory that shares a pod's lifetime)
    Medium:
    SizeLimit:  <unset>
  config-volume:
    Type:      ConfigMap (a volume populated by a ConfigMap)
    Name:      coredns
    Optional:  false
  kube-api-access-p988k:
    Type:                    Projected (a volume that contains injected data from multiple sources)
    TokenExpirationSeconds:  3607
    ConfigMapName:           kube-root-ca.crt
    ConfigMapOptional:       <nil>
    DownwardAPI:             true
QoS Class:                   Burstable
Node-Selectors:              <none>
Tolerations:                 CriticalAddonsOnly op=Exists
                             node-role.kubernetes.io/master:NoSchedule
                             node.kubernetes.io/not-ready:NoExecute op=Exists for 300s
                             node.kubernetes.io/unreachable:NoExecute op=Exists for 300s
Events:                      <none>

Versions:

Chart Version: 0.27.3
Kubernetes Version (kubectl version): 1.24

Please vote on this issue by adding a 👍 reaction to the original issue to help the community and maintainers prioritize this request
Please do not leave "+1" or "me too" comments, they generate extra noise for issue followers and do not help prioritize the request
If you are interested in working on this issue or have submitted a pull request, please leave a comment

Actually, I think I might see the issue here.

Based on the way that Karpenter performs pod batching and bin packing, it is not guaranteed that Karpenter will always choose the highest priority provisioner given specific requirements. For example, if a pod can’t be scheduled with the highest priority provisioner it will force creation of a node using a lower priority provisioner which may allow other pods from that batch to also schedule on that node. The behavior may also occur if existing capacity is available, as the kube-scheduler will schedule the pods instead of allowing Karpenter to provision a new node.

The behavior may also occur if existing capacity is available, as the kube-scheduler will schedule the pods instead of allowing Karpenter to provision a new node.
So is this possibly what happened?

karpenter spins up arm node for our spark pod
scheduler schedules coredns pod onto existing node, totally ignoring all karpenter configuration
spark pod exits, leaving only coredns pod
Because the arm provisioner does not have consolidation, the node is left at the current (very expensive!) size.

In this case non-consolidating provisioners seem very dangerous to have. It only takes one to run into a situation like this. I suppose I should put a taint on our non-consolidating provisioners?

Yes, Karpenter only provisions nodes when there is no available capacity for pods in the cluster. So if you have existing capacity, like what you mentioned, then kube-scheduler would utilize it.  If you have special nodes that you wouldn't want to use for general purpose pod scheduling, then you're right that you'd want to put a taint on that provisioner or make sure that all workloads have scheduling requirements that would force them to schedule where you want (using something like Kyverno, but taints are probably easiest).

Feel free to reopen if you are still having issues.

Thanks!


Use valid cert for HTTPS in development

Is this a bug report?
No. This is a question (for now) or discussion.
Which terms did you search for in User Guide?
HTTPS
Environment

OS: Windows 10
node -v: 8.4.0
yarn --version: 0.27.5/1.0.0
I haven’t ejected
Browser: Chrome 62

Steps to Reproduce:

Create a project
Run: 'set HTTPS=true&&yarn start' (or npm start)
Open https://localhost:3000 in the browser

Expected Behavior
Safe and secure HTTPS connection
Actual Behavior
Error message:

Your connection is not private

Main Question:
What I'm doing wrong and how to open secure connection without seeing errors or "unsafe connection warnings"?

I fixed this in very interesting and tricky way:

First I exported certificate from chrome (view this).
Imported the certificate into Windows (certmgr.msc).
Allowed chrome://flags/#allow-insecure-localhost flag. - https://goo.gl/VBV6uR

The most interesting is that 1 and 2 can't fix the security error without 3 and 3 can't without 1 and 2.

Out of curiosity: Why is this needed?
My (random) thoughts:

The warning is correct, as the certificate is not signed by a CA.
npm run start should be used only for development and for that http should suffice
When we deploy to an environment any other user wants to access, we should ensure proper certificates are in place.

@Dubes I know all of these things, but... Google wants it. I requested verification requested for using OAuth.
Dear Developer,
According to our information the following redirect URIs are not encrypted and secure.

post_message_origins: "http://localhost"
post_message_origins: "http://localhost:5000"

Please ensure your redirect URIs are encrypted and secure to ensure data theft does not take place. It can be done by using an HTTPS protocol (a secure, encrypted connection). Please reply back once you have updated all your redirect URIs for us to proceed with the verification process.

Sincerely,
Google Cloud Platform/API Trust & Safety

I'm not sure this is in scope of our project to fix, but I'm more than happy to accept a pull request and have some discussion about a possible solution for this.
Google shouldn't care if the cert isn't valid, I notice your error message says http:// and not https://, so maybe that's your problem?
It's going to be pretty low priority for us since you're the first to run into it (sorry!).

Yeah, you can add label priority: low, but please keep this open for discussion.
I'll try to learn more and if I can will open PR.

@Dubes you may use free CA like other js-lib do. I develop electron app, it can't open https://localhost:3000 if it has a invalid certificate, and, yes, use http can make it work, but it will be better as is support.


Add files via upload

@seenakhan: We may have to convert this docx file to md file.

@microsoft-github-policy-service agree company="Spektra Systems"


Comments Ending in Backslash also Comment following Command

Comments ending in backslashes prevent the following command from running. Is this intended behavior?
example:
  # following command won't run \
  echo Hello World!

Yah, that's correct, although I can definitely see why it's potentially confusing.


Use getpwnam_r instead of getpwnam in multi-threaded code

This pull request is an attempt to provide an upstream fix to multiple crashes seen in Alpine with cups. The crashes have in common that they happen on calls to getpwnam. Reading the documentation, I found that getpwnam and getpwuid are thread-unsafe functions. If I understand it correctly, the code under the cups folder is (potentially?) multi-threaded and could explain the crashes in getpwnam. Substituting all calls by getpwnam_r and getpwuid_r fix the crashes, at least in my limited testing.
Two things to note:

getpwnam and getpwuid are marked thread-unsafe by musl/POSIX, but also by glibc. Therefore, although I have not seen reports of crashes happening with glibc, they could potentially happen in the future if this fix is not applied in multi-threaded code.
There are also many calls to getpwnam and getpwuid under different files in the scheduler folder. However, if I understand it correctly by the documentation, the scheduler is single-threaded and using getpwnam and getpwuid should be safe. Otherwise, should I also change the calls there?

@pablofsf getpwnam_r goes back to the 1996 version of POSIX, so unless somebody reports a platform that doesn't support it let's avoid the complexity of checking and working around a missing implementation of it. Some platforms (macOS is one) use per-thread storage for the non-reentrant versions already so they are thread-safe, but I don't see a reason to special-case that.
I think 16k is more than enough - the passwd structure only has 6 string members and most are typically blank or short strings. The only potentially long string is pw_dir(home directory) but that will be limited to PATHMAX (1024 on most systems).
@zdohnal At this point I think my only concern would be in using the sysconf constants without an #ifdef check, but with the buffer in the per-thread globals I'm good.

Thank you so much for all the information! Should I send a similar PR for the calls under scheduler? Or is it ok to use thread-unsafe getpwnam there?


.name doesn't return the Microsoft gamertag so the docs shouldn't say .getName does

This is a known issue and has been patched in 47b47d4 for v2


COU should be more obvious when it's not going to upgrade nova-compute

This is the top of the output of running an upgrade with cou where there hypervisors are non-empty:
$ cou upgrade --auto-approve
Full execution log: '/home/ubuntu/.local/share/cou/log/cou-20240703060236.log'
Connected to 'openstack' ✔
Analyzing cloud... ✔
Generating upgrade plan... \2024-07-03 06:02:54 [WARNING] Not changing the install repository of app designate-bind: None already set to cloud:focal-victoria
Generating upgrade plan... /2024-07-03 06:03:01 [WARNING] Found non-empty hypervisors: [nova-compute/0, nova-compute/1, nova-compute/2, nova-compute/3, nova-compute/4]
Generating upgrade plan... ✔
Upgrade cloud from 'ussuri' to 'victoria'
        Verify that all OpenStack applications are in idle state
        Back up MySQL databases
...

It's not clear that the warning about finding non-empty hypervisors means that cou will skip upgrading these nova-compute units.  It simply states that it found non-empty hypervisors and doesn't not give any further information.
It should be much clearer.  Especially as this then blocks future steps, as some further steps require hypervisors to be upgraded.

Agreed that warning message could be improved and I approve your PR.


Update codes part3

@maneetgoyal sir the private repo has some issue with label visibility on HSL and Default color mode, we should fix it before releasing a new version here.

@maneetgoyal sir the private repo has some issue with label visibility on HSL and Default color mode, we should fix it before releasing a new version here.

Cool


build_db.py creates a prod domain without creating the Git branch

build_db.py initializes the Aquilon database with a prod domain whose content matches what is expected for running unit tests. But if unit tests are not run, the corresponding Git branch is not created in the template_king repository and aq add_domain --domain prod cannot create it as the domain already exists in the database. The only workaround is to create the branch manually in the Git repository.
2 actions are probably needed:

[ ] build_db.py should not create a prod domain by default. Such a creation should be done only if a specific option is given.
[ ] aq add_domain should have an option to fix the missing Git branch if the domain already exists in the database

There is a script that checks and fixes inconsistencies between DB and templates in the Git repo:
https://github.com/quattor/aquilon/blob/upstream/sbin/aqd_consistency_check.py
To ensure the build_db.py script only creates the DB but not adds data to it, you should not pass option '--populate' https://github.com/quattor/aquilon/blob/d7254a55af8a55533d21ca427eec00bcad241f94/tests/aqdb/build_db.py#L75
Or edit the contents of a data in this file (this is where the data is being populated from):
https://github.com/quattor/aquilon/blob/d7254a55af8a55533d21ca427eec00bcad241f94/tests/aqdb/data/unittest.dump

This came up at the Annecy workshop. It looks like the documentation is now correct, in that the "Git Daemon Configuration" section tells the user to create a prod branch manually.


attribute accent="true" in SVG output

I'm seeing accent="true" making it to the SVG output.
It shows up with a custom macro mimicking \accentset which does NodeUtil.setAttribute(accent, 'accent', true);). I realize it might be a bug on our end (not having updated after a change in MathJax) in which case I'd be grateful for guidance.
This might be affecting just the develop branch and open PRs.

What type of node is accent in NodeUtil.setAttribute(accent, 'accent', true);?  Note that the accent attribute is valid only on mo, msup, and msubsup, so if you are applying it elsewhere, MathJax will think it is a foreign attribute, and preserve it in the output.  It sounds like that may be what is happening for you,.

Thanks, Davide! That's what's happening. An isKind check should help avoid it, I think.


juno-msg-loadERROR: LoadError: ArgumentError: Module Atom not found in current path

I am trying to run Julia package on ATOM for the first time, but I'm getting this error...
Starting Julia...
juno-msg-loadERROR: LoadError: ArgumentError: Module Atom not found in current path.
Run Pkg.add("Atom") to install the Atom package.
while loading C:\Users\s1678502.atom\packages\julia-client\script\boot_repl.jl, in expression start
ing on line 16
When I try to Pkg.add("Atom") on Julia I get the following error:

Can anyone explain how to fix this? Appreciate your help!

Same as https://github.com/JuliaLang/julia/issues/26167.


macOS Monterey support

Update for official macOS Monterey support would be nice

The current version of the script supports creating macOS Monterey installer disk images. Updated README file: https://github.com/rtrouton/create_macos_vm_install_dmg/commit/ef94deefb07d96173e4af188d9e6465f5a9e3880


Slight changes from the book

Not sure if this is the new format, but quickly looking I couldn't wind a easy way to update it in the book
Objective

For sample project to compile after first pages of book

got:
  = note: clang: error: invalid linker name in argument '-fuse-ld=/usr/local/bin/zld'

error: linking with `cc` failed: exit status: 1

Solution
FIx to correct zld path

Not sure if this is the new format, but quickly looking I couldn't wind a easy way to update it in the book

A PR to https://github.com/bevyengine/bevy-website would be appreciated.

The website doesn't mention where zld would be installed on mac.
This PRs update the path to the one where homebrew will install zld, and homebrew is recommended just above for that.

bors r+

Hello,
I think the changes introduced in this PR might not be accurate or there might be more to it @mockersf .
I followed the current instruction in the bevy book which suggest using the path /opt/homebrew/bin/zld on mac os x86_64. and I ran into the clang error in the first post, although this time it was saying   = note: clang: error: invalid linker name in argument '-fuse-ld=/opt/homebrew/bin/zld'
I am using mac OS 10.15.7 x86_64.
Maybe this change depends on either the macOS version or the architecture ? Not sure would have to check the brew formula to see when each path is used.
Anyway for me changing back to '/usr/local/bin/zld' did the trick just thought I'd tell you.


NCP resolves before file copy is completed

Just to clarify, I have not experienced any issues while creating extensions. This is just to raise awareness of an issue I encountered when using NCP in a similar way as the extension system. You are probably avoiding the actual issue because of the template processing that happens between the initial copying of files (with NCP) and the removal of the tempDir.
For me, I was removing the temporary directory right after NCP resolved and it was leaving half the files uncopied or empty. While trying to understand why this was happening I found an old issue that still exists with NCP: https://github.com/AvianFlu/ncp/issues/127 Stll open and unresolved. At the end of the issue you see where facebook/react was able to sidestep the issue in their codebase by adding a 10ms timeout before resolving. Hacky but it apparently worked for them.
I was able to fix my problem with their same hacky solution:
const copy = (source: string, destination: string, options?: ncp.Options) => new Promise((resolve, reject) => {
    ncp(source, destination, options || {}, (err) => {
        if (err) {
            reject(err);
        } else {
            setTimeout(resolve, 10);
        }
    });
});

Again, no issues have been observed with the extension system as it is but I just wanted to make you aware of this potential issue when using NCP. Feel free to close this issue whenever you like.

thanks for reporting this! I think we won't change something for now since everything works as expected. But let's keep the issue open for some time so all maintainers can read it


Deprecate chalk.constructor() in favor of new chalk.Instance()

Closes #188
Todo:

[x] Somehow deprecate calls to chalk.constructor()
[x] Update readme

Here is the output from npm run bench:
(node:12838) [DEP0006] DeprecationWarning: child_process: options.customFds option is deprecated. Use options.stdio instead.

                      chalk
         191,041 op/s » single style
          64,176 op/s » several styles
         391,893 op/s » cached styles
          68,051 op/s » nested styles

  Suites:  1
  Benches: 4
  Elapsed: 4,760.06 ms

Compared to current version of chalk:
                      chalk
         193,479 op/s » single style
          67,814 op/s » several styles
         408,394 op/s » cached styles
          67,900 op/s » nested styles

  Suites:  1
  Benches: 4
  Elapsed: 4,706.14 ms

Sorry, this got totally lost in my open tabs.

@sindresorhus No problem! I've made the suggested changes.

Thank you! :)

As a new chalk user, I've got to ask: why?
What was wrong with new chalk()?

new chalk() never should have worked.

@Qix- Okay, but this doesn't explain what was actually wrong with it. How does moving to new chalk().Instance help?

Because chalk was already an instance of chalk. Using new on it doesn't make much semantic sense.
But moreover, IIRC there was a limitation with using new chalk, something with the template parser.

@grunge6 Please raise an issue with replication steps if you have found a bug or would like some support :-)

@grunge6 Such is OSS. We can't retain backwards compatibility forever, especially when we are careful about following semantic versioning.
If there is a bug, please open a new issue so we can address it. Be sure to include all of the relevant information (such as how to reproduce the bug).
Locking this as there is nothing further actionable.


update learn more link

Link to point to is https://aka.ms/azure-mysql-docs if the connection is Azure connection and if the connection is local <IP_ADDRESS> or localhost learn more link will point to https://dev.mysql.com/doc/

Not required


Remove debug webfont dependency

The main govuk_template.html file has a dependency on vendor/goog/webfont-debug.js. This dependency is bundled with the included assets.
This file seems to be an old debug version of the Google webfont loader. A search for its usage suggests alphagov and its derivatives are the only users: Search for 'google "webfont-debug.js"'.
While it currently serves it's purpose, there might be issues with using an old, debug version of the file. The Google Web Font Loader project suggests loading loading a production version using a CDN such as googleapis.com, jsDelivr or CDNJS. Alternatively, one of these newere, compressed versions could be downloaded and replace the current bundled version.

I’ve done some spelunking, and (although I wasn’t at GDS when this happened), it looks like this file was originally included to help with loading Gill from Monotype. After we stopped using Gill the google JS hung around for a while (eventually for IE8 and lower), but when govuk_template was set up and gained fonts they were loaded without JS. The loader was removed two months later, but the library has remained part of govuk_template until now.
Conclusion: it should probably be removed.


Update iqontrol to 1.9.10

Please update my adapter ioBroker.iqontrol to version 1.9.10.
This pull request was created by https://www.iobroker.dev 15f979c.

Sorry, received an bugreport for this version just not, please do not update stable and ignore the pr.


drawon() doesn't work properly (question)

I am trying to use the following code to draw on the existing svg file.
import ziamath as zm
from xml.etree import ElementTree as ET
ET.register_namespace("", "http://www.w3.org/2000/svg")

with open('test.svg', 'r') as file:
    tree = ET.parse(file)
    root = tree.getroot()

    myequation = zm.Math.fromlatex(r'x+y', size=28)
    myequation.drawon(root, 100, 2)

    with open('test2.svg', 'wb') as output_file:
        tree.write(output_file, encoding='UTF-8')

The output file is saved, but x, + and y are on the same place:

Size and coordinates doesn't have any effect. What I may be doing wrong?

Did you try zm.config.svg2 = False? It's likely whatever app you're using to view the output doesn't support SVG2.0 commands <symbol> and <use>, which means characters won't be translated to the correct location.

Thanks, it works.


Move tarmak socket to its own path

Is this a BUG REPORT or FEATURE REQUEST?:

Uncomment only one, leave it on its own line:
/kind bug
/kind feature

What happened:
Sockets are limited in path length. The socket now gets created inside the config directory of Tarmak. It is possible that that config directory path will be too long.
I propose to create the tarmak.sock in another path than the Tarmak config directory. This will not limit our users where to store the Tarmak config directory.
A possible location we could use is /tmp.
When we move it to a tmp folder, we should think about creating a unique socket name. This will be needed when we run Tarmak at the same time for different clusters. This can happen in CI/CD environments.
We did something similar for the controlpath in the SSH multiplexing.
https://github.com/jetstack/tarmak/pull/179

/assign


Button not clickable in header

Button is unresponsive when clicked on, but only when it's placed in the Header. In the Content block it works fine.
Here's the code:
    <LinearGradient colors={['#F0CB35', '#C02425']} style={styles.linearGradient}>
      <Container>
        <Header style={{backgroundColor: 'transparent', borderWidth: 0}}>
          <Left>
            <Button transparent
              onPress={ () => this.props.navigation.goBack(null) }>
              <Icon name='ios-arrow-back' />
            </Button>
          </Left>
          <Body />
          <Right />
        </Header>
         <Content style={styles.container}>
            <Button transparent
              onPress={ () => this.props.navigation.goBack(null) }>
              <Icon name='ios-arrow-back' />
            </Button>
          </Content>
      </Container>
    </LinearGradient>

The first button (in header) doesn't work and the second button (in content) works.
react-native, react and native-base version
react native 0.41. native base 2.0.12.
Is the bug present in both ios and android or in any one of them?
Only tried on ios emulator.
Also, I want to get rid of the bottom border line for the header. Tried setting borderColor to 'transparent' and borderWidth to 0. Neither works.

The button seems to work pretty well. Also just add the hasTabs prop to header for no border.
Can you just try doing a regular alert and check if the button responds?

@sankhadeeproy007 Thanks! I figured out what the problem was. I set the style of the Content block with {position: 'absolute', flex: 1}. It lay over the Header block and covered the button in header. But I didn't know it because the background is transparent. I deleted that style setting and it's now working. Sorry about the false alarm.

Glad it worked!


Can't use Phantom on Termux/UserLAnd

If I use UserLAnd or Termux, that are Linux environments, it should work on Android too, but it's not

@ItzLightyHD it's certainly the ARM version you should pick. As a matter of fact, I just tried the Linux ARM7 version out of curiosity and it works perfectly. Also I'd suggest Andronix instead of UserLand, as it has newer distro versions available.

woops, did I just spoil my server's IP? Let's unfuck this real professionally

@ItzLightyHD it's certainly the ARM version you should pick. As a matter of fact, I just tried the Linux ARM7 version out of curiosity and it works perfectly. Also I'd suggest Andronix instead of UserLand, as it has newer distro versions available.

What Android device are you using?

Huawei MYA-L11 with Android 6.0

ah looks like you might need ARM8. I built a new binary for 8, try that one? https://github.com/jhead/phantom/releases/tag/v0.5.1

ah looks like you might need ARM8. I built a new binary for 8, try that one? https://github.com/jhead/phantom/releases/tag/v0.5.1

Ok, i'll try. If something goes wrong, i'll write it here

Ok, now it works. But can you tell me why if I try to use phantom to join HyperLands, it says me "Can't connect to the server"?

It might be related to some incompatibility with certain servers. Have you tried other servers? For example, there's a known bug with phantom connecting to servers running Nukkit.


ReadTableFromDelimitedFile - need test for sub-second time precision

Sub-second time precision has been added to DateTime and need to test parsing for various number of digits.

I added a test using the ReadTableFromDelimitedFile command.  Close this issue.


Not hexadecimal string

keep getting this issue
keep getting this error when I try to run the port forward [*] Tasked beacon to list files in . [+] host called home, sent: 19 bytes [-] could not open C:\Windows\Temp\*: 5 [*] cd c:\users\public [*] Tasked beacon to upload /root/Downloads/taowu-cobalt-strike-english/script/iox.exe as iox.exe [+] host called home, sent: 1046589 bytes [+] host called home, sent: 1040460 bytes [+] host called home, sent: 228371 bytes [*] Tasked beacon to run: iox.exe fwd -r <IP_ADDRESS>:3389 -r *<IP_ADDRESS>:3389 -k 65535 [+] host called home, sent: 86 bytes [+] received output: Not hexadecimal string

-k needs to specify a hexadecimal string
Change -k 65535 to -k ffff

Thank you :) It worked


Fix/accordion react closed state changes

This PR will refactor the accordion React component so the closed prop is used directly in the render(), meaning any changes to this prop will cause a re-render where as before this was not the case.
The componentDidUpdate implementation is also changed to use the closed prop instead of props.state.closed, which is now removed from use.

Slight correction - changing the closed prop probably did cause a re-render before, but it wasn't changing the CSS closed class value as this was being set in the constructor. This is now determined on execution of render() so parent components that wish to align their state with closed on an accordion instance will be able to do so using <AUaccordion closed={this.state.accordionIsClosed} /> or similar.

I was starting to think I was crazy with the react component not functioning correctly.


New-DbaAgentJobStep: ensure the SMO job object is reloaded from the server

New-DbaAgentJobStep: ensure the SMO job object is reloaded from the server

Type of Change

[x] Bug fix (non-breaking change, fixes #7199 and fixes #7200 )
[ ] New feature (non-breaking change, adds functionality, fixes # )
[ ] Breaking change (effects multiple commands or functionality, fixes # )
[x] Ran manual Pester test and has passed (`.\tests\manual.pester.ps1)
[x] Adding code coverage to existing functionality
[x] Pester test is included
[ ] If new file reference added for test, has is been added to github.com/sqlcollaborative/appveyor-lab ?
[ ] Nunit test is included
[ ] Documentation
[ ] Build system

Purpose

Ensure the SMO job object is reloaded from the server, since updates to the job are not auto-reflected and the job object becomes stale.
Approach

The command already was retrieving the job object from the server, but wasn't using it in the subsequent script logic.
Commands to test

See the new integration test.

Thanks so much, Adam! @sanderstad - can you take a look and approve? 🙇🏼♀

Thanks to you both 🙇🏼


remove all mobile web views

These have been hanging around and confusing people for way too long. I propose we just rip them out. If we develop a responsive version of the core pages in the future, great, but currently they're probably doing more harm than good. Any one opposed?

nope

woo hoo! I don't like the mobile web pages, i always click to use the
regular ones if on the phone
On Wed, Oct 5, 2016 at 9:07 PM, Scott Loarie<EMAIL_ADDRESS>wrote:

nope
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/inaturalist/inaturalist/issues/1166#issuecomment-251842308,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AG1tkzJrKUXFzmQX5Ulu_zKKPFhFBvDVks5qxEnTgaJpZM4KPcgC
.

--
Charlie Hohn
Montpelier, Vermont

OK by me

Totally agree. The regular site looks so much better than the mobile view!


stringifyPath does not take root into account

this.$router.stringifyPath does not seem to respect the root option (http://router.vuejs.org/en/options.html#root), and therefore all paths returned by it do not include it.

Closing due to inactivity.
stringifyPath is undocumented so there is no warrant of it's behavior..