useProgress HLS bug on IOS

Describe the Bug
A clear and concise description of what the bug is.
Duration returned from useProgress while playing HLS live stream is not correct. All is well on android as I have observed.
Steps To Reproduce
How can someone else reproduce this bug?

Add an HLS track to Trackplayer and play.
Monitor the useProgress output via console.

Code To Reproduce
Please provide a simple code example that allows others to replicate the bug.
Trackplayer.add({
    ...track,
    type: "hls",
    isLiveStream: true,
    url: file.m3u8,
}).then(() => {
    Trackplayer.play();
});

const info = useProgress();
console.log(info); // normal if live stream is just starting, try joining at a much later time
Trackplayer.seekTo(secs); // triggers the issue

// ios
 LOG  {"buffered": 6700.753022675737, "duration": 17.972244897959182, "position": 6683.826420957}
 LOG  {"buffered": 6700.753022675737, "duration": 17.972244897959182, "position": 6683.826420957}
 LOG  {"buffered": 6700.753022675737, "duration": 17.972244897959182, "position": 6683.826420957}
 LOG  {"buffered": 6712.780959183673, "duration": 30.00018140589569, "position": 6674.442}
 LOG  {"buffered": 6712.780959183673, "duration": 30.00018140589569, "position": 6674.442}
 LOG  {"buffered": 6712.780959183673, "duration": 30.00018140589569, "position": 6674.442}
 LOG  {"buffered": 6712.780959183673, "duration": 30.00018140589569, "position": 6674.442}
 LOG  {"buffered": 6712.780959183673, "duration": 30.00018140589569, "position": 6674.442}
 LOG  {"buffered": 6671.623173922902, "duration": 0.8359183673469388, "position": 6670.85258876}
 LOG  {"buffered": 6671.623173922902, "duration": 0.8359183673469388, "position": 6670.85258876}
 LOG  {"buffered": 6688.782720408163, "duration": 17.99546485260771, "position": 6671.867501169}
 LOG  {"buffered": 6688.782720408163, "duration": 17.99546485260771, "position": 6671.867501169}
 LOG  {"buffered": 6694.773468707483, "duration": 23.986213151927437, "position": 6672.900312556}
 LOG  {"buffered": 6694.773468707483, "duration": 23.986213151927437, "position": 6672.900312556}

// android
 LOG  {"buffered": 0, "duration": 0, "position": 0}
 LOG  {"buffered": 0, "duration": 0, "position": 0}
 LOG  {"buffered": 30.014, "duration": 42.004, "position": 24.004}
 LOG  {"buffered": 30.014, "duration": 42.004, "position": 24.004}
 LOG  {"buffered": 42.004, "duration": 42.004, "position": 24.004}
 LOG  {"buffered": 42.004, "duration": 42.004, "position": 24.004}
 LOG  {"buffered": 42.004, "duration": 42.004, "position": 24.004}
 LOG  {"buffered": 42.004, "duration": 42.004, "position": 24.004}
 LOG  {"buffered": 48.011, "duration": 48.011, "position": 26.436}
 LOG  {"buffered": 48.011, "duration": 48.011, "position": 26.436}
 LOG  {"buffered": 50.055, "duration": 54.008, "position": 29.905}
 LOG  {"buffered": 50.055, "duration": 54.008, "position": 29.905}
 LOG  {"buffered": 54.008, "duration": 54.008, "position": 33.554}
 LOG  {"buffered": 54.008, "duration": 54.008, "position": 33.554}
 LOG  {"buffered": 54.008, "duration": 54.008, "position": 33.677}
 LOG  {"buffered": 54.008, "duration": 54.008, "position": 33.677}

Environment Info:
System:
OS: macOS 12.3.1
CPU: (8) x64 Intel(R) Core(TM) i7-4870HQ CPU @ 2.50GHz
Memory: 28.16 MB / 16.00 GB
Shell: 5.8 - /bin/zsh
Binaries:
Node: 14.17.0 - /usr/local/bin/node
Yarn: 1.22.18 - /usr/local/bin/yarn
npm: 6.14.13 - /usr/local/bin/npm
Watchman: 2<IP_ADDRESS> - /usr/local/bin/watchman
Managers:
CocoaPods: 1.10.1 - /usr/local/bin/pod
SDKs:
iOS SDK:
Platforms: DriverKit 21.4, iOS 15.4, macOS 12.3, tvOS 15.4, watchOS 8.5
Android SDK: Not Found
IDEs:
Android Studio: 4.2 AI-202.76<IP_ADDRESS>86908
Xcode: 13.3/13E113 - /usr/bin/xcodebuild
Languages:
Java: 11.0.11 - /usr/bin/javac
npmPackages:
@react-native-community/cli: Not Found
react: 17.0.2 => 17.0.2
react-native: 0.66.2 => 0.66.2
react-native-macos: Not Found
npmGlobalPackages:
react-native: Not Found
<EMAIL_ADDRESS>

Are you able to provide an HLS stream and playlist for us to test with?

@jspizziri  Thanks for the quick reply, and yes I sure do

@praisedavid787 if its something that we could add to this public repo to test HLS streams that would be super helpful.

@jspizziri I sent you an email about the constraints of adding this to the repo. Stream comes up on demand.
https://u-wayserver.herokuapp.com/streams/live/oLJK2eFvDTiOp0_wcfUBGPAa/index.m3u8

@jspizziri
https://pl.streamingvideoprovider.com/mp3-playlist/playlist.m3u8
This is one for a VOD stream that I have also tested with and gives desired behavior as it not a live stream.

@praisedavid787 Thanks for sending these. I'm currently taking some time but will try to look once back!

@jspizziri Anytime man, thank for all the help. Hoping things look up soon. Huge blocker for me this one. It's fine at the start but once I seekTo(), boom! Messed up.

Trying to be a hero but not familiar though. Something looks fishy when comparing the last else if statement in var duration with var bufferedPosition's return.  AvPlayerWrapper.swift in SwiftAudioEx
    var duration: TimeInterval {
        if let seconds = currentItem?.asset.duration.seconds, !seconds.isNaN {
            return seconds
        }
        else if let seconds = currentItem?.duration.seconds, !seconds.isNaN {
            return seconds
        }
        else if let seconds = currentItem?.loadedTimeRanges.first?.timeRangeValue.duration.seconds,
                !seconds.isNaN {
            return seconds
        }
        return 0.0
    }

    var bufferedPosition: TimeInterval {
        currentItem?.loadedTimeRanges.last?.timeRangeValue.end.seconds ?? 0
    }

@praisedavid787 Thanks for taking the initiative! You can try putting in some print statements or breakpoints and you should be able to debug the lib via Xcode. I can provide pointers there if you like.
@dcvz  any pointers or insight off of the top of your head?

@praisedavid787 another thing I'm wondering is if this is an issue with your HLS playlist metadata (I'm not an expert on HLS). But I did find that you can add the playlist length as a metadata field.
https://stackoverflow.com/questions/38216051/hls-fragment-length-and-playlist-length-with-nginx-rtmp-module

@jspizziri Eureka!!! After much research, implementation and testing, I have found what works:
     var duration: TimeInterval {
        if let seconds = currentItem?.asset.duration.seconds, !seconds.isNaN {
            return seconds
        }
        else if let seconds = currentItem?.duration.seconds, !seconds.isNaN {
            return seconds
        }
        else if let seconds = currentItem?.seekableTimeRanges.last?.timeRangeValue.duration.seconds,
                !seconds.isNaN {
            return seconds
        }
        return 0.0
    }

@praisedavid787 Nice work! The next step would be to open up a PR against the swiftaudioex repo with your changes. Then we'll get the feedback from the team there.
Kudos to you. For taking the initiative on looking into this!

@jspizziri Thanks for all the help, much appreciated.
For the PR, I'll have to fork the repo, is it?

Correct, you fork the main repo crest a branch off of main implement your change there, then push it up to your fork. Then you creat a PR from your repo to the main repo. Probably some better walkthroughs than that on the internet too haha

Makes a whole lot of sense man, thanks. See you at SwiftAudioEx!

@praisedavid787 Let's leave this issue open, as if it gets merged and released in swiftaudio we'll need to inevitably update this project too by upgrading the dependency version.

Ok sir, you're the boss man. Pull request up https://github.com/doublesymmetry/SwiftAudioEx/pull/18.

Great work tracking this down @praisedavid787! and for the help @jspizziri :)
The PR has been merged and I'll make a SwiftAudioEx release later today!

@praisedavid787

@praisedavid787 warch for the version release on swiftaudioex. Once it's been made you'll need to submit another PR to this repository updating the version number here
https://github.com/doublesymmetry/react-native-track-player/blob/main/react-native-track-player.podspec

Thank you @dcvz and @jspizziri for making things easy! Cheers to more collaboration for a better RNTP!

@praisedavid787 Here's the PR that solves the issue for this repo now that a new version is release for swiftaudioex.
https://github.com/doublesymmetry/react-native-track-player/pull/1534

@jspizziri Ok great! Anything you'll have me do?

No you're good. Just an fyi

The fix has been merged and will be released in the next RC

https://pl.streamingvideoprovider.com/mp3-playlist/playlist.m3u8

how did you created this play list. Usually, HLS provide .ts file but you have .mp3 file


[1.22] manual backport: allow missing svc port for sandwich #1054

Manual backport of https://github.com/istio/ztunnel/pull/1054

Closing for staleness


Update to Swift 4.2

Also updates the project settings and couple variable's access control to conform to new things.

Thanks!


Update: fix endpoint search bug.

@hanahmily In dev mode. going to endpoint page , and select one serivce, now select other service,this will case a type error. I found it was cased by the state data.


General banner

Adds a general banner to the site. Also reworks how the cookie banner works, will now use actual cookies and middleware to avoid layout shifts on initial load.
Needs extensive testing of links on the site before merging.

[ ] Replace all Link components with custom link component (strip state from links)
[ ] Update min side to work with new router

Tested on devices

[ ] Desktop 💻
[ ] Mobile 📱

Tests

[ ] All tests are running ✔
[ ] Test are updated 🧪
[ ] Code Review 👩💻
[ ] QA 👌

Checkpoints
Check these to flag for a more thurough review, as they could be potentially breaking changes

[ ] Packages updated
[ ] Other infrastructure updated (such as node version or similar)

⏲ Time spent on CR:
⏲ Time spent on QA:

When I click the logo the page does not respond (might not be related to banner).
Will the banner re-appear on re-load or is that only in the deployment?

Probably not worth spending time on now, we can keep it in mind for later ad se if people are bothered by it!


Fix CS.

Code sniffer is not happy about the additional unneeded newline in the docblocks.
This solves it.

Can we merge this?


Adding a view by option

Related Issue
Closes #56
Type of Change
Put x inside the square bracket to specify what type of change your PR is:

[x] New Feature
[ ] Bug Fix
[ ] Code Refactor
[ ] Documentation Update
[ ] Other (please specify):

Description of Change
Adding a View by option with the following values:

Category

Status

Demo
Screencast from 2024-12-22 03-11-39.webm

Hi @disvid thankyou for submitting.

Please change the PR title to something meaningful.
resolve merge conflict

@AyushDharDubey Done sir.

good work @disvid,
but although your fix is working, we won’t be able to merge this PR as there’s a protocol in place that requires all contributors to adhere to the rules outlined in our CONTRIBUTORS guide.
in this regard, please rename your branch to the required format and submit a new PR.

@AyushDharDubey Sir, somehow the PR gets closed as soon as i rename the branch, so i am creating a new PR for the same.


added check to stop printing parenthesis if the default is nil

Fix for #1834
Was planning on writing tests, but I don't really understand what's happening at question_spec.rb.
Short checklist

[x] Did you run bundle exec rubocop -a to fix linter issues?
[ ] If relevant, did you create a test?
[x] Did you confirm that the RSpec tests pass?
[x] If you are fixing a bug or introducing a new feature, did you add a CHANGELOG entry?

Looks like the new version of RuboCop is unhappy. I'll investigate.

@chamini2 Can you please rebase on top of the latest master? That should clear up the RuboCop errors. Thanks for the PR!

Done, @mattbrictson, something else that needs to get done?

Thanks! Could you also add a simple test for the no-default case? There are existing specs in question_spec.rb that you should be able to amend. Otherwise it looks good. 👍

@mattbrictson, added tests and rebased from master. I think it should be ready now? Let me know!

@chamini2 It looks like @capistrano-bot is still upset. Can you put back the * Your contribution here! line that you removed?
Once that it is green I'll merge the PR. Thanks!

Done! @mattbrictson

🙌 Thanks


create disabled version of the select

In GitLab by @01oseluiz on May 26, 2022, 18:37

mentioned in commit 9c38b08424fb337e4b9c42f01f78fb6ded51072c


Use Hamcrest assertions instead of JUnit in  tests/integration/native-image/se-1 - backport 2.x (#1749)

Part of #1749
Original PR

/trigger


Added localization support

Hi, I added some new features to the menu.

Added localization support (The menu will detect game language automatic)
Chinese translate (zh_CN and zh_TW)

Thanks for your great work!

Thanks for the contribution, glad to see the menu still being used :D I'm gonna build and test your features in a week or so once I have the time and then merge it if everything's fine.


When jquery-1.12.0 is used with vue2, the jQuery click event can not be executed

Vue.js version
2.0.2
`

onetwo
threefour

`
` $(document).ready(function(){
$("#table1").find("tbody").click(function() {
console.log('click running@@');
});
	var  vm = new Vue({
	      el:"#table1",
		  methods: {
		  }
	  });
 });`

When you do not call vue, the click event can be executed, call vue, click event can not be executed, vue2.0 and jquery can not be used together?

This is not a bug. , Vue replaces the elements after you added events to them in domready. Hint: call jquery code in the mounted() hook.
We encourage you to ask further questions on the forum  , Stack Overflow or on gitter and are happy to help you out there.

@LinusBorg  Vue replaces the elements after you added events to them in domready。why  replace  elements ？

@gcyStar, vue controls the DOM it's own way and you should never rely on DomElements or mutate DOM directly. Most times what you want to achieve can be done "vue-way", if you need more information please ask on the forum, Stack Overflow or on gitter. I prefer gitter and there's always someone online willing to help.


SDFSOF-92 - Updated actions to handle sep31 refunds and error handling

PR Checklist
PR Structure

[ ] This PR has reasonably narrow scope (if not, break it down into smaller PRs).
[ ] This PR avoids mixing refactoring changes with feature changes (split into two PRs
otherwise).
[ ] This PR's title starts with name of package that is most changed in the PR, ex.
paymentservice.stellar, or all or doc if the changes are broad or impact many
packages.

Thoroughness

[ ] This PR adds tests for the most critical parts of the new functionality or fixes.

What
Updated actions to handle sep31 refunds and error handling
Why
Replace PATCH /transactions used for SEP-31 flow
Known limitations
[TODO or N/A]

Reference Server Preview is available here:https://anchor-ref-pr1010.previews.kube001.services.stellar-ops.com/SEP Server Preview is available here:https://anchor-sep-pr1010.previews.kube001.services.stellar-ops.com/

Reference Server Preview is available here:https://anchor-ref-pr1010.previews.kube001.services.stellar-ops.com/SEP Server Preview is available here:https://anchor-sep-pr1010.previews.kube001.services.stellar-ops.com/


Support new REST API endpoint that evaluates if private vulnerability reporting is enabled

GitHub blog announcement: https://github.blog/changelog/2024-03-08-check-if-private-vulnerability-reporting-is-enabled-via-rest-api/
New GET endpoint: https://docs.github.com/en/rest/repos/repos?apiVersion=2022-11-28#check-if-private-vulnerability-reporting-is-enabled-for-a-repository

Thank you, @diogoteles08 !
Do you want to create a PR for this, or shall I open it up to other contributors?

No problem! ^^
Well I might take some time to implement it, as I'll probably use this feature in my project. But wouldn't be able to start right now, so let's leave it open in case someone else can start this work sooner than me.

gmlewis I'd be happy to take a look at this and see if I can implement a REST API endpoint to check if private vulnerability reporting is enabled. Is diogoteles08 currently working on this? If not, I'm ready to jump in.

Is @diogoteles08 currently working on this? If not, I'm ready to jump in.

I'm not! Please feel free to work on this, that'd be great 😊


Nomination Form - Demographics Updates

[x] Display a message to nominators if they enter a child that is older than fourteen years old. It will still allow the user to submit the form.
[x] Move "reason for nomination" away from child; put on head of household section instead. Make a required field.
[x] Change child age calculation to be based on their age on Christmas instead of the current date.
[x] Add text around the wishlist text area to only enter toys that are under $50
[ ] ~Change "# in household" to "Number of kids being nominated"~
[ ] ~When submitting the form, check whether the number of children actually entered does not match the number entered in the Number of kids field, alert the user of the discrepancy. We don't want to automatically calculate the number of children nominated; the field serves as a double-checking tool for the nominator.~

I am pretty sure that we got rid of the # of children field...striking through those requirements.
@Jillzey

@chimon2000 Those fields are actually supposed to be new for this year. They help the counselors verify that they filled out the digital form for as many children as they claimed they would be nominating. There has been a tendency for them to once in a while miss porting a child from the nomination form. Unless y'all decided against this in one of your discussions
cc @Jillzey

@AndrewNatoli ah ok.  Since this field is purely for validation purposes, I am guessing that we don't actually need to save it to the database?

@chimon2000 Since forms can be saved as drafts I think we'll have to save the field to the database. Additionally, the forms can be revised at a later date if a child is added / removed to the nomination for any reason.


Fix all warnings

@BGluth do we still need this... are they warning on build?

@vladimir-trifonov Nah I think we're good to close once #162 is merged in, since I think you also ended up cleaning up all of the warnings?


Unquote strings when expanding concat!

Fixes #7417.

Should we add a test here?

Should we add a test here?

It's already there :-).

It's already there :-).

bors r+

bors r+


feat(core): adopt WebHDFS LISTSTATUS_BATCH for better performance

Description

fixed: #1938

Background
Use the WebHDFS Iteratively_List_a_Directory API in WebHDFS as the default listing behavior, and fallback to One-time Listing API if the server doesn't support batch listing.

BTW, same dilemma on whether to use #[derive(default)] or impl Default. My first implementation used the derive macro, but it required four other sub struct/enum to implement the default macro. Anyway, I've learned a lot from these PR practices. I'll ask for help when I'm stuck again.

I'll ask for help when I'm stuck again.

Feel free to ask for help. I am willing to answer any questions you may have!


fix: adjust code to get rid of calc_cu() returns 0 case

related to:

https://github.com/threefoldtech/tfchain/issues/448

PR need to be reopened after a revert commit operation  https://github.com/threefoldtech/tfchain/commit/d883ca007dd58c45425e4eac17a7b655107ce2a5 due to new tfchain tag version https://github.com/threefoldtech/tfchain/commit/864ea673258360e22b79dc3acf9e3ece48e5ef27


Support for reading Dockerfile contents as args or piped in

This is in response to https://github.com/replicatedhq/dockerfilelint/issues/10

I don't think this quite works as expected. More info coming soon.

This allows the contents of a file to be passed in as an argument (instead of requiring a file name to be given), but it doesn't yet handle piping contents or stdin. I will follow up with a PR from this branch soon.


Implement a StrataSampler

To implement model fairness, need a pipeline component that can balance a dataset based on a combination of X input strata dimensions rather than by y target classes as done in scikit-lego's RandomOver/UnderSampler.

Committed to: https://github.com/eangius/scikit-lab/commit/d7b85373c9003906d07c7bcb91f118b164b2e80b


false positiv

Paper 1.20.1
ClickableLinks  1.1.5

normal text is recognised as a link

This plugin re-implements detection exactly as it was done prior to 1.19.1. It is not meant to be good, rather to retain what was there before.
I suppose exposing the URL regex may be a solution to this, but I really would prefer not to add a configuration file here.
You can edit the regex in the plugin (it is only a few lines and one class) and then compile with mvn package should you want different detection behaviour.

What exactly would have to be adapted for this?

It depends what you want. Detecting what is and isn't a URL isn't as easy as it seems.
The simplest solution to the issue you pointed out with test.test could be resolved by removing this ? here:

which would make it only detect links that start with http or https.
See this stackoverflow answer (or just google "url detection regex") to learn more about other options https://stackoverflow.com/questions/1500260/detect-urls-in-text-with-javascript

Thank you very much, I will try it out <3

I thought about this some more and I really do not want to add a config for this plugin-- just to keep it as simple as possible in restoring previous (pre-1.19.1) behaviour.
I hope you were able to resolve the issue.


[Feature] Multi-Machine Support for Distributed Inference

The growth in size of open-source models is outpacing the growth of memory capacity of Mac computers. The latest 70B version of Llama 3 is already pushing the limits of a fully loaded Mac Pro. The upcoming 400B version of Llama 3 will exceed available memory entirely unless heavily quantized.
While memory limits may increase in future Mac Pro and Mac Studio models, it is likely that LLMs will continue to grow in size at an even faster rate. This poses a challenge for running the latest large open-source models with MLX. Without changes, MLX could be restricted to handling small to medium-sized models or heavily quantized versions of large models, resulting in inevitable inaccuracies.
MLX may become unsuitable for scenarios where local GPT-4-equivalent open-source models are necessary for cost and/or privacy considerations. I'm particularly thinking of SMB businesses and power users.
If we discard lossy options like quantizing, there are alternative approaches to consider:

Optimizing memory usage. For instance, Air_LLM, which involves loading and unloading layers. It is unclear if every LLM supports unloading entire layers, and this method may be inefficient since the layers have to be cycled for each generated token.

Implementing multi-machine support for distributed inference, where inference is distributed across multiple Macs. I shared a tweet about this possible solution and received significant interest, even though it was just a spontaneous idea. One way this approach could function is by splitting the model across multiple Macs (Mini, Studio, or Pro) connected via IP over Thunderbolt.

I am not proposing a definitive solution, but if there is interest in this topic, this discussion could serve as a starting point for further exploration of the possibilities.

"Implementing multi-machine support for distributed inference": I am not sure how this would work in technical detail but this is an idea that would be very attractive in the context of using MLX for physics/engineering ML/AI+HPC applications.

I think we should extend the concept to distributed training and inference, like the deepseed library. This can enable incredible scenarios powered by Apple Silicons chips.

As a first step we are looking at adding the communication primitives you would use to implement both of these. Ops like send, receive, broadcast, reduce etc. Basically the ops in this table / MPI.
Both distributed training and inference should be implementable on top of those ops. Exactly what those APIs look like and where they live is still TBD. But we need those ops as a first step either way so we can do distributed work with MLX in a flexible way.

@awni That sounds like a good and doable first step. If I have time this week, I'm happy to take a first stab at it.
FWIW, there is a relevant discussion about how the accuracy of Llama 3 model may be disproportionately affected by quantization, which might be a side effect of using a large training set. This would be another argument for distributed inference. See
tweet and PR.

This is something I'd be happy to test and contribute to as well. I remember seeing the original tweet, and just now got the Thunderbolt 4 cable connected across two macs (macbook pro and a m2 ultra).
Can see this use case being common for Apple users with a work and home machine.

@awni That sounds like a good and doable first step. If I have time this week, I'm happy to take a first stab at it.
FWIW, there is a relevant discussion about how the accuracy of Llama 3 model may be disproportionately affected by quantization, which might be a side effect of using a large training set. This would be another argument for distributed inference. See tweet and PR.

From what i've read, this is a llama.cpp issue, more than just llama3 quants.

In progress #1097

In progress #1097

amazing. so ready.

So exciting to see this moving along!!  We have several Studios with M2 Ultras in the lab that we could hookup to test distributed computing  when we reach that point of maturity. Would be happy to get involved in the testing.

How cool would it be being able to run 70B models by distributing them over a few ipads, iphones and whatnot? That would give new life to older devices we don't use anymore. Not to mention that it could create a paid model where users could give some of their GPU time for $ :D I am so looking forward distributed inference with MLX!

I know this is an old thread. But for any of the folks above that are looking to try something out: https://github.com/exo-explore/exo supports distributed inference using either (or both) MLX and tinygrad backends (I've used it to successfully run the big Deepseek-Coder-v2 (236B parameters) across multiple Mac machines on a local network with pretty good speed on MLX)


Transition HashCodeCombiner to use the aspnet/Common variation.

Updated LocationTagged<TValue> to handle null implicit values.
Removed InternalsVisibleTo declaration on Microsoft.AspNet.Razor to the runtime test project.
Updated TagHelperDescriptor tests to utilize helper methods to construct TagHelperDescriptors. This was needed since the InternalsVisibleTo declaration was removed.
Formatted project.jsons in different commit.

#449

/cc @anurse

:shipit:

:watch:

:watch:
Suggestion: If we're going to rewrite tests to avoid a few internal "for testing only" constructors, it should be done separately, because it's necessary or at least because we've discussed the point and agreed (grudgingly is fine of course :smiley_cat:). Further, we should completely remove the constructors; not update one of the two test projects using them to avoid a harmless IVT attribute.

@dougbu The internal ctors are just one of the things used by Microsoft.AspNet.Razor.Tests. The IVT to the Microsoft.AspNet.Razor.Test project is used for muchhh more than internal ctors (method calls, classes etc.). Not seeing a compelling reason to break this PR apart, it's relatively small.

Not seeing a compelling reason to break this PR apart

We do individual commits / PRs for individual issues. My suggestion is to at least split the commits to match the two unrelated things in this PR::

Use the new aspnet/Common HashCodeCombiner
Remove IVT from Razor to Razor.Runtime.Test

Closing out this PR since the HashCodeCombiner bit was signed off on. WIll open another one for the IVT since that seems to have caused the most bit of conflict.


Fix a crash from empty-line-between-blocks

The tree traversal code that was checking for space between blocks in
.sass files was a little too lenient when checking where it should
traverse and was actually moving into the content of an ident node
when there wasn't an empty line after an @include. This looks like:
.item
  @include my-mixin
  color: blue

The tree traversal is looking for a space node and accidentally walks
into the ident node for my-mixin.
To prevent this, we now check to see if the parent is a node before
walking into it. If the parent isn't a node, then we should halt
execution and return a violation.
I have checked whether this problem happens in .scss files as well and
it appears it does not due to the presence of the braces for the blocks.
This leads me to believe that the problem is purely due to relying on
whitespace for delineate blocks.
Fixes #729
Fixes #793
<DCO 1.1 Signed-off-by: Michael Herold<EMAIL_ADDRESS>

Coverage remained the same at 97.528% when pulling 078060d39bb68a3c54964e63e8936e84f66cf54e on michaelherold:fix-empty-line-crash into deabaf663ccd268f8a979ccd4ca995123145d1af on sasstools:develop.

@michaelherold awesome, thanks for this. Our develop branch is currently a bit of work for V2 if you'd like to get this out sooner then you could rebase it off of our v1-dev branch and force push it back here and then change the base of your PR to v1-dev which would probably see it released this week. If not or I don't hear back from you then i'll just merge this to develop anyways and I think our v2 will be out soon too to deprecate some node versions with v3 then coming shortly after with a few rewrites. Up to you though.
Thanks again for the contribution! 👍

Whoops, I didn't notice that - I don't think it was in the contributing documentation at all.
It looks like #1125 fixes the same problem in a more elegant way. When I checked out the v1-dev branch, I ran the regression test that I wrote against HEAD and the test passed. That must mean that the patch just wasn't forward-ported.
Would you like me to push a commit with the regression test based onto v1-dev? Also, I can forward-port the patch in #1125 along with my regression test onto develop for your v2 if you'd like.
Please let me know how you'd like me to proceed.


fix(test-editor-events): add missing module index file

The plugin was actually not correctly working without the index.js of the bpmn-js extension.
This fixes it.

Is there any context issue or bug report? It is hard for me to understand / validate what is fixed.

I just found it when dealing with this forum thread ==> trying out the plugin and discovered it's not working properly (there was nothing logged when shapes were added).


Make walking-controllers compatible with robotology-superbuild

The aim of this PR is to make walking-controllers compatible with robotology-superbuild
More specifically:

use FindqpOASES.cmake for finding qpOASES #3
use FindThreads CMake module instead forcing using pthread #4
general refactoring of the folder structure #6
update README.md #2

I also see that WalkingModule.hpp includes iCub related stuff (integrators and filters) but it is not linked in the CMakeList. This seems to be an actual dependency.

Thanks!


BUG: Should allow numeric mysql table names

Closes #10255 .
Rewrote some of the code in test_sql.py so that it now properly drops tables with names that need to be quoted.

@artemyk Thanks!
Can you add a small note in the 0.16.2 whatsnew bugfix section?
And for the rewriting of the tests, I was thinking it would maybe be better to keep this for a separate PR if you don't mind? As this would give rahter some conflicts with the connection PR, and I am already slow with reviewing that one. So I would like to first get that one in

@jorisvandenbossche OK, I will submit another PR.

@jorisvandenbossche Ping.  This makes the testing code a bit more robust by making sure all tables are dropped after every test.

Whoops, forgot this one. Looking at it now

Thanks!


Where is me.tatarka.bindingcollectionadapter.sample.databinding.RecyclerViewBinding?

Your example app references an import:
import me.tatarka.bindingcollectionadapter.sample.databinding.RecyclerViewBinding;

However, I cannot find this class anywhere.  Is this supposed to be automatically generated or something?

That class is generated by databinding. Note: you are more likely to get a response for general databinding questions on somewhere like StackOverflow


Fix side effect of out-of-range temperature filtering

At the moment if a change event from an underlying has an out-of-range temperature, then the whole event is ignored. However this can lead to inconsisent states as relevant changes from the underlying might be ignored, i.e. hvac_action.
In this case the underlying will be in state off/idle, while the vtherm will stay in state off/heating. This will resolve itself eventually when another type of event comes in, e.g. from a temperature sensor. But until then it is confusing and annoying.
With this change just the detected target temperature change is filtered out, the other fields of the new underlying state can still be processed.
I'll add or change an unit test to cover this case, but won't have time for it until tonight. So please don't merge yet :smiley:

Merged thank you !

I could reopen it if you prefer.


Update package dependencies

Keeping things up to date.
Also removes coveralls as a dependency and uses it for builds only.

Coverage remained the same at 100.0% when pulling 61b96694a37d87c64ee6a664af4fb145f021df67 on fix/update-deps into 4d28431a03ec97adba62a5196161b93782871b53 on master.


Center custom logo horizontally

Is your feature request related to a problem? Please describe.
I'd be great if custom logos that don't exactly fit the measurements (155x55 px) could be centered horizontally.
Describe the solution you'd like
Logos that don't exactly fit the measurements e.g. 155x35 px should have a 10px margin on top and on the bottom.
Describe alternatives you've considered
It's possible to add these margins manually to the image, but it's impractical when using an image from another website.

I will re-open this as it should not be closed


Use of undeclared type 'DLRadioButton'

Hi. Using Xcode 9, swift. Installed using CocoaPods, using defaults.
Added buttons to storyboard no issue, and I can see them in the simulator.
But when I add
@IBOutlet weak var test: DLRadioButton!
@IBAction func test(_ sender: DLRadioButton) {
}
to my ViewController, I get the error "Use of undeclared type 'DLRadioButton'" for both.....
Any ideas? Thank you.....

Did you import DLRadioButton?

Actually, no I had not....
Adding 'import DLRadioButton' to the ViewController I am using the DLRadioButton's in gave this error: No such module 'DLRadioButton'
But I then added 'import DLRadioButton' to my main ViewController, and all is working....
Thank you! All sorted......


Zombie rules improved. More tunable overall, weaker victim zombies

About the PR

A wide set of zombie improvements based on player feedback and discussion on Discord and in other places. Also a much stronger set of data-driven zombie balance settings available in YML.
Player facing changes

Initial zombies are forced to convert after 10-15 minutes (down from 20-30). Also forced after 50% of players are zombies.
Zombies die permanently once they enter Death. If they enter Crit, they roll for a chance at a burst of healing (40% or 10% chance of returning).
Zombies coming back from crit have less life (50% - 75% health generally)
Hitting a crit zombie delays any healing which is always at least a set amount after death
Zombies biting other zombies do no damage.
Wait for minimum players to join to begin zombies, rather than just abandoning game round if people don't ready up in time.
Zombie players are infected after 5 minutes (rather than round / event start) and cannot convert until 10 minutes since event.
Shuttle auto-call at 75% and again at 90% zombies.

Important code changes

Added ZombieSettings which contains everything important incl damage, healing, heal from crit, speed and more.
Initial Zombies get EarlySettings and their victims get VictimSettings in ZombieRuleComponent so we can balance those separately
Zombie ratio and limits exposed in event. Zombie CVars have been removed.
If a sizable fraction of the station's crew is converted to zombies, the round will end on zombie extermination. This will happen every time on round-start zombies.
All zombies in any stage have a ZombieComponent. They also have one of : InitialInfected, PendingZombieComponent or LivingZombieComponent. This means queries don't need to check MobState or similar and can become empty and cheap.
Added: PassiveHealSystem flat-heals living zombies. BurstHealComponent brings them out of crit sometimes (40% or 10% chance).
Added: InitialInfectedSystem which manages Patient0 details (Relevant code split out of ZombieRuleSystem)
Added: PendingZombieSystem which manages the pre-zombie infection stage, taking relevant code from ZombieSystem.

Some code details

Every ZombieComponent knows which rule spawned them ZombieFamily.Rules
ZombifyOnDeathSystem has been combined into ZombieSystem (I used partial classes so as not to ruin the diff)
ZombifyOnDeathComponent is now redundant and has been removed. PendingZombie used instead.
Track which players infected who (in ZombieFamily) which allows better round end info in a later PR
Admin converted zombies attach themselves to a running ZombieRule, or if none, to the next one that runs
All timers now respect pause
PendingZombieSystem handles (new) GhostMoveAttempt to block accidental move->ghost while zombifying (console ghost still works)

Data Changes
Zombie Roundstart event
Initial Infected are similar to current zombies
Once zombies are wiped out, the round ends. When we have some post-zombie events, we can change this.
VIctim zombies:

Are much more likely to die on crit & death and take longer to convert or respawn
75% of current healing
Do no slash and less piercing damage on bite, so less likely to inflict bleeding. Also less likely to infect victims.

ZombieOutbreak End round event
Only possible after 60 minutes and with 30+ players
Similar to Zombie event. Zombies convert faster after it occurs (since it's already well after roundstart)
If 60% or more of players are turned, the round ends after they are wiped out.
Media

There is a demonstration on Youtube here: https://youtu.be/Pu1Zh_9q8VQ
Delay to turn undead

[X] I have added screenshots/videos to this PR showcasing its changes ingame, or this PR does not require an ingame showcase

Changelog

:cl:

tweak: Our Zombie-vaccine program has been (mostly) successful. We salute those who gave their lives for the company. As a result we expect anyone infected on-station to be less infectious, do less damage and be slightly slower.
tweak: Zombies are less likely to get back up after you Crit them and they take longer to do so. They will not respawn if you have just hit them. Once dead, they never come back.
tweak: We've been screening staff for the zombie virus, but some with latent infections have slipped through. Initial zombies must wait about 10 minutes and are forced to convert sooner.
add:  Due to employee concerns, Centcom has begun monitoring stations and will dispatch emergency shuttles to those overwhelmed by a zombie horde. Please do not bring any zombies with you on the shuttle.

Taking damage when biting nonliving objects is an obscene buff. Zombies have structure damage for a reason. They are meant to be able to break down structures in order to get into places. I know there are issues with them destroying power objects, but it's more feasible to do that than a blanket structure nerf. You're just going to encourage cheap tactics like vending machine barricades rather than actual counterplay.

Taking damage when biting nonliving objects is an obscene buff. Zombies have structure damage for a reason. They are meant to be able to break down structures in order to get into places. I know there are issues with them destroying power objects, but it's more feasible to do that than a blanket structure nerf. You're just going to encourage cheap tactics like vending machine barricades rather than actual counterplay.

Obviously we need to tune how much of this damage there is. Initial infected have a lower penalty. Even with 5 damage per hit, surely you could break through most reasonable barricades in 10 bites (50% health)?
At least it's an option now.

by 10 bites you must mean 100, since a door is destroyed at 500 damage
thatll more than kill a zombie so please remove that
just make ame shock you when attacked, or put that in a separate pr

Okay @EmoGarbage404 @deltanedas I hear ya. It was a pretty ugly solution and I will take it (damage on non-organic) out.
A better solution is to balance zombies enough that Eng can defend that stuff anyway.

I need to

Split out Passive Healing into its own system and component that can be used by non-zombie mobs and respects pausing.
Move much of PendingZombie into ZombieComponent and give ZC to pending zombies
Create a LivingZombieComponent as a marker to simplify iteration
Remove Revive entirely
Remove (don't add) Midround zombies (pity, I like a scary midround event)
Perhaps a specific InitialInfectedZombie component instead of PendingZombie for those who can delay their own infection to simplify that.

Removing midround zombies is cringe. I'd rather want it midround with InitInf than roundstart.

So some general thoughts on the changes here:

announcing zombies: this is pretty rough in terms of balance. We used to have announcements for zombies and various other midround events, but the general consensus is that players would overprepare and typically obliterate these events, causing them to be stomped by 10 million preped greytiders. Obviously you could make it announce after a percentage of the crew has been zombified, but at that point it just feels excessive and like it's impacting the stealth elements that add depth to initial infected.
 The whole point of zombies is the increase in difficulty as more and more people become overwhelming and begin to swarm everyone. I understand that it is very often overwhelming. It's meant to be like that. If the intent is to have combat situations that don't overwhelm the station, that situation shouldn't be zombies. They were intentionally designed to be round-ending threats. If they cause rounds to stall, that's a different issue (that you're addressing), but the core functionality of mass conversion shouldn't really be poked at.
Having a "minor zombie event," like i mentioned earlier, seems like a balancing nightmare due to the intentionally spiraling nature of how conversion antags work. Having secret settings like weaker zombies and debuffs after conversion percentages is really obscure and not something that players are going to understand. I'd rather have a more cohesive experience and just keep them rarer.
Forcing conversion seems kinda strange to me but i do understand that there have been situations where players do not convert at all and it is a detriment. At the very least, I'd still keep it solely on the timer rather than also having it happen on 50% zombification. It seems like yet another weird complicating factor that is likely to confuse initial infected players. As a good reference: players shouldn't have actions happen to them that they cannot understand the immediate causes of. If it's warned, the timer makes sense. Hitting a 50% quota is likely to be very obscure.
Self healing zombies is good. Healing out of crit is equally fine in my opinion, though some may disagree. Not healing out of dead state is perfectly understandable. Whichever option you pick is up to you. It's really just a question of if you want zombies to have that extra leeway and scare factor. It's perfectly fine balance to keep the healing out of crit so long as they don't heal out of dead.

Overall, I want to caution against balancing them too weak. Zombies were always meant to be a rare and deadly event. The initial infected mode was rarer than all the other modes for a purposeful reason. In terms of changes that are needed, I would prefer if they really just extended to recovery options (reversing zombification) and end round conditions (auto-calling shuttle)

announcing zombies: this is pretty rough in terms of balance.
Yeah, that's the general reaction. I think this feature is getting cut.

Limits for zombies ... seems like a strange choice for what is designed to be a snowballing conversion antag.
This is a very insightful point, especially around player confusion. It's a pity, since I'd like to see some complex midround stuff but I agree we should make other events for this.

Having a "minor zombie event,"
Having established that we shouldn't have various qualities, you make a good point here about cutting the nerf thing.

Forcing conversion seems kinda strange to me but i do understand that there have been situations where players do not convert at all and it is a detriment.
Yes, I can't think of a way to reasonably force that is flexible enough to handle the possibility of all zombies deciding to go late and also the possibility of most going early.

I've heard stories about groups of survivors on the shuttle and then "whoops, one of them was initial infected" which just seems unfair.

  Good paranoia IMO

Well the force system just starts the countdown, that person could be popping pills to try to outheal the conversion for a minute or two. And we can tune the setpoint for when the virus starts.

Post-death Revive has been removed. A burst heal is present while in crit (sometimes)
ZombieSystem has been split up, all timers respect pause now (I think)

Stuff I said I would do https://github.com/space-wizards/space-station-14/pull/17152#issuecomment-1581635410 is done
I think that all the comments from @metalgearsloth are resolved now. I hope they are happy with where this code is now.

* Zombies biting other zombies do no damage.

This is not cool, you guys are replacing my over 2 weeks old PR (https://github.com/space-wizards/space-station-14/pull/16963) with this one...

* Zombies biting other zombies do no damage.

This is not cool, you guys are replacing my over 2 weeks old PR (#16963), long completed and tested, with this one...

If you complain a 4th time about it I am going to close your PR.

* Zombies biting other zombies do no damage.

This is not cool, you guys are replacing my over 2 weeks old PR (#16963), long completed and tested, with this one...

If you complain a 4th time about it I am going to close your PR.

What's the point of this threat? Seems you're already closing it, since you're going to replace it with this new one. I've started working on that change 2 weeks ago and, desptite being new to both github and ss14's code, I worked on it with dedication and the PR was completed and tested in few days, waiting for approval. And it was just a waste of time: it seems to me you only care about your inner-circle of elite developers, the rest don't count.

I'm sorry @John-Doom that you feel I replicated your work. I've actually been gathering requests for zombie changes for about a month and the no damage one came up in conversation so I put it in.
I'm not blessed by any means. As you can see, I have 12 open PRs. Several of them have been open with little comment for 2-3 weeks. Patience is needed, the project simply does not have enough maintainers.
https://github.com/pulls?q=is%3Aopen+is%3Apr+author%3Atom-leys+archived%3Afalse+repo%3Aspace-wizards%2Fspace-station-14+
Just take it easy on the admin team, they're doing their best.

@John-Doom my first pr was like 2 weeks before merging you aint special kid

@John-Doom my first pr was like 2 weeks before merging you aint special kid

They got discord banned.

@John-Doom my first pr was like 2 weeks before merging you aint special kid

They got discord banned.

Do you feel proud of both closing my hard work and banning me from discord, so I can't talk with the other players or get the latests news?

@John-Doom my first pr was like 2 weeks before merging you aint special kid

They got discord banned.

Do you feel proud of both closing my hard work and banning me from discord, so I can't talk with the other players or get the latests news? What kind of rules did I break to be treated like a criminal?

You were specifically warned by mirror to stop stirring about it (and I also told you what needed doing twice) but instead you decided to come to an unrelated PR and continue.

Do you feel proud of both closing my hard work and banning me from discord, so I can't talk with the other players or get the latests news? What kind of rules did I break to be treated like a criminal?

Moderators in every community need to remove disruptive people or everyone else will either leave or become disruptive. It might look like a power trip, but it is not.
You can appeal your ban in the forums, but I suggest you promise not to be disruptive in the future and stick with that promise. https://forum.spacestation14.io/index.php?/forum/4-ban-appeals/
It takes what feels like a long time for PRs to be approved in this project. You just need to be patient. I suggest you work on multiple different things so you have multiple PRs "in flight" at a given time. Also, you might want to work on PRs that resolve issues, that way there is little debate that you are working on something a moderator will approve.

Had a longer-form discussion on discord on what people have started disliking about zombies.


Provide Model Serving snippet for Java base

@slemeur we have one base on OkHttp, do you think it is ok or should get one based on raw JDK runtime

The unirest snippet is correct for me

Closing as https://github.com/projectatomic/ai-studio/pull/626 has been merged and java unirest code is ok


Show keyboard automatically on prompt or login

To avoid the extra tap needed before keyboard shows up

Thanks @geirsagberg


Using Electron-Forge make, creating start menu shortcut on windows

[x] I have read the contribution documentation for this project.
[x] I agree to follow the code of conduct that this project follows, as appropriate.
[x] I have searched the issue tracker for an issue that matches the one I want to file, without success.

Please describe your issue:
I've spent the weekend trying to get a basic app installed on windows. The squirrel installer makes the executable just fine, and installs it in the %appdata% folder (even accessible from Control Panel -> Programs & Features), but does not create a shortcut anywhere that can be searched for from the start menu. I'm packaging the software for coworkers who I don't necessarily trust to find the executable without a shortcut.
I tried using the Electron-Winstaller blurb on handling Squirrel Events, but it doesn't create the shortcut, and when I run the setup.exe it asks me if my index.js (main) was compiled.
How do I use 'Electron-Forge make' and yet still influence squirrel to create a start menu shortcut?

Console output when you run electron-forge with the environment variable DEBUG=electron-forge:*. (Instructions on how to do so here). Please include the stack trace if one exists.
Put the console output here

What command line arguments are you passing?
Put the arguments here

What does your config.forge data in package.json look like?
"config": {
    "forge": {
      "make_targets": {
        "win32": [
          "squirrel"
        ],
        "darwin": [
          "dmg"
        ],
        "linux": [
          "deb",
          "rpm"
        ]
      },
      "electronPackagerConfig": {
        "asar": true,
        "quiet": true,
        "icon": "./build/icon",
        "win32metadata": {
          "CompanyName": "***",
          "ProductName": "Auth Time"
        }
      },
      "electronInstallerDMG": {
        "appPath": "***",
        "name": "AuthTime",
        "icon": "./build/icon"
      },
      "electronWinstallerConfig": {
        "appDirectory": "./out/AuthTime-win32",
        "setupIcon": "./build/icon.ico",
        "setupExe": "setup.exe",
        "setupMSI": "setup.msi",
        "noMsi": false
      },
      "electronInstallerDebian": {},
      "electronInstallerRedhat": {},
      "github_repository": {
        "owner": "jonpowers",
        "name": "authtime"
      },
      "windowsStoreConfig": {
        "packageName": "***"
      }
    }
  }

Please provide either a failing minimal testcase (with a link to the code) or detailed steps to
reproduce your problem. Using electron-forge init is a good starting point, if that is not the
source of your problem.

I tried using the Electron-Winstaller blurb on handling Squirrel Events

Without any code or tangible errors to share I cant help you with this.  If you post you're code or the error messages you are getting (with traces) I can try help you out.
That said, this is more of a question than an issue so closing out for now.  If you provide the information I'll help you out in this issue thread :+1:

I don't know how to get that output. I added DEBUG=electron-forge:* as a command line arg and it did nothing different. I also don't see any verbose flag for electron-forge make.

Can you provide the code you are using to try make the shortcut?

This is at the top of the entry point to my application, before mainWindow is created. I used the Electron-Forge init boilerplate.
var app = require('app');
var path = require('path');
var cp = require('child_process');

var handleSquirrelEvent = function() {
   if (process.platform != 'win32') {
      return false;
   }

   function executeSquirrelCommand(args, done) {
      var updateDotExe = path.resolve(path.dirname(process.execPath),
         '..', 'update.exe');
      var child = cp.spawn(updateDotExe, args, { detached: true });

      child.on('close', function(code) {
         done();
      });
   };

   function install(done) {
      var target = path.basename(process.execPath);
      executeSquirrelCommand(["--createShortcut", target], done);
   };

   function uninstall(done) {
      var target = path.basename(process.execPath);
      executeSquirrelCommand(["--removeShortcut", target], done);
   };

   var squirrelEvent = process.argv[1];

   switch (squirrelEvent) {

      case '--squirrel-install':
         install(app.quit);
         return true;

      case '--squirrel-updated':
         install(app.quit);
         return true;

      case '--squirrel-obsolete':
         app.quit();
         return true;

      case '--squirrel-uninstall':
         uninstall(app.quit);
         return true;
   }

   return false;
};

if (handleSquirrelEvent()) {
   return;
}

Can't see anything obviously wrong with that code.  Does your apps name have a space in it?
Can you upload the SquirrelSetup log file from your local install, should have some useful logging info
Lastly can you set the stdio of the child process to inherit and then launch the installed exe file from command prompt.  You should see any logs come through correctly.

I took out the space in my app's name, but it is still failing.
SquirrelSetup.log:
2017-04-03 10:13:31> Program: Starting Squirrel Updater: --install .
2017-04-03 10:13:36> Program: Starting install, writing to C:\Users\jpowers\AppData\Local\SquirrelTemp
2017-04-03 10:13:41> Program: About to install to: C:\Users\jpowers\AppData\Local\AuthTime
2017-04-03 10:13:47> CheckForUpdateImpl: Couldn't write out staging user ID, this user probably shouldn't get beta anything: System.IO.DirectoryNotFoundException: Could not find a part of the path 'C:\Users\jpowers\AppData\Local\AuthTime\packages\.betaId'.
   at System.IO.__Error.WinIOError(Int32 errorCode, String maybeFullPath)
   at System.IO.FileStream.Init(String path, FileMode mode, FileAccess access, Int32 rights, Boolean useRights, FileShare share, Int32 bufferSize, FileOptions options, SECURITY_ATTRIBUTES secAttrs, String msgPath, Boolean bFromProxy, Boolean useLongPath, Boolean checkHost)
   at System.IO.FileStream..ctor(String path, FileMode mode, FileAccess access, FileShare share, Int32 bufferSize, FileOptions options, String msgPath, Boolean bFromProxy, Boolean useLongPath, Boolean checkHost)
   at System.IO.StreamWriter.CreateFile(String path, Boolean append, Boolean checkHost)
   at System.IO.StreamWriter..ctor(String path, Boolean append, Encoding encoding, Int32 bufferSize, Boolean checkHost)
   at System.IO.File.InternalWriteAllText(String path, String contents, Encoding encoding, Boolean checkHost)
   at System.IO.File.WriteAllText(String path, String contents, Encoding encoding)
   at Squirrel.UpdateManager.CheckForUpdateImpl.getOrCreateStagedUserId()
2017-04-03 10:13:47> CheckForUpdateImpl: Failed to load local releases, starting from scratch: System.IO.DirectoryNotFoundException: Could not find a part of the path 'C:\Users\jpowers\AppData\Local\AuthTime\packages\RELEASES'.
   at System.IO.__Error.WinIOError(Int32 errorCode, String maybeFullPath)
   at System.IO.FileStream.Init(String path, FileMode mode, FileAccess access, Int32 rights, Boolean useRights, FileShare share, Int32 bufferSize, FileOptions options, SECURITY_ATTRIBUTES secAttrs, String msgPath, Boolean bFromProxy, Boolean useLongPath, Boolean checkHost)
   at System.IO.FileStream..ctor(String path, FileMode mode, FileAccess access, FileShare share)
   at Squirrel.Utility.LoadLocalReleases(String localReleaseFile)
   at Squirrel.UpdateManager.CheckForUpdateImpl.<CheckForUpdate>d__2.MoveNext()
2017-04-03 10:13:48> CheckForUpdateImpl: Reading RELEASES file from C:\Users\jpowers\AppData\Local\SquirrelTemp
2017-04-03 10:13:49> CheckForUpdateImpl: First run or local directory is corrupt, starting from scratch
2017-04-03 10:13:51> ApplyReleasesImpl: Writing files to app directory: C:\Users\jpowers\AppData\Local\AuthTime\app-1.0.0
2017-04-03 10:13:52> LogHost: Rigging execution stub for lib/net45/AuthTime_ExecutionStub.exe to C:\Users\jpowers\AppData\Local\AuthTime\AuthTime.exe
2017-04-03 10:14:28> ApplyReleasesImpl: Squirrel Enabled Apps: [C:\Users\jpowers\AppData\Local\AuthTime\app-1.0.0\AuthTime.exe]
2017-04-03 10:14:46> ApplyReleasesImpl: Couldn't run Squirrel hook, continuing: C:\Users\jpowers\AppData\Local\AuthTime\app-1.0.0\AuthTime.exe: System.OperationCanceledException: The operation was canceled.
   at System.Threading.CancellationToken.ThrowOperationCanceledException()
   at Squirrel.Utility.<>c__DisplayClass11_0.<InvokeProcessAsync>b__0()
   at System.Threading.Tasks.Task.InnerInvoke()
   at System.Threading.Tasks.Task.Execute()
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
   at Squirrel.Utility.<InvokeProcessAsync>d__11.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
   at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
   at Squirrel.UpdateManager.ApplyReleasesImpl.<>c__DisplayClass10_0.<<invokePostInstall>b__0>d.MoveNext()
2017-04-03 10:14:47> ApplyReleasesImpl: Starting fixPinnedExecutables
2017-04-03 10:14:47> ApplyReleasesImpl: Examining Pin: Adobe Acrobat Reader DC .lnk
2017-04-03 10:14:47> ApplyReleasesImpl: Examining Pin: Brackets.lnk
2017-04-03 10:14:47> ApplyReleasesImpl: Examining Pin: Command Prompt (2).lnk
2017-04-03 10:14:47> ApplyReleasesImpl: Examining Pin: Command Prompt.lnk
2017-04-03 10:14:48> ApplyReleasesImpl: Examining Pin: Google Chrome.lnk
2017-04-03 10:14:48> ApplyReleasesImpl: Examining Pin: GroupWise.lnk
2017-04-03 10:14:48> ApplyReleasesImpl: Examining Pin: JetBrains WebStorm 2016.3.3.lnk
2017-04-03 10:14:48> ApplyReleasesImpl: Examining Pin: Microsoft Office Excel 2007.lnk
2017-04-03 10:14:48> ApplyReleasesImpl: Examining Pin: notepad++.lnk
2017-04-03 10:14:49> ApplyReleasesImpl: Examining Pin: SQL Server 2014 Management Studio.lnk
2017-04-03 10:14:49> ApplyReleasesImpl: Examining Pin: Sublime Text 3.lnk
2017-04-03 10:14:49> ApplyReleasesImpl: Examining Pin: Visual Studio 2017.lnk
2017-04-03 10:14:49> ApplyReleasesImpl: Examining Pin: Windows Explorer.lnk
2017-04-03 10:14:49> ApplyReleasesImpl: Examining Pin: Windows PowerShell ISE.lnk
2017-04-03 10:14:49> ApplyReleasesImpl: Fixing up tray icons
2017-04-03 10:14:50> ApplyReleasesImpl: Couldn't rewrite shim RegKey, most likely no apps are shimmed: System.NullReferenceException: Object reference not set to an instance of an object.
   at Squirrel.UpdateManager.ApplyReleasesImpl.<unshimOurselves>b__13_0(RegistryView view)
2017-04-03 10:14:50> ApplyReleasesImpl: Couldn't rewrite shim RegKey, most likely no apps are shimmed: System.NullReferenceException: Object reference not set to an instance of an object.
   at Squirrel.UpdateManager.ApplyReleasesImpl.<unshimOurselves>b__13_0(RegistryView view)
2017-04-03 10:14:50> ApplyReleasesImpl: cleanDeadVersions: for version 1.0.0
2017-04-03 10:14:50> ApplyReleasesImpl: cleanDeadVersions: exclude folder app-1.0.0

And I honestly don't know how to approach your third request, but I'm searching

No need, there's your issue.

2017-04-03 10:14:46> ApplyReleasesImpl: Couldn't run Squirrel hook, continuing: C:\Users\jpowers\AppData\Local\AuthTime\app-1.0.0\AuthTime.exe: System.OperationCanceledException: The operation was canceled.

Not sure why the hook would fail with that message but that's why your shortcut isn't being made.  The hook isn't being called at still.
Should do some research into that error message and try figure out what's going on.  Peruse the issues over at Squirrel.Windows, one of them might point you in a good direction.

Thank you for your help! I may have a few more questions as this search isn't turning up too much.
I'm curious though, where is the better place to ask these questions?

But how can i just do not create this shortcut?
My client dont wanna it, there is a way?


API-based update downgraded cask

brew doctor output
This is from a long-lived install which I've also done formulae development on, so there are a lot of "installed kegs have no formulae" etc... really too much to quote.  No warnings related to casks though.

Verification

[X] My "brew doctor output" above says Your system is ready to brew. and am still able to reproduce my issue.
[X] I ran brew update twice and am still able to reproduce my issue.
[X] This issue's title and/or description do not reference a single formula e.g. brew install wget. If they do, open an issue at https://github.com/Homebrew/homebrew-core/issues/new/choose instead.

brew config output
HOMEBREW_VERSION: 3.6.20-189-g3af2f0e
ORIGIN: https://github.com/Homebrew/brew
HEAD: 3af2f0ececc848bca34ddbd5b876d20dc24a6aaa
Last commit: 39 minutes ago
Core tap origin: https://github.com/Homebrew/homebrew-core
Core tap HEAD: c7c38c40e91994ba2b59fa8570f836df0178d064
Core tap last commit: 83 minutes ago
Core tap branch: master
Core tap JSON: 01 Feb 01:49
HOMEBREW_PREFIX: /usr/local
HOMEBREW_CASK_OPTS: []
HOMEBREW_DISPLAY: /private/tmp/com.apple.launchd.TKKvVtmW0w/org.macosforge.xquartz:0
HOMEBREW_MAKE_JOBS: 12
Homebrew Ruby: 2.6.8 => /usr/local/Homebrew/Library/Homebrew/vendor/portable-ruby/2.6.8_1/bin/ruby
CPU: dodeca-core 64-bit kabylake
Clang: 14.0.0 build 1400
Git: 2.39.1 => /usr/local/bin/git
Curl: 7.85.0 => /usr/bin/curl
macOS: 13.1-x86_64
CLT: <IP_ADDRESS>.1.1668646533
Xcode: 14.2

(I elided HOMEBREW_EDITOR since it just pointed at my own vim-wrapping script I use as $VISUAL but not really relevant to this discussion)

### What were you trying to do (and why)?

I previously mentioned this in a comment on another issue ( https://github.com/Homebrew/brew/issues/14449#issuecomment-1414168433 ) and am filing it as a separate issue per a request there.

Normal `brew upgrade` just like I run frequently.

I've used this install for a lot of formulae (but not cask!) development.  My usual routine is:
```sh
$ cd $(brew --repo homebrew/core)/Formula
$ git pull
$ brew upgrade

I know that the git pull is something that brew can do automatically, but I just like to see what changes in the formulae repo day-to-day so I do it manually.
What happened (include all command output)?
One of my casks which homebrew had previously upgraded, got downgraded:
$ git pull
remote: Enumerating objects: 1044, done.
remote: Counting objects: 100% (821/821), done.
remote: Compressing objects: 100% (99/99), done.
remote: Total 1044 (delta 751), reused 789 (delta 722), pack-reused 223
Receiving objects: 100% (1044/1044), 607.67 KiB | 3.38 MiB/s, done.
Resolving deltas: 100% (754/754), completed with 131 local objects.
From https://github.com/Homebrew/homebrew-core
   0af99a0552a..c7c38c40e91  master     -> origin/master
Updating 0af99a0552a..c7c38c40e91
Fast-forward
[ ... eliding diffstat output ...]
122 files changed, 1215 insertions(+), 1183 deletions(-)
 create mode 100644 Formula/crfsuite.rb
time git pull: 0.41s/0.63s/28% user/kern, 3.604 elapsed

$ brew upgrade
Running `brew update --auto-update`...
==> Auto-updated Homebrew!
==> Updated Homebrew from 7a5f6143d to cdc6a4f11.
Updated 1 tap (homebrew/cask-versions).

You have 11 outdated formulae and 1 outdated cask installed.
You can upgrade them with brew upgrade
or list them with brew outdated.

==> Upgrading 11 outdated packages:

[... bunch of non-cask upgrades, nothing notable ...]

==> Casks with 'auto_updates true' or 'version :latest' will not be upgraded; pass `--greedy` to upgrade them.
==> Upgrading 1 outdated package:
kitty 0.27.0 -> 0.26.5
==> Upgrading kitty
==> Downloading https://github.com/kovidgoyal/kitty/releases/download/v0.26.5/kitty-0.26.5.dmg
Already downloaded: /Users/mitch/Library/Caches/Homebrew/downloads/a5773a93f0d48a59da4eff9758367dcf45bc8a14630b78ae0be7da93fa9ebd7c--kitty-0.26.5.dmg
==> Backing App 'kitty.app' up to '/usr/local/Caskroom/kitty/0.27.0/kitty.app'
==> Removing App '/Applications/kitty.app'
==> Unlinking Binary '/usr/local/bin/kitty'
==> Moving App 'kitty.app' to '/Applications/kitty.app'
==> Linking Binary 'kitty.wrapper.sh' to '/usr/local/bin/kitty'
==> Purging files for version 0.27.0 of Cask kitty

What did you expect to happen?
cask remains at the latest version.
Step-by-step reproduction instructions (by running brew commands)
It had been very insistent that it had the current version:

$ brew info --cask kitty
==> kitty: 0.26.5
https://github.com/kovidgoyal/kitty
/usr/local/Caskroom/kitty/0.26.5 (218B)
From: https://github.com/Homebrew/homebrew-cask/blob/HEAD/Casks/kitty.rb
==> Name
kitty
==> Description
GPU-based terminal emulator
==> Artifacts
kitty.app (App)
/usr/local/Caskroom/kitty/0.26.5/kitty.wrapper.sh -> kitty (Binary)
==> Analytics
install: 2,365 (30 days), 6,829 (90 days), 27,204 (365 days)

Looking at the status of the homebrew-cask git checkout showed that it hadn't been updated in 9 days (despite many "git upgrade"'s being run in that time)  I imagine that is due to some change in default wrt the new API stuff:
$ git l
f69317f19e - (HEAD -> master) adguard <IP_ADDRESS>4 (9 days ago) <stevezhengshiqi>
[...]

After disabling the API another update fixed the problem:
$ HOMEBREW_NO_INSTALL_FROM_API=1 brew upgrade
==> Casks with 'auto_updates true' or 'version :latest' will not be upgraded; pass `--greedy` to upgrade them.
==> Upgrading 1 outdated package:
kitty 0.26.5 -> 0.27.0
[...]

As I would expect this was because it then automatically updated the git checkout:
$ git l
cc05495e4c - (HEAD -> master, origin/master, origin/HEAD) Upgrade Headlamp version to 0.15.0 (#140522) (82 minutes ago) <Joaquim Rocha>
[...]

So:

Some change in the homebrew code caused it to stop updating my homebrew-cask git directory.  I gather this is intentional as part of the API transition
At some point the API code correctly updated the "kitty" cask
Today for some reason the API wasn't working (maybe an intermittent problem?) so I guess it fell back onto the now-outdated git checkout?!?  Unclear.
Downgrade hijinks.

At a minimum it seems that there is some poor fallback behavior where it can end up relying on git files without updating them first leading to incorrect results, at least for installs that predate the introduction of the API.  Either it should be using API data or up-to-date git data, but there seems to be some instance where it ends up using not-updated git data to make package upgrade decisions.

What's the output of
export HOMEBREW_NO_INSTALL_FROM_API
brew update
brew outdated --cask

?

Well right now they're not showing anything outdated, because the git pull from the successful update means that the git data is up-to-date.
If I try to force git into an old state, outdated shows kitty as "outdated"
$ git reset --hard f69317f19e
$ unset HOMEBREW_NO_INSTALL_FROM_API
$ brew outdated --cask
kitty (0.27.0) != 0.26.5

I don't know if there is a way to see if it is making an attempt to get newer data from the API?  brew outdated -v -d --cask doesn't print any extra info.  As far as I can see, it's idea of "outdated" only depends on the state of the git-repo, but unless HOMEBREW_NO_INSTALL_FROM_API is set it doesn't automatically update that.

Oops, I had meant HOMEBREW_NO_INSTALL_FROM_API=1 up there. I'm also a bit unclear on the sequence of events here, especially with the manual git pulls in the mix.
My guess is that when you did brew upgrade, the API data was slightly behind the Homebrew/cask repo and so thought the latest version was 0.26.5. Formulae will never be downgraded in this way, but I'm less sure about casks. It seems not, but if that's the case then I believe that was intentional. It just happens to interact weirdly with API installations.

My guess is that when you did brew upgrade, the API data was slightly behind the Homebrew/cask repo and so thought the latest version was 0.26.5.

Yes, this is what happened. My suggestion would be to either export HOMEBREW_NO_INSTALL_FROM_API=1 or not to rely on Git and the API being in sync.

Formulae will never be downgraded in this way, but I'm less sure about casks

brew outdated separates formulae versions with < and cask versions with !=, so I suspect cask logic only cares that "latest version is not the installed version".

Sorry, maybe I'm misunderstanding the issue here. Was the original (non-downgraded) cask built directly from the tap or is the API giving inconsistent version numbers?

Yes, this is what happened.

But as I mentioned, when the downgrade first happened my git cask data was 9 days stale, so the upgrade to kitty was done originally from API data.  So did the version of the cask advertised by the API go backwards?

brew outdated separates formulae versions with < and cask versions with !=

Then that would imply there is a strong dependency on the set of versions only monotonically increasing.  So, for instance, any divergence between multiple API servers could cause problems since a client can get version X from server A and later get version X-1 from server B.  Before homebrew --cask was only relying on git (and github) to not take the repo checkout backwards which was a safe bet.

My suggestion would be to either export HOMEBREW_NO_INSTALL_FROM_API=1...

Yes, I've done that on my installs after experiencing this problem.  I'm not against the API in principal (it sounds like it makes sense for the 95% case) but I often poke around the git directory to run git log etc to look at package history so I prefer the "full fat" version.
Still, I wanted to flag the issue with you developers since if the API is downgrade-prone (at least for casks) then I think it's going to become a source of problems.

So did the version of the cask advertised by the API go backwards?

Yeah, that's what I'm concerned about. As far as I can tell it shouldn't be possible for it to go backward like this which is why I'm confused. It seems like there's some edge case where casks are loaded from the taps when they shouldn't be, but I'll need to try to reproduce it on my machine to really tell.

Yes, I've done that on my installs after experiencing this problem. I'm not against the API in principal (it sounds like it makes sense for the 95% case) but I often poke around the git directory to run git log etc to look at package history so I prefer the "full fat" version.

Yeah, it sounds like your specific requirements might be best fit to have HOMEBREW_NO_INSTALL_FROM_API set. That being said, I still think there's a bug here that others could run into that I want to fix if possible since there's still clearly some unpredictable behavior. The goal is that only very specific cases (like yours, maybe) actually require HOMEBREW_NO_INSTALL_FROM_API and everything else will work fine without it.

Still, I wanted to flag the issue with you developers since if the API is downgrade-prone (at least for casks) then I think it's going to become a source of problems.

Thank you, I appreciate the report ❤

One way I could reproduce this kind of behavior is by changing the API URL that Homebrew uses to something that will definitely 404. If that happens, Homebrew tries to fall back to the existing cached JSON file but accidentally falls back to nil instead which causes it to fall even further back onto the taps.
I think that will fix your issue so this will be closed when that is merged, but TBH I'm not totally sure so if it continues to happen let me know.


How did you get the custom css elements

Hello, I was thinking about making a custom css for vscode, but was wondering where you got the elements from.

What do you mean? VSCode DOM elements? If so, you can run the > Developer: Toggle Developer Tools command and that will open up a devtools panel, just like you do in Chrome or other browsers. From there is just a matter of inspecting the DOM and targeting what you want.

@TheOld That is what I wanted. Thank you!


Create strandSort.cpp

This file containing strand sort algorithm in c++ language

Hello @brat00 , here in this PR, I can see that 26 files have been changed. So, you please pull the origin of this repository and then create PR with your strandSort.cpp file.

@AnanyaDas162 actually i try to solve the issue #68 that's why i also rearrange the file as when finally i update it it goes with my last pr i think  please review changes you can merge this will not cause any harm to your current repo

also i want a help how can i show my repository which i forked from your one so that i can show the folder to rearrange file

you can see i moved the files  to a folder name "important problems in cp"

@brat00 , yeah, but now you can see there are separate folders and inside them the corresponding files are located. So by merging this, the total folder division will be changed. So I am closing this PR and you please send a different PR for this sorting algorithm. And also thanks to you for collaborating in this repo. Meet you again!

@AnanyaDas162  thanks to you surely i will do that


[8.x] Changelog.md file has been updated.

The last pull requests not attached are included.
A few items have been corrected for readability.

https://github.com/laravel/framework/compare/v8.33.1...v8.34.0


Remove version: Stackers.Stack version 3.40.0

[Automated] It returns code over 400 in all urls
Microsoft Reviewers: Open in CodeFlow

URL: https://binaries.getstack.app/builds/prod/win/x64/Stack Setup 3.40.0-x64.exe
Response status code does not indicate success: 530 ().
(Automated message - build 896)


base checkpoint selection

I see in the code that two models (distilbert-base-uncased, msmarco-distilbert-margin-mse) are recommended to use as initial checkpoints. I tried to use other Sentence-Transformers models like all-mpnet-base-v2 but it didn't work. Is there a difference in the architecture of the models and the implementation out there? What models can be used here as initial checkpoints?

Hi @alsbhn, could you please tell me what you mean by "didn't work"? Do you mean the code was not runnable with this setting or something about the performance?

Thanks for pointing out this issue. I need some time to check what could be the exact reason. As I can imagine, there might be four potential reasons:
(1) The base checkpoint might be already stronger than the teacher cross-encoder;
(2) The training steps might be too few: For some target datasets, I found there could be degeneration at the beginning, but the final performance would be improved after longer training (e.g. 100K steps);
(3) The negative miner might be too weak. For this, we can try setting base_ckpt and retrievers to the same checkpoint, e.g. sentence-transformers/all-mpnet-base-v2. From my experience, I found this is very important when we use TAS-B as the base checkpoint;
(4) It might be due to the similarity function between dot product and cosine similarity. @nreimers recently found MarginMSE would result in poor in-domain performance if we use cosine similarity (compared with simple CrossEntropy loss). I am not sure whether this will be the same case for the domain-adaptation scenario. Note that both all-mpnet-base-v2 and all-MiniLM-L6-v2 were trained with cosine similarity.


Request for feature/help

I have an entity class with this fluent API (I do not use data declaration attributes)

this is what I have generated

is there a chance I can somehow translate at least Required and HasMaxLength to DTO object ???

@cherchyk I don't understand what you are trying to achieve here. Can you explain the use case further?


Add ability to provide a name when creating a service binding

Description of the Change
A user can now provide a name for their service binding. This name is BindingName until the API layer, where it is Name because that is what the CC API expects.
The default name, if none is provided, is an empty string.
CAPI story: https://www.pivotaltracker.com/story/show/152947740

@zrob

For the current story, when there is a binding name, please output:
Binding service s to app a with binding name [binding name] in org test / space test as admin...
If no binding name, output is unchanged.
We can update services in a later story.
OK for now.
Please use flag --binding-name.


facing error when code gen for sample app

Here are the err msgs:
.../test/appsync-ios-posts-starter/GraphQLOperations/queries.graphql: Cannot query field "getPost" on type "Query".
.../test/appsync-ios-posts-starter/GraphQLOperations/queries.graphql: Cannot query field "allPosts" on type "Query". Did you mean "posts" or "searchPosts"?
.../test/appsync-ios-posts-starter/GraphQLOperations/subscriptions.graphql: Unknown argument "author" on field "newPost" of type "Subscription".
.../test/appsync-ios-posts-starter/GraphQLOperations/subscriptions.graphql: Cannot query field "updatePost" on type "Subscription".
.../test/appsync-ios-posts-starter/GraphQLOperations/subscriptions.graphql: Cannot query field "deletePost" on type "Subscription". Did you mean "newPost"?
I followed instructions from:
https://docs.aws.amazon.com/appsync/latest/devguide/building-a-client-app-ios.html
The above url doesn't have info about where to get schema.json, so I downloaded from the appsync api console. Appreciated if you could give me some help!

Hello @fans3210 ,
We have an updated set of documentation here:
https://awslabs.github.io/aws-mobile-appsync-sdk-ios/ which should help resolve the issue.
Could you please go through this and see if it helps resolve the issue?
Thanks
Rohan

Facing a few errors. I don't know whether I'm doing correctly. But I changed some in the graphql file:
I changed
query AllPosts {
   listPosts {
    items {
       id
       title
       author
       content
       url
       version
       ups
       downs
      }
   }
}

to
query AllPosts {
listPosts {

id
title
author
content
url
version
ups
downs
}

}

by removing 'items'
I also added the missing contents in the console to make the api to be:
input CreatePostInput {
	id: ID!
	author: String!
	title: String
	content: String
	url: String
	ups: Int
	downs: Int
	version: Int!
}

input DeletePostInput {
	id: ID!
}

type Mutation {
	createPost(input: CreatePostInput!): Post
	updatePost(input: UpdatePostInput!): Post
	deletePost(input: DeletePostInput!):  Post
	listPosts: [Post]
}

type Subscription {
	onCreatePost: Post
		@aws_subscribe(mutations: ["createPost"])
}

input UpdatePostInput {
	id: ID!
	author: String!
	title: String
	content: String
	url: String
	ups: Int
	downs: Int
	version: Int!
}

schema {
	query: Query
	mutation: Mutation
	subscription: Subscription
}

After that the codegen finally works. I don't know whether I missed something or the doc error

After a few attempts. I still fail to subscribe the event of create post. I didn't make any change on update post yet since I'm focusing on create post. Below is a sample project, would be appreciated if anyone could provide some help.
10Apr-appsync-ios-posts-starter.zip

Same issue here, this repo looks dead. where's the aws support?

@hallwest AWS team (@minbi and @rohandubal ) are responding to issues, but it's unclear if the suggested steps have been helpful. The last suggestion from @minbi is correct, the @aws_subscribe directive is indented to be used on the GraphQL schema on the server and not in the client queries used to generate your Swift file. This is a common mistake hence the suggestion.
To be clear, please reference the flow in the README of this repo where you:

Create a GraphQL API, with subscriptions configured
Download the introspection schema (NOT the GraphQL schema, these are different files)
Create one or more GraphQL files with appropriate queries, mutations, and subscriptions
Use the introspection schema (it's a json file) from step 2 along with your queries/mutations/subscription files in step 3 to generate a Swift class
Call an appropriate GraphQL operation from your iOS app, in the case of a subscription it will be something like:

let subscriptionRequest = NewCommentOnPostSubscription(eventId: event!.id)
do {
  newCommentsSubscriptionWatcher = try appSyncClient?.subscribe(subscription: subscriptionRequest, resultHandler: { (res, transaction, err) in
//...handle received data

Call the mutation that was configured in the service with @aws_subscribe which will invoke this subscription

If you're just using GraphQL subscriptions with iOS for the first time, I suggest taking a few minutes and setting up the sample application: https://github.com/aws-samples/aws-mobile-appsync-events-starter-ios
With this sample app you only need to use the built-in schema in the console when you click "Create API" and then download the introspection schema after it completes. This sample schema has the @aws_subscribe directive for a mutation on event comments (think of this as comments on a blog post, same pattern). Then clone the repo and configure the appropriate constants and run the codegen. The events.graphql file already has the queries, mutations, and subscriptions created as an example of what is used for codegen.

Hello everybody! Just needed a little help
I've Integrated the https://docs.aws.amazon.com/appsync/latest/devguide/building-a-client-app-ios.html Example and that works well for me.
Then i started something from scratch i have a schema
type Mutation {
	#------ User ------#
	CreateUser(
		FirstName: String,
		Username: String,
		Password: String,
		CreatedOn: Int
	): User
	UpdateUser(UserId: ID!, FirstName: String!, Username: String!): User
	DeleteUser(UserId: ID!): User
}

type Query {
	####------ User ------#
	GetUser(UserId: ID!): User
	CheckUsername(Username: String): [User]
	CheckPhoneNumber(Phone: String): [User]
	#------ Login ------#
	CheckUserLogin(Username: String, Password: String): [User]
}

type User {
	UserId: ID!
	FirstName: String
	Username: String
	#Password: String
	Bio: String
	Phone: String
	Email: String
	Address: String
	WebsiteUrl: String
	ProfilePicture: String
	CreatedOn: Int
	Updated: Int
	Status: Boolean
	LastLoggedIn: Int
	IpAddress: String
}

schema {
	query: Query
	mutation: Mutation
}

A simple query in .graphql file
query CheckUsername($Username:String){
  CheckUsername(Username:$Username)
  {
      Username
  }
}

and generated a file called check Queries.swift
import AWSAppSync

public final class CheckUsernameQuery: GraphQLQuery {
    public static let operationString =
    "query CheckUsername {\n  CheckUsername(Username: \"user1_ajm\") {\n    __typename\n    Username\n  }\n}"

    public init() {
    }

    public struct Data: GraphQLSelectionSet {
        public static let possibleTypes = ["Query"]

        public static let selections: [GraphQLSelection] = [
            GraphQLField("CheckUsername", arguments: ["Username": "user1_ajm"], type: .list(.object(CheckUsername.selections))),
            ]

        public var snapshot: Snapshot

        public init(snapshot: Snapshot) {
            self.snapshot = snapshot
        }

        public init(checkUsername: [CheckUsername?]? = nil) {
            self.init(snapshot: ["__typename": "Query", "CheckUsername": checkUsername.flatMap { $0.map { $0.flatMap { $0.snapshot } } }])
        }

        public var checkUsername: [CheckUsername?]? {
            get {
                return (snapshot["CheckUsername"] as? [Snapshot?]).flatMap { $0.map { $0.flatMap { CheckUsername(snapshot: $0) } } }
            }
            set {
                snapshot.updateValue(newValue.flatMap { $0.map { $0.flatMap { $0.snapshot } } }, forKey: "CheckUsername")
            }
        }

        public struct CheckUsername: GraphQLSelectionSet {
            public static let possibleTypes = ["User"]

            public static let selections: [GraphQLSelection] = [
                GraphQLField("__typename", type: .nonNull(.scalar(String.self))),
                GraphQLField("Username", type: .scalar(String.self)),
                ]

            public var snapshot: Snapshot

            public init(snapshot: Snapshot) {
                self.snapshot = snapshot
            }

            public init(username: String? = nil) {
                self.init(snapshot: ["__typename": "User", "Username": username])
            }

            public var __typename: String {
                get {
                    return snapshot["__typename"]! as! String
                }
                set {
                    snapshot.updateValue(newValue, forKey: "__typename")
                }
            }

            public var username: String? {
                get {
                    return snapshot["Username"] as? String
                }
                set {
                    snapshot.updateValue(newValue, forKey: "Username")
                }
            }
        }
    }
}

Then i called this block of code to run the query
let query = CheckUsernameQuery()
AppSyncManager.Client().fetch(query: query, cachePolicy: .fetchIgnoringCacheData) {
    (result, error) in
    if error != nil {
        print(error?.localizedDescription ?? "")
        return
    }
    let username = result?.data?.checkUsername![0]?.username
}

And I'm Getting the following response
▿ Optional<GraphQLResult<Data>>
  ▿ some : GraphQLResult<Data>
    - data : nil
    - errors : nil
    - source : AWSAppSync.GraphQLResult<Ditto.CheckUsernameQuery.Data>.Source.server
    - dependentKeys : nil

The query works well when i test it in Appsync Api panel but not working in the app. No Error, No Data nothing at all? What am i possibly doing wrong if any of you have an idea just free to let me know, it would be a great help for me. Thanks.

The referenced issue confirmed that this was a problem with the wrong schema files being used. @fans3210 please let us know if you're still having trouble. Additionally, we will be retiring this repo soon as the new AWS Amplify CLI has the codegen features and more, including automatic graphql generation as well as a GraphQL transformer:
https://github.com/aws-amplify/amplify-cli/blob/master/native_guide.md
Could you look to see if the new AWS Amplify CLI flow resolves your issue and if not open something in that repo?

@undefobj Thanks a lot. Sorry forgot about this issue since it's indeed a schema issue and has been resolved on my side for long time.
Ok will take a look


the name "weather-chart-card" has already been used with this registry

HA info:
Core: 2024.7.3
Supervisor: 2024.06.2
Operating System: 12.4
Frontend: 20240710.0
Weather Card Card: V2.4.7
I see this error in the log file:
Error: Failed to execute 'define' on 'CustomElementRegistry': the name "weather-chart-card" has already been used with this registry
message (node_modules/core-js/modules/web.dom-exception.stack.js:25:24)
src/scoped-custom-element-registry.js:45:10
module code (/hacsfiles/weather-chart-card/weather-chart-card.js:19156:22)

Removed the card and restart HA. The error is gone.
Reinstalled the card, reload and the error is written again in the log

Checked the resources and there were more lines.
In removed:
/local/custom_cards/weather-chart-card.js
and now the error is gone.


Screenshot is outdated

Screenshot is outdated as azcopy now supports lots of types of storage account other than just Blob Containers thus making the terminal screenshot very misleading

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

ID: 15911160-5ba2-fb0a-0b24-16b3857e20ce
Version Independent ID: d925bc9f-f825-2f82-168c-6c30d716b9ce
Content: Copy or move data to Azure Storage by using AzCopy v10
Content Source: articles/storage/common/storage-use-azcopy-v10.md
Service: storage
Sub-service: common
GitHub Login: @normesta
Microsoft Alias: normesta

Thanks for reporting this. I have assigned the issue to the content author to investigate further and update the document as appropriate.

Hi @icaromedeiros. Thank you for raising this issue. The screenshot has been updated. Thank you!

#please-close


Fix anchor link in usage readme

While we're here, can someone (maybe @DonggeLiu) please clarify something for me:
Does a project need to already be integrated with FuzzBench in order to get results from this LLM-approach?
I read both blog posts and this is not clear... thank you

FWIW test failures seems to be auth-related, feel free to close this PR if you want, but the anchor link is not working

Thanks for noticing and fixing the link, @marktefftech!

Does a project need to already be integrated with FuzzBench in order to get results from this LLM-approach? I read both blog posts and this is not clear... thank you

Yep, but it is OSS-Fuzz, not FuzzBench : )
In order to build LLM-generated fuzz targets and get their coverage evaluation results, this platform needs to utilize the Dockerfile, build script, and fuzz target of an existing fuzz target of the project.
However, if evaluation is not required, people should still be able to use this framework to generate fuzz targets for projects not yet in OSS-Fuzz.
A side note: FuzzBench also makes use of OSS-Fuzz project. FuzzBench uses OSS-Fuzz fuzz targets as benchmarks to evaluate fuzzers.

FWIW test failures seems to be auth-related, feel free to close this PR if you want, but the anchor link is not working

Yep this is unrelated, I will merge your PR.


Document using Spegel with pull through registries

After carefully reviewing the following containerd-related errors, I deleted the configurations related to "mirror" in /etc/rancher/k3s/registries.yaml and /var/lib/rancher/k3s/agent/etc/containerd/config.toml.tmpl. Now, spegel is working properly.
time="2023-11-20T23:28:55.907708354+08:00" level=warning msg="failed to load plugin io.containerd.grpc.v1.cri" error="invalid plugin config: `mirrors` cannot be set when `config_path` is provided"

That's a bit unfortunate since the docs state:

Spegel does not aim to replace projects like Harbor or Zot but instead complements them.

I was hoping to use spegel in my cluster and zot as a pull thru cache deployed elsewhere. For what it's worth this is my current containerd mirrors:
mirrors:
  docker.io:
    endpoint:
      - https://zot.domain.tld/v2/docker.io
  ghcr.io:
    endpoint:
      - https://zot.domain.tld/v2/ghcr.io
  quay.io:
    endpoint:
      - https://zot.domain.tld/v2/quay.io
  gcr.io:
    endpoint:
      - https://zot.domain.tld/v2/gcr.io
  registry.k8s.io:
    endpoint:
      - https://zot.domain.tld/v2/registry.k8s.io
  public.ecr.aws:
    endpoint:
      - https://zot.domain.tld/v2/public.ecr.aws

I don't see a way to have spegel take over this responsibility it seems like you either have spegel or a pull thru cache, maybe this can be a feature request?
Originally posted by @onedr0p in https://github.com/XenitAB/spegel/issues/212#issuecomment-1830879118

@onedr0p I created a new issue for this as it is not realted to k3s, and can be fixed with more documentation.

Thanks @phillebaba, in the meantime do you have any tips on how to get that working? I don't mind contributing to the docs if I can get it working on my end.

Could you check the docs added to #280 and see if this is enough?

For example
spegel:
  additionalMirrorRegistries:
    - https://zot.example.com                       # not sure only the base domain would work?
    - https://zot.example.com/v2/docker.io # how would spegel know to pull dockerhub images from here?

@phillebaba I don't know if this is related, but I just want to point out how K3s' fork of Containerd handles these cases. They especially added support for regristry rewrites that we are using for exactly this use case, so that we can use our Harbor instance as a pull-through cache for docker hub. See the documentation here: https://docs.k3s.io/installation/private-registry#rewrites
Unfortunately they never managed to get this merged upstream at Containerd.

@ChristianCiach it looks like this could be baked into k3s which probably covers that use-case.
See: https://github.com/k3s-io/k3s/pull/8977

Hey folks, any update on this?


NuGet Commands: Remove EF.Commands.dll from bin on UWP

Currently, the commands package copies the net451 version of EntityFramework.Commands.dll to the output directory on UWP in order to work.
The problem is that .NET Native will try and compile all the assemblies in the output directory. We should look into a solution that doesn't leave this assembly in the output directory when compiling with .NET Native.

Also, we should ensure that libraries is moved when EF commands are in a UWP class library.
Cref https://github.com/aspnet/EntityFramework/issues/5069#issuecomment-216654757


Remove dependency to pandas

Is your feature request related to a problem? Please describe.
It takes a while to install schemachange
Describe the solution you'd like
Since pandas is not used very much in schemachange, it would be good to remove that dependency. You can probably do that with just plain python

Hi! When are you planning to release this PR? Numpy 2.x and Pandas 1.x are incompatible, so this dependency causes errors when using snowflake-python-connector[pandas].


Pmm 10162 verify that there is no grafana enterprise

This PR simply verify that there is no any Grafana ads on Admin page.

https://pmm.cd.percona.com/job/pmm2-ui-tests/17653/allure/#suites/2c502d0c22e43a4b5926a57a4e29e2f9/9942eb22c0e07a6b/


Could not find browser revision 818858. Run "PUPPETEER_PRODUCT=firefox npm install"

Error
Could not find browser revision 818858. Run "PUPPETEER_PRODUCT=firefox npm install" or "PUPPETEER_PRODUCT=firefox yarn install" to download a supported Firefox browser binary.

same issue...


ci: use remote org for docker images

Uses the org of the remote git origin to set docker credentials and docker image org.

Description

Motivation and Context

Regression

How Has This Been Tested?

Types of changes

[ ] 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)

Checklist:

[ ] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[ ] I have added unit tests to cover my changes.

bors merge


Host opcode ProcessEvents missing

VST plugins that send MIDI events use the host opcode audioMasterProcessEvents (8) as described here:
http://www.asseca.org/vst-24-specs/amProcessEvents.html
http://www.asseca.org/vst-24-specs/amCallbacks.html
But this opcode is missing from rust-vst2, it should be here in host.rs after GetTime:
https://github.com/overdrivenpotato/rust-vst2/blob/82a60b5c13e425e5ef4c30338f7eaaec179e14b5/src/host.rs#L42
As it is now, because of the missing opcode, all following opcodes have the wrong numerical representation, they are one less than they should be, e.g. IOChanged is 12 but should be 13 etc.

Should be fixed by 69488687b05858da09bdb2dfef68e7fdc6c78bf5. Nice catch, thanks!


[bug] pre-commit fatal: repository 'https://gitlab.com/pycqa/flake8.git/' not found

Trying to install pre-commit hooks I've got the following error
An unexpected error has occurred: CalledProcessError: command: ('/usr/bin/git', 'fetch', 'origin', '--tags')
return code: 128
expected return code: 0
stdout: (none)
stderr:
    remote: The project you were looking for could not be found or you don't have permission to view it.
    fatal: repository 'https://gitlab.com/pycqa/flake8.git/' not found

I've checked the link 'https://gitlab.com/pycqa/flake8.git/' and got 404, but there is https://gitlab.com/pycqa/flake8-engine.git for instance.
Steps to reproduce:
pre-commit install
pre-commit run

Yes, I saw the problem 2 days ago. It's not on our side, something has happened with the original repo.
As a temporary solution, I commented it in .pre-commit-config.yaml. So, just pull the main branch, please.

@dapladoc already fixed in https://github.com/OML-Team/open-metric-learning/pull/217