How to move files when album-artist changes?

Hi, I'm a bit new to beets... loving it so far!
Maybe this is a bug, or maybe I'm doing something wrong. There's one thing I can figure out...
If I import 2 albums by the same artist and the album-artist tags are messed up on 1 of them, they'll both get imported into 2 separate album artist folders.
i.e.
/full-album
    /Sun Ra
        /Astro Black
    /The Sun Ra Arkestra
       /Atlantis

In this case I want both artist and albumartist to be Sun Ra.
If I change the tags with beet mod artist="Sun Ra" albumartist="Sun Ra", everything works as expected and is confirmable via beet ls or beet info, but when I try to beet write or beet move nothing happens. The incorrect directory is still used, the directory structure above is still used but what I actually want is:
/full-album
    /Sun Ra
        /Astro Black
        /Atlantis

Is there a way to get this to work as I want? Or is this a bug with beet write?
(beets version 1.3.14)

You'll want to use the album flag to modify the album-level metadata (beet mod -a albumartist="Sun Ra").
This is indeed confusing; it's not just you! See also #561.

ah... thanks!


Object added to Camera appear incorrectly positioned.

I am facing a problem with reticle display after upgrading to Polyfill 0.10.4.  The Reticle (added fixed position 1 unit in front of camera) appears shifted horizontally towards edges on both left and right viewport.  Because of this, while scene distortion is accurate, reticles appear duplicate.  I am unable to pinpoint the root cause of the problem.  If I roll back to polyfill 0.9.x the problem disappears.
Screen Shots (Compare positions of the white circle)
0.9.x (Reticles are centered)

0.10.x (Reticles are shifted to left and right edges on each viewport)

If these two images are super imposed you can see the offset very clearly.

Do you have any clue?  Reticle is added to the camera, not scene.

Rewrote the same using polyfill 0.10.4 and rayinput by @borismus .  I can see the same offset issue for the gaze reticle.  Will build a smaller test case and share.  But this seems like a real issue to me.

What browser are you seeing this in? Could you try chrome m65 and m67 (canary)?

I think this is the same issue as https://github.com/immersive-web/webvr-polyfill/issues/312, thank you for the reproduction case, the reticle trick should help in tracking down the issue more

Confirming, ghosting has nothing to do with reticle.  It is purely a parameter of distance from camera.  Also observed some ghosting even when object is ~10m from camera.  But after some time (gyroscope movements)  ghosting disappeared for objects > ~10m.  This is not consistent.  But may be useful for you to troubleshoot.  For now I have moved the reticle to 12m as a work around.

This was tracked down to this: https://github.com/immersive-web/cardboard-vr-display/commit/82de07f2eae7306ee65cf05ea7b0f8185216b864
Also added was a reticle to the example to track issues with objects closer to the camera so we can avoid similar issues in the future


Update index.md

Update links to the maintained repo by Filecoin

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


ODL geometry interfaces

Hello! I just found out about this library and I'm very optimistic about it.
So far most implementations have used something like the as_tensorflow_layer     in ODL which calls ASTRA as a backend to get CUDA accelerated projections.
ODL already has rather thought out geometry objects that should be convertible to PYRO-NN geometries. This would allow seamless testing and integration between the framework, e.g. doing a TV reconstruction in ODL and training with PYRO-NN. It would also allow us to validate each others implementations and perform benchmarks.

Hello,
i was planning to contact you either way :).
I think that's a great idea, we definitely should do that. It should be pretty straight forward to bring the PRYO-NN-Layers to ODL. I think we have to ensure that we find all minor differences, or implicit assumptions (e.g. For the 3D projection matrices we assume a normalization such that the last entry is 1, also the channel handling could be a minor obstacle, etc.)


Syntax error when compiling clearInterval inside setInterval with optimization=true

🐞 Bug report
Command (mark with an x)

[ ] new
[x] build
[x] serve
[ ] test
[ ] e2e
[ ] generate
[ ] add
[ ] update
[ ] lint
[ ] xi18n
[ ] run
[ ] config
[ ] help
[ ] version
[ ] doc

Is this a regression?

Unknown.
Description

Given the code:

let name = setInterval(() => {
    clearInterval(name);
}, time);

Building or serving with optimization=true will result in a syntax error, and building or serving with optimization=false results in no such error.
Changing "let" to "var" will fix the issue but I feel the functionality should be consistent with or without optimization.
I speculate this issue may apply to anytime an anonymous function is used as a parameter in any function, and the anonymous function attempts to access the variable referencing the enclosing function, but I have no evidence to back that up.
🌍 Your Environment

Angular CLI: 7.3.8
Node: 10.15.3
OS: win32 x64
Angular: 7.2.15
... animations, common, compiler, compiler-cli, core, forms
... http, language-service, platform-browser
... platform-browser-dynamic, router

Package                                            Version
---------------------------------------------------------
@angular-devkit/architect               0.6.8
@angular-devkit/build-angular       0.6.8
@angular-devkit/build-optimizer    0.6.8
@angular-devkit/core                      0.6.8
@angular-devkit/schematics            7.3.8
@angular/cdk                                   7.3.7
@angular/cli                                     7.3.8
@angular/material                           7.3.7
@ngtools/webpack                          6.0.8
@schematics/angular                       7.3.7
@schematics/update                        0.13.8
rxjs                                                    6.5.1
typescript                                          3.2.2
webpack                                           4.8.3

Sorry for the formatting for copy-paste can't be performed.

Hi, is this still reproducible in version 9?

Closing as no further feedback provided by the issue author.
If the problem persists after upgrading, please open a new issue, provide a simple repository reproducing the problem, and describe the difference between the expected and current behavior.


Coordinate transformations

Discussed in https://github.com/desertfireballnetwork/DFN_darkflight/discussions/1

Originally posted by pgrebolt May 29, 2023
Hi all,
I am working with the DFN darkflight modelling code and, looking deep into it, I think there are some conversions between frames that are not done properly. I have seen that when converting from ECI coordinates to LLH, a transformation involving ECEF->LLH is applied, thus neglecting any Earth rotation effect. For example, this occurs in df_functions.py, in line 77 (AtmosphericModel). In my opinion, the steps should be ECI->ECEF->LLH. I have made the conversion as:
Pos_ECEF = ECI2ECEF_pos(Pos_ECI, t_jd)
Pos_LLH = ECEF2LLH(Pos_ECEF)

Similarly, this also happens in df_functions when converting the wind vector (in ENU) to ECI or in trajectory_utilities.py when computing  gravity_vector(). In the case of the wind vector I am now conducting the following conversion:
lat = float(Pos_LLH[0]); lon = float(Pos_LLH[1]) # (rad)
ENU2ECEF_matrix = ENU2ECEF(lat, lon)
Wind_ECEF = np.dot(ENU2ECEF_matrix, Wind_ENU)
Wind_ECI = ECEF2ECI(Pos_ECEF, Wind_ECEF, t_jd)[1] + v_rot

However, I am still wondering if in this case v_rot should be added.
Has anyone wondered the same? Is it possible that I am missing some approximations between ECI and ECEF?
Thank you

Hello, I have asked the same question to one of the author and we would like to repo his answer here:
%%
You may notice I've put an ECI position vector into a ECEF2LLH conversion. From memory, I believe that was because I wanted to save computational time and also because we don't have access to timing information at that point in the code to do the proper conversions from ECI -> ECEF -> LLH. I think the +-40m was calculated by me and is due to the small difference between Earth's ECEF z-axis and Earth's ECI z-axis. If I pretend those z-axes are the same (as I've done in the code), I'm actually shifting the Earth's ellipsoid in one of those frames very slightly, changing heights on the ground.
If I pretend those z-axes are the same (as I've done in the code), we end up with a possible height difference on the ground (height between the blue and green ellipses) of about 40m.
That said, I don't think this is a problem at all because (1) the meteoroid is probably travelling almost vertically by that point anyway so the impact site in lat/long would be almost the same, and (2) as far as I'm aware, we are not yet including proper ground height information, besides a constant height above sea-level set in a config file somewhere.
You are welcome to change it to use the proper conversions, i.e. ECI -> ECEF -> LLH, but you will slow down the code quite a bit and you will need to pass it the timing information somehow.
%%
That's the answer. Hope this will help you.


Attribution for EP data not updating when list moved to new contributor profile

Overview
When a list is moved from one contributor profile to another, the attribution for EP data points is still displaying as from the original contributor, even though the name and address update to the new contributor.
Expected Behavior
When a list moves to a different contributor profile, the attribution for all data points on that facility should be updated to the new contributor.
Steps to Reproduce

See this facility for an example: https://openapparel.org/facilities/BD2019241256YMR?contributors=1045&lists=1617
This was part of a list uploaded by OAR, then moved to ZHDC's profile.
The name is attributed to ZDHC (correctly), but the parent company is attributed to OAR.

Demo
Check name data, then check parent company data on this profile
Technical implementation
NOTE This issue only covers the development of the script needed to fix the data for a list

Create a SQL or one-line Django shell script (that can be run via ecsmanage) that giving the ID of a list, finds the current contributor associated with that list and corrects the contributor on all of the extended profile fields created from facility list items belonging to that list.

This SQL snippet should update all extended profile fields of a given list to match the contributor of the list's source:
UPDATE
  api_extendedfield
SET contributor_id = s.contributor_id
FROM api_extendedfield ef
JOIN api_facilitylistitem fli
  ON ef.facility_list_item_id = fli.id
JOIN api_source s
  ON fli.source_id = s.id
JOIN api_facilitylist fl
  ON s.facility_list_id = fl.id
WHERE fl.id = <list-id>;

That SQL statement ended up setting the contributor ID on all of the extended rows.
This is not yet tested and not very elegant, but I think it will have the proper update scope.
UPDATE api_extendedfield
   SET contributor_id = (
     SELECT contributor_id
       FROM api_source
      WHERE id = <source_id>
   )
 WHERE facility_list_item_id IN (
   SELECT id
     FROM api_facilitylistitem
    WHERE source_id = <source_id>
 )

I found the appropriate source with these queries
select name, c.id, facility_list_item_id
from api_extendedfield ef
join api_contributor c
on ef.contributor_id = c.id
where ef.facility_id = 'BD2019241256YMR';

  name                     |  id  | facility_list_item_id
  -------------------------+------+-----------------------
  Tom Tailor [Public List] |  908 |                681278
  Tom Tailor [Public List] |  908 |                681278
  Tom Tailor [Public List] |  908 |                681278
  ZDHC Foundation          | 1045 |                694033
  Open Apparel Registry    | 1183 |                718297
  (5 rows)

select source_id from api_facilitylistitem where id = 718297;

  source_id
  -----------
  332302

I counted the number of rows we expect to update with
select count(*)
from api_extendedfield
where facility_list_item_id in (
    select id from api_facilitylistitem
    where source_id = 332302
);

  count
 -------
  1902
  (1 row)

Then ran the update
UPDATE api_extendedfield
   SET contributor_id = (
     SELECT contributor_id
       FROM api_source
      WHERE id = 332302
   )
 WHERE facility_list_item_id IN (
   SELECT id
     FROM api_facilitylistitem
    WHERE source_id = 332302
 );

UPDATE 1902

Browsing https://staging.openapparel.org/facilities/BD2019241256YMR we see the attribution on the Parent Company fields is now correctly showing ZDHC

Verified by checking the extended fields table
select name, c.id, facility_list_item_id
from api_extendedfield ef
join api_contributor c
on ef.contributor_id = c.id
where ef.facility_id = 'BD2019241256YMR';
           name           |  id  | facility_list_item_id
--------------------------+------+-----------------------
 Tom Tailor [Public List] |  908 |                681278
 Tom Tailor [Public List] |  908 |                681278
 Tom Tailor [Public List] |  908 |                681278
 ZDHC Foundation          | 1045 |                694033
 ZDHC Foundation          | 1045 |                718297
(5 rows)

Satisfied with this test, I connected to prod and ran the following
select count(*) from (select distinct contributor_id from api_extendedfield) s;
 count
-------
   169

select count(*)
from api_extendedfield
where facility_list_item_id in (
    select id from api_facilitylistitem
    where source_id = 332302
);

 count
-------
  1902
(1 row)

UPDATE api_extendedfield
   SET contributor_id = (
     SELECT contributor_id
       FROM api_source
      WHERE id = 332302
   )
 WHERE facility_list_item_id IN (
   SELECT id
     FROM api_facilitylistitem
    WHERE source_id = 332302
 );

UPDATE 1902

select count(*) from (select distinct contributor_id from api_extendedfield) s;

 count
-------
   168

select name, c.id, facility_list_item_id
from api_extendedfield ef
join api_contributor c
on ef.contributor_id = c.id
where ef.facility_id = 'BD2019241256YMR';

           name           |  id  | facility_list_item_id
--------------------------+------+-----------------------
 Tom Tailor [Public List] |  908 |                681278
 Tom Tailor [Public List] |  908 |                681278
 Simple Approach          | 1740 |                734700
 Simple Approach          | 1740 |                734700
 ZDHC Foundation          | 1045 |                718297
 Simple Approach          | 1740 |                734700
 Simple Approach          | 1740 |                734700
 Tom Tailor [Public List] |  908 |                681278
 ZDHC Foundation          | 1045 |                694033
 Simple Approach          | 1740 |                734700


Added --monkeypatch option to pact mock service start up command and …

…pact provider verifier command
Fix #54
These changes have been made blind, as I have not yet been able to set up an environment where I can compile and test (help appreciated), but they have been reviewed by our local TS expert :-)

"when user specifies valid options
1) should return serverFactory using specified options"
fails with
"1) Pact Spec Create serverFactory when user specifies valid options should return serverFactory using specified options:
Error: Monkeypatch not found at path: /absolute/path/to/ruby/patch.rb
at Object.Server.create [as default] (src\server.ts:130:11)
at Pact.createServer (src\pact.ts:30:29)
at Context. (src\pact.spec.ts:94:23)"
because the named file obviously doesn't exist. Not sure how to specify a file such that it will be found in the builds.

Now,
"when user specifies invalid log
1) should return an error on invalid path"
fails with

Pact Spec Create serverFactory when user specifies invalid log should return an error on invalid path:
AssertionError: expected [Function] to throw Error
at Context. (src\pact.spec.ts:168:16)"

The origin version of the test,
pact.createServer({log: path.resolve(dirPath, "log.txt")});
expect(fs.statSync(dirPath).isDirectory()).to.be.true;

seemed not to do what its spec said, so I tried to fix it with,
expect(() => {
	pact.createServer({log: path.resolve(dirPath, "log.txt")});
}).to.throw(Error);

but since it does
try {
	fs.statSync(path.normalize(options.dir)).isDirectory();
} catch (e) {
	mkdirp.sync(path.normalize(options.dir));
}

it never fails - unless I find a name that is invalid in all possible environments. Linux only forbids '/' in file/dir names, but any '/' I use will be swallowed by path.normalize().
I propose to delete this test. Will comment it out for now.

Yes, I think the mock just creates whatever path you give it for the log.

Thanks for this @oliver-gramberg, it mostly looks OK to me. There's another fundamental PR in the works at the moment, this is probably best done after that.

Can you get the PWD via a function and prepend it to the relative path of the monkeypatch file to get its absolute path? I would actually expect there to be an absolute path function in the file utils class.

@bethesque, sorry not sure where exactly you want me to do that. I did not find the place where the CLI is being called. Different repo?

Not sure how to specify a file such that it will be found in the builds.

I was suggesting a solution for that. Something like "${Dir.pwd}/relative/path/to/file.rb".
Can I make a suggestion here? You seem to be testing the functionality, not the "contract" (interface). All your test needs to do is show that the absolute path is passed correctly to "--monkeypatch", rather than prove that the behaviour is correct. Along the lines of: https://docs.pact.io/best_practices/contract_tests_not_functional_tests.html

I did not find the place where the CLI is being called.

I'm not actually sure how it works, as I don't maintain the js repos, sorry.

OK, it's about the test code, not the production code. That answers my "where" question, at least partly. I was assuming the opposite, thinking you wanted me to actually get the monkeypatch parameter set.
Always happy to learn! I like the argument the article you referenced makes, and I think I will actually be able to apply it my current project. For this fork/PR: I copied existing test cases and adapted them to use the monkeypatch parameter. Yes, it's testing the functionality (of the wrapper code), but so are the tests where I copied from, and I am not testing deeper than those. Or am I? If I were to test the correct implementation of the monkeypatch, then shouldn't I have coded an expectation about the non-mangling of HTTP headers?
I am still not sure we're talking about the same pieces of code. Could we use the commenting feature of Github on the code diff tab of the PR?

I've added a comment next to the bit of code that made me think of the "test the interface, not the behaviour" approach.

Bump, what are we thinking on this issue folks?

I would like it in please. We don't need to advertise the feature, but it helps us work around limitations of certain libraries (eg. Rack::Test which expects headers to always be in Dasherized-Capitalized-Format)

I'm really not a fan of fixing this level since it is something inherently wrong with the core library, hence any language specific implementation that uses it will break.  Would be more efficient to fix it at the core.

it is something inherently wrong with the core library - there is an assumption in Rack::Test that all HTTP headers follow the same format. While it is recommended and common to use capitalized words with dashes, it is not actually mandatory, and there is at least one person out there who can't use Pact because of this code, as they use lower case and underscores. I am not in a position to change the behaviour of Rack::Test, as it is the most widely used HTTP testing library in Ruby, and for 99% of use cases, they are correct to make the assumption that they do. I can, however, provide a way for the probably less than 1% of users to alter the code to allow them to use our library.

Sure, I understand that, but again, all libraries that are using the core
will have this issue, so instead of fixing it only in pact-node, how about
we fix it closer to the root cause?
On Thu, Feb 15, 2018 at 1:48 PM Beth Skurrie<EMAIL_ADDRESS>wrote:

it is something inherently wrong with the core library - there is an
assumption in Rack::Test that all HTTP headers follow the same format.
While it is recommended and common to use capitalized words with dashes, it
is not actually mandatory, and there is at least one person out there who
can't use Pact because of this code, as they use lower case and
underscores. I am not in a position to change the behaviour of Rack::Test,
as it is the most widely used HTTP testing library in Ruby, and for 99% of
use cases, they are correct to make the assumption that they do. I can,
however, provide a way for the probably less than 1% of users to alter the
code to allow them to use our library.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/pact-foundation/pact-node/pull/60#issuecomment-365811478,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAjA5LlBycyzThtCWA6QPujsMMPuX39Eks5tU5rvgaJpZM4QZcXg
.

Are you suggesting that I fix the Ruby code in the standalone? As I've said, I can't fix it. I can only provide a way to load a Ruby file that can alter the behaviour. To do this, I have added a --monkeypatch file argument to the Ruby standalone CLI. This PR is just exposes that option to the javascript code.

@bethesque just thinking out loud here, could this be resolved by overriding the headers with the the --custom-provider-header flag?
Down the track, supporting "request filters" as a first class citizen could also be a nice catch here.

I spent quite a lot of time looking in to this, and this was the only solution I could come up with.

Here's an example of why this feature is really useful. I've been trying to fix this issue. The code that needs to be altered is embedded deep in the Ruby chain. By using the monkeypatch file, I can modify the code to allow me to test potential fixes without having to repackage everything (ruby, travelling ruby, js...) every time. Fast feedback loops lead to fast bug fixes. By exposing the monkeypatch option in the js, I'll be able to experiment with fixes really easily, or allow other people to try out fixes on their local machines if it's something that I can't reproduce (which seems to happen a lot with windows!)

Alright, fine, I'll get this done. Should we document this feature with an
extreme warning or not document it at all so people don't abuse it?
On Thu, Feb 22, 2018 at 2:10 PM Beth Skurrie<EMAIL_ADDRESS>wrote:

Here's an example of why this feature is really useful. I've been trying
to fix this https://github.com/pact-foundation/pact-js/issues/150
issue. The code that needs to be altered is embedded deep in the Ruby
chain. By using the monkeypatch
https://github.com/pact-foundation/pact-ruby-standalone-windows-test/commit/0352de0a464f00114c062cb0310585d67b1eef71
file, I can modify the code to allow me to test potential fixes without
having to repackage everything (ruby, travelling ruby, js...) every time.
Fast feedback loops lead to fast bug fixes. By exposing the monkeypatch
option in the js, I'll be able to experiment with fixes really easily, or
allow other people to try out fixes on their local machines if it's
something that I can't reproduce (which seems to happen a lot with windows!)
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/pact-foundation/pact-node/pull/60#issuecomment-367553538,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAjA5Mwo3RJKn1bKUGO6XntJY8EX4WcMks5tXNqNgaJpZM4QZcXg
.

Let's not document it. It'll only need to be used under "adult supervision" (ie. instructions from me!)

cool :)
On Mon, Feb 26, 2018 at 9:31 AM Beth Skurrie<EMAIL_ADDRESS>wrote:

Let's not document it. It'll only need to be used under "adult
supervision" (ie. instructions from me!)
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/pact-foundation/pact-node/pull/60#issuecomment-368351407,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAjA5KTQ-tyo5nWulwEDbv_Si_qOiTqCks5tYd9egaJpZM4QZcXg
.

I'm going to close this PR as I merged it manually because of all the major changes that had happened since and added more tests to make sure that the monkeypatch option actually works (thanks @bethesque for the mini test with the ruby file).
https://github.com/pact-foundation/pact-node/commit/8701b4db9bc5fd50bdc1e02cd076e2fe490a77ea
Version 6.11.0 will be going out soon with this.


opam-dune-lint gets the wrong package for some dependencies

e.g. with Eio, ocaml-ci is currently reporting:
eio_main.opam: changes needed:
  "fmt" {>= "0.9.0"}                       [from lib_eio_linux, lib_eio_posix, lib_eio_windows]
  "iomux" {>= "0"}                         [from lib_eio_posix]
  "logs" {>= "0.7.0"}                      [from lib_eio_linux]
  "uring" {>= "0"}                         [from lib_eio_linux]

But none of these directories is (directly) part of the eio_main package anyway.
For example, iomux is a dependency only of the eio_posix library, which is part of the eio_posix package (see https://github.com/ocaml-multicore/eio/blob/60e67472e3726192c4f6b9fbff81db5eb15b8a27/lib_eio_posix/dune#L10).
If I add (package eio_posix) to this, dune complains:
Error: This library has a public_name, it already belongs to the package eio_posix

And dune describe says:
  (library
   ((names (eio_posix))
    (extensions ())
    (package (eio_posix))
    (source_dir lib_eio_posix)
    (external_deps
     ((fmt required)
      (iomux required)))
    (internal_deps
     ((eio required)
      (eio.utils required)
      (eio.unix required)))))

So dune is reporting that iomux is part of eio_posix.

This is a bug, by default opam-dune-lint also resolve the transitive dependencies because a public library could depends on a private library. It doesn't resolve the internal_deps which are public.
dune describe doesn't printing lib_eio_linux, lib_eio_posix, lib_eio_windows as  internal_deps, because of the alternative select dune stanza.
(library
  (name eio_main)
  (public_name eio_main)
  (libraries
    (select linux_backend.ml from
            (eio_linux -> linux_backend.enabled.ml)
            (          -> linux_backend.disabled.ml))
    (select posix_backend.ml from
            (eio_posix -> posix_backend.enabled.ml)
            (          -> posix_backend.disabled.ml))
    (select windows_backend.ml from
            (eio_windows -> windows_backend.enabled.ml)
            (            -> windows_backend.disabled.ml))
    ))

This is why we're seeing this bug. It could be easily solved  by sharing a list of all the public libraries.

dune describe doesn't printing lib_eio_linux, lib_eio_posix, lib_eio_windows as internal_deps, because of the alternative select dune stanza.

My bad, my description above had some mistake. The internal optional public libs were resolve this is why the bug. Fixed here #54.

The fix is deployed on ocaml-ci, is it OK to close this issue ? @talex5

That seems to work - thanks for the quick fix!
(it's now saying WARNING: can't find opam package providing "runtime_events", but that's another problem!)


[beta] NPM introduction page rendering

Challenge Name

http://beta.freecodecamp.com/en/challenges/managing-packages-with-npm/introduction-to-the-managing-packages-with-npm-challenges
Issue Description

In the first "page" of the introduction for the NPM challenges (there's two pages) the list is rendered with a smaller font. Looking in the HTMl, everything below the
 opening tag is put outside the
 the rest of the test is put in. (see first screenshot)
But there's more. If you press "go to my next step", the list I mentioned stays there, below the second part of the introduction (see second screenshot).
Now, pressing "go to my previous step", the page is rendered correctly but the list is still there (see third screenshot). I tried to look for errors in the seed file but everything looks fine, could it be an issue with React?
Browser Information

Browser Name, Version: Chrome 57.0.2987.98 (64-bit)
Operating System: Windows 7 x64
Mobile, Desktop, or Tablet: Desktop

Screenshot

@Atomk nice find! Since you've already done a lot of the investigative work, would you be interested in refactoring the CSS to fix this?

@QuincyLarson I think it's a problem with how introduction text is rendered, because when all the text is inside the
...
 everything is fine, but it seems that for some reason React puts part of the description outside that element (see first screenshot), so it is styled in a different way and React can't replace it when loading the second part of the introduction. It seems that the text is split when a list element ul or ol is encountered. I tried to look at the code that handles this, but it looks a bit too complex for me at the moment to figure this out by myself.


PRM-447 Routing in Navigation in bereavement reason flow

Realise it was not part of your PR but can you add a test covering below case in the NavigationSpec as it has not been covered.

Added the test case for getNextURLBasedOnReasonableExcuse


feat: IF-MIB also get IFName

:tada: This PR is included in version 0.11.0-next.1 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


[Work in progress] Implement viewport scrolling commands

This is for #65.
I've become stuck. @ambientsound, any chance you can take a look and help me figure out why this isn't working?

Codecov Report

Merging #93 into master will decrease coverage by 1.02%.
The diff coverage is 0%.

@@            Coverage Diff             @@
##           master      #93      +/-   ##
==========================================
- Coverage   53.57%   52.55%   -1.03%
==========================================
  Files          50       51       +1
  Lines        1749     1783      +34
==========================================
  Hits          937      937
- Misses        790      824      +34
  Partials       22       22

Impacted Files
Coverage Δ

commands/command.go
65.45% <ø> (ø)
:arrow_up:

options/defaults.go
0% <ø> (ø)
:arrow_up:

commands/viewport.go
0% <0%> (ø)

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 b40cf1d...71723ed. Read the comment docs.

The problem here is that the views.ViewPort object is passed by value, and so the connection to the original object is lost. By returning a pointer in GetViewport, this problem is solved.
However, I would strongly prefer that the viewport object is kept private, and that a Scroll method (or similar) is exposed instead. Please see the upstream feature/viewport-commands.

Nice! OK, I'll finish this off soon.
If you have a moment, can you show me how I would have fixed my own code?
(I understand that it's better to keep it private instead, I just want to explicitly see what I was doing wrong.)

Ready for review!
Thanks for the help.

I believe I have this working correctly both with 'center' on and off, by the way. It works just like vim (with and without 'scrolloff'=999 in all cases I've tried) other than at the very bottom (vim lets you keep scrolling and gives you ~ lines).


[docs] Replace some images, gifs with videos

I think the videos are definitely more useful than the images! The only thing I noticed is that maybe some of them are too fast? Such as the one about theming. We do want to keep the videos short but maybe we can make it a bit easier for someone new to follow without having to pause the video?

I purposely made them 2x to reduce their size. I agree, users will have to pause and that's what I like about video over gifs.

Add some comments with what I think makes it more readable


Touch and hold drag to support touch scrolling

What changed?
This fixes #360.
I am building a commercial app that is 100% touch, so this feature is critical for me. I hope it will help others. This is my first PR here, so I appreciate your patience and understanding.
To simultaneously support touch drag and scrolling in a scrollable ItemsControl, I updated DragDrop.cs with a touch and hold gesture that will trigger a drag operation after configurable number of milliseconds.
When the user places their finger down, holds, and the timer ticks, it will display an animated adorner under their finger giving them a visual indicator that they can drag the item.
When the touch drag starts, the system sets the ScrollViewer.PanningMode="None" to prevent touch scrolling. The system will remember the previous setting and restore it once the drag is complete.
I added two attached properties to the DragDrop.Properties.cs file to support setting the background color of the touch and hold adorner as well as the touch and hold delay. The default delay is 600 milliseconds.

TouchHoldAdornerBackground (Brush)
TouchHoldDelay (int)

For the best touch user experience in a scrollable ItemsControl, the following properties should be set on the ItemsControl:

ScrollViewer.PanningMode="Both" (this allows touch scrolling)
Stylus.IsPressAndHoldEnabled="False"  (this turns off the context menu white box that appears)

It looks like this needs more work. It's not working reliably in my application. I will continue to work on it.

@khdc Are you still working on this feature? Even if you have closed this PR?

@punker76 I need a consistent way to handle the PreviewTouchUp event independent of where the drag is initiated. Do you have a preferred way that I do that?


[grpc] add gRPC API for running CLI commands

This PR enhances gRPC service of OTNS to run CLI commands.
Since this RPC can be used to add/delete/move nodes, other gRPC calls (e.g. CtrlAddNode, CtrlDeleteNode, CtrlSetTitle) will be removed in the future PRs. @ctan-g

Why prefer unstructured calls to structured ones? Versioning and type checking will be a bit harder.

Why prefer unstructured calls to structured ones? Versioning and type checking will be a bit harder.

That's a good question indeed.
The short answer is that it automatically brings all functionalities of OTNS CLI to gRPC service.
Currently, there are more OTNS CLI commands than gRPC RPCs. For example, there is no RPC for setting packet loss ratio.
Since OTNS CLI can also run OpenThread CLI commands for nodes, now we can also run CLI commands on nodes by gRPC.
We simply don't have the manpower to implement RPCs for all commands.
For type checking. We simply don't have type checking on the Javascript side anyway. For the server side, we already have type checking for all CLI commands.
Now that we are going to have this Command RPC, I tend to remove other CtrlXxxx RPCs so that we have only one way to do something.

Thanks for the explanation, I think it makes more sense now.


WIP - Convert to 4.0.1 - Need comments

I am working on updating sails-arangodb to use the newest ArangoJS 4.0.1.
This is a work in progress.
What I did:

Updating

Now using the new sync style of ArangoJS to get graph and collection
Added possibility to provide a database name, a username and a password

Formatting

Replaced spaces with tabs
Changed some multiple var assignment to use 1 var keyword per assignment
Indented everything using JsBeautifier

I get the following error when it tries to migrate the data. I can't figure out why. I tried to create another sails project using directly ArangoJS and the same ArangoJS calls works just fine. Do you have any idea?
I still have to go through all the ArangoJS calls and convert them. Perhaps we can close this PR and you create a branch for 4.0.1 integration and then I merge to it until it's ready to make it to master?
I was also wondering, are you still maintaining this project?

This PR is now able to:

connect using ArangoJS 4.0.1
initialize the DB (Edge Collections & Document Collections)
use the following waterline methods:

.find()
.create()

I am not actively using Arango anymore, so if you would like to take over, it would be fine by me.


Reason why Health Check status kept displayed as "Waiting for health check response" wanted to be mentioned in the FAQ section

Hi Team, I have doc update request.
When enabling Health Check feature, sometimes we see message "Waiting for health check response" in overview blade.

As far as I tested this feature, this error seems to be shown when app never get health check succeeded response internally.
For example, if we have app with force HTTPS redirection, and "HTTPS Only" feature disabled at App Service side, app returns 307 to Health Check ping (As ping is sent via HTTP) and this message kept shown.
It is easy to understand if Health Check result is "0%" in such case (we can understand health check ping is executed but it keeps failing), but actually it's displayed as "Waiting for health check response" so user couldn't understand what it means and how to troubleshoot it.
I want this (interpretation of this message and the way to troubleshoot) to be mentioned in this doc's FAQ section.
Thank you very much!

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 20c42a13-f6cb-1e60-0438-f7f32b86a336
Version Independent ID: 79e7dd15-24aa-2e6d-cee7-7714e1940c11
Content: Monitor the health of App Service instances - Azure App Service
Content Source: articles/app-service/monitor-instances-health-check.md
Service: app-service
GitHub Login: @msangapu-msft
Microsoft Alias: msangapu

@yutaro113 Thanks for the feedback! We are currently investigating and will update you shortly.

@yutaro113 Thank you for brining this up. We have made an edit to the document and it should be visible within 24 hours.
We value the input CSS can provide. If you have any insights in the future, please follow this same process so we can assist further.
We will now proceed to close this thread. If there are further questions regarding this matter, please tag me in your reply. We will gladly continue the discussion and we will reopen the issue.

Thank you very much for the very quick update!
Much appreciated for your help!


Properly initalize Column while aliasing

This includes the checkExpression and autoIncrement values while creating a new copy of the column.

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

Hey @EnriqueL8, any reason why this hasn't been merged?


wrong URI in powershell script

in the example powershell script for  Generating a signature(token) from a policy I can only make the sas working if in uri I add a / ath the end of URI:
$URI="myNamespace.servicebus.windows.net/myEventHub/"
instead of
$URI="myNamespace.servicebus.windows.net/myEventHub"

Document Details
⚠ Do not edit this section. It is required for learn.microsoft.com ➟ GitHub issue linking.

ID: f4642657-5800-d9af-69cd-9623b8e2c4bc
Version Independent ID: 5fb587c3-318f-56a9-6eb2-0d3935e98694
Content: Authenticate access to Azure Event Hubs with shared access signatures - Azure Event Hubs
Content Source: articles/event-hubs/authenticate-shared-access-signature.md
Service: event-hubs
GitHub Login: @spelluru
Microsoft Alias: spelluru

@adalfa
Thanks for your feedback! We will investigate and update as appropriate.

@adalfa
Thanks for reporting this! We have created a PR for this issue and the changes should go live soon.


fixed a input not registered in osx script editor

on OS X the input of the a key was only registered when alt is pressed...
Bugsquad edit: Fixes #18808.

Yeah, this code worked when I created the PR, but later changes to this file changed the logic. I'd recommend reverting my commit. It would be better to add the new functionality (being able to use ctrl-a and ctrl-e on macos) in the new code rather than fix this bug, imo.

I spend some time trying to understand the logic. But it was confusing to me.
It definitly should not only register the a key if alt is pressed.
Now the logic works.
But i dint know if wront modifier + key should result in adding an a, or if not.
I was not really seeing a paddern

@toger5 Yes, I agree it's confusing. Did you check if ctrl-a and ctrl-e work? That is by going to start and end of line. Idealyl shift-ctrl-a/e should also work by selecting everything from current cursor to start/end of line.

Yea those combinations work

Cherry picked into 3.0.3


Cherry-pick: Add fpgaflash to deb tools-extra package (#697)

Cherry pick for a deb package fix from the internal vc branch.

Coverage remained the same at 80.82% when pulling 284785d8aefa735c9d1d92af3a0462708e02c21b on itbarraz/cp_fpgaflash_deb_fix into 2d5a2249c59e5c702fed6248fedc258543de1588 on master.


Documentation Chaos: LICENSE Link in Readme file is not set

Describe the Documentation Update
I wanna add MIT LICENSE Link in the Readme file,  for better look.
Add References or Examples

Confirm

[X] I understand that my update will add to the chaos and I'm okay with it
[X] I agree to follow this project's Code of Conduct
[X] I want to work on this documentation update
[X] I'm GSSOC'24 Extd registered Contributor
[X] I'm Hacktoberfest registered Contributor

Go ahead but I can't give you any level labels for this

I have push the changes in pr please merge it.

Completed


First version of monitoring data storage

Here I started implementing a monitoring data storage making use of sortedcontainers to efficiently lookup data for observations.
Keys are aribiraty as long as they are hashable and implement comparison and subraction operations.

Closing this, as we will probably use fits files for monitoring data storage in cta and usage with those would be more straight forward using astropy tables.
ctapipe_io_lst also has code interpolating pointing tables and finding closest entry that matches a timestamp here:
https://github.com/cta-observatory/ctapipe_io_lst/blob/v0.19.0/src/ctapipe_io_lst/pointing.py


File Explorer File Preview Awesome but very slow to start - add in background thread

Description of the new feature / enhancement
The File Explorer File Preview is awesome and very useful, but it prevents me from using the File Explorer effectively. With it it is difficult to select files and delete them. Sometimes I even delete the wrong file because of it, because there is lag in the selecting. Can it not run on a background thread and show the preview when done leaving me to operate normally in the foreground thread.
Scenario when this would be used?
When selecting and deleting/copying files
Supporting information
No response

/bugreport

PowerToysReport_2024-04-30-16-09-55.zip

All of them

A good way to see the problem is if you unzip a 30GB zip file in another explorer window and then try and move around in explorer with the preview on. It just freezes explorer for minutes, since it is not a background thread.

The zip issue happens even though I used a different explorer window and I have the "Launch folders in a separate process" check box on

Also has an issue if I copy a font file to another directory and then I can't delete that font file again, because it is locked by the preview.


Add JavaScript tutorial to the docs

This PR adds a tutorial on Image Object Detection using vanilla JavaScript. It's meant as the entry point for newbie devs who might struggle with the React/Next/Node examples. It requires no setup or build tools to run, simply drag&drop the file into the browser.

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint.

The code is a mixture of spaces and tabs
Mixture of semicolons and lack thereof
Using jsx tags for JavaScript - any advantage over simply js?

But nice job :+1: :1st_place_medal:

Thanks @kungfooman!
I've changed it to 4 spaces for the indentation, and removed the jsx snippets. Not sure why they were there, probably just from Notion.
Also added semi colons :)

I just love standalone examples, just pressing . while viewing your fork, GitHub editor opens, right click on the examples dir and download only one folder for some quick testing :+1:

That armor is a :cake: ... I wonder if we could also make a tutorial about fine-tuning :joy: Nice work :rocket:

Thanks so much @perborgen! 😍 This is great! I'll review the whole thing soon. 🤗

Thanks for the review @xenova ! Just invited you to my fork so that you can make apply changes directly. Once I see how the final source code looks like, I'll record a new scrim :)

@perborgen I've done my first pass of simplifying the app even more, so we don't need an extra utils.js file. Some TODOs:

[ ] Update docs
[ ] Rename "JavaScript" -> "Vanilla JS"
[ ] Try avoid using separate span for labels.

I think it's a lot cleaner now, and we can continue discussing what more can be done.

I've just been proof-reading and improving some of the sections (mainly just trying to keep some terms consistent across the docs). I'll continue with step 4 and the rest tomorrow.


External interface

Write a top-level utility function that does everything EnhancedVoting
needs (they initialize us with JS objects that correspond to the manifest,
etc.). Where everything else is functional, this one would hide all the
state inside and would have an API similar to the encrypt-async code, where
you feed it individual bits of data and you eventually ask for the output.

Done in current encrypt-async

@danwallach while writing a usage example for the readme, I noticed that encrypt accepts a PlaintextContest, and that there isn't a version that accepts a JS object and deserializes it. Is this by design?

For now, it's deliberate. I want the client of our API to fill out the plaintext data structures. We could potentially build a wrapper API around it, but lets wait and see if anybody wants one.
The Plaintext data structures are a lot simpler.
The Manifest stuff, on the other hand, is complicated. That's a place we might want to help simplify things.


Login page

I believe there might be some confusion, this isn't the design we finalized for the login page. Currently, we are using a modal for login. In the Figma file, I have tagged the final designs, and you can refer to those.


make instance and make switch should optionally source ~/.bashrc

Discussed in https://github.com/EnigmaCurry/d.rymcg.tech/discussions/87

Originally posted by mcmikemn July 29, 2023
When you make instance, d.rymcg.tech creates a sub-shell for the new instance. Can that subshell retain or copy the environment from the main shell? For example, I have aliases set up in .bashrc in my main shell, but they're not available in the sub-shell.
Because we cannot control what a user has in their ~/.bashrc it should not be the default behaviour to source it in the make instance subshell, however it might be nice to extend the environment by sourcing a select file by configuration.
A root environment variable like MAKE_INSTANCE_SOURCE_RCFILE= would be blank (or /dev/null) by default, but if you set this variable to a file path, it should source that file it before the other variables in the make instance/switch script, thus extending the user's environment.

I also just noticed that completion is only partial in subshells. I can type d.r and press the  key and it autocompletes to d.rymcg.tech - which is good. But if I type d.rymcg.tech con and press , it does not autocomplete to  d.rymcg.tech context like it does in the main shell.

You just need to eval the completion code the same way as is in your .bashrc, so I think it will be solved by adding MAKE_INSTANCE_SOURCE_RCFILE:
export PATH=${PATH}:${HOME}/git/vendor/enigmacurry/d.rymcg.tech/_scripts/user
eval "$(d.rymcg.tech completion bash)"

Oh, I see. Completeion isn't configured within d.ry, it's configured within my shell, and hence not (currently) in the subshell.

I just pushed 6d793f7, but it doesn't work yet. I'm struggling with the syntax to make the subshell source the file.

open a pull request and i will review.


Unit tests for Job app

hello friends
i wrote some unit tests for Views and Models of Job application for #12
hope it can help
I can also change anything you think is needed in these tests to get better
:v:

Hello dear @MAM-SYS
sorry for late response, I will check your MR asap.

thank you @sbabashahi
Let me know if there is anything needs to be fix in UnitTests
i will make it better

First of all thanks for your contribution :) and sorry again for late response.

These tests are not unit tests, they are integration and functional tests, it's not a big deal, we appreciate writing any type of test.
it could be a good practice to create a special DJANGO_SETTINGS_MODULE for testing (because you are the first one writing test and it could help others) a quick hint:
write a tests.py in settings and in manage.py something like:

if 'test' in sys.argv:   os.environ.setdefault('DJANGO_SETTINGS_MODULE', 'ratecompany.settings.tests') else:   os.environ.setdefault('DJANGO_SETTINGS_MODULE',                         'ratecompany.settings.development')
3. Hint for unit test, test for default, required and optional fields.
As I said we appreciate your work and we could merge it right when you resolve conversations, but if you could improve it (add test settings and more test) we could wait. so your call :)

@sbabashahi
thank you boss.thanks for your guidance
certainly i will add and change them as you suggested to be better
it's my pleasure

Hello friends @sbabashahi
i fixed the codes as you mentioned
hope it is better now
but if still anything else that needs to be fix or add; just say
have a good Friday

@MAM-SYS Sorry to keep you waiting, I will check it asap.

thanks a lot @sbabashahi
if anything need to change, tell me.even all of the code

@MAM-SYS well done :) i approve it in this state. I add some hints that may help us work better

Try to set an in cache db for test
Separate func test dir from unit test

@sbabashahi  thanks again for your help in this issue
i consider your hints for my other pull requests for tests


clone step using RWX volume takes long time to complete

I used the helm chart to deploy woodpecker server and agent.
With `WOODPECKER_BACKEND_K8S_STORAGE_CLASS: "nfs-rwx-storage"`` which is RWX storage class, the clone step unusually takes long time (~2m) to complete. When I disable RWX, it take less than 20s to get completed.

I can see it logs, volume mount is immediateltly available to the build pod:

Has any one experienced this? Any way to disable volume mounting?

I also use a RWX (EFS on AWS) and I don't see such long clone times. In the end, this is likely an issue of the respective RWX driver or storage backend you're using.
Here is the description of the storage class
Name:                  efs-sc-delete
IsDefaultClass:        No
Annotations:           meta.helm.sh/release-name=aws-efs-csi-driver,meta.helm.sh/release-namespace=aws-efs-csi-driver
Provisioner:           efs.csi.aws.com
Parameters:            basePath=/dynamic_provisioning,directoryPerms=700,fileSystemId=fs-05b518491300978f2,provisioningMode=efs-ap
AllowVolumeExpansion:  <unset>
MountOptions:          <none>
ReclaimPolicy:         Delete
VolumeBindingMode:     Immediate
Events:                <none>

I used the OpenEBS NFS provisioner. From the logs I posted earlier, I can see volume is immediately available to the build pod.
Provisioner:           openebs.io/nfsrwx
Parameters:            <none>
AllowVolumeExpansion:  <unset>
MountOptions:          <none>
ReclaimPolicy:         Delete
VolumeBindingMode:     Immediate
Events:                <none>

I did further benchmark, using RWO volume. I use the following pipeline definition and tested using both WoodPecker:2.0.0 and Drone:2.21.0 on the same cluster. Drone completes the tasks in 23s but it takes over a minutes for Woodpecker.
steps:
- name: Init
  image: alpine
  commands:
    - echo "hello init"
    - sleep 5

- name: Fail test
  image: alpine
  commands:
  - id
  - sleep 5
  - env
  - sleep 5
  - exit 1

Woodpecker timing:

Drone timing:

So two things I noticed:

Drone doesn't mount a volume to the build pipe. Is there a way to disable default volume mounting in woodpecker?
Woodpecker waits longer to find the pod is in ready state before running the command. Is there a way to change the frequency of this check?

Thanks for the additional tests.
To make the timings meaningful, the image pull times are needed. They need to be subtracted from the overall time.

Drone doesn't mount a volume to the build pod. Is there a way to disable default volume mounting in the woodpecker's build pod?

It doesn't seem possible at the moment. The volume mount is hard-coded here and not managed within the helm chart: https://github.com/woodpecker-ci/woodpecker/blob/f5d9c92824713f548cf5555177bf4d6b816d81bf/pipeline/backend/kubernetes/pod.go#L162-L173
I think it's required for sharing output between steps, i.e. the cloned repo from the "clone" step to all further steps. I am not sure one can turn it off. If drone does without a PV, it would be interesting to know how they do it (i.e. porting over the cloned assets to the other steps).

Woodpecker waits longer to find the pod is in ready state before running the command. Is there a way to change the frequency of this check?

The chart defines the readiness as follows
          readinessProbe:
            httpGet:
              path: /healthz
              port: 8000

By default the checks are executed every 10s but these options are not exposed within chart right now. However, I would assume that this is not the issue but some process during pod startup (maybe the mounting process) might cause some delay.

FWIW, a "normal" clone step in most of my repos takes ~9s. With a RWX. I assume that the issue might be the RWX driver you're using and not in the general use of RWX (otherwise my timings would be higher as well).
Yet of course the differences to Drone are interesting. But before going further we need verification regarding the subtraction of image pull times.

Here are RWO storage class timing. From my read, it usually takes 10-15s for volume to attach. So the additional delay in getting the commands running as compare with Drone, seems to be related to volume bounding delay.
LAST SEEN   TYPE      REASON                   OBJECT                                                          MESSAGE
15s         Warning   FailedScheduling         pod/wp-01hk8s1qapv3vj2bn13y5dw2fg-0-clone                       0/4 nodes are available: pod has unbound immediate PersistentVolumeClaims. preemption: 0/4 nodes are available: 4 Preemption is not helpful for scheduling..
12s         Warning   FailedScheduling         pod/wp-01hk8s1qapv3vj2bn13y5dw2fg-0-clone                       0/4 nodes are available: pod has unbound immediate PersistentVolumeClaims. preemption: 0/4 nodes are available: 4 Preemption is not helpful for scheduling..
10s         Normal    Scheduled                pod/wp-01hk8s1qapv3vj2bn13y5dw2fg-0-clone                       Successfully assigned woodpecker/wp-01hk8s1qapv3vj2bn13y5dw2fg-0-clone to bn-amd-8gbram-4vcpu-x4nk2
5s          Normal    SuccessfulAttachVolume   pod/wp-01hk8s1qapv3vj2bn13y5dw2fg-0-clone                       AttachVolume.Attach succeeded for volume "pvc-4f8bd953-1b04-4fcc-bea3-5434490cb90d"
4s          Normal    Pulled                   pod/wp-01hk8s1qapv3vj2bn13y5dw2fg-0-clone                       Container image "docker.io/woodpeckerci/plugin-git:2.4.0" already present on machine
4s          Normal    Created                  pod/wp-01hk8s1qapv3vj2bn13y5dw2fg-0-clone                       Created container wp-01hk8s1qapv3vj2bn13y5dw2fg-0-clone
4s          Normal    Started                  pod/wp-01hk8s1qapv3vj2bn13y5dw2fg-0-clone                       Started container wp-01hk8s1qapv3vj2bn13y5dw2fg-0-clone
15s         Normal    ExternalProvisioning     persistentvolumeclaim/wp-01hk8s1qapv3vj2bn13y5dw2fg-0-default   waiting for a volume to be created, either by external provisioner or manually created by system administrator
15s         Normal    Provisioning             persistentvolumeclaim/wp-01hk8s1qapv3vj2bn13y5dw2fg-0-default   External provisioner is provisioning volume for claim "woodpecker/wp-01hk8s1qapv3vj2bn13y5dw2fg-0-default"
13s         Normal    ProvisioningSucceeded    persistentvolumeclaim/wp-01hk8s1qapv3vj2bn13y5dw2fg-0-default   Successfully provisioned volume pvc-4f8bd953-1b04-4fcc-bea3-5434490cb90d

RWX storage class timing. taking approximately 2m 2s to get the volume attached.
LAST SEEN   TYPE      REASON                   OBJECT                                                          MESSAGE
2m1s        Warning   FailedScheduling         pod/wp-01hk8sx15w474tq9feet3dbv9b-0-clone                       0/4 nodes are available: pod has unbound immediate PersistentVolumeClaims. preemption: 0/4 nodes are available: 4 Preemption is not helpful for scheduling..
2m          Warning   FailedScheduling         pod/wp-01hk8sx15w474tq9feet3dbv9b-0-clone                       0/4 nodes are available: pod has unbound immediate PersistentVolumeClaims. preemption: 0/4 nodes are available: 4 Preemption is not helpful for scheduling..
118s        Normal    Scheduled                pod/wp-01hk8sx15w474tq9feet3dbv9b-0-clone                       Successfully assigned woodpecker/wp-01hk8sx15w474tq9feet3dbv9b-0-clone to bn-amd-8gbram-4vcpu-x4nkp
106s        Normal    Pulled                   pod/wp-01hk8sx15w474tq9feet3dbv9b-0-clone                       Container image "docker.io/woodpeckerci/plugin-git:2.4.0" already present on machine
106s        Normal    Created                  pod/wp-01hk8sx15w474tq9feet3dbv9b-0-clone                       Created container wp-01hk8sx15w474tq9feet3dbv9b-0-clone
106s        Normal    Started                  pod/wp-01hk8sx15w474tq9feet3dbv9b-0-clone                       Started container wp-01hk8sx15w474tq9feet3dbv9b-0-clone
2m2s        Normal    Provisioning             persistentvolumeclaim/wp-01hk8sx15w474tq9feet3dbv9b-0-default   External provisioner is provisioning volume for claim "woodpecker/wp-01hk8sx15w474tq9feet3dbv9b-0-default"
2m2s        Normal    ExternalProvisioning     persistentvolumeclaim/wp-01hk8sx15w474tq9feet3dbv9b-0-default   waiting for a volume to be created, either by external provisioner "openebs.io/nfsrwx" or manually created by system administrator
2m          Normal    ProvisioningSucceeded    persistentvolumeclaim/wp-01hk8sx15w474tq9feet3dbv9b-0-default   Successfully provisioned volume pvc-ad9f928a-3571-4588-a046-138f976c456d

Once volume is attached to the pod, other steps, running as containers in the same pod, use the already attached volume. Hence the timing is less than the clone step.
I don't know why it is necessary to have PVC for sharing data between containers of the same pod, couldn't we just use a temporary/ephemeral memory volume that is shared. Probably that makes it faster and less costly.

FWIW, drone by default crate EmptyDir/Memory work volume: https://github.com/drone-runners/drone-runner-kube/blob/b57d0a9c80b0df00cfb92d59e3f965e8a29ac987/engine/compiler/compiler.go#L184
	workVolume := &engine.Volume{
		EmptyDir: &engine.VolumeEmptyDir{
			ID:   random(),
			Name: workMount.Name,
		},
	}

Thanks for the stats.
As mentioned in https://github.com/woodpecker-ci/woodpecker/issues/1594#issuecomment-1876760945, every step uses its own pod. Which is why a volume is required.
Both your RWO and RWX attach times are really high 😮 My RWX attach time is lower than your RWO 🤯
I haven't done research yet what might cause this but it seems to me that something is not so good WRT to your volume drivers or other related components. Indeed, when it takes that long, WP or any other app is annoying to use.
Leaving the emptyDir/one-pod/many-pod discussion aside, I still believe you are facing a local issues in your cluster which is responsible for the high attach times. Otherwise we would have seen other requests already and I also haven't faced such high timings in different clusters I have WP running on (all AWS/Azure).
I can help you debugging it but I think we can't do anything in the chart here. If we go for the one-pod/multiple-containers approach, this needs changes in the WP k8s backend code as mentioned in https://github.com/woodpecker-ci/woodpecker/issues/1594#issuecomment-1876760945.

Yeah make total sense. I am asking DO support to understand if this is normal or something wrong with our cluster. I did further testing using basic deployments and it took 2m for RWX to get attached.


Update angular.dart dependency to v1.0.0?

current pubspec.yaml prevents it from being used with angular v1.0.0, is it planned to be upgraded soon?

The demo pubspec is still limited to <1.0.0:
https://github.com/vsavkin/hammock/blob/master/demo/pubspec.yaml#L5
Also, the mirror_based_serializer.dart imports dart:mirrors without a @MirrorsUsed() attribute.
import 'dart:mirrors';
https://github.com/vsavkin/hammock/blob/master/demo/web/util/mirror_based_serializers.dart#L3
May be better to change the models to actual objects so you can enumerate them in @MirrorsUsed()

@monkeysecurity could you submit a PR?

@vsavkin Will do. Give me a day or two.


ADS 1.21.0 (User) Wont install Juypter and other Kernels, SSL error - CIS hardened hardware Windows 10

Azure Data Studio Version: 1.21.0 (User)

Steps to Reproduce:

Install ADS on a CIS hardened Windows 10 machine
Open new notebook
Change Kernel to PowerShell
Follow blades to install python and Juypter etc
Fails with various SSL errors

Encountered an error when trying to retrieve list of installed packages: Unexpected token C in JSON at position 85
Notebook dependencies installation is in progress
> "C:\Users\myuser\azuredatastudio-python\0.0.1\python.exe" -m pip install --user "jupyter>=1.0.0" "powershell-kernel>=0.1.3" --extra-index-url https://prose-python-packages.azurewebsites.net
stdout: Looking in indexes: https://pypi.org/simple, https://prose-python-packages.azurewebsites.net
stdout:
stdout: Collecting jupyter>=1.0.0
stdout:
stderr:   WARNING: Retrying (Retry(total=4, connect=None, read=None, redirect=None, status=None)) after connection broken by 'SSLError(SSLError(1, '[SSL: CERTIFICATE_VERIFY_FAILED] certificate verify failed (_ssl.c:841)'),)': /simple/jupyter/
stderr:
stderr:   WARNING: Retrying (Retry(total=3, connect=None, read=None, redirect=None, status=None)) after connection broken by 'SSLError(SSLError(1, '[SSL: CERTIFICATE_VERIFY_FAILED] certificate verify failed (_ssl.c:841)'),)': /simple/jupyter/
stderr:
stderr:   WARNING: Retrying (Retry(total=2, connect=None, read=None, redirect=None, status=None)) after connection broken by 'SSLError(SSLError(1, '[SSL: CERTIFICATE_VERIFY_FAILED] certificate verify failed (_ssl.c:841)'),)': /simple/jupyter/
stderr:
stderr:   WARNING: Retrying (Retry(total=1, connect=None, read=None, redirect=None, status=None)) after connection broken by 'SSLError(SSLError(1, '[SSL: CERTIFICATE_VERIFY_FAILED] certificate verify failed (_ssl.c:841)'),)': /simple/jupyter/
stderr:
stderr:   WARNING: Retrying (Retry(total=0, connect=None, read=None, redirect=None, status=None)) after connection broken by 'SSLError(SSLError(1, '[SSL: CERTIFICATE_VERIFY_FAILED] certificate verify failed (_ssl.c:841)'),)': /simple/jupyter/
stderr:
stdout:   Could not fetch URL https://pypi.org/simple/jupyter/: There was a problem confirming the ssl certificate: HTTPSConnectionPool(host='pypi.org', port=443): Max retries exceeded with url: /simple/jupyter/ (Caused by SSLError(SSLError(1, '[SSL: CERTIFICATE_VERIFY_FAILED] certificate verify failed (_ssl.c:841)'),)) - skipping
stdout:
stderr:   WARNING: Retrying (Retry(total=4, connect=None, read=None, redirect=None, status=None)) after connection broken by 'SSLError(SSLError(1, '[SSL: CERTIFICATE_VERIFY_FAILED] certificate verify failed (_ssl.c:841)'),)': /jupyter/
stderr:
stderr:   WARNING: Retrying (Retry(total=3, connect=None, read=None, redirect=None, status=None)) after connection broken by 'SSLError(SSLError(1, '[SSL: CERTIFICATE_VERIFY_FAILED] certificate verify failed (_ssl.c:841)'),)': /jupyter/
stderr:
stderr:   WARNING: Retrying (Retry(total=2, connect=None, read=None, redirect=None, status=None)) after connection broken by 'SSLError(SSLError(1, '[SSL: CERTIFICATE_VERIFY_FAILED] certificate verify failed (_ssl.c:841)'),)': /jupyter/
stderr:
stderr:   WARNING: Retrying (Retry(total=1, connect=None, read=None, redirect=None, status=None)) after connection broken by 'SSLError(SSLError(1, '[SSL: CERTIFICATE_VERIFY_FAILED] certificate verify failed (_ssl.c:841)'),)': /jupyter/
stderr:
stderr:   WARNING: Retrying (Retry(total=0, connect=None, read=None, redirect=None, status=None)) after connection broken by 'SSLError(SSLError(1, '[SSL: CERTIFICATE_VERIFY_FAILED] certificate verify failed (_ssl.c:841)'),)': /jupyter/
stderr:
stdout:   Could not fetch URL https://prose-python-packages.azurewebsites.net/jupyter/: There was a problem confirming the ssl certificate: HTTPSConnectionPool(host='prose-python-packages.azurewebsites.net', port=443): Max retries exceeded with url: /jupyter/ (Caused by SSLError(SSLError(1, '[SSL: CERTIFICATE_VERIFY_FAILED] certificate verify failed (_ssl.c:841)'),)) - skipping
stdout:
stderr:   ERROR: Could not find a version that satisfies the requirement jupyter>=1.0.0 (from versions: none)
stderr:
stderr: ERROR: No matching distribution found for jupyter>=1.0.0
stderr:
stdout: Could not fetch URL https://pypi.org/simple/pip/: There was a problem confirming the ssl certificate: HTTPSConnectionPool(host='pypi.org', port=443): Max retries exceeded with url: /simple/pip/ (Caused by SSLError(SSLError(1, '[SSL: CERTIFICATE_VERIFY_FAILED] certificate verify failed (_ssl.c:841)'),)) - skipping
stdout:
stdout: Could not fetch URL https://prose-python-packages.azurewebsites.net/pip/: There was a problem confirming the ssl certificate: HTTPSConnectionPool(host='prose-python-packages.azurewebsites.net', port=443): Max retries exceeded with url: /pip/ (Caused by SSLError(SSLError(1, '[SSL: CERTIFICATE_VERIFY_FAILED] certificate verify failed (_ssl.c:841)'),)) - skipping
stdout:
Installing Notebook dependencies failed with error: Process exited with error code: 1. StdErr Output:   WARNING: Retrying (Retry(total=4, connect=None, read=None, redirect=None, status=None)) after connection broken by 'SSLError(SSLError(1, '[SSL: CERTIFICATE_VERIFY_FAILED] certificate verify failed (_ssl.c:841)'),)': /simple/jupyter/
WARNING: Retrying (Retry(total=3, connect=None, read=None, redirect=None, status=None)) after connection broken by 'SSLError(SSLError(1, '[SSL: CERTIFICATE_VERIFY_FAILED] certificate verify failed (_ssl.c:841)'),)': /simple/jupyter/
WARNING: Retrying (Retry(total=2, connect=None, read=None, redirect=None, status=None)) after connection broken by 'SSLError(SSLError(1, '[SSL: CERTIFICATE_VERIFY_FAILED] certificate verify failed (_ssl.c:841)'),)': /simple/jupyter/
WARNING: Retrying (Retry(total=1, connect=None, read=None, redirect=None, status=None)) after connection broken by 'SSLError(SSLError(1, '[SSL: CERTIFICATE_VERIFY_FAILED] certificate verify failed (_ssl.c:841)'),)': /simple/jupyter/
WARNING: Retrying (Retry(total=0, connect=None, read=None, redirect=None, status=None)) after connection broken by 'SSLError(SSLError(1, '[SSL: CERTIFICATE_VERIFY_FAILED] certificate verify failed (_ssl.c:841)'),)': /simple/jupyter/
WARNING: Retrying (Retry(total=4, connect=None, read=None, redirect=None, status=None)) after connection broken by 'SSLError(SSLError(1, '[SSL: CERTIFICATE_VERIFY_FAILED] certificate verify failed (_ssl.c:841)'),)': /jupyter/
WARNING: Retrying (Retry(total=3, connect=None, read=None, redirect=None, status=None)) after connection broken by 'SSLError(SSLError(1, '[SSL: CERTIFICATE_VERIFY_FAILED] certificate verify failed (_ssl.c:841)'),)': /jupyter/
WARNING: Retrying (Retry(total=2, connect=None, read=None, redirect=None, status=None)) after connection broken by 'SSLError(SSLError(1, '[SSL: CERTIFICATE_VERIFY_FAILED] certificate verify failed (_ssl.c:841)'),)': /jupyter/
WARNING: Retrying (Retry(total=1, connect=None, read=None, redirect=None, status=None)) after connection broken by 'SSLError(SSLError(1, '[SSL: CERTIFICATE_VERIFY_FAILED] certificate verify failed (_ssl.c:841)'),)': /jupyter/
WARNING: Retrying (Retry(total=0, connect=None, read=None, redirect=None, status=None)) after connection broken by 'SSLError(SSLError(1, '[SSL: CERTIFICATE_VERIFY_FAILED] certificate verify failed (_ssl.c:841)'),)': /jupyter/
ERROR: Could not find a version that satisfies the requirement jupyter>=1.0.0 (from versions: none)
ERROR: No matching distribution found for jupyter>=1.0.0
Found issues for this for MacOS but not for Windows.  Is there a way to force Python to use TLS/SSL later versions on Windows.
I am able to browse PIP directly in Edge to download the ZIP/TAR files so must be a ADS or a Python issue using older TLS/SSL versions

Seems to be down to the host's not being trusted, unsure how you can force this within ADS itself, but I took the "pip" command and ran it in python outside of ADS like below and then ADS after a restart and a kernel change picked up all the stuff was installed.
Maybe an open to force the sites as trusted may be needed here.
"C:\Users<myuser>\azuredatastudio-python\0.0.1\python.exe" -m pip install --trusted-host=pypi.python.org --trusted-host=pypi.org --trusted-host=files.pythonhosted.org --upgrade pip --user
"C:\Users<myuser>\azuredatastudio-python\0.0.1\python.exe" -m pip install --trusted-host=pypi.org --trusted-host=files.pythonhosted.org --trusted-host=prose-python-packages.azurewebsites.net --user "jupyter>=1.0.0" "pandas>=0.24.2" "sparkmagic>=0.12.9" "prose-codeaccelerator>=1.3.0" "powershell-kernel>=0.1.3" --extra-index-url https://prose-python-packages.azurewebsites.net

It is 2021-06-15 and I am having this issue in v1.29.0. Ant-Green's workaround from 2020-08-19 was not successful for me. Are there plans to fix this soon? Azure Data Studio provides no value over SSDTs or SSMS if the only notebook kernel that works is SQL.
Version: 1.29.0 (system setup)
Commit: 0f5cfdc2c8c2433028842e16dc5ac1a65da24292
Date: 2021-05-15T01:12:11.112Z
VS Code: 1.53.0
Electron: 11.2.2
Chrome: 87.0.4280.141
Node.js: 12.18.3
V8: <IP_ADDRESS>-electron.0
OS: Windows_NT x64 10.0.16299

My solution was for 1.21.0
The kernels have moved on versions since then so you need to put in the right version numbers into the script.
If you check the output window it will list in that what packages it tried to download and you can then substitute out the old with the new.
You can also see what it tries to download on the blades which prompt for what Kernel to install and substitute that way.
Appreciate its not a full on fix for the issue but it is still a working, workaround as we have done this again recently.

"" -m pip install --trusted-host=pypi.org --trusted-host=files.pythonhosted.org --trusted-host=prose-python-packages.azurewebsites.net "jupyter>=1.0.0" "pandas>=0.24.2" "sparkmagic>=0.12.9" "prose-codeaccelerator>=1.3.0" "powershell-kernel>=0.1.4" "cryptography==3.2.1" --extra-index-url https://prose-python-packages.azurewebsites.net --no-warn-script-location

@Ant-Green Thanks, that got me closer. I am still not able to install any version of prose-codeaccelerator using this method, but that is beyond the scope of this issue.

Interesting, that worked on a vanilla ADS 1.30.0 install earlier on with no problems installing any of the kernels

BDC support in ADS is being deprecated so closing all related issues.


I meet a cache-related bug

Hello haiyang, thank you for your outstanding work in DSVT and UniTR. I can run unitr+lss.yaml normally, but when I run unitr+lss_caching.yaml, the network was able to calculate the loss, but then an error occurred:

Trying to backward through the graph a second time (or directly access saved tensors after they have already been freed). Saved intermediate values of the graph are freed when you call .backward() or autograd.grad()

This error always occurs in the second iter of the first epoch. Have you ever encountered this bug? It seems to be related to torch. cuda. amp. GradScaler. My GPU is GTX 1080 Ti, is it because it cannot support torch. cuda. amp?

I noticed that only unitr_cache.yaml is used as testing in README. Can you explain in detail the usage and the meaning of this sentence 'You can even cache all mapping calculations during the training phase, which can significantly accelerate your training speed'` Thank you very much.

Sorry for not explaining clearly, "mapping calculations" specifically refers to the image2lidar and lidar2image modules in UniTR, and does not include the cache calculation of lss. This can be achieved by only setting ACCELERATE to True in the mm_backbone part of the config.
Currently, the caching method of lss does not support use during training, because we did not cancel the gradient through torch.no_grad during calculation, resulting in an error in the second iteration of training (the calculation graph of the cache variable in the first iteration has been released after backpropagation).
In addition, UniTR does not support fp16 (mixed precision training), which will cause NaN in the loss calculation.

Sorry for this mistake. We will fix this bug as soon as possible. Now the cache mode only supports the test stage.
Transfusion has problems with fp16 training, try not to use it.

Thank you for your timely response. My understanding is to use unitr+lss.yaml in training and unitr+lss_cache.yaml in testing. Is that correct?

Thank you for your timely response. My understanding is to use unitr+lss.yaml in training and unitr+lss_cache.yaml in testing. Is that correct?

Yeah, there's no problem with that.

Please refer to issue#3, someone has reproduced our codebase. The code seems to be fine. Please feel free to contact me if you have any questions.

When I train using unitr+lss.yaml, can I achieve acceleration during evaluation after completing 10 epochs? I think it should not be possible, right? Because ACCELERATE is not specified as True during network initialization. Acceleration can only be achieved when testing with unitr+lss_cache.yaml.

When I train using unitr+lss.yaml, can I achieve acceleration during evaluation after completing 10 epochs? I think it should not be possible, right? Because ACCELERATE is not specified as True during network initialization. Acceleration can only be achieved when testing with unitr+lss_cache.yaml.

Yeah, you can evaluate the checkpoint trained by unitr+lss.yaml directly by unitr+lss_cache.yaml. ACCELERATE  option has nothing to do with network initialization and training. You can try this based on our provided checkpoint. We also provide the expected performance in cache testing section (-0.7 drop).

Thank you for your timely response! I have no further questions.

Thank you for your timely response! I have no further questions.

Great! Wish you all the best. :)
Haiyang


Don't throw the "empty heading" error when the element uses ng-bind or ng-bind-html

A fix for Issue #11! :tada:

Excellent, thanks!  Could you up the version number and release this to NPM?

Unless I made a mistake, I have already done this. v0.1.8

Oops, my bad!  I actually consume this though this gulp plugin and i forgot for a moment.  I need to ask them consume this new version.

I think if you do npm update you'll just get it.


net: create devnet network

Add a new devnet network, which will be used for local development tests.
-devnet is used to use it.
This network will generate a new genesis block each time the node boots.
Old codebase hints
Look for devnet occurrences in the old codebase for the needed changes, specially (but not exclusively):

src/chainparams.cpp
src/chainparamsbase.cpp/h

not needed, we can use regnet for local dev tasks


Allow for multiple smoothie configurations

Problem 1:
Currently only one smoothie configuration is possible, this PR allows for more than one.
Problem 2:
Sometimes a layout might not be desired, this PR allows the options mix smoothie.compile --no-layout and the template will not be embedded in the layout.
Feature Request 3:
Foundation-emails makes email great and easy... add foundation-email to npm repos and allow the use of inky templating. This is an optional feature and is only activate when passed as a parameter... mix smoothie.compile --foundation
See companion PR in the elixir-smoothie library
https://github.com/jfrolich/elixir-smoothie/pull/2

Thanks for the pull request, will review this soon!

cheers.

I am doing some major refactorings that support these use-cases. Simplified the API a bit, and added tests... Work in progress

Refactored the pull request to facilitate slightly different API.


Non-simple property used in cardinality restriction

MONDO asserts that disease has basis in feature is transitive, and RO asserts that disease has basis in feature is a subproperty of has phenotype. RO also asserts that the inverse of has phenotype, phenotype of is a functional property.
Because of the transitivity of a subproperty of its inverse, phenotype of becomes a "non-simple property", and this is incompatible with being used in a cardinality restriction (i.e. being functional).
The transitivity is not in RO.

there is a not a property called disesae has basis in feature, there is disease arises from feature, is that what you meant @balhoff?

Oh! This property has two labels!

I thought Protege wouldn't let you save the ontology when you have two labels. One of these should be a synonym?

Protege lets you do that but its against best practice in OBO

to do:
remove the label that comes from MONDO and rename the ID to match the RO label

@cmungall the rdfs:label asserted on this property in the MONDO file is "disease arises from feature", rather than "disease has basis in feature" which is in RO. Can you please replace all usages to use only the RO rdfs label? Also fix the id


Add a screen for displaying all trending Movies

As a movie enthusiast,
I want to browse a screen dedicated to trending movies displayed in a grid format with endless pagination
so that I can continuously discover and explore the most popular movies without interruptions.
Acceptance Criteria:

The trending movies screen should be easily accessible from home sections with a "More" button.
Movies should be displayed as thumbnails in a grid format.
Upon scrolling to the bottom of the list, the next set of trending movies should automatically load (endless pagination).
There should be a visual indication (e.g., loading spinner) while new movies are being fetched.
In case of a network error or no more trending movies available, an appropriate message should be displayed to the user.
Tapping on a movie thumbnail should take the user to the detailed movie description screen.

Feel Free to create separate PRs to implement this story

I started working on it.🖐

i am working on this issue

Hi, I'm starting to working on it!
I guess I will create a pr in a couple of days from now!

@Sobhan-mp Thanks


LightNovelPub release date bug

Source information
LightNovelPub (en)
Steps to reproduce

LightNovelPub, add Infinite Mana in the apocalipse to the library
download the last 100 chapters
Turn off internet from the device for a day minimum (absolutely no internet)
Turn on internet and update that entry

Expected behavior
The release date of chapters has a properly working time counter
Actual behavior
Time counter doesnt time count properly

Multiple release dates
https://github.com/LNReader/lnreader/assets/75091899/c14d2df1-861d-47bd-b779-54b1e9f5d6b9
LNReader version
1.1.18
Android version
Android 8
Other details
The reproduction steps are just what i did. I cant garantee it will be reproduced.
I thought it was a problem of the  app but Traitor said to report it here.
Acknowledgements

[X] I have searched the existing issues and this is a new ticket, NOT a duplicate or related to another open issue.
[X] I have written a short but informative title.
[X] I have updated the app to version 1.1.18.
[X] If this is an issue with the app itself, I should be opening an issue in the app repository.
[X] I will fill out all of the requested information in this form.

You should use v2 beta app or wait for its stable release


Feature request: Add refresh token example

It would be great to have the recommended method for the refresh token in the sample code. The online docs aren't exactly clear on what is best practice for an Ionic 2 App.

I need the same


Refactor transpose.js to make it readable

I just changed while-loop to for-loop of transpose.js beause I thinks it looks a little bit more readable. :)

Change while-loop to for-loop of transpose.js
Change var variables to const and let variables

While the Travis build failed because we're still not allowing const and other ES6 syntax into the code before v1.0, the bigger problem is that house style is the while loops.  One of the founding members of Ramda has made it a personal goal to never write another for loop.  And when Ramda started, there was still some truth to the folklore that while loops were more performant.  I think that time has now passed, but you shouldn't find any for loops in the Ramda code.

Not a founding member, but I too will never write a for loop in javascript, and will always suggest an alternative during code review.


Add specific example of building an R container with only R packages

Is your feature request related to a problem? Please describe.
I need to create a container with just R and some R packages. I found this link https://nf-co.re/developers/adding_modules#software-requirements.  But to a beginner (and I know this is a Developer documentation), it is not clear how to build an R specific type of container.
The closest language is in this section:
It is also possible for a new multi-tool container to be built and added to BioContainers by submitting a pull request on their multi-package-containers repository.
However, there are several difficulities for a beginner to understand.  For instance, installing R requires you to know to search for r-base on Anaconda.org. If you just search for R, you will never find it. Also, you need to know that R packages are prefixed with r- and bioconductor with bioconductor-
I have a list of commands that I executed to perform this task. I'm not sure if it is valuable to include as documentation.

I like this idea, I'd say go for it and start a PR, comments are always a good way to improve

Sounds like a great addition! Could have sections for common languages - R, Python etc.. This comes up quite a lot.

Should I write a separate tutorial webpage and add it to: https://github.com/nf-core/nf-co.re/blob/master/markdown/developers/developer_tutorials.md ?

yes that sounds good!

I'm having difficulty writing this tutorial. Allow me to explain.
I created a PR on Biocontainers that was merged recently: https://github.com/BioContainers/multi-package-containers/pull/2107.  Then I ran this command line
mulled-search --destination quay singularity conda --search r-dplyr r-readr r-optparse
to find the name of my mulled image (command is based on the one in https://nf-co.re/developers/modules#software-requirements). Unfortunately, I’m not getting any mulled search results.
However, there is another way to get the mulled image name! But, you have to go through the build logs here:  https://github.com/BioContainers/multi-package-containers/runs/5610436668?check_suite_focus=true. If you look under the "Upload images" it's quay.io/biocontainers/mulled-v2-4ba25ddb980af37c42ada81fe02785bc9ec0a264:fd1896f69951cb685c17367012009385d3356a3d-0.  This method does not seem sustainable but maybe for a developer that is all they need.
Unfortunately, there is more serious problem. When I tested the container with my R script, it failed because of a missing system library for the R readr library.  See this issue https://github.com/tidyverse/readr/issues/952#issuecomment-451255375.
So, this makes things difficult to write an effective tutorial for building an R container.  How am I supposed to know which system libraries are required for every R package? I’m not sure if I can write the tutorial...any ideas?


[Bug]:  (Android) NullPointerException when quickly loading / unloading map

Mapbox Implementation
Mapbox
Mapbox Version
default
Platform
Android
@rnmapbox/maps version
10.0.0-beta.64
Standalone component to reproduce
import React from 'react';
import {
  MapView,
  ShapeSource,
  LineLayer,
  Camera,
} from '@rnmapbox/maps';

const aLine = {
  type: 'LineString',
  coordinates: [
    [-74.00597, 40.71427],
    [-74.00697, 40.71527],
  ],
};

class BugReportExample extends React.Component {
  render() {
    return (
      <MapView style={{flex: 1}}>
        <Camera centerCoordinate={[-74.00597, 40.71427]} zoomLevel={14} />
        <ShapeSource id="idStreetLayer" shape={aLine}>
          <LineLayer id="idStreetLayer" />
        </ShapeSource>
      </MapView>
    );
  }
}

Observed behavior and steps to reproduce
Upon switching to and from a component that has a (partially?) loaded map, the following exception is thrown:
java.lang.NullPointerException
  com.mapbox.rctmgl.components.camera.RCTMGLCamera$mCameraCallback$1.onAnimationEnd(RCTMGLCamera.kt:96)
  com.mapbox.rctmgl.components.camera.CameraUpdateItem$run$callback$1.onAnimationEnd(CameraUpdateItem.kt:61)
  android.animation.Animator$AnimatorListener.onAnimationEnd(Animator.java:552)
  android.animation.AnimatorSet.endAnimation(AnimatorSet.java:1294)
  android.animation.AnimatorSet.doAnimationFrame(AnimatorSet.java:1079)
  android.animation.AnimationHandler.doAnimationFrame(AnimationHandler.java:146)
  android.animation.AnimationHandler.access$100(AnimationHandler.java:37)
  android.animation.AnimationHandler$1.doFrame(AnimationHandler.java:54)
  android.view.Choreographer$CallbackRecord.run(Choreographer.java:947)
  android.view.Choreographer.doCallbacks(Choreographer.java:761)
  android.view.Choreographer.doFrame(Choreographer.java:693)
  android.view.Choreographer$FrameDisplayEventReceiver.run(Choreographer.java:935)
  android.os.Handler.handleCallback(Handler.java:873)
  android.os.Handler.dispatchMessage(Handler.java:99)
  android.os.Looper.loop(Looper.java:214)
  android.app.ActivityThread.main(ActivityThread.java:7050)
  java.lang.reflect.Method.invoke(Native Method)
  com.android.internal.os.RuntimeInit$MethodAndArgsCaller.run(RuntimeInit.java:494)
  com.android.internal.os.ZygoteInit.main(ZygoteInit.java:965)

Please note that this only happens on Android, while on iOS this issue does not occur. It seems that the unmounting of the map creates some kind of problem if the map is asked to mount immediately after again.
The use case where I noticed this problem is:
User switches to map screen, switches away very quickly (before the map has finished fully initializing), and then opens the map screen again. The app will crash fully with the above error.
I've also seen this behavior happen in a non "react-navigation/tabs" navigator. Just a simple component mount / unmount also triggers this error. (For example someone presses a button to open a map sheet, closes it and pressed the button quickly again).
Expected behavior
The expected behaviour is that the map does not throw the exception and unmounts/mounts correctly, just like it seems to do on iOS, where I haven't managed to recreate this bug in both an emulator and/or physical device.
Notes / preliminary analysis
It seems like it's Android version / phone independent, I've had multiple people with different phones and releases of android have the bug happen. While testing on my S8+ running Android 9 pie, the problem also occurs. However, I have not managed to trigger it in my Android emulator (yet). It could simply be that the emulator is "too slow" to make this quick switch bug happen.
It also seems like that the problem is not related to the tab navigator for react native (which was a previous problem where the app would hang/crash after switching away from the maps tab for android). The problem also occurs in "generic" view mounts/unmounts.
Additional links and references
As it occurs on my physical android device (sorry for no screen cap, old s8 didn't like it :()
https://user-images.githubusercontent.com/13538135/214840315-05ae0f34-dec2-43aa-b922-35a0995bcb47.mp4
As it should be on my physical iPhone (12 Pro Max)
https://user-images.githubusercontent.com/13538135/214839924-b298906d-2c7e-466d-bd0a-6b35003d4bf6.MP4

@Arilith can you change it to a component that reproduces the issue without user interaction? Like using setTimeout to hide/show the map?

@Arilith can you change it to a component that reproduces the issue without user interaction? Like using setTimeout to hide/show the map?

@mfazekas Updated the reproduction component. I tested it on my S8 and it triggered the crash.


provide data for returns chart widget

Financing fee: data item on pool level

DROP APR: data item on pool level

DROP APY: see below screenshot

TIN APY: see below screenshot

Cash drag TIN: Liquidity reserve TIN / Pool value TIN

Cash drag DROP: Liquidity reserve DROP / Pool value DROP

Cash drag dilution TIN: Cash drag TIN * Financing fee

Cash drag dilution DROP: Cash drag DROP * DROP APR

Default rate TIN: Defaults TIN / TIN NAV

Default rate DROP: Defaults DROP / DROP NAV

Based on call with Dennis on 25th July we agreed the following:
We will not calculate cash drag and cash drag dilution on tranche level, as they do not allocate the cash drag based on tranche size (% of NAV). The reserve data items in subql on tranche level are available, but only used as buckets to calculate overall cash drag according to Dennis. Agreed procedure:

In returns widget we will show Financing fee (nominal) (gross of cash drag) and Financing fee (effective) as well as APR (nominal) and APR (effective), where the effective values will be adjusted for the liquidity reserve held within a pool (e.g. 10% financing fee (nominal); 20% Liquidity reserve -> 8% Financing fee (effective)


Set HOME environment variable

It is necessary to set the HOME environment variable when running a dotnet
core site through supervisor. If it is not set, the following exception is
thrown:
Unhandled Exception: System.ArgumentNullException: Value cannot be null.
Parameter name: path1
   at System.IO.Path.Combine(String path1, String path2, String path3)
   at Microsoft.DotNet.ProjectModel.Resolution.PackageDependencyProvider.ResolvePackagesPath(String rootDirectory, GlobalSettings settings)
   at Microsoft.DotNet.Configurer.NuGetCacheSentinel.get_NuGetCachePath()
   at Microsoft.DotNet.Configurer.NuGetCacheSentinel.Exists()
   at Microsoft.DotNet.Configurer.DotnetFirstTimeUseConfigurer.ShouldPrimeNugetCache()
   at Microsoft.DotNet.Configurer.DotnetFirstTimeUseConfigurer.Configure()
   at Microsoft.DotNet.Cli.Program.ConfigureDotNetForFirstTimeUse(INuGetCacheSentinel nugetCacheSentinel)
   at Microsoft.DotNet.Cli.Program.ProcessArgs(String[] args, ITelemetry telemetryClient)
   at Microsoft.DotNet.Cli.Program.Main(String[] args)

I guess that path1 in this case is the value of the HOME variable (which is
null unless set in supervisord.conf) and it seems to be used for the
default nuget package path for the www-data user. By including HOME as part
of the environment configuration, this expection is no longer thrown and the
site starts up as expected.

Hi @tmadsen, I'm your friendly neighborhood .NET Foundation Pull Request Bot (You can call me DNFBOT). Thanks for your contribution!

This seems like a small (but important) contribution, so no Contribution License Agreement is required at this point. Real humans will now evaluate your PR.

TTYL, DNFBOT;

@shirhatti can you review?

@glennc thoughts on this?

The HOME environment variable sets the root directory for the supervisor conf.  This is good. :shipit:


PowerToys Run fails to find VS Code Insiders and instead finds uninstalled stable

Environment
Windows build number: 10.0.19041.264
PowerToys version: 0.18.2
PowerToy module: PowerToys Run

Steps to reproduce
Install PT, install VS Code, then install VS Code Insiders, and then uninstall VS Code.
In start menu, I also renamed VSC Insiders from Visual Studio Code - Insiders to Visual Studio Code Insiders.
Expected behavior
PowerToys Run should not find VS Code anymore and instead find VS Code Insiders.
Actual behavior
PT Run finds VS Code which then fails to launch because it's not installed anymore. VS Code Insiders isn't found at all, not with 'Insiders', not with 'Visual Studio Code' etc.

Deleting %localappdata%/Microsoft/PowerToys/PowerToys Run fixed this issue.

This also fixed my issues with not finding slack, vs code, telegram or toggle.
But it'd be nice to know why and it still seems like there's a bug with something.


(BOLT-482) Bump puppetlabs-service to 0.3.0

Brings in agentless tasks for Windows and Linux.

CLA signed by all contributors.


Implement autofocus

Design document: https://docs.google.com/document/d/1odtedFcDg-F-DLUw-H9W37LWxCJxDzyYvsMi6jq1DrY/edit?usp=sharing

Autofocus support was added in adc0653a.