text:
Error in dsPlot with 'plot_type' set to power or raster

Running the following code produces an error
  eqns='dv/dt=@current+10; {iNa,iK}';
  data = dsSimulate(eqns);

Produce power plot
dsPlot(data,'plot_type','power')
@jsherfey  Any idea what's going on here? Normally I would say not to bother with it since we will switch to dsPlot2, but this is a simple use case (and is also one of the use cases in dsDemos) so it might make sense to address it.

Update: this also affects rastergram plots:
  eqns='dv/dt=@current+10; {iNa,iK}';
  data = dsSimulate(eqns);
dsPlot(data,'plot_type','raster')

i'm not able to reproduce this error on my local computer. is it still happening? what system are you using?

That works for me in MATLAB 2014a and 2017a. What is the error message? What version of MATLAB are you using?

@davestanley

Looking into this

I am running MATLAB 2016b on macOS Sierra.
The error is:
Error using line
Vectors must be the same length.

Error in dsPlot (line 996)
              line(thisAxes, [vlines(k) vlines(k)],ylim,'color','k','linestyle','--');

I suspect it is because the population in the example is of size 1

I just tested this on the cluster (MATLAB 2017a) and got the same error. I should also point out that it does produce a half-finished power plot before the error appears.
I am running this in the DynaSim dev branch
Here's the code I just ran
eqns='dv/dt=@current+10; {iNa,iK}';
data = dsSimulate(eqns);
% Produce power plot
dsPlot(data,'plot_type','power')

Do you get the same error on the master or debug branch?
On Wed, Mar 21, 2018 at 2:21 AM, Dave notifications@github.com wrote:

I just tested this on the cluster (MATLAB 2017a) and got the same error. I
should also point out that it does produce a half-finished power plot
before the error appears.
I am running this in the DynaSim dev branch
Here's the code I just ran
eqns='dv/dt=@current+10; {iNa,iK}';
data = dsSimulate(eqns);
% Produce power plot
dsPlot(data,'plot_type','power')
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/DynaSim/DynaSim/issues/398#issuecomment-374843061,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AQFleQ92Rug-n_cdlgW5qy8h_barE1Enks5tgfFTgaJpZM4SI5xX
.

On debug: It runs without error and produces the correct plot.
On master: a different error appears, associated with dsSimulate
>> eqns='dv/dt=@current+10; {iNa,iK}';
data = dsSimulate(eqns);
Error: Undefined function 'plus' for input arguments of type 'function_handle'.
	 in solve_ode_20180321140736_473 (line 28)
	 in dsSimulate/tryFn (line 997)
	 in dsSimulate (line 746)
Undefined operator '+' for input arguments of type 'function_handle'.

Error in solve_ode_20180321140736_473 (line 28)
  pop1_v_k1=@current+10;

Error in dsSimulate/tryFn (line 997)
            [outputs{1:length(output_variables)}]=feval(fname);

Error in dsSimulate (line 746)
    tryFn(nargout)

The offending line in the solve file is: pop1_v_k1=@current+10;
Full solve file below
function [T,pop1_v]=solve_ode
% ------------------------------------------------------------
% Parameters:
% ------------------------------------------------------------
params = load('params.mat','p');
p = params.p;
downsample_factor=p.downsample_factor;
dt=p.dt;
T=(p.tspan(1):dt:p.tspan(2))';
ntime=length(T);
nsamp=length(1:downsample_factor:ntime);
% ------------------------------------------------------------
% Initial conditions:
% ------------------------------------------------------------
% seed the random number generator
rng(p.random_seed);
t=0; k=1;

% STATE_VARIABLES:
pop1_v = zeros(nsamp,p.pop1_Npop);
  pop1_v(1,:) = zeros(1,p.pop1_Npop);
% ###########################################################
% Numerical integration:
% ###########################################################
n=2;
for k=2:ntime
  t=T(k-1);
  pop1_v_k1=@current+10;
  t=t+.5*dt;
  pop1_v_k2=@current+10;
  pop1_v_k3=@current+10;
  t=t+.5*dt;
  pop1_v_k4=@current+10;
  % ------------------------------------------------------------
  % Update state variables:
  % ------------------------------------------------------------
  pop1_v(n)=pop1_v(n-1)+(dt/6)*(pop1_v_k1+2*(pop1_v_k2+pop1_v_k3)+pop1_v_k4);
  n=n+1;
end
T=T(1:downsample_factor:ntime);

end

It looks like dsSimulate can't locate iNa.mech and iK.mech. Could it be a
path issue? Does it work if you execute the code from the DynaSim "models"
sub-directory?
On Wed, Mar 21, 2018 at 2:13 PM, Dave notifications@github.com wrote:

The offending line in the solve file is: pop1_v_k1=@current+10;
Full solve file below
function [T,pop1_v]=solve_ode
% ------------------------------------------------------------
% Parameters:
% ------------------------------------------------------------
params = load('params.mat','p');
p = params.p;
downsample_factor=p.downsample_factor;
dt=p.dt;
T=(p.tspan(1):dt:p.tspan(2))';
ntime=length(T);
nsamp=length(1:downsample_factor:ntime);
% ------------------------------------------------------------
% Initial conditions:
% ------------------------------------------------------------
% seed the random number generator
rng(p.random_seed);
t=0; k=1;
% STATE_VARIABLES:
pop1_v = zeros(nsamp,p.pop1_Npop);
pop1_v(1,:) = zeros(1,p.pop1_Npop);
% ###########################################################
% Numerical integration:
% ###########################################################
n=2;
for k=2:ntime
t=T(k-1);
pop1_v_k1=@current+10;
t=t+.5dt;
pop1_v_k2=@current+10;
pop1_v_k3=@current+10;
t=t+.5dt;
pop1_v_k4=@current+10;
% ------------------------------------------------------------
% Update state variables:
% ------------------------------------------------------------
pop1_v(n)=pop1_v(n-1)+(dt/6)(pop1_v_k1+2(pop1_v_k2+pop1_v_k3)+pop1_v_k4);
n=n+1;
end
T=T(1:downsample_factor:ntime);
end
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/DynaSim/DynaSim/issues/398#issuecomment-375044375,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AQFleTr063CVpRJs-7ca7g6Di6kvfJn7ks5tgphGgaJpZM4SI5xX
.

I just created a fresh clone of DynaSim and re-ran everything. This solved the dsSimulate bug on master, but the dsPlot bug on dev is still there. So now the situation is:
master: works
debug: works
dev: Same error as above.

Just a heads up, I am still getting this error. Was this fixed? I am running using the latest version of dev.
>> dsPlot(data,'plot_type','raster')
Error using line
Vectors must be the same length.
Error in dsPlot (line 910)
                  line(thisAxes, [spk spk],[c+ypos-.5 c+ypos+.5],'color','k'); hold on

UGH Okay, this appears to actually have been multiple bugs on this line of dsPlot. I thought the removal of initialization in tight_subplot2 that Erik did in April would work, but that's not the only source for this unhelpful error. I now believe the main issue is thus: the line command in Matlab (after 2014) seems to accept Axes objects like gca as the first argument to its function call, but does NOT accept the old-school version of Axes object handles where they are just a number. I believe that when you're using a number to represent the handle, even though you can do things like get(thisAxes, 'xticks') and it'll work correctly, it will NOT work with the line function. Instead, if you do line(gca, [vlines(k)]..., it'll work fine (after version-2014x that is). Doing a git blame for this shows what I think is the offending commit: f70c3b40 where all gca usage was stripped out replaced with thisAxes usage, even in line calls that did not formerly pass gca (like here on this line). Jason apparently was getting the same error from that commit that I'm getting now, and which appears to be this issue. I've tested both this commit and the one immediately before it, and I do NOT get this error using the commit before f70c3b40: I can run dsPlot(data, 'plot_type', 'power') just fine.
Finally, there's also a version issue: in version 2013a, which is the default for me on the cluster, the line function does NOT accept even Axes object handles, likely because this was before the big plotting rework in Matlab in 2014. This means that using gca as the first argument to line when running the default cluster version is always going to give an error. This is probably why many line uses in code before f70c3b40 did not supply the Axes handle.
Solution: Remove the thisAxes argument from the line calls and TEST if their handling breaks any other plotting functions like text, etc.

@erik-roberts Can you undo the changes to line from commit  f70c3b4 and any other plot functions that need it? It's probably cleanest to just do it as a new commit.

Let's use a test of the matlab version and do axes then line if pre 2014a, else do line with axis.

okay i fixed it to use the old version so it should work now.  as matlab warns, this method is a little slower, so it may be good to add a test and use the faster method when possible. also, it steals focus.

authors: DynaSim asoplata davestanley erik-roberts jsherfey ncompanytech

repo: DynaSim/DynaSim


text:
🚨 Linting

switch to rubocop-lts, closes #607 #606 (and prevents the like from popping up with every future RuboCop release!)

Adds:
  spec.add_development_dependency 'rubocop-lts', '~> 10.0'
  spec.add_development_dependency 'rubocop-md'
  spec.add_development_dependency 'rubocop-packaging'
  spec.add_development_dependency 'rubocop-performance'
  spec.add_development_dependency 'rubocop-rake'
  spec.add_development_dependency 'rubocop-rspec'

The only changes in this chageset are linting changes.  Nothing operational or logic related.

Looks like there are some issues in the build.  Not sure if these changes caused them.  Will investigate.

Closing this and will do the individual gem additions separately.

source: gharchive/pull-request

repo: Dynamoid/dynamoid

license: MIT

license_type: permissive

license_source: github-api


text:
Readme Container Link Invalid

The following files contain this comment: defaults to docker.io/dynatrace/oneagent

https://github.com/Dynatrace/dynatrace-oneagent-operator/blob/master/deploy/cr.yaml
https://github.com/Dynatrace/dynatrace-oneagent-operator/blob/master/pkg/apis/dynatrace/v1alpha1/defaults.go
https://github.com/Dynatrace/dynatrace-oneagent-operator/blob/master/pkg/apis/dynatrace/v1alpha1/oneagent_types.go

Presumably this should be: https://hub.docker.com/r/dynatrace/oneagent

This also affects the current public documentation page: https://www.dynatrace.com/support/help/technology-support/cloud-platforms/openshift/installation-and-operation/full-stack/deploy-oneagent-on-openshift-container-platform/#expand-1553custom-resource-snippet

Thank you Adam for your message. docker.io is the official Docker registry. The URL at  https://hub.docker.com/ is referring to the web page with the image description, but not where the image itself is available from.

source: gharchive/issue

authors: Dynatrace-Adam-Gardner lrgar

repo: Dynatrace/dynatrace-oneagent-operator


text:
expose failurePolicy in values .yaml

Is your feature request related to a problem? Please describe.
the following parameter failurePolicy is hardcoded to ignore and it will be useful if we make it configurable:
webhooks:

name: webhook.pod.dynatrace.com
reinvocationPolicy: IfNeeded
failurePolicy: Ignore

Describe the solution you'd like
change it to:
{{.Values.webhook.mutatingWebhook.failurePolicy}}
Describe alternatives you've considered
manual patching after the chart deployment
Additional context
link to code:
https://github.com/Dynatrace/dynatrace-operator/blob/v1.1.0/config/helm/chart/default/templates/Common/webhook/mutatingwebhookconfiguration.yaml#L25

This has the implication that a failure in the webhook or problems with the webhook deployment, in general, would block the rollout of any pods that are expected to be monitored. We have added many features to reduce the impact of failure in any of our monitoring components on the general cluster. Still, we can make this setting configurable.
We'll add this in a future release of the Operator.

repo: Dynatrace/dynatrace-operator


text:
How to handle scenarios where you can have many errors?

I recommend that tests on size, path, hash value (and possible other tests) are "aggregated", so that the user of the validation tool gets lists of:

files that are described in the METS(s) but which are not to be found in the IP. (path one direction)
files that are found in the IP but which are not to be found in the METS(s). (path the other direction)
files with a mismatch between actual file sizes and file sizes as given in METS(s). (size)
files with a mismatch between actual file hash value and the and the hash value as given the METS(s). (hash)

(The two tests for mismatches between sizes and hashes can seem redundant seen with consistency check lenses)
I think it is important to "aggregate" these tests into lists since the range can go from 1 error message to possible millions of error messages if our tool will report one message for each inconsistency (e.g. 1 IP having 1 million files with wrong hash values).
When creating testcase.xml's for:
-size: CSIP27, CSIP41, CSIP54, and CSIP69.
-path: CSIP24, CSIP38, CSIP51, CSIP79, and CSIP110.
-hash: CSIP29, CSIP43,CSIP56, CSIP71
it makes no sense to have an error message that describes the specific error message and I suggest a more generic one. For CSIP27 I will suggest that the error message for wrong size is formulated something like:
There is a mismatch between file size described in METS.xml versus the actual file size for [number of inconsistencies ] files. For a list of the files with mismatches, click [here(list)].
Let us imagine an IP with inconsistencies in CSIP27 in dmdSec in an EAD.xml where a user have altered the file - and also some inconsistencies in one of the representations where a tiff-conversion has gone wrong.
If you click the list, you will see something like:

METS-file
METS-ID
METS-path to actual file
METS-size
Actual size

(filepath) to root METS.xml
prefix_ID-42
metadata/descriptive/EAD.xml
10160
10260

(filepath) to representation METS.xml
prefix_ID-23523
representations/rep_tiff/data/historicpicture.tif
234345
0

(filepath) to representation METS.xml
prefix_ID-123232
representations/rep_tiff/data/historicpicture2.tif
234345
0

From: https://github.com/DILCISBoard/eark-ip-test-corpus/issues/206

repo: E-ARK-Software/eark-validator


text:
disable all formats since this is causing issues with PDF build

for some reason we are getting docs build failure see https://readthedocs.org/projects/e4s/builds/19632339/ so i disabled all the formats

looks like the docs were built @eugeneswalker see CI check


text:
Barrett's Esophagus subject URI is incorrect

BMDB RDF says that for biomarker http://edrn.jpl.nasa.gov/bmdb/biomarkers/view/53 the AssociatedDataset predicate is https://edrn-labcas.jpl.nasa.gov/labcas-ui/c/index.html?collection_id=Barretts_Esophagus_Methylation_Profile_Dataset.
However, the actual subject URI is https://edrn-labcas.jpl.nasa.gov/labcas-ui/c/index.html?collection_id=Barrett's_Esophagus_Methylation_Profile_Dataset.

Old eCAS datasets were associated with biomarkers and in eCAS it was Barretts_Esophagus_Methylation_Profile_Dataset with the apostrophe.
Manually updating biomarkers with Barrett's_Esophagus_Methylation_Profile_Dataset with the apostrophe fixes the problem, so no code changes necessary.
Closing this but @mryan4 can confirm in the portal.


text:
Implementar Entra ID

Descripción: Implementar Entra ID para la autenticación y autorización en el proyecto, sincronizando con la base de datos y restringiendo la vista por roles.
Tareas:

Actualizar los archivos de configuración para incluir los ajustes de Entra ID.
Sincronizar Entra ID con la base de datos.
Implementar restricciones de vista basadas en roles.
Probar la integración para asegurar que funcione correctamente.

Criterios de aceptación:

Entra ID está integrado y funcionando correctamente.
La sincronización con la base de datos es exitosa.
Las vistas están restringidas adecuadamente por roles.
Todas las pruebas pasan.

He estado trabajando en la implementación de Entra ID desde antes de la creación de este issue. A continuación, adjunto los commits relacionados con este trabajo:

8032aca32694ffffae68a9e296435bc960bb9408
c49b8ee16cd4b713e343255be4173ddc3d0c2b17
9628ff47bf60e08d396607109bd62fe8a58944a4

Hecho. La implementación de Entra ID está completa y funciona correctamente. A medida que se creen las páginas adicionales, se ajustarán las restricciones de vista según los roles definidos.


text:
Is There any possible way to have OnnxGemm converted into torch.nn.Linear?

Hi, It is kind of you to share your work.
Very appreciate it.
I have noticed that the GEMM layer of the onnx graph will be converted into a object of OnnxGemm.
for example:

then we check the pytorch model

and its forward

I wonder that is it possible to have these GEMMs converted into PyTorch native module?
And it will be much more scalable.
Thank you!!

Hello, @feanor21115
We do this if possible - all inputs except the first one are initializers.

Thank you!!!


text:
🛑 EC3 portal for EOSC-Synergy is down

In 8b2b828, EC3 portal for EOSC-Synergy (https://servproject.i3m.upv.es/ec3-synergy/) was down:

HTTP code: 0
Response time: 0 ms

Resolved: EC3 portal for EOSC-Synergy is back up in 7db5fa5.

repo: EOSC-synergy/status


text:
🛑 SWIFT Openstack - INCD is down

In 42db540, SWIFT Openstack - INCD (https://stratus-stor.ncg.ingrid.pt:8080/) was down:

HTTP code: 0
Response time: 0 ms

Resolved: SWIFT Openstack - INCD is back up in 789c7a9.


text:
LLVM memory error crash on 2.0

A few times now on Jungle running nodeos 2.0.0-rc2 (B1 packaged version) we have had nodeos crash with just this in the log:
LLVM ERROR: out of memory
Just before the crash our monitoring reports normal memory usage for the nodeos process (4-5GB with ~25GB free) so it doesn't seem to be a memory exhaustion issue. On the last crash we managed to get a core dump and here is a full backtrace:
gdb.log

Here is a backtrace from a test failure
backtrace-eosio-build-19078-amazonlinux-2-nodeos-forked-chain-lr-test.log

The test failure might be unrelated. Steven found a race condition which there is a PR (#8305) up to fix. However, I think the original issue reported here is probably something different.

And Steven found another race condition, fixed by #125, which should fix the original reported problem on this issue.

Resolved by #8311

repo: EOSIO/eos


text:
fix: Remove hardcoded require statement inside of module (esm) code.

What kind of change does this PR introduce (bug fix, feature, docs update, ...)?
There was a
/* eslint-disable-next-line */
const HttpProvider = require("web3-providers-http");

in the code which is illegal in a esm module.
What is the current behaviour (you can also link to an open issue here)?
When using a true esm compiler like esbuild or bun the require function will no longer be defined
in the esm (module) context.
What is the new behaviour (if this is a feature change)?
It will work in cjs and esm
Other information:
https://app.clickup.com/t/2645698/DEV-8268

Waiting for @skimaharvey answer in our chat before merging

authors: Hugoo richtera


text:
Cookie for Auth

Hi.  Great job.
I have found 2 issues:

On odoo_client.dart / _updateSessionIdFromCookies , data from response.headers['set-cookie'] comes with an error from Odoo response after authentication  (I have tested with Community Ed.  Versions 12 and 14).

Content of set-cookie:
__cfduid=d7aa416b09272df9c8ooooooo84f5d031615155878; expires=Tue, 06-Apr-21 22:24:38 GMT; path=/; domain=.mhfly.com; HttpOnly; SameSite=Lax,session_id=16ec9824767d8ca27ooooooo2fa61803fb35857b; Expires=Sat, 05-Jun-2021 22:24:38 GMT; Max-Age=7776000; HttpOnly; Path=/
Note that separator between  SameSite=Lax AND  session_id =...  is a comma, not a semicolon.  This Odoo bug cause bad parsing in _updateSessionIdFromCookies

For a good callRPC you must send  __cfduid + session_id in Cookie, something like:
__cfduid=d0c6c49b60848cc2d509ooooooo1261615152080; session_id=5af3e0bb806a96b7oooooooo5398bc4be1a25

Even sending whole original set-cookie obtained from authentication also works (just changing  comma separator by semicolon).
Best regards.

Hi!
Thanks for reporting. That's interesting topic.
According to RFC 6265 the only valid separator for set-cookie is semicolon.
SameSite cookie is not added by Odoo. See https://github.com/odoo/odoo/issues/51065
Maybe your frontend is adding it and it is possible to change separator it uses?
CF is dropping __cfduid so not worth bothering https://blog.cloudflare.com/deprecating-cfduid-cookie/

It might be dart's issue.
Headers with same name are joined with comma https://github.com/dart-lang/http/blob/master/lib/src/io_client.dart#L43

@anexa-digital please try version 0.4.3. It should handle you case now.
Except second point.
Setting all cookies requires proper handling of cookie rules as SameSite or Expires.
It may be required for cases like Sticky Session. It should be done as separate feature not a fix commit.

repo: ERP-Ukraine/odoo-rpc-dart


text:
update Docker settings

updated the Dockerfile and removed the old ones
Fixes the issue #68.

Looks good to me, but the usage instructions need some more updates. I have created a separate issue for that, see above.

authors: bouweandela fdiblen

repo: ESMValGroup/ESMValTool


text:
Windows x64: conflicting defintions of round

Hello,
Compiling this on windows 10 with msvc2015 I run into an error:
1>BPMOscControler.cpp(53): error C2382: 'round': redefinition; different exception specifications
1>  D:\Program Files\VC\include\cmath(278): note: see declaration of 'round'

In cmath, my definition of round is:
_Check_return_ inline float round(_In_ float _Xx) _NOEXCEPT

Commenting it out makes it compile properly, but I haven't poked far enough into the source yet to determine what kind of odd side effects a float being returned might cause.
Is there a reason you don't use qRound()?

quick patch to use qRound, which seems to do the same as the inline round that exists.
diff --git a/src/BPMOscControler.cpp b/src/BPMOscControler.cpp
index a498643..3d8d29b 100644
--- a/src/BPMOscControler.cpp
+++ b/src/BPMOscControler.cpp
@@ -50,7 +50,7 @@ void BPMOscControler::save(QSettings& settings) {
}
-inline int round(float value) { return (fmod(value,1.0) < 0.5) ? value : value + 1; }
+//inline float round(float value) { return (fmod(value,1.0) < 0.5) ? value : value + 1; }
// Called by the bpm detector to make the controller send the new bpm to the clients
void BPMOscControler::transmitBPM(float bpm)
@@ -63,21 +63,21 @@ void BPMOscControler::transmitBPM(float bpm)
continue;
}
QString message(command);

   message.replace("<BPM>", QString::number(round(bpm)));

   message.replace("<BPM1>", QString::number(round(bpm)));

   message.replace("<BPM1-2>", QString::number(round(bpm*0.5)));

   message.replace("<BPM1-4>", QString::number(round(bpm*0.25)));

   message.replace("<BPM1-8>", QString::number(round(bpm*0.125)));

   message.replace("<BPM1-16>", QString::number(round(bpm*0.0625)));

   message.replace("<BPM1-32>", QString::number(round(bpm*0.03125)));

   message.replace("<BPM2>", QString::number(round(bpm*2)));

   message.replace("<BPM4>", QString::number(round(bpm*4)));

   message.replace("<BPM8>", QString::number(round(bpm*8)));

   message.replace("<BPM16>", QString::number(round(bpm*16)));

   message.replace("<BPM32>", QString::number(round(bpm*32)));

   message.replace("<BPM>", QString::number(qRound(bpm)));

   message.replace("<BPM1>", QString::number(qRound(bpm)));

   message.replace("<BPM1-2>", QString::number(qRound(bpm*0.5)));

   message.replace("<BPM1-4>", QString::number(qRound(bpm*0.25)));

   message.replace("<BPM1-8>", QString::number(qRound(bpm*0.125)));

   message.replace("<BPM1-16>", QString::number(qRound(bpm*0.0625)));

   message.replace("<BPM1-32>", QString::number(qRound(bpm*0.03125)));

   message.replace("<BPM2>", QString::number(qRound(bpm*2)));

   message.replace("<BPM4>", QString::number(qRound(bpm*4)));

   message.replace("<BPM8>", QString::number(qRound(bpm*8)));

   message.replace("<BPM16>", QString::number(qRound(bpm*16)));

   message.replace("<BPM32>", QString::number(qRound(bpm*32)));
   m_osc.sendMessage(message);

}
// Send information command

m_osc.sendMessage("/s2l/out/bpm=" + QString::number(round(bpm)), true);

m_osc.sendMessage("/s2l/out/bpm=" + QString::number(qRound(bpm)), true);

Not using qRound was a mistake, thank you for notifying me. I will patch this issue as soon as possible

Fixed in https://github.com/ETCLabs/Sound2Light/pull/9

authors: HendrikNoeller dmeltzer luminosuslight


text:
Lot size with multiple profit targets

Hi,
Thanks for the great tool.
I use it with multiple profit targets in an account with little funds. The issue I observe is that the distribution of the lot size by the different portions of the trade doesn't look correct.
I attach an example where I have 4 PT with 25% each, and one of the portions has 0.04 and the others 0.01. Ideally, they should all have the same lot size but since that's not possible (unit is 0.01 lots), I believe a 0.02, 0.02, 0.02, 0.01 lots would make more sense.
There are other small issues relative to multiple PT:

When the number of PT is changed, the percentage distributions of the PT get messed up but don't become red to indicate an error.
If I have 3 PT, it accepts 33% in each (doesn't become red) but that gives different lot sizes than 34%, 33%, 33%. seems something is not quite right.

Kind Regards, Jaime.

Hello!
Thank you for reporting!
I will consider changing the volume distribution algorithm. I will also fixed the bug for when the TakeProfitsNumber is changed.
As for the third bug you mention, I don't quite understand the issue. Could you please provide more details?

Hi,
Firstly, how do one divide evenly between 3 positions (33.33..% x3) ? The expert doesn't allow decimals so, if I enter 33 in all positions, does that mean each has 33.33% ?
In that situation (3 x 33%), the lot distribution seems worse than in previous example I showed. See attached image.
Kind regards,
Jaime Machado

3 x 33% is correct, but for some reason it doesn't work well in this case. Your calculated position size was 0.03 lot?

3 x 33% is correct, but for some reason it doesn't work well in this case. Your calculated position size was 0.03 lot?

Yes, 0.03 lot total.

Should be addressed in 3.03.

repo: EarnForex/PositionSizer


text:
Allows to invite a client whose email and mobile number is missing.

Screenshots

repo: EasyCloudConsultants/EasyCloudBooks


text:
Display Back Matter media types in a human-readable format

As a user involved in the governance, risk, and compliance of systems and components exploring controls, I want the media type of the back matter resources to be displayed in a human readable format when possible.
Acceptance Criteria

Back matter media types have human-readable names for the common types (OSCAL documents, PDFs, HTML document, etc)
If the media type doesn't match to something well-known, the exact value from the media-type field is displayed
If the media type isn't specified, whichever heuristic is used to determine a media type results in a human readable name (such as using the file extension)
Product Owner agrees with the list of recognized types.

Implementation Notes

Per the specification, this is supposed to be an IANA registered media type; however, it seems that some things don't comply with this

Our default catalog seems to not specify media-type for many documents; however we seem to detect many of them as PDFs or "Unknown"
Some of the media types, such as application/oscal.content+json is not formally registered as an IANA media type

Find an (the?) authoritative source for media types and recognize those.
Enumerate the official list of recognized types and associate with human-friendly names. Consider icons (not required)

Core list plus OSCAL extensions.

@brian-easyd We have an awkward issue with this because of the very small number of back matter resources in existing documents that specify a media-type. So in most cases, the viewer today falls back to Unknown or a guess based off the file extension.
Determining Media Type
graph TD
    A([Displaying Media Type]) --> B{Is `media-type` present?}
    B -->|Present| C
    B -->|Absent| D[Use file extension to guess media type]
    D --> E{Can media type\n be determine from\nextension}
    E -->|Media Type Found| C[Convert to friendly-name\nusing lookup]
    E -->|Not Found| Fallback([Display 'Unknown Content'])
    C --> Z([Display friendly name for\nmedia-type])

Friendly names for MIME types
The IANA registrations do not necessarily contain a friendly name. This lists how we will represent specific media types and classes of media types in the viewer.
image/

Name
Display in Viewer

gif
GIF

heic
HEIC Image[^1]

jpeg
JPEC

png
PNG

tiff
TIFF

vnd.adobe.photoshop
Photoshop PSD

svg+xml
SVG

Fallback: "Image"
application/

Name
Display in Viewer

json
JSON

oscal.content+json
OSCAL (JSON)

xml[^2]
XML

oscal.content+xml
OSCAL (XML)

octet-stream
Binary data

msword
Microsoft Word (.doc)

application/vnd.openxmlformats-officedocument.wordprocessingml.document
Microsoft Word (.docx)

vnd.ms-powerpoint
Microsoft PowerPoint (.ppt)

vnd.openxmlformats-officedocument.presentationml.presentation
Microsoft PowerPoint (.pptx)

vnd.ms-excel
Microsoft Excel (.xls)

application/vnd.openxmlformats-officedocument.spreadsheetml.sheet
Microsoft Excel (.xlsx)

pdf
PDF

pem-certificate-chain
PEM Chain

pkcs7-signature
PKCS7 Signature

pgp-signature
PGP Signature

ogg
OGG

rtf
Rich Text (RTF)

Fallback: "Binary Data"
text/

Name
Display in Viewer

html
HTML

xml[^2]
XML

oscal.content+xml
OSCAL (XML)

Fallback: "Text Content"
example/
We will always report as "Unknown Type". Should never appear in real-world documents.
No special handling for specific types

Registry
Display in viewer

audio/*
Audio

font/*
Font

message/*
Message

model/*
Model

video/*
Video

[^1]: Using "Image" is intentional; while HEIC is becoming a more common format, many users do not recognize it as an image type
[^2]: XML appears in both text/ and application/ but application/ is recommended. We will display both as "XML" (and the corresponding OSCAL extension) because they are functionally the same to users.

@kylelaker first I want to say that I love this level of planning! Overall this is great and very near what I had in mind. I'd like to offer the following feedback:

In the flos diagram, where media-type is present, we then have the question of "Recognized" (Y/N). If Recognized, the rest of your flow is valid. If not recognized, we should simply display "Not Recognized".
I'd actually like to see all images represented as "Image (GIF)", "Image (JPG)", etc. Typically the user will care more about whether it's an image (yes or no), and won't care about the image format.
When we get into AR, audio and video are potentially valid media types, and should at least be displayed as "Audio" or "Video" with the type in parens if know -similar to audio above. Some auditors prefer to record interviews (although often the org or individual staff members don't love that), and may take a video of a screen interaction or a social engineering test.

Otherwise, I think this is great for an MVP. I eventually see our tool recognizing output from other tools, like Nessus scanner output. But we can address those down the road.

authors: brian-easyd kylelaker

repo: EasyDynamics/oscal-react-library


text:
CalDAV/CardDAV Integration

Hi,
Your idea with EasyEngine and the upcomming mail feature are greate! BUT! What if you added CalDAV/CardDAV support to the mix? Then you would have the ultimate product, you can sync your mobile calendar and contacts directly with your own system.
This is a feature i miss, i use Google Apps for some other projects, but after they started charging, and i dont use it for E-Mail any more i lack the support of Cal/CardDAV support, so i can stay updated 100%
The greate thing about this, is the different ways to do this integration, SabreDAV looks like a good way to go, and the best part it can be integrated with Roundecube plugin.

We have dropped webmail support from v4 so this is no longer relevant. Sorry.

repo: EasyEngine/easyengine

license: mit

license_source: bigquery


text:
Added free Dutch programming books for Java and PHP

What does this PR do?
Add Resources
For resources
Description
Added free Dutch programming books for Java and PHP from Wikibooks.
Why is this valuable (or not)?
There are currently no Dutch programming books listed in this repository yet for Java or PHP.
How do we know it's really free?
The pages are released under the GNU Free Documentation License (GFDL).
For book lists, is it a book? For course lists, is it a course? etc.
It is a book.
Checklist:

[x] Not a duplicate
[x] Included author(s) if appropriate
[x] Lists are in alphabetical order
[x] Needed indications added (PDF, access notes, under construction)

Thanks, and welcome to free-programming-books!

authors: Hero-Solutions eshellman

repo: EbookFoundation/free-programming-books


text:
Added python game development course

What does this PR do?
Add resource(s) | Remove resource(s) | Add info | Improve repo
For resources
Description
Why is this valuable (or not)?
How do we know it's really free?
For book lists, is it a book? For course lists, is it a course? etc.
Checklist:

[x] Read our contributing guidelines.
[x] Search for duplicates.
[x] Include author(s) and platform where appropriate.
[x] Put lists in alphabetical order, correct spacing.
[x] Add needed indications (PDF, access notes, under construction).
[x] Used an informative name for this pull request.

Follow-up

Check the status of GitHub Actions and resolve any reported warnings!

please resolve conflicts.

authors: eshellman niketmishra


text:
Update model_import.py

Hello. I often encountered an Attribute Error when loading my YAML file and I tried to slightly modify the code to have it show an error message when the error pops up due to an attribute error.
In particular, I move the loading file part forward before " if check:~~~" part runs. In this case, you would see the error message if you made a syntax error in writing your YAML file.
I am not sure this way of revision would be proper. Any suggestion from experienced users would be more than welcome. Thanks.

Thank you @altophohe . This should work, but I'm a bit surprised you had to move the linter since there is there seems to be a bit of code here https://github.com/EconForge/dolo/blob/master/dolo/linter.py (430-435), which does the same.
Actually, the best way to proceed would probably to:

in model_import.py import the file surrounded by try/except like you do
in lint.py, fix the lint function so that it can either take a string, in which case it parses it as it currently does or, optionnally a yaml structure
in model_import.py pass the parsed structure to lint so that the model text is parsed only once

Does it make sense to you? Any chance you can fix the pull request accordingly?

Hmm. I moved the try/except part before the linter since I found that the yaml syntax error cannot be caught otherwise, but an error pops up during linter. Since linter does not indicates the syntax error(as far as I know), then an user may get lost and cannot see what was her real problem.  If we move the try/except part forward, then the program barf the error message when it detects any syntax error during parsing YAML file.
I will try to follow your instruction for further improvement. thanks for advice.

And I have another question. My try/except part catches an attribute error due to a mistake in writing a yaml file. In contrast, the similar part in linter catches Exceptions when loading yaml file. Do those two try/except parts play the exactly same role and catch the same error(same case)? If not, I am afraid that the linter may not work properly if I get rid of the similar part in linter.  How do you think?

Well, the first order thing is to tell user he got an user that is due to
the yaml syntax, not anything to do with the model. So I'm good with
catching any exception here. Plus when you raise this exception it
ultimately provides information about what happened.
BTW, the way you currently print the exception before running the command
again is not useful. You can print the message if you want then raise the
exception directly (raise exc). No need to run the command again.
On Thu, Jun 6, 2019 at 12:38 PM altophohe notifications@github.com wrote:

And I have another question. My try/except part catches an attribute error
due to a mistake in writing a yaml file. In contrast, the similar part in
linter catches Exceptions when loading yaml file. Do those two try/except
parts play the exactly same role and catch the same error(same case)? If
not, I am afraid that the linter may not work properly if I get rid of the
similar part in linter. How do you think?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/EconForge/dolo/pull/151?email_source=notifications&email_token=AACDSKM5ETOL3JOG4IXPEZDPZDSJVA5CNFSM4HTW7RU2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODXCOJUI#issuecomment-499442897,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AACDSKO53N3BPZ7PRHHMQCDPZDSJVANCNFSM4HTW7RUQ
.

authors: albop altophohe

repo: EconForge/dolo


text:
feat: footer improvements

Fixes Issue

Changes proposed

Check List (Check all the applicable boxes)

[ ] My code follows the code style of this project.
[ ] My change requires changes to the documentation.
[ ] I have updated the documentation accordingly.
[ ] All new and existing tests passed.
[ ] This PR does not contain plagiarized content.
[ ] The title of my pull request is a short description of the requested changes.

Screenshots

Note to reviewers

Thank you @schmelto

repo: EddieHubCommunity/LinkFree


text:
Fixed Navbar close on link click

Closes #2464
Added the onClick event on the NavLink to fix the issue.

Check List (Check all the applicable boxes)

[ ] My code follows the code style of this project.
[ ] My change requires changes to the documentation.
[ ] I have updated the documentation accordingly.
[ ] All new and existing tests passed.
[ ] This PR does not contain plagiarized content.
[ ] The title of my pull request is a short description of the requested changes.

Screenshots

Note to reviewers

Hey @Somidh , I tried running the code on gitpod but it doesn't seem to close the navbar. I have left some comments in the code review I think that might be the case.

authors: ChinmayMhatre Somidh


text:
Add User Arjun-Kadam

Related issue

What changes does this PR do?
Screenshots (If any)
Questions

Hello @aparna2071 I have placed name in right alphabetical order.

Hello @aparna2071 I have placed name in right alphabetical order.

Hello Arjun, I have already approved your PR, you have to wait for one more approval from our maintainers to get this PR merged 😄


text:
When using the standup command, can it remove the original message?

I believe this can be done by deleting the message that started the command, like
if (message.deletable) {
  message.delete();
}

But to only do this on a particular command, we would need the command handler to receive the message parameter.

Oh awesome 👍 . Thanks for the example 🤓

I've tested the delete method, and in order for the bot to execute it it needs the "Manage Messages" permission:

or this error occurs:

I think our server has the bots role so maybe we could add the permission there or create a new role with it.

As for the message parameter on the command handlers, what do you guys think @garretcharp @eddiejaoude ? If we have it, this code that deletes the message would be only on the commands that want that. Also, we could have access to more information when creating the embed like adding message.author.username

Sounds good. Having access to the username would be good to include in the bot's response. Could pass it as a 3rd parameter?

this rocks

authors: BOLT04 eddiejaoude stemount

repo: EddieJaoudeCommunity/EddieBot


text:
Please invite me to the GitHub Community Organisation

Invitation will be sent for the GitHub Organisation soon. We look forward to having you part of our community :nerd:
Don't forget after accepting to make it public so it appears on your github profile for everyone else to see, you can do this by finding your name in the github organisation list and change the dropdown to public https://github.com/orgs/EddieJaoudeCommunity/people
Tips for practicing:

Customise your GitHub profile, here is a live stream on it https://www.youtube.com/watch?v=cT6GXCuS0Zo
Practice repo, instructions how to add your name to the README in the README https://github.com/EddieJaoudeCommunity/hacktoberfest-practice
Remember contributing to open source is not just about code, its about collaboration, communication and adding value

I hope that helps

Invitation sent for the GitHub Organisation. Welcome to the community 🎉
Don't forget after accepting to make it public so it appears on your GitHub profile for everyone else to see, you can do this by finding your name in the GitHub organisation list and change the dropdown to public https://github.com/orgs/EddieJaoudeCommunity/people
Tips for practising:
Customise your GitHub profile, here is a live stream on it https://www.youtube.com/watch?v=cT6GXCuS0Zo
Practice repo, instructions on how to add your name to the README in the README https://github.com/EddieJaoudeCommunity/hacktoberfest-practice
Remember contributing to open source is not just about code, its about collaboration, communication and adding value
I hope that helps

authors: eddiejaoude spartaabhishek

repo: EddieJaoudeCommunity/support


text:
Is WRITE_EXTERNAL_STORAGE really required for Android?

In plugin.xml, Android requires a special permission, but seems to work fine on my device when I comment it out. https://github.com/EddyVerbruggen/SocialSharing-PhoneGap-Plugin/blob/master/plugin.xml#L61
<config-file target="AndroidManifest.xml" parent="/*">
  <uses-permission android:name="android.permission.WRITE_EXTERNAL_STORAGE"/>
</config-file>

Yep, see the README for the case when it's required (and it would crash your app if it wasn't added, hence the default).

Ah ok, sorry about that!

I got the same question as @monove .... My app only shares text (app URL) and just got this message from the Play Store while updating my app:
New permissions added
Warning:

Users that have the APK with version code 10006 may need to accept the android.permission.WRITE_EXTERNAL_STORAGE permission, which may result in them not upgrading to this version of the app.

Tip:

Ensure that the new permissions are necessary, and consider mentioning these permissions in the What's new in this release text.

I guess only Android 4.4 will get this permission "warning" but still......

authors: EddyVerbruggen JTronLabs JarLowrey rodrigograca31


text:
Cannot change default options without sourcing nvim config again

Description
Hello.
I was trying to change the default option of solid backgroud to transparent but wasn't successful in doing so.
The changes i made are not being loaded unless i manually source my neovim config each time i open a new instance of neovim.
Tried changing comments to italic to test and indeed the nightfox.setup function doesn't work unless the config is sourced again.
Tried doing :NightfoxStatus and  :NightfoxClean too. It didn't throw any errors.
Behavior
Expected behavior
The changes i made should get automatically loaded each time.
Screenshots
 output of :NightfoxStatus

 before sourcing my config: comments aren't italic, transparent isn't working

 after sourcing my config again

Configuration
-- nightfox options
require('nightfox').setup({
  options = {
    transparent = true,
    styles = {
      comments = "italic",
    },
  },
})

Reproduce

install the theme through vim plug
set theme as nightfox through colorscheme dayfox, change relevant options inside a lua block in init.vim

Additional info
neovim version: NVIM v0.7.2
Build type: Release
terminal: alacritty
platform: arch linux

That did the job. Thanks a lot :D


text:
Error when running :NightfoxCompile in Neovim

Description
Running :NightfoxCompile upon obtaining a fresh installation of Nightfox on Neovim gives an error.
Behavior
Expected behavior
The Lua code should compile and the theme should be available for use shortly after.
Screenshots

Configuration
Reproduce
These instructions are specifically for Vim-Plug. I have tried to install Nightfox using Packer as well, but the error persists.

Install Neovim.
create a directory "colors" in ~/.config/nvim.
create a directory "autoload" in ~/.config/nvim.
Install Vim-Plug into autoload.
create an init.vim file with the contents of "minimal_init" below in ~/.config/nvim.
open init.vim and run :PlugInstall.
run :NightfoxCompile or :colorscheme nightfox.

minimal_init.vim
call plug#begin('~/.config/nvim/colors')
Plug 'EdenEast/nightfox.nvim'
call plug#end()

Additional info / context
I am using Neovim 0.6.1 on Ubuntu 22.04 LTS. The $TERM variable is xterm-256color.

Doesn't seem like vim.notify is supported in neovim 0.6.1
The latest version is 0.8.1, consider updating?

Yes it does look like the neovim needs to be updated. Currently I have the requirements section saying that it requires 0.5 but that has not been re-evaluated in a while. I think I would say that it supports 0.8. I say that because of the treesitter changes. The theme will work with 0.7 as it is but there will be the invalid group message. Updating the required section.

That's it. I installed with just apt, but for some reason the version that apt had was outdated. Installing the .deb file for 0.8.1 worked perfectly. Thanks a lot and sorry for the inconvenience!

Np, it was a good reminder to check the required section.

authors: EdenEast mile-stojakovic nullchilly


text:
🛑 Portal Repositorio is down

In d2c685d, Portal Repositorio (https://www.repositorio.una.ac.cr/) was down:

HTTP code: 0
Response time: 0 ms

Resolved: Portal Repositorio is back up in eb22fc0 after 529 days, 12 hours, 58 minutes.

repo: EdgardoRamosRoque/status


text:
Panic with completely black image

Hi,
I'm found a problem while i try to check dominant color on completely black image like the one below.

This is my code:
cols := prominentcolor.KmeansWithAll(3, img, prominentcolor.ArgumentNoCropping, 80, []prominentcolor.ColorBackgroundMask{})

col := cols[0].AsString()
fmt.Println("#"+col)

This is the output that i recive:
2018/01/21 16:31:05 Failed, k larger than len(allColors): 3 vs 0
panic: invalid argument to Intn

goroutine 1 [running]:
math/rand.(*Rand).Intn(0xc042072270, 0x0, 0x3)
	{GO_PATH}/src/math/rand/rand.go:142 +0xa3
math/rand.Intn(0x0, 0x0)
	{GO_PATH}/src/math/rand/rand.go:277 +0x3e
github.com/EdlinOrg/prominentcolor.kmeansPlusPlusSeed(0x3, 0x8, 0x614528, 0x0, 0x0, 0x0, 0x0, 0x0)
	{GO_PATH}/src/github.com/EdlinOrg/prominentcolor/kmeans.go:387 +0xd7
github.com/EdlinOrg/prominentcolor.kmeansSeed(0x3, 0x614528, 0x0, 0x0, 0x8, 0x0, 0xc0429de3c0, 0x5e3700)
	{GO_PATH}/src/github.com/EdlinOrg/prominentcolor/kmeans.go:357 +0x18e
github.com/EdlinOrg/prominentcolor.KmeansWithAll(0x3, 0x5e3700, 0xc0429de3c0, 0x8, 0x50, 0xc04296dec4, 0x3, 0x3, 0x501daa, 0xc042002020, ...)
	{GO_PATH}/src/github.com/EdlinOrg/prominentcolor/kmeans.go:111 +0x104
github.com/EdlinOrg/prominentcolor.KmeansWithArgs(0x8, 0x5e3700, 0xc0429de3c0, 0xc0429de3c0, 0x0, 0x0)
	{GO_PATH}/src/github.com/EdlinOrg/prominentcolor/kmeans.go:101 +0x104
main.main()
	{PROJECT_PATH}/main.go:38 +0x1d1

At the moment i moved to jakekausler/prominentcolor@7583347b935415ac0b34343888fc2b35c6f2e1e7 a fork of this project that return errors instead of panic, so i can handle exceptions.
How can i have #000000 instead of panic?

Nice, I pulled in those changes here. Odd that it finds 0 colors. The way you call it is correct to get it to ignore the masks, I tried with the image above and your call, and it found 1 color. I submitted some minor (unrelated) fixes, so please pull and try again. If you still have issues, can you please mail me the image you use, and I'll try again.

repo: EdlinOrg/prominentcolor


text:
Cannot find module './Autosuggest.vue'

Hi!
I have some problem when try to include with the gulp and the browserify.
Error: Cannot find module './Autosuggest.vue' from '/var/www/withpassion/docker-projects/wesele123.local/html/node_modules/vue-autosuggest/dist'
    at /var/www/withpassion/docker-projects/wesele123.local/html/node_modules/browser-resolve/node_modules/resolve/lib/async.js:55:21
    at load (/var/www/withpassion/docker-projects/wesele123.local/html/node_modules/browser-resolve/node_modules/resolve/lib/async.js:69:43)
    at onex (/var/www/withpassion/docker-projects/wesele123.local/html/node_modules/browser-resolve/node_modules/resolve/lib/async.js:92:31)
    at /var/www/withpassion/docker-projects/wesele123.local/html/node_modules/browser-resolve/node_modules/resolve/lib/async.js:22:47
    at FSReqWrap.oncomplete (fs.js:123:15)

Process finished with exit code 1

My run code
import VueAutosuggest from 'vue-autosuggest'
Any idea?

I am seeing the same thing from browserify builds. I've confirmed other vue libraries that are using a similar UMD bundling via webpack as the one I'm using receive the same issue while building via browserify. I'll look into this and report back!

@trocho should be fixed with 1.1.0 version. Thanks!

I have a some other problem now :)

vue.js:485 [Vue warn]: Failed to mount component: template or render function not defined.

found in

---> <VueAutosuggest>

VueAutosuggest is a plugin, so you will need to case Vue.use() to load it in your app so it can be used.
Example:
import Vue from 'vue'
import App from './App.vue'
import VueAutosuggest from 'vue-autosuggest';

Vue.use(VueAutosuggest);

new Vue({ // eslint-disable-line no-new
  el: '#app',
  render: (h) => h(App)
})

I'll update the README.md to make this more clear.

Yes. Good idea!
Thanks for help

authors: darrenjennings trocho

repo: Educents/vue-autosuggest


text:
Events section modified

I have updated the events section.
Kindly review and merge ,also if you wanna any other changes then please inform me.
https://github.com/Eduhub-Community/Eduhub-Community.github.io/assets/132778037/1519edaa-4d8e-4d42-8564-472619532289

I can't see any improvement

I can't see any improvement
Okie, I 'll add some more effects on this between
I have added box - shadows and change some padding.

resolved conflicts

@SahityaRoy  please add level number to the PR.

@SahityaRoy   , @AshAman999  please add level  number this PR.


text:
Highscore widget

I was thinking of adding a Highscore text at the top of the screen somewhere around the text which displays the current score. The solution I could think of is by creating a new array which stores the final scores of the current session and a new function which computes the maximum score and displays it. Can I start working on this? Can you think of any other way to make it work?

@Redarcher9 that's a great idea.
I think your solution sounds good, perhaps make it obvious that the score is the users highest score 'your highest score'.
Let me know if you need any help.

I tried to update my fork with the update instructions provided in the contributions.md file.But,the repository in my profile still says 'This branch is 2 commits ahead, 7 commits behind EdwardDunn:master'.

@Redarcher9 make sure you have the upstream repo as your remote.
'git remote -v' will show this.

Completed by @Redarcher9

repo: EdwardDunn/Platform-Game


text:
Value passed to parameter 'input_sizes' has DataType float32 not in list of allowed values: int32, int64

When I run you item, I get this error.
Traceback (most recent call last):
File "E:/GAN/7-3D-IWGAN-master/3D-IWGAN-master/3D-Generation/32-3D-Gan.py", line 49, in
net_g , G_train     = generator_32(z, is_train=True, reuse = False, sig= True, batch_size=args.batchsize)
File "E:\GAN\7-3D-IWGAN-master\3D-IWGAN-master\scripts\models.py", line 74, in generator_32
net_2 = Deconv(net_1, gf_dim, eighth, '2', batch_size)
File "E:\GAN\7-3D-IWGAN-master\3D-IWGAN-master\scripts\models.py", line 20, in Deconv
act=tf.identity, name='g/net_' + net + '/deconv')
File "D:\ProgramData\Anaconda3\envs\tensorflow\lib\site-packages\tensorlayer\layers.py", line 1512, in init
self.outputs = act( tf.nn.conv3d_transpose(self.inputs, W, output_shape=output_shape, strides=strides, padding=padding) + b )
File "D:\ProgramData\Anaconda3\envs\tensorflow\lib\site-packages\tensorflow\python\ops\nn_ops.py", line 2473, in conv3d_transpose
name=name)
File "D:\ProgramData\Anaconda3\envs\tensorflow\lib\site-packages\tensorflow\python\ops\nn_ops.py", line 2539, in conv3d_transpose_v2
name=name)
File "D:\ProgramData\Anaconda3\envs\tensorflow\lib\site-packages\tensorflow\python\ops\gen_nn_ops.py", line 2237, in conv3d_backprop_input_v2
dilations=dilations, name=name)
File "D:\ProgramData\Anaconda3\envs\tensorflow\lib\site-packages\tensorflow\python\framework\op_def_library.py", line 627, in _apply_op_helper
param_name=input_name)
File "D:\ProgramData\Anaconda3\envs\tensorflow\lib\site-packages\tensorflow\python\framework\op_def_library.py", line 61, in _SatisfiesTypeConstraint
", ".join(dtypes.as_dtype(x).name for x in allowed_list)))
TypeError: Value passed to parameter 'input_sizes' has DataType float32 not in list of allowed values: int32, int64
I find this error raise becaues 'Conv3DBackpropInputV2' not has float32 dtype, I don't konw why.
Do you have some ideas for this error.
By the why, what tensorflow and tensorlayer version do you used?

Hi, try tensorflow==1.13.2 and tensorlayer==1.5.0

Hi, try tensorflow==1.13.2 and tensorlayer==1.5.0

After replacing with tensorflow==1.13.2 and tensorlayer==1.5.0, the error still exists. Is there any other solution?

Hi, which python version are you using?

I found that the issue is in the Deconv function in ./scripts/models.py, where the out_shape should be of type int instead of float. So I modified it like this:
def Deconv(inputs, f_dim_in, dim, net, batch_size, f_dim_out=None, stride=2):
    if f_dim_out is None:
        f_dim_out = f_dim_in / 2
    output_shape = [batch_size, dim, dim, dim, f_dim_out]
    output_shape = tf.cast(output_shape, tf.int32)
    return tl.layers.DeConv3dLayer(inputs,
                                   shape=[4, 4, 4, f_dim_out, f_dim_in],
                                   output_shape=output_shape,
                                   strides=[1, stride, stride, stride, 1],
                                   W_init=tf.random_normal_initializer(stddev=0.02),
                                   act=tf.identity, name='g/net_' + net + '/deconv')

But this might not be the best solution, as it could lose some precision.

authors: EdwardSmith1884 EfreetWarfarin Julius-ZCJ Yucitrus


text:
fix: upload: adds button type props to show list item interface

SUMMARY:
Adds <button> type props to Upload showUploadList interface and defaults them to 'button' to prevent unintended Form submit

JIRA TASK (Eightfold Employees Only):
ENG-49788
CHANGE TYPE:

[X] Bugfix Pull Request
[ ] Feature Pull Request

TEST COVERAGE:

[ ] Tests for this change already exist
[X] I have added unittests for this change

TEST PLAN:
Pull the PR branch and run yarn and yarn storybook. Verify the Upload stories behave as expected.

Codecov Report

:exclamation: No coverage uploaded for pull request base (main@176d484). Click here to learn what that means.
Patch has no changes to coverable lines.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
Additional details and impacted files
@@           Coverage Diff           @@
##             main     #596   +/-   ##
=======================================
  Coverage        ?   84.60%
=======================================
  Files           ?      793
  Lines           ?    17348
  Branches        ?     6138
=======================================
  Hits            ?    14677
  Misses          ?     2605
  Partials        ?       66

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.

authors: codecov-commenter dkilgore-eightfold

repo: EightfoldAI/octuple


text:
mower gets invisible at some orientations

regulary the mower get's invisible / has no icon at some orientation.

Currently there is one image per robot rotation, I think the image should be able to be rotated instead with time image.rotate() function.
I might look into this at some point, unless @EinEinfach or someone else wants to do it first.

Currently there is one image per robot rotation, I imagine there is one missing.
I think we can have one image and just rotate it with image.rotate() function instead .
I might look into this at some point, unless @EinEinfach or someone else wants to do it first.

Good idea!

I commited some changes related to this issue. Pls test.

@disaster123 are you still seeing this issue? I think I was seeing that at times, but lately haven't noticed it.

I had too bad wheather the last days. Will need to check this the next days. Basically i can tell you that I'm still seeing the robot sometimes rotated by 180 degrees. So in some orientation it drives suddenly backwards in cassandra while driving forward in real.

Yeah, I think I have also seen it facing the wrong way when docked, in particular. I just started a mow task from the taskplanner page a moment ago, and when I went to the main page, I noticed I didn't see the mower at all. I could see the next point it was going to, but the mower wasn't shown. I tried refreshing the whole page, and it still isn't shown.

+1 same still here

Just noticed there has been another commit about the rover images from yesterday, and I haven't tried that out yet. Also wanted to note that mine did eventually show on this mow task, but it isn't clear what caused it to show again.

Are you use last version of Cassandra?

This is with latest master:

mower is currently docked and invisible....

Try to change mower image in settings section. Is the mower visible after that? If yes change it again to wanted image, maybe is something gone wrong at start up

no does not help. I switched the image multiple times - mower stays invisible until it moves / rotate again

Can you increase PIL logger level,  maybe then I can see more

Is the mower also invisible on mapping page?

authors: EinEinfach ShadedSelf disaster123 rothnic

repo: EinEinfach/CaSSAndRA


text:
Add tutorial for creating layers in frontend

Basics

[ ] The PR is rebased with current master
[ ] I added a line to changelog.md
[ ] Details of what I changed are in the commit messages
[ ] References to issues, e.g. close #X, are in the commit messages and changelog
[ ] The buildserver is happy

Checklist

[ ] I fully described what my PR does in the documentation
[ ] I fixed all affected documentation
[ ] I fixed the introduction tour
[ ] I wrote migrations in a way that they are compatible with already present data
[ ] I fixed all affected decisions
[ ] I added automated tests or a manual test protocol
[ ] I added code comments, logging, and assertions as appropriate
[ ] I translated all strings visible to the user
[ ] I mentioned every code or binary not directly written or done by me in reuse syntax
[ ] I created left-over issues for things that are still to be done
[ ] Code is conforming to our Architecture
[ ] Code is conforming to our Guidelines
[ ] Code is consistent to our Design Decisions
[ ] Exceptions to any guidelines are documented

First Time Checklist

[ ] I have installed and I am using pre-commit hooks
[ ] I am using Tailwind CSS Linting

Review

[ ] I've tested the code via issue description
[ ] I've tested the code via requirements
[ ] I've tested the code with concurrency (several browsers on the same map)
[ ] I've read through the code
[ ] I've read through the documentation
[ ] I've checked conformity to guidelines
[ ] I've checked conformity to requirements
[ ] I've checked that the requirements are automatically tested

Please state if you think something should be added or is unclear. Especially the part Implement Layer Logic might need some extenion or more details maybe.

repo: ElektraInitiative/PermaplanT


text:
[Feature Request] input-number provides placeholder functions

Existing Component
是
Component Name
el-input-number
Description
目前的el-input-number 如果input框里没有内容应该显示placeHolder的内容,但现在没有这个功能.我觉得加上会更加友好一点。

already fulfilled.

repo: ElemeFE/element


text:
[Bug Report] Text  's cursor height not correct in Safari

Element UI version
2.11.1
OS/Browsers version
macOS 10.14 and above
Vue version
2.6.10
Reproduction Link
https://jsfiddle.net/mmx38qxw/
Steps to reproduce

active the text input field
the cursor's height is as same as input height

What is Expected?
Default cursor's appearance need some padding, as same as in Chrome.
What is actually happening?
Default cursor's appearance without padding on top and bottom

Screenshot of Safari

Screenshot of Chrome

pr welcome


text:
[Feature Request] 能否优化一下以下几点功能呢?(Can you optimize the following functions?)

Existing Component
是
Component Name
el-image,el-calendar,el-tooltip
Description
1.el-image: 希望可以开放el-image的预览功能的遮罩点击不关闭的prop,现在是有这个属性但是没有放出来;
2.el-calendar: 希望同element-plus的展示,当最后一排都是下个月的天数时不展示出来,可以增加个属性或者什么,不是默认展示42个格子,如issue;
3.el-tooltip: 当文字非常多且有滚动条的时候展示的组件会有个left: -20px;导致文字展示不全,希望能优化一下..

基于目前现状，感觉以后基本上是以Vue3为主，所以大概率饿了么团队只修改一些bug或者就慢慢不维护（维护人员不能一直靠爱发电），精力投入到plus上。
如果老项目还需要持续使用Vue2+element，有精力的那就fork一份出来，自己维护修改发布，没精力的话那就只能摊手了……

好吧
------------------ 原始邮件 ------------------
发件人: @.>;
发送时间: 2023年2月22日(星期三) 中午12:27
收件人: @.>;
抄送: @.>; @.>;
主题: Re: [ElemeFE/element] [Feature Request] 能否优化一下以下几点功能呢?(Can you optimize the following functions?) (Issue #22403)
基于目前现状，感觉以后基本上是以Vue3为主，所以大概率饿了么团队只修改一些bug或者就慢慢不维护（维护人员不能一直靠爱发电），精力投入到plus上。
如果老项目还需要持续使用Vue2+element，有精力的那就fork一份出来，自己维护修改发布，没精力的话那就只能摊手了……
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you authored the thread.Message ID: @.***>

请前往讨论区征集意见~~~

authors: wangdaodao webvs2 zc-write-code


text:
[Feature Request] returns to submit the form

Existing Component
是
Component Name
Form
Description
是否可以和普通表单那样，在按下回车时提交表单？最好是在最后一个输入框时起效，其他的跳转到下一个输入框

We have no plan to support this, but you can implement it on your end easily.


text:
[Bug Report] [ElContainer] Vertical layout judgment

Element UI version
2.0.4
OS/Browsers version
chrome
Vue version
2.5.3
Reproduction Link
https://github.com/ElemeFE/element/blob/dev/packages/container/src/main.vue
Steps to reproduce
官网文档:
el-container：外层容器。当子元素中包含  el-header 或 el-footer 时，全部子元素会垂直上下排列，否则会水平左右排列。
把el-header封装成组件
return tag === 'el-header' || tag === 'el-footer'; //失效
What is Expected?
组件包装el-header生效
What is actually happening?
组件包装el-header不生效

是的，这样 Container 无法判断，因此提供了 direction 属性用于直接指定排列方向。


text:
[Bug Report] el-input-number not listening 'input' event

Element UI version
2.1.0
OS/Browsers version

Vue version
2.5.13
Reproduction Link
https://jsfiddle.net/Lvhcwuo3/1/
Steps to reproduce
随便用键盘输几个数
What is Expected?
el-input-number收到input事件时触发handleInputChange，同时触发上一级input事件，即重现内的alert
What is actually happening?
没触发

#9116 #9166 导致input事件无法传导到外面，事实上input事件必须监听

没听明白你想说什么。「上一级input」指什么？0.0x又是什么？

0.0x 指的是#9116 报告的错误，“需要触发上一级的input”指的是引用该组件时的input事件侦听，亦即在jsfiddle内，键盘输入数字不会触发input事件的问题；input事件在el-input内被触发了，但是被el-input-number截断了

那你期望的行为是什么。
用户未完成输入时不触发input-number的input有什么影响？如果有，需求是什么

期望触发input。
首先，必然有一部分用户的需求是获得input元素的input事件，不能因为包装而把原生事件吃掉了；
其次，原生input，el-input都能触发input事件，el-input-number不行，逻辑不一致；
第三，一个常见需求是网页前端做修改时，通过绑定向后端发请求，以同步用户设置，在此类页面，一般不提供保存按钮。一个很可能发生的操作是，用户做了修改，然后等了几秒（他认为设置已经同步了），然后关闭页面；此时来不及发起后端请求，或者后端请求被浏览器取消，则用户修改不会被保存。
需求是：用户通过键盘输入时，触发input事件，同时v-model绑定生效。

如果input和v-model同时都有，那么0.0这样的输入必然被截断，于是用户输入0.03这样的值会有问题。目前来说，发射input会引起value更新，从而刷新组件状态。
另外，因为组件值是number，number 0.0和0是无法区分的。
我并不认为逻辑不一致，date-picker/select之类可以接受输入的组件也仅在用户完成输入后（change）后才发射input，不会发射中间值或无效值。
如果需要同步数据并且服务器响应需要到秒级，这更像是服务器太慢而不是前端应该考虑的问题。
如果需要知道用户做了输入，input-number可以考虑提供类似原生composition的事件，然后在这个事件中自行处理用户输入决定要不要更新组件value。

@Leo

根据我对代码的理解，0.0发射input引起value更新从而刷新组件状态是可以避免的，在 handleInput 内，加入
if (newVal === oldVal) return
即可，正如判断小数点和减号一样。
逻辑不一致的点在于，data-picker和select都不是同一类原生元素的扩展，现在是原生的input元素及其扩展出来的el-input会发射input，然而el-input-number不会。
同步数据并不是服务器需要响应到秒级，而是当在el-input-number输入后，到用户关闭网页前，没有获得输入框内容的方法。
另外，如果仅仅将el-input发射出来的input事件再次发射出去，应该不会引起任何副作用。

handleInput方法是本次 向后不兼容 更新之前的方法，可以在某些特殊情况下在输入框内保留逻辑上相等的输入值（0,
-0, 0.0, 0.00等）而不是破坏它。

既然找到答案了的话就开个pr吧。记得写测试。

I'd like to add my use case: https://jsfiddle.net/9mbe0pbv/1/
Why is the input not emitting the input event? It makes the value not reactive to inputs, only to change... I am losing data-binding here. There isn't even an option.

@Leopoldthecoder
See also: https://github.com/ElemeFE/element/pull/9690/
Patched: https://jsfiddle.net/9mbe0pbv/4/

Greedily emit input (if parsed number is the same as text input: emit; otherwise, do not emit)
贪心的发射input，如果被发射出去的 number 的字符串形式和 input 里的一样，发射；如果不一样就把这个input吃掉（滑稽

@wacky6 Why isn't your correct emit input in master?

@Leopoldthecoder
See also: #9690
Patched: https://jsfiddle.net/9mbe0pbv/4/
Changes: wacky6@1f1a0af
Greedily emit input (if parsed number is the same as text input: emit; otherwise, do not emit)
贪心的发射input，如果被发射出去的 number 的字符串形式和 input 里的一样，发射；如果不一样就把这个input吃掉（滑稽

请问 merge 了吗

authors: Elfayer GreatFireWall dukkee garveen wacky6


text:
[Feature Request] Parent node highlight for tree component.

Existing Component
Yes
Component Name
Tree
Description
Would it possible to have an attribute in tree component to highlight the parent node of the TREE.
Like (highlight-current) but just the parent node, not children even if you click children node it highlights parent.
Thank you and help would be appreciated.

Sorry, we have no plan to support this.


text:
Click event on step.vue

Add click emit on step.vue.
Permit to make action when user click on step.
Please make sure these boxes are checked before submitting your PR, thank you!

[x] Make sure you follow Element's contributing guide (中文 | English | Español).
[x] Make sure you are merging your commits to dev branch.
[x] Add some descriptions and refer relative issues for you PR.

Deploy preview for element ready!
Built with commit ecf173ca0f1e802fb493a142cb39fe1d41c14a33
https://deploy-preview-11175--element.netlify.com

Duplicate of #11169. This is not necessary. You can use @click.native.

authors: Leopoldthecoder gparant ziyoung


text:
[Bug Report] Index List 无法正常滚动

Mint UI version
2.2.9
OS/Browsers version
iOS 10.3.3
Vue version
2.4.3
Reproduction Link
http://elemefe.github.io/mint-ui/#/index-list
Steps to reproduce
正常情况下都无法滚动
What is Expected?
可以正常滚动
What is actually happening?
正常情况下都无法滚动

+1   在ios 的微信 和  ios的谷歌浏览器都不能滑动

...前几天发现这个问题之后，想提issue, 发现已经有人提过了，收藏了下这个issue页面，现在自己做出来了发现还是没有人回应....再看时间，原来楼上那位早在2017年9月份就提出来了，然而官方demo还是没有解决，也没有回应。这个项目还在维护吗？（并没有很生气要抹黑这个UI框架或者是质问什么，毕竟知道别人开源也不容易，只是问问。）

@fefuns 你的问题是怎么解决的？希望指教一下

@lds8988 这上面沟通不是很方便，可以来前端群474471759找我一起交流下

2018-07-05 这个bug 还在

2018-07-11 这个bug 还在

2018-09-06 这个bug还在

repo: ElemeFE/mint-ui


text:
vue-infinite-scroll无限触发loadmore方法

每次请求完，回调里的数组list为空的时候 就会触发无限加载方法

之前写这个功能的时候看文档疏忽有遇到无限触发loadmore方法的问题，是因为没有设置容器高度。loadMore组件初始化会自动检测容器高度，如果没有撑满会自动调取bottom-method直到撑满或者人为阻止。
不太确定是不是你需要的，暂提个可能的思路。

@andylei18 返回的数组为空的时候，设置 infinite-scroll-disabled 为 true 似乎可以解决。

这样的话 你不能触发 再次加载啦？这样流程就断了

@Leopoldthecoder 那这样的话，你再次滚动就触发不了。。。。流程就断了

@xyDiva 列表容易高度我有计算，你说的bottom-method，autoFill:false  API里也没有，vue-infinite-scroll 那个issues里 N多 和我一样情况的人

如果返回的数组为空的时候，确实会引发无限加载，而且设置 infinite-scroll-disabled 为 true，没有用处，还是在一直加载。

我这还是会无限加载，这个问题解决了么？

我这里已经解决了，这个东西完全可以人工阻止。写几个判断判断是否需要加载

问题怎么解决的？

@th720309 忘记了，好久以前了，代码在家里的固态硬盘上。

@th720309 那个loadMore方法不是会调用很多次么。在 第一次调用的时候，就要拿到数据的总数(total)。然后之后每次调用loadMore的时候，在loadMore里面判断，如果当前vue组件里面的list 数据 和 total(总数)相等，说明数据已经全部加载完了，就在前端直接return，不发ajax去后台了 . 这样不管这个loadMore调用多少次，都只有第一次loadMore的数据。不要质疑这个解决方案。你可以在每次loadMore的时候都更新这个total值

@th720309 大概意思就loadMore方法 里面判断数据有没有加载完，数据全部加载完了，loadMore就直接返回，不在发ajax请求了。

试了很多方法，无奈只能这样处理了...
在极端的情况下，infinite-scroll-disabled这个根本没什么用

这个无限滚动真是用的 有点无语 , 默认请求两次 是什么鬼

authors: Leopoldthecoder YanwuSenlu androidxu andylei18 dinglittle th720309 watership wsqy182 xyDiva


text:
Typo

在阅读您的文章时
Network > 粘包 >方案1
少了一个 不 字

Thx, fixed.

repo: ElemeFE/node-interview


text:
[Refactor] Process Whitespace for greedy_until

@haileyschoelkopf after some thought, I think we should just add warnings regarding the use of whitespace in target_delimiter or in the target. While for some models, whitespace is required to be in the target ( yes and  no for example) some tokenizer like T5 or languages like Japanese don't necessarily use whitespaces prepended to the targets.
The warnings are:

If both the target_delimiter and the target have whitespace.
If both the target_delimiter and the target does not have whitespace.

Another addition is to allow target to be processed from doc_to_choice if doc_to_target is an integer.
PR for #696

I will review this tomorrow.

TODO: Add postprocessing to remove whitespace that's in the front of the generated text by the model.

authors: haileyschoelkopf lintangsutawika

repo: EleutherAI/lm-evaluation-harness


text:
feature/#117 : 메인 카드뷰 컴포넌트 작업

📑 제목

📎 관련 이슈
closes #117
✔ 셀프 체크리스트

[x] Warning Message가 발생하지 않았나요?
[x] Coding Convention을 준수했나요?
[ ] Test Code를 작성하였나요?

💬 작업 내용
메인컴포넌트내에 있는 카드뷰를 완성하였습니다.
🚧 PR 특이 사항
현재는 병원이름 , 동네 , 키워드순서대로 들어가있는데 가격을 넣는다면 어디에 넣으면 좋을까요???
또는 저 위에 순서가 바뀌어도 좋으니 순서를 추천해주셨으면 좋겠습니다.
[[FE] 메인 페이지]-[메인 컴포넌트]-[디자인 작업]
🕰 실제 소요 시간
5h

확인했습니다!
가격은 카드 크기를 조금 키워서 제목-위치-가격-키워드순으로 두면 어떨까요? 혹시 카드를 에어비엔비처럼 box-shadow 없이 뜨게 하는건 어떤가요?

제 기준에는 라인이 없다보니까 뭔가 쫌 휑한 느낌이 있어서 저렇게 진행했는데 다른분들은 어떻게 생각하시나요???

확인했습니다!
가격은 카드 크기를 조금 키워서 제목-위치-가격-키워드순으로 두면 어떨까요? 혹시 카드를 에어비엔비처럼 box-shadow 없이 뜨게 하는건 어떤가요?

제 기준에는 라인이 없다보니까 뭔가 쫌 휑한 느낌이 있어서 저렇게 진행했는데 다른분들은 어떻게 생각하시나요???

아 이렇게보니까 뭔가 어색하긴하네요 쉐도우 있는게 더 나은 것 같습니다

authors: ejaman hovelopin


text:
Unable to create character with a space in the name

Environment

OS version: Windows 7
Elona Foobar verision: develop

Details
You cannot create a character that has a space in the name, as it fails with this error:
boost::filesystem::copy_file: The system cannot find the path specified: "e:/build\ElonaFoobar\bin\debug\tmp\mdata_7_101.s2", "e:/build\ElonaFoobar\bin\debug\save\sav_ase asd \mdata_7_101.s2"

How to reproduce

Attempt to create a character with a space in the name.

On my environment, this issue occurs.
OS version: Windows 10 Home 1709
Elona Foobar version: 328a5646fdc5aa2841ca9c132026e935b1b1fd5b
I tried these names:

"abc"(a, b, c): ok
"a b c d e"(a, space, b, space, c, space, d, space, e): ok
"ase asd "(a, s, e, space, a, s, d, space): error
"a "(a, space): error

I tried also them in Japanese mode, then the results are the same.
Probably the last space cause the issue.

It seems to occurs only on Windows.
I can create a character whose name is "a "(a, space) on macOS High Sierra 10.13.3.

repo: ElonaFoobar/ElonaFoobar


text:
Improve Mac OS dmg GUI

Spinoff from #30
Improve the Mac OS dmg GUI so it looks prettier by adding a .DS_Store, a volume icon, and a background image.
@9Morello has taken up the task in #30.

So, I've made a simple background image we can use with the .dmg.
I believe we can add some scripts from here to build the .dmg with the background and the correct icon sizes to fit: https://asmaloney.com/2013/07/howto/packaging-a-mac-os-x-application-using-a-dmg/

As for a volume icon, I'll see if I can mock up something... its definitively harder to make something passable. hehe

Looks good so far. You don't have to work so hard to create a nice volume icon; you can even grab the Chromium logo and use that. The Windows and Debian builds aren't getting anything nearly as fancy as this. Something simple, understandable, and functional will do.
Minor nitpick: Could you change the text to "ungoogled-chromium"? It looks weird to me when the "u" is capitalized and the hyphen is removed, haha.

You're the boss.

If someone wants to submit a PR for this, feel free to do so.

repo: Eloston/ungoogled-chromium


text:
Add video MIME types

https://github.com/Elyx0/react-native-document-picker/blob/e4c03667a87ffba54832563b11e00c6e5dd183e0/index.js#L62
In the list of available types there's no option for video. Seems in master there's support for that, is there any reason why it isn't for v3?

Available in master, clossing.


text:
upgrade prettier

upgrade prettier
remove pretty lint

@wolverineks, let's resolve conflicts here plz, and then we will merge right away 🙏 Sorry the PR got out of sync that much

removes dependency on prettylint
requires adding ci step yarn prettier --list-different ./src/**/*.js

@wolverineks , resolve conflicts please, after the hocs PR got merged

authors: vantuz-subhuman wolverineks

repo: Emurgo/yoroi-mobile


text:
Systemtests: kubernetes client cannot remove address space plan config

there is test failure when after some test, which create new address space plan config, is thrown xception, that address space plan is still there and his configs for addresses are deleted.
14:36:46,292 [main] INFO io.enmasse.systemtest.AddressApiClient - Following payload will be used in POST request: {"apiVersion":"v1","kind":"AddressSpace","metadata":{"name":"test1","namespace":"test1"},"spec":{"type":"standard","plan":"test1","authenticationService":{"type":"standard"}}}
java.lang.RuntimeException: 500: {"description":"Error validating address space plan weak-plan: missing [standard-topic-weak, standard-queue-weak]"}
	at io.enmasse.systemtest.AddressApiClient.lambda$null$0(AddressApiClient.java:84)
	at io.vertx.ext.web.client.impl.HttpContext.lambda$null$0(HttpContext.java:125)
	at io.vertx.core.impl.ContextImpl.lambda$wrapTask$2(ContextImpl.java:339)
	at io.netty.util.concurrent.AbstractEventExecutor.safeExecute(AbstractEventExecutor.java:163)
	at io.netty.util.concurrent.SingleThreadEventExecutor.runAllTasks(SingleThreadEventExecutor.java:403)
	at io.netty.channel.nio.NioEventLoop.run(NioEventLoop.java:463)
	at io.netty.util.concurrent.SingleThreadEventExecutor$5.run(SingleThreadEventExecutor.java:858)
	at java.lang.Thread.run(Thread.java:748)

fixed by: https://github.com/EnMasseProject/enmasse/commit/21a5be048bb0ea25018d920c46d6466fcbe9b356

repo: EnMasseProject/enmasse


text:
Fix more test regressions introduced by #5265 (fabric8 kubernetes-client)

Type of change

Bugfix

Description
Fix test regression introduced by #5265 (the fabric8 kubernetes-client upgrade)
Checklist

[ ] Update/write design documentation in ./documentation/design
[ ] Write tests and make sure they pass
[ ] Update documentation
[ ] Check RBAC rights for Kubernetes / OpenShift roles
[ ] Try your changes from Pod inside your Kubernetes and OpenShift cluster, not just locally
[ ] Reference relevant issue(s) and close them after merging
[ ] Update CHANGELOG.md

@enmasse-ci run tests profile=shared AuthorizationTest,UserApiTest

:x: Test Summary :x:
TEST_PROFILE: shared
TEST_CASE:
TOTAL: 220
PASS: 204
FAIL: 3
SKIP: 13
BUILD_NUMBER: 1440
BUILD_ENV: oc cluster up
:heavy_exclamation_mark: Test Failures :heavy_exclamation_mark:

testMessagesStoredMetrics in io.enmasse.systemtest.shared.standard.web.ChromeConsoleTest
testMessagesStoredMetrics in io.enmasse.systemtest.shared.standard.web.FirefoxConsoleTest
testSortMessagesCount in io.enmasse.systemtest.shared.standard.web.FirefoxConsoleTest

authors: enmasse-ci k-wall


text:
(TEST) We can generate an accept message when a request for create X metering point has been accepted

Benefit hypothesis (Done by PO)
As a grid company
We want to be able to get a reply for our request for create X metering point
So that we know if our request has been approved.
Description (Done by PO)
The aim of this user story, is to be able to generate an accept message for the grid companies request for create X metering point, when their request has been approved.
The grid company needs a reply to know if their request has been approved.
Acceptance criteria (Done by team, is checked off by PO)

[ ] When a request for create metering point has been approved we are able to reply with an accept message.
[ ] We reply with an accept message  immediately after approval. Must be within 1 hour after the request has been approved
[ ] The accept message adheres to relevant CIM format.

Task (Done by developers, is checked off by developers)

[ ] Create ,,,,,

DoD (predefined, is checked off by developers and PO)

[ ] All tasks have been completed (Dev)
[ ] Has been tested by business (PO/SME)
[ ] Has been tested by Dev. (Can we be specifc about what tests?)
[ ] All acceptance criteria are met (Checked of by SME/PO)
[ ] Solution has been deployed to test environment (Dev)
[ ] Business/PO has signed off and approved story
[ ] Solution has been documented (technical doc)
[ ] .....

With team I mean everybody in team (PO, SM, Dev and SME)

Comment about DoD.
The most important thing for the team is to have a DoD. The aim is not to start off by making a long list. The aim is to come up with a list that ensures quality and to ensure that we with confidence can say that this story is done, and that the requested functionality has been delivered.
Lets start out with the DoD that we can come up with now, and down the road we can build further upon it.
Right now we just need some sort of way to now when a story is done.

This one is closed as it primarily has worked as a template.

repo: Energinet-DataHub/geh-metering-point


text:
Volt's test actor in U-001 is not able to peek confirmations for (UpdateChargePrices), while confirmations for (UpdateChargeInformation) can be peeked

Description:
Volt's test actor in U-001 is able peek confirmations for (UpdateChargeInformation), but cannot peek confirmations for (UpdateChargePrices)
Note: Batman's test actor is able to peek both in U-001. And it works fine in T-001.
Data details:

Volt test actor (id: 08ED22E3-615D-4F0E-993D-CFC1F4A833F8, actorId: A4ABD4AD-4FF3-4E93-A86D-7B164910CA34, gln: 9656626091925)
Batman's test actor (id: "7f7974dd-d2af-460e-8684-cc61f58f8117", actorId: A982DE1F-3703-4E3C-BE56-4E54418A2A59, gln: 0808118335003)
PartitionKey that showed some problem: "facef55d-e842-42c1-92fd-451e2d09d56c"

UpdateChargeInformation is peekable, why UpdateChargePrices is not.
How to reproduce:
(0) Probably the easiest thing is to involve Volt in the reproduction, otherwise :)...
(1) Go to Postman > Charges Domain workspace
(2) Find the following request and use it to create a new subscription:
Samples/ChargeIngestion/ 'Create Subscription (P1M) without price' => change the mRID used in line 18 to something that does not exist in charges db and submit it.
(3) Find the following request and use it to submit a new price to the newly created subscription
Samples/ChargeIngestion/UpdateChargePrices 'D08 UpdateChargePrices - Subscription - P1M' => remember to same mRID in line 18 as in step (2).
(4) Peek the first time and expect a 'ConfirmRequestChangeOfPriceList' with processType = D18
(5) Dequeue it and peek again, and expect another 'ConfirmRequestChangeOfPriceList' but this time with processType = D08.
No message is received in step 5 . A HTTP 204 No Content is provided.

Volt skriver, at det lader til at virke nu.

authors: defectiveAi prtandrup

repo: Energinet-DataHub/geh-post-office


text:
fix: avoid empty document in archived message

Description
Archived message stored empty market documents aggregated measure data was requested in the b2c flow.
References

Codecov Report
Attention: 513 lines in your changes are missing coverage. Please review.

Comparison is base (21978a1) 83.07% compared to head (06c88af) 10.75%.
Report is 314 commits behind head on main.

Files
Patch %
Lines

...ssages.Infrastructure/ArchivedMessageRepository.cs
10.00%
54 Missing :warning:

...ce/ArchivedMessages.Infrastructure/QueryBuilder.cs
0.00%
47 Missing :warning:

...essages/RequestAggregatedMeasureMessageReceiver.cs
0.00%
32 Missing :warning:

.../Api/Authentication/DevMarketActorAuthenticator.cs
0.00%
29 Missing :warning:

...uration/Middleware/UnHandledExceptionMiddleware.cs
0.00%
29 Missing :warning:

...rce/Api/Authentication/MarketActorAuthenticator.cs
0.00%
27 Missing :warning:

...thentication/MarketActorAuthenticatorMiddleware.cs
0.00%
25 Missing :warning:

source/Api/EventListeners/InboxEventListener.cs
0.00%
24 Missing :warning:

.../Authentication/BearerTokenAuthenticationMethod.cs
0.00%
23 Missing :warning:

source/Api/OutgoingMessages/PeekRequestListener.cs
0.00%
21 Missing :warning:

... and 29 more

Additional details and impacted files
@@             Coverage Diff             @@
##             main     #668       +/-   ##
===========================================
- Coverage   83.07%   10.75%   -72.33%
===========================================
  Files         456      410       -46
  Lines       10418     8860     -1558
===========================================
- Hits         8655      953     -7702
- Misses       1438     7897     +6459
+ Partials      325       10      -315

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

authors: MadsDue codecov-commenter

repo: Energinet-DataHub/opengeh-edi


text:
Deactivated tests for registration

Based on the discussed approach for solving the hen and egg problem for EMB, we decided to deactivate the tests of EMI while registering. This PR corresponds hence to satisfying all bullet points in the last comment of #28 .

In addition, nightly tests were deactivated due to problems with Mocking.jl.


text:
Unable to use pixeldata module

Hi, I have noticed that the repo add submodule pixeldata recently and would like to give it a try. However, I can't import it after specifying dependency in my Cargo.toml.
[dependencies]
dicom = {version = "0.4.0"}

use dicom::pixeldata // <- compiler complains about not finding pixeldata in dicom

After reading the comment in pixeldata/lib.rs, I also tried following configs.
[dependencies]
dicom_pixeldata = {version = "0.1", default_features = false}

This time cargo can't even locate the crate and I can't google for the pixeldata crate either. Could you give a hand how to correctly import pixeldata? Thank you very much.

Hello, @posutsai. The dicom-pixeldata crate is a recent introduction to the project, and it has not been released yet, nor has it been incorporated into the library parent crate dicom. To use it now (although you should know that heavy changes may still happen until it is released) you can depend on the upstream repository.
[dependencies]
dicom-pixeldata = { git = "https://github.com/Enet4/dicom-rs" }

See also #125 to keep track of the progress of this crate.

Thank you for quick reply. I've successfully installed dicom-pixeldata. I'll close the issue and keep track of its progress.


text:
After the update, the webpage cannot save the picture

As shown in the figure, after the image is generated, it disappears, and the entire page stops responding. Clicking the Generate button again does not respond. How can I solve this problem?
PS: I can't speak English, so I can only use machine translation. I'm sorry if it bothers your reading.

You just encountered the infamous Gradio bug. Basically, Gradio runs a reverse proxy service that connects your WebUI interface to the internet. Unfortunately, their proxy service is really shitty and can't proxy a file larger than 2MB.
There is literally no easy solution for this. Paperspace locks your account for using any tunneling proxy service such as ngrok or CloudFlare Tunnel. What I do is run a rathole server on a $5 DigitalOcean cloud VPS which I've had great success with. Here's some more tunneling proxy servers here: https://github.com/anderspitman/awesome-tunneling
I'm sorry, but you're on your own with this.

try rollback gradio in requirement file to 3.9 and it should be work.

https://github.com/Engineer-of-Stuff/stable-diffusion-paperspace/blob/master/docs/Tunneling Proxy for Paperspace.md

it's already fixed for me,try this new method that recently commit by auto, --gradio-queue,also to make sure remove --gradio-debug,since it break progress bar for ui

I was wondering why the progress bar was broken!

authors: Cyberes ShadeRedeye start1002

license: Unlicense


text:
tests for initial pin value

The fix was already merged into master, these are just tests to validate #34 and #35

These are just tests, doesn't touch main library. Moving forward with the merge.

repo: EnotionZ/gpio


text:
Feature/update requirement

Updated version for ES to target new version.

@vinay-ebi It's approved,m but the build fails with :
ERROR: Service 'celery-monitor' failed to build : Build failed
The command "docker-compose -f ./docker-compose-test.yml build" failed and exited with 1 during .
Any clue?

repo: Ensembl/ensembl-prodinf-event


text:
Option to add padding (to fix antialiasing)

First of all, thanks for awesome work!
I found a "bug", or maybe unwanted antialiasing feature when zooming webpage in Chrome
Example instead of 1000 words :)
Original sprite containing two images - blue and red square

Rendering only green square

Zooming Chrome to 75% produces leaking of the red square

Example to try in different browsers http://codepen.io/Strajk/pen/aOwREe
I believe that padding between images would fix this issue... or any other ideas?

This is one of the first features we added to spritesmith :tada:
https://github.com/Ensighten/spritesmith/issues/7
There is a padding parameter for you to add which should solve this problem:
https://github.com/Ensighten/grunt-spritesmith/tree/4.7.1#documentation
{
  // src, dest, destCss,
  padding: 1
}

Everything is awesome :sparkles: , thanks!

authors: Strajk twolfson

repo: Ensighten/grunt-spritesmith


text:
Support for Shanghai / Solidity 0.8.20

Tangentially related to other issues, but as an old version of revm is used, the PUSH0 opcode introduced in Shanghai will always cause a revert, meaning that anything compiled with 0.8.20 won't run in the simulator.

Indeed, now it makes sense why all my contract transactions got reverted without any apparent reason !
Tried with an older contract deployment using 0.8.0 and worked well

authors: ape-dev-cs polyphia

repo: EnsoFinance/transaction-simulator


text:
[Snyk] Security upgrade aws-amplify from 5.3.19 to 6.0.0

Snyk has created this PR to fix 1 vulnerabilities in the yarn dependencies of this project.
Snyk changed the following file(s):

services/ui-src/package.json
services/ui-src/yarn.lock

Note for zero-installs users
If you are using the Yarn feature zero-installs that was introduced in Yarn V2, note that this PR does not update the .yarn/cache/ directory meaning this code cannot be pulled and immediately developed on as one would expect for a zero-install project - you will need to run yarn to update the contents of the ./yarn/cache directory.
If you are not using zero-install you can ignore this as your flow should likely be unchanged.
Vulnerabilities that will be fixed with an upgrade:

Issue
Score

Cross-site Scripting (XSS) SNYK-JS-COOKIE-8163060
  44

[!IMPORTANT]

Check the changes in this PR to ensure they won't cause issues with your project.
Max score is 1000. Note that the real score may have changed since the PR was raised.
This PR was automatically created by Snyk using the credentials of a real user.

Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open fix PRs.
For more information:
🧐 View latest project report
📜 Customise PR templates
🛠 Adjust project settings
📚 Read about Snyk's upgrade logic

Learn how to fix vulnerabilities with free interactive lessons:
🦉 Cross-site Scripting (XSS)

addressed by https://github.com/Enterprise-CMCS/macpro-mdct-mfp/pull/791

authors: gmrabian mdct-github-service-account

repo: Enterprise-CMCS/macpro-mdct-mfp


text:
Move compilation out of threadsfor

Has a remaining bug in reverse mode, and prints need removal.

@vchuravy
wmoses@beast:~/git/Enzyme.jl (rtest) $ cat thread.jl
using Enzyme
Enzyme.API.printperf!(true)
function foo(y, s, e)
        Base.Threads.threading_run() do
            @inbounds y[s] *= 2
        end
        nothing
end

x = [1.0, 2.0, 3.0]

dx = [1., 1.0, 1.0]
Enzyme.API.printall!(true)
@show Enzyme.autodiff(foo, Duplicated(x, dx), 1, 1)
@show x, dx

Failed for me on 1.7.1 with:
ERROR: LoadError: MethodError: Cannot `convert` an object of type Core.LLVMPtr{UInt8, 0} to an object of type Ptr{Nothing}
Closest candidates are:
  convert(::Type{Ptr{T}}, ::Union{Int64, UInt64}) where T at ~/builds/julia-1.7.1/share/julia/base/pointer.jl:26
  convert(::Type{Ptr{T}}, ::Ptr{T}) where T at ~/builds/julia-1.7.1/share/julia/base/pointer.jl:29
  convert(::Type{Ptr{T}}, ::Ptr) where T at ~/builds/julia-1.7.1/share/julia/base/pointer.jl:30
  ...
Stacktrace:
 [1] Enzyme.Compiler.Tape(thunk::Enzyme.Compiler.AdjointThunk{typeof(Base.Threads.threading_run), Const{Nothing}, Tuple{Duplicated{var"#1#2"{Vector{Float64}, Int64}}}, Val{0x0000000000000001}, Nothing}, internal_tape::Core.LLVMPtr{UInt8, 0}, shadow_return::Nothing, resT::Type)
   @ Enzyme.Compiler ~/src/Enzyme/src/compiler.jl:304
in expression starting at /home/vchuravy/src/Enzyme/thread.jl:18

Replaced by #342

authors: vchuravy wsmoses


text:
support invalidation for user-defined rules

fixes #696

Codecov Report
Patch coverage: 88.88% and project coverage change: -3.91 :warning:

Comparison is base (939f9b4) 76.32% compared to head (58035e8) 72.41%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #702      +/-   ##
==========================================
- Coverage   76.32%   72.41%   -3.91%
==========================================
  Files          18       18
  Lines        7057     6993      -64
==========================================
- Hits         5386     5064     -322
- Misses       1671     1929     +258

Impacted Files
Coverage Δ

src/compiler.jl
74.32% <88.88%> (-2.30%)
:arrow_down:

... and 11 files with indirect coverage changes
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.

This is ready to go.

authors: aviatesk codecov-commenter


text:
Merge chargear and plrgear and fix wrong tool position and clipping in to baseplate

Fixes #1257
There is practically no reason to have these as two seperate commands
This also fixes the wrong tool position in both commands as well as resetting the currentplayer camera as well as the bug when you wield the tool in which your character glitches in to the baseplate

Awesome, I always wondered why this happened

repo: Epix-Incorporated/Adonis


text:
Add dbt-unit-testing to dbt package Hub

Hey,
Could we get the dbt-unit-testing to dbt's package hub?
Mainly for selfish reasons (our production runner cannot import from Git) but it should also help with better handling of duplicate dependencies + get more eyeballs on the package 😄
Love the package btw

I'm in strong support if the team is in support - we'd love to lean on the package syntax in the Package File rather than git refs except in cases where we pin to a ref/SHA for specific functionality.

Just want to chime in as well - I think it would be fantastic for this project to be added to dbt's package hub!

Hi @will-warner, we and the people who are using the package would appreciate it if you can find the time to raise the PR.
Thank you!

Thanks for the nod of approval @cdiniz - PR raised.

Closing out as now added to the Hub

authors: cdiniz daveworth lkell will-warner

repo: EqualExperts/dbt-unit-testing


text:
appearance?

styling via appearance so we don't need to do it one by one
thanks

it would be great if you fork and make pull request for it. Thanks

authors: ErAbhishekChandani dexcell

repo: ErAbhishekChandani/ACFloatingTextfield


text:
T2 branch

Note: Confirm the contribution guidelines before submitting a pull request.
Click the Preview tab and select a PR template:

Standard change
Small change

Do NOT send a PR with this description.

/cherry-pick PR_target

/cherry-pick PR_target

repo: Ericpotato/autoware


text:
[license] Change license

Closes #2697

I created a simple python script which can be used to replace license header in the source files:
#!/usr/bin/env python3

import fnmatch
import os
import re

root_dir = '.'

exclude_dirs = [
    './.git/*',
    './venv_dev/*',
    './build/*',
    './web/server/vendor/*',
    './site/*',
]

exclude_files = [
  'relicense.py'
]

old_license_with_hash = r"""
# [-]+
#                     The CodeChecker Infrastructure
#   This file is distributed under the University of Illinois Open Source
#   License\. See LICENSE\.TXT for details\.
# [-]+
""".strip()

new_license_with_hash = """
# -------------------------------------------------------------------------
#
#  Part of the CodeChecker project, under the Apache License v2.0 with
#  LLVM Exceptions. See LICENSE for license information.
#  SPDX-License-Identifier: Apache-2.0 WITH LLVM-exception
#
# -------------------------------------------------------------------------
""".strip()

old_license_with_slash = r"""
// [-]+
//                     The CodeChecker Infrastructure
//   This file is distributed under the University of Illinois Open Source
//   License\. See LICENSE\.TXT for details\.
// [-]+
""".strip()

new_license_with_slash = """
// -------------------------------------------------------------------------
//  Part of the CodeChecker project, under the Apache License v2.0 with
//  LLVM Exceptions. See LICENSE for license information.
//  SPDX-License-Identifier: Apache-2.0 WITH LLVM-exception
// -------------------------------------------------------------------------
""".strip()

old_license_with_multiline_comment = r"""
/\*\*
 \* [-]+
 \*                     The CodeChecker Infrastructure
 \*   This file is distributed under the University of Illinois Open Source
 \*   License\. See LICENSE\.TXT for details.
 \* [-]+
 \*/
""".strip()

new_license_with_multiline_comment = """
/**
 * -------------------------------------------------------------------------
 *  Part of the CodeChecker project, under the Apache License v2.0 with
 *  LLVM Exceptions. See LICENSE for license information.
 *  SPDX-License-Identifier: Apache-2.0 WITH LLVM-exception
 * -------------------------------------------------------------------------
 */
""".strip()

old_license_in_create_new_subcommand = r"""
#                     The CodeChecker Infrastructure
#   This file is distributed under the University of Illinois Open Source
#   License. See LICENSE.TXT for details.
"""

new_license_in_create_new_subcommand = """
#
#  Part of the CodeChecker project, under the Apache License v2.0 with
#  LLVM Exceptions. See LICENSE for license information.
#  SPDX-License-Identifier: Apache-2.0 WITH LLVM-exception
#
"""

licenses = [
  (re.compile(old_license_with_hash), new_license_with_hash),
  (re.compile(old_license_with_slash), new_license_with_slash),
  (re.compile(old_license_with_multiline_comment),
   new_license_with_multiline_comment),
  (re.compile(old_license_in_create_new_subcommand),
   new_license_in_create_new_subcommand),
]

for subdir, dirs, files in os.walk(root_dir):
    # Skip directories.
    if any([fnmatch.fnmatchcase(subdir, e) for e in exclude_dirs]):
        continue

    for file in files:
        # Skip files.
        if any([fnmatch.fnmatchcase(file, e) for e in exclude_files]):
            continue

        file_path = os.path.join(subdir, file)
        try:
            with open(file_path, 'r+',
                      encoding="utf=8",
                      errors="ignore") as f:
                content = f.read()

                for pattern, new_license in licenses:
                    print(file_path)
                    # Replace old license with the new one.
                    if pattern.search(content):
                        print("Replace license in:", file_path)
                        content = re.sub(pattern, new_license, content)

                        f.seek(0)
                        f.truncate()
                        f.write(content)

                        continue
        except Exception as ex:
            print("Failed to process: ", file_path)

repo: Ericsson/codechecker


text:
🛑 MiDesarrolloTec is down

In a6ba493, MiDesarrolloTec (https://midesarrollotec.mx) was down:

HTTP code: 0
Response time: 0 ms

Resolved: MiDesarrolloTec is back up in db78105 after 8 hours, 3 minutes.

repo: ErnesthoMtz/upptime


text:
chore: correction typos

Modified a few typos. Hope this helps.
Closes #<ISSUE_NUMBER>

This PR:

This PR does not:

Key places to review:

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

Hi @osrm! Thanks for your contribution to HotShot! We always value community collaboration, and we appreciate you taking the time to make these fixes. I'll get this merged.

authors: CLAassistant jparr721 osrm

repo: EspressoSystems/HotShot