Import from Calibre library

Is your feature request related to a problem? Please describe.
A Calibre user may have books already in a library. It would be convenient if books there are imported to Lute in batch.
Describe the solution you'd like
Read the Calibre library and add them in Lute.

Unfortunately this won't be easily possible.  Lute is a web app and so doesn't necessarily have access to the system libraries necessary for Calibre imports.  In addition, since (at this point) Lute is still in early stages, we can't consider features that will only be applicable to a relatively small segment of the user base.
I'll leave this open, but will tag it as "wont-fix" for now due to the above reasons.  It would be nice to support such a thing, but it feels out of scope at least for now.

Closing to manage the backlog only.


Order with payment "stuck" in processing state when complete request fails

Steps to reproduce

Mock the CompleteOrderApi::complete method to pass an additional header parameter to the capture action request so that Paypal will generate an error (doc: https://developer.paypal.com/docs/business/test-and-go-live/simulation-tests/#orders) like this:

        return $this->client->post(sprintf('v2/checkout/orders/%s/capture', $orderId), $token, null, [
            'PayPal-Mock-Response' => '{"mock_application_codes": "INSTRUMENT_DECLINED"}'
        ]);

Place an order
In "Pay with Paypal page" use the yellow paypal button, and confirm the payment on Paypal
You will be redirected to the homepage, but if you try to retry the payment it will fail cause it's in "PROCESSING" state but the CreatePayPalOrderAction class try to load the last payment with "NEW" state and it won't find any.

Possible solution
This does not happen with HostedFields cause there is an additional check here that cancel the payment if it is still processing after the complete call.
But i think that this solution is weak. If, for example, the user closes the page while the capture request is going on, it will never call the cancel payment. Or maybe due to another error, the cancel payment url could never be called, and the payment of the order will remain PROCESSING.
Probably, it should be responsibility of the CompletePayPalOrderAction controller to cancel the payment if it failed. What do you think?

Generally, i think that the PayPalClient class should "handle" properly the cases in which Paypal return a response with a status code that is not 20X. Actually the Client only logs the error when that happen. Maybe it could throw exception and the classes that use the PayPalClient will catch that exception and operate consequently.

Generally, i think that the PayPalClient class should "handle" properly the cases in which Paypal return a response with a status code that is not 20X. Actually the Client only logs the error when that happen. Maybe it could throw exception and the classes that use the PayPalClient will catch that exception and operate consequently.

When the payment status is "processing", and the paypal status is "captured", the cancel last payment should re-capture the payment instead of mark the payment as "canceled"  as the actual payment has already been done.


Refine CGContext's user<->device coordinate space boundary

CGContext will now strive, wherever possible, to store its internal state in fully-device-transformed units. This transformation includes the device scale (flip+translate from CG to D2D) and the user scale.
To enable this, this pull request also adds the ability to specify that coordinate system a drawn geometry is in: paths will use the premultiplied values, and user-space geometry values will use userspace->device transformation.
This allows us to leverage the Direct2D effect stack without having to apply input and output transformations to the effects, and simplifies shadow rendering (we no longer transform for userspace in the commandlist and then device space when we draw it.)

This change is

Refs #1268.

@rajsesh-msft @aballway tests done been added!


Extend timeout in Elasticbean to 5 minutes

should resould our timeout problem
not extremely confident about it since it don't know ElasticBean much
followed the instruction at
https://medium.com/tomincode/extending-gateway-timeouts-with-node-js-elastic-beanstalk-applications-5cb256f08f4b

If we do this we also need to modify it via in the load blancers, don't know if that's handled by Terraform or manually

Finally, unless you’re only running a single instance, you need to tell ELB (Elastic Load Balancer) to keep the connection open longer. You can do this from the EC2 configuration panel. Find Load Balancers in the sidebar, select the load balancer in front of your application (you can use the Instances tab to identify which balancer you need) and at the bottom of the Description tab, hit the Edit idle timeout button to up your timeout.

Easiest way to try is to deploy that branch on staging.

This fails when deploying:
Logs are:
[Thu Jun 17 05:15:15.752577 2021] [:error] [pid 9073] [remote <IP_ADDRESS>:212] mod_wsgi (pid=9073): Target WSGI script '/opt/python/current/app/hat/wsgi.py' cannot be loaded as Python module.
[Thu Jun 17 05:15:15.752618 2021] [:error] [pid 9073] [remote <IP_ADDRESS>:212] mod_wsgi (pid=9073): Exception occurred processing WSGI script '/opt/python/current/app/hat/wsgi.py'.

I'm seeing this in the deploy log
 2021-06-17 05:06:15    ERROR   [Instance: i-0216d3686039ae8a2] Command failed on instance. Return code: 2 Output: sed: can't read /tmp/deployment/config/#etc#nginx#conf.d#00_elastic_beanstalk_proxy.conf: No such file or directory.
container_command extend_proxy_timeout in .ebextensions/extend-proxy-timeout.config failed. For more detail, check /var/log/eb-activity.log using console or EB CLI.
2021-06-17 05:06:15    INFO    Command execution completed on all instances. Summary: [Successful: 0, Failed: 1].
2021-06-17 05:06:15    ERROR   Unsuccessful command execution on instance id(s) 'i-0216d3686039ae8a2'. Aborting the operation.

I thought the blog was outdated, but playing around a bit in the VM it seems we are using Apache anyway so there is no chance that would have worked?

New version for Apache.

For this, feel free to deploy on stating untill  it works and the merge

For this, feel free to deploy on stating until it works and the merge

I already tested it on staging and it was working, didn't you see the message I left you on slack about it?


rename incompletely implemented?

Link a new container that has the same name as a container that has been renamed and the old container is referenced which means the link is using the old container ID instead of respecting the new container name.
renamed database container:
docker rename DBContainerA DBContainerB
created a new database container with the same name as the original renamed container:
docker run --name="DBContainerA" ...
created a new application container supposedly linking the new database container:
docker run --name="AppContainer"  --link DBContainerA:myAlias ...
This doesn't work as expected because it's actually linking DBContainerB. It will only link DBContainerA if I link using the ID of the new DBContainerA.
If this is intended behaviour it needs to be documented.
uname -a
Linux zapta04meta01 3.13.0-45-generic #74-Ubuntu SMP Tue Jan 13 19:36:28 UTC 2015 x86_64 x86_64 x86_64 GNU/Linux
docker version
Client version: 1.5.0
Client API version: 1.17
Go version (client): go1.4.1
Git commit (client): a8a31ef
OS/Arch (client): linux/amd64
Server version: 1.5.0
Server API version: 1.17
Go version (server): go1.4.1
Git commit (server): a8a31ef
docker info
Containers: 4
Images: 206
Storage Driver: aufs
Root Dir: /var/lib/docker/aufs
Backing Filesystem: extfs
Dirs: 214
Execution Driver: native-0.2
Kernel Version: 3.13.0-45-generic
Operating System: Ubuntu 14.04.1 LTS
CPUs: 4
Total Memory: 3.86 GiB
Name: zapta04meta01
ID: RLGL:LARG:JZ3P:RPN3:MRNC:7EIJ:232S:C2RU:REDN:3LOO:4Q2B:BSRP
WARNING: No swap limit support

Thanks for reporting. Would you be able to provide a simple example to reproduce the steps you've taken? This may assist the maintainers to investigate. Also the output of docker version,  docker info and uname -a never hurt.

I think this is pretty much expected since you can't start a new container (pre 1.5)  with the same name and have the links work either... is it intentionally this way? I'm not certain.

Rename only changes the name of the container. The container should still use the old id.

@brahmaroutu is right, It's better to use a container ID to link rather than use the container name

I think all answers above miss the issue;

This doesn't work as expected because it's actually linking DBContainerB. It will only link DBContainerA if I link using the ID of the new DBContainerA.

If I read it correctly, the steps taken were (slightly reworded)

docker run --name="my-db" database
docker rename my-db my-old-db
docker run --name="my-db" database
docker run --link my-db:my-alias myApp

And the myApp container ends up being linked to my-old-db, not my-db, even though the myApp container was created and started after the database-container was renamed.
I haven't had time to test this scenario and see if I could reproduce, but if this is indeed what's happening! this looks like a bug.
Possibly the IP-addresses aren't updated properly, because the old container still exists, but is only renamed?

@thaJeztah you have it spot on.

I tried this out and I see the my-alias is correctly mapping to the IP address of the my-db container created in the step 3. Please check /etc/hosts file in the container created in the step 4 and see if it is using the IP address for the container from step 3. @thaJeztah  is right, we do not use the name of the container to link.
@gubuntu Please check the ipaddresses and connection setting in your environment.

I cannot repro this, here is my test case:
#!/bin/bash

docker run -d --name redis redis > /dev/null
docker rename redis redis_old > /dev/null
docker run -d --name redis redis > /dev/null
redis_ip=$(docker inspect -f '{{ .NetworkSettings.IPAddress }}' redis)
link_ip=$(docker run --link redis:db --rm busybox /bin/sh -c 'echo $DB_PORT_6379_TCP_ADDR')

[ "$redis_ip" == "$link_ip" ] || out=FAIL && \
  (
    # Remove the redis_old to make sure we are not talking to it
    docker rm -f redis_old > /dev/null
    docker run --rm --link redis:db busybox /bin/sh -c 'ping -c 1 db' > /dev/null
  )

if [ $? -gt 0 ]; then
  out=FAIL
else
  out=PASS
fi

docker rm -f redis > /dev/null

echo $out
[ "$out" == "FAIL" ] && exit 1

@cpuguy83 I have also been trying to reproduce, and wasn't able to reproduce the problem so far.
Only thing left I wanted to try is;

create db container
create app-container, linking to db-container
rename db container to db-old
create new db-container
create new app container and link that to the new db-container

Wondering if the keeping the "old" app-container from step 2. around would somehow cause trouble.

@thaJeztah As mentioned above, I had my container from step 3 running and it did not cause any link issues.

@brahmaroutu ah, yes.
In that case, I think this should be closed, unless @gubuntu is able to provide the exact steps to reproduce.

@cpuguy83 I think this issue can be closed because we cannot reproduce?

Closed since we don't have a reproducible case.
If you can provide a way for us to reproduce (exact steps... like Dockerfile + docker run commands) would be fantastic.
Thanks for the report!


review node type options to use basic types

Currently we map "int" to "irange", "float" to "drange" and sometimes these are too-much. Take GPIO pins, for instance, we're interested in a single integer to represent that as the other 3 (min, max, step) are meaningless in such context.
Then we should introduce new mappings for "irange_spec"/"irange" and "drange_spec"/"drange" and use them where a range specification is needed (but not the value) or the whole set of members are needed (ie: constants).

all changes merged. we're done.


Instagram Feed: Unable to fetch the given user/tag. Instagram responded with the status code: 0 5

now i have faced this issue how can solve it

Can confirm this happens, sometimes the answer it's the same, others are the XMLHttpRequest on the google side.

This library is working fine, but it's not working right now, so I checked my console and found some bugs related to the Instagram feed, changed Library 3 to 3.0.2, but I can not fix it . I shared my Instagram feed code and website URL.
URL : https://mifonemobilesalathur.mydomainbook.com/

--
  |

What's the solution? Same here

There is no solution at the time. Just wait a while.
For me, the script internal caching mitigates this problem most of the time.


Create first account

To test this PR, some actions require to setup

Comment out account information in .env file
Open extension service worker, and type chrome.storage.local.clear() in the console to clear all data in storage.

Summarize
Diagram

Demo
(Link)

I've discovered a bug: I receive a null userOpHash from the provider when using the eth_sendUserOperation method, triggered by insufficient account balance in app/page/authorization/userOperation.tsx.
If it detects that userOpHash is null, it should mark the userOpLog.status as Failed or Rejected and set the userOpLog.receipt.errorMessage to 'insufficient balance'.
I've created a new issue to document this discovery.

@oneleo This is a known issue about error handling. The status should be cateogorized as Failed.


When acorn login succeeds , the success message is presented in Std.Err instead of Std.out.

acorn version v0.6.0+c6ffc99d
Steps to reproduce the problem:
Running the following snippet of code results in test failing because the success message is actually set in "Err" instead of "Out"
	expectedOut := icmd.Expected{
		ExitCode: 0,
		Timeout:  false,
		Error:    "",
		Out:      "✔: Login to " + registryUrl + " succeeded",
		Err:      "",
	}
	icmd.RunCommand("acorn", "login", regUrl, "-u", regUsername, "-p", regPassword).Assert(t, expectedOut)

Failure:
=== RUN   TestAutoUpgradeExtRegUpgradeApp2
    autoupgradeexternalreg_test.go:183: assertion failed:
        Command:  acorn login registry-myregistry-23bc89b1.a-sangee2004-mytest1-3349a0ea-steve1.on-acorn.io -u wrtg7gtf -p fh76cqph9z2z45l7
        ExitCode: 0
        Stdout:
        Stderr:   ✔: Login to registry-myregistry-23bc89b1.a-sangee2004-mytest1-3349a0ea-steve1.on-acorn.io succeeded

        Failures:
        Expected stdout to contain "✔: Login to registry-myregistry-23bc89b1.a-sangee2004-mytest1-3349a0ea-steve1.on-acorn.io succeeded"
--- FAIL: TestAutoUpgradeExtRegUpgradeApp2 (0.46s)

Tests were changed to look for stderr. Not really sure why pterm is writing to stderr


Audio Component?

Does there exist a component like similar to Streamlit audio? Or plans to implement? Would be lovely :)

#26

Thanks for the suggestion, this is something we can add. Agree with @FHU-yezi  see that discussion for component requests.

yeah that would be a great addition


dbt_valid_to date should be set to udpated_at column date value

The dbt_valid_to date should be set to udpated_at column date value not the current timestamp.
Per the documentation: https://docs.getdbt.com/docs/build/snapshots#snapshot-meta-fields
https://github.com/dbt-athena/dbt-athena/blob/5598286b0069a6cf4e630891286af0b3743149f9/dbt/include/athena/macros/materializations/snapshots/snapshot.sql#L213
WHEN dbt_valid_to=CAST('9999-01-01' as timestamp) AND is_current_record=True THEN {{ current_timestamp() }}
This is causing overlaps in our valid date ranges.
Here are the meta data columns from one id:
id                                                                          updated_at                                   dbt_valid_from                          dbt_valid_to                              is_current_record
47a555a1-1455-481a-8937-d3cc03f02362  2021-09-02 13:58:09.961000  2021-09-02 13:58:09.961000  2023-02-14 20:05:44.433000  false
47a555a1-1455-481a-8937-d3cc03f02362  2021-09-03 13:57:59.941000  2021-09-03 13:57:59.941000  9999-01-01 00:00:00.000000  true
This was implemented with the following pull request:
#111

@rjhartings would you mind to open a pr with the fix?


Finance

🛠 Issue (Number) 887

Closes #887
Description
I have fixed the bug and the project is running smoothely.
Type of change

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

Checklist:

[x] I have made this from my own
[ ] I have taken help from some online resourses
[x] My code follows the style guidelines of this project
[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] My changes generate no new warnings
[x] The title of my pull request is a short description of the requested changes.

ATTACH SCREEN-SHOTS / DEPLOYMENT LINK

i had made the webpage responsive please review it.

above screenshots are of mobile screen

Make it responsive for all devices


Add support for trace spans to the infra attributes processor

What does this PR do?
Implements trace spans processor for infraattributesprocessor. Gets entities from resource attributes and applies entity tags and global tags as resource attributes.

Motivation

Additional Notes

Possible Drawbacks / Trade-offs

Describe how to test/QA your changes

/merge


Move the nextest config to the tools directory

Current
There is some nextest config here
https://github.com/build-trust/ockam/tree/develop/.config
Desired
Move it to https://github.com/build-trust/ockam/tree/develop/tools without breaking CI builds.

fixed in https://github.com/build-trust/ockam/pull/4730


Add order_by feature

Simple shortcut which allows to order by translated column as described in readme.

I really like that feature too, but in it's current version it is not enough, at least for me.
I'm relying on fallbacks and your code doesn't account for them.
I used your code as a starting point to "coalesce" all fallback translations and use that as order by statement.
Perhaps updating the readme with ordering instructions would be a nice compromise until the scope of hstore_translate is properly defined :hourglass_flowing_sand:

Would like this one too.  I mostly add stuff like this:
scope :ordered, -> { order('name') }

Which of course doesn't work in this case.  @daniel-rikowski mind sharing how you solved this without using this modification?

Ok found how to do this pretty easily:
scope :ordered, -> { order("name_translations -> '#{I18n.locale}'") }

@rept This still won't work with fallbacks.
Imagine this fallback configuration:
I18n.fallbacks[I18n.locale] == [:'de-DE', :de, :en]

And these database records:
[
  { name_translations: { 'de-DE' => 'Kanarienvogel', 'en-US' => 'Canary' },
  { name_translations: { 'de' => 'März', 'en' => 'March' },
  { name_translations: { 'en' => 'Wikipedia' },
]

Considering I18n.locale = :'de-DE' your SQL would sort these translations
'Kanarienvogel'
NULL      -- should be 'März'
NULL      -- should be 'Wikipedia'

To achive correct sorting with fallbacks you have to generate a dynamic SQL statement which takes I18n.fallbacks into account. Like this:
SELECT ... ORDER BY COALESCE(
    name_translations -> 'de-DE',
    name_translations -> 'de',
    name_translations -> 'en')

I've created a Gist with my - rather hacky - implementation. Use at your own risk!
https://gist.github.com/daniel-rikowski/1d94ea943bbcf8935fd0f23a5c8b37f6
Example:
Book.order_translated(:title)
Author.references(:books).order_translated(books: :title)

Any plans on getting this integrated in the main branch?


Issue trying to use physical device (Arduino, NodeMCU, ESP8266, etc.) to publish sensors data to ADT!

Hi,
Many thanks for the effort and hard work on this project.
The documentation provided is thorough and sufficient up to the point where only simulated devices are used to mimic data input into ADT.
The question is, if I want to send data from physical sensors using Arduino or NodeMCU or any other devices capable of MQTT connections, what's the payload that can be used to send data that can be updated and compatible with the ADT and what topic for publishing telemetry?
For Example, sending the data in the following format:
Client.publish(Telemetry_topic, TelemetryPayload)
Would be translated on Azure end as :
Client.publish("devices/{device_id}/messages/events/", "some property")
or:
Client.publish("devices/{device_id}/modules/{module_id}/messages/events/", "some property")
Trying to send the telemetry with some topic and payload data to the IoT device seems like the logical way to go. But would it be the right approach to use with ADT?  and if so, what topic and payload can be used to sync the data with ADT sensor properties since they are set as (bool, string, float, int, dateTime) values in UE4?
Any help or suggestion would be much appreciated.
Thank you very much.
lAvArt_Studio

I am interested in this as well.

@adamlash @Naimas
Could I ask for any advice on use physical devices with ADT?


Enable docker log rotate when deploying and add new node.

Coverage increased (+9.2%) to 68.891% when pulling d63fe772fc356cc971d78dd68d7bd9ed899c3e01 on yuye/docker-log-rotation into fbadd60b4fe218071ccd4dfc81432e6f2eb8a7dd on master.


Warn user if Bazel version is floating

(no bazelisk, or using latest in       .bazelversion, or no .bazelversion)

This is only valid if you run a globally installed aspect.


[request error] [unhandled] [500] Status code: 500 when tried to chat with a created KB.

Hit a new issue now:
Chat with knowledge base with id:  19
Knowledge base kb10 with embedding "nomic-embed-text:latest"
Creating embeddings for Ollama served model: nomic-embed-text:latest
Creating Chroma vector store
Initializing ParentDocumentRetriever with RedisDocstore
Redis client options:  { host: 'redis', port: 6379, username: undefined, password: undefined }
Chat with Ollama, Host: http://host.docker.internal:11434
User query:  connected vpc
Reformulated query:  connected vpc
Relevant documents:  [
Document {
pageContent: 'THE VPC CROSS-LINK AND CONNECTED VPC\n' +
'\n' +
'\n' +
'\n' +
'\n' +
'\n' +
'Every production SDDC must be cross-linked to a VPC within the customer-owned AWS account, referred to as the connected VPC. This\n' +
'cross-linking is accomplished using the Cross-Account Elastic network interface\n' +
'[https://docs.aws.amazon.com/AWSEC2/latest/UserGuide/using-eni.html] (ENI) feature of AWS and creates a connection between every\n' +
'host within the management cluster of the SDDC to a subnet within the cross-linked VPC. This cross-link provides the SDDC with a\n' +
'high-bandwidth and low-latency network forwarding path to services maintained within the customer-owned AWS account.\n' +
'\n' +
'[data:image/gif;base64,R0lGODlhAQABAAAAACH5BAEKAAEALAAAAAABAAEAAAICTAEAOw==]',
metadata: {
source: 'https://vmc.techzone.vmware.com/resource/vmware-cloud-aws-network-architecture',
title: 'VMware Cloud on AWS: Network Architecture | VMware',
description: 'Overview The networking architecture of a VMware Cloud on AWS Software-Defined Data Center (SDDC) is arguably one of the most',
language: 'en',
loc: [Object]
}
},
Document {
pageContent: 'These VMkernel adapters are visible from the networking section of the conﬁguration tab of the ESXi host. You can also get a sense\n' +
'of the setup by viewing the TCP/IP conﬁguration for the host.\n' +
'\n' +
'The last portion of the underlay network are the virtual switches. Again, these are visible from the networking section of the\n' +
'conﬁguration tab of the ESXi host. Here, you will see a mix of switches; some that are part of the underlay network and others\n' +
'which represent NSX network segments. The switches which are part of the underlay network are there to provide connectivity to the\n' +
'various management appliances of the SDDC. Of particular note is the management switch, which exists in order to provide the\n' +
'appliances with access to the management network. You can get a sense of some of the other important virtual switches in a host by\n' +
'examining their names as well as the management appliances which are attached to them.\n' +
'\n' +
'\n' +
'THE VPC CROSS-LINK AND CONNECTED VPC',
metadata: {
source: 'https://vmc.techzone.vmware.com/resource/vmware-cloud-aws-network-architecture',
title: 'VMware Cloud on AWS: Network Architecture | VMware',
description: 'Overview The networking architecture of a VMware Cloud on AWS Software-Defined Data Center (SDDC) is arguably one of the most',
language: 'en',
loc: [Object]
}
},
Document {
pageContent: 'The vCenter, NSX Managers and NSX edge appliances are typically deployed in the same AZ. In the event of an AZ failure, if\n' +
'vCenter, NSX Managers, and NSX edge appliances are in the impacted AZ, vSphere HA will restart these appliances and customers’\n' +
'workloads in the surviving AZ. These management appliances and NSX edge appliances will continue to operate there, even after the\n' +
'affected AZ is restored, until a failure or other scenario necessitates their recovery in another AZ. \n' +
'\n' +
'\n' +
'STRETCHED CLUSTER CONNECTED VPC\n' +
'\n' +
'\n' +
'\n' +
'\n' +
'\n' +
'Stretched clusters require an Amazon VPC with two subnets, one subnet per availability zone. The subnets determine ESXi hosts\n' +
'placement between the two AZs. In the event that the active edge appliance migrates between hosts or AZs, the route tables of the\n' +
'connected VPC will be updated to route traﬃc through the appropriate cross-link ENI.\n' +
'\n' +
'[data:image/gif;base64,R0lGODlhAQABAAAAACH5BAEKAAEALAAAAAABAAEAAAICTAEAOw==]\n' +
'\n' +
'\n' +
'CROSS-AZ DATA FLOWS',
metadata: {
source: 'https://vmc.techzone.vmware.com/resource/vmware-cloud-aws-network-architecture',
title: 'VMware Cloud on AWS: Network Architecture | VMware',
description: 'Overview The networking architecture of a VMware Cloud on AWS Software-Defined Data Center (SDDC) is arguably one of the most',
language: 'en',
loc: [Object]
}
},
Document {
pageContent: 'The MTU size for Internet Uplink is 1500 bytes, which is not configurable.\n' +
'\n' +
'\n' +
'SERVICES UPLINK\n' +
'\n' +
'The services uplink connects the SDDC edge appliances to the connected VPC in the customer-owned AWS account. A static route on\n' +
'the SDDC edge appliances is configured for the primary CIDR of the VPC which points to the VPC router as a next hop during the\n' +
'SDDC provisioning. It enables compute workloads to leverage this uplink to consume AWS services and resources in the connected\n' +
'VPC. For services not running in the connected VPC, it is still possible to using the uplink to get connected through VPC\n' +
'endpoints [https://docs.aws.amazon.com/whitepapers/latest/aws-privatelink/what-are-vpc-endpoints.html].',
metadata: {
source: 'https://vmc.techzone.vmware.com/resource/vmware-cloud-aws-network-architecture',
title: 'VMware Cloud on AWS: Network Architecture | VMware',
description: 'Overview The networking architecture of a VMware Cloud on AWS Software-Defined Data Center (SDDC) is arguably one of the most',
language: 'en',
loc: [Object]
}
},
Document {
pageContent: 'VMware Cloud on AWS supports three types of compute network segments: routed, extended and disconnected.\n' +
'\n' +
'The difference of three network types is primarily in their connectivity capability:\n' +
'\n' +
' * Routed – routed (default type) network segments have connectivity to other segments within an SDDC and to external networks.\n' +
' * Extended – extended network segments are used with the NSX L2VPN to create a common broadcast domain between a VMware Cloud on\n' +
'   AWS SDDC and on-premises networks.\n' +
' * Disconnected – disconnected network segments have no uplinks and create an isolated segment. Disconnected networks are\n' +
'   primarily created by HCX, that can be changed to routed networks.\n' +
'\n' +
'For more information, please refer to Understanding Segments in VMC on AWS\n' +
'[https://vmc.techzone.vmware.com/understanding-segments-vmc-aws].\n' +
'\n' +
'\n' +
'CONNECTED VPC CONNECTIVITY',
metadata: {
source: 'https://vmc.techzone.vmware.com/resource/vmware-cloud-aws-network-architecture',
title: 'VMware Cloud on AWS: Network Architecture | VMware',
description: 'Overview The networking architecture of a VMware Cloud on AWS Software-Defined Data Center (SDDC) is arguably one of the most',
language: 'en',
loc: [Object]
}
},
Document {
pageContent: 'For more information, please refer to Understanding Segments in VMC on AWS\n' +
'[https://vmc.techzone.vmware.com/understanding-segments-vmc-aws].\n' +
'\n' +
'\n' +
'CONNECTED VPC CONNECTIVITY\n' +
'\n' +
'\n' +
'\n' +
'\n' +
'\n' +
'Since we must always pass through the Edge SR whenever traﬃc leaves the SDDC, all traﬃc to and from the connected VPC must pass\n' +
'through the active edge appliance of the SDDC. Since this edge appliance is a VM, it resides on a speciﬁc ESXi host. As such, it\n' +
'will always use the Cross-Account ENI for that host. It should be noted that the hosts of the SDDC will be deployed within the\n' +
'same AZ as the cross-link subnet (the subnet which contains the Cross- Account ENIs). This mechanism controls the AZ placement of\n' +
'the SDDC (customer chooses the subnet and thus controls the AZ placement), but also serves to eliminate cross-AZ bandwidth charges\n' +
'between the edge and any native AWS resources within that same AZ.',
metadata: {
source: 'https://vmc.techzone.vmware.com/resource/vmware-cloud-aws-network-architecture',
title: 'VMware Cloud on AWS: Network Architecture | VMware',
description: 'Overview The networking architecture of a VMware Cloud on AWS Software-Defined Data Center (SDDC) is arguably one of the most',
language: 'en',
loc: [Object]
}
},
Document {
pageContent: 'ROUTE PRIORITY\n' +
'\n' +
'\n' +
'\n' +
'\n' +
'\n' +
'When the NSX learns the same route from different sources, the NSX T0 router will choose the route based on its source in the\n' +
'following order:\n' +
'\n' +
' * Connected VPC\n' +
' * Route-based VPN (“Use VPN as backup to Direct Connect” is OFF)\n' +
' * Policy-based VPN *\n' +
' * VMware Transit Connect\n' +
' * Direct Connect Private VIF\n' +
' * Route-based VPN (“Use VPN as Backup to Direct Connect” is ON) \n' +
' * AWS Internet Gateway \n' +
'\n' +
'IPsec VPN will never be used for traffic from ESXi hosts if SDDC is a member of an SDDC Group or has a DX Private VIF attached.\n' +
'\n' +
' Use VPN as Backup to Direct Connect option is not applied to routes learned through VMware Transit Connect.\n' +
'\n' +
' Default Route only.\n' +
'\n' +
'\n' +
'NETWORK SERVICES\n' +
'\n' +
'\n' +
'\n' +
'\n' +
'\n' +
'\n' +
'VPC PEERING FOR EXTERNAL STORAGE CONNECTION',
metadata: {
source: 'https://vmc.techzone.vmware.com/resource/vmware-cloud-aws-network-architecture',
title: 'VMware Cloud on AWS: Network Architecture | VMware',
description: 'Overview The networking architecture of a VMware Cloud on AWS Software-Defined Data Center (SDDC) is arguably one of the most',
language: 'en',
loc: [Object]
}
},
Document {
pageContent: ' Customers VPC connections are achieved through resource sharing from the VMware AWS account to the customer-owned AWS account.\n' +
'   Once the VTGW resource sharing request is accepted in the customers AWS      Resource Access Manager, customer VPCs can be\n' +
'   attached to VTGW through a VPC attachment from the AWS console. The attachment request must be accepted in VMware Cloud on AWS\n' +
'   before the attachment is completed.\n' +
' * In terms of interconnectivity for customers AWS Transit Connect Gateways, these TGWs would be interconnected to VTGW through\n' +
'   TGW peering. In a multi-region SDDC group, the choice of region for AWS TGW connection to is configurable.\n' +
" * Similarly, VTGW can be associated with customers DXGW through a TGW association. In the association configuration, 'allow\n" +
"   prefixes' will be defined to specify which routes will be advertised to on-premises networks over the DXGW, with an AWS limit\n" +
'   of up to 20 prefixes.',
metadata: {
source: 'https://vmc.techzone.vmware.com/resource/vmware-cloud-aws-network-architecture',
title: 'VMware Cloud on AWS: Network Architecture | VMware',
description: 'Overview The networking architecture of a VMware Cloud on AWS Software-Defined Data Center (SDDC) is arguably one of the most',
language: 'en',
loc: [Object]
}
},
Document {
pageContent: 'A screenshot of a computer Description automatically generated\n' +
'[data:image/gif;base64,R0lGODlhAQABAAAAACH5BAEKAAEALAAAAAABAAEAAAICTAEAOw==]\n' +
'\n' +
'For more information about the AWS Managed Prefix List Mode, please refer to Understanding Managed Prefix List Mode for Connected\n' +
'VPC in VMC on AWS [https://vmc.techzone.vmware.com/understanding-managed-prefix-list-mode-connected-vpc-vmc-aws].\n' +
'\n' +
'\n' +
'EDGE UPLINKS\n' +
'\n' +
'There are currently three uplinks from the tier-0 routers of the SDDC. These are described below.\n' +
'\n' +
'\n' +
'INTERNET UPLINK\n' +
'\n' +
'The internet uplink is connected to AWS IGW within the underlay VPC. Since the default from other sources such as Direct Connect,\n' +
'IPSec VPN or VMware Transit Connect (VTC) has a higher priority, the IGW path would be only used when no other default routes are\n' +
'learned.',
metadata: {
source: 'https://vmc.techzone.vmware.com/resource/vmware-cloud-aws-network-architecture',
title: 'VMware Cloud on AWS: Network Architecture | VMware',
description: 'Overview The networking architecture of a VMware Cloud on AWS Software-Defined Data Center (SDDC) is arguably one of the most',
language: 'en',
loc: [Object]
}
},
Document {
pageContent: '[data:image/gif;base64,R0lGODlhAQABAAAAACH5BAEKAAEALAAAAAABAAEAAAICTAEAOw==]\n' +
'\n' +
'An internet gateway [https://docs.aws.amazon.com/vpc/latest/userguide/VPC_Internet_Gateway.html] (IGW) and virtual private gateway\n' +
'[https://aws.amazon.com/directconnect/faqs/] (VGW) will also be created for this VPC. These gateways enable internet and Direct\n' +
"Connect connectivity to the VPC.  It's important to note that customers cannot directly see or manage these AWS components. Any\n" +
'interaction with or configuration of these components must be done through VMware-provided options such as the NSX UI, API, and\n' +
'Cloud Console.\n' +
'\n' +
'\n' +
'THE SDDC UNDERLAY\n' +
'\n' +
'\n' +
'\n' +
'\n' +
'\n' +
'\n' +
'ESXI NETWORKING',
metadata: {
source: 'https://vmc.techzone.vmware.com/resource/vmware-cloud-aws-network-architecture',
title: 'VMware Cloud on AWS: Network Architecture | VMware',
description: 'Overview The networking architecture of a VMware Cloud on AWS Software-Defined Data Center (SDDC) is arguably one of the most',
language: 'en',
loc: [Object]
}
}
]
Cohere Rerank Options:  {
apiKey: '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',
baseUrl: 'http://peanutshell:8000/v1',
model: 'ms-marco-MiniLM-L-6-v2',
topN: 4
}
[nuxt] [request error] [unhandled] [500] Status code: 500
Body: "Internal Server Error"
at CohereClient. (./.output/server/node_modules/cohere-ai/Client.js:481:27)
at Generator.next ()
at fulfilled (./.output/server/node_modules/cohere-ai/Client.js:31:58)
at process.processTicksAndRejections (node:internal/process/task_queues:95:5
Please help.

Looks it's some issue in the peanut-shell API.
Could you please try a couple of things at your end, following https://github.com/sugarforever/peanut-shell?

If the models endpoint works - curl --location 'http://localhost:8000/v1/models'
If the rerank endpoint works:

curl --location 'http://localhost:8000/v1/rerank/' \
--header 'Content-Type: application/json' \
--data '{
    "model": "ms-marco-MiniLM-L-6-v2",
    "query": "What is the capital of the United States?",
    "top_n": 3,
    "documents": [
        "Carson City is the capital city of the American state of Nevada.",
        "The Commonwealth of the Northern Mariana Islands is a group of islands in the Pacific Ocean. Its capital is Saipan.",
        "Washington, D.C. (also known as simply Washington or D.C., and officially as the District of Columbia) is the capital of the United States. It is a federal district.",
        "Capital punishment (the death penalty) has existed in the United States since beforethe United States was a country. As of 2017, capital punishment is legal in 30 of the 50 states."
    ]
}'

Please change the host localhost to your docker set up.
Let me know if you see any issue.


Image update

Hopefully this is all we need to update the icons

Ack https://github.com/Groestlcoin/BlueWallet/pull/1/commits/ab2d0aa7835f01ee5bad303fa233c1694713cd5a


feat: support bun plugin

🔗 Linked issue

close: #329
❓ Type of change

[ ] 📖 Documentation (updates to the documentation, readme, or JSdoc annotations)
[ ] 🐞 Bug fix (a non-breaking change that fixes an issue)
[ ] 👌 Enhancement (improving an existing functionality like performance)
[ ] ✨ New feature (a non-breaking change that adds functionality)
[ ] 🧹 Chore (updates to the build process or auxiliary tools and libraries)
[ ] ⚠ Breaking change (fix or feature that would cause existing functionality to change)

📚 Description

📝 Checklist

[x] I have linked an issue or discussion.
[ ] I have updated the documentation accordingly.

@sxzz would you like to work together on this?
@Dunqing what blockers do you see missing from having this PR ready? If there is limitations on bun, I think we may stated them in the README and get the basic feature shipped first.

@sxzz would you like to work together on this?
@Dunqing what blockers do you see missing from having this PR ready? If there is limitations on bun, I think we may stated them in the README and get the basic feature shipped first.

Currently, we can't run tests using the bunx --bun vitest because vitest doesn't support bun yet. We may need to wait until vitest supports bun
Alternatively, we can put the bun test cases in a separate directory rewrite the bun-related test cases with bun:test and run the tests with bun


Test: Add size mismatch log entry to failed ones.

Fail if size mismatch is found in the logs after executing the test.
Fix #7851
Signed-off-by: Eloy Coto<EMAIL_ADDRESS>

This change is

test-me-please

Coverage decreased (-0.07%) to 41.608% when pulling f562cff5031504d8a6ee39f63b07db18e408708e on eloycoto:issue7851 into a42961f597642365e7c92f9ab955017b033c99b7 on cilium:master.

test-me-please

test-me-please

@aanm why was this merged? The comments weren't addressed.

@aanm why was this merged? The comments weren't addressed.

@ianvernon which comments?

@aanm https://github.com/cilium/cilium/pull/7852#discussion_r278961783


Seller Model

Here is the seller model file

Also, please git pull before making your new changes and resolve every conflicts


Some times right click do not erase the annotations.

Description of the problem
I have a question regarding deleting interactive annotation during "raw.plot()".
When you try to delete the annotation,  a thin line remains...(orange annotation in this case)
I though it would be helpful if I can delete it easily using right click.
Steps to reproduce
matplotlib.use('Qt5Agg')
fig=raw.plot(start=2,duration=15, highpass=.5, lowpass=40)
fig.fake_keypress("a")

open the eeg using this code and annotate some data and try to erase it.

Link to data
No response
Expected results
the annotations are deleted(orange line disappears)
Actual results

Additional information
Platform:         Windows-10-10.0.19045-SP0
Python:           3.9.13 (main, Aug 25 2022, 23:51:50) [MSC v.1916 64 bit (AMD64)]
Executable:       [c:\Users\ADaida\Anaconda3\python.exe](file:///C:/Users/ADaida/Anaconda3/python.exe)
CPU:              Intel64 Family 6 Model 165 Stepping 5, GenuineIntel: 20 cores
Memory:           63.7 GB
mne:              1.3.0
numpy:            1.21.6 {OpenBLAS 0.3.17 with 20 threads}
scipy:            1.9.1
matplotlib:       3.5.2 {backend=Qt5Agg}
sklearn:          1.0.2
numba:            0.55.1
nibabel:          Not found
nilearn:          Not found
dipy:             Not found
openmeeg:         Not found
cupy:             Not found
pandas:           1.4.4
pyvista:          Not found
pyvistaqt:        Not found
ipyvtklink:       Not found
vtk:              Not found
qtpy:             2.2.0 {PyQt5=5.9.7}
ipympl:           Not found
pyqtgraph:        Not found
pooch:            [c:\Users\ADaida\Anaconda3\lib\site-packages\paramiko\transport.py:219](file:///C:/Users/ADaida/Anaconda3/lib/site-packages/paramiko/transport.py:219): CryptographyDeprecationWarning: Blowfish has been deprecated
"class": algorithms.Blowfish,
v1.6.0
mne_bids:         Not found
mne_nirs:         Not found
mne_features:     Not found
mne_qt_browser:   Not found
mne_connectivity: 0.5.0
mne_icalabel:     Not found

This issue has already been fixed in #11491.


PulseAudio support in ALSA by default

I just realised that I don’t have sound in packages installed with Nix which are built without explicit PulseAudio support. The problem is that alsaLib that they use doesn’t know anything about my system ALSA config that tells it to use PulseAudio. Furthermore, even if it knew, it woudn’t be able to load the PulseAudio module, because it knows nothing about alsaPlugins.
I don’t see any easy way out of this situation (except for switching to NixOS, of course), so here is the proposal: ship PulseAudio support in alsaLib by default, including the plugin and configuration that uses PulseAudio when it is available. It won’t break anything, since if there is no PulseAudio, the extra config will just do nothing. If there are space constraints, this behaviour can be optional (e.g. enabled by default, disabled for minimal builds).
CC maintainers: @MarcWeber @lovek323 @wkennington

This is not PulseAudio specific but applies to all alsa plugins. see https://github.com/NixOS/nixpkgs/issues/6860

(triage) maybe it’s already implemented?


[Feature Request]:  Optimising the navbar styling

Is there an existing issue for this?

[X] I have searched the existing issues

Feature Description
Hello @apu52 , I would like to optimise the navbar , change the styling and icons to it .
Kindly assign me this issue under GSSoC'24. I''ll do my best.
Use Case
This feature would increase the engagement of website and enhance the look of website.
Benefits
No response
Add ScreenShots
I would style navbar by adding icons similar to below screenshot but on hover they will display contents name such as home, about so that navbar can be made more spacious.

Priority
High
Record

[X] I have read the Contributing Guidelines
[X] I'm a GSSOC'24 contributor
[ ] I'm a VSOC'24 contributor
[X] I have starred the repository

Hey @Dilpreet0501, there is already an issues present related to this issues check #1129

@Dilpreet0501 there is some new changes in the navbar , after that changes can u implement the features u are asking in this issue , so that I will reopen this issue

Sure Sir @apu52 , I will do it.

Hello @apu52 , Kindly assign me this issue so I can update the navbar styling


Consider adding Dataset.filter

I discovered pandas.DataFrame.filter today, which is a surprisingly convenient way to filter columns with the like and regex arguments. It might make sense to add this for xarray.Dataset, too.
Note that the first argument of DataFrame.filter, items, is redundant with indexing.

Repeating what I said in #883:

I'm thinking a signature something like Dataset.filter(func=None, like=None, regex=None) (see here for how filter is implemented in pandas). This does have the downside that it departs from pandas syntax by combining filter and select, but these are very related ways of filtering a dataset so it seems better to keep them in one method.


UofT-DSI | Production - Assignment 1

What changes are you trying to make? (e.g. Adding or removing code, refactoring existing code, adding reports)

completion of hw assignment 1

What did you learn from the changes you have made?

dask is very important to use for large datasets, otherwise performance issues are prevalent

Was there another approach you were thinking about making? If so, what approach(es) were you thinking of?

access to cloud computing can help speed up the process, I just do not have access to a system

Were there any challenges? If so, what issue(s) did you face? How did you overcome it?

setting up the file to read parquet files was difficult, I was not correctly listing the file path to do this

How were these changes tested?

followed class notes and compared the performance

A reference to a related issue in your repository (if applicable)

n/a

Checklist

[ ] I can confirm that my changes are working as intended

A1 complete


Migrating to dropwizard 0.8+

Hi Stan,
I want to use this library. However it looks like the structure of classes has changed and this does not compiles. Is there a way you can help me fix this library (or if you have some spare cycles).
Thanks
Amit

Hi Amit,
I will start the migration to 0.8 version this week once I have more time. However I will probably change the approach to make it more flexible and this will require some effort. But I will keep the support for MVEL. JAX-RS 2.0 filters could be good way how to achieve this. Let me investigate on this.

Hey Thanks. I also feel your approach with MVEL is a good one. I saw this also: https://github.com/mathiasringhof/dropwizard-authrole but your approach allows using the familiar @Auth and allows applying complex role combinations.
Will wait for your upgrade.

Hi Stan, Did you get a chance to take a look at this?

Hi Amit, I am in the middle of that change. I need to solve many other things last days, but I think you may expect the first result soon. I am sorry for waiting.

Please don't say sorry. I just wanted to ask. I'll wait. Quite eager to use this code actually. As this is new I cannot do this myself else I would have.

Hi Amit, I was crazy busy last weeks so it took more time than I expected. You can try 0.1.1 testing version. Please see the readme file for the instructions. However it doesn't support MVEL expression authorization yet. I am going to release it as separated package ASAP.

Hi Stan,
I tried this but for some reason I am getting a host of errors that do not make sense to me.
! org.glassfish.jersey.server.model.ModelValidationException: Validation of the application resource model has failed during application initialization.
! [[FATAL] No injection source found for a parameter of type public com.aquevix.bazinga.models.Page com.aquevix.bazinga.resources.PageResource.update(com.aquevix.auth.models.User,io.dropwizard.jersey.params.UUIDParam,com.aquevix.bazinga.models.Page,io.dropwizard.jersey.params.UUIDParam,io.dropwizard.jersey.params.UUIDParam) at index 0.; source='ResourceMethod{httpMethod=PUT, consumedTypes=[application/json], producedTypes=[application/json], suspended=false, suspendTimeout=0, suspendTimeoutUnit=MILLISECONDS, invocable=Invocable{handler=ClassBasedMethodHandler{handlerClass=class com.aquevix.bazinga.resources.PageResource, handlerConstructors=[org.glassfish.jersey.server.model.HandlerConstructor@1c00d406]}, definitionMethod=public com.aquevix.bazinga.models.Page com.aquevix.bazinga.resources.PageResource.update(com.aquevix.auth.models.User,io.dropwizard.jersey.params.UUIDParam,com.aquevix.bazinga.models.Page,io.dropwizard.jersey.params.UUIDParam,io.dropwizard.jersey.params.UUIDParam),

This started happening after I added this:
        AuthorizationConfiguration authConfig = new AuthorizationConfiguration.Builder<User>()
                .setPolicy(ProtectionPolicy.PROTECT_ANNOTATED_ONLY)
                .addRole(new SecurityRole.USER())
                .addRole(new SecurityRole.ADMIN())
                .addRole(new SecurityRole.MANAGER())
                .addRole(new SecurityRole.SUPERADMIN())
                .setAuthentication(new JWTAuthFactory<>(new JWTAuthenticator(), "my-api", User.class, tokenVerifier, tokenParser))
                .build();
        environment.jersey().register(authConfig);  // <<=== THIS LINE CAUSES ERRORS.

Does this makes any sense to you?
Without this the application runs perfectly fine.

Hi Amit, that's unfortunate. Could you please share signature of method com.aquevix.bazinga.resources.PageResource.update(*) to see how this method and its parameters are annotated.

Also which Dropwizard version do you use pls?

Hi Amit, thank you for the reporting. I probably found the problem which may be related to your finding. You can see the failing test is here: https://github.com/StanSvec/dropwizard-authorization/blob/master/src/test/groovy/com/stansvec/dropwizard/auth/authorization/AuthorizationWithEntityTest.groovy
It looks like when @Auth annotation is used on the method level then injection resolver registered for this annotation is actually used for all method parameters causing the injection of custom entities failing. I afraid that only possible solution for this issue is to have two separated annotations: one for authentication / authorization and second one for principal instance injection. But let me double-check first. In that case I would probably use the same approach as planned for Dropwizard 0.9 and use SecurityContext for principal registration. That would unfortunately force the extension users to implement Java EE Principal interface for their principal objects.
Thanks again for cooperation. I haven't had an opportunity to use this extension in my own projects yet so it's good you already made some tests.

Hi Amit,
It should be fixed. If you need to inject principal instance into your resource methods please use @Principal annotation as described in the documentation. You can also already use MVEL expressions:  https://github.com/StanSvec/dropwizard-authorization-mvel.

Hey, Sorry I was out and could not reply sooner. Let me try this over the weekend.
I am currently using latest 0.8.x dropwizard.
Here's the signature:
    @PUT
    @Path("{id}")
    @Consumes(MediaType.APPLICATION_JSON)
    @Produces(MediaType.APPLICATION_JSON)
    @Timed
    @UnitOfWork
    @ApiOperation("Update Single")
    public Page update(@Auth(required = true) User user, @PathParam("id") UUIDParam id, Page entity,
                       @NotNull @PathParam("prj") UUIDParam projectId,
                       @NotNull @PathParam("domain") UUIDParam domainId) {

This method should work even with the old 0.1.1 implementation. I don't know what could be wrong in that case.
What if you use dropwizard-authentication module only, i.e. use Dropwizard @Auth annotation on that User parameter and register corresponding AuthFactory.binder(..) in Jersey. Do you experience the same error or does this fix it?
When you try the new 0.1.4 version of the authorization module you need to change your method like this:
    @PUT
    @Path("{id}")
    @Consumes(MediaType.APPLICATION_JSON)
    @Produces(MediaType.APPLICATION_JSON)
    @Timed
    @UnitOfWork
    @ApiOperation("Update Single")
    @Auth
    public Page update(@Principal User user, @PathParam("id") UUIDParam id, Page entity,
                       @NotNull @PathParam("prj") UUIDParam projectId,
                       @NotNull @PathParam("domain") UUIDParam domainId) {

If you have more methods like this in your resource class you can put @Auth annotation on class level.
But as I said I'm not convinced that the new version fix it. Please try and let me know.

The work is done. In case of any issue pls raise separate issue ticket.


ValueError: Dimensions must be equal

ValueError: Dimensions must be equal, but are 14 and 13 for 'position__embedding_1/add' (op: 'Add') with input shapes: [?,5,14], [?,5,13].

Please fill out the issue template, or your issue will be closed


fix(types): fix path to package types

trying to import this plugin results in the error
import htmlPlugin from "vite-plugin-html-config";
// TS2307: Cannot find module vite-plugin-html-config or its corresponding type declarations.

This PR fixes the path to package types

The new version has been released


ng-focus and ng-blur would be a cool feature

It would be really cool if ng-focus and ng-blur on a  element would work.

+1

I'd like to, but it's not something that should depend on Chosen by itself?
There's an open Issue there https://github.com/harvesthq/chosen/issues/1428

Hi @leocaseiro, on harvesthq/chosen#1428, @ababba15 mentioned a possible solution to the problem on April 3, 2016.
Unfortunately, I'm not a JavaScript developer but I would like to ask whether his suggested way of addressing the problem would be practical to be used for angular-chosen to support ng-focus and ng-blur... What do you think of it?

Hi @staubesv, thanks for your link and the update.
I've read his code and it's a little bit hack.
I'm not surprised he didn't send a Pull Request and I wouldn't be surprised if the Chosen team wouldn't accept his changes. They are quite rigorous and understand that. A library with so many users needs to be really tested and well maintained.
Either way, it's a solution that might help.


kernel requirements for RHEL/CentOS

File: engine/userguide/storagedriver/overlayfs-driver.md, CC @mstanleyjones
I think it is worth mentioning that modern RHEL/CentOS kernels support overlay2, but you need to override the kernel check (https://github.com/docker/docker/issues/24023). Took me about a day to figure that out :)

Docker does not support overlayfs on RHEL / CentOS, and we cannot document unsupported configurations. These distributions are supported on device-mapper: https://docs.docker.com/engine/userguide/storagedriver/device-mapper-driver/#configure-docker-with-devicemapper


Issue: 3529509: Remove echo form ufm-epilog.sh and ufm-prolog.sh files.

Issue: 3529515: fix permission of ufm_slurm.conf file

bot:retest


Just one more link in README.md, for future readers 😆

@marciojg 👍 👊

Codecov Report

Merging #11 (87071f5) into master (aa3c207) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##            master       #11   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files            3         3
  Lines          205       205
=========================================
  Hits           205       205

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 aa3c207...87071f5. Read the comment docs.


Link to tutorial video is broken

The player widget says that there is No video with supported format and MIME type found. Might be better to move the video to some publicly available hosting platform like YouTube or Vimeo.

Ah. This seems to be an issue of a missing codec on a vanilla Ubuntu + Firefox installation. Anyway, not working out-of-the-box on a kind of standard system is not a good thing.

Works on vanilla Ubuntu 22.04 w/ third-party repositories.


Spectator notification

If someone joins a game as a spectator, it would be good to have some sort of notification letting them know they are a spectator (and not able to play the game).

This issue is tracking spectator mode.


Header component

Initial work on header component.
This header is done in legacy non-mobile first design so we can get one component shipped. I'll refactor the header to be mobile-first in a subsequent PR.

I've no problem with the code, but on a related note moving forward we should discuss two items:

Font sources. I recommend using Google Fonts API, referenced remotely so we don't have to install fonts into our project - the latter unfortunately requires at least 3 types of font types per font.
Let's decide and use a SASS structure for our CSS so we're not writing CSS by hand. See this note in our Taiga Wiki for a suggested setup: https://taiga.pathfinder.gov.bc.ca/project/lukegonis-developergovbcca/wiki/css-implementation

how is this component going to be injested by the markdown file that is to present this content? Also I'm with @rajpersram on using Google Fonts API if that is allowed. From what I've checked though they don't have Myriad Web Pro as that is a Adobe licensed font?

@patricksimonian The html / css for the commonest are just samples. I'll actually copy the specific CSS and HTML (for the header) to the markdown file that will be (we hope) picked up by Gatsby. I might just delete the samples when done.

Looking good!
A couple things:
Positioning of logo/title - should be more indented. Logo should link back to gov.bc.ca
Logo - switch logo to sunrise with BC beside (not below)
Font - Site title and copy should be in Myriad Pro...I know there is a separate discussion about needing
a licensed font for (supposedly open) government work...
Changing the logo will make it possible to make the header tighter (it's a bit high now).
Visually, the header should resemble this (more/less, no nav) - https://portal.nrs.gov.bc.ca/web/client/home

@ksingbeil The only logo in that format is pretty low quality with the blue background embedded. It would be idea to get the logo in svg format. You can see in the NR link that the logo is blurry. Know where we can get a better copy?
Let's talk about the use of Myriad Pro we have an open and free one that looks almost identical. Its also hosted by google so much faster load times and it can be optimized. I think Myriad Pro might be legacy from printed material.

@ksingbeil btw, the font mentioned is a Google Font called PT Sans.

Thanks @jleach and @rajpersram . We're chatting with the branding folks this week and will raise these points - and find a better logo!

@orinevares Are you able to add your initial work on the harder markdown to this branch / RP? When we have the html / css dialled I'll port it to that file and we can go over it with @patricksimonian to make sure its something he can import.

@jleach no problem, ill upload the file now.

@orinevares @ksingbeil I've updated the header to look more like the NR header.
@rajpersram I've had to hard code some values in size. For example, the heard is fixed at 65px (no media queries in place just yet). And I've also put a 65px left and right border on each side. Your thoughts?
Check out the two images I put in the screenshots folder.

@orinevares FYI to make the header to edge to edge the body needs to have the margins corrected to 0 with <body style="margin: 0;">. This brought up the question of included some thoughts on a reset in the design system.

@jleach Further to our talk about a universal way to reset html tags via css, the original normalize.css (from MIT) was incorporated into Bootstrap and Google Material Design also uses this (with some sass changes). With that in mind, I suggest in general using the BootstrapCDN's reset css, called reboot.css, found here: https://stackpath.bootstrapcdn.com/bootstrap/4.1.3/css/bootstrap-reboot.min.css.

@jleach
Things are looking good, below are a few changes to the header.

The gold line should be a bit thicker, to mimic the NR site
The site title text should be a bit bigger, according to the developer guide that is 42 pts
When the browser window width decreases the site title should disappear instead of the words stacking on top of one another. Also just like the NR site.

The gold like is 2px. I think it looks smaller on our page because of the white below it. I also:
Increased the size to ~35pt to match the NR site;
Tweaked the media queries to hide the text at a certain size;
Cleaned up the blurry logo a bit.

On Oct 19, 2018, at 9:21 AM, Ori Nevares<EMAIL_ADDRESS>wrote:
@jleach https://github.com/jleach
Things are looking good, below are a few changes to the header.
The gold line should be a bit thicker, to mimic the NR site
The site title text should be a bit bigger, according to the developer guide that is 42 pts
When the browser window width decreases the site title should disappear instead of the words stacking on top of one another. Also just like the NR site.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/bcgov/design-system/pull/2#issuecomment-431418886, or mute the thread https://github.com/notifications/unsubscribe-auth/AAX26wBWhz7cEWfGrf_j0ewo2v5XgWl3ks5umfwHgaJpZM4XiPUh.

@orinevares @ksingbeil Are one of you able to approve the PR? There should be a button in the top right to start a review of the changes. Make comments in-line if needed, then approve or request changes. LMK if you need a walk through.

Useful document on the use of the BC Logo -
http://www.corporate.gov.bc.ca/print-ads/Govt_of_BC_Logos/Resources/Third-Party_Use_of_BCMarks.pdf

@jleach just approved it, let me know if there are any issues with it


disable ceph-utils

ceph-utils was designed to adjust ceph for large cluster as whitewater.
We disable it as it doesn't work well with old env which upgraded from
3.0 or less

how to know if an env is old env which upgraded from 3.0 or less?
run following command on a mon node, if the word default is in the output, then it's an old env.

ceph osd lspools

how to disable ceph-utils for existing problem envs?
login to the master mon node and follow below steps:
1.1 edit /etc/cron.d/ceph_utils to comment all lines
1.2 find the osds which have reweight value less than 1.

ceph osd df
ID WEIGHT  REWEIGHT SIZE   USE    AVAIL  %USE VAR  PGS
 1 0.85999  1.00000   883G  1940M   881G 0.21 1.05 178
 2 0.85999  1.00000   883G  1744M   881G 0.19 0.94 152
 3 0.85999  1.00000   883G  1904M   881G 0.21 1.03 172
 4 0.85999  1.00000   883G  1808M   881G 0.20 0.98 171
 5 0.85999  1.00000   883G  1852M   881G 0.20 1.00 173
 0 0.85999  1.00000   883G  1904M   881G 0.21 1.03 180
 6 0.85999  1.00000   883G  1797M   881G 0.20 0.97 169
 7 0.85999  1.00000   883G  1629M   881G 0.18 0.88 154
 8 0.85999  1.00000   883G  2031M   881G 0.22 1.10 178
 9 0.85999  1.00000   883G  1761M   881G 0.19 0.95 165
10 0.85999  0.95001   883G  1884M   881G 0.21 1.02 172
11 0.85999  1.00000   883G  1957M   881G 0.22 1.06 179
12 0.85999  1.00000   883G  1796M   881G 0.20 0.97 169
13 0.85999  0.95001   883G  1921M   881G 0.21 1.04 180
14 0.85999  0.95001   883G  1918M   881G 0.21 1.04 177
15 0.85999  1.00000   883G  1712M   881G 0.19 0.92 153
16 0.85999  1.00000   883G  1934M   881G 0.21 1.04 171
17 0.85999  1.00000   883G  1840M   881G 0.20 0.99 166

set reweight value to 1 for the osds found in step#2
for example, the osds are 10, 13, 14

ceph osd reweight 10 1
ceph osd reweight 13 1
ceph osd reweight 14 1

Can one of the admins verify this patch?

"ok to test" to accept this pull request for testing
"test this please" for a one time test run
"add to whitelist" to add the author to the whitelist

Can one of the admins verify this patch?

"ok to test" to accept this pull request for testing
"test this please" for a one time test run
"add to whitelist" to add the author to the whitelist

Can one of the admins verify this patch?

"ok to test" to accept this pull request for testing
"test this please" for a one time test run
"add to whitelist" to add the author to the whitelist


Default authorizer should allow some OperatorService calls with namespace claims

Current vs Desired Behavior
Currently default authorizer blocks all OperatorService calls to namespace-reader and namespace-writer (namespace-admin is required).
The UI and (at least) Java SDK call ListSearchAttributes as part of normal operation.
Default authorizer should allow  ListSearchAttributes to namespace-reader and Add/RemoveSearchAttributes & DeleteNamespace to namespace-admin.
The remote cluster calls should require system permissions.
Note that AddSearchAttributes on ES currently creates properties at the cluster level, so affects beyond a single namespace. This will be fixed eventually but users should be aware.

Thank you for reporting this issue.  We will investigate this behavior and look into adding the necessary permissions to the default authorizer. For now, you can achieve the desired behavior by customizing the authorizer to allow the necessary operations.
Please let me know if you have any other questions or concerns.

Thank you for reporting this issue. We understand the need to refine the default authorizer behavior for ListSearchAttributes and other operations.
To work around this issue in the meantime, you can grant the necessary permissions to the namespace-reader and namespace-admin roles. For more information about configuring the default authorizer, please refer to the Temporal CLI operator command reference.
We are actively working on improving the authorization model to address these concerns.

Thank you for reporting this issue. We understand the need for more granular access control for namespace-reader and namespace-writer roles. We are working on enhancing the authorization model to allow specific operations like ListSearchAttributes for namespace-reader and Add/RemoveSearchAttributes & DeleteNamespace for namespace-admin.
In the meantime, you can use the namespace-admin role for operations like ListSearchAttributes which are currently blocked for namespace-reader. However, please be aware that this could potentially grant more access than intended.
We will update you on any changes made to the authorization model.


Typo fix on list and recycler views

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


Implement value transforms

This PR implements value transform, available on both Database and Query objects:
transform = ->(h) { MyModel.new(h) }
db.query_transform_hash(transform, 'select * from foo')

transform = ->(a, b, c) { { a: a, b: b, c: JSON.parse(c) } }
db.query_transform_argv(transform, 'select a, b, c from foo')

# transforms on prepared queries
q = db.prepare('select * from foo').transform_hash { |h| MyModel.new(h) }
q.to_a #=> transformed rows

q = db.prepare('select a, b, c from foo').transform_argv { |a, b, c| { a: a, b: b, c: JSON.parse(c) } }
q.to_a #=> transformed rows

Benchmarks show no gain for the transform_hash flavor, but a significant improvement for transform_argv - up to 30% faster than the equivalent map.

That's a fantastic feature. I'll have a closer look at the beginning of next week.


Add architect to tools section of serverless landscape

Architect is a serverless provisioning and deployment tool at the JS Foundation. More info at https://arc.codes

I also added this to the Tools tab of the spreadsheet:

I added Architect to Tools in the yml. https://248-lev-add-arc-to-tools--interactive-landscape.netlify.com/grouping=landscape

Resolved: https://landscape.cncf.io/selected=architect

Thanks!


Allow setting of innerHTML and textContent properties

Setting innerHTML and textContent for an Element should be supported. To support this the following needs to be done:

Instead of calling setAttribute/removeAttribute for primitive (non object/function) values, check if it should be set using the property instead
For Elements that have innerHTML or textContent set, do not sweep out children on exit
Add assert to enforce elementOpen, etc. are not called inside an Element with innerHTML or textContent
Optionally, add semantically named attribute that behaves like textContent = '', allowing for Elements to act as insertion points for non-library managed DOM

Hi, sorry for resuscitating this, but what was the solution to set the innerHTML directly? Right now I'm using currentPointer()/skipNode() and manually changing the innerHTML of the element when it changes.

Hi, sorry for resuscitating this, but what was the solution to set the innerHTML directly? Right now I'm using currentPointer()/skipNode() and manually changing the innerHTML of the element only when it's supposed to change.

This is the right approach at the moment.


fix(robot-server): Home pipettes after calibration flows

Right now, when we finish calibration sessions, we leave the pipettes right above the tiprack they just returned a tip to, which is pretty annoying if you do another flow right after and want to swap out the tiprack. We should home the system to get the pipettes out of there.

@Laura-Danielle should we add this to cal check during the refactor?


Support for ONNX external data

It would be great if WinML added support for loading models with external data, currently creating a session with a model that has external data results in this error:
onnxruntime::Initializer::Initializer !model_path.IsEmpty() was false. model_path must not be empty. Ensure that a path is provided when the model is created or loaded.

this happens even if the model was loaded through LearningModel.LoadFromFilePath, this is because WinML always uses a stream to initialize the onnx runtime even if it was asked to use a path.

Thanks for posting. We'll look into this for the future.

Thanks for replying, I'm curious why the issue was closed as completed tho, is it moved to the internal tracker instead?

Yes, we are tracking this internally


Update AsArrayObject.php to use ARRAY_AS_PROPS flag

Added ARRAY_AS_PROPS flag to allow using array as an object
protected $casts = [
    'options' => AsArrayObject::class,
];

$model->options->something = 'whatever';
$model->options->something_else;

Before:
WARNING  Undefined property: Illuminate\Database\Eloquent\Casts\ArrayObject::$something
WARNING  Undefined property: Illuminate\Database\Eloquent\Casts\ArrayObject::$something_else
Now:
it just works
https://www.php.net/manual/en/class.arrayobject.php#arrayobject.constants.array-as-props
More info https://stackoverflow.com/questions/14610307/spl-arrayobject-arrayobjectstd-prop-list/16619183#16619183

Can you please add test case for it?


Do not allow user to clear selection in terra-form-select

Issue Description
The new terra-form-select always includes a "-Select-" option that lets the user clear the selection.  We need a way for a consumer of the control to turn off this functionality, as it allows the user to leave a valid state and go to an invalid state.
(This was possible in the old terra-form/select.)
Issue Type

[ ] New Feature
[X] Enhancement
[ ] Bug
[ ] Other

Expected Behavior
Allow the consumer of terra-form-select to indicate that the user is not allowed to clear a selection once it has been set/made.
Current Behavior
The terra-form-select control always includes a "-Select-" option that lets the user clear the selection.

It might make sense to allow the - Select - option to be removed or set as a different value, etc. Here's an example of how we'd like to use it. We default to "All" so including a Select option doesn't make much sense.

Tech Design
Summary
The Select component will now receive a new prop 'isClearSelectionHidden'. This prop will determine if the clear option- Select - should be visible.
React Props:
Select

Prop
Type
Default
Description

isClearSelectionHidden
boolean
false
Indicates whether the clear option should be visible

Examples
import React from 'react';
import Select from 'terra-form-select';

<Select
  name="foo"
  defaultValue="b"
  isClearSelectionHidden
>
  <Select.Option value="m" display="moo" key="m" />
  <Select.Option value="b" display="boo" key="b" disabled />
  <Select.Option value="z" display="zar" key="z" />
</Select>

In case when isInvalid and isClearSelectionHidden both are true for the Select component, then the default option will be a blank option.

+1 on tech design. I wish I could think of a better name for the isClearSelectionHidden prop.

Maybe some variation of:
isPlaceholderDisabled, allowBlankSelection, isNullValueDisabled, allowInvalidSelection, removeNullValue, isNullValueHidden, isDefaultValueHidden

@bjankord @dkasper-was-taken changed the prop to isPlaceholderDisabled

How about isPlaceholderHidden?

+1

Resolved in #1229

JIRA created


fix(NFT Standard): add doc strings to macros so generated methods have docs

Currently copy pasting them, but perhaps both methods could use a macro to include an external string.

If users use the macro to generate the methods then they won't get them doc
string when hovering over one in their IDE.
On Wed, Feb 16, 2022, 20:33 Austin Abell @.***> wrote:

@.**** requested changes on this pull request.
Rust docs are added to impls by default unless overridden, why the need
for duplicate docs? I don't see anything different here
—
Reply to this email directly, view it on GitHub
https://github.com/near/near-sdk-rs/pull/734#pullrequestreview-885331132,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AALKD3A5VXMK2774NAFSGNDU3RF5RANCNFSM5OSVE7LQ
.
You are receiving this because you authored the thread.Message ID:
@.***>

I would like to not duplicate the comments, which is what I was saying in my description.  It would be nice to be able to inject the comment.


[parsers][m]: parse xls booleans as booleans, not integeres

This PR fixes #230
As discussed in the issues above, xls booleans are parsed as integers - 1 for True, 0 for False. The PR checks if the type of cell ctype is type boolean and forces value to be boolean
includes tests
NOTE: Not sure why tests are failing, seems like they were failing before me - just run tests on master and they are failing as well.

@zelima
Thanks.
Can you please apply this change - https://github.com/frictionlessdata/tabulator-py/commit/9d5ee066967dc580a6ba2e45d934cba4d2f4fbb5 - to this PR?

@roll tests now are passing, thanks

@zelima
Thanks


Doesn't show weekday-and-week-number on journels when I use this format of title.

Doesn't show weekday-and-week-number on journels when I use this format of title.

I tried the newly updated version, the position might be wrong? I would prefer title appear left.

And I found the icon of calendar missed.


Add metapackages section and instructions

Description
Added metapackage section and instructions for installation.
Related Issue
Closes PWA-2398

@keharper I really don't like not being able to run through these steps locally. Damn valet+!


Docs for Croeseid Testnet (v0.8) upgrade

/staging

diff --git a/docs/getting-started/croeseid-testnet-nix.md b/docs/getting-started/croeseid-testnet-nix.md
index 47ef12e..df0659f 100644
--- a/docs/getting-started/croeseid-testnet-nix.md
+++ b/docs/getting-started/croeseid-testnet-nix.md
@@ -20,7 +20,7 @@ $ cachix use crypto-com
 ## Install chain utils for testnet

 ```shell
-$ nix-env -iA chain-utils-testnet -f https://github.com/crypto-com/chain-main/archive/f53418d87b8fede4fcee59078c996ee000eae4d2.tar.gz
+$ nix-env -iA chain-utils-testnet -f https://github.com/crypto-com/chain-main/archive/6c75214a0d4a017b957d5538eef86d125114bdc7.tar.gz

After installation, you'll have these commands in your PATH:
@@ -70,7 +70,7 @@ One of the strengths of nix package manager is you can have multiple isolated in
you want to manage both testnet and mainnet chains or different versions of them on a single machine.
-$ nix build -f https://github.com/crypto-com/chain-main/archive/f53418d87b8fede4fcee59078c996ee000eae4d2.tar.gz chain-utils-testnet -o testnet
+$ nix build -f https://github.com/crypto-com/chain-main/archive/6c75214a0d4a017b957d5538eef86d125114bdc7.tar.gz chain-utils-testnet -o testnet
$ export CHAINHOME=$PWD/testnet_data
$ MONIKER=testnode ./testnet/bin/init-node
$ ./testnet/bin/chain-main start --home $CHAINHOME

Can you add this change too?

nix build -f https://github.com/crypto-com/chain-main/archive/52d537481caf591066a2426d6bd3ef6c6b97e0db.tar.gz chain-utils-testnet -o testnet

Added 👍

/staging

/staging


Rule provider not initialized breaks dependency injection principles

In order to make use of DomainParser the rule provider it consumes must be initialised by calling the Build() method or else an exception is thrown (DomainDataStructure is not available).
In order to call the Build() method in an aspnet context,  some specific, non standard code needs to be executed in the startup class, which, as a side effect, requires that dependencies have to be registered as singleton. When rule providers are more advanced than the ones provided this github repo, for instance when data bases are involved, this may lead to a far from optimal setup.
Why is the Build() method not called by the DomainParser class. That is, rather than throwing an exception when _ruleProvider.GetDomainDataStructure() returns null, why not calling _ruleProvider.Build()?
I offer my help to discuss the design to see if it can be improved for more advance usage scenarios.

Any thoughts on this?

I don't have time to examine the topic in detail right now, I'll take a look at it when I get the chance

We just ran into the same issue when trying to update to the latest version. +1 on this suggestion from @merijndejonge:

Why is the Build() method not called by the DomainParser class. That is, rather than throwing an exception when _ruleProvider.GetDomainDataStructure() returns null, why not calling _ruleProvider.Build()?

I think that it's also confusing that we need to manually initialize the RuleProvider although we only want to use the DomainParser and wouldn't interact at all directly with the RuleProvider.
This now means that a class using the DomainParser needs to add another dependency on the RuleProvider and then initialize it to make the DomainParser work and this only works because the DomainParser will use the same RuleProvider instance due to the singleton config of the dependency injection.
Just letting the DomainParser initialize the RuleProvider itself like @merijndejonge suggested would solve this.

This is how I now solved it. A bit clumsy, but it makes the code more DI-compliant until the library itself is improved:
public class DiDomainParser(IRuleProvider ruleProvider, IDomainNormalizer? domainNormalizer = null)
    : IDomainParser
{
    private readonly DomainParser _domainParser = new(ruleProvider, domainNormalizer);

    public DomainInfo? Parse(string fullyQualifiedDomainName)
    {
        if (ruleProvider.GetDomainDataStructure() == null)
        {
            ruleProvider.BuildAsync().Wait();
        }
        return _domainParser.Parse(fullyQualifiedDomainName);
    }

    public DomainInfo? Parse(Uri fullyQualifiedDomainName)
    {
        if (ruleProvider.GetDomainDataStructure() == null)
        {
            ruleProvider.BuildAsync().Wait();
        }
        return _domainParser.Parse(fullyQualifiedDomainName);
    }

    public bool IsValidDomain(string fullyQualifiedDomainName)
    {
        if (ruleProvider.GetDomainDataStructure() == null)
        {
            ruleProvider.BuildAsync().Wait();
        }

        return _domainParser.IsValidDomain(fullyQualifiedDomainName);
    }
}

It is not possible for you to initialize your logic like I do here?
var ruleProvider = app.Services.GetService<IRuleProvider>();
if (ruleProvider != null)
{
    await ruleProvider.BuildAsync();
}

https://github.com/nager/Nager.PublicSuffixService/blob/main/src/Nager.PublicSuffixService/Program.cs

It requires that a rule provider including dependencies are all registered as singleton services. Why would you want to pose such a strong requirement on a rule provider?

@tinohager First of all thank you for what you've done here. As I see, this is a common problem.
In my opinion, user should not have to dig in the details about any provider object to create a simple DomainParser class as @FlorianHockmann states. Or at least a meaningful error message would also be great, such "DomainDataStructure is not available, IRuleProvider.BuildAsync has to be called in order to call DomainParser.Parse() method."

@merijndejonge @FlorianHockmann @bkarakaya01
If we call the creation logic in the BaseRuleProvider during instantiation, we can react less easily to any problems that may arise.
For example, we lose the option of canceling the process via the CancellationToken. We also cannot ensure that the process was successful.
What would you expect in these cases?
namespace Nager.PublicSuffix.RuleProviders
{
    /// <summary>
    /// BaseRuleProvider
    /// </summary>
    public abstract class BaseRuleProvider : IRuleProvider
    {
        /// <summary>
        /// Parsed public suffix list rules
        /// </summary>
        protected DomainDataStructure? _domainDataStructure;

        public BaseRuleProvider()
        {
            this.BuildAsync(ignoreCache: false).ConfigureAwait(false).GetAwaiter().GetResult();
        }

I'm aware of the situation here, but there should be a way to force the developer to call the related methods and ensure that build was success. Maybe we should discuss on this.

I would, in general, not do complex things in a constructor. I think that is against good coding practice.
If there really is a need for an explicit Build method, it should be part of a contract that belongs to the rule provider and only of concern to parties which directly interact with the rule provider. The obvious (and perhaps only) party that is using a rule provider is the domain parser. So, this guy should be able to understand the contract of the rule provider. That is why I proposed to let the domain parser make the call to the build method. I still recommend this approach.
As a result, users of the domain parser don't need to interact directly with the rule provider. Dealing with cancelation tokens and/or error handling can also be handled by the domain parser.

This is how I now solved it. A bit clumsy, but it makes the code more DI-compliant until the library itself is improved:
public class DiDomainParser(IRuleProvider ruleProvider, IDomainNormalizer? domainNormalizer = null)
    : IDomainParser
{
    private readonly DomainParser _domainParser = new(ruleProvider, domainNormalizer);

    public DomainInfo? Parse(string fullyQualifiedDomainName)
    {
        if (ruleProvider.GetDomainDataStructure() == null)
        {
            ruleProvider.BuildAsync().Wait();
        }
        return _domainParser.Parse(fullyQualifiedDomainName);
    }

    public DomainInfo? Parse(Uri fullyQualifiedDomainName)
    {
        if (ruleProvider.GetDomainDataStructure() == null)
        {
            ruleProvider.BuildAsync().Wait();
        }
        return _domainParser.Parse(fullyQualifiedDomainName);
    }

    public bool IsValidDomain(string fullyQualifiedDomainName)
    {
        if (ruleProvider.GetDomainDataStructure() == null)
        {
            ruleProvider.BuildAsync().Wait();
        }

        return _domainParser.IsValidDomain(fullyQualifiedDomainName);
    }
}

This works beautifully! Saved my day! Thanks!


Update build-first-app-overview.md

Get Started revamp - overview

Docs Build status updates of commit 4a38081:
:warning: Validation status: warnings

File
Status
Preview URL
Details

msteams-platform/build-your-first-app/build-first-app-overview.md
:warning:Warning
View
Details

msteams-platform/build-your-first-app/build-and-run.md
:warning:Warning
View
Details

msteams-platform/build-your-first-app/build-channel-tab.md
:warning:Warning
View
Details

msteams-platform/build-your-first-app/build-first-app-overview.md

Line 28, Column 5: [Warning-file-not-found] Invalid file link: '../overview'.
Line 54, Column 1: [Warning-file-not-found] Invalid file link: '../assets/images/build-your-first-app/your-app-in-cloud.png'.
Line 5, Column 12: [Suggestion-ms-author-invalid] Invalid value for 'ms.author', 'heath-hamilton' is not a valid Microsoft alias.

msteams-platform/build-your-first-app/build-and-run.md

Line 52, Column 173: [Warning-bookmark-not-found] Cannot find bookmark '#set-up-your-development-account' in 'build-your-first-app/build-first-app-overview.md'.

msteams-platform/build-your-first-app/build-channel-tab.md

Line 159, Column 160: [Warning-bookmark-not-found] Cannot find bookmark '#set-up-your-development-account' in 'build-your-first-app/build-first-app-overview.md'.

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
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.
Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Docs Build status updates of commit 641c808:
:warning: Validation status: warnings

File
Status
Preview URL
Details

msteams-platform/build-your-first-app/build-first-app-overview.md
:warning:Warning
View
Details

msteams-platform/build-your-first-app/build-and-run.md
:warning:Warning
View
Details

msteams-platform/build-your-first-app/build-channel-tab.md
:warning:Warning
View
Details

msteams-platform/build-your-first-app/build-first-app-overview.md

Line 54, Column 1: [Warning-file-not-found] Invalid file link: '../assets/images/build-your-first-app/your-app-in-cloud.png'.

msteams-platform/build-your-first-app/build-and-run.md

Line 52, Column 173: [Warning-bookmark-not-found] Cannot find bookmark '#set-up-your-development-account' in 'build-your-first-app/build-first-app-overview.md'.

msteams-platform/build-your-first-app/build-channel-tab.md

Line 159, Column 160: [Warning-bookmark-not-found] Cannot find bookmark '#set-up-your-development-account' in 'build-your-first-app/build-first-app-overview.md'.

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
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.
Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Docs Build status updates of commit 19fa784:
:warning: Validation status: warnings

File
Status
Preview URL
Details

msteams-platform/build-your-first-app/build-channel-tab.md
:warning:Warning
View
Details

msteams-platform/build-your-first-app/build-and-run.md
:warning:Warning
View
Details

msteams-platform/assets/images/build-your-first-app/your-app-in-cloud.png
:white_check_mark:Succeeded
View

msteams-platform/build-your-first-app/build-first-app-overview.md
:white_check_mark:Succeeded
View

msteams-platform/build-your-first-app/build-channel-tab.md

Line 159, Column 160: [Warning-bookmark-not-found] Cannot find bookmark '#set-up-your-development-account' in 'build-your-first-app/build-first-app-overview.md'.

msteams-platform/build-your-first-app/build-and-run.md

Line 52, Column 173: [Warning-bookmark-not-found] Cannot find bookmark '#set-up-your-development-account' in 'build-your-first-app/build-first-app-overview.md'.

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
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.
Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Docs Build status updates of commit bd51330:
:x: Validation status: errors
Please follow instructions here which may help to resolve issue.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
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.
Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Docs Build status updates of commit 9387984:
:x: Validation status: errors
Please follow instructions here which may help to resolve issue.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
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.
Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Docs Build status updates of commit 3e52178:
:warning: Validation status: warnings

File
Status
Preview URL
Details

msteams-platform/build-your-first-app/build-channel-tab.md
:warning:Warning
View
Details

msteams-platform/build-your-first-app/build-and-run.md
:warning:Warning
View
Details

msteams-platform/assets/images/build-your-first-app/your-app-in-cloud.png
:white_check_mark:Succeeded
View

msteams-platform/build-your-first-app/build-first-app-overview.md
:white_check_mark:Succeeded
View

msteams-platform/build-your-first-app/build-channel-tab.md

Line 159, Column 160: [Warning-bookmark-not-found] Cannot find bookmark '#set-up-your-development-account' in 'build-your-first-app/build-first-app-overview.md'.

msteams-platform/build-your-first-app/build-and-run.md

Line 52, Column 173: [Warning-bookmark-not-found] Cannot find bookmark '#set-up-your-development-account' in 'build-your-first-app/build-first-app-overview.md'.

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
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.
Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Closing this branch as I merged the changes into hello-world branch


Build with swift instead of xcodebuild

Change the build script to build with swift instead of xcodebuild.
This is part of #19. Currently aperture can't be build on linux (so maybe this is part of #7 as well 🤔 but haven't looked in detail inside the PR) which means it can't build in Docker as well.
swift is available on multiple operating systems (e.g. linux) but xcodebuild not.
So changing the build script to build with swift you only have to install swift on your given OS and you are fine to build aperture 🎉
Probably you want to modify the Package.swift to fits your needs. But I should that would be done later by you :)

😞  I found out that

you only have to install swift on your given OS and you are fine to build aperture
isn't true 😢

Because on Linux system (or Dockers) I got the message

aperture.js/swift/aperture/Recorder.swift:1:8: error: no such module 'AVFoundation'
Because AVFoundation isn't available on Linux 😞

Anyway. It would be great if we can merge it because it is one step closer to support other operating systems 👌

Wooo @StefMa, amazing! I tried that when I first started, but I never managed to get it working. Will test ASAP 🙌

I don't mind using swift directly to compile, but I don't understand the argument about Linux. The Swift code is highly coupled to macOS and will never run on Linux.

I thought when we compile with swift (because it is available for linux as well) instead of xcodebuild it will be possible to compile the code on linux environments.
But it can't build on linux because of the AVFoundation 😞

The idea behind this was to create a workaround for #19.

@StefMa You're overthinking it. All that's needed for #19 is to switch the compilation from postinstall to prepublish.

Oh, really?
Make sense 😁
Anyway, have now some experience with swift and xcodebuild 🤣
If you like to merge it feel free. Otherwise just close this PR...
On Jan 22, 2017 7:11 PM, "Sindre Sorhus"<EMAIL_ADDRESS>wrote:

The idea behind this was to create a workaround for #19
https://github.com/wulkano/aperture.js/issues/19.
@StefMa https://github.com/StefMa You're overthinking it. All that's
needed for #19 https://github.com/wulkano/aperture.js/issues/19 is to
switch the compilation from postinstall to prepublish.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/wulkano/aperture.js/pull/21#issuecomment-274347790,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AJwYe4y8_Ia_4nw7Ru75DA3HeLdi3nhYks5rU5vjgaJpZM4LqWgM
.

Actually, I don't see enough benefit in this change. It's safer to use xcodebuild as it follows the build settings in the Xcode project file. This is also incorrect as the binary needs to include Swift itself, as the Swift ABI is not stable. xcodebuild does this automatically.

Closed because the PR make no sense 😹


I can't enable TAlib in Windows 10 bash

I installed it and everything but I can't get it to work, every time I initiate it, it says TAlib disabled, I am starting node gekko --ui from bash.
Is there something I have to change in order to get it enabled? I already followed the tutorial on installing gekko on w10 with bash.
Thanks.

ping @thegamecat

Try node 7.6 and<EMAIL_ADDRESS>I have a strange memory that 1.0.3 doesn't work in bash but could be wrong.

closing this for inactivity!


Improve benchmark for distributed tree

Changes:

Properly align output,
Only print information on MPI rank 0,
Print relevant run parameters.
Replace overlap by shift such that values different from one and zero have a more intuitive meaning.

@masterleinad Can you please post the output before and after?

before:
$ mpiexec -np 2 ./ArborX_DistributedTree.exe
ArborX version: 0.9 (dev)
ArborX hash   : 0b73981
ArborX version: 0.9 (dev)
ArborX hash   : 0b73981
contruction done
knn done
radius done
========================================

TimeMonitor results over 2 processors
Timer Name	MinOverProcs	MeanOverProcs	MaxOverProcs
----------------------------------------
construction	0.0406743	0.0406757	0.0406771
knn	1.03524	1.03534	1.03543
radius	0.57317	0.573342	0.573514
========================================

after:
$ mpiexec -np 2 ./ArborX_DistributedTree.exe
ArborX version: 0.9 (dev)
ArborX hash   : 0b73981

Running with arguments:
perform knn search      : true
perform radius search   : true
#points/MPI process     : 50000
#queries/MPI process    : 20000
size of shift           : 1
dimension               : 3

contruction done
knn done
radius done
==========================================================

TimeMonitor results over 2 processors
Timer Name   | MinOverProcs | MeanOverProcs | MaxOverProcs
----------------------------------------------------------
construction | 4.230867e-02 |  4.230957e-02 | 4.231046e-02
knn          | 1.042842e+00 |  1.043093e+00 | 1.043345e+00
radius       | 5.693861e-01 |  5.695536e-01 | 5.697212e-01
==========================================================

Also the name of the PR is misleading.  Please, at the very least, edit the description and make a list or an enumeration to emphasize the changes for controlling the overlap between regions.

BTW https://github.com/arborx/ArborX/pull/78#issuecomment-500978403 demonstrates something interesting I had already ran into.  The hash returned by the runtime is garbage if you do not reconfigure.

Merge when you are ready.  You may chose to ignore my last comment.  I would be fine with you adding the Arguments class you introduced in #83 before we merge.


Version 7 no artisan command

In the tutorial it is stated to run
php artisan datatables:make UsersDataTable
but running this results in artisan saying there is n namespace for datatables.

Code snippet of problem

System details

Operating System - Windows 10 with IIS
PHP Version - 7
Laravel Version -5.4
Laravel-Datatables Version - 7

Please see upgrade guide. Thanks!

Jim Elliott
I am following the tutorial for using it as a service
On 18/02/2017 13:48, Arjay Angeles wrote:

Please see upgrade guide
https://yajrabox.com/docs/laravel-datatables/7.0/upgrade. Thanks!
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/yajra/laravel-datatables/issues/1018#issuecomment-280846650,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAHhMHH9yHWGGSwpBGH_ergsoNk2zEiJks5rdvasgaJpZM4MFEJn.

Have updated the tutorial and add some note for upgrade guide for Laravel 5.4.
Will try to make a new tutorial for Laravel 5.4 when I had the chance. Thanks!


WiFiC3 - config(ip) didn't set default value for DNS IP

@facchinm this little fix could be merged too. it sets the gateway IP as default DNS IP in the 'automatic' config function


SpleenTweenManager: Cont active tweens

I would not set the active tween count in every frame, but make a public "getter" that returns the .count value.
With this it only gets read, when it really needs to.

Done. Great suggestion thank you


Integration test for Verified registry actor

:link: zboto Link

Closing this issue as it was completed here https://github.com/Zondax/filecoin-solidity-api/pull/59.