ReadOnlyAttribute creates property readonly instead of readOnly

Hi, I'm pretty new to using json shema, but from what I can see in the json hyperschema spec is that the property should be named readOnly and not readonly. From what I can tell from CreateJsonSerializerContractResolver(SchemaType schemaType), I can probably use the SchemaType.Swagger2 to achieve the expected result, but that will also change the behavior of RequiredAttribute, which is not desired.
Might just be a typo or does the spec perhaps say that it should be all lower letters in some case?

I think the default for JSON Schema is x-readOnly, no? And only for Swagger/OpenAPI where it is defined we rename it.
https://github.com/RicoSuter/NJsonSchema/blob/master/src/NJsonSchema/JsonSchemaProperty.cs#L78
Is it now also in the standard for JSON Schema?

Well I'm very new to this but with some googling I ended up on a thread that said that it was in spec 3 but removed in spec 4, but it was actually not removed, just moved to jsonschema-hyperschema.
Just went to the github for hyperschema and found some references for it here: https://github.com/json-schema-org/json-schema-spec/blob/master/jsonschema-hyperschema.xml
I'm using react-jsonschema-form to render data based on the schema, and it does not work for readonly but it does for readOnly.

Based on this:
https://json-schema.org/draft/2019-09/json-schema-validation.html#rfc.section.9.4
it should be readOnly, not readonly.
I'm also having problems generating UI forms from the schema because of casing.

Readonly handling on class level is currently missing. The casing needs to be checked in the static serializer class where the casing is configured per schema type.


Rename Mask class and add origin arg to plot methods

According to discussion in https://github.com/astropy/photutils/issues/684

Added origin arg to plot methods (which is present in photutils)
Added to_region and plot methods to BoundingBox
Rename Mask to RegionMask.

Now, RegionMask are ApertureMask are same both API and code wise.
Geometry functions and BoundingBox are also same in both packages.

@sushobhana - Please rebase against upstream master to resolve the merge conflicts.

@keflavich Added an example and rebased it. Ready to merge??

Thanks @keflavich !


Fix formatting on "MongoDB Storage Plugin" page

https://drill.apache.org/docs/mongodb-storage-plugin/#configuring-mongodb block has invalid formatting

Hi, Is it possible to add the mongo docs link about Connection String URI Format  for this. thanks

Hi, Is it possible to add the mongo docs link about Connection String URI Format  for this. thanks

Hi, Is it possible to add the mongo docs link about Connection String URI Format for this. thanks

ok, the only thing I will add the reference to mongo 4.2 page, since Drill has 3.12.7 Mongo Java driver and looks like it supports Mongo 4.2 version.

Hi, Is it possible to add the mongo docs link about Connection String URI Format for this. thanks

ok, the only thing I will add the reference to mongo 4.2 page, since Drill has 3.12.7 Mongo Java driver and looks like it supports Mongo 4.2 version.

Hi, Is it possible to add the mongo docs link about Connection String URI Format for this. thanks

ok, the only thing I will add the reference to mongo 4.2 page, since Drill has 3.12.7 Mongo Java driver and looks like it supports Mongo 4.2 version.

Looks like from Docs Drill support 3.0 version officially. Need to check the compatibility with newer Mongo versions before update the doc.
Looks like we should support 3.6 version:
https://issues.apache.org/jira/browse/DRILL-7094
But we need to check and update to latest Mongo version in separate ticket. So added link to 3.0 version for now.

Hi, Is it possible to add the mongo docs link about Connection String URI Format for this. thanks

ok, the only thing I will add the reference to mongo 4.2 page, since Drill has 3.12.7 Mongo Java driver and looks like it supports Mongo 4.2 version.

Looks like from Docs Drill support 3.0 version officially. Need to check the compatibility with newer Mongo versions before update the doc.
Looks like we should support 3.6 version:
https://issues.apache.org/jira/browse/DRILL-7094
But we need to check and update to latest Mongo version in separate ticket. So added link to 3.0 version for now.


Logout session is not working...

when i tried logout and pressed back button, i still have have access to previous session, it seems that session is not deleted...
Regards
Rama.

@movibe do you have time to check out this issue? :) If not, i'll take a look at it!

Yes @beeman , I will send new fixes and a new module chat with socket.io for you to analyze

Awesome @movibe , really looking forward to that!  Could you send it in two separate branches?

Yes! @beeman :-)

@gowram could you check if #43 fixed this?

Thanks guys it working fine right now..
thanks for your quick fix.


background-image with gradient

Because the current background-image is wrapping the value provided as url:
https://github.com/jxnblk/styled-system/blob/9add1edd4d5855aecc2a1dabb7aa6bdda67e0bbb/src/styles.js#L320
it is not possible combine it with a gradient, like:
backgroundImage='linear-gradient(rgba(0, 0, 255, 0.5), rgba(255, 255, 0, 0.5)), url("/static/img/hero/3.jpeg")'>

Thanks! Yeah, I've thought about this a little bit, but maybe with #232 it'd be good to remove that default value transform. I've also thought that it could be helpful to have a gradient style helper specifically for gradients

With v3, backgroundImage no longer adds the url syntax


Initial pass of autocomplete; removal of detritus

Implements simple auto-complete system while search input is being typed/pasted in.

Phrase search support only
Ranks matches in the title 10 points, url 5, keywords 3, description 1, and multiplies points x # of matches, showing the top three scoring topics
Highlights matches in yellow
Use arrow keys to navigate to matching topics. If no topic preview is selected (or "See all" is selected) then hitting return runs a full search.

CC @joaofnfernandes @sanscontext @thaJeztah @mstanleyjones @londoncalling @docker/design
See the deploy preview for a demo -- type any term in the search box on the top-right:
https://deploy-preview-723--docker-docs.netlify.com

Deploy preview ready!
Built with commit ed538b4c02d6463d65fd7a62d293337620e567f5
https://deploy-preview-723--docker-docs.netlify.com

Alright, I'm pulling this in - everyone I've shown it to, including Hans and Rafael, seem to like it, I've shared it in Slack, I've shared it in the team meeting. We can always iterate. But we need this pretty bad - users are running wayyyy too many searches.


Code | Alphabetic Pattern 140

@aryashah2k can you assign me alphabet pattern from 140 -165.

sure, all the best!


feature/salesforce-marketing-cloud

Adding a new Provider & Client for Salesforce Marketing Cloud (SFMC).
Following similar pattern to existing Salesforce Sales/Services Cloud provider by utilising existing API User's credentials (has to be manually created in SFMC) to rotate its own secret. Zeep package added to assist with constructing and making SOAP calls.
Pre-requisite prior to use:

API User setup in SFMC
API User provided with permissions (Admin -> API Access)

Ps. Has been a while since I used python. Let me know if anything's butchered real bad 😅

Looks pretty good @mmsarom! Just one typo which is not a blocker, and just calling out you've neglected to add any doco for the new provider.
I'll approve/merge this now, maybe once you have it working in your environment you can circle back and add the docs?
Thanks for your contribution!


GIVbacks - use on-chain data to calculate GIVbacks

@mohammadranjbarz creating things issue as per your comment in the dev call! we don't want users to be able to "game the system" by using just the APIs to determine donation amounts and therefore GIVback allocations.
Want to use on-chain data for this.
Keep us posed here if you have any updates.

Shal I close this? @mohammadranjbarz


Question - remapping default movement

Now ive edited vimrc's before but I couldn't get this to work.
i wanted to remap hjkl in normal mode to match hand placement for touch typing, i.e.:
j = h
k = j
l = k
; = l
I tried using nmap, and even tried using map
what I noticed is that the keys overlap so if I say j = h now, then j will send me left as h used to do. great.
but when i remap k to be j or to send me "done" as j used to do, then it copies j's functionality and will also send me left.
one avenue that almost worked was to map the keys like
j = g1h
k = gj
l = gk
; = g1l
but at that point the ; key doesn't have the intended effect of sending me 1 character to the right as lt has been remapped to mean the operation of the l key.
my question is, do you know of a way within the confines of this remplecated vim environment to remap the hjkl keys to being jkl; for the sake of touch typing?

I believe that map ; <Right> should fix the last issue you are describing.

I did try that, and it's sending me upwards for some reason
https://user-images.githubusercontent.com/29872822/103373834-df771e00-4a8a-11eb-89eb-bb2fcbdab094.mov

Strange, it does work for me.
I assume this problem occurs regardless of whether you have the Vimrc plugin enabled or not. Does it recreate in the CodeMirror Vim demo? If so, I suggest opening an issue in the CodeMirror Github repo, since they're most likely to be able to help.


Error updating to v3.2.3

I get the following output when upgrading to the latest version as of writing.
Updating dependencies (including require-dev)
  - Removing fenos/notifynder (3.1.6)
  - Installing fenos/notifynder (3.2.3)
    Loading from cache

Writing lock file
Generating autoload files
> Illuminate\Foundation\ComposerScripts::postUpdate
> php artisan optimize

  [ErrorException]
  Invalid argument supplied for foreach()

Ok, this is an error @fenos also had. But I wasn't able to reproduce it.
I will publish a fix that should fix this problem. @computerfr33k can you tell me in which file, class, method, line or something like this the error occured?

PR #114 is merged - should solve your @computerfr33k problem!?

When i/it runs artisan optimize. Not sure if there is a debug option for artisan optimize.
Now it tells me this:
  [ErrorException]                               array_merge(): Argument #2 is not an array

@computerfr33k: Ok, before I do some more blind-fixes: can you pls tell me which Laravel-Version you are using and add a full stacktrace for this exception? If there is no in the console it is in the logs.

I am using laravel 5.2, using homestead at the moment which ships with php 7. I wonder if php 7 is causing the issue as well.
http://slexy.org/view/s2sPCeDuP4

Ok, was another place - can you pls checkout the new fix: dev-issue-113 it is untagged so pls use the dev branch.

Seems to be working now.

Great - now I'm mobile but I will Tag it if I'm home.

PR is merged and new tag is published 3.2.5 - Issue is solved and closed!?


feat: allow hourly aggregation for other date ranges

Describe the feature or enhancement
Currently hourly stats seems to only be supported for the "last 24hrs" range. It would be great to look into hourly data for +1 week ranges.

Are you looking to set the aggregation range within the app, or just an update to the API to allow it? We set minimum units of aggregation per date range to avoid resource intensive queries like hourly by year, etc.

We set minimum units of aggregation per date range to avoid resource intensive queries like hourly by year

That seems totally fair and reasonable. Although hourly by week or sometimes even month still seems to make sense without breaking the bank.
I'd be happy to open a PR for this.

Sorry for delayed response. I can see the hour option being available up to 7 days (maybe 30?) and the day option being available up to 1 year. Don't feel a need to minutes / weeks for this first iteration. Feel free to put in a PR. I'll mark as enhancement so it doesn't become stale.

@franciscao633 suggesting one implementation here: #3013
The idea is that there's a new icon right beside the arrows on the DateFilter component. That would be an "engine" icon, that would open a modal, allowing the user to set the timeUnit.
Because there's this (valid) concern about long running queries, we could add a logic that shows a warning if the user selects a non-recommended unit based on the date range.
Suggestion this specific UX since it would allow for other secondary settings (eg. chart type) as well, and wouldn't clutter the current chart view.

@franciscao633 had a change to take a look at this? happy to continue this work, it's a feature we would very much value in prod

@ccrvlh Is the PR ready for review? It's still marked WIP.

It is a WIP. Would be great to get validation on the approach / UI before polishing it for review.

@mikecao just added a couple of screenshots.


Please add a quick start to the Readme

Ideally I'd like a Rails Quick Start and a separate Rack Quick Start. @remi says there is specific support for Sinatra built in to this gem and calling that out would be helpful too.

What's the Sinatra support?

The google-cloud-error_reporting gem under stackdriver gem has code to capture and report exceptions from Sinatra even if Sinatra error handling mechanism is enabled.
As far as I remember, this is currently the only Sinatra specific support we have. @dazuma, am I missing any other Sinatra features?

@dazuma Do you want to take this issue?


unknown option -nostatic

Running into this error when running make under /QAT_Engine.
Right after it tries to run perl /openssl/util/mkerr.pl

Hi @GCM707,
The error you are encountering is caused when you build against the OpenSSL master branch.
It was introduced by the following Openssl commit:
make error tables const and separate header file
https://github.com/openssl/openssl/commit/52df25cf2e656146cb3b206d8220124f0417d03f
The QAT Engine is written to build against the OpenSSL 1.1.0 branch, We specifically test against the OpenSSL 1.1.0f release.
Unfortunately the commit above in Master branch breaks backwards compatibility.
We plan to add a new configure option very shortly which will allow the QAT Engine to build against Master branch again although we would recommend you stick to building against the stable OpenSSL 1.1.0 release unless there is a feature you need in Master branch.
Hope that is helpful,
Steve.

Thanks for the info. I’ve been trying to get this working in the latest beta version of vyos. Looking forward to your next update, I will see what else I can do in the mean time.

Hi @GCM707,
I've just posted the latest QAT Engine release v0.5.26 that contains the fix for the nostatic bug you are seeing.
The specific commit that fixes it is:
Makefile Changes to accommodate latest mkerr.pl in OpenSSL
https://github.com/01org/QAT_Engine/commit/17a1808cdfa02afa6eed5d35c2027c1522a24203
Note: you'll need to run configure with the option:
--enable-qat_for_openssl_master
which will then configure the Makefile to use the required different options when calling mkerr.pl
If that solves your problem then please close this issue, any issues then let me know.
Steve.

it gets further. stops at this message:
In file included from e_qat.c:80:0:
e_qat.h:53:18: fatal error: cpa.h: No such file or directory
include "cpa.h"
              ^

compilation terminated.
Unsure if that comes with something i should have already installed, im a bit new to this.

Hi @GCM707,
Apologies for the slow reply.
The cpa.h header file is one of the QuickAssist API files. It is included with the QAT Driver source code. If you follow the installation instructions in the README.md file included with the QAT Engine it contains pointers on installing the QAT Driver and example QAT Engine configure options to point the QAT Engine build to where the QAT Driver source is so that it can find the Quickassist API header files. The configure option you need to use is:
--with-qat_dir=/path/to/qat_driver
Ensure that the option is pointing to the correct folder level within the QAT Driver. For instance if you have installed the QAT Driver to /QAT and are using an Intel® Communications Chipset 8925 to 8955 Series device then you would use the following setting:
 --with-qat_dir=/QAT/QAT1.6
Kind Regards,
Steve.

We've dealt with this issue, feel free to close this issue.


High Resource Utilization

It would appear when a significant number of actions are displayed in history, high CPU usage is seen. On the order of 50-60% of an i7 mobile CPU. This should be easy to recreate by browsing to st2-build's history page.

Can someone see if this can be reproduced on, say, a Mac? If it's isolated to Linux then I'm fine with it bing ignored. I'll try on other browsers too.

This spikes most likely related to browser trying to highlight big chunks of code. We're fighting this problem one way or the other since the first days, but have only managed to make it not to block the browser itself.
On my i7 mac, cpu utilization rarely spikes higher than 30% of a single core in both chrome and firefox. The UI still seem sluggish, but st2-build doesn't run the latest version of st2web either.


Does firechat works in react native application?

import firechat from 'firechat';
the line above fails with SyntaxError

Interested in the above question!


fitToCoordinates not working with edgePadding

I have an issue when setPadding to move Google logo to other position in the MapView by use method fitToCoordinates in onLayout callback. But, It's not working.
I have found the cause of this issue in the native method:
https://imgur.com/AKOIQXE

Could you please review and fix soon?

Getting the same issue. Please update if it is fixed.

Confirming this issue. Passing edgePadding to fitToCoordinates on Android doesn't have any effect.

Same problem, edgePadding only works on iOS

It works for Android but for some reason it calculates paddings in raw pixel units. You can use PixelRatio.getPixelSizeForLayoutSize to convert units to raw pixels. For example if you want to have padding that is size half of height you can use it like this:
edgePadding = {
...
bottom: (Platform.OS === 'android') ? PixelRatio.getPixelSizeForLayoutSize(height / 2) : (height / 2)
};

@kenobi91 could you open an issue that shows exactly the problem  (repro and solution) and link it here? 🤗

Ofc i will do it later np. :)


migrate to drizzle notification routes

✨ Codu Pull Request 💻

Pull Request details
I've refactored the following notification.ts routes:

delete
deleteAll
getCount

The return value from delete and deleteAll routes is not used on the front end. Despite this, I followed the same return pattern as the Prisma queries I was replacing. ie

delete : The id of the deleted record
deleteAll : The count of deleted records.

The return value of getCount is used on the front end.
I will include the get route  in a seperate PR because it is more complicated.
Any Breaking changes

Hopefully not, but I would still like to test each route

Summary by CodeRabbit

Refactor

Updated database access method to improve performance.
Enhanced notification fetching and deleting logic for better efficiency.
Modified return values from delete operations to ensure accuracy.
Improved notification count operation for more reliable results.

Nice 🚀 we are getting there with the migration!
Great to see some of them are mapping 1:1 like this

Nice 🚀 we are getting there with the migration!
Great to see some of them are mapping 1:1 like this

Yes, getting there and I'm learning a lot in the process.
I will leave the hard ones to you 😀


Towards the e-book

http://sdwfrost.github.io/mevr/
for inspiration:
https://github.com/sdwfrost/mevr
See the book.json
See also the Glossary.md
Summary.md for the book structure
https://github.com/jonschlinkert/markdown-toc for the TOC (clickable)
Use

Nearly there. Things to do:


pop-location after execution was cancelled

this PR changes behaviour of Inoke-DbcCheck in an event of test cancellation (ctrl+c).
So far we were left in the custom or default repo folder. Now, with this change, even if the test is cancelled the pop-location is executed, and the location is exactly as when the execution started. As a bonus a warning message appears to notify the user that the execution was cancelled.

All the tests have been run, no errors reported.

YAY! you found a solution. Thank you! I'll let @SQLDBAWithABeard merge but I love this and will use the idea in the future too!

Well, if that's OK I'd like to try merging, @SQLDBAWithABeard please review, but if that's OK I'd like to try the merge, finally.


enabled assetic integration of LiipThemeBundle

actually use the fix @dantleech made in https://github.com/liip/LiipThemeBundle/pull/118
informations:

q
a

Fixed tickets
none

Related PRs
none

BC breaks
none

Documentation PR
none

@c00n84 Does it ring a bell? :sweat_smile:
@wachterjohannes Could you please review that?


Prediction explanations cannot be written to json when target is of type bool

Repro
from evalml.model_understanding import explain_predictions_best_worst
from evalml.pipelines import BinaryClassificationPipeline
from evalml.demos import load_fraud
import json
import pytest

pipeline = BinaryClassificationPipeline(["Text Featurization Component", "DateTime Featurization Component",
                                         "One Hot Encoder", "CatBoost Classifier"])
X, y = load_fraud(1000)
pipeline.fit(X, y)

report = explain_predictions_best_worst(pipeline, X, y, output_format="dict")

with pytest.raises(TypeError, match="keys must be str, int, float, bool or None, not numpy.bool_"):
    json.dumps(report)

The issue here is that the column names for the predicted probabilities dataframe when the target is boolean changed from base python bools to numpy bool_ dtype.
On the left is main and on the left is 0.24.2:

This is the line that is causing this change.
I think there are two fixes here:

Update predict_proba to not have np.bool_ types in the column names
Apply _make_json_serializable to the keys of the probabilities dictionary in the explain predictions module. Probably here

Root cause: can't use default python JSON serializer to serialize numpy bool type. So we need to sanitize the types before trying to serialize, as @freddyaboulton wrote in description.


Add comment about Sink and OutputStream

I'm having the same issue described in #525  regarding Sink and OutputStream lifetimes. Since there is no relevant description in the documentation, I added the relevant description.

Thanks a lot @senlizishi! It seems rustfmt does not like it :sweat_smile: we can easily fix that though. If you run cargo fmt and then push again it should be fixed.


Training checkpoint stopped creating decode.output

Hi,
After passing some checkpoints successfully, sockeye stops calculating BLEU score during checkpoints but it keeps logging perplexity and saves new params.NUM files.
Last successful Checkpoint

[2020-05-14:17:46:36:INFO:sockeye.training:fit] Checkpoint [88] Updates=440000 Epoch=16 Samples=280841176 Time-cost=2919.597 Updates/sec=1.713
[2020-05-14:17:46:36:INFO:sockeye.training:fit] Checkpoint [88] Train-perplexity=2.603048
[2020-05-14:17:46:38:INFO:sockeye.training:fit] Checkpoint [88] Validation-perplexity=1.350713
[2020-05-14:17:46:38:INFO:sockeye.training:collect_results] Decoder-87 finished: {'bleu-val': 0.7527509099234959, 'chrf-val': 0.8385353050482615, 'rouge1-val': 0.7619336112171186, 'rouge2-val': 0.6748560114329887, 'rougel-val': 0.7457107039294799, 'length-ratio-mse': 1.0308577832975823, 'avg-sec-per-sent-val': 0.08381631994247436, 'decode-walltime-val': 41.90815997123718}
[2020-05-14:17:46:38:INFO:sockeye.training:start_decoder] Starting process: Decoder-88
[2020-05-14:17:46:40:INFO:sockeye.training:fit] Validation-perplexity improved to 1.350713 (delta=0.000705).
[2020-05-14:17:46:41:INFO:sockeye.utils:log_gpu_memory_usage] GPU 1: 24486/32480 MB (75.39%) GPU 2: 24218/32480 MB (74.56%)
[2020-05-14:17:47:28:INFO:sockeye.training:call] Epoch[16] Batch [440050]   Speed: 634.01 samples/sec 21833.64 tokens/sec 0.94 updates/sec   perplexity=2.607029

Next Checkpoint

[2020-05-14:18:35:35:INFO:sockeye.training:fit] Checkpoint [89] Updates=445000 Epoch=16 Samples=284026656 Time-cost=2915.589 Updates/sec=1.715
[2020-05-14:18:35:35:INFO:sockeye.training:fit] Checkpoint [89] Train-perplexity=2.601719
[2020-05-14:18:35:37:INFO:sockeye.training:fit] Checkpoint [89] Validation-perplexity=1.352920
[2020-05-14:18:35:37:INFO:sockeye.training:start_decoder] Starting process: Decoder-89
[2020-05-14:18:35:39:INFO:sockeye.training:fit] Validation-perplexity has not improved for 1 checkpoints, best so far: 1.350713
[2020-05-14:18:35:41:INFO:sockeye.utils:log_gpu_memory_usage] GPU 1: 24486/32480 MB (75.39%) GPU 2: 24218/32480 MB (74.56%)
[2020-05-14:18:36:28:INFO:sockeye.training:call] Epoch[16] Batch [445050]   Speed: 568.60 samples/sec 21589.55 tokens/sec 0.94 updates/sec   perplexity=2.603905

After this point, validation set is not evaluated with BLEU and no decode.output.NUM created. There is neither a warning log.
Here is the running environment:
Python 3.6.10 :: Anaconda, Inc.
sockeye==1.18.106
mxboard==0.1.0
mxnet-cu101==1.6.0 # CUDA 10.1 is installed in device
mxnet-mkl==1.4.1
I have another device with CUDA 10.0 and there is no such a problem there. So you may not easily reproduce the issue. I hope you can help.

I am calling the sockeye through Jupyter notebook using sockeye.arguments.add_train_cli_args
When I directly use a shell script, everything works fine. I think it is not a sockeye related problem but a Jupyter environment problem. So you can close the issue but it would be even better if you could add an INFO or WARNING level log when a checkpoint Decoder can't be finished as a nice to have feature.

Hi @ogunoz
In this version of Sockeye, the decoder process to translate the validation data is actually spawned in a subprocess and its results are collected when the main training process reaches the next checkpoint. It could be that this setup does not play well with running side a Jupyter notebook. Are there restrictions to subprocesses/logging of output from subprocesses inside notebooks?

Newer versions of Sockeye (>2.0) will not decode in a subprocess and therefore this issue should not be present once you update.

Hi @fhieber
Thank you so much for your response. After our one week investigations, I realized that it is not related with Jupyter notebook as well. I suspect that it is related with the shared folder structure in the cloud machine. I will check the new Sockeye! If there is no reason for me to use the older one, I will migrate to 2.0. Thank you also for this information.

Just another note, when the decoder cannot translate and generate BLEU score, if validation perplexity is improved, it keeps updating the params.best. For this reason, when I try to go on from interrupted point, I can't actually use the "real" params.best. For your information :)

I believe with the new version >= 2.0, it is going to be fixed. So I am closing the issue now.

I have one more question to specify "Sockeye 2" version. I couldn't find any released package for that. Should I directly clone it from your sockeye2 branch?

Yes, you can clone the sockeye_2 branch to use it. We haven’t released to pypi yet.


Tweaks to WMS support

Builds on #799

Thanks for the pull request @kring!

:heavy_check_mark: Signed CLA found.
:grey_question: CHANGES.md was not updated.

If this change updates the public API in any way, please add a bullet point to CHANGES.md.

Reviewers, don't forget to make sure that:

[ ] Cesium for Unreal Samples works.


Inflector::camelize unexpected behaviour after recent upgrade

What steps will reproduce the problem?
A bit of an edge case, but I was passing strings to the camelize function that may or may not already been in camelized format.  This changed with the latest update here 629a33fd0e2f469eef29de7d30f0f23c66e5791f. To reproduce:
echo \yii\helpers\Inflector::camelize('HelloWorld');
What is the expected result?
HelloWorld
What do you get instead?
Helloworld
Additional info
See commit 629a33fd0e2f469eef29de7d30f0f23c66e5791f

I think that current behavior is correct. Uppercase letters should indicate word separators and everything else should be lowercased. If you pass string without word separators, only first letter should be uppercased.

It makes perfect sense, but it's still a BC braking change.
Should we add it to an UPGRADE.md (👍), fix it to keep BC (😕 ) or move the entire change to 2.1 (❤ )

By reverting this you're introducing another BC break :/.
Also: https://xkcd.com/1172/ ;).

By reverting this you're introducing another BC break :/.

No. It wasn't released yet.

I'd either make it compatible or move to 2.1.


Update icon assets

Need to update the icon and launch screen assets in the web manifest.
Assets are located here:
https://app.zeplin.io/project/5e7e4dc925feed6b33480e5d/screen/5e87b75ed23311bc803cca15

@ke

@kbakeann the asset labeled as 50x50 is actually 72x72 in zeplin

Fixed!


feat(security): security section for website

@robzienert do you know if the<EMAIL_ADDRESS>email has been setup?

@dorbin @robzienert can this be merged please?


modal maximum call stack

When I open react-native-datepicker in modal component show this error: "maximum call stack size exceeded". I prove with manually method but not resolved

Well I used a similar plugin as modal: react-native-popup-dialog


Need more info on how InteractivityService works with selection

While building new IVisual, I'm struggling to understand exactly how InteractivityService works with selection, especially on how the selection of one IVisual would drive the update of one or more IVisuals.  The playground project uses only the DefaultVisualHostService and it doesn't show how IVisualHostServices.onSelect() is being handled.
I have been studying how Slicer works but I can only find WebInteractivityService.sendSelectToHost() being called but I can't figure out how the host service is supposed to handle the select event and drive other related IVisuals to be updated.
Are there E2E example of how one IVisual's selection interacts with other IVisuals?  Any more details doc on this topic?

Use Selection Manager instead of Interactivity Service. There may be compatibility issues with Interactivity Service. Many of the newer sample visuals showcase how the Selection Manager can be used.

Hi @deldersveld,
I am also having questions about how selection works in the framework, and I am using SelectionManager. My confusion is mainly on "onSelect" of DefaultVisualHostServices and how the selection of one IVisual can affect another.
I tried to figure out how DefaultVisualHostServices handles the selection by walking through the code of "AreaRangeChart" in the playground app. However, at the very end, SelectionManager will call "onSelect" of DefaultVisualHostServices, which seems to be an empty function:
DefaultVisualHostServices.prototype.onSelect = function () { };
According to the "Hello IVisual" page, it seems like it is the host that takes care of notifying other IVisuals when a selection is made to one of them.
So could you please shed some light on both DefaultVisualHostServices and how the selection of one IVisual can affect another.
Thank you.

@UncleDolan, answered it here #139


[Bug] insert时数值超出类型表示范围仍导入成功

Search before asking

[X] I had searched in the issues and found no similar issues.

Version
2.1.5
What's Wrong?
使用新优化器，insert时数值超出类型表示范围仍导入成功
这个问题与新优化器有关，之前的优化器在stmt.analyze阶段就抛异常了，新优化器没考虑这个异常
What You Expected?
insert时数值超出类型表示范围导入失败并抛出异常
How to Reproduce?
insert导入的值超出int范围：
CREATE TABLE int_tb ( k1 INT NULL, v1 INT SUM NULL, v2 INT MAX NULL, v3 INT MIN NULL, v4 INT REPLACE NULL ) AGGREGATE KEY(k1) DISTRIBUTED BY HASH(k1) BUCKETS 15 PROPERTIES ("replication_allocation" = "tag.location.default: 1");

insert into int_tb select 0, -2147483649, 0, 0, 0;

insert导入的值超出char范围：
CREATE TABLE stream_insert_datatype_db.varchar_tb ( k1 VARCHAR(1) NULL, v1 VARCHAR(1) REPLACE NULL ) AGGREGATE KEY(k1) DISTRIBUTED BY HASH(k1) BUCKETS 15 PROPERTIES ("replication_allocation" = "tag.location.default: 1");

insert into varchar_tb select "a", 100;

Anything Else?
无
Are you willing to submit PR?

[ ] Yes I am willing to submit a PR!

Code of Conduct

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

|   0:VUNION(43)                                                                                |
|      constant exprs:                                                                                |
|          0 | CAST(-2147483649 AS int) | 0 | 0 | 0                                                                    |
|      tuple ids: 0

现在新优化器上会考虑规划一个cast，而cast的实现中，目前是不会检查是否超出范围的。
有在考虑超出范围的时候报错，这里有个demo
https://github.com/apache/doris/pull/40334
mysql [test]>set enable_strict_cast_mode = false;
Query OK, 0 rows affected (0.00 sec)

mysql [test]>select cast(k2 as int) from testdb;
+-----------------+
| cast(k2 as INT) |
+-----------------+
|     -2147483648 |
+-----------------+
1 row in set (0.02 sec)

mysql [test]>set enable_strict_cast_mode = true;
Query OK, 0 rows affected (0.00 sec)

mysql [test]>select cast(k2 as int) from testdb;
ERROR 1105 (HY000): errCode = 2, detailMessage = (<IP_ADDRESS>)[INVALID_ARGUMENT]Cast to type Int32 ,out of range 9.2173984e+18

|   0:VUNION(43)                                                                                |
|      constant exprs:                                                                                |
|          0 | CAST(-2147483649 AS int) | 0 | 0 | 0                                                                    |
|      tuple ids: 0

现在新优化器上会考虑规划一个cast，而cast的实现中，目前是不会检查是否超出范围的。 有在考虑超出范围的时候报错，这里有个demo #40334
mysql [test]>set enable_strict_cast_mode = false;
Query OK, 0 rows affected (0.00 sec)

mysql [test]>select cast(k2 as int) from testdb;
+-----------------+
| cast(k2 as INT) |
+-----------------+
|     -2147483648 |
+-----------------+
1 row in set (0.02 sec)

mysql [test]>set enable_strict_cast_mode = true;
Query OK, 0 rows affected (0.00 sec)

mysql [test]>select cast(k2 as int) from testdb;
ERROR 1105 (HY000): errCode = 2, detailMessage = (<IP_ADDRESS>)[INVALID_ARGUMENT]Cast to type Int32 ,out of range 9.2173984e+18

Which version can support?


Build fails on Apple M1 macs

Expected Behavior
Code should build on M1 (ARM64) macs
Actual Behavior
Build fails with errors on M1 macs (build succeeds on x86 mac)
==> Installing google/verible/verible
==> bazel build --jobs=8 --compilation_mode=opt --copt=-march=native //...
Last 15 lines from /Users/reportaman/Library/Logs/Homebrew/verible/01.bazel:
Analyzing: 1789 targets (68 packages loaded, 605 targets configured)
WARNING: Download from https://mirror.bazel.build/ftp.gnu.org/gnu/m4/m4-1.4.18.tar.xz failed: class com.google.devtools.build.lib.bazel.repository.downloader.UnrecoverableHttpException GET returned 404 Not FoundWARNING: Download from https://mirror.bazel.build/github.com/jmillikin/rules_m4/releases/download/v0.1/m4-gnulib-788db09a9f88abbef73c97e8d7291c40455336d8.tar.xz failed: class com.google.devtools.build.lib.bazel.repository.downloader.UnrecoverableHttpException
GET returned 404 Not FoundWARNING: Download from https://mirror.bazel.build/ftp.gnu.org/gnu/bison/bison-3.3.2.tar.xz failed: class com.google.devtools.build.lib.bazel.repository.downloader.UnrecoverableHttpException GET returned 404 Not FoundWARNING: Download from https://mirror.bazel.build/github.com/jmillikin/rules_bison/releases/download/v0.1/bison-gnulib-788db09a9f88abbef73c97e8d7291c40455336d8.tar.xz failed: class com.google.devtools.build.lib.bazel.repository.downloader.UnrecoverableHttpExce
ption GET returned 404 Not FoundWARNING: Download from https://mirror.bazel.build/github.com/westes/flex/releases/download/v2.6.4/flex-2.6.4.tar.gz failed: class com.google.devtools.build.lib.bazel.repository.downloader.UnrecoverableHttpException GET returned 404 Not FoundAnalyzing: 1789 targets (75 packages loaded, 607 targets configured)
ERROR: /private/tmp/verible-20210524-4463-1xwgue4/.brew_home/_bazel/91a8a79b286e633105eb5e6515121504/external/bazel_tools/tools/cpp/BUILD:469:14: Target '@bazel_tools//tools/cpp:compiler' depends on toolchain '@local_config_cc//:cc-compiler-darwin_arm64', which cannot be found: no such target '@local_config_cc//:cc-compiler-darwin_arm64': target 'cc-compiler-darwin_arm64' not declared in package '' defined by /private/tmp/verible-20210524-4463-1xwgue4/.brew_home/_bazel/91a8a79b286e633105eb5e6515121504/external/local_config_cc/BUILD'
ERROR: While resolving configuration keys for @com_google_protobuf//:protoc: Analysis failed
ERROR: While resolving toolchains for target @com_google_protobuf//:protoc: Analysis failed
ERROR: Analysis of target '//third_party/proto/kythe:storage_proto' failed; build aborted: Analysis failed
INFO: Elapsed time: 16.901s
INFO: 0 processes.
FAILED: Build did NOT complete successfully (85 packages loaded, 607 targets configured)
FAILED: Build did NOT complete successfully (85 packages loaded, 607 targets configured)

Do not report this issue to Homebrew/brew or Homebrew/core!

Steps to Reproduce the Problem
brew tap google/verible
brew install verible

Specifications

Version: macOS BigSur 11.3
Platform: Apple M1 (ARM64) MacMini

Passing on to Johan as he is working on the brew package and might know some of the things still needed in M1

Seems like compiling it directly on machine gives the same error. Link: https://github.com/google/verible/issues/260#issuecomment-848079292.


analyzer: Have exited processes read all data

This commit changes the behavior for analyzer processes so that processes that have successfully exited without reading all the data will continue to consume data from the byte stream insteading of returning an error and putting a stop to the copy goroutine.
Closes #331

tl;dr
This is a functional 👍 for me!
Details
I pointed my Zui at the Brimcap commit from this branch and tested it out with the pcaps I mentioned in #331 comments and saw the improvements expected in both cases.

For the one in https://github.com/brimdata/brimcap/issues/331#issuecomment-1907185603, the error changes from this incorrect one:

to this accurate one:

and that latter error message will go away entirely when the Zeek v6.0.3-based artifact is in use since that has support for this link type.

For the one in https://github.com/brimdata/brimcap/issues/331#issuecomment-1907207180, it goes from showing this incorrect error message:

to a successful import:

i.e., Suricata quietly refused to do anything with this pcap since it doesn't support the link layer protocol, but since it returned an exit code of 0 the Zui user gets only Zeek events. This seems like a fine place to be while we wait to see if the Suricata people ever catch up and address the existing issues.


Best way to lazy load a service used in multiple controllers using ocLazyLoad?

 So that controller file is loaded when its needed.
but I want to know what is the best way to lazy load service file which is being used in multiple controllers?
As loading a single service file in resolves of multiple states will cause lots repetition of code.
Like this I have many common services which are needed in many places and loading all of them in index file will make system loading slow at first.

Perhaps you could bundle the service names array to be returned in a couple functions and then use that in the resolve to reduce the duplication so if a service names changes or whatever you can just change it in the function instead of all the routes?


Any particular instructions to pool multiple gpu's ?

Hi,
As a home user, just to test out some inference and large models, i have 2 gpu's lying around, but in different form factors (one in desktop, one in a laptop). Is there a way for this driver abstraction to use both of these gpu's?

looks like the faq says that its in dev : "Yes, watch this space for imminent developments!"

https://github.com/Juice-Labs/Juice-Labs/wiki/Multi-GPU-Support


Visual Studio Winforms style Editor and Drag and drop gallery planned?

Visual Studio Winforms style Editor and Drag and drop gallery planned?
This is not so much a question of my wants, but I am trying to convince the Developer of AdvancedHMI to port his code from Wibforms to something cross platform. His biggest hang up is no visual editing support with Avalonia.
The user base for the software is mostly non programmers.

That's a very common question, so I went ahead and made an enhancement issue for it: https://github.com/modern-forms/Modern.Forms/issues/18.  As stated there, a visual editor is probably not happening any time soon due to other priorities.
While I do not have experience with it, Avalonia does have a visual editor: https://avaloniaui.net/docs/quickstart/vs-designer that might be worth checking out.

As far as I know, the Avalonia VS-Designer is just a preview tool, it doesn't let you do a complete design visually. :/
Thank I understand. Can actual true visual designing be clarified in the issue. I know it is will be way off.

Sure! I added "drag and drop" to it to clarify.

Thank you!!!


Typo in fl_asr_train example parameter

Bug Description
Not a bug itself but had to open an issue to do the pull request.
There is a small typo in flashlight/flashlight/app/asr/README.md in an example on the usage of fl_asr_train:
The flags to the train binary can be passed in a flags file or as flags on the command line. Example of flags file:

--dadadir=/tmp
--train=train.lst

Should be datadir instead of dadadir.
Reproduction Steps
None
Platform and Hardware
None
Additional Context
None

I'm not signing anything with Meta to fix a typo. I close this.


pkg/kf/commands: removes KfParams.Output

Replaces all usage with Command.OutOrStdout(). KfParams.Output was
redundant.
fixes #60

Cool, does this open us up to using KfParams as an on-disk config format so we can start to do things like target a namespace and save a container registry?

@josephlewis42 It should


HHH-13565

https://hibernate.atlassian.net/browse/HHH-13565

I've polished it a bit more, updated, and been re-benchmarking. Awesome results.
Will now push as soon as the JDK matrix test completes.


Purpose of this settings?

I don't get it. What's the real world goal / application of this project? The README doesn't explain what one would use this for.

Hello,
So, imagine that in your system you have Users, and a Notification system, that notifies the User about several system events. You can use this package to store the user notification preferences.

Specific settings for an entity. Ex. Users model: you want to manage notifications (in app, slack, push notif. etc.),

Come to think of it, this could likely be used to support "meta" fields too... information you may want to have about a record but not necessarily stuff you need to have as columns in your table.


♻ Remove totalSupply. Add checked balance arithmetic.

Description
Removes totalSupply in accordance with EIP-6909 update.
Adds checked arithmetic on balance increases, as they previously relied on a checked addition to totalSupply.
Checklist
Ensure you completed all of the steps below before submitting your pull request:

[x] Ran forge fmt?
[x] Ran forge snapshot?
[x] Ran forge test?

LGTM.

LGTM.


InoSkyTech Pop-up Design

Develop a suitable popup design for the promotion of the InoSkyTech company.

I made InoskyTech and Modest pop-up designs. while making them, I paid attention to where the object should be located and to make a design that fits the user's eye.


Init connections once per session for testing

Currently, when testing  jupysql, there are many tests, which initialize their own connections for every single test case like src/tests/test_magic_plot.py::test_sqlplot at this, even though a connection to the database has already been made in a test earlier
Even though pytest.parametrize parallelize the tests, I think one way it might be improved is by using pytest.fixture in conjunction with the scope argument. Most probable session or module (may be package) level will do(needs more research)
@edublancas

I think the tests you're showing are only using DuckDB. Since DuckDB is a local database, it's OK to open a new connection (we're creating an entirely new database) because that ensures better isolation.
Unless this is causing any issues, we can keep it.


dev: adds php script

Adds php script that formats untranslated tags copied directly from the log into a json format
context: https://discord.com/channels/890222432605057044/1188449993019904120

Lets hope curseforge doesn't flag this is a virus or something lol


Handle GitHub's tricky site loading behavior better

This flow almost always works:

Click on Pull Requests in the right bar
Click on New pull request, go from there

This doesn't:

Click on green shortcut button on the repo home page, middle-ish left
Click on a branch, keep going but script never fires

Instead of relying on Chrome to load us on certain paths, we should instead load on every GitHub page load and then fire relevant parts of the extension when we detect that the URL is a match.

I have this working on my own local refactoring branch. :-)

Are you a wizard


change imagePullPolicy to IfNotPresent

Well in  mainland the network is always a problem.
Sometimes the image is hard to be pulled and have to be imported by docker.
Always imagePullPolicy  makes  some trouble.
So the imagePullPolicy  IfNotPresent is very suitable.

DCO failed, you need to sign off your commit. @alongL


Hook request: Biome.js

I want to implement a hook for the following development tool:
https://biomejs.dev/
https://github.com/NixOS/nixpkgs/tree/7feb69e10527655647fe211ff9872cf14ab56b9a/pkgs/development/tools/biome

Closed in favor of https://github.com/cachix/git-hooks.nix/issues/428.


Can I set 4 processors for 8 channels?

I know the synthesis in the current form does not support this, however, is there a way to run two pass for less processors by setting the registers? (I am comfortable with the registers on the accelerator, so you can explain using them)

Yes, try the "--max-proc" command line option. For example, "--max-proc 32". This is not officially supported, so it's possible you'll run into problems. Let me know if that happens.

If I synthesize 2 models for 32 processors each, it will each have configurations for 32 processors.
Can the two be allocated for processors 0-31 and 32-63?
They would have different row/column count registers and will have settings for themselves.
Problems I can imagine is when the aggregator tries to fetch data from all 64 processors, or when the processors would want run in sync.

Correct, unfortunately this is not supported in the general sense.

Thanks!!
I think I got a somewhat understanding. I just need to process on how the aggregator works.


Added a cors option to allow cross origin requests

This PR adds the cors headers to allow cross origin requests when running broccoli serve.
I needed this for being able to serve fonts.

I would have loved to see this feature :'(


GraphQL classes not compatible with upcoming GraphQL 4 module

The new GraphQL 4 module removed the need for some of the classes, such as QueryCreator. Therefore the installation of this module next to GraphQL 4 will result in fatal errors during dev/build
Maybe the issue could be solved similarly to the asset-admin module, where they moved these classes into a _legacy folder: https://github.com/silverstripe/silverstripe-asset-admin/tree/1/_legacy/GraphQL and prevented class loading like this: https://github.com/silverstripe/silverstripe-asset-admin/blob/1/_legacy/GraphQL/ReadFileUsageQueryCreator.php#L14
I'm unsure about the  ScaffoldingProvider (https://github.com/gorriecoe/silverstripe-link/blob/master/src/models/Link.php#L32) though, as it is no longer available in the latest version.

The upgrade-path for scaffolders is described as follows: https://docs.silverstripe.org/en/4/developer_guides/graphql/upgrading/#scaffoldingproviders-are-now-schemaupdaters
Not sure if that can be accomplished within the same codebase though (eg. creating a module that is compatible to both GraphQL 3 and 4). Maybe @unclecheese knows…

Experienced issues when updating to SS4.10. Reported them in silverstripe-linkfield, see the issue below;
https://github.com/gorriecoe/silverstripe-linkfield/issues/33

Lovely, thanks a lot. I've been using UncleCheese's fork until now, glad this is now merged/fixed. ❤


Enable better diagnostics: Example images

Example images are no longer written during segmentation model training. In the new PL training, this can only happen at rank zero, and means that we don't always get the same images coming in. Switch to doing it for the same subject across more ranks? Or always the first images on rank zero?
Check if store_and_upload_example is of any use, remove if not.
Tests should be updated: end of _test_model_train()
AB#3909

pick a set of subject IDs at random, and have the option to fix a set of subject IDs that you put into the config
for each of those subjects, we want to monitor the performance across training. Write thumbnails every N epochs (every 10 epochs maybe, configurable)
Thumbnails in axial view (horizontal) for every structure we want to predict, centered at the Z position where ground truth has the most pixels.
thumbnails should show a contour for the ground truth segmentation and for the model prediction

Thumbnails are printed for model evaluation, not for training. You can reuse/extend plot_contours_for_all_classes

Ensure that this all works fine for multi-node training. You can use the BasicModel as a testbed, with --num_nodes=2 to run on two nodes


Incorrect window resizing after t.maximizeWindow

Are you requesting a feature or reporting a bug?
Bug
What is the current behavior?
A browser window is not resized to the specified size.
What is the expected behavior?
A browser window is resized to the specified size.
How would you reproduce the current behavior (if this is a bug)?
Use t.resizeWindow or t.resizeWindowToFitDevice after using t.maximizeWindow
Provide the test code and the tested page URL (if applicable)
Specify your

operating system: Windows
testcafe version: 0.11.0-alpha2
node.js version: 6.5.0

The problem is not reproduced with<EMAIL_ADDRESS>fixture `Fixture`;

test('test', async t => {
	await t
	   .maximizeWindow()
	   .resizeWindow(500, 500);
});

The problem is not reproduced with<EMAIL_ADDRESS>fixture `Fixture`;

test('test', async t => {
	await t
	   .maximizeWindow()
	   .resizeWindow(500, 500);
});


jackett: 0.21.2586 -> 0.21.2663

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for jackett is: API Support for your favorite torrent trackers
meta.homepage for jackett is: https://github.com/Jackett/Jackett/
meta.changelog for jackett is: https://github.com/Jackett/Jackett/releases/tag/v0.21.2628
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 0.21.2628 with grep in /nix/store/c7n6xhwhqq4is377i08vy6s52ypbjz5p-jackett-0.21.2628
found 0.21.2628 in filename of file in /nix/store/c7n6xhwhqq4is377i08vy6s52ypbjz5p-jackett-0.21.2628

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

2 total rebuild path(s)

2 package rebuild(s)

First fifty rebuilds by attrpath

jackett

Instructions to test this update (click to expand)

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

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

Or:
nix build github:r-ryantm/nixpkgs/3998d92c1f2cb6d958f34564df84f9c83b4dd6ec#jackett

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

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

1 package built:

jackett

Maintainer pings
cc @edwtjo @nyanloutre @purcell for testing.

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for jackett is: API Support for your favorite torrent trackers
meta.homepage for jackett is: https://github.com/Jackett/Jackett/
meta.changelog for jackett is: https://github.com/Jackett/Jackett/releases/tag/v0.21.2663
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 0.21.2663 with grep in /nix/store/i6c6sasa1cgwz4hr17530cm3brwlqr0d-jackett-0.21.2663
found 0.21.2663 in filename of file in /nix/store/i6c6sasa1cgwz4hr17530cm3brwlqr0d-jackett-0.21.2663

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

2 total rebuild path(s)

2 package rebuild(s)

First fifty rebuilds by attrpath

jackett

Instructions to test this update (click to expand)

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

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

Or:
nix build github:r-ryantm/nixpkgs/6bde9cc262c671a27ac8b494480fd5d6e6aba1e2#jackett

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

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

1 package built:

jackett

Maintainer pings
cc @edwtjo @nyanloutre @purcell for testing.

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for jackett is: API Support for your favorite torrent trackers
meta.homepage for jackett is: https://github.com/Jackett/Jackett/
meta.changelog for jackett is: https://github.com/Jackett/Jackett/releases/tag/v0.21.2831
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 0.21.2831 with grep in /nix/store/jc8k277n12pdrw079j108mwf7p32ryzz-jackett-0.21.2831
found 0.21.2831 in filename of file in /nix/store/jc8k277n12pdrw079j108mwf7p32ryzz-jackett-0.21.2831

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

2 total rebuild path(s)

2 package rebuild(s)

First fifty rebuilds by attrpath

jackett

Instructions to test this update (click to expand)

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

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

Or:
nix build github:r-ryantm/nixpkgs/613920111c048f1f0e27e0b7a69d027453d9fdf5#jackett

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

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

1 package built:

jackett

Maintainer pings
cc @edwtjo @nyanloutre @purcell for testing.

Result of nixpkgs-review pr 312752 run on aarch64-darwin 1

1 package built:

jackett


How to delete 1gb partition with VitaDeploy ?

I created internal memory card (1Gb) on PSP Vita 1000.  (firmware now 3.65 with reset)
I want  back to stock memory.
I think i need install Henkaku,  install VitaDeploy with replace app Near.
What i need do then?

VitaDeploy's create an internal memory card option.
It will launch imcunlock, you can remove the 1gb from there ("Default 1XXX..." option).
Keep in mind that to avoid a memory card slot softbrick, if you dont have a memory card inserted it will reduce the partition size to 0MB instead of removing it.

Does Memory card Sony  will format?

no

Thx. i back to stock
Do you know how  to unlink from account memory card Sony?
My account PSN hacked. Sticker with serial number lost. I can't revover account.

I check hack PSP Vita 1000 with create 1Gb imc. And in settings VitaDeploy choose unlink memory card, but its unlink 1Gb imc.
Now i back to stock.

i need copy files from memory card Sony, but i cant unlock


Fix and test for updating SequenceFields

Current code doesn't support "update" on SequenceField where the underlying python sequence is modified rather than replaced outright.  The result of the update operation is a purge of the Sequence from the db.
I've got a couple of test cases here.  I initially thought the issue was confined to SubDocument lists but determined it was not, so there are two cases, one with IntFields and the other with SubDocuments.
I'm not entirely happy with the fix I'm proposing here, which is to detect when the list fetched from the DB differs from the current list and simply replace the old list with the new one.  I suppose the "perfect" solution would be to $push and $pull the added and removed elements from the list, but only when that is more efficient that simply $setting the new list.  I know for our use case, we aren't really using lists of more than a few dozen elements of fairly small size, so simply blowing away the whole thing isn't a big deal.
The location in which I'm stuffing the old value is probably not the best place as well.  If there's a better place to cache the old value I'd be happy to move it there.

I think the ideal solution would be to augment the list (or dict, etc) with wrapper methods that could record its mutation operations into the object.  I'll pull this change, make sure it passes the tests, and then release it, since it seems better than nothing for now.

0.11 is released with this change

I definitely agree with your ideal solution, and dicts currently suffer from the same issue on update.


Improve integration tests

I made some notes on the integration tests, including adding a script to initialize PGVector's test DB and some environment variables to increase the flexibility of the configuration, such as making it possible to run PHP inside a container spinned up with DB containers.

Hey @f-lombardo ,
Thanks a lot for these improvements! 2 questions otherwise, it's good for me

Thanks @f-lombardo !


akira-unstable: unbreak by removing vala-lint dependency

Description of changes
ZHF #265948
fixes https://hydra.nixos.org/build/241858923
Our vala-lint seems hit by https://github.com/vala-lang/vala-lint/issues/181, (https://hydra.nixos.org/build/241858665)

Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[x] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
23.11 Release Notes (or backporting 23.05 Release 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

[ ] Fits CONTRIBUTING.md.

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

Thanks

I tried reverting that commit and i still get the same error

The commit I bisected to is part of https://github.com/NixOS/nixpkgs/commit/bce876b8cd2e4bba28af72bd9ce2de62ef284c2b.


Refactor internals, add from.dirty & form.valid

Fixes #22 #24 #40

👏 👏 👏
So sorry that I didn't manage to finish it myself. Was pulled into a completely different task at work that suddenly got urgent.


Importing existing Feeds not working

Describe what happened
I am trying to import an existing Azure DevOps Feed so it is managed by Pulumi.
Seems like the read during the import is not issued with the correct parameters.
I can read the feed just fine with the Get or GetFeed Methods.
Sample program
$ pulumi import azuredevops:index/feed:Feed internal-feed 52b41ced-793e-4779-9048-9c42c4ed6782
Output
Previewing import (dev):
     Type                       Name                         Plan       Info
 +   pulumi:pulumi:Stack        azuredevops-feed-import-dev  create     1 error
 =   └─ azuredevops:index:Feed  internal-feed                import     1 error
Diagnostics:
  azuredevops:index:Feed (internal-feed):
    error: Preview failed: refreshing urn:pulumi:dev::azuredevops-feed-import::azuredevops:index/feed:Feed::internal-feed: 1 error occurred:
        *  reading feed during read: Argument args.FeedId can not be nil or empty

  pulumi:pulumi:Stack (azuredevops-feed-import-dev):
    error: preview failed

Log output
https://gist.github.com/kerko/572035cc37d43b069fbd5031c50bcd2e
Affected Resource(s)
azuredevops.Feed
Output of pulumi about
$ pulumi about
running 'dotnet build -nologo .'
  Determining projects to restore...

  Alle Projekte sind für die Wiederherstellung auf dem neuesten Stand.

  azuredevops-feed-import -> D:\bugs\pulumi\azuredevops-feed-import\bin\Debug\net6.0\azuredevops-feed-import.dll

Der Buildvorgang wurde erfolgreich ausgeführt.

    0 Warnung(en)
    0 Fehler

Verstrichene Zeit 00:00:01.09

'dotnet build -nologo .' completed successfully
CLI
Version      3.111.1
Go Version   go1.22.1
Go Compiler  gc

Plugins
NAME         VERSION
azuredevops  3.1.1
dotnet       unknown

Host
OS       Microsoft Windows 11 Pro
Version  10.0.22631 Build 22631
Arch     x86_64

This project is written in dotnet: executable='C:\Program Files\dotnet\dotnet.exe' version='8.0.206'

Backend
Name           DESKTOP-JK
URL            file://~
User           AzureAD\JanKerkenhoff
Organizations
Token type     personal

Dependencies:
NAME                VERSION
Pulumi              3.63.1
Pulumi.AzureDevOps  3.1.1

Pulumi locates its logs in C:\Users\JANKER~1\AppData\Local\Temp by default
warning: Failed to get information about the current stack: No current snapshot

Additional context
No response
Contributing
Vote on this issue by adding a 👍 reaction.
To contribute a fix for this issue, leave a comment (and link to your pull request, if you've opened one already).

It appears this is not implemented in the upstream provider. The import section in both their and our docs is missing: https://registry.terraform.io/providers/microsoft/azuredevops/latest/docs/resources/feed
Additionally, the implementation for the create and update operations do not call d.SetId(...): https://github.com/microsoft/terraform-provider-azuredevops/blob/7f801b0cda433ecd68bac6ea9888c4a6f7a7d9a7/azuredevops/internal/service/feed/resource_feed.go#L56-L85
You could try filing an issue in the upstream project to see if this enhancement could be added.


[WIP] Cleanup and remove pass by value

Starting this branch to track some of the work described in #132. This PR aims to

[ ] Remove normaization cache
[ ] Change the parameters that have been passed by value to const& to prevent copying so many times (apart from the ones that clang tidy changed anyway).

cc @apendleton (this isn't done yet)

We'll want to be careful about the functions that mutate the strings they accept as arguments. This is safe for pass by copy, but not for pass by reference -- we might end up mutating someone else's string. So we'll want to change them to const&, and then if they mutate anything, we'll want to make the copy explicit.


⬆ upgrade dependencies

Upgrade all dependencies to latest
Drops support for python 3.6, 3.7, 3.8

Brave to upgrade everything in one go, but let's see what breaks :-)

👀 no breaking changes!
@dbluhm Going through each dependency it seemed like there shouldn't be any major changes - but, too good to be true? 😄 Any changes that you can see that are still necessary?


Fix impress status CLI

Closes: https://github.com/metarhia/impress-cli/issues/13
Ref: https://github.com/metarhia/impress/issues/858

This commit should also fix https://github.com/metarhia/impress/issues/858

Landed in 592f8238db869ccd81613c2557cb23346fdd85b4.


Fix apparent freeze when building -sys with --release

rustc has trouble dealing with a large amount of returns within the same expression when optimization is enabled, causing the build to appear to halt. This problem was introduced in #309 , which used many try expressions in a single constructor.
Although the exact cause within rustc is unknown, separating the try expressions into let bindings resolved this problem.
This PR also added release builds to CI, in order to detect similar problems before merging in the future.

Could this have anything to do with #323 maybe?

I think it's unlikely. The library in the other issue depends on version 0.8 from crates.io, which does not contain #309. I also didn't notice excessive memory consumption while trying to reproduce this bug.


Update for OT 1.16

@conda-forge-admin please rerender

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.


[5.8] Update preset setting for bootstrap option

Add JQuery and popper.js in bootstrap.js file which is required for bootstrap.

Hello! Would you mind sending this to laravel/ui repository, which is where these things are moving?

Sure thing.


[WIP] A Maths library for Silk.NET

Summary of the PR
A short summary of this PR and what it adds/removes/fixes.
Related issues, Discord discussions, or proposals
Links go here.
Further Comments

Cancelled for the time being.


env vars for polling intervals

Description
Closes #1033

Add env vars for 4 different common polling intervals, fast (1000), normal (30000), long (60000), extra long (300000):

NEXT_PUBLIC_FAST_POLL_INTERVAL=1000
NEXT_PUBLIC_NORMAL_POLL_INTERVAL=30000
NEXT_PUBLIC_LONG_POLL_INTERVAL=60000
NEXT_PUBLIC_EXTRA_LONG_POLL_INTERVAL=300000

NEXT_PUBLIC_ prefix is used since these are consumed in client-side code.
Use env vars in all pollInterval params to useQuery in the codebase
The original intent behind this PR is just to be able to reduce network noise in local dev, for easier debugging.
Updated the PR template to ask if new env vars are being introduced

Screenshots
N/A
Additional Context
N/A
Checklist

[X] Are your changes backwards compatible?
[X] Did you QA this? Could we deploy this straight to production? ⚠ Needs env vars configured in prod ⚠
[ ] For frontend changes: Tested on mobile?
[X] ⚠ New environment variables introduced ⚠

LGTM! Pulled code and checked that polling still works as expected.
While searching for setInterval, I only found one location where we could also replace a number with FAST_POLL_INTERVAL:
https://github.com/stackernews/stacker.news/blob/1d154ec9b5a373247ea55615c2cd1b2de92e7689/components/invoice.js#L345-L365

Good spot, I didn't think to search setInterval. I can make this adjustment, too.


Updated README

What

Updated README

Why

We need comprehensive README

Still WIP

@s2504s maybe you should also explain the minimum possible variables needed to use the module. Sometimes, like me yesterday, you only want to give the module a quick try...


Assets

Here's the figma file
The Site page has all of the actual pages.
The tailwindui login is in OnePassword
The project has been setup with tailwindui, and all deps needed in order to get moving


Tested on EdgeRouter 6P - works flawlessly

Hi @x70b1,
first of all thank you for this great tool!
I was using Telegraf on various other network components already but to be able to gather stats directly on the router and sending them to my existing InfluxDB/Grafana setup is bliss!! Very well done for this mips package of telegraf.

So to extend your compatibility list for the ER-6P:
Internal plugins: Check
firmware: Check
interfaces: Check
power: Check
temperatures: Cross (this is not working)
Here's a screen of my Dashboard:

Again thank you and let me know if I can help in the future (I was thinking of tapping into the DPI data of the router somehow).

I am happy that this project is also helpful for you :sunglasses:
Thanks for the feedback, I will update the table for ER-6P.
If you have ideas about improvements, just share them. DPI is something that I dont use. The chances that I will work on this is very low. So if you want to start to work on it or other topics, that would be cool.


No Sound, No Vibration

Hi guys, i hope you can help with this.
This happened to me since i was in Beta2:
I activated the "Use faster animations" options and every since then my UI Sounds and vibrations messed up, here is a little list of my "problems":

No sound: Notifications, typing on keyboard, unlocking the device with LockGlyphX, that "pop" sound when liking something on Facebook, the sound of the bubble when you send a message from the message app, when connecting my usb to charge my phone etc
No vibrations: Notifications, setting up a personal vibration, when connecting my usb to charge my phone, getting "failed touch ID" (try again) etc
Sound working on: Receiving calls, Alarms (to wake up), music, etc
Vibration working on: Receiving calls, Alarms (to wake up, 3D touch, haptic feedback (even with the Taptic keyboard tweak), etc

So far its been 3 days and its all i've been able to discover, I am not 100% sure if this is a problem with LockGlyphX or not, hope you can tell me, fix it or help me to modify something in the system through iFile or Filza.
Thank you for the hard work!
Ps: I already checked my iOS settings like "Vibrate on Silent" or "Vibration" inside accessibility.

I don't see how it could possibly be related to LockGlyphX, especially once it's removed.

Neither do i, i have no idea how it happened, i dont know if maybe it changed some code or something, again..im not a 100% sure that this is caused by your tweak, but i just thought about it because it happened when i activate that :( and i was hoping to see if you guys vould help me, if not..i totally understand :(
Thanks again.

Do you have proper sounds and vibrations when in non-jailbroken mode?

Hi again! Wow, thank you very much for the answer! You give me hope haha. Unfortunately no :(
I even just tried uninstalling all my tweaks and still didn't change :/
Sent from my iPhone
On Mar 3, 2017, at 08:13, Sticktron <notifications<EMAIL_ADDRESS>wrote:
Do you have proper sounds and vibrations when in non-jailbroken mode?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/Sticktron/LockGlyphX/issues/41#issuecomment-283949584, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AY6bp8o_rMnHsPGYJpr-pgXWa3lvhBNWks5riBIEgaJpZM4MQHN9.


Request: configuration for DOM insertion

Hi Again!
I love making these mo.js shapes- it would be really powerful to be able to decide where in the DOM the SVG object is inserted. Right now it is z-indexed above everything, but I have a situation where I would like to insert the SVG inside of an existing SVG so that they can scale together seamlessly. This would also be really great for when you have complicated illustrations or UI/UX where you'd like the DOM insertion to happen in a lower z-index.
Apologies if you can already do this and I just haven't found the docs for it. Thank you!

Hi Sarah!
Thanks for the issue!
We have parent option, which defines where to insert a module. There is one problem though, mojs modules are wrapped into <div /> root so it cannot be inserted in the SVG. The outer HTML is responsible for translate/scale animations.
This will be resolved when we will have refactored Shape module and maybe SVG module for SVG animations.

Thank you for your time and effort in this! Much appreciated. I think it will be very useful.

@legomushroom I would love to use mo-js but this issue let me switch to svg.js. Looking forward to seeing this feature landing in mo-js soon.

Not inside a SVG, so not applicable for this- But my way of doing it would be inserting the mobject as a child of an object with the z-index I want.
Related to this -and I might be missing something-, but it seems like changing the parent inside tune does not have any effect? :( Should it work fine?


Unable to download dataset

I am unable to download the dataset directly from your server using the link:
wget http://<IP_ADDRESS>/GraSP/<file_name>.
I get the following error:
Connecting to <IP_ADDRESS>:80... failed: Connection refused..

Hello
We've been having some trouble with our servers, so we have now uploaded the entire dataset into the Google Drive link provided in the README file.


Thaumaturgy research requirements

This is just a suggestion, but I feel that the requirements for unlocking thaumaturgy are much more difficult and excessive than the requisites for alchemy and artifice. I'm pretty unlucky when it comes to world gen, and I usually have to resort to teleporting myself randomly around the world until I land in a desert, ocean, or extreme hills. For unlucky players like myself, or for anyone using mods that replace vanilla world gen/boimes, unlocking thaumaturgy can be a disaster.
I would suggest either having a config option that lets you discover thaumaturgy by some other means, or by lessening the difficulty of the research entirely  (maybe only make us search for a magic forest biome?)

I do agree that there is a disparity in the difficulty there and I was planning on balancing it out a bit more.

This issue was moved to Azanor/thaumcraft-suggestions#169


UHF-5985: Debug log ad groups

UHF-5985

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (95844e1) 72.95% compared to head (0a54ed3) 73.91%.

:exclamation: Current head 0a54ed3 differs from pull request most recent head 326c070. Consider uploading reports for the commit 326c070 to get more accurate results

Additional details and impacted files
@@             Coverage Diff              @@
##               main      #31      +/-   ##
============================================
+ Coverage     72.95%   73.91%   +0.95%
- Complexity       55       57       +2
============================================
  Files             4        4
  Lines           196      207      +11
============================================
+ Hits            143      153      +10
- Misses           53       54       +1

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


Inconsistent use of bool and BOOL data type

Hi Tnunnink,
First of all... Awesome project, lots of great stuff here!
I'm having some trouble loading some L5X files that have been generated by Rockwell's Application Code Manager. I'm sure you're aware that ACM creates L5X files which are a bit inconsistent in the use of "0"/"1" or "false"/"true". Logix Designer is similarly inconsistent when importing these L5X files.
Here's an example of a controller element in an L5X file:
<Controller ... ShareUnusedTimeSlice="0" ... ReportMinorOverflow="false">

This will happily import into Logix Designer but will throw an error when loaded as a L5X object as it expects the ShareUnusedTimeSlice property to be a bool.
When changed to this:
<Controller ... ShareUnusedTimeSlice="true" ... ReportMinorOverflow="false">

The opposite is true - failing to import into Logix Designer but happily loaded as an L5X object.
I've had this problem with Controller's ShareUnusedTimeSlice property and Program's Disabled property so far.
I see the BOOL.cs class that is being used as a solution to this in a variety of places. Would be happy to make a PR where I fix some problematic properties but thought I'd first pop by to see if you had any insight to offer as to why it's not more widely used and/or if there's a better approach.
Cheers!

Oh sorry, you are saying the L5X object cannot parse 0/1 as a bool and is throwing an exception, correct?

Yes, that's right. I'm attempting to load an L5X file into an L5X object and an exception is being thrown where a "0" or "1" is found where a bool was expected.
So far I've noticed it in two places.
The Disabled property of Program, and the ShareUnusedTimeSlice property of Controller.
If I modify the L5X file to change these to be "true" or "false" instead then they will fail to load in Studio.
Have found it quite difficult to pin down how ACM decides, when generating an L5X file, which properties are "0"/"1" or "true"/"false". It almost seems like [BOOL](public sealed class BOOL : AtomicData, IComparable, IConvertible, ILogixParsable) should be used in most places.

I see. Thanks for clarifying. I'm adding a couple things to fix this.
For reading Booleans, I am adding a custom parse function to LogixParser to handle properties/attributes that are being parsed as bool, in a similar way to BOOL. This will fix issues with L5X ability to read 0/1 and covert that to true/false, and we can leave properties as bool in this case.
For writing Booleans, I'm adding a new set method the LogixElement called SetBit which will write 0/1 using the provided Boolean value. This will have to be replaced with the SetValue method for each property/attribute that requires the 0/1 instead of true/false. Using BOOL here doesn't necessarily fix the problem, since we would still need to decide which value to write. BOOL just handles parsing.
In regards to the Disable property for Program, Rockwell's documentation says it serializes as true/false.

When I export a Program, I see true/false as well.
<Programs Use="Context">
<Program Use="Target" Name="Empty" TestEdits="false" Disabled="false" UseAsFolder="false">
<Tags/>
<Routines/>
</Program>

But I tested changing this to 1 and it imports just fine and indeed inhibits the program. Out of curiosity, which version of Studio are you using? I suppose I can change Disable to be 1/0 but it's strange that it's not working as true/false for you.

Version 2.3.1 adds the fixes described above and implements the SetBit for ShareUnusedTimeSlice.
I added a test to verify but let me know if you still have issues or not.

No problem! Thanks for sharing this issue. I don't have time to test exporting/importing every single thing, so you finding this helps this library become better.

Beautifully done! That's every L5X file that I have on hand being loaded with seeming no problems at all.
Again, thank you a lot for your efforts here - it's greatly appreciated!


Creators auto-complete

Changes introduced from the creatibutors auto-complete feature branch. Some notes for the reviewer:

We went with using Refs (which is horrible practice), since refactoring the underlying components to make state change handlers parametrized via props ended up being very messy/difficult, with unknown implications for other uses. If this is too hack-ish, we're up for refactoring, but we'll need some prior discussion/guidance for the specifics.

(Closes #397)

Squash-merged manually in https://github.com/inveniosoftware/react-invenio-deposit/commit/04eba96448077d5242efa4ac947dba9b9d40bf8c (but GitHub doesn't detect that...)


Cannot start http server on non-default port

Hey - Trying to expose an influxdb backend on port 8083.  Getting the following...
time="2022-05-02T17:43:56Z" level=info msg="registered backend" backend=InfluxDB
time="2022-05-02T17:43:56Z" level=info msg="starting ingester"
time="2022-05-02T17:43:56Z" level=info msg="starting http server" listen_addr=8083
time="2022-05-02T17:43:56Z" level=panic msg="cannot start http server"
panic: (*logrus.Entry) 0xc00018a1c0
goroutine 21 [running]:
github.com/sirupsen/logrus.Entry.log({0xc0001147e0, 0xc0001880c0, {0x0, 0x0, 0x0}, 0x0, 0x0, {0x0, 0x0}, 0x0, ...}, ...)
	/go/pkg/mod/github.com/sirupsen/logrus@v1.6.0/entry.go:259 +0x2de
github.com/sirupsen/logrus.(*Entry).Log(0xc00018a000, 0x0, {0xc000098f10, 0x0, 0x0})
	/go/pkg/mod/github.com/sirupsen/logrus@v1.6.0/entry.go:287 +0xa8
github.com/sirupsen/logrus.(*Entry).Logf(0xc00018a000, 0x0, {0x7b4fb4, 0x6162f8}, {0x0, 0x0, 0x7ffc92e5de88})
	/go/pkg/mod/github.com/sirupsen/logrus@v1.6.0/entry.go:333 +0x85
github.com/sirupsen/logrus.(*Logger).Logf(0xc0001147e0, 0x0, {0x7b4fb4, 0x18}, {0x0, 0x0, 0x0})
	/go/pkg/mod/github.com/sirupsen/logrus@v1.6.0/logger.go:146 +0x85
github.com/sirupsen/logrus.(*Logger).Panicf(...)
	/go/pkg/mod/github.com/sirupsen/logrus@v1.6.0/logger.go:187
github.com/sirupsen/logrus.Panicf(...)
	/go/pkg/mod/github.com/sirupsen/logrus@v1.6.0/exported.go:174
main.main.func1()
	/go/src/workspace/cmd/ingester/main.go:62 +0xe7
created by main.main
	/go/src/workspace/cmd/ingester/main.go:59 +0x3c5

Hey @eliotmartin, thanks for opening this!
The error is most likely due to some socket error, perhaps the port is already in use. Unfortunately I didn't log the error when this happens, but this should be now fixed and latest image pushed to Docker Hub.
Could you try to pull the latest image again, and if you still cannot figure out the error you can report back! Cheers

Actually I just noticed the issue. You need to use :8083 instead of just 8083, which means to listen on all interfaces at port 8083. Maybe you can try that!

I'll close the issue now, as it seems like it's a configuration issue :)


Upload Card bug

On my iPad in Unplugged, when you tap on "Select Photo" nothing happens.  It should take you to the photo library.   I'm concerned that the Rotate and Upload functions may also be broken since I wasn't able to test them.
This function is also broken on Unplugged on Android
(I've just re-synced on iOS and it appears to be working again)

Seems to be working fine for me.


AttributeError: 'generator' object has no attribute 'next'

When trying to import from a file. The traceback is
Traceback (most recent call last):
  File "/home/moshe/bin/gcalcli", line 11, in <module>
    load_entry_point('gcalcli==4.1.0', 'console_scripts', 'gcalcli')()
  File "/home/moshe/.local/lib64/python3.6/site-packages/gcalcli/gcalcli.py", line 2057, in main
    FLAGS.verbose, FLAGS.dump, FLAGS.reminder, FLAGS.file.name)
  File "/home/moshe/.local/lib64/python3.6/site-packages/gcalcli/gcalcli.py", line 1569, in ImportICS
    v = vobject.readComponents(f).next()
AttributeError: 'generator' object has no attribute 'next'

Github version. ics file: vv.txt

Yes, py3 changes how generators work.  We need to look up the right way to do this with six.  I assume there is some right way.  Thanks!


Allow configuration of a theme

Feature request:
Allow specifying an external theme, or integrate some themes such as the following Dark Theme:
https://github.com/ptrkcsk/one-dark-plantuml-theme/blob/main/theme.puml

I discovered that there is an option available under "Tools>Options>PlantUML>Diagram: Header file" .
And also that Plantuml has a new directive "!theme" and that includes several themes.

The Header file function has been requested with the issue #1 to allow using default theme for all diagrams.


Consider adding support for Authz docker plugin support

dockerd can be extended to support an access authorization plugin. https://docs.docker.com/engine/extend/plugins_authorization/
Ratify plugin can be extended to support Authz so that all content fetch commands (image pull, build) can implicitly invoke Ratify and allow/deny the pull operation.

In order to support this, not only does the existing docker plugin implementation have to be updated; but also, Ratify's cli needs to be updated to support more authentication support for key management providers and ORAS store. Ratify cli also needs to add key management provider support which does not exist today.
https://github.com/ratify-project/ratify/issues/1300
https://github.com/ratify-project/ratify/issues/1630


Strange issues with partial mocking.

We're struggling a bit with our application which is half built at the moment. Specifically, we have two different ways of requesting a type of data. One is implemented, the other is not currently (details below). The one that is implemented is working well but the one that is not implemented is using the mock data generators to generate up to the reference to Event but it's using the live implementation of the Event resolvers to handle the data from there.
This sounds reasonable, but I've been struggling to pass in a source object out from the mock so that the real resolvers don't get undefined and break horribly.
For instance (typescript):
export const EventPostEdge = (): { node: {} } => {
  console.log("In the mocked EventPostEdge.");

  return {
    node: {
      title: faker.name.title(),
      uuid: uuid(),
    },
  };
};

Which just produces the following output if I run the query:
server_1  | In the mocked EventPostEdge.
server_1  | In the mocked EventPostEdge.
server_1  | TypeError: Cannot read property 'title' of undefined
server_1  |     at title (/usr/src/app/src/graphql/types/posts/EventPost.ts:108:43)

Schema generation in the application:
const schema = makeExecutableSchema({ resolvers, typeDefs: loadSchema(name) });
addMockFunctionsToSchema({
  mocks,
  preserveResolvers: true,
  schema,
});

Relevant parts of the schema:
type Query {
  latestEvents: EventConnection!
  event(id: UUID!): EventPost!
}

scalar UUID;

type EventConnection {
  edges: [EventPostEdge!]!
}

type EventPostEdge {
  node: EventPost!
}

type EventPost {
  uuid: UUID!
  title: String!
}

Broken query:
query events {
  latestEvents {
    edges {
      node {
        uuid
        title
      }
    }
  }
}

Ok, I tracked down the issue. The resolver was defined as such:
export const EventPost = {
    title: (source): string => source.parent_event.title,
}

Which means I had to alter the mock for EventPostEdge as follows:
export const EventPostEdge = (): { node: {} } => {
  return {
    node: {
      parent_event: {},
    },
  };
};

This means I'm providing the parent_event property in the source and the resolver succeeds but returns undefined.
This... works, but feels slightly unintuitive. I would have expected the mocking to handle the error and return mocked data since the parent elements were mocked. This resolves my issue and I can move forward but I'd like to leave the ticket open for discussion about whether the current behaviour is correct or not.

Basically, this was an issue where if you have mocks mixed in with production code and your call tree of resolvers includes mocks which return implemented types then the mock needs to return a valid source type for a real object because the implementation will override any mocked version of the type. This means the mocks need to be aware of the real implementations. We quickly ran into issues because a lot of our mocks were contributed by the client team.
For example:
Query -> EventConnection (mocked) -> EventEdge (mocked) -> Event (implemented)

It would be useful to be able to have the above tree use a mocked version of the Event types resolvers till the actual search connection was implemented otherwise EventEdge has to return something realistic as a source object rather than a dummy empty node (or more realistically, completely omit this mock because it does nothing) which an Event mock can then fill in with details for each node in the mocked up search. It feels weird to have the mocks need to know how the implementations work and mirror them to be able to work correctly. If that is what you wanted to do you'd just do stub implementations instead.

Could you share your apollo setup? I'm trying to implement partial mocking. But I'm getting mocked Hello World data on the non mocked resolvers.

Sorry so late with reply, we just came under new management recently!
To avoid undefined, the mock should be:
{
  EventPostEdge: () => ({
    node: {
      parent_event: {
        title: faker.name.title(),
      }
    },
  }),
}

I think you are essentially correct, however, that it makes sense to stick with just implementing mocks and no resolvers, or putting in stub implementations of all the resolvers. Things can get messy fast.
I am going to close this for now, feel free to reopen as needed or if you think we can find and adopt a better approach.


Metro map draft for nf-core/chipseq

Description of feature
Hello everyone,
Here is a metro map draft for nf-core/chipseq.
Feel free to give feedback on it and to modify it yourself if you feel like it!
I wanted to provide the svg file here but for some reason it implodes when I try to attach it so I'll make it available on the chipseq channel of the nf-core slack.
Have a nice day!

I closed this issue since the metro map has been added in https://github.com/nf-core/chipseq/pull/327


Demonstrate #1485, #1487, #2662, #2664 in functional and unit tests

test additional cases close to these bugs as well

for #1485

show odd @Html.CheckBox(), @Html.Hidden() behaviour in unit tests
show odd @Html.TextBox() behaviour in functional tests (templates)

for #1487

show odd @Html.Value() behaviour in unit tests
show odd @Html.RadioButton(), @Html.TextArea() behaviour in functional tests
show lack of validation attributes for @Html.RadioButton(), <select> tag helper

for #2662

show odd @Html.RadioButton(string.Empty) behaviour in functional tests

for #2664

show failures with @Html.ListBox() in unit tests

nits:

test IHtmlHelper methods, not extensions
use ViewData, not ViewBag in HtmlGeneration_FormController
name test methods a bit more consistently
rename HtmlHelperValueExtensionsTest to HtmlHelperValueTest

/cc @Eilon let's discuss tomorrow and decide if the expectations in the skipped tests match what we want.

Will reopen with correct base bracn


doc Improve and supplement hight-concurrent-point-query documentation

Proposed changes
Issue Number: close #xxx

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

run buildall


fix(verify-bundle): add support for top-level directories

Adds support for top-level directories in app-bundles.
Related to https://github.com/dhis2/dhis2-core/pull/11831

Hi @Birkbjo - I'm cleaning up open reviews that have been requested from me and I see that this never got merged... is it waiting on something or did it already get fixed elsewhere so this PR can be closed?  Happy new year!

@amcgee Don't think this is waiting on anything. You merged the same functionality in core, so I guess I just waited for your approval.
Of course it needs to be updated to base, but I will fix that.

@Birkbjo following up on this again, no need to wait for my approval - feel free to update and merge


Add release date for Rooms SDK Public Preview .Net

Contributing to the Azure SDK
Please see our CONTRIBUTING.md if you are not familiar with contributing to this repository or have questions.
For specific information about pull request etiquette and best practices, see this section.

API change check
API changes are not detected in this pull request.

/check-enforcer evaluate


General docs improvements

Pull Request Test Coverage Report for Build<PHONE_NUMBER>5
Details

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

Totals

Change from base Build<PHONE_NUMBER>5:
0.0%

Covered Lines:
824

Relevant Lines:
824

💛  - Coveralls


BioLink API "GET /bioentity/phenotype/{id}/anatomy/" Not Functioning

Original ticket by @Superraptor in monarch-app
Hi! First, I'd like to say amazing work here, I'm a huge fan of the Monarch Initiative application and the API has been an amazing resource as of late.
Second, my apologies if this is the wrong place to post this. I'm specifically referring to the BioLink API documentation as seen here.
Essentially, there are two provided examples, "ZP:0004204" and "MP:0008521". The first provides the following output:
[
  {
    "label": "muscle pioneer",
    "categories": null,
    "types": null,
    "synonyms": null,
    "description": null,
    "consider": null,
    "deprecated": null,
    "id": "ZFA:0001086",
    "replaced_by": null
  }
]

While the second produces:
[]

I thought it might work with another Mammalian Phenotype identifier, so I tried MP:0003279 (aneurysm), but it still provided:
[]

Further, I tried a Human Phenotype Ontology identifier (HP:0100790, hernia) and got the same result. All of these calls resulted in a 200 response.
Is this how the function should be working? Or is it still in development? Thanks so much!

we have this in scigraph_util
        objs = self.traverse_chain(id, [HAS_PART, INHERES_IN], "anatomical entity")

the actual path we need is has_part/{inheres_in|inheres_in_part_of}.
Currently the traverse_chain method doesn't support for path expressions... nor should it! No sense reinventing cypher / sparql property paths here. We should just pass this in as a direct cypher query
@kshefchek and @deepakunni3 should we take the opportunity to go with BOLT? Or just do this via /cypher/execute in the existing SciGraph API?

In the short term I think the only option is /cypher/execute until someone finishes https://github.com/SciGraph/SciGraph/pull/238

@cmungall Do you want me to take a look at SciGraph/SciGraph#238?
It looks like the bulk of the work is done. The only thing pending is making it configurable.
@kshefchek Let me know if there is more to that PR than what I assume.

@deepakunni3 it might not be too much work, I keep meaning to take a closer look.
@cmungall do we aim to support complex owl restrictions, such as http://compbio.charite.de/hpoweb/showterm?id=HP:0012792? I'm not sure how this is translated in scigraph.

Hmm, I think I prefer it being outside the scope of SG and having ontologies produce pre-reasoned versions with shortcut relations connecting the phenotype directory with the a anatomy term.
But in the interim it may be easier to replicate the pattern in a path query

bump, @realmarcin was asking about this today.
See also https://github.com/biolink/biolink-api/issues/115


8.2.0 g8ossal

This code requires https://github.com/g8os/core0/commit/c438fdeda11dccdb080d66665bca1ec7ad34cd20 to be merge into core0 to work

LGTM

@grimpy Thx for the review.
I'll wait for the PR on core0 to be merge before merging this one.


fix(metrics): cluster_status_capacity_used doesn't update when kafka count goes down to 0

Description

Fixes https://issues.redhat.com/browse/MGDSTRM-8866
This depends on https://github.com/bf2fc6cc711aee1a0c2a/kas-fleet-manager/pull/1100 for the test to pass, so I'll open it as draft.
Verification Steps

Checklist (Definition of Done)

[ ] All acceptance criteria specified in JIRA have been completed
[ ] Unit and integration tests added that prove the fix is effective or the feature works (tested against emulated and non-emulated OCM environment)
[ ] Documentation added for the feature
[ ] CI and all relevant tests are passing
[ ] Code Review completed
[ ] Verified independently by reviewer
[ ] All PR comments are resolved either by addressing them or creating follow up tasks
[ ] Required metrics/dashboards/alerts have been added (or PR created).
[ ] Required Standard Operating Procedure (SOP) is added.
[ ] JIRA has been created for changes required on the client side

Verified locally and works as expected.
Comments above around refactoring the cluster/kafka object builder is an optional enhancement and not a blocker for me as it may be covered in this issue: https://issues.redhat.com/browse/MGDSTRM-7993

Thanks @JameelB I've added the builder as well.


OTB configs / results don't match

Hi,
I trained models with siamrpn_r50_l234_dwxcorr_8gpu / siamrpn_r50_l234_dwxcorr_otb_8gpu configs and tested on OTB. The highest score I got after hyper-parameter tuning is 0.687 / 0.676 resp.
I trained siamrpn_r50_l234_dwxcorr_8gpu on 1 1080Ti and siamrpn_r50_l234_dwxcorr_otb_8gpu on 4 1080Ti's.
I am a bit confused as 0.696 is reported in Model zoo for OTB. Did I miss something during the training? Is training results are dependent on # of gpus?

Have you adjusted batch size, learning rate, etc.?

@zyc-ai No, I didn't modify configs

@zyc-ai No, I didn't modify configs

Then you probably should do it.

Can you tell what should I adjust in configs?