Unable to play some videos.

🐛 Bug Report

unable to play some videos, as it throw an exception
Expected behavior
The video  should play normaly, as its type is an mp4
Reproduction steps
The video is of type mp4, just by playing it, it throw some kind of java exception.
Exception detalj
Com.Google.Android.Exoplayer2.ExoPlaybackException: Exception of type 'Com.Google.Android.Exoplayer2.ExoPlaybackException' was thrown. ---> Java.Lang.NullPointerException: Attempt to invoke interface method 'com.google.android.exoplayer2.PlayerMessage com.google.android.exoplayer2.ExoPlayer.createMessage(com.google.android.exoplayer2.PlayerMessage$Target)' on a null object reference
   --- End of inner exception stack trace ---
  --- End of managed Com.Google.Android.Exoplayer2.ExoPlaybackException stack trace ---
com.google.android.exoplayer2.ExoPlaybackException
	at com.google.android.exoplayer2.ExoPlayerImplInternal.handleMessage(ExoPlayerImplInternal.java:354)
	at android.os.Handler.dispatchMessage(Handler.java:102)
	at android.os.Looper.loop(Looper.java:214)
	at android.os.HandlerThread.run(HandlerThread.java:65)
Caused by: java.lang.NullPointerException: Attempt to invoke interface method 'com.google.android.exoplayer2.PlayerMessage com.google.android.exoplayer2.ExoPlayer.createMessage(com.google.android.exoplayer2.PlayerMessage$Target)' on a null object reference
	at com.google.android.exoplayer2.source.ConcatenatingMediaSource.scheduleListenerNotification(ConcatenatingMediaSource.java:513)
	at com.google.android.exoplayer2.source.ConcatenatingMediaSource.handleMessage(ConcatenatingMediaSource.java:495)
	at com.google.android.exoplayer2.ExoPlayerImplInternal.deliverMessage(ExoPlayerImplInternal.java:861)
	at com.google.android.exoplayer2.ExoPlayerImplInternal.sendMessageToTarget(ExoPlayerImplInternal.java:829)
	at com.google.android.exoplayer2.ExoPlayerImplInternal.sendMessageInternal(ExoPlayerImplInternal.java:811)
	at com.google.android.exoplayer2.ExoPlayerImplInternal.handleMessage(ExoPlayerImplInternal.java:328)
	... 3 more

I could post the video if you like.
Configuration
Version: 0.6.8
Platform:

[ ] :robot: Android
[ ] :monkey: Xamarin.Forms

Yes, can you post the url?

The video is a local video.
Here you can retrive it
https://mega.nz/#!loxVQAzI!PPfTuR8xiCJYyTaxm-HPVqAFEX9bDX-oksVDqrZygF8
The name of the file is
#جميلة - #دمدومة | Jamila - #Damdouma[I=exdfhcEjbwY][F=18][P=6].mp4
as its sometimes can become problems.

Was able to resolve the issue by creatng new URI
new Uri("#جميلة - #دمدومة | Jamila - #Damdouma[I=exdfhcEjbwY][F=18][P=6].mp4").AbsoluteUri;
This is where a Resolver  could do its work.
I dont know if this issue should be handled by the library or the user.

I'm not sure if it is possible to use a url resolver right before playing. Would it be useful to add a resolver that works on the beginning, when you add the url to play, or to the queue?

In this case it easy to do it as you said. But there is other cases, when thr url need decripting like i mentiened in MediaItem Resolver #510
I was thinking of somthink like this
in IMediaPlayer have a property.
IMediaItemResolver ItemResolver { get; set; }
The IMediaItemResolver look like below
  /// <summary>
   /// Validate & resolve Mediaitem before playing it
   /// </summary>
   public interface IMediaItemResolver
   {
       /// <summary>
       /// Return null will force the MediaPlayer, to stop playing the item.
       /// Validate and Make changes to mediaItem like MediaUrl
       /// </summary>
       /// <param name="mediaItem"></param>
       /// <returns></returns>
       IMediaItem Resolve(IMediaItem mediaItem);

       /// <summary>
       /// When the resolver fail, you could handle it. like playing the next video , or event change to a
       /// diffrent MediaPlayer that can handle the MediaItem. for example try to decript a youtube URI
       /// when it failed then play the item in youtube player instead eg changing the MediaPlayer.
       /// </summary>
       Action<IMediaManager> OnFailed { get; set; }
   }

And then having an extention like below
        public static bool ResolveMediaItem(this IMediaPlayer mediaPlayer, IMediaItem mediaItem)
        {
            IMediaItemResolver itemResolver = null;
#if ANDROID
            // this is only temporary as the itemResolver exist only in Android
            itemResolver = mediaPlayer.ItemResolver;
#endif
            if (itemResolver != null)
            {
                var resolvedMediaItem = itemResolver.Resolve(mediaItem);
                if (resolvedMediaItem == null)
                {
                    itemResolver.OnFailed?.Invoke(CrossMediaManager.Current);
                    return false; // will prevent setting the source to the mediaplayer
                }
                if (resolvedMediaItem != mediaItem) /// new created IMediaItem from resolver, try and get all the changes
                {
                    foreach (var p in typeof(IMediaItem).GetType().GetProperties())
                    {
                        var value = p.GetValue(resolvedMediaItem);
                        var currentValue = p.GetValue(mediaItem);
                        if (value != currentValue && value != null)
                        {
                            p.SetValue(mediaItem, value);
                        }
                    }
                }
            }
            return true;
        }

If this is a very big change then forget, its alright i will find another way :)


Frequently requires me to "reopen"

Issue Type: Bug
Terminal and files will freeze, then I am prompted to reopen or close visual studio code. This happens frequently (3 times in 10 minutes). Usually when I have my application started and resave a file.
VS Code version: Code 1.24.1 (24f62626b222e9a8313213fb64b10d741a326288, 2018-06-13T17:51:32.889Z)
OS version: Windows_NT x64 10.0.17134

System Info

Item
Value

CPUs
Intel(R) Core(TM) i7-6700HQ CPU @ 2.60GHz (8 x 2592)

GPU Status
2d_canvas: enabledflash_3d: enabledflash_stage3d: enabledflash_stage3d_baseline: enabledgpu_compositing: enabledmultiple_raster_threads: enabled_onnative_gpu_memory_buffers: disabled_softwarerasterization: disabled_softwarevideo_decode: enabledvideo_encode: enabledvpx_decode: unavailable_softwarewebgl: enabledwebgl2: enabled

Memory (System)
15.87GB (8.85GB free)

Process Argv
C:\Program Files\Microsoft VS Code\Code.exe

Screen Reader
no

VM
0%

Extensions (5)

Extension
Author (truncated)
Version

vscode-eslint
dba
1.4.12

prettier-vscode
esb
1.5.0

dotenv
mik
1.0.1

html5-boilerplate
sid
1.0.3

vscode-babel-repl
t-s
0.1.2

What do you get for code --status when this happens? Also check https://github.com/Microsoft/vscode/wiki/Performance-Issues for more.

Do you mean what does it say during the time it freezes if I do a code --status or after? During I can't type anything in the terminal.

I'm not able to type during the issue, however this is what my "code --status" shows afterwards and also a few of the pop-ups that show during the error.

cc @Tyriar for the terminal. can the gpu-renderer freeze the window?

@KileyAdams92 When the freeze happens can you follow the second part of this section: https://github.com/Microsoft/vscode/wiki/Performance-Issues#the-rendererwindow-process-consumes-a-lot-of-cpu. Mainly, creating a CPU profile using dev tools? That will precise point to the code that going crazy

I've heard this once before, were you able to use the DOM-based renderer as suggested in the notification?

I'm going to be completely honest, I'm not sure how to do that.

It looks like this might be the same as #51490.
Can you try going into your preferences and set terminal.integrated.rendererType to dom?

@mark-adams yep that's it. @KileyAdams92 that notification will do it for you by hitting the yes button, if it gave you a chance that is.


No autogenerated commit message when merging branch with conflict

Does this issue occur when all extensions are disabled?: Yes

Version: 1.78.1 (Universal)
OS Version: Darwin x64 22.4.0

Steps to Reproduce:

Merge conflicting  branch through Vscode palette
Expect to see auto-filled message in the commit form

UPD: Looks like the problem exists only when I'm merging branches inside of the git submodule

/gifPlease

Thanks for reporting this issue! Unfortunately, it's hard for us to understand what issue you're seeing. Please help us out by providing a screen recording showing exactly what isn't working as expected. While we can work with most standard formats, .gif files are preferred as they are displayed inline on GitHub. You may find https://gifcap.dev helpful as a browser-based gif recording tool.
If the issue depends on keyboard input, you can help us by enabling screencast mode for the recording (Developer: Toggle Screencast Mode in the command palette). Lastly, please attach this file via the GitHub web interface as emailed responses will strip files out from the issue.
Happy coding!

Not reproducible on latest version.


Add more results / plots

Blocked because animations do not work and heat maps for floating particles are not documented (see also https://github.com/digitalfabrik/opendrift-leeway-webgui/issues/63#issuecomment-1350582193):
Traceback (most recent call last):
 File "/code/leeway/simulation.py", line 141, in <module>
   main()
 File "/code/leeway/simulation.py", line 120, in main
   simulation.animation(background=['x_sea_water_velocity', 'y_sea_water_velocity'],
 File "/code/opendrift/models/basemodel/__init__.py", line 2746, in animation
   self.get_map_background(ax, background, crs,
 File "/code/opendrift/models/basemodel/__init__.py", line 3748, in get_map_background
   data = reader.get_variables(background, time, reader_x, reader_y, None)
 File "/code/opendrift/readers/reader_lazy.py", line 54, in get_variables
   return self.reader.get_variables(*args, **kwargs)
 File "/code/opendrift/readers/reader_netCDF_CF_generic.py", line 330, in get_variables
   self.nearest_time(time)
 File "/code/opendrift/readers/basereader/variables.py", line 410, in nearest_time
   if (time - time_before) < (time_after - time):
 File "src/cftime/_cftime.pyx", line 1574, in cftime._cftime.datetime.__sub__
TypeError: cannot compute the time difference between dates with different calendars

Superseded by #95


Price Importer > add check for invalid json

In my first step I created a json what looked like this
{
    "prices": [
        {
            "sku": "4001513102164",
            "prices": {
                    "value": {
                        "centAmount": 199,
                        "currencyCode": "EUR"
                    },
                "country": "DE"
                }
        }
}

When I just have one price per SKU this is a valid json. Well when I used the importer he logged a lot of wraps to say "nothing to update". After fixing my json to a "valid" json for your endpoint the import finally worked. so my "valid" json looked like this:
{
     "prices": [
        {
            "sku": "4001513102164",
            "prices": [
                {
                    "value": {
                        "centAmount": 199,
                         "currencyCode": "EUR"
                    },
                     "country": "DE"
                }
            ]
        }
 ]
}

I really would love to get some kind of validation error when my json is "not valid" for your import endpoint instead of just getting nothing.

The validation part is still missing yes, but we have a json schema for that.
https://github.com/sphereio/sphere-json-schemas/blob/master/schema/price.schema.json
Have you also looked at the documentation?
https://github.com/sphereio/sphere-product-import/wiki/Price-Importer#sample-input
Also as a side note, prices usually implies more then one, so an array right? ;)

You're absolutely right, but sometimes a customer wishes imports in around 5 minutes hacked so when you think its easy, its not because of too less time to think. I looked into the schema+docs, yes, and I also started my own documentation (https://github.com/emtii/commercetools-json-examples) because its easier/faster for me to read & understand just as best practices ...

Cool!
Anyway, this is still WIP, so any feedback is very welcomed. We will try to improve the documentation on our side as well.
PS: be aware that the schema for the different imports doesn't necessary correspond 1:1 to the actual schema of the API.


Add as_ptr and as_mut_ptr methods

This is useful for moving across a matrix/slice in unsafe code and for converting slices to other types (e.g. ndarray::ArrayView).
Edit: If you'd like to guarantee that the pointer is aligned and non-null for empty matrices too, that would be great. I just didn't know if you'd like to make that guarantee, so I left it out.

Thanks!

If you'd like to guarantee that the pointer is aligned and non-null for empty matrices too, that would be great. I just didn't know if you'd like to make that guarantee, so I left it out.

How would this be done?

It always struck me as a little odd that ptr is a method on Storage, instead of ContiguousStorage.

Having it be a method on Storage seems like it introduces the implicit requirement that it's possible to construct a pointer to the start of the storage space of the matrix, which would preclude implementing, for instance, a HashMapStorage.

I assume you're thinking about a sparse representation with indices as keys and matrix elements as values? From what I can tell, Storage is restricted to accessing elements by offsetting a pointer. (For example, it has a strides method.) Note that getting a pointer and shape/strides is necessary for non-contiguous matrix types like MatrixSlice in addition to contiguous matrix types, and a trait like Storage allows operating in a uniform way on both contiguous and non-contiguous matrices. If sparse representations are desired in the future, I'd suggest adding a separate, more general trait.

Thanks @jturner314! I will merge this now for today's release.
The discussion about alignment is worth keeping in mind so I will create an issue for that.


Can't compile on Windows 10

Hello.
I am unable to compile the project.
Steps I've taken:

Install Visual Studio 2019 Community
Install Qt 4.8.7 (qt-opensource-windows-x86-vs2010-4.8.7.exe)
Install Qt Vs Tools extension
Add Qt 4.8.7 in the Qt Versions settings in the extension
Open MedianXLOfflineTools.sln (from MedianXLOfflineTools-0.6.4.zip) and try to build it

I get the following error:

1>qtmaind.lib(qtmain_win.obj) : error LNK2038: mismatch detected for '_MSC_VER': value '1600' doesn't match value '1900' in allstatsdialog.obj
1>E:\dev\cpp\MedianXLOfflineTools-0.6.4..\Debug\MedianXLOfflineTools.exe : fatal error LNK1319: 1 mismatches detected
1>Done building project "MedianXLOfflineTools.vcxproj" -- FAILED.
========== Rebuild All: 0 succeeded, 1 failed, 0 skipped ==========

Detailed log:
https://pastebin.com/ERcYRXYt

You must use Qt version that was built with the same compiler you're using. In your case:

Visual Studio 2019
qt-opensource-windows-x86-vs2010-4.8.7

Either compile 4.8.7 from source with VS 2019 or use Qt 5 (it has binary package for 2019).

Okay, I have made it work in VS2013 and now at least know that it does compile, that will suffice for now.
It seems that making it work in VS2019 will take some more tinkering, but thanks for pointing me in the right direction.


Simulation framework

This introduces a simulation framework designed to test the server by simulating multiple clients performing various actions over several rounds. The framework reads simulation configurations from YAML files, validates them against a predefined schema, and executes the simulation accordingly.
This contains code for local and remote(AWS) simulation, check README.
Few issues were fixed while testing (af492a79, 28116d05) @louisinger take a look.
Also there is separate PR for handling panic recovery in go routines started before server startup.
Note that AWS setup doesnt work always for many users (100) as there is rate limit for creating many ECS tasks in parallel (for bigger setup AWS support needs to be contacted to increase rate limit).
This should be enhanced with Signet support.
Prev PR was closed cause it had logs of commits and resolving conflicts while rebasing was complicated.
@tiero @altafan @louisinger please review.

@tiero @altafan this now has support for signed, considering it contains two bug fixes i think it should be merged to master asap.

two bug fixes

worth cherry-picking them in a dedicated PR, so we dont hold them for the simulation to be reviewed?


[AppConfig] Support for configuration snapshot

Add APIs to the SDK for the new REST APIs of the configuration snapshot feture in App Config.

Duplicate of #31722


Incorrect identification and extraction of keywords

Example: Who wrote We Will Rock You?
Extraction output:
What X [verb] Y question.
[(5, 'question_id'), ('Who', 'property'), ('Rock You', 'entity'), ('wrote', 'root')]

Another identification problem.
Example: "Bohemian Rhapsody" is a song by which group?
Extraction output:
X of Y question.
[(2, 'question_id'), ('which', 'entity'), ('Bohemian Rhapsody', 'property'), ('song', 'question_word')]

Another identification problem.
Example: What is the country of origin of the Rolling Sones?
Extraction output:
X of Y question.
[(2, 'question_id'), ('origin', 'entity'), ('country', 'property'), ('What', 'question_word')]

Another identification problem.
Example: Who composed the Matthew's Passion?
Extraction output:
X's Y question.
[(4, 'question_id'), ('Matthew', 'entity')]
Matthew->Matthew the Apostle
Q43600

Another identification problem.
Example:Where is David Grohl from?
Extraction output:
{}


feat: Add Table of Contents

Add a Table of Contents section just under Resources, linking to each resource category section within Readme.md.
This will aid navigation.

Thanks so much for the suggestion. Github now does this automatically (if you click on the
 icon to the left of the file title.


[DOCFIX] Add Allocator & Reviewer sections to Caching.md

This change:

Adds Allocator as a section. This is an open interface configuring the allocation strategy, but is not documented.
Adds Reviewer as a section. This is a new interface added in 2.4.1.
Fixes multiple typos and updates a few doc link formats.

@LuQQiu FYI

alluxio-bot, merge this please

alluxio-bot, cherry-pick this to branch-2.4 please


Client Crash

Hi I am having an issue on my Fabric 1.20.1 server where everytime I try to tp to the creative world it crashes and says that it is missing the dimension Building_Dimension:Overworld and I have to be teleported back using a repeating command block. I have Fabric API and Fantasy Library installed as well as the Multiworld Mod.
I have attached the latest log, hope you can figure this out. I really want a creative world to keep people on my server! Please let me know if you need more info or context.
latest (4).log

additionally, I tried doing it in a singleplayer and it crashed me to desktop so it isn't a server issue.

Hello, thanks for opening an issue. I've looked a bit into the file and cant find the issue immediately.
I'll look more into it this week, I've never seen packet, is that the server log or single player log ? If it's single player, if you have the server only logs that could also help.
Are you using a premade modpack ? If so, I could download it to test more in depth.

Little follow up, Could you try removing ImmersivePortals.
If it works again, I'll try to integrate it to work.

Thanks for the response! I removed Immersive portals, and it worked! So, if it is possible to integrate that mod, that would be awesome! I am also having a new issue where every time I tp to the creative world and back (in single player) my advancements get reset. If you want to test the modpack it is a custom modpack but it was approved by CurseForge, so just search "SteamCraft - Train focus". You will have to add the Building Dimension mod since it is server-side only. Thank you again for responding so quickly!
Here are the singleplayer logs:
latest.log

No problem, for the achievements It should give them back when going back I'll try to test it a bit more as giving Achievements is a bit hard and something might be broken.

Okay, after a bit more testing I do retain my advancements it just gives the popups every time. A tad annoying, but not the end of the world. I would rather have immersive portals be compatible :)

Yes i tried to patched it by turning the gamerule off but it seems like it's not working, might be due to the server-side only. I'll test it more.
For immersive portals I've looked a bit online and haven't found a quick fix but I hope it's not due to the server-side only or the fact I'm loading dimension just after the server started.

I wont be avaliable for a few hours, if you need to reach me you may contact me on discord "lordmaximus05". I seem to be having a lot of unique problems such as: I logged off in the creative world and lost my inventory lol (have replicated so not a one time issue) so I will do some more intensive testing tonight by removing mods and such so you don't have to do so much work!

Sure, I'll send you a request.
Also if you want to only include the mod for your server, you could do a server zip for example like : https://www.curseforge.com/minecraft/modpacks/create-astral/files/4729689
At the bottom of the page there is "Additional files" with the server zip, also useful if there are others server side mods.


Documenation Improvements and Updates

How to Review:
This is a large documentation PR with lots of changes to formatting, wording, and site organization. In order to effectively review,  please check out this branch (tyler/more-code-samples) and run make develop-docs-comprehensive to see a development version of the site.
Changes (selected)

Update documentation to reflect that we support Python 3.9+
Document usage of ask_ml and ask_confident
Reorganize site to improve consistency and flow (most significantly, docs for developing applications are separate from docs detailing sample applications)
Document use of framegrab and tuning of motion detection
Document Alerts, update documentation for the edge-endpoint and groundlight/stream

Preview of the changes / reorganization

Thanks brandon! merged your changes into this branch

There are lots of line changes for sure, but the reason there are tens of thousands is due to the node package updates in package-lock.json


Update index.html

Added images and the real-time prices of the new cars.
🚀 Pull Request
Description
I wanted to inform you that I have made significant updates to the cab rental website. Here are the key changes:
Updated Images: I have replaced the car images with real cab photos to provide a more authentic representation of our fleet.
Modified Pricing: The prices for daily rentals have been revised to better reflect current market rates and enhance our competitiveness.
Enhanced User Experience: I believe these changes will improve the overall user experience and encourage more bookings.
Fixes # (issue)

Added images of cars
Name of cars
Pricing

Screenshot of final output/video

@07sumit1002 Kindly Check and let me know.


Impact on Data Flows?

The Synapse data flows currently use the Spark 3.1 runtime. There is no way to configure the Spark version there.
What would happen when the runtime reaches EOL next week?
Are there plans to migrate data flows to a newer Spark runtime? If so, when?

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

ID: d04f52b2-6225-3f26-cf16-0fbb9671ead2
Version Independent ID: 45a04e29-843c-448a-dacb-c4f9610d1249
Content: Azure Synapse Runtime for Apache Spark 3.1 (EOLA) - Azure Synapse Analytics
Content Source: articles/synapse-analytics/spark/apache-spark-3-runtime.md
Service: synapse-analytics
Sub-service: spark
GitHub Login: @ekote
Microsoft Alias: eskot

@VitalyMCT
Thanks for your feedback! We will investigate and update as appropriate.

@VitalyMCT
Based on the provided document, the Azure Synapse Runtime for Apache Spark 3.1 will reach its end of life on January 26, 2023. After that date, the runtime will not receive bug and feature fixes, but security fixes will be backported based on risk assessment.
Regarding the migration of data flows to a newer Spark runtime, I'm not sure about the specific plans of Microsoft. However, it is recommended to upgrade your Apache Spark 3.1 workloads to version 3.2 or 3.3 at your earliest convenience. You can review the official Spark 3.2 release notes and migration guidelines between Spark 3.1 and 3.2 to assess potential changes to your applications. For more details, you can read the Azure Synapse documentation on Apache Spark version support and Azure Synapse Runtime for Apache Spark 3.2.

@RamanathanChinnappan-MSFT
Thanks for your response.
To clarify, the question is specifically about the Spark runtime used for (formerly called "mapping") data flows. Not Spark Pools. Data flows have an internal, Microsoft-controlled Spark runtime. Customers have no control over its versioning.
An example from one of the data flow runs:

@VitalyMCT
I've delegated this to @ekote, a content author, to review and share their valuable insights.

@VitalyMCT - Mapping Data Flows Spark runtime migration from 3.1 to 3.3 is ongoing at present. Migration is planned to be completed soon.

@ekote Following up - can this issue please be addressed?
Our customers are currently getting emails that include the following. This is very confusing. Their data flows are on 3.1 and they cannot migrate. What's the impact?

Followed up with author.

@VitalyMCT - If the reason for using Azure Synapse Spark Pool 3.1 is DataFlows, then that is correct - no work on your side. The Azure Data Factory team is responsible for migrating customers from 3.1 to 3.3 and then to 3.4.

@ekote
Regarding

If the reason for using Azure Synapse Spark Pool 3.1 is DataFlows

That's part of the question. What's triggering those alerts? Is it the data flows? We don't have Spark 3.1 pools running.
We've also been awaiting Microsoft's reply to the questions from April 8th in this issue thread.

@VitalyMCT - Today, that thread landed in my inbox, and I replied immediately. Thank you.

@VitalyMCT I agree with @ekote, we can't give any non-public info or other roadmap info here.  These questions might have been best answered in an Azure support case. This method - GitHub Issues - is for documentation problems only and has no SLA. Docs really can't make future promises, so we're unlikely to document some of the answers you've asked, but they may be addressed future Azure blog posts or Azure updates.  I do understand the frustration here, but this communication method isn't what you're looking for, as is evident by the turnaround time.
I recommend Microsoft Q & A (Azure Synapse Analytics), Azure Synapse Analytics feedback, Microsoft Support, or your support partners. These avenues donger product group attention and can receive official updates.
Regardless, thanks again for this feedback.

#please-close

For anyone following this thread, it seems the Data Flow runtime was migrated to 3.3 recently. This is based on pipeline output telemetry showing that Spark version.
In the absence of any communication it remains unclear whether this upgrade has been completed globally, in selected regions only, or with any other scope.
According to https://learn.microsoft.com/en-us/azure/synapse-analytics/spark/apache-spark-33-runtime, this runtime provides supportability up to March 31st of next year.
There is definitely remains a communication gap here that needs to be closed here for customers to be able to rely on Data Flows in Synapse in production scenarios.


systemd Client Nodes Not Properly Leaving Cluster on Shutdown

Details
OS: Ubuntu 16.04
Consul: 0.6.4 and 0.7.0
Puppet: 4.6.1
Puppet Module: 1.1.0
Puppet Config:
consul::config_hash:
  client_addr: <IP_ADDRESS>
  data_dir: /opt/consul
  datacenter: "datacenter"
  dogstatsd_addr: "localhost:8125"
  encrypt: --OMITTED--
  enable_syslog: true
  log_level: INFO
  retry_join:
    - "consul.example.com"
consul::pretty_config: !!bool true
consul::version: 0.7.0

Behavior
On shutting down servers running the Consul client on Ubuntu 16.04, they show up as failed in consul members and the web UI afterwards. This was not an issue in 14.04 using Upstart where they'd be gracefully shutdown, e.g. left. Tried both Consul 0.6.4 and Consul 0.7.0 (with its new leave defaults for clients), to no avail. Seems like maybe the systemd service configuration might not be properly ordered.

That’s interesting, I will look into that and submit a patch (or update #301 if it doesn’t get merged before then). Alternatively, we can file a bug to hashicorp/consul for not handling the TERM signal like any other POSIX daemon. (According to documentation, they use the INT signal instead for that purpose.)

A lot of folks add a consul leave to the init script on shutdown to keep this clean....

@armon: I see that the behaviour for client-mode agents has changed in 0.7, and they are now shutting down gracefully by default – good!
Can you expand on why leave_on_terminate still defaults to false on server-mode agents?
Thanks,
Amir

It's mostly a safeguard to keep people from shooting themselves in the foot with respect to the consensus protocol. If you have servers leave when they terminate, they will remove themselves from the Raft consensus pool. Usually adding/removing servers is very rare, so we preferred to be conservative and avoid having automation accidentally cause a quorum loss or split brain.

It's mostly a safeguard to keep people from shooting themselves in the foot with respect to the consensus protocol. If you have servers leave when they terminate, they will remove themselves from the Raft consensus pool.

Absolutely right! Thanks, I almost missed that. Going to revert my patch at https://github.com/solarkennedy/puppet-consul/pull/301/commits/bcb5e92e7dbd7f9d10500e062b7cf9f5c9b86397 with a different one, that allows customisation of KillSignal=, and defaults to SIGTERM.


Warp non-VRT output files

Fixes #13

codecov points out this needs a test for the new warning -- check class explicitly

also, the file size test is cool and all, but maybe we test explicitly against a VRT to guarantee


EntitySpawner Class Needs Method To Set Builder Class

The director attribute in entity spawner can can be final, however it needs to be able to update the builder class it uses since the entity spawner needs to be able to construct enemies and players of different body types (classes).
I recommend creating a method in EntitySpawner that passes to the directors method.
public void changeBuilder(IEntityBuilder builder) {
        this.builder = builder;
    }

Decided to fix this as I have some time to kill.


Seeking in video failed Operation not permitted, will use first frame

Not exactly an issue with this gem, but I decided it makes sense to mention it here.

dirkvdb/ffmpegthumbnailer #165

@x-yuri nice note, thnx


Extract is not proposed when a class and a generic version of it are in the same file

I have a file with two classes in it, with the second class being a generic version of the first (Class1 and Class1<T> in a file named Class1.cs).
When i try to move the second class to a new file, the refactoring is not proposed. But if i change the name of the second class to Class2<T>, the refactoring is present.
I guess it has something to do with the fact that Class1<T> would be moved to a file named Class1.cs (i expected a name like Class`1.cs) and that file is already here.

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

ID: 8139dca2-5127-1385-f3a3-d945b12b9d2e
Version Independent ID: c455e23d-6d9b-c0f0-5f7c-be5dea7a5259
Content: Move type to matching file refactoring in Visual Studio - Visual Studio
Content Source: docs/ide/reference/move-type-to-matching-file.md
Service: unspecified
Product: visual-studio-dev15
GitHub Login: @gewarren
Microsoft Alias: gewarren

I realized that it is more a product feedback than a documentation feedback. I will close it and open a new ticket on the product feedback platform.


Add documentation about locking scroll

FYI, there is a popular library for the same purpose: https://github.com/willmcpo/body-scroll-lock
Isn't it worth mentioning though it is a third-party library?
The most basic usage is just calling disableBodyScroll() and enableBodyScroll() on target element.

@chalkygames123 I think we could totally mention it! Although the lib is 1Kb, which is basically as big as the entire of a11y-dialog. But I assume it’s a little more elegant than the solution I mention in the docs, so probably worth suggesting it as well. :)

Glad to hear that 👍 I'm looking forward to the update.


Floor division by zero result differs if index unaligned

Depending on if the indices of two series/dataframes are aligned, floor division by zero can result in either NaN or inf:
In [68]: import pandas as pd

In [69]: l = pd.Series([1, 2, 3], index=[1, 2, 3])

In [70]: r = pd.Series([1, 1, 0], index=[0, 2, 3])

In [71]: l // r    # Unaligned indices
Out[71]:
0    NaN
1    NaN
2    2.0
3    NaN
dtype: float64

In [72]: l.iloc[1:] // r.iloc[1:]    # Aligned indices
Out[72]:
2    2.000000
3         inf
dtype: float64

In [73]: pd.__version__
Out[73]: u'0.18.0'

I would expect the result for index 3 to be the same in either case. In pandas 0.17.1, both result in inf.

so this is expected.
when these are aligned they are converted to floats
In [9]: l2, r2 = l.align(r)

In [10]: l2
Out[10]:
0    NaN
1    1.0
2    2.0
3    3.0
dtype: float64

In [11]: r2
Out[11]:
0    1.0
1    NaN
2    1.0
3    0.0
dtype: float64

In [12]: l2 // r2
Out[12]:
0    NaN
1    NaN
2    2.0
3    NaN
dtype: float64

we only do the 'filling' part on integer series (IOW how the 0/0 -> inf) happens)
In [13]: np.array([0])/np.array([0])
Out[13]: array([0])

In [14]: np.array([0.])/np.array([0])
Out[14]: array([ nan])

IOW for 13 the 0's get filled, but for 14 they don't (they remain NaN).

From a user perspective was not very expected.  It was actually pretty surprising.
Why are integer series converted to floats when aligned?
Also, my NumPy behaves differently than yours apparently:
In [1]: import numpy as np

In [2]: np.array([0]) / np.array([0])
/home/mrocklin/Software/anaconda/bin/ipython:1: RuntimeWarning: invalid value encountered in true_divide
  #!/home/mrocklin/Software/anaconda/bin/python3
Out[2]: array([ nan])

In [3]: np.array([0.]) / np.array([0.])
/home/mrocklin/Software/anaconda/bin/ipython:1: RuntimeWarning: invalid value encountered in true_divide
  #!/home/mrocklin/Software/anaconda/bin/python3
Out[3]: array([ nan])

This actually seems to be a change in how numpy handles things.

Numpy 1.10.4

In [4]: l // r
Out[4]:
0         NaN
1         NaN
2    2.000000
3         inf
dtype: float64

In [5]: import numpy as np

In [6]: np.__version__
Out[6]: '1.10.4'

In [7]: pd.__version__
Out[7]: u'0.18.0'

Numpy 1.11.0

In [4]: l // r
Out[4]:
0    NaN
1    NaN
2    2.0
3    NaN
dtype: float64

In [5]: import numpy as np

In [6]: np.__version__
Out[6]: '1.11.0'

In [7]: pd.__version__
Out[7]: u'0.18.0'

Does pandas special case integer floor division? For aligned partitions, the behavior of 1//0 == inf differs from numpy's of 0 or nan (depending on float or int dtype).
In [51]: l = pd.Series([1, 2])

In [52]: r = pd.Series([1, 0])

In [53]: l // r
Out[53]:
0    1.000000
1         inf
dtype: float64

In [54]: l.values // r.values
Out[54]: array([1, 0])

In [55]: l.values // r.values.astype('float64')
Out[55]: array([  1.,  nan])

In [56]: l.values.astype('float64') // r.values
Out[56]: array([  1.,  nan])

In [57]: l.values.astype('float64') // r.values.astype('float64')
Out[57]: array([  1.,  nan])

In [58]: pd.__version__
Out[58]: u'0.18.0'

In [59]: np.__version__
Out[59]: '1.11.0'


send files to trash rather than deleting entirely

use package Send2Trash. check electron installer builds as expected

Electron has shell.moveItemToTrash which may make this entire issue moot. Try it out.

should be added in 2.3.7, please close once tested

It is sending the folder to trash on deleting a project. I will close this issue.


Add CustomPaint Widget to the Widget Catalog

CustomPaint Widget needs to be added to the Widget Catalog. Make sure to include the description of the widget in the Description tab and add an argument to include link to the official docs of the widget (Refer to any example in widgetList.dart for better understanding. Also add an argument for category of the widget and give it a category accordingly. Showcase all properties of the widget. (Refer to the official docs in case of any doubts)
Please attach screen recording of the same once done in the PR.

@AryanKhubchandani Pls assign this to me.

Sure, good luck!


Use ubuntu 16.04 for coverage build

Image with new ubuntu:20.04 doesn't produce consistent coverage report. we need to investigate that separately, but no need to block current PRs. so we can use ubuntu:16.04 base image temporarily.

Coverage remained the same at 96.19% when pulling 5c1cd58ec340e70e9bd6fe61f069f79e8bcc76aa on narekgharibyan:use_old_coverage into 1d262f0f08697aa715b2c887f4a63880a8765c30 on KeyviDev:master.


Fix 17 migrations - hopefully

This still is not the final solution for the swappable model, but still should be better implementation of the migrations.
Real solution will imply the use of swappable meta attribute, but being still undocumented it may be tricky to properly implement it

@jsma care to review this?

Coverage decreased (-0.06%) when pulling 093d07357ee13d4ea830db136ef037180824ddae on nephila:feature/fix_17_migrations into fe4b2a1965d69b4afcffc511b532109a0916487b on stefanfoulis:develop.

Coverage decreased (-0.06%) when pulling 093d07357ee13d4ea830db136ef037180824ddae on nephila:feature/fix_17_migrations into fe4b2a1965d69b4afcffc511b532109a0916487b on stefanfoulis:develop.

Coverage decreased (-0.46%) when pulling 5f42a54226c7ce5f7a56a282fc4827f26906d051 on nephila:feature/fix_17_migrations into fe4b2a1965d69b4afcffc511b532109a0916487b on stefanfoulis:develop.

Coverage decreased (-0.19%) when pulling d4959702618141c5ee0af80eee6b7a5b4d40007f on nephila:feature/fix_17_migrations into fe4b2a1965d69b4afcffc511b532109a0916487b on stefanfoulis:develop.

Ping @jsma would you review if this solution fixes the issues with Django 1.7?

Sorry, literally in the middle of launching a big project. Might be able to regroup this weekend.

Thanks!

Very belated reply but can confirm no issues so far after upgrading to the latest on develop branch.


EKF2_AID_MASK -- Optical Flow With no GPS -- LeddarOne -- IntelAero

When attempting to arm and takeoff using optical flow in an indoor lab using a LeddarOne sensor on the Intel Aero with the standard downward facing camera, the position lock command fails its check and refuses to take off. Previously I understand that running this EKF2 setup was not possible but I have reason to believe that it may have been patched in. At this point, I am unsure whether it is supported and the problem is user error or if a position lock is required. Has anyone made this setup work? If so, can they share the settings that gave them their positive results? If this particular set up has a known issue, is there a supported optical flow framework using the intel aero to achieve localization in GPS deprived environments?

I am having the same issue with the Lidar Lite V3. Have you been able to over come this? I am on V1.8.2 but trying V1.6.5 today.

No we currently are pursuing using the IR cameras in the lab to run it using visual fusion so that we can get results on our control system. The environment that we would like to operate the drone in will have limited access to absolute position methods though. However I've read the Lite has chronic problems. I've seen people have success with optical flow when flown in conjunction with GPS but I haven't seen anyone state that they've managed to fully use flow


Use SRV lookups for DNS resolving

Fixes #1.

When using the plugin with this PR I get the following error when hazelcast nodes try and connect with each other:
INFO: [<IP_ADDRESS>]:5701 [dev] [3.6-EA2] Connection [Address[<IP_ADDRESS>]:80] lost. Reason: java.io.EOFException[Could not read protocol type!]
What's interesting is <IP_ADDRESS> is not the IP of a pod.
Pod ips are:
<IP_ADDRESS>
172.17.04
So it's trying to connect to the wrong nodes who answer with something unexpected.
This would imply the SRV records don't contain the IP  addresses of the pods.

@jimmidyson any idea? haven't tested yet.

BTW my quickstart that demonstrates the issue can be seen here:
https://github.com/purplefox/ipaas-quickstarts/tree/master/quickstart/vertx/eventbus

Even before the SRV change I'm not really sure how it would work - with an A record, the IP address of the service would be returned.
This would then proxy through to one of the nodes in the cluster. The assumption being I guess that the node would then tell the connecting node about the other nodes in the cluster. However, couldn't the service just connect the node back to itself, as the loadbalancing is random, in which case it would never know about the other nodes.
Perhaps I am missing something but not really sure how this is supposed to work.

Well it looked like it was working properly but I just realised I'd used a headless service for the discovery. That creates SRV records for each pod/port combination targeted in the selector so works properly.

I see, lemme try configuring it as a headless service :)

Ah, but won't a headless service mean no load balancing? We want load-balancing in this case as it's a webapp

Yeah the dns was broken before, i wrote this when I planned to tunnel hazelcast through https

Do you use the same service for web traffic & cluster comms? They should be separate services shouldn't they?
This is my preferred way to do clustering discovery in Kubernetes. The service doesn't need an allocated IP (i.e. it's headless) as no traffic will go to the cluster comms port.
As an example with Elasticsearch I use 2 services, one for REST API (called elasticsearch) that clients connect to & one headless service for cluster discovery (called elasticsearch-masters) that cluster members use for discovery. No cluster traffic goes through the service IP.
DNS queries for cluster comms:
$ dig elasticsearch-masters.demo.svc.cluster.local SRV +short
10 100 9300 8f89f1b7._unknown-port-9300._tcp.elasticsearch-masters.demo.svc.cluster.local

$ dig 8f89f1b7._unknown-port-9300._tcp.elasticsearch-masters.demo.svc.cluster.local A +short
<IP_ADDRESS>

You don't want load balancing for Hazelcast (client -> server, server -> server)

I don't want a service for the cluster communication. I only need a service for the web traffic.
The hazelcast nodes need a mechanism to discover each other. Basically I just want some way for the nodes to be able to see the pod ip addresses of the other nodes.

Right & the way to do that is by a headless service & use DNS or by querying the API server. Querying the API server requires karma for the service account the pod runs under.

But if I use a headless service won't I lose load balancing for web traffic?

With the Elasticsearch example I used above, there are 2 services involved, one headless for cluster discovery, one normal service for client's connecting to REST API.

you configure multiple services, only the hazelcast one is headless

I am pretty much a k8s newb - is it possible to have multiple services pointing at the same set of pods? Any particular pod is both a web server and a hazelcast node.

me too, sorry :)

Absolutely possible. Service endpoints are selected by the label selector of the service. These can overlap, be subset/superset of other service endpoints, etc - do anything you like really.
(Don't get this confused with replication controllers which should never overlap selectors.)

Cool!. Do you have an example Maven project I can copy where a pod is configured with two services?

Looking here:
https://github.com/purplefox/ipaas-quickstarts/blob/master/quickstart/vertx/eventbus/pom.xml#L55
I can see how the service name is configured for a quickstart, but not sure how to configure multiple.

Should I revert this commit for now?

Aiui, I think this is good if it's documented it must be used with headless services.
But I don't think this is sufficient. I believe the HZ Kubernetes plugin should allow pods from different services to also form a cluster - so I think you should be able to supply a set of service names, or perhaps better a selector that allowed you to include services based on their names, e.g. all services starting with "foo." (e.g. foo.microservice1, foo.microservice2, etc)

@purplefox You would probably be better off doing that with a single service that selects pods based on labels. Each pod would have a common subset of labels you could use for the cluster.

Cool. I didn't know you could do that. So it's possible to create a service that forms a "view" over other services? Perfect.

Absolutely.


Change the synced folder to ENV['GOPATH'], "ENV["HOME"] + "/go"

The behaviour is the same with less code :)

thanks!


Animated Sprite setCurrentFrame doesnt imediately set the frame

setCurrentFrame should visibly change the frame that is being displayed instead of waiting for the animation to run.

Requires 199 because of Demo changes

@lavinrp what was alex's original use case here? Currently, you have it so setCurrentFrame requires runAnimation to have been called. This seems ok, and obviously would work if it was paused. But I feel like I feel like I remember alex using this to set the animation, potentially of the button. What was the reason runAnimation wasn't just called here? I am fine with this fix, I just want to check all the use cases we are trying to cover are in fact covered.

Add a better integration to the demo

@gyurgyma This demo is holding up this bug fix. Lets try to put this to bed by 9/14

merged into dev


[stable/prometheus-operator] Update grafana to 4.0.0.

Checklist
[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

[x] DCO signed
[x] Chart Version bumped
[x] Variables are documented in the README.md
[x] Title of the PR starts with chart name (e.g. [stable/chart])

/retest

Hm. Is it another test failure?
Error: object is being deleted: customresourcedefinitions.apiextensions.k8s.io "alertmanagers.monitoring.coreos.com" already exists

/retest

/hold
I got reports of issues with this version of the grafana helm chart.

Hm. Is it another test failure?
Error: object is being deleted: customresourcedefinitions.apiextensions.k8s.io "alertmanagers.monitoring.coreos.com" already exists
@desaintmartin
I think this means CRD does not get deleted properly after the chart has been tested. You might consider to implement hook. See other stable charts having this implemented.

@zanhsieh It is already implemented. Obviously something is wrong here but I don't know what.
https://github.com/helm/charts/blob/master/stable/prometheus-operator/ci/test-values.yaml#L1029
https://github.com/helm/charts/blob/master/stable/prometheus-operator/templates/prometheus-operator/cleanup-crds.yaml

/retest

@desaintmartin
Let's see retest result.

/lgtm

Seems to pass!
Note that this change requires helm >= 2.12.0 (https://github.com/helm/charts/pull/18176, not yet merged, maybe better to wait, hence the /hold). Should we advertise it as well in this chart?

/hold

It would be ok if we bump the major version of this chart to indicate the breaking change with Helm minimum version requirement

Still waiting for https://github.com/helm/charts/pull/18176 to be merged in order to update this PR.

/retest

/hold cancel

/retest

CRD cleanup issue again:
...
I1024 07:52:11.349] Error: object is being deleted: customresourcedefinitions.apiextensions.k8s.io "alertmanagers.monitoring.coreos.com" already exists
...

I have a more permanent fix for this here, waiting for it to get merged: https://github.com/helm/charts/pull/18271

Rebased.

/lgtm

@desaintmartin
Would you volunteer to be another owner of this chart please? @gianrubio does not have interest in this chart anymore (this chart, in other words, literally only has single owner). See:
https://github.com/helm/charts/issues/13867

I can help maintaining this chart but I won't have the time maintaining it on my own! I already maintain way too many charts for smooth PR merging. ;)


[Bug]: There are 2 selections, please use .inds.

Bug Description
I am working on a solution which Combining Text-to-SQL with Semantic Search. Query engine is not working as expected some times. It throws error - "There are 2 selections, please use .inds."
It works some time. In some cases it throws error.
i am using  for local vector_index and sqlite for database, SQLAutoVectorQueryEngine for queryEngine.
Version
0.10.37
Steps to Reproduce
https://github.com/run-llama/llama_index/issues/6982
Relevant Logs/Tracbacks
Traceback (most recent call last):
  File "/home/zwf_llama3_2.py", line 344, in <module>
    response = query_engine.query(text)
               ^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/local/anaconda/lib/python3.11/site-packages/llama_index/core/instrumentation/dispatcher.py", line 274, in wrapper
    result = func(*args, **kwargs)
             ^^^^^^^^^^^^^^^^^^^^^
  File "/usr/local/anaconda/lib/python3.11/site-packages/llama_index/core/base/base_query_engine.py", line 53, in query
    query_result = self._query(str_or_query_bundle)
                   ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/local/anaconda/lib/python3.11/site-packages/llama_index/core/query_engine/sql_join_query_engine.py", line 312, in _query
    if result.ind == 0:
       ^^^^^^^^^^
  File "/usr/local/anaconda/lib/python3.11/site-packages/llama_index/core/base/base_selector.py", line 31, in ind
    raise ValueError(
ValueError: There are 2 selections, please use .inds.

The LLM is supposed to pick one option. For some reason, it was ambiguous and picked two

def load_SQLAutoVectorQueryEngine(sql_tool,vector_tool):
query_engine = SQLAutoVectorQueryEngine(
sql_query_tool = sql_tool,
vector_query_tool = vector_tool,
selector=LLMSingleSelector.from_defaults(
prompt_template_str="Some choices are given below. It is provided in a numbered list "
"(1 to {num_choices}), "
"where each item in the list corresponds to a summary.\n"
"---------------------\n"
"{context_list}"
"\n---------------------\n"
"Using only the choices above and not prior knowledge, return "
"the choice most relevant to the question: '{query_str}'\n",
output_parser=RobustSelectionOutputParser(),
),
sql_vector_synthesis_prompt=DEFAULT_SQL_VECTOR_SYNTHESIS_PROMPT,
callback_manager=callback_manager
)
return query_engine

My prompt words are limited and I can only choose one. The problem is solved.
"choose only the one most relevant to the question: '{query_str}'\n",


fix: data missing from moduleInfo.meta

Follow-up to #6811
Fixes #6766

@aleclarson There's a lot of changes unrelated to the original fix which makes it hard to review. Are you able to rebase the changes and apply the fix for #6766 specifically? Or feel free to close this too and we can continue tracking progress in that issue.


XQuery support

I'm working with the XQuery language at work. BracketPair prior to the PrismJS update supported XQuery okay, but since that update it doesn't handle the language at all. For now, I've reverted to BracketPair 0.10.8, but it's a bit obnoxious when VSCode randomly decides to remove BracketPair inbetween sessions.
Running VS Code 1.21.0 with XQuery support from XML Tools 1.9.2

Thanks for your report. I took a look at XQuery, and while it isn't supported by PrismJs, it seems to have a similar structure as SQL which is.
Could you test if manually changing the language ID to SQL (ctrl-k, m) makes it colorize properly? If it does I will map it also in the code.

I pulled example 9-18 from the Xquery book and took some screenshots.
The best sample code I can recommend is the FunctX Function Library, which contains most combinations of (), {}, [], and comments that I can find.
Also not visible in my screenshots, BracketPair 0.10.8 would highlight the comment parenthesis. Which is not a big deal at all.
Running BracketPair 0.10.8:

Running BracketPair 1.0.20, language set to XQuery:

Running BracketPair 1.0.20, language set to SQL:

That would be great!
On Tue, 24 Apr 2018, 4:22 PM KWeaver87<EMAIL_ADDRESS>wrote:

Hi CoenraadS, I'm curious as to your thoughts with this issue. Should I go
file an issue against prismJs and let that filter down to here?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/CoenraadS/BracketPair/issues/93#issuecomment-383950310,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AFVa4aXODV8_2bCI05ibrHZCk7eQi4ioks5trzUQgaJpZM4SpRZ9
.

Created new issue here: PrismJS/prism#1405

New PrismJS is included in v1.59. Let me know if it doesn't work

Everything seems to work good. Thanks for the update!


effects for hotspots

Would it be complicated to implement effects behavior from layers to hotspotsso we can apply the same color styling?

The restriction is that hotspots are HTML elements, so we can only implement effects that can be expressed in CSS. That means opacity is ok, but colorMatrix and colorOffset aren't (unless there's a clever way to implement them in terms of CSS filter primitives).
In terms of implementation, you'd want to add a Scene#setEffects method, and propagate it to both Layer and HotspotContainer. Scene transitions are a potential complication, since they also use the effects.

How about using feColormatrix if the hotspot has an SVG element and fallback to default behavior in case of IMG element?

I'd prefer HotspotContainer not to mess around with the internal contents of hotspot elements.
However, it appears that the CSS filter property can be set to the URL of an SVG filter element (first example in https://developer.mozilla.org/en-US/docs/Web/CSS/filter). So, assuming this is well supported, HotspotContainer could create an appropriate offscreen SVG element and set filter accordingly.


Revert "run-resinhup.sh: support only btrfs"

This reverts commit aa97eb23e37a2b731ecdd442899e9675a3251232.

@agherzan @floion

@michal-mazurek can you check why the builds are failing and if it is from other causes then merge this PR?

@michal-mazurek did the build succeed?


Skip initial sync if already synced

Should help increase the stability of sim tests.
However does not fix the spammed error
 error: Failed to subscribe to committee subnet message=Cannot collect attestations before regular sync
Error: Cannot collect attestations before regular sync

closing in favor of #2090


💡 [REQUEST] - Blogs Heading Hover Effect Improvement

[x] I am from GSSoC'23.
[x] I would like to be assigned this task.

Section
Description

Concise Description
Improvement in the Hover effect of the Blogs heading

Solution Proposed
Changing some styling of the effect, I can make it look even better

Alternative Solutions
Style changes in the heading design

Additional Context
N/A

Screenshots

Code of Conduct
By submitting this issue, you agree to follow our Code of Conduct.

[x] I follow the Contributing Guidelines of this project.
[x] I have checked the current issues for duplicate problems.
[x] I have read the readme of this repository.

Already assigned issue


fix: added contact form

###Description
This PR Created a  Simple Contact Us Form

Created a simple contact us form in index.html file.
Additional Context

Merged

This is an automated message from Fork, Commit, Merge [BOT].

Thank you for your contribution! Your pull request has been merged. The files have been reset for the next contributor.
What's next?
If you're looking for more ways to contribute, I invite you to check out my other projects. Just click here to find more. These projects contain real issues that you can help resolve. You can also check out the Influences section in the README to find more projects similar to this one.
Also please leave a star to this project if you feel it helped you, i would really appreciate it.
I look forward to seeing your contributions!


refactor: cleanup deferred config

Moving towards unified configuration via config providers.
Overview
Sets defaults, provides config providers that override configuration.
Changes
Describe your changes and implementation choices. More details make PRs easier to review.

Remove all deferred config calls (and removing dependency)
Specifies defaults for dev and test so that development isn't broken.

Testing
Same testing.

Coverage remained the same at ?% when pulling 9ab1b1b8015406c0ccc67ff8453f09fd8dc63b9f on 579-config-cleanup into b753a33dd8fc84d89fb0c8a7985b9120925ccc2e on master.

It seems to me that coveralls ignore_files doesn't really work...

It seems to me that coveralls ignore_files doesn't really work...

aaah, yes! I think the "new" coveralls approach uses paths relative to the project dir, not the app/something dir. I see apps/omg/test/support is reporting coverage
Let's fix it in a separate PR, I can tackle that as a follow up to the coveralls change

Lemme try testing these config providers-it would be better anyhow!

Also @pdobacz, please checkout the branch and let me know if any defaults break your workflow! Ktnx

OK, I've run a couple quick checks:

mix test :ok:
mix test --only integration locally :ok:
mix run ... perf test (no system envs) - just the Alarm install is missing as in the perf fix PR
mix xomg.watcher.start... --config ~/config_samrong.exs - the postgres urls have changed. I think the old ones were ok (never use omisego without suffices dev/test, to steer clear away from clashes). I'd propose:

"postgres://omisego_dev:omisego_dev@localhost/omisego_dev" the default (used in dev)
"postgres://omisego_dev:omisego_dev@localhost/omisego_test" the test one.
An alternative would be to not have a top level (config.exs) default, but define the dev/dev/dev entry only in dev.exs, and thereby force the prod env to define properly
Also I've noticed there can be quite a few of 2019-08-08 09:50:26.100 [error] module=telemetry function=execute/3 ⋅Handler "measure-db" has failed and has been detached. failures (from one to four per start of the apps, in random instants after the bootup). As they show in red and stand out they might be seen as a serious error. Maybe wind down the level to :warn?

I'm pushing a quick fix to the above issues, feel free to discard/squash/mold as you see fit

OK, I've run a couple quick checks:
* `mix test` 🆗

* `mix test --only integration` locally 🆗

* `mix run ... perf test` (no system envs) - just the Alarm install is missing as in the perf fix PR

* `mix xomg.watcher.start... --config ~/config_samrong.exs` - the postgres urls have changed. I think the old ones were ok (never use `omisego` without suffices `dev/test`, to steer clear away from clashes). I'd propose:

  * `"postgres://omisego_dev:omisego_dev@localhost/omisego_dev"` the default (used in `dev`)
  * `"postgres://omisego_dev:omisego_dev@localhost/omisego_test"` the `test` one.
    An alternative would be to not have a top level (`config.exs`) default, but define the `dev/dev/dev` entry only in `dev.exs`, and thereby force the `prod` env to define properly
    Also I've noticed there can be quite a few of `2019-08-08 09:50:26.100 [error] module=telemetry function=execute/3 ⋅Handler "measure-db" has failed and has been detached.` failures (from one to four per start of the apps, in random instants after the bootup). As they show in red and stand out they might be seen as a serious error. Maybe wind down the level to `:warn`?

I'm pushing a quick fix to the above issues, feel free to discard/squash/mold as you see fit

mix run perf test should probably go away now with the latest merge into master.
Telemetry logs come from telemetry, not me.

there's a bit of commit trash because coveralls died in the middle of everything :)

@InoMurko could you please extend :pr: description of WHY deffered config was removed.
It doesn't play well with releases?

@pnowosie since distillery supports runtime config providers, there’s no need to keep them, but they also clash with config providers.

@InoMurko :+1: What's preventing this from being merged? Looks like it's being maintained for a while

currently, this is blocked by performance measurements against trunk...

@InoMurko can we go ahead and move this forward? If needs be, we can come back and fix the performance measurements. I think getting this refactor in is going to help make the application better to work with and implementing the performance measurement fixes we need.

We could, but it needs a synchronous merge with the PR on the devops side #114


Pass APIRequestOptions to the response interceptor

Description
In the response middleware we do not have access to the request method.
This prevents us from being able to differentiate between the PUT|PATCH methods for services with the same endpoints but different methods.

@desean1625 this is available in the new Fetch API client. You can view the demo where I added an example interceptor checking for method type. Would love to hear your thoughts!

Took a look at the changes ultimately it looks like does what I need. I think that dropping the handle bars templates makes the whole repo a little less complex and much easier to follow, but the resulting generated code isn't as clean as the hbs template.
My branch I changes the hbs to include the request options, and added another interceptor hook to grab the results after the parsing, but before it throws an error.

/**
 * Request method
 * @param config The OpenAPI configuration object
 * @param options The request options from the service
 * @returns CancelablePromise<{{#equals @root.$config.serviceResponse 'response'}}ApiResult<T>{{else}}T{{/equals}}>
 * @throws ApiError
 */
export const request = <T>(config: OpenAPIConfig, options: ApiRequestOptions): CancelablePromise<{{#equals @root.$config.serviceResponse 'response'}}ApiResult<T>{{else}}T{{/equals}}> => {
	return new CancelablePromise(async (resolve, reject, onCancel) => {
		try {
			const url = getUrl(config, options);
			const formData = getFormData(options);
			const body = getRequestBody(options);
			const headers = await getHeaders(config, options);

			if (!onCancel.isCancelled) {
				let response = await sendRequest(config, options, url, body, formData, headers, onCancel);

				for (const fn of config.interceptors.response._fns) {

send options to response interceptors
					response = await fn(response,options);

				}

				const responseBody = await getResponseBody(response);
				const responseHeader = getResponseHeader(response, options.responseHeader);

				const result: ApiResult = {
					url,
					ok: response.ok,
					status: response.status,
					statusText: response.statusText,
					body: responseHeader ?? responseBody,
				};

add result interceptor so we can grab errors at an application level, or do special things with results
				for (const fn of config.interceptors.result._fns) {
					response = await fn(result,options);
				}

				catchErrorCodes(options, result);

				resolve({{#equals @root.$config.serviceResponse 'body'}}result.body{{else}}result{{/equals}});
			}
		} catch (error) {
			reject(error);
		}
	});
};

Also the new fetch client doesn't have as obvious results as the ApiResult

Either way great work


adding the test line

/azp run

/azp run

this was testing the lang end to end credentials issue, so closing this PR


Failed to import - @pyscript/core

Failing module

GitHub: https://github.com/pyscript/pyscript/tree/next/pyscript.core
npm: https://www.npmjs.com/package/@pyscript/core

<script type="module" src="https://esm.sh/<EMAIL_ADDRESS>
Error message
This file just works on Chrome but it miserably fails on Safari resulting into a 500 Server Error ... the amount of weird things happening here is more than concerning:
/* esm.sh - error */
throw new Error("[esm.sh] " + "esbuild: Top-level await is not available in the configured target environment (\"es2021\")");
export default null;

You are deliberately breaking working code out of assumptions I believe made via User Agent sniffing and that breaks completely our users in Safari for stuff that instead just works otherwise.
The error is a 500 which is unrelated with the error that is thrown on the JS side.
With Chrome/ium or Firefox browser that end point just produces the expected result.
This is more than unexpected and bad in general ... you can't detect JS capabilities out of user agent sniffing but it's also none of your business to provide errors otherwise perfectly handled by browsers and devtools themselves.
Please fix this or we'll be forced to move away all our demos and docs from your otherwise lovely service, but this incident smells a bit because we work on very latest APIs and JS features and by no mean we need you to tell us what works and what doesn't ... we already test what works and what doesn't in real browsers, not user agents strings.
Thanks.
Additional info

esm.sh version:
Browser version: any Safari or WebKit, including TP

From the docs:

ESBuild Options
By default, esm.sh checks the User-Agent header to determine the build target. You can also specify the target by adding ?target, available targets are: es2015 - es2022, esnext, deno, and denonext.

Just use https://esm.sh/@pyscript/core@0.1.5/core.js?target=esnext

@nicolo-ribaudo not sure what you are suggesting ... this bug is about having this link:
https://esm.sh/<EMAIL_ADDRESS>showing this result in Chrome/ium
/* esm.sh -<EMAIL_ADDRESS>*/
export * from "/v131/<EMAIL_ADDRESS>
showing this result in Firefox
/* esm.sh -<EMAIL_ADDRESS>*/
export * from "/v131/<EMAIL_ADDRESS>
but showing this result in Safari
/* esm.sh - error */
throw new Error("[esm.sh] " + "esbuild: Top-level await is not available in the configured target environment (\"es2021\")");
export default null;

we're just using the CDN and the CDN does something completely unexpected taking over browsers responsibility to tell users what works and what doesn't out of valid JS code that works outside your CDN service.

to clarify, this bug is about esm.sh User Agent sniffing being obtrusive ... I don't think anyone needs to know that https://esm.sh/@pyscript/core@0.1.5/core.js?target=esnext works, I think developers might need to know that if they are targeting a specific version of ECMAScript that shoould be opt-in, not opt-out because by default stuff breaks without reasons to do so ... User Agent sniffing has never been a Web good practice, so this bug is about underlying that's still he case in 2023.
I hope this makes sense.

my question is does safari support top await? if so this is a bug of esm.sh, or even target=esnext wouldn't work in safari

just checked, safari supports top await since version 15:

now i think this is an issue of esm.sh, i will fix ASAP.

i think there are two tricks that can fix this:

using esnext instead of checking use-agent, but considering this is a break change i will not take
adding more specific target, for example using target=safari15 instead of  target=es2021

just checked, safari supports top await since version 15:

do you agree this way forward doesn’t scale and create friction for both you, as maintainer, and users, as consumer of your service, for every single new version of a browser or every possible experimental flag, technical preview, dev or canary channel, of every possible browser?
most importantly, most modules use tools to create already their target / optimized artifact to serve via CDN or at least I do, so that whatever esm .sh does is kinda redundant but even if “smart” it should never break developers intents after build.
I hence see a third option: check a CDN field in package json and, if present, use that exact file AS IS without transforming a single char, eventually adding a comment on top, if strictly necessary.
I’ve been using unpkg field for years now to tell unpkg which file should be served out of the box if the default entry of a module is explicitly defined and I think this won’t break anything because it relies on module updates same way not specifying the version of the module in the URL points at its latest … you can keep the current logic and handle the target burden but you can add this feature which could be handy for every other tool / service too, if widely accepted and used by popular modules.
the target solution smells to me regardless because I might want to use something that should never change or break and might not be even in esbuild yet so please consider some escape hatch that’s not on the url for users to remember but defined in the published module for authors to explicitly describe their intent, thank you.

P.S.

using esnext instead of checking use-agent, but considering this is a break change i will not take

if you do that for safari it’s not a breaking change because safari is broken already without explicit flags.
On top of that, responding with a 500 (a server error) to throw on browsers breaks fetch expectations too so the current state is currently very broken for Service Workers too or even crawlers or testing tools using WebKit… I’m not sure how it’s even possible nobody else filed issues around the current state … apparently nobody test in real browsers or nobody test Safari/WebKit but that’s another story and changing this can’t break more than it is now, imho.

An example of my suggestion for a generic module:
{
  "name": "esm",
  "type": "module",
  "exports": {
    ".": {
      "types": "./types/index.d.ts",
      "import": "./esm/index.js",
      "default": "./cjs/index.js",
      "CDN": "./build/index.js"
    },
    "./client": {
      "types": "./types/client.d.ts",
      "import": "./esm/client.js",
      "default": "./cjs/client.js",
      "CDN": "./build/client.js"
    },
    "./server": {
      "types": "./types/server.d.ts",
      "import": "./esm/server.js",
      "default": "./cjs/server.js"
    },
    "./package.json": "./package.json"
  }
}

Tools already use the exports field and to me it looks logical and simple to add this as explicit opt-in for modules, and default behavior for CDN services like esm.sh ... note that the server doesn't have a CDN field so that it can always be optional but, if present, it should tell the CDN there is a file already prepared to be served AS-IS from the network.
What do you think? In alternative I still think your option 1 would be the best for Safari users which, in the worst case scenario, will have sites trusting this service just working instead of being broken.

An example of my suggestion for a generic module:
{
  "name": "esm",
  "type": "module",
  "exports": {
    ".": {
      "types": "./types/index.d.ts",
      "import": "./esm/index.js",
      "default": "./cjs/index.js",
      "CDN": "./build/index.js"
    },
    "./client": {
      "types": "./types/client.d.ts",
      "import": "./esm/client.js",
      "default": "./cjs/client.js",
      "CDN": "./build/client.js"
    },
    "./server": {
      "types": "./types/server.d.ts",
      "import": "./esm/server.js",
      "default": "./cjs/server.js"
    },
    "./package.json": "./package.json"
  }
}

Vary interesting!! however i don't want to add complexity/cdn-specification to the package.json, currently the exports.import just works, even commonjs module like React.

do you agree this way forward doesn’t scale and create friction for both you, as maintainer, and users, as consumer of your service, for every single new version of a browser or every possible experimental flag, technical preview, dev or canary channel, of every possible browser?

In fact if the server doesn't recognize the User-Agent, then the target esnext is used. Besides the server also serves source map for debugging.
Currently the problem is esbuild can't transform top await and for await to lower version because the wrong target was used for Safari/iOS. So my solution is adding browser target instead of esma version.
For example, https://esm.sh/<EMAIL_ADDRESS>will point to https://esm.sh/v131/<EMAIL_ADDRESS>

however i don't want to add complexity/cdn-specification to the package.json

unpkg uses the unpkg field in the top level of the package.json file and that worked well for years ... webpack or other tools check for the module field so it looks like everyone else didn't have thoughts about polluting package.json ... here I am proposing to use the standard approach for modules imports / requires resolution, adding a field like TypeScript did for types ... it's not obtrusive, it doesn't conflict, it's based on the same standard structure that allows already extensions to it, but if that's too much to handle I don't mind your solution, as long as it happens sooner than later ... that being said:

In fact if the server doesn't recognize the User-Agent, then the target esnext is used.

the server can't recognize runtime/bootstrap flags neither out of the UA, so this approach is once again problematic to me and it requires you to sync on features per every browser release. If you're willing to do so then good luck, but this issue already shows you can't possibly track all features of every browser with ease and you were stuck at Safari 14 (2 major versions behind) and who knows how many sites avoid this lively service because it didn't work OK in Safari Desktop/Mobile.
I hope you'll reconsider the approach in the future and make the target an opt-in, not an opt-out ... sane defaults mean working code as written/intended, magic transpilation/transformations should be explicit, imho.

bw, if I were you, I would at least factor out this table in a separate folder/project so that it's easier for the community to contribute with updates and features: https://github.com/esm-dev/esm.sh/blob/a37fb5604ed627f4ff289d3a74d6bc56e21b1a87/packages/esm-worker/src/compat.ts#L20-L565
this could help other services too and it could allow anyone to build a static page that verifies all fields and, accordingly with the current browser, show if stuff is indeed supported or not ... evaluating snippets too, helping automating any possible update for that huge table of details.

I hope you'll reconsider the approach in the future and make the target an opt-in, not an opt-out ... sane defaults mean working code as written/intended, magic transpilation/transformations should be explicit, imho.

yeah, i like this idea! just a lot of works LOL. and some edge cases should be handled as well, for example if a package has some ohter npm deps then the transpilation(resolver) is a must.

to clarify, bundling (where possible, ignoring dynamic imports, etc) after resolving modules (ESM vs CJS) and minify that code would be the minimal acceptable transpilation as long as the code is not transformed. I have seen Bun and Deno also get some special treatment, but then again these could also fetch stuff via a fake user agent (let's say curl) and just get code as is via esnext ... the thing is that target=auto might do what you do now with UA sniffing, browsers, and that table, but the default should be just esnext and let consumers opt-in for any different target only when desired.
anyway, if Safari now works, I am happy regardless and you can close this ... hopefully there won't be other issues to me for the time being.

just pushed a patch, i think this should fix the safari builds

i agree! using User-agent breaks the consistency of output, i will push it more deep

to clarify, bundling (where possible, ignoring dynamic imports, etc) after resolving modules (ESM vs CJS) and minify that code would be the minimal acceptable transpilation as long as the code is not transformed. I have seen Bun and Deno also get some special treatment, but then again these could also fetch stuff via a fake user agent (let's say curl) and just get code as is via esnext ... the thing is that target=auto might do what you do now with UA sniffing, browsers, and that table, but the default should be just esnext and let consumers opt-in for any different target only when desired.
anyway, if Safari now works, I am happy regardless and you can close this ... hopefully there won't be other issues to me for the time being.

i agree! using User-agent breaks the consistency of output, i will push it more deep.

just pushed a patch, i think this should fix the safari builds

and i added a testing for it, will deploy the patch after testing pass.


Deploy CAP Java App to SAP Business Technology Platform

Tutorials: https://developers.sap.com/tutorials/cp-cap-java-deploy-cf.html
In Step 7 of the tutorial:

In item 4, an example URL should be /odata/v4/BooksService/Books if the user had followed the entire tutorial of Build a Business Application Using CAP for Java, instead of .../catalog/Books
In item 5, not sure what routes mean, there's no mention of routes in the previous step.

@renejeglinsky please respond to this user's feedback.

Hi @zbroadwell ,
thanks a lot for opening this issue. I'll provide a fix soon.
All the best,
René


Fix miscellaneous warning in C++ files of Windows plugin

Pre-launch Checklist

[x] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[x] I read the Tree Hygiene wiki page, which explains my responsibilities.
[x] I read and followed the Flutter Style Guide, including Features we expect every widget to implement.
[x] I signed the CLA.
[x] I listed at least one issue that this PR fixes in the description above.
[x] I updated/added relevant documentation (doc comments with ///).
[x] I added new tests to check the change I am making, or this PR is test-exempt.
[x] All existing and new tests are passing.

If you need help, consider asking for advice on the #hackers-new channel on Discord.

@cbracken can you take a look at this one?

Thank you for your contribution.
I'm going to close this for now since most of these changes don't fix an issue in the template, and in at least one case (removal of the memory header), violates the style guide.
If these are generating warnings in your project, you can certainly make those changes there.


sharded meshes, manifest calls with verify=false

continuing from #470, multiple rebases in that pr seem to have complicated git history

branch name was too long, app engine did not deploy successfully


ImpalaAsyncQuery.status raises exception

Not urgent, but noting

Pushing to future, since this isn't necessary for the next ibis release.


Introduced OutbreakSignatureValidationModule in Kotlin

Android - validate Outbreak payload signature from server.

Ref: https://github.com/cds-snc/covid-alert-app/pull/1614


Flamework Integration

I've had a few back-and-forths with myself regarding the usage of Flamework, and I think it's likely a good idea to integrate it. It will reduce the amount of code duplication with regard to my current approach, and the opinionated nature will ensure I don't do wacky things.
The current issue regarding state synchronization may be easier to solve as I believe we can interface with a component on the server and client, but our logic will remain safe due to the Service/Controller seperation.
I do have a few concerns, the primary one being the potentially large amount of ore components. I think the best way to see if things will work out is probably to just give it a shot.

 As mentioned previously, this is not really a problem with Flamework, just myself, and is probably indicative of me doing something wrong. This might take a bit longer than usual, as I'm doing a refactor of the entire codebase in combination with the resolution of #6.

I think I get things a bit more now. Our current ItemComponent will instead be a PlacedItemComponent, and will be only on the server. It will have the logic for creating ores, functioning as a conveyor, et cetera. As for the client, it will have SelectedItemComponent, and has all the placement-related stuff.
The OreComponent will likely remain shared (still figuring this out).
The PlayerComponent will be moved to the server, and we'll instead have a LocalPlayerComponent on the client for stuff that won't happen on the server.

I'm still wondering how exactly to use OreComponent on the server and client.
It should be readable on the client and server, but only (persistantly) mutable on the server.
It should be reactive to changes on the client and the server. (We can use the attributes, or if that's too unperformant because of the vast amount, disable the change-watching and use manual signals.)
The client should be able to selectively examine these components, not having every single one be sent to them. (How?)
There should be able to be ideally thousands of OreComponents at once.

This might be too much to ask though. Perhaps we should just use a regular old class? I'm not a fan of having to handle the selective-sending and state-synchronization. Maybe we should just rethink our approach.

Update: The restructure is going alright. I’ve noticed a few issues I’m not a fan of, but I think things will sort themselves out. A shame I can’t really spread these commits out.

Pretty much fully migrated after a major rewrite, though there are still a few kinks to be worked out.
More specifically, the “register” system feels a bit weirded within this context, but I’m struggling to think of a better approach, so I suppose it’s okay.
The item placement system is harder to follow using the new system, but it works okay enough, and can be expanded.
As I continue using Flamework, I find myself putting less methods and data on components, which feels like a good thing and a bad thing. I still need to figure out how I feel about it.
I should hopefully have everything merged in a few days. Be wary of the fact this will be one massive refactoring commit, and is absolutely not spaced out at all.

Yuh. #8


Added assertion of recordable list

fixes https://github.com/flutter/flutter/issues/76205
Pre-launch Checklist

[ ] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[ ] I read the Tree Hygiene wiki page, which explains my responsibilities.
[ ] I read and followed the Flutter Style Guide, including Features we expect every widget to implement.
[ ] I listed at least one issue that this PR fixes in the description above.
[ ] I added new tests to check the change I am making or feature I am adding, or Hixie said the PR is test-exempt.
[ ] I updated/added relevant documentation (doc comments with ///).
[ ] I signed the CLA.
[ ] All existing and new tests are passing.

If you need help, consider asking for advice on the #hackers-new channel on Discord.

we are getting an error of missing a key.


No LICENSE file

There is no LICENSE file for this repository.  Is it MIT licensed or something else?

It's already here since https://github.com/Microsoft/Windows-driver-samples/commit/fb242817317ff9b144fad28f4e3884207d101e35: Microsoft Public License


Moving focus using keybinds when there are two windows in the same workspace on some games causes them to black screen (Proton, Steam)

Regression?
No
System Info and Version
Hyprland, built from branch  at commit 9a09eac79b85c846e3a865a9078a3f8ff65a9259  (props: bump version to 0.42.0).
Date: Wed Aug 7 19:17:10 2024
Tag: v0.42.0, commits: 5069
flags: (if any)
System Information:
System name: Linux
Node name: arch
Release: 6.10.3-zen1-2-zen
Version: #1 ZEN SMP PREEMPT_DYNAMIC Tue, 06 Aug 2024 07:47:21 +0000
GPU information:
09:00.0 VGA compatible controller [0300]: Advanced Micro Devices, Inc. [AMD/ATI] Navi 23 [Radeon RX 6600/6600 XT/6600M] [1002:73ff] (rev c1) (prog-if 00 [VGA controller])
os-release: NAME="Arch Linux"
PRETTY_NAME="Arch Linux"
ID=arch
BUILD_ID=rolling
ANSI_COLOR="38;2;23;147;209"
HOME_URL="https://archlinux.org/"
DOCUMENTATION_URL="https://wiki.archlinux.org/"
SUPPORT_URL="https://bbs.archlinux.org/"
BUG_REPORT_URL="https://gitlab.archlinux.org/groups/archlinux/-/issues"
PRIVACY_POLICY_URL="https://terms.archlinux.org/docs/privacy-policy/"
LOGO=archlinux-logo
plugins:
======Config-Start======
hyprlock.txt
keybindings.txt
windowrules.txt
======Config-End========
Description
when im playing titanfall2 and I use the keybinds to change focus to the next window, because there are two windows in the current workspace and because titanfall2 is fullscreen, it rotates the windows, causing titanfall2 to blackscreen.
I'd like to add that moving the focus to another monitor using the mouse, or changing workspaces using binds doesn't trigger this.
How to reproduce

Open titanfall 2
have another window in the workspace (titanfall 2 will always have an empty EA window for some reason)
use movefocus, r keybind to attempt to move focus to other monitor, or other apps
in the current workspace, the windows will swap because they're both fullscreen
watch as titanfall 2's screen turns completely black

Crash reports, logs, images, videos
https://github.com/user-attachments/assets/03a145f7-ea8f-49e8-a809-a7e45760b6a1

same, i don't remember the exact steps i do to produce this but to make it work again i either unfullscreen and refullscreen the game or move the focus between steam and the game

#6914

#6914

isn't this issue for switching workspaces? I've opened a seperate issue because it doesn't happen on workspace switch.

it's related to focus changes

it's related to focus changes

alright thank you


feature request: use zeroed MAC 00:00:00:00:00:00 instead of randomized MAC_AP on AP-LESS attacks

I have known for a long time that sometimes the captured points do not have the correct mac address. The name is correct and the password is also correct, but the mac of wifi station is not correct. It's randomly generated or something?
And sometimes it is not clear whether the real mac is or not, until you manually check it. Maybe you can add some parameter --zero_mac that you write and then such points, instead of an incomprehensible mac that looks like a real one, in hccapx file wrote 00:00:00:00:00:00 or something like that so that it was immediately clear that the mac address is not correct. Or to do it with hcxpcaptool, because this is very confusing.

When converting to cap, I use -O, that is raw. Could it be because of this?

Most of state of the art CLIENTs doing MAC randomization to prevent tracking.
hcxdumptool/hcxlabtool doing the same MAC randomization to prevent counter measures against it.
In addition to that, on directed PROBEREQUESTs, hcxdumpttol/hcxlabtool will use the probed MAC from the CLIENT.
Zeroed MACs are out of the protocol and most of the CLIENTs will not respond to them. So it is not a good idea to use them for an attack and it doesn't make sense to add such an option to hcxdumptto/hcxlabtool.
Please notice that calculating a PMKID or a PKE (from EAPOL) is based on the MAC addresses of AP and CLIENT used during the AUTHENTICATION. We can't change them by the conversion tool because that will lead to uncrackable hashes!
There is no chance to add such an option to the conversion tool.
If you don't want to feed hashcat with EAPOL MESSAGEPAIRs from hcxdumptool ROGUE attacks filter them out and get only authenticated M2M3 MESSAGEPAIRs as described here or use the PMKID:
https://hashcat.net/forum/thread-10253-post-53580.html#pid53580
BTW:
hcxpcaptool is deprecated and not longer maintained. You should use hcxpcapngtool instead.
hash mode 2500 (hccapx) and 16800 are deprecated, too and replaced by hash mode 22000:
https://hashcat.net/forum/thread-10253-post-53580.html#pid53580

I mean, after the AP-LESS Attack, during the records of the information to the HCCAPX file about the AP, write its Mac like 00:00:00:00:00:00 or something like that. Something like a label for the future.
When viewing an APs in a CAP file, immediately know what wifi points with the wrong MAC address.

Partly this information is stored in the pcapng header file and hcxpcapngtool will show you the basic start MAC:
$ hcxpcapngtool test.pcapng
hcxpcapngtool 6.2.4-1-g34a4170 reading from test.pcapng...

summary capture file
--------------------
file name................................: test.pcapng
version (pcapng).........................: 1.0
operating system.........................: Linux 5.13.13-arch1-1
application..............................: hcxdumptool 6.2.0-40-gab5deb3
interface name...........................: wlp5s0f4u2
interface vendor.........................: 784476
openSSL version..........................: 1.1
weak candidate...........................: 12345678
MAC ACCESS POINT.........................: 00269f581765 (incremented on every new client)
MAC CLIENT...............................: e0cb1d72ccd8

This basic MAC_AP (00269f581765) is incremented on every new ESSID (00269f581766, 00269f581767, 00269f5817658,...).
If a CLIENT probe via undirected BROADCAST MAC (ffffffffffff), we use this replacement.
If a client use a directed MAC, we use this one instead.
After converting it to a hash line this information get lost, except of the value of the MESSAGE PAIR field bit 4.
If bit 4 is set, we know that the attack was ap-less and the MAC is possible a random one.
You can use hcxhashtool to identify them:
$ hcxhashtool -i test.hc22000 --apless -o apless.hc22000

OUI information file..........: /home/zerobeat/.hcxtools/oui.txt
OUI entires...................: 30555
total lines read..............: 97388
valid hash lines..............: 97388
PMKID hash lines..............: 13171
EAPOL hash lines..............: 84217
filter by M2..................: requested from client (AP-LESS)
EAPOL written.................: 36490

hashlines containing hashes from AP-LESS attacks will have message pair field *10 and you can assume that the used MAC_AP is mostly a random one.

The same can be done by bash tools to get all AP-LESS hashes:
$ cat hash.hc22000 | grep WPA.02 | grep 10$
or to get all authorizde hashes:
$ cat hash.hc22000 | grep WPA.02 | grep 02$
There is no other way to do this.

Please notice, if we are able to identify a randomized MAC, an AP or a CLIENT are able to do identify our attack, too. In that case they will initiate counter measures against us.

If you don't want that hcxdumptool use randomized MAC_APs, you have to disable AP-LESS attacks.
In that case hcxdumptool will not respond to requests from CLIENTs.

If you don't want that hcxdumptool use randomized MAC_APs, you have to disable AP-LESS attacks.
In that case hcxdumptool will not respond to requests from CLIENTs.

I think I will do so, this is the simplest solution in my case.


Feature Request: API Elements

Howdy!
This is a bit of a big one, but theoretically it would not be a task entirely on your shoulders.
The Apiary folks have done a fair bit of work on API Elements, to help them build out tooling around both OpenAPI and their own API Blueprint format. They found that supporting both languages directly in their code was a giant effort, so they built this abstraction layer to improve things.
I bring it up here, because the entire API specification ecosystem could really benefit from ReDoc supporting API Elements instead of OpenAPI directly. It would not only solve #312 (and split the burden of upgrading OAI in the future), but it would bring a bunch of users this way due to API Blueprint lacking an actively maintained doc renderer.
Snowboard and Aglio are both way behind in features, functionality, looks, and... every other metric.
I don't want to speak for Apiary people, but it's my impression they'd be into working on this with you. I'm sure the React switch is going to take up a lot of your time, but I think if a bit of forward thinking was done into supporting API Elements you could make the React switch part of that effort.
API Elements and a lot of background on API Blueprint is kinda in this podcast if you're interested.
I'd love to see the API community come together with an excellent renderer that can support multiple formats, to help avoid vendor lockin and split the work of turning metadata into human viewable awesome docs.

Hey @philsturgeon,
I was contacted by Emmanuel from Apiary about API Elements support in ReDoc.
I've considered this option from two perspectives:

having this as an underlying data structure for ReDoc
just supporting it

I've investigated into API Elements a bit and so far I won't introduce it as an underlying level:

it's too verbose (just my personal opinion)
potential performance issues (it relies on Refract and ReDoc has many users with huge specs)
no native support for oneOf, discriminator and other complex patterns (while many ReDoc users rely on these heavily)
no ready-to-use converters from other formats

Most likely I will have OpenAPI 3 as an underlying data structure or a bit modified version of it.
I am open to discussion though!
Regarding just supporting API Elements as a ReDoc input we can consider this in the future.
Btw, would just using "Blueprint to OpenAPI converter" fit your needs?

Thanks for the explanation of all that! I'm not aware of Refract and what that means for performance issues, so I trust your judgement there. I'm glad you folks had the chat all the same.
My own needs were solved with APImatic Transformer and would in the future be probably better solved with https://github.com/luckymarmot/API-Flow, which does a similar thing to the tool you linked.
It's less about my needs, and more about "the API ecosystems needs". Whilst things can be converted from one format to another in a one-off fashion, it's usually a fairly lossy and imperfect process, and I'd not try to do it as part of a ongoing pipeline.
Really API Blueprint needs a decent renderer, and ReDoc is amazing. I hoped this would be a cool way to bring ReDoc into the fold, and break down a few walls in the various sub-ecosystems out there. :)

Thanks for linking to API-Flow, it looks very interesting, I will definitely spend some time looking into it!
I see your points and they are really reasonable.
I think it would be great for ReDoc to support other formats so I will try to abstract OpenAPI from ReDoc core through some abstraction layer as much as possible (just to keep rewrite time reasonable). So in the future, we'll be able to finish this abstraction and just implement API Blueprint or other parser which will break down walls :)

I’m really happy you’re into those ideas!
It’s darn unfortunate that API Elements cannot be that abstraction layer though, that’s what it sets out to do! I wonder if there are others around.
Feel free to close this, or keep it for the “abstracting OpenAPI out” issue. Whichever!
Thanks for all the replies.

Hey! I'm the product manager for API Elements. I just wanted to share here a slightly modified version of the email I sent to @RomanGotsiy in response to his questions around API Elements
API Elements Data Structures does support oneOf. The API Elements are meant to be an abstract way of describing any data — not just JSON. That means they support most of the JSON schema functionality, but not all of it.
Fury.js — the wrapper for API Blueprint / OAS2 producing the API elements is currently partially converting basic JSON schemas from OAS2 documents to Data Structures. At this moment for the sake of rendering nice data structure documentation with Attributes Kit even for JSON schema - not only MSON.
The API description team is currently working on implementing the rest of the JSON Schema functionality. I can share the exact supported/unsupported/never-going-to-be-supported JSON Schema features later this week.
Performance of the Fury.js parser is going to be definitely lower than just parsing and dereferencing YAML documents because it’s doing more things. API Elements is decoupling our entire product from the API Description format battle/hype/development and for instance, in case of implementing OAS3 - it’s just matter of creating a new adaptor for it and the format is introduced across the entire product surface. The same for RAML, Postman collections etc...
The parser is very robust and actively maintained. It's handling huge documents (thousands LOCs) in the production Apiary already and we care about the performance of the parser a lot. Fury.js an essential and heavily used component of the UX of the editor in Apiary - it validates the API description as-you-type in the editor with every other keystroke. We’re measuring basic performance continuously and we also monitor our internal parsing service and immediately react to any changes in the editor-parser-editor roundtrip time. Also, any exception in the production parser is causing an incident and our API description team is handling it as a production issue.


SSH keychain items are too separated

Dev Home version
0.5
Windows build number
No response
Other software
No response
Steps to reproduce the bug
Pin the SSH keychain widget and have at least 2 connections listed
Expected result
No response
Actual result
Padding between each list item in SSH card is too tall compared to design spec, which has 4px for the padding

Included System Information
No response
Included Extensions Information
No response

Plan is to update spec to have the larger, 8 px margins. This aligns with the space between the items in the GitHub widgets.


Fix image names

Add suffix -image to image names and -container to generated containers.

@yoheikikuta Can you please review this pull request when you have a time 🙏

Thanks as always @yoheikikuta!


fix(deps): bump monaco from 0.37 to 0.48

Bump monaco in order to get the latest bugfixes and features.
Triggered by attempting to fix diff alignment issue, supposedly fixed in v0.45.

npm publish


Cant get code to execute = Notebook / Pycharm

Hi Altimis, thanks a stack for putting this together, I think I will get really good use of this if I can get it to work. I fear my lack of experience probably half the problem but when I copy your code into Jupyter Lab and change the run line (data = scrap(....)) to the following:
data=scrap(max_date=2020-1-5,start_date=2020-1-1, to_account="tesla", interval=1,limit=10)
(Ive also tried using the format 2020-01-05)
I get the an error:
"usage: ipykernel_launcher.py [-h] [--words WORDS]
[--from_account FROM_ACCOUNT]
[--to_account TO_ACCOUNT] --max_date MAX_DATE
--start_date START_DATE [--interval INTERVAL]
[--navig NAVIG] [--lang LANG]
[--headless HEADLESS] [--limit LIMIT]
[--display_type DISPLAY_TYPE] [--resume RESUME]
[--proxy PROXY]
ipykernel_launcher.py: error: the following arguments are required: --max_date, --start_date
An exception has occurred, use %tb to see the full traceback."
I assume this means I cant run it out of Notebook?
Secondly the modules import datetimeKeys and from selenium.common.exceptions import NoSuchElementExceptiondoesn't doesnt seem to exist? so when using pycharm it just goes grey.
Finally if it's not too much to ask can you add a few lines to the readme.txt with some more examples as I used your exact example in the python terminal and count get it to execute?
I see it working for others in the issues commentary so thanks for the code and appreciate the work given the loss of old school scrapers.

Hi @cunnial , Thank you. You can check this notebook. I'm  currently working on the library, but you can work with Scweet as a module.