Bugfix: internal links in remote-hosted epubs open in new tab

This PR fixes an issue where, when an epub is hosted on a separate remote server, its internal links are considered external and open in a new tab.
You can see an example of this issue here:  https://drb-qa.nypl.org/read/6585394 (navigate to "contents")
This doesn't seem to be a complete fix - this fix will not catch the case where the same host serves both the epub and also some other content - eg:  if www.company.com serves wwww.company.com/files/epub_id/manifest.json and this epub links to www.company.com/about/team, this second page will be interpreted as internal to the epub.
I'd like to know how much of an edge case this is - I think if it's possible to check the link location against the manfiest URL, that might give us more confidence, but I'm not quite sure how to get the manifest URL inside the EventHandler, or if we even want to.

@aferditamuriqi This sounds good!  Yes, if you have ideas on how the EventHandler can access the manifestUrl that'd be super helpful to me!  Right now it's all being called via a listener, so I don't know how this information could be passed.


BulkData $import API improvement - Exit Status #949

Updated the ChunkJob.xml to include missing parameters (the passed
values didn't agree)
Add guard for null condition
Added support for operational outcome bucket

{
    "transactionTime": "2020-04-25T20:35:50.699-04:00",
    "request": "$import",
    "requiresAccessToken": false,
    "output": [
        {
            "type": "OperationOutcome",
            "url": "test-import.ndjson_oo_success.ndjson",
            "count": 3
        }
    ],
    "error": [
        {
            "type": "OperationOutcome",
            "url": "test-import.ndjson_oo_errors.ndjson",
            "count": 0
        }
    ]
}```

Signed-off-by: Paul Bastide<EMAIL_ADDRESS>

I'm merging it and then sync to my local for my export refactoring.


[SPARK-38633][SQL][FOLLOWUP] JDBCSQLBuilder should build cast to type of databases

What changes were proposed in this pull request?
DS V2 supports push down CAST to database.
The current implement only uses the typeName of DataType.
For example: Cast(column, StringType) will be build to CAST(column AS String).
But it should be CAST(column AS TEXT) for Postgres or CAST(column AS VARCHAR2(255)) for Oracle.
Why are the changes needed?
Improve the implement of push down CAST.
Does this PR introduce any user-facing change?
'No'.
Just new feature.
How was this patch tested?
Exists tests

ping @huaxingao cc @cloud-fan

thanks, merging to master!


hub issue only shows the first 30 open issues

I know that it's listed as "beta", and I'm guessing that it's doing some sort of paging and only showing the first page, but it's a bit of a pain that the hub issue command only seems to display the first 30 newest open issues.

Yes, the command is still in heavy beta and isn't ready for consumption. Adding proper pagination is definitely on our roadmap, though.
Closing in the meantime to help clear out our issue backlog.


Facebook, twitter and Website in Menu

In the menu there is a link to the Facebook page of WCEU and below that to the twitter account.

The link to the site should also be in that list:  https://2018.europe.wordcamp.org/
It might be better to just have icons next to each other on 1 line to save space in the menu with all 3 (website, facebook, twitter)

Almost forgot: we also have Instagram! https://www.instagram.com/wceurope/

Updated.

@luisherranz Do you have these icons? Or do you have similar ones?

@ePascalC we've finally purchased FontAwesome Pro so no problem anymore :)


switch to range-based for loops

Reference Issue

What does this implement/fix? Explain your changes.
I switched some old-style for loops to newer range-based for loops.
This should help readability and might reduce off-by-one errors.
Let me know what you think.
Any other comments?

Thanks @AnnaBruenisholz !


How to combine parameters from different haiku modules?

In PyTorch, it is possible to have a single optimizer for different nn.Module sets of parameters, and there are various ways to combine different modules' parameters.
For example, from https://github.com/altosaar/variational-autoencoder/blob/dfb452b5421e9e5b97315c6420b8766ac86f3f4f/train_variational_autoencoder_pytorch.py#L216:
    optimizer = torch.optim.RMSprop(        list(model.parameters()) + list(variational.parameters()),        lr=cfg.learning_rate,        centered=True,    )
What is the equivalent in optax? Is it chaining optimizers, or is such functionality not supported at this time, requiring different instances of optax optimizers, one per haiku.Module?
Thanks so much!

Yes, you can simply create a collection combining all your parameters, then pass that to tx.init. Below's an example:
import jax.numpy as jnp
import optax

# Suppose the collections below are arbitrary PyTrees with parameters
# from Haiku (or any other framework)
model_params = {'linear': {'w': jnp.zeros((5, 5)), 'b': jnp.zeros(5)}}
other_params = {'other': jnp.zeros(3)}

# Create another PyTree (here, just a tuple) with all the params
all_params = (model_params, other_params)

tx = optax.rmsprop(learning_rate=0.001, centered=True)
state = tx.init(all_params)

# suppose you have gradients for both
model_param_grads = ...
other_param_grads = ...

# Combine the grads the same way you combined the params
all_param_grads = (model_param_grads, other_param_grads)

# Apply model updates
updates, new_state = tx.update(all_param_grads, state)
new_all_params = optax.apply_updates(all_params, updates)

# Unpack new params as needed
new_model_params, new_other_params = new_all_params

How does that sound?

Thank you so much! This is what I was looking for :) and solves the problem nicely!!!

Hi. I need help with this topic. I am using Haiku for creating my neural networks. The parameters for the networks are not training when I am using the approach you mentioned above. Can someone please help me figure out how to properly apply the optimizer updates to the two neural network parameters?
Thank you.
@jaanli @n2cholas

@rsmath could you send a snippet/gist of code demonstrating the issue? If it is structured like the suggestion above it should work.

I was able to get it work. Thank you for your help :)


Edit Appendix E Correlation Test Run Data

As an ECMPS user (logged-in), I want to update Appendix E Correlation Test Run data so that I can add information to a test
Acceptance Criteria:
Given the user has selected Appendix E Correlation Summary from the Test Type Group dropdown
When the user selects 'Edit' for Appendix E Correlation Test Run
Then the system displays the edit modal
And the user can edit Appendix E Correlation Test Run data
Run Number (Text)
Reference Value (Text)
Hourly Heat Input Rate (Text)
Total Heat Input (Text)
Response Time (Text)
Begin Date (Text+Date Picker)
Begin Hour (Dropdown)
Begin Minute (Dropdown)
End Date (Text+Date Picker)
End Hour (Dropdown)
End Minute (Dropdown)
Given the user has updated Appendix E Correlation Test Run data
When the user chooses to save and close
Then the data is saved
And the modal closes
And the user is returned to the Landing page for Appendix E Correlation data
And the updates are reflected in the Appendix E Correlation Test Run data section
Warning message to pop up when a user updates data and closes the modal without saving (Cancel or clicks on X)
Warning message to read: Closing this window will discard any unsaved changes
Reference Invision:
https://projects.invisionapp.com/freehand/document/38vmVWIc0?zoomToItems=Y2w2ZjR5Mmw2MDBzdzNiN25ucHNoaHhkdg%3D%3D
Testing
Test Case: TC1011
Tester: @mosesdeeCVP

Test Passed


Can't get CoreUtilites Example running (JDK 11)

Hey 👋🏻
I'm trying to enrich my existing AWS Lambda Java setup with the structured logging functionality. However, I'm failing to get it running with Java 11.
To confirm that it's not something project-specific I forked this sample project and tried to run the helloworld.App on a Java 11 default AWS Lambda runtime and get the following error:
Error loading class helloworld.App: java.lang.ExceptionInInitializerError
java.lang.ExceptionInInitializerError
	at java.base/java.lang.Class.forName0(Native Method)
	at java.base/java.lang.Class.forName(Unknown Source)
Caused by: java.lang.UnsupportedOperationException: No class provided, and an appropriate one cannot be found.
	at org.apache.logging.log4j.LogManager.callerClass(LogManager.java:573)
	at org.apache.logging.log4j.LogManager.getLogger(LogManager.java:598)
	at org.apache.logging.log4j.LogManager.getLogger(LogManager.java:585)
	at helloworld.App.<clinit>(App.java:37)
	... 2 more

Some basic research pointed me to this issue https://issues.apache.org/jira/browse/LOG4J2-2834 that mentions a fix by adding Multi-Release:true to the Manifest.md.
I tried to add this by enriching the transformer setup but still get the same issue when running the sample Lambda of this repo.
What's weird is that this issue seems to only occur on the AWS Lambda runtime. If I manually add the Lambda handler as the Main-Class with a basic main method that invokes the function, and run java -jar target/... I don't see this exception locally.
Reproducible example: https://github.com/rieckpil/aws-lambda-powertools-examples
With a java8.al2 runtime it works fine, but I'd like to avoid downgrading all my lambdas to Java 8 in 2022 😅
Any ideas?

Hi @rieckpil Thanks for opening the issue.
The utility for sure works well with java 11 and further too. Infact we have that validated as part of the build process of utility too, before we make a release.
That being said, I just tried the example as is and using aws sam template provided in the example and everything worked fine as it should have.
I also tried the serverless framework example you had, there off course it failed, like you mentioned above.
I tried changing the function code to
   private final static Logger log = LogManager.getLogger**(App.class)**;
instead of
   private final static Logger log = LogManager.getLogger();
and then it works fine. So there has to be some difference for sure between how sam is packaging thing vs just deploying the jar directly.
I will recommend explicitly specifying the class when initializing logger until i see why log4j is not able to figure things out on its own when running as plain jar vs what SAM packages and builds

After digging a bit more, the real reason here is and this post was really helpful as well.

maven-shade-plugin is used to create an "uber"-jar containing all the dependencies of my lambda function. This approach is recommended one as well official AWS documentation. the maven-shade-plugin extracts all jar file dependencies and repackages them into a single, flat jar file. If one of your dependencies is a multi-release jar (as is log4j2), then you can configure the maven-shade-plugin to reconstruct an appropriate META-INF directory and if you run the jar as a jar file then everything still works.

But because AWS Lambda extracts the jar, the META-INF directory is not "seen" by the JVM anymore, and anything that was in META-INF/versions is ignored.
So basically, this is not just specific to powertools but basically, log4j2 and java lambda function configuration. That being said, what we are going to do in powertools specifically:

Update docs and samples to specify class names directly when initialising logger.  Thats better from cold start perspective anyways.
Clarify and address this bit in FAQ section of doc as well with a workaround on what config might be needed, if users still want to depend on log4j to figure out class name.

I will copy this issue to powertools repo itself.

Here is the issue https://github.com/awslabs/aws-lambda-powertools-java/issues/825

Wow, thanks for the quick help @pankajagrawal16.
Oh yikes, I was digging so deep into this :D
Thanks for your investigation effort 🥳  I'll also take a closer look into favoring the assembly plugin over the shade plugin.
PS: I can confirm that with the suggested change it works 👍


fixed a minor bug

Found a minor syntactical bug when I was packaging with PyInstaller on Windows and Mac.
On Windows, the build process fails immediately.
On Mac, the build succeeds, but when the program is launched it shows up and kills the app.
The error is the same on Windows and Mac.
Traceback (most recent call last):
  File "<frozen importlib._bootstrap>", line 983, in _find_and_load
  File "<frozen importlib._bootstrap>", line 967, in _find_and_load_unlocked
  File "<frozen importlib._bootstrap>", line 677, in _load_unlocked
  File "PyInstaller/loader/pyimod03_importers.py", line 531, in exec_module
  File "gooey/__init__.py", line 7, in <module>
  File "<frozen importlib._bootstrap>", line 983, in _find_and_load
  File "<frozen importlib._bootstrap>", line 967, in _find_and_load_unlocked
  File "<frozen importlib._bootstrap>", line 677, in _load_unlocked
  File "PyInstaller/loader/pyimod03_importers.py", line 531, in exec_module
  File "gooey/gui/components/options/options.py", line 90, in <module>
  File "gooey/gui/components/options/options.py", line 10, in _include_layout_docs
TypeError: can only concatenate str (not "NoneType") to str

Thanks for the fix! Could you update your PR to target the release branch? The contributing guidelines in the PR notes (*cough*that text you deleted while opening the pr*cough* ^_^) will point you in the right direction.

Closing. Contributing.md guidelines not followed. PR opened against wrong branch.


Alpha blending for better text visibility

Hello @ankush-me,
In poisson_reconstruct.py is the:
IM_TOP -> text instance
IM_BACK -> background image
There says:
combine images using poission editing.
IM_TOP and IM_BACK should be of the same size.

But what images script combines?
Also in the poisson_reconstruct.py what is these gxs,gys,gxd,gyd? This is X and Y coordinates of something?
[gxs,gys] = get_grads(ims)
[gxd,gyd] = get_grads(imd)

Is the alpha blend done here?
elif mode=='blend': # from recursive call:
           # just do an alpha blend
           gx = gxs+gxd
           gy = gys+gyd

When I replace 'blend' instead 'max' in function blit_images(), nothing happens?

I don't understand what should I do??


Modify Student tests for ProgrammerError's use case

Fixed incorrect renaming of Person to Student
Modified tests to be in line with ProgrammerError's definition of a unique student (i.e. having unique student id only)

Codecov Report

Merging #103 (405cb79) into master (6c0e789) will increase coverage by 0.50%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             master     #103      +/-   ##
============================================
+ Coverage     66.24%   66.75%   +0.50%
- Complexity      359      365       +6
============================================
  Files            69       69
  Lines          1182     1182
  Branches        118      118
============================================
+ Hits            783      789       +6
  Misses          344      344
+ Partials         55       49       -6

Impacted Files
Coverage Δ

...ain/java/seedu/programmer/model/student/Grade.java
80.00% <0.00%> (+10.00%)
:arrow_up:

...n/java/seedu/programmer/model/student/ClassId.java
80.00% <0.00%> (+10.00%)
:arrow_up:

...n/java/seedu/programmer/model/student/Student.java
91.17% <0.00%> (+11.76%)
: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 6c0e789...405cb79. Read the comment docs.


PIP 45: Update zookeeperServers to metadataStoreUrl

As discussed with @codelipenghui, https://github.com/apache/pulsar/pull/13077 changes zookeeperServers to metadataStoreUrl.
Consequently, the flag zookeeperServers of pulsar-perf managed-ledger should be changed to metadataStoreUrl. Code files should be updated.
https://pulsar.apache.org/tools/pulsar-perf/2.10.0-SNAPSHOT/#managed-ledger

Any one would like to contribute? Thanks

Thanks for your confirmation, i will update for it.

https://github.com/apache/pulsar/pull/13777 resolved this issue.

Thanks @liangyuanpeng


systemd v254 Behaviour of sandboxing options for the per-user service manager units has changed

in systemd 254 Behaviour of sandboxing options for the per-user service manager units has changed. They now imply PrivateUsers=yes see https://github.com/systemd/systemd/releases/tag/v254
it will be enabled for rust-u2f due to
https://github.com/danstiner/rust-u2f/blob/da1a256e804395588c21c0dd9891310506746e7a/linux/user-daemon/softu2f.service#L8
this makes the root check fail https://github.com/danstiner/rust-u2f/blob/da1a256e804395588c21c0dd9891310506746e7a/linux/user-daemon/src/main.rs#L131

Thanks for the report, fixed in master branch with 121a81db2a1b412be5723e22081578b074d5f7ea


get/set in JS/TS

As you can see it should be the same as static, talking about color and italic.

If needed I can create a PR.
/cc @sdras

Yeah, this is a spot where it makes sense to make them the same, I would accept a PR for this, thank you!

Okay. I will create a PR today :sparkles:

@sdras PR already opened: https://github.com/sdras/night-owl-vscode-theme/pull/110 :fire:

Thanks!


Question - Is there any permission need to be provided so SMS can be read by the Flutter App?

I tried running the app on actual mobile device but SMS was not read by the app, I am suspecting I might have to ask user for permission to read sms. Is it something necessary?
if so, does it need to be part of manifest file?

Follow the README carefully, you should not face any issue setting this up. BTW, OTP auto-fetch only works on android.

No permission is required though..

Don't forget to add you SHA keys in the firebase console

Hi @rajesh-h, this issue looks inactive. I'm closing this, feel free to open another one.


Missing dartdoc

Checking the source code I was a bit surprised to find zero comments.
There is a full article on that widget, I expected at least some dartdoc

Good pt, will add some in next version.


Not-in-place real-to-complex 2D forward FFT modifies the input

What is the expected behavior
Not-in-place real-to-complex 2D forward FFT should not modify the input.
What actually happens
Input is modified after rocfft_execution completes. There is a pattern in the data that could be a clue to rocFFT developers on what has gone wrong.
How to reproduce
I could reproduce in two ways:

In standard rocfft sample at docs/samples/real2complex_2d.cpp, dump and compare input after forward rocfft_execution completes.
Use the sample code I provided (failure_repro.cpp.txt).

Environment
CPU => Intel(R) Core(TM) i7-4790K CPU @ 4.00GHz
GPU => gfx900
| Software | version |
Ubuntu 18.04
HIP version: 3.3.20126-2dbba46b
clang version 11.0.0
rocm 3.3.0 => standard RPM
rocfft => latest built from source (revision<PHONE_NUMBER>22a9cc303fb28ef3a8b8196124f139 May 28)
rocFFT compiled using HIP_PLATFORM=clang

The real/complex algorithms require that they overwrite the work buffer; we would have to create an entirely separate work buffer in order to avoid this, which would be massively inefficient.
Moreover, other implementations (eg FFTW: http://www.fftw.org/fftw3_doc/One_002dDimensional-DFTs-of-Real-Data.html) overwrite input data as well.

@lashgar I totally understand why you might think that.  The property of 'in-place' versus not 'in-place' simply determines whether or not the result is placed in the input array.  Of course, in-place transforms therefore must modify the input array, because that's where they put the result.  Transforms that are 'not in-place' don't put the result in the input array, but that doesn't necessarily mean that they can't put other things in the input array, like intermediate answers or other temporary computations.
As @malmcolmroberts pointed out, FFTW behaves the same way.  We tried to design rocfft to echo some of the semantics of FFTW so that folks using either felt comfortable.
If you need to ensure that the input array is unmodified, as is common in a lot of pseudo-spectral codes, then make a copy before calling rocfft.

Thanks @malcolmroberts & @dmcdougall for clarification. I will wrap my FFT exec calls to buffer the input.


server: avoid flushing while a flush is in progress

In order to support calls to Server::Instance::flushStats (let's call this an "external flush") while also having a flush timer activated we need to handle an external flush being requested while in the process of doing a periodic flush. As it stands,
this currently runs the risk of triggering an ASSERT due to the histogram merging being async. See https://github.com/envoyproxy/envoy-mobile/issues/748 for an example of this happening.
This PR proposes a simple solution where we simply ignore calls to flushStats when we're still in the process of merging
histograms.
Signed-off-by: Snow Pettersen<EMAIL_ADDRESS>Risk Level: Medium
Testing: UTs
Docs Changes: n/a
Release Notes: n/a

@jmarantz This is a fairly naive solution to the problem so wanted to put this up for feedback before I started writing tests. I'm not sure if it makes sense to support other ways of handling this (e.g. enqueue a flush instead of dropping it), or if it makes more sense to make this part of the stats store (e.g. have mergeHistogram do nothing if merging is already happening).

I like this approach. I think if stat-flushing is slow you don't want to queue them. You want to drop flushes if they are not done.
I would suggest adding a state counting the dropped flushes. I think that would be more useful to us than info-logs.

tsan failure in //test/integration:multiplexed_integration_test  , 2/5 times. Not sure whether that's related.

I see the same failures listed in #test-flaky, so I believe these are also happening on main


Use of term topologic

I wonder whether you should use the term topologicAL instead of topologic. Both terms exist in English, but it seems that the one with the "al" suffix is used more frequently.
I mean here for example the "TopologicRelationTypeValue"
Maybe a native speaker can help?
Best regards,
GA

Topological

This requires a change to the name of the TopologicRelationTypeValue Class. The term "topological" is used in all other cases with the exception of 7.2.3. In that case the term "topologic" is appropriate since it is used in conjunction with the terms "geometric" and "semantic".

Class name and definition have been changed to "Topological" in the UML model.

Good call (sorry to be late). "Topologic" is a very rare term. I'd never seen it & had to Google itl it doesn't occur in the web accessible versions of standard dictionaries.

Agreed by unanimous consent, closed 3 Sept meeting


Fixes #437 : Updated GetAvailableDiskSize function

Description
in internal/validations.go
we have syscall.Statfs and Statfs_t which are os specific methods and does not work in  windows
so what i have does add a pkf "github.com/shirou/gopsutil/disk"
which is used to calculate the available disk and it is os independent and maintained regularly
Related Issue(s)
Fixes # 437
How to test
clone repo on a windows machine and you will straight away see that

Hey @sundaram2021! Would you mind bumping/updating the go.mod and go.sum here? A go mod tidy would suffice.
We would love to merge this whenever possible! Thank you!


getTotalCost as float

The CartItem class accepts price as float, but when retrieving the total cost, the SimpleCalculator cast them as integer

It happens because CartItem::getCost() method uses function ceil(). In the database, the productFieldPrice can be a string, float or an integer, but will be returned as an integer. And it is much better to store prices in the database as an integer than a float.

Thanks Devanych!
In my case, I need to keep it as float.
In case someone needs a workaround:

Create a FloatCalculator class implementing devanych\cart\calculators\CalculatorInterface
Copy the methods from SimpleCalculator
Replace $item->getCost() by $item->getPrice() in getCost method
Remember to specify this new class in main.php as calculatorClass

Thanks, closing.


Strange...

on lstm_sin, I change the training_steps from 10000 to 10, and the batch size from 100 to 5, and get the same results...
Are you sure the whole thing is really predicting?

can you provide more information about your setup, tensorflow version, ...

I just run the example with 10 steps and a batch size 5, and got this result:

Obviously it did not learn yet to predict a correct sin function. And even if it's the case, the sin function is very simple to learn for the model. since it's monotonic, this would not work for more complex function or time series, try to do the same for x*sin and see for your self.
One last thing, if you ran the example before with 10000/100 and then changed the values, of course it will predict with the same accuracy because you basically just added a few steps to already trained model. The variables are saved after each execution, you need to remove the folder where the data is stored.
Closing this issue.

You're right, the model was cached in the filesystem.


Add two new chip tlv apis

Problems:
Chip IM Circular Eventing needs the below new APIs to set backing store
for tlvreader and get the totla remaning number of bytes.
Summary of Changes:
-- Add SetBackingStore to set backstore
-- Add GetRemainingFreeLength to get the total remaining number of bytes
for current tlv writer.

Problem
<<<<<FILL ME IN - the issue this PR is intended to address>>>>>

Summary of Changes
<<<<<FILL ME IN - what's in this PR>>>>>

Fixes #<<<<<FILL ME IN  - issue number(s). If no issue, please create one>>>>>

Could we add some tests for these APIs?

Could we add some tests for these APIs?

+1

Could we add some tests for these APIs?

Done, thanks


Use app.driver.Stat for registry health check

driver.List on "/" is very expensive if registry contains significant amount of images. And the result isn't used anyways.  In most (if not all) storage drivers, Stat has a cheaper implementation, so use it instead to achieve the same goal.
Test plan:
docker build -t registry:latest .
Run the registry with cmd/registry/config-example.yml and try push a image to the local registry at several different time point: immediately when registry boot up; 10s after registry boot up; 30s after registry boot up. And all pushes finished without problem.

Please sign your commits following these rules:
https://github.com/docker/docker/blob/master/CONTRIBUTING.md#sign-your-work
The easiest way to do this is to amend the last commit:
$ git clone -b "health" git@github.com:yixizhang/distribution.git somewhere
$ cd somewhere
$ git commit --amend -s --no-edit
$ git push -f

Amending updates the existing PR. You DO NOT need to open a new one.

Current coverage is 51.20% (diff: 0.00%)

Merging #2115 into master will decrease coverage by 10.02%

@@             master      #2115   diff @@
==========================================
  Files           129        129
  Lines         11416      11416
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
- Hits           6990       5846   -1144
- Misses         3535       4821   +1286
+ Partials        891        749    -142

Powered by Codecov. Last update 2d50093...ced2694

LGTM

@aaronlehmann can you take a look at this, is there any reason we would want to enforce / cannot be statted

LGTM
Seems fine to me. If List allows / I can't see why Stat shouldn't as well.


labe_image bazel build failed!

Please make sure that this is a build/installation issue. As per our GitHub Policy, we only address code/doc bugs, performance issues, feature requests and build/installation issues on GitHub. tag:build_template
System information

OS Platform and Distribution (e.g., Linux Ubuntu 16.04): Linux Ubuntu 16.04
Mobile device (e.g. iPhone 8, Pixel 2, Samsung Galaxy) if the issue happens on mobile device:
TensorFlow installed from (source or binary):binary
TensorFlow version: 1.12
Python version:2.7
Installed using virtualenv? pip? conda?:pip
Bazel version (if compiling from source):0.24.0
GCC/Compiler version (if compiling from source): gcc  5.4.0 20160609
CUDA/cuDNN version: 9.0
GPU model and memory:

Describe the problem
To build label_image for android ARMv8:
> bazel build --config android_arm64 --config monolithic --cxxopt=-std=c++11 \
  //tensorflow/lite/examples/label_image:label_image

error infor:
ERROR: /home/apuser/deeplearning/tensorflow/tensorflow-master/tensorflow/lite/kernels/internal/BUILD:496:1: no such package '@com_google_absl//absl/base': java.io.IOException: Error downloading [http://mirror.tensorflow.org/github.com/abseil/abseil-cpp/archive/2c8421e1c6cef0da9e8a20b01c15256ec9ec116d.tar.gz, https://github.com/abseil/abseil-cpp/archive/2c8421e1c6cef0da9e8a20b01c15256ec9ec116d.tar.gz] to /home/apuser/.cache/bazel/_bazel_apuser/4972b2c1cfd6d41e3f1f5f386fde1d1a/external/com_google_absl/2c8421e1c6cef0da9e8a20b01c15256ec9ec116d.tar.gz: All mirrors are down: [Connection reset, GET returned 404 Not Found] and referenced by '//tensorflow/lite/kernels/internal:tensor_utils'
ERROR: Analysis of target '//tensorflow/lite/examples/label_image:label_image' failed; build aborted: no such package '@com_google_absl//absl/base': java.io.IOException: Error downloading [http://mirror.tensorflow.org/github.com/abseil/abseil-cpp/archive/2c8421e1c6cef0da9e8a20b01c15256ec9ec116d.tar.gz, https://github.com/abseil/abseil-cpp/archive/2c8421e1c6cef0da9e8a20b01c15256ec9ec116d.tar.gz] to /home/apuser/.cache/bazel/_bazel_apuser/4972b2c1cfd6d41e3f1f5f386fde1d1a/external/com_google_absl/2c8421e1c6cef0da9e8a20b01c15256ec9ec116d.tar.gz: All mirrors are down: [Connection reset, GET returned 404 Not Found]

as shown in the error message you posted,
Error downloading [http://mirror.tensorflow.org/github.com/abseil/abseil-cpp/archive/2c8421e1c6cef0da9e8a20b01c15256ec9ec116d.tar.gz, https://github.com/abseil/abseil-cpp/archive/2c8421e1c6cef0da9e8a20b01c15256ec9ec116d.tar.gz] to ...

Mostly, it's problem of your internet connection, e.g., behind a firewall or proxy

Thanks,  the reason is that the version of bazel is too new.


Update README.md

Pull Request Test Coverage Report for Build 961f9ee5f09c14b768e9fa9bcddb1ae8120f699e-PR-12

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

Totals

Change from base Build 7a727fdb5be78fb098e6da689dd5cf4b55154bb0:
0.0%

Covered Lines:
1174

Relevant Lines:
1177

💛  - Coveralls


Support for Git's Large File Storage?

Will the Docker Hub have support for git-lfs?
So we can create automated builds without worrying about the repository size (cache history).

If you are reporting a new issue, make sure that we do not have any duplicates already open. You can ensure this by searching the issue list for this repository. If there is a duplicate, please close your issue and add a comment to the existing issue instead.
If you suspect your issue is a bug, please edit your issue description to include the BUG REPORT INFORMATION shown below. If you fail to provide this information within 7 days, we cannot debug your issue and will close it. We will, however, reopen it if you later provide the information.
For more information about reporting issues, see CONTRIBUTING.md.
You don't have to include this information if this is a feature request
(This is an automated, informational response)

BUG REPORT INFORMATION
Use the commands below to provide key information from your environment:
docker version:
docker info:
Provide additional environment details (AWS, VirtualBox, physical, etc.):
List the steps to reproduce the issue:
1.
2.
3.
Describe the results you received:
Describe the results you expected:
Provide additional info you think is important:
----------END REPORT ---------
#ENEEDMOREINFO

Hi @rafaelsoaresbr thanks for the suggestion; can you open this issue in the https://github.com/docker/hub-feedback/ issue tracker? This issue tracker is only for Docker itself, and Docker Hub is a different team, so nothing we can do about that here :smile:  You can also open a ticket through<EMAIL_ADDRESS>I'll close this issue here, but feel free to comment here after I've closed (nice idea btw)

Thanks for your quick reply! It looks like someone already requested this o(5 Dec 2015)
https://github.com/docker/hub-feedback/issues/500


Collapsing adjacent text nodes is too aggressive

Ruby code demonstrating the problem: bug_nokogiri.rb
Output from ruby code demonstrating the problem: bug_nokogiri.txt
What problems are you experiencing?
The example runs the offending code 3 times.

When path == 1, this is my initial version, converted from a JavaScript version that's been running for a long time.
When path == 2, this is my first workaround. It works for two consecutive text nodes but fails for more.
When path == 3, this is my final workaround

This code is trying to convert traditional HTML to use HTML5 <section> tags.
It does this by locating <h1>, inserting an empty <section> before it and then moving everything up to the next <h1> into that section. (then repeat for other header levels.)
Earlier code (unwrap) causes the DOM tree to have multiple text nodes that are adjacent to each other.
In path 1 and path 2 the result is corrupt. It appears that when add_child appends a text node to the <section> and the last child was already a text node, these are combined into a single text node (good). The problem is that the pointer to the next node is broken.
In other words
If:
sect.last_child.text? && sect.next_sibling == n1 && n1.text? && n2.text? && n2 == n1.next_sibling

then the sequence
n1.unlink
sect.add_child(n1)

sometimes causes n2 to become nil
What's the output from nokogiri -v?
$ nokogiri -v
Nokogiri (<IP_ADDRESS>)
---
warnings: []
nokogiri: <IP_ADDRESS>
ruby:
  version: 2.4.0
  platform: x86_64-darwin16
  description: ruby 2.4.0p0 (2016-12-24 revision 57164) [x86_64-darwin16]
  engine: ruby
libxml:
  binding: extension
  source: packaged
  libxml2_path: "/usr/local/lib/ruby/gems/2.4.0/gems/nokogiri-<IP_ADDRESS>/ports/x86_64-apple-darwin16.3.0/libxml2/2.9.4"
  libxslt_path: "/usr/local/lib/ruby/gems/2.4.0/gems/nokogiri-<IP_ADDRESS>/ports/x86_64-apple-darwin16.3.0/libxslt/1.1.29"
  libxml2_patches: []
  libxslt_patches: []
  compiled: 2.9.4
  loaded: 2.9.4

Can you provide a self-contained script that reproduces what you're seeing?
See attached

Thanks for reporting this. I'll take a look as soon as I can.

The code example provided here is complex, and I don't have a ton of time at the moment to dig in and make sure I understand it. Can you help me understand what document structure you are trying to generate?
Text mode merging is beyond the control of Nokogiri, and is done by the underlying XML library. There are places in Nokogiri where we merge nodes during certain operations, but we do it defensively, because if we let libxml2 do it we are left with dangling pointers and the ensuing memory corruption.
I can go into more detail if you like, but understanding what you're trying to do will perhaps focus the conversation on your immediate problem.

In step 3 I misspoke. It should say that the loop stops at the enclosing ,  or whatever. No <h\d> elements are changed.
Steveg

On Jan 13, 2017, at 8:39 AM, Steve Glaser<EMAIL_ADDRESS>wrote:
I'm trying to convert a sequence of <h\d> headings into HTML5 by adding  tags.

heading 1

heading 2
paragraph 1
paragraph 2
paragraph 3

heading 2a

The original looks exactly the same if you remove the  tags.
The code converts the  items to text nodes (the unwrap routine in the code). These end up as distinct text nodes (I would be OK if they got merged but they didn't).
For each  element it then:

Inserts  in front of the  element.

It then loops through nodes beginning with the  element following the new  until it hits the next  element or the end of the enclosing element (nil returned from next_sibling). For each node, it unlinks it and uses add_child to move it into the new section.

Repeat steps 1 and 2 for , , ... This makes sure that the hierarchy is maintained. When doing , the loop stops at the end of the enclosing . When doing , the loop stops at the end of the enclosing , etc.

When it moves the text node containing "paragraph 1", that node gets moved correctly. When it moves the text node containing "paragraph 2" that gets merged with the "paragraph 1" node (perfectly legal), but something screws up and also merges "paragraph 3" or sets the loop pointer used in step 2 to nil. The resulting nil is too early and causes loop 2 to exit early. Sometimes "paragraph 3" gets duplicated in the output (one copy inside the  and another one after the ).
The path == 3 workaround does a manual "merge" of the text nodes in loop 2 by appending adjacent text nodes to the first text node of the sequence.
Steveg

On Jan 13, 2017, at 7:20 AM, Mike Dalessio<EMAIL_ADDRESS>wrote:
The code example provided here is complex, and I don't have a ton of time at the moment to dig in and make sure I understand it. Can you help me understand what document structure you are trying to generate?
Text mode merging is beyond the control of Nokogiri, and is done by the underlying XML library. There are places in Nokogiri where we merge nodes during certain operations, but we do it defensively, because if we let libxml2 do it we are left with dangling pointers and the ensuing memory corruption.
I can go into more detail if you like, but understanding what you're trying to do will perhaps focus the conversation on your immediate problem.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.

Hi @sglaser,
Sorry for the slow response. I think I understand now what you're trying to do; but I'm having trouble following the code, and so I'll respond to your description above.
My advice would be to avoid operating on the text nodes, and instead operate on the p nodes. This will avoid having text nodes get merged by libxml2 (or nokogiri) while the structure is still being changed.
You may also want to consider building a second document based on the content of the first (perhaps even leveraging the SAX parser on the first).
I hope I haven't misunderstood the problem again; if so, happy to try once more. Otherwise, I'll close this out soon.


feat: remove all cookie methods

Cookie-related methods and helpers should be implemented where they are used. For example, in the auth-helpers library directly.

Bundle size has improved too:
https://bundlephobia.com/package/<EMAIL_ADDRESS>https://bundlephobia.com/package/<EMAIL_ADDRESS>


Missing taskbar icon

It seems PicoTorrent's cute icon doesn't actually register on the Windows 10 taskbar, even though the system tray, start menu and window icons work well. Maybe this is something easy to fix during the build process.

Odd, can you try with the latest installer? https://ci.appveyor.com/project/picotorrent/picotorrent

Is there a certain trick to downloading this? I haven't used AppVeyor before.

https://ci.appveyor.com/project/picotorrent/picotorrent/build/job/hlt4s1h1t09umx10/artifacts
The exe file in this list is the latest beta installer. You might have to
uninstall your current version first.
On Fri, 16 Mar 2018 at 11:21, Pixel Eater<EMAIL_ADDRESS>wrote:

Is there a certain trick to downloading this? I haven't used AppVeyor
before.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/picotorrent/picotorrent/issues/546#issuecomment-373669157,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABbDcGjoDl02_a83fpXRNNICdQyO1LO-ks5te5INgaJpZM4Sl7V3
.

I think the problem returned with Windows 11:
(installed latest via choco)

it somehow fails to register the icon file. If I manually recreate the shortcut, everything is OK.


Warning for 4.0.0-beta.4 - circular dependency

Warning during build:

Nuxt config:

Thank you.

No longer getting this with PrimeVue 4.0.0-beta.5
Closing.


Better building name ESP

Is it possible to find out the exit of a given teleporter entrance? That would allow to create some cool features


Doc that Init is not required if you use the contract API

Signed-off-by: NIKHIL E GUPTA<EMAIL_ADDRESS>Type of change

Documentation update

Description
Clarify that users to not need to use the init function if they use the Fabric contract API. Part of directing users away from using it.

@denyeart @mastersingh24 Mea culpa on this. This section has been incrementally changed until it has become unreadable, something the old review system unfortunately encouraged. I finally took the step to re write the whole section, and hopefully it makes more sense:
"If you use the low level APIs provided by the Fabric Chaincode Shim API, you need
to include an Init method in your chaincode that will serve as the initialization
function. This function is required by the chaincode interface, but does not
need to invoked by your applications. The chaincode will start and be ready to use
after the chanced definition is committed to the channel, without the Init function
being called. However, you can request that Init become an executable function.
Fabric will ensure that the Init function is invoked before any other function
in the chaincode and is only invoked once. Requesting the execution of the Init
function allows you to implement logic that is run when the chaincode is initialized.
You will need to call Init to initialize the chaincode every time the
chaincode is started or upgraded.
You can use the chaincode definition to request the execution of the Init
function by using the --init-required flag. You can then call the Init
function by using the peer chaincode invoke command and passing the --isInit
flag. For more information, see :doc:chaincode_lifecycle.
If you are using the Fabric contract API, you do not need to include an Init
method in your chaincode. If you use the --init-required flag to request the
that the chaincode be initialized by a call from your applications, you can
initialize the chaincode by calling any function in the chaincode."

/azp run

@Mergifyio backport release-2.1


feat(nixos/maintainers): add myself under maintainers

This PR lists myself under maintainers so that I can reference myself when building new packages.

You can just do it in the PR when you add yourself as a maintainer to a package.
Usually we call the commit that does this maintainers: Add uroshercog.

Thank you for the response. I'm new and didn't know this, also didn't see it in the docs (I should probably this).
I'll close this issue now and reopen one with the module.
Thanks!


Automatic display of workshops when selecting filters

Story link
#186 Story
Task description
As a user, I want the available workshops to be automatically displayed when I select filters without having to refresh the entire page to find the information I need faster.
To do

[ ] The system should automatically update the list of circles after changing any of the filters without reloading the page.

[ ] Filters should affect the display of available circles in real time.

https://github.com/ita-social-projects/OoS-Frontend/issues/2676


Update jquery.businessHours.js

if someone want only to display the program, and the user cannot change it

@giannisdag thx

be welcome


Error evaluating 'LockModule.init'

Hi,
I'm trying to use your module yet I hit this error (followed your instructions 100%)

Any resolutions to this?
Thanks

@aviadhahami can you provide a little more context information like how you are instantiating Lock in your code?

@hzalaz yes, sure (sorry for not doing this initially).
I have a component called App which instantiates the Auth0 module like this (nothing fancy):
import React, { Component } from 'react'
import {View, Text} from 'react-native'
import Auth0Lock from 'react-native-lock'

class App extends Component{
	constructor(props){
		super(props)
		this.state = {
			profile:'',
			token:'',
		}
	}

	componentWillMount(){
	}
	componentDidMount(){
		let lock = new Auth0Lock({clientId: 'xxxxxx', domain: 'aviadhahami.eu.auth0.com'});
		lock.show({}, (err, profile, token) => {
			if (err) {
				console.log(err);
				return;
			}
			// Authentication worked!
			console.log('Logged in with Auth0!');
			this.setState({
				profile: profile,
				token: token,
			})
		});
	}

	render(){
		console.log(this.state.profile)
		console.log(this.state.token)
		return (
			<View>
				<Text>
					UN: {this.state.profile.name}
				</Text>
				<Text>
					Tkn: {this.state.token}
				</Text>
			</View>
		)
	}
}

export default App

My package.json has this:
"dependencies": {
    "react": "^15.1.0",
    "react-native": "0.27.2",
    "react-native-lock": "^0.3.0"
  }

Rather than this, I've followed what you guys stated on your Readme (regarding the gradle and the manifest)

I ran in to same issue. I did installation using rnpm  rnpm link react-native-lock. After opening the new  workspace created by cocoapods I got this same error. I got it working when I manually added A0RNLock.xcodeproj from under node_modules/react-native-lock/ios/ to my libraries and also added libA0RNLock.a to "link binary with libraries".
I wonder if this step is missing from the instructions or should this be happening automatically?

@jniemin imo it should happen during the rnpm link process..
@hzalaz - Any updates?

+1

found the problem. when running the rnpm link automation it's skipping a stage.
i followed the manual instructions in the readme.md file to see what is wrong.
apparently, the LockReactPackage is missing from the 'getPackages' method.
it should be in MainApplication.java instead of MainActivity.java as instructed (and probably as the linker is trying to init).
MainApplication.java:
... import com.auth0.lock.react.LockReactPackage; ... @Override     protected List<ReactPackage> getPackages() {       return Arrays.<ReactPackage>asList(           new MainReactPackage(),           new LockReactPackage()       );     } ...

The first time I read this, I updated MainActivity.java because I didn't read it the whole way.
Don't make that mistake. It goes in MainApplication.java.
@sagic This worked for me as well once I got the file right!
Thanks

This is because RN introduced yet another breaking change in v0.29 by moving the place where native modules are registered to a custom Android Application object. As @lanceblais said, this should be done in MainApplication.java (We'll update the manual installation steps).
If you are using rnpm, seems it's bundled in react-native-cli, so calling react-native link is the recommended step now (until it changes again). Just make sure you are using version 0.29.2 of react-native since they fixed rnpm to work with MainApplication.java.
In iOS rnpm (or react-native link now) should handle adding the xcodeproj itself and I see no breaking changes in iOS so far.
Closing this now, feel free to open new issues for iOS if it's not working with some details

@sagic I tried what you suggested and am still getting the error :( I'm on Android as well. MainApplication.java looks like this.
`
import com.auth0.lock.react.LockReactPackage;
public class MainApplication extends Application implements ReactApplication {
private final ReactNativeHost mReactNativeHost = new ReactNativeHost(this) {
@Override
protected boolean getUseDeveloperSupport() {
return BuildConfig.DEBUG;
}
@Override
protected List<ReactPackage> getPackages() {
  return Arrays.<ReactPackage>asList(
      new MainReactPackage(),
      new LockReactPackage()
      new OAuthManagerPackage(),
      new FirestackPackage(),
      new RCTCameraPackage()
  );
}

};`

If you see this error on iOS, I posted the fix that worked for us at:
https://github.com/auth0/react-native-lock/issues/133

Thanks a lot: I had this issue on Ubuntu because while trying to link I always had an issue because of CocoaPods (impossible to install)
So actually the mannual installation steps fixed the issue for me!

For those who are going down the same path as I did and found this issue. Please note that the create-react-native-app project is not intended for projects that need to link to native code such as this one.


SchemaStorage::getSchema() should not add schema

The method SchemaStorage::getSchema is currently not just getting a schema, but adding if missing.
    public function getSchema($id)
    {
        if (!array_key_exists($id, $this->schemas)) {
            $this->addSchema($id);
        }

        return $this->schemas[$id];
    }

Based on the documentation of the interface this is wrong:
Returns schema for given identifier, or null if it does not exist

@estahn The behavior is correct. The addSchema step is used to load remote schemas when they are first needed. Note that 'doesn't exist' is not the same thing as 'not currently loaded'.
The documentation could definitely use some clarification :-).

@erayd Fair enough.


LIFX color temperature is incorrectly set

Make sure you are running the latest version of Home Assistant before reporting an issue.
You should only file an issue if you found a bug. Feature and enhancement requests should go in the Feature Requests section of our community forum:
Home Assistant release (hass --version):
0.25
Python release (python3 --version):
3.5
Component/platform:
lights/lifx
Description of problem:
The LIFX component has a FIXME about a missing mir to Kelvin conversion for colour temperature - this missing call ends up with the wrong colour temperature being sent to the bulbs. Setting a color temperature from the front end results in a different temperature to the one that was intended being set on the bulb (this also causes the frontend to show the wrong color temp again, after two incorrect conversions during the round trip).
Expected:
Color temperature correctly set on the bulb.
Problem-relevant configuration.yaml entries and steps to reproduce:

Set color temperature in the front end
Observe vastly different colour on the bulb
Observe front end snapping to another different colour temperature

Actually, after a bit of logging tonight, the correct color temperature is being sent and received after massaging into Kelvin/Mir. The issue seems to be that the front-end doesn't update to reflect the change in temperature - it will apply the update to the bulb but the slider in the device info will immediately snap back to its previous position.

The frontend updates to the state of the state machine. We should probably
add the same two seconds delay till we update as we have for on/off
On Mon, Aug 1, 2016, 04:28 Dean Camera<EMAIL_ADDRESS>wrote:

Actually, after a bit of logging tonight, the correct color temperature is
being sent and received after massaging into Kelvin/Mir. The issue seems to
be that the front-end doesn't update to reflect the change in temperature -
it will apply the update to the bulb but the slider in the device info will
immediately snap back to its previous position.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/home-assistant/home-assistant/issues/2686#issuecomment-236555997,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABYJ2v4D7dzrrC0Pa-xJka1aIeTEuFDMks5qbdhMgaJpZM4JZJVk
.

I believe adding the 2s delay for reading the update after setting state should also help with the current issue where sliders (brightness and temperature) "jump" back to the old position right after setting a new value

Has this been fixed?


Failed to work with repo via SSH

Dear Sander,
This is the first time I use your extension, and I got an issue right at step "Gerrit: Check connection"
The URL to my repo is not HTTP, but SSH.
And it seems that your extension doesn't support SSH at the moment.
My question: Is it a bug? If not, then it would be very helpful (for me and lots of people) if SSH can be supported in the furture.
Note: Because this a business repo, I'm allowed to access it via SSH only.
Here is the output log:
GET request to "ssh://xxx@vgit.yyy.com:29420/config/server/version" UnsupportedProtocolError: Unsupported protocol "ssh:"    GET request to "ssh://xxx@vgit.yyy.com:29420/a/changes/I032887887115ebd0d0fd49527a0eedf3f5d6af8a/detail/" UnsupportedProtocolError: Unsupported protocol "ssh:"    Invalid response GET request to "ssh://xxx@vgit.yyy.com:29420/a/changes/I032887887115ebd0d0fd49527a0eedf3f5d6af8a/detail/" UnsupportedProtocolError: Unsupported protocol "ssh:"    Invalid response GET request to "ssh://xxx@vgit.yyy.com:29420/a/changes/I032887887115ebd0d0fd49527a0eedf3f5d6af8a/detail/" UnsupportedProtocolError: Unsupported protocol "ssh:"    Invalid response GET request to "ssh://xxx@vgit.yyy.com:29420/a/accounts/self" UnsupportedProtocolError: Unsupported protocol "ssh:"    Invalid response GET request to "ssh://xxx@vgit.yyy.com:29420/a/accounts/self" UnsupportedProtocolError: Unsupported protocol "ssh:"    Invalid response GET request to "ssh://xxx@vgit.yyy.com:29420/a/config/server/version" UnsupportedProtocolError: Unsupported protocol "ssh:"    GET request to "ssh://xxx@vgit.yyy.com:29420/a/changes/" GET request to "ssh://xxx@vgit.yyy.com:29420/a/changes/" GET request to "ssh://xxx@vgit.yyy.com:29420/a/changes/" GET request to "ssh://xxx@vgit.yyy.com:29420/a/changes/" GET request to "ssh://xxx@vgit.yyy.com:29420/a/changes/" UnsupportedProtocolError: Unsupported protocol "ssh:"    UnsupportedProtocolError: Unsupported protocol "ssh:"    UnsupportedProtocolError: Unsupported protocol "ssh:"    UnsupportedProtocolError: Unsupported protocol "ssh:"    UnsupportedProtocolError: Unsupported protocol "ssh:"    GET request to "ssh://xxx@vgit.yyy.com:29420/a/changes/I032887887115ebd0d0fd49527a0eedf3f5d6af8a/detail/" GET request to "ssh://xxx@vgit.yyy.com:29420/a/changes/I032887887115ebd0d0fd49527a0eedf3f5d6af8a/detail/" GET request to "ssh://xxx@vgit.yyy.com:29420/a/changes/I032887887115ebd0d0fd49527a0eedf3f5d6af8a/suggest_reviewers/" GET request to "ssh://xxx@vgit.yyy.com:29420/a/changes/I032887887115ebd0d0fd49527a0eedf3f5d6af8a/suggest_reviewers/" GET request to "ssh://xxx@vgit.yyy.com:29420/a/changes/I032887887115ebd0d0fd49527a0eedf3f5d6af8a/drafts/" GET request to "ssh://xxx@vgit.yyy.com:29420/a/accounts/self" GET request to "ssh://xxx@vgit.yyy.com:29420/a/changes/I032887887115ebd0d0fd49527a0eedf3f5d6af8a/revisions/current//mergeable" UnsupportedProtocolError: Unsupported protocol "ssh:"    UnsupportedProtocolError: Unsupported protocol "ssh:"    UnsupportedProtocolError: Unsupported protocol "ssh:"    UnsupportedProtocolError: Unsupported protocol "ssh:"    UnsupportedProtocolError: Unsupported protocol "ssh:"    UnsupportedProtocolError: Unsupported protocol "ssh:"    UnsupportedProtocolError: Unsupported protocol "ssh:"    Invalid response Invalid response Invalid response Invalid response Invalid response Invalid response Invalid response GET request to "ssh://xxx@vgit.yyy.com:29420/a/config/server/version" UnsupportedProtocolError: Unsupported protocol "ssh:"    GET request to "ssh://xxx@vgit.yyy.com:29420/a/changes/I032887887115ebd0d0fd49527a0eedf3f5d6af8a/detail/" UnsupportedProtocolError: Unsupported protocol "ssh:"    Invalid response GET request to<EMAIL_ADDRESS>HTTPError: Response code 404 (Not Found) 404 [object Object] Not Found GET request to "ssh://xxx@vgit.yyy.com:29420/a/changes/I032887887115ebd0d0fd49527a0eedf3f5d6af8a/detail/" UnsupportedProtocolError: Unsupported protocol "ssh:"    Invalid response GET request to<EMAIL_ADDRESS>HTTPError: Response code 404 (Not Found) 404 [object Object] Not Found GET request to "ssh://xxx@vgit.yyy.com:29420/a/changes/I032887887115ebd0d0fd49527a0eedf3f5d6af8a/detail/" UnsupportedProtocolError: Unsupported protocol "ssh:"    Invalid response

This is intentional. The SSH API supports significantly fewer features than the HTTP API, and a lot of those are very essential to the extension working, so it's not really possible to have the extension work with "just" the SSH API.
Out of interest, is there no HTTP URL to your repo at all? I'd assume you use the web interface too right? Or is the HTTP API specifically disabled for that?
(will close for now since the issue's not really actionable but feel free to keep commenting)

Thank Sander,
"is there no HTTP URL to your repo at all?" No, I actually have the HTTP URL, but due to the security policy of the company, I can only use SSH URL at the moment. Maybe I'll request HTTP URL permission.
Anyway, your answer is clear. I have nothing to ask more. Thanks a lot


Roll Flutter Engine from 44309c9f385c to 4dc54074d0cd (1 revision)

https://github.com/flutter/engine/compare/44309c9f385c...4dc54074d0cd
2023-10-30<EMAIL_ADDRESS>[macOS] Wrap external texture tests in autorelease pool (flutter/engine#47492)
If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Please CC aaclarke@google.com,rmistry<EMAIL_ADDRESS>on the revert to ensure that a human
is aware of the problem.
To file a bug in Flutter: https://github.com/flutter/flutter/issues/new/choose
To report a problem with the AutoRoller itself, please file a bug:
https://issues.skia.org/issues/new?component=1389291&template=1850622
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md

Commit queue failed; closing this roll.


support a custom eventindexer api

Would be nice to have a custom eventindexer api option in the connections page, as a backup or for not having to rely on the official one.

#15


Automatically sort the input sequence in links.loss.CRF1d

[x] flake8
[x] compatibility
[x] test

CRF1d needs to sort the input sequences
in descending order of lengths and transpose the sequences.
But in NStepLSTM, this procedure is not needed
because links.connection.NStepRNNBase already does.
It would be better that CRF1d also does this, which allows users
to concentrate on defining the network architecture.
So I changed the links.CRF1d to sort the input sequence automatically and send the PR.
After deal with the API compatibility policy, I'll remove WIP prefix.

I checked and updated the links.loss.CRF1d.argmax to return a list of Variable.
In this modification, this PR seems not to break compatibility.
So I removed WIP prefix, please check it.
There are two remaining which I can do.

There is no test for links.loss.CRF1d (if it is needed, I'll create some tests)
NStepRNN and CRF1d use the same permutate_list and argsort_list_descent

I think it would be better to unify but I'm not sure where module is the best for them.

cc: @aonotas @unnonouno

Check comments.

@unnonouno I'm very sorry but I could not find what I should do (I overlooked something important?).
Could you please tell me what I can check?

Thanks, I updated the PR.
BTW, what the situation does it break compatibility?
(I thought sorting and reindexing is totally done in forward and argmax, which keep the compatibility)

Sorry for late reply. If the default value of transpose argument is True, programs which use the old interface will be broken.

Can you write a test using transpose=True?

@unnonouno
Sorry for late reply...
I'm not sure what I should test ...
I'm sorry but could you please tell me what I should check or please take over my PR:bow:
(If you will take over my work, I'll close this PR:pray:)
By the way, I previously added transpose option to forward and argmax.
But now, I think it would be better to add transpose option to __init__.
Please please tell me what you thinks about it.

Now, I will make two changes to this PR.

Remove transpose option from __call__ and argmax and add it to __init__
Add test to check whether if the output from __call__ in transpose=False and it in transpose=True are the same

@unnonouno
Thanks for your advice, I updated my PR.

And, OK i'll take over this PR and write a unit test. Please keep this PR open. I'll send another one.
I got it, I keep this PR open :bow:

@unnonouno What is the status of this PR?
I'm sorry to leave it to you all.
If you are busy, please let me know how I should write tests :bow:

Now this PR should be merged after #5326

@unnonouno ping

https://github.com/chainer/chainer/pull/6351 took over this PR.
So, I close this PR, thanks a lot, @unnonouno :bow:


Reword intro for csi-driver-spiffe

Fix some broken English / grammar, and rewords things slightly

/hold
This is blocked behind https://github.com/cert-manager/csi-driver-spiffe/pull/25 which will update the examples.
Once that PR merges, I'll update the docs page to point at the new updated examples (since they're - correctly - referred to by commit SHA, it'll need to be updated)

/unhold
I added a hold in a previous comment, but then decided to go in a different direction

Linking here; https://github.com/cert-manager/csi-driver-spiffe/pull/25
That PR applies the same changes to the relevant part of the csi-driver-spiffe README


Resolve issue where withdrawal time changes

Trello: https://trello.com/c/GGxgDv3W/533-add-form-to-let-users-withdraw-a-document
We previously used the time a status was created to show a user when a
piece of content was withdrawn, however this is not accurate if you
update the explanation of a withdrawal as this creates a new status.
Thus the way I've approached resolving this is by passing a timestamp
among the details of Withdrawal objects so that this data is persisted.
If a user undoes a withdraw and re-withdraws this timestamp will be
reset.
Note: this migration relies on there not being any withdrawal records persisted, as is the case in production. In a dev environment it may well require a db:reset. For integration it should be straightforward if the data sync has ran, if not it’ll need any documents that have been withdrawn to be deleted or statuses changed

@benthorner thanks for review, all comments addressed


README: Fix link to EKF/ECL tuning guide

Fixes #576

@hamishwillee   The link not work:  https://docs.px4.io/en/advanced_config/tuning_the_ecl_ekf.html

Thanks @xdwgood - fixed by https://github.com/PX4/ecl/pull/599
Just FYI, the link worked when I fixed it and the old link did not. This broke due to addition of docs versioning for user guide.


docs: explain how to test single block/superblock

Checklist:

[x] I have read and followed the contribution guidelines.
[x] I have read and followed the how to open a pull request guide.
[x] My pull request targets the main branch of freeCodeCamp.
[x] I have tested these changes either locally on my machine, or GitPod.

Closes https://github.com/freeCodeCamp/freeCodeCamp/issues/49403

@ojeytonwilliams Hey how's this going :)

Hi,
We are closing old issues/pull requests that we think are stale. If you still need help or are fixing this, please open a new issue/pull request.
Thanks for your understanding.
Happy contributing.


Automated cherry pick of #3873: fix: 修复rds重置密码未正确接收参数问题

Cherry pick of #3873 on release/2.12.
#3873: fix: 修复rds重置密码未正确接收参数问题

Codecov Report

:exclamation: No coverage uploaded for pull request base (release/2.12@a322c4e). Click here to learn what that means.
The diff coverage is 0%.

@@              Coverage Diff               @@
##             release/2.12   #3875   +/-   ##
==============================================
  Coverage                ?   6.92%
==============================================
  Files                   ?     500
  Lines                   ?   84438
  Branches                ?       0
==============================================
  Hits                    ?    5850
  Misses                  ?   78094
  Partials                ?     494

Flag
Coverage Δ

#aFlag
6.92% <0%> (?)

Impacted Files
Coverage Δ

pkg/compute/models/dbinstance_accounts.go
2.54% <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 a322c4e...a5eb231. Read the comment docs.

/lgtm

/approve


Any way to debug map not showing on iOS?

Hi! Thank you for your time developing this plugin. Just recently, I started integrating this on a Cordova application, using Vue and VueRouter. I installed V2 with the proper map keys with no problems. However, the map is not showing for me using the following code:
<template>

</template>

The "LOADED" alert gets fired, no evident errors, but I can't see the map. iOS emulator console just shows:
2017-05-22 16:41:54.085 MyApp[98075:1466235] Google Maps SDK for iOS version: 2.2.30010.0
Running the app on my iPhone and debugging with Safari Developer Console, I just see the div with nothing inside:

Can you help me see what am I doing wrong? Any way to further debug this?
Thank you very much!

map.showDialog() is not available in the v2 plugin.

And the safari debugger does not keep errors before you open it.
For that time, you need to reload your html using the safari debugger, then the safari debugger captures the errors.
Please try it. Maybe you will get some JS errors (I guess).

@wf9a5m75 Thank you for your quick answer! Did as you told me and sure enough, an error is revealed
TypeError: map.showDialog is not a function. (In 'map.showDialog()', 'map.showDialog' is undefined)
Because as you said map.showDialog() is not available in V2. Removed that line, error goes away. No more errors and still no map :/ any other idea?
Thanks!

I see. Maybe like ionic, you need to fire the plugin_touch event.
https://github.com/mapsplugin/cordova-plugin-googlemaps/issues/1454#issuecomment-302461067
var map = plugin.google.maps.Map.getMap(div, { ... });
  map.on(plugin.google.maps.event.MAP_READY, function() {

     setTimeout(function() {
       cordova.fireDocumentEvent('plugin_touch', {});
     }, 200);
  });

@wf9a5m75 Thanks again for your help. Tried adding what you told me, still no difference. Tried emulating on Android, same problem. Could this be an issue related on how Vue Router renders the view? I've read issues where the view overlays the map. Tried manually putting styles like z-index: -1 and background-color: rgba(0,0,0,0) to all the parent elements, but still no luck :(

Since I'm not familiar with the Vue framework at all, I can not detect what the wrong point is so far.
In order to inspect your case more deeply, please share your project file (all project files) using github or bitbacket (or other git repository services are acceptable).
(Don't send me a zip file)

have you tried setting width and height of #map_canvas in px?

ping

we had the same issue just keep the height 99.99vh it works

Since there is no update, I close this issue.

"@wf9a5m75 Thanks again for your help. Tried adding what you told me, still no difference. Tried emulating on Android, same problem. Could this be an issue related on how Vue Router renders the view? I've read issues where the view overlays the map. Tried manually putting styles like z-index: -1 and background-color: rgba(0,0,0,0) to all the parent elements, but still no luck :( "

This works for me, I'm using onsen for the UI so the problem was that the map was rendering behind the div that is used for display the background, and also behind the div that is used for display the content.
Thanks.


Moved greedo-layout-for-ios to /Image

I also thought we could put under /TableView & CollectionView
Project URL

https://github.com/500px/greedo-layout-for-ios
Category

/Image
Description

Full aspect ratio grid layout for iOS

          1 Warning

:warning:
Found 1 link issue, a project collaborator will take care of these, thanks :)

Link issue by awesome_bot

Line
Status
Link

2877
302
http://www.ios-blog.co.uk/ redirects tohttp://www.ios-blog.co.uk/cgi-sys/suspendedpage.cgi

  Generated by :no_entry_sign: Danger


Reference module from GoogleCloudPlatform

To match the links in the guide at https://cloud.google.com/solutions/using-vault-for-secret-management

I signed it! (As<EMAIL_ADDRESS>

@madmod great catch, thanks!


Updated romanian translation

Thanks for the update! I will incorporate t in the next days...

The updated romanian translation is live. Thanks again!


Test IntelliSense sorting

Test for: https://github.com/Microsoft/vscode/issues/23643
Complexity: 1

[x] Any OS - @rebornix

IntelliSense results are not sorted by how well they match the text left of the cursor. Test that

suggestions are sorted nicely while typing
suggestion are sorted lexicographical when there is no text left of the cursor
snippet sorting rules (top, inline, bottom) still apply

The sorting result looks reasonable to me.


fix(tests): fix some untyped codes and unused descriptions

Checklist

[x] npm test passes
[x] tests and/or benchmarks are included
[x] commit message follows commit guidelines

Description of change

remove deprecated docs
remove unused codes
codes releated with ShapeStyle not changed, it will work after core released

感谢 PR！


bar series with polar and markPoint will error

Version
Reproduction link
https://codepen.io/lgl017/pen/LYWBQXy
Steps to reproduce
see codepan
What is expected?
chart show normally
What is actually happening?
clampData is not a function

I am temporarily modifying the code to make it work, please implement polar coordSys.clampData, or add a judgment to aviod error

Hi, I'm working on this project and now trying to solve some easy tasks to get familiar with echarts.


Remove src folder from npm package

This will reduce the size of the package :+1:

Sorry we will not do this. It will break npm build and rebuild as well
install for many users.
On 22 Aug 2016 8:20 PM, "Evilebot Tnawi"<EMAIL_ADDRESS>wrote:

This will reduce the size of the package 👍
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/sass/node-sass/issues/1674, or mute the thread
https://github.com/notifications/unsubscribe-auth/AAjZWEdkzaEPkvolsfxvx4VFFg2b7QU_ks5qiXgHgaJpZM4JptuT
.


Export content

I removed it because there the value was not used anywhere. The export is automatically disabled when there are no data.


Incorrect @var name removal in multi-line inline phpdoc

The PHP version you are using ($ php -v):
7.4
PHP CS Fixer version you are using ($ php-cs-fixer -V):
2.15.1
The command you use to run PHP CS Fixer:
./vendor/bin/php-cs-fixer fix --config=.php_cs.dist
The configuration file you are using, if any:
<?php

return PhpCsFixer\Config::create()
    ->setRules([
        '@Symfony' => true,
    ])
    ->setFinder(
        PhpCsFixer\Finder::create()
            ->in(__DIR__.'/src')
            ->in(__DIR__.'/tests')
    )
;

If applicable, please provide minimum samples of PHP code (as plain text, not screenshots):

before running PHP CS Fixer (no changes):

/**
 * @var \DateTime $dateTime
 * @var State     $state
 */
foreach ($states as [$dateTime, $state]) {
    // ...
}

with unexpected changes applied when running PHP CS Fixer:

/**
 * @var \DateTime
 * @var State     $state
 */
foreach ($states as [$dateTime, $state]) {
    // ...
}

with the changes you expected instead: (no changes)

/**
 * @var \DateTime $dateTime
 * @var State     $state
 */
foreach ($states as [$dateTime, $state]) {
    // ...
}```

---

I discovered the `PhpdocVarWithoutNameFixer` caused this, as it assumes all inline PHPdoc comments are single line. We cannot make this single line and afaik, defining two vars for this foreach is perfectly fine (at least phpstorm supports it).

I'm not sure how to fix it (maybe check if there is more than 1 var tag?).

Nvm, should have clicked on the suggested titles instead of blindly clicking "submit". Closing in favor of https://github.com/FriendsOfPHP/PHP-CS-Fixer/issues/3112


SdkWorkflow.fetch is broken

wf = SdkWorkflow.fetch(project, domain, name, version)
returns:
Traceback (most recent call last):
File "app/workflows/registerar_workflow.py", line 70, in
bump_cache_versions.unit_test(project="flytekit", domain="development", name="tests.flytekit.common.workflows.batch.BatchTasksWorkflow", version="49aa6fefb9d718b2cceb418b56987a4bd68eae18", desired_cache_version="")
File "/Users/hamabuelfutuh/.virtualenvs/flyteexamples/lib/python2.7/site-packages/flytekit/common/exceptions/scopes.py", line 160, in system_entry_point
_reraise(ex.type, ex.value, ex.traceback)
File "/Users/hamabuelfutuh/.virtualenvs/flyteexamples/lib/python2.7/site-packages/flytekit/common/exceptions/scopes.py", line 163, in system_entry_point
return wrapped(*args, **kwargs)
File "/Users/hamabuelfutuh/.virtualenvs/flyteexamples/lib/python2.7/site-packages/flytekit/common/workflow.py", line 169, in fetch
sdk_workflow = cls.promote_from_model(admin_workflow.closure.compiled_workflow.primary.template)
File "/Users/hamabuelfutuh/.virtualenvs/flyteexamples/lib/python2.7/site-packages/flytekit/common/workflow.py", line 189, in promote_from_model
for n in base_model.nodes
File "/Users/hamabuelfutuh/.virtualenvs/flyteexamples/lib/python2.7/site-packages/flytekit/common/workflow.py", line 189, in
for n in base_model.nodes
File "/Users/hamabuelfutuh/.virtualenvs/flyteexamples/lib/python2.7/site-packages/flytekit/common/nodes.py", line 188, in init
non_none_entities
flytekit.common.exceptions.user.FlyteAssertion: An SDK node must have one underlying entity specified at once.  Received the following entities: []

This has been fixed.


Remove ldap-modify script and replace with community.general.ldap_ tasks

https://github.com/SURFscz/SRAM-deploy/issues/360
Depends on
https://github.com/ansible-collections/community.general/pull/5385 (merged in 5.8.0)
https://github.com/ansible-collections/community.general/pull/5435
https://github.com/ansible-collections/community.general/pull/5450

All necessary PR's have been merged in community.general 6.0.0


fix network remove

Remove network fix.
Network takes time to be really removed after docker/docker rm command (>20 secondes)
added loop to wait it's really removed before continue.
This issue make docker engine in bad shape after 'make test' command.
test: make test, several times

@freignat91 Please rebase / resolve conflicts and I'll merge. Thanks!

This fix has been merged with the PR#304 on which the fix has been reported.
So we can just close it.


Generate Haddock for test-suite components

Although the '--haddock-all' flag was set and Haddock was generated for all components including test packages, this documentation was not properly copied to the output directory. This small change ensures this is the case. This PR can be a useful complement to #4248.
Checklist

Branch

[x] Commit sequence broadly makes sense
[x] Commits have useful messages
[ ] New tests are added if needed and existing tests are updated
[ ] If this branch changes Consensus and has any consequences for downstream repositories or end users, said changes must be documented in interface-CHANGELOG.md
[ ] If this branch changes Network and has any consequences for downstream repositories or end users, said changes must be documented in interface-CHANGELOG.md
[ ] If serialization changes, user-facing consequences (e.g. replay from genesis) are confirmed to be intentional.

Pull Request

[x] Self-reviewed the diff
[x] Useful pull request description at least containing the following information:

What does this PR change?
Why these changes were needed?
How does this affect downstream repositories and/or end-users?
Which ticket does this PR close (if any)? If it does, is it linked?

[ ] Reviewer requested

bors r+


Fixed double free.

Codecov Report

Merging #20 into master will increase coverage by 0.09%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master      #20      +/-   ##
==========================================
+ Coverage   61.57%   61.66%   +0.09%
==========================================
  Files           4        4
  Lines         242      240       -2
  Branches       44       43       -1
==========================================
- Hits          149      148       -1
  Misses         61       61
+ Partials       32       31       -1

Impacted Files
Coverage Δ

src/wrp_interface.c
50% <ø> (ø)
: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 abd022a...aad308e. Read the comment docs.

Looks correct, you may need to a dd a comment to the header file that the message must be freed by the caller. (If the comment isn't there already ;-)

This is the message allocated by libparodus_receive() and it is correctly commented so in libparodus.h. I was being overzealous in freeing the message one too many a time. :-)


Resolve issue #664, exception encountered

Resolve issue #664, exception encountered when using the --single-instance option. This refactors the deprecated call to app.makeSingleInstance(). See https://electronjs.org/docs/all?q=app.requestSingleInstanceLock.

Closing, will resolve lint issues and make another PR.


[MINOR] Remove unused import. Naming convention.

What is this PR for?
Change method name (Method names should comply with a naming convention).  Remove unused imports. Using append makes code simpler to read sb.append("\nAccessTime = ").append(accessTime); instead sb.append("\nAccessTime = " + accessTime);
What type of PR is it?
[Refactoring]
Todos

[ ] - Task

What is the Jira issue?

https://issues.apache.org/jira/browse/ZEPPELIN-1839

How should this be tested?
HDFSFileInterpreterTest.java (still working)
Questions:

Does the licenses files need update? (no)
Is there breaking changes for older versions? (no)
Does this needs documentation? (no)

Remove this unused import,  naming convention

Could you please update the title and the PR description with high level description of your changes even if we still mar it as [MINOR].

@lresende Sure.

Overall, the code changes LGTM.
As for the PR, I would recommend :

Add [MINOR] to the title, as this is going to be the commit title
Respond the "What is this pr for"
Respond what type of PR this is
Remove todos and all other sections below

Then it should be good to go.

@lresende is it right?

LGTM. Merging if there's no more discussion


Implement all filter effects.

[x] FEBlend
[x] FEColorMatrix
[x] FEComponentTransfer
[x] FEComposite
[ ] FEConvolveMatrix
[ ] FEDiffuseLighting
[x] FEDisplacementMap
[ ] FEDropShadow
[x] FEFlood
[x] FEFuncA
[x] FEFuncB
[x] FEFuncG
[x] FEFuncR
[x] FEGaussianBlur
[x] FEImage
[x] FEMerge
[x] FEMergeNode
[ ] FEMorphology
[x] FEOffset
[ ] FESpecularLighting
[x] FETile
[x] FETurbulence
[x] FENone

I found this library while studying Manin and I have to say this is a very fine piece of work, congratulations. Do you have any reference about this effects, I'm skimming through the issues to see if I can contribute with the project. Regards.

Hi @joseedil.
The filter effects are documented by Mozilla. For example, you can find the documentation for <feBlend> here: https://developer.mozilla.org/en-US/docs/Web/SVG/Element/feBlend
To implement <feBlend> in Haskell, we create a data structure which contains the specific attributes and possible children (there are no children in this case). Concretely, the FEBlend data structure should contain the in, in2 and mode attributes.
This work has been completed for FEColorMatrix, FEComposite, FEDisplacementMap, FEGaussianBlur, and FETurbulence. All the other nodes are still missing. Without a Haskell representation for the missing nodes, it's quite limited which filter effects can be used in reanimate.
Note that this ticket is actually for this repository: https://github.com/reanimate/reanimate-svg

This could be a good first task as it is not too difficult and no design decisions have to be made; it is just a bit tedious to do. I can guide you through the process if you want to work on this.

Still would be a useful addition to the project. I'm studying your code base, I'm not too keen on SVGs.
Let me see if I correctly understood the thing, in the end an SVG is just a XML text file and the functions on Graphics.SvgTree.Printer serialize the internal SVG representation to a String which is saved during the generateFrames call in Reanimate.Render.
I just did not get how the actual raster images are rendered, the heavy work is lifted to ffmpeg?

Got it after looking better how the Raster data type is handled in the render module. Now I understand better the documentation comments about caching and other optimizations.

@Lemmih Let me know how would you check/test the implementations.

reanimate-svg is lacking an automated test suite. To test the filters, I just write animations that use them.

I just checked out other Haskell SVG libraries, but could not find any test suites. W3C has a markup check service (https://validator.w3.org/) which I'm thinking about how to use for automated testing.
I also studied the code base, if I got it right every constructor of the FilterElement data type has to be extended with the corresponding fields. In addition the XMLUpdatable instance for the FilterElement and its internal fields has to be extended for each implemented filter effect as well as the lens generators.
Am I missing something related to this issue?
I'm not a Haskell expert, but I'm missing contributing to nice projects which force me to study more. Reanimate relates very much to one research project I work in partnership with a Swedish university which is a nice side effect of understanding it better (besides being much more amenable than Manim IMHO).

Don't think you're missing anything. The data types, the XMLUpdatable instances, and the lens instances should be all that is needed for this ticket.
Working on the testsuite might be even more impactful, though. I've written a bit about it here: https://github.com/reanimate/reanimate-svg/issues/1
There are SVG test files which cover the entire spec. We can do two things with those files:

Check that parse . print . parse == id. This should be fairly quick to run.
Check that print . parse doesn't affect how the SVG is rendered. This is a bit more expensive to run.

We could do some pair programming to knock these tickets out of the park.

@Lemmih May I ask you to check this commit?
I added the feBlend implementation and got the whole thing compiling after a while but when checking the dump against this test file I don't think it got the filters right and I couldn't find so far where to look for.
Note that the dumped tree seems to get the filter tag in line 56 of the SVG but not its children which actually define the filter.
Looks like the serialiser ignores tags it does understand, which is quite understandable, but seems to be a bug for loading SVGs which comply to this standard. I have not yet checked the creation of SVGs from the primitive constructors.
filters-blend-01-b.svg.txt
filters-blend-01-b.dump.txt

Maybe something related to the Tree XMLUpdatable instance?

Ah, yes, don't worry about that. The parser is very broken ATM. I've only been adding printing support for a while now. Your code looks really good.

Just submitted a pull request so you can check if this is the right track. I figured out how to extend the parser so now it parses the filter effects as well. If this is it I'll proceed with the others.

Code looks perfect. FEBlend is done.

Just submitted another pull request. In total, the following effects are done:
feBlend
feColorMatrix
feComponentTransfer
feComposite
feDisplecementMap
feFlood
feFuncA
feFuncR
feFuncG
feFuncB
feGaussianBlur
feImage
feMerge
feMergeNode
feOffset
feTile
feTurbulence
feNone

Looks good. Will test, write some examples, and then update this ticket.

I started to expand the library and refactor somethings to organize it better. Almost all attributes are there as well as the CSS classes used in SVG tags.
So far it broke the parser and I did not submitted because of that. I could keep working on it, but there are probably more useful things you might recommend putting some effort on.

We could merge the code to a branch and then work on it together. Making reanimate-svg rock solid is probably the most important task at the moment.
Here's a list of other things I'll be working on. Maybe some of them will interest you:

Filter effects tutorial. This will show off your work and demonstrate how to easily manage chunks of hand-written SVG.
Bind external data sets such as OpenMoji. Ideal interface: openMoji :: String -> SVG. The functions in Reanimate.External makes it easy to download such data sets on demand.
Building a tool for creating shapes with boolean operators (eg. union, intersection, exclusion, difference). The core operations are defined here: https://hackage.haskell.org/package/cubicbezier-<IP_ADDRESS>/docs/Geom2D-CubicBezier-Overlap.html. Right now it's quite a chore to define shapes that could easily be described by boolean operations over rectangles, circles and ellipses.
Tools for making presentations with reanimate. I think reanimate would be great for captivating presentations.

How do I submit a pull request to a branch that does not exist in the original repo?
I can help with the tutorial in the next days, will be good for me to better understand the matter as well.
It would probably be good to open issues with these ideias on reanimate-svg repo and migrate others like #81, #127 and #97.

I'm still a little lost in the repos structures. The ideia is to keep in reanimate-svg only the basic data structures, parser and serialization functions for SVGs and in reanimate the facilities to build and manipulate those data structures (mkCircle, mkRect, etc)?
Would not make more sense to have everything related to static SVGs in reanimate-svg, including the renderSvg function, and keep reanimate dealing only with the Animation and Scene data structures and functions?

I'm still a little lost in the repos structures. The ideia is to keep in reanimate-svg only the basic data structures, parser and serialization functions for SVGs and in reanimate the facilities to build and manipulate those data structures (mkCircle, mkRect, etc)?
Would not make more sense to have everything related to static SVGs in reanimate-svg, including the renderSvg function, and keep reanimate dealing only with the Animation and Scene data structures and functions?

Yes, it would make a lot more sense. The current organization is an artefact of history. Early versions of reanimate used svg-tree (which I later forked into reanimate-svg) which wasn't my repository and I therefore couldn't add any functions to it.
Feel free to reorganize and move stuff from reanimate into reanimate-svg. Most (all?) of the SVG specific helper functions definitely fit better in reanimate-svg.

How do I submit a pull request to a branch that does not exist in the original repo?

Easiest way is to directly push your own branch:
git remote add upstream git://github.com/reanimate/reanimate.git
git push upstream your-branch-here


Fix #114

ok

This only changes the tests, not the runtime code?

It's my fault of tiredness. Now it is fixed.


Eventhubs: Added validator for rights

Fix for Issue :  https://github.com/Azure/azure-cli/issues/5824
This checklist is used to make sure that common guidelines for a pull request are followed.

[ ] The PR has modified HISTORY.rst describing any customer-facing, functional changes. Note that this does not include changes only to help content. (see Modifying change log).

[X] I adhere to the Command Guidelines.

/azurepipelines run


#1016: Add text justification feature

I added as constructor option the marginRight to be able to justify the right margin.
If specified the justification will be on the pageWidth - x (pos) - marginRight.

@gpietro Thanks for this, the build seems to fail in IE for some reason?

@MrRio I have seen the failing check but I don't know for which reason. It could be that IE tests needs to be updated. My bet is that IE threats Tw PDF property differently. The code changes are minimal, with a 0 Tw line, added in the pdf output.

It seems get this output:
28.35 566.93 Td
(This is a test!) Tj

When it expects:
28.35 813.54 Td
0 Tw
(This is a test) Tj

tbh this justification method is not well written. I wrote already a proper justification method in my refactor text method PR, which repairs right allignment too, which passes all tests. #1448

@MrRio it looks like that the only difference with IE it's that when Tw is equal to 0 is removed from the output.
This:
28.35 813.54 Td
(This is a test) Tj

Instead of:
28.35 813.54 Td
0 Tw
(This is a test) Tj

This could lead to PDF rendering errors.
If @arasabbasi solution is already working and tested I would proceed that way.
@MrRio how much do you think it will take to merge his PR?
Thanks

I did update the algorithm of justification using number of spaces instead of words-1. This solved a problem where there could be more than one space concat in a text row and reduced the Tw coefficient.

Will be part of #1433


ffe-dropdown: Chevron does not render properly in IE10

Found a fix: add width and height attributes to the background image SVG. Included fix in #184.


Song progress and Webplaying are faulty

it isn't able to detect track progress and there isn't any option to remove it(As I type this I realized I can just alter its setting file but come on, you've made a classy settings panel for it).
As for the Chrome extension, It's just missing... shows error 404 from the given link and turns up nothing upon manual search.
hope you can fix these or atleast provide with a workaround, thank you.

This is a WNP issue, not Cleartext. Please open an issue on the WNP GitHub instead. I believe a new version of the WNP plugin is being worked on. For now you will have to manually install it.


Catalog

An toggle element  which display a tree of predefined WMS layers.
See http://api3.geo.admin.ch/examples/geoadmin_catalog.html for an example

We'd need also infobox for the "strukturverbesserungen" layers.
Thanks

I create an issue for that. #25

CSS problem for Infobox: class-modalgialog
Thanks

You can create separate issues  @p1d1d1   https://github.com/geoadmin/web-emapis/issues/26


Support for cratedb

Cratedb is created by some folks living near to me ;)
Notes:

95% of this code has been generated by GPT.
CLI does not work as the crate command doesn't run as root (therefore removed it), because user is intended to use crash.

used a newer version of GPT (haha), and added some things:

postgres wire protocl
crash interactive shell (devkit cratedb cli)

see feature branch: feature/cratedb. let me know if the changes are alright to merge

LGTM (where G stands for 'great'). How did U figure out/test arm64 compat?


Updated spring boot version to 1.4.3.RELEASE

Update application.properties datasource configuration
Update deprecated hibernate naming strategy (due to hibernate 5)
Update spring security schema to spring-security-4.1
Update swagger dependeny #277

see: https://github.com/spring-projects/spring-boot/wiki/Spring-Boot-1.4-Release-Notes

Coverage remained the same at 68.858% when pulling 39f584fd3ec08cf86eb247de0801d52d69c76e93 on spring-boot-update into 0554e1c05b372231d6dc101dcd7c4a53524b19d4 on master.

Coverage remained the same at 68.858% when pulling 39f584fd3ec08cf86eb247de0801d52d69c76e93 on spring-boot-update into 0554e1c05b372231d6dc101dcd7c4a53524b19d4 on master.

Coverage remained the same at 68.858% when pulling a040526727d31b71176f2907d3b8a190511e6d68 on spring-boot-update into 0554e1c05b372231d6dc101dcd7c4a53524b19d4 on master.


2 minutes delay to start to build each time

In UI I can see spinning circle on step to build image. It takes around 2 minutes. Building image after this 2 minutes is + 5-20 seconds depend on cache.
I have super simple Dockerfile for tests:
FROM nginx
I have installed concourse by helm in kubernetes.
Summary up: It takes around 2 minutes freeze each time to start.

I was trying it about 10 times with cache and without
each time UI spinning circle is showing and there is no output in task
after this this 2 minutes delay it start to build and success

How I run it in jobs:
      - task: build-image
        privileged: true
        config:
          platform: linux
          image_resource:
            type: registry-image
            source: {repository: concourse/builder-task}
          params:
            REPOSITORY: kwladyka/ory-hydra-auth-provider
            TAG: latest
            CONTEXT: auth-provider:master
          inputs:
          - name: auth-provider:master
          outputs:
          - name: image
          caches:
          - path: cache
          run: {path: build}

I have the same issue. With wait times as high as 17 minutes. I was hoping for the exact opposite in switching from the docker-image-resource.
INFO
the parts of the pipeline of interest
...

(the resources section)
- name: img
  icon: docker
  type: registry-image
  source:
    repository: a/repo
    username: myUser
    password: ((myPassword))

(the JOBS section)
- name: Deploy-ToTest
  serial: true
  plan:
  - get: repo-develop
    trigger: true
  # BUILD IMAGE
  - task: build-image
    file: repo-develop/concourse/tasks/container-img-develop-repo.yml
    privileged: true
  # PUSH IMAGE
  - put: img
    params:
      image: image/image.tar
      additional_tags: repo-develop/.git/ref
...

concourse

v5.1.0
1 worker
Concourse is running in Docker containers. 1 for the db, 1 for the web-instance and 1 for the worker

WHAT HAPPENS

the repo-develop step is executed
then wait, wait wait for 'x' amount of time. Up to 17 minutes as I write. As @kwladyka writes the UI spinning cirlcle cycles round and round
the build-image step is executed

However, I can go back to the main overview of configured pipelines in Concourse and then go back to the job I want to view the build output from and I can see it again. Strange.

What filesystem driver are y'all using for baggageclaim? Is it overlay or btrfs? overlay has known performance issues with privileged tasks.

part of the helm values
  worker:
    baggageclaim:
      # volumes:
      ## Driver to use for managing volumes.
      driver: overlay


Option to display a list of connections & servers on the right hand side

Please provide an option to display a list of connections & servers on the right hand side just like in a browser you can move developers tools from left to right, it would be great for the AZ Studio to have this flexibility as well
Thanks!

You can actually do this already! Right click on the side bar and select Move side bar right

(There's also a Toggle Side Bar Position command that can be ran from the command palette to accomplish the same thing)


Make gpu_cache optional

(This PR is based on #16 )
I think gpu_cache module and related features should be optional, so how is it to use "feature" feature?
This change will help solve also #12, by not turning on the gpu_cache feature on Redox os.
The feature is disabled by default, but enabled by building with cargo build --features="gpu_cache", and gpu_cache example can run by cargo run --example=gpu_cache --features="gpu_cache".
(See http://doc.crates.io/manifest.html#the-features-section )

https://travis-ci.org/dylanede/rusttype/builds/136678828#L184
I think CI test should be run with gpu_cache feature, by cargo test --features="gpu_cache".

master is merged, no conflicts!

I am requesting transfer to another maintainer, please read and comment here: https://github.com/dylanede/rusttype/issues/55

Oops, resolved conflits with old master...

Resolved conflicts with the current master.
I have two questions:

Should gpu_cache feature be default-enabled?

In this PR, currently default-disabled.

When gpu_cache feature is disabled, should gpu_cache example successfully compile and panic at runtime? Or should it fail to compile (with message by std::compile_error! macro)?

In this PR, currently the example compiles successfully and panic at runtime.


[g3log] Update g3log to 2.1

Updates g3log to version 2.1, I also updated the cmake commands as written in the mainter guide and updated the json deps.

What does your PR fix?

//

Which triplets are supported/not supported? Have you updated the CI baseline?

I personally tested it with x64-windows theorically it should work on all platforms that g3log supports.

Does your PR follow the maintainer guide?

I think yes

If you have added/updated a port: Have you run ./vcpkg x-add-version --all and committed the result?

Yes

Suggestions addressed

Thank you!


update dependencies part 2 (#5499)

Pull request opened by the merge tool on behalf of #5499

Merged in b872e55d50888dd055fcbf158382a4d0d3e0abce


Re-enables errorprone on JDK 10+

This uses the commandline instructions here:
http://errorprone.info/docs/installation
However, there's a glitch as I'm not sure why
-XepDisableWarningsInGeneratedCode doesn't work.
See https://github.com/google/error-prone/issues/860

Works with the problem of crashing on args (commented about this)
$ ./mvnw clean compile -pl brave
--snip--
[INFO] --- maven-compiler-plugin:3.8.0:compile (default-compile) @ brave ---
[INFO] Changes detected - recompiling the module!
[INFO] Compiling 50 source files to /Users/acole/oss/brave/brave/target/classes
[WARNING] [options] bootstrap class path not set in conjunction with -source 1.8
[WARNING] WARNING: An illegal reflective access operation has occurred
WARNING: Illegal reflective access by com.google.errorprone.bugpatterns.FutureReturnValueIgnored (file:/Users/acole/.m2/repository/com/google/errorprone/error_prone_ant/2.3.1/error_prone_ant-2.3.1.jar) to field com.sun.tools.javac.code.Type$StructuralTypeMapping$4.this$0
WARNING: Please consider reporting this to the maintainers of com.google.errorprone.bugpatterns.FutureReturnValueIgnored
WARNING: Use --illegal-access=warn to enable warnings of further illegal reflective access operations
WARNING: All illegal access operations will be denied in a future release
/Users/acole/oss/brave/brave/src/main/java/brave/propagation/ThreadLocalSpan.java:[99,6] [ThreadLocalUsage] ThreadLocals should be stored in static fields
    (see http://errorprone.info/bugpattern/ThreadLocalUsage)
[INFO] ------------------------------------------------------------------------
[INFO] BUILD SUCCESS
[INFO] ------------------------------------------------------------------------
[INFO] Total time: 4.289 s
[INFO] Finished at: 2018-08-20T09:38:30+08:00
[INFO] ------------------------------------------------------------------------

oops also need to figure our how to disable this in certain sub-projects, like JMH..

circleci is angry for unrelated reasons


NLB, internal with static IPs. How to specify static internal IPs?

When creating NLB like:
load_balancer_type = "network"
internal           = true

How do you specify static IPs like you can in the UI? I don't believe this is via subnet_mapping because that returns an error:  Error: creating network Load Balancer: InvalidConfigurationRequest: Elastic IPs are not supported for load balancers with scheme 'internal'

Thank you!

Use
private_ipv4_address versus allocation_id in your subnet_mapping