Create svc causing Deployment spec lost if the spec is not supported in Rancher API

v2.3.0-rc8
Steps:

Create a deployment with windowsOptions fields via YAML
Deployment up and working as expected
Create a node port svc with label selector

Results:
Deployment will be updated and new pods will be created.
And the windowsOptions fields lost in the spec.
Not sure if the fields lost when Rancher update the annotations for the deployment
In k8s 1.16, windowsOptions fields are introduced. They are very important fields for Windows container. However, they are not supported in Rancher API.
https://kubernetes.io/docs/reference/generated/kubernetes-api/v1.16/#windowssecuritycontextoptions-v1-core f

Types vendor uses kubernetes 1.14.5 so these fields are not available in Rancher API at the moment.

@loganhz Could you please provide the yml used for getting a deployment with windowsOptions fields  ?

Reproduced the issue on rancher:2.3.0-rc8 (single node install)
Steps:

Create a custom windows cluster with these args in the edit as yaml file

    kube-api:
      extra_args:
        feature-gates: WindowsRunAsUserName=true

Cluster spec: 1 linux all roles, 1 windows worker node - AWS nodes, k8s1.16
Import a workload using the yaml file with field windowsOptions

apiVersion: apps/v1
kind: Deployment
metadata:
  labels:
    test: three
  name: wk02
  namespace: test2
spec:
  replicas: 1
  selector:
    matchLabels:
      test: three
  template:
    metadata:
      labels:
        test: three
    spec:
      securityContext:
        windowsOptions:
          runAsUserName: "ContainerUser"
      containers:
      - image: maiwj/testcontainer:windows-1903
        name: test-image
        command: ["ping", "-t", "localhost"]
        ports:
        - containerPort: 80
          protocol: TCP

The workload should get deployed successfully.
Do a kubectl get deployment  -n  -o json
Field windowsOptions is available in the spec

"securityContext": {
                    "windowsOptions": {
                        "runAsUserName": "ContainerUser"
                    }
                },

Create a service discovery entry with selector labels (test: three), and a node port service.
You will see the node port is exposed on the Workload.
Do a kubectl get deployment  -n  -o json
The field windowsOptions is not available anymore in the spec

"securityContext": {}

Verified this issue on master-head - commit id: de4448e87 and see that the fields on workload update, do NOT go missing
Steps:

Create a custom windows cluster with these args in the edit as yaml file

    kube-api:
      extra_args:
        feature-gates: WindowsRunAsUserName=true

Cluster spec: 1 linux all roles, 1 windows worker node - AWS nodes, k8s1.16
Import a workload using the yaml file with field windowsOptions (same as given above)
The workload should get deployed successfully.
Do a kubectl get deployment  -n  -o json
Field windowsOptions is available in the spec

"securityContext": {
                    "windowsOptions": {
                        "runAsUserName": "ContainerUser"
                    }
                },

Create a service discovery entry with selector labels (test: three), and a node port service.
You will see the node port is exposed on the Workload.
Do a kubectl get deployment  -n  -o json
The field windowsOptions is available in the spec

"securityContext": {
                    "windowsOptions": {
                        "runAsUserName": "ContainerUser"
                    }
                },


Enable fine grained configuration with external property file

Configuration of the HTTP client
Configuration of the Cqlsession with a .conf file

astra.http.connectTimeout=3
astra.http.readtimeout=3
astra.http.keep-alive=3

Done as of 0.3.0


Pelorus Operator development improvements

We want to to add links to Pelorus Operator form view options description (scripts/pelorus-operator-patches/07_spec_description.diff file description keys), but each time we make a release in Pelorus repo, we would need to change them. Editing diff files is not easy.
Requirements:

Automate process of updating links when a release is created in the repository.
Automate process of getting descriptions to each field (get them from the docs folder).
Have an easier way to develop changes to operator (without diff files)

related to #806

when working on this task, also work to resolve #772


Adding support for migrations

Resolves
Fixes #120
Description
Adding support for migrations

To support migrations we need some re-structuring. We'll most likely need separate migrations for each storage provider (which is why we have specific MLOpsDbContexts)

@Brett-Parker  / @AnoojNair  - this should be ready for a review, which will enable migrations for SQL Server and SQLite. I'm going to write up some docs shortly on how that's is done when we change the underlying models

I've updated the Wiki with documentation on how to run and create migrations: https://github.com/aslotte/MLOps.NET/wiki/Contributing-to-the-repository

If approving this PR, hold off on merging it just so I get a chance to re-base and point to the main branch :)

Closing and re-opening this PR to kick of checks.


Split testing of CloudRecordView into two files

test_cloud_record_helper.py tests the underlying helper methods.

this file contains a new test test_signer_is_valid, which only tests the _signer_is_valid

test_cloud_record_post.py tests the overall post method by making POST requests

this file is essentially the old test_cloud_record.py

Splitting the two type of test makes the files more manageable/smaller. The additional test may catch cases when a POST request is made, _signer_is_valid fails, but the POST request carries on to fail in the way the test was expected, thus not causing a test failure.
I have also moved class variables to the __init__.py file, as PROVIDERS was now being defined twice in two different classes (and I felt like MESSAGE could also be moved encase future tests want to make use of it.)

rebased


Updates variables to be double letter

Also exposes the asteroid class number as a variable for automated laser alignment purposes
New Variables Names are:
Active: AC
Index: ID
ScanResults: SR
Material: MA
Volume: VO
Scan: SC
Reset: RE
Asteroid Class: SZ
This prevents conflicts with ISAN

The  SCC edition version needs adjusting too, but there's also https://github.com/DerPfandadler/Pfandadler-YOLOL/pull/4#issuecomment-919918438

The SCC edition version needs adjusting too, but there's also #4 (comment)

That's why I didn't update it. Just offering an alternative not trying to compete, no need to get upset.

No need to make this personal @th3fallen, but since we're here now: what I usually do if I created a PR that solves an issue that is already being solved by an existing PR, is to add my new findings (if any) to the existing one and close mine as duplicate.

No need to make this personal @th3fallen, but since we're here now: what I usually do if I end up creating a PR that solves an issue that is already being solved by an existing PR, is to add my new findings (if any) to the existing one and close mine as duplicate.

there was nothing personal in my previous response. but since your comments are now bordering on harassment I'll kindly ask you to refrain from polluting this PR with pointless comments.

I will expose the asteroid class as a global variable and adjust device field names.


Fixed:  Prevent release generation in forks

When I just updated my fork, the release workflow created me a release.  The workflow should only work in the original repository, so I fixed this.

@kevinmatthes Thanks for taking this.
I didn't think of this case while reviewing the release PR too :+1:


Fix: LH Unused CSS + Organizes CMS Sections and components styles.

What's the purpose of this pull request?

[x] Enables SlideOver/Modal components to be used as Sections (CartSidebar, NavbarSlider, FilterSlider,RegionModal).
[x] Organizes Sections/Common components into Layout and Pages.
[x] Uses CSS Modules per Sections (3 styles customizations levels, see video in slack channel).
[x] Removes CSS Modules from other components and move styles to @faststore/ui.
[x] Adjusts styles to use the same pattern used in the Atomic Design Epic approach in @faststore/ui.
[x] Removes some stories (Storybook).
[x] Fix Unused CSS Problem for Homepage, PDP, PLP.
[x] Fix conflicts.

How it works?
It prepares the repo to uses the CMS Section approach. It also enables the CSS Modules per section instead of importing all styles globally as before. Finally it fixes the Unused CSS Problem for Homepage, PDP, PLP, Search etc.
How to test it?
All the pages should keep the same visually in Desktop and Mobile versions. You can see the coverage tab in devtools and check the unused CSS compared to previous version using the starter PR.
Note that the unused CSS problem is gone in the LH Check from the starter PR (But there are others LH Issues yet).
Starters Deploy Preview
https://github.com/vtex-sites/starter.store/pull/44

LGTM! I found just a margin in the footer, but will pre-approve!

Ya! Same think for desktop version

@ArthurTriis1  @hellofanny what do you think about this change to fix the footer padding? I wonder if would be better place some styles in the data-fs-footer instead.

@ArthurTriis1 @hellofanny what do you think about this change to fix the footer padding? I wonder if would be better place some styles in the data-fs-footer instead.

I think we can do this, and when the footer is moved, we put these styles in data-attributes

58034cf

Nice catch, Fan.
I changed it here. Can you double check it again?


Usage of non-inclusive language in docs

I noticed that this page makes use of the word "whitelisting". Given that this word is non-inclusive and it is a public facing doc, we should replace that with inclusive wording such as "trusted" or "approved" or "allowed"

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

ID: b067a175-f640-7503-9c1e-f0130c6dbeda
Version Independent ID: ff743c7b-a103-eae6-4478-62ba995a4b36
Content: Pipeline deployment approvals - Azure Pipelines
Content Source: docs/pipelines/process/approvals.md
Product: devops
Technology: devops-cicd-process
GitHub Login: @shashban
Microsoft Alias: shashban

@shashban The term "whitelisted" appears in several screen captures later in the article. Since they are not in text, they didn't throw build suggestions.


Remove PHP v5 support from composer.json

We stopped testing this (PHP v5.4) in v2.6, so we need to remove all support in v3.

We will remove all PHP 5 support from v3.  PHP5 is no longer officially supported.

Since a significant portion of PHP out there is still on 5, we're moving this to v4.


add placement image in clustermanager and factoring version/tag flag

based on this comment https://github.com/open-cluster-management-io/clusteradm/pull/80#issuecomment-989485226
The placement image's tag could be different from registration/work. We might need to have a release version map to the tags maintained here.

/assign ilan-pinto

@qiujian16 we have the following images to render
registration-operator, registration, work

Thanks!

yes.
I think so.

/close


always use "%s"-style format for printf()-style functions

ncuses-6.3 added printf-style function attributes and now makes
it easier to catch cases when user input is used in palce of format
string when built with CFLAGS=-Werror=format-security:
musikcube/cursespp/cursespp/curses_config.h:54:36:
 error: format not a string literal and no format arguments [-Werror=format-security]
   54 |     if (window && format) { wprintw(window, format, ##__VA_ARGS__); }
      |                             ~~~~~~~^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
musikcube/src/musikcube/app/window/TransportWindow.cpp:640:5: note: in expansion of macro 'checked_wprintw'
  640 |     checked_wprintw(c, shuffleLabel.c_str(

Let's wrap all the missing places with "%s" format.

Cleaned up conflicts and opened a new PR with some other community contributions.
Your change will be merged today as part of this PR: https://github.com/clangen/musikcube/pull/504
Closing this one.
Thanks again!


persian translation

You need to create the PR from your fork:branch to hugogiraudel/sass-guidelines:gh-pages. :)

i'm sorry, i said before that i'm not pro but i will figure it out


More tests for all data types

I need to add more tests to detect breakage earlier. Ideally I'd like to cover all of the Nl implementations at minimum.

Work started in branch v0.5.0-dev.


Make it easier to use the Branch test key - AIS-287

Right now, switching between the key_test and the key_live confusing.
If the user can set a boolean or explicitly set the key before initialization then switching between live/test is a simple one line code change for the partner.
Options

Result of Integration 1
Duration: 5 minutes and 39 seconds
Result: Perfect build! All 179 tests passed. :+1:
Test Coverage: 58%

I also vote for not removing the getTestInstance because it will break for current partners and it has been in our docs for a long time.

Result of Integration 2
Duration: 5 minutes and 29 seconds
Result: Build failed 7 tests out of 179

I've added back getTestInstance.  It's now deprecated.
Once the Branch object is instantiated with a key, test or live, the key can't be changed.  So getTestInstance returns the same instance that getInstance will.  It's what ever instance was created first.  So the order of use is important and a little confusing.  That's why I want to deprecate it.

As far as different configurations go, I usually use an xcconfig file.  The values is the xcconfig file can be used in the Info.plist, so I can switch between live and test pretty easily.

Result of Integration 3
Duration: 4 minutes and 30 seconds
Result: Perfect build! All 179 tests passed. :+1:
Test Coverage: 58%


Document analysis engine build/debug/contributions

Included in https://github.com/Microsoft/vscode-python/pull/1231. See https://github.com/Microsoft/vscode-python/blob/master/CONTRIBUTING - PYTHON_ANALYSIS.md


Update documentation for NB GLMMs

Parallel to #56, need to add accompanying documentation / code examples (both in package & in README)

this has long since been addressed, unsure why i didn't close it earlier


Reduced severity of enterprise license bootstrap log from warn to info

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@b8c2950). Click here to learn what that means.
The diff coverage is 100%.

@@            Coverage Diff            @@
##             master    #6058   +/-   ##
=========================================
  Coverage          ?   83.07%
  Complexity        ?    12434
=========================================
  Files             ?     1436
  Lines             ?    47062
  Branches          ?     5795
=========================================
  Hits              ?    39095
  Misses            ?     5265
  Partials          ?     2702

Impacted Files
Coverage Δ
Complexity Δ

...src/main/java/io/crate/module/CrateCoreModule.java
90% <100%> (ø)
3 <0> (?)

Continue to review full report at Codecov.

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


Clean up GuiHandle #459

Part of #459.
Proposed merge commit messages:
GuiHandle contains unnecessary parameters in its constructor. It also
contains methods that are more specialized to individual components.

Let's remove these constructor parameters, and move the methods to their
appropriate component handle classes.

v1
@yamgent submitted v1 for review.

[v1 1/6] GuiHandle: Remove guiRobot from constructor parameters
[v1 2/6] GuiHandle: Remove primaryStage constructor parameter
[v1 3/6] GuiHandle: Move pressEnter() to CommandBoxHandle
[v1 4/6] GuiHandle: Inline setTextFieldText()
[v1 5/6] GuiHandle: Inline GuiHandle#getTextFieldText(String)
[v1 6/6] GuiHandle: Inline getTextFromLabel(String, Node)

(:books: Archive)

Checkout this PR version locally
git fetch https://github.com/se-edu/addressbook-level4.git refs/pr/515/1/head:BRANCHNAME

where BRANCHNAME is the name of the local branch you wish to fetch this PR to.

[v1 5/6] GuiHandle: Inline ~~GuiHandle#~~getTextFieldText(String)

Maybe change the make the commit message

:O
*restarts @limmlingg*

v2
@yamgent submitted v2 for review.

[v2 1/6] GuiHandle: Remove guiRobot from constructor parameters
[v2 2/6] GuiHandle: Remove primaryStage constructor parameter
[v2 3/6] GuiHandle: Move pressEnter() to CommandBoxHandle
[v2 4/6] Inline GuiHandle#setTextFieldText()
[v2 5/6] Inline GuiHandle#getTextFieldText(String)
[v2 6/6] Inline GuiHandle#getTextFromLabel(String, Node)

(:books: Archive) (:chart_with_upwards_trend: Interdiff between v1 and v2)

Checkout this PR version locally
git fetch https://github.com/se-edu/addressbook-level4.git refs/pr/515/2/head:BRANCHNAME

where BRANCHNAME is the name of the local branch you wish to fetch this PR to.

The rest of the changes makes sense, thanks.

I don't quite understand? GuiRobot is not a singleton?

Apologies for the mistake, it was cherry picked from #512 and there was some confusion about which PR got merged.

v3
@yamgent submitted v3 for review.

[v3 1/6] GuiHandle: Remove guiRobot from constructor parameters
[v3 2/6] GuiHandle: Remove primaryStage constructor parameter
[v3 3/6] GuiHandle: Move pressEnter() to CommandBoxHandle
[v3 4/6] Inline GuiHandle#setTextFieldText()
[v3 5/6] Inline GuiHandle#getTextFieldText(String)
[v3 6/6] Inline GuiHandle#getTextFromLabel(String, Node)

(:books: Archive) (:chart_with_upwards_trend: Interdiff between v2 and v3)

Checkout this PR version locally
git fetch https://github.com/se-edu/addressbook-level4.git refs/pr/515/3/head:BRANCHNAME

where BRANCHNAME is the name of the local branch you wish to fetch this PR to.

v4
@yamgent submitted v4 for review.

[v4 1/6] GuiHandle: Remove guiRobot from constructor parameters
[v4 2/6] GuiHandle: Remove primaryStage constructor parameter
[v4 3/6] GuiHandle: Move pressEnter() to CommandBoxHandle
[v4 4/6] Inline GuiHandle#setTextFieldText()
[v4 5/6] Inline GuiHandle#getTextFieldText(String)
[v4 6/6] Inline GuiHandle#getTextFromLabel(String, Node)

(:books: Archive) (:chart_with_upwards_trend: Interdiff between v3 and v4)

Checkout this PR version locally
git fetch https://github.com/se-edu/addressbook-level4.git refs/pr/515/4/head:BRANCHNAME

where BRANCHNAME is the name of the local branch you wish to fetch this PR to.

v4
Rebased to master.
@pyokagan reminder for review.

v5
@yamgent submitted v5 for review.

[v5 1/6] GuiHandle: Remove guiRobot from constructor parameters
[v5 2/6] GuiHandle: Remove primaryStage constructor parameter
[v5 3/6] GuiHandle: Move pressEnter() to CommandBoxHandle
[v5 4/6] Inline GuiHandle#setTextFieldText()
[v5 5/6] Inline GuiHandle#getTextFieldText(String)
[v5 6/6] Inline GuiHandle#getTextFromLabel(String, Node)

(:books: Archive) (:chart_with_upwards_trend: Interdiff between v4 and v5)

Checkout this PR version locally
git fetch https://github.com/se-edu/addressbook-level4.git refs/pr/515/5/head:BRANCHNAME

where BRANCHNAME is the name of the local branch you wish to fetch this PR to.

v5
Rebased.


First cut of a new rama CLI 'probe' command

This commit introduces the first version of a new probe command to the rama CLI
It has two methods of input:

STDIN: pass a domain directly to the rama probe command like so:

rama probe google.com

FILE: pass -l to rama probe with a txt file containing all domains to probe:

rama probe -l domains.txt
where domains.txt is a file containing entries like:
google.com
github.com

Hi @GlenDC, wanted to follow up on this PR. I noticed that when the command would run against a domain it would always give a 301 status code so I wanted to ask if you have any suggestions to resolve that 😃


Nnable to add 2nd sensor

Before you submit a new bug report, please check that

[X] I have read frequently asked questions and limitations
[X] I'm running the newest version of Garbage Collection, or the next pre-release.
[X] I have enabled debug logging for my installation.

Describe the bug
I have added first sensor and I right after that wanted to add few more. Unfortunatelly one I click on CONFIG button it shows be config of sensor I have created already. Removed/ installed/ Restarted HA several times already.
Please advise.
Configuration
{
  "home_assistant": {
    "installation_type": "Home Assistant OS",
    "version": "2022.3.7",
    "dev": false,
    "hassio": true,
    "virtualenv": false,
    "python_version": "3.9.9",
    "docker": true,
    "arch": "x86_64",
    "timezone": "Europe/Warsaw",
    "os_name": "Linux",
    "os_version": "5.10.103",
    "supervisor": "2022.03.5",
    "host_os": "Home Assistant OS 7.5",
    "docker_version": "20.10.9",
    "chassis": "vm",
    "run_as_root": true
  },
  "custom_components": {
    "daikin_residential": {
      "version": "2.2.0",
      "requirements": [
        "oic==1.2.1"
      ]
    },
    "dahua": {
      "version": "0.5.0",
      "requirements": []
    },
    "tauron_amiplus": {
      "version": "v2.0.4",
      "requirements": [
        "requests"
      ]
    },
    "webrtc": {
      "version": "v2.2.0",
      "requirements": []
    },
    "solis": {
      "version": "2.1.0",
      "requirements": []
    },
    "hacs": {
      "version": "1.24.0",
      "requirements": [
        "aiogithubapi>=21.11.0"
      ]
    },
    "garbage_collection": {
      "version": "4.6",
      "requirements": [
        "python-dateutil>=2.8.2"
      ]
    }
  },
  "integration_manifest": {
    "domain": "garbage_collection",
    "name": "Garbage Collection",
    "version": "4.6",
    "documentation": "https://github.com/bruxy70/Garbage-Collection/",
    "issue_tracker": "https://github.com/bruxy70/Garbage-Collection/issues",
    "iot_class": "calculated",
    "dependencies": [],
    "config_flow": true,
    "codeowners": [
      "@bruxy70"
    ],
    "requirements": [
      "python-dateutil>=2.8.2"
    ],
    "is_built_in": false
  },
  "data": {
    "entity_id": "sensor.test3",
    "state": 1,
    "attributes": {
      "next_date": "2022-03-29T00:00:00+02:00",
      "days": 1,
      "last_collection": null,
      "last_updated": "2022-03-28T14:41:58.140862+02:00",
      "device_class": "garbage_collection__schedule"
    },
    "config_entry": {
      "entry_id": "dc83e9e0a0b74147fd4990fadfa7ae30",
      "version": 4.5,
      "domain": "garbage_collection",
      "title": "test3",
      "data": {
        "unique_id": "b2c62cd4-83a4-4c6d-97f4-f7a04c1fa628",
        "frequency": "weekly",
        "icon_normal": "mdi:trash-can",
        "icon_today": "mdi:delete-restore",
        "icon_tomorrow": "mdi:delete-circle",
        "first_month": "jan",
        "last_month": "dec",
        "collection_days": [
          "tue"
        ]
      },
      "options": {},
      "pref_disable_new_entities": false,
      "pref_disable_polling": false,
      "source": "user",
      "unique_id": null,
      "disabled_by": null
    }
  }
}

Debug logs
?

Just click add integration and add the second entity.

I'm an i***t. Sorry for reporting this as bug.


Add telemetry for which language server is being used on Python notebooks

Right now when we're trying to find usage or errors in notebooks telemetry for those using Pylance, we have to join queries on sessions. If we had telemetry to indicate which language server is being used on Python notebooks, this would make it faster/easier for us.
cc @debonte

@luabud the language server right now is based on the 'python.languageServer' setting. Aren't you tracking this in the python extension? We don't have a separate setting.

We are tracking it, but we have to use "joins" on session ID to get this info and it's not always accurate (e.g. if Pylance failed to load and Jedi starts, it's not trivial to query for that). The queries often time out or run out of memory too.

Wouldn't the python extension then add a specific telemetry event for this? The language server for notebooks is never different than the language server for python files. Seems weird to add a telemetry event for notebooks.

That's a good point, if we were to log something it would probably make more sense to do it from the Python extension.


Missing encode.py

In the README and your presentation, there is a encode.py file that is missing in this repository.
Is this intended, did I misunderstand something here, or did it maybe get lost?
Thanks in advance!

Hi, thanks for reporting this and sorry for the late reply. This was a leftover of a major refactoring I did on the codebase during which I renamed encode.py to codasm.py. I updated the README accordingly. Hope this helps!


Add “Edit this Page” links to editable topics for #8

… branch.

Initial implementation for #8.

@georgebina Thanks, I'll test this locally this evening.
Are you able to push another commit to this feature branch to move the JavaScript to a separate file as @jelovirt suggested?

In my initial testing last night, the results look promising — seems like a good start.
A few comments & questions:

Existing Git commit metadata in footers should be preserved.
In my test environment, the links were appended to the fallback footer with the license info instead.
We may want to simplify the Edit link text further:
Page content generated from DITA source: dita-ot/docs/commit/4fd9a82. [Edit]
(but we can always do that later)
Would it be possible to always create/use a fork, even if author has commit rights?
I'd rather not push any changes to the main repo from web app without code review via the normal GitHub pull request mechanism.
To facilitate code review in pull requests, edits created via the web app should always create a new branch (maybe webedits, webapp-edit?)
Currently, the changes go to develop by default, or allow the user to select an existing branch, which would not isolate the changes from other commits on these branches.
Will each commit create a new PR, or how are multiple edits/saves/commits handled?
Works in Firefox 44, not in Safari 9.1 or Safari Technology Preview 9.1.1
The web inspector shows this JS error:
TypeError: file.contains is not a function. (In 'file.contains('/build-')', 'file.contains' is undefined)
What browsers/OS combinations have you tested this with?

I cannot see the commit locally either... but that should not have anything to do with my changes, the footer template is back to its initial form now.
Done.
It is not possible now. I guess this is a problem only for you and a couple of people :), the rest will get the fork automatically when they try to save.
Currently the web author works almost similarly with the "edit this file" link from GitHub, with a late creation of the fork, we do that only if the user tries to commit a change, otherwise we show the content from the original repository.
PR are not created automatically, the web author will commit the content but the user needs to initiate the PR
Fixed in c1d6160

Thanks for refactoring, getting closer. More comments:

On 1) Missing commit metadata may be related to changes in the org.dita-ot.html plug-in.
(Will investigate this separately.)
On 3) True, most users will be prompted to create/use fork, so it should be OK.
On 4) Is there any mechanism to create a new branch, or can users only select existing branches?
On 6) That error is gone in Safari, but links to source topics are still unreliable.
On first load, the Edit link on every page seems to open either index.dita or the previously viewed topic. If the page is reloaded in the browser, the link seems to be “fixed” and the correct topic opens in the web app. (Links seem to work fine in Firefox without page reloads.)

On Save you one can also type a new branch name.
The same issue appears also with Firefox, the link is not updated unless the page is refreshed, I suppose it may be how 'jekyll serve' caches the footer? If I add just {{page.url}} to the footer, that does not change also unless you refresh the page.

On the automatic branching: we will update the current workflow to propose a branch like develop-webAuthor-timestamp and offer also an option to send a pull request automatically - hopefully we will have this available in a few days.

On 2: the footer is not updated when a page is loaded by clicking on a link in the ToC - the code in Common.js will intercept all clicks on links and it will perform loading of a new page by updating the main content and the ToC. I can change the code to update the edit link when the user will click on the link but the link will appear also for generated topics, so we need to decide what we want to show to the user in that case - the map, a massage that says that the current content is generated, something else.

@georgebina the footer update should be handled in a separate issue, #15.

In connection to https://github.com/dita-ot/org.dita-ot.html/pull/2, if https://github.com/dita-ot/dita-ot.github.io/issues/15 is fixed then we can create the edit link in the footer template only if the page is not generated.

This should be ready now - the footer template generates the link and the href is updated to edit the correct topic by the editLink.js JS script. If the HTML is marked as generated then no link is added. Until https://github.com/dita-ot/dita-ot.github.io/issues/15 is fixed the link to generated topics will be visible if the topic is loaded from a navigation link, but that will open the root map, not a missing topic.
We need also the additional PRs https://github.com/dita-ot/org.dita-ot.html/pull/2 and https://github.com/dita-ot/docs/pull/96 to have the generated flag information available.


Document good pagination practices

Note: this is a section under "efficient querying" under performance. But it could also be its own page under the top-level "Query data", since this isn't only about performance, but also about general correctness.
Closes #3582

@ajcvickers I moved this into its own page under the query section and did various improvements - can you take another look?

:shipit:


CCTBX Conversion: Read in MTZ and CCP4/MRC Maps

Change the reading of maps from qFit infrastructure to CCTBX.
The two main functions to be converted are:
fromfile()
canonical_unit_cell()
This will include storing information about the spacegroup, unit cell, origin, voxel spacing, resolution, and grid parameters. The majority of this functionality is in the script volume.py and functions called within.
The output of this conversion should still be an xmap object as this is used in the rest of qFit.
These changes will need to be applied to the following qFit input functions:
-qFit_protein.py
-qFit_residue.py
-qFit_ligand.py

I think I need more examples of this - my code works fine with synthetic test cases but these are all maps generated by CCTBX.


Replacing unconfirmed with pending in github issue templates

Change Summary
Replacing unconfirmed with pending in github issue templates
Checklist

[x] The pull request title is a good summary of the changes - it will be used in the changelog
[ ] Unit tests for the changes exist
[x] Tests pass on CI
[ ] Documentation reflects the changes where applicable
[x] My PR is ready to review, please add a comment including the phrase "please review" to assign reviewers

Please review


Transfer ansible-lint to zuul job

Delete Github Action check ansible-lint
Activate zuul job ansible-lint
Partly osism/issues#395

Signed-off-by: Ramona Beermann<EMAIL_ADDRESS>

In this repository we need another way of checks, because zuul can´t check hisself


Compatible variables generate empty plot with negative x-y coordination

In vizualization section, some compatible variables generate empty plot with negative x-y coordination. For example, in india cohort study, This happens in bar plot when main=" Treated with antimalarials" or " Clinic or hospital" and overlay=" Untreated bednet count"," ITN bednet count, treated >6 months ago"," ITN bednet count, treated <6 months ago" or " Long-lasting ITN bednet count".

the request succeeds on the backend. the result looks like {"barplot":{"data":[{"overlayVariableDetails":{"variableId":"EUPATH_0000401","entityId":"PCO_0000024","value":"1.0"},"label":["No","Yes"],"value":[50,3]},{"overlayVariableDetails":{"variableId":"EUPATH_0000401","entityId":"PCO_0000024","value":"2.0"},"label":["No"],"value":[26]},{"overlayVariableDetails":{"variableId":"EUPATH_0000401","entityId":"PCO_0000024","value":"3.0"},"label":["No"],"value":[6]}],"config":{"completeCasesAllVars":85,"completeCasesAxesVars":1249,"xVariableDetails":{"variableId":"EUPATH_0000410","entityId":"EUPATH_0000738"}}},"sampleSizeTable":[{"overlayVariableDetails":{"variableId":"EUPATH_0000401","entityId":"PCO_0000024","value":"1.0"},"xVariableDetails":{"variableId":"EUPATH_0000410","entityId":"EUPATH_0000738","value":["No","Yes"]},"size":[50,3]},{"overlayVariableDetails":{"variableId":"EUPATH_0000401","entityId":"PCO_0000024","value":"2.0"},"xVariableDetails":{"variableId":"EUPATH_0000410","entityId":"EUPATH_0000738","value":"No"},"size":[26]},{"overlayVariableDetails":{"variableId":"EUPATH_0000401","entityId":"PCO_0000024","value":"3.0"},"xVariableDetails":{"variableId":"EUPATH_0000410","entityId":"EUPATH_0000738","value":"No"},"size":[6]}],"completeCasesTable":[{"variableDetails":{"variableId":"EUPATH_0000410","entityId":"EUPATH_0000738"},"completeCases":1249},{"variableDetails":{"variableId":"EUPATH_0000401","entityId":"PCO_0000024"},"completeCases":85}]}
i wonder if this is a problem mapping values to vocabulary again? that was the culprit last time i saw the frontend not plotting data it had received. in any case, i dont think this is for the data service.


Certificate rotation frequency and storage

"Microsoft automatically rotates these certificates in compliance with the internal security policy and the root key is protected by a Microsoft internal secret store."
Any additional information on the frequency of rotation (monthly, quarterly, yearly, etc.)?
Also, any details about what the "internal secret store" is? Is that blob storage, Key Vault, something else?

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

ID: a44a0e0d-f0a8-fe91-8ae5-a901e103f89e
Version Independent ID: 834b8bc9-d661-0204-9ece-1f370067ec96
Content: Transparent data encryption for Azure SQL Database and Data Warehouse
Content Source: articles/sql-database/transparent-data-encryption-azure-sql.md
Service: sql-database
Sub-service: security
GitHub Login: @aliceku
Microsoft Alias: aliceku

The frequency of rotation can change based on our compliance requirements, so we do not want to make statements here that need frequent updates.  The Trust Center link provides the audit reports that check on key rotation policies and Microsoft's compliance with them.  The internal secret store is not customer facing, and we do not disclose these details for security reasons.

@aliceku Thanks for the quick response. 👍

@dmarlow Closing this issue for now. If there are further questions regarding this matter or questions or feedback on the documentation, please do let us know and we will gladly continue the discussion.


SLIM-247 Feat convert reads to uniform unit

SonarQube analysis reported 2 issues:

 2 major

Watch the comments in this conversation to review them.
Note: the following issues could not be reported as comments because they are located on lines that are not displayed in this pull request:

 Constructor has 8 parameters, which is greater than 7 authorized.

All is well.

SonarQube analysis reported 2 issues:

 2 major

Watch the comments in this conversation to review them.
Note: the following issues could not be reported as comments because they are located on lines that are not displayed in this pull request:

 Constructor has 8 parameters, which is greater than 7 authorized.

All is well.

SonarQube analysis reported 2 issues:

 2 major

Watch the comments in this conversation to review them.
Note: the following issues could not be reported as comments because they are located on lines that are not displayed in this pull request:

 Constructor has 8 parameters, which is greater than 7 authorized.

All is well.


Any alternatives to this package out there?

This works, but its a bit hacky... there are some lines of code that are not reachable that alter test coverage percentages

I wrote a similar plugin with latest istanbul dependencies, some coverage excludes for unreachable code and coverage reporting /w thresholds.
Currently it's part of our test utils: https://github.com/SAP/ui5-project/blob/3174d9f21f471252d2a39b8cb085eeeb5debe0a6/test/utils/AjvCoverage.js
I'd be willing to contribute my features to this project. @epoberezkin what do you think?
Otherwise I would create a separate project.


Add vendor directory to the list of Golang exclusions

Signed-off-by: Niladri Halder<EMAIL_ADDRESS>This is the add the vendor directory to the list of Golang vendor directory exclusions.
Presently, this library does not recognise this directory as a vendoring directory.
Refer to https://golang.org/ref/mod#go-mod-vendor for official documentation.

Description
Added an entry under the comment '# Go dependencies'.
Checklist:

[ ] I am associating a language with a new file extension.

[ ] The new extension is used in hundreds of repositories on GitHub.com

Search results for each extension:

https://github.com/search?utf8=✓&type=Code&ref=searchresults&q=extension%3AFOOBAR+KEYWORDS+NOT+nothack

[ ] I have included a real-world usage sample for all extensions added in this PR:

Sample source(s):

[URL to each sample source, if applicable]

Sample license(s):

[ ] I have included a change to the heuristics to distinguish my language from others using the same extension.

[ ] I am adding a new language.

[ ] The extension of the new language is used in hundreds of repositories on GitHub.com.

Search results for each extension:

https://github.com/search?utf8=✓&type=Code&ref=searchresults&q=extension%3AFOOBAR+KEYWORDS+NOT+nothack

[ ] I have included a real-world usage sample for all extensions added in this PR:

Sample source(s):

[URL to each sample source, if applicable]

Sample license(s):

[ ] I have included a syntax highlighting grammar: https://github-lightshow.herokuapp.com/
[ ] I have updated the heuristics to distinguish my language from others using the same extension.

[ ] I am fixing a misclassified language

[ ] I have included a new sample for the misclassified language:

Sample source(s):

[URL to each sample source, if applicable]

Sample license(s):

[ ] I have included a change to the heuristics to distinguish my language from others using the same extension.

[ ] I am changing the source of a syntax highlighting grammar

Old: https://github-lightshow.herokuapp.com/
New: https://github-lightshow.herokuapp.com/

[ ] I am updating a grammar submodule

[ ] I am adding new or changing current functionality

[ ] I have added or updated the tests for the new or changed functionality.

[ ] I am changing the color associated with a language

[ ] I have obtained agreement from the wider language community on this color change.

[URL to public discussion]
[Optional: URL to official branding guidelines for the language]

Right! Missed it. Closing this pull request.


update build-from-scratch.md for clarity, velero install cmd

Signed-off-by: Steve Kriss<EMAIL_ADDRESS>Fixes #1424

Could we rename this to Build from source? It's the "norm" "out there".

addressed code review comments. Since we're renaming the file, I'll need to do TOC updates to change the link (in a separate PR to gh-pages).


chore: Readonly root FS for supervisord configuration

This is part of supporting running Appsmith with readonly root FS. This moves the supervisord configuration, and runtime files, like the unix socket file, and the PID file, to $TMP.

/ok-to-test


Updated gradle usage to take advantage of new features in 2.2

I updated the gradle version, added enableBuildCache in the gradle.properties (as per instructions here http://tools.android.com/tech-docs/build-cache) (it doesn't seem to work yet, but hopefully it will do something eventually), and added a dependency on the ConstraintLayout library ("For improved layout performance, you should convert older layouts to ConstraintLayout" - see https://developer.android.com/studio/write/layout-editor.html#convert-constraint and https://developer.android.com/training/constraint-layout/index.html)

This change is

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

Comments from Reviewable

Review status: all files reviewed at latest revision, all discussions resolved.

Comments from Reviewable


Theming isn't properly supported in extensions.

@vishwacsena  FYI
We had a bug where extensions weren't working all together but I just fixed it and now realized that we need a theming story for extensions.

Basic theming infrastructure in place with themes applied superficially. A full theming sweep is recommended.


Add TextBox.ScrollToLine

What does the pull request do?
Adds the method ScrollToLine to the TextBox control.
What is the current behavior?
No obvious API for moving the caret to a specific line. This is not "compatible" with WPF.
What is the updated/expected behavior with this PR?

Calling ScrollToLine with a line index will move the caret index to the first position on the specified line index.
There are two new test cases in TextBoxTests.cs

How was the solution implemented (if it's not obvious)?
ScrollToLine will get the TextLine object from the collection and use the TextPresenter.MoveCaretToTextPosition API to move the caret.
Perhaps this is a bit naive implementation. The WPF implementation gets a bounding rectangle and brings that into view.
Checklist

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

Unsure if this method requires updated docs since not everything is included in the docs. Happy to do so if suggested.
Breaking changes
None that I know of.
Fixed issues
Addresses #3036 at least partially. It would be nice with convenience API:s like ScrollToHome and ScrollToEnd. Those would delegate to existing internal API or to the same API on the ScrollViewer.
I'll leave it to the maintainers to decide if #3036 can be closed or not.

We need a test for Arabic text as well so right to left is also covered.

Thanks @Gillibald! Please consider adding the hacktoberfest-accepted label when/if PR gets merged.


big big speedup w/ numba

one order of magnitude speedup

Thanks for the contribution! I love the idea of using numba for this repo—the library reminds me a bit of how Julia works.
I have a couple of technical details I'm not sure about—could you take a look and either change the code or address my concerns?
In general, I'd be happy to merge in your changes :+1: modulo the details.


Fix python generation for enum tag ranges

Could we commit a generate output file so that we can follow along in the evolution via those?

I can do that, but that will fall outside the scope of this CL

Okay, sure!


Potential false positives on BooleanPropertyNaming with constants

Context
BooleanPropertyNaming considers naming of properties, while technically these const vals are properties, they also have a different naming convention and I have a feeling they rarely start with IS_ prefix.
Expected Behavior of the rule
I propose to exclude const from this check fully.
object WidgetPreferences { // Detekt: or a companion object

	/**
	 * User selection of whether to show the part of day or not.
	 *
	 * *type*: [Boolean]
	 * *description*: true=show
	 * *default*: [DEFAULT_SHOW_PART_OF_DAY]
	 */
	const val PREF_SHOW_PART_OF_DAY = "showPartOfDay"
	const val DEFAULT_SHOW_PART_OF_DAY = true // Detekt: BooleanPropertyNaming shouldn't be reported
}

Yup that's legit from my point of view 👍

May I try this?

Sure @gouri-panda, I've assigned this to you :+1:

As there is no recent activity on this, I am taking this up.

Thank you @amitdash291 !


types(runtime-core): fix types of createElementBlock and createBlock

fix: #4391
supersedes: #4406
This aligns the arguments of createElementBlock and createBlock closer to the ones of the underlying createBaseVNode and createVNode.
packages/runtime-core/src/helpers/renderSlot.ts had to be changed since the type of props.key had to be string | number | symbol | undefined instead of unknown.
I already tested it with my fail case (#4391) and it fixes the issue.

These functions are only used in compiler-generated code so their type checking should be lax.


Revert "bower angular dependency to ~1.4.0"

This reverts commit eee6efeeae2763a3f5c4b44cce842123cfce8a3a.
As requested in #49

Oops should have committed against the main repo.


one of the commands exited with non-zero exit code; note that snakemake uses bash strict mode!

Hi,
I'm using snakemake but in all my rules no one is excited correctly !!
I had this error : one of the commands exited with non-zero exit code; note that snakemake uses bash strict mode!
Here an example of a rule in my workflow:
rule sort_bed:
input: expand("inp/bed/{bed}.bed", bed = BED_SMP)
output:  expand("out/bedtools/sort/{bed}_sorted.bed", bed = BED_SMP)
shell:"""
bedtools sort -k1,1 -k2,2n -i {input} > {output}
"""
Note that when I execute it in interactif way it works
Any help please ?
Thank you.

Hi- I would suggest you post this question to stackoverflow and close this issue since you may get more help there and most likely this is not a bug in snakemake. Before posting though, add more detail about your settings. E.g. show the output of running snakemake with and without --dry-run option. As it is, it is difficult if not impossible to tell what went wrong.


[DO NOT MERGE] [#155856825] Remove CC ELB

What
Now that the API is behind GoRouter we can remove the old ELB and associated gubbins.
How to review

deploy on top of https://github.com/alphagov/paas-cf/pull/1287
make sure all tests pass

Check there isn't anything I may have forgotten to remove.
Before merging ⚠

merge https://github.com/alphagov/paas-cf/pull/1287
rebase on master

Who can review
Anyone but me.

Marked as DO NOT MERGE because we had to revert the previous PR in this story's series.

Rebased with master


Update generated thumbnails

The default thumbnail size was enlarged some time ago.  DSpace's media filter will not regenerate thumbnails if they already exist unless forced to, so the nightly job does not handles regeneration for the small legacy thumbnails.  The legacy thumbnails should be either deleted or undergo a forced media filer.  The small thumbnails can be tracked with this query.

Query for generated thumbnails with corresponding item and collection handles:
select b.uuid,b.size_bytes,i.uuid as itemid,ci.collection_id as collectionid,idfi.handle as item_handle,idf.handle as coll_handle,m.text_value
from bitstream b
left join metadatavalue m on m.dspace_object_id=b.uuid
left join bundle2bitstream bb on bb.bitstream_id=b.uuid
left join item2bundle ib on ib.bundle_id=bb.bundle_id
left join item i on i.uuid=ib.item_id
left join dspaceobject di on di.uuid=i.uuid
left join handle idfi on (idfi.resource_id=di.uuid and idfi.resource_type_id=2)
left join collection2item ci on ci.item_id=i.uuid
left join dspaceobject d on d.uuid=ci.collection_id
left join handle idf on (idf.resource_id=d.uuid and idf.resource_type_id=3)
where m.text_value='Generated Thumbnail' order by size_bytes asc limit 100;

Note that as DSpace is not lately able to run lengthy batch operations, individual media filter commands will need to be run serially.  A handy script to do this for a list of handles can be found here: https://github.com/TAMULib/saf-scripts/blob/development/dspace/repeatcommand.sh

From dspace as root
PGPASSWORD=<password> psql -h <host> -p <port> -d <database> -U <username> -c 'SELECT handle FROM handle WHERE resource_type_id = 2' -t > item_handles.txt

while read handle; do /data/DSpace/bin/dspace filter-media -f -i $handle; done < item_handles.txt


Filtering using Lambda expressions not adding grouping parenthesis

I have create a query using a lambda expression like this one
query.Filter((x => x.WhenFrom >= from && x.WhenFrom <= to) || r(x => x.WhenTo >= from && x.WhenTo <= to))
when I use the GetCommandTextAsync method, I get the following (I have decoded the url)
$filter=WhenFrom ge datetime'2016-06-10T00:00:00' and WhenFrom le datetime'2016-06-17T00:00:00' or WhenTo ge datetime'2016-06-10T00:00:00' and WhenTo le datetime'2016-06-17T00:00:00'
Is this the right way of grouping multiple or statements or is there another?
Thanks

But why should it add parenthesis? AND has higher precedence than OR, so parenthesis should not be necessary.

If that's the case then OK, but will it start to add parenthesis as the query requires it, i.e. becomes more complex? I'm refering to http://www.odata.org/documentation/odata-version-3-0/url-conventions/ section <IP_ADDRESS>. Operator Precedence regarding grouping.

Yes it should. If if you find an example where it doesn't please let me know.


kairos-test-utils/cctl: add deploy contract function

Hold off on merging this. There might be a bug.

Hold off on merging this. There might be a bug.

Let me know if I can help.


Remove node crash

This should make the HierarchyId map more reliable

That's much cleaner.  Thanks!

@amcasey going to merge this now... I'll make the stylistic changes tomorrow, but this way we have a build to test.


TCP/HTTP health check for agent

Is your feature request related to a problem? Please describe.
In cloud environment I would like to make netbird highly available and put it behind load balancer. To determine netbird is able to process connections I need TCP or HTTP health check.
Netbird should respond in health check if it is "Connected" to API

This is important to us as well


The focus misses borderlines after clicking the '<<' icon in front of the directory path

Storage Explorer Version: 1.2.0
Platform/OS Version: All
Architecture: ia32/x64
Build Number: 20180611.2
Commit: e00d7640
Regression From: Not a regression
Steps to Reproduce:

Launch Storage Explorer and expand the Storage Accounts.
Open one file share and go to a deep directory.
Make sure '<<' icon displays in front of the path.
Click this icon and check the focus.

Expected Experience:
The focus displays well.
Actual Experience:
The focus misses borderlines.

Verified on build 20181129.6. This issue doesn't reproduce. Could you help to resolve it?

This issue doesn't reproduce. So we close it.


[Task] Add EDAM annotations

Following up on #2209, add EDAM annotations curated by @gaiaandreoletti to the OC db.

Added to Sprint 10.11

Actually I'm sending that one back to Backlog. We need to better validate the existing data (#2064) before we can add more properties to the data mode.

Update 2024-03-04
The API property Challenge.operation is now described with EDAM Operation. We have one more task about representing challenge input data type with EDAM before be can close this story.

This task is done. There is a pending task tracked by @gaiaandreoletti that consist in making sure that only "Data" concepts are used for the challenge input data types. The existing implementation supports this decision.
@gaiaandreoletti We are welcome to revisit this decision any time, e.g. if you want to start using "Format" or "Topic" concepts to annotate the input data types of challenges.

only data concepts of edam were added to the challenge format


fixed URL joining problem

fixed a general ResolveReference problem and a reCAPTCHA error.

The following line (in utils/url.go):
if !baseURL.IsAbs() && !strings.HasPrefix(base, "/") {
is a typo error?
if !baseURL.IsAbs() && strings.HasPrefix(base, "/") {

@mzch yeah I think that might be a typo but I'm not sure what that line is supposed to be doing at all. I think it could do with some tests to ensure that this is doing what it says it should be doing, that is acting like path.Join but for URLs.

@zacheryph This line will delete a top char of relative URL. e.g. "wiki" + "?user=bob" -> "iki?user=bob"

How do I get people to stop tagging me in tickets? This is the third ticket this month I have been tagged in that I am completely unrelated to. :slightly_frowning_face:

@mzch yeah I realise it would do that clearly incorect thing but what I'm trying to work out is what it was ever supposed to do...
@zacheryph sorry, I think they've all meant to be to me...

Sorry @zacheryph your username is so close to @zeripath :)

Imho this could break markdown URLs... If you specify url in markdown like: [test](/test) it  should result in https://domain/test not https://domain/repo/subpath/test

Imho previous PR with was correct

@lafriks OK. I'll remake PR.

@lafriks OK. I'll remake PR.

@mzch You probably only need to reopen it.

Ok this dropping to / isn't going to work when Gitea is mounted on a suburl.
It's probably the cause of multiple bugs with suburls and it disagrees with the comment that the function says it does.
https://github.com/go-gitea/gitea/blob/69d81b656978a03ff277a611f5c3d9ef1814d001/modules/util/url.go#L26
If we're gonna keep it this way we need to investigate every use of urljoin and think about how it's going to work with Gitea mounted on suburls. Any use of urljoin without removing an initial slash outside of markdown is suspect.

If we're gonna keep it this way we need to investigate every use of urljoin and think about how it's going to work with Gitea mounted on suburls. Any use of urljoin without removing an initial slash outside of markdown is suspect.

Is it useful to keep two versions of the function, with different behaviors?


BR problems

and 9 --->  9.0

Nice catch! Fixed: https://github.com/ImVexed/ThunderView/commit/5a21063373748a2620737c16cf72eaabd7880306


Unable to load wiki database any more

A few days ago my WFInfo started giving the status "ERROR (ParseScreen)", after which it wouldn't display any overlays.
I was using 8.2.3 at the time, but I've downloaded 8.2.6 and had the same issue, with and without blowing away my AppData directory.
Looking at the log it seems very likely that this is due to edits on the wiki's data dump, when it gets parsed as a Lua script at {Data.vb:603}}.  I've had a quick look through the wiki page's history and nothing jumps out as obviously wrong (timing-wise the two edits on the 31st would seem to be the culprit but they look benign).
I'm also surprised that no-one else seems to be reporting this as it ought to be affecting everyone (after the one-day refresh window).  Perhaps I have a bad NLua DLL being used somehow?
If you can't reproduce this, let me know if there's anything I can do to provide more debugging info.

It's an error in the Warframe wiki we parse with NLua, however seeing as we've moved over onto a complete re-write of the software (that doesn't use Nlua anymore) we've little to no interest in supporting the older versions.
You're able to download the latest beta here, v9.0. It's core feature is fully functional and even better than the old one. The additional features are currently in the works and are hopefully done before the end of the month.

https://github.com/WFCD/WFinfo/releases/tag/v8.2.7


chore(test): bump solc to 0.26.0

Motivation
Compilation in smoke test was failing with default evm version (cancun) as we were using an incompatible solc version.
Solution
Use solc 0.26.0 for tests

So paris is incompatible with 0.8.26?

It is. But we changed our default to cancun and 0.8.20 does not support this, that's why we were setting evm to paris in our tests.


Implied columns tests and messaging

TL;DR --> adds implied columns parsing context and tests and slightly improves existing functionality.
This adds some context to the messaging around implied columns parsing. It also adds tests for this new feature. Along for the ride is the removal of the restriction to Series on type of input data to Sample constructor, and ensuring that exception type reflects the access mode for a Sample data item. That is, dot-notation produces AttributeError and getitem syntax produces KeyError when the name of the data item requested isn't defined for the Sample instance in question. There's a small handful of tests here for those changes as well. Other changes are stylistic.

Ticking off checklist tasks 1 & 2 from https://github.com/epigen/looper/pull/94


Fix typo in PR comment find method documentation

looks like the coveralls library for elixir needs a patch - it should not fail the build when it does not manage to push the coverage statistics.


Effect Length should be a box instead of a slider

As it is fairly hard to get the correct number with the mouse, it would be better for the Number Entity's Mode to be box instead of slider or auto.
Relevant docs: https://developers.home-assistant.io/docs/core/entity/number/#:~:text=Type of number.-,mode,-string

This will be available in the next release


Private State Token Issuer Request - MAIL.RU

Issuer Name
MAIL.RU
Origin
https://privacy-ad.mail.ru/trust-token
Contact Email
<EMAIL_ADDRESS>Key Commitment Endpoint URL
https://privacy-cs.mail.ru/private_state_tokens/key_commitment
Purpose
Checking user authorization on sites that use authorization services and detect fraud and bots.
Disclosure and Acknowledgement

I understand the technical restrictions on key rotation frequency of 60 days in the PST API.
I understand that my issuer registration will be valid for a period of six months after the key commitment is accepted, and that I will need to re-register in this repository following that six-month period.
I understand that in the future renewing my registration for this API may have additional requirements, to reduce the risk of abuse by token issuers.

Your key commitment endpoint appears to 404 and doesn't return valid PST keys.

My apologies, the domain was indeed unavailable. This endpoint is now operational https://privacy-cs.mail.ru/private_state_tokens/key_commitment

Is it intentional that the origin for the issuer is privacy-ad..., while the key commitment endpoint is privacy-cs...?
If they should be the same, can you please edit the original issue message with the correct domains.
Additionally it appears that privacy-cs.mail.ru/robots.txt  is disallowing automated fetching of the key commitments with a wildcard Disallow. You'll need to permit fetching of the key commitments so we're able to correctly fetch and parse the keys.


Question - Get Bitmap Signature Path

First, many thanks for this awesome library.
I have a quick question regarding the upload of the Bitmap Signature on a server.
When calling the getSignatureBitmap() and receiving the Bitmap:

Is the Bitmap already stored somewhere?
Can we upload it as is to a server? If so how can we retrieve the Bitmap path?
If not, I guess I have to copy the Bitmap to the user storage?

Thanks a lot in advance for your assistance,
Isabelle,

I've found the answer in the example provided. Thanks.


feature/normalize l2

I'm submitting a ...
[/] feature request

Summary
This ticket aims to implement normalize API that can be used for a wide variety of regression models.

Equation:
X_normalized = X - min(X)/(max(X) - min(X))
The process of L2 normalization
Given an example
>>> X = [[ 1., -1.,  2.],
...      [ 2.,  0.,  0.],
...      [ 0.,  1., -1.]]
>>> X_normalized = preprocessing.normalize(X, norm='l2')

>>> X_normalized
array([[ 0.40..., -0.40...,  0.81...],
       [ 1.  ...,  0.  ...,  0.  ...],
       [ 0.  ...,  0.70..., -0.70...]])

Take all the X-axis then √1^2 + -1^2 + 2^2 = √6
Divide each number of the X-axis with √6 and reconstruct the matrix

References:
http://scikit-learn.org/stable/modules/generated/sklearn.preprocessing.normalize.html

I will implement it, probably tomorrow =)

PR for this open at https://github.com/JasonShin/kalimdorjs/pull/91/

Merged =)


The gpg commiter of the initial commit behaves weird

🥧

Gitea version (or commit ref): 1.11.6+2-g5d4251eb7 (06/09/2020 06:51:55)
Git version: 2.20.1-2+deb10u3
Operating system: Debian GNU/Linux 10 (buster) (10.4)
Database: PostgreSQL 12+215.pgdg100+1
Can you reproduce the bug at https://try.gitea.io: No
Log gist:

Description
I have transfered my workflow to gitea for a short time. I realize that gitea is suitable for a portable code manage platform on Raspberry 4B as a permanent service. Therefore, it's essencial for me to show gpg signature of commits.
I flow the user guide step by step: GPG Commit Signatures.
Firstly, I generated gpg key for my raspi.
Then, I import secret-subkey and set password to null.
Next, I change relavant configurations in /home/git/.gitconfig and /etc/gitea/app.ini.
Finally, I restart the gitea services and copy public key to admin user named gitea.
At the beginning, I create a repo as gitea and everything goes well.

I'm so excite but tragedy cames one after another.
I create a normal user name jueti and add my own gpg public keys. Then, I create a repo but a warning was occured:
WARNING! Although there is a key with this ID in the database it does not verify this commit! This commit is SUSPICIOUS.

So I tried to use git log command to see what was happened, and I found an unusal result.

compare to an initial commit of Github.

So it's obvious that the initial commit of jueti/test should be committed by gitea<EMAIL_ADDRESS>instead of jueti<EMAIL_ADDRESS>So it's obvious that the commiter of the initial commit should be gitea<EMAIL_ADDRESS>Configurations

/home/git/.gitconfig
[user]
        name = gitea
        email =<EMAIL_ADDRESS>        signingkey = C5A84090
[commit]
        gpgsign = true
[gpg]
        program = /usr/bin/gpg
[core]
        quotepath = false
        commitGraph = true
[gc]
        writeCommitGraph = true

/etc/gitea/app.ini
[repository.signing]
SIGNING_KEY    = C5A84090
SIGNING_NAME   = gitea
SIGNING_EMAIL  =<EMAIL_ADDRESS>INITIAL_COMMIT = always
CRUD_ACTIONS   = pubkey, twofa, parentsigned
WIKI           = pubkey, twofa, parentsigned
MERGES         = pubkey, twofa, basesigned, commitssigned

gpg -K --keyid-format short
/home/git/.gnupg/pubring.kbx
----------------------------
sec#  rsa4096/BE6A1CD3 2020-06-14 [C]
      0916836C9F9B437341DAD5A1B8C833ECBE6A1CD3
uid         [ultimate] gitea<EMAIL_ADDRESS>ssb   ed25519/C5A84090 2020-06-14 [S] [expires: 2023-06-14]
ssb   ed25519/50E06E86 2020-06-14 [A] [expires: 2023-06-14]
ssb   cv25519/19D7986D 2020-06-14 [E] [expires: 2023-06-14]

First of all let me disabuse of the notion that the Committer has to be the Signer.
There is no such requirement in Git and git verify-commit will simply check if the signature matches a key in your keyring and report which key did so.
Github has decided that it will use Signatures to verify Committer but that is their decision and Gitea does not require that: This is why Gitea always shows the avatar and information of the Signer separate from the Committer. It is a different Trust model to Github - one which allows you to re-sign commits following revocation of keys or as part of a fully signed tree with one key without loss of committer information - (albeit with change of SHAs)
There is a PR which will allow users to use the Github style of committer-verification but it is not merged as yet.
However, there is clearly a problem if you have a commit that should verify as per git verify-commit but it does not verify on Gitea.
I believe that #11713 and its backport to 1.12 #11747 should fix this - so if it is at all possible I would ask you to test this with the latest 1.12 rc or master and report back if that works - if it does I can send a backport to 1.11 to get that fix in.

I'm agree with you. My expression is unclear so I change it slightly.
I'm plesure to migrate to 1.12 rc and it's not troublesome.
Thanks for your work. 🎉

Gitea version (or commit ref): 1.12.0-rc2 (06/08/2020 08:04:54)
Git version: 2.20.1-2+deb10u3
Operating system: Debian GNU/Linux 10 (buster) (10.4)
Raspi model: Raspberry Pi 4 Model B Rev 1.4
Database: PostgreSQL 12+215.pgdg100+1
Can you reproduce the bug at https://try.gitea.io: No
Log gist:
/etc/gitea/app.ini

[repository.signing]
SIGNING_KEY    = C5A84090
SIGNING_NAME   = gitea
SIGNING_EMAIL  =<EMAIL_ADDRESS>INITIAL_COMMIT = always
CRUD_ACTIONS   = pubkey, twofa, parentsigned
WIKI           = pubkey, twofa, parentsigned
MERGES         = pubkey, twofa, basesigned, commitssigned

~/.gitconfig

[user]
        name = gitea
        email =<EMAIL_ADDRESS>        signingkey = C5A84090
[commit]
        gpgsign = true
[gpg]
        program = /usr/bin/gpg
[core]
        quotepath = false
        commitGraph = true
[gc]
        writeCommitGraph = true

gpg -K --keyid-format short

/home/git/.gnupg/pubring.kbx
----------------------------
sec#  rsa4096/BE6A1CD3 2020-06-14 [C]
      0916836C9F9B437341DAD5A1B8C833ECBE6A1CD3
uid         [ultimate] gitea<EMAIL_ADDRESS>ssb   ed25519/C5A84090 2020-06-14 [S] [expires: 2023-06-14]
ssb   ed25519/50E06E86 2020-06-14 [A] [expires: 2023-06-14]
ssb   cv25519/19D7986D 2020-06-14 [E] [expires: 2023-06-14]

Everything goes well as you said.


Initial pass at Unicode plumbing.

Fixes #85.
This is a pass at the following tasks, needed for unicode support:

Decode utf-8 wherever we process input. The places where we expect input are relatively few -- all through the loader
import unicode_literal from __future__ on all files with string literals
str conversions go through the Python 2/3-compatible six.text_type

This may not be a comprehensive list, and my first pass may not have knocked them all out thoroughly. Let's see how Travis handles this.

Codecov Report

Merging #86 into master will increase coverage by 0.13%.
The diff coverage is 95.83%.

@@            Coverage Diff             @@
##           master      #86      +/-   ##
==========================================
+ Coverage   88.31%   88.45%   +0.13%
==========================================
  Files           7        7
  Lines         676      684       +8
==========================================
+ Hits          597      605       +8
  Misses         79       79

Impacted Files
Coverage Δ

apacheconfig/wloader.py
46.93% <100%> (+1.1%)
:arrow_up:

apacheconfig/reader.py
100% <100%> (ø)
:arrow_up:

apacheconfig/lexer.py
96.68% <100%> (+0.03%)
:arrow_up:

apacheconfig/loader.py
89.24% <100%> (+0.07%)
:arrow_up:

apacheconfig/parser.py
85.71% <87.5%> (+0.25%)
:arrow_up:

Continue to review full report at Codecov.

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

One more thing: would you consider this change worth noting in CHANGELOG? Not sure if it fixes a bug of introduces a feature or both? %-)

@sydneyli I honestly tried to resolve the conflicts, but have eventually given up in fear to break something. %-)

Thank you!

Somehow, this failure occurs only with Py3.7 job:
Traceback (most recent call last):
 File "/home/travis/build/etingof/apacheconfig/tests/integration/test_perl_samples.py", line 139, in testParseFiles
   with make_loader() as loader:
 File "/opt/python/3.7.1/lib/python3.7/contextlib.py", line 112, in __enter__
  return next(self.gen)
  File "/home/travis/build/etingof/apacheconfig/apacheconfig/__init__.py", line 14, in make_loader
   ApacheConfigLexer = make_lexer(**options)
 File "/home/travis/build/etingof/apacheconfig/apacheconfig/lexer.py", line 366, in make_lexer
   lexer_class = type(six.binary_type('ApacheConfigLexer'),
TypeError: string argument without an encoding

Thanks for the note-- might be from merging the unicode & new naming branch. I'll see if I can fix things in #89 !


[FEATURE] MicroBlink - Expose customization of MBCBlinkCardRecognizer

Is your feature request related to a problem? Please describe.
We want the ability to not have microblink attempt to gather the owner information
Describe the solution you'd like
Microblink already exposes the ability to not require owner information collection.
https://blinkcard.github.io/blinkcard-ios/Classes/MBCBlinkCardRecognizer.html#/c:objc(cs)MBCBlinkCardRecognizer(py)extractOwner
Expose a way for users to pass false for extractOwner

@trevor-holliday-instacart ok, I will add it in next release and let you know when it's ready.

@trevor-holliday-instacart this is released in v1.16.1.


Deleting from mongo

HI This is great App.
I have it working for inserting data in mongo and it syncs to Elasticsearch, however in MongoData there is a _id field which it didnt like so i
addeddocument.MongoID = document._id;
document = _.omit(document, '_id');

to the transformFunction, which was ok, but when deleting records from Mongo, it gives error

/Users/collia01/Node/rabbittomongo/node_modules/node-elasticsearch-sync/lib/util.js:143
if (!response.found) {
^
TypeError: Cannot read property 'found' of undefined
at /Users/collia01/Node/rabbittomongo/node_modules/node-elasticsearch-sync/lib/util.js:143:20
at Object.utils.applyArgs (/Users/collia01/Node/rabbittomongo/node_modules/elasticsearch/src/lib/utils.js:252:17)
at wrapper (/Users/collia01/Node/rabbittomongo/node_modules/elasticsearch/node_modules/lodash/index.js:3533:19)
at nextTickCallbackWith0Args (node.js:433:9)
at process._tickCallback (node.js:362:13)

Is this because the _id has been renamed in elasticseach?  Is there a way around this.
Thank you
Andy

It's a good thing to remove rename the _id field before indexing in elasticsearch, so that is not the issue here. It's probably from the checks being done to ensure that documents really exist in elasticsearch cluster. Will try to replicate that and see what fix can be applied.

@andymcollins this #5  might be a possible fix. Update, test and let me know if it works as it works down here on my machine without any error.

When i remove the _Id from the document using
document = _.omit(document, '_id'); i get message
ESMongoSync: Inserted -  Index:  test  Type:  external  Id:  undefined
then when i try to delete a record in MongoDB i get message
ESMongoSync: Not Found -  Document with id 5728aefd9191198f6ca9991d not found.
if i try to rename the _id field to id with
document.id = document._id;
i dont get any message to say it has been inserted, and nothing appears in Elasticsearch.
How do i get the data into Elasticsearch so that any updates/deletes in Mongo are replicated to elasticseach. It seems to just be the ID field.
Thanks

this is data i'm adding, id is the renamed string coming from MongoDB
{"processedData":"2016-05-03T14:04:18.948Z",
"queue":"AndyJobs",
"messageID":"1200319843",
"oracleProcessedDate":null,
"Details":{"VAC_REF":"1200319843",
"JOB_TITLE":"string TEXT HERE",
"LOCATION":"Surrey",
"AGENCY_ID":18462,
"EMP_MKT_CD":"EN",
"NeutralForm":"String textr",
"Discipline":"Construction",
"SubDiscipline":"Water",
"RemainingSource":"string"},
"id":"5728aff39191198f6ca9997f"}

Fixed with my latest pull request


Native Federation: Library with different version is not shared as expected due to config of required version

Hello,
I have create a dynamic shell and a remote as described under
https://www.npmjs.com/package/@angular-architects/native-federation
I have installed<EMAIL_ADDRESS>in the shell and<EMAIL_ADDRESS>in the remote application.
After integrating the remote into the shell I would expect that lib version 1.0.1 is shared, but the library is loaded in both versions.

The source code can be found under
https://github.com/maan1112/native-federation-shell
https://github.com/maan1112/native-federation-remote
How to configure remote to use shells library?
Thanks M.

Yes, that's true. Currently, Native Federation uses a simplified strategy: It only shares libs if they have exactly the same version. This makes sense in the case of Angular because compiled Angular code uses parts of Angular's private API and there are no guarantees in terms of semver for private APIs.
Nevertheless, sooner or later, we will provide the flexibility known from module federation.


fix(website): Add "read blog" tag above Hero title, adjust hero margins and font sizes

I think there's a slight black rounded background on the link? Can that be removed?

wait wdym rounded background? i think i added it on purpose to give it more contrast


Safer installation method

Piping the output of curl or wget -O- to a shell is a very bad idea for multiple reasons; security concerns aside, doing so provides no opportunity for the script's download to finish before execution starts, so if the download is interrupted for some reason (perhaps because of an intermittent network failure), the incompletely-downloaded script will still run with potentially disastrous results.  See here for a more in-depth description.

FWIW, that command would run the following:

https://github.com/progrium/dokku/blob/master/bootstrap.sh#L33

Which is what is in our debian installation notes.


update react to v17.0.1

Tested the plugin with latest changes.
https://reactjs.org/blog/2020/08/10/react-v17-rc.html#other-breaking-changes

To fix the build issues you will need to

remove the es3Check() from the applicationinsights-react-js/rollup.config.js (as this is not supported on ES3 (IE 7/8))
Add a note to the README.md stating that this extension will no longer work with ES3 by default because react 17.x does not support ES3 without the react-app-polyfill. (I don't this is a breaking issue as react browser support by default is IE9+ (ES5)

Looks like we already have second point stated in the readme:


Compression level has no effect on ZlibEncoder write

I tried zlibencoder-write.rs example with different compression levels (none, default, fast, and best), and the result is the same for all levels.

What content are you encoding? Perhaps it's small enough that zlib doesn't change based on the level?

I tried it with bytes from an 40 MB file. The output still has [0x78, 0x01] as the first two bytes indicates the lowest compression level.

Can you provide an example test case for this?

Those two first bytes are the zlib header structure. The compression level flag in the header doesn't actually do anything, so miniz is lazy and just writes the same values no matter the compression level. It doesn't have any bearing on the actual compression level.

With the larger file, there does seem to be difference in compression, although the zlib header is the same.


Cloned and executed hugo server and didn't work

Hello,
Just followed your installation suggestion and ran hugo server on the main folder and also on the exampleSite and it didn't work.
Can you clarify this part?
Best regards,

hello @ricardopt ,
Please follow this doc to run this theme locally https://docs.gethugothemes.com/guide/#run-theme-locally

Hi @ricardopt
I am going to close this issue, If you have any query to ask, feel free to re-open it, or create a new issue.


live

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

windows-iotcore/TOC.md
:white_check_mark:Succeeded

windows-iotcore/commercialize-your-device/OEMLicenseRequirements.md
:white_check_mark:Succeeded

windows-iotcore/connect-your-device/MBB.md
:white_check_mark:Succeeded

windows-iotcore/develop-your-app/IoTCoreShell.md
:white_check_mark:Succeeded

windows-iotcore/learn-about-hardware/SoCsAndCustomBoards.md
:white_check_mark:Succeeded

windows-iotcore/media/adafruitkit/pack.jpg
:white_check_mark:Succeeded

windows-iotcore/secure-your-device/SecureBootAndBitLocker.md
:white_check_mark:Succeeded

windows-iotcore/secure-your-device/TPM.md
:white_check_mark:Succeeded

windows-iotcore/tutorials/Tutorials.md
:white_check_mark:Succeeded

windows-iotcore/tutorials/adafruitkit.md
:white_check_mark:Succeeded

windows-iotcore/windows-iot-enterprise.md
:white_check_mark:Succeeded

windows-iotcore/windows-iot.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Can you help compile a version with magic

I want to use the gif function, but it always fails during compilation

If this is for Windows, you'll need to base it on the "all" formats upstream release rather than only "web" formats.
https://github.com/libvips/build-win64-mxe

I don’t understand here, how to increase the dependency

In addition, it is not windows, but linux 64 。 Is this the command.
~] bash build.sh --PLATFORM="linux-x64"

Don't know why it became win

The docs at https://github.com/lovell/sharp-libvips#creating-a-tarball say:

Run the top-level build script without parameters for help.

Doing so reveals:
$ ./build.sh

Usage: ./build.sh VERSION [PLATFORM]
Build shared libraries for libvips and its dependencies via containers

Please specify the libvips VERSION, e.g. 8.9.2

Optionally build for only one PLATFORM, defaults to building for all

Possible values for PLATFORM are:
- win32-ia32
- win32-x64
- linux-x64
- linuxmusl-x64
- linux-armv6
- linux-armv7
- linux-arm64v8
- darwin-x64

So for example you might try the following:
$ ./build.sh 8.10.0 linux-64

A word of warning: modifying the scripts in this repo is not straightforward and will require experience with shell scripting and compiling Linux software from source.

Yes, it is more difficult for me. Do I have other ways to compile? I tried to compile directly with libvips, but it doesn’t have some dependencies by default, such as svg, I don’t know how to add

@lovell
I echoed a lot of breakpoints and found that it was a problem during version checking, possibly caused by my network. The compilation is successful, but I may need to clone a sharp to install my own sharp.
Do you think it is necessary for sharp to directly compile a version with magic, and then install this version by reading the configuration of package.json or other files when npm i sharp. There may be other users who need it, but it is too troublesome to compile by yourself

It has been successful. Install vips globally. Record it

First need to install some dependencies

yum install -y libjpeg* libpng* libtiff* libungif* freetype zlib
yum install ImageMagick
yum install ImageMagick-devel
yum install librsvg2-devel

If there is a loader library for some pictures (such as yum install librsvg2-devel), vips will be called, otherwise it will be processed by ImageMagick.

Follow the libvips document to install, the configuration should be noted:

./configure --with-magickpackage=ImageMagick --with-modules

Thanks for the update, glad to hear you got it working.

Could you please post changes you've made on the bash script to enable imagemagick to be used by vips?
I tried installing imagemagick and set --with-magick flag but the output library wasn't using imagemagick


zeek: 4.0.3 -> 4.1.0

Motivation for this change
https://github.com/zeek/zeek/releases/tag/v4.1.0
Things done

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

[x] NixOS
[x] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
21.11 Release Notes (or backporting 21.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

@ofborg eval

Result of nixpkgs-review pr 133574 run on x86_64-linux 1

3 packages built:

caf
python38Packages.zeek
zeek (python39Packages.zeek)


Support for additional dotnet publish flags.

Many build scenarios and scripts require certain flags to succeed.  For example nuget restore might be required prior to running electronize build and support for the --no-restore flag in the generated publish command is required.
This PR adds the capability to add any of the dotnet publish flags to the electronize build command by appending the command with -- [--dotnet-publish-flag(s)]
Backwards compatibility to keep current builds running is maintained by adding the --self-contained flag if not present or overridden by --self-contained false|--no-self-contained.  Additionally any other flags that are already built into the design of the command are ignored or stripped.

I've got a logic error somewhere in here... working on it


Unexpectedly Excited When Clicking the Open Preview Button

Does this issue occur when all extensions are disabled?: Yes/No

VS Code Version: 1.90.1
OS Version: Windows_NT x64 10.0. 19045

Steps to Reproduce:

Initial Markdown Preview Activation:

Open a Markdown file from the Explorer in Visual Studio Code.
Click the "Open Preview" button to activate the side-by-side preview mode, where the Markdown code is displayed on one side and the rendered preview on the other.

Switching to Another Markdown File:

Click on another Markdown file in the Explorer.
The editor will display the Markdown code for this new file.
Click the "Open Preview" button again for this file.

Unexpected Behavior:

Upon clicking the "Open Preview" button for the second file, Visual Studio Code unexpectedly exits.

The issue is not happened on my side anymore. I'm not sure why this was happened on my side at that time


debug connection issues

This builds on top of https://github.com/intel/pmem-csi/pull/738 and adds commits that may address the sanity test connection issues.

In https://cloudnative-k8sci.southcentralus.cloudapp.azure.com/job/pmem-csi/job/version-skew-2/17/testReport/junit/fedora-1_18.lvm-testing/sanity/Testing___1_18___PMEM_CSI_can_publish_volume_after_a_node_driver_restart/ there was another dial tcp <IP_ADDRESS>:9735: connect: connection refused error despite including these changes here.
Full log in https://cloudnative-k8sci.southcentralus.cloudapp.azure.com/job/pmem-csi/job/version-skew-2/17/execution/node/138/log/?consoleFull
So while this PR here may help a bit (more obvious error reporting, avoid excessive CI test runtime), it doesn't help when the cluster gets into that state where the socat port cannot be reached.
I'm wondering whether this started to happen because the docker.io/alpine/socat image changed. We don't lock it down to a specific version.

I'm wondering whether this started to happen because the docker.io/alpine/socat image changed. We don't lock it down to a specific version.

No, https://hub.docker.com/r/alpine/socat/tags shows that "latest" hasn't changed for over 8 months.

I think I have finally nailed the root cause for the "connection refused" issue - see commit f4a7062.
@avalluri : please review, then we can merge this and thus also PR #738.


[TextField] onChange not called when using setValue/clearValue

I have a case where I want to change the value of a TextField programmatically:
var SearchBox = React.createClass({
...
changeSearch: function(q) { this.refs.searchBox.setValue(q); },
clearSearch: function() { this.refs.searchBox.clearValue(); },
...
render: function() {
    return (
        <div className='search-box'>
            <TextField ref='searchBox' onChange={this.props.changeHandler}/>
        </div>
    );
}
});

I've noticed that an onChange event is never triggered when I use setValue/clearValue: is this a normal behavior?

Seems like the normal behaviour. There's no call to the onChange callback.
https://github.com/callemall/material-ui/blob/master/src/text-field.jsx#L352
I've done something like this in the past:
<TextField
    onChange={this.props.changeHandler}
    value={this.state.searchBoxValue} />

If you now want to manipulate the value you do it by chaning the state.

You're right, thanks!
Realized it's also the normal behavior in vanilla Javascript when programmatically manipulating input values.


Separate tool/endpoint for running deployDockerStack

Probably it's time to have a single-button release management

Maybe consider single-button release already for Kubernetes? It will require simply connection Github actions to Kubernetes API server using Github secrets and using well-established set of tools like Helm, instead of dealing with remote Gradle invocation.


CVCalendar has no Package.swift manifest for version 1.6.2

[x] I have verified there are no duplicate active or recent bugs, questions, or requests
[ ] I have verified that I am using the latest version of CVCalendar.
[ ] I have given my issue a non-generic title.

@Dilan-Anuruddha That's correct, the project is not distributed via Swift Package Manager due to it's heavy reliance on UIKit which still seemed to be hard to deal with via SPM. If you have an idea on how to solve this, please submit a PR!

Please see the latest answers to this SO question


L1 to L3 Bridgers latest

gas estimation is probably the most complicated part of this code
CI is failing because the teleporter contracts npm package is out of date
this is how the Erc20L1L3Bridger should be used, note that getDepositRequest returns a feetoken amount
// our deposit parameters
const depositParams = {
  erc20L1Address: l1Token.address,
  to: l3Recipient,
  amount,
  l1Signer,
  l2Provider: l2Signer.provider!,
  l3Provider,
}

// get a deposit request
const depositRequest = await bridger.getDepositRequest(depositParams)

// approve fee token if necessary
if (depositRequest.feeTokenAmount.gt(0)) {
  await (
    await bridger.approveFeeToken({
      l1Signer,
      l2Provider: l2Signer.provider!,
      amount: depositRequest.feeTokenAmount,
    })
  ).wait()
}

// approve token
await (
  await bridger.approveToken({
    erc20L1Address: l1Token.address,
    l1Signer,
    amount: depositParams.amount,
  })
).wait()

// deposit, use the deposit request we generated earlier
// if the fee token is already approved, you don't really need to getDepositRequest first, you can just pass depositParams to deposit
const depositTx = await bridger.deposit({
  l1Signer,
  txRequest: depositRequest.txRequest,
})

const depositReceipt = await depositTx.wait()

// get deposit status, can pass in txHash, tx, or txReceipt
const status = await bridger.getDepositMessages({
  txReceipt: depositReceipt,
  l1Provider: l1Signer.provider!,
  l2Provider: l2Signer.provider!,
  l3Provider,
})

to test:
link the contracts repo
cd <teleport contracts> && yarn link

cd <sdk>
yarn link "@offchainlabs/l1-l3-teleport-contracts"
yarn install

spin up the testnode in custom or eth fee mode
# to spin up a custom fee L3:
./test-node.bash --init \
    --l3node \
    --tokenbridge \
    --l3-token-bridge \
    --l3-fee-token

# to spin up an ETH fee L3:
./test-node.bash --init \
    --l3node \
    --tokenbridge \
    --l3-token-bridge

finally:
yarn gen:network && yarn mocha tests/integration/l1l3Bridger.test.ts --timeout 100000 --bail

would it be better for Erc20L1L3Bridger to have a private constructor, and instead have an async create ... function? that way it could check that the l1feetoken exists and is valid before you can actually use the bridger. It will also remove the need for an extra provider for some functions that need to get the l1 fee token.


Windows | Build failing at junits

Expected Behaviour
Build should be successful
Actual Behaviour
Build fail at junits
Reproduce Scenario (including but not limited to)
mvn clean install at master branch for 2.7.1-SNAPSHOT
Steps to Reproduce
Checkout code and build using mvn clean install
Platform and Version
Windows
Logs taken while reproducing problem
ERROR] Errors:
[ERROR] CommerceLinksTransformerTest.testTransformerDisabled:158 ▒ InvalidPath Illegal...
[INFO]
[ERROR] Tests run: 626, Failures: 0, Errors: 1, Skipped: 0
[INFO]
17:18:04.761 [main] INFO c.a.c.c.c.c.i.s.CommerceLinksTransformerFactory - Commerce links transformer enabled.
17:18:05.095 [main] INFO c.a.c.c.c.c.i.s.CommerceLinksTransformerFactory - Commerce links transformer disabled.
[ERROR] Tests run: 2, Failures: 0, Errors: 1, Skipped: 0, Time elapsed: 0.477 s <<< FAILURE! - in com.adobe.cq.commerce.core.components.internal.services.CommerceLinksTransformerTest
[ERROR] testTransformerDisabled(com.adobe.cq.commerce.core.components.internal.services.CommerceLinksTransformerTest) Time elapsed: 0.22 s <<< ERROR!
java.nio.file.InvalidPathException: Illegal char <:> at index 2: /C:/Users/anupatel/Desktop/Unilever/cif-core-components/aem-core-cif-components/bundles/core/target/test-classes/rewriter/ciflinks.html
at com.adobe.cq.commerce.core.components.internal.services.CommerceLinksTransformerTest.testTransformerDisabled(CommerceLinksTransformerTest.java:158)

This would require inlining the html from the ciflinks.html into the java class. Please feel free to provide a pull request for this.

@Buuhuu I have gone through the code base of java.
can you please me with changes..
// verify transformed HTML
String transformedHtml = writer.toString();
    Path filePath = Paths.get(classLoader.getResource(TEST_HTML).getPath());

    String originalHtml = Files.lines(filePath).collect(Collectors.joining(System.lineSeparator()));

    assertTrue(transformedHtml.endsWith(originalHtml));

Thanks,


Feat: Resolve 'connect' functionality for UI

UI is meant to be publicly facing, but we do not have a way for the UI to connect to a remote backend. Need to resolve the intended use case and how to apply it.

Start by discussing with team how it currently works and if the UI only can connect to the Climate Explorer on localhost or if there's a way to connect it to a remote instance.


Discovery - Reactive Contact Tracing - Interview Scipt

Background
TBD
Old Script to build from

Creating new script here for personas/customer pool outlined in our customer pool.
CC @Mcscott142 @HauwaAguillard @maryolivier @bmccrossen

Moved to LDH Repo
https://github.com/revelrylabs/LDH/issues/2
https://github.com/revelrylabs/LDH/issues/3


Support Rails 7.2+

The schema_migration method was moved in https://github.com/rails/rails/pull/51162
Red tests are due to https://github.com/DatabaseCleaner/database_cleaner-active_record/pull/93

@casperisfine would Rails ever consider a public API for accessing this model or table?

would Rails ever consider a public API for accessing this model or table name?

I'd be fine with schema_migration being public but not the class. When I removed the inheritance on Base it also removed the ability to do most queries, so it only responds to some methods. I don't want PRs making that class (or InternalMetadata) more robust.

Not necessarily a breaking change for 7.2 but it would be great if 102 could also get included with this.

@ghiculescu Looks good, thank you!


[Azure Search] Investigate frequency of latest version delete or unlist

With some custom, code, I found 92 removals per day, on average, in the year of 2018.

removed-with-grouping.xlsx

The max value was 791, on March 29, when LEADTOOLS unlisted many different package IDs.


WIP: Fix codespaces

Somehow I force pushed and broke my working codespaces env in #165

Codecov Report

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

@@           Coverage Diff           @@
##           master     #173   +/-   ##
=======================================
  Coverage   97.93%   97.93%
=======================================
  Files          34       34
  Lines         870      870
=======================================
  Hits          852      852
  Misses         18       18

Continue to review full report at Codecov.

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


feat: Add Snowflake table last updated timestamp extractor

Summary of Changes
Related issue : https://github.com/amundsen-io/amundsen/issues/664
Adding a new extractor to extract table last updated timestamp for tables in Snowflake.
Tests
Added new unit test test_snowflake_last_updated_timestamp_extractor.py to improve coverage.
Documentation
Updated the README.md file to update extractor list and added some notes on how to use the new extractor
CheckList
Make sure you have checked all steps below to ensure a timely review.

[X] PR title addresses the issue accurately and concisely. Example: "Updates the version of Flask to v1.0.2"

In case you are adding a dependency, check if the license complies with the ASF 3rd Party License Policy.

[X] PR includes a summary of changes.
[X] PR adds unit tests, updates existing unit tests, OR documents why no test additions or modifications are needed.
[X] In case of new functionality, my PR adds documentation that describes how to use it.

All the public functions and the classes in the PR contain docstrings that explain what it does

[X] PR passes make test

cc @feng-tao

cc @jinhyukchang

@feng-tao Please let me know what you think. Would like to get this merged if we are okay with adding this extractor.

@feng-tao PR updated. Please take a look and let me know what you think. Thanks!