arrow-cpp: add flight feature flag

Motivation for this change
Add the option of building arrow-cpp with Flight enabled.
Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested basic functionality 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.

This is a semi-automatic executed nixpkgs-review with nixpkgs-review-checks extension. It is checked by a human on a best effort basis and does not build all packages (e.g. lumo, tensorflow or pytorch).
Result of nixpkgs-review pr 144609 run on x86_64-linux 1

21 packages built:

arrow-cpp
python38Packages.datasets
python38Packages.google-cloud-bigquery
python38Packages.intake
python38Packages.intake-parquet
python38Packages.pyarrow
python38Packages.snowflake-connector-python
python38Packages.snowflake-sqlalchemy
python38Packages.spacy-transformers
python38Packages.tokenizers
python38Packages.transformers
python39Packages.datasets
python39Packages.google-cloud-bigquery
python39Packages.intake
python39Packages.intake-parquet
python39Packages.pyarrow
python39Packages.snowflake-connector-python
python39Packages.snowflake-sqlalchemy
python39Packages.spacy-transformers
python39Packages.tokenizers
python39Packages.transformers


Does the "Get an Album" Spotify API ever return any genres?

Looks like it does not.

Hi @dandv,
Yeah, looks like that most of the time Track.genres is most of the time empty.
I know that most examples I saw on Github used the linked Artist.genres which is most of the time filled.

Note: add documentation on this point for Track.genres in<EMAIL_ADDRESS>cf: #79

Hi @dandv,
Yeah, looks like that most of the time Track.genres is most of the time empty.
I know that most examples I saw on Github used the linked Artist.genres which is most of the time filled.

Note: add documentation on this point for Track.genres in<EMAIL_ADDRESS>cf: #79


method names changed in asyncAirtable.js, tests, and readme

won't let me work on tests without an API key, sorry!

Hi,
Please also change the main.test.js file so it has all the correct method names.

Handled with #15


Renaming Bundles: Auto focus edit box

When renaming a Bundle, the edit box in the dialog should be focused.

@R3voA3 This enhancement was implemented with #3634.


How to mount azure fileshare when using AKS to  host azure functions with KEDA?

I cannot find any option to mount azure fileshare to the deployment file when I deploy azure functions to kubernetes cluster.

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

ID: 2cbf4053-0a3d-a803-3e0d-8cf28db358e8
Version Independent ID: 782efd9c-beea-249c-f8e8-4ee4389bfb07
Content: Azure Functions on Kubernetes with KEDA
Content Source: articles/azure-functions/functions-kubernetes-keda.md
Service: azure-functions
GitHub Login: @eamonoreilly
Microsoft Alias: eamono

Hello @sanjeebSubedi What is the actual document you are referring to?

https://github.com/MicrosoftDocs/azure-docs/blob/main/articles/azure-functions/functions-kubernetes-keda.md I am referring to this one. I used to mount azure fileshare on azure functions and access my files from functions, now I have to migrate to using keda due to resource limits. I want to mount the fileshare onto the pod when deployment is created.

@sanjeebSubedi , Answered similar question in the below forum
https://docs.microsoft.com/en-us/answers/questions/833306/how-to-mount-azure-fileshare-to-kubernetes-for-use.html?childToView=844597#answer-844597
@eamonoreilly , please feel free to add anything here


go/packages: fake command-line-args package isn't created on windows when ad-hoc package file is missing

I haven't been able to reproduce this for go list, so I'm setting this as a go/packages bug for now.
It seems like on Windows, when doing a go list -e of an ad-hoc package where a source file doesn't exist, the fake command-line-args package that contains the error isn't created. See the comment on golang.org/x/tools/go/packages.TestErrorMissingFile.

CC @jayconrod

Is this in module mode, GOPATH mode, or both?

It looks like this is happening for both.


Druid villager needs new texture, since 1.11 will be using the green coat

Also there are some pixels out of place on the side of the sage.

Fixed in new release https://minecraft.curseforge.com/projects/cyclic/files/2333207


[Bug]: replay from system tray

Describe the bug
When attempting to replay a recording from the system tray:
Process Process-1:5:
Traceback (most recent call last):
  File "/Users/abrichr/Library/Caches/pypoetry/virtualenvs/openadapt-VBXg4jpm-py3.10/lib/python3.10/site-packages/sqlalchemy/engine/base.py", line 1900, in _execute_context
    self.dialect.do_execute(
  File "/Users/abrichr/Library/Caches/pypoetry/virtualenvs/openadapt-VBXg4jpm-py3.10/lib/python3.10/site-packages/sqlalchemy/engine/default.py", line 736, in do_execute
    cursor.execute(statement, parameters)
sqlite3.OperationalError: database is locked

The above exception was the direct cause of the following exception:

Traceback (most recent call last):
  File<EMAIL_ADDRESS>line 314, in _bootstrap
    self.run()
  File<EMAIL_ADDRESS>line 108, in run
    self._target(*self._args, **self._kwargs)
  File "/Users/abrichr/oa/OpenAdapt/openadapt/record.py", line 808, in memory_writer
    crud.insert_memory_stat(
  File "/Users/abrichr/oa/OpenAdapt/openadapt/db/crud.py", line 177, in insert_memory_stat
    _insert(memory_stat, MemoryStat, memory_stats)
  File "/Users/abrichr/oa/OpenAdapt/openadapt/db/crud.py", line 64, in _insert
    result = db.execute(sa.insert(table), to_insert)
  File "/Users/abrichr/Library/Caches/pypoetry/virtualenvs/openadapt-VBXg4jpm-py3.10/lib/python3.10/site-packages/sqlalchemy/orm/session.py", line 1714, in execute
    result = conn._execute_20(statement, params or {}, execution_options)
  File "/Users/abrichr/Library/Caches/pypoetry/virtualenvs/openadapt-VBXg4jpm-py3.10/lib/python3.10/site-packages/sqlalchemy/engine/base.py", line 1705, in _execute_20
    return meth(self, args_10style, kwargs_10style, execution_options)
  File "/Users/abrichr/Library/Caches/pypoetry/virtualenvs/openadapt-VBXg4jpm-py3.10/lib/python3.10/site-packages/sqlalchemy/sql/elements.py", line 333, in _execute_on_connection
    return connection._execute_clauseelement(
  File "/Users/abrichr/Library/Caches/pypoetry/virtualenvs/openadapt-VBXg4jpm-py3.10/lib/python3.10/site-packages/sqlalchemy/engine/base.py", line 1572, in _execute_clauseelement
    ret = self._execute_context(
  File "/Users/abrichr/Library/Caches/pypoetry/virtualenvs/openadapt-VBXg4jpm-py3.10/lib/python3.10/site-packages/sqlalchemy/engine/base.py", line 1943, in _execute_context
    self._handle_dbapi_exception(
  File "/Users/abrichr/Library/Caches/pypoetry/virtualenvs/openadapt-VBXg4jpm-py3.10/lib/python3.10/site-packages/sqlalchemy/engine/base.py", line 2124, in _handle_dbapi_exception
    util.raise_(
  File "/Users/abrichr/Library/Caches/pypoetry/virtualenvs/openadapt-VBXg4jpm-py3.10/lib/python3.10/site-packages/sqlalchemy/util/compat.py", line 208, in raise_
    raise exception
  File "/Users/abrichr/Library/Caches/pypoetry/virtualenvs/openadapt-VBXg4jpm-py3.10/lib/python3.10/site-packages/sqlalchemy/engine/base.py", line 1900, in _execute_context
    self.dialect.do_execute(
  File "/Users/abrichr/Library/Caches/pypoetry/virtualenvs/openadapt-VBXg4jpm-py3.10/lib/python3.10/site-packages/sqlalchemy/engine/default.py", line 736, in do_execute
    cursor.execute(statement, parameters)
sqlalchemy.exc.OperationalError: (sqlite3.OperationalError) database is locked
[SQL: INSERT INTO memory_stat (id, recording_timestamp, memory_usage_bytes, timestamp) VALUES (?, ?, ?, ?)]
[parameters: (None,<PHONE_NUMBER>.935582,<PHONE_NUMBER>.0,<PHONE_NUMBER>.0527885)]
(Background on this error at: https://sqlalche.me/e/14/e3q8)

To Reproduce

run python -m openadapt.app.tray
click system tray -> record
wait for recording to start
type some words on the keyboard
click system tray -> stop recording type oa.stop
click system tray -> replay -> latest recording
observe system tray still shows "Stop recording", even though recording has stopped

Fixed


Cannot install moco 0.25.0 without admissionregistration.k8s.io/v1beta1=true for k8s 1.30 or above

Describe the bug
#751 introduces ValidatingAdmissionPolicy. This feature is GA in Kubernetes 1.30.
https://kubernetes.io/blog/2024/04/24/validating-admission-policy-ga/
Kubernetes 1.30 or above only accepts admissionregistration.k8s.io/v1, not admissionregistration.k8s.io/v1beta1 by default.
Environments

Version: 0.25.0
K8s: 1.30.6

To Reproduce
kind create cluster --name moco --image kindest/node:v1.30.6
curl -fsL https://github.com/jetstack/cert-manager/releases/latest/download/cert-manager.yaml | kubectl apply -f -
helm install --create-namespace --namespace moco-system moco moco/moco

Following errors are shown:
Error: INSTALLATION FAILED: unable to build kubernetes objects from release manifest: [resource mapping not found for name: "moco-delete-validator" namespace: "" from "": no matches for kind "ValidatingAdmissionPolicy" in version "admissionregistration.k8s.io/v1beta1"
ensure CRDs are installed first, resource mapping not found for name: "moco-delete-validator" namespace: "" from "": no matches for kind "ValidatingAdmissionPolicyBinding" in version "admissionregistration.k8s.io/v1beta1"
ensure CRDs are installed first]

Expected behavior
moco 0.25.0 can be installed for any supported kubernetes versions.
Additional context
K8s cluster used in E2E Test enables the admissionregistration.k8s.io/v1beta1.
https://github.com/cybozu-go/moco/blob/389ae1248865a443f9fd152406d630faaecf0cd5/e2e/kind-config.yaml#L3-L6
So the tests passed, but installation fails for the cluster with default configuration.
Replace admissionregistration.k8s.io/v1beta1 with admissionregistration.k8s.io/v1, then it can be installed without any configuration.
helm template --namespace moco-system moco moco/moco > manifests.yaml
kubectl create ns moco-system
sed s%admissionregistration.k8s.io/v1beta1%admissionregistration.k8s.io/v1%g manifests.yaml | kubectl apply -f -

❯ sed s%admissionregistration.k8s.io/v1beta1%admissionregistration.k8s.io/v1%g manifests.yaml | kubectl apply -f -
serviceaccount/moco-controller-manager created
customresourcedefinition.apiextensions.k8s.io/backuppolicies.moco.cybozu.com created
customresourcedefinition.apiextensions.k8s.io/mysqlclusters.moco.cybozu.com created
clusterrole.rbac.authorization.k8s.io/moco-backuppolicy-editor-role created
clusterrole.rbac.authorization.k8s.io/moco-backuppolicy-viewer-role created
clusterrole.rbac.authorization.k8s.io/moco-manager-role created
clusterrole.rbac.authorization.k8s.io/moco-mysqlcluster-editor-role created
clusterrole.rbac.authorization.k8s.io/moco-mysqlcluster-viewer-role created
clusterrolebinding.rbac.authorization.k8s.io/moco-manager-rolebinding created
role.rbac.authorization.k8s.io/moco-leader-election-role created
rolebinding.rbac.authorization.k8s.io/moco-leader-election-rolebinding created
service/moco-webhook-service created
deployment.apps/moco-controller created
certificate.cert-manager.io/moco-controller-grpc created
certificate.cert-manager.io/moco-grpc-ca created
certificate.cert-manager.io/moco-serving-cert created
issuer.cert-manager.io/moco-grpc-issuer created
issuer.cert-manager.io/moco-selfsigned-issuer created
mutatingwebhookconfiguration.admissionregistration.k8s.io/moco-mutating-webhook-configuration created
validatingadmissionpolicy.admissionregistration.k8s.io/moco-delete-validator created
validatingadmissionpolicybinding.admissionregistration.k8s.io/moco-delete-validator created
validatingwebhookconfiguration.admissionregistration.k8s.io/moco-validating-webhook-configuration created

❯ kubectl get po -n moco-system
NAME                               READY   STATUS    RESTARTS   AGE
moco-controller-6d7867d984-gl8n8   1/1     Running   0          29s
moco-controller-6d7867d984-gwrdk   1/1     Running   0          29s

❯ kubectl get validatingadmissionpolicy
NAME                    VALIDATIONS   PARAMKIND   AGE
moco-delete-validator   1             <unset>     60s

❯ kubectl get validatingadmissionpolicybinding
NAME                    POLICYNAME              PARAMREF   AGE
moco-delete-validator   moco-delete-validator   <unset>    78s

If additional FeatureGates are required for installation, it should be documented.
ValidatingAdmissionPolicy is not available in K8s 1.29 by default.

@pddg
Thank you for the report.
It seems there was an omission in the documentation, so I will take care of it.

Fixed after https://github.com/cybozu-go/moco/pull/760 (tested by installing chart from git repo), We should wait for chart release :)

Chart just have been released. chart-v0.15.1


fix: auto-upgrade requirements + charts

add a custom upgrade mechanism so that if folks are using the old tekton chart or build packs we auto-upgrade to the new shiny tekton engine based build pack and latest tekton chart

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To complete the pull request process, please assign jstrachan
You can assign the PR to them by writing /assign @jstrachan in a comment when ready.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

/approve

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: jstrachan
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [jstrachan]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

/approve

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: jstrachan
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [jstrachan]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: jstrachan
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [jstrachan]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


Improve fork config generation for berkeley

Explain your changes:

Merge changes from #14266 before they can land in compatible.

Explain how you tested your changes:

Connect to a testnet and extract the ledger.
CI.

Checklist:

[x] Dependency versions are unchanged

Notify Velocity team if dependencies must change in CI

[x] Modified the current draft of release notes with details on what is completed or incomplete within this project
[x] Document code purpose, how to use it

Mention expected invariants, implicit constraints

[x] Tests were added for the new behavior

Document test purpose, significance of failures
Test names should reflect their purpose

[x] All tests pass (CI will check this if you didn't)
[x] Serialized types are in stable-versioned modules
[x] Does this close issues? List them

Closes https://github.com/MinaFoundation/mina/issues/95

!ci-build-me

!ci-build-me

!ci-build-me

@bkase any chance to review this PR ?

!ci-build-me

!ci-build-me

Can you cherry-pick the changes you did here: https://github.com/MinaProtocol/mina/pull/14285/commits/27157c9f2cfccea40304fcdd826197888936ee5d into this PR too?

Hi @ghost-not-in-the-shell . I've cherry-picked the changes. Can you review when you are free and I can merge? Thanks.

!ci-build-me

!ci-build-me

!ci-build-me

!ci-build-me

Heads up: I've reverted https://github.com/MinaProtocol/mina/pull/14285 for now, as this PR needs reviewed first. Any suggested changes on that merge PR should be ported back to here, and the merge PR can be recreated once the review here is completed (if still necessary).

This will be redone after https://github.com/MinaProtocol/mina/pull/14266 gets approved to more easily include all the commits that went into it.


Compare-Object with [char] instances is unexpectedly case-sensitive by default

Prerequisites

[X] Write a descriptive title.
[X] Make sure you are able to repro it on the latest released version
[X] Search the existing issues.
[X] Refer to the FAQ.
[X] Refer to Differences between Windows PowerShell 5.1 and PowerShell.

Steps to reproduce
If [char] instances were consistently compared case-insensitively to each other and to strings, all of the following commands would output 0.
# OK: [string] vs. [string]
(Compare-Object ('a') ('A')).Count
# OK: [string] vs. [char]
(Compare-Object ('a') ([char] 'A')).Count
# !! BROKEN: [char] vs. [char]
(Compare-Object ([char] 'a') ([char] 'A')).Count
# !! BROKEN: [char] vs. [string]
(Compare-Object ([char] 'a') ('A')).Count

Note that [char] 'a' -eq [char] 'A' is $true (even though with -gt, -lt, ... it is the code points that get compared, resulting in case-sensitive behavior).
#12498 is conceptually related, but would not fix this issue, because use of the IEquatable<T> interface would also result in case-sensitive behavior.
Expected behavior
0
0
0
0

Actual behavior
0
0
2
2

Error details
No response
Environment data
PowerShell Core 7.3.0-preview.6

Visuals
No response

When comparing [char] converted into [string]? Compare-Object'sCaseSensitive parameter does not affect the result (and this is true behavior for nonstrings, IMO). But:
[char] 'a' -eq [char] 'A'     # True
[char] 'a' -ceq [char] 'A'    # False

@237dmitry: They're not technically compared as strings with -eq (if memory serves, .ToUpperInvariant() is called before comparing), but the intent is clearly the same: compare [char]s case-INsensitively.
Therefore, Compare-Object, which is also built on equality comparison, should behave the same (with -CaseSensitive acting as opt-out).

Thanks for the clarification.

How do chars compare, as chars or as strings? Compare-Object'sCaseSensitive parameter does not affect the result (and this is expected behavior for non strings, IMO). But:
[char] 'a' -eq [char] 'A'     # True
[char] 'a' -ceq [char] 'A'    # False

I think -eq is always -ieq The char type itself only does case sensitive comparisons
PS>  [char]::Equals(([char]'a'),([char]'A'))
False

But string comparisons are case sensitive by default with the option to select culture and case sensitivity.
PS>  [string]::Equals(([char]'a'),([char]'A'))
False

PS>  [string]::Equals(([char]'a'),([char]'A'), [System.StringComparison]::CurrentCultureIgnoreCase)
True

IEQ might call ToUpper or ToLower on both strings and a chars and compare the result or it might specify the comparison type for strings.    My hunch  is Compare-Object is only requesting insensitive for strings and not for chars.
Because there is only a -CaseSensitive option for  Compare-Object the natural expectation is chars and strings both use case insensitive comparisons by default and only case sensitive when the switch is specified.


feat: remove hlx2 support

Fix #225

Legacy message:

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


Adding non list components out of queue causes list components to not get rendered

Consider the following admin structure
<Admin>
    <Resource name="clients" list={ClientList} edit={ClientEdit} create={ClientCreate} />
    <Resource name="genres" />
    <Resource name="appUsers" list={UserList} edit={AdminUserEdit} />
</Admin>

This when one navigates to the appUsers link, nothing is displayed
However if I do this
<Admin>
    <Resource name="clients" list={ClientList} edit={ClientEdit} create={ClientCreate} />
     <Resource name="appUsers" list={UserList} edit={AdminUserEdit} />
    <Resource name="genres" />
</Admin>

The appUsers page is now rendered and is usable. This is definitely a bug, but can be resolved by adding the above info to the docs.

Really? There is not reason for that. I'm curious to see a failing test case for that bug. Could you create one based on this sandbox?

I can try reproducing at the sandbox, but that will take tomorrow (have to run at the moment)
I have simplified the component greatly in the code above to highlight the issue. However just moving around that 1 resource entry makes the difference between the component rendering or not. I wish I could share the github link with you but its a private repo.

So I couldn't really make it work in the sandbox. But I have some more details from my current codebase.
<Admin>
     <Resource name="clients" list={ClientList} edit={ClientEdit} create={ClientCreate} />
     <Resource name="appUsers" />
     <Resource name="appUsers/usersByRole" list={UserList} edit={AdminUserEdit} />
</Admin>

When a resource with a matching first part of the path is before in the resource queue, AOR is somehow stripping the route by the / and then even though the menu points to the route -> /appUsers/usersByRole
AOR is loading the component for 'appUsers' which does not really exist.

On more investigation I can say that this is because the
exact: true
prop is not being passed to react router in AdminRoutes.js.
A very simple fix would be to make exact: true a default prop to this part of the code in AdminRoutes.js
        {resources.map(resource =>
            <Route
                exact: true
                path={`/${resource.name}`}
                key={resource.name}
                render={() => <CrudRoute
                    resource={resource.name}
                    list={resource.list}
                    create={resource.create}
                    edit={resource.edit}
                    show={resource.show}
                    remove={resource.remove}
                    options={resource.options}
                />}
            />
        )}

Please do advise.

Interestingly enough. Adding exact: true to this comp also results in all CRUD routes to NOT get rendered. So supersimple fix may not be that easy.

Now I understand. That's a limitation of react-router and the way you use it: if you have several resources that start with the same letters, you must place the longest first. I don't consider it as a bug, and this won't be fixed.
As for resources with a / in their name, they are not supported.

As for resources with a / in their name, they are not supported.

I feel that this should be made part of the docs then. I am yet to work at a place where single word rest routes sufficed.


MINOR: [Java] Bump com.google.errorprone:error_prone_core from 2.10.0 to 2.28.0 in /java - Manual Fix

Rationale for this change
Based on the dependabot PR , making correct amendments to reflect the necessary error-prone library upgrade.
What changes are included in this PR?

[X] Upgrade error-prone version for JDK 11+
[X] Keeping the existing error-prone version for JDK 8, and added notes and references as reasoning.

Are these changes tested?
Tested by existing build.
Are there any user-facing changes?
No

@github-actions crossbow submit -g java

@lidavidm I created this manual fix for error-prone upgrade. Also kept the jdk8's version to use the existing one and not go for additional burden in maintaining it.


Remove server telemetry

Motivation
The client side is implemented in https://github.com/Shopify/vscode-ruby-lsp/pull/571.
Doing benchmarks in the server itself is not reliable and doesn't include the entire roundtrip that requests make until they are processed in the editor.
Additionally, benchmarking stuff on the server means it will spend time sending telemetry events to the client for every single request and notification.
Implementation

Stopped sending telemetry/event notifications completely
Started returning error information as part of the request response, which we can read in the client

Automated Tests
Adjusted the related tests for the removed functionality.

Moving to backlog as discussed at team meeting on 2023-06-12.


Searching for text in webpages

This is a Bug Report

Problem:  User experience when searching for text in webpage : https://kubernetes.io/docs/reference/kubectl/quick-reference/
E.g. when searching for rollout word first 6 hits are most probably from the navigation tree and they're not highlighted nor the cursor is moved to the result, making the search experience somehow confusing.

Proposed Solution:
NA
Page to Update:
https://kubernetes.io/docs/reference/kubectl/quick-reference/

/language en
/sig docs

/retitle Lack of visual cues for search matches in left-side menu items using browser find on webpage
/area web-development

/remove-language en

it is the browser feature not a k8s web functionality.
it cannot be resolved.

/remove kind-bug

/remove-kind bug

it is the browser feature not a k8s web functionality. it cannot be resolved. @tamilselvan1102  how did you reach this conclusion?
The issue is reproducible on other browsers e.g. edge

Interesting. @dejanu, there is a solution of hiding the hits using ~magic~ code. But I can see some future complaints rising from that too.

To be quite honest I've heard others complaining too, maybe make that the search results from the page content is shown first, instead of the search hits from the navigation tree

@dejanu
[Ctrl+f] function is common for all the browsers. if do search after expanding the left menu, browser automatically navigate to the keyword.  [Ctrl+f] function we can override but it is not prescribed.

@dejanu
attached image for your reference

A workaround: put the left nav last on the page in terms of its order in the HTML source, but leave it where it is visually.

This might duplicate https://github.com/kubernetes/website/issues/41469

/triage duplicate
/close not-planned
See issue #41469


e2e test ACNPFQDNPolicy is flaky

Describe the bug

=== RUN   TestAntreaPolicy/TestGroupNoK8sNP/Case=ACNPFQDNPolicy
I0818 03:23:51.108737    2096 k8s_util.go:859] Creating/updating ClusterNetworkPolicy test-acnp-drop-all-google
D0818 03:23:51.113984    2096 k8s_util.go:862] Creating ClusterNetworkPolicy test-acnp-drop-all-google
    antreapolicy_test.go:4445: Waiting for ACNP 'test-acnp-drop-all-google' to be realized
I0818 03:23:51.239599    2096 util.go:44] Confirming ready status costs 103.417132ms
T0818 03:23:51.239625    2096 antreapolicy_test.go:3204] Probing: a -> drive.google.com
T0818 03:23:51.239794    2096 k8s_util.go:149] Running: kubectl exec x-9xr9ws07a-55589794c8-vjlsv -c c80 -n x-9xr9ws07 -- /bin/sh -c for i in $(seq 1 3); do /agnhost connect drive.google.com:80 --timeout=1s --protocol=tcp; done;
T0818 03:23:52.027289    2096 k8s_util.go:156] x-9xr9ws07/a -> drive.google.com: error when running command: err - command terminated with exit code 1 /// stdout -  /// stderr - REFUSED
REFUSED
REFUSED

T0818 03:23:52.027384    2096 antreapolicy_test.go:3204] Probing: b -> maps.google.com
T0818 03:23:52.027421    2096 k8s_util.go:149] Running: kubectl exec x-9xr9ws07b-79766db6c8-zdgfc -c c80 -n x-9xr9ws07 -- /bin/sh -c for i in $(seq 1 3); do /agnhost connect maps.google.com:80 --timeout=1s --protocol=tcp; done;
T0818 03:23:52.840628    2096 k8s_util.go:156] x-9xr9ws07/b -> maps.google.com: error when running command: err - command terminated with exit code 1 /// stdout -  /// stderr - REFUSED

    antreapolicy_test.go:3210: failure -- wrong results for probe: Source x-9xr9ws07/b --> Dest maps.google.com:80 connectivity: Err, expected: Rej
T0818 03:23:52.840827    2096 antreapolicy_test.go:3204] Probing: a -> wayfair.com
T0818 03:23:52.840876    2096 k8s_util.go:149] Running: kubectl exec y-9xr9ws07a-57f7bdf8cc-5wzpb -c c80 -n y-9xr9ws07 -- /bin/sh -c for i in $(seq 1 3); do /agnhost connect wayfair.com:80 --timeout=1s --protocol=tcp; done;
T0818 03:23:57.556695    2096 k8s_util.go:156] y-9xr9ws07/a -> wayfair.com: error when running command: err - command terminated with exit code 1 /// stdout -  /// stderr - TIMEOUT
TIMEOUT
TIMEOUT

T0818 03:23:57.556815    2096 antreapolicy_test.go:3204] Probing: b -> facebook.com
T0818 03:23:57.556869    2096 k8s_util.go:149] Running: kubectl exec y-9xr9ws07b-584bfdc78f-s9xkk -c c80 -n y-9xr9ws07 -- /bin/sh -c for i in $(seq 1 3); do /agnhost connect facebook.com:80 --timeout=1s --protocol=tcp; done;
I0818 03:23:58.297803    2096 k8s_util.go:888] Deleting AntreaClusterNetworkPolicies test-acnp-drop-all-google

antrea-test-logs.tar.gz
The above logs are from a build of flexiable IPAM, but I saw this failure in other environment too.

The issue was because the DNS response was so large that the response was truncated and the client sent another query via TCP protocol, see https://en.wikipedia.org/wiki/Domain_Name_System#DNS-over-TCP/53_("Do53/TCP")

DNS-over-TCP/53 ("Do53/TCP")
In 1989, RFC 1123 specified optional Transmission Control Protocol (TCP) transport for DNS queries, replies and, particularly, zone transfers. Via fragmentation of long replies, TCP allows longer responses, reliable delivery, and re-use of long-lived connections between clients and servers.

It can be confirmed by the flag "tc" in the following response I captured from a DNS response, which means the message is truncated.
        ;; opcode: QUERY, status: NOERROR, id: 16000
        ;; flags: qr aa tc rd ra; QUERY: 1, ANSWER: 1, AUTHORITY: 13, ADDITIONAL: 10

        ;; QUESTION SECTION:
        ;drive.google.com.      IN       A

        ;; ANSWER SECTION:
        drive.google.com.       25      IN      A       <IP_ADDRESS>

        ;; AUTHORITY SECTION:
        .       25      IN      NS      i.root-servers.net.
        .       25      IN      NS      f.root-servers.net.
        .       25      IN      NS      l.root-servers.net.
        .       25      IN      NS      a.root-servers.net.
        .       25      IN      NS      j.root-servers.net.
        .       25      IN      NS      b.root-servers.net.
        .       25      IN      NS      m.root-servers.net.
        .       25      IN      NS      e.root-servers.net.
        .       25      IN      NS      d.root-servers.net.
        .       25      IN      NS      k.root-servers.net.
        .       25      IN      NS      g.root-servers.net.
        .       25      IN      NS      c.root-servers.net.
        .       25      IN      NS      h.root-servers.net.

        ;; ADDITIONAL SECTION:
        l.root-servers.net.     25      IN      AAAA    2001:500:9f::42
        c.root-servers.net.     25      IN      AAAA    2001:500:2::c
        k.root-servers.net.     25      IN      AAAA    2001:7fd::1
        d.root-servers.net.     25      IN      A       <IP_ADDRESS>
        d.root-servers.net.     25      IN      AAAA    2001:500:2d::d
        a.root-servers.net.     25      IN      AAAA    2001:503:ba3e::2:30
        e.root-servers.net.     25      IN      AAAA    2001:500:a8::e
        c.root-servers.net.     25      IN      A       <IP_ADDRESS>
        m.root-servers.net.     25      IN      A       <IP_ADDRESS>
        i.root-servers.net.     25      IN      AAAA    2001:7fe::53

I added some code to packet-in TCP/53 response to test Pods and drop them, the logs show there were such traffic captured and test Pods reported the following error, which indicated it indeed queried DNS over TCP.
DNS: lookup drive.google.com on <IP_ADDRESS>:53: dial tcp <IP_ADDRESS>:53: i/o timeout
DNS: lookup drive.google.com on <IP_ADDRESS>:53: dial tcp <IP_ADDRESS>:53: i/o timeout
DNS: lookup drive.google.com on <IP_ADDRESS>:53: dial tcp <IP_ADDRESS>:53: i/o timeout

Because antrea-agent didn't inspect DNS over TCP traffic, it may miss the IPs resolved via TCP response and cause the test traffic bypass the policy.
As a workaround, perhaps disabling IPv6 could reduce the size of the response so clients won't need to use TCP and the test flake can be avoided.
In long term, we may need to inspect DNS over TCP traffic as well, or see if there is a way to disable DNS over TCP in a cluster.


Should we throw an exception for TaskSeq.zip when the sequences are of unequal length?

While writing tests for #31, I started pondering this ☝.
For comparison, F# throws an ArgumentException for Array.zip and List.zip, but not for Seq.zip. Most likely, this is because "getting the next item" can have an unwanted side effect, so instead, they choose to stop processing as soon as one of the sequences is exhausted.
Arguably, such side effect can still happen. I.e., for zip we have to call MoveNextAsync, which will give us a bool that tells us whether we reached the end of the sequence. But, if the first sequence still has items, but the second sequence is exhausted, the side effect of the first sequence for "one item past the last" has already happened.
In fact, I think the Seq.zip in F# should probably come with a warning. What if each item is "getting a web page of several megabytes"? AFAIK, there's no peek function that would potentially tell me, without side effects, that a sequence is exhausted.
Edit: linking the discussion in the F# repo: https://github.com/dotnet/fsharp/issues/14121

Of relevance, for infinite sequences, this example is relevant: https://stackoverflow.com/questions/7226004/seq-zip-in-f-may-require-seemingly-extra-sequence-element-to-complete.
Basically: if you have an infinite sequence, and you have a filter that returns only two elements of the infinite set, then, combining that with another sequence using zip, map2, iter2 and their variants, will always blow up!
The cause being: we need to "peek", and (at least one of) the sequence(s) will try to MoveNext, but since no other element in the infinite sequence yields a proper result, we will run forever.
Currently, the way this is implemented in F# is such that it is deterministic: if one of the two sequences exhibits this behavior (i.e. MoveNext not finding the "next" item, ever), the combination of the two will run forever.

/cc @gusty, what would you think? I tend to raise an exception, as we do know whether there is a "next" item and it feels like this was an oversight in F# Core. But at the same time, parity is a good thing as well.

I would definitely align with the mechanism for Seq.zip.
IMHO the fact that it hangs in that specific situation is a corner case, which normally is not encountered in the wild, as doing a filter like that is an unsafe operation.

I’m considering adding a shadowed function set that can be opened, that have an overload that takes a CancelationToken. This would not be specifically for infinite sequences, but everything here is async, so it makes sense to make them cancelable.
I’m still a little on the fence about parity with F#. They made a mistake (IMO) by not having the same behaviour for all collection types (where that makes sense and doesn’t require potential extra side effects). But there’s also the “principle of least surprise”. Hmm, yeah, ultimately I guess I have to bite the bullet and go for parity.

I agree in that they made a mistake, but to me the mistake was on List and Array, while seq has the right behavior.
Of interest, in F#+ there is *.zipShortest defined for List and arrays.

I agree with you there. I mean: the different collections should have parity in behaviour. Bit if you can get away without exceptions it’s generally better, but then that should have applied to all of them.

This has now been solved in #102.


drymartini lt file issues

Hello!
I've been playing around with the dry martini model incldued with moltemplate - I had an error that certain atom masses and atom types were undefined. It seems like this issue cropped up especially surrounding the small bead naming schemes. In particular, throughout the lt file there were references to SP4_bSP4_aSP4_dSP4_iSP4, but the replace command read replace{ @atom:SP4 @atom:SP4_SP4_aP4_dP4_iP4} missing S's before the P's in the longer name causing moltemplate to think there weren't masses defined for these atoms.
There was also one bond coeff not defined - P1-C1, I wasn't sure whether that should be defined or not - I added a coeff for it and marked the location (line  828). The coeff I chose was 1.49375 4.8 (which I think is correct).
I've gone through and corrected (I think) all of these issues - I don't make any promises that anything else in the file is correct. (I marked things I changed with comments). Moltemplate runs with dry martini now at least with these changes.
I've attached the new lt file to this issue report (I had to swap the extension to get github to accept it). Let me know if you have any questions, I hope this helps!
Best,
Zeke
drymartini.txt

Hello,
Sorry for the second message but I found another issue I think. It seems like the pair coefficient sigmas in the moltemplate force field file for dry martini is still in units of nm rather than Angstroms (like is used for units real in lammps and defined at the bottom of the force field). If I manage to make a corrected version with this change, I'll post it here.

Hi Zeke!
The "drymartini.lt" file was created by Daniel Stelter back when he was a postdoc in Tom Keyes' lab (also at BU).  I went ahead and included his version of "drymartini.lt" with moltemplate, but  I haven't used MARTINI or DRYMARTINI yet.  So I rely on people like you to find the problems with these files, and I'm very grateful.
I am a little bit embarassed by the sorry state of the MARTINI (and DRYMARTINI) examples.
In the original ITP files posted on the cgmartini.nl website, each molecule type (for example, each type of lipid) comes with a completely independent list of bonds and angles.  In the files from the cgmartini.nl web site, the @angle type is not one-to-one (not uniquely determined) the @atom types.  Most of the time, the type of angle used (@angle) can be determined uniquely by the types of the 3 atoms (the three @atoms).
Unfortunately, instead of converting those files, instead we copied our MARTINI and DRYMARTINI force field parameters from the "PRM" files that are distributed with the EMC software.  The EMC software requires that angles are looked up according to atom types.  So the @angle must be one-to-one with the @atom types.  Whenever EMC discovers a new molecule or lipid type where the same atom types being associated with a different @angle type, it creates a new @atom type, which is only used in that kind of molecule.  This only happens in a minority of the lipid types, but it is an issue you need to be aware of.
Lately, I've been thinking about starting over, and writing a converter that converts the simple ITP files that are on the cgmartini web site (for example, download the ZIP file for DRYMARTINI here.  (I feel bad doing that because I know David Stelter put a lot of work into his EMC converter tool.)  But the ITP file format used at cgmartini.nl is simple to understand and elegant.  (At least, the ITP files for MARTINI and DRYMARTINI are simple.  I don't know if GROMOS files are more complicated.)  In the long run, I think writing a simple ITP-to-Moltemplate converter would actually be easier than trying to figure how to fix the problems with the "martini.lt" and "drymartini.lt" files derived from EMC (and figure out what EMC did to the atom type names).  And we could get the force field parameters straight from the source (cgmartini.nl), and update them to the latest version of MARTINI.  (The new Martini 3.0 parameters just came out a month or so ago, but "martini.lt" is still using version 2.1.)  Perhaps I should remove all of the MARTINI and DRYMARTINI files and examples from the moltemplate distribution until this is done.
If you plan to use DRYMARTINI or MARTINI in LAMMPS have any time to devote to this, I would probably spend it doing writing that conversion tool.  It will probably be 4 or 5 months before I can devote any time to this.
If that's too much work for you, but you find a way to edit the existing "drymartini.lt" file so that it works for your purposes, let me know and at least we can post that.
Thanks again for your post!
Andrew

Andrew,
Thanks for the detailed reply! I’ve actually been chatting with David a bit already because I am working with gREMD of lipid bilayers which he worked on a bit.
I actually already wrote a code that starts to read some of the MARTINI files to extract and build a moltemplate lt file for individual lipids, so hopefully it won’t be too hard to actually go the rest of the way and build an overall code like you are describing to actually build the rest of the force field file. I’ll work on it a bit and I will reach out to you again you once I have it!
Thanks again for the response!
Zeke
On Jun 25, 2021, at 4:09 AM, Andrew Jewett<EMAIL_ADDRESS>wrote:
Hi Zeke!
The "drymartini.lt" file was created by Daniel Stelter back when he was a postdoc in Tom Keyes' lab (also at BU). I went ahead and included his version of "drymartini.lt" with moltemplate, but I haven't used MARTINI or DRYMARTINI yet. So I rely on people like you to find the problems with these files, and I'm very grateful.
I am a little bit embarassed by the sorry state of the MARTINI (and DRYMARTINI) examples.
In the original ITP files posted on the cgmartini.nlhttp://cgmartini.nl website, each molecule type (for example, each type of lipid) comes with a completely independent list of bonds and angles. In the files from the cgmartini.nlhttp://cgmartini.nl web site, the @Anglehttps://nam10.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FAngle&data=04|01|piskuliche%40ku.edu|c65b812109014faa3b6908d937b0984d|3c176536afe643f5b96636feabbe3c1a|0|0|637602053900635063|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=QJUwFZrOAlYW7AquIgC1ffL4AJ4QE0C%2BQWiNWsHsZqk%3D&reserved=0 type is not one-to-one (not uniquely determined) the @atomhttps://nam10.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fatom&data=04|01|piskuliche%40ku.edu|c65b812109014faa3b6908d937b0984d|3c176536afe643f5b96636feabbe3c1a|0|0|637602053900635063|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=XQUyaEqII%2FLREmd37fA%2FTr2y0xELDZbToc12xMeutBw%3D&reserved=0 types. Most of the time, the type of angle used (@Anglehttps://nam10.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FAngle&data=04|01|piskuliche%40ku.edu|c65b812109014faa3b6908d937b0984d|3c176536afe643f5b96636feabbe3c1a|0|0|637602053900645027|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=H4hHmsInPFXrEAOvYbqpIgpuqfoykIzeZHkF1xMMCEE%3D&reserved=0) can be determined uniquely by the types of the 3 atoms (the three @Atomshttps://nam10.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FAtoms&data=04|01|piskuliche%40ku.edu|c65b812109014faa3b6908d937b0984d|3c176536afe643f5b96636feabbe3c1a|0|0|637602053900645027|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=Tu0HuhD7opgeCprNhsAgqJTGgPLXbg3GDlMNCKTIrjA%3D&reserved=0).
Unfortunately, instead of converting those files, instead we copied our MARTINI and DRYMARTINI force field parameters from the "PRM" files that are distributed with the EMC software. The EMC software requires that angles are looked up according to atom types. So the @Anglehttps://nam10.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FAngle&data=04|01|piskuliche%40ku.edu|c65b812109014faa3b6908d937b0984d|3c176536afe643f5b96636feabbe3c1a|0|0|637602053900645027|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=H4hHmsInPFXrEAOvYbqpIgpuqfoykIzeZHkF1xMMCEE%3D&reserved=0 must be one-to-one with the @atomhttps://nam10.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fatom&data=04|01|piskuliche%40ku.edu|c65b812109014faa3b6908d937b0984d|3c176536afe643f5b96636feabbe3c1a|0|0|637602053900654986|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=OHx73QsgFyyWZQiZ9jgzRgM6AImdVVL1jgcPgwlfxE8%3D&reserved=0 types. Whenever EMC discovers a new molecule or lipid type where the same atom types being associated with a different @Anglehttps://nam10.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FAngle&data=04|01|piskuliche%40ku.edu|c65b812109014faa3b6908d937b0984d|3c176536afe643f5b96636feabbe3c1a|0|0|637602053900654986|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=NIqF7%2FgQKxbK3LZypVB1clKB0IBpfzRUZsLY8ALUfW0%3D&reserved=0 type, it creates a new @atomhttps://nam10.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fatom&data=04|01|piskuliche%40ku.edu|c65b812109014faa3b6908d937b0984d|3c176536afe643f5b96636feabbe3c1a|0|0|637602053900664939|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=0wWWEpPtUY%2Fpbg3kSHsTg7ENEztxSsPtld41TY3X%2B1E%3D&reserved=0 type, which is only used in that kind of molecule. This only happens in a minority of the lipid types, but it is an issue you need to be aware of.
Lately, I've been thinking about starting over, and writing a converter that converts the simple ITP files that are on the cgmartini web site (for example, download the ZIP filehttps://nam10.safelinks.protection.outlook.com/?url=http%3A%2F%2Fcgmartini.nl%2Fimages%2Fparameters%2Fdry_martini%2F20160523%2FDry_Martini_v2.1.zip&data=04|01|piskuliche%40ku.edu|c65b812109014faa3b6908d937b0984d|3c176536afe643f5b96636feabbe3c1a|0|0|637602053900664939|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=XyjuxGY89Zrb8qYrZc0hAwXmBqLtydzG429vCeKrWGE%3D&reserved=0 for DRYMARTINI herehttps://nam10.safelinks.protection.outlook.com/?url=http%3A%2F%2Fcgmartini.nl%2Findex.php%2Fforce-field-parameters%2Fdry-martini&data=04|01|piskuliche%40ku.edu|c65b812109014faa3b6908d937b0984d|3c176536afe643f5b96636feabbe3c1a|0|0|637602053900664939|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=gjMhE%2FAFbvk3ZM64Ym7S1L8MKMuiGS8Q57%2BE0Z%2F%2B4%2BM%3D&reserved=0. (I feel bad doing that because I know David Stelter put a lot of work into his EMC converter tool.) But the ITP file format used at cgmartini.nlhttp://cgmartini.nl is simple to understand and elegant. (At least, the ITP files for MARTINI and DRYMARTINI are simple. I don't know if GROMOS files are more complicated.) In the long run, I think writing a simple ITP-to-Moltemplate converter would actually be easier than trying to figure how to fix the problems with the "martini.lt" and "drymartini.lt" files derived from EMC (and figure out what EMC did to the atom type names). And we could get the force field parameters straight from the source (cgmartini.nlhttp://cgmartini.nl), and update them to the latest version of MARTINI. (The new Martini 3.0 parameters just came out a month or so ago, but "martini.lt" is still using version 2.1.) Perhaps I should remove all of the MARTINI and DRYMARTINI files and examples from the moltemplate distribution until this is done.
If you plan to use DRYMARTINI or MARTINI in LAMMPS have any time to devote to this, I would probably spend it doing writing that conversion tool. It will probably be 4 or 5 months before I can devote any time to this.
If that's too much work for you, but you find a way to edit the existing "drymartini.lt" file so that it works for your purposes, let me know and at least we can post that.
Thanks again for your post!
Andrew
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://nam10.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fjewettaij%2Fmoltemplate%2Fissues%2F51%23issuecomment-868309019&data=04|01|piskuliche%40ku.edu|c65b812109014faa3b6908d937b0984d|3c176536afe643f5b96636feabbe3c1a|0|0|637602053900674896|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=YWu1aYXgjANso5lKACrvt3r2CKUpOhJB3FFneeaCIDM%3D&reserved=0, or unsubscribehttps://nam10.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAEXNNMZ3AXU3XTSZR3V6U23TUQ2UTANCNFSM47CDYNCQ&data=04|01|piskuliche%40ku.edu|c65b812109014faa3b6908d937b0984d|3c176536afe643f5b96636feabbe3c1a|0|0|637602053900674896|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=pCMTzk6c6yRb8bw%2B35aQ7TlIcj0V877yEF5KcNK%2BgfQ%3D&reserved=0.

Cool!  Glad to hear you are working with David.  (not Daniel)  I think the gREMD method is underappreciated.  And I'm also glad to hear you are writing a converter.  I would love to include that with moltemplate when it's ready, if that's okay with you.  (Along with the lipid files.)
Thanks for posting.  I will change the title of this post to "drymartini and martini issues".


Location override

Expected Behavior
I edited my son file location to a new location and I expected my location to be sent there.
However, I get this message:
[13:49:44] [x] Parsing cached location... [13:49:44]  [13:49:44] [x] Last location flag used. Overriding passed in location [13:49:44] [x] Last in-game location was set as: (37.76695739625344, -122.42211926362825, 0) [13:49:44]
To fix this i set my location cache variable to false and I just got soft-banned. Is this the right way to prevent the override is there a better method?

You need to give it time if you are jumping around locations. The larger the distance, the longer you need to wait.
In the data folder is a file called last-location-username.json. Delete that file and it will place you at your config's location

Behavior is correct, albeit confusing and maybe could use a y/n confirmation.
Expected behavior:
Location in CLI > Cache in config > location in config

Agree with @CapCap, although I'm seeing the cached location take precedence over the CLI. The CLI should always have precedence.


packer/build_deb_image: Improve error handling

Fix success verification
Error when missing target parameter

Tested: https://jenkins.scylladb.com/job/scylla-master/job/releng-testing/job/gce-image/479/ OK
I also tested it with missing target parameter (but build was already erased).


[clone_app.py] code refinement and testing

clone_app.py contains a single huge function, clone.
This is really hard for writting unittests. There is tooo many logcial blocks in it.
So I will try to split it into several smaller functions if possible.
I just took a glance at the code. I will list what I want to do continuously.

[ ] Replace _replace_slash function with os.path.normpath
[ ] Separate logical block: create files from manifest

I decide to rewrite function clone into a callable class.
Because i just found that split the logical blocks directly will encounter some problem: they have dependencies and share lots of vars.
Assume our clone function looks like this:
def clone(...):
    # logical block A
    # We init some vars here.

    # logical block B
    # some mutable state: modify vars from block A
    # Or create some new vars

    # logical block C
    # using the vars from A & B

So.... If we split those blocks as functions, we need to do large amount of vars passing.
I'm considering using callable class to solve this, like:
class clone:
    def block_A():
        ...
    def block_B():
        # we can access those shared vars via self.x
    def block_C():
        ...
    def __call__():
        self.block_A()
        self.block_B()
        ...
        return something

And, i need more suggestions. Please comment!


java.lang.NoClassDefFoundError: net/dv8tion/jda/JDABuilder

I have the jar implemented in my project... But why would it say this? java.lang.NoClassDefFoundError: net/dv8tion/jda/JDABuilder

Caused by JDA (and most likely its dependencies) not being present in the built jar. Try using something like Maven's Shade plugin or Gradle's ShadowJar plugin.


Distributing development versions

Is your feature request related to a problem? Please describe.
I want to make development versions of Files feedback, but I couldn't build it because I haven't used VS.
Describe the solution you'd like
Please distribute development versions.
Describe alternatives you've considered
I'm sorry, I couldn't find it.
Additional context
N/A

@pklion You can join the insider flight group by filling out this form.

Thank you!


Create a FAQ page

Adding it on docs will be better sollution

@tachyons @nitin10s can close this. Since, currently, we are not planning to add an FAQ page or are we?

Basically no activity, no elaboration. the bot will close it in 2 days


can't make react-tap-event-plugin to work in my typescript react environement

Im trying to work with material-ui and react-tap-event-plugin I am getting this error when the app is loaded:
react-dom.js:18238 Warning: Unknown proponTouchTapon <button> tag. Remove this prop from the element. For details, see https://....
    in button (created by EnhancedButton)
    in EnhancedButton (created by IconButton)
    in IconButton (created by AppBar)
    in div (created by Paper)
    in Paper (created by AppBar)
    in AppBar (created by AppLayout)
    in div (created by AppLayout)
    in div (created by AppLayout)
    in AppLayout
    in MuiThemeProvider

I can't just ignore the error, the events are not working at all.
this is my index.tsx file:
import * as injectTapEventPlugin from 'react-tap-event-plugin';

injectTapEventPlugin();//I am able to get into this in the js debugger

ReactDOM.render(....)
This is my index.html file:

this is my tsconfig:
{
  "compilerOptions": {
    "outDir": "./dist/",

    "allowJs": true,
    "sourceMap": true,
    "noImplicitAny": false,
    "noEmit": true,
    "baseUrl": ".",
    "suppressImplicitAnyIndexErrors": true,
    "module": "commonjs",
    "target": "es5",
    "jsx": "react"
    },
  "include": [
    "./src/**/*",
  ],
    "exclude": [
      "node_modules" // don't run on any code in the node_modules directory
    ]
  }

package.json:
{
  "name": "affiliate_portal_client",
  "version": "1.0.0",
  "description": "",
  "main": "index.js",
  "scripts": {
    "test": "echo \"Error: no test specified\" && exit 1",
    "watch": "webpack"
  },
  "author": "",
  "license": "ISC",
  "dependencies": {
    "@types/react": "^15.0.25",
    "@types/react-dom": "^15.5.0",
    "react": "^15.5.4",
    "react-addons-css-transition-group": "^15.5.2",
    "react-dom": "^15.5.4",
    "react-tap-event-plugin": "^2.0.1",
    "material-ui": "^0.18.1"
  },
  "devDependencies": {
    "@types/react-tap-event-plugin": "0.0.30",
    "awesome-typescript-loader": "^3.1.3",
    "source-map-loader": "^0.2.1",
    "typescript": "^2.3.3"
  }
}

I don't think its related to typescript, the function injectTapEventPlugin is invoked, i see it in the debugger...
I read all the possibly posts about that. top to botton, non of them are really working.
Here is a link to the source code: https://bitbucket.org/avifatal/public-portal/src/047d385d6c0b69078b62065f9f3fe00013e4acfe/AffiliatePortal.Client.MVC/src/?at=master
Thanks

Sorry, I'm not sure we can provide more help here. We have removed the react-tap-event-plugin on the next branch, I'm closing the issue for now.


Code Coverage html view in Azure Devops is not styled

The 'Code Coverage' view in Azure Devops is not styled because the html cannot import the styles files. This is a known limitation of Azure Devops: https://visualstudio.uservoice.com/forums/330519-visual-studio-team-services/suggestions/15926806-code-coverage-report-on-build-should-not-strip-sty
Down the road we can try solving this but it seems out of scope and something either Istanbul or Azure Devops should fix.

Maybe a solution: https://davidsekar.com/aspnetcore/code-coverage-html-reports-are-missing-styles-in-vsts


Scala 2.10.6 Crossbuild missing

A crossbuild for Scala 2.10.5 exists but not 2.10.6

This is fixed in master, should go out in 0.5.1


Compatibility with symfony/mime package upgrade

Fixes #27

Thanks @fosron for looking into it.
For backward compatibility, can your team work on it as it breaks one of our production servers.
Regards,

@zeshan77 the message you posted is a warning, not an error, that should not cause a problem. We are not going to fix it at this time, but later down the line.


0.0.97 'npm run scully:serve' not serving any page other than home.

🐞 Bug report
Really sorry for being a pain. but the latest release 0.0.97 only serves the homepage on the Angular app when doing npm run scully:serve.
It looks like a 404 option has been added, but by default (without setting this 404 to index) a basic Angular route (with no params) in the app is not served and therefore an error is created in the static page.
Description
🔬 Minimal Reproduction
💻Your Environment
Angular Version:

9

Scully Version:

0.0.97

🔥 Exception or Error

Route "/faq" not provided by angular app

@etavener This should be fixed in the latest released version, 0.0.98.

Yep, just tried it. I will close this issue.


websocket_refactor branch error upon make run

The npm error below was encountered:
make run
sudo -H pip3 install -r ~/tellina_task_interface/requirements.txt
Requirement already satisfied (use --upgrade to upgrade): Django==1.10.4 in /usr/local/lib/python3.5/dist-packages (from -r /home/vagrant/tellina_task_interface/requirements.txt (line 1))
Requirement already satisfied (use --upgrade to upgrade): docker-py==1.10.6 in /usr/local/lib/python3.5/dist-packages (from -r /home/vagrant/tellina_task_interface/requirements.txt (line 2))
Requirement already satisfied (use --upgrade to upgrade): django-extensions==1.7.5 in /usr/local/lib/python3.5/dist-packages (from -r /home/vagrant/tellina_task_interface/requirements.txt (line 3))
Requirement already satisfied (use --upgrade to upgrade): websocket-client>=0.32.0 in /usr/local/lib/python3.5/dist-packages (from docker-py==1.10.6->-r /home/vagrant/tellina_task_interface/requirements.txt (line 2))
Requirement already satisfied (use --upgrade to upgrade): docker-pycreds>=0.2.1 in /usr/local/lib/python3.5/dist-packages (from docker-py==1.10.6->-r /home/vagrant/tellina_task_interface/requirements.txt (line 2))
Requirement already satisfied (use --upgrade to upgrade): requests!=2.11.0,>=2.5.2 in /usr/lib/python3/dist-packages (from docker-py==1.10.6->-r /home/vagrant/tellina_task_interface/requirements.txt (line 2))
Requirement already satisfied (use --upgrade to upgrade): six>=1.4.0 in /usr/lib/python3/dist-packages (from docker-py==1.10.6->-r /home/vagrant/tellina_task_interface/requirements.txt (line 2))
You are using pip version 8.1.1, however version 9.0.1 is available.
You should consider upgrading via the 'pip install --upgrade pip' command.
sudo docker build -t proxy proxy_image
Sending build context to Docker daemon  5.12 kB
Step 1 : FROM node:6-onbuild
# Executing 5 build triggers...
Step 1 : ARG NODE_ENV
 ---> Using cache
Step 1 : ENV NODE_ENV $NODE_ENV
 ---> Using cache
Step 1 : COPY package.json /usr/src/app/
 ---> Using cache
Step 1 : RUN npm install
 ---> Running in 001414e203bc
npm info it worked if it ends with ok
npm info using<EMAIL_ADDRESS>npm info using<EMAIL_ADDRESS>npm info attempt registry request try #1 at 1:43:40 PM
npm http request GET https://registry.npmjs.org/ws
npm info attempt registry request try #1 at 1:43:40 PM
npm http request GET https://registry.npmjs.org/pty.js
npm http 200 https://registry.npmjs.org/pty.js
npm http 200 https://registry.npmjs.org/ws
npm info retry fetch attempt 1 at 1:43:40 PM
npm info attempt registry request try #1 at 1:43:40 PM
npm http fetch GET https://registry.npmjs.org/pty.js/-/pty.js-0.3.1.tgz
npm info retry fetch attempt 1 at 1:43:40 PM
npm info attempt registry request try #1 at 1:43:40 PM
npm http fetch GET https://registry.npmjs.org/ws/-/ws-1.1.1.tgz
npm http fetch 200 https://registry.npmjs.org/pty.js/-/pty.js-0.3.1.tgz
npm http fetch 200 https://registry.npmjs.org/ws/-/ws-1.1.1.tgz
npm info attempt registry request try #1 at 1:43:41 PM
npm http request GET https://registry.npmjs.org/extend
npm info attempt registry request try #1 at 1:43:41 PM
npm http request GET https://registry.npmjs.org/nan
npm http 200 https://registry.npmjs.org/extend
npm info retry fetch attempt 1 at 1:43:41 PM
npm info attempt registry request try #1 at 1:43:41 PM
npm http fetch GET https://registry.npmjs.org/extend/-/extend-1.2.1.tgz
npm http 200 https://registry.npmjs.org/nan
npm info retry fetch attempt 1 at 1:43:41 PM
npm info attempt registry request try #1 at 1:43:41 PM
npm http fetch GET https://registry.npmjs.org/nan/-/nan-2.3.5.tgz
npm http fetch 200 https://registry.npmjs.org/extend/-/extend-1.2.1.tgz
npm http fetch 200 https://registry.npmjs.org/nan/-/nan-2.3.5.tgz
npm info attempt registry request try #1 at 1:43:41 PM
npm http request GET https://registry.npmjs.org/options
npm info attempt registry request try #1 at 1:43:41 PM
npm http request GET https://registry.npmjs.org/ultron
npm http 200 https://registry.npmjs.org/options
npm http 200 https://registry.npmjs.org/ultron
npm info retry fetch attempt 1 at 1:43:41 PM
npm info attempt registry request try #1 at 1:43:41 PM
npm http fetch GET https://registry.npmjs.org/options/-/options-0.0.6.tgz
npm info retry fetch attempt 1 at 1:43:41 PM
npm info attempt registry request try #1 at 1:43:41 PM
npm http fetch GET https://registry.npmjs.org/ultron/-/ultron-1.0.2.tgz
npm http fetch 200 https://registry.npmjs.org/options/-/options-0.0.6.tgz
npm http fetch 200 https://registry.npmjs.org/ultron/-/ultron-1.0.2.tgz
npm info lifecycle undefined~preinstall: undefined
npm info lifecycle<EMAIL_ADDRESS><EMAIL_ADDRESS>npm info lifecycle<EMAIL_ADDRESS><EMAIL_ADDRESS>npm info lifecycle<EMAIL_ADDRESS><EMAIL_ADDRESS>npm info lifecycle<EMAIL_ADDRESS><EMAIL_ADDRESS>npm info lifecycle<EMAIL_ADDRESS><EMAIL_ADDRESS>npm info lifecycle<EMAIL_ADDRESS><EMAIL_ADDRESS>npm info linkStuff<EMAIL_ADDRESS>npm info linkStuff<EMAIL_ADDRESS>npm info linkStuff<EMAIL_ADDRESS>npm info linkStuff<EMAIL_ADDRESS>npm info linkStuff<EMAIL_ADDRESS>npm info linkStuff<EMAIL_ADDRESS>npm info lifecycle<EMAIL_ADDRESS><EMAIL_ADDRESS>npm info lifecycle<EMAIL_ADDRESS><EMAIL_ADDRESS>npm info lifecycle<EMAIL_ADDRESS><EMAIL_ADDRESS>npm info lifecycle<EMAIL_ADDRESS><EMAIL_ADDRESS>npm info lifecycle<EMAIL_ADDRESS><EMAIL_ADDRESS>
><EMAIL_ADDRESS>install /usr/src/app/node_modules/pty.js
> node-gyp rebuild

gyp info it worked if it ends with ok
gyp info using<EMAIL_ADDRESS>gyp info using<EMAIL_ADDRESS>| linux | x64
gyp http GET https://nodejs.org/download/release/v6.9.2/node-v6.9.2-headers.tar.gz
gyp http 200 https://nodejs.org/download/release/v6.9.2/node-v6.9.2-headers.tar.gz
gyp WARN install got an error, rolling back install
gyp ERR! configure error
gyp ERR! stack Error: read ECONNRESET
gyp ERR! stack     at exports._errnoException (util.js:1022:11)
gyp ERR! stack     at TLSWrap.onread (net.js:569:26)
gyp ERR! System Linux 4.4.0-51-generic
gyp ERR! command "/usr/local/bin/node" "/usr/local/lib/node_modules/npm/node_modules/node-gyp/bin/node-gyp.js" "rebuild"
gyp ERR! cwd /usr/src/app/node_modules/pty.js
gyp ERR! node -v v6.9.2
gyp ERR! node-gyp -v v3.4.0
gyp ERR! not ok
npm info lifecycle<EMAIL_ADDRESS>Failed to exec install script
/usr/src/app
`--<EMAIL_ADDRESS>  +--<EMAIL_ADDRESS>  `--<EMAIL_ADDRESS>
npm WARN app No description
npm WARN app No repository field.
npm WARN app No license field.
npm ERR! Linux 4.4.0-51-generic
npm ERR! argv "/usr/local/bin/node" "/usr/local/bin/npm" "install"
npm ERR! node v6.9.2
npm ERR! npm  v3.10.9
npm ERR! code ELIFECYCLE

npm ERR<EMAIL_ADDRESS>install: `node-gyp rebuild`
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>install script 'node-gyp rebuild'.
npm ERR! Make sure you have the latest version of node.js and npm installed.
npm ERR! If you do, this is most likely a problem with the pty.js package,
npm ERR! not with npm itself.
npm ERR! Tell the author that this fails on your system:
npm ERR!     node-gyp rebuild
npm ERR! You can get information on how to open an issue for this project with:
npm ERR!     npm bugs pty.js
npm ERR! Or if that isn't available, you can get their info via:
npm ERR!     npm owner ls pty.js
npm ERR! There is likely additional logging output above.

npm ERR! Please include the following file with any support request:
npm ERR!     /usr/src/app/npm-debug.log
The command '/bin/sh -c npm install' returned a non-zero code: 1
Makefile:19: recipe for target 'build_images' failed
make: *** [build_images] Error 1

TLDR:

Seems like a download issue, try make run again.
If it still fails, try vagrant destroy -f && vagrant up and make run

The key part of the error is:
sudo docker build -t proxy proxy_image
...
gyp http GET https://nodejs.org/download/release/v6.9.2/node-v6.9.2-headers.tar.gz
gyp http 200 https://nodejs.org/download/release/v6.9.2/node-v6.9.2-headers.tar.gz
gyp WARN install got an error, rolling back install
gyp ERR! configure error
gyp ERR! stack Error: read ECONNRESET
gyp ERR! stack     at exports._errnoException (util.js:1022:11)
gyp ERR! stack     at TLSWrap.onread (net.js:569:26)

When Docker builds proxy_image/Dockerfile, it runs npm install which pings https://nodejs.org to install Node.js packages. Looks like one of those download calls failed (ECONNRESET) and a dependency didn't get installed, so the Docker build failed.

@todpole3 Are you still running into this issue?

@kvu787 It's gone. Thanks!

This happened again today.
Restart vagrant is not effective.

The error is suppressed by using a different node image: replace FROM node:6-onbuild with FROM nodesource/trusty:5.1 in proxy_image/Dockerfile and backend_container_image/Dockerfile.


Fix docker-compose start up order

I found that last PR #216 have some problem. If CRM container be ready faster than DB container, build by docker-compose will failed.
db.env is for storing shared database environment variables
wait-for-postgres.sh is for waiting DB container ready to connect

Coverage remained the same at 69.846% when pulling a826e5d4180cd01e3f9b649d4058e205f8d7bd5a on jtr860830:master into e4f59edfc1480912c267a3017672e9004b3e7699 on MicroPyramid:master.


Help

how allow access to all users?

https://github.com/ciuzaak/Claude-Telegram-Bot/blob/f9a2495b375c7052a291062a34562f2aae64335d/bot.py#L380
Change to:
user_filter = filters.CHAT

But I wouldn't recommend doing that, because both Claude and Bard have rate limits, and their usage by multiple people could potentially cause conflicts. 😓

Thanks


How can I get the general date format

When I parse the file, I will get different time formats, such as excel time or others.
Is there a way to get a unified format that I expect?

Thanks for your feedback. This issue is similar with #62, #660, #675, #759, #1055, #1258. The Excelize not support to parse all the number formats of the cell currently. This feature are working in progress. As the documentation says, if the cell format can be applied to the value of a cell, the applied value will be returned, otherwise the original value will be returned. I think this can be close this and merge it into issue #660 is still discussing this feature.

Hi @d630200x, I closed this issue. If you have any questions, please let me know to reopen this anytime.


Parse mass from object's geophysical properties

Property table is probably the most inconsistent bit of the API.

not going to happen


[Module 2 Bug] Fixing wrong args passed to DPOTrainer

Notes:

Wrong args were passed to DPOTrainer which had to be passed for DPOConfig.

Also, For me report_to='none' worked instead of report_to=None,. Should I make that change as well ? @burtenshaw

Partially it is fixed by #120 by revoking those properties, this PR would move them to config


Added a Function for DocArrayInMemorySearch

Hi @Sammybams,
I have implemented a function that processes the results from the extracted contents using Azure Document Intelligence. The function performs the following tasks:

Chunking: Uses the previously implemented chunking function to split the extracted content into manageable pieces.
Embedding: Leverages the OpenAI embedding model to create embeddings for the chunked content.
Vector Store: Stores the resulting embeddings in a DocArrayInMemorySearch vector store.

Additionally, I have included a resource that helped guide my implementation. Could you please review the code and provide feedback? I would appreciate any suggestions or improvements.
Thank you!

Hi @Sammybams This is how it works. I had to change the initialization of the open ai embedding model from OpenAIEmbeddings to AzureOpenAIEmbeddings the documentation has been updated so to mitigate the error of resource not found when calling the API key, the update had to be done.

Hi @Sammybams This is how it works. I had to change the initialization of the open ai embedding model from OpenAIEmbeddings to AzureOpenAIEmbeddings the documentation has been updated so to mitigate the error of resource not found when calling the API key, the update had to be done.

Hi @Odeyiany2,
Kindly confirm what version of openai you are using. You shouldn't have that issue.

Hi @Sammybams I used the versions provided in the requirements.txt file and it worked smoothly. The versions I used previously were higher than what was expected which made to make the previous changes. I have made the changes and it still worked. Thank you.


What permissions are needed to use the Active Directory - password option?

Note
Issues in this repo are for tracking bugs, feature requests and questions for the tasks in this repo
For a list:
https://github.com/Microsoft/azure-pipelines-tasks/tree/master/Tasks
If you have an issue or request for the Azure Pipelines service, use developer community instead:
https://developercommunity.visualstudio.com/spaces/21/index.html )
Required Information
Entering this information will route you directly to the right team and expedite traction.
Question, Bug, or Feature?
Type: Question
Enter Task Name: Azure SQL Database deployment
list here (V# not needed):
https://github.com/Microsoft/azure-pipelines-tasks/tree/master/Tasks
Environment

Server - Azure Pipelines or TFS on-premises?

If using TFS on-premises, provide the version:

If using Azure Pipelines, provide the account name, team project name, build definition name/build number: internal account, please follow-up for these details.

Agent - Hosted or Private:

If using Hosted agent, provide agent queue name: Hosted on Azure Pipelines agents

If using private agent, provide the OS of the machine running the agent and the agent version:

Issue Description
I am trying to enable the Active Directory-Password option on my Azure SQL Database deployment task in my release pipeline. I'm attempting to do this using an Azure AD App Registration. I am having failures in deploying, and I think it's because my AAD App doesn't have all of the permissions it needs. What permissions are needed in order for my AAD app to deploy a DACPAC to my Azure SQL Db?
Task logs
[Enable debug logging and please provide the zip file containing all the logs for a speedy resolution]
Troubleshooting
Checkout how to troubleshoot failures and collect debug logs: https://docs.microsoft.com/en-us/vsts/build-release/actions/troubleshooting
Error logs
[Insert error from the logs here for a quick overview]

As discussed, try connecting to Azure SQL database with active directory password from SSMS. if you are able to connect then you should be able to connect from Task as well. for Task no additional permissions are required.
Closing this issue.

I followed up with the SQL AAD team who confirmed the issue is not on their end. Would love support with this issue

@AmrutaKawade - pls review this clarification

can you pass /Diagnostics:true in additional arguments and run pipeline ?
it looks like auth is succeeding from your previously shared pipeline but while executing dacpac it failed


chore!: change fetchData output to non object

BREAKING CHANGE: if fetchData is called with a string, the output will be the value itself, not an object anymore.
Before:
const dataOneKey = await erc725.fetchData('LSP3Profile');
/**
{
  LSP3Profile: {
    LSP3Profile: {
      name: 'patrick-mcdowell',
      links: [Array],
      description: "Beautiful clothing that doesn't cost the Earth. A sustainable designer based in London Patrick works with brand partners to refocus on systemic change centred around creative education. ",
      profileImage: [Array],
      backgroundImage: [Array],
      tags: [Array]
    }
  }
}
*/

after:
const dataOneKey = await erc725.fetchData('LSP3Profile');
/**
{
  LSP3Profile: {
    name: 'patrick-mcdowell',
    links: [Array],
    description: "Beautiful clothing that doesn't cost the Earth. A sustainable designer based in London Patrick works with brand partners to refocus on systemic change centred around creative education. ",
    profileImage: [Array],
    backgroundImage: [Array],
    tags: [Array]
  }
}
*/

Closes #119.

Codecov Report

Merging #120 (1d4d570) into develop (49eb5cf) will decrease coverage by 0.53%.
The diff coverage is 33.33%.

@@             Coverage Diff             @@
##           develop     #120      +/-   ##
===========================================
- Coverage    87.09%   86.56%   -0.54%
===========================================
  Files           11       11
  Lines          620      625       +5
  Branches       122      124       +2
===========================================
+ Hits           540      541       +1
- Misses          40       42       +2
- Partials        40       42       +2

Impacted Files
Coverage Δ

src/index.ts
84.61% <33.33%> (-2.35%)
:arrow_down:

Continue to review full report at Codecov.

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


Make path_vec_from_text_list more idiomatic

Making the helper function a bit more idiomatic Rust.

LGTM this does look more idiomatic


Only unsubscribe if we have an array of listeners.

Possible solution to resolve #3.

Please pull......................

@JadenH Hi, once again sorry... I really miss time atm.
But thanks for it, I quickly push v0.5.3 following this merge.
I hope I'll have time soon to work on it...


feat: add etag and collections

This update introduces the addition of ETag to collection responses, allowing clients to track versioning for individual resources within a collection. By including an ETag for each item, we enable efficient caching and conditional operations, reducing unnecessary API calls and improving performance. The ETag is added as a read-only field in the response body to ensure version consistency without altering the API's general object model.
This solution can serve as an interim approach until a standardized HATEOAS implementation is adopted.

@omnipitous / @nickclarity / @PavlenkoV - I appreciate your expertise, effort, and review on this one.


Incorrect Placement of nodeId "ND-LotTenderingProcess" in NTDs

In 1.6.0 the group "GR-Lot-Purpose" received the nodeId "ND-LotTenderingProcess".
According to https://docs.ted.europa.eu/eforms/latest/notice-types/index.html my assumption is that all fields having this node as an ancestor should be a part of this group, but for some of the fields like BT-131(d)-Lot and BT-131(t)-Lot that is not the case.

Thanks for your feedback.
As indicated in the documentation page:

There is no constraint on non-repeatable groups. An arbitrary number of intermediate, non-repeatable groups can be created to organize the notice fields in a more convenient way.

In this case, fields like BT-131(d)-Lot are in a different non-repeatable group than other fields that are also under the node "ND-LotTenderingProcess". But the closest repeatable parent is the same : GR-Lot.
So the notice type definition is still "aligned" with the XML structure: Both only allow one occurrence of BT-131(d)-Lot in a lot.
The association of a group to a node is relevant for the structure only for repeatable groups. We have added other associations to help relate the XML with the form shown to a user (in particular for validation messages), but they are not necessary from a structural perspective.
I'm closing this as it does not seem to be a problem in the SDK.


Very old releases should be hosted

The oldest release versions are hosted on SourceForge. We should download them all into our archives so they are served from one place. And pages like this should have our download link:
https://www.stage.boost.cppalliance.org/releases/boost-1-21-1/

Sourceforge files are here: https://sourceforge.net/projects/boost/files/boost/
Current boost downloads are hosted at : https://archives.boost.io/
Notice that each archive is accompanied by a json file with this format:
{
"commit": "65c1319bb92fe7a9a4abd588eff5818d9c2bccf9",
"file": "boost_1_86_0.zip",
"created": "2024-08-07T22:03:02Z",
"sha256": "cd20a5694e753683e1dc2ee10e2d1bb11704e65893ebcc6ced234ba68e5d8646"
}

git commit
filename
creation date
The sha256 hash of the archive itself.

Action items:  write and test a python or bash script which does the following:

downloads all sourceforge archives before version 1.63.0.
generates a json file for each archive
downloads all the archives a second time in order to re-check and re-verifying the sha hash.
arrange the downloads in the directory structure that archives.boost.io uses.  The destination will be an S3 bucket s3://boost-archives.   Directory format:

boost-archives/release/1.62.0/source/filename
boost-archives/release/1.61.0/source/filename
boost-archives/release/1.60.0/source/filename

Finally, an "aws s3 cp" command to upload the files to a bucket such as  s3://boost-archives/release/ .

Test the entire process with another test s3 bucket first.   If an S3 bucket is needed let me know.

After the script is ready and tested, provide me a copy of the script.   I will run that "in production".

@rbbeeston  you could assign this to one of the developers.

when this is complete, revisit https://github.com/boostorg/website-v2/issues/584

@sdarwin do we have a way to get the commit (for the json file) for sourceforge releases?

@gavinwahl
reference the git tags in https://github.com/boostorg/boost
From there,
commit 65c1319bb92fe7a9a4abd588eff5818d9c2bccf9 (HEAD, tag: boost-1.86.0)
matches the example in this issue above,
"commit": "65c1319bb92fe7a9a4abd588eff5818d9c2bccf9"

@gavinwahl apologies - I missed that this was assigned to you. I've been working on this since yesterday per this slack thread. Let's connect tomorrow and compare notes. If you'd like to peek at what I have so far, you can look at this very WIP commit.

Per your question above, note that we have commit SHAs stored in the db already via Version.data["commit"]["sha"].

Sourceforge has an rss feed for all downloads, might be easier than scraping their html: https://sourceforge.net/projects/boost/rss?limit=100000
It also includes md5sums, which would be more reliable to verify instead of downloading the file twice.

md5 checksums

Yes, good idea

stored in the db already via Version.data["commit"]["sha"].

I had envisioned this as a one-time cli-driven external script.  it wouldn't necessarily have access to the database, right?   The commit can be found in this way:
git rev-list -n 1 $TAG
git rev-list -n 1 boost-1.86.0

@rbbeeston both me and Greg started on this independently -- who do you want to finish it?

@gavinwahl & @GregKaleka, can you compare notes on it and see who's further along?  @gavinwahl I see you don't have #1230 marked in progress yet, and have a couple other items still in progress. can you update your prioritizations so I can see what you currently have to work on?
Thanks!

md5 checksums
Yes, good idea

That's fine with me - though boost.org and all the newer versions we host show SHA256 hashes.

The sha256 ought to be computed. The contents of the json would include the sha256.
The purpose of the md5 is limited - it is to double-check the download from sourceforge was correct.  Just for that step it can be used.

OK, I've got a version of this working on the sourceforge-import branch. I ran the script, using a test bucket, and it appears to be working as expected. After running it, I verified the files seem to be stored correctly:
In [1]: from storages.backends.s3boto3 import S3Boto3Storage
   ...: storage = S3Boto3Storage(bucket_name="stage-rob.boost.org.v2")
   ...: directory_path = "test/boost-archives/release/1.62.0/source/"
   ...: files = storage.listdir(directory_path)[1]
   ...:
   ...: for file in files:
   ...:     print(file)
   ...:
boost_1_62_0.7z
boost_1_62_0.7z.json
boost_1_62_0.tar.bz2
boost_1_62_0.tar.bz2.json
boost_1_62_0.tar.gz
boost_1_62_0.tar.gz.json
boost_1_62_0.zip
boost_1_62_0.zip.json

In [2]: with storage.open(file_name, mode='r') as json_file:
   ...:     content = json.load(json_file)
   ...:     print(json.dumps(content, indent=4))
   ...:
{
    "commit": "4f2bdeb93a4be13ba0dc5e9f44920a2bf67191fc",
    "file": "boost_1_62_0.tar.gz",
    "created": "2016-09-28T14:18:40Z",
    "sha256": "440a59f8bc4023dbe6285c9998b0f7fa288468b889746b1ef00e8b36c559dce1"
}

Right now my script is scraping the webpage. I'm going to modify it to use the RSS feed as Gavin suggested, so that I can access the MD5 checksum for verification (it's not readily available on webpage).

OK, @sdarwin I've completed testing of this script. I don't have UI access to our S3 buckets, but if you'd like to do some checking, the tested files have been stored in the bucket stage-rob.boost.org.v2 and are stored in the directory test/boost-archives/release.
Let me know if you need anything else from me to verify this worked.
If you're ready to run this "in production", you can check out the sourceforge-import branch, and change the constants BUCKET_NAME and S3_KEY_PREFIX at the top of the file versions/management/commands/host_sourceforge_release_files.py. You'll also need to ensure your AWS credentials are set correctly in your .env file. Specifically:
AWS_S3_ENDPOINT_URL="https://s3.us-east-2.amazonaws.com"
AWS_ACCESS_KEY_ID="foo"
AWS_SECRET_ACCESS_KEY="bar"
AWS_S3_REGION_NAME="us-east-2"

In a test environment, going file by file, the script is uploading to S3.  Great!!   Probably on Monday I will run it "live" then.

The main set of archives (zip, 7z, bz2, gz) advertised on boost.org and boost.io have been uploaded from sourceforge.
https://archives.boost.io/release/1.62.0/source/


Update Ingress godep

@aledbf fyi

LGTM.
Out of curiosity, can this tag be used for release-1.2 as well, or is there an incompatibility there?

It can, I'm not sure how I'm going to handle the 1.2 update. We don't really "release" cluster addons, the l7 controller runs on the master so it should just rev forward automatically on hosted offerings. If someone sets up a new cluster (with old 1.2 code) on gce/prem etc, they'll get an old l7. They can use kubectl edit to update the image on this old controller forward to 0.7.0.
It should be safe to just update this tag to 1.2  because the resource didn't change, but I need to test it some more.

We don't really "release" cluster addons, the l7 controller runs on the master so it should just rev forward automatically on hosted offerings.

I meant we don't branch, we release eg: https://github.com/kubernetes/contrib/releases, no reason we can't other than it's in beta so it's still in contrib/

We're about to cut a 1.2.5. If you get the PR into master for bumping kubernetes/kubernetes/cluster/saltbase/salt/l7-gcp/glbc.manifest, we can get it cherry-picked to release-1.2, and it'll go out when we update hosted masters in GKE.

Does this Travis ever work?

(Feel free to merge if not.)

Yeah I think it broke maybe because of some change, looking. Thanks for lgtm will merge when i fix. I think we wouldn't have the famed "green badge" if it didn't work: https://github.com/kubernetes/contrib

Green, merging


Waves effect don't working inside owl-carousel plugin

I know this is not a real issue but maybe someone had previusly had the same problem and can help me. It will be great if you would help me or give me a hint about and maybe it can also help improving this awesome framework.
I have use the floating button with waves for the Owl carousel jquery plugin navigation buttons, and for some reason the waves effect doesn't work. I supouse that something in the owl carousel css or js is interfering but I can't find it.
This is my code:
("#intro-slider").owlCarousel({
	navigation : true, // Show next and prev buttons
	navigationText: ['',''],
	slideSpeed : 300,
	paginationSpeed : 400,
	singleItem:true,
	autoPlay : 5000,
});

Thanks in advance!

Owl Carousel is probably capturing the click event.

Solution is
$("#intro-slider").owlCarousel({
	pagination:true,
	touchDrag: false
});


chore(main): release 1.1.0

:rocket: New release :bookmark:
1.1.0 (2023-05-04)
:sparkles: Features

add go to top button (#34) (170da4f)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/hugo-sid/hugo-blog-awesome/releases/tag/v1.1.0 :sunflower:


wasm: 3x performance overhead of using webassembly in node 8

Im really not sure where to submit this issue, golang, node or chromium. So I am starting here. Feel free to close the issue and provide feedback if you think I should go elsewhere.
What version of Go are you using (go version)?
Master Head // b001ffb864ce5486c6edbe98202d3e0687313ce2
Does this issue reproduce with the latest release?
N/A latest release does not have wasm.
What operating system and processor architecture are you using (go env)?
wasm on linux
node 8.11
What did you do?
I built a connection from webassembly to node http and benchmarked it against native http version
recipe for reproducing
https://github.com/trashhalo/go_wasm_node_http
benchmark results
https://gist.github.com/trashhalo/b2f120fb9d20bd4003bf125c0200b601
node hello world
https://gist.github.com/thebinarypenguin/122cda772438c80ad683
What did you expect to see?
I expected a performance hit but not one so large
What did you see instead?
A 3x hit to performance of equivalent javascript code.

If you want to skip the docker build
app.wasm.gz

If I understand this correctly, there is very little "actual" webassembly here. This is still running a node http server, in a very roundabout fashion. What I believe you are seeing is the js-wasm transfer overhead.
/cc @neelance

I bet a good portion of it is due to the setTimeout here: https://github.com/golang/go/blob/5cb1b1773fba1e1503493eea5c4ffa49fc5b5f08/misc/wasm/wasm_exec.js#L319
Performance optimizations were not yet in scope.

@neelance I played around to see if I can shave off time. Removing settimeout didn't change too much. I ran it through chrome profiler and it seems the hotspot is loading strings out of the memory blob and decoding them on the js side. The strings in my example on the method names i want to call on my js values.
Perhaps some sort of constant pool is in order?

Perhaps some sort of constant pool is in order?

Maybe. We might even need some additional type like this:
idFoo := js.ID("foo")
for i := 0; i < 1000000; i++ {
   v.CallByID(idFoo)
}

This way, we could avoid referencing a method by string each time.
But we should wait with this until WebAssembly has its improved JS interop features ready.

That would be cool!
I dont know how safe this is but I changed loadString in the wasm_exec to have a js side string cache. Its not bombing... and now im getting ~10932.22 requests/second, vs original implementation 5540.49.
			const strCache = {};
			const loadString = (addr) => {
				const saddr = getInt64(addr + 0);
				const cache = strCache[saddr.toString()];
				if (cache) {
					return cache;
				}
				const len = getInt64(addr + 8);
				const ret = decoder.decode(new DataView(this._inst.exports.mem.buffer, saddr, len));
				strCache[saddr.toString()] = ret;
				return ret;
			}

This will now fall under https://github.com/golang/go/issues/32591. Closing this in favor of that.


Add local channel

What is the expected feature or enhancement?
QiskitRuntimeLocalService was created to support local testing mode and is meant to be similar to QiskitRuntimeService. Today a user is not expected to instantiate this class directly, since it only has a run() method that is called under the cover by the primitives. As we add more methods to it (see #1606 and #1607), users will want to instantiate it. Instead of having to import / create another class, we should just add a "local" channel.
Acceptance criteria
User can do
from qiskit_ibm_runtime import QiskitRuntimeService, Sampler

service = QiskitRuntimeService(channel="local")  # <--- this will return an instance of `QiskitRuntimeLocalService`

I was having second thoughts about whether to use QiskitRuntimeService(channel="local"), since we want to allow one to refresh the fake backend's data (see #1707), which requires a real service. So people may think they can do
service = QiskitRuntimeService(channel="local")
fake_backend = FakeSherbrooke()
fake_backend.refresh(service)

But that won't work since service here is local service, not a real QRT service. On the other hand, even if we make people to initialize service = QiskitRuntimeLocalService(), they may still think it's a "service" they can pass to refresh().
We should just check if the service passed to refresh is a non-local QiskitRuntimeService


Extension not working

Appears that the extension has stopped working for me in the last 2 weeks.
Not sure if an issue just on my end or something with DnD Beyond/Edge.
Edge Version 91.0.864.37 (Official build) (64-bit)
Would normally expect the character parts to expand and show their stats

I don't know if it's something related to Edge, as I am using that without issues in the Chrome.

I've done some playing around and found that, at least for me, it doesn't work in Chrome either.
I then looked at the settings and found that the URL in the includes didn't match the URL DND Beyond gives me.
By adding in a manual include I've got it working again.

Thats odd as %2f is just a /


nixos/kubernetes: forces etcd to wait for etcd.pem and adds tests

Description of changes

This PR forces etcd.service to wait for etcd.pem and adds tests about the journal logs searching for etcd erros. With this PR it starts cleanly.
Things done

The etcd unit was failing a few times, it spams journal logs. It was producing messages like:
"Failed to start etcd key-value store."

Command to get the massage:
journalctl --unit etcd.service --boot \
--catalog --no-pager --full --priority=err \
--output="json" --output-fields="MESSAGE"

It confuses people (including my self) trying to starting to use kubernetes from/in NixOS. The "classical error" about only etcd pods pending, that can be caused by many things, would be suspected to be etcd faults because of this "Failed to start ..." messages.
Details
Tested it with:
nix \
build \
--no-link \
--print-out-paths \
--override-flake \
nixpkgs \
'github:PedroRegisPOAR/nixpkgs/db15dfcb888dbb9fc02882d16a05a8fb471a4329' \
'nixpkgs#nixosTests.kubernetes.dns-single-node' \
'nixpkgs#nixosTests.kubernetes.dns-multi-node' \
'nixpkgs#nixosTests.kubernetes.rbac-single-node' \
'nixpkgs#nixosTests.kubernetes.rbac-multi-node'

Outputs:
/nix/store/0rgizg8lgflajgz5b2bk7zf95w7p0iq6-vm-test-run-kubernetes-dns-multinode
/nix/store/lrrj8j74jywjz183lla97vfjc9p28xwr-vm-test-run-kubernetes-dns-singlenode
/nix/store/78zvarvyxphhqf8gb5s6ig073nmbiaky-vm-test-run-kubernetes-rbac-multinode
/nix/store/nvpi7wqvda15qy9bj3z9yk85gzdlcarc-vm-test-run-kubernetes-rbac-singlenode

For local testing:
nix \
build \
--no-link \
--print-out-paths \
'.#nixosTests.kubernetes.dns-single-node' \
'.#nixosTests.kubernetes.dns-multi-node' \
'.#nixosTests.kubernetes.rbac-single-node' \
'.#nixosTests.kubernetes.rbac-multi-node'

Description of changes

Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[x] sandbox = true

[x] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[x] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
24.11 Release Notes (or backporting 23.11 and 24.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.

Add a :+1: reaction to pull requests you find important.

Friendly ping: @johanot @offlinehacker @saschagrunert @srhb

@OfBorg test kubernetes.dns-single-node kubernetes.dns-multi-node kubernetes.rbac-single-node kubernetes.rbac-multi-node

I think this looks good to me too. Is it also safe on apiserver nodes where the etcd service is not colocated?

Is it also safe on apiserver nodes where the etcd service is not colocated?

I do not know how to test/verify that. Any tips?
I did try to add in the multiNodeTest using for machine2 the same logic for testing used for machine1, but it did not work because it falsely fails because etcd unit is not present in the machine2.

I think I'm retracting my tentative approval. As I understand this, this would change a crashing systemd service into to an "all clean state" where etcd just doesn't start, which seems worse for monitoring purposes.
Please argue if you disagree with this position.


Using LocalFileImageDataProvider not working after a while

Hi
I store my images to file manager manually then use kingfisher's LocalFileImageDataProvider to load it onto ui image view. It was working perfectly for a few days until it wasn't. Currently, setting image using LocalFileImageDataProvider, image is nil, while image file still exist.
I was wondering if there is a certain duration that LocalFileImageDataProvider works or?
Below is my code:
let options = [.keepCurrentImageWhileLoading, .loadDiskFileSynchronously]
private func fetchStoredImage(using filePath: String) {
let url = URL(fileURLWithPath: filePath)
let provider = LocalFileImageDataProvider(fileURL: url)
self.imageView.kf.setImage(with: provider, options: self.options)
}
Below function returns true, image still exist.
func isImageSaved(atFileName fileName: String, completion: @escaping (Bool) -> Void) {
getFilePath(forFileName: fileName) { filePath in
guard let filePath = filePath else {
completion(false)
return
}
        let fileManager = FileManager.default
        completion(fileManager.fileExists(atPath: filePath))
    }

}

@onevcat please assist. Thanks.

@ElrB Sorry for the late reply.
I don't think there is a "certain duration that LocalFileImageDataProvider works"...Regardless of a local data provider or a remote URL data fetching, both are working in similar ways: Check the cache, if not hit, then get the data from the origin place (the local file or URL) and cache the image.
From your description, it seems that once the cache is expired, it stops loading from the original local file. It should not happen.
I also tried to build a simple sample with LocalFileImageDataProvider and a very short disk cache expiration duration (to simulate your case), but I cannot reproduce it at all.
Can you try:

Make sure that the url you use in fetchStoredImage has exactly the same path as you get in isImageSaved.
Try to print some result to see if there is an error and/or where the image is from when you set it.

self.imageView.kf.setImage(with: provider, options: self.options) { result in
  switch result {
    case .success(let r):
      print(r.cacheType)
      print(r.source)
    case .failure(let error):
      print(error)
  }
}

By the way, you can use the .diskCacheExpiration option change the disk cache duration to a smaller value to boost the verification.


-fix retry, implement mutation pool/batching, refactor bulk add, crea…

…te common library

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

moving this pull request against branch v1.1.5-fixes