Refactored translatable rules functionality

Description
I have refactored the translatable rules functionality to add some features and fix a bug.

Not published locales now getting validated (required rules will be disabled)
Now supports defining rules for fields as Array (like Laravel it does)
Fixed bug in the If-Condition where it checks for 'required_' (haystack and needle were interchanged)

We came across this when we received a database error because it tried to save a text that was too long. Unfortunately, the validation was not performed because the language was not (yet) published.

Hey @domihagen,
Thank you for the pull request! I have checked it and it seems to work fine.
This would however introduce a backward compatibility issue. For this I think we should have a switch on the Request, something like:
protected $validateInactiveLanguage = true; // True is the default as this is the current behavior.

What do you think? Also, what are your thoughts @ifox?

Hey @domihagen,
Sure, and please correct me if I am wrong.
Currently, the validation will check for inactive languages as well, this is the current design and users might rely on this behavior.
Once this pr is merged, that changes as the inactive languages are no longer validated. Which might be unwanted by users relying on that behavior.

hey @domihagen,
I did a check on 2.x this morning, the video illustrates the current behavior:

Sorry if I am still missing the point.

@domihagen Ok, I got it now completely! Thank you for your patience. Looks good to me.


Add example of using ClickHouse

ClickHouse is quite popular for data analysis tasks.

LGTM - @owahltinez any objections to merging this?


Jordan Data Quality Checks

As discussed with Gunnar.
jor_quality_checks.xlsx

have we applied these data quality checks?

no


"border-corner-shape"

"border-corner-shape"
http://dev.w3.org/csswg/css-backgrounds-4/#corner-shaping
http://leaverou.github.io/corner-shape/

This issue was imported from Google Moderator
Moderator votes: +4
Issue added by zasdasda on 2014-04-08
To vote this issue up or down, simply include +1 or -1 in your comment.

@Fyrd:
#1477 and this one request the same.
As there are more votes on #1477, close this one as duplicate (and maybe slightly rename the other)?

+1

Yup, duplicate of #1477


Add trie coprocessor circuits.

Closes #397.
Some issues for further work are noted in TODOs and are, I believe, captured in #745.

If merging doesn't work, rebasing might solve it due to https://github.com/lurk-lab/lurk-rs/pull/764


add != to PartTreeFirestoreQuery

fixes #2563

Looks good. @meltsufin @dmitry-s -- Wanted to revisit our discussion of the new repo. Do you plan to include these changes in the new repo? I think we are getting a little out of sync with a few PRs related to datastore/firestore.

adding support for NOT_IN as well


Match PLDM: correct Gard to Guard

Fixes 586859
This is 1060 only.
https://github.com/ibm-openbmc/pldm/pull/478 went into 1060 and changed the spelling of pvm_rpd_gard_policy to pvm_rpd_guard_policy. Need to match that bios attribute spelling.
Tested: None, but searched and verified this is the same spelling.

Also, Change need to be made in translation file.

I tried to change everywhere.
Used
grep -R 'gard'  --exclude=*NoticesText.vue  *
to find.


add rule (rrd): component uses js and not ts

This should be only warning level and implement only when we have the cli options to filter out rules as this one might be quite controversal.
As I understand nuxt 4 might be defaults to ts, so <script setup> will mean ts.

@rrd108  Isnt this rule covered by the plainScript rule?

As I understand nuxt 4 might be defaults to ts, so

I created a PR for this @rrd108


jsos/Soften Rainbow artifacting

Reduce Rainbow artifacting at long distances
Fade out contouring at distance
Fixes AB#1218

As per discussion with @Jossos, I'm going to close this PR. It is very close to solving the issue, but we've had a discussion about other solutions which may work better.


GSD Request

--- GSD JSON ---
{
  "vendor_name": "Linux",
  "product_name": "Kernel",
  "product_version": "versions from v4.15 to before v5.4.169",
  "vulnerability_type": "unspecified",
  "affected_component": "unspecified",
  "attack_vector": "unspecified",
  "impact": "unspecified",
  "credit": "",
  "references": [
    "https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux.git/commit/?id=b2cfd8ab4add53c2070367bfee2f5b738f51698d",
    "https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux.git/commit/?id=992649b8b16843d27eb39ceea5f9cf85ffb50a18"
  ],
  "extended_references": [
    {
      "type": "commit",
      "value": "b2cfd8ab4add53c2070367bfee2f5b738f51698d",
      "note": "introduced"
    },
    {
      "type": "commit",
      "value": "992649b8b16843d27eb39ceea5f9cf85ffb50a18",
      "note": "fixed"
    }
  ],
  "reporter": "joshbressers",
  "reporter_id": 1692786,
  "notes": "",
  "description": "ipmi: Fix UAF when uninstall ipmi_si and ipmi_msghandler module\n\nThis is an automated ID intended to aid in discovery of potential security vulnerabilities. The actual impact and attack plausibility have not yet been proven.\nThis ID is fixed in Linux Kernel version v5.4.169 by commit 992649b8b16843d27eb39ceea5f9cf85ffb50a18, it was introduced in version v4.15 by commit b2cfd8ab4add53c2070367bfee2f5b738f51698d. For more details please see the references link."
}
--- GSD JSON ---

This issue has been assigned GSD-2021-1002793


comp report log likelihoods not changed for DM

Thanks Grant Thompson for reporting the following:

...I was hoping that the log likelihoods listed in CompReport.sso for a given year/fleet/kind would sum to the value listed in the FIT_LEN_COMPS or FIT_AGE_COMPS section of Report.sso, but they are not even close.
I am using the Dirichlet-multinomial option, and I note that the SS change log includes an entry for 4/5/21 referencing "change logL for D-M composition," so I was wondering if perhaps the change did not get carried over into the code that generates the CompReport.sso file (just a wild guess on my part). ...

Investigate this and see if that is what is going on. (KD: we don't test CompReport.sso at all in our testing routines, so this is certainly possible)

I think we need a code review here.
In ss_prelim, there is:
int z1=tails_a(f,i,1);
int z2=tails_a(f,i,2);
offset_a(f,i) += gammln(nsamp_a(f,i) + 1.) -
sum(gammln(1. + nsamp_a(f,i)*obs_a(f,i)(z1,z2)));
then in ss_objfun, there is:
age_like(f,i) = -offset_a(f,i);  //  so a perfect fit will approach 0.0
and
if(Comp_Err_A(f)==1) dirichlet_Parm=mfexp(selparm(Comp_Err_Parm_Start+Comp_Err_A2(f)))nsamp_a(f,i);
if(Comp_Err_A(f)==2) dirichlet_Parm=mfexp(selparm(Comp_Err_Parm_Start+Comp_Err_A2(f)));
and
temp+=sum(gammln(nsamp_a(f,i)  obs_a(f,i)(tails_w(1),tails_w(2))
+ dirichlet_Parmexp_a(f,i)(tails_w(1),tails_w(2))));
temp-=sum(gammln(dirichlet_Parmexp_a(f,i)(tails_w(1),tails_w(2))));
and
age_like(f,i)-=temp;
then in write_report, the age_like are output for each obs in fit_age table, and these sum to the total reported logL
age_like(f,i) = -offset_a(f,i);  //  so a perfect fit will approach 0.0

I'm worried that I made an error in the changes to the DM likelihood associated with change in the code to add the constant of integration as discussed under https://github.com/nmfs-stock-synthesis/stock-synthesis/issues/31.
Grant confirmed by email back in August that a model run in versions before and after that change had the same parameter estimates, so the issue seems to just be with the added constant.

I created a test case from "simple" in which I verified that for multinomial with both length and age comp:
a) logL reported in fit_age and fit_length matches the sum of the bin specific logL in compreport.sso
b) by creating an input observation from the expected values output, verify that the model fits this perfectly and that it's logL is near 0.0, as it should be because it includes subtraction of an offset which is the logL for a perfect fit.
Then switched to test case from hake_2018 that has only age comp and uses D-M.
It passes neither of the comparisons that are passed for multinomial.
Problem 1:  calculation of offset seems not to be using the dirichlet parameter
Problem 2:  reporting of bin-specific logL is not using D-M calcs.  But Pearson residuals do appear to be using D-M calcs
Problem 3:  logL of D-M has a component that is not bin-specific.  If we want the compreport logL output to sum to match the overall logL of that observation, then this component will need to be spread across the bins of that observation.

Sorry for all these issues. I just looked through the changes in https://github.com/nmfs-stock-synthesis/stock-synthesis/pull/132/files. I thought that the whole point of the offset was that it represents the part of the likelihood which is independent of the parameters, and thus can be calculated in SS_prelim.tpl.
I tried to find notes related to checking the likelihood against the Thorson et al. paper but unfortunately haven't found them yet. Also, I'm on leave the next two weeks but can prioritize working on this for the week of Jan 3.

The offset for D-M that is in place now may well be as good as it can be.  Don't sweat it.

@Rick-Methot-NOAA, I finally took the time to double-check the implementation of the Dirichlet-multinomial likelihood and reread your note above https://github.com/nmfs-stock-synthesis/stock-synthesis/issues/219#issuecomment-997059352 from December.
I verified outside of SS3 that the DM likelihood (when choosing the linear parameterization) matches equation (10) from Thorson et al. (2017) shown below.

When revising the DM offset in Feb 2021 (#31), I failed to understand the role of the offset for multinomial likelihood as you noted in the comment referenced above. That is, for the multinomial, I now see that the goal of the offset is to make the logL approach 0.0 for a perfect fit, not to match the full likelihood equation including normalizing constant. So given the important log-likelihood term of n*obs*log(exp), the offset is set to -n*obs*log(obs).
Having the logL approach 0 for perfect fit probably has lots of benefits, including keeping the scale of all the likelihood contributions on a similar order of magnitude (in contrast to the current DM likelihood which can have really large values, and can be negative as well). However, this approach gets in the way of the one perceived benefit of having the DM likelihood include the constant term, which is facilitating comparing the DM vs multinomial likelihoods, as noted in https://github.com/nmfs-stock-synthesis/stock-synthesis/issues/31#issue-737136387.
Now that I understand the difference between the two implementations I have a few questions on possible few ways forward.

Would there be any purpose in giving the user the option of the multinomial likelihood with the full constant term to facilitate comparison of the two likelihood options? I presume this comparison would only work if the input sample sizes were not adjusted, so maybe this wouldn't be valuable after all since we typically always weight the comp sample sizes when using the multinomial likelihood.
Is there a better offset for the DM likelihood that would bring the scale of the values down, or should we just stop messing with it and leave it as is, matching the published equations?

@james-thorson-NOAA, do you have any thoughts on this?

Looking back over https://github.com/nmfs-stock-synthesis/stock-synthesis/issues/219#issuecomment-997059352 above yet again, I see that the DM likelihood values should change in future models no matter what we decide on the two questions I just posed in order to resolve the problems related to bin-specific logL so that the reported values are correct and the compreport's logL output sums to match the overall logL of that observation.

My recommendation is that we want to allow the loglikehood to also be comparable between the DM and multinomial as primary goal (thus including the integration constant for both), but allowing a switch to compute the loglikelihood measured relative to a saturated fit.  This is one step along the path towards computing percent-deviance-explained ... a standard diagnostic for GLMs, which involves calculating the saturated value (which I assume is what is meant by a renormalization that results in a 0 value for a "perfect fit") but also one or more "null values" that could be computed by stripped out recruitment deviations or something.  So I see the value of the renormalization of log-likelihoods to have 0 correspond to perfect fit, but think that's a secondary concern that should be accessed through an explicit flag.
just my two cents :)

Reporting bin-specific contribution to the logL for each bin was the challenge I saw because the offset depends on the D-M adjusted sample size, so cannot be pre-computed.
So, whatever we do, I think we should start by creating multinomial and D-M functions that can be called, rather than the current spaghetti code that repeats these complex equations for length and age and males and females, and the for generalized size freq.
Then we could create offset on the fly as needed by calling fxn once to get the offset using (n, obs_comp, obs_comp, tails) and then again with  (n, obs_comp, exp_comp, tails).

My TMB code is below ... I recommend writing it out more spaciously like this (and assume a compiler can manually compress the multiple additions with no loss of efficiency) and putting it in some distributions.cpp file or whatever, please tell me if you want help.
template<class Type>
Type ddirmult( vector<Type> x, vector<Type> prob, Type ln_theta, int give_log=0 ){

  // Pre-processing
  int n_c = x.size();
  vector<Type> p_exp(n_c);
  vector<Type> p_obs(n_c);
  Type Ntotal = x.sum();
  p_exp = prob / prob.sum();
  p_obs = x / Ntotal;
  Type dirichlet_Parm = exp(ln_theta) * Ntotal;

  // https://github.com/nmfs-stock-synthesis/stock-synthesis/blob/main/SS_objfunc.tpl#L306-L314
  // https://github.com/James-Thorson/CCSRA/blob/master/inst/executables/CCSRA_v8.cpp#L237-L242
  // https://www.sciencedirect.com/science/article/pii/S0165783620303696

  // 1st term -- integration constant that could be dropped
  Type logres = lgamma( Ntotal+1 );
  for( int c=0; c<n_c; c++ ){
    logres -= lgamma( Ntotal*p_obs(c) + 1.0 );
  }

  // 2nd term in formula
  logres += lgamma( dirichlet_Parm ) - lgamma( Ntotal+dirichlet_Parm );

  // Summation in 3rd term
  for( int c=0; c<n_c; c++ ){
    logres += lgamma( Ntotal*p_obs(c) + dirichlet_Parm*p_exp(c) );
    logres -= lgamma( dirichlet_Parm * p_exp(c) );
  }

  if(give_log) return logres; else return exp(logres);
}

Thanks Jim.  This code is exactly what I planned to write out in ADMB.  Will put it into the SS file SS_miscfxn.tpl


local pe execution

Execute EXE and DLL files from memory, with support of TLS callbacks and exception handling

This is awesome! thank u <3


feat: single sidecars exe and versioned templates

There are a few things that happen here:

combine utilities is a single executable that contains the proxy and cloning utilities we run in sidecars
enable templating of the cloning and auth proxy containers/sidecars that is based on the version of the image passed in the CRD

This later part about "versioned templating" is required to prevent the following scenario:

we will publish the sidecar image with every release
the sidecar image is hardcoded as a const in the code
someone updates this hardcoded constant
redeploys amalthea on a cluster that already has many sessions
amalthea restarts and patches all existing images to use the new sidecar image
this causes all existing running sessions in the cluster to restart

There are other reasons too. For example we upgrade the version of the sidecar image but its api has changed. Then either new sessions with the new sidecar work or the old ones are patched restarted but now when they start back up they keep crashing because the sidecars image API has changed.

It's looking good however I am just not sure about what "template" refers to in this context.

There's the fact that these are template functions but in the k8s context, some resources such as deployment uses template in the sense of "reference pod declaration to be used when creating pods".

In this case "template" means the real k8s children resources I generate from the CR specification. I.e. the PVC, Ingress, Service, Statefulset, etc.

This change is part of the following stack:

#745 ◀

#748

Change managed by git-spice.


Where to find available offering types to create cases?

Hi,
Are all the available offering types that can be used to create a case documented somewhere?
I found an example here for Cloud Object Storage: https://cloud.ibm.com/apidocs/case-management?code=python#createcase
Where can I find the group, key and name for the rest of available offering types?
Thanks.

I belive this was answered via slack.  If not, please re-open.


ImportError: cannot import name 'Trainer' from 'transformers'

Hi, I am getting this error with your transformer environment. I have tried python version 3.7 and 3.8.
Any clue on how to solve this?
Thanks!

Just solved the issue. There seems to be a problem with apex installation.


squashctl --help & --version should NOT require a kubecontext

squashctl --help and squashctl --version should work correctly even when run without a kubecontext environment set. Currently these panic looking for the kubecontext.

Can we close this issue as #153 has fixed the issue.


Support for cascading inheritance

RIght now it is easy to inherit from Learn2_new thanks to the inheritance_template. However, it can be messy to inherit from one or multiple children of Learn2_new.
For example, intrinsically_interpretable_probabilistic_regression could benefit from inheriting from probabilistic_regression and committor_projection_NN. Instead, right now, it inherits directly from Learn2_new, with the consequence that much of its code is duplicated with its similar siblings. This has already made maintaining the code troublesome.
It would then be interesting to implement more flexibility for the inheritance

After some tests, everything seems to be working fine


.NET Core 2.1 fails on linux cluster.

I have a simple rest service deployed to an ubuntu service fabric cluster which starts perfectly when using .NET core version 2.0 but fails with the following error with version 2.1. Is 2.1 officially supported for a linux cluster ? I could not find this information anywhere. Note that 2.1 works on our Windows cluster.
Unhealthy event: SourceId='System.RA', Property='ReplicaOpenStatus', HealthState='Warning', ConsiderWarningAsError=false. Replica had multiple failures during open on _NodeType2_2. API call: IStatelessServiceInstance.Open(); Error = System.ArgumentException (-2147024809) Item has already been added. Key in dictionary: 'LD_LIBRARY_PATH'  Key being added: 'LD_LIBRARY_PATH'    at System.Collections.Hashtable.Insert(Object key, Object nvalue, Boolean add)    at System.Environment.ToHashtable(IEnumerable1 pairs)
at Microsoft.Extensions.Configuration.EnvironmentVariables.EnvironmentVariablesConfigurationProvider.Load()
at Microsoft.Extensions.Configuration.ConfigurationRoot..ctor(IList1 providers)    at Microsoft.Extensions.Configuration.ConfigurationBuilder.Build()    at Microsoft.AspNetCore.Hosting.WebHostBuilder..ctor()    at HelloWorld.HelloWorld.<>c__DisplayClass1_0.b__1(String url, AspNetCoreCommunicationListener listener) in c:\users\davies\source\repos\HelloWorldCloud\HelloWorld\HelloWorld.cs:line 38    at Microsoft.ServiceFabric.Services.Communication.AspNetCore.AspNetCoreCommunicationListener.OpenAsync(CancellationToken cancellationToken)    at Microsoft.ServiceFabric.Services.Runtime.StatelessServiceInstanceAdapter.OpenCommunicationListenersAsync(CancellationToken cancellationToken)    at Microsoft.ServiceFabric.Services.Runtime.StatelessServiceInstanceAdapter.System.Fabric.IStatelessServiceInstance.OpenAsync(IStatelessServicePartition partition, CancellationToken cancellationToken) For more information see: http://aka.ms/sfhealth

Issue posted on https://github.com/Azure/service-fabric-issues/issues/1178


Update to add extended distro support and one-liners

The title says it all.

The title says it all.

I don't understand

@spider863644 Is my contribution needed?

@spider863644 Is my contribution needed?

Yes

@spider863644 Is my contribution needed?

Yes

You are welcome to merge it anytime. @spider863644

Thanks ❤


Configure etcd CNAME as a service

Instead of the semi-hard-coded CNAME generation in the coredns-mdns
plugin, use an mdns service record to create the etcd CNAMEs. This
is more flexible and will allow the plugin to be more generic since
it won't need to have OpenShift-specific logic in it.

Build SUCCESS, see build http://<IP_ADDRESS>:8080/job/dev-tools/366/

No longer needed. This was included as part of #224


删除文件导致上级文件夹也被删除

alluxio fs mkdir /t1
alluxio fs mkdir /t1/t2.txt
alluxio fs rm /t1/t2.txt
alluxio fs ls /t1  路径找不到

Hi, I can't reproduce this on my Alluxio instance. Can you share more details, e.g. what's your Alluxio setup (local, cluster, under storage), what commands did you use, and what were their output, and any error logs?

@JYHarvest may you share your master.log? I feel alluxio fs mkdir /t1 may not succeed thus alluxio fs ls /t1 failed

feel free to re-open this issue when you can provide more info


As the Designer, I want to use Homebrew so that my Darwin and Linux configurations are similar.

[x] Use Linuxbew
[x] install.sh should use darwin and linux directories.
[x] Test with Darwin
[x] Test with Linux

Git-commit 20774c0 completed this task.


Cursor on query ignores trashed relations

Laravel Version: 6.20.18
PHP Version: 7.3.25
Database Driver & Version: MySQL 5.7.31

Description:
Cursor on query ignores withTrashed() on relations.
Steps To Reproduce:
Lets say we have a Contact model with phones that have types and we want to take deleted phones also
$query = Contact::query();
$query->with([
    'phone'             => function($q) {
        $q->with('types');
        $q->withTrashed();
    },
]);
$query->whereHas('cars', function(Builder $builder1) use ($car) {
    $builder1->where('car_id', '=', $car);
});

if you have 10 phones and 2 are deleted and you do
$query->get()
it will bring all 10 results
If you do loop with cursor
 $query->cursor() as $item
and try to get $item->phone->number
it will bring return null for those 2 deleted records

pretty sure with() does not work at all with cursor()

pretty sure with() does not work at all with cursor()

If you do the below you will see that works...
 $query = Contact::query();         $query->with('phone');         foreach ($query->cursor() as $item){             logger($item->phone->number);         }

Hi there,
Thanks for reporting but it looks like this is a question which can be asked on a support channel. Please only use this issue tracker for reporting bugs with the library itself. If you have a question on how to use functionality provided by this repo you can try one of the following channels:

Laracasts Forums
Laravel.io Forums
StackOverflow
Discord
Larachat
IRC

However, this issue will not be locked and everyone is still free to discuss solutions to your problem!
Thanks.


Introduction of requirements tracking feature

Add support for basic requirement tracking
Generates a by requirement summary as well as a JSON report containing each requirement and the test status

This feature seems very interesting!
Could you give some more detail on how it works?

@svancau This is a prime example of a feature that should have an issue opened and some level of discussion about it, even if you think that delays things. Personally I feel this sort of feature is out of the scope of cocotb. I maybe alone on that though.
Such work though should also include

Documentation on how to use it
Examples of how to use it
A running testcase


Readme example is not working

Copy-pasting the example from readme produces this:
INFO: ('<IP_ADDRESS>', 33118) - "GET / HTTP/1.1" 500
ERROR: Exception in ASGI application
Traceback (most recent call last):
  File "<venv>/lib/python3.7/site-packages/uvicorn/protocols/http/httptools_impl.py", line 372, in run_asgi
    result = await asgi(self.receive, self.send)
  File "<venv>/lib/python3.7/site-packages/starlette/middleware/errors.py", line 125, in asgi
    raise exc from None
  File "<venv>/lib/python3.7/site-packages/starlette/middleware/errors.py", line 103, in asgi
    await asgi(receive, _send)
  File "<venv>/lib/python3.7/site-packages/starlette/exceptions.py", line 74, in app
    raise exc from None
  File "<venv>/lib/python3.7/site-packages/starlette/exceptions.py", line 63, in app
    await instance(receive, sender)
  File "<venv>/lib/python3.7/site-packages/starlette_api/routing.py", line 168, in awaitable
    injected_func = await app.injector.inject(endpoint, state)
  File "<venv>/lib/python3.7/site-packages/starlette_api/injection.py", line 173, in inject
    state[output_name] = resolver(**kw)
  File "<venv>/lib/python3.7/site-packages/starlette_api/components/validation.py", line 79, in resolve
    query_params = validator().load(dict(query_params), unknown=marshmallow.EXCLUDE)
AttributeError: module 'marshmallow' has no attribute 'EXCLUDE'

UPD: I used starlette-api[full] for install

It seems that you're using an older version of marshmallow (2.x) and it's required a 3.x version.

I have no marshmallow defined in Pipfile, which means I got it installed with starlette-api, and that means that the project dependencies are off somehow. The version of marshmallow specified in pyproject.toml is ">2" which means 2.19.1 suits fine.

It should be fixed with version 0.6.4


DNM: riff function local commands

Introduces two new commands:

riff function local build which builds a local directory and publishes an image
riff function local run which builds a local directory and runs the function

Both commands have no dependency on a Kubernetes cluster or an installed riff system. They do require a local docker daemon.
This PR is a superset of #1070, it must merge first.
So far this PR only contains superficial docs and tests. It is demo quality.

Fixes #972

I think we should add flag for env vars to the local run command
  --env stringArray                environment variable expressed in a 'key=value' format

If I specify a port we should set environment variable for the port as PORT and SERVER_PORT so the app can pick these up

I think we should add flag for env vars to the local run command

I assume you're proposing these as runtime env vars? Would we ever need to also set build env vars, if so how would a user know the difference?

If I specify a port using function local run --port we should set environment variable for the port as PORT and SERVER_PORT so the app can pick these up

The running container shouldn't care about the value of --port. The flag exposes a port that is running inside the container to the host. The build pack already exposes 8080, which is what provides the default --port value. We could hard code PORT=8080 which is normally set as part of the runtime contract.

Nit: so far, the riff command structure has always been riff <resource_type> <action>, it would be nice if these new commands could stick to this scheme.

I assume you're proposing these as runtime env vars

Yes, that was my intention, although looking at pack run I don't see a way to pass those in, so ignore my env var and port suggestions

This PR is now focused on only the run command, the description has been updated.


failed to install code-server by script

OS/Web Information

Web Browser: N/A
Local OS: Ubuntu 20.04.3 LTS
Remote OS: N/A
Remote Architecture: N/A
code-server --version: 3.12.0
shell: zsh installed by omz

Steps to Reproduce

open terminal
execute curl -fsSLV https://code-server.dev/install.sh | sh

Expected

this command will download the installation script and execute it
Actual

thrown error:
➜  ~ curl -fsSLV https://code-server.dev/install.sh | sh
sh: 1: Syntax error: "(" unexpected

Logs

➜  ~ curl -fsSLV https://code-server.dev/install.sh | sh
sh: 1: Syntax error: "(" unexpected

Screenshot

Notes

This issue can be reproduced in VS Code: No

-V makes cURL output version info instead of fetching the URL so removing it should fix this.


Issues with contrib settings/examples for check_vmware_disk_consolidation plugin

While configuring new service checks on our prod Nagios instance I realized that the contrib files (which I used as a template) had some issues:

referenced wrong service group
missing alternative service group
service_description contains VMware VMs - prefix

instead of VMware - as other service check descriptions (e.g., Vmware vCPUs Allocation)
by itself not a problem, but it is inconsistent with the other examples

https://github.com/atc0005/check-vmware/blob/947dbe158ba100f9257bdca4f15e6aae95bc69f5/contrib/nagios/etc/nagios3/conf/hosts/servers/vc1.example.com.cfg#L333-L343

@tubby1981 FYI (just in case you were using the template as a reference)

Fixed in GH-189.


Self-modifying operators for constexpr complex and imaginary

In C++14, constexpr evaluations can modify variables, so it's nice to have compile-time classes which can be manipulated.
I wrote a compile-time polynomial class, and I wanted to use it with complex coefficients. Unfortunately, std::complex does not have constexpr operators, so I couldn't use it.
To use smath::complex instead, the member variables should not be const, and it's handy to have the self-assigning operators. This PR adds those.
I also added a polar function (like std::polar, except constexpr.)
Thanks.

Uh, it's true that this library is a bit rusty and its original design choices when it was still C++11-only couldn't take into account C++14's enhanced constexpr. Now could be a good time to change that.
Anyway, thanks for your pull request, I will review it tonight (UTC+1). Apparnetly it looks mostly fine, so you can expect it to be merged soon enough :)

I merged your pull request and then added more things to complex and imaginary even though they're still a bit incomplete. Some things might break your code, most notably the constructor to construct an imaginary from a real value has been made explicit for obvious reasons. If anything else breaks your code, don't hesitate to submit an issue.
Thanks again for your additions!


unfold returns wrong results for next page requests

What did you implement:
unfold actually picks the last item from the next page as first item.
How did you verify your change:
let mut opts = RepoListOptions::builder();
opts.sort(Sort::FullName);

let list = rt.block_on(github.repos().list(&opts.build()))?;
opts.per_page(2);
let iter = rt.block_on(github.repos().iter(&opts.build()).collect())?;

for (e1, e2) in list.iter().zip(iter.iter()) {
    println!("{}  {}", e1.name, e2.name);
    assert_eq!(e1.name, e2.name);
}

Any ideas why items is reversed?  It always confused me.

yes, because of the use of items.pop() src/lib.rs#L1056


Netlify CMS "authorized popup"

Hi!
I've met a weird CMS behaviour. I have a private repo on github and the website is stored on AWS with a custom domain.
After setting up the custom domain (like it is in placeholder

) I could go to /admin page and see a "login with github" button. But after authorization process was complete I only had a hanging popup window with a line "Authorized". No errors,  console was clean.

Finally I've found the way to complete the process - I had to remove "www." from custom domain field, and it worked well. (Thanks to https://github.com/erquhart)
But solution wasn't obvious without any warnings or errors. And placeholder has "www" prefix.
Thanks!

same like delwin here,
My site has hosted with Netlify subdomain and I can't log into netlify-cms.
For your reproduce, below is site domain and repository.
Site: http://blog.kataba.me/
Repo: https://github.com/katabame/kataba.me/tree/blog

Thank you so much for the repro case @katabame! I think I've found the issue.
The problematic code is on https://api.netlify.com/auth/done, the page that shows the "Authorized" screen.
console.log("Got host: %o", host);
if ( host === "katabame.netlify.com" ||  host === "kataba.me" ||  host == "localhost") {
  window.opener.postMessage(
    'authorization:github:success:{\"token\":\"an-example-token\",\"provider\":\"github\"}',
    e.origin
  );
}

The problem is that you are logging in from a subdomain, blog.kataba.me, but the Netlify authorization code only works for the root domain, kataba.me.
@erquhart @bcomnes @biilmann Can this code be fixed, or do logins need to be restricted to top-level domains only?

@erquhart I assume that we'll still have a "real" fix at some point? Not everybody can or will use Git Gateway.

Not really a bug, auth only works against your custom domain. @delwin would need to change his Netlify site domain to "dcv.ampul.com", and @katabame would change his to "blog.kataba.me".
To request improvements in functionality, or to get other ideas about how to work around this, reach out in Netlify's community Gitter or submit the request to<EMAIL_ADDRESS>

I leave a note for @delwin or who in similar situation.
As https://github.com/netlify/netlify-cms/issues/770#issuecomment-377334439 say, I need to auth with git-gateway.
Here is detailed steps that I did.

Add <script src="https://identity-js.netlify.com/v1/netlify-identity-widget.js"></script> in your CMS index.html header.
See this for example.
Rewrite your CMS config.yml backend section.
You'll need specify branch if you're using subbranch for your site.
See this for example.
Add Git-Gateway Identity from your netlify dashboard.
I recommend change Registration preferences setting to Invite Only.
Invite yourself
You'll receive email from netlify. When you click Accept the invite link, you'll jump to YourTopDomain.com/#invite_token=TOKEN (for me, https://kataba.me/#invite_token=TOKEN).
But that link will not work, you need to rewrite&access link to Subdomain.YourTopDomain.com/#invite_token=TOKEN (for me, https://blog.kataba.me/#invite_token=TOKEN)
Then netlify identity will ask you for set your password.
Now you can login to netlify-cms dashboard.

@katabame Thanks for that blow-by-blow; it's really helpful!
Can I suggest linking point 3 to the Identity docs (here: https://www.netlify.com/docs/identity/ ) and/or including the current step-by-step instructions:
i) From the Netlify Dashboard, Select 'Identity' and "Enable Identity".
ii) (Recommended) In the settings page that then opens up scroll down to Registration and set "Invite Only".
iii) Scoll further down to Services and click "Enable GitGateway".
(If you could edit that into your post, I think it'll make this thread more useful for other devs - it's always annoying to trawl through a thread trying to piece together the necessary steps from multiple posts. :)

@Brondahl

(If you could edit that into your post, I think it'll make this thread more useful for other devs - it's always annoying to trawl through a thread trying to piece together the necessary steps from multiple posts. :)

Yes, it's so annoying. I edited the post, no annoying anymore!

@tech4him1 @Brondahl I think you're misunderstanding - Git Gateway and Netlify Identity aren't a temporary workaround, they're Netlify's answer for (among other things) authenticating against a Netlify site domain and it's subdomains. The GitHub/Bitbucket/GitLab backends on their own make use of a freebie auth server that we provide to everyone at cms.netlify.com, but if you want to authenticate for all of your branch subdomains on Netlify, use Netlify Identity.

@erquhart , hi!
Please, is there a way to login on my staging domain using BitBucket? Thank you!

@erquhart Will the "https://api.netlify.com/auth" server be deprecated then, and people be forced to us Git Gateway?

@Brondahl Netlify's own auth works only for a single domain, you'll need to create a separate Netlify site for your staging domain.
@tech4him1 no, not at all, just not as robust as Identity, that's all.

Like some other users, I couldn't login to netlify-cms on my website, because after trying to login with my github account, the login popup said Authorized but then nothing happened. I was trying to setup a Jekyll website using github pages and github backend (and netlify authentication server). In my case, just adding the property site_domain: <username>.github.io to backend field in config.yml did not solve the problem. I also had to set <username>.github.io as my site domain in netlify dashboard. That solved the problem for me and I could login using my github account.


Java chaincode implementation of ERC-20 token standard.

Java chaincode implementation of ERC-20 token standard. It follows the same logic of the existing golang and javascript chaincode implementation in the repo.

@renjithpta I think there is also a lingering error in that a client can reach a negative balance by BURNing a token quantity higher than the current available balance.
e.g. :
mint 5000 tokens for account1
transfer 100 tokens from account1 -> account2
BURN 5000 tokens.

At this point the token balance in the account is -100.   I think this is an error case and should be reflected as INSUFFICIENT_FUNDS

Hi @renjith
This looks great!! Thanks for the new addition!
A couple of really minor comments. And maybe one question...

The SPDX copyrights are missing from a few of the source files.
Please add a log / printf / output / ... line at the return from the CC contract methods.
Why does ClientAccountID return an encoded cert description from golang, and a certificate summary in the Java/Node contracts?
Move the Genson out of Transfer and Approval classes -> ERC20TokenContract class.  This can be in a marshal() routine in the contract class.


Recipe does not include arsenic

Solution to issue cannot be found in the documentation.

[X] I checked the documentation.

Issue
The conda recipe should include the OpenFF arsenic package in the run requirements since we use this in analysis of the benchmarks.
Installed packages
N/A

Environment info
N/A

Good catch! Opened a PR to fix this!


treeview, sorting by configure args

package require oowidgets
namespace eval ::test { }
oowidgets::widget ::test::Treeview {
constructor {path args} {
my install ttk::treeview $path
my configure {*}$args
}
}
#ttk::treeview .tree -columns "Row1  Row2" -displaycolumns "Row2 Row1"
test::treeview .tree -columns "Row1  Row2" -displaycolumns "Row2 Row1"
.tree heading Row1 -text "row1" -anchor center
.tree heading Row2 -text "row2" -anchor center
pack .tree
Error:
Invalid column index Row2
while executing
"$widget configure $opt $val "
...
oowidgets.tcl
oo::class create ::oowidgets::BaseWidget
method configure
in
foreach opt [array names opts] {
$widget configure $opt $val
executes : -displaycolumns "Row2 Row1"
then: -columns "Row1  Row2"

Thanks for the report. I recognize the problem. You are right, the configuration was done option  by option and in this case displaycolumns internally was configured before columns. I know configure at the end all at once after the loop:
$widget configure {*}$args
So the current version on Github should fix the problem, version is now 0.3.1 for oowidgets.
Thanks for the report.
Detlef


[Documentation] Could the main readme briefly explain the difference between scumm-8 and scummvm?

Could the main readme briefly explain the difference between scumm-8 and scummVM?
Please feel free to close this issue here at any moment in time.

Hi,
I thought I kinda did in the Read Me:

While it is heavily "inspired" by the SCUMM engine, it isn't a true replica (for that you'll want good ol' SCUMM-VM). However, SCUMM-8 attempts to stay as true as possible to the original SCUMM command reference.

...but I suppose you're right - I'm not explicitly stating the difference between the two projects. 🤔
I'll see if I can put a clear, short/one-liner statement covering the differences, as well.
Thanks for the suggestion


chore(ci): release main

:robot: I have created a release beep boop
tailwind: 0.10.0
0.10.0 (2024-09-27)
Features

add tabs for react (#228) (8f676ee)

html: 0.9.1
0.9.1 (2024-09-27)
Bug Fixes

html package publish not required (3e1b38c)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/ogcio/govie-ds/releases/tag/tailwind-v0.10.0 :sunflower:

:robot: Release is at https://github.com/ogcio/govie-ds/releases/tag/html-v0.9.1 :sunflower:


Please rename this artifact to conform to guidelines set out in Spring Boot user guide

Thank you for your interest in the Spring Boot, and for publishing this code which looks very useful. Please can you change the artifact id to align with the suggested guidelines in the spring boot user guide (http://docs.spring.io/spring-boot/docs/current-SNAPSHOT/reference/htmlsingle/#boot-features-custom-starter)?

Thank you for the comments and interest. I'll have the project renamed shortly.


fix(neo-tree): use last window for preview

Similar to "trouble.nvim" I want the preview to open in the last active window. Currently the neo-tree preview opens in a floating window, which (to me) doesn’t look very good.
What do you think about doing preview window the same everywhere (trouble & neo-tree)?
Before:
https://github.com/LazyVim/LazyVim/assets/3313023/7d7c21c9-158d-40ef-82f2-62e0dc795555
After:
https://github.com/LazyVim/LazyVim/assets/3313023/bafc5bae-20cb-4dd1-8e40-1358ada573d9

Makes sense. Thanks!


HTTP uTransport Implementation

This doesn't belong in this project, need to add it to the roadmap and then figure out where to go afterwards


Implement the equality delta writer

After #340 #341, we can implement the equality delta writer.
There is a rust implementation in icelake can be refer

Hi, @ZENOTME Since we have finished the framework, I'm thinking about providing a high level api such as append in python first rather providing these low level apis, what do you think?

Hi, @ZENOTME Since we have finished the framework, I'm thinking about providing a high level api such as append in python first rather providing these low level apis, what do you think?

Sure. I can work on it later and add the e2e test for the writer. But can this work be done in parallel with low-level writers, seems there are someone interested in them.

Please assign it to me, I will do it later : )

Please assign it to me, I will do it later : )

Sure. Thanks! But this issue was blocked by #341 #340 now.

Please assign it to me, I will do it later : )

Sure. Thanks! But this issue was blocked by #341 #340 now.

OK, I get it. I will proceed after #340 and #341 are completed.


Not webscale enough

Hi, I would love to use this module for my upcoming hip startup social network, but I'm afraid it'll be too slow. I need it to support async and possibly those "promises" things? I'm not sure yet if I'll need those too. Please fix asap.
Thanks

Dude, so late. See #1.


run laravel new blog is failed

For the install instruction, I run below command to create a new project.
$ cat /usr/local/bin/composer
#!/bin/sh
export PATH=/sbin:/bin:/usr/sbin:/usr/bin:/usr/local/sbin:/usr/local/bin
echo "Current working directory: '"$(pwd)"'"
docker run --rm -v $(pwd):/app -v ~/.ssh:/root/.ssh composer/composer $@

$ composer
Current working directory: '/Users/bill/composer'
   ______
  / ____/___  ____ ___  ____  ____  ________  _____
 / /   / __ \/ __ `__ \/ __ \/ __ \/ ___/ _ \/ ___/
/ /___/ /_/ / / / / / / /_/ / /_/ (__  )  __/ /
\____/\____/_/ /_/ /_/ .___/\____/____/\___/_/
                    /_/
Composer version 1.0.0 2016-04-05 13:27:25

problem 1
If I run composer global require "laravel/installer", the command runs well, but there is no new folders generated.
So I have to manually created a composer.json file, and this time I got all vendor folders.
$ cat composer.json
{
    "require": {
        "laravel/installer": ">=1.3.3"
    }
}

$ composer install

problem 2
I need create a new project, I run below command:
$ vendor/bin/laravel new blog
...
phpunit/phpunit suggests installing phpunit/php-invoker (~1.1)
Generating autoload files
Application ready! Build something amazing.

I found the APP Key is not automatically generated, if I run the command manually:
$ php artisan key:generate

Warning: require(/Users/bill/composer/blog/bootstrap/../vendor/autoload.php): failed to open stream: No such file or directory in /Users/bill/composer/blog/bootstrap/autoload.php on line 17

Fatal error: require(): Failed opening required '/Users/bill/composer/blog/bootstrap/../vendor/autoload.php' (include_path='.:') in /Users/bill/composer/blog/bootstrap/autoload.php on line 17

then I run with composer create-project --prefer-dist laravel/laravel blog, this time everything works fine.
$ composer create-project --prefer-dist laravel/laravel blog
...
phpunit/phpunit suggests installing phpunit/php-invoker (~1.1)
Writing lock file
Generating autoload files
> Illuminate\Foundation\ComposerScripts::postUpdate
> php artisan optimize
Generating optimized class loader

Nice job on the composer script. @renatomefidf brought up some good permissions tips in https://github.com/RobLoach/docker-composer/issues/35 . Would end up adding --user $(id -u) to the command.
What is APP_Key in this case?

@RobLoach
I follow the instruction in Laravel installation

Maybe I knew the reason for the first problem.
$ composer global config bin-dir --absolute
Current working directory: '/Users/bill/study/bitbucket.org/homestay/yii2'
Changed current directory to /composer
/composer/vendor/bin

With global option, the package were installed to container's folder /composer/vendor/bin, but our container only exposes /app folder. That's why I didn't see anything after I run composer global require "laravel/installer"

@RobLoach Seems you have added the global path into Dockerfile at
https://github.com/RobLoach/docker-composer/blob/585d9053c7a288385a861dd8cf79886710bf21f4/base/alpine/Dockerfile#L43
Can you explain how to make it work with composer global require "laravel/installer"

Since I can't build the base image directly (reported in #66), here is my work around to fix this issue.
$ cat Dockerfile
FROM composer/composer
WORKDIR /composer

$ cat /usr/local/bin/composer
#!/bin/sh

export PATH=/sbin:/bin:/usr/sbin:/usr/bin:/usr/local/sbin:/usr/local/bin
echo "Current working directory: '"$(pwd)"'"
docker run --rm -v $(pwd):/composer -v ~/.ssh:/root/.ssh composer $@

So I change the WORKDIR to /composer in Dockerfile and composer script.
Now I am fine to run composer with global option.

@SydOps Thanks for looking into this... Updated the installer hash at https://github.com/RobLoach/docker-composer/pull/67 , mind reviewing?


Added phoenix framework

Reasons for making this change:
Phoenix could have a similar gitignore to Elixir's file, but Phoneix needs some additional lines.
This template contains lines used in a Elixir's project and a Phoenix's project.
Links to documentation supporting these rule changes:
https://github.com/phoenixframework/phoenix/blob/master/.gitignore
https://hexdocs.pm/phoenix/Phoenix.html
If this is a new template:

Link to application or project’s homepage: https://phoenixframework.org/

@juanpabloaj apologies for the delay in replying to this.
I had a lot of these templates requested, but I couldn't quite figure out the right place for them. I ended up settling on #2857, and you'll see a Phoenix template can now be found at community/Elixir/Phoenix.gitignore.
Feel free to submit any future improvements to that template, and thanks again for the contribution!


Updated regex to allow + symbol and find more common email errors suc…

This pull request enhances the isEmail function to provide more robust and accurate email validation. The updated function now better adheres to email format standards, ensuring reliability and correctness in identifying valid email addresses.
Changes Made

Enhanced Validation: The function now disallows consecutive special characters and trailing dots in the local part of the email address, ensuring stricter compliance with common email standards. Function now allows + symbol in the local part for those that use plus email addressing.
Expanded Test Coverage: The test suite has been broadened to include a wider range of valid and invalid email scenarios, verifying the function's improved validation capabilities.

Phew finally fixed CI tests with Bun and ready to merge! Love the commit!


Epiphany browser does not recognize freshwrapper anymore

it was working a while on Epiphany but since I updated my system Fedora 23 64bits and git pull freshwrapper now Epiphany does not see any flash plugin

Tested in VM with F23 upgraded from F22 and current version of freshwrapper. Works fine mostly, but transparency doesn't work properly, plugin content is opaque.
Try to remove file ~/.cache/webkitgtk/plugin and start Epiphany again. It's possible that Epiphany decided that Flash plugin is not working, and cached that information.

this seems to make the trick.
also I installed another Fedora 23 on an another USB key
and everything is working well, even with Icecat browser.
the only issue I had with epiphany was SElinux that blocked the libfreshwrapper to read from libflashplayer so it needed to save another selinux rule


Add success/error feedback after function creation

There is not indicative that the function creation has been successful or not.
migmartri ~ $ kubeless function create miguel --runtime python27 \
>                               --handler test.foobar \
>                               --from-file test.py \
>                               --trigger-http

migmartri ~ $

It does not output on failure either:
 kubectl get lambda --all-namespaces
No resources found.

Something we can tackle very quickly. /assign myself

we should check with Lambdas or Google functions what the behavior is.
The no resources found is normal for kubectl if not resources exist.
I have a feeling this was a side effect of the controller GC not working properly which cause the functions to vanish in vacuum with no errors whatsoever.

I am taking this, we can return the controller logs in case an issue is found and return the control to the user just when the function is successfully deployed

I think the function ls call should return a status, like the Pod Status for that function.
We can also add the number of replicas in the deployment, so that we can compute the number of Pods that are READY for that function.
I would not want to block waiting for a successful deployment to become READY.

People won't know at the beginning that there is a function ls so if the command doesn't tell them anything it would be difficult for them to know if the deployment went fine or wrong (or what happened).
I just submitted a PR with the feedback I would expect but happy to change the approach if I am wrong.

So we need to check the CLI behavior on lambda and google cloud function. I think this is not a blocking event since the lambda deploy actually uploads to S3 then deploys...it could take several minutes the first time around, what happens if your network fails during that time ?

So we need to check the CLI behavior on lambda and google cloud function
I will check it

it could take several minutes the first time around, what happens if your network fails during that time ?
There is a timeout of 10 minutes (by default). If the network fails it will retry 3 times and then it will return an error saying that it is not possible to retrieve the deployment status.

So we need to check the CLI behavior on lambda and google cloud function
I will check it

So Lambda doesn't load the function until the it's triggered so there is no "deployment". It gives the error log in case the function fails.
Google Cloud Functions takes a similar approach than us. It deploys the function without giving feedback. The documentation tells you to manually check the status of the function afterwards: https://cloud.google.com/functions/docs/quickstart#deploy_a_function.

So I appreciate what you have done, I just don't think we should block on deploy. Otherwise the user could seat there waiting for the call to return for 10 minutes.
Instead I really think we should add a state to a function, using the state of the Pod.

Sounds reasonable to me as far as we tell the user to check that status.

Now the PR is pretty trivial. It is ready for landing.


update to reference STAC API spec v1.0.0

Resolves #16

There version number there is the version of the extension (which is 1.0.0-rc.1), not the version of the API.


#hgetall() calls both then() and catch() when key doesn't exists

redis.hgetall('does_not_exists_key')
  .then(photo => {
     console.error(photo);
  })
  .catch(err => {
    console.error(err);
  });

=> {}
=> Error

I tried on my computer, and only got a true. Could you please also log the value of err?

@luin Sorry, it's because I did assert.ok() inside then(). My fault.


Multiserver Content Manager Wrapper

It seems that in a multiserver setup only one server can run content manager wrapper, even when using different ports.

I'm not sure of the origin of this, but our own servers do exactly this without issue.


TestKeywordArguments#test_rest_keyrest - some kind of extra array being added, most likely splat related

Working on this at https://github.com/ruby/ruby/pull/9788


CollectCounts task ignores bam index parameter

If the bam index name can't be inferred from the bam name, CollectCounts fails with the following error:
A USER ERROR has occurred: Traversal by intervals was requested but some input files are not indexed.
I believe that we need to add the --read-index parameter to our invocation of GATK CollectReadCounts.
This issue may affect other tasks.

Closed via https://github.com/broadinstitute/gatk-sv/pull/54


Support OAuth2 Authentication

Documentation: https://github.com/AdobeDocs/analytics-2.0-apis/blob/master/oauth-curl.md

Came here to request this. My org unfortunately does not provide me access for JWT. Your project looks great and I would really like to try it. Thanks!

Hi @melanoma, had in mind for a long time to do this. Just never had the chance. will try to pick this up again! will keep you posted!

Hi @melanoma Have updated the appropriate branch to support OAuth authentication (https://github.com/konosp/adobe-analytics-reports-api-v2.0/tree/feature-oauth-authentication). Would you mind giving it a try?
To so you will need:

Install the appropriate branch from git by running pip install git+https://github.com/konosp/adobe-analytics-reports-api-v2.0.git@feature-oauth-authentication. This would install locally this particular branch. In case you have previously installed the version from PyPI you will need to uninstall it by pip uninstall analytics-mayhem-adobe
Create an OAuth integration in the adobe console. Follow the instructions at  https://github.com/AdobeDocs/analytics-2.0-apis/blob/master/create-oauth-client.md
Try the notebook created at https://github.com/konosp/adobe-analytics-reports-api-v2.0/blob/feature-oauth-authentication/OAuth Demo.ipynb

Let me know how it goes so I can push it into the live version!
Cheers

Hi @melanoma , any chance you looked into this? Eager to know if it works for you without issues. Cheers


Updated card number range for Discover

According to https://en.wikipedia.org/wiki/Bank_card_number, the range between 622126 and 622925 is under Discover but according to http://www.stevemorse.org/ssn/List_of_Bank_Identification_Numbers.html, such range is under China Union Pay. I actually have a Union Pay card that starts at 6225. After discussion with @DannyVancura , our thought is that there should be some cooperation between Discover and Union Pay and Union pay used to be treated as Discover in the US. Since we are supporting Union Pay, it would be better to treat them as Union Pay itself instead of Discover.

65 is also part of Discover.

:+1:

👍


Using variable fonts

https://github.com/flutter/flutter/issues/33709#issuecomment-730660020
https://wakamaifondue.com/beta/

https://api.flutter.dev/flutter/painting/TextStyle/fontVariations.html


Plotting

Add a .show() method.

n.b. there's only one commit in this PR to look at, 30484cf. I branched from #36 .

Interesting point. I'll look into this. Sometimes the rasterio ds does actually have data associated with it, most notably if using the classmethod from_array.

@rhugonnet have you looked at my earlier comment? I'm concerned that using rasterio's plotting function might load the data a second time, which is not ideal for large data set...

@rhugonnet have you looked at my earlier comment? I'm concerned that using rasterio's plotting function might load the data a second time, which is not ideal for large data set...

So should we plot using something else instead? Cartopy?

I think the idea for now is to have some plotting capability in order to rapidly check the function outputs and move forward. I can  open an issue on this and works towards a better solution?

I'm inclined to agree with @rhugonnet , we can change the code underneath the plot method later if/when we find there to be a problem.

Thanks Amaury, I have linked an issue from your comment to check the data loading of rio plotting.

Sounds good!


Complete stopped working after update to 2.9.3

Describe the bug
Updated the chrome extension diversion 2.9.3. After the update the ETV value would just be a spinning wheel. Unable to send anything to the vine discord. I tried disabling and reenabling the extension and that did not work. I didn't removed the extension restarted chrome and reinstalled the extension and restarted chrome. I no longer have anything that Devine helper extension offered before. It's just a standard plain screen.
To Reproduce
Steps to reproduce the behavior:
0. Using amazon.com (US)

Go to '...'
Click on '....'
Scroll down to '....'
See error

Expected behavior
The extension is installed, but doesn't show that it's installed
(optional) Screenshots
If applicable, add screenshots to help explain your problem.
A screenshot also help knowing what display settings you are using to reproduce your problem.
(optional) Error in the developer console
(Right click anywhere, Inspect. Then check the console tab for any red error related to Vine Helper)
(optional) Last page of Vine Helper debug log
If you have an issue where the tabs are no longer showing the item count:
With the keybinding activated, press 'd' to show the debug log of vine helper, scroll down to the last page and copy its content. This should tell us where the execution of the script ended if an error was encountered.
Browser
[e.g. chrome, firefox,  kiwi]
Additional context
Add any other context about the problem here.

Update it came back but does not show the ETV, it just spins

Are you also the author of UltraViner?

On Sep 30, 2024, at 10:26 AM, Francois Mazerolle @.***> wrote:
Yes, the release of the instant notification feature crashed the server. Currently working on restoring functionality. V2.9.5 (not 2.9.4) should work again (without instant notifications)
—
Reply to this email directly, view it on GitHub https://github.com/FMaz008/VineHelper/issues/132#issuecomment-2383366602, or unsubscribe https://github.com/notifications/unsubscribe-auth/ARBD7D6BUUG6ZYJ5F67P43DZZFNSPAVCNFSM6AAAAABPDJTN6WVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGOBTGM3DMNRQGI.
You are receiving this because you authored the thread.

Nop.

Should be fixed by now, if not, please re-open


Update yarte to 0.8

Update yarte example with latest features.
And fix simple-auth-server depend on<EMAIL_ADDRESS>incompatible with latest versions of clang

lgtm, but why did you disable form and simple auth examples?

thanks

lgtm, but why did you disable form and simple auth examples?

Form tests fail to create server or return 404.
And  simple-auth-server depend on<EMAIL_ADDRESS>incompatible with latest versions of clang


Replace custom slice code by Access.slice/1

Now that Elixir 1.14 has Access.slice, we can use it instead of the custom code.

I've taken this issue and created a pull request.
Please see my comments in the pull request, because this will be a breaking change.


Does it has color picker?

Is your feature request related to a problem? Please describe.
Does it has color picker?
Describe the solution you'd like
I wan't a color picker

http://vue-color-picker.rxshc.com/


Cannot read property 'render' of undefined

Hi,
I am trying to implement zingcharts in a simple create-react-app but it is giving me error locally but when i tried it on sandbox it works perfect.
https://codesandbox.io/s/create-react-app-x5bps
Uncaught TypeError: Cannot read property 'render' of undefined
    at t.value (zingchart-react.min.js?8408:69)
    at commitLifeCycles (react-dom.development.js?61bb:20049)
    at commitLayoutEffects (react-dom.development.js?61bb:22813)
    at HTMLUnknownElement.callCallback (react-dom.development.js?61bb:347)
    at Object.invokeGuardedCallbackDev (react-dom.development.js?61bb:397)
    at invokeGuardedCallback (react-dom.development.js?61bb:454)
    at commitRootImpl (react-dom.development.js?61bb:22585)
    at unstable_runWithPriority (scheduler.development.js?3069:643)
    at runWithPriority$2 (react-dom.development.js?61bb:11305)
    at commitRoot (react-dom.development.js?61bb:22414)

I have the following files
App.js
import React from 'react';
import Zingchart from './ZingChart';
import './App.css';

function App() {
  return (
    <div className="App">
      <header className="App-header">
        <Zingchart />
      </header>
    </div>
  );
}

export default App;

ZingChart.js
import React from 'react';
import { core as ZingChart } from 'zingchart-react';

var myDataSimple = {
 	type: "bar",
	series : [
        {values : [35,42,67,89,25,34,67,85]}
	]
};
class Zingchart extends React.Component{
    render(){
        return(
            <ZingChart id="chart1" height="300" width="600" data= {myDataSimple}/>
        )
    }
}
export default Zingchart;

package.json
{
  "name": "zingchart",
  "version": "0.1.0",
  "private": true,
  "dependencies": {
    "react": "^16.9.0",
    "react-dom": "^16.9.0",
    "react-scripts": "3.1.1",
    "zingchart": "^2.8.7",
    "zingchart-react": "^1.0.7"
  },
  "scripts": {
    "start": "react-scripts start",
    "build": "react-scripts build",
    "test": "react-scripts test",
    "eject": "react-scripts eject"
  },
  "eslintConfig": {
    "extends": "react-app"
  },
  "browserslist": {
    "production": [
      ">0.2%",
      "not dead",
      "not op_mini all"
    ],
    "development": [
      "last 1 chrome version",
      "last 1 firefox version",
      "last 1 safari version"
    ]
  }
}

Hi @richarepo, there was an issue with the main zingchart repository and how it exports out the library. That issue has been resolved in 1.0.8 of this package. Also note, we will be overhauling this wrapper in the coming weeks with more features and a different syntax.


v2.0.2 has a bug in log_request

When enabling logging, I got NoMethodError (undefined method > for nil:NilClass) in RestClient::Payload::Base#short_inspect.
https://github.com/rest-client/rest-client/blob/4ba9f7ee6374ec25abb870f769059b162ad7642a/lib/restclient/payload.rb#L88-L90
It seems that the issue has been fixed at v2.1.x

Yeah, this is a known issue with #size not always being implemented. It ought to be fixed in 2.1.0, which was just released.


Plex usage 100%

Plex uses 90% or more and JMMServer usually at 25% cpu when using the plex channel plugin for JMM the latest versions that are on the github.io page.

that is the plex transcoder. plex will auto transcode by default if you your codec isn't supported on the player device. meaning if you say play something on your phone for example it will transcode most of the time. nothing can be done with this unless you re-encode your videos in a codec that is supported by your client.
by default the plex transcoder can be a bit buggy at times, this isn't a JMM Plugin issue as far as i know. if you are not watching anything on plex and the transcoder is under load i would recommend to restart your plex server.

also for us to actually see whats going on you should share your logs with us, logs of the plex server aswell as the myanime plugin (the JMM plugin).
also on note, this issue is in the wrong place. it should be here: https://github.com/japanesemediamanager/plexplugin/issues

Eagle97 you can come on our Discord channel to discuss this :)

maybe share the link if you invite him, as he is already on the wrong project on this issue he might not be up-to-date on whats where just yet.
we are chatting here:
https://discord.gg/0XKJW7TObKJiXMIq
thats the support chat (not used just yet, but you can be the first! :D )
plz try to use the right text channels when on discord as we like to keep things neat.

Alright thanks I be sure to drop in.

But as I no longer use plex and just stick to jmmdesktop and direct streaming without transcoding. If someone wants to open this issue they can do so.


graphql:dump rake task is only available for Rails apps

The Rake task is currently loaded as a Railtie:
https://github.com/Shopify/shopify_api/blob/master/lib/shopify_api/graphql/railtie.rb
We should investigate if we can make this available for non-Rails apps (see #764).

@catlee I solved this by creating a Rakefile with the following in it:
require 'shopify_api'

ShopifyAPI::GraphQL.initialize_clients

load 'shopify_api/graphql/task.rake'

Given that you have both shopify_api and rake in the Gemfile for your script you just need to run (for a public app):
$ bundle exec rake shopify_api:graphql:dump SHOP_DOMAIN="SHOP_NAME.myshopify.com" ACCESS_TOKEN="SHOP_TOKEN" API_VERSION=2020-01

Hope this helps! :)


Added streamlit import in utils and bubbled up outcome of add_embeddings

This PR fixes 3 problems I discovered while testing the Q&A.

A non handled exception in utilities.utils.add_embeddings where st (streamlit) was referenced but it was not imported in this module. (Potentially the utils module should be independent from streamlit and have a way to bubble up the error /error message, however this will require a more involved refactoring which is best if left to you to consider)
If utilities.utils.add_embeddings failed and is not able to add the embeddings to Redis db, the code calling add_embeddings was not aware and still recorded in the blob metadata that embeddings were successfully added (embeddings_added set to true).  I have changed the add_embeddings function so that returns a boolean indicating whether the operation was successful. This allows the caller function to act accordingly.
BatchStartProcessing function was set its authLevel to "anonymous", changed to "function"

@gromag The function has to stay anonymous for local docker execution. It's properly secured when deployed to Azure.
https://learn.microsoft.com/en-us/azure/azure-functions/functions-create-function-linux-custom-image?tabs=in-process%2Cbash%2Cazure-cli&pivots=programming-language-python#create-and-test-the-local-functions-project

@ruoccofabrizio thanks for getting back. Are you sure it is secured when deployed to Azure? I have deployed it to Azure using the deployment.json ARM template included in the project and the function remain anonymous. I can see the clientKey in the function keys being created by the ARM and the ARM appends the clientKey to CONVERT_ADD_EMBEDDINGS_URL  but the function itself remains anonymous and can be triggered without any authentication. Can you double check at your end.

Thanks for your contribution! The repo has gone through substantial refactoring given the fast pace environment around LLM. This feature seems to be implemented in that macro update. If this is not the case, please submit a new PR. Thanks again!


Unable to send command! Node source

org.redisson.client.RedisTimeoutException: Unable to send command! Node source: NodeSource [slot=null, addr=null, redisClient=null, redirect=null, entry=MasterSlaveEntry [masterEntry=[freeSubscribeConnectionsAmount=1, freeSubscribeConnectionsCounter=50, freeConnectionsAmount=31, freeConnectionsCounter=63, freezed=false, freezeReason=null, client=[addr=redis://<IP_ADDRESS>:6379], nodeType=MASTER, firstFail=0]]], connection: RedisConnection@1152922437 [redisClient=[addr=redis://<IP_ADDRESS>:6379], channel=[id: 0x76d81725, L:/<IP_ADDRESS>:53758 - R:<IP_ADDRESS>/<IP_ADDRESS>:6379]], command: (EVAL), command params: [if (redis.call('exists', KEYS[1]) == 0) then redis.call('hset', KEYS[1], ARGV[2], 1); redis.call('pe..., 1, InventorySycLock--1001-YY101131449719808386739, 1800000, 1ab08d95-f14d-4d90-b8bf-7f6b0533bf07:856] after 3 retry attempts
at org.redisson.command.CommandAsyncService$10.run(CommandAsyncService.java:715)
at io.netty.util.HashedWheelTimer$HashedWheelTimeout.expire(HashedWheelTimer.java:672)
at io.netty.util.HashedWheelTimer$HashedWheelBucket.expireTimeouts(HashedWheelTimer.java:747)
at io.netty.util.HashedWheelTimer$Worker.run(HashedWheelTimer.java:472)
at java.lang.Thread.run(Thread.java:748)
redisson version: 3.9.1
singleServerConfig:
address: xxx
database: xxx
password: xxx
pingConnectionInterval: 30000
other conifg is default;

I want to know if this is a bug or if there are some configuration errors?

https://github.com/redisson/redisson/wiki/16.-FAQ#q-what-is-the-cause-of-redistimeoutexception

I don't know how to solve it，Can you give me some advice

Try to set nettyThreads=32

I need to update the version of redission?

I would update. Since 3.9.1 is 5 years old.

Which version is currently the most stable and recommended for use.

@ykrenz
You can find it in release page  https://github.com/redisson/redisson/releases/


Why does comments.txt have a date?

I can't seem to figure out where the comments.txt date field is used. Why do we need this variable?

As described in README#file-structure, KirbyComments does not ship with a template for comments.txt files. Because of that those pages are not styles nor reachable by clicking internal links. However, developers are free to make comments.txt a publicly accessible page.
For that reason, a creation date is stored alongside the comments page. It does not wast a lot of space and is useful for creating a pages dedicated to listing all comments of other pages. Personally, I don’t have strong feeling on whether to keep it or not.
To conclude:

the date field is not used by the plugin
we do not need it, it’s simply a convenience access to the creation date of the first comment
it is redundant information and wastes space, but the impact is (I think) negligible
I am happy to remove it, if any reasons for doing so are brought forth

I have no strong opinions one way or the other. I'm just doing some debugging (imported a lot of comments from Disqus) and wanted to make sure to rule out this date field as a potential cause of bugs.


Fix typo in alerting.mdx

Description of the change
Small typo in documentation
Type of change

[X] Bug fix (non-breaking change that fixes an issue)
[ ] New feature (non-breaking change that adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)

Related issues

Fix #1

Checklists
Development

[ ] Lint rules pass locally
[ ] The code changed/added as part of this pull request has been covered with tests
[ ] All tests related to the changed code pass in development

Code review

[ ]  This pull request has a descriptive title and information useful to a reviewer. There may be a screenshot or screencast attached
[ ] "Ready for review" label attached to the PR and reviewers mentioned in a comment
[ ] Changes have been reviewed by at least one other engineer
[ ] Issue from task tracker has a link to this pull request

Thanks for your contribution to the docs, @gassiss :) Before we push this change, request you to please sign the CLA here: https://rudderlabs.wufoo.com/forms/rudderlabs-contributor-license-agreement. Thanks!

Thanks for your contribution to the docs, @gassiss :) Before we push this change, request you to please sign the CLA here: https://rudderlabs.wufoo.com/forms/rudderlabs-contributor-license-agreement. Thanks!

Done :)


fixing documentation contained in the element commands js files

Proposed changes
Updated the inline documentation for all element command js files.

removed references to selector being a param
updated examples to use browser.$() for locating an element before interacting with it

Types of changes
Documentation update.

[ ] Bugfix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)

Checklist

[x ] I have read the CONTRIBUTING doc
[ x] I have added tests that prove my fix is effective or that my feature works
[ x] I have added necessary documentation (if appropriate)

Further comments
Reviewers: @webdriverio/technical-committee

Codecov Report

Merging #38 into master will not change coverage.
The diff coverage is n/a.

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

@christian-bromann I think I got all the updates


0.10: ReST API http method change

We are trying out the 0.10 branch and found that most of the original GET methods in 0.9 for modifications have changed to POST. I understand that theoretically a ReST API is supposed to use the right http method for each type of action, but we already have a lot of client code that uses the GET only API in the current master.
Are we intending to maintain backward compatibility in 0.10 and keep the GET version of the API?

The change to POST only for APIs that mutate state was intentional, previously having APIs like stop/restart/delete/etc exposed via GET was just a recipe for disaster.  Unfortunately I believe that this is one place where breaking backward compatibility is worth the pain.
I will note the change in the 0.10 release notes.  I recommend updating your tooling to use POST for everything, which will work with both 0.9 and 0.10 versions of the scheduler.

@steveniemitz that's fair enough, so for the mutation methods, are we gonna have PUT/DELETE as well (PUT for update and DELETE for remove), or is it POST only?

I think in general the API needs a rethink.  What I'd really like to see is a protobuf described API with a REST interface.
However, to answer your question, I don't have any plans to modify the existing interface any more (changing methods to PUT/DELETE for example).


Update to latest Font Awesome version

There are new icons in the latest version (now 5.12.0)
Thanks!

I've updated to FontAwesome 5.13.0, but I cannot push my PR to this repository.
I got this error when publish my PR.
remote: Permission to devgg/fonticon.git denied to fysidm.
fatal: unable to access 'https://github.com/devgg/fonticon.git/': The requested URL returned error: 403

Never mind.
It was my first PR. lol
#39


how to customize 404 page layout

I declared global layout as common,but i want to use another layout for 404 page.
Router.configure({
    notFoundTemplate: '404',
    layoutTemplate: 'common',
});
Router.route('/', function () {
    this.render('home');
});

i do not want to use routerController

@lifenglsf call for Router.go('your404page') inside Template.your404page.onCreated callback and create separate route for it with different layout template.
Use this suggestion unless this will be fixed in IR


Use Stackrox builder to build Scanner for arm64 / apple silicon

This is an alternative to fix Scanner builds (make image) errors on Apple silicon Macs (see also https://github.com/stackrox/scanner/pull/1038), that uses a native arm64 builder image.
The current builder image has Go v17 and cannot build the current source.
This PR changes the builder image to use the current Stackrox builder, which has Go 18.4. It should be acceptable to use this image (the main difference between the Scanner and the Stackrox builder images is that the Stackrox one also contains the RocksDB binaries and its dependencies).

Thanks for looking into #1038 @vladbologa!
A few thoughts for consideration:
quay.io/rhacs-org is not accessible to outside contributors, relying instead on the quay.io/stackrox builder image(s) + emulation may be more inviting until multi-arch build images are made avail
I believe the IMAGE_BUILD_FLAGS env bug fix (from #1038) should be made regardless to avoid future confusion / debt - even though it will not have a noticable impact when using the arm64 builder image
Leveraging the amd64 build image now may prevent having to make future changes if/when the scanner image diverges or becomes multi-arch
If you still prefer to use the proposed stackrox BUILD_IMAGE, I'll update #1038 accordingly - either way the local builds for apple silicon are fixed which works for me! 😉

You're right, if it works properly, I guess the only advantage of the arm64 build is speed. I should've have just commented on your PR instead of making a new one anyway, so I'll just close this one.


Convert binary exceptions into YSON text

Starting issue: https://github.com/ytsaurus/ytsaurus-ui/issues/405
Issue:
CHYT converts clickhouse-thrown std::exception into YT TError directly, in binary format (using exception.what())
Then it is stored in dynamic table in binary format:
"inner_errors" = [
        {
            "attributes" = {
                "datetime" = "2024-03-08T17:27:47.033681Z";
                "fid" = 0u;
                "host" = "our-host.net";
                "pid" = 3673;
                "thread" = "ThreadPool";
                "tid" =<PHONE_NUMBER>553949491u
            };
            "code" = 1;
            "message" = 53 79 6e 74 61 78 20 65 72 72 6f 72 3a 20 66 61 69 6c 65 64 20 61 74 20 70 6f 73 69 74 69 6f 6e 20 31 20 28 27 d0 27 29 3a 20 d0 be d1 88 d0 b8 d0 b1 d0 ba d0 b0 31 2e 20 55 6e 72 65 63 6f 67 6e 69 7a 65 64 20 74 6f 6b 65 6e 3a 20 27 d0 27
        }

This can't be decoded into UTF8 json on server side, resulting in failure in get_query(id, format='<encode_utf8=%false>json'.
Table reader (read_query_result) uses web_json format and somehow fixes it. web_json is not available for get_query
Solution:
In this PR I make 2 changes:

Convert clickhouse's exception.what() into text YSON string
Convert all TErrors before writing to the table into text YSON string
The format says it escapes non-text characters

Feel free to propose a different solution or leave only one of two changes
I wasn't able to write tests for it. Please help me find a correct place for a new test

After some discussion we decided that binary exceptions are okay and UI should learn how to handle them.
UI can handle binary data on Operation page, since it does get_operation format=json and decodes output to correct UTF8 or shows it in binary
UI can handle binary data from tables by using web_json format. web_json isn't available for non-streaming APIs so it is not a solution
A suggested solution for UI is to never use <encode_utf8=false>json since it fails on non-utf8 data. UI should use other format that is okay with binary data and perform additional decoding


Bundle and compile test helper functions

I wanted to use the helper functions in zeppelin-solidity/test/helpers/ in my project, but had a hard time importing them. They’re exported using ES2015 module syntax, so I had to add babel-register as a dependency and run:
require('babel-register')({
    ignore: /node_modules\/(?!zeppelin-solidity)/
});

It would be easier to add as a dependency in other projects if they were bundled and transformed to CommonJS

@maraoz I created https://github.com/levelkdev/zeppelin-testing. It has all the js from test/helpers, exports each named function, and bundles them into a dist file. Haven't tested it yet, but could be a good starting point if you want to add to OpenZeppelin

@mikec awesome! we'll review it and maybe we can add it to the OpenZeppelin org :)

Went through the exact same process a few days ago, great work!

@martriay can you take over the task of moving our existing test helpers to a separate module, and make it part of the OpenZeppelin org? That'd be helpful for other projects too.
@frangio thoughts?

@mikec has already done some of the work here. We should definitely bundle the test helpers for the community.

Many thanks to @mikec for a babel configuration excerpt as everywhere advised require('babel-register'); does not work and I had very hard time making it work. Still does not know why it does not work (I guess default ignore?) without ({ ignore: ... }) but at least it works!

@mikec Well done.

Would be nice to get these helper functions compiled down so babel is no longer needed. I'm using Typescript so babel is no longer needed, would have to include it just for this purpose.


Minor fixes in the upstream doc - fusedoc 2397

Corrected typos and made minor changes to the text

ok to test

Thanks!


Testing not picking up custom variable in testing.conf

Setting custom_attributes = "slurm.constraint": "knl" in testing.conf but job is submitted to different partition (bdwall).

Can you send me or post the testing.log file?

           2666432    bdwall 646d566b    wilke PD       0:00      1 (Priority)

in testing.conf
#
# If a site needs specific job attributes to be specified, such as billing
# accounts, they can be added here. The value is a comma separated list
# of "executor.name": "value" pairs.
#
# custom_attributes = "slurm.account": "xyz", \
#                     "slurm.constraint": "knl"

custom_attributes = "slurm.constraint": "knl"

What's the output of psij-ci-run?

tests/test_callbacks.py::TestCallbacks::test_job_callbacks PASSED                                                         [  0%]
tests/test_callbacks.py::TestCallbacks::test_job_executor_callbacks PASSED                                                [  1%]
tests/test_doc_examples.py::test_submit_and_wait_for_n_jobs PASSED                                                        [  2%]
tests/test_doc_examples.py::test_run_n_throttle_m PASSED                                                                  [  2%]
tests/test_doc_examples.py::test_job_parameters PASSED                                                                    [  3%]
tests/test_executor.py::test_simple_job[local:single] PASSED                                                              [  4%]
tests/test_executor.py::test_simple_job[local:multiple] PASSED                                                            [  5%]
tests/test_executor.py::test_simple_job[batch-test:single] PASSED                                                         [  5%]
tests/test_executor.py::test_simple_job[batch-test:multiple] PASSED                                                       [  6%]
tests/test_executor.py::test_simple_job[batch-test:batch-test] PASSED                                                     [  7%]
tests/test_executor.py::test_simple_job[local:mpirun] PASSED                                                              [  8%]
tests/test_executor.py::test_simple_job[batch-test:mpirun] PASSED                                                         [  8%]
tests/test_executor.py::test_simple_job[slurm:single]

In ~/.psij/work/slurm, there should be a bunch of .job files. While the job is still waiting in the queue, can you find the latest .job file and post it here?

#!/bin/bash

#SBATCH --export=ALL

#SBATCH --time=0:10:00

#SBATCH --constraint="knl"

#SBATCH -e /dev/null
#SBATCH -o /dev/null

exec &>> "/home/wilke/.psij/work/slurm/$SLURM_JOB_ID.out"

/bin/bash /home/wilke/.psij/work/f65b111560da52a6d41f52a732be0faf4914eb6d7af2230a8de326017979e684/single_launch.sh e5869c75-bdfb-448f-b1ee-a911526c6326 /home/wilke/.psij/work/slurm/e5869c75-bdfb-448f-b1ee-a911526c6326_launcher.log '' '' /dev/null /dev/null /dev/null /bin/date

echo "$?" > "/home/wilke/.psij/work/slurm/$SLURM_JOB_ID.ec"

It looks to me like the tests are picking up the custom variable in testing.conf just fine, but perhaps that variable does not do what you were expecting it to do?

Could be a user error, removing the constrains entry and replacing it with partition and correct partition name seems to work. Closing for now.
             JOBID PARTITION     NAME     USER ST       TIME  NODES NODELIST(REASON)
           2732663    knlall 725575df    wilke PD       0:00      1 (Priority)
           2732666    knlall a82c39a0    wilke PD       0:00      1 (Priority)


[AIRFLOW-1881] Make operator log in task log

Make sure you have checked all steps below.
JIRA

[X] My PR addresses the following Airflow JIRA issues and references them in the PR title. For example, "[AIRFLOW-XXX] My Airflow PR"

https://issues.apache.org/jira/browse/AIRFLOW-1881

Description

[X] Here are some details about my PR, including screenshots of any UI changes:
Previously operators logged under airflow.operators or
airflow.contrib.operators. This unifies them under
airflow.task.operators allowing the task log to pick
them up and not have 'double' logging.

Tests

[X] My PR adds the following unit tests OR does not need testing for this extremely good reason:
Covered by existing tests

Commits

[X] My commits all reference JIRA issues in their subject lines, and I have squashed multiple commits if they address the same issue. In addition, my commits follow the guidelines from "How to write a good git commit message":

Subject is separated from body by a blank line
Subject is limited to 50 characters
Subject does not end with a period
Subject uses the imperative mood ("add", not "adding")
Body wraps at 72 characters
Body explains "what" and "why", not "how"

[X] Passes git diff upstream/master -u -- "*.py" | flake8 --diff

cc: @Fokko @jgao54 @ashb

Codecov Report

Merging #2838 into master will decrease coverage by 0.07%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master    #2838      +/-   ##
==========================================
- Coverage   74.01%   73.93%   -0.08%
==========================================
  Files         159      159
  Lines       12076    12086      +10
==========================================
- Hits         8938     8936       -2
- Misses       3138     3150      +12

Impacted Files
Coverage Δ

airflow/models.py
87.37% <100%> (+0.01%)
:arrow_up:

airflow/task_runner/bash_task_runner.py
93.33% <0%> (-6.67%)
:arrow_down:

airflow/utils/helpers.py
53.44% <0%> (-2.88%)
:arrow_down:

airflow/jobs.py
79.53% <0%> (-0.45%)
:arrow_down:

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 406d738...ec7e032. Read the comment docs.

As _log is set in init it would change the signature and the tests would catch it richt away. So would it be really required?

sounds good to me


Azure automation toolset?

In the docs there is use of both PowerShell modules and Azure CLI commands.
There are times when we have to choose one tool because it offers features or command surface area that the other does not it appears to me that everything that is being done in script at the moment could be done purely in Azure CLI. Possibly in using PowerShell modules, but TBH I'm not that well versed in the Azure AD PowerShell modules.
Is there a preferred toolset for this project?
Should the project care?
(Please forgive the flurry of issues, I think that this stuff is super cool and I'm looking to offer my help)

you really excel at scripts AND open-source and its a pleasure to have you as a contributor. Let me know your T-shirt size 💙


Limit ini/mdu parsing to most common elements

and ignore multilines or weird comments? See documentation by @priscavdsluis

Please do support these syntax elements:

Comment character # (and don't interpret ; as comment character).
Multiple [sections] with same name allowed, order should be preserved.
Order of sections should be preserved in general.
Keys may have an empty righthandside value (name =    # comment).
Key values may contain any character except for comment character (in particular: also ; is a valid character).
Key values in scientific notation should allow D (not only E) as the exponent character (UnifFrictCoef = 2.3D-2). (Case insensitive, d and e also fine.)

We do not directly need to support these rarely used syntax elements:

Multiline values using line continuation with \ character
Quoting of (String) key values using enclosing #, e.g.:
key = #some value with spaces# # Comment: Not needed for now

Specific for MDU, not all ini files in general:

A String key value containing semicolons represents a list of values, typically multiple file names. For recursive parsing it is good to split such kind of values and maintain them as a list.
Example:

[geometry]
FrictFile         = roughness-Main.ini;roughness-Sewer1.ini;roughness-Sewer2.ini

With regards to the scientific notation with the D as in UnifFrictCoef = 2.3D-2, we probably could use a validator in Pydantic to quickly rewrite any Ds in to Es. See https://pydantic-docs.helpmanual.io/usage/validators/


[IMPROVEMENT] JSON rewrite

Please prefix your pull request with one of the following: [FEATURE] [FIX] [IMPROVEMENT].
In raising this pull request, I confirm the following (please check boxes):

[x] I have read and understood the contributors guide.
[x] I have checked that another pull request for this purpose does not exist.
[x] I have considered, and confirmed that this submission will be valuable to others.
[x] I accept that this submission may not be used, and the pull request closed at the will of the maintainer.
[x] I give this submission freely, and claim no ownership to its content.

My familiarity with the project is as follows (check one):

[x] I have never used the project.
[ ] I have used the project briefly.
[ ] I have used the project extensively, but have not contributed previously.
[ ] I am an active contributor to the project.

This removes the beautifulsoup4 dependency from the bot and uses JSON and regex to parse the lyrics from the html.
It's speed improvements are:

HTML Type
This PR
Current Method
Improvement

Legacy UI
0.014 s
0.043 s
~3x

React UI
0.0031 s
0.042 s
~13x

THIS PR IS NOT READY FOR MERGE
I have not tested it directly since I don't have Spotify. Please ensure it is working. Also, I didn't test with many songs.
NOTE: Remove rewrite.py once it is ready for merge.

Hello @FadedCoder! Thanks for opening this PR.

In the file rewrite.py:

Line 82:1: E402 module level import not at top of file

Do see the Hitchhiker's guide to code style

@aadibajpai fix the .editorconfig, it has end_of_line=crlf.

@aadibajpai fix the .editorconfig, it has end_of_line=crlf.

wontfix

I've thought about this a lot, and I feel we shouldn't go for speed at the cost of maintainability since we're not actively developing anymore so any overhead would just come back and bite us :(
Honestly thanks so much for coming up with a significant speed improvement tho!


Add explicit CRD configuration for MetalLB

Fixes #385
Fixes #387

Hi, please see - https://github.com/equinix/cloud-provider-equinix-metal/issues/385#issuecomment-1457830600.
The configmap/namespace parser is not working correctly. In case there is only namespace defined (METAL_LOAD_BALANCER: "metallb:///kube-system" as per docs) the namespace variable will be set to default value (metallb-system).
This needs to be fixed as well.

So metallb is throwing this warning:
metallb.io v1beta1 AddressPool is deprecated, consider using IPAddressPool

So metallb is throwing this warning:
metallb.io v1beta1 AddressPool is deprecated, consider using IPAddressPool

This may be a false positive, I've checked and it seems we're using IPAddressPool.


Add to_ordinal utility

This PR will update the RELEASE.md for recently added feature tf.keras.utils.to_ordinal on Keras (keras-team/keras/pull/17419).

@haifeng-jin  Thanks for letting me know. Any idea when to_ordinal will be added for Release 2.13.0 ?

@haifeng-jin my fork was out of sync, I fixed it just now. Can you please approve the workflow?


Warn about manufacturer firmware updates

Adds the following piece of advice (feel free to improve the wording):

It is known that the manufacturer's firmware updates can remove the possibility to successfully flash the devices with tuya-convert. Hence, it is best to disable all Internet connectivity (e.g., disconnect the DSL or cable line from your router) when powering the device until the alternative firmware has successfully been flashed.

Example: https://github.com/ct-Open-Source/tuya-convert/issues/197

This. More and more reports of people not being able to flash their devices after firmware upgrades are coming in. It should be a priority to warn those that are going to buy devices in the future or aren't affected just yet.

Hi @probonopd thanks for the contribution!
I like the idea, but I feel the language is a bit paranoid. It suggests that the device could inadvertently connect to your home network and register to the cloud, bypassing our sandbox, through normal use of tuya-convert, which I don't believe is a legitimate risk.
The pairing procedure we use would make any attempt to register with the authentic cloud fail, as we're assigning our own token. Disconnecting your router shouldn't be necessary. In fact this would prevent users from installing firmware specified by /flash3?url=... that are not hosted on the local network.
The idea we should be conveying to users is that a new, incompatible, firmware could be shipped with your device, or installed to your device after connecting with the vendor supplied application. If your device was shipped with the new firmware, obviously nothing can be done (yet!). To mitigate the latter case, one should avoid adding the device to the vendor supplied app, as this would allow the authentic cloud to force an update.

Check that the rewritten passage reflects your intentions and let me know if you are satisfied. I'll merge on your approval.

Yes, sure, please go ahead.


Initial implementation for simple scheduler

Scheduler assumes that input is ModuleOp which contains single FuncOp
Scheduler constructs dependency graph using TTNN ops (we can change IR or template it away) which implement DPS interface.
This excludes ops like ttnn.open_device, ttnn.close_device, we need to find some way how to handle them i.e either scheduler puts them at the start/end of schedule or we handle them in some different way.
Snapshotting creates deep copy of scheduler members
Tested by creating pass which printed dependency graph of some more complex graph that was handcrafted (didn't include pass nor test code in PR but I can do if needed).

closes #571

Nice. And just at the right moment :).

Is this related to 6AM moment?


fix: Icon alignment changes automatically every time the icon is changed in Menu button widget

Description
Every time we click on an icon and change the existing icon, the alignment of the icon changes automatically to default one
Fixes #11482
Type of change

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

How Has This Been Tested?

Test A
Test B

Checklist:

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes

Test coverage results :test_tube:
:white_circle: Total coverage has not changed
// Code coverage diff between base branch:release and head branch: fix/11482-menu-button-icon-alignment-reset

Status
File
% Stmts
% Branch
% Funcs
% Lines

:green_circle:
app/client/src/utils/WorkerUtil.ts
89.76 (0.78)
72.55 (1.96)
100 (0)
93.33 (0.95)

/ok-to-test sha=36af281

/ok-to-test sha=7885e83

@wmdev0808 tested this PR. DP is working fine.
Moving to Done.

/ok-to-test sha=1594a93

/ok-to-test sha=b23137b


I can't setup it successfully!

when I use the official document to start my mmdet, I got the error
"Traceback (most recent call last):
File "/usr/local/bin/mim", line 8, in
sys.exit(cli())
File "/usr/local/lib/python3.8/dist-packages/click/core.py", line 1157, in call
return self.main(*args, **kwargs)
File "/usr/local/lib/python3.8/dist-packages/click/core.py", line 1078, in main
rv = self.invoke(ctx)
File "/usr/local/lib/python3.8/dist-packages/click/core.py", line 1688, in invoke
return _process_result(sub_ctx.command.invoke(sub_ctx))
File "/usr/local/lib/python3.8/dist-packages/click/core.py", line 1434, in invoke
return ctx.invoke(self.callback, **ctx.params)
File "/usr/local/lib/python3.8/dist-packages/click/core.py", line 783, in invoke
return __callback(*args, **kwargs)
File "/usr/local/lib/python3.8/dist-packages/mim/commands/download.py", line 70, in cli
download(package, configs, dest_root, check_certificate, dataset)
File "/usr/local/lib/python3.8/dist-packages/mim/commands/download.py", line 107, in download
return _download_configs(package, configs, dest_root,
File "/usr/local/lib/python3.8/dist-packages/mim/commands/download.py", line 127, in _download_configs
raise RuntimeError(
RuntimeError: mmdet is not installed. Please install it first."
I promise I follow  your document!

By the way, when I use 'pip list', it didn't display 'mmdet'

I have the same problem


Development: Remove disabled annotations from server tests

Checklist
General

[x] Language: I followed the guidelines for inclusive, diversity-sensitive, and appreciative language.
[x] I chose a title conforming to the naming conventions for pull requests.

Server

[x] I followed the coding and design guidelines.
[x] I documented the Java code using JavaDoc style.

Motivation and Context

There are currently 6 tests that are labled with @Disabled and are not run due to different TODOs. We want to resolve this TODOs and in some cases remove the tests.
Description

In this PR @Disabled is removed by resolving the TODOs  from:

ProgrammingExerciseBitbucketBambooIntegrationTest.testAutomaticCleanUpBuildPlans()
EmailSummaryServiceTest.testIfPrepareWeeklyEmailSummariesCorrectlySelectsExercisesAndCreatesEmail()

The following tests are about functionalities that should be implemented in the futue:

ResultTest.testRemoveTestCaseNames()
ResultTest.keepTestNamesWhenExerciseSettingActive()
ProgrammingExerciseResultBambooIntegrationTest.shouldRemoveTestCaseNamesFromWebsocketNotification()
ProgrammingExerciseResultJenkinsIntegrationTest.shouldRemoveTestCaseNamesFromWebsocketNotification()

The tests are not very complicated and can easily be written again once the functionality is implemented. ResultTest.keepTestNamesWhenExerciseSettingActive() can still be kept since it passes per default.
Review Progress

Code Review

[ ] Code Review 1
[ ] Code Review 2

Test Coverage

unchanged

@Strohgelaender these are (partially) your tests