Make image / document upload forms work with custom tag models

The recipe in the documentation for custom tag models assumes that your custom tag model is going to be used on your own page / snippet model, while the image and document models continue to use the stock taggit.Tag model. With a custom image/document model and some careful field overriding, it ought to be possible to use a custom tag model for images and documents, making it possible to disable free tagging, attach extra metadata (e.g. descriptions) to tags, or just have the autocomplete suggestions come from separate lists...
from django.db import models
from taggit.managers import TaggableManager

from taggit.models import ItemBase, TagBase
from wagtail.documents.models import AbstractDocument, Document
from wagtail.snippets.models import register_snippet

@register_snippet
class DocumentTag(TagBase):
    free_tagging = False
    site = models.CharField(max_length=20, choices=[('site1', 'Site 1'), ('site2', 'Site 2')])

    class Meta:
        verbose_name = "document tag"
        verbose_name_plural = "document tags"

class TaggedDocument(ItemBase):
    tag = models.ForeignKey(
        DocumentTag, related_name="tagged_documents", on_delete=models.CASCADE
    )
    content_object = models.ForeignKey(
        to='customdocs.CustomDocument',
        on_delete=models.CASCADE,
        related_name='tagged_documents'
    )

class CustomDocument(AbstractDocument):
    tags = TaggableManager(help_text=None, blank=True, verbose_name='tags', through=TaggedDocument)

    admin_form_fields = Document.admin_form_fields

However, this doesn't quite work, because the upload forms for images and documents use a plain unconfigured AdminTagWidget, which ends up always using the taggit.Tag model. This PR adds the necessary glue to pick up the tag model actually used by the image/document model and pass it on to AdminTagWidget.

rebased


Test error messages for update compat tests

What should be done?
in testUpdateCompat we should test more than just error codes for writeErrors. We expect to have some failing cases afterwards (especially for $rename operator), every of them should be fixed. Smaller differences should be covered with altMessage field, which is also in the scope of this issue. As in some integration tests - if altMessage != "", we should compare it's content with target's error.
DoD

[ ] Comparing writeError messages in testUpdateCompat helper.
[ ] Implementing altMsg string field in updateCompatTestCase
[ ] Fixing error differnces (especially in $rename)

https://github.com/FerretDB/FerretDB/issues/2345#issuecomment-1494035888
Please leave it open for now


Ctrulib upgrade

This pull requests upgrades svdt to compile with ctrulib-1+. I tested the anti-anit hacking functionality with Smash4, and it seems to be working. The only thing that doesn't work reliably is returning to the HBL after pressing Start-- the program sometimes hangs, sometimes doesn't (sounds like a possible memory management problem somewhere).
Also did some changes to make some formatting/line-endings (more) consistent.

Fixes #22

It seems the crashing problem was already a thing (issue #19), so it doesn't seem like this work causes a regression.


Add support for development AWS environment

What does this pull request aim to achieve? 💯
Adding a deployment script to the developer sandbox environment, as well as updating the main.tf Terraform file to support the developer sandbox. This environment will be used for testing new DevOps/infrastructure changes. As such, it does not require Google Maps, SendGrid, Sentry, or Papertrail. By having a separate sandbox environment for testing DevOps changes, we eliminate the risk of impacting the review, staging, and production environments with untested changes.

@mannykary , code looks good. Do you think these changes require README update?
Here is what I reviewed. It looks good.
When fp_context is 'dvelopment', instance count will be 0 for:
sendgrid_api_key
google_maps_api_key
sendgrid_contact-list_id
sentry_dsn
logger_host
logger_port
New locals:
app_domain:
development = fightpandemics.online
auth_app_url:
development = "https://development....."
main module updates:

Following backend variables will get no values for 'development'
GOOGLE_MAPS_API_KEY
SENDGRID_API_KEY
SENDGRID_CONTACTS_LIST_ID
SENTRY_DSN
LOGGER_HOST
LOGGER_PORT

Yes, looks like your understanding is correct!
As for the readme, I decided to add it to the one in the infrastructure repo rather than this one because the development sandbox environment is only going to be used by the DevOps team for testing out changes in Terraform code.


runnable openjdk-11-runtime image

Inside a chained build, I was expecting to only have to copy a bootable JAR into the runtime image /deployments directory then have the run script to take over on image run and start my app with correct JVM settings.
Instead it is displaying the message:  "This is the Red Hat UBI8 OpenJDK 11 Runtime container".
I am wandering if the runtime image could take advantages of the automatic (and configurable) JVM configuration we find in the builder image and offer an entry point that would automatically run/configure a java application?
Thank-you.

Hi @jfdenise

I am wandering if the runtime image could take advantages of the automatic (and configurable) JVM configuration we find in the builder image and offer an entry point that would automatically run/configure a java application?

Yes this is a limitation. We decided against including the existing run-java.sh in the runtime image for launch, as we need to go through it and make sure it is not too tightly coupled to maven, etc., which are not going to be present. That should happen but we didn't want to delay releasing something in the meantime.

@jmtd , thank-you, makes full sense. So I understand that the long term plan is to have a runnable runtime-image.

I've filed https://issues.redhat.com/browse/OPENJDK-413 to track this on the product  side.

Addressed from 1.12 onwards


Document that coredns is available as an option for k8s 1.12+ versions only

CoreDNS was added as an optional provider in Rancher 2.2.x. There is one limitation that we have to document - CoreDNS is available for k8s 1.12+ versions only. If you try to use it for the earlier version of clusters, it is going to fail.

Closing in favor of this docs bug: https://github.com/rancher/docs/issues/1578


airspy: 1.0.9 -> 1.0.10

Motivation for this change
Minor update: https://github.com/airspy/airspyone_host/releases
Things done

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

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 120564 at 3e1c0665 run on aarch64-linux 1

3 packages marked as broken and skipped:

gnuradio3_7Packages.ais
gnuradio3_7Packages.gsm
gnuradio3_7Packages.osmosdr

1 package skipped due to time constraints:

sdrangel

13 packages built successfully:

airspy
cubicsdr
gnss-sdr
gnuradio3_8Packages.osmosdr
gqrx
pothos
python38Packages.soapysdr-with-plugins
python39Packages.soapysdr-with-plugins
qradiolink
rtl_433
soapyairspy
soapysdr-with-plugins
urh


[BRD-144] - Loading Page

created loading page and swapped with skeleton

Pull Request Test Coverage Report for Build 764155308

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

Totals

Change from base Build 763149716:
0.0%

Covered Lines:
77

Relevant Lines:
77

💛  - Coveralls


Updated Contributing Guide

Hey, I just made a Pull Request!
Added a few items that were missing and some other minor adjustments.
:heavy_check_mark: Checklist

[ ] A changeset describing the change and affected packages. (more info)
[ ] Tests for new functionality and regression tests for bug fixes
[ ] Screenshots attached (for UI changes)
[x] All your commits have a Signed-off-by line in the message. (more info)

Hi @BethGriggs, could you give this a look? I sort of just dumped the things I knew were missing but I feel like the order and some of the wording could use some work.


How does zuul2 deal with the situation where the upstream connection is directly disconnected?

In zuul2, if the upstream disconnects, it seems that HttpOutboundFilter will not be executed. Where should I catch this exception and handle it?

It appears that the HandleDisconnectEventFilter class is not present in the main Zuul repository on GitHub.


Detect Head Output

The output of the forward Method of the Detection Head needs to be a torch.Tensor, instead of a tuple. This would facilitate integration with our original YOLOv8 model. Would it be possible to modify the mct pipeline to accept a torch.Tensor with shape (B, 84, 8400) instead of a tuple within split y_bb with shape (B, 8400, 4) and y_cls with shape (B, 840, 80)?
Sony Implementation:
    def forward(self, x: Tensor) -> Tuple[Tensor, Tensor]:
        shape = x[0].shape  # BCHW
        for i in range(self.nl):
            x[i] = torch.cat((self.cv2[i](x[i]), self.cv3[i](x[i])), 1)
        box, cls = torch.cat([xi.view(shape[0], self.no, -1) for xi in x], 2).split(
            (self.reg_max * 4, self.nc), 1)

        y_cls = cls.sigmoid().transpose(1, 2)

        dfl = self.dfl(box)
        dfl = dfl * self.strides

        # box decoding
        lt, rb = dfl.chunk(2, 1)
        y1 = self.relu1(self.anchors.unsqueeze(0)[:, 0, :] - lt[:, 0, :])
        x1 = self.relu2(self.anchors.unsqueeze(0)[:, 1, :] - lt[:, 1, :])
        y2 = self.relu3(self.anchors.unsqueeze(0)[:, 0, :] + rb[:, 0, :])
        x2 = self.relu4(self.anchors.unsqueeze(0)[:, 1, :] + rb[:, 1, :])
        y_bb = torch.stack((x1, y1, x2, y2), 1).transpose(1, 2)
        return y_bb, y_cls

Original YOLOv8 implementation:
    def forward(self, x):
        """Concatenates and returns predicted bounding boxes and class probabilities."""
        if self.end2end:
            return self.forward_end2end(x)

        for i in range(self.nl):
            x[i] = torch.cat((self.cv2[i](x[i]), self.cv3[i](x[i])), 1)
        if self.training:  # Training path
            return x
        y = self._inference(x)
        return y if self.export else (y, x)

Hi @ambitious-octopus,
The reason we have this modification (tuple instead single Tensor), is due to the different values range of those 2 parts of the original Tensor.
y_bb (bounding boxes coordinates 4x8400) values range in [0, 640] and y_cls (score per class - 80x8400) range in [0,1].
This fact makes the quantization of this single tensor very problematic, resulting in bad accuracy of the quantized model. (for one part we need high resolution in the range of [0,1] while for the other part we need to stretch the values up to 640).
The solution we suggested is therefore to keep the 2 parts separately during MCT quantization.
Let me know if you need more detailed explanation.
Thanks
Idan

@Idan-BenAmi Hi!
Thanks for the explanation!
However modifying the original tensor output to a tuple in ultralytics package would break all our current inference pipelines. Is it possible to split the original tensor to a tuple in model_optimization repo before actual starting the MCT quantization?
Thanks

Hi @Laughing-q and @ambitious-octopus ,
I missed your last question, sorry for the delay.
The MCT isn't designed to manage this kind of manipulation. What do you think about keeping the split operation within the export code?


[RFC] simplify the jbuilder invocation in opam files

The jbuilder command line in opam files is too long:
build: [
  ["jbuilder" "build" "@install" "--root" "." "--only" "utop" "-j" jobs]
]

And it might get longer if we add support for configuration files in the future:
build: [
  ["jbuilder" "build" "@install" "--root" "." "--only" "utop" "--no-config" "-j" jobs]
]

This PR adds a --release pkg option which is a shorthand for --root . --only-package pkg, and later might also include --no-config. With this and #47, we can simplify the invocation to:
build: [
  ["jbuilder" "build" "--release" "utop" "-j" jobs]
]

I'm not sure about the name of the option though. Suggestions welcome. Another possibility is to add a separate command, however we need the same thing for tests, so an option seems better.

One thing that's slightly weird is that this looks like the opposite of --dev but --dev only changes the ocaml flags. @hhugo and suggested --for-release <package>.
Another possibility is to have a descriptive long option --for-release-of-packages <pkgs> with a short version -r <pkgs>:
build: [
  ["jbuilder" "build" "-r" "utop" "-j" jobs]
]
test: [
  ["jbuilder" "runtest" "-r" "utop" "-j" jobs]
]

Implemented the version with the short option, but used -p which IMO reads better as the argument is a package name

Incidentally, it might be a good idea to advise people to write "%{name}%" instead of a package name in the build line, so that exactly the same build command can be copy-pasted between opam files.
e.g. https://github.com/stedolan/malfunction/blob/master/malfunction.opam#L9

Indeed, that's even better. BTW, shoudn't just name instead of "%{name}%" work as well?

I checked that build: [["jbuilder" "build" "-p" name "-j" jobs]] works properly in my latest opam PR and updated the manual accordingly

Thanks!


Add python version to README

So, we (I,sorry!) use f-strings, which require Python 3.6. Do we have anything other features to think about? Are we okay requiring >3,6?

Added in #295


[Bug]: The approval initiated in the sql check result contains sql that did not perform the sql check

ODC version
ODC 4.2.3
OB version

What happened?

The approval initiated in the sql check result contains sql that did not perform the sql check.
What did you expect to happen?
Initiate approval for the sql that performs the sql check.
How can we reproduce it (as minimally and precisely as possible)?
1.Write some sql statements into the sql window.
2.sql check is performed on only part of the sql statements.
3.Initiate approval in sql result.
Anything else we need to know?
No response
Cloud
No response

Fixed In Commit f7de18b7fae938488489828a3ca0fc4614012c40
------
陈朝俊
Wed Dec 13 09:27:46 2023 +0800

pass


Fix spotbug warning in KafkaIO PreCommit

Please add a meaningful description for your change here

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Mention the appropriate issue in your description (for example: addresses #123), if applicable. This will automatically add a link to the pull request in the issue. If you would like the issue to automatically close on merging the pull request, comment fixes #<ISSUE NUMBER> instead.
[ ] Update CHANGES.md with noteworthy changes.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

See the Contributor Guide for more tips on how to make review process smoother.
To check the build health, please visit https://github.com/apache/beam/blob/master/.test-infra/BUILD_STATUS.md
GitHub Actions Tests Status (on master branch)

See CI.md for more information about GitHub Actions CI or the workflows README to see a list of phrases to trigger workflows.

assign set of reviewers

PreCommit Java Examples Dataflow E2EWordcount failed with quota issue (cannot bring up any worker), not related


add south polar stereo zooming example

Quick addition of a zooming-in example for a South Polar Stereo projection. I think this is helpful since it uses different keywords than rectangular projections.
Notes:

I can't build the sphinx to check that this worked out as envisioned. Having issues getting the sphinxext.automodapi extension installed (make html returns(exception: No module named 'sphinxext') after running pip install sphinx-automodapi). So you might have to check this PR out and try building.
I don't know your naming convention for showcase .pngs. Can change to a specific number if that is desired.
I don't have a cartopy example, as it seems you don't have SouthPolarStereo installed in proplot from cartopy. (https://scitools.org.uk/cartopy/docs/v0.15/crs/projections.html) Any reason for this?
Something in your .gitignore is ignoring some of the showcase png files. I noticed this because when I tried to push showcase_spstere.png it made me force add. Despite nothing showing up in your .gitignore to really cause this... Also some of your newer pngs didn't download with my git clone.

Note: I added this zoom-in example to the documentation.
Unfortunately, as things stand, contributors cannot add examples, because I generate examples from a big ipython notebook and convert them to RST with a shell script that calls nbconvert and does some regex magic to repair RST-style Sphinx module links embedded in the Markdown notebook cells (e.g. by default, nbconvert turns ~proplot.subplots.Figure into proplot.subplots.Figure -- is this something nbsphinx repairs?).
In the near future, I need to commit the example notebook, use nbsphinx to automatically run each cell instead of running the cells manually myself, and configure nbsphinx with a formal nbconvert template/filter that repairs the links.


go-peerflix continues to download even when you close vlc

So first of all i like the idea - kudos on that.
Second of all i hope you manage that todo for OS detection it would make things a lot easier (even tho on Ubuntu you just need to remove "open","-a" and it works).
But what i really don't like (don' take this personally) is that when i close VLC that the download keeps going.  Is that on your check/todo list? :)
Thanks and keep up the great work.

Hey,
I think we can close the program when the playing finished and -seed is not passed as an argument.
I'll work on it, thanks for the idea =)


changelog cleanups

Signed-off-by: Ryan Northey<EMAIL_ADDRESS>
Commit Message:
Additional Description:
Risk Level:
Testing:
Docs Changes:
Release Notes:
Platform Specific Features:
[Optional Runtime guard:]
[Optional Fixes #Issue]
[Optional Fixes commit #PR or SHA]
[Optional Deprecated:]
[Optional API Considerations:]

for merging into release commit/pr - tested here https://github.com/envoyproxy/envoy/pull/24989


KF NGPU

@MihaelOreshenko lib/domains/ru/KF: this request has an incorrect file path format, please ensure that you are familiar with     this repository rules located at the bottom of this page and change     your request accordingly. If you don't know how to change request, close it and open a new one.     If you are not able to make the pull request according the instructions, please send us an email to<EMAIL_ADDRESS>Please mention this request URL in the message.


Grouping functions by input and/or usecase

Hello!
I'm new to Ramda and i have hard time to learn all the functions; There is so many of them.
Because of that, it just came to my mind that it would be a lot easier to understand ramda and it's functions by grouping the somehow. For example: R.object.where, or R.math.sum. This way it would be easier to find in editor the correct function for each use case and understand straight away where it should be used.
I just want to raise some conversation how you guys feel about the idea. Of course for older users like ramdas core developer its always clear how and where to use the function, but for newcomers the learning curve is quite high because of the amount of functions.
I know it's more verbose, but i think it would pay back by making it clearer for user what is the purpose and/or what the function takes in. It could also be optional: reserving for example words R.object, R.math, R.function, R.list/array R.string etc for more declarative use, and let the current api as it is.

welcome @villesau !
I love the idea of grouping common functions for the documentation. That should absolutely happen. But I think adding namespaces to the library will be a tough sell.

Is there any drawbacks in namespacing them? It could be just an alias for the R. so the function could work in both ways. I haven't tried of course, but as a newcomer and fresh eyes to Ramda i think it would make the learning and understanding the library a lot easier, at least with an editor/ide which has some sort of autocomplete.
Now it's not very intuitive when there is a huge list of functions for totally different purposes.
I don't see any harm in namespacing/aliasing the functions, so why not give it a try? If i'm right, Ramdas api is not freezed yet so if  after gathering some feedback about the approach it turns out to be a bad idea for some reason, it is always possible to revert. Only drawbacks what i see, is that it's more verbose(but more declarative) and it pollutes the api a bit.

Well if it were just an alias for R, then weird constructs like these would be legal, which would bother me:
R.math.map // should be R.list.map
R.math.list.string.object.function.logic.relation.filter // Well, why not?

I agree with @buzzdecafe that this belongs as part of documentation, and not as part of the runtime code.  But it definitely belongs.  The only challenge is that some of our functions really should be listed in multiple categories.  (Does max belong in List or Math?)  But there seems to be no good reason to duplicate the listings...

Closing since we've decided not to pursue the original proposal.
As for documentation, the Ramda Manual will provide thematic function grouping.


When a Mailable is added on the queue collection loses order.

Laravel Version: 5.5.40
PHP Version: 7.0.29
Database Driver & Version:
MySql-InnoDB 5.6.34

Description:
When a Mailable is added on the queue collection loses order.
I added the email to the queue then returned the email view. The dates when returned via the view are in desc order. The email version seems random.
        $mail = new UpcomingMatches($user, $registered, $following, $previous, 'daily', $notifications);
        Mail::to($user->email)->queue($mail);

        return $mail;

Email Version

View Version

Just re-order the collection.


feat: add and support builder_boost_factor query param to produceBlockV3 api

Add the builder_boost_factor query param to produce blockV3 api.
while the PR doesn't remove builder_selection custom param, but its now deprecated and can be safely removed from the api .
However its not supported on the validator as a cli option so that the validator can reject the produced block if its not aligned with the builder_selection the user has provided.
this PR also allows to set builder_boost_factor on the key manager api
Closes https://github.com/ChainSafe/lodestar/issues/6226

Linking issue #6226 🙂

@jeluard thanks for the reviews, added some cleanups to todo list here: https://github.com/ChainSafe/lodestar/issues/5279

:tada: This PR is included in v1.14.0 :tada:


fix: Validation requests

Data like note hash read requests, nullifier read requests, etc, are not revertible. When a tx reverts, we still need to verify those requests in the kernel.
Moving them out of (Non)RevertibleAccumulcatedData and creating a new ValidationRequests for aggregating these data.

Benchmark results
Metrics with a significant change:

note_history_successful_decrypting_time_in_ms (5): 1,193 (-20%)
note_successful_decrypting_time_in_ms (8): 243 (+19%)
note_trial_decrypting_time_in_ms (64): 110 (+90%)

Detailed results
All benchmarks are run on txs on the Benchmarking contract on the repository. Each tx consists of a batch call  to create_note and increment_balance, which guarantees that each tx has a private call, a nested private call, a public call, and a nested public call, as well as an emitted private note, an unencrypted log, and public storage read and write.
This benchmark source data is available in JSON format on S3 here.
Values are compared against data from master at commit 258ff4a0 and shown if the difference exceeds 1%.
L2 block published to L1
Each column represents the number of txs on an L2 block published to L1.

Metric
8 txs
32 txs
64 txs

l1_rollup_calldata_size_in_bytes
5,668
18,820
36,356

l1_rollup_calldata_gas
66,244
238,588
469,076

l1_rollup_execution_gas
666,925
953,498
1,336,433

l2_block_processing_time_in_ms
1,288 (-7%)
4,616 (-4%)
8,910

note_successful_decrypting_time_in_ms
:warning: 243 (+19%)
528 (-3%)
979 (+5%)

note_trial_decrypting_time_in_ms
48.8 (-55%)
44.4 (-31%)
:warning: 110 (+90%)

l2_block_building_time_in_ms
18,126 (-2%)
68,658 (-2%)
135,684 (-2%)

l2_block_rollup_simulation_time_in_ms
8,171
28,834 (-1%)
56,181 (-1%)

l2_block_public_tx_process_time_in_ms
9,934 (-4%)
39,765 (-3%)
79,404 (-3%)

L2 chain processing
Each column represents the number of blocks on the L2 chain where each block has 16 txs.

Metric
5 blocks
10 blocks

node_history_sync_time_in_ms
13,831 (-14%)
27,110 (-7%)

note_history_successful_decrypting_time_in_ms
:warning: 1,193 (-20%)
2,512 (-12%)

note_history_trial_decrypting_time_in_ms
116 (+68%)
112 (-22%)

node_database_size_in_bytes
19,062,864
35,676,240

pxe_database_size_in_bytes
29,859
59,414

Circuits stats
Stats on running time and I/O sizes collected for every circuit run across all benchmarks.

Circuit
circuit_simulation_time_in_ms
circuit_input_size_in_bytes
circuit_output_size_in_bytes

private-kernel-init
283 (-3%)
44,370
28,244 (+2%)

private-kernel-ordering
213 (-6%)
52,868 (+1%)
14,326

base-parity
1,833 (-4%)
128
311

base-rollup
725 (-3%)
165,787 (-6%)
925

root-parity
1,602 (-4%)
1,244
311

root-rollup
69.2 (-4%)
4,487
789

private-kernel-inner
637 (-3%)
73,773 (+1%)
28,244 (+2%)

public-kernel-app-logic
443 (-5%)
35,262
28,215

public-kernel-tail
172 (-7%)
40,926
28,215

merge-rollup
8.28 (-4%)
2,696
925

Tree insertion stats
The duration to insert a fixed batch of leaves into each tree type.

Metric
1 leaves
16 leaves
64 leaves
128 leaves
512 leaves
1024 leaves
2048 leaves
4096 leaves
32 leaves

batch_insert_into_append_only_tree_16_depth_ms
10.1 (-3%)
15.9 (-5%)
N/A
N/A
N/A
N/A
N/A
N/A
N/A

batch_insert_into_append_only_tree_16_depth_hash_count
16.8
31.6
N/A
N/A
N/A
N/A
N/A
N/A
N/A

batch_insert_into_append_only_tree_16_depth_hash_ms
0.588 (-3%)
0.493 (-5%)
N/A
N/A
N/A
N/A
N/A
N/A
N/A

batch_insert_into_append_only_tree_32_depth_ms
N/A
N/A
46.1 (-2%)
72.1 (-2%)
231
444 (-9%)
869
1,728
N/A

batch_insert_into_append_only_tree_32_depth_hash_count
N/A
N/A
96.0
159
543
1,055
2,079
4,127
N/A

batch_insert_into_append_only_tree_32_depth_hash_ms
N/A
N/A
0.473 (-2%)
0.445 (-3%)
0.422
0.416 (-9%)
0.415
0.415
N/A

batch_insert_into_indexed_tree_20_depth_ms
N/A
N/A
53.5 (-3%)
108 (-2%)
336 (-2%)
668 (-9%)
1,311
2,607
N/A

batch_insert_into_indexed_tree_20_depth_hash_count
N/A
N/A
104
207
691
1,363
2,707
5,395
N/A

batch_insert_into_indexed_tree_20_depth_hash_ms
N/A
N/A
0.477 (-3%)
0.485 (-2%)
0.458 (-2%)
0.459 (-9%)
0.455 (-1%)
0.454
N/A

batch_insert_into_indexed_tree_40_depth_ms
N/A
N/A
N/A
N/A
N/A
N/A
N/A
N/A
61.3 (-4%)

batch_insert_into_indexed_tree_40_depth_hash_count
N/A
N/A
N/A
N/A
N/A
N/A
N/A
N/A
109

batch_insert_into_indexed_tree_40_depth_hash_ms
N/A
N/A
N/A
N/A
N/A
N/A
N/A
N/A
0.537 (-4%)

Miscellaneous
Transaction sizes based on how many contract classes are registered in the tx.

Metric
0 registered classes

tx_size_in_bytes
22,014

Transaction processing duration by data writes.

Metric
0 new note hashes
1 new note hashes

tx_pxe_processing_time_ms
3,309
1,740 (-3%)

Metric
0 public data writes
1 public data writes

tx_sequencer_processing_time_ms
13.3 (-1%)
1,237 (-6%)


modify to the correct expected location fields

@atalyaalon
Why was this merged without waiting for approval?
Does the algorithm outputs these results?
Are these what we want the algorithm to output, or what it really does?

@Mano3 it's what the algorithm does, but in this case it seems like the location is indeed accurate. That's why it's merged. Tests are passing well in dev.

It merely fixes a mistake I made due to google-maps issues on my machine.

Alright, previous comments from other PRs confused me.


rejected by a future version of Rust: quick-xml v0.17.2

Describe the bug
warning: the following packages contain code that will be rejected by a future version of Rust: quick-xml v0.17.2
note: to see what the problems were, use the option `--future-incompat-report`, or run `cargo report future-incompatibilities --id 1`

To Reproduce
Steps to reproduce the behavior:

cargo install target-gen --git https://github.com/probe-rs/probe-rs --locked --force

Expected behavior
installation process free of errors and warnings
Desktop (please complete the following information):

Linux (Ubuntu)

This is due to an outdated dependency of the cmsis-pack carte. Unfortunately it's non-trivial to update, so I don't think it will happen anytime soon.


knife status NoMethodError: undefined method `first' for nil:NilClass

Description
When running knife status  "name:*node*" -F json the expected result is to show the status of the node searched for, instead there is an error given.
Chef Version
Chef Infra Client 14.13.11
Chef Workstation Version latest stable
Platform Version
Any Flavor Linux/Windows EC2 Instance
Replication Case
Run the knife status "name:*node*" command against any EC2 instance that doesn't have a public IP.
Client Output
/opt/chef-workstation/embedded/lib/ruby/2.7.0/net/http/exceptions.rb:25: warning: already initialized constant Net::HTTPClientException
/opt/chef-workstation/embedded/lib/ruby/gems/2.7.0/gems/chef-16.6.14/lib/chef/monkey_patches/net_http.rb:16: warning: previous definition of HTTPClientException was here
ERROR: knife encountered an unexpected error
This may be a bug in the 'status' knife command or plugin

Stacktrace
/opt/chef-workstation/embedded/lib/ruby/gems/2.7.0/gems/chef-16.5.77/lib/chef/knife/core/status_presenter.rb:70:in `block in summarize_json': undefined method `first' for nil:NilClass (NoMethodError)
	from /opt/chef-workstation/embedded/lib/ruby/gems/2.7.0/gems/chef-16.5.77/lib/chef/knife/core/status_presenter.rb:65:in `each'
	from /opt/chef-workstation/embedded/lib/ruby/gems/2.7.0/gems/chef-16.5.77/lib/chef/knife/core/status_presenter.rb:65:in `summarize_json'
	from /opt/chef-workstation/embedded/lib/ruby/gems/2.7.0/gems/chef-16.5.77/lib/chef/knife/core/status_presenter.rb:57:in `format'
	from /opt/chef-workstation/embedded/lib/ruby/gems/2.7.0/gems/chef-16.5.77/lib/chef/knife/core/ui.rb:184:in `output'
	from /opt/chef-workstation/embedded/lib/ruby/2.7.0/forwardable.rb:235:in `output'
	from /opt/chef-workstation/embedded/lib/ruby/gems/2.7.0/gems/chef-16.5.77/lib/chef/knife/status.rb:87:in `run'
	from /opt/chef-workstation/embedded/lib/ruby/gems/2.7.0/gems/chef-16.5.77/lib/chef/knife.rb:490:in `block in run_with_pretty_exceptions'
	from /opt/chef-workstation/embedded/lib/ruby/gems/2.7.0/gems/chef-16.5.77/lib/chef/local_mode.rb:42:in `with_server_connectivity'
	from /opt/chef-workstation/embedded/lib/ruby/gems/2.7.0/gems/chef-16.5.77/lib/chef/knife.rb:489:in `run_with_pretty_exceptions'
	from /opt/chef-workstation/embedded/lib/ruby/gems/2.7.0/gems/chef-16.5.77/lib/chef/knife.rb:229:in `run'
	from /opt/chef-workstation/embedded/lib/ruby/gems/2.7.0/gems/chef-16.5.77/lib/chef/application/knife.rb:165:in `run'
	from /opt/chef-workstation/embedded/lib/ruby/gems/2.7.0/gems/chef-16.5.77/bin/knife:24:in `<top (required)>'
	from /opt/chef-workstation/bin/knife:353:in `load'
	from /opt/chef-workstation/bin/knife:353:in `<main>'

I was able to reproduce this with version 16.8.14 of chef-client as well

Fixed in #10781.


Column rendering layout broken with small screen size

When I reduce the screen size such that breakpoint xs is hit, then the table rendering seems broken. The column header renders into each row (see attached screenshot). This may be due to some CSS issue in my project (I have forked the project from an existing template and don't yet know my way around yet).
Still, I believe it is all fairly standard and such a behavior should not be caused by outer CSS. Also, other components of the app like Tabs handle the transition well.
This is what my theme preakpoint config looks like:
{ "breakpoints": {
    "keys": ["xs", "sm", "md", "lg", "xl"],
    "values": { "xs": 0, "lg": 1280, "sm": 600, "xl": 1920, "md": 960 }
  }}

This is the screen I get before hitting the xs breakpoint.

And this is what it looks like when layout passes the breakpoint (and several components respond with layout changes).

Is this a bug in mui-datatables or more likely of my project layout?

Tech
Version

Material-UI
4.2.1

MUI-datatables
2.7.0

React
16.8.6

browser
firefox, chrome (Ubuntu)

etc

Hello @bennidi. It's hard to say at this point where the issue is, as I'd need to know more about your table options and code. Are you able to reproduce this issue from one of the examples in the project? You could take a simple example, add it to a codesandbox, and then progressively add features from your own project until it breaks. If you can do that, I would be happy to take a look.

Yes, the code is taken from the example in main README.md. I just render it within the layout of my app.
import React from 'react';
import MUIDataTable from "mui-datatables";

const columns = [
  {
    name: "name",
    label: "Name",
    options: {
      filter: true,
      sort: true,
    }
  },
  {
    name: "company",
    label: "Company",
    options: {
      filter: true,
      sort: false,
    }
  },
  {
    name: "city",
    label: "City",
    options: {
      filter: true,
      sort: false,
    }
  },
  {
    name: "state",
    label: "State",
    options: {
      filter: true,
      sort: false,
    }
  },
];

const data = [
  { name: "Joe James", company: "Test Corp", city: "Yonkers", state: "NY" },
  { name: "John Walsh", company: "Test Corp", city: "Hartford", state: "CT" },
  { name: "Bob Herm", company: "Test Corp", city: "Tampa", state: "FL" },
  { name: "James Houston", company: "Test Corp", city: "Dallas", state: "TX" },
];

const options = {
  filterType: 'checkbox',
  resizableColumns:true
};

export default function CatalogueList(){

  return (<MUIDataTable
    title="Employee List"
    data={data}
    columns={columns}
    options={options}
  />)

}

The same happens when exporting to pdf. Even with wide screen size and correct rendering in the browser, the generated pdf shows the same issue of column headers leaking into each row

I don't think that I can isolate parts of my project setup and build a trace of when the app breaks the table. I wouldnt know where to start and it seems quite a bit of work. Have you ever seen this behavior before?

Unfortunately, I can't be of much help without seeing and interacting with your code. There are too many possibilities. Unless you can show a simple example that breaks, there is a good chance that the issue is not specifically in the library, but in the way it is being utilized in your project.

Understood. Was just giving it a try since it might have been a known issue.


Match unified NDK headers

Before Android NDK r14b, each android api version has their own libc headers set which were not totally correct.
Since Android NDK r14b, Google unified the headers so that they apply to all api version (using #ifdef when needed to differentiate between mismatching versions). These new headers are directly taken from the bionic source which was not apparently the case before.
Unfortunately, this crate was based on these not unified headers and contain some incompatibilities with the new correct headers.
For now, we manage to update the NDK used to r15b by skipping the tests of the functions and structs that have changed. But ideally, we should update these functions to match the new headers for the next major version of libc.
So, for new symbols, please do not look into $NDK_PATH/platforms/android-xx/arch-arm/usr/include for reference but into $NDK_PATH/sysroot/usr/include as this path contains the new headers.
Here is a list of the changes written in C that we need to cover for the next major release:

int getpriotity(int, int) --> int getpriority(int, id_t) (id_t is uin32_t and is already defined as libc::id_t)

int setpriority(int, int, int) --> int setpriority(int, id_t, int)

int personality (unsigned long) --> personality(unsigned int)

stat struct is probably a bit harder to fix:
These fields:
long st_atime;
unsigned long st_atime_nsec;
long st_mtime;
unsigned long st_mtime_nsec;
long st_ctime;
unsigned long st_ctime_nsec;

became:
struct timespec st_atim;
struct timespec st_mtim;
struct timespec st_ctim;

where timespec is defined as:
struct timespec {
  time_t tv_sec;
  long tv_nsec;
};

and some #define allow to still have fields like st_mtime_nsec:
#define st_atime st_atim.tv_sec
#define st_mtime st_mtim.tv_sec
#define st_ctime st_ctim.tv_sec

#define st_atime_nsec st_atim.tv_nsec
#define st_mtime_nsec st_mtim.tv_nsec
#define st_ctime_nsec st_ctim.tv_nsec

#define st_atimensec st_atim.tv_nsec
#define st_mtimensec st_mtim.tv_nsec
#define st_ctimensec st_ctim.tv_nsec

It means we could just keep the stat struct as is and just change the types of *_mtime and *_nsec to match the nex types and drop the new fields st_atim (without final e) and st_atimensec an alike (without the underscore before nsec) or find a way to present the field with different names.

int strerror_r(int, char*, size_t) --> char* strerror_r(int, char*, size_t)
The old function is still present and accessible through the symbol strerror_r, the new one is accessible through _gnu_strerror_r. The solution is more probably simply just to ignore the new function and let strerror_r as is.

int madvise(const void*, size_t, int) --> int madvise(void*, size_t, int)

int msync(const void*, size_t, int) --> int msync(void*, size_t, int)

int mprotect(const void*, size_t, int) --> int mprotect(void*, size_t, int)

ssize_t recvfrom(int, void*, size_t, int, const struct sockaddr*, socklen_t*) --> ssize_t recvfrom(int, void*, size_t, int, struct sockaddr*, socklen_t*)

int bind(int, const struct sockaddr*, int) --> int bind(int, const struct sockaddr*, socklen_t) (only applies to android 64 bits, do not know why we bounded it correctly in 32 bits)

int writev(int, const struct iovec *, int) --> ssize_t writev(int, const struct iovec*, int) (same remark, only applies to android 64 bits)

int readv(int, const struct iovec *, int) --> ssize_t readv(int, const struct iovec*, int) (same remark)

int sendmsg(int, const struct msghdr*, int) --> ssize_t sendmsg(int, const struct msghdr*, int) (same remark)

int recvmsg(int, struct msghdr*, int) --> ssize_t recvmsg(int, struct msghdr*, int) (same remark)

Thanks for opening this @ndusart!

This is fixed (or should be) fixed on master.


macOS Monterey. Can't rebuild kext cache. Permission denied.

When I run the utility I get the following code followed by a shower of commands in red:
spctl --master-disable
mount -uw /
mount_apfs: volume could not be mounted: Permission denied
mount: / failed with 66
Can you help me to mount the disk in macOS Monterey?  I have disabled SIP and Gatekeeper and no luck.

UPDATE: I found a good answer to the issue of mounting th edisk as read write at this link:
https://stackoverflow.com/questions/73048614/can-i-mount-the-root-system-filesystem-as-writable-in-macos-monterey
It looks like it's a no go on Monterey. Will you eventually be removing the rebuild kext cache utility since it seems Apple has shut down that option going forward or does that utility still function despite the cascade of red "Read only" prompts in the Hackintool terminal window?


Access is denied

Seems to me that JNA is giving off this exception when trying to access the Jserial dll. Could you please guide me in the right direction on how to solve this? :) Running as administrator does not solve the issue.
dk.thibaut.serial.SerialException: Access is denied.

at dk.thibaut.serial.SerialPortWindows.getLastException(SerialPortWindows.java:92)
at dk.thibaut.serial.SerialPortWindows.<init>(SerialPortWindows.java:99)
at dk.thibaut.serial.SerialPort.open(SerialPort.java:75)
at SerialConnection.<init>(SerialConnection.java:32)

Hello,
This error is returned by windows (it's teh result of CreateFile() Win32 API function). Usually it means that the COM port is used by an other application. Is it the case ?

Tried rebooting to make sure, doesn't seem to be the case unfortunately. I am using a Prolific USB-to-Serial Comm Port, running on Windows 10... Using some very old drivers which seems to work (2008).
Maybe my setup is at fault here.

If you try with an other software like putty or termite, does it work ?
Le 28 oct. 2015 17:45, "Poul Erik Venø"<EMAIL_ADDRESS>a écrit :

Tried rebooting to make sure, doesn't seem to be the case unfortunately. I
am using a Prolific USB-to-Serial Comm Port, running on Windows 10... Using
some very old drivers which seems to work (2008).
Maybe my setup is at fault here.
—
Reply to this email directly or view it on GitHub
https://github.com/thibautd/JSerial/issues/1#issuecomment-151905722.

I just realised that you where indeed correct before - I am using the port twice. I am shortcircuiting pin 2 and 3 in order to test communication on the same com port.
Tried not opening both a read and a write and the program launched without any issues at all.
Im sorry for wasting your time :)

No problem ! I think you are the first user of JSerial, so I'm glad to see that you're interested in the project, enjoy :-)


[FALSE-POSITIVE]

Domains or links
https://adclicker.io/
More Information
How did you discover your web site or domain was listed here?
Incorrectly marked as Phishing on Phishtank or OpenPhish?
Have you requested removal from other sources?
CyRadar
Sophos
G-Data
SOCRadar
Additional context
Hello, I removed the false positive pishing/malicious content from the URLs. All the files have been cleared and the website is secured now.
The identified URLs that were malicious are as follows:
https://adclicker.io/api/v1/links/1P6OuL
https://adclicker.io/api/v1/links/wLHj3l
https://adclicker.io/f/wLHj3l
https://adclicker.io/f/iYi4Vb
The official website is
hxxps://adclicker[.]com
Please could you check it again,
:exclamation:
We understand being listed on a Phishing Database like this can be frustrating and embarrassing for many web site owners. The first step is to remain calm. The second step is to rest assured one of our maintainers will address your issue as soon as possible. Please make sure you have provided as much information as possible to help speed up the process.
Send a Pull Request for faster removal
Users who understand github and creating Pull Requests can assist us with faster removals by sending a PR to mitchellkrogza/phishing repository, on the falsepositive.list file

https://github.com/mitchellkrogza/phishing/blob/main/falsepositive.list
Please include the same above information to help speed up the whitelisting process.

It's done? Or do I need some more steps like creating a pull request or something?


[path_provider] Update README to list Android support for getDownloadsDirectory

What package does this bug report belong to?
path_provider
What target platforms are you seeing this bug on?
Android
Have you already upgraded your packages?
Yes
Dependency versions
path_provider: ^2.1.2

Steps to reproduce
https://pub.dev/packages/path_provider#supported-platforms-and-paths
https://pub.dev/packages/path_provider_android/changelog#220
Android has supported getDownloadsDirectory() but document isn't update.
Expected results
Update pubspec.yaml of path_provider (link) to using path_provider_android: ^2.2.2, update doc and type of getDownloadsDirectory().
Actual results
It's still path_provider_android: ^2.1.0 now.
Flutter Doctor output
Doctor output
[√] Flutter (Channel stable, 3.16.3, on Microsoft Windows [版本 10.0.22631.3007], locale
    zh-CN)
    • Flutter version 3.16.3 on channel stable at D:\Flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision b0366e0a3f (8 weeks ago), 2023-12-05 19:46:39 -0800
    • Engine revision 54a7145303
    • Dart version 3.2.3
    • DevTools version 2.28.4

[√] Windows Version (Installed version of Windows is version 10 or higher)

[√] Android toolchain - develop for Android devices (Android SDK version 34.0.0)
    • Android SDK at C:\Users\28642\AppData\Local\Android\Sdk
    • Platform android-34, build-tools 34.0.0
    • ANDROID_HOME = C:\Users\28642\AppData\Local\Android\Sdk
    • Java binary at: D:\Program Files\Android\Android Studio\jbr\bin\java
    • Java version OpenJDK Runtime Environment (build 17.0.7+0-b2043.56-10550314)
    • All Android licenses accepted.

[√] Chrome - develop for the web
    • Chrome at C:\Program Files\Google\Chrome\Application\chrome.exe

[√] Visual Studio - develop Windows apps (Visual Studio Community 2022 17.7.4)
    • Visual Studio at C:\Program Files\Microsoft Visual Studio\2022\Community
    • Visual Studio Community 2022 version 17.7.34031.279
    • Windows 10 SDK version 10.0.22000.0

[√] Android Studio (version 2023.1)
    • Android Studio at D:\Program Files\Android\Android Studio
    • Flutter plugin can be installed from:
       https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
       https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 17.0.7+0-b2043.56-10550314)

[√] Connected device (3 available)
    • Windows (desktop) • windows • windows-x64    • Microsoft Windows [版本 10.0.22631.3007]
    • Chrome (web)      • chrome  • web-javascript • Google Chrome 120.0.6099.216
    • Edge (web)        • edge    • web-javascript • Microsoft Edge 121.0.2277.83

[√] Network resources
    • All expected network resources are available.

• No issues found!

Close by https://pub.dev/packages/path_provider/changelog#214


feat(ui): Response Limit

Hold on this story for now. Still figuring out what the limit should be

User Story
As a LFAI user
I want to know when the response has reached it's limit and be able to continue
So that I can have longer responses
Acceptance Criteria
Given a responses is being generated
When the response limit is reached (Response Limit: ??? characters)
Then the response stops
And a "Continue Generating" button appears
When I click on the "Continue Generating" button
Then the response continues to generate in the existing response container
When I DO NOT click on the "Continue Generating" button
And I send a new prompt message
Then the "Continue Generating" button disappears
Describe alternatives you've considered
(optional) A clear and concise description of any alternative solutions or features you've considered.
Additional context
Design link: (incoming)

Continue Response Generating Button link
Response Generation Continues link

blocked by #463

Seconding this - ran into this testing today with some simple yaml generation (also seemed to strip spaces / not understand indentation?)

@Racer159 Thanks for seconding this!! As an interim solution you can type "continue" after the response gets cut off and it will continue generating the previous response in a new response block. Hope this help until we get this addressed.

The default LeapfrogAI Bot assistant seems to have a response token limit. If you create a new assistant without RAG (or with RAG) you do not get a response limit.
Default assistant (LFAI Bot):

Non-RAG assistant:

Some of the comments on this issue are in reference to a bug (default parameter) which will be resolved by the following PR. The original intent of this issue is to implement a continue generation button, which wasn't entirely clear from the title. For those encountering this bug (@unicorn-kp @Racer159) hopefully we'll get this addressed in the next release!
https://github.com/defenseunicorns/leapfrogai/pull/699


COLLECTIONS-661: fix for concurrency issue in HashSetValuedHashMapTest

The getMap() method, when testing a HashSetValuedHashMap, would return an object of this type. Which is an instance of SetValuedMap.
Running it in debug mode would - most of the times - run the tests and succeed. Running normally - especially on Windows - would result in intermittent, but very frequent, failures.
The getMap() method sometimes, depending on the order and execution of tests, will return null. So the collection added to the map will be either a Hashset (when it is not null), or a Arrays$ArrayList (otherwise). When the types are different, hashCode() and equals() calls return incorrect values, resulting in the errors we have seen in COLLECTIONS-661.
A good solution would be to re-design the tests. The TestMultiValuedMapAsMap is testing MultiValuedMap's, which include HashSetValuedHashMap. However, some of its methods contain extra logic for when the type under test has some characteristics like being an instance of SetValuedMap.
It might be possible to come up with a better design, where there are multiple test classes, for MultiValuedMap's that use SetValuedMap's; MultieValuedMap's that use List's, and so it goes.
Or we could add a class to the parent class, with a flag defining the type under test. For now, I have used the makeObject() method, which returns the collection under test. Then I validate its instance type. There is also a comment above the code to indicate why we are using makeObject() and not getMap().
It was a fun ticket. Happy to get feedback on better solutions, or feel free to edit this pull request if you have right to it, or merge if you are happy and it has gathered some consensus.
Cheers,
Bruno

Coverage increased (+0.008%) to 85.126% when pulling 6e8951ed0325abe3e07e32aded0b27aacdbc1011 on kinow:COLLECTIONS-661 into 1d21a49c27d9eab8d02785a783fcfba387a3e8e1 on apache:master.

Merged inhttps://github.com/apache/commons-collections/commit/eed8a7adb3de7441751e846f4a75d110dd205f23. Closing it.


Make emulator connection retries configurable

We're seeing test flakiness when using the Bigtable and Pub/Sub emulators in CI. As far as we can tell this is caused by the connection timing out. I've tested this change in our CI (by bumping the timeout to 30 seconds) and it fixes the issue.
There are almost certainly better ways to pass this config in, so please feel free to suggest improvements.

Sure, it makes sense to me that this be configurable. At some point it probably makes sense to switch to a builder-style configuration, like Emulator::new().project("hello").connection_limit(100).build(), but maybe there aren't yet enough configuration parameters to justify it...

+1 on a builder. Those constructor names get unwieldy over 2 params IMO

Yep, makes sense. I'll try to find time to make that change today. Thanks folks.

I updated the code to use builders, and I agree that it's much nicer.
I did it in two different ways, split across two different commits. The first time I essentially just copied the way the existing constructors worked, which you can find in 372ca8c.
While doing this, I noticed that the default values for the project & instance only really made sense for the test suite in this repository. It seemed a little odd to have Emulator::new().await return something that was actually very unlikely to be what consumers wanted, so I removed the defaults and then used typestates to force consumers to provide a project (and, in the case of the bigtable emulator, an instance). I think this works pretty nicely, but I deliberately did it in its own commit so that I can drop it in case not everyone is as enamoured by typestates as I am. :grin:

This fails on 1.63 as into_future wasn't stablised until 1.64. Any objections to updating the version on CI?


Fix label lower case

I made the label the lower case. See #3.
I do not know about LoC so please correct it after merging.
(Cargo.lock change is a message automatically inserted by cargo.)

Thank you for your PR! I'm sorry that I didn't get to this sooner as there are now merge conflicts. If you could resolve them and address the feedback I've left. I'd be happy to merge this.

rebased!

fix code lines to lines of code.
You also prefer to me use lines of blanks and lines of comments?

No that's fine. Thank you again for your PR, and congrats on your first contribution! 🎉

OK. Thanks!


fix for CVE-2022-27191

Signed-off-by: Chris Ahl<EMAIL_ADDRESS>For issue https://github.com/stolostron/backlog/issues/21804

/lgtm


I need help with contextTypes.

I want add dynamic height for items of List. And I add in contextTypes :
 list: PropTypes.instanceOf(Immutable.List).isRequired
As in example. But this.context.list: undefined all time. I can't understant what do. Thanks!

https://github.com/bvaughn/react-virtualized/blob/master/source/demo/Application.js#L48-L53
https://github.com/bvaughn/react-virtualized/blob/master/source/demo/Application.js#L68-L76

Hi @Zulfia73,
For contextTypes to work in your component, it will need a parent component that declares a matching childContextTypes.
You can read all about context in the official React docs here: https://reactjs.org/docs/context.html#how-to-use-context


Create 0000-01-02-sophiacheung.md

Add sophiacheung's file

created a branch, created a file and made a commit, and opened a pull request


Adding npm packages one-by-one

This is a change to resolve #31
Each package sourced from the standard registry is added individually, to give best compatibility for the various problematic versions of npm 3.

Thanks for that, great job!


chore: update docker to v25

This depends on https://github.com/openshift/source-to-image/pull/1140

/test all

/approve
/lgtm

Feel free to unhold, adding it just in case anyone wants to look at the PR.
/hold

PTAL @gauron99 @lkingland @matzew

/unhold


Partial Chromatic Scan and Storybook update to 7.1.0 ;; WLFE-82

Also:

Make the Token Colors story display all colors without paging

Issue: https://gsoftdev.atlassian.net/browse/WLFE-82

Light Colors:

Dark colors:


Remove rounding of utm values

I don't know why the variables 'utmEasting' and 'utmNorthing' are rounded. This commit remove the Math.round() function and leave the precision as is.

I would be happy to reopen the PR and help you in maintaining the code. I am closing the PR for now.


Travis: only run standard, not tape

If you run npm test in Travis, you get this message:
Test failed.
Command failed: /bin/sh -c which codesign

Oddly enough, it doesn't fail (even though it should). This PR just runs standard so contributors don't get confused by the Travis output.
Let me know if there's a more idiomatic Node way of doing this.
References #18.

@sethlu I would just merge this myself, but I didn't know what direction you wanted to take with this.

npm needs to be called for the test because standard is not in PATH. The alternative is to have script be:
script: $(npm bin)/standard

Thanks @malept!
Yea, standard's not globally available in Travis. :+1:  I'll merge this now.


Robots.txt for prod

I think it's safer if you merge from develop -> master in this repo, to make sure the changes have gone through the pipeline. i.e. dont fork for this? I might be wrong.

Reason for this process was that SEO content is on staging that isnt ready to be on Prod


StaticFiles extension serving truncated file for symbolic link

(Moving from this issue)
I have a SPA app that uses an config.json file with the following contents:
{
   "api": {
     "baseUri": "http://<IP_ADDRESS>:30081/api"
   }
}

In the Startup.cs file I configure StaticFiles:
    public void Configure(IApplicationBuilder app, IHostingEnvironment env, ILoggerFactory loggerFactory)
    {
      ...

      app.UseDefaultFiles();
      app.UseStaticFiles();
    }

I'm deploying the app in a docker container (FROM microsoft/aspnetcore:1.1) using Kubernetes. In the Kubernetes Pod I map a configMap volume to override the config file. Kubernetes creates a symlink in the contianer and the file contents are correct within the container:
host ~> kubectl exec -it $podname -c frontend /bin/bash
container:/app# ls -l wwwroot/config/config.json
lrwxrwxrwx 1 root root 18 Jun 13 15:30 wwwroot/config/config.json -> ..data/config.json
container:/app# cat wwwroot/config/config.json
{
    "api": {
        "baseUri": "http://kubernetes-value:30081/api"
    }
}

However, doing a curl only returns 18 bytes:
host ~> curl http://<IP_ADDRESS>:30080/config/config.json
{
    "api": {
host ~>

I think that the StaticFile is seeing the file length as 18 (the length for the symlink, which is the length of the path to the physical file: in this case, ../data/config.json which is 18 chars) and only serving that length even though the file proper is longer.
I don't think this is expected behavior - but perhaps there is some other setting that I should use for StaticFiles?

I doubt StaticFiles is doing any special here. I'd bet you could repro the issue using IFileProvider directly. See IHostingEnvironment.WebRootFileProvider and see if you can read your file as expected.

Thanks @Tratcher - I'm busy digging in now. The FileProvider is providing the length field and StaticFiles is returning 0 - length bytes in the response. I suspect that the length property is being set incorrectly for symlinks. Tracking down the point in the clr where the length is read from the OS to see if there's a workaround.

Yep @Tratcher , I've confirmed it. If you do
var fi = new FileInfo("path/to/symLink");

then fi.Length is the length of the symlink (as described above), not the lenght of the file! I think this is a bug on the FileInfo object for Unix.

Here is the change that made unix FileInfo not follow symlinks and a comment about why https://github.com/dotnet/corefx/pull/5020#r47822345
TLDR;
FileInfo behavior on Unix matches Windows behavior for symlinks.

Thanks @brennanconroy. So is there an IFileProvider that uses Stat for symlinks so that I get the expected length? I could roll my own if there isn't one, but was hoping you might know of one before I start hacking ☺!

Turns out it was pretty easy to write my own IFileProvider to work around the symlink length issue.
Once that was in place, I just changed Startup.cs:
      app.UseStaticFiles(new StaticFileOptions()
      {
          FileProvider = new SymlinkFileProvider(env.WebRootPath)
      });

Now all is well!


Data Update

the latest data update is 04-04-20?

Hi @tiagoalmeida10, Is this in your local setup or on our hosted version? Ive just checked the hosted version and it seems to be updating ok?

Hi @FortDigital. My installation is local.
I also ran the python script manually.
Tks

Which of the python scripts is this? If its the alternative dataset script then i have an issue open with the data owner as their files have stopped updating. ive had a few issues with that particular datasource so will probably be dropping it soon in favour of another one im working with.

I'm using the covid19.py script. I will try another script.
Tks

Do you get any sort of error when you run the script manually? The only thing that comes to mind is that the 5th was the start of BST, and the script uses a GMT timestamp, this shouldn't stop it from working though as far as i can tell.

@FortDigital the covid19 script runs without error but after a few seconds it stops without updating the dashboard

@netrosec  the covid19 script runs without error but after a few seconds it stops without updating the dashboard

Try setting cron to something like this.
10 17 * * * /usr/bin/python3 /opt/covid/covid19.py >/dev/null 2>&1
The script runs 10min after 5PM local time.  The dataset is updated (from JHU)  at 23:59UTC, which for me, is 4:59PM PST.  See if that runs.


How to treat duplicate data in Item Similarity Recommender

What's the recommended way of treating duplicate data points when I am trying to train an item_similarity_recommender? You can assume my model is for recommending topics to readers and some readers may have read the same topics multiple times and therefore the dataset contains duplicates. For example ["User1", "Topic1"], ["User1", "Topic1"], ["User1", "Topic1"], should I keep them repeated in the data or should I only keep one ["User1", "Topic1"]?

Hello @dogahe,
The item similarity recommender should handle multiple inputs, but by default duplicate entries won't have an effect on the resulting model.  If you want them to mean something, then scoring each user-item pair with something that increases with the number of hits could give you some gains.  For example, one interaction is 0.8, two is 0.9, 3 is 0.95, etc.


Add read free function to read some bytes

Hi!
I've found that there is a way to read from the socket either until EOF or exact number of bytes, but there is no way to read unknown number of bytes. This PR fill that gap.
Usage:
let rdbuf = [0; 4096].to_vec();
read_some(rd, rdbuf).and_then(|(rd, rdbuf, nread)| {
    println!("read {} bytes: {:?}", nread, &rdbuf[..nread]);
    Ok(())
})

If everything is OK, I'll add documentation.

Perhaps this should be generalized to read_at_least(), which would take a usize signifying the minimum number of bytes desired?

I have no strong opinion about your suggestion. Waiting for maintainer's word.

Thanks for the PR! We may want to actually stick to the standard library conventions and just call this read instead of read_some? I forget what the conclusion was about read_at_least was in the standard library, but it might be relevant here as well...

Just curious, is there any reason to introduce free functions instead of some kind of:
trait AsyncRead: Read {
    fn read<T>(self, buf: T) -> impl Future<Item=(Self, T, usize)>
        where T: AsMut<[u8]>;

    fn read_to_end<T>(self, buf: T) -> impl Future<Item=(Self, T)>
        where T: AsMut<[u8]>;
}

?

I'd personally prefer to avoid prematurely adding traits, but we could consider that at a later date yeah.

@3Hren what do you think about renaming this to read?

Sorry, completely forgot about this PR. Fixed!

Fixed again!
Look, is there any way to try to read from the R immediately (not after a future is polled), i.e. directly inside the read function?

Thanks!

What's the motivation for removing the EOF? Now users will have to check for it everywhere (and possibly hit infinite loops until they spot this, true story)

Just like in std::io::Read::read and elsewhere in linux world.

Yeah this is the same as the standard library as @3Hren mentioned, which is what this is intended to mirror.


[cephfs] stale subvolumes in the CI run

Jun  7 07:45:29.594: subvolumes [[{nfs-export-eb40d884-e62b-11ec-932d-2a8066a6beb8} {csi-vol-5fac3991-e62d-11ec-8ee0-7aaed4ab8f87}]]. subvolume count 2 not matching expected count 1

Looks like we got a bug? logs at https://jenkins-ceph-csi.apps.ocp.ci.centos.org/blue/organizations/jenkins/mini-e2e_k8s-1.23/detail/mini-e2e_k8s-1.23/1899/pipeline
Originally posted by @Madhu-1 in https://github.com/ceph/ceph-csi/issues/3148#issuecomment-1148537147

closing it, looks like jenkins job doesnt have any logs


Feral Combat Training Flurry

I have a bloodrager monk, with the Abyssal bloodline. Feral combat training does not allow the use of my claws with flurry when raging. The other damages and abilities appear the same, however I only have 3 attacks when raging (with claws), and 5 attacks when not-raged (without claws)
I tried rebuilding the character using the training and toybox to toggle the feat with no positive result.
Quickly built a feral warpriest/monk. The static claws also did not benefit from flurry of blows.
Seems that aspect of the feat does not work as written.

It's the third time this topic comes up. But the first time it's not about getting extra attacks. People assume this would get them 5 weapon attacks and additionally their natural attacks.
By RAW and RAI you would get 5 claw attacks and they do get the normal BAB penalty (0/0/-5/-10/-15/-20).

There are a lot of combination of features and no way to select which weapon attacks a character should do. E.g. you would be able to do 5 claw attack. Or instead 5 bite attacks.
So it's rather difficult to come up with a smart way to implement this. If you have a suggestion, I am all ears.

I am sorry if I wasn't clear in my post. The issue is not which weapons I use in those attacks, it is that they do not get 5 natural attacks as RAW/RAI and the feat description. The issue very much is about not getting extra attacks. I tested multiple sources of natural attacks and none worked with FCT/Flurry.
I have very little experience in coding, I understand it can be difficult to find a solution, I just wanted to make you aware of the bug.

I have done a bit of playing with it, even started a new monk just to have a clean test. It does use the claws with the flurry ability now. Even reloaded the previous version.
1.15.17:
unarmed flurry (0/0/-5/-10/-15)
FCT clawed flurry (0/0/-5/-10/-15) (0/-5/-10/-15)
1.15.15:
Unarmed flurry (0/0/-5/-10/-15)
FCT Clawed flurry (0/0)(0)
I hope this clarifies the issue. From my limited experience, I would guess the game engine is at fault for the weird interactions. I should have been more specific from the start, so I do want to apologize. I will play around with the other abilities too over the next couple days as well. I do want to add that I really enjoy your mod and want to sincerely thank you for all the work you have done.

Ah... that's a lot of offhand attacks. Do you have two-weapon-fighting?
Anyway, thanks for testing and you are welcome :heart:

I went through every ability/feat on the character, there is nothing that gives them additional attacks other than the high BAB at that level.


Correct get elastic tensor

Patch

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

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

Totals

Change from base Build<PHONE_NUMBER>9:
0.0%

Covered Lines:
602

Relevant Lines:
614

💛  - Coveralls


defaultDate in datetime_picker does not post value unless changed

Bug report
When adding the defaultDate option to a datetime_picker field type, it does not actually post it in the request unless you change the date.
What I did:
Add the field to the crud:
$this->crud->addField([
    'name' => 'start_at',
    'label' => 'Start date',
    'type' => 'datetime_picker',
    'allows_null' => false,
    'datetime_picker_options' => [
        'defaultDate' => (new Date())->toDateString()
    ],
]);

What I expected to happen:
To get the defaultDate value in the request if the user does not touch it.
What happened:
If the user does not change the date, it will be null
Backpack, Laravel, PHP, DB version:
Backpack: 3.4.8
Laravel: 5.6.20
PHP: 7.2.4
Mysql: 5.7.22

Hi @patrickreck ,
Thanks for reporting this. Never used defaultDate myself, I always use Backpack's default attribute, since it works across all fields - just easier to remember imho.
        $this->crud->addField([   // DateTime
            'name'  => 'datetime_picker',
            'label' => 'Datetime picker (jQuery plugin)',
            'type'  => 'datetime_picker',
            'allows_null' => true,
            'default' => '2017-05-12 11:59:59',
            // optional:
            'datetime_picker_options' => [
                'format'   => 'DD/MM/YYYY HH:mm',
                'language' => 'en',
            ],
        ]);

It should work for you with default - just checked. Also, I've just updated the docs to show this (even if commented) so that people understand this is the preferred way to specify a default date. Let me know if it doesn't work for you and we'll reopen the issue.
Thanks, cheers!


"Commitment to open source" and lawsuit

Dear Modest Tree:
I've read your "Commitment to open source" message in the front page of Zenject here in GitHub. I just wanted to ask if that commitment includes the withdrawal of the lawsuit you filled against the main author of Zenject, @svermeulen.
I'm asking this because you simply cannot say you have a commitment to open source while simultaneously sueing one of the authors of that open source tool. It would not only be hypocritical, but also a danger alert to any contributor or forker of your open projects: I contributed in a seemingly "open source" project and somehow got sued for forking the project or trying to take credit for my participation on it.
I (and I believe other users and potential contributors on the project) would appreciate an answer on this matter.

bump!

Good Afternoon,
Thank you for reaching out to voice your concern and to seek clarification.
Zenject is and continues to be licensed by Modest Tree under the MIT
Licenses. As such, users may contribute and fork the project under this
licensing. MIT licensing is very broad and gives a high level of
flexibility. Many publicly known games, including Pokemon Go utilize the
framework and have this integrated into their development environment.
The legal matter referenced is not in relation to actions for being a
contributor to Modest Tree's Zenject.  It arises from signed contracts
between the founders of Modest Tree.  Specifically, there are intellectual
property, non-competition and other protections contained in those
contracts which are essential for us as a small business.
We look forward to you contributing/forking/using Zenject and hope that
provides some comfort. We have and continue to support the open source
community and the use of Zenject widely and encourage the contribution and
forking on Zenject from community users. We appreciate you reaching out and
hope that this addresses your potential concerns.
Kindest Regards,
Emily Smits CPA,CMA, PMP
Chief Operating Officer
Modest Tree
www.modesttree.com | Follow us on LinkedIn
https://www.linkedin.com/company/modest-tree-media-inc and Twitter
https://twitter.com/modesttree
On Tue, Jun 2, 2020 at 12:28 PM Racso<EMAIL_ADDRESS>wrote:

Dear Modest Tree:
I've read your "Commitment to open source" message in the front page of
Zenject here in GitHub. I just wanted to ask if that commitment includes
the withdrawal of the lawsuit you filled against the main author of
Zenject, @svermeulen https://github.com/svermeulen.
I'm asking this because you simply cannot say you have a commitment to
open source while simultaneously sueing one of the authors of that open
source tool. It would not only be hypocritical, but also a danger alert to
any contributor or forker of your open projects: I contributed in a
seemingly "open source" project and somehow got sued for forking the
project or trying to take credit for my participation on it.
I (and I believe other users and potential contributors on the project)
would appreciate an answer on this matter.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/modesttree/Zenject/issues/675, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AA6SAZTLAJST4U7B3NLOP6DRUULBPANCNFSM4NQ2TWWA
.

 April's fools joke.

IANAL and don't know much about it but the pdfs with the lawsuit information from the extenject github page sure make it sound related to the extenject fork

"We look forward to you contributing/forking/using Zenject and hope that
provides some comfort. "
Please give some courtesy and respect to other developers, and put up a link to Extenject, please. This is really irresponsible of an open source initiative, when you haven't developed the code since February of 2019.
That was more than a year ago. Everyone, just go here Extenject


Badge works incompatible in release and debug modes.

Each time the createNotification function is run in debug mode, the badge increases automatically. In release mode, this does not work properly.
I was not sure whether it was a bug or a quest because there was not enough information to use the badge.

@tyypgzl you erased the issue form with some crutial questions that i need to know about your issue, because the badge behaves differently on each platform and Android distribution. I just can't help you.
I suggest you to recreate this issue and answer all the questions propertly. This way i can understand better your case.
I will mark this issue as invalid as it violates the community guidelines to post issues.


Add option to set event id manually

See: https://github.com/snowplow/snowplow-android-tracker/issues/59

Depends on https://github.com/snowplow/snowplow-objc-tracker/issues/223 for ease of implementation.


added sbtserver linter

Per w0rp's request and to make it easier for ongoing vim maintainability/portability, this pull request only supports sbtserver in tcp mode

addresses #1283

active.json can potentially be different every time the sbtserver starts up, especially if there are multiple running servers (i.e., working on two separate projects at once necessitates different ports for the different server). Requiring the user to change the setting manually every time would be poor UX; is there a way to read files in vim asynchronously? If not, I can go back to using the wrapper at https://github.com/Ophirr33/sbtio

I guess there are two other options we could do:

recommend that the user override the serverPort value in their global.sbt to a default that matches the var's default value. This would still break on attempts to run more than one project at a time, however.
We could try to replicate the algorithm they use to generate the port, which is computing the sha1 hash of the directory name, transforming it into a hex string, then converting it into a number via hashcode computation and adding the result % 1000 to 5000: https://github.com/sbt/sbt/blob/4c3cd4e535a89d220a0d20f360f36212ceff9b09/main/src/main/scala/sbt/Defaults.scala#L288

Thoughts on what would be best for me to implement?

Well at least for now I'll just leave it as a variable and recommend that the user overrides their global sbt config to connect to the specified port. It still won't work out of the box for multiple concurrent projects, but I can set up a separate pull request for that later.


[stable/mongodb] Disable IPv6 by default

What this PR does / why we need it:
This PR disable the IPv6 support by default by setting to false the mongodbEnableIPv6 variable.
Mainly, this variable set the ipv6 parameter to true or false in the mongodb.conf file:
# network interfaces
net:
  port: 27017
  unixDomainSocket:
    enabled: true
    pathPrefix: /opt/bitnami/mongodb/tmp
  ipv6: true
  bindIpAll: true

In some k8s cluster, mongodb service is not running because of the following issue (mongodb.log):
2019-06-21T14:09:17.987+0000 I CONTROL  [main] Automatically disabling TLS 1.0, to force-enable TLS 1.0 specify --sslDisabledProtocols 'none'
2019-06-21T14:09:17.992+0000 I CONTROL  [initandlisten] MongoDB starting : pid=36 port=27017 dbpath=/opt/bitnami/mongodb/data/db 64-bit host=carlosrh-mongo-mongodb-76bc884646-ztpxl
2019-06-21T14:09:17.992+0000 I CONTROL  [initandlisten] db version v4.0.10
2019-06-21T14:09:17.992+0000 I CONTROL  [initandlisten] git version: c389e7f69f637f7a1ac3cc9fae843b635f20b766
2019-06-21T14:09:17.992+0000 I CONTROL  [initandlisten] OpenSSL version: OpenSSL 1.1.0j  20 Nov 2018
2019-06-21T14:09:17.992+0000 I CONTROL  [initandlisten] allocator: tcmalloc
2019-06-21T14:09:17.992+0000 I CONTROL  [initandlisten] modules: none
2019-06-21T14:09:17.992+0000 I CONTROL  [initandlisten] build environment:
2019-06-21T14:09:17.992+0000 I CONTROL  [initandlisten]     distmod: debian92
2019-06-21T14:09:17.992+0000 I CONTROL  [initandlisten]     distarch: x86_64
2019-06-21T14:09:17.992+0000 I CONTROL  [initandlisten]     target_arch: x86_64
2019-06-21T14:09:17.993+0000 I CONTROL  [initandlisten] options: { config: "/opt/bitnami/mongodb/conf/mongodb.conf", net: { bindIpAll: true, ipv6: true, port: 27017, unixDomainSocket: { enabled: true, pathPrefix: "/opt/bitnami/mongodb/tmp" } }, processManagement: { fork: false, pidFilePath: "/opt/bitnami/mongodb/tmp/mongodb.pid" }, security: { authorization: "disabled" }, setParameter: { enableLocalhostAuthBypass: "true" }, storage: { dbPath: "/opt/bitnami/mongodb/data/db", directoryPerDB: false, journal: { enabled: true } }, systemLog: { destination: "file", logAppend: true, logRotate: "reopen", path: "/opt/bitnami/mongodb/logs/mongodb.log", quiet: false, verbosity: 0 } }
2019-06-21T14:09:17.995+0000 I STORAGE  [initandlisten] exception in initAndListen std::exception: open: Address family not supported by protocol, terminating
2019-06-21T14:09:17.996+0000 I CONTROL  [initandlisten] now exiting
2019-06-21T14:09:17.996+0000 I CONTROL  [initandlisten] shutting down with code:100

This message is very descriptive:
2019-06-21T14:09:17.995+0000 I STORAGE  [initandlisten] exception in initAndListen std::exception: open: Address family not supported by protocol, terminating

Although the host has disabled IPv6, it is expected that MongoDB will try to use IPv4 in its absence, in fact, we tried it when using this parameter for the first time, so why in some k8s cluster works and not in others?
We can disable IPv6 support by running
sudo sysctl -w net.ipv6.conf.*.disable_ipv6=1

in this case, mongodb server does not complain about unsupported address family when IPv6 is enabled in the configs. This reflects with our earlier findings as we had used the same method to disable IPv6.
Continuing on our investigation, we found that the alternate way of disabling IPv6 support is to pass ipv6.disable=1 as a bootarg to the Linux kernel and with this, we were able to see the unsupported address family error with mongodb.
With these findings, we cannot make the assumption that IPv6 enabled in the mongodb configs by default will always work regardless of whether the underlying host supports IPv6 or not.
Which issue this PR fixes

https://github.com/bitnami/bitnami-docker-mongodb/issues/108
https://github.com/kubeapps/kubeapps/issues/577
https://github.com/helm/charts/issues/14002
https://github.com/jenkins-x/jx/issues/1597

Special notes for your reviewer:
We are working to use false as default value in the image itself, but we can merge this PR without waiting to the image changes as this is an env. variable that we're passing to the image and the image already support both values, we are only changing the default value.
Checklist
[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

[X] DCO signed
[X] Chart Version bumped
[X] Variables are documented in the README.md
[X] Title of the PR starts with chart name (e.g. [stable/chart])

/ok-to-test
/lgtm


error CS1061: 'Task' does not contain a definition for 'Result'

Hello, can you share with me more info? Thankss

Hi David
I think the post got deleted by mistake. I am using Unity 2022.3.20  on chapter 6 Adding Registration and our users.
I knew getting the following errors ;
Assets\Scripts\SignUpManager.cs(69,36): error CS0029: Cannot implicitly convert type 'Firebase.Auth.AuthResult' to 'Firebase.Auth.FirebaseUser'
Assets\Scripts\SignInManager.cs(58,62): error CS0103: The name 'existingUser' does not exist in the current context
Assets\Scripts\SignUpManager.cs(69,41): error CS1061: 'Task' does not contain a definition for 'Result' and no accessible extension method 'Result' accepting a first argument of type 'Task' could be found (are you missing a using directive or an assembly reference?)
I have tried different solutions I believe it might be a version issue.


Serialization of internal stores

Now any internal store of this library serialized in SSR-flow.
Shall we add { serialize: 'ignore' } to internal stores?

I think not. Can't come up with use case for using history on server side, but it can cause state inconsistency if someone will do it

Imagine the situation
const $data = createStore(new Date()) // or other non-json structure

const history = createHistory({
  source: {
    data: $data,
  },
  maxLength: 20
})

After serialization and deserialization cycle, client will find invalid state in history and after undo-call it will propagate to original store.

Got it. But I think it'd better to pass serialize as an option for createHistory so users can decide whether they need it or not


Timeouts in scrapyjs

I am a little nervous to post this but can you please see at this SO question:
http://stackoverflow.com/questions/34656424/scrapyjs-example-from-official-github-not-running
I exactly have the same problem. It constantly times out. Though I used solutions from these:

http://stackoverflow.com/questions/30345623/scraping-dynamic-content-using-python-scrapy/30378765#30378765
http://stackoverflow.com/questions/32141967/how-can-scrapy-deal-with-javascript

Even opening splash in browser and rendering a simple page like google.com fails as well. I just pulled the new image for docker but it is not working.
PS. I am nervous because with everyone else it seems to work fine. :)

Hey @shafaq,
It looks like either Splash configuration error or a problem with a website. Maybe you're trying to access Splash at a wrong IP address, or maybe you haven't exposed the port when running docker container. I don't think scrapy-splash may cause this timeout; Splash bug tracker (or Splash gitter chat) is a better place to discuss this issue.


Retrain Tokenizer for Python

https://huggingface.co/learn/nlp-course/en/chapter6/2
Follow this:

Load GPT-2 Tokenizer
Create iterator dataset for our training data
Train GPT-2 Tokenizer with iterator
Save weights to file

Draft: https://github.com/MichiganDataScienceTeam/F24-mini-copilot/commit/b4f1f3cc0f6c046cf3d747ce09d056fc48cf53a2

Is this completed and good to close?

yes


Wattesigma on Linux

for the Linux users out there, I was able to run the Windows build using bottles after changing the runner to kron4ek-wine-9.11-stable. My system's wine-8.2 did not work (app crashed on launch), and the bottles default of soda-9.0 didnt work either (CEF failed to load). If you try to run this without bottles, make sure you use a 64-bit Wine build.

Yes, you're right the GitHub CI runs on 64-bits archi. Not sure how to set the CI to compile for 32-bits (Linux/Windows) and if this makes sense nowadays to keep maintaining 32-bits.

Linux support will not be added to the Releases tab, although you'll be able to compile the cef_artifcts for Linux hopefully soon, which will allow you to use the browser normally if you export it as an executable file through Godot.


Installation problem

Hello,
I would like to ask if there's something I missed out because i have tried both install method, but I can't make the command work. I attach the pict of my terminal here. I hope you can give me some advice on how to fix this. Thank you very much for your help.

Hello, after installing the required libraries, you have to clone the repo and run pip install as the next step

Mark as resolved


Fiat value of yCurve token is 0

Describe the bug
Fiat value of yCurve token is 0
To Reproduce
https://safe-transaction.gnosis.io/api/v1/safes/0xFEB4acf3df3cDEA7399794D0869ef76A6EfAff52/balances/usd/
The response at the time i'm writing the ticket:
[
  {
    "tokenAddress": "0xdF5e0e81Dff6FAF3A7e52BA697820c5e32D806A8",
    "token": {
      "name": "Curve.fi yDAI/yUSDC/yUSDT/yTUSD",
      "symbol": "yDAI+yUSDC+yUSDT+yTUSD",
      "decimals": 18,
      "logoUri": "https://gnosis-safe-token-logos.s3.amazonaws.com/0xdF5e0e81Dff6FAF3A7e52BA697820c5e32D806A8.png"
    },
    "balance": "139276488795081812837140",
    "balanceUsd": "0.0",
    "usdConversion": "0.0"
  }
]

Expected behavior
The user expected the value to be somewhat close to this source https://www.coingecko.com/en/coins/lp-ycurve
Additional context
Reported by our user in telegram https://t.me/GnosisSafe/6102

I would say this is a feature request. We should check if we have alternative oracles or conversion rate source (e.g. Zerion or coingecko). Currently we expect that a token has a price either on Kyber, Uniswap or Kraken.

I would say this is a feature request. We should check if we have alternative oracles or conversion rate source (e.g. Zerion or coingecko). Currently we expect that a token has a price either on Kyber, Uniswap or Kraken.

Automatic price estimation uses Kyber and Uniswap. Kraken is just used for ETH/DAI. I've checked and there's no price for that token on those platforms, so we need to decide if more oracles must be added to the service, but this would be another ticket. What do you think @tschubotz @lukasschor , is this required?

Hmm. I think it would be nice to have prices for most used tokens.

This has come up quite often recently where tokens (especially DeFi / Pool tokens) where valued at 0 USD. It would indeed be good to have more price oracles to cover these.

There is some research requires to find potential oracles.
New potential oracles:

Uniswap v2
...

New potential oracles:

Uniswap v2

FYI, Uniswap V2 was added 2 weeks ago. Now the oracles supported are:

Kraken (for ETHUSD)

Are we using Kraken for ETHUSD or for DAIUSD? (https://gnosisinc.slack.com/archives/CQB729LLF/p1601303396122100?thread_ts=1601303359.121900&cid=CQB729LLF)

I recently checked how many prices are missing.
https://docs.google.com/spreadsheets/d/1AdIG21NLmTqrvKkG0w3guWMFGURZsbEqzK_VsZ8I0gE/edit#gid=0 From the top 100 tokens in Safes (by number of Safes that hold this token), only aUSDC and idleUSDCYield don't have a price. So currently, I would not consider this important enough to add additional oracle. Hence I would close this ticket until this changes and/or more users provide feedback that they are missing a token price.

Are we using Kraken for ETHUSD or for DAIUSD? (https://gnosisinc.slack.com/archives/CQB729LLF/p1601303396122100?thread_ts=1601303359.121900&cid=CQB729LLF)

I've just checked, it's ETHUSD, sorry for the typo


Visual Editor not working

Hi,
I built a website for a client on Gatsby and I used Storyblok to manage content. After deploying it to Netlify the visual editor is not working saying https://webstarsltd.netlify.app/ refused to connect. I thought maybe Netlify has some issues so I deployed it on Gatsby Cloud but again the same issue https://webstarsfinalmaster.gatsbyjs.io/ refused to connect.
Any workaround for the issue, please?
One thing I notice, For both URLs, if I open up the developer's console I saw an error
You need to provide an access token to interact with Storyblok API.
And a warning
You are not in Draft Mode or in the Visual Editor.
Please help me find the solution.

Try using https://preview-webstarsfinalmaster.gatsbyjs.io/ after hardcoding the Storyblok API key. If that doesn't work please share your gatsby-config.js and any relevant code snippets
P.S. you may want to consider adding a check so that you only load SB bridge when you're in the visual editor for performance sake

@bugattie Did you provide a public API key token in Netlify? https://docs.netlify.com/environment-variables/overview/
It's been a while, so it's not relevant to keep this issue open. But let me know/feel free to re-open this issue if you still face this issue.


Correct and improve documentation for Photosets

I have never written RDoc documentation (I'm a YARD kind of guy) so I apologize if the formatting is a bit strange.

Thanks @maxcal


created layer name GOTOPLOT

src/js/collection.js
223-229 I think we can make this more generic by just adding the layer when a map is created.  In mercator.js, in mercator.createMap, line 648, add the goToPlot layer when the map is created. Use visible: false and zIndex: 100.
Use mercator.setLayerVisibilityByLayerId or mercator.setVisibleLayer to hide / show the layer when necessary.
the layer "id" can be a string.  Make the layer id "goToPlot" insted of -1. -1 in this app means something more like not found, invalid selection, etc.  Its easier to understand what you are doing if the layers id has something meaningful.
JS uses camel case so use goToPlot instead of GOTOPLOT.
268 currentImagery.sourceConfig.type === "PlanetDaily" || currentImagery.sourceConfig.type === "SecureWatch") can be simplified to ["PlanetDaily", "SecureWatch"].includes(currentImagery.sourceConfig.type)
src/js/utils/mercator.js
463-469 I dont think you need a source type here.  You can just add the goToPlot layer when the map is created and hide / show it.
464 Is there a difference between the TileWMS layer and the XYZ layer (see layer 471)? The XYZ layer is one line verses a couple.

src/js/utils/mercator.js
652-656 I think you only need url here.  If you need more than url here, use XYZ layer, it only needs url.

fix #647


evict_expired_blocked_clients

https://github.com/turms-im/turms/blob/c53f2905f6b81cfd9e5896b11a5b040bf47945c5/turms-server-common/src/main/resources/redis/blocklist/evict_expired_blocked_clients.lua#L3
but BlocklistServiceManager#encodeBlockEndTime use millis, evict_expired_blocked_clients will not work

Thanks for reporting this bug! The latest docker images and code of turms servers contain the bugfix, and you can pull them currently.


A possible typo in command for creating application gateway v1

In "Create the application gateway", at "for V1 SKU use the below command", should the command be
$appgw = New-AzApplicationGateway -Name appgateway -SSLCertificates $cert -ResourceGroupName "appgw-rg" -Location "West US" -BackendAddressPools $pool -BackendHttpSettingsCollection $poolSetting -FrontendIpConfigurations $fipconfig -GatewayIpConfigurations $gipconfig -FrontendPorts $fp -HttpListeners $listener -RequestRoutingRules $rule -Sku $sku -SSLPolicy $SSLPolicy -AuthenticationCertificates $authcert -Verbose
in which pool setting is $poolSetting instead of $poolSetting01

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

ID: d404ca02-47ed-d3c9-bf4e-52cd0b0d61ba
Version Independent ID: 5fadfbea-ddcb-8342-32b3-00a8595f3437
Content: Configure end-to-end SSL with Azure Application Gateway
Content Source: articles/application-gateway/application-gateway-end-to-end-ssl-powershell.md
Service: application-gateway
GitHub Login: @vhorne
Microsoft Alias: victorh

@MasaSip , Thank you for your feedback. I will do a PR to fix this issue.

PR: https://github.com/MicrosoftDocs/azure-docs-pr/pull/91414


106 feature extended kalman filter for more precise location measurements

Description
A functioning kalman_filter.py file is now in the perception package.
It is able to beat the older filter by about 50 percent to get about 0.5 meters more accuracy.
It can still be tuned more.
It is also not yet publishing a new current_pos, or heading, which is what i'd like to discuss with you on monday
Fixes #106, #151
Type of change
Please delete options that are not relevant.

Bug fix (non-breaking change which fixes an issue)
New feature (non-breaking change which adds functionality)
This change requires a documentation update

Does this PR introduce a breaking change?
no, every other file stayed the same for now
Most important changes
the kalman_filter.py file is the most important change
Checklist:

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[x] I have added tests that prove my fix is effective or that my feature works (might be obsolete with CI later on)
[x] New and existing unit tests pass locally with my changes (might be obsolete with CI later on)

Review failed checks!


Fix for pre-bound snapshot empty source error

Use GetSnapshotStatus for pre-bound snapshots with nil Source field.

The pre-bound snapshots have  empty Source field in their Spec. Because of it, the call to getCreateSnapshotInput fails when the function tries to get the source volume. If we want to get the status for snapshot without Source field set, we can switch using GetSnapshotStatus.

@hakanmemisoglu We are in the process of creating consistent guidelines for managing forked repos. I'll document those, but in short, we want to keep the master branch unmodified and use that branch to track the original repository's master branch. And use a kasten branch that is local to the forked repo as the branch with the kasten-specific changes.
The workflow would be:

Develop in local branch
Open PR in the Kasten's fork to merge changes to the kasten branch, review and merge.
Open a separate PR (either from the kasten branch or a separate branch) to merge the changes upstream.

Periodically,

The fork's master branch will be synced with the upstream master branch.
The fork's master branch can be merged (via a PR) into the kasten branch to consume upstream changes.

I'll put the document together and tag you.

@hakanmemisoglu @julio-lopez
https://gist.github.com/Chaser324/ce0505fbed06b947d962 seems useful too.

Closing this one... I will send a PR to upstream.


[GOBBLIN-883] Add docker files and compose

Dear Gobblin maintainers,
Please accept this PR. I understand that it will not be reviewed until I have checked off all the steps below!
JIRA

[x] My PR addresses the following Gobblin JIRA issues and references them in the PR title. For example, "[GOBBLIN-883] My Gobblin PR"

https://issues.apache.org/jira/browse/GOBBLIN-XXX

Description

[x] Here are some details about my PR, including screenshots (if applicable):

Adds a DockerFile, docker compose and documentation on how to run the GaaS API Service.
Builds a docker image from the latest source code.
Tests

[x] My PR adds the following unit tests OR does not need testing for this extremely good reason:

Commits

[x] My commits all reference JIRA issues in their subject lines, and I have squashed multiple commits if they address the same issue. In addition, my commits follow the guidelines from "How to write a good git commit message":

Subject is separated from body by a blank line
Subject is limited to 50 characters
Subject does not end with a period
Subject uses the imperative mood ("add", not "adding")
Body wraps at 72 characters
Body explains "what" and "why", not "how"

@zxcware @sv2000 Please take a look, thanks!


linkage_checker: restrict RPATH test to --strict

[x] Have you followed the guidelines in our Contributing document?
[x] Have you checked to ensure there aren't other open Pull Requests for the same change?
[x] Have you added an explanation of what your changes do and why you'd like us to include them?
[ ] Have you written new tests for your changes? Here's an example.

[x] Have you successfully run brew typecheck with your changes locally?
[ ] Have you successfully run brew tests with your changes locally?

This test is causing some rebuilds due to failed linkage upon upgrade.
That's a problem because rebuilds won't fix the problem that the RPATH
check identifies.

Review period skipped due to critical label.

thank you!

Thanks @carlocab!


program stuck (when ”Loading cached shuffled indices for dataset at ...“)

When running the training code, the loading cached shuffled indexes program is stuck? How to solve it?

Same issue here, have you solved it? @ccx06 @sangmichaelxie

Maybe there is some kind of OOM issue or num_workers is set too high? Does this happen every time and on all datasets?

I set the num_workers as 1. I tried to reduce the RANDOM_BATCH_SIZE in dataloader.py but it does not work.
It seems the issue is related to caching
06/25/2024 03:50:31 - INFO - datasets.arrow_dataset - Caching indices mapping at ~/doremi/preprocessed/train/Pile-CC/66/cache-abce09a69c09a6c6.arrow


SAM-3132 - Extended time should validate that user/group is entered w…

…hen initial button is pressed

Okay LGTM


Change config path for vendor:publish

Hello,
how can I change the path for the "vendor:publish" command?
Currently all configs directly published in config-folder. My target is a subfolder, e.g. config/modules/
Greetings.

You set the publishedTo method, to choose where to publish. More on this here: http://laravel.com/docs/5.0/packages#publishing-file-groups

Thanks.


Add fail_with_status host function, fix #146

Addresses #146 by providing a fail_with_status function that users can call to fail the current contract call (trapping its vm) with a user-provided status. The status has to be the newly-added ScStatusType::ContractError with an opaque user-provided u32 body.

cc @jonjove and @leighmcculloch I am posting this to make discussion concrete -- we shouldn't merge this until we have consensus that it's the right thing to do.