Initial TorchScript version of the core classes

This is still missing a bunch of magic functions (__getitem__, …) and other convenience which exist on equistore-core, but should be at a good starting point for review!

:books: Documentation preview :books:: https://equistore--263.org.readthedocs.build/en/263/

This should be ready for review and merge! For anyone wanting to give this a look, this PR is better reviewed one commit at the time.

Hello Guillaume,
Thanks for the work! I tried it out with the spherical expansion code from @frostedoyster and it worked after some changes.
The only problem I had with the core package is that the __iter__ function is not available which is convenient to use. Do I understand correct that to include the __iter__ function, we would need to do another change like this one https://github.com/pytorch/pytorch/pull/100724/ ?
In this case it is probably is not even possible because the output type is generic? I am also confused about where this items() function is implemented. I could only find the declaration.
Another point the __repr__ behavior is different
In [6]: tm_ref # equistore
Out[6]:
TensorMap with 6 blocks
keys: ['a_i' 'lam' 'sigma']
         1     0      1
         6     0      1
         8     0      1
         1     1      1
         6     1      1
         8     1      1

In [125]: tm_ref #equistore.torch
Out[125]: <torch.ScriptObject at 0x7f47cd423e70>

I am not sure if this is something that is part of this PR, but I wanted to mention it in case it is: When I tried to add equistore-torch and equistore-operations as setup requirements,
https://github.com/agoscinski/torch_spex/blob/cdac1549948250cd3ff407465bdc7478882d16e8/setup.py#L19-L32
I get a nonsensical dependency conflict on the same version.
ERROR: Cannot install equistore-operations==0.1.0.dev286 and equistore-torch==0.1.0.dev286 because these package versions have conflicting dependencies.
ERROR: Resolution Impossible: for help visit https://pip.pypa.io/en/latest/topics/dependency-resolution/#dealing-with-dependency-conflicts
The conflict is caused by:
    equistore-operations 0.1.0.dev286 depends on equistore-core 0.1.0.dev286 (from /tmp/pip-install-6b03c972/equistore-operations_e14fe4088fca448786f5b9b3654f02b8/python/equistore-core)
    equistore-torch 0.1.0.dev286 depends on equistore-core 0.1.0.dev286 (from /tmp/pip-install-6b03c972/equistore-torch_a4828d7d7a304d0eb6b2b79c1aceaca7/python/equistore-core)

https://github.com/agoscinski/torch_spex/actions/runs/5171050634/jobs/9314351880
In the end I made it work by installing them separately in the tox file, and it worked fine.

The only problem I had with the core package is that the iter function is not available which is convenient to use. Do I understand correct that to include the iter function, we would need to do another change like this one https://github.com/pytorch/pytorch/pull/100724 ?

No, we need much more than a simple change in torch: the __iter__ function is returning a generator (i.e. it uses yield statements), but these are not supported by TorchScript, so we would have to add a significant feature to TorchScript. The plan is to replace for key, block in tensor with a more explicit for key, block in tensor.items().

Another point the repr behavior is different

https://github.com/pytorch/pytorch/pull/100724 fixes this, but will only be available in torch 2.1

I am not sure if this is something that is part of this PR, but I wanted to mention it in case it is: When I tried to add equistore-torch and equistore-operations as setup requirements,
https://github.com/agoscinski/torch_spex/blob/cdac1549948250cd3ff407465bdc7478882d16e8/setup.py#L19-L32
I get a nonsensical dependency conflict on the same version.

This is a known issue, but it is getting very annoying. I'll check if there is a solution, but I think the only way out is to make a release on PyPI

Thanks for the answers.
I encountered a problem when scripting LabelsView which you get when accessing a Labels with square brackets __getitem__
import torch
from equistore.torch import Labels

@torch.jit.script
def foo():
    samples=Labels(
        names=["s_1", "s_2"], values=torch.IntTensor([[0, 0], [1, 1], [2, 2]])
    )
    return samples["s_1"].values

Error
Traceback (most recent call last):
  File "/home/alexgo/code/torch_spex/labelsview-scripting.py", line 5, in <module>
    def foo():
  File "/opt/anaconda3/envs/equitorch/lib/python3.10/site-packages/torch/jit/_script.py", line 1341, in script
    fn = torch._C._jit_script_compile(
RuntimeError:
'Any' object has no attribute or method 'values'.:
  File "/home/alexgo/code/torch_spex/labelsview-scripting.py", line 9
        names=["s_1", "s_2"], values=torch.IntTensor([[0, 0], [1, 1], [2, 2]])
    )
    return samples["s_1"].values
           ~~~~~~~~~~~~~~~~~~~~ <--- HERE

This works
import torch
from equistore.torch import Labels

@torch.jit.script
def foo():
    samples=Labels(
        names=["s_1", "s_2"], values=torch.IntTensor([[0, 0], [1, 1], [2, 2]])
    )
    return samples.values[:, samples.names.index("s_1")]

Yes, this is a known limitation: https://github.com/lab-cosmo/equistore/blob/f995ba2c479941cc4feff19f666ea73c83679a89/python/equistore-torch/equistore/torch/documentation.py#L251-L254
You can either use the view() function explicitly
samples.view("s_1").values

or help the compiler refine types:
view = samples["s_1"]
assert isinstance(view, Labels)


Fix reported values for Tuya Air sensor

CO2 and Formaldehyd values were mixed up. Also the formaldehyd value always reported wrong values, I can prove it by being still alive :)
The formaldehyd value is now in the 0.00X ppm range normaly. That means we need also the increase the decimals in HA zha/sensor.py from 0 to 3 or the value will always be zero.  Shall I create also a pull request for HA?
@cloudbr34k84 you seem to had the same problem in https://github.com/zigpy/zha-device-handlers/issues/1313 , could you verify that this is solving the issue for you as well?

I'm pretty sure that there are 2 diferent devices/firmware/implementations for these devices.
I would suggest to duplicate the TuyaCO2ManufCluster implementation and use the new one just for the afected device also duplicating the TuyaCO2ManufCluster.
(That way also will prevent the tests from failing)

okay, but I close this for now, since I do not trust the values at all so much. CO2 stayed around 360 ppm the entire days. (The sensor raises to 500 when breathing against it, so it does react, but having 350-370 the entire day is not really believable).
Need to find out more details of the _TZE200_dwcarsat device first.
(without the patch CO2 is report as 6 ppm and formaldehyde jumps to 40000 though which seems to be complete garbage).

For future reference:
https://github.com/Koenkk/zigbee-herdsman-converters/pull/3982


Chrome/Firefox do not collapse inline-flex correctly

This was just brought up on the mailing list and can be hit already in the wild (which was found by a webdev). Here is the issue, needs a workaround. @tabatkins points out here why they should flex to 0 based on flex-basis defaults.

Interesting.
The linked case seems somewhat unlikely, but I imagine a fair amount of people who don't fully understand the flex shorthand will do things like combine width:100px and flex:1, and not realize that flex:1 implies a flex-basis of 0%.
I suppose the workaround is to not use the flex shorthand with width/height.


Incorrect added mass effect for tapered Morison members in certain orientations

This bug affects version of OpenFAST up through v2.5
This bug report is in case any questions arise about HydroDynamic added mass effects in the affected versions of OpenFAST.  This has been corrected in a development branch that will form the basis of the OpenFAST 3.x release series.  There are no plans to fix this in OpenFAST <3.0 releases.
This bug was also present in FAST v8 releases after 2013.
Bug description
Tapered Morison elements may experience incorrect added mass effects if they are partially aligned with direction of acceleration where one end of the tapered element is higher than the other.  This does not affect tapered elements that are perpendicular to the acceleration.
Cases that are affected will experience a non-zero offset of the resulting reaction force to the acceleration.  The amount of offset appears correlated with the percentage diameter change from largest end to smallest in the member.
This effect is generally very small, but may become apparent in cases with many tapered elements.
Example test case
A single tapered member of 20 m length, tapered from 2 m to 10 m in diameter, is located with ends at (-10, 0 -28)  (narrow end) and (10,0,-26) (wide end).  Only the SimplCa coefficient is set with a value of 2.0.  All other values are set to 0.  A time series acceleration is applied along the global X direction only (all displacements, velocities, and other accelerations are held at 0).
The resulting HydroFxi reaction force the the applied accelerations then appears with an offset in the force values. Tipping the tapered member downwards along X flips the sign of the offset.
Figure: The blue line and red dots illustrate the expected results from the acceleration (generated using the updated HydroDyn that will be included in OpenFAST 3.0).  The green dash-dot line and red dotted lines show the results from OpenFAST v2.5 and earlier for upwards and downwards tilted tapered members.

Sample input files:
Uploading Bug633.zip…

This has been corrected in the branch for PR #537

This has been corrected in the branch for PR #537

Thanks to @jjonkman for debugging help.

Thanks to @jjonkman for debugging help.

Fixed with the merge of PR #537

Fixed with the merge of PR #537


fix: pass cookieOptions with the cacheUerLang

published in<EMAIL_ADDRESS>


[DESIGN] Write an ADR about the UFO fragment

We have work planned on the efficient checking of the restricted fragment of TLA+. This fragment contains uninterpreted first-order logic, which is used, e.g., in Ivy. We should write the design document that explains this fragment and its relation to efficient SMT techniques. This is a summary of the preliminary discussions with @Kukovec:

Who is the user? It is not the systems engineer, but a verification engineer.
What's included:

Boolean connectives,
uninterpreted functions (to Booleans and uninterpreted sorts),
quantifiers over special sets that capture sorts, e.g, \A x \in Values_FOO: P and \E x \in Values_BAR. The set Values_X captures all values of the sort X,
operators and let-in definitions, potentially inlined by the preprocessing steps,
potentially, flat sets as syntax sugar for predicates.

An example of two-phase commit in this fragment: TwoPhaseUFO
This fragment would require us to develop special rewriting rules (which are much simpler than what we have) and write a custom model checker. Alternatively, we could translate specs of this fragment into VMT and use the tools that support this format. For instance, we could run nuxmv or ic3po.
We need an ADR to describe this fragment in more detail and to scope the work in Apalache.

@Kukovec , this is done, right?

@Kukovec shall we close this issue?

This was closed by #1246 I suppose


HOME SCREEN IS BLANK THERE'S NOTHING IN THE VERISON 2.9.0

Bro I downloaded the new update and I faced this problem but there wasn't any problem like this in 2.6.0 it started from 2.9.0 plzzz fix it and yeah please add Hindi dubbed it will help me a lot and also give the option of English dub with English subtitles please I beg you 😭

It started right after 2.6.0 which includes versions since then which were at least 3 including 2.90 so version 2.86, 2.8.8, and 2.89 this may have to do with the type of device being used this app is dedicated to Android TV OS...certain types of budget tv box users it will be a hit and miss using a modified mobile OS

Bro I downloaded the new update and I faced this problem but there wasn't any problem like this in 2.6.0 it started from 2.9.0 plzzz fix it and yeah please add Hindi dubbed it will help me a lot and also give the option of English dub with English subtitles please I beg you 😭

You maybe using a budget chinese tv box the app is not truly optimized for those devices 100%

Bro I downloaded the new update and I faced this problem but there wasn't any problem like this in 2.6.0 it started from 2.9.0 plzzz fix it and yeah please add Hindi dubbed it will help me a lot and also give the option of English dub with English subtitles please I beg you 😭

You maybe using a budget chinese tv box the app is not truly optimized for those devices 100%

Thanks brother for telling me

No bro I use a smart TV (wisdom share) it is a Chinese TV btw

There is many options to try at settings:

Change Source
Change Http Client
Enable/disable DoH

Can you add Hindi dubbing please btw Assalamwalaikum brother are you fasting?

There is many options to try at settings:

Change Source
Change Http Client
Enable/disable DoH

Btw what is these for?

Ramadhan Kareem
Currently source don't have any Hindi dub, don't know about other sources.
I don't running any source, it just fetch/scrapping data from existing anime streaming website.

There is many options to try at settings:

Change Source
Change Http Client
Enable/disable DoH

Btw what is these for?

There is 5 source to select, if one didn't work for you, you can try select other source.
There also 3 httpclient type that may work for you, okhttp, generic http or cronet.
DoH is DNS over https, if your ISP block source DNS, it may fix the problem by resolving domain IP address via https


Allow to configure where "target" folder is created

it will ensure that the folder is written in a safe place.
see https://github.com/jboss-fuse/vscode-atlasmap/issues/85

e.g. -Datlasmap.working.dir


Connecting Circuit Termination to RearPort Issue

Environment

Python version:  3.6
NetBox version:  2.5 beta 1

Steps to Reproduce
Connect circuit termination to a RearPort of a patch panel.  Select patch panel device, select rear_port, select the port instance.

Expected Behavior
A new cable should be created connecting the circuit to the device.

Observed Behavior
I see a new cable is created, but receive the following error:
<class 'django.db.utils.IntegrityError'>

duplicate key value violates unique constraint "dcim_interface__connected_circuittermination_id_key"
DETAIL:  Key (_connected_circuittermination_id)=(126) already exists.

In addition, when looking at the device connected to the corresponding FrontPort, I see a cable connected but there is no far end device.  If I trace the connection, it does properly trace to the circuit.
On a separate/slightly related issue, there is no button to disconnect  a circuit termination from a specific port.  I deleted the cable and was able to reconnect to a new port.  However, the device is currently showing that the old port is still connected to the circuit but does not have a cable attached. (see screenshot)

Connect circuit termination to a RearPort of a patch panel. Select patch panel device, select rear_port, select the port instance.

I'm not able to recreate this. Connecting the cable works as expected. Please see if you can work out the entire workflow needed to reproduce the bug. It likely involves an interface connected at some point in the path.

On a separate/slightly related issue

Needs a separate GitHub issue.

This might be related to #2571

Moved second part of original message to https://github.com/digitalocean/netbox/issues/2572

I am not able to reproduce the issue now.  Not sure if I am following the same steps or if I was trying too many things at once.  🤷♂   I guess this can be closed and I'll see if I can break beta 2 when it comes out.

i run into this issue today, and have a slightly differnt scenario:
i have created a cwdm system with one rear and a lot of front ports. we conneted a circuit (darkfiber in our case) to the rear port.
by adding the first front port, everything looks good.
after adding the second port i get the same error like above.
--> i think there is a 1to1 relationship between circuits and it's connected items. after adding a second front port there is a problem with this object and how it is designed to work.

@jeremystretch should i open a new issue for that, or can you re-open this one for validation?


Change affine transform

affine.yaml requires a 3x3 matrix for the affine transformation. I think the idea was that this may go to higher dimensions in the future. Pyasdf splits this into matrix and translation in order to initialize modeling.AffineTransformation2D which uses the two quantities separately.
I was assuming pyasdf was writing to disk the augmented matrix but it simply attaches the translation part as an additional row to the matrix. This is confusing.
array([[   0.92913257,   -0.36974676,  100.        ],
       [   0.36974676,    0.92913257,   20.        ],
       [   0.        ,    0.        ,    1.        ]])

vs
matrix: !core/ndarray
    data:
    - [0.92913257, -0.36974676, 0.0]
    - [0.36974676, 0.92913257, 0.0]
    - [100.0, 20.0, 0.0]
    datatype: float64
    shape: [3, 3]

In addition,  modeling.AffineTranslation2D is the only option currently that can be used to apply the PC matrix in WCS transformations. However I am only able to write a 3x3 matrix as an affine transformation. This is also confusing because it may mean the data is 3 dimensional in the WCS context.
Any ideas how to resolve this? This is also related to astropy.modeling issue #3548 which would solve this problem if accepted.

Fixed in #80.


chore: bdd-spring-1584635725 to 0.0.1

chore: Promote bdd-spring-1584635725 to version 0.0.1

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approvers:
If they are not already assigned, you can assign the PR to them by writing /assign  in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


give cli a bit more intuitive name.

Good idea to use subparser, Syuan-Ming. I will edit tomorrow.
sent on the move, please excuse typos
On Mon, Apr 27, 2020, 4:42 PM Bryant<EMAIL_ADDRESS>wrote:

@bryantChhun approved this pull request.
I have no opinion regarding using
parser.add_mutually_exclusive_group
vs
parser.add_subparsers  # as per Syuan-Ming's suggestion
I might lean towards subparsers if only because it enables us to run both
pipelines (analysis and interpretation) in one command.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/czbiohub/pysero/pull/51#pullrequestreview-401402018,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAWMLJZRL7CAKHLXDJBZNDDROYJ5BANCNFSM4MQIASCA
.

Thanks for the ideas. I tried the subparser, but didn't get the right behavior (See issue #55). Merging this PR now. @jennyfolkesson  please merge the PR on unittests next.


GROOVY-10898: Provide tail/init for primitive arrays

Codecov Report

Merging #1842 (81ed389) into master (8f80995) will decrease coverage by 0.0173%.
The diff coverage is 33.3333%.

@@                Coverage Diff                 @@
##               master      #1842        +/-   ##
==================================================
- Coverage     67.7474%   67.7300%   -0.0173%
- Complexity      28300      28316        +16
==================================================
  Files            1427       1427
  Lines          112611     112659        +48
  Branches        19302      19302
==================================================
+ Hits            76291      76304        +13
- Misses          29795      29812        +17
- Partials         6525       6543        +18

Impacted Files
Coverage Δ

...rg/codehaus/groovy/runtime/ArrayGroovyMethods.java
51.8625% <33.3333%> (-2.9548%)
:arrow_down:

...vy/util/concurrent/ConcurrentReferenceHashMap.java
35.2349% <0.0000%> (-0.5034%)
:arrow_down:


Create test-random_init.py

to test gym.make("fishing-v1",init="random")

Thanks @Pierre-Cortambert . Can you run make format to fix the formatting issues (see above on the CI check).  You might take a look at some of the recommendations on Codacy Static Code analysis too.  (These are mostly stylistic rather than functional things, but they do make it easier to maintain code in the long run).
I'll try and take a closer look at the implementation soon.  nice work here.


Check for captureStackTrace in WError

https://github.com/davepacheco/node-verror/issues/18

Thanks for this, and sorry for the slow response.  This issue (along with other inconsistencies between VError and WError) should be fixed by my major refactoring in #10.  I'm going to close this PR for now, but leave #18 open until the other change gets merged.  Thanks!


Spawning Issue

I have the latest version of Minetest.  When I enable this Dragons and Wyverns spawn insanely everywhere.  Within a minute 15+ of them are everywhere.  I have tried reinstalling it several times incase there was a glitch.  Not sure what is happening.  I have used this mod plenty of times before this has never happened till I started using the current version of game.

I have the exact same issue, the mod is unplayable due to this issue...

same


chore(deps): upgrade dependencies

Upgrades project dependencies. See details in workflow run.

Automatically created by projen via the "upgrade-k8s-22-main" workflow

⚪ Backport skipped
The pull request was not backported as there were no branches to backport to. If this is a mistake, please apply the desired version labels or run the backport tool manually.
Manual backport
To create the backport manually run:
backport --pr 1238

Questions ?
Please refer to the Backport tool documentation


3D Convolution

Hi!
I am trying to run a 3D Convolution but I am getting an error that IMOpen only supports 4D Tensors, Not 5D which I need. But this list  says that Conv3D is implemented in ROCm.
Could someone please explain to me why it doesn’t work and how (if possible) I can fix it.
Yours,
Max

@maxzoech the list is a bit misleading that basically it means operators in those files are successfully built and registered with TensorFlow ROCm port so it can be used from Python. But to get the the operators running properly we need StreamExecutor to pick up the right APIs in MIOpen. And thus far we are only using 2D convolution algorithms in MIOpen from TensorFlow at this moment, hence you are experiencing this issue.
@zjing14 knows better about the status of 3D convolution algorithms in MIOpen. @zjing14 , could you help weigh in here?

Hi @maxzoech , could you ask the question in MIOpen repo?
https://github.com/ROCmSoftwarePlatform/MIOpen

Hi @daniellowell , could you update the status of 3D convolution algorithms in MIOpen?

Yes any update on this?

I opened an issue about it on MIOpen 7 months ago. Also looking forward to hearing back about this tatus.
https://github.com/ROCmSoftwarePlatform/MIOpen/issues/46

3D convolutions are slated for MIOpen 1.8 release.


Potential typo in SimpleMessageListenerContainer when logging exceptions?

Hello.
We recently experienced RabbitMQ outage and got the following logs from our RabbitMQ listener:

ERROR 6 --- [ntContainer#3-5] o.s.a.r.l.SimpleMessageListenerContainer : Consumer received fatal=false exception on startup
org.springframework.amqp.rabbit.listener.QueuesNotAvailableException: Cannot prepare queue for listener. Either the queue doesn't exist or the broker will not allow us to use it.
ERROR 6 --- [ntContainer#3-5] o.s.a.r.l.SimpleMessageListenerContainer : Stopping container from aborted consumer

And while investigating issues with our RabbitMQ configuration, we got really confused why container was stopped if it says fatal=false. Further investigation lead us to this code in org.springframework.amqp.rabbit.listener.SimpleMessageListenerContainer:
catch (QueuesNotAvailableException ex) {
    logger.error("Consumer received fatal=" + isMismatchedQueuesFatal() + " exception on startup", ex);
    if (isMissingQueuesFatal()) {
        this.startupException = ex;
        // Fatal, but no point re-throwing, so just abort.
        aborted = true;
    }
    publishConsumerFailedEvent("Consumer queue(s) not available", aborted, ex);
}

For logging this code checks isMismatchedQueuesFatal() method, but for actual logic it uses isMissingQueuesFatal().
Searching for other usages of isMismatchedQueuesFatal() shows that it's mainly used when configuring listener and declaring queues. The only other place where its usage is not configuration-related is the snippet above.
Is this intentional or just a typo?

Yes; it's a typo; thanks for reporting - but it's also wrong to just log the value; the error should explicitly refer to the missingQueuesFatal property.


'FramebufferTexture' is not exported from 'three'

Awesome package! Just a heads up, when trying to use the component, I get the error:
./node_modules/three-stdlib/objects/Lensflare.js
Attempted import error: 'FramebufferTexture' is not exported from 'three'.

Thanks! Fixed in 5.2.0. See:
https://twitter.com/onirenaud/status/1495407672581300225
Changelog will follow


Added documentation for local golangci-lint use.

Changes
Added documentation for local golangci-lint use.
Contributes toward #5899
/kind documentation
Submitter Checklist
As the author of this PR, please check off the items in this checklist:

[x] Has Docs included if any changes are user facing
[N/A] Has Tests included if any functionality added or changed
[x] Follows the commit message standard
[x] Meets the Tekton contributor standards (including
functionality, content, code)
[x] Has a kind label. You can add one by adding a comment on this PR that contains /kind <type>. Valid types are bug, cleanup, design, documentation, feature, flake, misc, question, tep
[N/A] Release notes block below has been updated with any user facing changes (API changes, bug fixes, changes requiring upgrade notices or deprecation warnings)
[N/A] Release notes contains the string "action required" if the change requires additional action from users switching to the new release

Release Notes
NONE

/test pull-tekton-pipeline-alpha-integration-tests
Flake: TestExamples/v1beta1/pipelineruns/workspace-from-volumeclaimtemplate

thank you!
/lgtm


update to list of past presentations

added 4 presentations in the list
titles still missing, awaiting slides from presenters, will update further based on those.

merge now or wait for titles?

Will merge, can add titles later.


Remove space between [] and () in links

Spacing between [] and () prevents commonmark based parsers, like GitHub flavored markdown, from being able to identify text as a link.
This removes the space to ensure it will be parsed correctly.
Also normalize line endings to LF.

Contributor License Agreement adherence:

[x] This Contribution is under the terms of Individual Contributor License Agreements (and also Corporate Contributor License Agreements to the extent applicable) appearing in the Apereo CLA roster.

Changes Unknown when pulling a23a21417771522221ac91c28c35faa258babecd on ChristianMurphy:markdown-links into ** on UW-Madison-DoIT:master**.


Pretraining dataset and code request

Will the pretraining datasets and corresponding code be open-sourced?
Thanks!

Hi, thanks for the great work. I'd also be interested particularly in training code or at least if you can share some multi-node settings details. What tech did you use for parallelization across GPU nodes?

Same.
Looking forward to the open-source training code and details.

Thanks for the amazing work and the paper.  Really would love to explore your training code.

+1

https://huggingface.co/jetmoe/jetmoe-8b/discussions/5


Gradle jacoco coverage report with more than one submodule(s)?

Currently JacocoEverywhere merges jacoco reports from different tests in the same submodule. i.e.

myProject/src/main
myProject/src/test
myProject/src/androidtest

Is it possible to have the report over the tests of different sub-moduls as described in
http://stackoverflow.com/questions/38675853/gradle-jacoco-coverage-report-with-more-than-one-submodules

myProject1/src/main
myProject1/src/test
myProject2/src/main
myProject2/src/test

I alredy tried as decribed in https://discuss.gradle.org/t/merge-jacoco-coverage-reports-for-multiproject-setups/12100/6
task jacocoMerge(type: JacocoMerge) {
   subprojects.each { subproject ->
	  executionData subproject.tasks.withType(Test)
   }
}

but got the error message (using gradle-2.14)
* What went wrong:
Some problems were found with the configuration of task ':jacocoMerge'.
> No value has been specified for property 'jacocoClasspath'.
> No value has been specified for property 'executionData'.
> No value has been specified for property 'destinationFile'.

Seems like you found your answer already according to http://stackoverflow.com/questions/38675853/gradle-jacoco-coverage-report-with-more-than-one-submodules


ColorTool: Fix the included all.bat batch file

all.bat doesn't work if you use build.bat to build the program, as all.bat looks for ct.exe rather than colortool.exe. Fortunately, fixing the batch file is almost as easy as working around its bug manually.

@metathinker,
Thanks for having already signed the Contribution License Agreement. Your agreement has not been validated yet. We will now review your pull request.
Thanks,
Microsoft Pull Request Bot

Ah, good catch! Thought I fixed this before we released but looks like not. Thanks!


mget in the cluster performance issue on the client side

We have a use case that we want to mget 100 keys in the cluster use RedisAdvancedClusterAsyncCommandsImpl.mget().
When we do stress tests on a single virtual machine with 4 cpu, more than 100 call mget(100...keys) times per second, vm cpu idle will down fastly and context switching high. Program run on the edge of crash.

As the official doc(https://github.com/lettuce-io/lettuce-core/wiki/Redis-Cluster) says, regular Redis Cluster commands are limited to single-slot keys operation, mget 100 keys will be dispersed 100 slots approximately, and then send commands to redis server concurrently by netty NIO, one context switching for sending command, next context switching for receiving response.
Will I give up use redis cluster mode in this use case, and use client side partitioning ?

Please provide a minimal complete verifiable example so we can trace what's happening here.

ExecutorService executors = Executors.newFixedThreadPool(4);

List<RedisURI> urls = new ArrayList<>();
urls.add(RedisURI.create("<IP_ADDRESS>", 7000));
urls.add(RedisURI.create("<IP_ADDRESS>", 7001));
urls.add(RedisURI.create("<IP_ADDRESS>", 7000));
urls.add(RedisURI.create("<IP_ADDRESS>", 7001));
RedisClusterClient client = RedisClusterClient.create(urls);
RedisAdvancedClusterAsyncCommands commands = client.connect().async();

List<List<String>> groupKeys = ... list of 100 str list

for (List<String> keys : groupKeys) {
    executors.submit(new Task(keys));
}

private class Task implements Runnable {
    private List<String> keys;
    Task(List<String> keys) {
        this.keys = keys;
    }
    @Override
    public void run() {
        try {
            RedisFuture<List<KeyValue<String, String>>> future = commands.mget(keys.toArray(new String[keys.size()]));
            List<KeyValue<String, String>> result = future.get();
        } catch (Exception e) {
             e.printStackTrace();
        }
    }
}

@mp911de thanks for your reply, I feedback example

According to the code here all is as it is expected. The JVM properly uses resources because of Lettuce's non-blocking I/O layer. If you find issues that we can streamline CPU usage with more light-weight activity (i.e. preventing unnecessary actions), I'm happy to assist you.
As of now, there's nothing left to do.

Hi @LinGoWei Were you able to RCA this?
We are facing similar issue, mget on 100 keys using lettuce mget RedisAdvancedClusterCommands<K,V>
Redis is receiving the commands but not able to fulfil in 500ms also and leading to timeout at client end followed by CB open.
Kindly let me know how you resolved it. Note: in our case, this is intermittent

E-mail 已收到啦，谢谢！


artichoke-dev is shipping Apple Silicon binaries

Hi folks! 💎 Artichoke Ruby nightlies (available in ruby-build as artichoke-dev) are now building aarch64-apple-darwin (aka Apple Silicon) artifacts: https://github.com/artichoke/nightly/pull/17.
Example release: https://github.com/artichoke/nightly/releases/tag/test-2021-01-02-apple-silicon-2
Artifact name: artichoke-nightly-aarch64-apple-darwin.tar.gz
I'm not sure how to add this artifact to ruby-build. cc @deepj since you added artichoke-dev to ruby-build.
Another thing I'm unsure of is whether distributing these binaries through ruby-build will work as is or if I need to setup code signing of Mac artifacts first: https://github.com/artichoke/nightly/issues/18.

 I'll link back here once a scheduled build completes.

 I'll link back here once a scheduled build completes.

Today's scheduled nightly build succeeded: https://github.com/artichoke/nightly/releases/tag/nightly-2021-01-03.
The Apple Silicon artifact is: https://github.com/artichoke/nightly/releases/download/nightly-2021-01-03/artichoke-nightly-aarch64-apple-darwin.tar.gz. This artifact has the same contents as the existing x86_64 Mac and Linux artifacts.

Today's scheduled nightly build succeeded: https://github.com/artichoke/nightly/releases/tag/nightly-2021-01-03.
The Apple Silicon artifact is: https://github.com/artichoke/nightly/releases/download/nightly-2021-01-03/artichoke-nightly-aarch64-apple-darwin.tar.gz. This artifact has the same contents as the existing x86_64 Mac and Linux artifacts.

@lopopolo Here my proposal https://gist.github.com/deepj/3c1f85893ef16e618690b6959ef7fa5c
If @hsbt is OK with that I can make a PR.
ARM
$ uname -m
arm64
$ ruby-build artichoke-dev ~/.rubies/artichoke-dev
Downloading artichoke-nightly-aarch64-apple-darwin.tar.gz...
-> https://github.com/artichoke/nightly/releases/latest/download/artichoke-nightly-aarch64-apple-darwin.tar.gz
Installing artichoke-nightly...
Installed artichoke-nightly to ~/.rubies/artichoke-dev

$ ~/.rubies/artichoke-dev/bin/ruby --version
artichoke 0.1.0-pre.0
$ ~/.rubies/artichoke-dev/bin/ruby -e "puts RUBY_PLATFORM"
aarch64-apple-darwin

Under x86_64
$ uname -m
x86_64
$ ruby-build artichoke-dev ~/.rubies/artichoke-dev
Downloading artichoke-nightly-x86_64-apple-darwin.tar.gz...
-> https://github.com/artichoke/nightly/releases/latest/download/artichoke-nightly-x86_64-apple-darwin.tar.gz
Installing artichoke-nightly...
Installed artichoke-nightly to ~/.rubies/artichoke-dev

$ ~/.rubies/artichoke-dev/bin/ruby --version
artichoke 0.1.0-pre.0
$ ~/.rubies/artichoke-dev/bin/ruby -e "puts RUBY_PLATFORM"
x86_64-apple-darwin

@lopopolo Here my proposal https://gist.github.com/deepj/3c1f85893ef16e618690b6959ef7fa5c
If @hsbt is OK with that I can make a PR.
ARM
$ uname -m
arm64
$ ruby-build artichoke-dev ~/.rubies/artichoke-dev
Downloading artichoke-nightly-aarch64-apple-darwin.tar.gz...
-> https://github.com/artichoke/nightly/releases/latest/download/artichoke-nightly-aarch64-apple-darwin.tar.gz
Installing artichoke-nightly...
Installed artichoke-nightly to ~/.rubies/artichoke-dev

$ ~/.rubies/artichoke-dev/bin/ruby --version
artichoke 0.1.0-pre.0
$ ~/.rubies/artichoke-dev/bin/ruby -e "puts RUBY_PLATFORM"
aarch64-apple-darwin

Under x86_64
$ uname -m
x86_64
$ ruby-build artichoke-dev ~/.rubies/artichoke-dev
Downloading artichoke-nightly-x86_64-apple-darwin.tar.gz...
-> https://github.com/artichoke/nightly/releases/latest/download/artichoke-nightly-x86_64-apple-darwin.tar.gz
Installing artichoke-nightly...
Installed artichoke-nightly to ~/.rubies/artichoke-dev

$ ~/.rubies/artichoke-dev/bin/ruby --version
artichoke 0.1.0-pre.0
$ ~/.rubies/artichoke-dev/bin/ruby -e "puts RUBY_PLATFORM"
x86_64-apple-darwin

@lopopolo there is a new release of ruby-build containing the option to install a native version Artichoke Ruby on Apple Silicon's Macs

@lopopolo there is a new release of ruby-build containing the option to install a native version Artichoke Ruby on Apple Silicon's Macs


cellxgene-schema must perform basic 2.0.0 -> 3.0.0 migration

To make it easier for Lattice and/or the Data Portal to migrate datasets from 2.0.0 to 3.0.0:
The cellxgene-schema CLI should have an option to update the datasets prior to re-validation:

Rename ethnicity_term_ontology_id to self_reported_ethnicity_term_ontology_id
Rename ethnicity to self_reported_ethnicity
Delete all deprecated fields in the table below:

Reserved Name
AnnData
Deprecated in

X_normalization
uns
3.0.0

default_field
uns
2.0.0

layer_descriptions
uns
2.0.0

tags
uns
2.0.0

version
uns
2.0.0

contributors
uns
1.1.0

preprint_doi
uns
1.1.0

project_description
uns
1.1.0

project_links
uns
1.1.0

project_name
uns
1.1.0

publication_doi
uns
1.1.0

Set suspension_type based on the corresponding assay_ontology_term_id if possible - some assays support multiple choices.
Update schema_version to "3.0.0"

I think 3. Delete X_normalization should be updated to
3. Delete any deprecated field (#264 )

@jahilton - so you're comfortable if migration deletes a publication-doi in uns for example? Just saying that it's a sharp tool ...

Yes, I'm actually counting on it.

I update the top-level summary comment to include that requirement.

documenting one more feature to include, from Jason:

Would it be possible to include an ontology update function, as well?
For the cases where we know there is a direct replacement for a deprecated term
As of now, there are 3 cases in 2-to-3 migration
obs.disease - MONDO:0008345 --> MONDO:0800029
obs.cell_type - CL:0002609 --> CL:0010012
obs.cell_type - CL:0011107 --> CL:0000636

For suspension_type column, it would be great to have the following behavior and logging to help track what was done by the "convert" command:

If suspension_type column is already in obs, leave it in the object and report what the categories currently are for this column
If suspension_type is not present in obs, add suspension_type for assays that have only one possible value and report the value that was added
Else, if NaN was added for suspension_type, report this as well

Logging looks good! Thank you! Closing ticket.

Test Case
Expected Result
Obtained Result
Test File

no suspension_type column for 10x 3' v2 and v3
Logging and add suspension_type nan
column 'suspension_type' not found in obs. Adding column and auto-assigning value where possible.Dataset contains row(s) with assay_ontology_term_id EFO:0009899. Automatically assigned suspension_type nanDataset contains row(s) with assay_ontology_term_id EFO:0009922. Automatically assigned suspension_type nan
heart_v3_nosus.h5ad

suspension_type 'cell' and 'nucleus' for 10x 3' v2 and v3
Logging
suspension_type already exists in obs, with categories Index(['cell', 'nucleus'], dtype='object')
heart_cell_nucleus_v3.h5ad

no suspension_type column for spatial
Logging and add suspension_type na
column 'suspension_type' not found in obs. Adding column and auto-assigning value where possible.Dataset contains row(s) with assay_ontology_term_id EFO:0010961. Automatically assigned suspension_type na
heart_spatial_v3.h5ad

no suspension_type column for DroNc-seq
Logging and add suspension_type "nucleus"
column 'suspension_type' not found in obs. Adding column and auto-assigning value where possible.Dataset contains row(s) with assay_ontology_term_id EFO:0008720. Automatically assigned suspension_type nucleus
heart_droncseq_v3.h5ad

no suspension_type column for DroNc-seq and Drop-seq
Logging and add suspension_type "nucleus" or "cell"
column 'suspension_type' not found in obs. Adding column and auto-assigning value where possible.Dataset contains row(s) with assay_ontology_term_id EFO:0008720. Automatically assigned suspension_type nucleusDataset contains row(s) with assay_ontology_term_id EFO:0008722. Automatically assigned suspension_type cell
heart_droncseq_dropseq_v3.h5ad


Add possibilty to add two Volvo Accounts

Me and my family are having two Volvo cars and would be lovely to have the opportunity to add two accounts. Is this maybe possible already?

No sorry. If you are using docker, just run the addon twice.

@Dielee Would it be also possible to fork your repo and then install it twice as HA AddOn?


Adapt for events

Coverage decreased (-0.05%) to 88.123% when pulling 99b0b8ab3caa04861913b0149e9e45be06cb36b1 on feature/event-tracking into 951073da0ad0ec18f65812be712092b9aeb864fb on master.


"pip install scons" when NOT using virtualenv results in a broken installation

Based on a discussion in the SCons mailing list
https://pairlist4.pair.net/pipermail/scons-users/2018-October/007321.html
I would like to report an issue with Scons, when being installed with pip, not using a virtual environment.

SCons version: 3.0.1
Python version: 2.7.14
Python distribution: Manually compiled from source code downloaded on Python.org
Installation procedure: pip install scons
Plattform: CentOS Linux release 7.5.1804 (Core)
How to reproduce the issue issue?:
Issue can be reproduced by running the 2 commands below:

pip install scons
scons

[sfux at lo-login-02 ~]$ pip install scons
Collecting scons
Installing collected packages: scons
Successfully installed scons-3.0.1
[sfux at lo-login-02 ~]$ scons
SCons import failed. Unable to find engine files in:
  /cluster/apps/python/2.7.14_cpu/bin/../engine
  /cluster/apps/python/2.7.14_cpu/bin/scons-local-3.0.1
  /cluster/apps/python/2.7.14_cpu/bin/scons-local
  /cluster/apps/python/2.7.14_cpu/lib64/python2.7/site-packages/lib/scons-3.0.1
  /cluster/apps/python/2.7.14_cpu/lib/scons-3.0.1
  /cluster/apps/python/2.7.14_cpu/lib/scons-3.0.1
  /cluster/apps/python/2.7.14_cpu/lib64/python2.7/site-packages/lib/python2.7/site-packages/scons-3.0.1
  /cluster/apps/python/2.7.14_cpu/lib/python2.7/site-packages/scons-3.0.1
  /cluster/apps/python/2.7.14_cpu/lib/python2.7/site-packages/scons-3.0.1
  /cluster/apps/python/2.7.14_cpu/lib64/scons-3.0.1
  /cluster/apps/python/2.7.14_cpu/lib64/python2.7/site-packages/lib/scons
  /cluster/apps/python/2.7.14_cpu/lib/scons
  /cluster/apps/python/2.7.14_cpu/lib/scons
  /cluster/apps/python/2.7.14_cpu/lib64/python2.7/site-packages/lib/python2.7/site-packages/scons
  /cluster/apps/python/2.7.14_cpu/lib/python2.7/site-packages/scons
  /cluster/apps/python/2.7.14_cpu/lib/python2.7/site-packages/scons
  /cluster/apps/python/2.7.14_cpu/lib64/scons
Traceback (most recent call last):
  File "/cluster/apps/python/2.7.14_cpu/bin/scons", line 192, in <module>
    import SCons.Script
ImportError: No module named SCons.Script
[sfux at lo-login-02 ~]$

How you invoke scons ? just running scons without specifying any option

Analysis of the problem and proposal of a solution:
SCons searches it own files in
$PYTHONHOME/lib64/python2.7/site-packages/lib/scons-3.0.1
instead of
$PYTHONHOME/lib64/python2.7/site-packages/scons-3.0.1
One workaround is to set $SCONS_LIB_DIR to $PYTHONHOME/lib64/python2.7/site-packages/scons-3.0.1, but it would be better to solve the root cause of the problem.
It arises from $PYTHONHOME/bin/scons. The "/lib" is added on line 157 of the script:
temp = [os.path.join(x, 'lib') for x in prefs]
If I change this line to
temp = [os.path.join(x, '') for x in prefs]
then the error does no longer occur.
I will use this workaround as it avoids setting environment variables. It is clear to me, that this only fixes the problem on our system, and maybe for other sites there is a different solution required.
The guessing of the paths is in general difficult,  since it is OS dependent. Would using site package might be a good idea for getting the installation directory of the SCons module ?
[sfux@lo-login-01 ~]$ python
Python 2.7.14 (default, Apr 10 2018, 16:12:16)
[GCC 4.8.5 20150623 (Red Hat 4.8.5-16)] on linux2
Type "help", "copyright", "credits" or "license" for more information.
>>> import site
>>> print site.PREFIXES
['/cluster/apps/python/2.7.14_cpu', '/cluster/apps/python/2.7.14_cpu']
>>> print site.USER_SITE
/cluster/home/sfux/.local/lib64/python2.7/site-packages
>>> print site.getsitepackages()
['/cluster/apps/python/2.7.14_cpu/lib64/python2.7/site-packages', '/cluster/apps/python/2.7.14_cpu/lib64/site-python']
>>> print site.getusersitepackages()
/cluster/home/sfux/.local/lib64/python2.7/site-packages

instead of using sys.prefix with some additional guessing ?
        prefs.append(sys.prefix)

    temp = [os.path.join(x, '') for x in prefs]
    temp.extend([os.path.join(x,
                                           'lib',
                                           'python' + sys.version[:3],
                                           'site-packages') for x in prefs])

Could you please consider fixing this in a future SCons release, as it seems that many people are facing this problem ?

https://github.com/FPGAwars/apio/issues/143
https://github.com/Homebrew/legacy-homebrew/issues/5794
https://stackoverflow.com/questions/24897687/import-failed-unable-to-find-scons-files-import-scons-script
http://scons.1086193.n5.nabble.com/SCons-can-t-correctly-install-via-pip-when-not-in-a-virtualenv-td42715.html
https://pairlist4.pair.net/pipermail/scons-users/2017-December/006589.html

Thank you.
Best regards
Sam

Please try a package I just built from master and uploaded to testpypi
pip install --index-url https://test.pypi.org/simple/ scons==3.1.0.alpha.20181011
This issue is fixed in master.
It will be resolved with the next release.
The patch for this was merged in May
https://github.com/SCons/scons/pull/3134

Still the same issue:
[sfux@lo-login-01 ~]$ pip install --index-url https://test.pypi.org/simple/ scons==3.1.0.alpha.20181011
Looking in indexes: https://test.pypi.org/simple/
Collecting scons==3.1.0.alpha.20181011
Installing collected packages: scons
Successfully installed scons-3.1.0a20181011
[sfux@lo-login-01 ~]$ scons
SCons import failed. Unable to find engine files in:
/cluster/apps/python/2.7.14/bin/../engine
/cluster/apps/python/2.7.14/bin/scons-local-3.1.0.alpha.20181011
/cluster/apps/python/2.7.14/bin/scons-local
/cluster/apps/python/2.7.14/lib64/python2.7/site-packages/lib/scons-3.1.0.alpha.20181011
/cluster/apps/python/2.7.14/lib/scons-3.1.0.alpha.20181011
/cluster/apps/python/2.7.14/lib/scons-3.1.0.alpha.20181011
/cluster/apps/python/2.7.14/lib64/python2.7/site-packages/lib/python2.7/site-packages/scons-3.1.0.alpha.20181011
/cluster/apps/python/2.7.14/lib/python2.7/site-packages/scons-3.1.0.alpha.20181011
/cluster/apps/python/2.7.14/lib/python2.7/site-packages/scons-3.1.0.alpha.20181011
/cluster/apps/python/2.7.14/lib64/scons-3.1.0.alpha.20181011
/cluster/apps/python/2.7.14/lib64/python2.7/site-packages/lib/scons
/cluster/apps/python/2.7.14/lib/scons
/cluster/apps/python/2.7.14/lib/scons
/cluster/apps/python/2.7.14/lib64/python2.7/site-packages/lib/python2.7/site-packages/scons
/cluster/apps/python/2.7.14/lib/python2.7/site-packages/scons
/cluster/apps/python/2.7.14/lib/python2.7/site-packages/scons
/cluster/apps/python/2.7.14/lib64/scons
Traceback (most recent call last):
File "/cluster/apps/python/2.7.14/bin/scons", line 192, in
import SCons.Script
ImportError: No module named SCons.Script
[sfux@lo-login-01 ~]$

Whether or not using non-versioned install locations improves the situation, scons itself (src/script/scons.py) is still generating a bunch of dodgy paths that will never match, and should be cleaned up. The single most promising path to try if you are not running a local install or source copy is the one that comes from a query of the Python packaging support. For the example in the previous comment, that path is /cluster/apps/python/2.7.14/lib64/python2.7/site-packages and if suffixed by /scons and /scons-version would have almost certainly succeeded. Instead it has been transformed into these four failing paths from the listing, none of which would work on any installation I have ever seen, that is, trying to use a lib subdirectory of the site-packages directory.
/cluster/apps/python/2.7.14/lib64/python2.7/site-packages/lib/scons-3.1.0.alpha.20181011
/cluster/apps/python/2.7.14/lib64/python2.7/site-packages/lib/python2.7/site-packages/scons-3.1.0.alpha.20181011
/cluster/apps/python/2.7.14/lib64/python2.7/site-packages/lib/scons
/cluster/apps/python/2.7.14/lib64/python2.7/site-packages/lib/python2.7/site-packages/scons

I can easily supply a PR to take care of that issue, it just involves reserving that first path  and not subjecting it to one of the transformations.

PR #3218 proposes a new scons.py. I didn't set it up to reference this issue since technically it's closed already, but it would be useful to know if it helps this situation.


⚠ Bofrost.de has degraded performance

In bbbb629, Bofrost.de (https://www.bofrost.de) experienced degraded performance:

HTTP code: 200
Response time: 1573 ms

Resolved: Bofrost.de performance has improved in 7237c6f after 9 minutes.


Update docs to cover new pipeline config

Overview/Summary
Update documentation for setting up Azure Pipelines
This PR fixes/adds/changes/removes

Adds script to assign RBAC role to service principal
Moved links to docs under Prerequisites to a Further Reading section
Changed order between "Configure in Portal" and "Configure with script"
Adds script for setting up everything using Azure CLI and PowerShell
Moved and merged table for settings to dedicated file (added link)
Changed some wording and formatting

Related to #459
Partial conflict with #494
As part of this Pull Request I have

[x] Checked for duplicate Pull Requests
[x] Associated it with relevant issues, for tracking and closure.
[x] Ensured my code/branch is up-to-date with the latest changes in the main branch
[x] Performed testing and provided evidence.
[x] Updated relevant and associated documentation.
[x] Updated the "What's New?" wiki page (located in the Enterprise-Scale repo in the directory: /docs/wiki/whats-new.md)
Prepared in fork

Sorry for the force push, realized I left a hardcoded value in there, fixed and restructured the permissions part to make it more clear which GUIDs are well-known and "global" and which are customer specific.


[PDI-16890]-UI Glitch on tab name

added configurable parameter of max length of tab

Build Completed
:fire: This pull request has some issues. It would be preferable to fix them in order for it to be just perfect. See below for more details. Some links are also available below for further assistance in addressing those issues.
Build Commands
mvn -B -fn -f 'pom.xml' -pl 'ui,engine,core' -P '!assemblies' -Dsurefire.runOrder=alphabetical -Daudit -amd clean install

Cleanup Commands
mvn -B -f 'pom.xml' -pl 'ui,engine,core' -P '!assemblies' -amd build-helper:remove-project-artifact

Changed files
  core/src/main/java/org/pentaho/di/core/Const.java
  engine/src/main/resources/kettle-variables.xml
  ui/src/main/java/org/pentaho/di/ui/spoon/delegates/SpoonDelegate.java
  ui/src/main/java/org/pentaho/di/ui/spoon/delegates/SpoonJobDelegate.java
  ui/src/main/java/org/pentaho/di/ui/spoon/delegates/SpoonTransformationDelegate.java

Unit Test Coverage
These statistics help you identify how your changes have affected the coverage of the following files. If a file is not in this list, then its coverage was not affected by your changes. To get some help interpreting these metrics, please refer to Jacoco's documentation.

:warning: Coverage Changes (click to expand)
org.pentaho.di.job.entries.sql.JobEntrySQL

Instruction Change: -3.4% :small_red_triangle_down:
Line Change: -3.8% :small_red_triangle_down:

org.pentaho.di.trans.step.StepMeta

Branch Change: -0.7% :small_red_triangle_down:
Complexity Change: -0.6% :small_red_triangle_down:
Instruction Change: -0.2% :small_red_triangle_down:
Line Change: -0.3% :small_red_triangle_down:

org.pentaho.di.trans.steps.constant.ConstantMeta

Instruction Change: -0.1% :small_red_triangle_down:

org.pentaho.di.trans.steps.csvinput.CsvInputMeta

Instruction Change: -0.1% :small_red_triangle_down:

org.pentaho.di.trans.steps.dbproc.DBProcMeta

Instruction Change: -0.1% :small_red_triangle_down:

org.pentaho.di.trans.steps.memgroupby.MemoryGroupByMeta

Branch Change: -4.4% :small_red_triangle_down:
Complexity Change: -5.1% :small_red_triangle_down:
Instruction Change: -0.3% :small_red_triangle_down:
Line Change: -0.9% :small_red_triangle_down:

org.pentaho.di.trans.steps.replacestring.ReplaceStringMeta

Instruction Change: -0.1% :small_red_triangle_down:

org.pentaho.di.trans.steps.symmetriccrypto.symmetriccryptotrans.SymmetricCryptoTransMeta

Branch Change: -3.3% :small_red_triangle_down:
Complexity Change: -1.8% :small_red_triangle_down:
Instruction Change: -0.3% :small_red_triangle_down:

org.pentaho.di.trans.steps.tableoutput.TableOutputMeta

Branch Change: -0.9% :small_red_triangle_down:
Complexity Change: -0.8% :small_red_triangle_down:

org.pentaho.di.trans.steps.textfileinput.TextFileInputMeta

Branch Change: -0.5% :small_red_triangle_down:

org.pentaho.di.trans.steps.gpload.GPLoadMeta

Branch Change: -0.9% :small_red_triangle_down:
Complexity Change: -0.8% :small_red_triangle_down:
Instruction Change: -0.2% :small_red_triangle_down:

org.pentaho.di.ui.repository.pur.repositoryexplorer.model.UISecurityRole

Branch Change: -1.8% :small_red_triangle_down:
Complexity Change: -1.5% :small_red_triangle_down:
Instruction Change: -0.2% :small_red_triangle_down:
Line Change: -0.5% :small_red_triangle_down:

org.pentaho.di.ui.spoon.delegates.SpoonDelegate

Complexity Change: -33.3% :small_red_triangle_down:
Instruction Change: -21.7% :small_red_triangle_down:
Line Change: -16.7% :small_red_triangle_down:
Method Change: -33.3% :small_red_triangle_down:

org.pentaho.di.ui.spoon.delegates.SpoonJobDelegate

Branch Change: -0.1% :small_red_triangle_down:
Complexity Change: -0.1% :small_red_triangle_down:

org.pentaho.di.ui.spoon.delegates.SpoonTransformationDelegate

Branch Change: -0.1% :small_red_triangle_down:
Complexity Change: -0.1% :small_red_triangle_down:
Instruction Change: -0.1% :small_red_triangle_down:

Note: The list of offending coverage changes extends beyond the presented results and was truncated to prevent reaching the comment size limit.


Add query commands for groups, workbooks, datasources, projects

It's annoying to have to look up a workbook/etc name somewhere else when I'm trying to run a command in tabcmd.
We do have listsites and listdomains as standalone commands already. Following that pattern would give us
listprojects --project --parent-project-path
listworkbooks --project
listdatasources --project [?--type? what would the output show?]
listgroups [?--user?]
listusers --group
? any others?
@smarvin what do you think about those as potential commands/arguments?

This is implemented as tabcmd list x


How to add Backdrop - Example code?

Hello together,
I tried many different ways to add a Backdrop to the RN Action Button. But I got some Problems with the Type "Object" etc.
Could someone help me with some Example Code for a blurred Backdrop?
Thanks and regards
Alex

Hey, did you try to give the ActionButton component a prop like this:
<ActionButton backdrop={<VibrancyView blurType="light" style={styles.blur} />}> ...
styles.blur needs to have the width and height of the device in order to blur the background.

Hey,
thanks for that fast Tipp. Now i do not get any Message, BUT instead of having the blurred look, i only see a red border around my complete screen, when clicking the ActionButton.
Any Idea, how that might come.

Nevermind I have the Solution :) THANKS SUPER GREAT ANSWER 👍

@alexanderniebuhr Can you share your solution please? :) I am getting the same red border...

I think he forgot to link the library (react-native-blur) to his project, if I remember correctly
On Fri, Aug 26, 2016 at 11:34 AM +0200, "nirshubz"<EMAIL_ADDRESS>wrote:
@alexanderniebuhr Can you share your solution please? :) I am getting the same red border...
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or mute the thread.

For some reason I am still getting Native component for "VibrancyView" does not exist after using rnpm link, but I ll look into it.
Thanks very much!

I tried a different approach instead of using react-native-blur library. I wrote a simple component with absolute positioning, black background, opacity tweaks and used it as backdrop.
Component :
import React,{Component} from 'react';
import {View,Dimensions} from 'react-native';
export default class BlurComponent extends Component {
    render(){
        width = Dimensions.get('window').width;
        height = Dimensions.get('window').height;
        return (
            <View style={{position:'absolute',top:0,left:0,height:height,width:width,backgroundColor:'black',opacity:0.7}}></View>
        )
    }
}

and reference it in backdrop prop
<ActionButton backdrop={<BlurComponent />}>

I felt React Native Blur library was too much for this simple use case. Let me know if there are any issues with this approach.

@ranjithkumar8352 : You can try their default props bgColor .

<ActionButton buttonColor="red" **bgColor="rgba(0,0,0,.4)"** >
    <ActionButton.Item buttonColor='#fff'  title="Add Property" >
         <Icon name="md-beer" style={styles.actionButtonIcon} />
    </ActionButton.Item>
</ActionButton>

Thanks @Sonukr I think this prop was added recently


Rex vs VS Code search?

VS Code seems to have all the features of Rex and more out of the box using the $# number syntax. Are there features of Rex I haven't discovered yet? I'm just curious.
In VS Code $0 gets the entire match, and $1 gets the first match, $2 second... etc.

https://code.visualstudio.com/updates/v1_29#_multiline-search for even more.

Actually this is just a tool for my own need, I didn't expect ppl to use it (would be great if anyone find it useful, but not the goal). So, feel free to use whatever make sense to you :)
For the difference, as I said in the readme:

Replacing via RegExp sounds like a good options to me, but somehow most editors can only do replacement but lack of the feature "keep only what matched".

VS Code provides the functionality of string.replace but sometimes I'd need string.matchAll, which for the most of the time, I would need to write a script every time to do the transform before I made this tool.


Test data for comparison with FreeType

There are several test cases in read-fonts and punchcut where the condition is “produces results identical to FreeType.” The expected output is currently hand curated which is not ideal. We’d like to be able to generate this programmatically to both ensure consistent data and to facilitate adding new test cases.
I see two potential solutions to this problem:

Take on the freetype crate as a test only dependency in these projects. Use this inline to generate expected values on the fly.

Add a new crate to the repo that is used to generate the test data which is then included in the relevant places.

I’m not sure if 1 violates our new dependency rule. I suppose we should clarify if build and test dependencies are subject to it.

A prospective "real user" is likely to need to compile our code so I would think that rules out build dependencies?
Why not continue the current fontations model - one we've used in other places with considerable success - used for test fonts: submit both the output and the tooling, e.g. a script or a wee tool, to update the output. You would:

Submit a ttx and the compiled result of an "interesting" scenario, say a novel glyph setup
Run a script or wee tool to generate the freetype data for the same and emit it to a text file you commit

the prospective "real user" doesn't ever have to do this

In your test load the text file - the expected result - and the glyph punchcut gets from the font and compare them

May I suggest comparative fuzzing, using FreeType as the oracle? 👀

I suppose the “wee tool” would be equivalent to option 2? We ultimately need to execute FreeType code to generate the output so a script won’t suffice.
As for build and test dependencies, they’re simply additions to Cargo.toml that are only included during the relevant phases so shouldn’t add any friction for the user.

I agree it's basically the same as option 2, just removing the requirement to be a crate. For example, a wee python script or C++ tool become permissable.


consume consul from latest(270) cf-release

[#150161180]

LGTM

@rohitsharma04 Updated the V1 manifest

@boyang9527 Yes, this issue has been handled in cf-deployment.

LGTM


Fix per sample fastq bug

During processing of the HMP2 metagenomic data I found an issue where it was possible to assign  the same run_prefix to multiple files.

Codecov Report

Merging #2386 into master will increase coverage by <.01%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master    #2386      +/-   ##
==========================================
+ Coverage   93.76%   93.77%   +<.01%
==========================================
  Files         163      163
  Lines       18667    18674       +7
==========================================
+ Hits        17504    17511       +7
  Misses       1163     1163

Impacted Files
Coverage Δ

qiita_pet/handlers/api_proxy/studies.py
87.2% <100%> (+0.54%)
:arrow_up:

Continue to review full report at Codecov.

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

This is ready for review.
Before:

After:


Get a cost centre code

As a team member
I want to know what cost centre to charge
So I can pay for goods

Jeremy has instructed to use Joe's postdoc research costs code for purchasing goods for this project (517746302)

This issue now closed.


Compile partial blocks to allow nested partial-block usage

Also took the occasion to simplify the PartialBinder logic a bit.
As a plus, this change compiles the template fallback value.

👍
Ran into this issue with nested partials - your fork works as expected for me and the tests cover nesting a combination of content and partials.
@rexm any chance this can be merged in?

Will release this weekend

I've just run into this issue as well, compiled the master branch locally and confirmed that this fixes our issue with nested partials.
Any chance for a release of the current state of master?
Thanks for the great library by the way, @rexm!


python312Packages.influxdb-client: 1.47.0 -> 1.48.0

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for python312Packages.influxdb-client is: InfluxDB client library
meta.homepage for python312Packages.influxdb-client is: https://github.com/influxdata/influxdb-client-python
meta.changelog for python312Packages.influxdb-client is: https://github.com/influxdata/influxdb-client-python/blob/v1.48.0/CHANGELOG.md
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes
Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 1.48.0 with grep in /nix/store/7khnrij150vhprhd7hkw887ldw0rnfvy-python3.12-influxdb-client-1.48.0
found 1.48.0 in filename of file in /nix/store/7khnrij150vhprhd7hkw887ldw0rnfvy-python3.12-influxdb-client-1.48.0

Rebuild report (if merged into master) (click to expand)

5 total rebuild path(s)

5 package rebuild(s)

First fifty rebuilds by attrpath

home-assistant-component-tests.influxdb
influxdb2-provision
python311Packages.influxdb-client
python312Packages.influxdb-client

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/7khnrij150vhprhd7hkw887ldw0rnfvy-python3.12-influxdb-client-1.48.0 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A python312Packages.influxdb-client https://github.com/r-ryantm/nixpkgs/archive/118327ee921198191db5681115aab67d35bf47fb.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/118327ee921198191db5681115aab67d35bf47fb#python312Packages.influxdb-client

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/7khnrij150vhprhd7hkw887ldw0rnfvy-python3.12-influxdb-client-1.48.0
ls -la /nix/store/7khnrij150vhprhd7hkw887ldw0rnfvy-python3.12-influxdb-client-1.48.0/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
nixpkgs-review result
Generated using nixpkgs-review.
Command: nixpkgs-review

x86_64-linux

:white_check_mark: 6 packages built:

home-assistant-component-tests.influxdb
influxdb2-provision
python311Packages.influxdb-client
python311Packages.influxdb-client.dist
python312Packages.influxdb-client
python312Packages.influxdb-client.dist

Maintainer pings
cc @Mic92 for testing.

[!TIP]
As a maintainer, if your package is located under pkgs/by-name/*, you can comment @NixOS/nixpkgs-merge-bot merge to automatically merge this update using the nixpkgs-merge-bot.

Add a :+1: reaction to pull requests you find important.

@NixOS/nixpkgs-merge-bot merge


[python] make early_stopping callback pickleable

Hey, I'm a maintainer of LightGBM-Ray. I was working on early stopping support for that library, however the early stopping callback on LightGBM's master branch raises exceptions in our tests (see test_linux_cutting_edge (3.7)).
I assumed this is due to the nonlocal variables present in the callback and them not interacting well with Ray serialization using cloudpickle. I would imagine there would be similar issues with other distributed computing or serialization libraries.
In order to solve that issue, I took the liberty of rewriting that callback into a callable class, with the nonlocal variables becoming attributes. This solved the issues with serialization. No functionality has been affected.
Please let me know if this approach is suitable, and if not, I would love to discuss other potential solutions to the problem I am facing.

@Yard1 Hey, thank you very much for the interest in LightGBM and this contribution!
As a early opinion I would say I support conversion complex functions with closures into classes. I think this will make callbacks usage more user friendly and LightGBM codebase simpler.
Just a note: we have received a feedback about the need in callbacks serializability from Neptune maintainers recently #4719. Also, XGBoost migrated to callback classes: https://github.com/dmlc/xgboost/pull/6199. So I believe this is the right direction.

I agree with @StrikerRUS , I definitely support the proposed direction. I'll mark this PR as "in-progress" for now. Please get the unit tests working again, and then we'll be able to provide a review. Let us know if you need help with the tests.

@Yard1 since you're a new contributor, we'll have to manually approve CI runs on each commit

This is something GitHub added last year (I think?) to combat abuse of GitHub Actions.
Sorry for the inconvenience!

@jameslamb All's good, was on the accepting side myself many times 😂 The tests pass locally for me now, so it should be good!

@jameslamb I've applied your feedback, please take a look. Thanks!

@jameslamb I have added a test, please take a look!

The tests are failing due to a scheduled Windows brownout in CI.

Hey @jameslamb the Windows CI env doesn't seem to have cloudpickle installed. How can we fix that?

@jameslamb Had to merge with master, please rerun tests

@jameslamb Gentle ping :)

Let me merge master again, maybe that will fix the CI

@StrikerRUS @jameslamb CI is green now 🙏

@jameslamb Could you please take another look before merging?

Thanks again for this!
I changed the title to "make early_stopping callback pickleable" because PR titles in this project automatically become items in the release notes, and I think the fact that lgb.early_stopping is now pickleable will be more important to users than the particular mechanism used to achieve that.
I've also added #5080 documenting the work to ensure that all other callbacks are pickleable. Thanks for bringing the issue to our attention.
Good luck with lightgbm-ray, and we hope we'll see more contributions from you in the future 👋

Thanks!


 label falls behind default placeholder text

[x] The issue is present in the latest release.
[x] I have searched the issues of this repository and believe that this is not a duplicate.

Current Behavior 😯
Label falls behind placeholder text of date input in FireFox

Expected Behavior 🤔
Label should be above/there should be no default placeholder text

Steps to Reproduce 🕹

https://codesandbox.io/s/amazing-wing-xfgo6
Steps:

Inspect TextField, it does not show label
Click on TextField
Now label is visible

Context 🔦

Just implementing a date-picker
Your Environment 🌎
Ubuntu 20, FireFox 76.0.1

Tech
Version

Material-UI
v4.9.9

React
16.13.1

Browser
Firefox 76.0.1

TypeScript
3.8.3

etc.
Ubuntu 20. LG Monitor

Duplicate of #8131


Unable to execute.

issued the following comand
#vina12 i renamed and put it in the folder
#the config file contains receptor ligand info
"receptor=rec.pdbqt
ligand=lig.pdbqt
center_x =  27.898
center_y =  5.175
center_z = 59.160
size_x = 25
size_y = 25
size_z = 25
exhaustiveness=50"
./vina12 --config config.txt --maps rec.*.map --scoring vinardo  --out lig_out.pdbqt --cpu 8
Command line parse error: too many positional options have been specified on the command line

Hi,
try changing --maps rec.*.map to --maps rec

lower_isomer-2/Vina12$ ./vina12 --config config.txt --maps rec --scoring vinardo --out lig_out.pdbqt --cpu 8
#################################################################
If you used AutoDock Vina in your work, please cite:

O. Trott, A. J. Olson,
AutoDock Vina: improving the speed and accuracy of docking
with a new scoring function, efficient optimization and
multithreading, Journal of Computational Chemistry 31 (2010)
455-461

DOI 10.1002/jcc.21334

Please see http://vina.scripps.edu for more information.
#################################################################
ERROR: Cannot specify both receptor and affinity maps at the same time, --flex argument is allowed with receptor or maps.

@debanjansen48 You almost have it working :-) The 1iep_receptor_vina_box.txt example config file only contains the parameters for the search box but not the ligand which you would need to specify in this example (i.e. --ligand 1iep_ligand.pdbqt).
Here is our writeup for this example:
https://autodock-vina.readthedocs.io/en/latest/docking_basic.html#running-autodock-vina

I am having a prefix map issue.  How in the "Hello World" does this not work?
% ls Q00534_fill.*.map
Q00534_fill.A.map       Q00534_fill.HD.map      Q00534_fill.NA.map      Q00534_fill.d.map
Q00534_fill.C.map       Q00534_fill.N.map       Q00534_fill.OA.map      Q00534_fill.e.map

vina --flex Q00534_fill.BL00010001_ligand_conect.fixed.out_flex.pdbqt --ligand trilaciclib.pdbqt --maps Q00534_fill --scoring vina --exhaustiveness 32 --out Q00534_fill_flex_trilaciclib_vina_out.pdbqt

Scoring function : vina
Flex receptor: Q00534_fill.BL00010001_ligand_conect.fixed.out_flex.pdbqt
Ligand: trilaciclib.pdbqt
Exhaustiveness: 32
CPU: 0
Verbosity: 1

Reading Vina maps ...
ERROR: No *.map files with prefix "Q00534_fill"

I added some debug print lines to print out the atom type and file name.  This atom types don't match those normally from any sort of run Usage: prepare_receptor4.py -r filename.  Is there a new  prepare_receptor for vina?
CHECKING atom type: "C_H"

File name: "Q00534_fill.C_H.map"

CHECKING atom type: "C_P"

File name: "Q00534_fill.C_P.map"

CHECKING atom type: "N_P"

File name: "Q00534_fill.N_P.map"

CHECKING atom type: "N_D"

File name: "Q00534_fill.N_D.map"

CHECKING atom type: "N_A"

File name: "Q00534_fill.N_A.map"

CHECKING atom type: "N_DA"

File name: "Q00534_fill.N_DA.map"

CHECKING atom type: "O_P"

File name: "Q00534_fill.O_P.map"

CHECKING atom type: "O_D"

File name: "Q00534_fill.O_D.map"

CHECKING atom type: "O_A"

File name: "Q00534_fill.O_A.map"

CHECKING atom type: "O_DA"

File name: "Q00534_fill.O_DA.map"

CHECKING atom type: "S_P"

File name: "Q00534_fill.S_P.map"

CHECKING atom type: "P_P"

File name: "Q00534_fill.P_P.map"

CHECKING atom type: "F_H"

File name: "Q00534_fill.F_H.map"

CHECKING atom type: "Cl_H"

File name: "Q00534_fill.Cl_H.map"

CHECKING atom type: "Br_H"

File name: "Q00534_fill.Br_H.map"

CHECKING atom type: "I_H"

File name: "Q00534_fill.I_H.map"

CHECKING atom type: "Si"

File name: "Q00534_fill.Si.map"

CHECKING atom type: "At"

File name: "Q00534_fill.At.map"

CHECKING atom type: "Met_D"

File name: "Q00534_fill.Met_D.map"

CHECKING atom type: "C_H"

File name: "Q00534_fill.C_H.map"

CHECKING atom type: "C_P"

File name: "Q00534_fill.C_P.map"

CHECKING atom type: "W"

File name: "Q00534_fill.W.map"

OK this only works for ad4 scoring function.  This needs to be added to readthedocs as a warning under https://autodock-vina.readthedocs.io/en/latest/docking_basic.html?highlight=map#a-using-autodock4-forcefield


Modify the order of taxonomies

This is an enhancement or feature.
Summary
Two modifications:

Categories and tags on each post are sorted by alphabetic.
Categories and tags on their archive pages are first sorted by the counts and then sorted by alphabetic.

Nice work!


Update community.json with free monthly workshops

PR Type
What kind of change does this PR introduce?

[ ] Bugfix
[ ] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[ ] CI related changes
[ ] Documentation content changes
[X] Other... Please describe: New event listed

Does this PR introduce a breaking change?

[ ] Yes
[x] No

Other information

Hello Alain, do you have an alternative image with logos above/below the text? It would make the content bigger and easier to read.

Where can I see a preview of what it looks like now? Also, what size of image would work best? Thanks!

@alcfeoh You can run the project locally, check the contributing guide as you need an .env file.
otherwise, here is how it looks:

I don't have standards as it depends on communities logos but most of them are square ones.

Hi @alcfeoh, I merged it as the event is in a few days. Feel free to open a new pull request to change the image.

@geromegrignon I created a new PR with a square image


Estimated Effects in random effect

Hi @KseniaNedorezova Is there an issue to ask about?  Could you post some code demonstrating the issue?

Hello everyone! I worked with panel data
All estimated effects are equal 0.
Why estimated effects in random effect model equal 0? P-value of the model 0,0017.

Can you please post some demo code that reproduces the problem.

import pandas as pd
from linearmodels import PanelOLS, RandomEffects
data = pd.read_excel(r"...", sheet_name = "....").reset_index(drop = True)
data = data.set_index(['Name', 'Year'])
mod_re_prod = RandomEffects(data['Percent'], data[['value_1', "value_2"]]).fit()
mod_EF_re_prod = mod_re_prod.estimated_effects
print(mod_EF_re_prod )

Any chance you could use one of the demo datasets included with linearmodels to reproduce?  That way I can run without needing any data, which might not be available to share.

Can you post some runnable code so I can see if I can help you?  This doesn't usually happen and so seems to be specific to your setup.


insecure_docker_registry_list not being applied

Hi
We have a private docker registry with a self signed certificate.
We update the property in stager job:
capi.stager.insecure_docker_registry_list: [ "somehostname:443" ]
When we try to stage an image we get this message:
Iniciando app testDocker en la organización system / espacio test como admin...
Creating container
Successfully created container
Staging...
Staging process started ...
Failed to talk to docker registry: Get https://somehostname:443/v2/: x509: certificate signed by unknown authority
Staging process failed: Exit trace for group:
builder exited with error: failed to fetch metadata from [test] with tag [latest] and insecure registries [] due to
....
....
Exit status 2
Staging Failed: STG: Exited with status 2

Additionally we added in the manifest instance_groups diego-cell, job garden:
garden.insecure_docker_registry_list: [ "somehostname:443" ]
Do you know what else we have to do?

Hi
we were able to deploy it as an app in cloud foundry. I understand that this is not a feature that is available for the open source cf.
Best regards

I'm glad to hear you were able to get this working for you.  This feature should be available to all users of CF, so I'll re-open this and look at resolving the issue.

Unfortunately the stager component and other "bridge" components have been deprecated for some time and we rarely update them anymore. Fortunately, we just tested insecure docker registries in a post-bridge deployment and it works as expected. You'll need to set BOTH cc.diego.insecure_docker_registry_list on the API jobs AND garden.insecure_docker_registry_list on the garden jobs. Instructions on removing the bridge components can be found here.


Correct for bad links: classtut.rakudoc

The problem
Bad links in file has and Attributes are not source files.
Solution provided

L<> markup is used for has, but probably should be I/B. Chose B
Missing # in attributes link, to refer to heading later

closing as not correct. Missing link exists, but correction is wrong


Simple suggestion + tips for noobs (like me)

One thing that might be helpful is to have the docker_run.sh name the container it creates the same as the config file.  If I knew how to work Git worth a damn, I'd submit a pull req. for this since I assume it's easy.
Since I was usually creating multiple containers, one for each site, it was then difficult to tell which was for which site.
CONTAINER ID        IMAGE               COMMAND                  CREATED             STATUS              PORTS               NAMES
b4ea0984ef2e        inventory-hunter    "python /src/run.py …"   3 minutes ago       Up 3 minutes                            bold_solomon
265636d8ccdb        inventory-hunter    "python /src/run.py …"   20 minutes ago      Up 20 minutes                           sleepy_golick
d6c9da1a5081        inventory-hunter    "python /src/run.py …"   20 minutes ago      Up 20 minutes                           blissful_tharp

Tip for noobs -  to display running containers:  docker ps
How do I tell if it's working?
After running docker ps copy the name you want to see if it's working and run docker logs -f <name>
Example from the command above: docker logs -f bold_solomon
Output - this is what i see when it's working.  If it's not working, you'll find hints here as to why not.
I2020-11-28 14:30:11,123 gigabyte-geforce-rtx-2070-super-gv-n207swf3oc-8gd: still in stock at the same price
I2020-11-28 14:30:13,143 gigabyte-geforce-rtx-3070-gv-n3070aorus-m-8gd: not in stock
I2020-11-28 14:30:15,113 gigabyte-geforce-rtx-3070-gv-n3070eagle-8gd: not in stock
I2020-11-28 14:30:17,111 gigabyte-geforce-rtx-3070-gv-n3070eagle-oc-8gd: not in stock
I2020-11-28 14:30:19,117 gigabyte-geforce-rtx-3070-gv-n3070gaming-oc-8gd: not in stock
I2020-11-28 14:30:21,094 gigabyte-geforce-rtx-3070-gv-n3070vision-oc-8gd: not in stock

Great idea! Working on it now.

Just committed a change to docker_run.bash with these suggestions. Thanks!


RID to String conversion

Hi all,
Could you please suggest a proper way to convert RID of existing Vertex to String?
Getting following error message:
"java.lang.IllegalArgumentException: Argument '?' is not a RecordId in form of string. Format must be: : [ONetworkProtocolHttpDb]".
Code:
var cmdCreateRel = "create edge Child from ? to (select from Site where code='" + code +"')";
print("3:::cmdCreateRel: " + cmdCreateRel);
var cityRidAsString = "" + cities[ci].getRecord().field('@rid');
print("3:::c rid: " + cityRidAsString);
gdb.command("sql", cmdCreateRel, [cityRidAsString]);
Server console:
3:::
3:::cmdCreateRel: create edge Child from ? to (select from Site where code='County-0_City-99_Site-0')
3:::c rid: #15:2001
2015-06-10 09:19:15:309 SEVERE Internal server error:
java.lang.IllegalArgumentException: Argument '?' is not a RecordId in form of string. Format must be: : [ONetworkProtocolHttpDb]
===============
But following statement works in Studio:
create edge Child from #15:2001 to (select from Site where code='County-0_City-99_Site-0')

Hi @aleks-n
which OrientDB version? Input parameters as query target are supported since 2.1

Hi,
It is 2.0.10 Community Edition for Windows (orientdb-community-2.0.10.zip). And following test code works fine.
for (var i = firstFloorNumber; i < runNum; i++){
var floorName = "Floor-" + i;
var floorCode = buildingCode + "_" + floorName;
var cmdCreateFloor = "create vertex Floor set name=?, code=?";
var floor = gdb.command("sql", cmdCreateFloor, [floorName, floorCode]);
}


This fixes issue #34. Using the wrong hostname.

When registering as a slave we are using the wrong hostname. It should
be our hostname, not the hostname name of the server we are connecting
to.

Hi @gdey , I guess if we don't register slave, we can still sync binlog from mysql master.
Maybe we can remove register slave later?

I'm going to close this and reopen it. Goofed in my repo. And started committing to master, instead of a feature branch. I'm not sure what you are asking with the second comment.


Change parameter for $currentDate operator

This changes the $currentDate operator to no longer accept a timestamp parameter, instead accepting the type directly.
Fixes #217

LGTM!


Meeting closes instantly after joining in iOS

As soon as I join the meeting the meeting closes again.
This is the behaviour in simulator, I haven't tested on real iOS device.

Using the latest master code and not the one from pub dev

This issue was only happening in example project, not replicable in new project


Active track on user page keeps playing on playlist page

To reproduce:

Go to a user's page. The user must have at least one playlist
Play a track from their page
While the track is playing, navigate to the playlist page
Play a track from the playlist
Both tracks are playing simultaneously

Expected: The first track to stop before playing the second track.

Thanks for this! This is probably a bug with the recent turbo refactor.


ModuleNotFoundError: No module named 'socketIO_client'

I've just installed it on Windows and Python 3.6 and I get this error:
ModuleNotFoundError: No module named 'socketIO_client'
I tried using pip install socketIO-client with and without "-U" parameter.
It was working fine before, on Python 3.4 before I reinstalled Windows.

Apparently, because it's import socketio_client and not import socketIO_client now. Fix required in documentation.

import socketio_client does not work for me either. I've pulled version 0.7.0 for py3.4 from anaconda

import socketio_client works for me.

ef.txt


Add mute button to push notifications

https://twitter.com/_ryannystrom/status/1050070500708376576
Would be ace if we can expand the notification and click "Mute" or "Unsubscribe".
(Potentially both, where mute just silences it for 24hr or something if there's a heated discussion)
Not sure with local notifications if you can prevent the app from opening? (I know it's possible with actual push notifications) but either way I think it's a nice time saver and convenience factor!

Love it!


Infer doi, use docker targets remove redundant flag

3 things in play here:

without you around to review the code before Friday, I decided to look more closely at how you were preparing Dockerfiles in other repo's and I like the idea of not even having to reference the script when we run the docker container and using target to help with that.
remove the need to pass the doi into as a parameter and we just infer it from the XML.
I was trying to do something clear to put tests around the log output and I allowed us to strip out the log prefix which includes a date so that we can predictable assert what the output of the log should be. I've decided that it is better to just not do any of that. I preserve the log of the project tests as an artefact of a PR run and it can be there if needed.

This will require a minor adjust in the integration with enhanced-preprints-data once in. I will take care of this once the images are up.

Will leave for @scottaubrey to review

@nlisgo - which entrypoint do you intend to be the published image? I think the test one would be as it is now? (though not 100% sure)

I've not used target before but the default entrypoint should be /app/scripts/transform.sh

I was trying to avoid specifying a target for the prod published image. I don't think I can without using multiple Dockerfile's. So have amended my Dockerfile to specify apply-xslt first as this requires least setup, then prod and finally test. The one we build and push should be prod.
The following should run the /app/scripts/transform.sh as ENTRYPOINT:
docker buildx build -t epp-biorxiv-xslt --target prod .
cat ./test/fixtures/kitchen-sink.xml | docker run --rm -i epp-biorxiv-xslt

The following should run the xslt only with apply-xslt as ENTRYPOINT:
docker buildx build -t epp-biorxiv-xslt:apply --target apply-xslt .
docker run --rm -i -v "./test/fixtures/kitchen-sink.xml:/tmp/input.xml" -v "./src/change-label-and-title-elements.xsl:/tmp/stylesheet.xsl" epp-biorxiv-xslt:apply /tmp/input.xml /tmp/stylesheet.xsl

The following should run the tests with /app/project_tests.sh as ENTRYPOINT:
docker buildx build -t epp-biorxiv-xslt:test --target test .
docker run --rm epp-biorxiv-xslt:test


2.6.0: Unable to add criteria to new rule

st2 2.6.0, using Safari on macOS. Also seen on Chrome.
Go to Rules tab, add a new rule. Give it a name & pack, and pick a trigger. Add criteria. Add something in the criteria field, e.g. this:

Now move away from that input box, e.g. by hitting  or clicking somewhere else. The criteria is removed - the box is now empty. It is not white text on white background, the criteria is gone.
Expected behavior: I should be able to add criteria to a new or existing rule.

NB: Workaround is to create a rule via CLI, and register it with sudo st2ctl reload --register-rules. Same as the process here: https://docs.stackstorm.com/rules.html

Resolved by https://github.com/StackStorm/st2web/pull/508


Decreasing the number of filesystem operations

Seems like after moving the execution of lifecycle scripts out of install.js, it is possible to decrease the number of filesystem operations. No need in a temp folder and a lock file.

i love this change, though as far as i remember it correctly locking was needed to fix simultaneous pnpm i

lgtm

But seems like I don't understand something correctly because this changes are making installation longer when packages are already cached in the store

I returned make. It checks whether the package is already in the store and does not refetch it in case it is there already.


[HOLD for payment 2024-11-21] QAB - Scan QAB is not displayed for offline created workspace

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Version Number: v9.0.59-0
Reproducible in staging?: Y
Reproducible in production?: N/A - new feature, doesn't exist in prod
Issue was found when executing this PR: https://github.com/Expensify/App/pull/51836
Email or phone of affected tester (no customers<EMAIL_ADDRESS>Issue reported by: Applause Internal Team
Action Performed:

Sign up with a new account
Apply offline network
Create a workspace
Click on FAB
Apply online network

Expected Result:
On step 4, a Scan receipt QAB should be displayed pointing to the newly created workspace
Actual Result:
There is no Scan receipt QAB
Workaround:
Unknown
Platforms:

[ ] Android: Standalone
[ ] Android: HybridApp
[ ] Android: mWeb Chrome
[ ] iOS: Standalone
[ ] iOS: HybridApp
[ ] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[x] MacOS: Desktop

Screenshots/Videos
https://github.com/user-attachments/assets/64a331f3-2413-4c57-a287-3311ece8a870

View all open jobs on GitHub
Issue OwnerCurrent Issue Owner: @OfstadC

As per the op, this is a new feature and its not broken to the user so demoting - not a blocker

@ishpaul777 @dukenv0307 @puneetlath can you have a look please?

@ishpaul777 I'm guessing this is because we don't yet have the workspace chat reportID, right? Because we don't create it optimistically and so we have to wait for the back-end to return it? If so, I feel like it's not really a bug and the behavior is fine as-is.

Oh cool, I like that. That makes sense to me.

@ishpaul777 yeah that sounds good, if its your first group policy, set it as active. Can you put up a PR for it please as a follow up

We'll need the checklist to be completed before I can issue payment tomorrow. Thanks!

@dukenv0307 could you please complete the BZ checklist so we can issue payment here? Thank you!

Or - actually I think @ishpaul777 would be responsible for the BZ checklist 😅 Sorry about that!

Regression Test Issue

Thanks @ishpaul777 - I'll hold payment here until @puneetlath weighs in. I believe they are out today -- so if you prefer, I can raise a Slack discussion to decide on payment sooner if you'd like?
I can confirm Payment Summary for @dukenv0307 is accurate.

I don't think it's a regression. It wasn't part of the scope for the original, so I think we should pay everyone out on this one.

@ishpaul777 Offer sent

$250 approved for @dukenv0307

Updated Payment Summary - All set here 😃
Thanks everyone!


Dart/Flutter SDK

Hi, thank you for driving the open spec :)
Not sure if a community-maintained Dart SDK is desired but if so, I'm willing to be a contributor.
I've started the Dart SDK as a hobby project and it's somewhat working already (It's pretty much based on the Java/.NET SDK).
Repo: https://github.com/marandaneto/openfeature-dart-sdk
SDK: https://pub.dev/packages/openfeature
There's no provider besides the NoOP and Env. Var. Provider, but I'm willing to add the most used ones within the next few weeks/months.
I still have to write the spec tests, but feature-wise should be usable already.
Let me know your thoughts.
Thanks!

Hey @marandaneto, that's great! Thanks for your interest and support. We haven't looked into a Dart SDK yet, so your contributions would be very much appreciated!
You may be interested in syncing with @toddbaert. He's been working on updating the spec to better support client-side feature flags. It would be wonderful to get your feedback on those improvements from a mobile dev perspective.

Exciting!
I suspect that a Dart SDK would be most valuable for use in clients, as @beeme1mr suggests. Is your intent? Our research and vendor collaboration leads us to believe that first-class client SDKs require a few additional APIs to meet the needs of that use-case, so we're working on enhancements to the spec. I think with your mobile experience you could be a great asset to the project here (sorry if that sounds presumptuous :sweat_smile: ).
An overview of the client-related specification enhancements can be viewed here. The OFEPs (OpenFeature enhancement proposals) might be of particular interest.
I'd be happy to meet to talk about the Dart SDK further. If you'd like you can reach out to me directly on our CNCF channel. I can also invite you to the org.

Nice, yeah it is my intent, so any Mobile related topic would be interesting to catch up on.
I'll take a look at https://github.com/open-feature/ofep/issues/34 as well.
I'll get some time to read the spec enhancements and digest them.
I added the meetings to my cal but I can't promise to attend the next one because the day and time coincide with a few other meetings, let's see if I can move stuff around.

Hey @marandaneto ! I just wanted to highlight we recently released a new version of the spec: v0.6.0.
We believe that the last thing needed for a robust client implementation is this remaining spec PR: https://github.com/open-feature/spec/pull/171
I'd love to get your thoughts on this. as well as what was release in v0.6.0, especially in how in pertains to the potential Dart SDK.

@toddbaert ups sorry, missed this thread really.
I'll take a look over the weekend, hold on.

@marandaneto, stickiness can be achieved using evaluation context. We have an optional, named propertied called targeting key defined in the spec, but the evaluation context is just a map. All properties are optional because they can be set at various levels, and not all providers require it. However, the provider can require a targeting key and throw the TARGETING_KEY_MISSING error. This requirement should be clearly documented in the provider docs.

@marandaneto : @beeme1mr is :100: correct, "sticky" pseudo-random assignment is one of the reasons we define the targeting-key field.

Gotcha, that makes sense, the difference is that with openfeature, you have to do the stickiness algo yourself, but usually feature flag providers already have this built-in in the SDK, I guess it's fine since we already have the capability to do so.
Maybe it'd be something that we offer built-in in the spec as well in the future.
All good for now, thanks for the clarification.

Gotcha, that makes sense, the difference is that with openfeature, you have to do the stickiness algo yourself, but usually feature flag providers already have this built-in in the SDK

Most providers that exist at the moment wrap existing SDKs (or portions of them). These providers "handle the stickiness algo" by simply mapping the incoming targetingKey to their particilar user identifier, then that SDK uses it's existing stickiness algorithm on that identifier.
If somebody is implementing a provider entirely from scratch, not wrapping any existing SDK at all, you're correct that they would need to implement "stickiness" based on the targetingKey themselves, behind the scenes. That's what we've done with our reference implementation, flagd.

@toddbaert I see, yeah I wasn't thinking about that, if people are writing providers that wrap their own SDKs, makes sense.
Alright then, we are good.
Left a bit of feedback here as well https://github.com/open-feature/ofep/issues/34#issuecomment-1575528283
Do you need more feedback on something specific? wondering how else I can help with, besides my comments on this issue and https://github.com/open-feature/ofep/issues/34, the spec would cover the needs I have off the top of my head.


Explain background disk and memory usage growth

To address #957. I'll move these wherever you want them, but put them here for the sake of sending them out.
If they look good, I'll also throw them up on stack overflow.

This change is

http://cockroach-docs-review.s3-website-us-east-1.amazonaws.com/de2f19e4c11d96f73ec765c2828102b4239d9803/

http://cockroach-docs-review.s3-website-us-east-1.amazonaws.com/3326a323097654e050e8976ede5b5f205901c2c7/

Continuing in https://github.com/cockroachdb/docs/pull/1450/files. Closing this PR.


Add support for pix2pix and inpainting models

Merging a pix2pix or inpainting model with a normal model yet does not work. We need to catch dimension mismatches in the shapes of the tensors and take appropriate action. We need to look around on github, I'm pretty sure there is code for this already somewhere.

This?
https://github.com/vladmandic/automatic/blob/3c952675fefd2c94b817940ffbd4cd94fd5876c9/modules/merging/merge.py#L335

Yes, but I'm not sure this is how it's supposed to be done. Instead of ignoring the keys, we should test slicing, merging and concat.