Transpose Application Version to JSON for NFT storage

160  Bounty
Take a application version and all of the associated information from the Postgres db and transpose to json for nft storage.
Deliverable: This method will need to read the Postgres DB to retrieve the data for a application version, transpose it to json, store in nft storage, retrieve the nft and compare to ensure it is the same as the original one.

Waiting on architecture doc finalization before assigning.

Closing at this time.  Starting from scratch to add data to ArangoDB and NTF Storage.  Will reopen if needed for customer migration.


Added I.I.S. "Galilei-Vetrone"

Added my school and its website. I attend an IT course

@nick747 I have to close this request without merging, because you requested to add a non-existent domain to this repository: Host iisgalileivetrone.it not found: 3(NXDOMAIN) Please ensure that you are familiar with this repository rules located at the bottom of this page and submit a new request with the correct information. Thank you for your understanding!

I created another pull request, I forgot the ".edu" in my website: iisgalileivetrone.edu.it


<EMAIL_ADDRESS>vulnerable

<EMAIL_ADDRESS>requires<EMAIL_ADDRESS>via a transitive dependency on<EMAIL_ADDRESS>No patched version available for got

Cant add the label Security, some issue with repo setting i guess.

Also https://www.cve.org/CVERecord?id=CVE-2021-3807 via ansi-regex dependency.

Any update on this?

https://github.com/royNiladri/js-big-decimal/issues/75 should fix it

Waiting for it to be merged. Also when is a npm release expected?

Thanks for the fix! Merged and new version created v1.3.6. Please check and confirm.


KCP Connection problem

Hello
There is an ongoing problem with kcp that I know has been reported by others in the past but is still present. If you are using KCP in the game and you submit an update for apple, it will be rejected. Because kcp doesn't work on Apple testers.
If it's any clue, I know they use VPNs.

I found another tip for Android. kcp does not work in the VPN program called Hotspot Shield, which is widely used. Could be the same problem with Apple testers. If you want to test it, you can use the hotspot shield vpn program.


End-to-end latency react demo /  Support for real-time synthesizers

Hi,
The React demo had too much latency for my use cases. When I mean latency, I mean the time between someone saying something and the ability for that person to hear the first word of an answer.  Was the React demo using the streaming API on the backend? I could not find the source code of the backend of the react code.
Also is there a way with the current synthesizers to support real-time / streaming text-to-speech services such as Polly? My understanding is that none of the text-to-speech services / synthesizer in vocode.turn_based.synthesizer support a streaming API like Polly. Please let me know if that's not true.
Best,
J

the React demo uses Azure, which is relatively fast in my experience - and uses a streaming API: https://learn.microsoft.com/en-us/azure/cognitive-services/speech-service/how-to-speech-synthesis?tabs=browserjs%2Cterminal&pivots=programming-language-python#get-a-result-as-an-in-memory-stream
you're right that vocode.turn_based.synthesizer doesn't use any streaming APIs - these are all implemented in vocode.streaming.synthesizer.
summarizing the feature requests here:

use streaming APIs in vocode.turn_based.synthesizer so that the response comes back more quickly
add support for Amazon Polly

Actually now I've been working with Vocode for a week, I have a much better understanding of the codebase and the overall latency. I personally don't need Polly - Azure is fine for me now and I'm eagerly waiting for PlayHT streaming implementation to go fully live.
Regarding using Streaming for turn-based, I could see that being useful but I personally don't use turn-based so no ask on my end.


 Whilst it is working, I would ideally want the ability to preload at least the next image along in the slider. I notice in your documentation you mention the 'Intelligent prefetch/preload' which I can see does indeed work very well on vertically scrolling images (like if I disable the slider and just have the images one on top each other) - I assume there is a certain amount of pixels it has to get near to the image to trigger the preloading?
However, within the slider this works less well, and the images are a little slow and "jumpy" when cycling through the slideshow and don't appear to be preloading. I also saw in the documentation the lazySizesConfig.preloadClass where you say "Note: This class can be also dynamically set ($currentSlide.next().find('.lazyload').addClass('lazypreload');)." Now it seems like this might provide the answer I'm looking for but as my programming knowledge is pitiful at best, I am uncertain how exactly to use this code. If you could point me in the right direction I'd be very grateful!
This is the page I'm working on if you'd like to see what I mean: http://frontwardsdesign.co.uk/geoffreypreston/project/the-new-house/
Many thanks
Oliver

will look into this, either in 9 or in 20h.

Thank you, much appreciated!

I have created a small jQuery plugin for you:
(function($){
	$.fn.lazysizesRoyal = function(){
		var addPrevNextPreload = function(){
			var setPreload;
			var slider = $(this).data('royalSlider');

			if(!slider){return;}
			setPreload = function(){
				slider.currSlide.holder
					.next()
					.add(slider.currSlide.holder.prev())
					.find('.lazyload')
					.addClass('lazypreload')
				;
			};
			setPreload();
			slider.ev.on('rsAfterSlideChange', setPreload);
		};
		return this.each(addPrevNextPreload);
	};
})(jQuery);

Usage would look something like this:
jQuery('.royalSlider').lazysizesRoyal();

Note:

This has to be run after the royalSlider script!

Hello
That is incredibly kind of you! Thank you so much. However, I cannot get it to work. I have put your plugin code in a separate js file which I am enqueueing after Royal Slider, and then I am adding the initialising code jQuery('.royalSlider').lazysizesRoyal(); just before the closing body tag, again after the initialising code of Royal Slider - is this correct? Do I need to change any of the '$' in the code to 'jQuery' as I am within Wordpress? I am not getting any errors in the console and the script is enqueueing correctly.
Thanks again for your help.
Oliver

Please ignore me - I figured out what I was doing wrong, it was a classic schoolboy error (I'm too embarrassed to admit what exactly)! It works like a dream now - thank you once again!
Oliver


Medical image classification and segmentation

I, William, could hold a presentation on our work on classification of polyps from within medical imaging. We could also provide some code, maybe though notebooks.azure.com, as we are mostly working with public datasets, and some models can be trained relatively quickly, but most models need GPU support for training in normal times so probably don't work easily.
Could be done any time in 2019, after January.

A slight idea is also to spend 5 mins introducing notebooks (in the case that we share code).

Great! February would be perfect

Hey, er vi klar for i morgen i forhold til mat og rom? Jeg ser på meetup nettsiden at det er i A150, så antar at alt er klar, men bedre å sjekk.
Jeg har slides og en notebook klar, og trenger ikke noe spesiell. Ses i morgen.

Ja, alt er klart. Hvis du vil, kan du sjekke at du har tilgang til rommet og at utstyret (projektor og mikrofon) fungerer.
Vi sees i morgen 👍


Feature: Response Class

A custom response class which handles converting flask_hal.document.Document view return values into standard flask.Response objects.
This means views still follow the standard tuple return value pattern and can still return none HAL document responses if needed.
The user also has the ability to set their own response_class if needed.

Example
# Third Party Libs
from flask import Flask

# First Party Libs
from flask_hal import HAL, document

app = Flask(__name__)
HAL(app)  # Initialise HAL

@app.route('/hello')
def hello():
    return document.Document(data={
        'message': 'Hello World'
    })

if __name__ == "__main__":
    app.run(debug=True)

GET /hello HTTP/1.1
Accept: */*
Accept-Encoding: gzip, deflate
Connection: keep-alive
Host: localhost:5000
User-Agent: HTTPie/0.9.2

HTTP/1.0 200 OK
Content-Length: 87
Content-Type: application/hal+json
Date: Thu, 06 Aug 2015 14:12:37 GMT
Server: Werkzeug/0.10.4 Python/2.7.6

{
    "_links": {
        "self": {
            "href": "http://localhost:5000/hello"
        }
    },
    "message": "Hello World"
}

@peeklondon & @radeklos :eyes: pwease :smile:

nice, I like :+1:

:+1:


Hero Block

Please always provide the GitHub issue(s) your PR is for, as well as test URLs where your change can be observed (before and after):
Fix #3
Test URLs:

After: https://feat-hero--takeda-ihs--hlxsites.hlx.page/drafts/stopp/hero
After: https://feat-hero--takeda-ihs--hlxsites.hlx.page/drafts/stopp/hero-emphasis
After: https://feat-hero--takeda-ihs--hlxsites.hlx.page/drafts/stopp/hero-product
After: https://feat-hero--takeda-ihs--hlxsites.hlx.page/drafts/stopp/hero-card

I'm still working on a CLS issue on the Product variation. I don't know what the fix is yet.

@lwintch - i've done as much as i can do for the CLS, can you please re-review.

I'm still working on a CLS issue on the Product variation. I don't know what the fix is yet. (I should have left this as a draft)

All good.  It happens.


[BUG] Read-only admin: destroy volume icon is active

Describe the bug
When logged in as read-only admin (with allowed only list* actions in the role), the destroy volume icon is active. Since it doesn't have permissions, it fails with error message but still the volume is removed from the list and there is a neverending 'in progress' circle in the notification box.
To Reproduce
Steps to reproduce the behavior:

Login as read-only admin user (admin role with only list* actions allowed).
Go to Storage > Volumes, then click on a volume and press the 'Destroy' icon. There will be an error message in a pop-up notification but the volume is removed from the list.

Expected behavior
The 'Destroy' icon should be hidden when the user has no rights for this operation.
Screenshots

Desktop (please complete the following information):

OS: Ubuntu 18.04 LTS
Browser: Chrome
Version: 81.0.4044.138 (Official Build) (64-bit)

Additional context
None.

I created an aggregated issue - #347 so I'm closing this one.


Problems installing trimethyl

Hey,
your widget seems to be really great- but i can't get trimethyl to run correctly.
I've used "npm install trimethyl" to install it to my project dir, but it still throws a warning, that it is not available.
Am i missing something?
Best, Nico

I'll push a new version on NPM with this fix.
You can easily bypass this warning by defining Ti.Trimethyl = true
Otherwise, it's just a warning:
if (Ti.Trimethyl == null) Ti.API.warn(WNAME + ': This widget require Trimethyl to be installed (https://github.com/CaffeinaLab/Trimethyl)');


MINOR: Fix null exception in coordinator log

More detailed description of your change
Found that the fatalException is always null when calling log.info("xxx", fatalException), maybe we should first assign a value to it.
Summary of testing strategy (including rationale)
Test locally.
from
[2021-03-03 10:18:06,203] INFO FindCoordinator request hit fatal exception (org.apache.kafka.clients.consumer.internals.AbstractCoordinatorTest$DummyCoordinator:260)

to
[2021-03-03 10:17:37,123] INFO FindCoordinator request hit fatal exception (org.apache.kafka.clients.consumer.internals.AbstractCoordinatorTest$DummyCoordinator:260)
org.apache.kafka.common.errors.AuthenticationException: Authentication failed

Committer Checklist (excluded from commit message)

[ ] Verify design and implementation
[ ] Verify test coverage and CI build status
[ ] Verify documentation (including upgrade notes)

@ableegoldman PTAL.

One unrelated test failure which is known to be flaky: TransactionsBounceTest.testWithGroupMetadata

@chia7712 do you want to merge this? We should also cherrypick it all the way back to 2.6

do you want to merge this? We should also cherrypick it all the way back to 2.6

Sure. will merge and backport this patch.


[httparty] Add RBS files for HTTParty(0.18.1) gem

Description
Complete RBS write-up of HTTParty(0.18.1)

Added RBS files for HTTParty (0.18.1)
Added test script in _scripts folder to run rbs validate & steep check
Added tests & Steepfile in _test folder
Added git module referencing the source code of the gem

Signed-off-by: Sushanth Sathesh Rao<EMAIL_ADDRESS>

[ ] Update the test script & Steepfile to require Net::HTTP, once RBS for Net::HTTP is added into rbs/stdlib

Thanks for your pull request!
Could you move the directory from gems/httparty/0.18.1/ to  gems/httparty/0.18/?

Specify the major and minor version you are using would be great for most cases.
https://github.com/ruby/gem_rbs_collection#make-a-directory

This patch depends on https://github.com/ruby/rbs/pull/686, so we need to review https://github.com/ruby/rbs/pull/686 before reviewing this pull request.

This patch depends on ruby/rbs#686, so we need to review ruby/rbs#686 before reviewing this pull request.

Net::HTTP is internally used by HTTParty and hence ruby/rbs#686 would need to be reviewed & merged into ruby/rbs prior to this PR.

Could you move the directory from gems/httparty/0.18.1/ to gems/httparty/0.18/?

Moved the directory from gems/httparty/0.18.1 to gems/httparty/0.18


test(grpc): GetMempoolTx and GetMempoolStream test

Motivation
We have this 2 calls missing from the grpc tests. https://github.com/ZcashFoundation/zebra/issues/4350
Solution
One of the calls seems to be not working.
Review
Reviewer Checklist

[ ] Code implements Specs and Designs
[ ] Tests for Expected Behaviour
[ ] Tests for Errors

Follow Up Work

@oxarbitrage can you please sync this again now that the lightwalletd image issue was fixed?

@mergifyio update

Same issue persists after the CI fix... I'm investigating in https://github.com/ZcashFoundation/zebra/pull/4626

This PR depends on CI and test fixes in PRs:

#4669
#4663

It might also depend on the sync fixes in PR:

#4670

After those PRs merge, we should be able to make more progress on these tests.

@mergifyio update

It looks like we need to fix the full sync test, then this test should work.
I've opened ticket #4894 in the next sprint to fix that.
We can't make any progress on this PR until that's done, so I've moved this ticket to the next sprint as well.
Feel free to re-open this PR when we start work on it again.


PlayerIndex Error

@wwayne
seem to be getting this error
2016-01-18 19:35:04.721 [warn][tid:com.facebook.React.JavaScript] null is not an object (evaluating 'playerList.filter')
2016-01-18 19:35:04.722 [error][tid:com.facebook.React.RCTExceptionsManagerQueue] Unhandled JS Exception: null is not an object (evaluating 'playerList.filter')
any help on why?

Looks this is a bug in Player tab, I will check this tmr.

looks like this is at fault
actions.getSearchRecord()
   .then(() => {
      actions.getPlayerList()
})

if you do this
actions.getSearchRecord()
      .then(() => {
        actions.getPlayerList()
      })
actions.getPlayerList()

Fixed, thanks @cyprusglobe


Lookup failure

Lookup on "ss" for the code below fails. Lookup on "Sim" does work.
func (ss *Sim) ConfigEnv() {
ss.TrainEnv.Defaults()
ss.TrainEnv.VoiceOrder = Random
...

wasn't using proper local scoping -- now is


Unable to span across multiple zones with FancyZones

Microsoft PowerToys version
0.71.0
Installation method
GitHub
Running as admin
Yes
Area(s) with issue?
FancyZones
Steps to reproduce
Attempt to span window across more than 1 zone
✔ Expected Behavior
Multiple adjacent zones are highlighted
❌ Actual Behavior
Only 1 zone at a time can be highlighted
Other Software
None
Workaround
Restart PowerToys

PowerToysReport_2023-07-06-04-12-04.zip
My bad, must've forgotten to attach.
Yes, it works again after a restart. The issue occurs 2-5 times per day (for the past 2 days).

Are you still experiencing this issue? /needinfo

Hi,
I have not experienced this issue for many months now. Seems to be resolved.


unrecognized font family 'ionicons'

I've followed the directions to install this framework.

I've installed the npm package
I've installed the font folder into my xcode project.
I've put in the .ttf fonts into my info.plist http://prntscr.com/grriv0
I've tried restarting the simulator.
I've tried deleting the build folder and starting everything again.
I've tired  react-native link
I've tried react-native link react-native-vector-icons and than react-native run-ios.

Pretty frustrated at this point tbh. Also, it's not just ionicons. it's all of them.

Fixed by manually putting in stuff needed in bundle phases

I have get the error, although I have put it in bundle phases, and rebuild...

thank you masudhossain


Restructure for easy library installation

Move the example into its own folder and the library into a src folder.

Thank you!


Dynamic relation setters should replace instead of attach

Right now the dynamic relation setters attach.
$user->posts = $posts is the same as $user->posts()->attach($posts), which merges in with any existing related models.
The setter should probably replace everything that's there instead, so change Record::setRelated() to use Relation::replace() (which isn't public), or use Relation::clear() before attaching.

74eacbd...37be2b71.
Just need to finish that last test.

Done (bc77d654)!
I think it's still weird that $relation->dissociate() doesn't remove all models and $relation->purge() does, but $relation->detach() does indeed detach all models.
Need a purge() equivalent for detachment some day.


Weird naming of Labels

"amp_extensions" - Main window
"move_completed_downloads" and "only_move_from_default_save_path" - Download tab
"require_encryption_incoming" and "require_encryption_outgoing"  Privacy tab
"privacy"-tab - First letter is lowercase, which should be uppercase

Sorry, these are missing translations. I'll add them to the next version 😃

Which language are you seeing this?

German. Thanks for fixing this!

Am 06.11.2016 um 14:50 schrieb Viktor Elofsson<EMAIL_ADDRESS>Which language are you seeing this?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.

Ping @VoodaGod 😄 German is not my strong suit.

i'll check the now strings out ;)


validate_signature(googleplay) Unable to accept purchase data signature

I am trying to verify the product purchased from google play with the Google Play (receipt + signature) method, but validate_signature function does not accept the "signature" transmitted by google play by warning "Bad signature".
Same library is working to verify purchases in AppStore but not working in Google Play!
Do you have any idea why? @keeprocking

Hi, sorry for the late reply.
I stopped working with mobile applications almost 7 years ago, so I'm afraid I can't help you with this.


Crash: failed to initialize fontconfig

$ nix shell github:l4l/yofi
$ yofi
[2023-01-30][18:51:43][yofi::input][ERROR] failed to map keyboard on seat seat0: XKBNotFound
thread 'main' panicked at 'failed to initialize fontconfig', src/font/fdue.rs:15:36
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace
[2023-01-30][18:51:43][yofi][ERROR] panic: failed to initialize fontconfig

Also for me, nixos with hyprland

Yeah, nixpkg seems to be broken after #129, the fontconfig dependency need to be specified I guess. Contribution is really welcome here as I don't really use it myself

I've added --prefix LD_LIBRARY_PATH : ${fontconfig.lib}/lib to wrapProgram and it is recognized but I get thread 'main' panicked at 'cannot load default font: The face index is larger than the number of faces in the font.', src/font/fdue.rs:122:14

https://github.com/l4l/yofi/pull/138 fixes the issue for me


[BUG] Price tooltip shown for collectable fish which are market prohibited

For example Fleeting Brand has two versions: one caught with Collector's Glove buff up and one caught without that buff. The former is market prohibited but shows a tooltip price, too.

I'll have a look into this. Not sure what the item id of the collectible fish is.


Rename repository to ServerGroveTranslationEditorBundle?

It is common practice to include the vendor name in the name of a git repository. Doing this allows for multiple vendors to have a TranslationEditorBundle, while a user can still fork all of them. :)
I see you're other bundles already follow a naming scheme like that?
Other than that, interesting bundle!

this is a very valid request. I was undecided and tried to avoid a very long bundle name. We may change it after all.

Well the bundle stays ServerGrove\TranslationEditorBundle as it is now. Only the name of the repository should be adjusted. :)


How do I know I can use flash_attn?

I find it still slow, flash attn is already installed.

Look for these lines in your console:

Also the Gemma node should say (in console) if it's using "flash_attention_2" or "sdpa"


Aliyun Windows Server 2019 unable to run dapr

Aliyun Windows Server 2019 unable to run dapr
docker: failed to register layer: failed to start service utility VM (applydiff 88e57cc977280a42ef93829fad29a6ffbe1be1320278764f77575546056c1800): container 88e57cc977280a42ef93829fad29a6ffbe1be1320278764f77575546056c1800_svm encountered an error during CreateContainer: failure in a Windows system call: The virtual machine could not be started because a required feature is not installed. (0xc0370102) extra info: {"SystemType":"container","Name":"88e57cc977280a42ef93829fad29a6ffbe1be1320278764f77575546056c1800_svm","Layers":null,"HvPartition":true,"HvRuntime":{"ImagePath":"C:\Program Files\Linux Containers","LinuxInitrdFile":"initrd.img","LinuxKernelFile":"bootx64.efi"},"ContainerType":"linux","TerminateOnLastHandleClosed":true}.

@Seanxwy This seems to be an error log from docker, are you able to run other docker commands and pull down and run images normally ?
Also what version of CLI are you using and what version of runtime are you trying to initialise ?

Two things:

Looks like you're trying to use Dapr for Linux on Windows Server. Is there a reason why you want to use the Linux bits on top of Windows? We publish Dapr for Windows too, including Docker containers for Windows 2019.
If you do need to use the Linux containers, they require virtualizing a Linux environment, and on a VM like on Aliyun it requires support for nested virtualization. I'm not familiar with Aliyun, but please make sure that the kind of VM you're running does have support for nested virtualization.

@ItalyPaleAle   Can dapr be deployed based on Windows containers?There is no related description in the document.Can build deployment be done entirely based on Windows containers?

@ItalyPaleAle  Windows Server 2019 with Docker installed.You need a Linux container to install dapr.Windows containers are not supported

Yes, there are windows server 2019 containers. See here: https://docs.dapr.io/operations/hosting/kubernetes/kubernetes-hybrid-clusters/
Docker images are here: https://hub.docker.com/layers/dapr/daprio/dapr/latest/images/sha256-ef2fa7ef389025a9627c9c61a06a244bbd8da39e77f016a69f931dabccbc0ef6?context=explore
Note that you need to install Docker for windows too: https://docs.microsoft.com/en-us/virtualization/windowscontainers/quick-start/run-your-first-container

Must be deployed based on kubernetes to use windows containers?
If you use docker swarm cluster deployment, you can't use windows container? There is no plan to use kubernetes for the time being. What should I do?

You can use Swarm on windows: https://docs.microsoft.com/en-us/virtualization/windowscontainers/manage-containers/swarm-mode

Pretty tedious, hope to enhance the Windows ecosystem application


Setting to switch status bar side

Usually, editor line-dependent status bar contributions are supposed to be on the right-hand side of the statusbar. It's very odd that this one sits on the left. Switching the default by now would upset users, so I suggest to add a setting.

Hey João!
Thank you for the feature request. I'm working on a new version right now. I will see if I can add this to it.

A new version with this feature was released yesterday. I hope it works as you expect!

Thanks! Here's your reward 🥇 https://github.com/Sertion/vscode-gitblame/pull/112


v2: References to custom fonts stored across frames

Your plugin stores references to custom fonts across frames. This leads to crashes in the VST when reloading a project.
For example (but there might be others):
SpeckDisplay() {
   font = APP->window->loadFont(asset::plugin(pluginInstance, "res/DejaVuSansMono.ttf"));
}

Please see the following link on how to resolve this issue and re-submit to the Library:
https://vcvrack.com/manual/Migrate2#2-1-Don-t-store-Font-and-Image-references-across-multiple-frames

Thank you,
I have fixed this, and pushed my changes (new manifest 2.0.3)
I would test it on a DAW but I'm stuck with Ardour and Rack Studio on Linux crashing even without my modules.
Can you tell me how is it going now?

Your changes fix the issues. Thank you. You can submit the SHA to the library.


Lines were not taking into account aspect ratio.

Angle at the ends of lines weren't quite right.  If a map was not square, the line itself was not a consistent width, but depending on its orientation.
Switched the arctangents to calculate only when needed, and then use the atan(y, x) form rather than the convolute if-else statements and atan(y / x).

Go Math! :heavy_plus_sign: :heavy_multiplication_x:

Looks great to me :heavy_division_sign:  :smile: :


Scan autocompletion of directories

Partial fix for #128.
Tested locally through:

cd spring-boot-migrator/applications/spring-shell/
mvn package -DskipTests
java -jar target/spring-boot-migrator.jar
scan <tab> ../../../ <tab>

That should provide you with auto complete examples located next to spring-boot-migrator.

Needs a slash ../ before you're able to navigate directories; I think that's fair enough though, and not worth changing.

Hi @timtebeek,
first of all, thank you for your contribution!
Sorry for my late reply, I've been on PTO.
I like the auto-completion. For path auto-completion, I think we could improve the handling a bit to make it easier to use.
I will create a new issue and merge this PR.

[ ] Improve handling for Path auto-select or use Path Input component


Event Hub + Event Grid not supported with Service Endpoints

As per the Warning in https://docs.microsoft.com/en-us/azure/event-hubs/event-hubs-service-endpoints , if we enable service endpoints, Event Grid won't work. Hence, the comment at the top of this page is wrong "Secure your data connection sources (Event Hub and Event Grid) with service endpoints."
The Event Hub will have to be available to Any Networks if Event Grids must be used to get notifications, such as this architecture https://docs.microsoft.com/en-us/azure/data-explorer/ingest-data-event-grid

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

ID: 501c33cd-0ac0-685b-0da1-03a269616d08
Version Independent ID: cc306505-d4d8-b9fd-49eb-7c5a74f95fcf
Content: Deploy Azure Data Explorer into your Virtual Network (Preview)
Content Source: articles/data-explorer/vnet-deployment.md
Service: data-explorer
GitHub Login: @basaba
Microsoft Alias: basaba

only Storage accounts allow Trusted Services (https://docs.microsoft.com/en-us/azure/storage/common/storage-network-security?toc=%2Fazure%2Fvirtual-network%2Ftoc.json#trusted-microsoft-services)

@marcosgm Thank you for bringing this to our attention. I will have the content author review this and update the document as appropriate.

Adding Warning from: https://docs.microsoft.com/en-us/azure/event-hubs/event-hubs-service-endpoints

Where there should be some additional information to communicate this implications of enabling this functionality.

@JasonWHowell @orspod The current assigned content owner (@basaba) is not reachable via this repository. Should there be a new assignee? Thank you!

Thanks @marcosgm for the feedback, I've sent a PR to fix this.

@marcosgm thanks for the docs feedback.
@basaba  thanks for the doc update. https://github.com/MicrosoftDocs/azure-docs/pull/43761

@marcosgm I believe your issue has been addressed and the doc updated.
#please-close


Use placeholders when @extend'ing in prose scope

This updates the prose scope (and the things the prose scope depends on) such that we only ever @extend placeholder selectors.
This abstraction gives us control over which occurrences of the selector are included by the @extend. Without this change, occurrences of e.g. the heading classes within the fieldset component are also ending up being ‘applied’ to the prose scope, leading to selectors like .govuk-c-fieldset__legend .govuk-prose-scope h4 in the CSS. This makes no sense, as the prose scope should never be used within the fieldset legend.
With this change, only occurrences of the placeholder selectors are 'included' when @extend’ing. This means that as we define e.g. the typographic contextual adjustments at the placeholder level and these will be applied to the prose scope, but other references to the concrete classes will have no effect on the prose scope.
It also changes the linting rules to enforce that @extends can only be used to extend placeholder classes.
Difference in generated CSS
Reversal of simple selectors in compound .govuk-link / attribute selectors:
<     .govuk-link[href^="/"]::after, .govuk-prose-scope a[href^="/"]::after, .govuk-link[href^="http://"]::after, .govuk-prose-scope a[href^="http://"]::after, .govuk-link[href^="https://"]::after, .govuk-prose-scope a[href^="https://"]::after,
< .govuk-link[href^="http.\://"]::after,
---
>     [href^="/"].govuk-link::after, .govuk-prose-scope a[href^="/"]::after, [href^="http://"].govuk-link::after, .govuk-prose-scope a[href^="http://"]::after, [href^="https://"].govuk-link::after, .govuk-prose-scope a[href^="https://"]::after,
> [href^="http.\://"].govuk-link::after,

< .govuk-link[href^="https.\://"]::after,
---
> [href^="https.\://"].govuk-link::after,

Change in whitespace around selectors:
< .govuk-body-m + .govuk-heading-l, .govuk-body + .govuk-heading-l, .govuk-prose-scope p + .govuk-heading-l, .govuk-prose-scope .govuk-body-m + h2, .govuk-prose-scope .govuk-body + h2, .govuk-prose-scope p + h2,
< .govuk-body-s + .govuk-heading-l, .govuk-prose-scope
< .govuk-body-s + h2,
< .govuk-list + .govuk-heading-l, .govuk-prose-scope ul + .govuk-heading-l,
< .govuk-prose-scope ol + .govuk-heading-l, .govuk-prose-scope
< .govuk-list + h2, .govuk-prose-scope ul + h2, .govuk-prose-scope ol + h2 {
---
> .govuk-body-m + .govuk-heading-l, .govuk-body + .govuk-heading-l, .govuk-prose-scope p + .govuk-heading-l, .govuk-prose-scope .govuk-body-m + h2, .govuk-prose-scope .govuk-body + h2, .govuk-prose-scope p + h2, .govuk-body-s + .govuk-heading-l, .govuk-prose-scope .govuk-body-s + h2, .govuk-list + .govuk-heading-l, .govuk-prose-scope ul + .govuk-heading-l,
> .govuk-prose-scope ol + .govuk-heading-l, .govuk-prose-scope .govuk-list + h2, .govuk-prose-scope ul + h2, .govuk-prose-scope ol + h2 {

<     .govuk-body-m + .govuk-heading-l, .govuk-body + .govuk-heading-l, .govuk-prose-scope p + .govuk-heading-l, .govuk-prose-scope .govuk-body-m + h2, .govuk-prose-scope .govuk-body + h2, .govuk-prose-scope p + h2,
<     .govuk-body-s + .govuk-heading-l, .govuk-prose-scope
<     .govuk-body-s + h2,
<     .govuk-list + .govuk-heading-l, .govuk-prose-scope ul + .govuk-heading-l,
<     .govuk-prose-scope ol + .govuk-heading-l, .govuk-prose-scope
<     .govuk-list + h2, .govuk-prose-scope ul + h2, .govuk-prose-scope ol + h2 {
---
>     .govuk-body-m + .govuk-heading-l, .govuk-body + .govuk-heading-l, .govuk-prose-scope p + .govuk-heading-l, .govuk-prose-scope .govuk-body-m + h2, .govuk-prose-scope .govuk-body + h2, .govuk-prose-scope p + h2, .govuk-body-s + .govuk-heading-l, .govuk-prose-scope .govuk-body-s + h2, .govuk-list + .govuk-heading-l, .govuk-prose-scope ul + .govuk-heading-l,
>     .govuk-prose-scope ol + .govuk-heading-l, .govuk-prose-scope .govuk-list + h2, .govuk-prose-scope ul + h2, .govuk-prose-scope ol + h2 {

< .govuk-body-m + .govuk-heading-m, .govuk-body + .govuk-heading-m, .govuk-prose-scope p + .govuk-heading-m, .govuk-prose-scope .govuk-body-m + h3, .govuk-prose-scope .govuk-body + h3, .govuk-prose-scope p + h3,
< .govuk-body-s + .govuk-heading-m, .govuk-prose-scope
< .govuk-body-s + h3,
< .govuk-list + .govuk-heading-m, .govuk-prose-scope ul + .govuk-heading-m,
< .govuk-prose-scope ol + .govuk-heading-m, .govuk-prose-scope
< .govuk-list + h3, .govuk-prose-scope ul + h3, .govuk-prose-scope ol + h3,
< .govuk-body-m + .govuk-heading-s, .govuk-body + .govuk-heading-s, .govuk-prose-scope p + .govuk-heading-s, .govuk-prose-scope
< .govuk-body-m + h4, .govuk-prose-scope .govuk-body + h4, .govuk-prose-scope p + h4,
< .govuk-body-s + .govuk-heading-s, .govuk-prose-scope
< .govuk-body-s + h4,
< .govuk-list + .govuk-heading-s, .govuk-prose-scope ul + .govuk-heading-s,
< .govuk-prose-scope ol + .govuk-heading-s, .govuk-prose-scope
< .govuk-list + h4, .govuk-prose-scope ul + h4, .govuk-prose-scope ol + h4 {
---
> .govuk-body-m + .govuk-heading-m, .govuk-body + .govuk-heading-m, .govuk-prose-scope p + .govuk-heading-m, .govuk-prose-scope .govuk-body-m + h3, .govuk-prose-scope .govuk-body + h3, .govuk-prose-scope p + h3, .govuk-body-s + .govuk-heading-m, .govuk-prose-scope .govuk-body-s + h3, .govuk-list + .govuk-heading-m, .govuk-prose-scope ul + .govuk-heading-m,
> .govuk-prose-scope ol + .govuk-heading-m, .govuk-prose-scope .govuk-list + h3, .govuk-prose-scope ul + h3, .govuk-prose-scope ol + h3, .govuk-body-m + .govuk-heading-s, .govuk-body + .govuk-heading-s, .govuk-prose-scope p + .govuk-heading-s, .govuk-prose-scope .govuk-body-m + h4, .govuk-prose-scope .govuk-body + h4, .govuk-prose-scope p + h4, .govuk-body-s + .govuk-heading-s, .govuk-prose-scope .govuk-body-s + h4, .govuk-list + .govuk-heading-s, .govuk-prose-scope ul + .govuk-heading-s,
> .govuk-prose-scope ol + .govuk-heading-s, .govuk-prose-scope .govuk-list + h4, .govuk-prose-scope ul + h4, .govuk-prose-scope ol + h4 {

<     .govuk-body-m + .govuk-heading-m, .govuk-body + .govuk-heading-m, .govuk-prose-scope p + .govuk-heading-m, .govuk-prose-scope .govuk-body-m + h3, .govuk-prose-scope .govuk-body + h3, .govuk-prose-scope p + h3,
<     .govuk-body-s + .govuk-heading-m, .govuk-prose-scope
<     .govuk-body-s + h3,
<     .govuk-list + .govuk-heading-m, .govuk-prose-scope ul + .govuk-heading-m,
<     .govuk-prose-scope ol + .govuk-heading-m, .govuk-prose-scope
<     .govuk-list + h3, .govuk-prose-scope ul + h3, .govuk-prose-scope ol + h3,
<     .govuk-body-m + .govuk-heading-s, .govuk-body + .govuk-heading-s, .govuk-prose-scope p + .govuk-heading-s, .govuk-prose-scope
<     .govuk-body-m + h4, .govuk-prose-scope .govuk-body + h4, .govuk-prose-scope p + h4,
<     .govuk-body-s + .govuk-heading-s, .govuk-prose-scope
<     .govuk-body-s + h4,
<     .govuk-list + .govuk-heading-s, .govuk-prose-scope ul + .govuk-heading-s,
<     .govuk-prose-scope ol + .govuk-heading-s, .govuk-prose-scope
<     .govuk-list + h4, .govuk-prose-scope ul + h4, .govuk-prose-scope ol + h4 {
---
>     .govuk-body-m + .govuk-heading-m, .govuk-body + .govuk-heading-m, .govuk-prose-scope p + .govuk-heading-m, .govuk-prose-scope .govuk-body-m + h3, .govuk-prose-scope .govuk-body + h3, .govuk-prose-scope p + h3, .govuk-body-s + .govuk-heading-m, .govuk-prose-scope .govuk-body-s + h3, .govuk-list + .govuk-heading-m, .govuk-prose-scope ul + .govuk-heading-m,
>     .govuk-prose-scope ol + .govuk-heading-m, .govuk-prose-scope .govuk-list + h3, .govuk-prose-scope ul + h3, .govuk-prose-scope ol + h3, .govuk-body-m + .govuk-heading-s, .govuk-body + .govuk-heading-s, .govuk-prose-scope p + .govuk-heading-s, .govuk-prose-scope .govuk-body-m + h4, .govuk-prose-scope .govuk-body + h4, .govuk-prose-scope p + h4, .govuk-body-s + .govuk-heading-s, .govuk-prose-scope .govuk-body-s + h4, .govuk-list + .govuk-heading-s, .govuk-prose-scope ul + .govuk-heading-s,
>     .govuk-prose-scope ol + .govuk-heading-s, .govuk-prose-scope .govuk-list + h4, .govuk-prose-scope ul + h4, .govuk-prose-scope ol + h4 {

Addition of .govuk-prose-scope hr here now that it extends %govuk-section-break:
< .govuk-section-break {
---
> .govuk-prose-scope hr, .govuk-section-break {

Re-ordering of selectors:
< .govuk-section-break--xl, .govuk-prose-scope hr {
---
> .govuk-prose-scope hr, .govuk-section-break--xl {

<     .govuk-section-break--xl, .govuk-prose-scope hr {
---
>     .govuk-prose-scope hr, .govuk-section-break--xl {

```diff
<     .govuk-section-break--xl, .govuk-prose-scope hr {
---
>     .govuk-prose-scope hr, .govuk-section-break--xl {

< .govuk-section-break--visible, .govuk-prose-scope hr {
---
> .govuk-prose-scope hr, .govuk-section-break--visible {

Removal of non-sensical prose scope adjustments for fieldset component :tada::
<   .govuk-c-fieldset__legend .govuk-heading-s, .govuk-c-fieldset__legend .govuk-prose-scope h4, .govuk-prose-scope .govuk-c-fieldset__legend h4 {
---
>   .govuk-c-fieldset__legend .govuk-heading-s {

<   .govuk-c-fieldset__legend .govuk-heading-m, .govuk-c-fieldset__legend .govuk-prose-scope h3, .govuk-prose-scope .govuk-c-fieldset__legend h3 {
---
>   .govuk-c-fieldset__legend .govuk-heading-m {

<   .govuk-c-fieldset__legend .govuk-heading-l, .govuk-c-fieldset__legend .govuk-prose-scope h2, .govuk-prose-scope .govuk-c-fieldset__legend h2,
<   .govuk-c-fieldset__legend .govuk-heading-xl,
<   .govuk-c-fieldset__legend .govuk-prose-scope h1, .govuk-prose-scope
<   .govuk-c-fieldset__legend h1 {
---
>   .govuk-c-fieldset__legend .govuk-heading-l,
>   .govuk-c-fieldset__legend .govuk-heading-xl {

@igloosi happy with the changelog entry?

👍


Using whisper-large-v3 to transcribe an audio buffer from a video fails

System Info
<EMAIL_ADDRESS><EMAIL_ADDRESS>Environment/Platform

[X] Website/web-app
[ ] Browser extension
[ ] Server-side (e.g., Node.js, Deno, Bun)
[ ] Desktop app (e.g., Electron)
[ ] Other (e.g., VSCode extension)

Description
I'm building a quick POC to see if I can transcribe audio to text from a video file. The process is:

User selects video
Get the audio channel from the video via AudioContext
Convert to Float32Array
Invoke transcriber pipeline

I get this error:

Doesn't seem to be video-specific, you can use any .mp4 file and the error will happen.
Also not using a WebWorker, just trying to get this working to validate I can do this in the browser.
Reproduction
Here's my code:
<Input
        id="dropzone-file"
        type="file"
        accept="video/mp4"
        className="hidden"
        onChange={async (e) => {
          const fileList = e.target.files;
          const file = fileList?.[0];

          if (!file) {
            return;
          }

          const url = URL.createObjectURL(file);
          const video: HTMLVideoElement = document.createElement('video');
          video.onloadedmetadata = async () => {
            setProjectState({
              file,
              height: video.videoHeight,
              width: video.videoWidth,
              duration: video.duration,
            });

            const audioCtx = new AudioContext();
            const audioElement = document.createElement('audio');

            // Set the source of the audio element to the same source as the video
            audioElement.src = video.src;

            // Wait for the audio element to load
            await audioElement.load();

            // // Create a media element source node
            // const sourceNode = audioCtx.createMediaElementSource(audioElement);

            // Decode the audio data into a buffer array
            const audioBuffer = await audioCtx.decodeAudioData(
              await (await fetch(audioElement.src)).arrayBuffer(),
            );
            console.log(audioBuffer);

            // Convert the audio buffer into a Float32Array
            const numberOfChannels = audioBuffer.numberOfChannels;
            const length = audioBuffer.length;
            const float32Array = new Float32Array(numberOfChannels * length);
            // Concatenate the audio data of all channels
            for (let channel = 0; channel < numberOfChannels; channel++) {
              const channelData = audioBuffer.getChannelData(channel);
              float32Array.set(channelData, channel * length);
            }
            debugger;

            const transcriber = await pipeline(
              'automatic-speech-recognition',
              'Xenova/whisper-large-v3',
            );
            const output = await transcriber(float32Array, {
              return_timestamps: 'word',
            });

            console.log(output);
          };

          video.src = url;
          video.load();
        }}
      />

@xenova were you able to take a look at this?

@xenova will try that out and report back, thanks

sorry, i didn't give enough information.
whisper-large-v3 allows for word-level timestamps, I've taken a look at the other models and they don't seem to have this capability
so i was looking for a smaller v3 model per your suggestion, but doesn't seem like any is available.

Non-v3 models (see here) should also support word-level timestamps. For example, as shown in the model card of https://huggingface.co/Xenova/whisper-tiny.en:
// npm i @xenova/transformers
import { pipeline } from '@xenova/transformers';

let url = 'https://huggingface.co/datasets/Xenova/transformers.js-docs/resolve/main/jfk.wav';

// Create translation pipeline
let transcriber = await pipeline('automatic-speech-recognition', 'Xenova/whisper-tiny.en');
let output = await transcriber(url, { return_timestamps: 'word' });
// {
//   "text": " And so my fellow Americans ask not what your country can do for you ask what you can do for your country.",
//   "chunks": [
//     { "text": " And", "timestamp": [0, 0.78] },
//     { "text": " so", "timestamp": [0.78, 1.06] },
//     { "text": " my", "timestamp": [1.06, 1.46] },
//     ...
//     { "text": " for", "timestamp": [9.72, 9.92] },
//     { "text": " your", "timestamp": [9.92, 10.22] },
//     { "text": " country.", "timestamp": [10.22, 13.5] }
//   ]
// }

oh interesting, their documentation didn't show this, thanks.
do you have example code on how to take a local video, pull out the audio, and get the transcript? i tried whisper-tiny with my code above and it outputs empty, i feel I'm constructing the audio array (variable float32Array) incorrectly.

You need to specify the sampleRate when constructing the AudioContext. Whisper uses a sampling rate of 16000Hz, while the default is 44100Hz. For example:
const audioCtx = new AudioContext({
  sampleRate: 16000,
});

@xenova that works! thanks.

@xenova oh one more thing, do you have a tutorial on how to do chunked+streamed inference? is this even possible? I know there's the chunk_length_s and stride parameter that let's the model process long audio in chunks, but I'd like to stream each processed chunk to the user so they can cancel if needed.

was able to find your example here: https://github.com/xenova/whisper-web/blob/main/src/worker.js
based on the code, seems like this can invoke a callback per chunk


Linter complains about valid BEM

somehow the linter is complaining about this classname: .table__row--align-top
I configured the linter to enforce BEM class names, but according to this (http://csswizardry.com/2013/01/mindbemding-getting-your-head-round-bem-syntax/) article it should. Any idea what's going wrong?

Using this sample from http://csswizardry.com/2013/01/mindbemding-getting-your-head-round-bem-syntax/:
.person {
  display: block;
}

.person__hand {
  display: block;
}

.person--female {
  display: block;
}

.person--female__hand {
  display: block;
}

.person__hand--left {
  display: block;
}

I am seeing:
[15:07:23] test.scss:9 [W] Selector `person--female` should be written in BEM (Block Element Modifier) format
[15:07:23] test.scss:13 [W] Selector `person--female__hand` should be written in BEM (Block Element Modifier) format
[15:07:23] test.scss:17 [W] Selector `person__hand--left` should be written in BEM (Block Element Modifier) format

Looks like we are doing BEM all wrong.

No actually CSS Wizardry is advocating a BEM syntax which is not the standard (although that's the one I use too). To lint according to this article's syntax you need to set convention to hyphenated_BEM

According to @csswizardry on http://cssguidelin.es/#bem-like-naming
The convention is:

Elements are delimited with two (2) underscores (__), and Modifiers are delimited by two (2) hyphens (--).

According to BEM developers on (http://bem.info/tools/bem/bem-naming/):

According to original BEM-naming convention it looks like the following:
'block[_block-mod-name[_block-mod-val]][__elem-name[_elem-mod-name[_elem-mod-val]]]'
(Parameters whithin square brackets are optional)
Block — block-name.
Block's modifier in key-value format — block-name_mod-name_mod-val.
Block's boolean modifier — block-name_mod.
Block's element — block-name__elem-name.
Element's modifier in key-value format — block-name__elem-name_mod-name_mod-val.
Element's boolean modifier — block-name__elem_mod.

Duplicate of #319


Terminal exits when moving window to another display

Environment
Windows build number: Windows 10 18363
Windows Terminal version (if applicable): 0.6.2911.0

Steps to reproduce
Drag the terminal window to other display. Terminal will exit after a second or so of being unresponsive.

/feedback

https://aka.ms/AA6cvr1

Based on your feedback archive, this looks like https://github.com/microsoft/terminal/issues/2277#issuecomment-521064993
Thanks @rjfmachado!
/dup #2277


ci: setup gitlab CI

another try at #17

Ok, I got it. the notification doesn't work, because it's not using https://github.com/apps/gepetto-dashboard to push the status, but a token from @hrp2-14. Can this bot be added in the organisation / project ?

I just sent an invitation. Who has access to the bot?

@nim65s What is the current status?

@nim65s Do you still have the invitation for hrp2 bot, or do you want me to send it again?

It works now, thanks :)


add Task count to Epics table

Description

Adds a column for showing the number of Tasks in each Epic on the Project detail page.

Steps to test/reproduce

Go to a project detail page.
Click on the Epics tab

Please explain how to best reproduce the issue and/or test the changes locally (including the pages/URLs/views/states to review).
Show me
Provide screenshots/animated gifs/videos if necessary.
REMEMBER: Attach this PR to the Trello card

 I want to see the # of Tasks in each Epic in the epics table.

@jgerigmeyer This kind of works. The correct task count shows for each epic ... eventually.
I've noticed that the task count doesn't update immediately after I go through the steps to add a task on the Add/Create Task form.
The task shows up but the task_count doesn't update until I trigger the FETCH_OBJECTS_SUCCEEDED action.
Am I missing firing a dispatch somewhere? And would that require changing something in the addSuccess function?


docs(fold/za): use keywords for easy search

I constantly forget this map and start searching for "toggle" in :help fold.txt, but no matches, so I have to open google. This will solve it.
(From https://github.com/neovim/neovim/pull/22642)

I can add something like this.  It's easier to read when it comes first.


Update zio-logging, zio-logging-slf4j to 2.1.2

Updates

dev.zio:zio-logging
dev.zio:zio-logging-slf4j

from 2.1.0 to 2.1.2.
GitHub Release Notes - Version Diff
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Adjust future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "dev.zio" } ]

Or, add this to slow down future updates of this dependency:
dependencyOverrides = [{
  pullRequests = { frequency = "@monthly" },
  dependency = { groupId = "dev.zio" }
}]

labels: library-update, early-semver-patch, semver-spec-patch, commit-count:1

Superseded by #461.


Using -s option makes netsh unit tests fail

Describe the bug
The netsh tests rely on checkin the stdout output.
However the REQUIRE() in GlueCloseHandle() generates stdout output when -s is used.  This makes the tests fail.
OS information
No response
Steps taken to reproduce bug
Run unit_tests.exe with -s.
Expected behavior
Unit tests should succeed.
Actual outcome
Unit tests fail.
Additional details
No response

Proposal is to change the require to an assert in any place outside the top-level test.


Is moduleResolution=Node a requirement?

I'm currently migrating a project to ESM and noticed that all my buf generate commands fail when I use "moduleResolution": "NodeNext" in my tsconfig.json files. Is the use of "moduleResolution": "Node" an absolute requirement or is there some way to work around this so that NodeNext can be used as resolution mechanism?
source/index_pb.ts:42:38 - error TS2576: Property 'fromBinary' does not exist on type 'TransactionChecksum'. Did you mean to access the static member 'TransactionChecksum.fromBinary' instead?

42     return new TransactionChecksum().fromBinary(bytes, options);
                                        ~~~~~~~~~~

source/index_pb.ts:46:38 - error TS2576: Property 'fromJson' does not exist on type 'TransactionChecksum'. Did you mean to access the static member 'TransactionChecksum.fromJson' instead?

46     return new TransactionChecksum().fromJson(jsonValue, options);
                                        ~~~~~~~~

source/index_pb.ts:50:38 - error TS2576: Property 'fromJsonString' does not exist on type 'TransactionChecksum'. Did you mean to access the static member 'TransactionChecksum.fromJsonString' instead?

50     return new TransactionChecksum().fromJsonString(jsonString, options);

Hey @faustbrian.  moduleResolution="Node" is definitely not a requirement.  After some digging it turns out that this is due to a missing types property in the conditional exports in our package.json file.  I will put up a PR to fix this.  Thanks for pointing this out!


Use new Assignments page provided by PortalPages

Use new Assignments page provided by PortalPages.
Offering API and Classes API had to be slightly updated. I've provided new tests. I've also removed a bunch of old code that was no longer necessary (views, some controller methods, and Cucumber tests).
Depends on: https://github.com/concord-consortium/portal-pages/pull/48

As I was looking at the effects of this. I found this ITSI customization feature:
In the ITSI portal when a teacher clicks the 'assign materials' button on the class page. They are taken to this link: https://itsi.portal.concord.org/itsi?assign_to_class=[class_id]
This is customized by: https://github.com/concord-consortium/rigse/blob/74b995a542af1a321996ad97b726af85d4ac1c92/themes/itsi-learn/views/portal/clazzes/materials.html.haml#L2
One way to support this with the migration to portal-pages is to add some conditional logic in portal-pages that checks the Portal.theme and customizes this button appropriately.  Another way would be to pass some additional Portal.xyz variable that in the itsi case would be the URL https://itsi.portal.concord.org/itsi?assign_to_class=999 then the portal pages would replace 999 with the class_id.  I don't like this 999 stuff but that is what we use elsewhere.

👍


feature: Allow backup creation to specify target directory

Instead of using a predefined directory for storing the backup files, karapace_schema_backup get should allow to specify the name of the directory where the files will be stored.

I'm in the midst of looking at this, I just need to convince myself there's no foot-gun here, I think we should be safe.
Just for posterity, linking the discussion that lead to current behavior: https://github.com/aiven/karapace/pull/584#discussion_r1198894936


Add templates for Asynchronous Invocation and Parallel Service Orchestration

Purpose

Add a module template for Asynchronous Invocation.
Add a module template for Parallel Service Orchestration.
Create a separate subdirectory inside the templates folder for templates related to services.

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


Updated the JSON body

Removed "select=" from the document

@MonikaReddy-MSFT : Thanks for your contribution! The author(s) have been notified to review your proposed change.

#sign-off


Setting up deploy preview with editorial workflow on GitLab results in two pipelines per commit

Describe the bug
I am using GitLab as backend and have the editorial workflow enabled. After configuring a GitLab environment/review app in my .gitlab-ci.yml to make deploy preview links work, every push to an existing merge request results in two pipelines per commit. The first pipeline is still associated with the previous commit and fails immediately, the second pipeline is associated with the new commit and succeeds. This repeats for every subsequent push through Netlify CMS (but not for manual git commits).
To Reproduce

.gitlab-ci.yml
image: monachus/hugo

stages:
  - deploy

variables:
  GIT_SUBMODULE_STRATEGY: recursive

pages:
  stage: deploy
  script:
    - hugo
  artifacts:
    paths:
      - public
  only:
    - main

deploy_review:
  stage: deploy
  script:
    - hugo
  artifacts:
    paths:
      - public
  environment:
    name: $CI_COMMIT_REF_NAME
    url: "https://$CI_PROJECT_NAMESPACE.gitlab.io/-/$CI_PROJECT_NAME/-/jobs/$CI_JOB_ID/artifacts/public/"
    auto_stop_in: 1 hour
  only:
    - branches
  except:
    - main

Save changes to an existing article
Make some more changes to the same article and save
Receive two emails, one for a failed pipeline and one for a fixed pipeline

Expected behavior
The first pipeline for the old commit should not be triggered.
Applicable Versions:

Netlify CMS version: 2.10.78
Git provider: GitLab
OS: Fedora 32
Browser version Firefox 83.0

CMS configuration

Netlify CMS config.yml
backend:
  name: gitlab
  repo: <repo>
  branch: main
  auth_type: implicit
  app_id: <id>
  preview_context: "pages:deploy"
publish_mode: editorial_workflow
media_folder: "static/images"
public_folder: "/images"
site_url: <url>
logo_url: "/images/site-logo.svg"
collections:
  - name: "articles"
    label: "Articles"
    label_singular: "Article"
    folder: "content"
    create: true
    slug: "{{slug}}"
    extension: "md"
    format: "yaml-frontmatter"
    preview_path: "{{title}}/index.html"
    fields:
      - {label: "Title", name: "title", widget: "string"}
      - {label: "Author", name: "author", widget: "string", required: false}
      - {label: "Publish Date", name: "date", widget: "datetime", date_format: "YYYY-MM-DD", time_format: "HH:mm"}
      - {label: "Last Updated", name: "lastmod", widget: "datetime", date_format: "YYYY-MM-DD", time_format: "HH:mm", required: false}
      - {label: "Slug", name: "slug", widget: "string", required: false}
      - {label: "Menu", name: "menu", widget: "string", default: "main", required: false}
      - {label: "Draft", name: "draft", widget: "boolean", required: false}
      - {label: "Body", name: "body", widget: "markdown", required: false}
  - name: "posts"
    label: "Posts"
    label_singular: "Post"
    folder: "content/posts"
    path: '{{slug}}/index'
    media_folder: ''
    public_folder: ''
    create: true
    slug: "{{year}}-{{month}}-{{day}}_{{slug}}"
    extension: "md"
    format: "yaml-frontmatter"
    preview_path: "posts/{{year}}/{{month}}/{{title}}/index.html"
    fields:
      - {label: "Title", name: "title", widget: "string"}
      - {label: "Author", name: "author", widget: "string", default: "Florian", required: false}
      - {label: "Publish Date", name: "date", widget: "datetime", date_format: "YYYY-MM-DD", time_format: "HH:mm"}
      - {label: "Last Updated", name: "lastmod", widget: "datetime", date_format: "YYYY-MM-DD", time_format: "HH:mm", required: false}
      - {label: "Slug", name: "slug", widget: "string", required: false}
#      - {label: "Description", name: "description", widget: "string", required: false}
      - {label: "Summary", name: "summary", widget: "string", required: false}
      - {label: "Draft", name: "draft", widget: "boolean", required: false}
      - {label: "Body", name: "body", widget: "markdown", required: false}

Additional context
My actual .gitlab-ci.yml is a little larger, I shortened it for the purpose of this bug report.

full .gitlab-ci.yml
image: monachus/hugo

stages:
  - pre-deploy
  - deploy
  - post-deploy

variables:
  GIT_SUBMODULE_STRATEGY: recursive

pages:
  stage: deploy
  script:
    - hugo
  artifacts:
    paths:
      - public
  rules:
    - if: $CI_COMMIT_BRANCH == 'main'

deploy_review:
  stage: deploy
  script:
    - hugo
  artifacts:
    paths:
      - public
  environment:
    name: $CI_COMMIT_REF_NAME
    url: "https://$CI_PROJECT_NAMESPACE.gitlab.io/-/$CI_PROJECT_NAME/-/jobs/$CI_JOB_ID/artifacts/public/"
    auto_stop_in: 1 hour
  only:
    - /^cms\/.*$/

announce_target_pending:
  stage: pre-deploy
  environment:
    name: demo
  script:
    - 'curl --verbose --request POST --header "PRIVATE-TOKEN: ${API_PRIVATE_TOKEN}" "${CI_API_V4_URL}/projects/${CI_PROJECT_NAMESPACE}%2F${CI_PROJECT_NAME}/statuses/$CI_COMMIT_SHA?state=pending&context=pages:deploy&target_url=${CI_PAGES_URL}"'
  only:
    - /^cms\/.*$/

announce_target_success:
  stage: post-deploy
  environment:
    name: demo
    auto_stop_in: 1 hour
  script:
    - 'apt-get update && apt-get install -y jq'
    - 'JSON=$(curl --header "PRIVATE-TOKEN: ${API_PRIVATE_TOKEN}" "${CI_API_V4_URL}/projects/${CI_PROJECT_NAMESPACE}%2F${CI_PROJECT_NAME}/environments?name=$CI_COMMIT_REF_NAME" 2> /dev/null)'
    - 'EXTERNAL_URL=$(echo $JSON | jq -r ".[0].external_url")'
    - 'curl --verbose --request POST --header "PRIVATE-TOKEN: ${API_PRIVATE_TOKEN}" "${CI_API_V4_URL}/projects/${CI_PROJECT_NAMESPACE}%2F${CI_PROJECT_NAME}/statuses/$CI_COMMIT_SHA?state=success&context=pages:deploy&target_url=${EXTERNAL_URL}"'
  needs: ["deploy_review"]
  only:
    - /^cms\/.*$/

Hi @fkohrt, I'm wondering if this is coming from the CMS rebasing the the PR:
https://github.com/netlify/netlify-cms/blob/a92d53f88914fb73dd109686ee28c6240432f690/packages/netlify-cms-backend-gitlab/src/API.ts#L748 before pushing the commit.
If that is the case is there a way to ignore that event on GitLab's side?

Hi @fkohrt, I'm wondering if this is coming from the CMS rebasing the the PR:
https://github.com/netlify/netlify-cms/blob/a92d53f88914fb73dd109686ee28c6240432f690/packages/netlify-cms-backend-gitlab/src/API.ts#L748 before pushing the commit.
If that is the case is there a way to ignore that event on GitLab's side?

Perfect, that's it!

If that is the case is there a way to ignore that event on GitLab's side?

There is, I'll provide a fix that skips the pipeline creation.

Perfect, that's it!

If that is the case is there a way to ignore that event on GitLab's side?

There is, I'll provide a fix that skips the pipeline creation.

Thanks @fkohrt! Glad to hear it. Do you mind sharing the solution in this issue once you have it?

Thanks @fkohrt! Glad to hear it. Do you mind sharing the solution in this issue once you have it?


Add offline support to jobs

Hey there!
I had a little free time to do a little work on TE. So I decided to do a little tuning on the job system.
I enabled changing jobs for other players including offline support.
Since the offline support required the JobManager to change to User objects the whole thing should now support offline players.
I also re-enabled pagination in the job info and refactored the class structure a little.
@Erigitic Take your time and look over this. I hope this brings TE forward a little :)
Included in this PR are the following changes:

Merge the JobCommands into one class with inner classes (represents the child structure of the command)
Use User instead of Player in the commands and the JobManager so they support offline players
Add userOrSource argument to JobCommand.Set to allow changing jobs of other users. Resolves #166
Change #setJobs return type to boolean and exclude player feedback from it
-> Moved to calling methods
Collect permissions in one class using static final Strings (currently only used by JobCommand.Set) so they can be centralized so they're easier to change.
Re-enable pagination for JobCommand.Info to display large jobSets more readable. (May need more formatting though)
Change #getJobLists return type to Text and include TextActions so players can click the jobs on the list to change to it. (Maybe change this to just running the info command)

Tested on:
SpongeForge stable-5 branch.

This PR allows edits from maintainers.

What's the reason to support offline players for jobs?

As asked in #166 there's now support for changing another players job. I though that aside from allowing this for players who are offline this support would open up the plugin a little so future features which may require this don't need those changes.
And aside from that I would've suggested that anyways since the change doesn't require any actual code changes other than changing some vars from Player to User due to the only method used being #getUniqueID() which is derived from User anyways.

//Notification @yololetsplay

Updated the PR according to your comments.

Thanks for the help again @MarkL4YG. Always appreciated. Great work!


Suppressing logging of internal grate migrations doesn't appear to be working correctly

Describe the bug
Grate still logs everything while running grate internal migrations.
From debugging the code I think the reason it still logs everything when running internal migrations is because the logger is created before setting the new configuration into the DbMigrator instance. The method IsInternalMigration uses the configuration from inside the DbMigrator instance as you can see inside code snippet 2.
https://github.com/erikbra/grate/blob/13132276d62123b21946b436d81bc63d1b6b7ee5/src/grate.core/Migration/GrateMigrator.cs#L16-L31
https://github.com/erikbra/grate/blob/13132276d62123b21946b436d81bc63d1b6b7ee5/src/grate.core/Migration/GrateMigrator.cs#L282-L283
I tried lowering the line where the logger is created below the line where the new configuration (value) is set into the newly created DbMigrator instance, and it worked from what I can see. Although still some things are logged because they are logged with a logger provided by dependency injection and not the custom created logger (I'm not sure if this is a bad thing).
This is what I get when I apply the above-mentioned change.
Initializing connections.
Running grate v1.0.0 (build date 05/12/2024 17:58:26) against localhost,1435 - TestDb.
Looking in D:\Repos\TestDb for scripts to run.
================================================================================
Setup, Backup, Create/Restore/Drop
================================================================================
================================================================================
Grate Structure
================================================================================
Initializing connections.
 Versioning TestDb database with version 1.0.0.
Initializing connections.
 Versioning TestDb database with version 1.0.0.
Initializing connections.
 Versioning TestDb database with version 1.0.0.
Initializing connections.
 Versioning TestDb database with version 1.0.0.
================================================================================
Versioning
================================================================================
 Migrating TestDb from version <IP_ADDRESS> to <IP_ADDRESS>.
 Versioning TestDb database with version <IP_ADDRESS>.
================================================================================
Migration Scripts
================================================================================
Skipping 'BeforeMigration', beforeMigration does not exist.
Skipping 'AlterDatabase', alterDatabase does not exist.
Skipping 'Run After Create Database', runAfterCreateDatabase does not exist.
Skipping 'Run Before Update', runBeforeUp does not exist.

Looking for Update scripts in "D:\Repos\TestDb\./ddl/05.up.OneTime". These should be one time only scripts.
--------------------------------------------------------------------------------
  Running '20240312_01_CreateTables.sql'.
--------------------------------------------------------------------------------

Skipping 'Run First After Update', runFirstAfterUp does not exist.
Skipping 'Functions', functions does not exist.
Skipping 'Views', views does not exist.
Skipping 'Stored Procedures', sprocs does not exist.
Skipping 'Triggers', triggers does not exist.
Skipping 'Indexes', indexes does not exist.
Skipping 'Run after Other Anytime Scripts', runAfterOtherAnyTimeScripts does not exist.
Skipping 'Permissions', permissions does not exist.
Skipping 'AfterMigration', afterMigration does not exist.

grate v1.0.0 (build date 05/12/2024 17:58:26) has grated your database (TestDb)! You are now at version <IP_ADDRESS>. All changes and backups can be found at "C:\Users\jaduyve\AppData\Local\grate\migrations\TestDb\localhost_1435\2024-05-12T17_58_29.6460636_02_00".

To Reproduce
Just run grate when the internal migrations haven't been run.
Expected behavior
Logs with log level below warning during internal migration should not be visible.
Desktop (please complete the following information):

OS: Windows, MacOS
Version: 1.7.3
Database: MSSQL

I created a PR which applies the above-mentioned change. Do you think this change would be suitable for this issue 😃?  https://github.com/erikbra/grate/pull/520

Thank you. Bad testing on my part there 😄  unit tests doesn't test everything...

Fixed in #520


VCF batch ingestion

Add tiledb.cloud.vcf.ingest to support ingestion of VCF files.
Example:
from tiledb.cloud.vcf import ingest
from tiledb.cloud.utilities import read_config

# Set location of VCF dataset
dataset_uri = "s3://..."

# Set location and pattern to search for VCF files
search_uri = "s3://1000genomes-dragen-v3.7.6/data/individuals/hg38-graph-based"
pattern = "*.hard-filtered.vcf.gz"

# Read AWS credentials from ~/.aws/credentials
config = read_config()

# Start the ingestion
ingest(
    dataset_uri,
    config=config,
    search_uri=search_uri,
    pattern=pattern,
)

VCF ingestion is ready for review and release as an experimental feature while we continue to make improvements.

Thanks for the detailed review @thetorpedodog! I addressed all of the comments except the ones related to find_uris_udf, which we plan to rewrite and replace in a future PR.


Update should return WriteError when given a bad update path

After we fix panic in #1954 / #1873, let's return correct error in this issue.
Versions
v0.8.0
What did you do?
This only seems to happen with $set, see repro in #1743.
What did you expect to see?
WriteResult({
        "nMatched" : 0,
        "nUpserted" : 0,
        "nModified" : 0,
        "writeError" : {
                "code" : 56,
                "errmsg" : "The update path '0..' contains an empty field name, which is not allowed."
        }
})

What did you see instead?
dance_ferretdb  | 2023-01-03T23:04:10.275+0545  DPANIC  // <IP_ADDRESS>:45710 -> <IP_ADDRESS>:27017         clientconn/conn.go:158  err must be non-nil
dance_ferretdb  | (err = <nil>)
dance_ferretdb  | github.com/FerretDB/FerretDB/internal/clientconn.(*conn).run.func2
dance_ferretdb  |       github.com/FerretDB/FerretDB/internal/clientconn/conn.go:158
dance_ferretdb  | runtime.gopanic
dance_ferretdb  |       runtime/panic.go:890
dance_ferretdb  | github.com/FerretDB/FerretDB/internal/clientconn.(*conn).run.func5
dance_ferretdb  |       github.com/FerretDB/FerretDB/internal/clientconn/conn.go:230
dance_ferretdb  | runtime.gopanic
dance_ferretdb  |       runtime/panic.go:890
dance_ferretdb  | github.com/FerretDB/FerretDB/internal/types.NewPath
dance_ferretdb  |       github.com/FerretDB/FerretDB/internal/types/path.go:42
dance_ferretdb  | github.com/FerretDB/FerretDB/internal/types.NewPathFromString
dance_ferretdb  |       github.com/FerretDB/FerretDB/internal/types/path.go:54
dance_ferretdb  | github.com/FerretDB/FerretDB/internal/handlers/common.processSetFieldExpression
dance_ferretdb  |       github.com/FerretDB/FerretDB/internal/handlers/common/update.go:157
dance_ferretdb  | github.com/FerretDB/FerretDB/internal/handlers/common.UpdateDocument
dance_ferretdb  |       github.com/FerretDB/FerretDB/internal/handlers/common/update.go:60
dance_ferretdb  | github.com/FerretDB/FerretDB/internal/handlers/pg.(*Handler).MsgUpdate.func2
dance_ferretdb  |       github.com/FerretDB/FerretDB/internal/handlers/pg/msg_update.go:183
dance_ferretdb  | github.com/FerretDB/FerretDB/internal/handlers/pg/pgdb.(*Pool).InTransaction
dance_ferretdb  |       github.com/FerretDB/FerretDB/internal/handlers/pg/pgdb/transactions.go:71
dance_ferretdb  | github.com/FerretDB/FerretDB/internal/handlers/pg.(*Handler).MsgUpdate
dance_ferretdb  |       github.com/FerretDB/FerretDB/internal/handlers/pg/msg_update.go:90
dance_ferretdb  | github.com/FerretDB/FerretDB/internal/clientconn.(*conn).handleOpMsg
dance_ferretdb  |       github.com/FerretDB/FerretDB/internal/clientconn/conn.go:519
dance_ferretdb  | github.com/FerretDB/FerretDB/internal/clientconn.(*conn).route
dance_ferretdb  |       github.com/FerretDB/FerretDB/internal/clientconn/conn.go:405
dance_ferretdb  | github.com/FerretDB/FerretDB/internal/clientconn.(*conn).run
dance_ferretdb  |       github.com/FerretDB/FerretDB/internal/clientconn/conn.go:294
dance_ferretdb  | github.com/FerretDB/FerretDB/internal/clientconn.acceptLoop.func1
dance_ferretdb  |       github.com/FerretDB/FerretDB/internal/clientconn/listener.go:276
dance_ferretdb  | panic: path element must not be empty
dance_ferretdb  |       panic: err must be non-nil [recovered]
dance_ferretdb  |       panic: err must be non-nil
dance_ferretdb  | (err = <nil>)

Environment
- OS: Ubuntu
- Architecture: x86_64
- Version: 22.10
- Go: go1.19.3 linux/amd64
- Git: 2.37.2
- Deployment: Docker
- Deployment details: 20.10.16

DoD

[ ] Return correct error codes, error types, if possible - messages
[ ] Add tests for negative path-related cases:

[ ] missing field in the path (foo..)
[ ] negative array indexes (foo.-1)
[ ] array indexes bigger than length (foo.100)
[ ] field doesn't exist in the path (v.does-not-exist) (the prefix v exists and is an object, part of the path doesn't exist)
[ ] v.does-not-exist where v exists and not an object) - compat tests should cover it as part of the previous case
[ ] maybe some other cases if we discover them

[ ] If we find new root causes for such tests to fail, we should create new issues (and skip tests)
[ ] Operators to cover with tests: update (all non-array update operators), find (some tests might already exist, we don't repeat them)
[ ] Create a separate task to cover array update operators.

While working on dot notation tests I discovered that v.-1.bar is a valid path for MongoDB with -1 treated as a field name.
I'd like to suggest return an error in that case and create an issue for dance tests. @AlekSi @rumyantseva, are we ok with that?


Added the ability to change the weekday symbol type for the CVCalendarMenu

Hello, all! First pull request here. Would like to add the ability to change the weekday symbol type for the Calendar Menu. Uses an enum to allow easy selection. Selection is made on the delegate for CVCalendarMenuViewDelegate. 🖖🏻

Hello @roymckenzie,
Thank you for your useful contribution!
Please, could you update the changes to the most recent code on the master branch, so it won't be conflicting while merging?
Regards,
Eugene

Fixed @Mozharovsky!

Thank you so much once more! :+1:

Please, could you document the changes?

Hey @Mozharovsky I don't see documentation for the CVCalendarMenuViewDelegate. Can you point me to it and I will update?

@roymckenzie, there is no page on wiki but feel free to add one! :)

Ohh! I see what you are trying to do! :) Happy to help and I will try to add one this evening.

On Oct 13, 2015, at 12:57 PM, Eugene Mozharovsky<EMAIL_ADDRESS>wrote:
@roymckenzie https://github.com/roymckenzie, there is no page on wiki but feel free to add one! :)
—
Reply to this email directly or view it on GitHub https://github.com/Mozharovsky/CVCalendar/pull/127#issuecomment-147834323.


Add development environment setup guide for Fedora 28.

Steps provided for Ubuntu 18.04, did a test run on a VM this evening.
Fixed a small error in the Fedora docs, addressed the problems indicated by @karfai.


Avoid duplicating specs in the lockfile after updating w/ the gem uninstalled

Closes https://github.com/bundler/bundler/issues/5599
I decided to not compare using full_name in SpecSet#merge for the sake of performance, since #full_name is uncached

@bundlerbot r+


Add dispatch username and password to dapper_env

closed in favor of https://github.com/rancher/k3s/pull/1537


How to write loader width HMR?

`
resolveLoader: {
alias: {
'my-loader': resolvePath('./lib/index.js')
}
},
and
devServer: {
open: true,
compress: true,
hot: true,
historyApiFallback: true,
contentBase: resolvePath('dist'),
disableHostCheck: true
},
`
and my loader change, about demos do not update

It looks like you just deleted our lovely crafted issue template. It was there for good reasons. Please help us solving your issue by answering the questions asked in this template. I'm closing this. Please either update the issue with the template and reopen, or open a new issue.
Here docs https://webpack.js.org/concepts/hot-module-replacement/ and https://webpack.js.org/guides/hot-module-replacement/


Commit changes to dockers.json when PR is merged

This PR updates Docker Images to commit the list of updated docker images in dockers.json to the master branch when a PR is merged. A commit is signed by gatk-sv-bot and is made only if the dockers.json is updated.
Note that, by design, this commit does not trigger invocation of any Github actions.

Thanks @mwalker174 for the review! I like the idea of time-stamping image tags, and better to be consistent with GATK, I'll make a separate PR for that.


Translations and exceptions handling

Translation of sync modified files since today
Handling of exceptions if code is modified in the future

Hi,
I amended my previous pull request
I kept the handling of exceptions that could lead to issues in the future if code is modified
Best regards

Hi,
I think "TODAY" is fine for most of the synchronizations, but if the synchronization starts at 23:59 and ends at 00:05, after 00:00 it will be "YESTERDAY" instead of "TODAY", so I don't agree with changing FILTER_FILE_DATE_TYPE_. SYNC_BEGIN_DAY to FILTER_FILE_DATE_TYPE_TODAY.
The current code has a bug and will be fixed.
Best regards.

Hi,
No, I did not change it today, but "Since Today", that is from the current day at 00:00:00 till next day at 23:59:00
I also thought at it and changed my mind to not use "today"
So the labelling "Since today" is fine in the english, french and italian translations I changed (Russian is gogle, and I don't understand Russian)
I will look at your next changes
Best regards

Hi,
Changed to cover the period from 00:00:00.000 to 23:59:59.999 of the synchronization start date.
https://github.com/Sentaroh/SMBSync2/commit/8b8f35502526a212a1f7e9e2a77c255b48438a42
Best regards.

Hi,
I will fix it, please wait until we update it.
Best regards.

Hi,
SYNC_BEGIN_DATE in the File date filter now targets the start date of synchronization after 00:00:00.000.
https://github.com/Sentaroh/SMBSync2/commit/5b18a618cb820501b9b072e8506613cfa900ba4d
Best regards.


Draft: Add resources configuration to initcontainers

What this PR does / why we need it:
Allow to deploy that chart on Kube clusters which force to have resources configuration for every pods (even initContainers)
Checklist

[ ] Add ressources to other components
[ ] Update values
[ ] Update doc

@aigptteam Hi, thank you for providing this patch, please make sure that the sign-off is added before we cloud merge it.
check this link for detail https://github.com/zilliztech/milvus-helm/pull/60/checks?check_run_id=21731562172


some question about training

Hello, thank you very much for your wonderful work, but I ran into some issues while reproducing the code. When I was training, I couldn't save coarse_segmentation_gui_tar, and I didn't see the code to save this file when I checked the code. Could you please tell me how to save this file?

Hello, thank you very much for your wonderful work, but I ran into some issues while reproducing the code. When I was training, I couldn't save coarse_segmentation_gui_tar, and I didn't see the code to save this file when I checked the code. Could you please tell me how to save this file?


Use @EnabledIfOidcExists instead of several Gradle test tasks

Summary
The idea is to keep a single test task only, which would make IDE UI easier: it won't ask "which of test, testGithubOidc, testManual tasks would you like to use to execute the test".
Release Note
NONE
Documentation
NONE

A side-effect is that ./gradlew build would attempt executing test with oidcprovider in (ANY, MANUAL).
I'm inclined to add a property like skipOidc to allow skipping OIDC flows when testing on dev machine.

LGTM, but I would like a way to disable manual testing locally (calling to sign in)

disable manual testing locally

Run with -PskipOidc=true or add skipOidc=true to your $HOME/.gradle/gradle.properties.
Is that good enough?
PS https://docs.gradle.org/current/userguide/build_environment.html#sec:gradle_configuration_properties
PPS https://docs.gradle.org/current/userguide/build_environment.html#sec:project_properties

So this will work because: id-token:write will never be accessible to an external PR? But those test just wont run anyway because of these checks?

I'm inclined to add id-token: write to the regular ci.yaml

Id tokens are scoped to workflow, so there appears to be reduced risk is leaked.


[builder] COLRv0: Reorder glyphs in font to have empty glyph in GID1

Reacting to a Windows rendering bug, a COLRv0 font needs to have an empty glyph in GID1, such as the space glyph.
Could the builder make that happen in a post-processing stage?
I think it's not in the responsibility of a source author to ensure or even know that glyphs are to be ordered a certain way to react to stupid bugs.
The post-processing would happen on the binary, because AFAIK gftools has no control over the UFO sources, given that it can process several different kinds of sources.
I'm offering to write a fonttools script to make that happen.

It should already do this, https://github.com/googlefonts/gftools/blob/main/Lib/gftools/fix.py#L694-L708

Ah, just seen it may not be getting called due to https://github.com/googlefonts/gftools/blob/main/Lib/gftools/fix.py#L730-L732. Let me fix this.

Okay, let me check using the repo version of gftools

I just retried using the repo gftools, to same result. No reordering.

Ah, just seen it may not be getting called due to https://github.com/googlefonts/gftools/blob/main/Lib/gftools/fix.py#L730-L732.

From that code I don't see how it wouldn't run, honestly.

Alright, there's a bug in the existing code for making GID 1 a blank glyph. I'll take a look now.

may I just say that it would be much better if this fix were applied on the glyphOrder parameter (or public.glyphOrder lib key) directly in the input to fontmake instead of hacking the font post-build?
I know I myself suggested Marc to use nanoemoji reolder_glyphs functionality, but given that glyph order is something that touches basically all glyph-related tables it's a very deep surgery that should best be avoided if we can...

I would gladly merge a PR to ufo2ft that creates a glyphOrder with empty GID1 to be used for the build whenever we detect that the font will contain a COLRv0 table


[RFC] unset environment variable

In current implementation, vim doesn't have a way to unset environment variable. Most of programs works well with empty value as same as no-set variable. But for example, TZ variable
$ TZ=EST date
Tue May  1 00:45:54 EST 2018
$ TZ= date
Tue May  1 05:45:57 UTC 2018
$ date
Tue May  1 14:46:01 JST 2018
$

If vim doesn't have a way to unset environemnt variable, we can't reset TZ. Let's add a way to unset environment variables with unlet $FOO?

One another example.
let $TZ='EST'
echo localtime() " This is EST
let $TZ=''
echo localtime() " This is UTC

Once we set TZ, we can not get result of localtime in default time zone since ever.

This is dup of https://github.com/vim/vim/issues/1116 but no progresses.

closing as duplicate of #1116


This is for a config file for the darkmode settings to keep user's settings saved

Tweaked the readme to match original repo's readme.
Added in config file settings and the like.

Thank you a lot! Your pull request really help me! 😁


Quote and weather rotation does not work

https://forums.balena.io/t/inkyshot-variables-not-working/203394

fast fix

default --> quote only.
ALTERNATE_FREQUENCY set --> switch between quote/weather
DISPLAY_QUOTE = 0 --> showing only weather

substitute the last block of code in update_display.py after the comment
Alternate to other display code
if "DISPLAY_QUOTE" in os.environ and os.environ["DISPLAY_QUOTE"]=='0':
        os.environ["NEXT_DISPLAY"] = 'weather'
else:
        if(NEXT_DISPLAY == 'quote' and "ALTERNATE_FREQUENCY" in os.environ):
            os.environ["NEXT_DISPLAY"] = 'weather'
        else :
            os.environ["NEXT_DISPLAY"] = 'quote'

Hey @manuelemac thanks for chiming in. Would you like to open a pull request with your solution?
Then we could test it out and merge it once we confirm that this solves the issue.


SAK-45674 - Calendar: Replace UI by fullcalendar

Re-tested after all of the requested changes, works fine in Chrome, Firefox and Edge.


Code get unselected when menu pops up

Bug description
When I select a code (using visual selection with neovim plugin) and call the which-key menu, the code gets unselected and cannot perform action on selected code
To Reproduce

Select code
Call which-key

Expected behavior
I would expect the selected code to remain selected
System information

FWIW this doesn't happen with vscodevim. I don't have much experience with the neovim extension. But how did you call the whichkey extension?


Links in TOC are bad

They're missing /reference/ in their URL.

Having issues reproducing this to work on a fix, the build of F#F I'm getting right now doesn't seem to be replacing the {{list-of-namespaces}} template parameter in the html template.

Looking at the F#F code though, the doc comments on the UrlBaseName members (which are used to generate the namespace list html) explicitly mention not having the /reference/ segment on there, so it might be as simple as prefixing that wherever those members are used.

Though seeing how prevalent the use of those members are, I question the value of not prepending the /reference/ to the UrlBaseName, so they they are correct by construction.

Though seeing how prevalent the use of those members are, I question the value of not prepending the /reference/ to the UrlBaseName, so they they are correct by construction.

This is likely right.  BTW I've been trying to clarify the various namings or URL things in F#F - a persistent source of bugs - might be better to make a type distinction

I'm working on fixing this

A bit of a snafu on the docs right now, sorting it out now.  F# Formatting is setting root incorrectly.
Hilariously it means the old F# logo turned up on the site :-)
https://fsharp.github.io/img/logo.png

Fixed by https://github.com/fsprojects/FSharp.Formatting/pull/570


Fixes issue1248: Sort the tx inputs and outputs

The idea is to have a defined order for the inputs and outputs within a transaction. This is to make it deterministic so matches will more easily be made during nomination consensus and also make it not possible to re-arrange the transaction contents to create new hashes..

Rebased and should be green.

@hewison-chris Is the rule that the first output is a refund kept?
This rule has been added in the PR below. I'd like to check, please.
https://github.com/bosagora/agora/pull/1664

Yes, I checked. But please check if what I have in mind is right.
Only 40,000 BOA cannot be frozen on UTXO with 90,000 BOA.
About 50,000 BOA, which is the refund amount, is also frozen.

Only less than 40,000 BOA will be considered not frozen.
If you have UTXO with say 90,000 BOA then it needs to be split into two UTXO first if you want refund unfrozen.

Yes, I checked. But please check if what I have in mind is right.
Only 40,000 BOA cannot be frozen on UTXO with 90,000 BOA.
About 50,000 BOA, which is the refund amount, is also frozen.

Only less than 40,000 BOA will be considered not frozen.
If you have UTXO with say 90,000 BOA then it needs to be split into two UTXO first if you want refund unfrozen.

In order not to do so, we proceeded with the PR below.
Also, I think there is a problem with the fact that it is not clear which output is frozen.

Yes, I checked. But please check if what I have in mind is right.
Only 40,000 BOA cannot be frozen on UTXO with 90,000 BOA.
About 50,000 BOA, which is the refund amount, is also frozen.

Only less than 40,000 BOA will be considered not frozen.
If you have UTXO with say 90,000 BOA then it needs to be split into two UTXO first if you want refund unfrozen.

In order not to do so, we proceeded with the PR below.
#1664
Also, I think there is a problem with the fact that it is not clear which output is frozen.

It is clear which output is frozen. The outputs are sorted and then the first output that is less than the minimum freeze amount.

@hewison-chris I'll take a little more time to organize my thoughts.
And I'll let you know my opinion again.

@hewison-chris I'll take a little more time to organize my thoughts.
And I'll let you know my opinion again.

We can discuss more next week if you like.

Yes, I checked. But please check if what I have in mind is right.
Only 40,000 BOA cannot be frozen on UTXO with 90,000 BOA.
About 50,000 BOA, which is the refund amount, is also frozen.

Only less than 40,000 BOA will be considered not frozen.
If you have UTXO with say 90,000 BOA then it needs to be split into two UTXO first if you want refund unfrozen.

This method seems to be the most efficient solution in Agora.
There has been this discussion before, and I have asked the following questions.
If a user needs to split the funds first to freeze, he or she must wait (10 minutes) for the new block to create.
It can be difficult for users to understand and make it difficult to use a wallet.
In addition, unnecessary keypairs must be created in this process.
Therefore, I would like to ask you not only to consider the efficiency of Agora, but also to consider its convenience when used in Wallet and other tools.

Yes, I checked. But please check if what I have in mind is right.
Only 40,000 BOA cannot be frozen on UTXO with 90,000 BOA.
About 50,000 BOA, which is the refund amount, is also frozen.

Only less than 40,000 BOA will be considered not frozen.
If you have UTXO with say 90,000 BOA then it needs to be split into two UTXO first if you want refund unfrozen.

This method seems to be the most efficient solution in Agora.
There has been this discussion before, and I have asked the following questions.
If a user needs to split the funds first to freeze, he or she must wait (10 minutes) for the new block to create.
It can be difficult for users to understand and make it difficult to use a wallet.
In addition, unnecessary keypairs must be created in this process.
Therefore, I would like to ask you not only to consider the efficiency of Agora, but also to consider its convenience when used in Wallet and other tools.

@Geod24: Could we add a refund field to the Output struct. Then there would be no need to limit the size of the unfrozen amount when the transaction type is frozen?

@Geod24 If we don't know the refund output, how can we change the fee using the replace function?
https://github.com/bosagora/agora/pull/1881

Yes, I checked. But please check if what I have in mind is right.
Only 40,000 BOA cannot be frozen on UTXO with 90,000 BOA.
About 50,000 BOA, which is the refund amount, is also frozen.

Only less than 40,000 BOA will be considered not frozen.
If you have UTXO with say 90,000 BOA then it needs to be split into two UTXO first if you want refund unfrozen.

This method seems to be the most efficient solution in Agora.
There has been this discussion before, and I have asked the following questions.
If a user needs to split the funds first to freeze, he or she must wait (10 minutes) for the new block to create.
It can be difficult for users to understand and make it difficult to use a wallet.
In addition, unnecessary keypairs must be created in this process.
Therefore, I would like to ask you not only to consider the efficiency of Agora, but also to consider its convenience when used in Wallet and other tools.

@Geod24: Could we add a refund field to the Output struct. Then there would be no need to limit the size of the unfrozen amount when the transaction type is freeze?

The last commit of https://github.com/bosagora/agora/pull/2107 shows how it could work. (A test to fix still)


Scaling for all pages in the app

Adjusted scaling for the whole app and small grammar fix.
Description
App has values that didn't scale well, for example, if the width was 50, 50*1.25 = 62.5, and it would round up, and cause an offset of 0.5 pixels. I have gone through every page and tried to update the values to scale well.
Motivation and Context
Make the app scale well.
How Has This Been Tested?
Building the application
Types of changes

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

I think it would be better to merge and #313 and #314 into this PR since they all aim to achieve the same.

I think it would be better to merge and #313 and #314 into this PR since they all aim to achieve the same.

#313 has some changes from #312, but I could merge #314 into this

I think it would be better to merge and #313 and #314 into this PR since they all aim to achieve the same.

#313 has some changes from #312, but I could merge #314 into this

What I mean is that is better to have one PR that contains all changes from #312,#313 and #314 since they all are fixing the same thing. 😅

Closing as there are unaddressed comments. @salmanmkc - please feel free to submit a new PR if desired.

Closing as there are unaddressed comments. @salmanmkc - please feel free to submit a new PR if desired.


zug: add a patch to include algorithm

Fix error: 'sort' is not a member of 'std'; did you mean 'qsort'?
https://github.com/arximboldi/zug/pull/42

Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[x] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
25.05 Release Notes (or backporting 24.11 and 25.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

@ofborg build krita

@ofborg test zug


subsys/settings: Allow to register custom backends in settings subsystem

This is a rework of the settings subsystem based on closed PR #12998
Major changes are:

Expose settings backend API to enable custom backend support.
Add a new CONFIG_SETTINGS_CUSTOM backend to allow registering a custom
backend.
Change api of the handlers h_set() routines to allow for
backend-specific read callbacks.
Provide a customizable settings_backend_init() routine for custom
backends.
Move runtime settings support to be its own backend.

Signed-off-by: François Delawarde (FNDE<EMAIL_ADDRESS>

It looks like there are a lot of unrelated changes to bluetooth, mostly code style changes which probably should be either left alone or submited in a different patch.

@Vudentz, Sorry about that, I followed the contribution guidelines and ran uncrustify and checkpatch on all files in the diff, and the previous contributor might have skipped that step. I will try to move those unrelated code style changes into another commit.

@nvlsianpu can you take a look at this one?

Merging this one as requested by @aescolar as most of the needed people are in wrong timezone.


Edit message from another stream

Hello,
At the moment it is possible to edit a message in a stream from another stream. This seems to be an effect of #2757 .
Preview:

@zulipbot claim

This is an old issue.
There are several PR to fix this
https://github.com/zulip/zulip-mobile/pull/2911


Vector of Pairs for PlutoUI.Select not parsed properly?

Or am I doing something wrong?
The options input should be of the form ["Display Value" => assigned_value] with assigned_value isa Any, right?

If you see from the documentation open on the helpbox it does mention that what is returned from the @bind is the key and not the value of the pair.
I do agree with you though that it does seem a bit counterintuitive as behavior

Exactly. Note that the type of the argument is key::String => value::Any, which means that it's disallowed to have non-string keys.
You can easily find the value you want by doing
options = ["1" => "one", "2" => "two"]
@bind selectedkey Select(options, default=options[1][1])

--- # Another cell
value = options[ findfirst(x -> first(x) === selectedkey, options)][2]

Thanks guys. Yes, I did notice the help doc, and I was wondering if it was just a mistaken decision, or optimal given the context of a notebook.
Is it designed this way because in Pluto notebooks, you cannot assign, say, a custom type to the @binded variable?
Also, what is the purpose of having each second in the input Pair be type Any? It just converts it to String anyway. It would definitely make more sense for even an integer to be assigned to the bound variable. It's not even possible to do something like, say,
@bind x parse(Int64, Select(["1" => "one", "5" => "five"]))

because of course Select doesn't output the integer value selected.
I'd be happy to close this issue seeing as this design seems intentional, but just wanted to check and ask if it's possible to design it the more intuitive way.

FWIW, I use the following pattern with basically all Select-like widgets:
x_options = OrderedDict(["str1" => object1, "str2" => object2, ...])
x_sel = @bind x_str Select(collect(keys(x_options)))

x = x_options[x_str]

It would be nice if there was an API that achieved this directly in the @bind line.

See #3 , which will be fixed by #148


[BUG] No way to specify an alternate transfer-pvc image

What version of crane are you running, and what are your clutsters+platform
0.0.4 dev
What did you expect to happen?
There's no way to override the transfer-pvc image.
What actually happened?
There's no way to override the transfer-pvc (rsync) pod. This means it always pulls from quay.
Please include any relevant logs or errors
It looks like crane-lib has a default set that can be overridden, but it is not exposed in crane.
I think we need to roughly approximate the tunnel-api options:
     --destination-image string     The container image to use on the destination cluster. Defaults to quay.io/konveyor/openvpn:latest
     --source-image string          The container image to use on the source cluster. Defaults to quay.io/konveyor/openvpn:latest

This will be important for downstream as well, since we'll probably have many cases where pulling from quay is not desired or even possible.

Need to consider impact to https://github.com/konveyor/crane-operator ... specifically if we need to add "relatedImages" to the CSV.

We have relatedImages for most of these already and logic for handling image overrides.
You are right...this would just be another on that list.

--source-image and --destination-image flags are now available. Issues have been opened for relatedImages


Stuck when split editor

Description
Stuck when split editor
To Reproduce

Open a file
Click the split button to split the editor

Expected Behavior
No stuck
Version Information
CodeEdit: [e.g. 0.0.x-alpha.y] 0.0.3-dev (37)
macOS: [e.g. 14.0] 14.0
Xcode: [e.g. 15.0] 15.0
Additional Context
I add a runtime issues breakpoint and it looks like entering a infinite loop here.

Screenshots
https://github.com/CodeEditApp/CodeEdit/assets/22616933/4417c6ad-a64c-4295-b271-4bc37784e686

Closed. Duplicate with #1545


Add Suspension Type column to source datasets

Add After "Assay"
Add on the pages:

Source study/id/data-sets
Component Atlas linked source datasets  e.g. https://tracker.data.humancellatlas.org/atlases/id/component-atlases/id
Source dataset linker popup e.g.  the popup from selecting "Link Source Dataset" button from the screen above.

Complete thx