feat<events.cognito>: add support for custom email sender

Issue #, if available:
https://github.com/aws/aws-lambda-go/issues/370
Description of changes:
added new event struct for support of customEmailSender messages.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Codecov Report

Merging #419 (1125b60) into main (96ab249) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #419   +/-   ##
=======================================
  Coverage   71.66%   71.66%
=======================================
  Files          19       19
  Lines        1041     1041
=======================================
  Hits          746      746
  Misses        228      228
  Partials       67       67

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 96ab249...1125b60. Read the comment docs.

LGTM, do you have an AWS docs link for this feature that I can reference?

Can we cover custom SMS sender as well?

Sure, I will add the same for SMS, I have few time available atm, but asap.
@bmoffatt I'll add the link from documentation

Hey, do you have any updates on this? @BorisLeMeec, do you need any help?

Is this change still happening? Running into the need for the Email/SMS event models in a project I'm working on

Hey, @bmoffatt is there an update regarding the support for the custom email sender event? Are there any blockers for merging this?

Bump, any updates on this one? Anyone uses a workaround?


Add --yes apt install commands

https://github.com/MiczFlor/RPi-Jukebox-RFID/blob/7e12d45adfa566d80761af4adf3c87c5ecd21aa7/scripts/installscripts/stretch-install-spotify.sh#L459
This could be more "automated" when adding --yes to the command, as this automatically accepts the apt packages to be installed. (also relevant for the other install scripts as well)

Hi @lied
this suggestion flatters me, because you seem to trust the install process :) I will keep this open and hope the day will come where I happily add this.

@MiczFlor hm, I don't get it, what bad could happen? Either it fails anyway after manually accepting the apt changes or it fails automatically :)

+1 or add a --unattended option to the script so the user can choose to trust or not

I added this to the code.


Beta12 issues

Not sure what's happening, I get random failures on opening editor (fixed by purging .godot cached folder), no documentation for D classes, crashes on exit...

This was probably either a bugged .godot project cache, or godot merges out of order specifically db7d8c2d87a9.
Closing this as beta 14 seems to be ok now.


Add a initialization toggle to the Helm chart #3771

This fixes #3771

LGTM, Thanks for implementing @wromansky .


Autostart .py file

Hey again, short question:
what does the Face-Rec-SMAI.py file start?  Is it a cronjob?
Greetings Toby :)

There are probably many ways to do it. But I solved It by using pm2 (also user for MM to autostart). And use a sh script started by pm2 containing the following line:
DISPLAY=:0 python3 ~/MagicMirror/modules/MMM-Face-Recognition-SMAI/MMM-Face-Recognition-SMAI.py

OK, make sense.
Is this the .sh script?

If I look at it with "sudo nano run-face-rec.sh" I only can find this:

My thing is:
I want to stop the Face-Rec for 20 secounds to be able to use a Selfie-Module. The cam is not able to do the 2 modules at the same time. Can you please help me bro?
Toby :)

@opdoffer as u can see in my post above there is an run-face-rec.sh file, but nothing is in there. The face-reg.py starts with my MM (PID 631) but now by the run-face.sh file (think so :o )

@EbenKouao who executes the command "python3 ~/MagicMirror/modules/MMM-Face-Recognition-SMAI/MMM-Face-Recognition-SMAI.py" ? thx

I solved it like this, dunno its this is the correct way but it works for me.

Create a file and give it a name whatever you like:
sudo nano ~/face-rec-autostart.sh
Add the following line
python3 ~/MagicMirror/modules/MMM-Face-Recognition-SMAI/MMM-Face-Recognition-SMAI.py
Hit CTRL-X the n Yes and enter to save the file
Change the flights to execute permissions:
chmod +x ~/face-rec-autostart.sh
Add this script to pm2 with the following command:
pm2 start ~/face-rec-autostart.sh
Save the pm2:
pm2 save

reboot and try
In case you want to use the camera you can stop the face-rec-autostart script with the following command:
pm2 stop face-rec-autostart

@opdoffer Thank you man! that would be great for me! But in my image the module was pre-installed and there is somewhere an ecisting thing that starts the .py file... and I dont know how I can stop it (only with "kill [PID] )
If I create a file like you, the .py script would be executed twice, doasnt it?

Ah so. I installed all of it manually. You used the image. Sorry my bad.
Yes if you create a file like I did then it will be executed twice and the second one will fail. Hopefully Ebenkouao will answer your question soon.
I will try the pre-installed version soon and if I found how it is configured I will let you know.

@opdoffer ok thank you, If you will find something with the pre-installed image let me know please <3

@EbenKouao I just want to deactivate the autostart of face-recognition-SMAI.py ... can u help me?

Hi @thetobyde @opdoffer,
Thanks for trying it out.
In order to deactivate the autostart of face-recognition-SMAI.py (in LXTerminal at boot)
Go to the Pi directory, Right-click, show hidden files, and open up .bashrc
sudo nano /home/pi/.bashrc
Remove the line at the bottom. This is what autostarts at boot:
sudo python3 /home/pi/dlib/build/face_recognition/examples/MMM-Face-Recognition-SMAI.py
That should avoid the camera module python script conflicts. Let me know how it goes.
BTW, you can customize the python script face recognition, to include multiple profiles. As of now, it's limited to one.
Eben.

Thnx for the info Eben. And btw great work you’ve done! It is much fun.
Leaves me one question: when is the one user limitation resolved?

Hi @opdoffer, the 'one user limitation' has been resolved. You will observe that your face can be differentiated from others. You can now develop multiple profiles as you wish. Let me know how it goes and if you develop the module further, feel free to commit to the git!
Eben.


Possible memory leak on dhcp server (IDFGH-4594)

I'm running an project where ESP32 behaves as wifi station and AP. From time to time I get an entry on log like this:
dhcps: send_nak>>udp_sendto result 0

After that heap looses about 60 bytes that are never recovered.
As I'm using Arduino core I do not have access to dhcpserver.c code so that I'm not able to do any test on it. Please, check if code on https://github.com/espressif/esp-idf/blob/master/components/lwip/apps/dhcpserver/dhcpserver.c#L553 may cause this leak.

@gmag11 Thanks for the report, indeed there's a theoretical chance of allocating 20 bytes (struct dhcps_pool + struct _list_node) which were not free'd immediately after DHCP server replies with NAK. These structures are used to hold information about the client (that has requested another address than offered). The data are kept in the internal list of client information and reused when the same client (with the same MAC) reapplies for a lease. I think this could cause some memory leaks only if we repeatably request different IPs from clients that constantly change MACs.

@gmag11 Thanks for the report, indeed there's a theoretical chance of allocating 20 bytes (struct dhcps_pool + struct _list_node) which were not free'd immediately after DHCP server replies with NAK. These structures are used to hold information about the client (that has requested another address than offered). The data are kept in the internal list of client information and reused when the same client (with the same MAC) reapplies for a lease. I think this could cause some memory leaks only if we repeatably request different IPs from clients that constantly change MACs.

I'm posting a patch which modifies the behavior. Could you please check if this the issue you're seeing? (perhaps you can use the arduino lib builder to test in with Arduino?)
file:///home/david/esp/idf/0001-lwip-Fix-DHCP-Server-to-remove-clients-record-if-req.patch
The current status seems to be in line with the RFC (https://tools.ietf.org/html/rfc2131#page-16):
A server MAY choose to mark addresses offered to clients in
DHCPOFFER messages as unavailable.

And at the same time, the record is stored and reused when needed, so for most usecases it just saves the heap from alloc/free cycles, and it doesn't in fact leak the memory, as there's a mechanism to discard leases.

I'm posting a patch which modifies the behavior. Could you please check if this the issue you're seeing? (perhaps you can use the arduino lib builder to test in with Arduino?)
file:///home/david/esp/idf/0001-lwip-Fix-DHCP-Server-to-remove-clients-record-if-req.patch
The current status seems to be in line with the RFC (https://tools.ietf.org/html/rfc2131#page-16):
A server MAY choose to mark addresses offered to clients in
DHCPOFFER messages as unavailable.

And at the same time, the record is stored and reused when needed, so for most usecases it just saves the heap from alloc/free cycles, and it doesn't in fact leak the memory, as there's a mechanism to discard leases.

Hi, I'll try to setup Arduino lib builder although I've never used it.


fix(theme-tokens): dark and light tokens fix

Description
Fixing a few things that I found wrong in the theme tokens files

Is this PR necessary? these tokens should be fixed via design changes, as it couldn't impact their flow if we merge any changes to files within packages/@momentum-design/tokens/src and prevent them from pushing.

This is being addressed in an earlier pr, closing it.


Wrong network used for choosing the action

Hi,
I believe, that the target actor network should be used for choosing the actions. This network is more stable as it is soft-updated, and this is the usual choice in DQN algorithms if the target network is used.
Even though it is inactive for 2 years, I raise this issue, because this repo is the first result in my search engine, when given the DDPG phrase, so I guess some people might use it as a reference implementation.

I now see that there are no target networks here! Is this even considered while talking about DDPG? I though this is one of the required features.


Integrate Model Registry issues into ODH dashboard

Need to integrate Model Registry issues in the ODH dashboard. For that automatically take place we need to add the following YAML file in the repo

Maybe we should first create the Model Registry Dashboard project here https://github.com/orgs/opendatahub-io/projects?query=is%3Aopen as I see there are project specific for each area, wdyt?

@rareddy let us know which Dashboard (by providing link) or which new to be created then we can create PR with the file

will proceed by creating a Dashboard as Andrea mentioned, will sink with Serving if there is any "cross-dashboard" needed.

https://github.com/orgs/opendatahub-io/projects/49 it is private now looking to how I make it public

Can we update the file details like this here https://github.com/opendatahub-io/odh-dashboard/pull/1899/files see how step one generates the token in there and used by the project in next step

@rareddy updating as requested in https://github.com/opendatahub-io/model-registry/pull/56


Request Failed 400 on initial integration

Hi,
I've read through some of the issues and I know "Request Failed with status 400 - Bad Request" is a common indicator that GM changed something or just an intermittent server issue. I've seen issue 233.
I'm setting this up for the first time, using new config values though, and I'm not sure if the issue is in my config, or the connection to onstar. Is there a way I can tell? I've ran the docker a few times in the last 12 hours. I'm pretty sure the username/password/pin/vin are right. I'm not 100% sure my mqtt settings are right (port/tls, for example).
Do you have any reason to think it would be down today and yesterday? I did get an email saying they were implementing MFA soon.

My instance stopped working as well. Not sure when exactly since I was gone for a week.
Not sure where to find any logs.

docker logs onstar2mqtt will give you the text log from npm. There is personal information in it, so be careful posting any of it.

Same issue here as well - it was working fine for a few months and stopped working on or around 2/21/24 for me as well. In looking at the JSON response I do see:
"data": {      "error": "invalid_client" },
so perhaps GM has locked down API access more tightly.
Has anyone found an updated library or a workaround?

I forgot to report back. The BigThunder version is working well for me. Onstar has its limitations. But this integration is impressive. I ended up installing it as an add on.

I forgot to report back. The BigThunder version is working well for me. Onstar has its limitations. But this integration is impressive. I ended up installing it as an add on.

Just read back in the thread and BigThunder is working for me as well. Thank you @BigThunderSR!

Pushed a build with the latest OnstarJS

Pulled latest and im getting the error 400 bad request


Fix empty server certs being loaded. Adds C++ header installation

Not sure if the C++ headers not being installed was meant to be there - can't see why it would be. A /usr/local/ install now works as expected for me.
But mainly the fix ensures that string views of zero length strings are not used inadvertently.

The C++ headers are not going to be distributed with the product currently, so they are missing on purpose, but the other piece of this PR is valid.

Reverted the CMake changes. Just the fix now.
Timelines on the C++ headers being "official"?

It has not been considered yet.  But if you are interested in it then @rajagp will probably want to know.


Creating a set does not update affected records

Reproduce

Install Invenio v3.2 (https://inveniosoftware.org/products/framework/)
Login as an admin (i.e. create an account and assign admin role)
Create a new OAI set via the administration interface.

Problem
Creating the sets, starts a background task in celery which then seems to be rate limited. See error below.
[2020-02-17 10:41:54,759: INFO/MainProcess] Received task: invenio_oaiserver.tasks.update_affected_records[73318bac-d8a3-4501-a17b-d53652826b8b]
<IP_ADDRESS> - - [17/Feb/2020 10:41:54] "POST /admin/oaiset/edit/?id=1&url=%2Fadmin%2Foaiset%2F HTTP/1.1" 302 -
[2020-02-17 10:41:55,065: ERROR/ForkPoolWorker-4] Task invenio_oaiserver.tasks.update_affected_records[73318bac-d8a3-4501-a17b-d53652826b8b] raised unexpected: TypeError('must be str, not NoneType',)
Traceback (most recent call last):
  File "/Users/lnielsen/envs/my-site-ZNC6mTtx/lib/python3.6/site-packages/celery/app/trace.py", line 382, in trace_task
    R = retval = fun(*args, **kwargs)
  File "/Users/lnielsen/envs/my-site-ZNC6mTtx/lib/python3.6/site-packages/flask_celeryext/app.py", line 100, in __call__
    self.app.flask_app.preprocess_request()
  File "/Users/lnielsen/envs/my-site-ZNC6mTtx/lib/python3.6/site-packages/flask/app.py", line 2241, in preprocess_request
    rv = func()
  File "/Users/lnielsen/envs/my-site-ZNC6mTtx/lib/python3.6/site-packages/flask_limiter/extension.py", line 511, in __check_request_limit
    six.reraise(*sys.exc_info())
  File "/Users/lnielsen/envs/my-site-ZNC6mTtx/lib/python3.6/site-packages/six.py", line 703, in reraise
    raise value
  File "/Users/lnielsen/envs/my-site-ZNC6mTtx/lib/python3.6/site-packages/flask_limiter/extension.py", line 494, in __check_request_limit
    self.__evaluate_limits(endpoint, all_limits)
  File "/Users/lnielsen/envs/my-site-ZNC6mTtx/lib/python3.6/site-packages/flask_limiter/extension.py", line 366, in __evaluate_limits
    limit_key = lim.key_func()
  File "/Users/lnielsen/envs/my-site-ZNC6mTtx/lib/python3.6/site-packages/invenio_app/limiter.py", line 19, in useragent_and_ip_limit_key
    return str(request.user_agent) + request.remote_addr
TypeError: must be str, not NoneType

@lnielsen with the new dynamic sets, this can be closed right?


OpenMined Course 1 Lesson 2 "Conflict, Political Science & Information Flows" Missing A Clip

Description
This video from 11:53 to 11:58 misses a clip of a real-world example of a 3rd party facilitating the mediation to avoid nuclear actions from India and Pakistan.
How to Reproduce
Check out the video.
Expected Behavior
The example from the guest lecturer should be inserted.
Screenshots
N/A
System Information
N/A
Additional Context
N/A

@em-blue @mcleonard

@em-blue @mcleonard

@iamtrask

@iamtrask

Thanks for the report @nahuakang ! Unfortunately, this is an issue with content. I'm going to ask that you instead post this on the courses discussion board, while I close this issue. I'd like to reserve Github issues to strictly be technical in nature.
https://github.com/OpenMined/courses/discussions

Thanks for the report @nahuakang ! Unfortunately, this is an issue with content. I'm going to ask that you instead post this on the courses discussion board, while I close this issue. I'd like to reserve Github issues to strictly be technical in nature.
https://github.com/OpenMined/courses/discussions


How to train for the case of a real life object and blender-like camera setting

hello thanks for the sharing~!
im trying to train barf with my custom dataset. I dont know the camera intrinsics and
the camera is located at 3 position(0, 30, 60 degree) spherically and an object is on the turn-table and took pics of it every 15 degrees.
so i have 72 pics ((pitch 0 yaw 15, 30, 45,...,360) and (pitch 30 yaw 15, 30, 45,...,360) and (pitch 60 yaw 15, 30, 45,...,360)).
so, even the camera is fixed per each pitch, its similar as blender style camera movement.
Barf support blender and llff but i failed to provide applicable camera pose information(.json or npz) as blender or llff dataloader.py need. so i tried to use iphone configuration.
i seems to train at a level. but is quite far from succeed after 200000steps.
i tried to initialize the camera pose spherically manually but it gives worse results.
please give me some hints to solve it.
thanks^^.

Hi @dedoogong, from your description I'm guessing there could be several issues:

The most critical is probably the pose initialization. BARF is still a local registration method, which means that the pose initializations have to be close enough to the underlying ground-truth poses. Since your multi-view data is object-centric and captured 360˚ spherically, I don't expect BARF to be able to make the cameras automagically "wrap" around the object from the same (all-identity) pose. Since you already know your capture configuration of the 72 viewpoints, it would be more realistic to initialize from the spherical angles you described. I would suggest using such poses to train a NeRF first to make sure it could at least get you some reasonable results, and then switch to BARF to see if it improves.
It sounds like you're turning the table and capturing the object every 15˚. If it's true, then the background would not be in correspondence, and BARF would have a hard time using the photometric cues for pose optimization. This wouldn't work even for the original NeRF (i.e. even when ground-truth poses are given).
Using camera intrinsics different from your actual sensor could have an impact, but it probably isn't the major issue. You could consider also optimizing the intrinsic parameters (e.g. focal lengths), as in NeRF-- or Self-calibrating NeRF.

Typically if you don't see signs of BARF converging in 20k steps, then it probably won't in the end either.
Hope these help!

Hi @chenhsuanlin ! thanks so much for your kind thoughtful reply!
I tried to use an estimated camera intrinsics(focal length) from colmap but still failed.
I agree your opinions(1,3) and I will try to find a better initial pose manually even though is would require a lot of trial and error.
Thank you!


Investigate issues in landscape on mobile

Received a report that candidate list appeared below the evidence list, even though there should have been enough room because the mobile device was in landscape orientation.
I have not been able to reproduce this on Google Pixel / Firefox

Reporter is using Chrome 70.0.3538.110, Android 8.0.0, Galaxy 7.
For my reference, viewport size should be somewhere in these links:

https://stackoverflow.com/questions/35723011/what-actual-viewport-size-are-the-samsung-s6-and-samsung-s7-mobile-phones
https://stackoverflow.com/questions/38013700/samsung-s7-media-queries
https://yesviz.com/devices/s7/


fixed typo in readme

Changed "create customs callbacks" to "create custom callbacks" in "Advanced Usage" part of readme

thanks 🙏


Updating mbed-os to mbed-os-5.4.4

Please test/merge this PR and then tag Master with mbed-os-5.4.4

Commit manually rebased and merged here


Directory does not exist in live website

Hi guys, I got an error when I upload my project live but it is working on my local machine. If I use the stand alone button, then no user's folder yet in the directory, it returns an error, But if I add the user's folder renamed it as the id before clicking the button then it works fine, here's the errror:

InvalidArgumentException in Finder.php line 632:
directory does not exist.

Version of LFM: v<IP_ADDRESS>
I've also noticed this part in the lfm.php:
// permissions to be set when create a new folder or when it creates automatically with thumbnails
    'create_folder_mode' => 0755,

// permissions to be set on file upload.
'create_file_mode' => 0644,

What folder/file do I have to apply it?
Thanks
@g0110280
@gwleuverink

Have the same problem, how did you resolved it?

Hi @fencermd  . Try to update your LFM.

me too

i also face .


experiment with tarball conda-package template

https://docs.conda.io/projects/conda-build/en/latest/resources/package-spec.html


⚠ Bofrost.be has degraded performance

In b89fbd8, Bofrost.be (https://www.bofrost.be) experienced degraded performance:

HTTP code: 200
Response time: 887 ms

Resolved: Bofrost.be performance has improved in cba4cda after 6 minutes.


Add support for sending secure cookies over localhost

Fixes #1308. See the discussions in Firefox and Chrome for more context.
P.S: We do not support localhost aliases. As far as I understand, Firefox also uses a similiar technique for this.

Codecov Report

Merging #1327 (efb5ed9) into master (4d7d6b6) will decrease coverage by 0.46%.
The diff coverage is 96.12%.

:exclamation: Current head efb5ed9 differs from pull request most recent head 17ec3be. Consider uploading reports for the commit 17ec3be to get more accurate results

@@            Coverage Diff             @@
##           master    #1327      +/-   ##
==========================================
- Coverage   97.28%   96.81%   -0.47%
==========================================
  Files          67       97      +30
  Lines        4235     6883    +2648
==========================================
+ Hits         4120     6664    +2544
- Misses        115      219     +104

Impacted Files
Coverage Δ

tests/test_binary.py
100.00% <ø> (ø)

httpie/compat.py
35.41% <32.60%> (-64.59%)
:arrow_down:

tests/conftest.py
76.59% <68.00%> (-10.37%)
:arrow_down:

httpie/manager/__main__.py
82.35% <82.35%> (ø)

httpie/output/lexers/metadata.py
82.35% <82.35%> (ø)

httpie/output/lexers/http.py
86.36% <86.36%> (ø)

tests/test_ssl.py
92.66% <87.50%> (-2.28%)
:arrow_down:

httpie/models.py
95.19% <90.00%> (-2.18%)
:arrow_down:

httpie/manager/tasks/sessions.py
92.45% <92.45%> (ø)

tests/test_uploads.py
97.70% <92.59%> (-2.30%)
:arrow_down:

... and 82 more

Continue to review full report at Codecov.

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


partial addition of the Windows platform

Windows Platform

In summary

Compilation with Visual Studio + configuration with VSCode (for Windows).
Addition of Windows equivalents for Linux code.
Addition of the "stdafx" file to group libraries.
Compilation in C99 with union.
Left todos and comments on unclear parts of the code.

What I plan to do

I will improve this version; I worked on it for 4 hours; it's an alpha version.
I will add a testing system.

Let me know if the code suits you and if there's anything I can do to better adapt it to the initial project.
Thank you for your feedback, have a great day! (sorry for my english, i'm french )

Hello @julesgrc0, Thank you for your contribution. the code is good overall but one thing I noticed is you changed the formatting a bit, as discussed in contributing.md file this project uses the k&r coding style which is the default style for clang-format. I would be more than happy if you format the code, another thing is that for now i don't want to use VS solution files but maybe in the future I will so remove those for now.

Oh, and I forgot to say, can you please modify stdafx to get rid of some includes that are used? like time.h and stdarg and manually include them?

Thanks for your feedback, I'll fix it tonight when I get home.
I wanted to know if I could change the structure of the project to make it easier to add other platforms in the future? For example, there would be a "platform" folder to add the different platforms, like this:
platform/win.h
platform/linux.h
platform/...

Thanks for your feedback, I'll fix it tonight when I get home.
I wanted to know if I could change the structure of the project to make it easier to add other platforms in the future? For example, there would be a "platform" folder to add the different platforms, like this:
platform/win.h platform/linux.h platform/...

not now. Thank you for your contribution!


Add racetracks as a kind of roads.

This is because luge/bobsled runs appear to be mapped as leisure=track, sport=luge, so it seems easiest to just enable that whole class of racetracks, which include athletics tracks, karting tracks and disc golf amongst others.
Connects to mapzen/vector-datasource#344.
@rmarianski could you review, please?

:+1:


Location problem

I can't change my Location, it is already displaying on a false Location and it stays there. I can't change it. Some help would be appreciated.
Also I have some Javascript Error popping from time to time.

If you are using an older version you can only change the location by logging out and scrolling down and inputting a new location.
No one can help you unless you tell us what error (screenshot or at least some information).


Emacs freezing when saving rs file

Hi, thanks a lot for the config !
Everything seemed to work well except when I try to save a rust file, emacs freezes and I need eventually to kill the process and restart. The modifications on the file are saved when I reopen it.
I am on windows 10, I use clang/LLVM and the last rust-analyzer version.
Does anyone have encountered the same problem ?

Hi @waldganger, I haven't seen this happen yet, but it might be related to the pretty printing as this is what is happening when saving. I recently made a change to the config as a workaround of https://github.com/brotzeit/rustic/issues/253. Not sure if this is related but maybe it makes a difference.
If not, try to disable the pretty printing and see if this helps.


Quadratic performance on left-associated binds

The problem
Consider walking over a tree using a streaming library to do something at every leaf:
data Tree = Branch Tree Tree | Leaf Int

walkTreeTransformer :: (Monad (m IO), MonadTrans m)
                    => (Int -> IO ()) -> Tree -> m IO ()
walkTreeTransformer doSomething = loop
  where loop = \case
          Leaf i -> lift (doSomething i)
          Branch t1 t2 -> do
            loop t1
            loop t2

On left-skewed trees, i.e. trees that will end up generating left-associated binds, for example those generated by leftSkewed
-- Fully evaluated left-skewed tree
leftSkewed :: Int -> Tree
leftSkewed 0 = Leaf 0
leftSkewed n = (Branch $! leftSkewed (n - 1)) (Leaf n)

the performance of your streaming library is quadratic. In fact performance is quadratic under

streaming: https://github.com/haskell-streaming/streaming/issues/109
pipes

(Neither streamly nor conduit have quadratic performance)
The plot below demonstrates the claim. The code to generate the plot is available at https://github.com/tomjaguarpaw/streaming-benchmark. The compiler was GHC 8.10.7.

The solution
Firstly, before talking about a solution, do you actually consider this a bug? I do, and I think the risk of hitting unexpected quadratic performance makes this library unsuitable for production use as it is. On the other hand maybe you have a good reason that I shouldn't think that. If so I would be interested to hear it.
If you do think this is a bug then let's think about what can be done. I know of two straightforward options:

Add an explicit constructor for binds and then reassociate binds during elimination
Use Codensity (streamly essentially uses handwritten Codensity.)

In my benchmarks explicit bind comes out slightly ahead. (The examples are implemented to match streaming but the same techniques ought to apply to your library.)
Perhaps there is another option more suitable for your library.
I welcome your thoughts.
(Related to https://github.com/Gabriel439/Haskell-Pipes-Library/issues/131)

[N.B. conduit doesn't have quadratic performance. I made a mistake when setting up the benchmarks: https://github.com/tomjaguarpaw/streaming-benchmark/commit/2a965a790c23c759c70ac580b25c59fd22ae922c]

For what it's worth, this is mentioned in the tutorial: https://hackage.haskell.org/package/pipes-4.3.16/docs/Pipes-Tutorial.html#g:10
Here's an old ticket about it: https://github.com/Gabriel439/Haskell-Pipes-Library/issues/100

Thanks for pointing that out. My comments:

The tell-tale sign of the issue is the use of "List done wrong" (i.e. a monad action returning a list).

I don't agree with this (the same claim is made in the tutorial. There's no monadic action returning a list in my example.

conduit does not do this optimization either, but is currently the mostly highly used streaming library, so I don't think this flaw is as serious as you make it out to be.

This claim is no longer true. Michael Snoyman corrected me about it this morning.

I do not agree with the argument that just because an uninformed user can potentially write code with poor time complexity that it is a serious bug. If I did, then I would also conclude that we should abandon all linked lists for difference lists because uninformed Haskell beginners can (and do) write linked list code that gives quadratic time complexit

Personally I do think that we should abandon linked lists but not for difference lists, for Seq, Vector/Array or for a streaming library as appropriate.

I don't think that the option of a Bind constructor plus carefully written run has been considered. Perhaps that would be considered less complex than Codensity.

I don't have plans to fix this myself, but I'd accept a pull request if it didn't lead to a significant performance regression for other benchmarks

Thanks Gabriella. To clarify, I don't have plans to fix this either but I think it's helpful to have the requirements spelled out like this in case anyone else wants to take it on.


generateChangelog --author=Markus results in: Error parsing command line: Ausgabe-ChangeLogFile 'dbchangelog.xml' existiert bereits!

Search first

[X] I searched and no similar issues were found

Description
It seems the --author is not detected correctly (at least on German Windows 10 Pro), as applying it to generateChangelog always leads to the following output:
Error parsing command line: Ausgabe-ChangeLogFile 'dbchangelog.xml' existiert bereits!

Steps To Reproduce
liquibase generateChangelog --author=Markus

(The same problem occurs with other parameters, like --include-objects=MyTable)
Expected/Desired Behavior
Liquibase generates changelog file where all author attributes have the value Markus.
Liquibase Version
master
Database Vendor & Version
SAP SQL Anywhere 17.0.11
Liquibase Integration
N/A
Liquibase Extensions
N/A
OS and/or Infrastructure Type/Provider
Windows 10 Pro (German)
Additional Context
N/A
Are you willing to submit a PR?

[X] I'm willing to submit a PR (Thank you!)

This bug report is invalid. The documentation clearly says that output files are not overwritten by default. The solution is to provide --overwrite-output-file=true.


Scraping on DMM fails when videos are removed from r18

Expected Behavior
A valid URL should be returned from dmmja even though video URLs are missing from r18.
Current Behavior
The current behavior of will invalidate dmmja
Steps to Reproduce (for bugs)

Search for ABW-047 on dmmja and r18.
On r18, the video URL has been removed
on dmmja, the video still exists, but no valid URL to dmmja were returned

Additional Notes
The current implementation of Get-DmmUrl relies on searching r18 and constructing the dmm equivalent of the video URL. In my opinion isn't a very good solution as now both dmm and r18 are coupled. What happens if r18 goes down? Dmm will fail to be searched.
In this particular situation, ABW-047 was removed from r18, but it still exists on dmmja.
I think a fallback search on dmmja should still happen.

I added a fallback for a native dmm url match, though it's mostly using a hardcoded URL and the movie contentId. I tried to test with your example ABW-047 but I wasn't able to get any matches on either R18 or DMM. Do you have another movie to test on?

I think the exact code name on dmm is 118abw047
I think you can try any any movies under the maker prestige (プレステージ) as I noticed they are no longer listed as official makers on dmm, but their movies are still available for some reason.
Here are a couple of other movies: ABW-049 (118abw049), SGA-148 (118sga148)
I think hardcoding the links on dmm would not be the best idea, but we should fallback to searching on dmm whenever r18 URLs are not returned.

Thanks for those CID values. Did some testing on the URLs I have used traditionally for both the EN/JA versions of dmm:
En = "https://www.dmm.co.jp/en/mono/dvd/-/detail/=/cid=118sga148"
Ja = "https://www.dmm.co.jp/digital/videoa/-/detail/=/cid=118sga148"

From what I can see, the Ja version (with /digital/videoa/...) is returning not found, while the En version works (with /mono/dvd/...).
Unless I'm misunderstanding how their search engine works, I'm unable to get any matches for the contentId values using the DVD search. This is in contrast to the digital video search which works as expected, though it misses the point as the Prestige videos don't show up in the digital section. Were you able to use their search engine to find Prestige videos?
The DVD search url that I'm using is:
https://www.dmm.co.jp/mono/dvd/-/search/=/searchstr=[...]

Thanks for checking it out.
Unfortunately, I'm still not sure how to access dmm.co.jp/en so I can't test this out (even with a VPN, I'm still getting redirected back to the Japanese version).
On the Japanese version of DMM, the URL that you just provided,
https://www.dmm.co.jp/mono/dvd/-/search/=/searchstr=[...]

works perfectly fine and I could get a return result to a DVD prestige page.

Unless I'm misunderstanding how their search engine works, I'm unable to get any matches for the contentId values using the DVD search. This is in contrast to the digital video search which works as expected, though it misses the point as the Prestige videos don't show up in the digital section. Were you able to use their search engine to find Prestige videos?

Does that mean that CIDs are not shown to you when you browse DVD type work? I could see CID on both digital and DVD pages.

Here's my result from searching すべて:

And my result from DVD:

Trying ABW in すべて and DVD yields no results either:

Thanks for sharing the screenshots. This is really interesting. I'm speculating that VPN may not work to its 100% efficiency and there's some way DMM could figure out our exact locations. I'm not sure, but in Japan, I'm getting results without problems on both すべて and DVD. Here's the screenshot:
すべて

DVD

Edit: I've tested with a VPN now and it seems like in US, prestige series are not searchable at all, but here in Japan, results are returned.

Just tested on my VPN as well. Same results.
That does make sense since they removed the entries on R18 which is the international version. I'll update the search method and test with the VPN.

@seeyabye If you have a chance, could you test the matching from the changes on the dev branch? My initial tests passed but since this change will mostly affect your use case since you can natively see Prestige videos I'd like some additional confirmation.

@jvlflame thanks for the fallback. I've tested it and it seems good, but there are some edge cases that I've found which failed.
I've made a PR to fix this issue: https://github.com/jvlflame/Javinizer/pull/262


Add FrostedGlassFill fill type

What does it do?
Added a new Fill type called FrostedGlassFill. This new fill type allows users to specify a color using RGB format (so you can specify the opacity of the color) and a blur in px, which is later converted to rem.
Why is it needed?
I wanted a way to be able to have see-through components with a blur added.
How to test it?
Create a rectangle and set the fill to FrostedGlassFill, providing color and blur.
ex: fill=rio.FrostedGlassFill(rio.Color.from_rgb(0, 0, 0, 0.4), 10),

Thanks for the PR! You honestly did a good job navigating our code base, but unfortunately there are a few issues:

The fill only works for Rectangles, but not for Text and Icons.
fillToCss must always return the same properties. Otherwise components will look incorrect if they switch from e.g. a FrostedGlassFill to a SolidFill, because properties like backdropFilter will be left behind.
The blur is measured in pixels instead of rem like everything else in rio.

Looking at all this code again made me realize that it's overdue for a refactor, so I'm going to take care of that and then fix all of these issues. So unfortunately I can't accept this PR, but don't worry, your commits will be merged.


Add more localizations / internationalization coverage

On the Engaging Networks Strip integration page EN says they work in the following countries ( https://stripe.com/partners/engaging-networks). I'm sure they work in "all" countries but these seem to be their focus. We could add in localization / internationalization coverage for these and feel confident we have all the most commonly used ones covered.
Australia
Austria
Belgium
Brazil
Czech Republic
Cyprus
Canada
Bulgaria
Denmark
Germany
Estonia
Greece
Finland
Hong Kong
Hungary
France
Malaysia
Japan
Italy
Luxembourg
Lithuania
Ireland
India
Latvia
Malta
Norway
Romania
Singapore
Sweden
Switzerland
Slovakia
Poland
Mexico
Netherlands
New Zealand
Portugal
Slovenia
Spain
United Kingdom
United States

No need


Catch warning unstable simulation

Hi,
I'm a researcher and I'm trying to use MuJoCo for simulating deformable linear objects and specifically acquiring synthetic datasets. Therefore, I have written a script that runs many simulations each with different physical parameters. However, some combinations of physical parameters lead to unstable simulations, and the following warning is shown in the terminal.
WARNING: Nan, Inf or huge value in QACC at DOF 0. The simulation is unstable. Time = 0.0280.
I would like to catch that warning so I can run the simulation with new parameters. I am using Python bindings.

You want to inspect mjData.warning


Hypernym + Paragraph + Reducing re-rendering time

-Ability to annotate hypernym on the cluster level after the coreference annotation, by adding hypernym:true in the .json config.
-Split each document into paragraphs, by adding paragraph number in the token list paragraph.
-Reducing significantly the re-rendering time

Please merge first into Dev so I can test the changes.


entire library very slow since installing kometa

Version Number
2.1.0
What branch are you on?
master
Describe the Bug
ever since i installed kometa  my entire library has slowed down dramatically. adding files take 5 times longer than what they used to. seems like everything is just hanging.  very annoying.
Relevant Collection/Overlay/Playlist Definition
the issue is happening on plex now even if i end kometa's run.

Logs
No response

This is outside of Kometa's control, and could be an indicator that you maybe have database corruption. Maybe look into using Plex DBRepair to check for and resolve any issues.
Closing as not a Kometa issue.


Edit Profile

fixes #15
Added Edit profile functionality
Users can update their teams id anytime
For changing other information email will be sent

@DuP-491 Few more fixes needed -

Make Username a hyperlink it should redirect to UserProfile of that User.
In Course, the Course Name should be visible instead of Integer Number corresponding to that Course.

@pirateksh fixed the issue


Female Genital Mutilation Identification Method Code

FEMALE GENITAL MUTILATION IDENTIFICATION METHOD CODE:

Codes are on both the Attribute and Data Element
The Data Element is not using as being used in the data set

I think this ticket can be closed now - unless I'm missing something, so assigned to Angela for review/closure.

The item is used in the Female Genital Mutilation Data Set but that is not showing on the Data Element "Where Used".
I have checked one other item from another data set: "DATE FIRST SEEN" and it shows it is used in the National Cancer Waiting Times Monitoring Data Set.
I have checked another FGM item: Female Genital Mutilation Identification Method Code and it is not showing as being used in FGM.
Is there an issue with the FGM data set Where Used?

Await next release from James

This is the same as #32 and #34

Data Elements now showing as used in the data sets, issue closed.


Adding txt file

Great job, make sure you also push your files for PS0 in your repository here: https://github.com/ECON368-fall2023-big-data-and-economics/ps0-smathias20


Add some tests for the graph data structures etc.

This PR doesn't contain any UI tests. Includes the changes from #68

Freya merged this by hand


Add Microsoft.Resources@2024-07-01 to Python SDK readme

ARM (Control Plane) API Specification Update Pull Request

[!TIP]
Overwhelmed by all this guidance? See the Getting help section at the bottom of this PR description.

PR review workflow diagram
Please understand this diagram before proceeding. It explains how to get your PR approved & merged.

Purpose of this PR
What's the purpose of this PR? Check the specific option that applies. This is mandatory!

[ ] New resource provider.
[ ] New API version for an existing resource provider. (If API spec is not defined in TypeSpec, the PR should have been created in adherence to OpenAPI specs PR creation guidance).
[ ] Update existing version for a new feature. (This is applicable only when you are revising a private preview API version.)
[ ] Update existing version to fix OpenAPI spec quality issues in S360.
[ ] Convert existing OpenAPI spec to TypeSpec spec (do not combine this with implementing changes for a new API version).
[X] Other, please clarify:

The 2024-07-01 API version for Microsoft.Resources was not explicitly added to the python readme when the API version was published, so this version is not yet available via the python SDK. This PR corrects this oversight.

Due diligence checklist
To merge this PR, you must go through the following checklist and confirm you understood
and followed the instructions by checking all the boxes:

[X] I confirm this PR is modifying Azure Resource Manager (ARM) related specifications, and not data plane related specifications.
[X] I have reviewed following Resource Provider guidelines, including
ARM resource provider contract and
REST guidelines (estimated time: 4 hours).
I understand this is required before I can proceed to the diagram Step 2, "ARM API changes review", for this PR.

Additional information

 Viewing API changes
For convenient view of the API changes made by this PR, refer to the URLs provided in the table
in the Generated ApiView comment added to this PR. You can use ApiView to show API versions diff.

Suppressing failures
If one or multiple validation error/warning suppression(s) is detected in your PR, please follow the
suppressions guide to get approval.

Getting help

First, please carefully read through this PR description, from top to bottom. Please fill out the Purpose of this PR and Due diligence checklist.
If you don't have permissions to remove or add labels to the PR, request write access per aka.ms/azsdk/access#request-access-to-rest-api-or-sdk-repositories
To understand what you must do next to merge this PR, see the Next Steps to Merge comment. It will appear within few minutes of submitting this PR and will continue to be up-to-date with current PR state.
For guidance on fixing this PR CI check failures, see the hyperlinks provided in given failure
and https://aka.ms/ci-fix.
For help with ARM review (PR workflow diagram Step 2), see https://aka.ms/azsdk/pr-arm-review.
If the PR CI checks appear to be stuck in queued state, please add a comment with contents /azp run.
This should result in a new comment denoting a PR validation pipeline has started and the checks should be updated after few minutes.
If the help provided by the previous points is not enough, post to https://aka.ms/azsdk/support/specreview-channel and link to this PR.

@msyyc Could you clarify what ARM needs to review here? This PR is not updating any service models, just a readme.python.md to expose an already-published API spec.

This PR is only related with Python SDK so I will merge it directly as Python SDK owner.


Needs work

There is a "see also" link here which links to this same page. Remove it.  This article claims that "C" and "C++" are used with strings and then goes on to give examples with chars and ints. Last I heard, a char was not a string and certainly an int is not. There is no explanation, nor reference to what exactly C (string?) linking and C++ (string?) linking means. Null terminated vs a (structured, with internal components (fields, functions)) data object? The potential implications of linkages between files are deeper than simply null terminated or structured stings, are they not? Aren't there also implications for data/program security and compatibility?

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 933730ba-c23b-b25a-657e-c9dc5d85bcf2
Version Independent ID: 89890352-1314-9d68-e25f-64c33522ee5d
Content: Using extern to Specify Linkage
Content Source: docs/cpp/using-extern-to-specify-linkage.md
Service: unspecified
Product: visual-cpp
GitHub Login: @mikeblome
Microsoft Alias: mblome

@Dave-51 Thanks for the feedback. In addition to the issues you found, this section of the documentation in general is very out-of-date. We plan to make some major improvements in the near future. I'll keep this issue open until the new topics are published.

@Dave-51 Page has been rewritten and hopefully improved.


Misleading error when using pageTrackerScreenviewEnabled with pageTrackerTemplate

I was wrestling with this error for a bit:
To use the screenview, add the appName to the plugin options
with this configuration:
    Vue.use(VueGtag, {
      config: {
        id: options.gTag,
        params: {
          language: i18n.locale
        }
      },
      pageTrackerScreenviewEnabled: true,
      appName: "Participant Portal",
      pageTrackerTemplate(to) {
        return {
          screen_name: to.name,
          user_id: store.getters.getParticipant ? store.getters.getParticipant.id : null,
        }
      }
    }, router);

After digging into the source code it became apparent, that the top level option of appView only gets set in the template if you're not using a custom one, otherwise you need to set app_view within your pageTrackerTemplate method. Would it make sense to report a different error if they have a custom pageTrackerTemplate that's missing app_view?

If you agree with the idea of more descriptive errors when using a customTemplate, I can submit a PR with that change.

the error is correct, the thing is that by having a custom template like this you are making the error yourself by not passing the app_name.
this is how I create the template
let template;
  const customTemplate = pageTrackerTemplate(to, from);

  if (customTemplate) {
    template = customTemplate;
  } else if (pageTrackerScreenviewEnabled) {
    template = {
      app_name: appName,
      screen_name: to.name
    };
  } else {
    template = {
      page_title: to.name,
      page_path: to.path,
      page_location: window.location.href
    };
  }

so when you create the custom one, u are taking care of everything basically
sorry for the huge delay of the response 🙏


Move succeedsWithin to FutureAssert

succeedsWithin is currently only defined on CompletableFutureAssert but is compatible with a regular Future also.
This means if I have a method that returns a Future (but I know it is a CompletableFuture), I have to cast it in order to use this assertion.
Placing this assertion on FutureAssert instead would let it work in either case.

sure!

Awesome, thanks! I'll put a PR together.

Done.
As far as I can see, the assertions need to stay on CompletableFutureAssert also. I've tried to share as much code between them but let me know if there is a better way.
Please let me know if you'd like any changes.

succeedsWithin had been added to FutureAssert: https://www.javadoc.io/static/org.assertj/assertj-core/3.18.1/org/assertj/core/api/AbstractFutureAssert.html#method.summary


Funny behavior when inside of a container that is hidden on page load

When the main (initial) tag div is inside of a container that is hidden on page load, this library does not work very well. Behaves strangely. I had to write some code to wait to initialize bootstrap-tags until the div they were in was displayed.

Has there been any progress on this?  I'm having the same problem.  I've stuck of tags inside a modal window that is hidden until a button is clicked.  The size of the input box is 0 width until I type something in.  Then it sets to normal, and correctly persists upon modal close and open.  @maxwells
@rtconner
I tried implementing something similar but it didn't fix the problem.  How do you initialize your tags?

.. sheesh asking me about code I wrote 8 months ago? haha.
@havok2063 anyways .. i just had an event that initialized the library on show() of the div (instead of on page load)

@rtconner  Yeah, I'm just started trying to implement this nice tag feature in my code, and of course, I end up using it in a way that's broken. :(  Could you provide an example of how you got it to work?  I've tried a few different ways, but it doesn't seem to fix the problem.
The issue is that when jquery gets the width of a hidden element via $(selector).width(), it returns 0 instead of the actual div width.  The tag code does this at line 333 in bootstrap-tags-js.  Once the div is visible, it can correctly get the actual width, which persists even if the div element is hidden again.  So the solution seems to be to modify that line so it grabs the actual width of the div element, regardless of it being hidden or not.  However I don't know how to do that. :)   Plus not sure how often things get updated here.  The last update was months ago.   Any thoughts?

FYI
I encountered a similar problem when using the control inside the bootstrap-table control: every next row/record, the tags control was getting bigger.
For anyone interested I fixed it for me like this:

passed an extra parameter fixedWidth, like .tags( { fixedWidth: 200, ...})
as havok2063 pointed out above, at line 333 you can find this:

pWidth = _this.$element.width() - pLeft;

replace it with:
if (this.fixedWidth) {
	pWidth = this.fixedWidth - pLeft;
} else {
	pWidth = _this.$element.width() - pLeft;
}


[HOLD for payment] Attendee Tracking feature C+ reviews

Please pay out @ZhenjaHorbach for their work as contributor+ on these Attendee tracking feature PRs.

https://github.com/Expensify/App/pull/48008
https://github.com/Expensify/App/pull/48870
https://github.com/Expensify/App/pull/49008

The first two PRs contained a huge number of changes that could have been easily broken into smaller PRs. We also delayed the payment by quite a few months, so I suggest we increase the payment from $750 to $1000.

So just to be sure, $1000 for all 3 PRs? @Julesssss

Offer sent for payment here https://www.upwork.com/jobs/~021851043127029144459

Paid and contract ended @ZhenjaHorbach !
Thanks


Not deleting the added image in the locale.

Terms

[X] Before you openning this issue, i have checked if the issue has already been reported.

Bagisto Version(s) affected
master
Issue Description
Not deleting the added image in the locale.
Preconditions

Admin should be logged in.
Admin should have at least one locale and the image should be added.

Steps to reproduce

Go to 'Admin panel'
Click on 'Edit Locale' inside the Settings/Locale
Delete the image and save
Reopen the same locale
See the error

Expected Result
Should be able to delete.
Actual Result
Not able to delete it. Here
Screenshots
No response
Additional context
No response

Now, this issue is fixed. so I am closing this issue.
And I have attached a screenshot Here, for future reference.


Return Null Generic Using C#8 Nullable

I'm having trouble implementing C#8 Nullable feature.
Very simple case.
    public class NullableGeneric<T>
    {
        public T Default => default;
    }

This now returns "Possible null reference return.
How can I fix this? T can be either a class or struct.

I believe one way is to suppress the warning using ! and then respecifying that the property can return null by using [MaybeNull]:
public class NullableGeneric<T>
{
    [MaybeNull]
    public T Default => default!;
}

Though I'm not sure if this is the best way to do it.

Simply adding ! at the end of default suppresses the warning; but I really don't see why it would be necessary here

@mysteryx93 Because e.g. NullableGeneric<string>.Default would return null, even though its return type is a non-nullable string.


envied, astronomically (logical-conflict)

This pull request was generated by the 'mq' tool

/trunk merge


Make some blocks waterloggable

[x] Beams
[x] Vertical stairs
[x] Jars
[x] Tables
[x] Chairs
[x] Footstools
[x] Shelves
[x] Covers
[x] Display cases

Added in dev. This will be released with the next version (hopefully this weekend, but we'll see)


ATProto: convert polling for posts/notifs to task per user

Not urgent, but eventually.

Obsoleted by #978


Reappearance of placeholder in RichText Field

Issue Summary
Once text entered by user, ideally the placeholder should disappear and no longer prompt the user.
But in the case of RichText Field, even after user enters some amount of text, when "ENTER" key pressed(with or without edit request) we can see that the placeholder which disappeared(after any first text was inputted), reappears prompting user to enter some text
Introduced as part of the HTML5 specification, the placeholder attribute “represents a short hint (a word or short phrase) intended to aid the user with data entry when the control has no value. A hint could be a sample value or a brief description of the expected format.”
Here is a sample of the behaviour in a new wagtail project

(ENTER key pressed multiple times)

Steps to Reproduce

Choose any Page with a RichTextField
Create a new entry or edit an exisitng entry.
Enter some text. Press "ENTER" key(placeholder reappears)

Any other relevant information. For example, why do you consider this a bug and what did you expect to happen instead?
I believe once user has entered any text, the placeholder should disappear completly without any scope of reappearance unless the field value becomes empty(null)

I have confirmed that this issue can be reproduced as described on a fresh Wagtail project: yes

Technical details

Python version: 3.10.9
Django version: 4.1.7
Wagtail version: 4.2.1
Browser version: Firefox/111.0

Alright.
Thankyou @gasman for the clarification


How to allow 'homeFillsViewer' option to show the top part of the image?

When I am using the homeFillsViewer option. It is showing the image from the center part.

I want that is should show the top part of the image like this:

Can anyone tell me how can I achieve that?
This is my code:

this.viewer = new OpenSeadragon( {
element: this.$openseadragonDiv[ 0 ],
zoomInButton: 'openseadragon-zoomIn',
zoomOutButton: 'openseadragon-zoomOut',
homeButton: 'openseadragon-home',
showNavigator: true,
showFullPageControl: false,
navigatorHeight: '140px',
navigatorWidth: '80px',
useCanvas: false,
visibilityRatio: 1,
homeFillsViewer: true,
tileSources: {
type: 'legacy-image-pyramid',
levels: [ {
url: url,
height: 889,
width: 600
},
{
url: url2,
height: 2201,
width: 1485
},
{
url: url3,
height: 4402,
width: 2970
}
]
}
} );

Unfortunately we don't have a good mechanism for customizing the home location. My suggestion is we should reinstate setHomeBounds (issue https://github.com/openseadragon/openseadragon/issues/1090). There's also some discussion on workarounds in https://github.com/openseadragon/openseadragon/issues/643.


Auto-updates

The application should check for updates on startup - If an update is found, it should prompt the user with an InboxImportDialog (which will need to renamed). They should be able to check a box that allows future updates to silently install. - The update will take affect the next time the user runs the app.

https://github.com/MediaComplete/MediaComplete.Lib/issues/16


Why should I use openlayers instead of leaflet

Sorry guys, this maybe a wrong location.
What is the reason that I should use openlayers instead of leaflet?
I found a few article talk about openlayers can deal with more complexed gis issue. but what is this kind of issue?
We used openlayers a lot, it is great, but seems more and more people are using leaflet, we are trying to review if we need to switch.
Thanks

@johnluan I had that question myself and read this article to get a better understanding:
https://mapsvg.com/blog/openlayers-vs-leaflet

I think the reason why a lot of people using Leaflet is that it is sufficient for the standard use case of showing a map with some simple info functions. That's the focus of Leaflet and therefore it is simple especially for non-developers with no GIS know how (like web designers) to setup a map in web page.
If you need more than this OpenLayers is your friend. It has a more powerful and cleaner API and a cleaner overall architecture. For example the separation of layers, datasources and formats. In Leaflet "GeoJSON" is a layer 😄 .

I found a few article talk about openlayers can deal with more complexed gis issue. but what is this kind of issue?

You are using OL a lot but don't know it? 😉

We used openlayers a lot, it is great, but seems more and more people are using leaflet, we are trying to review if we need to switch.

Hehe, sounds like the only reason for you to switch is that a lot of people using Leaflet. If you think OL is great and it fits your requirements why there should be a need to switch?

@TDesjardins

If you think OL is great and it fits your requirements why there should be a need to switch?
Because it has never been a bad idea to compare with other solutions. For quite a lot of reasons...

@TDesjardins

If you think OL is great and it fits your requirements why there should be a need to switch?

Because it has never been a bad idea to compare with other solutions. For quite a lot of reasons.

@rowild Sure, but comparing is another step than switching. My question is meant rhetorical and could be used to find answers that could be used for making such decisions. Other reasons than "A lot of people using it.". This could be at most an indicator because you don't know if the requirements of those peoples match yours.


Can't update device's status in the application.

Overview
The motion controllers' input and the headset tracking can only be detected and set BEFORE i open my
master build UWP application. It's highly likely that i set something wrong in the Unity, but i still need to know how to fix it.（They work perfectly in play mode）
Expected Behavior
I can power on the motion controller or set the headset tracking AFTER i open my application, and still get the correct tracking.
Actual Behavior
I have to power on the motion controller and make sure the headset already got the tracking signal BEFORE i start my application, otherwise they won't work correctly.
Unity Editor Version
Unity 2017.2 MRTP5
Mixed Reality Toolkit Release Version
2<IP_ADDRESS>

This sounds like a controller detection bug that I believe was present in MRTP5. Please re-open if you're still seeing this issue, but this should have been resolved in later versions of Unity. The current recommendation is Unity 2017.4.


Synchronize multiple chart's x-axis.

I found the solution for x-axis synchronize for two chart(pasted below). But I stuck with multi chart synchronize. I have 8 charts in my dashboard. I want to synchronize all of them.
Is there any fiddle code to do it? It will be very helpful if you put them in examples folder.
Existing code for two charts:
$scope.chart1 = {
    options: {
        chart: {
            events: {
                selection: function(event) {
                    self.onZoomHandler(event, 'chart2');
                }
            }
        }
    }
}

self.onZoomHandler = function(event, chartToUpdate) {
    var chart = $scope[chartToUpdate].getHighcharts();
    if (event.xAxis) {
        chart.xAxis[0].setExtremes(event.xAxis[0].min, event.xAxis[0].max);
        if (!chart.resetZoomButton) {
            chart.showResetZoom();
        }
    } else {
        chart.xAxis[0].setExtremes(null, null);
        if (chart.resetZoomButton) {
            chart.resetZoomButton = chart.resetZoomButton.destroy();
        }
    }
};

This could be done with angular events. Not really an issue with this framework though. Maybe try stackoverflow?

@pablojim Thanks.


Authentication / Authorization

[ ] Authenticate via authentik IdP
[ ] Protected OGC API services
[ ] Protected WMS + maps
[ ] Authorize via QWC permissions
[ ] Authorize via bbox config

OIDC implemented in 81a81ec19
Other tasks will be tracked seperately.


Fix for TFframe reference and coloring of CUBELIST/SPHERELIST

The current issue was markers were not rendering under the correct TFframe parent from ROS into Unity. The TFtree did not incorporate the transform of the the gameobject. Thus this fix simply finds the game object in unity, caches it into the list, and uses the trasnform found in the list as the reference frame, instead of the one from the TFtree.
Also added individual coloring of cubelist and spherelist

I'm not sure how to remove the scenes from this pull request, however, it should be synced with upstream/master


Support for annotating tags with a message

Support for annotating tags with a message

Et voilá, here you have my reverse birthday present. :-)


Audit Logging support K8s Networkpolicy

Previously audit logging only works for Antrea NetworkPolicy and ClusterNetworkPolicy when setting the enableLogging field in the rule. This PR is to address this issue.
Method overview of this PR is at this design doc.
To enable logging for K8s NetworkPolicy, set annotations in metadata for the Namespace as policy.antrea.io/enable-np-logging: "true". Then K8s NetworkPolicy applied to this Namespace will be logged in the directory /var/log/antrea/networkpolicy/np.log.

Reviewers, there's a problem for the Namespace UPDATE event. When modifying the drop flow in an UPDATE event, we might need to compare the previous drop flow and the newly generated one to determine if a modification is necessary. Code here.
Currently the method is to always modify the drop flow due to lack of available information. If we want to compare the flows, we need to change the Flow struct to add useful methods, which involves much more changes to the OVS struct.
Any suggestions or thoughts? Thanks.

Reviewers, there's a problem for the Namespace UPDATE event. When modifying the drop flow in an UPDATE event, we might need to compare the previous drop flow and the newly generated one to determine if a modification is necessary. Code here. Currently the method is to always modify the drop flow due to lack of available information. If we want to compare the flows, we need to change the Flow struct to add useful methods, which involves much more changes to the OVS struct. Any suggestions or thoughts? Thanks.

If enableLogging changes, the ID of rule also changes, the original rule would be deleted and the new rule would be installed:
https://github.com/antrea-io/antrea/blob/2a37aec1e4df5e976601342a72eb76a42058e335/pkg/agent/controller/networkpolicy/cache.go#L650-L665
So it's already handled. Otherwise current code won't work because reconciler doesn't call functions like AddPolicyRuleAddress for a previously install rule if there is no address change.
I guess Code is not needed?

So it's already handled. Otherwise current code won't work because reconciler doesn't call functions like AddPolicyRuleAddress for a previously install rule if there is no address change.
I guess Code is not needed?

Probably I didn't express it clearly, for the flow generated by the rule directly, it indeed changes due to ruleID update, so the IngressRuleTable is updated correspondingly.
However, for K8s NetworkPolicies, the default drop flow in IngressDefaultTable would not update since address was not changed. The pathway is Reconcile->add->installOFRule->InstallPolicyRuleFlows->calculateMatchFlowChangesForRule, calculateChangesForRuleCreation->addAddrFlows->addConjunctiveMatchFlow. In this function, it searches for globalConjMatchFlowCache where the key only depends on tableID, priority and matchPairs, it does not depend on enableLogging.
I did some tests as well, Code here is the place to update default drop flow by K8s NP, but the problem remains whether to compare the flows so that we only update when it is necessary. If we need to compare but there is no easy way to modify Flow, then I believe the UPDATE event should be marked TODO, to solve together with the race condition. Thanks.

@qiyueyao thanks for the explanation. Now I understand the problem is the dropFlow is not specific to a rule but specific to a match condition. Then does it work if we store enableLogging in conjMatchFlowContext and update the dropFlow when it changes?

I think Quan's suggestion could work. Since we only use it for dropFlow, maybe name it as something like: dropFlowEnableLogging?

Storing enableLogging in conjMatchFlowContext works, the latest commit updates that and is ready. Thanks!

/test-all

@qiyueyao I had to rewrite the commit message and close the corresponding issue manually when merging the PR.  Could you please write detailed commit message (and perhaps squash commits timely) for future PRs? If you prefer appending commits instead of force pushing, the first commit should have a proper commit message which could be used when merging.
The PR description and commit message should use the well known format to link the issue, like "Fixes #XXX", instead of a hyperlink "This PR is to address this issue", which couldn't close issue automatically. You could find more details here.


Add Discard button to editor Touch Bar

This partly fixes #246 by adding a "Discard" button to the touch bar. Not 100% sure about the color since it's the same as "Record" which might cause confusion.

Can you add some space between Discard and the other buttons? We don't want people tapping it by mistake.

@kevva Can you fix the merge conflict?

Done.

Great! I decided to move it all the way to the right:

Like flexible too! I thought large looked somewhat out of place.

This is awesome.


ADLS Gen2 and Azure Databricks using the Spark connector

Hi,
could we please get insights by when ADLS Gen2 and Azure Databricks using the Spark Connector will be supported? What are the dependencies?
Referring to https://docs.microsoft.com/en-us/azure/analysis-services/analysis-services-datasource
Thanks,
Isa

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: ea45d88b-212b-51ae-b902-6efd5873f6c5
Version Independent ID: 6f86e171-7553-3d01-00fa-e0865d8d817b
Content: Data sources supported in Azure Analysis Services
Content Source: articles/analysis-services/analysis-services-datasource.md
Service: azure-analysis-services
GitHub Login: @Minewiskan
Microsoft Alias: owend

@isamllr, Thank you for sharing the query. We are looking into this and will get back to you soon.

@Minewiskan Could you please look into the ask?

@isamllr  - We try to keep this article specific to supported data sources. ADLS Gen 2 is not currently supported, however, you can connect Azure AS to ADLS Gen 2  by using the preview multi-protocol access Blob APIs. Check out the following links to learn more.
https://powerbi.microsoft.com/en-us/blog/connecting-azure-analysis-services-to-azure-data-lake-storage-gen2/
https://docs.microsoft.com/en-us/azure/storage/blobs/data-lake-storage-multi-protocol-access
If/when ADLS Gen 2 and databricks connectors are available for AS models, the blog and this article will be updated.
For other questions like this, it's best to post to the AS forum.
Thanks
#please-close


Dangerous public interface in iati.core.default

A number of functions in iati.core.default have a use_cache parameter. Setting this value to True is extremely dangerous since it allows the defaults to be overwritten in strange ways.
Due to the danger of this parameter, it is suggested that private functions with this parameter are made available for optimisation purposes. The public interface would then become a fair bit safer since it would ensure the returned values are indeed the defaults.

#147 now merged to dev.


Explain post delete method

Fixes #4054

@guardrex,
Thanks for having already signed the Contribution License Agreement. Your agreement was validated by .NET Foundation. We will now review your pull request.
Thanks,
.NET Foundation Pull Request Bot

Thx @scottaddie ... got it.

@guardrex Awesome work! Thanks again for your help.


Option to set a minimum cluster size

It would be helpful to set a minimum cluster size so that a small amount of markers (like 2) close to each other are not being clustered

Just in case this is picked up at some point, would be super useful for our use case (areas of high density clusters, near areas with sparse data where clusters not necessary).


Meditor must not freeze web ui

It might be just too long, but must be beyond 30 sec as we do get through on 000026 with slightly over 30 sec (see https://github.com/dandi/dandi-api-webshots/#000026)  but browser (when trying locally directly without webshots) does just say that Aw Snap eventually)

probably the same for 000041
2021-05-09T13:36:50-0400 [INFO    ] 000041 edit-metadata
Traceback (most recent call last):
  File "/home/yoh/proj/dandi/dandi-api-webshots/tools/make_webshots.py", line 237, in <module>
    stats = process_dandiset(driver, ds)
  File "/home/yoh/proj/dandi/dandi-api-webshots/tools/make_webshots.py", line 150, in process_dandiset
    act()
  File "/home/yoh/proj/dandi/dandi-api-webshots/tools/make_webshots.py", line 112, in click_edit
    edit_button.click()
  File "/usr/lib/python3/dist-packages/selenium/webdriver/remote/webelement.py", line 80, in click
    self._execute(Command.CLICK_ELEMENT)
  File "/usr/lib/python3/dist-packages/selenium/webdriver/remote/webelement.py", line 672, in _execute
    return self._parent.execute(command, params)
  File "/usr/lib/python3/dist-packages/selenium/webdriver/remote/webdriver.py", line 318, in execute
    self.error_handler.check_response(response)
  File "/usr/lib/python3/dist-packages/selenium/webdriver/remote/errorhandler.py", line 242, in check_response
    raise exception_class(message, screen, stacktrace)
selenium.common.exceptions.WebDriverException: Message: unknown error: session deleted because of page crash
from unknown error: cannot determine loading status
from tab crashed
  (Session info: headless chrome=88.0.4324.182)

we should check validity of dandi metadata for all dandisets. in dandi cli, we need to turn models to contain all fields even if optional for base models, so that we can run through using model validation. i'll give this a shot later today.

@waxlamp @mvandenburgh - do you see a way to establish some "watchdog" for a loading meditor, so if it fails to load e.g. in 20 seconds, it would time out and provide an error message to file an issue here? (could even prepopulate the body of the message via URL content with details on the browser and exact url)

It's a good question. Unfortunately, I am pretty sure the answer is "no", but maybe there is something out there I'm not aware of.
Are you finding long loading times still on a particular dandiset?

Are you finding long loading times still on a particular dandiset?

Not at the moment!  But I think to get this particular issue "Closed" fully, if there is an opportunity to ensure "must not freeze", that would be the way.  Thus decided to run an idea by you.

Well, there's still a ~10 second lag on https://gui.dandiarchive.org/#/dandiset/000026 (not as bad as the loading times we used to see, but still not acceptable from a UX standpoint I think)

yeap, max time somehow consistent between deployed and staging instances ATM, it is 9 sec on https://github.com/dandi/dandi-api-webshots-staging#000032 .  Indeed not optimal UX but at least not halting the entire page (this issue) for minutes causing browser to announce the ultimate death ;)

I did a bit of profiling on 000026, and I see that the bulk of the slowdown time is spent in Vue's DOM/layout/etc. computations, specifically within a watcher. I didn't dive much deeper than that. @mvandenburgh, maybe you can do a bit more investigation here, play around with VJSF options, and maybe even ask VJSF for help in diagnosing what's going on here.


IsoSphere tangents may not be calculated correctly

Describe the bug
I'm not sure about this one. Looks like IsoSphere tangents are not calculated correctly?
I've tried to calculate tangents for an STL file with a code fragment from IsoSphere as suggested here https://github.com/google/filament/issues/959#issuecomment-472189487 but got some weird results with a simple plastic.filamat material. So I've checked more carefully the provided redball.html demo. Looks like the material is not applied correctly to IsoSphere if it has a different geometry than the default demo value?
To Reproduce
Steps to reproduce the behavior:

Copy redball.html demo
Set IsoSphere nsubdivs to 0 (zero)

Expected behavior
I would expect that IsoSphere with low res to be correctly covered with the set material.
Screenshots
Redball demo plastic.filamat with IsoSphere nsubdivs set to 0:

IsoSphere with default nsubdivs set to 5 for reference:

Same plastic.filamat applied to susanne.html demo looks ok:

Desktop (please complete the following information):

OS: Mac OS
GPU: Intel Iris Plus Graphics 655 1536 MB
Backend: OpenGL probably

Hard to tell whether the lighting is correct or not. What do you expect that you are not getting?

The material texture looked stretched and the geometry lost its edges. But you're right, it's the lighting. Pretty obvious now... it's set to make the sphere look good  🤦  Sorry.
And thank you.


Moved surface create/change code back into normal lifecycle

The SurfaceCreated and SurfaceChanged are the ideal places to handle the camera and preview setup, however if the code only lives there, it's impossible to restart the camera after it has been stopped unless the surface is destroyed and recreated.
This refactors the setup out into methods that can be called from SurfaceCreated and SurfaceChanged, but also from outside of these (eg: StartScanning).  This way we can stop scanning and restart it without recreating the surface.  The StartScanning method will check to ensure SurfaceCreated and SurfaceChanged have already been called before running the setup again, to ensure it's not the first caller to them.
This should manually merge the intent of #286

Does version 2.1.0-beta1 on NuGet include this commit? Because it does seem to fix the NullReferenceException while resuming the app from issue #274, but the app now hangs and fails on getting exclusive access to the camera.

It includes it in a slightly different way.  Can you tell me the steps to reproduce the app hanging?  I've tried resuming the app and it's working fine here getting exclusive access, but if it's hanging for you somehow it must not be properly releasing the access when the app is paused...

The issue can be reproduced by running the Sample.Android application:

Click 'Scan with Default Overlay'
Press the 'Recent Apps' / Application switcher button.
Press the ZXing.Net.Mobile app

Result: App hangs 10 seconds and throws an excpetion with the message: "Couldn't get exclusive access to the camera"
Stacktrace:
04-21 15:06:00.521 I/MonoDroid( 2830): UNHANDLED EXCEPTION: 04-21 15:06:00.521 I/MonoDroid( 2830): System.Exception: Couldn't get exclusive access to the camera 04-21 15:06:00.521 I/MonoDroid( 2830):   at ZXing.Mobile.ZXingSurfaceView.GetExclusiveAccess () [0x0002d] in D:\Projects\ZXing.Net.Mobile\Source\ZXing.Net.Mobile.Android\ZXingSurfaceView.cs:473  04-21 15:06:00.521 I/MonoDroid( 2830):   at ZXing.Mobile.ZXingSurfaceView.OnSurfaceCreated () [0x00046] in D:\Projects\ZXing.Net.Mobile\Source\ZXing.Net.Mobile.Android\ZXingSurfaceView.cs:99  04-21 15:06:00.521 I/MonoDroid( 2830):   at ZXing.Mobile.ZXingSurfaceView.SurfaceCreated (ISurfaceHolder holder) [0x00001] in D:\Projects\ZXing.Net.Mobile\Source\ZXing.Net.Mobile.Android\ZXingSurfaceView.cs:82  04-21 15:06:00.521 I/MonoDroid( 2830):   at Android.Views.ISurfaceHolderCallbackInvoker.n_SurfaceCreated_Landroid_view_SurfaceHolder_ (IntPtr jnienv, IntPtr native__this, IntPtr native_holder) [0x00011] in /Users/builder/data/lanes/3053/a94a03b5/source/monodroid/src/Mono.Android/platforms/android-23/src/generated/Android.Views.ISurfaceHolder.cs:194  04-21 15:06:00.521 I/MonoDroid( 2830):   at (wrapper dynamic-method) System.Object:a0a375b6-7053-4635-ac91-9641e9d106f9 (intptr,intptr,intptr)

Anyway, I downgraded to version 1.5.4 because it pausing/resuming is broken in all current 2.x versions.


Dragging columns is unpredictable

When dragging a column left or right of its original position, the resulting location is not where you'd expect it to be. For example, using the "Render Image" example in the documentation (https://material-table-core.com/demos/columns/render-image) try to drag the Birth Year column between Name and Surname (I'm specifically aiming for the white space between the columns before I drop). For me it jumps to the left of the name column rather than rendering in between them. When comparing it to the project from which this was forked, the mbrn version has a visual indication of where the column would land which doesn't seem to be available here.
I have determined that it works better if you drop the column on the column you wish to switch with rather than to the left or right of that column, but it still results in an unexpected order.
Using the Render Image example, if I drag Birth Place and drop it on top of the Surname column, the resulting order is Name, Birth Place, Surname, Birth Year.
If I then drag Birth Place and drop it on Name, it results in Birth Year, Name, Birth Place, Surname.
I'd have expected it to be Birth Place, Name, Surname, Birth Year.
I'm experiencing this in Chrome 104.0.5112.102 in Windows.

I have the same problem
Video: https://watch.screencastify.com/v/Wvsh0irPBbMGLnrPoHWv

Is there any updates on this? or any other workarounds which could help?

This should work now


Update PHP Images

The Images are still PHP 7.3. Already 7.4 is out since one year so far

what are you looking for @shyim ?

The Images are still PHP 7.3 😅

Got it. Looking for contributions here

When it's just updating the docker image. I can go for it. I don't know how complex all these things are working here :D

https://github.com/kubeless/runtimes/pull/98


Add dynamic attributes tracking

Currently mongoid-history considers only changes to attributes, which are described in model as "fields". There were no way to track changes to dynamic attributes, which could be set if you include Mongoid::Attributes::Dynamic to your model. I think that gem should support dynamic attributes tracking.
Please let me know if you have any remarks about implementation.

I think we don't need an option to enable/disable dynamic attribute tracking, we should just track them. Why would one not want to track dynamic fields?
Please fix the build, update CHANGELOG, etc.

Coverage decreased (-2.15%) to 97.15% when pulling fe32fb0d39b78b5598177c29d715b0a88ac56ff2 on Minisai:master into 81077c3f0b1014f19ace4c33a2d39f042154fcce on aq1018:master.

Coverage decreased (-2.15%) to 97.15% when pulling fe32fb0d39b78b5598177c29d715b0a88ac56ff2 on Minisai:master into 81077c3f0b1014f19ace4c33a2d39f042154fcce on aq1018:master.

Note that Dynamic didn't exist before Mongoid 3, so you have to figure something out for the build.

Coverage increased (+0.01%) to 99.31% when pulling 14fb609173193e6aafba29a6f2668432db2aac21 on Minisai:master into 8fb5cb8170b396dbe10aadb3b0cccdbc4738cc14 on aq1018:master.

Coverage increased (+0.01%) to 99.31% when pulling 14fb609173193e6aafba29a6f2668432db2aac21 on Minisai:master into 8fb5cb8170b396dbe10aadb3b0cccdbc4738cc14 on aq1018:master.

Coverage increased (+0.01%) to 99.31% when pulling 14fb609173193e6aafba29a6f2668432db2aac21 on Minisai:master into 8fb5cb8170b396dbe10aadb3b0cccdbc4738cc14 on aq1018:master.

Coverage increased (+0.01%) to 99.31% when pulling 335a32d5ac7eafa52f7257518f46d4a6a31727d6 on Minisai:master into 8fb5cb8170b396dbe10aadb3b0cccdbc4738cc14 on aq1018:master.

Coverage increased (+0.01%) to 99.31% when pulling 335a32d5ac7eafa52f7257518f46d4a6a31727d6 on Minisai:master into 8fb5cb8170b396dbe10aadb3b0cccdbc4738cc14 on aq1018:master.

Coverage increased (+0.01%) to 99.31% when pulling 335a32d5ac7eafa52f7257518f46d4a6a31727d6 on Minisai:master into 8fb5cb8170b396dbe10aadb3b0cccdbc4738cc14 on aq1018:master.

@dblock Thanks for the notice! Build is ok now.

Very nice, thanks.


Migrating notification templates

Summary of the Pull Request
What is this about?
Adds support for migrating our jinja templates to scriban on the fly.
PR Checklist

[ ] Applies to work item: #1754

Info on Pull Request
What does this include?

Detect whether a template is a jinja template
Convert our jinja templates to scriban template
Tests
Small docs improvement for NodeDisposalStrategy

Validation Steps Performed
How does someone test & validate?
Test ADO/github/teams notification creation with an old jinja template (so long as the template is not very complicated).
The tests that are added in this PR are testing the jinja template features we use in prod (if, for, and a combination of the 2).

Codecov Report

Merging #2486 (f66ba59) into main (0e9e32a) will increase coverage by 5.68%.
 Consider uploading reports for the commit 06829b7 to get more accurate results

@@            Coverage Diff             @@
##             main    #2486      +/-   ##
==========================================
+ Coverage   29.88%   35.57%   +5.68%
==========================================
  Files         288      135     -153
  Lines       35180    17103   -18077
==========================================
- Hits        10515     6084    -4431
+ Misses      24665    11019   -13646

Impacted Files
Coverage Δ

src/agent/coverage/src/test.rs
92.85% <0.00%> (-7.15%)
:arrow_down:

src/agent/coverage/src/code.rs
8.29% <0.00%> (-6.12%)
:arrow_down:

src/agent/onefuzz/src/machine_id.rs
51.21% <0.00%> (-5.63%)
:arrow_down:

src/agent/onefuzz/src/sanitizer.rs
69.69% <0.00%> (-2.91%)
:arrow_down:

src/agent/onefuzz-agent/src/worker/tests.rs
99.10% <0.00%> (-0.11%)
:arrow_down:

src/agent/onefuzz/src/asan.rs
0.00% <0.00%> (ø)

src/agent/coverage/src/cache.rs
0.00% <0.00%> (ø)

src/agent/onefuzz-agent/src/setup.rs
0.00% <0.00%> (ø)

src/agent/onefuzz-agent/src/reboot.rs
0.00% <0.00%> (ø)

src/agent/onefuzz/src/input_tester.rs
0.00% <0.00%> (ø)

... and 177 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


没有xtuner.parallel: 'No module named 'xtuner.parallel''

ubuntu22.04
安装步骤
conda create --name xtuner-env python=3.10 -y
 conda activate xtuner-env
pip install -U 'xtuner[deepspeed]' -i https://pypi.tuna.tsinghua.edu.cn/simple/

运行
 xtuner train ./internlm2_7b_qlora_oasst1_512_e3_copy.py  --deepspeed deepspeed_zero2
/root/anaconda3/envs/xtuner-env/lib/python3.10/site-packages/torch/cuda/__init__.py:138: UserWarning: CUDA initialization: CUDA driver initialization failed, you might not have a CUDA gpu. (Triggered internally at ../c10/cuda/CUDAFunctions.cpp:108.)
  return torch._C._cuda_getDeviceCount() > 0
/root/anaconda3/envs/xtuner-env/lib/python3.10/site-packages/bitsandbytes/cextension.py:31: UserWarning: The installed version of bitsandbytes was compiled without GPU support. 8-bit optimizers, 8-bit multiplication, and GPU quantization are unavailable.
  warn("The installed version of bitsandbytes was compiled without GPU support. "
/root/anaconda3/envs/xtuner-env/lib/python3.10/site-packages/bitsandbytes/libbitsandbytes_cpu.so: undefined symbol: cadam32bit_grad_fp32
[2024-04-02 11:05:56,914] [INFO] [real_accelerator.py:191:get_accelerator] Setting ds_accelerator to cuda (auto detect)
04/02 11:05:58 - mmengine - WARNING - WARNING: command error: 'No module named 'xtuner.parallel''!
04/02 11:05:58 - mmengine - WARNING -
    Arguments received: ['xtuner', 'train', './internlm2_7b_qlora_oasst1_512_e3_copy.py', '--deepspeed', 'deepspeed_zero2']. xtuner commands use the following syntax:

        xtuner MODE MODE_ARGS ARGS

        Where   MODE (required) is one of ('list-cfg', 'copy-cfg', 'log-dataset', 'check-custom-dataset', 'train', 'test', 'chat', 'convert', 'preprocess', 'mmbench', 'eval_refcoco')
                MODE_ARG (optional) is the argument for specific mode
                ARGS (optional) are the arguments for specific command

    Some usages for xtuner commands: (See more by using -h for specific command!)

Hi! @menkeyi001
So sorry for that. This is a known issue about xtuner v0.1.16, and we are working on it!
Before being fixed, you can degrade xtuner to 0.1.15, or install xtuner from source, to resolve this bug.
# Approach 1: degrade
pip install xtuner==0.1.15
# Approach 2: install from source
git clone https://github.com/InternLM/xtuner.git
cd xtuner
pip install -e .

@menkeyi001
We have fixed this bug.
You can install xtuner>=0.1.17 to solve this issue.
pip install 'xtuner>=0.1.17'


passing other arguments to customValidator

I'm currently using the custom-validator feature to return the result of sever side field validations. In this use case it would be ideal to pass the name of the field being validated instead of only the value to the custom-validator function.
So after my ajax promise is resolved I set a boolean for the validated field in a validations object  and then I call the validate method which triggers the custom validator:
ajaxValidator: function (value,fieldName) {
            if(typeof this.validations[fieldName] != 'undefined') return this.validations[fieldName];
            return true;
 }

Can your code be adapted to accommodate this easily or Is there a better way to approach this?

Closing Vue 1 compatibile version issues.


api.html: Recommend pip install for installing API bindings.

We have an updated PyPI package now for our API bindings, so we
now recommend pip install rather than providing a download link
to the package. :)

Automated message from Dropbox CLA bot
@eeshangarg, it looks like you've already signed the Dropbox CLA. Thanks!

Yay, it's great to have this fixed :)


first stab at a federation page

Let's start working on #544 ...
Here is a first draft!

The image is a gif:

yes, it is quite slow, that was intentional. Don't want to create a flashing page, do we?
@jancborchardt because you might want to say "try without the background gradient", here's a shot without that...

Let me know what you think, @karlitschek @jancborchardt @schiesbn

i like it :+1:

No, the background is great! :dancer:
Few things:

the address in the gif is different on almost every screenshot. Just use<EMAIL_ADDRESS>for every one of them
do not use "I" or "my" in body copy. Simply cut the »How do I share blabla« header. The top header suffices
say »personal settings« instead of »user settings«. That’s how it’s called in the interface
cut the horizontal line. The whitespace is enough

Done. If that is all - we can merge this whenever you guys want.

@LukasReschke all done, thanks for the reading.

@jospoortvliet looks good except one thing: What are the images federated_sharing_1.png and federated_sharing_2.png, and why do they still have this strange name?

@jancborchardt they can be removed, they were part of the gif sources. Will remove them and merge.


Automated Package Upgrade

:tada: This PR is included in version 5.0.91 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


[x]XP ([x]XP total) doesn't render and your two column doesn't render properly.

Excellent webapp though.

ah right the [x]XP isn't really a token since you could choose to award an amount of XP that isn't equivalent to the CR of the creature. I could add tokens that let you specify the amount of XP through the CR (like [XP:4] or something)
The width of the exported png stat block is determined by the current size of the stat block on the right. You can click and drag the center divider to control the width (it's admitted a bit of a hack but it's the easiest method at the moment).

Got it. I will manually fix the XP then.
Considering the scope of the project, yeah, easiest works. What you need to do next is do this same app, but for items, and equipment, and spells, and everything else.


Rename to scipy_doctest

Also,

run ruff, fix violations
update pyproject.toml: trove classifiers etc
bump version: 0.1 is prehistory (before pytest layer); 1.0 was tagged a while ago; what is now is working towards 1.1

Something fishy is going on with local files under pytest, xfail for now (https://github.com/ev-br/scpdt/pull/153/commits/902e2e824b9fe41e6a6a0d995ee22b39192006b9) to keep the ball rolling.


Storage: Update location type fields

#1951 introduced support for location type. It works with the current state of the API, however, "MULTI_REGION", "REGION", and" DUAL_REGION" were supposed to be "multi-region", "region", and "dual-region". This will be changed in the API after a rollout on June 26 and should also be reflected here.

@JesseLovelace is the rollout still on track for today? Could you let me know if anything changes? I'll make the change tomorrow if everything goes well.


Rails 5 Upgrade - [Work In Progress]

Hey y'all,
Been testing out this gem with a Rails 5 application lately. Would love to introduce the availability of Rails 5 into the gem codebase! Running through basic setup and testing at the moment to make sure that everything works alright!

Rails 5 support would be awesome! Thanks for the effort!


Issue 445 LINT fixes

Various LINT issues addressed. All changes are passing RTL->RTL formal equivalence.

@amullick007 can we run our LINT checks on this and check if we still see any leftover issues ?

I ran our Spyglass Rule Deck locally using these changes and I am still getting around ~3k violations in RTL apart for those within veer_el2.
spyglass_warnings_mar11_non_veer_el2.txt

I flipped through the list and a lot of these are really strict style enforcements.
Are all of these rules strictly enforced on every IP you integrate?

From: Avirup Mullick @.>
Sent: Monday, March 11, 2024 2:51 PM
To: chipsalliance/caliptra-rtl @.>
Cc: Michael Norris @.>; Review requested @.>
Subject: Re: [chipsalliance/caliptra-rtl] Issue 445 LINT fixes (PR #448)
I ran our Spyglass Rule Deck locally using these changes and I am still getting around ~3k violations in RTL apart for those within veer_el2.
spyglass_warnings_mar11_non_veer_el2.txthttps://github.com/chipsalliance/caliptra-rtl/files/14564816/spyglass_warnings_mar11_non_veer_el2.txt
—
Reply to this email directly, view it on GitHubhttps://github.com/chipsalliance/caliptra-rtl/pull/448#issuecomment-1989508726, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AZ2ZUQQHUK6N2HEJBSNUFZDYXYYQJAVCNFSM6AAAAABECO47S2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSOBZGUYDQNZSGY.
You are receiving this because your review was requested.

Hi @mborder-amd -- in the process of deleting the main_1.0 branch I unintentially closed this PR. Can you please re-open, targeting branch main? Once it is completed we can create a cherry-pick PR to bring the same changes to patch_v1.0, which is where 1.0 patches will be applied/maintained.

@amullick007 can we run our LINT checks on this and check if we still see any leftover issues ?

I flipped through the list and a lot of these are really strict style enforcements.
Are all of these rules strictly enforced on every IP you integrate?

@amullick007 can we run our LINT checks on this and check if we still see any leftover issues ?

I flipped through the list and a lot of these are really strict style enforcements.
Are all of these rules strictly enforced on every IP you integrate?

Yes, most of the IPs that we integrate from other teams are enforced to have no spyglass warnings. We add in-line waivers on code to suppress these. For third party IPs, I need to check what we do. However if the current violations are waivable, we can add rule based waivers as well.

@calebofearth - As requested, I have created a new PR targeting the main branch.


No matching distribution found for pytorch-directml

I am following this:
https://github.com/microsoft/DirectML/commit/eb663f7b5b88a1312e0d6c02015ac404a66e39bc
ERROR: Could not find a version that satisfies the requirement pytorch-directml (from versions: none)
ERROR: No matching distribution found for pytorch-directml
It was with python 3.9, tried downgrade to python 3.7 still does not work

But it doesn't work even my Python‘s version is 3.8