Update junit to 4.13.2

Updates junit:junit from 4.13.1 to 4.13.2.
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "junit", artifactId = "junit" } ]

labels: library-update, semver-patch

+1

[X] Do all automated continuous integration checks pass?
Yes
[X] Is the update a patch, minor, or major update?
Patch release, likely changes have no affect on Daffodil
[X] Is the license still compatible with ASF License Policy?
Yes, same license and no changes to ASF policy, note that we do not distribute junit with Daffodil
[X] Have any changes been made to LICENSE/NOTICE files that need to be incorporated?
No changes
[X] Have any transitive dependencies been added or changed?
No changes to transitive dependencies


fix: fix the typo in scheduler crp integration tests

Description of your changes
fix a bug in #489
Fixes #
I have:

[X] Run make reviewable to ensure this PR is ready for review.

How has this code been tested

Special notes for your reviewer

LGTM.


Adding scaffold to gem

This fixes #67 by changing the gemspec to include the scaffold folder.
Also updating the the gemspec so it's a simple bash pipeline that includes the grep, which makes it easy to see what it returns.
and removed adding the spec.files value twice.

Not sure why the Ruby grep is any less direct.

Thanks for the fix!

@parkr, I changed it to simply copy and paste the line in the terminal to triage what is the output... adding the 'scaffold/' pattern is what counts.

@parkr, I changed it to simply copy and paste the line in the terminal to triage what is the output... adding the 'scaffold/' pattern is what counts.

Ok. Just worried about systems without grep. The Ruby grep was still quite clear in my opinion while still offering cross-platform capability. I was curious because the PR description seemed to indicate that a shell grep offered an advantage.


[Bug] Unstable Test case MySqlSyncDatabaseTable

Search before asking

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

Paimon version
master
Compute Engine
flink
Minimal reproduce step

https://github.com/apache/incubator-paimon/actions/runs/5830902368/job/15813230854?pr=1734
What doesn't meet your expectations?
make it stable
Anything else?
No response
Are you willing to submit a PR?

[ ] I'm willing to submit a PR!

I think #1783 fixes it.


REST-API to manage Zone Records by Accounts - Feature Request

Found a similar feature on powerdns-auth-proxy and DjangoPowerDNS. It would be great if PowerDNS-Admin would expose it's own API key for each zone so that multiple teams could update their own zone records programmatically.
List Records in a zone
Create a New Record
View a specific Record
Edit a specific Record
Delete a specific Record

I think we can integrate with flask-restplus which has Swagger to expose needed API.


WIP: Add middleware to process multiple tokens concurrently

This is done through a new middleware structure called
MultiTokenMiddleware, which will concurrently validate tokens against
multiple endpoints. These endpoints are meant to be generic, but an
initial implementation is the JWT middleware that we already have in
this repo.

This is no longer needed.

We might need this after all

Codecov Report

Merging #4 (905a902) into main (5ed6c76) will increase coverage by 4.69%.
The diff coverage is 86.92%.

@@            Coverage Diff             @@
##             main       #4      +/-   ##
==========================================
+ Coverage   72.30%   77.00%   +4.69%
==========================================
  Files           4        6       +2
  Lines         130      200      +70
==========================================
+ Hits           94      154      +60
- Misses         27       35       +8
- Partials        9       11       +2

Flag
Coverage Δ

unittests
77.00% <86.92%> (+4.69%)
:arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

ginjwt/multitokenmiddleware.go
79.48% <79.48%> (ø)

ginjwt/testtools.go
72.97% <84.61%> (-2.03%)
:arrow_down:

ginjwt/jwt.go
82.60% <88.33%> (+1.65%)
:arrow_up:

ginjwt/errors.go
100.00% <100.00%> (ø)

Continue to review full report at Codecov.

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


More video window configuration options

Add some more options as proposed in this KVR forum post:

show timecode
show frames
window size

I add some more ideas:

window position
full screen (if it's possible to send the display number as well)
disable user interaction with the video window (found in the remote control commands)

I think it's the best to realize these video window options as global preferences and not per-project but time and users will tell ;-)

timecode and frame number options will be added with #4


is there any plan to migrate to .net core 3.1 and ef core 3.1

is there any plan to migrate to .net core 3.1 and ef core 3.1, MS has fixed 150 issues in ef 3.1. i will be glad if this is made available.

Fixed with #69


Visualization error: segmentation fault

I tried to create a video using multiAgentMakeVideo.py but I got the following error:  Segmentation fault in expression starting at no file:0
getproperty at ./sysimg.jl:18 [inlined]
getindex at ./refvalue.jl:32 [inlined]
handle_new_from_data at /home/user/.julia/packages/Rsvg/uwc9D/src/calls.jl:109
render_fancy_car at /home/user.julia/packages/AutoViz/Kccwt/src/fancy_render.jl:40
I tried to change AutoViz to v.0.6.0 but then I got other issues. Suggestions are appreciated!

I successfully reproduced your error with AutoViz v0.7.3. Reverting back to AutoViz v0.7.1 resolved the issue for me and I was able to create a video. Could you please provide further information about what other issues you faced?


Turn off the keyboard

Hey, great module it's been really useful! Is there a way to just disable the keyboard completely? I've tried keyboard = false but no luck.
Cheers.

Hey @daviddiamond1 I'm curious, what is the use case of completely removing the keyboard? I mean, why do you want an input where you can't write?

@ajimix I want to be able to turn off the virtual keyboard, for web

Oh ok, you should use the virtualKeyboard option. Like this:
input = new InputModule.Input
  virtualKeyboard: false # Enable or disable virtual keyboard for when viewing on computer

I'm closing this issue. Please reopen it if you still have problems.

Ah, thanks Adria, might be worth adding to the list of options in the readme too 👍

It was already there, it's the second option :)

🤦♂


Forcing width when resizing

Hi!
I looked over similar questions but none of the answers work for me, so. My problem is simple: I want to enforce specific width in @resizing event. The point is for the object being resized to be constrained by its parent, but setting :parent prop doesn't work for me. Maybe because the parent is a table cell, so my resizable blocks appear somewhere else but not in cells when I set parent restriction. But anyway, I also support dragging between cells, so restricting it to parent won't work for me.
So I wanted to implement the constraint manually, but changing :width in @resizing event doesn't do anything. Could you give me some advice on how to change object's width so that when user is resizing it, the width won't go over the specific value (will be forced to stay at some max value that is dynamically calculated).
Thanks!
P.S. Setting maxwidth won't do as well because my object may be placed for example in a middle of a cell. So then user would resize to the right and maxwidth will go over the cell's border.

At the moment I solved it by setting this.$el.style.width to the maximum allowed value when user tries to go over borders. Though it seems like a hack and I really would love to find a more component-related solution, like changing :width reactively (which I expected to work but it doesn't).
It also have some downsides:

While the width set up this way works visually (user sees that the resizeable component won't grow further), component's width value still increases in the background. That could lead to misunderstanding if user would try to decrease the width that was set to a maximum value, because inside the component the width is greater.

To fix 1) I had to reset component's width by changing the prop from parent component. Eventually it resulted in the component being recreated which is all right by itself. But when the component is recreated it looses resize handles and user may still want to resize it more.

To fix 2) I used active prop for components re-created after resizing, so that newly created components would "keep" the handles. But it resulted in some weird behaviour: while setting this.$el.style.width works the way I described above, it doesn't work if the draggable component has it's active prop initially set to true. Somehow in this case user's resizing overlaps my way of width enforcing and user can visually pass the borders. Though if user would deactivate the component by clicking on the outside and then make it active manually, the width enforcing works again.

So it raises an additional question: what is different in the component when the active prop is initially set to true in comparison to the component made active later by user clicking on it?

Hi!
First of all, thank you for reporting the issue with this level of details. Much appreciated!

changing :width in @resizing event doesn't do anything

w and h are reactive props, but they are disabled during dragging or resizing because otherwise they would interfere with the internals of the component. I know that this is not the best idea and often it causes confusion, but I'm afraid that solving this problem would require a huge rewrite of the component.
Back to your problem: there's no way right now to force a specific width during resizing or dragging, but you can use the resizestop and dragstop events to revert the size of the component to the desired values.

Hello!
Thank you for the reply!
Reverting width on resizestop and dragstop events is what I did and it eventually resulted in the small issue I described in my second comment. In a nutshell, to process new width and position in the parent component I had to recreate the draggable-resizable component with new width and position. That's okay but the new component appears without resize handle which is not good if user was resizing the component. So I'm using active prop to keep handles visible. But when active prop is initially set to true, my $el.style.width hack behaves differently than when user manually clicks on the component to make it active. Basically, it doesn't work when the component is initially active.
So maybe you know of some difference in the component behavior when it is active from start comparing to made active by the user?

w and h are reactive props, but they are disabled during dragging or resizing

Please specify this point in the documentation, I spent an hour before I found this thread.

Sure, but it's an open source project which means that PRs are always welcome.

@graker  I have the same issue with re-setting width as you describe, but also when I want to reset x,y of the component, not active as in your case. But I think the source of the issue is the same.
While playing around with await nextTick(); I have found out that if I reset w,h first, then wait for the next tick, and then reset x,y in a separate operation - sometimes it works as I expected. And sometimes not.
My issue is that if I did drag and resize, so I changed x,y, and w,h - when I reset them back, the component processes w,h change first and somehow internally calculates new x,y based on changed w,h.
This calculation might be async, therefore sometimes after w,h change when I apply my stored x,y it does place the component in the right previous position, and sometimes it places it based in the top-left corner, where the new resized coordinates were.
I think the same happens in your case. You change width, it updates something internally and asynchronously, and then you change active, and it causes a second update cycle, but the behavior is not predictable, as the first async update might have ended or not yet.
Because of this update issue, I find this component not usable, as I simply cannot reset w,h,x,y to initial values by setting them back in a rectangle without experiencing weird async rendering issues.
@mauricius FYI

this is the solution that works for me. I do reset rectangles to an empty array, removing them, then wait until they are removed from being rendered with nextTick and then add them back from saved state.
  const resetToInitialState = async () => {
    state.rectangles = [];

    await nextTick();

    state.rectangles = [
      ...state.initialRectangles.map((initial) => reactive({ ...initial })),
    ];
  };

this is the only way it was possible to render dragged and resized rectangle back to the initial state and with the proper visual position.


Feature Request: example of :+:

Although I can understand not supporting sum types out of the box, as it isn't obvious what those instances should look like, I would find it helpful/useful to have some example code that facilitates automatic derivation of CStorable instances for sum types, perhaps as part of the documentation if not an official part of the library.
Feels a little strange jumping right in to this request, without saying it so let me also add: Thankyou for what you are doing/have done. This is a useful package.

CStorable is intended to represent the way to access in memory the C analog of a given type.
As far as I know, C does not actually support true sum types, thus the lack of support.
It might be sensible to write a CStorable instance for sum types in the case where all constructors of the type took no arguments (by translation into an enum).
Is that what you were thinking of, or have I missed the mark? (Sorry for the slow response)

well, i think it would be more useful to have something like

data Thing = Foo { foo1 :: CInt} | Bar { bar1 :: CChar }

becomes two c types, and a union for the fields like so
enum ThingType {
  thingTypeFoo,
  thingTypeBar
};

/* for missing field names just append a number to the data constructor made lower case */
union ThingFieldsUnionTag {
   foo1 :: Int
   bar1:: char
};
typedef union ThingFieldsUnionTag ThingFields;

struct ThingStructTag {
  ThingType type;
  ThingFields fields;
};

typedef struct ThingStructTag Thing;

The idea is to have a common representation that works, like i said, it doesn't necessarily have to be implemented in the library, but it just would be nice to at least have an example documenting how to do this.

A more general example, with multiple fields:

data Thing = Foo CInt CChar CInt | Bar CChar CInt | Baz

becomes in c:
enum ThingType {
  thingTypeFoo,
  thingTypeBar,
  thingTypeBaz
};

struct FooStructTag {
   int foo1;
   char foo2;
   int foo3;
};
typedef struct FooStructTag Foo;

struct BarStructTag {
  char bar1;
  int bar2;
};
typedef struct BarStructTag Bar;

struct BazStructTag {
};
typedef struct BazStructTag Baz;

/* for missing field names just append a number to the data constructor made lower case */
union ThingFieldsUnionTag {
   Foo foo;
   Bar bar;
   Baz baz;
};
typedef union ThingFieldsUnionTag ThingFields;

struct ThingStructTag {
  ThingType type;
  ThingFields fields;
};

typedef struct ThingStructTag Thing;

Sorry I've been so slow to respond.
I'm fully aware that it's possible to represent a union in C by type tagging a union. However, that's not the point of CStorable. CStorable is for typing out the direct analog of a C type and being able to interact with the C representation. What you're describing sounds like it'd make sense as an implementation strategy for Storable, possibly using Generic to allow your implementation to be easily plugged in for a given type.
I think the miscommunication here is that CStorable is not for "here is a way of representing this in C", it is for "there is already a C representation for this, how do I gain the ability to read/write it in Haskell". While the type tagged union is a common pattern in C, unless you're seeing this exact formulation in a bunch of APIs (as otherwise it'd be incompatible) I don't think this library, or even the typeclass CStorable is the right place for it.


Log everything

Hi guys, in advanced thanks for the effort, really.
I need to log everything that my users do in the app... How can I achieve that without writing model per model the fields to be listened?
thanks a lot

https://docs.spatie.be/laravel-activitylog/v2/advanced-usage/logging-model-events
Just use $logFillables.

@Gummibeer Thanks a lot for the response Gummibeer. That is exactly my question; following the documentation and your link, we all have to write each column to listen at for changes. Isn't there an option to listen just ALL the columns in all the models?

@leopinzon no, after the first code-block it says:

If you want to log changes to all the $fillable attributes of the model, you can specify protected static $logFillable = true; on the model. Let's see what gets logged when creating an instance of that model.

With this it will log changes on every fillable attribute. If you want to log every-everything you can also use the wildcard * - just to say it: this will also log potential private informations (passwords, tokens and other sensitive data).

The attributes that need to be logged can be defined either by their name or you can put in a wildcard '*' to log any attribute that has changed.

I tried these options without much luck. Maybe (and I apologize for not having mentioned before) it is because I'm using the 1.* version of the package since my Laravel version is 5.3.  Anyway, writing each important column to listen at in the $logAttributes variable is working like a swiss clock. More writing at the beginning but awesome reward in the long run. Thanks gummi!

You couldn't be more right, the only problem in my case is that the system is built on a LaraAdmin system that supports - at the most- Laravel 5.3. Not happy with that but trying to make the best of it.

Hi @leopinzon, this is what I did (laravel 5.6), I hope it helps you on 5.3...
I created a Class named BaseModel which extends from Eloquent's Model and added the attributes for log there:
protected static $logAttributes = ['*'];
protected static $ignoreChangedAttributes = ['updated_by','updated_at','last_action'];
protected static $logOnlyDirty = true;
The simply extend this Class in all of your models instead of Eloquent's Model

@JoseDguez the wildcard isn't available, following the docs, in v1. But the idea with the BaseModel is good - I've switched to traits cause they are also bootable and more modular. If I need one model without a single feature but with all other I can simply don't use the particular trait. With the BaseModel I had to copy all needed features or add feature switch properties.


Inference on a multiprocessing CPU

I am trying to do an inference on a a cpu :  i7-6700K CPU. The CPU has a total of seven cores . Does the inference_detector take into account multiprocessing when computing inference on a CPU?
If not is there a work around that exists ?

Not sopport yet.


[BUG] Could not run WebCam notebook end to end - should this be our first notebook?

Description

Could not run it , the last cell on show widget ran successfully but did not show the widget. There is a duplicate issue for it. But overall my feedback is should we keep webcam notebook as the first notebook, because many users will face similar issue and could have bad experience and can easily give up on other notebooks, is their a way to simplify this notebook?
In which platform does it happen?

Linux Ubuntu

GPU

Yes
How do we replicate the issue?

Expected behavior (i.e. solution)

Other Comments

@AnupamMicrosoft May I ask which webcam notebook?

https://github.com/microsoft/ComputerVision/blob/master/scenarios/detection/00_webcam.ipynb

@AnupamMicrosoft Could you give more details about how you run the notebook?  I recorded a video of running the notebook on LInux GPU DSVM.  Could you tell me any difference?


Inconsistencies in model_json_schema with custom fields

Initial Checks

[X] I confirm that I'm using Pydantic V2

Description
Situation
As a user, I have a pydantic model with multiple fields which use, or not custom JSON fields. The type of theses fields is a custom type with custom JSON fields. I want to use .model_json_schema() to export my instantiated model
Expected Result
Each attribute has it's own in custom fields and the custom fields of the type. If there is a conflict, the custom fields of the attribute takes precedence.
Actual Result

If the attribute has no custom fields: the custom fields of the custom class will be used
If the attribute overwrite the custom fields of the custom class: only the custom fields of the attribute will be used
If the attribute use a new custom field: only the custom fields of the attribute will be used

Additional information

I tried to use json_schema_extra in Field and to define it as Field(custom_field='foo',  ...)
'Normal' arguments in fields foes not seem to affect the result (I tried with alias)

Example Code
from typing import Annotated

from pydantic import (
    BaseModel,
    Field,
)

StrWithFields = Annotated[
    str,
    Field(json_schema_extra={
        'custom_field': True,
        'other_field': 'other_field'
    }),
]

class Model(BaseModel):
    my_field: StrWithFields = Field('foo')
    my_field_with_overwrite: StrWithFields = Field('bar',
                                                   custom_field=False)
    my_field_with_other_field: StrWithFields = Field(
        'bar',
        new_field='my_field'
    )

print(Model().model_json_schema())
# Gives
# {
#     'properties': {
#         'my_field': {
#             'custom_field': True,
#             'default': 'foo',
#             'other_field': 'other_field',
#             'title': 'My Field',
#             'type': 'string'
#         },
#         'my_field_with_overwrite': {
#             'custom_field': False,
#             'default': 'bar',
#             'title': 'My Field With Overwrite',
#             'type': 'string'
#         },
#         'my_field_with_other_field': {
#             'default': 'bar',
#             'new_field': 'my_field',
#             'title': 'My Field With Other Field',
#             'type': 'string'
#         }
#     },
#     'title': 'Model',
#     'type': 'object'
# }

Python, Pydantic & OS Version
pydantic version: 2.3.0

pydantic-core version: 2.6.3

pydantic-core build: profile=release pgo=false

python version: 3.11.3 (main, Jun  9 2023, 09:44:01) [Clang 14.0.0 (clang-14<IP_ADDRESS>)]

platform: macOS-12.6.6-x86_64-i386-64bit

Hi @Phoetaim,
Thanks for reporting this. Just for clarity's sake, could you include an example of what you'd expect the result of model_json_schema() to be compared to what is currently produced?
Thanks for the detailed explanation thus far.

Hi @sydney-runkle I would expect this json:
{
    'properties': {
        'my_field': {
            'custom_field': True,
            'default': 'foo',
            'other_field': 'other_field',
            'title': 'My Field',
            'type': 'string'
        },
        'my_field_with_overwrite': {
            'custom_field': False,
            'default': 'bar',
            'other_field': 'other_field',
            'title': 'My Field With Overwrite',
            'type': 'string'
        },
        'my_field_with_other_field': {
            'custom_field': True,
            'default': 'bar',
            'new_field': 'my_field',
            'other_field': 'other_field',
            'title': 'My Field With Other Field',
            'type': 'string'
        }
    },
    'title': 'Model',
    'type': 'object'
}

Thanks for the quick response

Hi @Phoetaim,
Thanks for the example + comments. I have some time later today to check this out in detail and will get back to you 😄.

Hi @Phoetaim,
After looking into this a bit further, I'm a bit concerned about the breaking nature of this change. Theoretically, there could be some users counting on the fact that specifying json_schema_extra later on in a field completely overrides that specified in earlier defined fields. I don't think it makes sense to introduce an additional flag to allow your requested behavior, but rather we should just have one policy for the behavior of how multiple json_schema_extra specifications behave.
Notably, there's not a super clear way to enforce an inclusion / overriding pattern with json_schema_extra specifications that are Callable types. Whereas with dict type specifications we can easily merge, there's not such an obvious solution with Callable types.
I think it might make sense to implement this change if there's a larger volume of community support, but I think for now it might make sense to stick with the current behavior. I think that you can still achieve the result that you want, you'll just have to specify a few extra fields at the final Field level.
Let me know what you think about all of this - I'm excited to get your feedback.

Hi @sydney-runkle,
Oh indeed, I didn't think far enough, it seemed the natural way to do it but it might not be for everyone. I didn't know about Callable type json_schema_extra, I'll look into it later.
I hope it will receive enough support then :).
Have a good day!

Hi @Phoetaim,
No worries - I didn't consider all of the permutations at first either. Sounds great. Excited to hear more about what you think.
We can leave this issue open in the meantime to see what kind of support / demand there is for it 😄.
Hope you have a good day as well.


multi language support

Missing functionality
The whole web page is English, and cannot display other language natively.
Proposed feature
Could we have a multi-language support profiling? The community can support the ability to add more languages
Alternatives considered
No response
Additional context
No response

You can try this, add a .ttf file to replace the default path :
def _plot_word_cloud(
series: Union[pd.Series, List[pd.Series]],
figsize: tuple = (6, 4),
) -> plt.Figure:
if not isinstance(series, list):
series = [series]
plot = plt.figure(figsize=figsize)
for i, series_data in enumerate(series):
word_dict = series_data.to_dict()
wordcloud = WordCloud(
background_color="white", random_state=123, width=300, height=200, scale=2
,font_path='/home/STFangSong.ttf'
).generate_from_frequencies(word_dict)
    ax = plot.add_subplot(1, len(series), i + 1)
    ax.imshow(wordcloud)
    ax.axis("off")

return plot


Problems configuring SGDK on Visual Studio 2005 Professional

I was followed the instructions of this video for use SGDK on Visual Studio:
https://www.youtube.com/watch?v=Kx8__lr4Ao4
But, on that video, was used Visual Studio 2017, and I use Visual Studio 2005 Professional (Because my computer is older and I cannot install Visual Studio modern versions on it).
On that video, for use the Sega Mega Drive SGDK codes, it add the SGDK\inc root to "include directory" on "VC++" section on the project properties.
But, on Visual Studio 2005 Professional, I not found that feature that is present on Visual Studio 2017.
Where is that feature on Visual Studio 2005?
Or, How I can configure and use SGDK on Visual Studio 2005?

Definitely not a bug, just require a proper tutorial :)

Definitely not a bug, just require a proper tutorial :)

I was afraid you'd say that, haha!
So the game compiles and runs fine, it just stopped compiling straight from VSC, what could've led to that, because I'm very curious!
Also, Stephen. I'm an indie dev and would love to be able to make a future game on the Mega Drive alongside modern systems for my future projects, and using the MEGA CD in some capacity would be a dream, so is there any chance of an expansion to the SGDK library to be compatible with the MEGA CD in the future?
With Terraonion's Mega SD and Krikzz EverDrive, development and testing on actual hardware is now a genuine reality, so I'm curious to know what you're thinking there.
Finally, what is realistically, the largest size a Mega Drive rom file can be? With EverDrives, we're no longer confined to 4MBs, so what would the scientific limit be? I'm asking in case the MEGA CD option is out, we could just take the SNES EverDrive SD card extra capacity approach if that's possible.
Sorry for all the questions, but its been a very exciting week for me working with this thing and a life dream is now a genuine reality thanks to you!


[FLINK-22573][datastream] Fix AsyncIO calls timeout on completed element. [1.12]

Unchanged backport of #15836 (well squashed into 1 commit).

What is the purpose of the change
As long as the mailbox is blocked, timers are not cancelled, such that a completed element might still get a timeout.
Brief change log
The fix is to check the completed flag when the timer triggers.
Verifying this change
Added AsyncWaitOperatorTest#testTimeoutAfterComplete.
Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): (yes / no)
The public API, i.e., is any changed class annotated with @Public(Evolving): (yes / no)
The serializers: (yes / no / don't know)
The runtime per-record code paths (performance sensitive): (yes / no / don't know)
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Kubernetes/Yarn/Mesos, ZooKeeper: (yes / no / don't know)
The S3 file system connector: (yes / no / don't know)

Documentation

Does this pull request introduce a new feature? (yes / no)
If yes, how is the feature documented? (not applicable / docs / JavaDocs / not documented)

Failure unrelated.


[8.x] Is dirty giving false negative when null value changed to false

I stumbled into this today and created a test to make sure I was not going crazy...
So basically it looks like Laravel does a loose comparison when checking if objects are equal each other, meaning if you change a json object from having a null value to a false value, it will not save it, because isDirty is not detecting any changes.
If anyone else have this issue, then the new AsArrayObject cast seems to fix it as it uses strict comparison.

Is this issue also present in 6.x?

@driesvints looking at the code, yes it should be an issue in 6.x also.

@olivernybroe can you send this to 6.x instead? So we can fix it there and merge upstream.

@driesvints sure thing 👍 https://github.com/laravel/framework/pull/36326


badly packaged library

ERROR in ./node_modules/ng2-breadcrumb/ng2-breadcrumb.ts
Module build failed: Error: /home/mengqingyun/iserver/src/frontend/node_modules/ng2-breadcrumb/ng2-breadcrumb.ts is missing from the TypeScript compilats' or 'include' property.
The missing file seems to be part of a third party library. TS files in published libraries are often a sign of a badly packaged library. Please open an issuem to package the library using the Angular Package Format (https://goo.gl/jB3GVv).

Have the same problem...

same problem..any updates ?!!


Plugin fetch all Specs on the local machine

Hi, if i use the remote cocoa pods repo, the plugin will save all cocoa pods specs on my local machine (~756 MB). In my podfile i define only one dependency (Alamofire). I expected only the spec for Alamofire on my local machine.  Artifactory runs on a remote system.
I follow your instructions in the Screencast Sample and doesn`t finde the mistake.

Cocoapods relies on the existence of the entire index locally, it does not query the remote for specific packages like soem other package managers do - the same behavior can be observed if you just cocoapods init or try to resolve dependencies when there is nothing under .cocoapods/repos - what the client does by default is a git clone of the entire specs repo (https://github.com/CocoaPods/Specs), resulting in the same amount of files (and size)


.swift-version file missing

Environment: XLPagerTabStrip 8.1.0, Xcode 9.4.1 and iOS 10.3.1 you are using.

http://blog.cocoapods.org/CocoaPods-1.1.0/#xcode-8-and-ios-10

Thanks for reporting it, next release will contain it.


Add Option To Rerun Failures (Only and or First)

We may not want to rerun passing tests when trying to fix failures. We could introduce an option to rerun the failed tests from our previous run. We could have this option specifically run ONLY the failed tests (even ignoring passing methods within the test script).
With this enabled, users could also tell their CI platforms to rerun their failed tests for an automated defect localization.

I'll likely choose to rerun only the failures. The rerunning first may cause unnecessary issues and just ends up slowing down the issue anyway.

Closing. See #132.

Opening apparently unresolved issue (might be resolved)

Seems this issue was already resolved in #132
see: https://github.com/AlexDarigan/WAT/issues/132#issuecomment-619452345


Update the OkHttp version this depends on

Hey, I was having an issue with the IDE I supposed while using this library which I think would be solved by bumping which OkHttp version this depends on.
Since this pulls an earlier version of OkHttp, it somehow makes it so that when I am calling methods like .body on a Response object, the IDE looks at the older version and returns a String instead of a ResponseBody. It would make our lives inside the IDE if this was updated. I think this may be happening since we're transitively depending on OkHttp from some other dependency, so we're using 4.x.x on runtime and the IDE sees the 3.x.x version. Not sure though.
It seems like importing this library or copying the method it provides is still the only way to use okhttp with coroutine support, which is why we're still using it. Is this something you use yourself as well? Maybe there are some alternatives I'm missing out on which is why you haven't felt the need to update what this library depends on? Or have you not encountered this IDE problem that I am referring to?

I found this actually, describes what I was experiencing as well https://stackoverflow.com/questions/59158745/okhttp4-cannot-access-body-it-is-package-private-in-response I hope it helps you with some context


Modified PSService script to have some additional functionality

Change 1:  Added $User and $Password parameters.  Modified the installation routine to utilize the user Id and password provided when setting the RunAs account for the service.
Change 2: Sub folder called "functions"
When using this framework in my own project, I found that I did not want to rely solely on this particular script for all of the logic of the application.  So I created a sub-folder called "functions", and put my functions files in that folder.  Modified the installation routine to scan this folder for any .ps1 files, and dot source them all.
Modified the uninstall routine to ensure that the directory is removed recursively, to include the functions directory.

Hi Nathan,
Thanks for submitting this, but there are several issues that need to be addressed before I can merge it...

I like the user and password parameters you've added during the setup stage. Several people have already asked for something like this. But I'm afraid that you've not finished the implementation: The $RunasAccount variable is commented out. And the things you mentioned in https://github.com/JFLarvoire/SysToolsLib/issues/7#issuecomment-347910930 are not there either. Yet I'm pretty sure that implementing something like that is necessary for the new paramenters to be usable, and the service to work fine, whether you configure a specific user or not. (Actually, for doing that, I prefer the variables proposed by @furt1v3ly in https://github.com/JFLarvoire/SysToolsLib/issues/7#issuecomment-294571471.) Anyway, I think there's still much work to do to make this ability to run as another user fully functional, and test it in all cases.
On the other hand, I don't like the copying of other "function" files. This is very specific to your personal needs. Other people reusing the script may put it anywhere before running -Setup. And every file in subdirectories there would be copied along with the service. This is very dangerous. Please remove that for now.
Finally, one minor annoyance: You're indenting blocks with tabs, instead of the 2 spaces used in the rest of the script. This breaks the code blocks alignment in my editor. I would have changed it myself, but I'd appreciate if you could do it beforehand if you submit an updated versions.
Jean-François

Okay, apologize for being new to GitHub pull requests.  I tried to sync my code up with the recent accepted merge requests, and realized it created more changes than I anticipated.
I reverted the changes and re-did them, making just the proposed changes.
I believe I have addressed your concerns - I've renamed the User and Password parameters to ServiceUser and ServicePassword.
I've completed the isSystem check. I didn't see where this was being used when I originally updated the code, so I wasn't sure it was neccessary - but now I see where it is being used.
Additionally, I have changed any tabs that I added to spaces.

I can't make it work with any user account except the LocalSystem account.
Did you test it with other accounts?
And if so, what should I do to make it work?

Answering part of my own question:
The user must have the "Log on as a service" right. I'll add that information to the help screen.

I've played a lot with your code over the week end.
The first issue I had was the failure to start the service using my own account. First there was that problem about giving it the correct rights, as mentioned in my previous comment. This was purely a documentation problem, and I've added explanations in the help screen.
But there still was a problem with the way we had been detecting if the Start and Stop commands came from the user or from the Service Control Manager. The initial method I had used (relying on the user account detection) had become unusable, once it was possible to run the service on the same account as the user. None of the alternatives suggested by you, or earlier by others, could address that issue
It took me a surprisingly long time to find the obvious solution: Use different options in the PSService.exe stub, when passing on the Start and Stop commands. I called them -SCMStart and -SCMStop. That change is simple, and gives a reliable code that works in all account combinations cases.
A second change I did was to rename your -ServiceUser and -ServicePassword arguments to the more standard names -UserName and -Password. I hope you'll forgive me for that. This required renaming the existing variable $userName to $currentUserName. Maybe it is because of that variable that you couldn't call the argument -UserName to begin with.
Then, as entering a password on the command line is very convenient but considered unsafe, I added code to securely prompt for the password if the -UserName is specified, but not the -Password.
Finally, as an alternative to entering the -UserName, I added the ability to pass the whole credential directly with a standard -Credential argument.
That last change caused a minor problem with the old PowerShell version 2 on Windows XP: If you use -Setup with no -UserName or -Credential, PS complains about not knowing which parameter set to use. This is obviously a bug in PSv2, as the command completion does work. So I've documented that in the help, along with the obvious workaround, which is to use -Setup -UserName LocalSystem.
Anyway, I've pushed the updated script to the GitHub repository.
It's working fine with the default LocalSystem account, and with my own account.
There still is a problem with the LocalService and NetworkService accounts. The SCM seems to be perfectly happy about them, and reports success in all cases. Yet, even though the PSService.exe stub starts, the PSService.ps1 -Service instance does not. I don't know why. I've written in the help screen that theses accounts do not have enough privileges to run PowerShell, but this is just a speculation. I'm still hoping that somebody will find a solution for these two accounts, which are precisely the recommended ones to use for running services!


Returning tensor instead of dict for state_dict causes failure

🐛 Describe the bug
class TensorStateDataset(torch.utils.data.IterableDataset, Stateful, Iterator):
    def __init__(self, length):
        self.length = length
        self.i = 0

    def __iter__(self):
        return self

    def __next__(self):
        if self.i >= self.length:
            raise StopIteration
        self.i += 1
        return self.i

    def state_dict(self):
        return torch.rand(2, 2)

    def load_state_dict(self, state_dict):
		pass

class TestSimple(TestCase):
    def test(self):
        dataset = TensorStateDataset(100)
        dl = StatefulDataLoader(
            dataset=dataset,
            num_workers=1,
        )
        it = iter(dl)
        for _ in range(30):
            next(it)
        self.assertTrue(False)

Running this, I hit an error as follows:

self = <torch._utils.ExceptionWrapper object at 0x7f921c5fde10>

    def reraise(self):
        r"""Reraises the wrapped exception in the current thread"""
        # Format a message such as: "Caught ValueError in DataLoader worker
        # process 2. Original Traceback:", followed by the traceback.
        msg = f"Caught {self.exc_type.__name__} {self.where}.\nOriginal {self.exc_msg}"
        if self.exc_type == KeyError:
            # KeyError calls repr() on its argument (usually a dict key). This
            # makes stack traces unreadable. It will not be changed in Python
            # (https://bugs.python.org/issue2651), so we work around it.
            msg = KeyErrorMessage(msg)
        elif getattr(self.exc_type, "message", None):
            # Some exceptions have first argument as non-str but explicitly
            # have message field
            raise self.exc_type(message=msg)
        try:
            exception = self.exc_type(msg)
        except TypeError:
            # If the exception takes multiple arguments, don't try to
            # instantiate since we don't know how to
            raise RuntimeError(msg) from None
>       raise exception
E       RuntimeError: Caught RuntimeError in DataLoader worker process 0.
E       Original Traceback (most recent call last):
E         File "/home/gokulg/torchdata/data/torchdata/stateful_dataloader/worker.py", line 233, in _worker_loop
E           delta_state_dict = incremental_worker_state.generate_delta(state_dict)
E         File "/home/gokulg/torchdata/data/torchdata/stateful_dataloader/incremental_state.py", line 142, in generate_delta
E           if iter_state := fetcher_state.get(_DATASET_ITER_STATE, None):
E       RuntimeError: Boolean value of Tensor with more than one value is ambiguous
E
E
E       To execute this test, run the following from the base repo dir:
E            python test/stateful_dataloader/test_state_dict.py -k test2
E
E       This message can be suppressed by setting PYTORCH_PRINT_REPRO_ON_FAILURE=0

../../.conda/envs/basetorch/lib/python3.10/site-packages/torch/_utils.py:722: RuntimeError

If an integer is returned or say dict (with value as tensor) is returned, there is no error.
Versions
Latest git commit 82918dd

@gokulavasan we should probably try and fix this before release-branch cut, would this be straightforward?

Fix is in this PR - https://github.com/pytorch/data/pull/1272


Can not embed a Youtube player anymore

In the old forum a youtube address in the post embedded a small player.
That does not work anymore and the old posts contain only plain text now.
Example here: http://forum.arduino.cc/index.php?topic=238567.30

Guys, but what security threats is youtube making to your forum ? I don;t understand where is the security issue for embeding youitube videos? I'm not saying to allow embeded flash files, but just youtube video.
Can anyone tell us more details, why this normal/usual/usefully function is disabled ?
Thank you.

Guys, but what security threats is YouTube making to your forum ?

As long as the embed code is restricted to trusted websites like youtube (which it was, as SMF converted a plain text URL into the embed).
The only other reason for a block is because the content of the embedded videos is not controlled.

For the content of the videos, I think the forum has a few moderators that can ban or delete some post if the content is inappropriate, but this is not a reason to make visualization of all users harder.
I really think it's time to rethink why the bbcode was removed, this is one of the few forums that doesn embed youtube videos automatically.

It's working again after the switch to using the Discourse software for the Arduino forum this week.


Usage of targetObject in example app

I noticed that your example app at http://offirgolan.github.io/ember-cp-validations/ uses the private targetObject API to read didValidate off the targetObject (which happens to be the controller in this case, but isn't guaranteed to be).
This is going to be deprecated in 2.9 (related issue: https://github.com/emberjs/ember.js/issues/14168) and I think, it's fair to say, is considered a really bad practice. Components should be passed the data they need to do their job, not read data magically from the context they're evaluated inside of. Metaphorically, in JS, this would be akin to doing something like arguments.caller.someLocalVariable.

@ebryn I totally agree with you here. Once I have some spare time, Ill remove the targetObject dependency on the validated-input component.


Layout/IndentationConsistency adds spaces to multi-line strings

Expected behavior
Ignore multi-line strings
Steps to reproduce the problem
$ cat test.rb
  x = 'a'
y = 'b
c'

$ rubocop --auto-correct --only Layout/IndentationConsistency ./test.rb
Inspecting 1 file
C

Offenses:

test.rb:2:1: C: [Corrected] Layout/IndentationConsistency: Inconsistent indentation detected.
y = 'b ...
^^^^^^

1 file inspected, 1 offense detected, 1 offense corrected

$ cat test.rb
  x = 'a'
  y = 'b
  c'

"b\nc" == "b\n  c"
 => false

RuboCop version
0.74.0 (using Parser <IP_ADDRESS>, running on ruby 2.3.3 x86_64-darwin18)

This has the same root cause as #7177.

And here is some work on this from 3 months ago: https://github.com/rubocop-hq/rubocop/compare/master...buehmann:spec/alignment_corrector
I haven't gotten around to continuing on that. I still don't like the way I am testing the AlignmentCorrector.


Backport diagnostic reporting to Lean 3.

Does so by introducing infoview components (which in the future
possibly is a way to customize what's in the infoview display).
Closes: #41
Refs: #17
@gebner if you happen to have a spare moment I wouldn't mind a sanity check on whether this crude splitting of Lean 3 goal state is trivially incorrect in some way. It seems to work for the files / state I checked manually. If not no worries, can always revert later if it goes sideways.

Thanks for cleaning up infoview.lua. This is interesting, I'm curious do you think it would be possible to do this by adding a new type of provider to to lean-language-server that does the parsing rather than doing that here?

Thanks for cleaning up infoview.lua. This is interesting, I'm curious do you think it would be possible to do this by adding a new type of provider to lean-language-server that does the parsing rather than doing that here?
I think it'd be possible but it still wouldn't be 100% seamless, there's still some heuristic'ing because I think the upstream Lean 3 server message doesn't separate things out. And combined with knowing the lean-language-server repo isn't super active I thought it was easier to just shove here, though now Gabriel's merged the outstanding PR I had there.

Dah, well done @rish987 -- I added is_empty precisely because I thought it'd fix it (and changed where you changed), but forgot the table.concat... le facepalm.
Appreciated!

Fixed infoview/lsp_spec.lua LSP wait condition. LGTM at this point!


Add log statements to project and configure log level

As a project developer, I want an application wide log level configured, and I want each existing log statement to have an appropriate level applied, so that application-wide log output becomes configurable

Mentioned in #231


I want make the dialog backgroundcolor to transparent, How do that?

You cannot

I need to make an ad pop-up window, and the position where the close button is displayed always has a white background. So I asked this question, is there any other solution?

please。。。


finalized the quiz

added some css files and restart quiz button
all tasks except "A user can refresh the page and still have his/her given answers available" are completed

Make sure you delete commented out code before checking in to keep the code clean. Otherwise looks good!


How can I get SMPL2d keypoints

hello, thanks for your excting works. I just want to know how can I get this format keypoints to inference in the wild videos.

Hi, thank you for your interests and good question! Different from benchmark testing where ground truth 2D keypoints are typically used, in-the-wild videos can only have detected 2D keypoints. However, the format between 2D detectors and SMPL can differ significantly. Possible workarounds are (i) Train a detector directly in the SMPL format; or (ii) Learn a mapping from OpenPose to the SMPL format. More details and discussions on this can be found in Section G of our supplementary materials.

As discussed in our limitations and Sec. G of supp, KITRO can be sensitive to the 2D noise. Enhancing the robustness to 2D noise and providing full support for in-the-wild 2D detection are ongoing areas of future work.


[#165677471] Enable S3 for everyone!

What
Enables the aws-s3-bucket service's default plan for everyone on the GOV.UK PaaS! 🎉
How to review
Code review. This change worked in my miki dev environment.
Who can review
Not @46bit

🙇 @henrytk 🙇


Use initial commit if no previous tag is available

if only a single tag is available resolve the initial commit as fallback. this is meant to support initial release scenarios

FIX https://github.com/mikepenz/release-changelog-builder-action/issues/276
Thanks @ccremer for debugging and testing

Thanks a lot for your effort!

@ccremer it's now released as v2.1.0 :) thanks again


Add shipping breakdown

This PR will add a new shipping_breakdown to orders and carts. This will allow developers to add additional shipping costs and provide a more complete breakdown similar to discounts and taxes.
The idea being is that you may have outside factors which affect the cost of shipping outside of what's provided by the manifest, such as split delivery or additional charges for awkward items, this should facilitate that.

@alecritson can you fix the conflicts?


MINIFICPP-449 Add cURL external project build with static linking

Thank you for submitting a contribution to Apache NiFi - MiNiFi C++.
In order to streamline the review of the contribution we ask you
to ensure the following steps have been taken:
For all changes:

[x] Is there a JIRA ticket associated with this PR? Is it referenced
in the commit message?

[x] Does your PR title start with MINIFI-XXXX where XXXX is the JIRA number you are trying to resolve? Pay particular attention to the hyphen "-" character.

[x] Has your PR been rebased against the latest commit within the target branch (typically master)?

[x] Is your initial contribution a single, squashed commit?

For code changes:

[x] If adding new dependencies to the code, are these dependencies licensed in a way that is compatible for inclusion under ASF 2.0?
[x] If applicable, have you updated the LICENSE file?
[x] If applicable, have you updated the NOTICE file?

For documentation related changes:

[x] Have you ensured that format looks appropriate for the output in which it is rendered?

Note:
Please ensure that once the PR is submitted, you check travis-ci for build issues and submit an update to your PR as soon as possible.

Moved to curl extension and fixed external static lib cmake dependencies by using PROPERTIES INTERFACE_LINK_LIBRARIES.


When creating Organizations, create Contacts as extradata

This includes adding the newly enhanced schema to the mapping-file-creator.

Proposed sequence of events:

Mapping: the OrganizationMapper fetches the Contacts schema and integrate it into
Mapping: during mapping transformation, the contact (objects are built similar to other array objects)
Post-processing: take the objects in the Contacts array and save them as extra data
Post-processing: removes everything except the UUID string(s) from the Contacts array

@branchedelac I like this strategy!

Another option:

Mapping: the OrganizationMapper treats the contacts as a an array of strings (which it is)
Mapping: during mapping transformation, a UUID is added to the Contacts array for each mapped contact
Post-processing: for each uuid, a contacts object bearing this UUID is created based on the contacts schema (a contacts mapper?) and printed as extradata.

Taking these independent objects out of the depths of organization mapping might make the code a bit easier to follow.
The downside is the contacts are not -- out of the box, but it could be done -- included in the mapping file. On the other hand, the Orgs mapping file creation UX is already so overwhelming I am a bit concerned about adding more stuff.

@branchedelac this seems more on-par with how Notes are done, except for the mapping part I guess.

So this (also) seems like a good approach.

I think Contacts on Organizations are very much like Notes (except one of them that is Notes), with a few exceptions:

They link is from the organization to the sub-object, not the other way around. Comparable to Instructors, I think.
They contain many of the same sub-objects as the Organization itself, like "addresses".
They are not (currently) shared with any other apps.

How is mapping handled for Notes?

Are the notes still extra data?

Are the notes also extra data?

Yes. See the NotesMapper

How is mapping handled for Notes?
Follow the calls to NotesMapper.map_notes()

I was wondering if you could give me a summary to speed things up.

Broke this out as a bug issue (not a showstopper): https://github.com/FOLIO-FSE/folio_migration_tools/issues/464

Broke this out: https://github.com/FOLIO-FSE/folio_migration_tools/issues/468


Update torch dependency to >= 1.1.0

Current master fails with torch 1.0.1. Fix this by depending on torch >= 1.1.0.

That's great, thank you so much :)


Fixed the more button in my Favourites page

Fixes #1573
Describe the changes you have made in this PR -
Changed the URL for more button in the favourites page
Screenshots of the changes (If any) -
Note: Please check Allow edits from maintainers. if you would like us to assist in the PR.

Coverage remained the same at 85.862% when pulling 19f48e4cef79e8c462e1cef7eb72449685c698a2 on nileshdas:favouritesPage into aee080ea19615612ca8fc98a5c8d8e0deaab5d07 on CircuitVerse:master.

@aayush-05

No need to change path to url

@aayush-05 Changed it.

Thanks for your contribution! 😄


How to run it on juputer notebook?

should i directly run the emotin recognition file

This is not made to run in a jupyter notebook but directly from the terminal. Follow the readme instructions.


aux window: WYSIWYG HTML Editor not possible to open as Floating window

Does this issue occur when all extensions are disabled?: Yes

VS Code Version: 1.86.0-insider
OS Version: Windows 11

Steps to Reproduce:

Install WYSIWYG HTML Editor Extension https://marketplace.visualstudio.com/items?itemName=node-projects.vscode-designer-addon
Open a HTML File in it, then try to drag it out, does not work

Probably related to #195715
Extension author should know, and meanwhile may be able to update the extension so it tells VS Code not to offer the option.

Thanks for creating this issue! We figured it's covering the same as another one we already have. Thus, we closed this one as a duplicate. You can search for similar existing issues. See also our issue reporting guidelines.
Happy Coding!


Fix model validation json names

Codecov Report

Merging #5 into master will not change coverage.
The diff coverage is 100%.

@@           Coverage Diff           @@
##           master       #5   +/-   ##
=======================================
  Coverage   99.56%   99.56%
=======================================
  Files          12       12
  Lines        1381     1381
=======================================
  Hits         1375     1375
  Misses          4        4
  Partials        2        2

Impacted Files
Coverage Δ

modelvalidator.go
100% <100%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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


Fix mismatching operand size when lifting movd/movq/vmovd/vmovq instructions

Moved from https://github.com/Vector35/arch-x86/pull/38.

Copied this over from the original PR:

Prior to this fix movd eax, xmm0 was lifted to eax = xmm0 instead of eax = xmm0.d.
In SSA form, this resulted in a semantically invalid zero-extend instruction that made no sense: rax#1 = zx.q(zmm0.xmm0), which is wrong since xmm0 is bigger than a quad-word.
Fixed by specifying the explicit size of the source operand:
https://github.com/Vector35/arch-x86/blob/a87d5df489eed969dec7f36235ebd81c3ff87acb/il.cpp#L2104-L2115

I am closing this PR, as there were a number of problems with it. I will link to the corrected commit here once it's pushed (I'm still testing it).


smtp password creation and domain

Hi, some suggestions:

Fixing Postgres version in docker-compose to avoid a mess when you have previous versions installed.
For some reason redirection for organizations is not woking with a port different than 80 in localhost.
I've found that SECRET_KEY_BASE is needed for the encryption of passwords. Particularly SMTP password in organization creation.
Also without some value in SMTP_DOMAIN, SMTP won't work.

And I had to comment out RAILS_MASTER_KEY to fix #5

Hello !
Thank you very much for your contribution !

I'm ok for fixing postgres version, it's a good practise !
I don't understand why are you changing the port on the host side, it should work whatever port is chosen, and it's best to use unprivileged ports. You had some trouble with it ?
Thanks a lot ! I'm always confused with those two values (RAILS_MASTER_KEY and SECRET_KEY_BASE), thanks for you correction !
Thanks for your correction for SMTP !

Can you explain more the second point ? Thanks a lot !

Happy to contribute, this repo has helped me a lot, so.
On the second point. I know it should work with any port but for some reason after creating an organization and setting the host, localhost:3000, Decidim will always redirect to /system thus no letting me go to homepage or /admin. I've had trouble before with that host field in new organization form. For example it won't work if you add the scheme (http, https). Maybe this is related somehow.
Can you reproduce this?

Hi, new commits fixes #4 and brings back 3000 port to compose.


Fix stable baseline 3 example and add a requirement for older version of numpy

Set up the wrong upstream. It was meant for a fork.


Port 80 is already in use error.

Bug Report or Feature Request (mark with an x)
- [x] bug report -> please search issues before submitting
- [ ] feature request

Versions.

    _                      _                 ____ _     ___
   / \   _ __   __ _ _   _| | __ _ _ __     / ___| |   |_ _|
  / △ \ | '_ \ / _` | | | | |/ _` | '__|   | |   | |    | |
 / ___ \| | | | (_| | |_| | | (_| | |      | |___| |___ | |
/_/   \_\_| |_|\__, |\__,_|_|\__,_|_|       \____|_____|___|
               |___/
@angular/cli: 1.3.0
node: 6.11.2
os: darwin x64
@angular/common: 4.3.3
@angular/compiler: 4.3.3
@angular/core: 4.3.3
@angular/forms: 4.3.3
@angular/http: 4.3.3
@angular/platform-browser: 4.3.3
@angular/platform-browser-dynamic: 4.3.3
@angular/router: 4.3.3
@angular/cli: 1.3.0
@angular/compiler-cli: 4.3.3

Repro steps.

"serve": {
  "host": "<IP_ADDRESS>",
  "port": 80
}

$ ng serve --disable-host-check --e alpha-selleradmin --proxy-config ./proxy/selleradmin.conf.js

The log given by the failure.

Port 80 is already in use. Use '--port' to specify a different port.

Desired functionality.

Server work on 80 port.
Mention any other details that might be useful.

throw Error on angular/cli 's check-port.js module.
I have no process that using 80 port.
$ sudo lsof -t -i tcp:80 -s tcp:listen
// nothing
$ sudo lsof -i ':80'
// nothing
$ sudo lsof -n -i4TCP:$PORT | grep LISTEN
// nothing
$ lsof -n -iTCP:$PORT | grep LISTEN
// nothing
$ lsof -n -i:$PORT | grep LISTEN
// nothing

ports below 1048 needs to root privileges. issue close.

ports below 1048 needs to root privileges. issue close.

Then the error message should be changed to something meaningful please.


Dockerfile*: bump Go to 1.7.5

- What I did
I've bumped the Go version to 1.7.5.
- How I did it
I've edited all the relevant files.
- How to verify it
This can be done by taking a look at the changelog.
- Description for the changelog
Dockerfile*: bump Go to 1.7.5

LGTM

https://jenkins.dockerproject.org/job/Docker-PRs-arm/358/console seems to have run into errors.

Looked at some failed builds for arm-master. Found TestSwarmPublishDuplicatePorts also failing in https://jenkins.dockerproject.org/job/Docker-PRs-arm/359/console . Didn't see TestLogsFollowSlowStdoutConsumer but we've had trouble with this test before in #29030

@tonistiigi is the issue with master or with this PR ? If it is only with master I think we should merge this.

@tiborvass As I wrote I found one case where one of the failed tests failed in master and the other test failed in another PR(although that pr also updates go). I would say it is likely that the problem is not in this PR but hard to be 100% certain.


How return error when a key does not exist?

Suppose that I have the following test.yaml file.
name: "MyName !!"
subvalue:
    how-much: 1.1
    things:
        - first
        - second
        - third
    other-things: [a, b, c]
    maintainer: "Valentin Lab"
    description: |
        Multiline description:
        Line 1
        Line 2

The following command would return an exit status of 0 event the key subvalue.maintainer1 does not exist. Is there a way to have shyaml return an non-zero status in such a case? Thanks.
~$ cat test.yaml | shyaml get-value subvalue.maintainer1
~$ echo $?
0

Can you test branch PR7 ? this was already tackled in this branch, and it's waiting a little bit more tests to be pushed as master.
The (bad) rationale behind the previous behavior you are speaking about is that there's a third argument to shyaml which is the default value if we don't find the key. The default of this default value was the empty string. (did you follow ?)
In the PR7, if you don't specify a third argument as a default value, then an error will be casted and errorlevel will be 1.
So if you can tell me if PR7 solves your issue, and doesn't break your use cases of shyaml, I might push PR7 as the new version of shyaml.

Closing this as the fix was released since a while now.


port-forward fails if Pachd is in a namespace

port-forward can't find pachyderm if it's deployed in a namespace. Probably need to add a --namepsace flag to port-forward.
Had to run this to make it work:
kubectl --namespace=pachyderm port-forward kubectl --namespace=pachyderm get pods -l app=pachd -o json | jq '.items[0]tadata.name' -r 30650:650 &
There may be other places where we call kubectl commands under the hood and need that flag.

We may also want to add the namespace to the ~/.pachyderm/config.json file ... since we include details that tell pachctl how to connect to a cluster in there (right now the pachd address, user token).

Adding a namespace flag to port-forward is easy enough, (#2749). We could also add this to the config. However I think the best practice is actually to set this at the k8s level with:
$ kubectl config set-context dev --namespace=development \
  --cluster=<cluster> \
  --user=<user>

That way things like kubectl get all and kubectl logs will just work.

Just hit this as well. Need to decide on the best solution. Then I can document it.

We now have a --namespace flag on port-forward we should get that documented.


Added basic editor for overriden build script for GUI.

According #41 and #31 I've created simple editor (without syntax highlighter) for custom build script.
Functionality is really simple, if there any script is added, it creates folder and file for .flux-build in overrides. If there is no text, build script is removed.
Saving into file could be little bit buggy, it could require additional check.

that was fast :D

that was fast :D

Damn right. 😄 Awesome! Thanks

that was fast :D. the idea behind override folder was not bad right?

I think the idea is brilliant!

I would go for adding a checkbox something like "[ ] update build script by the thing below", so override folder gets not something written in when e.g. changing ref-whitelist

I was considering it too but I believe configuring repo for CI is not something what anybody does reckless.


[Request for external] page 12156 "Subcontracting Prices"

In page 12156 "Subcontracting Prices" (italian localization) could you change the scope of the following variables to protected?
protected var
VendNoFilter: Code[30];
ItemNoFilter: Code[30];
WorkCenterFilter: Code[30];
TaskCodeFilter: Code[30];
StartingDateFilter: Text[30];
Thank you

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.
Build ID: 47754.


bash: odtk: command not found

I followed the steps:
git clone https://github.com/nvidia/retinanet-examples
git checkout -b 19.10 origin/19.10
docker build -t odtk:latest retinanet-examples/
docker run --gpus all --rm --ipc=host -it odtk:latest

the container started successfully:
=============
== PyTorch ==
=============

NVIDIA Release 19.10 (build 8472689)
PyTorch Version 1.3.0a0+24ae9b5

Container image Copyright (c) 2019, NVIDIA CORPORATION.  All rights reserved.

Copyright (c) 2014-2019 Facebook Inc.
Copyright (c) 2011-2014 Idiap Research Institute (Ronan Collobert)
Copyright (c) 2012-2014 Deepmind Technologies    (Koray Kavukcuoglu)
Copyright (c) 2011-2012 NEC Laboratories America (Koray Kavukcuoglu)
Copyright (c) 2011-2013 NYU                      (Clement Farabet)
Copyright (c) 2006-2010 NEC Laboratories America (Ronan Collobert, Leon Bottou, Iain Melvin, Jason Weston)
Copyright (c) 2006      Idiap Research Institute (Samy Bengio)
Copyright (c) 2001-2004 Idiap Research Institute (Ronan Collobert, Samy Bengio, Johnny Mariethoz)
Copyright (c) 2015      Google Inc.
Copyright (c) 2015      Yangqing Jia
Copyright (c) 2013-2016 The Caffe contributors
All rights reserved.

Various files include modifications (c) NVIDIA CORPORATION.  All rights reserved.
NVIDIA modifications are covered by the license terms that apply to the underlying project or file.

NOTE: MOFED driver for multi-node communication was not detected.
      Multi-node communication performance may be reduced.

Then execute the command:
odtk train retinanet_rn50fpn.pth --backbone ResNet50FPN --images /dataset --annotations train_0303_3clz.csv
Then comes the error:
bash: odtk: command not found

You can't use instructions from master on branch 19.10
Use retinanet instead of odtk for 19.10


Fix 19

Fixes #19 , though there's no user-friendly solution for an empty directory and the interface is just left empty.  Also has knock on effect that the page is reset to the first one every time the user changes the rows_per_page.
(Apologies if squashed commits would be better, but github's web editing doesn't seem to allow it yet.  If it's vital I might eventually get around to a local clone and proper git client, but it might have to wait quite a while.  Feel free to squash them yourself if you want though!)

LGTM. Thanks!


Please add a method to place skulls in the world.

As per PM on spigot.
Please add some kind of method to allow us to place the skulls in the world without player interaction. Something like SkullCreator.setSkullBlock(Block block, ItemStack skull);

I think the way I'm going to set it is with... the default /setblock command 😅
It's really just a terrible solution, but so was using Bukkit.getUnsafe(), but it's the most version-compatible way to do it.

These don't work in other worlds...

Supported in 3.0.0


Add gtts-cli

edge-tts seem more advance than this, but who knows how long it will continue to work.  https://github.com/pndurette/gTTS
it doesn't support different voices, just different languages; don't know if worth it or possible; seems to produce mp3s
gtts-cli --all
  af: Afrikaans
  am: Amharic
  ar: Arabic
  bg: Bulgarian
  bn: Bengali
  bs: Bosnian
  ca: Catalan
  cs: Czech
  cy: Welsh
  da: Danish
  de: German
  el: Greek
  en: English
  es: Spanish
  et: Estonian
  eu: Basque
  fi: Finnish
  fr: French
  gl: Galician
  gu: Gujarati
  ha: Hausa
  hi: Hindi
  hr: Croatian
  hu: Hungarian
  id: Indonesian
  is: Icelandic
  it: Italian
  iw: Hebrew
  ja: Japanese
  jw: Javanese
  km: Khmer
  kn: Kannada
  ko: Korean
  la: Latin
  lt: Lithuanian
  lv: Latvian
  ml: Malayalam
  mr: Marathi
  ms: Malay
  my: Myanmar (Burmese)
  ne: Nepali
  nl: Dutch
  no: Norwegian
  pa: Punjabi
  pl: Polish
  pt: Portuguese
  ro: Romanian
  ru: Russian
  si: Sinhala
  sk: Slovak
  sq: Albanian
  sr: Serbian
  su: Sundanese
  sv: Swedish
  sw: Swahili
  ta: Tamil
  te: Telugu
  th: Thai
  tl: Filipino
  tr: Turkish
  uk: Ukrainian
  ur: Urdu
  vi: Vietnamese
  zh-CN: Chinese (Simplified)
  zh-TW: Chinese (Mandarin/Taiwan)
  zh: Chinese (Mandarin)

Examples:
echo "Victor jagt zwölf Boxkämpfer quer über den großen Sylter Deich"  | gtts-cli -l de -f - | mpv -
echo "The five boxing wizards jump quickly."  | gtts-cli -l en -f - | mpv -
echo "Portez ce vieux whisky au juge blond qui fume"  | gtts-cli -l fr -f - | mpv -

Interesting, I was was not familiar with this. I probably won't have spare time to add this for a while but I'll keep this open to remind me to do it one of these days.


redis: Add link, target completion & target search

Part of #160, #161, #165, and #166. Implements the RedisClient operations necessary to support these search, autocomplete, and target collision features.
Uses the lexicographic completion approach described in: http://oldblog.antirez.com/post/autocomplete-with-redis.html
Note that we'll need a migration script to create the new search indexes for existing installations.

Coverage increased (+0.02%) to 98.083% when pulling dc3ccd7bc8abd7ea9ebf0bddc880c9854c3065cb on autocomplete-and-target-search into 1b587f1208ae308599c7f94c0d9d24a039c7a5c0 on master.

BTW, this PR only adds items to the search corpora; I've filed #172 to follow up with removing them when links are deleted.

The Code Climate coverage check is failing, but I think it's faulty. The "Details" link shows no details, and Coveralls shows a small increase in coverage. Ignore the failure.

I've decided to eliminate the target URL autocompletion, as I think that would get messy in a hurry and isn't as useful as a regular search.
I've also:

restricted autocompletion to a three-character minimum
updated createLink to resolve to the Error resulting from a search index insertion failure, as the link will still have been successfully created
performed an internal rename of the external redis npm client to identify it as client.impl and clientImpl

I'm going to go ahead and merge to get things moving along, but feel free to comment, ask questions, or open a new issue if there are any concerns.


[3.2] Add mouse event pass-through support for window.

Same as #39944 for 3.2

Thanks!

Why is the Window not drawn in Windows? Is it not possible to do it that way?


The workflow activity logs exist the  syntax/Type error

Describe the bug
I found two kinds of syntax/Type error in logs.

SyntaxError: Unexpected token U in JSON at position 0(e.g: this request )
TypeError: Cannot read property ‘slug’ of undefined(e.g:activity)

For the second Type error, as i checked the worker.flow source code, if it can’t get the team value, then it will occur to can’t read property ‘slug’ of undefined error, maybe that's due to team.slug is no value?
const team = await RetrieveTeamByOrgName(url, token, organizationName, teamName);

What project does this issue relate to?
Boomerang Flow
To Reproduce
This is the "Invite Member to Team via GitHub userName" Http request and response result from request
request {
method: 'GET',
baseUrl: 'https://github.ibm.com/api/v3',
headers: {
accept: 'application/vnd.github.v3+json',
'user-agent': 'Boomerang Flow Joe Bot octokit-rest.js/17.11.2 octokit-core.js/2.5.4 Node.js/14.19.0 (Linux 4.18; x64)'
},
mediaType: { format: '', previews: [] },
request: { agent: undefined, hook: [Function: bound bound register] },
url: '/orgs/{org}/teams',
org: 'Boomerang-Delivery',
per_page: '100',
page: 1
}
GET /orgs/Boomerang-Delivery/teams?per_page=100&page=1 - 200 in 265ms
06/21/23 09:24:13 🔍  Successful response:  [
{
name: 'Abu Dhabi Health Authority - DOH',
id: 22310,
node_id: 'MDQ6VGVhbTIyMzEw',
slug: 'abu-dhabi-health-authority-doh',
description: '',
privacy: 'secret',
url: 'https://github.ibm.com/api/v3/organizations/132952/team/22310',
html_url: 'https://github.ibm.com/orgs/Boomerang-Delivery/teams/abu-dhabi-health-authority-doh',
members_url: 'https://github.ibm.com/api/v3/organizations/132952/team/22310/members{/member}',
repositories_url: 'https://github.ibm.com/api/v3/organizations/132952/team/22310/repos',
permission: 'pull',
parent: null
},
Environment
prod and stage env
Additional Contenxt
No response

FYI @gchickma


client_side argument for H2Connection is undocumented

The examples seem to use False but the default argument is True. I suspect it means client vs. server and there simply isn't any client-side example code (yet).

Ah, there is some client-side example code in the docs here. However, good spot that client_side is undocumented. I'll add that today.


Add support for responseType=json

When working with FakeXMLHttpRequest I found that there were several instances where my tests were failing because xhr.response wasn't correctly parsed as JSON.
I have added code to support setting responseType = "json" and return a response inline with the spec here: https://xhr.spec.whatwg.org/#dom-xmlhttprequest-responsetype
There are tests around this new functionality and nothing else is reporting as a fail.
Thanks for the great package!

Thanks. The use of defineProperty will kill support for IE < 9. Not sure if we're there yet. @mantoni @mroderick ?

Personally, I think we should help killing anything < IE 9. However, we should still make it a major release then.

I feel the same way. A 2.0 release where we no linger actively support IE<9 would be good.
People can still use 1.x, which is awesome
Sent from phone, apologies for brevity and auto-correct weirdness.

On 05/06/2015, at 11.42, Maximilian Antoni<EMAIL_ADDRESS>wrote:
Personally, I think we should help killing anything < IE 9. However, we should still make it a major release then.
—
Reply to this email directly or view it on GitHub.

Hey, if you like I could re-work this to include support < ie 9?
Thanks :)

Hey, if you like I could re-work this to include support < ie 9?

Sure, that'd be great :)

Ok, I'll try and take a look at this a little later today.  Would you want me to kill off Object.defineProperty completely or would you prefer to check for the capability and provide a fallback if not present?

I've refactored to remove the Object.defineProperty usage - it has meant that the behaviour doesn't exactly match the specification but I think it's as close as will be needed. Obviously if support for < IE9 is dropped then I'd advocate going back to the first version as it offered a more robust copy of the intended behaviour.
Cheers! :)

:+1:


Flutter Engine should consider banning static initializers

See Chromium's policy and reasoning:
https://chromium.googlesource.com/chromium/src/+/lkcr/docs/static_initializers.md
http://neugierig.org/software/chromium/notes/2011/08/static-initializers.html
Unless maybe already our build rules do?  @chinmaygarde might know.

No. Our current build rules do not ban static initializers. Though, I have tried addding the same in the past. There are a few dependencies that do not compile with that updated options. I think we should ban them in engine code to start and then expand scope from there.

Is this still a valid issue?

Yes. We have fewer globals and singletons in the engine than when this issue was first written. But getting rid of a few (especially some in our dependencies) is hard. Maybe we can consider a ban for additions.


Meteor-service-worker vs appcache

Hi NitroBay,
Great looking plugin and am interested in trying it out. My question is, what are the main differences between your service worker plugin and the meteor appcache (https://docs.meteor.com/packages/appcache.html). They seem to solve the same problem of caching assets in the browser. I'm developing an offline first web app and am not sure what the pros/cons are of the 2 possible solutions.
Any feedback is appreciated. Thanks and will talk with you later.

@centinel3
"Before service worker, there was one other API that gave users an offline experience on the web called AppCache. There are a number of issues with the AppCache API that service workers were designed to avoid."
this link may help explain - https://developers.google.com/web/fundamentals/getting-started/primers/service-workers

@centinel3 Very sorry for the late answer but it could still be useful to some to I answer now. @sferoze has summed up pretty well. Appcache was a promising solution when it has been published but then a lot of reasons makes it a solution not liked by devs. The core problems was appcache is a declarative* high-level API (so you couldn't make a lot of things as high level = easy to use but less options. I.E : HTML with its english-like tags is a high-level markup language) which brought all cons explained in the article I've linked.
Whereas Service Worker is a very low level imperative API so you can do an infinite number of things literally .
Appcache has been deprecated anyway.
Applied to the Meteor use case for now both are acting pretty much the same but as appcache has been deprecated you should use Service Worker. Moreover if you can understand how SW are working you can do whatever you want and store whatever you want. Modify my code as much as you want. In the future I shall offer a more complex API and support plugins etc. I've still have to wait for Meteor to let package creators register the service worker automatically. They're waiting for Apple to implement the Service Worker (Apple recently have started the development of this feature).

“imperative programming is like how you do something, and declarative programming is more like what you do, or something.”

As for the last bit of your message I'd be pleased to talk with you for sure even though I now I've answered way too late.


SQLite3 error

I'm getting this error running barnabas commands:
~ ᐅ barnabas list
unable to open database file (SQLite3::Exception)
[4339580434] *CallStack::unwind:Array(Pointer(Void)) +82
[4339580337] *CallStack#initialize:Array(Pointer(Void)) +17
[4339580296] *CallStack::new:CallStack +40
[4339549641] *raise<SQLite3::Exception>:NoReturn +25
[4339849892] *SQLite3::Connection#check<Int32>:Nil +36
[4339849640] *SQLite3::Connection#initialize<DB::Database>:Nil +120
[4339849499] *SQLite3::Connection::new<DB::Database>:SQLite3::Connection +123
[4339848980] *SQLite3::Driver#build_connection<DB::Database>:SQLite3::Connection +20
[4339549539] ~procProc(DB::Connection)@./libs/db/db/database.cr:39 +35
[4339841548] *DB::Pool(DB::Connection+)@DB::Pool(T)#build_resource:DB::Connection+ +60
[4339841426] *DB::Pool(DB::Connection+)@DB::Pool(T)#initialize<Int32, Int32, Int32, Float64, Int32, Float64, &Proc(DB::Connection+)>:Mutex +146
[4339835448] *DB::Pool(T)::new:initial_pool_size:max_pool_size:max_idle_pool_size:checkout_timeout:retry_attempts:retry_delay<Int32, Int32, Int32, Float64, Int32, Float64, &Proc(DB::Connection+)>:DB::Pool(DB::Con[4339835448] *DB::Pool(T)::new:initial_pool_size:max_pool_size:max_idle_pool_size:checkout_timeout:retry_attempts:retry_delay<Int32, Int32, Int32, Float64, Int32, Float64, &Proc(DB::Connection+)>:DB::Pool(DB::Connection+) +280
Unhandled exception in spawn:
end of file reached (IO::EOFError)
[4339580434] *CallStack::unwind:Array(Pointer(Void)) +82
[4339580337] *CallStack#initialize:Array(Pointer(Void)) +17
[4339580296] *CallStack::new:CallStack +40
[4339544473] *raise<IO::EOFError>:NoReturn +25
[4339682655] *IO::FileDescriptor+@IO#read_fully<Slice(UInt8)>:Int32 +111
[4339717316] *IO::ByteFormat::LittleEndian::decode<Int32:Class, IO::FileDescriptor+>:Int32 +68
[4339623618] *Int32@Int::from_io<IO::FileDescriptor+, IO::ByteFormat::LittleEndian:Module>:Int32 +34
[4339682530] *IO::FileDescriptor+@IO#read_bytes<Int32:Class>:Int32 +34
[4339712901] *Event::SignalHandler#run:NoReturn +53
[4339544316] ~procProc(Nil)@/usr/local/Cellar/crystal-lang/0.19.4_1/src/event/signal_handler.cr:93 +12
[4339573332] *Fiber#run:(IO::FileDescriptor | Nil) +164
[4339544297] ~proc2Proc(Fiber, (IO::FileDescriptor | Nil))@/usr/local/Cellar/crystal-lang/0.19.4_1/src/fiber.cr:29 +9
nection+) +280
[4339824620] *DB::Database#initialize<DB::Driver+, URI>:DB::Pool(DB::Connection+) +412
[4339824183] *DB::Database::new<DB::Driver+, URI>:DB::Database +151
[4339783643] *DB::build_database<URI>:DB::Database +43
[4339783585] *DB::build_database<String>:DB::Database +17
[4339809083] *RecipientRepository#load:Array(Recipient) +91
[4339808961] *RecipientRepository#all:Array(Recipient) +49
[4339550837] ~procProc(Array(String), RecipientRepository, Symbol)@./src/command/list.cr:2 +37
[4339792276] *Command::Base#process<Array(String), RecipientRepository>:Symbol +52
[4339488471] __crystal_main +2503
[4339545192] main +40

Fresh install with homebrew using OSX 10.11.16 and latest ZSH

@patrick-armitage sorry to hear that!
if you run brew info barnabas, which version does it tell you is installed?

Thanks for following up on this!  brew info barnabas yields:
~ ᐅ brew info barnabas
anicholson/andrewdotnich/barnabas: stable 1, HEAD
Encourage your friends from your terminal
https://github.com/anicholson/barnabas
/usr/local/Cellar/barnabas/1 (7 files, 964.6K) *
  Built from source on 2016-10-31 at 09:37:29
From: https://github.com/anicholson/homebrew-andrewdotnich/blob/master/barnabas.rb
==> Dependencies
Required: crystal-lang ✔

@patrick-armitage I think I worked it out! Barnabas depends on SQLite, but I'd forgotten to flag it as an explicit dependency.
Could you please run brew update, then brew install barnabas --HEAD and see if you still get the error?

@anicholson thanks, tried brew install barnabas --HEAD after uninstalling and now getting an SQLite3 error during install process:
~ ᐅ brew uninstall --force barnabas
Uninstalling barnabas... (7 files, 964.7K)
~ ᐅ brew install barnabas --HEAD
==> Installing barnabas from anicholson/andrewdotnich
==> Cloning https://github.com/anicholson/barnabas.git
Cloning into '/Users/patrick.armitage/Library/Caches/Homebrew/barnabas--git'...
remote: Counting objects: 254, done.
remote: Total 254 (delta 0), reused 0 (delta 0), pack-reused 254
Receiving objects: 100% (254/254), 35.28 KiB | 0 bytes/s, done.
Resolving deltas: 100% (151/151), done.
==> Checking out revision master
==> crystal deps update
==> crystal build src/barnabas.cr
==> crystal src/setup.cr
Last 15 lines from /Users/patrick.armitage/Library/Logs/Homebrew/barnabas/03.crystal:
[4393140068] *SQLite3::Connection#check<Int32>:Nil +36
[4393139816] *SQLite3::Connection#initialize<DB::Database>:Nil +120
[4393139675] *SQLite3::Connection::new<DB::Database>:SQLite3::Connection +123
[4393139540] *SQLite3::Driver#build_connection<DB::Database>:SQLite3::Connection +20
[4392992963] ~procProc(DB::Connection)@./libs/db/db/database.cr:39 +35
[4393135052] *DB::Pool(DB::Connection+)@DB::Pool(T)#build_resource:DB::Connection+ +60
[4393134930] *DB::Pool(DB::Connection+)@DB::Pool(T)#initialize<Int32, Int32, Int32, Float64, Int32, Float64, &Proc(DB::Connection+)>:Mutex +146
[4393130008] *DB::Pool(T)::new:initial_pool_size:max_pool_size:max_idle_pool_size:checkout_timeout:retry_attempts:retry_delay<Int32, Int32, Int32, Float64, Int32, Float64, &Proc(DB::Connection+)>:DB::Pool(DB::Connection+) +280
[4393122140] *DB::Database#initialize<DB::Driver+, URI>:DB::Pool(DB::Connection+) +412
[4393121703] *DB::Database::new<DB::Driver+, URI>:DB::Database +151
[4393095051] *DB::build_database<URI>:DB::Database +43
[4393094993] *DB::build_database<String>:DB::Database +17
[4393112820] *RecipientRepository#create:DB::ExecResult +68
[4392934762] __crystal_main +1866
[4392990984] main +40

READ THIS: https://git.io/brew-troubleshooting
If reporting this issue please do so at (not Homebrew/brew):
  https://github.com/anicholson/homebrew-andrewdotnich/issues

@patrick-armitage I was able to reproduce the error you just saw, and pushed a fix. Can you try again, please? :)

@anicholson it installs without error now, so we are definitely making progress :)
~ ᐅ brew install barnabas --HEAD
==> Auto-updated Homebrew!
Updated 2 taps (anicholson/andrewdotnich, homebrew/core).
==> New Formulae
snap-telemetry
==> Updated Formulae
anicholson/andrewdotnich/barnabas         boost-build                               docker-swarm                              mercurial                                 s6
awscli                                    clib                                      freeipmi                                  mysql-sandbox                             sqlmap
bazel                                     convox                                    libvirt                                   roswell                                   zorba

==> Installing barnabas from anicholson/andrewdotnich
==> Cloning https://github.com/anicholson/barnabas.git
Updating /Users/patrick.armitage/Library/Caches/Homebrew/barnabas--git
==> Checking out revision master
==> crystal deps update
==> crystal build src/barnabas.cr
==> Installation complete!
==> Run 'barnabas setup' to get started
==> Shell integration
==>  If you use bash for your shell, add
==> source /usr/local/Cellar/barnabas/HEAD-b0e761c/share/suggest.bash
==> to your .bashrc file.
==> If you use ZSH instead, add
==> source /usr/local/Cellar/barnabas/HEAD-b0e761c/share/suggest.zsh
==> to your .zshrc file.
🍺  /usr/local/Cellar/barnabas/HEAD-b0e761c: 7 files, 979.3K, built in 4 seconds
~ ᐅ source /usr/local/Cellar/barnabas/HEAD-b0e761c/share/suggest.zsh

The problem seems to be, first, that the barnabas setup command brew asks me to run is not a valid command:
~ ᐅ barnabas setup
setup is not a valid command.
Known commands:
help			Print this help
add			Add a new contact to your list
list			List your contacts
send			Send a message
suggest			suggest someone to contact

And also, that it seems there's still an SQLite error when using other commands:
~ ᐅ barnabas list
unable to open database file (SQLite3::Exception)
[4429549826] *CallStack::unwind:Array(Pointer(Void)) +82
[4429549729] *CallStack#initialize:Array(Pointer(Void)) +17
[4429549688] *CallStack::new:CallStack +40
[4429516761] *raise<SQLite3::Exception>:NoReturn +25
[4429819940] *SQLite3::Connection#check<Int32>:Nil +36
[4429819688] *SQLite3::Connection#initialize<DB::Database>:Nil +120
[4429819547] *SQLite3::Connection::new<DB::Database>:SQLite3::Connection +123
[4429819028] *SQLite3::Driver#build_connection<DB::Database>:SQLite3::Connection +20
[4429516659] ~procProc(DB::Connection)@./libs/db/db/database.cr:39 +35
[4429811596] *DB::Pool(DB::Connection+)@DB::Pool(T)#build_resource:DB::Connection+ +60
[4429811474] *DB::Pool(DB::Connection+)@DB::Pool(T)#initialize<Int32, Int32, Int32, Float64, Int32, Float64, &Proc(DB::Connection+)>:Mutex +146
[4429805496] *DB::Pool(T)::new:initial_pool_size:max_pool_size:max_idle_pool_size:checkout_timeout:retry_attempts:retry_delay<Int32, Int32, Int32, Float64, Int32, Float64, &Proc(DB::Connection+)>:DB::Pool(DB::Con[4429805496] *DB::Pool(T)::new:initial_pool_size:max_pool_size:max_idle_pool_size:checkout_timeout:retry_attempts:retry_delay<Int32, Int32, Int32, Float64, Int32, Float64, &Proc(DB::Connection+)>:DB::Pool(DB::Connection+) +280
Unhandled exception in spawn:
end of file reached (IO::EOFError)
[4429549826] *CallStack::unwind:Array(Pointer(Void)) +82
[4429549729] *CallStack#initialize:Array(Pointer(Void)) +17
[4429549688] *CallStack::new:CallStack +40
[4429511593] *raise<IO::EOFError>:NoReturn +25
[4429652399] *IO::FileDescriptor+@IO#read_fully<Slice(UInt8)>:Int32 +111
[4429687060] *IO::ByteFormat::LittleEndian::decode<Int32:Class, IO::FileDescriptor+>:Int32 +68
[4429593234] *Int32@Int::from_io<IO::FileDescriptor+, IO::ByteFormat::LittleEndian:Module>:Int32 +34
[4429652274] *IO::FileDescriptor+@IO#read_bytes<Int32:Class>:Int32 +34
[4429682645] *Event::SignalHandler#run:NoReturn +53
[4429511436] ~procProc(Nil)@/usr/local/Cellar/crystal-lang/0.19.4_1/src/event/signal_handler.cr:93 +12
[4429542724] *Fiber#run:(IO::FileDescriptor | Nil) +164
[4429511417] ~proc2Proc(Fiber, (IO::FileDescriptor | Nil))@/usr/local/Cellar/crystal-lang/0.19.4_1/src/fiber.cr:29 +9
nection+) +280
[4429794668] *DB::Database#initialize<DB::Driver+, URI>:DB::Pool(DB::Connection+) +412
[4429794231] *DB::Database::new<DB::Driver+, URI>:DB::Database +151
[4429753403] *DB::build_database<URI>:DB::Database +43
[4429753345] *DB::build_database<String>:DB::Database +17
[4429779131] *RecipientRepository#load:Array(Recipient) +91
[4429779009] *RecipientRepository#all:Array(Recipient) +49
[4429517957] ~procProc(Array(String), RecipientRepository, Symbol)@./src/command/list.cr:2 +37
[4429762324] *Command::Base#process<Array(String), RecipientRepository>:Symbol +52
[4429455543] __crystal_main +2503
[4429512312] main +40

That's a strange error, because the update to the formula should have happened after I'd pushed the setup command!
I moved some of the setup actions the formula was doing into this setup command. Once that shows up in your list of commands, you should be good to go :)
I'm sure you have been, but have you been brew updateing between these? That might help if not.
Thanks for sticking with me while I work this out! @patrick-armitage :)

Thanks @anicholson!  Hmmm you're right, that is weird.  I just ran brew cleanup --force which cleared out some previous barnabas installs, then brew update, then brew install barnabas --HEAD, and still no setup command exists.  I wonder why my formula won't update...  Maybe I need to remove and retap your repo?

@patrick-armitage that is odd! If you cd to /usr/local/Cellar/barnabas, find the git version, and do a manual git pull, that should punch it through.

@anicholson i built from source and it works!  could not figure out how to git pull in brew cellar.  anyway, i'd say this one is resolved in master.  thanks!

Excellent! I'm glad :)
Please let me know your experiences - feature requests, bug reports, etc.
:)


[ot_certs,codegen] Avoid the use of strlen for string literals.

We do not have an implementation of strlen on devices.

I suppose we can also just use __builtin_strlen


Removed rendering test

Speaks for itself, I have removed the test rendering, maybe make a release of this as well?

Good one on my part, thanks for the fix.

No problem, I really like the mod, but seeing a random green line somewhere in the world was quite annoying 😛

Indeed. I need to find a better way to handle testing in a fabric mod.


Update HelpWindow

Closes #26
HelpWindow displaying only a link to the user guide felt unfriendly and not very useful.
The link is updated, and the window now shows basic command usage. This makes it easier for the users to get help.

Codecov Report

Merging #41 (f43791b) into master (abecde1) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master      #41   +/-   ##
=========================================
  Coverage     71.96%   71.96%
  Complexity      399      399
=========================================
  Files            70       70
  Lines          1234     1234
  Branches        125      125
=========================================
  Hits            888      888
  Misses          314      314
  Partials         32       32

Impacted Files
Coverage Δ

...eatwhere/foodguide/logic/commands/HelpCommand.java
100.00% <ø> (ø)

...rc/main/java/eatwhere/foodguide/ui/HelpWindow.java
0.00% <ø> (ø)

...here/foodguide/logic/parser/AddressBookParser.java

...twhere/foodguide/logic/parser/FoodGuideParser.java
100.00% <0.00%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


figma design element for home page

📑 Description
figma design element for home page

[ ] Not Completed
[x] Completed

✅ Checks

[ ] My pull request adheres to the code style of this project
[ ] My code requires changes to the documentation
[ ] I have updated the documentation as required
[ ] I have checked the page with https://validator.unl.edu/
[ ] All the tests have passed
[ ] My pull request is based on the latest commit (not the npm version).

ℹ Additional Information

There are no links are added. Where should we direct to?

@zoltanszogyenyi I think the link for Preview in Figma is https://www.figma.com/file/5pHMkriSz9q98zawojb4mx/flowbite-pro-figma-v2.5.0?node-id=18-0&t=X431WUvSP7jsPiEI-0, and the link for Duplicate Flowbite Design System is https://flowbite.com/figma/. But the title should be "Learn more" or "Figma design system".

On iPad, I get this.
I think we should hide the image for iPad width or make the image smaller or responsive.


Update informations about PropTypes lib

[x] I have read freeCodeCamp's contribution guidelines.
[x] My pull request has a descriptive title (not a vague title like Update index.md)
[x] My pull request targets the master branch of freeCodeCamp.
[x] None of my changes are plagiarized from another source without proper attribution.
[x] My article does not contain shortened URLs or affiliate links.

@fjeansilva Thank for trying to address the error CamperBot identified, but you still have not corrected the issue.  Currently the frontmatter block looks like:
title: Typechecking With PropTypes
localeTitle: Checagem de tipo com PropTypes

---
Checagem de tipo com PropTypes
---

and it should look like:
---
title: Typechecking With PropTypes
localeTitle: Checagem de tipo com PropTypes
---

@RandellDawson Thank you for notifying.

@fjeansilva Looks like you have resolved all the front matter issues.  Now, we just have to wait for one of our Portuguese translation reviewers to get to this PR to review the actual content.
Thanks for you quick response.

@RandellDawson welcome.

Thank you for your contribution to the FreeCodeCamp Portuguese Translation!

Hope you continue to help us with more contributions.


Numerically stable derivative for split_node

It would be very nice for TensorNetwork's net.split_node and net.split_node_full_svd to have a numerically stable derivative out-of-the-box, since this is (I think) the major barrier to TensorNetwork being fully automatically differentiable for practical use.
Using this paper's algorithm and corresponding code, I have (very easily) implemented such an SVD for TensorNetwork's PyTorch backend, which works well in calculations, even up to the third derivative. [1]
I guess the questions I'd like to discuss are:

Would it be good to implement numerically stable derivatives for split_node, split_node_full_svd? Perhaps just for the jax and pytorch backends?

The algorithm I linked above uses a Lorentzian broadening to make the derivative numerically stable. This introduces some error in the gradient.
a) Is there a better algorithm to implement, hopefully one which doesn't introduce error?
b) If the best algorithm is the Lorentzian broadening one, perhaps it's best to add an optional argument broadening to split_node, maybe with default value 0, so that people can choose an error/stability tradeoff, and are aware that they're making one? That said, having such an argument might make people expect that split_node has backend-independent stable derivative... and if that's not the case, it might lead to unexpected errors for TensorFlow users.

If this seems like a good idea, I'm happy to work on implementing it -- though I can't guarantee that the work will happen in a timely fashion if I do it, and anyone willing to pick this up in a more timely fashion is of course welcome to do so.
[1]:  When I tried to implement the very same algorithm in the TensorFlow backend, I ran into significant issues (of many kinds) trying to take even the second-order automatic derivative. I haven't tried implementing it in jax, but knowledgeable people have told me that the issues I faced in TensorFlow should not occur there, perhaps another argument for #183 .

I do think this is a good idea!
 I'm not surprised it was hard to get working.
In general I say go for it! we can discuss how the API should look once we are reasonably certain it's working. (It might also might be a good idea to just include this in the upstream PyTorch and JAX libraries.)

 I'm not surprised it was hard to get working.

I think I can tell this is not an "official Google product" 😄

Just wanted to weigh in with this article on differentiable linear algebra. They go into a bit of detail/discussion on differentiable matrix operators. Unfortunately they work for Amazon so they only have implementations in MXNet. Remembered it as having an implementation for SVD, but on checking it only has symmetric Eigen, but maybe you can draw something from it anyway.

Hi,
I would like to contribute to this issue.

Great! Go for it.

Thanks a lot, could you assign me the same?
I am going through Lorentzian broadening and MXNet. Will keep updating with my findings. Any leads wrt previous experiments (how they were run) / results apart from torchsvd code would be very helpful.

Assigned it to you. As for experiments, we don't have any in particular you can run, unless @sidek has something.

@sidek @Bonnevie It would be great to hear your inputs on implementing Lorentzian broadening and about a good starting point in the code base where we would introduce the change?
Do we intend on developing wrappers for TensorFlow, jax and PyTorch backends?