⚠ Bofrost.nl has degraded performance

In 53d7d0b, Bofrost.nl (https://www.bofrost.nl) experienced degraded performance:

HTTP code: 200
Response time: 1297 ms

Resolved: Bofrost.nl performance has improved in aec4805 after 6 minutes.


Implement to_path feature for File.extname

With the added tests from Ruby Spec we're failing for File.extname because to_path isn't supported.
Consider https://github.com/danielpclark/faster_path/issues/69 as a potential resolution.

File.extname accepts an object that has a #to_path method ERROR

File.extname accepts an object that has a #to_path method FAILED

Written for refinement and monkey patch.  Not performant and advised against using but now fully compatible.


Support for computations from pre-calculated replicates

As mentioned in #34, we need a way to allow computation from pre-calculated replicates.
My original idea was to reuse the existing interface to do this, but documentation and argument types then become a bit ugly, which was also @dsaxton 's concern.
We need to resolve this before publishing release 1.0, in case it has implications for our interface overhaul (not necessarily the case, but it could have).
To address the points raised, I drafted a solution here where the interface of the bias function in the jackknife module is left as is. In addition, there is a bias_from_precalculated (name to be refined), which accepts the pre-calculated replicates. Internally, bias calls bias_from_precalculated, of course.
@dsaxton Would that be a way to go for all functions? We need to introduce X_from_precalculated then for

bias
bias_corrected
variance
confidence_interval

in both the jackknife and bootstrap modules.
This would be an acceptable solution for me, but whenever I see a common prefix/suffix, I am thinking of namespaces. I think it would be more organized to put these in a separate module, so that one can do
from resample.jackknife.precalculated import bias # the version in which you pass `theta` and `resampled_thetas`

or
from resample.jackknife import bias # the version in which you pass `fn` and `sample`

I think to make this work, we need to make jackknife and bootstrap into sub-packages, which then can have a sub-module precalculated. The directory structure would look like this.
resample
  __init__.py
  jackknife
     __init__.py
     precalculated.py
  bootstrap
     __init__.py
     precalculated.py

Still not sure how I feel about complicating the library to accommodate this, it feels a bit like over-optimization.
To me I wouldn't worry a whole lot if my computer has to do some extra work when calling resample.bootstrap.variance followed by resample.bootstrap.bias. If a user is especially concerned about reusing the replicates it's not that hard to write custom code using what we already have that persists the replicates and then does what he / she wants with them.

Still not sure how I feel about complicating the library to accommodate this, it feels a bit like over-optimization.

And for me it is difficult to understand why you don't see how essential this is.
Imagine that evaluating your estimator fn cost you 1 min. If fn is a complex maximum-likelihood fit, then this is not unrealistic. It could be more than 1 min, it could be 10, 100 min. If computing fn is costly like that, but you want to know its bias and variance, you certaintly don't want to compute 100 replicates twice.

If you think that using the formulas directly is trivial, then why write a python library in the first place. The formulas, while simple, are not trivial. The bias is computed differently for the jackknife and bootstrap. We write libraries like this so that people don't have to worry about details like that.

I would appreciate it if we start to discuss the "how" and not the "whether at all" of this, since I am 100 % convinced of the latter and I won't change my mind.

Add functions if you want, submodules seem like overkill

Sorry, for the delayed answer. Making further submodules is easy, that shouldn't stop us. It would move this functionality out of sight of those who don't care about this. Isn't this in your interest? I do agree with you that using this functionality is an advanced feature for power users, so it is fair to put it a bit deeper into the package.

precalculated could also be named formulas or something.

precalculated could also be named formulas or something.

cached could also make for an interesting name (since we are using "cached" replicates). I suppose submodules aren't too bad if they're easy to add.

@HDembinski Were you still interesting in working on this? I think it would be nice to release a new version in the not too distant future (your plan was to include this I believe).

@dsaxton I am still working on this, but if we agree on the plan laid out here, it can be done after the 1.0 release. Feel free to go ahead with the release, but please leave this open for me to work on, ok?


Rename power to repeat

What is the expected enhancement?
To apply an operator multiple times we can use the power method of the OperatorBase.
To streamline the notation with Gate and QuantumCircuit this should be change to repeat.
There exists a power method in Gate but this takes the matrix of the operation and computes the matrix power.

For power - since its the method that is called by the overidden power operator - do you really want to rename it? It will make the expression form and code seem different. E.g op ** 2  or op.power(2) today. Would a synonym be better i.e one can call power or repeat which do the same things?

I favor sticking close to the standard mathematical terms for objects and operations, when possible. Using power is probably better. But, it looks like this is a stub in any case ?
https://github.com/Qiskit/qiskit-aqua/blob/b54a4b6b42199ec9cef23384b1c55e2e4ef9d355/qiskit/aqua/operators/operator_base.py#L518-L527
Furthermore, I am in favor in general of minimal APIs with orthogonal elements. Does the advantage gained from adding an alias offer enough benefit to break this rule ?  From the issue description, I guess that the Gate (in a different repo) has a power function that is called repeat. I think power would be a better choice there. But, apparently power is reserved for a function that both converts and takes the power. I guess I'm not sure that adding repeat here would make things more clear.

The Gate has both power and repeat, where power computes the matrix representation of the gate (this class allows only unitary operations, so its possible), does matrix exponentiation and synthesizes this to a gate. repeat on the other hand, just repeats the gate several times.
For algos such as phase estimation we need to repeat circuits, gates and operators and to avoid writing special cases for each class it would be useful if they all provide the same methods. Therefore I think we should have a thing that repeats on all of them -- with the same name.
As for repeat vs power I don't have a very strong opinion, though I do see the need to differentiate between the exponentially expensive operation of computing the matrix, exponentiating it and then synthesizing it and just repeating the operation. If an implementation just repeats something, calling it repeat is probably clearer than power.

This is no longer necessary, we added power to the QuantumCircuit because even if you know an efficient way to implement the power of a circuit, repeat should just repeat. The power method however can implement a more efficient implementation if it knows one. See Qiskit/qiskit-terra#4685.


Staryu and Starmie unexpected move set

Expected Behavior
No errors
Actual Behavior
Spammed with Staryu and Starmie moveset errors
Your FULL config.json (remove your username, password, gmapkey and any other private info)
{
"auth_service": "XXX",
"username": "XXX",
"password": "XXX",
"location": "XXX",
"gmapkey": "XXX",
"encrypt_location": "",
"websocket_server": false,
"tasks": [
{
"type": "HandleSoftBan"
},
{
"type": "SleepSchedule",
"config": {
"enabled": true,
"time": "22:54",
"duration":"7:46",
"time_random_offset": "00:24",
"duration_random_offset": "00:43"
}
},
{
"type": "CollectLevelUpReward"
},
{
"type": "IncubateEggs",
"config": {
"longer_eggs_first": true
}
},
{
"type": "UpdateLiveStats",
"config": {
"enabled": false,
"min_interval": 10,
"stats": ["username", "uptime", "stardust_earned", "xp_earned", "xp_per_hour", "stops_visited"],
"terminal_log": true,
"terminal_title": true
}
},
{
"type": "TransferPokemon"
},
{
"type": "NicknamePokemon",
"config": {
"enabled": false,
"nickname_template": "{iv_pct}_{iv_ads}"
}
},
{
"type": "EvolvePokemon",
"config": {
"evolve_all": "Caterpie, Weedle, Pidgey, Rattata, Spearow, Psyduck, Poliwag, Slowpoke, Drowzee, Goldeen, Staryu, Magikarp",
"first_evolve_by": "iv",
"evolve_above_cp": 500,
"evolve_above_iv": 0.8,
"logic": "or",
"evolve_speed": 20,
"use_lucky_egg": false
}
},
{
"type": "RecycleItems",
"config": {
"min_empty_space": 15,
"max_balls_keep": 150,
"max_potions_keep": 50,
"max_berries_keep": 70,
"max_revives_keep": 70,
"item_filter": {
"Pokeball":       { "keep" : 100 },
"Potion":         { "keep" : 10 },
"Super Potion":   { "keep" : 20 },
"Hyper Potion":   { "keep" : 30 },
"Revive":         { "keep" : 30 },
"Razz Berry":     { "keep" : 100 }
}
}
},
{
"type": "CatchVisiblePokemon"
},
{
"type": "CatchLuredPokemon"
},
{
"type": "SpinFort"
},
{
"type": "MoveToFort",
"config": {
"lure_attraction": true,
"lure_max_distance": 2000
}
},
{
"type": "FollowSpiral",
"config": {
"diameter": 9,
"step_size": 70
}
}
],
"map_object_cache_time": 5,
"forts": {
"avoid_circles": true,
"max_circle_size": 100,
"cache_recent_forts": true
},
"walk_max": 4.16,
"walk_min": 2.16,
"action_wait_min": 1,
"action_wait_max": 4,
"debug": false,
"test": false,
"health_record": true,
"location_cache": true,
"distance_unit": "km",
"reconnecting_timeout": 15,
"catch_randomize_reticle_factor": 1.0,
"catch_randomize_spin_factor": 1.0,
"min_ultraball_to_keep": 10,
"logging_color": true,
"catch": {
"any": {"catch_above_cp": 0, "catch_above_iv": 0, "logic": "or"},
"// Example of always catching Rattata:": {},
"// Rattata": { "always_catch" : true }
},
"catch_throw_parameters": {
"excellent_rate": 0.1,
"great_rate": 0.5,
"nice_rate": 0.3,
"normal_rate": 0.1,
"spin_success_rate" : 0.6
},
"release": {
"any": {"release_below_cp": 0, "release_below_iv": 0, "logic": "or"},
"// Example of always releasing Rattata:": {},
"// Rattata": {"always_release": true},
"// Example of keeping 3 stronger (based on CP) Pidgey:": {},
"// Pidgey": {"keep_best_cp": 3},
"// Example of keeping 2 stronger (based on IV) Zubat:": {},
"// Zubat": {"keep_best_iv": 2},
"// Also, it is working with any": {},
"any": {"keep_best_cp": 2, "keep_best_iv": 3},
"// Example of keeping the 2 strongest (based on CP) and 3 best (based on IV) Zubat:": {},
"// Zubat": {"keep_best_cp": 2, "keep_best_iv": 3}
},
"vips" : {
"Any pokemon put here directly force to use Berry & Best Ball to capture, to secure the capture rate!": {},
"any": {"catch_above_cp": 1200, "catch_above_iv": 0.9, "logic": "or" },
"Lapras": {},
"Moltres": {},
"Zapdos": {},
"Articuno": {},
    "// S-Tier pokemons (if pokemon can be evolved into tier, list the representative)": {},
    "Mewtwo": {},
    "Dragonite": {},
    "Snorlax": {},
    "// Mew evolves to Mewtwo": {},
    "Mew": {},
    "Arcanine": {},
    "Vaporeon": {},
    "Gyarados": {},
    "Exeggutor": {},
    "Muk": {},
    "Weezing": {},
    "Flareon": {}

}

}
Output when issue occurred
2016-08-20 13:18:55,018 [IncubateEggs] [INFO] [next_egg_incubates] Next egg incubates in 1.30 km
2016-08-20 13:18:55,263 [   Pokemon] [ERROR] Unexpected moveset [Tackle, Hydro Pump] for #121 Starmie, please update info in pokemon.json and create issue/PR
2016-08-20 13:18:55,266 [   Pokemon] [ERROR] Unexpected moveset [Tackle, Power Gem] for #120 Staryu, please update info in pokemon.json and create issue/PR
2016-08-20 13:18:55,519 [   Pokemon] [ERROR] Unexpected moveset [Tackle, Hydro Pump] for #121 Starmie, please update info in pokemon.json and create issue/PR
2016-08-20 13:18:55,522 [   Pokemon] [ERROR] Unexpected moveset [Tackle, Power Gem] for #120 Staryu, please update info in pokemon.json and create issue/PR
2016-08-20 13:18:56,658 [FollowSpiral] [INFO] [position_update] Walking from (52.37387117004409, 4.868485243980463, 0) to (52.37387117004409, 4.868485243980463, 0) (0.0 m)
2016-08-20 13:18:56,675 [IncubateEggs] [INFO] [next_egg_incubates] Next egg incubates in 1.21 km
2016-08-20 13:18:56,916 [   Pokemon] [ERROR] Unexpected moveset [Tackle, Hydro Pump] for #121 Starmie, please update info in pokemon.json and create issue/PR
2016-08-20 13:18:56,920 [   Pokemon] [ERROR] Unexpected moveset [Tackle, Power Gem] for #120 Staryu, please update info in pokemon.json and create issue/PR
2016-08-20 13:18:57,407 [   Pokemon] [ERROR] Unexpected moveset [Tackle, Hydro Pump] for #121 Starmie, please update info in pokemon.json and create issue/PR
2016-08-20 13:18:57,410 [   Pokemon] [ERROR] Unexpected moveset [Tackle, Power Gem] for #120 Staryu, please update info in pokemon.json and create issue/PR
2016-08-20 13:18:57,432 [FollowSpiral] [INFO] [position_update] Walking from (52.373875908841576, 4.8685093954200935, 0) to (52.37449882666645, 4.868485243980463, 0) (69.2846995058 m)
2016-08-20 13:18:58,698 [   Pokemon] [ERROR] Unexpected moveset [Tackle, Hydro Pump] for #121 Starmie, please update info in pokemon.json and create issue/PR
2016-08-20 13:18:58,702 [   Pokemon] [ERROR] Unexpected moveset [Tackle, Power Gem] for #120 Staryu, please update info in pokemon.json and create issue/PR
2016-08-20 13:18:58,946 [   Pokemon] [ERROR] Unexpected moveset [Tackle, Hydro Pump] for #121 Starmie, please update info in pokemon.json and create issue/PR
2016-08-20 13:18:58,949 [   Pokemon] [ERROR] Unexpected moveset [Tackle, Power Gem] for #120 Staryu, please update info in pokemon.json and create issue/PR
Steps to Reproduce
Other Information
OS:
OSX 10.11.6
Branch:
Master
Git Commit:
70347a3f0b76286969ccfa1a1912958aec9df0c8
Python Version:
Python 2.7.12

I missed the other issue, sorry!


DRILL-4198: Enhance StoragePlugin interface to expose logical space r…

…ules for planning purpose
Also move Hive partition pruning rules to logical storage plugin rulesets.

@jinfengni Could you please review the patch?

+1. LGTM.


Add views for posts and users

In this milestone of the project, I added the Views associated with the Models and Controllers for the App.

Implemented the design from the sneak peek wireframes.
Executed minimal styling with focus on functionality

I made sure the app works as the given review.

I've worked on the issue with the posts controller. Here there are two posts for the first user and both of them are displayed.

Here the post with id 1 is displayed.

Below comes the post with id 2.

I've not added a post with id 3.


Implement receiver redirect support

Summary
The Event Hubs service supports redirects for AMQP receiver links in order to provide a direct connection to a partition node rather than routing through the gateway.  When the redirect is opted-into,  upon creation of a receiver link, the Event Hubs service will respond with an amqp:link:redirect error if it also supports redirects. The error details contains the information needed to create a new connection to the service.
The main benefit of receiver redirect is it allows a more direct connection to the host actually sending events to the client which should reduce latency.
Scope of Work

Implement the design for supporting receiver redirects.

Success Criteria

The design has been implemented and the targeted clients are able to opt-in to the receiver redirect.

The tests necessary for its validation have been created or adjusted and pass reliably; tests that do not focus on the extension points have been removed and no base functionality is included.

The existing live test suite continues to produce deterministic results and pass reliably.

Related Issues and References

Design for receiver redirect support (#10588)

Hi @jsquire, we deeply appreciate your input into this project. Regrettably, this issue has remained inactive for over 2 years, leading us to the decision to close it. We've implemented this policy to maintain the relevance of our issue queue and facilitate easier navigation for new contributors. If you still believe this topic requires attention, please feel free to create a new issue, referencing this one. Thank you for your understanding and ongoing support.


Suggestion: allowClear and BACKSPACE behavior change in multiple select

Hi.
If we specified allowClear option for multiple select then we'll not get clear button (as for single select). But instead BACKSPACE will be handled as clearing all values.
I think that removing all values at once on a BACKSPACE press is not very good solution. But I understand that multiple select should support allowClear in some way and it was chosen as that way. But why did you decide to not add clear button (cross) in the same way as for single select?

This is a duplicate of #3335.


fix: getFunInfo, inferType to use withAtLeastTransparency, not withTransparency

when the transparency mode is .all, then one expects getFunInfo and
inferType to also work with that transparency mode.
Fixes #5562
Fixes #2975
Fixes #2194

Ah, finally. Prefer_native helped.


errors

Hey @acalejos. I wanted to play a bit with it but when installing I saw an error:

Then I see this but I think it's due to the error on compilation:
Failed to load the widget JS module, got the following error:

    error loading dynamically imported module: http://localhost:8090/public/sessions/wzg6ynplofxwbcgljniopopbcarbgiovzyna5ktg6gzmjvqt/assets/2qoyzwmpaczaj2mabgmoz6ccpy/main.js

See the browser console for more details. If running behind an authentication proxy, make sure the /public/* routes are publicly accessible.

Hey thanks for bringing this to my attention.
I think I forgot to add the build directory to the mix.exs to tell Hex to upload it. I'll put out a patch here shortly

@dkuku Try now on the 0.2.1 patch

@acalejos It loads now 👍🏻 , thanks


Backwards compatibility out of bounds for 1.2.1

http://jenkins.mxnet-ci.amazon-ml.com/blue/organizations/jenkins/restricted-backwards-compatibility-checker/detail/restricted-backwards-compatibility-checker/380/pipeline
INFO:root:=================================

INFO:root:Fetching files for MXNet version : 1.2.1 and model lenet_gluon_hybrid_export_api

[06:36:44] src/nnvm/legacy_json_util.cc:209: Loading symbol saved by previous version v1.2.1. Attempting to upgrade...

[06:36:44] src/nnvm/legacy_json_util.cc:217: Symbol successfully upgraded!

Traceback (most recent call last):

  File "model_backwards_compat_inference.py", line 136, in <module>

    test_lenet_gluon_hybrid_imports_api()

  File "model_backwards_compat_inference.py", line 95, in test_lenet_gluon_hybrid_imports_api

    assert_almost_equal(old_inference_results.asnumpy(), output.asnumpy(), rtol=rtol_default, atol=atol_default)

  File "/work/mxnet/python/mxnet/test_utils.py", line 495, in assert_almost_equal

    raise AssertionError(msg)

AssertionError:

Items are not equal:

Error 1962.383789 exceeds tolerance rtol=0.000010, atol=0.000010.  Location of maximum error:(15, 0), a=0.061246, b=0.040821

 a: array([[ 0.03364218,  0.24863665],

       [-0.03897328,  0.28473783],

       [ 0.00993963,  0.18869999],...

 b: array([[ 0.03364221,  0.24863653],

       [-0.03897329,  0.28473788],

       [ 0.00993963,  0.18870012],...

@mxnet-label-bot add [Bug}

@marcoabreu Haven't seen this failing again. Maybe something intermittently went in that broke it and now it seems to be fixed.
http://jenkins.mxnet-ci.amazon-ml.com/job/restricted-backwards-compatibility-checker/
Can this issue be closed ?

No, MXNet yields different results for the same input, so we should track down why.

@mxnet-label-bot add [CI]

Another failure:
http://jenkins.mxnet-ci.amazon-ml.com/job/restricted-backwards-compatibility-checker/420/console
INFO:root:Fetching files for MXNet version : 1.2.1 and model lenet_gluon_save_params_api
Traceback (most recent call last):
  File "model_backwards_compat_inference.py", line 135, in <module>
    test_lenet_gluon_load_params_api()
  File "model_backwards_compat_inference.py", line 72, in test_lenet_gluon_load_params_api
    assert_almost_equal(old_inference_results.asnumpy(), output.asnumpy(), rtol=rtol_default, atol=atol_default)
  File "/work/mxnet/python/mxnet/test_utils.py", line 495, in assert_almost_equal
    raise AssertionError(msg)
AssertionError:
Items are not equal:
Error 9591.478516 exceeds tolerance rtol=0.000010, atol=0.000010.  Location of maximum error:(17, 0), a=-0.441335, b=-0.315189
 a: array([[-0.40397233, -0.19248717],
       [-0.34466907, -0.15791757],
       [-0.39881065, -0.2201823 ],...
 b: array([[-0.4039724 , -0.19248715],
       [-0.3446689 , -0.15791774],
       [-0.39881057, -0.22018239],...

Maybe this should be tracked separately as it's a different failure?

Created separate issue https://github.com/apache/incubator-mxnet/issues/14524

Probably this issue can be closed.

@mxnet-label-bot update [flaky, gluon, ci]


Introduce crt imds client.

The main goal for using crt imds client is to query ec2 instance type mountpoint-s3 running on.

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license and I agree to the terms of the Developer Certificate of Origin (DCO).

Maybe we considered this already: should we be using CRT for this or should we use a Rust-based HTTP client?
CRT of course will make things easier in terms of how to use IMDS, but it does mean we have to put thought into the safety interacting with its C code.

Also, how do you think we should test this code?

Nice work!


Migrate AdminStyles.PRIMARY_BUTTON_STYLES to ButtonStyles.SOLID_BLUE

Description
The first step towards removing button styles in AdminStyles - we want all button styling for both applicants and admins to be in ButtonStyles.
Here, we set AdminStyles.PRIMARY_BUTTON_STYLES to ButtonStyles.SOLID_BLUE. In a future PR we will remove this variable altogether.
Checklist
General

[x] Set AdminStyles.PRIMARY_BUTTON_STYLES to ButtonStyles.SOLID_BLUE
[x] Remove two unused variables in AdminStyles
[x] Update screenshots

User visible changes
Tiny changes to the existing admin buttons - see screenshots.
Issue(s) this completes
None.

Can you add any relevant labels to the PR?

Done, apologies for forgetting this repeatedly


Add onChangesCompleted callback to ModelRoot

This handler is similar to the "onChange" callback, but is invoked after all updates to the cache have finished, all output has been delivered, and all outstanding Observables have terminated with an error or completion.
@jhusain @sdesai

We are not going to do this afaik.  If I am incorrect @jhusain please feel free to re-open pr.


update Logging Interceptor to log connection failures

BEFORE
05-06 08:21:41.939 10449-11016/com.blah.blah D/OkHttp: --> GET https://somewhere.com/ http/1.1
05-06 08:22:11.205 10449-11016/com.blah.blah D/libc: [NET] android_getaddrinfofornetcontext+,hn 24(0x736f6c6f2d7175),sn(),hints(known),family 0,flags 1024, proc=com.blah.blah
05-06 08:22:11.207 10449-11016/com.blah.blah D/libc: [NET] android_getaddrinfo_proxy get netid:0
05-06 08:22:11.557 10449-11016/com.blah.blah D/libc: [NET] android_getaddrinfo_proxy-, NODATA
AFTER
05-10 09:10:42.693 17637-19160/com.blah.blah D/OkHttp: --> GET https://somewhere.com/ http/1.1
05-10 09:10:42.697 17637-19160/com.blah.blah D/libc: [NET] android_getaddrinfofornetcontext+,hn 25(0x736f6c6f2d7175),sn(),hints(known),family 0,flags 1024, proc=com.blah.blah
05-10 09:10:42.697 17637-19160/com.blah.blah D/libc: [NET] android_getaddrinfo_proxy get netid:0
05-10 09:10:43.202 17637-19160/com.blah.blah D/libc: [NET] android_getaddrinfo_proxy-, NODATA
05-10 09:10:43.203 17637-19160/com.blah.blah D/OkHttp: <-- Failed to send: java.net.UnknownHostException: Unable to resolve host "somewhere.com": No address associated with hostname

Can you add a test?

Could someone restart the Travis build?  Seems to have had a timeout installing the JDK from Oracle.  Hopefully a transient problem.

Thanks! I fixed a couple tiny nitpicks (indentation! imports!)  and tweaked your message. But otherwise this is great. I wonder if it’s worthwhile to do something similar to this if streaming the response body fails.


Fix ambiguity in LineaStateBridge

transferOwnership method that initiates ownership transfer in Linea has the same name as "normal" transferOwnership from Ownable2Step. It's better to rename it back to transferOwnershipLinea.
Don't forget to update all mentions in tests and scripts

Hey @kfastov I'd like to implement this

Hi @kfastov can I work on this?

I am applying to this issue via OnlyDust platform.
My background and how it can be leveraged
Hi, my name is Adrian, I have experience with solidity and have worked in some projects using this language
How I plan on tackling this issue
I will rename the method and check all the mentions in code making sure to rename them. Then I would run the tests to make sure everything is fine.

heyy @kfastov can i work on this?
i will...

Communicate with Your Team:
•	Notify your team about the need to rename the transferOwnership method in the Linea contract to transferOwnershipLinea to avoid conflicts with Ownable2Step.
•	Ensure they are aware that all mentions of this method in tests and scripts will need to be updated accordingly.

Update Documentation:
•	Document this change in your project’s documentation, specifying that transferOwnership in Linea has been renamed to transferOwnershipLinea.
•	Include instructions on where this change needs to be reflected (e.g., in tests, deployment scripts, etc.).

Coordinate with QA/Testing:
•	Inform the QA team to update and verify all test cases that involve transferOwnership to ensure they now refer to transferOwnershipLinea.

Schedule the Change:
•	Set a timeline or schedule for implementing this change, ensuring that all dependent code is updated simultaneously to avoid any disruption.


ThematicBreak literal is lost in the Markdown renderer

Corresponding to the specification, thematic breaks can consist of 3 or more consecutive on of characters -, _ or * with 0-3 leading spaces (i.e. regex ^\s{0,3}[-_*]{3,}$).
This logic seem to be correct when reading Markdown - the thematic breaks are correctly recognised and captured as ThematicBreak node with literal parameter containing the actual value from the Markdown.
However, during rendering the literal is dropped and replaced with static ___
Expected behaviour: this markdown looks the same after parse/render
foo

   *******

bar

---

baz

Actual behaviour: rendered transforms it into
foo

___

bar

___

baz

Yeah. MarkdownRenderer does not yet preserve everything from the input, it's main focus is on producing an equivalent document (___ is also a thematic break).
Would you want to try to make that change yourself and raise a PR? Looks like you've already found the right place where the fix should go :). I think in this case, if the node has a literal, we can use it, otherwise use ___ to not be ambiguous with lists.


Generate-sw from node module

R: @jeffposnick @addyosmani
This PR is just a re-jig of the utils and making the generateSW method work as a node module (with an ok API).
The next PR I'll get the build manifest working (Should enable anyone to play with adding it to their build process).

👍


Ignore the PATH fix if it is not set

If it is not set the fix makes no sense
Fixes #1436

In https://github.com/haskell/haskell-ide-engine/issues/1436#issuecomment-550301084 it is said that the path is set. Maybe dont merge this until it is clear why this issue arises?

In #1436 (comment) it is said that the path is set. Maybe dont merge this until it is clear why this issue arises?

Sure, but i think the actual code does what i expected in the fisrt place (return Nothing if the var is not informed)

Is catching an exception the best way to detect if the environment variable is set? This seems a bit hacky to me, but I have no experience with the System.Environment module

It was a surprise to me that lookupEnv throws an Exception instead Nothing if the variable is not set (why return Maybe String then?)
The docs  for lookupEnv states:

For POSIX users, this is equivalent to getEnv.

...and getenv trhows an IOException if the variable is not set  🤦♂

...and getenv trhows an IOException if the variable is not set 🤦♂

then the patch makes sense. Maybe we should only catch exceptions that satisfy the predicate isDoesNotExistError? Otherwise, this PR is good to go.

@jneira Merge when you are ready!


Debug Node sends when deactivated

What are the steps to reproduce?

Place Debug Node while it is deactivated.
Deploy
activate Debug Node, dont deploy.
Ctrl+Z deactivates Debug Node

What happens?
Debug Node sends incoming msgs to Debug window.
What do you expect to happen?
No msg to Debug window.
(I love the feature that you can activate a Debug Node without Deploying, makes Debugging on big projects way easier, but when you Ctrl+Z it and still get messages it gets confusing real fast)
Please tell us about your environment:
Tested on:
Debian GNU/Linux 9.11 (stretch)
Raspbian GNU/Linux 10 (buster)
Windows 10

[10.0.3] Node-RED version:
[12.13.1] Node.js version:
[6.13.1] npm version:
[Debian/Windows] Platform/OS:
[Chrome] Browser:

This is one of those lingering issues I've been aware of for a long time, but in absence of a quick fix, have left alone until someone raised it.
The underlying reason for this is that act of clicking the button does two things - it modifies the node's property (which gets put on the undo-history stack) as well as send an HTTP request to the runtime to (de)activate the node.
The undo framework in the editor only knows how to revert a node's properties - it doesn't have anyway to trigger node-specific side-effects such as sending the http request to (de)activate the node.
The fix for this will require some way of attaching custom callback events onto the events that are placed into the history stack (for both undo and redo).

This has now been fixed in the dev branch for 1.1.0. It adds the ability for a history event to have a callback that is triggered when the event is undone.


Update README.md

@uttamaniket  I do not understand this PR. It seems to include a lot more than README.md changes.

No response. Closing for now! Please reopen if it's still relevant.

Yes, I was changed more then README.md file. It  contend more then values.


Deploy API Gateway v1

TODO

[ ] Finish and merge the SKOSMOS integration (https://github.com/ts4nfdi/api-gateway/pull/5)
[ ] Create the first release of API Gateway
[x] Deploy to our Kubernetes cluster (see the deployment template here https://github.com/ts4nfdi/api-gateway-deployment)

[ ] Remove gfbio
[ ] Check/test endpoints for autocomplete with roman


FP16 accuracy issue minimal onnx example

@onnx_test()
def div_mul_bcst_fp16_test():
    channels = 32
    hw = 32
    shape = [1, channels, hw, hw, hw]
    scale = helper.make_tensor_value_info('scale', TensorProto.FLOAT16,
                                          [1, channels, 1, 1, 1])
    x = helper.make_tensor_value_info('0', TensorProto.FLOAT16, shape)
    y = helper.make_tensor_value_info('1', TensorProto.FLOAT16, shape)
    z = helper.make_tensor_value_info('2', TensorProto.FLOAT16, shape)
    div = onnx.helper.make_node('Div', inputs=['0', '1'], outputs=['div_out'])
    mul = onnx.helper.make_node('Mul',
                                inputs=['scale', 'div_out'],
                                outputs=['2'])
    return ([div, mul], [x, y, scale], [z], [])

ONNX file for this test can be generated using gen_onnx.py.
And when testing for accuracy against onnx-rt using accuracy_checker.py it fails.

If you have zeros generated in parameter 1(very likely) then there is division by zero, which you will have nans. If you insert an add with a constant before, it will pass verification even with --tolerance 1:
p = migraphx.program()
mm = p.get_main_module()
x0 = mm.add_literal(migraphx.fill_argument(migraphx.shape(type="half_type", lens=[1]), 1000))
x_0 = mm.add_parameter("0",migraphx.shape(type="half_type", lens=[1,32,32,32,32]))
x_1 = mm.add_parameter("1",migraphx.shape(type="half_type", lens=[1,32,32,32,32]))
x_scale = mm.add_parameter("scale",migraphx.shape(type="half_type", lens=[1,32,1,1,1]))
x4 = mm.add_instruction(migraphx.op("multibroadcast", out_lens=[1,32,32,32,32]), [x0])
x5 = mm.add_instruction(migraphx.op("add"), [x_1, x4])
x6 = mm.add_instruction(migraphx.op("div"), [x_0, x5])
x7 = mm.add_instruction(migraphx.op("multibroadcast", out_lens=[1,32,32,32,32]), [x_scale])
x8 = mm.add_instruction(migraphx.op("mul"), [x7, x6])
mm.add_return([x8])

I ran the above onnx file in the accuracy checker and found that it failed because of overflow or some math error resulting in an inf value in the output.
When I constrained the input range of the randomly generated numbers, I didn't see any accuracy issue even with default tolerance of 1e-3.

I ran the above onnx file in the accuracy checker and found that it failed because of overflow or some math error resulting in an inf value in the output.

The inf most likely comes from division by zero.

Original issue was closed by : https://github.com/ROCmSoftwarePlatform/AMDMIGraphX/pull/1779/files


Support for linux/arm64 wheels

Hey @jllllll I manage to figure out how to use github to build linux/arm64 wheels. Is this something that could be added to your repo?
name: Build Wheels

on: [push, pull_request]

jobs:
  build-wheels:
    runs-on: ubuntu-latest
    steps:
      - uses: actions/checkout@v4
        with:
          repository: 'abetlen/llama-cpp-python'
          ref: 'v0.2.20'
          submodules: 'recursive'
      - name: Set up QEMU
        uses: docker/setup-qemu-action@v3
        with:
          platforms: all
      - name: Set up Docker Buildx
        uses: docker/setup-buildx-action@v3
      - name: Set up Python
        uses: actions/setup-python@v4
        with:
          python-version: '3.8'
      - name: Install cibuildwheel
        run: python -m pip install cibuildwheel
      - name: Build wheels
        run: CIBW_SKIP="*musllinux* pp*" CIBW_REPAIR_WHEEL_COMMAND="" cibuildwheel --platform linux --archs "aarch64" --output-dir wheelhouse
      - name: List wheelhouse contents
        run: ls -la wheelhouse/

The whole CI takes around 21 mins to run and generates these:
14:23 llama_cpp_python-0 .2.20- cp31 0- cp310- manylinux_2_17_aarch64. whl
-rW-r- -r--
14:27 llama_cpp_python-0.2. 20-cp311-cp311 - manylinux_2__17_aarch64.whl
-rW-r- -r--
14:31 1llama_cpp_python-0.2.20- cp31 2-cp312- manylinux_2_17_aarch64. whl
14:15 llama_cpp_python-0.2. 20-cp38-cp38- manylinux_2_17_aarch64. whl
-rW-- -r--
14:191lama_cpp_python- 0. 2. 20-cp39- cp39 - manylinux_2_17_aarch64.whl

This would allow raspberry pi's and macos using linux to install wheels.

I can probably simplify it more by using the official cibuildwheel action.

While I get that it can be convenient to have pre-built wheels for ARM systems, I don't see why it would be worth adding an additional 21 minutes of build time to my 3 hours 41 minutes build process for wheels that barely anyone is going to use.
For MacOS, this is especially pointless as you are sacrificing a huge amount of performance by not taking advantage of the MacOS Metal backend that all ARM-based Macs have.
Keep in mind that these wheels would end up being CPU-only wheels that would take less than a minute to build on your own system. The only reason this repo hosts some CPU-only wheels in the first place is for some projects like text-generation-webui to use.
The primary focus for this repo is hosting pre-built wheels compiled for GPU-acceleration. The CPU-only builds are simply much easier to build as needed locally. You should ask abetlen to make wheels for Linux arm64 on the main llama-cpp-python repo.

@gaby hello, did you share the linux aarch version with llama-cpp-python team? I am trying with LLM on a linux aarch platform, please @gaby

@gaby thanks a lot ! ! I have got it in your repo


feat: add embed-only message ctor

This PR adds an embed-only constructor to dpp::message. Since message(snowflake channel_id, const embed& _embed) exists, it seemed natural and useful to have a version without the channel id.
// this is probably the most common reply on my bot
event.reply(dpp::message{}.add_embed(embed));

// now you could just do this
event.reply(dpp::message{embed});

// or this with the implicit conversion
event.reply(embed);

Code change checklist

[x] I have ensured that all methods and functions are fully documented using doxygen style comments.
[x] My code follows the coding style guide.
[x] I tested that my change works before raising the PR.
[x] I have ensured that I did not break any existing API calls.
[x] I have not built my pull request using AI, a static analysis tool or similar without any human oversight.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


Question: can this be incorporated into xESMF?

What about xESMF? xESMF is a terrific for regridding logically rectangular grids, however, it has limited support for higher-order grids like cubed-sphere grids (see here).

Are there any limitations that would prevent incorporating this into xESMF so that its support for cubed-sphere grids is expanded? Using xESMF is useful because it keeps the regridding tools all in one place.

I don't think so, but I'd welcome it. Ultimately, I don't want this to be it's own pacakge, but the fundamental limitation of xESMF is it's a wrapper around pyesmf so it's limited to what that pacakge supports, which doesn't include cubed-sphere regridding.
I have an open issue with xESMF on exactly this: https://github.com/pangeo-data/xESMF/issues/96. I'd welcome incorperating sparselt into xESMF, but as I say in that post, I don't think it fits because xESMF is based around wrapping pyesmf.

Lets continue discussion in https://github.com/pangeo-data/xESMF/issues/96 if you have ideas about inclusion.


Responsive the Image

Responsive  the Circular images section

I would like to work on this issue. Please assign me

Wow, you did a great job.


feat(angular): adding angular.io search

Adding angular io search in frontend-search plugin. Sharing some points about the reason for this pull request.

"AngularJS" now is called "Angular" and some folks contacted me talking about it. The current feature (angularjs) is appointing to angularjs.org documentation and the new one (angular) is appointing to angular.io documentation. So that everybody knows the current behavior and which one will use, based on the current framework used in their apps;
How AngularJS applications are in migration is a good approach keep both options for now;

Ping @robbyrussell

Any updates about this PR? @robbyrussell @mcornella

Oh damn, I didn't see this one. I merged #7830 instead. Sorry!

No worries, mate. I'm keeping this project outside of this repository https://github.com/willmendesneto/frontend-search
Happy to move the changes back, but not sure how long is taking to add any changes in this repository.


convert to use eth-enr library

What was wrong?
The code for handling ENR records has been extracted into a standalone library:  https://github.com/ethereum/eth-enr
How was it fixed?
Updated the local code to use the API exposed by eth-enr as well as removing the discovery v5 implementation rather than updating it (since it is slated to be replaced by https://github.com/ethereum/ddht/)
Cute Animal Picture

Waiting to merge until I've also completed the migration in https://github.com/ethereum/ddht/ which is looking like it will fuel at least a patch release in the eth-enr library.

This PR removed tests-trio/p2p-trio/test_discovery.py, which was for the discv4 implementation


Task - Comprehensive - identifier error message

When I open an existing collecting event with an identifier, I have this error message

Easiest fix, perhaps, is to state 'Identifier already exists, and won't be (re)saved.'  We need just identifier.cached, not object tag in the highlight as well I think.

Blocked by extend/embed branch.

@jlpereira we might be able to tackle this, unsure?

Yes, I think. We need to check if the existing identifier is the same loaded to avoid display the message


On click of button with transition

Hi,
We're using Gemini to run some visual regression checks on our app and it's been working great so far.
We've got a problem however with a button that has CSS on it that performs a transition between colors on click. What seems to be happening is Gemini doesn't take the screenshot at the same consistent time each run so when comparing to the original it fails as the color is not correct as it hasn't fully transitioned.
Is there any sort of workaround we can use so we can fix this?
Cheers

@ballwood Can you provide a minimally reproducible case, please? This will help to solve your problem faster.

Yeah no problem I'll try and put something together tomorrow

Hi seems it was an issue our end. Apologies!


Do "pointClick" events work with Ember?

How do you use click listeners for a stock chart? I want to get the date for a click and create a drilldown chart.

@wonderful123 It's possible with a listener and some extra code. Here is a sample of implementing this feature. You can use the same approach in Ember project. Please, download our AnyChart-Ember demo. Then replace AnyChart-Ember/tests/dummy/app/routes/simple-stock-chart.js file with modified file.
Then you can launch demo and choose 'simple-stock-chart' and try to click on the plot. You will see date and data from the clicked point in the chart title and in the browser console.

Very cool. I don't think I would have come up with all that easily!

This code is actually buggy. The returned date is depending on which side of the selection line is clicked.
https://www.screencast.com/t/hkkwrnHn7x1V
You can't see the cursor with the screenshot software but I'm clicking either side of the selection line.

@wonderful123
This may come from different dateTime formats (UTC and using timeZone). Do you set your data as UTC dateTime? Do you define any dateTime formats in your project?
Javascript methodes use timeZone offset, sometimes it brings troubles.

Oh I think you're right. I'm at 12pm for each time. Thanks very much!


Change CoreOS to Flatcar Linux in production install

What type of PR is this?
/kind content
/area documentation
What this PR does / why we need it:
CoreOS the way we knew is deprecated :) The de-facto replacement is Flatcar Linux.
Also, I've removed the binding from toolbox, as it generates errors (toolbox already mounts everything)
As a followup, it would be good to remove the '-q' from yum install script, as this leaves anxious users (as me) wondering if something wrong happened, when it's just my internet that is slow :P

Thanks for your pull request. Before we can look at it, you'll need to add a 'DCO signoff' to your commits.
:memo: Please follow instructions in the contributing guide to update your commits with the DCO
Full details of the Developer Certificate of Origin can be found at developercertificate.org.
The list of commits missing DCO signoff:

ed67a11 Change CoreOS to Flatcar Linux

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To complete the pull request process, please assign lucperkins after the PR has been reviewed.
You can assign the PR to them by writing /assign @lucperkins in a comment when ready.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

Welcome @rikatz! It looks like this is your first PR to falcosecurity/falco-website 🎉

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


Delete files on google cloud and localy before uploading/dowloading

The merge request #180 fix this bug better... So i will close this issue


[MPLUGIN-492] Documentation for plugins in general: Goals comprises m…

…ore than that
This closes #252

@Reissner have a look.

I hope that translations are ok 😄

It is, hand-crafted, native level.


Tweaks to manage script

Signed-off-by: Emiliano Suñé<EMAIL_ADDRESS>

Integrated the updates into https://github.com/bcgov/a2a-trust-over-ip-configurations/pull/56, and fixed some additional bugs in both accepttaa and registerSovrin commands.


Update tedivm/jshrink to upstream version 1.3.2

Apply all the changes in tedivm/jshrink between versions 1.1.0 and 1.3.2.
Related discussion:
https://github.com/concrete5/concrete5/issues/7907

Did you try to apply both jshrink patches (the one currently in this repo and your one)? I think that they are conflicting...

@mlocati Yes, I ran composer install against this patched version. Here is how the previous patche's lines look like after composer install:

@mlocati Also note that your patch is applied first which is why I think it is working properly.

I think that we may discard the patch currently present in this repository, and keep just yours, but merging into it the two changes of that patch.

Sure. I think they are actually already included since they come from this commit:
https://github.com/tedious/JShrink/commit/91105810dafedba0390608d7465abd602beb6410
That is included in the patch.

That is included in the patch.

You are right, I just read the first patch. So, I'd just drop the current patch

So, I'd just drop the current patch

Done already at e6c779790e1ea591bfa458e4e2b4b598a87e1c64.

I tried this with an installation where we already had the previous patch applied.
Here's the relevant parts:
$ composer update concrete5/dependency-patches

  - Updating concrete5/dependency-patches (1.2.0 => dev-master 0f13eb6):  Checking out 0f13eb61ea

Applying patch concrete5/dependency-patches/Update to upstream version 1.3.2 to tedivm/jshrink... patch was already applied.

So I think we have to do something to "revert" the previous patch, maybe splitting it in two, so that the "reverting" patch may fail (when the previous patch isn't already applied). In any case, the second one (the one in this PR) should be executed by composer-patcher

I think it is just loading it from some cache where you have already applied the patch.
I get the exact same message when I try the update to the concrete5 main repository by pulling the dependency from this git repository.
I don't completely understand how those patches are applied in the background (haven't looked into it) but it seems to be working correctly.

To test:
Clone the c5 repository:
$ git clone git@github.com:concrete5/concrete5.git c5dptest

Run composer install:
$ cd c5dptest
$ composer install

Add the following lines in the composer.json at the root of the project:
  "repositories": [
    {
      "type": "vcs",
      "url": "https://github.com/concrete5/dependency-patches.git"
    }
  ],

And change the following line:
  "require": {
    ...
    "concrete5/dependency-patches": "dev-master"
  },

Run composer update for the dependency patches package:
$ composer update concrete5/dependency-patches

Now, check the file at vendor/tedivm/jshrink/src/JShrink/JShrink.php.
It should have all the patches applied.
I was particularly looking for the changes done here:
https://github.com/tedious/JShrink/pull/84/files

I don't completely understand how those patches are applied in the background (haven't looked into it)

They simply run a command like this:
patch -p1 --no-backup-if-mismatch -N -d "path/to/vendor/tedivm/jshrink" -i "path/to/file.patch"

Yeah, hard to say where that message is coming from. I also get the patch was already applied message from all the current patches when I run them against a clean cache directory.
To verify, you can change the local cache directory with the test provided above before running the first composer install:
$ mkdir .composer-cache
$ composer config cache-dir "$PWD/.composer-cache"

After this, copy the current Minifier.php version for reference (before updating the dependency to this repository):
$ cp concrete/vendor/tedivm/jshrink/src/JShrink/Minifier.php Minifier-orig.php

Then run the rest of the test and finally compare the Minifier.php after applying the latest patch:
$ diff Minifier-orig.php concrete/vendor/tedivm/jshrink/src/JShrink/Minifier.php

You can see that it has changed regardless of the message.

With https://github.com/concrete5/dependency-patches/pull/4, starting from the state after dependency-patches v1.3.0 applied its patches, we have:
Applying patch concrete5/dependency-patches/Fix continue switch in FileGenerator and MethodReflection to tedivm/jshrink... patch was already applied.
Applying patch concrete5/dependency-patches/Update to upstream version 1.3.2 to tedivm/jshrink... done.

(and BTW we should really change that "Fix continue switch in FileGenerator and MethodReflection" description, it was a mistake of mine)

@ahukkanen maybe we are using two different version of the patch command (I'm using the one that comes with git for Windows).
What do you have now with the current master of this repo?

@mlocati When pulling from master I now get this when updating concrete5/dependency-patches with a clean composer cache:
Applying patch concrete5/dependency-patches/Fix continue switch in Minifier to tedivm/jshrink... patch was already applied.
Applying patch concrete5/dependency-patches/Update to upstream version 1.3.2 to tedivm/jshrink... done.

See https://github.com/concrete5/concrete5/pull/7930


Reimplement \spell macro

This is an an alternative implementation of \spell (#40) that uses a description environment. Description items can contain multiple paragraphs.
This changeset also sets the default serif font family to Palatino. description environments use their own font families (i.e., Computer Modern). I figured setting Palatino as the document default made more sense than hard-coding it inside the macro.

Thanks for your contributions coming so quickly! A couple questions:

I was under the understanding that cmr was not being used at all in the document and bookman was the closest we could find, with lmss substituted in when we need bold small caps.  Someone who wants to merge this change into their branch but does not want the font change is going to have to modify their branch after the merge. The font change should probably be split out into a separate pull request to be debated there. In the meantime, if description is using a built in font we need to figure out how to make that match what the rest of the document is using. Is it possible to set /normalfont/selectfont where necessary? LaTeX fonts are not my specialty. So I am asking questions here partially out of ignorance.
This solution allows line breaks in the spell description, but it does not actually allow paragraphs (by inserting a blank line) in the spell description. This is the same issue as #40. As I was going through that, I realized that the problem is easily sidestepped. The paragraphs of a spell are typeset exactly the same way as a normal paragraph. The \spell macro therefore only has to worry about everything other than the description. I like the use of the description environment with the hanging indent.
Overall, I like the formatting, but the use of \vspace{-5pt} concerns me. I know that it is in the definition of \subtitlesection as well, but I would like to figure out how to modify both of these so there is no need for hard-coded adjustment lengths in points. I do not think they would hold up well if the spacing of subsubsection was modified.
As a user, the less typing I need to do, the better. While I can always copy and past spell definitions, it would be nice to just skip having to type "type=", "duration=", etc. Every spell that I can see always includes the all the different elements of the header. So is there a way to just define this with \newcommand instead of \newkeycommand? The problem I ran into when I tried that is some of the parameters inexplicable moved down in the typesetting. So range was blank and the range parameter was typeset under the "Components:" heading. It was really weird, and I would appreciate any insight anyone has into the reason why that happens.

I was under the understanding that cmr was not being used at all in the document and bookman was the closest we could find, with lmss substituted in when we need bold small caps.

The example document uses Palatino, but I see that was introduced as a side-effect in #1. I'll remove the font change.
You can disregard the CM note. I was confused, the description environment was using the style default (Bookman), not CM.

This solution allows line breaks in the spell description, but it does not actually allow paragraphs (by inserting a blank line) in the spell description. This is the same issue as #40. As I was going through that, I realized that the problem is easily sidestepped.

Shoot, the paragraph indenting works with \RaggedRight but not without it. I'll see what I can do. I would prefer to keep all the spell information in the same command.

Overall, I like the formatting, but the use of \vspace{-5pt} concerns me. I know that it is in the definition of \subtitlesection as well, but I would like to figure out how to modify both of these so there is no need for hard-coded adjustment lengths in points. I do not think they would hold up well if the spacing of subsubsection was modified.

Absolutely agree; that 5pt should be 1ex in both places. It's just a coincidence that it works right now with the general font size 10 pt.

As a user, the less typing I need to do, the better. While I can always copy and past spell definitions, it would be nice to just skip having to type "type=", "duration=", etc. Every spell that I can see always includes the all the different elements of the header. So is there a way to just define this with \newcommand instead of \newkeycommand?

Yep, I'll give this a shot.

Here we are. Switching to \newcommand magically cured the paragraph problem.

Sorry to keep asking questions as you are refining this, but seeing what you are producing is causing new ideas and questions to pop up. One of the reasons I was thinking that it might make sense for \spell to only handle the header is that the description can sometimes be a full page long and include environments such as tables. The final curly brace could sometimes get a bit lost around the description, and I do not know how parameters of newcommand handle environments nested inside them.
Alternatively, would it make more sense for \spell to be an environment? Could the header information be mandatory parameters, with the spell description in between \begin and \end? This might make it easier to sight the beginning and end of a spell definition.
%Spell definition
\newenvironment{spell}[6]
    {\subtitlesection{#1}{#2}\vspace{-1ex} % subtitlesection artefact; topsep also changes bottom spacing.
        \begin{description}[font=\normalfont\textbf,noitemsep,topsep=1ex,leftmargin=1em]
            \item[Casting Time:] #3
            \item[Range:] #4
            \item[Components:] #5
            \item[Duration:] #6
        \end{description}
    }
% Description goes here
    {
    }

Which would then be used by:
\begin{spell}
    {Beautiful Typesetting}
    {4th-level illusion}
    {1 action}
    {5 feet}
    {S, M (ink and parchment, which the spell consumes)}
    {Until dispelled}
    You are able to transform a written message of any length into a beautiful scroll.

    All creatures within range that can see the scroll must make a wisdom saving throw or be charmed by you until the spell ends.

    While the creature is charmed by you, they cannot take their eyes off the scroll and cannot willingly move away from the scroll.

    Also, the targets can make a wisdom saving throw at the end of each of their turns. On a success, they are no longer charmed.
\end{spell}

What does everyone think?

I was actually going to suggest using an environment for the same reason (I re-read Wish yesterday evening).
Here it is. \spellsummary exists if someone needs it.


Added Object Tracking feature.

This pull request is to include Milestone 3's Object Tracking feature that integrates to the EPD codebase as well as introducing hot fixes for unreported issues.
The edits are:

Updated documentation for Tracking Use Case Configuration.
Updated Python-based GUI for Tracking Use Case Configuration.
Introduced hot fix for Anaconda environments in setting up multiple EPD package on the same workstation.
Integrated Tracking feature as extension of p3_ort_base's Precision Level 3 Localization feature.

This pull request serves as easy_perception_deployment (EPD) 's full release.
@AlanXue-ROSI @Briancbn @tanjpg

@AlanXue-ROSI @Briancbn @tanjpg
Public CI failure has been rectified. Ready to merge Full-Release variant.

Codecov Report

Merging #20 (7395e21) into master (22e6a15) will decrease coverage by 8.68%.
The diff coverage is 63.90%.

:exclamation: Current head 7395e21 differs from pull request most recent head 095ac89. Consider uploading reports for the commit 095ac89 to get more accurate results

@@            Coverage Diff             @@
##           master      #20      +/-   ##
==========================================
- Coverage   89.19%   80.50%   -8.69%
==========================================
  Files          12       12
  Lines        1249     1795     +546
==========================================
+ Hits         1114     1445     +331
- Misses        135      350     +215

Impacted Files
Coverage Δ

...eployment/include/epd_utils_lib/usecase_config.hpp
98.91% <0.00%> (ø)

...ion_deployment/include/ort_cpp_lib/p3_ort_base.hpp
100.00% <ø> (ø)

...ion_deployment/include/epd_utils_lib/processor.hpp
33.76% <3.65%> (-16.57%)
:arrow_down:

...ion_deployment/include/ort_cpp_lib/p3_ort_base.cpp
82.30% <72.92%> (-11.35%)
:arrow_down:

...deployment/include/epd_utils_lib/epd_container.cpp
95.10% <88.88%> (-0.46%)
:arrow_down:

...deployment/include/epd_utils_lib/message_utils.hpp
100.00% <100.00%> (ø)

Continue to review full report at Codecov.

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


feat(index): add optional schemaName support

Closes #8

🍪  Really glad this was a quick fix for the schema problem! 👏


Feature/example project

Updated the Obj-C Project to Swift
@Haud @jlmendezbonini

Looks good to me. Lets wait for @Haud blessing, but in the mean time you can start with some of the issues: https://github.com/prolificinteractive/PIDatePicker/issues

Looks good!


LoadError: cannot load such file -- java

I'm building Mirah from source as described in the documentation:
$ git clone http://github.com/mirah/mirah.git
$ cd mirah
$ bundle install
$ rake gem
$ gem install pkg/mirah-*.gem

But when I run rake gem I get the following error:
rake aborted!
LoadError: cannot load such file -- java
/home/tkeen9/mirah/Rakefile:24:in `require'
/home/tkeen9/mirah/Rakefile:24:in `<top (required)>'

When run with the --trace parameter there is more info:
rake aborted!
LoadError: cannot load such file -- java
/home/tkeen9/mirah/Rakefile:24:in `require'
/home/tkeen9/mirah/Rakefile:24:in `<top (required)>'
/var/lib/gems/1.9.1/gems/rake-10.4.2/lib/rake/rake_module.rb:28:in `load'
/var/lib/gems/1.9.1/gems/rake-10.4.2/lib/rake/rake_module.rb:28:in `load_rakefile'
/var/lib/gems/1.9.1/gems/rake-10.4.2/lib/rake/application.rb:689:in `raw_load_rakefile'
/var/lib/gems/1.9.1/gems/rake-10.4.2/lib/rake/application.rb:94:in `block in load_rakefile'
/var/lib/gems/1.9.1/gems/rake-10.4.2/lib/rake/application.rb:176:in `standard_exception_handling'
/var/lib/gems/1.9.1/gems/rake-10.4.2/lib/rake/application.rb:93:in `load_rakefile'
/var/lib/gems/1.9.1/gems/rake-10.4.2/lib/rake/application.rb:77:in `block in run'
/var/lib/gems/1.9.1/gems/rake-10.4.2/lib/rake/application.rb:176:in `standard_exception_handling'
/var/lib/gems/1.9.1/gems/rake-10.4.2/lib/rake/application.rb:75:in `run'
/var/lib/gems/1.9.1/gems/rake-10.4.2/bin/rake:33:in `<top (required)>'
/usr/local/bin/rake:23:in `load'
/usr/local/bin/rake:23:in `<main>'

Solved by installing the latest JRuby version.

Glad you figured it out. Do you think the docs could be updated to make
that less likely?
On May 1, 2015 4:30 PM, "tkeen9"<EMAIL_ADDRESS>wrote:

Solved by installing the latest JRuby version.
—
Reply to this email directly or view it on GitHub
https://github.com/mirah/mirah/issues/315#issuecomment-98255573.

@baroquebobcat Yep, I think so. I'm using Debian, but any other distribution based on Debian like Ubuntu install older JRuby versions, namely 1.5. I had to build JRuby 1.7 (the latest as of now) from source.


Support recent urllib3 release

https://github.com/neuropacs/neuropacs-py-api/blob/8a1206741a24572449cceee41399864aa7524747/setup.py#L18
It appears that in https://github.com/neuropacs/neuropacs-py-api/commit/b45120a4af393b47e41965a1b34c3e2ec08a2825, an upper pin was added to urllib3 to 1.26.6 released June 25th 2021. Is there a reason why I couldn’t continue to use urllib3 version 1.26.20 or any other future 1.26.x release? https://github.com/urllib3/urllib3/releases/tag/1.26.20
urllib3<2?
Is it possible to use urllib3 version 2? Is there a known compatibility issue with this library? The migration guide is available at:
https://urllib3.readthedocs.io/en/stable/v2-migration-guide.html

@neuropacman Reviewing the latest commit on main, this upper pin still exists. See below.
https://github.com/neuropacs/neuropacs-py-api/blob/19f4332a7a1a7005b823ed50d689c5e12b03a002/setup.py#L20
What is your intention by closing this issue as "completed" now? Is this actually a wont-fix indication?

@jamesobutler Closing this issue was a mistake. I have reopened it.
I will attempt to migrate to urllib3 version 2 this week. I believe this version was upper pinned due to compatibility issues in early development.
Thank you for catching this.

@jamesobutler
The upper pin of urllib3 has been removed.
https://github.com/neuropacs/neuropacs-py-api/commit/7286a2f4c311395c48ea8bb1ba9964582a72d2cc#diff-60f61ab7a8d1910d86d9fda2261620314edcae5894d5aaa236b821c7256badd7L20 (ignore the typo for 'author')
This was originally added due to a compatibility issue between urllib3>=2.0 and system Python on macos in which this API was originally developed. But as of recent, we use a newer version of Python making this obsolete.
Hence, there is no longer a need to pin urllib3.


Remove resource_name tag from webhook stats

Proposed in relation to https://github.com/tektoncd/pipeline/issues/2872.
Motivation: Common use cases for this webhook involve using Kubernetes's generateName API to randomise resource names (this is a good idea in Tekton pipelines, for example, where there are uniqueness constraints. That means that the webhook metrics here end up with very high cardinality, which makes Prometheus fall over. Even without generateName, it is possible to shoot oneself in the foot this way.
This commit just removes the resource_name label altogether.

/ok-to-test

Would converting this to report actual different opencensus.Resource values (rather than tags) solve the issue, or is the issue that the cardinality of resource_name is so high that it's not actually a good candidate for a metric.
I kind of suspect the latter -- right now it seems like we could be reporting a metric for every single custom object that goes through the webhook, which could be thousands of names...

is the issue that the cardinality of resource_name is so high that it's not actually a good candidate for a metric.

In my experience running tekton, resource_name is unambiguously a metric of unworkable cardinality. It is possible that this is the tekton folks 'doing it wrong' by randomising resource names, but if they are, I expect many others are too.

@evankanderson any advance on this? Just got indirectly reminded of its existence ...
I understand if there's more serious reorganisation of this metrics code afoot, but this change should be orthogonal to all that and will disarm a big old footgun.

/lgtm
/approve
/hold
I think you have to remove the "draft" bit and we need to make sure the CLA bot is happy with you 😉

@tragiclifestories have you filled out the CLA? Details here: https://knative.dev/community/contributing/contributing/#contributor-license-agreements

/assign @dprotaso

/check-cla

bump @tragiclifestories - were you able to sign the CLA?

Given it's been more than a month, I'd vote to just re-do the work.

@dprotaso just signed it now - though I thought I already had for some Tekton thing long ago.

/check-cla

though I thought I already had for some Tekton thing long ago.

The Tekton CLA is distinct from the Google CLA

@googlebot rescan

🎉

/hold cancel

somanyclas

/retest

though I thought I already had for some Tekton thing long ago.

The Tekton CLA is distinct from the Google CLA

It didn't use to be, though - I remember because there was a hassle about signing the Google one with work ...

I think the latest unit failure is legit
/retest

or not - weird

runs away


using opencv 2.4.9

HI, thank you for you sources.
I am working with opencv now, but I don't like the opencv3.0 and some of the library I want to use is not included in your source. How can I Use OpenCV 2.4.9 with your sources?
Thank you

I want to use  "opencv2/xfeatures2d.hpp". But I cant find it in the library you created.

Hi,
Unfortunately, CV Drone does not provide opencv_contrib module.
You should build OpenCV 2.4.9 and opencv_contrib by yourself.
This is the last version which uses OpenCV 2.4.9.
https://github.com/puku0x/cvdrone/tree/5e6ffc21e6e6b9b79d632e2991e1270e7f611f02
I'm not sure which environment are you using but this will help you how to set the path of libraries.
Thank you,
puku


ROX-18537: remove unused ocp core bpf qa e2e tests

Description
With changes to the testing, the ocp-qa-e2e-tests were switched to using core_bpf. Therefore, the ocp-core-bpf-qa-e2e-tests job was removed and its associated script is no longer needed. See https://github.com/openshift/release/pull/41738
Checklist

[ ] Investigated and inspected CI test results

Testing Performed
CI is sufficient

/test ocp-4-10-operator-e2e-tests
/test ocp-4-13-qa-e2e-tests
/test ocp-4-13-operator-e2e-tests
/test ocp-4-13-ebpf-qa-e2e-tests
/test ocp-4-10-ebpf-qa-e2e-tests


Updated keyboardcleantool to use HTTPS

Thank you for the contribution. It was merged directly as eee2cbbe9ab4e95bb45b906e1346b4356202b19b to keep commit history cleaner. Your contribution is still credited to you.


[Documentation Update]: Replace NPM install with bun i for setting up test cases.

Is there an existing issue for this?

[X] I have searched the existing issues

Issue Description
IN- Documentation for test inside store folder it is told to use NPM for installing dependencies where as the project uses bun to install dependencies
Suggested Change
IN readme.md inside frontend/lib/store/tests NPM is being used to install dependencies. Replace it with bun.
Rationale
To use bun instead of NPM for installing dependencies.
Urgency
Medium
Record

[X] I have read the Contributing Guidelines
[X] I have starred the repository

Can you please assign me this issue.

@Dedsec933 assigned.


Endpoint to show all the routes and their details

What bug or feature you wants to report.
No response

Do it go ahead @Kuldeep2001 ! ;)

Its already there, sorry I assigned you with this! You can raise another and build a new feature!


mill: 0.4.1 -> 0.5.0

Motivation for this change
Upgrade mill to latest release
Things done

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

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

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

cc @scalavision

@GrahamcOfBorg build mill


Authrep for OAuth function is missing

In the README it mentions there is no authrep call for OAuth. This call is actually documented as can be seen here.
It would be great to have a function to make this request which would also help @picsoung extend the lambda functions used in the 3scale authorizer plugin.
cc @avilatusell

Work in progress in [PR#46] (https://github.com/3scale/3scale_ws_api_for_nodejs/pull/46)

PR #46 merged


regexp correction:

email type function now matches for multiemail styles for [email name]

i get it, it will be merge soon.


Support playwright

Integration with Microsoft's Playwright
Issue: https://github.com/Visual-Regression-Tracker/Visual-Regression-Tracker/issues/129

Tests still coming

Tests done now. This PR only covers the synchronous API. I'll do the async work later.

Note, code coverage results are affected by: https://github.com/microsoft/playwright-python/issues/313

I've added async support now.

@pashidlos I think its finally ready for review

I'm on it!

not sure if I'm doing it right
but I'm getting error
Traceback (most recent call last):
  File "example_playwright.py", line 25, in <module>
    vrt.trackPage(page, 'Home page')
  File "/Users/pashidlos/work/vrt/sdk-python/visual_regression_tracker/playwright.py", line 75, in trackPage
    self.track(TestRun(
  File "/Users/pashidlos/work/vrt/sdk-python/visual_regression_tracker/visualRegressionTracker.py", line 92, in track
    result = self._submitTestResult(test)
  File "/Users/pashidlos/work/vrt/sdk-python/visual_regression_tracker/visualRegressionTracker.py", line 80, in _submitTestResult
    result = _http_request(
  File "/Users/pashidlos/work/vrt/sdk-python/visual_regression_tracker/visualRegressionTracker.py", line 111, in _http_request
    response = request(url, json=data, headers=headers)
  File "/Users/pashidlos/.conda/envs/sdk-python/lib/python3.8/site-packages/requests/api.py", line 119, in post
    return request('post', url, data=data, json=json, **kwargs)
  File "/Users/pashidlos/.conda/envs/sdk-python/lib/python3.8/site-packages/requests/api.py", line 61, in request
    return session.request(method=method, url=url, **kwargs)
  File "/Users/pashidlos/.conda/envs/sdk-python/lib/python3.8/site-packages/requests/sessions.py", line 516, in request
    prep = self.prepare_request(req)
  File "/Users/pashidlos/.conda/envs/sdk-python/lib/python3.8/site-packages/requests/sessions.py", line 449, in prepare_request
    p.prepare(
  File "/Users/pashidlos/.conda/envs/sdk-python/lib/python3.8/site-packages/requests/models.py", line 317, in prepare
    self.prepare_body(data, files, json)
  File "/Users/pashidlos/.conda/envs/sdk-python/lib/python3.8/site-packages/requests/models.py", line 467, in prepare_body
    body = complexjson.dumps(json)
  File "/Users/pashidlos/.conda/envs/sdk-python/lib/python3.8/json/__init__.py", line 231, in dumps
    return _default_encoder.encode(obj)
  File "/Users/pashidlos/.conda/envs/sdk-python/lib/python3.8/json/encoder.py", line 199, in encode
    chunks = self.iterencode(o, _one_shot=True)
  File "/Users/pashidlos/.conda/envs/sdk-python/lib/python3.8/json/encoder.py", line 257, in iterencode
    return _iterencode(o, 0)
  File "/Users/pashidlos/.conda/envs/sdk-python/lib/python3.8/json/encoder.py", line 179, in default
    raise TypeError(f'Object of type {o.__class__.__name__} '
TypeError: Object of type bytes is not JSON serializable

the code I'm executing
from playwright import sync_playwright
from visual_regression_tracker import Config, TestRun
from visual_regression_tracker.playwright import PlaywrightVisualRegressionTracker

config = Config(
    apiUrl='http://localhost:4200',
    project='Default project',
    apiKey='0TK0P0NQP6MNFQQPTYYBN27JRAA5',
    branchName='develop',
    enableSoftAssert=False,
)

playwright = sync_playwright().start()
browserType = playwright.chromium
browser = browserType.launch(headless=False)
page = browser.newPage()
page.goto('https://www.python.org/')

vrt = PlaywrightVisualRegressionTracker(config, browserType)

with vrt:
    try:
        vrt.trackPage(page, 'Home page')
    finally:
        playwright.stop()

released in 4.6.0
https://twitter.com/VisualReTracker/status/1331639937259745284


Prevent from using illegal patterns as conditions

The types do not prevent from using certain illegal patterns as conditions: any, local, var, etc.  For instance, this is a legal pattern:
any `with` any

It would be good to rule out such patterns at the level of types.

Currently, condition any will simply fail, as it is not "closeable".  Maybe ruling this out at the level of types is not so important.  The general rule is that, if a pattern is not closeable, it fails as a condition.
Condition var x makes sense, especially if x refers to a Boolean value (which can be type checked).  For instance:
any `with` var x

Using local x as a condition doesn't make sense.  It should be only used in letIn expressions, but we do not type check that at the moment.  This is therefore a different issue, and both local and letIn are experimental.


python3Packages.black: disable all tests on aarch64-linux

For now at least.  I'm tired of this channel-blocking chase:
https://github.com/NixOS/nixpkgs/pull/176991#issuecomment-1150736907

@ofBorg build black


Emit metadata instead of sessionid to magellan

Emit metadata to expose more information to executors.

Codecov Report

Merging #78 into master will decrease coverage by <.01%.
The diff coverage is n/a.

@@            Coverage Diff            @@
##           master     #78      +/-   ##
=========================================
- Coverage   98.81%   98.8%   -0.01%
=========================================
  Files          21      21
  Lines         927     924       -3
  Branches      153     153
=========================================
- Hits          916     913       -3
  Misses         11      11

Impacted Files
Coverage Δ

lib/worker/magellan.js
88.23% <0%> (-1.77%)
:arrow_down:

lib/base-test-class.js
100% <0%> (ø)
: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 de52c5b...4e9fe2e. Read the comment docs.


EVEREST-1087 OIDC config

EVEREST-1087
What's new:

Added the CLI oidc configure command.
Usage:  everestctl oidc configure --issuer-url=http://url.com --client-id=034rkslld

Added the everest-settings ConfigMap to store the Everest settings, including the OIDC config

% kubectl get configmap/everest-settings -o yaml -n everest-system
apiVersion: v1
data:
  oidcClientId: 034rkslld
  oidcIssuerUrl: http://url.com
kind: ConfigMap
metadata:
  creationTimestamp: "2024-05-27T11:04:19Z"
  name: everest-settings
  namespace: everest-system
  resourceVersion: "19619"
  uid: de6da5e3-11cd-4a99-b77c-aa17d9e7a8a6

Out of scope:
An API method to get the Everest Settings. Coming in a separate PR.

@mayankshah1607 regarding the comment:

what do you think about grouping together all the OIDC settings

It was my first idea too, but then during the implementation I realised that the grouping makes it much more cumbersome to read and write the actual values bc of the format conversion but doesn't bing that much of a value. So for the sake of keeping things simple I used the flat values.
Let's check for more opinions on the topic, @recharte what data format do you think it would be better to use in the settings ConfigMap?

@oksana-grishchenko I agree that with the grouping, we now need to manipulate a YAML string, so it becomes a bit more tricky. But I think that with a little bit of of code refactoring, we can reduce the complexity.
For example, we're dealing with the same level of complexity with the Everest users Secrets as well, where we need to manipulate a YAML string, but I tried to organise the code such that its not too hard to deal with: https://github.com/percona/everest/blob/main/pkg/kubernetes/client/accounts/accounts.go#L122

Let's check for more opinions on the topic, @recharte what data format do you think it would be better to use in the settings ConfigMap?

Looking at the CM alone I do prefer the grouping like Mayank was suggesting. @oksana-grishchenko What do you think about Mayank's suggestion of processing it similarly to what we did with the accounts secret? Do you still think it's too complex?


Can i use svg loader？

This tool really works well. Do it now support SVG loader?

I want to use it in antd mobile

Your given the config object so for the most part you can alter however you need.

The official example is simple. Can you give an example of SVG configuration?Thank you very much

@walkinMrwang if you're referring to SVG React Loader I just wrote a rewire for it:
https://github.com/codebandits/react-app-rewire-svg-react-loader


[stable/datadog] fix several reported issues

What this PR does / why we need it:
This PR fixes severals reported issues
Which issue this PR fixes

fixes #21806, #21920, #21991, #21853, #21663, #21622
includes: #21656

Special notes for your reviewer:
Checklist
[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

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

/assign @CharlyF

/lgtm


luci-protocols-openconnect: add an option for "--no-cert-check"

What title says, instead of editing /usr/sbin/openconnect-wrapper and adding it there.

No action for years; closing


Unable to pass memlevel compression parameter while creating gzip files

While creating gzip files with this module I'm unable to use the memlevel compression parameter. I've only see the compression level param but not the memory compression level param.
The GZip class used in Pack.js allow to pass level and memlevel params. The memlevel param allow to get a smaller gzipped file.
There's a way to do that?

tar.create({
  gzip: {
    memLevel: 9
  },
  file: 'files.tgz'
}, [file, file, file])

Note that the default memLevel is 8 because there are bugs in some versions of zlib with unzipping memLevel 9 compression, making memLevel 9 less portable.

Your code works (this is mine):
...
let _aFileList = fs.readdirSync(_sSource);
tar.create(
    {
	    gzip: {
		    level: 9,
		    memlevel: 9
	    },
	    file: _sTarget,
	    cwd: _sSource,
	    prefix: '/./'
    },
    _aFileList,
    function(err){
	    if(err){
		    return fReject();
	    }
	    else{
		    return fResolve();
	    }
    }
    );
...

but with this code I can get a smaller gzipped file:
...
let _aFileList = fs.readdirSync(_sSource);
let _sTarTarget = _sTarget+'.tmp';
tar.create(
	{
		file: _sTarTarget,
		cwd: _sSource,
		prefix: '/./'
	},
	_aFileList,
	function(err){
		if(err){
			return fReject();
		}
		else{
			let input = fs.createReadStream(_sTarTarget);
			let output = fs.createWriteStream(_sTarget);
			input.pipe(zlib.createGzip({level:9, memLevel:9})).pipe(output).on('finish', function(){
				fs.unlink(_sTarTarget, function(){
					return fResolve();
				})
			});
		}
	}
);
...```


Bug: Create and Validate w Collections causes error

Summary
User receives an error message when selecting "create and validate" on a new bulkrax csv importer.
The error is valid because "create and validate" does not import anything. Therefore, when the code checks for the collection's id, it is correct that it does not exist.
The code should not be checking for the existence of a collection at this point, when "create and validate" is chosen.
Current bulkrax version: 3.0.0beta5
Acceptance Criteria

[ ] selecting "create and validate" on a csv bulkrax importer should not throw an error.
Csv should be set up to create works and collections the 3.0+ way.

Screenshots or Video
URL (Staging): https://lv-hyku-staging.notch8.cloud/importers/26?locale=en#collection-entries
DEMO: https://share.getcloudapp.com/RBun27ZP

Testing Instructions

Create and Validate the sample file using a csv importer
sample file: shanas-rhino.zip

solved and merged per PR


Unicode in filenames

Hi,
I just had an issue with national characters in filenames not being stored correctly. I fixed the problem through changing the getter for UseUnicode in Zip\ZipStrings.cs, line 83 to reference CodePage instead of codePage. This will include the interpretation of AutomaticCodePage given in the getter for CodePage line 46. AutomaticCodePage is chosen as the default for codePage in line 31 - being good citizens.
I am just wondering if this is a bug in SharpZipLib or if I have solved my problem in a wrong way? - or perhaps neither or both?
Morten

Closing this due to lack of activity.


New version?

someone can upgrade a new version of RN?
🧐

@pherm Done.

have you upgrade ignite bro?

Yes I have updated this boilerplate.

👍👏


Upgrade Stellar SDK to v13.0.0-rc.2

Preview is available here:http://laboratory-pr1159.previews.kube001.services.stellar-ops.com/

I tagged off the stable build since everything seemed to check out, so please use that, instead :)


Document the Configurations and Extra Resources fields in the Notebooks setup screen

Document the Configurations and Extra Resources fields in the Notebooks setup screen. (We currently mention Extra Resources when discussing GPUs, but we should document other available options too.)
Doc to update:
https://www.kubeflow.org/docs/notebooks/setup/#create-a-jupyter-notebook-server-and-add-a-notebook
Screenshot:

/help-wanted

Guide to updating the docs:
https://github.com/kubeflow/website/blob/master/README.md

CPU, RAM and GPU are the main resources for cluster. for gpu, nvidia is common and amd also i think. the plugin set for extending Kubernetes are listed here. based on that, if somebody uses those plugins, then they may add those resources as extra. Also, enabling gpu and tpu suggests ways for adding gpu and nodeSelector for selecting a distinct type of gpu for particular notebook. nodeSelector is important for on-prem users as well but this feature not implemented yet. same the case for adding tpu in gke. For configurations.. are they low-level things users should be aware of ?

@lluunn Maybe you can help? We need to tell users (a) which options are supported, and (b) the format supported by the UI fields. If you have any useful info/links that you can add to this issue, then I can get started with a PR.
/assign

cc @kimwnasptd

For extra resource, the value should be a json string value for container.resouces.limit: https://kubernetes.io/docs/concepts/configuration/manage-compute-resources-container/#resource-requests-and-limits-of-pod-and-container
For example: {"memory": "128Mi", "cpu": "500m"}

Configuration is a list of labels: https://github.com/kubeflow/kubeflow/blob/master/components/jupyter-web-app/backend/kubeflow_jupyter/common/utils.py#L365
It can be used to set PodDefault. For example, to use the poddefault with:
matchLabels:
    addgcpsecret: "true"

Add configurations: addgcpsecret

@kimwnasptd please correct me if there is anything wrong.
@sarahmaddox please let me know if you need more info

Thanks @lluunn! I'll work on a PR for this doc update.


Don't modify .zprofile

Is your feature request related to a problem? Please describe.
I don't generally like when programs modify the .zprofile or .zshrc file. I prefer adding ~/.orbstack/bin to PATH manually or through direnv.
Describe the solution you'd like
I much prefer orbstack showing the pop up about the information that need to be added to the PATH(similar to docker desktop)
Describe alternatives you've considered
I tried to delete what orbstack added, but orbstack keeping chaning the .zprofile.
Additional context

I'd prefer not to have to add it myself :)

Duplicate of #552
As you saw, the current solution is that we add it automatically and show an alert, making you aware of it and allowing you to revert it immediately if you don't like it. We could actually add an Undo button to make that easier.

OrbStack won't add it back if you add any of OrbStack's bin directories to your PATH manually.

Ah, then subscribe to #552.

Actually, commenting the line will stop OrbStack from adding it back.


feat(web): adopt browser routing for cleaner URIs

This PR changes docat to use browser history for routing, for simpler implementation and cleaner URIs.
Before:
https://doc.example.org/#/project/version/path%2Fto%2Fpage.html#hash

After:
https://doc.example.org/project/version/path/to/page.html#hash

Mark as draft until backward compatibility is addressed.

Thanks for the changes @stevapple, we just had a 'facelift' of the frontend page of the UI, would you mind rebasing onto main before I have a look?

Thanks for the changes @stevapple, we just had a 'facelift' of the frontend page of the UI #907, would you mind rebasing onto main before I have a look?

Done

@randombenj @fliiiix bother for a second review?


Request tests of BaseXXX/XXXImpl interface time sequence verification

Is your feature request related to a problem?
Related to :
https://github.com/pingcap/tiflow/issues/5764
We has no specified global ci tests to cover the overall time seqence of Master/Worker.
Describe the feature you'd like
Add tests of BaseXXX/XXXImpl interface time sequence verification
Describe alternatives you've considered
May be we can use etcd mvcc feature to verify the time sequence of different sequence of BaseXXX/XXXImpl interface.
For example:
<job1, (MasterImpl::OnWorkerDispatch, 1)>
<job1, (WorkerImpl::InitImpl, 2)>
<job1, (WorkerImpl::CloseImpl, 3)>
....

Teachability, Documentation, Adoption, Migration Strategy
No response

/cc @liuzix
FYI

/area engine


Enable passing of RayDMatrix to sklearn API

@krfricke The test failure on Ray master seems to be unrelated


ftp class file upload: gives error but the file is uploaded

codeignites behave strangely.
i am trying to upload a one file but get error message "Unable to upload the specified file. Please check your path." which seems strange since the file was uploaded. I have tried with and without passive but I get the same result.
I have tried to change the mode but it does not change that I get the error message.
CI_VERSION: 3.1.10
PHP 7.0.33-0
Config:
$config['hostname'] = 'ip';
$config['username'] = 'user';
$config['password'] = 'password';
$config['debug'] = TRUE;
$config['passive'] = FALSE;

$this->ftp->connect($config);
$this->ftp->upload($fileFrom, $fileTo);
$this->ftp->close();

We simply use the return value of ftp_put() to determine whether an error has occured or not. If you see that error message, it means ftp_put() has returned false.
I can only suggest that you provide a specific upload() mode (via the third parameter), but outside of that there's nothing we can do.


Game name not pulling in the json output?

Hi there,
I attempted the example and it seems that the game name does not show in the output:
Json Output:

{"name":"","metascritic_score":89,"users_score":8.9,"rating":"M","genres":"Role-Playing","developers":"Black Isle Studios","publishers":"Interplay","release_date":"Sep 30, 1997","thumbnail_url":"","cheat_url":"http:\/\/www.gamefaqs.com\/console\/pc\/code\/197289.html"}

The game I searched for was "Fallout"

Hi,
Thanks for your report. I guess the website as been updated, so I need to parse on another element. I will look at it ASAP.
Kind regards,
Melroy


Update README.markdown

fixes #2

It can also be downloaded :)

okay

no need to close it @morenoh149, I'll let @kenwheeler decide on that, appreciate your feedback

I'll leave it closed for now, going to fix this with a builder archetype when init is ready