Connect the Corona test center in Munich (Theresienwiese) to the Corona-Warn-App System

Current Implementation
Currently the COVID-19 test center in Munich, located on the Theresienwiese, seems to be not connected to the CWA infrastructure. If you are tested there, you get a sheet of paper and the number of your test to find out your test result on https://www.doctorbox.de/covid19.jsp.
A QR-Code for CWA is never issued.
Suggested Enhancement
Connect the lab/test center to the CWA infrastructure so that users of the Corona-Warn-App can get the test result in the App.
Expected Benefits
More people who got tested there and their test came back positive will be (more easily) able to share the test result.
Additional Information
I thought about sending an e-mail to<EMAIL_ADDRESS>but decided against it since this is not about i.e. that one test result wasn't delivered to the app or similar, it's a more general problem.
The website of the test center is: https://www.corona-testung.de/ (German only).
I will ping @GisoSchroederSAP here since it seems like he is responsible for such things.

@Ein-Tim
Thanks, I forwarded the info internally.

@Ein-Tim
Thanks, I forwarded the info internally.

@Ein-Tim Update: They are aware of that specific case and working on it.

@Ein-Tim Update: They are aware of that specific case and working on it.

@dsarkar is there a centralized place to report cases like this? I know of two test centres which are also not connected to the CWA infrastructure.

@dsarkar is there a centralized place to report cases like this? I know of two test centres which are also not connected to the CWA infrastructure.

@achisto
There is a dedicated issue for this on "Connectivity to laboratories / Transfer of QR code / QR code not working" https://github.com/corona-warn-app/cwa-documentation/issues/400
The e-mail address<EMAIL_ADDRESS>has also been published for feedback on QR code issues.

@achisto
There is a dedicated issue for this on "Connectivity to laboratories / Transfer of QR code / QR code not working" https://github.com/corona-warn-app/cwa-documentation/issues/400
The e-mail address<EMAIL_ADDRESS>has also been published for feedback on QR code issues.

@achisto
This is the text the user will get in the AppStores regarding lab connectivity issues:

Vielen Dank für Ihr Feedback. Die automatische Benachrichtigung für das Testergebnis nach Registrierung des QR-Codes funktioniert noch nicht reibungslos.  Testcenter/Gesundheitsamt an, um Ihr Testergebnis zu erhalten.   Je mehr Informationen wir erhalten, desto besser können wir diesen Prozess korrigieren und die Qualität der Corona-Warn-App weiter verbessern.

So, yes<EMAIL_ADDRESS>is the best option I think. It might take a while, but my understanding is that they will look into every single case and answer you.
Thank you for reporting!

@achisto
This is the text the user will get in the AppStores regarding lab connectivity issues:

Vielen Dank für Ihr Feedback. Die automatische Benachrichtigung für das Testergebnis nach Registrierung des QR-Codes funktioniert noch nicht reibungslos.  Testcenter/Gesundheitsamt an, um Ihr Testergebnis zu erhalten.   Je mehr Informationen wir erhalten, desto besser können wir diesen Prozess korrigieren und die Qualität der Corona-Warn-App weiter verbessern.

So, yes<EMAIL_ADDRESS>is the best option I think. It might take a while, but my understanding is that they will look into every single case and answer you.
Thank you for reporting!

So, yes<EMAIL_ADDRESS>is the best option I think. It might take a while, but my understanding is that they will look into every single case and answer you.

@achisto I can confirm that this is a good email. I recently used it for a similar issue. I got immediate reply from a robot, but shortly after this (I think even the same day) I got a nice reply and feedback. And I am sure they collect and apppreciate all kind of informations that helps them to address this issues and to improve integration of labs and test centers with cwa.

So, yes<EMAIL_ADDRESS>is the best option I think. It might take a while, but my understanding is that they will look into every single case and answer you.

@achisto I can confirm that this is a good email. I recently used it for a similar issue. I got immediate reply from a robot, but shortly after this (I think even the same day) I got a nice reply and feedback. And I am sure they collect and apppreciate all kind of informations that helps them to address this issues and to improve integration of labs and test centers with cwa.

@dsarkar

They are aware of that specific case and working on it.

Please keep us (me) in the loop, since I don't get tested every day I can not confirm when they are connected 😉
Thanks! 🙂

@dsarkar

They are aware of that specific case and working on it.

Please keep us (me) in the loop, since I don't get tested every day I can not confirm when they are connected 😉
Thanks! 🙂

All,
please note, we follow up on every single case of known process disruption during the Corona testing. This includes delayed and even missing test results in the app. Meanwhile, based on multiple hundreds of investigated issues we found more than 20 reasons that lead to disappointment of users (and of our team as well). The missing technical connection of a lab to the CWA infrastructure is only one of those 20+ root causes.
However, following up each and every single case allows us to "cluster" the issues and to work on strategies to resolve the one or the other cluster. Additionally, we are currently enhance the process monitoring capabilities to get informed as early as possible if the process chain between test registration (scan) and delivery of result (into the app) gets stuck.
For the initial case: Munich Theresienwiese is a big player performing the Corona tests with at least two labs. Unfortunately, it seems the rich southern German federal states have high medial visibility and but low impact to make the lab facilities supporting the CWA. A big facility chain, with locations in Konstanz, Erlangen, Gelsenkirchen, Ebersberg and Munich still ha no major interest in extra effort to support the CWA test result process. Instead, users get redirected to other websites (like doktorbox.de or mein-laborergebnis.de).
We are still trying to continue to evaluate the recurring cases and to fix them.

All,
please note, we follow up on every single case of known process disruption during the Corona testing. This includes delayed and even missing test results in the app. Meanwhile, based on multiple hundreds of investigated issues we found more than 20 reasons that lead to disappointment of users (and of our team as well). The missing technical connection of a lab to the CWA infrastructure is only one of those 20+ root causes.
However, following up each and every single case allows us to "cluster" the issues and to work on strategies to resolve the one or the other cluster. Additionally, we are currently enhance the process monitoring capabilities to get informed as early as possible if the process chain between test registration (scan) and delivery of result (into the app) gets stuck.
For the initial case: Munich Theresienwiese is a big player performing the Corona tests with at least two labs. Unfortunately, it seems the rich southern German federal states have high medial visibility and but low impact to make the lab facilities supporting the CWA. A big facility chain, with locations in Konstanz, Erlangen, Gelsenkirchen, Ebersberg and Munich still ha no major interest in extra effort to support the CWA test result process. Instead, users get redirected to other websites (like doktorbox.de or mein-laborergebnis.de).
We are still trying to continue to evaluate the recurring cases and to fix them.

Unfortunately, it seems the rich southern German federal states have high medial visibility and but low impact to make the lab facilities supporting the CWA.

@GisoSchroederSAP I suggest that you discuss these topics with our Bavarian minister of health, and convince him of the strategic importance of cwa and of the seamless integration of cwa with test center and labs infrastructure (and maybe you are already in this discussion) 😀
Here in the rich South, we have setup very good and low-threshold testing services for everybody, and we are proud of this. I remember a comment of a not-so-lucky cwa user living in the North, who was eagerly waiting for his red cwa warning so that he could get a test. Here in my county I don't need to wait; I just show up at the test center, self-registrate, and get the test. The process is fully digitized and fast; I got my result after only 15 hours via password-protected email. This is the fastest possible, cwa just cannot speed this up.
But the part which is indeed disrupted is not the delivery of test result to the user, but feeding a positive result back into cwa infrastructure and warning others.
This is what you need to use as arguments. Instead of just pressing a few buttons, the user must call an overloaded hotline to get the TeleTAN (and maybe not everybody knows this, or they don't take the time, or ... - you are better aware than me of the shortcomings of this TeleTAN path).

Unfortunately, it seems the rich southern German federal states have high medial visibility and but low impact to make the lab facilities supporting the CWA.

@GisoSchroederSAP I suggest that you discuss these topics with our Bavarian minister of health, and convince him of the strategic importance of cwa and of the seamless integration of cwa with test center and labs infrastructure (and maybe you are already in this discussion) 😀
Here in the rich South, we have setup very good and low-threshold testing services for everybody, and we are proud of this. I remember a comment of a not-so-lucky cwa user living in the North, who was eagerly waiting for his red cwa warning so that he could get a test. Here in my county I don't need to wait; I just show up at the test center, self-registrate, and get the test. The process is fully digitized and fast; I got my result after only 15 hours via password-protected email. This is the fastest possible, cwa just cannot speed this up.
But the part which is indeed disrupted is not the delivery of test result to the user, but feeding a positive result back into cwa infrastructure and warning others.
This is what you need to use as arguments. Instead of just pressing a few buttons, the user must call an overloaded hotline to get the TeleTAN (and maybe not everybody knows this, or they don't take the time, or ... - you are better aware than me of the shortcomings of this TeleTAN path).

Yes, we are aware, that there are various spots where we need to pay extra attention.
Together with RKI we also created this picture that explains the behavior quite user-friendly (and has sufficient resolution to get printed as a banner or poster).
However, our observation also is, that there are some cases where the test result is delivered at a certain day, but it appears in the app up to 5 days later. Strange, so we need to investigate there as well.  Those delays are unacceptable, from the user perspective and as well from our perspective.

@GisoSchroederSAP actually, I saw your poster at my local test center when I showed up last week for a test 😀 and I wanted even to take a photo 😀 but I saw it too late and the waiting line was very short and there was no time 😀
So my local test center is meanwhile integrated with cwa (and it is not Theresienwiese).
Still I would like to give feedback on the part of the process / data flow that was most diruptive in my case.
The process at my local test center is fully digitized, even the cwa QR code I got via email as link to a website where it was shown as png.
But cwa wants to scan it; so I need either a printer or a second device.
See also suggestions for improvements #227, #302, and #389

A Twitter user told me that he had a PCR test done at the test center on the Theresienwiese and he got his test result via the Corona-Warn-App.
So it seems like that the test center is now connected to the CWA infrastructure.
@GisoSchroederSAP Could you maybe cross check this, please?

Unfortunately, I am not able to cross-check. However, Theresienwiese will be served by one of the labs of the Eurofins group. As far as we are informed, all labs of this group in Germany deliver their PCR test results into one central location (Konstanz), that finally consolidates the data and is connected since February to the CWA infrastructure. After initial difficulties with the technical setup and reliable data exchange it should work now continuously.

@GisoSchroederSAP Thank you, so I am closing this issue now because it seems like they are connected to the CWA System now. I'll cross check the next time I do a PCR test there.
Closing as implemented.

I'm totally aware of the difficulties that come with the fulli-digitalized process, where you don't even get something to scan, but just a link to a QR code or an email carrying a QR code.
We addressed this topic already multiple times. There is a slight chance, it becomes added if (and only if) we are permitted to go for a face-lifting of the app. Main objective is to optimize the user experience, to streamline navigation and to make the UI more consistent on the way it works.

@GisoSchroederSAP I assume you wanted to comment in #389 or was this comment intended to be in this thread?

Unfortunately, I am old and was too slow when you closed this ticket.
As @ndegendogo mentioned this issue here above, this response should stay here, but certainly also matches #389. Shall I copy/paste it there as well - or will you just link my comment into the other ticket?

@GisoSchroederSAP
I'll link to your comment. Thanks for the update!


Add DS record support

Add DS record support
Confirmed code coverage
Tested against AWS Route53 account: Create, Update and Delete.  Single and multiple values.

Got it.  Thanks @ross.


findfile returns nonexistent paths when editing protocol paths

Steps to reproduce

gvim --clean c:/scratch/file.txt
cd c:/scratch
write
edit res://cars/dummy.txt
echo findfile("file.txt", ".;")
echo filereadable(findfile("file.txt", ".;"))

Vim outputs a nonsense path that is not readable:
res://cars/file.txt
0

Expected behaviour
If vim finds a real file, it should output a readable path to it (c:/scratch/file.txt).
If you replace step edit res://cars/dummy.txt with edit dummy.txt then everything works as expected. I don't see why vim should report paths that don't exist when it clearly knows the actual valid path.
I ran into this issue in trying to remap paths prefixed with res://.
Version of Vim
8.2.3399
Environment
Windows 10 21H2
Gvim
Logs and stack traces
No response

Steps to reproduce

gvim --clean c:/scratch/file.txt
cd c:/scratch
write
edit res://cars/dummy.txt
echo findfile("file.txt", ".;")
echo filereadable(findfile("file.txt", ".;"))

Vim outputs a nonsense path that is not readable:
res://cars/file.txt
0

Expected behaviour
If vim finds a real file, it should output a readable path to it
(c:/scratch/file.txt).

Yeah, when finding files it knows that URLs can't be checked for
existence, it accepts any URL as existing.
I suppose that findfile() could completely ignore URLs, assuming they
never exist.  However, in your example it would then never find a match,
is that useful?  I can't think of a way how passing a URL to findfile()
(either explicitly or as the directory of the current file) would ever
find a match.  Thus it would change it from always returning the first
match to not returning anything.  That doesn't appear to be useful.
--
login: yes
password: I don't know, please tell me
password is incorrect
login: yes
password: incorrect
/// Bram Moolenaar -- @.*** -- http://www.Moolenaar.net   \
///                                                                      \
\\        sponsor Vim, vote for features -- http://www.Vim.org/sponsor/ ///
\\            help me help AIDS victims -- http://ICCF-Holland.org    ///

I can't think of a way how passing a URL to findfile()
(either explicitly or as the directory of the current file) would ever
find a match.

I see now. findfile("file.txt", ".;") is implicitly findfile("file.txt", expand("%:h")..";") so it expands to the url and findfile("noexist.txt", "res://cars;") doesn't do any finding -- it just returns the file in the it acts as if the file exists in the first searched directory.

Thus it would change it from always returning the first
match to not returning anything.  That doesn't appear to be useful.

It seems more correct to me that it returns nothing (since it didn't even try to find) so we can assume findfile only returns valid paths. What is the invalid path useful for if we have no evidence that it exists? Is there some way where findfile does work with protocols?
I think this code should be sufficient error checking:
let f = findfile("file.txt", ".;")
if empty(f)
    return "error message"
endif
return readfile(f)

Currently, you need to use filereadable instead of empty.

Assuming it is never valid (does not exist) has the problem that you cannot use the
functions to find other matches.

I feel like the current behaviour is the opposite. Since urls match everything, they block valid matches from being discovered and you have to look past the first result:
:edit /scratch/hello.txt
:write /scratch/file.txt
:ec findfile('file.txt', 'res://;,.;')
res://file.txt
:ec findfile('file.txt', 'res://;,.;', 2)
\scratch\file.txt
:ec findfile('file.txt', '/;,.;')
\scratch\file.txt

Doesn't that seem like more reason to consider always consider URL paths to be "not found"?

Assuming it is never valid (does not exist) has the problem that you
cannot use the functions to find other matches.

I feel like the current behaviour is the opposite. Since urls match
everything, they block valid matches from being discovered and you
have to look past the first result:
:edit /scratch/hello.txt
:write /scratch/file.txt
:ec findfile('file.txt', 'res://;,.;')
res://file.txt

Why would you have "res://" in the path?  I cannot think of a reason.

:ec findfile('file.txt', 'res://;,.;', 2)
\scratch\file.txt
:ec findfile('file.txt', '/;,.;')
\scratch\file.txt

Doesn't that seem like more reason to consider always consider URL
paths to be "not found"?

--
How To Keep A Healthy Level Of Insanity:
7. Finish all your sentences with "in accordance with the prophecy".
/// Bram Moolenaar -- @.*** -- http://www.Moolenaar.net   \
///                                                                      \
\\        sponsor Vim, vote for features -- http://www.Vim.org/sponsor/ ///
\\            help me help AIDS victims -- http://ICCF-Holland.org    ///

Godot (the game engine) scripts sometimes contain lines like:
extends "res://cars/driveable.gd"

If my project is in c:/project, then that line refers to the path c:/project/cars/car_base.gd. That line is analogous to #include "driveable.h" in C --  a reference from one code file to another.
When I use vim to edit Godot code, I want to gf to jump to that file. I tried changing 'includeexpr', but it seems that protocol paths are always "valid" so it's never invoked.


Open by default the settings row

Hello Open by default the settings row on page ?
Thanks for help.

Closing old issue. Please reopen if a bug or feature request.


Improve type annotations and PHPDocs.

This PR adds improved type support and PHPDocs for the library.

Read here:
https://github.com/HPWebdeveloper/laravel-pay-pocket/pull/16


When the next release?

There been a few good PR that been merged, when would it be a good time to do another release?
p.s: I would prefer not to track master branch

v5.0.2 was released 3 days ago. Hope that helps!


Semicolon as chunks delimiter in js tech

Currently enb uses justJoinFilesWithComments in js and js-i18n techs to concatenate javascript sources. This function separates files with /*begin*/ and /*end*/ markers.
Other build systems use semicolon as delimeter, when dealing with js files:

bem-tools  https://github.com/bem/bem-tools/blob/support/0.9.x/lib/techs/js.js
grunt suggests to use semicolon as separator of js files https://www.npmjs.com/package/grunt-contrib-concat#separator

The reason for using semicolon is clear: we prevent content of some file to be treated as part of function call
Consider following example: lets say, we have 2 files with following content:
file1.js
(function(){/*Some code in IIF*/}())

file2.js
(function(){/*Another code in IIF*/}())

Concatenation with semicolon would work as expected
(function(){/*Some code in IIF*/}());(function(){/*Another code in IIF*/}())

Whereas concatenation without semicolon
(function(){/*Some code in IIF*/}())(function(){/*Another code in IIF*/}())

would produce error
As a workaround, I now use js-with-semicolons tech which is exactly like js tech, with exception of that  .justJoinFilesWithComments() call is replaced with
.justJoinFiles(function (filename, data) {
    var fn = this.node.relativePath(filename);
    return '/* begin: ' + fn + ' *' + '/\n' + data + '\n;/* end: ' + fn + ' *' + '/';
})

But, I belive that everyone would benefit from having above change "out of the box".
The question is: would PR with the described change be accepted, or would it be recognized as breaking change?

justJoinFilesWithComments can be used for both JS and CSS right now. Your change will break CSS support.

justJoinFilesWithComments should be left as is. I propose changes in js.js and js-i18n.js

@hsalkaline use browser-js tech from enb-js package with iife option.


XML libraries alternatives

"The JSON and XML libraries are also fully pluggable". What are the alternatives for parsing XML responses ? I found google-http-client-xml which uses MXParser pretty buggy. For example if my element text contains a large [!<CDATA[ -- large text -- ]]> the cdata is not fully returned.
Could you also post examples of how to use them ?

I don't have any examples of using XML or any other pluggable implementation. If you have a parsing library in mind, I am more than willing to help you implement ObjectParser with that library.
My XML knowledge is pretty foggy, am I remembering correctly that there are subtleties parsing CDATA blocks; like you have to buffer the whole thing before you start parsing? Is that what MXP is tripping over?

In a stackoverflow post, Sean Vieira explains that "CDATA stands for Character Data and it means that the data in between these strings includes data that could be interpreted as XML markup, but should not be."
So when it's about  --text here--  blocks you just want to get the text inside it. What MPX does now is that it returns only a part of the text not the entire one. The problem is here:
    public String getText()     {         if(eventType == START_DOCUMENT || eventType == END_DOCUMENT) {             //throw new XmlPullParserException("no content available to read");             //      if(roundtripSupported) {             //          text = new String(buf, posStart, posEnd - posStart);             //      } else {             return null;             //      }         } else if(eventType == ENTITY_REF) {             return text;         }         if(text == null) {             if(!usePC || eventType == START_TAG || eventType == END_TAG) {                 text = new String(buf, posStart, posEnd - posStart);             } else {                 text = new String(pc, pcStart, pcEnd - pcStart);             }         }         return text;     }
The value of the posEnd variable is wrong. My text has more than 1000 characters and posEnd is 343.
Later edit: Oh. I think i got it. The eventType in the above code is 4 which stands for text. Shoudn't it be 5 ?
/**
 * Character data was read and will is available by calling getText().
 *
Please note: next() will
 * accumulate multiple
 * events into one TEXT event, skipping IGNORABLE_WHITESPACE,
 * PROCESSING_INSTRUCTION and COMMENT events,
 * In contrast, nextToken() will stop reading
 * text when any other event is observed.
 * Also, when the state was reached by calling next(), the text value will
 * be normalized, whereas getText() will
 * return unnormalized content in the case of nextToken(). This allows
 * an exact roundtrip without chnanging line ends when examining low
 * level events, whereas for high level applications the text is
 * normalized apropriately.
 *
 * @see #next
 * @see #nextToken
 * @see #getText
 */
int TEXT = 4;

/**
 * A CDATA sections was just read;
 * this token is available only from calls to nextToken().
 * A call to next() will accumulate various text events into a single event
 * of type TEXT. The text contained in the CDATA section is available
 * by callling getText().
 *
 * @see #nextToken
 * @see #getText
 */
int CDSECT = 5;

I didn't had time for that. I parsed it a as string and then I threw it in an XML parser implemented by myself.

Alright. Give us a holler if you need anything else.


[WIP] Use helm-toolkit for DB initialization

This PS enables shipyard and airflow to use newly created
helm-toolkit for postgre DB initialization.

Change-Id: I093671f9bce747b491f22dd8f38f597bd9dae9af

Thank you for contributing to openstack/airship-shipyard!
openstack/airship-shipyard uses Gerrit for code review.
If you have never contributed to OpenStack before make sure you have read the
getting started documentation:
https://docs.openstack.org/infra/manual/developers.html#getting-started
Otherwise please visit
https://docs.openstack.org/infra/manual/developers.html#development-workflow
and follow the instructions there to upload your change to Gerrit.


Monitor condition number of the Jacobian

I asked how to do this on Slack and it seems there's a way for us to redefine a SNES Monitor:
"
Julian Andrej [3:54 PM]
"I'd implement a SNES Monitor for that
You can then access the jacobian after every nonlinear iteration
so if you have
solver = NonlinearVariationalSolver(...)
mysnes = solver.snes

def monitor(snes, its, fgnorm):
  myjacobian = snes.jacobian

snes.setMonitor(monitor)

myjacobian then contains a Petsc Mat
"

So in the Model class in fempy/model.py we can define something like
def init_solver(self, solver_parameters = {
            "snes_type": "newtonls",
            "snes_monitor": True,
            "ksp_type": "preonly",
            "pc_type": "lu",
            "mat_type": "aij",
            "pc_factor_mat_solver_type": "mumps"}):

        self.solver = fe.NonlinearVariationalSolver(
            self.problem, solver_parameters = solver_parameters)

        def monitor(snes, its, fgnorm):

            J = snes.getJacobian()[0]

            condition_number = scipy.sparse.linalg.cond(J)

            print("Jacobian condition number = " + str(condition_number))

        self.solver.snes.setMonitor(monitor)

but scipy.sparse.linalg doesn't actually have cond like numpy.linalg does, numpy doesn't understand sparse matrices, and also I don't know if the sparse PETSc Mat can be understood by scipy, though I'm sure others are also converting between these.


Caching on CI seems to be broken

Last few times, the cache was 45 bytes...  Something's definitely quite broken. I'd appreciate some help, as my knowledge in this area is thin (and that's a generous description.)

Still is broken. It's causing extra slowness in being able to accept PRs.

Might be related to this issue. And also getting various warnings. Latest cache was also bogus.

Maybe related to https://github.com/actions/cache/issues/624?  Also, maybe the cache action should be updated to v3?

Plausibly the issue is that cabal seems to now be installing to ~/.local/state/cabal instead of ~/.cabal?

Thanks - I'll investigate all those.

Didn't mean to close this quite yet, I guess I used the 'wrong' words in my commit!

Looks like you might also need to pass --bindir explicitly now, or else cache ~/.local/bin/, because it seems that cabal is stuffing a symlink to the executables in ~/.local/bin, and there aren't anywhere else in the cabal tree except /home/runner/.local/state/cabal/store/ghc-8.6.5/Agda-2.6.3-c7bd11606a8a3dcf5f3eb623c3877b0e1a7a34d151782f77a271cd5a0e48afca/bin/agda?

Looks like this is now fixed. I'll wait for an independent PR to verify this before closing.

I merged master into #382. Looks like CI is still not working on PRs

We've got partial success: the cache is there and has lots of stuff in it, and it is picked up. Part-way there.
Now to make sure the right things are in the path! Looks like various pieces disagree as to where to expect things to be.

Still not working, sadly. I'm going to have to spend real time on it and test locally first.

Replace echo "~/.local/bin" with echo "$HOME/.local/bin" when saving the path.  Pass --installdir explicitly to cabal install and make sure it's a directory that's cached.  Otherwise you can look at what I had to change to get https://github.com/JasonGross/lawvere/blob/main/.github/workflows/ci.yml working (which I pseudorandomly based off the CI in this repo)


Allow to add version metadata check rules before triggering a release

Ref: https://github.com/biothings/biothings.api/issues/284

@sengineer0 the implementation looks good to me. Just one question, if A hub instance needs to override this method with its own validation rules in a AutoHubFeature subclass, is there an easy way to specify to use this new subclass?

@sengineer0 the implementation looks good to me. Just one question, if A hub instance needs to override this method with its own validation rules in a AutoHubFeature subclass, is there an easy way to specify to use this new subclass?

@newgene as I see, currently we doesn't support an easy way to specify subclass of AutoHubFeature, when hub configures autohub feature, it always import and instantiate  the biothings.hub.standalone.AutoHubFeature
I suggest that, we can add a config, to allow specify what class to use as autohub manager, default can be the biothings.hub.standalone.AutoHubFeature
What do you think ?

@sengineer0 every hub instance will have this bin/hub.py, where we typically customize a hub. If you can see an easy way to use a AutoHubFeature subclass there, we should be good. A customization example in that hub.py is the use of MyChemDataBuilder for that hub.

I will merge this PR first, but please update this PR comment (or code) if you have more info regarding how to use it.


possibility to disable edit template version

To ensure stability we would like to disable editing feature in templateSpecs. We could provide additional parameter by creating a templaceSpec "-edit $false"

This seems more of an RBAC action by defining and assigning role(s) based on
Microsoft.Resources/templateSpecs/read|write|delete
Microsoft.Resources/templateSpecs/versions/read|write|delete

There aren't any specific roles defined yet but nothing that can't be handled with a custom role
https://docs.microsoft.com/en-us/azure/role-based-access-control/resource-provider-operations

we want to allow people to create a new versions, and we don't want to stop them from doing that. But we want to be sure that new versions are created, instead of overwriting existing one. Just say so that we have a stable version 1.0 and if someone tries to update the same version will be denied instead of prompt to overwrite
Template Spec version '1' already exists and this action will overwrite existing data for this version. Are you sure you'd like to overwrite existing Template Spec version data for Template Spec 'a' version '1'?

my question is: how to disable overwriting existing version?

There is no way to disable it today so that we can allow for hotfixes if needed. That being said, the ask makes sense. We will discuss on our side to see if we could add a property that would disable overwrites. cc @apclouds

Understand
We mitigated this risk by only allowing our pipeline to write templateSpecs
Everyone else has read except if you've been granted specific RBAC rights
The version is either

while in development

the name of the branch

released via PR

the major.minor.0 (tagged) semantic version
we can redeploy by tag if we suspect the version was compromised

theoretically speaking, we also have a pipeline that creates a new template versions and that just works. But in reality few people have direct access to template specs...


create CODEOWNERS

Create a code owners list to enforce PR reviews by the people listed in https://github.com/HumanCellAtlas/dcp2/blob/main/docs/dcp2_operating_procedures.rst#designated-reviewers.
This is in conjunction with employing branch protection rules: mandatory PRS, min required reviews, reviews by code owners required
see related dcp2 spec update HumanCellAtlas/dcp2#65

@NoopDog can you please review this?


Query parameter resolved wrong via image api proxy

Observed/problematic behavior
I use shields.io to create badges of the latest version and other stuff. However the badges are resolved via a proxy I guess.
The image url for the badge itself is https://img.shields.io/nexus/maven-releases/de.eldoria.schematic-sanitizer/api?label=Release&logo=Release&server=https%3A%2F%2Feldonexus.de&style=for-the-badge, which works totally fine.
However once posted it gets wrapped by hangars image api resulting in
https://hangar.papermc.io/api/internal/image/https://img.shields.io/nexus/maven-releases/de.eldoria.schematic-sanitizer/api?label=Release&logo=Release&server=https%3A%2F%2Feldonexus.de&style=for-the-badge
This looks right for me at least, but I am not a web dev. However if you look at the image behind the api you get an error from shields.io that the server query parameter could not be resolved.
Expected behavior
That the shield.io image is displayed as it is given in the link embedded into the markdown
Steps to reproduce
embed the link https://img.shields.io/nexus/maven-releases/de.eldoria.schematic-sanitizer/api?label=Release&logo=Release&server=https%3A%2F%2Feldonexus.de&style=for-the-badge into an markdown image tag
![image](https://img.shields.io/nexus/maven-releases/de.eldoria.schematic-sanitizer/api?label=Release&logo=Release&server=https%3A%2F%2Feldonexus.de&style=for-the-badge)
The issue can be seen here as well https://hangar.papermc.io/Eldoria/SchematicSanitizer. i copied the badge url and formatting from GitHub to verify that it is indeed different behaviour: https://github.com/eldoriarpg/schematic-sanitizer
Other
No response

Thanks for the quick fix ♥


Update README.md

Update test instructions in the README.md
In general, the section should be more compact now and will not provide information on the options that cargo offers. For this, the cargo documentation provides a more up to date description.
REDIS_URI is the only environment variable that needs to be set for the tests to execute currently.

README.md preview – https://github.com/gnosis/safe-client-gateway/blob/update-tests-instructions/README.md

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 88.344%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
11126

Relevant Lines:
12594

💛  - Coveralls


lapce: Update to version 0.1.0, switched from wgpu to opengl

Release Notes: https://github.com/lapce/lapce/releases/tag/v0.1.0
TL;DR

switched from WGPU to OpenGL for better compatibility
subpixel text rendering
WSL remote development support
syntax highlighting for new languages

[x] I have read the Contributing Guide.

/try

/try

/verify

/verify

The package was automatically updated by our Excavator bot, see Actions for details.
I will still merge this PR (as a mark), so that it is easier to track for potential problems in the future.


Improve location handling

also use network provider
filter new locations by quality before using them
only submit "precise" locations to server
some cleanup and refactoring

All the parameters used to filter the location and the definition of what we consider precise should be okay for a start. I'd appreciate it if you could take a look and do a quick test yourselves to verify and maybe correct me ;)

Looks good on a first glance - but want to give it a bit more attention before merging

I am for merging this but would be great to merge it into development branch for beta-testing in play - @stephanlindauer has set up open-beta on play

development branch is created. i'd suggest that we stick to the git flow rules when doing things on those different branches. http://nvie.com/posts/a-successful-git-branching-model/

:+1:  @cbalster can you redirect your PR

replaced by #123


PLAT-16501 moon.Input provides validator to prevent inputting invalid number

Issue
This is new UX requirement in 2017 TV.
User can input the invalid value out of correct range. So moon.input provides validator to prevent inputting invalid number.
Fix
I added published variables(min, max) to make range from default values.
Also I implemented change handler to make it dynamically.
If moon.input has number type, min and max value, contexturePopup is shown by validity when user input the wrong number.
Enyo-DCO-1.1-Signed-off-by: Sangwook Lee<EMAIL_ADDRESS>

Please create one more PR or 2.6.0-dev.

It looks like the popup is only show when input changes.
Hidden on blur and not shown on focus.


sync with upstream

I think if no changes are being introduced, it's ok to just fast-forward, without introducing an additional merge commit.

That was my plan all along. I could have mentioned that, sorry.

Perfect!


server freeze when doing /changemap on a badly ported map

This is a bug I got on my server when Sweet ported some maps from unv to trem, the reason is still unknown to me. It can currently be observed when doing /changemap testmap on nightly build server.
Symptoms are those:

server does not move to next map
a weird string is printed, looking like this: changemap: map 'testmap' started by foobar $3t$4$5$

An example of map triggering this behavior has been attached to this issue, renamed to add .zip so that github does not annoys me.
map-testmap_0.dpk.zip

what defines a badly ported map?
there is no testmap.bsp in your testmap
i would say pak name and bsp name mismatch is the problem

what defines a badly ported map?

Honestly, I have no idea. It's just stuff I made at my 1st porting attempts, I have no clue about what I did wrong, but I have not seen that anew since long on my server.

i would say pak name and bsp name mismatch is the problem

Yes, I probably forgot to mention but this should likely be renamed map-methane_0.dpk, and it should still reproduce the problem. Or maybe not? I'll try to play with this a bit more to complete this issue.


fix(debugger): fix crash for machs with log level 0

Machines without any log msgs break debugger's log view.

Welcome to Codecov :tada:
Once you merge this PR into your default branch, you're all set! Codecov will compare coverage reports and display results in all future pull requests.
:information_source: You can also turn on project coverage checks and project coverage reporting on Pull Request comment
Thanks for integrating Codecov - We've got you covered :open_umbrella:


Miles Lubin, Iain Dunning, "Numerical Optimization in Julia"

Miles Lubin, Iain Dunning, Numerical Optimization in Julia, https://www.youtube.com/watch?v=O1icUP6sajU
When someone ask question, volume was quite low, so I could make a lot of mistakes.
Contents
00:00 What is Linear Programming (LP)?
01:23 Modeling languages for Linear Programming
03:05 Question: can you use more vector-like notation in AMPL?
03:35 Statements in AMPL can be more complicated that one show now
03:50 Short summary of current state of languages for LP
04:00 Modeling in Julia
04:53 Julia macros allow us to generate fast code
06:58 Question: what are you benchmarked in this example?
07:20 Question: does this macro takes expression and generated actual matrix?
07:52 Question: is writing a file procedure the reason that Julia is slower than AMPL?
08:50 Question: can you run this example asynchronously?
09:06 Question: how much times is take to solve actually LP problem?
09:55 Question: dose language for solving LP problems always use this type of interface?
10:35 Question: what is the external representations of the input?
11:09 Question: what are Model and Variable functions doing?
11:36 Variable functions and operator overloading
12:09 Question: dumb variables in presented problem
13:47 "The whole reason that language as AMPL exists, is that people like to think about it that way"
14:31 Algorithms for solving LP problems
15:04 Question: is this your PhD thesis?
15:25 Question: what happens to different implementations of Dantzig’s simplex algorithm
16:04 Question: what is the name of this thesis?
16:15 Computational and algorithmic challenges in LP
17:02 Simple benchmarks for solving LP problems
18:12 Comments on bounds checking of arrays in Julia
18:24 Results of benchmarks
18:40 Comments on cost of features from higher-level languages
18:57 Question: does Python implementations use numpy?
19:07 Comments on sparse matrix and vectors in Python
19:36 Question: do you write your own code in Julia for this task?
21:12 Question: do you know sparsity pattern before starting the calculation?
21:39 Question: would someone use aforementioned PhD thesis to implement Dantzig’s simplex algorithm in Julia?
22:00 "No factor of 2 will be left untouched"
22:36 "Nothing special is happening in this code"
22:53 Fuzzed dot operator inside a loop
23:56 Overview of tutorial, that is mostly unrelated to LP

Timestamps were added to the video. We will close this issue, when we get access rights to this GitHub repository.


Adding Watermark to section that has not initialized headers will throw an error

Error thrown:

System.NullReferenceException
HResult=0x80004003
Message=Object reference not set to an instance of an object.
Source=OfficeIMO.Word
StackTrace:
at OfficeIMO.Word.WordWatermark..ctor(WordDocument wordDocument, WordSection wordSection, WordHeader wordHeader, WordWatermarkStyle style, String text) in C:\Support\GitHub\OfficeIMO\OfficeIMO.Word\WordWatermark.cs:line 98

I've read the document - and I have no clue what I am deciding on about 🤣🤣 and how it impacts this project or this particular issue. I am noob when it comes to programming so you need to treat me as such 😊


Consider reverting #345595 Due to memory usage issues in qBittorrent 5.0

Describe the bug
https://github.com/qbittorrent/qBittorrent/issues/21502
Steps To Reproduce
Steps to reproduce the behavior:

Open qBittorrent 5.0
Monitor it's memory usage

Expected behavior
qBittorrent should not use excess RAM
Screenshots

Additional context
This issue makes the torrent client utterly unusable, as it just takes RAM and doesn't even seed the torrents
Notify maintainers
@Anton-Latukha @kashw2

Metadata

system: "x86_64-linux"
host os: Linux 6.11.2, NixOS, 24.11 (Vicuna), 24.11pre690827.5633bcff0c61
multi-user?: yes
sandbox: yes
version: nix-env (Nix) 2.18.8
channels(root): "home-manager, nixos"
channels(ember): "nixos"
nixpkgs: /nix/var/nix/profiles/per-user/root/channels/nixos

Add a :+1: reaction to issues you find important.

Ouch, if maintainers aren't available, I could get a revert PR going in a couple hours

Ouch, if maintainers aren't available, I could get a revert PR going in a couple hours

XRef: #352499
The version < 5.0.1 contains a RCE CVE. If we need to revert back to an older version, we need to mark it insecure.

rock and a hard place here, huh.

I think its probably better to leave the package as is, between picking a potential vulnerability and a memory leak I'd reckon most would settle for the latter.
If there's been any upstream commits with a potential fix I could get it added here as a patch as a temporary fix before a full release, though that will have to wait for rn, since it hasnt been fixed.

Upstream issue seems to be fixed in https://github.com/qbittorrent/qBittorrent/pull/21618 which is part a v5.0.1

this issue does seem to be fixed in 5.0.1

or.. not?

Have you tested on Qbit v5.0.2

Have you tested on Qbit v5.0.2

hadn't at the time, but nope, no change

it was working for a while after 5.0.1, but the issue returned for some reason
may be this issue https://github.com/arvidn/libtorrent/issues/6667


Question

When I connect to my LGTV from my laptop (Windows 10) on WiFi, it asks me once to allow pairing, and then the API works.  I can restart node over and over and I'm not asked to pair again.
When I connect from a machine on ethernet (Ubuntu), I'm asked to pair (by the TV) every time I restart node.
Any ideas?

Seems like the saving of the keyFile doesn't work under Windows... I'm preparing the 1.4.0 release that also fixes your Issue #9 - I think that could fix the issue. Should be done this weekend.


Runtime exception on dashboard getProxyData

=> Starting app on port 3000...
{"line":"106","file":"percolate_migrations.js","message":"Migrations: Not migrating, already at version 2","time":{"$date":1482167053954},"level":"info"}
spiderable: phantomjs not installed. Download and install from http://phantomjs.org/
Exception while invoking method 'getProxyData' TypeError: Cannot read property '0' of undefined
    at [object Object].getProxyData (dashboard/server/methods.js:25:39)
    at maybeAuditArgumentChecks (packages/ddp-server/livedata_server.js:1711:12)
    at packages/ddp-server/livedata_server.js:711:19
    at [object Object]._.extend.withValue (packages/meteor.js:1122:17)
    at packages/ddp-server/livedata_server.js:709:40
    at [object Object]._.extend.withValue (packages/meteor.js:1122:17)
    at packages/ddp-server/livedata_server.js:707:46
    at Session.method (packages/ddp-server/livedata_server.js:681:23)
    at packages/ddp-server/livedata_server.js:551:43

Helpfult hint: Check proxyBackend has apiUmbrella object before using it https://github.com/Digipalvelutehdas/APIKA/blob/fec765d2deaa23dba58402a4cc3b2617709bc6a0/dashboard/server/methods.js#L25

Steps to reproduce

Log into apika.digipalvelutehdas.fi as admin
Go to dashboard
Go to Select API list and select one API

Repeat step nr. 3 until you get 504 Gateway Time-out
Cannot tell yet whether it is a certain API that causes this or a certain nr of selections.

This issue was moved to apinf/platform#2040


Script.sh works when it's run manually but doesn't run when executed from outside docker.

Script.sh has many commands out of which only one command is now currently not working fine:
php artisan serve --port=3000 --host=<IP_ADDRESS>
alternative
php -S <IP_ADDRESS>:3000 server.php
Both of them does not work when remotely executed.

Works well now.


Facing error while create-react-app

If you have a general question about Create React App or about building an app with Create React App we encourage you to post in GitHub Discussions instead of this issue tracker. The maintainers and other community members can provide help and answer your questions there: https://github.com/facebook/create-react-app/discussions
If you're looking for general information on using React, the React docs have a list of resources: https://reactjs.org/community/support.html
If you've discovered a bug or would like to propose a change please use one of the other issue templates.
Thanks!

Steps to Reproduce
(Write your steps here:)
C:\Users\Admin>create-react-app new-react-app
Creating a new React app in C:\Users\Admin\new-react-app.
Installing packages. This might take a couple of minutes.
Installing react, react-dom, and react-scripts...
events.js:292
throw er; // Unhandled 'error' event                          ////I don't know why this error is occured
^
Error: spawn C:\Windows\system64\cmd.exe ENOENT
at Process.ChildProcess._handle.onexit (internal/child_process.js:267:19)
at onErrorNT (internal/child_process.js:469:16)
at processTicksAndRejections (internal/process/task_queues.js:84:21)
Emitted 'error' event on ChildProcess instance at:
at ChildProcess.cp.emit (C:\Users\Admin\AppData\Roaming\npm\node_modules\create-react-app\node_modules\cross-spawn\lib\enoent.js:34:29)
at Process.ChildProcess._handle.onexit (internal/child_process.js:273:12)
at onErrorNT (internal/child_process.js:469:16)
at processTicksAndRejections (internal/process/task_queues.js:84:21) {
errno: 'ENOENT',
code: 'ENOENT',
syscall: 'spawn C:\Windows\system64\cmd.exe',
path: 'C:\Windows\system64\cmd.exe',
spawnargs: [
'/d',
'/s',
'/c',
'"npm ^"install^" ^"--save^" ^"--save-exact^" ^"--loglevel^" ^"error^" ^"react^" ^"react-dom^"<EMAIL_ADDRESS>]
}
I am not able to start the next step because of these issues. I don't understand why npm is giving this issue is there any problem with my installation.

have the same issue, please help ((

any solution ?

any solution?

I updated my windows, and then use the yarn instead of npm, worked for me, after two weeks of struggling


fetch should not execute if id is not set; otherwise it gets the whole collection

Consider this case
var app.my_book = new Backbone.Model({ url_root: 'https://myserver/v1/books');
app.my_book.fetch();  // I forgot to set the book's id before fetching
Now app.my_book contains the list of all the books from REST server. This is because fetch was called without an id.
Suggest: if id is not set, fetch should return null and does nothing.
Thanks.

Some models only have URLs, and not ids.

Ok thanks. I ended up extending the Model, but I guess it's just me then.
Keep up the good work!


typescript error with vite

when using unplugin-auto-import in vite+vue+ts project, linter fails with following error:
Cannot find name 'Plugin$1'. Did you mean 'Plugin'?
./node_modules/unplugin-auto-import/dist/vite.d.ts:5:60

indeed, there seems to be some kind of error in file unplugin-auto-import/dist/vite.d.ts:
import { Options } from './types';
import '@antfu/utils';
import '@rollup/pluginutils';

declare const _default: (options?: Options | undefined) => Plugin$1;

export { _default as default };

token Plugin$1 is not defined anywhere, thus it throws an error.
Plugin$1 is used internally in node_modules/unplugin/dist/index.d.ts:5 as an alias to vite's plugin: import { Plugin as Plugin$1 } from 'vite';
Here is corresponding file from unplugin-vue-components:
import * as vite from 'vite';
import { Options } from './types';
import '@rollup/pluginutils';
import 'unplugin';

declare const _default: (options?: Options | undefined) => vite.Plugin;

export { _default as default };

Here, vite plugin's types are imported directly from vite, thus available to the ts parser and it works.
Unfortunately, I have no idea what to change in source code in order to fix this here.

I am totally stunned by the quickness of your fix! Everything works now, thank you!


Build failure: libpeas2 cannot find lua lgi package

Steps To Reproduce
Steps to reproduce the behavior:

Build libpeas2 on nixos-unstable

NOTE: an earlier version of this report contained misleading info, preserved for posterity within the collapsed content
~Unclear to me.~
~I have only been able to reproduce this error as a dependency of my (flake-based) NixOS system configuration on aarch64-linux.~
~None of these alone result in an error:~
# IGNORE THESE! SEE ABOVE UPDATE FOR CORRECT INFO
# nix build nixpkgs#libpeas
# nix build nixpkgs#gnome-builder

Build log
Full: https://gist.github.com/montchr/bdfd613a3b747a33f32bb669a30ed265
Excerpt of nixos-rebuild build --verbose output (removed unrelated lines):
building '/nix/store/d9qijlw975ag76bj5vv82j6l94srcg62-libpeas-2.0.1.drv'...
error: builder for '/nix/store/d9qijlw975ag76bj5vv82j6l94srcg62-libpeas-2.0.1.drv' failed with exit code 1;
       last 10 log lines:
       > Compiler for C supports link arguments -Wl,-z,relro: YES
       > Compiler for C supports link arguments -Wl,-z,now: YES
       > Compiler for C supports link arguments -Wl,-Bsymbolic: YES
       > Compiler for C supports link arguments -fno-plt: YES
       > Compiler for C supports arguments -fvisibility=hidden: YES
       > Configuring config.h using configuration
       >
       > meson.build:404:2: ERROR: Problem encountered: Lua51 requested but failed to locate suitable Lua51 and LGI support
       >
       > A full log can be found at /build/libpeas-2.0.1/build/meson-logs/meson-log.txt
       For full logs, run 'nix log /nix/store/d9qijlw975ag76bj5vv82j6l94srcg62-libpeas-2.0.1.drv'.
error: 1 dependencies of derivation '/nix/store/zhishld4zw0dg7wh8zipbzq29ilwyvyi-gnome-builder-45.0.drv' failed to build
error: 1 dependencies of derivation '/nix/store/bjidb7n0krz58w16rwavz966g4p55yg9-gnome-builder-45.0_fish-completions.drv' failed to build
error: 1 dependencies of derivation '/nix/store/kdjyyxmaxni2pk1q2n3z6i3gmkva4abg-man-paths.drv' failed to build
building '/nix/store/vm6dm6wzasgvh7y6mcxgqzd6x51d9h87-nnn-4.9.drv'...
error: 1 dependencies of derivation '/nix/store/5y654pcc4swqsf6sy1wr4g54nc9gcj5b-system-path.drv' failed to build
...
error: 1 dependencies of derivation '/nix/store/4l71d3j310ipbi29nglnlnjp8215hi1r-nixos-system-tuvok-24.05.20240403.08b9151.drv' failed to build

Additional context
Flake-based configuration for Asahi (Apple Silicon) system tracking nixos-unstable (08b9151ed40350725eb40b1fe96b0b86304a654b)
[Update]:
After further searching, I found this upstream issue and fix from 4 years ago:
https://gitlab.gnome.org/GNOME/libpeas/-/issues/33
https://gitlab.gnome.org/GNOME/libpeas/-/merge_requests/20
I am not sure what to do with that, however.
Notify maintainers
@NixOS/gnome
Metadata
Please run nix-shell -p nix-info --run "nix-info -m" and paste the result.
[user@system:~]$ nix-shell -p nix-info --run "nix-info -m"
 - system: `"aarch64-linux"`
 - host os: `Linux 6.6.0-asahi, NixOS, 24.05 (Uakari), 24.05.20240323.44d0940`
 - multi-user?: `yes`
 - sandbox: `yes`
 - version: `nix-env (Nix) 2.18.2`
 - nixpkgs: `/nix/store/06b0vfbjzpnn5qdar7yw83c6gzqsz5yd-amxd2p02wx78nyaa4bkb0hjvgwhz1dq7-source`

Add a :+1: reaction to issues you find important.

My build attempt of libpeas2 is attached:
build.log

I was greeted with the same issue when I attempted to updatepkgs.valent alongside my flake setup.

That is because gnome-builder depends on libpeas2.
I am currently bisecting it but it goes slow without https://github.com/timokau/nix-bisect/issues/25.

That is because gnome-builder depends on libpeas2.
I am currently bisecting it but it goes slow without timokau/nix-bisect#25.

I added that because the original issue states that they can build gnome-builder without the issue.

Maybe they have cached older version of nixpkgs flake ref.

That is because gnome-builder depends on libpeas2.
I am currently bisecting it but it goes slow without timokau/nix-bisect#25.

This comment appears to state that the linked PR is what caused libpeas2 to fail to build: https://github.com/NixOS/nixpkgs/pull/289135#issuecomment-1978270396

Weird. It was supposed to have been fixed by https://github.com/NixOS/nixpkgs/commit/0f6ddea2ff3b20c536b1530f66ee306014b39d83, which is in nixos-unstable according to https://nixpk.gs/pr-tracker.html?pr=292500 but libpeas2 still does not build for me.

Apologies for the confusing info I provided in the opening description.  I do see build failures for both libpeas2 and gnome-builder now:
[cdom@tuvok]% nix build github:nixos/nixpkgs/nixos-unstable#libpeas2
error: builder for '/nix/store/d9qijlw975ag76bj5vv82j6l94srcg62-libpeas-2.0.1.drv' failed with exit code 1;
       last 10 log lines:
       > Compiler for C supports link arguments -Wl,-z,relro: YES
       > Compiler for C supports link arguments -Wl,-z,now: YES
       > Compiler for C supports link arguments -Wl,-Bsymbolic: YES
       > Compiler for C supports link arguments -fno-plt: YES
       > Compiler for C supports arguments -fvisibility=hidden: YES
       > Configuring config.h using configuration
       >
       > meson.build:404:2: ERROR: Problem encountered: Lua51 requested but failed to locate suitable Lua51 and LGI support
       >
       > A full log can be found at /build/libpeas-2.0.1/build/meson-logs/meson-log.txt
       For full logs, run 'nix log /nix/store/d9qijlw975ag76bj5vv82j6l94srcg62-libpeas-2.0.1.drv'.
[cdom@tuvok]% nix build github:nixos/nixpkgs/nixos-unstable#gnome-builder
error: builder for '/nix/store/d9qijlw975ag76bj5vv82j6l94srcg62-libpeas-2.0.1.drv' failed with exit code 1;
       last 10 log lines:
       > Compiler for C supports link arguments -Wl,-z,relro: YES
       > Compiler for C supports link arguments -Wl,-z,now: YES
       > Compiler for C supports link arguments -Wl,-Bsymbolic: YES
       > Compiler for C supports link arguments -fno-plt: YES
       > Compiler for C supports arguments -fvisibility=hidden: YES
       > Configuring config.h using configuration
       >
       > meson.build:404:2: ERROR: Problem encountered: Lua51 requested but failed to locate suitable Lua51 and LGI support
       >
       > A full log can be found at /build/libpeas-2.0.1/build/meson-logs/meson-log.txt
       For full logs, run 'nix log /nix/store/d9qijlw975ag76bj5vv82j6l94srcg62-libpeas-2.0.1.drv'.

Maybe they have cached older version of nixpkgs flake ref.

@jtojnar You're right, thanks.  Now I know better for next time to verify stuff like this with github:nixos/nixpkgs/nixos-unstable rather than nixpkgs, which corresponds to the ref of my current NixOS generation.  I'll update the description.

Does not build here, nixpkgs @ fd281bd6b7d3e32ddfa399853946f782553163b5

cc @lolbinarycat and @vcunat who last touched the lua stuff

@vcunat yes, that fixes it, thanks!
libpeas2 builds when cherry picking b8be0164260e61c1859d12b5d973c0f3f67d379c and c55b4dbcd2af7ff8065ee4fece2c8793427ec88b onto nixos-unstable.

libpeas2 builds when cherry picking b8be016 and c55b4db onto nixos-unstable.

For others wishing to track the status of #300905, the PR adding these commits:
https://nixpk.gs/pr-tracker.html?pr=300905
Unfortunately, it looks as though the changes have been stuck on the staging branch for a few days.  I haven't yet cherry-picked those commits to verify since I expected them to appear in master sooner.  But I will try to get to verifying a few hours from now.
I'm having some trouble following the chain of related PRs.  @vcunat (or anyone else) do you have a better sense of when we might see those changes make it out of staging?

Roughly two weeks from now.


Provide Scaler for Oracle NoSQL Database

Proposal
https://www.oracle.com/database/
Scaler Source
Oracle Database
Scaling Mechanics
Scale based on an Oracle SQL query outcome
Authentication Source
Password authentication will be used
Anything else?
No response

Great idea, are you willing to contribute it?

Yes :)

Awesome, thank you @nilayasiktoprak!

Hi, I'm using this library to develop the Oracle scaler: https://github.com/oracle/nosql-go-sdk
and it supports 3 environments (Oracle Cloud, Oracle NoSQL Cloud Simulator, and On-Premise): https://github.com/oracle/nosql-go-sdk#configuring-the-sdk
I have developed the scaler for on-premise and I would like to ask you if you think I should develop seperate scalers for these 3 environments or combine them all in one scaler? An example of a combined one here: https://github.com/oracle/nosql-go-sdk#simple-example
Or maybe you can say that there is no need to develop a scaler for Cloud Simulator environment and I can just skip it.
What are your suggestions?

Thanks for the research! ❤
IMO, having one scaler for all if they do the same for different envs it's the best option because it reduces the maintenance work.
BTW, I thought that Oracle as a relational db, not NoSQL, do they use the same SDK?

There are Oracle NoSQL and Oracle (RDBMS). I decided to do the NoSQL one first so that's why I'm using that SDK.
Alright, I'll put them all in one scaler. And for which environments do you think I should develop the scaler? All the three of them?
Also maybe it would be better to change the name of the issue to Oracle NoSQL, right?

TBH, I don't know enough about Orcale to know which environments could be better, but as far as I could see in the docs you sent, supporting more than 1 only changes the connection part, right?
I mean, if support all of them is just adding more cases to the switch...
But I can be missing something. At least (IMO), we should support on prem and cloud (but again, IDK about Oracle to know if both are used)

+1 to changing name to Oracle NoSQL
+1 to merging this to one scaler
Thanks for the work!

Recently the CNCF has offered credit for Oracle Cloud. Could it be useful if we request it?


Web form pattern added

Added new pattern for web form.

Standard form
Form with errors
Success page

This pattern requires documentation.

 Web Form

Im thinking it should be in a folder named send-a-web-form rather than webform

Agree. I will update that now.

Im thinking it should be in a folder named send-a-web-form rather than webform

Agree. I will update that now.
Looking at other patterns it may be good to follow the same naming pattern. Others do look inconsistent.

Yeah I agree we should standardise it across the repo


himitsu: 0.4 -> 0.5

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://repology.org/project/himitsu/versions.
meta.description for himitsu is: A secret storage manager
meta.homepage for himitsu is: https://himitsustore.org/
Updates performed

Version update

To inspect upstream changes
Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
found 0.5 in filename of file in /nix/store/vqxd9v9mr52ymsq4dynygjj189cynl5m-himitsu-0.5

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

2 total rebuild path(s)

2 package rebuild(s)

First fifty rebuilds by attrpath
himitsu
himitsu-firefox

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/vqxd9v9mr52ymsq4dynygjj189cynl5m-himitsu-0.5 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

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

Or:
nix build github:r-ryantm/nixpkgs/b04aa8e72e752713af6944aefa9d74c6f7b1dedd#himitsu

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/vqxd9v9mr52ymsq4dynygjj189cynl5m-himitsu-0.5
ls -la /nix/store/vqxd9v9mr52ymsq4dynygjj189cynl5m-himitsu-0.5/bin

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

1 package marked as broken and skipped:

himitsu-firefox

1 package built:

himitsu

Maintainer pings
cc @auchter for testing.

Result of nixpkgs-review pr 272519 run on x86_64-linux 1

1 package marked as broken and skipped:

himitsu-firefox

1 package built:

himitsu

Result of nixpkgs-review pr 272519 run on x86_64-linux 1

1 package marked as broken and skipped:

himitsu-firefox

1 package built:

himitsu

merging since the maintainer didn't respond within a week and the package update seems fine.


Change RabbitMqUser ValidateSet

added "impersonator" and "none" as options to the validate set.

Looks good to me, and simple enough to not bother others for review, merging!


NCO Frequency cannot be set with LIQUID_VCO_DIRECT

The user can specify nco_crcf_create(LIQUID_VCO_DIRECT) but then later cannot set the frequency using the common interface nco_crcf_set_frequency(q, 0.12345f). The frequency can only be set with an internal method nco_crcf_set_vcodirect_frequency() which is not exposed to the user.

Tagging @ArtemPisarenko.

Issue isn't clear for me. nco_crcf_set_vcodirect_frequency() method isn't internal. It's exposed to user. Otherwise object would be non-functional.
Did you mean that NCO objects created with different types have different methods for setting/getting frequency? If so, then it's a choice of whole library design considerations. Should same module objects have fully identical API (allowing different sub-types to change only internal behaviour and characteristics)? If so, LIQUID_VCO_DIRECT nco should be reimplemented as separate module. If library would written in C++ then common part of API could be separated to base class (NCO interface) and specific module classes (NCO types) could be derived from it. But it's C lang. So I decided the most flexible design option is to add "vco-direct" implementation as additional run-time type and differentiate API partially. Because this kind of NCO have more common than differences. Its specifics cannot be hidden from user. There are no universal optimal way to convert arbitrary floating-point dtheta value to the pair of n and m integer values. Also it's not possible to change phase arbitrarily during stepping (although current implementation might be improved to specify starting phase, i.e. initial theta floating-point value). So, user should be aware of such limitations, i.e. LIQUID_VCO_DIRECT is application-specific. It's a price user must pay for achieved NCO characteristics.

Ok, my apologies. There are two things going on that are related. You're correct that the user can set the frequency of LIQUID_VCO_DIRECT with the nco_crcf_set_vcodirect_frequency() method. My apologies; when I rebased your branch onto master I somehow missed this in the global header. Thank you for reminding me of this.
The second (which was the origin for this issue) is that there are now separate methods for getting/setting frequencies based on types. As you point out, this is a choice of library design considerations. I really see one of two options for how the NCO functionality should work:

There is one object that can be created several ways at runtime, but with identical interfaces
There are completely separate objects with separate interfaces

My preference would be for option 1, but I understand your reasoning for choosing the interface you did.
Currently, we have separate functional objects packed into one: nco_crcf, but separate interfaces for them after instantiation.
In my opinion, the nco_crcf object should have a fixed-point (e.g. 32-bit) phase representation because its phase and frequency can be adjusted extremely quickly using integer operations and masking. Then the implementation for the type (look-up table, LUT with linear interpolation, etc.) can be defined by LIQUID_NCO or LIQUID_VCO etc. If we want an NCO-like object that doesn't support this functionality, I think it makes sense to have it as a separate object.
What are your thoughts?
(p.s. thank you for supporting this conversation; I've found it very useful!)

I haven't any preferences about design choices, because I haven't used the library for a long time and hardly remember things I did :)
I agree that phi and theta values would be better represented as fixed-point (both interface and internals), but it has nothing to do with functionality. LIQUID_VCO_DIRECT algorithm is natural fixed-point, but it still differs in what nco_crcf methods are supported. So these are two separate issues.
If you prefer option 2, extracting current LIQUID_VCO_DIRECT implementation to separate module would be pretty straightforward. I mean that it's an easy and trivial work compared to design decisions we discuss.
The only important thing I see is how much user be happy with limitations of current LIQUID_VCO_DIRECT implementation. Some limitations can't be either avoided at all, or some non-trivial compromises could be found (e.g. setting phase during stepping). Some ones aren't actually limitations but rather missing trivial features I didn't care about when implemented for my needs (e.g. defining initial phase or getting phase). I have time to discuss, but I'm not sure I'll be able to support my commitment further.


Remove version: WHONET.2023 version 23.14.16

Microsoft Reviewers: Open in CodeFlow

URL: https://whonet.org/WHONET2023-Setup-x86.exe
Status Code: 200
(Automated message - build 796)


Can't use multiple tensorboard callbaks in TF2.8

System information.

Have I written custom code (as opposed to using a stock example script provided in Keras): yes
OS Platform and Distribution (e.g., Linux Ubuntu 16.04): MacOS
TensorFlow installed from (source or binary): binary
TensorFlow version (use command below): v2.8.0-rc1-32-g3f878cff5b6 2.8.0
Python version: 3.8.12
Bazel version (if compiling from source):
GPU model and memory: None
Exact command to reproduce:

Describe the problem.
I am suing multiple tensoboard callbacks with the same Model. Since I upgraded Tensorflow from 2.5 to 2.8, I have an error at the end of the training when closing the Tensorboard writers.
Describe the current behavior.
The training doesn't end correctly because of a Tensrboard callback error.
Describe the expected behavior.
It used to work in TF2.5 and it should still do.
Contributing.


Next outline element

This adds the ability to retrieve the next outline node for any node.
This could probably be re-written in a nicer way using statement address iterators introduced in #61 , I'll look into that.

Actually maybe I could merge this as-is if you agree, and then rework later?


tutorial/react not working with aem-guides-wknd 2.0.1-snapshot

Expected Behavior
To display the content based on the content fragments in aem-guides-wknd
Actual Behavior
No content is being displayed with an error
Error: SyntaxError: Unexpected end of JSON input
Reproduce Scenario (including but not limited to)
Steps to Reproduce
Follow the instructions based on the GraphQL > Quick Setup > Local SDK with latest AEM SDK
https://experienceleague.adobe.com/docs/experience-manager-learn/getting-started-with-aem-headless/graphql/quick-setup/local-sdk.html?lang=en
Platform and Version
MacOS 12.4
Oraclle Java version "<IP_ADDRESS>" 2022-04-22 LTS
Node.js v18.4.0
npm 8.13.1
Adobe Experience Manager 2022.6.7904.20220629T070041Z-220600 LocalSDK
aem-guides-wknd version 2.0.1-snapshot
Sample Code that illustrates the problem
 {
    adventureList {
      items {
        _path
        adventureTitle
        adventurePrice
        adventureTripLength
        adventurePrimaryImage {
          ... on ImageRef {
            _path
            mimeType
            width
            height
          }
        }
      }
    }
  }

http://localhost:3000/content/cq:graphql/wknd-shared/endpoint.json
The fieldlabels and endpoints have changed in 2.0.0 from 1.1.0 in aem-guides-wknd and has broken the headless app
Logs taken while reproducing problem

Hello @deltaworld, we recently updated the tutorial to use persisted GraphQL queries and performed content restructuring, can you please re-try and let us know. Also, there is a basic GraphQL tutorial to understand, learn headless capabilities
https://experienceleague.adobe.com/docs/experience-manager-learn/getting-started-with-aem-headless/graphql/multi-step/overview.html?lang=en


Drop "only master branch" travis restriction

I get that one may want to restrict where travis runs (resource contentions,
reserving travis runs for master, etc), but this makes it difficult for others
to enable travis on their forks -- unless they're willing to do all their work
on master.
Would you be amenable to this change?  Or, in the alternative, creating a
white/black list
to allow for a broader range of branches?

I'll take this. I set that restriction when I was creating a few unstable branches and I didn't want to see the emails where Travis complained about them.

Awesome, thanks :)


fix(deposit instruction): incorrect expected accounts count

Nine accounts are listed as required as per the instruction

Thanks. Fixed this in branch v3.2.7.


GUVNOR-2923: [Guided Decision Table] Audit log is not paginated

See https://issues.jboss.org/browse/GUVNOR-2923

@jomarko Care to review? I refactored a little; but nothing too serious.

@manstis sure. I would like to have a look. I will put comment here when I am done.

Tried locally and pagination works now. I have also nothing against the code changes. Thanks @manstis for solving this issue in such short time.


Anchor pages to screen side

Describe your suggested feature
I'd like to be able to anchor the page to the top of the screen instead of to the center. So if I'm holding my phone from the bottom in reverse portrait I essentially have a borderless view of the manga, since the border is where I'm holding the phone.
Other details
No response
Acknowledgements

[X] I have searched the existing issues and this is a new ticket, NOT a duplicate or related to another open or closed issue.
[X] I have written a short but informative title.
[X] I have updated the app to version 0.16.5.
[X] I will fill out all of the requested information in this form.

what?

left aligned instead of centre aligned?

Basically, if the phone is in portrait view, there are normally back/white bars at the top and bottom. I'm suggesting that you can have one large bar on the top or the bottom instead of two smaller ones on the top and bottom.

What black bar? There's no black bar

This part.

If you have a tall phone, the back bars are there. Theoretically, you could have one big bar instead to anchor the page to the top or bottom of the screen.

Given that you'll probably gonna be the only one to use it it's a no.

That's fair. I think tall phones are fairly common, but it's understandable that most people wouldn't use that option.


Custom Keyboard Implementation

As discussed in issue #3

I removed the selector view (not needed in this implementation) and set the coloredBorderWidth to 0 to avoid the crash mentioned above (looks better in this implementation anyways)


Cannot make it work

I downloaded sample from GitHub, substituted connection string, hub name, google-service.json with my data, but still couldn't get test notification.
What is interesting, that I don't hit breakpoint in MyFirebaseIIDService.OnTokenRefresh, which I think is root of the problem. Any ideas or advice is highly appreciated. Thank you.

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

ID: 2a03ab5f-3dda-f469-33e0-9504c90d468c
Version Independent ID: 4f8e4309-cd06-f09f-00f0-d978b3d16777
Content: Push notifications to Xamarin.Android apps using Azure Notification Hubs
Content Source: articles/notification-hubs/xamarin-notification-hubs-push-notifications-android-gcm.md
Service: notification-hubs
GitHub Login: @jwargo
Microsoft Alias: jowargo

Please, close the issue.
I was able to make it work. I had to update my Android SDK to 9.0 (API 28) and set 9.0 as target platform. Then it worked.

@maxal1917  Glad to hear that the issue was resolved. I will now proceed to close this issue. Feel free to still comment if you have any queries.


Remove obsolete section

The "Why is this written in Java and not Kotlin?" section is no longer relevant,
since the library is now 100% Kotlin.
Closes #28

 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.

Thanks for the PR!


Drop ember-composable-helpers dependency

Fixes #731
Essentially, the newest versions of ember require v3 of ember composable helpers which in turn only supports ember 3.13+. We can't bump our own dependency as we are trying to maintain support back to ember 3.4.
Pending a more elaborate refactor in v3, I'm proposing we lift the compute helper directly from ember-composable-helpers to enable us to drop the dependency entirely.
I'm going to mark this as breaking as dropping the dependency may catch a few people out who are using it elsewhere in their applications.

I've also done a little test locally, and when ember-composable-helpers is installed it appears to favour their helper over ours so this shouldn't cause any issues for people with ember-composable-helpers installed.


Allow custom names for channels

The use case I have is that I have a bunch of Russian channels I sub to and I'd love to quickly jump to them with new cmd+O feature.
But if you search for say:

It won't return any results because it needs to be in Russian.

Not sure how feasible it is to implement but would be nice. ♥

Good suggestion, thank you! Instead of custom names I made it work with built-in latin transliteration, so your example with "vdud" will show both channels now.
Done in https://github.com/agentcooper/Telik/commit/cc9ba1bc9f38657d6813e76db69b42ac8ee4881f, released in https://github.com/agentcooper/Telik/releases/tag/1.5.

This is a very nice change, thank you.
One thing that it does bring is a regression in speed. On my list of subscriptions if I type vd really fast, the first v will bring results quite fast but after typing d it would lag and only update the search input after ~ 2 seconds. And that's on latest M1 mac.
I'll try to look into the issue as I am certain this is only an issue when you have as many subscriptions as me which few will have. 😄

I didn't see much lag when testing with many channels, but I think there is clearly room for improvement.
It would be cool if you can try to optimize it. Right now it is mostly a dumb loop through the whole list with caching only for the transliteration results.
Maybe the search needs to become async and debounced.


Added overloads for Tasks.ParkVehicle and changed the default radius …

…to 20.0f instead of 0.0f.

Couldn't there be an overload that includes the Park Mode option (which is in the Native)? If you don't know the heading you will be approaching a parking destination from, it makes it impossible to predict what heading you will need, which is why the Parking Mode has an "Ignore heading" option.


Improve assignment bot logic

The assignment bot should only assign an issue to a particular user if the user comments !assign or !assign me or please !assign me
https://github.com/codinasion/codinasion/blob/0fa9dad5c599f318a8d417b4987e4507afc16b66/github-bots/assign-me/src/index.ts#L8
Reference

please !assign me

Testing.

Check if bot assigns only when we use !assign

It works as expected. What is the issue?

It should assign a user if a suggestion is made to another user

Understood. Working on it.


sap_hypervisor_node_preconfigure: Minor fixes

LGTM.
@newkit: What was the reason for the double quotes around true in https://github.com/sap-linuxlab/community.sap_install/pull/553/commits/df5b58604a06c83ec48701ded90015d70f5bb7b7 (just for the records)?

@berndfinger Simply the fact that the double quotes are needed there in order to work.

@berndfinger it is the definition of a k8s configuration and in this case feature.node.kubernetes.io/network-sriov.capable takes a string and not a boolean.
You can find the documentation for this component in here https://docs.openshift.com/container-platform/4.9/networking/hardware_networks/about-sriov.html
And also a configuration example in here https://docs.openshift.com/container-platform/4.8/networking/hardware_networks/configuring-sriov-device.html#nw-sriov-networknodepolicy-object_configuring-sriov-device

@Wabri Many thanks!


Issue with k8s.io/docs/tutorials/kubernetes-basics/update/update-interactive/

On Module 6, Step 2 of the KataKoda exercise, curl $(minikube ip):$NODE_PORT does not work because the service is deployed on a different IP than minikube ip. Please replace with
curl $(kubectl get service -o json kubernetes-bootcamp | jq -r '.spec.clusterIP + ":"+ (.spec.ports[0].port|tostring)')

/good-first-issue

Can I take this up?
@zhangqx2010 Can you please point me to the source code of the katacoda scenarios?

/remove-good-first-issue
@palnabarun Thanks for you help.
Sorry for the wrong information. This is out of scope. My bad. I am trying to contact relative person to fix it.

@domq
I retried on the lab but it seems work. Output like:
$ echo NODE_PORT=$NODE_PORT
NODE_PORT=30757
$ curl $(minikube ip):$NODE_PORT
Hello Kubernetes bootcamp! | Running on: kubernetes-bootcamp-5bf4d5689b-qkdwd | v=2
$

Could you please provide more detail how to reproduce the issue?

/triage needs-information

/remove-help

As @zhangqx2010 pointed out, recent changes to Minikube meant this has been resolved.
We believe this issue can be closed.

/close


Please post a maintenance status note

"I had a job writing software, then I open sourced it. Now I have two jobs."
@nullgemm Thank you for sharing Ly. Clearing it's popular with a number of people and now pull requests and issues are piling up. You've given enough and don't need to take on the second job of dealing with all the work that open sourcing Ly has generated if you don't want to.
But you are in a unique position as the primary Github repo owner to decide what happens next. You could add more co-maintainers, like @SartoxSoftware who has already started to maintain a fork with bug fixes. Or you could enable the "Discussions" feature to at least move some of the chat out of issues.
One option is to create a free, "ly" Github organization and give some other people permission help run it. You could contribute as much or as little as you want.
But right now Ly is stuck with no clear path forward with maintenance. Posting a note about your intentions or interest in alternate or co-maintainers would be helpful. Thanks!

Hi @markstos, first I appreciate your indulgence, so: thank you.
About Ly's status : I had plans to make some rather extensive changes in the UI system. That was four years ago. I was then in the process of designing a UI library (which I finally started implementing last year) based on the idea all positions and sizes would be computed in per-widget callbacks. This allows for easy GUI creation for me, non-artist, and has a special advantage: it translates very well to TUI, so it should be possible to generate TUI apps from GUI code transparently, by changing a simple define. I wanted Ly to use this system to both have a TUI version and a GUI version using the Linux frame buffer, for people who don't like startup logs polluting the TTY (which, sadly, we can't fix). I know, I know, this is overkill, but I thought it would be useful for other projects, and not just Ly.
Now obviously "we do these things not because they are easy, but because we thought they were going to be easy". And here we are, four years later, with the situation you described. I am going to finish this UI library and all associated projects, but it is clearly more than time to update Ly now. As you can see I'm not very active on open-source projects currently, this is because I am busy right now, and it's not going to get better before several months. So what I can do is create an organization with people interested in helping out as maintainers to at least fix the bugs. I'll have a look at this and contact potential maintainers as soon as I have some more time available.
Thank you for caring about this project.

Having recently setup a Github organization myself, the whole process of creating the repo and transferring the project to it should take < 10 minutes. Github will handle automatically redirecting the old URLs.
Start here to create the organization: https://github.com/organizations/plan
Once setup, return to the project settings here ( https://github.com/nullgemm/ly/settings ) then find  "Danger Zone" and
the option to transfer the repo to the organization.
The username ly is already taken so it can't be used for an org name, so the hardest part may be picking the organization name. Maybe ly-dm.

As some of you might have noticed, @AnErrupTion joined me as a maintainer, and some major bug fixes and PRs have already been merged. I am slightly less busy right now so I will be helping out for some time before going back to my other projects, when I feel comfortable doing so.


Vscode on github codespace issues

Hi
I have this error when i run : npm start  in Vscode GitHub codespace:
Starting the development server...
The build failed because the process exited too early. This probably means the system ran out of memory or someone called kill -9 on the process.

/extCodespaces

It looks like this is caused by the Codespaces extension. Please file the issue in the Codespaces Discussion Forum. Make sure to check their issue reporting template and provide them relevant information such as the extension version you're using. See also our issue reporting guidelines for more information.
Happy Coding!


Bug: list contributor search hitting <CR> doesn't do anything

Describe the bug
Here's a sample video of what I'm seeing on the List edit / creation page:
https://github.com/open-sauced/app/assets/23109390/4cdbd3de-8502-47fc-bf04-dd79cd8fac08
You'll notice the drop down 'pops': that's when I'm hitting "enter" (or more agnostically, <CR>)
My expectation when hitting enter on a list would be that it returns search results for that query on a fuzzy basis. The only way to really make the returned list useful is to click on an entry (which you'll notice I do in the video above)
TLDR: hitting enter when searching should return and generate a list.
Noticing this on beta and prod
Steps to reproduce

Create a List
Add some collaborators
Use the search bar with a query
Hit enter / <CR>

Browsers
Firefox
Additional context (Is this in dev or production?)
No response
Code of Conduct

[X] I agree to follow this project's Code of Conduct

Contributing Docs

[X] I agree to follow this project's Contribution Docs

Possibly also related to: https://github.com/open-sauced/app/issues/988 which seems to have similar problems with the search for repos

can i work on this issue ?

This might need to be core team work: I don't have a good intuition on what's happening with the search under the hood or if it will require API changes


chore: update delegate routing deps to fix branches

This updates the delegated routing dependencies for failing tests due to an issue with the latest ipfs http client
Depends on:

[ ] https://github.com/libp2p/js-libp2p-delegated-peer-routing/pull/30
[ ] https://github.com/libp2p/js-libp2p-delegated-content-routing/pull/37

cc @achingbrain All the delegate routing tests are now passing with the latest ipfs-utils and http-client RC.


Failure with build tool plugin on Windows

Description
AFAICT, all build tool plugins fail on Windows because PackagePlugin.dll cannot be found.
FWIW, related thread on the forums:
https://forums.swift.org/t/known-case-of-a-working-build-tool-plugin-on-windows/58825
Expected behavior
The plugin should execute
Actual behavior

Steps to reproduce

Start with https://github.com/abertelrud/swiftpm-buildtool-plugin-examples
cd "Example 1 - SwiftGen"
swift build

In this particular case, what should happen is that the plugin should be compiled and executed, and then we should get some kind of failure due to the fact that binary target is Linux specific.  But it never gets that far.  FWIW, I've tried lots of combinations wherein I add an actual Windows exe for the binary target, as well as cases where I change the plugin itself to be simpler.  None of that matters because the plugin never actually launches.
Swift Package Manager version/commit hash
No response
Swift & OS version (output of swift --version && uname -a)
I've tried this with 5.6.2, 5.7 snapshot, and main snapshot.  They all give the same result.  OS is Windows 11.

Looking at apple/swift-installer-scripts:
https://github.com/apple/swift-installer-scripts/blob/main/platforms/Windows/devtools-amd64.wxs#L149-L151
I think that this is the result of a simple oversight: we need to adjust Path when executing the plugin on Windows as we do tests: https://github.com/apple/swift-package-manager/blob/57af70f2f9f5645514a9ce8e6279995c701333b0/Sources/Commands/Utilities/TestingSupport.swift#L133


Issue with sudo npm install npm -g

Hi all,
i'm a newbie to programming and following the instruction to install both heroku and node.
I'm facing an issue that when i type sudo npm install npm -g into node Command Line Prompt. It show that there's syntax error.
Could anyone help to solve this issue? I tried to search this issue in google but couldn't figure out anything.
thanks and appreciate for your help.

curl -X POST "https://graph.facebook.com/v2.6/me/subscribed_apps?access_token=<PAGE_ACCESS_TOKEN>"
i solve this by simply type "npm install npm -g"
however, i'm facing issue that in step"curl -X POST "https://graph.facebook.com/v2.6/me/subscribed_apps?access_token=<PAGE_ACCESS_TOKEN>"
i type this code in cmd but it doesn't work...

problem solved by installed curl


Add trailing newlines to files missing them

Somebody's editor doesn't add a trailing newline, which is making some linters
unhappy.

I have also seen the failures. Thanks for fixing.


[Snyk] Security upgrade ansible-core from 2.11.12 to 2.12.0b1

This PR was automatically created by Snyk using the credentials of a real user.Snyk has created this PR to fix one or more vulnerable packages in the `pip` dependencies of this project.
Changes included in this PR

Changes to the following files to upgrade the vulnerable dependencies to a fixed version:

test-requirements.txt

⚠ Warning
virtualenv 20.24.6 has requirement importlib-metadata>=6.6; python_version < "3.8", but you have importlib-metadata 4.2.0.
molecule 2.22 has requirement anyconfig==0.9.7, but you have anyconfig 0.13.0.

Vulnerabilities that will be fixed
By pinning:

Severity
Priority Score (*)
Issue
Upgrade
Breaking Change
Exploit Maturity

536/1000  Why? Recently disclosed, Has a fix available, CVSS 5
Race Condition SNYK-PYTHON-ANSIBLECORE-6069873
ansible-core:  2.11.12 -> 2.12.0b1
No
No Known Exploit

(*) Note that the real score may have changed since the PR was raised.
Some vulnerabilities couldn't be fully fixed and so Snyk will still find them when the project is tested again. This may be because the vulnerability existed within more than one direct dependency, but not all of the affected dependencies could be upgraded.
Check the changes in this PR to ensure they won't cause issues with your project.

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

Learn how to fix vulnerabilities with free interactive lessons:
🦉 Learn about vulnerability in an interactive lesson of Snyk Learn.

Superseded by: https://github.com/mesoform/ansible-dockerswarm/pull/10


Add ReZip Parameter for MSOnline compatibility (fixes #240)

@JustinGrote WooHoo, I can confirm that this patch works.  I am able to send email with an Excel file created via Export-Excel with your patch through Microsoft Edge servers to Outlook 365.

Thanks @JustinGrote for the fix and testing. Thank you @CrashM for confirming.
Also, thanks for including the Write-Verbose


Config for preventing checking database schema

Describe the bug
In the CI environment, we do not have a database for now.
I would expect the plugin to keep working even without a database. I understand the plugin trying to get the type from schema. Is it possible to torn this off?
Impacted Versions
Please paste the output of composer show | grep -E 'psalm|laravel' so we can see what versions are impacted
root@cf7568939b49:/app/bpg# composer show | grep -E 'psalm|laravel'
barryvdh/laravel-debugbar                      v3.5.2    PHP Debugbar integration for Laravel
barryvdh/laravel-ide-helper                    v2.8.2    Laravel IDE Helper, generates correct PHPDocs for all Facade classes, to improve auto-completion.
bensampo/laravel-enum                          v1.38.0   Simple, extensible and powerful enumeration implementation for Laravel.
laravel-doctrine/migrations                    1.3.3     Doctrine Migrations for Laravel
laravel-doctrine/orm                           1.5.10    A Doctrine ORM bridge for Laravel 5
laravel/framework                              v6.20.19  The Laravel Framework.
laravel/socialite                              v5.2.2    Laravel wrapper around OAuth 1 & OAuth 2 libraries.
laravel/tinker                                 v2.6.1    Powerful REPL for the Laravel framework.
laravelcollective/html                         v6.2.1    HTML and Form Builders for the Laravel Framework
paxoffice/laravel-permission-gateway-auth-php  1.26.0    Laravel Permission Gateway Auth provider
psalm/plugin-laravel                           v1.5.1    A Laravel plugin for Psalm
sentry/sentry-laravel                          1.9.0     Laravel SDK for Sentry (https://sentry.io)
vimeo/psalm                                    4.20.0    A static analysis tool for finding errors in PHP applications

Additional context
Add any other context about the problem here.

good question! I don't think there is a way right now, but a PR would be welcome!

The current problme is that it going to return 0 status code if plugging throw exception while trying to get return type.
I suggest to catch Throwable to eliminate chance of such error.

https://github.com/psalm/psalm-plugin-laravel/pull/228

I need it in previuse laravel version?
Could we add it to 1.6.*?

https://github.com/psalm/psalm-plugin-laravel/pull/231


Ci exclude arm64 with Noble and uses the candidate for grafana-agent

Part of #339, was added Noble support on our CI in #342, however it was not possible to use

grafana-agent from latest/stable on amd64 with<EMAIL_ADDRESS>base, since it's not promoted yet
grafana-agent on arm64 with<EMAIL_ADDRESS>
This bug will by removing exclude for arm64 on Noble and using latest/stable channel.
Related bug on grafana-agent: https://github.com/canonical/grafana-agent-operator/issues/207

@rgildein is this fixed after you added the exclusion to the matrix for this combination?


Need option to disable (git) integration(s) completely

What feature or improvement do you think would benefit Files?
In general I think Files should allow users to enable and disable different integration. But what personally bugs me is the git integration - details view is slow even for smaller repos and other views (especially columns) gets slow with bigger repos.
Try setting default view as columns and open the files to a certain file from for example from vscode, it takes way too long to open files fully.
Requirements

Allow users to enable what they actually need/want
Make details view usable, right now if you use that view and go to any git repo's, it's slow because it tries to get the commit info for each separate files
Overall performance inpact

Files Version
<IP_ADDRESS>
Windows Version
10.0.22621.2506
Comments
No response

Thanks for the feedback, Performance issues are being discussed at https://github.com/files-community/Files/issues/12737. You can check there or click subscribe in the right sidebar for updates.
You can also turn off the columns by right clicking them and go under git for that content to not load.

@Josh65-2201
Even then I still think there should be options to disable and enable integrations completely for people who don't want them.

Thank you for the feedback. Considering that the Git columns can be hidden (they are by default) and that the options on the status bar don't need to be used, we don't have plans to add this option. That said, we're aware of a performance issue and we're actively working on fixing it #12737.

@GameBurrow thank you for your patience, the next release will have a fix for the performance issues. Not only that, the improvements work without disabling the git features so you'll be able to take advantage of them if you wish.


Selected date not being retained when we move for 2 or more months in either direction and comeback to selected date.

I have given last selectable date and first/oldest selectable date, and disabled few days in between...
For example I selected Apr 25th 2018, I swipe to the month May and come back, apr 25th is retained. But If i go to June or more and go back to Apr , Apr 25th is not preselected and in side CVCalender related classes, calendarView is becoming nil,
My requirement is to show dates in 3-4 months, disable few dates in between, toggle between them.
Retain selected date and keep it highlighted even if user swipes to other months and come back to selected month, unless user selects another date.

Hi @vamsivannam, please use the issue template and provide a proper example project for us to investigate. Otherwise we won't be able to provide any assistance, thank you.

same issue for me also when tooggleing from week to month view.
juste launch the sample code, select a day and switch from month view to week view : the selected day is lost

@vassily-fr @vamsivannam Please re-test with the latest release 1.6.2, thanks!


fix(test): fix Jest tests

What kind of change does this PR introduce? (check at least one)

[ ] Bugfix
[ ] Feature
[ ] New Binding Issue #___
[ ] Code style update
[ ] Refactor
[ ] Build-related changes
[x] Other, please describe: Fixes tests

Does this PR introduce a breaking change? (check one)

[ ] Yes. Issue #___
[x] No

The PR fulfills these requirements:

[x] It's submitted to the dev branch and not the master branch
[ ] When resolving a specific issue, it's referenced in the PR's title (e.g. fix: #xxx[,#xxx], where "xxx" is the issue number)

If adding a new feature, the PR's description includes:

[ ] A convincing reason for adding this feature (to avoid wasting your time, it's best to open a suggestion issue first and wait for approval before working on it)

Other information:
Fixed file paths which were changed during project reorganization

I'm not sure what's going on with the JS tests on macOS, as they take a long time on my laptop as well. I have a hunch that process.exit(0) is killing the process even though it's mocked, but I can't figure out why that would happen. The issue doesn't seem to occur if the tests are run with --runInBand

And thanks for the --runInBand tip
I actually generally only ever run my tests in development with wallaby.js - which has no problem with that.


Can you add a game ?

Can you add "Marbles on Stream" on the game list ?
Thx you

It's already in there.


Fix typo

Sorry for the failed test, your change has nothing to do with it. We'll take a look at the test ASAP.
As for the actual proposed change, from the first sight it seems reasonable, but either me or someone from my team will get back to you later.

np)

Thank you for your contribution. We just checked with someone who wrote the code in the first place, and he things the tests are dormant and ought to be removed. See https://github.com/uber/ringpop-node/pull/286.
Closing.