How to convert a displacement field in numpy array to sitk.transform type ?

Hi,
I want to apply a displacement field in matrix(numpy array), with shape: N * M * K * 3 to deform images.
May I know how to convert the np.array to sitk.transform type?
I am using below code for deforming images with known DVF:
resampler = sitk.ResampleImageFilter()
resampler.SetReferenceImage(fix)
resampler.SetInterpolator(sitk.sitkBSpline)
resampler.SetDefaultPixelValue(0)
resampler.SetTransform(DVF in .tfm)                 -> where np.array to .tfm is needed
regmov = resampler.Execute(mov)

I tried couples of functions but all fail to convert np.array to .tfm .
I will appreciate for any of your suggestions!

Hello @XXZhou25,
The following code creates a deformation field from a numpy array:
import SimpleITK as sitk
import numpy as np

arr = np.random.random((10,20,30,3))
displacement_image = sitk.GetImageFromArray(arr, isVector=True)
# Set the displacement image's origin and spacing to relevant values
#displacement_image.SetOrigin()
#displacement_image.SetSpacing()
tx = sitk.DisplacementFieldTransform(displacement_image)

Hello @XXZhou25,
The following code creates a deformation field from a numpy array:
import SimpleITK as sitk
import numpy as np

arr = np.random.random((10,20,30,3))
displacement_image = sitk.GetImageFromArray(arr, isVector=True)
# Set the displacement image's origin and spacing to relevant values
#displacement_image.SetOrigin()
#displacement_image.SetSpacing()
tx = sitk.DisplacementFieldTransform(displacement_image)

Hello @XXZhou25,
The following code creates a deformation field from a numpy array:
import SimpleITK as sitk
import numpy as np

arr = np.random.random((10,20,30,3))
displacement_image = sitk.GetImageFromArray(arr, isVector=True)
# Set the displacement image's origin and spacing to relevant values
#displacement_image.SetOrigin()
#displacement_image.SetSpacing()
tx = sitk.DisplacementFieldTransform(displacement_image)

Hi @zivy ,
Thank you so much for your prompt reply!
It works perfect!
Happy new 2021 year:)

Hello @XXZhou25,
The following code creates a deformation field from a numpy array:
import SimpleITK as sitk
import numpy as np

arr = np.random.random((10,20,30,3))
displacement_image = sitk.GetImageFromArray(arr, isVector=True)
# Set the displacement image's origin and spacing to relevant values
#displacement_image.SetOrigin()
#displacement_image.SetSpacing()
tx = sitk.DisplacementFieldTransform(displacement_image)

Hi @zivy ,
Thank you so much for your prompt reply!
It works perfect!
Happy new 2021 year:)


terraform: 0.7.4 -> 0.7.6

Motivation for this change
Things done

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

[x] NixOS
[ ] OS X
[ ] Linux

[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[x] Fits CONTRIBUTING.md.

Weird, terraform should already be at 0.7.6: https://github.com/NixOS/nixpkgs/tree/master/pkgs/applications/networking/cluster/terraform

mmm, old master in my side? I thought it would diff with the current master in the PR though

Yeah that's what I am expecting too. Mind rebasing to make sure?

When attempting a rebase, it errored out saying it was an empty commit:
The previous cherry-pick is now empty, possibly due to conflict resolution.
If you wish to commit it anyway, use:

    git commit --allow-empty

Thank you for the initiative, though :)


[RUN-3894] Create Java version migration document

From now on, we will mention all known issues due to Java version migration in this document. It will be a growing document, with Java 21 issues as well when we start supporting that version.

Approval from the Runtime Core team is required.

Looks good to me. I am not sure about the location of the document, but I will let the docs team decide on that.

Hi @jveldhuizen
I think you make a good case for moving it to the Java Programming section, so I'll do that.
Done in this commit: https://github.com/mendix/docs/pull/7352/commits/a2df27f866fb77f2e7013a2f73b6d8dc3463ed3c
OK to publish now?

@MarkvanMents, thanks. Yes you can publish it. Is its 'weight' correct, which is the lowest now in this directory, 4?

Thanks for the reminder - I had made the change locally but forgotten to commit it.
It is now above Managed Dependencies but below the rest of the Java documentation.
I'll merge this and make a Jira issue to pick up the improvement to the Java Programming section.


Write docs for aggregation and sum

@frederikbosch Can you write some docs for these topics before we tag a release? Not sure if I'll have the time before next week, so if you get there before could you pick this one up?

@sagikazarmark Shall we create the tag then?


⚠ AVS (kty) has degraded performance

In 9a4cd40, AVS (kty) (https://avus.com.br) experienced degraded performance:

HTTP code: 200
Response time: 8627 ms

Resolved: AVS (kty) performance has improved in 043c634.


Spike: Export HESA data

Context
We'd like to export applications including HESA codes in a convenient format.

Changes proposed in this pull request
Adds an export class to export in the requested format.
There are some fields where we do not yet have data (ie. diversity information), the only other problematic field is QLAIM, which I can't find data for in Course#qualifications so this needs further investigation
Sample output:

id
status
first name
last name
date of birth
nationality
domicile
email address
recruitment cycle
provider code
accredited body
course code
site code
study mode
SBJCA
QLAIM
FIRSTDEG
DEGTYPE
DEGSBJ
DEGCLSS
institution country
DEGSTDT
DEGENDDT
institution details
sex
disability
ethnicity

22
recruited
Jeraldine
Bode
1967-08-19
British
GB
<EMAIL_ADDRESS>2020
1JA
Keele and North Staffordshire Teacher Education
2ZRM
-
2
100511
none
1
93
100166
13
GB
2020
2010
22

48
recruited
Ashlea
Yost
1983-03-03
British
GB
<EMAIL_ADDRESS>2020
1JA
The OAKS (Ormiston and Keele SCITT)
3282
D
1
100433
none
1
200
100069
7
GB
2020
2020
122
not specified
not specified
not specified

Guidance to review

Link to Trello card
https://trello.com/c/9Cado6Zz/2806-spike-hesa-data-export

Things to check

[x] This code does not rely on migrations in the same Pull Request
[x] If this code includes a migration adding or changing columns, it also backfills existing records for consistency
[x] API release notes have been updated if necessary
[x] New environment variables have been added to the Azure config

Closing as the implementation is here https://github.com/DFE-Digital/apply-for-teacher-training/pull/3135


chore: add Gitpod

What type of change does this PR introduce?

[ ] Bugfix
[ ] Feature
[ ] Refactor
[ ] Documentation
[x] Not Sure?

Does this PR introduce breaking changes?

[ ] Yes
[x] No

Description:
Add Gitpod to ease the contribution process.
Basically, every person that have a browser could contribute to Leon without configuration, without the need to have Node.js or Python installed.
It is not mandatory and local developement workflow will still happen, but for people that don't want to configure too many things for Leon, they would have the choice to use Gitpod. :smile:
You can already test it out by visiting this URL: https://gitpod.io/#https://github.com/leon-ai/leon/tree/chore/add-gitpod

 ✨ [1.0.0-beta.4] Gitpod


Implement Jensen' Device

I'm not really sure if I've grokked the original Algol 60 idea and if a closure is really a reasonable substitution (hope it is, due to Rust's static dispatch).

Great! Thanks for this!


Update component example for v2

Problem/Motivation
The add-on changed its default port and admin location since v2 of the add-on.
The documentation was not updated to reflect this.

Want to back this issue? Post a bounty on it! We accept bounties via Bountysource.

Both panel_iframe and sensor components needs updating.
The example you posted on the forums however, seems incorrect:
sensor:
  - platform: pi_hole
    host: localhost:4865
    location: ""

Isn't the admin panel still on /admin (and the matching API endpoint on /admin/api.php as per the docs?

No it is not at this point, see #39

Oh great, missed that!


Add some files from the MetOffice feed for testing

It would be useful to have a few representative files checked into the repo for testing purposes (i.e. to write unit tests against). Also if we want to change the way we store things (e.g. #17) so we can test changes locally.

Definitely agree!
The Met Office NetCDF files are about 100 MBytes.
100 MB maybe feels a little large to put into our git repo.  What do you think?
We could put a handful of testing NetCDF files in an OCF public S3 bucket (which would also help us test the code that reads from S3.  But that would slow the tests down (when running locally, at least)).  Or we could reduce the filesize by throwing away most of the data, and then saving the NetCDF in git.  Or we could do both?!?  Or something else?  What would you prefer?

Scrap what I said above!  Some of the NetCDF files are less than a MB (so we can probably just stick them into git).
For reference, here's the mean size (in MB) of MOGREPS-UK NetCDF files from the Met Office:
air_pressure                                                                             229.484640
air_pressure_at_sea_level                                                                  3.359660
air_temperature                                                                           21.432869
atmosphere_boundary_layer_thickness                                                        3.463518
cloud_area_fraction                                                                        2.139006
cloud_area_fraction_assuming_only_consider_surface_to_1000_feet_asl                        1.191044
cloud_base_altitude_assuming_only_consider_cloud_area_fraction_greater_than_2p5_oktas      1.198854
cloud_base_altitude_assuming_only_consider_cloud_area_fraction_greater_than_4p5_oktas      1.142050
cloud_volume_fraction_in_atmosphere_layer                                                 31.550393
dew_point_temperature                                                                      4.072501
fog_area_fraction                                                                          0.287845
freezing_level_altitude                                                                    4.102886
high_type_cloud_area_fraction                                                              1.457913
land_binary_mask                                                                           0.209811
low_type_cloud_area_fraction                                                               2.073978
lwe_graupel_and_hail_fall_rate                                                             0.145724
lwe_snowfall_rate                                                                          0.142409
lwe_thickness_of_graupel_and_hail_fall_amount                                              0.145957
lwe_thickness_of_snowfall_amount                                                           0.143660
lwe_thickness_of_surface_snow_amount                                                       0.368421
medium_type_cloud_area_fraction                                                            1.509519
number_of_lightning_flashes_per_unit_area                                                  0.139348
rainfall_rate                                                                              0.545779
relative_humidity                                                                         70.426761
soil_temperature                                                                           5.251025
surface_air_pressure                                                                       7.087338
surface_altitude                                                                           2.405621
surface_diffusive_downwelling_shortwave_flux_in_air                                        4.592893
surface_direct_downwelling_shortwave_flux_in_air                                           4.460242
surface_downwelling_longwave_flux_in_air                                                   6.106000
surface_downwelling_shortwave_flux_in_air                                                  5.235519
surface_downwelling_ultraviolet_flux_in_air                                                1.054016
surface_temperature                                                                        2.256696
surface_upward_sensible_heat_flux                                                          4.923685
surface_upwelling_ultraviolet_flux_in_air                                                  0.400496
thickness_of_rainfall_amount                                                               0.620976
total_radar_reflectivity                                                                  24.705189
total_radar_reflectivity_max_in_column                                                     3.865009
upward_air_velocity                                                                      112.851796
visibility_in_air                                                                          7.156086
wet_bulb_freezing_level_altitude                                                           4.419323
wet_bulb_potential_temperature                                                             4.963114
wind_from_direction                                                                       81.070012
wind_speed                                                                                33.663551
wind_speed_of_gust                                                                         3.214056```


TypeScript declarations

Hi,
first of all thanks to you for your solid work!
We are planning to use your library within one of our TypeScript-based projects and noticed that there aren't any type declarations yet.
I'm willing to provide those declarations and it would be nice when some one could test / extend the types to include serial and UDP, as we probably only use either TCP or UDP in our project.
I will publish the types in my own repository first of all and if everything looks okay a PR to the official @types namespace managed by https://github.com/DefinitelyTyped/DefinitelyTyped will be created.
There are no changes to the actual codebase necessary, though this project would really benefit from a migration to typescript (codecompletion for driver / stream options etc.)

Thank you. When you have it ready, just pull it here 👍


Docs: v2 todo list

This is a tracking issue for tasks related to documentation updates that need to happen before the v2 release.

[ ] #1285
[ ] #1050
[ ] the OpenAPI generation CLI doc isn't up to date with the current CLI: the new command is starlite schema typescript (for instance, to get the typescript client lib)
[ ] the tutorial on JWT Auth uses the old typing API for the Request type. The V2 type is Request[User,Token,State].
[ ] #1429
[ ] #1619

The docs for AbstractAuthenticationMiddleware has these imports at the very top:
from litestar.authentication import (
    AbstractAuthenticationMiddleware,
    AuthenticationResult,
)

It seems like the correct import for these is now:
from litestar.middleware import (
    AbstractAuthenticationMiddleware,
    AuthenticationResult,
)

The docs for AbstractAuthenticationMiddleware has these imports at the very top:

Fixed in #1814


Kustomize 2.0 migration issue

As noted in https://github.com/replicatedhq/ship/issues/859, Ship is still on the 1.x branch of Kustomize.
Questions

Can we upgrade kustomize without breaking ship?
If changes are required in ship code, do these also require changes to end user's state.json files or will we be able to migrate seamlessly? What are possible solutions and their costs in terms of effort or added complexity?

We're currently on 2.0.2, we just aren't filling out apiVersion and kind - something we should be doing though!
https://github.com/replicatedhq/ship/blob/ce0e98ade7d08b76975cdc27ffe50b845590d908/Gopkg.toml#L90-L92

ah gotcha, nice catch -- closing then, thanks @laverya


[AutoPR network/resource-manager] Add enableTcpReset property to load balancer

Created to sync https://github.com/Azure/azure-rest-api-specs/pull/3545

(message created by the CI based on PR content)


chore: re-generate project with latest ember-cli

This project was insanely far out of date. Simply upgrading ember-cli-babel (#75) unfortunately doesn't seem to cut it. Furthermore, CI has been broken for ages.
Considering it's small scope, I've opted for re-generating the project from scratch by doing the following:

delete all files in the repo
ember addon --yarn --skip-git --skip-npm @ember-intl/cp-validations
move over all new files
compare diff and restore actual code
modernize
patch ember-cp-validations

based on qonto/ember-cp-validations
and additional patches

patch ember-validator

Unless ember-cp-validations & ember-validator are published upstream with those fixes this PR cannot be published for production use.

@buschtoens Thanks for the work done to revive this addon :) So basically what we should do now is publish https://github.com/qonto/ember-cp-validations to npm so we can use it here? v4.0.0-beta.13.1 uses the latest version of https://github.com/offirgolan/ember-validators
Let me know if this is what we need as the next step.

Hi @jasonmit @herzzanu @Turbo87,
Wanted to follow-up on this PR. What work is needed to get this merged with main? These changes are needed for this library to be compatible with Ember 4.x. It looks like this PR has been in limbo for a few months now.
cc: @buschtoens

@justinbaltazar I've been out of the ember community for a while but verified @buschtoens has the proper github & npm permissions to push forward this change.

Superseded by https://github.com/ember-intl/cp-validations/pull/87


Add netlink support to VdpaDevice interface

This PR is based on top of #9 . **API breaking change **
In order to add netlink support to the vdpa device API this PR does some heavy preliminary refactoring:

kvdpa.go is renamed to device.go
virtio-net device and vhost-vdpa device information handling is moved to their own files (virtio.go and vhost.go) and encapsulated into their own interfaces VirtioNet and VhostVdpa which can hold different information without contaminating the same API.
Many free functions are now part of the vdpaDev struct, the end API looks like this:

type VdpaDevice interface {
	Driver() string
	Name() string
	VirtioNet() VirtioNet
	VhostVdpa() VhostVdpa
	ParentDevicePath() (string, error)
}

@lmilleri PTAL


Include missing root directory files in wp_dash_exceptions

Example: Patching package.json via grunt patch:31337 fails with the following error:
Result:
can't find file to patch at input line 5
Perhaps you used the wrong -p or --strip option?
The text leading up to this was:
--------------------------
|Index: package.json
|===================================================================
|--- package.json	(revision 31464)
|+++ package.json	(working copy)
--------------------------
File to patch:
Skip this patch? [y]
Skipping patch.
1 out of 1 hunk ignored

Tests pass for Node.js 0.10 and fail on 0.8


Add module initializer with context corresponding to NODE_MODULE_INITIALIZER

Node 10 added Context-aware addons, using the NODE_MODULE_INITIALIZER. To be able to make such an addon with compatibility with older versions we need a corresponding macro in nan.

Hi @murgatroid99,
I think that NAN_MODULE_WORKER_ENALBED is the macro that you need. https://github.com/nodejs/nan/blob/master/nan.h#L158
Can anyone say if I'm right or not?

That does look relevant, but it looks like it just ignores the context parameter, instead of providing some kind of shim or fallback. So, it looks like it doesn't help with actually writing an addon that is context-aware in Node 10+.

@NickNaso are there any plans to resolve this issue or is there anything I could contribute?
This one is currently blocking https://github.com/grpc/grpc-node/issues/778 which in turn is causing other trouble further down stream (https://github.com/zeit/next.js/issues/7894) so I would be interested in getting this resolved.

According to electron/electron#18397, as of Electron 11 this will be required to load all native addons in Electron. Therefore, changing this should be a priority.

That does look relevant, but it looks like it just ignores the context parameter, instead of providing some kind of shim or fallback.

What should that shim/fallback do in your opinion? Add-ons can already obtain the context in their initializer with Nan::GetCurrentContext().

Look, you have this definition:
#if NODE_MAJOR_VERSION >= 10 || \
    NODE_MAJOR_VERSION == 9 && NODE_MINOR_VERSION >= 3
#define NAN_MODULE_WORKER_ENABLED(module_name, registration)                   \
    extern "C" NODE_MODULE_EXPORT void                                         \
      NAN_CONCAT(node_register_module_v, NODE_MODULE_VERSION)(                 \
        v8::Local<v8::Object> exports, v8::Local<v8::Value> module,            \
        v8::Local<v8::Context> context)                                        \
    {                                                                          \
        registration(exports);                                                 \
    }

That applies to the version range we are interested in, and it discards the context parameter instead of passing it to the registration function. This macro appears to correspond to the NODE_MODULE_INIT macro described in the Context-aware addons documentation, and that documentation states that the context variable is available in the function body passed to that macro. Based on this information, it seems like NAN_MODULE_WORKER_ENABLED does not facilitate the creation of a context-aware addon the way NODE_MODULE_INIT does.
Honestly, I was unaware of Nan::GetCurrentContext() and nobody mentioned it in this issue before, and the documentation doesn't clearly indicate that it can be used in that function as a replacement for the context variable.

@murgatroid99 Do you want to update the API or the docs? In both cases a pull request is the best way forward. :-)

I don't know what the right change here is. I'm just a user of this library; that doesn't mean I'm knowledgeable about the nuances of the Node API or its changes across versions.
In particular, you say that Nan::GetCurrentContext() can access the current context in the module initializer. If that is the case, why does Node's API also pass it in as an argument to the initializer? I think someone who knows the answer to that question is in a better position to determine what change should be made here.

why does Node's API also pass it in as an argument to the initializer?

As the one who introduced the original multi-context support in Node: because it seemed like a good idea at the time. :-)


Clarification of instructions for Add the seed initializer

On the page "Work with SQL Server LocalDB in ASP.NET Core" in the section "Add the seed initializer" it should state to replace the contents of Program.cs with the following code. This might be obvious to more experienced individuals but not so to beginners.

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

ID: a0a4022c-7ef1-3c30-4f9d-cf521a57a44a
Version Independent ID: a0b5e15c-d747-8710-95fb-a55c09861a1e
Content: Work with SQL Server LocalDB in ASP.NET Core
Content Source: aspnetcore/tutorials/first-mvc-app/working-with-sql.md
Product: aspnet-core
GitHub Login: @Rick-Anderson
Microsoft Alias: riande

@Stumpy842  agreed.

This part was causing errors for me: saying that Movie object already exists. When I deleted the dbo.Movie object problem was fixed. Later on I realized I don't need that part of the code since it trying to migrate the database again:
var context = services.GetRequiredService();
context.Database.Migrate();
just delete this 2 lines of code and it works just fine.

In Razor Pages tutorial, that was pointed out, exactly as you said it. My chosen workaround was also to delete the dbo.Movie table (while I was following the Razor Pages tutorial). Good catch, by the way. Kudos!

I forgot to mention, when encountering the same error, the other fix is to stop IIS Express, so this will 'force' the recreation of the table. I believe it is worth mentioning this, too. Just in case somebody does not like the idea of deleting the dbo.Movie table. I got this to work, without deleting the table, but stopping IIS Express.


Do first release

Determine what we think the milestone is for a first release and then do so.

Two thoughts on a first release:

We need at least one method in the membership class. Since we moved the various get_students methods into the gradebook class, this will be new work.
Since the membership class is a green field now, can we just have a very basic method that just passes requests through directly to the membership service? This would also simply our docs, since we can just point to the IS&T docs on the membership API.

So talking with @pdpinch and realizing more and more about how these APIs are completely independent but heavily overlapping, I agree and propose we do one last refactor before release. The proposal:

Refactor base url in the constructor to just be http(s)://<domain-name>[:port]/
Refactor Membership to append services/membership in the constructor
Refactor Gradebook to append services/gradebook to the constructor
Drop Client entirely since this change will make it so get, post, and delete are unresolvably ambiguous
Refactor grades2stellar to use Gradebook and truncate the default base url

This will give us a good release point since Membership will legitimately work with the Base methods and have some functioning code that differentiates it from Gradebook (i.e. the shortcut constructor append).
We could also add a trivial call like get_staff to membership as well.  That should be fairly quick and easy to write, but I don't think that should hold back the release neccessarily as long as we have a doc showing that as an example for how to use Membership

Good plan !

This sounds all good to me, especially adding the get_staff() function to nail down a pattern for new membership functions since the earliest implementation seems to win.

I've made this a check list now, with completed work being checked off, feel free to tackle issues and mark more off (I think you can do that, if not tell me and I'll check them off)


Not able to run in iOS 16

I have upgraded my visual studio version and trying to run with iOS 16 simulator. Not able to run.
Error MT4109: Failed to compile the generated registrar code. Please file a bug report at https://github.com/xamarin/xamarin-macios/issues/new

Microsoft support for Xamarin will end on May 1, 2024 for all Xamarin SDKs including Xamarin.Forms. In preparation for this, all issues and PRs in this repository are being closed.
.NET MAUI is the replacement for Xamarin.Forms, and it's documentation can be found here.


Modified cardDetails.json (new card)

Added new card for myself in cardDetails.json.

PR MERGED!
Thanks for Contributing :)


scipy.interpolate needs flexible ways of constructing smoothing splines and to influence the number of knots (including manual selection of number)

http://docs.scipy.org/doc/scipy/reference/generated/scipy.interpolate.UnivariateSpline.html#scipy.interpolate.UnivariateSpline
I've been using UnivariateSpline for smoothing noisy data, and it's great -- except that the smoothing factor is a pain to choose properly. If I have a lot of data, and by mistake I choose too low of a smoothing factor, it takes a looooooooong time for the spline to be constructed. Sensitivity to the smoothing factor is very high: if I use s0 it might give me a spline with 100 knots, but if I use s0*0.99 it might give me a spline with 5000 knots.
In the end, I don't really care about the smoothing factor; what I do care about is the number of knots. The optimal smoothing for my application is the point of diminishing returns: if there is a figure of merit F(n) used for spline fit (mean squared error or whatever), where n is the number of knots, what I want is to choose the largest n such that the gain of F(n) - F(n-1) is not small. Yes, this is an imprecise number, but I think it would be very helpful to allow either of the following parameters in the constructor:

N -- number of knots. If I have 10,000 points of noisy data, let me specify that I want 100 spline knots. UnivariateSpline() gets to pick where they are, but I want the best spline you can give me with N=100 knots.
a termination function f() which receives key parameters of one iteration of the UnivariateSpline's attempt to smooth, and returns either True (the knots are good enough) or False (not good enough, keep adding knots). If I care about comparing to previous attempts I'll make it a class with a __call__() method. Maybe all I care about is how long it takes, in which case I'll use the time since calling UnivariateSpline as a metric for when to terminate. :-)

Or better yet, how about a generator that yields successive values of (spline, otherstuff) where spline is the spline with the next higher # of knots, and otherstuff is information about the spline fit. Then I can keep calling next() until I get a spline that I'm happy with.

Completely agree - ended up writing a loop on smoothing factor to get a desired number of knots, which is ridiculous.

I just stumbled onto this same issue, and I 100% agree. We're trying to migrate to SciPy from R's smooth.spline method, which provides a parameter nknots. This is a current barrier that we're trying to work through.

We definitely want more flexible ways to construct smoothing splines.
The first activity in this direction is gh-16653 which implements GCV smoothing splines with knots at datapoints. My current thinking is that the next step is to allow a user-specified knot vector. An interface can be  make_smoothing_spline(x, y, t=...), where the t parameter selects knots.
Help is welcome and needed, on several fronts:

[ ] review gh-16653 which serves as a basis for further improvements
[ ] implement the construction of a smoothing spline given the knot vector and a smoothing parameter lambda
[ ] research and summarize the prior art for selecting knot vectors. This includes patsy, statsmodels, scikit-learn, possibly others in the python land. Outside of the pydata ecosystem, it's useful to have understanding an a summary of how R's smooth.spline selects knots, also matlab etc. For these, need to be careful to not look into license-incompatible code.
[ ] For the use case of "I want no more than N knots, place them automatically", would be nice to understand what FITPACK does (need to read through the Dierckx papers/book). What is the criterion for adding a knot?
[ ] Understand and implement alternative knot placement strategies, e.g. de Boor's

All of these can be useful to have as python prototypes first. Once the protypes are there, we can

[ ] craft a user interface: is a single t=... parameter, possibly overloaded for an integer number, a string strategy or a vector of knots enough? do we really want a generator? do we need a way to say "insert a knot automatically into this BSpline using this smoothness criterion"? Etc.
[ ] See what functionality needs to be moved from pure python to compiled code.

Implementation-wise, we will definitely not add things to the UnivariateSpline. All of this is best done as standalone functions which return a specific BSpline instance.

Experimenting with knot placement strategies should get easier now that gh-19970 is in. One thing that it includes is the generate_knots generator to replicate what FITPACK does.


Support import multiple repositories by uploading a manifest file

Feature Description
AOSP repo's manifest.xml can import multiple git repositories at once, which is very useful for large projects.
This feature is a copy of gitlab feature
Screenshots
No response

Please give more detail

Please give more detail

https://github.com/GerritCodeReview/git-repo
https://github.com/GerritCodeReview/git-repo/blob/main/docs/manifest-format.md

But how does exactly is meant to be supported in Gitea?

But how does exactly is meant to be supported in Gitea?

This feature is a copy of gitlab feature.

Sorry for my description is not very clear.

GitLab allows you to import all the required Git repositories based on a manifest file like the one used by the Android repository. Use the manifest to import a project with many repositories like the Android Open Source Project (AOSP).

See also #2569

This issue proposals supporting for the manifest file when mass importing existing repositories. So it's not very same as #2569

Are there any plans to implement this feature?


config/default.js not formed properly after npx install [BUG]

Summary
config/default.js not formed properly after npx install
Steps To Reproduce
run "npx @salesforce/pwa-kit-create-app " and go through installation, connecting to a sandbox
Expected result
when running "npm start" it should result in a running local PWA
Actual result
when running "npm start" an error is thrown because there is no comma in default.js on line 16-ish
`/Users/terrancegrahn/Documents/dev/pwa/whoop/node_modules/cosmiconfig/dist/ExplorerSync.js:93
throw e;
^
SyntaxError: /Users/terrancegrahn/Documents/dev/pwa/whoop/config/default.js: Unexpected token, expected "," (20:12)
The fix was simply to add a comma in the applicable area, but the installer should incorporate this.
System Information (as applicable)
Browser:
Node version:
npm 9.8.1
node v18.18.2
pwa-kit version:
3 (where is the exact version located?)
Desktop OS:
Mobile Device Info:
Additional information

Thanks for catching this, @tgrahn2! I actually also spotted this recently, and the fix is ready in #1612.

Fixed in v3.4.0


Add the footnote links for comparing releases #270

Follow-up to PR #271 (for issue #270)

looks good, but closing (in favor of #275)


Info message is printed during binary component creation

[kind/bug]
What versions of software are you using?

Operating System: macOS
Output of odo version: v1.0.0-beta1

How did you run odo exactly?
clone https://github.com/grahamdumpleton/parksmap-web
mvn package
odo create java frontend --binary target/parksmap-web.jar --app parksmap

Actual behavior
odo create java frontend --binary target/parksmap-web.jar --app parksmap
I0425 19:11:24.732607    1769 create.go:138] target/parksmap-web.jar
 ✓  Checking component
 ✓  Checking component version
Please use `odo push` command to create the component with source deployed

Expected behavior
I wouldn't expect that info message to be printed:
I0425 19:11:24.732607    1769 create.go:138] target/parksmap-web.jar
https://github.com/openshift/odo/blob/2b98a06e811a60a08c9a6545c9698896d91e8f0b/pkg/odo/cli/component/create.go#L138
Any logs, error output, etc?

Hey all, I've opened up a PR here: #1657


Add p2p port range option into Subutai configuration

Subutai user should be able to specify a port range for p2p to use. This will be helpful for advanced users, who can open specific range of ports for p2p to avoid usage of control peers.
In a subutai console, under network settings should be an input field with two input boxes separated by '-', representing a range of ports, that should be used by p2p.
If no ports were specified (default value) - p2p start command should be executed without changes, but if port range specified, special -ports flag should be provided during startup.

Implemented


Static build system + drone CI integration

This PR:

adds the static dependency build system from loki-core/lokinet for producing a build with dependencies (rather than needing an out-of-build-system script to deal with them):

updates dependencies of the static build dep; these were painfully out of date, especially for openssl which has had several security updates since the version that was being bundled here
devendors sqlite3; sqlite3 is widely available with a very stable API so building against a system one by default seems more appropriate (and matches what loki-core and lokinet builds do).  (For the static build, we now download and build sqlite3 as part of the build via the above static dep mechanism).
the above all required some minor cmake reworking to use the static target dependencies when active.

adds drone CI builds with uploads to https://oxen.rocks with builds under various distros/versions/architectures

adds "strip" and "create_archive" targets, which get used in the drone CI scripts, that are able to build a release-ready package (rather than needing to unpack and repack for a release).

fixes some minor compiler warnings

Edit:
Closes #191 because this static deps system works fine on macos (and even windows, though that doesn't really matter here).

(https://ci.oxen.rocks/jagerman/loki-storage-server/91 is the CI job for this PR)

(https://ci.oxen.rocks/jagerman/loki-storage-server/91 is the CI job for this PR)


improved blog page,scroll reveal

Added scroll reveal effect to blog template
fixes issue no #600
hey @vishanurag
I have added the scroll reveal effect and also improved the overall aesthetics of the blog template/page, i have attached a demo video for your reference , kindly check
https://github.com/user-attachments/assets/55d3ea54-3f18-458f-a8a5-962ca4bca239
add the labels : gssoc extd , hacktoberfest
If you like my work please add level 3 label

can u pl add level 3 ?


Outgoing call doesn't disconnect properly

Description
Hello, dear Twilio team. I have a problem with outgoing calls. When I make a call from my app to GSM number, it calls properly, but when receiver hang up the call, in my app the call doesn't finish, and callDidDisconnect function doesn't get call. My app still calling, and finish after receiver hang up call after two-three times.
Expected Behavior
I expect to finish call properly
Reproduces How Often
20-30%
Versions
All relevant version information for the issue.
Voice iOS SDK
[e.g. 2.0.0-beta16 via CocoaPods]
Xcode
[e.g. 8.3.1]
iOS Version
[e.g. 10.3.3]
iOS Device
[e.g. iPhone 6]

Hi @talalt
Thanks reporting the issue.
Can you provide us the Call-SID (you can find it in the TVOCall.sid object once the call is connected) when you are experiencing the issue? We can use the Call-SID to dig up more information about the lifecycle of the call and see what happened.
A better way of providing us debugging info is to enable TVOLogLevelVerbose log-level of the SDK and send the debug log to us. We don't want to post any of your credentials here publicly so please go to Twilio Support and create a ticket. Please tag the ticket with Programmable Voice iOS SDK and bchen so that the support experts know where to forward the issue.
Thanks
-bobie

Hi @bchen-twilio
Thanks for your answer.
Call-SID: CAe6d270443918d77a9c7248e9b0bf6906
Log file:  twilio-log.txt
Also I usually faced with problem described here. I use TwilioVoice 2.0.0-beta16 via CocoaPods. Could you please look that problem too? Thanks!

Hi @talalt
Thanks for providing us the Call-SID and SDK log.
From the log I can actually see the client getting the disconnected-by-remote event and the call did end properly. Did you get any TVOCallDelegate callbacks (call disconnected or failed) in this call?

Hi @bchen-twilio
Yes, call disconnects, but before that the receiver needs to hang up two-three times, because after first hang up, the call continues to calling. I didn't get any callbacks when receiver hangs up the call.

Hi @talalt
If I am understanding this correctly, the situation you are seeing is:

the receiver (GSM phone number) hangs up the call
the call on the SDK client side (mobile app) doesn't get disconnected immediately
even if the call is disconnected (on the mobile side) eventually, you never get the callDidDisconnect: callback?

thanks,
bobie

Hi @bchen-twilio
Yes, situation looks like that, except that when the call disconnects properly, I properly get callDidDisconnect: callback. No problem with that. Also when receiver hangs up the call, call disconnects on his side, but not on the mobile side. And after few seconds call ring on the receiver side again. Thanks!

Hi @talalt
Thanks for the update.
Based on your description I think this is beyond the SDK can handle. Could you please reach out to the Twilio Support and open a ticket with the Call-SIDs that you are experiencing problematic calls. We experts from the Voice service team to dig up more insights why the calls behaved weirdly.
Thanks
bobie

I have the same issue, the callback is never called.also I noticed call.delegate = self cannot be used since it is a get-only property

Hi @logesh45
Please use Twilio Support to open a new ticket with the SDK (verbose log level) log so we can take a look why the delegate didn't get the callback.
The attribute of the TVOCallDelegate is intentionally designed that way so you will need to provide the delegate when making or accepting calls.
Thanks.

We had redesigned the entire SDK in 3.x. The latest version of SDK is 5.2.0. See changelogs for 5.2.0 here. I
Closing an old ticket. Feel free to reopen if you still run into the same issue or have any questions.

Hello
I am still facing issues while an outgoing call is disconnected from another side. I am using latest sdk version.

Hello
I am still facing issues while an outgoing call is disconnected from another side. I am using latest sdk version.

Hi @Nisha-spaceo,
Any chance you have the Call SID of such call? We can use that to check the signaling records and the TwiML so we can help you resolve the issue.

Hi @Nisha-spaceo,
Any chance you have the Call SID of such call? We can use that to check the signaling records and the TwiML so we can help you resolve the issue.


Format

Codecov Report

:exclamation: No coverage uploaded for pull request base (main@4f7ba00). Click here to learn what that means.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main      #17   +/-   ##
=======================================
  Coverage        ?   24.85%
=======================================
  Files           ?        6
  Lines           ?      515
  Branches        ?        0
=======================================
  Hits            ?      128
  Misses          ?      387
  Partials        ?        0

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 4f7ba00...07dea98. Read the comment docs.


Nest column with array or object?

Could use nest column array, object like this
"aliases": [
    "stormagedden",
    "bob"
  ]
---
"address": {
      "street": "12 Beaver Court",
      "city": "Snowmass",
      "state": "CO",
      "zip": "81615"
    }

Hi @thearabbit, could you describe better the idea?

Like excel-as-json

thanks 👍


Support binary type in take

Implementation is shared with take_string.

Codecov Report

Merging #5 (389643e) into cubestore-2021-01-02 (5813558) will decrease coverage by 0.00%.
The diff coverage is 69.23%.

@@                   Coverage Diff                    @@
##           cubestore-2021-01-02       #5      +/-   ##
========================================================
- Coverage                 81.35%   81.35%   -0.01%
========================================================
  Files                       208      208
  Lines                     52730    52744      +14
========================================================
+ Hits                      42901    42908       +7
- Misses                     9829     9836       +7

Impacted Files
Coverage Δ

rust/arrow/src/compute/kernels/take.rs
92.85% <69.23%> (-1.32%)
:arrow_down:

rust/arrow/src/array/transform/fixed_binary.rs
84.21% <0.00%> (+5.26%)
: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 5813558...389643e. Read the comment docs.

Codecov Report

Merging #5 (389643e) into cubestore-2021-01-02 (5813558) will decrease coverage by 0.00%.
The diff coverage is 69.23%.

@@                   Coverage Diff                    @@
##           cubestore-2021-01-02       #5      +/-   ##
========================================================
- Coverage                 81.35%   81.35%   -0.01%
========================================================
  Files                       208      208
  Lines                     52730    52744      +14
========================================================
+ Hits                      42901    42908       +7
- Misses                     9829     9836       +7

Impacted Files
Coverage Δ

rust/arrow/src/compute/kernels/take.rs
92.85% <69.23%> (-1.32%)
:arrow_down:

rust/arrow/src/array/transform/fixed_binary.rs
84.21% <0.00%> (+5.26%)
: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 5813558...389643e. Read the comment docs.


document that output template types can be separated by ,

DO NOT REMOVE OR SKIP THE ISSUE TEMPLATE

[X] I understand that I will be blocked if I intentionally remove or skip any mandatory* field

Checklist

[X] I'm requesting a feature unrelated to a specific site
[X] I've looked through the README
[X] I've verified that I'm running yt-dlp version 2023.10.13 (update instructions) or later (specify commit)
[X] I've searched known issues and the bugtracker for similar issues including closed ones. DO NOT post duplicates
[X] I've read the guidelines for opening an issue

Provide a description that is worded well enough to be understood
Hey.
To my best knowledge it seems undocumented so far that one can apparently use a ,-separated list for the TYPE in output templates.
I neither found anything at -P. --output or the OUTPUT TEMPLATE section.
This makes it however much easier to e.g. move all meta-data to a subdirectory like via:
--paths subtitle,thumbnail,description,annotation,infojson,link,pl_thumbnail,pl_description,pl_infojson,chapter,pl_video:metadata

instead of:
--paths subtitle:metadata --paths thumbnail:metadata --paths description:metadata --paths annotation:metadata --paths infojson:metadata --paths link:metadata --paths pl_thumbnail:metadata --paths pl_description:metadata --paths pl_infojson:metadata --paths chapter:metadata --paths pl_video:metadata

Perhaps it makes even sense to add this (i.e. placing everything except for the video file itself in some metadata subdir) as another one in the list of examples.
Thanks,
Philippe.
Provide verbose output that clearly demonstrates the problem

[ ] Run your yt-dlp command with -vU flag added (yt-dlp -vU <your command line>)
[ ] If using API, add 'verbose': True to YoutubeDL params instead
[ ] Copy the WHOLE output (starting with [debug] Command-line config) and insert it below

Complete Verbose Output
No response

This affects all the following options:
--color
--retry-sleep
--downloader
--downloader-args
--progress-template
--paths
--output

Any suggestions on how to document it?

Well I guess that it may not be enough to simply add a generic sentence like "multiple values can be separated by ,", because for e.g. the case of --path that would already be ambiguous (is it subtitle,thumbnail:dir or is it subtitle:dir,thumbnail:dir or both?). So perhaps really giving a precise description on where the , can be used for each of the options?


oncoplot is sort of blank

Describe the issue
I am not able to plot the oncoplot with bars, I get a blan center.

Command
oncoplot(
  maf = metb1,genes=brcagenes,
  sortByAnnotation = TRUE,
  clinicalFeatures = "cat_risk_core",
)

**Session info**
sessionInfo()

R version 4.2.1 (2022-06-23 ucrt)
Platform: x86_64-w64-mingw32/x64 (64-bit)
Running under: Windows 10 x64 (build 17763)

Matrix products: default

locale:
[1] LC_COLLATE=English_United States.1252  LC_CTYPE=English_United States.1252
[3] LC_MONETARY=English_United States.1252 LC_NUMERIC=C
[5] LC_TIME=English_United States.1252

attached base packages:
[1] stats     graphics  grDevices utils     datasets  methods   base

other attached packages:
 [1] maftools_2.12.0    RColorBrewer_1.1-3 hrbrthemes_0.8.0   viridis_0.6.2      viridisLite_0.4.1
 [6] ggplot2_3.4.0      NGCHM_0.13.1       MBatch_1.7.6       jsonlite_1.8.3     readxl_1.4.1
[11] stringr_1.5.0      dplyr_1.0.10       purrr_0.3.5

loaded via a namespace (and not attached):
  [1] sf_1.0-9             lubridate_1.9.0      httr_1.4.4           rprojroot_2.0.3
  [5] tools_4.2.1          utf8_1.2.2           R6_2.5.1             KernSmooth_2.23-20
  [9] uwot_0.1.14          DBI_1.1.3            colorspace_2.0-3     DNAcopy_1.70.0
 [13] withr_2.5.0          prettyunits_1.1.1    processx_3.8.0       tidyselect_1.2.0
 [17] gridExtra_2.3        curl_4.3.3           compiler_4.2.1       extrafontdb_1.0
 [21] cli_3.5.0            BiasedUrn_2.0.8      Cairo_1.6-0          flextable_0.8.3
 [25] xml2_1.3.3           desc_1.4.2           officer_0.5.0        labeling_0.4.2
 [29] scales_1.2.1         classInt_0.4-8       callr_3.7.3          proxy_0.4-27
 [33] systemfonts_1.0.4    digest_0.6.30        R.utils_2.12.2       rmarkdown_2.19
 [37] base64enc_0.1-3      pkgconfig_2.0.3      htmltools_0.5.4      oompaBase_3.2.9
 [41] extrafont_0.18       oompaData_3.1.3      fastmap_1.1.0        rlang_1.0.6
 [45] rstudioapi_0.14      generics_0.1.3       farver_2.1.1         zoo_1.8-11
 [49] mclust_6.0.0         gtools_3.9.4         R.oo_1.25.0          zip_2.2.2
 [53] magrittr_2.0.3       Matrix_1.5-1         Rcpp_1.0.9           munsell_0.5.0
 [57] fansi_1.0.3          logger_0.2.2         gdtools_0.2.4        R.methodsS3_1.8.2
 [61] lifecycle_1.0.3      stringi_1.7.8        ClassDiscovery_3.4.0 pkgbuild_1.4.0
 [65] Rtsne_0.16           plyr_1.8.7           grid_4.2.1           parallel_4.2.1
 [69] randomcoloR_<IP_ADDRESS>  crayon_1.5.2         lattice_0.20-45      splines_4.2.1
 [73] pander_0.6.5         ps_1.7.2             knitr_1.41           pillar_1.8.1
 [77] epiR_2.0.54          uuid_1.1-0           reshape2_1.4.4       glue_1.6.2
 [81] evaluate_0.19        squash_1.0.9         V8_4.2.2             data.table_1.14.4
 [85] remotes_2.4.2        vctrs_0.5.1          Rttf2pt1_1.3.11      cellranger_1.1.0
 [89] gtable_0.3.1         assertthat_0.2.1     xfun_0.36            eulerr_7.0.0
 [93] e1071_1.7-12         class_7.3-20         survival_3.4-0       tsvio_0.0.14
 [97] tibble_3.1.8         rJava_1.0-6          vizier_0.4.2         units_0.8-1
[101] cluster_2.1.4        timechange_0.1.1
R version 4.2.1 (2022-06-23 ucrt)
Platform: x86_64-w64-mingw32/x64Run `sessionInfo()` and post the output below

Hi,
Sorry for the late reply.  You will have to play around with the arguments sepwd_samples and sepwd_genes to make them visible. Also, please save the plot to a pdf with large width and height to make them visible.
I hope this helps.

Hi Anand, Thank you. I kind of figured because I was plotting mutations for
cell lines. Another not so important question I had if there is a way to
plot the oncoplot by sorting samples based on a quantitative variable ( I
know one can plot based on a categorical one). For eg, I want to look at
IC50 value (continuous) distribution across the cell lines(in an
ascending order), would I be able to do so?
Shwetha
Thank you so much for your help
On Wed, Mar 1, 2023 at 8:02 AM Anand Mayakonda @.***>
wrote:

Hi,
Sorry for the late reply.  You will have to play around with the arguments sepwd_samples
and sepwd_genes to make them visible. Also, please save the plot to a pdf
with large width and height to make them visible.
I hope this helps.
—
Reply to this email directly, view it on GitHub
https://github.com/PoisonAlien/maftools/issues/897#issuecomment-1450202339,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AETL4GXUELBI5WSKQYS3JNDWZ5JJBANCNFSM6AAAAAAT4GJRS4
.
You are receiving this because you authored the thread.Message ID:
@.***>


Modal close animation

Thanks for this wonderful library.
How would I add a small animation, so the modal closes softly, as it does when it opens? I find it slightly inharmonious as of now.
Many thanks in advance.

Any update or help available?

Hi @20Fourteen,
This is not possible for the moment because the closing event remove tingle from the dom immediately (using display: none).
I will take a look a this when I will have some free time.

Thank you. That's fine for me.
I wanted to know if this is possible with the current code. Now this changes to a feature request , let me know if you'll consider it when you have time.
Best regards.

Thank you! I'll have a look and see how this works out.

Hi @20Fourteen,
This is not possible for the moment because the closing event remove tingle from the dom immediately (using display: none).
I will try to take a look at this when I will have some free time.

Hi @robinparisi ,
i've just created a pr with a fix for this.
For the others before the issue is fixed it's possible to use the following workaround:
(function(close) {
    tingle.modal.prototype.close = function(force) {
        force = typeof force === 'boolean' ? force : undefined;
        close.call(this, force);
    };
}(tingle.modal.prototype.close));

Just execute it before new tingle.modal().
Additionally take a look at the transition of .tingle-modal. It should have something like:
    transition: visibility .2s ease, opacity .2s ease;

Best regards,
dr0bz

Hello
Excelent work by the way.
I´m trying to make the close smooth transittion work on a prestashop, and it simply does not work, i´ve tried replacing the modules tingle.min.css and tingle.min.js withe this new version and made the sugested changes on #50 but the close animation is not working.
I´ve noticed that it also does not work on this demos page -> https://tingle.robinparisi.com/
Any fix for this?
Best Regards

Hey @ivoduarte ,
there are to options:

first try out my branch https://github.com/dr0bz/tingle/tree/50-close-fix
if you would create a CodePen (or what you prefer) example, i could look at it:)

Best regards

Hey @ivoduarte ,
there are to options:
first try out my branch https://github.com/dr0bz/tingle/tree/50-close-fix
if you would create a CodePen (or what you prefer) example, i could look at it:)
Best regards

Hello @dr0bz
Perfect, it works, how did i miss that :)
Thanks friend
Best Regards

I managed to make it work but I used the destroy() option with a setTimeout. My Tingle modal is called 'dcTableModal' and i set my timer to match the animation e.g. 500
setTimeout(function () { 	var modal = dcTableModal.opts; 	dcTableModal.destroy(); 	dcTableModal = new tingle.modal(modal); 	return true; }, 500);


Google captcha integration to verify on sign-up

We'll want to have a captcha presented to the user when they click submit on their sign-up form.

There's some privacy concerns with Google's reCAPTCHA service. See here for more info

I don't see where there was any indication Google was using any data provided for advertising, or really collecting very much), just that people were suspicious of it.

Their privacy policy doesn't explicitly disallow data collection. Someone reverse-engineered it when it first came out, they're definitely collecting a lot of data:
https://github.com/toogle/InsideReCaptcha

Google servers will receive and process, at least, the following information:

Plug-ins
User-agent
Screen resolution
Execution time, timezone
Number of click/keyboard/touch actions in the  of the captcha
It tests the behavior of many browser-specific functions and CSS rules
It checks the rendering of canvas elements
Likely cookies server-side (it's executed on the www.google.com domain)
And likely other stuff...


Add self-uninstall command

We should make it easy to cleanly uninstall swiftly in case a user no longer needs it on their system.

while that pr is merged but still there is no way to uninstall swiftly itself, how can we uninstall it completely?

I think the best way right now would be the following:
$ swiftly uninstall all
$ rm -r ~/.local/share/swiftly
$ rm ~/.local/bin/swiftly

This is assuming you've installed to the default location.
You may also want to delete any swiftly-specific lines from your .profile, .bash_profile, .bash_login, or .zprofile, if there are any in any of those files.

Note, when going from 0.2.0 -> 0.3.0 one may get into trouble if one hasn't uninstalled all / removed all the symlinks
find ~/.local/bin/ -maxdepth 1 -type l
find ~/.local/bin/ -maxdepth 1 -type l -delete


Test branch with different slider behaviors

This is a test branch for us to experiment different slider behaviors in gallery & table view.
This branch is based on: https://github.com/SETI/pds-opus/pull/811
Local branch: test_branch_with_different_slider_behavior_based_on_slider_issue_807_and_infiniteScroll_issue_761
Current behavior:

Example 1: (13 images per row in gallery view)

Go to gallery view.
Move slider to a number (7190), it will match startObs (7190) in gallery view.
Switch to table view, slider number will match the one in gallery view (7190).
In table view, scroll the scrollbar so that 7198 is the top item (slider will move from 7190 to 7198 with step size 1).
Switch to gallery view, slider will move back to row boundary 7190.
Switch to table view, slider will move back to 7198.
In table view, scroll the scrollbar so that infiniteScroll load is triggered and 7283 is the top item (slider is 7283).
Switch to gallery view, the slider will move to new row boundary 7281.
Switch to table view, the slider will move back to 7283.
Switch to gallery view, and move the slider from 7281 to 13040.
Switch to table view, the slider will be 13040.

Example 2: (13 images per row in gallery view)

Go to table view.
Move slider to 3867, top item will be obs 3867.
Switch to gallery view, slider will move to 3862 (row boundary). Obs 3867 is one of item in top row.
Switch to table view, slider will move to 3867.
Switch to gallery view and scroll up scrollbar so that infiniteScrolll load is triggered, and slider move to 3797.
Switch to table view, slider is at 3797.

Update based on comments.
In gallery view, when all images fit on the screen, slider will be disabled. Switch to table view, slider will be enabled again (except when the time that all items in the table can fit on the screen).
When pasting an URL with a startObs not aligned with browser size, the startObs will be adjusted to a new value aligned with the browser size.
In browse view:

In gallery view:

When we resize the browser, some obs will be removed if necessary. The previous startObs will still be in the top row, and the new startObs will be smaller than previous one (startObs will increase when browser is resized).
When we keep resizing the browser, more and more DOM are deleted, at one point when scrollbar reaches to the top, infiniteScroll load event will be triggered and more data will be rendered.

In table view:

When we resize the browser, no DOM will be removed, and the top obs will not change (slider number will stay the same).
When we switch from table to gallery view, some obs will be removed to align with the browser size.

The latest update has fix to issue #793.

Scrolling will be smooth when infiniteScroll load is triggered.
Pasting an URL with startObs:

In gallery view, when pasting an URL with a startObs not aligned with current browser size, the startObs will be rounded down (aligned with current browser size).
In table view, when pasting an URL with a startObs not aligned with current browser size, the startObs will stay the same and not be rounded down.


Some basic setup/compilation changes

Generalized the cudnn path to any user's home directory
If the initial CUDA installation is invalid, running setup.sh will always fail. Moving CUDA_HAVE_GPU to a global variable allows it to be updated on each run of setup.sh.

Excellent! I will look into the change regarding those two CMake vairables and come back to you. Before that, could you do a git pull to update your entire repository? I made a huge change yesterday because of supporting composed mode.

Should be all set.


fixed minor typo in migration guide

Cheer for the PR. Will get it merged.

@davidtsadler Thanks!


Questions on the functional

Good afternoon.
Let me ask you a couple of questions:

Is it possible to check whether the user is authorized in the wallet?
Is it possible to get the address of the user's wallet?

Thank you in advance.

@dmitry-kanunnikoff once you receive signed message you can recover signed address.

@dmitry-kanunnikoff once you receive signed message you can recover signed address.

@vikmeup I'm sorry, but could you tell me how I can do this? Maybe there is some documentation?

And tell me, please, that in the end comes in response.result for, say, a signTransaction()? Is this the transaction ID hash?

The signed transaction is returned - the transaction data is serialized rlp, encrypted and again serialized rlp. These data can be deserialized and get a signature and transaction data. Look at https://github.com/ethereum/wiki/wiki/RLP.

@madcake Thank you very much for your reply! I'll look in this direction.

@madcake Good afternoon!
Thank you for your help!
I was able to decode the presentation of the transaction. However, I can not get the value of the 'from' field. An error pops up: "Invalid transaction v,r,s values".
It was discussed here: https://github.com/ethereum/go-ethereum/issues/14599.
But, as I understand, the solution boils down to the fact that now you need to specify the identifier of the chain.
Please tell me how can I get this chain ID with only a signed transaction?
Thank you in advance for any help!
https://pix.my/eQ9z1j

I found that, it turns out, the chain ID is not an arbitrary number. Turned to this list, and was able to pick up the ID I needed: https://ethereum.stackexchange.com/a/17101/46174
The problem is solved.
Thank you very much, guys!

@madcake Good afternoon.
I apologize for the disturbance. Tell me, please, how can I get the address of the wallet from the signed message (Trust.signMessage())? And is it even possible?
With the transaction, I did this:
val tx = Geth.newTransactionFromRLP(Numeric.hexStringToByteArray(response.result))
But how to decode the data from the signed message?

Look at 'recovery'

@madcake I am sorry, but I couldn't find the direction where to search.
Tell me, please, where I need to search 'recovery'? Can you post the direct link?

@madcake Maxim, could you, please, share more details?


Issues running in a WSL environment

In the process of getting acquainted with the repository, I discovered that the application will give this output before closing when it is run in a WSL environment
t$ ./run.sh
./run.sh: line 4: $'\r': command not found
"* Serving Flask app "main.py

Environment: production
WARNING: This is a development server. Do not use it in a production deployment.
Use a production WSGI server instead.
Debug mode: off
Ignore the duplicate column errors below, I had to catch it as a workaround
duplicate column name: CSRFCompleted
duplicate column name: Insecure Direct Object ReferenceCompleted
duplicate column name: Insecure DeserializationCompleted
duplicate column name: Proxy TestCompleted
duplicate column name: Test SQL InjectionCompleted
duplicate column name: Test XSSCompleted
duplicate column name: XXECompleted
initializing CSRF
initializing Insecure Direct Object Reference
initializing Insecure Deserialization
initializing Proxy Test
initializing Test SQL Injection
initializing Test XSS
initializing XXE
Usage: python -m flask run [OPTIONS]

".ror: Could not import "main.py
The reason I say that this is specific to the WSL environment is because I duplicated my actions within a virtual box and the application launched without complaining or throwing an error. This may extend to the Windows environment as well but my laptop does not have python installed in the plain windows environment to test this hypothesis.
Since the PyGoat application is being launched in the spirit of education, I believe solving this issue would make the app much more accessible to prospective cyber security students who may not find using a virtual box environment to their liking.
Currently I am attempting to work out a reason why the error is being thrown. My working theory is that it is a peculiarity with how the flask framework for python works, and I am researching the build process for flask to verify this.

Update:
while I do not understand fully what this line in the output stream is indicating
./run.sh: line 4: $'\r': command not found
since line 4 in run.sh is an empty line, I have a feeling it might be because of how bash scripts work they may not accept \r as a line terminating character.
Regardless, deleting the line cleaned up the output stream resulting in this output before closing once again:
$ ./run.sh
"* Serving Flask app "main.py

Environment: production
WARNING: This is a development server. Do not use it in a production deployment.
Use a production WSGI server instead.
Debug mode: off
Ignore the duplicate column errors below, I had to catch it as a workaround
duplicate column name: CSRFCompleted
duplicate column name: Insecure Direct Object ReferenceCompleted
duplicate column name: Insecure DeserializationCompleted
duplicate column name: Proxy TestCompleted
duplicate column name: Test SQL InjectionCompleted
duplicate column name: Test XSSCompleted
duplicate column name: XXECompleted
initializing CSRF
initializing Insecure Direct Object Reference
initializing Insecure Deserialization
initializing Proxy Test
initializing Test SQL Injection
initializing Test XSS
initializing XXE
Usage: python -m flask run [OPTIONS]

".ror: Could not import "main.py
The majority of the output is unimportant for the scope of this error, only the last line seems significant. I must research more about the flask framework before I know more

Fixed:
I solved the issue by copying the contents of the run.sh file to a new .bashrc file. The .bashrc allows a user on WSL to run the app by running the following commands:
chmod +x .bashrc
./.bashrc
The process for running the app in a Linux environment is still the same
chmod +x run.sh
./run.sh
I never discovered why the .sh file could not find main.py on execution, but a simple alternative execution path solved the problem


AddJournalQuestEntry

Source Code
Using
        public static void ModuleClientEnterJournal(ModuleEvents.OnClientEnter enter)
        {
            enter.Player.AddJournalQuestEntry("test", 1, false);
        }

With this set in toolset

Produces this result inside the game.

I've done journal entries in toolset before with nwscript. Almost same syntax used in both functions.  Basically I'm unsure if I'm doing something wrong or there is a bug in managed with this function?

As of latest update this seems to be fixed.

As of latest update this seems to be fixed.


Difference with debug enabled deadlocks on compareArray

Enabling the debug flag can cause lock contention as compareArray can loop on itself calling diff.Difference twice: https://github.com/google/go-cmp/blob/master/cmp/compare.go#L403
goroutine 5 [semacquire]:
sync.runtime_SemacquireMutex(0x4dffcc4, 0x900000000)
	GOROOT/src/runtime/sema.go:71 +0x3d
sync.(*Mutex).Lock(0x4dffcc0)
	GOROOT/src/sync/mutex.go:134 +0x153
repo.jazznetworks.com/jazz/main/vendor/github.com/google/go-cmp/cmp/internal/diff.(*debugger).Lock(0x4dffcc0)
	vendor/github.com/google/go-cmp/cmp/internal/diff/debug_enable.go:78 +0x7b
repo.jazznetworks.com/jazz/main/vendor/github.com/google/go-cmp/cmp/internal/diff.(*debugger).Begin(0x4dffcc0, 0x1, 0x1, 0xc000328550, 0xc0003466d8, 0xc000346708, 0xc00032e810)
	vendor/github.com/google/go-cmp/cmp/internal/diff/debug_enable.go:82 +0x54
repo.jazznetworks.com/jazz/main/vendor/github.com/google/go-cmp/cmp/internal/diff.Difference(0x1, 0x1, 0xc000328550, 0x1, 0x3, 0xc000326540)
	vendor/github.com/google/go-cmp/cmp/internal/diff/diff.go:200 +0x381
repo.jazznetworks.com/jazz/main/vendor/github.com/google/go-cmp/cmp.(*state).compareArray(0xc000342850, 0x477cd20, 0xc00032ad40, 0x197, 0x477cd20, 0xc0001a6c40, 0x197, 0x4947940, 0x477cd20)
	vendor/github.com/google/go-cmp/cmp/compare.go:403 +0x3e3
repo.jazznetworks.com/jazz/main/vendor/github.com/google/go-cmp/cmp.(*state).compareAny(0xc000342850, 0x477cd20, 0xc00032ad40, 0x197, 0x477cd20, 0xc0001a6c40, 0x197)
	vendor/github.com/google/go-cmp/cmp/compare.go:272 +0x1189
repo.jazznetworks.com/jazz/main/vendor/github.com/google/go-cmp/cmp.(*state).statelessCompare(0xc000342850, 0x477cd20, 0xc00032ad40, 0x197, 0x477cd20, 0xc0001a6c40, 0x197, 0x4102374, 0xc00032cf00)
	vendor/github.com/google/go-cmp/cmp/compare.go:181 +0x149
repo.jazznetworks.com/jazz/main/vendor/github.com/google/go-cmp/cmp.(*state).compareArray.func1(0x0, 0x0, 0x4933420, 0xc0000d4008)
	vendor/github.com/google/go-cmp/cmp/compare.go:405 +0x170
repo.jazznetworks.com/jazz/main/vendor/github.com/google/go-cmp/cmp/internal/diff.(*debugger).Begin.func1(0x0, 0x0, 0x1, 0x10)
	vendor/github.com/google/go-cmp/cmp/internal/diff/debug_enable.go:97 +0x141
repo.jazznetworks.com/jazz/main/vendor/github.com/google/go-cmp/cmp/internal/diff.Difference(0x1, 0x1, 0xc000326500, 0x1, 0x2, 0xc00032ade0)
	vendor/github.com/google/go-cmp/cmp/internal/diff/diff.go:216 +0x693
repo.jazznetworks.com/jazz/main/vendor/github.com/google/go-cmp/cmp.(*state).compareArray(0xc000342850, 0x477bda0, 0xc00032ad60, 0x97, 0x477bda0, 0xc00032ad80, 0x97, 0x4947940, 0x477bda0)
	vendor/github.com/google/go-cmp/cmp/compare.go:403 +0x3e3
repo.jazznetworks.com/jazz/main/vendor/github.com/google/go-cmp/cmp.(*state).compareAny(0xc000342850, 0x477bda0, 0xc00032ad60, 0x97, 0x477bda0, 0xc00032ad80, 0x97)
	vendor/github.com/google/go-cmp/cmp/compare.go:272 +0x1189
repo.jazznetworks.com/jazz/main/vendor/github.com/google/go-cmp/cmp.Equal(0x477bda0, 0xc00032ad60, 0x477bda0, 0xc00032ad80, 0xc00032ada0, 0x2, 0x2, 0x46c6b3a)
	vendor/github.com/google/go-cmp/cmp/compare.go:87 +0xda
repo.jazznetworks.com/jazz/main/vendor/github.com/google/go-cmp/cmp.Diff(0x477bda0, 0xc00032ad60, 0x477bda0, 0xc00032ad80, 0x0, 0x0, 0x0, 0x1, 0xc000040120)
	vendor/github.com/google/go-cmp/cmp/compare.go:101 +0x182
repo.jazznetworks.com/jazz/main/vendor/gocloud.dev/blob/drivertest.testList.func4(0xc000374000)
	vendor/gocloud.dev/blob/drivertest/drivertest.go:346 +0x619
testing.tRunner(0xc000374000, 0xc000348000)
	GOROOT/src/testing/testing.go:827 +0x163
created by testing.(*T).Run
	GOROOT/src/testing/testing.go:878 +0x65a

Are you reporting this because:

you were trying to debug the cmp library itself and noticed this?
you happened to have the debug tag set for an unrelated purpose and then noticed this?

Second option, this was a side effect I didn't expect.

Copy. The trivial fix is to change the build tag to something more specific (e.g., debugcmp).

Would be happy for that, not a big deal I just turned off the debug flags while testing.


pythonPackages.simplefix: init at 1.0.12

Motivation for this change
Things done

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

[ ] NixOS
[ ] macOS
[X] other Linux distributions

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

Notify maintainers

None of the tests get executed by this package as it is. To get them to work you've got to change the src to the github archive (which includes the tests):
  src = fetchFromGitHub {
    repo = "simplefix";
    owner = "da4089";
    rev = "v${version}";
    sha256 = "0pnyqxpki1ija0kha7axi6irgiifcz4w77libagkv46b1z11cc4r";
  };

and add
  checkPhase = ''
    cd test
    ${python.interpreter} -m unittest all
  '';

Great point, updated as you suggest to discover the tests; they all pass.

Feel free to ping me for merge when @jonringer changes are applied.

@worldofpeace Applied those changes and the checks have passed

@GrahamcOfBorg build python27Packages.simplefix python35Packages.simplefix python36Packages.simplefix python37Packages.simplefix

Thanks @catern :sparkles:


Upgrade tests against present branch

note
Please assume that I am referencing the examples folder in the v3 branch of ibctest.
Goal
In github actions, test an upgrade from Juno v11 to Juno v12, before the v12 release has been tagged
Problems

we end up with another go.mod file
IBCTest is looking up juno versions presumably in the chain registry
tying this into CI has never been done before
unlikely to be possible with standard ci (github actions)
requires tagged releases

we end up with another go.mod file

You mean in the Juno repo? I'm presuming you mean something like this
That could possibly be avoided if the main package for the repo is using same dep versions as the version of ibctest you are using. Regardless I don't see why this is necessarily a problem, Go modules as a dependency management system are incredibly simple and straightforward and do not add much overhead/technical debt.

IBCTest is looking up juno versions presumably in the chain registry

By default ibctest will attempt to look for the Docker images in heighliner but you can point to to whatever repo you want where the Docker image is built and the versioning/tags can be whatever you want them to be.

tying this into CI has never been done before
unlikely to be possible with standard ci (github actions)

We run a CI job in the relayer for ibctest and I believe the ibc-go team is also using ibctest in their CI. You are essentially just running standard Go tests so there is a lot of flexibility as far as how you could run the tests in CI.
https://github.com/cosmos/relayer/blob/main/.github/workflows/ibctest.yml
https://github.com/cosmos/ibc-go/search?q=ibctest

requires tagged releases

You don't need tagged releases of the chain binaries, you need tagged Docker images that can be pulled from some repository. That means you can build the Docker images from any arbitrary branch or codebase and push to a remote Docker repo like GHCR and you are up and running. It's possible that you could build the Docker images in CI and pipe the image information into the test cases and the CI job for running the test but I'm not 100% sure what that would look like
There is already an example showing how a test case for upgrading Juno would look like here

Dan's documentation work has made this more available now as seen here
https://github.com/strangelove-ventures/interchaintest/blob/main/docs/writeCustomTests.md
Closing since this + Juno, Noble, and PersistanceOne show how to do this in prod.


Add clustering to the map

Add clustering to the map as discussed in github/services-web#655.
cc/ @brianamarie @swinton

var markerCluster = new MarkerClusterer(map, markers, {
  imagePath: 'https://developers.google.com/maps/documentation/javascript/examples/markerclusterer/m',
  averageCenter: true,
  minimumClusterSize: 42
});

@stoe I ❤ how the clustering looks in your screenshot! When I spin up locally, though, I'm not seeing the clustered pins. Could I be missing a step or a piece of this?

@brianamarie, are seeing the numbers?

@stoe Nope, I believe I'm on the correct branch but I am likely doing something wrong. 😄

Nope, I believe I'm on the correct branch but I am likely doing something wrong.

@brianamarie, you're probably not doing anything wrong, often it is the browser cache that will show you old content.
Can you try and open dev tools in Chrome via Cmd + Option + I and then hit F1 to go to the settings.

You can disable cache for web development whenever dev tools are open via the ☝  checkbox. Saves a lot of time when developing websites/webapps.
Alternatively you can hit Cmd + Shift + R to hard reload with dev tools open, or long press on the reload icon left of the address bar and select "Empty Cache and Hard Reload".

@stoe, Thank you, that worked like a charm!

Looks good to me, thanks @stoe :+1:

☝ 😍  :+1: :ship:
I think we also need a separate PR on training-web to pull in these changes, it looks like the existing functionality is duplicated here.

I think we also need a separate PR on training-web to pull in these changes, it looks like the existing functionality is duplicated here.

@swinton that is all pulled in via a submodule update! Once this is merged we can take care of that!

@swinton, any last minute 💭s on this re/ your comments before we :ship:?

any last minute 💭s

Nope, I think we should :ship:, apologies for the delay in responding @stoe :+1:


Docker run return  No module named 'autosub'

I build the docker image as the instruction. It returns this error after running.
Traceback (most recent call last):
  File "./autosub/main.py", line 10, in <module>
    from autosub import logger
ModuleNotFoundError: No module named 'autosub'

I think the problem is I didn't have autosub installed in the image. So I add  RUN pip3 install . to Dockerfile. Then everything works out fine.
BTW: With this solution, it's also necessary to have  COPY README.md ./ in the image. Also, I have to add  encoding='utf-8' in  with open("README.md", "r") as fh: in  setup.py, otherwise it would use ASCII encoding as default in my case.

Another suggestion: I think it would be more elegant and concise if you can merge those "RUN pip3" steps in one RUN command.


Add arabic translation

Looking good, thank you!
However, there's a problem: currently your translation would use the flag of Argentina. I know using flags isn't the most optimal solution but there are no language emojis that can convey the same meaning and fit into the header as compactly as flags.
I also don't know of any other emoji that could represent Arabic as a whole. Also, I am not familiar with the use of Arabic in general. (I googled a bit and found out that there's the Arab League, but unlike EU it has no emoji). So I don't really know what to pick here. Is there any emoji I overlooked or a country that can be primarily associated with the Arabic language / has a flag frequently used to denote the Arabic language?
Otherwise I guess I'll have tocome up with a different solution for the language links.

Hii, it's ok to use any Arabic country since all speak Arabic here is :united_arab_emirates: that can be used

:united_arab_emirates:


Panic: Add Handler

@stevvooe
Enabling log hook to capture the registry panic, when it happens.
Refer to #394.
Signed-off-by: xiekeyang<EMAIL_ADDRESS>

@stevvooe  Done.

LGTM!
@RichardScothern @dmcgowan @endophage

LGTM


tooltipAnchor has limited or no effect for some Tooltip directions

[x] I've looked at the documentation to make sure the behavior is documented and expected

The coordinates of the point from which tooltips will "open", relative to the icon anchor.

[x] I'm sure this is a Leaflet code issue, not an issue with my own code nor with the framework I'm using (Cordova, Ionic, Angular, React…)
[x] I've searched through the issues to make sure it's not yet reported

Steps to reproduce
Steps to reproduce the behavior:

Define an Icon with the tooltipAnchor option with a non-zero value for x.
Create a Marker using that Icon, that has a Tooltip bound with an top, bottom or center direction.

Expected behavior
The anchor of the Tooltip should move relative to the iconAnchor according to the values defined in the tooltipAnchor. It should have the same behavior as popupAnchor according to the documentation, e.g. if given the same values it should position itself at the same position as the Popup.
Current behavior
tooltipAnchor.x has no effect for the directions top, bottom and center.
tooltipAnchor.y has no effect for the bottom direction.
Environment

Leaflet version: 1.5.1
Browser (with version): Chrome 76.0.3809.100, Waterfox 56.2.12
OS/Platform (with version): Windows 7 (6.1 Build 7601), Ubuntu 18.04

Additional context
Source code responsible for the issue:
https://github.com/Leaflet/Leaflet/blob/1e0d43b11d477e0c89de4dace33123aa871c76b8/src/layer/Tooltip.js#L144-L156
Minimal example reproducing the issue
https://jsfiddle.net/590jt8fx/
const map = L.map('map').setView([0.0, 0.0], 12)

const icon = L.icon({
  iconUrl: 'https://leafletjs.com/docs/images/logo.png',
  iconSize: [120, 31.8], // (600 x 159)/5
  iconAnchor: [60, 15.9], // center
  popupAnchor: [50, -15.9], // top right
  tooltipAnchor: [50, -15.9], // top right
})

L.marker([0, 0], { icon } )
  .bindPopup("popup is working correctly")
  .bindTooltip("tooltipAnchor.x has no effect", { direction: 'top' })
  .addTo(map)

L.marker([-0.02, 0], { icon } )
  .bindTooltip("offset is needed", { direction: 'top', offset: [50, 0] })
  .addTo(map)

[x] this example is as simple as possible
[x] this example does not rely on any third party code

same here

@Istador
@vangenechtenbert
Is it fixable by #6116?

@Istador
@vangenechtenbert
Is it fixable by #6116?

No, it's even more broken than before.

For comparison, this is the current behavior of my minimal example back then when I created the issue. The most recent master branch version shows the same behavior.
Hovering the first image:

Hovering the second image:

And this is the behavior of the latest commit 20784d7fbd7ef934e5c205caa4c98b891404eea6 from pull request #6116 with my minimal example code.
Hovering the first image:

Hovering the second image:

Can you provide own PR to fix this issue?

@Istador
Isn't this issue already fixed by #7155?

Yes, you're correct. I wrongly assumed my commit 5b30f7d34461a5bca7927ff368547abe516a8de8 would auto close this issue when being merged (I wrote too much in the first sentence of the commit message).


Fix error when output filename is an url

Fixes #16
mb_trackid contains an URL populated by some metadata source plugins (eg.
bandcamp)
use the md5 filename generation lines (that were already there for cases
where mb_trackid is not present at all) in case mb_trackid contains a slash
(/) character
could be advanced later if other unwanted characters populated by source
plugins are found in mb_trackid but should be a good enough for now fix.

Hi @adamjakab I'll try to submit some fixes that were reported. How should we proceed? I know you are a busy man but quick reviews/approvals from you would help me a lot. This one should be a no-brainer though, since @doctorfree uses it in his latest musicplayerplus release already. I think it's a perfectly fine fix.
I'll submit some more fixes and addition in the next hours/days and I'll try to keep them tiny so they are quick to review.


Python3 지원

https://github.com/namongk/django-summernote/blob/master/django_summernote/migrations/0002_update-help_text.py#L18 help_text의 타입이 byte 이기 때문에 python3 지원을 위해 새로운 마이그레이션이 필요함

Codecov Report

Merging #226 into master will increase coverage by 0.07%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master     #226      +/-   ##
==========================================
+ Coverage   96.66%   96.74%   +0.07%
==========================================
  Files          11       12       +1
  Lines         210      215       +5
==========================================
+ Hits          203      208       +5
  Misses          7        7

Impacted Files
Coverage Δ

...o_summernote/migrations/0003_auto_20170914_1311.py
100% <100%> (ø)

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 2b89450...efa8ced. Read the comment docs.

I found that it varies by Python version. If I run it on Python 2, it creates with byte literal. But on Python 3, it doesn't. Weird. :(


deposit form: enable file modifications depending on external DOi

disable "New version" button on extrenal DOI
enable file upload components on external DOI
closes https://github.com/inveniosoftware/invenio-app-rdm/issues/2259

Warning message preview:

File upload box for records with external DOI:

For the comparison: file upload box for records with our DOI (not touched):

closing as alternative approach was decided to follow


🐛 BUG: css variable in Layout component not present in autocomplete of child

Describe the Bug
When I define CSS variables in a Layout or page
<style>
	:root {
		--yellow: #FF0;
		--red: #F00;
	}
</style>

I don't get autocomplete for these in my component files.
Steps to Reproduce

npm init astro using template
Create Layout with CSS variable
Try and consume in child component
No autocomplete

Link to Minimal Reproducible Example
https://github.com/nojaf/astro-variable-repro

This is not a bug, the feature just isn't there.


community: import fix

Issue: When the third-party package is not installed, whenever we need to pip install <package> the ImportError is raised.
But sometimes, the ValueError or ModuleNotFoundError is raised. It is bad for consistency.
Change: replaced the  ValueError or ModuleNotFoundError with ImportError when we raise an error with the pip install <package> message.
Note: Ideally, we replace all try: import... except... raise ... with helper functions like import_aim or just use the existing langchain_core.utils.utils.guard_import But it would be much bigger refactoring. @baskaryan Please, advice on this.

@baskaryan Thanks!


Improve Caching in Docker builds

I've played around with this a bit, but our Docker builds are still pretty slow, especially in GitHub actions.
BuildKit has some nice features that we might be able to use to improve things. Could we have a cache of rust packages pulled in from another image use --cache-from?

There's an example here: https://docs.github.com/en/free-pro-team@latest/packages/getting-started-with-github-container-registry/migrating-to-github-container-registry-for-docker-images#updating-your-github-actions-workflow
Also, manifest-tool should be updated.

I'm willing to help out here! I think I could help speed it up, by introducing an Earthfile, which can help reproducibility and uses the baked-in Buildkit caching. We see pretty decent speed improvements on our own builds from the Buildkit cache.
Of course, as long as you and your project are open to the idea of course.

Interesting! I've not seen this before. I'm open to this idea, but I'm also reluctant to introduce more tech - it's really easy to just get started with a Dockerfile.


unknown configuration key: "os_type"

I am using packer version 0.10.0 and attempting to use this plugin. I understand that azure-arm is now built in to packer but deploying windows is not yet supported (from my understanding), which is why i'm using this plugin. I have placed the plugin files in the same directory as packer.exe and am trying to use the template here:
https://github.com/Azure/packer-azure/blob/master/config_examples/arm/WindowsServer-2012-R2.json
but I keep getting "unknown configuration key: "os_type""
I'm sure this is an error on my part but I can't seem to figure out what it is.
thanks!
Update: Let me also mention that I am using the version of the plugin downloaded from the release tab.

The plugin doesn't provide support for azure-arm as this is a builtin builder in Packer, which at this time only supports Linux
There is an azure type which this plugin provides support for, see: https://github.com/Azure/packer-azure/blob/master/config_examples/packer-azure_Win2012r2.json

Thanks!

I sent a PR to add WIndows support for Azure to the official Packer repository.  All development on ARM will move to the official Packer repository going forward.  ASM (Service Management aka SM aka the old API) will continue on in this repository.

Any idea when it will be merged? And, after it is merged, any idea how long before they'd cut a release?

Device login support. User doesn't have to create an SPN to actually use the builder.

@boumenot Is there a way in the new build to use a setting/key file (or setup oauth/api keys?) Moving forward, we're looking to automate the builds and clicking a link to login is going to cause issues here

The short answer is I don't know.
It is currently under review by the team.  I don't know when the next release will be cut, but the last two were frequent, but the ones before were not.

@liv3d Yes.  If you're going to automate the build, which is what we expect most people to do, the correct way is via an SPN.  The use of a SPN avoids any need for an interactive login.
If you just want to try out Packer on Azure then Device Login is a good way to start.  The process of creating an SPN can be daunting, and delays gratification :smile:.  We introduced Device Login support so people could actually see if Packer would fit their needs without having to create an SPN.


3.0.0 gives twice the build time as 2.*

Do you want to request a feature or report a bug?
bug

What is the current behavior?
Build time for development bundle with<EMAIL_ADDRESS>gives me an increase (double the size) in build time. The bundle was a steady 6 seconds before, now, it's increased to 12seconds +
I think the culprit is<EMAIL_ADDRESS>or<EMAIL_ADDRESS>or most likely the<EMAIL_ADDRESS>plugin
If the current behavior is a bug, please provide the steps to reproduce.

upgrade webpack to latest version
run it on current webpack config

What is the expected behavior?
Equal or reduced build time
If this is a feature request, what is motivation or use case for changing the behavior?
Please mention other relevant information such as the browser version, Node.js version, webpack version and Operating System.
<EMAIL_ADDRESS><EMAIL_ADDRESS>ubuntu@15

Please investigate to find the causing plugin and file the issue there.


Added info for loading data from MySQL to notebook

Updated both DataLoad notebooks to contain information on how to load from MySQL in Scala and Python

Code blocks are used for user input. Closing Pull Request


Fix Quick Look not closing in file upload dialog

Before submitting, please confirm you've

[x] read and understood our Contributing Guidelines
[x] completed Mattermost Contributor Agreement
[x] executed npm run lint:js for proper code formatting

Please provide the following information:
Summary
Fix Quick Look not closing in file upload dialog.
https://github.com/electron/electron/releases/tag/v1.6.2
But still there is a problem where the app craches when the dialog is canceled without closing Quick Look.
Issue link
#498
Test Cases

Open file upload dialog
Use Quick Look to preview a file.
Quick Look can be closed via close button or space bar.

Additional Notes
https://circleci.com/gh/yuya-oc/desktop/213#artifacts

Awesome, thanks Yuya!
Posted here to help with testing: https://pre-release.mattermost.com/core/pl/9rs914una7drdm7ndfd6hqstwe

@jasonblais @yuya-oc , I tested as noted in the permalink and testing steps above. Looks great! Quick Look closes as expected by clicking 'x' or pressing space bar. Other related functions still work as expected, including uploading the file after closing Quick Look. No issues found.
One question: The description says "But still there is a problem where the app crashes when the dialog is canceled without closing Quick Look." Does that mean that is a separate issue, and continues to be known/expected that the app will crash if I cancel the dialog while Quick Look is open? Because I was able to reproduce that behavior. If it's known that that is still happening and is a separate issue, then all is well with this testing.

@lindalumitchell Thank you for testing! Yeah, I think it's a separate and known issue. It's already written at the changelog.
It looks an upstream bug of electron, so I will create an issue if I could.

Thanks! This should be good to merge @yuya-oc

Thanks to confirm @jasonblais


ObjectMapper requires configuration in Kotlin?

I had a little trouble getting started with a kotlin app and receiving a post request. I followed the CRUD tutorial in order to see how to unmap a json object from a post into a Kotlin data class. That didn't seem to work out of the box, as I received the following  exception:
com.fasterxml.jackson.databind.exc.InvalidDefinitionException: Cannot construct instance of `com.foo.project.domain.Email` (no Creators, like default construct, exist): cannot deserialize from Object value (no delegate- or property-based Creator)
The Email data class was defined as:
data class Email(val recipient: String, val message: String)

And I was unpacking the data as follows:
ctx.bodyAsClass(Email::class.java)

I eventually figured that by adding an extra configuration for the ObjectMapper, I could successfully unmap the json body to a Kotlin data class:
JavalinJacksonPlugin.configure(ObjectMapper().registerModule(KotlinModule()))

Would it be possible to automatically configure Javalin to do this mapping correctly?

Welcome @JamesGuthrie, thanks for letting us know. I remember having some issues with data-classes that I "solved" by adding default values. Maybe I should have done this instead?
Either way, it should be possible to fix. The object-mapper is initialized in Jackson.kt: https://github.com/tipsy/javalin/blob/master/src/main/java/io/javalin/translator/json/Jackson.kt
I think we could replace ObjectMapper() with a new functin createObjectMapper(), which checks for the presence of KotlinModule and registers it (we could also make the mapper properly lazy).
Would you like to make a PR ?


Jit the BCD optimization solver. Fixes #6164.

What does this implement/fix?
The goal is to jit the  Block Coordinate Descent (BCD) solvers, in particula _mixed_norm_solver_bcd, using numba.

Please also add to requirements.txt

Since jitting the code does not lead to improvements in speed, I tried something else, using blas instead of np.dot. The current PR is thus useless and I close it.
The code accelerated with blas (through scipy) is proposed in a new pull request https://github.com/mne-tools/mne-python/pull/6220