Obscuring Mist fix

This adds accurate particle effects for Obscuring Mist.
The existing effects are about 5ft in radius, but the spell creates a 20ft radius effect. I'm not sure how the merging works for this file, so it might not override the old particles. However, if it isn't, it doesn't look to bad to me.
I have another suggestion for the spell which I can add to the PR. The rules file could be changed so that the targeting acts more like Bless. You can only target yourself, but you get to see the radius before the spell is cast. Right now it immediately casts as soon as you select it from the radial, because it's set up as a personal spell.
I wasn't sure what other people would think of the latter change, though, so I haven't included it yet.

I sounds like it'd be better if all personal + range spells behaves this way at the engine level (requiring click to comfirm) rather than manually changing each one. As a bonus, it'd display the radius without having to mouse your character.
Fortunately the picker system is largely replaced already so it shouldn't be too hard to do.

Oh. That'd be really nice, actually.


Setup Ubuntu template missing some ami id's for some regions (Tokyo, etc.)

ISSUE TYPE

Bug Report

COMPONENT NAME
(https://s3.amazonaws.com/big-iq-quickstart-cf-templates/6.0.0/Setup-Ubuntu-Trial.template)
SUMMARY
When trying to launch the CFT in the tokyo region (ap-northeast-1), stack launch fails because unable to map to a tokyo region ami id for ubuntu. This isn't listed in the CFT for Setup-Ubuntu-Trial.template.
"Unable to get mapping for AWSRegionArch2AMI::ap-northeast-1::64"
Tokyo Region ubuntu 16.0.4:
Ubuntu Server 16.04 LTS (HVM), SSD Volume Type - ami-940cdceb
STEPS TO REPRODUCE

Click Launch button here: https://github.com/f5devcentral/f5-big-ip-cloud-edition-trial-quick-start
Attempt to launch in Tokyo region
confirm failure in events.

WORKAROUND ATTEMPTED

Download both nested template "Setup-Ubuntu-Trial.template" and the "bigiq-cm-dcd-pair-with-ssg.template"
Modify the "Setup-Ubuntu-Trial.template" with the ami id for Ubuntu in tokyo region
UPload the modified "Setup-Ubuntu-Trial.template" to S3 and make read only
Modify "bigiq-cm-dcd-pair-with-ssg.template" to point to the location of the modified "Setup-Ubuntu-Trial.template"
Upload stack to AWS and launch

EXPECTED RESULTS
CFT to deploy in Tokyo region
ACTUAL RESULTS
Failed to launch stack
Template error: Unable to get mapping for AWSRegionArch2AMI::ap-northeast-1::64

Fixed in new branch 6.0.1


PR/Branch Name

Is there any format you want for branch names (and/or the PR name itself) when doing a pull request?  If you take a page from openHAB - the PR (and branch) would be named "XXX-YYY" where XXX is the issue ID and YYY would be some descriptive name.  So if I did a pull request based on this issue for an example named 'stuff' - the branch and PR would be named "3-Stuff"

hi @tmrobert8
currently a name is enough for a pr.
thanks


DD-1036 Upgrade DropWizard

Fixes DD-1036
Description of changes

Upgraded to parent pom 7.4.0 and to DropWizard 2.1.1
Moved the health checks to the standard location
Changed dependency to io.dropwizard.modules:dropwizard-health to io.dropwizard:dropwizard-health (no modules) and made necessary code changes.

Notify
@DANS-KNAW/dataversedans

Looks fine after all. After checking the output from the console, it showed this message:
INFO  [2022-08-16 10:06:31,165] [nl.knaw.dans.virusscan.DdWorkflowStepVirusScanApplication.main()] io.dropwizard.health.HealthCheckConfigValidator: The following health check(s) were registered, but are not configured with a schedule:
  * deadlocks

Which I confused for "these are the only health checks configured" because it scrolled by pretty fast. This is not the case, after some testing I can confirm the health checks are working as expected.


[Suggestion] Show ingredients when pinned

Suggestion
When you pin the Factory Planner window, you miss out on seeing the ingredients. I find it’s useful to know the ingredients when building, so I keep having to unpin the window. It would be fantastic if you could see the ingredients when the window is pinned.

Hi, thanks for the suggestion. This has been suggested before in #180, and I'll definitely add this at some point!

Thank you so much for finally adding this! 💙

You're welcome :)


Add coin divisibility to crypto listings

Coverage decreased (-0.4%) to 26.152% when pulling<PHONE_NUMBER>bc360fbdcb965ed5f5bad8a39bc994 on TS_post_crypto_listings_divisibility into 83381b0beecb030381a60f2f841142bf1cc09102 on master.

@placer14 It wasn't incorrect, but I've changed it from 10e7 to 1e8 to be more clear.


Update Instagram Report Bot2.py

bro why would u put ur password on here

bro msg me my Instagram id @sharma_daks.h
msg me fast i ban someone account you help me?
On Wed, 1 Dec, 2021, 1:19 pm Nick Brady, @.***> wrote:

bro why would u put ur password on here
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/Khanejo/Instagram-mass-reporter/pull/22#issuecomment-983376163,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AVYWNOTM7BTO4UIL7STZKY3UOXHOZANCNFSM5BPZZQQQ
.


Precompute nn ranking for DR datasets

Adding a bigger dataset to the dimensionality reduction task accidentally cost us $1000, because we're computing pairwise distances on each dataset n_methods * n_nn_ranking_metrics times (which is a lot). This PR is a hack -- we precompute the NN ranking matrix, which is costly upfront, but it only happens once. Ideally we could define bespoke metric dependencies, but that sounds like a v2 feature.
cc @rcannood

I agree this is a bit hacky. @michalk8 I think you added this originally during the jamboree, do you have any comments or maybe other suggestions for how this could be sped up?

I agree this is a bit hacky. @michalk8 I think you added this originally during the jamboree, do you have any comments or maybe other suggestions for how this could be sped up?

The simplest would be to update the update data on `figsharee with the pre-computed ranking metrics, if possible.

I'd rather precompute it actively once per run than have it precomputed in the cloud -- better at least to have it be transparent about how it's calculated. In v2 we will be able to compose metrics of multiple components and pass data directly from the dataset to the metric (bypassing the method) which solves this problem in a much cleaner way.

In terms of how this could be properly sped up, it seems to me that most of these metrics don't actually need the full knn matrix computed. The big problem here is the memory cost of storing the full nxn dense matrix -- if we're only evaluating most of these metrics with _K = 30, without understanding the metrics completely I imagine we could probably also compute them with something like the 300 nearest neighbors for each cell rather than the 80000 nearest neighbors.

Tests passing at https://tower.nf/orgs/openproblems-bio/workspaces/openproblems-bio/watch/1jOQdpGEMso24E . I propose we merge it this way for now while @michalk8 proposes a more efficient computation so we don't have to subsample datasets for the entire task just for this metric. (One possibility -- just sample datasets before computing this metric??)


Hovering over a Programmatically Pre-Selected disabled date changes Background to Transparent

Describe the bug
When a programmatically pre-selected date is disabled (for example, due to excludeDate), we still have select background on it.

In the above example, eventhough 18th July is disabled as we set the same date programatically, the color highlight is getting applied to it.  But the issue comes when we hover the cursor over that disabled pre-selected date.  It's background color will become transparent, because of the hover style we set to it.

As the sudden background change happens over the disabled day, it'll confuse users.  So, for the disabled days we shouldn't change the background and retain the existing background color even while hover, because anyway user can't select it manually.
Issue Code
() => {
  const [startDate, setStartDate] = useState(new Date());
  return (
    <DatePicker
      selected={startDate}
      onChange={(date) => setStartDate(date)}
      excludeDates={[new Date(), subDays(new Date(), 1)]}
      placeholderText="Select a date other than today or yesterday"
    />
  );
};

Expected behavior
The pre-selected disabled date should retain it's highlighted background color when hovered over.
Screenshots
The background color of the pre-selected disabled date changes to transparent when hovered over.

Hi @martijnrusschen, Please assign this issue to me.  I'll work on it.


Improved the UI of the Home screen and clear the Categories data

Description

Header styling improvement
Fixed the snapshot issues when a new category added
Fixed the sign-in issue by clearing Categories data

Fixes # (issue)
Type of change
Please delete options that are not relevant.

[ x] Bug fix (non-breaking change which fixes an issue)

Change Status

[x ] Complete, but not tested (may need new tests)

Checklist

[ x] There are no merge conflicts

awesome


Global allowlist for codesearch on projects

Feature Description
Due to the size of code index not all repos should be indexed, an admin may wish to only allow certain repos (or namespaces to be indexed).
Perhaps a global setting such as CODESEARCH_ALLOWLIST: org/*, user/project,... and if a repo is not in that list then it wouldn't be indexed. This could later be extended to add options to disregard archived/forked repos, etc..
Screenshots
No response

For our use case at Blender I think we would want to disable it on all forks, and that would be enough. A specific list of repos is not something we would use probably, since keeping that up to date is not so convenient.

For our use case at Blender I think we would want to disable it on all forks, and that would be enough. A specific list of repos is not something we would use probably, since keeping that up to date is not so convenient.

I think just disabling fork repositories is an easier implementation than allowlist, but there are still mirrors and even users in fact could do a localhost fork and push but not from Gitea itself.


new style trigonometric functions

This PR implements new style trigonometric functions.
This PR needs to be rebased after #3603.

Rebased.

LGTM!


how to customize defult profile ingestion  query

i want customize defult profile ingestion  query in open metadata 1.1.7 Because defult profile ingestion query >> blocks our databases
Please advise.

hi @alitehranim can you give us some more details? What do you mean that the profiler queries block the database?

hi pmbrull ,in my  operational database ,when run profile ingestion ,all task in database slow and It will be disturbed .I want to customize the query related to this case to prevent this problem.
Thankful

@pmbrull ,
To write a SQL query in an operational database without blocking, you can use the NOLOCK hint. The NOLOCK hint allows a query to read data without acquiring locks on the tables or rows being accessed. This can improve query performance but also introduces the risk of dirty reads, where the data being read might not be consistent or accurate if it is being modified by another transaction at the same time[8].
SELECT * FROM TableName WITH (NOLOCK) WHERE ...
Replace TableName with the actual name of the table you want to query, and add any necessary conditions in the WHERE clause.
It's important to note that using the NOLOCK hint is not always recommended, as it can lead to inconsistent or inaccurate data. It's advisable to use it only when you understand the potential risks and have determined that the trade-off is acceptable for your specific use case.

hi @pmbrull https://github.com/pmbrull ,in my operational database ,when
run profile ingestion ,all task in database slow and It will be disturbed
.I want to customize the query related to this case to prevent this problem.
To write a SQL query in an operational database without blocking, you can
use the NOLOCK hint. The NOLOCK hint allows a query to read data without
acquiring locks on the tables or rows being accessed. This can improve
query performance but also introduces the risk of dirty reads, where the
data being read might not be consistent or accurate if it is being modified
by another transaction at the same time[8]
https://www.mssqltips.com/sqlservertip/2470/understanding-the-sql-server-nolock-hint/
.
SELECT * FROM TableName WITH (NOLOCK) WHERE ...
Replace TableName with the actual name of the table you want to query, and
add any necessary conditions in the WHERE clause.
It's important to note that using the NOLOCK hint is not always
recommended, as it can lead to inconsistent or inaccurate data. It's
advisable to use it only when you understand the potential risks and have
determined that the trade-off is acceptable for your specific use case.
Thankful
On Fri, Oct 27, 2023 at 11:27 AM Pere Miquel Brull @.***>
wrote:

hi @alitehranim https://github.com/alitehranim can you give us some
more details? What do you mean that the profiler queries block the database?
—
Reply to this email directly, view it on GitHub
https://github.com/open-metadata/OpenMetadata/issues/13685#issuecomment-1782469432,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A75IXMYEUAWE7XGLM3GRH2LYBNSNHAVCNFSM6AAAAAA6L7UKFGVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTOOBSGQ3DSNBTGI
.
You are receiving this because you were mentioned.Message ID:
@.***>


Introduce config map watcher for feature flags

Changes
Taskrun reconciler now uses a configmap watcher for feature-flags
configmap to remove the need for an extra Kubernetes API call to get the
configmap's value every time a pod is being created. Fixes #2117.
Submitter Checklist
These are the criteria that every PR should meet, please check them off as you
review them:

[x] Includes tests (if functionality changed/added)
[ ] Includes docs (if user facing)
[x] Commit messages follow commit message best practices

See the contribution guide for more details.
Double check this list of stuff that's easy to miss:

If you are adding a new binary/image to the cmd dir, please update
the release Task to build and release this image.

Reviewer Notes
If API changes are included, additive changes must be approved by at least two OWNERS and backwards incompatible changes must be approved by more than 50% of the OWNERS, and they must first be added in a backwards compatible way.
Release Notes

/ok-to-test
/kind feature
/cc @sbwsg

I was only trying to understand if we had to duplicate the logic that manages the configmaps.

Oh I see, apologies!  This makes perfect sense to me.  Agree on it being a possible improvement!

There are two new feature-flags added to this configMap in https://github.com/tektoncd/pipeline/pull/2158 and https://github.com/tektoncd/pipeline/pull/2630

There are two new feature-flags added to this configMap in #2158 and #2630

Thank you for the reminder. Fixed.

/retest

/lgtm


Mosquitto installation

Has anyone tried install the Mosquitto? I couldn't manage to get it install keeps saying cannot complete please reinstall.. Does butler have to run with a MQTT application? Any other choice?

What operating system did this happen on?
Installing MQTT has always been 100% smooth for me, on Mac OS and Linux at least. Never tried it on Windows.
You can disable MQTT in Butler SOS by setting the "enableMQTT" parameter in the config file to false.

Thank you mate! I am installing on Windows unfortunately.. Maybe I should try on Linux..
Another silly question.. Without MQTT, what functionality do I loose when using butler?

MQTT is a machine-to-machine protocol, often used in Internet of Things applications. Such as sending a sensor reading from one computer to a server.
Butler SOS uses MQTT as the transport layer for sending the Sense related measurements it collects   to other systems.
For example, Node RED can be used to visually prototype new applications that use the metrics coming from Sense (via Butler SOS) - thanks to both Butler SOS and Node RED talking MQTT.
If you don't have any plans to send the sensor data to other systems, you can safely start off with MQTT disabled.

That's very clear. I will give it a try and see how it goes. Thanks a lot!!


Issues When attaching file with email in sendgrid v3 APIs

I was using this package for sending email with sendgrid. Everything was working fine but some days ago one of my email where I attach an excel sheet started raising errors.

On debugging I find out that it was because of new Behaviour of sendgrid V3 APIs.

The issue I was getting is bad Request error. Complete error is as follows :

{'errors': [{'message': 'The content value must be a string at least one character in length.', 'help': 'http://sendgrid.com/docs/API_Reference/Web_API_v3/Mail/errors.html#message.content.value', 'field': 'content.0.value'}]}

Seems kile sendgrid don't accept content=None anymore.

In sendgrid package file named mail.py class Mail() they accept content kwargs but this package doesn't seems to send content when using Mail() in sgbackend/mail.py _build_sg_mail().

You can also find out the same issue raised here

Can you post code to replicate the issue?  At least I can verify that this will be resolved by either #69 or #55 which are upgrades to the sendgrid dependency.

@jayhale  Below is the code which I am using to send an email :
from django.core.mail import EmailMultiAlternatives

class EmailNotification(BaseNotification):
    """
    Notification class for Email notification
    """

    def __init__(self, destination, message, case, content_object, subject, message_html, attachment_name=None, attachment_type=None, attachment=None):
        super().__init__(destination, message, case, content_object)
        self.attachment = attachment
        self.attachment_name = attachment_name
        self.attachment_type = attachment_type
        self.subject = subject
        self.message_html = message_html

        self.email_message = EmailMultiAlternatives(self.subject, self.message<EMAIL_ADDRESS>                                                    [self.destination])
        if self.attachment:
            self.email_message.attach_file(self.attachment)

        if self.message_html:
            self.email_message.attach_alternative(x
                self.message_html, 'text/html')

    def send(self):
        """Send email to destination
        """
        return self.email_message.send()

@jayhale Today Again I got the same error (this email do not have any attachment with it).
Below are my package info
sendgrid==3.6.5
sendgrid-django==4.0.4

Error tracback :
{'errors': [{'field': 'content.0.value', 'help': 'http://sendgrid.com/docs/API_Reference/Web_API_v3/Mail/errors.html#message.content.value', 'message': 'The content value must be a string at least one character in length.'}]}

Yes I see - the package should raise an error when the content is None or '' before sending.

@jayhale But Howcome earlier the same code was working and even if package raise an error then what content one must send and where (in which variable)?

notification = EmailNotification(
    message='Mail contents', # Any non-zero-length string
    **the_other_args)

Your notification object is being instantiated with a zero-length-string or None in message. That is then being passed to SendGridBackend() which instantiates a Mail() object from the sendgrid-python package which sends it on to the API which replies with an error telling you that the content of the message must be longer than 0 in length.
I don't know if anything changed in the API. This package has used the v3 API for the past year:
https://github.com/elbuo8/sendgrid-django/commit/9cb266812d6beaf063fa26359f12ebc3f71f4b8d

Yes, issue id fixed.
Actually I was using two params , message_txt and  message_html. So in some emails message_txt length was 0 and this is where sendgrid was raising errors and I was thinking I am sending the message content with email (message_html).
Anyway Thanks a lot.


Automatically generated registrants for web plugins

Description
This change adds a web platform section to the platforms section of pubspec.yaml so that plugins can define their web implementations. For a Flutter Web project, it generates a registrant which automatically registers all web plugin dependencies and imports the registrant from the generated entrypoint.
Related Issues

Closes https://github.com/flutter/flutter/issues/39591

Tests
I added the following tests:

Added a test in plugin_parsing_test.dart to verify that the web platform section is properly parsed.

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

[x] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[x] I signed the CLA.
[x] I read and followed the Flutter Style Guide, including Features we expect every widget to implement.
[x] I updated/added relevant documentation (doc comments with ///).
[x] All existing and new tests are passing.
[x] The analyzer (flutter analyze --flutter-repo) does not report any problems on my PR.
[x] I am willing to follow-up on review comments in a timely manner.

Breaking Change
Does your PR require Flutter developers to manually update their apps to accommodate your change?

[ ] Yes, this is a breaking change (Please read Handling breaking changes). Replace this with a link to the e-mail where you asked for input on this proposed change.
[x] No, this is not a breaking change.

Here is the code to generate the plugin registrants. With this change I am able to add a web implementation to url_launcher and run the url_launcher demo with no changes to the project.

You'll need to run flutter update-packages --force-upgrade due to the addition of a new pubspec. You can leave that out until the end if there is a large diff.

Thanks for the quick reviews Jonah. I added some tests (which will have to be skipped until the next engine roll, they pass locally when running with --local-engine) and responded to the other comments. PTAL

Thanks! I'll land this when work starts up again tomorrow. What is the schedule of engine rolls? If it rolls tomorrow I may update this PR to enable the tests before landing

Engine rolls should happen automatically as soon as the bots finish (at most a few hours). unless the engine build is broken, which I don't think it is

Ah, I just merged with latest master and was able to enable the tests. I'll land later today when all tests are passing.

You'll want to enable it here:
https://github.com/flutter/flutter/blob/master/dev/bots/test.dart#L465
Otherwise it won't be run automatically on CI


Address Xeonus V3 feedback

Discuss wording and other suggestions:
https://balancerecosystem.slack.com/archives/C04KJE1U8TU/p1732968937461009

Suggestions for wording / UX:

boosted pool text, replace with: This Boosted pool's underlying mechanics are powered by wrapped Aave tokens, which generate yield from lending activities on the Aave protocol, resulting in continuous appreciation of the pool's total value.

where do we see buffers if they are used?

no hook details present on pool overview page (yet)

My Portfolio: all mixed, no indication of what is v2 and what is v3, needs a label IMO. See my deployer with lots of positions, impossible to sort stuff

For boosted poos: why are there no APRs on testnet? No tested lending happening? How can we make sure everything will work correctly with rate providers on mainnet?

UI testing:

Every time I want to swap sepolia ETH to a token I need to sign permit: ETH. I expect this to only be once per token I want to swap to authorize the vault? Is there a mis-label and it should be "sign permit: tokenname"?

Permit 2 breaks completely for me for this pair, also higher amounts

adds not showing up in "My Transactions", might be data issue

@Xeonus I will move this ticket as done. There are still some pending issues but they are already tracker in other issues. Thanks for reporting! 🙌


Hide support for PostgreSQL arrays behind a config switch

As we want to replace existing mapping with something more suitable (https://github.com/prestosql/presto/issues/682), the
first step is to provide a config toggle that will help during
transition period.

cc @vincentpoon @guyco33

We discussed this on Slack and decided to ship this for now, to avoid locking in the current behavior (which might not be the correct default behavior).


Enhanced the Team Page Card Design

fixes #21
improved the layout , colour and transition on hover

:white_check_mark: Pull Request Approved :white_check_mark:
PR Summary

Fixes #21: Improved the layout, color scheme, and hover transition for a better user experience.

Remarks

Looks good! The enhancements contribute significantly to the overall aesthetics and usability.

Next Steps

Merged into the main branch for deployment.

Thank you for your contribution!


Only one event for caldav calendars

When my calendars are caldav, only one event is ever shown, and it seems to be random as to which event is shown (they're not all from the same calendar, either).
Using the home assistant API directly to get calendar events, I see the expected number of events from multiple calendars.
I originally thought this was because my caldav calendars contain a uid member, not an id member (and the fix for #32 explicitly references the id member).  However, modifying the caldav component to use id instead of uid didn't make a difference.

I'm wrong.  The problem is the caldav calendars using uid instead of id.  I forgot to clear my browser cache when I made the change. :(  I'll submit a PR against home assistant -- I can't think of any good reason for the two components to be returning data in a different format.

Looks like somebody already suggested such a change, and it was rejected.  See https://github.com/home-assistant/home-assistant/pull/24831. Basically, they want the google calendar code to be modified to match the output of the caldav component, instead of just forwarding the info from Google, and for projects like this to adjust and assume the output of the caldav component is correct.

I tried adding uid as a fallback if id doesnt exist for event caching. See if that works in 3.11.3

It does not; I still see only one event this way.  I think the fix for #32 interferes, because it uses 'id' from the returned data directly, before the CalendarEvent objects are created.

I don't have a caldav server setup so it's hard to test. I can get a docker container up if there's one you know of thats simple to setup

Quoting Leonardo Merza<EMAIL_ADDRESS>
I don't have a caldav server setup so it's hard to test. I can get a
docker container up if there's one you know of thats simple to setup
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub[1], or mute the thread[2].

https://www.caldavserver.com/. I haven't looked at it much, but I
think that would provide a test platform for you.
Links:
[1]
https://github.com/ljmerza/calendar-card/issues/37?email_source=notifications&email_token=AAYXEZJDEJBCPGS6IKFK5OTQCRKBHA5CNFSM4IIUCVPKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD3OEQAY#issuecomment-517752835
[2]
https://github.com/notifications/unsubscribe-auth/AAYXEZMXNMZDBR3P7ID76ODQCRKBHANCNFSM4IIUCVPA
Preserve wildlife! Pickle a squirrel!

Confirming that 3.11.4 works.


Simple Wasmplugin failed loading in waypoint

Is this the right place to submit this?

[X] This is not a security vulnerability or a crashing bug
[X] This is not a question about how to use Istio

Bug Description
I have a very simple wasm image and it works fine with sidecar.
When I use it on waypoint:
apiVersion: extensions.istio.io/v1alpha1
kind: WasmPlugin
metadata:
  name: hello
  namespace: default
spec:
  selector:
    matchLabels:
      istio.io/gateway-name: httpbin
  pluginName: add_header
  imagePullPolicy: Always
  url: oci://webassemblyhub.io/linsun/helloworld:latest

The waypoint failed to validate the TypedExtensionConfig with the following in the log:
2023-10-06T19:01:58.987901Z	warning	envoy config external/envoy/source/extensions/config_subscription/grpc/grpc_subscription_impl.cc:128	gRPC config for type.googleapis.com/envoy.config.core.v3.TypedExtensionConfig rejected: Proto constraint validation failed (WasmValidationError.Config: embedded message failed validation | caused by PluginConfigValidationError.VmConfig: embedded message failed validation | caused by VmConfigValidationError.Code: embedded message failed validation | caused by AsyncDataSourceValidationError.Remote: embedded message failed validation | caused by RemoteDataSourceValidationError.Sha256: value length must be at least 1 characters): config {
  name: "default.hello"
  root_id: "add_header"
  vm_config {
    runtime: "envoy.wasm.runtime.v8"
    code {
      remote {
        http_uri {
          uri: "oci://webassemblyhub.io/linsun/helloworld:latest"
          cluster: "_"
          timeout {
            seconds: 30
          }
        }
      }
    }
    environment_variables {
      key_values {
        key: "ISTIO_META_WASM_IMAGE_PULL_POLICY"
        value: "Always"
      }
      key_values {
        key: "ISTIO_META_WASM_PLUGIN_RESOURCE_VERSION"
        value: "11088"
      }
    }
  }
  configuration {
  }
}
	thread=21

Version
1.19.1

Additional Information
No response

Confirmed this is indeed caused by ecds config not able to work with delta response. I got wasm oci image downloaded fine after disable delta per @kyessenov 's suggestion in https://github.com/istio/istio/blob/359678f4bc7be37143dd9b1b726e950d1a6be99d/pkg/bootstrap/config.go#L94

This exists not only in waypoints but also sidecars using delta xDS with ECDS. I think this is related to upstream Envoy after correcting the forward logic in https://github.com/istio/istio/pull/47302. I've also sent an issue to track https://github.com/envoyproxy/envoy/issues/30379.

Still exist

@howardjohn Just tested https://github.com/envoyproxy/envoy/issues/30379 again. The new behavior observed is that the init configuration dump information is no longer shown. However, proxies that enable Delta XDS, such as waypoint or sidecars with ISTIO_DELTA_XDS: "true", still cannot be ready. I'm not sure if this is now an issue on the Envoy side, so I raise it here for investigation.
To reproduce, enable delta XDS for the sidecar or use a waypoint, and add a WasmPlugin resource as follows:
apiVersion: extensions.istio.io/v1alpha1
kind: WasmPlugin
metadata:
  name: hello
  namespace: default
spec:
  selector:
    matchLabels:
      app: httpbin
  pluginName: hello
  imagePullPolicy: Always
  url: <wasm-address>

Then, redeploy the pod. The proxy will not be ready, with an ECDS config like:
{
  "@type": "type.googleapis.com/envoy.admin.v3.EcdsConfigDump",
  "ecds_filters": [
  {
    "ecds_filter": {
      "@type": "type.googleapis.com/envoy.config.core.v3.TypedExtensionConfig",
      "name": "extenstions.istio.io/wasmplugin/default.hello"
    },
  "last_updated": "2024-02-08T07:01:16.286Z"
  }
  ]
},

What does Istiod logs say? Is it responding to this delta request?

What does Istiod logs say? Is it responding to this delta request?

@ramaraochavali I have attached the logs from istiod, and both the sidecar httpbin.bookinfo and the waypoint httpbin-istio-waypoint.default; they are both blocked. From the logs, it appears they did not receive the delta response.
logs.zip

ok. thank you. I will take a look


Updating a date field using update([]) does not convert it to mongo date.

Laravel-mongodb Version: 3.6.6
PHP Version: 7.3
Database Driver & Version:

Description:
Updating documents with the ->update() method saves Carbon dates as Object instead of Date in the database. Doing the same using ->save() on a model works fine. Yes, the models have the property registered as a date in the $dates array.
Steps to reproduce

Create model with property 'end_date'. Add property to $dates array on the model.
Create date $date = Carbon::parse('2021-01-08 10:00');
Run query: Model::where('match', true)->update(['end_date' => $date]);
Check db.

Expected behaviour
The end_dateshould be saved as a field of type Date instead of Object

Actual behaviour
end_date is saved as object

Additionally, if you were to do ->update(['end_date' => $date->toIso8601String()]) the date would be saved as a String field

Hello,
That's correct behavior. You're updating the field which is saved as a string.
If you need to save it as a date field then typecast it to https://github.com/jenssegers/laravel-mongodb/blob/ae864fbca560419cca65b503c5da0a1315bf177a/src/Jenssegers/Mongodb/Eloquent/Model.php#L92
Thanks!

Hello,
That's correct behavior. You're updating the field which is saved as a string.
If you need to save it as a date field then typecast it to https://github.com/jenssegers/laravel-mongodb/blob/ae864fbca560419cca65b503c5da0a1315bf177a/src/Jenssegers/Mongodb/Eloquent/Model.php#L92
Thanks!

But why isn't it consistent?
If I do:
$model->end_date = $date;
$model->save();
then it is saved as date.

But why isn't it consistent?
If I do:
$model->end_date = $date;
$model->save();
then it is saved as date.

But why isn't it consistent?
If I do:
$model->end_date = $date;
$model->save();
then it is saved as date.

Yes, should be same but not taking care.
Thanks!

But why isn't it consistent?
If I do:
$model->end_date = $date;
$model->save();
then it is saved as date.

Yes, should be same but not taking care.
Thanks!

Just chiming in that I would agree the update() behaviour should obey the $dates fields.
$model = new Model();
Model::update([
    'date_field' => Carbon::now()
]);

Should be functionally the same as:
$model = new Model();
$model->date_field = Carbon::now();
$model->save();

Just run into this problem now, after trying $dates to no success, it seems $casts did the trick.
protected $casts = [         'date_field' => 'datetime'  ];
Then it works as expected!


Make Bark a HF Pre-Trained Model

PR to make the Bark model a HF PreTrainedModel. The PreTrainedModel class takes care of all loading / saving logic, enabling checkpoints to be downloaded / pushed directly to the HF Hub:
from bark import GPT

model = GPT.from_pretrained("suno/bark-text")  # load model weights from Hub

Model weights on the HF Hub have version control and download counters. Users can also filter HF Hub models by type, e.g. by TTS.
Preliminarily, this PR only makes the required modelling code changes. The next step of the PR is to update generation.py, namely replacing the _load_model functionality with a single .from_pretrained call:
https://github.com/suno-ai/bark/blob/d621ee3088f29f6d12c2d8b0503e2368f18fabd9/bark/generation.py#L169
Since Transformers is already a dependency of Bark, this adds no new dependency requirements. It also has no effect on the .forward call (functionality remains 1-to-1 the same).
For details on the PreTrainedModel class, refer to the code and docs.

awesome, thanks sanchit. what do we need to do on the HF side for this to work?

Hi @gkucsko - I'm VB, and I work with @sanchit-gandhi on the Open Source Audio team at Hugging Face. IMO in terms of next steps, we would need the open source model checkpoints to: https://huggingface.co/suno and then the .from_pretrained method should work with the checkpoints on suno org. Happy to help with it.
We can also add the model card there along with inference details to help with the discovery.

awesome thanks, super swamped rn, but promise i'll get back to it later in the week!


webpack works well with vizmake process

PR #6 makes everything happen, BUT you have to run unpublish() and then vizmake() to get webpack to re-bundle the js modules if you edit anything since they are not a target.
We could...

make every js module a target in viz.yaml and have the bundle publish step depend on each one.
re-run the webpack bundle step every time vizmake is called no matter what
work webpack into vizmake function

Potential things for number 3:

parameters:
  -
    id: webpack_input_js
    location: src/app.js
  -
    id: webpack_output_js
    location: bundle.js
  -
    id: webpack_module_dir
    dir: src/modules/
...

publish:
  -
    id: js_bundle # get appropriate script tag
    relpath: ??????
    mimetype: application/javascript
    depends: webpack_output_js
  -
    id: figure_section # need this so that d3 code gets run after d3 has been loaded
    template: layout/templates/main_fig.mustache
    publisher: section
    depends:
      script: js_bundle
    context:
      script: script
  -
    id: webpack_config
    template: webpack_config.mustache
    publisher: ??????
    depends:
    	input: webpack_input_js
	output: webpack_output_js
	context:
		input: input
		output: output

---------- webpack_config.mustache ----------

const webpack = require("webpack");

module.exports = {
    entry: './{{input}}',
    output: {
        filename: './{{output}}'
    },
    plugins: [
      new webpack.ProvidePlugin({
          d3: 'd3'
      })
    ]
};

---------- vizmake() calls a function called webpacker() ----------

check to see if package.json existed. if not, run `npm init -y`
check to see if correct node_modules are installed. if not install them.
check to make sure webpack.config.js exists (throw error if it doesn't because that should come from publish step)
?? when does package-lock.json get created?

once everything else is published `npm run start`

Right now this is functioning as I would expect with vizlab v0.3.8 or greater. Specific things can be created as new issues.


Add PyTorch Live Demo for a Text Classification Task using Lightning Flash.

Summary

This PR follows the discussion in #4 for adding a demo using Lightning Flash.
Changelog

[CATEGORY] [TYPE] - Add a new demo using Lightning Flash for the Text Classification task.

Thanks for submitting the text classification tutorial, @karthikrangasai!
Currently, the website build fails due to missing assets that are referenced in the tutorial. We can take a look and eventually merge if you submit the assets :)
Thanks again!

Hello @raedle ,
I am working on creating the live.spec.json and the model and the assets. So I will be using this tutorial to create and fix mistakes in the tutorial if I find any. (Like a loop in itself)
Currently I just have the tutorial file based on other demos.


Upgrade

Coverage remained the same at 95.848% when pulling 21bfebc722fa3c517bf66758aa611235cba9ffb3 on upgrade into 8e177f586b22b630ea94c640759074f1910ab6a6 on master.


How to use more than one optimizer at each step (jointly train multiple modules within one model)?

❓ Questions and Help
What is your question?
I have a model which consists of two blocks, let's call them first_module and second_module.
Code (simplified)
Training Step
def training_step(self, batch, batch_idx, optimizer_idx):
    out = self.first_module(batch)
    out = self.second_module(out)

    loss = criterion(out, batch['target'])

    metrics = {'train_loss': loss}
    output = {'loss': loss,
                  'log': metrics,
                  'progress_bar': metrics}

    return output

Optimizers
def configure_optimizers(self):
    train_params = self.train_params

    optimizer_first_module = torch.optim.Adam(
        self.first_module.parameters(),
        lr=train_params['lr_first_module'], betas=(0.5, 0.999))

    optimizer_second_module = torch.optim.Adam(
        self.second_module.parameters(),
        lr=train_params['lr_second_module'], betas=(0.5, 0.999))

    return [optimizer_first_module, optimizer_second_module]

Question
How to do optimizer_first_module.step() and optimizer_second_module.step() at each batch and ignore batch_idx?
It may be seen that optimizer_step always passes only one optimizer per step
    def optimizer_step(
        ...
        optimizer: Optimizer,
        optimizer_idx: int,
        ...
    ) -> None:

Possible solution (?)

Blending two optimizers into one (hacky way, not sure if pl would see the result as a correct optimizer class)
Modify training loop in PL (this option is even worse)

def training_step(self, batch, batch_idx, optimizer_idx):
    if optimizer_idx == 0:
        out = self.first_module(batch)
        out = self.second_module(out)

        loss = criterion(out, batch['target'])

        metrics = {'train_loss': loss}
        self.output = {'loss': loss,
                  'log': metrics,
                  'progress_bar': metrics}

    return self.output

def backward(self, closure_loss, optimizer, opt_idx):
    if opt_idx == 0:
        super().backward(closure_loss, optimizer, opt_idx)

As of now, lightning doesn't support returning no loss in training_step so you can attach it to self. But once it does you will be able to return results with no loss or None in future.


Indent rules don't work for some coding styles

VSCode Version: Code 1.15.0 (8b95971d8cccd3afd86b35d4a0e098c189294ff2, 2017-08-09T20:09:15.006Z)
OS Version: Linux x64 4.4.0-83-generic
Extensions:

Extension
Author (truncated)
Version

you-complete-me
Ric
1.0.36

togglehs
bbe
0.1.2

chromium-codesearch
cha
0.0.7

vscode-instant-markdown
dba
1.2.1

python
don
0.7.0

gn
lju
0.0.1

cpptools
ms-
0.12.2

difftomaster
pav
0.0.1

proto
pet
0.0.2

annotator
ryu
0.10.1

rewrap
stk
1.4.2

change-case
wma
1.0.0

(1 theme extensions excluded)

Steps to Reproduce:
Write a function in C++ with parameters spread over 2 lines. The indent rules currently puts you 2 characters (or however many) in relative to the already indented second parameter line.
void Test(bool param1,
          bool param2) {
            ^ Indent rule puts you here
}

Instead, the indent rules should put you 2 characters in relative to the line where the function starts.
void Test(bool param1,
          bool param2) {
  ^ Indent should be here
}

This is extremely distracting, especially when pasting code from elsewhere.

Reproduces without extensions: Yes

Duplicate of https://github.com/Microsoft/vscode/issues/30859, let's track it there.


Bug: ActiveQuery::one

PHP information:
PHP 5.4.38 (cli) (built: Feb 18 2015 16:46:56) with eAccelerator v1.0-dev, Copyright (c) 2004-2012 eAccelerator, by eAccelerator
In yii\db\ActiveQuery::one if $models is not empty array next code returned null
return reset($models) ?: null;

if i replaced to
return reset($models)? reset($models) : null;

This code returned first element of array. I think the cause of for this behavior is eAccelerator.

Ternary shortsyntax ?: is widely used accross the framework - not only at yii\db\ActiveQuery::one()

@grigorieff are you using latest eaccelerator version? It looks like the one you're using isn't fully compatible with PHP 5.4...

Description says eAccelerator v1.0-dev, although I can't find related one at https://github.com/eaccelerator/eaccelerator
?: syntax has been introduced at PHP 5.3. If eAccelerator has a problem with it, it should revealed itself long ago.

Anyway, I don't think we should change short ternary into long one. It's definitely eaccelerator issue and it should be fixed there.

@samdark I think the same. Thanks! eAccelerator v. <IP_ADDRESS>

@grigorieff, if you are able to reproduce this issue with plain PHP, please submit it to https://github.com/eaccelerator/eaccelerator/issues/new

Installed OPCache instead of eAccelerator and ?: was working now is good ;)


'"MZmine" is damaged and can't be opened. You should move it to the Trash' on macOS

Basic information

My operating system: macOS 12.6 (21G115)
My MZmine version: 3.2.8

What happened
Ran the installer. Double-clicked the MZmine icon. Got the message:
"MZmine" is damaged and can't be opened. You should move it to the Trash

Hi, please see the relevant documentation page regarding installation on macOS:
https://mzmine.github.io/mzmine_documentation/getting_started.html#on-macos


Design an ".etch" format

This format should be able to hold all the extended information roughly described in https://github.com/resin-io/etcher/pull/707, plus allow the client to consume it in a streaming fashion.

This should probably be closed in favour of #707 ?

Sounds good!


Issues with istio-installation.md

from hackathon feedback:
installing-istio:

Installation link for kubectl in the prerequisites. Useful for the option of using google container engine.
Better to mention that both source and istioctl tars has to be extracted in step 1 (installation on existing cluster)
Better to have absolute links with home directory reference like (<ISTIO_HOME/) in all the kubectl steps (kubectl apply -f <istio_home>/isitio-1.0/install/kubernetes/addons/grafana.yaml)
Good to mention the optional components (like grafana) in the list svc and pod (steps 1 and 2 verifying installations)

this is fixed in https://github.com/istio/istio.github.io/pull/198 and later

I did not close it on purpose, there are a couple of things left.

ah sorry, what's left ?


App removed from Google Play

The app has been remove from Google Play, and cannot be downloaded anymore.

Hi! Yes, I had to unpublish it because I was exceeding the free quotas on Google App Engine. I'm sorry about it.

Hi! Yes, I had to unpublish it because I was exceeding the free quotas on Google App Engine. I'm sorry about it.

Much appreciated you provided it up until that point :)

Well, this is a shame.
What does it cost to get whatever you need to add more capacity?
If it's nominal, would it be feasible to setup a donation link of some kind so the community might be able to potentially contribute to keep this helpful utility alive?
Thanks

I concur : this app is/was very useful to e.g. test microG GCM installations.
Would a donation link help out ?

Thanks for your kind words. In the month I stopped it was already 3-4€ a month. Probably the instance settings on App Engine could have been tweaked to bring it down, however I really don't want to ask the community for money to pay the server bills.

AH thank you for the app. It has been very useful in testing with microg. Hope you find a way to get it working again. I second the donation option if you feel for it in the future...

@bbindreiter: Isn't possible to compile with microG libs and put it on F-Droid?

Of course. Source code is available for everyone to use.
Problem is on server side though. Backend is not available anymore as I took it offline.

is there any alternative


[SPARK-43863][CONNECT] Remove redundant toSeq from SparkConnectPlanner for Scala 2.13

What changes were proposed in this pull request?
This pr aims to remove redundant toSeq from SparkConnectPlanner for Scala 2.13.
Why are the changes needed?
Remove redundant toSeq for Scala 2.13 to avoid unnecessary collection copies
Does this PR introduce any user-facing change?
No
How was this patch tested?

Pass Github Actions
Manually checked connect and connect-client-jvm module for Scala 2.12

The only question will be whether this was somehow needed in Scala 2.12. I'm always slightly worried that the copy is accidentally needed, but I doubt it here

For Scala 2.12, there are over 50 redundant toSeq in SparkConnectPlanner, I have selected 14 of them, which are also redundant toSeq for Scala 2.13.
These changes are all for local vals and I think they should be safe, and I have also manually verified the testing of Scala 2.12, they passed.

Merged to master


1.0.0RC2 always times out on kong quit (more than 10 seconds)

Consider a small test script ./test.sh (using a 10 minute timeout)
date
docker exec -it $1 kong version
docker exec -it $1 kong quit --timeout=600
date

With an 0.14.0 container it takes less than a second:
Thijss-MBP:alpine thijs$ ./test.sh 0168ae733082b33ed47d3df1574cfd38cf17bade3fa031cf36882b71723404a9
Tue Nov 13 14:40:13 CET 2018
0.14.0
Tue Nov 13 14:40:13 CET 2018
Thijss-MBP:alpine thijs$

With an 1.0.0rc2 container it takes 1 minute:
Thijss-MBP:alpine thijs$ ./test.sh cd4c6d97f48a738487bed88da52c7d18d4cc94bf5dc26f9629759db6c5718f34
Tue Nov 13 14:55:01 CET 2018
1.0.0rc2
Tue Nov 13 14:56:01 CET 2018
Thijss-MBP:alpine thijs$

Note: I created the RC2 image myself based on Alpine (using kong/docker-kong repo)
Logs from the container:
Thijss-MBP:alpine thijs$ docker logs cd4c6d97f48a738487bed88da52c7d18d4cc94bf5dc26f9629759db6c5718f34
2018/11/13 13:54:44 [notice] 1#0: using the "epoll" event method
2018/11/13 13:54:44 [notice] 1#0: openresty/<IP_ADDRESS>
2018/11/13 13:54:44 [notice] 1#0: built by gcc 6.3.0 (Alpine 6.3.0)
2018/11/13 13:54:44 [notice] 1#0: OS: Linux 4.9.93-linuxkit-aufs
2018/11/13 13:54:44 [notice] 1#0: getrlimit(RLIMIT_NOFILE): 1048576:1048576
2018/11/13 13:54:44 [notice] 1#0: start worker processes
2018/11/13 13:54:44 [notice] 1#0: start worker process 29
2018/11/13 13:54:44 [notice] 1#0: start worker process 30
2018/11/13 13:54:44 [notice] 1#0: start worker process 31
2018/11/13 13:54:44 [notice] 1#0: start worker process 32
2018/11/13 13:55:01 [notice] 1#0: signal 3 (SIGQUIT) received from 56, shutting down
2018/11/13 13:55:01 [notice] 29#0: gracefully shutting down
2018/11/13 13:55:01 [notice] 30#0: gracefully shutting down
2018/11/13 13:55:01 [notice] 32#0: gracefully shutting down
2018/11/13 13:55:01 [notice] 31#0: gracefully shutting down
2018/11/13 13:55:01 [notice] 31#0: exiting
2018/11/13 13:55:01 [notice] 32#0: exiting
2018/11/13 13:55:01 [notice] 30#0: exiting
2018/11/13 13:55:01 [notice] 31#0: exit
2018/11/13 13:55:01 [notice] 32#0: exit
2018/11/13 13:55:01 [notice] 1#0: signal 17 (SIGCHLD) received from 31
2018/11/13 13:55:01 [notice] 1#0: worker process 31 exited with code 0
2018/11/13 13:55:01 [notice] 1#0: signal 29 (SIGIO) received
2018/11/13 13:55:01 [notice] 1#0: signal 17 (SIGCHLD) received from 32
2018/11/13 13:55:01 [notice] 1#0: worker process 32 exited with code 0
2018/11/13 13:55:01 [notice] 1#0: signal 29 (SIGIO) received
2018/11/13 13:55:01 [notice] 30#0: exit
2018/11/13 13:55:01 [notice] 1#0: signal 17 (SIGCHLD) received from 30
2018/11/13 13:55:01 [notice] 1#0: worker process 30 exited with code 0
2018/11/13 13:55:01 [notice] 1#0: signal 29 (SIGIO) received
2018/11/13 13:56:01 [notice] 29#0: exiting
2018/11/13 13:56:01 [notice] 29#0: exit
2018/11/13 13:56:01 [notice] 1#0: signal 17 (SIGCHLD) received from 29
2018/11/13 13:56:01 [notice] 1#0: worker process 29 exited with code 0
2018/11/13 13:56:01 [notice] 1#0: exit
Thijss-MBP:alpine thijs$

I repeatedly ran this, and it always takes 1 minute.

Just tested with an image for rc2 that I pulled, instead of creating it myself, same problem.

Ok, does this happen with both Pg and C*?
Can you try commenting this out:
https://github.com/Kong/kong/blob/next/kong/db/strategies/postgres/connector.lua#L293-L303

Yes, I think ^ that is it.

Merged in next.


Fix shells and lmax argurments

Proposed changes
Resolves #34.
Fixes the arguments to pass space-separated command line arguments (snakemake) as comma-separated to mrtrix rules (as necessary)
Types of changes
What types of changes does your code introduce? Put an x in the boxes that apply

[x] Bugfix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionalitiy)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Other (if none of the other choices apply)

Checklist
Put an x in the boxes that apply. You can also fill these out after creating the PR. If you are unsure about any of the choices, don't hesitate to ask!

[x] Changes have been tested to ensure that fix is effective or that a feature works.
[ ] Changes pass the unit tests
[ ] Code has been run through the poe quality task
[ ] I have included necessary documentation or comments (as necessary)
[ ] Any dependent changes have been merged and published

Notes
All PRs will undergo the unit testing before being reviewed. You may be requested to explain or make additional changes before the PR is accepted.

Do you mind merging from main so the linters work properly?

Do you mind merging from main so the linters work properly?

Rebased from main, workflows should work now.


Null safety migration

This pull request addresses migration to Dart's new safety feature and it brings many improvements related to lint rules which haven't been followed thus far (for instance, preferring var).

@alfonsocejudo can you merge this and publish a new pub.dev?

I have an error using the null safety version
E/flutter (  687): [ERROR:flutter/lib/ui/ui_dart_state.cc(186)] Unhandled Exception: type '(BaseCluster, double, double) => MapMarker' is not a subtype of type '((BaseCluster?, double?, double?) => MapMarker)?'
I initialised the cluster in this way:
return Fluster<MapMarker>(       minZoom: minZoom,       maxZoom: maxZoom,       radius: 150,       extent: 4096,       nodeSize: 64,       points: markers,       createCluster: (         BaseCluster cluster,         double lng,         double lat,       ) =>           MapMarker(         id: cluster.id.toString(),         latitude: lat,         longitude: lng,         isCluster: cluster.isCluster,         clusterId: cluster.id,         pointsSize: cluster.pointsSize,         childMarkerId: cluster.childMarkerId,       ),     );
Do you know what the problem is?


Serializing Computed Values with onSnapshot/onPatches: Valid Mobx-Keystone Use-Case?

Say I have a model Guests with an array of model Guest. Then I have a computed value which is some subset of the array.
@model('Guests')
class Guests extends Model({array: prop<Guest[]>(() => []) {
  @computed
  get someGuests() {
    return this.array.filter(...);
  }
}

Is it within the intended paradigm of Mobx Keystone to wish to serialize this computed array of models using onSnapshot or onPatches?
Currently, I am doing this by setting up a reaction in onAttachedToRootStore in order to track the computed and update a model prop. However, this "violates" a cardinal MobX best practice, which is to avoid/never update observables from reactions.
@model('Guests')
class Guests extends Model({
  array: prop<Guest[]>(() => []),
  ids: prop<string[]>(() => []).withSetter()
}) {
  @computed
  get someGuests() {
    return this.array.filter(...);
  }
  onAttachedToRootStore() {
    const idsDisposer = autorun(() => {
	    this.setIds(this.someGuests.map(guest => guest.id));
    });

    return () => {
	    idsDisposer();
    };
  }
}

Is this simply not a valid use of MobX Keystone? How might I think about my models differently in order to adapt to my use-case, which is to (reactively) send an array of Guest.ids to the back end?
I understand that one way of thinking about this is that the local tree can be re-created elsewhere by essentially running the same computed with the total state of the local tree. However, in my case, the computation is more efficiently done on the client, and the results sent to the server.

do you need ids to be a mobx keystone property? perhaps it could be a plain variable if that feels less "impure"

By "plain variable" do you mean if it was a normal, observable property on the class?
One thing I'm trying to wrap my head around with using Mobx Keystone is whether an entity without the need for serializable state should or should not be a Keystone Model.

Yes, that's what I mean.
As for if it should be a mobx keystone model or not, depends. Do you need serialization or a middleware (such as undo/redo)? Then yes, else no.
Basically, your document and/or config could be part of mobx-keystone, while everything else (e.g. UI state) could be in plain mobx classes or the react state.

Ah! Thank you for that description, I find it very helpful. So, for example, even if I don't require something to be serializable, the middleware patterns can be useful even for plain observables.
I'll still have to do some thinking, however, about the clearest way to serialize a computed to send to the server. I was just hoping there was a "native" keystone pattern already.


Inserting multiple rows into a table

In the test I find:
tx.executeSql('INSERT INTO Employees (name, office, department) VALUES ("Sylvester Stallone", 2,  4);');
tx.executeSql('INSERT INTO Employees (name, office, department) VALUES ("Elvis Presley", 2, 4);');
tx.executeSql('INSERT INTO Employees (name, office, department) VALUES ("Leslie Nelson", 3,  4);');
tx.executeSql('INSERT INTO Employees (name, office, department) VALUES ("Fidel Castro", 3, 3);');
tx.executeSql('INSERT INTO Employees (name, office, department) VALUES ("Bill Clinton", 1, 3);');
tx.executeSql('INSERT INTO Employees (name, office, department) VALUES ("Margaret Thatcher", 1, 3);');
tx.executeSql('INSERT INTO Employees (name, office, department) VALUES ("Donald Trump", 1, 3);');
tx.executeSql('INSERT INTO Employees (name, office, department) VALUES ("Dr DRE", 2, 2);');
tx.executeSql('INSERT INTO Employees (name, office, department) VALUES ("Samantha Fox", 2, 1);');

Is there a way to do this with one statement with passing an array?

Please raise this sort of question on stack overflow.

ok. https://stackoverflow.com/questions/57711001/how-can-i-insert-multiple-rows-into-a-table-with-react-native-sqlite-storage

This worked for me:

db.transaction(tx => {
  for (let i = 0; i < my_list.length; i++) {
    tx.executeSql(
      'INSERT OR REPLACE INTO category (id,name,age) VALUES (?,?,?,)',
      [
        my_list[i].id,
        my_list[i].name,
        my_list[i].age,
      ],
      (tx, results) => {
      },
      error => {
      },
    );
  }
});


translate communityguide.md into Chinese

may be the translation should be a separate version @Otsar-Raikou ? or may be we can add readme for different languages ?

I think a separate version is better

Yeah lets do a separate version.
On Wed, Oct 25, 2023 at 11:30 AM miles @.***> wrote:

I think a separate version is better
—
Reply to this email directly, view it on GitHub
https://github.com/ingonyama-zk/ingopedia/pull/86#issuecomment-1778772179,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A5E2DTEW4ZF2THCLUATECZLYBDEZZAVCNFSM6AAAAAA6O4C5ZSVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTONZYG43TEMJXHE
.
You are receiving this because you were mentioned.Message ID:
@.***>

--
Otsar Shalmoni
Marketing Manager

can this pr be merged？ so i will continue other part

Hi Miles, Thank you for your effort.
At this point, we would like to keep the main ingopedia page (GitHub mdbook version) in English
https://ingonyama-zk.github.io/ingopedia/ So I am not including this PR.
There is a html version of the same page, https://www.ingonyama.com/ingopedia/communityguide which can be translated easily into any language. We feel that it is better to do it at this level, to avoid changing the base level ingopedia hosted in GitHub differently in different languages.
If you would like to contribute there please let me know and we willl get back to you. Thanks!


Hangs in JSDom for the latest version

Downgrading to ~10.0 is the solution.

Please provide details of how to reproduce.

Closing due to inactivity and failure to provide any additional information.  Can always be reopened if more information or a reproduce-able test case is provided.


torch.compile fails

Using it like this:
optimizer = heavyball.PrecondSchedulePaLMSOAP(raw_model.parameters(), lr=0.05)

The heavyball/utils.py line 31 return compiled is where it torch.compile fails.
This is called from precond_schedule_palm_foreach_soap.py line 69 grad_projected = project(g, state['Q'], False)
The specific error is unhashable type: set

Seeing this as well. May be fixed by heavyball 1.0.0


Adding a blueprint to the top with ordering@: 0 not working

Looks like ordering@: 0  (https://learn.getgrav.org/forms/blueprints/advanced-features#changing-field-ordering) is not working. 1 and above work, but 0 is not, a new field is always at the bottom (edited)
ordering@: -10000 works
@mahagr hint:

yeah, there's likely (!$field['ordering@'])
should likely be !isset()

it's an issue with yaml, 0 is evaluating to false, try !!int 0


Fix some compiler warnings.

Again, given that char does not define whether it's signed or unsigned
in C, some compilers may issue warnings on these functions (leftover
from "various: Avoid using 'char' as an index" commit). At
power-supply.c, we cast to the right type for
sol_power_supply_get_capacity() usage.

:+1:

Merged with fixes.


DBZ-3672 Remove unused code

https://issues.redhat.com/browse/DBZ-3672

Does this require any doc updates? If those metrics are exposed in a different way now, is this reflected in the current docs?

@gunnarmorling Hi!  There should be no docs update required as we had already merged all these metrics into the streaming metrics class, I just forget when doing so that we should have removed these classes as part of that code consolidation effort.

Ok, cool. Thx for clarifying, @Naros! Merging.


improved javadoc for the generated env var json schema

Refer to this link for build results (access rights to CI server needed):
https://fabric8-ci.fusesource.com/job/fabric8-quickstarts-master-pullreq/217/
Test PASSed.

Merged


fix: allow selection on mobile ios

On mobile ios up until ios9, no mousedown event is fired, because ios keeps the focus
on the currently active input element. However, it does fire a click event.
To work around this problem, but keep the current behavior, we add a click event
handler to the existing mousedown handler, which will now only preventDefault,
which keeps blur from firing, but allows the click handler to do the actual selection.
This new behavior additionally means that clicking and holding (dragging) the mouse away
from the autocomplete list will not result in a selection, mimicking the normal
mouse behavior.
Unit tests pass in Chrome 54
Manually tested on Win 10 Chrome 54, FF 49, Edge, IE11; on ios with 9.3.5 and 10.1
FF
This is an alternative to #16715 and fixes #16712

Thank you @Narretz, I gave it a go and this does indeed fix the issue I'm having. Good work.

@MHanslo That's good to hear, thanks for testing. Follow up question, did you test it in browser or inside an app (e.g. cordova)? I only tested in browser but as I found out the problem was actually reported for an app.

@Narretz, sorry, I've been quite busy lately. I recall I tested this in the browser; using the Chrome iPhone emulator I could always replicate the touch bug on our app. Dropping in your fix corrected the bug; I could actually perform a selection.

@LeaVerou Just stumbled over this again. Are you still maintaining this library?

@LeaVerou I've rebased the PR

Merged, thanks!


Can't run dependent-project example

I was able to get the dependent-project to work via git sobmodules but if I try the npm approach, I first get
You should not need to recompile .ts files there, but if you really want to, use the allowTsInNodeModules option. See: https://github.com/Microsoft/TypeScript/issues/12358

If I enable that option, I just get a lot of errors like
ERROR in ./node_modules/neuroglancer/src/neuroglancer/datasource/render/frontend.ts
    Module build failed (from ./node_modules/ts-loader/index.js):
    Error: TypeScript emitted no output for /Users/chrisj/dev/tmp/neuroglancer/examples/dependent-project/node_modules/neuroglancer/src/neuroglancer/datasource/render/frontend.ts.

How do you enable that option? I have the webpack config:
`'use strict';
const path = require('path');
const webpack_helpers = require('./webpack_helpers');
module.exports = env => {
env = env || 'dev';
return webpack_helpers.getViewerConfigFromEnv(
{outputPath: path.resolve(__dirname, '../dist/' + env)}, env);
};
`

Nevermind. I added a param in the path property in the tsconfig.json:
"neuroglancer/": ["./node_modules/neuroglancer/src/neuroglancer/"],
then ran "npm link" from the neuroglancer project and "npm link neuroglancer" from the dependent-project. Then I ran it and it worked.


How much computing resources does we needs with Google Cloud ?

Hi,
We are considering using GATKSVPipelineBacth.wdl of GATK-SV to analyze and process CRAM data for 100 people in our possession.
When I ran a test with a sample of 5 persons, the 1TB of disk space I had prepared as a work area was full halfway through the process, and we were unable to process the data.
At that time, we provided the VM resources on Google Cloud were 16 CPUs, 64GB memory, 200GB Boot disk and 1TB work disk. And the processing environment is based on CentOS7.9, using Singularity instead of Docker, and running on Cromwell.
How much computing resources on the Google Cloud are actually needed to process the data of 100 people?

Hi @kt-hrd, it sounds like you're trying to run on Cromwell configured with a local backend on your VM. This would likely take months to process your samples.
We really recommend you instead use Google cloud as a backend as it will be able to take advantage of the Google Pipelines API to parallelize the workflow. There is documentation available for setting up GCS backends here.

Hello @mwalker174, thank you so much for your reply.
We have just started processing 100 GATK-SV samples using Google's PAPI as per your recommendation.
We have also just found a description in Google's Cloud Life Science Tutorial regarding specific requirements for local computational resources. So we need a reasonable amount of computational resources on Google Cloud.


Made parseDocument generic

Fixes #245

@jbicker Still on it? There is a small merge conflict in the langium-parser.ts If you fix the other comment this PR is good to go!

squashed and force pushed the changes we directly talked about.


Hide/Show fields by user

Laravel Version: 5.5.13
Voyager Version: 1.0.4
PHP Version: 7.1.0

Question:
Hi Everyone,
There is a way to hide some fields for some users?
The only way i found is overriding views with additional custom checks.
Any other suggestion?
Thank you very much.
Alex

You will need to add the checks yourself in overridden views.  I recommend adding permissions to control this instead of checking the roles directly.  If you create permissions (e.g. edit_name_user), it will automatically be checked by @can('edit_name', Auth::user()).  The key is that the permission name MUST end with the model's BREAD "name"


Support html5 hdpi?

Related links:
http://caniuse.com/#feat=devicepixelratio
http://www.html5rocks.com/en/tutorials/canvas/hidpi/
I have seen that the html5 doesn't seems to support hdpi. It just updates width/height of the canvas, but should update style.width and style.height too for hdpi.

Added in 3.4.0 :smile:


UI Issue in iPhone 5s

Xcode 10.3 and iOS 12.4
Where need to change the code any suggestions ?

I got it. We can solve by changing font size
calendar.appearance.headerTitleFont = [UIFont systemFontOfSize:5];


help ansible find repo template from installs subdirectory

here is a PR with just the change to the path for the repo template - if you run without this you get an error like this.  Ansible seems to be checking ../templates, assuming that all yaml files will run from inside tasks/ subdirectory and not a directory nested inside it.
fatal: [rhs-cli-01] => input file not found at /mnt/oldroot/home/ben/ceph/dm-cache/ceph-ansible/roles/ceph-common/tasks/templates/redhat_storage_repo.j2 or /mnt/oldroot/home/ben/ceph/dm-cache/ceph-ansible/redhat_storage_repo.j2
BTW a similar change would be needed for RHCS 1.2 (Inktank Ceph Enterprise).

@bengland2 thanks.

Hi @leseb @bengland2
(Using RHCS ISO)
I tried with the latest master branch and v1.04 release but got this error
TASK [ceph.ceph-common : include] **********************************************
included: /usr/share/ceph-ansible/roles/ceph-common/tasks/./installs/install_rh_storage_on_redhat.yml for rhcs1

TASK [ceph.ceph-common : install red hat storage repository key] ***************
ok: [rhcs1]

TASK [ceph.ceph-common : add red hat storage repository] ***********************
fatal: [rhcs1]: FAILED! => {"changed": false, "failed": true, "msg": "IOError: [Errno 2] No such file or directory: u'/usr/templates/redhat_storage_repo.j2'"}

PLAY RECAP *********************************************************************
rhcs1                      : ok=23   changed=3    unreachable=0    failed=1

Fixed by editing roles/ceph-common/tasks/./installs/install_rh_storage_on_redhat.yml and changingsrc: ../../templates/redhat_storage_repo.j2 to src: redhat_storage_repo.j2
- name: add red hat storage repository
  template:
    src: redhat_storage_repo.j2
    dest: /etc/yum.repos.d/rh_storage.repo
    owner: root
    group: root
    mode: 0644
  when:
    - ceph_stable_rh_storage_iso_install

Myself and one of our partners who is also using ceph-ansible have seen this very problem. Have you seen this lately ?

@ksingh7 I haven't used ISO install recently
Which ansible version are you using?

@leseb ansible <IP_ADDRESS>

please try with <IP_ADDRESS>

@leseb
[root@client1 ceph-ansible]# ansible --version
ansible <IP_ADDRESS>

TASK [ceph.ceph-common : include] **********************************************
included: /root/ceph-ansible/roles/ceph-common/tasks/./installs/install_rh_storage_on_redhat.yml for ceph-mon1

TASK [ceph.ceph-common : install red hat storage repository key] ***************
ok: [ceph-mon1]

TASK [ceph.ceph-common : add red hat storage repository] ***********************
fatal: [ceph-mon1]: FAILED! => {"changed": false, "failed": true, "msg": "IOError: [Errno 2] No such file or directory: u'/templates/redhat_storage_repo.j2'"}
	to retry, use: --limit @site.retry

PLAY RECAP *********************************************************************
ceph-mon1                  : ok=15   changed=6    unreachable=0    failed=1

Fixed by changing to  src: redhat_storage_repo.j2

@alfredodeza
I am hitting this with RHCS2.0

It's definitely an issue with your ansible version, did you try with <IP_ADDRESS>, from which repo did you get <IP_ADDRESS>?

@leseb
earlier i was using epel for ansible and got ansible version as <IP_ADDRESS>
This time i used pip to get <IP_ADDRESS>

so was this failing with <IP_ADDRESS> also?

Haven't tried with <IP_ADDRESS> yet.

@ksingh7 we are using 1.9 downstream. Would you mind trying that with 1.9?

@alfredodeza @leseb works fine with 1.9
@alfredodeza so ansible v1.9 is the recommended one for RHCS ?


add linters files and README.md file

Hi @BANCUNGUYE66 ,
Your project is complete!  There is nothing else to say other than... it's time to merge it :shipit:
Congratulations! 🎉

Highlights

Nice job reviewing your project from trials ✔

Optional suggestions
Every comment with the [OPTIONAL] prefix won't stop the approval of this PR. However, I strongly recommend you to take them into account as they can make your code better. Some of them were simply missed by the previous reviewer and addressing them will really improve your application.
Cheers and Happy coding!👏👏👏
Feel free to leave any questions or comments in the PR thread if something is not 100% clear.
Please, remember to tag me in your question so I can receive the notification.

As described in the Code reviews limits policy  you have a limited number of reviews per project (check the exact number in your Dashboard). If you think that the code review was not fair, you can request a second opinion using this form.


Not able to pass arbitrary field values in User Registration

In my user registration template I have several arbitrary fields. And somehow they are not included in the arbitrary list with the exception of "password", "name" and "email". I tried several things:
ab.Config.Modules.RegisterPreserveFields = []string{"name", "surname", "gender",
  "description", "clientweightmax_int", "clientheightmax_int", "phonenumber","email", "username"}

ab.Config.Core.BodyReader = defaults.NewHTTPBodyReader(false, true)
ab.Config.Core.BodyReader.(*defaults.HTTPBodyReader).Whitelist["register"] = []string{"name",
  "surname", "gender", "description", "clientweightmax_int", "clientheightmax_int",
  "phonenumber","email", "username"}

But the only non-authboss field I get is "name" (very weird I know...), which gets correctly stored in the arbitrary list.
My template looks like:
<input type="text" class="form-control" name="name" placeholder="{{l "Name" $.lang}}" value="{{.name}}" autofocus required/> (<-- this works)
<input type="text" class="form-control" name="surname" placeholder="{{l "Surname" $.lang}}" value="{{.surname}}" required/> (<-- this does not)

I might be doing something really silly here but I don't seem to find the issue.
Thank you in advance for your help,
Daniele

I did implement the Arbitrary user interface. And I'm aware that preserve fields are just to refill the form in case of error. But I had to try... :)
// PutArbitrary inserts registration field names into user
func (u *User) PutArbitrary(values map[string]string) {
  fmt.Println("PutArbitrary. values: ", values)
  var err error
	if n, ok := values["name"]; ok {
		u.Name, err = utils.ValidateName(n)
	}
	if n, ok := values["surname"]; ok {
		u.Surname, err = utils.ValidateName(n)
	}
	if n, ok := values["gender"]; ok {
		u.Gender, err = utils.ValidateGender(n)
	}
	if n, ok := values["description"]; ok {
		u.Description, err = utils.ValidateDescription(n)
	}
	if n, ok := values["clientweightmax_int"]; ok {
		u.Clientweightmax, err = utils.ValidateInt(n, cfg.Constants.MinWeight, cfg.Constants.MaxWeight)
	}
	if n, ok := values["clientheightmax_int"]; ok {
		u.Clientheightmax, err = utils.ValidateInt(n, cfg.Constants.MinHeight, cfg.Constants.MaxHeight)
	}
	if n, ok := values["phonenumber"]; ok {
		u.Phonenumber, err = utils.ValidatePhone(n)
	}
	if n, ok := values["email"]; ok {
		u.Email, err = utils.ValidateEmail(n)
	}
	if err != nil {
	  log.LogError("Some fields couldn't be validated and won't be insterted in the DB", err, false)
	}
	ab_updateUser(u.Username, *u)
}

// GetArbitrary from user
func (u User) GetArbitrary() map[string]string {
  return map[string]string{
    "name": u.Name,
    "surname": u.Surname,
    "gender": u.Gender,
    "description": u.Description,
    "clientweightmax": strconv.Itoa(u.Clientweightmax),
    "clientheightmax": strconv.Itoa(u.Clientheightmax),
    "phonenumber": u.Phonenumber,
    "email": u.Email,
  }
}

Yep, I'm not concerned about the details of the implementations. As long as they're implemented that's good enough for me. (Though I am concerned you have a function name with an underscore in it!)
Like I said in a long winded way, you're not getting preserved fields correctly because in this snippet you showed me you are not referencing .preserve which is where all the preserved fields live.
<input type="text" class="form-control" name="name" placeholder="{{l "Name" $.lang}}" value="{{.name}}" autofocus required/> (<-- this works)
<input type="text" class="form-control" name="surname" placeholder="{{l "Surname" $.lang}}" value="{{.surname}}" required/> (<-- this does not)

Which is why I showed you the authboss sample template which is correctly referencing the preserved fields:
https://github.com/volatiletech/authboss-sample/blob/v2/ab_views/html-templates/register.tpl#L7
Fix your templates to correctly look in the .preserve field of the data for preserved fields and it should start working. Though I can't explain why name is working now, that's crazy to me.

Hi Aaron,
I think I explained myself in a very confusing way - sorry:), or I'm not
understanding what you meant. I'm not saying that the preserve feature
doesn't work (didn't test it yet), what I'm saying is that I don't manage
to save the arbitrary fields back in the DB with the sole exception of the
"name". "username" doesn't make it to the DB either even though I am using
it as primary ID for authboss. I nevertheless tried to put in the suggested
preserve template variable like in the authboss example for the "surname",
but it doesn't work for that either:
<input type="text" class="form-control" name="surname" placeholder="{{l
"Surname" $.lang}}" value="{{with .preserve}}{{with
.surname}}{{.}}{{end}}{{end}}" required/>
If you think there is no issue with authboss, then I guess I will just
debug my code deeper to find the bug (it might very well be something silly
on my side, that somehow I can't see), and you may close this issue.
Thanks again for your help,
Daniele
2018-07-06 17:59 GMT+02:00 Aaron L<EMAIL_ADDRESS>
Yep, I'm not concerned about the details of the implementations. As long
as they're implemented that's good enough for me. (Though I am concerned
you have a function name with an underscore in it!)
Like I said in a long winded way, you're not getting preserved fields
correctly because in this snippet you showed me you are not referencing
.preserve which is where all the preserved fields live.
<input type="text" class="form-control" name="name" placeholder="{{l "Name" $.lang}}" value="{{.name}}" autofocus required/> (<-- this works)
<input type="text" class="form-control" name="surname" placeholder="{{l "Surname" $.lang}}" value="{{.surname}}" required/> (<-- this does not)
Which is why I showed you the authboss sample template which is correctly
referencing the preserved fields:
https://github.com/volatiletech/authboss-sample/blob/v2/ab_views/html-
templates/register.tpl#L7
Fix your templates to correctly look in the .preserve field of the data
for preserved fields and it should start working. Though I can't explain
why name is working now, that's crazy to me.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/volatiletech/authboss/issues/173#issuecomment-403075443,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AeHCdo0EmVYlVbkE9YY2FjvGYmdiDkL9ks5uD4lggaJpZM4VFiWZ
.

BTW, thanks for the heads up on the function with underscore, I'm quite new to golang and wasn't aware of the convention:)

In the end it was my own bug, I was overwriting my configuration:
ab.Config.Core.BodyReader.(*defaults.HTTPBodyReader).Whitelist["register"] = []string{"name",
  "surname", "gender", "description", "clientweightmax_int", "clientheightmax_int",
  "phonenumber","email", "username"}

with the Authboss v2 example one:
Whitelist: map[string][]string{"register": []string{"email", "name", "password"},},

Sorry for the noise, please close this ticket.


Update fiat_0.yaml

Added info for user that entering a PIN actually provides additional functionality.
Should we also mention that currently the API has trouble (403 errors) if the car is not connected to the charging station?

This change must be done in core. The file you've changed is auto-generated.
See: https://github.com/evcc-io/evcc/blob/master/templates/definition/vehicle/fiat.yaml


Update reference to OpenZeppelin Contracts in docs

Documentation referenced installation of OpenZeppelin Contracts erroneously. I have updated the docs to follow the latest version of OpenZeppelin Contracts.
Thanks!

The docs have been updated to reflect this, but not as part of this PR. Thanks for contributing 😄


Use Xcode 9.2 for CI. Update CI config.

podspec validation has failed because of the prototype warnings. 😬


Drag & Drop Text Snippets

The problem
Determination messages take up a long time to write and are not standardized in content or tone across different round leads.
When round leads write a determination message, they often copy & paste standard texts from a shared document. This additional step can be frustrating and time-consuming.
These standard texts ("templates") could also be better grouped/tagged.
Possible solution
An interface element that allows round leads to drag and drop standard texts and then edit them in the input interface.

Requirements

Drag and drop, perhaps preview of full-text upon hover
Allows for some form of grouping (colors, dropdown filter etc.)
Has to paste into input field and allow for editing

Priority

Low priority (annoying, would be nice to not see)

Affected roles

Reviewers

Ideal deadline
New review and determination forms ship on July 1, but this doesn't need to be part of that release.

(Just to be sure: this is only a murky idea, we think it could greatly benefit determination message writers, but want to hear what you all think!)

Related #2080 #721 #1948

@ei8fdb potentially applicable to your current work. Thanks!


Scalar parser

This is a rebased version of https://github.com/chyh1990/yaml-rust/pull/37

In the last commit I replaced the ... inclusive patterns with ..= since it's getting deprecated in 2 weeks. It got stabilized in rust 1.26 in may 2018, but appveyor and travis are using rust 1.17 and 1.24.
If supporting such old compilers is a priority, the last commit should be skippable.

I updated the tag directive-handling code to not rely on NLL (it uses an intermediate "Action" enum to separate the read-borrowing peek from the write-borrowing update), and it now compiles on rust 1.27.0 (because of dyn syntax) and runs tests on 1.28.0 (because of a slice::from_ref in regex-syntax).

I tested this with a naive rebase against upstream master ( 0ecbc23 ) this morning, it still passes all tests (there's a dead code warning). This functionality would be quite useful for me personally, but unfortunately I'm still rather new to rust so I'm unable to adequately review the code. I'd be willing to provide a bounty for successful merger with upstream though - 100 AUD.

What would it take to push this forward?

If needed I would be willing to pick up where this left off

FWIW I've merged this PR into my fork: https://github.com/davvid/yaml-rust


The difference between minimum and maximum values should be at least 4

Hello!
Not sure if its a bug but it is worth to notice:
create sequence seq0
    MINVALUE -10000
    start -10000
    MAXVALUE<PHONE_NUMBER>854775800
    CACHE 1;

SELECT permute_nextval('seq0'::regclass,<PHONE_NUMBER>12345 ) FROM generate_series(-10000,-7000);

ERROR:  sequence too short to encrypt.
HINT:  The difference between minimum and maximum values should be at least 4.

If MINVALUE = 0 everything is ok.

It's an integer overflow bug when checking the sequence's range. I've just committed a fix, thanks for the report!

Fixed by 11c1f86


KubeJS Wiki with Full Version

Your wiki was last updated 153 days ago. I would like to know when you will finish the Docs or in how many days it will be updated? Or what plans do you have for the future?

The wiki was last committed to yesterday (abed77d95fb0d28e99e7eaa241921d08e228daf7).
The ETA for when it is done is the heat death of the universe, because KubeJS, like minecraft, is ever evolving and needs new documentation for new updates.

The wiki was last committed to yesterday (abed77d). The ETA for when it is done is the heat death of the universe, because KubeJS, like minecraft, is ever evolving and needs new documentation for new updates.

can i ask? https://kubejs.com/wiki/events/ServerEvents have any update? or it have fully complete in old version?


can generate-tarballs exclude webui

Is your feature request related to a problem? Please describe.
A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]
tag v2.0.0
webui building cost much time
Describe the solution you'd like
A clear and concise description of what you want to happen.
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Urgency
Explain why the feature is important
Additional context
Add any other context or screenshots about the feature request here.

@zrss Thanks for this issue! Could you fill out all the sections in the description?

@zrss Currently, generate-tarballs is used to bundle Alluxio binaries into a full release for end-users. For this reason, it wouldn't make sense to remove the webui as part of this build. Internally it runs this command with some added profiles.
mvn -T 2C -am clean install -DskipTests -Dfindbugs.skip -Dmaven.javadoc.skip -Dcheckstyle.skip
If you want to skip building the web-ui, I would add "-pl webui" and run the mvn command locally in the root of the alluxio repo. I've tried adding the arguments to the generate tarball script, but it seems that the generate-tarball script doesn't parse the argument correctly for mvn.

thx ns1123. we deploy the alluxio by container in k8s, or could we give a doc about building a docker image of alluxio after the mvn building ?
currently, i just see the generate-tarballs can build a tarball in a convenient way then untar this tarball to docker image for image building

@zrss I've raised a PR for your change. I'm not sure if it will get accepted, but if not, you can checkout the file from my repo and add the flag for skipping the webui stage. Testing shows that the flag shaves off 2 minutes from the build time.

oh thx @ns1123  ~ i can borrow your mind and apply to my local alluxio

Glad to help :) If this solves your problem, remember to close this issue.


your compose solution is actually working?

Hi there,
I tried your docker-compose and see several things

it might not work as it is because your nginx only filter ext .php and pass to the php fpm container however serving wordpress static files like css, js nginx has no clue. Thus my installation wizard display in browser is strange (no style at all)
I need to fix the wordpress permissions by execting into teh ct and run

chown -R www-data:www-data /var/www
find /var/www/ -type d -exec chmod 0755 {} ;
find /var/www/ -type f -exec chmod 644 {} ;

Not sure if this is using podman-compose is the problems but I dont think the first issues is due to that as it wont be a different if run on docker or podman.
I dont know how to fix that cleanly yet, might be mounting the first container volume into the nginx in full (rather than only the upload folder)

Hi,
I think the volumes_from line should be mounting the static files also into the nginx container. Only .php files are passed to the wordpress container, this is correct. Static files are supposed to be served directly from the nginx container. Perhaps something changed in later versions of the wordpress container which breaks it.
However I would not recommend using these projects anymore. The sqlite integration plugin has not been updated in 6 years and is closed. I myself also switched to a MariaDB.

Hi,
It is just for a small site so might be ok with sqlite3 however I will take your advice and see if security wise the plugin is good enough.
For your docker-compose the volume_from is fine however it requires files of the whole /var/www/html, and the volume form wont cover that. Mine one look like this and it works nicely
version: '2'
services:
    wp:
        image: milanb/wordpress-sqlite
        environment:
          - WP_HOME=https://store.kaykraft.org
          - WP_SITEURL=https://store.kaykraft.org
        volumes:
            - db:/var/www/db
            - uploads:/var/www/html/wp-content/uploads
            - webroot:/var/www/html
        restart: always
    http:
        image: milanb/wordpress-nginx
        links:
            - wp:wordpress
        volumes_from:
          - wp
        volumes:
          - ./ssl:/etc/nginx/ssl
          - ./conf.d:/etc/nginx/conf.d
        ports:
            - "443:443"
        restart: always
volumes:
    db:
        external: false
    uploads:
        external: false
    webroot:
        external: false


Parsing strategy .optional

I wanted to add a —draft option to my command but I wanted to be able to optionally specify the draft text i.e. —draft or —draft Entwurf. Unfortunately I couldn't think of a way to handle it.
I think that there is a case for having optional arrays although I think there that specifying a minimum and maximum number of elements would be better.
I don't think that there is a good way of handling this with standard types so maybe a custom type should be a requirement, one that could take an optional value as an argument.

After giving it some thought I think that rather than flag and option a minimum and maximum number of arguments is more flexible. A flag is then 0…0, an option is 1…1, what I want is 0…1.

Sorry, I'm not sure that I understand you.
You can use Flag like 0 or 1 and Option like 0 or 1 as well. But also, Option can be an array. So it will be a collection with zero or more elements.
But I need the third state of Option:
Command                         | Swift
---------------------------------------------------------------------------------
rugby				| let focus: [String]? = nil
rugby --focus			| let focus: [String]? = []
rugby --focus Alamofire SnapKit | let focus: [String]? = ["Alamofire", "SnapKit"]

Not 0 or 1 but 0 to 1, it is a closed range of the number of arguments allowed, a flag has none or 0…0, an option 1…1 or 1… depending on your parsing strategy in the current setup.

I found that there is init with initial: nil. But when I run my command its demands this argument.
https://github.com/apple/swift-argument-parser/blob/992a7451d546ede8825284530ffdb541c9a316ef/Sources/ArgumentParser/Parsable Properties/Option.swift#L496-L509

This isn't currently supported, mostly because there isn't a great way to model it cleanly. It might make sense as a double Optional, but those are awkward to work with. Maybe we could add an Optional-esque type:
struct Example: ParsableCommand {
    @Option var kind: OptionValue<String>?

    func run() {
        print(kind as Any)
    }
}

$ example
nil
$ example --kind
.flagOnly
$ example --kind foo
.value(foo)

Still strikes me as a bit gross… Any other ideas?

Hey, @natecook1000 why we can use it like this:
@Option var kind: [String]?

Like it already works with non-collection types.
I can try to make a draft if it suits.
I see that need to change the array parsing strategy, but maybe there are some pitfalls.

I found another way to do it

@msalmonse Mind sharing your solution? :)

I wrote my own parser with a min and max count for parameters. I fixed the negative numbers problem as well and threw in environmental variables.
SAP is very good but… I remember a comparison of Windows with Linux: in Windows the easy is easy and the difficult is impossible while in Linux it's all just hard.

@msalmonse Thank you for the reply!


Will @vue/theme be the theme for vitepress?

Hello~
The new vue3 documentation will be the best development documentation.
Thanks to the vue team for bringing us such a good documentation experience.
Also want to know, @vue/theme will optimize the hardcoded logic specific to Vue documentation to become the official vitepress theme?
Thanks!

For example: https://github.com/vuejs/theme/blob/main/src/vitepress/components/VPNavBarTitle.vue

In the README:

Please note this theme is solely dedicated for vuejs.org and isn't meant to be used as a content-neutral theme. It does not follow semver and may contain hard-coded logic specific to the Vue documentation.

@kiaking is designing a new theme for VitePress tho 👀

In the README:

Please note this theme is solely dedicated for vuejs.org and isn't meant to be used as a content-neutral theme. It does not follow semver and may contain hard-coded logic specific to the Vue documentation.

@kiaking is designing a new theme for VitePress tho 👀

Thank you for your answer.

@antfu Thank you very much.


[Update Request]: upgrade Firefox ESR

What type of update are you requesting?
A new version of an existing package
Current Package Identifier
Firefox.ESR
Package Version
115.3.1
Please describe the changes you would like to see
There is a new release 115.3.1.
download page:
https://www.mozilla.org/en-US/firefox/enterprise/#download

Close with reason: Resolved in #122067

Completed. Could someone please close


On research finished crashes when some entities are open

The LRM mod crashes when the UI for the SE Satellite Receiver, LTN Combinator, Crafting Combinator, or any other entities are open when the research finishes that do not have a logistic point.

I have a potential fix for this that I am currently going to test a little bit first but I will create a PR for what I currently have

I implemented  the same logic in some more places after your hint - should work fine now.
Thanks again!


[Bug] Possible incompat with other mods [Connector Forge 1.20.1]

What happened?
Creates a failed to join error when loading into a world, by itself it works just fine so I'm assuming it's some mod conflict
I'll do a large sample binary search when I have the free time but hopefully latest can point some things out.
additional mods that aren't in the pastebin due to word count limit:

Terrastorage Version
1.1.4
Fabric API Version
Forgified Fabric API 0.92.2
Minecraft Version
1.20.1
Relevant Log Output
https://mclo.gs/H4tP8N5

Other Mods
https://pastebin.com/yZPRdx5H

Hi! thanks a lot for taking the time to report this issue, and for offering to do a binary search on the mods. That would be extremely helpful in pinpointing the root cause.
I'd love to be able to test things out on my own, but with the number of mods you're using, gathering them all manually would be quite the challenge. If it happens to be part of a modpack, that would simplify things quite a bit. If not, would it be possible for you to upload a zip or rar archive of your mods to a file-sharing platform like Google Drive or Mega? this would make testing on my end a lot more doable.
Let me know if that's possible, and thanks again for your time in reporting this bug.

config conflict too perhaps? what if you sent those too...

Hi! thanks a lot for taking the time to report this issue, and for offering to do a binary search on the mods. That would be extremely helpful in pinpointing the root cause.
I'd love to be able to test things out on my own, but with the number of mods you're using, gathering them all manually would be quite the challenge. If it happens to be part of a modpack, that would simplify things quite a bit. If not, would it be possible for you to upload a zip or rar archive of your mods to a file-sharing platform like Google Drive or Mega? this would make testing on my end a lot more doable.
Let me know if that's possible, and thanks again for your time in reporting this bug.

https://drive.google.com/file/d/1eHjiq6fZF8WtXmm8zNRWO2curFUEGHz2/view?usp=sharing here you go, it's quite big, my bad. I know the other arrows are there but I intended to disable them in favor of your mod (if I can find which it is). But I know whichever mod it is, it's not the problematic one

Hi again! Thanks for uploading the mods and configs. I tested it on Forge 1.20.1 - 47.3.0, and everything seemed to work fine on my end. I created and joined a new world without any errors during loading.
The issue might be specific to your world setup. Could you try creating a new world to see if the error still happens there? If possible, a screenshot of the error screen might also help pinpoint the problem.
And don’t worry about the mod list size, it's totally fine. Just let me know if there’s any other detail you think might help.

I don't know what to say, it was happening on every world I created (failed to join server with a long error) and only with this mod installed and I could replicate it perfectly each time. Now everything is seemingly fine? Idk if this is more concerning or a blessing but I'll close this issue for now

Glad to hear everything’s working now! It’s possible the initial issue resolved itself after a PC restart or due to some other unrelated factor. If you ever run into this issue again, please feel free to reopen this issue, have an awesome day!


Followers: Can follow the same person more than once

if possible, this should be disabled/ blocked


Need help with bodyFilter

Hi ! Although it is not an issue, just wondering if can be helped. I am trying to mask one of the field in my request with the name as Value, which works fine with below.
Properties is my set with that field's name.
However I have another field with same name in my Json request and it is masking that too. Is it possible if I can specify or mark somehow that which field needs to be masked?
Logbook.builder().sink(new DefaultSink(new SplunkHttpLogFormatter(), new DefaultHttpLogWriter()))
.bodyFilter(JsonBodyFilters.replaceJsonStringProperty(properties, "")

Also, I tried using JsonPath like below, which worked only for desired object. However this object is not present in my HttpResponse hence while displaying response logbook is throwing exception.
.sink(new DefaultSink(new SplunkHttpLogFormatter(), new DefaultHttpLogWriter()))
.bodyFilter(jsonPath("$.object.value").replace("*****"))
.build();

As far as I know there is not build in way to only filter the field some of the time (other than using JsonPath). Your can probably implement a custom filter to archive this.
JsonPath has to be configured to allow (not throw an exception) "missing" paths. I guess logbook does not do this, therefore you get an error.

Thank you, This helps!


Pause / Resume

Add Pause / Resume.

In UI use switch toggle
XMRig has RPC API for pause / resume, we can pause the miner and keep the process runnig

Pause
curl -X POST http://<IP_ADDRESS>:8088/json_rpc -H 'Content-Type: application/json' -H "Authorization: Bearer 1234" -d '{"method":"pause","id":1}'
Resume
curl -X POST http://<IP_ADDRESS>:8088/json_rpc -H 'Content-Type: application/json' -H "Authorization: Bearer 1234" -d '{"method":"resume","id":1}'

/cib


I hava the same issues with #2

zhangsan

##zhangsan

zhangsan
zhangsan

I just for test

Hi, we understand you might have been learning GitHub lately, but we have quite a few subscribers to this repo that might be receiving these messages. Please don't learn with this repo in a way that creates pull requests/issues/etc that would notify everyone else. You're more than welcome to create a free new github repo of your own to try things out.