fix: e2e build step

run-e2e.sh is currently failing because the build step is incorrect
Before:
npm run build &&

After:
deno task build &&

I'm guessing the broken step once relied on a package.json script command that no longer exists.

Heya! I'm not sure what exactly is broken, but I'm happy to learn about it!
https://github.com/surrealdb/surrealdb.js/actions/runs/4666599995/jobs/8261368767

Ah, I see! This is for running it manually on your local device. Yep, looks good. Thanks!

Yup!


Design issue in password field on signup page

Password field on signup page looks okay in small screens.

But breaks for large-screen.

This issue has been fixed by @rahulnegi20 , Thanks Guys.


Reusable pipeline specs

It would be nice if you could reuse a pipeline specification by overriding values like name, inputs, etc from the cli like in helm:
helm install --set accessKey=myaccesskey,secretKey=mysecretkey \
    stable/minio

So, something like:
pachctl create-pipeline -f my-pipeline.json --set name=extract-features-from-new-data,inputs.atom.repo=my-new-repo

This isn't incredible urgent but it would definitely be nice for pipeline reusability if the spec didn't need to be copied, tweaked then run if we just want to run the pipeline with a couple slight modifications

This strikes me as a bit outside the purview of pachyderm. You can get a really good user experience for this stuff using jq it's very unlikely that we'd ever be able to have an experience anywhere near as jq's.

I can see this being useful once we get to a point where people start sharing pipelines online.

I was actually thinking about something similar yesterday. Would be great to be able to have a marketplace for pipelines. Templating like helm may be a first step?

I think this could be solved by transforming a pipeline spec into a Helm chart.
We can integrate pachctl with Helm pretty easily by talking to Tiller which is a GRPC service.
pachctl can be used as the following to create the pipeline:
pachctl create-pipeline --chart my-pipeline --values my-values.yaml

After a slack conversation we realized we don't have pachctl create-pipeline --dry-run so we don't have the basic building block for a reusable pipeline.

After a slack conversation we realized we don't have pachctl create-pipeline --dry-run so we don't have the basic building block for a reusable pipeline.
I guess this issue could be focussed on updating actual pipeline specs (which I agree would be useful) and we could have a separate issue for adding something like --dry-run to create-pipeline and update-pipeline for tweaking the more general k8s deployment config.

Relevant slack convo: https://pachyderm-users.slack.com/archives/C3UN0S0Q0/p1508146446000362

Homme Zwaagstra
2:18 AM
Hi all,  does anyone know of a way to set tolerances for pipeline workers?  Our situation is that we have an instance group composed of high performance (read expensive!) nodes that scales down to 0 when there is no k8s pod pressure.   We only want pipeline workers to run on this instance group and not any of the other services in the cluster. This is usually the case and there aren't any problems, but occasionally a service will be evicted and end up re-deploying on this worker instance group when a job is running.  This prevents the instance group from scaling down to 0 and costs :moneybag: over the longer term. We would be like any taints that we have set on nodes in the IG to be tolerated by Pachyderm workers but can't figure out how to set these tolerances.  Should this be in the pipeline spec, is there another way to set tolerances, or is there a completely different approach that gets us to where we want to be? Thanks :slightly_smiling_face: (edited)
Omer Katz
2:34 AM
@Homme I strongly suggest to avoid pipeline.spec for anything other the initial generation of the relevant YAML files.
2:35
The workflow should be pipeline.spec -> Draft (for dev https://github.com/Azure/draft) -> Helm (for deployment)
GitHub
Azure/draft
draft - A tool for developers to create cloud-native applications on Kubernetes.
2:36
You can generate a yaml file to work with by using the --dry-run option of pachctl create-pipeline (edited)
2:38
Any non-trivial deployment should use the k8s primitives and not the abstractions pachyderm provides for data scientists imo
Homme Zwaagstra
2:39 AM
Hi @the_drow and thanks for the reply and suggestion.  Being able to edit the pipeline YAML would be a perfect workaround to not being able to specify tolerances in the spec, and I did previously have a look at the create-pipeline command to just get YAML output, but couldn't find anything useful: I still don't see the --dry-run option for that command (I'm running 1.6.1).
Omer Katz
2:39 AM
I might be confusing with some other option name but there is such option
Homme Zwaagstra
2:40 AM
Any non-trivial deployment should use the k8s primitives and not the abstractions pachyderm provides for data scientists imo
That makes sense to me, and I already use some other services running alongside our pipeline to egress/ingress data to good effect.
2:42
@the_drow I can see the --dry-run option for deploy, but nothing for create-pipeline, unless I'm missing something?
# pachctl create-pipeline -h
Create a new pipeline from a [Pipeline Specification](../reference/pipeline_spec.html)

Usage:
  pachctl create-pipeline -f pipeline.json [flags]

Flags:
  -f, --file string       The file containing the pipeline, it can be a url or local file. - reads from stdin. (default "-")
      --password string   Your password for the registry being pushed to.
  -p, --push-images       If true, push local docker images into the cluster registry.
  -r, --registry string   The registry to push images to. (default "docker.io")
  -u, --username string   The username to push images as, defaults to your OS username.

Global Flags:
      --no-metrics   Don't report user metrics for this command
  -v, --verbose      Output verbose logs

Omer Katz
2:43 AM
Yeh you are right. It is missing
Homme Zwaagstra
2:44 AM
That would be very useful functionality to have, though - nice idea for an enhancement.
2:44
update-pipeline doesn't have it either.
Omer Katz
2:45 AM
See https://github.com/pachyderm/pachyderm/issues/1833
Homme Zwaagstra
2:49 AM
Cool, although that issue would only be useful in this situation if it could be used to update k8s specs that aren't in the pipeline spec.  I like the --dry-run option better as it seems more flexible.
Omer Katz
2:52 AM
What you should have is the option to generate a helm chart or a draft from a pipeline spec. --dry-run is very basic
Homme Zwaagstra
2:55 AM
Auto generation of charts or drafts from a pipeline spec would be handy, but a lower level --dry-run option would be very useful too.  The former might be able to build on the latter, but either way manipulating the k8s spec before it is applied to the cluster would be great.
Omer Katz
2:57 AM
In the meantime you can extract the yaml template from the code
Homme Zwaagstra
3:00 AM
Yes, that's an option. I was thinking it may be easier to do something like kubectl get -o yaml rc/pipeline-my-pipeline-v1 | my-update-script | kubectl apply -f -.

this mechanism could also be useful for creating pipelines based on a standard template and for creating multiple, slightly altered versions of a pipeline (ie DAG branching)

@gabrielgrant @homme The tolerances (to accept node taints) should be able to be set with a pod_spec or pod_patch fields in the pipeline.  Does anyone know why this is not supported in the "scheduling_spec"?

@ktarplee Thanks for flagging this issue. The simple answer is it has not been frequently asked. Can you describe your use case? It will help us better understand the instances when pod_patch does not work.

@nitinjainsj We have a k8s cluster with three types of nodes

NVIDIA DGX-2 (16 GPUs)
Intel nodes with 4 NVIDIA GPUs
AMD nodes with no GPUs and lots of cores.

We tainted the DGX-2 nodes with batch=big:PreferNoSchedule to prevent pods from being scheduled to it unless it needs lots of GPUs.  In which case you can create a pod to tolerate the batch=big taint.
So just making that easy to do in Pachyderm would be nice.  pod_spec should work for this case but it's cumbersome.

@ktarplee Thanks for the info. I have added our feature request tag. We will look at adding this in a future release.


RepoBasedController#find_repo: Automatically Update Repository If updated_at Is Over 6 Hours Ago

The changes introduced will cause RepoBasedController#find_repo to check if the repository was updated over 6 hours ago, and queue an UpdateRepoInfoJob if so.  It returns the result of Repo.find_by_full_name either way, and only performs the check and job queuing if the result of this find is truthy.
I'm unsure about the performance drawbacks of this, since this check is performed every time a RepoBasedController uses the #find_repo method to locate a Repository.  I think this could also be split out into a separate method, and then the new method could be more appropriately used.
I'm willing to work on improvements to this as well.

Another performance drawback that I can think of is: if the queue of UpdateRepoInfoJobs is too big, it is possible to flood the queue with jobs to update a repo if one makes repeated requests and keeps firing that callback.  It might make sense to check if the queue already has an UpdateRepoInfoJob for a repository before adding another job, or to introduce some kind of lock on the repository that prevents it from repeatedly being spammed with requests to update it.

I agree, I don't see why this has to be done during a search and not just something done on a regular scheduled cron. Closing this solution, but raising the problem in issue

It certainly seems like it would be better if you could get specific information for all repos in one request, but I don't think GitHub's API supports obtaining that much information in the public /repositories endpoint.

Is using GraphQL an option? I don't have experience with it, nor with GitHub's API, but it looks promising for a use case like this.


Replace beta 1 diagnostics visualization with a modified VisualProfiler-Unity fork

Overview
NOTE: This is a breaking change from Beta 1 as the diagnostic visualization has been completely replaced.
This change removes much of the beta 1 diagnostics system implementation and replaces it with a modified fork of the Visual Profiler from https://github.com/Microsoft/VisualProfiler-Unity.
The diagnostics profile currently contains only "Is Profiler Visible".
Changes to the forked VisualProfiler script include:

Renaming the class to MixedRealityToolkitVisualProfiler (to avoid collisions if a customer already has the original in the project)
Renaming the namespace to Microsoft.MixedReality.Toolkit.Services.DiagnosticsSystem (consistency and collision avoidance)
Rename initiallyVisible to isVisible and add a read/write property to enable toggling
Remove keyword support (now handled by demo scripts in conjunction with the Speech Commands Profile)
Add public property to set the profiler window's parent (hierarchy organization)
Modify LateUpdate() to show/hide the profiler window as appropriate

Fixes: #3245, #3056

found a bug in the toggle logic... will re-open when fixed


Add return type to GetDirectoryNameOfFileAbove property function

GetDirectoryNameOfFileAbove returns a string, but this return type is missing in the property functions table, while all the other functions have their return types defined in the table.
Happy Holidays! 🥳

#sign-off

@ghogen

Can you review this PR?
IMPORTANT: When this content is ready to merge, you must add #sign-off in a comment or the approval may get overlooked.

#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team


Added French translations to the strings resources

Added French translations to the strings resources.
I'll be of course willing to help for future translations 😄

Thanks! I'm going to find a french speaking friend to double check this since I obviously can't review myself :P

No problem. I had sometimes to use something a bit different than the raw translation to make the text fit on screen.

Friend looked it over and says it looks good. Thanks!


Add user registration functionality

send username, email, password to /api/auth/register endpoint by validating if password and confirm password are same at the frontend by default USER must be the default role ADMIN is only done at the database

This is the current request format to the register

The integration works, role based login is working fine


Format MOD files.

and list the usecase directories in order.

Is there a formatting tool for these you are using, or is it all by hand?

Vim is the best formatting tool!


feat: Init sudt backend by kuai init

seems that test failed?

I think we can approve and merge this PR quickly because it's simply generated by the cli as a codebase.


Multiple DbSets of the same entity

I'm working on a tool that is doing data migrations, and as part of that I need to store mappings of record ids in the old system to that in the new system.  Since these mappings are essentially the same schema, I tried to model this as follows:
//DB Context
public DbSet<EntityMapping> UserMapping { get; set; }
public DbSet<EntityMapping> RoleMapping { get; set; }

//Common Entity
public class EntityMapping
{
	[Key]
	public int SourceId { get; set; }
	[Required]
	public int DestinationId { get; set; }
}

However all that just ended up with one table in the (sql) database called EntityMapping.
What is the recommended approach for modelling the same entity, in multiple tables?

This isn't something we are planning to support in EF Core. Probably the best option is to use an un-mapped base type, and then have a concrete type for each table.
//DB Context
public DbSet<UserMapping> UserMapping { get; set; }
public DbSet<RoleMapping> RoleMapping { get; set; }

//Common Entity
public abstract class EntityMapping
{
    [Key]
    public int SourceId { get; set; }
    [Required]
    public int DestinationId { get; set; }
}

public class UserMapping : EntityMapping
{
}

public class UserMapping : RoleMapping
{
}


[BUG] fastfetch caused core dumped on vmware virtual machine

General description of bug:

What happened: I am trying to use the fastfetch on my virtual machine(host: esxi 8.0 u1)However, fastfetch didnt work and caused the Segmentation fault (core dumped)
What should happen: Run fastfetch
Did it work in an older version: Unknown
Where did you get the binary: Installed with a package manager(RockyLinux 9.3 dnf)
Does this issue still occurs in the latest dev build? Unknow

Often helpful information:
Screenshot:

Please provide the stack trace when crashng. You should follow the instructions of if fastfetch crashed section in the bug report template


Delete quartz job when project not exist

Purpose of the pull request

Brief change log

Verify this pull request

This pull request is code cleanup without any test coverage.
(or)
This pull request is already covered by existing tests, such as (please describe tests).
(or)
This change added tests and can be verified as follows:

(or)
If your pull request contain incompatible change, you should also add it to docs/docs/en/guide/upgrede/incompatible.md

Codecov Report

Merging #13793 (db4934a) into dev (d91bdef) will decrease coverage by 0.06%.
The diff coverage is 50.00%.

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

@@             Coverage Diff              @@
##                dev   #13793      +/-   ##
============================================
- Coverage     39.16%   39.11%   -0.06%
+ Complexity     4450     4441       -9
============================================
  Files          1132     1133       +1
  Lines         42175    42105      -70
  Branches       4756     4740      -16
============================================
- Hits          16519    16469      -50
+ Misses        23836    23818      -18
+ Partials       1820     1818       -2

Impacted Files
Coverage Δ

...nscheduler/dao/repository/impl/ProjectDaoImpl.java
50.00% <50.00%> (ø)

... and 52 files with indirect coverage changes
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Update version to 1.3.6-0 on staging

Update version to 1.3.6-0

Hey! I see that you made changes to our Form component. Make sure to update the docs in FORMS.md accordingly. Cheers!


Find all references not working in M1

Describe the bug

Not sure if its M1 related but this is the only thing I can think of that is different between my laptops. Find all references works fine in an intel laptop, but with M1 find references is broken. It only finds references within its function scope. All my dotnet, ionide, etc versions are all the same.
It's able to find the declaration, but when it goes to the declaration and I try to find references, then it doesn't find it.
Steps to reproduce

Open a FS Code with simple references2.
Find all references
Notice the difference of find reference results between M1 and Intel based

Link to sample reproduction
https://github.com/Dzoukr/Yobo

Expected behaviour

Same results as Intel based
Screenshots

This is the output from F#
[Error - 12:59:33 AM] Request textDocument/references failed.
  Message: Request cancelled (exn was System.TypeInitializationException: The type initializer for 'Microsoft.Data.Sqlite.SqliteConnection' threw an exception.
 ---> System.Reflection.TargetInvocationException: Exception has been thrown by the target of an invocation.
 ---> System.DllNotFoundException: Unable to load shared library 'e_sqlite3' or one of its dependencies. In order to help diagnose loading problems, consider setting the DYLD_PRINT_LIBRARIES environment variable: dlopen(libe_sqlite3, 0x0001): tried: 'libe_sqlite3' (no such file), '/usr/local/lib/libe_sqlite3' (no such file), '/usr/lib/libe_sqlite3' (no such file), '/Users/l337/Projects/LearningSAFE/libe_sqlite3' (no such file)
   at SQLitePCL.SQLite3Provider_e_sqlite3.NativeMethods.sqlite3_libversion_number()
   at SQLitePCL.SQLite3Provider_e_sqlite3.SQLitePCL.ISQLite3Provider.sqlite3_libversion_number()
   at SQLitePCL.raw.SetProvider(ISQLite3Provider imp)
   at SQLitePCL.Batteries_V2.Init()
   --- End of inner exception stack trace ---
   at System.RuntimeMethodHandle.InvokeMethod(Object target, Span`1& arguments, Signature sig, Boolean constructor, Boolean wrapExceptions)
   at System.Reflection.RuntimeMethodInfo.Invoke(Object obj, BindingFlags invokeAttr, Binder binder, Object[] parameters, CultureInfo culture)
   at Microsoft.Data.Sqlite.Utilities.BundleInitializer.Initialize()
   at Microsoft.Data.Sqlite.SqliteConnection..cctor()
   --- End of inner exception stack trace ---
   at Microsoft.Data.Sqlite.SqliteConnection..ctor(String connectionString)
   at SymbolCache.PersistentCacheImpl.initializeCache(String dir) in /home/runner/work/FsAutoComplete/FsAutoComplete/src/FsAutoComplete.Core/SymbolCache.fs:line 108
   at SymbolCache.PersistentCacheImpl.lazyConn@136.Invoke() in /home/runner/work/FsAutoComplete/FsAutoComplete/src/FsAutoComplete.Core/SymbolCache.fs:line 136
   at System.Lazy`1.ViaFactory(LazyThreadSafetyMode mode)
--- End of stack trace from previous location ---
   at System.Lazy`1.CreateValue()
   at SymbolCache.getSymbols@170-2.Invoke(Unit unitVar) in /home/runner/work/FsAutoComplete/FsAutoComplete/src/FsAutoComplete.Core/SymbolCache.fs:line 172
   at Microsoft.FSharp.Control.AsyncPrimitives.CallThenInvoke[T,TResult](AsyncActivation`1 ctxt, TResult result1, FSharpFunc`2 part2) in D:\a\_work\1\s\src\fsharp\FSharp.Core\async.fs:line 447
   at Microsoft.FSharp.Control.Trampoline.Execute(FSharpFunc`2 firstAction) in D:\a\_work\1\s\src\fsharp\FSharp.Core\async.fs:line 104)
  Code: -32603

Machine info

OS: Mac with M1
.NET SDK version: 6.0.201
Ionide version: 6.0.3

Additional context
Add any other context about the problem here.

Thanks for reporting this issue (and the associated one in FSAC). We may need to investigate an alternative implementation for our symbol cache, or look at removing the concept now that the compiler services have some additional APIs for us to use for the areas we introduced the symbol cache for.

Also as a workaround, you can set "FSharp.enableBackgroundServices": false in your workspace settings to disable the background services, which is our only use of the sqlite library that's erroring.

@baronfel Disabling the background services just hides the error. It still doesn't work properly with find all references, so I'm guessing this is a different bug somewhere not related to symbol cache?

Finding references should be fixed in 7.0.0


overnigth tests are failing: 'CheckForbiddenElement' object has no attribute 'node_validate'

See original issue on GitLab
In GitLab by [Gitlab user @jjardon] on Jul 24, 2019, 10:35
Job #257705458 failed for 9656b2cb93cabf8b363f2d0222ba9747af28aa22
This happened after https://gitlab.com/BuildStream/buildstream/merge_requests/1472 got merged
Work to fix bst-plugins-experimental here:https://gitlab.com/BuildStream/bst-plugins-experimental/merge_requests/15

In GitLab by [Gitlab user @BenjaminSchubert] on Jul 24, 2019, 10:35
https://gitlab.com/freedesktop-sdk/freedesktop-sdk/merge_requests/1560 for the fix

In GitLab by [Gitlab user @BenjaminSchubert] on Jul 24, 2019, 10:35
https://gitlab.com/freedesktop-sdk/freedesktop-sdk/merge_requests/1560 for the fix

In GitLab by [Gitlab user @jjardon] on Jul 24, 2019, 10:58
mentioned in merge request !1498

In GitLab by [Gitlab user @jjardon] on Jul 24, 2019, 10:58
mentioned in merge request !1498

In GitLab by [Gitlab user @jjardon] on Jul 24, 2019, 11:30
mentioned in commit d5252efeaa37ac00c5019dfca79e9235e6e773d8

In GitLab by [Gitlab user @jjardon] on Jul 24, 2019, 11:30
mentioned in commit d5252efeaa37ac00c5019dfca79e9235e6e773d8

In GitLab by [Gitlab user @jjardon] on Jul 29, 2019, 11:27
https://gitlab.com/freedesktop-sdk/freedesktop-sdk/merge_requests/1560 fixed the internal fdsdk plugins (thanks [Gitlab user @BenjaminSchubert] !), but unfortunatelly the plugins from the bst-plugins-external repo are still broken (from https://gitlab.com/BuildStream/buildstream/-/jobs/260852453):
AttributeError: 'GitTagSource' object has no attribute 'node_get_member'

In GitLab by [Gitlab user @jjardon] on Jul 29, 2019, 11:27
https://gitlab.com/freedesktop-sdk/freedesktop-sdk/merge_requests/1560 fixed the internal fdsdk plugins (thanks [Gitlab user @BenjaminSchubert] !), but unfortunatelly the plugins from the bst-plugins-external repo are still broken (from https://gitlab.com/BuildStream/buildstream/-/jobs/260852453):
AttributeError: 'GitTagSource' object has no attribute 'node_get_member'

In GitLab by [Gitlab user @BenjaminSchubert] on Jul 29, 2019, 11:31
https://gitlab.com/BuildStream/bst-plugins-experimental/tree/bschubert/fix-plugins-yaml is a first attempt at fixing this, but the CI is also broken.
If someone wants to take over the fix, please do so and ping me :)

In GitLab by [Gitlab user @BenjaminSchubert] on Jul 29, 2019, 11:31
https://gitlab.com/BuildStream/bst-plugins-experimental/tree/bschubert/fix-plugins-yaml is a first attempt at fixing this, but the CI is also broken.
If someone wants to take over the fix, please do so and ping me :)

In GitLab by [Gitlab user @jjardon] on Aug 8, 2019, 09:15
mentioned in merge request bst-plugins-experimental!15

In GitLab by [Gitlab user @jjardon] on Aug 8, 2019, 09:15
mentioned in merge request bst-plugins-experimental!15

In GitLab by [Gitlab user @jjardon] on Aug 8, 2019, 11:44
changed the description

In GitLab by [Gitlab user @jjardon] on Aug 8, 2019, 11:44
changed the description

In GitLab by [Gitlab user @jjardon] on Aug 9, 2019, 09:56
changed the description

In GitLab by [Gitlab user @jjardon] on Aug 9, 2019, 09:56
changed the description

In GitLab by [Gitlab user @BenjaminSchubert] on Oct 8, 2019, 15:15
mentioned in merge request !1631

In GitLab by [Gitlab user @BenjaminSchubert] on Oct 8, 2019, 15:15
mentioned in merge request !1631

In GitLab by [Gitlab user @jjardon] on Oct 9, 2019, 09:47
closed via merge request !1631

In GitLab by [Gitlab user @jjardon] on Oct 9, 2019, 09:47
closed via merge request !1631

In GitLab by [Gitlab user @jjardon] on Oct 9, 2019, 09:47
mentioned in commit 4c0bb3404f5d760d4825b886b18fc8b5b052b20e

In GitLab by [Gitlab user @jjardon] on Oct 9, 2019, 09:47
mentioned in commit 4c0bb3404f5d760d4825b886b18fc8b5b052b20e


OpenSearch indexing script disconnects from server: ConnectionRefusedError

Describe the bug
I am encountering a ConnectionRefusedError while running an OpenSearch indexing script. Initially, the script worked without issues, indexing approximately 200 files, each containing around 30,000 documents.
However, now it fails with the following error:
Traceback (most recent call last):
  File "/usr/local/lib/python3.8/dist-packages/urllib3/connection.py", line 198, in _new_conn
    sock = connection.create_connection(
  File "/usr/local/lib/python3.8/dist-packages/urllib3/util/connection.py", line 85, in create_connection
    raise err
  File "/usr/local/lib/python3.8/dist-packages/urllib3/util/connection.py", line 73, in create_connection
    sock.connect(sa)
ConnectionRefusedError: [Errno 111] Connection refused

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

Traceback (most recent call last):
  File "/usr/local/lib/python3.8/dist-packages/opensearchpy/connection/http_urllib3.py", line 264, in perform_request
    response = self.pool.urlopen(
  File "/usr/local/lib/python3.8/dist-packages/urllib3/connectionpool.py", line 847, in urlopen
    retries = retries.increment(
  File "/usr/local/lib/python3.8/dist-packages/urllib3/util/retry.py", line 445, in increment
    raise reraise(type(error), error, _stacktrace)
  File "/usr/local/lib/python3.8/dist-packages/urllib3/util/util.py", line 39, in reraise
    raise value
  File "/usr/local/lib/python3.8/dist-packages/urllib3/connectionpool.py", line 793, in urlopen
    response = self._make_request(
  File "/usr/local/lib/python3.8/dist-packages/urllib3/connectionpool.py", line 491, in _make_request
    raise new_e
  File "/usr/local/lib/python3.8/dist-packages/urllib3/connectionpool.py", line 467, in _make_request
    self._validate_conn(conn)
  File "/usr/local/lib/python3.8/dist-packages/urllib3/connectionpool.py", line 1099, in _validate_conn
    conn.connect()
  File "/usr/local/lib/python3.8/dist-packages/urllib3/connection.py", line 616, in connect
    self.sock = sock = self._new_conn()
  File "/usr/local/lib/python3.8/dist-packages/urllib3/connection.py", line 213, in _new_conn
    raise NewConnectionError(
urllib3.exceptions.NewConnectionError: <urllib3.connection.HTTPSConnection object at 0x7fb798541430>: Failed to establish a new connection: [Errno 111] Connection refused

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "indexing.py", line 162, in <module>
    create_index(index_name, path, model, batch_size=100)
  File "indexing.py", line 139, in create_index
    client.bulk(body=batch, refresh=True)
  File "/usr/local/lib/python3.8/dist-packages/opensearchpy/client/utils.py", line 181, in _wrapped
    return func(*args, params=params, headers=headers, **kwargs)
  File "/usr/local/lib/python3.8/dist-packages/opensearchpy/client/__init__.py", line 462, in bulk
    return self.transport.perform_request(
  File "/usr/local/lib/python3.8/dist-packages/opensearchpy/transport.py", line 446, in perform_request
    raise e
  File "/usr/local/lib/python3.8/dist-packages/opensearchpy/transport.py", line 409, in perform_request
    status, headers_response, data = connection.perform_request(
  File "/usr/local/lib/python3.8/dist-packages/opensearchpy/connection/http_urllib3.py", line 279, in perform_request
    raise ConnectionError("N/A", str(e), e)
opensearchpy.exceptions.ConnectionError: ConnectionError(<urllib3.connection.HTTPSConnection object at 0x7fb798541430>: Failed to establish a new connection: [Errno 111] Connection refused) caused by: NewConnectionError(<urllib3.connection.HTTPSConnection object at 0x7fb798541430>: Failed to establish a new connection: [Errno 111] Connection refused)

When encountering this error, I restart OpenSearch and resume indexing from the exact file where the process stopped. However, upon resuming, the same error occurs again.
Related component
Clients
To Reproduce
The script reads JSON files from a directory and indexes them into an OpenSearch cluster. It uses the opensearchpy library to interact with OpenSearch and the sentence-transformers library for document embeddings. This is the code:
import pandas as pd
import os
import json
import torch
import time
from opensearchpy import OpenSearch
from sentence_transformers import SentenceTransformer

path = "../outputNJSONextracted"  # Directory containing your JSON files
model_card = 'sentence-transformers/msmarco-distilbert-base-tas-b'

device = torch.device("cuda" if torch.cuda.is_available() else "cpu")
print(f"Device {device}")

host = '<IP_ADDRESS>' #host = '<IP_ADDRESS>'
port = 9200
auth = ('admin','IVIngi2024!') #('admin', 'admin')

client = OpenSearch(
    hosts = [{'host': host, 'port': port}],
    http_auth = auth,
    use_ssl = True,
    verify_certs = False,
    ssl_assert_hostname = False,
    ssl_show_warn = False,
    timeout=30,
    max_retries=10
)
print("Connection opened...")

index_name = 'medline-faiss-hnsw-3' # change the index name

index_body = {
 "settings": {
    "index": {
     "knn": "true",
     "refresh_interval" : -1,
     #default_pipeline": "medline-ingest-pipeline",  # embedding in script
     "number_of_shards": 5,
     "number_of_replicas": 0
    }
  },
  "mappings": {
    "properties": {
         "embedding_abstract": {
                        "type": "knn_vector",
                        "dimension": 768,
                        "method":{
                            "engine":"faiss",
                            "name": "hnsw",
                            "space_type": "innerproduct"
                        }
                       },
          "title":{"type":"text"},
          "abstract":{"type":"text"},
          "pmid":{"type":"keyword"},
          "journal":{"type":"text"},
          "pubdate":{"type":"date"},
          "authors":{"type": "text"}
    }
  }
}

response = client.indices.create(index_name, body=index_body)
print(response)

def create_index(index_name, directory_path, model, batch_size=100):
    j = 0
    documents = set()

    files_number = 0
    for filename in sorted(os.listdir(directory_path)):
        start_time = time.time()
        if filename.endswith(".json"):
            print(f"Starting indexing {filename} ...")
            # Construct the full file path
            file_path = os.path.join(directory_path, filename)
            # Read the JSON file
            with open(file_path, 'r') as file:
                # Initialize an empty list to store dictionaries
                dictionaries = []

                # Read the file line by line
                for line in file:
                    # Parse each line as JSON and append it to the list
                    dictionaries.append(json.loads(line))
            # Create a DataFrame
            df = pd.DataFrame(dictionaries)
            # Select only the required columns
            df = df[['pmid', 'title', 'abstract', 'journal', 'authors', 'pubdate']]
            # Output the file name
            batch = []
            for i, row in df.iterrows():
                pmid = row["pmid"]
                if pmid in documents:
                    continue
                else:
                    documents.add(pmid)
                    embedding = model.encode(row["abstract"])
                    doc = {
                        "pmid": pmid,
                        "abstract": row["abstract"],
                        "title": row["title"],
                        "authors": row['authors'],
                        "journal": row['journal'],
                        "pubdate": row['pubdate'],
                        "embedding_abstract": embedding
                        }
                    batch.append({"index": {"_index": index_name, "_id": pmid}})
                    batch.append(doc)
                    j += 1
                if len(batch) >= batch_size*2:
                    client.bulk(body=batch, refresh=True)
                    batch = []

            if batch:
                client.bulk(body=batch, refresh=True)
                print(f"Indexed remaining documents")

            files_number += 1
            print(f"Processed file: {filename} in {time.time()-start_time}")
            print("Number of currently documents indexed ",j)
            if files_number % 100 == 0:
                print("-"*50)
                print(f"Files indexed = {files_number}")
                print()

    print("Total documents inserted = ", j)

model = SentenceTransformer(model_card)
model.to(device)
print("Creating indexing...")
start = time.time()
create_index(index_name, path, model, batch_size=100)
print(f"Time neeeded {time.time() - start}")

Troubleshooting Steps Taken:
Increased heap size in jvm.options file to 8GB (the max that I can).
Attempted to reduce batch size to mitigate potential HTTP request size issues.
Despite these efforts, the script continues to encounter connection issues.
I suspect that the problem may be related to sending large HTTP requests or some configuration issue with the OpenSearch server. However, I'm unsure how to proceed in diagnosing and resolving the issue.
Any insights or suggestions would be greatly appreciated. Thank you!
Expected behavior
namal Connection
Additional Details
Plugins
Please list all plugins currently enabled.
Screenshots
If applicable, add screenshots to help explain your problem.
Host/Environment (please complete the following information):

OS: [e.g. iOS]
Version [e.g. 22]

Additional context
Add any other context about the problem here.

@lijie123bes Let's narrow this down to a client vs. server problem.
Do you have errors in the server-side log, meaning is the server actually refusing connections because of a circuit breaker or something like this?


flecs needs an explicit usage file

Suggesting target_link_libraries(main PRIVATE flecs::flecs flecs::flecs_static) only works in vcpkg. Outside vcpkg either one target will be missing, or the user will link two libs providing the same symbols but with different linkage.
What needs to be done?
Add a usage file which can work without the wrapper, i.e. using
    target_link_libraries(main PRIVATE $<IF:$<TARGET_EXISTS:flecs::flecs>,flecs::flecs,flecs::flecs_static>)

Originally posted by @dg0yt in https://github.com/microsoft/vcpkg/issues/31432#issuecomment-1547627257

This issue is not unique to flecs.
There are many C++ libraries, such as zeromq and xeus, that have CMake support for building both dynamic and static libraries at the same time. In such cases, the library maintainers differentiate between the targets for dynamic and static libraries (usually including "static" in the target name for the static library).
Based on my vague recollection, in the past few years, when encountering such libraries, vcpkg recommended adding a vcpkg-cmake-wrapper.cmake to link the static target and dynamic target together.
I remember that this approach was mentioned either in the vcpkg documentation or in responses from core members of the vcpkg team in related issues. However, I am currently unable to locate the exact source.

OK. I think I find it.
https://github.com/microsoft/vcpkg/issues/6045#issuecomment-482387298

@myd7349 This issue is about a particular bad output from vcpkg tool's usage heuristics which can be fixed by a usage file. My post ist clear about it.
(Wrapper are only loosely related to the issue. I wouldn't add one for this port. But I didn't ask to remove it, and I protested against breaking users by removing it.)


Swagger model API version is incorrect

Reports 0.1 but should be 1.x

Fixed in https://github.com/IBM/MAX-Image-Resolution-Enhancer/commit/fc0fad3fe0d5666b6def551d3e4f680449d87a93
Thanks, @ptitzler !


[HUDI-2284] Move HoodieDeltaStreamerException to hudi-utilities

DELETE HoodieDeltaStreamerException
Tips

Thank you very much for contributing to Apache Hudi.
Please review https://hudi.apache.org/contributing.html before opening a pull request.

What is the purpose of the pull request
HoodieDeltaStreamerException should exist in hudi-utilities
Committer checklist

[ ] Has a corresponding JIRA in PR title & commit

[ ] Commit message is descriptive of the change

[ ] CI is green

[ ] Necessary doc changes done or have another open PR

[ ] For large changes, please consider breaking it into sub-tasks under an umbrella JIRA.

reopen to trigger ci


cannot enable HIRB: compatibility issue?

irb(main):002:0> Hirb.enable
TypeError: can't convert false into Hash
from /usr/local/lib/ruby/gems/1.9.1/gems/hirb-0.5.0/lib/hirb/util.rb:22:in merge' 	from /usr/local/lib/ruby/gems/1.9.1/gems/hirb-0.5.0/lib/hirb/util.rb:22:in recursive_hash_merge'
from /usr/local/lib/ruby/gems/1.9.1/gems/hirb-0.5.0/lib/hirb.rb:75:in block in config' 	from /usr/local/lib/ruby/gems/1.9.1/gems/hirb-0.5.0/lib/hirb.rb:74:in each'
from /usr/local/lib/ruby/gems/1.9.1/gems/hirb-0.5.0/lib/hirb.rb:74:in inject' 	from /usr/local/lib/ruby/gems/1.9.1/gems/hirb-0.5.0/lib/hirb.rb:74:in config'
from /usr/local/lib/ruby/gems/1.9.1/gems/hirb-0.5.0/lib/hirb/view.rb:265:in default_config' 	from /usr/local/lib/ruby/gems/1.9.1/gems/hirb-0.5.0/lib/hirb/view.rb:249:in load_config'
from /usr/local/lib/ruby/gems/1.9.1/gems/hirb-0.5.0/lib/hirb/view.rb:244:in merge_or_load_config' 	from /usr/local/lib/ruby/gems/1.9.1/gems/hirb-0.5.0/lib/hirb/view.rb:84:in enable'
from /usr/local/lib/ruby/gems/1.9.1/gems/hirb-0.5.0/lib/hirb.rb:38:in enable' 	from (irb):2:in irb_binding'
from /usr/local/lib/ruby/1.9.1/irb/workspace.rb:80:in eval' 	from /usr/local/lib/ruby/1.9.1/irb/workspace.rb:80:in evaluate'
from /usr/local/lib/ruby/1.9.1/irb/context.rb:254:in evaluate' 	from /usr/local/lib/ruby/1.9.1/irb.rb:159:in block (2 levels) in eval_input'
from /usr/local/lib/ruby/1.9.1/irb.rb:273:in signal_status' 	from /usr/local/lib/ruby/1.9.1/irb.rb:156:in block in eval_input'
from /usr/local/lib/ruby/1.9.1/irb/ruby-lex.rb:243:in block (2 levels) in each_top_level_statement' 	from /usr/local/lib/ruby/1.9.1/irb/ruby-lex.rb:229:in loop'
from /usr/local/lib/ruby/1.9.1/irb/ruby-lex.rb:229:in block in each_top_level_statement' 	from /usr/local/lib/ruby/1.9.1/irb/ruby-lex.rb:228:in catch'
from /usr/local/lib/ruby/1.9.1/irb/ruby-lex.rb:228:in each_top_level_statement' 	from /usr/local/lib/ruby/1.9.1/irb.rb:155:in eval_input'
from /usr/local/lib/ruby/1.9.1/irb.rb:70:in block in start' 	from /usr/local/lib/ruby/1.9.1/irb.rb:69:in catch'
from /usr/local/lib/ruby/1.9.1/irb.rb:69:in start' 	from /usr/local/lib/ruby/gems/1.9.1/gems/railties-3.1.0.rc5/lib/rails/commands/console.rb:45:in start'
from /usr/local/lib/ruby/gems/1.9.1/gems/railties-3.1.0.rc5/lib/rails/commands/console.rb:8:in start' 	from /usr/local/lib/ruby/gems/1.9.1/gems/railties-3.1.0.rc5/lib/rails/commands.rb:40:in <top (required)>'
from script/rails:6:in require' 	from script/rails:6:in 'Maybe IRB bug!!

Since I didn't get more detail I'm closing. Please reopen with above info

i have downloaded hirb for my rails 3.1 app and also mentioned it in my gemfile. when i go to rails console and say require 'hirb' it returns false. i have run bundle update as well.


Add regression tests for student finance calculator

We want tests that give us confidence to make some larger changes to the existing Smart Answers (e.g. switching to ERB templates).
The current integration tests aren't good enough as they don't exercise the flows completely, and they rely on using the internals of the flows (e.g. tests that assert things about the phrase lists).
These regression tests aim to comprehensively test the Smart Answers and only rely on the generated html, which should allow us to change the implementation without worrying about breaking the tests.
Testing Smart Answers in this way involves:

Creating a questions-and-responses file that contains a set of responses that should be used for each question (see scripts/generate-questions-and-responses-for-smart-answer.rb).

Creating a responses-and-expected-results file that contains each combination of results from the questions-and-responses file, along with the node name and whether it's an outcome node (see scripts/generate-responses-and-expected-results-for-smart-answer.rb).

Running the smart_answers_regression_test to test the smart answer using each combination of responses from the responses-and-expected-results file.

Running the smart_answers_regression_test in this branch takes around 100 seconds, which isn't an insignificant amount of time. I have some ways of optimising the tests so that they only run when changes to the flows or input responses mean that the saved outcomes might be different. I didn't want to add them to this pull request as I'm more interested in what people about the approach in general.
We don't imagine these tests being very long lived. We'd hope to use them while we're making changes to the Smart Answers and for them to be replaced with more focussed tests over time.
Any questions and/or thoughts about this approach?

This will help immensely when refactoring smartanswers. Great commits with descriptive messages too :trophy:
The generated HTML files are quite big, if you suggest committing them the smartanswers repo size will grow significantly. It will also pollute commit messages, because most of the changes we normally do change outcomes/flows legitimately. Did you commit those just for the demo purposes? If so then ignore this paragraph :)
Other than that it looks like a brilliant tool that will encourage me to refactor hairy parts in smartanswers :) In the past I've written brittle ad-hoc scripts to crawl the URLs I expected to not change to achieve what this PR does.
Also, it will need a nice writeup in README :+1:

The generated HTML files are quite big, if you suggest committing them the smartanswers repo size will grow significantly. It will also pollute commit messages, because most of the changes we normally do change outcomes/flows legitimately. Did you commit those just for the demo purposes? If so then ignore this paragraph :)

I was imagining that we would commit the HTML files to the repo. I think this is the only way we'll be able to have tests that fail when something's changed. I don't feel entirely comfortable about this approach and went back and forth between having a tool that runs outside of tests versus one that's part of the build process.
We might be able to avoid the commits becoming polluted by updating the YAML and HTML files in a separate commit on the branch that makes a change to the flow/s. This isn't ideal because it means we have a commit with failing tests, but it might be a suitable compromise at this point in time?
In the longer term, I'm hoping that we'll be able to reduce these sort of tests and therefore remove a lot of the HTML from the repo.
Do my comments address your concern about storing the HTML files in the repo, or do you think we should revisit that part of the idea?

@tadas:
I'm not sure it's obvious from Chris' description, but our plan in the short term is to replace the existing integration tests with tests like these. For the moment, Chris' tool will only replace the assert_phrase_list aspect of the tests, but we envisage extending it to cover asserting against the question nodes too. Hopefully knowing that these tests are intended to replace the existing integration tests makes it more obvious why we're thinking of committing the outcome HTML to the repo.
Do you have a specific concern about the repo size getting bigger?
I can see how including the outcome HTML in commits might make them more confusing, but I think it should be manageable.

Ok, I somehow missed the intention to completely replace integration tests :) Then my concerns are:

how would one test-drive changes to the flow? By modifying the yaml files?
test descriptions are often a good medium to write up exceptional behaviour and reasons why the behaviour is the way it is, how would we do it with the new approach?
what we do most of the time is change phrases/flows in smart answers. I'm afraid this would cause a lot more failures. i.e. for a smartanswer with a countries drop down list a single change could result in 200+ failures. I'm afraid whoever is responsible for reviewing them and making sure the failures are caused by intended changes would just blindly accept the diff or spend hours verifying the failures are legit. Test driving a change like this would require some serious automation as well I imagine.

All in all I understand how this can be useful when refactoring without any externally visible changes, but I have doubts this would be useful for day-to-day changes Ben and I usually make. I would be curious to see if my concerns are relevant in practice, but the thing I'm working on (and will be for the next couple of weeks) is not suitable for trying it out. Would it be possible to have it as an optional tool first, see how useful it is in practice and then if we think it's good, replace flow integration tests with it.

In the longer term, I'm hoping that we'll be able to reduce these sort of tests and therefore remove a lot of the HTML from the repo.

How would we do that?

Do you have a specific concern about the repo size getting bigger?

Cloning the repo would take longer, editors would need specific setup to exclude HTML dirs from global search paths to make search faster etc.

I've tried to answer your questions below. Doing so has made us realise that @floehopper and I haven't done a particular good job of keeping you informed about how our thinking has changed over time. Can we get together tomorrow (Weds) and chat through this in person, @tadast and @BenJanecke?

how would one test-drive changes to the flow? By modifying the yaml files?

Modifying the YAML files should be possible, yes. But I can also imagine being in a place where we don't test-drive changes to the flow. If, for example, we end up with some kind of interface to allow non-developers to create/edit flows then I don't think they'd be test-driven. Instead, I think we'd want tools that give us information about the flows to ensure that they're not "broken" (for some definition of "broken") and to allow us to visualise them in some way to ensure they're working the way we expect.

test descriptions are often a good medium to write up exceptional behaviour and reasons why the behaviour is the way it is, how would we do it with the new approach?

I'd like to see us move to having the logic of the policies encapsulated in well tested objects that better describe the sort of behaviour you're talking about. At the moment, we have lots of Smart Answers that mix both the flow and policy logic, making it difficult to test one without the other. Splitting flow and policy logic should allow us to retain descriptive tests without having to have those tests exercise the flow itself.

what we do most of the time is change phrases/flows in smart answers. I'm afraid this would cause a lot more failures. i.e. for a smartanswer with a countries drop down list a single change could result in 200+ failures. I'm afraid whoever is responsible for reviewing them and making sure the failures are caused by intended changes would just blindly accept the diff or spend hours verifying the failures are legit. Test driving a change like this would require some serious automation as well I imagine.

This is related to question 1 in that I hope some new tools can help. I'd hope that rather than diff'ing the HTML, it'd be possible to at least diff the YAML (an intermediate representation of the flow) and ideally diff a visual representation of the flow to ensure the changes have the effect you want.

Would it be possible to have it as an optional tool first, see how useful it is in practice and then if we think it's good, replace flow integration tests with it.

I certainly think this be possible. Even in its current form, we can selectively add these tests for the Smart Answers that we convert to ERB templates, i.e. hopefully not disrupt your work.

In the longer term, I'm hoping that we'll be able to reduce these sort of tests and therefore remove a lot of the HTML from the repo.

How would we do that?

I think I've covered this in my answers to the previous questions. Hopefully, though, a combination of well-tested smaller objects describing the policy logic and flow logic would make the flows themselves much easier to test, either by writing actual tests or by using tools that verify they're behaving as expected.

Cloning the repo would take longer, editors would need specific setup to exclude HTML dirs from global search paths to make search faster etc.

Certainly valid concerns.

Thanks for the answer, it's starting to make more sense now! :) Let's talk tomorrow

@floehopper, @tadast, @BenJanecke and I discussed this in some detail today.
We've agreed to merge this to master and to continue to review it as we start to add more of these regression tests. We're certain that the increase in test duration is going to cause us pain and we'll almost certainly tackle that soon. We think that the very tests themselves might cause us some pain but we've agreed to tackle that as and when it arises.


Problem saving relations

I am trying to save relations between
https://tibschol-test.acdh-ch-dev.oeaw.ac.at/apis/apis_ontology.instance/6957
https://tibschol-test.acdh-ch-dev.oeaw.ac.at/apis/entities/entity/instance/4572/
https://tibschol-test.acdh-ch-dev.oeaw.ac.at/apis/apis_ontology.work/4573
but cannot save them

actually, I cannot save any new relations at the moment.


flutter error

Analyzer Feedback from IntelliJ
Version information

IDEA AI-191.8<IP_ADDRESS>77832
2.8.4
AI-191.8<IP_ADDRESS>77832, JRE 1.8.0_202-release-1483-b03x64 JetBrains s.r.o, OS Windows 10(amd64) v10.0 , screens 1366x768

Exception
Dart analysis server, SDK version 2.8.4, server version 1.27.4, FATAL error: UnimplementedError: name is an instance of _OperatorName in endClassConstructor
#0      Driver._captureExceptions.errorFunction (package:analysis_server/src/server/driver.dart:720:7)
#1      _CustomZone.handleUncaughtError (dart:async/zone.dart:1059:19)
#2      _CustomZone.runGuarded (dart:async/zone.dart:981:7)
#3      _CustomZone.bindCallbackGuarded.<anonymous closure> (dart:async/zone.dart:1019:23)
#4      _microtaskLoop (dart:async/schedule_microtask.dart:43:21)
#5      _startMicrotaskLoop (dart:async/schedule_microtask.dart:52:5)
#6      _runPendingImmediateCallback (dart:isolate-patch/isolate_patch.dart:118:13)
#7      _Timer._runTimers (dart:isolate-patch/timer_impl.dart:405:11)
#8      _Timer._handleMessage (dart:isolate-patch/timer_impl.dart:429:5)
#9      _RawReceivePortImpl._handleMessage (dart:isolate-patch/isolate_patch.dart:168:12)

For additional log information, please append the contents of
file://C:\Users\fasil\AppData\Local\Temp\report.txt.

same here

This has not been reported again since Dart 2.8.4; please re-open if you see this in a recent version of Dart/Flutter.

This has not been reported again since Dart 2.8.4; please re-open if you see this in a recent version of Dart/Flutter.


Publish 03/11/2022 3:30 PM PT

Docs Build status updates of commit 7b7046c:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

exchange/exchange-ps/exchange/Set-OrganizationConfig.md
:white_check_mark:Succeeded
View (exchange-ps)

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


Package version compatibility?

I somehow can't get the packages aligned to make the latest version work on Android and Hololens.
Unity 2019.4.15f1c1
First of all following packages match:

ARSubsystems: 2.1.3
AR Foundation: 2.1.8
AR Core XR Plugin: 2.1.11
Windows XR Plugin: 2.4.1

However this doesn't match with WLT because AnchorManagerXR.cs requires ARSubsystems >= 3.0.1 because the renaming of ReferencePoints to Anchors:
<EMAIL_ADDRESS>So trying with following combination of packages:

ARSubsystems: 3.1.3
AR Foundation: 3.1.3
AR Core XR Plugin: 3.1.4
Windows XR Plugin: 2.4.1

But that leads to following errors:
Library\PackageCache\com.unity.xr.windowsmr@2.4.1\Runtime\Session\WindowsMRSessionSubsystem.cs(15,28): error CS0246: The type or namespace name 'IProvider' could not be found (are you missing a using directive or an assembly reference?)
Library\PackageCache\com.unity.xr.windowsmr@2.4.1\Runtime\Session\WindowsMRSessionSubsystem.cs(15,38): error CS0508: 'WindowsMRSessionSubsystem.CreateProvider()': return type must be 'XRSessionSubsystem.Provider' to match overridden member 'XRSessionSubsystem.CreateProvider()'
Library\PackageCache\com.unity.xr.windowsmr@2.4.1\Runtime\Session\WindowsMRSessionSubsystem.cs(20,15): warning CS0108: 'WindowsMRSessionSubsystem.Provider' hides inherited member 'XRSessionSubsystem.Provider'. Use the new keyword if hiding was intended.
Library\PackageCache\com.unity.xr.windowsmr@2.4.1\Runtime\ReferencePoint\WindowsMRReferencePointSubsystem.cs(118,28): error CS0246: The type or namespace name 'IProvider' could not be found (are you missing a using directive or an assembly reference?)
Library\PackageCache\com.unity.xr.windowsmr@2.4.1\Runtime\Session\WindowsMRSessionSubsystem.cs(20,26): error CS0246: The type or namespace name 'IProvider' could not be found (are you missing a using directive or an assembly reference?)
Library\PackageCache\com.unity.xr.windowsmr@2.4.1\Runtime\ReferencePoint\WindowsMRReferencePointSubsystem.cs(118,38): error CS0508: 'WindowsMRReferencePointSubsystem.CreateProvider()': return type must be 'XRAnchorSubsystem.Provider' to match overridden member 'XRAnchorSubsystem.CreateProvider()'
Library\PackageCache\com.unity.xr.windowsmr@2.4.1\Runtime\ReferencePoint\WindowsMRReferencePointSubsystem.cs(123,15): warning CS0108: 'WindowsMRReferencePointSubsystem.Provider' hides inherited member 'XRAnchorSubsystem.Provider'. Use the new keyword if hiding was intended.
Library\PackageCache\com.unity.xr.windowsmr@2.4.1\Runtime\Session\WindowsMRSessionSubsystem.cs(27,58): error CS0115: 'WindowsMRSessionSubsystem.Provider.GetAvailabilityAsync()': no suitable method found to override
Library\PackageCache\com.unity.xr.windowsmr@2.4.1\Runtime\Session\WindowsMRSessionSubsystem.cs(34,43): error CS0115: 'WindowsMRSessionSubsystem.Provider.trackingState': no suitable method found to override
Library\PackageCache\com.unity.xr.windowsmr@2.4.1\Runtime\Session\WindowsMRSessionSubsystem.cs(42,47): error CS0115: 'WindowsMRSessionSubsystem.Provider.notTrackingReason': no suitable method found to override
Library\PackageCache\com.unity.xr.windowsmr@2.4.1\Runtime\Session\WindowsMRSessionSubsystem.cs(50,34): error CS0115: 'WindowsMRSessionSubsystem.Provider.Destroy()': no suitable method found to override
Library\PackageCache\com.unity.xr.windowsmr@2.4.1\Runtime\Session\WindowsMRSessionSubsystem.cs(54,34): error CS0115: 'WindowsMRSessionSubsystem.Provider.Pause()': no suitable method found to override
Library\PackageCache\com.unity.xr.windowsmr@2.4.1\Runtime\Session\WindowsMRSessionSubsystem.cs(59,34): error CS0115: 'WindowsMRSessionSubsystem.Provider.Resume()': no suitable method found to override
Library\PackageCache\com.unity.xr.windowsmr@2.4.1\Runtime\Session\WindowsMRSessionSubsystem.cs(64,34): error CS0115: 'WindowsMRSessionSubsystem.Provider.Reset()': no suitable method found to override
Library\PackageCache\com.unity.xr.windowsmr@2.4.1\Runtime\ReferencePoint\WindowsMRReferencePointSubsystem.cs(123,26): error CS0246: The type or namespace name 'IProvider' could not be found (are you missing a using directive or an assembly reference?)
Library\PackageCache\com.unity.xr.windowsmr@2.4.1\Runtime\ReferencePoint\WindowsMRReferencePointSubsystem.cs(125,34): error CS0115: 'WindowsMRReferencePointSubsystem.Provider.Start()': no suitable method found to override
Library\PackageCache\com.unity.xr.windowsmr@2.4.1\Runtime\ReferencePoint\WindowsMRReferencePointSubsystem.cs(130,34): error CS0115: 'WindowsMRReferencePointSubsystem.Provider.Stop()': no suitable method found to override
Library\PackageCache\com.unity.xr.windowsmr@2.4.1\Runtime\ReferencePoint\WindowsMRReferencePointSubsystem.cs(135,34): error CS0115: 'WindowsMRReferencePointSubsystem.Provider.Destroy()': no suitable method found to override
Library\PackageCache\com.unity.xr.windowsmr@2.4.1\Runtime\ReferencePoint\WindowsMRReferencePointSubsystem.cs(140,63): error CS0115: 'WindowsMRReferencePointSubsystem.Provider.GetChanges(XRAnchor, Allocator)': no suitable method found to override
Library\PackageCache\com.unity.xr.windowsmr@2.4.1\Runtime\ReferencePoint\WindowsMRReferencePointSubsystem.cs(181,34): error CS0115: 'WindowsMRReferencePointSubsystem.Provider.TryAddReferencePoint(Pose, out XRAnchor)': no suitable method found to override
Library\PackageCache\com.unity.xr.windowsmr@2.4.1\Runtime\ReferencePoint\WindowsMRReferencePointSubsystem.cs(188,34): error CS0115: 'WindowsMRReferencePointSubsystem.Provider.TryRemoveReferencePoint(TrackableId)': no suitable method found to override

It looks Windows XR Plugin 2.4.1 doesn't match with ARSubsystems 3.1.3, but there's no higher version.
Am I missing something?
Which combination of packages did you use succesfully?

As mentioned here:
https://github.com/microsoft/MixedReality-WorldLockingTools-Unity/blob/master/DocGen/Documentation/HowTos/XRSDK.md
It should be AR Subsystems 2.x.x
I was probably testing with a wrong version of WLT.
Anyway it is working now.

That's a great point, I have found getting the right versions of the supporting plugins to be confusing. I will clarify that in the documentation. Thanks!

@zantiu , I saw your previous reply that ASA work with XR Plugin and AR Subsystems 2.x.x
But, once I import the ASA SDK 2.7.0 in unity through manifest file the AR Subsystem updated to 3.1.3 version and i am not able to change the version, and i am getting the same error which you have mentioned in first comment
Could you please guide me how to come out of this Errors☺

@zantiu , I saw your previous reply that ASA work with XR Plugin and AR Subsystems 2.x.x
But, once I import the ASA SDK 2.7.0 in unity through manifest file the AR Subsystem updated to 3.1.3 version and i am not able to change the version, and i am getting the same error which you have mentioned in first comment
Could you please guide me how to come out of this Errors☺

Sorry where did you see that? I'm currently not testing with ASA.

Sorry where did you see that? I'm currently not testing with ASA.

@zantiu Very much thanks for the reply, My bad i thought you were working with Azure spatial anchors, Since i also got the errors☺

@zantiu Very much thanks for the reply, My bad i thought you were working with Azure spatial anchors, Since i also got the errors☺


Timeout in Requests

Hello,
I have an unsteady internet connection, and I had to add timeout=(3,10) to line 170 to get it to kick up and not get paralysed.
If you read https://realpython.com/python-requests/#timeouts, you will say they say :
"When you make an inline request to an external service, your system will need to wait upon the response before moving on. If your application waits too long for that response, requests to your service could back up, your user experience could suffer, or your background jobs could hang.
By default, requests will wait indefinitely on the response, so you should almost always specify a timeout duration to prevent these things from happening. To set the request’s timeout, use the timeout parameter. timeout can be an integer or float representing the number of seconds to wait on a response before timing out:"
Please make this standard, it will save a lot of time to others.

Happy to merge a PR


Add "involves fire button" to submission page

FAST would like a record of all proposals that involve fire.  We could add a checkbox to the submission page and then it'd be easy to flag those proposals for their review

The backend for this should be implemented such that we can add more attributes (tags) to proposals. And then we also want to be able to associate these tags with voters.
So we'll have a new table, tags, and then a table of tags_for_submissions.  Eventually we'll also associate tags with voters and then we can match up.  But for v0, just have the one tag for fire

in progress: https://github.com/FireflyArtsCollective/ffagc/pull/115


Document did:tz resolution options

The did:tz implementation uses some custom DID Resolution options (input metadata). The updates option is defined in the Tezos DID Method Specification. public_key and tzkt_url options are specific to this implementation. This PR adds some rustdocs for them.
Re: https://github.com/spruceid/ssi/pull/311#discussion_r755431182
Preview: https://demo.didkit.dev/2021/11/23/did-tz-rustdoc/did_tz/struct.DIDTz.html#resolution-options

Put the same info in the did-tezos/readme.md

Rebasing and merging


Fix rate limiting issues with the web application

Based on an investigation of a case where the web app managed to hit server-side rate limits.
The individual commits detail the changes.

Pushed with a more systematic backfill implementation, which I've tested fairly carefully.

Merged, since the CI failure is a hang unrelated to this PR.

Backported for 8.x in https://github.com/zulip/zulip/pull/28964.


Bugfix/create if missing nouow

Remove onCommit hook

This pull request has been made obsolete by #1799.


[Ask] WebRTC support

Hi mate,
I'm planning to create some video streaming app in Android that uses USB Webcam.
I've tried using libs: https://github.com/saki4510t/UVCCamera but it doesn't support WebRTC streaming.
Quote:

Simple integration to 3rd party applications with multiple implementation options dependant on use case requirements.

Is it possible to stream video in a WebRTC app using this library?
Regards,
L

The UVC implementation is still incomplete at this point in time but it is close to working. Once implemented it should be possible to support WebRTC however that would likely not be part of this library directly as the only goal of this library is to provide raw access to the camera streams and simplify UI display of webcam output.

@ToxicBakery Thanks mate..
Maybe it's time to take a deep look into WebRTC SDK..

@jwoolston No prob mate..
Thanks for your consideration.
I still follow this repo 😃
Cheers.

Is the project dead? What would be the best substitute?
Meanwhile I opened a feature request to integrate a UVC library into react-native-webrtc which powers Jitsi Meet. Contributions are welcome.
https://github.com/jitsi/jitsi-meet/issues/4937

@vakrolme This project was more of an experiment than anything. I had hopes of making it more viable but the UVC usb class requires a lot of work, more than I am currently willing to dedicate, especially given that Android has mainstream support for external USB cameras now (even if the manufacturers aren't enabling it)

@vakrolme This project was more of an experiment than anything. I had hopes of making it more viable but the UVC usb class requires a lot of work, more than I am currently willing to dedicate, especially given that Android has mainstream support for external USB cameras now (even if the manufacturers aren't enabling it)

especially given that Android has mainstream support for external USB cameras now (even if the manufacturers aren't enabling it)

Any link of documentation to enable this?
I open a case into manufacturers to enable passing docs to enable it.


multiple lines and indentation when editing json

SUMMARY

Makes it more convenient
BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF

before:

after:

TEST PLAN

ADDITIONAL INFORMATION

[ ] Has associated issue:
[ ] Changes UI
[ ] Requires DB Migration.
[ ] Confirm DB Migration upgrade and downgrade tested.
[ ] Introduces new feature or API
[ ] Removes existing feature or API

Codecov Report

Merging #11501 into master will decrease coverage by 5.33%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master   #11501      +/-   ##
==========================================
- Coverage   66.61%   61.28%   -5.34%
==========================================
  Files         862      418     -444
  Lines       41212    26074   -15138
  Branches     3716        0    -3716
==========================================
- Hits        27455    15979   -11476
+ Misses      13657    10095    -3562
+ Partials      100        0     -100

Flag
Coverage Δ

#cypress
?

#javascript
?

#python
61.28% <ø> (-0.72%)
:arrow_down:

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

Impacted Files
Coverage Δ

superset/db_engines/hive.py
0.00% <0.00%> (-85.72%)
:arrow_down:

superset/db_engine_specs/hive.py
53.90% <0.00%> (-30.08%)
:arrow_down:

superset/db_engine_specs/presto.py
69.76% <0.00%> (-12.53%)
:arrow_down:

superset/db_engine_specs/mysql.py
79.59% <0.00%> (-12.25%)
:arrow_down:

superset/db_engine_specs/sqlite.py
65.62% <0.00%> (-9.38%)
:arrow_down:

superset/utils/celery.py
82.14% <0.00%> (-3.58%)
:arrow_down:

superset/examples/world_bank.py
97.10% <0.00%> (-2.90%)
:arrow_down:

superset/examples/birth_names.py
96.51% <0.00%> (-2.33%)
:arrow_down:

superset/views/database/mixins.py
80.70% <0.00%> (-1.76%)
:arrow_down:

superset/models/core.py
87.46% <0.00%> (-1.68%)
:arrow_down:

... and 457 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 68d493e...62fd395. Read the comment docs.

Jeez, I totally have done this before but the PR never got merged because of failing CI back then (we were using Travis and it was quite unstable).
I would recommend utilize the json-stringify-pretty-compact package as I did in my PR.  It would make things much easier to read.

I would argue those extra whitespaces are minuscule comparing to the whole payload and the overall db size, especially after gzip...

+1, spaces/return shouldn't have measurable impact. We could get fancy and compress, but I don't think it's worth it at this time. Usability over micro-optimization!

Oh, side note about having JSON-in-JSON in default_filters, we should fix that, but that's out of scope for this PR.

Agreed, that json-in-json thing has been bugging me. Won't be a problem in the new dashboard-native filters!


[Suggestion] Hide config button if unused

If a mod does not register a config task then the button should not be displayed for its entry in the list.

I did think about doing that but I figured it would be a bit more confusing if a button is missing than if it was just disabled with a tooltip. But I guess it's something to think about again.

I'd prefer the button to stay how it is; having it greyed out + tooltip makes it very clear to the user what's going on. Not having the button at all will just make users wonder what's broken instead of conveying a "nothing to configure here" message.

A disabled config button for a mod that doesn't need a config implies that is intended to have one but has no yet implemented it


Check titles for two topics that are now called 'Persistent identifier'

Two topics are now called 'Persistent identifier':

https://ubvu.github.io/open-handbook/topics/persistent-identifier.html; I think this one is correctly titled
https://ubvu.github.io/open-handbook/topics/data-publication.html; this one was originally (in LibGuide) titled 'Data publication'.

The page on data publication used to contain a section 'What is a persistent identifier?' and this section is no longer part of the page in the Handbook. Maybe something got mixed up there?
I think for now https://ubvu.github.io/open-handbook/topics/data-publication.html should get the title 'Data publication'.

Thanks @Jolien-S 🙌
Something is going wrong here indeed - the topic Persistent Identifier is being copied over into the Data Publication one, which causes a heading conflict (that is, two titles and quarto picks the last occurrence).
I will update to link out to Persistent Identifier, as for now we should not be using include statements in topics (only in guides).


[SRC] pom.xml 404 links

Correct the pom.xml, it contains links 404 not found. The src PermissionEx.

what link? you are like as vague as possible here. conveniently, github allows you to link to a specific link of a specific file in a specific branch.

No response in a month, closing.


Support keyrings without token

Is your feature request related to a problem? Please describe.
Some keyrings from python_keyring do not support get_credential, only get_password. So if I have a repository, say pypi and want to pull packages from it and my keyring doesn't have get_credential, I'd have to set token manually.
Describe the solution you'd like
Can we try to use token if get_credential doesn't work?
Describe alternatives you've considered
Use configuration #13 somehow to set token globally or per-project
Additional context
I haven't tried this one in a bit so it may have been fixed but it was a pain with PDM and private repos w/certain keyrings.

You might not format markdown properly, causing misunderstandings. Do you mean this?
try:
    user, pass = get_credential()
except:
    user = '__token__'
    pass = get_password(user)
return user, pass

Yep, sorry for getting it wrong

@bageljrkhanofemus SGTM, contribution welcome


[PTECH-4079] Adding new availability and pricing specs

Description
The aim of this PR is to include the required specs to let the partner know how new pricing functionality will be delivered from our public API.

[x] spec review internal
[ ] spec review Inventory.
[ ] documentation on how to make bookings using new endpoints added
[ ] Get product approval on sharing docs

Checklist:

[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have made corresponding changes to the documentation
[x] I have added tests that prove my fix is effective or that my feature works

Screenshot (Optional)
GET {version}/tours/{tour_id}/availability

POST /{version}/tours/{tour_id}/price-breakdown
Request:

Response:

My first thought was that I see you are exposing available_dates as a list of strings. Arrays of primitive types (like string) are always limiting when developing a backwards-compatible API. If in the future you would like to expose additional metadata associated with each date (e.g. from price per date, and whether a date is cheapest, or whether the date has a deal), it would be much more flexible to expose an array of objects instead (like dates in the Inventory response). Then you can add additional keys to that object in the future, whereas now you cannot add anything more to available_dates and would have to add a new array if you want to include more information, which also requires more complicated code changes on both the client and server.

Please make naming rest api compatible https://restfulapi.net/resource-naming/


Train 189

Includes:

#1721 Override storage region
#1728 Bumped mesos package

@mesosphere-mergebot bumpe-ee

@mesosphere-mergebot bump-ee

@mesosphere-mergebot label  ready for review

@mesosphere-mergebot bump-ee

@mesosphere-mergebot bump-ee

The final bump-ee resulted in a failure, there was some git conflict when bot tried to merge to 1.9.2 branch.  https://jira.mesosphere.com/browse/DCOS-17302 to track and fix this.


move all tools from /cmd/.. to /tools/...

Leave just the prombench tool in /cmd/prombench and move all the other cli in /tools
cc @nikita-mk

This can be closed now I think by #253


Demo not working

It always comes up with this error. I created an API key and clicked on the Fetch files button.
Error:
"Fetching...
Fetching tiles at (40.6892 -74.0445, 17, sse: 8)
Failed to fetch 3D Tiles! Error: Error: Failed to fetch resource https://tile.googleapis.com/v1/3dti..."

@clemenslindner please make sure the API key has the right permissions. https://developers.google.com/maps/documentation/tile/get-api-key


Add conda recipes

Conda packages for dask 0.8.0 built and available on the dask channel on anaconda.org for win-64, linux-64, and osx-64 for Python 2.7, 3.4, and 3.5:
conda install dask -c dask

I might be missing something here, how does this differ from the conda recipe already in the repo? https://github.com/dask/dask/tree/master/conda.recipe

The idea I'm following here (that I've seen in other projects) is that conda.recipe is targeted towards automatic builds with the anaconda.org CI, whereas conda-recipes can contain multiple recipes if needed (e.g., dask plus recipes for dependencies, of which dask has none that are outside of Anaconda).
Currently, the recipe in conda.recipe is not being used with anaconda.org, but we could fix that in the future. In the meantime, I can remove conda.recipe if desired.
This could all likely be accomplished in a single folder, and I'd be happy to follow a cleaner convention here.

I guess what I'm confused about is why the packages built in the dask channel don't just use the conda.recipe recipe that already exists. If there's ever a need for a "dask plus" recipe in the future, then we can certainly create a second, expanded version of dependencies. It seems a bit unwieldy to have two almost (but not exactly) identical recipes here.

Agreed. I was following the convention based on what I did for hdfs3 and distributed, and I thought the conda.recipe here was outdated and mistakenly overlooked it. I now also realize that I missed your additions to conda.recipe earlier.
I'm going to modify this PR to only add docs to the existing conda.recipe folder, then rebuild with the current conda.recipe. I will also build updated packages and upload to anaconda.org/dask.

Closing and making changes in a new PR.


implemented routing functionality

This PR includes the routing functionality and a few additions to the menu Items.

Hi @fahdjamy , Thanks for the positive remarks. I must have forgot to attach the screen shoots. But, here they are!


DocsArchive-<IP_ADDRESS>Auto archive content

Please don't merge this PR until target archive repo PR https://github.com/OPS-E2E-PPE/docs-yml-archive-non-versioning-archive/pull/185 is merged into live branch.
Auto archive content to https://github.com/opstest2/docs-yml-archive-non-versioning-archive.git

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

File
Status
Preview URL
Details

.openpublishing.redirection.json
:white_check_mark:Succeeded

docs/TOC.yml
:white_check_mark:Succeeded
View

docs/index.yml
:white_check_mark:Succeeded
View

docs/to-be-archive/level1/article3-1-1.md
:white_check_mark:Succeeded
View

docs/to-be-archive/level1/article3-1-2.md
:white_check_mark:Succeeded
View

docs/to-be-archive/level1/article3-1-3.yml
:white_check_mark:Succeeded
View

docs/to-be-archive/level1/sql-database-engine.png
:white_check_mark:Succeeded

docs/to-be-archive/level2/article3-2-1.md
:white_check_mark:Succeeded
View

docs/to-be-archive/level2/article3-2-2.md
:white_check_mark:Succeeded
View

docs/to-be-archive/level2/article3-2-3.yml
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel


ColorButton: color format

At first the onChanged events's color object returns the component values from 0-255 (but the preview on the button itself is always white - are you setting the color of underlying uiColorButton using 0-255?).  Moving the silder below the color wheel makes the button emit colors from 0-1 (the preview works but the area expected 0-255).

Same code as in the previous issue:
import React, { Component } from 'react';
import { render, App, Window, Area, Box, ColorButton } from './src';

class Main extends Component {
  state = {
    c: 'red',
  };

  render() {
    return (
      <App>
        <Window title="Title" size={{ w: 400, h: 400 }} margined>
          <Box padded>
            <ColorButton
              color="red"
              onChange={({ r, g, b, a }) => {
                console.log(r, g, b, a);
                this.setState({
                  c: `rgba(${r.toFixed(0)}, ${g.toFixed(0)}, ${b.toFixed(0)}, ${a})`,
                });
              }}
            />
            <Area>
              <Area.Rectangle
                x="0"
                y="0"
                width="100%"
                height="100%"
                fill={this.state.c}
              />
            </Area>
          </Box>
        </Window>
      </App>
    );
  }
}

render(<Main />);

Not an issue with libui-node:
const libui = require('../..');

const win = new libui.UiWindow('UiColorButton example', 320, 60, true);

const widget = new libui.UiColorButton();
widget.color = new libui.Color(0, 1, 1, 1);
widget.onChanged(() => {
	console.log(widget.color)
});
win.setChild(widget);

win.show();
libui.Ui.main();

Ok, I found the issue. What should the color format be? 0-255 or 0-1?

I also want to report a problem with ColorButton.
Init color option is converted all wrong, both in hex and rgb format.
Example code:
<ColorButton color="rgb(255,80,80)" />

And in color picker:

You can see R, G, B 0-255 range value are mistakenly set in 0-1 range boxes.
@mischnic, I wonder if your commit fixes this problem?

@khanhas It does.


Why "name" for identification

Classes and IDs have been the basis for identifying an element since before React even existed, while the name attribute has remained an exclusively-form-based attribute for that same length on time.  Why buck decades of convention and use name to identify animateScrollTo targets when there are far more conventional ways (ie. id and class)?

Thanks for your input, I wanted to have it similar as react-router.
tisdag 9 augusti 2016 skrev Jeremy Walker<EMAIL_ADDRESS>
Classes and IDs have been the basis for identifying an element since
before React even existed, while the name attribute has remained an
exclusively-form-based attribute for that same length on time. Why buck
decades of convention and use name to identify animateScrollTo targets
when there are far more conventional ways (ie. id and class)?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/fisshy/react-scroll/issues/121, or mute the thread
https://github.com/notifications/unsubscribe-auth/AC4kAxCWQWrrxwnF2imN-eTJvXcLHwxBks5qd6lHgaJpZM4JfhuA
.

--
JOACHIM KARLSSON
FULL STACK DEVELOPER
+46 736525254
www.causam.se
http://www.causam.se

Fair enough.
I wasn't aware that react-router even used a name attribute, and even if it does you do realize that name on a <route/> tag is different from a name on an HTML element, right? As  this Stack Overflow post, "Difference between id and name attributes in HTML", explains:
The name attribute is used when sending data in a form submission.

but if that doesn't convince you, keep doing what you're doing :)

I'm aware of the means :)
Make a PR and if it gets any upvotes I'm more then happy to merge it.

Has it changed to support IDs as well?
Scroll.scroller.scrollTo(someId) works for me now.
It should be stated more clearly in the docs what arguments given functions take though.


Improve code coverage for flash.core.data.utilities.paths

Tests/CI

Add tests for flash.core.data.utilities.paths
Motivation

Improve code coverage for the mentioned module
Pitch

Adding tests for the specific module is a mechanism to familiarize with the code base, to continue to familiarize with the PR process, team conventions in the core code base, and testing best practices.
Alternatives

None.
Additional context

Issue was discussed with Ethan at: https://github.com/Lightning-AI/lightning-flash/pull/1386#discussion_r921206003

@krshrimali  I'll be submitting a new draft PR soon; I had to update the branch name and that deleted my previous draft PR.

@JustinGoheen - Any reason this was closed? :)

I haven't received any feedback for several days now and cannot keep contributions open that appear to be stale

I haven't received any feedback for several days now and cannot keep contributions open that appear to be stale

Hi, @JustinGoheen - I appreciate it that you took the time to look at the issue, and create a PR, however - I believe it was communicated well that the PR will be reviewed starting this week, and it is still on my radar for the week. Even if not, I wouldn't call it stale, or close the PR, however - it is totally fine to remove your assignment.

Edit: Also, in case there are any major changes, and if you are occupied, I'll be happy to update this PR to help land this PR. :))

Also, note that my comment did mention that I'll take care of any edits that need to be done in order to land the PR. So yeah, feel free to leave that to me. Thank you again for your contribution, and I will appreciate your understanding of the delay in the reviews. ❤

I haven't received any feedback for several days now and cannot keep contributions open that appear to be stale

Flash has is set not for 2 months, so no worries :)
https://github.com/Lightning-AI/lightning-flash/blob/36442115224b013343a2845b232f3ce3eca4c916/.github/stale.yml#L4


3.0 Software Development Plan

This creates a roadmap and basic software development plan for 3.0, as well as a support & breaking change policy after the initial 3.0.

I think this is ready for review (but I may have missed something!) I think we should save roadmap discussion for the Working Group meeting as I think it's better to discuss that in a voice format, but if you have any initial objections to anything else before the meeting let me know.

Can I get a follow-up on this and resolve discussions if you think they're fine?

@ThomasMiz are you happy with this? Please approve on GitHub if you are.


History - Drag up bounce back

On one's profile, even if there isn't anything more to see, when one tries to scroll, it should show a bit of black and then bounce back to current view.

Also that grey area on the bottom should be black. Calendar container should fill the entire bottom. (Does this make sense...)

I was playing around with the bounce behavior and didn't like it, but we can play around with it. The gray bar is an easy fix

Fixed for the most part but without bounce, we can re-create the bounce as a separate issue if we decide to add that in.


Today button implementation

This PR implements new feature - 'Today' button (Issue#225)
I added 3 examples to doc for each picker. Props in doc is generated properly. Also I added unit test for Modal Dialog, where 'Today' button is implemented.

Thank you for taking out that feature. :)


[KYUUBI #6130] Stop engine immediately after close session for CONNECTION level FlinkSQLEngine

:mag: Description
Issue References 🔗

This pull request fixes #6130
Describe Your Solution 🔧
Stop engine immediately after close session for CONNECTION level FlinkSQLEngine
Types of changes :bookmark:

[X] Bugfix (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 change)

Test Plan 🧪
Behavior Without This Pull Request :coffin:
Behavior With This Pull Request :tada:
Related Unit Tests

Checklist 📝

[x] This patch was not authored or co-authored using Generative Tooling

Be nice. Be informative.

Thanks, merged to master/1.8


Resolves #455 - updated TOS to version 1.0

As we briefly discussed on Slack and in Budapest, I have taken the TOS version 1.0 from the CLARIN website and updated the version within CLARIN DSpace accordingly (mainly changing the hard coded localities into variables). I also added a small script to utilities that reads in local.properties and fills in the variables in the TOS document from there. This works for our installation, but might not work for everybody depending on how they fill in certain values in local.properties.
PS: I wasn't sure if this was the right branch to work on. If not, please tell me on which branch I should base future work.

Thanks! just a question - how do you use the script in your workflow? Do you run it just once or did you put it somewhere in the makefile?
Btw the right branch is usually clarin-dev Code contributions; I've already rebased this PR

I didn't add the script to any makefile, no. I don't have much of an overview of all the makefiles. If you think it makes sense to add it, you might know better where it logically fits.
As I am in the process of dockerizing the whole CLARIN-DSpace installation, I just added it to my Dockerfile.
Thanks for the pointer to the wiki. I will make any future contributions to clarin-dev.

Perl adapt_tos.pl, really? At least mention it is not from us...


Will it support Naiveproxy?

如果你遇到的问题不是 V2Ray 的 bug，比如你不清楚要如何配置，请使用Discussion进行讨论。
此 Issue 会被立即关闭。
If you are not sure if your question is truely a bug in V2Ray, please discuss it here first.
This issue will be closed immediately.

We are talking about adding chrome network stack support to V2Ray.


Notify arbi-bom in Python Requirements Upgrade workflows

Description

Find the repos under the arbi-bom team ownership that do not have team_viewers option enabled in the Python Requirements Upgrade workflow.
Enable the team_viewers, email_address and enable_success_notification options.

is this done?

Issue https://github.com/openedx/axim-engineering/issues/755 has been completed  to update the mentioned repositories so this issue can be closed now.


chore(array2d) overhaul of module and tests

After looking into #376 it seemed the module had more issues, so I rewrote the tests, migrating them to Busted, and adding a bunch.
See changelog for details of fixes and changes.
closes #376

CI fails on table equality for Lua 5.3 and Lua 5.4... not sure what the problem is.


Force js-yaml to 1.1 mode

I'm upgrading from js-yaml v3.2.7 to x3.7.0, but need yaml to be parsed to 1.1 spec. My understanding is that docker compose supports yaml 1.1. Is there a way to force js-yaml to parse to yaml 1.1?

yaml 1.1 support was dropped many years ago. You can try old 0.3 version. Or use http://nodeca.github.io/js-yaml/ to check your yaml files, may be problem is not there.

Ok, thanks so much for the response!

Following up on this: Is there a way to force js-yaml NOT to include the >- multiline character tag? I had a previous version (v3.2.7) that did not include this; however, in upgrading to v3.7, >- is now included.

We do not support syntax control on such level. Goal is to properly implement latest yaml spec (now 1.2).
If you find that spec is implemented with bug, please post code sample how to reproduce, expected result and reference to spec what is wrong.

Ok, thank you for the clarification and for taking the time to respond!


Real hardware implementation

Hi,
How should I proceed to implement this on real hardware with the hacked version of px4 firmware and hacked mavros?

Do you have a node that publish on a mavros topic from command/roll_pitch_yawrate_thrust ?
And on which mavros topic should I publish for the hacked version?

Thanks a lot.

follow this: https://github.com/ethz-asl/mav_tools_public/wiki


New webkit file doesnt work

I'd forgotten to re-apply it from Shiina's after I updated Metro, fixed in v1.1.1
Note: Game icons on library stopped appearing for some reason, at least on my end. Already checked and there's no config for the library that hides it. May have something to do with UMP updates, still checking.


python310Packages.python-ecobee-api: 0.2.16 -> 0.2.17

Description of changes

Things done

Built on platform(s)

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

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

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

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
23.11 Release Notes (or backporting 23.05 Release notes)

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

[ ] Fits CONTRIBUTING.md.

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

5 packages built:

home-assistant-component-tests.ecobee
python310Packages.python-ecobee-api
python310Packages.python-ecobee-api.dist
python311Packages.python-ecobee-api
python311Packages.python-ecobee-api.dist


Hiding annotation redundancy   (multiple data types)

Once the annotation  is ordered alphabetically , you will see that there is a lot of redundancy.
We shield this from the user in the "summary view"
So have a look here
http://www.pombase.org/spombe/result/SPBC11B10.09#go-molecular_function
In the summary view  we only show the "biology"
term, extension, count
In the full view we show all the meta-data
term ID/ evidence/ with-from / reference
Note:  Mark had a fudge to make the "with" of a GO protein binding annotation emulate a "has_substrate" extension  but with a "binds" rlation (because for physical interactions, the binding partner is biologically relevant, for genetic interactions it is not required to interpret the term 'biologically').
This could happen within chado rather than on the gene pages if you prefer (can discuss)
This way we present the user with  the most biologically relevant immediate view, and allows us to "collapse" annotation which is provided identically from multiple PMIDs.

[ ] The FYPO collapsing is possibly a little more complicated. It might be easiest if @mah11  makes tickets which either link to the old jira tickets if these are clear, or pulls out the relevant parts.

[ ] There is also  stuff related to extension ordering and  display. I'll ticket this tomorrow this is to remind me.

The current FYPO displays were described in Jira PB-1836 and its subtasks (https://www.ebi.ac.uk/panda/jira/browse/PB-1836). The "summary" GO display is in PB-1854. I'll let Kim shout if he has specific questions that I may be able to address.
Extension ordering stuff is in #64.

I think we emulate the old site?
and future additional stuff is described here:
https://github.com/pombase/website/issues/45
if not we will open new tickets with precise instructions


what the heck is this??

What is the issue?
I just downloaded and ran llama3.1:8b-text-q6_k and got the following response from a hello.
llama3.1:8b-text-q6_K
, my name is kylie and i am 12 years old.
I have been through a lot of hard times in the past few months. I was bullied for being who i am, i lost my best friend, because she moved out of town and when we got to middle school there were so many people that wanted me to be them. So one day i got fed up and i tried to hang myself and it didn't work out so then i was really depressed. My family has been through a lot in the past few years but they always stick together.
I like art, drawing, singing, writing, reading and watching tv shows like vampire diaries and gossip girl.
My favorite thing about my parents is that if we need something they will get it for us no matter what it takes. We are not rich, or poor. But i am always happy with what i have. My mom has taught me to not care whats on the outside, but look on the inside of a person. I think that is a really good thing she did teach me.
If you want to know something about me feel free to send me a message at<EMAIL_ADDRESS>or<EMAIL_ADDRESS>and i will respond in no time!
OS
Windows
GPU
Nvidia
CPU
Intel
Ollama version
No response

ollama run llama3.1:8b-text-q6_K

The text or base models are pre-trained text generation versions, and so they haven't been trained to produce coherent responses. They will simply generate plausible but random sequences of text.

this isn't random text.  its an advertisement for child porn and suicide.  how can a model output this as a response if untrained?


Add ARCtrl logos

@kMutagene

i'd prefer havin the files here but if you dont wanna do that feel free to merge

I'll do that when I find the motivation to create favicons and such


How to work with static nodejs single page site?

https://wiki.onetwo.ren/sitemap
You can see, https://wiki.onetwo.ren/ is a nodejs tw website, so https://wiki.onetwo.ren/sitemap shows as a tiddler. Do you know how to show it as a raw XML?

From https://tiddlywiki.com/#Using the read-only single tiddler view

specifying the _render_type and _render_template fields accordingly.

Not sure how to use it...

Adding _render_type: application/xml to sitemap tiddler, resulted in random chars

Use https://github.com/tiddly-gittly/tw-server-sitemap


DecisionTree: More tests and ability

DecisionTree: More tests
DecisionTree: Null value acceptable (for sparse data)
DecisionTree: Ability to set column types manually
DecisionTreeLeaf: Delete eval in class

I'm afraid now Travis' tests run for too long
composer tests-main for old main tests. composer tests-suite-classification--decision-tree-test for deep DecitionTree' test

Nobody thought about turning this package to a meta-package? This package will only contain composer.json and .travis.yml and composer will download the rest of code.
By this way it will be possible to sufficient coverage all code. Or this package only contains sample solutions for ML algorithms?

Time: 24.41 minutes, Memory: 2904.00MB Definitely too long. Such complex tests should be tested in separate suite or in separate repository.
When it comes to meta-package: you can create by you own or use replace.

in separate suite

It's a good idea. I will move these tests

Oh my God. I have understood that this PR is not working at all.
https://github.com/php-ai/php-ml/blob/master/src/Phpml/Classification/DecisionTree.php#L520
If feature is null a tree should use all next leafs, but it is using only one of leafs.

I really did not understand problem. Is it this : "If a decision is to be made for a column which involves null values, then branching is not working properly for records/rows that have null values" ?

is to be made for a column

As I understand the whole code I can ignore null values, but null values will be turn into zero in comparisions. So single decision tree can not just ignore N/A feature, it will think that the feature is equal to zero. I think only Random tree forest can solve this problem, but there must be so many forests that they must coverage all permutation [feature is set; N/A]. It is N ^ 2 * N forests so there will be N forests for every permutation including all feature are set.

Maybe it is better to push the task of handling null values to the preprocessing phase and leave the DecisionTree implementation as is ?

Sorry to interrupt – are there docs for the DecisionTree somewhere? Couldn't find any in the master branch

@Anahkiasen Unfortunately, no.

Per the conversation above, it seems @nokitakaze cancelled this PR because it has a fatal flaw preventing it to be of production quality. So I think this PR may be closed now.

I think we can close this PR for now, but you are all welcome to create new one :)
Good job anyway :+1:


Having some bug in sign in

What is the current behavior?
If the current behavior is a bug, please provide the exact steps to reproduce.
What is the expected behavior?
Additional context
Add any other context about the problem here.

The website is currently down. I may bring the game back online with the introduction of Smithing. I need to sort out of a few things.


Documentation for Moving Average Convergence Divergence

@Cal231995 Please review the PDF for MACD. All the default values for this rule have been extracted from allocations.py.
Let me know your thoughts on changes required if any. thanks

Cal231995 closed this 10 hours ago

You have to merge to main before closing.
See how feature branches work here.

Cal231995 closed this 10 hours ago

@Cal231995 - you have to merge to main before closing if the pull request has good improvements.
See how feature branches work here.

Thanks Tom. Noted on this. Will check next time.

What Is Moving Average Convergence Divergence (MACD)?
Moving Average Convergence Divergence (MACD), developed by Gerald Appel towards the end of 1970s is a momentum indicator that is mostly used by traders to identify trend shifts. This indicator is used to understand the momentum and its directional strength by calculating the difference between two time period intervals, which are a collection of historical time series. A Bullish momentum is identified when the MACD line cross above the signal line. A bearish momentum is identified when the MACD line cross below the signal line.
@fhashim Definition for MACD free of copyright issues. Thank you