DeviantArt could be deploying a defence mechanism right now

Ripme version: 1.7.84
Java version:

java version "1.8.0_211"
Java(TM) SE Runtime Environment (build 1.8.0_211-b12)
Java HotSpot(TM) 64-Bit Server VM (build 25.211-b12, mixed mode)

Operating system:  Window$ 10.0.17134.706 Enterprise x64

Exact URL you were trying to rip when the problem occurred: https://www.deviantart.com/malte279/gallery/?catpath=/
Please include any additional information about how to reproduce the problem: -

Expected Behavior
Rips normally.
Actual Behavior
Errors out as "No full-size image found for <page URL>". Recently within a week's time RipMe has been consistently expressing strange inability to continue ripping DA after successfully working for a while first.
Take the link given in the template for example: https://gist.github.com/rautamiekka/4a0d947069fc6eb1f23406d5a3989208
At first, it works fine for a time. Then, no matter how many times I abort and restart, it's just reporting no image found although the page works just fine, and the folder-specific log (.\rips\deviantart_malte279_gallery_all) gives good hope in terms of functional links and detection: https://gist.github.com/rautamiekka/3ca0998f09785711e9f3f8fd240ce753
Here's the .\ripme.log: https://gist.github.com/rautamiekka/0e3eea98940072114c8a5e8732b4cd2b
Thus, I'm starting to think DA is deploying a defence mechanism when using RipMe, whatever that mechanism is. I suggest dumping each page RipMe fetches.

18 minutes later it worked fine:
RipMe output: https://gist.github.com/rautamiekka/81cf27bb659d8f13979ff10b9add3b98
.\rips\deviantart_malte279_gallery_all\log.txt: https://gist.github.com/rautamiekka/2479864b742d0ece3c5c18f3e81fc8fe
.\ripme.log:
2019-06-06 00:14:45,190 DEBUG ripper.DownloadThreadPool .initialize() Initializing deviantart thread pool with 1 threads
2019-06-06 00:14:45,191 DEBUG ripper.AbstractRipper .getRipper() Found album ripper: com.rarchives.ripme.ripper.rippers.DeviantartRipper
2019-06-06 00:14:45,924 DEBUG ripper.DownloadThreadPool .initialize() Initializing deviantart thread pool with 1 threads
2019-06-06 00:14:45,924 DEBUG ripper.AbstractRipper .getRipper() Found album ripper: com.rarchives.ripme.ripper.rippers.DeviantartRipper
2019-06-06 00:14:45,925 INFO  ripper.AbstractRipper .getGID() Album Name: malte279_gallery_all
2019-06-06 00:14:45,925 DEBUG ripper.AbstractRipper .setWorkingDir() Using album title 'deviantart_malte279_gallery_all'
2019-06-06 00:14:45,925 DEBUG ripper.AbstractRipper .setWorkingDir() Set working directory to: D:\_DOWNLOADS_\RipMe\rips\deviantart_malte279_gallery_all
2019-06-06 00:53:17,442 INFO  ripper.DownloadFileThread .run() [+] Saved https://api-da.wixmp.com/_api/download/file?downloadToken=eyJ0eXAiOiJKV1QiLCJhbGciOiJIUzI1NiJ9.eyJzdWIiOiJ1cm46YXBwOjdlMGQxODg5ODIyNjQzNzNhNWYwZDQxNWVhMGQyNmUwIiwiaXNzIjoidXJuOmFwcDo3ZTBkMTg4OTgyMjY0MzczYTVmMGQ0MTVlYTBkMjZlMCIsImV4cCI6MTU1OTc3MjE5MiwiaWF0IjoxNTU5NzcxNTgyLCJqdGkiOiI1Y2Y4MzljODMxMGRmIiwib2JqIjpudWxsLCJhdWQiOlsidXJuOnNlcnZpY2U6ZmlsZS5kb3dubG9hZCJdLCJwYXlsb2FkIjp7InBhdGgiOiJcL2ZcLzc2Njk0OTk2LTk2OWMtNGZmNy05ZDZmLWZkM2NlMTMzODNkY1wvZGE0MnRrMC1jNjkwOGFmNS01NGU4LTRhOGEtYjIwZS1iZTdlZTQ0MjU5YTIuanBnIiwiYXR0YWNobWVudCI6eyJmaWxlbmFtZSI6InNvYXBzdG9uZV9wcmluY2Vzc19sdW5hX2J5X21hbHRlMjc5X2RhNDJ0azAuanBnIn19fQ.7izvee7ok1eOFAte_rwATeF7a2LPM0XzC8X0RrWy-tY as .\rips\deviantart_malte279_gallery_all\soapstone_princess_luna.jpg

Ripping deviant art using different tool a few months ago i came across a defense mechanism that sounds like this.
Large amounts of activity seamed to trigger it, once triggered the web page would load normally, yet the CDN server that hosts the actual image file would not respond at all. Images hosted on other CDN servers seamed to still load. The cooldown period for it seamed to be over a day, but just changing my IP address got it working again. So my solution was to slow down the ripper tool and hop to a different IP if tripped.


fix: fixing typo preventing correct type-checking for the Alert in svelte

@fbasso Thank you for your review!


Feature: Report on level lower than the selected severity

Adding a new drop-down item in the UI to pick the
severity level of issues that will be displayed.
This enables the option of picking two issue levels

one for posting the score (Failing the commit with -1)
one for displaying issues.

Duplicate of https://github.com/jenkinsci/sonar-gerrit-plugin/pull/21 ? Maybe merge efforts of those two PRs?

Thank you for the contribution and for pointing out this feature. I have implemented it in slightly different way.


Can not build Dockerfile.example.tensorflow-mpi

I'd like to build the Dockerfile.example.tensorflow-mpi image by myself, I follow these steps:
1.build the Dockerfile.build.mpi  image.
2.build the Dockerfile.example.tensorflow-mpi image.
But I meet mistakes during the step 2. I cannot execute the command bazel build -c opt --config=cuda \ 	--cxxopt="-D_GLIBCXX_USE_CXX11_ABI=0" \         tensorflow/tools/pip_package:build_pip_package
There're some error, and I fix some of them:
1.apt-get update && apt-get -y upgrade && apt-get install -y git
2.pip install numpy
However, there still an error that I cannot solve:"openmpi/ompi/mpi/cxx/mpicxx.h: No such file or directory"
I have tried several methods to solve it according to  this page:
1.upload bazel version to 0.5.4
2.export OMPI_SKIP_MPICXX=1
3.export  CC_OPT_FLAGS="-DOMPI_SKIP_MPICXX=1 -march=native"
But they all don't work.

By the way, is "OMPI_SKIP_MPICXX=1" suitable? It means skip cxx when building.

Building TensorFlow r1.4 with Bazel 0.11.0 doesn't work. We can cherry pick https://github.com/tensorflow/tensorflow/commit/3f57956725b553d196974c9ad31badeb3eabf8bb to fix the Bazel versoin check bug or downgrade Bazel to 0.5.4.
Adding OMPI_SKIP_MPICXX is OK, pls refer to https://github.com/tensorflow/tensorflow/commit/f73d7c90ed05bcf9f36f6a3be0c29efa5fef0f6e#diff-7f5f80d91bf584c6c77b2d1bf874ee9b, TensorFlow r1.4 missed that.

I will submit a PR after I have tested it.

OK, after fixing the docker image, I'll run the mpi tensorflow example again.

is it possible to solve the problem by upgrading tensorflow to higher version?

I have tried it when I test the tensorflow benchmark on cifar-10 example. When the version of tensorflow reach over 1.4(1.5,1.7,1.8), even though I used cuda9.0+cudnn7, there still some error due to nvidia driver version.

2018-09-14 10:36:49.520632: E tensorflow/stream_executor/cuda/cuda_driver.cc:406] failed call to cuInit: CUDA_ERROR_UNKNOWN
2018-09-14 10:36:49.520675: I tensorflow/stream_executor/cuda/cuda_diagnostics.cc:152] no NVIDIA GPU device is present: /dev/nvidia0 does not exist
[paigcr-a-gpu-1010:00954] mca: base: component_find: unable to open /usr/local/mpi/lib/openmpi/mca_shmem_posix: /usr/local/mpi/lib/openmpi/mca_shmem_posix.so: undefined symbol: opal_shmem_base_framework (ignored)
[paigcr-a-gpu-1010:00954] mca: base: component_find: unable to open /usr/local/mpi/lib/openmpi/mca_shmem_mmap: /usr/local/mpi/lib/openmpi/mca_shmem_mmap.so: undefined symbol: opal_show_help (ignored)
[paigcr-a-gpu-1010:00954] mca: base: component_find: unable to open /usr/local/mpi/lib/openmpi/mca_shmem_sysv: /usr/local/mpi/lib/openmpi/mca_shmem_sysv.so: undefined symbol: opal_show_help (ignored)
It looks like opal_init failed for some reason; your parallel process is
likely to abort.  There are many reasons that a parallel process can
fail during opal_init; some of which are due to configuration or
environment problems.  This failure appears to be an internal failure;
here's some additional information (which may only be relevant to an
Open MPI developer):
opal_shmem_base_select failed
--> Returned value -1 instead of OPAL_SUCCESS

It looks like orte_init failed for some reason; your parallel process is
likely to abort.  There are many reasons that a parallel process can
fail during orte_init; some of which are due to configuration or
environment problems.  This failure appears to be an internal failure;
here's some additional information (which may only be relevant to an
Open MPI developer):
opal_init failed
--> Returned value Error (-1) instead of ORTE_SUCCESS

It looks like MPI_INIT failed for some reason; your parallel process is
likely to abort.  There are many reasons that a parallel process can
fail during MPI_INIT; some of which are due to configuration or environment
problems.  This failure appears to be an internal failure; here's some
additional information (which may only be relevant to an Open MPI
developer):
ompi_mpi_init: ompi_rte_init failed
--> Returned "Error" (-1) instead of "Success" (0)
*** An error occurred in MPI_Init
*** on a NULL communicator
*** MPI_ERRORS_ARE_FATAL (processes in this communicator will now abort,
***    and potentially your MPI job)
[paigcr-a-gpu-1010:954] Local abort before MPI_INIT completed successfully; not able to aggregate error messages, and not able to guarantee that all other processes were killed!

Now, I can build the docker image, but it seems that the open MPI was not installed correctly.

By the way, I can run the same code with the same data by using tensoflow1.4 without mpi.

Maybe there are some error in mpi base image, are there some methods to test it?

Hi @qyyy, did submit the job to PAI or run the Docker on the host?

tensorflow/stream_executor/cuda/cuda_driver.cc:406] failed call to cuInit: CUDA_ERROR_UNKNOWN

It seems the driver path wasn't mounted correctly in docker run.

tensorflow/stream_executor/cuda/cuda_diagnostics.cc:152] no NVIDIA GPU device is present: /dev/nvidia0 does not exist

It seems the gpu devices wasn't mounted correctly in docker run.
Please also check there's enough share memory in docker.
Because the openmpi is built with cuda, the gpu problems will also cause the failure of mpi_init.
The mpi base image referred to the openmpi installation part in official cntk image, and Docker.example.cntk-mpi built on the top of mpi base image also works well using openmpi.

I run it on PAI. And I try to run it again with 16GB memory, but neither the server nor the worker cannot run correctly. The error message is the same as above. I have test the cntk_mpi example, it's no problem. So, I think it's still some error of the building process of tensorflow.

@qyyy Could you please try this simpler mpi example code? Here's an example of its json file:
{
  "jobName": "tensorflow-mpi",
  "image": "openpai/pai.example.tensorflow-mpi",
  "taskRoles": [
    {
      "name": "mpi",
      "taskNumber": 1,
      "cpuNumber": 8,
      "memoryMB": 16384,
      "gpuNumber": 0,
      "command": "mpirun --allow-run-as-root -np 2 --host tf-0,tf-1 python tf-mpi.py",
      "minSucceededTaskCount": 1
    },
    {
      "name": "tf",
      "taskNumber": 2,
      "cpuNumber": 8,
      "memoryMB": 16384,
      "gpuNumber": 2,
      "command": "/bin/bash"
    }
  ]
}

Where is the tf-mpi.py file?

Could you please try this simpler mpi example code ...

here's tf-mpi.py

Traceback (most recent call last):
File "tf-mpi.py", line 5, in
import tensorflow.contrib.mpi as mpi
ImportError: No module named mpi

I have tried this, but it doesn't work.

I have put an issue onto tensorflow. Here is the issue.

import tensorflow.contrib.mpi as mpi
ImportError: No module named mpi

It's caused by tf version, you can use tensorflow.contrib.mpi_collectives instead.
I have tried those two tensorflow mpi examples. The benchmark example succeed with grpc protocol but failed with grpc+mpi protocal. However, openmpi itself worked fine in that Docker image. The problem may lied in mpi extension in tensorflow compilation, which causes the "An error occurred in MPI_Init" issue. It might be mpi version mismatch, or bugs in tensorflow mpi code.
There exists a few docs on how to build tensorflow with openmpi, including the official readme. The only switch I can make sure is TF_NEED_MPI=1 or Do you wish to build TensorFlow with MPI support [y/N] in tensorflow compilation. And we can only build tensorflow r1.4 with openmpi successfully at present.
I will figure out how to compile and use tensorflow with openmpi, which may take a moment.

By the way, Dockerfile.example.cntk-mpi cannot be build correctly, either.
The error message is following:
# _/hdfs-mount
import cycle not allowed in test
package _/hdfs-mount (test)
        imports _/hdfs-mount

FAIL    _/hdfs-mount [setup failed]
make: *** [test] Error 1
Makefile:52: recipe for target 'test' failed

@abuccts - per discussion, team won't support the docker anymore, clean up the docker and related samples.


Java8 workshop getting errors

Got issue related to lab where it says do "Cdk bootstrap"
after doing cdk bootstrap running the command gives below error
it will be good to mention which POM.xml to update for adding maven dependencies - it should be pipeline pom but because there is pom in application code it would be a good idea to mention the pipeline pom

[ERROR] Failed to execute goal org.apache.maven.plugins:maven-compiler-plugin:3.8.1:compile (default-compile) on project pipeline: Compilation failure: Compilation failure:
[ERROR] /home/ec2-user/environment/sam-app/pipeline/src/main/java/com/myorg/PipelineStack.java:[3,35] package software.amazon.awscdk.core does not exist
[ERROR] /home/ec2-user/environment/sam-app/pipeline/src/main/java/com/myorg/PipelineStack.java:[4,35] package software.amazon.awscdk.core does not exist
[ERROR] /home/ec2-user/environment/sam-app/pipeline/src/main/java/com/myorg/PipelineStack.java:[5,35] package software.amazon.awscdk.core does not exist
[ERROR] /home/ec2-user/environment/sam-app/pipeline/src/main/java/com/myorg/PipelineStack.java:[8,36] cannot find symbol
[ERROR]   symbol: class Stack
[ERROR] /home/ec2-user/environment/sam-app/pipeline/src/main/java/com/myorg/PipelineStack.java:[9,32] cannot find symbol
[ERROR]   symbol:   class Construct
[ERROR]   location: class com.myorg.PipelineStack
[ERROR] /home/ec2-user/environment/sam-app/pipeline/src/main/java/com/myorg/PipelineStack.java:[13,32] cannot find symbol
[ERROR]   symbol:   class Construct
[ERROR]   location: class com.myorg.PipelineStack
[ERROR] /home/ec2-user/environment/sam-app/pipeline/src/main/java/com/myorg/PipelineStack.java:[13,72] cannot find symbol
[ERROR]   symbol:   class StackProps
[ERROR]   location: class com.myorg.PipelineStack
[ERROR] /home/ec2-user/environment/sam-app/pipeline/src/main/java/com/myorg/PipelineStack.java:[16,45] incompatible types: com.myorg.PipelineStack cannot be converted to software.constructs.Construct

This workshop has been reorganized to use SAM Pipelines. As such, CDK errors are no longer and issue.


support flake8-fixme

Hey,
until know I could find rules for all my flake8 plugins or at least a feature request to support those.
The only one I'm missing is flake8-fixme.
Would be great to see this supported :)
fin swimmer

Maybe https://github.com/orsinium-labs/flake8-todos would be preferred?
It seems more comprehensive

T001: use TODO instead of FIXME (or BUG) for consistency.
T002: add author into TODO (motivation).
T003: add link on issue into TODO.
T004: missed colon in TODO.
T005: missed text in TODO.
T006: write TODO instead of ToDo (use upper case).
T007: missed space after colon in TODO.

Maybe https://github.com/orsinium-labs/flake8-todos would be preferred?

The purpose if flake8-todos seems to another one, compared to flake8-fixme. The later one ensures that there are not TODOs, FIXME etc. in the code base. And that's the functionality I like to have in ruff :smiley:

You're right, the two plugins address different problems! I'll create a separate issue for my suggestion.

@charliermarsh can you please assign this to me? Once #3921 lands, I'll be able to refactor some of the TODO detection out to a common location - when that's done, it'll be pretty trivial to clear this as well.

The fix I have in mind depends on this PR being merged so I can rebase on top of it


License

What license are the code/pre-trained models in this repo licensed under?

Hi @acairncross ,
Thanks for asking, I added the license in #31 , please have a look.


Getting error with OAuth Password Grant Type

I am getting invalid client issue as shown in below screenshot. Please can you guide me where I am going wrong. I am using MongoDB and below you can find oauth_clients resultset as well.

Can anyone help me on this?

yes. at first look looks like you're using "email" as a property key instead of "username".

Thanks for the response. I have already tried with username as well but its same error.

Just to help anyone else who might face same issue, I was using MongoDB so it was all related to that. I am planning to extend this package to support MongoDB and will share with you guys if I am able to do so.


About the Rest API

Title: Error 400 when adding a new memory using REST API
Description:
I have been working with the API to create a session-based memory in my application. I've implemented basic GET and DELETE operations for memory, and they seem to be working properly. However, I'm encountering an Error 400 when trying to add a new memory using the POST method.
I've tried various approaches to pass the Memory object, including using the zep-python library, custom JSON, and other methods, but none of them seem to work. The API documentation lacks examples for the REST API part, making it difficult to troubleshoot the issue.
Steps to Reproduce:
Perform a POST request to add a new memory using the REST API.
Include the necessary parameters in the request (e.g., session ID, memory data).
Observe the Error 400 response.
Expected Behavior:
The POST request should add a new memory to the session without any errors.
Additional Information:
I have successfully used the GET and DELETE methods for memory, which suggests that the authentication and basic functionality of the API are working as expected.
The issue seems to be specific to the POST method when adding a new memory.
Note:
The API documentation lacks clear examples for the REST API part, especially when it comes to the POST method for adding a new memory. This makes it challenging to understand the correct format and parameters required for the request. It would be highly beneficial to include detailed examples in the documentation to help developers use the API effectively.

Swagger docs may be found in the Zep repo: https://github.com/getzep/zep/tree/main/docs
You should be able to render the UI for the swagger file using an IDE such as VS Code or PyCharm and point it to your Zep server, allowing you to use the "Try it out" functionality.
We also have a Swagger UI for Zep available here: https://getzep.github.io/zep/
Both of the above provide example payloads based on Zep's models.

Got it ! Thank you so much for the fast response.


Add uml diagram for retag, untag, Label commands

Codecov Report

Merging #322 (10eabae) into master (5cb3303) will decrease coverage by 0.03%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             master     #322      +/-   ##
============================================
- Coverage     67.62%   67.58%   -0.04%
- Complexity      593      594       +1
============================================
  Files           100      100
  Lines          1838     1842       +4
  Branches        194      194
============================================
+ Hits           1243     1245       +2
  Misses          521      521
- Partials         74       76       +2

Impacted Files
Coverage Δ
Complexity Δ

...n/java/seedu/address/logic/commands/CdCommand.java
58.53% <0.00%> (-0.93%)
11.00% <0.00%> (+1.00%)
:arrow_down:

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 5cb3303...10eabae. Read the comment docs.


Add DNS Policy Field

Part of: https://github.com/rancher/rke/issues/1531

The format for the PR is 1 commit with changed code, and 1 with generated code as in https://github.com/rancher/types/pull/935/commits

LGTM


Discussion: Improvements around app.message(...)

Proposal
As described at #463, app.message(...)'s behavior with the latest payloads is not intuitive for developers. It's surprising for developers to receive more events than the number of messages in a channel.
I propose to change the behavior to receive only message typed events that don't have a subtype. With this, Bolt apps can receive only events telling the arrival of new messages.
Also. there is one additional thing we may need to care about in the future. As far as I know, the server-side no longer sends bot_message subtyped events. Instead, it sends no-subtype events with bot_id and bot_profile properties. If the server-side gets back to the previous behavior in the future, we may need to add bot_message events as well.
For reference: @aoberoi 's https://github.com/slackapi/hubot-slack/issues/589

What happened to the subtype: 'bot_message'? It seems to be missing on at least some, but maybe all, events where it should be.

References
The list of sutyped message events
There are a number of subtype events for messages. Some of them may have a quite different data structure compared with message typed events without a subtype.

bot_message
ekm_access_denied
me_message
message_changed
message_deleted
message_replied
reply_broadcast
thread_broadcast

Bolt for Java's approach
Bolt for Java's app.message listeners receive only message events that don't have a subtype.
https://github.com/slackapi/java-slack-sdk/blob/v1.0.3/bolt/src/main/java/com/slack/api/bolt/App.java#L509-L526
There are two reasons for that. It's technically impossible to include other events due to the Reason 2.

Reason 1: @seratch believes developers expect to receive only new incoming messages when they use keyword matching in text
Reason 2: It's not possible to support completely different structure in a statically-typed language (we're already aware of the same issue with TypeScript https://github.com/slackapi/bolt/issues/311)

The MessageEvent class in this line is a representation of message typed event without any subtype. Other events with subtype look like this.

Regarding bot_message events, Bolt doesn't invoke listeners given to app.message due to the difference in the structure of payloads. If the server-side is changed in the future, Bolt for Java may need to consider introducing a new routing method.
What type of issue is this? (place an x in one of the [ ])

[ ] bug
[ ] enhancement (feature request)
[ ] question
[ ] documentation related
[ ] testing related
[x] discussion

Requirements (place an x in each of the [ ])

[x] I've read and understood the Contributing guidelines and have done my best effort to follow them.
[x] I've read and agree to the Code of Conduct.
[x] I've searched for any related issues and avoided creating a duplicate issue.

This is a well thought out and thorough proposal!
Question: In Bolt for Java, would it be possible to use constraints properties to vary the type of arguments passed to a listener? This is how we've been able to solve this problem in Bolt for JavaScript in the app.event<EventType>() method.  I think this is a clean solution that doesn't expand the App API too much, but it might depend on unique capabilities of the TypeScript type system.

Question: In Bolt for Java, would it be possible to use constraints properties to vary the type of arguments passed to a listener?

Yes, it's possible and even is necessary for Bolt for Java apps. Developers are not allowed to make the type ambiguous.
Here is a simple example demonstrating how to use it:
https://github.com/slackapi/java-slack-sdk/blob/v1.0.3/bolt-servlet/src/test/java/samples/EventsSample.java#L21-L29
The first one in the listener args is typed with the given event class like MessageEvent. The type constraint is implemented this way: https://github.com/slackapi/java-slack-sdk/blob/v1.0.3/bolt/src/main/java/com/slack/api/bolt/App.java#L492

Thanks for writing this out @seratch!
So the proposal is to have app.message only handle message type events. Sounds fine to me.
Questions:

would this be a breaking change?
Is there any other way for bolt users to still handle events such as message_changed?

My concern with this proposal is the behavior with threaded messages. The message_replied subtype event only fires for messages that are contained in thread and I think the assumption would be that, as an app listening to messages in a channel, you'd receive all messages in that channel (including the ones in threads). Listening to messages in threads is a part of core functionality for some apps (IT ticketing apps, standup apps, etc.).
In addition, if my app didn't listen to messages in threads, I may still assume it would receive threaded messages sent to channel (reply_broadcast subtype) since it appears in the main channel window.
Because of threading, I am slightly against making this change. If we moved forward and made this change, I think we would need to carefully consider how we document support for threads so developers aren't confused why they aren't receiving messages.

If we were looking for other approaches to the problem, I think we could create a built-in middleware (https://github.com/slackapi/bolt/blob/master/src/middleware/builtin.ts) that specifies noSubtype messages, or something to that effect.
Perhaps in that regard, we may want to surface built-in middleware like that more easily so developers could more easily find and use the ones available.

would this be a breaking change?

Good point. Yes, it is a breaking change. So, it may be better to consider a bit careful transition like providing an option to change the behavior in the next minor release, and then changing the default in the next major release.

Is there any other way for bolt users to still handle events such as message_changed?

I think app.evnt is the best way to handle those. Developers can check what the subtype is inside a listener or middleware.

As far as I know, the server-side no longer sends bot_message subtyped events.

This was a wrong assumption. If other bot users in the same workspace are still classic apps, they may generate message events with bot_message subtype.
As of version 1.4.4, bolt-python handles only non-subtyped and "bot_message" subtype events by app.message listeners. This is the behavior most developers should expect. We can revisit this issue and it's worth considering to apply the behavior changes in v4.0 (of course, with a clear mention about it in the migration guide).

We can revisit this issue and it's worth considering to apply the behavior changes i

Hey so if I am building a slack app currently and receiving events via the slack events web api should I handle the cases with subtype as bot_message or consider they will not be subtyped with the presence of bot_id and bot_profile properties?

@batflarrow As I mentioned above, both no subtype and "bot_message" subtype patterns can still co-exist. If other bots in your workspace are still based on legacy permission model (we call it "classic" app), their message events don't have the bot_message subtype.

It's the other way round right "classic" app's messages with have a sub_type as bot_message and not the new apps?

@batflarrow As I mentioned above, both no subtype and "bot_message" subtype patterns can still co-exist. If other bots in your workspace are still based on legacy permission model (we call it "classic" app), their message events don't have the bot_message subtype.

It's the other way round right "classic" app's messages with have a sub_type as bot_message and not the new apps?

Also is there a list of these "classic" apps, how can I test my code for cases with sub_type as bot_message or is there a way just to know the payload in cases where the events will be sub_typed as bot_message

If you don't have so many apps in your workspace, it may not be so hard to review the scopes of the apps listed at https://slack.com/apps/manage . However, I think that your goal is to safely respond to all new messages. If so, all you need to do are:

Check if a message event has "bot_message" subtype or does not have any subtype
If yes, extract text property from the event data and respond to it

If you don't have so many apps in your workspace, it may not be so hard to review the scopes of the apps listed at https://slack.com/apps/manage . However, I think that your goal is to safely respond to all new messages. If so, all you need to do are:

Check if a message event has "bot_message" subtype or does not have any subtype
If yes, extract text property from the event data and respond to it

yeah but I am building a slack app myself for people to use and they can have any apps installed on their systems themselves so I will have to handle the cases with the "bot_message" subtype. It would be great if I could find a event with the sub_type as bot_message but anyways  I'll look for it . Also I got your method will probably handle those in this way. Also can you confirm "classic" apps have the subtype right and not the new apps coz you mentioned it the otherway round here and here.


Fix AutoSync when a etcd cluster is unhealthy

When a etcd cluster is unhealthy, the AutoSync will be broken. if etcd cluster node change it's advise ip then docker client will never known even we use domain name in docker opts.

Please sign your commits following these rules:
https://github.com/moby/moby/blob/master/CONTRIBUTING.md#sign-your-work
The easiest way to do this is to amend the last commit:
$ git clone -b "master" git@github.com:ringtail/libkv.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.


Update geckodriver for compatibility with recent Firefox versions

The ci.sh script installs the latest Firefox (currently 68.9.0esr), but an old version of geckodriver. Locally, tests seemed to hang until I killed them after a few minutes. Once upgraded, hokusai test succeeded.
This version compatibility chart was helpful: https://firefox-source-docs.mozilla.org/testing/geckodriver/Support.html#supported-platforms

Thanks @joeyAghion !


Using the SDK in Chrome generates lots of 'Refused to set unsafe header "accept-encoding"' errors'

Might well be a resin-request issue, haven't investigated yet. Reproduced by just running the test suite in Chrome, where almost every test hits this at least once.
Doesn't break anything, but lots of noise, and we should fix it before we put the SDK to use in the UI.

Resin-requests issue, moving to resin-io-modules/resin-request#91


Refactor SPI for SES channel addition

Description
Refactor SPI for SES channel addition
[Tests]
All unit and integration test passed
Signed-off-by: @akbhatta
Issues Resolved
N/A
Check List

[x] New functionality includes testing.

[x] All tests pass, including unit test, integration test and doctest

[x] New functionality has been documented.

[x] New functionality has javadoc added
[x] New functionality has user manual doc added

[x] Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Codecov Report

Merging #257 (b6097c9) into develop (a8fa959) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff            @@
##           develop     #257   +/-   ##
========================================
  Coverage    80.53%   80.53%
========================================
  Files           60       60
  Lines         1634     1634
  Branches       415      415
========================================
  Hits          1316     1316
  Misses         310      310
  Partials         8        8

Flag
Coverage Δ

dashboards-notifications
80.53% <ø> (ø)

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

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 a8fa959...b6097c9. Read the comment docs.


Add visualstudio2017 to windows il2cpp docker images

Context
The windows docker image requires VC++ with an IDE to build windows IL2CPP applications
Suggested solution
Add choco install visualstudio2017-workload-vctools -y --no-progress to the image with the "il2cpp-windows" module
Considered alternatives
Any user has to build their own image for building their application
Additional details
The version of VS can be discussed. I tried visualstudio2019buildtools package at first, but it doesn't work because Unity needs a full IDE.
Here is a build log when "visualstudio2017-workload-vctools" is not installed yet
Initializing Microsoft Media Foundation failed.  It seems that the Microsoft Media Foundation is not installed on this machine or a newer version of Microsoft Media Foundation is required.  To install the necessary lib
raries please install the Media Feature Pack from https://www.microsoft.com/en-us/search/result.aspx?q=Media+Feature+Pack for your version of Windows.

Exception: C++ code builder is unable to build C++ code. In order to build C++ code for Windows Desktop, you must have one of these installed:
        Visual Studio 2015 with C++ compilers and Windows 10 SDK (it cannot build C++ code because it is not installed or missing C++ workload component)
        Visual Studio 2015 installation is found by looking at "SOFTWARE\Microsoft\VisualStudio\14.0_Config\InstallDir" in the registry
        Windows 10 SDK is found by looking at "SOFTWARE\Wow6432Node\Microsoft\Microsoft SDKs\Windows\v10.0\InstallationFolder" in the registry

        Visual Studio 2017 (or newer) with C++ compilers and Windows 10 SDK (it cannot build C++ code because it is not installed or missing C++ workload component)
        Visual Studio 2017 (or newer) installation is found using Microsoft.VisualStudio.Setup.Configuration COM APIs
        Windows 10 SDK is found by looking at "SOFTWARE\Wow6432Node\Microsoft\Microsoft SDKs\Windows\v10.0\InstallationFolder" in the registry

Is-this the case for all windows based images?

We can't actually add Visual Studio with VC++ build tools to the images due to licensing issues. See microsoft/Windows-Containers#102 for details. We instead use the Visual Studio that is installed on the GitHub runners and pass it through to the container, so it currently uses VS2019 Enterprise. Since we pass through the entire installation, we don't install Visual Studio into the image which also helps save on the already massive Windows image size. If you are creating private docker images for your own use, then you can install Visual Studio with the VC++ build tools without running into the redistribution issue. Otherwise, if you are running GitHub Actions and are using a self hosted runner, just make sure you have Visual Studio 2017 or 2019 installed on the host with the workloads you need.

Should we add it to the documentation?
The problem and both available solutions, I mean
I also faced a problem that every statring windows docker container on the same machine requires a new docker license
I've fixed all changable values from a license file (including the container hostname) but the hash of the machine is always different. That's why we had to acquire and return a license for build every run
Is it OK, if I create a PR including this information to a Docs ?

@AndrewKahr
Can you tell us, please, how do you ?

pass it through to the container

Passing Microsoft Visual Studio directories from "Porgram Files" (and x86) didn't work. vswhere wouldn't find it

I'm not sure if you meant Unity License instead of docker license, but if you did, make sure you have your unity email and password set in addition to the license file when trying to build on Windows/Mac. The docs are now written such that there is a credentials section to point people towards setting it up: https://game.ci/docs/github/builder#credentials
As for passing through to the container, it should happen automatically for Github Actions, unless it was installed in a non-standard location. If you are manually using it, you'll need to pass C:/Program Files (x86)/Microsoft Visual Studio and C:/ProgramData/Microsoft/VisualStudio as well as C:/Program Files (x86)/Windows Kits for the Windows 10 SDK. You can find that process here: https://github.com/game-ci/unity-builder/blob/main/src/model/platform-validation/validate-windows.ts

Thanks you a lot!
I ended up with volumes
SDK: C:\Program Files (x86)\Microsoft SDKs
x86 VS: C:\Program Files (x86)\Microsoft Visual Studio
x64 VS: C:\Program Files\Microsoft Visual Studio
Installer data: C:\ProgramData\Microsoft Visual Studio
I'll add this information to the Gitlab docker docs

Thank you for sharing your solution! Would you share a reference here once it's added to the docs?
After that I think we can close this issue.


Split filter/2 in two functions

@cigrainger I was wondering if we should break filter/2 in two functions: mask(dataframe, series) and filter(dataframe, callback).
The rationale for doing so is that we don't want to encourage filter(dataframe, series), as that won't be efficient as a lazy query. So I thought we should make filter/2 to be exclusively function based (and across based in the future based on #223) and provide a specific mask/2 function for masking.
Thoughts?

Sorry I meant to say ages ago that I agree with this. Mask is perfect.

@cigrainger so last time we talked we were not sure if we should allow the mask in filter_with (instead of filter) or introduce the mask function. Between those options, any preference?
The mask in filter_with doesn't have the ambiguity this issue first comments on... so it may be theoretically fine, although some adapters like SQL cannot implement the mask version anyway.

I think a separate mask function makes more sense to me. It seems cleaner. But I don't hold a strong opinion on this one. How much do we think folks should use filter_with directly?

Once we land the macro version of filter/2, there is little to no reason to use filter_with/2? So yeah, it makes sense to have mask/2 then!

Agreed! Let's do it.


feat: add translation of README.md into French

Summary
Add translation into French of README.mdand update links to include the French translation.

New translation: add README.fr.md file with translation into French of the README.md file.
Update links: update README.md, README.hi.md, README.ja.md, README.kr.md, README.pt-BR.md, README.ru.md, README.uk.md, README.zh-CN.md and README.zh-TW.md files to include a link to the French translation.

Merci pour ta contribution ! J'ai laissé quelques commentaires / suggestions :)


Error deploying Vision AI Application with custom trained model

Hi,
I have been using Vision AI platform for a while with a custom trained model.
Our model has been updated and registered registered on 2023-02-09 and has been working correctly.
I could undeploy the application on 2023-05-15 to adjust the input streams of the application and could deploy without problem.
However, I found out that there was mismatching information in the metadata sent to our model. I went into the application view panel and found out that the entire application setup UI is changed.
Now, there is "Universal input" instead of "Streams" and I could see that the Universal input node was not connected to my model. In order to fix that, I undeployed the application and deleted the application.
I followed the step by step guide published on https://cloud.google.com/vision-ai/docs/build-app. However, the "Add streams input" section does not match to the UI and I could not follow the steps.
Finally, I registered the streams as "Streaming" and registered 3 cameras to test. I added my custom trained model, with all the options, as an "Output" of the "Universal input" node and I could see that the two nodes were connected on the graph.
Finally I tried to Deploy the application and received a deployment error. I tried different combinations, including the deployment by API, but without a positive result. I am attaching the deployment error.
{
"name": "projects/c5i-data-lake/locations/us-central1/operations/operation-1684582139755-5fc1e5808fc81-2f7c5bcc-5d50bdbc",
"metadata": {
"@type": "type.googleapis.com/google.cloud.visionai.v1.OperationMetadata",
"createTime": "2023-05-20T11:28:59.981037429Z",
"endTime": "2023-05-20T11:40:47.157448423Z",
"target": "projects/c5i-data-lake/locations/us-central1/applications/c5i-20230520-placas",
"verb": "update",
"statusMessage": "Custom Model Deployment Step 2 Pending....",
"requestedCancellation": false,
"apiVersion": "v1"
},
"done": true,
"error": {
"code": 3,
"message": "Instance Deployment Step 1 failed, step: projects/c5i-data-lake/locations/us-central1/applications/c5i-20230520-placas/instances/c5i-test-pedernales-lpr-DeployInstance. Detailed App Platform Error: Invalid application instance graph, it cannot be deployed:  Detailed App Platform Error:  Invalid Application Instance, there should only be 1 stream input, but now: 0"
}
}
I am also attaching two images from the UI.

Thanks for reporting; we are looking into it.

Will the documentation be updated according to the final fix?

Yeah, though it might trail the fix by a few days, because the documentation update is part of a launch train. It will be out some time next week.


Using URL as key to Map is unsafe

https://github.com/apache/dubbo/blob/3257ef48f524c748c0651e97f6a8d06ff2963c94/dubbo-remoting/dubbo-remoting-p2p/src/main/java/org/apache/dubbo/remoting/p2p/exchange/support/AbstractExchangeGroup.java#L49
https://news.ycombinator.com/item?id=21765788

Thanks.
The URL you mentioned is defined by Dubbo and it behaves differently from the one in JDK. So I think we are not facing the issue you mentioned in the link below.
Reopen if necessary


Scale CC clusters up and down on the managed console (change node size)

Background: https://airtable.com/tblD3oZPLJgGhCmch/viw1DKmbKhg2MIECH/rechmiQfoXTbajvfw
Description: Once a particular node size is picked, users can change the node size - to smaller or larger.
Team: Jeremy Bloom

Replaced by https://github.com/cockroachdb/docs/issues/5973.


ipsec: support multiple phase 1 proposals

As prodded by @mimugmail -- if @AdSchellevis wants to take this it's fine with me.

This would be very nice feature since it guarantuees compatibility to older and newer clients.
For the beginning, a checkbox with a warning for unsetting the "!" in ike and/or esp would be enough.
Then older clients would negotiate lower encryption:
              Defaults  to  aes128-sha256.   The  daemon  adds  its  extensive
              default proposal to this default or the  configured  value.   To
              restrict  it  to the configured proposal an exclamation mark (!)
              can be added at the end.

              Note: As a responder, the daemon defaults to selecting the first
              configured  proposal that's also supported by the peer. This may
              be changed via strongswan.conf(5) to selecting the first accept-
              able  proposal  sent by the peer instead. In order to restrict a
              responder to only accept specific cipher suites, the strict flag
              (!,  exclamation mark) can be used, e.g: aes256-sha512-modp4096!

https://forum.opnsense.org/index.php?topic=6544.0
This could fit the thing with "!" .. a button "Disable strict policy" would be nice.

For me it seems if the other endpoint has multiple offers and the first one doesn't match, the connections fails. If the "!" is removed it tries the next one and so one.
So I'd still suggest a new Checkbox "Disable strict policy" to remove it (default unchecked).
It's a compatibilty enhancement for devices supporting multiple values for P1 and P2 like ASA.

This would be nice, since I have to configure both Android 8 and Windows 10 roadwarrior clients, which is not possible, since there is no configuration that both support.

I'm also fighting ATM since Android 7 only support 1024 PFS in P1 and Mac requires 2048.
Trying to build a support matrix which client supports which VPN types .. but right now it doesn't make sense. Perhaps also multiple DHs would be nice.
Some months ago I opened an issue to add a checkbox which would just remove the ! from the ike and esp .. perhaps calling it Disable strict cipher selection

@mimugmail first part in 037a92f2cade -- it only works in IKEv2, IKEv1 only takes the first and maybe we should warn users in the GUI about this. The enc algo modelling is less flexible and we need glue to bring it all together but we'll have a solution for 19.1 in any case. :)
Please test!

@mimugmail 36cde51 job done? :D

This looks great!! Thanks! :)
Do you think we ca reverse the sort order?
ATM it's sha1 and 1024 first and sha512 and 2048 at last ...

Hmm, we could reverse the write order, assuming that the values are filled from smallest to highest, but it's somewhat dependent on the browser.

Legacy code doesn't support ordering so we could order from config.xml to ipsec.conf?

The issue is that in the current modelling it supported only one entry, so there is no order by intention... now that we can choose multiple values we're left to guess what the best order is. 2 hashes and 2 dh groups are 4 combinations and even if we order by both hashes and dh groups from best to worst we have a middle gap where order is subject to interpretation due to the inner and outer loop. I'll add a final reverse, but that's as far as we can go before rewriting the entire handling of ike and esp entries.

Thanks, I'll test. But for this I'd left P2 untouched (which also has wrong ordering regarding hash) since it could break existing setups

I haven't changed P2 other than changing the hash checkboxes to a selectpicker for unified style.

Ok, works perfect. Tested with Andoid, before the patch both devices were DH1024, now Android is DH1024 and iPAD is at DH2048, so, working as expected with highest security available 👍

Ok, close?

Close, thanks 🥇

Pew pew!


Invalid position at 334 at https://docs.remirror.org/editors/wysiwyg

Go to https://docs.remirror.org/editors/wysiwyg
On load, select all content delete press undo and it breaks the page. there are error logs on console

@joonhocho thanks for raising this. I have confirmed the issue 👍
The offending line is in the positioners.
https://github.com/ifiokjr/remirror/blob/26f51ae94ca102c4282de45d3f123a929f0fc368/%40remirror/react/src/positioners.ts#L61
While there are several easy fixes, the fundamental issue is that the EditorState held by the remirror component is one step ahead of the EditorState stored by Prosemirror.
When updating the editor state the remirror component first calls this.setState({ newState: state }) and then after the component renders, view.updateState is called.
The updateHandler can be found here.
https://github.com/ifiokjr/remirror/blob/26f51ae94ca102c4282de45d3f123a929f0fc368/%40remirror/react/src/components/remirror.tsx#L374-L380
By simply moving the view.updateState to before the this.setState call, this bug and probably a whole host of future bugs are fixed.
The only problem is that currently, the @remirror/extension-mention package depends on this weird implementation. The package was developed at the very beginning of my journey with Prosemirror and for some reason delaying the update of the view's state fixed an issue I couldn't fix otherwise.
Before I can merge in the simple fix I'll need to update @remirror/extension-mention to be a Prosemirror mark rather than node. This will also provide much better support on Android keyboards.

@ifiokjr Thank you so much for details. Awesome work on your repo, btw. I've been searching for editors for past few days and ,after trying out so many options, decided yours is the best one for my use case (prosmirror + react + typescript). I still see performance issues with slate. I don't have much experience on editors yet, though I've read ProsMirror documentations.
You said your work is inspired by tiptap. Have you also taken a look at Atlaskit's editor? It's built on top of react + prosmirror + typescript. https://bitbucket.org/atlassian/atlaskit-mk-2/src/master/packages/editor/

@joonhocho thanks for the pointer in the atlassian direction.
Credit where credit is due, I've used a lot of code from them in making this library. The current implementation of NodeView's with React components is basically a direct copy.
I've also got some work coming (at some point) for tables which borrows heavily from their library (albeit simplified).


remove ecr from soak-on-release service list

Description of changes:

While running soak test on every release of ecr-controller, there were resource leaks and no monitoring was set on those soak tests.
Since these soak tests were running in isolated e2e test account, it created friction on ecr-controller PR testing
For now I am removing the soak tests on every release of ecr controller and they can be enabled again once ecr team owns the controller and want this feature.

Error from soak-test-job
        # Create k8s resource
        ref = k8s.CustomResourceReference(CRD_GROUP, CRD_VERSION, RESOURCE_PLURAL,resource_name,namespace="default",)
        k8s.create_custom_resource(ref, resource_data)
        cr = k8s.wait_resource_consumed_by_controller(ref)
>      assert cr is not None
E      assert None is not None
tests/test_repository.py:148: AssertionError
 ------------------------------ Captured log call -------------------------------
ERROR    root:resource.py:227 Wait for resource default:ecr-repository-n87zogcx2 to be consumed by controller timed out

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

/lgtm


SelfDefine device based on user input

Description
Implements a new device model class (SelfDefineDevice) where user can define up/down pulse and weights.

@bobo-nums @kaoutar55 I polished the device quite a bit. There were quite some unecessary loops in the device which I removed and optimized. You can take a look. I also enabled the full device-to-device variation. I also renamed the device and the fields to be more consistent with the other devices.
Moreover I added a fitting tool to any device data.

This is automatically fitted from:

Maybe one could add fit to the variability as well (will add tomorrow).
Would be great of you would double check whether everything looks correct.
I will try to add the CUDA part tomorrow.

@kaoutar55 I added the CUDA device and the write noise fitting in the example. Would be nice if one could add some documentation. The CSV read might also be added directly to the toolkit instead of to the example.
Black dots are the automatically fitted device data:

Thank you @bobo-nums and @maljoras the new feature and update look great! Will work on adding documentation for this.


Vectors of NodeStats data

This:
https://github.com/PyVRP/PyVRP/blob/a21c07f5c7d19e8692b9bd9ca49d5f581c783b2d/pyvrp/cpp/search/Route.h#L80-L89
Should probably become separate vectors by now since we're also adding load segments to it in #471, rather than one vector of this struct. That'd probably help with the cache.

I'm going to try to get a draft PR ready for this tonight, and then I'll run a benchmark to see if it matters.


Execution issue

Please can you give me the steps to running on my as400 system.

My bad, completely forgot to add a setup section to my README.
I added the following section:
Setup

Setup Twilio account and a Twilio phone number - https://www.twilio.com/
Connect to IBMi via SSH - ssh USER@YOUR400
Clone project - git clone https://github.com/barrettotte/RPGLE-Twilio.git
Navigate to project - cd RPLGE-Twilio
Give permissions to build script - chmod u+x build.sh
Run build script - ./build.sh
The build script will create the following

A Library named TWILIO to hold all of the Twilio objects
A Log table for logging SMS requests - sms_log
A simplified view over the log table -> sms_log_v
A stored procedure to call Twilio SMS API -> send_sms
An RPGLE service program to call Twilio SMS API

Example call in example/src/testtwilio.sqlrpgle

closing this issue since i did forget to include the setup instructions.
This is not meant to be a production-ready service program, just something I messed around with in my free time. I'm sure it can be adapted to be something cooler.
Thanks for taking a look at it


init failed: Error: Invalid configuration settings

Hello guys, im new to AWS services and when i enter " amplify init' , they prompt me whether to create an account or not, i pressed no and they prompt me for the accessKeyID and secretAccessKey and when i entered them , the error "invalid configuration settings" came out.
Just to be sure, i got my accessKeyID and secretAccess key from Workbench > Account Details

Hi @renebrandel , i've followed the documentation but i seem to have a problem with the newly created account
Attached is the error, i cannot proceed with the configuration without the access keys

@nypsystem - it looks like your organization is prohibiting the creation of Administrator accounts. This doesn't seem like something that we can engage in. Please reopen this issue if can confirm that your organization allows your account to perform this action.


net/rpmsg: add multi-iov support

Summary
Add multi-iov support for rpmsg sockets
Impact
Testing

I've never worked with the RPMSG subsystem, I'd need some time to understand these changes.

Sure, the key change is support send iovec in one batch.


Add Row ID specification to protocol

Description
This change updates the Delta protocol to add the specification for the Row ID feature.
Row IDs are integers that are used to uniquely identify rows within a table.
See Row ID design document for additional context:

@johanl-db the design doc is not public available

@johanl-db is the plan to send the implementation PR soon?
Identity column spec has been created more than an year ago and we don't have a implementation. I'm afraid the same could happen to Row Id.
Without reference implementation other libraries are more likely to deviate from spec and create compatibility issues.

@johanl-db is the plan to send the implementation PR soon? Identity column spec has been created more than an year ago and we don't have a implementation. I'm afraid the same could happen to Row Id.
Without reference implementation other libraries are more likely to deviate from spec and create compatibility issues.

Yes, I'm actively working on it and I plan to open the first PRs implementing writing default row ids in the next few days.
Support for reading and writing materialized row ids is also in progress though it requires changes in Spark. I started shipping these (e.g. https://github.com/apache/spark/pull/40545) but I'll need to wait for Delta to pick up the changes in a later Spark release before moving forward which may take some time. When that happens, I'll open PRs to support reading and writing materialized row ids.


Fedora 32 version is lower than F31

/kind bug
I've just updated to F32 and my podman downgraded from 1.9.0 to 1.8.2.
Describe the results you received:
$ podman version
Version:            1.8.2
RemoteAPI Version:  1
Go Version:         go1.14
OS/Arch:            linux/amd64

Describe the results you expected:
Expected to use latest podman 2.0 with #5883 merged in (1.9.1 doesn't have the fix).

1.9.1 should land soon in F32.

Expected to use latest podman 2.0 ...

2.0 does not exist yet. Note that the API is still experimental, so I am not sure if #5883 will make it into any release before 2.0.

I checked that #5883 works and would like to see that released for F32.

F32 is now with podman 1.9.1. Backporting #5883 is another issue.


some questions about inference

Do I have to download the entire dataset before running inference.py? I have already placed the pre-trained model, but I still can't run it.

I think you miss some files in the dataset -- Frames.m4v, you can check whether this file is downloaded correctly

I think you miss some files in the dataset -- Frames.m4v, you can check whether this file is downloaded correctly

Thank you，but I encountered a new problem: the sfm_model does not exist. Does the sfm_model need to be generated, or am I missing something in the dataset?

you should generate them. please set the redo in configs/preprocess/sfm_spp_spg_sample.yaml to True. It applies to other split (train / test / val).

Thank you！

hello, I have set the redo and the AssertionError about sfm_model disappeared. But  the  "Error executing job with overrides" still exists

could you help me?
thanks


Enhancement: Multi-png animation support

When using Apple's SpriteKit to develop games, they always want animations and sequences done like this:
name_000.png
<EMAIL_ADDRESS>name_001.png
<EMAIL_ADDRESS>name_002.png
<EMAIL_ADDRESS>name_003.png
<EMAIL_ADDRESS>I know it's kind of a simple / dumb feature to add, but being able to get these out would be FREAKING AMAZING to the pipeline of someone building OS X / iPhone games.

This was fixed by #214 AFAICT. Feel free to reopen if needed!


Open modal is shifting body content to the left

This issue (9855) was supposedly fixed in 3.2.
I still see this problem using Bootstrap v3.3.7
MacOSx El Capitan in
Chrome 52.0.2743.116
Firefox 48.01
Safara 9.1.1
Does anyone else still see this?

Ok I just realised this was because my root div was container-fluid instead of container.


Using "jQuery" instead of "$"

Minor trouble : in slick.js, there is "slider = $(element);", which explodes on our site where $ is not defined (it must remain free for other uses), and only "jQuery" is.
To prevent collisions, how about sticking to "jQuery", or using a closure to define "$" as is often done ?

We have the same problem
var $ = jQuery;
this works

What about using JQuery's no conflict wrapper?
http://learn.jquery.com/using-jquery-core/avoid-conflicts-other-libraries/


fix: #2114 for redis-server not support Hello

fix: #2114 Using strings.HasPrefix instead of equal

Thanks


[FEATURE REQ] Update Microsoft.Rest.ClientRuntime to use Newtonsoft.Json v11.0.2

Library or service name.

Microsoft.Rest.ClientRuntime

Is your feature request related to a problem? Please describe.

Reduce dependencies for NetStandard2.  Newtonsoft.Json v10.0.3 has no NetStandard2-target so it fallback to NetStandard1 dependencies.

Solution

Update to Newtonsoft.Json v11.0.2, or maybe switch to your own new System.Text.Json

@snakefoot thanks for your suggestion.
We are in the process of updating the client libraries for data plane to follow the Azure SDK guidelines and as part of that, we are moving to use System.Text.Json.


context.userId is inconsistent and should almost always be PractitionerRole

The PractitonerRole resource identifies a clinical user acting within a specific organization with given credentials, authorizations and role.
It seems like the userId field in context SHOULD be valued with a PractitionerRole resources, if the hook is fired during a provider-facing workflow. In the case where a PractitionerRole isn't supported (e.g. DSTU2), Practitioner should be sent.
For patient-facing workflows, userId SHOULD be Patient or, in proxy-scenarios, RelatedPerson.

We should update each hook to clarify when Patient/RelatedPerson may be used and that PractitionerRole is appropriate.


Logprobas

Again, no way to test at runtime at the moment, so let's just hope for the best

A) The I/O issue is connected to the everlasting problem of the proto structures. As of now, such functions only make sense in the Algorithm class. I can move them out, my points about the previous I/O functions still remain.
B) If the writing function does get moved, then it absolutely makes sense to also get rid of the std::pair, since one would call the write function from the main or something
C) About the density evaluation on single iterations, I can change the function so that it returns an Eigen::Matrix in which the i-th column is the evaluation at the i-th iteration. Wouldn't its size be too large, though?

A) The I/O issue is connected to the everlasting problem of the proto structures. As of now, such functions only make sense in the Algorithm class. I can move them out, but my points about the previous I/O functions still stand

Please clarify this point!

B) If the writing function does get moved, then it absolutely makes sense to also get rid of the std::pair, since one would call the write function from the main or something

Ok great

C) About the density evaluation on single iterations, I can change the function so that it returns an Eigen::Matrix in which the i-th column is the evaluation at the i-th iteration. Wouldn't its size be too large, though?

I don't think, for 1D data  (or 2D max, where it the visual representation of the uncertainty of the estimate is still possible) it would still make sense. Then again in more than 3 dimension estimating the density over a grid is not something so common... we could have another function / method that is highly optimized for estimating the mean, as you do now...

A) The I/O issue is connected to the everlasting problem of the proto structures. As of now, such functions only make sense in the Algorithm class. I can move them out, but my points about the previous I/O functions still stand

Please clarify this point!

Never mind, I moved out everything I could. That is, everything bar get_state_as_proto(), which uses private members allocations and unique_values. I guess it is ok to keep this one in Algorithm.
Everything should be fine now. Please check and approve the PR and I will proceed with the merge, thank you

Great, yes I believe that get_state_as_proto should be part of the algorithm, because different algorithms (e.g. conditional or marginal) will have different kind of states.
I think this is good, approving the PR right now, thanks for the great work!


CLOUD-51759 after stop-start the registrator tries to register ambari…

… a bit too early when the consul is not ready yet.  Set up registrator to re-register periodically (even the first registration is unsuccessful the registrator tries to register again after 60s)
@keyki
Please note that the formatter included to config/idea_settings.jar changed the import order and the whitespaces.

Jenkins build finished, all tests passed.
Refer to this link for build results: http://ci.sequenceiq.com/job/cloudbreak-pull-request/2013/


how to import libraries from cdn in this library?

Bug description:
Currently i need apple music login integartion in my react native app. since apple only provides a cdn script fo this. I would like to know how i can use it with react-native-webview
Environment:

OS:
OS version:
react-native version:
react-native-webview version:

@jasir-m have you found a solution?


Do you understand why it gets stuck and crash?

From what I can find, the exit code is a STATUS_ACCESS_VIOLATION error, which means something is trying to access something it shouldnt. I don't really know what or why though, and the windows CI is really finicky about this. I also don't have a windows setup myself, so I can't even reproduce.
Just gathering things here: as far as I can tell, this can definitely be merged, am I correct?

I think so. My code did not raise any exception during the night.
On Thu, Aug 1, 2024 at 11:30 AM Lukas @.***> wrote:

From what I can find, the exit code is a STATUS_ACCESS_VIOLATION error,
which means something is trying to access something it shouldnt. I don't
really know what or why though, and the windows CI is really finicky about
this. I also don't have a windows setup myself, so I can't even reproduce.
Just gathering things here: as far as I can tell, this can definitely be
merged, am I correct?
—
Reply to this email directly, view it on GitHub
https://github.com/QuantumKitHub/SUNRepresentations.jl/pull/21#issuecomment-2262583691,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AXMRNR5YJU3NN3EK6IU5MHLZPH5ZRAVCNFSM6AAAAABLYSE7C6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDENRSGU4DGNRZGE
.
You are receiving this because you authored the thread.Message ID:
@.***>


How to get raw packet data in real time

Is it possible to get the raw hex data of a packet via pktmon.exe when log-mode is set to real-time.
I tried the following:
pktmon start --capture --log-mode real-time --flags 0x010
but was unable to get the raw data/payload of the packet.

We hugely appreciate your contribution, feedback, and support to our technical documentation.
We’ve phased out GitHub Issues on this repository as the feedback mechanism for content and have replaced it with a new feedback system. For more information see, Content User Feedback. As part of this rollout, we are migrating your issue where our team will use your comments to improve the documentation experience. Unfortunately, this does mean you won't be able to access this issue after the migration is complete.
All issues will be migrated by Thursday 29th August 2024.
Once again, thank you for your valuable feedback, we hope you’ll continue to contribute using our new feedback system.


Zero-padding not supported in compiled output

It appears that zero-padded numeric sequences are advertised, but not properly supported.
// Example from README.md
console.log(braces('a/{001..300}/b')); //=> ['a/(0{2}[1-9]|0[1-9][0-9]|[12][0-9]{2}|300)/b']

// Actual output is:
[ 'a/(0{0,2}[1-9]|0?[1-9][0-9]|[12][0-9]{2}|300)/b' ]

Will follow up with PR with failing test cases.
Please advice whether the fix for this is in this module, or upstream in to-regex-range.
As always, thanks for you hard work on this module and so many others.

Okay, I dug into this a bit more...
It seems that fill-range is the culprit, and 'to-regex-range' is not actually used. The undocumented strictZeros option can be passed to fix zero-padding in braces.
assert.deepEqual( braces(['a/{01..20}/b', 'a/{1..5}/b'], { strictZeros: true } ), [ 'a/(0[1-9]|1[0-9]|20)/b', 'a/([1-5])/b' ] );

My recommended solution here is to make { strictZeros: true } is the default behavior in braces, since ( to my mind anyway ) it represents a more correct output.
Thoughts?

The undocumented strictZeros option can be passed to fix zero-padding in braces.

Thank you, I had a hard time remembering and was about to look. Yes, I agree with that. However, I think we should do a major bump because it will result in a breaking change. Would you like to do a PR?
FWIW I have a couple of other fixes that I'll be publishing as a patch before I bump the major.

I will make a PR for this in the next few days, making { strictZeros: true } the default behavior and adding docs and test cases. Is there any reason to use to-regex-range, or is fill-range sufficient?

This should be resolved by https://github.com/micromatch/braces/issues/29.


nuitka crashes if python package has text in the version

System:
1.1.6
Commercial: None
Python: 3.8.13 (default, Aug  9 2022, 11:20:38)
Flavor: Unknown
Executable: /some/path/to/venv/python3.8
OS: Linux
Arch: i686
Distribution: Debian 7
nuitka was installed through pip
pip freeze:
Events==0.4
Nuitka==1.1.6
ordered-set==4.1.0
Shapely==1.8.5.post1
zstandard==0.18.0
When running nuitka --standalone I get a traceback that ends in
ValueError: invalid literal for int() with base 10: 'post1'
The traceback mentioned module versions so I checked pip freeze and found the culprit
Fixed the issue by getting another version of Shapely.

Thanks for your report, this is fixed on the factory branch, which is a development version under rapid development. You can try it out by going here: https://nuitka.net/doc/factory.html
Feedback if this is working is very welcome, just please do not share plans of doing it, but rather confirmations or denials of it working.

Can we start posting Nuitka bugs related to Python 3.11 or it isn't allowed yet?

Works on factory, thanks!

This is part of the hotfix release 1.1.7 that I just made.


bug

https://github.com/metaphysicser/PS-Mixer/blob/3e4fc7fc75b73c90f34e699bf5faac22d18d7022/new_models.py#L124
Hi,
I found a bug "'PS_Mixer' object has no attribute 'vlayer_norm'"

Thank you for finding the bug.  I have completed the missing code in 77-79 line of new_model.py


LAB_05-Implement_Intersite_Connectivity.md

English:
Virtual machine
Current Translation:
Komputer virtual
Suggested Translation:
Mesin virtual

Hello contributor,
Thank you for your contribution! We are processing your suggestions and we will update you shortly.
Kind regards,
Microsoft Worldwide Learning Team

Run the bugfix tool and waiting for result. ICMS QUERY

This issue has been fixed, so I close the PR.


feat: 🎸 Angular in-page wizard step card

Does the component already exist

[x]  I confirm that I have checked if the component already exists in any of the Green libraries

For which framework/library you are requesting the component
Angular
Description
The In-page wizard step card as a Angular component

Who will benefit from this feature?
No response
Anything else?
Added disableNext option for disabling next button if validation of input values fails.

@splashdust Updated with tests

@Fredrik-Oberg Thank you for the added tests!
I made a small change to the code, renaming the module to simply InPageWizardModule, since we may want to extend the functionality to include a complete generalized wizard implementation in the future. In that case I think the InPageWizardStepCardComponent should be one of a handful of components exported by that module.
Let me know if you disagree!
Otherwise everything looks good to me, and this PR is good to merge.


remove legacy compatibility with jekyll-paginate

I noticed that some code was marked as REMOVE AFTER 2018-01-01, so I removed it.

Hi @sverrirs,
It doesn't look like this PR conflicts to me according to GitHub? Maybe it did when you wrote this? I might have based this PR on top of #80, so since you merged that (thanks! :+1: ) it no longer conflicts?
Let me know if it still conflicts now and I'll check again with a clone locally.


Move reset to DisplayModeTrait

The functionality is common, so the implementation can be as well.
Hi! Thank you for helping out with SSD1306 development! Please:

[ ] Check that you've added documentation to any new methods
[ ] Rebase from master if you're not already up to date
[ ] Add or modify an example if there are changes to the public API
[ ] Add a CHANGELOG.md entry in the Unreleased section under the appropriate heading (Added, Fixed, Changed, etc)
[ ] Run rustfmt on the project with cargo fmt --all - CI will not pass without this step
[ ] Check that your branch is up to date with master and that CI is passing once the PR is opened

PR description
Just a bit of cleanup. Should be backwards compatible since the trait is in the prelude. I don't think there should be a changelog entry for this one.

Fair enough. I also marked it as breaking

No worries, thanks for accepting my contributions :)

Any time!


Filters in Data tab are not saved after applying

After applying a filter in the data tab of a table this filter is not saved in the list of previously used filters (I think the caret down button to the right of a text field is supposed to hold these filters)
MacOS 10.14.1
DBeaver 5.2.3

Yes, it's very handy function but it couldn't save. and can't remember the previous histories.

Duplicate of https://github.com/dbeaver/dbeaver/issues/10343.


Css class sorting changes if formatted on save or by a script

Hello,
With a specific config, the css classes order is different when formatted on save and when formatted by a script.
I made a repo with a minimal reproduction to show the bug: https://github.com/francoisromain/bug-svelte-tailwind
To reproduce the bug, in VS Code:

Clone this repo
Install the dependencies : npm i
Install the required VS Code extensions : svelte.svelte-vscode, dbaeumer.vscode-eslint, esbenp.prettier-vscode and bradlc.vscode-tailwindcss.
Open the file src/routes/index.svelte and save the file. The order of the css classes changes.
Run npm run format. The order of the css classes changes again.

If we remove the boxShadow property in tailwind.config.cjs, the css order stays the same when formatted on save and by a script.

However, if I add a prettier.config.cjs file, it works fine.

Brilliant, thanks for confirming! I'll close this since your issue is resolved 👍


Home ????

So many questions including, Why?

There were certain issues that needed to be fixed, so fixed it. check now. It was declared to give a homepage to the doodlecollab, more fixes are pending will do soon

@ItsRoy69 the home page still looks the same in vercel, if there are things to do in homepage, I would like to do that, you can assign me

NHA! stills looks same to me.


Feedback to incorrect send & seed import

https://github.com/clemahieu/raiblocks/issues/151

unlock wallets with empty passwords


Live2d demo crash

axmol version: 2.1
devices test on: Windows10

I known the reason of the crash is mismatched GLSL version.

After modified the glsl at here(the screenshot is origin code, I've modified ,such as: attribute -> in; gl_FragColor -> out fragColor...):

I still get the crash at here:

Any idea ? Thank you very much!

The screenshots you provided show that the shader code is embedded into your CPP code, and if that is the case, then that is most likely the problem.  Axmol 2.0 requires that the shader source code is in separate files, as they are processed during build time to generate the correct shaders for different platforms.
Refer to #1279
This may not be documented clearly at the moment, so that needs to be addressed.

No need to do anything.  A PR will be up soon with the updates to get Live2D working with Axmol 2.x

Thank you very much!


Another fake polkastarter, "polkastarter.fund"

Thank you, closing. Was PR-ed in https://github.com/polkadot-js/phishing/pull/229 a short while before this.


Use rebuild image from docker hub

Now that there's an image on docker hub.

Thanks :+1:

Looks like this didn't get pushed to https://hub.docker.com/r/markstory/lint-review/

Dockerhub failed to build the images because of issues with ubuntu. I'll figure out how to retry the build.


Populates linked dataset and attributes

This PR populates the created links panel (left) and the identity links (right).
It also display the actual name of the attributes, the ones in the data file and not the renamed ones in session file.
Code changes

adds attributes in the sharedModel
adds a model in the link editor to compute the component links only once

Thanks @brichet


Documentation: add PD to production users

PTAL @xiang90

@siddontang please update the commit title to Documentation: add PD to production users so the commit conforms with https://github.com/coreos/etcd/blob/master/CONTRIBUTING.md#format-of-the-commit-message. Thanks!

@siddontang I meant the title / first line in the commit itself-- change with git rebase -i master and squash into a single commit? Thanks!

@heyitsanthony Done :-)

Caught some typos. Should be good to merge after the fixup. Sorry about the back and forth. Thanks again!

Thanks @heyitsanthony

LGTM. Thanks.

Codecov Report

Merging #7428 into master will decrease coverage by -0.25%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #7428      +/-   ##
==========================================
- Coverage   70.88%   70.64%   -0.25%
==========================================
  Files         236      236
  Lines       21106    21106
==========================================
- Hits        14962    14911      -51
- Misses       5017     5063      +46
- Partials     1127     1132       +5

Impacted Files
Coverage Δ

proxy/grpcproxy/register.go
60.71% <0%> (-17.86%)
:x:

pkg/adt/interval_tree.go
77.7% <0%> (-11.49%)
:x:

pkg/testutil/recorder.go
66.66% <0%> (-3.71%)
:x:

clientv3/txn.go
96.36% <0%> (-3.64%)
:x:

clientv3/watch.go
91.9% <0%> (-1.83%)
:x:

etcdmain/etcd.go
45.64% <0%> (-1.25%)
:x:

clientv3/lease.go
91.11% <0%> (-0.89%)
:x:

etcdserver/server.go
79.17% <0%> (-0.36%)
:x:

etcdserver/v3_server.go
62.71% <0%> (ø)
:white_check_mark:

pkg/transport/timeout_conn.go
80% <0%> (ø)
:white_check_mark:

... and 3 more

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 e5d94a2...a4a8418. Read the comment docs.


Cache XNames

Fixes #19
This contains two commits.
The first caches XName lookups for ClrPropertyInfo. This replaces the codegen of 4 XName.Get calls with a single one: (1) getter, (2) setter, (3) code model in ctor, (4) local types dictionary.
(1) and (2) are esp. interesting because they can be in hot paths if you're processing a lot of large documents.
The second caches the single IXMetaData.SchemaName lookup. This one is called a lot during document processing so it's worth avoiding the expensive XName lookup.
One could try to improve the codegen further by reusing the SchemaName cache for ClrPropertyInfo or getting rid of the XName lookups in the global types dictionary. Those tasks are more complex and have virtually no runtime benefit so I stopped here.
For your convenience, I have attached a before/after of one of my schemas.
test.zip

@mamift Before you do a release, I'm gonna fix a bug blunder in this PR!!
The caching fields are declared as string, they should be XName of course!
I'll do a PR to fix that today.


External kwarg to TMJets solver

With this PR we can call TMJets with an external solver from TaylorModels using the external=true flag. Pass a solver_name to specify using a function different than validated_integ.
cc: @UzielLinares

@UzielLineares, @lbenet this is ready from my side. see an example notebook here.  more proper docs and testing can come in a follow up pr.
note that you can pass

external=true (required to use a solver different than the one in src/Algorithms/TMJets/reach.jl
solver_name (optional) if you want to pass a function name which is different than TaylorModels.validated_integ
solver_kwargs (optional) if you want to pass additional keyword arguments to the solver function

Looks great! I'll check out.

Thanks a lot!


KHR_materials_Specular

This branch contains the change for KHR_materials_specular support. Tested on test model and some edtited models.
For android, due to level support, we force material changes as below:

KHR_materials_unlit and KHR_materials_specular, the ulit is dropped.
KHR_materials_specular but we also have clearcoat normal texture or clear coat roghness texture, in this case we drop the latter two.
KHR_materials_sheen and we have specular color texture, in this case we drop specular color texture.
KHR_materials_volume and has specular color texture, in this case we drop specular color texture.

I have pushed change in google/draco so not sure if I still need to sign the form again?

@PsycoTodd would you be able to update the branch? I would like to try and get this PR in again. thanks!

Trying this weekend. Will let u know how it goes
On Wed, May 29, 2024 at 12:11 PM Mathias Agopian @.***>
wrote:

@PsycoTodd https://github.com/PsycoTodd would you be able to update the
branch? I would like to try and get this PR in again. thanks!
—
Reply to this email directly, view it on GitHub
https://github.com/google/filament/pull/7564#issuecomment-2138089166,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACCQBDW2VPZONSAMV3TPHGLZEYR4VAVCNFSM6AAAAABDAO4VI6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCMZYGA4DSMJWGY
.
You are receiving this because you were mentioned.Message ID:
@.***>

Merged my change from the current mainline and have the pull request.

Will the readme and version update by someone else? I think some release job need to be done from the team?

@PsycoTodd would you be able to rebase this PR one more time (it has conflicts again), I'll try to merge it this week.

sure, let me do this. And I suppose I need also update the readme a little
bit to include the extension support announcement.
On Tue, Jun 18, 2024 at 12:08 AM Mathias Agopian @.***>
wrote:

@PsycoTodd https://github.com/PsycoTodd would you be able to rebase
this PR one more time (it has conflicts again), I'll try to merge it this
week.
—
Reply to this email directly, view it on GitHub
https://github.com/google/filament/pull/7564#issuecomment-2175326238,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACCQBDWJ2G45QZ6KNV6FCILZH7MIVAVCNFSM6AAAAABDAO4VI6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCNZVGMZDMMRTHA
.
You are receiving this because you were mentioned.Message ID:
@.***>


[ENH] Define and add small dataset used to showcase and test hierarchical forecasting. Add synthetic data generators.

For development and testing in hierarchical forecasting we'll need:

[ ] at least one data set like airline, but hierarchical (at least two levels of hierarchy)
[x] a data generator like _make_series, but hierarchical, input should be: number variables, number of levels, perhaps also number of units per level

task 2 is solved by #2093

task 1 is solved by _bottom_hier_datagen


Perl/tk menus not working for graphical applications

Windows Build Number
Microsoft Windows [Version 10.0.22000.132]
WSL Version

[X] WSL 2
[ ] WSL 1

Kernel Version
5.10.43
Distro Version
Ubuntu  20.10
Other Software
Texlive 2021
Perl/tk 1:804.033-2build3
Repro Steps

Install texlive 2021
Install sudo apt-get perl-tk
Open tlmgr --gui
Open a menu
See the menu floating around.  Also, the menu doesn't work

Expected Behavior
Working menus in the correct places on the graphical interface (based on python-tk)
Actual Behavior
Menus floating around and not working at all.  Clicking in the options makes the menu disappear and opening it again doesn't work.
Diagnostic Logs
No response

Can you please file this issue on the WLSg repo as that is where we track GUI app related issues! Thank you!


On Windows, multiple duplicate tag files generated with increasing number of backslashes

Describe the bug
On Windows 10, I have source code on A:\ drive. Opening a file with gutentags correctly generates the tag file in a tmp directory I have specified, and editing the file causes that tag file to be correctly updated. But if I attempt to use the tag file in vim (jump to definition in same file) gutentags does not use the file it generated, it generates another duplicate tag file, this time with an extra backslashes in the path. Each time I jump to definition, it regenerates the tag file with more backslashes. It does jump to the correct tab, but spits out an error relating to chdir function.
Steps to reproduce

Open file on secondary drive, observe that tag file is generated correctly with no error messages in g:gutentags_cache_dir
In the same file, jump to the definition of a symbol (g C-])
Observe that gutentags reports that no tag file is found (but it just generated it), it regenerates a second one with extra backslash after drive letter name, then uses that to jump to definition. It reports an error regarding chdir. But it does jump to the correct definition.
Do step 2 again and observe the same thing, this time with yet another backslash in generated tag name.

Share your setup

OS: Windows 10 Home x64 ver. 1803
Vim: 8.1 w/ patches 1-524, compiled Nov 12 2018 (a full compiled version from https://tuxproject.de/projects/vim/)
ctags: Universal Ctags 0.0.0(befcb888)
g:gutentags_cache_dir=C:\Users\Chris\vimfiles\.tmp\tags

Gutentags config:
  let g:gutentags_enabled                   = 1
  let g:gutentags_trace                     = 1
  let g:gutentags_add_default_project_roots = 1
  let g:gutentags_project_root              = ['package.json']
  let g:gutentags_exclude_filetypes         = []
  let g:gutentags_exclude_project_root      = []
  let g:gutentags_generate_on_missing       = 1
  let g:gutentags_generate_on_new           = 1
  let g:gutentags_generate_on_write         = 1
  let g:gutentags_generate_on_empty_buffer  = 0
  let g:gutentags_cache_dir                 = expand(g:crf_tmp_vim_dir . '/tags')
  let g:gutentags_ctags_tagfile             = 'tags'
  let g:gutentags_ctags_auto_set_tags       = 1
  let g:gutentags_resolve_symlinks          = 0
  let g:gutentags_ctags_exclude_wildignore  = 1
  noremap [tags]u :<C-u>GutentagsUpdate<CR>
  noremap [tags]d :<C-u>GutentagsToggleEnabled<CR>
  if exists("g:plugs['lightline.vim']")
    augroup crf_vimrc
      autocmd User GutentagsUpdating call lightline#update()
      autocmd User GutentagsUpdated call lightline#update()
    augroup END
  endif

Post the logs
This is the logging for step 1 above:
Locking tags file...
Removing references to: A:\Development\pcm-app\lambdas\api\src\model\simulation.js
findstr /V /C:"A:\Development\pcm-app\lambdas\api\src\model\simulation.js" "C:\Users\Chris\vimfiles\.tmp\tags\A--Development-pcm-app-lambdas-api-tags" > "C:\Users\Chris\vimfiles\.tmp\tags\A--Development-pcm-app-lambdas-api-tags.temp"
Running ctags
call "ctags" -f "C:\Users\Chris\vimfiles\.tmp\tags\A--Development-pcm-app-lambdas-api-tags.temp"  --options="C:\Users\Chris\vimfiles\.tmp\plug\vim-gutentags\res\ctags_recursive.options" --exclude="@C:\Users\Chris\vimfiles\.tmp\tags/_wildignore.options" --append "A:\Development\pcm-app\lambdas\api\src\model\simulation.js"
Replacing tags file
move /Y "C:\Users\Chris\vimfiles\.tmp\tags\A--Development-pcm-app-lambdas-api-tags.temp" "C:\Users\Chris\vimfiles\.tmp\tags\A--Development-pcm-app-lambdas-api-tags"
        1 file(s) moved.
Unlocking tags file...
Done.

Now this is the logging for step 2 (when things go wrong):
Locking tags file...
Running ctags
call "ctags" -f "C:\Users\Chris\vimfiles\.tmp\tags\A---Development-pcm-app-lambdas-api-tags.temp"  --options="C:\Users\Chris\vimfiles\.tmp\plug\vim-gutentags\res\ctags_recursive.options" --exclude="@C:\Users\Chris\vimfiles\.tmp\tags/_wildignore.options" "A:\\Development\pcm-app\lambdas\api"
Replacing tags file
move /Y "C:\Users\Chris\vimfiles\.tmp\tags\A---Development-pcm-app-lambdas-api-tags.temp" "C:\Users\Chris\vimfiles\.tmp\tags\A---Development-pcm-app-lambdas-api-tags"
        1 file(s) moved.
Unlocking tags file...
Done.

And the third time:
Locking tags file...
Running ctags
call "ctags" -f "C:\Users\Chris\vimfiles\.tmp\tags\A-----Development-pcm-app-lambdas-api-tags.temp"  --options="C:\Users\Chris\vimfiles\.tmp\plug\vim-gutentags\res\ctags_recursive.options" --exclude="@C:\Users\Chris\vimfiles\.tmp\tags/_wildignore.options" "A:\\\\Development\pcm-app\lambdas\api"
Replacing tags file
move /Y "C:\Users\Chris\vimfiles\.tmp\tags\A-----Development-pcm-app-lambdas-api-tags.temp" "C:\Users\Chris\vimfiles\.tmp\tags\A-----Development-pcm-app-lambdas-api-tags"
        1 file(s) moved.
Unlocking tags file...
Done.

The chdir error is not captured in the logs, but looks like this. it spits out after trying to jump to definition, assuming it's related to the duplicate tag file problem:

Additional context
Here you can see the duplicate tag files it generates for a file located at A:\Development\pcm-app\lambdas\api\src\model\simulation.js:

i have this bug, could you tell me the way to solve this bug.
Thank you very much!

I don't think I ever solved it (not on Windows anymore so can't check). I believe it will work if you keep your files on C:/ drive though.


Stats for Open Access week campaign

This week is OA week and we can have a few tweets on the theme for both Open Targets Genetics and Platform.
If you had some stats on the data in Open Targets Genetics I could write a tweet like:
**Q**: Where can you find ??? million variants, ??? thousand human genes, ??? traits, ??? eQTLs, and ??? pQTLs that are #openaccess in the #OAweek & every other week? **A**: Open Targets Genetics: https://genetics.opentargets.org/
It'd be nice to fill in the blanks in the tweet, but other stats that you think are relevant too would be welcomed :)
The stats for the Open Targets Platform that we are interested in usually are:

number of targets
number of diseases
number of evidence strings
number of associations

It's very easy with the Platform to get the stats based on this API call:
https://api.opentargets.io/v3/platform/public/utils/stats
(useful for dissemination of our tools and what they provide, or a based on).

Sorry, we missed this. We should look to add such stats to the API in the future.

Updated queries using big query
# 19,922 protein-coding genes linked to 9,096 GWAS studies
SELECT
  COUNT(DISTINCT(study_id)),
  COUNT(DISTINCT(gene_id))
FROM
  `open-targets-genetics.190303.disease2variant2gene`
JOIN
  `open-targets-genetics.190303.genes`
USING
  (gene_id)
WHERE
  biotype = "protein_coding"

# 299,463,830 evidence strings linking disease to genes
SELECT
  COUNT(*)
FROM
  `open-targets-genetics.190303.disease2variant2gene`

# 140,799,9517 evidence strings linking variants to genes
SELECT
  COUNT(*)
FROM
  `open-targets-genetics.190303.variant2gene`

# 99,562 GWAS disease-associated loci from GWAS Catalog and UK Biobank
SELECT
  COUNT(*)
FROM (
  SELECT
    DISTINCT study_id,
    lead_chrom,
    lead_pos,
    lead_ref,
    lead_alt
  FROM
    `open-targets-genetics.190303.variant2disease`)


Add Github Enterprise Support

Added --enterprise-url parameter to override default github or gitlab url and use a personal token to authenticate with basic auth per git-go's documentation here https://github.com/go-git/go-git/blob/master/_examples/clone/auth/basic/access_token/main.go.  Will also update the links on the results web page to point at the override url specified. This needs to be tested for gitlab... I don't have an enterprise instance to work with.

@funkwhatyouheard  I reviewed the other, you can make any changes necessary in that #60 in this branch and then I will make a note in #60 and we can continue in #61. :+1:

@mattyjones implemented most of the changes. Think all that's outstanding is moving auth into NewClient() and ambiguity around enterprise vs. public github variable names, logging, etc. I left comments on https://github.com/N0MoreSecr3ts/wraith/pull/60 to give them more context, but we can switch over here going forward with that's preferred. Let me know your thoughts.

@funkwhatyouheard You are all set, thanks for the effort and understanding through this. As an FYI, Hacktoberfest is coming and there are and will be smaller issues that can be worked on, that will count towards your pr count. I have done this for many years and it is a lot of fun and a great way to get some cool swag from Github and its partners.

@mattyjones Awesome, and not a problem at all. I'll certainly keep an eye out for Hacktoberfest. I have another branch that I implemented outputting the results to a csv, but the code is atrocious. My Go is pretty infantile at this point but I'd like to find a cleaner way to get it working. Logic should be pretty near identical for json output as well once it's good. I'll work on getting something over soon for those two features. This project will save me quite a bit of time and effort so thanks for all your hard work to get it where it is!


Allow value mapping for aggregation bucket labels

An example configuration for this functionality would be: [{'aggName': 'access_right',            'field': 'access_right',            'title': 'Access_right',            'mapping': {'open': 'Open'}},           {'aggName': 'resource_type',            'field': 'resource_type.type',            'title': 'Resource_type'}]

Should be done on the REST API level.

This is was discussed in the vocabularies context and ILS implementation: such mapping is too simplistic. Limitations:

thousands of possible values, which cannot be easily done via an object mapping like in the example
it does not allow translations

We should find a solution when implementing vocabularies.


Fix NullPointer when clicking on image in MediaDetailFragment (#3730)…

Description (required)
Uri.parse would throw NullPointerException if the image in MediaDetailFragment was clicked before the image URL is available
Tests performed (required)
Tested betaDebug on S7

Seems like Travis failed?

Looking into it @misaochan

Actually 2.13 somehow has a compile-time error

Fixed @misaochan


ui: statements page search and pagination

PR: https://github.com/cockroachdb/cockroach/pull/41641
From release notes:

Added page search and pagination to the Statements page. [#41641][#41641] {% comment %}doc{% endcomment %}

Resolved with #6821.