Setting of super.named parameter is recognized as unused or incorrect

Snippet
class A{
  final bool b;

  A({this.b = false});
}

class _B extends A{
  _B({super.b = true});
}

class _C extends A{
  _C(): super.b = true;
}

void fn(){
  _B();
  _C();
}

What I want
I want to avoid _B({b: true}) when I can set in constructor and to pass analyzer lints
Problem
B.b: is unused. (Okay. It looks like expected. But it has not hotfix by analyzer and I create hotfix as _C class)
C.b: error: 'b' isn't a field in the enclosing class.

Version
Dart SDK version: 3.5.4 (stable) (Wed Oct 16 16:18:51 2024 +0000) on "macos_arm64"

For C this should work:

The code is recognized as incorrect. It is incorrect, the syntax, super.id = e in an initializer list, is not allowed, and it's not even clear what it should mean.
It looks like it assigns a value to a superclass instance variable, but that's not allowed and likely never will be. It breaks abstraction to rely on a superclass implementing a getter using an instance variable. The subclass only less that the superclass has a getter and a constructor parameter, the rest is implementation details.
It can mean passive the expression's value to the super-constructor parameter of the same name, but that's not how you pass parameters (and even if it was made to work, it won't work with positional parameters).
If that's what you want, the syntax is _C(): super(b: true);.


Fix bug where the configuration file can not be serialized

The "resources" key in the pulse-api.php config file is a Collection instance.
On running config:cache an error is thrown: "Your configuration files are not serializable."
This PR changes the "resources" value to an array.

PR for existing issue
https://github.com/robertogallea/pulse-api/issues/5

Nice one, thanks!


Submission Audit Query (CAT)

Queries can be developed instead of developing functionality: 24 hrs
Based on a meeting held with Chris on 1/25/24, this functionality will be included in the initial 2.0 release.
Submission Audit Query

Stalled submissions
Unloaded submissions
Missing events

Linked to #6205

Created ticket #6381 to create scripts instead of developing functionality.


Team-members sharing the same dev backend

I'm following the instructions here:
https://aws-amplify.github.io/docs/cli-toolchain/quickstart#team-members-sharing-the-same-dev-backend
When the developer follows the instructions he is taken to the AWS console but he is logged into his account.
I couldn't find anything in the documentation to complete the setup.
I copied the IAM user that was created when I configured my amplify account and created that for him.
We ended up editing the files directly in the .aws directory.
How is this supposed to be done? Does amplify required developers to be added to the account?

There are two ways to do this.

In the IAM console of the team's account, add an IAM user for the team member, and the team member should log into the aws using the team IAM user's credentials. Then the team member should put the credentials into the configuration files  ~/.aws/credentials and ~/.aws/config as a profile. Run amplify configure project on the project and select this profile for aws accesses.

If an outside contributor is also working on the same project, and he needs access to the same backend, you can create a role for the contributor in the IAM. There's a detailed write up for it in our document on how this should be done: https://aws-amplify.github.io/docs/cli-toolchain/usage#iam-roles--mfa

Thanks @UnleashedMind. Closing.


Sparcv9_solaris builds broken

Sparcv9_solaris builds appear to be broken since Dec 7th with the following errors:
When it fails quickly:
07:21:09  Execution error: hudson.remoting.RequestAbortedException: java.io.IOException: Unexpected termination of the channel
[Pipeline] echo
07:21:09  hudson.remoting.RequestAbortedException: java.io.IOException: Unexpected termination of the channel
07:21:09  	at hudson.remoting.Request.abort(Request.java:345)
07:21:09  	at hudson.remoting.Channel.terminate(Channel.java:1080)
07:21:09  	at hudson.remoting.SynchronousCommandTransport$ReaderThread.run(SynchronousCommandTransport.java:90)
07:21:09  	Suppressed: hudson.remoting.Channel$CallSiteStackTrace: Remote call to build-siteox-solaris10u11-sparcv9-1

When it builds for hours, then fails differently:
19:39:55  Execution error: org.eclipse.jgit.errors.InvalidObjectIdException: Invalid id: b8f651cfbf403edfdf136c1587b4d9bf1d681084
19:39:55  Process leaked file descriptors. See https://www.jenkins.io/redirect/troubleshooting/process-leaked-file-descriptors for more information
[Pipeline] echo
19:39:55  org.eclipse.jgit.errors.InvalidObjectIdException: Invalid id: b8f651cfbf403edfdf136c1587b4d9bf1d681084
19:39:55  Process leaked file descriptors. See https://www.jenkins.io/redirect/troubleshooting/process-leaked-file-descriptors for more information
19:39:55  	at org.eclipse.jgit.lib.ObjectId.fromString(ObjectId.java:203)

To make it easy for the infrastructure team to repeat and diagnose, please
answer the following questions:

test suite/name (e.g, BUILD_LIST, TARGET, CUSTOM_TARGET)?
a link into recent Test_ job on https://ci.adoptopenjdk.net which showed the failure: Any recent build https://ci.adoptopenjdk.net/job/build-scripts/job/jobs/job/jdk8u/job/jdk8u-solaris-sparcv9-temurin/
Hyperlink to re-run in Grinder:
Is there an existing issue elsewhere covering this? No
Which machine(s) does it work on? there is only one build machine
Which machine(s) does it fail on? build-siteox-solaris10u11-sparcv9-1

Any other details:

That timing may be coincident with trying to use a Zulu 11 JDK for the jenkins agent. Although the most recent build appears to have been ok

I've kicked off another at https://ci.adoptopenjdk.net/job/build-scripts/job/jobs/job/jdk8u/job/jdk8u-solaris-sparcv9-temurin/224 (I've also switched the test system over to Zulu11 to see if that shows any problems after the switch)
@steelhead31 if you're able to keep an eye on this for me since I'm not 100% today that would be appreciated.

A quick SAR shows the system is at 99% usage..
bash-3.2# sar 1 10
SunOS build-siteox-solaris10u11-sparcv9-1 5.10 Generic_147147-26 sun4v    12/21/2022
00:18:28    %usr    %sys    %wio   %idle
00:18:31       3      97       0       0
00:18:34       2      98       0       0
00:18:36       1      99       0       0
00:18:39       2      98       0       0

Of the list at https://ci.adoptopenjdk.net/label/solaris||sw.os.sunos/ all active ones are using Liberica 11, although https://ci.adoptopenjdk.net/computer/test-equinix_esxi-solaris10-x64-1-NEW has it in /usr/lib/jvm/jdk-11.0.17 instead of /usr/lib/jvm/bell-jdk-11.0.17 like the others - I have now corrected that for clarity and consistency so this can "now mission" for this issue can now be considered complete.


Hot Reloading

Need to figure out how to get hot-reloading working with current configuration. I am thinking this would be done in the WebpackShellPluginNext near the end of the webpack.config.js I just haven't found the right way to get it to work.

What kind of HMR do you need? styles?

Hey @felixmosh thanks for helping out.
The immediate issue I am looking for is when we are working in development mode we look for a file change on .liquid files and then re-run the scripts.

build tailwindcss
build webpack
deploy changed files to theme
watch for changes

I've managed to start a dev env with preview of the template on Shopify.
I see that if i change . liquid file, it won't re upload it.
Is it part of the HMR task?

Correct @felixmosh it would be great to watch for changes on .liquid files and re-upload the changed file as part of the HMR task.

This has been fixed #19


Istio installed with istioctl results in an inaccessible istio-ingressgateway

Describe the bug
I am not sure if this is an orbstack or istio problem, but I am unable to reach the declared LoadBalancer IP of the istio-ingressgateway when deploying with istioctl per the orbstack instructions.  The same works fine when deploying via helm-chart, and other ingresses (e.g. NGINX).
To Reproduce
Running orbstack v1.8.2 and istio 1.24.1:

create a fresh k8s cluster in orbstack
run istioctl install as per the orbstack instructions

istioctl install --set values.cni.cniBinDir=/var/lib/rancher/k3s/data/current/bin/ --set values.cni.cniConfDir=/var/lib/rancher/k3s/agent/etc/cni/net.d

observe that a LoadBalancer has been registered

$ kubectl get svc -A
NAMESPACE      NAME                   TYPE           CLUSTER-IP        EXTERNAL-IP    PORT(S)                                      AGE
default        kubernetes             ClusterIP      <IP_ADDRESS>   <none>         443/TCP                                      60s
kube-system    kube-dns               ClusterIP      <IP_ADDRESS>   <none>         53/UDP,53/TCP,9153/TCP                       55s
istio-system   istiod                 ClusterIP      <IP_ADDRESS>   <none>         15010/TCP,15012/TCP,443/TCP,15014/TCP        18s
istio-system   istio-ingressgateway   LoadBalancer   <IP_ADDRESS>   <IP_ADDRESS>   15021:30459/TCP,80:30245/TCP,443:30141/TCP   14s

run nmap k8s.orb.local

Expected behavior
The nmap should show that ports 80/443 are open.  However, with the above sequence, we get
$ nmap k8s.orb.local
Starting Nmap 7.95 ( https://nmap.org ) at 2024-11-29 18:03 EST
Strange SO_ERROR from connection to <IP_ADDRESS> (50 - 'Network is down') -- bailing scan
QUITTING!

Diagnostic report (REQUIRED)
OrbStack info:
Version: 1.8.2
Commit: 44cf8269710cd29802bfee176a7cec65b55cc822 (v1.8.2)
System info:
macOS: 15.1.1 (24B2091)
CPU: arm64, 16 cores
CPU model: Apple M4 Max
Model: Mac16,5
Memory: 128 GiB
Full report: https://orbstack.dev/_admin/diag/orbstack-diagreport_2024-11-29T23-06-40.404581Z.zip
Screenshots and additional context (optional)
If we substitute helm-chart installation for istio for istioctl, everything works as expected
kubectl create namespace istio-system
helm -n istio-system install istio-base istio/base
helm -n istio-system install istiod istio/istiod
helm -n istio-system install istio-gw istio/gateway

$ helm ls -A
WARNING: Kubernetes configuration file is group-readable. This is insecure. Location: /Users/ghaskins/.kube/config
WARNING: Kubernetes configuration file is world-readable. This is insecure. Location: /Users/ghaskins/.kube/config
NAME      	NAMESPACE   	REVISION	UPDATED                             	STATUS  	CHART         	APP VERSION
istio-base	istio-system	1       	2024-11-29 18:10:18.357209 -0500 EST	deployed	base-1.24.1   	1.24.1
istio-gw  	istio-system	1       	2024-11-29 18:10:20.749453 -0500 EST	deployed	gateway-1.24.1	1.24.1
istiod    	istio-system	1       	2024-11-29 18:10:19.099704 -0500 EST	deployed	istiod-1.24.1 	1.24.1

We may now run the nmap and obtain the expected results
$ nmap k8s.orb.local
Starting Nmap 7.95 ( https://nmap.org ) at 2024-11-29 18:10 EST
Nmap scan report for k8s.orb.local (<IP_ADDRESS>)
Host is up (0.00010s latency).
Other addresses for k8s.orb.local (not scanned): fd07:b51a:cc66:0:a617:db5e:c613:f803
Not shown: 998 closed tcp ports (conn-refused)
PORT    STATE SERVICE
80/tcp  open  http
443/tcp open  https

Nmap done: 1 IP address (1 host up) scanned in 0.26 seconds

I am open to the notion that I am doing something wrong.  However, I found this while trying to deploy my app, which indeed has a virtual gateway.  This app generally works elsewhere but failed when I tried adding orbstack to the mix.  I didn't include a VS in the reproduction steps as it didn't seem material, but perhaps that was a tactical error.
I confirmed the wiring between the ingress-pod/gateway/virtualservice objects, and they all appear correct, yet I cannot reach the 80/443 ports on the ingress. If there are any suggestions to try, please let me know.

Update: I'm sorry, you were correct.  Your comment made me realize a slight incongruence in the labels between the ingress <-> gateway objects that effectively resulted in no gateways/virtualservice mappings being picked up.  It's working now!  Thank you.


Vickel: Watch number<PHONE_NUMBER>8

Vickel requests the watch of the watch_keyword number<PHONE_NUMBER>8. See the MS search here and the Stack Exchange search in text, in URLs, and in code.

number<PHONE_NUMBER>8 has been seen in 0 true positives, 0 false positives, and 0 NAAs.

Approved by Jeff Schaller in Charcoal HQ


Decimal object created without initial value throws an exception during comparison

Expected
No JavaScript error
Actual
Uncaught TypeError: this.value.comparedTo is not a function
Steps To Reproduce
Live Test Case
public class App
{
    [Ready]
    public static void Main()
    {
        decimal x = new Decimal();
        int y = 1;

        Console.WriteLine(x > y);
    }
}

Fixed in https://github.com/bridgedotnet/Bridge/pull/1252/commits/da619dcaeab826964f314dadba812bb232a51f9f


Added Conda Environment Yaml for MacOS

I have never been able to successfully build the Olympia Conda environment on my Mac, so I'm checking in a yaml that works for me. It's derived from the Sparta Conda environment with some modifications (i.e. removing version specifications from a few libraries like boost).

Sorry I can't try this out, I don't use macs.

I will wait to merge until another MacOS user can test this out.

Ugh, don't be too frustrated with this, but we actually have a MacOS environment yaml file here: https://github.com/riscv-software-src/riscv-perf-model/blob/master/.github/workflows/conda/macos_env.yml
Moving forward, can you either merge your new file into it OR have the GitHub CI use yours?  This is the line in CI that you'd update: https://github.com/riscv-software-src/riscv-perf-model/blob/791d133283861273f3649c0e05aedccd946b29a3/.github/workflows/macos-build.yml#L72


[Bug] weblog:lol Truncation appendix: doesn’t appear on tag pages

Bug Description
The configuration setting Truncation appendix: that’s part of titleless post handling appears in {post-list} but not in post lists on tag pages.
Steps to Reproduce
No response

Fixed! Sorry about that.

Oh, hold up. I did fix it, but haven't pushed the change yet. Stand by.

Should be good now! Sorry for the false alarm.


PARAM_PAGINGACTION "_getpages" not being recognized correctly.

NOTE: Before filing a ticket, please see the following URL:
https://github.com/jamesagnew/hapi-fhir/wiki/Getting-Help
Describe the bug
When coding and running against the PARAM_PAGINGACTION "_getpages" query-string name like this:
Patient/?_getpages=aaaaaaaa-d6e3-4ab3-8665-f131941f323f&_getpagesoffset=50&_count=25&_pretty=true&_bundletype=searchset

Note, the above query is the result of a pagination search.
Getting postman error like this:
{
    "resourceType": "OperationOutcome",
    "issue": [
        {
            "severity": "error",
            "code": "processing",
            "diagnostics": "Invalid request: The FHIR endpoint on this server does not know how to handle GET operation[Patient/] with parameters [[_getpages, _pretty, _count, _getpagesoffset, _bundletype]]"
        }
    ]
}

To Reproduce
Steps to reproduce the behavior:
Create a Patient Resource Provider (r4)
import org.hl7.fhir.r4.model.Patient;

public class MyPatientResourceProviderBase implements IResourceProvider {

    /**
     * The getResourceType method comes from IResourceProvider, and must be overridden to indicate what type of resource this provider supplies.
     */
    @Override
    public Class<Patient> getResourceType() {
        return Patient.class;
    }

Add this @Search  method:

import static ca.uhn.fhir.rest.api.Constants.PARAM_BUNDLETYPE;
import static ca.uhn.fhir.rest.api.Constants.PARAM_COUNT;
import static ca.uhn.fhir.rest.api.Constants.PARAM_PAGINGACTION;
import static ca.uhn.fhir.rest.api.Constants.PARAM_PAGINGOFFSET;
import static ca.uhn.fhir.rest.api.Constants.PARAM_PRETTY;

    @Search
    public IBundleProvider findPatientsByPaging(@OptionalParam(name = PARAM_PAGINGACTION) String theGetPages,
                                                 @OptionalParam(name = PARAM_PAGINGOFFSET) String thePagingOffSet,
                                                 @OptionalParam(name = PARAM_COUNT) String theCount,
                                                 @OptionalParam(name = PARAM_PRETTY) String thePretty,
                                                 @OptionalParam(name = PARAM_BUNDLETYPE) String theBundleType
    ) {

return null;

}

Current Version:
Expected behavior
PARAM_PAGINGACTION does not error out.
Environment (please complete the following information):
Gradle import/hapi fhir server version

implementation group: 'ca.uhn.hapi.fhir', name: 'hapi-fhir-server', version: '5.1.0'

Additional context
Note, if I change it to
    @Search
    public IBundleProvider findPatientsByPaging(@OptionalParam(name = "_getpagesx") String theGetPages,
                                                 @OptionalParam(name = PARAM_PAGINGOFFSET) String thePagingOffSet,
                                                 @OptionalParam(name = PARAM_COUNT) String theCount,
                                                 @OptionalParam(name = PARAM_PRETTY) String thePretty,
                                                 @OptionalParam(name = PARAM_BUNDLETYPE) String theBundleType
    ) {

and slightly adjust the request to:
Patient/?_getpagesx=aaaaaaaa-d6e3-4ab3-8665-f131941f323f&_getpagesoffset=50&_count=25&_pretty=true&_bundletype=searchset

It works fine.
Something about that
PARAM_PAGINGACTION
Note sure if these being so close is an issue:
public static final String PARAM_PAGINGACTION = "_getpages";
public static final String PARAM_PAGINGOFFSET = "_getpagesoffset";

HAPI FHIR paging URLs have to be executed against the base of the server, and have to come from existing search result bundles (i.e. be created by HAPI FHIR). In other words, it is not valid to create your own.
Did HAPI FHIR give you a paging URL with Patient/? in it?

(Note that this is FHIR's rule, not HAPI's)

Ok, maybe I'm not understanding something then.
Here is my "regular" search...but I've added pagination.
myr4fhirserver/Patient/?
And I get back:
{
"resourceType": "Bundle",
"id": "5555f71b7469458cb34286e9638febf9",
"meta": {
"lastUpdated": "2020-11-09T17:36:50.300-05:00"
},
"type": "searchset",
"total": 100,
"link": [
{
"relation": "self",
"url": "/myr4fhirserver//Patient/"
},
{
"relation": "next",
"url": "/myr4fhirserver/?_getpages=aaaaaaaa-db90-46c7-8ca6-b3eb565249d5&_getpagesoffset=25&_count=25&_pretty=true&_bundletype=searchset"
}
],
"entry": [
So I don't take the "next" .. and run it through /Patient/ again?
Is that my miscue?

I have injected this into my "context" via xml
    <bean id="zzzFifoMemoryPagingProviderBean"
          class="ca.uhn.fhir.rest.server.FifoMemoryPagingProvider">
        <constructor-arg value="10"/>
    </bean>

which is the (close xml version of
https://hapifhir.io/hapi-fhir/docs/server_plain/paging.html
  FifoMemoryPagingProvider pp = new FifoMemoryPagingProvider(10);
  pp.setDefaultPageSize(10);
  pp.setMaximumPageSize(100);
  setPagingProvider(pp);

and am calling
RestfulServer.setPagingProvider (with the above injection)

You would just run it through / (i.e. your server base URL) again, not
[baseUrl]/Patient/.
That's really weird through that HAPI isn't already adding a base URL to
those next/self links. Are you using a custom server base URL strategy or
something?
On Mon, Nov 9, 2020 at 5:40 PM granadaCoder<EMAIL_ADDRESS>wrote:

Ok, maybe I'm not understanding something then.
Here is my "regular" search...but I've added pagination.
myr4fhirserver/Patient/?
And I get back:
{
"resourceType": "Bundle",
"id": "5555f71b7469458cb34286e9638febf9",
"meta": {
"lastUpdated": "2020-11-09T17:36:50.300-05:00"
},
"type": "searchset",
"total": 100,
"link": [
{
"relation": "self",
"url": "/myr4fhirserver/
/Patient/" }, { "relation": "next", "url": "/myr4fhirserver/
?_getpages=aaaaaaaa-db90-46c7-8ca6-b3eb565249d5&_getpagesoffset=25&_count=25&_pretty=true&_bundletype=searchset"
}
],
"entry": [
So I don't take the "next" .. and run it through /Patient/ again?
Is that my miscue?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/jamesagnew/hapi-fhir/issues/2165#issuecomment-724321430,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AA2N7HII7A33NJ5L3WXXD2TSPBVVVANCNFSM4TP5MAUQ
.

public final class FullOrShortServerAddressStrategy implements IServerAddressStrategy {
private boolean showFullUrl = false;

You've done it again!
Tomorrow I will see if my custom FullOrShortServerAddressStrategy is screwing things up and report back.

Ok, I temporarily removed my custom address strategy.
and now I see a "healthier" NEXT link
        "relation": "next",
        "url": "http://me.com/myr4fhirserver?_getpages=aaaaaaaa-6b26-4d81-a398-e2d154a59e09&_getpagesoffset=25&_count=25&_pretty=true&_bundletype=searchset"
    }

and now I "see" better how the _getpages is off the root.
S'oh boy.  Now I gotta tell update my teams.
It is still weird to me that it isn't off of /Patient/

Closing this as it appears solved. But of course please feel free to comment if it is not.

Everything is ok.  I had a small bug in my IServerAddressStrategy. (That's where that "*" was coming from).
Thanks for your quick feedback.
Once again, this was not a real bug, but a lack of understanding on my part.
To finish out the conversation...here is my return payload:
and the "next" link (and later one, the "previous" links) work correctly.
{
"resourceType": "Bundle",
"id": "abcblahblahblah1228be50d",
"meta": {
"lastUpdated": "2020-11-10T16:06:18.775-05:00"
},
"type": "searchset",
"total": 100,
"link": [
{
"relation": "self",
"url": "http://localhost:8080/myr4fhirserver/Patient/?_count=13"
},
{
"relation": "next",
"url": "http://localhost:8080/myr4fhirserver?_getpages=bbbbcd07-6bfe-4f0a-bc17-e5035746261a&_getpagesoffset=13&_count=13&_pretty=true&_bundletype=searchset"
}
],
"entry": [
{
"fullUrl": "http://localhost:8080/myr4fhirserver/Patient/14",
"resource": {
"resourceType": "Patient",

PS, I wrote a "PayAsYouGo" IBundleProvider.  with an injectable method to handle the "getting".
#foodForThought

import ca.uhn.fhir.model.primitive.InstantDt;
import ca.uhn.fhir.rest.api.server.IBundleProvider;

import org.hl7.fhir.instance.model.api.IBaseResource;
import org.hl7.fhir.instance.model.api.IPrimitiveType;

import javax.annotation.Nonnull;
import java.util.Collection;
import java.util.Date;
import java.util.List;
import java.util.function.BiFunction;
import java.util.stream.Collectors;

/**
 * PayAsYouGoBundleProvider allows for an injected method where the method is able to get the subset of items based on fromIndex and toIndex.
 *
 * @param <R>
 */
public final class PayAsYouGoBundleProvider<R extends YouPickSomethingHereReadTheComments> implements IBundleProvider {

    private final String uuid;

    private Integer preferredPageSize;

    /* Must be the TRUE overall size of the resources. */
    private Integer overallSize;

    private IPrimitiveType<Date> publishedDate = InstantDt.withCurrentTime();

    private BiFunction<Integer, Integer, R> pagingDataRetriever;

    /**
     * @param uuid random uuid
     * @param overallItemsCount this must be the actual and true OVERALL count, not just the "current page of data" count
     * @param pagingDataRetriever a BiFunction where the "R" should be List<IBaseResource> or something that can be converted to a List<IBaseResource>
     */
    public PayAsYouGoBundleProvider(String uuid, int overallItemsCount, BiFunction<Integer, Integer, R> pagingDataRetriever) {
        this.uuid = uuid;
        setSize(overallItemsCount);
        this.pagingDataRetriever = pagingDataRetriever;
    }

    @Override
    public IPrimitiveType<Date> getPublished() {
        return this.publishedDate;
    }

    /**
     * By default this class uses the object creation date/time (for this object)
     * to determine {@link #getPublished() the published date} but this
     * method may be used to specify an alternate date/time.
     */
    public void setPublished(IPrimitiveType<Date> newPublishedDate) {

        this.publishedDate = newPublishedDate;
    }

    @Nonnull
    @Override
    public List<IBaseResource> getResources(int fromIndex, int toIndex) {
        R rItem = this.pagingDataRetriever.apply(fromIndex, toIndex);
        List<IBaseResource> returnItems = rItem; /* OR rItem can be something translate-able to List<IBaseResource> */
        return returnItems;
    }

    @Override
    public String getUuid() {
        return uuid;
    }

    /**
     * Defaults to null.
     */
    @Override
    public Integer preferredPageSize() {
        return preferredPageSize;
    }

    /**
     * Sets the preferred page size to be returned by {@link #preferredPageSize()}.
     * Default is null.
     */
    public void setPreferredPageSize(Integer preferredPageSize) {
        this.preferredPageSize = preferredPageSize;

    }

    /**
     * Sets the total number of results, if this provider corresponds to a single page within a larger search result.
     */
    public PayAsYouGoBundleProvider setSize(Integer theSize) {
        this.overallSize = theSize;
        return this;
    }

    @Override
    public Integer size() {
        return this.overallSize;
    }
}


Require loops possible solution.

Hi @koskimas , I had this require loop issue for a long time and been using the solution #1 from the guide, but today doing a refactor on my code I've managed to fixed it by not using default exports on my Models, so I've created this PR in case you consider it proper to have in the docs, is a short addition to comment that not using default exports may help avoid the issue.
BTW: I love Objection! Congratulations on your work! Is really great!

Coverage increased (+0.2%) to 97.171% when pulling 20c13eee7b9a7fbd774339cf92178ed192dd5ac5 on damian-pastorini:damian/docs-loops-solution into 794a036b6e621af38631378eed85188dfb973d3f on Vincit:master.

This is not true.


Applying hash to URL

It would be nice if upon clicking a button, it would apply the data-filter value to the url string. That way, we can easily link to a page that will sort automatically via the data-filter category.

You can trigger click event.
Try this
      var parts = window.location.hash;
      if (parts) {
        setTimeout( function () { jQuery(parts).click(); }, 950);
      }


[WIP] Next Animation

@tmijieux Changed Animation from a static version to an instance version. Animation is created for MapControl. Disadvantage is, that Animation could only be used from objects, which have access to MapControl. Perhaps you have a better idea.

This is not what I had in mind.
What I was thinking was to keep the Animation class static but remove this list of AnimationEntries. Call AnimationUpdate on those things that are themselves animated. Or better, don't call AnimationUpdate at all but leave it up to the animated item to return it's current animation state given the time of update. This is how the AnimatedFeatures work. There are some things in that part of the code that needs to be improved, but the way animation works is how I want it. You just call GetFeatures at a certain time. The class knows the current time and knows the start and end of the animation so that it can return a feature in the state that corresponds to the animation at that time.
To use the same mechanism for viewport animations and style animations will need more work. I will work on this, but there is little I can do the coming week, this is the busiest time of the year for me.

@pauldendulk But how should this work? You don't want to have a list of animations, but you want to call them from within the render loop. You want a static Animation class, but it should be different between different MapControls.
If you do it on Feature basis, then you have to check each Feature, if there is an animation running. Or you have somewhere a list, which Features are animated. And for navigation you need a different system.
But I have to wait to see your results.

@pauldendulk But how should this work? You don't want to have a list of animations, but you want to call them from within the render loop. You want a static Animation class, but it should be different between different MapControls.

The static Animation class will have no static list of AnimationEntries. The AnimationEntries will be stored more locally. This could be: MapControl, Map, Layer, Feature. I currently playing with a solution where I store it in the Layer.

If you do it on Feature basis, then you have to check each Feature, if there is an animation running.

Yes, pushing it all the way down to the individual feature may be a step too far.

Or you have somewhere a list, which Features are animated. And for navigation you need a different system.

Yes, navigation as it currently works does not fit in this logic. Perhaps animation should be inside the Viewport. Or we should change the parameters to the rendering loop.


Fix Volume calculations

Currently the volume calculations in Augur Node are done by taking the trade amount multiplied by the trade price and summing that with previous volume.
Instead when we update the volumetrics we should increase overall volume by:

(The lowest of makerSharesProvided and takerSharesProvided) * numTicks
makerTokensProvided
takerTokensProvided

Shouldn't this actually be data from https://github.com/AugurProject/augur-core/blob/master/source/contracts/trading/FillOrder.sol#L397 so:
Instead when we update the volumetrics we should increase overall volume by:
(The lowest of makerSharesDepleted and takerSharesDepleted) * numTicks
makerTokensDepleted
takerTokensDepleted
Basically using the amounts that actually got "depleted" / filled instead of the amounts provided (which may not necessarily get used in a trade)?

The actual log data will use the terms:

numCreatorShares
numCreatorTokens
numFillerShares
numFillerTokens


Feature: Install tailwind css

Description
Upgrade the styling of the website with tailwind css
Proposed Solution
Current website uses vanilla css for styling, using Tailwind css will reduce code duplication and help in increasing consistency.
Screenshots
No response
🥦 Browser
Google Chrome
Checklist ✅

[X] I checked and didn't find similar issue
[X] I have read the Contributing Guidelines
[X] I am willing to work on this issue (blank for no)

@akash0708 converting whole css code to tailwind css ? It takes more time

hello @akash0708 I can install the Tailwind CSS so further from here people can use tailwind instead of raw CSS
you can assign this issue to me

Can you assign this to me!

A great issue this is, however, this would take a lot of time so I think we can go step by step. Let me close this issue real quick for now and first discuss how we are thinking to proceed.


SST data process

Hi，
Thanks for your great work and your nice code;
I just can't figure why you use STree.txt and SOStr.txt when process the STS datasets?
why not just use dictionary.txt and sentiment_label.txt ?

Hi,
Thanks for your interests.
The reason is that I once tested the recursive neural model in the project and I kept those module. Therefore, the files to indicate the parsing tree results were needed.


ISLE IDE functionality

Currently ISLE doesn't have any IDE functionality. It would be nice to get the following:

[ ] Syntax highlighting
[ ] Code formatting
[ ] Inline error reporting
[ ] Documentation on hover
[ ] Go to definition
[ ] Go to references
[ ] Show compiled rust source for rule
[ ] Inlay hints

[ ] Types
[ ] Argument names

I would love for an LSP implementation to exist someday for ISLE -- if anyone is looking for a fairly substantial and impactful, but well-defined and orthogonal, project, then this would be a great one.
Regarding syntax highlighting and automatic code formatting: I've gotten decent results by treating ISLE like Lisp, FWIW: lisp-mode in Emacs auto-indents the s-expressions and lets me navigate the structure (paired parens, up a level, etc). Vim likely has equivalents too. A very janky first-draft implementation of isle-fmt might just be Emacs in batch mode running indent-region over the file! (Ideally we have something more self-contained and efficient than that eventually, of course.)

Apparently somebody has already implemented the "run Emacs in batch mode" approach to formatting Lisp source: https://github.com/eschulte/lisp-format/blob/master/lisp-format

As a quick way to get go-to-definition working, it would be pretty easy to emit a tags file from islec. vim and emacs both know how to navigate those, and that could make working with the codebase a little bit easier.

I don't think it would be much harder to implement a basic lsp server. Rust-analyzer has the lsp-server crate that does half the work.

https://github.com/yuyang-ok/isle-analyzer/blob/main/editors/code/README.md
I have been developing IDE support for ISLE language right now I think it is ready to lauch.

Cool! What license does it have?

@bjorn3  Apache License 2.0 :-)

I made a tree sitter grammar which is working pretty well for syntax highlighting and navigation in Helix, but I haven't tested it very thoroughly yet.
https://github.com/jwnrt/tree-sitter-isle
FYI, I think the grammar in the language reference has fallen out of sync with the parser. It's missing some additions like if-let, if, partial, pure, and infallible.

@jwnrt this is pretty cool! If you'd like to link to it from the ISLE docs, please feel free to submit a PR for that (maybe cranelift/isle/docs/language-reference.md -- we'd be happy to take updates to the grammar there too if you have them top-of-mind, as it seems likely we've missed updates there as you note).


mathext: Digamma should have the full polynomial expansion

The digamma code is adapted from http://web.science.mq.edu.au/~mjohnson/code/digamma.c, but that code gives a polynomial expansion out to 20 orders while our code only goes out to 8. We should match the extended version.

Note that the asymptotic series is divergent, so using more terms is actually not beneficial here. Some gains would be achieved by using the recurrence relation a little longer, i.e. until x > 10. This is where Boost/Cephes cross over. (For reference, Cephes uses 7 terms.) You will still have problems with high (relative) error in the interval [1, 2] because it contains the sole positive zero of the digamma function. Boost gets around this with a rational approximation; we recently switched to using this in SciPy.
The asymptotic expansion should be evaluated using Horner's method, however.

Thanks for the information. Agreed on Horner's method, we use it elsewhere in the suite, it should be here too.


GH Action - Clean step in Publish Action

Clean step in publishing workflow. Important for self-hosted runners.
Signed-off-by: Ruslan Dautov<EMAIL_ADDRESS>

You probably need to make sure this always runs (even if the previous steps fail)

Yes, good point! Thank you, changed and added one more clean step.

You probably need to make sure this always runs (even if the previous steps fail)

Yes, good point! Thank you, changed and added one more clean step.

Assume we can use something like this: https://github.com/lf-edge/eve/blob/f415f206f090c8a282dee933779ca8631ffc99b9/.github/workflows/eden.yml#L104


Release azure-cli-keyvault 2.0.5

Reviewers, please verify the following for this PR:

[ ] The PR title has format Release <component-name> <version>.
[ ] setup.py has been modified with the same version as in the PR title.
[ ] HISTORY.rst has been modified with appropriate release notes.
[ ] If releasing azure-cli or azure-cli-core, __version__ defined in __init__.py should also be modified to match.

Started releasing azure-cli-keyvault 2.0.5...

Release of 'azure-cli-keyvault' with version '2.0.5' successful. View at https://pypi.python.org/pypi/azure-cli-keyvault/2.0.5.

GitHub release created. https://github.com/Azure/azure-cli/releases.


Optimise Snakemake code

Expected behaviour
Optimisation of Snakemake pipelines
Actual behaviour
Snakemake pipelines work, but there could be optimisations to apply (e.g. output files, scalability).
Steps to reproduce the behaviour
Revisit the code.

sbpipe_sim.snake pipeline optimisation.

Snakemake pipelines have been cleaned and optimised with the current code. The current analysis rule can be organised in sub rules. This would allow for better resource usage and increased parallelisation. To do so, sbpiper requires more modularisation.

Analyses for snakemake pipelines sim() and ps2() are modular.

Analyses for snakemake pipelines pe() and ps1() are modular.


extract widgets mod

Codecov Report

Merging #26 (849b508) into master (00efeb0) will increase coverage by 0.02%.
The diff coverage is 17.17%.

@@            Coverage Diff             @@
##           master      #26      +/-   ##
==========================================
+ Coverage   72.57%   72.60%   +0.02%
==========================================
  Files          53       54       +1
  Lines        3884     4005     +121
==========================================
+ Hits         2819     2908      +89
- Misses       1065     1097      +32

Impacted Files
Coverage Δ

quake_tui/src/command.rs
89.65% <ø> (ø)

quake_tui/src/lib.rs
1.75% <0.00%> (-0.04%)
:arrow_down:

quake_tui/src/ui.rs
0.00% <0.00%> (ø)

quake_tui/src/widgets.rs
1.33% <1.33%> (ø)

quake_tui/src/app.rs
70.83% <88.88%> (+52.46%)
:arrow_up:

quake_core/src/parser/ast.rs
80.00% <0.00%> (-20.00%)
:arrow_down:

src/cli/entry_action.rs
39.21% <0.00%> (-7.78%)
:arrow_down:

src/helper/search_config.rs
98.59% <0.00%> (-1.41%)
:arrow_down:

src/cli/quake_action.rs
0.00% <0.00%> (ø)

src/helper/exec_wrapper/meili_exec.rs
0.00% <0.00%> (ø)

... and 3 more

Continue to review full report at Codecov.

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


code comment error

https://github.com/prometheus/procfs/blob/c5d92abccdcd53df4a03188556ca078f17462b89/proc_status.go#L26
after correcting
// read from /proc/[pid]/status.

Is this still an issue after the prior commits? If not, let's close this bug


Doesn't support basic auth of alertmanager.url?

Hi.
My alertmanager is set to be using basic auth and I setup pipeline as following:
jobs:
- name: job-hello-world
  public: true
  plan:
  - get: time
    trigger: true
  - put: silence-test-alerts
    resource: am
    params:
      operation: silence
      matchers: "service=cf"
      expires: 30m

resources:
- name: am
  type: alertmanager-resource
  source:
    url: http://admin:password@<IP_ADDRESS>:9093/
- name: time
  type: time
  source:
    interval: 1m

resource_types :
- name: alertmanager-resource
  type: docker-image
  source:
    repository: frodenas/alertmanager-resource

but concourse(amtool) prints error.
0e207f34e107: Pull complete
154c7d3824e9: Pull complete
d1fdece7f9a9: Pull complete
Digest: sha256:64d9a5163223cedb990a20e07fa171aaf9e90925ddea5d55a62732ead65745cd
Status: Downloaded newer image for frodenas/alertmanager-resource@sha256:64d9a5163223cedb990a20e07fa171aaf9e90925ddea5d55a62732ead65745cd

Successfully pulled frodenas/alertmanager-resource@sha256:64d9a5163223cedb990a20e07fa171aaf9e90925ddea5d55a62732ead65745cd.

amtool: error: unable to parse silence json response from http://<IP_ADDRESS>:9093/api/v1/silence

I can get a response successfully by curl.
$ curl http://admin:password@<IP_ADDRESS>:9093/api/v1/silences
{"status":"success","data":[]}

It did work with
resources:
- name: am
  type: alertmanager-resource
  source:
    url: http://admin:password@<IP_ADDRESS>:9093   # if there is / at end, it doesn't work

comment | Silence added by CI pipeline http://<IP_ADDRESS>/teams/main/pipelines/alert-manager-pipeline/jobs/job-hello-world/builds/6
-- | --
created_by |
ends_at | 2018-05-18T04:39:06.110679986Z
starts_at | 2018-05-18T04:09:06.11522511Z
updated_at | 2018-05-18T04:09:06.115243322Z


SOLR 15749 created a v2 equivalent of v1 'rename'

https://issues.apache.org/jira/browse/SOLR-15749
Description
Create a v2 equivalent of v1 'RENAME' /solr/admin/collections endpoint
Solution
Created a RenameClusterAPI class and RenameClusterPayload class with an endpoint @EndPoint(path ={"/clusters/rename/{cluster}"}, method = POST, permission = COLL_EDIT_PERM)
Tests
Tested mapping to API endpoint
Checklist
Please review the following and check all that apply:

[√] I have reviewed the guidelines for How to Contribute and my code conforms to the standards described there to the best of my ability.
[√] I have created a Jira issue and added the issue ID to my pull request title.
[√] I have given Solr maintainers access to contribute to my PR branch. (optional but recommended)
[√] I have developed this patch against the main branch.
[√] I have run ./gradlew check.
[√] I have added tests for my changes.
[] I have added documentation for the Reference Guide

Okay thanks, I will fix that shortly
On Mon, Oct 31, 2022 at 3:21 PM Jason Gerlowski @.***>
wrote:

@.**** commented on this pull request.
In
solr/core/src/test/org/apache/solr/handler/admin/api/V2CollectionAPIMappingTest.java
https://github.com/apache/solr/pull/1080#discussion_r1009477355:

@@ -85,6 +85,18 @@ public void testGetCollectionStatus() throws Exception {
assertEquals("shard2", v1Params.get(SHARD));
}

@Test
public void testRenameCollectionAllParams() throws Exception {
final SolrParams v1Params = captureConvertedV1Params(

       "/collections/collName", "POST", "{\"rename\": {\"to\": \"targetColl\"}}");

The first thing that jumps out at me in this test code is that you
restructured the API endpoint to look the way we discussed. (i.e. now it
looks like POST /collections/collName/commands/rename)....but this test
still uses the old appearance/syntax. The path, request-body, etc. that
gets sent into captureConvertedV1Params is all using the old syntax we
moved away from. If you fix that, I'm betting this error will go away!
As a general note on "process": generally speaking its easier for me (or
others) to help review if you push up the code changes you've made, even if
things aren't 100% working yet. That makes it easier for me to comment on
particular lines. It makes it easier for me to pull the code down and run
that test myself, etc. It saves you from having to paste your code in a
comment box and fight the markdown/formatting, etc.
Totally fine if you had reasons for not doing that here; just figured I'd
mention the convention as a "heads up" in case it's helpful since you're
newer to the community
—
Reply to this email directly, view it on GitHub
https://github.com/apache/solr/pull/1080#discussion_r1009477355, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AWIEDZG2KINRYWJVZZDM74TWF7IYLANCNFSM6AAAAAARGAWM5E
.
You are receiving this because you were mentioned.Message ID:
@.***>

Thank you Jason, you do the same.
On Sat, Oct 15, 2022 at 8:51 PM sonatype-lift[bot] @.***>
wrote:

⚠ 314 God Classes were detected by Lift in this project. Visit the
Lift web console
https://lift.sonatype.com/results/github.com/apache/solr/01GFEGDZ81KM2ZP1W0ZBMPP3N9?tab=technical-debt&utm_source=github.com&utm_campaign=lift-comment&utm_content=apache solr
for more details.
—
Reply to this email directly, view it on GitHub
https://github.com/apache/solr/pull/1080#issuecomment-1279821768, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AWIEDZD65G22DAN66STNGC3WDMDM5ANCNFSM6AAAAAARGAWM5E
.
You are receiving this because you authored the thread.Message ID:
@.***>


Azure Web App Support for CodeIgniter

Added Deploy to Azure button which would take the user to Azure and Deploy a Web App. It also deploys In App MYSQL Database when it deploys CodeIgniter.

I don't think this will fit...
It's just a particular case, for your setup/project, for instance. CodeIgniter doesn't have to do anything with deployment.

It's just and add on, would help users of Azure to deploy CodeIgniter on
single click.

You are well-intentioned, but I suggest you create your own Repository, this is not the responsibility of the framework

Even if we put everything else aside, this is basically a request to promote Azure, and as a side-effect - MySQL too. We do not and will not promote any particular platform.
I don't even believe this is "well-intentioned", given that you want to put this on the very top of our README, above even our own project name.
And that you're converting it from rst to Markdown is just more annoying.

@muhammad92 If you want to help other CI devs, you can always post your ideas and solutions to CI forums, in Addins section, for example. :)

Apologies from my side. Was my first pull request, just wanted to value add what I could do.
If you really hate the idea of deployment on azure. I could be placed at bottom may be :) under heading of one click deployment.
By the way thanks for your idea. I was never here to promote Azure or MYSQL but to just add something that could help many of the CI users.
Thanks.

@gxgpet  sure :)

You're free to blog about it, post about it on our forums, publish it as a helper package on Packagist, etc.
But while I appreciate how it may make deployment easier for Azure users, it doesn't belong in our README.
Unless we provide equally convenient stuff for every other platform, doing this would mean that we are favoring/promoting Azure as a platform - that's just how it works.


Class 'Laravel\Cashier\CashierServiceProvider' not found

when I trying to add cashierServiceProvider in providers (config/app.php) it showing not found error. I am using laravel 4.2. how to solve this issue?

Hey there,
Unfortunately we don't support this version anymore. Please check out our support policy on which versions we are currently supporting. Can you please try to upgrade to the latest version and see if your problem persists? We'll help you out and re-open this issue if so.
Thanks!


Envs with python from previous Conda Env and Python version

Testing in another MacOS, we find out that the python version and the way the Environment is created might affect the correct installation. 1) if users have python already from conda envs, which can be quite common,  thepoetry installshows
zsh:1: no such file or directory: /opt/concourse/worker/volumes/live/c1a1a6ef-e724-4ad9-52a7-d6d68451dacb/volume/python-split_1631807121927/_build_env/bin/llvm-ar
        zsh:1: no such file or directory: /opt/concourse/worker/volumes/live/c1a1a6ef-e724-4ad9-52a7-d6d68451dacb/volume/python-split_1631807121927/_build_env/bin/llvm-ar
        error: Command "/opt/concourse/worker/volumes/live/c1a1a6ef-e724-4ad9-52a7-d6d68451dacb/volume/python-split_1631807121927/_build_env/bin/llvm-ar rcs build/temp.macosx-10.9-x86_64-3.9/libnpymath.a build/temp.macosx-10.9-x86_64-3.9/numpy/core/src/npymath/npy_math.o build/temp.macosx-10.9-x86_64-3.9/build/src.macosx-10.9-x86_64-3.9/numpy/core/src/npymath/ieee754.o build/temp.macosx-10.9-x86_64-3.9/build/src.macosx-10.9-x86_64-3.9/numpy/core/src/npymath/npy_math_complex.o build/temp.macosx-10.9-x86_64-3.9/numpy/core/src/npymath/halffloat.o" failed with exit status 127
        ----------------------------------------
    ERROR: Command errored out with exit status 1: /Users/jding/opt/anaconda3/envs/ua/bin/python -u -c 'import io, os, sys, setuptools, tokenize; sys.argv[0] = '"'"'/private/var/folders/_p/_n8j38ls4wq303jqcy3mt0cr0000gr/T/pip-install-yt692vf0/numpy_846fe9e98d334320be8531dc9d52f948/setup.py'"'"'; __file__='"'"'/private/var/folders/_p/_n8j38ls4wq303jqcy3mt0cr0000gr/T/pip-install-yt692vf0/numpy_846fe9e98d334320be8531dc9d52f948/setup.py'"'"';f = getattr(tokenize, '"'"'open'"'"', open)(__file__) if os.path.exists(__file__) else io.StringIO('"'"'from setuptools import setup; setup()'"'"');code = f.read().replace('"'"'\r\n'"'"', '"'"'\n'"'"');f.close();exec(compile(code, __file__, '"'"'exec'"'"'))' install --record /private/var/folders/_p/_n8j38ls4wq303jqcy3mt0cr0000gr/T/pip-record-_cno4xb0/install-record.txt --single-version-externally-managed --prefix /private/var/folders/_p/_n8j38ls4wq303jqcy3mt0cr0000gr/T/pip-build-env-iwnc5r0v/overlay --compile --install-headers /private/var/folders/_p/_n8j38ls4wq303jqcy3mt0cr0000gr/T/pip-build-env-iwnc5r0v/overlay/include/python3.9/numpy Check the logs for full command output.
    ----------------------------------------
  WARNING: Discarding file:///Users/jding/Library/Caches/pypoetry/artifacts/1b/f8/77/cd6cb033665a9495af3468fb24629d33fa626db7b10a8abeb47a016bf1/pandas-1.0.3.tar.gz. Command errored out with exit status 1: /Users/jding/opt/anaconda3/envs/ua/bin/python /private/var/folders/_p/_n8j38ls4wq303jqcy3mt0cr0000gr/T/pip-standalone-pip-p1dzj8yo/__env_pip__.zip/pip install --ignore-installed --no-user --prefix /private/var/folders/_p/_n8j38ls4wq303jqcy3mt0cr0000gr/T/pip-build-env-iwnc5r0v/overlay --no-warn-script-location --no-binary :none: --only-binary :none: -i https://pypi.org/simple -- setuptools wheel 'Cython>=0.29.13' 'numpy==1.13.3; python_version=='"'"'3.6'"'"' and platform_system!='"'"'AIX'"'"'' 'numpy==1.14.5; python_version>='"'"'3.7'"'"' and platform_system!='"'"'AIX'"'"'' 'numpy==1.16.0; python_version=='"'"'3.6'"'"' and platform_system=='"'"'AIX'"'"'' 'numpy==1.16.0; python_version>='"'"'3.7'"'"' and platform_system=='"'"'AIX'"'"'' Check the logs for full command output.
  ERROR: Command errored out with exit status 1: /Users/jding/opt/anaconda3/envs/ua/bin/python /private/var/folders/_p/_n8j38ls4wq303jqcy3mt0cr0000gr/T/pip-standalone-pip-p1dzj8yo/__env_pip__.zip/pip install --ignore-installed --no-user --prefix /private/var/folders/_p/_n8j38ls4wq303jqcy3mt0cr0000gr/T/pip-build-env-iwnc5r0v/overlay --no-warn-script-location --no-binary :none: --only-binary :none: -i https://pypi.org/simple -- setuptools wheel 'Cython>=0.29.13' 'numpy==1.13.3; python_version=='"'"'3.6'"'"' and platform_system!='"'"'AIX'"'"'' 'numpy==1.14.5; python_version>='"'"'3.7'"'"' and platform_system!='"'"'AIX'"'"'' 'numpy==1.16.0; python_version=='"'"'3.6'"'"' and platform_system=='"'"'AIX'"'"'' 'numpy==1.16.0; python_version>='"'"'3.7'"'"' and platform_system=='"'"'AIX'"'"'' Check the logs for full command output.

  at ~/.poetry/lib/poetry/utils/env.py:1195 in _run
      1191│                 output = subprocess.check_output(
      1192│                     cmd, stderr=subprocess.STDOUT, **kwargs
      1193│                 )
      1194│         except CalledProcessError as e:
    → 1195│             raise EnvCommandError(e, input=input_)
      1196│
      1197│         return decode(output)
      1198│
      1199│     def execute(self, bin, *args, **kwargs):

then the issue was about the python version in which we got
Command ['/Users/jding/Library/Caches/pypoetry/virtualenvs/ramp-u2zQ2mbi-py3.10/bin/pip', 'install', '--no-deps', '/Users/jding/Library/Caches/pypoetry/artifacts/1d/39/69/fd592161731e8c197899f57a8d2142e2e2fd24626c2ee55ba72942ec9f/pandas-1.0.3.tar.gz'] errored with the following return code 1, and output:
  Processing /Users/jding/Library/Caches/pypoetry/artifacts/1d/39/69/fd592161731e8c197899f57a8d2142e2e2fd24626c2ee55ba72942ec9f/pandas-1.0.3.tar.gz
    Installing build dependencies: started
    Installing build dependencies: still running...
    Installing build dependencies: still running...
    Installing build dependencies: still running...
    Installing build dependencies: still running...
    Installing build dependencies: finished with status 'error'
    error: subprocess-exited-with-error

× pip subprocess to install build dependencies did not run successfully.
  │ exit code: 1
  ╰─> [4308 lines of output]
      Ignoring numpy: markers 'python_version == "3.6" and platform_system != "AIX"' don't match your environment
      Ignoring numpy: markers 'python_version == "3.6" and platform_system == "AIX"' don't match your environment
      Ignoring numpy: markers 'python_version >= "3.7" and platform_system == "AIX"' don't match your environment
      Collecting setuptools
        Using cached setuptools-60.9.3-py3-none-any.whl (1.1 MB)
      Collecting wheel
        Using cached wheel-0.37.1-py2.py3-none-any.whl (35 kB)
      Collecting Cython>=0.29.13
        Using cached Cython-0.29.28-py2.py3-none-any.whl (983 kB)
      Collecting numpy==1.14.5
        Using cached numpy-1.14.5.zip (4.9 MB)
        Preparing metadata (setup.py): started
        Preparing metadata (setup.py): finished with status 'done'
      Building wheels for collected packages: numpy
        Building wheel for numpy (setup.py): started
        Building wheel for numpy (setup.py): still running...
        Building wheel for numpy (setup.py): still running...
        Building wheel for numpy (setup.py): still running...
        Building wheel for numpy (setup.py): still running...
        Building wheel for numpy (setup.py): finished with status 'error'
        error: subprocess-exited-with-error

        × python setup.py bdist_wheel did not run successfully.
        │ exit code: 1

Then the solution was to install 3.8.8 version, to let poetry install correctly, and then we got the issue reported in #14 , we solved it as it was described there, and then we got the issue #15  where nothing much could be done. I guess there is a need to make clear in the Readme that users needs to install the ASPICS requirements out of any Conda env previously installed.

Getting things working with poetry is too much of a time sink. I'm going to copy over the Conda env from RAMP-UA repo and get it working here. If that still doesn't work, pull out a bigger hammer and just use Docker -- though I have doubts that'll make the OpenGL / OpenCL part easy. Hoping to get you something working by EOD today.

Super. I tried already with miniconda, using the dependencies listed in the TOM file, but did not work, I got the same error. I will wait to you new branch and try again, in case I did something wrong in my previous attempt

For what it's worth: I think updating pandas is all that's needed to be able to install on osx. I think this is separate from the issue of managing the poetry environment within rust (could make another issue for that if you like?).
Unfortunately I'm using the master branch of poetry and it's heavily altering the format of poetry.lock so I don't want to mess that up in a way that might be backwards incompatible.
[tool.poetry.dependencies]
 python = "^3.8"
-pandas = "=1.0.3"
+pandas = "^1.4"
 click = "^8.0.3"
 PyYAML = "^6.0"
 tqdm = "^4.62.3"
@@ -16,6 +16,7 @@ glfw = "^2.4.0"
 imgui = "^1.4.1"
 PyOpenGL = "=3.1.5"
 pyopencl = "=2020.2.2"
+numpy = "^1.22.3"

I could try a proper pull request if you'd rather but for ease here are the modifications I've gotten to. Given numpy is a pandas dependency that line might not be necessary and I think the glfw line is simply a formatting fix.

Assuming you'd also like to automate cmake, that can be added within [build-setup]. An example from scikit (and again, can put together a pull request or separate ticket if you like:
[build-system]
requires = [
  "setuptools>=42",
  "wheel",
  "scikit-build",
  "cmake",
  "ninja; platform_system!='Windows'"
]
build-backend = "setuptools.build_meta"

https://scikit-build.readthedocs.io/en/latest/usage.html#example-of-setup-py-cmakelists-txt-and-pyproject-toml


Navigator not displaying startup script users in quickstart

If I follow the instructions for the quickstart (SDK 2.5.0), but instead of running daml studio, I run daml start --start-navigator=yes, I don't see the list of users created in the script in the log-in page.
Doc on this feature of navigator implies this should just work: https://docs.daml.com/tools/navigator/index.html#logging-in

Same goes for daml start followed by daml navigator server separately.


Added new version 2021-06-01-preview to support datasource auth crede…

…ntials input.
MSFT employees can try out our new experience at OpenAPI Hub  - one location for using our validation tools and finding your workflow.

Changelog
Please ensure to add changelog with this PR by answering the following questions.

What's the purpose of the update?

[ ] new service onboarding
[ ] new API version
[ ] update existing version for new feature
[ ] update existing version to fix swagger quality issue in s360
[ ] Other, please clarify

When you are targeting to deploy new service/feature to public regions? Please provide date, or month to public if date is not available yet.
When you expect to publish swagger? Please provide date, or month to public if date is not available yet.
If it's an update to existing version,  please select SDKs of specific language and CLIs that require refresh after swagger is published.

[ ] SDK of .NET (need service team to ensure code readiness)
[ ] SDK of Python
[ ] SDK of Java
[ ] SDK of Js
[ ] SDK of Go
[ ] PowerShell
[ ] CLI
[ ] Terraform
[ ] No, no need to refresh for updates in this PR

Contribution checklist:

[ ] I commit to follow the Breaking Change Policy of "no breaking changes"
[ ] I have reviewed the documentation for the workflow.
[ ] Validation tools were run on swagger spec(s) and errors have all been fixed in this PR. How to fix?

If any further question about AME onboarding or validation tools, please view the FAQ.
ARM API Review Checklist

[ ] Ensure to check this box if one of the following scenarios meet updates in the PR, so that label “WaitForARMFeedback” will be added automatically to involve ARM API Review. Failure to comply may result in delays for manifest application. Note this does not apply to data plane APIs, all “removals” and “adding a new property” no more require ARM API review.

Adding new API(s)
Adding a new API version
[ ] Ensure to copy the existing version into new directory structure for first commit (including refactoring) and then push new changes including version updates in separate commits. This is required to review the changes efficiently.
Adding a new service

[ ] Please ensure you've reviewed following guidelines including ARM resource provider contract and REST guidelines. Estimated time (4 hours). This is required before you can request review from ARM API Review board.

[ ] If you are blocked on ARM review and want to get the PR merged with urgency, please get the ARM oncall for reviews (RP Manifest Approvers team under Azure Resource Manager service) from IcM and reach out to them.

Breaking Change Review Checklist
If there are following updates in the PR, ensure to request an approval from Breaking Change Review Board as defined in the Breaking Change Policy.

[ ] Removing API(s) in stable version
[ ] Removing properties in stable version
[ ] Removing API version(s) in stable version
[ ] Updating API in stable or public preview version with Breaking Change Validation errors
[ ] Updating API(s) in public preview over 1 year (refer to Retirement of Previews)

Action: to initiate an evaluation of the breaking change, create a new intake using the template for breaking changes. Addition details on the process and office hours are on the Breaking change Wiki.
Please follow the link to find more details on PR review process.

Created a different pull request for changes abandoning this https://github.com/Azure/azure-rest-api-specs/pull/14932

abandoning this as I created a new one for same changes that is cleaner https://github.com/Azure/azure-rest-api-specs/pull/14932


Send button only responds to touch on left half of button

Using iOS 9 in the Simulator rightBarButtonItem responds perfectly to touch.
Using iOS 9 on an actual device (iPhone 5s) the Send (rightBarButtonItem) button only seems to respond to touch on its left side. Including when button width is resized.
This appears in the project demo as well as my production app. I haven't tested iOS 8/7 so maybe Apple made some changes perhaps surrounding buttons near the edge of screens?
I've checked for the button exceeding superview bounds and for other views capturing the touch event, but can see nothing. UIWindow and the View Controller bounds also appear fine.

Hello everyone!
I'm sorry to inform the community that I'm officially deprecating this project. 😢 Please read my blog post for details:
http://www.jessesquires.com/blog/officially-deprecating-jsqmessagesviewcontroller/
Thus, I'm closing all issues and pull requests and making the necessary updates to formally deprecate the library. I'm sorry if this is unexpected or disappointing. Please know that this was an extremely difficult decision to make. I'd like to thank everyone here for contributing and making this project so great. It was a fun 4 years. 😊
Thanks for understanding,
— jsq


Expired queue remain in the waiting room size

Describe the bug
expired queue should not be count under waiting room size as they are not waiting and expired.
To Reproduce

Create 6 user
add 4 serving count
user 1 & 2 enter website
user 3 & 4 wait until queue expiry
user 5 & 6 can now enter the website
but the Waiting Room Size keep showing 2 which the queue should be 0 now

Expected behavior
Expected Waiting Room Size update to 0
Screenshots
If applicable, add screenshots to help explain your problem (please DO NOT include sensitive information).

Additional context
Version: [v1.1.0]

Thanks for reporting this. We have added it to our backlog to look into.

Hi @cccdi,
The fix for this issue has been included in release 1.1.3. Please feel free to verify the fix, and don't hesitate to reach out if you encounter any further issues.
Thank you!


Create Source Collector Main Prototype

Source Collector Main will connect

The PDAP API Client
The Source Collector Database (#120)
The Collector Manager (#118)

With each other, enabling a pipeline that streamlines the collection of urls into a database for additional processing.
This prototype would include:

A Command Line Interface (CLI) for managing commands to the system (possibly a precursor to a full-fledged API)
A Batch Preprocessor which does the following

Retrieve raw data from collector
Create Batch metadata based on URLS
Eliminate existing duplicate URLs from original batch through combination of calls to Data Sources App Database (via PDAP API Client) and internal Collector Manage Database
Provide batch data and urls to be submitted into database

Things To Include Later

A means of sifting quality URLs using machine learning
Using the PDAP API Client to submit urls as data sources to the Data Sources App
Upgrading (or complementing) the CLI with a full-fledged API.

For the database backend, I'll be doing what I did before and using PostgreSQL.
However, I'll be using a fully SQLAlchemy-based backend this time around. And, once we get to the point of need to do database migrations, look into using Alembic to assist with that.


Error while unmountComponentAtNode

I try to destroy my React application using "unmountComponentAtNode" I get an error when my app includes a Dialog  component.

[x] I have searched the issues of this repository and believe that this is not a duplicate.

Expected Behavior
Application unmounted without errors.
Current Behavior
These errors:
Uncaught DOMException: Failed to execute 'removeChild' on 'Node': The node to be removed is not a child of this node.

index.js:2177 The above error occurred in the <DialogInline> component:
    in DialogInline

Steps to Reproduce (for bugs)
https://codesandbox.io/s/7mojo3pzzq (project not working for some reason)

Call unmountComponentAtNode

Context
I'm trying to unmount a React application that has a Dialog.
Your Environment

Tech
Version

Material-UI
0.20.0

React
16.2.0

browser
Any

@RaedsLab We have been porting the component on the v1-beta branch. We have reimplemented it from the ground-up.
We encourage people starting with Material-UI to go with v1-beta. The master branch is in a low maintenance mode. We don't plan any work on it. However, we still do our best to review and merge pull requests that fix bugs on this branch.


Feature me message

rendering /me messages works \o/
but sending /me actions depends on https://github.com/yakyak/hangupsjs/pull/49

imho, it doesn't make sense to have a rendering feature only. So we have to wait for hangupsjs release I think.

travis ci build fails (?)

@davibe I've not fixed it yet since this feature cannot be used without a new hangupsjs release.

i believe @algesten wants to embed hangupjs source code in yakyak at some point. But ok, let's wait for him to answer

just rebased on master. however, one unit test still fails (hangupsjs dependency)
@davibe @algesten actually i appreciate the separation of yakyak and hangupsjs.
yakyak is the ui client and hangupsjs the api abstraction for hangouts.

I did not understand correctly before. Martin (algesten) moved the repository under the yakyak github group. Basically he's loosing interest in the project so we will have to maintain the lib as best as we can.
This means I am free to merge your pull request and i will do it as soon as i can so you can complete this PR too

I merged your stuff in hangupsjs so you can go on

yay, thanks :)


Error handling

Describe the bug
There are lots of edge cases and instances where bot crashes due to some unexpected syntax or error.
Example : Unexpected syntax in /addmeetup
Expected behavior
The bot shouldn't crash instead it should respond with something like " Unexpected behaviour, please try again "
** Additional Context **
We are using log.Fatal / log.Fatalln when an error occurs. This causes the program to call os.exit(1) which is an undesirable behaviour.
Instead we should log.Print  the errors as well as a bot response for the same.

@daemon1024 : let me pick this one :

removing log.Fatal
convert log.Print & send a error message

@OgmaJ sure.
https://github.com/osdc/bots/blob/master/telegram-bot/meetup.go#L139-L141
This may help you out.


Remove fast_finish flag

It has been confirmed that the fast_finish flag is responsible for triggering multiple build notifications on #zftalk.dev.

@froschdesign what about this in 3.2.0?

@Koopzington
Merged with 43d209995f054fa62557652192c4943625b4145a.
Thanks! 👍


Unable to run React samples (dev branch)

I've tried multiple times with various different versions of the current .net core sdk and every time I try to run any of the react samples (MusicStore/ReactGrid) I get the following error: (running on Win10)
The current project is not valid because of the following errors:
C:\Users\<user>\Documents\vs-workspace\JavaScriptServices\samples\react\MusicStore(1,0): error DOTNET1017: Project file does not exist 'C:\Users\<user>\Documents\vs-workspace\JavaScriptServices\samples\react\MusicStore\project.json'

Reading around it seems the project.json file is now gone since moving to VS2017, which is totally fine. However why can't I build/run from the command line anymore?
Is it only supported to do this in VS2017 now? No VS Code? Command line? I haven't installed VS2017 yet but it shouldn't matter to my knowledge.
I'd imagine that it should build from the command line just fine, so what should I do? What version of the .net core sdk do I need? The newest version from the downloads page is confusing. The file name and dotnet --version say it's 1.0.0-preview2-1-003177 but running just 'dotnet' you see it says 1.1.0 and it's the same on the site.
I'd love to be able to run the samples, but can someone also please explain to me what on earth is going on with the versions?
Thanks!

You need the RC4 tools (or later) to build/run csproj on the command line: https://github.com/dotnet/core/blob/master/release-notes/download-archives/rc4-download.md


Address flaws in "Peer disconnected too quickly" approach

What is wrong?
https://github.com/ethereum/trinity/pull/629#issuecomment-493607901
As pointed out in that comment, if we treat X seconds as our minimum session length, a peer who is trying to disrupt our sync can wait X + 1 seconds before disconnecting.
How can this be fixed?
Ideas:

introduce randomness for minimum session length on a per-connection basis.
Take into account our existing sessions and scale blacklist timeout using this data:

e.g.  Apply a scaled blacklist to every peer that disconnects from us, very minimal for peers who stay long enough, and move exponentially towards an upper bound when you get closer to the zero.

Another approach: track EMA of session length, averaging in 0 session length when the peer does not accept a connection. We can (still?) do an exponential backoff of attempts to connect when they refuse a connection, to avoid punishing the long-session-length peers too quickly when they become unavailable.
(Subtlety: we should not modify the average session length if we disconnect from a peer and it was a shorter than usual session. But we should modify it when we disconnect from a peer and it was a longer that average session.)


[PKG-4789] Initial build 0.1.800

autoviml 0.1.800 :snowflake:
Destination channel: Snowflake
Links

PKG-4789
Upstream repository
Relevant dependency PRs:

https://github.com/AnacondaRecipes/emoji-feedstock/pull/2
https://github.com/AnacondaRecipes/imbalanced-ensemble-feedstock/pull/1

Explanation of changes:

Patched 2 broken imports in upstream:

imbalanced_ensemble -> imbens
scipy.interp -> numpy.interp

scipy.interp has been deprecated in favor of numpy.interp, and removed since scipy 1.12.

Submitted a PR to upstream with the same fix.

No upstream tests available.

Linter check found the following problems:
ERROR conda.cli.main_run:execute(125): `conda run conda-lint /tmp/abs_2e4b43i1u_/clone` failed. (See above for error)
The following problems have been found:
===== WARNINGS =====

clone/recipe/meta.yaml:73: missing_license_file: The recipe is missing the about/license_file or about/license_url key.
===== Final Report: =====
0 Errors and 1 Warning were found

@lorepirri It would seem that the upstream xgboost pinning is necessary. Running the code in the example notebook fails for xgboost >=1.7,<2.0, but works fine with the upstream pinnings. We can either proceed with the rebuild for xgboost or deliver only Python 3.9 and 3.10 versions.

https://github.com/AnacondaRecipes/xgboost-feedstock/pull/15

Linter check found the following problems:
ERROR conda.cli.main_run:execute(125): `conda run conda-lint /tmp/abs_502cgbbcsu/clone` failed. (See above for error)
The following problems have been found:
===== ERRORS =====

clone/recipe/meta.yaml:62: missing_pip_check: For pypi packages, pip check should be present in the test commands
===== Final Report: =====
1 Error and 0 Warnings were found

Linter check found the following problems:
ERROR conda.cli.main_run:execute(125): `conda run conda-lint /tmp/abs_e078jaguks/clone` failed. (See above for error)
The following problems have been found:
===== ERRORS =====

clone/recipe/meta.yaml:62: missing_pip_check: For pypi packages, pip check should be present in the test commands
===== Final Report: =====
1 Error and 0 Warnings were found

Linter check found the following problems:
ERROR conda.cli.main_run:execute(125): `conda run conda-lint /tmp/abs_9a7wybdzh4/clone` failed. (See above for error)
The following problems have been found:
===== ERRORS =====

clone/recipe/meta.yaml:62: missing_pip_check: For pypi packages, pip check should be present in the test commands
===== Final Report: =====
1 Error and 0 Warnings were found


LibWeb: Improve alignment of table width distribution algorithm with the specification

The expand_columns_to_fill_available_width lambda doesn't have an obvious match in the distribution algorithm specification. It looks like we're, at least in part, implementing the specification at the end of TableFormattingContext::distribute_width_to_columns. Moreover, the expand_columns_to_fill_available_width lambda assigns the entire remaining width based on the preferred increment of a column relative to the total preferred increment for all columns, which doesn't seem to be in the specification and is unstable with respect to modest increments of max width vs min width for a few columns.

Mostly fixed by https://github.com/axgallo/serenity/commit/303f62b1572fd2fd2e49464dce3c8b3b2b35693f, will track individual issues instead.


Use prepare-for-rsem.py script directly from UMI-tools package

Description of feature
See https://github.com/nf-core/rnaseq/issues/828#issuecomment-1135673459

Still not in a UMI-tools release, but will be in the next one.

Solved in https://github.com/nf-core/rnaseq/pull/1214


Add --add-opens for java.lang + java.io in system_custom by default

Resolves https://github.com/adoptium/aqa-systemtest/issues/418
Signed-off-by<EMAIL_ADDRESS><EMAIL_ADDRESS>

Test with JDK 8 with a non-MiniMix test (where -add-opens can't be part of command line) : Internal/Grinder/14420
Test with JDK11 with a non-MiniMinx test (where -add-opens is not required) : Internal/Grinder/14419
Test with JDK 11 with MiniMix test (where -add-opens is required) : Internal/Grinder/14418

Note: https://twitter.com/DanHeidinga/status/1372405925647155202 - JDK_JAVA_OPTIONS
Also relates to https://github.com/AdoptOpenJDK/openjdk-tests/issues/2253 (where we could define a variable in perf.mk in similar fashion).

Oh.. Just noticed your comment @ShelleyLambert .
if simply defining env var JDK_JAVA_OPTIONS solves the issue, then we should do this instead of what I've just done! I'll give this one a try.

@ShelleyLambert , I tried it, but it doesn't seem to work:
17:34:49  NOTE: Picked up JDK_JAVA_OPTIONS: "--add-opens java.base/java.lang=ALL-UNNAMED --add-opens java.base/java.io=ALL-UNNAMED"
17:34:49  JVMJ9VM007E Command-line option unrecognised: --add-opens java.base/java.lang=ALL-UNNAMED --add-opens java.base/java.io=ALL-UNNAMED
17:34:49  Error: Could not create the Java Virtual Machine.
17:34:49  Error: A fatal exception has occurred. Program will exit.
17:34:49  NOTE: Picked up JDK_JAVA_OPTIONS: "--add-opens java.base/java.lang=ALL-UNNAMED --add-opens java.base/java.io=ALL-UNNAMED"
17:34:49  JVMJ9VM007E Command-line option unrecognised: --add-opens java.base/java.lang=ALL-UNNAMED --add-opens java.base/java.io=ALL-UNNAMED
17:34:49  Error: Could not create the Java Virtual Machine.
17:34:49  Error: A fatal exception has occurred. Program will **exit.**

Moving back to what worked.
Perhaps we could investigate how to make use of JDK_JAVA_OPTIONS in a separate issue?

Latest tests :
JDK 11 with MiniMix_5m test : Grinder/14448
JDK 11 with MinIMix test run via system_custom : Grinder/14447
JDK 8 with MinIMix test run via system_custom : Grinder/14446/


USHIFT-1848: add periodic test scenario for the standard tests on rpm-based system

Depends on #2568
/hold

/hold cancel

Rebased to see if that helps with the flakes. The SELinux failure looks like it might be real because I can reproduce it locally.

/retest-required

/lgtm

/cherry-pick release-4.15

/cherry-pick release-4.14


Error: running react-native run-ios (back-icon.png doesn't exist)

Updating latest watchman and node.
Installing lastest react-native-cli
Initialising react-native project
Using react-native run-ios without any changes
Error

transformed 349/394 (89%)
(node:25247) UnhandledPromiseRejectionWarning: Unhandled promise rejection
(rejection id: 893): UnableToResolveError:Unable to resolve module ./assets/back-icon.png from weekdays/node_modules/react-native/Libraries/CustomComponents/NavigationExperimental/NavigationHeaderBackButton.js: Directory weekdays/node_modules/react-native/Libraries/CustomComponents/NavigationExperimental/assets/back-icon.png doesn't exist

Node: v7.2.0
react-native-cli: v4.1.1
OSX: ElCapitan: v10.11.5 (15F34)
Simulator: iPhone 6 - iOS 10.2 (14C89)
Xcode: v8.2.1 (8C1002)

Opening
/node_modules/react-native/Libraries/CustomComponents/NavigationExperimental/NavigationHeaderBackButton.js
and changing
43: <Image style={styles.button} source={require('./assets/back-icon.png')} />
to
43: <Image style={styles.button} />
resolves issue, but i'm pretty sure i should not do that
Reinstalling node_modules inside project directiory, and inside /node_modules/react-native with
npm cache clean
rm -rf node_modules
npm install
doesn't do the trick.
also chaning package.json dependencies react-native from "0.40.0" to "0.38.0" fixes the problem
How to fix?

Hmm, I haven't seen this issue.
Can you try running watchman watch-del-all and running the RN packager server with the --resetCache option?

Closing, bigger thread here #6691

Hi guys,
I also want to create simple hello word based on this: https://facebook.github.io/react-native/docs/getting-started.html and I faced on this red error:
"node_modules/react-native/Libraries/CustomComponents/NavigationExperimental/assets/back-icon.png doesn't exist"

Can you help me to fix this?
I worked with React-Native later without any issue but after install new version this error appear on any app (also a simple hello world)

This is my complete error:

Hi guys,
I also want to create simple hello word based on this: https://facebook.github.io/react-native/docs/getting-started.html and I faced on this red error:
/node_modules/react-native/Libraries/CustomComponents/NavigationExperimental/assets/back-icon.png doesn't exist
Can you help me to fix this?
Please, please, please, please!!!!!!

sudo chmod -R 755 your-project

Hi @RaThierry,
See #6691 and check project folder path. Maybe special character exists in path. For example +. For me it's the main issue.

@cyclops24 That was our problem too. We had a space in one of our directory names.

@Jimmy-YMJ thank you, that worked for me :D

@Jimmy-YMJ thank you, that worked for and me :D


Keyboard Shortcuts Do Not Work

module.js:440 Uncaught Error: Cannot find module 'remote'
Failed to parse SourceMap: https://www.youtube.com/Applications/YouTube.app/Contents/Resources/app/lib/static/preload.js.map

Thanks for the heads up, it was a change in electron, try now please!

Thank you so much. Been looking for something like this.

Happy to hear this, I've written a little medium article about-it: https://medium.com/front-end-wtf/youtube-desktop-app-with-multimedia-keyboard-shortcuts-f69aad3c5de2#.17a76yx2t


fix: patch integration test argv

Patches sys.argv in testing, so that Lug propagates the intended command-line arguments to remote execution (instead of the arguments given to pytest for CI).

Closed, moving towards an alternate method of passing in command-line arguments.


[WIP] Re-add support for all colorspaces

@mac-cain13 @tomlokhorst
Upon further research into #301 and #309, I believe that the original implementation of colors that normalized all colors to the NSCalibratedRGBColorSpace (aka Generic RGB) was correct.
colors[key] = colorList.color(withKey: key)?.usingColorSpaceName(NSCalibratedRGBColorSpace)

What this is doing is it takes an NSColor in any colorspace, then converts it to Generic RGB. If the original color was already RGB, you get the same color as a result. If you look at the test cases I've added in this PR, you'll see that R.swift in fact creating the correct RGB values for red, green and blue in the Display P3 and sRGB .clr files.
When converting between colorspaces, depending on the color, there can be some loss in precision, which is reflected in the blue test case for the sRGB .clr file that I added. The conversion to RGB results in a blue value of 254.538, which rounds up to 255. Since this loss is minimal, I believe that it should be ignored. This should be fine because this should be the "real" value of the intended color, because it was created in a different colorspace.
In this PR, I have returned the color conversion to its original form and added tests for the Display P3 .clr file, as well as a new sRGB .clr file and tests to go along with it. I've also updated the Colors.md documentation to indicate that when colors are created in a colorspace that isn't RGB, R.swift's generated RGB values will appear to be wrong, but are in fact correct.
I'm almost 100% sure that my logic regarding this issue is correct, but I would appreciate some independent verification. Do you guys think you could take a look and see if there's anything wrong with this PR, either logically or implementation-wise?
Thanks!

Hmm, so I don't think that this is correct.
For one, this doesn't actually add support for Display P3 colors. In order to do that, I need to have R.swift generate the following code if the colorspace is Display P3.
/// #FF0000 Red
///
/// UIColor(displayP3Red: 1.0, green: 0.0, blue: 0.0, alpha: 1.0) or UIColor(red: 1.0, green: 0.0,
static func red(_: Void = ()) -> UIKit.UIColor {
  if #available(iOS 10.0, *) {
    return UIKit.UIColor(displayP3Red: 1.0, green: 0.0, blue: 0.0, alpha: 1.0)
  } else {
    return UIKit.UIColor(red: 1.0, green: 0.0, blue: 0.0, alpha: 1.0)
  }
}

This leaves two issues:

What to do with sRGB? Convert it to RGB like before?
What to generate for comments in the case of Dislpay P3? Something like the above example?

Any thoughts?

Is there a difference in colours or range between "RGB" and "sRGB", or are they the same?
For P3 colours, I think we should generate those with an @available annotation, so that they only appear on iOS 10.
Regarding the comments; I don't think the background-color css properties are working anymore in the current Xcode. If so, we can remove them.

Generic RGB and sRGB definitely seem to be different. sRGB seems to be somewhat of a standard because of its limited range, meaning that most other wide ranged colorspaces fully encompass it, so it's somewhat of a lowest common denominator. Here is a Wiki article about sRGB. Generic RGB is described in this Apple document, although it doesn't give much information about how or when to use it in iOS apps.
Here's a quote from UIKit's UIColor reference.

On applications linked for iOS 10 or later, the color is specified in an extended range sRGB color space. On earlier versions of iOS, the color is specified in a device RGB colorspace.

When combined with the information from this blog post about how to get the correct colors in your iOS app with Xcode from 2014 (prior to iOS10), it seems that Generic RGB is correct for iOS 9 and lower when using UIColor(red:green:blue:alpha:) and sRGB is appropriate for iOS 10 and above.
Here's a pretty good article explaining the difference between Display P3 and sRGB.
I feel like what all of this means is:

If the color space is Display P3 and the device is >= iOS10, then UIColor(displayP3Red:green:blue:alpha:) should be used without converting the colorspace.
(I think that) If the color space is sRGB and the device is >= iOS10, then UIColor(red:green:blue:alpha:) should be used without converting the colorspace. (More investigation should be done for this case and other colorspaces on iOS10 before implementing.)
Display P3 should be ignored for iOS9 and below.
Other colorspaces should be converted to Generic RGB for iOS9 and below. (This should be tested more before implementing.)

I'll try playing around with various colorspaces to try and see what the best combination for different OS versions and colorspaces is.
Please let me know if you have any other thoughts on the subject.

Your approach sounds good!
In general the goal of R.swift is the generate static references to existing resources.
If we can't be sure we're generating code that 100% of the time works, we'd rather not generate code at all. See coding guidelines.
Applying this to colours; Only generate code for colours if we're 100% it produces the exact color that was in the resource file. E.g. If we're not sure if the color reproduction is exact on iOS 9, then don't generate code for iOS 9.

What's the status of this issue?
I'm using R.swift 3.2.0 and I'm facing a similar issue.
I have created my .clr file using Apple Color Picker tool, my colors are all defined using the sRGB colorspace. However, if I set colors using Interface Builder or using R.color, I don't get the same color at all.
I have tried to look into some workaround and the only solution is to use a different .clr file for R.swift with Generic RGB colorspace. This is not an acceptable solution because I have to maintain 2 files (sRGB and Generic RGB).
SwiftGen offers the same feature but when I put the sRGB or Generic RGB file, I get the exact same output. I think this is somehow related to the colorspace conversion done here.

During WWDC17 Apple announced support voor named Color Assets in Xcode asset folders. I think this is the way forward for predefined colors. So in R.swift 4 we will deprecate clr support and add support voor the named Color Assets. This has already been released in the 4.0.0.alpha.2 thanks to @tomlokhorst.
We could merge a fix that first converts to RGB color space like SwiftGen does, I'm okay with that, but on the long term we will drop the CLR support.

Closing due to inactivity.
Like @mac-cain13 said, the way forward is Color Assets in Xcode 9 / iOS 11. For iOS 10 support, you can use something like ColorAssetCatalog. I haven't used that myself, it's just something I came across.
We can merge full colourspace support for R.clr if there's a complete, well tested implementation. But since R.clr is already deprecated for R.swift 4, I don't want to spent too much time on it.


The userProfile creation

This is for the userProfile app.

Please, can you check it now, I've done some fix to it.


allow basic controller tests to run

This addresses a "FIXME" note that was left in a controller spec.  Apparently a previous iteration of the code base prevented it from running a shared example test.  Now, it runs just fine.  The instance variable mentioned in the comment, @file_base_or_name, is by default set to nil in the controller :show method, which seems to eliminate the need for a mock.

👍


Transitory error with bal build

Got this just now (5 minutes ago);
jc@brie:~/nballerina/compiler$ bal build
Compiling source
	wso2/nballerina:0.1.0
ballerina/io:1.0.1 [central.ballerina.io -> home repo]  100% [==========] 129/
	ballerina/io:1.0.1 pulled from central successfully
ballerina/os:1.0.1 [central.ballerina.io -> home repo]  100% [==========] 79/7
	ballerina/os:1.0.1 pulled from central successfully
ballerina/log:2.0.1 [central.ballerina.io -> home repo]  100% [==========] 17/
	ballerina/log:2.0.1 pulled from central successfully
ballerina/file:1.0.1 [central.ballerina.io -> home repo]  100% [==========] 75
	ballerina/file:1.0.1 pulled from central successfully

	ballerina/regex:1.0.1 pulled from central successfully
ballerina/time:2.0.1 [central.ballerina.io -> home repo]  100% [==========] 25
	ballerina/time:2.0.1 pulled from central successfully
ERROR [ballerina/log/2.0.1::natives.bal:(31:34,31:50)] unknown type 'StackFrame'
ERROR [ballerina/log/2.0.1::natives.bal:(43:5,43:10)] unknown type 'Value'
ERROR [ballerina/log/2.0.1::natives.bal:(179:22,179:27)] unknown type 'Value'
ERROR [ballerina/log/2.0.1::natives.bal:(180:17,180:33)] unknown type 'StackFrame'
ERROR [ballerina/log/2.0.1::natives.bal:(187:43,187:46)] undefined function 'v'
error: compilation contains errors

Trying again gave the same errors.
Then  I did bal dist update which updated the update tool to 1.3.5.
Then I did bal build --offline which completed successfully.
Then I did bal build which also completed successfully.

This was due to an issue in Ballerina Central. This is fixed now.


Isomorphic usage of react-router

Previously, I could use Router.run() for server and for client, the question is, can I use match() in this way?
For example
let params = {
 routes: routes,
 location: location
};

if (history) {
 params.history = history;
}

match(params, (/*args*/) => {
 // do something
});

do you mean https://github.com/rackt/react-router/blob/master/docs/advanced/ServerRendering.md ?

This example is just for server rendering, but I guess that match() can be used and on client, because I can pass history in params and probably it can be one of the history instances (Browser, Memory, Hash).

on the client:
const history = createHistory();

// It is not possible currently to get the location object synchronously through the history object.
// so we end up having two sources of truth for the current location or nesting the rest of this in a callback.

const location = createLocation(location.pathname + location.search);
match({ location, history, routes }, (err, nextLocation, props) => {
  // props already has a history object
  // unfortunately this history object is wrapped by match, and Router will attempt to wrap it again
  // which will give you headaches. So here we override that history object with the first.
  // not a clean solution but it is the only solution at the moment.
  props.history = history;
  const inst = React.render(
      React.createElement(Router, props),
      DOMContianer
  );
});

Its been commented here and here that match was never intended to be used client side (and paradoxically that it was never intended to be public api), which means, by intent, if you are not using match client side you will end up writing code using different api's depending on the environment (server vs client) which is not really isomorphic friendly IMHO.

Agree with you. In such case we will probably have something like this.
// client
let props = {
  history,
  location,
  children: routes
};

React.render(<Router {...props}/>, document.body);

//server
match({location, props}, (err, redirectInfo, renderedProps) => {
 // send response
});

I still have problems, even this history replacements. Then routes changed on client, nothing happens.

// client
let props = {
  history,
  location,
  children: routes
};

React.render(<Router {...props}/>, document.body);

//server
match({ location, routes }, (err, redirectInfo, renderedProps) => {
 // send response
});

So, to be clear, I can not use match() on client and on server even if I replace history?

Yes you can. I gave an example above on how to accomplish this. Are you seeing errors when using match?

No, without errors, but and nothing happens. Routing does not work after initialization in this way.

@BerkeleyTrue, thanks for sharing your solution here. I thought that I was losing my mind(!) earlier today while trying to cook up an isomorphic app using the latest react-router. The client-side code was not receiving the right history information until I added the code you presented in this comment. I'm really hoping there's a better solution to this, but I'm not yet up-to-speed enough on all of the changes for v1.0.0 to suggest one.

@BerkeleyTrue thank you, your solutions works.
An example of usage.
// client
import router from 'path/to/isomorphic/router';
import createLocation from 'history/lib/createLocation';
import createBrowserHistory from 'history/lib/createBrowserHistory';

const location = createLocation(document.location.pathname, document.location.search);
const history = createBrowserHistory();

router(location, history)
  .then((reactEl) => {
    React.render(reactEl, dist);
  }, (err) => {
    throw err;
  })

// server
import routing from 'path/to/routing/middleware';
// ...
app.use(routing)
// ...

// routing middleware
import React from 'react';
import createLocation from 'history/lib/createLocation'
import createMemoryHistory from 'history/lib/createMemoryHistory';
import router from 'path/to/isomorphic/router';

export default function(req, res) {
  let location = createLocation(req.url);

  router(location, history, res, req)
    .then((reactEl) => {
      try {
        let reactStr = React.renderToString(reactEl);
        res.send(reactStr);
      } catch (err) {
        res.status(500).send({error: err.toString()});
      }
    }, (err) => {
      console.log('err', err);

      res.status(500).send({error: err});
    });
}

// router
import Promise from 'promise';
import Router, {match, RoutingContext} from 'react-router';
import routes from './routes.js';

function getRootComponent(renderProps) {
  let component = null;

  if (__SERVER__) {
    component = (<RoutingContext {...renderedProps}/>);
  } else {
    component = React.createElement(Router, renderedProps);
  }

  return component;
}

export default function (location, history, res, req) {
  return new Promise((fullfill, reject) => {
    match({routes, location, history}, (error, redirectLocation, renderProps) => {
      if (!error) {
        if (renderProps) {
          renderProps.history = history;
        }

        fullfill(getRootComponent(renderProps));
      } else {
        reject(error);
      }
    });
  });
}

I think this issue can be closed for v1.0.0-rc1.

@mikhail-riabokon In your example code how would you pass in props to your root component on the server?

If I understood your questions correct, the answer is
function getRootComponent(renderProps) {
  let component = null;

  if (__SERVER__) {
    component = (<RoutingContext {...renderedProps}/>);
  } else {
    component = React.createElement(Router, renderedProps);
  }

  return component;
}

If not, could you explain which props do you want to pass?

Thanks for responding so quickly.
I have two objects website and page that my root component App will require in order to render properly.  How you would recommend passing those two props to App.  I hope this is clearer.

maybe this will help.  I keep getting this:
Warning: Failed propType: Required prop website was not specified in Page. Check the render method of RoutingContext.
Can't figure out how to pass in the prop when rendering on the server...

I understand that you mean.
For example you express app
...
const app = express();
...
app.use(express.static('path/to/assets'))
app.use(routing()) // isomorphic router
...

Then you go by route for example /, isomorphic router will receive renderedProps, but if you have some assets on the page, for example /asssets/images/home.png isomorphic router will not receive renderedProps, that's why you have such type of warnings

I have updated an example of isomorphic usage redux-forms, where you can reach simple solution to your question.
https://github.com/mikhail-riabokon/redux-form-universal-example

Thanks - checking it out now.

You're welcome. Let me know in case of any questions.

@webular Did you find a solution? I'm running into the same issue and can't find the solution in @mikhail-riabokon 's repo. TIA!

@nodu Do you use for server rendering <RoutingContext />?

Yes @mikhail-riabokon I'm using it like this with Redux and Koa:
  html = ReactDOMServer.renderToString(
        <div>
          <Provider store={store}>
              <RoutingContext {...renderProps}/>
          </Provider>
        </div>)
  this.body = renderFullPage(html, initialState)

My use case was that I needed to pass two objects into my RoutingContext -- website and page.
Here is what I ended up doing.
function createElement(Component, props){
   return <Component {...props} website={website} page={page} />
}

return (<RoutingContext createElement={createElement} {...renderProps} />);

In order to pass those props into your Routing Context you need to use createElement.
https://github.com/rackt/react-router/blob/master/docs/API.md#createelementcomponent-props
In my App component I was then able to reference website and page via props.  i.e.  this.props.website

Thanks @webular for the code snippet, it's now working.  But I have SERIOUS misgivings from this approach.    This just seems really inefficient and creates more boilerplate to write (and forget you need to write). I also pass actions as props to my dumb components in my flux implementation (Redux), so I would have to pass those into the RoutingContext as well.  Is there a better, more sane solution?

Will creating a parent route around all your routes not work?  Then I would sort out which props get passed down in the parent component?
I've got a parent component App which passes props to my "smart components".
Will that not work in your case?

You can pass props into children from parents by cloning them. Otherwise React context is pretty good for this sort of thing.
Otherwise this discussion seems to have no bearing on this closed issue.


Python project as asset.

I was thinking if it would be possible to give an existing project in python to gpt with an explanation what this project does and the question if it could build the same app in a language like c.

with about the same success rate as normal based on the quality of prompt and the overall workflow of changes step type and refining the prompt, giving feedback at run-end (then running use_feedback if its working. the other day i was having problems with 2_word step types with an underscore.. haven't checked today since i pulled again. if and when it works its a good tool for that, always ad a note to the last question, reminding it that  its "transposing"/"transliterating" from one format to another ) ... ive had really good results with quick tests of prompt templates like this
###<(interpreted as a full stop) HEADER MESSAGE/TITLE BAR:
# <(like a \n\n indent into a sub menu/value array)BRIEF DESCRIPTION:

###
# 1. BULLET
# 2. POINTS
# 3. OF
# 4. FEATURES

### CURRENT REVISION:
``` FILETYPE

CODEBLOCK BODY

FULL DESCRIPTION:

EXECUTE INSTRUCTIONS:
1. LIST
2. OF
3. ADDITIONAL
4. INSTRUCTIONS
! RETURN ALL REVISED OUTPUT AS FILE TYPE!

the required backticks keep messing everything up xP


added: timestamp to base tx

Closes #9

I think that makes sense to add it once it mined
Best regards,
Andrei
On Mon, 1 Aug 2022, 22:09 Argeare5, @.***> wrote:

@.**** commented on this pull request.
In src/web3/store/transactionsSlice.ts
https://github.com/bgd-labs/fe-shared/pull/10#discussion_r934510196:

@@ -82,6 +83,7 @@ export function createTransactionsSlice({
from: tx.from,
to: tx.to,
nonce: tx.nonce,

   timestamp: tx.timestamp || Math.floor(Date.now() / 1000),

after a little research, I saw that there is no timestamp in the tx, we
need to get timestamp from block number, but we don't have block number
when tx just created.
I suggest doing this:

Pass the current date to the timestamp when tx just created. Since
we need timestamp while the tx is pending.
Update tx timestamp at the moment when we update the statuses.

What do you think about this @kyzia551 https://github.com/kyzia551
@satanworker https://github.com/satanworker ?
—
Reply to this email directly, view it on GitHub
https://github.com/bgd-labs/fe-shared/pull/10#discussion_r934510196, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AEFXTWWCQQ2S4FJCRYEWDNDVW7EBBANCNFSM55GXR4PA
.
You are receiving this because you were mentioned.Message ID:
@.***>


make omar happy

with this change 'EUNIT' will only be set when running the eunit provider and 'COMMON_TEST' will be set by the ct provider. this does mean users that run both eunit and ct tests will see a recompile when alternating between the two test runners

also this makes @omarkj happy

I'm ok with this change but not positive it is what everyone wants? @ferd

Well it sure worked before then. PErsonally the macro definition changes nothing since including eunit.hrl defines both anyway and I always include eunit's assertions. I'm okay with it, merging.