download.sh not Working

Issue Type
Bug
OS
Ubuntu
OS architecture
x86_64
Programming Language
Python
Framework
TensorFlowLite
Model name and Weights/Checkpoints URL
https://github.com/PINTO0309/PINTO_model_zoo/tree/main/033_Hand_Detection_and_Tracking
Description

download.sh returns a html file instead of tat file.
Relevant Log Output
resources.tar.gz: HTML document, ASCII text, with very long lines

URL or source code for simple inference testing code
No response

Also, it appears that Google Drive's specifications have changed and it is no longer possible to download files via curl. The specifications of Google Drive change about once every three months, and it is becoming more and more tedious.
I may close the Google Drive of this repository model in the near future as the same issues are posted over and over again.

Workaround

gdown

### e.g.
gdown --id 1ve-bkElmfNra_-EYGUWrQStSQBoXWfWD

Type the URL directly into the browser

https://drive.google.com/uc?export=download&id=1ve-bkElmfNra_-EYGUWrQStSQBoXWfWD


Performance - progressive preview

Would the UI be more responsive if we show a "progressive preview" at lower resolution while we take a higher-resolution screenshot in the background and update it when it's ready?
It would potentially take longer for the higher-res preview to be displayed as we'd have to generate the lower-res one first, but that might improve the user experience if we can show something sooner.
E.g., show this 10x low-res preview within a very short space of being activated:

and replace it with this higher res version as soon as it's available:

Spike a test version and see what it looks like...

Fixed in FancyMouse v0.0.11-preview


Use compute_ctl to manage Postgres in tests.

This adds test coverage for 'compute_ctl', as it is now used by all
the python tests.
There are a few differences in how 'compute_ctl' is called in the
tests, compared to the real web console:

In the tests, the postgresql.conf file is included as one large
string in the spec file, and it is written out as it is to the data
directory.  I added a new field for that to the spec file. The real
web console, however, sets all the necessary settings in the
'settings' field, and 'compute_ctl' creates the postgresql.conf from
those settings.

In the tests, the information needed to connect to the storage, i.e.
tenant_id, timeline_id, connection strings to pageserver and
safekeepers, are now passed as new fields in the spec file. The real
web console includes them as the GUCs in the 'settings' field. (Both
of these are different from what the test control plane used to do:
It used to write the GUCs directly in the postgresql.conf file). The
plan is to change the control plane to use the new method, and
remove the old method, but for now, support both.

Some tests that were sensitive to the amount of WAL generated needed
small changes, to accommodate that compute_ctl runs the background
health monitor which makes a few small updates. Also some tests shut
down the pageserver, and now that the background health check can run
some queries while the pageserver is down, that can produce a few
extra errors in the logs, which needed to be allowlisted.
Other changes:

remove obsolete comments about PostgresNode;
Create standby.signal file for Static compute node;
log output of compute_ctl and postgres is merged into endpoints/compute.log.

I rebased this, and split off the spec file format V2 commit to a separate PR.

LGTM, thanks!


add note that contig64 is broken

Please add a note on the contig sysinternals page that the 64-bit version contig64 is broken. I was consistently receiving STATUS_ALREADY_COMMITTED. The problem is shared by others, as per Google.
This isn't a bugfix request. But a note not to use the 64-bit version would save others after me a lot of pain. I was for a long time convinced it can't be made to work. See background.

Background:
I'm using a 2TB partition for bulk data storage. Bundled Windows defrag utility hangs while analyzing it.
A third-party defrag program skips over two 100 GB files, being preallocated VM disks. The files had lots of small fragments and a great amount of fragments in total.

Thanks for reporting. I have added this to our backlog


drivers: display: st7789v: Add support to disable inversion mode

For historical reasons, inversion mode is enabled by default in the current implementation. This commit introduces an inversion-off boolean DTS property, allowing it to be disabled if necessary.

Wouldn't it be more consistent with other drivers to name the new property display-inversion and have it default to true if you want to preserve backwards compatibility?

Wouldn't it be more consistent with other drivers to name the new property display-inversion and have it default to true if you want to preserve backwards compatibility?

I agree that display-inversion is a more reasonable. But boolean properties do not support default values.


Dart 3.3 extension types don't work with dart_mappable

Using the definition of the SimpleMapper custom Mapper as a reference:
/// An interface to define a custom mapper.
///
/// Implementation should extend this interface and implement the
/// [decode] and [encode] methods.
/// For a generic type with one or two generic type arguments, extend the
/// [SimpleMapper1] or [SimpleMapper2] interface instead, respectively.
///
/// {@category Custom Mappers}
abstract class SimpleMapper<T extends Object> extends _SimpleMapperBase<T> {
  const SimpleMapper();

  T decode(Object value);
  Object? encode(T self);

  @override
  T _decode(Object value, DecodingContext context) {
    return decode(value);
  }

  @override
  Object? _encode(T value, EncodingContext context) {
    return encode(value);
  }
}

Extension types don't seem to extend Object, so you cannot use them as the type parameter T.
And because the return type of decode has to be the same as T, you cannot define the Mapper on the wrapped value instead.

Ok, it turns out it does work, one just has to implement Object on the extension type 🤦


removed ubuntu 22.10 support due to EoL

This fixes the failures in the PR automated checks due to ubuntu 22.10's end of life

This GHA run (from my branch on my fork) shows that it fixes the problem outlined in the issue 774:
https://github.com/jmjaffe37/openjdk-installer/actions/runs/6332563576

https://github.com/adoptium/containers/issues/359 and https://github.com/adoptium/containers/issues/423 are related to this.  I think we need to start a policy doc on the website

Agreed, although I wasn't thinking of a policy (this change is fine), rather the actual description of what is in/out of support for Temurin on Debian - as we do with other OSes.

Heh, had missed that, looks like we never stated we would support 22.10 (which isn't an LTS), I wonder if we mis stepped in adding it in the first place.


Add SCFBI, SCFCEN, and SCFORB handlers to io.scf

Need to add support to the SCF class to read the other simulation output files. Currently only supports SCFPAR (to get units) and SNAP files.

Moved to adrn/scf


Option control renders item list at the wrong location

Hey all, been playing around with Godot for a bit now and ran into a problem regarding the option control. When I use a Node2D as the main scene node, the option item list shows up at the top left of the option control. When using a Control as the main scene node, the option item list shows at the top left the game window. (See screenshots).
The following link is a zip file containing my test project: https://app.box.com/s/sc0bobjgufiz5ok1cey6
Excuse me if the following explanation may be a bit sluggish, I am new to github, and would just like to see this bug solved.
What I've basically done is setup a hierachy in my main scene such as this:
Node2D -> "main"

WindowDialog -> "dialog_mainmenu"
Button -> "button_startgame"
Button -> "button_quitgame"
Optionbox -> "option_difficulty"

After that, I've added code, in the _ready() function of the Node2D -> "main" script, in order to add the items to the option control.

While trying to figure out where the problem lies in C++, I also noticed that with the clean source code, the Tooltip positions incorrectly as well.

Any updates on this yet? Haven't been able to figure it out myself yet due to a lack of better knowledge about the C++ code :)

i got back to the city some days ago and working on fixing several issues,
will get to this one briefly
On Sat, Jan 3, 2015 at 2:27 PM, Mike<EMAIL_ADDRESS>wrote:

Any updates on this yet? Haven't been able to figure it out myself yet due
to a lack of better knowledge about the C++ code :)
—
Reply to this email directly or view it on GitHub
https://github.com/okamstudio/godot/issues/1008#issuecomment-68602332.

well, I was almost certain that this surprisingly wasn't a bug but API misuse, but it really was a strange and obscure bug, so congratulations on your find :P

It's surprising that no one else found it, but I can see why, as you put a Sprite when everyone else would have used TextureFrame (there is not much of a merit mixing 2D and GUI stuff), I just can't believe this bug was around for so long, so i'm glad you reported it.


[FR] Assorted minor feature requests

Thanks a bunch for making this! I'd like to get rid of maintaining my separate script that does  mostly the same thing as this package. There are a two features that would be nice to have here to make that possible.

Fail gracefully when a package fails to build

It's pretty common that a package might fail to build for a certain version of Python or something. Currently, this kills the whole job. IMO it would be nice to catch this and continue through the matrix even if a package fails to build, but then to set the exit status so that the calling script is notified about the failure. IIRC in omnia-md, we set the exit status to the number of failed builds, so if it's nonzero you still get a failure inside your CI system.

Print summary of the packages that will be built.

Currently, the script prints out, for each possible build matrix item, whether or not it will be built here. That's useful but it's a little overwhelming if you have a lot of packages but are using --inspect-channels, so probably very few of them will be built. IMO it would be nice, at end maybe, to print out just the ones that will be built. Or maybe to use color highlighting for the ones that will be built?

Fail gracefully when a package fails to build

Yes, this is something I've thought about too, but not something I've looked into too much. I'm aware that conda-build does a number of sys.exit() calls, but that shouldn't be too hard to deal with. :+1: in being able to do this (as a toggle).

Print summary of the packages that will be built.

Definitely in favour of adding UI improvements. I'm keen to know which packages are not being built too (so that I know I'm not missing some valuable matrix items with my conditions), but there is no reason that the two lists have to be output at the same time. :+1:
Are you interested in getting involved? It'd be great to get more :eyes: on the codebase if so.

P.S. Hope you had a good :christmas_tree: - I was pretty much offline for the duration, so sorry for not replying sooner.


ubuntu 16.04 install video

Hello to everyone. I am using ubuntu 16.04LTS (from aws). I want to build Zenbot, every time I'm doing wrong because I'm not pro user
Is there anyone who wants to take video about the installation on ubuntu 16.04? For installation video, I will share the vps server information and he can share this video on your own YouTube channel ...
email for communication<EMAIL_ADDRESS>As soon as I wait for the return. Best regards..  (sorry bad english, just google translate :)

It can also be an automated script for installation. :)

See #645

I followed those steps but I did not succeed to the line. As I said I am not a pro user :(

those steps are line for line what is required on a new aws ubuntu 16.04 instance. You must have skipped something.

in fact I do not know how to use ubuntu, especially the command line. I would like to use the ZenBot web interface as it is. I need the installation videos for windows or ubuntu (it does not matter). Could you take a video at a time when you are free?

There is no web interface.

:( ok. For ubuntu command line, using video? Because there is no video on the internet

please need it...:(

imho, if you are already having this much trouble typing commands into the command line that I linked to you, you probably want to look for something else because zenbot is entirely command line based.
https://youtu.be/BEhU55W9pBI

Thank you so much, I needed it too. Must have web interface...

thank you @sniper7kills :) :smile:

@ahmethakandd if that solved it for you, you should close this issue

I need to learn zenbot commands urgently : smile:
I do not know how to close issue

Is there a "command line generator"? or do you intend to do it?

Making the web interface may be difficult, but at least such this can be done

@ahmethakandd "I need to learn zenbot commands urgently" You'll want to read though the Read Me. It has all of the commands documented.
Good luck, I won't be able to assist any further.

one last question:
for example, the boat started and putty shutdown. What happens when I reconnect the putty's screen? that is, it will continue again? Is there a command to see the state of the bot?
really respectful

@ahmethakandd I wanted to ask those questions. thank you. I'm waiting for the answer. :smile:

You could use the "screen -S zenbot nodejs zenbot.js whateverzenbotcommand" for the process to persist different login sessions.

And to attach to the process later use "screen -rx"

@halvors Thanks for the tip. I have been looking for this.  So this would work if say I wanted to run paper trading, close the terminal session, and log back on later? Would it be something like:
screen -S zenbot nodejs zenbot.js trade --paper
and in a later session...
screen -rx
Do I have that right?

This issue isn't really about the project but more about how to Linux. There's plenty of tutorials out there to follow...


Revert commits instead of checkout at specific commit

Should fix "being behind" from main repository.

This doesn't matter and is unnecessary. Won't fix.


A fix for tags not mimicking AWS

This is a very simple fix for the tags problem metioned in #2148 be aware a similar problem may occur in other places. It is also not easy to find what it should act like aside from probing boto3 this means a similar problem will no doubt have to be fixed at somepoint in the future.
The fix is increadibly simple all it does is remove tagSet from the response when there are no tags to be contained within it. This means it acts the same as AWS and by virtue boto3.

Coverage remained the same at 92.93% when pulling c7b0905c236ad0c1fad1c2418d1610196e9eccf7 on TBurnip:master into 66dafc6679e979c8ad102c0a782d9731179945c6 on spulec:master.

Coverage remained the same at 92.93% when pulling c7b0905c236ad0c1fad1c2418d1610196e9eccf7 on TBurnip:master into 66dafc6679e979c8ad102c0a782d9731179945c6 on spulec:master.

Coverage remained the same at 92.93% when pulling c7b0905c236ad0c1fad1c2418d1610196e9eccf7 on TBurnip:master into 66dafc6679e979c8ad102c0a782d9731179945c6 on spulec:master.

For some reason, the last test is failing.  Can you try rebasing off of master and seeing if that fixes it?

What do you mean rebasing?

@TBurnip rebase your branch with the current master branch. You would then need to push override your current branch to update the PR: https://help.github.com/en/articles/using-git-rebase-on-the-command-line.

I have now rebased it and it seems to be passing the checks.


GMX-2065 GMX-2066 GMX-2067 fixing issue with adaptive going to the wr…

…ong API
@weddingwire/gmx please review

They are now :)


ImportError: cannot import name 'metanet_pb2' from partially initialized module 'caffe2.proto'

I have been trying to get this lib to work for three days. I have tried python 3.5, 3.6, 3.7, and 3.10.
I have tried installing pytext using conda, pip, and build from source.
It all ends up with the error in the issue title.
from caffe2.proto import caffe2_pb2
File "/data/anaconda3/envs/pytext/lib/python3.8/site-packages/caffe2/proto/init.py", line 15, in
from caffe2.proto import caffe2_pb2, metanet_pb2, torch_pb2
ImportError: cannot import name 'metanet_pb2' from partially initialized module 'caffe2.proto'
googling ends up in dead trails where people continue to see the same problem, but previously raised tickets are closed.
Please give indication how to get past this step?

I use Python 3.7.6. I have not seen this error before. Only advice I can give is to do a clean install by deleting previous packages. Maybe use a new Python environment to install everything?


Keybinding loses control after pressing Enter key on a date

So this is my process, the issues I am having I have capitalized

I navigate to the input that the date picker is attached to
Pressing the down key on the input, I open the date picker
I select a date by using the arrow keys to navigate, then pressing the Enter key
I navigate back into the input box
DOWN KEY NO LONGER WORKS so I click the calendar icon
The calendar opens but THE ARROW KEYS NO LONGER NAVIGATE instead they move about the input box content

Has anyone else experienced this issue or knows a fix? It happens in the demo too.

I figured out the issue.  It was being caused by the tab key not being handled.  I was able to fix it in a jsfiddle, please review to see if it can be put into the base code (I added in some comments with //FIX in two locations for easy viewing) https://jsfiddle.net/mfeola/a0zkz5zx/

Hi  @Spydr331,
We got a similar issue here in the project, once we select the date and came back with the tabkeys the arrow keys for left and right no longer navigate not even to move around the field.
The problem was that in the default logic of the plugin if there is no widget open it doesn't do anything, it just return. What I did to fixed was rebind the keys with this, showing the widget if the arrows are pressed.
$datepicker.datetimepicker({       format: 'DD-MMM-YYYY',       useStrict: true,       minDate: moment('01-Jan-1890', 'DD-MMM-YYYY'),       maxDate: moment(),        keepInvalid: true,        useCurrent: false,        widgetPositioning: {         horizontal: 'left',         vertical: 'bottom'       },       ignoreReadonly: true,       keyBinds: {         't': null, //we need 't' for dates such as Oct, disable the 't' = today's date feature         left: function (widget) {               if (!widget) {                   this.show();                   return;               }               var d = this.date() || this.getMoment();               if (widget.find('.datepicker').is(':visible')) {                   this.date(d.clone().subtract(1, 'd'));               }             },         right: function (widget) {             if (!widget) {                 this.show();                 return;             }             var d = this.date() || this.getMoment();             if (widget.find('.datepicker').is(':visible')) {                 this.date(d.clone().add(1, 'd'));             }         }       }     });
But we still face a problem that if they press enter the field loose focus, can you please repost your fiddle so I can have a look ? (it seems not working)
Thanks!

Hi @rachirib,
I just want to ask where I'm gonna put that code of yours? Sorry I'm just a newbie.

Sorry, but the Issue still exists...

The error is that keyState gets never reset after selecting a date with Enter or canceling with Escape. To add keyState = {}; in the hide method fixes this bug.


Update k8s deployment template

A couple of other notes for follow up:

I think we should remove the try blocks from here: https://github.com/massdriver-cloud/terraform-modules/blob/main/massdriver-application/main.tf#L5-L7  If those files aren't present, it should be an error, not a silent failure with default values (my params file was named wrong and it took a few minutes to figure out why it didn't work).
We should update commonLabels with md_metadata.default_tags here: https://github.com/massdriver-cloud/terraform-modules/blob/main/massdriver-application-helm/main.tf#L2  so we get the tags as labels in MD. If not there, it needs to be somewhere.  Thoughts @coryodaniel
Check the other comments on naming convention.

Otherwise, this was tested locally with terraform and deployed.

Is how we get CI to pass. Not sure if there is a way to “raise an error” in terraform. One option would be to remove the defaults from the try block and  lox the same file without the path.root in the failure side of try - that will always fail and tell you the file name,  it the path would be wrong.

More importantly, mass app new should create your dev auto var files - we should fix that if it does not. @WillBeebe

makes sense

Is how we get CI to pass. Not sure if there is a way to “raise an error” in terraform. One option would be to remove the defaults from the try block and load the same file without the path.root in the failure side of try - that will always fail and tell you the file name,  but the path would be wrong.

More importantly, mass app new should create your dev auto var files - we should fix that if it does not. @WillBeebe

makes sense

The CLI doesn't make the new name _params.auto... or _connections.auto files yet. I have most of it on a branch and can get a PR out today.


Editing a test file clears test status in test explorer

Work flow:

Run tests
5 passed, 1 failed
Edit failing test file
Extension will re-discover tests
All status will be wiped clean

Questions:

How do we re-run failed tests
Should 5 passed tests sill appear as 5 passed
Should we just display 1 failed and other 5 have no status icon

Proposal:

If the same test exists after re-discovering tests, keep the last run status
Optionally change icon from "green pass" to a "white pass" icon to indicate pass status is fail

Todo:

Modify src/client/unittests/common/services/storageService.ts
Merge results in storeTests method

validated


Updated to the latest ouroboros-network depenencies

The sha256 hashes in the cabal.project file also need to be updated. The current version of cardanp-repo-tool can do this.


Add the option to test without testing cases in examples/

This PR adds the option --no-examples to ./mfc.sh test so that pre_process doesn't have to be compiled a billion times when testing and waste so much of my precious valuable time.

Good call. Can you add this to the docs? Do we even document all/most such options for build/test/run?

Most of the options for testing haven't been added to the documentation yet. I'll add them.


LogReader: clean up max segment getting

The sourcing stuff for LogReader is a bit confusing. This (for the SegmentRange class):
https://github.com/commaai/openpilot/blob/6552d4ecb87211b91310ca5d7a7bcb918df36021/tools/lib/route.py#L238-L246
is doing the same thing as the Route class:
https://github.com/commaai/openpilot/blob/6552d4ecb87211b91310ca5d7a7bcb918df36021/tools/lib/route.py#L28
but both in slightly different ways. There's probably not a good reason for that, so we should combine and clean things up in the process. I think we should prefer Route's method as if maxqlog isn't updated in the database, you won't find those files.

Actually it really only makes sense to use the azure files since we proceed to just get them from there anyway.
Will fix my problem a better way


fix: Increase nexus-staging timeout

Increase timeout for nexus-staging plugin
PR Checklist
Please make sure that your PR fulfills the following requirements:

[ ] The commit message follows the Angular Commit Message Guidelines.
[ ] Tests for the changes have been added (for bug fixes / features)
[ ] Docs have been added / updated (for bug fixes / features)

PR Type

[ ] Bugfix
[ ] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] New tests
[ X] Build/CI related changes
[ ] Documentation content changes
[ ] Other (please describe)

What is the current behavior?

What is the new behavior?

Does this PR introduce a breaking change?

[ ] Yes
[ X] No

Other information

:tada: This PR is included in version 0.21.2 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


feature: add multiple pictures in the same paragraph

Hi, Scanny,
For docx add_picture function, is it support add multiple pictures in the same paragraph?
in your pptx, add_picture(left, top, width, height) can add multiple graph to a single row
by input parameter. There are only width and height for docx.
Thanks,
Le

@zhongle - not with the current API. But of course Word and the XML format support this. This page in the technical documentation may give you an idea how inline pictures in Word work. http://python-docx.readthedocs.org/en/latest/dev/analysis/features/shapes-inline.html
By going down to the lxml level you should be able to accomplish what you're after.

Hi, Scanny,
Thanks for the guide! I am trying with your suggestion to add multiple graphs with the same inline instance(inline.graphic.graphicData), but the generated docx file have some syntax error that can not be opened, would you pls give some suggestions, thanks a lot in advance!
my main flow code as below:
class EXT_Picture(CT_Picture):
'''extended class from CT to support customize operation 12/7/2015'''
@classmethod
def add(cls, pic_id, filename, rId, cx, cy, off_x):
'''add graph 12/07/2015'''
pic = cls.new(pic_id, filename, rId, cx, cy)
    if pic and pic.spPr and pic.spPr.xfrm:
        pic.spPr.xfrm.off.x = off_x
    return pic

#main flow
for graph_index in range(0, len(graphs_list)):
graph_name = graphs_list[graph_index]
graph_path = graphs_dict[graph_name]
  if 0 == graph_index: #first graph of row
       self.cur_inline = run.add_picture(graph_path)._inline

  elif self.cur_inline and run.part: # second graph to the end of the row
       rId, image = run.part.get_or_add_image(graph_path)
       cx, cy = image.scaled_dimensions(None, None)#(width, height)
       shape_id, filename = run.part.next_id, image.filename
       pic = EXT_Picture.add(0, filename, rId, cx, cy, cx/2)
       self.cur_inline.graphic.graphicData._insert_pic(pic)
       run._r.add_drawing(self.cur_inline)

Hi, Scanny,
I have solve the multi-pic issue by adding multi-inline shapes under drawing, thanks for your suggestions.
Thanks,
Le

That's great @zhongle, glad you got it working. If you can share the code you developed as a Gist and link it here, that may be helpful to others who want to do this sort of thing :)

Hi, Scanny,
Sure, glad to share, main flow code as below:
run = doc.add_paragraph().add_run()
def add_picture(self, run, graph_path, graph_width=None, graph_height=None):
'''add picture extended function for offset 12/08/2015'''
r_id, image = run.part.get_or_add_image(graph_path)
width, height = image.scaled_dimensions(graph_width, graph_height)
shape_id, filename = run.part.next_id, image.filename
inline = CT_Inline.new_pic_inline(shape_id, r_id, filename, width, height)
run._r.add_drawing(inline)
if want to add multi-picture in same row,  use the same run instance.
Thanks,
Le

Super, thanks @zhongle, nice job! :)
I cleaned up a couple aspects of the code. In particular there's no need for a self parameter since this is a free-standing function rather than a method on the Run object.

No problem, agreed! :)

`
def add_picture_to_run(run, picture_file, width=None, height=None):
"""
Add a picture at the end of a run.
"""
rId, image = run.part.get_or_add_image(picture_file)
cx, cy = image.scaled_dimensions(width, height)
shape_id, filename = run.part.next_id, image.filename
inline = CT_Inline.new_pic_inline(shape_id, rId, filename, cx, cy)
run._r.add_drawing(inline)
run = doc.add_paragraph().add_run()
picture = 'foobar.png'
add_picture_to_run(run, picture)
`
It add picture after the line, how to add picture before the line?
need cheng rId and shape_id?


View stacking transaction - This page is missing

https://explorer.stacks.co/txid/0xa07a4b7b37a1a894208c4aa13a6e3938ad0e1edf355533758df8f2f71a50117c

Thanks for the report, once the site is deployed next this should be fixed :)

Thanks for the report, once the site is deployed next this should be fixed :)

@unclemantis should be working now!

@unclemantis should be working now!


Count_occurrence does not work with tokenizer?

Hello!
Sorry if this is something simple that I missed, but I tried to count occurrences of words in 'rajpurkar/squad' dataset and after downloading data, tokenization and suffix array creation, the binary search from count_occurrence script does not return the correct number of word occurrences (it returns 0 most of the time...). I then digged deeper and found that the problem might be related to tokenization, where the same words can be tokenized differently based on context by a BPE tokenizer. This happens with GPT2Tokenizer from huggingface and with tiktoken tokenizer also... Below is a script to showcase this difference:
from transformers import GPT2Tokenizer
# Initialize the GPT-2 tokenizer
tokenizer = GPT2Tokenizer.from_pretrained("gpt2")

a = "blah blah blah... Architecturally... blah blah"
b = "Architecturally"

# Encode both strings
encoded_a = tokenizer.encode(a)
encoded_b = tokenizer.encode(b)

print("encoded_a: ", encoded_a)
print("encoded_b: ", encoded_b)

# Check if encoded_b is a subsequence of encoded_a
def is_subsequence(sub, main):
    sub_len = len(sub)
    for i in range(len(main) - sub_len + 1):
        if main[i:i + sub_len] == sub:
            return True
    return False

print("Is encoded_b a subsequence of encoded_a?", is_subsequence(encoded_b, encoded_a))

The output on my machine is:
encoded_a:  [2436, 993, 33367, 33367, 986, 17340, 20221, 986, 33367, 33367]
encoded_b:  [19895, 5712, 20221]
Is encoded_b a subsequence of encoded_a? False

This leads to the binary search not finding the correct number of occurrences. Am I missing something? Is there a setting where I can set a BPE tokenizer to always tokenize the same text to the same integers independent  of context?
Thank you!

Yeah. This project works in byte-space, not "word"-space.  If you tokenize your dataset, then you'll have to handle things yourself if you want different tokens. (If your dataset isn't that big, you can also just work in byte-space and everything will be fine and easy.)
For your specific example, you'll want to make sure to add an initial space b = " Architecturally" and everything should match what you expect. This is because most tokenizers add the space to the following word. So it's not "that" context-dependent, just what the one prior byte was.
If you want to do this properly to do word searching, you'll want to find all first tokens that contain your word and start from there. I don't know exactly how "Architecturally" is tokenized but let's pretend it's "Arch" "itect" "urally". This means you'll want to find all tokens that contain "Arch" which might be "\nArch", " Arch", "Arch", "\tArch". Then run your search for each of those followed by the remaining 2 tokens.

Hi Nicholas,
Okay thank you for prompt reply! This helps a lot and indeed by adding a space at the start I was able to find most of the sequences in SQUAD.
Have a good day!


bug: #361 caused a regression of #350

Provide environment information
  System:
    OS: macOS 14.4.1
    CPU: (10) arm64 Apple M1 Pro
    Memory: 113.14 MB / 16.00 GB
    Shell: 5.9 - /bin/zsh
  Binaries:
    Node: 18.18.2 - ~/.nvm/versions/node/v18.18.2/bin/node
    Yarn: 1.22.19 - /opt/homebrew/bin/yarn
    npm: 9.8.1 - ~/.nvm/versions/node/v18.18.2/bin/npm
    pnpm: 8.15.4 - ~/Library/pnpm/pnpm
    bun: 1.0.35 - ~/.bun/bin/bun

Describe the bug
The new drag handle regresses the fix from #350, causing the drag handle not to work in dialogs.

Link to reproduction
TBD
To reproduce

Install Novel
Render Novel in a Radix Dialog
Examine the Drag Handle position

Additional information
No response

Ok will add a PR to fix this in the new repo
https://github.com/NiclasDev63/tiptap-extension-global-drag-handle

Update novel demo example to also show a dialog, so we can better track bugs related to this in the future
Added PR adding changes from #350
I'll update novel after @NiclasDev63 publishes a new version

@haydenbleasel published<EMAIL_ADDRESS>should fix this issue


Make CodeBlockItemListSyntax expressible by string literal

This PR is to enable creating a CodeBlockItemList from one multi-line string, which is mentioned in #1165.
BTW, I also noticed that there are two linked TODOs in ConvenienceInitializers.swift and SyntaxExpressibleByStringInterpolationConformances.swift
// TODO: This should be inlined in SyntaxParseable.init(stringInterpolation:),
// but is currently used in `ConvenienceInitializers.swift`.
// See the corresponding TODO there.
func performParse<SyntaxType: SyntaxProtocol>(

// TODO: We should split FunctionParameter into separate nodes
extension FunctionParameterSyntax {

I'm a little confused about We should split FunctionParameter into separate nodes. Some examples or elaborations would be of much help!

As I mentioned in https://github.com/apple/swift-syntax/issues/1165#issuecomment-1471192005, I would prefer to make CodeBlockItemSyntax expressible by string literal instead of CodeBlockItemListSyntax. I think that generalizes better because you can then still use result builder constructs such as loops to create the final CodeBlockItemListSyntax.
Also, regarding the implementation, instead of adding parserCodeBlockItemListForBuilder you should make CodeBlockItemSyntax conform to SyntaxParsable in Parser+Entry.swift.

I'm a little confused about We should split FunctionParameter into separate nodes. Some examples or elaborations would be of much help!

We currently use FunctionParameterSyntax for a variety of parameters like function parameters, enum parameters and closure parameters and they all have different needs. Closure parameters for example don’t require a type and enum parameters no parameter label. Thus all the children of FunctionParameterSyntax are optional. If we had a separate node for each of these uses we could get rid of ParameterSubject and make the children that are required for each of these parameter kinds non-optional.

Hi @ahoppen, thanks for your comments!
Sorry for misunderstanding your comments in #1165. I have updated the codes according to your suggestions. To prevent the undefined behavior of parsing a code block item, I added an assertion in the Parser+Entry.swift. I wonder whether this is reasonable?
As for FunctionParameterSyntax, I would like to make a try to separate it into different nodes. Should I create an issue for that to follow up?
Thanks in advance!

@swift-ci Please test macOS

@swift-ci Please test

@swift-ci Please test Windows

@swift-ci please test

@swift-ci please test Windows

I think there is something wrong with the Windows CI. Could you please trigger that again? @kimdv

@swift-ci please test Windows

Thank you 😁


Generate code in pre-build step

To make it more clear that users should not edit these files directly. Will reset their changes on each local build, similar to what build server does.
Fixes #199.

@JKSnd for your information. I'll just merge this in right away.


Video Bitrate Issues

Converted a 25.3 GiB, 21.3 Mb/s overall bit rate 1080p BR rip MKV file to MP4 while leaving the video-bitrate option with the default setting of nothing, and wound up with a 8.66 GiB, 7 283 kb/s overall bit rate file. As far as I can see, the default option is to not limit the bit rate so I would expect to see a file with both similar size and overall bit rate. I do not believe that I have any other settings specified that would be limiting this, but can share my config if it's requested.
Any/all help/information is greatly appreciated.

Post your autoProcess.ini file
Generally if you're gonna convert something you need to specify some degree of quality otherwise it just defaults to FFMPEG's default, you need either a CRF or bitrate specified

Ok, that makes sense. New to this and just copied a friend's file. Here's the relevant portion:
[MP4]
ffmpeg = C:\Programs\ffmpeg\bin\ffmpeg.exe
ffprobe = C:\Programs\ffmpeg\bin\ffprobe.exe
threads = auto
output_directory =
copy_to =
move_to =
output_extension = mp4
output_format = mp4
delete_original = True
relocate_moov = True
video-codec = h264,x264
video-bitrate =
video-crf =
video-max-width = 1920
h264-max-level = 4
use-qsv-decoder-with-encoder = True
ios-audio = True
ios-first-track-only = True
ios-audio-filter =
max-audio-channels = 5
audio-codec = ac3,mp3
audio-language =
audio-default-language =
audio-channel-bitrate =
audio-filter =
subtitle-codec = mov_text
subtitle-language = eng
subtitle-default-language = eng
subtitle-encoding =
fullpathguess = True
convert-mp4 = False
tagfile = True
tag-language = eng
download-artwork = Thumbnail
download-subs = False
embed-subs = True
sub-providers = addic7ed,podnapisi,thesubdb,opensubtitles
permissions = 0777
post-process = False
pix-fmt =
aac_adtstoasc = False
postopts =
preopts =
audio-copy-original = False
enable_dxva2_gpu_decode = False
ios-move-last = False
use-hevc-qsv-decoder = False
embed-only-internal-subs = False

So yeah you should specify some quality options if you're going to convert
However, the script should estimate a bitrate similar to the source bitrate
Out of curiosity, using those settings if you run a file through manual.py what json options does it spit out prior to processing?

That's what I thought the expected behavior would be. I did three of these by hand with the same result, but didn't note down the JSON output. I will do it again and grab all the info.

Just thought to see if there's a log, and there is, so here's the logged info for that manual conversion:
2018-08-06 16:06:42 - MANUAL - INFO - Manual processor started.
2018-08-06 16:06:42 - MANUAL - INFO - C:\Python27\python.exe
2018-08-06 16:06:42 - MANUAL - WARNING - Audio bitrate was invalid, defaulting to 256 per channel.
2018-08-06 16:06:45 - MANUAL - INFO - Reading video stream.
2018-08-06 16:06:45 - MANUAL - INFO - Video codec detected: h264.
2018-08-06 16:06:45 - MANUAL - INFO - Pix Fmt: yuv420p.
2018-08-06 16:06:45 - MANUAL - INFO - Video level 4.1.
2018-08-06 16:06:45 - MANUAL - INFO - Reading audio streams.
2018-08-06 16:06:45 - MANUAL - INFO - No audio streams detected in any appropriate language, relaxing restrictions so there will be some audio stream present.
2018-08-06 16:06:45 - MANUAL - INFO - Audio detected for stream #1: ac3 [eng].
2018-08-06 16:06:45 - MANUAL - INFO - Creating audio stream 0 from source audio stream 1 [iOS-audio].
2018-08-06 16:06:45 - MANUAL - INFO - Audio track is number 0 setting disposition to default
2018-08-06 16:06:45 - MANUAL - INFO - Creating audio stream 1 from source stream 1.
2018-08-06 16:06:45 - MANUAL - INFO - Audio Track is number 1 setting disposition to none
2018-08-06 16:06:45 - MANUAL - INFO - Audio detected for stream #2: aac [eng].
2018-08-06 16:06:45 - MANUAL - INFO - Creating audio stream 2 from source stream 2.
2018-08-06 16:06:45 - MANUAL - INFO - Audio Track is number 2 setting disposition to none
2018-08-06 16:06:45 - MANUAL - INFO - Reading subtitle streams.
2018-08-06 16:06:45 - MANUAL - INFO - Subtitle detected for stream #3: subrip [eng].
2018-08-06 16:06:45 - MANUAL - INFO - Creating subtitle stream 0 from source stream 3.
2018-08-06 16:06:45 - MANUAL - INFO - Subtitle detected for stream #4: subrip [eng].
2018-08-06 16:06:45 - MANUAL - INFO - Creating subtitle stream 1 from source stream 4.
2018-08-06 16:06:45 - MANUAL - INFO - Subtitle detected for stream #5: subrip [eng].
2018-08-06 16:06:45 - MANUAL - INFO - Creating subtitle stream 2 from source stream 5.
2018-08-06 16:06:45 - MANUAL - INFO - {
    "preopts": [
        "-fix_sub_duration"
    ],
    "subtitle": {
        "0": {
            "map": 3,
            "codec": "mov_text",
            "language": "eng",
            "encoding": null
        },
        "1": {
            "map": 4,
            "codec": "mov_text",
            "language": "eng",
            "encoding": null
        },
        "2": {
            "map": 5,
            "codec": "mov_text",
            "language": "eng",
            "encoding": null
        }
    },
    "format": "mp4",
    "postopts": [
        "-threads",
        "auto"
    ],
    "video": {
        "map": 0,
        "codec": "h264",
        "bitrate": 21266.839,
        "level": 4.0
    },
    "audio": {
        "0": {
            "map": 1,
            "language": "eng",
            "channels": 2,
            "codec": "aac",
            "disposition": "default",
            "bitrate": 256,
            "filter": null
        },
        "1": {
            "map": 1,
            "language": "eng",
            "channels": 5,
            "codec": "ac3",
            "disposition": "none",
            "bitrate": 1280,
            "filter": null
        },
        "2": {
            "map": 2,
            "language": "eng",
            "channels": 2,
            "codec": "ac3",
            "disposition": "none",
            "bitrate": 512,
            "filter": null
        }
    }
}
2018-08-06 16:06:45 - MANUAL - INFO - Starting conversion.
2018-08-06 17:40:32 - MANUAL - INFO - G:\Completed\Movies\The.Godfather.Part.III.1990\The.Godfather.Part.III.1990.1080p.BluRay.DD5.1.x264-CtrlHD-002.mp4 created.
2018-08-06 17:40:46 - MANUAL - INFO - Tagging file: G:\Completed\Movies\The.Godfather.Part.III.1990\The.Godfather.Part.III.1990.1080p.BluRay.DD5.1.x264-CtrlHD-002.mp4.
2018-08-06 17:40:46 - MANUAL - INFO - Trying to write tags.
2018-08-06 17:40:47 - MANUAL - INFO - Tags written successfully.
2018-08-06 17:40:47 - MANUAL - INFO - Relocating MOOV atom to start of file.
2018-08-06 18:15:16 - MANUAL - INFO - Manual processor started.
2018-08-06 18:15:16 - MANUAL - INFO - C:\Python27\python.exe
2018-08-06 18:15:16 - MANUAL - WARNING - Audio bitrate was invalid, defaulting to 256 per channel.

Yeah its trying to guess the bitrate but chances are its just too high because it just uses the prior bitrate, but given that you're reducing the resolution you'll need a lower bitrate or you can use the CRF setting but this is primarily an FFMPEG behavior


InvalidEnumException for Enum::is() update

InvalidEnumException for Enum::is() now mentions the class name if an object was passed to the function.

Thanks @CupOfTea696 for your contribution and interest.
Can I please ask you to send this PR to our develop branch? Optionally do you mind to provide a test covering your introduced changes?
Thanks in advance.


GitHub: Rename nginx-ingress-controller to ingress-nginx.

Same as in devctl I think this reference should be removed as ingress nginx is no longer part of Giant Swarm releases

Opened another PR for removing it and requested your review.


How to use Material Design theme in Avalonia Reactive UI WPF(.Net Core)

Hello,
I am developing app using following technologies:
WPF( .Net Core) , Avalonia Reactiveui
I would like to use Material Design theme in this app.
I tried to add Material design resource in app.xaml as below:
<Application.Resources> 		<ResourceDictionary> 			<ResourceDictionary.MergedDictionaries> 				<!-- Material design theme --> 				<ResourceDictionary Source="pack://application:,,,/MaterialDesignThemes.Wpf;component/Themes/MaterialDesignTheme.Light.xaml" /> 				<ResourceDictionary Source="pack://application:,,,/MaterialDesignThemes.Wpf;component/Themes/MaterialDesignTheme.Defaults.xaml" /> 				<ResourceDictionary Source="pack://application:,,,/MaterialDesignColors;component/Themes/Recommended/Primary/MaterialDesignColor.Teal.xaml" /> 				<ResourceDictionary Source="pack://application:,,,/MaterialDesignColors;component/Themes/Recommended/Accent/MaterialDesignColor.Teal.xaml" /> </ResourceDictionary.MergedDictionaries> </ResourceDictionary> </Application.Resources>
But it throws error at runtime.
This is because Avalonia using xmlns="https://github.com/avaloniaui"
whereas resource dictionary file used xmlns="http://schemas.microsoft.com/winfx/2006/xaml/presentation" in plain WPF.
Could you please suggest how to resolve this problem.

The material design library you are using needs to be ported.

Could you please elaborate more on this. I didn't get you.
how can we port material design library?

There are ported Material Design icons here https://github.com/ahopper/Avalonia.IconPacks

Thanks ahopper for your reply.
How to use this ported Material Design Icons in app.
I need to use Material Design Theme and Color in Avalonia Reactive UI WPF?
What I need to do? how can I port theme and use it?
Thanks in advance.

The IconPacks app is an example in itself. MainWindow.xaml https://github.com/ahopper/Avalonia.IconPacks/blob/master/MainWindow.xaml uses an icon for search. The icon is in a file in the icons directory and it is referenced as a StyleInclude in App.xaml. You typically don't want to include all icons so the IconPacks program is there to allow easy copy and pasting of just the icons you want.  I think the ControlCatalog sample has an example of themes.

I am not able to find ControlCatalog  there.
Is it a separate repo?

https://github.com/AvaloniaUI/Avalonia/tree/master/samples

I have downloaded whole solution (it has 70+ projects) but I am not able to find any project which is using Material Design them

There is not a current Material Design theme but there are examples of theme changing which might help you port/create a Material theme, this may also be useful https://github.com/wieslawsoltes/Avalonia.ThemeManager

and https://github.com/wieslawsoltes/ThemeEditor

Do you mean that Material Design won't integrate with Avalonia Reactiveui as  it is.
Do I need to Adapt theme of Avalonia using ThemeManger to make similar to Material Design theme?

Do you mean that Material Design won't integrate with Avalonia Reactiveui as it is.
yes, the wpf library wont magically work.

Do I need to Adapt theme of Avalonia using ThemeManger to make similar to Material Design theme?
I'm no expert on themes but ThemeEditor might be a good place to start

Thanks ahopper for your quick response.

I have looked at Avalonia Theme Editor and play around it.
It support only Color modification. I would like make style similar to Material Design.
How can I do this.

Avalonia has lookless controls like WPF. To have controls to look like you want you have to override the default templates.
https://github.com/AvaloniaUI/Avalonia/tree/master/src/Avalonia.Themes.Default


[ANCHOR-220] Refactor README

PR Checklist
PR Structure

[ ] This PR has reasonably narrow scope (if not, break it down into smaller PRs).
[ ] This PR avoids mixing refactoring changes with feature changes (split into two PRs
otherwise).
[ ] This PR's title starts with name of package that is most changed in the PR, ex.
paymentservice.stellar, or all or doc if the changes are broad or impact many
packages.

Thoroughness

[ ] This PR adds tests for the most critical parts of the new functionality or fixes.

What
[TODO: Short statement about what is changing.]
Why
[TODO: Why this change is being made. Include any context required to understand the why.]
Known limitations
[TODO or N/A]

Reference Server Preview is available here:https://anchor-ref-pr882.previews.kube001.services.stellar-ops.com/SEP Server Preview is available here:https://anchor-sep-pr882.previews.kube001.services.stellar-ops.com/


Add support for ignore_changes lifecycle Meta argument

I am using the terraform-aws-apigateway-v2 module to manage my API Gateway resources. In my architecture, I am following a Cognito app-based multitenancy approach. Due to this, certain configurations such as authorizer.jwt_configuration.audience and api_gw.cors_configuration.allow_origin need to be dynamic because clients can be added or removed dynamically with their own app client IDs and domains.
Currently, these configurations are tightly managed by the Terraform module, causing unnecessary changes and disruptions on every deployment. To handle this more efficiently, I need support for the ignore_changes lifecycle Meta argument to be added to the module. This would allow specific configurations to be ignored by Terraform, thus preventing them from being updated on each deployment.
Feature Request:
Please add support for the ignore_changes lifecycle Meta argument for the resources included in the module.
By adding this feature, it will allow for dynamic updates to these lists without causing disruptions or requiring manual intervention with each deployment.
Use Case:
In a multitenant environment, the list of clients can change frequently. Without the ability to ignore changes to these configurations, the Terraform module attempts to update them on every deployment, which is not desirable. Enabling ignore_changes for these configurations will improve the flexibility and usability of the module in dynamic environments.
Thank you for considering this feature request. It will greatly enhance the module's applicability in real-world scenarios where dynamic client management is required.
Environment:
Terraform version: v1.8.2 on linux_amd64
Module version: 4.0.0
AWS Provider version: v5.45.0
Please let me know if there is any other workaround for this, I'm open to a different and viable solution.

I see that lifecycle settings only allow literal values. Closing issue.


Improve .gitignore

When I tried to create the repository, I had to re-upload my work a couple times because I did not properly set ut the .gitignore.
It is relatively simple, but I would like to add documentation to it; and I am not sure how to go about doing so.

It looks fine.
It was the Dev Branch .gitignore.


Ollie: Watch habibigp.ir

Ollie requests the watch of the watch_keyword habibigp\.ir. See the MS search here and the Stack Exchange search in text, in URLs, and in code.

habibigp\.ir has been seen in 1 true positive, 0 false positives, and 0 NAAs.

Approved by Jeff Schaller in Charcoal HQ


Max thread number for platform task runner thread

Hello, in my projet, i use a plugin to do a certain jobs(List), in which will create new threads for each job in jobs(job finish in few seconds normally)
now in dart, it will invoke the methodfor 10 jobs per time(10 threads will be created) recursively.
the problem is after app run some time randomly, the app will crush and ouptput error like below. i guess the problem is the plugin create too much threads, i am not for sure.
Logs

F/libc    (25280): Fatal signal 11 (SIGSEGV), code 1, fault addr 0x1f0ffed65f03d0 in tid 25280 (nieben.yiqice)

Build fingerprint: 'HONOR/LLD-AL30/HWLLD-H2:8.0.0/HONORLLD-AL30/133(C00):user/release-keys'
Revision: '0'
ABI: 'arm64'
pid: 25280, tid: 25280, name: nieben.yiqice  >>> nieben.yiqice <<<
signal 11 (SIGSEGV), code 1 (SEGV_MAPERR), fault addr 0x1f0ffed65f03d0
x0<PHONE_NUMBER>000060  x1   0000007c8f437dc0  x2   0000007c8f400000  x3<PHONE_NUMBER>000006
x4<PHONE_NUMBER>000037  x5   0000007fc10ce078  x6   0000007cb8f23000  x7<PHONE_NUMBER>794276
x8   0000007c96de9898  x9<PHONE_NUMBER>000016  x10<PHONE_NUMBER>000016  x11<PHONE_NUMBER>000009
x12  0000007c965a0c40  x13<PHONE_NUMBER>000010  x14  0000007c965a0c40  x15  f81f0ffed65f03c0
x16<PHONE_NUMBER>09c108  x17  0000007cb7ce7948  x18  000000001a5a3a48  x19  0000007cb417f388
x20<PHONE_NUMBER>000017  x21<PHONE_NUMBER>000017  x22<PHONE_NUMBER>00000b  x23<PHONE_NUMBER>000044
x24<PHONE_NUMBER>00000b  x25  0000007cb417f398  x26  0000007cb40ade70  x27  0000007ca9a0e818
x28<PHONE_NUMBER>000040  x29  0000007ca9a0e818  x30  0000007c965b4810
sp   0000007fc10ce0f0  pc   0000007c96cee860  pstate<PHONE_NUMBER>000000
backtrace:
#00 pc<PHONE_NUMBER>3e6860  /data/app/nieben.yiqice-kHJTi3gGe9hMTdLsxaaXXQ==/lib/arm64/libflutter.so (offset 0xc90000)
#01 pc<PHONE_NUMBER>cac80c  /data/app/nieben.yiqice-kHJTi3gGe9hMTdLsxaaXXQ==/lib/arm64/libflutter.so (offset 0xc90000)
#02 pc<PHONE_NUMBER>cac540  /data/app/nieben.yiqice-kHJTi3gGe9hMTdLsxaaXXQ==/lib/arm64/libflutter.so (offset 0xc90000)
#03 pc<PHONE_NUMBER>cac260  /data/app/nieben.yiqice-kHJTi3gGe9hMTdLsxaaXXQ==/lib/arm64/libflutter.so (offset 0xc90000)
#04 pc<PHONE_NUMBER>ce6154  /data/app/nieben.yiqice-kHJTi3gGe9hMTdLsxaaXXQ==/lib/arm64/libflutter.so (offset 0xc90000)
#05 pc<PHONE_NUMBER>cb9d60  /data/app/nieben.yiqice-kHJTi3gGe9hMTdLsxaaXXQ==/lib/arm64/libflutter.so (offset 0xc90000)
#06 pc<PHONE_NUMBER>cbba94  /data/app/nieben.yiqice-kHJTi3gGe9hMTdLsxaaXXQ==/lib/arm64/libflutter.so (offset 0xc90000)
#07 pc<PHONE_NUMBER>015f18  /system/lib64/libutils.so (_ZN7android6Looper9pollInnerEi+808)
#08 pc<PHONE_NUMBER>015b68  /system/lib64/libutils.so (_ZN7android6Looper8pollOnceEiPiS1_PPv+108)
#09 pc<PHONE_NUMBER>11b31c  /system/lib64/libandroid_runtime.so (_ZN7androidL38android_os_MessageQueue_nativePollOnceEP7_JNIEnvP8_jobjectli+44)
#10 pc<PHONE_NUMBER>82769c  /system/framework/arm64/boot-framework.oat (offset 0x626000) (android.app.NativeActivity.onWindowFocusChangedNative [DEDUPED]+140)
#11 pc<PHONE_NUMBER>015c50  /dev/ashmem/dalvik-jit-code-cache (deleted)

[√] Flutter (Channel master, v1.3.14-pre.38, on Microsoft Windows [Version 10.0.17134.648], locale zh-CN)
• Flutter version 1.3.14-pre.38 at C:\src\flutter
• Framework revision 479be44479 (5 days ago), 2019-03-21 02:11:40 -0400
• Engine revision 250b99bd27
• Dart version 2.2.1 (build 2.2.1-dev.2.0 None)
[√] Android toolchain - develop for Android devices (Android SDK version 28.0.3)
• Android SDK at C:\Users\niebe\AppData\Local\Android\Sdk
• Android NDK location not configured (optional; useful for native profiling support)
• Platform android-28, build-tools 28.0.3
• ANDROID_HOME = C:\Users\niebe\AppData\Local\Android\Sdk
• Java binary at: C:\Program Files\Android\Android Studio\jre\bin\java
• Java version OpenJDK Runtime Environment (build 1.8.0_152-release-1248-b01)
• All Android licenses accepted.
[√] Android Studio (version 3.3)
• Android Studio at C:\Program Files\Android\Android Studio
• Flutter plugin version 33.3.1
• Dart plugin version 182.5215
• Java version OpenJDK Runtime Environment (build 1.8.0_152-release-1248-b01)
[!] IntelliJ IDEA Ultimate Edition (version 2017.2)
• IntelliJ at C:\Program Files\JetBrains\IntelliJ IDEA 2017.2.1
X Flutter plugin not installed; this adds Flutter specific functionality.
X Dart plugin not installed; this adds Dart specific functionality.
• For information about installing plugins, see
https://flutter.io/intellij-setup/#installing-the-plugins
[√] VS Code (version 1.32.3)
• VS Code at C:\Users\niebe\AppData\Local\Programs\Microsoft VS Code
• Flutter extension version 2.24.0
[√] Connected device (1 available)
• LLD AL30 • 6SJRX18B02004681 • android-arm64 • Android 8.0.0 (API 26)

Hi, could you please confirm if this problem still exists with the latest versions on stable or master?
Thanks a lot

Without additional information, we are unfortunately not sure how to resolve this issue. We are therefore reluctantly going to close this bug for now. Please don't hesitate to comment on the bug if you have any more information for us; we will reopen it right away!
Thanks for your contribution.


implement_methods, dont ignore check if there are super classes

dont do workaround, https://github.com/abaplint/abaplint/pull/1366

fixed in https://github.com/abaplint/abaplint/pull/1379, closing


Recreate the Navbar

Your task is to recreate the navbar of the Forum website.
Design is provided in the Figma link
Guidelines

The code should be clean and reusable.
You have to make changes inside THIS.

Note

Make sure you follow the given template before making the pull request.

claim

claim

Go for it @Kali-Decoder

Updates @Kali-Decoder ?


Expand group dataset metadata field for slice media types

What changes are proposed in this pull request?
For group datasets, the metadata field does not expand its schema when a new slice media type is implicitly added causing unwanted side effects like unnecessary metadata computation in the App server. This PR adds default metadata schema handling of new slice media types when adding samples to a group dataset.
How is this patch tested? If it is not, please explain why.
New integration/unit test
Release Notes
Is this a user-facing change that should be mentioned in the release notes?

Fixed metadata field expansion for group datasets when implicitly adding new slice media types, e.g. image and video

What areas of FiftyOne does this PR affect?

[x] App: FiftyOne application changes
[ ] Build: Build and test infrastructure changes
[x] Core: Core fiftyone Python library changes
[ ] Documentation: FiftyOne documentation changes
[ ] Other

This test is failing, should we do something about it?

[chromium] › oss/specs/regression-tests/group-video/group-video-label.spec.ts:87:3 › groups video labels › video plays with correct label for each slice

This test is failing, should we do something about it?

[chromium] › oss/specs/regression-tests/group-video/group-video-label.spec.ts:87:3 › groups video labels › video plays with correct label for each slice

Right. I'll take a look locally this afternoon

Just to clarify what this PR is addressing, currently on develop in the following situation:
import fiftyone as fo

group = fo.Group()
sample1 = fo.Sample(filepath="/Users/Brian/Desktop/test.png", group=group.element("img"))
sample2 = fo.Sample(filepath="/Users/Brian/Desktop/test.mov", group=group.element("vid"))

dataset = fo.Dataset()
dataset.add_samples([sample1, sample2])

dataset.compute_metadata()

fo.pprint(dataset.select_group_slices(_allow_mixed=True).head())

the samples contain the image and video-specific fields as desired:
[
    <SampleView: {
        'id': '65e7f45243c6316183d89d9a',
        'media_type': 'image',
        'filepath': '/Users/Brian/Desktop/test.png',
        'tags': [],
        'metadata': <ImageMetadata: {
            'size_bytes': 90227,
            'mime_type': 'image/png',
            'width': 1906,
            'height': 422,
            'num_channels': 4,
        }>,
        'group': <Group: {'id': '65e7f45143c6316183d89d98', 'name': 'img'}>,
    }>,
    <SampleView: {
        'id': '65e7f45243c6316183d89d9b',
        'media_type': 'video',
        'filepath': '/Users/Brian/Desktop/test.mov',
        'tags': [],
        'metadata': <VideoMetadata: {
            'size_bytes': 1647380,
            'mime_type': 'video/quicktime',
            'frame_width': 2940,
            'frame_height': 1758,
            'frame_rate': 58.96551724137931,
            'total_frame_count': 228,
            'duration': 3.816667,
            'encoding_str': 'avc1',
        }>,
        'group': <Group: {'id': '65e7f45143c6316183d89d98', 'name': 'vid'}>,
        'frames': <FramesView: 0>,
    }>,
]

However, these media type-specific fields are not added to the dataset's metadata field schema:
fo.pprint(dataset.get_field_schema(flat=True))

{
    'id': <fiftyone.core.fields.ObjectIdField object at 0x7f77e27a7be0>,
    'filepath': <fiftyone.core.fields.StringField object at 0x7f77e27a7c10>,
    'tags': <fiftyone.core.fields.ListField object at 0x7f77e27a7c70>,
    'metadata': <fiftyone.core.fields.EmbeddedDocumentField object at 0x7f77b0cb9160>,
    'metadata.size_bytes': <fiftyone.core.fields.IntField object at 0x7f77e3fbefa0>,
    'metadata.mime_type': <fiftyone.core.fields.StringField object at 0x7f77e3fbef70>,
    'group': <fiftyone.core.fields.EmbeddedDocumentField object at 0x7f77e3fbec10>,
    'group.id': <fiftyone.core.fields.ObjectIdField object at 0x7f77e3fbec40>,
    'group.name': <fiftyone.core.fields.StringField object at 0x7f77e3fcef70>,
}

This change makes it so that the metadata field in this situation (a grouped dataset with mixed media type slices) will now have all mixed fields declared on it:
{
    'id': <fiftyone.core.fields.ObjectIdField object at 0x7fd818859760>,
    'filepath': <fiftyone.core.fields.StringField object at 0x7fd818875fa0>,
    'tags': <fiftyone.core.fields.ListField object at 0x7fd818878fd0>,
    'metadata': <fiftyone.core.fields.EmbeddedDocumentField object at 0x7fd82a9fe370>,
    'metadata.size_bytes': <fiftyone.core.fields.IntField object at 0x7fd82a9fe0a0>,
    'metadata.mime_type': <fiftyone.core.fields.StringField object at 0x7fd82a9feee0>,
    'metadata.width': <fiftyone.core.fields.IntField object at 0x7fd82a9fe8b0>,
    'metadata.height': <fiftyone.core.fields.IntField object at 0x7fd82ab98520>,
    'metadata.num_channels': <fiftyone.core.fields.IntField object at 0x7fd82ab989d0>,
    'metadata.frame_width': <fiftyone.core.fields.IntField object at 0x7fd82ab31040>,
    'metadata.frame_height': <fiftyone.core.fields.IntField object at 0x7fd809114220>,
    'metadata.frame_rate': <fiftyone.core.fields.FloatField object at 0x7fd809114d30>,
    'metadata.total_frame_count': <fiftyone.core.fields.IntField object at 0x7fd809114ca0>,
    'metadata.duration': <fiftyone.core.fields.FloatField object at 0x7fd809114130>,
    'metadata.encoding_str': <fiftyone.core.fields.StringField object at 0x7fd809114fa0>,
    'group': <fiftyone.core.fields.EmbeddedDocumentField object at 0x7fd82ab2dd00>,
    'group.id': <fiftyone.core.fields.ObjectIdField object at 0x7fd82aa68880>,
    'group.name': <fiftyone.core.fields.StringField object at 0x7fd82aa68400>,
}

although the type of the metadata field remains the generic Metadata type:
print(dataset.get_field("metadata").document_type)
# fiftyone.core.metadata.Metadata

@benjaminpkane Out of curiosity, can you clarify why Flashlight (<-- I assume?) needs to have the subfields explicitly declared in the dataset's schema in order to avoid recomputing metadata? If the App just reads in the full metadata field values it will find the subfields despite not being formally declared in the dataset's schema.

Yes, historically this wouldn't be an issue. But the grid optimizations that filter sample data in _project_pagination_paths remove the undeclared fields. The alternative is to skip projections for the metadata field, but that would leave image/video metadata fields out of the sidebar which could be misleading


Validation for reference types seems nonfunctional

Very simple test class:
    public class ValitTestModel
    {
        public bool testAttributeA;
        public ushort testAttributeB;

        public bool validate()
        {
            var validator = ValitRules<ValitTestModel>
                .Create()
                .Ensure(m => m.testAttributeA, _ => _
                    .Required())
                .Ensure(m => m.testAttributeB, _ => _
                    .Required())
                .For(this)
                .Validate();

            return validator.Succeeded;
        }
    }

The error returned is that The type <bool|ushort> must be a reference type to use it as parameter TProperty for ValitRulePropertyExtensions.Required<TObject, TProperty> where both are defined as classes. C# assumes these to be structs, so it doesn't work.
I did notice however that bool and a few other types have type specific extension methods defined, how do we go about using them?

It works if the fields are made nullable, which is what we ended up doing.


It will outputs error in lstm_ptb_bucketing.py after one epoch training ends

hi, @pluskid , i just run the lstm_ptb_bucketing.py demo, and it outputs the error message as below,
2016-03-29 15:16:23,366 Epoch[0] Batch [1700]    Speed: 135.28 samples/sec    Train-accuracy=0.318865
2016-03-29 15:16:27,722 Epoch[0] Resetting Data Iterator
2016-03-29 15:16:27,723 Epoch[0] Train-accuracy=0.318873
2016-03-29 15:16:27,723 Epoch[0] Time cost=410.278
2016-03-29 15:16:56,348 Epoch[0] Validation-accuracy=0.351199
Traceback (most recent call last):
  File "lstm_ptb_bucketing.py", line 312, in <module>
    batch_end_callback=mx.callback.Speedometer(batch_size, 50),)
  File "../../python/mxnet/model.py", line 770, in fit
    sym_gen=self.sym_gen)
  File "../../python/mxnet/model.py", line 220, in _train_multi_device
    for data_batch in train_data:
  File "lstm_ptb_bucketing.py", line 174, in __iter__
    data[:] = self.data[i_bucket][idx]
ValueError: could not broadcast input array from shape (0,20) into shape (32,20)

it happend when one epoch finish training, in the line data[:] = self.data[i_bucket][idx], the data.shape is (32,20), but why the self.data[i_bucket][idx].shape is (0,20) in the second epoch?
i think it may (may) due to that we do nothing in reset of train_data, but how to fixed this? thanks~

The recently example is ok


Add parse_js_objects function.

Currently the library extracts only the first JSON-like thing it encounters in the input string:
>>> import chompjs
>>> data = """
... {'abc': 'test', 'foo': 12}
...
... // some JS code
...
... {'ghi': 'test', 'bar': 13}
... """)
>>> chompjs.parse_js_object(data)
{'abc': 'test', 'foo': 12}

Add a second method, parse_js_objects returning an iterator yielding all objects that were found and successfully parsed. Expected behavior:
>>> import chompjs
>>> data = """
... {'abc': 'test', 'foo': 12}
...
... // some JS code
...
... {'ghi': 'test', 'bar': 13}
... """)
>>> for result in chompjs.parse_js_objects(data):
...     print(result)
...
{'abc': 'test', 'foo': 12}
{'ghi': 'test', 'bar': 13}

@Nykakin ,  could you please consider the possibility to keep/have in the result JSON then name of JS dict?
for example if we have js like this:
const testField1 = {
  name: 'name1',
  value: 'value1',
}

const testField2 = {
  name2: 'name2',
  value2: 'value2',
}

in a result we should have:
[{"testField1": {"name": "name1", "value": "value2"}}, {"testField2": {"name2": "name2", "value2": "value2"}}]

or this will be wrong or overcomplicated?

This would be wrong. JavaScript objects aren't always immediately assigned to a variable and don't always have a clear concept of a name. For example, they can be passed down as arguments to anonymous function:
window.__NUXT__ = (function(a, b, c) {
    /*
   Some JavaScript code here
   */
})({"a": 123, "b": 232}, 11, [1, 2, 3, 4]));


exclude the error folder from sphinx toc

Description
Sphinx was throwing two warnings about the files in the error folder not being in a table of contents (toctree). These don't need to be in a toctree since they're just custom error pages.
I've excluded the error folder in the Sphinx conf.py, and also sorted the excludes so it's easier to manage.
The html for the error pages still render, and the warnings go away. Now down from 17 warnings/errors to 15 (for Sphinx/website/python specific errors)!
See this preview:
http://<IP_ADDRESS>/versions/error/error/404.html

@mxnet-label-bot add [pr-awaiting-review]
Thanks for your contribution @aaronmarkham

Failed a Perl test... restarting the job...

Restarting this one due to failure on ARM test. https://github.com/apache/incubator-mxnet/issues/13410
That's 2 for 2 for me so far this morning... wondering if this ARM test is broken for all the PRs...


ios working examples

Hey all, this is more of a question, trying to locate a working ios example using cca, i found none so far! .. any suggestions will be greatly appreciated it, many thanks

Closing this issue based on the previous answer.  Feel free to open if there still questions about the sample apps.


Error during installation

ubuntu@ip-123-45-67-789:~$ ./node_modules/.bin/spotswap-install
/home/ubuntu/node_modules/@mapbox/spotswap/index.js:18
if (!process.env[key]) throw new Error('Env var ' + key + ' is required');
^
Error: Env var INSTANCE_ID is required
at /home/ubuntu/node_modules/@mapbox/spotswap/index.js:18:34
at Array.forEach (native)
at envCheck (/home/ubuntu/node_modules/@mapbox/spotswap/index.js:17:5)
at Object. (/home/ubuntu/node_modules/@mapbox/spotswap/bin/install.js:9:1)
at Module._compile (module.js:410:26)
at Object.Module._extensions..js (module.js:417:10)
at Module.load (module.js:344:32)
at Function.Module._load (module.js:301:12)
at Function.Module.runMain (module.js:442:10)
at startup (node.js:136:18)
What are the actual steps to install and deploy this to an existing AWS environment? There seem to be a lot of missing info in the documentation. I'm willing to help once I have it up and running. Thanks.
cc:
@arunasank
@emilymcafee
@jakepruitt

@henryn001 sorry for this shameless plug on this issue, but you should have a better getting started experience using my similar project https://github.com/cristim/autospotting

@cristim I have your project up and running. It does work well however, we need something that has the termination logic that will spin up the back-up on-demand instances once a spot instance has been tagged for termination.

@henryn001 thanks, I'll definitely consider this idea for implementation soon, it's on the roadmap for a lot of time.

Word @henryn001 - appreciate the feedback (also +1 for feedback and collaboration @cristim - we're thinking of using cloudwatch events in the near future too).
It looks like the EnvironmentVariables section is lacking and needs a few additions:

AWS_REGION: the region of the instance
INSTANCE_ID: the ID of the instance. We get this by using the metadata endpoint at curl -s -m 3 http://<IP_ADDRESS>/latest/meta-data/instance-id.
SpotGroup or SpotFleet: the ID of the spot autoscaling group or spotfleet. We get this by making a cloudformation API call to describe the stack the instance is a part of, and query for the physical resource ID, with something like this:

asg=$(aws cloudformation describe-stack-resource \
    --stack-name ${stack} \
    --logical-resource-id AutoScalingGroup \
    --query StackResourceDetail.PhysicalResourceId \
    --output text)

I'll cut a PR to add this to the readme. Thanks for the catch!

@jakepruitt Thanks for the update. I'm now getting an "Error: EACCES: permission denied, open '/etc/init/spotswap-poll.conf'". Also, is INSTANCE_ID for an instance (any) that's not part of the scaling group? More like an Admin box?
I'm more of an Ops than a Dev so perhaps I'm approaching this the wrong way. Is there a CloudFormation template I can use and a Lambda function zip I can just upload?

INSTANCE_ID should be the ID of the instance that spotswap is currently running on, pulled from the metadata endpoint http://<IP_ADDRESS>/latest/meta-data/instance-id during a UserData script run on an instance. An example UserData script you'll probably need should look something like this (you'll need to clone your repo into ${deploy_dir} on the instance, and set ${stack} as the AWS::StackName):
instance=$(curl -s -m 3 http://<IP_ADDRESS>/latest/meta-data/instance-id)
asg=$(aws cloudformation describe-stack-resource \
    --stack-name ${stack} \
    --logical-resource-id AutoScalingGroup \
    --query StackResourceDetail.PhysicalResourceId \
    --output text)
SpotGroup=${asg} \
terminationTimeout=0 \
INSTANCE_ID=${instance} \
AWS_REGION=${region} \
${deploy_dir}/../node_modules/.bin/spotswap-install

I'm not sure about the user permissions of the UserData script, but I assume it already is or can be configured to be a user that has access to the /etc/init directory, which sounds like the problem you ran into.
In terms of the lambda function zip and cloudformation template, you'll need to write your own template in javascript and include the @mapbox/spotswap module as a npm dependency (Check out the usage section of the readme for an example of the things you'll need in your template).
In terms of lambda bundles, I suggest zipping up the repo of the code you're planning on running for the cloudformation stack (with the node_modules installed, and @mapbox/spotswap defined as a dependency) and uploading it to S3. At Mapbox, we use https://github.com/mapbox/stork/ to use CodeBuild to build our github repositories into zipped bundles on S3 (if you're doing anything with lambda, I'd definitely recommend taking a look at this from a devops standpoint).


ELB Healthcheck Intervals are too lax

We use the default HealthCheck when building ELBs, and those are far too lax.  We might want to pull this into the extended procfile support.
http://docs.aws.amazon.com/ElasticLoadBalancing/latest/DeveloperGuide/elb-healthchecks.html

This one got closed an accident from https://github.com/remind101/empire/pull/803, but very easy to change now.
Right now, the cloudformation stack doesn't specify any health checks, which means you can alter the health check by hand after the ELB is created (like before). I think it'd be good to wait for health checks to be defined in the extended Procfile before we add health checks to the CloudFormation stack, so that they can still be configured.


Add timestamp metrics to ecr

It is fairly common to see timestamps in seconds as gauge metrics for events. These should be the UNIX timestamp in seconds and NOT a "time since the event" in seconds.

Released in #5


[docs] Fix Typo in BottomNavigationAction label

@ovidiumihaibelciug My bad, I have added the extra s to check that the hook version was correctly mounted in the page.


Rename "drawer" to "tabs + drawer"

When choosing a navigation type in the cli, you currently select between "stack", "tabs", or "drawer". This is misleading as "drawer" actually includes a tab controller as well.

can i work on this?

Go for it

I was looking at the code but couldn't found where this needs to be changed, can you please help me with this ?

cli/src/utilities/runCLI.ts
Line 43 and 55
I would also make sure that you've read through the contribution guide linked in the readme of this monorepo

In the event that someone wishes to work on this, please do so, due to other commitments I will not be able to participate, I apologize for the inconvenience.

I would like to take this.

@neerajkumarc all yours! ty


Allow user system CA's for Android

Summary
This will allow Android to establish a network connection with any server that uses a Self sign certificate that has the CA certificate installed on the device
More info here  and in this blog post
Ticket Link
Resolves #2139

Thank you @enahum!

Android build App: https://releases.mattermost.com/mattermost-mobile-pr/PR-2301/PR-2301.apk
iOS build App: itms-services://?action=download-manifest&url=https://releases.mattermost.com/mattermost-mobile-pr/PR-2301/PR-2301.plist

@bbodenmiller great to hear, thank you


Tabs to navigate the app

users can open maximum 5 tabs
tabs can be closed
all tabs are rendered, but only one at a time is visible
persistent navigation state is no longer supported

Maybe we can remove the word 'search' ? Try to fit as much descriptive text as possible

Also the dropdown doesn't close when you click on the button again

Something weird with search input when resizing


Combine Execution and Audit Logs in Request Status Endpoint [#1024]

Purpose
We have AuditLogs that store user actions at the PrivacyRequest level, and ExecutionLogs that describe system actions at the individual Collection level.  There is information from both of these tables that is useful in establishing the timeline of the request.
Unify/combine this information into a single endpoint.
Changes

Update existing request status endpoint, so when the verbose query param is used, e.g. http://<IP_ADDRESS>:8080/api/v1/privacy-request?include_identities=true&request_id=<privacy_request_id>&verbose=true, both execution logs and audit logs are included in the response

Behind the scenes, this is a union all query.  Both tables share some columns and have unique columns of their own, so if a column is missing from one table, we return null.
Current API returns collection logs grouped by dataset name, so we just give the audit log a grouping related to its request status, and then it has a single log associated, the audit log.
Existing pagination is used, in that we limit embedded logs per dataset to 50.

Checklist

[x] Update CHANGELOG.md file

[x] Merge in main so the most recent CHANGELOG.md file is being appended to
[x] Add description within the Unreleased section in an appropriate category. Add a new category from the list at the top of the file if the needed one isn't already there.
[x] Add a link to this PR at the end of the description with the PR number as the text. example: #1

[x] Applicable documentation updated (guides, quickstart, postman collections, tutorial, fidesdemo, database diagram.
If docs updated (select one):

[ ] documentation complete, or draft/outline provided (tag docs-team to complete/review on this branch)
[ ] documentation issue created (tag docs-team to complete issue separately)

[x] Good unit test/integration test coverage
[ ] This PR contains a DB migration. If checked, the reviewer should confirm with the author that the down_revision correctly references the previous migration before merging
[ ] The Run Unsafe PR Checks label has been applied, and checks have passed, if this PR touches any external services

Ticket
Fixes #1024

It's ready to merge once the changelog conflicts are fixed.

thanks for the review @TheAndrewJackson !


improve startup error logging

Description
improves startup error logging
Related Issue

Fixes https://github.com/owncloud/ocis/issues/3460

Motivation and Context

How Has This Been Tested?

first start ocis server then start ocis xxx server and see a speaking error message

Screenshots (if appropriate):
Types of changes

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] Technical debt
[ ] Tests only (no source changes)

Checklist:

[x] Code changes
[ ] Unit tests added
[ ] Acceptance tests added
[ ] Documentation ticket raised:

:boom: Acceptance test Core-API-Tests-ocis-storage-10 failed. Further test are cancelled...

The sonarcloud scanner is failing because of the high amount of duplication. This is unavoidable here so I'll force merge it.


How to add borders in neofetch

Description
Hello, I don't quite know if neofetch has this but i wanted to put a border to the other side but cant figure out how, can anyone please help?
If you're suggesting a new feature then just a description will suffice.

[X] Does this issue still occur in the master branch? (Required if issue)

Neofetch version
7.1.0
Screenshot

Config file
Here
Verbose log
Not needed

I like this!

Currently there is no way to add a right border to neofetch ?


[Splashscreen][Android] Fails to deploy/build after first successful run on device/emulator below API 31

Create a Blank of Default Uno app from the templates
Launch application onto a device/emulator at API level 30 or below
Attempt to launch the application for Visual Studio again (no matter that API level)
Observe:

APT2062: failed linking references
resource drawable/splash_screen (aka com.companyname.UnoApp12:drawable/splash_screen) not found"

Performing a clean or rebuild avoids the problem

cc: @nickrandolph + @pictos

This has already been reported on both uno and resizetizer repositories, but a third time can't hurt :-)
#resize

Should be fixed now.


请问原始训练的代码能开源吗?

RT，想学习学习。

Very Good! Could you supply a train.protoxt in GoogleNet ?I want to finetune this model


Bugfix HTTPS SNI and IP Address

Motivation:
Solving the SNI Bug
Modifications:
Added an internal extension on String for checking if the hostname is an IP Address -- see the private extension on SNI. Additionally using the IPv4Address and IPv6Address Function from Network above 10.14 as protecting with #availabe.
Adding the test for HTTPS and IP in as hostname
Result:
We get results with an IP as Hostname

anyone else want to review ? to fix that bug?

@artemredkin or @weissi want to take a look here as well?

@swift-server-bot test this please

@swift-server-bot test this please

@swift-server-bot test this please


added PlayerProfile

Created PlayerProfile (as a separate entity from User)
A User can be both a Player and a Host, if we implement Player and Host as subclass of User, everyone that is both Player and Host will have 2 User objects relevant to them (which is pretty wonky)

Mapped PlayerProfile as OneToOne with User
I added @JsonIgnore in PlayerProfile to get rid of the cyclic dependency between PlayerProfile and User (or else when you try to add PlayerProfile to a User, they recursively print each others' value). I don't know if this is the right way to handle it but ya

Changed Club references to User to playerProfilleID
I think it makes more sense to take in the ID and handle the getting of PlayerProfile using ID at the service level

Added http tests for Club and Player

All relevant improvements in code and makes the back-end implementation a lot cleaner, nice work. HTTP tests is a nice addition as well, to make it easier for us to test our classes as we go.
To build on this, we may consider using @JsonManagedReference and @JsonBackReference for a cleaner way to handle cyclic dependencies between PlayerProfile and User, but @JsonIgnore works fine for now.


material-ui-next.com is down

material-ui-next.com has been down for a couple of minutes now :(

@messutied I'm migrating from Firebase to Cloudfront.

Thanks, its already back up :)

It's down again


Kanban Board Info

Click here to view the GitHub project.
There are three members in the team, the following are their names;
Team member 1: @sameedmohsin28
Team member 2: @federicaulzurrun
Team member 3: @Mylo16

Hi @TEAM,
Good job so far!
There are some issues that you still need to work on to go to the next project but you are almost there!

Highlights

No linter errors ✔
Group work ✔
Good implementations ✔

Required Changes ♻

[ ] [REQUIRED] Good job so far! Please each card on the board should have a descriptive description of the task 🙏🏼 Kindly add description to these cards:
#14,
#21,
#28

Optional suggestions
Every comment with the [OPTIONAL] prefix is not crucial enough to stop the approval of this PR. However, I strongly recommend you to take them into account as they can make your code better.
Cheers and Happy coding!👏👏👏
Feel free to leave any questions or comments in the PR thread if something is not 100% clear.
Please, remember to tag me in your question so I can receive the notification.
Please, do not open a new Pull Request for re-reviews. You should use the same Pull Request submitted for the first review, either valid or invalid unless it is requested otherwise.

As described in the Code reviews limits policy you have a limited number of reviews per project (check the exact number in your Dashboard). If you think that the code review was not fair, you can request a second opinion using this form.

Hi Team 👋,
STATUS: APPROVED 😃
In my opinion, your project is complete!
Good luck! 🎉
Feel free to leave any questions or comments in the PR thread if something is not 100% clear.


qrcode for reception reads bitcoin:someaddress

zbarimg Screenshot\ from\ 2017-08-09\ 16-59-13.png  QR-Code:bitcoin:1D3zcccccccccccccccccccccccccccccccccayH scanned 1 barcode symbols from 1 images in 0.01 seconds
Message from coinomi_angelos: https://www.reddit.com/r/COINOMI/comments/6s0zv5/cant_send_bch_out_of_wallet/dla2nmv/
"This is a known issue, and it involves the URI in the QR of the service providing the address you're sending to. Most commonly it has the "bitcoin:" URI, while our application expects "bitcoincash:" "
So, I don't know if a standard is set up for this, but coinomi seems to expect bitcoincash: as tag instead. Another solution is to just drop the bitcoin: tag.

duplicate of #35

This might be a serious issue.
What if someone send BTC to  electron cash address by accident?
It seems to me it can happen and once it's sent, it will be lost forever.

Not lost forever, you can see the transaction in a bitcoin wallet (for instance electrum) using the same seed and key derivation path.
By the way, I discovered the problem when I wanted to pay to an electron cash wallet from coinomi. I used the qrcode including the bitcoin: uri, and the coinomi wallet too helpfully arranged a trade with shapeshift to receive a corresponding value of btc in the address. In my case I stopped the action, but reports on reddit r btc indicates that at least one person (thinks) he lost the coins.

that's something that needs to be fixed right away to prevent further mistakes.


Implement Day 4 Part 1

Link to problem description: https://adventofcode.com/2023/day/4

Ugh... Just a classic array search problem... I could go through and optimize the heck out of this thing; implementing some sorting algorithms, and some clever search method. Butttt, I really don't feel like going crazy with it. Let's just aim to make the simplest solution possible, using only Node.js built-in tools.


[Bug]: CSSOM.parse is not exposed in the package used by jest-environment-jsdom

Version
29.0.3
Steps to reproduce
I don't have a repro currently but error is generated from @material-ui/styles since switching to jest>28 and  jest-environment-jsdom.
Expected behavior
The issue is already identified https://github.com/NV/CSSOM/pull/113/files and a PR is even available but CSSOM doesn't seem to be the most maintained lib.
Actual behavior
Jest tests are generating a lot of warning in our monorepo setup
● Console

    console.warn
      Warning: [JSS] CSSOM.parse is not a function

      53 |   const registerScroll = useRegisterScrollElement()
      54 |
    > 55 |   const classes = useAppLayoutStyles({
         |                                     ^
      56 |     innerHeight,
      57 |     showNavInDesktopOnly: navType === 'desktop-only',
      58 |   })

Additional context
No response
Environment
System:
    OS: macOS 12.3
    CPU: (10) arm64 Apple M1 Pro
  Binaries:
    Node: 16.10.0 - ~/Library/Caches/fnm_multishells/1912_1663926248111/bin/node
    npm: 7.24.0 - ~/Library/Caches/fnm_multishells/1912_1663926248111/bin/npm
  npmPackages:
    jest: 29.0.3 => 29.0.3

having same issue as well

Sounds like an issue for JSDOM and not Jest?

Yeah, but JSDOM is not actively maintained and fixed PR not being merged: https://github.com/NV/CSSOM/pull/113
A fork version with fix has been created: https://github.com/rrweb-io/rrweb/pull/925
Can we use https://github.com/rrweb-io/CSSOM instead?

JSDOM is very much actively maintained. CSSOM is a separate project.
If you want JSDOM to use a fork, you should ask them: https://github.com/jsdom/jsdom/blob/c3c421c364510e053478520500bccafd97f5fa39/package.json#L26


Resolving a good amount of open issues

Posting in draft so people can see what's coming.
There's a few more things I'm considering. I've listed all of them below (I think). Feel free to leave comments / ask questions.

Version number scheme (was 0.9.5, but changed to 0.0.1, then to <IP_ADDRESS>)
Checksum support
Atomic ACLs API plugin support
Default configuration values
TLS/SSL with iRODS server
Naming of things
Including buffer sizes and other config info in /info response
Documentation

For the most part, I think the majority of things are supported.

So far, all python tests are passing.
@trel Let's tackle the correctness of the API documentation after merging this PR. Linking your comments to the wiki issue.

Let's tackle the correctness of the API documentation after merging this PR

agreed.

All squashed.
Working on creating/assigning issue numbers.

All python tests pass.
Added #'s.
@trel Please merge when you get a chance.


[Reviewed] [Autotyping] Simplify the description

Don't show in changelog

Did you create a wiki page to share the info you are deleting?  (I didn't see the link to it)

Did you create a wiki page to share the info you are deleting? (I didn't see the link to it)

I moved useful information in other descriptions that better fit the purpose and removed what is obvious.
For instance:

I removed "Change the typing speed with the Change interval action" because this is action description is already "Change the time between characters being typed.
I moved "When the text changes, the automatic typing starts again from the beginning with the new text" in the "Start at the beginning of the text" action description because this is where users need to know that calling this action maybe not be necessary. If they read the extension description entirely (which I doubt), I bet they wouldn't remember it at this point.

Should we go ahead with this?


Wrong bracket pair colorization

Issue Type: Bug
The match for "end" in bracket pair colorization should read something like [^0-9a-z]end[^0-9a-z].
The current match will colorize the "end" part of extends.
VS Code version: Code 1.60.0 (e7d7e9a9348e6a8cc8c03f877d39cb72e5dfb1ff, 2021-09-01T10:41:52.311Z)
OS version: Windows_NT x64 10.0.19042
Restricted Mode: No

Duplicate of #132162

/duplicate


Fix test ContainerizedDispatchManagerTest#testRestartFlow to use thread.join instead of wait

Pull Request Test Coverage Report for Build 6524

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 45.472%

Totals

Change from base Build 6522:
0.0%

Covered Lines:
17990

Relevant Lines:
39563

💛  - Coveralls


[SPARK-31932][SQL][TESTS] Add date/timestamp benchmarks for HiveResult.hiveResultString()

What changes were proposed in this pull request?
Add benchmarks for HiveResult.hiveResultString()/toHiveString() to measure throughput of toHiveString for the date/timestamp types:

java.sql.Date/Timestamp
java.time.Instant
java.time.LocalDate

Benchmark results were generated in the environment:

Item
Description

Region
us-west-2 (Oregon)

Instance
r3.xlarge

AMI
ubuntu/images/hvm-ssd/ubuntu-bionic-18.04-amd64-server-20190722.1 (ami-06f2f779464715dc5)

Java
OpenJDK 64-Bit Server VM 1.8.0_242 and OpenJDK 64-Bit Server VM 11.0.6+10

Why are the changes needed?
To detect perf regressions of toHiveString in the future.
Does this PR introduce any user-facing change?
No
How was this patch tested?
By running DateTimeBenchmark and check dataset content.

@cloud-fan @juliuszsompolski @HyukjinKwon Please, review this PR.

Test build #123646 has started for PR 28757 at commit 37f5ba1.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/28269/
Test PASSed.

Test build #123646 has finished for PR 28757 at commit 37f5ba1.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/123646/
Test PASSed.

I'm merging it to 3.0 as well, so that it's easier to compare the perf between 3.0 and 3.1 in the future.
thanks, merging to master/3.0!


Think about video vs audio chat

This is likely low enough priority we won't get to it by the event, but wanted to word-vomit anyway.
Right now, we default to video chat, although you can disable your camera (and theoretically in the future can disable others').
Videochat is more communicative but more daunting and emotionally exhausting. Audio has a lower emotional barrier to entry.
Our current design encourages video over audio-only chat. I'm not sure this is the right choice. Worth thinking about whether we want to nudge people towards videochat, or if we by default put them in audio and then give them ways to opt-in to vide.
When I say I'm not sure our current solution is the right choice, that isn't me saying "we should be more audio-focused". I legitimately don't know the right answer here.

It looks like this could be treated on... spectra? This new-ish project is very promising: https://github.com/highfidelity/Spatial-Audio-API-Examples/tree/main/experiments/nodejs/videochat-twilio


Update to Akka 2.4

Update to akka 2.4.0-RC1 (and Scala 2.11.7 and sbt 0.13.8), don't merge yet!

LGTM

updated doc links in https://github.com/typesafehub/activator-akka-supervision/pull/12


android php compatibility with this library support ?

hi, i want to use this library in my php code , so please can you help to implement this library in php side ?

Hi Achinverma. Thanks for your interest in our library. We don't have a PHP implementation. Are you looking for something in PHP that will be compatible (e.g. on the server side), or just something to perform similar functionality (encrypting strings).
thanks,
isaac

hi @SyntaxPolice i am looking for a library which is compatible  with both android and php server side for encrypting and decrypting strings , please let me know if you know this kind of library.

Achinverma. Sorry I hadn't gotten back to you. I didn't realize this issue was still open after Aug. 20.
We don't know if a PHP library that has the same functionality. Implementing one may or may not be difficult, depending on whether PHP crypto libraries offer the same ciphers and modes. In practice, we had tried to implement a compatible library in Javascript and found it challenging because of the restrictions in many languages in the availability of aes modes.
It may be that you are looking for something more like our upcoming product tozny.com/e3db. If that looks interesting, please sign up for the pilot.
thanks,
isaac


A cell containing the character ; breaks the CSV output

Hi, I'm converting a file containing a ; character in a cell, and the resulting CSV is broken because that's the delimiter character.
A good rule would be to wrap the text value with double quotes " when the text contains the delimiter character, since many CSV parsers and serializers also use this logic. (see Papaparse and d3.csvParse)
Also, it would be really useful to be able to change the delimiter and not have it hardcoded to ;.
It seems pretty straightforward to change both in the Rust sources, but I don't have much Rust knowledge. Would you like me to try a PR?

Hi,
Thank for these proposal.
Everything is implemented in the new release v2.0.0.
Please take a look, also added some documentation.

Example:
import {convertFile} from 'your-library-name';

// Convert an Excel file to CSV and save it to a specific path
convertFile('example.xlsx', {
    sheet: 'Sheet1',
    delimiter: ',',
    dest: 'output.csv',
    datetimeFormat: '%d-%m-%Y %H:%M'
});

// Convert an Excel file to CSV and return the result as a string
const csvData = convertFile('example.xlsx', {
    sheet: 'Sheet1',
    delimiter: ';',
    dest: 'return',
    datetimeFormat: '%Y-%m-%d'
});

console.log(csvData);

Wonderful thanks! Will try that right away. 🎉


New Challenge Mode times overwrite Reverse times

I know the Challenge and Reverse Challenge modes are new, but I noticed today when I completed Morqut Ascent on the Challenge mode, the addon announced a new time for the Reverse version of the Advanced track.
Please let me know if I can provide any additional details. I really love this addon and want to help if I can. Thank you!!

Was the saved time updated?

Yes, the Challenge mode time was saved as the Reverse time. My Reverse time was originally 51.336, but now shows as the Challenge time of 55.521.

When rerunning the Reverse course, the Reverse entry reverts to my best Reverse time.

I will take a look over it this weekend.

Popping in to check up on this issue, and make sure everything with you is good. No rush for an update, just wanted to make sure you were doing well.


Allow conditionally exporting variables

This is done except for the final syntax. See #922 for discussion.

This is a bit insane, and I can't figure out how to make this less insane, so I'm going to close it until such time as less insane syntax and semantics are discovered.