Add .net core build and nuget package

error: Package RunProcessAsTask 1.1.3 is not compatible with netcoreapp1.0

Thanks for the repo! Very helpful :)

@ajbeaven FYI, I'm looking around and trying to figure out how to add support for .net core now - certainly nothing in the current code is Windows-specific AFAIK (doesn't use shell execute, for instance), but I haven't targeted .net core before.

Legend, thanks for that! Yes, I've currently just copy pasted the source code and everything but the securitystring was supported it seemed. Fortunately I didn't need that so stripped it out.

@ajbeaven nice!  I'll stick that overload in an appropriate #if once I can figure out the rest of it. 😄

After looking into this more and starting down the path of getting a .net core build working (with a functional project.json and a new csproj that target .netstandard 1.6), since there's a pretty simple current workaround, I'm going to hold off on having this target .net core until VS15 goes RTM since my understanding of the tooling is that at that point I should be able to fully target .netstandard 1.6 from just the csproj and won't need to create/maintain project.json (which is going away).
The VS 2015 tooling isn't horrible, but there's a bit too much hackery with the tooling as it stands and I'd rather use the better tooling and target .netstandard the Right Way since what I've got building now seems to work, but I'm a bit fearful would do more harm than good

need to go edit project properties after creation to change from targeting portable to .net standard, then need to change version to 1.6 https://docs.microsoft.com/en-us/dotnet/articles/core/tutorials/target-dotnetcore-with-msbuild
need <NuGetTargetMoniker>.NETStandard,Version=v1.6</NuGetTargetMoniker> added near end of csproj https://github.com/dotnet/roslyn/issues/12918

Hopefully VS15 RTM will happen soon enough - it just feels like at the moment it's a lot more work and duct tape to get this up and working than it will be once VS15 is out and if I can avoid having to adopt project.json just to turn around and abandon it, I'd rather just wait a little. 😄

Totally understand and really appreciate you taking the time to look in to it.

@ajbeaven since my earlier effort to hold off on this was until Visual Studio 2017 was RTM and that has now happened, I'm revisiting this and hoping I can get a .NETStandard 1.6 build going. 😄

Great, thanks James!

@jamesmanning If it helps with the exeption to the use of SecureString the entire codebase is compatable with .net 1.x.

@MaxCheetham
Seems like SecureString is available in .NET Standard based on this:
https://github.com/dotnet/corefx/issues/1387
https://www.nuget.org/packages/System.Security.SecureString/
But, not available in .NET Standard based on the docs:
https://docs.microsoft.com/en-us/dotnet/api/system.security.securestring?view=netcore-1.1#remarks
Have you tested it or have additional info?

Interesting, I can test after work tonight?

@bbrandt looking at their implementation linked from the issue, no actual encryption seems to take place, so at the moment I'm much more likely to just kill off that overload since it seems to be more trouble than it's worth, and the (hopefully few) people that really need a SecureString version can always just use Process directly.  Does that sound ok?
@MaxCheetham that seem ok?

@jamesmanning yeah that seems fine, people can always write there own extension method if needed. Is there anything you need a hand with?

@jamesmanning I'm good with that decision.

@MaxCheetham honestly? yeah, a PR to modify/recreate this as a .net core project would be wonderful 😄 AFAICT the resulting nuget package would only need to target .net standard 1.0 and not multiple targets (keeping around a net46 target for instance) but I haven't actually verified that. I'm happy that NuGet 4.x now lets us do away with the nuspec too. Depending on VS2017 would be fine with me if that makes it easier. 😄

@jamesmanning Sure I can do that. i'll look into NuGet for Core, any pointers?

@MaxCheetham i just have the link I had started with @ https://docs.microsoft.com/en-us/dotnet/articles/core/deploying/creating-nuget-packages

@jamesmanning What branch do you want the pr aimed at?

@MaxCheetham master's fine with me

@jamesmanning Any chance you know how to force appvoyer to use msbuild version '15.0'?

Use the Visual Studio 2017 image?  Maybe?
https://github.com/appveyor/ci/issues/1179
https://github.com/lexicalunit/nanodbc/issues/247
environment:
  matrix:
  - platform: x64
    APPVEYOR_BUILD_WORKER_IMAGE: Visual Studio 2015
  - platform: x64
    APPVEYOR_BUILD_WORKER_IMAGE: Visual Studio 2017

@jamesmanning  #12 is locally passing the test suit whilst running netcoreapp1.1. I'm pretty sure that the integration tests on here are failing due to their config. Can you offer any advice?

@MaxCheetham bear with me since I'm still quite new to all this.
TBH what I expected to have happen was that the project would change to be netstandard1.0 instead of netcoreapp since I think that means it can continue being consumed by .NET (at least 4.6?) but also be consumable by .NET Core.  My understanding (which may be wrong!) is netcoreapp would limit its usefulness and we'd be stuck going down the path of multiple builds for different frameworks, which I was hoping to avoid since they did all that work to create the .NET Standard and I'm hoping that means we can avoid the multiple-targets route.
Ignoring that for the moment, if things are passing for you locally and the issue is just the CI hosts not supporting .NET Core at the moment, I wouldn't worry about continuing to bang on it.

@jamesmanning righto. I'll give the .yml's a look into over the weekend and try and get to the point where we are all green on integration tests. Sorry about the .NET Core/ netstandard1.0 mixup. I'm more than happy to look into that over the weekend.
What would your suggestions with the pr be moving forward?

@MaxCheetham if netstandard1.0 is possible (which API-wise I believe it is), then that'd be a great target for the weekend.
The use case I'm most worried about is one I think should work but I'm not 100% sure.

User is currently using the existing nuget package with a .NET Framework app/library
User upgrades to new version which only targets netstandard1.0
Everything Just Works for them (this might require that they're using 4.6 for netstandard1.0 to work, but I'm not overly worried about that at the moment)

@MaxCheetham as long as the 'existing version nuget consumer' scenario is working (and, of course, '.net core nuget consumer' since that's what we're enabling 😄  ) and the tests pass locally, I'm not as worried about the CI's failing.  It's great if you can get the CI's working as well, but that's a lower priority IMHO than the nuget package use cases, especially since the CI setups are, at least currently, just testing the code and not a built nuget package, which is what I'm more worried about breaking at the moment. 😄

@jamesmanning unfortunently System.Diagnostics.Process has a requierment of .NETStandard, Version=v1.3 #c72d266 is now supportive of netstandard1.4. So if you want to package that up then the tests are green on my local.

So, I'd recommend merging soon enough and doing some post-work.
There's a few things to consider, which don't all need to wait.
Hand-editing: First of all, it's more normal now (and easier) to edit the project files by hand.  You're going to want to be able to do that because the UI isn't always very good; sometimes it's downright misleading (it's clearly still playing catch-up).  And to do that, it's important for the project file to have a limited amount of "gunk".  The default project files the old tooling generated had lots of gunk; most you can simply throw away.  #12 is already pretty slimmed down, but I'd also get rid of all the conditional stuff you can.  You need very little.
At least consider multi-targetting: Sure, in theory you can target .net standard 1.x and support both .net core and .net framework, but in practice, that provides a pretty poor .net framework experience for two reasons.

Firstly, for some reason I don't fully understand, .net standard support has a high .net framework requirement.  even v1.3 means a minimum .net framework of 4.6, and if you want  1.6 (the highest version supported by .net core 1.0, which is still a considerably smaller api than the full framework), then you've raised the bar to .net framework 4.7.  Basic reflection support was added in 1.5, so since you can avoid that dependency, this applies less to RunProcessAsTask.
Secondly, if you reference a .net standard lib from a .net framework project, that means adding lots of dependencies on "core system" assemblies from the .net standard stack.  When I updated a very simple project I maintain to be based on .net standard 1.1, somebody noted that they weren't happy with seeing lots of extra dependencies in nuget, and needing 22 extra dlls on disk: https://github.com/EamonNerbonne/a-vs-an/issues/2  Enabling multitargeting can be really easy: https://github.com/EamonNerbonne/a-vs-an/commit/fa1e67a5eede46af0b0fe3f0bef1e0097ca0088d . Now, I think that lots and lots of dependencies is likely at least partially a sign of the times that you're not entirely going to be able to avoid, but I minimizing dependencies is still an entirely reasonable goal.  It's a bit of a pain that lib authors need to take steps to multitarget, but it's not a huge deal (well, except for testing, I have no idea how to solve that yet).

@EamonNerbonne You are correct when it comes to the dependencies everywhere. @jamesmanning Am i correct in the asumption that only the RunProcessAsTask solution that needs to be migrated. You don't pack the test suit?

@MaxCheetham correct, only the main project gets packaged.  The test suite is just run via the built dll, no packaging needed/done.

The issue with testing that I meant is that if you multitarget, then really should should be testing every target, especially if you're doing anything that might plausibly be target dependent.

.NET Core Now Supported (at least on Windows) @ajbeaven #14

Thanks guys! Looking forward to the nuget package being updated.

@ajbeaven I've uploaded the 1.2.0 nuget package so please give it a shot when you have a chance.
@EamonNerbonne @MaxCheetham changes worked great, I just did msbuild /t:pack /p:Configuration=Release then nuget push - great job guys! 👍


[camera] Fix flash/torch not working on some Android devices.

Description
This PR fixes the new flash & torch functionality not currently working for some Android devices, causing picture capture to break.
It has been tested to work with the following devices:

OnePlus 6T
LG Nexus 5X
Samsung Galaxy Tab A 10.5

Related Issues

flutter/flutter#72649

Checklist
Before you create this PR confirm that it meets all requirements listed below by checking the relevant checkboxes ([x]). This will ensure a smooth and quick review process.

[x] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[x] My PR includes unit or integration tests for all changed/updated/fixed behaviors (See Contributor Guide).
[x] All existing and new tests are passing.
[x] I updated/added relevant documentation (doc comments with ///).
[x] The analyzer (flutter analyze) does not report any problems on my PR.
[x] I read and followed the Flutter Style Guide.
[x] The title of the PR starts with the name of the plugin surrounded by square brackets, e.g. [shared_preferences]
[x] I updated pubspec.yaml with an appropriate new version according to the pub versioning philosophy.
[x] I updated CHANGELOG.md to add a description of the change.
[x] I signed the CLA.
[x] I am willing to follow-up on review comments in a timely manner.

Breaking Change
Does your PR require plugin users to manually update their apps to accommodate your change?

[ ] Yes, this is a breaking change (please indicate a breaking change in CHANGELOG.md and increment major revision).
[x] No, this is not a breaking change.

@googlebot I consent.


Project Details page - preliminary automatic settings

User Story
As a Data Sponsor, when I review the details of a given project, I need to see what parameters were set automatically, so I can understand what I'm able to do with the project and decide whether I need to request any changes.
Notes
As of Dec. 13, we don't have an "Automatic Settings" section on the Project Details page (aka "show" page):

The goal is to have it look like the wireframe shown in the user journey from #247, i.e.:

In this preliminary phase, we are suppressing some required project metadata fields from the new project request form, but the Standard Metadata Schema includes default values for 3 out of 4 that we want to include in the preliminary Project Details page: "Storage Capacity" ("500 GB"); "Storage Performance Expectations" ("Standard"); and "Project Purpose" ("Research"). The other field to display under "Automatic Settings" is "Project ID," for which #257 is meant to establish a process for automatic assignment. However, I don't think #257 needs to be a blocker for this issue; we can in the interim use placeholder values for Project IDs (e.g., 09az-0002).
I also do not want this issue to be a blocker for #287, though they are certainly related. If we can simply display all four of the automated settings on the "Project Details" page, even without including all of them in the metadata model, then I will be satisfied for now. But of course, in the future, we will need the metadata model to include all fields in the standard schema.
This issue is dependent on #264, which establishes all of the headings for the "Project Details" page
Completion Criteria
Note there is no querying of mediaflux or tracking provenance here, ONLY taking data from a config file and displaying it on the screen

[ ] The preliminary "Project Details" page includes a section with the heading "Automatic Settings" below the "Description" section
[ ]  The "Automatic Settings" section includes 4 fields (in order): "Project ID"; "Storage Capacity"; "Storage Performance Expectations"; and "Project Purpose"
[ ] The "Project ID" value shown for a given project is either a placeholder value (e.g. 09az-0003) or the draft DOI reserved for that project
[ ] There is a config file where these automatic values are managed
[ ] The "Storage Capacity" value shown is the default "500 GB"
[ ] The "Storage Performance Expectations" value shown is the default "Standard"
[ ] The "Project Purpose" value shown is the default "Research"

Hey team! Please add your planning poker estimate with Zenhub @carolyncole @hectorcorrea @JaymeeH @leefaisonr


Add a Time TypedInput Widget

Similar to the Date Input widget, one for times

There is now a DateTime input widget, but not yet just a Time widget. This will take some more effort and refactoring to produce, and is not a burning need now.
Developers can use DateTime as a field in the meantime. The only problem being that it will result in users being prompted with the date picker before they can choose the time, which will be confusing if only Time is needed.
We can add a Time form field that works properly, but it will take some extra effort.


Animated cycle plotters

Description
Provide a high-level summary of the change and which issue (either on Github or Notion) is fixed. Please also include relevant motivation and context.
Fixes #(issue) (if the issue is on Github, simply link to it using the '#' symbol; otherwise, provide a notion page link)
Type of change:
Delete as appropriate:

Bug fix (non-breaking change which fixes an issue)
New feature (non-breaking change which adds functionality)
Breaking change (fix or feature that would cause existing functionality to not work as expected)
Documentation update
Anything else (provide your category)

Features:

bulleted list of features, describing the code contributions in this pull request

Questions:

potential questions for the code reviewer (e.g., if you want feedback on a certain method or class; any feedback required from other code contributors with their mentioning)

Remarks:

any additional remarks for the code reviewer
if your remarks or questions pertain to individual lines in the code, you may also include them as comments in the code review window

[Optional] Screenshots:
If your PR is related to UI change or anything visual, feel free to include screenshots to give the reviewer straightforward understandings on your changes.

I am learning GitHub etiquette and this project was not yet ready for a pull request. Will close for now and open pull request when appropriate.


How to backup and restore etcd cluster by docker container env?

Our env:
Three docker containers(on three hosts)for etcd and volume mount /usr/local/etcd/data:/data mount on each host.
node1: <IP_ADDRESS>
node2: <IP_ADDRESS>
node3: <IP_ADDRESS>
Backup and Restore process:
When use v2 with cmd like etcdctl backup --data-dir /data --backup-dir /data/data_bak,
and restore it firstly into a single member cluster by docker-compose.yml with command: command: /usr/local/bin/etcd --data-dir /data/data_bak  --force-new-cluster, then change the peerurl with cmd:   etcdctl  member update 8e9e05c52164694d http://<IP_ADDRESS>:2380
info for this single member cluster:

/ # etcdctl  member update 8e9e05c52164694d http://<IP_ADDRESS>:2380
Updated member with ID 8e9e05c52164694d in cluster
/ # etcdctl member list
8e9e05c52164694d: name=default peerURLs=http://<IP_ADDRESS>:2380 clientURLs=http://localhost:2379 isLeader=true
/ # netstat -tnulp|grep 2380
tcp        0      0 <IP_ADDRESS>:2380          <IP_ADDRESS>:*               LISTEN      -

You know, when reload by docker restart, the port was still listening on <IP_ADDRESS>, but not <IP_ADDRESS>, thus i could not add the node2, node3 to this cluster. So how can i fulfill this backup/restore function in such env?Do your experts has any recommend method?

@regardfs is etcd restarted with --listen-peer-urls set? etcdctl member update will update the advertised peer url, but etcd still has to be called with --listen-peer-urls to bind to the right interface/address for listening.

@heyitsanthony, When i start docker-compose with this command:
    command: /usr/local/bin/etcd       --data-dir /data/data_bak       --listen-peer-urls http://<IP_ADDRESS>:2380
There are mismatch errors in each node:
node1:

etcd1          | 2017-04-11 02:32:21.959753 E | rafthttp: request cluster ID mismatch (got b853b134c9640cad want cdf818194e3a8c32)
etcd1          | 2017-04-11 02:32:22.061261 E | rafthttp: request cluster ID mismatch (got b853b134c9640cad want cdf818194e3a8c32)
etcd1          | 2017-04-11 02:32:22.063836 E | rafthttp: request cluster ID mismatch (got b853b134c9640cad want cdf818194e3a8c32)
etcd1          | 2017-04-11 02:32:22.163369 E | rafthttp: request cluster ID mismatch (got b853b134c9640cad want cdf818194e3a8c32)
etcd1          | 2017-04-11 02:32:22.166902 E | rafthttp: request cluster ID mismatch (got b853b134c9640cad want cdf818194e3a8c32)
etcd1          | 2017-04-11 02:32:22.266822 E | rafthttp: request cluster ID mismatch (got b853b134c9640cad want cdf818194e3a8c32)
etcd1          | 2017-04-11 02:32:22.268380 E | rafthttp: request cluster ID mismatch (got b853b134c9640cad want cdf818194e3a8c32)
etcd1          | 2017-04-11 02:32:22.369789 E | rafthttp: request cluster ID mismatch (got b853b134c9640cad want cdf818194e3a8c32)
etcd1          | 2017-04-11 02:32:22.373123 E | rafthttp: request cluster ID mismatch (got b853b134c9640cad want cdf818194e3a8c32)

node2 and node3(already delete all /data!!!):

etcd2          | 2017-04-11 02:32:14.531294 E | rafthttp: request sent was ignored (cluster ID mismatch: peer[12558efa96ea1646]=cdf818194e3a8c32, local=b853b134c9640cad)
etcd2          | 2017-04-11 02:32:14.533902 E | rafthttp: request sent was ignored (cluster ID mismatch: peer[12558efa96ea1646]=cdf818194e3a8c32, local=b853b134c9640cad)
etcd2          | 2017-04-11 02:32:14.635218 E | rafthttp: request sent was ignored (cluster ID mismatch: peer[12558efa96ea1646]=cdf818194e3a8c32, local=b853b134c9640cad)
etcd2          | 2017-04-11 02:32:14.637433 E | rafthttp: request sent was ignored (cluster ID mismatch: peer[12558efa96ea1646]=cdf818194e3a8c32, local=b853b134c9640cad)
etcd2          | 2017-04-11 02:32:14.736936 E | rafthttp: request sent was ignored (cluster ID mismatch: peer[12558efa96ea1646]=cdf818194e3a8c32, local=b853b1

I have viewed some bugs and issues which quite like this scenario, such as https://github.com/coreos/etcd/issues/3651, but not work...

@heyitsanthony , Hi , in docker compose env, all depend on docker-compose.yml, so finally, I change my docker-compose.yml command part to and etcd could start with listening on right ip and port, thanks a lot, i hope this experience could help those guys who do this process in docker env.

command: /usr/local/bin/etcd
--data-dir /data/data_bak
--listen-peer-urls http://<IP_ADDRESS>:2380
--advertise-client-urls http://<IP_ADDRESS>:2379,http://<IP_ADDRESS>:4001
--force-new-cluster


VictoriaLogs has no context view function bug

VictoriaLogs has no context view function bug

Hi @chown1987 ! Your issue is a duplicate of the https://github.com/VictoriaMetrics/victorialogs-datasource/issues/41. I close this issue please see the progress in the mentioned issue


[API] Use "PUT" method for update

Currently editing API uses "POST" so use PUT and change URL if nessecery

https://github.com/FlowingSPDG/get5-web-go/pull/102


Added readonly as metadata config option

Each field in a metadata catalog can now be set to be editable by the user or not.
Furthermore, this reworks how settings are written in the editor-settings.toml to allow specifiying multiple attributes for one metadata field. If an attribute is not specified, a default is assumed instead.
To test this, play around with example definitions in the editor-settings.toml and observe how they change the metadata ui.

Theoretically, this should solve #362, shouldn't it?

It should. Closing that one.


Can not import data when using multi page document annotation

Validation error
Error at item 0: "document" key is expected in task data [assume: item["data"] = task root with values] :: {'data': {'document[{{idx}}].page': '/data/upload/1/5d355b15-document_page_0001.jpg'}, 'file_upload_id': 38}
Version: 1.7.3

Hello Lonepic,
Is it possible to show how your are importing the data and the labeling interface?

I'm getting the same error.

Create a project, select "Multi-Page document annotation" in Labeling Setup, and leave other parameters as default;
After the project is successfully created, select a picture and then import it, and get the error message:

Error at item 0: "document" key is expected in task data [assume: item["data"] = task root with values] :: {'data': {'document[{{idx}}].page': '/data/upload/6/3a72a25b-page_1.jpg'}, 'file_upload_id': 13}

Same probleme here, I try to import a pdf located at https://app.heartex.ai/static/samples/sample.pdf
And get the error below:

Error at item 0: "document" key is expected in task data [assume: item["data"] = task root with values] :: {'data': {'document[{{idx}}].page': '/data/upload/7/2cdc34dd-sample.pdf'}, 'file_upload_id': 15}

When you create a new project from "Multi-page document annotation" template, you have a task data example in the Code section:

You can't merely import an image file, because this template uses Repeater tag, it's a complex annotation type with multiple import sources:
<View>
  <Repeater on="$document" indexFlag="{{idx}}" mode="pagination">
      <Image name="page_{{idx}}" value="$document[{{idx}}].page" inline="true"/>
      <RectangleLabels name="labels_{{idx}}" toName="page_{{idx}}">
        <Label value="Title" background="green" />
        <Label value="Date" background="blue" />
        <Label value="Author" background="yellow"/>
        <Label value="Organization" background="orange"/>
        <Label value="Amount" background="red"/>
      </RectangleLabels>
      <View style="display: flex; justify-content: center;">
        <Header value="Rate this page:"/>
        <Rating name="rating_{{idx}}" toName="page_{{idx}}"/>
      </View>
  </Repeater>
</View>

Logically it's not possible to combine all images per one task in one import step, so you have to orchestrate them into a JSON file with your tasks that points to your images on some hosting (https, s3, gcs, azure). Example of such a JSON file:
 {
"document": [
    {
      "page": "https://htx-pub.s3.amazonaws.com/demo/images/demo_stock_purchase_agreement/0001.jpg"
    },
    {
      "page": "https://htx-pub.s3.amazonaws.com/demo/images/demo_stock_purchase_agreement/0002.jpg"
    },
    {
      "page": "https://htx-pub.s3.amazonaws.com/demo/images/demo_stock_purchase_agreement/0003.jpg"
    }
  ]
}


Incorrect nightly DV report

I received nightly DV reports via email which sometimes showed error tests for IP (see below screenshot). However, when I click into failed IP (e.g. pattgen IP), it showed all tests passed. There might be issues in scripts used to collect and generate the nightly reports.

Thank you for raising this issue Tung.
Here is what happened:

When you received the email it is Tuesday March 30th, and when you click the link to pattgen, it refers to the latest report (for today it is actually March 31st).
So that means if you check it tomorrow, the link should refer to April 1st.
If you go down pattgen and click "2021.03.30_14.34.56", this should refer to the March 30th result.

So the issue here is the email always refer to the latest results, and the latest results get updated daily.

@cindychip thank you for the explanation


Define JSON structure for subtitles

Example
{
   "imdb_id":1,
   "name":"Orange is the New Black",
   "type":"TV_SHOW",
   "season":1,
   "episode":1,
   "phrases":[
      {
         "time":"01:00:00-01:05:00",
         "phrase":"Today is a great day!"
      },
      {
         "time":"01:05:00-01:06:00",
         "phrase":"Isn't it?"
      }
   ]
}

or
{
   "imdb_id":1,
   "name":"Orange is the New Black",
   "type":"TV_SHOW",
   "season":1,
   "episode":1,
   "phrases":[
         "Today is a great day!",
         "Isn't it?"
   ]
}

@joaomcar do you think we should provide the time? It varies from video to video. Perhaps, it'snot a valuable info.

No @danielfc, I think it's not necessary to keep this information.

@danielfc, I think the year is an important info.
{      "imdb_id":1,    "name":"Orange is the New Black",    "type":"TV_SHOW",    "season":1,    "episode":1,    "year": 2015,    "phrases":[            "Today is a great day!",          "Isn't it?"    ] }


Archives for Windows for 1.2.2 Invalid

Attempting to extract a downloaded release for x86 and x64 versions using Window's built in .zip functionality results in an error notifying that the archive is invalid.
Extracting with 7zip works fine. I have not verified beyond current release.
I'm not entirely sure why, to be honest. And this could very well be something that's going over my head. But I thought I may bring it to your attention. Could maybe be an OS-related issue?

Hello,
it's definitely a OS relates issue, due to long path names (after electron packaging) in ZIP file, windows default archiver doesn't allow to unzip it. But 7zip and winrar works fine as you mentioned.
duplicate of #81
Cheers


OPG-472: Documentation for nodeFilter label and value formatting

Documentation for the nodeFilter labelFormat and valueFormat feature, see PR #937 and PR #940 for more info.
External References

JIRA (Issue Tracker): http://issues.opennms.org/browse/OPG-472
Continuous Integration: CircleCI

Good suggestions, made all of the changes.


VS crash if you add a wrong condition to project reference

Repro Steps

Create Console Application (.Net Core)
Create Class Library (.Net Framework)
Add reference from Console to class library
Build
Edit Console Application and add following condition to the

<ItemGroup>
    <ProjectReference Include="..\ClassLibrary1\ClassLibrary1.csproj" Condition="net452" />
  </ItemGroup>

Expected
We should probably indicate there is a wrong input and unload that project
Actual
VS will crash
Note: Right way to condition this is
<ItemGroup>
    <ProjectReference Include="..\ClassLibrary1\ClassLibrary1.csproj" Condition="'$(TargetFramework)' == 'net452'" />
  </ItemGroup>

Build : 26213.d15rel

VS crashes?

Dup'ing against an uber issue tracking this: https://github.com/dotnet/roslyn-project-system/issues/1570

just to answer the question, it was a crash for me


useInterval 没有返回值

Describe the bug
useInterval 返回值为undefined
Steps to reproduce
No response
System Info
谷歌浏览器 vhookplus 2.1

Used Package Manager
pnpm
Validations

[X] Follow our Code of Conduct.
[X] Read the Contributing Guidelines.
[X] Read the docs.
[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
[X] Make sure this is a VueHooks Plus issue and not a framework-specific issue. For example, if it's a Vue SFC related bug, it should likely be reported to vuejs/core instead.
[X] Check that this is a concrete bug. For Q&A open a GitHub Discussion.
[X] The provided reproduction is a minimal reproducible example of the bug.

@huxiguo  我看了一下这个 hook 并没有提供 return value 给到外部使用，这是 一个 bug，我会在这几天更新一个优化包，会在里面包含这个 bug 的修复，感谢你的反馈。

已经修复
文档：https://inhiblabcore.github.io/docs/hooks/useInterval/index.html#代码演示
版本：v2.2.1


Error due to old DLP policy

Hi
I have installed the CoE and the additional auditing components.  I added the required custom connector to a DLP Policy and it appeared in the Business section of my DLP with the other connectors required for the CoE.
However I have now noticed that the child flow "[child] | Admin |Sync Logs" is not being run and the parent flow is erroring.  I therefore deleted the DLP, however this error did not go away.  I then created another DLP with a different name, however I am getting the same error.
The first DLP was called "CoE Policy" and the second is called " CoE DLP"
I have attached an image of the second DLP policy showing the custom connector within the Business Only section.
However the error the I am getting when starting the child flow is, which seems to be referencing a DLP called CoE which I have never created so it makes not sense to me.
"Your flow could not turn on: [Child] Admin | Sync Logs. Error: Flow client error returned with status code "Forbidden" and details "{"error":{"code":"DlpViolation","message":"Admin data policy 'CoE' restricts the use of /providers/Microsoft.PowerApps/apis/shared_commondataserviceforapps with /providers/Microsoft.PowerApps/apis/shared_admin-5foffice-20365-20management-20api-5f00e1c83cedecf792."}}"."
Note: There are 2 additional Environment DLP policies in place on the tenant which are not referencing the environment I have the CoE installed on.
Thank you for your help in advance
Paul

Entering in incorrect place.


Fix $validate-code result with ValueSet.compose.exclude.valueSet.

Fix $validate-code result with ValueSet.compose.exclude.valueSet when ValueSet.compose.exclude.system is not present (which is normally expected).

Sure.  I have to admit that I don't think I fully understand the use of the FNoValueSetExpansion flag - exactly what case it's trying to cover.  So I tried to make sure that logic where it is needed wasn't altered.  But FNoValueSetExpansion was being set to 'true' in a case where it seems that it clearly was not intended - when compose.exclude.system is not present (which results in cs = nil) but compose.exclude.valueSet is present.    If both 'system' and 'valueSet' are specified in the 'exclude', the existing logic works, because in that case 'cs' is not 'nil' which sets FNoValueSetExpansion to 'false' and the value set expansion proceeds normally.  But if 'valueSet' is present alone (without 'system'), FNoValueSetExpansion is then set to 'true' and the expansion fails.  But it's not required or normally expected (and could be erroneous) to also send 'system' with 'valueSet, so the logic needs to be updated to work without it.  The update that I made by adding  and (s = '') keeps FNoValueSetExpansion set as 'false' and allows the expansion to proceed as expected if either 'system' or 'valueSet' are present in the 'exclude'.
I have to admit that I also thought about whether the current logic works correctly for 'include', but I didn't thoroughly investigate that because that seems to be working (as far as I know).  I've certainly used compose.include.valueSet without also specifying compose.include.system and I haven't seen others complaining about it.  So I decided to stay focused on only the part that I knew was broken.

This issue still exists.  I've reworked the PR so that the the logic is more robust and more clearly understandable (hopefully).  I'll also add some include and exclude test cases.

POST localhost/r4/ValueSet/$validate-code?system=http://loinc.org&code=2160-0
with body:
{
  "resourceType" : "ValueSet",
  "id" : "valueset-include-test",
  "url" : "http://hl7.org/fhir/uv/ips/ValueSet/valueset-include-test",
  "version" : "1.0.0",
  "name" : "ValuesetIncludeTest",
  "title" : "Valueset Include Test",
  "status" : "active",
  "publisher" : "Health Level Seven International - Patient Care Work Group",
  "description" : "Valueset Include Test",
  "jurisdiction" : [
    {
      "coding" : [
        {
          "system" : "http://unstats.un.org/unsd/methods/m49/m49.htm",
          "code" : "001"
        }
      ]
    }
  ],
  "immutable" : false,
  "copyright" : "This artefact includes content from SNOMED Clinical Terms® (SNOMED CT®) which is copyrighted material of the International Health Terminology Standards Development Organisation (IHTSDO). Where an implementation of this artefact makes use of SNOMED CT content, the implementer must have the appropriate SNOMED CT Affiliate license - for more information contact http://www.snomed.org/snomed-ct/get-snomed-ct or<EMAIL_ADDRESS>  "compose" : {
    "include" : [
      {
        "valueSet" : [
          "http://terminology.hl7.org/ValueSet/snomed-intl-gps"
        ]
      }
    ]
  }
}

returns:
{
  "resourceType" : "Parameters",
  "parameter" : [
    {
      "name" : "display",
      "valueString" : "Creatinine [Mass/volume] in Serum or Plasma"
    },
    {
      "name" : "result",
      "valueBoolean" : true
    }
  ]
}

but should return:
{
  "resourceType" : "Parameters",
  "parameter" : [
    {
      "name" : "result",
      "valueBoolean" : false
    },
    {
      "name" : "message",
      "valueString" : "The code provided (http://loinc.org#2160-0) is not valid in the value set 'ValuesetIncludeTest'"
    },
    {
      "name" : "cause",
      "valueString" : "unknown"
    }
  ]
}

POST localhost/r4/ValueSet/$validate-code?system=http://loinc.org&code=2160-0
with body:
{
  "resourceType" : "ValueSet",
  "id" : "valueset-exclude-test",
  "url" : "http://hl7.org/fhir/uv/ips/ValueSet/valueset-exclude-test",
  "version" : "1.0.0",
  "name" : "ValuesetExcludeTest",
  "title" : "Valueset Exclude Test",
  "status" : "active",
  "publisher" : "Health Level Seven International - Patient Care Work Group",
  "description" : "Valueset Exclude Test",
  "jurisdiction" : [
    {
      "coding" : [
        {
          "system" : "http://unstats.un.org/unsd/methods/m49/m49.htm",
          "code" : "001"
        }
      ]
    }
  ],
  "immutable" : false,
  "copyright" : "This artefact includes content from SNOMED Clinical Terms® (SNOMED CT®) which is copyrighted material of the International Health Terminology Standards Development Organisation (IHTSDO). Where an implementation of this artefact makes use of SNOMED CT content, the implementer must have the appropriate SNOMED CT Affiliate license - for more information contact http://www.snomed.org/snomed-ct/get-snomed-ct or<EMAIL_ADDRESS>  "compose" : {
    "include" : [
      {
        "system" : "http://snomed.info/sct",
        "filter" : [
          {
            "property" : "concept",
            "op" : "descendent-of",
            "value" : "836369007"
          }
        ]
      }
    ],
    "exclude" : [
      {
        "valueSet" : [
          "http://terminology.hl7.org/ValueSet/snomed-intl-gps"
        ]
      }
    ]
  }
}

returns:
{
  "resourceType" : "Parameters",
  "parameter" : [
    {
      "name" : "display",
      "valueString" : "Creatinine [Mass/volume] in Serum or Plasma"
    },
    {
      "name" : "result",
      "valueBoolean" : true
    }
  ]
}

but should return:
{
  "resourceType" : "Parameters",
  "parameter" : [
    {
      "name" : "result",
      "valueBoolean" : false
    },
    {
      "name" : "message",
      "valueString" : "The code provided (http://loinc.org#2160-0) is not valid in the value set 'ValuesetExcludeTest'"
    },
    {
      "name" : "cause",
      "valueString" : "unknown"
    }
  ]
}

POST localhost/r4/ValueSet/$validate-code?system=http://snomed.info/sct&code=836388000
with body:
{
  "resourceType" : "ValueSet",
  "id" : "valueset-exclude-test",
  "url" : "http://hl7.org/fhir/uv/ips/ValueSet/valueset-exclude-test",
  "version" : "1.0.0",
  "name" : "ValuesetExcludeTest",
  "title" : "Valueset Exclude Test",
  "status" : "active",
  "publisher" : "Health Level Seven International - Patient Care Work Group",
  "description" : "Valueset Exclude Test",
  "jurisdiction" : [
    {
      "coding" : [
        {
          "system" : "http://unstats.un.org/unsd/methods/m49/m49.htm",
          "code" : "001"
        }
      ]
    }
  ],
  "immutable" : false,
  "copyright" : "This artefact includes content from SNOMED Clinical Terms® (SNOMED CT®) which is copyrighted material of the International Health Terminology Standards Development Organisation (IHTSDO). Where an implementation of this artefact makes use of SNOMED CT content, the implementer must have the appropriate SNOMED CT Affiliate license - for more information contact http://www.snomed.org/snomed-ct/get-snomed-ct or<EMAIL_ADDRESS>  "compose" : {
    "include" : [
      {
        "system" : "http://snomed.info/sct",
        "filter" : [
          {
            "property" : "concept",
            "op" : "descendent-of",
            "value" : "836369007"
          }
        ]
      }
    ],
    "exclude" : [
      {
        "valueSet" : [
          "http://terminology.hl7.org/ValueSet/snomed-intl-gps"
        ]
      }
    ]
  }
}

returns:
{
  "resourceType" : "Parameters",
  "parameter" : [
    {
      "name" : "display",
      "valueString" : "Vaccine product containing Rubella virus antigen (medicinal product)"
    },
    {
      "name" : "result",
      "valueBoolean" : true
    }
  ]
}

but should return:
{
  "resourceType" : "Parameters",
  "parameter" : [
    {
      "name" : "result",
      "valueBoolean" : false
    },
    {
      "name" : "message",
      "valueString" : "The code provided (http://snomed.info/sct#836388000) is not valid in the value set 'ValuesetExcludeTest'"
    },
    {
      "name" : "cause",
      "valueString" : "unknown"
    }
  ]
}

If you need them, I can add some positive cases (that should return 'true'), too.

This has been addressed by Grahame's latest commit.


Remove MonadResource's duplication of Base re-exports

In addition to simply removing a bunch of code, this also removes some unnecessary typeclass constraints. For example, get and put don't require any resource cleanup, so it's odd to give them a MonadResource constraint when MonadIO will do.

Thanks


Unable to test your code

In line 46 of the models_test.py code (base_model_name = 'nets_test.C2D_' + self.net), 'nets_test.C2D' cannot be found, can you please provide it?

Thanks for your attention, I have already revised the code. Testing is now available.

Thanks for your attention, I have already revised the code. Testing is now available.

Sorry, I still can't test your code. When I use your updated code, the following error is reported in models_tsn.py (Line 233) :
return output.squeeze(1)
AttributeError: 'NoneType' object has no attribute 'squeeze'

If the network receive data in the right format, the output of the network could not be 'NoneType'. Checking out the data flow  in your case step by step may be helpful


Module not found: Can't resolve 'classnames'

Getting this error when integrating into a NextJS app. Adding the classnames to my package.json solves it, but you may want to move classnames to the this modules dependencies instead of devDependencies?

Thanks for your interest in my project.
The new version of the package already has a fix.
https://github.com/coded-bear/react-otp-field/pull/11/files


require("luasnip.loaders.from_vscode").lazy_load() doesn't work

Problem
require("luasnip.loaders.from_vscode").lazy_load()

Doesn't load the snippets from friendly-snippets plugin.
However it works when I manually load it by running
:lua require("luasnip.loaders.from_vscode").load({ include = { "python" } })

Config files

packer config
nvim-cmp config

NOTE: I write my config files in fennel, the above links are to the compiled lua output, so it looks unconventional.

Looks (? I'm not that sure when it comes to packer's features) like you're adding luasnip and friendly-snippets on some event, could you check if the problem persists if they are loaded directly, eg on startup?

I'll check it out tomorrow, sorry for the delay.

No problem, it's still working for me, so not a general issue I think

Yeah, loading it on startup works. Thanks.


Support RequestOutput in V2 Inference Request

/kind feature
Describe the solution you'd like
[A clear and concise description of what you want to happen.]
https://kserve.github.io/website/0.11/modelserving/data_plane/v2_protocol/#request-output
RequestOutput in Inference Request is supported according to the V2 Protocol.
It is not supported currently. https://github.com/kserve/kserve/blob/c1c5efec646462f34b8848c3264849f8ed7a65c5/python/kserve/kserve/protocol/infer_type.py#L225
Anything else you would like to add:
[Miscellaneous information that will assist in solving the issue.]
Links to the design documents:
[Optional, start with the short-form RFC template to outline your ideas and get early feedback.]
[Required, use the longer-form design doc template to specify and discuss your design in more detail]

We often deal with models that return more than just the prediction results, for example, losses or other metadata that are not useful for inference. It would be practical to enable the selection of response output!!


update links to be relative

Description

changes links to be relative (no hostname included anymore)

Please create a new PR that only contains the relevant changes, but not the deletion of your task solution


The Copy Bundle Resources build phase contains this target's Info.plist file

The framework target contains its own Info.plist file in the copy bundle resources phase. The Info.plist doesn't need to be in the copy bundle resources phase and it causes a warning when building the framework. It's not causing any actual bugs but I'd appreciate if you could remove the file from the copy bundle resources phase anyway!

Thanks for catching this 👍 - I'll create a new version of the Pod over the weekend.


PROPOSAL: Add instructions on creating a new column from a sequence

This took me a while to figure out and there doesn't appear to be a straightforward tutorial for doing this in DuckDB. The use case was I had a huge table that already existed and would have taken forever to either recreate or update each row individually. I simply needed a primary key column and eventually I found out that I can create a new column and set its default to the next value of a sequence! I propose adding this SQL code to the CREATE SEQUENCE page in the documentation so others can know how to do this too.
CREATE SEQUENCE id_sequence START 1;
ALTER TABLE MyTable ADD COLUMN id INT DEFAULT nextval('id_sequence');

Let me know if you'd like me to make a pull request for this.

@erikhuck thanks for the suggestions – sure, I'm happy to review a PR about this!


Add helper function and properties for -with-udebs and -architectures

Description
Add -with-udebs and -architectures to aptly mirror create.
Issues Resolved
#49
Check List

[ ] All tests pass. See https://github.com/chef-cookbooks/community_cookbook_documentation/blob/master/TESTING.MD
[ ] New functionality includes testing.
[x] New functionality has been documented in the README if applicable

I ran tests locally and they succeeded.


Avoid unnecessary margin-bottom

Fixes #149.

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

hi @nya3jp please sign the Contributor License Agreement :)

Hi! I've just signed CLA.

Thanks for reviewing! How do we merge this change?


Put a link to CuPy installation guide in README instead of a command instruction

pip install cupy is not recommended for general users.

imosCI, test this please.

Jenkins CI test (for commit f47a51f8a9a05a7d6ade1c48a36171518f967e88, target branch master) succeeded!

@Crissman This pull-request is marked as to-be-backported, but the corresponding backport PR could not be found. Could you check?

@Crissman This pull-request is marked as to-be-backported, but the corresponding backport PR could not be found. Could you check?

@Crissman This pull-request is marked as to-be-backported, but the corresponding backport PR could not be found. Could you check?

@Crissman This pull-request is marked as to-be-backported, but the corresponding backport PR could not be found. Could you check?

@Crissman This pull-request is marked as to-be-backported, but the corresponding backport PR could not be found. Could you check?

@Crissman This pull-request is marked as to-be-backported, but the corresponding backport PR could not be found. Could you check?

@Crissman This pull-request is marked as to-be-backported, but the corresponding backport PR could not be found. Could you check?


LogicValue division with negative numbers doesn't work as expected

Describe the bug
It appears that doing division on LogicValues with numbers that may be interpreted as negative can give unexpected and incorrect results.
See https://github.com/intel/rohd/pull/319/files#r1325020368
To Reproduce

8 / -2 should be -4, but it is 0.

Expected behavior
No response
Actual behavior
No response
Additional: Dart SDK info
No response
Additional: pubspec.yaml
No response
Additional: Context
No response

Fixed in #319


”Coșbuc” is listed in the drop-down menu

Description:
"Coșbuc" is listed in the drop-down menu alongside its corresponding county and commune, according to the law.
Precondition:
The website is up an running.

Step 1
Type "Coșbuc" in the search bar.

Expected results
The location is listed in the drop-down menu as "Coșbuc, Bistrița-Năsăud (Coșbuc)".


Refine /api/competitions/ so that it isn't unnecessarily huge and contains what mtggoldfish needs

In particular providing card details and not just card names is pretty psychotic.

I did this.

Or someone did. I forget :D


Make minor improvements to memberrewards notebook

Improvements are:

simplifying definition of df.treatment
changing + to - for simulation of monthly effect to be consistent with the comment
change i=6 to i=3 to be consistent with markdown explanation
simplify post-processing code, using .mean() instead of explicitly summing and dividing by count.
apply black auto-formatting to the post-processing cell to make it easier to read
minor rewordings and additions of comments

Let me know if these are sensible and/or if I have overlooked anything by making these changes.

Closing the PR as I prematurely signed CLA. Though this is a personal contribution, the only reason I came across this package is via my work, so I should check this first.

Thank you for these improvements, @LovKush-A. Look forward to your response about the CLA--I'm hoping that it should not be an issue.


About data prepare

Hi! I see the dense flow code, when prepare data, you simply resize the image to (340x256) whatever the origin size of the image is, is it right? and compute flow with images of 340x256.
Thank you!

and then convert the flow to 0~255?

It described in both paper and the codebase. Please do no abuse GitHub issues.


Pyinstaller hook failing on MacOS GitHub runner

Describe the bug
Pyinstaller hook failing on MacOS GitHub runner, causing the game to crash on launch.
To Reproduce
Steps to reproduce the behaviour:

Freeze a program that uses pygame-gui using Pyinstaller on the macos-11 runner on GitHub
Attempt to run the program on a Mac
Program crashes

Expected behaviour
The program does not crash.
Screenshots
GitHub Actions log:

Traceback:

Platform and software (please complete the following information):

OS: Mac
Pygame GUI version: 0.6.8
Pygame version 2.1.3

Additional context
This does not occur when I run Pyinstaller locally on my own system. In the workflow, we're using actions/setup-python to setup Python and installing packages using pip. The hook did not fail on Pygame 2.1.2.

I wonder if this is related to a bug we had over on pygame-ce with mac runners no longer installing pkg-config by default, so freetype didn't install properly - then cascading from there?
see: https://github.com/pygame-community/pygame-ce/pull/1888

Or it is the mac font rendering bug:
https://www.reddit.com/r/pygame/comments/11505q9/pygamefontinit/


Droid Issue

After updating the control, it seems to throw generic exceptions on android projects only.

Which exceptions?

I am releasing an update making the Android version to match Xamarin.Android.Support.v4 version used by Xamarin.Forms.

If my work is helping you, please help me back: https://xamarinhq.wufoo.com/forms/nominate-a-xamarin-mvp/
This is what i have done that is community visible:

Alex has been pretty active sharing his knowledge and answering questions in StackOverflow, GitHub and Xamarin Forums.
Alex has been open sourcing Xamarin components and plugin with the community, with more of 10 repositories being maintained on GitHub.
One of Alex's Xamarin.Forms plugin, the CarouselView, is currently being pulled into the Xamarin.Forms core (https://github.com/xamarin/Xamarin.Forms/pull/853). As an stable widget, this control has become the standard for Xamarin.Forms carousel screens with more that 16K downloads in NuGet and more than 100 solved and not open issues.
Another Alex's plugin, the SegmentedControl (https://github.com/alexrainman/SegmentedControl), is a big candidate to be also pulled into Xamarin.Forms core.
Plus Xamarin.Forms UI related plugins, Alex shares plugins to set app icon Badge, Google Analytics and KeyChain access.
Alex shares his knowledge about cutting edge technologies like Couchbase NoSQL and MobileIron integration with Xamarin Apps.


Cryptic error when trying to build files when a class has a getter that returns a function

import 'package:isar/isar.dart';

part 'example.g.dart';

typedef MyFunc = int Function(int i);

@Collection()
class Example {
  MyFunc get myFunction {
    return (i) {
      return i * i;
    };
  }

  Id id = Isar.autoIncrement;
}

Outputs:
[INFO] Generating build script completed, took 357ms
[INFO] Reading cached asset graph completed, took 887ms
[INFO] Checking for updates since last build completed, took 1.9s
[SEVERE] isar_generator on lib/infrastructure/data_sources/dispatch_hub_data_source/model_configuration/example.dart:

Null check operator used on a null value
[INFO] Running build completed, took 1.6s
[INFO] Caching finalized dependency graph completed, took 394ms
[SEVERE] Failed after 2.0s

While this is solved easily by adding ignore: {"myFunction"}, the error threw me off for a while. I would expect this to output something more specific like this:
Unsupported type. Please annotate the property with @ignore.
package:ironsight_mobile/infrastructure/data_sources/dispatch_hub_data_source/model_configuration/example.dart:15:13
   ╷
15 │   Set<Set>? sets;
   │             ^^^^
   ╵
[INFO] Running build completed, took 1.6s

The error message suggests to support the property with @ignore. That would've solved the issue 🙂

I think you are looking at what I expected to see, not what I actually saw. The actual output has a null check error:
[INFO] Generating build script completed, took 357ms
[INFO] Reading cached asset graph completed, took 887ms
[INFO] Checking for updates since last build completed, took 1.9s
[SEVERE] isar_generator on lib/infrastructure/data_sources/dispatch_hub_data_source/model_configuration/example.dart:

Null check operator used on a null value
[INFO] Running build completed, took 1.6s
[INFO] Caching finalized dependency graph completed, took 394ms
[SEVERE] Failed after 2.0s

+1
I was searching this issue and this solved it.
The error could be clearer.
Thanks so much!

@simc could you take another look at this please?


New Payment Flow UI

Detailed Description
We should implement our new designs for the end of the send payment/transaction flow.
https://app.zeplin.io/project/5bacdbad33af886d419f1b34/screen/5bc75189140d6a09ac630b23
Context
After clicking send the user is taken to an old payment flow that doesn't match current designs and is a bit of a hack job. In the new flow, the payment would be added to the activity list with an indication that it's in the process of sending, and then visually update once it has finished sending.
Possible Implementation

Add sending transactions and payments to state with sending flag
Display sending payments in activity list
Remove sending flag after no less than 3 seconds to avoid flicker for quick sends
Animate new payments appearing in the list

closing due to #1060


Enhancement: Embedding and Vector Search Chunking

Goal-State/What/Result
Compute, compare and store several embeddings per row, per column for a dataset by intelligently chunking the content.
Why/Purpose
Chunking long data to create several embedding is a common strategy to improve vector search quality.
By When
Issue/Spec written and reviewed: 25th September
Done-Done: v1.0-rc.2
Done-Done

[ ] First Principles Driven
[ ] The Algorithm
[ ] PM/Design Review
[ ] UX Review
[ ] Threat Model / Security Review
[ ] Tests
[ ] Documentation
[ ] Samples/Quickstarts/Tutorials
[ ] Release Notes

The Algorithm

[ ] Every requirement questioned?
[ ] Delete (Scope) any part you can.
[ ] Simplify.
[ ] Break down into smaller iterations/milestones.
[ ] Opportunities for automation.

Spec
Spicepod UX

User's should be able to automatically enable chunking of the embedding column for their dataset.

Fortunate rust library https://github.com/benbrandt/text-splitter

Sane defaults for predominately text based chunking, but then can infer better defaults from the dataset.

E.g. if params.file_format provides a hint, there are better chunkers in benbrandt/text-splitter

Default options:
datasets:
  - from: github:github.com/spiceai/spiceai/files/trunk
    name: docs
    params:
      include: docs/**/*.md
    embeddings:
      - column: content
        use: embedding_model_name
        chunking:
          enabled: true

Overrides/schema
embeddings:
  - column: string
    use: string
   chunking:
     enabled: bool # Optional, default false
     desired_chunk_size: usize # Optional, default to maximum (number of tokens)
     overlap_size: usize # Optional, default 0. (number of tokens)
     trim: bool, # optional, default true

SQL UX

when .datasets[*].embeddings[*].chunking.enabled == true, the {embedding_column}_embedding will be of type List(FixedSizeList(Float)) (currently, and if false, FixedSizeList(Float)).
The embedding_column will not be chunked. Instead we need auxiliary data structure. Either

{embedding_column}_offsets of type List(Uint32) that show the sequential offsets of each {embedding_column}_embedding into {embedding_column}. Might be more SQL efficient for this to be List(fixedList(uint32, 2)).
An in-memory offset list (in TableProvider), that can be used in VectorSearch to show the specific subsection of embedding_column that relates to the vector, without exposing the offsets to users.

Vector Search SQL

Ensure the underlying SQL query we would need to do based on above decisions is reasonable and performant.

SELECT
    {other fields + primary keys},
    content,
    content_offset,
    array_distance(embedding, :X) AS distance
FROM documents,
    UNNEST(content_embedding, content_offsets) AS t(embedding, content_offset)
WHERE {predicates}
ORDER BY distance ASC
LIMIT N;

We need embedding models to possibly implement to know when/how to recursively chunk.

pub trait ChunkSizer {
    /// Determine the size of a given chunk to use for validation
    fn size(&self, chunk: &str) -> usize;
}


function setContext

Hello, what function for setContext on java implementations? Thanks

@mazis0  set context helps to switch between Native, Web, or Flutter context. Your app might be developed using different ways it could be Native, Hybrid, or Flutter.

Hi @ashwithpoojary98 how to change context on appium java client 9? do you have example repo for appium flutter using appium java client 9?

How to change context to native app and flutter there? Is it possible?


reset_cache() Decrease the Generation Quality of Consecutive Inferences

When conducting generation for multiple consecutive inputs on a LoRA fine-tuned LLaMA, I noticed that using 'reset_cache' after each generation for one input will affect the performance of generation on the next input. However, if you load the model again after each generation, the performance stays good. But reloading consumes lots of time. Could you help provide some explanation why 'reset_cache' will decrease the performance of the generation on the next consecutive inputs?
Code: I modified the code of 'generate/lora.py' to enable consecutive generation on multiple inputs. Basically, just add a for loop and model.reset_cache()
  # support multiple inference
  outputs = []
  num_samples = len(input)
  for i in range(num_samples):
      sample = {"instruction": prompt[i], "input": input[i]}
      prompt = generate_prompt(sample)
      encoded = tokenizer.encode(prompt, bos=True, eos=False, device=model.device)

      t0 = time.perf_counter()
      output = generate(
          model,
          idx=encoded,
          max_new_tokens=max_new_tokens,
          temperature=temperature,
          top_k=top_k,
          eos_id=tokenizer.eos_id
      )
      t = time.perf_counter() - t0

      model.reset_cache()
      output = tokenizer.decode(output)
      output = output.split("### Response:")[1].strip()
      print(output)
      print(f"Time for inference: {t:.02f} sec total, {max_new_tokens / t:.02f} tokens/sec", file=sys.stderr)
      outputs.append(output)

我也很好奇这一点，我预测前100次和前1000次的准确率就不一样，你邮箱是什么，咱们讨论一下


More emotes replacing common words

I've discovered "snap" displays a camera emoji. There are a few others I can't think of right now, but it may warrant going through the list and removing other common word replacements. I've noticed they're not Twitch emotes, so they may be from the other packs (BTTV, Franker, etc.)

Nope, again I'd blame it on twitch for allowing common words to use emotes. This time it is someone that goes by fwiz.
"fwiz": {
  "title": "Fwiz",
  "link": "http://twitch.tv/fwiz",
  "desc": null,
  "id": "fwiz",
  "badge": "//static-cdn.jtvnw.net/jtv_user_pictures/subscriber-star.png",
  "set": 170,
  "emotes": [
    {
      "code": "FwizzyKid",
      "image_id": 1419
    },
    {
      "code": "PLAYS",
      "image_id": 4568
    },
    {
      "code": "KAPPA",
      "image_id": 4569
    },
    {
      "code": "SLEEPY",
      "image_id": 4570
    },
    {
      "code": "RAGE",
      "image_id": 4571
    },
    {
      "code": "CHYABOI",
      "image_id": 4572
    },
    {
      "code": "FAIL",
      "image_id": 4573
    },
    {
      "code": "snap",
      "image_id": 9182
    }
  ]
},

Yeah they are old partners whose emotes were never removed from the database.
New emote blacklist is now loaded from: https://github.com/Jiiks/BetterDiscordApp/blob/master/emotefilter.json
Just edit that and submit a pull request if you find any bad emotes :+1:

Ok, cool. Thanks Jiiks!


Remote Control Tizen / WebOS not working

the remote control dpad and numeric are doing nothing is there a keyhandler that should be implemented ?

@MwareSolutons basic focus management will be added in #125 . should be released in few days

dpad works fine, for other buttons please read this article: https://developer.samsung.com/tv/develop/guides/user-interaction/remote-control

@MwareSolutons basic focus support now deployed and built into hello world template.
available in 0.23.34

Hello there ,
I have tried to install the hello world template but navigation between the two buttons did not really work neither on webOS nor on web ,  with the exception of Enter button

What rnv version are you using?
On Tue, Oct 15, 2019, 13:31 Ahmad ElMadi<EMAIL_ADDRESS>wrote:

Hello there ,
I have tried to install the hello world template but navigation between
the two buttons did not really work neither on webOS nor on web , with the
exception of Enter button
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/pavjacko/renative/issues/127?email_source=notifications&email_token=ABEQ5OWFKAOF7DPLZMNAHD3QOWSXVA5CNFSM4H5G62E2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEBINDJI#issuecomment-542167461,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABEQ5OUA6KTFSCLWRRE6ZTLQOWSXVANCNFSM4H5G62EQ
.

0.27.5

1.0.0-rc.18
No focus on both webOS and Tizen devices.
related to https://github.com/flexn-io/renative/issues/869

Solved in 1.0.0


Implement incrmcov

Moving incremental covariance. See @stdlib/math/statistics/incr/mvariance.

This has now been implemented beginning with commit e28d0b4e90d41efb85dbbb6b47cdcb88a208a73f.


Fix: Race condition for simultaneous NN assignment requests

A race condition existed when two NN assignment requests are received concurrently, where both requests would get the same NN. This was probably never gonna happen, but once I wrote a test case confirming it was possible I had to fix it :)

Idk locking a random arbitrary row works for this but just feels bad. There’s got to be a better way

Yeah agreed. I think there might be room here for a better strategy. Now
that we have the tests to reproduce the race condition we can easily
evaluate different locking options
Andrew Dickinson
Computer Science BSE (December 2020) - University of Michigan
On Fri, Feb 9, 2024 at 20:26 Willard Nilges @.***>
wrote:

@.**** commented on this pull request.
I suppose I'd like to have bit of a conversation about the async stuff
before I approve it just so I understand the architecture better, and to
make sure that we don't do something that would shoot us in the foot later.
In src/meshapi/views/forms.py
https://github.com/WillNilges/meshdb/pull/160#discussion_r1484923250:

@@ -88,6 +108,11 @@ def join_form(request):
{"detail": "Your address could not be validated."}, status=status.HTTP_500_INTERNAL_SERVER_ERROR
)

No need to lock all DB objects since we don't modify these tables,

We don't? Just to be clear, do you mean to say that all we do is append?
What if we get a response from the same Member with different info? IIRC,
we update their information, don't we?
In src/meshapi/views/forms.py
https://github.com/WillNilges/meshdb/pull/160#discussion_r1484923687:

This call is a bit out of order, but we need to do this here so that we acquire the lock

before we call Install.objects.select_for_update().get() below in order to prevent deadlocks

free_nn = get_next_free_nn()

This does so to lock as soon as we know the request is valid? Makes sense,
because I don't think that get_next_free_nn has any side effects.
In src/meshapi/views/forms.py
https://github.com/WillNilges/meshdb/pull/160#discussion_r1484923435:

@@ -191,13 +215,37 @@ def join_form(request):
)

+def get_next_free_nn() -> int:

We don't need to lock all the DB objects, just the ones that impact the free NN calculations

Just the ones that impact the free NN calculations
Do you have an example?
Just the first one
Not totally clear on what locking the first one does.

In src/meshapi/views/forms.py
https://github.com/WillNilges/meshdb/pull/160#discussion_r1484922631:

@@ -35,8 +37,25 @@ class JoinFormRequest:
ncl: bool

+def acquire_lock_for_queryset(queryset: QuerySet, object_count=None):

This code may look useless but is extremely important. Here we use select_for_update()

to acquire locks on every single row in the given table. This is a bit hacky, but based

What are the implications of that?
—
Reply to this email directly, view it on GitHub
https://github.com/WillNilges/meshdb/pull/160#pullrequestreview-1873369680,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAX6K6LWDYD3EJDRTM3EIM3YS3EEJAVCNFSM6AAAAABDA3YB3KVHI2DSMVQWIX3LMV43YUDVNRWFEZLROVSXG5CSMV3GSZLXHMYTQNZTGM3DSNRYGA
.
You are receiving this because you authored the thread.Message ID:
@.***>

Okay this is overhauled to move away from hacky-row based locks and use a postgres feature that is specifically intended for this purpose. I feel a lot better about the robustness of this, and the code is a lot cleaner too :)


Clarify when users need to add custom SCITOKENS mappings

(SOFTWARE-4685)

Link failures due to Tiger outage. Merging


Ability to customize editorHoverWidget.foreground color

Issue Type: Feature Request
There is currently support for editorHoverWidget.background and editorHoverWidget.border, but no option to edit the foreground color. For one of my dark themes, I would like to change the foreground color to black and the background color to a lighter color (e.g. white), but this isn't possible at the moment. It would be great if this color customization could be supported.
https://code.visualstudio.com/docs/getstarted/theme-color-reference#_editor-widget-colors
VS Code version: Code 1.25.1 (1dfc5e557209371715f655691b1235b6b26a06be, 2018-07-11T15:33:29.235Z)
OS version: Darwin x64 18.0.0

Already implemented this. We are going to make the pull request this week!


Add cache cleaning target

Issue #, if available:
Description of changes:
Adds cache cleaning to presubmits to help with out-of-space issues as the builder base gets larger
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

/test emissary-tooling-presubmit

/approve


fix: Adjusting the layout of the predicted results

:tada: This PR is included in version 1.14.0-alpha.4 :tada:
The release is available on npm package (@alpha dist-tag)
Your semantic-release bot :package::rocket:

:tada: This PR is included in version 1.18.0-alpha.4 :tada:
The release is available on npm package (@alpha dist-tag)
Your semantic-release bot :package::rocket:

:tada: This PR is included in version 1.10.0-alpha.4 :tada:
The release is available on npm package (@alpha dist-tag)
Your semantic-release bot :package::rocket:

:tada: This PR is included in version 1.10.0 :tada:
The release is available on npm package (@latest dist-tag)
Your semantic-release bot :package::rocket:

:tada: This PR is included in version 1.18.0 :tada:
The release is available on npm package (@latest dist-tag)
Your semantic-release bot :package::rocket:

:tada: This PR is included in version 1.14.0 :tada:
The release is available on npm package (@latest dist-tag)
Your semantic-release bot :package::rocket:

:tada: This PR is included in version 1.0.0 :tada:
The release is available on npm package (@latest dist-tag)
Your semantic-release bot :package::rocket:


Browse: Curate choices should be Effective and Tool

As with the sort choices discussed in #495, the Curate choices should be Effective and Tool.
We will have to come up with other ways to talk about the domain-specific scores (e.g., described and licensed and secure, ...)

@jeffmcaffer as per your comment and changes on #495 , should this one change as well somehow?

This has been addressed in #564  and can be closed


gen2-human-machine-safety: improvements

improved clarity by reducing copied code and improving variable names
vectorised calc_spatials depth averaging
added easier support for alternative averaging methods (e.g. np.median)

Thanks @ES-Alexander and @VanDavv !

I found some issues

Fair enough - I probably should have been more rigorous with, you know, actually testing it :-P
The depth averaging was linked to from somewhere (can't remember if it was discord or in the OAK-D-Lite kickstarter FAQs or something). I swung by to take a look and saw that it could be done better, and once I'd fixed that I saw a couple of other repetitive things that I thought I may as well fix while I was in there :-)

I went ahead quickly and self-addressed the comments below in this PR - #212 - where you can find both your commit and my improvements

Thanks!


Update libsqlite-sys to 0.25.x

Is your feature request related to a problem? Please describe.
I'm getting conflicts in libsqlite-sys library due to different versions used by dependent libraries (0.24 sqlx, and 0.25 in another dependency chain)
Describe the solution you'd like
Update libsqlite-sys / rusqlite crate to a newer version.
Describe alternatives you've considered
Will have to look for a solution.
Additional context
There is a newer version of libsqlite available. It might be a good idea to update for bugfixes and features.

This is already updated in the 0.7 branch. It's just pending a release


pull request for e2e test at 01/29/2017 00:45:14

:white_check_mark: Validation status: passed
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.


Add docker tcp certificate path. Issue #175

Add a cert-path parameter to allow private docker certs for tcp connections.
https://github.com/goodwithtech/dockle/issues/175

@craftyc0der Thank you for your contribution!


Update websocket all processes

Codecov Report

Merging #74 (41be4ba) into main (5bcec27) will decrease coverage by 0.24%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main      #74      +/-   ##
==========================================
- Coverage   84.51%   84.27%   -0.25%
==========================================
  Files          84       84
  Lines        4793     4757      -36
  Branches      804      788      -16
==========================================
- Hits         4051     4009      -42
- Misses        601      613      +12
+ Partials      141      135       -6

Impacted Files
Coverage Δ

orchestrator/api/api_v1/endpoints/processes.py
71.35% <100.00%> (-3.08%)
:arrow_down:

orchestrator/services/processes.py
89.84% <100.00%> (+2.09%)
:arrow_up:

orchestrator/utils/show_process.py
100.00% <100.00%> (ø)

orchestrator/websocket/__init__.py
80.64% <100.00%> (-4.29%)
:arrow_down:

...tor/websocket/managers/memory_websocket_manager.py
56.81% <0.00%> (-23.19%)
:arrow_down:

orchestrator/websocket/websocket_manager.py
69.04% <0.00%> (-2.39%)
:arrow_down:

orchestrator/cli/scheduler.py
69.69% <0.00%> (-0.90%)
:arrow_down:

orchestrator/domain/base.py
90.34% <0.00%> (-0.55%)
:arrow_down:

.../websocket/managers/broadcast_websocket_manager.py
40.00% <0.00%> (ø)

... and 2 more

Continue to review full report at Codecov.

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


fix: update FUSION_LOG_LEVEL configuration in vite config generation

Why
missing process env in production building of applications
closes: https://github.com/equinor/fusion/issues/343
Check off the following:

[x] Confirm that I checked changes to branch which I am merging into.

I have validated included files
My code does not generate new linting warnings
My PR is not a duplicate, check existing pr`s

[x] Confirm that the I have completed the self-review checklist.

[x] Confirm that my changes meet our code of conduct.

@eikeland can you checkout branch, build the context cookbook (fusion-framework-cli app pack) and test it in a dev-portal.
Also check with fusion-framework-cli app dev and verify that local log is set to '3'


Julia installation instructions do not work

Hello and congratulations on this very nice software package! I followed the instructions on how to install the Julia interface to the solver, but it looks like OSQP.jl is not yet a registered package.
I suppose the correct procedure should be that of issuing
Pkg.clone("https://github.com/oxfordcontrol/OSQP.jl.git")

rather than using Pkg.add.

Hi! Thank you for using OSQP!
OSQP is already a registered package in METADATA.jl.
Which version of Julia are you using? With Julia 0.6 it should download and install without any problems. No need to use Pkg.clone.

Woops, my bad, my METADATA was probably out of date. Everything working as expected.
Keep up the good work!


Make Postgres async query public

Follow up to #248. Last step in making the new Query API public.

Codecov Report

Merging #249 (415336a) into main (136cf4c) will increase coverage by 4.04%.
The diff coverage is 76.25%.

:exclamation: Current head 415336a differs from pull request most recent head ab77d02. Consider uploading reports for the commit ab77d02 to get more accurate results

@@            Coverage Diff             @@
##             main     #249      +/-   ##
==========================================
+ Coverage   39.72%   43.77%   +4.04%
==========================================
  Files         115      115
  Lines        9505     9506       +1
==========================================
+ Hits         3776     4161     +385
+ Misses       5729     5345     -384

Flag
Coverage Δ

unittests
43.77% <76.25%> (+4.04%)
:arrow_up:

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

Impacted Files
Coverage Δ

...es/PostgresNIO/Connection/PostgresConnection.swift
15.42% <0.00%> (+0.72%)
:arrow_up:

...s/PostgresNIO/New/Extensions/ByteBuffer+PSQL.swift
100.00% <ø> (ø)

Sources/PostgresNIO/New/PostgresCodable.swift
91.66% <ø> (ø)

...s/PostgresNIO/New/Data/Bytes+PostgresCodable.swift
64.70% <33.33%> (-8.83%)
:arrow_down:

...es/PostgresNIO/New/Data/Date+PostgresCodable.swift
89.28% <50.00%> (ø)

...PostgresNIO/New/Data/Decimal+PostgresCodable.swift
74.19% <50.00%> (+6.45%)
:arrow_up:

...s/PostgresNIO/New/Data/Float+PostgresCodable.swift
76.66% <50.00%> (+3.33%)
:arrow_up:

...IO/New/Data/RawRepresentable+PostgresCodable.swift
82.35% <50.00%> (ø)

...ces/PostgresNIO/New/Data/Int+PostgresCodable.swift
36.60% <64.70%> (+11.11%)
:arrow_up:

...s/PostgresNIO/New/Data/Array+PostgresCodable.swift
93.25% <90.00%> (-0.43%)
:arrow_down:

... and 36 more


Dropdown menu of Weather site

Describe the bug
There is a line after the second place name which is not required in the dropdown menu of places.
Screenshots

I want to remove that line and make it look good,
Please assign me this issue @TusharKesarwani

That website is quite static otherwise even. Try to make that dropdown clickable, add pages leading to usage guide, and about this app, also make the card's length a bit longer in order to make that visually attractive.

Can I know what information to include in about and usage guide pages, should it be my own or could you give any idea please

Write on your own, an abstract about how the app work, and its motive


chore(main): release 0.7.0

:robot: I have created a release beep boop
0.7.0 (2023-03-04)
Features

enhance please-release action (27ae895)
enhance please-release action (580286a)
enhance release-please action (8bef7cb)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/samuele-cozzi/obsidian-marp-slides/releases/tag/v0.7.0 :sunflower:


dark mode option

added dark mode option

Hey @sakettt25 update your branch with the main branch because the custom scroll bar is not visible in the deployment preview and resolve the merge conflicts.


GEODE-3718: shutdownNow should also interrupt non executing runnables

This is a follow up commit for GEODE-3718.  shutdownNow was causing some threads waiting for future.get() to get hung.

@kirklund @pivotal-jbarrett @boglesby @dschneider-pivotal
Would you be able to review or determine if this change is ok?  It's either this change (or something similar) or reverting the previous change https://github.com/apache/geode/pull/3655

I'm going to merge this and if needed, we can revert.  Thanks @kirklund @onichols-pivotal @jujoramos for the reviews!


Build error with the plgin

/flutter/.pub-cache/hosted/pub.dartlang.org/device_calendar-0.1.2/ios/Classes/D
eviceCalendarPlugin.m:2:9: fatal error: 'device_calendar/device_calendar-Swift.h' file not found
#import <device_calendar/device_calendar-Swift.h>

I didn't even add any code yet. That was me just including the plugin into the project.

There's a general issue with Flutter consuming plugins developed in Swift within an Objective-C iOS project, which I assume is the scenario here as well. A link has been added to the readme with a link to the issue being tracked by the Flutter team. You may need to consider converting the iOS side of your project into Swift

Well, I definitely have Objective C Plugins that are working in my project but the work I do is primarily in dart.

From what I recall, Objective-C plugins should work fine within a Swift project. It's more of an issue when it's the other way around. The plugins and your project don't necessarily need to be the same language

Just for anyone checking this in the future (like me), I fixed this issue by making sure 'use_frameworks!' was in the Podfile.


Extract rustc-link-search from library path

This is needed for libraries in non-default lib dirs to be found

I didn't change the behavior on Windows because I'm not sure what is expected/needed there


Remove unnecessary options

I'm considering removing the canonicalUris and replaceInExtensions options.
When using rev-manifest.json, filepaths already use forward slashes by default. The canonicalUris option thus only intervenes when gulp-rev and gulp-rev-rewrite are used in the same stream. However, forward slashes are prevalent in web development, which is what this plugin would be used for in the first place.
As for replaceInExtensions, it seems better to filter by file extension in the initial gulp.src() glob.
I'm opening this issue to see if people have use cases for these options that I'm not considering and that would prevent their removal.

Closed in ded7170739ca5826192d2c5f657279b4a0660ac2 and 6ec52e41eec9d875d2dbceb377c6ddcc17927383.


Add support for OPENSEARCH_JAVA_HOME

Signed-off-by: Andriy Redko<EMAIL_ADDRESS>Description
Add support for OPENSEARCH_JAVA_HOME
Issues Resolved
Closes https://github.com/opensearch-project/security/issues/1600
Testing
The scripts have been tested under Windows / Linux
Check List

[ ] New functionality includes testing
[ ] New functionality has been documented
[X] Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

@dblock whenever you have time please

Codecov Report

Merging #1603 (7aee239) into main (933dc8f) will decrease coverage by 0.35%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##               main    #1603      +/-   ##
============================================
- Coverage     64.68%   64.33%   -0.36%
+ Complexity     3219     3214       -5
============================================
  Files           247      247
  Lines         17351    17342       -9
  Branches       3082     3068      -14
============================================
- Hits          11224    11157      -67
- Misses         4577     4635      +58
  Partials       1550     1550

Impacted Files
Coverage Δ

...t/keybyoidc/AuthenticatorUnavailableException.java
0.00% <0.00%> (-40.00%)
:arrow_down:

...ava/com/amazon/dlic/auth/ldap2/MakeJava9Happy.java
8.69% <0.00%> (-34.79%)
:arrow_down:

...org/opensearch/security/rest/TenantInfoAction.java
77.94% <0.00%> (-10.30%)
:arrow_down:

.../auth/http/jwt/keybyoidc/SelfRefreshingKeySet.java
59.85% <0.00%> (-8.46%)
:arrow_down:

...com/amazon/dlic/auth/ldap2/PrivilegedProvider.java
44.26% <0.00%> (-6.56%)
:arrow_down:

...ic/auth/ldap/backend/LDAPAuthorizationBackend.java
53.59% <0.00%> (-4.36%)
:arrow_down:

...ic/auth/http/jwt/AbstractHTTPJwtAuthenticator.java
55.81% <0.00%> (-3.49%)
:arrow_down:

.../dlic/auth/http/jwt/keybyoidc/KeySetRetriever.java
76.82% <0.00%> (-2.44%)
:arrow_down:

.../org/opensearch/security/support/ConfigHelper.java
80.39% <0.00%> (-1.97%)
:arrow_down:

...zon/dlic/auth/ldap2/LDAPAuthorizationBackend2.java
28.35% <0.00%> (-0.38%)
:arrow_down:

... and 9 more

Continue to review full report at Codecov.

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


Clock bugs when kicking a player

Thank you for the update, I'm thrilled you did my request.
The console outputs that it's sending the help message repeatedly after kicking a player.
Not when banning a player. Can you replicate?
I did some digging and my best guess is that the clock is counting incorrectly after kicking?
I'm referring to Server::Update(float dt)

Resolved


React with Typescript error

I continue to get this error when I try to add props on a component. I cannot find the solution anf or work around. Is this a bug?
[ts] Property 'title' does not exist on type 'IntrinsicAttributes & IntrinsicClassAttributes & Readonly<{ children?: ReactNode; }> &...'.
TypeScript Version:  2.5.7
Code
import * as React from 'react';
import AppFooter from './Footer';
import AppHeader from './Header';

export default class Layout extends React.Component {
  render() {
    const title = 'Welcome Mark';
    return (
      <div>
        <AppHeader title={title}/>
        <AppFooter />
      </div>
    )
  }
}

This is not a support forum. If you think you've found a bug, please include a way to reproduce the bug. We don't have the definition of AppHeader so we have no way of knowing what might be going wrong with your code.


[Orca-107]/tier 1 validation

This PR adds the Tier 1 validation steps to the OME-TIF validation workflow and refactors some reusable python code. The first new process MD5_VALIDATE calculates an MD5 Checksum and checks it against the MD5 Checksum provided in the input CSV. The FILE_EXT_VALIDATE process takes the extension from the downloaded file's name and checks whether it is either "ome.tiff" or "ome.tif". The script reusable.py now holds two functions package_validation_dict and json_dump which will be used to generate the output JSON files at each validation step.

Excellent PR!  I went ahead and reviewed first, but i'm sure Bruno will also have comments.


Use resuable workflows instead of Chef Delivery

Signed-off-by: Dan Webb<EMAIL_ADDRESS>

Released as: 4.0.3


added more detailed Multus/SRIOV config to Readme

Updates to the Readme to show how to configure the Bond CNI with modern incarnations of Multus and SRIOV CNI.

/lgtm

/lgtm
Let's alos add a simpler docs section under https://github.com/intel/sriov-network-device-plugin/tree/master/docs with the sample deployments files showing the SR-IOV usecase with link aggregation and HA.


Add SBom Manifest Generation

Platforms Impacted

[X] Android

Description of changes
Add required steps to generate an SBOM manifest and archive the manifest.

@mishramayank1, @aishwarya56, @iaulakh: Friendly ping if you could have a look at this PR to make it EO Compliant

hi @dannyvv ,
Kindly raise the PR to develop branch in order to maintain parity with master branch.


@Delegate: idea doesn't know that geter is used by @delegate

Suppose I've got class A with getter getData. Class B delegates some calls to A. Some method calls (new B()).getData(). If I open A and try to find usages of getData (alt+f7) IDEA will not find anything + getData will be grey (which means it's not used)
I think it makes usage of @Delegate pretty dangerous (in case of refactoring). Can you fix it?

This is duplicate of #76 .


chore: update dependencies

:tada: This PR is included in version 5.0.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Influxdb java 182 org

test

Codecov Report

Merging #2 into master will increase coverage by 0.43%.
The diff coverage is 87.09%.

@@             Coverage Diff              @@
##             master       #2      +/-   ##
============================================
+ Coverage     86.04%   86.48%   +0.43%
- Complexity      293      301       +8
============================================
  Files            20       20
  Lines          1261     1287      +26
  Branches        133      134       +1
============================================
+ Hits           1085     1113      +28
+ Misses          116      114       -2
  Partials         60       60

Impacted Files
Coverage Δ
Complexity Δ

src/main/java/org/influxdb/InfluxDB.java
100% <ø> (ø)
0 <0> (ø)
:arrow_down:

src/main/java/org/influxdb/impl/InfluxDBImpl.java
87.34% <100%> (+1.06%)
67 <3> (+3)
:arrow_up:

src/main/java/org/influxdb/dto/BatchPoints.java
72.34% <75%> (+3.29%)
22 <2> (+2)
:arrow_up:

src/main/java/org/influxdb/dto/Point.java
83.52% <91.66%> (+0.59%)
34 <3> (+3)
:arrow_up:

Continue to review full report at Codecov.

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


Last release looks broken

Hi again, the last release seems to be broken. When installing with yarn you get:
error Couldn't find a package.json (or bower.json) file in /Users/posva/.yarn-cache/npm-rollup-plugin-vue-2.2.9

I searched but couldn't find any useful information. The package.json is there and removing node modules + cleaning the cache does nothing. 😔

I have to test it once (Used yarn first time to publish a release).

Fixed with v2.2.10


Error: Cannot find module 'gulp-cli'

root@benz-iMac:/home/benz/Downloads/bitcore-8.22.2/packages/bitcore-lib# gulp browser
node:internal/modules/cjs/loader:998
  throw err;
  ^

Error: Cannot find module 'gulp-cli'
Require stack:
- /usr/share/nodejs/gulp/bin/gulp.js
    at Module._resolveFilename (node:internal/modules/cjs/loader:995:15)
    at Module._load (node:internal/modules/cjs/loader:841:27)
    at Module.require (node:internal/modules/cjs/loader:1061:19)
    at require (node:internal/modules/cjs/helpers:103:18)
    at Object.<anonymous> (/usr/share/nodejs/gulp/bin/gulp.js:3:1)
    at Module._compile (node:internal/modules/cjs/loader:1159:14)
    at Module._extensions..js (node:internal/modules/cjs/loader:1213:10)
    at Module.load (node:internal/modules/cjs/loader:1037:32)
    at Module._load (node:internal/modules/cjs/loader:878:12)
    at Function.executeUserEntryPoint [as runMain] (node:internal/modules/run_main:81:12) {
  code: 'MODULE_NOT_FOUND',
  requireStack: [ '/usr/share/nodejs/gulp/bin/gulp.js' ]
}

Node.js v18.12.1
root@benz-iMac:/home/benz/Downloads/bitcore-8.22.2/packages/bitcore-lib# npm install glup-cli -g
npm ERR! code EEXIST
npm ERR! path /home/benz/.nvm/versions/node/v18.12.1/bin/gulp
npm ERR! EEXIST: file already exists
npm ERR! File exists: /home/benz/.nvm/versions/node/v18.12.1/bin/gulp
npm ERR! Remove the existing file and try again, or run npm
npm ERR! with --force to overwrite files recklessly.

npm ERR! A complete log of this run can be found in: /root/.npm/_logs/2023-08-31T12_01_22_435Z-debug-0.log
root@benz-iMac:/home/benz/Downloads/bitcore-8.22.2/packages/bitcore-lib#
root@benz-iMac:/home/benz/Downloads/bitcore-8.22.2/packages/bitcore-lib#
root@benz-iMac:/home/benz/Downloads/bitcore-8.22.2/packages/bitcore-lib#
root@benz-iMac:/home/benz/Downloads/bitcore-8.22.2/packages/bitcore-lib#
root@benz-iMac:/home/benz/Downloads/bitcore-8.22.2/packages/bitcore-lib# npm remove gulp --force
npm WARN using --force Recommended protections disabled.
npm WARN old lockfile
npm WARN old lockfile The package-lock.json file was created with an old version of npm,
npm WARN old lockfile so supplemental metadata must be fetched from the registry.
npm WARN old lockfile
npm WARN old lockfile This is a one-time fix-up, please be patient...
npm WARN old lockfile
npm WARN deprecated<EMAIL_ADDRESS>See https://github.com/lydell/source-map-resolve#deprecated
npm WARN deprecated<EMAIL_ADDRESS>Chokidar 2 does not receive security updates since 2019. Upgrade to chokidar 3 with 15x fewer dependencies
npm WARN deprecated<EMAIL_ADDRESS>this library is no longer supported
npm WARN deprecated<EMAIL_ADDRESS>Please upgrade  to version 7 or higher.  Older versions may use Math.random() in certain circumstances, which is known to be problematic.  See https://v8.dev/blog/math-random for details.
npm WARN deprecated<EMAIL_ADDRESS>request has been deprecated, see https://github.com/request/request/issues/3142
npm WARN deprecated<EMAIL_ADDRESS>Legacy versions of mkdirp are no longer supported. Please update to mkdirp 1.x. (Note that the API surface has changed to use Promises in 1.x.)
npm WARN deprecated<EMAIL_ADDRESS>Debug versions >=3.2.0 <3.2.7 || >=4 <4.3.1 have a low-severity ReDos regression when used in a Node.js environment. It is recommended you upgrade to 3.2.7 or 4.3.1. (https://github.com/visionmedia/debug/issues/797)
npm WARN deprecated<EMAIL_ADDRESS>See https://github.com/lydell/source-map-url#deprecated
npm WARN deprecated<EMAIL_ADDRESS>This module is no longer maintained, try this instead:
npm WARN deprecated   npm i nyc
npm WARN deprecated Visit https://istanbul.js.org/integrations for other alternatives.
npm WARN deprecated<EMAIL_ADDRESS>Please update to ini >=1.3.6 to avoid a prototype pollution issue
npm WARN deprecated<EMAIL_ADDRESS>Please see https://github.com/lydell/urix#deprecated
npm WARN deprecated<EMAIL_ADDRESS>https://github.com/lydell/resolve-url#deprecated

added 847 packages, and audited 848 packages in 28s

76 packages are looking for funding
  run `npm fund` for details

29 vulnerabilities (9 moderate, 15 high, 5 critical)

To address issues that do not require attention, run:
  npm audit fix

To address all issues (including breaking changes), run:
  npm audit fix --force

Run `npm audit` for details.
root@benz-iMac:/home/benz/Downloads/bitcore-8.22.2/packages/bitcore-lib#
root@benz-iMac:/home/benz/Downloads/bitcore-8.22.2/packages/bitcore-lib#
root@benz-iMac:/home/benz/Downloads/bitcore-8.22.2/packages/bitcore-lib# npm install glup-cli -g
npm ERR! code EEXIST
npm ERR! path /home/benz/.nvm/versions/node/v18.12.1/bin/gulp
npm ERR! EEXIST: file already exists
npm ERR! File exists: /home/benz/.nvm/versions/node/v18.12.1/bin/gulp
npm ERR! Remove the existing file and try again, or run npm
npm ERR! with --force to overwrite files recklessly.

npm ERR! A complete log of this run can be found in: /root/.npm/_logs/2023-08-31T12_02_48_485Z-debug-0.log
root@benz-iMac:/home/benz/Downloads/bitcore-8.22.2/packages/bitcore-lib# rm -R /home/benz/.nvm/versions/node/v18.12.1/bin/gulp
root@benz-iMac:/home/benz/Downloads/bitcore-8.22.2/packages/bitcore-lib# npm install glup-cli -g
npm WARN deprecated<EMAIL_ADDRESS>this package has been deprecated

added 20 packages in 1s

2 packages are looking for funding
  run `npm fund` for details
root@benz-iMac:/home/benz/Downloads/bitcore-8.22.2/packages/bitcore-lib#
root@benz-iMac:/home/benz/Downloads/bitcore-8.22.2/packages/bitcore-lib#
root@benz-iMac:/home/benz/Downloads/bitcore-8.22.2/packages/bitcore-lib#
root@benz-iMac:/home/benz/Downloads/bitcore-8.22.2/packages/bitcore-lib#
root@benz-iMac:/home/benz/Downloads/bitcore-8.22.2/packages/bitcore-lib# gulp browser
node:internal/modules/cjs/loader:998
  throw err;
  ^

Error: Cannot find module 'gulp-cli'
Require stack:
- /usr/share/nodejs/gulp/bin/gulp.js
    at Module._resolveFilename (node:internal/modules/cjs/loader:995:15)
    at Module._load (node:internal/modules/cjs/loader:841:27)
    at Module.require (node:internal/modules/cjs/loader:1061:19)
    at require (node:internal/modules/cjs/helpers:103:18)
    at Object.<anonymous> (/usr/share/nodejs/gulp/bin/gulp.js:3:1)
    at Module._compile (node:internal/modules/cjs/loader:1159:14)
    at Module._extensions..js (node:internal/modules/cjs/loader:1213:10)
    at Module.load (node:internal/modules/cjs/loader:1037:32)
    at Module._load (node:internal/modules/cjs/loader:878:12)
    at Function.executeUserEntryPoint [as runMain] (node:internal/modules/run_main:81:12) {
  code: 'MODULE_NOT_FOUND',
  requireStack: [ '/usr/share/nodejs/gulp/bin/gulp.js' ]
}

Node.js v18.12.1

@benzmuircroft
Since I found the word bitcore-lib in your report, I downloaded it and tried to run the following commands:
$ node -v
v18.12.1
$ git clone https://github.com/bitpay/bitcore.git
$ cd bitcore/packages/bitcore-lib
$ npm install
$ which gulp
gulp not found
$ npx gulp browser
[12:27:37] Using gulpfile /path/to/bitcore/packages/bitcore-lib/gulpfile.js
[12:27:37] Starting 'browser'...
[12:27:37] Starting '<anonymous>'...
[12:27:40] Finished '<anonymous>' after 2.64 s
[12:27:40] Starting '<anonymous>'...
[12:27:45] Finished '<anonymous>' after 4.81 s
[12:27:45] Finished 'browser' after 7.46 s
$ npm install -g gulp-cli
$ which gulp
/path/to/home/.nodebrew/current/bin/gulp
$ gulp browser
[12:30:57] Using gulpfile /path/to/bitcore/packages/bitcore-lib/gulpfile.js
[12:30:57] Starting 'browser'...
[12:30:57] Starting '<anonymous>'...
[12:30:59] Finished '<anonymous>' after 1.91 s
[12:30:59] Starting '<anonymous>'...
[12:31:03] Finished '<anonymous>' after 4.77 s
[12:31:03] Finished 'browser' after 6.68 s$

Gulp is seemd to run normally. Please check where gulp and gulp-cli are installed and retry again.
I'll close this issue, but if you find any further problems, please reopen this issue and report them.

@sttk please don't answer issues like this that get raised. Support issues are supposed to be submitted as Discussions and assisted by the community.


Optional Map

Add optionalMap and optionalFlatMap methods to EventLoopFuture  where the callback is only fired if the optional value in the future is not nil:
let id: Future<Int?> = ...
id.optionalMap { id in
    type(of: id) // Int
}

+1 to this.
I think it could make sense to flip the words though, to better match #2.
let id: Future<Int?> = ...
id.mapOptional { id in
    type(of: id) // Int
}
id.thenOptional { id in
    type(of: id) // Int
}

Maybe it would be useful to add more monad transformers? For instance.
let ids: Future<[Int]> = ...
ids.mapArray { id in
    type(of: ids) // Int
}
ids.thenArray { id in
    type(of: id) // Int
}

Unfortunately I don't think there is a generic way of writing these without Higher Kinded types.

Just realized there was already an issue for this, oops. Ignore my previous comment! I think adding these is a great idea.


Generators to SDK 4.22.0

#minor

Pull Request Test Coverage Report for Build 384141

0 of 0   changed or added relevant lines in 0 files are covered.
6 unchanged lines in 3 files lost coverage.
Overall coverage decreased (-0.02%) to 78.372%

Files with Coverage Reduction
New Missed Lines
%

/libraries/AdaptiveExpressions/BuiltinFunctions/GetNextViableTime.cs
1
90.91%

/libraries/AdaptiveExpressions/BuiltinFunctions/GetPreviousViableTime.cs
1
90.91%

/libraries/AdaptiveExpressions/LRUCache.cs
4
87.18%

Totals

Change from base Build 384139:
-0.02%

Covered Lines:
26170

Relevant Lines:
33392

💛  - Coveralls


Don't notify using a forcerenamed name for namelocked users

On second thought, this seems like the incorrect solution.

It should still warn if other, non-namelocked users use that name, right?

Hm, actually, other users will still get namelocked for trying to use that name, so I guess the warning isn't necessary in that case, either.


Add Gutenberg

Add hugo-theme-gutenberg from https://github.com/gcaracuel/hugo-theme-gutenberg

Screenshot meets requirements as all the other PRs but tested some of them and all them seems imàcted by same issue I would say on this project code that prevents the screenshot.pnf file to be loaded once entered on theme section.
Any idea what could be the issue? I am happy to help.


Can we expose terminal::previous_buffer() to read the content on the screen?

Hi, thank you for the great crate.
Is your feature request related to a problem? Please describe.
I'm adding mouse event support to my CLI tool - I would like to get the text at the mouse position. I initially thought that terminal::current_buffer_mut() would be useful, however after a .draw() call, the current buffer refers to the reset buffer.
If I had a reference to the other buffer with state from the .draw() call, it would be easy to get the symbol at the cursor position:
// terminal.draw(..);
// ...

Event::Mouse(mouse_event) => match mouse_event {
    MouseEvent::Press(MouseButton::Left, x, y) => {
        let buffer = terminal.previous_buffer();
        let index = x + (y * buffer.area.width);
        let cell = &buffer.content[index as usize];
        let symbol = cell.symbol.trim();
    }
}

It would be bad for the user to mutate this buffer, so we could expose a read-only reference.
Describe the solution you'd like
Terminal exposes a read-only reference to the previous buffer (which contains the cell state after a .draw()) via a public method.
Additional context
I'm able to hack my way around this by cloning the current_buffer_mut() before the draw call and rendering the widgets into it directly.
// HACK: capture the buffer only if we want to read from it after drawing.
let mut buffer = if let Event::Mouse(MouseEvent::Press(MouseButton::Left, _, _)) = event
{
    Some(terminal.current_buffer_mut().clone())
} else {
    None
};

terminal.draw(|f| {
     // ..
    let widget = ..;
    if let Some(buffer) = buffer.as_mut() {
        widget.clone().render(chunks[0], buffer);
    }
    f.render_widget(widget, chunks[0]);
});

if let Event::Mouse(MouseEvent::Press(MouseButton::Left, _, _)) = event {
    // read symbol from buffer, everything works as expected
}

Obviously this is not ideal since it clones the widget and renders it twice.

Thanks for the detailed report. This seems like a valid feature request and as you rightfully exposed probably mandatory if you wish to get the terminal content as soon as you handle the mouse events yourself. Let me know if the changes in #440 would fit your use case.

Thanks for the detailed report. This seems like a valid feature request and as you rightfully exposed probably mandatory if you wish to get the terminal content as soon as you handle the mouse events yourself. Let me know if the changes in #440 would fit your use case.

Those changes look good 👍🏻  and would work for my use-case!

Those changes look good 👍🏻  and would work for my use-case!