[fix] str.split can not handle surrogate pair, replaced with Array.from

example
> Array.from("\n𝐀")
[ '\n', '𝐀' ]

> "\n𝐀".split('')
[ '\n', '\ud835', '\udc00' ]

Hmm, interesting. I fundamentally agree with the proposed change; operating on Unicode code points instead of UTF-16 code units is the correct default behaviour (and frankly no code should operate on UTF-16 code units unless it's a library explicitly for dealing with UTF-16). However, I don't want to merge any change that modifies the results that jsdiff emits without adding unit tests, adding release notes, and doing a major version number bump. I also want to carefully audit the code line by line to make sure there's nowhere else where we're similarly treating strings as sequences of UTF-16 code units instead of Unicode code points.
I want to churn through some of the more straightforward-to-handle issues and PRs before doing the above - but I do intend to return to this PR in due course!

Adding docs and stuff over at https://github.com/kpdecker/jsdiff/pull/500


Reuse branch name

When a branch is deleted, GitHub sends a deletion event but they also send a push event. In the past, depending on the order the events are processed, this push event would accidentally reactivate the branch. Does this PR handle that case?

:ship

🚢


TouchableHighlight's onPress event is fired without touching the screen.

Description
I have a modal which I present to the user. Upon clicking yes, the modal's text is changed and it displays the ActivityIndicator.
`submit() {
var summary = "Are you sure you want to submit attendance with "+this.getSummaryString()+ " children present?"

this.props.dispatch(
  appActions.setModal({
  modalVisible: true,
  modalText:  summary ,
  modalMode: ModalMode.CONFIRM,
  modalOnPositive: () => {
    this.props.dispatch(appActions.setModal({
      modalVisible: true,
      modalText: "Getting location",
      modalMode: ModalMode.WAITING
    }))
    navigator.geolocation.getCurrentPosition((location) => {
      this.actions.submit(
        location,
        this.centreId,
        this.classId,
        this.attendanceData,
        this.token,
        this.props.navigator
      )
      this.props.dispatch(appActions.setModal({
        modalVisible: true,
        modalText: "Uploading Data",
        modalMode: ModalMode.WAITING
      }))
    }, (error) => {
      this.props.dispatch(appActions.setModal({
        modalText: "Could not get location. Please ensure location is enabled on your phone.",
        modalMode: ModalMode.OKAY,
        modalVisible: true
      }))
    },{
      timeout: 5000
    })
  }
}))

}`
When the background network activity completes, the modal's state is set to visible = true throughout. Logging Redux revealed that the modal's this.close is being fired without the button being touched. In my WaitModal component
`close() {
this.props.dispatch(appActions.setModal({modalVisible: false}))
}
onPositive(){
if(this.props.onPositive)
this.props.onPositive()
this.close()
}
onNegative(){
if(this.props.onNegative)
this.props.onNegative()
this.close()
}
...
<TouchableHighlight style={{ borderRadius: 5 }}
underlayColor={'white'}
onPress={ () => this.onPositive() }
>
<View style={{
padding: 15,
borderRadius: 5,
backgroundColor: Colours.primary ,
alignItems: 'center',
justifyContent: 'center',
width: 200
}}>
<Text style={{color: 'white', fontWeight: 'bold', fontSize: FontSizes.p}}>Okay

`
Solution
Perhaps there is a problem with my javascript, but I see no reason for the onPress event  to fire without me touching the screen so, onPositive()/onNegative() and close() should never run.
Additional Information

React Native version: 0.37.0
Platform: Android
Operating System: Mac OS X 10.12.1

Closing this issue because it has been inactive for a while. If you think it should still be opened let us know why.


zoom in and zoom out event is not working

Attach (recommended) or Link to PDF file here:
Configuration:

Web browser and its version:
Operating system and its version:
PDF.js version:
Is a browser extension:

Steps to reproduce the problem:
1.
2.
What is the expected behavior? (add screenshot)
What went wrong? (add screenshot)
Link to a viewer (if hosted on a site other than mozilla.github.io/pdf.js or as Firefox/Chrome extension):

the zoom in and zoom out event is not working
when ever i click on the event the show the error in console
the error is "offsetParent is not set -- cannot scroll"

Hy
Pada tanggal Jum, 18 Des 2020 16:39, Rajat Sharma<EMAIL_ADDRESS>menulis:

the zoom in and zoom out event is not working
when ever i click on the event the show the error in console
the error is "offsetParent is not set -- cannot scroll"
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/mozilla/pdf.js/issues/12749#issuecomment-747979770,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ARLGI57JOBM2X6BINX2JP3LSVMPOZANCNFSM4VA5L42A
.

When opening an issue, please provide all of the information requested in https://github.com/mozilla/pdf.js/blob/master/.github/ISSUE_TEMPLATE.md since as-is this issue contains no actionable information and will be closed as INVALID.
Furthermore, please see https://github.com/mozilla/pdf.js/blob/master/.github/CONTRIBUTING.md (emphasis mine):

If you are developing a custom solution, first check the examples at https://github.com/mozilla/pdf.js#learning and search existing issues. If this does not help, please prepare a short well-documented example that demonstrates the problem and make it accessible online on your website, JS Bin, GitHub, etc. before opening a new issue or contacting us in the Matrix room -- keep in mind that just code snippets won't help us troubleshoot the problem.


Add nodeSelector, tolerations, affinity configuration to helm chart

I added nodeSelector, tolerations, affinity configuration to helm chart to assign pods to nodes.

Coverage decreased (-0.02%) to 64.531% when pulling d852c68c4da6416be48eda85e83376c75f282675 on YunSangJun:assign-pod-to-node into 2f4950b80c3a955bae3a398f8486134ac7d3b87c on vmware:master.

@vito @jessehu @edevil
What does mean the following error message from continuous-integration/vic/pr
+ echo YunSangJun
YunSangJun
+ echo $SKIP_CHECK_MEMBERSHIP

+ if $SKIP_CHECK_MEMBERSHIP == true; then echo 'check-org-membership step skipped'; else /bin/bash -c '[[ ! $(curl --silent "https://api.github.com/orgs/vmware/members/YunSangJun?access_token=$GITHUB_AUTOMATION_API_KEY") ]]'; fi
/bin/sh: 23: ==: not found

And one more, In coverage/coveralls test, I do not know what is the problem. I hope you give me a hint.

@YunSangJun No worry. That CI does not allow PR from non-VMware users. I triggered a new one here https://ci-vic.vmware.com/vmware/harbor/4481 . Please wait for a while.

Could you describe the purpose of adding these attributes? Does your project require them ?

@jessehu Thank you.
Our team wants to run Harbor only at certain nodes. Helm 2.8.0+ also generate these configuration by default. The Kubernetes Official Chart Guide also recommends including this attribute.

Make sense. Thanks. And could you let us know which company you work for if it's convenient ? We'd like to update the internal list of Harbor customers :)

I work at SK Holdings C&C. We provide IT outsourcing services to our affiliates.


FI-291: enable sorting of TestResult objects by test id, and sort them on the results page

Problem:
TestResults used to display in a non-obvious/nondeterministic order (see the "before" screenshot below) after a test was cancelled. Probably they were being returned in an array in the order they were last modified. See ticket https://oncprojectracking.healthit.gov/support/browse/FI-291 for more information.
Solution:
Explicitly sort TestResults on the test_sets page by test_id (lexicographically), using the Comparable mixin on TestResult. See the "after" screenshots below to verify that the affected cancelled test suite is correctly sorted, and completed/not run test suites are still correctly sorted.
Before:

After:
Cancelled test suite:

Completed test suite:

Not-run test suite:

Could we have the database do this sort rather than ruby?

I just gave that a try, and it does seem to do the job (changing the TestResult association in SequenceResult to has n, :test_results, order: [:test_id.asc]). I'll switch the PR around to do that instead.

@arscan Rebased into development. After some discussion in the lab, the fix for this now removes a line from extends_sequence in sequence_base, which is something you added in a refactor. Are you able to re-review to ensure that we haven't broken something in fixing this ordering issue?


Way to view all currently applied ignored dependencies / versions

Is there an existing issue for this?

[X] I have searched the existing issues

Feature description
Resurrecting https://github.com/dependabot/dependabot-core/issues/2255#issuecomment-2152921722.

bump


[Change Request] Prefix the attrs values.

Description:
Would be possible to prefix the attrs values in order to not conflict with other libraries?
At the moment, the attribute  attr/viewType  conflicts with one with the same name in the new Androidx Media2 library androidx.media2:media2-widget. Prefixing to something like pin_viewType pin_backgroundColor would almost 100% prevent conflict issues of attributes.

same issue here

jetified-pinview-1.4.3/res/values/values.xml:14:5-32:25: AAPT: error: duplicate value for resource 'attr/viewType' with config ''.
still there

This commit exists on "develop" branch and must be added (cherrypick) to "master" branch as well

This commit exists on "develop" branch and must be added (cherrypick) to "master" branch as well


chore: update avo events LESQ-529

Description
Music year: {owa_music_year}

Updates Avo search events on homepage and searchpage

Acceptance Criteria:

[x]  Search Attempted tracking event
[x]  Search Journey Initiated
[x]  Search Result Opened
[x]  Search Results Displayed

Issue(s)
Fixes #LESQ-529
How to test

Go to https://deploy-preview-2145--oak-web-application.netlify.thenational.academy
Click on ______
You should see ______

Checklist

[ ] Added or updated tests where appropriate
[ ] Manually tested across browsers / devices
[ ] Considered impact on accessibility
[ ] Design sign-off
[ ] Approved by product owner
[ ] Does this PR update a package with a breaking change

Looks good - only this is the 'only' in the tests and Analytics Use Case = null from home page search - is this expected ?

Fixed the .only - I think this may be because the URL is /#teachers this was pre-exisiting behaviour should I add it to the ticket?

earch journey initiated" has a depth of "refine", I can't confirm if that is right or wrong but I feel it should be "interest"?

Removed the depth field as we're going to do more work on defining it
Search journey is initiated after entering some text, not just on selection of the search box
Unit and Lesson slug bugs should be fixed
Filters will be added on the examboard PR
Search results expanded will be on Joe's PR

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


Collect telemetry about usage

Collect visits per section, number of compiles (incl. failures) and create overview page to show usage stats of the tour.

can't you include Google Analytics for a start? It's free an pretty easy to setup.

Good suggestion. I already used Google Analytics so I confirm I would be easy to integrate it into the current solution. Basically Google Analytics could always be used even if some more fancy telemetry will be built at a later stage.


Provide Arrow Schema Hint to Parquet Reader

Which issue does this PR close?

Closes #5657
Rationale for this change

What changes are included in this PR?

Are there any user-facing changes?

seems like the integration process has some problem

FYI there is another alternative idea here: https://github.com/apache/arrow-rs/pull/5939 which looks promising

Closed by https://github.com/apache/arrow-rs/pull/5939


[#45] Bounding Volume Hierarchy

Finally working.

I'm about to rewrite commit history (squash things that can be squashed, remove irrelevant stuff and all that), so don't review this yet.

This PR will be opened again after some major changes in master.


Additional Windows Defender endpoints.

The online documentation Manage connection endpoints for Windows 10, version 1809 contains additional service endpoints for defender
https://docs.microsoft.com/en-us/windows/privacy/manage-windows-1809-endpoints#windows-defender
I would suggest that both pages contain the same list of service endpoints.

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

ID: f86b0db3-f979-bd1f-78a4-41247510721a
Version Independent ID: 5289bb5f-f015-7f38-05a1-26a615ae07aa
Content: Configure and validate Windows Defender Antivirus network connections
Content Source: windows/security/threat-protection/windows-defender-antivirus/configure-network-connections-windows-defender-antivirus.md
Product: w10
Technology: windows
GitHub Login: @Dansimp
Microsoft Alias: dansimp

@alexverboon  - Thank you for submitting feedback.
I will get this issue over to the Win10 ITPro writing team for investigation.
Thank you for reporting and making the docs better. Much appreciated.
I made a note to request the team to update this when the work is complete.

@alexverboon - From our understanding, the issue has been resolved based on this merged commit https://github.com/MicrosoftDocs/windows-itpro-docs/commit/ca19f9632298c2c9ad22ceb4164d759ff5045b83. If you feel it hasn't been resolved, please re-open this issue.
Thank you for your contribution to make the docs better! Much appreciated!


Testing login via LoginOverlay

The example applications have a LoginView that extends com.vaadin.flow.component.login.LoginOverlay. The LoginViewTest tests navigation to LoginView and (failed, since not logged in) navigation to another view. But it does not test the login-process (i.e. entering UN/PW) itself. Although this is not generally necessary (since it just tests a vaadin component) such a test would be nice since it shows how to acces fields of a component.
I've tried _get(TextField.class, spec -> spec.withId("vaadinLoginUsername")).setValue(username);  since this is how com.vaadin.flow.component.login.testbench.LoginFormElement does this. But that did not work:
java.lang.AssertionError: /login: No visible TextField in UI[] matching TextField and id='vaadinLoginUsername': []. Component tree:
└── UI[]
    └── LoginView[]

Now I am not sure if accessing fields of a component is even possible with karibu-testing, or if I did anything wrong.

Hi, unfortunately those fields are only present in the browser - those fields are not present on the server-side. I think it's even mentioned in Karibu's documentation 🤔 let me check

Yup, mentioned at https://github.com/mvysny/karibu-testing/tree/master/karibu-testing-v10#support-for-loginform-and-loginoverlay . Closing, please reopen if the documentation could be improved.

Thanks for that superquick answer - and sorry for not having found that documentation myself. The reason for that is that I've just checked the README.md in https://github.com/mvysny/karibu-testing and was not aware of the much more extensive README.md documentation in https://github.com/mvysny/karibu-testing/tree/master/karibu-testing-v10. Totally my fault, since it is mentioned in the main README.md.

No problem! I'll try to make the main page link more prominent; also I'll try to describe the login view issue in more detail.


[Feature request] Remember connected device.

Is your feature request related to a problem? Please describe.
This app is so useful that I always use it to transfer files across platforms.
However, when I'm at school, devices cannot find each other even when they are in the same network, so I have to type in IP address of my devices every time to transfer files.
Describe the solution you'd like
Add a "remembered devices" feature so that I can save IP address of all my devices since I seem to have a static nat IP address.
Whenever I want to send files to a device, I can simply choose the device in the remembered devices list without typing address all the time.
Describe alternatives you've considered
No response
Additional context
No response

Duplicate of #7


compatibility with supermaven-nvim (indent issue)

Make sure you have done the following

[X] I have updated to the latest version of blink.cmp and blink.compat
[X] I have read the README

Bug Description
Supermaven's suggestions seem to expect nvim-cmp to clear the line before inserting the suggestion. For this reason, indentation can double up when accepting the suggestion.
Cmp:

Blink:

Relevant configuration
{
  "saghen/blink.cmp",
  opts = {
    sources = {
      compat = { "supermaven" },
    },
    windows = {
      autocomplete = {
        selection = "auto_insert",
      },
    },
  },
  dependencies = {
    "saghen/blink.compat",
    {
      "supermaven-inc/supermaven-nvim",
      opts = {
        keymaps = {
          accept_suggestion = nil,
        },
        disable_inline_completion = true,
      },
    },
  },
},

neovim version
NVIM v0.10.2 Build type: Release LuaJIT 2.1.1713484068 Run "nvim -V1 -v" for more info
blink.cmp version: branch, tag, or commit
v0.6.1
blink.compat version: branch, tag, or commit
0f368c3

Something else I noticed:

with auto_insert, but then hitting enter after, will delete the entire line
without auto_insert, accept simply does nothing other than close the completion window

@theol0403 I believe I know what the issue is. Could you try mannually pinning blink.cmp to 0.5.0 and blink.compat to 0bb4b68e8f09d66ca69f482c6aa782c0f643829b to see if that works?
@Saghen it looks like supermaven expects the regular completion to continue after execute, which as I understand is different to how blink handles execute. Is this something that could be made configurable by the source (ex by introducing opts into the callback parameters)?

@stefanboca thank you for the reply! unfortunately, I am having trouble downgrading blink.cmp... I am within the LazyVim environment, and while downgrading is possible, there are breaking changes in LazyVim's configuration... and when completely resetting LazyVim's default configuration... supermaven (or LSP for that matter) do not work.
I can try to keep debugging a fresh install of blink.cmp outside of the LazyVim context but I am unsure if it is worth the time given that you seem to understand the issue.

@stefanboca Just pushed the change on main, it'll run the execute function after applying

@theol0403 could you check if you still have the same issue?
@sagehen thanks, just saw this.
Unrelated but right before I saw this I merged #15 and released v1.0.1 which accidentally included 6d914f7396cf026f04156266a5b134a25cbbca97, so there might be some slight incompatibility until the next release of blink.cmp.

Reverted the commit and made a new release without it, should be all good now, thanks for noticing

@Saghen @stefanboca thanks for the work on this!
With auto_insert, the behavior is now improved -- cycling the selections maintains proper indentation! However, hitting accept cancels the entire suggestion and reverts to an empty line.
Similarly, without auto_insert, hitting accept simply closes the completion window without inserting the selected suggestion.

@theol0403 I just installed supermaven to try replicate this issue and I couldn't. Accepting both with and without auto_insert worked fine for me (although there were some errors with ghost text). Here's the config I used:
{
  "supermaven-inc/supermaven-nvim",
  opts = {
    disable_inline_completion = true,
  },
},
{
  "saghen/blink.cmp",
  opts = {
    sources = {
      completion = {
        enabled_providers = { "supermaven" },
      },
      providers = {
        supermaven = {
          name = "supermaven",
          module = "blink.compat.source",
          score_offset = 3,
        },
      },
    },
  },
},


Cannot write a column of type DataType::List containing a DataType::Struct to parquet

Describe the bug
When trying to write a column that is of type List that contains a Struct, the parquet writer throws an error Error: ParquetError(General("Incorrect number of rows, expected 4 != 0 rows")). This seems to be a regression as this works fine in datafusion v32.0.0 but not in v33 or v34. It also works using write_json instead of write_parquet
Example dataframe:
+------------------------------------+
| filters                            |
+------------------------------------+
| [{filterTypeId: 3, label: LABEL3}] |
| [{filterTypeId: 2, label: LABEL2}] |
+------------------------------------+

To Reproduce
dependencies (working):
[dependencies]
tokio = { version = "1.35.1", features = ["macros"] }
datafusion = { version = "32.0.0", features = ["backtrace"] }

dependencies (broken):
[dependencies]
tokio = { version = "1.35.1", features = ["macros"] }
datafusion = { version = "33.0.0", features = ["backtrace"] }

example.json
{"filters":[{"filterTypeId":3,"label":"LABEL3"}]}
{"filters":[{"filterTypeId":2,"label":"LABEL2"}]}

main.rs
use datafusion::{dataframe::DataFrameWriteOptions, error::DataFusionError, prelude::*};

#[tokio::main]
async fn main() -> Result<(), DataFusionError> {
    let ctx = SessionContext::new();

    let df = ctx
        .read_json("example.json", NdJsonReadOptions::default())
        .await?;

    df.write_parquet("result", DataFrameWriteOptions::default(), None)
        .await?;
    Ok(())
}

Expected behavior
The parquet writer supports writing this kind of datatype as in v32
Additional context
Maybe related to: https://github.com/apache/arrow-rs/issues/1744
I found this issue trying to debug a different one that came up while trying to upgrade from v32 to v34. If the struct contains a timestamp the error instead becomes a Error: Internal("Unable to send array to writer!") with a source error internal error: entered unreachable code: cannot downcast Int64 to byte array.
An example of such a df:
+---------------------------------------------------------------------------------+
| filters                                                                         |
+---------------------------------------------------------------------------------+
| [{assignmentStartTs: 2023-11-11T11:11:11.000Z, filterTypeId: 3, label: LABEL1}] |
| [{assignmentStartTs: 2023-11-11T11:11:11.000Z, filterTypeId: 2, label: LABEL2}] |
+--------------------------------------------------------------------------------+

I tried to debug this issue myself looking into the arrow-rs implementation however I didn't manage to find the relevant commit that could have changed this behavior. Also I wasn't sure if I should open the bug in this project or in the arrow-rs project so I hope this is ok 😃.

Also I wasn't sure if I should open the bug in this project or in the arrow-rs project so I hope this is ok 😃.

I think this project is the correct place to start. We can distill your example down and see if we can get an arrow-rs only reproducer
@tustvold  do you have any thoughts?

It could be something in the arrow upgrade from 47 to 48, but nothing immediately springs out to me.

Edit: the DF parallel parquet writer looks to have landed around then, I wonder if it is the cause.

We can try the following setting and see if that fixes this bug to verify if it is the cause.
datafusion.execution.parquet.allow_single_file_parallelism false

thank you all for the quick response! For now setting the variable is a good workaround for my usecase 👍🏻.

@devinjdangelo  set the feature allow_single_file_parallelism to false by default in  https://github.com/apache/arrow-datafusion/pull/8854 and added test coverage so this is no longer a "regression" (instead it is a bug in a feature that is not enabled by default)
Let's leave this issue open until the code is sorted but I think it is much less severe now


feat: experiment using @floating-ui for tooltips

seems like a better option than rolling our own solution from scratch.

Closing in favor of this work: https://github.com/mergestat/blocks/issues/42


[FEAT]: toggle notes to flashcards

Is your feature request related to a problem? Please describe.
I create toggle notes in Obsidian to help with active recall but can I create this into flash card to help with spaced repetition
Describe the solution you'd like
A feature to convert toggle notes to flash cards without having to create flash cards from scratch
Additional context
No response

What are toggle notes? Can you give an example of how your file looks and what you like the flashcard to look like.

If I understand you correctly formatting of your answer was hard to read.
you would like to have

It is possible with kind of a trick but requires a bit of adjustment to your files
Set for multi line setting the >

And than you need to have a empty line like this

Than it is possible to get the requested result. But in my opinion complete flashcard marking needs a rework there are some issues with the current system.


Make network and netmask configurable via environment parameter

This will enable the setup to have different networks and netmasks being used.

One example for setting up a aka cluster with own IP network/netmask:
version: '2'
services:
  seed:
    image: mhamrah/clustering:0.3
    ports:
      - '1600:1600'
    environment:
      CLUSTER_PORT: 1600
#      CLUSTER_IP: seed
      SEED_PORT_1600_TCP_ADDR: <IP_ADDRESS>
      HOST: seed
    networks:
      backend:
        ipv4_address: <IP_ADDRESS>

  c1:
    image: mhamrah/clustering:0.3
    ports:
      - '1601:1601'
    environment:
      CLUSTER_PORT: 1601
#      CLUSTER_IP: <IP_ADDRESS>
      SEED_PORT_1600_TCP_ADDR: <IP_ADDRESS>
      HOST: c1
    networks:
      backend:
        ipv4_address: <IP_ADDRESS>

  c2:
    image: mhamrah/clustering:0.3
    ports:
      - '1602:1602'
    environment:
      CLUSTER_PORT: 1602
#      CLUSTER_IP: <IP_ADDRESS>
      SEED_PORT_1600_TCP_ADDR: <IP_ADDRESS>
      HOST: c2
    networks:
      backend:
        ipv4_address: <IP_ADDRESS>

  proxy:
    image: tkaefer/docker-mac-network
    ports:
      - "<IP_ADDRESS>:13194:13194"
    command: TCP-LISTEN:13194,fork TCP:<IP_ADDRESS>:1194
    networks:
      backend:

  openvpn:
    image: kylemanna/openvpn
    volumes:
      - ./config:/etc/openvpn
      - ./local:/local
    cap_add:
      - NET_ADMIN
    environment:
      dest: akka-mac-docker.ovpn
      DEBUG: '1'
      DOCKER_NETWORK: <IP_ADDRESS>
      DOCKER_NETMASK: <IP_ADDRESS>
    command: /local/helpers/run.sh
    networks:
      backend:
        ipv4_address: <IP_ADDRESS>

networks:
  backend:
    driver: bridge
    ipam:
      driver: default
      config:
        - subnet: <IP_ADDRESS>/26
          gateway: <IP_ADDRESS>

The environment variables exposed as:
      DOCKER_NETWORK: <IP_ADDRESS>
      DOCKER_NETMASK: <IP_ADDRESS>

will ensure the right routing in the OpenVPN client config file.

@tkaefer Sorry for the late response. Merging, thanks!


Status of this project

Hi Manolis,
id like to know about the status of the project. Right now i am trying to port a project of mine from traditional Jenkins configuration to Pipeline, running ZAP is one of the tasks i have to solve. I have found your repo by  coincidence and it seems to solve some problems of mine.
Id like to know:

is it or will it be useable in Jenkins
does it start the ZAP Server (startZap seems to have some issues or isnt finished yet)
does it run the attack
does it shut down the Server at the end of the build
is there any type of schedule :D?

I havent worked on Jenkins Plugins yet, but maybe i can give you a hand.

Hey, if you want to try the plugin out you will just need to build it, you can just run mvn install at the root, which will generate a HPI file for jenkins.
I had the line to actually start ZAP commented out by mistake, so that's why (if you tried) it wasn't starting 😢, get the latest version for a quick fix
To run an attack you can call runZapAttack(), it will start attacks on the hosts you provide in allowedHosts (in startZap), and localhost.
It shuts down ZAP automatically after archiveZap() has finished
What do you mean by schedule? Run zap every x? You could create a new pipeline job that calls the functions above and runs at certain times (Configure job -> Build periodically)
There is a bunch of stuff that still needs doing, eg a way to load sessions, URL lists & attack policies, ajaxSpider, more detailed reports with raw request/resposes, Windows support (easy), ect, so feel free to contribute 😄

Hey,
after some configuration and try error approache i managed to get things running. We are still missing some features to port our projekt to Jenkins Pipeline but this looks like a good starting point. I will discuss this with my colleagues and hopefully i will be able to contribute. I will let you know in a couple of days.
By schedule i mean some kind of release plan, like any specific features you want to implement or when (if) you want to release the plugin to Jenkins Plugin repo.

Hey
What problems did you have running the plugin (so I can make it better)? It meets all of our requirements currently but I am planning on adding more features, what do you need?
I am planning on releasing this to the Jenkins plugin repo ~soon~ after I clean things up a little & add some more features (above)

Here is a short list of the problems i had and features i need:

problems to get the configuration right because none of the exceptions (occurring in ZapDriver) are propagated to the Jenkins Output.
some kind of integration test with a local installation of ZAP would be helpful to speed things up (i didnt want to build and install the plugin over and over again)
Method to check if the ZAP Server is currently running (startup takes some time)
Load an existing session
Load a file with URLs to scan
Change scan/attack options (path traversal on/off)

I implemented some of the things in a quick and dirty way to evaluate the possibilities but i have to get permission to keep working on it and contribute.

Not done, I agree it is hard to see exactly what's failing without looking in the zap log
This is the script I use for quickly building & installing, there are no integrations test yet.

sudo service jenkins stop
sudo mvn install
sudo cp ./target/zap-comp.hpi /var/lib/jenkins/plugins/
sudo service jenkins start

startZap will wait until ZAP has started before continuing & fail the build if it never goes up. The code for this is in StartZapExecution.java

I have added a sessionPath parameter to startZap that loads a session at the path you specify. If you want to run a scan as a specific user you can use the userId parameter in runZapAttack - eg runZapAttack(userId: 2)

I added importZapUrls("path/to/urls"), but you need the import urls plugin to use it

You can now import scan policies using importZapScanPolicy("path/to/.policy/file"). You can then run runZapAttack(scanPolicyName: "nameofpolicy") to run an attack with a specific policy.

Do these changes help?

Closing due to no reply


feat(extensions): enable mock resets

Change Summary
This allows you to reset mocks for components, functions, or both. Closes #466.

I approved, but please rebase and remove your version bump before you merge 😸


Refactor pageant to be an importable package

A new Pageant struct wraps the configuration and proxy logic. A new instance can be created with pageant.New. The Pageant proxy can then be started using Run().
This makes it easier to embed the functionality in another application, such as a custom SSH agent implementation.

This is an awesome idea! thanks for the work. I didn't think anyone would want an importable package out of this functionality but it appears I'm wrong.
I'm on lunch right now so I won't get time to in-depth review this, but I did take a cursory glance. Nothing immediately pops out as wrong, so that's great. I have some questions/discussion for you though:

Have you built this and ran/tested it locally? (I know I need to write integration tests for this)
Out of curiousity, what project are you trying to use this new pageant package in?

I'd like to make what wndProc does with the data it receives configurable. I.E we give the Pageant struct a function handler (something like handlerFunc Func([]byte)) and call it on line run_windows.go:213, and we move the lines below that into the default handlerFunc.

* Have you built this and ran/tested it locally? (I know I need to write integration tests for this)

Yes, compiled it for a Windows VM, configured Putty for<EMAIL_ADDRESS>and loaded a test SSH key with ssh-add. Connection was successful 🙂. Haven't tested packaging it, but that should just work.

* Out of curiousity, what project are you trying to use this new `pageant` package in?

I have several ideas that revolve around SSH agents, and I would like them to work on Windows without having to install separate binaries. The most concrete is the ecosystem we're building at Smallstep, amongst which is the CLI: https://github.com/smallstep/cli. We already have some integrations with OpenSSH, and our CLI generally works well on Windows, but people are still often using Putty for SSH. It may be an option to include this functionality in the CLI (or a different component) to make it easier for people to use Putty with the rest of our ecosystem. Combining that with SSH certificates in Putty, which we're strong proponents of, might be even more fun. Another potential application may be to add support to https://github.com/FiloSottile/yubikey-agent. Windows support is WIP; embedding this for Putty support might be nice.

I'd like to make what wndProc does with the data it receives configurable. I.E we give the Pageant struct a function handler (something like handlerFunc Func([]byte)) and call it on line run_windows.go:213, and we move the lines below that into the default handlerFunc.
If I understand you correctly you want the handlerFunc to be called with result? And by default that should only do the copy(sharedMemoryArray[:], result)? The rest of the code is only cleanup; and the sharedMemoryArray actually needs setup before the call, so doesn't seem suitable to put in the handlerFunc itself?. I can imagine just calling the handlerFunc to be used like a callback with result, then continue with the cleanup as it is right now.

As a preparation for this change I've made New a variadic function taking options.

Yes, compiled it for a Windows VM, configured Putty for<EMAIL_ADDRESS>and loaded a test SSH key with ssh-add. Connection was successful 🙂

Awesome, thank you.

If I understand you correctly you want the handlerFunc to be called with result?

Close, very close. You did a good job interpreting my rushed lunch response. Here's what I imagined:
Original:
size := binary.BigEndian.Uint32(sharedMemoryArray[:4]) + 4 // +4 for the size uint itself
if size > sshagent.AgentMaxMessageLength {
    return 0
}

// Query the windows OpenSSH agent via the windows named pipe
result, err := sshagent.QueryAgent(p.sshPipe, sharedMemoryArray[:size])
if err != nil {
    log.Println(err)
    return 0
}
copy(sharedMemoryArray[:], result)

With a handler func:
size := binary.BigEndian.Uint32(sharedMemoryArray[:4]) + 4 // +4 for the size uint itself
if size > sshagent.AgentMaxMessageLength {
    return 0
}

result, err := p.HandlerFunc(sharedMemoryArray[:size])
if err != nil {
    log.Println(err)
    return 0
}

copy(sharedMemoryArray[:], result)

Where the signature of p.HandlerFunc is func(p *Pageant, pageantResult []byte) ([] byte, err)
And the default HandlerFunc is
result, err := sshagent.QueryAgent(p.sshPipe, sharedMemoryArray[:size])

The names of the function/variable names is just to illustrate meaning, I'm not married to them being that exact name, but I hope I got my meaning across.
Are you okay with me committing changes to this branch too (that is, if you haven't made these above changes before I get to it)?

Hey @ndbeals, I added the HandlerFunc like you described above. Via an Option is not likely to work, because of the function signature. It's calling a method on p with p as the first argument. If provided through New(), then p is not yet known, so that doesn't work.
Maybe there's a way to make this work via New with some more refactoring or if the SSH pipe name is passed instead of p.
Feel free to commit to this branch 🙂

Hey @ndbeals, what are your thoughts on my latest changes and the proposed options? Without doing some more refactoring, it'll remain a bit messy. It works, but could be made a bit more neat, I guess 🙂

Sorry this took me so long @hslatman. I've made some changes and I'm going to merge this.

Thank you, @ndbeals!
Changes look OK and usable to me 🙂


Rebase CPVPA to distroless

This is part of the effort described in kep kubernetes/enhancements#900
It mitigates the impact of CVE issues and keep the image light weighted.

/assign @MrHohn

I don't have approval privilege though. Adding @thockin for approval :)
/assign @thockin

Ha, I think you should have. They just put the wrong approval name

Ah good find, I will try to update that.

/lgtm
/approve


Converted to python 3 and added tenant_id

I converted the storage module to python 3 and added the tenant_id to the configuration (including the README.md). The latter fix I found via a user called James Wettenhal who posted it here: https://groups.google.com/forum/#!topic/tardis-devel/wBAyHyz6HuU

Thank you! :+1:


NETMQ transport support

Is this a bug report?
NO
Looking around for some solutions on messaging I stumbled on your library. I also saw the different transports. I would like to keep the infrastructure simple for now, so I would like to use something like ZeroMQ (NETMQ) so I can do messaging between processes on the same server without installing additional software.
Is this possible? I saw there was a PR (in 2012) related to ZeroMQ.

There is no support for 0MQ/NETMQ in MassTransit, and there isn't anything on the roadmap at this time nor is it being considered.
I do think there was a team that had built a named pipes transport, but I don't know details about it. You might ask in the chat room.


build.jl: Fix word repetition

Warning message contains two 'without's, delete one of them

Thanks!


Linking(?) issue when using find_package after installation

Issue
After building and installing both cpr and influxdb-cxx i get the following error if I build and run the test_package example.

./test_package: error while loading shared libraries: libcpr.so.1: cannot open shared object file: No such file or directory

Even though the file exists and cpr works on its own.
Details
I've been struggling with this issue for a while and I don't understand why it occurs. I'm running a 64bit Ubuntu 22.04 LTS VM on a Windows host. I'm building and installing cpr as described in their repo with the flag CPR_USE_SYSTEM_CURL=ON.
I'm building this library with the flags INFLUXCXX_TESTING=OFF and INFLUXCXX_WITH_BOOST=OFF
I've tried with the latest master of both cpr and influxdb-cxx as well as cpr 1.10.0 together with influxdb-cxx v0.7.1 with the same outcome. If I try to build and run a similar minimalistic example using just cpr, that works fine which is why I believe that the issue isn't with cpr.
By coincidence I noticed that if I make an example that includes and uses cpr as well as influxdb-cxx it runs without the error, so something is not working when l just use influxdb-cxx in the project.
I might be missing something obvious, let me know if there's any more details needed.
For a minimal example that runs and doesn't give the error:
CMakeLists.txt:
cmake_minimum_required(VERSION 3.12)
project(test_package CXX)

find_package(InfluxDB REQUIRED)
find_package(cpr REQUIRED)

add_executable(test_package example.cpp)
target_link_libraries(test_package PRIVATE InfluxData::InfluxDB)
target_link_libraries(test_package PRIVATE cpr::cpr)
target_compile_features(test_package PRIVATE cxx_std_17)

example.cpp:
#include <InfluxDBFactory.h>
#include "cpr/cpr.h"

int main()
{
    auto influxdb = influxdb::InfluxDBFactory::Get("http://localhost:8086?db=test");
    influxdb->addGlobalTag("test", "value");

    cpr::Response r = cpr::Get(cpr::Url{"https://api.github.com/repos/whoshuu/cpr/contributors"},
                               cpr::Authentication{"user", "pass", cpr::AuthMode::BASIC},
                               cpr::Parameters{{"anon", "true"}, {"key", "value"}});
}

After building and installing both cpr and influxdb-cxx i get the following error if I build and run the test_package example.

Marginally: The test_package directory is used for Conan packaging tests.

By coincidence I noticed that if I make an example that includes and uses cpr as well as influxdb-cxx it runs without the error, so something is not working when l just use influxdb-cxx in the project.

If you build the example without the cpr code in both listings, what is the error you get? Also can you verify whether cpr has installed it's cmake files correctly (either listed on make install or available on the filesystem).

Update: I was able to reproduce the issue using these steps:
#!/bin/bash

set -ex

BASE_PATH="/tmp/ifdbcxx_repro"

rm -rf "${BASE_PATH}"
mkdir -p "${BASE_PATH}"
cd "${BASE_PATH}"

git clone --depth=1 -b 1.10.4 https://github.com/libcpr/cpr.git
mkdir -p cpr/build
cd cpr/build

cmake -DCPR_USE_SYSTEM_CURL=ON -DCMAKE_INSTALL_PREFIX="${BASE_PATH}/deps" ..
cmake --build . -j
cmake --build . --target install

cd "${BASE_PATH}"

git clone --depth=1 https://github.com/offa/influxdb-cxx.git
mkdir -p influxdb-cxx/build
cd influxdb-cxx/build

cmake -DINFLUXCXX_TESTING=OFF -DINFLUXCXX_WITH_BOOST=OFF -Dcpr_DIR="${BASE_PATH}/deps/lib/cmake/cpr/" -DCMAKE_INSTALL_PREFIX="${BASE_PATH}/deps" ..
cmake --build . -j
cmake --build . --target install

cd "${BASE_PATH}/influxdb-cxx/test_package"
mkdir build
cd build

cmake -DInfluxDB_DIR="${BASE_PATH}/deps/lib/cmake/InfluxDB/" -Dcpr_DIR="${BASE_PATH}/deps/lib/cmake/cpr/" ..
cmake --build . -j
./test_package

Well that seemed to work.
Fresh install of ubuntu 20.04
apt-get install git
apt-get install cmake
apt-get install build-essential
apt-get install openssl
apt-get install libssl-dev
apt-get install curl
apt-get install libcurl4-openssl-dev
root@dev03:~# ./test.sh

BASE_PATH=/tmp/ifdbcxx_repro
rm -rf /tmp/ifdbcxx_repro
mkdir -p /tmp/ifdbcxx_repro
cd /tmp/ifdbcxx_repro
git clone --depth=1 -b 1.10.4 https://github.com/libcpr/cpr.git
Cloning into 'cpr'...
remote: Enumerating objects: 189, done.
remote: Counting objects: 100% (189/189), done.
remote: Compressing objects: 100% (178/178), done.
remote: Total 189 (delta 21), reused 77 (delta 5), pack-reused 0
Receiving objects: 100% (189/189), 153.87 KiB | 2.65 MiB/s, done.
Resolving deltas: 100% (21/21), done.
Note: switching to '2553fc41450301cd09a9271c8d2c3e0cf3546b73'.

You are in 'detached HEAD' state. You can look around, make experimental
changes and commit them, and you can discard any commits you make in this
state without impacting any branches by switching back to a branch.
If you want to create a new branch to retain commits you create, you may
do so (now or later) by using -c with the switch command. Example:
git switch -c
Or undo this operation with:
git switch -
Turn off this advice by setting config variable advice.detachedHead to false

mkdir -p cpr/build
cd cpr/build
cmake -DCPR_USE_SYSTEM_CURL=ON -DCMAKE_INSTALL_PREFIX=/tmp/ifdbcxx_repro/deps ..

-- Detecting SSL backend...
-- Found OpenSSL: /usr/lib/x86_64-linux-gnu/libcrypto.so (found version "1.1.1f")
-- SSL auto detect: Using OpenSSL.
-- Found CURL: /usr/lib/x86_64-linux-gnu/libcurl.so (found version "7.68.0") found components: HTTP HTTPS
-- Curl 7.68.0 found on this system.

-- The CXX compiler identification is GNU 9.4.0
-- Check for working CXX compiler: /usr/bin/c++
-- Check for working CXX compiler: /usr/bin/c++ -- works
-- Detecting CXX compiler ABI info
-- Detecting CXX compiler ABI info - done
-- Detecting CXX compile features
-- Detecting CXX compile features - done
-- ~~~ influxdb-cxx v0.7.1 ~~~
-- Build Type : RelWithDebInfo
-- Boost support : OFF
-- Unit Tests : OFF
-- System Tests : ON
-- Looking for C++ include pthread.h
-- Looking for C++ include pthread.h - found
-- Performing Test CMAKE_HAVE_LIBC_PTHREAD
-- Performing Test CMAKE_HAVE_LIBC_PTHREAD - Failed
-- Looking for pthread_create in pthreads
-- Looking for pthread_create in pthreads - not found
-- Looking for pthread_create in pthread
-- Looking for pthread_create in pthread - found
-- Found Threads: TRUE
-- Found CURL: /usr/lib/x86_64-linux-gnu/libcurl.so (found version "7.68.0")
-- Performing Test COMPILER_HAS_HIDDEN_VISIBILITY
-- Performing Test COMPILER_HAS_HIDDEN_VISIBILITY - Success
-- Performing Test COMPILER_HAS_HIDDEN_INLINE_VISIBILITY
-- Performing Test COMPILER_HAS_HIDDEN_INLINE_VISIBILITY - Success
-- Performing Test COMPILER_HAS_DEPRECATED_ATTR
-- Performing Test COMPILER_HAS_DEPRECATED_ATTR - Success
-- Configuring done
-- Generating done
-- Build files have been written to: /tmp/ifdbcxx_repro/influxdb-cxx/build
cmake --build . -j
Scanning dependencies of target InfluxDB-BoostSupport
Scanning dependencies of target InfluxDB-Internal
Scanning dependencies of target InfluxDB-Core
[ 12%] Building CXX object src/CMakeFiles/InfluxDB-BoostSupport.dir/NoBoostSupport.cxx.o
[ 37%] Building CXX object src/CMakeFiles/InfluxDB-Internal.dir/LineProtocol.cxx.o
[ 37%] Building CXX object src/CMakeFiles/InfluxDB-Internal.dir/HTTP.cxx.o
[ 50%] Building CXX object src/CMakeFiles/InfluxDB-Core.dir/Point.cxx.o
[ 62%] Building CXX object src/CMakeFiles/InfluxDB-Core.dir/InfluxDB.cxx.o
[ 87%] Building CXX object src/CMakeFiles/InfluxDB-Core.dir/InfluxDBFactory.cxx.o
[ 87%] Building CXX object src/CMakeFiles/InfluxDB-Core.dir/Proxy.cxx.o
[ 87%] Built target InfluxDB-BoostSupport
[ 87%] Built target InfluxDB-Core
[ 87%] Built target InfluxDB-Internal
Scanning dependencies of target InfluxDB
[100%] Linking CXX shared library libInfluxDB.so
[100%] Built target InfluxDB
cmake --build . --target install
[ 50%] Built target InfluxDB-Core
[ 62%] Built target InfluxDB-BoostSupport
[ 87%] Built target InfluxDB-Internal
[100%] Built target InfluxDB
Install the project...
-- Install configuration: "RelWithDebInfo"
-- Installing: /tmp/ifdbcxx_repro/deps/lib/libInfluxDB.so.1.0.0
-- Installing: /tmp/ifdbcxx_repro/deps/lib/libInfluxDB.so.1
-- Set runtime path of "/tmp/ifdbcxx_repro/deps/lib/libInfluxDB.so.1.0.0" to ""
-- Installing: /tmp/ifdbcxx_repro/deps/lib/libInfluxDB.so
-- Up-to-date: /tmp/ifdbcxx_repro/deps/include
-- Installing: /tmp/ifdbcxx_repro/deps/include/Proxy.h
-- Installing: /tmp/ifdbcxx_repro/deps/include/Point.h
-- Installing: /tmp/ifdbcxx_repro/deps/include/InfluxDB.h
-- Installing: /tmp/ifdbcxx_repro/deps/include/InfluxDBException.h
-- Installing: /tmp/ifdbcxx_repro/deps/include/InfluxDBFactory.h
-- Installing: /tmp/ifdbcxx_repro/deps/include/Transport.h
-- Installing: /tmp/ifdbcxx_repro/deps/include/influxdb_export.h
-- Installing: /tmp/ifdbcxx_repro/deps/lib/cmake/InfluxDB/InfluxDBTargets.cmake
-- Installing: /tmp/ifdbcxx_repro/deps/lib/cmake/InfluxDB/InfluxDBTargets-relwithdebinfo.cmake
-- Installing: /tmp/ifdbcxx_repro/deps/lib/cmake/InfluxDB/InfluxDBConfig.cmake
-- Installing: /tmp/ifdbcxx_repro/deps/lib/cmake/InfluxDB/InfluxDBConfigVersion.cmake
cd /tmp/ifdbcxx_repro/influxdb-cxx/test_package
mkdir build
cd build
cmake -DInfluxDB_DIR=/tmp/ifdbcxx_repro/deps/lib/cmake/InfluxDB/ -Dcpr_DIR=/tmp/ifdbcxx_repro/deps/lib/cmake/cpr/ ..
-- The CXX compiler identification is GNU 9.4.0
-- Check for working CXX compiler: /usr/bin/c++
-- Check for working CXX compiler: /usr/bin/c++ -- works
-- Detecting CXX compiler ABI info
-- Detecting CXX compiler ABI info - done
-- Detecting CXX compile features
-- Detecting CXX compile features - done
-- Found CURL: /usr/lib/x86_64-linux-gnu/libcurl.so (found version "7.68.0")
-- Looking for C++ include pthread.h
-- Looking for C++ include pthread.h - found
-- Performing Test CMAKE_HAVE_LIBC_PTHREAD
-- Performing Test CMAKE_HAVE_LIBC_PTHREAD - Failed
-- Looking for pthread_create in pthreads
-- Looking for pthread_create in pthreads - not found
-- Looking for pthread_create in pthread
-- Looking for pthread_create in pthread - found
-- Found Threads: TRUE
-- InfluxDB 0.7.1 found
-- Configuring done
-- Generating done
-- Build files have been written to: /tmp/ifdbcxx_repro/influxdb-cxx/test_package/build
cmake --build . -j
Scanning dependencies of target test_package
[ 50%] Building CXX object CMakeFiles/test_package.dir/example.cpp.o
[100%] Linking CXX executable test_package
[100%] Built target test_package

Well that seemed to work.

But the final call to test_package fails due to a missing cpr SO, right?
At least that's the error I can reproduce.

Yes, I do get that error when trying to run it.
./test_package: error while loading shared libraries: libcpr.so.1: cannot open shared object file: No such file or directory
root@dev03:/tmp/ifdbcxx_repro/influxdb-cxx/test_package/build#

I have pushed a fix to 201-cpr_so_linking branch, could you give it a try please?
--> The script above works and ldd shows correct results.

Hello, thanks for looking into it!
Tried running your script but still get the same error on a clean Ubuntu install. Added "-b 201-cpr_so_linking" in the git clone command so it checks out the correct commit. Seems strange that it works for you but not me since you could reproduce the error before. I checked the CMakeLists.txt in src and it had the changes you made so seems like something is still not working.
simon@simon-VirtualBox:/tmp/ifdbcxx_repro/influxdb-cxx/test_package/build$ ldd test_package
	linux-vdso.so.1 (0x00007ffdc57e9000)
	libInfluxDB.so.1 => /tmp/ifdbcxx_repro/deps/lib/libInfluxDB.so.1 (0x00007f6b366fd000)
	libstdc++.so.6 => /lib/x86_64-linux-gnu/libstdc++.so.6 (0x00007f6b36400000)
	libgcc_s.so.1 => /lib/x86_64-linux-gnu/libgcc_s.so.1 (0x00007f6b366cf000)
	libc.so.6 => /lib/x86_64-linux-gnu/libc.so.6 (0x00007f6b36000000)
	libcpr.so.1 => not found
	libm.so.6 => /lib/x86_64-linux-gnu/libm.so.6 (0x00007f6b36319000)
	/lib64/ld-linux-x86-64.so.2 (0x00007f6b36724000)

Strange … We both do exact the same steps, but get different results.
I've tested again: The error is reproducible on master but using the branch (same change as you did) works every time.
Is the ldd output complete? I do have a longer list, but those are system libs only – libcurl is included btw..

Hello!
I've tried again and it still does not work. I attached the terminal output as  .txt file if that could help. Now I tried on Ubuntu 23 instead using g++ 12. If I manually set LD_LIBRARY_PATH with export LD_LIBRARY_PATH=/tmp/ifdbcxx_repro/deps/lib and then run the test_package file it works. I find it really strange that you're not getting the same error.
I tried installing both cpr and this to the normal install directories as well and still had the same issue, but if I set LD_LIBRARY_PATH to /usr/local/lib it worked.
log.txt

I was able to reproduce the issue on Ubuntu 22.04.2 based Github Action runners. :+1:
This is interesting, since the images used for testing above – that did fail on master only – are Ubuntu too.

Calling ldconfig after installation of cpr solves the issue (on both branches).

I have merged the branch mentioned above. While not a solution for everything, it still fixes some issues.

@s1m0n-f were you able to resolve the issue?

Yes, running ldconfig solved the problem for me as well, thanks for the help!


Change default port of CMS

The current default port of the CMS - 3000 - by default interferes with the Spec API.

This issue was fixed in the feature/official-cra merge with the addition of a .env.development file.


Input size limit of 256 bytes

Hello guys,
I really like your library and was hoping to use in a new project for added security but I ran into the input size limit of 256 bytes when saving a value.
I need to store OpenId tokens and I've read that I need some 1024 bytes so the 256 bytes limit really makes it impossible.
In the future, would there be a solution to be able to store bigger strings?
Thanks!

Hi @FilipeMLFreitas ,
That is a shortcoming of the RSA encryption protocol that is used in SecureStorage.
You can take a look at this https://github.com/adorsys/secure-storage-android/issues/28#issuecomment-424394160 as a workaround for what you are trying to achieve.
I am also now in the final stages of developing SecureStorage 2.0 which handles the whole encryption process differently and doesn't have the shortcomings of its predecessor.
So with SecureStorage 2.0 you would have no problem storing long strings, tokens etc.
I am hoping that I can finish and release version 2.0 in the next few weeks so maybe that is something that can be of benefit for you.
SecureStorage2 logic is currently in the develop branch.
Thank you for using SecureStorage

@drilonrecica do you have any update on the SecureStorage 2.0 release that fixes the size limit? Thanks!

I will finish the development and deployment of SecureStorage2 on my fork of SecureStorage (at least temporarily). Watch and star that repository to be informed about it.


kclvm not work on Ubuntu 20.10+

Bug Report
I installed kclvm on ubuntu 22.04 and run kcl and it produces following error:
ImportError: libffi.so.7: cannot open shared object file: No such file or directory

https://askubuntu.com/questions/1286772/libffi-so-7-cannot-open-shared-object-file-no-such-file-or-directory
1. Minimal reproduce step (Required)
docker run -it mcr.microsoft.com/devcontainers/base:ubuntu sh
root ➜ / $ cat  /etc/os-release
PRETTY_NAME="Ubuntu 22.04.1 LTS"
NAME="Ubuntu"
VERSION_ID="22.04"
VERSION="22.04.1 LTS (Jammy Jellyfish)"
VERSION_CODENAME=jammy
ID=ubuntu
ID_LIKE=debian
HOME_URL="https://www.ubuntu.com/"
SUPPORT_URL="https://help.ubuntu.com/"
BUG_REPORT_URL="https://bugs.launchpad.net/ubuntu/"
PRIVACY_POLICY_URL="https://www.ubuntu.com/legal/terms-and-policies/privacy-policy"
UBUNTU_CODENAME=jammy
root ➜ / $ mkdir -p /home/vscode/.kusion
root ➜ / $ wget -c https://github.com/KusionStack/kusion/releases/download/v0.7.1/kusion-ubuntu.tgz -qO - | tar xz -C /home/vscode/.kusion
root ➜ / $ export PATH=$PATH:/home/vscode/.kusion/bin:/home/vscode/.kusion/kclvm/bin
root ➜ / $ kcl

2. What did you expect to see? (Required)
kcl help message
3. What did you see instead (Required)
Traceback (most recent call last):
  File "/home/vscode/.kusion/kclvm/lib/python3.7/runpy.py", line 193, in _run_module_as_main
        "__main__", mod_spec)
  File "/home/vscode/.kusion/kclvm/lib/python3.7/runpy.py", line 85, in _run_code
    exec(code, run_globals)
  File "/home/vscode/.kusion/kclvm/lib/python3.7/kclvm/__main__.py", line 3, in <module>
    import kclvm.internal.kclvm_internal.main as main
  File "/home/vscode/.kusion/kclvm/lib/python3.7/kclvm/internal/kclvm_internal/__init__.py", line 1, in <module>
    import kclvm.config as config
  File "/home/vscode/.kusion/kclvm/lib/python3.7/kclvm/config/__init__.py", line 26, in <module>
    from .modfile_pb2 import KclModFile
  File "/home/vscode/.kusion/kclvm/lib/python3.7/kclvm/config/modfile_pb2.py", line 8, in <module>
    from google.protobuf import reflection as _reflection
  File "/home/vscode/.kusion/kclvm/lib/python3.7/site-packages/google/protobuf/reflection.py", line 51, in <module>

     from google.protobuf import message_factory
  File "/home/vscode/.kusion/kclvm/lib/python3.7/site-packages/google/protobuf/message_factory.py", line 49, in <module>
    from google.protobuf.internal import python_message as message_impl
  File "/home/vscode/.kusion/kclvm/lib/python3.7/site-packages/google/protobuf/internal/python_message.py", line 64, in <module>
    from google.protobuf.internal import extension_dict
  File "/home/vscode/.kusion/kclvm/lib/python3.7/site-packages/google/protobuf/internal/extension_dict.py", line 34, in <module>
    from google.protobuf.internal import type_checkers
  File "/home/vscode/.kusion/kclvm/lib/python3.7/site-packages/google/protobuf/internal/type_checkers.py", line 48, in <module>
    import ctypes
  File "/home/vscode/.kusion/kclvm/lib/python3.7/ctypes/__init__.py", line 7, in <module>
    from _ctypes import Union, Structure, Array
ImportError: libffi.so.7: cannot open shared object file: No such file or directory

4. What is your KusionStack components version? (Required)
v0.7.1

Related issue #111

The KCL with versions above v0.4.4-alpha.2 no longer has built-in python3. This error will not be encountered. We can use the following command to install python3 3 and pip in ubuntu 22+ like #327
apt-get install python3 python3-pip -y


is it possible to make MFE not show up in system tray

Is your feature request related to a problem? Please describe.
A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]
Describe the solution you'd like
A clear and concise description of what you want to happen.
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
Add any other context or screenshots about the feature request here.

Duplicate of #56 and #135


Fix use of CoerceCaretIndex on Text change

What does the pull request do?
fixes exception on clicking in TextBox #2552
What is the current behavior?
Under rare circumstances clicking at the end of a TextBox throws an exception
What is the updated/expected behavior with this PR?
exception free clicking
How was the solution implemented (if it's not obvious)?
The check for \r\n could fail if there was a \r at the end of the Text.
Calls to CoerceCaretIndex were using the previous text on Text changed which I can't believe was intended.
Checklist

[ ] Added unit tests (if possible)?
[ ] Added XML documentation to any related classes?
[ ] Consider submitting a PR to https://github.com/AvaloniaUI/Avaloniaui.net with user documentation

Fixed issues
Fixes #2552

Can you add a unit tests for this?


from oml.lightning.entrypoints.validate import pl_val Error

When I run from oml.lightning.entrypoints.validate import pl_val, it shows error:
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/conda/envs/oml-py3.7/lib/python3.7/site-packages/oml/lightning/entrypoints/validate.py", line 18, in <module>
    from oml.registry.postprocessors import get_postprocessor_by_cfg
  File "/conda/envs/oml-py3.7/lib/python3.7/site-packages/oml/registry/postprocessors.py", line 7, in <module>
    from oml.retrieval.postprocessors.pairwise import (
  File "/conda/envs/oml-py3.7/lib/python3.7/site-packages/oml/retrieval/postprocessors/pairwise.py", line 11, in <module>
    from oml.inference.pairs import (
  File "/conda/envs/oml-py3.7/lib/python3.7/site-packages/oml/inference/pairs.py", line 6, in <module>
    from oml.datasets.pairs import EmbeddingPairsDataset, ImagePairsDataset
  File "/conda/envs/oml-py3.7/lib/python3.7/site-packages/oml/datasets/pairs.py", line 7, in <module>
    from oml.datasets.list_dataset import ListDataset, TBBoxes
  File "<fstring>", line 1
    (len(bboxes)=)
                ^
SyntaxError: invalid syntax

How to fix it ?

@PapaMadeleine2022
Hey, this kind of syntax is only working for python 3.8 and later. I will refactor this part of the code so it does not need 3.8.
You can wait for it or update your python.

@AlekseySh  Get it! thanks

@PapaMadeleine2022 I've released a new version of the repo where I fixed the problem above: 0.3.14. It will be available from PIP soon (usually in ~30 minutes)


Linker wrapper to produce VELF files

Rely on rust-psp implementation while development. Use Microsoft DLLs-like structure.

Wrapped linker is not included, it should exist within user's environment. Use PSVITA_WRAP_LINKER environment variable to deduce which linker to wrap, otherwise rust-lld (subject to change);
Linker should implement ld flavor, rely on ld flavor of the wrapped linker;
Input libraries can be:

Static library (.a, .o);
Vita shared library (.vso);

Produced file can be:

Static library (.a, .o);
Vita shared library (.vso);
Vita executable (.vexe);

Vita executable (.vexe) and vita shared library (.vso) are used to create SELF files;
Vita shared library (.vso) is a zip file containing a public interface static library with nids, and optional implementation module;


Add new service builder codegen

Motivation and Context
https://github.com/awslabs/smithy-rs/pull/1679
Description

Add ServerProtocol interface to allow for server side protocol specific methods.
Make public the structs merged in https://github.com/awslabs/smithy-rs/pull/1679.
Add ServerOperationGenerator, which generates a ZST and implements OperationShape on it.
Add ServerServiceGeneratorV2, which generates the service newtype around a router and a service builder.
Add hidden argument to RustModule which allows modules to be marked with #[doc(hidden)].
Add BuildModifier trait to provide a common interface for extending service builders.
Add Upgradable trait to simplifying bounds when upgrading from an Operation to a HTTP service.
Add FromRequest, FromParts, and IntoResponse implementations.
Make RoutingService accept general body types B for the inner services http::Response<B>.

Notes

All new generated code is placed in operation_shape.rs and service.rs. These are both marked as #[doc(hidden)] for now.
The generated diff should be examined extensively.
Follow up tasks:

https://github.com/awslabs/smithy-rs/issues/1707
https://github.com/awslabs/smithy-rs/issues/1706
https://github.com/awslabs/smithy-rs/issues/1705
https://github.com/awslabs/smithy-rs/issues/1703

Closes https://github.com/awslabs/smithy-rs/issues/1705


introduce linter/formater

We need a linter/formater to unsure that our code will have same style everytime
Possible way to handle that

standard
noe standard
eslint + prettier
biome

Things need to be done:

have the format script that call the linter and the formater
have vsc settings enabled to directly format the code on save (because Jakob and I use vsc)
have commit hooks (maybe husky)
have pr/main workflow that check code quality

Let's go for biome :)


Add removeAttestationSigner to ReleaseGold contract

TLDR:

There is no wrapper function for removeAttestationSigner() in ReleaseGold.sol from Accounts.sol
ASv1 operators that use ReleaseGold (i.e. former cLabs employees) can authorise but not (!) deauthorise their attestation signers
We’re sunsetting ASv1, so ReleaseGold validators will want to call removeAttestationSigner() to complete the sunsetting.

https://gist.github.com/0xarthurxyz/cad6535d5ba69f749804b89e832c8034

Eng team decided against patching this small issue because:

no actual behaviour change (after Attestations.sol will be made read-only)
best practice: “minimise changes to smart contracts”


Add Pettier

What does this project do?
Randomizes your Prettier config.
Why is that entertaining?
Anything that uses semicolons is a joke. 😉
How hard did you laugh? Was it just a silent giggle or was it full blown laughing and howling and screaming "OH MY GOD! EVERYONE! OH MY GOD! THIS IS THE FUNNIEST THING EVER! YOU'VE GOT TO SEE THIS!!!"
NaN
CC @coleturner thanks for seizing the memes of production

Lmao @ semicolons


fix StringLengthPtr documentation error for SQLGetEnvAttr, SQLGetConnectAttr, SQLGetStmtAttr

Closes  #5885

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

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

Thanks for the contribution!
#sign-off

Thanks for the contribution!
#sign-off

Thanks for the contribution!
#sign-off

Thanks for the contribution!
#sign-off

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

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


EEException: Earth Engine memory capacity exceeded

hydro-engine version: 0.0.30
Python version: 3.6.6
Operating System: OS

Description
I'm trying to download many ~100,000 AOIs using the hydroengine.get_catchment() function, but after ~10,000 AOIs, the script fails with the error EEException: Earth Engine memory capacity exceeded, I've made sure to include a sleep in the code so to not exceed the GEE rate limit, and I feel confident that my AOIs are not too big.
What I Did
    raise HTTPError(http_error_msg, response=self)
requests.exceptions.HTTPError: 500 Server Error: INTERNAL SERVER ERROR for url: http://hydro-engine.appspot.com/get_catchments

Do you have any suggestions for how to avoid the EEException: Earth Engine memory capacity exceeded error? Thanks for your time.

Most probably this has to do with the size of the asset. Currently, this is limited to ~1GB (the limit of EE for getDownloadURL() function). I only wonder if it's EE that limits that or if something else happening. Did you try to repeat request only for that last failing catchment after some longer delay?
The solution could be to reduce the region size and querying data per smaller sub-catchment, but you've indicated that it is already not too large.
It could be that EE enforces some quota limits per service account as well, we may need to check with EE team to see if that is the case.

Hello thanks for your response.
So this is an example tile:
{'type': 'Polygon',
 'coordinates': (((-101.6015625, 32.6953125),
   (-101.6015625, 33.046875),
   (-101.953125, 33.046875),
   (-101.953125, 32.6953125),
   (-101.6015625, 32.6953125)),)}

I just repeated the request for just the single failing catchment (geometry included above) and it ran without any issue.
I think you're right that  there is an EE quota going on...  Do you know how I should contact them to get more information about that?

@tylere, do you maybe know what kind of quota is there used for service accounts here?

@martham93, for this kind of global exports it might be easier to turn directly to Earth Engine and export these datasets for the whole world, and then just clip them with the regions you're interested in locally. I can share a script to access the underlying datasets if that is an option.

okay that would be great thank you so much! I would love to see the scrip you use with EE directly

The following script basically emulates get-raster: https://code.earthengine.google.com/ac907d9fad3793ba58e829049b2febf7, you can probably export what is needed using Export.image.toDrive() or to Cloud Storage.

thank you so much I'll work with that script.

Service accounts have the same quota restrictions as standard user accounts.
For exports of this size, you will be better off using a batch Export, as @gena suggested.


test_id:1511 is failing often on k8s-1.19 lane

Is this a BUG REPORT or FEATURE REQUEST?:

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

What happened:
On all 1.19 lane fails the test test_id:1511 is the one to be seen most. Before making the lane 1.19 a default we need to investigate this failure.
Examples:

https://prow.apps.ovirt.org/view/gcs/kubevirt-prow/pr-logs/pull/kubevirt_kubevirt/4247/pull-kubevirt-e2e-k8s-1.19/1308907636502040577
https://prow.apps.ovirt.org/view/gcs/kubevirt-prow/pr-logs/pull/kubevirt_kubevirt/4235/pull-kubevirt-e2e-k8s-1.19/1308892285437480960
https://prow.apps.ovirt.org/view/gcs/kubevirt-prow/pr-logs/pull/kubevirt_kubevirt/4156/pull-kubevirt-e2e-k8s-1.19/1308846233468538880
https://prow.apps.ovirt.org/view/gcs/kubevirt-prow/pr-logs/pull/kubevirt_kubevirt/4051/pull-kubevirt-e2e-k8s-1.19/1308807856631320576
https://prow.apps.ovirt.org/view/gcs/kubevirt-prow/pr-logs/pull/kubevirt_kubevirt/4248/pull-kubevirt-e2e-k8s-1.19/1308772372039340032
https://prow.apps.ovirt.org/view/gcs/kubevirt-prow/pr-logs/pull/kubevirt_kubevirt/4235/pull-kubevirt-e2e-k8s-1.19/1308756016162148352

Output:
[rfe_id:150][crit:high][vendor:cnv-qe@redhat.com][level:component]Networkpolicy when three cirros VMs with default networking are started and serverVMI start an HTTP server on port 80 and 81 and connectivity between VMI/s is blocked by Default-deny networkpolicy [test_id:1511] should fail to reach serverVMI from clientVMI

/root/go/src/kubevirt.io/kubevirt/tests/utils.go:3008
Test Panicked
/gimme/.gimme/versions/go1.13.14.linux.amd64/src/runtime/panic.go:679

Panic: interface conversion: runtime.Object is *v1.Status, not *v1.Event

Full stack:
panic(0x1d2eca0, 0xc001f34c90)
	/gimme/.gimme/versions/go1.13.14.linux.amd64/src/runtime/panic.go:679 +0x1b2
kubevirt.io/kubevirt/tests.(*ObjectEventWatcher).Watch.func3(0x24a4be0, 0xc000d50040, 0xc001a763a0, 0xc00315de00)
	/root/go/src/kubevirt.io/kubevirt/tests/utils.go:359 +0x144
created by kubevirt.io/kubevirt/tests.(*ObjectEventWatcher).Watch
	/root/go/src/kubevirt.io/kubevirt/tests/utils.go:356 +0x513

What you expected to happen:
Test should not fail.
How to reproduce it (as minimally and precisely as possible):
Anything else we need to know?:
Environment:

KubeVirt version (use virtctl version):
Kubernetes version (use kubectl version):
VM or VMI specifications:
Cloud provider or hardware configuration:
OS (e.g. from /etc/os-release):
Kernel (e.g. uname -a):
Install tools:
Others:

Related kubevirt build watcher issue: https://github.com/kubevirt/kubevirt/issues/4239

Closing this in favor of other issue, PR related seems ready: https://github.com/kubevirt/kubevirt/issues/4239

I could not reproduce this issue on the latest master with 2 k8s 1.19 nodes, seems fixed indeed.


Use the eigenvalue estimation of PreconditionRelaxation

Description of the problem
The eigenvalue estimation for the smoother of the multigrid preconditioners was done through a PreconditionChebyshev.
Description of the solution
Due to recent changes in deal.II (https://github.com/dealii/dealii/pull/16742), now we are able to do it using the PreconditionRelaxation directly. This allows us to remove the estimate omega function since it is done internally by the smoother if the relaxation parameter is set to 1. This allows also to remove the degree parameter that was only used in the case of Chebyshev.

I guess we need to drop deal.II 9.5 CI?

I will check the deal.II version as we did here: https://github.com/lethe-cfd/lethe/blob/82095bab9286844d9f1cec0dab67cf3575c1791a/source/solvers/mf_navier_stokes.cc#L651


Pluggable Widget Latest version is not working!

When i installed pluggable widget code from yoeman generator today, tt installs the folders and structure well but when building the widget it fails with below error.
Running MX Widgets Tools script build:web...
[!] Error: Cannot find module '../dist/typings-generator'
When i inspected i saw that, the @mendix/pluggable-widget-tools (9.23.0) is searching for ../dist/typings-generator folder in it. however the whole dist folder is missing!
Now i tried adding version 9.18.0 and all works well.
Can you please fix it?

Hi @khazaddoom, thanks for reporting the problem.
A MR with the fix has been opened. We will soon release a new patch to fix this.
https://github.com/mendix/widgets-tools/pull/31

Bug fixed. You can now install the latest pluggable-widgets-tools package to solve the problem.
npm i @mendix/pluggable-widgets-tools@latest


[WFLY-19339] Use Phase.STRUCTURE_NAMING_JDK_DEPENDENCIES that was created for JdkDependenciesProcessor

https://issues.redhat.com/browse/WFLY-19339

I'm, going to ignore the "Cloud Tests Trigger" CI failure as that's https://wildfly.zulipchat.com/#narrow/stream/174184-wildfly-developers/topic/Cloud.20Test.20failures/near/456711214.

Thanks @ropalka


[BUG] - Frontend - Feature MatchViewportRes not quite working.

**UE Version: **
E.g. UE 5.1.1
**Frontend Version: **
0.5.1
Problem component
Frontend
Description
The Flag MatchViewportRes doesn't seems to alway do its job, in particular when click and dragging the side of the browser.
This seems to be caused by the event filtering that is happening here
https://github.com/EpicGames/PixelStreamingInfrastructure/blob/eea7839f513056ebbd0f98930eb8b8ca895d6bf1/Frontend/library/src/VideoPlayer/VideoPlayer.ts#L213
(removing that filter fix the problem)
In particular the function referenced by setTimeout seems to never beeing called due to the timeout
https://github.com/EpicGames/PixelStreamingInfrastructure/blob/eea7839f513056ebbd0f98930eb8b8ca895d6bf1/Frontend/library/src/VideoPlayer/VideoPlayer.ts#L232
Frontend dev is not my speciality so I am not sure why this is hapenning and how to fix it but I'm curious to understand what's going on here.
Steps to Reproduce:

Set the MatchViewportRes flag to 'true' in the react frontend implementation
Start react frontend
Resize the window by click and dragging the size of the browser.
Observe the black bands around the video stream (ie the resolution of the stream doesn't match the resolution of the window.

Platform (please complete the following information):

OS: Windows 11 Pro
Chrome Version 113.0.5672.127

Same here. Seems it does ignore resize of any kind


Check stack count before opening shulker box

Some servers make empty shulker boxes stackable via stacking plugins, need to check the stack count before opening them.
I do this in other plugins
public void onClick(InventoryClickEvent e) {
    ItemStack shulkerItem = e.getCurrentItem();
    // Do not open if stacked: compatible stacking plugin
    if(shulkerItem != null && shulkerItem.getAmount() != 1) return;

    // ...
}

Thanks for the request
I think the fix will not be just as simple as checking the stack count
What happens if someone stacks empty shulker boxes while this plugin has an open shulker box?
Changes are only saved to the stack when the inventory gets closed so I think you would be able to get a duplication bug by:

Opening an empty shulker box
Putting items in it
Stacking the shulker box with other empty boxes
Close shulker box (my plugin will now save the contents to every shulker box in the stack)
You have duplicated items

I hope that makes sense, what do you think about this?
Also, if you have a plugin to hand that allows empty shulker stacking can you send it to me as that means I can do thorough testing with both plugins installed

I think, you fixed it, other shulker boxes cannot be left-clicked while opening a shulker box, but this leads me to another possibility: the player picks up an empty shulker box item after opening , they will be merged
plugin: https://www.spigotmc.org/resources/simple-stack-stack-any-items-to-64.83044/
Please add shulker boxes using the Unique Iten List feature of this plugin, in the config GUI

Yes, setting random NBT data on the shulker box should prevent it from stacking with other boxes and causing issues
So the goal is:

The instant a shulker box is right clicked, ensure it is a stack of 1 + mark it with random NBT
The instant a shulker box is closed (including if the player dies or disconnects), ensure the NBT is removed

Now during a shulker box being opened it should be impossible for identical shulker boxes to stack with it
Is there a way this could fail if the server crashes while someone has a shulker box open and this leaves the NBT tag on the box forever? If so, should we just override that data when it's next opened and that will fix any glitched boxes?
Also, do you know a good way to set NBT data on item stacks without using net.minecraft.server?
I would prefer not to use it as then the plugin will only target a specific version of Spigot and won't be future proof.
I'm not a very experienced plugin developer so I wouldn't have the time to set up a system to update the plugin for each version of NMS/Spigot

I am studying how to operate nbt, there may be a better nbt library.
I may update the code on another branch: https://github.com/ApliNi/Shulkerbox


Add Docker support

Add Dockerfile to enable image building.
By now, I use the official Node image, 'onbuild' tag. More info at https://hub.docker.com/_/node/ . Should the build be pinned to any specific version, let me know.
Build:
$ docker build -t webtorrent-cli .

Run:
$ docker run -rm -it --name webtorrent -p 8000:8000 webtorrent-cli <magnet-link>

FYI, there's a still quicker to test, already built image on my Docker Hub. Test it by running:
$ docker run -rm -it --name webtorrent -p 8000:8000 pataquets/webtorrent-cli <magnet-link>

Using --rm instead of -d makes the container not go background and it to be deleted after stop. Stop it by CTRL+C'ing it. Also, you might want to add one (or more) -v /host/dir/path:/container/path/ options in order to mount host's directories into the container to use .torrent files.
More optional improvements to come:

Create an 'official' automated build at Docker Hub for the image: https://docs.docker.com/docker-hub/builds/

@feross: Bump

Thanks for the PR, but I don't use Docker and I don't understand why this can't be done in your own repo.

Thanks for the review @feross .
Aside from easing non-devel people using and supporting the tool, more and more devs are using Docker as a repeatable local dev/test environment in their workflows. For newly entering contributors, the docker build/docker run dev/test setup lowers the entry barrier a lot.
It also keeps the Docker recipe a canonical place where it is maintained and discussed and where ppl can contribute improvements to, or at least have a minimal base to start from. For people using Docker (like myself) in absence of an official one, just pulling upstream changes it's all what's needed to build one. In addition, Docker-friendliness gives confidence to anyone thinking to step up as Docker official image maintainer. It also might help yourself to adopt Docker as just another way of packaging and distributing the software by just creating an official Automated Build in Docker Hub (has even a image pulls counter). I think that with time, more people will ask for an 'official' (automatically built from source, that is trustable) Docker Hub image.
Overall, makes people aware of Docker as another runtime environment and can attract other useful contributions in the long run, IMHO, even if you are not using it (by now :wink:). If you have problems or other requirements with using Docker, I can help you, also.
It can't hurt having just another tool at everyone's disposal, even if you warn clearly in the readme that it is not officially supported. Said warning might even attract more hands eager to help.

Aside from easing non-devel people using and supporting the tool

I think people that knows how to use Docker are proficient enough to know how to use or install this package.

For newly entering contributors, the docker build/docker run dev/test setup lowers the entry barrier a lot.

For Node.js users, it's already really easy to get started:
npm install
npm test

I appreciate where you're coming from, but I respectfully disagree. Docker doesn't solve any problems that I have.


Curated_List_of_all_Online_Python_resources Issue No#598

Description 📜
Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context.
Fixes #598

Type of change 📝

[ ] Au#dio (Should be in mp3 format Includes speech clarity, Concise ,Low distortion)
[ ] Vi#deo (Animations, screen-recordings, presentations and regular explanatory films are all possibilties etc)
[x] Doc#umentation (Content Creation in the form of codes or tutorials)
[ ] Other (If you choose other, Please mention changes below)

Domain of Contribution 📊

[ ] Android Dev #(Flutter)
[ ] Android Dev #(Java)
[ ] Android Dev #(Kotlin)
[ ] Backend Dev #(Java)
[ ] Backend Dev #(.NET)
[ ] Backend Dev #(PHP)
[ ] Backend Dev #(Python)
[ ] C/#CPP
[ ] Competitive #Programming
[ ] Cyber #Security
[ ] DS#A
[ ] Data#base
[ ] Datascience with #Python
[ ] Datascience with #R
[ ] Frontend Dev #HTML/CSS/JS
[ ] Frontend Dev #React/Angular/Vue
[ ] Go#lang
[ ] Interview #Prep
[ ] Java_#Domain
[ ] Java#Script
[ ] ME#RN
[ ] Machine #Learning
[ ] Open #Source
[x] Python_#Domain
[ ] Ru#st
[ ] Statis#tics
[ ] UI/#UX

Checklist ✅

[x] I follow Contributing Guidelines & Code of conduct of this project.
[x] I have performed a self-review of my own code or work.
[x] I have commented my code, particularly in hard-to-understand areas.
[x] My changes generates no new warnings.
[x] I'm GWOC'21 contributor

Screenshots / Gif (Optional) 📸

@Randrita Created a new PR, everything looks good to me. Kindly review it and if you think any more changes are required do let me know. Thanks again!!

will you please update the readme of the python folder as well @Varun270 ?
I personally feel the list is so straightforward and short! What's your view on this? @kumarankm

Yes, add little more info like what are the things they teach in that videos and sites in a short description. And you didn't fetch upstream kindly fetch it @Varun270

will you please update the readme of the python folder as well @Varun270 ?
I personally feel the list is so straightforward and short! What's your view on this? @kumarankm

Yes, add little more info like what are the things they teach in that videos and sites in a short description. And you didn't fetch upstream kindly fetch it @Varun270

Okay , I will add some more info.

will you please update the readme of the python folder as well @Varun270 ?
I personally feel the list is so straightforward and short! What's your view on this? @kumarankm

Where should I add the folder name that I created Learning path or Index?

@Varun270 the readme.md file of python folder! check how all the contributors have added the folder name. I am attaching the link!
https://github.com/girlscript/winter-of-contributing/blob/Python/Python/README.md

@Randrita I have pushed some more changes in the PR. Kindly review it.


scipy.spatial.distance.wminkowski behaviour change in 1.0.0b1

scipy.spatial.distance.wminkowski does not give the same result in 0.19.1 and in 1.0.0b1. We noticed that because it was breaking some tests in scikit-learn. It feels to me that even if wminkowski is considered deprecated it should return the same result in 1.0 as in 0.19.1.
This seems related to https://github.com/scipy/scipy/pull/7630 and https://github.com/scipy/scipy/issues/5718. It seems like the discussion was around whether to raise the weights to the power of p or not.
Reproducing code example:
import numpy as np

import scipy
from scipy.spatial.distance import wminkowski

print(scipy.__version__)
arr = np.arange(4)
print(wminkowski(arr, arr + 1, p=2, w=4))

Output
scipy 1.0.0.b1 (weights are not squared):
1.0.0b1
4.0

scipy 0.19.1 (weights are squared):
0.19.1
8.0

You're totally right. We should restore the wminkowski implementation and clearly document that it differs from minkowski and that the latter is preferred.
@apbard would you be able to do this?

@rgommers this was expected I think and I added this to the release note in #7881. As @metaperture noted in #6931 the implementation of wminkowski was not implemented according to the right definition. Thus, IMHO this behaviour change could be regarded as a fix for a longstanding bug rather than a real behaviour change. Beside keeping the compatibility, is there some advantage in keeping the wrong implementation?

Beside keeping the compatibility, is there some advantage in keeping the wrong implementation?

backward-compatibility is the main point here I think.

I missed that comment in the very long history of changes to pdist & co.

Beside keeping the compatibility, is there some advantage in keeping the wrong implementation?

Two reasons.

Not giving users, including libs like scikit-learn, an unnecessary amount of work. This is also why I wanted a doc-only deprecation.

We try to never silently change numerical results. Doing that is worse than breaking their code by deprecation and then removal (and introducing a new function with the desired behavior).

This isn't really a bug, it's a matter of interpretation of weights. The new minkowski is arguably better, but I don't think that's enough to say "it's a bug" and be done.
So I think we'll want to:

now restore the old behavior with a doc-only deprecation
deprecate ``wminkowski in 1.2 or 1.3
remove it 2 releases after that

@rgommers ok, agreed. i'll do it (probably next week though).

Not giving users, including libs like scikit-learn, an unnecessary amount of work. This is also why I wanted a doc-only deprecation.

Maybe silly question, why not directly the "hard" deprecation (in this case but also in a more general)? Users can still use the function and at the same time they gets a warning about the behaviour change.

Not a silly question. A library like scikit-learn wants to avoid deprecation warnings, so if we deprecate something they want to get rid of it. However, they can't just replace it with something else now, because they support multiple scipy releases, and pre-1.0 we don't give them the alternative. So they're going to have to do some switching based on parsing the scipy version string. That's annoying and also easy to make mistakes in.

Closed by gh-7905


Hotfix/local perf issue

Updated the postNiccUrl and added bearer token to headers. The bearer token needs to be added manually to get a successful run till the auth code is implemented

Ran smoke test locally after updating postNiccUrl


Need to remove Share, Publish items from context menu for empty prompt/chat folder

Now:

Need to remove Share, Publish items from context menu for empty prompt/chat folder

Verified on staging successfully


:arrow_up:one::two: WMC_portal

@Ramil-Mukhametzyanov make your tests please

@Ramil-Mukhametzyanov Module is not related to the error, I reproduced the error even without the module on our runbot with all modules installed


CSDM origin offset and reference issue

The CSDM LinearDimension defines coordinates as
coordinates = $\Delta x(J_k - Z_k) + b_k$,
where $J_k = [0, 1, ... N_k -1]$. When complex_fft=False,  $Z_k$ = 0. When complex fft=True, $Z_k = N_k/2$ for even $N_k$ else $(N_k-1)/2$

coordinates_offset = $b_k$
increment = $\Delta x_k$
count = N

For NMR datasets, here are two cases---when origin_offset is defined at the receiver frequency, and origin_offset is defined at the reference frequency.

origin_offset $= w_\text{RF}$
origin_offset $= w_\text{ref}$

complex_fft = True
$w_\text{ref} = w_\text{RF} - b$
$w_\text{RF} = w_\text{ref} + b$

complex_fft = False, $N$ is odd
$w_\text{ref} = w_\text{RF} - (b + \frac{N-1}{2} \Delta x)$
$w_\text{RF} = w_\text{ref} + (b + \frac{N-1}{2} \Delta x)$

complex_fft = False, $N$ is even, $\Delta x$ > 0
$w_\text{ref} = w_\text{RF} - (b + \frac{N}{2} \Delta x)$
$w_\text{RF} = w_\text{ref} + (b + \frac{N}{2} \Delta x)$

complex_fft = False, $N$ is even, $\Delta x$ < 0
$w_\text{ref} = w_\text{RF} - (b + (\frac{N}{2} - 1) \Delta x)$
$w_\text{RF} = w_\text{ref} + (b + (\frac{N}{2} - 1) \Delta x)$

The conversion from Hz -> ppm is coordinates / ${w_\text{ref}}$. The difference is that when origin_offset is ${w_\text{RF}}$, the value of ${w_\text{ref}}$ depends on how the LinearDimension is defined.

What happens to the origin offset for a data subset?

If the origin offset is $w_\text{ref}$, we retain both frequency and ppm dimension scale for the subset data but lose the information on $w_\text{RF}$. However, does $w_\text{RF}$ has meaning for a data subset?

If the origin offset is $w_\text{RF}$, we lose the information on $w_\text{ref}$. Therefore, the subset data can only be represented in frequency dimension, not dimensionless ppm scale. We retain $w_\text{RF}$, which again seems meaning less for a data subset.

The assumption that the carrier frequency is at the center of the spectrum is the only way to encode both wRF and wref into o and b.


Filename fuzzy match unexpected results

Issue Type: Bug
When using file fuzzy search I regularly get unexpected search results. Illustrative example:

In the above image, the search was for cartenums. The top match is for a file that contains all those letters, in that order, however the next 6 results contain all those letters, in that order, closely together, in what are clearly more relevant files to any human reading the list of results.
For some context which may or may not be useful, this repository is a web app that contains both a large frontend and a large backend codebase. In the file listing, all frontend files are alphabetically before all the backend files. Anecdotally, it seems that the issue only happens this way around.
If I were to guess about the source of the issue, it would be that the search algorithm does not weight results with characters in groups higher than those that just contain the characters in order, however it may be that it does do this and that the weighting is just not at the level needed for my particular use.
I'm happy to help work on this and contribute a fix if I can have some pointers towards where to look and/or the go-ahead that a fix would be wanted. My idea would be to implement this weighting mentioned above in some form.
VS Code version: Code 1.62.3 (ccbaa2d27e38e5afa3e5c21c1c7bef4657064247, 2021-11-17T07:59:13.865Z)
OS version: Darwin x64 20.6.0
Restricted Mode: No

System Info

Item
Value

CPUs
Intel(R) Core(TM) i7-8559U CPU @ 2.70GHz (8 x 2700)

GPU Status
2d_canvas: enabledgpu_compositing: enabledmetal: disabled_offmultiple_raster_threads: enabled_onoop_rasterization: enabledopengl: enabled_onrasterization: enabledskia_renderer: disabled_off_okvideo_decode: enabledwebgl: enabledwebgl2: enabled

Load (avg)
4, 4, 4

Memory (System)
16.00GB (0.09GB free)

Process Argv

Screen Reader
no

VM
0%

Extensions (70)

Extension
Author (truncated)
Version

vscode-apollo
apo
1.19.9

vscode-mjml
att
1.6.0

vscode-bazel
Baz
0.5.0

django-html
bib
1.3.0

unique-lines
bib
1.0.0

markdown-mermaid
bie
1.13.0

touch-bar-display
bla
1.5.0

mermaid-markdown-syntax-highlighting
bpr
1.2.2

pgformatter
bra
1.17.0

better-toml
bun
0.3.2

path-intellisense
chr
2.6.0

gitignore
cod
0.7.0

systemd-unit-file
coo
1.0.6

vscode-mac-color-picker
dae
1.0.2

vscode-eslint
dba
2.2.2

vscode-dash
dee
2.4.0

xml
Dot
2.5.1

monitrc
dru
0.0.1

gitlens
eam
11.7.0

EditorConfig
Edi
0.16.4

LogFileHighlighter
emi
2.13.0

prettier-vscode
esb
9.0.0

file-icons
fil
1.0.29

copilot
Git
1.7.4016

vscode-pull-request-github
Git
0.32.0

vscode-graphql
Gra
0.3.41

vscode-nginx-conf-hint
han
0.3.0

terraform
has
2.17.0

jsonnet
hep
0.1.0

magit
kah
0.6.24

procfile-support
log
0.0.1

syntax-project-pbxproj
mar
0.1.3

vscode-github-actions
me-
3.0.1

rainbow-csv
mec
1.10.1

xcodestrings
mhc
1.0.0

dotenv
mik
1.0.1

vscode-html-format
moh
0.0.4

vscode-duplicate
mrm
1.2.1

language-gettext
mro
0.2.0

vscode-docker
ms-
1.18.0

vscode-kubernetes-tools
ms-
1.3.4

python
ms-
2021.12.1559732655

vscode-pylance
ms-
2021.12.1

jupyter
ms-
2021.10.1101450599

jupyter-keymap
ms-
1.0.0

jupyter-renderers
ms-
1.0.4

remote-containers
ms-
0.205.2

vsliveshare
ms-
1.0.5196

vs-sequential-number
nep
1.1.0

tuck
Pet
0.1.3

erlang
pgo
0.8.2

vscode-commons
red
0.0.6

vscode-yaml
red
1.2.2

rust
rus
0.7.8

partial-diff
ryu
1.4.3

jinjahtml
sam
0.16.0

elm
sbr
0.26.0

crates
ser
0.5.10

vscode-nginx
sha
0.6.0

markdown-preview-enhanced
shd
0.6.1

stylelint
shi
0.51.0

fish-vscode
sky
0.2.1

vscode-fileutils
sle
3.4.5

rewrap
stk
1.15.4

vscode-mermaid-editor
tom
0.11.4

change-case
wma
1.0.0

jsonnet-formatter
xrc
0.3.0

markdown-pdf
yza
1.4.4

vscode-open-in-github
ziy
1.3.6

vscode-proto3
zxh
0.5.5

(9 theme extensions excluded)

Would be interested in any developer thoughts on whether this should be solved at the VSCode level, in vscode-ripgrep, or in ripgrep itself.

Duplicate of https://github.com/microsoft/vscode/issues/25925

Yeah this is because the first item in your quick pick contains all of the letters inside of the name of the file while the other items don't - they have all of the letters inside the full path. This is a dupe of #25925


Add a comment explaining why we say can handle false always

We always return false since we don't want people to actually use it with the JDBC connector and we should clarify that in a comment that we're just doing that to access some of Spark's internals.

Cleaned the obsolete issue.


Rendering or wording is strange for "Waypoint: Style Text Inputs as Form Controls"

In this Waypoint the wording "Give the text input in your form the class "form-control"." or perhaps the highlighting is hard to understand.
If I understood my misgivings better, I would correct it in this line of code...  but unsure how or why it seems strange.

Seems this has been reported a few times - bug #'s 1383, 1328, 1281, 1082
Looks like QuincyLarson fixed this on 20 June, so I'm assuming it just hasn't yet been pushed out.


Javascript type inference problem for classes if constructor argument number doesn't match

From @chanep on December 19, 2016 13:53

VSCode Version: 1.8.0
OS Version: Any

VScode doesn't detect correctly instance type if class constructor is called with different number of arguments.
Example:
class Person{
    constructor(arg){
        this.arg = arg;
    }
}

let p1 = new Person(null);      //OK, p1 is type Person
let p2 = new Person();           //WRONG, p2 is type "any"

I think it is a little annoying to have to put null's all over the place
Copied from original issue: Microsoft/vscode#17539

For JavaScript, I think that TypeScript should be more relaxed when looking at constructor arguments.  Users can currently workaround this issue by explicitly specifying that arg is optional:
class Person{
    constructor(arg = null){
        this.arg = arg;
    }
}

but leaving arguments implicitly specified is still a common pattern.

I believe we do this correctly for call signatures, but it's not working for construct signatures for some reason

Hi, is there any news abnout this?

No longer repros - seems this has been fixed


ENH: add instructions for drift check

Closes #305

This view also makes it easier to check if the thresholding is good.


Refreshextracts returns 403 but working

Hi,
As part of the testing of the TabCmd 2.0 we noticed that there is an issue when content owners have a creator license when using the refreshextract parameter they get a 403 returned to them, even though they are the owner of said data source they are requesting to refresh. If the user is a Site Administrator they do not encounter this issue.
Interestingly enough it is actually working but form some reason there is an error getting returned making it look like it's not working. In the screenshot below you can see that I executed the command and receive a 403 but when I immediately do the same command again I get a different response saying it's already queued. I can also confirm on the jobs tab of Tableau Online that the job was kicked off to refresh the data source.

That's very odd - I have noticed the error, but never actually checked to realize that it was scheduling the job anyway. That might make it easier to fix. Thanks!

I got someone else to look into this and they couldn't repro it - now I can't either, although I'm sure I saw it in the past. Is this still happening for you, @RobertFoltzDBRS ?

Hi @jacalata and @fholt
I haven't tested the latest releases, and I personally have just been using the API. I'll see if using version 2.0.10 of tabcmd works.
Thanks,
Robert

Hi @RobertFoltzDBRS @fholt,
It would help greatly if you could share the following information for your error:

Tableau Cloud server name
Tableau Cloud site name
User name
Timestamp

@RobertFoltzDBRS we have most of the information from your screenshot, please include the timestamp as well.
Thank you,
Don

"c:\tabcmd\tabcmd" refreshextracts --server "https://us-east-1.online.tableau.comhttps://us-east-1.online.tableau.com/" --site "skylarcapital" --token-name "FosterPAT" --token-value "" --project "Default" --datasource "wx_Latest_Daily (EuropeDB)"
User name is<EMAIL_ADDRESS>Not sure what you mean by Timestamp.
From: dkwon-salesforce @.>
Sent: Monday, February 6, 2023 2:25 PM
To: tableau/tabcmd @.>
Cc: Foster Holt @.>; Mention @.>
Subject: Re: [External] [tableau/tabcmd] Refreshextracts returns 403 but working (Issue #189)
Hi @RobertFoltzDBRShttps://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_RobertFoltzDBRS&d=DwMCaQ&c=euGZstcaTDllvimEN8b7jXrwqOf-v5A_CdpgnVfiiMM&r=15KpEzJNOnn_sj5fZQzthEZXcqNfnTZRJA9J1O2CeKs&m=CJ5x-K8TvxUo3_AxghyXheaM8PZsM93HTmOkAt2t0-k&s=yK6LEtWDzpVrJAVtNkoYedC3Of_8GOrsIpTFAeEM3gI&e= @fholthttps://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_fholt&d=DwMCaQ&c=euGZstcaTDllvimEN8b7jXrwqOf-v5A_CdpgnVfiiMM&r=15KpEzJNOnn_sj5fZQzthEZXcqNfnTZRJA9J1O2CeKs&m=CJ5x-K8TvxUo3_AxghyXheaM8PZsM93HTmOkAt2t0-k&s=xPY8ZrbjdRr51iUb_XvxenBku2TBQLIPbtyHlZqGwrU&e=,
It would help greatly if you could share the following information for your error:

Tableau Cloud server name
Tableau Cloud site name
User name
Timestamp

@RobertFoltzDBRShttps://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_RobertFoltzDBRS&d=DwMCaQ&c=euGZstcaTDllvimEN8b7jXrwqOf-v5A_CdpgnVfiiMM&r=15KpEzJNOnn_sj5fZQzthEZXcqNfnTZRJA9J1O2CeKs&m=CJ5x-K8TvxUo3_AxghyXheaM8PZsM93HTmOkAt2t0-k&s=yK6LEtWDzpVrJAVtNkoYedC3Of_8GOrsIpTFAeEM3gI&e= we have most of the information from your screenshot, please include the timestamp as well.
Thank you,
Don
—
Reply to this email directly, view it on GitHubhttps://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_tableau_tabcmd_issues_189-23issuecomment-2D1419703303&d=DwMCaQ&c=euGZstcaTDllvimEN8b7jXrwqOf-v5A_CdpgnVfiiMM&r=15KpEzJNOnn_sj5fZQzthEZXcqNfnTZRJA9J1O2CeKs&m=CJ5x-K8TvxUo3_AxghyXheaM8PZsM93HTmOkAt2t0-k&s=SHn9lCY_RBcf4h9wnnv5NJ9_WQiHn9rkUdrsBwDw67g&e=, or unsubscribehttps://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_notifications_unsubscribe-2Dauth_AI4BO2XW26KWF7YLPETGM4DWWFM3TANCNFSM6AAAAAARDMWLBA&d=DwMCaQ&c=euGZstcaTDllvimEN8b7jXrwqOf-v5A_CdpgnVfiiMM&r=15KpEzJNOnn_sj5fZQzthEZXcqNfnTZRJA9J1O2CeKs&m=CJ5x-K8TvxUo3_AxghyXheaM8PZsM93HTmOkAt2t0-k&s=qKPYO51JQG-3ls88Q736JIm5BCTrSiyCls9ya0qRjzc&e=.
You are receiving this because you were mentioned.Message ID<EMAIL_ADDRESS>

This has been fixed internally, I believe will be in the 23.2.0 release.


alphabetize malayam courses list

What does this PR do?
The f-p-b linter doesn't check the ordering af malayam lists
Why is this valuable (or not)?
As lists grow, it's hard to identify duplicates unless there is a standardized ordering.

@RijoKsd would appreciate a quick review

What does this PR do?
Added resources and improved repo
Why is this valuable (or not)?
This open-source contribution is valuable because it allows Malayali students to learn coding in their native language, Malayalam. By making coding more accessible to students in their native language, this open-source contribution can help to increase access to coding resources and potentially open up a new world of opportunities for Malayali students.
NB:  Malayalam is the native language of Kerala. Kerala is situated in INDIA


ssh connection error

running this
npm install cypress-visual-regression --save-dev
getting this:
Error while executing:
npm ERR! C:\Program Files\Git\cmd\git.EXE ls-remote -h -t<EMAIL_ADDRESS>npm ERR!
npm ERR! ssh: connect to host github.com port 22: Connection timed out
npm ERR! fatal: Could not read from remote repository.
I am able to  install other things just fine. any ideas? on windows.

Not sure. Does this help?