Question about specific query

Hi,
I'm trying to make a specific query to the router, and it seems that the parser doesn't supports or I'm doing something wrong. Here's an example:
ip dhcp-server lease print where dynamic=no and active-address in <IP_ADDRESS>/24

This works from the terminal or in a ssh session. The invalid bit seems to be and active-address in <IP_ADDRESS>/24.
Is there a way to do this? I skimmed through the API documentation but I still can't figure out. Thanks in advance!

I can lab this to test it, but before I do that it would be helpful if you can give me a bit more information:

Can you show me the Java code snippet where you generate the command and call the API?
Can you show me the stack trace of the error you receive?

Hey, thanks for responding.
Sure, here's the code I'm using:
    try {
        ApiConnection con = ApiConnection.connectTLS(Config.HOST, ApiConnection.DEFAULT_TLS_PORT, Config.TIMEOUT);
        con.login(Config.USERNAME, Config.PASSWORD);

        List<Map<String, String>> rs1 = con.execute("/ip/dhcp-server/lease/getall where dynamic=no return address,comment");
        rs1.stream().forEach((r) -> {
            System.out.println(r);
        });

        /*
          Everything is ok here, a list of leases is printed
          ....
          tag=3, data={address=<IP_ADDRESS>, comment=Taller}
          tag=3, data={address=<IP_ADDRESS>}
          tag=3, data={address=<IP_ADDRESS>, comment=fire tv stick}
        */

        List<Map<String, String>> rs2 =  con.execute("/ip/dhcp-server/lease/getall where server=CMTS and address in <IP_ADDRESS>/24");

        rs1.stream().forEach((r) -> {
            System.out.println(r);
        });

        con.close();
    } catch { ...

Here's the stacktrace:
Nov 10, 2015 8:12:47 AM com.dschulz.getleases.GenerarScript main
SEVERE: null
me.legrange.mikrotik.impl.ParseException: Expected [=, <, >, !=] but found TEXT at position 62
    at me.legrange.mikrotik.impl.Parser.expect(Parser.java:183)
    at me.legrange.mikrotik.impl.Parser.expr(Parser.java:90)
    at me.legrange.mikrotik.impl.Parser.andExpr(Parser.java:121)
    at me.legrange.mikrotik.impl.Parser.expr(Parser.java:112)
    at me.legrange.mikrotik.impl.Parser.where(Parser.java:78)
    at me.legrange.mikrotik.impl.Parser.parse(Parser.java:27)
    at me.legrange.mikrotik.impl.Parser.parse(Parser.java:17)
    at me.legrange.mikrotik.impl.ApiConnectionImpl.execute(ApiConnectionImpl.java:75)
    at com.dschulz.getleases.GenerarScript.main(GenerarScript.java:43)

@dschulz I see the command language does support 'in', so apologies for that tangent. It doesn't look (according to the documentation) as if the API language supports "in". I will do a few tests though and see if there is a way of doing it.

Hi Gideon,
Thank you for taking the time to review my question.
I had to improvise, so what I did in the end was simply querying all the leases and then do the filtering in the application. I used a IP subnet calculator class that popped up clicking "I'm feeling lucky" :-)
I think it would be cool if you find a way to use the 'in' keyword. Unfortunately I have no idea on how to achieve this. I'll keep an eye on your commits and test the library as regularly as I can.
Thanks again!

@dschulz I went back to the API "documentation" and I experimented a bit, and I think that while the command line has the "in" syntax, the underlying API does not.
I have posted a question on the Mikrotik forum to see if somebody can help me out.
I'll let you know, but I think for the moment your post-processing solution is most likely the only good option.

@dschulz I've got confirmation on the Mikrotik forum that "in" is not supported in the API. So your current solution is the only one.


Build method without OPAM

Could you detail how to build ppx_deriving without OPAM for ppl with OCaml on MinGW?
From opam file, it seems to be
$ sed -e 's/%{version}%/1.1/g' pkg/META.in > pkg/META
$ ocaml pkg/build.ml native=true native-dynlink=true

In my environment the above builds things fine. The hard part is the installation. I had to run ocamlifind manually getting the file information from ppx_deriving.install.
Thanks!

I was able to compile ppx_deriving in the MSVC port of Ocaml with make. The only change that I had to do was to make was to change the ".a" extension to ".lib" in the pkg/topkg.ml file.
After building I had to manually fix the version in the META file and copy all the compiled files to my site-lib directory. Also I had to rename the ppx_deriving_main.native to ppx_deriving.exe

I've been using a newer topkg version for a while that has better Windows support--could you please recheck? @modlfo @camlspotter

There's still an issue installing in Windows. I get this error:
Installed C:\OCAMLM~1\lib\site-lib\ppx_deriving\META
mv `ocamlfind query ppx_deriving`/ppx_deriving_main.native `ocamlfind query ppx_deriving`/ppx_deriving.exe
mv: cannot stat ‘C:\\OCAMLM~1\\lib\\site-lib\\ppx_deriving\r/ppx_deriving_main.native’: No such file or directory
Makefile:52: recipe for target 'install' failed
make: *** [install] Error 1

there's an strange \r in the path name. I don't know if this happens only in my computer.

@modlfo can you try again with the last command being...
mv `ocamlfind query ppx_deriving -suffix /ppx_deriving_main.native` `ocamlfind query ppx_deriving -suffix /ppx_deriving$(EXE)`

?

That las command works correctly. I get the ppx_deriving.exe in the library folder. Thanks!


set mode explicitly to notebook

This PR fixes #18088
VSCode added autodetectLanguage step whenever file content changes. And unless the file mode (language) is explicitly set, it tries to pick a language for the file.
Added setMode in notebookInput to explicitly set the language.

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 increased (+0.01%) to 48.285%

Totals

Change from base Build<PHONE_NUMBER>:
0.01%

Covered Lines:
14749

Relevant Lines:
29007

💛  - Coveralls

Please also make sure we have a test in the signoff plan for this scenario

Please also make sure we have a test in the signoff plan for this scenario

https://mssqltools.visualstudio.com/CrossPlatBuildScripts/_workitems/edit/9622


ARROW-13842: [C++] Bump vendored date library

This fixed an undefined behaviour issue on extreme timestamp values.

Will undraft after #10610 is merged.

@github-actions crossbow submit -g cpp

@github-actions crossbow submit -g example-cpp


Allow creating an empty VisModel and setting its state programmatically

[ ] Decouple VisModel form VizJSON (here)
[ ] Define the visModel public API
[ ] Update unit tests for this api

After this PR, you should be able to create a new VisModel and manage layers, dataviews, analysis, etc.

Wrong approach. Closing in favor of https://github.com/CartoDB/cartodb.js/issues/1749.


Where did you found the Flow API

Hi,
How where you able to figure out how the flow api works?
I'm trying to build my own integration ( with Grafana ) and would like to know how to authenticate.
I see flow now uses SingleKey as oauth backend instead of the Bosch one.
Maybe something you have to fix in your implementation as well.
Wouter

+1


Unable connect to Spark 2.1.0

I tried to run devtools::install_github("rstudio/sparklyr") to install the package. I refer to https://github.com/rstudio/sparklyr/issues/288 for the solution but there error keep exist.
When I tried to connect to Spark, the system prompt me invalid java's version.
spark_version <- "2.1.0"
sc <- spark_connect(master = "local", version = spark_version)
- Error in get_java(throws = TRUE) :
-   Java is required to connect to Spark. JAVA_HOME is set but does not point to a valid version. Please fix - JAVA_HOME or reinstall from: https://www.java.com/en/

But I had installed Java when I tried to check if system detected.

> devtools::session_info()$platform
 setting  value
 version  R version 3.4.1 (2017-06-30)
 os       Windows >= 8 x64
 system   x86_64, mingw32
 ui       RStudio
 language en
 collate  Japanese_Japan.932
 tz       Asia/Tokyo
 date     2017-09-21

@englianhu looks like JAVA_HOME is not pointing to the right path, from PS I believe we want to run $Env:JAVA_HOME to see what the value is, could you follow that by listing the directory it points to with Get-ChildItem $Env:JAVA_HOME?

Solved with modify the path https://d.cosx.org/d/419485-sparklyr-spark-2-1-0


Fix avatar creation for not logged-in authors

If conversation message authors are not logged in, we use the EmptyAvatar class and its userInfo property does not have any entity associated.
Thus, calling $this->userInfo->getUserName() leads to a Call to a member function getUserName() on null exception.

Can you specify the steps necessary to reproduce the error that this solves? Someone else reported this but the message formatter should take care of this use case
Sent from my iPhone

On Aug 19, 2016, at 3:47 AM, Michele Locati<EMAIL_ADDRESS>wrote:
If conversation message authors are not logged in, we use the EmptyAvatar class and its userInfo property does not have any entity associated.
Thus, calling $this->userInfo->getUserName() leads to a Call to a member function getUserName() on null exception.
You can view, comment on, or merge this pull request online at:
https://github.com/concrete5/concrete5/pull/4232
Commit Summary
Fix avatar creation for not logged-in authors
File Changes
M concrete/src/User/Avatar/EmptyAvatar.php (5)
M concrete/src/User/Avatar/StandardAvatar.php (11)
Patch Links:
https://github.com/concrete5/concrete5/pull/4232.patch
https://github.com/concrete5/concrete5/pull/4232.diff
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.

Brand new c5 installation with sample contents -> add a new conversation message as guest, not as logged-in user -> KABOOM

Ah, I see; you are correct. This is due to some refactoring between 5.7 and v8. My mistake.

Fixed in a different way


Machine match

Internal Winboard software engine (theirs not our) vs our LegalPlayer (black).
Menu Machine Match!
Try running. It's interesting : our player did several moves and then crashed. Need to simulate and check what's happening.

It looks now it works fine!


[SPARK-25131]Event logs missing applicationAttemptId for SparkListenerApplicationStart

When master=yarn and deploy-mode=client, event logs do not contain applicationAttemptId for SparkListenerApplicationStart. This is caused at org.apache.spark.scheduler.cluster.YarnClientSchedulerBackend#start where we do bindToYarn(client.submitApplication(), None) which sets appAttemptId to None. We can however, get the appAttemptId after waitForApplication() and set it
This i have tested manually and verified

Can one of the admins verify this patch?

Can one of the admins verify this patch?

Can one of the admins verify this patch?

Is this really necessary? It will always be "1", since client-mode apps are not re-tried (the YARN AM might be, but the driver is not). That makes it not really useful.

It is not that the change is trivial, it's that I don't see the point of it. There is no concept of "attempts" in Spark client mode. So why fill in this information at all?
It's an "option" in the event for a reason: not all modes support re-attempts. If some listener is confused by that, it's a problem with that listener.
Sorry but I don't think this should go in.


Postprocessing in compassR vs manuscript

In the manuscript the scores are said to be scaled with -log(1+score) while in compassR get_reaction_consistencies with -log2(score) followed by some filtering. Which should now be used?

Also, I have one reaction that has scores below 0 R_NADtx_neg  from Recon3D on mouse.

The compassR code is slated for substantial revisions. We recommend -log(1+x) currently, as used in the manuscript here.
To scale the raw compass scores we subtracted the minimum entry in the matrix after taking -log(1+x) which makes them all non-negative. However, the reaction scores you are getting look unusually small if they are all on the order of 1e-12. In fact we removed rows in the matrix where the difference between maximum and minimum is less than 10^-3 for the manuscript.
If you can give some more details on how you added Recon3 and what dataset you're running this on, I can definitely take a look at what's occurring.

Thank you for the explanation.
I added this data  compass/Resources/Metabolic Models/RECON3/Recon3D.xml.gz and changed possible input options https://github.com/Hrovatin/Compass/commit/84ea2f9a6425673f44e8b89506d6adae8bbd4238
I get only one such reaction where score is below 0. Others look normal (top - scores, bottom - consistencies).

I have removed reactions with any score below 0 or all scores==0 for consistencies.


[UNKNOWN PROPERTY] issue

[Y] I have read the contribution documentation for this project.
[Y] I agree to follow the code of conduct that this project follows, as appropriate.
[Y] I have searched the issue tracker for an issue that matches the one I want to file, without success.

Please describe your issue:
Hello. I just downloaded electron packager and i runned the command
electron-packager ./ Client --platform=win32 --icon=icon.ico
and i ran it. I've used this tool before but instead of the output, it said that "Unable to determine [UNKNOWN PROPERTY]. Please specify a [UNKNOWN PROPERTY].
I've gotten this error and i've looked at everything for about ~20 to ~30 minutes. I didn't find anything, help!

Console output when you run electron-packager with the environment variable DEBUG=electron-packager. Please include the stack trace if one exists.
Unable to determine [UNKNOWN PROPERTY]. Please specify a [UNKNOWN PROPERTY]

What command line arguments are you passing? Alternatively, if you are using the API, what
parameters are you passing to the packager() function?
./ Client --platform=win32 --icon=icon.ico

Please provide either a failing minimal testcase (with a link to the code) or detailed steps to
reproduce your problem. Using electron-quick-start
is a good starting point.

Thanks for filing an issue! In order to help you with your problem, we're going to need more information about it. In particular:

Console output when you run electron-packager with the environment variable DEBUG=electron-packager, for example DEBUG=electron-packager electron-packager ./ Client --platform=win32 --icon=icon.ico on macOS or Linux.

Please provide either a failing minimal testcase (with code) or detailed steps to reproduce your
problem. Using electron-quick-start is a good starting point. Alternatively, please provide your package.json file.

Will do. The testcase is something i cannot provide, due to my computer causing the error, and yours wouldn't be able to replicate.

Since the original issue reporter has not responded to the request for additional information in about two weeks, I'm going to assume that they either found the answer in another manner or decided not to pursue this issue. If I'm incorrect, they can respond with answers to the questions posed, and I'll reopen.


Fix README, make sure it renders

If it does not render, we won't be able to publish a release on PyPI, see https://github.com/scrapinghub/extruct/pull/168#issuecomment-751741909

D'oh, it'd have been useful for me to double-check this during #162.  I guess GitHub's markup rendering is more lenient than PyPi.  Thanks @lopuhin!

No worries, it wasn't rendering correctly for quite a while, good that it's now fixed and looks much prettier on PyPI.


added example of edge highlighter

Hi Marko, Im trying to add an example of the edge highlighter.

merging


Gradle support

Provide gradle build support for GitFX

Good tutorial http://www.vogella.com/tutorials/Gradle/article.html

Fixed in #64


[Mixer/CEL] Mixer's match function with CEL counterpart (matches).

Mixer currently has a "match" function that performs a prefix/suffix match:
match("ns1.svc.local", "ns1.*") => true
match("ns1.svc.local", "ns2.*") => false
match("svc1.ns1.cluster", "*.ns1.cluster") => true
match("svc1.ns1.cluster", "*.ns1.cluster1") => false
CEL also introduces a similar function matches:
https://github.com/google/cel-spec/blob/master/doc/langdef.md
// matches second argument against regular expression in first argument
matches: (string, string) -> bool
i.e.
match("ns1\\.svc\\.local","ns1.svc.local") => true
match("ns1.*","ns1.svc.local") => true
match("ns2.*","ns1.svc.local") => false
These two functions have very similar names, and are prone to cause confusion. Given that we both want CEL alignment and want to have regex matching support, it makes sense to refactor the existing match function.
The most straightforward refactoring is to introduce two special case methods for prefix or postfix matching:
// Picked starts & ends as names, to start the discussion. We can replace with better names.
starts(prefix, expression)
ends(suffix, expression)
i.e.
starts("ns1", foo)
starts("ns1.", "ns1.svc.local") => true
starts("ns2.", "ns1.svc.local") => false
ends(".ns1.cluster", "svc1.ns1.cluster") => true
ends(".ns2.cluster", "svc1.ns1.cluster") => false
Alternatives are:

Keep things as is:

Names are too close to each other and is bound to cause confusion.

Rename matches (i.e. mask): mask("ns1.*", foo)

Not a commonly encountered type of string function: Specialized format, involving "*", that suspiciously looks like regex. Many languages have (separate) string suffix/prefix check functions natively.

Remove the function altogether and depend on "matches".

Writing some of these constraints in regex can be awkward:

starts("ns1.svc.", foo) v.s. match("ns1\\.svc\\..*", foo)

@geeknoid @mandarjog @guptasu @douglas-reid

I like starts/ends, they're simple and intuitive.

starts/ends seems reasonable. As for better names, I might be inclined to use startsWith and endsWith, or hasPrefix and hasSuffix.
I also wonder whether or not the prefix/suffix arg should be second in the args order (hasPrefix(foo, "ns1")), but I guess we should follow whatever the CEL convention is.

I did a cursory research in CEL implementations and, there is very little reliable information to glean from.
It looks like matches has two implementations in two different places: one instance method, the other not (i.e. "foo.".matches("fooo") v.s. matches("foo.", "fooo") ). The matches parameter ordering makes sense for use as instance method model.
For prefix/suffix functions, my initial thinking was to align with non-instance matches, and also tried to avoid composite words since it can cause speed bumps in reading & writing. However, if we can model these as instance methods (i.e. "foo".startsWith("f")), then it makes sense to change both the parameter ordering and the name as well.
I am following up with the CEL folks, will update.

Here is the latest:
matches function in CEL will be an instance method.
There is an ongoing proposal for startsWith/endsWith instance methods.
With this, the final shape will be:
"regex".matches("input") => bool
"str".startsWith("prefix") => bool
"str".endsWith("suffix") => bool


(FACT-1075) Add flag to display legacy facts

Facter 3 moved to only showing structured facts, hiding most flat fact
names used by Facter 2 in the default command-line output. This is
difficult for tools parsing the command-line output, particularly when
using json or yaml output.
Add a flag to enable displaying all legacy facts as part of command-line
output querying all facts.

CLA signed by all contributors.

:+1: this did end up being pretty simple! I verified that the option works as expected in both OSX and Ubuntu 14.04.

@MikaelSmith oh one other thing - we'll want to add this option to the man page.

Updated.

:+1: will merge after CI passes.


Missing getmempoolentry

Here's a sample call:
getmempoolentry abed2bc7cc77a4bc342a67e29da974b9cc69d47afa6279463d916009b1bcb6a3
￼
{
"size": 372,
"fee": 0.00008280,
"modifiedfee": 0.00008280,
"time":<PHONE_NUMBER>,
"height": 1085243,
"startingpriority":<PHONE_NUMBER>052631,
"currentpriority":<PHONE_NUMBER>526316,
"descendantcount": 1,
"descendantsize": 372,
"descendantfees": 8280,
"ancestorcount": 1,
"ancestorsize": 372,
"ancestorfees": 8280,
"depends": [
]
}

In my particular use-case, the fields I need are `ancestorsize` and `ancestorfees` as I'm trying to compute the feerate to guestimate if it'll confirm. I notice these fields are also missing from btcjson.GetRawMempoolVerboseResult.  So it'd be nice if they were also made complete

https://github.com/btcsuite/btcd/pull/905

@dajohi: For reference, even if you add those fields to the result as 905 currently does, none of the ancestor/descendant information is available to populate the results.  The mempool doesn't current keep track of that information.

@FrozenPrincess Are you able to give #117 a try?


CategoryRelations xcat_trims argument with single None gives empty  df

Work already complete - held in Unit Test 262.


Enforcing BlockKit limits

In looking to replace our internal BlockKit implementation with this one, something that came up is that we'd lose the support we built for enforcing Slack's BlockKit limits on text fields and number of elements in various blocks.
Is this support that you'd be interested in accepting a pull request for?  Do you have thoughts about how this should be implemented in this library?

Hey, this would definitely be a welcome addition!
I would be interested in hearing about how you approached solving this internally and any strategies in particular for managing these validation failures (and if those have/would impact the design of any interface!).
It might be worth also exploring whether we can leverage existing tools to validate payloads -- rather than having to implement some kind of validation interface into this lib.
It looks like (although somewhat undocumented) it is possible to extract a JSON schema for block kit messages from their online block builder:

https://github.com/slackapi/slack-api-specs/issues/18
https://gist.github.com/renatorib/1fb1a9bd71435b41bee602d15bc56899

This could be functionality that lives within this library or independently as a "validator" for block kit JSON -- which may or may not have been generated via this gem 🤔

We've got a pretty straightforward system right now:

If you add an element to an array that exceeds max value, an exception is raised
If you try to create a field with a text field longer than allowed, an exception is raised

I agree that using the JSON schema would be the right way to do this.  I took a swing at using that gist after digging out the non-truncated one that's buried in there.  I couldn't get it to validate properly -- it seemed like it was only for a portion of the BlockKit spec or was too old to be useful, but I only took a pretty quick pass on it.
We've also reached out to Slack to see if they can provide this to us, but we haven't heard anything back yet.
If we were to do the former, I think it'd have to be embedded in the library.  If we can take the schema-validator approach, I agree that we could have some broader utility if we keep it independent.

It does seem that maybe they are no longer exposing that schema anywhere anymore.
The current block builder UI now makes an RPC call to /api/blocks.format in order to run validation, rather than having any kind of client-side validation happening it seems.
If Slack don't provide a schema, I think it could be beneficial to scrape one together and keep it up to date over time, especially if it is in a language agnostic form.
To some extent, if there were an exposed schema, most of the code currently in this gem could simply be generated, which might be quite nice and guarantee a level of consistency in the API, depending on how nice the generator was!

Would be willing to help move this forward, is there a working branch/fork available?

Hey there. I sketched out a possible approach here https://github.com/CGA1123/slack-ruby-block-kit/pull/180 and would love to hear if this might help. The PR adds the ability to flexibly check for limit/schema violations (though doesn't use json schema but rather just checks limits advertised in the documentation) as well as sanitize the json produced during serialization. This allows for, say, option labels to be truncated rather than raising an error. You could imagine implementing your own "limiter" to additionally provide tracing or logging in such cases.
This is just a sketch to illustrate a few different scenarios but I'd love feedback on whether this seems useful. Thanks!

Hey @lritter -- thanks for taking the time to look at this and for providing a potential solution. Much appreciated!
I apologise for the delayed reply. I haven't yet been able to make the time to think this through in full, but wanted to ACK this and will do my best to make some time in the evening this week or over this coming weekend.
In the meantime, would be interested to hear from @bougyman + @bmorton about thoughts and whether this would cater to your use-cases -- you are better placed than I am to shed light on clear requirements/wants from a solution!

Appreciate it @CGA1123!
I'll say that one of the things I don't like about my solution is that it mixes the "validation" concern with the "make it work" concern (modifying supplied values such as long labels to allow things to work). Doing the "make it work" stuff at the application layer makes sense but being able to hook into the framework to do this greatly eases development and reduces the chance of missing stuff. The PR here seems like "the simplest thing that could work" but I'm definitively curious about other options.
Thanks!

I think the idea of a serialisation middleware stack is slick, and allows for all sorts of flexibility!
There's then probably a set of common implementations for field types that have common restrictions (text length, enum types, etc...) which probably need to have some different ways to "fail": notify, raise, "try to autofix" (e.g. truncate).
I like this because it also keeps the door open for curating or finding a JSON-Schema based approach for all these components, and potentially leveraging existing tooling for validation around that.


Prevent using card images in other languages

We should always use images in english if at all possible. So we should check if we can use another printing, if possible or if not find a different card.

This is only a problem if the card was only released in a foreign language. So this is not a real problem…


When auto-selecting GPUs, prioritize previously used GPUs

Proposed refactoring or deprecation
When auto-selecting GPUs, Lightning iterates over available CUDA devices in sequence as follows:
https://github.com/PyTorchLightning/pytorch-lightning/blob/4755ae60b71a857a17f39dba1189c2c3245660f5/pytorch_lightning/tuner/auto_gpu_select.py#L48-L58
We should first check GPUs that have been previously used, and then check the unused ones.
Motivation
Say we have processes A, B and C, and two GPUs configured in exclusive process mode. Consider the following situation:

Process A is using GPU 0
Process B, with Lightning, runs a Trainer that auto-selects GPU 1 (since A is using GPU 0).
Process A exits, clearing up GPU 0.
Process B finished using the Trainer, and thus 'frees' up GPU 1. However, B doesn't exit, and so there is still some small amount of memory reserved by PyTorch on GPU 1.
Process B creates a new Trainer, that invokes the auto-selection again. It sees that GPU 0 is available, so it reserves it.
Process C wants to get a GPU. However, since B is fully using GPU 0, and it's reserved GPU 1 (even though it's not using it), it doesn't get any GPU.

Pitch
The GPU selection routine should be changed to:
previously_used_gpus = []
unused_gpus = []
for i in range(torch.cuda.device_count()):
    if torch.cuda.memory_reserved(f"cuda:{i}") > 0:
        previously_used_gpus.append(i)
    else:
        unused_gpus.append(i)

# Prioritize previously used GPUs
for i in previously_used_gpus + unused_gpus:
    if i in exclude_gpus:
        continue
    # Try to allocate on device:
    device = torch.device(f"cuda:{i}")
    try:
        torch.ones(1).to(device)
    except RuntimeError:
        continue
    return i
raise RuntimeError("No GPUs available.")

I've made the changes here: https://github.com/rharish101/pytorch-lightning/tree/feature/10456-better_gpu_select
cc @justusschock @awaelchli @akihironitta

@kaushikb11 Any thoughts ?

I also noticed that pick_single_gpu is not used anywhere in the code.
I've added this to my branch (https://github.com/rharish101/pytorch-lightning/tree/feature/10456-better_gpu_select) as follows:
https://github.com/rharish101/pytorch-lightning/blob/81ef04f1af2dbf8c327906cb59e37bd16a006f90/pytorch_lightning/tuner/auto_gpu_select.py#L21-L64
I can open a PR if you want.

Hi @rharish101, the proposal sounds good.
I have a question, here you check if the GPU is being used before.
if torch.cuda.memory_reserved(f"cuda:{i}") > 0:
	...

Let's consider GPU 0 has some PyTorch program running, it will return as True and crash if it is selected by Trainer, right?
Wouldn't that be bad user experience? As there would be other GPUs available.

Hi @kaushikb11, AFAIK torch.cuda.memory_reserved only shows memory reserved by the current process, not by any other PyTorch process. That way, we are actually making it safer.

@rharish101 Sounds good! :)
You could go ahead with the PR. Thanks!

Thanks for the approval! I've opened a PR (#10485) for the prioritization. I'll open another PR for the refactoring after this PR is merged (otherwise there would be a merge conflict with the PR merged later).

I'm hesitant about this change for the reasons @kaushikb11 mentioned.

only shows memory reserved by the current process, not by any other PyTorch process

Do you have any references for this? The docs don't say it:
https://pytorch.org/docs/stable/generated/torch.cuda.memory_reserved.html#torch.cuda.memory_reserve.
In the past, when I've had to share GPUs I preferred the opposite, prioritizing unused GPUs to minimize the impact of different scripts on memory and throughput.

Do you have any references for this?

Nope, I manually ran two scripts, each of which reserved memory by allocating a tensor, and printed torch.cuda.memory_reserved.

In the past, when I've had to share GPUs I preferred the opposite, prioritizing unused GPUs to minimize the impact of different scripts on memory and throughput.

Do you mean that when you have multiple GPUs and multiple processes, you would prefer them to use separate GPUs? Because that is what my proposal does, as this tells each process to prioritise GPUs that it previously used, and not infringe on GPUs that could be used by other processes.
If you prefer the opposite, then it wouldn't work with the example mentioned in the issue (which would lead to unnecessary GPU sharing), especially when running in exclusive process mode.


Fix for #8918 The Community Administrator should not be able to view all communities/collections in the create/edit community and collection sections.

References

Fixes #8918

Description
The Community Administrator should not be able to view all communities/collections in the create/edit community and collection sections.
Instructions for Reviewers
Currently, users can see all communities/collections in the create/edit community and collection sections. However, the expected behavior is that only the communities/collections where the user is a community or collection admin should be displayed.
List of changes in this PR:
•	Added a method retrieveObjectWithAdmin in the AuthorizeService and implemented it in the AuthorizedServiceImp class. This method returns a filtered result, showing only the communities/collections where the user is a community or collection admin.
How to Test

Make a user the admin of a few communities.
Navigate to the “New” section and click on "Community" or “Collection”. Only the communities where the user is community admin should be displayed.
For the Edit section:
o	Go to Edit > click “Community”. Again, only the communities where the user is an admin should be shown.
o	For collections, go to Edit > click “Collection”. Only the collections where the user is a collection or community admin should be listed. If the user is a community admin for a community, all collections within that community should also be shown.

Bug Reproduction Steps

New Section:
o	Log in as a community admin, go to New > Community. Currently, all communities are shown in the list.
o	Similarly, for collections, go to the New section > click Collection. Here, all collections are shown in the list.
Edit Section:
o	Log in as a community admin, go to Edit > Community. Currently, all communities are shown in the list.
o	Similarly, for collections, go to Edit > click Collection. Here, all collections are shown in the list.

Thank you @im-shubham-vish. The patch applies on DSpace 7.6.3-SNAPSHOT so I tested it there by creating a user who is a member of a community administrator group and checking the behavior of "Create Community" and "Create Collection" before and after. After applying the patch I only saw communities where the user had administrator permissions. :+1:
Side note: this patch has some whitespace issues according to git apply, and seems to be out of date with current main, including some extra commits here. I would suggest rebasing this pull request on top of current main. @tdonohue do we have a lint to apply consistent formatting of Java code?

Thank you @alanorth for the feedbacks.
We are done with the rebasing of this pull request on top of current main. Regarding the whitespace issue mentioned, we had ran the lint locally which got passed over there before raising this pull request.
Please let me know in case you've any other feedbacks which we need to incorporate.

@im-shubham-vish it seems you have accidentally included a dozen commits from DSpace main along with your changes (I see 13 commits here, and only two are related to your fix). I can help you clean it up if you like. Basically, I would recommend cherry-picking those two commits, squashing them into one, and rebasing on top of DSpace main.
Also, you're developing directly on your main branch which is a bit messy. I would recommend doing your development from a feature branch instead, for example git checkout -b fix-8918, because it makes pull requests cleaner. Your main is downstream from DSpace's so you should make sure that it does not diverge.

Thank you @alanorth for the help.
I'm done with the changes. Can you please check and let me know your feedback.

Thanks @im-shubham-vish. I tested this again and it is still working as expected and the PR is cleaner now. Thanks!


Running from windows: command line question

Hi !
Can you please explain these lines on the docker run configuration ?
--device=/dev/net/tun --cap-add=NET_ADMIN
-v /etc/localtime:/etc/localtime:ro \
I'm trying to start it from windows but I don't have a volume /etc/localtime and it doesn't seems to work...
Thanks

Ok I've tried initially in git bash but git bash doesn't recognize --device=/dev/net/tun, --cap-add=NET_ADMIN is to add capabilities for managing network for what I saw. So I launched it without the volume and in powershell and it works.

I've never used Docker on Windows, so could help. Thanks for reporting back. :star:


Expose the size parameter to Stream#read

There was an oversight in the previous PR in that we did not expose the optional size parameter: https://nodejs.org/api/stream.html#stream_readable_read_size.
I've opted for a breaking change that adds Maybe Int to all relevant signatures. However, we could
also add prime variants: read', readString' and readEither'.

👍 LGTM. What are you using this for, by the way?

I have released 0.6.0 which includes this.

Oh also, would you be able to include some documentation for these functions? If you send another PR I can do a patch-level release.

Thanks! Yeah I can add docs later today. I am using these functions in an experimental module I am building that lets you use nodejs streams with purescript-pipes: https://github.com/felixSchl/purescript-node-pipes/blob/master/src/Pipes/Node.purs


WIP: Can specify fn branch to build from with --fn-branch

Requires https://github.com/fnproject/fn/pull/664

No activity since Jan 10. Closing...


chatgpt.gpt3.apiBaseUrl setting not working after v3.20.0

I am using oobabooga's text-generation-webui with its openai extension to serve local llms for ChatGPT Reborn to use.
The only setting I have changed is "chatgpt.gpt3.apiBaseUrl": "http://<IP_ADDRESS>:5000/v1", and I have provided my own API key
Versions v3.20.0 works correctly.
Maybe this is to blame: https://github.com/Christopher-Hayes/vscode-chatgpt-reborn/blame/41107ed508a050d42d995c06334ca1279c7c4f3d/src/api-provider.ts#L34

Thanks for the report, will take a look. OpenAI made breaking changes to their API schema to support GPTs and it looks like proxies broke in the process of updating to support the new API schema.

Fixed in v3.23.0


Jquery 1.9 considered insecure

Hi,
We've been informed that the jquery version is considered insecure (the jquery-ui security ticket that you already have open was also considered):
https://nvd.nist.gov/vuln/detail/CVE-2015-9251
https://nvd.nist.gov/vuln/detail/CVE-2019-11358#vulnCurrentDescriptionTitle
What would it take to bump jquery to a newer version - happy to try and allocate some resource to help.
Thank you,
Damian

Nice, go !


Databank keeps crashing

Describe the bug
My Data bank keeps crashing with the same Error over and over and i don't understand why
Environment Details (please complete the following information):

OS: Ubuntu (Docker)
Chrome
Jellystat Version latest
Jellyfin Version latest

Docker Config
version: '3'
services:
jellystat-db:
image: postgres:15.2
environment:
POSTGRES_DB: 'xxx'
POSTGRES_USER: xxx
POSTGRES_PASSWORD: xxx
volumes:
- ./postgres-data:/var/lib/postgresql/data # Mounting the volume
networks:
- codeserver_default  # Hinzugefügte Netzwerkzuordnung
jellystat:
image: cyfershepard/jellystat:latest
environment:
POSTGRES_USER: xxx
POSTGRES_PASSWORD: xxx
POSTGRES_IP: jellystat-db
POSTGRES_PORT: 5432
JWT_SECRET: 'my-secret-jwt-key'
ports:
- "824:3000" #Server Port
volumes:
- ./backup-data:/app/backend/backup-data # Mounting the volume
depends_on:
- jellystat-db
networks:
- codeserver_default  # Hinzugefügte Netzwerkzuordnung
restart: on-failure
networks:
codeserver_default:
external: true
To Reproduce
Steps to reproduce the behavior:

Set up a fresh instance
Wait until it crashes (At least for me)

Expected behavior
My Data bank keeps crashing, and I couldn't figure out why. So I did a fresh install and imported my old Backup. But I get the same Error.
Task Logs
PostgreSQL Database directory appears to contain a database; Skipping initialization
2024-02-25 22:39:39.102 UTC [1] LOG:  starting PostgreSQL 15.2 (Debian 15.2-1.pgdg110+1) on x86_64-pc-linux-gnu, compiled by gcc (Debian 10.2.1-6) 10.2.1 20210110, 64-bit
2024-02-25 22:39:39.102 UTC [1] LOG:  listening on IPv4 address "<IP_ADDRESS>", port 5432
2024-02-25 22:39:39.102 UTC [1] LOG:  listening on IPv6 address "::", port 5432
2024-02-25 22:39:39.107 UTC [1] LOG:  listening on Unix socket "/var/run/postgresql/.s.PGSQL.5432"
2024-02-25 22:39:39.117 UTC [29] LOG:  database system was shut down at 2024-02-25 22:39:27 UTC
2024-02-25 22:39:39.130 UTC [1] LOG:  database system is ready to accept connections
2024-02-25 22:39:44.575 UTC [33] ERROR:  database "jfstat" already exists
2024-02-25 22:39:44.575 UTC [33] STATEMENT:  CREATE DATABASE jfstat
Additional context
I think this is the Problem: "ERROR:  database "jfstat" already exists" but I have no Idea how to fix that.

Hey @neuleo , that error is fine, all its saying is we tried to create the database but it found an existing one, this is then handled inside jellystat to proceed with the rest of the setup. Do you have any other logs that could help?
Are you sure the database is crashing and not jellystat itself? You will have to also check in the Postgres forums for help as this issue doesn't really  look like its related to jellystat from what i can see.


Update RouteItem and add test cases

Currently the method property in RouteItem is typed as HTTPMethods but with fastify.route you can also add an array of routes. I updated the types accordingly and added test cases

Done :)


Bumping version number after adding the spread operator to the button.

What does this PR do? (please provide any background)
This PR changes bumps the version number and details the change in the CHANGELOG.
What tests does this PR have?
How can this be tested?
Screenshots / Screencast
What gif best describes how you feel about this work?

[ ] I have checked our general contributing document and the project specific contributing document (if present) and I'm happy for this to be reviewed.

Reviewers
Review 1

[ ] :+1:

Review 2

[ ] :+1:

Review 3

[ ] :+1:

By adding a +1 you are confirming you have...

Witnessed the work behaving as expected (this could be on the author's machine or screencast).
Checked for coding anti-patterns.
Checked for appropriate test coverage.
Checked all the tests are passing.

plus one


EPR Compliance Page Added

Related Issue
Completed #1535
@arghadipmanna101 Please review and merge this PR as soon as possible.
Description
EPR Compliance Page Added
Type of PR

[x] Bug fix
[x] Feature enhancement
[x] Documentation update
[x] Other (specify): _______________

Checklist:

[x] I have performed a self-review of my code
[x] I have read and followed the Contribution Guidelines.
[x] I have tested the changes thoroughly before submitting this pull request.
[x] I have provided relevant issue numbers, screenshots, and videos after making the changes.
[x] I have commented my code, particularly in hard-to-understand areas.

resolve conflicts.

resolve conflicts.

@arghadipmanna101 I have resolved all the conflict issues. Please merge this PR as soon as possible.

@arghadipmanna101 I have resolved all the conflict issues. Please merge this PR as soon as possible.

@arghadipmanna101 I have resolved all the conflict issues. Please merge this PR as soon as possible.

resolve conflicts.


JDBC support for public Date getDate(int columnIndex, Calendar cal) throws SQLException

JDBC support for:
java.sql.ResultSet: public Date getDate(int columnIndex, Calendar cal) throws SQLException
java.sql.ResultSet: public Date getTime(int columnIndex, Calendar cal) throws SQLException

Also timestamp:
public Timestamp getTimestamp(int columnIndex, Calendar cal) throws SQLException
BTW, the return of getTime should be Time, not Date

Thanks,
The error it throws Identifies these methods as "Not supported yet"
We are slowly working through all methods, as internal priorities allow, but welcome contributions from the broader community if you need these fixed sooner.

I implement d those methods last December to make the driver work with microstrategy; but I don’t know if I implemented them correctly. It worked but I haven’t control on ms calls so I am not sure the result is correct.
I can share the code anyway
Ottieni Outlook per iOShttps://aka.ms/o0ukef

From: Michael Thomson<EMAIL_ADDRESS>Sent: Friday, March 30, 2018 10:13:26 PM
To: mapd/mapd-core
Cc: Subscribed
Subject: Re: [mapd/mapd-core] JDBC support for public Date getDate(int columnIndex, Calendar cal) throws SQLException (#204)
Thanks,
The error it throws Identifies these methods as "Not supported yet"
We are slowly working through all methods, as internal priorities allow, but welcome contributions from the broader community if you need these fixed sooner.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHubhttps://github.com/mapd/mapd-core/issues/204#issuecomment-377612151, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AdZQl_5jskPpBIkUWAUB9DEVzC7Jw3cBks5tjpHmgaJpZM4TB2sx.

@cdessanti  thanks , awesome!  I wonder whether we can push them to main/next branch? In that way everyone takes advantage of it.

Fixed by #331


trading complete sets, user cover short, order confirmation is wrong

This is only an issue if user has existing complete sets.
If a user has a short position, and goes to cover their short position, adding a buy order to the order ticket, the order ticket only displays "New Position" info and is not displaying the "Closing Position"info

Complete sets, have market maker create liquidity.
Have second user take all orders. market maker should be left with complete sets.

You can also get complete sets by making orders for each outcome (in binaries and scalars they must not cross)

That was easy, worked like a charm!

Things look good with small change
Scenario: User has complete sets but order form confirmation looks correct.
User can buy/sell and the order form confirmation says user is opening new position.
Issue come in with synthetics when user buys/sells shares when they have complete sets.
There is a netPosition of 0 on No with no realized/unrelaized it shows up as a Yes long position with zeros. UI can filter that position out.
With this change the user can create new positions and close position and not get mixed up.
See what you think @Chwy5

Makes sense to me when there is ranges between dates....Like Sept 8, Sept 12, Sept 16th etc, Little bit weird to me for daily dates though when the dot doesnt match up with the coordinating date.

tested and fixed


Support imports

Is proto-convert able to handle import in proto files?

Like this?
https://github.com/iamazeem/protobuf-log-sample/blob/6fb30653e595629cead29a26db03f55ce5590e5e/log.proto#L5

Hello.
With very similar proto file, we are facing below issue.
Types.proto file:
syntax = "proto3";

package net.sample;

import "google/protobuf/any.proto";

option java_multiple_files = true;

message Type {
  string traceId = 1;
  google.protobuf.Any header = 2;
  google.protobuf.Any body = 3;
}

test.json file:
{
  "traceId": "2",
  "header": {},
  "body": {},
}

Results:
./proto-convert -m j2b -p Types.proto -t net.sample.Type -i test.json -o test.bin

Possible 'import' issue! Use a single self-contianed .proto file! cannot load such file -- google/protobuf

Looking at the code that error message it's here:
https://github.com/iamazeem/proto-convert/blob/77ed15facc343191e9a3a0e80f9d5581c1e65a6d/bin/proto-convert#L182
btw, has a typo there, it's "self-contained", not "self-contianed" it should say. Plus it could say "Try" instead of "Use" if that is just a suggestion for cases where the importing failed for someother reason (can't it show some msg for that reason?) and the tool does support imports

Another thing I was looking for was how it uses "proto_path" command-line param of protoc, seems it's being used here:
https://github.com/iamazeem/proto-convert/blob/77ed15facc343191e9a3a0e80f9d5581c1e65a6d/bin/proto-convert#L45
from what I understand it tries to use the folder of the passed-in file as the root folder. But that will be a relative path and not sure if protoc compiler treats such relative paths as relative to its own folder.
Maybe should use an absolute path there, that is:
file_dir = File.expand_path(File.dirname(file_path))
see the replies at:
https://stackoverflow.com/questions/15358669/confusing-behavior-of-file-dirname
and related doc:
https://ruby-doc.org/core-2.0.0/File.html#method-c-expand_path
seems this will also support "~" :

The given pathname may start with a “~'', which expands to the process owner's home directory (the environment variable HOME must be set correctly). “~user'' expands to the named user's home directory.

./proto-convert -m j2b -p Types.proto -t net.sample.Type -i test.json -o test.bin

Possible 'import' issue! Use a single self-contianed .proto file! cannot load such file -- google/protobuf

@pawelantczak
Hi,
Looks like protoc was not able to find the google/protobuf:
cannot load such file -- google/protobuf

Please make sure that the files from the include folder are available.
There's no information about your environment so I won't be able to suggest anything.
On my Ubuntu 18.04 machine, the installed paths are:
/usr/local/bin/protoc
/usr/local/include/google/protobuf/

Once installed, it should work.
I tested with your proto and it was compiled successfully.
However, I'm observing another issue with the protobuf itself.
Please test on your side and let me know if there's anything.
I'd appreciate it if you could open a separate issue for that one.
Thanks!

Looking at the code that error message it's here:
https://github.com/iamazeem/proto-convert/blob/77ed15facc343191e9a3a0e80f9d5581c1e65a6d/bin/proto-convert#L182
btw, has a typo there, it's "self-contained", not "self-contianed" it should say. Plus it could say "Try" instead of "Use" if that is just a suggestion for cases where the importing failed for someother reason (can't it show some msg for that reason?) and the tool does support imports

@birbilis
Good catch, thanks!
The message was mainly added for the recursive import cases.
For the reason part, the ERROR_INFO is being printed also.
https://github.com/iamazeem/proto-convert/blob/77ed15facc343191e9a3a0e80f9d5581c1e65a6d/bin/proto-convert#L182
I'm thinking about changing it only to:
LoadError: #{$ERROR_INFO}

For recursive cases, the error returned by protoc will be printed.
What do you think?

@birbilis
Regarding your comment about --proto_path, please see:
https://github.com/iamazeem/proto-convert/blob/77ed15facc343191e9a3a0e80f9d5581c1e65a6d/bin/proto-convert#L42-L45
Please test it on your side with relative and absolute paths.
Let me know if you observe any discrepancies.


fzf_color_opts does not support hex colours

-- This works just fine
fzf_color_opts = {
    fg = { 'fg', 'Normal' },
},

-- However this does not
fzf_color_opts = {
    fg = { 'fg', '#707880' },
},

the request looks a good feature! thanks

This requirement is done in #614 .


Better AutoRun - continue running/sprinting after jumping

Very useful mod, thank you for sharing! Before you released it, I had some janky AutoHotkey script to do autosprint and keep sprinting after strafing, but it didn't work very well. So you definitely saved me some trouble, thanks!
Would it be possible to continue sprinting after jumping? When running around, I occasionally need to jump an obstacle and would like to keep running or sprinting, without having to press Q again. That would be a nice little QOL improvement.
Thanks for consideration.

Yes, that sounds reasonable. Shouldn't be too difficult to do, either.

Included the changes in version 1.0.3. Glad you enjoyed the mod, thanks for the suggestion!

I tried it out, and it works like a charm. Thanks a bunch!


Fix for extraction of the NMState result for manager.get_state

The manager.get_state() method currently fails with error "Wrong response type".
Looking at the DBUS API, the type for the "state" method return is "u" (uint32):
https://developer.gnome.org/NetworkManager/stable/gdbus-org.freedesktop.NetworkManager.html
I modified the From<> and manager invocation, and it seems to have fixed the error in my application.

Thanks for the PR! I am able to reproduce locally as well.

@spiderkeys I created a new PR for this so that I can easily merge it due to our VersionBot requirements: #114

Closing this one in favor of the other.


Livejoin: Audio is lost if meeting is disconected

If the meeting disconects for some seconds and reconnects, the recorder looses the audio and does not reconnect as a listener. This happened in a 3 hour session on which at 1:30 the server disconected for some seconds, and the last half of the meeting there is only video without sound.

This is expected. You are sending a bot user. It can't understand what's happening there. It'll just record screen & initially click to join as listen only mood. It don't know in which situation what need to do. You can read this section: https://github.com/jibon57/bbb-recorder#how-it-will-work


Fixing spacing around chaise buttons

This PR will address the issue
Summary of changes:
The aim of this PR change is to fix the inconsistency in button spacing issue.
With this PR we have center aligned the buttons using display: 'inline-flex'.
Also for view details action button a height is given and padding is removed to fix the alignment
The deployed version is here

@RFSH I have addressed all comments. Please have a look.
here
I tested on chrome and firefox in different zooming levels

@PoornimaVKrishnan Please be more attentive. I'm assuming you were only focused on the create/copy/edit/delete button. But the "Export" and "Permalink" buttons in recordset now show up in two lines.
Also did you look at my comment? I asked you to try to fix the alignment of the action buttons but you only reverted the changes that you did. Like I said try 3px padding and see if that is better than what we have or not. In our current version the action buttons are a few pixel lower than they should be.

@PoornimaVKrishnan Please be more attentive. I'm assuming you were only focused on the create/copy/edit/delete button. But the "Export" and "Permalink" buttons in recordset now show up in two lines.
Also did you look at my comment? I asked you to try to fix the alignment of the action buttons but you only reverted the changes that you did. Like I said try 3px padding and see if that is better than what we have or not. In our current version the action buttons are a few pixel lower than they should be.

I fixed the issue. Added comments. 3px looked better

@PoornimaVKrishnan  The main change in this PR is addition of display: inline-flex to buttons. We should not override this rule to try to solve other issues. Because of that change, the "Export" and "Permalink" button labels in recordset were misaligned. So I pushed a fix for this. To fix this we just needed to add display: inline-block to the .export-menu.dropdown.
 This was more obvious on the facet and foreign key popups. So I also pushed some changes related to that.
@jrchudy To me this PR is ready to be merged. Can you please review it again?

After discussing this with @hongsudt, we decided to reduce the size of action buttons. This was done because in the single select mode the button was taking up the whole available height and everything was very close to each other.
While doing this change I also fixed the display of checkboxes on Firefox. Firefox doesn't like the way we were attaching :before and :after to input, so I had to move it to label instead. Now all browsers will display the same checkbox.


I've prevented students from spamming multiple times with the same error, but I'm not sure there's a good automated way to decide if a student's report is in good faith or not.  So I guess we deal with it, and lament that the world isn't a better place.


Support MongoDB 3.4 for Kontena Master

We would like to run a Kontena Master using a replicated cluster provided via a managed solution. Most managed solutions are dropping support for MongoDB 2.x, though. For example, MongoDB Atlas only offers the choice of 3.2 or 3.4.
Please update the Kontena Master to support current versions of MongoDB and use the newest stable by default.

For >= 3.2 MongoDB support we need to upgrade Mongoid, see #1290.

Fixed in #2257, will be released with 1.3.0


What type of update are you requesting?
A new version of an existing package
Current Package Identifier
spyshelter.spyshelter
Package Version
<IP_ADDRESS>0
Please describe the changes you would like to see
There is a new release available  <IP_ADDRESS>3 as can be confirmed at https://www.spyshelter.com/ under the download button.
https://www.spyshelter.com/files/SpyShelterSetup.exe

Thank you.


dev guide: mention that example component enhancements are lacking tests

The example code we give for component enhancements don't have tests. We should mention that in the dev guide and ask the reader to implement tests as well.
Alternatively, we can add the missing tests to those PRs.

Currently the rest of the component enhancements seem to be updated. Only one left is this:
https://github.com/se-edu/addressbook-level4/blob/master/docs/DeveloperGuide.adoc#storage-component


text-input focussed field, doesn't replace text with new input

Describe the bug
First off, I love this library to the moon. Best datepicker I've come across in a long time.
I'm not 100% sure this is a bug or intended, however I haven't found a way to work around it either.
When focussing a text-input enabled datepicker the text is highlighted.
Normally in an input field you would expect a keystroke would override the selected text.
In this case it opens the datepicker and removes the highlight.
While the datepicker is very good for general usage, superusers doesn't require the picker and often just want to write the date tabbing through a form.
To Reproduce
Steps to reproduce the behavior:
I made stackblitz with a minimal setup. Can be edited here: Stackblitz

Go to https://vue-6ukup4.stackblitz.io/
Hit tab on the page, field should now be in focus and text highlighted
Start writing the date.
Unexpected behavior is the modal opening and first keystroke is ignored while the rest is appended to the date.

Expected behavior
When the text is highlighted and text is entered, the text should replace the selection.
Desktop & mobile (please complete the following information):

Browser: chrome
Library version: 5.4.0

The fix is included in the released v6


Refactor alarm handling

When using the soco.alarms module in tandem with subscription callback events, I found it was difficult to know if the current state of the alarms was accurate. When an alarm is changed and events are sent out (from each speaker), it can take some time for updated alarms to propagate to all speakers. The zone.alarmClock.ListAlarms() call returns a CurrentAlarmListVersion which can be matched against the same one provided in the event payload, but this does not exist in the current implementation.
While adding support for this value, I found it was easier to think of things in terms of a parent storage class (Alarms) which contains all Alarm instances instead of using the weakref class attribute.
This PR adds new functionality while attempting to preserve existing behavior:

Adds a new Alarms storage class to hold all Alarm instances (acts as singleton like SoCo instances)
Parses and stores CurrentAlarmListVersion to allow synchronizing with event callbacks
Throttles unnecessary updates using CurrentAlarmListVersion
Attempts to preserve Alarm instances by updating them in place (legacy behavior)
Adds typing to all methods and attributes

Breaking changes:

Alarm.alarm_id is an int instead of str

Non-breaking changes to existing behavior:

Alarm.remove() returns a bool to indicate success

Coverage decreased (-0.2%) to 52.466% when pulling 6b19b958eab8042318faa147d91232f7364846af on jjlawren:alarms_refactor into 5602d2880af8a3daa9158a2982fca8130d41b13f on SoCo:master.

Breaking changes:

Alarm.alarm_id is an int instead of str

I'm concerned that this will cause widespread breakages, and I'd like to explore ways of retaining backward compatibility. I appreciate that this could make the code suboptimal.

I'm concerned that this will cause widespread breakages, and I'd like to explore ways of retaining backward compatibility. I appreciate that this could make the code suboptimal.

To clarify, the values used remain the same, but they're now cast to int.

I'm concerned that this will cause widespread breakages, and I'd like to explore ways of retaining backward compatibility. I appreciate that this could make the code suboptimal.

To clarify, the values used remain the same, but they're now cast to int.

Yes, I realise that. The type change will break code, though, as you noted. (It would break my code, for example.)

Fair point. I can revert that change after I make all the CI versions happy.

Breaking change has been reverted. I've also removed type hinting for now as using the "old" pre-3.7 hinting is messy and no other code in the library uses it yet.

Breaking change has been reverted. I've also removed type hinting for now as using the "old" pre-3.7 hinting is messy and no other code in the library uses it yet.

Thanks for retaining backward compatibility.
I gave the refactored code a quick test run against the alarms actions in my code (SoCo-CLI), and it all worked fine.

I've refactored the Home Assistant integration against the current state of this PR and I haven't run into any issues. I'm now able to coordinate and throttle alarm updates based on event callbacks across multiple speakers.

LGTM -- thanks.


Can we make path join() func accepts a variable number of arguments?

I want to make my Justfile run on both Windows and macOS, for the following lines:
_termix := env_var('TERMIX_DIR')
# Used to handle the path separator on different OS
_s := if os_family() == "windows" { '\' } else { '/' }
AGE_SCRIPT :=  join(_termix, 'git' + _s + 'age.nu')

If join() func accepts a variable number of arguments, that would be much easier:
_termix := env_var('TERMIX_DIR')
AGE_SCRIPT :=  join(_termix, 'git', 'age.nu')

The actual path of my script is like /Users/abc/termix-nu/git/age.nu on mac and D:\work\termix-nu\git\age.nu for Windows.
Thanks.

I wouldn't be opposed to this. join should probably accept two or more arguments.
For now, you can do join(a, join(b, join(c))) but that's pretty gross.


Design for data check recommended actions

Start with a meeting setup between Raymond Peck, Chris Mancuso, Karsten Chu, Dylan Sherry and yourself!

Design doc is here: https://alteryx.quip.com/3yqBAaGDDjWk/Data-Checks-Recommended-Actions
Closing!


Makefile: support LIBC and DESTDIR options.

Fixes:#39

Hi, @liangzhou121, this PR will resolve your issue :)


ConfigurationManager.AppSettings is getting null on Azure Pipelines on 16.3-preview

Steps to reproduce
Since Azure Pipelines has been updated to 16.3 preview all our test runs are now failing.
I'm not 100% sure what the full root cause is yet, but this appears to be related to reading a value from an app.config via System.Configuration.ConfigurationManager.AppSettings (this is on .NET Framework).
Previously our code was reading the value OK, now it is null.
Expected behavior
Tests using System.Configuration.ConfigurationManager.AppSettings get a value.
Actual behavior
null is returned.
Diagnostic logs
Tests run against:

"C:\Program Files (x86)\Microsoft Visual Studio\2019\Enterprise\Common7\IDE\Extensions\TestPlatform\vstest.console.exe" @d:\a_temp\7369bca1-e130-11e9-8628-396c9d60e171.txt
Microsoft (R) Test Execution Command Line Tool Version 16.2.0

were passing.

Environment
Running on Azure Pipelines.

We are still using VS2017 for developers at the moment. But we have managed to re-create by installing VS2019 (16.3) on one machine, so this might be a bigger problem than just console / Pipelines

There is also this in the Failing build that doesn't exist in the Passing one:
d:\a\_tasks\VSTest_ef087383-ee5e-42c7-9a53-ab56c98420f9\2.157.4\Modules\DTAExecutionHost.exe --inputFile d:\a\_temp\input_96a10d90-e365-11e9-9046-67324e9a5d6c.json
======================================================
##########################################################################
DtaExecutionHost version 17.157.29325.1.
Starting TestExecution Model...
Updated Run Settings:
<RunSettings>
  <RunConfiguration>
    <MaxCpuCount>0</MaxCpuCount>
    <BatchSize>1000</BatchSize>
    <ResultsDirectory>d:\a\_temp\TestResults</ResultsDirectory>
  </RunConfiguration>
</RunSettings>

Hi,
I can still reproduce it, to workaround the issue we added the following task:
  - task: VisualStudioTestPlatformInstaller@1
    inputs:
      packageFeedSelector: 'nugetOrg'
      versionSelector: 'specificVersion'
      testPlatformVersion: '16.2.0'

And added vsTestVersion: 'toolsInstaller' to our VSTest task.
If I revert this change, the problem re-occurs.
Our project is large with over 800 projects.
The test project is .NET Framework 4.5.2, using MSTest TestAdaptor 1.3.2 for running the tests.
The code that gets the AppConfig value is a .NET 4.0 DLL.
The dependencies go something like this:
Test -> net40/45 Project -> net40 Reading App Config -> net40 executable with app.config

We also build the assemblies into sub folders so following the same projects as above
BuildOutput\Tests -> BuildOutput\Client -> BuildOutput\Common -> BuildOutput\Client

If I get time I'll see if I can create a sample project that mimics this and build it.

Sorry, I'm not able to repro, every time I run these tests they fail, even with the same workaround: https://dev.azure.com/taeguk/Public/_git/TestFailures
I'm not sure what it is in a production code that makes these pass when using 16.2.0 and fail on newer versions.

I tried this project on my local box with VS with test platform version 16.2.0 and the test is still failing as you said. I will need a repro project to see what exactly is causing the failure.

Please feel free to reopen the issue with a repro.


Redstone updates forcing crops on elemental soil to grow very fast

Hello, not sure if you consider this a bug, but I do.
Describe the bug:
Crops on elemental soil (all kinds, but tested with aqueous) are forced to grow by redstone updates. With a redstone update I mean the changing of a redstone signal.
To Reproduce:
First construct a standard farm with elemental soil with crops planted on the soil. Then put redstone next to the crops. Construct a redstone clock (any variant will do) to rapidly change the redstone signal. Hook the clock up to the redstone next to the crops, you will see the crops grow much faster than they should (best is to put some other crops a bit away from the redstone to compare with). This behaviour does not happen when using normal minecraft soil (tilled with hoe), which is why I am reporting it here, it seems to occur due to the elemental soil.
Environment Versions
I am using Roots-1.12.2-<IP_ADDRESS>.jar
Mystic Mods Versions

MysticalLib: mysticallib-1.12.2-1.13.0.jar
MysticalWorld: mysticalworld-1.12.2-1.11.0.jar

Other Versions:

Forge: <IP_ADDRESS>59
Minecraft: 1.12.2
Modpack (if available): No other mods installed.

Please let me know if you need any further clarification. Would love to be of help.

So, this definitely seems like a bug. I'm guessing it's the additional growth ticks that get added to crops; the updateTick method is probably being forcibly called via the redstone change, so I'm guessing that's the issue there.
	@Override
	public void updateTick(World world, BlockPos pos, IBlockState state, Random rand) {
		super.updateTick(world, pos, state, rand);

		if (world.isAirBlock(pos.up())) {
			return;
		}

		IBlockState upState = world.getBlockState(pos.up());
		Block upBlock = upState.getBlock();
		doHarvest(world, pos.up(), world.getBlockState(pos), upState);

		if (!(upBlock instanceof IGrowable))
			return;

		// TODO: Who knows if this value is any good
		if (rand.nextInt(5) == 0) {
			upBlock.randomTick(world, pos.up(), upState, rand);
		}
	}
	```

Probably that entire section for randomly ticking the `upBlock` needs to be removed or adjusted in some way, or it should be moved to a separate method that isn't going to be accelerated by redstone ticks. Don't currently have a functional build environment for 1.12.2, but I'm working on it at the minute hopefully.

Thank you for the fast response! I am glad you agree it is a bug and are working on a fix, thank you very much for your time :)

This bug was around long before I ever got here, so I assumed it was intended.  Guess not.
Elemental soil was already in desperate need of a rework, as it has a ton of things happening every single tick.  I took a stab at optimizing it a long time ago, but there was just too much spaghetti there so I abandoned the project.  I guess another look is about due if both of us work on it.


GH1796: Obsolete DotNet aliases

Fixes #1796

@mholo65 your changes have been merged, thanks for your contribution 👍

Warning


Arcade that sends you to OpenDream Paradise station

About the PR

Adds an arcade that, when used, sends you to paradise station. It forces you to sleep for 60 seconds while adding a little text upon examination that indicates that you are in paradise station.
Why / Balance

Suggestion from #contributors.
I do have indirect permission from ike709, moderator and contributor for open dream, to connect people to open dream servers. But I have no idea if that is enough. I asked around on the Open Dream discord server and will update, till then I will keep this PR as a draft.
See this discord message.
Update: According to amylizzle, the open dream servers are hosted by PJB, which means this is allowed per Hub rules. Hooray! Also because amylizzle said that OpenDream is administered largely by wixoa, I also asked him.
Technical details

Honestly, this is the first time I ever coded UI, and I am so sure that I produced shitcode. But I hope it's good enough for an april fools joke.
Media

https://github.com/space-wizards/space-station-14/assets/63975668/90fc36b4-40f8-4d45-8096-bb2878e1b7f2

[x] I have added screenshots/videos to this PR showcasing its changes ingame, or this PR does not require an ingame showcase

Changelog
i dont even think the CL works for april fools

This adds it to the random arcade spawner, meaning it will spawn in with the other arcades. At least on most maps.

This adds it to the random arcade spawner, meaning it will spawn in with the other arcades. At least on most maps.

I immediately noticed my mistake after posting, oops.
Anyways, you could probably still serve to weigh this more then the others by putting them in a rare prototype group. You'd probably want more people to play it anyways since it's only for april fools.

Anyways, you could probably still serve to weigh this more then the others by putting them in a rare prototype group. You'd probably want more people to play it anyways since it's only for april fools.

How? I am unsure how they work.

Add

Anyways, you could probably still serve to weigh this more then the others by putting them in a rare prototype group. You'd probably want more people to play it anyways since it's only for april fools.

How? I am unsure how they work.

To my understanding, from looking at the maint loot spawner: you need to add a rarePrototypes field and set rareChance under it. You should probably set chance to make up the difference of whatever you have rarePrototypes set to, otherwise I think there's a chance the spawner will just spawn nothing.
You'd be looking at something like this, to my understanding:
    prototypes:
      - ParadiseArcade
      chance: 0.50
    rarePrototypes:
      - BlockGameArcade
      - SpaceVillainArcadeFilled
      rareChance: 0.50

Alright, thanks. Will implement soon.

the open dream servers are hosted by PJB, which means this is allowed per Hub rules

Still wanna see permission from her and a quick glance that this is fine.
@PJB3005 require your review.

Imagine being able to cross-message between servers

Permission from PJB aquired.


Added Animation

Added Test animation on MOMOMIA on header. Here the text spins 360 degree. Also removed extra nav bar from the main body.  Refer the two demo images for reference.
Hope you will like it and merge my pull request.

Good job @AbhishekKumarRay


Flow temperature adjustment  accordingly surplus energy from PV

Automating heating and climate control with  flow temperature adjustment , thermostat automation ,  Smart Thermostat with PID controller  ,adjust the temperature accordingly surplus energy from PV ,Automating climate control with Solar PV output   Estimated Power Production - Now (in Watt) ,Estimated Power Production - Next Hour (in Watt) .  The resulting optimized flow temperatures mean increased efficiencies, potential savings,
I need some help to write this scrip or to find a similar script  ( I don't have programming experience )
1.connection  to  heat pump is via local tuya TCPIP   ( number.set4temp ; switch.pdc_on_off ;sensor.21senzor_return;sensor.22senzor_flow)
2.connection to PV inverter  Huawei FusionSolar inverter or via local modbus TCPIP , Power Sensor  (sensor.power_sensor_active_power ;sensor.power_sensor_realtime_power
3.connection with a room thermometer  is via local tuya  (sensor.t1_3_real_temp  second room/  sensor.t2_realtemp )
4.connection  external temperature and humidity is done via API(sensor.wupws_temp,sensor.wupws_humidity)
5.Solar production forecast   (sensor.power_production_now ;sensor.energy_current_hour;sensor.energy_next_hour
Some connection are lost-est over some issue with signal , is necessary a control loop and some extra feedback info .

Hi. You might be wrong here? This custom implementation is only maintained for EUROTRONIC Zigbee SPZB0001 thermostats. It is based on the official HA Generic Thermostat. It is not meant to be used as a generic thermostat for other purposes. For this please contact the Home Assistant community directly.
By the way: this custom implementation is not maintained anymore. And I don't have any knowledge about your request, sorry.


[WIP] Modern SASS

Hi, this is a work in progress, direct manual port to the modern SASS compiler
It employs stdlib sass:math instead of mathsass external library and works with latest dart-sass implementation
Things to do:

[x] Add more tests
[x] Investigate new features of dart-sass which may be used here
[x] Port the rest functions under utils module

I decided to change the old API a bit. This pull request still needs to be polished, but the main work is done and I want to hear your thoughts on this @apexskier @boronine
Also, I wish to add color constrast module later and it will need some agreement on API too (as well as ideas)

Hi @Ptico thank you for modernizing this repo! The pull request looks good to me. Are you able to get Travis tests to pass again? Feel free to drop support for older versions of Sass/etc. that don't pass. When we publish your work we will refer users of old Sass versions to hsluv-sass v1.0.0.
For contrast I did some experiments here:
https://github.com/hsluv/hsluv/blob/master/math/contrast.mac
https://github.com/hsluv/hsluv/blob/master/haxe/src/hsluv/Contrast.hx

Great, I will update Travis and Readme this week and will let you know when everything ready

Just a quick status update: I've restructured the project for better readability and clarity and added a snapshot testing, but some strange glitch doesn't allow me to push to my own repo. GitHub support is aware of situation and promised to take a look soon

Hi @boronine , I've regained an ability to push. There is a basic color contrast function added, but because of SASS inaccuracy, sometimes it fails online check by a small fraction (like 0.02 : 1). While it's still relevant and factually accessible, it will not allow to pass some linters and automated checkers.
There is a few ways to solve this:

Introduce an additional values for $ratio argument, like 'AA' and 'AAA' which will have a small margin around 4.51/1 and 7.01/1 and leave an ability to specify exact ratio. It looks most promising for me as it gains a control over result.
Add a margin to the formula itself
Leave it as is and cover this nuance in a documentation, suggesting to increase a ratio in order to pass automated checking

Also, I did try to use it today and I'm not very satisfied with a result. The problem is that the current public API is good for casual usage, like translate HSLuv to RGB in place. But it doesn't take an advantage from HSLuv features.
I'm thinking about a little bit more complicated, but flexible design, where we treat HSLuv as a first-class sitizen, like https://sass-lang.com/documentation/modules/color do. Something like:
@use 'hsluv' as hsluv;

$main-color: hsluv.create(150deg, 100%, 35%);

$secondary-color: hsluv.scale($main-color, $hue: 10%);
$background-color: hsluv.contrast-lighten($main-color);

.example {
  color: hsluv.hsluv($main-color);
  background-color: hsluv.hsluv($secondary-color);
}

Hi @Ptico I might be dull but I'm not understanding your comment about the SASS inaccuracy. Can you provide an example? I see your code passing tests on Travis so I'm not sure where you're expecting a test failure.
I like your idea of adding a full color API to hsluv-sass, although I think there will be quite a few nuances to work through. I wouldn't be able to contribute much time to this, but if you're willing to do it, we could start by deciding on a short list of functions that are worthy of implementing.
Having said that, I think it would be best if we split this work into two parts: 1. modernize the repo, 2. make additions to the API. Seems that 1 is basically finished, so when you're ready we should merge that into master and release hsluv-sass v2.0. Then we can work on new APIs for v2.1. We just need to make sure v2.1 doesn't break v2.0 imports, i.e. is backwards compatible.

@boronine you are right, let's ship a compatible 2.0 first. I pulled new features and will create a separate PR for them

@Ptico just released v2! Added you to the LICENSE file. Thank you! Could you open a new issue/PR to discuss contrast and/or new API?

Thanks for all the work here! I'm obviously not as involved with this project these days and didn't get around to looking until now, but it looks pretty slick - I appreciate the more modern test library and sass tooling.


Need updated information.

It simply says proceed or cancel.  It doesn't explain validation check, etc. that needs to be checked before proceeding.

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

ID: e438bc46-868d-3795-62d8-2ee14d690b4c
Version Independent ID: 8f22521d-2695-94a1-2871-4f5ae694996a
Content: Publish Topology
Content Source: Skype/SfbServer/help-topics/help-topobld/publish-topology.md
Product: skype-for-business-itpro
GitHub Login: @kenwith
Microsoft Alias: kenwith

Thank you for this feedback and for contributing to the wellness of the content! This is a good idea and we have added it into our backlog for triage and prioritization.

@tseward Is investigating this.

Hi @nsshommo
Tasks description on the Publish Topology page of the Deployment Wizard try to be self-explanatory and as the lists of things that need to be ready before proceeding it's so diverse that would be difficult to explain all on the same "Publish Topology" document, but you can find what needs to be done for every one of this bullet points in docs, for example "A file share has been created..." it's here https://docs.microsoft.com/en-us/skypeforbusiness/deploy/install/create-a-file-share and so on.
Hope this info has helped you.
Thanks


Option to change string of total score modifier in difficulty menu

Id like to change it to something else like "overall difficulty value" since "score" doesnt matter for rt

Provided in 0.6.6.


fix: add missing dependency iproute2 to vpnc-scripts

Motivation for this change
When running vpnc-scripts in a restricted environment (systemd service), I noticed that a dependency is missing:
/nix/store/bq0mhi257vrxl8vhf0srd2kpdxndpiza-vpnc-scripts-unstable-2021-03-21/bin/.vpnc-script-wrapped: line 807: ip: command not found

Notice: This triggers a rebuild of quite a few packages, so maybe the 'staging' branch is a better place for this.
Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[ ] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
21.11 Release Notes (or backporting 21.05 Relase notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 130941 at 45e376fa run on x86_64-linux 1

1 package marked as broken and skipped:

googleearth-pro

469 packages skipped due to time constraints:

adapta-gtk-theme
alfis
almanah
apostrophe
appgate-sdp
arc-theme
areca
arx-libertatis
asciidoc-full
asciidoc-full-with-plugins
...

11 packages built successfully:

connman
connman-gtk
connman-ncurses
connmanFull
connman_dmenu
networkmanager-openconnect (gnome.networkmanager-openconnect)
networkmanager
openconnect (openconnect_gnutls)
openconnect_head
openconnect_openssl
vpnc-scripts

1 suggestion:

warning: missing-phase-hooks
installPhase should probably contain runHook preInstall and runHook postInstall.
Near pkgs/tools/networking/vpnc-scripts/default.nix:17:3:
   |
17 |   installPhase = ''
   |   ^


Missing XACML doc issues

Is your suggestion related to a missing or misleading document? Please describe.
Missing docs: https://is.docs.wso2.com/en/5.11.0/learn/configuring-access-control-policy-for-a-service-provider/#

This issue is being closed due to extended inactivity. Please feel free to reopen it if further attention is needed. Thank you for helping us keep the issue list relevant and focused!


Rules not populating from Tenant

Working through a few different tenants, ones that I know for certain have rules uploaded to them.
The Rules folder consistently displays, what looks to be, 4 example rules (After AD AccountCreate, After AD AccountModify, After Operation Rule Template for Web services, Before Operation Rule Template for web services), but does not show any of the other rules populated on the tenant.

Hi,
The extension is leveraging the beta API for connector rules: https://developer.sailpoint.com/apis/beta/#operation/getConnectorRuleList. Using this API allows to create, update and delete rules.
It does not include the "Cloud Rule", rules that execute only in the cloud.
Can you check with a REST client and the API that nothing else is returned? Can you confirm that you are using cloud rules or rules pushed by PS?


[Request] Support flatPortal method

It seems to be already supported. Check the example datapack's command portals.


Pi 8496 legacy product detail page opens

Coverage increased (+0.1%) to 33.371% when pulling 5c4f8be920e8f3dca5b48c15763c4e1d9a4be954 on PI-8496-legacy-product-detail-page-opens into 0b7925c095f8f59619b483ddfb40f3f155e092f4 on master.


Request for new term [inborn error of surfactant metabolism]

Label
inborn error of surfactant metabolism
Synonyms
hereditary pulmonary alveolar proteinosis
Synonym type
related
Definition
Lung disease caused by the presence of a genetic variant negatively affecting pulmonary surfactant associated proteins or processes.
Parent term
MONDO:0015925
Children term(s)
MONDO:0016323, MONDO:0012582, MONDO:0018603, MONDO:0009929,  MONDO:0014206, MONDO:0024465, MONDO:0010424, MONDO:0013712
ORCID Identifier
https://orcid.org/0009-0007-1636-9645
Website URL
No response
Additional comments
I am submitting the request to introduce this term on behalf of the ClinGen Interstitial Lung Disease Gene Curation Expert Panel. Our panel believes that the term "inborn error of surfactant metabolism" should replace "hereditary pulmonary alveolar proteinosis" as it is a more accurate parent term for the hereditary surfactant related diseases that reside under it in the disease ontology tree.
Attached is an image to clarify our request to replace the term "hereditary pulmonary alveolar proteinosis" with "inborn error of surfactant metabolism" and move this parent term and the gene related syndromes beneath it to the parent term "interstitial lung disease". Please disregard the part of this image showing "interstitial lung disease due to SFTPB dysfunction" as a term; we will submit a separate request to introduce that term.

Hi @peastc - Nice to virtually meet you, I'm Nicole, I'm a curator with Mondo, I don't believe we've worked together before.
Thanks for your request. if I understand correctly, you are requesting that 'hereditary pulmonary alveolar proteinosis' be relabeled to 'inborn error of surfactant metabolism'. In cases like these, you can request a term to be relabled (there is a template for that). However, I'd like to point you to our disease naming conventions in Mondo.
In Mondo, we aim to use the OMIM labels as the primary labels for our terms. We can add your term as an exact synonym and add a 'ClinGen label' tag to it. We hope that your ClinGen technical team can enable the terms tagged with 'ClinGen label' to display in your curation interface, so you will see that as the primary label in your system, but the primary label in Mondo will remain as 'hereditary pulmonary alveolar proteinosis'.
Our other Mondo curators and technical developers @sabrinatoro and @twhetzel can advise you too, if you have further questions, or you are welcome to join one of our developers calls to learn more.
Thanks for contributing to Mondo.

Hi @nicolevasilevsky! Nice to e-meet you! I am fairly new to the ClinGen team and I am excited to continue to work with you.
Thank you for your guidance here and for adding this synonym. I will discuss the introduction of new terms as exact synonyms with our GCEP and UNC ClinGen team.


Added boa projects and app,infra pipeline projects

Added example projects for Bank of Anthos and pipeline projects.

@bharathkkb  I got past the license error but not sure how I can get past this error since the single_project module won't exist in this repo.
Error: Unreadable module directory
Unable to evaluate directory symlink: lstat ../../modules: no such file or
directory

@bharathkkb I got past the lint test. Let me know if your ok with add of modules directory and folder.tf & variable.tf files. I know there not needed, but are required to pass the lint tests

@googlebot I fixed it.

@googlebot I fixed it.

@googlebot I fixed it.

@googlebot I fixed it.

@googlebot I consent


suppressMethod should take var args of method names

From<EMAIL_ADDRESS>on September 26, 2008 21:02:50
We should add suppressMethod(Class<?> clazz, String...methodNamesToSuppress);
Original issue: http://code.google.com/p/powermock/issues/detail?id=17

From<EMAIL_ADDRESS>on October 13, 2008 09:32:19
Added support to easy suppress several methods in the same class
Status: Fixed


Only one document parsed in reponse from server

As of #41 in the 1.0 branch, Message::read_reply only parses one document from the server. This will be fixed in the future when we get around to implementing cursors.
Because of this, making multiple queries in a row may not get the correct results (again, just in the 1.0 branch) if the first query returns more than one document, so if you're testing this out and get a weird issue when making multiple queries, try limiting the first query (or any query besides the last) to just one document.

Fixed in e01326fd97c232f24faa179aa981d75ec679244


Get rid of CC formatting tags in package descriptions

Self-descriptive

https://pkg.go.dev/regexp#Regexp.ReplaceAllString
ReplaceAllString returns a copy of src, [...]

Also the regular expression is re-compiled every time.


Add code coverage back in

It looks like code-coverage is no longer being reported, and we should add this back in.

Closed by #265


Completion and linting not working

Issue Report Checklist

[* ] Searched the issues page for similar reports
[ *] Read the relevant sections of the Spyder Troubleshooting Guide and followed its advice
[ *] Reproduced the issue after updating with conda update spyder (or pip, if not using Anaconda)
[ *] Could not reproduce inside jupyter qtconsole (if console-related)
[ *] Tried basic troubleshooting (if a bug/error)

[* ] Restarted Spyder
[* ] Reset preferences with spyder --reset
[* ] Reinstalled the latest version of Anaconda
[* ] Tried the other applicable steps from the Troubleshooting Guide

[ *] Completed the Problem Description, Steps to Reproduce and Version sections below

Problem Description
Completion and linting in the editor for python files will not work in the current session or stopped working
To try to fix this please verify that your firewall or antivirus allows python processes to open ports or restart spyder
What steps reproduce the problem?

open spyder

What is the expected output? What do you see instead?
Completion is not working
Paste Traceback/Error Below (if applicable)

PASTE TRACEBACK HERE
<!--- Copy from error dialog or View > Panes > Internal Console --->
Completion and linting in the editor for python files will not work in the current session or stopped working
To try to fix this please verify that your firewall or antivirus allows python processes to open ports or restart spyder

Versions

Spyder version: 5.0.5
Python version: 3.8.10 64 bit
Qt version: 5.9.7
PyQt version: 5.9.2
Operating System name/version: Windows 10 64 bit

Dependencies

PASTE DEPENDENCIES HERE

# Mandatory:
atomicwrites >=1.2.0          :  1.4.0 (OK)
chardet >=2.0.0               :  4.0.0 (OK)
cloudpickle >=0.5.0           :  1.6.0 (OK)
cookiecutter >=1.6.0          :  1.7.2 (OK)
diff_match_patch >=20181111   :  20200713 (OK)
intervaltree >=3.0.2          :  3.1.0 (OK)
IPython >=7.6.0               :  7.22.0 (OK)
jedi =0.17.2                  :  0.17.2 (OK)
jsonschema >=3.2.0            :  3.2.0 (OK)
keyring >=17.0.0              :  23.0.1 (OK)
nbconvert >=4.0               :  6.1.0 (OK)
numpydoc >=0.6.0              :  1.1.0 (OK)
paramiko >=2.4.0              :  2.7.2 (OK)
parso =0.7.0                  :  0.7.0 (OK)
pexpect >=4.4.0               :  4.8.0 (OK)
pickleshare >=0.4             :  0.7.5 (OK)
psutil >=5.3                  :  5.8.0 (OK)
pygments >=2.0                :  2.9.0 (OK)
pylint >=1.0                  :  2.9.5 (OK)
pyls >=0.36.2;<1.0.0          :  0.36.2 (OK)
pyls_black >=0.4.6            :  0.4.6 (OK)
pyls_spyder >=0.3.2;<0.4.0    :  0.3.2 (OK)
qdarkstyle =3.0.2             :  3.0.2 (OK)
qstylizer >=0.1.10            :  0.1.10 (OK)
qtawesome >=1.0.2             :  1.0.2 (OK)
qtconsole >=5.1.0             :  5.1.0 (OK)
qtpy >=1.5.0                  :  1.9.0 (OK)
rtree >=0.9.7                 :  0.9.7 (OK)
setuptools >=39.0.0           :  57.0.0 (OK)
sphinx >=0.6.6                :  4.0.2 (OK)
spyder_kernels >=2.0.4;<2.1.0 :  2.0.5 (OK)
textdistance >=4.2.0          :  4.2.1 (OK)
three_merge >=0.1.1           :  0.1.1 (OK)
watchdog >=0.10.3             :  2.1.3 (OK)
zmq >=17                      :  20.0.0 (OK)

# Optional:
cython >=0.21                 :  0.29.24 (OK)
matplotlib >=2.0.0            :  3.3.4 (OK)
numpy >=1.7                   :  1.20.3 (OK)
pandas >=1.1.1                :  1.3.0 (OK)
scipy >=0.17.0                :  1.6.2 (OK)
sympy >=0.7.3                 :  1.8 (OK)

Hi @pachu-um thank you for the feedback, as the message says there is most probably a problem with your firewall or antivirus preventing the process used for Spyder to give completions to start. Did you try to check already your firewall/antivirus to allow python process as the message says? A simple way to test if the problem is actually related to your firewall or antivirus is to disable them momentarily and launch again Spyder. If completions work then you need to add the exceptions rules for either your firewall, your antivirus or both.
Also, you could check some documentation we have regarding completion or help not working and a way to try to restart the service from a running Spyder instance: https://docs.spyder-ide.org/current/troubleshooting/common-illnesses.html#completion-help-not-working
Let us know if something of the above helps :)

Dear Daniel Althviz More,
thanks for the reply... I did set the firewall access..i don't have any antivirus installed.
But i noticed that the LSP Python is down... If i try to restart, it kind of hangs... it doesnt get restarted.
When i restart spyder, it try to restart the LSP and fails and show the message "LSP Python down", post this, i get the completion and linting error box
As per the link shared above i try to update the python-language-server, but i got the message saying , all packages required are already installed.
thanks and regards

Thank you for the info @pachu-um maybe you need to check Windows Security/Defender then (the built-in element that includes an 'antivirus' on Windows 10). Also, did you install Spyder with the Anaconda distribution and a normal user or a user with administrator rights?
Let us know!

Dear Daniel,
I have attached the firewall picture... the firewall is turned off
And regarding anaconda installation, i guess i was logged in as admin when i installed. But is there a way to check it ?

Thanks for the screenshot @pachu-um however besides the firewall configuration that you showed there is another view related with the security and the built-in antivirus that is included with Windows 10: https://docs.microsoft.com/en-us/microsoft-365/security/defender-endpoint/microsoft-defender-security-center-antivirus?view=o365-worldwide could you check there is there is more settings you need to put in place?
Regarding the admin rights then maybe you could try to launch Spyder with admin privileges but I think in the long term will be better to check the proper configuration to allow the completion process (firewall and antivirus exceptions).
Let us know :)

Dear Daniel,
I took some time, discussed with our IT team and got it checked, there seems to be no blocking in firewalls or antivirus.
Not sure how to proceed now... any other logs that i could share ?
please do let me know ,
thanks and regards

@pachu-um maybe you could try to run in debug mode and send us the logs.
For that please run from an Anaconda Prompt (with the env were you have Spyder activated) Spyder with the debug-info and debug-output flags.
Something like:

To see the log in the console:
    spyder --debug-info verbose

To see the log in the console and save it in a spyder-debug.log file in the current directory were the console is:
    spyder --debug-info verbose --debug-output file

That should let you see a log and maybe a traceback will appear regarding the completion process not being able to start

Dear Daniel,
I found something in another post
https://github.com/spyder-ide/spyder/issues/12904
i ran a conda list and found that some of the packages are installed using pip
So i created a new environment and install spyder using conda list . Now the completion and linting error does not come and am also able to see the completion...
thanks for your support...
Best regards

Thanks for the feedback @pachu-um ! Indeed mixing packages (pip installed vs conda installed) can cause things to not work properly. Glad is working for you now, will close this then :)


Upgrade from 2.0.0 to 2.0.1 alters schema.db file format

Upgrading schema_plus and running any pending migrations will result in a changed db/schema.rb file:
#> bundle update schema_plus
#> rake db:migrate

The changed db/schema.rb file will use the old Ruby format where
t.string   "description",         default: "", null: false

will become
t.string   "description",         :default=>"", :null=>false

Yes this change is a byproduct of other changes.  I didn't consider it a regression or breaking change since it's functionally equivalent.
TL;DR: the simplest cleanest way that I know to print a ruby hash that contains only primitives, in a form that can be read back in as valid ruby, is simply to use the builtin Hash#to_s -- which prints using the old Ruby format.   Could write a custom dumper in some form but that seems like extra code (with more opportunities for bugs) for little functional benefit.

This seems reasonable to me too.
@nesrual, is this breaking something specific for you? If the behavior of schema_plus changed as a result of the functionally equivalent hash changing, that's a bug. Otherwise this isn't really any different than if schema_plus decided to write:
t.string "foo"

@nerusal I should add that I was disappointed by this change too, since the modern style makes for clear looking output IMHO.   But I couldn't justify to myself muddying the code (and taking the time) just to make the output arguably prettier.
But of course as @fj commented, if the change in output format really does break something then of course it'd need to be fixed.

Besides the output change in the schema.rb file all works fine - so not really a "bug" in that sense. However it would be nice to keep the format of the file in the "proper" format :+1:

On further reflection, I think hash-rocket may actually be preferred. Unusual but theoretically legal column names (e.g. "!!!X&&&") might not also be legal Ruby symbol names (e.g. you can't write !!!X&&&: false), so you would have to quote everything.
But if you quote everything, then there is an ambiguity between these three choices:

{"!foo": 1}: key is the symbol :"!foo"
{:"!foo" => 1}: key is the symbol :"!foo"
{"!foo" => 1}: key is the string "!foo"

So I think not using hashrockets as the canonical representation would actually be a bug.

BTW, if you want to keep your migrations "clean", one thing I've done is perform an empty migration, commit the change, and then continue with my subsequent migrations. That isolates the commit with the schema_plus syntax changes so that the rest of your commits are clean.

FWIW, 👎

@fj re needing hash-rocket for non-symbol keys -- I don't think that's a huge issue here, since the hash is used for column options, which are defined by reasonable humans to have reasonable names.  Conceivably some nested values for some options could be arbitrary hashes though, I suppose.  But still, a pretty output formatter could use symbol-colon form whenever the key is a symbol, and revert to hash-rocket only when necessary.
@braindev FWIW I'm willing to accept a PR that prettifies the output (with sufficient tests of course!), just don't have the time to do it myself right now.   or perhaps following SOC, maybe somebody could write or find a simple gem that will do this, and schema_plus_core could use it.


Lack of dependencies installation

When I run python gradio.py
There are more dependencies needed than the requirements.txt

I forgot to add somedependencies to the requirements.txt, I've updated it and it should work.
I've tested it locally and it seems to work fine.
Could you confirm that it works for you?


Unsupported PrimOp: mkWeakNoFinalizer# ,  Unsupported PrimOp: deRefWeak#

When you install elerea as a dependency to a project you get warned that these two PrimOps are unspported : mkWeakNoFinalizer# and deRefWeak# . indeed when you run an application that uses elerea you also get the error  Uncaught Unsupported PrimOp: mkWeakNoFinalizer#
example application that triggers the crash:
import Data.IORef
import Control.Monad
import FRP.Elerea.Simple

main = do
  smp <- start $ do
    ref <- execute $ newIORef 0
    effectful $ do
      x <- readIORef ref
      putStrLn "Count: " ++ show x
      writeIORef ref $! x+1
      return ()
  replicateM_ 5 smp

I'll check if this fixes my code today.

Assuming this to be fixed: the primops are there and there are no further reports of problems.