Can't retrieve top posts

Is there a way to retrieve top posts from a hashtag? I'm only getting the most recent ones and then sorting them by most popular..

I know this is very old, but it's no longer possible with Instgram's current API.


Add WDK to VS 2022 image

Tool name
WDK
Tool license
Microsoft
Add or update?

[X] Add
[ ] Update

Desired version
Latest
Approximate size
No response
Brief description of tool
The WDK is used for building kernel mode binaries. It was added to the 2019 image, but because a 2022 was not available at all until recently,
https://github.com/actions/virtual-environments/issues/4139 was made by our team earlier, and closed pending a WDK release. Now that one has been released (Even though its a preview) we would like it installed so we can test our pipelines.
URL for tool's homepage
https://docs.microsoft.com/en-us/windows-hardware/drivers/installing-preview-versions-wdk
Provide a basic test case to validate the tool's functionality.
No response
Virtual environments affected

[ ] Ubuntu 18.04
[ ] Ubuntu 20.04
[ ] macOS 10.15
[ ] macOS 11
[ ] Windows Server 2016
[ ] Windows Server 2019
[X] Windows Server 2022

Can this tool be installed during the build?
No response
Tool installation time in runtime
No response
Are you willing to submit a PR?
No response

Hello @thhous-msft , are you willing to submit PR?

As per discussion with @thhous-msft , there are not public links to download WDK/SDK in CI, we should wait an official release.

@thhous-msft please re-open this one or open a new issue when the public release is available.
Thank you!

@thhous-msft thanks for the update! We will take care of adding wdk to win2022

We also need the WDK. How about adding it and including a release note in the image readme?

@al-cheb  I don't think anyone is going to debug drivers with the VS gui in a CI pipeline, so the regression is not relevant to this issue, you could simply just not install the debug extensions.

@namazso , I agree but it breaks VS first initialization creating additional windows with errors.

If you don't install it it doesn't. It's optional in the WDK installer

@al-cheb Here is the full list of features from wdksetup.exe /help:
Windows Driver Kit - Windows 10.0.22621.1
---------------------------
Available features: (Features marked with * can be downloaded but cannot be installed on this computer)

   OptionId.WindowsDriverKitComplete
   OptionId.WindowsDriverKitARMAddons
   OptionId.WDKVisualStudioDev17
   OptionId.WDKVisualStudioDownlevel
   OptionId.ARMSoftwareDevelopmentKit
   OptionId.ARMWindowsDesktopDebuggers
   OptionId.MinWLPC

For CI pipelines, the first is likely sufficient: wdksetup.exe /features OptionId.WindowsDriverKitComplete /quiet

how to use it?
can any tell me?


add ebank

Hi
I added a project I wrote myself.
I hope I can help some people.

Sorry, not enough stars: https://github.com/chiraggude/awesome-laravel/blob/master/CONTRIBUTING.md


Broken on OS X?

Hi there, tried mkv, avi, wmv, mp4 with no avail (multiple test files from different sources neither work).
This is the following output:
BUILD LOG: merge_mat: -D DOUBLE_SUPPORT
Error getting function data from server
OpenCV Error: Gpu API call (CL_BUILD_PROGRAM_FAILURE) in getOrBuildProgram, file /tmp/opencv20160201-24683-1mh5c76/opencv-2.4.12/modules/ocl/src/cl_programcache.cpp, line 445
libc++abi.dylib: terminating with uncaught exception of type cv::Exception: /tmp/opencv20160201-24683-1mh5c76/opencv-2.4.12/modules/ocl/src/cl_programcache.cpp:445: error: (-217) CL_BUILD_PROGRAM_FAILURE in function getOrBuildProgram

[1]    33626 abort      butterflow

Tried with: bufferflow -sm <file>, bufferflow <file>, bufferflow -sm -r 60 <file>, bufferflow -sm -r 2x <file>.

Hey, butterflow still works for me on my late 2008 MacBook, albeit only with the -sw flag enabled, and on my newer machines.
Can you tell me:

Did you install butterflow and opencv with Homebrew, or from source?
If you used Homebrew, what the output of brew info butterflow opencv is?
What the output of butterflow -d is?

Also, to make sure we're on the same page, in terms of testing, you should use this video and try running butterflow -a -mrk -vs 0.9 -r 3x -t -s a=0,b=0.5,spd=1 ~/Desktop/coraline.mp4 against it.

Hello!

I installed from Homebrew, I initially tried brew install homebrew/science/butterflow but didn't work at all - I don't think that command installed any deps. So I simply did brew install butterflow instead (our current state).

brew info butterflow opencv

homebrew/science/butterflow: stable 0.2.0 (bottled)
Makes fluid slowmo and motion interpolated videos
https://github.com/dthpham/butterflow
/usr/local/Cellar/butterflow/0.2.0_1 (32 files, 210.0K) *
  Poured from bottle
From: https://github.com/Homebrew/homebrew-science/blob/master/butterflow.rb
==> Dependencies
Required: ffmpeg ✔, opencv ✔

homebrew/science/opencv: stable 2.4.12 (bottled), HEAD
Open source computer vision library
http://opencv.org/
/usr/local/Cellar/opencv/2.4.12_2 (225 files, 35.4M) *
  Built from source with: --with-ffmpeg --with-opengl
From: https://github.com/Homebrew/homebrew-science/blob/master/opencv.rb
==> Dependencies
Build: cmake ✔, pkg-config ✔
Required: jpeg ✔, libpng ✔, libtiff ✔
Recommended: eigen ✔, openexr ✔, homebrew/python/numpy ✔
Optional: gstreamer ✘, jasper ✘, libdc1394 ✘, openni ✘, qt ✔, tbb ✘, vtk ✘, ffmpeg ✔
==> Options
--32-bit
        Build 32-bit only
--c++11
        Build using C++11 mode
--universal
        Build a universal binary
--with-cuda
        Build with CUDA support
--with-ffmpeg
        Build with ffmpeg support
--with-gstreamer
        Build with gstreamer support
--with-jasper
        Build with jasper support
--with-java
        Build with Java support
--with-libdc1394
        Build with libdc1394 support
--with-opengl
        Build with OpenGL support
--with-openni
        Build with openni support
--with-qt
        Build the Qt4 backend to HighGUI
--with-quicktime
        Use QuickTime for Video I/O instead of QTKit
--with-tbb
        Enable parallel code in OpenCV using Intel TBB
--with-vtk
        Build with vtk support
--with-ximea
        Build with XIMEA support
--without-eigen
        Build without eigen support
--without-numpy
        Use a numpy you've installed yourself instead of a Homebrew-packaged numpy
--without-opencl
        Disable GPU code in OpenCV using OpenCL
--without-openexr
        Build without openexr support
--without-python
        Build without Python support
--without-tests
        Build without accuracy & performance tests
--HEAD
        Install HEAD version
==> Caveats
Python modules have been installed and Homebrew's site-packages is not
in your Python sys.path, so you will not be able to import the modules
this formula installed. If you plan to develop with these modules,
please run:
  mkdir -p /Users/georgij/Library/Python/2.7/lib/python/site-packages
  echo 'import site; site.addsitedir("/usr/local/lib/python2.7/site-packages")' >> /Users/georgij/Library/Python/2.7/lib/python/site-packages/homebrew.pth

bufferflow -d

OpenCL devices:
  Platform              : Apple
  Platform Vendor       : Apple
  Platform Version      : OpenCL 1.2 (Feb 27 2015 01:29:10)
  Platform Profile      : FULL_PROFILE
    Device              : Intel(R) Core(TM) i7-3770 CPU @ 3.40GHz
      Version           : OpenCL 1.2
      Version           : 1.1
      Profile           : FULL_PROFILE
    Device              : GeForce GTX 675MX
      Version           : OpenCL 1.2
      Version           : 10.2.7 310.41.25f01
      Profile           : FULL_PROFILE

Downloaded coraline.mp4 added it to desktop.
butterflow -a -mrk -vs 0.9 -r 3x -t -s a=0,b=0.5,spd=1 ~/Desktop/coraline.mp4

window appears Python quit unexpectedly while using the libopencv_core.2.4.dylib plug-in.
and this is the following log is outputted on to the terminal:
BUILD LOG: merge_mat: -D DOUBLE_SUPPORT
Error getting function data from server
OpenCV Error: Gpu API call (CL_BUILD_PROGRAM_FAILURE) in getOrBuildProgram, file /tmp/opencv20160201-24683-1mh5c76/opencv-2.4.12/modules/ocl/src/cl_programcache.cpp, line 445
libc++abi.dylib: terminating with uncaught exception of type cv::Exception: /tmp/opencv20160201-24683-1mh5c76/opencv-2.4.12/modules/ocl/src/cl_programcache.cpp:445: error: (-217) CL_BUILD_PROGRAM_FAILURE in function getOrBuildProgram

[1]    1335 abort      butterflow -a -mrk -vs 0.9 -r 3x -t -s a=0,b=0.5,spd=1 ~/Desktop/coraline.mp4

Thanks!

Thanks for the output.
Try this, in a Terminal, and tell me if either command works:
export OPENCV_OPENCL_DEVICE=:CPU; butterflow -r2x coraline.mp4
export OPENCV_OPENCL_DEVICE=:GPU; butterflow -r2x coraline.mp4

Thanks for trying to help me,
here's the log for both commands:
Desktop ❯ export OPENCV_OPENCL_DEVICE=:CPU; butterflow -r2x coraline.mp4
OpenCV Error: Assertion failed (localThreads[1] <= ctx->getDeviceInfo().maxWorkItemSizes[1]) in openCLVerifyKernel, file /tmp/opencv20160201-24683-1mh5c76/opencv-2.4.12/modules/ocl/src/cl_operations.cpp, line 347
libc++abi.dylib: terminating with uncaught exception of type cv::Exception: /tmp/opencv20160201-24683-1mh5c76/opencv-2.4.12/modules/ocl/src/cl_operations.cpp:347: error: (-215) localThreads[1] <= ctx->getDeviceInfo().maxWorkItemSizes[1] in function openCLVerifyKernel

[1]    9353 abort      butterflow -r2x coraline.mp4

Desktop ❯ export OPENCV_OPENCL_DEVICE=:GPU; butterflow -r2x coraline.mp4

BUILD LOG: merge_mat: -D DOUBLE_SUPPORT
Error getting function data from server
OpenCV Error: Gpu API call (CL_BUILD_PROGRAM_FAILURE) in getOrBuildProgram, file /tmp/opencv20160201-24683-1mh5c76/opencv-2.4.12/modules/ocl/src/cl_programcache.cpp, line 445
libc++abi.dylib: terminating with uncaught exception of type cv::Exception: /tmp/opencv20160201-24683-1mh5c76/opencv-2.4.12/modules/ocl/src/cl_programcache.cpp:445: error: (-217) CL_BUILD_PROGRAM_FAILURE in function getOrBuildProgram

[1]    9389 abort      butterflow -r2x coraline.mp4

I'm having this same issue as well;
Mac-Pro-Server:pipeline neg2led$ butterflow -v --poly-s 0.9 -r 60 -o /Volumes/WDBlack/earthscope/output/theworld/20160115000000-20160116000000/video/interpolated-2-60fps-0.75x.mp4 -s a=0,b=8,spd=0.75:a=8,b=8.5,spd=0.375:a=8.5,b=43.5,spd=0.75:a=43.5,b=44,spd=0.375:a=44,b=end,spd=0.75 /tmp/animate-earth/theworld/20160115000000-20160116000000/original-2fps-lossless.mp4
DEBUG  : Rendering sequence:
DEBUG  : subregion: 0,15,16 0,8,8 0,0.112,0.112
DEBUG  : subregion: 15,16,2 8,8.5,0.5 0.112,0.119,0.00699
DEBUG  : subregion: 16,86,71 8.5,43.5,35 0.119,0.608,0.49
DEBUG  : subregion: 86,87,2 43.5,44,0.5 0.608,0.615,0.00699
DEBUG  : subregion: 87,142,56 44,71.5,27.5 0.615,1,0.385
DEBUG  : Working on subregion: 2
DEBUG  : fa: 0
DEBUG  : fb: 15
DEBUG  : ta: 0.0
DEBUG  : tb: 8000.0
DEBUG  : reg_dur: 8000.0
DEBUG  : reg_len: 16
DEBUG  : tgt_fps: None
DEBUG  : tgt_dur: None
DEBUG  : tgt_spd: 0.75 1.3x
DEBUG  : tgt_frs: 640
DEBUG  : ts: 0.02..0.98
DEBUG  : int_each_go: 42
DEBUG  : wr_per_pair: 43
DEBUG  : pairs: 15
DEBUG  : will_make: 645
DEBUG  : extra_frs: 5
DEBUG  : dup_every: 0
DEBUG  : drp_every: 129.0
DEBUG  : est_dur: 10.6666666667
DEBUG  : pot_aud_drift: 0.0833333333333
DEBUG  : wrt_one: False
DEBUG  : runs: 16

BUILD LOG: merge_mat: -D DOUBLE_SUPPORT
Error getting function data from server
OpenCV Error: Gpu API call (CL_BUILD_PROGRAM_FAILURE) in getOrBuildProgram, file /tmp/opencv20160317-4444-12p3z8i/opencv-2.4.12/modules/ocl/src/cl_programcache.cpp, line 445
libc++abi.dylib: terminating with uncaught exception of type cv::Exception: /tmp/opencv20160317-4444-12p3z8i/opencv-2.4.12/modules/ocl/src/cl_programcache.cpp:445: error: (-217) CL_BUILD_PROGRAM_FAILURE in function getOrBuildProgram

Abort trap: 6

This is on a MacPro5,1 with a GTX660 and the latest nVidia/CUDA drivers.
I am also using butterflow version 0.2.1.dev0 to deal with issue #13

OK, this appears to be a driver issue. I swapped back to the GPU driver included with OS X instead of nVidia's latest, and it works fine.


Visual Studio Test (V 2.*) Task: How to retrieve TestRunId when Test Selection uses Test Plan.

Question, Bug, or Feature?
Type: Question
Enter Task Name:  Visual Studio Test V2
Environment
Azure Pipelines
Issue Description
Hi Team,
We are using Visual Studio Test (V 2.*) Task to execute Automated Tests under given Test plan. In Automated Test methods , we have custom logic to add more detail to Test Results using TestRunID. When we are executing from TestHub , we can see that $(test.RunId) is available. But we would like to get this when Test Selection uses Test Plan.
RunId Available upfront during Select tests using as Test Run

Question:
When Select tests using as TestPlan like below, TestRunId is getting created during this task execution internally and we don’t have any control. Actually we want to use that TestRunID in our Automated Test method implementations.
Is there any way to pass on this TestRunID to associated TestMethods ducting execution.

Added more details to query along with screenshots.

@mailtopavanc are you on AzureDevOps services or Azure DevOps server (TFS onprem)?

Closing due to no reply

@ShreyasRmsft
So, is it possible somehow to get TestRunId from VS test task when Test Selection uses Test Plan?
We are also interested in this topic.
We are using AzureDevOps services.
Thanks in advance.


Warped sceptre not calculating max hit or dps

Pretty straight forward, the new powered staff doesn't work in the dps calc. Would love to have this working as I use it quite frequently.

I apologize it seems to be working now, for some reason runelite did not update the plugin and reinstalling it fixed the issue.


Empty TextField size

In Flash, the size of an empty TF is 5x5 for any font and its size

When does 100x100 come up? Isn't that the default size when there is text?

Maybe the size is 4x4 if you use text auto-size? @RblSb Are you using auto-size, or is this a vanilla new TextField?
Thanks!

@jgranick I forgot that I'm using TextFieldAutoSize, you're right. 4x4 will be with any of its value except NONE.
So, if you remove the text == "" check here, then empty textfield will calculate the size correctly by calling getLineMeasurements () i think.

I guess it's the same either way. It would be more consistent to use getLineMeasurements() on the empty text, but you already know what the results will be for each property.

I just fixed the behavior of textWidth and textHeight when not type INPUT, I believe the current behavior is correct, in the tests I am running. Please let me know if we're missing something :smile:


add draw score

Oh yeah you are right. Forgot about that.
I'm on vacation right now till friday. Will update this pr as soon as I can.

Ok I added it.


Reentrant human input

Hi,
I'm looking at the code for the first time, and I'm trying to understand how to involve a human. Looking at the conversable_agent, here's the default implementation:
def get_human_input(self, prompt: str) -> str:
        reply = input(prompt)
        return reply

What if the human is an HTTP response away? What if they're an SMTP relay away? Slack message?
Whenever the human is somewhere else on the network, it seems like I need to keep the autogen process alive. If they take 1 day to respond, that's not ideal, that's a long time to keep an idle process running. How are you planning on dealing with such a scenario?

Yep, I had the same observation. For real-world usage, e.g., in a business applications scenario, we need some kind of a workflow engine in addition to the agent system.

If the architecture here doesn't support something like this, one way I would (will probably) approach it is to run autogen in some sort of lightweight VM, like firecracker, and when autogen signals that it's idle, pause the whole VM and save it for later reentrance.

I agree with the need to support reentrance or similar. In the two-agent scenario, we've worked around it in stateless environments (e.g., a http), by repopulating the histories, running without human intervention, and controlling the number of steps taken for each request.

@victordibia @gagb for awareness
I've seen many similar questions, and some doc or FAQ will be helpful.

Is there a plan for this feature in the current roadmap @sonichi ?

There are patterns for this in the cookbook. 0.4 has a different model and enables this easily. closing. please reopen with a specific proposal if desired.

There are patterns for this in the cookbook.

Unfortunately couldn't find any:
https://microsoft.github.io/autogen/dev//user-guide/core-user-guide/cookbook/index.html
Am I looking in the right place?

@attilavetesi-epam
What you are looking for is slow human input.
Core Sample: python/packages/autogen-core/samples/slow_human_in_loop.py
AgentChat also supports human/application input in the outer loop: https://microsoft.github.io/autogen/dev/user-guide/agentchat-user-guide/tutorial/teams.html#running-team


[CCP] Add some needed variables to cloud-socok8s

We need to set the INTERNAL_SUBNET variable for the deploy to work,
and set a unique ANSIBLE_RUNNER_DIR for each job to prevent cross-test
environment pollution.

But #3317 does changes the regular job testing master and does not touch the job that runs on Github PRs. This PR is the other way around. As said above, it would be nicer if both used the same job or at least same code.

But INTERNAL_SUBNET is not getting set based on the PREFIX in the cloud-socok8s job:
13:51:51 ~/workspace/cloud-socok8s/cloud-socok8s.TK98k ~/workspace/cloud-socok8s/cloud-socok8s.TK98k
13:51:51 Deploying on OpenStack
13:51:51 INTERNAL_SUBNET name not given. export INTERNAL_SUBNET=...
13:51:51 + ret=1

via https://ci.suse.de/job/cloud-socok8s/164/console

I don't know what the cloud-socok8s job does. The full ccp periodic doesn't need it because everywhere internal subnet is necessary, the variable deploy_on_openstack_internal_subnet is used. This is defined in vars as "{{ deploy_on_openstack_prefix }}-subnet" If there is a place where this is incorrectly defined, then we need to fix it in socok8s, not force more variables to be exported.

But INTERNAL_SUBNET is not getting set based on the PREFIX in the cloud-socok8s job:
13:51:51 ~/workspace/cloud-socok8s/cloud-socok8s.TK98k ~/workspace/cloud-socok8s/cloud-socok8s.TK98k
13:51:51 Deploying on OpenStack
13:51:51 INTERNAL_SUBNET name not given. export INTERNAL_SUBNET=...
13:51:51 + ret=1

via ci.suse.de/job/cloud-socok8s/164/console

After checking we seem to be defining INTERNAL_SUBNET in other jobs too.

I think the best way forward would be to use the same content as the other ccp job. If possible adapt said job to use a single script. I don't really know why this came to life, as the card to explain what to do in CI are in the backlog...


Support for Persistence

Description
Add function that create the Datavolume wich automatically create the pvc and wait for pvc import.
Add function that create persistent virtual machine.
Add function in creation_test and in controller_test that test the two function written above.
How Has This Been Tested?
Please describe the tests that you ran to verify your changes. Please also note any relevant details for your test configuration.

[ ] Test A
[ ] Test B

/rebase

Please take care of writing the proper description to your PR (starting with simple things such as "." at the end of a sentence, and capital letter at the beginning).
Persistance ==> Persistence
(a typo in the PR title looks a very bad start....)

Let's wait until #401 is merged, then we review this one. I fear we can create several conflicts otherwise.

Let's wait until #401 is merged, then we review this one. I fear we can create several conflicts otherwise.

/deploy-staging

/deploy-staging

/deploy-staging

/deploy-staging

/deploy-staging

/deploy-staging

/deploy-staging

/deploy-staging

/deploy-staging

/deploy-staging

/deploy-staging

/deploy-staging

/deploy-staging

/deploy-staging

/deploy-staging

/deploy-staging

/deploy-staging

/deploy-staging

/deploy-staging

/deploy-staging

/deploy-staging

/deploy-staging

/deploy-staging

/deploy-staging

/deploy-staging

/deploy-staging

/deploy-staging

/deploy-staging

/deploy-staging

/deploy-staging

/deploy-staging

/deploy-staging

/deploy-staging

/deploy-staging

/deploy-staging

/deploy-staging

/deploy-staging

/deploy-staging

/deploy-staging


Error during inference if patch size is larger than image

If the patch size is larger than the image size during inference with TorchIO, an error is raised. This implies that this situation wasn't being covered by the tests.
To do:

[ ] Write a unit test for this
[ ] Constrain the patch size to the image size during inference

Error:
Traceback (most recent call last):
  File "InnerEyePrivate/ML/runner.py", line 50, in <module>
    main()
  File "InnerEyePrivate/ML/runner.py", line 46, in main
    post_cross_validation_hook=runner.default_post_cross_validation_hook)
  File "/mnt/azureml/cr/j/5a59e32ccd764940b8143a7327f35cb5/exe/wd/innereye-deeplearning/InnerEye/ML/runner.py", line 451, in run
    return runner.run()
  File "/mnt/azureml/cr/j/5a59e32ccd764940b8143a7327f35cb5/exe/wd/innereye-deeplearning/InnerEye/ML/runner.py", line 218, in run
    self.run_in_situ(azure_run_info)
  File "/mnt/azureml/cr/j/5a59e32ccd764940b8143a7327f35cb5/exe/wd/innereye-deeplearning/InnerEye/ML/runner.py", line 407, in run_in_situ
    self.ml_runner.run()
  File "/mnt/azureml/cr/j/5a59e32ccd764940b8143a7327f35cb5/exe/wd/innereye-deeplearning/InnerEye/ML/run_ml.py", line 351, in run
    self.run_inference(checkpoint_paths_for_testing, ModelProcessing.DEFAULT)
  File "/mnt/azureml/cr/j/5a59e32ccd764940b8143a7327f35cb5/exe/wd/innereye-deeplearning/InnerEye/ML/run_ml.py", line 474, in run_inference
    model_proc=model_proc)
  File "/mnt/azureml/cr/j/5a59e32ccd764940b8143a7327f35cb5/exe/wd/innereye-deeplearning/InnerEye/ML/run_ml.py", line 698, in model_inference_train_and_test
    model_proc=model_proc)
  File "/mnt/azureml/cr/j/5a59e32ccd764940b8143a7327f35cb5/exe/wd/innereye-deeplearning/InnerEye/ML/model_testing.py", line 72, in model_test
    return segmentation_model_test(config, data_split, checkpoint_paths, model_proc)
  File "/mnt/azureml/cr/j/5a59e32ccd764940b8143a7327f35cb5/exe/wd/innereye-deeplearning/InnerEye/ML/model_testing.py", line 102, in segmentation_model_test
    epoch_and_split=epoch_and_split)
  File "/mnt/azureml/cr/j/5a59e32ccd764940b8143a7327f35cb5/exe/wd/innereye-deeplearning/InnerEye/ML/model_testing.py", line 166, in segmentation_model_test_epoch
    voxel_spacing_mm=sample.metadata.image_header.spacing
  File "/mnt/azureml/cr/j/5a59e32ccd764940b8143a7327f35cb5/exe/wd/innereye-deeplearning/InnerEye/ML/pipelines/inference.py", line 47, in predict_and_post_process_whole_image
    return self.post_process(self.predict_whole_image(image_channels, voxel_spacing_mm, mask, patient_id))
  File "/azureml-envs/azureml_019d9b5c5814e5f53d425814bccd13a8/lib/python3.7/site-packages/torch/autograd/grad_mode.py", line 28, in decorate_context
    return func(*args, **kwargs)
  File "/mnt/azureml/cr/j/5a59e32ccd764940b8143a7327f35cb5/exe/wd/innereye-deeplearning/InnerEye/ML/pipelines/inference.py", line 272, in predict_whole_image
    padding_mode=self.model_config.padding_mode.value,
  File "/azureml-envs/azureml_019d9b5c5814e5f53d425814bccd13a8/lib/python3.7/site-packages/torchio/data/sampler/grid.py", line 77, in __init__
    self.locations = self._compute_locations(self.subject)
  File "/azureml-envs/azureml_019d9b5c5814e5f53d425814bccd13a8/lib/python3.7/site-packages/torchio/data/sampler/grid.py", line 102, in _compute_locations
    self._parse_sizes(*sizes)
  File "/azureml-envs/azureml_019d9b5c5814e5f53d425814bccd13a8/lib/python3.7/site-packages/torchio/data/sampler/grid.py", line 131, in _parse_sizes
    raise ValueError(message)
ValueError: Patch size (48, 64, 48) cannot be larger than image size (35, 51, 35)

AB#5114

For some reason, this was not automatically corrected by
https://github.com/microsoft/InnerEye-DeepLearning/blob/821cb3be7a1b9cebbd06d49481e2024aa7cd0b06/InnerEye/ML/models/architectures/base_model.py#L77-L80


Adding subdomain  for student.mercubuana.ac.id

@dini4529 I am sorry, the domain mercubuana.ac.id is removed from our trusted domains list because the school allows to register an email address there even if you are not a student of the school. If you are the student of this school, please apply with official document (OFFICIAL DOCUMENT tab at https://www.jetbrains.com/shop/eform/students).


updates from previous review

incorporate review feedback from https://github.com/flutter/flutter/pull/4572

@yjbanov

Lgtm


Run shellcheck from the project root

This allows project specific shellcheck settings to be applied.
It took me quite a while to figure out why this plugin wasn't using the settings in the .shellcheckrc at the root of the repository. It's because shellcheck walks up the directory path until it finds the first .shellcheckrc. The default directory where the shellcheck process is executed is ~/Library/Application Support/Nova/Extensions/[extension directory].

@samdoran thanks for the pull request. Apologies for the delay in getting back to you. I've changed your modifications slightly to keep the existing behaviour (where it runs with the working directory in the Application Support folder) when there's no workspace path configured.
I'll endeavour to get this built and released ASAP.

Thank you so much!


add explicit request source type to label the external request like lightning/br

if use
set @tidb_request_source_type='lightling'
select count(*) from t;

the request source field will like:
external_Select_lightning

ptal @HuSharp  @glorv

Some tests fails.

@overvenus Can you leave a merge label? thx!

/merge


[Bug]: Value inside the date field is getting corrupted

Is there an existing issue for this?

[X] I have searched the existing issues

Current Behavior
I am fetching data from the select query on the users database and getting a proper response. The response object contains a date field which shows date in YYYY-MM-DD HH:mm format.

When I connect the data from the same query to the table widget, the column dob shows values as invalid date . The column input type is also set to plain text.

The data seems to be getting corrupted after it's getting stored in the query.data object. This is happening in both release and production environments.
Steps To Reproduce

Run the select query from the users sample db
Connect it to the table widget

Environment
Release
Version
Cloud

@parth-appsmith I'm not able to reproduce this. Can you please share a video?


add logseq-better-zotero-plugin

Submit a new Plugin to Marketplace
Plugin Github repo URL: https://github.com/schubsijan/logseq-better-zotero-plugin
Github releases checklist

[x] a legal package.json file.
[x] a valid CI workflow build action for Github releases. (Optional for theme plugin only).
[] a release which includes a release zip pkg from a successful build.
[x] a clear README file, ideally with an image or gif showcase. (For more friendly to users, it is recommended to have English version description).
[x] a license in the LICENSE file.

Hi @schubsijan You don't need to upload the source code of the plugin to this repository, just submit plugin metadata :)
A helpful PR https://github.com/logseq/marketplace/pull/394/files


Change the links in the single page to facet links (done for techniques, needs to be done for locations)

done


Use nameof in StoredGeneratedTestBase

Just because it's no longer to possible to see member names in double-quotes... :)

Merged. Thanks @roji


merge master into v1

includes changes in (105cae8, a057c9f]

I am looking to minimize the number of commits included in the massively-conflicting merge that will happen when we merge https://github.com/openshift/hive/pull/694 into v1.

/lgtm


./main.go:491: undefined: Asset

I'm not sure where the problem comes from, but I thought I'd report it in case you know what's up.
building with a Dockerfile:
FROM golang

RUN apt-get update \
        && apt-get install -yq libssl-dev

# FROM golang:onbuild
RUN mkdir -p /go/src/app
WORKDIR /go/src/app

# this will ideally be built by the ONBUILD below ;)
CMD ["go-wrapper", "run"]

COPY . /go/src/app
RUN go-wrapper download
RUN go-wrapper install

ends with the error:
Step 7 : RUN go-wrapper install
 ---> Running in 27b8c64c17c3
+ exec go install -v
github.com/alecrn/git-remote-grave/tarpack
github.com/gcmurphy/getpass
golang.org/x/crypto/poly1305
golang.org/x/crypto/salsa20/salsa
golang.org/x/crypto/nacl/secretbox
app
# app
./main.go:491: undefined: Asset
./main.go:621: undefined: Asset

Thanks for pointing this out.  The Asset function is provided by bindata.go, which is generated by running 'go generate'.  I didn't think that 'go get' wouldn't run it.
To fix it, I've checked in the files generated by go generate.


Fix link to the Symfony blog

The readme is using rST, not markdown, so a markdown link does not work well.

Thank you @stof.


external jenkins  401 Unauthorized

What is version of KubeSphere DevOps has the issue?
3.2.1
How did you install the Kubernetes? Or what is the Kubernetes distribution?
No response
Describe this feature
devops-config
devops:
  host: http://<IP_ADDRESS>:8080
  maxConnections: "100"
  namespace: kubesphere-devops-system
  password: eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJ1c2VybmFtZSI6ImFkbWluIiwidG9rZW5fdHlwZSI6ImFjY2Vzc190b2tlbiIsImlhdCI6MTY0OTM5MDgyOSwiaXNzIjoia3ViZXNwaGVyZSIsIm5iZiI6MTY0OTM5MDgyOX0.85dhN45U_kyf_50CbfCWPelZxYf7_Jb_QeMuYrDxR7g
  username: admin
  workerNamespace: kubesphere-devops-worker

this password how to make? jenkins api token ?
Additional information
No response

/kind support

You could use the API token generated from the Jenkins user. Then add a flag to the apiserver and controller.
--override-jenkins-token=false

See also https://github.com/kubesphere/ks-devops/blob/b2362c1d0ac4f86908070d1499f8a0a79774bcdd/cmd/tools/jwt/app/jwt_root.go#L56

You could use the API token generated from the Jenkins user. Then add a flag to the apiserver and controller.
--override-jenkins-token=false

See also
https://github.com/kubesphere/ks-devops/blob/b2362c1d0ac4f86908070d1499f8a0a79774bcdd/cmd/tools/jwt/app/jwt_root.go#L56

I set override-jenkins-token=false, but it looks like invalid。

the apiserver and controller also set --override-jenkins-token=false

Please use the token from Jenkins user instead.

Please use the token from Jenkins user instead.

i am sure, i used jenkins api token.

I refer to this article。
https://github.com/kubesphere/ks-devops/issues/429
but i met 401 Unauthorized

Please use the token from Jenkins user instead.

Using regular users is the same mistake

 devops-controller logs

@LinuxSuRen help please!  thank you!

Just make sure the following fields are correct if you got 401.
devops:
  host:
  password:
  username:


feat: show message when no positivi predictions available for evaluation

Codecov Report

Merging #519 (e17af65) into main (8fc0ddc) will decrease coverage by 0.07%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main     #519      +/-   ##
==========================================
- Coverage   79.82%   79.76%   -0.07%
==========================================
  Files          63       63
  Lines        4343     4343
==========================================
- Hits         3467     3464       -3
- Misses        876      879       +3

see 2 files with indirect coverage changes
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Fix agent flare crash caused by diagnose if the flare runs locally (regression)

What does this PR do?
Previous Move agent diagnose execution into agent process context #18792) PR had optimization to trigger internal Agent function calls because flare and diagnostics run in the same process. However, the code behind agent diagnose command assumes access to the running agent environment. However it is not true in cases when the agent is not running and flare executes locally.
Two fixes are applied. First, always presume that agent diagnose will be executed fully even if the agent is running (meaning to call a local port listener even when the diagnosis code is running inside the running Agent process). Second, add extra validation of internal state before "trusting" passed configuration.

Motivation

Additional Notes

Possible Drawbacks / Trade-offs

Describe how to test/QA your changes

Reviewer's Checklist

[ ] If known, an appropriate milestone has been selected; otherwise the Triage milestone is set.
[ ] Use the major_change label if your change either has a major impact on the code base, is impacting multiple teams or is changing important well-established internals of the Agent. This label will be use during QA to make sure each team pay extra attention to the changed behavior. For any customer facing change use a releasenote.
[ ] A release note has been added or the changelog/no-changelog label has been applied.
[ ] Changed code has automated tests for its functionality.
[ ] Adequate QA/testing plan information is provided if the qa/skip-qa label is not applied.
[ ] At least one team/.. label has been applied, indicating the team(s) that should QA this change.
[ ] If applicable, docs team has been notified or an issue has been opened on the documentation repo.
[ ] If applicable, the need-change/operator and need-change/helm labels have been applied.
[ ] If applicable, the k8s/<min-version> label, indicating the lowest Kubernetes version compatible with this feature.
[ ] If applicable, the config template has been updated.

I hit the flare --local crash when doing QA on rc1 and I can confirm that with this branch it works properly.


rid-catalog: Remove rhel minor versions

RHEL minor versions are binary compatible. The rhel major should be used for publishing.
cfr https://github.com/ellismg/core-setup/commit/b16864201a3a33f0e55f68c4539bbafcffc718ea
CC @omajid @ellismg

@tmds Thanks ... I can see that this is causing dev grief from the issue you linked. Let's merge and @mairaw can shoot me :boy: :gun: later if I'm being a bad dino today.


Search Fix

Inner Join :arrow_right: Left Join.

There are several other places in the search query where I use INNER JOIN. I don't want to just assume they all need to be changed, but I will do some more testing with searches with tags, user photos, etc. to make sure the query is performing as expected.


asset_id as full_path if contains "projects"

Minimal change to allow users to delete asset_ids that are in projects.  Similar control should probably be added to uploads as well.

Nice approach, but my worry here is that user might have a directory 'projects', e.g. users/user/projects - how would we know which to delete? Adding a switch is not nicest option, but explicit. Another option would be to remove relative paths completely. Simplifies the code, no ambiguity. It kills though backward compatibility and calls for a bit more typing .

After some thinking I decided to remove altogether the option for using 'relative' path. The code and command line interface is now more clear. What do you think?

Certainly - only allowing a full-path would be unambiguous.
That said, If you do want to keep the relative path an easy solution would be to use regex  with/^users\//  and /^projects\// - using the carrot (^) to ensure that projects  or users comes at the beginning and the escaped forward slash (\/) to ensure they are followed by a trailing forward slash (though you could probably drop this part).

Thanks, the regex would work just great, short of one situation:
projects/stuff/mycollection
Now, is it relative or absolute? Naturally 99% of users would mean here fully qualified path, but in theory there could be this one person that meant:
users/user/projects/stuff/mycollection
Similar thing with:
projects/mycollection
It's not a valid absolute path, but perfectly fine relative.
These are edge cases that are unlikely to happen, I agree. I am actually more concerned that in future there might be more name spaces added next to users and projects.
However, if there's a strong demand to have the short, relative path, then indeed your regex would be the best solution here.

I don't think there's a strong demand and absolute path idea is probably
best.  As long as were hashing out ideas though - you could protect against
projects/mycollection with regex as well - and for future name spaces you
could just put all-absolute-path-regex's in a config variable so that you
can easily add new paths.  I personally wouldn't worry about  projects/stuff/mycollection as a relative path.
But again I think your first instinct was right - sometimes simplest is
best :)
On Wed, Dec 7, 2016 at 1:32 PM, Lukasz Tracewski<EMAIL_ADDRESS>wrote:

Thanks, the regex would work just great, short of one situation:
projects/stuff/mycollection
Now, is it relative or absolute? Naturally 99% of users would mean here
fully qualified path, but in theory there could be this one person that
meant:
users/user/projects/stuff/mycollection
Similar thing with:
projects/mycollection
It's not a valid absolute path, but perfectly fine relative.
These are edge cases that are unlikely to happen, I agree. I am actually
more concerned that in future there might be more name spaces added next to
users and projects.
However, if there's a strong demand to have the short, relative path, then
indeed your regex would be the best solution here.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/tracek/gee_asset_manager/pull/18#issuecomment-265579785,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AARLLgxi5_d9PbrDkaWQTPnXTk31KPd5ks5rFyXfgaJpZM4LDFcT
.


Implementation for range with multiple look buged when pre-release are not present

These tests look wrong https://github.com/dtolnay/semver/blob/0080ed9bb561a9cb603f86aff5fe4ea8ce738814/tests/test_version_req.rs#L111-L115
According to the comment: https://github.com/dtolnay/semver/blob/0080ed9bb561a9cb603f86aff5fe4ea8ce738814/src/eval.rs#L14-L16
That should match, It's very strange no ? Specially:, https://github.com/dtolnay/semver/blob/0080ed9bb561a9cb603f86aff5fe4ea8ce738814/tests/test_version_req.rs#L150-L161
Is as expected. And: https://github.com/dtolnay/semver/blob/0080ed9bb561a9cb603f86aff5fe4ea8ce738814/tests/test_version_req.rs#L108-L109
Clearly show that 1.0.0-alpha < 1.0.0-beta < 1.0.0) Do I miss something ?

This matches the proto-spec in https://github.com/semver/semver/pull/584 and matches npm's semver implementation. The spec's treatment of <I.J is consistent with the behavior that I would want it to have.


color code not responding

I don't know what happened.
I linked my my css file to my html.
Then when I'm coding for a background color
it's not appearing on the preview.
And when I try redownloading the app
it says createprocess failed 740 and
something with the user account elevation??
Pls.. help..
Thank you

How do you preview your HTML page? How is VS Code involved?

I “open with chrome”
Then I try “open with live server”
I also download sublime text and it has the same issue.I don’t know how to
fix it.
On Mon, Apr 11, 2022 at 10:11 PM Martin Aeschlimann <
@.***> wrote:

How do you preview your HTML page? How is VS Code involved?
—
Reply to this email directly, view it on GitHub
https://github.com/microsoft/vscode/issues/147195#issuecomment-1095103511,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AYU34QJFBB4KL2VO5FFKXYLVEQXKJANCNFSM5TBY3RPQ
.
You are receiving this because you authored the thread.Message ID:
@.***>

So it is probably related to your HTML and CSS source files?
Could also be the Live Preview extension you a using, but if you see the same when opening with Chrome, I doubt that.
Sorry, this is the issue inbox for VS Code issues. Please try to get help about HTML and CSS in another forum.


Pants plugin required to run?

Have been trying to get the binary to work, but it appears pants needs a plugin "export-fastpass"
Howe do you get this plugin for pants?

Thank you for reporting! There are no installation instructions since this project is still evolving and it's unclear what the Pants requirements will be (Pants version, Pants plugin, etc). I recommend using the Pants IntelliJ plugin, which is the officially recommended IDE solution for Pants users https://plugins.jetbrains.com/plugin/7412-pants-support/versions

We just updated the readme and issue template with further details https://github.com/scalameta/fastpass/pull/24


Automatically install Python library dependencies

One thing we could try is moving the set of commands in the CI for installing AutoHOOT and cotengra into a file deps/build.jl, which Julia will run as a build step when the ITensorNetworkAD package gets installed. Similar to the discussions here:
https://discourse.julialang.org/t/pycall-pre-installing-a-python-package-required-by-a-julia-package/3316
https://gist.github.com/Luthaf/368a23981c8ec095c3eb

I guess we should actually add this as a build script in the AutoHOOT.jl package instead.


Combination Finisher affects MAP even without Panache

If Finisher was checked on when Panache was removed, MAP is still affected as though the character is using a finisher with the Combination Finisher feat. Because the Finisher checkbox is inaccessible when Panache is not on, the Panache effect needs to be re-added before it can be unchecked.

Fixed by https://github.com/foundryvtt/pf2e/pull/13058


gazetteer.File ignores termSeparator parameter

You can set the termSeparator parameter - and it looks like it gets passed in the config to uk.gov.dstl.baleen.resources.gazetteer.FileGazetteer. However, the init() method ignores it, so you always get the default comma separator.

This is now fixed. Thanks jbaker-nca!


test(connector-corda): fix flaky monitor transactions v4.8

Fix race condition that caused user session removal during request processing,
that caused test to fail occasionaly.
Correlated fixes / improvements:

Use paging when watching for vault changes.
This was necessary to perform long-running runs of a failing test.
Without it, start monitor of a state with 200+ changes was failing instantly.
I thought that paging was needed when there are 200+ changes in single observer push,
but it seems that it breaks when total number of changes are that large,
which makes it more critical to fix.
To prevent deadlock of cleanup thread I introduced thread pool to corda kotlin server.
Number of threads is configurable in environment variable.
Updated readme to show this option.

Closes: #1995
Signed-off-by: Michal Bajer<EMAIL_ADDRESS>

@petermetz @izuru0 Please review.
Also, please upload image ghcr.io/outsh/cactus-connector-corda-server:corda-dev-4 to hyperledger ghcr so I can update the reference in the test, and remove my name from cspell exclusion.

@petermetz @izuru0 Please review.

Also, please upload image ghcr.io/outsh/cactus-connector-corda-server:corda-dev-4 to hyperledger ghcr so I can update the reference in the test, and remove my name from cspell exclusion.
@outSH Thank you for fixing the flake, much appreciated! Where is the diff for the Dockerfile changes?

The changes in image B are not done through the dockerfile and therefore if next week we change something in the dockerfile and publish image C

@petermetz
The changes are replicable in dockerfile, but current image in hyperledger ghcr is older than one from this PR. I'm afraid I might be missing something important here - otherwise I'd just ask Izuru-san to upload this ;) - so I'll explain it in more detail:

I have a functional test monitor-transactions-v4.8.test.ts
This test used to pull cactus-connector-corda-server image from hyperledger ghcr, which contains kotlin connector (packages/cactus-plugin-ledger-connector-corda/src/main-server). This image is based on sources from couple commits ago.
I've done some changes in cactus-plugin-ledger-connector-corda/src/main-server source files.
If I run the test now, it would still use upstream ghcr  kotlin-server from couple commits ago, i.e. not including my changes.
For this reason, I've built a new image, using the same, unmodified dockerfile, to have an image that represents kotlin-server state from this commit (i.e. including my changes).
I've uploaded them to my ghcr, and updated the test, so that now the functional test uses the kotlin-server from this commit.
I need to have this image uploaded to hyperledger ghcr, which will contain the source version of kotlin-server from this commit.
I think normally this image would be updated with this PR sources after merge, but I want to have this earlier so we don't have to commit a test with reference to my GHCR.

@outSH Oh, okay, now I get it. Sorry, I thought you meant you were doing the changes with the docker commit command and that's why you wanted me to upload your custom image directly to ghcr. If we are talking about me rebuilding the image myself and then uploading it, then it's all good of course.

Oh, OK, now I see I'v asked you to upload my image from my ghcr.io/outsh, that was misleading, my bad. Good that we came to an understanding, eventually ;)
I've rebased, updated the image using in the test and fixed cspell ignore file. Should be good to go now.


Should prevent ROS_DISTRO from polluting build

As encountered in Anzu build, if a ROS 1 workspace / setup is sourced, then the build may fail, e.g.:
INFO: Repository ros2 instantiated at:
  {source}/WORKSPACE:56:25: in <toplevel>
  {source}/tools/workspace/default.bzl:282:24: in add_default_repositories
  {source}/tools/workspace/ros2/repository.bzl:53:26: in ros2_repository
Repository rule _ros2_repository_rule defined at:
  {source}/tools/workspace/ros2/repository.bzl:42:40: in <toplevel>
ERROR: An error occurred during the fetch of repository 'ros2':
   Traceback (most recent call last):
    File "{source}/tools/workspace/ros2/repository.bzl", line 27, column 25, in _ros2_repository_impl
        base_ros2_repository(
    File "{workspace}/external/bazel_ros2_rules/ros2/defs.bzl", line 87, column 29, in base_ros2_repository
        result = execute_or_fail(repo_ctx, cmd, quiet=True)
    File "{workspace}/external/bazel_ros2_rules/tools/execute.bzl", line 16, column 13, in execute_or_fail
        fail("Failed to setup @{} repository: {}".format(
Error in fail: Failed to setup @ros2 repository: './run.bash {workspace}/external/bazel_ros2_rules/ros2/generate_distro_file.py -s {workspace}/external/ros2/ros2-linux:ros2-linux -d distro_metadata.json ros2' exited with 1
--- captured stdout ---
ROS_DISTRO was set to 'noetic' before. Please make sure that the environment does not mix paths from different distributions.

--- captured stderr ---
Traceback (most recent call last):
  File "{workspace}/external/bazel_ros2_rules/ros2/generate_distro_file.py", line 105, in <module>
    main()
  File "{workspace}/external/bazel_ros2_rules/ros2/generate_distro_file.py", line 98, in main
    args = parse_arguments()
  File "{workspace}/external/bazel_ros2_rules/ros2/generate_distro_file.py", line 64, in parse_arguments
    args.distro = json.load(args.distro_file)
  File "/usr/lib/python3.8/json/__init__.py", line 293, in load
    return loads(fp.read(),
  File "/usr/lib/python3.8/json/__init__.py", line 357, in loads
    return _default_decoder.decode(s)
  File "/usr/lib/python3.8/json/decoder.py", line 337, in decode
    obj, end = self.raw_decode(s, idx=_w(s, 0).end())
  File "/usr/lib/python3.8/json/decoder.py", line 355, in raw_decode
    raise JSONDecodeError("Expecting value", s, err.value) from None
json.decoder.JSONDecodeError: Expecting value: line 1 column 1 (char 0)
ERROR: {source}/WORKSPACE:56:25: fetching _ros2_repository_rule rule //external:ros2: Traceback (most recent call last):
    File "{source}/tools/workspace/ros2/repository.bzl", line 27, column 25, in _ros2_repository_impl
        base_ros2_repository(
    File "{workspace}/external/bazel_ros2_rules/ros2/defs.bzl", line 87, column 29, in base_ros2_repository
        result = execute_or_fail(repo_ctx, cmd, quiet=True)
    File "{workspace}/external/bazel_ros2_rules/tools/execute.bzl", line 16, column 13, in execute_or_fail
        fail("Failed to setup @{} repository: {}".format(

\cc @IanTheEngineer

There must be some issue beyond the ROS1 environment being sourced. I completed builds with ROS1 sourced and unsourced without encountering this issue. I was only able to reproduce this when using a bazel environment built from a branch  that contained the bazel_ros2_rules, then switching to and attempting to build a branch that did not contain the bazel_ros2_rules external package (without a bazel clean first). bazel clean before building the branch after the switch resulted in a successful build. We first need to root cause this issue.

As discussed out-of-band with @EricCousineau-TRI, this is a lower priority issue.

I was only able to reproduce this when using a bazel environment built from a branch that [...]

Bazel attempts to be hermetic, but it's not always perfect. IMO, this isn't entirely surprising, and I think we can either place fixes or mitigations to address it.

We first need to root cause this issue.

I think the stacktrace is sufficient root cause, at least for this acute failure?

Ah, you mean "root cause why bazel clean didn't cause failure".
I think in this case, you forgot to source ROS 1 in the environment.
I am able to reproduce this error in Anzu by doing bazel clean --expunge --async, then doing bazel fetch @ros2//:rosidl.

I can reproduce this error in this repository under similar circumstances, using 685301d

Well, I keep getting notifications (should probably unwatch) so I'll chime in and hopefully save some of @sloretz's time :smile:

ROS_DISTRO was set to 'noetic' before. Please make sure that the environment does not mix paths from different distributions.

That message is likely coming from https://github.com/RobotLocomotion/drake-ros/blob/685301d189caef09b3599ddc8718d3043b6a0bbe/bazel_ros2_rules/ros2/resources/templates/run.bash.in#L14 The same message finds its way into the metadata file in the previous step, which causes all subsequent parsing to break.

One solution would be to nuke the environment in run.bash.in before all sourcing. Alternatively, you could use the -o/--output flag in all ros2bzl-powered scripts instead of piping their stdout to a file through Bazel (at the time I believed it was connected to with spurious repo invalidation, but maybe I was wrong).

Ah, I think I get - this is a warning that is sent to stdout (ewww). Using -o allows us to actually capture what we want, rather than letting this warning scribble into our stdout input.
Do we know what code is responsible for this? Can we file an issue / PR to make that go into stderr instead?
(or is it an issue w/ our code?)

Do we know what code is responsible for this? Can we file an issue / PR to make that go into stderr instead?
(or is it an issue w/ our code?)

I think it's from here:
https://github.com/ros/ros_environment/blob/64678bac5fd733146ba6d07a38d1db733398898e/env-hooks/0.ros_distro_check.sh.in#L3-L5

Can we file an issue / PR to make that go into stderr instead?

Opened PR: https://github.com/ros/ros_environment/pull/30

Thanks!

I think this is resolved by #54; closing.


docs: make subnav sticky

Closes #1060
What I did

make the subnav's container sticky, calculating the height of the whitespace and header

Testing Instructions

check out an elements page in the dp

Notes to Reviewers

Looks good on screens < 1400px, though I think we might need a different stylesheet for >= 1400px

Looks like when the screen is >= 1400px, the .header class gets a padding left, top, and right.  Also the padding on the h1 element changes because the font-size changes to 48px.

Thanks @brianferry. PTAL. should fix large screens. mobile still has some issues, but I think we can merge anyways and go back after this release to fix the styles in a more fundamental way (i.e. use tokens)
that being said if you wanna tinker, go right ahead, just commit here and merge when it looks good to you

should fix large screens

Nice work, this looks great!

@bennypowers Looks good to me.


Use ipTree to determine the global IP

Use ipTree to determine the global IP

Looks good, merging.


Add transform to fuse activations into TFL pooling ops

TFLite average and max pool operations support fused activation functions similar to the convolution ops. This PR adds a MLIR transform to the TFLite converter to fuse activations functions into preceding pooling operations.

@joker-eph @abattery do you mind giving this PR a quick review?

@abattery Thanks for approving the PR!
Would you also be able to take a look at #41790 which is a crutial bug fix for #39572 and touches similar files?


docker build is throwing error

Problem encountered on https://dotnet.microsoft.com/en-us/learn/aspnet/microservice-tutorial/docker-image
Operating System: windows
Error Stack:
[+] Building 1.0s (7/14)
=> [internal] load build definition from Dockerfile                                                                                0.0s
=> => transferring dockerfile: 32B                                                                                0.0s
=> [internal] load .dockerignore                                                                                0.0s
=> => transferring context: 34B                                                                                0.0s
=> [internal] load metadata for mcr.microsoft.com/dotnet/aspnet:6.0                                                                                0.5s
=> [internal] load metadata for mcr.microsoft.com/dotnet/sdk:6.0                                                                                0.5s
=> CANCELED [build 1/6] FROM mcr.microsoft.com/dotnet/sdk:6.0@sha256:e656fe19bc303356acdbfec4695cb2bdd42c7a30677a8775a15eed7fd88edf60                 0.1s
=> => resolve mcr.microsoft.com/dotnet/sdk:6.0@sha256:e656fe19bc303356acdbfec4695cb2bdd42c7a30677a8775a15eed7fd88edf60                                0.0s
=> => sha256:e656fe19bc303356acdbfec4695cb2bdd42c7a30677a8775a15eed7fd88edf60 2.17kB / 2.17kB                                                         0.0s
=> => sha256:148a3465a035ddc2e0ac2eebcd5f5cb3db715843d784d1b303d1464cd978a391 2.01kB / 2.01kB                                                         0.0s
=> ERROR [internal] load build context                                                                                0.1s
=> => transferring context:                                                                                0.0s
=> CANCELED [stage-1 1/3] FROM mcr.microsoft.com/dotnet/aspnet:6.0@sha256:bb2428448b9bbc46171905a7a3e8dbdb696a5583e8ba10307f15be4fea8483e9            0.3s
=> => resolve mcr.microsoft.com/dotnet/aspnet:6.0@sha256:bb2428448b9bbc46171905a7a3e8dbdb696a5583e8ba10307f15be4fea8483e9                             0.0s
=> => sha256:8d32e18b77a4db7f10ec4985cc85c1e385dc6abd16f9573a8c2bc268cad4aab9 3.38kB / 3.38kB                                                         0.0s
=> => sha256:bb2428448b9bbc46171905a7a3e8dbdb696a5583e8ba10307f15be4fea8483e9 2.17kB / 2.17kB                                                         0.0s
=> => sha256:5b4a077a17943113fee94818046e6f9839e11ec692481bf122ffacb849cf67de 1.37kB / 1.37kB                                                         0.0s

[internal] load build context:

rpc error: code = Internal desc = stream terminated by RST_STREAM with error code: PROTOCOL_ERROR

It's going well with .Net5. Having issues with .Net6 though.

It looks like something else in your Docker build is failing, which instructs other parts of the build pipeline to be cancelled. The log doesn't tell me anything about what is going on.
Probably best to flag this as NMI.

stream terminated by RST_STREAM with error code: PROTOCOL_ERROR
This sounds like something weird went wrong in networking.  Is your error still happening?  Are there any other errors, like compile errors before the docker phase?

stream terminated by RST_STREAM with error code: PROTOCOL_ERROR
This sounds like something weird went wrong in networking. Is your error still happening? Are there any other errors, like compile errors before the docker phase?

Yes, this is still happening. And no, there's no other error before the docker phase. It works perfectly fine if I change it to Net5 instead of Net6.

@MichaelSimons Is this sort of docker/build failure something you happen to know off the top of your head, or can otherwise assist with?

@MichaelSimons Is this sort of docker/build failure something you happen to know off the top of your head, or can otherwise assist with?

Sorry this doesn't look familiar to me.  The error makes me suspect docker versus .NET.  @AshrCode - are you using the latest docker version?

@MichaelSimons Is this sort of docker/build failure something you happen to know off the top of your head, or can otherwise assist with?

Sorry this doesn't look familiar to me. The error makes me suspect docker versus .NET. @AshrCode - are you using the latest docker version?

@MichaelSimons , Yes I am using the latest version of docker.


feat: add German language support

Closes #240

Did all the changes - can you please check again @fmaclen?

:tada: This PR is included in version 0.23.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Pre-process attributes with a class-isolated attribute cache

WIP to close #467
I’m not 100% on the naming at this point.

@joelmoss if you want to try this out, you can define process_attributes? to return true and process_attributes(**attributes) to return a Hash of processed attributes. The cache should be isolated to the class.

One thing I don't like about this solution is it binds pre-processing attributes with a class-isolated attribute cache, and you may want to pre-process attributes without a class-isolated attribute cache if your pre-processing is global for all view classes.

ok, so this all works perfectly as far as I'm concerned 👌

One thing I don't like about this solution is it binds pre-processing attributes with a class-isolated attribute cache, and you may want to pre-process attributes without a class-isolated attribute cache if your pre-processing is global for all view classes.

I’ve been thinking more about this and I do think it makes sense to switch to class-isolated attribute cache keys when process_attributes is defined. However, it may be appropriate to provide a configuration setting for a global attribute processor that wouldn't affect the cache keys.
I could see something like this in the future:
Phlex.configure do |c|
  c.process_attributes do |attributes|
    ...
  end
end

ok, so I spoke too soon :( The shared attribute cache is still biting me.
The shared cache is only a problem for me, because I am also using process_attributes to call code that should be called every time a view is rendered, but only if specific attribute conditions are met.
But because process_attributes is only called the once, no matter how many instances of the view are created and rendered, my code is only called the once.
Yes there are the [before|after]_template hooks where I could call my code. But that won't work because I still only want to call my code if the my attributes conditions are met.
So this new hook is great for processing attributes, but not for triggering actions based on attributes used, as the hook only gets called the once no matter how many view instances are rendered.

I can't think of a solution to this without losing all the benefits of caching which is a ~52% performance cost. Can you help me understand what you need this for? There might be another way.

so I think I found a solution that is working... so far 😁

I'm still using process_attributes. I have to, because that is the only way to know about the attributes.
When attribute conditions are met, I append to a class attribute (defined as a Set using attr_accessor on the view class, not instance). I then call my code from the after_template hook when the class attribute is populated.
Something like this...
class_methods do
  attr_accessor :side_load_cache
end

def before_template
  self.class.side_load_cache ||= Set.new
  super
end

def process_attributes(**attributes)
  if attributes.key?(:class) && attributes[:class].starts_with?('@')
    self.class.side_load_cache << "path/to/#{attributes[:class]}"
  end

  attributes
end

def after_template
  super

  self.class.side_load_cache&.each do |path|
    Proscenium::SideLoad.append! path
  end
end

So far it works, and I avoid duplicates by using Set.
How would a global hook be better?

Oh that's great, that's a nice solution. A global hook is just simpler, faster and easier to maintain since it can use the global attributes cache. If there are use-cases for process_attributes, I’m happy to keep it.
You may want to use Concurrent::Set from the concurrent-ruby project if you want this to be thread-safe. concurrent-ruby is already a dependency of Phlex, so you may as well.

Global hook sounds good, but we would lose its context, such as the view instance.

If the view instance is helpful to someone, I’m happy to keep process_attributes as it is.


Support migrations with existing pgmigrations table

Currently you can only start from scratch which is only useful for testing scenarios.

https://github.com/PDMLab/massive-migrate/commit/230ca3ebdbb56497fd13e13c2bace2070de92ab6 verifies that it works basically.
Next: verify that it returns an error if same migration is run multiple times.


Laravel 6 Support

Hello,
Do you plan to upgrade to Laravel 6 ?
Thx for your work

I'm no longer actively working with Laravel. But PR's and maintainers for this project would be welcomed

Hi @thomaswelton I have added pull #33. Can we get this merged please? Thanks.
HHVM was breaking on the other pull request that was opened by @robbens. It also looks to be common practice to future proof packages for Laravel 7, in advance of release.

@wells Merged and tagged a 1.2.0 release.


:+1: show constraint error warning

Fixes #401, closes #403

Summary by CodeRabbit

New Features

Enhanced error handling and logging in the Denops server, providing users with actionable feedback for specific errors.
New test cases added to ensure the server behaves correctly when encountering constraint errors.

Bug Fixes

Added checks to prevent the server from restarting after a stop command under error conditions, improving reliability.

Chores

Updated configuration to exclude specific test data paths, streamlining operations.

Force pushed remove let g:denops#debug = 1, which was used to debug the test implementation but is no longer needed.

I thought it would be better to set g:denops#disabled = 1 on error.

Nice. But with other PR.


Realtime swap updates

Refactor realtime updates to support multiple types of operation
Refactor substrate extrinsic handlers to use unified interface
Implemented handler for realtime assetConversion's swaps, both native and custom fee tokens are supported
Fixed - disabled transfers on Westend and Rococo due to missing transfer function
#869347nvf
#86934amx4

Task linked: CU-869347nvf Android. Realtime Swap Updates

Task linked: CU-86934amx4 Android. Disabled transfers on Westend and Rococo


Fix broken TDML append, copy and execute functionality

closes #1013
Fixed TDML functionality. Now copyTDML, Append and Execute TDML works as expected.
Following functionality still need re-visit and further enhancement issues needs to be created.

in copy, it is currently being hardcoded to getTmpTDMLFile location. If a user is specifying a custom value in the launch config, we won't be able to pick that up right now (append is probably working the same way) (#1027 )
in copy, there's no error message when it doesn't work (#1027)
in append, if you are trying to add a duplicate, there is no error message letting the user know that's why nothing happened (#1028 )
in execute, if the user takes longer than the timeout to enter name/description, the server will disconnect. #596 should fix this issue. (#1029 )
in generate, TDML name, description and infoset path is defaults to string "undefined" (#1024 )
Execute functionality should only require TDML test case name only to execute, currently it requires name and description. #596 should fix this issue. (#1030 )
append is listed as a valid action in the launch wizard - Append is not a server action and should not be available as an option in launch config. (#1031 )
TDML Copy function creates the wrong path in some Windows 10 environments (#1034)
Fix TDML Copy, Execute and Append functionality in MacOS Platform (#1035)

Steps to test TDML copy, execute, append
Configure launch.json to generate a TDML file.

Run the debug extension, choose a dfdl schema and data file.  After the schema opens make sure the language mode is DFDL.

Press the continue button to produce the infoset.

When the infoset generates, a temporary TDML schema will generate.

Close any open window except the dfdl schema window.  Click “Copy TDML File” in the dropdown at the top tight of the editor window.

Enter a name for the TDML file, click “Save TDML File in the lower right corner of the popup.

Close the dfdl schema in the editor window.  Click on the explore tab in the top left corner to display the file view. Click on the save TDML file.

After the TDML file opens click the menu in the top right corner and select “Execute TDML”

Quickly select a test case a press enter.  Then quickly accept the description and press enter.

The previously selected dfdl schema will open and the previously selected data file will be used and empty infoset will also open.

To append to the existing TDML file, open the test file and click the button in the upper right corner to open in a text editor.

Change the test case name and save the file.

Select append from the TDML dropdown menu at the upper right.

The original default test case will be appended to saved TDML file with the new test case.

M1 Macbook Testing
Testing this on my M1 macbook resulted in no generatedTDML file in any temporary directory.
Pop!OS (Ubuntu 22.04) Testing
The generated file is created successfully and I can run the Copy, Append, and Execute operations. Though when executing there is an uninformative info dialog message that pops up that only contains the string of the infoset file name:

Didn't want to merge this yet until @stricklandrbls approves.

M1 Macbook Pro Test Failure
@hdalsania @scholarsmate @shanedell - I just tested this again on my M1 Macbook Pro and I still cannot Copy / Append because the generated tmp file is not created.
The Copy TDML File button exists but will not save the .tdml file anywhere once the data debugger is finished and I attempt to save.
VSCode Debug Output
simulatorDirPath: undefined
deviceFilePath: undefined
simulatorDirPath: undefined
deviceFilePath: undefined
detected java homes: [{"path":"/Library/Java/JavaVirtualMachines/jdk-<IP_ADDRESS>.jdk/Contents/Home","version":"11.0.16","security":0,"isJDK":true,"is64Bit":true,"executables":{"java":"/Library/Java/JavaVirtualMachines/jdk-<IP_ADDRESS>.jdk/Contents/Home/bin/java","javac":"/Library/Java/JavaVirtualMachines/jdk-<IP_ADDRESS>.jdk/Contents/Home/bin/javac","javap":"/Library/Java/JavaVirtualMachines/jdk-<IP_ADDRESS>.jdk/Contents/Home/bin/javap"}},{"path":"/Library/Java/JavaVirtualMachines/microsoft-11.jdk/Contents/Home","version":"11.0.16","security":0,"isJDK":true,"is64Bit":true,"executables":{"java":"/Library/Java/JavaVirtualMachines/microsoft-11.jdk/Contents/Home/bin/java","javac":"/Library/Java/JavaVirtualMachines/microsoft-11.jdk/Contents/Home/bin/javac","javap":"/Library/Java/JavaVirtualMachines/microsoft-11.jdk/Contents/Home/bin/javap"}},{"path":"/Library/Java/JavaVirtualMachines/temurin-8.jdk/Contents/Home","version":"1.8.0","security":302,"isJDK":true,"is64Bit":true,"executables":{"java":"/Library/Java/JavaVirtualMachines/temurin-8.jdk/Contents/Home/bin/java","javac":"/Library/Java/JavaVirtualMachines/temurin-8.jdk/Contents/Home/bin/javac","javap":"/Library/Java/JavaVirtualMachines/temurin-8.jdk/Contents/Home/bin/javap"}}]
choosing java home at /Library/Java/JavaVirtualMachines/jdk-<IP_ADDRESS>.jdk/Contents/Home, version 11.0.16, is at least JDK 17: false
No test suite found in source XML buffer

The output line: No test suite found in source XML buffer gets outputted every time I attempt to save the generate .tdml file via Copy TDML File.

@hdalsania @stricklandrbls I also get a No test suite found in source XML buffer when I try to Copy TDML File or when I try to Append TDML.

@hdalsania @scholarsmate @shanedell I did a little digging and running the extension in debug mode w/ breakpoints. It looks like this is coming from this scope of code:
https://github.com/apache/daffodil-vscode/blob/5bd426e1dbdf25b40193ba0a2eb17dbc9d861ba1/src/tdmlEditor/utilities/tdmlXmlUtils.ts#L277-L295
I think it has something to do with this function being async but not returning a Promise because this block of code throws with all of the scoped variables being undefined or uninitialized.

We created a bug issue #1035 to address MacOS platform and changed the scope of this PR to be limited to Windows/Linux platform.


Python: function to remove requirements.txt/setup.py constraints

Issue description
Python developers often pin their dependencies to exact versions. This is bad practice and most of the times absolutely not necessary either.
Typically we use substituteInPlace to remove the dependency requirement. It would be useful to have a function that would strip such exact requirements.
I've also opened an issue at setuptools with a request for a flag that ignores version pinning.
https://github.com/pypa/setuptools/issues/894

Example why we should have this: https://github.com/NixOS/nixpkgs/pull/21321/files

I can see the need in regards to the setup.py file. I've myself sometimes patched away a specific version pinning in a setup.py.
How do you see the use related to a requirements.txt file? Wouldn't this typically be used in a scenario to generate nix code e.g. via pip2nix or pypi2nix?

Actually, many developers wrongly load the requirements.txt file and add the items to install_requires. Therefore, a method to just filter the install_requires and setup_requires values should be sufficient.

Are there any updates to this issue, please?

Reading https://github.com/pypa/setuptools/issues/894 again. The buildPhase will produce a wheel, and that will contain the constraints (in METADATA, the Requires-Dist rows). We can thus fix it at our side by patching the wheel. Would also need to fix then the hash for METADATA in RECORD. The are tools to manipulate wheels. This can be implemented in a hook.

continuing my line of though with sed patching, I came with following hook:
function pythonJailbreak {
    local file="$1"; shift

    for dep in $@; do
        if [[ "$file" =~ requirements*.txt ]]; then
            sed -r -i "s/$dep[><=].*?/'$dep'/g" $file
        else
            sed -r -i "s/['\"]$dep[><=].*?['\"]/'$dep'/g" $file
        fi
    done
}

function toJailbreak {
    local file="$1"
    if [[ "$file" =~ requirements* ]]; then
        sed -rn "s/([a-zA-Z0-9+-]+)[><=].*/\1/p" $file
    else
        sed -rn "s/.*['\"]([a-zA-Z0-9_-]+)[><=].*['\"].*/\1/p" $file
    fi
}

function jailbreakHook {
    if [[ "$doJailbreak" != "false" ]]; then
        if [ -f setup.py ]; then
            pythonJailbreak setup.py $(toJailbreak setup.py)
        fi
        for reqtxt in $(ls -d requirements*.txt); do
            pythonJailbreak "$reqtxt" $(toJailbreak $reqtxt)
        done
    fi
}

It jailbreaks ~95% of dependencies for apache-airflow, so is not "production-grade", but package is built and tested fine.

Added by https://github.com/NixOS/nixpkgs/pull/170813


Adding Mirascope Third Party Integration Docs Page

Added a new third_party/ directory in the docs/integrations/ directory for all future third party integrations.
Added the mirascope.md docs page describing the integration to the third_party/ directory.
Tried to match existing docs pages wrt. internal links.
NOTE: I have purposefully excluded an image for the extraction section since Logfire UI does not yet have Function Calling supported (see #54). I will add that screenshot once that feature is released.

I don't think I have rights to push to this branch. Can you apply this patch?
diff --git a/docs/integrations/third_party/mirascope.md b/docs/integrations/third_party/mirascope.md
index a0e07f89..91ce9b45 100644
--- a/docs/integrations/third_party/mirascope.md
+++ b/docs/integrations/third_party/mirascope.md
@@ -1,5 +1,3 @@
-# Mirascope
-
 [Mirascope](https://github.com/Mirascope/mirascope) is an intuitive approach to building AI-powered applications using LLMs. Their library is the first third party integration with Logfire to make observability and monitoring for LLMs easy and seamless.

 You can enable it using their [`@with_logire`](https://docs.mirascope.io/latest/integrations/logfire/#how-to-use-logfire-with-mirascope) decorator, which will work with all of the [model providers that they support](https://docs.mirascope.io/latest/concepts/supported_llm_providers/) (e.g. OpenAI, Anthropic, Groq, and more).
diff --git a/docs/plugins/main.py b/docs/plugins/main.py
index b184ba44..57cf3c9b 100644
--- a/docs/plugins/main.py
+++ b/docs/plugins/main.py
@@ -20,8 +20,8 @@ def on_page_markdown(markdown: str, page: Page, config: Config, files: Files) ->
     markdown = build_environment_variables_table(markdown, page)
     markdown = logfire_print_help(markdown, page)
     markdown = install_logfire(markdown, page)
-    if page.file.src_uri == 'guides/onboarding_checklist/06_add_metrics.md':
-        check_documented_system_metrics(markdown, page)
+    markdown = check_documented_system_metrics(markdown, page)
+    markdown = warning_on_third_party(markdown, page)
     return markdown

@@ -36,6 +36,9 @@ def check_documented_system_metrics(markdown: str, page: Page) -> str:

     This function checks that all the metrics in `DEFAULT_CONFIG` are documented.
     """
+    if page.file.src_uri != 'guides/onboarding_checklist/06_add_metrics.md':
+        return markdown
+
     metrics_documented: set[str] = set()
     for line in markdown.splitlines():
         match = re.search(r'\* `(.*)`: ', line)
@@ -134,3 +137,17 @@ def install_logfire(markdown: str, page: Page) -> str:
     ```
 """
     return re.sub(r'{{ *install_logfire\(.*\) *}}', instructions, markdown)
+
+
+def warning_on_third_party(markdown: str, page: Page) -> str:
+    if not page.file.src_uri.startswith('integrations/third_party/'):
+        return markdown
+
+    note = """
+!!! note "Third-party integrations"
+    Third-party integrations are not officially supported by **Logfire**.
+
+    They are maintained by the community and may not be as reliable as the integrations developed by **Logfire**.
+"""
+
+    return note + markdown

I applied the patch and addressed the comments (but have left things unresolved for you to resolve).
@Kludex please let me know if I missed anything.

@Kludex I just updated the docs to include a screenshot of extraction now that function calling shows up nice and pretty in the UI :)

I don't know exactly what the problem is with this branch - I'm not sure if it's because it was created on an org's main branch, or I can't edit the branch, but I've copied the commits here to https://github.com/pydantic/logfire/pull/133.
I'll merge that PR, thanks @willbakst ! :)

Ah yeah not sure what the issue was, maybe cause it's a fork? Thanks for taking care of this and merging in!


Can't find React context during hydration in dev

Reproduction
This was brought up by Brian on Twitter
https://stackblitz.com/edit/remix-run-remix-egqqa7?file=app%2Froutes%2F_index%2Froute.tsx,app%2Froutes%2F_index%2Fstuff.tsx
System Info
StackBlitz

Used Package Manager
npm
Expected Behavior
No error should be thrown (only happens in hydration)
Actual Behavior

It looks simillar: https://github.com/remix-run/remix/issues/10113


[ERROR] Failed to install crash handler. No exceptions will be reported!

Please checkout our documentation for guides, references and tutorials.
If you have questions about your integration please contact us at<EMAIL_ADDRESS>Alternatively, view additional options at support.md.
Dears, good morning!
I have this error in the debug:
[Bugsnag] [ERROR] Failed to install crash handler. No exceptions will be reported!
Can you give me a light on what I'm missing?
I`m using objective c.
Thanks so much.

Hi @vmagnani,
The error message you're seeing is a bit misleading - it just means that due to the current configuration / environment there wasn't a crash handler to install. We have actually removed this message in https://github.com/bugsnag/bugsnag-cocoa/pull/1091 (released in v6.9.2)
Closing this issue as we believe it's already been fixed, but if you're still experiencing this issue on 6.9.2 or greater do let us know and we'd be happy to investigate further.

@yousif-bugsnag thanks for the answer, but I forgot to mention that I'm using version 6.9.6.
I'm a little more relieved to know that it's not a problem.

we are still facing this issue on v6.10.2, pls help take a look!

@pandaApe - could you write in to<EMAIL_ADDRESS>with full details of how you're configuring Bugsnag in code and what the issue is you're seeing? i.e. is it just that you're seeing that warning or are crashes not being reported?


Fix tutorial & add tutorial testing

Codecov Report

Merging #1095 (b89eb79) into master (2efd47f) will not change coverage.
The diff coverage is n/a.

Additional details and impacted files
@@            Coverage Diff            @@
##            master     #1095   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files           36        36
  Lines         2450      2450
=========================================
  Hits          2450      2450


Add ChangeRegion to readme

Closes #17

:tada: This PR is included in version 1.2.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


updates available platforms and 1080p support

Please follow the guide below

You will be asked some questions, please read them carefully and answer honestly
Put an x into all the boxes [ ] relevant to your pull request (like that [x])
Use Preview tab to see how your pull request will actually look like

Before submitting a pull request make sure you have:

[x] At least skimmed through adding new extractor tutorial and youtube-dl coding conventions sections
[x] Searched the bugtracker for similar pull requests

In order to be accepted and merged into youtube-dl each piece of code must be in public domain or released under Unlicense. Check one of the following options:

[x] I am the original author of this code and I am willing to release it under Unlicense
[ ] I am not the original author of this code but it is in public domain or released under Unlicense (provide reliable evidence)

What is the purpose of your pull request?

[ ] Bug fix
[x] Improvement
[ ] New extractor
[x] New feature

Description of your pull request and other information
Resending the pull request for adding 1080p support for Hotstar as the format is now available on their CDN and this PR enables support for more than one User Agents(Formats & Platforms).

i don't think that this is needed all formats are already extracted:
youtube-dl -F http://www.hotstar.com/on-air-with-aib--english-1000076273
[HotStar]<PHONE_NUMBER>: Downloading JSON metadata
[HotStar]<PHONE_NUMBER>: Downloading JIO JSON metadata
[HotStar]<PHONE_NUMBER>: Downloading m3u8 information
[info] Available formats for<PHONE_NUMBER>:
format code  extension  resolution note
hls-64       mp4        250x140      64k , avc1.66.30, mp4a.40.2
hls-120      mp4        320x180     120k , avc1.66.30, mp4a.40.2
hls-240      mp4        320x180     240k , avc1.66.30, mp4a.40.2
hls-460      mp4        416x234     460k , avc1.66.30, mp4a.40.2
hls-860      mp4        640x360     860k , avc1.66.30, mp4a.40.2
hls-1360     mp4        720x404    1360k , avc1.66.30, mp4a.40.2
hls-2060     mp4        1280x720   2060k , avc1.77.30, mp4a.40.2
hls-3059     mp4        1600x900   3059k , avc1.77.30, mp4a.40.2
hls-4559     mp4        1920x1080  4559k , avc1.77.30, mp4a.40.2 (best)

also while i was testing other values for channel query param, some of them will serve a video saying that the video is not available on the platform.

Not Required. Closing.


Initial pull request - Nightscout RTD.

This is fantastic. Huge kudos, @andrew-warrington . I +1 this for merging and getting the live RTD link up, so we can get some volunteers to help finish the last few WIP pages.

Updates:

Added a front page linking to OpenAPS and AndroidAPS documentation as well (since this intended to be a main page for #WeAreNotWaiting documentation)
Put all Nightscout docs in an EN subfolder to allow for translations with Crowdin
Moved all images to their own subfolder and updated links

Apologies for the 7 billion commits. It is not overly simple to work with TOC in RTD, so I needed to iterate the build of the docs often to learn and get that right.


tour: Short definition of exported name would be helpful.

Context: https://tour.golang.org/basics/3
This page tells that exported name must start with upper character, but it does not say what the "exported name" means.   Also, "Exported identifiers" maybe more aligned with other material.
This made me think "ok... I understand what it looks like, but what does this exported name do?".
One sentence like "exported names can be accessed from another package, while unexported names remains local".

I'll have a go at clarifying this.


Migrate to Swift 3

How would you like to approach this, using a branch, or simply going ahead in master?
Expecting new shiny tools tomorrow due to WWDC...
Personally, I wouldn't mind migrating to S3 in master as things are still a bit in flux (and preferably looking at support for the package manager), but doing a swift3 branch is also more than completely fine of course. Thoughts?
Also related to #55.

Totally agree on that. I will just tag the latest Swift 2.2 version.
Only if someone is using FlatBuffersSwift in production and don't want to migrate to Swift 3.
In this case people should approach me directly. It is always possible to make a branch from a Tag.

Ok, cool. Do you want help with this one? I can create a PR and you can tag the 2.2 rev. before merging in that case.

Is S3 already stable? I would rather wait a bit. Other wise the master will be in constant refactoring mode.

It's probably not stable enough yet - looking at https://swift.org/blog/swift-3-0-release-process/ - when swift-3.0-branch is created, it should be good enough to work against with need to refactor as the feature set would be frozen. Let's see what the timeline becomes (not communicated yet).
Another alternative would be to track the swift-3.0-preview--branch:es and refactor (if needed...) every 6 weeks or so, each such refactor would likely be a smaller chunk of work.

Hm in this case a separate branch makes more sense I guess, because people who want to try out FlatBuffersSwift can do it from master. Advanced users who are already playing around with S3 will be able to find out about the branch :)

will you support Swift 3 soon?

Yes, now as it is officially released I will put as high priority. I reserved couple of days for this project next week, hopefully it will be enough to have the first swift 3 version.
I am optimistic 😬

Any update on the progress of the Swift 3 upgrade?

I am also eagerly awaiting the Swift3 upgrade:)

Ok was a bit to optimistic before. Will need a couple more weeks. Sorry.

No problem, thanks for your work.  I'm building a game with Java servers that use flatbuffers and really needed your swift library on the client side, so appreciate it very much.
Ken
Sent from my iPhone

On Sep 29, 2016, at 9:49 AM, Maxim Zaks<EMAIL_ADDRESS>wrote:
Ok was a bit to optimistic before. Will need a couple more weeks. Sorry.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or mute the thread.

@mzaks why not draft an official swift3 fork - people can pick up from you and upgrade code.

FlatBuffersSwift ported to Swift3. 🎉
I also cleaned it up a bit.
In this repository I keep only the lib and the UnitTests.
Performance tests now have there own home.
https://github.com/mzaks/FlatBuffersSwiftPerformanceTest
Btw. with Swift3 we are a bit slower again. Everyone is welcome to make make pull requests ;)
Also I don't use Swift3 in production yet.
If you generate your schemas and there are some problems, please let me know.

Nice, good job! Will be some time before I will be able to give feedback due to other commitments, but looking forward to trying it out, thanks!


Refer to nimbleSMC JSS paper in User Manual Ch. 8

It might be nice to refer directly to “Sequential Monte Carlo Methods in the nimble and nimbleSMC R Packages.”. Currently in Ch. 8 we say the SMC methods are in nimbleSMC and leave a reader to find their way to the paper from nimbleSMC. However Ch. 8 then proceeds to walk through examples, which might throw people off the scent of looking at nimbleSMC documentation and finding the JSS paper, which could really be more helpful.

Doing now on devel.


txn: use region pessimsitic lock rollback to speed up cleanup

Ref: https://github.com/tikv/tikv/issues/16158
Use region pessimistic rollback to speed up expired pessimistic lock cleanup, it would take effect for:

write-write conflict pessimistic lock resolve, for example when a pessimistic lock is encountered by a prewrite request or another pessimistic request.

It would not impact:

GC lock resolving as it should ensure no expired locks are left.
Other lock resoving like snapshot.scan.

To be discussed, if it's necesary to removing for_update_ts check.
The benefit of removing  for_update_ts check in tikv:
All the expired locks belonging to the same transction could be cleaned up at one time.
The side effect of removing  for_update_ts check in tikv:
When the primary lock key is changed (e.g the first statement fails and the related pessimitic locks are left), it's possble the normal pessimistic locks are cleaned up mistakenly if https://github.com/tikv/tikv/pull/16185#discussion_r1436859199 for_update_ts check is removed.

Why the integration test cases could not pass on the CI..

@zyguan @MyonKeminta
At present, the CI integration test fails when the region resolve is enabled for the prewrite and pessimistic commands during lock resolving. Strangely, the test passes on my local machine. While the feature is not currently enabled, we might proceed with the PR review.
I plan to investigate the integration test failure on CI and, if needed, implement the region resolve on the mockstore to ensure its success.


Libpod container creation logic

The core logic for container creation. @baude will be integrating this with CLI work to produce a working version of 'kpod create'.

Alright, this should be ready for review. Going to wait for travis to return to trigger the tests

@rhatdan PTAL

@nalind @mitrmac PTAL

We want to get this in, so ignore the TODO statements.  We can get back to those after it is merged.

LGTM.
I've been using this code for a while now too.
One global comment would be to add logrus.debug statements for significant actions (start, create, storage create, and so on).  Great work @mheon

Pushed two commits. One refactors the way pods are handled in container creation, another addresses comments from review.

/test all

Tests are green, but I'd like to see a few more reviews before we merge

@mrunalp @runcom Could one of you guys take a look.

@rhatdan Doubt we're getting any more reviews any time soon. Let's get this merged so @baude can get his frontend code out for review.

LGTM

This code is a work in progress.  But I will merge so we can move forward. When we start to move CRI-O to using it, we can get a better review.


Add the ability to save a full docsend which is not downloadable

Hello,
thanks for the great work you've done here.
Everything is in the title, I have docsend share which is not downloadable and I would like to save it

Addressed in #3

I think you misunderstood, I'm not talking about a full document but about a full share tree

Can you reopen this issue please ?

Sorry, not sure I understand the "Full share tree". Do you mean a docsend that contains multiple documents, like a Space?

Yes, exactly


Yahoo is having trouble opening accented folders or files (invalid path)

Yahoo is having trouble opening accented folders or files. It's a problem with Yahoo servers.
But when yahoo-group-archiver tries to download one of those files/folders it crashes.
"httpStatus":200,"errorMessage":"Invalid path","errorCode":1309
Traceback (most recent call last):
File "./yahoo.py", line 195, in
archive_files(yga)
File "./yahoo.py", line 96, in archive_files
archive_files(yga, subdir=pathURI)
File "./yahoo.py", line 73, in archive_files
file_json = yga.files(sfpath=subdir)
File "/root/yahoo-group-archiver/yahoogroupsapi.py", line 83, in get_json
raise e
KeyError: 'ygData'

httpStatus":200,"errorMessage":"Invalid path","errorCode":1309
Same error, different reason.  In my case, clicking on the folder in Yahoo groups gives an invalid path error, as well.  So the path is broke at the source.  I would like either default behavior to skip and go to the next, or a command line tag to enable skipping of invalid paths.

Could you give an example of a group (if public) that shows this bug so that I may test a fix?

This public group has several album names with characters (# " ;) not allowed in file or path names.  The download keeps crashing the yahoo.py script at the 3rd album which contains a " in the album name.  Another group download crashed at the | character but that group is private.
BugHLG

Another group crashes the download at a album name like this: Untitled album 01.03.2015, 13:49:35
Of course, the lack of being able to restart at some place in the middle makes it difficult to get all the photos when a group has 3600 photos!  Need some sort of ignore errors switch with a skipped files/photos log.

My group is private so I can't give you a link, but I have an error when trying to switch to a directory with an ö in the directory name.

Fetching directory 'V Gröbl' (3/17)

Looking into it a bit more, it looks like it's encoding the name a bit strange?  If I browse to the page, the string is:
/files/Membership%20/2018%20New%20Member%20Applications/V%20Gr%C3%B6bl/
but the script is trying to navigate to:
/files?sfpath=%2FMembership+%2F2018+New+Member+Applications%2FV+Gr%C3%83%C2%B6bl%2F
Some of it is different because it's an API call vs the web interface, but when you decode it the difference is:
/files/Membership /2018 New Member Applications/V Gröbl/
/files?sfpath=/Membership+/2018+New+Member+Applications/V+GrÃ¶bl/
I'm not sure where the extra %83%C2 is coming from but I bet if we can remove it or whatever is adding it then the script would allow special characters.

I just tried the latest commit and it works now.  Looks like this issue isn't a problem anymore.

Yahoo has a long history of spectacularly bad handling of character encodings. There are several issues, some of which may be reported in #9 which also contains a draft for a (partial?) fix.

With the last commit (3bd73d6) I no longer have this issue.


forge compiles despite unresolved imports

Component
Forge
Have you ensured that all of these are up to date?

[X] Foundry
[X] Foundryup

What version of Foundry are you on?
forge 0.2.0 (57bb12e 2024-11-06T00:26:30.417023000Z)
What command(s) is the bug in?
forge build
Operating System
macOS (Apple Silicon)
Describe the bug

Clone this repo
forge build

Some imports will be noted as unresolved by forge, but compilation + tests will succeed.
May be related to #9271.

cc @adhusson nightly has been released, please re-run foundryup

reopen as this wasn't something related to the regression introduced, and still reproducible

this happens because of how imports recurse is handled - that is capped to project root


Port forward now listens on IPv4 and IPv6 localhost address

Tries to listen <IP_ADDRESS> and the given port
Tries to listen [::1] and the given port
If both of them fails port-forward fails
If at least one succeeds, port-forward listens on the corresponding interface and displays a warning for the other
If both succeeds, port-portfward listens on both the interfaces

Hi,
this PR fixes the workaround previously set-up to fix the unit test failing which revealed that port-forward should try to listen on addresses rather than on hostnames. #6278
Now, port-forward will try to listen on both the "localhost" ip adresses which are <IP_ADDRESS> and [::1] for respectively IPv4 and IPv6. Port-forwarding will continue unless both the listens fails.
@ncdc could you please review ?
cc @derekwaynecarr

LGTM


TokenId created from Managed Guardian Service published with UUID

Problem description
When I imported and published an iRec7 policy with
Step to reproduce
Steps to reproduce the behavior:

Go to 'MGS'
Use Standard Registry to import iRec7 Envision policy form open source policy list
Publish the policy
Go to the Standard Registry user topic in https://explore.lworks.io/
View the Payload tab and look at the token information in the ESG API payload

Alternatively use the cid in the policy field to query ipfs and get the token information which has the UUID format for the tokenId.
Topic 0.0.10565568 - Guardian TokenID Regression.txt

Published policy: https://explore.lworks.io/testnet/topics/0.0.10565576/messages/57
Expected behavior
The tokenId property should have the Hedera tokenId value.
Screenshots

This was reproduced in a previous local instance of the Guardian related to this closed issue https://github.com/hashgraph/guardian/issues/1913


DOC: document limit and stop price args

Coverage remained the same at 81.621% when pulling 278b39f03ff713d04c81f1dbb4d47ee02466edd6 on api-doc-update into 977e5571a16cc6999db8897caf0bdedf23700562 on master.

Coverage remained the same at 81.619% when pulling aa94724372faf8e29a63f55331c989e17439c25d on api-doc-update into 65de1215e01ceda1755b4095f87ed815e2b04e6b on master.


Loom tests, and a couple basic other improvements

Codecov Report

Merging #4 into main will increase coverage by 0.08%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main       #4      +/-   ##
==========================================
+ Coverage   95.91%   96.00%   +0.08%
==========================================
  Files           3        3
  Lines         392      400       +8
==========================================
+ Hits          376      384       +8
  Misses         16       16

Impacted Files
Coverage Δ

src/arc_str.rs
93.71% <ø> (ø)

tests/arc_str.rs
100.00% <100.00%> (ø)

Continue to review full report at Codecov.

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


Provide a full rerun functionality

Need to be able to remove all CSV files (daily/monthly) in the s3 aloft bucket and replace them by a new run with a given schema-version implementation.
Need to be operational to have a rerun after is merged. https://github.com/enram/vpts-csv/pull/42

The full rerun can be triggered by running the ECS task with a custom defined modified-days as input. Need to check how this config can be best overwritten, but the setup supports overwriting the CLI command of the docker so this will be the first try out.


Setting xmlDeclaration and doctypeDeclaration to false generates a "falsefalse" preamble

Reproduced in version 1.2.2
I configure svg-sprite like that:
{
      svg: {
        xmlDeclaration:  false
        doctypeDeclaration: false
      },
      transform: [{
        svgo: {
          plugins: [{
            removeUnknownsAndDefaults: false
          }]
        }
      }],
      mode: {
        symbol: {
          dest: '.',
          sprite: 'sprite.svg'
        }
      }
    }

I get an output file that starts with falsefalse<svg xmlns="http://www.w3.org/2000/svg" xmlns:xlink="http://www.w3.org/1999/xlink"><symbol.... Seems like the boolean value is treated as a string instead of disabling the declarations.

Note: a workaround is to set the value of both properties to a blank space (empty string won't work)

Hi @jraoult,
thanks for reporting this. Interesting. I'll look into this — it definitely worked at some point. Could you please tell me a little more about your platform (OS, Node version etc.). Thanks!
Cheers,
Joschi

@jkphl yes sure. I'm on a mac os (Yosemite). I reproduced on Node 0.12.2 and iojs 2.2.1. Last time I had it working was the 1.1.0 of svg-sprite.
My guess is that this bug was introduced when the rootAttributes  setting was added.

Should be fixed as of release v1.2.3.


Rough draft of prioritized updating.

This hasn't been tested at all so don't merge it. Also I apologize about the formatting (makes it difficult to see the changes).

Also did a bit of work on this, though the implementation is a bit simpler since I'm just assigning a priority to each player: https://github.com/garyttierney/apollo/commit/d6bb11a3285035a9252f33ecf2d333fd73ca7ac9#diff-e1a08984a7ac500f4ed521a9b2d8e4f7R25