Update RELEASE.md

Proposed Changes
Update releaseability steps to capture Module Release

Codecov Report

Merging #365 (885381f) into main (5fc1e08) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #365   +/-   ##
=======================================
  Coverage   80.57%   80.57%
=======================================
  Files          20       20
  Lines         767      767
=======================================
  Hits          618      618
  Misses        135      135
  Partials       14       14

Continue to review full report at Codecov.

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


Error in detect.py with custom model

I have a problem to visualize my predictions in the tf.image.combined_non_max_suppression() dimensions inputs, maybe Im doing something wrong but I alreary did so many changes and this error keeps.
File "detect.py", line 83, in main
score_threshold=FLAGS.score
File "/usr/local/lib/python3.6/dist-packages/tensorflow/python/util/dispatch.py", line 201, in wrapper
return target(*args, **kwargs)
File "/usr/local/lib/python3.6/dist-packages/tensorflow/python/ops/image_ops_impl.py", line 4335, in combined_non_max_suppression
score_threshold, pad_per_class, clip_boxes)
File "/usr/local/lib/python3.6/dist-packages/tensorflow/python/ops/gen_image_ops.py", line 358, in combined_non_max_suppression
_ops.raise_from_not_ok_status(e, name)
File "/usr/local/lib/python3.6/dist-packages/tensorflow/python/framework/ops.py", line 6843, in raise_from_not_ok_status
six.raise_from(core._status_to_exception(e.code, message), None)
File "", line 3, in raise_from
tensorflow.python.framework.errors_impl.InvalidArgumentError: scores has incompatible shape [Op:CombinedNonMaxSuppression]
My models summary outputs seem like this
tf_op_layer_Sigmoid_10 (TensorF [(None, 52, 52, 3, 1 0           tf_op_layer_split_3[0][2]

tf_op_layer_Sigmoid_11 (TensorF [(None, 52, 52, 3, 1 0           tf_op_layer_split_3[0][3]

tf_op_layer_concat_18 (TensorFl [(None, 26, 26, 3, 4 0           tf_op_layer_Mul_85[0][0]
tf_op_layer_Mul_86[0][0]

tf_op_layer_Sigmoid_13 (TensorF [(None, 26, 26, 3, 1 0           tf_op_layer_split_4[0][2]

tf_op_layer_Sigmoid_14 (TensorF [(None, 26, 26, 3, 1 0           tf_op_layer_split_4[0][3]

tf_op_layer_concat_20 (TensorFl [(None, 13, 13, 3, 4 0           tf_op_layer_Mul_88[0][0]
tf_op_layer_Mul_89[0][0]

tf_op_layer_Sigmoid_16 (TensorF [(None, 13, 13, 3, 1 0           tf_op_layer_split_5[0][2]

tf_op_layer_Sigmoid_17 (TensorF [(None, 13, 13, 3, 1 0           tf_op_layer_split_5[0][3]

tf_op_layer_concat_17 (TensorFl [(None, 52, 52, 3, 6 0           tf_op_layer_concat_16[0][0]
tf_op_layer_Sigmoid_10[0][0]
tf_op_layer_Sigmoid_11[0][0]

tf_op_layer_concat_19 (TensorFl [(None, 26, 26, 3, 6 0           tf_op_layer_concat_18[0][0]
tf_op_layer_Sigmoid_13[0][0]
tf_op_layer_Sigmoid_14[0][0]

tf_op_layer_concat_21 (TensorFl [(None, 13, 13, 3, 6 0           tf_op_layer_concat_20[0][0]
tf_op_layer_Sigmoid_16[0][0]
tf_op_layer_Sigmoid_17[0][0]
Total params: 64,003,990
Trainable params: 63,937,686
Non-trainable params: 66,304

Did you execute save_model.py before run detect.py, mentioned in Demo?:
# Convert darknet weights to tensorflow
## yolov4
python save_model.py --weights ./data/yolov4.weights --output ./checkpoints/yolov4-416 --input_size 416 --model yolov4

## yolov4-tiny
python save_model.py --weights ./data/yolov4-tiny.weights --output ./checkpoints/yolov4-tiny-416 --input_size 416 --model yolov4 --tiny

# Run demo tensorflow
python detect.py --weights ./checkpoints/yolov4-416 --size 416 --model yolov4 --image ./data/kite.jpg

python detect.py --weights ./checkpoints/yolov4-tiny-416 --size 416 --model yolov4 --image ./data/kite.jpg --tiny

It seems that you are trying to use the output of decode_train, which is output of train.py model.


/contributing/parity-testing not found, linked to by "snapshot parity testing" on coverage page

On each coverage page, there seems to two explanations of what "Snapshot tested" means both of which link to https://docs.localstack.cloud/contributing/parity-testing/ which is now a "not found" page.

For example https://docs.localstack.cloud/references/coverage/coverage_appsync/#terminology

Snapshot Tested: the operation is part of a snapshot parity test, which verifies the responses by LocalStack and AWS are the same
...
Snapshot Tested this is a snapshot parity test, meaning the responses are validated against AWS

https://github.com/localstack/docs/blob/0f246cd5f5325fad6fc13722d16c951afcc5c375/layouts/partials/coverage/coverage_table.html#L37
https://github.com/localstack/docs/blob/0f246cd5f5325fad6fc13722d16c951afcc5c375/layouts/partials/coverage/coverage_details.html#L19

For now, the article can be found at https://github.com/localstack/localstack/blob/master/docs/testing/parity-testing/README.md


bug: Spelling error on delegated realms profile section

Provide environment information
MAC OS Chrome @ realms.world/account/delegates
Describe the bug

The text for "Delegated" is misspelt as "Delgated"
Link to reproduction
https://realms.world/account/delegates
To reproduce
https://realms.world/account/delegates
Additional information
No response

Fixed in #249


Automatic References CodeLens

Option to automatically provide references CodeLens for members using the References provider.
Motivation

Extensions currently have to manually provide references CodeLens, and it would make sense if VSCode could do it automatically, as the extensions already provide a References provider, so it feels a bit like duplicating work.
Some extensions (such as Dart), currently lack References CodeLens, but provide a References Provider.

I think the best way to do this is for extensions to opt-in to this in the contributes.

Last, you can implement all of this by implementing an extension

I think it might be difficult to do this as an independent extension right now, because you don't really know what locations to send the requests for (the API takes a Position)?
I agree it'd be nicer as an editor/LSP feature than having each language server do it (because it would provide some consistency and a single set of options for all languages), but probably it should have some API that allows requesting them for a given range (eg. the visible part of the document) in a single call.


Feature: support sbt-projectmatrix

After #438 is merged I would like to attempt to build in support for sbt-projectmatrix and adjust the generated workflows based on what type of crossbuilding setup the project is using.

I created a PR on my #438 branch in my fork: https://github.com/ThijsBroersen/zio-sbt/pull/1
It has quite some changes to detect and adjust for all types of crossbuilding. I will add descriptions and documentation later this week.


Ch 9 relative URLs

Noticed a couple places where I think relative urls should of been used.

pg 107

link back to homepage
action attribute on form

pg 112

action attribute on form

pg 113

link to login on base.html

Can send PR on this.

Hey Adam,
Thanks for your comment and sorry for the delay in replying. Things have moved around a bit since you raised this. So the page numbers are probably a little out.
Page 107 (now 111 I believe) - the links are using the Django reverse lookup function. I think they are okay as is - maybe before they were written down statically?
For the second form (page 112, now 116) we again use the reverse lookup function. Do you mean to say that action should just be "."? I guess we could do that - but I would say it's better to be explicit. You could argue either way I guess.
I tweaked the last one - thanks for spotting that.
I'll leave this open if you want to comment!
Cheers,
David

We've gone through and tried... to fix all these..


Align form title edit with prototype

We currently have the form edit title screen, but it should be aligned with the prototype

Closed by #56


[docs] /volumes with "filter" or "filters" query parameter?

The GET /volumes remote api query parameter "filters" (plural) is consistent with similar list commands (e.g. GET /images), but the v1.21 docs state that the parameter is named "filter" (singular).
Which one is correct? I guess the docs are wrong, and if so, I could provide a PR to fix them. If the docs are right and the param was meant to be filter (singular), I'd like to discuss about consistency to the other endpoints ;-)

Hi!
Please read this important information about creating issues.
If you are reporting a new issue, make sure that we do not have any duplicates already open. You can ensure this by searching the issue list for this repository. If there is a duplicate, please close your issue and add a comment to the existing issue instead.
If you suspect your issue is a bug, please edit your issue description to include the BUG REPORT INFORMATION shown below. If you fail to provide this information within 7 days, we cannot debug your issue and will close it. We will, however, reopen it if you later provide the information.
This is an automated, informational response.
Thank you.
For more information about reporting issues, see https://github.com/docker/docker/blob/master/CONTRIBUTING.md#reporting-other-issues

BUG REPORT INFORMATION
Use the commands below to provide key information from your environment:
docker version:
docker info:
uname -a:
Provide additional environment details (AWS, VirtualBox, physical, etc.):
List the steps to reproduce the issue:
1.
2.
3.
Describe the results you received:
Describe the results you expected:
Provide additional info you think is important:
----------END REPORT ---------
#ENEEDMOREINFO

P.S. the same applies to GET /networks and the docs for that endpoint.

Indeed, the doc is incorrect. filters is the right name in both cases.

:+1:  ok, I'll create a PR

@cpuguy83 see #17127


Material Design Overhaul / Day Night Theme [Ready For Review]

The screenshots you posted on the forums of the light/dark themes look really nice! Let me know when this is closer to being ready and I'll do some testing as well.

I'm quite happy with it now @Twinklebear, I dropped the [WIP] from the pull request. Its feature complete, ideally the only changes I make will be based off your's and other's feedback.

It looks like the Tags & Similar Artists headers for the torrent group description wasn't re-styled, same with various headers for a request (the artists & top contributors are updated but it seems like tags/acceptable media etc. aren't)
It also seemed to start me in the light material theme even though the setting wasn't selected. The login progress dialog also seems to flash as white when using the dark theme.
Besides those smaller things this looks really good, I think the teal is pretty nice.

Updated @Twinklebear, please check out the changes on the Request page and the profile page (recent snatches and uploads with images enables/disabled)

It looks like I still get defaulted to the light theme although the setting isn't checked, where the artist page tags/similar artists headers updated as well?
It also seems like the card views shown for uploads/snatches may not be getting updated when changing to the dark theme (they did switch previously iirc).

I've been testing on android 6 too, I can't reproduce the issue where you start in light theme. It could be that the entire system is set to light theme because its day time? This should be overwritten for the app though.
You're right about the recent's in dark theme, I'll fix that and the artists headers tonight.

Let me know when the album description headers & artist headers are updated. I did also notice that when going to an artist page there's no loading feedback shown now, I'm not sure if these changes removed it (or maybe it was gone previously), but some loading feedback for users would be good as well. Otherwise it looks like the app is hanging/broken and showing a blank page.

Let me know when the album description headers & artist headers are updated.
This has been added
I did also notice that when going to an artist page there's no loading feedback shown now
Can you give me steps to reproduce this? I see a loading indicator when I do an artist search

Ah so if you're directly to an artist page from the search we've already loaded them, it's a bit of an interesting thing in how the site handles searching for artists. If you search and match an artist name e.g. "the sword" the search response you get is actually the artist page, so no loading is done in the artist fragment, we just forward that data we got from the search response.
Try going to an artist from an album or similar artist list, (albums seem to be missing a loader too actually).

Oh ok, I can see it now. I'll have a quick look tonight and see if I did anything weird to break it.
Is that the last outstanding issue before we can close this PR?

I think so, I might've missed a commit when I was noticing that the album description headers & artist description page headers weren't updated.

I have found the problem. It is because the app used to use a spinner in the AppBar for paged fragment activities. Since the migration to Toolbar, this hasn't been supported. That is why Artist and torrent group don't have indicators.
I won't have time to work on this for a while and I don't see a quick fix. I guess we can either park this PR,  merge with know issue or you could have a look at it?

Interesting, I thought there was also a big spinner shown in the fragment while it was loading. Yea I can take a look at this and see what happened to the loading spinner.

Hey @sxo, I checked the git blame on the ArtistFragment and it looks like we may have only ever shown progress in the menu bar, or a while back stopped showing the spinner in the fragment. So I don't think these changes removed any of that, and if you're ready to merge it I think everything looks good!
Adding back in some loading feedback to fragments missing it will have to be added as a todo.

I just pushed a fix for another missing background on dark theme. I feel like there may be other obscure things found once we roll it out. I'm happy to do a minor bug fix PR if a few things come back related to dark theme.
With that, I'm happy to merge.

Cool, sounds good!


Query Frontend: add query instant tripperware for query sharding

Signed-off-by: Ben Ye<EMAIL_ADDRESS>

[ ] I added CHANGELOG entry for this change.
[ ] Change is not relevant to the end user.

Changes

This pr adds a simple tripperware for sharding instant queries.
Verification

One question, do you think it makes sense to have separate sharding number for query range and query instant? It would mean we change --query-frontend.vertical-shards to
--query-range.vertical-shards
--query-instant.vertical-shards

which is a "breaking change", but we also haven't released the feature yet.

One question, do you think it makes sense to have separate sharding number for query range and query instant? It would mean we change --query-frontend.vertical-shards to
--query-range.vertical-shards
--query-instant.vertical-shards

which is a "breaking change", but we also haven't released the feature yet.

What about using the same NumShards flag?
type Config struct {
	QueryRangeConfig
	LabelsConfig
	DownstreamTripperConfig

	CortexHandlerConfig    *transport.HandlerConfig
	CompressResponses      bool
	CacheCompression       string
	RequestLoggingDecision string
	DownstreamURL          string
	ForwardHeaders         []string
	NumShards              int
}

Now the NumShards is defined in the root level instead of inside QueryRangeConfig so I think it is totally okay to use it in instant queries.
Maintaining two flags are also okay but I cannot think of a usecase for setting vertical shards differently for different query types.

Got it. Let's start with a single flag and we can revisit if a use case comes up.

After we vendor the cortex package internally, is there a way to import the cortex proto file we vendored? Tried multiple times but didn't find a way to get it working with existing genproto.sh script.

After we vendor the cortex package internally, is there a way to import the cortex proto file we vendored? Tried multiple times but didn't find a way to get it working with existing genproto.sh script.

Yeah, I think it hasn't been updated. You probably need to add -I option that points to the Cortex directory.

Still cannot get it working by adding the Cortex dir to the proto dependency. Wondering what's the future plan on this.
Are we allowed to modify the vendored Cortex code for some use cases? We may need to build the Cortex proto ourselves.

Could you share the command that fails for you currently?

Could you share the command that fails for you currently?

Thanks. I figured out a way to compile the Cortex protobuf instead and make changes on it directly. Not sure if it is a acceptable way but I got it working finally. Seems query response parsing and instant query sharding are working properly on my local tests.
I will add more tests soon.

This is ready for another round of review. @fpetkovski @GiedriusS

This looks good, my only concern is that we need to modify the vendored Cortex files. I am not sure though what's the strategy for keeping up to date with upstream Cortex.

Yeah, I see your concern. I am not sure about this as well. If not needed I can try to avoid modifying the Cortex code. But this causes more code duplication. For example, we need to copy Cortex unexposed functions like merging query stats, etc.

Even though it's not ideal for syncing up with the upstream Cortex, we'll need to modify the vendored Cortex files. If I'm not mistaken, @GiedriusS also working on a PR that modifies the vendored files. It'll be a half-automated process to upgrade Cortex in Thanos, but I don't see any other options. If someone has a brilliant idea to do this, please let us know :)

Amazing stuff, one of our instant queries went from 36s to 8s.

Whoops, I didn't expect the PR got merged so quickly. I haven't added some unit tests and changelog, too. I will add them in follow up prs.
One important thing I want to make sure is to have PromQL compatibility test for Thanos Query behind the query frontend to make sure no regression. Since the pr is already large I will do it in another pr, too.


🐛 BUG: wrangler deploy deploy fails with code: 10013

Which Cloudflare product(s) does this pertain to?
Workers Runtime
What version(s) of the tool(s) are you using?
3.22.3 [wrangler]
What version of Node are you using?
v20.8.0
What operating system are you using?
Linux (Ubuntu)
Describe the Bug
wrangler deploy deploy fails with workers.api.error.unknown [code: 10013]
Worked some hours ago today, seems to not be working anymore
On status page everythign is green in "Europe" and "Cloudflare Sites and Services" sections in https://www.cloudflarestatus.com/
Please provide a link to a minimal reproduction
No response
Please provide any relevant error logs
npx wrangler deploy
 ⛅ wrangler 3.22.3
-------------------

✘ [ERROR] A request to the Cloudflare API (/accounts/ACCOUNT_ID/workers/services/edge-encoder) failed.

  workers.api.error.unknown [code: 10013]

  If you think this is a bug, please open an issue at:
  https://github.com/cloudflare/workers-sdk/issues/new/choose

Closing, since it resolved itself after some hours


Feature request: add support for more customisations of Draftail

Previously at #4315 / #4436. Related: #4445, #4223.
At the moment, rich text extensions can only customise the following parts of Draftail:

Blocks
Inline styles
Entities (e.g. links, images, embeds)
Horizontal rules being enabled or not

Here are things that are not customisable and would be very valuable to make customisable:
// Displayed when the editor is empty. Hidden if the user changes styling.
placeholder: "Write here",
// Show undo control in the toolbar.
showUndoControl: true
// Show redo control in the toolbar.
showRedoControl: true,
// List of active decorators.
decorators: [],
// Additional React components to render in the toolbar.
controls: [],
// List of plugins of the draft-js-plugins architecture.
plugins: [],
// Max level of nesting for list items. 0 = no nesting. Maximum = 10.
maxListNesting: 4,

And things that I think would be valuable for some extensions, but probably not that valuable to make customisable just yet, except to allow for experimentation:
// Called when the editor receives focus.
onFocus: null,
// Called when the editor loses focus.
onBlur: null,
// Optionally override the default Draftail toolbar, removing or replacing it.
topToolbar: Toolbar,
// Optionally add a custom toolbar underneath the editor, e.g. for metrics.
bottomToolbar: null,

Additionally, for those customisations that would be done with Wagtail’s rich text features, I think it’ll be important that a single rich text feature can map to multiple Draftail extensions – for example, a code-block rich text feature could configure both a block type for code (as is supported already) and a decorator that would do syntax highlighting. I think this might be possible already in the rich text features API, but the Draftail integration of this might not support it.
Implementation
I think the options described above fall in different categories:

Things that map well to "rich text features" or "rich text formatting" because they determine the formatting available in the editor in a widget-agnostic way. For example an "emoji" feature that maps to a plugin and a control, and we would likely want to enable either site-wide, or per-field
Things that would ideally map to field or widget attributes, because they aren’t widget-agnostic. For example providing a custom placeholder string for some fields, or hiding the undo/redo controls site-wide.
It might also make sense to have customisation APIs that are client-side only – for more advanced customisations that we might not want to have APIs for, but where it would still be valuable to reuse existing code.

As a way to assess API designs, here are types of editor extensions that I would like those new APIs to support:

Indent/dedent buttons. This is the only operation currently doable with the keyboard only, thus unusable on touch screens with virtual keyboards that generally don’t have a "tab" key.
Paste from Word button – content import features
Clear formatting button – Resets the editor to plain-text
Character replacements – smart quotes, convert --> to → like Google Docs
Slash-commands – interact with the editor, and Wagtail, by typing CLI-like commands with autocomplete
Emojis – emoji picker, copy-paste support
Content linting – validation rules, highlighted words
Content metrics – live-updating measures of the content’s readability, SEO-keywords-ratio, sentiment analysis.
Single-line input – the editor constrained to a single block, with no block-level formats in the toolbar. Pasting causes the content to be added in a single line.

I’m interested in this and have used the same approach as above to inject some functionality into the draftail instances. In my case I wanted to replace ASCII style quote-characters with guillemets, which can be solved by a relatively simple draft-js plugin. Unfortunately there is no way to add draft-js plugins through the Wagtail API.

Thanks @kmohrf, that’s good to know. Do you have any code to share for reference? There aren’t that many people publicly talking about customizations like this, so any real-world implementation is appreciated.
For this type of character replacement – it’s something I would like to support out of the box eventually (https://github.com/springload/draftail/issues/113). As you say though, it’s also desirable to be able to add with draft-js-plugins.

I’ve published the code as a gist for reference :)

Hi, any update on this?
I would love to have an "add emoji" button!

It looks like (as predicted), the monkeypatch suggested by @thibaudcolas above may no longer be working with current versions of wagtail, as per this issue from the wagtail-readinglevel package (that the above example was taken from).
https://github.com/wagtail/wagtail/issues/5580#issuecomment-574693649

This example from @alxbridge updates the approach for more recent wagtail versions
https://github.com/vixdigital/wagtail-readinglevel/pull/32

Clever! This approach seems to rely on defining those extensions as entities, whose JS files will then be loaded as configured.  This seems to then register an entity server-side, and a control client-side using the API monkeypatch. I was surprised this doesn’t result in two buttons in the editor toolbar, but I guess this might be working since Draftail v1.4.0 ("Fix empty buttons appearing when providing custom formats without a defined label or icon"). I believe we’re only using that release in Wagtail starting with Wagtail v2.13 – in older versions, this would result in two buttons in the toolbar, one of which would either break the editor when clicked, or do nothing.
So – seems pretty good to me, with the caveat that this is again relying on implementation details of existing APIs rather than something we intend to support.

Sharing more info of our use case of Draftail customisation.
One of these is using a slightly modified version of the word count:

Our contents are dual language (Chinese and English), and in Chinese, one characters equals to one word (so we need to extend the default counting via space between words)
In addition to that we also have Total Word counts! (Excluding some fields that we don't need to count) We placed it near the save button below, neat placement I would say. This was requested by our editors, since we need the total word counts for freelance writers.

Oh, and also, we managed to do a quote sign replacement! (from '' and "",  to ‘’ and “”).
Not included in this example to keep the sharing simple for now


[34249] jsoup:HtmlFuzzer: Uncaught exception in org.jsoup.parser.CharacterReader.unconsume

Labels: Restrict-View-Commit ClusterFuzz Reproducible Engine-libfuzzer OS-Linux Reported-2021-05-13 Proj-jsoup
Type: Bug
New issue 34249 by ClusterFuzz-External: jsoup:HtmlFuzzer: Uncaught exception in org.jsoup.parser.CharacterReader.unconsume
https://bugs.chromium.org/p/oss-fuzz/issues/detail?id=34249
Detailed Report: https://oss-fuzz.com/testcase?key=5267962723434496
Project: jsoup
Fuzzing Engine: libFuzzer
Fuzz Target: HtmlFuzzer
Job Type: libfuzzer_asan_jsoup
Platform Id: linux
Crash Type: Uncaught exception
Crash Address:
Crash State:
org.jsoup.parser.CharacterReader.unconsume
org.jsoup.parser.TokeniserState$11.read
org.jsoup.parser.Tokeniser.read
Sanitizer: address (ASAN)
Crash Revision: https://oss-fuzz.com/revisions?job=libfuzzer_asan_jsoup&revision=202105130607
Reproducer Testcase: https://oss-fuzz.com/download?testcase_id=5267962723434496
Issue filed automatically.
See https://google.github.io/oss-fuzz/advanced-topics/reproducing for instructions to reproduce this bug locally.
When you fix this bug, please

mention the fix revision(s).
state whether the bug was a short-lived regression or an old bug in any stable releases.
add any other useful information.
This information can help downstream consumers.

I haven't been able to reproduce this one with the provided data file. See testcase at: https://github.com/jhy/jsoup/commit/e718ddd847e15331d7d204a021b7f0aa80628a68
Driver just calls Jsoup.parse(string) - so not sure what I'm missing:
https://github.com/google/oss-fuzz/blob/master/projects/jsoup/HtmlFuzzer.java
Any idea?

@0roman thanks very much for setting up the fuzz integration. I've fixed the bulk of the issues tracked here: #1537
This one I can't replicate. I am guessing that I'm decoding the sample test case incorrectly but can't figure what I'm doing wrong. According to:
https://github.com/CodeIntelligenceTesting/jazzer#reproducing-a-bug there should be a .java main class to repro the bug, but I can't find that on the linked report.
Any suggestions?

I wasn't able to directly test this with the sample fuzz output, but was able to reconstruct the same code execution path and error with a standard unit test.
Also reviewed all 14 other uses of unconsume() and verified that these all are directly after a consume() and there's no bufferUp() potential, so those cannot trigger this underrun.


Improve default logging experience on App Engine Flex

The current idea I have is to have the following in our default nginx.conf:
location ~ ^/_ah/health$ {
  access_log off;
  error_log off;
  return 200 'ok';
}

This is defined as regex and it's intentional (see the reason below). This directive will suppress all the logs from your log in the app category.
The Flex log viewer has another log category 'nginx.health_check' and at least you can see the HTTP status there. The nginx will just return 200 'ok' so this log is probably fine.
To override this behavior, you can have a custom nginx-app.conf file in your project root:
location = /_ah/health {
}

Why this works. It works because nginx follows the rules below:
Excerpt from: https://www.linode.com/docs/websites/nginx/how-to-configure-nginx

Directives are processed in the following order:

Exact string matches are processed first. If a match is found, nginx stops searching and fulfills the request.
Remaining literal string directives are processed next. If nginx encounters a match where the ^~ argument is used, it stops here and fulfills the request. Otherwise, nginx continues to process location directives.
All location directives with regular expressions (~ and ~*) are processed. If a regular expression matches the request, nginx stops searching and fulfills the request.
If no regular expressions match, the most specific literal string match is used.

@bshaffer WDYT?

This seems great! The downside is if your application is sick, it won't return a 500 because nginx would never actually hit it. But the health check is for the health of the container and not necessarily the application.
If we document how to override it in the runtime docs, I think this is ok!

I think it's ok to say "the default health check handler just returns 200", no?

We should make it clear that the health check response does not actually touch the application layer. Something like "by default the NGINX web server returns 200 for health checks" or something along these lines.

@tmatsuo This needs to be implemented before we can go beta.


(#152) make sure function crontimes isn called, when disabling the cronjob

This addresses #152 by preventing calling function mcollective::crontimes when mcollective::facts_refresh_interval is set to 0.

weird, problem is fqdn_rand doesnt take a 0 as max.  Looks good thanks.
Do you mind updating the commit message and pr header as others in the git log?
(#ticket) description

Yep, done! Thanks!

Is there anything further from stopping this PR from being landed now the title has been updated ?

I think it’s good. I was on holiday for a week that’s the main hang up :) will get to it this week

Thanks currently building against this fork would be nice to work of master.

thank you!


Empty preds when trained on my own data

Hello! I am trying to train on my own data based on exported_model.h5. After 100 epochs i get empty preds:

My config:
train:
    dataset_builder: &ds_builder
        table_path: 'data/table.txt'
        # 1: Grayscale image, 3: RGB image
        img_channels: 3
        # The image that width greater than max img_width will be dropped.
        # Only work with image width is null.
        max_img_width: 400
        ignore_case: false
        # If it is not null, the image will be distorted.
        img_width: null
        # If change height, change the net.
        img_height: 32
    train_ann_paths:
        - 'data/dataset/annotation_train.txt'
        - 'data/dataset/annotation_val.txt'
    val_ann_paths:
        - 'data/dataset/annotation_test.txt'
    batch_size_per_replica: 256
    # The model for restore, even if the number of characters is different
    restore: 'exported_model.h5'
    learning_rate: 0.001
    # Number of epochs to train.
    epochs: 100
    # Reduce learning rate when a metric has stopped improving.
    reduce_lr:
        factor: 0.5
        patience: 5
        min_lr: 0.0001
    # Tensorboard
    tensorboard:
        histogram_freq: 1
        profile_batch: 0

eval:
    dataset_builder:
        <<: *ds_builder
    ann_paths:
        - '/datasets/ICDAR/2013/Challenge2_Test_Task3_gt.txt'
    batch_size: 1

Tensorboard:

If i run demo with exported_model.h5 i get preds. What i am doing wrong?

It seems that your model has not converged yet, the loss is high, and the accuracy is low.

Thanks for your answer. So i need to continue training with more epochs? Is it okay that model cant predict any symbols when training is based on pre-trained exported_model.h5 ?

@FLming i extend dataset, train more epochs and get some results. Thanks!
But i faced with some issues: example images from you repo was recognized incorrectly now. My dataset contains images with serial numbers on it, so it's just random sequences of a-z and 0-9. It looks like i need to turn off LSTM layers?
To do so i need to comment lines 48-52 in models.py ?
x = layers.Bidirectional(
        layers.LSTM(units=256, return_sequences=True), name='bi_lstm1')(x)
x = layers.Bidirectional(
        layers.LSTM(units=256, return_sequences=True), name='bi_lstm2')(x)
x = layers.Dense(units=num_classes, name='fc1')(x)

Is it correct?
Another one question about confidence. If i run prediction on image with symbols on another language, nn gives me some results. This results will be incorrect ofcourse because nn doesnt see such symbols before. I want to filter such predictions. Is there any change to get confidence for word of each char of word?

About the first question, according to the conclusion of the paper, removing the LSTM layer will reduce the capacity of the model, usually only reducing the performance.
The second question, it's about the decode method, the greedy method will give the predicted probability (softmax) of each character, you can do the postprocessing to get the probability of the sequence. Other methods can also give the predicted
sequence probability accordingly.

Thanks!


KeyboardAware

:tada: This issue has been resolved in version 7.25.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


missing call to Close on Fetch return value (ReadCloser)

Current coverage is 68.37%

Merging #12 into master will increase coverage by +0.08% as of e1bc73c

@@            master     #12   diff @@
======================================
  Files           17      17
  Stmts         1205    1208     +3
  Branches       192     192
  Methods          0       0
======================================
+ Hit            823     826     +3
  Partial         93      93
  Missed         289     289

Review entire Coverage Diff as of e1bc73c

Powered by Codecov. Updated on successful CI builds.

LGTM


WebApplicationTestFixture should provide a default WebHostBuilder

According to the announcement WebApplicationTestFixture tries to look for a function with the signature public static IWebHostBuilder CreateWebHostBuilder(string [] args) in the assembly where the Startup class is located.
That sounds really strange to me. My CreateWebHostBuilder method in my Program class is for use in production. I don't want my tests to be reading configuration from environment variables and would prefer to avoid writing test code in my application.
I think the WebApplicationTestFixture should provide a basic default WebHostBuilder (example below) instead of or in addition to looking in the Program class for an implementation of some method.
var builder = new WebHostBuilder()
    .ConfigureAppConfiguration((hostingContext, config) =>
        config.AddJsonFile("appsettings.json", optional: true, reloadOnChange: true))
    .UseDefaultServiceProvider((context, options) => options.ValidateScopes = true)
    .UseStartup<T>();

As it is, WebApplicationTestFixture will definately require me to override it to make it useful. With some sensible defaults, it could be made useful without doing so.
See also WebApplicationTestFixture Should Find the Project under Test

@RehanSaeed I believe the idea here is that if you have a custom scenario, you can override one of the virtual methods on WebApplicationTestFixture to suit your needs:
https://github.com/aspnet/Mvc/blob/008a562bf5b1ab89e4b83c60faaabd1d12ffcb82/src/Microsoft.AspNetCore.Mvc.Testing/WebApplicationTestFixture.cs#L122
In general the app's startup code should be aware of multiple environments and use conditional logic to enable/disable certain parts as needed. The reason the test fixture calls the default method is that in any non-trivial scenario there are many services that need to be configured, so having two copies of that code seems less than ideal.
@javiercn do you have additional thoughts on this?

@Eilon is exactly right. Things like setting up application configuration etc will not work properly with just doing new WebHostBuilder().
By default the tests reflects the config of your application as close as possible, that's the starting point. You can always override CreateWebHostBuilder and setup dependencies and anything else you need for your tests

I'm closing this as by design.
The solution is simply to extend WebApplicationTestFixture and override CreateWebHostBuilder to create the builder yourself (and depend on this class for your tests)


Windows Does Not Recognize this device after modifying firmware

Hi, I went through all of the steps to configure my bad USB and before changing the firmware, I confirmed that my flash drive had the correct controller, yet now windows doesn't recognise the device at all. It tries for a while, then it gives up and the data light on my USB hub turns off. I tried it in my main PC USB port and it does nothing too. I have a few screenshots of what I found in device manager.

Device not migrated -
Device USB\VID_13FE&PID_5201\7&504b82f&0&1 was not migrated due to partial or ambiguous match.
Last Device Instance Id: USB\VID_04F2&PID_B40E\6&3455550a&0&5
Class Guid: {36fc9e60-c465-11cf-8056-444553540000}
Location Path:
Migration Rank: 0xF000FFFF0000F120
Present: false
Status: 0xC0000719
Device configured (usb.inf) -
Device USB\VID_13FE&PID_5201\7&504b82f&0&1 was configured.
Driver Name: usb.inf
Class Guid: {36fc9e60-c465-11cf-8056-444553540000}
Driver Date: 06/21/2006
Driver Version: 10.0.19041.488
Driver Provider: Microsoft
Driver Section: Composite.Dev.NT
Driver Rank: 0xFF2003
Matching Device Id: USB\COMPOSITE
Outranked Drivers:
Device Updated: false
Parent Device: USB\VID_2109&PID_2812\6&8edca8f&0&3
Device not started -
Device USB\VID_13FE&PID_5201\7&504b82f&0&1 had a problem starting.
Driver Name: usb.inf
Class Guid: {36fc9e60-c465-11cf-8056-444553540000}
Service: usbccgp
Lower Filters:
Upper Filters:
Problem: 0xA
Problem Status: 0xC0000001

I have tried several times to restore the original (dumped) firmware and I keep getting this error (my guess is that the device isn't recognised by the app since windows doesn't recognise it) -
./DriveCom.exe /drive=D /action=SendFirmware /burner=C:\Users\User\Downloads\BadUSB\Burner_Image/BN03V104M.BIN /firmware=C:\Users\User\Downloads\Psychson\originalfw.bin
Action specified: SendFirmware
Gathering information...
FATAL: System.InvalidOperationException: DeviceIoControl failed: 0006
at DriveCom.PhisonDevice._SendCommand(SafeFileHandle handle, Byte[] cmd, Byte[] data, Int32 bytesExpected) in C:\Users\User\Downloads\Psychson-master\DriveCom\DriveCom\PhisonDevice.cs:line 365
at DriveCom.PhisonDevice.SendCommand(Byte[] cmd, Int32 bytesExpected) in C:\Users\User\Downloads\Psychson-master\DriveCom\DriveCom\PhisonDevice.cs:line 295
at DriveCom.PhisonDevice.RequestVendorInfo() in C:\Users\User\Downloads\Psychson-master\DriveCom\DriveCom\PhisonDevice.cs:line 140
at DriveCom.PhisonDevice.GetChipType() in C:\Users\User\Downloads\Psychson-master\DriveCom\DriveCom\PhisonDevice.cs:line 170
at DriveCom.Startup._GetInfo() in C:\Users\User\Downloads\Psychson-master\DriveCom\DriveCom\Startup.cs:line 384
at DriveCom.Startup._SendFirmware() in C:\Users\User\Downloads\Psychson-master\DriveCom\DriveCom\Startup.cs:line 365
at DriveCom.Startup.Main(String[] args) in C:\Users\User\Downloads\Psychson-master\DriveCom\DriveCom\Startup.cs:line 114
I have included as much detail as possible and I hope it can help as to figure out why I'm unable to use my rubber ducky. Thanks in advance :D

I've found an old Samsung Galaxy S5 with an OTG micro USB port that I can use so I no longer have use for these phison 2303 USBs (and I probably bricked mine anyway).


option for allowing pre-extension

For example .spec.ts in test.spec.ts should be ignored. currently it ignores only .ts.

#15 Ignore all pre-extensions and it's good enough for 0.1.0.


Correcting YAML syntax for branch names

Also, this is reason<PHONE_NUMBER>437 for YAML being a shitty configuration format.
Fitting quote:

The story so far:
In the beginning the Universe was created.
This has made a lot of people very angry and been widely regarded as a bad move.

:+1:

this looks like something which should be added to the validator script to be automated, no ?

@stof not sure if it can be automated, as it fails in the Yaml component itself. Maybe an addition to the YAML stuff?

@Ocramius A solution could be to enforce branches not to have numeric names in the validator (the branch is generally set as 1.x or 2.4.x for versionned branches). This would solve it as this would parse as a Yaml string.
What we could do in the Yaml component to avoid loosing float keys is to cast them to string before adding them to the array instead of letting PHP cast it (which will cast to int)

What we could do in the Yaml component to avoid loosing float keys

The actual bug, I suppose. Can you report it, if you got time? Currently being kicked out...

Reported in symfony/symfony#15667


Not getting direct messages

When connecting via IRC, I am able to send direct messages to people without problems, however I am not getting their responses (I have checked several IRC clients: IRCcloud, Textual5).
[update]: Channels (public) work fine, but not receiving direct messages.

Just to note, you don't join one to ones like normal rooms. Instead /msg foouser
I can send and receive messages in a one to one room via IRC but it seems to only work for a one to one I have already done this with on IRC previously.
I suspect this is an issue with node-gitter's room.join and will need to wait for https://github.com/gitterHQ/docs/pull/41

Hey @chriskacerguis,
I looked into this a bit more and I don't think room.join is the issue.
What is your default notification setting set to? I suspect you are falling in this trap https://github.com/gitterHQ/gitter/issues/1275
I checked the user I was reproducing this issue with and it was set to Announcements which won't send out any user_notification for a normal message. I am pretty sure @suprememoocow did some work around this by always sending a notification for a one to one but those changes seemed to have never worked (or never worked from my understanding of it), see https://github.com/troupe/gitter-webapp/pull/1753

@MadLittleMods Can you confirm that link to troupe/gitter-webapp it returns 404.  Also, I'm set to mute with Announcement on a few 1:1 conversations. I'll change it tomorrow and let you know.
That said, what is the status of the different notifications settings for 1:1 vs rooms, which was asked https://github.com/gitterHQ/gitter/issues/1275

@chriskacerguis It's a link to an internal project, you won't be able to see it.
Status would be on that issue. Actually separate settings for one-to-ones probably won't be addressed in the near future.

@MadLittleMods So, with the person that I was testing with on Gitter (and not getting their messages) I was set to "Announcement", as soon as I switched it to "All" it started to work.


Deprecation request

Hi! I was hoping you might be able to do a deprecation of the ember-cli-pusher module as the main repo is now an Ember module....
npm deprecate ember-cli-pusher
I was also hoping you could unlist the bower module since you've taken the real name of the package ember-pusher....
https://github.com/bower/bower/issues/120
curl -X DELETE "https://bower.herokuapp.com/packages/ember-pusher?access_token=TOKEN"
where TOKEN is a github personal access token.
If you need a hand with anything, let me know!

Hi. It's done. Just in case, I added you as a maintainer of ember-cli-pusher npm package. I personally don't use it anymore, but some people might do. So feel free to do anything with the npm package to make the transition to ember-pusher for them better.

Thanks Ivan!


Distribution parameters

TFP Normal uses NumPy's loc and scale.
PyMC3 used mu and sigma (or alternatively tau or sd).
What are our thoughts on parametrization of distributions? Follow TFP only, or provide the alternative parametrizations as well?
Speaking for myself here, I believe that it's better for UX to keep PyMC3 parametrizations - it means fewer things to change API-wise, for an end-user.

We settled this to use greek symbols like in PyMC3.

*greek names. Greek unicode symbols would make things super challenging! :smile:


A new app should have at least 1 qr template

list index out of range
Traceback (most recent call last):
  File "/base/data/home/runtimes/python27_experiment/python27_lib/versions/third_party/webapp2-2.3/webapp2.py", line 1511, in __call__
    rv = self.handle_exception(request, response, e)
  File "/base/data/home/runtimes/python27_experiment/python27_lib/versions/third_party/webapp2-2.3/webapp2.py", line 1505, in __call__
    rv = self.router.dispatch(request, response)
  File "/base/data/home/runtimes/python27_experiment/python27_lib/versions/third_party/webapp2-2.3/webapp2.py", line 1253, in default_dispatcher
    return route.handler_adapter(request, response)
  File "/base/data/home/runtimes/python27_experiment/python27_lib/versions/third_party/webapp2-2.3/webapp2.py", line 1077, in __call__
    return handler.dispatch()
  File "/base/data/home/runtimes/python27_experiment/python27_lib/versions/third_party/webapp2-2.3/webapp2.py", line 547, in dispatch
    return self.handle_exception(e, self.app.debug)
  File "/base/data/home/runtimes/python27_experiment/python27_lib/versions/third_party/webapp2-2.3/webapp2.py", line 545, in dispatch
    return method(*args, **kwargs)
  File "/base/data/home/apps/s~mobicagecloudhr/51.401199075546206538/rogerthat/pages/service_interact.py", line 59, in get
    qrtemplate, color = get_default_qr_template_by_app_id(si.app_id)
  File "/base/data/home/apps/s~mobicagecloudhr/51.401199075546206538/mcfw/rpc.py", line 164, in typechecked_return
    result = f(*args, **kwargs)
  File "/base/data/home/apps/s~mobicagecloudhr/51.401199075546206538/mcfw/rpc.py", line 142, in typechecked_f
    return f(**kwargs)
  File "/base/data/home/apps/s~mobicagecloudhr/51.401199075546206538/rogerthat/bizz/service/__init__.py", line 571, in get_default_qr_template_by_app_id
    qr_template = QRTemplate.get_by_key_name(app.qrtemplate_keys[0])
IndexError: list index out of range

Create a default QR code template when creating an app

Wait until this is properly spec'ed

Use this one for all apps except for city apps: https://github.com/rogerthat-platform/rogerthat-backend/blob/master/src/rogerthat/bizz/qr-hand-only.png
Use this one for city apps: https://github.com/rogerthat-platform/rogerthat-backend/blob/master/src/rogerthat/bizz/qr-brand-oca.png

Merged on staging branch


Close #1684: Adapting to new stat architecture

Require: https://github.com/Terracotta-OSS/terracotta-platform/pull/259
See #1684 and https://github.com/Terracotta-OSS/terracotta-platform/issues/217 and email sent to you guys

Note: waiting for release 5.2.0-pre4 of tc-platform to be available

@henri-tremblay @ljacomet : I ran a complete ./gradlew clean build with the new release 5.2.0-pre5 of tc-platform and build pass :-)
We just need to wait for the new release to be deployed then.

test this please


Fix Jaeger format spanId to Wavefront spanId conversion

Fix issue where high 64bits of spanId are set to 0 while converting to UUID format
Add unit tests for JaegerWavefrontPropagator propagator

@anilkun Can you please revert the delete of the codeStyleSettings file? Also we follow a different style guideline. In the future please format using the provided codeStyleSettings file.


UpgradeError Not yet support for changing primary Key -- with 3.0.0-rc.7

Hi,
Sorry I haven't log because this is a user remark.
I migrated from : 3.0.0-rc.2 to 3.0.0-rc.7
And now, a user sent me a mail with the problem :
OpenFailedError;UpgradeError Not yet support for changing primary Key
Is it not corrected in 3.0.0-rc.7 ??
Thanks,
Jonathan

Yea #1004 and was resolved in rc.6.
There might be another issue. Would it be possible to get a repro?

Tried to trigger this bug with various browsers and configs but the only way I've been able to do it, is by changing the actual primary key, such as renaming its property, or changing auto-incrementation flag. It seems to work as expected for both inbound and outbound keys.
Could it be that your user for some reason had a very old version that was not compatible? Or had a another database with the same name at the same origin? Just trying to figure out possible reason. Dexie prior to 3.0.0-rc.3 would not detect this problem but pretend everything is normal until you try a request on the primary index.

First of all : I will ask the user the "exact" configuration of his system (browser and version).
Then, I will try to test to reproduce.
If I can't... Then I will "wait" another user to have the problem (because for the user, which encounter the problem I ask him to use a drop database function I wrote in my app, which drop and reload local from server -> after : everything OK).
Dexie prior to 3.0.0-rc.3 would not detect this problem but pretend everything is normal until you try a request on the primary index.
==> Maybe I've always had the problem without catching it properly ;)

Encountered this error today!
In my case, I am removing the auto-increment from primary key. Because I am converting the PK to UUID.
Is this type of upgrade not supported?

Autoincrement is only a fallback. It's still allowed to set UUIDs as primary keys on autoincremented tables. So you could basically leave it as autoincremented if you just keep the same name of the property of the primary key.
If that feels odd or you must change which property to use as primary key, there's an example here:
You need to do it in 3 versions with upgraders on each: https://github.com/dfahlander/Dexie.js/issues/781#issuecomment-443237897

Update: I've actually refactored my code as per your suggestion here  i.e. without dropping auto-increment flag
and its working so far...

PS: Thank you for all your work!

Really confused by this - is it simpler if I am willing to delete the table in an upgrade? Had no idea something simple like this would require anything more than simply changing the schema and updating the version. Seems like dexie.js should handle this process behind the scenes rather than users having to implement the abomination mentioned here and elsewhere?

Really confused by this - is it simpler if I am willing to delete the table in an upgrade? Had no idea something simple like this would require anything more than simply changing the schema and updating the version. Seems like dexie.js should handle this process behind the scenes rather than users having to implement the abomination mentioned here and elsewhere?

The process of changing primary key could be improved and there are lots of ideas around this. It's not supported natively though and there are a few unanswered questions on how what the developer would expect from changing the primary key property:

Should the database fail to open if there are one or more existing rows that lacks the new primary key?
Should the database fail to open if there are two rows containing the same value for primary key?
Would it be expected by dexie to copy the old prop into the new or would that be the responsibility of the developer to add an upgrader to do this?

Good questions - do you know what MongoDB and other popular databases do in those cases? IIRC MongoDB:

Uses 'undefined' as the index value if it's missing in a doc/row
Fails when trying to create a new unique index if there's a key collision for that property

Not sure what you mean by point 3 - isn't the dev just specifying changes in indexes/uniqueness? Why would that involve any copying of values?

Any chance that Dexie.js v4 will have the ability to change primary key? This would be really helpful. I've only just started using Dexie in the last couple of months and I've hit into this problem 3 times now.

Hello, I'd like to necro this and ask what is the support for changing primary keys. The last messages are from 10 months ago.
Thanks


Remove unneeded ui

dps update rate no longer makes sense given that the miniparse event
source is sending dps updates.
Additionally, as logs aren't sent to anything that don't need it,
there's no need for a log-related checkbox either.

This is an update for https://github.com/quisquous/cactbot/pull/577.


Issue #1176 assign wells with point filters

Fixes #1176
Description

Assign wells with point filters (filters with a zero length) to layers. Before, these would be removed in the compute_overlap method.
Add test case for wells with a point filter
Set validation so that screen_top can equal screen_bottom
Rename vague variable name "df" to "df_factor"

Checklist

[x] Links to correct issue
[x] Update changelog, if changes affect users
[x] PR title starts with Issue #nr, e.g. Issue #737
[x] Unit tests were added
[ ] If feature added: Added/extended example

The request for type hints lead to some extra required type annotations to be implemented for MyPy. A no-regret, but the slight downside that this changeset now influences more places of the code.


[V3] TypeError: this.commands.refreshToken is not a function

on my old iPhone 4s
Version: 9.3.6 (13G37)
I get the following error message when I try to request the token.
i confused why the file NotificationsAndroid.js is in this stack

System:
    OS: macOS 10.15.3
    CPU: (4) x64 Intel(R) Core(TM) i5-6267U CPU @ 2.90GHz
    Memory: 639.40 MB / 16.00 GB
    Shell: 5.7.1 - /bin/zsh
  Binaries:
    Node: 10.16.0 - /usr/local/bin/node
    Yarn: 1.22.0 - /usr/local/bin/yarn
    npm: 6.9.0 - ~/.nvm/versions/node/v10.16.0/bin/npm
    Watchman: 4.9.0 - /usr/local/bin/watchman
  SDKs:
    iOS SDK:
      Platforms: iOS 13.2, DriverKit 19.0, macOS 10.15, tvOS 13.2, watchOS 6.1
    Android SDK:
      API Levels: 22, 23, 25, 26, 27, 28
      Build Tools: 23.0.1, 25.0.0, 25.0.1, 25.0.2, 26.0.0, 26.0.1, 26.0.2, 26.0.3, 27.0.1, 27.0.3, 28.0.3
      System Images: android-26 | Google Play Intel x86 Atom, android-28 | Google Play Intel x86 Atom, android-29 | Google Play Intel x86 Atom
  IDEs:
    Android Studio: 3.5 AI-191.8<IP_ADDRESS>10548
    Xcode: 11.3.1/11C504 - /usr/bin/xcodebuild
  npmPackages:
    react-native: 0.61.4 => 0.61.4
  npmGlobalPackages:
    react-native-cli: 2.0.1

react-native-notifications: 3.1.2

also happens on iPod Touch 5, 16GB, Modell MGG82FD/A mit iOS 9.3.5.

Issue still exists! 😢


make builtin directories a cycle

enable/disable stuffs for program/apps and program/apps/appvars, maybe also a toggle to disable entirely?

can you just tell me exactly what to do lol

Put the Apps directory in Programs, put Appvars in Apps and put Programs in Appvars
Have a radio button to enable/disable the Appvar dir, and maybe one for Apps.

I assume you meant Programs in Programs, not in Appvars.

Added with ba4292d


Add from typing import * to boilerplate

I imagine that imports were left out to avoid giving clues, but it would be nice to have all the type hints imported into the namespace.

I tend to not add it, given import * is discouraged in the first place. Also writing the import is just a few characters typing..

Plus, there are things you don't need to import, like tuple, dict, list, etc; and things are better imported from collections.abc instead of typing.


[WIP] Setup karma unit tests

This PR sets up karma for executing renderer process tests in electron and adds one test file for testing spellchecker, refactors tests dir into tests/e2e & tests/unit.
It picks up parts of #350 and improves upon them.
Address #82

@priyankp10  @akashnimare Can you guys give this a test run on macOS?

@abhigyank can you rebase this so that I can review?

@akashnimare rebased.

@abhigyank is this ready to review?

I'll test it once and update you.

I think currently this may not work due to failure in the spectron package as we had discussed in the retreat.

@priyank-p updated the PR.
npm run test-e2e works as expected, npm run test-unit works, but would prefer someone to test it on macOS. If it doesn't we can remove the spellchecker unit test and keep the framework as unit test framework is working correctly.

@abhigyank can add this to .travis.yml so these tests run in CI.

@priyank-p can do a test on this locally? Both test works for me on my local, but seems to fail on the travis-ci. I am unable to understand the occurrence of the error on travis-ci, can you give a look?

Isn't the spectron tests failing in the CI? I think that is just because of travis' hardware being slow so the tests are timing out. We should just get rid of them by porting them to karma or disable them just like before in CI.

Removed the e2e spectron test from travis (passes locally).
unit-test passes on macOS, on linux no unit test is present so fails.So, for now running unit-tests on macOS only in travis.

@akashnimare I think its ready to merge.
@priyank-p give both the tests a try .


Discovery Bluetooth (not tested) and the ALL Printers

Upgraded the .jar and so files to the newest that i found.
Implemented the Bluetooth and all devices discovery ( I don't have any bluetooth but it's the same method)
Refact the onDiscoveryPrinter , now there is only one method to find printers and i put the device in parameter
Fix some null errors in example project

Thanks for your package it's SIMPLY AWESOME!

wow, honestly this package I not maintenance about 1 year but you make me want to improve that. Thanks so much @brasizza , I appreciated that.

@brasizza do you have any plans to reopen this PR? These changes are great and would be very useful to the package.

i have but there is a lot of breakchange that i need to arrange it.
One thing that i fix is to use this package into x86 android devices as well

No worries, if there's anything you'd like help with / tested I am available :)

@TrentStarky  try mine
if its ok i can do a pull request again
https://github.com/brasizza/flutter_epson_epos


The official TodoMVC example is still using imperative state updates?

This line with imperative state updates makes me puzzled to see it in the Redux architecture
https://github.com/reactjs/redux/blob/master/examples/todomvc/src/components/TodoTextInput.js#L22
      if (this.props.newTodo) {
        this.setState({ text: '' })
      }

This seems to be exactly the style making functions impure and harder to test, that Redux is aimed to discourage or better ban altogether.
Was it an oversight to leave it like that?

It's intentional. In many cases it's easier to use React state for small isolated updates.
http://redux.js.org/docs/faq/OrganizingState.html#organizing-state-only-redux-state

@gaearon
Thank you very much for quickly commenting on that
and I can see where your reasoning come from,
but isn't the real reason why it is easier
is due to the enforced global state model?
If the local state management were supported,
you could just write a small reducer function next to the component
with its own private action,
making your component completely pure and testable?
The problem I see here is not so much for this particular example,
where things are indeed simple enough this way because the component itself is simple,
but once you try to imitate it in the more complex advanced cases,
where the imperative style will increasingly begin to "bite",
pushing you into more complex but still imperative-style-based architecture,
until and at some stage, you would have to throw away much of it
and move state to Redux, which could be considerably harder at that later stage.
It feels like a fundamental problem of co-existing of two different architecture patterns,
competing with each other to solve the same problem,
which inevitably creates additional complexity,
and the painful question which way you implement this and which way that,
will "haunt" every piece of the app :)
Whereas, if local state was allowed by Redux, it could have been easy the other way around --
have pure functional uniform structure instead of the imperative one mixed in here and there.
Here is an example of how it may look:
https://github.com/dmitriz/un/blob/master/examples/basic/submit-with-reset/index.js#L5
It uses streams but the architecture is very close to redux,
with the additional flexibility of local reducers.
I can see there is a long discussion with many people expressing their reasons
to depart from the monolith single state model that is even linked from the page in your link :),
but no complete answer to justify the current situation seems provided unless I missed it: https://github.com/reactjs/redux/issues/1385


OSM Compass is Disappeared

Issue Type
[ ] Question
[ ] Bug
Description and/or steps/code to reproduce the problem
I have a problem regarding to the compass usage in OSM, previously i was implement it and it worked, but the compass is not shown recently, question is it known issue? or there are some of my code that need to be adjusted if any changes related with dependency or else. Below is sample code how i implement it. @spyhunter99
CompassOverlay co = new CompassOverlay(context, osmMapView);
co.enableCompass();
osmMapView.getOverlays().add(co);
Environment
If it's a bug, version(s) of android this affects:
All
Version of osmdroid the issue relates to:
4.2

Even I am facing the same issue. Compass overlay is working on my old device(Moto x play). The same code is not displaying the compass in the new sevice (Moto G5s plus). Is it because of anychange in the orientation sensor? I am using osmdroid distribution 6.0.1.

are you sure it is running in your old device? because i also try it using 4.1.2 OS and still same, no compass displayed.

@moonlotus The Moto G5s Plus doesn't seem to have a magnetometer sensor, see https://forums.lenovo.com/t5/Moto-G5S-Moto-G5S-Plus/Moto-G5s-Plus-compass/td-p/3769776

As stated by @secaw, if you don't have a sensor the compass won't be displayed (CompassOverlay.mAzimuth == Float.NaN)
Am I dreaming or there used to be in osmdroid a "map orientation compass" similar to Google Map's? If not, shouldn't we implement it: a compass that follows the map orientation, and that restores the north on top when clicked upon? Of course that would be part of another issue, like a new feature issue.

Thank you for @secaw and @monsieurtanuki , i was found the root cause, and it's already solved. Inside InternalCompassOrientationProvider.class , it still use Sensor.TYPE_ORIENTATION as a sensor trigger to show/hide the compass which is already deprecated. So, the compass will be defined such as like this :
CompasOverlay compass = new CompassOverlay(context, new AdjustCompassOrientation(context), osm);
//AdjustCompassOrientation is a class that implement SensorEventListener, IOrientationProvider

u should use  getApplicationContext() instead of using context .

Hello to all,
Due to the fact that according to some users of my app, the problem with the missing compass rose seems to persist on many Xiaomi (Redmi and Poco) phones in 2024, I modified the InternalCompassOrientationProvider slightly.
I don't know if this is still useful after such a long time, but with the integrated fallbacks, the compass rose is now also displayed on these phones and the compass function works.
The entire code is attached (zip file).
public boolean startOrientationProvider(IOrientationConsumer orientationConsumer) {
    mOrientationConsumer = orientationConsumer;
    boolean result = false;

    // first old OSMDroid handling
    final Sensor sensor = mSensorManager.getDefaultSensor(Sensor.TYPE_ORIENTATION);
    if (sensor != null)
        result = mSensorManager.registerListener(this, sensor, SensorManager.SENSOR_DELAY_UI);
    else // now the fallbacks
    {
        final Sensor rotationsensor = mSensorManager.getDefaultSensor(Sensor.TYPE_ROTATION_VECTOR);
        if (rotationsensor != null)
            result = mSensorManager.registerListener(this, rotationsensor, SensorManager.SENSOR_DELAY_UI);
        else {
            final Sensor accelerometer = mSensorManager.getDefaultSensor(Sensor.TYPE_ACCELEROMETER);
            final Sensor magnetometer = mSensorManager.getDefaultSensor(Sensor.TYPE_MAGNETIC_FIELD);
            if (accelerometer != null && magnetometer != null)
                result = mSensorManager.registerListener(this, accelerometer, SensorManager.SENSOR_DELAY_UI) &&
                        mSensorManager.registerListener(this, magnetometer, SensorManager.SENSOR_DELAY_UI);
        }
    }
    return result;
}

...
//
// SensorEventListener
//

@Override
public void onAccuracyChanged(final Sensor sensor, final int accuracy) {
    // This is not interesting for us at the moment
}

private static final float ALPHA = 0.2F; //1 / 16F;//adjust sensitivity

private float[] LPF(float[] input, float[] output) {
    if (output == null) return input;
    for (int i = 0; i < input.length; i++) {
        output[i] = output[i] + ALPHA * (input[i] - output[i]);
    }
    return output;
}

private float[] mMatrixR = new float[9];
private float[] mMatrixValues = new float[3];
private float[] gravity;
private float[] geomagnetic;

@Override
public void onSensorChanged(final SensorEvent event) {
    // first old OSMDroid handling
    if (event.sensor.getType() == Sensor.TYPE_ORIENTATION) {
        if (event.values != null) {
            mAzimuth = event.values[0];
            if (mOrientationConsumer != null)
                mOrientationConsumer.onOrientationChanged(mAzimuth, this);
        }
    } else // now the fallbacks
    {
        if (event.sensor.getType() == Sensor.TYPE_ROTATION_VECTOR) {
            // Get rotation matrix
            SensorManager.getRotationMatrixFromVector(mMatrixR, event.values);
            SensorManager.getOrientation(mMatrixR, mMatrixValues);
            mAzimuth = (float) Math.toDegrees(mMatrixValues[0]);
            if (mOrientationConsumer != null)
                mOrientationConsumer.onOrientationChanged(mAzimuth, this);
        } else {
            if (event.sensor.getType() == Sensor.TYPE_ACCELEROMETER)
                // gravity = applyLowPassFilter(event.values.clone(), gravity);
                gravity = LPF(event.values.clone(), gravity);
            if (event.sensor.getType() == Sensor.TYPE_MAGNETIC_FIELD)
                // geomagnetic = applyLowPassFilter(event.values.clone(), geomagnetic);
                geomagnetic = LPF(event.values.clone(), geomagnetic);
            if (gravity != null && geomagnetic != null) {
                float R[] = new float[9];
                float I[] = new float[9];
                boolean success = SensorManager.getRotationMatrix(R, I, gravity, geomagnetic);
                if (success) {
                    float orientation[] = new float[3];
                    SensorManager.getOrientation(R, orientation);
                    mAzimuth = (float) Math.toDegrees(orientation[0]);
                    if (mOrientationConsumer != null)
                        mOrientationConsumer.onOrientationChanged(mAzimuth, this);
                }
            }
        }
    }
}

MyInternalCompassOrientationProvider.zip


Replacing WindowDialog with another node leaves behind "x" button

godot 2.0.1 stable
If you create a WindowDialog node then replace it with any other node, the new node will keep drawing the "x" button that the windowdialog created. It seems to be anchored to the new node as well.

I can confirm this on 15d1fca.


Fix missing icons and colorpicker when mounted on suburl

icons and colorpicker are missing when mounted on a suburl. This PR fixes this.
Signed-off-by: Andrew Thornton<EMAIL_ADDRESS>

The global flag is necessary
From try.gitea.io/index.css unminified:
@font-face {
    font-family: Icons;
    src: url(themes/default/assets/fonts/icons.eot);
    src: url(themes/default/assets/fonts/icons.eot?#iefix) format("embedded-opentype"),url(/fonts/icons.woff2) format("woff2"),url(themes/default/assets/fonts/icons.woff) format("woff"),url(themes/default/assets/fonts/icons.ttf) format("truetype"),url(themes/default/assets/fonts/icons.svg#icons) format("svg");
    font-style: normal;
    font-weight: 400;
    font-variant: normal;
    text-decoration:inherit;text-transform:none}

I think we even need to do the eot ones too

I think we even need to do the eot ones too

Modern browsers (e.g. those we support) should not hit on EOT/SVG fonts but prefer WOFF.

The global flag is necessary

Why? global flag is pointless for simple regex.test which I think is what webpack would perform. It does not matter if there is more than 1 match.

you're right. Strangely I was sure that the g did the thing I wanted it to do.
Sorry.

please send backport

I don't think this actually does need a backport


Why test.error() ?

t.ok(value, msg)
Assert that value is truthy with an optional description message msg.

t.error(err, msg)
Assert that err is falsy. If err is non-falsy, use its err.message as the description message.

Is really t.error() a proper name? IMHO it should be t.noError() or whatever but t.error().

There are a lot of aliases in tape that are confusing.
It's actually t.ifError(err, msg)

Even the ifError alias is confusing. It seems that is waiting that to show the message if there's an error and is doing the opposite. t.checkNoError would be better or t.noError, like @ibc said, would be nice.

The API matches that of assert and as such is chained to it, convenience not withstanding.

ok is tied to _assert(value,
notOK is tied to _assert(!value,
error is tied to _assert(!err,
I agree with @ibc , error is the wrong name and it should be noError.

The various forms of ifError make no sense to me either.


Authentication error (401)

Hi everyone, recently I downloaded the unity sample to test the project and when I press the button for continuous recognition I get this error:
CANCELED: ErrorDetails=WebSocket upgrade failed: Authentication error (401). Please check subscription information and region name. SessionId: 8c9bb8ab475a483083382edb6d7f8073
UnityEngine.Debug:LogFormat(String, Object[])
i follow the steps of the creation of the keys to use the sdk and still didnt work.
im using Microsoft.CognitiveServices.Speech.1.15.0
My operating system is on windows x64 on c#
Ill add some pictures of the project and the errors

Thanks All!

@jhakulin Please check.
@Mallo-Lucas I removed one of your screenshots because you probably don't want to share your subscription key in public...

@Mallo-Lucas Have you checked your subscription key and region are valid?  Unity sample should work for that purpose as well, however if this is your first try of Speech SDK, you may want to try with simpler .NET version first, e.g.
https://github.com/Azure-Samples/cognitive-services-speech-sdk/tree/master/quickstart/csharp/dotnet/from-microphone
Let us know if you have any problems with that.

thanks for reply, how i know if there are correct? i mean i just create the resource of the picture and choose southbrasil becouse is near where i live

@Mallo-Lucas I would like you to try with some other sample, like the one I have pointed out below to figure out is there a problem with the Unity sample or with your keys. Let us know if the sample below works with the key and region you have.
https://github.com/Azure-Samples/cognitive-services-speech-sdk/tree/master/quickstart/csharp/dotnet/from-microphone

Hi, in the end I realized that the error was that I chose the wrong azure service, I had to choose cognitive services and not the speech service, thanks also for the help.


Provide fixed size for rich text plugin forms

Rich text plugin forms currently change size based on their content, sometimes during use:

A default form size should be provided, potentially configurable through the editor component configuration object.
Requirements

Provide a default static width for rich text editor plugin forms
Allow configuration by accepting a css length string in the editor component configuration object

Fixed in 1.0 🎉
Plugin forms are now inline blocks that exist as a part of the content flow in the markdown editor.


Executing a method onSelect

Is it possible to execute a method after the user makes a selection?
For example:
<ng-select *ngIf="data" [allowClear]="false"
              [options]="data"
			  (selected)="selected($event)"
              placeholder="Choose data"
			  >
  </ng-select>

the $event is the object value and label
An example use will be when someone needs to select an user from the select box and execute a method to load the user data from an API once the user has chosen from the box

Thanks for raising this missing feature @theunreal, it was on my list for some time already, but I didn't have much time to spend on this project lately.
I just added output events opened, closed, selected, and deselected (see events documentation). The selected event is exactly the one you asked for, it provides you with the selected item (object with value and label).


Added a time chart as Sign-in Trend over Time

Fixes #
Proposed Changes
Added this time chart to display Sign-in Trend over Time.
This has a TimeBrush enabled for the rest of the widgets to be filtered based on that.

@v-rucdu - please let me know for any updates needed here.

Request @v-rucdu & @liatlishams for your review.
Let me know for any further queries.
Thank you.

Please add workbookMetadata.json file. Thanks

Please add workbookMetadata.json file. Thanks

@NikTripathi done.

. Where can I set it?

@NikTripathi  - Thank you for sharing this. Request you to have a look

If the SigninLogs & AADNonInteractiveUserSignInLogs are available in the test environment.

SigninLogs
| summarize Time = max(TimeGenerated)
| where isnotempty(Time)

AADNonInteractiveUserSignInLogs
| summarize Time = max(TimeGenerated)
| where isnotempty(Time)

If existing Azure AD Sign-in logs workbook render any data.
As can see the current workbook loads where SigninLogs & AADNonInteractiveUserSignInLogs are present

@liatlishams  - Request you to review this.

Thank you @v-rucdu for approving this.


Fixed .clone.meta file deserialization bug

Got error when running cast run --tweak
cast run --tweak UniswapV3 0x2cc0341cbd7b7ce5d72a488079eace1773b583d951a9b6d371c74c6fd4bf72d4

Error:
failed to load tweak project from path: "~/pocs/UniswapV3"

Context:
- missing field `target_contract` at line 1 column 7266

Solution
forge clone used camelCase to store the fields in .clone.meta
https://github.com/MEDGA-eth/foundry/blob/3f87f29083febd31d6d22dec3b0b71949655cbf7/crates/forge/bin/cmd/clone.rs#L30-L50
So cast run --tweak should also use camelCase to read the .clone.meta file accordingly
https://github.com/0xstonegm/foundry/blob/6add3ce1a241cb11ac1e6d27707df6f8db4ae901/crates/tweak/src/metadata.rs#L157
#[derive(Debug, Clone, Default, serde::Deserialize)]
+ #[serde(rename_all = "camelCase")]
pub struct CloneMetadata {
    /// The path to the source file that contains the contract declaration.
    /// The path is relative to the root directory of the project.

@ZhangZhuoSJTU tried running cargo test in the crates/tweak all tests passed:
running 7 tests
test compatibility::tests::test_storage_layout_additional ... ok
test compatibility::tests::test_same_storage_layout ... ok
test compatibility::tests::test_storage_layout_shift ... ok
test compatibility::tests::test_storage_layout_different_type ... ok
test compatibility::tests::test_storage_layout_struct ... ok
test code::tests::test_direct_deployment ... ok
test code::tests::test_factory_deployment ... ok

test result: ok. 7 passed; 0 failed; 0 ignored; 0 measured; 0 filtered out; finished in 7.41s

   Doc-tests foundry_tweak

running 0 tests

test result: ok. 0 passed; 0 failed; 0 ignored; 0 measured; 0 filtered out; finished in 0.00s

i also tried running all tests by just clone the repo (https://github.com/MEDGA-eth/foundry), without this fix or any modifications:
failures:

---- anvil_api::can_get_metadata_on_fork stdout ----
thread 'anvil_api::can_get_metadata_on_fork' panicked at crates/anvil/src/config.rs:1061:57:
Failed to get fork block number: Transport(Custom(reqwest::Error { kind: Request, url: Url { scheme: "https", cannot_be_a_base: false, username: "", password: None, host: Some(Domain("bsc-dataseed.binance.org")), port: None, path: "/", query: None, fragment: None }, source: TimedOut }))

---- anvil_api::metadata_changes_on_reset stdout ----
thread 'anvil_api::metadata_changes_on_reset' panicked at crates/anvil/src/config.rs:1061:57:
Failed to get fork block number: Transport(Custom(reqwest::Error { kind: Request, url: Url { scheme: "https", cannot_be_a_base: false, username: "", password: None, host: Some(Domain("bsc-dataseed.binance.org")), port: None, path: "/", query: None, fragment: None }, source: TimedOut }))

---- anvil_api::test_can_set_storage_bsc_fork stdout ----
thread 'anvil_api::test_can_set_storage_bsc_fork' panicked at crates/anvil/src/config.rs:1061:57:
Failed to get fork block number: Transport(Custom(reqwest::Error { kind: Request, url: Url { scheme: "https", cannot_be_a_base: false, username: "", password: None, host: Some(Domain("bsc-dataseed.binance.org")), port: None, path: "/", query: None, fragment: None }, source: TimedOut }))

failures:
    anvil_api::can_get_metadata_on_fork
    anvil_api::metadata_changes_on_reset
    anvil_api::test_can_set_storage_bsc_fork

test result: FAILED. 179 passed; 3 failed; 0 ignored; 0 measured; 0 filtered out; finished in 48.64s

it seems like bsc-dataseed.binance.org rpc endpoint issue but still not related with the errors in CI


Smaller flushes

Why :
stabilizing write performance.
slight increase in write performance due to stabilization
What :
reduce the amount of immutable memtables per flush
flushes will be in smaller steps that finishes faster allowing the proper write delay configuration to maintain more stable performance
Who :
write intensive workload that suffer from gittery write delays

looking at different approaches, abandoned for now


33 unfreed memory blocks : 2922

Hi
WHen launching the tomboy-ng from the command line, then choosing "quit" from the menu, the app complains of objects not beeing freed
FreeandNil() missing ?
Heap dump by heaptrc unit
27664 memory blocks allocated : 3253938/3323144
27631 memory blocks freed     : 3251016/3320152
33 unfreed memory blocks : 2922
True heap size : 622592
True free heap : 614016
Should be : 615376
Call trace for block $00007FE0F06FFAE0 size 69
  $00000000004F5D76
  $00000000007F7DF0 line 230 of oauth.pas
  $0000000000490855 line 83 of ncsetup.pas
  $000000000057EA62 line 2913 of include/control.inc
  $00000000005E305A line 55 of include/buttoncontrol.inc
  $00000000005E38A3 line 169 of include/buttons.inc
  $00000000005E2F46 line 21 of include/buttoncontrol.inc
  $0000000000432B1F
Call trace for block $00007FE0F04CD180 size 42
  $00000000004F5D76
  $00000000007F7DF0 line 230 of oauth.pas
  $0000000000490855 line 83 of ncsetup.pas
  $000000000057EA62 line 2913 of include/control.inc
  $00000000005E305A line 55 of include/buttoncontrol.inc
  $00000000005E38A3 line 169 of include/buttons.inc
  $00000000005E2F46 line 21 of include/buttoncontrol.inc
  $0000000000432B1F

Please apply PR #164

Sorry, please slow down a bit Joan, I am having to make quite a lot of changes to you patch from yesterday to endure backward comparability.  You will have a lot of problems merging with what I have done.  If you can wait a short while, I will push and then we can see how well this one merges ....
David

My guess is that your memory leak is from where you create the OAuth, if you create, so must you free !
We could have a look at how that works too, it might be a good idea to only create it when its about to be used and release it immediately afterward.  Thats a bit kinder on memory usage.
For 'regional' vars like OAuth, I like to expressly set them to nil in the FormCreate function then when we want to use it, you might do something like this -
if OAuth <> nil then freeandnil(OAuth);
OAuth := TOAuth.Create
try
....
finally
freeandnil(OAuth)
end;
Its a safe ....
davo

Yes, all the memory leaks are fixed in my PR


Promote KafkaChannel to v1

Problem
KafkaChannel should be promoted to v1. Would be best as a follow up on after the refactorings and unifications:

https://github.com/knative-sandbox/eventing-kafka/issues/386
https://github.com/knative-sandbox/eventing-kafka/issues/387

Persona:
Persona:
Which persona is this feature for?
Exit Criteria
A measurable (binary) test that would indicate that the problem has been resolved.
Time Estimate (optional):
How many developer-days do you think this may take to resolve?
Additional context (optional)
Add any other context about the feature request here.

/remove-lifecycle stale

/reopen
/remove-lifecycle stale

/triage accepted


PR: Reduce row height in dependencies dialog

By default, Qt uses an unnecessarily large row height for tables. This results in not many rows being visible in the dependency dialog.
This patch reduces the row height of the dependency table.
Before:

After:

Thanks for your contribution!
@rlaverde, please review this one.


fix(CX-3619): change tabs text size

The type of this PR is: FIX

This PR solves CX-3619
Description

Adjust font size for Tabs in Collector Profile mweb to xs
https://user-images.githubusercontent.com/36167539/231989099-af248376-c24c-4c43-ba51-b951080fd3f3.mov

Why does this keep happening. This is not correct. If you want to change this then we should change it for the entire design system. But I do not recommend that, tap areas should not be smaller on mobile.

(I've rolled this back in 02af4ae80ff73fa2481e75906e3c5ec470fca4c3 just so we don't have a random one-off inconsistent tab bar for the time being.)


sources.rs: fix test_get_download_url

From my testing, the test_get_download_url fails with:
     Running unittests src/main.rs (target/x86_64-unknown-linux-gnu/release/deps/rye-f2fb693ac01ef08b)

running 1 test
test sources::test_get_download_url ... FAILED

failures:

---- sources::test_get_download_url stdout ----
thread 'sources::test_get_download_url' panicked at 'assertion failed: `(left == right)`
  left: `Some((PythonVersion { kind: "cpython", major: 3, minor: 8, patch: 14 }, "https://github.com/indygreg/python-build-standalone/releases/download/20221002/cpython-3.8.14%2B20221002-aarch64-apple-darwin-pgo-full.tar.zst"))`,
 right: `Some((PythonVersion { kind: "cpython", major: 3, minor: 8, patch: 14 }, "https://github.com/indygreg/python-build-standalone/releases/download/20221002/cpython-3.8.14%2B20221002-aarch64-apple-darwin-debug-full.tar.zst"))`', rye/src/sources.rs:123:5
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace

failures:
    sources::test_get_download_url

test result: FAILED. 0 passed; 1 failed; 0 ignored; 0 measured; 0 filtered out; finished in 0.00s

This PR changes the expected value for the URLs and hence fixes those tests.
I am not sure rather this is correct though.

Thanks :)


typedoc 0.9.0 no longer processes files in tsconfig.json "files" order

In typedoc 0.8.0, the following command:
typedoc --mode file --out foo .

processes files in the order they appear in the "files" array of tsconfig.json.
In typedoc 0.9.0, the same command processes files in the order they appear in the directory, regardless of what's in the "files" array of tsconfig.json.  This can lead to compile errors, because order of compile matters.
Expected behavior: Files are processed in the order they appear in tsconfig.json's files list.
I isolated the issue to https://github.com/TypeStrong/typedoc/commit/482821ce51c10909c642638e8b85ff9300e7aed8 . Before that commit, everything still works the way 0.8.0 did.  The associated PR is https://github.com/TypeStrong/typedoc/pull/587

Interesting. My guess is that the TypedocReader or ArgumentsReader is (or both are) overwriting(/riding) the order from the TSConfigReader.  I'll look around when I get a chance this week to see if the files option is actually getting passed to TypeDoc. It might not be (e.g. #588).

After some further testing, I suspect that the "files" array in tsconfig.json is being ignored completely.  It appears to be processing all *.ts files in the specified directory, in the order they appear.

I was having trouble with that with #587 and CLI-specified files. I think I
have a pretty good idea where to fix this if you don't beat me to it.
On Oct 10, 2017 7:19 PM, "Brett Vickers"<EMAIL_ADDRESS>wrote:

After some further testing, I suspect that the "files" array in
tsconfig.json is being ignored completely. It appears to be processing
all *.ts files in the specified directory, in the order they appear.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/TypeStrong/typedoc/issues/617#issuecomment-335644881,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AF9gku9wSDFWDpJduiOx2cxj5kfCINyWks5srAmigaJpZM4P0fdr
.

#610 should fix that

I added more detail about the issue on #610 for those interested.

Just tried 0.11.0 and this bug still seems to be there.

@beevik The #610 PR was to change the read order of the option readers (arguments/tsconfig/typedoc) and NOT the individual loaded file order.
Looking at the source for the typedoc reader, its using the Typescript config reader which handles its own load order:
const { config } = ts.readConfigFile(fileName, ts.sys.readFile);

...

const { fileNames, options, raw: { typedocOptions }} = ts.parseJsonConfigFileContent(
            config,
            ts.sys,
            Path.resolve(Path.dirname(fileName)),
            {},
            Path.resolve(fileName));

One option would be to specify the files array in the typedoc.js config file, since that reader reads the files array as-is.
Example typedoc.js:
const path = require('path');
const tsconfig = require(path.resolve(__dirname, './tsconfig.json'));
tsconfig.out = path.resolve(__dirname, './docs');
tsconfig.name = 'Source Documentation';
tsconfig.mode = 'modules';
tsconfig.files = [
  ...
];
module.exports = tsconfig;

Im not entirely sure if there is any sorting later on down the typedoc pipeline, but I would suggest starting there.

If typedoc is using typescript's tsconfig.json parser, I wonder why it ends up compiling files in an order different than the TypeScript compiler itself does.


What is the target BLEU score for machine translation

Hi,
I am running the machine translation examples, however the best BLEU score reported is 7.71 with more than 400000 iterations' training. However the paper neural machine translation by jointly learning to align and translate reports over 17.82 or 26.75.
Any idea on that?

The results reported in the mentioned paper, are the BLEU scores of the trained models computed
on the test set for English to French language pairs. As you can see in the prepare_data.py (https://github.com/mila-udem/blocks-examples/blob/1c6ccb9caffc199313265df3a30f20af1d92d3b6/machine_translation/prepare_data.py#L40-L43), the languge pairs are Cs to En. Obviously, different language pair leads to different results. If you want to replicate the results in the paper, you have to ensure all of your parameters and settings are the same.

@papar22 yes, but I also check the website hosting the data (http://www.statmt.org/wmt15/translation-task.html), which has a statistics of BLEU for all language pairs over all dataset. The cs to en, still has something over 25.

@zhangtemplar, here are the reasons why you dont see the state of the art BLEU scores by running the example,

In this example, we do not use the entire cs-en corpus but only a small chunk of it. Entire corpus has 12M parallel sentences, but the provided prepare data script only downloads a subset of it (newscommentary-v10) which only has 150K parallel sentences.

State of the art systems are using a lot of additional methods/tricks such as large vocabularies, ensembles, unk-replacements, language models, rescoring etc. which are not implemented here in this example.
Finally, download the entire data, play with hyper-parameters and give it some time :)

I tested with En-Fr, used the same data as in the paper, got 26.57 (the paper reported 26.75 after 5 days) after 5 days 8 hours using GPU (7-80k iterations/day). Just FYI.

@orhanf thanks for your explanation. But I still doubt why the difference is so huge, even considering I have less data and less tricks.
However, I realized they may have a different metrics for BLEU. I found the code reports five value for BLEU, namely BLEU-1, BLEU-2, BLEU-3, BLEU-4 and finally BLEU, which seems to be an exponential average of the previous four BLEUs. For BLEU I can something close to 30.
@tnq177 for 26.57, are you referring to BLEU or BLEU-1?

@zhangtemplar BLEU

@tnq177 interesting. I may need to try en-fr instead.

@tnq177  Did you just use the code provided by tensorflow?
can you tell me the details of the model?( tensorflow version, stack size, layer size, sampled_loss(sample size), voca_size, etc.)
Thanks

@ad26kt no, I used this blocks-examples/machine-translation code. I didn't make any change in the configuration, except using en-fr data as detailed in Bahdanau's paper.

@tnq177 can you tell me which part of the prepare_data.py should I fix to using en-fr data as detailed in Bahdanau's paper? I tried several way, but it caused an error like :
File "prepare_data.py", line 135, in create_vocabularies
    if n.endswith(args.source)][0]]) + '.tok'
IndexError: list index out of range

I'm a little bit urgent , so your help will be really appreciated

@ad26kt I juse use that script as reference. You can take the data here http://www-lium.univ-lemans.fr/~schwenk/cslm_joint_paper/ (noted in Bahdanau's paper), preprocess the data as the authors mention in the paper (only tokenize, I believe.). Then to create vocabulary files, you can use GroundHog's preprocess.py (PREPROCESS_URL in prepare_data.py); the command should be something like python preprocess.py -d vocab_file_name.pkl -v number_of_unique_tokens_used train_file. You can use the function shuffle_parallel in prepare_data.py to shuffle the training data files. Now, write the correct configuration function in configurations.py with correct path to train/dev/test/vocab... files. Basically just follow the steps in prepare_data.py.

@tnq177 Thanks a lot!


Choose best package for PurlDB matched resources

In PurlDB resource matching step we may match single resource to multiple package, this post process step insures that we keep only the best matched package for every resource and remove the rest. ATM we visit archives's extract directory and rank all the packages based on the higest number of resource matched, and only keep those resource-package relationship where package has highest resource count.

@keshav-space could you fix the conflicting files?

@keshav-space could you fix the conflicting files?

Done!


Milestone: Mobile menu

Milestone: Mobile menu
In this branch, my coding partners and I worked on the following features:

Add new linters
Add events.js file
Add a mobile menu
Add events to the buttons

Changes Done
These were the changes applied:

Change the color
Add margin to the bottom line


Focus Group session

Session with Nate on 12/27/2022 to focus group features and flow. Notes below

Round length selection:

Dropdowns for selecting days, hours, minutes
OR a game stop time, auto-calculate round lengths

Might want to extend the idea of dropdowns for user input to game selection for other actions, so that users don't need to remember/search for game IDs

Resolved! Using dropdowns to select round interval, and dropdowns in each View to provide user with game selection


Item modifier fix #834

Account for modifier rank when applying bonuses to rolls. Issue #834

Closing this one as the diff was already merged as part of #875 (same source branch so both commits were already present)


Update maven repo to latest version

Hello,
could you update the maven repository to the latest version? Latest one ist still 2.5.0.
https://mvnrepository.com/artifact/com.mangopay/mangopay2-java-sdk
Thanks!

Is there any update on this?

solved with jitpack.


Add basic 'boot new' functionality

Still cleanup and tests to add.

Some sort of test directory is left behind:
$ java -jar build/libs/spring-up-0.0.1-SNAPSHOT.jar boot new --project-name test --template-name jpa --package-name io.example
Setting history file to /home/jvalkealahti/spring-shell.log
Project test created in /tmp/source-repo-10680700830669244648/rd-1-2022-rpt-spring-data-jpa-da76048
~/repos/spring/spring-up [gh7-review|…17]

$ git status
On branch gh7-review
Untracked files:
  (use "git add <file>..." to include in what will be committed)
	test/

nothing added to commit but untracked files present (use "git add" to track)
~/repos/spring/spring-up [gh7-review|…17]

$ ls test
build.gradle  gradle  gradlew  gradlew.bat  LICENSE  mvnw  mvnw.cmd  pom.xml  README.md  settings.gradle  src

Looks to be merged in main.


basePath topic unresolved

Hi
I have an OAS2 definition that specifies paths like /lemons and a basePath like /api/v1. Furthermore I am using a code generator that fixes those together like /api/v1/lemons in the api client.
Let's say it is totally impossible to upgrade to OAS3 or use anything but that generated basePath-concatenating api client.
How can I use prism to serve those endpoints at exactly http://localhost:7000/api/v1/lemons, etc, please?
TIA

@cge can you provide us with an example OAS2 spec that illustrates your question?

Hi @chohmann, I guess I have the same question/issue. Reading from the FAQ it seems that is the expected behaviour but I don't understand why. If I have a client that I want to test against the mock then I will simply get 404 for the requests.
I also have a v2 swagger file with a basePath, and as the prism output says it's going to be resolved without it.


Hostname source

Remove hostname from metric name. Replace hardcoded sensu source with hostname source.

@portertech @petecheslock don't suppose I could ask a favor for an opinion on this :)? More wondering if this would break anything major that you know of. It's a lot better for Librato users to have the ephemeral/locality context (e.g. hostname) in the source name separated out from the permanent definition of the metric name. Enables users to toggle their sensu metrics in a template dashboard.


Can't writeValue

Hi,
I'm trying to use tinyb (Java) to control my BLE device (Ubuntu 16.04, Bluez 5.37 + -E flag). Discovering devices, services, and charateristics works. However, when I am trying to use writeValue (like in the sample app) to write to a characteristic, I get the following exception:
tinyb.BluetoothException: GDBus.Error:org.freedesktop.DBus.Error.UnknownMethod: Method "WriteValue" with signature "aya{sv}" on interface "org.bluez.GattCharacteristic1" doesn't exist
Using the bluetoothctl cli-tool, I can send the commands without an issue.
Any idea how to get around this?

You need to update BlueZ to version 5.42 or newer, or use TinyB 0.5.

Upgrading to Bluez 5.43 helped indeed.
Thanks a lot!


Update Set-RemoteMailbox.md

Issue #1894

Docs Build status updates of commit 334a05d:
:clock10: Full build: the average full build time is 18 min(s) 6 sec(s), this is based on the last 1000 build(s) for this repository.

Docs Build status updates of commit 334a05d:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

exchange/exchange-ps/exchange/Set-RemoteMailbox.md
:white_check_mark:Succeeded
View (exchange-ps)

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel

Docs Build status updates of commit 03d0869:
:clock10: Full build: the average full build time is 18 min(s) 5 sec(s), this is based on the last 1000 build(s) for this repository.

Docs Build status updates of commit 03d0869:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

exchange/exchange-ps/exchange/Set-RemoteMailbox.md
:white_check_mark:Succeeded
View (exchange-ps)

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel