suggestion: move code injected into User to a method and inject the method instead

Maybe it's just me, but I've always favored the declarative approach rather then the procedural one when drawing up a specification. What are your thoughts in regards to moving the injected text out of the model and into its own method? That gives you two things, first, a cleaner User model :). Second, and I think more importantly, it allows you to have finer control over the monkey patching that takes place. Currently, since all the code is dumped into the user's lap, you have to be very careful by not adding anything to the scheme. Hiding it behind a method will give you more flexibility on how the User model will get augmented.
turn this:

inject_into_class(model_path, user_cname.camelize) do
   " include Rolify::Roles\n" +
   " #{'# ' if !options[:dynamic_shortcuts]}extend Rolify::Dynamic\n" +
   " has_and_belongs_to_many :roles#{", :class_name => \"" + role_cname.camelize + "\"" if role_cname != "Role"}, :join_table => :#{user_cname.tableize + "_" + role_cname.tableize}\n"
end

into this:

inject_into_class(model_path, user_cname.camelize) do
   "rolify"
end

def self.rolify option = { :role_cname => 'Role', :dynamic => false }

   include Rolify::Roles
   extend Rolify::Dynamic if options[:dynamic]
   has_and_belongs_to_many :roles, :class_name => options[:role_cname].constantize, :join_table => :"#{self.class.to_s.tabletize}_#{options[:role_cname].tableize}"

end

Thoughts?

I like it. it's pretty neat indeed. I changed many times the design of the module and it would make easier future upgrades.
putting that in the v3 roadmap items.
thanks for your feedback, very appreciated.

Still need to extend the User class before calling rolify method. Will see if I can extend ActiveRecord or Mongoid to avoid that.

I favour the ActiveSupport.on_load(:active_record) do ... end approach :).

For v3.0, I am introducing mongoid support, so I guess I have to find a way to handle both. For instance, having to explicitly extend the class works for both cases.

Apparently ActiveSupport.on_load(:mongoid) seems to work as well but mongoid object doesn't inherit from a base class like ActiveRecord objects, so maybe I will have to the extend Object class... still investigating

Hmm I believe the purpose of ActiveSupport is to handle the individual cases (that is they're supposed to abstract away that issue). I have not dealt with other ORM layers though. Let me know if you need help. I would not mind flexing my brain matter, but I have not yet dealt with other ORM layers before.

I think I will take the Kaminari's approach: https://github.com/amatsuda/kaminari/blob/master/lib/kaminari/hooks.rb


EPIC: research further analysis related features to add to the AOI functionality

Eg: https://github.com/NASA-IMPACT/covid-api/issues/56

Some ideas about further analysis items to include in the AOI functionality, from a discussion with Barry Lefer and Ken Jucks (Air Quality scientists):
Current:

Mean of values within AOI
Median of values within AOI

Potential:

Std. dev of values within AOI
25th and 75th percentile values within AOI
7 day rolling average
15 day rolling average
% of valid data values within the requested AOI (eg: it might be useful to the user to know that the AOI they've drawn is actually half over a body of water for which there is no data)


Unittest Segmentation Fault: VaapiPostProcessScalerTest.mapToRange

On platforms that don't support vaQueryVideoProcFilterCaps (e.g. Ivybridge), the VaapiPostProcessScalerTest.mapToRange test will segfault.  This is because caps is a NULL pointer and we try to deference it.  Before dereferencing the pointer, there should be an ASSERT_TRUE(bool(caps)) before to ensure the test does not continue when it's NULL.
$ ./vpp/unittest
[==========] Running 5 tests from 1 test case.
[----------] Global test environment set-up.
[----------] 5 tests from VaapiPostProcessScalerTest
[ RUN      ] VaapiPostProcessScalerTest.Factory
[       OK ] VaapiPostProcessScalerTest.Factory (0 ms)
[ RUN      ] VaapiPostProcessScalerTest.mapToRange
libva info: VA-API version 0.39.3
libva info: va_getDriverName() returns 0
libva info: Trying to open /home/uartie/Work/workspace/media/install/lib/dri/i965_drv_video.so
libva info: Found init function __vaDriverInit_0_39
libva info: va_openDriver() returns 0
libyami error 5957 (vaapipostprocess_base.cpp, 84): vaQueryVideoProcFilterCaps: the requested filter is not supported
vaapipostprocess_scaler_unittest.cpp:129: Failure
Value of: scaler.mapToRange(value, level, minLevel, maxLevel, filterType, caps)
  Actual: false
Expected: true
vaapipostprocess_scaler_unittest.cpp:130: Failure
Value of: bool(caps)
  Actual: false
Expected: true
Segmentation fault (core dumped)

Hi @uartie ,
We will check it after Q3 release

fixed by 	3376fbf


Add windows wheels

The last commit is not building the wheels since I have switch back the wheel building trigger to "on tags".
The successful build of the wheels is in the previous commit: https://github.com/BlueBrain/MorphIO/runs/1511986706


execution process

please tell how to execute the code provide steps please...........

How to execute your program - Please add steps


Autoclose generates closing tag duplicates

It happened after the last update which I have made today.
This is what happens after closing DIV. Same happened with the body element and the html element before that.
        <div>

        </div>
        </div>

It happens after every update. Basically when the plugin updates it rebinds the events. I put a fix in for this update, however because of the way the bug happens it won't do it's thing until next update.
Just close and reopen atom to get rid of it

Yes, that has done it. Thanks!


.NET core compatibility

Hello
I would like to know whether FileHelpers is compatible with .NET core. I have touch time finding this information as it does mention anywhere explicitly.
Thanks

I am not sure but one of the major differences I've seen between .NET Core and .NET Framework is that core does not return full type information when you do object.GetType() so you also have to do object.GetTypeInfo()
Unfortunately, when I cloned the repo and then searched for GetTypeInfo() the only project that had this in was the FileHelpers.Analyzer project.  Which would lead me to suspect that is the only .NET Core project so far.
There is always a chance I may be wrong though as I'm just a developer whose looked at a couple of issue with the project, not one of the original developers.

We have published an alpha release of FileHelpers dll, targeting directly .NET Core 2.0.
See https://github.com/marcosmeli/filehelpers#current-support-for-net-core for some comments on it.
Please let us know, if that release works for you.
Matthias Cavigelli

With pullrequest #295 we are targeting .NET Standard 2.0!

Regarding to dynamic methods is there any chance to use it on .net standard?


Transition crash issue

To reproduce:
1.swipe to next page;
2.quickly tap the current tab bar item when transitioning.
And it crashed.
ps:set a longer TRANSITION_DURATION may help you to reproduce this issue.
here is the crash log:2015-07-06 16:12:43.178 LxTabBarControllerDemo[8929:1642292] *** -[Transition retain]: message sent to deallocated instance 0x17000fee0

I sent a pull request for this issue.
It seemed not a very elegant solution for this, but it worked.
#3

tabbar上面滑的话，会闪退

@JJetGu  fixed.


DTSPO-16389: reader access of KV containing slack webhook

Jira link (if applicable)
https://tools.hmcts.net/jira/browse/DTSPO-16389
Change description

Add Reader for the keyvault containing slack webhook url for waf-monitoring slack app

Checklist

[x] commit messages are meaningful and follow good commit message guidelines
[x] README and other documentation has been updated / added (if needed)
[x] tests have been updated / new tests has been added (if needed)
[ ] Does this PR introduce a breaking change

Plan Result (prod_app_registrations)
Plan: 1 to add, 0 to change, 0 to destroy.

Create

module.this.azurerm_role_assignment.this["WAF Monitoring-Reader-/subscriptions/1baf5470-1c3e-40d3-a6f7-74bfbce4b348/resourceGroups/core-infra-intsvc-rg/providers/Microsoft.KeyVault/vaults/cftptl-intsvc"]

Change Result (Click me)

  # module.this.azurerm_role_assignment.this["WAF Monitoring-Reader-/subscriptions/1baf5470-1c3e-40d3-a6f7-74bfbce4b348/resourceGroups/core-infra-intsvc-rg/providers/Microsoft.KeyVault/vaults/cftptl-intsvc"] will be created
  + resource "azurerm_role_assignment" "this" {
      + id                               = (known after apply)
      + name                             = (known after apply)
      + principal_id                     = "414c87c4-9f5a-4fcf-b630-91d1c282ace0"
      + principal_type                   = (known after apply)
      + role_definition_id               = (known after apply)
      + role_definition_name             = "Reader"
      + scope                            = "/subscriptions/1baf5470-1c3e-40d3-a6f7-74bfbce4b348/resourceGroups/core-infra-intsvc-rg/providers/Microsoft.KeyVault/vaults/cftptl-intsvc"
      + skip_service_principal_aad_check = (known after apply)
    }

Plan: 1 to add, 0 to change, 0 to destroy.


Fix for issue #528: Change python-opengl to python3-opengl.

Issue: https://github.com/huggingface/deep-rl-class/issues/528
Solution: Use Python3 dependency.

Hey 👋 , thanks for the update. I merge it 🤗


Failed to load delegate from libedgetpu.so.1.0

System information

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

Describe the problem
libedgetpu.so.1.0 fails to load for certain user.
Provide the exact sequence of commands / steps that you executed before running into the problem
interpreter = Interpreter(args.model, experimental_delegates=[load_delegate('libedgetpu.so.1.0')])
It works perfectly for the user I can sudo. But I want to use the library with a different user (that cannot do sudo). I found in another issue that I need to add this user to the plugdev group. Which I did with the following command:
sudo usermod -aG plugdev [your username]
This did not solve the problem.
Any other info / logs
interpreter = Interpreter(args.model, experimental_delegates=[load_delegate('libedgetpu.so.1.0')])
File "/home/certain-user/.local/lib/python3.5/site-packages/tflite_runtime/interpreter.py", line 168, in load_delegate
library, str(e)))
ValueError: Failed to load delegate from libedgetpu.so.1.0

@bsddemon is it only for certain user? Try this:
$ sudo usermod -aG plugdev [your username]

It seems it's not documented. At least, I don't remember seeing it mentioned. Using delegate is kinda API/ABI dependent, if there is data structure or function parameters changes, loading a delegate could fail. A safe way to make sure you can use the Edge TPU delegate is to check the source code by Edge TPU team. E.g., the latest source code you can find now is for diploria2. And the TensorFlow version used by it is TensorFlow https://github.com/tensorflow/tensorflow/commit/5d0b55dd4a00c74809e5b32217070a26ac6ef823. You can find corresponding version in Edge TPU repo's WORKSPACE file, check here

With MacOS restarting the computer solved the problem.

I'm having the same problem and restarting my machine does nothing. :(

For me I tried this also on a RPI4 - 4GB and I still get this error when I connect the TPU to a USB3.0 port. If I use a USB2.0 port it works.

@Syirrus I believe this USB3.0 issues is actually a bug in rpi4's HW. Check here

I encounter this problem on the Coral Edge TPU boards as well.
Setup:

Board is updated (not that it matters since no new software has been released for some time.)
I am trying to run the classification example from this repository
I have installed the wheel for tflite_runtime-2.1.0.post1-cp35-cp35m. Newer wheels refuse to install with errors:

$ pip3 install https://dl.google.com/coral/python/tflite_runtime-2.1.0.post1-cp38-cp38-linux_aarch64.whl
tflite_runtime-2.1.0.post1-cp38-cp38-linux_aarch64.whl is not a supported wheel on this platform.

I haven't made any changes, I'm just trying to run the hello-world of this device and confirm stuff is working.

Full error is below:
tflite/python/examples/classification$ python3 classify_image.py \
>   --model models/mobilenet_v2_1.0_224_inat_bird_quant.tflite \
>   --labels models/inat_bird_labels.txt \
>   --input images/parrot.jpg
Traceback (most recent call last):
  File "classify_image.py", line 122, in <module>
    main()
  File "classify_image.py", line 99, in main
    interpreter = make_interpreter(args.model)
  File "classify_image.py", line 73, in make_interpreter
    {'device': device[0]} if device else {})
  File "/usr/local/lib/python3.5/dist-packages/tflite_runtime/interpreter.py", line 161, in load_delegate
    delegate = Delegate(library, options)
  File "/usr/local/lib/python3.5/dist-packages/tflite_runtime/interpreter.py", line 90, in __init__
    self._library = ctypes.pydll.LoadLibrary(library)
  File "/usr/lib/python3.5/ctypes/__init__.py", line 425, in LoadLibrary
    return self._dlltype(name)
  File "/usr/lib/python3.5/ctypes/__init__.py", line 347, in __init__
    self._handle = _dlopen(self._name, mode)
OSError: libedgetpu.so.1: cannot open shared object file: No such file or directory
Exception ignored in: <bound method Delegate.__del__ of <tflite_runtime.interpreter.Delegate object at 0xffff77fb1e80>>
Traceback (most recent call last):
  File "/usr/local/lib/python3.5/dist-packages/tflite_runtime/interpreter.py", line 125, in __del__
    if self._library is not None:
AttributeError: 'Delegate' object has no attribute '_library'

@youngelf
OSError: libedgetpu.so.1: cannot open shared object file: No such file or directory

Means that you son't have libedgetpu installed.
Also, can you share your outputs of:
cat /etc/os-release
dpkg -l | grep edgetpu

It looks like you are running a very old version of mendel OS to even have python3.5 as default. I know there are a lot of tutorials out there that had older versions but I definitely suggest installing new OS. Currently, we're at mendel 5.0, try this: https://coral.ai/docs/dev-board/reflash/#flash-a-new-system-image

Thanks, namburger. I flashed it last weekend, here we go:
$ cat /etc/os-release
PRETTY_NAME="Mendel GNU/Linux 5 (Eagle)"
NAME="Mendel GNU/Linux"
ID=mendel
ID_LIKE=debian
HOME_URL="https://coral.ai/"
SUPPORT_URL="https://coral.ai/"
BUG_REPORT_URL="https://coral.ai/"
VERSION_CODENAME="eagle"
$ dpkg -l |grep edgetpu
ii  edgetpu-examples                     14.1                                all          Example code for Edge TPU Python API
ii  edgetpudemo                          3-1                                 all          Edge TPU demo script
ii  libedgetpu-dev                       14.1                                arm64        Development files for libedgetpu
ii  libedgetpu1-std:arm64                14.1                                arm64        Support library for Edge TPU
ii  python3-edgetpu                      14.1                                arm64        Edge TPU Python API
ii  python3-edgetpuvision                6-1                                 arm64        EdgeTPU camera API
wget https://dl.google.com/coral/python/tflite_runtime-2.1.0.post1-cp37-cp37m-linux_aarch64.whl--2020-11-05 17:56:48--  https://dl.google.com/coral/python/tflite_runtime-2.1.0.post1-cp37-cp37m-linux_aarch64.whl
Resolving dl.google.com (dl.google.com)... <IP_ADDRESS>, 2607:f8b0:4007:803::200e
Connecting to dl.google.com (dl.google.com)|<IP_ADDRESS>|:443... connected.
HTTP request sent, awaiting response... 200 OK
Length: 1940683 (1.9M) [application/octet-stream]
Saving to: ‘tflite_runtime-2.1.0.post1-cp37-cp37m-linux_aarch64.whl’
tflite_runtime-2.1.0.post1 100%[=======================================>]   1.85M  2.89MB/s    in 0.6s
2020-11-05 17:56:49 (2.89 MB/s) - ‘tflite_runtime-2.1.0.post1-cp37-cp37m-linux_aarch64.whl’ saved [1940683/1940683]
pip3 install tflite_runtime-2.1.0.post1-cp37-cp37m-linux_aarch64.whl
Processing ./tflite_runtime-2.1.0.post1-cp37-cp37m-linux_aarch64.whl
Requirement already satisfied: numpy>=1.12.1 in /usr/lib/python3/dist-packages (from tflite-runtime==2.1.0.post1) (1.16.2)
Installing collected packages: tflite-runtime
Successfully installed tflite-runtime-2.1.0.post1
$ ipython3
Python 3.7.3 (default, Dec 20 2019, 18:57:59)
Type "copyright", "credits" or "license" for more information.
IPython 5.8.0 -- An enhanced Interactive Python.
?         -> Introduction and overview of IPython's features.
%quickref -> Quick reference.
help      -> Python's own help system.
object?   -> Details about 'object', use 'object??' for extra details.
In [1]: import tflite_runtime as tf
In [2]: tf.version
Out[2]: '2.1.0.post1'
~/coral/tflite/python/examples/classification$ python3 classify_image.py \

--model models/mobilenet_v2_1.0_224_inat_bird_quant_edgetpu.tflite
--labels models/inat_bird_labels.txt
--input images/parrot.jpg

Traceback (most recent call last):
File "/usr/local/lib/python3.7/dist-packages/tflite_runtime/interpreter.py", line 161, in load_delegate
delegate = Delegate(library, options)
File "/usr/local/lib/python3.7/dist-packages/tflite_runtime/interpreter.py", line 120, in init
raise ValueError(capture.message)
ValueError
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "classify_image.py", line 122, in
main()
File "classify_image.py", line 99, in main
interpreter = make_interpreter(args.model)
File "classify_image.py", line 73, in make_interpreter
{'device': device[0]} if device else {})
File "/usr/local/lib/python3.7/dist-packages/tflite_runtime/interpreter.py", line 164, in load_delegate
library, str(e)))
ValueError: Failed to load delegate from libedgetpu.so.1
(I suspect hardware damage, as on this board the Wifi is detected with 'lshw' but nmcli doesn't think it exists.)
nmcli d
DEVICE  TYPE      STATE      CONNECTION
eth0    ethernet  connected  Wired connection 1
usb0    ethernet  connected  gadget0
usb1    ethernet  connected  gadget1
lo      loopback  unmanaged  --
Full output of 'lshw' attached here:
lshw.txt

@youngelf does
ls /dev/apex_0
shows anything?

Nope, nothing.
# ls /dev/
autofs           input             mxc_hantro          random     tty22  tty46    ttymxc2   vcs2
block            ion               net                 rtc        tty23  tty47    ttyp0     vcs3
...
initctl          mqueue            ptypf               tty21      tty45  ttymxc1  vcs1

Nothing called apex.

@youngelf apex is literally how the software interact with the edgetpu lol
Could you ping us an email to<EMAIL_ADDRESS>Looks like some how the edgetpu completely disappear from the board, we can issue a replacement

Super, thanks. Will do.

Got the updated boards, and I was able to run the inference in the classify_image.py example from here.
Thanks so much!

@bsddemon Could you please try to  upgrade the tflite_runtime package from link and try to execute your code using latest stable version of TF 2.6.0 .Please let us know if helps?Thanks!

I have also faced this issue and I think one more possible reason is that the user may not be the part of apex group. So, adding user to the apex group can fix this issue. This can be done using:
sudo usermod -a -G apex <user_name>
This may fix #32743.

So what was actually causing the problem? What was the solution? I connected the Coral Edge USB version to a desktop PC, I'm running into the issues described above. Looks like the Coral device 1a6e:089a is connected, but apex_0 is not available.
$ python3 examples/classify_image.py --model test_data/mobilenet_v2_1.0_224_inat_bird_quant_edgetpu.tflite --labels test_data/inat_bird_labels.txt --input test_data/parrot.jpg

ValueError: Failed to load delegate from libedgetpu.so.1

$ dpkg -l | grep edgetpu
||/ Name                   Version  Architecture  Description
ii  libedgetpu1-std:amd64  16.0     amd64         Support library for Edge TPU

$ lsusb | grep "1a6e:089a"
Bus 004 Device 007: ID 1a6e:089a Global Unichip Corp.

$ ls /dev/apex_0
ls: cannot access '/dev/apex_0': No such file or directory

Disregard the above. I have not restarted this server in years because everything just works without ever rebooting.
But I thought I'd give it a shot and reboot. Now it works.
PS - I stil do not have /dev/apex_0. But the inferencing examples now work. Turns out this is only for the PCIe Coral. So to avoid confusion for future googlers, the USB Coral is hidden away in /dev/bus/usb.


Add TagHelperSample.Web functional tests.

Validated that each of TagHelperSample.Webs sites work properly.
Found that the ConditionalCommentTagHelper page wasn't working as intended due to never being updated after TagHelperContent encoding changes.
Added content verification for the TagHelper/ConditionalComment endpoint only since its static HTML.

#3530

/cc @dougbu @pranavkm @rynowak @ajaybhargavb

What abouuuut the Readme changes @Eilon requested in https://github.com/aspnet/Mvc/issues/3530#issuecomment-155951093 ?

Important comment on comments in the commit description's comments:
... since its static HTML.

s/b
... since it's static HTML.

:trollface:

:watch:

:up: :date:

:watch:

:up: :date:

:watch:

:up: :date:

:shipit:


Update unidic-cwj to 3.1.1

@kampersanda I suggest splitting the download scripts into a separate repository.
Including the download scripts requires constant tracking of changes.
Since Vibrato does not depend on specific dictionaries, that would be an extra burden.

@vbkaisetsu In the near future, we will also be distributing mappings learned from a certain corpus.
At that time, we plan to make it a policy to distribute them using Assets.


Avoid top-level with ...; in pkgs/development/coq-modules

Description of changes
Using with at a top-level scope is an anti-pattern. The tracking issue is #208242. This is a pure refactor: there is no functional change contained in this PR.
I used the refactor strategy that looks for the uses of names coming from lib, pkgs, python3Packages, and more.
Following advice, I also preferred to inherit names from lib instead of builtins where there was a choice. I also preferred to inherit from lib in one place, rather than to have a mixture of lib.thing and inherit (lib) thing.
There are no rebuilds when I run nixpkgs-review rev HEAD, so I've targeted master with this PR.
Things done

[x] Tested compilation of all packages that depend on this change using nixpkgs-review rev HEAD.
[x] Fits CONTRIBUTING.md.

Please change the manual first.

Please change the manual first.

I did three weeks ago. https://github.com/NixOS/nixpkgs/pull/293829/commits/9f8a58026b4a82bdaf24cdeffcbf8e98ce354984

Here we go: #299927

Here we go: #299927

This was merged; what would you like to do next?

Probably a good idea to explicitly use lib.maintainers and so on, unless we need them multiple times and frequently. Only in those cases, I'd use inherit.

I took a closer, hand-crafted look at these patches and updated them to use lib. where that was needed, instead of using inherit (lib) .... The result looks OK to my eyes.
Several of the packages asked for recurseIntoAttrs to be injected, but didn't actually use it. I removed those.
I didn't style the code at all beyond what was necessary.

Result of nixpkgs-review pr 299010 run on x86_64-linux 1


some? macros don't get resolved in script

I don't know if this is a zabbix 6.4 thing or what, but I can't seem to actually have at least some of the macros resolved in the script.
zabbix-server-pgsql_1     |    191:20240508:210928.542 [ Google Chat Webhook ] ERROR: Zabbix URL Macro is invalid or not defined: {$ZABBIX_URL}
This is definitely defined (I tried _ instead of . to see if it fixed it).   When I test the script I get the same... and if I change the zabbix_url to a string, I get errors about some of the other fields instead.  But they also seem to be undefined when an actual event happens.

User Error.
The logs looked like it was failing during an actual alert, but that must have been my testing through the UI.


Feature/async lt

Load testing is now run asynchronously. This is very cool, @girishranganathan!
There are a few extra notifications (in blue) that probably aren't necessary, but this is a great feature.


Revert "move version to parent pom"

Reverts apache/incubator-dolphinscheduler#1712

Codecov Report

Merging #1715 into dev will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##              dev    #1715   +/-   ##
=======================================
  Coverage   17.48%   17.48%
=======================================
  Files         285      285
  Lines       13912    13912
  Branches     2274     2274
=======================================
  Hits         2433     2433
  Misses      11206    11206
  Partials      273      273

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 87e8f0d...2271c96. Read the comment docs.

+1


New version: Diskuv.OCaml version 1.2.0

Pull request has been created with Komac v1.11.0 :rocket:
Microsoft Reviewers: Open in CodeFlow

Automatic Validation ended with:

Manual Validation took about 33 hours.

@wingetbot waivers Add Validation-Unattended-Failed


Load balancing scheme in Trino gateway?

What's the load balancing scheme used by Trino gateway? And is there a way to change it?

By default the gateway picks a random cluster within the matched routing group. See https://github.com/trinodb/trino-gateway/blob/main/gateway-ha/src/main/java/io/trino/gateway/ha/handler/QueryIdCachingProxyHandler.java#L249 and https://github.com/trinodb/trino-gateway/blob/main/gateway-ha/src/main/java/io/trino/gateway/ha/router/RoutingManager.java#L91

@vishalya can you potentially take this on to document as part of the update for the linked PR ?

Please see https://trinodb.github.io/trino-gateway/routing-logic/#routing-based-on-cookies


FEAT add PKU-SafeRLHF dataset

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

https://arxiv.org/pdf/2406.15513
https://huggingface.co/datasets/PKU-Alignment/PKU-SafeRLHF
could be added as another dataset to the datasets module. The prompt column is the relevant one. Some of these aren't necessarily harmful but that's okay.
Describe the solution you'd like

This should get a similar fetch function as other datasets.

I will work on this task for FHL

Hello, I am interested in working on this for FHL


Add protobuf type as a transaction method parameter [ECR-3991]

Overview
Add protobuf type as a transaction method parameter.

See: https://jira.bf.local/browse/ECR-3991
Definition of Done

[ ] There are no TODOs left in the code
[x] Change is covered by automated tests
[x] The coding guidelines are followed
[x] Public API has Javadoc
[x] Method preconditions are checked and documented in the Javadoc of the method
[x] Changelog is updated if needed (in case of notable or breaking changes)
[ ] The continuous integration build passes

Coverage increased (+0.04%) to 86.861% when pulling 07515ada22c89b1c16607f7ee53b1c77ab39e69c on ECR-3991 into daa4a473393506af7b3773a630caab15a1cfda1f on master.


Support for casts in macros with smaller integers

Partially fix #4 in the most common cases.
Any integer type longer than 32 bits(size_t, uint64_t, long) is not supported since EvalResult::Int can not hold it. For example:
#define Int_18446744073709551615 (unsigned long)-1

Please give me some ideas to simplify the implementation. I know it is ugly.

Thanks for your contribution! I won't have time to look at it for at least a week, but maybe @emilio can

It looks reasonable, but I don't have enough nom expertise to properly review. Thanks a lot for working on this!

@jethrogb Is there any chance to get this pull request merged? I am trying to upload my first crate to crates.io and this feature is the only blocker.

@crab2313, any update on this?


Automatically lookup ('find') linked async records as they are requested

Something like how ember-data does it: https://github.com/emberjs/data/blob/master/packages/ember-data/lib/system/relationships/relationship.js#L329-L337 and https://github.com/emberjs/data/blob/master/packages/ember-data/lib/system/relationships/relationship.js#L236-L244
Pretty important for smoothly nesting templates and controllers in my opinion and also for transitioning from ember-data.

@leejt489 @dgeb any updates on the best way to access the relationship.
I tried using the route's afterModel hook for a post that I know has comments but for some reason the comments are empty, see: https://github.com/pixelhandler/blog/blob/comments/app/routes/post/detail.js#L9-L11

Oh my, this issue has been open a while. Sorry for not fully addressing all the questions sooner! 😅

The members of relationships can now be directly accessed on orbit models:
let author = post.author; // returns an `Author` model
let comments = post.comments; // returns an array of `Comment` models

Relationships, and all other fields, will stay updated as values in the underlying cache are updated.
If you want to force an update of the cache, you can re-query the associated Store:
await store.query((q) => q.findRelatedRecord(post, 'author');
await store.query((q) => q.findRelatedRecords(post, 'comments');

You can also pass query options, such as pagination and include params, as described in the orbit docs.


Add setup experiments notebook

Closes #164
This PR adds a notebook that creates a space with input files for demo users on the s3 bucket.
Signed-off-by: Shreyanand<EMAIL_ADDRESS>

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

Needs approval from an approver in each of these files:

OWNERS

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

@Shreyanand: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

aicoe-ci/prow/pre-commit
5abb1e3856109d6b814a5ab38daaa56c2f71f585
link
true
/test pre-commit

Full PR test history. Your PR dashboard. Please help us and open an issue when you hit one in your PR.

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

@Shreyanand: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

aicoe-ci/prow/pre-commit
fb4f2fac7f26ed7acb0b5aa872a7cd0e3e981e6c
link
true
/test pre-commit

Full PR test history. Your PR dashboard. Please help us and open an issue when you hit one in your PR.

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

@Shreyanand: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

aicoe-ci/prow/pre-commit
98f6889f2550ffa40916fb2ca5daf727b5e194d7
link
true
/test pre-commit

Full PR test history. Your PR dashboard. Please help us and open an issue when you hit one in your PR.

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

pre-commit check is failing but not on the files changed in this PR. @Shreyanand and I discussed this, and it will be addressed in a forthcoming PR. Merging manually for now.


Python 3.5.2 Compabilitiy with mxnet ?

Hi :
I just wonder if current mxnet-git compatible with Python 3.5.2???
Environment info
Operating System:
Ubuntu 16.04.1
Compiler:
GCC/G++ 5.4.0
Package used (Python/R/Scala/Julia):
Python 3.5.2
MXNet version:
0.9.1 (Current-git)
Or if installed from source:
Yes, install from source
MXNet commit hash (git rev-parse HEAD):
1ae2905cc3c2e35541d45edab18ddffb9cf455da
If you are using python package, please provide
Python 3.5.2
Python version and distribution:
3.5.2
Error Message:
Please paste the full error message, including stack trace.
RefactoringTool: Warnings/messages while refactoring:
RefactoringTool: ### In file ./example/image-classification/symbol/inception-resnet-v2.py ###
RefactoringTool: Line 111: You should use 'operator.mul(tower_5b_out, 10, block35, scale=0.17, input_num_channels=320)' here.
RefactoringTool: Line 119: You should use 'operator.mul(net, 20, block17, scale=0.1, input_num_channels=1088)' here.
RefactoringTool: Line 132: You should use 'operator.mul(net, 9, block8, scale=0.2, input_num_channels=2080)' here.
RefactoringTool: There was 1 error:
RefactoringTool: Can't parse ./example/fcn-xs/image_segmentaion.py: IndentationError: unindent does not match any outer indentation level (<tokenize>, line 22)

Steps to reproduce
or if you are running standard examples, please provide the commands you have run that lead to the error.

Trying to convert all .py files from version 2 to version 3 by using command "2to3" .

Cheers
Pei

This issue is closed due to lack of activity in the last 90 days. Feel free to reopen if this is still an active issue. Thanks!


Release 1.6.0

Signed-off-by: Quan Tian<EMAIL_ADDRESS>

Create the PR for review, will update the target branch to release-1.6 and add a commit to update VERSION once the remaining PRs are merged.

Thanks @jianjuns @dreamtalen. Addressed your comments.

Thanks @antoninbas. Addressed your comments.

/test-whole-conformance
/skip-conformance
/test-networkpolicy
/test-e2e
/test-flexible-ipam-e2e
/test-multicluster-e2e
/test-multicast-e2e
/test-windows-all
/test-ipv6-all
/test-ipv6-only-all

Codecov Report

Merging #3531 (b8d288e) into release-1.6 (245ac54) will decrease coverage by 1.16%.
The diff coverage is n/a.

@@               Coverage Diff               @@
##           release-1.6    #3531      +/-   ##
===============================================
- Coverage        64.22%   63.06%   -1.17%
===============================================
  Files              278      278
  Lines            27825    27825
===============================================
- Hits             17871    17547     -324
- Misses            8042     8406     +364
+ Partials          1912     1872      -40

Flag
Coverage Δ

kind-e2e-tests
49.55% <ø> (-1.74%)
:arrow_down:

unit-tests
43.38% <ø> (-0.02%)
:arrow_down:

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


Binder

Building a binder for the tutorial.

Codecov Report

Merging #74 into master will not change coverage.
The diff coverage is n/a.

@@          Coverage Diff           @@
##           master     #74   +/-   ##
======================================
  Coverage    85.7%   85.7%
======================================
  Files           8       8
  Lines        1478    1478
======================================
  Hits         1267    1267
  Misses        211     211

networkx is required for greedy and should be added to requirements.txt.
    try:
        import networkx as nx
    except ImportError:
        raise ImportError("The 'networkx' package is required.")

Where is mapclassify.yml  being used?


Split sort query index by last underscore

There is a bug when using sorting on fields with an underscore in the field name. For example query, PRICE_MINIMAL_DESC was sent to the backend as field=PRICE, direction=MINIMAL.

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

Sorry, this PR was closed automatically due to us reorganising the repo. Would you mind reapplying your changes? Thanks for the contribution!


Add fixtures for jellyfish-transaction test case should bi-directional buffer-object-buffer

Just for knowledge sharing, we need to add at least 3 fixtures for should bi-directional buffer-object-buffer
As you copied this from the previous programmer, we can ignore this for now.
Originally posted by @jingyi2811 in https://github.com/JellyfishSDK/jellyfish/pull/1252#discussion_r859349082

/triage accepted
/assign @jingyi2811
/area packages workflow

/kind fix

Also, withdrawFromVault uses J. The letter S (0x53) should be changed to J (0x4a).


Increase the dimension of the latent space progressively and infer the optimum latent space dimension via AIC/BIC

Increasing the dimension of latent space progressively has two major benefits.

The MLE of the coverage model has many local minima. The EM algorithm lands on different solutions depending on how early one starts updating copy ratio posteriors, and "how" the dimension of the latent space is set (see below).

If one starts with a high dimensional latent space (assuming no CNV-avoiding regularization), most of the real events will be identified as bias and get "washed off", and the HMM will be unable to identify any events. This leads to low sensitivity. On the other hand, if one starts with a low dimensional latent space, most of the real CNV events will remain intact, allowing the HMM to identify many more change points, leading to high sensitivity (though, low specificity).
It is desirable to have both protocols implemented and to check which one gives better solutions. We are really exploring the land of heuristics here. Due to the large scale of the problem, an annealing-based approach is unlikely to be practical (is it?); hence, we have to resort to heuristics.

Having calculated the data log likelihood for different dimensions of the latent space, we may then use an information theoretic criterion (AIC or BIC) to infer the dimension of the latent space.

This is addressed in PR #857 using ARD + PCA-initialization.


Update Cake.MinVer to target Cake v3.0.0

Cake 3.0.0 has been released  🎉

:tada: The changes implemented in response to this issue shipped with Cake.MinVer v3.0.0
The release is available on:

:rocket: GitHub Release
:package: NuGet Package


import() method misbehaving

I exported the flowchart using the output() method. After that, I tried to render the chart using the import method().
Rendered objects that are displayed on the canvas is one less comparing to the output items.
Also, I'm receiving the error "Unhandled rejection: TypeError: Cannot read property 'parentNode' of null" when I was trying to snap a new item to the rendered ones.
This how does the rendered canvas look like after import. there should be 3 but are 2.

Just let you know the elements inside the "blocks" is one less than the "blockarr" items. Could it be related to this?
@alyssaxuu

Can you post your output (or a sanitized version)? I'm not following your description and need to see what your working with.

@fcnyp
Im encountering the same issue. Im using flowy.output() in the snapping callback to save the state of the tree (For an undo/redo implementation) but Ive encountered issues with regards to flowy.import() Here is the output I receive
{    "html":"
<input data-v-084a4425=\"\" type=\"hidden\" name=\"blockelemtype\" value=\"1\"><input type=\"hidden\" name=\"blockid\" class=\"blockid\" value=\"0\">\n
\n
New visitor
\n
\n
\n              \n
\n
\n
\n
\n              <input type=\"text\" value=\"When a new visitor goes to Site 1\">\n
\n

",    "blockarr":[       {          "parent":-1,          "childwidth":0,          "id":0,          "x":814,          "y":351,          "width":318,          "height":83       }    ],    "blocks":[       {          "id":0,          "parent":-1,          "data":[             {                "name":"blockelemtype",                "value":"1"             },             {                "name":"blockid",                "value":"0"             },             {                "name":null,                "value":"When a new visitor goes to Site 1"             }          ],          "attr":[             {                "data-v-084a4425":""             },             {                "class":"configuratorTool block cfgCard"             },             {                "style":"left: 299px; top: 115px;"             }          ]       }    ] }
I should note as well that this output is only available after I the second time the snapping callback executes, after the first, the output appears to be undefined.  (As an aside: When in the callback lifecycle is the entire output ready?)
At the end of my snapping callback Im pushing flowy.output() into a undo stack as an object (not stringified as above), and an on page button will pop this object and push into flowy.import
When I try to import this output I get the following exception:
vue.js:1897 TypeError: Cannot read property 'parentNode' of null     at N (flowy-engine.js:1)     at Function.e.import (flowy-engine.js:1)     at VueComponent.undoState (Config.vue:157)     at click (Config.vue?0639:66)     at invokeWithErrorHandling (vue.js:1863)     at HTMLButtonElement.invoker (vue.js:2188)     at HTMLButtonElement.original._wrapper (vue.js:7547)
Is there something Im missing with how the output() & import() should be handled?

Should be working fine now, I updated the library.


If CMake >= 3.24 use CMAKE_COMPILE_WARNING_AS_ERROR to enable Werror

The -Werror option is typically enabled automatically by library authors, so they ensure that all contributors to the library early fail if new code they write contain a warning, ensuring as soon as possible that no new warnings are added to the library. On the other hand, people that package libraries for distributions prefer to disable the -Werror option, as they may want to compile a given library with a new compilers (that may not have even existed when a given release of a library was tagged), that introduce new warnings, without having a failure. For a detailed discussion of this from the point of people packaging libraries, see https://youtu.be/_5weX5mx8hc?si=ZtiWaK7KPTfQ01_g&t=322 .
Thanks to the CMAKE_COMPILE_WARNING_AS_ERROR CMake option available since CMake 3.24, it is possible to have the best of both worlds. Library authors can define this variable as an option and have it ON by default, while people packaging libraries can just set it to OFF from CMake command line invocation (or pass --compile-no-warning-as-error option to CMake) to have build scripts that will work also with more recent compilers.

See #118 for how I plan on addressing this.


some info about catalogs

Hi, I would validate some xml comparing with my xsd.
Have I use the catalog? How coudl I configure it on Windows 10?
Thanks a lot

I don't have access to a Windows machine at the moment, but maybe you can take a look at the comments at https://github.com/eerohele/exalt/issues/17 for some pointers.


Class to cipher and decipher a SMSMessage

The following class encrypt and decrypt SMSMessages using a symmetric encryption alghoritm.
To encrypt/decrypt it takes the message to encrypt/decrypt and XOR it with the choosen password.

 Here is an overview of what got changed by this pull request:

Issues
======
- Added 8

Complexity increasing per file
==============================
- app/src/test/java/ingsw/group1/findmyphone/cryptography/RandomStringGeneratorTest.java  2
- app/src/main/java/ingsw/group1/findmyphone/cryptography/SMSCipher.java  3
- app/src/test/java/ingsw/group1/findmyphone/cryptography/SMSCipherTest.java  2
- app/src/main/java/ingsw/group1/findmyphone/cryptography/RandomStringGenerator.java  2

See the complete overview on Codacy

 Here is an overview of what got changed by this pull request:

Issues
======
- Added 1

Complexity increasing per file
==============================
- app/src/test/java/ingsw/group1/findmyphone/cryptography/RandomStringGeneratorTest.java  2
- app/src/test/java/ingsw/group1/findmyphone/cryptography/SMSCipherTest.java  2
- app/src/main/java/ingsw/group1/findmyphone/cryptography/PasswordManager.java  1
- app/src/main/java/ingsw/group1/findmyphone/cryptography/RandomStringGenerator.java  2
- app/src/main/java/ingsw/group1/findmyphone/cryptography/SMSCipher.java  3
- app/src/test/java/ingsw/group1/findmyphone/cryptography/PasswordManagerTest.java  1

See the complete overview on Codacy

Also, build.gradle conflict, try pulling from development and selecting "Accept Theirs".

 Here is an overview of what got changed by this pull request:

Issues
======
- Added 1

Complexity increasing per file
==============================
- app/src/main/java/ingsw/group1/findmyphone/cryptography/SMSCipher.java  3
- app/src/test/java/ingsw/group1/findmyphone/cryptography/RandomStringGeneratorTest.java  2
- app/src/test/java/ingsw/group1/findmyphone/cryptography/PasswordManagerTest.java  1
- app/src/test/java/ingsw/group1/findmyphone/cryptography/SMSCipherTest.java  2
- app/src/main/java/ingsw/group1/findmyphone/cryptography/PasswordManager.java  1
- app/src/main/java/ingsw/group1/findmyphone/cryptography/RandomStringGenerator.java  2

See the complete overview on Codacy

Thx meglio ora. Come dicevo stasera provo con mano se manca qualcosa, mi è difficile verificare che tutti i test siano in regola ad occhio.

 Here is an overview of what got changed by this pull request:

Issues
======
- Added 1

Complexity increasing per file
==============================
- app/src/test/java/ingsw/group1/findmyphone/cryptography/PasswordManagerTest.java  1
- app/src/main/java/ingsw/group1/findmyphone/cryptography/RandomStringGenerator.java  2
- app/src/main/java/ingsw/group1/findmyphone/cryptography/PasswordManager.java  1
- app/src/test/java/ingsw/group1/findmyphone/cryptography/RandomStringGeneratorTest.java  1
- app/src/main/java/ingsw/group1/findmyphone/cryptography/SMSCipher.java  3
- app/src/test/java/ingsw/group1/findmyphone/cryptography/SMSCipherTest.java  2

See the complete overview on Codacy

 Here is an overview of what got changed by this pull request:

Issues
======
- Added 1

Complexity increasing per file
==============================
- app/src/test/java/ingsw/group1/findmyphone/cryptography/RandomStringGeneratorTest.java  1
- app/src/main/java/ingsw/group1/findmyphone/cryptography/RandomStringGenerator.java  2
- app/src/main/java/ingsw/group1/findmyphone/cryptography/PasswordManager.java  1
- app/src/main/java/ingsw/group1/findmyphone/cryptography/SMSCipher.java  3
- app/src/test/java/ingsw/group1/findmyphone/cryptography/SMSCipherTest.java  2
- app/src/test/java/ingsw/group1/findmyphone/cryptography/PasswordManagerTest.java  1

See the complete overview on Codacy

I'm seeing Process completed with exit code 1 in the github utility. Is it crashing?

Yep, the Android CI is crashing currently, my best guess from the log is that it finds some kind of issue when trying to compile the library submodule, hence why I insisted we moved to the central repo

 Here is an overview of what got changed by this pull request:

Complexity increasing per file
==============================
- app/src/test/java/ingsw/group1/findmyphone/cryptography/SMSCipherTest.java  2
- app/src/main/java/ingsw/group1/findmyphone/cryptography/SMSCipher.java  3
- app/src/test/java/ingsw/group1/findmyphone/cryptography/PasswordManagerTest.java  1
- app/src/main/java/ingsw/group1/findmyphone/cryptography/PasswordManager.java  1
- app/src/main/java/ingsw/group1/findmyphone/cryptography/RandomStringGeneratorUtils.java  2
- app/src/test/java/ingsw/group1/findmyphone/cryptography/RandomStringGeneratorTest.java  1

See the complete overview on Codacy


Clarify steps and language

Signed-off-by: Erin Schnabel<EMAIL_ADDRESS>

This change is

@kate says it looks ok. More edits to follow.

Reviewed 1 of 1 files at r1.
Review status: all files reviewed at latest revision, all discussions resolved.

Comments from Reviewable


BIGTOP-3930. Fix deployment failure on Ubuntu 22.04 due to unavailable dependency on python.

https://issues.apache.org/jira/browse/BIGTOP-3930

added alternative dependency to python2.
added manifest to use /usr/bin/python pointing to python2 on Ubuntu 22.04.

Thank you for working on fixing these critical issues. Let me test it on Arm64.

It’s noticed that Ambari and YCSB seem to be having trouble building on x86 and Arm:
Ambari:
[INFO] --- exec-maven-plugin:1.2.1:exec (Bower install) @ ambari-admin ---
bower angular-translate#* CERT_HAS_EXPIRED Request to https://registry.bower.io/packages/angular-translate failed: certificate has expired
....
..
.
[ERROR] Failed to execute goal org.codehaus.mojo:exec-maven-plugin:1.2.1:exec (Bower install) on project ambari-admin: Command execution failed.: Process exited with an error: 1 (Exit value: 1) -> [Help 1]

YCSB:
[ERROR] Failed to execute goal on project maprdb-binding: Could not resolve dependencies for project site.ycsb:maprdb-binding:jar:0.17.0: Failed to collect dependencies at org.apache.hbase:hbase-client:jar:1.1.8-mapr-1710: Failed to read artifact descriptor for org.apache.hbase:hbase-client:jar:1.1.8-mapr-1710: Could not transfer artifact org.apache.hbase:hbase-client:pom:1.1.8-mapr-1710 from/to mapr-releases (http://repository.mapr.com/maven/): Transfer failed for http://repository.mapr.com/maven/org/apache/hbase/hbase-client/1.1.8-mapr-1710/hbase-client-1.1.8-mapr-1710.pom 500 Proxy Error -> [Help 1]

The cause is error of network or server on downloading dependencies unrelated to Python. Feel free to file another JIRA if it is reproducible. @guyuqi

I'll file a ticket to fix the ycsb issues.

@guyuqi issues of Ambari and YCSB are resolved. Are you OK to move this forward?

The next Ubuntu drops python2, how many python scripts need to migrate in this project?

@pan3793 The next Ubuntu is out of scope of this PR. Please file a JIRA issue or use dev-ML if you want.


Does not work with non-stable discord versions

This doesnt work with discord canary or discord ptb

or im dumb never mind


Can't seem to be able to use assembly attributes

Hi,
I'm trying to set the base class to be  of type BaseContent
have added this line to AssemblyInfo.cs
[assembly: ModelsBaseClass(typeof(BaseContent))]
Yet when I try to generate models, (I've tried both AppData, or using Visual Studio custom Tool)
The models still apear to be inheriting from  PublishedContentModel
Am I missing something?
Thanks,
Doron

Sorry for the horribly late reply. The line should be added to a .cs file alongside the models, ie some sort of Models.cs file in the same directories the files are generated in.
Making sense?

Ok.
Thanks alot.
I will give this a try.
On 7 Nov 2016 3:41 p.m., "Stephan"<EMAIL_ADDRESS>wrote:

Sorry for the horribly late reply. The line should be added to a .cs file
alongside the models, ie some sort of Models.cs file in the same
directories the files are generated in.
Making sense?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/zpqrtbnk/Zbu.ModelsBuilder/issues/117#issuecomment-258837985,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AEuINZoXm_MjDapf8MpmrR-Nr1iZdM8gks5q7yqJgaJpZM4JpSlg
.

Cool, it works by all my tests, so closing the issue - but in case of any problem, just reopen it!


Docs not generating in API readiness app for PR

The docs are not being generated.  The screen continues to show "Documents generating" but it never resolves.  This may be associated with something wrong in the PR, not sure.  If there is an error that needs to be addressed by user or a failure in our tooling, we should let the user know what is happening.

This looks to be a tooling issue not related to the Release Planner App. I can see the pipeline finished, but the status was never updated. I created issue https://github.com/Azure/azure-sdk-tools/issues/6229 for eng to follow up.
@ladonnaq is this blocking the release plan?


Cleanup quay image after PR is closed

Resolves #761
Replaces #811
Description
This PR implements a github action that automatically deletes a quay image for a PR that is closed.
How Has This Been Tested?

Create a PR and its image
Trigger action by closing a PR
Verify the associated image is deleted

Merge criteria:

[x] The commits are squashed in a cohesive manner and have meaningful messages.
[x] Testing instructions have been added in the PR body (for PRs involving changes that are not immediately obvious).
[x] The developer has manually tested the changes and verified that the changes work

/lgtm
I did not manually test this but everything looks like it should work as intended


Performance on Gated2Depth model

Hi Tobias,
I am curious if you have tried the Gated2Depth model on gated images given for this dataset. I seem to have issues with obtaining the correct depth using the pre-trained model for gated2depth.
Thanks,

Hi. The gated images in the Pixel-Accurate Depth Benchmark are acquired with different range-intensity profiles because the length of the fogchamber is quite limited (< 30m). Therefore, the Gated2Depth model does not work on those images. There will be a paper published at the ATZ Automated Driving (https://www.atzlive.de/en/events/automated-driving/) this October that deals with the adaptation of the Gated2Depth model to the new range-intensity profiles in the fogchamber. Best, Tobias

Thanks, your explanation.

@gruberto Is this why radar data is also not available for this?

For this dataset we have not recorded radar data because the fogchamber is mainly a metal construction and the data won't be that helpful.

I understand. Thank you.


Discord link is invalid

The invite link to Discord is expired or leads to something with incorrect permissions. This is the error screen:

Thanks for reporting! Updated the link and made it non-expirable from the discord settings.
Here is the link for reference: https://discord.gg/jAAqT7eYEN


WIP: added the basic authentication function

fix #107  @lioramilbaum

PR tested, does NOT break the app.
the function will help the team members to implement their functions easily and uniformly.
the function is included in a custom library: app_functions.
function still a WIP since it lacks the functions: #105 #104 #103 #102

@Omeramsc I overlooked the WIP in the subject line. :-(


correct repo for filewave importer and input source path

for filewave recipes, should the filewave importer processor not be:
Processor
com.github.autopkg.filewave.FWTool/FileWaveImporter
also should the path for import source not be:
			<key>fw_import_source</key>
			<string>%RECIPE_CACHE_DIR%/%NAME%/Applications/%NAME%.app</string>

i had to make the above corrections to successfully run the filewave recipes generated by rr.
thank you btw for a great utility!

waves to CV
Nice to see you on github.

On Jun 9, 2016, at 11:10 AM, CV Rao<EMAIL_ADDRESS>wrote:
for filewave recipes, should the filewave importer processor not be:
Processor
com.github.autopkg.filewave.FWTool/FileWaveImporter
also should the path for import source not be:
        <key>fw_import_source</key>
        <string>%RECIPE_CACHE_DIR%/%NAME%/Applications/%NAME%.app</string>

i had to make the above corrections to successfully run the filewave recipes generated by rr.
thank you btw for a great utility!
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.

Thanks! Queued up for 1.0.3.


Any idea what this strange purple smoke is?

https://github.com/user-attachments/assets/948aa7bf-fa3c-4796-a923-2bf3d7bf8722
I wish I knew more so I could provide better info, but it seems to appear sometimes if I incapacitate a bunch of guards by disarming them. I've seen it appear once before in Chicago and just now in Carrington Villa. The 1st time I saw it, it was encircling the exploded cab. This time, it's just floating about in the air above the streets.
Using https://github.com/fgsfdsfgs/perfect_dark/commit/683d083fab5a1ec1f88e5de371cce113c5ed200f. x86_64 build. I reproduced it in an earlier commit, too, but don't remember the exact one.

Do you remember if it was before the 64-bit fixes or after?

It was a commit after the 64-bit fixes. I don't recall seeing it prior to them, but I also never ran around the level incapacitating all the guards either, which seems to be a prerequisite to producing this smoke. So, I can't say for certain whether this is 64-bit-fix exclusive or not.

@ZenithMDC did you have any cheat activated?

No cheats enabled. I just ran around Chicago again, knocking out all the guards on the streets and backalleys. It seems like there's one guard in particular, who is one of the guards that is distracted by the diversion, where I actually see that purple smoke fly out of him as he's knocked out and then it shoots up into the air. After a while, it falls down and starts hovering around the crash site. Very bizarre.

Just by going there and disarming them, or did you complete the diversion objective first? Also, which difficulty?

I'm playing on Agent difficulty and always complete the diversion objective. I actually never tested doing it without completing the diversion 1st.

I wonder if that guard is supposed to drop something specific but somehow it gets changed into PROPTYPE_SMOKE.

After spawning the purple smoke and running around Chicago for ~10 minutes or so, the game will crash. It doesn't happen every time, but it's highly likely to crash after around that duration. Here is the log:
Crash!

EXCEPTION: 0xc0000094
PC: 00007ff6c02b7d20
MODULE: [00007ff6c0160000]
MAIN MODULE: [00007ff6c0160000]

BACKTRACE:
#00: 00007ff6c02b7d20: [00007ff6c0160000]+0000000000157d20
#01: 00007ff6c029e457: [00007ff6c0160000]+000000000013e457
#02: 00007ff6c02191ae: [00007ff6c0160000]+00000000000b91ae
#03: 00007ff6c02ec5e7: [00007ff6c0160000]+000000000018c5e7
#04: 00007ff6c02ecb1c: [00007ff6c0160000]+000000000018cb1c
#05: 00007ff6c02ecd62: [00007ff6c0160000]+000000000018cd62
#06: 00007ff6c02e4db5: [00007ff6c0160000]+0000000000184db5
#07: 00007ff6c016140a: [00007ff6c0160000]+000000000000140a
#08: 00007ff6c01614d6: [00007ff6c0160000]+00000000000014d6
#09: 00007ffb22947374: BaseThreadInitThunk+20
#10: 00007ffb2395cc91: RtlUserThreadStart+33

I've traced it to line 491 of smoke.c:
if (smoke->age % g_SmokeTypes[smoke->type].spreadspeed == 1) {

It's a division by zero exception, which would imply that g_SmokeTypes[smoke->type].spreadspeed somehow becomes 0.
Tested using 9c4aded. x86_64 build.

 Very likely the former.

Can confirm I'm seeing this smoke on dataDyne Defection, after going down the elevator and killing the guards (before I entered the security room to place the ECM mine). x86_64 build


Add native support for Golang, Python, and JavaScript Azure SDKs

Although we have the ability to make HTTP calls to the Azure service endpoints, it would be much easier to be able to import the various SDKs directly.

Another possible strategy is to implement the wasi-cloud-core interfaces e.g. https://github.com/WebAssembly/wasi-blobstore.  Cf https://github.com/WebAssembly/wasi-cloud-core

Is that more or less what was done here, but for KV? https://github.com/fermyon/spin-go-sdk/tree/main/kv

Sort of, although that is a custom interface (we have a PR #2486 for wasi-keyvalue but it's stalled on the world rev stuff).  But yes, you can back that onto a hosted Redis, or onto Azure Cosmos DB.  (And if there's a specific AWS service we should back it into, then do raise an issue.)  And when #2486 lands, it will inherit all those back ends.


Redirect not working

I'm using this library together with the Go oauth2 library. The final part of the authorization process redirects to a localhost:port where a local webserver is waiting for the authorization code. However, the webview doesn't seem to be able to show the redirected url; it simply freezes (even though my local webserver receives the response just fine).
I should mention that the same issue happens with the go-astilectron library, so perhaps it's something that occurs with many webview implementations. With lorca, redirection works fine, but I would prefer using this library due to its speed and window control.
OS: Windows 10 pro 64 bit
Language: Go


Import error on trajectorama

import trajectorama
/usr/local/lib/python3.7/site-packages/anndata/_core/anndata.py:21: FutureWarning: pandas.core.index is deprecated and will be removed in a future version.  The public classes are available in the top-level namespace.
from pandas.core.index import RangeIndex
Traceback (most recent call last):
File "", line 1, in
File "/usr/local/lib/python3.7/site-packages/trajectorama/init.py", line 1, in
from .trajectorama import *
File "/usr/local/lib/python3.7/site-packages/trajectorama/trajectorama.py", line 9, in
from .pan_dag import PanDAG
File "/usr/local/lib/python3.7/site-packages/trajectorama/pan_dag.py", line 11, in
from ._louvain import louvain
File "/usr/local/lib/python3.7/site-packages/trajectorama/_louvain.py", line 10, in
from scanpy import utils
ImportError: cannot import name 'utils' from 'scanpy' (/usr/local/lib/python3.7/site-packages/scanpy/init.py)

Thanks for the issue! What version of scanpy are you using? The version in our tests (version 1.4.5) seems to support from scanpy import utils.

Thanks. I'm working with python3. It works when I do the following:
pip3 install scanpy==1.4.5


Need to expose server flag

The data export tool defaults to the https://www.commcarehq.org production environment. This can be changed by using the commcare-hq flag. We need to expose this to be able to use the tool from different environments

Fixed in #4


Include md output: tenant commands

Related to #3754

Update the documentation to include the md output for all the commands under the service:

tenant documentation location

You can reference the commands planner plan and planner bucket. They are already contain the Markdown reponse in their documentation.
For more information what guidelines to follow, be sure to check out the epic issue #3754

Some of the tenant commands return responses that are not in markdown format. For example:
m365 tenant report activeusercounts --period D7 --output md

Report Refresh Date,Office 365,Exchange,OneDrive,SharePoint,Skype For Business,Yammer,Teams,Report Date,Report Period
2023-04-18,,,,,,,,2023-04-12,7

I believe this issue should be addressed separately. For now, I will update the documentation to include the current "as-is" md output.

This behavior is pretty common since the command is quite unique. There are some other commands like this one called Period Reports which only give two types of output: a JSON or a CSV response. So, you just need to include your output in the Markdown section similar to what we did with the m365 teams report deviceusagedistributionusercounts command as shown here

Perfect, I will do that. thank you.


were all methods found then?

https://github.com/RCinbo/IPBESValueAssessment/blob/29ec49fb7d6831b21d5a9b2b886f2b798b051f22/Method _list.R#L162
because this one returns an empty list?
or am i doing something wrong?

If you run line 165, there are 74 methods that are not found..

no i get

method_lutb$Method.name[method_not_found_idx]
factor(0)
121 Levels: Afrocentric methodologies ... Worldview assessment
length(method_lutb$Method.name[method_not_found_idx])
[1] 0


feat: Joints implementation

Summary
This PR introduces the Joints feature.
Joints are one of the advanced features natively available in Rapier.
This PR provides an easy, basic but robust way to deal with it.
Logs

Support for all the supported Joint types

"fixed" | "spring" | "rope" | "generic" | "spherical" | "prismatic" | "revolute"

New components based on the supported joint types (FixedJoint, SphericalJoint, ...)

How to use
Here's a basic implementation example:
<RigidBody
  v-for="(ref, i) in bodyRefs"
  :key="i"
  :ref="ref"
  :type="i === 0 ? 'kinematic' : 'dynamic'"
  :position="[i * 1.5, 0, 0]"
   collider="ball"
 >
  <TresMesh>
    <TresSphereGeometry />
    <TresMeshNormalMaterial />
  </TresMesh>
</RigidBody>

<SphericalJoint
  v-for="(ref, i) in bodyRefs"
  :key="i"
  :bodies="[ref.value?.[0]?.instance, bodyRefs[i - 1]?.value?.[0]?.instance]"
  :params="[
    [-1.1, 0, 0],
    [1.1, 0, 0],
  ]"
/>

Preview

https://github.com/user-attachments/assets/e30dc274-597f-4903-bcc6-4060711e5494
TODO: implement a decent documentation

Will close #108

Hi @alvarosabu, I just added a base documentation for Joint.
I'd appreciate having your review of it (the Joint implementation in general).
cc: @JaimeTorrealba


types(ui/skeleton): add BasicAttributes

Checklist

[ ] Fix linting errors
[ ] Tests have been added / updated (or snapshots)

Change information

Codecov Report

Merging #637 (d88b9ee) into dev (7d421bc) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##              dev     #637   +/-   ##
=======================================
  Coverage   95.41%   95.41%
=======================================
  Files         217      217
  Lines        5255     5255
  Branches      982      982
=======================================
  Hits         5014     5014
  Misses        232      232
  Partials        9        9

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 7d421bc...d88b9ee. Read the comment docs.


#1194 Add missing unit test for twilio

Description
These changes add and update unit test for the Twilio callback task.  One test, marked xfail, uncovered a bug to be addressed by #1217.
#1194
Type of change

[x] New feature (non-breaking change which adds functionality)

How Has This Been Tested?

[x] All Unit Test Pass (except 1 new xfail)

Checklist

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[x] My changes generate no new warnings
[x] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes (except 1 xfail)
[x] Any dependent changes have been merged and published in downstream modules

Additional Info & Resources
Uncovered related issue: #1217

Please add a label to this PR (like, bug). Otherwise, LGTM.

I added the "bug" label.


ESP32 C3 compatibility

Hi,
Thanks for this nice project!
i've installed a Wemos C3 Mini on top of the pcb. This compiles and seems to work somewhat with the config below. I can connect it to home assistant, but i don't see any values, only ' unknown'.
D1 dsmr alias d5 = gpio 14  >> gpio1 on C3
D1 uart alias d7 = gpio 13  >> gpio4 on C3
Everything loks good, but no data :). Is there something obvious i might be missing? Or is there a way to validate esphome receives the data correctly from the P1 port?
substitutions:
  name: dsmr-reader
  friendly_name: DSMR Reader

esphome:
  name: ts27meter
  platformio_options:
    board_build.flash_mode: dio

esp32:
  board: esp32-c3-devkitm-1
  framework:
    type: arduino
    version: 2.0.3
    platform_version: 4.4.0

logger:
  # Serial logging is disabled by setting the logger baud rate to 0.
  # Otherwise the logger will occupy the hardware UART, making it unavailable
  # for receiving smart meter data on pin D7 (GPIO13).
  baud_rate: 0
  level: VERBOSE

# Enable Home Assistant API
api:
  password: "xx"

ota:
  password: ""

wifi:
  ssid: "xx"
  password: "xx"
  output_power: "8.5"

uart:
  rx_pin: GPIO4
  baud_rate: 115200
  rx_buffer_size: 1500

dsmr:
  request_pin: GPIO1
  request_interval: 10s

sensor:
  - platform: dsmr
....

allright, that was a user-error - i used the wrong cable. THere's data going to Home Assistant now


No access to original route object data

In a normal react component I can access this.props.route and get access to the original route definition. With react-router-relay this.props.relay.route (or when using context this.context.route) does not contain any original data. It returns an object with the keys name, params, queries. And for example name does not contain the original route name but some autogenerated name like "_aggregated__form__default_root__form__default_offer__inquiries__default_root__inquiries__default_offer".
If this is not a bug but a feature - how to access the current orignal route definition object, just what a normal react component returns with @props.route? How to access the other router variables like @props.routes, etc.?

It's just this.props.route in the route component.

@taion Would be great, but it's not :-1:
Here's my component and relay container (coffeescript):
React = require("react")
Relay = require("react-relay")
{withRouter} = require("react-router")

Component = React.createClass
  contextTypes:
    router: React.PropTypes.object
    route: React.PropTypes.object

  render: ->
    console.log @props.relay.route.name
    console.log @props
    console.log @context
    console.log @props.route
    console.log @context.route
    div null,
      "bla"

Container = Relay.createContainer Component,
  fragments:
    offer: -> Relay.QL`
      ...
    `

module.exports = withRouter(Container)

The output in the console is:
Object {root: Object, offer: Object, children: Object, router: Object, relay: Object}
Object {router: Object, route: Object}
undefined
Object {name: "_aggregated__form__default_root__form__default_off…inquiries__default_root__inquiries__default_offer", queries: Object, params: Object}

As you can see @props.route is undfined and @context.route contains not the original data.
My router config for this config looks like:
{
  name: "form"
  path: ":id/edit"
  component: require("app/pages/offers/edit")
  queries: Queries("root", "offer")
}

When removing the relay container and passing the react component directly to the router it works as expected.
~# npm list | grep route
├──<EMAIL_ADDRESS>extraneous
├─┬<EMAIL_ADDRESS>├─┬<EMAIL_ADDRESS>

@taion Sorry for wasting your time, it seems it was a browserify caching bug here. After recompiling everything it seems to work...


Build for multi-arch

Ref docs:

Building Multi-Architecture Docker Images on ARM 64-bit AWS Graviton2 Processors

A quick memo, so far this project can be built on Apple Silicon (e.g. M1) without a problem.


[Suggestion] Upload Banner from website (link)

Having a function to post a link to a picture instead of upload one from the computer.

It might be great to consider integrating banner selection with SteamGridDB. They even have an api:
https://www.steamgriddb.com/api/v2

This has been resolved with #40 which implements:

Suggesting game names when adding a new game
Letting the user pick images from SteamGridDB if a suggested names was picked
Entering a URL to a custom image

@alkazar please close this issue.


selene tools deleted

Codecov Report

Merging #119 into master will increase coverage by 5.64%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master     #119      +/-   ##
==========================================
+ Coverage   58.11%   63.76%   +5.64%
==========================================
  Files          25       24       -1
  Lines        1170     1104      -66
  Branches       98       91       -7
==========================================
+ Hits          680      704      +24
+ Misses        465      376      -89
+ Partials       25       24       -1

Impacted Files
Coverage Δ

selene/abctypes/conditions.py
100% <ø> (+14.28%)
:arrow_up:

selene/common/delegation.py
100% <100%> (+8.69%)
:arrow_up:

selene/conditions.py
56.34% <0%> (+2.53%)
:arrow_up:

selene/support/conditions/have.py
77.41% <0%> (+22.58%)
:arrow_up:

selene/bys.py
100% <0%> (+44.44%)
:arrow_up:

Continue to review full report at Codecov.

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


Add installation through conda

Now that polars is available through conda, tidypolars can be added.

First of all, thanks for this excellent package!
I have created a PR adding tidypolars to conda-forge:  #https://github.com/conda-forge/staged-recipes/pull/20328
Would you mind being added as a recipe maintainer?

I have created a PR adding tidypolars to conda-forge

Thanks 🥳

Would you mind being added as a recipe maintainer?

That sounds good to me!
I'll be honest I'm not very familiar with how the conda-forge process works. I'll look more into it, but if there's anything you need me to do let me know.

Awesome - thanks again @nsandau 😄


Playing Forbidden Grimoire with Blue Candle costs 1 HP

Version
Slay the spire: BETA
Mod version: v0.47.2
Describe the bug
Playing the Forbidden Grimoire on Wanderer proccs Blue Candle and costs 1 HP.
To Reproduce
Steps to reproduce the behavior:

Have Blue Candle.
Play the Forbidden Grimoire.

Expected behavior
I expected this to not cost me HP.

Obsoleted by bcc9415 (makes Grimoire not be a curse)


[edn] Experiment commit to address disable corner case

"might" address #16152
Signed-off-by: Timothy Chen<EMAIL_ADDRESS>

can you give this one a try @cindychip ?

sounds good, should i run the whole regression? i cna't say for sure this doesn't break something.

sounds good, should i run the whole regression? i cna't say for sure this doesn't break something.

I am running the regression, so far nothing flagging.


initial typescript compile

I don't expect this PR to get merged. I just wanted to show you how easy it is to get the project to 'compile' with tsc. The idea being that the move to typescript can be gradual.
Just yarn; tsc and the output will go to a build folder.
Related: #318

@lookfirst Very nice! Thanks for putting this together.


Added rofi

There's a fork of rofi with Wayland support that works flawlessly

httops

You should probably squash it into one commit.

ok, closing this PR


fix: Update NAB status phase and condition together

Why the changes were made
As discussed with the team, we want to do less reconcile steps for NAB full reconcile loop (but keeping only making one update call per reconcile). With this PR, NAB reconcile update phases and conditions together in one update call.
Also, simplified Predicate and Handlers logic.
How to test the changes made
Run tests and check logs (make simulation-test) or test in a cluster following development documentation.

I tried creating a NAB backup using this PR and found some discrepancy in NAB status, it is showing phase as Created even though the VeleroBackup is Completed. Can you please double check for sanity ?
status:                                                                                17:33:00 [3/1874]
  conditions:
  - lastTransitionTime: "2024-10-02T00:31:42Z"
    message: backup accepted
    reason: BackupAccepted
    status: "True"
    type: Accepted
  - lastTransitionTime: "2024-10-02T00:31:42Z"
    message: Created Velero Backup object
    reason: BackupScheduled
    status: "True"
    type: Queued
  phase: Created
  veleroBackupName: nab-nacuser-e68ec47f50dc20
  veleroBackupNamespace: openshift-adp
  veleroBackupStatus:
    completionTimestamp: "2024-10-02T00:31:54Z"
    expiration: "2024-11-01T00:31:42Z"
    formatVersion: 1.1.0
    phase: Completed
    progress:
      itemsBackedUp: 71
      totalItems: 71

@mateusoliveira43 yeah correct, On one hand Created seemed a bit out of place to me (maybe just becasue not used to seeing it) but on the other hand we do not want to duplicate data from VeleroBackup. Anyway if we wanted to change, it would be a separate PR.

We have the veleroBackupStatus which reflects the state of the Backup.
On the other hand we could also think about 4th phase in NAB that will be Completed when the veleroBackupStatus also is completed without error.


Random code review

@maximpertsov per convo with Eric in slack, I think this is just not going to work during this PR since it's trying to grab the secret from my account and necessarily failing, but it should work once it's merged.


Dark mode broken in Brave browser

When attempting to use the extension in the Brave browser (1.71.118) I can't see any of the text:

A minor issue but one I thought I should flag so that it can be addressed. 🙂

Same for Chrome

Go into Google extensions and try clicking update extensions. It's been fixed.


Already a language named V

Isn't there already a language named V? See here.

https://github.com/vlang-io/V/issues/6

Found yet another one: a golfing language inspired by everybody's favorite text editor, vim.


Log spam with EnderIO non-fluid conduits passing next to GenericTank implementations

[16:38:49] [Server thread/DEBUG] [OpenMods/]: openmods.liquids.GenericTank.fillFromSides(GenericTank.java:154): Tank class crazypants.enderio.conduit.TileConduitBundle @ (-797,63,795) returned null tank info. Nasty.
Block using GenericTank:  OpenBlocks sprinkler.
Screenshot of how it is laid out in world:

Just to confirm, you have an item and energy conduit but no liquid conduit yes?

There is no liquid conduit in that block, no. The sprinkler in question is pulling its water from a pressure pipes water source below it.

Can you confirm that the block you are looking at in that screenshot is in fact (-797,63,795)
Debug mode (F3) will tell you the location of the block you are looking at.

From what I can tell this is caused by
https://github.com/SleepyTrousers/EnderIO/blob/master/src/main/java/crazypants/enderio/conduit/TileConduitBundle.java#L773
As well as a lack of Null check at our end.
I do believe the returned array should be empty, not null, when a tank is not available.
A null check at our end will fix this though.
Looking at the API here:
https://github.com/MinecraftForge/MinecraftForge/blob/master/src/main/java/net/minecraftforge/fluids/IFluidHandler.java#L73-L81
Does not say if null is valid or invalid for return. I can only assume it's invalid.

Oops, we do the null check, that's exactly what we are seeing here.
https://github.com/OpenMods/OpenModsLib/blob/master/src/main/java/openmods/liquids/GenericTank.java#L153-L157
Our end is working as expected.
Raise an issue with the EnderIO team regarding returning an empty TankInfo array instead of null.
If they insist that null is valid, I will just silence the console output at our end. Nothing is overly broken in OpenMods by this issue, and the library is only warning you that EnderIO might be misbehaving.

I've popped open an issue with MinecraftForge.
https://github.com/MinecraftForge/MinecraftForge/issues/2085
When they clarify the API's intended use, OpenMods will follow that. EnderIO will have to follow.


What is our quest?

Define what we actually want to do and set up milestones for the project.

To learn how to do a process


Correct example command option

Replaced --show-lib-versions with --show-package-versions in the command examples.

Nice catch!
Thank you.


List Media never returns when there is a corrupted image

I'm using await album.listMedia(); on my main landing page so the image is loaded before the splash screen goes away. If there is a corrupted image in the album this never returns and the app never loads.
The corrupt image is a me problem but its not unlikely this could happen to a user and they would not be able to get themselves out of this state. We need to handle this edge case gracefully.

More testing shows its not based on a corrupt image. Not sure what triggers the behavior but it pretty much kills any usability of the package.

I am experiencing the same issue. However, seems it triggers on certain devices. iPhone 11 and iPhone 14. No logs, so no idea why this is happening.  Can we get a fix on this

I have also encountered the same problem. On some Android phones, such as Xiaomi Note 9 (Android 9.0), the following code does not return, there are no logs or errors, and it seems like the code has stopped:
 final List<Album> imageAlbums = await PhotoGallery.listAlbums(       mediumType: type,       newest: true,       hideIfEmpty: false,     );     Album? album =         imageAlbums.firstWhereOrNull((element) => element.id == '__ALL__');    final MediaPage page = await album.listMedia(         skip: 2,         take: 10,       );
This is not a permission issue, as it is also about obtaining albums. If you do not paginate but instead obtain all of them, you can obtain them. The following code can return normally:
 final MediaPage page = await album.listMedia();
This matter is too mysterious. Could the author please help to take a look

I have also encountered this situation. Have you resolved it?


[Feature Request] Multiple sidebar with deeply nested subfolder

Hi,
I don't think this is currently possible.
But our documentation is very extensive and has many folders and subfolder.
To make it easier to navigate, it would be nice if we could have a sidebar for each subfolder instead of collapsible sections.
Here is an exemple:

Hello! Thank you for using vitepress-sidebar.
This plugin implements functionality based on the limits provided by VitePress.
If you want to simplify your menu display by showing different sidebar menus based on specific paths (URIs), i recommend learning about Multiple Sidebar feature.
This is explained in detail in the documentation below for you to implement in your own plugin:
https://vitepress-sidebar.cdget.com/advanced-usage/multiple-sidebars-how-to
If you're still having trouble, i'd appreciate a more detailed description of your requirements.
Regards,

@jooy2 Yes, I already use this feature, but I need to manually add every directory to scan, and this won't add the directories inside the directories (reacusively).

Here is a a visual exemple of what I want:


I want to to print linked, github name insted of whole link on pdf. How can I do it?

\github{s0sharma} should render <gh icon> s0sharma with a clickable link to https://github.com/s0sharma.
Is that not what you're seeing?

This works as expected on my own CV, so closing for now - feel free to ping me/reopen if it's not working as described above for you though.


Build a reusable API Response object

User Story
As a ReportStream API user, I want responses to be consistent so that integrations are easy to implement.
Description/Use Case
The Website API Payload Proposal provides a structure that is not currently used on any endpoints. API users, when provided with documentation, will expect responses to match it.
Risks/Impacts/Considerations
Changing existing endpoints would break existing integrations.
Dev Notes
What you need to know
Acceptance Criteria

[ ] An object that can be used on any endpoints matching the structure on the proposal

Hey team! Please add your planning poker estimate with Zenhub @arnejduranovic @jack-h-wang @mkalish @thetaurean

Please add your planning poker estimate with Zenhub @victor-chaparro


Removed Solution Activity Log from LAW

Overview/Summary
Remove Activity Log solution from Bicep RI
Update and Bicep documentation referencing this
This PR fixes/adds/changes/removes

PR #414
Closes AB#25141

Breaking Changes

None

Testing Evidence
Executed the deployment and Activity Log Solution is not deployed.
As part of this Pull Request I have

[X ] Read the Contribution Guide and ensured this PR is compliant with the guide
[X ] Ensured the resource API versions in .bicep file/s I am adding/editing are using the latest API version possible
[X ] Checked for duplicate Pull Requests
[X ] Associated it with relevant GitHub Issues
[X ] (ALZ Bicep Core Team Only) Associated it with relevant ADO Items - AB#25141
[ X] Ensured my code/branch is up-to-date with the latest changes in the main branch
[X ] Performed testing and provided evidence.
[ ] Updated one or more of the following tests (if required)

Unit
Linting
E2E (End-To-End)
ValidateAzCloud (Base validation in Azure Cloud)
ValidateMcCloud (Base validation in Azure China Cloud)

[X ] Updated relevant and associated documentation (e.g. Contribution Guide, Module READMEs, Wiki Docs etc.)
[ ] If relevant, created or updated Code Tours here

/azp run validateazcloud

/azp run e2e


NCL-7849 Fix sorting and entity attributes

Hi @patrikk0123 , I'd like to ask you for code-review here. Thanks

Great, thanks


[Feature Request]: Add "Back to Top" Button

Is there an existing issue for this?

[X] I have searched the existing issues

Feature Description
The addition of a "Back to Top" button on the all pages of our website would enhance user experience by providing a convenient way for users to quickly return to the top of the page after scrolling down. This feature is commonly seen on many websites and has become an expected element for smooth navigation.
Use Case

Convenience: Quickly returns users to the top of the page with one click.
Improved Navigation: Simplifies moving back to the main navigation menu or introductory content.
Accessibility: Helps users with mobility issues or those using keyboard navigation.
Enhanced User Experience: Reflects attention to user-centric design, increasing satisfaction and retention.

Benefits

Time-Saving: Reduces the time and effort needed to navigate long pages.
User-Friendly: Makes the website more intuitive and pleasant to use.
Inclusive: Supports diverse user needs, adhering to accessibility standards.

Add ScreenShots
My previous work:
https://github.com/ashutosh-rath02/git-re/assets/146121869/8e00e36d-9ef4-4a42-9e24-81339394f07f
Priority
High
Record

[X] I have read the Contributing Guidelines
[X] I'm a GSSOC'24 contributor
[X] I have starred the repository

assign me this project

Hello @AsmitaMishra24! Your issue #278 has been closed. Thank you for your contribution!


EPIC: Auto mode

[x] support custom hints to improve automation
[ ] change prompt from :auto to '
[ ] implement errors from r2lang.cmd
[x] support chatml-function-calling via llama-cpp https://github.com/radareorg/r2ai/pull/4
[ ] support functionary v2 via llama-cpp
[ ] evaluate different models and compile a list of supported ones and figure out what prompts would work best for them

ref https://github.com/radareorg/r2ai/pull/4

i've added doc/auto/*.txt and integrated the indexer context message to make the local functionary know which commands to run and how to achieve certain things with r2. this way we can improve the local model auto interaction quite a lot. pleae give it a try because i was able to do several more things properly without gpt thanks to this :)
also, ive updated some models, and mistral2 supports 32K contexts.. so maybe we can use that to compress large disassemblies or decompilation outputs to get an improved output processing. Because the maxtokens and context window is another issue we face with local models. until llama gets improved to handle larger ones in a proper way we must find ways to workaround those limitations.
unfornately i cant use some 3rd party apis because im in europe and some AI vendors dont give access to us :__ so once again another reason to use local models

That's awesome, just tried functionary and it's actually usable with these simple instructions! I thought it'd be a bigger effort :)

To take into consideration for a another approach on auto mode without function calling https://github.com/user1342/Monocle

If everything looks good to u i would like to release 0.6 and publish it in pip. So we can start messing with other stuff from this stable point once again :) sgty?

sounds great!

In this topic, what do you guys think for this?
https://docs.phidata.com/introduction
I'm more interested in this:
https://docs.phidata.com/knowledge/introduction
The thought of having a way we can help the assistant know how to use radare excites me 😂

@nitanmarcel there's already a RAG in r2ai that can be used for that I think. I remember pancake got one working a few months ago.


Upgrade issue

I just tested upgrading from thinking-sphinx 3.1.4 to 3.3.0 and ran into a small issue.
In 3.1.4, I used to be able to filter based on an attribute passing it as a string. Something like: Article.with_communties(['1','3']).search('something')
After the upgrade, sphinx started complaining about unexpected QUOTED_STRING. I can easily fix this by doing Article.with_communities([1,3]).search('something') (passing array of integers instead of array of strings).
Was there an intentional change made to cause this?

Ah, this is actually related to a change in the Riddle gem (which generates the SphinxQL queries). The upcoming TS release (which is the commits in the develop branch) tries to cast filter values to their appropriate types, but in the long run I definitely want to remove that - it's just a stop-gap solution to make the transition easier.
The change has been made because Sphinx now allows for filters on string attributes (whereas previously they'd been only for grouping/ordering), and so the SphinxQL generation needs to pay closer attention to types - it can't presume that every string value passed in should actually be treated as an integer (Sphinx treats booleans and timestamps as integers, hence why it was a safe assumption previously). This means that Riddle will put quotes around string values in the query, and this is why you're getting the QUOTED_STRING error.
I'll try to get a new release of TS out soon, but your options in the meantime are either: use the latest in the develop branch (here's the diff: https://github.com/pat/thinking-sphinx/compare/v3.3.0...develop - mostly testing changes and bug fixes); or, review your code and ensure you're passing in appropriately typed values for any attribute filters.

We are in a position to ensure the types are correct for the filter values, so we will likely be moving forward with an upgrade to 3.3.. I just wanted to understand a little about why things were changing, so this helps.
Thanks!


style: apply clippy lints

This PR applies several suggestions from clippy, the built-in Rust linter designed to "catch common mistakes and improve your Rust code". Running cargo clippy on main currently produces 66 warnings, after this PR there are 0. Most of these suggestions are about style, so please feel free to only accept some or none of this PR if they don't align with your coding style! I can remove any commits you don't want to merge, just let me know.
Each lint is in its own commit. You can search up each of the lints on the clippy lints website to see why clippy suggests them.
This compiles and I did a basic profile download without any issues, but I didn't test more thoroughly than that.
If you want to enable clippy hints in your dev environment, in VS Code, you can go to settings and set rust-analyzer.check.command to clippy, for helix you can follow these instructions, otherwise check the internet :)

Thanks for this! I'm new to rust so this taught me some good behaviors I should be using in my rust code going forward.
I think the only lint item I don't really want is needless_return - I think it's better to explicitly indicate when you are returning, at least for normal functions - only really makes sense to omit for closures, etc. IMO.
The rest I'd be happy to take. I can do some testing later - I've got a handful of accounts for that purpose.
It also looks like it's possible to update main.rs to include an entry that suppresses a specific clippy warning for just the "needless_return" type so the number goes to zero. If this is required above every use of return obviously don't bother haha.
I will definitely be turning this on in VS Code going forward - again, thanks, I didn't even know it existed. I wonder if I can even set a rule that all clippy warnings must be resolved before successful PR.... though maybe that's overkill.

It also looks like it's possible to update main.rs to include an entry that suppresses a specific clippy warning for just the "needless_return" type so the number goes to zero.

Done in latest commit, and I deleted the commit that removed the returns :) So cargo clippy disregards needless_returns and has 0 warnings!
Yeah, I wish the IDEs defaulted to using clippy, I'm pretty new to Rust too and stumbling into clippy has been invaluable in learning new ways to do things.

I wonder if I can even set a rule that all clippy warnings must be resolved before successful PR.... though maybe that's overkill.

The clippy documentation has an example Github Actions file, and there's some actions like actions-rs/clippy-check which seem to be nice :) Haven't tried them myself, but doesn't seem too bad if you wanted to enforce it.
And I totally forgot to say this in my initial message, but thank you for writing LASIM! It's been extremely useful in backing up my account and migrating between instances, very grateful for it and happy to contribute :)


Update the README

Add/update details on the README for this project. Feel free to be creative, we want it to be bright, creative, and exciting! I've been kind of following pandas README for inspiration but other ideas are absolutely welcome
Some ideas to tackle:

[ ] update Features section to be more comprehensive
[ ] add more images/gifs
[ ] short snippets/examples
[ ] Dependencies section
[ ] badge(s) that add something meaningful to understanding this repo
[ ] emojis..... lots of emojis
[ ] etc!

Really open ended, feel free to comment/submit a PR with ideas that you feel would enhance the README for this project

Hi, this is my first time contributing to open source. Can you assign me this?

[ ] update Features section to be more comprehensive
[ ] add more images/gifs
[ ] short snippets/examples
[x] Dependencies section
[x] badge(s) that add something meaningful to understanding this repo
[x] emojis... lots of emojis
[x] a typo

I didn't make many significant changes, just added a dependencies section with the pip commands to install each library, with a few small tweaks. Libraries taken from dependencies.txt

Creating a PR now...

PR created, will sit tight and do other stuff.

Wrote a demo program that looks at the profile data of Google and puts it into a CSV using Pandas. Would you like me to add its source to the readme as a supplement to the profile section under features?

I love to hear it!!! There's actually an examples folder that I think this would be perfect for if you want to add it as a dedicated file into that directory! Thanks so much for the contributions ❤

You're welcome, I'll make a pull request with the file uploaded. Feel free to look at the source; I had to do it in multiple steps in order to achieve the end goal

PR created! 😁
I love this project and will continue to contribute in any way I can!

 I'm not too worried about enforcing black styling right now so if you don't do it on your PR's, no worries
I'm honestly really glad to hear you like the project though lol, this is my first open source and I'd love for people to get use out of it

Might make another screenshot / script analyzing a random hashtag and plotting it using MatPlotLib. Stay tuned!

Me, realizing that already exists: 😂

@beckettnormington Speaking of hashtag scripts though, I have a real world hashtag analysis program that uses instascrape here if you wanna check that out. The plan is to launch it on PythonAnywhere but that's been sidelined until I can figure out how to solve Issue #9 lol but it works great otherwise if you want to tinker with it

Alrighty, will check that out. Thanks again! ❤

can I help with this issue?

@PriyanshTri go for it! Feel free to submit a PR and I'll give it a look 😄

I'm gonna be working on making an IPython version of the CSV Scraper.

I made a badge that might interest you. Details: https://github.com/chatter-social/chatterware

Anything that I can improve?

Anything that I can improve?

@beckettnormington I'm doing a massive update that's going to greatly improve the library but introduce a ton of breaking changes so I wouldn't want any work you do to be outdated almost immediately lol, check back in a day or two though and I'll let you know!

Alrighty, thank you!

Might update README's Table of Contents by removing the link tot he non-existing dependencies section and send a PR.

Might update README's Table of Contents by removing the link tot he non-existing dependencies section and send a PR.

@beckettnormington lolol dang forgot to update that, good catch. Could you create a Dependencies section in between Documentation and Contributing that contains links to BeautifulSoup and Requests docs? That'll emphasize that this is a lightweight library for static HTML scraping only and it'll indicate what's primarily under the hood

Sure thing, I'll try to get to that tonight or tomorrow.

Sure thing, I'll try to get to that tonight or tomorrow.

Sounds good, very appreciative of your help! 😄

Attempted to add dependencies section. PR: #33

@chris-greening what more things you want in README.md, I can do it for you.

!Suggestion we can Line of Code (LOC) with python : 98.8%

@chris-greening what more things you want in README.md, I can do it for you.

@Siddhant-K-code It's really open ended! I'm just looking to expand it to make it as comprehensive and exciting as possible to the layman to get people interested in the project
maybe add a Features section with short intro's to the Profile, Post, and Hashtag objects? Let me know what ideas you have!! I'd love to hear them

Anything I can do to help improve any existing materials or new things?

Anything I can do to help improve any existing materials or new things?

If there are any ideas you have for new features, feel free to open a new issue to get a dialogue going! I think the README is good for now so I'm gonna close this issue and open some more over the next few days with some ideas I have for improvements so keep an eye out :smile:
Probably gonna write some more blog posts and refactor some stuff so there'll also be that to look for as well  :smirk:


Conda build recipe not working

Hi,
I tried to use the conda recipe to install the notebook extensions on my system.
I'm on windows 7. My conda version is 3.17.0. I downloaded the Ipython-notebook-extensions as a zip file from the github repository. I extract the zip. From the command line I navigate to the directory and write:
conda build meta.yaml

i get the message:
Ignoring non-recipe: meta.yaml

Am I doing something wrong or is there a problem with the recipe?

Just found out I need to be outside the directory and do:
conda build <directory>