Reduce Resources for Gradebook Fetching for Specific Users

When we specify specific users to get a gradebook for, we still pre-fetch all the course users.
Limit that fetch to only relevant users.
Depends on #3.

Complete with 62f1e7b2d8bfdd44324a0bab029020b024af1692.
We fetch the relevant users at the same time we resolve user queries.


Kura init mode, wsv serialization basics

Description
This PR is not done. But I want to get feedback on the strategy taken for serializing the WSV. If you think I should do something differently or that I've missed something please tell me.
Linked issue
Closes #1915
Benefits
Fast init and boot for peers after power outage.
Checklist

[ ] Draft PR released
[ ] Strategy analysed
[ ] Weakpoints discovered
[ ] Weakpoints addressed
[ ] Regular PR
[ ] Tests (via Pytest)

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

126 of 158 (79.75%)  changed or added relevant lines in 4 files are covered.
1121 unchanged lines in 28 files lost coverage.
Overall coverage increased (+0.04%) to 59.478%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

core/src/validator.rs
0
15
0.0%

core/src/smartcontracts/isi/triggers/set.rs
0
17
0.0%

Files with Coverage Reduction
New Missed Lines
%

client_cli/src/main.rs
1
0%

client/src/http_default.rs
1
0%

cli/src/main.rs
1
0%

core/src/smartcontracts/isi/asset.rs
1
29.79%

data_model/src/transaction.rs
1
48.08%

derive_primitives/src/lib.rs
1
0%

macro/derive/src/lib.rs
1
94.59%

wasm_codec/derive/src/lib.rs
1
90.68%

config/base/src/lib.rs
2
91.94%

tools/kagami/src/main.rs
3
2.01%

Totals

Change from base Build<PHONE_NUMBER>:
0.04%

Covered Lines:
19984

Relevant Lines:
33599

💛  - Coveralls

I'm fine with the general approach, but I think it has a few places where more work is known to be required.

will this affect @QuentinI 's work on #3651 ?

Closed in favor of #3724


Simplify GuardHelpers@{check, guest} a bit

It seems to be simpler without double negation

Prefer not to change if not broken.


fix: [Bug-4717] reverted all 200% zoom changes

Purpose
Revert changes for Resize of the content without overlap.
Does this introduce a breaking change?

[ ] Yes
[X] No

How to Test
1.  High contrast modes
Open CWYD URL.
Go to system setting ->Accessibility->Contrast themes->select aquatic in drop down-> apply.
Go to CWYD URL, you can find the screen, does not have border, so there is no separation between citation panel and other section.
git clone [repo-address]
cd [repo-name]
git checkout [branch-name]
npm install
Please find attached screenshot for reference

![image](https://github.com/Azure-Samples/chat-with-your-data-solution-accelerator/assets/168007985/c8d00492-31e0-4e55-867c-98d907810c7f)

![image](https://github.com/Azure-Samples/chat-with-your-data-solution-accelerator/assets/168007985/a356e0aa-7d11-4f28-89c1-9f619a029524)

Heya, why is this being reverted? I see you've said it's out of scope, but this seems to work as intended.

Heya, why is this being reverted? I see you've said it's out of scope, but this seems to work as intended.

Hi Adam,
We have provided the fix for 200% zoom in the previous PR and seems to be not completely fixed and bug got re-open. But later we just identified that we are supporting only till 125% as Bob confirmed. So we planned to remove the additional code related to 200% which added by PSL team.


xds: add api ref to the extension

Add api ref to the extension config creation.
Commit Message:
Additional Description:
Risk Level:
Testing:
Docs Changes:
Release Notes:
Platform Specific Features:
[Optional Runtime guard:]
[Optional Fixes #Issue]
[Optional Fixes commit #PR or SHA]
[Optional Deprecated:]
[Optional API Considerations:]

/assign @adisuissa

Thanks @adisuissa, e.g., sometimes some objects stored in the processContext are needed.


documentation shows wrong api for activating an eligible role

I was trying to follow this documentation to selfactivate an eligible role.
This page states to use the roleElibilityScheduleRequests api to activate an eligible request.
What works is using the roleAssignmentScheduleRequests api to activate a linkedroleeligibilityscheduleid

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

ID: 3a831056-b759-8539-ca86-9fbb84f331d7
Version Independent ID: 8fca63cc-bd36-d774-1f31-eca19502b006
Content: Activate Azure resource roles in PIM - Azure AD - Microsoft Entra
Content Source: articles/active-directory/privileged-identity-management/pim-resource-roles-activate-your-roles.md
Service: active-directory
Sub-service: pim
GitHub Login: @curtand
Microsoft Alias: curtand

@ikiris04
Thanks for your feedback! We will investigate and update as appropriate.

@ikiris04
Thank you for pointing this out! I've created a PR to update the documentation and it's currently waiting for the author to review and sign off.
I will now close out this issue. Please allow some time for the author to review and the changes to reflect.
Thank you for your time and patience!


Update git-mirrors.sh

while loop executes in subshell and value of STATUS is lost
this fixes that

Thanks for the PR.  I'll test and merge.

Nice use of the HERE string.


Test the updated GitHub action

My pyright test action, which is forked from Jake Bailey's but adds the warn on partial option, just got updated with the latest changes from Jake's, which are quite extensive. It would be good to kick off a test run of the stubs (I don't have permission). If things aren't working please open an issue at https://github.com/gramster/pyright-action

I was on vacation last week.   Kicked off a run now and everything looks good.
@gramster Would you like to be added as a maintainer of this project?


Handle common disk failures in the spill code path

In a JBOD setup disk failures are quite common. In the absence of basic failure
handling, a presto worker will not recover until a config is changed and process
is restarted -- which is difficult to handle operationally.
Spill path discovery routine already handles the case when one or more paths
are out of free space, this change adds couple of additional checks to cover
out of band disk failures.

How does it differ to https://github.com/prestosql/presto/pull/2175?

How does it differ to #2175?

Please nevermind, I just noticed the comment: https://github.com/prestosql/presto/pull/2175#issuecomment-572324028

@dfinninger Please sing CLA: https://github.com/prestosql/cla

@shrijeet, @dfinninger have you been using this patch in production?

As a follow up, we should proactively run the check (or clear the cache to force a check) anytime we get an IO error while reading or writing in the spill code.

Ha, great time for my printer to be out of ink. I'll need to pick up some replacement cartridges and I'll get the CLA signed and back to y'all by tomorrow.

@kokosing I have not used this in production, no.

Alright, just submitted the CLA.

@cla-bot check

As a follow up, we should proactively run the check (or clear the cache to force a check) anytime we get an IO error while reading or writing in the spill code.

@dfinninger are you going to handle that?

@kokosing I could take a look at it this week. And I'm assuming you mean in a follow-up PR?

And I'm assuming you mean in a follow-up PR?

Yes.

Merged, thanks!


.Net: Align Chroma memory store behavior with other stores

Motivation and Context

This is an attempt to account for the following PR.
Chroma logs an error when calling DoesCollectionExistAsync on a non existing collection, it should log at debug level instead
Also Volatile Store and QDrant return an empty collection when calling GetNearestMatchesAsync whereas Chroma throws an exception. This PR aligns behavior with other stores.
Description

Align chroma store behavior with other common memory stores
Contribution Checklist

[ ] The code builds clean without any errors or warnings
[ ] The PR follows the SK Contribution Guidelines and the pre-submission formatting script raises no violations
[ ] All unit tests pass, and I have added new tests where possible
[ ] I didn't break anyone :smile:

@dmytrostruk OK I'm happy either :

to revert the change in GetNearestMatchesAsync,, only keep the log event type change in this PR, leaving the distinct memory providers non aligned in their behavior concerning NearestMatch for missing collections.
to revert this one to throwing exceptions and also align the Volatile, Qdrant and potentially other providers to throw exceptions systematically in GetNearestMatchesAsync if the collection does not exist, but that will break the Copilot App until my other PR is merged, and potentially other apps that use the same pattern of create-on-insert / Search regardless of existence, so this is a breaking change that should be advertised as such, whether it is considered in this PR or postponed.
to keep the PR where the behavior is aligned to be more lenient allowing GetNearestMatchesAsync to return empty when collection does not exist yet. That was my choice because I think that this is the currently expected behavior, and it would make sense for all the providers to change in sync with the appropriate warning for better coherence.

Can you make a firm decision?

@jsboige Please use this option for now:

to revert the change in GetNearestMatchesAsync,, only keep the log event type change in this PR, leaving the distinct memory providers non aligned in their behavior concerning NearestMatch for missing collections.

We will focus on aligning every connector to follow the same flow, I created issue for that - https://github.com/microsoft/semantic-kernel/issues/2269
The preferred way would be to throw an exception when collection doesn't exist.
Thank you!

@dmytrostruk I reverted the over reach to only keep the logger update


wrong base cleaning in _missingfy() when dealing with grid slices...

...when NaNs are found in a grid slice before an exclusion of codes happens, the cleaning of the base vectors will incorrectly turn all elements of the slice into NaN.
This results empty grid slices.

fixed long time ago.


Modularization

I think it would be a good idea to break this project up into separate modules to ease the process of development/testing and also make it easier for incoming developers to be able to understand the project. I have already begun doing this in #421, #422, and #423 but I think there is more to be done. I foresee the project having one "main" module where the overall game is ran, including all other modules, and the set of modules which control the different components of the game (game logic, chat client, canvas, socket handlers, etc).

Thanks for contributing! I do agree that separating functionality into different modules is easier to understand than a large mixed JS file.
Also, I would suggest that If you're going to continue work on modularization, I think it would be better to make one larger pull request with a big batch of commits rather than a bunch of small PRs.

No problem! And will do. Thanks for the feedback.


Throw error in CSG when adding a mesh that lacks indices, positions or normals

Small issue found, where Babylon reads from a null array. Added some checks that throw more descriptive errors and adjusted the typing as well to better represent the data usage.

Please make sure to label your PR with "bug", "new feature" or "breaking change" label(s).
To prevent this PR from going to the changelog marked it with the "skip changelog" label.

Please make sure to label your PR with "bug", "new feature" or "breaking change" label(s).
To prevent this PR from going to the changelog marked it with the "skip changelog" label.

Snapshot stored with reference name:
refs/pull/15237/merge
Test environment:
https://babylonsnapshots.z22.web.core.windows.net/refs/pull/15237/merge/index.html
To test a playground add it to the URL, for example:
https://babylonsnapshots.z22.web.core.windows.net/refs/pull/15237/merge/index.html#WGZLGJ#4600
Links to test babylon tools with this snapshot:
https://playground.babylonjs.com/?snapshot=refs/pull/15237/merge
https://sandbox.babylonjs.com/?snapshot=refs/pull/15237/merge
https://gui.babylonjs.com/?snapshot=refs/pull/15237/merge
https://nme.babylonjs.com/?snapshot=refs/pull/15237/merge
To test the snapshot in the playground with a playground ID add it after the snapshot query string:
https://playground.babylonjs.com/?snapshot=refs/pull/15237/merge#BCU1XR#0

WebGL2 visualization test reporter:
https://babylonsnapshots.z22.web.core.windows.net/refs/pull/15237/merge/testResults/webgl2playwright/index.html

Visualization tests for WebGPU (Experimental)
https://babylonsnapshots.z22.web.core.windows.net/refs/pull/15237/merge/testResults/webgpuplaywright/index.html


Butterknife Gradle Plugin 9.0.0 SNAPSHOT + AGP 3.2.0 can't generate R2 class in library

when upgrade to Android Studio 3.2 and AGP 3.2.0, can't generate R2 class in library. use AGP 3.1.4 are fine.
FAILURE: Build failed with an exception.

* What went wrong:
Execution failed for task ':...:generateDebugRFile'.
> java.io.FileNotFoundException: ... R.java (No such file or directory)

my versions
versions.butterknife_plugin = "9.0.0-SNAPSHOT"
butterknife.plugin = "com.jakewharton:butterknife-gradle-plugin:$versions.butterknife_plugin"

How to avoid this problem forever!

check your project level gradle file,i downgraded it to  classpath 'com.android.tools.build:gradle:3.3.0-alpha10' from classpath 'com.android.tools.build:gradle:3.3.0-alpha12' till there is a fix.
P.S Using Butterknife with AndroidX

This is a dupe of #1377

@goonerDroid @hzsweers thank you for the help~

H

Hi


Authentication

I think it would be much more convenient if we can add authentication like HTTP Authentication, query parameter,IP restriction etc.

That's right. But generally for all AuthN concerns, I usually rely on a third-party service with a high level of security and configuration. For example, I use webhookd in production behind an API gateway called Kong. You can put all the restrictions you want and plug in almost all AuthN and AuthZ mechanisms.
However, I will see if I can find time to add a very simple authentication feature (Basic Auth or API key).
Besides, you can do a lot of things into the shell script! Validating an API key is simple and validating a Basic Auth header should not be more difficult ;)

I think auth methods should live as HTTP middlewares living in an auth package. I could submit a PR if you want :)

You are truly welcome! :smile:


Mapping kmehr -> fhir?

@costateixeira @bdc-ehealth any progress is this ?

This also means that the implementation of a mapping gets pushed to each vendor I assume?
For the record: I have no issue with that, I just want this clarified.
If that is the case I feel like this issue can be closed as well.

@JensPenny ,
I do not want to make any statements on who will implement "a" mapping. The point is that the resulting FHIR will have to be confirmed by a physician, because an automatic translation that is always correct, is mathematically not feasible.


Default for source code header documentation

Hi guys,
On my last PR https://github.com/go-gitea/gitea/pull/430#discussion_r93372087 I add Gitea copyright on changed files, but I put copyright in year asc, as below:
// Copyright 2014 The Gogs Authors. All rights reserved.
// Copyright 2016 The Gitea Authors. All rights reserved.

On PR, @lunny asked to put in order desc, but for me is strange, because on PHP and Python projects is normal to put copyright in order year asc.
On Golang doesn't  have a standard or definition for this yet, then I think that this is a chance to set default source code header and documentation for Gitea in general. With this it would even be possible to show for other projects for we make tools for API reference documentation in the future like PHPDoc, ApiGen, Doxygen, Sphinx, Javadoc and others.
Looking for principal Go projects on GitHub that I know, really doesn't exists a standard for this, as examples below:

Minio: https://github.com/minio/minio/blob/master/main.go
Caddy: https://github.com/mholt/caddy/blob/master/caddy.go
Docker: https://github.com/docker/docker/blob/master/cli/cobra.go
Kubernets: https://github.com/kubernetes/kubernetes/blob/master/cmd/kube-discovery/kubediscovery.go
Beego: https://github.com/astaxie/beego/blob/master/beego.go
Hugo: https://github.com/spf13/hugo/blob/master/main.go

Then, what my propose about this? Each one of us create one header documentation that like and discuss about this to create one standard.
What you think guys?

// Copyright 2015 The Gogs Authors. All rights reserved.
// Copyright 2016 The Gitea Authors. All rights reserved.
//
// Licensed under the Apache License, Version 2.0 (the "License");
// you may not use this file except in compliance with the License.
// You may obtain a copy of the License at
// http://www.apache.org/licenses/LICENSE-2.0
//
// Unless required by applicable law or agreed to in writing, software
// distributed under the License is distributed on an "AS IS" BASIS,
// WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
// See the License for the specific language governing permissions and
// limitations under the License.

You mean order by year asc? @tboerger

You mean order by year asc? @tboerger

Yes, that's how I also know that from other projects and languages.

OK. I will chek all the code and send a PR to fix this.

It seems all the files are correct. So let's close this issue.

for now we solved problem about copyright ordenation.
But for future we can consider discuss about a possible new standard for we documentate all code?
@lunny @tboerger

Current it's enough, I think


If first added person in list is owner, all other users added will be owner

Steps:

Create 3 environments
First environment, add rancherdev & rancherdev2 as members (this is previous bug #813)
Second environment, add rancherdev & ranchderdev2 as members (this works)
Third environment, add rancherdev as owner & rancherdev2 as member

Results:  Both rancherdev and rancherdev2 have owner rights
Expected:  rancherdev as owner  & rancherdev2 as member
This seem to be because the first person I added has owner rights and then everyone below them has owner rights.

v0.23.0-rc5
Verified fixed


Harden validations, report server version & deprecate TTL fields

This PR contains three separate commits:
Remove obsolete TTL fields
Proposals and validations include a TTL field. This was part of an earlier idea which never really materialized. The fields were never really used in practice (a TTL value is not set by default). This change simply makes this official: the fields are deprecated, should not be used and are not coming back.
Ripple engineers will be implementing the system outlined by @JoelKatz in this post.
Include version in validations
Currently, it is impossible to know what version of the software a given validator is running. This information is useful for server operators, for developing who are putting together dashboards, and the community at large.
This change adds a "packed" version in a validation message. The message will, by default, only be sent every 256 ledgers.
Harden validations
Several changes are included in this PR.

Track how many validations a given server publishes for a given ledger. This should be precisely one. When such validations are detected, the server will now log a message to notify server operators of this issue.
The commit adds a 64-bit "cookie" in every validation which can be used to attempt to decipher whether multiple validations are the result of accidental misconfiguration vs. outright byzantine behavior. It is important to note that a malicious validator operator can manipulate the cookie.
Adds the hash of the last ledger a validator considers to be fully validated in the validation message.

(an unofficial lookover of 99f907b by @bachase would be awesome!)

When such validations are detected, the server will now log a message to notify server operators of this issue.

Could we optionally also get a full log (or sqlite database, if you feel fancy) of all validation messages received? As far as I know, this data is currently only available on WebSocket streams.

When such validations are detected, the server will now log a message to notify server operators of this issue.

Could we optionally also get a full log (or sqlite database, if you feel fancy) of all validation messages received? As far as I know, this data is currently only available on WebSocket streams.

We used to record validations in an SQL table, and we removed that functionality (see https://github.com/ripple/rippled/commit/c5a95f1eb5a5eedc7ece46ad26505209c1703486), because it was not really used by anyone/anything.
I think (and this is just my personal position) that the best option is to use the subscription and to index the received data separately. But would we, for example, report conflicting validations in the subscription stream?
Happy to discuss what to do here.

I don't see any documentation impacts beyond the new amendment. Am I missing anything? Are there changes to the fields provided in any of the API subscription streams for example?

I think I've addressed all your concerns, except for improved unit test @bachase. I've avoided rebasing this to make reviewing easier; once you sign-off, I'll do it and force-push the branch.
In the meantime, I'm running a rebased version against the network.

I'm going to rebase this against b4 and squash.


Gutter too narrow

When using Dark Atom Syntax, the gutter width is 49px; with Seti Syntax it's 43px.
When there is a line with .git-line-added, the .icon-right div gets cut (see screenshot).
I have been unable to pinpoint the exact css property that is causing this difference.

:+1:

:+1:  I also have that problem with errors


Lost workspace widget on Hyprland

For the past few days, I've not had the workspace area on the Waybar.
I use Hyprland on Arch.
My settings in config.jconc:
"wlr/workspaces": {
    "format": "{name}: {icon}",
    "format-icons": {
      "1": "",
      "2": "",
      "3": "",
      "4": "",
      "5": "",
      "active": "",
      "default": ""
    },
    "disable-scroll": true,
    "all-outputs": true,
    "on-click": "activate",
    "persistent_workspaces": {
      "1": [],
      "2": [],
      "3": [],
      "4": [],
      "5": [],
      "6": [],
      "7": [],
      "8": [],
      "9": [],
      "10": []
    }
  },

I've gone through the multiple fixes posted on the web including building from scratch but nothing has worked.
Am I missing something?

Same happening for me too!!
My config :
"wlr/workspaces": {
       "format": "{name}",
        "on-click": "activate",
         "all-outputs": true,
        "sort-by-number": true,
        "format-icons": {
              "1": "main  ",
              "2": "2  ",
              "3": "3  ",
              "4": "4  ",
              "5": "5  ",
              "6": "6  ",
              "7": "7  ",
              "8": "8  ",
              "9": "9  ",
              "10": " 10 ",
              "focused": "",
              "default": ""    },
        "on-scroll-up": "hyprctl dispatch workspace e+1",
        "on-scroll-down": "hyprctl dispatch workspace e-1"
},

And the style:
#workspaces {
	background: #282828;
	margin: 5px 5px 5px 5px;
  padding: 0px 5px 0px 5px;
	border-radius: 16px;
  border: solid 0px #f4d9e1;
  font-weight: normal;
  font-style: normal;
}
#workspaces button {
    padding: 0px 5px;
    border-radius: 16px;
    color: #928374;
}

#workspaces button.active {
    color: #f4d9e1;
    background-color: transparent;
    border-radius: 16px;
}

#workspaces button:hover {
	background-color: #E6B9C6;
	color: black;
	border-radius: 16px;
}

Glad it's not just man. I tried setting up EWW as my new widget manager thing but it wasn't having any of it. Don't even know if I'm posting this on the right repo because it's the waybar hypr version.

Mine is gone too.

Same problem here, pretty minimal config:
"wlr/workspaces": {
     "sort-by-name": true,
     "on-click": "activate"
},

I think this is due to a change in Hyprland, their release notes for v0.29.1 (released 3 days ago) state:

support for the ext_workspace_v1 protocol has been dropped, meaning waybar's wlr/workspaces and > alike will not work anymore (for waybar I recommend just switching to hyprland/workspaces)

Yes, this is a duplicate of #2463 and #2462, among others. Please change wlr/workspaces to hyprland/workspaces. Keep in mind not every option is exactly the same, so behaviour might change. Read the wiki if you come across any issues.

I had the same issue but to fix it I uninstall waybar-hyprland and installed waybar-hyprland-git.
The problem is waybar-hyprland is out of date.

I had the same issue but to fix it I uninstall waybar-hyprland and installed waybar-hyprland-git. The problem is waybar-hyprland is out of date.

As far as I know, and from my experience with Hyprland/Waybar in Arch, the AUR package waybar-hyprland has not been needed anymore for some time now: just use waybar from the official repositories.

I got it to work by using hyprland/workspaces instead of the wlr one

I had the same issue yesterday a rollback solved it for my.

I had the same issue but to fix it I uninstall waybar-hyprland and installed waybar-hyprland-git. The problem is waybar-hyprland is out of date.

As @LorenzoBettini said, you can just swap to using waybar now instead or you could yolo it and build it directly from the repo with a clone ;)


added design guideline links for tab and bots

Added design guideline links for tabs and bots. Also rearranged a section for tab mobile.

Docs Build status updates of commit dd2d601:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

msteams-platform/concepts/design/personal-apps.md
:white_check_mark:Succeeded

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

Docs Build status updates of commit d6a8017:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

msteams-platform/concepts/design/personal-apps.md
:white_check_mark:Succeeded
View

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

Docs Build status updates of commit 8cab2aa:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

msteams-platform/concepts/design/personal-apps.md
:white_check_mark:Succeeded
View

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


Return annotations in search endpoint via fieldList

In the OLS3 search API it was possible to request individual annotation fields be included in the results via fieldList. You could either include annotations_trimmed, or specify an annotation by including <field_name>_annotation in fieldList.
Example OLS3 request:
https://www.ebi.ac.uk/ols/api/search?q=holstein&ontology=vbo&queryFields=label,synonym&fieldList=iri,label,synonym,subset,exists_in_country_annotation,annotations_trimmed

This would return result items like:
 {
    "iri": "http://purl.obolibrary.org/obo/VBO_0003160",
    "label": "Holstein, Ecuador (Cattle)",
    "synonym": [
	    "Holstein"
    ],
    "annotations_trimmed": [
	    "https://orcid.org/0000-0002-4142-7153",
	    "https://orcid.org/0000-0002-5520-6597",
	    "https://orcid.org/0000-0002-9178-3965",
	    "http://www.wikidata.org/entity/Q736",
	    "https://www.fao.org/dad-is",
	    "ECU"
    ],
    "exists_in_country_annotation": [
	    "http://www.wikidata.org/entity/Q736"
    ]
}

Currently, neither form of the annotation seems to be supported in OLS4.
Including these annotations in search results is very useful for my use case because it means I can search for multiple results, without having to do individual requests on each term when I need to filter on one of these annotation properties.

I am looking into reimplementing annotations_trimmed to enhance the search boosting, but interested in your use case for returning them to the API consumer? Aren't the annotations values pretty much useless without the properties they correspond to?

This is an undocumented feature in OLS3 and was not intended to be used in this way.
Having said that, this could be an interesting use case and is worth exploring further.

Yes, I've never found having annotations_trimmed in the returned fields that useful, because you can't see what fields they correspond to. But the undocumented feature in OLS3 where you could request specific annotations turned out to be very useful, as 1 or 2 extra fields in the search result items can mean the difference between just using the search results, and having to do individual lookups of every term.
Possibly I only discovered this by reading the OLS3 source code, after trying to work out which annotations could actually be retrieved in the search endpoint. So I've made life hard for myself by relying on an undocumented feature. I've attempted a pull request for retrieving the specific annotations in #561


external tests can't get DOCKERIMAGE_TAG set

external tests can't get DOCKERIMAGE_TAG set, which caused
"/home/runner/work/runaqaTest/runaqaTest/openjdk-tests/TKG/../TKG/output_16173068587866/TestTargetResult";
Unable to recognize DOCKER_OS from DOCKERIMAGE_TAG = --version!
docker run  --rm adoptopenjdk-derby-test:8-jdk-ubuntu-hotspot-full
Unable to find image 'adoptopenjdk-derby-test:8-jdk-ubuntu-hotspot-full' locally
docker: Error response from daemon: pull access denied for adoptopenjdk-derby-test, repository does not exist or may require 'docker login': denied: requested access to the resource is denied.

https://github.com/sophia-guo/runaqaTest/runs/2249127830?check_suite_focus=true
Need to investigate why the if condition( which should set dockerImageTag=nighlty) doesn't work https://github.com/AdoptOpenJDK/openjdk-tests/blob/master/external/build.xml#L28

DOCKERIMAGE_TAG works as an environment variable for external tests. For test build stage if no DOCKERIMAGE_TAG set  nightly is used as default.  For test running DOCKERIMAGE_TAG is specifically used as a  shell script parameter. That is for local run or run-aqa if DOCKERIMAGE_TAG is not set as an environment variable external tests will fail.

Currently locally run with external tests manual may not work. The fix will also need to update the readme

Fix is in https://github.com/AdoptOpenJDK/openjdk-tests/pull/2462.
Re https://github.com/adoptium/run-aqa/issues/80#issuecomment-814237556
Based on the current fix no need to update Readme.

@sophia-guo so this can be closed?

Yes, it can be closed.


allow higher than rand 0.6

this is needed to have better dependency management in the future.
Alot of crates depend on this.
But portpicker using 0.6 and NOT 0.7 will mean for alot of rust programs to include both.
Which increases the compile time for some projects.
So in order to be future prof we should change it to >= 0.6, so it will accept newer too without recent updates necessary.
Best regards,
Marcel

rand has not released a 1.0 yet, and they have done breaking changes in 0.x minor updates, so I'm not going to do that. I will still update to lastest version in the next release.


Arch Linux Usan 3.x/ build bot with -fno-sanitize=function

Closes #517

Ah it's a minor detail, but the bot is named AMD64 Arch Linux Usan Function (so we could assume that it's testing the USAN for functions). It's not really an issue though I think.


TLS/SSL connection of Publisher to IoT-hub fails

Hello,
I try to update the OPC-Publisher on an embedded Linux (arm) device to the tagged version 2.3.2 .
As the log below indicates, the connection of the publisher to the IoT-Hub fails, due to one missing file.
Unfortunally the log message does not say the exact name of the missing file.
2019-02-13 13:43:15.895 +00:00 [INF] Current directory is: /var/www/Azure
2019-02-13 13:43:16.548 +00:00 [INF] Log file is: /var/tmp/publisher.txt
2019-02-13 13:43:16.549 +00:00 [INF] Log level is: info
2019-02-13 13:43:16.659 +00:00 [INF] OPC Publisher V2.3.0 starting up...
2019-02-13 13:43:16.678 +00:00 [INF] OPC UA monitored item notifications with one of the following 2 status codes will not generate telemetry events:
2019-02-13 13:43:16.682 +00:00 [INF] StatusCode: BadNoCommunication (dec:<PHONE_NUMBER>, hex: 80310000)
2019-02-13 13:43:16.682 +00:00 [INF] StatusCode: BadWaitingForInitialData (dec:<PHONE_NUMBER>, hex: 80320000)
2019-02-13 13:43:16.844 +00:00 [INF] opcstacktracemask set to: 0x0
2019-02-13 13:43:16.852 +00:00 [INF] OperationTimeout set to 120000
2019-02-13 13:43:16.876 +00:00 [INF] OPC UA server base address: opc.tcp://edgegate00f5:62222/UA/Publisher
2019-02-13 13:43:16.883 +00:00 [INF] Security policy http://opcfoundation.org/UA/SecurityPolicy#Basic256Sha256 with mode SignAndEncrypt added
2019-02-13 13:43:17.075 +00:00 [INF] Trusted Issuer store type is: Directory
2019-02-13 13:43:17.078 +00:00 [INF] Trusted Issuer Certificate store path is: pki/issuer
2019-02-13 13:43:17.080 +00:00 [INF] Trusted Peer Certificate store type is: Directory
2019-02-13 13:43:17.080 +00:00 [INF] Trusted Peer Certificate store path is: pki/trusted
2019-02-13 13:43:17.082 +00:00 [INF] Rejected certificate store type is: Directory
2019-02-13 13:43:17.082 +00:00 [INF] Rejected Certificate store path is: pki/rejected
2019-02-13 13:43:17.084 +00:00 [INF] Rejection of SHA1 signed certificates is disabled
2019-02-13 13:43:17.086 +00:00 [INF] Minimum certificate key size set to 1024
2019-02-13 13:43:17.094 +00:00 [INF] Application Certificate store type is: Directory
2019-02-13 13:43:17.097 +00:00 [INF] Application Certificate store path is: pki/own
2019-02-13 13:43:17.098 +00:00 [INF] Application Certificate subject name is: edgegate00F5
2019-02-13 13:43:17.099 +00:00 [WRN] WARNING: Automatically accepting certificates. This is a security risk.
2019-02-13 13:43:17.425 +00:00 [INF] Application certificate with thumbprint 'CC819F8673D96091825A5FC9FED09BAA151F3155' found in the application certificate store.
2019-02-13 13:43:17.674 +00:00 [INF] Application certificate is for ApplicationUri 'urn:edgegate00f5:edgegate00F5.softing-testnet-dmz.org:microsoft:', ApplicationName 'edgegate00F5' and Subject is 'edgegate00F5'
2019-02-13 13:43:17.676 +00:00 [INF] LDS(-ME) registration intervall set to 0 ms (0 means no registration)
2019-02-13 13:43:17.776 +00:00 [INF] Trusted issuer store contains 0 certs
2019-02-13 13:43:17.787 +00:00 [INF] Trusted issuer store has 0 CRLs.
2019-02-13 13:43:17.789 +00:00 [INF] Trusted peer store contains 0 certs
2019-02-13 13:43:17.790 +00:00 [INF] Trusted peer store has 0 CRLs.
2019-02-13 13:43:17.790 +00:00 [INF] Rejected certificate store contains 0 certs
2019-02-13 13:43:17.792 +00:00 [INF] There is no site configured.
2019-02-13 13:43:17.794 +00:00 [INF] Starting server on endpoint opc.tcp://edgegate00f5:62222/UA/Publisher ...
2019-02-13 13:43:32.117 +00:00 [INF] Server started.
2019-02-13 13:43:32.205 +00:00 [INF] Using default telemetry configuration.
2019-02-13 13:43:32.304 +00:00 [INF] The name of the configuration file for published nodes is: /var/www/Azure/publishednodes.json
2019-02-13 13:43:32.305 +00:00 [INF] The node configuration file '/var/www/Azure/publishednodes.json' does not exist. Continue and wait for remote configuration requests.
2019-02-13 13:43:32.305 +00:00 [INF] There are 0 nodes to publish.
2019-02-13 13:43:32.421 +00:00 [INF] IoTHub device cert store type is: X509Store
2019-02-13 13:43:32.422 +00:00 [INF] IoTHub device cert path is: My
2019-02-13 13:43:32.422 +00:00 [INF] Attempting to register ourselves with IoT Hub using owner connection string.
2019-02-13 13:43:34.975 +00:00 [FTL] Error in IoTHub initialization.
Microsoft.Azure.Devices.Common.Exceptions.IotHubCommunicationException: The SSL connection could not be established, see inner exception. ---> System.Net.Http.HttpRequestException: The SSL connection could not be established, see inner exception. ---> Interop+Crypto+OpenSslCryptographicException: error:2006D080:BIO routines:BIO_new_file:no such file
   at Interop.Crypto.CheckValidOpenSslHandle(SafeHandle handle)
   at Internal.Cryptography.Pal.StorePal.LoadMachineStores()
   at Internal.Cryptography.Pal.StorePal.FromSystemStore(String storeName, StoreLocation storeLocation, OpenFlags openFlags)
   at System.Security.Cryptography.X509Certificates.X509Store.Open(OpenFlags flags)
   at Internal.Cryptography.Pal.OpenSslX509ChainProcessor.FindCandidates(X509Certificate2 leaf, X509Certificate2Collection extraStore, HashSet`1 downloaded, HashSet`1 systemTrusted, TimeSpan& remainingDownloadTime)
   at Internal.Cryptography.Pal.ChainPal.BuildChain(Boolean useMachineContext, ICertificatePal cert, X509Certificate2Collection extraStore, OidCollection applicationPolicy, OidCollection certificatePolicy, X509RevocationMode revocationMode, X509RevocationFlag revocationFlag, DateTime verificationTime, TimeSpan timeout)
   at System.Security.Cryptography.X509Certificates.X509Chain.Build(X509Certificate2 certificate, Boolean throwOnException)
   at System.Security.Cryptography.X509Certificates.X509Chain.Build(X509Certificate2 certificate)
   at System.Net.Security.CertificateValidation.BuildChainAndVerifyProperties(X509Chain chain, X509Certificate2 remoteCertificate, Boolean checkCertName, String hostName)
   at System.Net.Security.SecureChannel.VerifyRemoteCertificate(RemoteCertValidationCallback remoteCertValidationCallback, ProtocolToken& alertToken)
   at System.Net.Security.SslState.CompleteHandshake(ProtocolToken& alertToken)
   at System.Net.Security.SslState.CheckCompletionBeforeNextReceive(ProtocolToken message, AsyncProtocolRequest asyncRequest)
   at System.Net.Security.SslState.StartSendBlob(Byte[] incoming, Int32 count, AsyncProtocolRequest asyncRequest)
   at System.Net.Security.SslState.ProcessReceivedBlob(Byte[] buffer, Int32 count, AsyncProtocolRequest asyncRequest)
   at System.Net.Security.SslState.StartReadFrame(Byte[] buffer, Int32 readBytes, AsyncProtocolRequest asyncRequest)
   at System.Net.Security.SslState.StartReceiveBlob(Byte[] buffer, AsyncProtocolRequest asyncRequest)
   at System.Net.Security.SslState.CheckCompletionBeforeNextReceive(ProtocolToken message, AsyncProtocolRequest asyncRequest)
   at System.Net.Security.SslState.StartSendBlob(Byte[] incoming, Int32 count, AsyncProtocolRequest asyncRequest)
   at System.Net.Security.SslState.ProcessReceivedBlob(Byte[] buffer, Int32 count, AsyncProtocolRequest asyncRequest)
   at System.Net.Security.SslState.StartReadFrame(Byte[] buffer, Int32 readBytes, AsyncProtocolRequest asyncRequest)
   at System.Net.Security.SslState.PartialFrameCallback(AsyncProtocolRequest asyncRequest)
--- End of stack trace from previous location where exception was thrown ---
   at System.Net.Security.SslState.ThrowIfExceptional()
   at System.Net.Security.SslState.InternalEndProcessAuthentication(LazyAsyncResult lazyResult)
   at System.Net.Security.SslState.EndProcessAuthentication(IAsyncResult result)
   at System.Net.Security.SslStream.EndAuthenticateAsClient(IAsyncResult asyncResult)
   at System.Net.Security.SslStream.<>c.<AuthenticateAsClientAsync>b__47_1(IAsyncResult iar)
   at System.Threading.Tasks.TaskFactory`1.FromAsyncCoreLogic(IAsyncResult iar, Func`2 endFunction, Action`1 endAction, Task`1 promise, Boolean requiresSynchronization)
--- End of stack trace from previous location where exception was thrown ---
   at System.Net.Http.ConnectHelper.EstablishSslConnectionAsyncCore(Stream stream, SslClientAuthenticationOptions sslOptions, CancellationToken cancellationToken)
   --- End of inner exception stack trace ---
   at System.Net.Http.ConnectHelper.EstablishSslConnectionAsyncCore(Stream stream, SslClientAuthenticationOptions sslOptions, CancellationToken cancellationToken)
   at System.Threading.Tasks.ValueTask`1.get_Result()
   at System.Net.Http.HttpConnectionPool.CreateConnectionAsync(HttpRequestMessage request, CancellationToken cancellationToken)
   at System.Threading.Tasks.ValueTask`1.get_Result()
   at System.Net.Http.HttpConnectionPool.WaitForCreatedConnectionAsync(ValueTask`1 creationTask)
   at System.Threading.Tasks.ValueTask`1.get_Result()
   at System.Net.Http.HttpConnectionPool.SendWithRetryAsync(HttpRequestMessage request, Boolean doRequestAuth, CancellationToken cancellationToken)
   at System.Net.Http.RedirectHandler.SendAsync(HttpRequestMessage request, CancellationToken cancellationToken)
   at System.Net.Http.HttpClient.FinishSendAsyncBuffered(Task`1 sendTask, HttpRequestMessage request, CancellationTokenSource cts, Boolean disposeCts)
   at Microsoft.Azure.Devices.HttpClientHelper.ExecuteAsync(HttpClient httpClient, HttpMethod httpMethod, Uri requestUri, Func`3 modifyRequestMessageAsync, Func`2 isMappedToException, Func`3 processResponseMessageAsync, IDictionary`2 errorMappingOverrides, CancellationToken cancellationToken)
   --- End of inner exception stack trace ---
   at Microsoft.Azure.Devices.HttpClientHelper.ExecuteAsync(HttpClient httpClient, HttpMethod httpMethod, Uri requestUri, Func`3 modifyRequestMessageAsync, Func`2 isMappedToException, Func`3 processResponseMessageAsync, IDictionary`2 errorMappingOverrides, CancellationToken cancellationToken)
   at Microsoft.Azure.Devices.HttpClientHelper.GetAsync[T](Uri requestUri, TimeSpan operationTimeout, IDictionary`2 errorMappingOverrides, IDictionary`2 customHeaders, Boolean throwIfNotFound, CancellationToken cancellationToken)
   at OpcPublisher.IotHubCommunication.InitAsync()

BTW: We have build the publisher for linux-arm as described in https://github.com/dotnet/core/blob/master/samples/RaspberryPiInstructions.md . And after the build, the generated files had been copied from a Windows file system to a case sensitive Linux file system.
So I faced a problem due to rename of OpcPublisher.dll into opcpublisher.dll.
Could the issue above also be caused by some thing similar?
Greetings
_   Juergen

@JuergenKosel Can you check if the DigiCert Baltimore Root CA Certificate is in the store?
Can you give me some more info on the case issue or submit a PR?

Hello @hansgschossmann ,
the problem was an incorrect update of the dotnet runtime environment.
Now I extract the arm binaries from your docker image with a helper docker image:
Dockerfile
# Docker image to copy opc publisher from docker image for arm architexture to a volume
# See https://ownyourbits.com/2018/06/27/running-and-building-arm-docker-containers-in-x86/
# docker container run --rm --name publisher-extract -w $(pwd) -v $(pwd):$(pwd) opc-publisher:2.3.2-linux-arm32v7-2amd64

FROM mcr.microsoft.com/iotedge/opc-publisher:2.3.2-linux-arm32v7

COPY qemu-arm-static /usr/bin

ENTRYPOINT ["cp", "-r",  "/app", "/usr/share/dotnet", "./" ]

So I can copy the binaries from an image which was build for linux-arm.


package repositories should be processed during build, not generate

Because the repositories are only required during the image build and they are specified through the configuration, I think it makes sense to copy them only when doing the build.  This will allow for generation of the image and details without including any sensitive information, e.g. using chained builds in openshift (generate -> build), where the generated code lives in an image that is fed into the docker build.
Therefore, if we're doing just a generate, we should key off the repositories section of the packages element and use that when generating the Dockerfile.  During the build, the user will have to copy the relevant files into the Dockerfile/../repos/ folder, as necessary.

Generate should prepare everything to build image. Target directory structure is considered implementation detail and no one should mess with it - we do not document how it works and one of the feature is that we can change how target directory is created in future version.
Also we support OSBS as different build and we will soon support buildah ( with kpod and cri-o maybe there will be no docker soon in Kubernetes/OpenShift) as build engine which can again change structure of target directory for these builders in future. Dockerfile is probably only of supported use cases and we should not design any specific work-flow to this.
User should be interested in sources and finalized image only. The generated stuff should not be altered as it can break reproducible builds.
Running concreate with same descriptor and same configuration should result in completely same image every time.

Agreed, but the repo files are not part of the image definition, but are injected via the configuration mechanism.  This differs from other artifacts used to create the image.
This issue was uncovered while putting together a template and chained build for use in openshift.  Because a two step process was used, generate -> build, it meant that the repo files would be stored in the image with the generated artifacts.  I'm guessing this is not desirable, since the idea is that the repo files would be contributed through a secret/config map.  I don't know how sensitive this information is.  The main reason for separating it is because it appears specific to the build and not the image (i.e. contributed via .concreate vs. image.yaml).

Specific repo files are probably needed only in our use cases, because we are using internal repositories instead of official RHEL ones. So we need to hide these urls. If you are on subscribed RHEL system on OCP installation - you shouldn't need any of these repo files, because you can just enable corresponding subscription/repositories on your RHEL host and this repositories are then accessible in containers.

We might want to consider how we manage these in the yaml files, since we're using --enablerepo with the repository name when we generate the image.  This would imply the repo name we use in the yaml file would need to match what's installed on the system.  Are these names standaradized?  Or, should we not be specifying repo names at all in packages?

This is something that will change, there will be no "--enablerepo" switches at all.

I think we do need to keep a way for adding additional repositories that may be required for building.  For example, a required repository may be disabled or non-existent on the host machine.

This is not doable. Some repositories will be always required, but at the same time these will not be public. This is something that will be solved using internal documentation.

That would address this issue then.

This issue should be resolved by repository changes in Cekit 2.0. WE now use all repositories that exists in the image plus the ones we explicitly configure via url/ODCS/package name.


Tests failure when building with nanobind 2.4.0

I got some Invalid type  ndarray received errors when running tests after accidentally built MLX with nanobind 2.4.0, not sure if it is a regression of nanobind.
======================================================================
ERROR: test_array_from_noncontiguous_np (test_array.TestArray.test_array_from_noncontiguous_np)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/Users/zcbenz/codes/mlx/python/tests/test_array.py", line 697, in test_array_from_noncontiguous_np
    mx_arr = mx.array(np_arr)
ValueError: Invalid type  ndarray received in array initialization.

======================================================================
ERROR: test_dlpack (test_array.TestArray.test_dlpack)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/Users/zcbenz/codes/mlx/python/tests/test_array.py", line 1808, in test_dlpack
    self.assertTrue(mx.array_equal(y, x))
                    ~~~~~~~~~~~~~~^^^^^^
ValueError: Invalid type  ndarray received in array initialization.

======================================================================
ERROR: test_diag (test_ops.TestOps.test_diag)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/Users/zcbenz/codes/mlx/python/tests/test_ops.py", line 2304, in test_diag
    expected = mx.array(np.diag(x))
ValueError: Invalid type  ndarray received in array initialization.

======================================================================
ERROR: test_put_along_axis (test_ops.TestOps.test_put_along_axis)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/Users/zcbenz/codes/mlx/python/tests/test_ops.py", line 1115, in test_put_along_axis
    idx_mlx = mx.array(idx_np)
ValueError: Invalid type  ndarray received in array initialization.

Hmm.. that's interesting. Non-contiguous NumPy arrays should be cast to contiguous by Nanobind.

When no perfect match can be found, nanobind will try each overload once more while performing basic implicit conversions: it will convert strided arrays into C- or F-contiguous arrays (if requested) and perform type conversion. T

It seems like a nanobind bug. We'll probably need to make a minimal repro and pass it to them to fix.

File an issue with Nanobind.. let's see what Wenzel says. We may need a fix in MLX depending or not if the behavior is the new default.


add a direct dlsym test

@bors r+

:pushpin: Commit c73c2a92342faa304f22ced629c442e36e4e59cd has been approved by RalfJung
It is now in the queue for this repository.

:hourglass: Testing commit c73c2a92342faa304f22ced629c442e36e4e59cd with merge 370a961611cb477e9b158f2166fe552de65a912e...

:sunny: Test successful - checks-actions
Approved by: RalfJung
Pushing 370a961611cb477e9b158f2166fe552de65a912e to master...


Settings Sync activity doesn't show specific User Tasks entry

Testing #141275

Open Sync Activity (Remote) view using the command Settings Sync: Show Synced Data
Make a change in one machine's tasks.json, change takes effect a few seconds later in second machine's tasks.json
❓ Expected to see User Tasks as one of the most recent entries, but only see UI State from today

I see User Tasks entries from sync testing yesterday, so thought I'd see it again today, especially as UI State doesn't mention tasks.json
Is it covered by globalState.json?

I refreshed the Sync Activity, and User Tasks appeared - is it expected for Sync Activity to automatically update (i.e. after 10 minutes, since the initial sync was 6 minutes ago, as shown below), or does the user need to refresh / run the command again?

It does not auto appear, you have to refresh manually. Since this is just for trouble shooting it is not aggressive to update.


Implement Control-Left/Right/Up/Down for Windows.

@amjith:
Implemented in https://github.com/jonathanslenders/python-prompt-toolkit/commit/c61b8dc7d25a7e7e0cd5ddfb044890a3c47861c7

:+1: Thank you!


Large and Distracting Scroll Bars in Navie Views

Description
The scroll bars in the Navie response pane, context window, and code block panes appear very large and distracting compared to the other scrollbars in the JetBrains UI and the scrollbars as they render in Visual Studio Code. These scroll bars are very bright, wide, and numerous, which creates a distracting and unfinished experience when navigating Navie responses.
Expected Behavior
The scroll bars in Navie views should match the appearance and behavior of scroll bars in the rest of the JetBrains Specifically:

Scroll bars should be less bright and more subtle.
Scroll bars should be narrower.
The overall prominence and visibility of scroll bars should be reduced to avoid distraction.

Actual Behavior

Scroll bars in Navie views are very bright and wide.
The scroll bars appear numerous and stand out compared to the rest of the IDE.
Changing themes in the IDE does not affect the appearance of the scroll bars in Navie views.

Impact
The large and distracting scroll bars give the experience of navigating Navie responses an unfinished feeling, making it uncomfortable and distracting for users.
Steps to Reproduce

Open JetBrains and navigate to a Navie chat.
View the response pane, context window, and code block panes within Navie.
Observe the appearance and size of the scroll bars in these panes.
Compare these scroll bars to those in the rest of the JetBrains UI and Visual Studio Code.

Environment Details
JetBrains: .72.0
Visual Studio Code: 0.125
Additional Information

With 2023.2 it's possible to style the scrollbar, https://youtrack.jetbrains.com/issue/IJPL-120103/JCEF-customize-scrollbars-in-JCEF,
but we can't use this yet because the plugin must be compatible with 2023.1 and later.
JetBrains is using https://kingsora.github.io/OverlayScrollbars/, it we want to keep compatibility with 2023.1, then we could perhaps  include this library and css instead of using CSS provided by the IDE?
But using the styling provided by the IDE would be preferred as we would not have to maintain our own solution.

On Slack it was discussed to put this on hold, for now.


SyntaxError: Unexpected token on line 28 : /node_modules/apn/index.js:28

/node_modules/apn/index.js:28
tls,
^
SyntaxError: Unexpected token ,
Can you help me fixing the issue?

@sivapc What version of Node are you running?

Node version is : v0.10.25

node-apn requires node v4 or greater. I suggest updating to an LTS release, node v4 or better v6: https://github.com/nodejs/LTS#lts-schedule

@florianreinhart : Thanks for your reply, I will update the version and check.


Add "Concurrency" section similar to that on the Queue storage

Would be nice to understand how concurrency is handled, similar to the "Concurrency" section on the Queue storage documentation.
I see that max concurrency is configurable in host.json, under serviceBus.maxConcurrentCalls, but it took me a little digging to find it.
Moreover, I'm not sure how the concurrency is determined, and what the relationship to scaling the functions is.
Example: If we get 200,000 queued messages in the service bus queue, will the function scale out, and the 16 be multiplied by each function? What determines how many are pulled at a given time, and when scaling would occur?

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

ID: 1d50399c-0873-e903-35ca-1845c84b6c6c
Version Independent ID: 1a3150b4-5eb9-0ac5-ee2b-d1bb6b6fd63e
Content: Azure Service Bus bindings for Azure Functions
Content Source: articles/azure-functions/functions-bindings-service-bus-trigger.md
Service: azure-functions
GitHub Login: @craigshoemaker
Microsoft Alias: cshoe

@redtaped Thank you for your feedback! We have assigned this issue to the author to review further and take the right course of action.

@jeffhollan - can you help provide some details on the concurrency rules here?


ios/safari not working

I've tried boilerplate and vanilla repo but it's not working with ios(have both MM & trust wallet).
Can you provide a working ios auth example?

Same, react-native boilerplate not working neither iOS nor Android. I get an Ethereum image logo overlay and that is it.

Post issues in https://forum.Moralis.io

Updated react-native in the boilerplate app.json to 0.63.4 and the app started


Is this project dead?

Last release is from 2015.
A lot of the plugins do not work anymore.
Many open issues.

What is going on here? Help needed?

This original project is dead. Look up Streamlink which has forked from this project and is being kept up to date with plugin fixes and all sorts of updates by many of the same developers who worked on this project.
On Apr 5, 2017, at 11:52 AM, Adam <notifications<EMAIL_ADDRESS>wrote:

Last release is from 2015.
A lot of the plugins do not work anymore.
Many open issues.

What is going on here? Help needed?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHubhttps://github.com/chrippa/livestreamer/issues/1602, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AESJ1kzl_TMaHPGrOExF61Nf5w7pnb0Nks5rs7iUgaJpZM4M0cu6.

@quantenschaum https://github.com/streamlink/streamlink
Yes, @chrippa has dropped off the face of the earth.


Adding three GIO terms Feb 22 2021

Also includes some accompanying subclassof changes and label changes

@pporrasebi
I've added a few terms and made some other minor changes


No SSR support

Workaround:
render component only after mounting

@goldylucks not sure if it is possible, does the typeform library allow it?
I'll try to do some research. Thanks!

Same issue.
Another workaround would be to check if window is defined and do nothing if it's not.
      if (typeof window !== 'undefined') {

      }

Too bad because the lib looks promising :/
@goldylucks What platform did you build?

@Vadorequest Hi! I'm facing the same situation with next.js. and so far getting this error
Uncaught TypeError: Cannot read property 'border-radius:0;display:block;height:2px;width:25px;position:absolute;right:6px;top:6px;' of undefined

Could you please share with your solution for embed typeform if you have any?
Thank you!

@dvakatsiienko My solution may not work for you, I figured that I didn't need to bother making typeform work with react, since I wanted to display it at the bottom of my page, I just followed the guidelines using "TypeForm Embed" documentation and ended up adding it outside of the react app, directly in the html body:

      You need to enable JavaScript to run this app.

Is there a way to do this in Gatsby @alexgarces ?

I approached it a different way that doesn't require any additional packages. It's true that just doing a regular 'import' of the package (and not rendering it until the client) will cause SSR errors. This package is amateur-hour by having 'window' references before the render of the component. Do they not care about SSR at all?
export default class TypeFormForm extends Component {
  static propTypes = {
    height: PropTypes.number,
    url: PropTypes.string.isRequired,
    buttonText: PropTypes.string,
  }

  static defaultProps = {
    height: 600,
  }

  constructor(props) {
    super(props)
    this.state = { Form: null }
  }

  componentDidMount() {
    const Form = require('react-typeform-embed').ReactTypeformEmbed
    this.setState({ Form })
  }

  render() {
    const { height, url, buttonText } = this.props
    const { Form } = this.state;
    return (

        {Form && (
          <Form
            url={url}
            buttonText={buttonText}
          />
        )}

    )
  }
}

So, basically, the react-typeform-embed package is not required/initialized until the client code is ready to render it. If you need more props to be passed through, do it. I just included the code that I use. This is only set up for an embedded form (no auto-open, full-screen, etc). Adapt the code to your site's needs and then just always use the wrapper. If you're using a full screen pop-up, you won't want to set the height on the containing div. I put that there so that the server renders the page with a space for the form so that it doesn't suddenly reflow the page when the form comes in as the client renders.

I've been working with Next.js for a long time. There is a work around for this for static site generators like Next.js. As we know window is not accessible in a server we can access it after the component is mounted in the client side.
Add the script CDN in the custom document js file (_document.js  in Next.js). I hope similar approach can be taken with other static site generators as well.
For eg. Gatsby - https://www.gatsbyjs.com/docs/custom-html/


Mustache parser should not need to be copy-pasta'd

0.5.0 introduces a new mustache parser:
https://github.com/machty/emblem.js/blob/0.5.0-beta/lib/mustache-parser.pegjs
This file is currently copy-pasted into the main parser. It should be imported.
PEG doesn't support an import syntax, but this plugin seems promising:
https://github.com/casetext/pegjs-import

Fixed in https://github.com/machty/emblem.js/commit/59e9f59d9721d5545920a4eecd5a381958d047e6


Convert sampled consumers to being Instructions instead of IDs

Changing the stored value for a sampled image consumer to be the
instruction instead of result ID, since not all instructions have
result IDs. Using result IDs led to a potential crash when using
OpReturnValue, which doesn't have a result ID. OpReturnValue is not a
legal consumer, but the validator needs to look at the instruction to
determine this, thus storing the pointer to the instruction, instead
of trying to fetch the pointer using the instruction.
Issue #1528 covers fixing the check.
Fixes #2463

Thanks for this. Can you add a test for this that validates till the other issue is resolved so we catch any regressions here.

Done, created a reduced case of the example provided in the bug.


Subsequent import of basic.json displays errors

Describe the bug
When you import basic.json for the first time into empty database it is completed with no issues.
Though when you attempt to run it one more time, the same basic.json, it will show a few errors like this:
Cannot insert duplicate key row in object 'dbo.CMS_WebPageUrlPath' with unique index 'IX_CMS_WebPageUrlPath_WebPageUrlPathHash_WebPageUrlPathWebsiteChannelID_WebPageUrlPathContentLanguageID_WebPageUrlPathIsDraft'. The duplicate key value is (AA3543E5BD7D7DDDB1AE5F40C5AD0B9B7DE56583EF678C6C57E62F8662B729D8, 2, 4, 0).
To Reproduce
Steps to reproduce the behavior:

Restore the clean database
Import basic.json
Import basic.json one more time without restoring the database
Inspect the import log for errors

Expected behavior
Subsequent import of the same data should not produce errors.
Library Version
1.9.0
Additional context
Below is the log file containing these errors when I run it locally:
import.log

Hi @diger74
fixed in 2.0.0.
Thanks for reporting.


Create Technical Documentation for doctors

The documentation should include every function that Clario Admin has to offer

Assigneed to Vasil Georov

Updated docs
https://help.myclario.com/


Fix README.md template

Codecov Report

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

@@           Coverage Diff           @@
##           master     #178   +/-   ##
=======================================
  Coverage   92.35%   92.35%
=======================================
  Files          85       85
  Lines        1072     1072
  Branches      179      179
=======================================
  Hits          990      990
  Misses         67       67
  Partials       15       15

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 83cecf0...007d081. Read the comment docs.


Add Option::into_iter()

I found it odd that the stdlib seems to use separate iterator types for
Option<T> and &Option<T>.  For this, &Option<T> reuses Option<T>
by calling option.as_ref().

Thanks!!


CreatedAt, UpdatedAt, Version are always Null

I'm trying to use this framework for sync as well as cache however the models returned by TableControlleralways have null for CreatedAt, UpdatedAt, and Versionproperties. Thus for as i understand it making sync impossible. I'm using a EntityDomainManager and Nuget Version <IP_ADDRESS>.
I believe my model has the correct properties so would expect them to populate correctly.
    #region System properties
    public string Id { get; set; }
    public byte[] Version { get; set; }
    public DateTimeOffset? CreatedAt { get; set; }
    public DateTimeOffset? UpdatedAt { get; set; }
    public bool Deleted { get; set; }
    #endregion

Digging deeper I tried to use a MappedEntityDomainManagerinstead however it actual had trouble writing the objects headers for the response due to the Version Tag.resulting in this error.
    message: "An error has occurred.",
    exceptionMessage: "The value cannot be null or empty. Parameter name: tag",
    exceptionType: "System.ArgumentException",
    stackTrace: " at System.Net.Http.Headers.EntityTagHeaderValue..ctor(String tag, Boolean isWeak) at Microsoft.Azure.Mobile.Server.Tables.TableQueryFilter.GetETagFromVersion(Byte[] version) at Microsoft.Azure.Mobile.Server.Tables.TableQueryFilter.AddETagResponseHeader(HttpResponseMessage response)

i can make it not error by setting the version byte array field to anything, just for testing.
    //TODO: Hack, don't do this, test only
    result.First().Version = new byte[] { 0x1 };

Am i doing something wrong or am i misunderstanding something basic. What experience are others having?

I think I've solved my problems with the Azure Mobile SDK Syncing. My problem was two fold.
First I failed to understand that the UpdatedAt, CreatedAt and Version will all be null on the client untill the objected inserted on the server. The server database will generate these properties and return them to the client with the sync.
Second make sure all the correct Attributes are populated on the Server Model. This is really important as if you are using EF Code First this affects SQL Triggers and Constrains that populate the CreatedAt, UpdatedAt Columns. As well the Version Column is a timestamp type.
In my case my code first was not generating the trigger and EF didn't see the columns as Computed so it was inserting null into these fields.
Azure Sync Client Model
Azure Sync Server Model
Currently i'm working on a DB First solution and I'll try to follow up with more infomation to help others not doing code first EF.

Thanks Chris.  I'm glad it's sorted.
Feel free to ping me offline with your observations.   We can always put a wiki article together around this topic.

Having the same issue - tried to remove the fields completely (in my server & client solution) as documented. Since that did not work, I tried to use @ChrisTowles solution - no luck, I always get complaints about trying to insert NULL as CreatedAt. Any ideas?

Was able to solve it, did not follow this template for my DB since creating from existing tables.
CREATE TABLE [dbo].[ConstructionSite](
[Site_Heads] [bigint] NULL,
[Id]NVARCHAR (128) DEFAULT (newid()) NOT NULL,
[Version] [timestamp] NOT NULL,
[CreatedAt] DATETIMEOFFSET (7) DEFAULT (sysutcdatetime()) NOT NULL,
[UpdatedAt] datetimeoffset NULL,
[Deleted] [bit] NOT NULL,
CONSTRAINT [PK_dbo.ConstructionSite] PRIMARY KEY NONCLUSTERED ([Id] ASC)
);
CREATE CLUSTERED INDEX [IX_CreatedAt]
ON [dbo].[ConstructionSiteDBs]([CreatedAt] ASC);
GO
I did not find this in the documentation (might be my fault). The documentation should mention somewhere what the schema is supposed to look like if not created automatically.

When I was debugging locally my code first was able to create the database. After publishing for some reason it keeps timing out even though I increased both the connection and command timeouts.
I went to just script the table over but initially forgot to grab the trigger which generates these values so that was my mistake.

Good documentation for this feature: https://adrianhall.gitlab.io/develop-mobile-apps-with-csharp-and-azure/chapter3/server/
In addition, there has to be Trigger for UpdatedAt as well
CREATE TRIGGER [TR_dbo_Examples_InsertUpdateDelete] ON [dbo].[Examples]     AFTER INSERT, UPDATE, DELETE AS     BEGIN         UPDATE [dbo].[Examples]             SET [dbo].[Examples].[UpdatedAt] = CONVERT(DATETIMEOFFSET, SYSUTCDATETIME())             FROM INSERTED WHERE inserted.[Id] = [dbo].[Examples].[Id]     END;

@ChrisTowles do you have an example of what you did....I am facing the same issue as you.  I am using code first.  I am absolutely confused on what my models should look like on the client vs the server involving attributes.  I saw that some of yours are TableData???


Support better directory structure

@carymrobbins suggested that the directory structure should look something like this:
example/
├── example.cabal
├── Setup.hs
└── src/
    ├── main/
    │   ├── eta/
    │   │   └── Example/
    │   │       └── Transform.hs
    │   └── java/
    │       └── Main.java
    └── test/
        └── eta/
            └── Example/
                └── TransformTest.hs

The layout is a lot better than the existing layout, which places the .cabal file inside of src/main/eta.

I  always forget this, but the Main.java probably needs to go in the src/main/eta directory. Build tools usually try to compile all of the src/main/java sources with just javac and before any other sources, so if Main.java references anything from src/main/eta it'll fail to compile. Maybe sbt is different, but either way this is a potential gotcha.

So in these inter-project setups, there's always the question of where the dependencies flow. SBT has a mechanism which allows you to select whether you want to compile Scala or Java dependencies first. We probably need to add a new option that chooses the order among, Java, Scala, Eta (in case anybody wants such complicated setups).

In this particular case, it would make sense to disable javac building altogether and add the following to the cabal file -
java-sources: src/main/java/Main.java


Add getgrgid, getgrnam, getgrnam_r and getgrgid_r for emscripten

I added getgrgid, getgrnam, getgrnam_r and getgrgid_r for emscripten.
Header sources:
getgrgid
getgrnam
getgrgid_r
getgrnam_r

:umbrella: The latest upstream changes (presumably 7cab757d7b090db039cdc3782a1ec81333113ccf) made this pull request unmergeable. Please resolve the merge conflicts.


doc: Add documentation for data structures

Add a new document describing the list, tree and ring buffer data
structures under lib/os.  We had doxygen references for this stuff,
but never had an overview document.
Signed-off-by: Andy Ross<EMAIL_ADDRESS>

Split out the mergeable data structure documentation from #19201, which is blocked on the timeout rework that it documents.  This should be able to go in right now.  No other changes.

Think I fixed all the editing notes.

Closing. Stale.


Convert the request body correctly when it is a String

so hashie coercion doesnt error when it retrieves and the requests have body content.
As currently if it retrieves a request with body content it errors out with the following;
Cannot coerce property :body from String to MockServer::Model::Body: undefined method `each_pair' for "Hello this is a message":String

It appears that the mockserver java core doesn't return the type of body and this appears to be implied by this java spec - https://github.com/jamesdbloom/mockserver/blob/master/mockserver-core/src/test/java/org/mockserver/model/HttpRequestTest.java#L96-L114
My gut feeling tells me it ideally should returned from the overridden toString methods in all the classes extending Body like StringBody class below
https://github.com/jamesdbloom/mockserver/blob/master/mockserver-core/src/main/java/org/mockserver/model/StringBody.java#L65-L68

@jamesdbloom on second thoughts this doesn't seem write if a request body is of type binary then this would break that (not that it is working currently) - so I guess the mockserver request response should also include the bodyType ?

When the body is a binary it should include the bodyType, only bodies which are strings support the shorthand of no bodyType.


feat: implement delete_language in client

Closes #206

Failing due to https://bugs.limesurvey.org/view.php?id=17873.
Possible fix in https://github.com/LimeSurvey/LimeSurvey/pull/2235


Using AVX-512

Looking at the code, I see that only vector species up to 256 bits are used.
Is there a reason why 512 is not supported?
Asking because with the advent of AMD's Zen 5 architecture, we finally have true AVX-512 support on consumer grade machine, so it would be very nice to use for Llama3 :)

512-bit vector support (AVX512) added in 45301fc98c9aef8862e0e2fe17f15987c2c8f236.
Use -Dllama.VectorBitSize=0|128|256|512 to force the vector size, 0 to disable.
The JIT compilers in the JVM will pick up AVX512 automatically. For native-image the -march=native|x86-64-v4 option is required (already included in make native).
If you, or someone, has access to an AVX512 CPU, I'd love some feedback/benchmarks/comparisons for the different vector sizes.

Thanks!

I don't yet have access to such a machine, but I hope to get one in the coming months


Variable Template Query Tag Values does not work for each bucket

Tools US 2.x Cluster
Dashboard Log File Studies V1.x - Shards
Variable shard defined as query:
import "influxdata/influxdb/v1"
v1.tagValues(bucket: v.bucket, tag: "shard_id")
The above sometimes works and sometimes does not based on the bucket.  For the following bucket, I actually created a cell to show that the value appears valid.
https://influxdb.aws.influxdata.io/orgs/844910ece80be8bc/dashboards/0569ce471d8dd000?lower=2020-04-01T01%3A00%3A00.000Z&upper=2020-04-01T02%3A00%3A00.000Z

I have run through a few dozen buckets (log files from customers) and sometimes the drop down works for the variable, other times it fails.  The l-creditkarma-apr bucket does not enable the drop down but the tag value list should just have the 10 numbers as shown above.

Ping me if you cannot repro the problem or if the bucket shown above is gone.

The shard variable is a query type variable that depends on another variable (v.bucket).

dupe of #17593

while more stuff started showing up, this didn't get fixed in the tools cluster. Check out the buckets variable in the data explorer as an example, also.. the shard value dropdown isn't loading even though the v1.tagOutput For shard is showing up


Implement Multi-Language Support for the Website

Description
The website is currently available only in English, which can be a barrier for developers who are not fluent in the language. To make the website more accessible, I propose adding a feature that allows users to select their preferred language via a button, dropdown menu, or another selection method.
Proposed Solution
I suggest implementing this feature using the i18n framework, which leverages a collection of JSON files for different languages. Each language would have its own JSON file, and the website would automatically translate based on the user’s selection.
Additionally, we could consider implementing automatic language detection based on the user's location. However, it might be preferable to give users the option to enable or disable this feature.
Contribution:
I'm willing to work on this feature if approved.

@rodriveiga01 sure go for it, it'll enhance the accessibility greatly

Just need to know what languages should I put. Maybe if there's a lot I should put all of them in one file. Let me know what languages do you want the website to be translated

In the meanwhile I will start to implement the logic

@rodriveiga01 At first just implement whatever you are comfortable with, after that we can discuss what other languages to include


Fix serialization and deserializattion in DF classification

Description
Fix serialization and deserializattion in DF classification

make test

make test

make test


Amazon web-scraper

Aim
Recieve an email when the price of an Amazon product drops below your budget
Details
This script inputs three things- the link of the Amazon product, your budget for it and how frequently you wish to monitor its price. That's it- keep the script running in background and you will receive an email alerting you when the price is within your budget!
PS- The exact instructions to do so will be provided in the README.md

Hey, I would like to contribute to this issue. Could I please be assigned the same?
Thanks and Regards
Sarthak Saxena

@sarthak1905 Assigned to you.. Happy Coding..😊


API token expires and does not refresh

Hi, I have been having some issues recently with the API token stopping refreshing. I saw there were some commits related to the token refresh, and so this afternoon I have also tried running the most up to date code from here, but the issue persists. I can only get a single update by reloading the integration. It identifies that the API token will expire in 5 minutes and then (as expected) it expires 5 minutes later and nothing updates until I reload the integration again.
Any ideas? Happy to provide the relevant logs if someone signposts me to where the most useful ones are. Thanks!

Digging into this a little more, practically every second line of my debugging logs is "Skipping update, already in progress"
Looking at the code (as a total noob) this looks like it gets caught at:
        if not self.updating.acquire(blocking=False):
            self.logger.debug("Skipping update, already in progress")
            return

which prevents it from ever getting to the refresh token step:
       try:
            if self.auth.need_token_refresh():
                await self.auth.get_token(refresh=True)

The only other thing notable thing I can see in the logs is:
2024-11-09 14:18:15.036 ERROR (MainThread) [custom_components.polestar_api.polestar] Unexpected Error on update data 'PolestarAuth' object has no attribute 'logger'
But my custom_components/polestar_api/pypolestar/auth.py does have:         self.logger = _LOGGER.getChild(unique_id) if unique_id else _LOGGER from dd7f27e.

I'm working on cleaning up updates now, hope to have a new release tomorrow.

Pre-release 1.9.0rc1 now published

Unfortunately it does not seem to fix it. No updates on the sensors after initial loading.

maybe its helpful having a look over here?
https://github.com/evcc-io/evcc/issues/17186

Yes, this has already been addressed. Some issues with caching left, sorting that out now.

https://github.com/pypolestar/polestar_api/releases/tag/1.9.0rc3 released, please test


Hero Animation Not Working As Expected

I thought the Hero was a visual transition from one element into the next page. I expect the thumbnail image to visually transition from the small thumbnail into the fullsize image when expanded. also it should reverse so you can see the image shrink back down into the thumbnail.
how do we get this working?
thanks

Could you explain in more detail what you mean by that? Is it like this image?

yes like that.  you can see it go back "home".
also, say you initially open on book 2 thumbnail to expand the view. then scroll to the 18th book, when you dismiss, the list of thumbnails should also follow so you can see it animate down into the 18th thumbnail 👍

I haven't tried it yet, but I think you can make the same thing with regular hero animation.
Say, you have a list of thumbnails, and tapping one of them navigates to its details page.
Hero(
    tag: "thumbnail",
    child: YourSmallThumbnailWidget(
        onTap: () => showPageViewAsDialog(),
    ),
),

You can use showModalExprollable function to show a page view as a dialog.
showModalExprollable(
    context,
    builder: (context) {
    return ExprollablePageView(
      itemBuilder: (context, page) {
        return SingleChildScrollView(
          controller: PageContentScrollController.of(context),
          child: Column(
              children: [
                  Hero(
                      tag: "thumbnail",
                      child: YourLargeThumbnailWidget(...),
                  ),
              ],
          ),
        );
      },
    ),
  },
);

I will try to make an example of a Hero animation like this one this weekend.

The animation on the package page looks like it's working as expected.
Which animation is it? No hero animation is used in this package.

oh i'm sorry, i mean in your complex example you use a hero animation. and the image above shows the thumbnail animating but when i run the example it does not

This one? If so, this example doesn't use a hero animation at all. Expanding the page to full screen is not a hero animation, but a scroll animation using the Transform widget. And the image above is not part of this package, it is a snapshot of the Apple's Books app that this package tries to imitate. Sorry, that's confusing.

got it. so i think a hero is what you need if you want it to really look and feel like the books app. you see how the animation from thumb to expanded is? that gives visual feedback about what's happening. i think an updated example with hero animation would polish the look and feel quite a bit

hi, did you get that hero demo working?

Hi, sorry for the late reply. I tried a little bit and found it to be more time-consuming than I expected. I will work on it tomorrow.

ah no big deal :)

@jtkeyva Hi, I have added an new class to support hero animations and its example usage to the dev branch. Please try it, and if you have no problem, close this issue. If you have any problems, please let me know here again. It is very helpful.
Until the next release with the new features, the following must be included in your pubspec.yaml.
dependencies:
  exprollable_page_view:
    git:
      url: https://github.com/fujidaiti/exprollable_page_view.git
      ref: dev
      path: package/


Removal of live fire map from titan tag

I noticed that Stacks is listed under Titan Tag so I removed it :)
Might also fix #386

Hey, now I can remove my workaround in Modevote


Push http metrics with statsd

Push metrics, such as response time, response count ..., to a statsd server.
First part of #102
Description
Besides the default configuration, there is a negroni middleware to push http metrics to statsd.
Motivation and Context
Have control metrics of individual parts of the architecture, e.g.: response times, individual processes breakdown, project health in general.
How Has This Been Tested?
An statsd server started locally, checked if the metrics are sent.
Types of changes

[x] New feature (non-breaking change which adds functionality)

Checklist:

[x] My code follows the code style of this project.
[x] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[ ] I have added tests to cover my changes.
[x] All new and existing tests passed.

Thanks, this is awesome to have some statsd support! Is it compatible with datadog?

@zhouzhuojie it is, but it doesn't have all the features, e.g.: tags

And another thing is to add such metrics to individual parts of the code, such as, calling database/kafka calls, so we can track down any possible latency or spikes. But I would probably cover that on another PR.

@zhouzhuojie updated.

Using Datadog client (supports tags and it is backwards compatible with other statsd)
Use tags for http status, url and method (allows better filtering later on, e.g.: duration of gets to /foo/bar)

I was considering moving the middlewares to sub-packages on their own files, since the file is getting a bit big and there is a lot of logic that would be better handled on individual files. But, I probably will do that on another PR.

Thanks @marceloboeira! Left one comment, and I think it's good to go!

@zhouzhuojie yeap , no problem.

@zhouzhuojie done.
I don't know why the dep ensure didn't work the first time.
btw, I started using the statsd metrics on a couple of pods running in production yesterday, pretty neat:


Export sign function

This is a request that the sign function be exported so it can be used by services that want to generate a cookie (like creating and driving a headless browser session for screenshot/PDF generation). Currently, we're doing this by having our own function that matches the behavior, but this is fragile and it would be nice to be able to use the actual function.
Follow on from this issue: https://github.com/expressjs/session/issues/805

Hi Dave, both the sign and unsign functions are exported by this library, e.g.
import {sign} from 'cookie-signature';

let result = sign("thing", SOME_SECRET);
// …use `result`

should already work. Feel free to re-open if I've misunderstood your request.

Hi Dave, both the sign and unsign functions are exported by this library, e.g.
import {sign} from 'cookie-signature';

let result = sign("thing", SOME_SECRET);
// …use `result`

should already work. Feel free to re-open if I've misunderstood your request.

🤦 sorry for the noise

🤦 sorry for the noise


Cordova - Re-sign app & upload new release fails

What App Center service does this affect?
Cordova Re-sign app & upload new release constantly says validating.
Describe the bug
Cordova registering 2 unprovisioned devices for a new release fails when Re-signing the app. Constantly says validating.
To Reproduce
Steps to reproduce the behavior:

States "There are 2 unprovisioned devices"
Click on "Register Devices"
Check Resign App
Click Register

Expected behavior
App should Re-signs and I can distribute app to these users.
Screenshots

Desktop (please complete the following information):

OS: MacOS High Sierra
Browser Chrome & Firefox

Hi @ciaranmccaughey - thanks for opening an issue! Can you start a support thread via the blue chat bubble on the bottom left? This will put you in touch with our support and engineering team so they can help troubleshoot.
In the meantime, @Oddj0b can jump in if there are any suggestions.

@ciaranmccaughey did you get help through our support channel?


Feature: Replace bundles with tags widget

While tags and bundles are a little different, they are similar enough that we can remove bundles and replace it with a tags widget. This will lead to less duplication and will provide a more useful user experience.
Requirements

Rename bundles and related settings to "Tags"
Each tag should display a widget on the home page
Each tag should have a button to navigate to a page with the full list of files

Discussion points

Should all files be displayed in the widget or just the first 10-15?
Should there be a menu option to add the tag to a specific file similar to bundles? (I think we should try without this for now)
Should there be an option to create a new tag from the widget? (I think we should try without this for now)
Should existing bundles be ported over to tags and if yes, should users be prompted, or should it happen automatically?

Should all files be displayed in the widget or just the first 10-15?

@mdtauk do you have any input on this?

I can see the logic of using existing tags to replace the "Bundle".  My initial thinking is that people may see tags as an organisational tool, but see a Bundle as a group like Quick Access or Favourites.
If someone makes a tag specifically for having quick access to files or folders on their home screen - will they be annoyed to see that tag showing up in various folder views?
Also, could the tag group/bundles be responsive, so displaying more items (or displaying their items differently) depending on the space for them?
Do we introduce the idea of perhaps special tags for displaying on the Home screen maybe?

If someone makes a tag specifically for having quick access to files or folders on their home screen - will they be annoyed to see that tag showing up in various folder views?

My hunch is that users wanting to pin items on the home page will be okay with the tag displaying in other views as well. At the very least, they can hide the tag column if they don't want it displayed.

Also, could the tag group/bundles be responsive, so displaying more items (or displaying their items differently) depending on the space for them?

The first iteration will have a fixed size, if we do add support for custom sizes in the future, we'll still need to decide on the max items to display.

Do we introduce the idea of perhaps special tags for displaying on the Home screen maybe? (like a pre-defined Home tag)

How would this work if users want multiple tags on the home page?

I think they can both serve separate purposes


StyleSheet.create and other refactors

This is the other set of followups from https://github.com/zulip/zulip-mobile/pull/4101#issuecomment-669814479 where I merged #4101: some small refactors to the ZulipButton component, and then that led me to some digging into StyleSheet.create and responding to an unpleasant discovery there, and to a bit of Flow metaprogramming.
(I actually still have some further changes following up on these, going deeper into tools for the Flow metaprogramming with some of the things I learned while writing this SubsetProperties. Those aren't quite ready and I'll send them separately.)

Thanks for the review!
Yeah, the added blank lines were not intended. I noticed a couple of them while developing the change -- they were actually introduced by eslint --fix, as when it deduplicates two imports from the same module:
import styles, { ThemeContext } from '../styles';
import { createStyleSheet } from '../styles';

->
import styles, { createStyleSheet, ThemeContext } from '../styles';

it leaves behind a blank line where the removed import was. Seems like a bug in that ESLint rule, though not a super bad one.
I'll fix those up and merge.


Proper validation for models on app or models namespace

Fix for #97.
This PR validates Models on \App\ namespace and \App\Models\ namespace.
If both exists user is prompt to select wish one to use.
I had to move all Artisan::call() to $this->call() to be able to use prompts.
Controller already exists!

Multiple models with this name were found, which one do you want to use?:
  [1] Use App\Test
  [2] Use App\Models\Test
 > 1
1

Model already uses CrudTrait.
Request already exists!
Route already exists!
Sidebar item already exists!

Also, by using $this->call() messages are properly highlighted.

I agree with this, thanks @promatik !
I've made one change, in terms of output. Let's use:

green text for when things were done successfully  - $this->info()
yellow text for when things have not been done because nothing was needed  - $this->comment()
red text for when things have NOT been done successfully so "Houston we have a problem" - $this->error()

I think it makes for a more informative output:

I agree with this, thanks @promatik !
I've made one change, in terms of output. Let's use:

green text for when things were done successfully  - $this->info()
yellow text for when things have not been done because nothing was needed  - $this->comment()
red text for when things have NOT been done successfully so "Houston we have a problem" - $this->error()

I think it makes for a more informative output:

Merged and tagged! Thanks @promatik .
When testing this I did discover a small inconvenience that remains when using Models both outside and inside app/Models: while it DID use my App\User model to add the CrudTrait to it, my UserCrudControler still referenced App\Models\User instead of App\User.
This is a minor problem though. And one that I think will be less and less relevant as we drop support for old versions of Laravel. Since people are nudged towards using App\Models by Laravel 8, I think everyone's going to do that going forward. Also, it would be kind of difficult to implement a solution to it since they're created by different commands, so I don't think it's worth working on it at this time. I did want to mention it though, because it's a thing, and we might want to do something about it in the future 😀
Cheers! Good work!

Merged and tagged! Thanks @promatik .
When testing this I did discover a small inconvenience that remains when using Models both outside and inside app/Models: while it DID use my App\User model to add the CrudTrait to it, my UserCrudControler still referenced App\Models\User instead of App\User.
This is a minor problem though. And one that I think will be less and less relevant as we drop support for old versions of Laravel. Since people are nudged towards using App\Models by Laravel 8, I think everyone's going to do that going forward. Also, it would be kind of difficult to implement a solution to it since they're created by different commands, so I don't think it's worth working on it at this time. I did want to mention it though, because it's a thing, and we might want to do something about it in the future 😀
Cheers! Good work!


remote mode does not currently work for ungrouped rows

Issue Type (delete the irrelevant ones)

[x] Bug

Specs
What version are you using?
Latest (v2.0.3)
What browser?
Firefox
Expected Behavior
What did you expect to happen?
I was trying out the new undocumented remote mode. Using the remote mode property, I expected to be able to provide the rows property as a regular array.
Actual Behavior
What actually happened?
No rows showed up, I had to format the rows parameter as an array of grouped rows.
Steps to Reproduce the Problem

Set the mode property to remote
Provide an array in the rows property

jsfiddle
https://jsfiddle.net/57a2a4ce/135/
Additional info
I believe that flattening the rows should be happening after the mode check in the paginated computed property: https://github.com/xaksis/vue-good-table/blob/new-build/src/components/Table.vue#L477..L484

whoops! this is clearly a bug... thank you for pointing it out. I'll check in the fix shortly

this is now fixed in v2.1.0. @skray remote mode has not been through a litmus test yet, I plan on using it myself over the next few days, so there might be more changes coming soon.
closing this for now.


UK is eu True

IP addresses from united kingdom are returning country_eu: true although UK has exited the EU. Can you please update. Thanks

The database provider addressed this recently. Websites using Maxmind will reflect this change once they update their copy of the database. It is not something echoip has direct influence over. Once the new database version is pulled the issue will disappear.
You can see it is gone in the new version of the database:
curl -s https://eth0.dev/json?ip=<IP_ADDRESS> | jq
{
  "ip": "<IP_ADDRESS>",
  "ip_decimal":<PHONE_NUMBER>,
  "country": "United Kingdom",
  "country_iso": "GB",
  **_"country_eu": false,_**
  "region_name": "England",
  "region_code": "ENG",
  "zip_code": "EC2V",
  "city": "London",
  "latitude": 51.5164,
  "longitude": -0.093,
  "time_zone": "Europe/London",
  "asn": "AS20473",
  "asn_org": "AS-CHOOPA",
  "hostname": "<IP_ADDRESS>.vultr.com",
  "user_agent": {
    "product": "curl",
    "version": "7.64.0",
    "raw_value": "curl/7.64.0"
  }
}


iceshelf: init at unstable-2019-07-03

Motivation for this change
iceshelf: init at unstable-2019-07-03
Things done

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

[X] NixOS
[ ] macOS
[ ] other Linux distributions

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

@jonringer good point, however, if the package works fine with python2, I would prefer to leave it up for the distribution to make that decision. Just because the upstream will stop the support doesn't mean all the distributions will, therefore I don't feel comfortable to force this on python3 unless I have to (I tested it on python3 too tho).

Just mentioned it because https://github.com/mrworf/iceshelf/wiki/Installing-iceshelf doesn't really mention anything about being tied to a particular version, and I'm disillusioned with python2 and it's all it's quarks

I think it would be better to just use python3.pkgs in the expression.
I don't think we lose anything by using python3 by default since it's just an application.
Eventually we'll have to switch it over anyway.

Thank you, rebased.


Unpin numpy

Summary
docplex 2.28.240 was released on Aug 2 with numpy 2 support. So, I unpin the numpy version
https://pypi.org/project/docplex/2.28.240/#history
requires #630 to pass CI
Details and comments

Pull Request Test Coverage Report for Build<PHONE_NUMBER>8
Details

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 92.857%

Totals

Change from base Build<PHONE_NUMBER>1:
0.0%

Covered Lines:
4446

Relevant Lines:
4788

💛  - Coveralls

Yes, let's backport it.


Fix main.rs for rustc 1.17.0-nightly (0e7727795 2017-02-19)

Please update your travis scripts to the latest.


feat: DaemonSets best practices on Autopilot tutorial

This PR contains manifest files needed for the tutorial on running DaemonSet on GKE Autopilot using best practices recommendations.

@TheRealSpaceShip please rebase and merge

@TheRealSpaceShip this is still ready to be merged whenever it's ready to go, just let me know

@bourgeoisor this PR is done and ready to be merged.


Update deps

Hi,
i found that dependencies here were pretty old, so i have bumped all where i found that there are not breaking changes, And after running grunt it seems that everything is fine.
I have also
Please let me know if there is anything else you would like to do here.
Best,
Dejan

Coverage remained the same at 98.734% when pulling 5e12ca6f4941349d43ba94e2907a8d55671e69a2 on dejanr:update-deps into 07c6ebecb7dd7fe1d84135591dca46b710873427 on yaniswang:master.

Coverage remained the same at 98.734% when pulling 6bdf71589fa0cf77df18663d57009bc6008cfb8f on dejanr:update-deps into 07c6ebecb7dd7fe1d84135591dca46b710873427 on yaniswang:master.