Design authentication / tenant flow

Current setup: uses Twilio for Whatsapp integration.

In order to get access to the WhatsApp number (and have the messages being forwarded to our call, a user has to send a specific message to the whatsapp number (this is now hard coded).
Ideal setup would be to have an online tenant management (in the web app?) where a tenant can be created and users can send an activation message to get access to the tenant.
Based on the activated users, we could link their phone number to a specific tenant for the future
Limitation: there would only be one tenant per user selected

Have been thinking about the following :

design agreed - code will be visualized in better way (accessibility!), as per issue #47


thonny: 4.1.4 -> 4.1.6

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for thonny is: Python IDE for beginners
meta.homepage for thonny is: https://www.thonny.org/
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes
Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 4.1.6 with grep in /nix/store/fv6hlrqr4cl3n225cr6lw221nbbk0mh5-thonny-4.1.6
found 4.1.6 in filename of file in /nix/store/fv6hlrqr4cl3n225cr6lw221nbbk0mh5-thonny-4.1.6

Rebuild report (if merged into master) (click to expand)

2 total rebuild path(s)

2 package rebuild(s)

First fifty rebuilds by attrpath

thonny

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/fv6hlrqr4cl3n225cr6lw221nbbk0mh5-thonny-4.1.6 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A thonny https://github.com/r-ryantm/nixpkgs/archive/02aba12b71bd0cdd3df71752f8a2318c5ecb9aa6.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/02aba12b71bd0cdd3df71752f8a2318c5ecb9aa6#thonny

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/fv6hlrqr4cl3n225cr6lw221nbbk0mh5-thonny-4.1.6
ls -la /nix/store/fv6hlrqr4cl3n225cr6lw221nbbk0mh5-thonny-4.1.6/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

2 packages built:

thonny
thonny.dist

Maintainer pings
cc @leenaars for testing.

[!TIP]
As a maintainer, if your package is located under pkgs/by-name/*, you can comment @NixOS/nixpkgs-merge-bot merge to automatically merge this update using the nixpkgs-merge-bot.

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

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

2 packages built:

thonny
thonny.dist

Result of nixpkgs-review pr 341989 run on x86_64-darwin 1

2 packages built:

thonny
thonny.dist

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

2 packages built:

thonny
thonny.dist

Result of nixpkgs-review pr 341989 run on x86_64-darwin 1

2 packages built:

thonny
thonny.dist


Reduce ActivationMapUploader workers to 1 for Semantic Segmentation example

Linked issue(s):
Having 32 workers causes the test_run to hang until the process is manually killed. Having 1 worker allows the run to complete and based on testing is faster than 4 workers.
What change does this PR introduce and why?
Please check if the PR fulfills these requirements

[ ] Include reference to internal ticket and/or GitHub issue "Fixes #NNNN" (if applicable)
[ ] Relevant tests for the changes have been added
[ ] Relevant docs have been added / updated

If we're running this with a single worker, why even use a threadpool? It's also surprising to me that this is faster than 4 workers — this upload process should be IO-bound, no? Slowdown with concurrency hints at this uploader being CPU-bound.

Removed the Threadpool because having one with a single does not have any benefit other than allowing the masks to be uploaded in the background. When I was referering to timing was meant the time for the results to be uploaded (did not have the numbers on the masks itself). However doing this synchronously seems to preform well.

Closing in favor of https://github.com/kolenaIO/kolena/pull/291


The object name contains a Chinese message indicating a signature error

The object_name and path contain Chinese characters, so a signature error will be displayed if the object_name or path is uploaded or downloaded
2023-11-14T03:02:44Z INFO  minio_manager::minio_util] 上传文件名称："test/绿色指针说明手册.pdf"
[2023-11-14T03:02:46Z INFO  minio_manager::minio_util] Err(S3Error(ErrorResponse { code: "SignatureDoesNotMatch", message: "The request signature we calculated does not match the sig
nature you provided. Check your key and signing method.", resource: "/test/test/绿色指针说明手册.pdf", request_id: "17975E351E27D69A", host_id: "e3b0c44298fc1c149afbf4c8996fb92427ae41e4649b934ca495991b7852b855", bucket_name: "", object_name: "test/绿色指针说明手册.pdf" }))

The crates problem is solved by switching to minio-rsc


Rendering error pages on server with RoutingContext

Current guide for server rendering suggest to return simple strings for error pages. But on real sites we often want to render error pages with React too. And we want to use <Link> on error pages as well. The problem is <Link> needs <RoutingContext> in order to work correctly, but we don't have renderProps for it in case of 404 at least (according to the guide).
So what is the best approach to render error pages with React and RoutingContext?

I've manage to work around this problem.
For 404 rendering I've added <Route path="*" component={NotFound} /> at the end of my routes config. And we have different solution for status codes.
And I think it's ok to render simple string in case of some error during routes resolution, which should not happen normally.
But still it's seems a bit awkward. Maybe there is a better solution? Or maybe the server rendering API should be changed somehow?

I think the not found route is the right thing to do. How do you figure out if you're rendering that route for purposes of setting the status code?

If there is some sort of error during routing, then you can't expect the router to still function for your Link components. You should render a static error message and force the client side to re-request via plain a tags, in the hopes that they don't encounter the error again.
The exception being the 404 case, which I've found the NotFound option works pretty well.

How do you figure out if you're rendering that route for purposes of setting the status code?
We use https://github.com/gaearon/react-side-effect for pages meta data like title, description etc., and also specify status codes using it.

Ok, problem seems to be solved for me.
Thanks you for clarification about the NotFound route @taion and @timdorr !
Not sure if this should be closed or not. I'm totally fine with it being closed.


DeprecationWarning: Calling an asynchronous function without callback is deprecated.

> cssnano --no-map.inline dist/styles/main.posted.css dist/styles/main.posted.min.css

(node:6608) DeprecationWarning: Calling an asynchronous function without callback is deprecated.

Which Node version is this?

v7.3.0


Training correlations when applied MBM model to another dataset

HI! I really appreciate your work and plan to follow it in our area. By the way, may I ask a question about the training correlations calculated by torch.corrcoef?
I have applied the MBM model to another dataset and made several changes. However, I have recently found that the model may not perform well under the correlation metric. Could you please tell me the training correlations in your settings, and what is your opinion on this phenomenon?

The attached image shows the models that we have trained recently, with the red and grey lines representing the models.

Hi there,
Thanks for your interest in our work! I actually feel like this correlation looks pretty good already (if you are referring to the correlation between reconstructed fMRI and original fMRI). Also, the correlation may or may not reflect the performance of your downstream task.
You may try on the image reconstruction task and see how it goes! :)
Good luck!

Thank you for your response! I appreciate your feedback on the correlation between the reconstructed fMRI and the original fMRI. I am glad to hear that you think the correlation looks good.
I understand that the correlation may not necessarily reflect the performance of my downstream task, but I will certainly take your suggestion and try it out on the our tasks.


function argument with var

The beautifier formats fine with function function fn(x, y) but with function fn(var x, var y) add a new line per argument that is
function fn(var x,
     var y)

I want all to behave as function fn( x,  y)
Is there an option for that?

Is var valid in that location?  I've never seen that done.

Can you explain what you mean?
I don't understand your comment

I think this is valid javascript:
function fn(x, y)

I think this is not valid javascript:
function fn(var x, var y)

Please point to documentation for this syntax.
Even if it is valid javascript why would you use it?

Mhh, I found it on one of my JS in project (not written by me). I since deleted the Project files as they caused me troubles. Since am not JS maverick, I thought may be its something advanced.
I will close this issue and If I happen to remember it, I will update it.
Sorry!

No problem! :smile:


[GDnative] autogenerate gdnative_api_struct.h from a json

As discussed with @karroffel the other day ^^
It may worth to wait until #11780 is done before merging this one given it add the need to integrate the string_name function in the json then.

X11 and MSVC builds are both failing:

https://travis-ci.org/godotengine/godot/jobs/282548498
https://ci.appveyor.com/project/akien-mga/godot/build/1.0.281

scons: *** [modules\gdnative\include\gdnative_api_struct.gen.h] Source `modules\gdnative\gdnative_api.json' not found, needed by target `modules\gdnative\include\gdnative_api_struct.gen.h'.

thanks @akien-mga I've forget to had the json file...

@touilleMan I'd rather merge this one and then modify #11780, I'm still not sure what to do about the ptr_hash, so I'd like to merge this so we can start working on making the struct of pointers the new default way without that PR getting in the way. It would be a trivial change after that anyway.
If CI is happy then I'll merge, I need this :^)


[NCL-3159] Create tag that matches ver for /adjust

Repour currently creates a commit and tag after and adjustment is done
to a repository. The tag name is using the format 'repour-'.
This commit attempts to change the tag name format to
'<version>'. The new tag format is more meaningful and should
help humans better understand what the tag is used for.
The version information is extracted only when we use PME for the
adjustment phase. PME generates a result JSON file that contains the
version of the Java project after processing. That version is extracted
from the JSON file and is used to generate the new tag name.

I'd prefer that we don't prepend "repour" to the tag name.  Since we are appending the -redhat-X suffix to the version it should already be unique.  And if it's not unique, then repour should return an error saying that we are potentially conflicting with an already released version.
I think adding the name "repour" in the tag annotation comment (if it's not already included) should make it clear enough that the tag was automatically generated.

@pgier Gotcha! we don't use tag annotation, but this can be done. I'll do the necessary changes. Will push the new changes here and will edit the description.

As for me +1 for "not to add the repour prefix"

Repour will now create tag with '' and add to its annotated tag message that the tag was created by repour

LGTM!

Thanks for the review everyone! :)


Ekf2 replay

Replaces #3606

Tested, works well!

Does that mean good to merge?

@LorenzMeier Somebody should review it!

@tumbili Travis fail.

@tumbili Good to merge if my review determines its ok?

@LorenzMeier Mainly have a look at the diff in the logging app. The rest is mostly the replay app itself which won't break anything else.

@LorenzMeier I realized that the sdlog2 app used all of the stack. So I increased it such that it has a bit of free stack left. However, I don't understand this concept of frame size so maybe you can explain this to me.

Added final changes. Will give this one more test flight in replay and non-replay mode. Should be good to go.

@LorenzMeier This is good to go from my side. I was able to reduce the stack size and the frame size again. If someone could just quickly test that logging is working for non replay mode (EKF2_REC_RPL = 0) then it can go in.

Rebased and applied.


Remove debug-leftovers

Followup to #8308

Thanks @Sija!

I noticed this in some debugging and was curious if there was a reason for it.


Load closest match pdf url

I'm updating some code in https://github.com/makeabilitylab/makeabilitylabwebsite/issues/1166 that will regenerate some filenames. I'm worried about how this might create inaccurate links back to our website. I've already found instances using Google Search Console. For example, two of these links won't work after our file renames are completed:

https://makeabilitylab.cs.washington.edu/media/publications/Gamifying_Green_Gamification_and_Environmental_Sustainability_yy7Jx99.pdf
https://makeabilitylab.cs.washington.edu/media/publications/Froehlich_GrandChallengesInAccessibleMaps_2019.pdf

So, then I had the idea of writing a feature where--rather than loading a 404 page--if the specified PDF url is not found, we look for the closest PDF name in our database. I asked Copilot to help... and this seems like a good start:
Copilot
That sounds like a useful feature! You can achieve this by using a combination of Django views and a string similarity algorithm like Levenshtein distance to find the closest match. Here's a step-by-step guide:
1.  Install the python-Levenshtein library:
pip install python-Levenshtein

2. Create a function to find the closest match:
import os
from Levenshtein import distance as levenshtein_distance

def find_closest_match(requested_filename, directory):
    files = os.listdir(directory)
    closest_match = None
    min_distance = float('inf')

    for file in files:
        if file.endswith('.pdf'):
            dist = levenshtein_distance(requested_filename, file)
            if dist < min_distance:
                min_distance = dist
                closest_match = file

    return closest_match if min_distance <= 5 else None  # Adjust threshold as needed

3. Create a Django view to handle the request:
from django.http import HttpResponse, Http404
from django.conf import settings
import os

def serve_pdf(request, filename):
    directory = os.path.join(settings.MEDIA_ROOT, 'pdfs')
    closest_match = find_closest_match(filename, directory)

    if closest_match:
        file_path = os.path.join(directory, closest_match)
        with open(file_path, 'rb') as pdf:
            response = HttpResponse(pdf.read(), content_type='application/pdf')
            response['Content-Disposition'] = f'inline; filename="{closest_match}"'
            return response
    else:
        raise Http404("PDF not found")

4. Update your urls.py:
from django.urls import path
from .views import serve_pdf

urlpatterns = [
    path('pdfs/<str:filename>/', serve_pdf, name='serve_pdf'),
]

This setup will attempt to find the closest matching PDF file based on the filename requested by the user and serve it if the similarity is within the defined threshold. Adjust the threshold in the find_closest_match function as needed to suit your requirements.
Would you like more details on any specific part of this process?

Gemini suggested using difflib:

I'd be glad to help you with that. Here's a comprehensive approach to implementing the requested functionality in Django:
1. Define a Model:
Create a Django model to store information about your PDFs, including their filenames and any relevant metadata:
from django.db import models

class PDF(models.Model):
    filename = models.CharField(max_length=255, unique=True)
    # Add other fields as needed (e.g., description, upload date)

    def __str__(self):
        return self.filename

2. Implement a Custom View:
Create a Django view that handles PDF requests and searches for the closest match:
from django.shortcuts import render, HttpResponse
from .models import PDF
import difflib

def pdf_view(request, filename):
    try:
        pdf_instance = PDF.objects.get(filename=filename)
        # Handle the case where the PDF exists and return it
    except PDF.DoesNotExist:
        # Search for closest match
        all_filenames = PDF.objects.values_list('filename', flat=True)
        closest_match = difflib.get_close_matches(filename, all_filenames, n=1, cutoff=0.8)[0]

        # Check if the match is close enough
        if difflib.SequenceMatcher(None, filename, closest_match).ratio() >= 0.8:
            # Redirect to the closest match
            return HttpResponseRedirect(reverse('pdf_view', args=[closest_match]))
        else:
            # Handle the case where no suitable match is found
            raise Http404("PDF not found")

3. Customize the Search Logic:

Threshold: Adjust the cutoff parameter in difflib.get_close_matches to control the minimum similarity required for a match.
Matching Algorithm: Consider using other string matching algorithms like Levenshtein distance or TF-IDF if necessary.
Additional Criteria: If you have more metadata about your PDFs (e.g., tags, categories), you can incorporate them into the search logic to improve accuracy.

4. Handle PDF Responses:
In the try block of the view, you can use appropriate Django mechanisms to serve the PDF content. For example:
with open(pdf_instance.file.path, 'rb') as pdf:
    response = HttpResponse(pdf.read(), content_type='application/pdf')
    response['Content-Disposition'] = 'attachment; filename="%s"' % pdf_instance.filename
    return response

5. Configure URLs:
Map the view to a URL pattern in your urls.py file:
from django.urls import path
from .views import pdf_view

urlpatterns = [
    path('pdf/<str:filename>/', pdf_view, name='pdf_view'),
]

By following these steps and customizing the search logic to your specific requirements, you can effectively implement a mechanism to find and serve the closest matching PDF in Django when a requested file is not found.

This works on localhost but not on test or production. Not sure why. Still investigating.

Still working on localhost but not on test or production. One idea might be the web hosting service...

Web Server Configuration: If you are using a web server like Nginx or Apache, ensure it is not intercepting the request and serving the file directly. You might need to adjust your web server configuration to pass the request to Django.

ChatGPT suggests we might need to configure things on our Apache server. Not sure why we wouldn't have to do this locally as well (on localhost)

Great, thanks for confirming! Since you're using Apache, you'll need to ensure that your Apache configuration is set up to pass requests for media files to Django rather than serving them directly. Here are some steps to help you configure Apache:

Apache Configuration:
Locate your Apache configuration file, which might be named something like 000-default.conf or your_site.conf, typically found in /etc/apache2/sites-available/.

ProxyPass Configuration:
Ensure that you have a ProxyPass directive to pass requests to your Django application. Here’s an example configuration:
<VirtualHost *:80>
    ServerName yourdomain.com

    # Serve static files directly
    Alias /static /path/to/static
    <Directory /path/to/static>
        Require all granted
    </Directory>

    # Serve media files directly
    Alias /media /path/to/media
    <Directory /path/to/media>
        Require all granted
    </Directory>

    # Proxy requests to Django application
    ProxyPass / http://<IP_ADDRESS>:8000/
    ProxyPassReverse / http://<IP_ADDRESS>:8000/

    # Ensure the following modules are enabled
    # a2enmod proxy
    # a2enmod proxy_http
</VirtualHost>

Enable Required Modules:
Ensure that the necessary Apache modules are enabled:
sudo a2enmod proxy
sudo a2enmod proxy_http
sudo systemctl restart apache2

Check Permissions:
Make sure the media files have the correct permissions and are accessible by the Apache server.

Restart Apache:
After making changes to the configuration, restart Apache to apply the changes:
sudo systemctl restart apache2

By following these steps, you should be able to configure Apache to correctly pass requests for media files to your Django application. If you encounter any issues or need further assistance, feel free to ask!


PEERS banned

This issue tracker is only for technical issues related to HTMLCOIN.
Describe the issue
After upgrqding from v<IP_ADDRESS> to <IP_ADDRESS> , all my peers seem to be banned and appear in the section "BANNED PEERS". I performed a "-reindex" in the hope that would solve the issue... I downgraded back to <IP_ADDRESS> and <IP_ADDRESS> but the issue persists. At the moment, I'm back on <IP_ADDRESS>, but it is still the same
Can you reliably reproduce the issue?
YES
If so, please list the steps to reproduce below:

Start the wallet
Unlock the Wallet
View PEERS tab

Expected behaviour
Peers should be connectable
Actual behaviour
All my peers are in the section "BANNED PEERS". When unbanning the peers, they immediately flip back to banned or disappear. I rebooted my router, but to no avail.
Screenshots.

What version of HTMLCOIN are you using?
v<IP_ADDRESS>-beta
Machine specs:

OS: Win10 Pro x64
CPU: i5-3570K
RAM: 16GB
Disk size: 80GB SSD

Any extra information that might be useful in the debugging process.
debug.log
Any help is appreciated...
UPDATE:
After hours leaving my computer running and wallet unlocked, I have 3 connections and 66 banned peers. Also, I'm still unable to stake, it keeps returning the message "not staking because wallet is syncing".

It is banning all those pre-fork nodes as they are trying to serve your client blocks that are now invalid port-fork.


New functionalities

It would be nice if we could add some more functionalities, like:

A callback for when user has reached the time limit on recording
Possibility to open the trim view controller from an existing audio
Possibility to override icons

Those all are good ideas, and for the second one, it's already there, you can present IQAudioCropperViewController using below methods for existing audio.
- (void)presentAudioCropperViewControllerAnimated:(nonnull IQAudioCropperViewController *)audioCropperViewController;
- (void)presentBlurredAudioCropperViewControllerAnimated:(nonnull IQAudioCropperViewController *)audioCropperViewController;

Can we use the AVAssets ( fetched from NSURL) for cropping?

Can you update docs on how to use existing audio for cropping? and not use record.

Hmm, I think you can use url to open for cropping

Would love any update in the documentation though, since i ll be using a bridging header to use the library in swift

Getting repeatedly error : IQ_FDWaveformView could not load asset: The requested URL was not found on this server.
// Using a button to call function cropAction with the url of the media Item
class ViewController: UIViewController, IQAudioCropperViewControllerDelegate ,MPMediaPickerControllerDelegate {
var test : String? = nil
var data		: [MPMediaItem]	= []
override func viewDidLoad() {
    super.viewDidLoad()
    // Do any additional setup after loading the view, typically from a nib.

    let tempData = MPMediaQuery.songs().items ?? []
    self.data = []
    for mediaItem in tempData {
        if (mediaItem.isCloudItem == false && mediaItem.assetURL != nil) {
            print(mediaItem.assetURL)
            // get title for bpm
            print(mediaItem.title)
            print(mediaItem.playbackDuration)
            self.data.append(mediaItem)
        }
    }

    print(data[0].assetURL)
    test = String(describing:data[0].assetURL)
}

override func didReceiveMemoryWarning() {
    super.didReceiveMemoryWarning()
    // Dispose of any resources that can be recreated.
}

@IBAction func crop(_ sender: Any) {

    cropAction(test)
}
func cropAction(_ item: Any) {
let controller = IQAudioCropperViewController(filePath: item as! String)
controller.delegate = self
controller.title = "Edit"
    controller.barStyle = UIBarStyle.default

controller.normalTintColor = UIColor.magenta
    controller.highlightedTintColor = UIColor.orange;

presentBlurredAudioCropperViewControllerAnimated(controller)
}

func audioCropperController(_ controller: IQAudioCropperViewController, didFinishWithAudioAtPath filePath: String) {
    //Do your custom work with file at filePath.
    controller.dismiss(animated: true) { _ in }
}

func audioCropperControllerDidCancel(_ controller: IQAudioCropperViewController) {
    //Notifying that user has clicked cancel.
    controller.dismiss(animated: true) { _ in }
}

}


Extensions Support

Any thoughts on how to implement extensions?

@sbchisholm, I have encountered extensions rarely and haven't really thought of implementing it yet.
Any suggestions / ideas?

For the use case when serializing objects to protobuf, extensions provide a way to mirror the objects in the protobuf message and support inheritance.  Here's an example of how it can be accomplished in python:
import message_pb2

class A(object):
  def __init__(self, x, y, z):
    self.x = x
    self.y = y
    self.z = z

  def to_protobuf(self):
    proto = message_pb2.A()
    proto.x = self.x
    proto.y = self.y
    proto.z = self.z
    return proto

class B(A):
  def __init__(self, x, y, z, b):
    super(B, self).__init__(x, y, z)
    self.b = b

  def to_protobuf(self):
    proto = super(B, self).to_protobuf()
    ext = proto.Extensions[message_pb2.b_ext]
    ext.b = b
    return proto

message A{
  required string x = 1;
  required string y = 2;
  required string z = 3;
  extensions 100 to max;
}

message B {
  required string b = 1;
}

extend A {
  optional B b_ext = 101;
}

Extensions in python seem to be supported by the ProtoBuf object using a dictionary indexed by the extension number.  I think something similar might work for Julia.  We would probably want to have accessors for get_extenstion, set_extension, and has_extension.

proto3 deprecates Extensions and introduces an Any type which seems simpler.
May be it's better to jump to proto3 instead? Unless there is an immediate requirement.

Moving ahead with proto3 sounds like a good plan, Any does look like a much cleaner approach than the Extensions.


update policy doc to make it easy for read

Signed-off-by: jichenjc<EMAIL_ADDRESS>

These changes look good to me. I am going to let the policy team to take another quick look.
/approve
/assign mprahl
/assign dhaiducek

Thanks for these updates! I'm assuming we need to copy the changes over to the zn/ files? Other than that, I think this is good to be merged.


[Bug] Statistics needed in getReport2()

I would call it a bug but maybe is mandatory for some weird thing. But when I call to getReport2() without statistics block it returns an error.
p:\programdata\anaconda3\lib\site-packages\aanalytics2\aanalytics2.py in getReport2(self, request, limit, n_results, allowRemoteLoad, useCache, useResultsCache, includeOberonXml, includePredictiveObjects, returnsNone, countRepeatInstances, ignoreZeroes, rsid, resolveColumns, save, returnClass)
   2720             dataRequest["statistics"]["ignoreZeroes"] = True
   2721         else:
-> 2722             dataRequest["statistics"]["ignoreZeroes"] = False
   2723         ### Request data
   2724         if self.loggingEnabled:

KeyError: 'statistics'

My json file (modified just in case) is this one, in which I don't have the statistics block.
jsn = """{
    "rsid": "id1",
    "globalFilters": [
        {
            "type": "segment",
            "segmentId": "s21_59c218..."
        },
        {
            "type": "dateRange",
            "dateRange": "2022-01-01T00:00:00.000/2022-02-01T00:00:00.000"
        }
    ],
    "metricContainer": {
        "metrics": [
            {
                "columnId": "0",
                "id": "metrics/visitors"
            },
            {
                "columnId": "2",
                "id": "metrics/visitors",
                "filters": [
                    "0"
                ]
            }
        ],
        "metricFilters": [
            {
                "id": "0",
                "type": "segment",
                "segmentId": "s21_59c218..."
            }
        ]
    },
    "dimension": "variables/daterangeday",
    "settings": {
        "countRepeatInstances": true,
        "includeAnnotations": false,
        "dimensionSort": "asc"
    }
}"""
jsn = json.loads(jsn)

The fix is easy, since I only have to add a random one like this one at the end.
,
    "statistics": {
        "functions": [
            "col-max",
            "col-min"
        ]
    }

But in general, I don't want the statistics but the filteredTotals and totals from summaryData (which I already have), not the col-max and col-min.
Also, with getReport() I could send the json as string, but now I need to parse it with json.loads() before using getReport2(), is not a problem because this way I can modify more stuff, but just in case you wanted to maintain the same behavior in both calls.
Thanks for the library and the new version 😄

Thanks for providing this, probably miss to test that case.
I can probably fix that so you do not have to add a statistics key.
I will do a fix this week.
Some questions as you tested the new version :)
Do you use the RequestCreator and do you find the Workspace class returned by the result better than the old getReport result ?

Tbh I'm using the simplest things. Basically in my company there is some department of analytics that generates this type of reports for business, CEOs, etc... And, although I'm not certain why me xD, I'm doing the automatization of those reports, my normal job is data scientist but the guy who tend to do this job manually doesn't know how to program in python, so he creates the segments in Adobe Analytics and send me the JSONs, which I use to call the API, join the dataframes and put them in BigQuery to use datastudio as tool of visualization for business.
Therefore, I don't use RequestCreator, and there are some differences with the new method (like dict vs params, n_limit vs n_limits or the need to parse it as json), but in general are pretty similar, I like that know I can check the summaryData, which is useful when I'm doing monthly calls and then I have the mean of the data in the summaryData instead of needing to do another call, which will delay the rest of the calls, so I'm trying to do few and big calls, instead of small and many calls to not have problems with the limitations of the API xD.

thanks, appreciate the feedback.
I was not scrupulous enough for the parameters (n_results vs n_result), I will try to support both via kwargs, so you can change the method easily.
Normally, you can pass a dict for the request, not mandatory have a JSON.
It may be badly documented.

The new release should fix this @set92.
Let me know if it works for you to get it to version 0.3.1.post2
I will add the kwargs supports later with another release.

Works, thanks :D


.phased not correct for VCFs containing phased & unphased genotypes.

Hi,
I have a VCF that was processed with HiPhase. The file contains lines with "|" as well as "/" for the GT value. Example:
chrX	1189233	.	T	A	7.4	PASS	.	GT:GQ:DP:AD:VAF:PL:PS	0|1:7:4:1,3:0.75:6,0,19:1189207
chrX	1189258	.	G	A	30.7	PASS	.	GT:GQ:DP:AD:VAF:PL	1/1:13:4:0,4:1:30,12,0

nft-vcf incorrectly assumes that the file is not phased. Looking into the code I realize that a single slash in any line is sufficient to set phased to false (it's true by default and gets updated when reading the file).  I rather think it should be the other way around: A single pipe ("|") should be sufficient to assume the VCF is phased (and set phased to false by default) - assuming the code stays roughly as is.
Further thoughts: The VCF 4.2 spec says that a PS field should be added for phased variants. Maybe one could check for this field instead of looping over the variants. The current approach assumes that a single unphased genotype means the whole VCF is unphased. That seems too strict (or even wrong). If the phasing information is something that is expected to change on a per-variant base, you might rather want to have a phased boolean array with a value for each variant.

Thank you for bringing this up! In our primary use case (genotype imputation from microarray data), a single pipe isn’t sufficient to ensure the file is correctly phased. However, I agree that supporting a single-pipe configuration could be useful for other workflows.
If you’re interested, we’d love to see a Pull Request (PR) from you to implement this feature. My suggestion would be to add a new parameter to the load function to handle an alternative strategy.
See here: https://github.com/seppinho/nft-vcf/blob/a0ff5c90920efcbc12d38500f8b16408bfb8d35a/src/main/java/genepi/nf/test/vcf/VcfFileUtil.java#L14


Comments!

I'm going to regret this in the morning.
As per usual for me, I haven't written any comments. I need to go back and add them.

I've commented the client code.
Good god I did a bad job writing this thing. Serves me right I guess for trying to code at 10pm.

Since I've been rewriting this (see #5), I've been doing better with comments. We'll see how well this goes.
Not closing this until we have a proper release that's well commented.

I've done it! :tada:
It's all thanks to the rewrite (see #5). Contrary to my previous statement, I'll now be closing this when I merge #6.

Resolved in #6.


Typos and unformatted html file

Description
Typo in contributing.md and readme
Screenshots

I would like to fix it, pls assign it
What browsers are you seeing the problem on?
No response
Ready to Work?

[X] I want to work on this issue

@oyepriyansh I had raised a PR. please check  it out

sorry but there is no need to format the current html file, and those are not typos


[Package Issue]: FireDaemon.OpenSSL - installation directory not added to path

Please confirm these before moving forward

[X] I have searched for my issue and not found a work-in-progress/duplicate/resolved issue.
[X] I have not been informed if the issue is resolved in a preview version of the winget client.

Category of the issue
Installation issue.
Brief description of your issue
The folder C:\Program Files\FireDaemon OpenSSL 3\bin is not added the the system path after installation.
Steps to reproduce

winget install FireDaemon.OpenSSL
Open new console session
Run openssl version

Actual behavior
openssl version
openssl: The term openssl is not recognized as a name of a cmdlet, function, script file, or executable program.
Check the spelling of the name, or if a path was included, verify that the path is correct and try again.

Expected behavior
openssl version
OpenSSL 3.1.4 24 Oct 2023 (Library: OpenSSL 3.1.4 24 Oct 2023)

Environment
winget --info
Windows Package Manager v1.6.3133
Copyright (c) Microsoft Corporation. All rights reserved.

Windows: Windows.Server v10.0.20348.2113
System Architecture: X64
Package: Microsoft.DesktopAppInstaller v1.21.3133.0

Screenshots and Logs
No response

This seems to be an installer issue. Because FireDaemon-OpenSSL uses an exe installer, WinGet has very limited control over what happens. In effect, all WinGet does is run the installer with the switches - ./FireDaemon-OpenSSL-x64-3.1.4.exe /exenoui /qn /norestart REBOOT=ReallySuppress.
You can verify the same behavior occurs if you download the installer from FireDaemon and run it directly; Even after a manual install and starting a new terminal session, the directory has not been added to path.
Unless there is an installer switch for adding the directory to path, there really isn't anything WinGet can do to fix this.

Thanks for the quick reply.
I could not find any related options for this in version 3.1.4, however - I found this for the newer 3.2.0 version:

I does not seem to be any installer switches available, though:

Hi @janegilring,
It's nice to see others using and integrating the OpenSSL binaries and installer provided by us (FireDaemon Technologies Limited)!
The executable installer is a bootstrapper created with Advanced Installer. But Windows Installer generally does not provide a way to view public installer properties, so you would either have to query the msi file or - much better of course - rely on a vendor's useful documentation.
You are correct to note that the ability to set the system environment is a recent addition and has been available since the last installer for the latest version of OpenSSL 3.2.0.
You can set the following public installer properties by passing them on the command line:

APPDIR="<folder>" optionally change the installation folder (default %PROGRAMFILES%\FireDaemon OpenSSL 3)
ADJUSTSYSTEMPATHENV=yes optionally prepend the APPDIR\bin folder to the System environment.

This works regardless of whether the installation is performed silently or with a user interface.
E.g.
FireDaemon-OpenSSL-x64-3.2.0.exe ADJUSTSYSTEMPATHENV=yes "APPDIR=C:\Program Files\FireDaemon OpenSSL 3" /exenoui /qn

We will update our documentation accordingly.

@firedaemon-cto Excellent, thanks a lot for the information Klaus.


Come up with RelocationSize for future use for AArch64

This is a preparation change for AArch64 where various relocation values are stored partially:
https://github.com/ARM-software/abi-aa/blob/main/aaelf64/aaelf64.rst#id53
285	15	R_<CLS>_ LDST32_ABS_LO12_NC	S + A	Set the LD/ST immediate value to bits [11:2] of X. No overflow check

and so I introduce RelocationSize::BitRange. I wish the write_to_buffer implementation would be simpler, but it seems Rust is quite strict when it comes to overflow of the shift operations and thus I combine masking with SHR.
Plus, I would like to ask why do you add and subtract rel_info.byte_size for RelocationKind::Relative relocation value calculation? That would make it more complicated for BitRange type.

Plus, I would like to ask why do you add and subtract rel_info.byte_size for RelocationKind::Relative relocation value calculation?

I guess adding, then later subtracting can give a different result if the relocation references a merged string. Adding the byte size of the relocation is likely assuming that the relocation is at the end of the instruction and thus the instruction pointer when the instruction is being evaluated will be pointing to the byte immediately after the relocation. Removing it, does appear to still pass the tests. So maybe we should remove it. If it turns out that it is needed, we can then add additional tests when we add it back again. Might be best to remove it in a separate commit though.
I wonder if the enum will optimise away on x86 where only a single branch is constructed. I guess we should watch the benchmarks as you add the ARM code to make sure x86 doesn't slow down. If it does, then we could consider making RelocationSize be an associated type on the Arch trait - that way the x86 code will only have to deal with bytes. Does ARM always deal with bits? If to does, maybe an associated type might be the way to go.
Removing Copy from a few types seems like a shame. Perhaps skip on using Range and just have start and stop fields?

Does ARM always deal with bits? If to does, maybe an associated type might be the way to go.

I would say the majority of the relocations would be of the BitRange, however, there are also standard relocations that will use the other type. Based on my preliminary measurements, I haven't noticed any measurable slowdown when linking Clang on x86_64 if I replaced all ByteSize(X) with BitRange(0, 8 * X).

Removing Copy from a few types seems like a shame. Perhaps skip on using Range and just have start and stop fields?

Yep, makes sense.
Anyway, I tend to close this PR as I have a more complex patch-set that can add the rudimentary support for AArch64.


Trouble with Expert Mode using the GROQ API

What's the issue? I am unable to perform a query search in Expert Mode using the GROQ API. I need to identify the cause in order to find a solution. Anybody?

Same as https://github.com/rashadphz/farfalle/issues/66 unfortunately no answers yet.

Considering that this is an issue that we are experiencing in at least two people, wouldn't have been better to try asking for logs or something else to try reproduce the error instead of just closing it with "can't reproduce"?


Varnish 7.2.0 is out

Need to add support for the newer varnish versions.
https://varnish-cache.org/releases/rel7.2.0.html#rel7-2-0

Good call @shveik. One thing we discussed many moons ago was supporting both 6.x and 7.x. I'm not sure how different the CLIs and APIs are for varnish between the versions offhand, but hopefully their differences won't impact the operator/controllers in this project. Will try it out next week.

Any news on supporting 7.x?

I started this a while back but got pulled into other things. TBH, I don't remember where I was with it. I have a feature branch for it here. Feel free to take it over it you have time to do so. :) I currently don't have the bandwidth to take this on.


Range input: implement 'disabled'.

In the WebP options "Auto adjust filter strength" disables the subsequent range. The range input in https://github.com/GoogleChromeLabs/squoosh/pull/136 doesn't support this.

Dropping this from v1, as we no longer use disabled ranges.


Ensure test duration is output with a decimal point

Currently, if running tests on an emulator/device with a locale which uses decimal commas instead of decimal points, the test duration attribute is output with a comma.  This can upset JUnit XML parsers, certainly that of Jenkins.
Now the locale is fixed to English when writing out this XML attribute.
Tested before/after on an emulator with German locale.

Thanks Chris!  If I can convince you to give Pulse a try one day (in lieu of Jenkins), then shoot me an email and I'll rustle up a free license for you ;).


Keep scroll position after reloaded.

What's fixed

Fixed  #228

Looks good to me, and build succeeds 👍
I'll merge this soon


Support for container registries exposed on a custom port

Please provide us with the following information:

This issue is a: (mark with an x)

[x] bug report -> please search issues before submitting
[ ] documentation issue or request
[ ] regression (a behavior that used to work and stopped in a new release)

Issue description
Azure Container Apps prevents users from using a private Docker Registry (valid Docker Registry 2.0) that uses a non-standard port. For example: myregistry.domain.com:9443
When creating a new Container App and specifying an registry having a port  in its server field, deployment will not succeed.
Expected behavior [What you expected to happen.]
Azure Container Apps should support private container registries exposed on various TCP ports
Actual behavior [What actually happened.]
Azure Container Apps  does not accept registries that explicitly specify a port

Any update regarding this one?

Any updates? We were planning on using the container app service, but the lack of support for registries running on specific ports would force us to use AWS ECS instead.

This is a significant blocker for our project, as the images we plan to deploy are hosted in an internal docker repo on a non-standard port. Is there any workaround available for this issue, or any timeline for when this issue will be corrected?

Maybe @dariagrigoriu can share some thoughts / updates / roadmap

This should be fixed now. Please reopen if still an issue.


Move compatible version(s) to Initialize-cmdlet

It's much better if we have one place to store what the current compatible version is for Institution Config etc.

Solved by #162


Add a MTGO-compatible export format

Currently, you can export a decklist in plain text or Arena format. It would be really nice to have an option to export it in MTGO format. Although it can mostly consume the plain text format, there are a few situations where these formats differ (MTGO, for example, expects split cards to be A/B rather than A // B). In an ideal world, you'd be able to download a .dek format which would include the correct versions of each card as well... but I think this would require having a little bit more Scryfall data on hand than is currently available.
Due to the nature of MTGO cubing where one player typically has the entire cube in their account and using other players' decklists to trade, it would also be nice if these decklists didn't include basic lands.
I may be able to find some time to work on this myself at some point.

Exporting to the .dek format should be doable (see #347).
I don't use MTGO myself, but after some quick testing it seems pretty lax about the Name field,  but outright crashes if CatID is missing. It seems pretty reasonable to make some requests to Scryfall to get the MTGO ids, so I went with that.
Not sure what to do about the basics, in the case of a regular set draft I guess the user would expect the MTGO export to behave like the other options. Should I allow turning off the completion to 40 cards with basics, rather than make the MTGO export a special case?
Note: The retro artifacts from The Brothers' War do not have a MTGO id in the Scryfall DB (yet?).

Should I allow turning off the completion to 40 cards with basics, rather than make the MTGO export a special case?

Making this configurable per-draft would also work, especially if that persists across sessions.

I realized you can already set the Deck Size to 0 to quickly get the same behavior, this is a little obtuse but may be enough for what's I think is a fringe use case? I'm a little hesitant to add yet another checkbox in this dropdown.
Other than that I think I'm done with my PR #347 for now if you want to give it a try.

I realized you can already set the Deck Size to 0 to quickly get the same behavior, this is a little obtuse but may be enough for what I think is a fringe use case? I'm a little hesitant to add yet another checkbox in this dropdown.

The problem is I'd like to avoid having to teach each of my drafters to do it individually, so having a way to set it for the whole draft would be really valuable.

Other than that I think I'm done with my PR https://github.com/Senryoku/MTGADraft/pull/347 for now if you want to give it a try.

Is there a good way to run a draft against this, or do I need to build it locally?

I realized you can already set the Deck Size to 0 to quickly get the same behavior, this is a little obtuse but may be enough for what I think is a fringe use case? I'm a little hesitant to add yet another checkbox in this dropdown.

The problem is I'd like to avoid having to teach each of my drafters to do it individually, so having a way to set it for the whole draft would be really valuable.

Ah, I see. These settings are client side only right now and I'd rather keep it that way.
If you don't mind trading more cards than absolutely necessary, perhaps you can try exporting the raw list of drafted cards from the draft log yourself instead of asking each player to share their deck list?

Other than that I think I'm done with my PR #347 for now if you want to give it a try.

Is there a good way to run a draft against this, or do I need to build it locally?

Sadly I can't spin up instances for each PR anymore, you'll have to build it locally :(

PR merged and deployed. You can test it on the main instance.
Closing this issue, please open a new one if you find any problem when you get the chance to try it.


Enhancement: deprecation warning should be written to stderr instead of stdout

Description
Command 'accesstoken get' is deprecated, but it still works and outputs wrongly the warning message in stdout stream and invalidates the token.
Steps to reproduce
$ token=$(o365 accesstoken get --resource https://graph.microsoft.com)

$ echo $token
Command 'accesstoken get' is deprecated. Please use 'util accesstoken get' instead eyJ0eXAiOiJKV1QiLCJub25jZSI6ImVKSnlKTkFzNlI3N1libTJWaTFWVExoOXp1dFE1djA1YUxtc3JhcmVvLUkiLCJhbGciOiJSUzI1NiIsIng1dCI6IkN0VHVoTUpt...

Expected result
$ echo $token
eyJ0eXAiOiJKV1QiLCJub25jZSI6ImVKSnlKTkFzNlI3N1libTJWaTFWVExoOXp1dFE1djA1YUxtc3JhcmVvLUkiLCJhbGciOiJSUzI1NiIsIng1dCI6IkN0VHVoTUpt...

Actual result
$ echo $token
Command 'accesstoken get' is deprecated. Please use 'util accesstoken get' instead eyJ0eXAiOiJKV1QiLCJub25jZSI6ImVKSnlKTkFzNlI3N1libTJWaTFWVExoOXp1dFE1djA1YUxtc3JhcmVvLUkiLCJhbGciOiJSUzI1NiIsIng1dCI6IkN0VHVoTUpt...

This is indeed the current behavior. Would you say that outputting deprecation messages to stderr would be a better choice?

The current behavior breaks scripts. So, the message needs to go into some stream other than stdout. I know that "accesstoken get" is depricated, but it still works and it's actually mentioned in help as an alias for "util acceestoken get". We need to assume that people will continue to use it.

We have a number of choices how we could inform people about deprecation. Our goal is to give everyone enough time to update their scripts.
We could print the deprecation in stderr rather than stdout but I'm not sure if that would change the exit code which might be undesirable as well. We could show the message only in verbose or debug mode, with the risk that it's not seen and people are surprised by the command being removed. Does any of these options seem better than what we have now? Do you have other ideas that we should consider to communicate deprecation?

I've had a conversation with the Azure CLI team. They are printing the deprecation and other warning messages to stderr.
I would suggest to do the same in Office 365 CLI.

Thank you for checking with them and I appreciate your help. We'll adjust accordingly to follow the az cli behavior.

Azure CLI team has shared the doc about deprecating commands and arguments and some links to code examples (I know they are developing in Python and Office 365 CLI is based on Node.js, but it still might be useful, I hope.)
A doc about deprecating commands and arguments.
"Our CLI repo depends on knack, which is a basic command-line interface framework. Deprecated information is handled in knack. Azure CLI output it to stderr for normal commands and to stdout for help commands."
You can reference the code here:
Output to stderr for normal commands
Output to stdout for help commands

Thank you for the pointer. That's helpful 👍

We're working on removing Vorpal which gives us more control about CLI's output. Once we've closed #1737, we'll be able to look into this issue.

We've just release a preview version where all secondary output is sent to stderr and only the main command output is sent to stdout. Once again, thank you for your suggestion @alexandair 👏


update to use maintained fork of go bindata

https://github.com/intercom/intercom/issues/93512

Context here: https://www.reddit.com/r/golang/comments/7vv9zz/popular_lib_gobindata_removed_from_github_or_why/


Add welcome splash screen

Added a Welcome Splash Screen, that would be displayed in the start, when the
App is launched.

Codecov Report

Merging #68 (61ee58f) into master (b4e1802) will decrease coverage by 2.71%.
The diff coverage is 0.00%.

@@             Coverage Diff              @@
##             master      #68      +/-   ##
============================================
- Coverage     64.32%   61.60%   -2.72%
- Complexity      425      451      +26
============================================
  Files            78       82       +4
  Lines          1480     1615     +135
  Branches        154      167      +13
============================================
+ Hits            952      995      +43
- Misses          484      564      +80
- Partials         44       56      +12

Impacted Files
Coverage Δ

src/main/java/seedu/address/ui/UiManager.java
0.00% <0.00%> (ø)

src/main/java/seedu/address/ui/WelcomeWindow.java
0.00% <0.00%> (ø)

...rc/main/java/seedu/address/logic/LogicManager.java
60.00% <0.00%> (-15.00%)
:arrow_down:

...in/java/seedu/address/commons/util/StringUtil.java
85.00% <0.00%> (-9.74%)
:arrow_down:

...rc/main/java/seedu/address/model/ModelManager.java
90.38% <0.00%> (-9.62%)
:arrow_down:

.../seedu/address/logic/parser/AddressBookParser.java
80.00% <0.00%> (-4.22%)
:arrow_down:

src/main/java/seedu/address/model/Model.java
100.00% <0.00%> (ø)

src/main/java/seedu/address/ui/MainWindow.java
0.00% <0.00%> (ø)

...rc/main/java/seedu/address/ui/PersonListPanel.java
0.00% <0.00%> (ø)

...main/java/seedu/address/commons/util/FileUtil.java
72.41% <0.00%> (ø)

... and 7 more

Continue to review full report at Codecov.

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


Allow page.HasMenuCurrent() and node.HasMenuCurrent() to proceed with multi-level nested menus

new PR, with fixed tests (first PR was #1226)
The following test fixture was buggy i think, because when i look into the menuItem from
findTestMenuEntryByID(s, "grandparent", "grandparentId") I can see my home menu.
This test worked before because HasMenuCurrent returned after checking the first children, without checking children of children... and so on...
@bep ,  please can you review :)

Will have a look tomorrow.

Merged in 2c22a8b


Better support for _Imports.razor

https://github.com/aspnet/AspNetCore/issues/6374

@rynowak @pranavkm do these changes look reasonable? I wasn't totally sure where to put this logic.

I'm not very familiar with the ProjectSystem, but the change in GenerateCommand seems reasonable.

ProjectSystem side looks fine, however, you'll need to do the same thing for VS4Mac

@NTaylorMullen, I couldn't find the exact place the filekind is set in VS for Mac. It just calls into ProjectSnapshotManager. https://github.com/aspnet/AspNetCore-Tooling/blob/master/src/Razor/src/Microsoft.VisualStudio.Mac.LanguageServices.Razor/ProjectSystem/DefaultRazorProjectHost.cs#L46

@NTaylorMullen, I couldn't find the exact place the filekind is set in VS for Mac. It just calls into ProjectSnapshotManager. https://github.com/aspnet/AspNetCore-Tooling/blob/master/src/Razor/src/Microsoft.VisualStudio.Mac.LanguageServices.Razor/ProjectSystem/DefaultRazorProjectHost.cs#L46

Oh shiiittt, totally forgot we haven't added the host document stuff yet to VS4Mac derp

Oh shiiittt, totally forgot we haven't added the host document stuff yet to VS4Mac derp

We have an item tracking overall components support for VS4Mac - it's not currently in a milestone. I think we should ignore it until we have a schedule and a plan.

🆙 📅


Unable to make stdlib

I tried running make in stdlib as mentioned in #1652. Getting the following error:
error: unknown target triple 'sbf', please use -triple or -arch
make: *** [../target/bpf/solana.bc] Error 1```

You need to use the llvm build with includes our changes,see https://solang.readthedocs.io/en/v0.3.3/installing.html#step-1-install-the-llvm-libraries

I did used that, still the same issue is coming! @seanyoung


docs: Remove partitions and LVM limitation from import guide

LVM and different partition schemes are supported

/ok-to-test


Tabbar spotlight appears 60-80pts above target.

When TabBar type spotlight used, the spotlight appears 60-80pts above the target area.

Can you do a workaround correction about it? Also i am evaluating if adopting this repo or the official one Gecco for a company project. Do you think this project is well suited for future updates and support ?


add resource for datastream lifecycle

resolves #838

Thanks for taking the time to add this in.
Is it possible to have this resource built on the Plugin Framework instead? There's a strong preference here for new code to utilise that framework over the old SDK v2 and we're not re-using any existing SDK code here. This commit is an example of converting to the plugin framework if that helps.
It looks like lint is failing because the generated docs don't match what's committed, which can be fixed by committing the result of make docs-generate

ok, not a problem.

updated


Wallet client trace ID error

Description
When sending a transaction, we see this error:
 An internal error occurred: requesting the transaction review failed: the trace IDs between the request and the response mismatch: expected "...", got "..."
! The request has been canceled.

This is most likely due to sending a request two times from the dApps.
Steps to Reproduce
The bug can't be reproduced consistently (at least we haven't found a way), it will require some further investigation.

connect your wallet to a dApp (e.g. console)
submit an order

Expected behavior
Not having a request error.

https://github.com/vegaprotocol/feedback/discussions/124
Looks like there is already a ticket and fix on its way


EyouCMS v1.5.7 has a vulnerability, stored cross-site scripting (XSS)

Background upload image, by modifying the image name to generate cross-site request forgery
1, first open content management, click Publish document, click Upload file

2. Set up proxy packet capture before clicking upload image, upload a valid JPG image, find the name of the image and change it to:

b949ada.png)
Other impacts: The file name of the vulnerability is saved in the database. When the system is built using phpstudy, a word written into the Trojan horse will appear in "ey_uploads.MYD"  in the mysql directory. If any file contains the vulnerability, the vulnerability will be directly getshell

已修复


feat: allow copying service name from service/dataplane list views

feat(services): add copy button to service name in list view
refactor(CopyButton): use type syntax for props
feat(dataplanes): show copy button for services in list view
Add support for showing a copy button to the TagList component.
Resolves #1747.
Signed-off-by: Philipp Rudloff<EMAIL_ADDRESS>

clicking on the copy button in the listing view navigates me to the detail view.

Good catch. I’ve fixed this now. We had this before and the strategy was making sure the links are nested inside TextWithCopyButton and not the other way around.

I'm not totally sold on the copy button being in the tag thing, but as I said I just took a very quick look and noticed the above thing, so leave that one with me some more.

Me neither. I was thinking to stop using the TagList component specifically for the Dataplane list view. Let me try that. That might just be the solution here.

I was thinking to stop using the TagList component specifically for the Dataplane list view. Let me try that. That might just be the solution here.

I’ve implemented this now. I think I like this better.

Oh, one thing I noticed which might not be fixable easily here (but could be, lmk) is the non-rounded corners on the tooltip/notification things:

I added the Kongponents default border radius to this now. I have work planned to replace CopyButton with KCopy completely, but that’s not priority right now.

I added the Kongponents default border radius to this now. I have work planned to replace CopyButton with KCopy completely, but that’s not priority right now.

Huh, I didn’t realize this already had a border radius. Let me look at this some more.
Okay, so specifically it’s not applied for the services in the Dataplane list. Strange. I’ll look at this a bit. If it’s not easy to fix, let’s do that separately.

Okay, yes, let me look at that separately. The issue is the KTruncate use using overflow: hidden which cuts-off parts of the tooltip making it look like it has no border-radius. That’s not exactly trivial to fix.

That’s not exactly trivial to fix.

Cool don't worry on that one then thanks for looking.
I noticed this also, looks like the service names have turned blue in this PR:

I noticed this also, looks like the service names have turned blue in this PR:

I changed the AppCollection styles for this to fix that and also removed an unnecessary text-decoration: none;.

and also removed an unnecessary text-decoration: none;.

Oh wait, that means we get an underline on hover which I don't think we want

Oh wait, that means we get an underline on hover which I don't think we want

Isn’t that exactly what we want? We have the following base rule for links:
a:hover,
a:focus {
  text-decoration: underline
}

So why shouldn’t it apply to the name cells in AppCollection?

It's not what we want (unless something has changed that I'm not aware of). We spoke about this a good while ago now.
The name of the thing in the row (i.e. the first cell) shouldn't look/act like a link, the entire row is the clickable thing. The text in the first cell shouldn't be blue/underlined/underline on hover or anything that makes it look like a link. It should be black and slightly bold, as close as possible to this:

We have the following base rule for links:

This is why this rule is there, to overwrite the link base rule.

It's not what we want (unless something has changed that I'm not aware of). We spoke about this a good while ago now.
The name of the thing in the row (i.e. the first cell) shouldn't look/act like a link, the entire row is the clickable thing. The text in the first cell shouldn't be blue/underlined/underline on hover or anything that makes it look like a link. It should be black and slightly bold, as close as possible to this:

I’ve set this declaration again as it’s not related to this PR’s purpose, but I really, really disagree with this entire statement.
It’s not clear why something that’s a link shouldn’t look like any other link in the application by default. In this case, a special rule applies to the color and font-weight as a first cell item in tables, but why the link shouldn’t be underlined on hover is unclear.
The link also very much acts like a link: You can activate it to navigate, you can focus it, you can use the whole array of browser link interactions like MouseWheel+Click, Ctrl+Click, etc.


Replace init delegations in Mirror with calls to Builtin.unreachable() when reflection is disabled

These initializer delegations, which ought to have been Builtin.unreachable() calls, were used to work around an issue that was fixed a while ago: #57622.

Hey, could I get assigned to this issue for my first contribution? Thanks

Sure!


[FEATURE REQUEST] Ability to save and work on a collection/doc in local file system

Insomnia Version: 6.6.2
Operating System: Linux BL7TBH2 4.15.0-46-generic #49~16.04.1-Ubuntu SMP Tue Feb 12 17:45:24 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux (linux mint)

Details

Hello there, I did try to search through the issues, find anything, so hopefully this isn't duplicate.
I wanted to be able to save and open workspace from any location on our disk. The motivation behind this is that I want to commit this data on my source control and others can use it directly, I know about the teams and sharing, but there another issue, I want to be able to run those requests through CLI on CI system.
Actually saving to a directory on my repository, without a huge single json file would be better, it can use the same dir structure as the app, and each request could be separate file, that way version control would track much better.
Thanks

Was about to open an issue for this. I have a different use case. We as a team store everything next to our project, so we would love to have the API JSON file there too. Paw supports storing it to a location, so we were able to save the file to our NextCloud and share it with the team. Maybe you could add this too with a lowered priced team sub.

The local DB files probably won't work very well with VCS but there currently exists an INSOMNIA_DATA_PATH environment variable that can be used to control where local app data is stored. https://github.com/getinsomnia/insomnia/pull/1230
There's also a plugin hook to export the current workspace. It should be possible to write a plugin that exports the current workspace every few seconds to a desired folder.
Saving to the FS with the same folder/request structure as Insomnia uses would require replacing the current local database (NeDB) with a new file-based one so that's likely not an option for now.

Sounds like what I wanted @gschier. Where do I add this env var, just in my e.g. .zshrc?

I'm not sure. You could try it there.
If you need it to be different for different projects than you'd likely be better off setting it when launching it from the command line.
INSOMNIA_DATA_PATH=/my/data/path /path/to/insomnia

I'm not familiar enough with Ubuntu to suggest the best method.

Great, thanks! I am on macOS so it is different, this works: INSOMNIA_DATA_PATH=~/Downloads/Data /Applications/Insomnia.app/Contents/MacOS/Insomnia.
Unfortunately this is as you have mentioned the hole Insomnia instance with all settings not only the workspace. Would be great to have something like this in the future. I'll use it like this for now and check out the plugin hooks method.

So I had a quick look into plugins but none of the hooks seems to work here. There is context.data.export which would work I guess but I can't find a hook that has access to it.
Do you have a reference/link that has more information? I just found https://support.insomnia.rest/article/26-plugins#context.data and https://github.com/getinsomnia/insomnia/blob/develop/packages/insomnia-app/app/plugins/index.js#L174-L219

Ah right, we might need a new plugin hook for that. It might be good to have an init hook that a plugin could use to do initial setup of things like repeating commands.

actually I'd still like this to be opened

I was really hoping for this feature when I discovered insomnia. Postman won't do this because it goes against their business model.
Honestly it would be fine if the db was committed to the repo (though obviously not ideal), as long as there was decent ui for opening and closing a workspace.
Hell, even just a hotkey to "save" which could just be an export, and open from an exported folder would work.
It's bizarre to me that all the api tools seem to want to sync using teams instead of letting people control their data in vc.

Or have an extension, which is a simple json file or whatever db file insomnia uses, and allow us to double click on it, I really need it on VC because I want to have tests there

+1 to this feature request. Being able to save a workspace in a project's repository would be very valuable.

+1 to this feature request

Hey all, Insomnia Designer does support directly syncing your workspace with a git repository. Specifically for @cyberhck, Unit Tests in Designer also sync to your repository, and you can run them in CI using the companion CLI, Inso - inso run test TestSuite --env Staging.
Git Sync, and the ability to create and run Unit Tests, have not been included in the Core application just yet, but it is entirely possible to migrate across to Designer to make use of these features.
Please reach out if there is anything I can help to clarify. 🤗

It will be nice if we can tell the workspace exporter not to minimize the resulting JSON. This feature (the export/import) is ok for me as it is but having it not minimized will help with diffs. It will also allow to implement a VCS backed workspace even if you have to import/export it manually.

+1 for this feature request. @filfreire any possibility of this happening in the near future?

Im supporting this as well, my usecase is just simply to avoid accidentaly wiping all your workspaces.
I usually have all my developing stuff on a separate harddrive, so that way i am able to avoid losing all my data when my OS HDD dies.
Also, all developing stuff is organized on a single hard drive.
So yes, reinstalled windows, and guess what happened, i didnÄt think of Insomnia storing all my stuff in AppData.
So many Tools tend to force Users into their own specific save Folders, it starts becoming a Mess.
And for many tools forcing Users to do this, it becomes easy to overlook one of them.

seriously? You're storing your code in a external hard drive? what if it's lost or damaged? Why aren't you using git and github?

Why do we have to be tied into using Inso’s git tool? It would be much better and easier to just allow for Inso spec files to be saved to a file location that is source controlled through another tool. Why re-invent the wheel?

Why re-invent the wheel?

I guess money?

seriously? You're storing your code in a external hard drive? what if it's lost or damaged? Why aren't you using git and github?

No, not external Hard Drive, a separate HardDrive in my Computer, and there isnt only code on this drive, (local Servers, Tools, local maven repos etc.), basically all the stuff needed for developing. And ofc, the code itself also gets pushed to git.

This problem deserves more attention but I am not sure whether allowing external storage of existing data file solves the issue completely.
For instance, currently Insomnia stores data in (on Windows) %AppData%/Insomnia as various JSON-like DB files, e.g. insomnia.Request.db. Such files are not friendly for version control.

And once again, my workmachine died, and guess what, everything in Insomnia is gone again, because i am not able to "Save as..." and Version Control my Files ...


Update template.json

This seems like a small (but important) contribution, so no Contribution License Agreement is required at this point. We will now review your pull request.
Thanks,
.NET Foundation Pull Request Bot

Thanks, merged.


error="open /host/opt/cni/bin/portmap: text file busy"

Expected Behavior

Calico pod should be able to up and running
Current Behavior

Possible Solution

Steps to Reproduce (for bugs)

Upgrade the system os
root partition switch over, and install the kubernetes with backup etcd on the new root partition
Install kubeadm and run kubeadm init with existing etcd
Calico pod crash loop

Context

master-1:~$ kubectl logs calico-node-pvc2n -n kube-system -c install-cni -p
time="2024-06-20T07:51:02Z" level=info msg="Running as a Kubernetes pod" source="install.go:140"
time="2024-06-20T07:51:02Z" level=info msg="Installed /host/opt/cni/bin/bandwidth"
time="2024-06-20T07:51:02Z" level=info msg="Installed /host/opt/cni/bin/calico"
time="2024-06-20T07:51:02Z" level=info msg="Installed /host/opt/cni/bin/calico-ipam"
time="2024-06-20T07:51:02Z" level=info msg="Installed /host/opt/cni/bin/flannel"
time="2024-06-20T07:51:02Z" level=info msg="Installed /host/opt/cni/bin/host-local"
time="2024-06-20T07:51:02Z" level=info msg="Installed /host/opt/cni/bin/install"
time="2024-06-20T07:51:02Z" level=info msg="Installed /host/opt/cni/bin/loopback"
time="2024-06-20T07:51:02Z" level=error msg="Failed to install /host/opt/cni/bin/portmap" error="open /host/opt/cni/bin/portmap: text file busy"
Your Environment

Calico version 3.24
Orchestrator version (e.g. kubernetes, mesos, rkt): Kubernetes
Operating System and version:Ubuntu 20.04
Link to your project (optional):

error="open /host/opt/cni/bin/portmap: text file busy"

@geotransformer could you run lsof /opt/cni/bin/portmap on your host to check what's preventing that file from being overwritten by the install-cni initContainer?

@geotransformer Closing this issue. Feel free to reopen if you have more information.


Enhance ALS TCP to support periodic reports when the TCP connection is not closed

Title: Enhance ALS TCP to support periodic reports when the TCP connection is not closed.
Description:

Describe the desired behavior, what scenario it enables and how it
would be used.

Currently, the HTTP ALS reports for every HTTP request, and TCP ALS only reports after the connection is closed.
In the observability case of SkyWalking for Istio mesh, we rely on the ALS to generate the topology map with metrics. But when TCP is set through service mesh, envoy TCP ALS would not report TCP ALS until the connection is closed.
Meanwhile, a lot of TCP connections are running in keep-alive mode, so, it lasts days to not close. As a result, we can't observe the relationship and metrics.
We hope this could be enhanced to add a new option to ALS to support TCP ALS reports periodically. And we should consider a new field in the ALS indicating the reporting mode, a) end-of-the-session(same as now), b) intermediate/periodical(report every several seconds even if the connection is still kept)
cc @cpakulski @dio
FYI @wbpcode As you are well known SkyWalking using ALS :)
[optional Relevant Links:]

Any extra documentation required to understand the issue.

SkyWalking ALS analysis doc, https://skywalking.apache.org/docs/main/next/en/setup/envoy/als_setting/

Sound reasonable to me. It would be helpful for the users who using raw TCP connections.

👍


file not found error

The script runs txt2img fine, but then looks for the *u.png upscaled image before any upscaling is performed. I get a file not found error.
FileNotFoundError: [Errno 2] No such file or directory: 'outputs/txt2imghd-samples/samples/00000u.png'
The base file 00000.png exists. I believe I have realesrgan installed correctly (i've tried placing the binary in stable-diffusion/ and stable-diffusion/realesrgan-ncnn-vulkan/) but I get the same error.

Turns out the realesrgan binary was not running b/c of this.
If there is any way to run your script with python libraries instead of vulkan binaries, it would be much appreciated. Your results look better than any other variation on SD that I've seen.

Seconding this, updated to ubuntu 22.04 and tried every which way to resolve but I always get 'No such file or directory'

Maybe check #34
I stumbled upon a similar issue it might be of some help

Same error here. Any update on this issue?


fix(connection): added missed tls option ecdhCurve

Hi, this is related with #242.

This PR not provide a support of ecdhCurve to usage in combine with mongo-native-driver. It should be done separately. This PR allows to establish connection from mongodb-core itself


Demo is broken

The demo has an invalid URL for pouchdb.js
"NetworkError: 404 Not Found - http://mazemap.github.io/Leaflet.TileLayer.PouchDBCached/bower_components/pouchdb/dist/pouchdb.js"
pouchdb.js
ReferenceError: PouchDB is not defined

Additionally, when that link is fixed to point to a working pouchdb.js, the seed demo fails, seemingly because of #9.
TypeError: this._getTileSize is not a function
var tileSize   = this._getTileSize();
L.TileL...ched.js (line 154, col 21)

Both issues should be fixed.
If that is still an issue for you please re-open


[Unity] How do I clear the cached ParseConfig object for development purposes?

For development I want to clear ParseConfig so I can test flow when there is no cached object. How is this done? Is there a file somewhere I can delete?

There's a Parse.settings file that I found under Application Support. Deleting this does the trick, however I'm not sure if the file will also contain other important things?

Yes, Parse.settings will contain things other than config. Most notably, the installation ID, and user ID.
There's no good way to clear just the config from the settings file, without managing to use the private API of the SDK. I'm not too sure about having something like that as a public API, seeing as we have GetAsync() on ParseConfig already which will always ignore cache.

I'm closing this issue due to inactivity. If you have anything else on this topic you'd like to ask, feel free to re-open the issue.


JSQPhotoMediaItem - Image was hidden

Library version(s): 8.0_release
iOS version(s): iOS 9.0
Devices/Simulators affected: iPad Air 2
Complete steps to reproduce the issue:
a) buid & run Demo jsqmessages project (choose iPAD Air 2)
b) "push via storyboar"
c) "Cmd + ->" to rotate the simulator
d) scroll the collection view
e) during scrolling, you will find that image is hidden when the image comes near the top or bottom

Hi,jessesquires ,I found same issue. Had you solve this problem yet?

Hey, @KennyWZhang @Lisajoy512. Does the issue still exist from 7.3.4? I could not replicate it. Can you guys provide more information based on the current version? Thx. 😄

@Lucashuang0802 I came into the issue on 8.0_release.

@KennyWZhang I am so sorry that I have a hard time to replicate the problem by iPad Air 2 with iOS 9.0. Do you think you can post a video for better clarification? That will be extremely helpful. 😄

最新的开发分支也有这个bug

Hello everyone!
I'm sorry to inform the community that I'm officially deprecating this project. 😢 Please read my blog post for details:
http://www.jessesquires.com/blog/officially-deprecating-jsqmessagesviewcontroller/
Thus, I'm closing all issues and pull requests and making the necessary updates to formally deprecate the library. I'm sorry if this is unexpected or disappointing. Please know that this was an extremely difficult decision to make. I'd like to thank everyone here for contributing and making this project so great. It was a fun 4 years. 😊
Thanks for understanding,
— jsq


Trip page and bug fixes

Summary by Sourcery
Refactor cache key management into nested classes for improved organization and readability. Introduce a new trip page with real-time schedule display, including delay information and color-coded on-time status. Fix the display issue for schedules with unknown updates. Update unit tests to align with the new cache key structure.
New Features:

Introduce a new trip page with enhanced real-time schedule display, including delay information and color-coded on-time status.

Bug Fixes:

Fix the issue where schedules with no updates were incorrectly displayed by filtering out schedules with unknown on-time status.

Enhancements:

Refactor cache key management by organizing cache keys into nested classes for better structure and readability.
Improve the trip page by updating the schedule table to include real-time arrival times and delay information.
Enhance the navbar logo display by adjusting its dimensions for better visual consistency.

Tests:

Update unit tests for cache key management to reflect the new nested class structure and ensure correct functionality.

@sourcery-ai review


Is it necessary to select each character in LSTM Training?

Environment
Tesseract Version: tesseract.exe --version
tesseract v5.0.0-alpha.20200223
leptonica-1.78.0
libgif 5.1.4 : libjpeg 8d (libjpeg-turbo 1.5.3) : libpng 1.6.34 : libtiff 4.0.9 : zlib 1.2.11 : libwebp 0.6.1 : libopenjp2 2.3.0
Found AVX2
Found AVX
Found FMA
Found SSE
Found libarchive 3.3.2 zlib/1.2.11 liblzma/5.2.3 bz2lib/1.0.6 liblz4/1.7.5
Found libcurl/7.59.0 OpenSSL/1.0.2o (WinSSL) zlib/1.2.11 WinIDN libssh2/1.7.0 nghttp2/1.31.0
Commit Number: 2020-09-03
Platform: Windows 7
In LSTM training, do I need to adjust the boundary of each box data?

Change to

In LSTM training, do I need to adjust the boundary of each box data?

No. If you have correct bounding box for the line, it is ok.


dts: boards: provide generic dtsi file for 'generic' boards

Some boards belonging to the same board family share common settings.
Being able to include provide a <generic_board>.dtsi file by each board adhering to the generic type would allow to factorize board description.
For instance st nucleo boards share:
-User Button (SW0), LD0 settings
-Console being routed to uart2
-uart2 pinmux and pinmuxing in general
These settings could be defined in a nucleo.dtsi file included by nucleo_xxxx.dts files.
Drawback of this proposal would be loss of readability of dts files as most information would be removed from board dts files.

I think device tree in zephyr could use a lot of work, this being one of them. I'm of the belief that all nucleo boards should have a single maintainer that can manage what you're mentioning. This goes for other 'groups' of boards as well.
Of course, someone else can do the work, but the maintainer should be responsible for keeping things sane.

@psidhu : If you're interested into dt work, maybe you could have a look to following RFC and provide feedback:
https://github.com/zephyrproject-rtos/zephyr/pull/5021

@psidhu : If you're interested into dt work, maybe you could have a look to following RFC and provide feedback:
https://github.com/zephyrproject-rtos/zephyr/pull/5021

No interest in this for now. Closing


Add gitignore so that vim-plug can be managed by vim-plug

If you set up vim-plug to manage vim-plug itself, it will create a doc/tags file, which means the repo it is saved in is dirty. Fix this by adding a gitignore.

I have to mention this:

https://github.com/junegunn/vim-plug/wiki/faq#shouldnt-vim-plug-update-itself-on-plugupdate-like-vundle
https://github.com/junegunn/vim-plug/pull/240#issuecomment-110538613

But, having a .gitignore file wouldn't hurt.


Self alias as struct constructor

Consider the following code
struct Foo { a: i32 }

impl Default for Foo {
    fn default() -> Self {
        Foo { a: 42 }
    }
}

We can't replace the last use of Foo in the impl with Self (you get E0071). But Self is a kind of type alias, or it acts like one anyway, and normally you can construct a struct through a type alias. So we should allow usage of Self like this.
brought up here cc @huonw

Yes. I'd like to do it too.

An alternative that I've been thinking of is allowing _ instead. (fitting in with a possible extension to enum matching)

That's something I intended to implement.

@arielb1 do you mean you don't think this requires an RFC?

Implemented at #37035, thanks @petrochenkov!


New reviewer responder docs

This PR adds docs to the new rOpenSci Reviewers with due date responder.
Here is a preview:

Thank you!!


Properties in JSON output are now always sorted by property name

Coverage remained the same at 32.638% when pulling 8a29bc97acf15ca2477602092bc6200a322179ba on andy/sort-json into 2e17bcabfb6fbf2e99967e19701a0f523c8d3721 on master.


workflows/check-nix-format: add python modules

Description of changes
We accepted nixfmt as a formatter for python-modules in https://github.com/NixOS/nixpkgs/pull/313628.

Things done

Built on platform(s)

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

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

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

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

[ ] Fits CONTRIBUTING.md.

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

Looks like we need to format again.
I will wait until staging-next is merged into master to avoid unnecessary conflicts.

I'm trying to focus on https://github.com/NixOS/nixpkgs/pull/322537 right now, so that we have a generic solution for all parts of Nixpkgs, would be great to support that :D

Btw with https://github.com/NixOS/nixpkgs/pull/326407 merged now, we shouldn't have regressions anymore whenever we once format something :D

Yeah, thanks for your great work!


Nodependencymanager don't work fine

Running the command

don't show any error and don't import dependencies
This is making it impossible to create my noframework

That is a common beginner mistake. That is the intended result.


[Bug] Homepage linking to @Next

On the homepage (https://material-ui.com) under Installation, the suggestion is to install @next. Is this a bug?
I would expect to install the normal stable version of material-ui.

@eluchsinger Oops, and that's not the only place where we don't link the right target.

@oliviertassinari  I was wondering if I could be assigned this issue.  Pretty new to open source contributions.
In my forked local build, I just modified the HomeStep.js file to say "npm install @material-ui/core". I searched other areas where there might be @next, but they seem to be correct as they're talking about migrating. Where else did you see incorrect link targets?

@davidoffyuy Sure, thank you for volunteering! I have found the following extra issues:
diff --git a/docs/src/modules/components/HomeSteps.js b/docs/src/modules/components/HomeSteps.js
index 392a6fe3d..ce1ac957b 100644
--- a/docs/src/modules/components/HomeSteps.js
+++ b/docs/src/modules/components/HomeSteps.js
@@ -122,7 +122,7 @@ function HomeSteps(props) {
           <Link
             variant="subtitle1"
             color="inherit"
-            href="https://github.com/mui-org/material-ui/tree/master/examples/cdn-next"
+            href="https://github.com/mui-org/material-ui/tree/master/examples/cdn"
             gutterBottom
           >
             {t('cdn')}
diff --git a/docs/src/pages/guides/migration-v3/migration-v3.md b/docs/src/pages/guides/migration-v3/migration-v3.md
index 5d0695bf3..b1bdd8b74 100644
--- a/docs/src/pages/guides/migration-v3/migration-v3.md
+++ b/docs/src/pages/guides/migration-v3/migration-v3.md
@@ -36,11 +36,11 @@ You need to update your `package.json` to use the latest version of Material-UI.
 Or run

 ```sh
-npm install @material-ui/core@next
+npm install @material-ui/core

 or

-yarn add @material-ui/core@next
+yarn add @material-ui/core

diff --git a/docs/src/pages/guides/minimizing-bundle-size/minimizing-bundle-size.md b/docs/src/pages/guides/minimizing-bundle-size/minimizing-bundle-size.md
index 33bec0b7a..c45995859 100644
--- a/docs/src/pages/guides/minimizing-bundle-size/minimizing-bundle-size.md
+++ b/docs/src/pages/guides/minimizing-bundle-size/minimizing-bundle-size.md
@@ -102,7 +102,7 @@ Pick one of the following plugins:
 The package published on npm is **transpiled**, with [Babel](https://github.com/babel/babel), to take into account the [supported platforms](/getting-started/supported-platforms/).

 We also publish a second version of the components.
-You can find this version under the [`/es` folder](https://unpkg.com/@material-ui/core@next/es/).
+You can find this version under the [`/es` folder](https://unpkg.com/@material-ui/core/es/).
 All the non-official syntax is transpiled to the [ECMA-262 standard](https://www.ecma-international.org/publications/standards/Ecma-262.htm), nothing more.
 This can be used to make separate bundles targeting different browsers.
 Older browsers will require more JavaScript features to be transpiled,

@oliviertassinari Thank you!
I will submit the pull request later tonight (US Central Time). Should I update the migration pages and minimizing file pages for the various other languages too? Also, I noticed that those pages are still in English. Hopefully that is normal.

If you can handle all the occurrences of the problem at once, it would be perfect 👌 .
The diff I have shared only touches the source of truth of the documentation. The translations are handled in Crowdin, they should be ignored.

@oliviertassinari
Hey. Last question. In migration-v3.md, should I remove the @next reference as well for the Styles component?
If you were previously using @material-ui/styleswith v3 you need to update yourpackage.json` to use the latest version of Material-UI Styles.
"dependencies": {
  "@material-ui/styles": "^4.0.0"
}

Or run
npm install @material-ui/styles@next

or

yarn add @material-ui/styles@next

Yes, well spotted.

@oliviertassinari  Sorry, just realized you said NOT to touch the translations. I misread. Thanks for fixing that and remove the @next after styles.

@davidoffyuy No problem :)


Blizzard's Objective Tracker not reappearing after leaving an M+

Blizzard's Objective Tracker doesn't reappear after exiting mythic keystone dungeons, nor if I checked Demo Mode in options for longer than a few seconds (spam clicking Demo Mode on and off for some reason always allowed them to reappear).
I've managed a temporary fix for myself by adding "ObjectiveTrackerFrame:Show()" in function WarpDeplete:ShowBlizzardObjectiveTracker below SetAlpha on line 214 of the most recent Retail update at the time of writing while you work on a more permanent fix.
Thank you for the addon!

Thanks for the bug report!
Since blizzard changed a lot when it comes to the objective frame in TWW API wise we had to do quite a bit of fiddling with it to get it to stay hidden. It's possible they rolled back some of the changes (which they did with other things too) which probably causes this behavior.
I'll add your suggestion later today and do a release.

Fixed in v3.1.2 🎉


Closes ciena-frost/ember-frost-modal#118

This project uses semver, please check the scope of this pr:

[ ] #none# - documentation fixes and/or test additions
[x] #patch# - backwards-compatible bug fix
[ ] #minor# - adding functionality in a backwards-compatible manner
[ ] #major# - incompatible API change

CHANGELOG

#118 - Bind context to call of this._super.included() in index.js

👍

Changes Unknown when pulling 45ff81a7317dd30901fe5e3b0d43953128e55c46 on notmessenger:Issue118 into ** on ciena-frost:master**.


chore(deps): upgrade dev dependencies

Upgrades project dependencies. See details in workflow run.

Automatically created by projen via the "upgrade-dev-dependencies-1.x" workflow

⚪ Backport skipped
The pull request was not backported as there were no branches to backport to. If this is a mistake, please apply the desired version labels or run the backport tool manually.
Manual backport
To create the backport manually run:
backport --pr 2445

Questions ?
Please refer to the Backport tool documentation


Replace larapack/dd with symfony/var-dumper

larapack/dd no longer provides any value, because now symfony/var-dumper ships dd and dump helpers on its own.
I set the version to "^6.4|^7.0", but lmk if you think another option would be more suitable.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>0
Details

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 92.977%

Totals

Change from base Build<PHONE_NUMBER>6:
0.0%

Covered Lines:
1549

Relevant Lines:
1666

💛  - Coveralls

Cool, thanks!


Fix go.mod

mod name should be full name

Thanks so so much :)
Merged and tagged with the new release.


Cannot run devcontainer-cli outside of its command folder (Possible fix)

I can't say when it started or if it was after a devcontainer update or somethig else but devcontainer cli command stopped working from WSL.
Now i'm testing it on version 0.295.0 but previous versions or pre-release still do not work.
The command devcontainer --help (or other commands) works fine inside Windows (in cmd) but hangs indefinitely in wsl console.
I debugged the script and it works fine in WLS if i run inside devcontainer command folder (if i cd into '/mnt/c/Users//AppData/Roaming/Code/User/globalStorage/ms-vscode-remote.remote-containers/cli-bin/' folder that i got with which devcontainer).
I managed to make it work by adding a cd $basedir on line 47 (two lines before  exit $? in wsl part of the final if)
I will write here my temporary, please do add a proper fix to this problem.
My temporary fix
#!/bin/sh
script_file=$0
while [ -h "$script_file" ]; do
        script_file=$(readlink $script_file)
done
basedir=$(dirname "$(echo "$script_file" | sed -e 's,\\,/,g')")

# This file is placed in a stable (version-independent) location
# and forwards to the currently installed version

# WSL detection borrowed from VS Code
IN_WSL=false
if [ -n "$WSL_DISTRO_NAME" ]; then
        # $WSL_DISTRO_NAME is available since WSL builds 18362, also for WSL2
        IN_WSL=true
else
        WSL_BUILD=$(uname -r | sed -E 's/^[0-9.]+-([0-9]+)-Microsoft.*|.*/\1/')
        if [ -n "$WSL_BUILD" ]; then
                IN_WSL=true
        fi
fi
VSCODE_PATH=$(cat "$basedir/vscode-path")
DEVCONTAINER_CLI_PATH=$script_file
REMOTE_CONTAINERS_PATH=
if [ -f "$basedir/remote-containers-path" ]; then
        REMOTE_CONTAINERS_PATH=$(cat "$basedir/remote-containers-path")
        if [ $IN_WSL = true ]; then
                REMOTE_CONTAINERS_PATH=$(wslpath -u $REMOTE_CONTAINERS_PATH)
        fi
        if [ ! -f "$REMOTE_CONTAINERS_PATH/dev-containers-user-cli/cli.js" ]; then
                REMOTE_CONTAINERS_PATH=
        fi
fi

if [ -z "$REMOTE_CONTAINERS_PATH" ]; then
        echo "Failed to determine Dev Containers path"
        exit 1
fi

if [ $IN_WSL = true ]; then
        export WSLENV="ELECTRON_RUN_AS_NODE/w:DEVCONTAINER_CLI_PATH/p:IN_WSL:WSL_DISTRO_NAME:$WSLENV"
        CLI=$(wslpath -m "$REMOTE_CONTAINERS_PATH/dev-containers-user-cli/cli.js")
        ELECTRON=$(wslpath -u "$VSCODE_PATH")

        # TODO - piping to `cat` is a temporary workaround for the CRLF->LF conversion
        # that occurs when calling Windows Electron binary from WSL
        cd $basedir
        IN_WSL=$IN_WSL DEVCONTAINER_CLI_PATH="$DEVCONTAINER_CLI_PATH" ELECTRON_RUN_AS_NODE=1 "$ELECTRON" --ms-enable-electron-run-as-node "$CLI" "$@" | cat
        exit $?
else
        ELECTRON="$VSCODE_PATH"
        CLI=$REMOTE_CONTAINERS_PATH/dev-containers-user-cli/cli.js
        DEVCONTAINER_CLI_PATH="$DEVCONTAINER_CLI_PATH" ELECTRON_RUN_AS_NODE=1 "$ELECTRON" --ms-enable-electron-run-as-node "$CLI" "$@"
        exit $?
fi

Original file
#!/bin/sh
script_file=$0
while [ -h "$script_file" ]; do
        script_file=$(readlink $script_file)
done
basedir=$(dirname "$(echo "$script_file" | sed -e 's,\\,/,g')")

# This file is placed in a stable (version-independent) location
# and forwards to the currently installed version

# WSL detection borrowed from VS Code
IN_WSL=false
if [ -n "$WSL_DISTRO_NAME" ]; then
        # $WSL_DISTRO_NAME is available since WSL builds 18362, also for WSL2
        IN_WSL=true
else
        WSL_BUILD=$(uname -r | sed -E 's/^[0-9.]+-([0-9]+)-Microsoft.*|.*/\1/')
        if [ -n "$WSL_BUILD" ]; then
                IN_WSL=true
        fi
fi
VSCODE_PATH=$(cat "$basedir/vscode-path")
DEVCONTAINER_CLI_PATH=$script_file
REMOTE_CONTAINERS_PATH=
if [ -f "$basedir/remote-containers-path" ]; then
        REMOTE_CONTAINERS_PATH=$(cat "$basedir/remote-containers-path")
        if [ $IN_WSL = true ]; then
                REMOTE_CONTAINERS_PATH=$(wslpath -u $REMOTE_CONTAINERS_PATH)
        fi
        if [ ! -f "$REMOTE_CONTAINERS_PATH/dev-containers-user-cli/cli.js" ]; then
                REMOTE_CONTAINERS_PATH=
        fi
fi

if [ -z "$REMOTE_CONTAINERS_PATH" ]; then
        echo "Failed to determine Dev Containers path"
        exit 1
fi

if [ $IN_WSL = true ]; then
        export WSLENV="ELECTRON_RUN_AS_NODE/w:DEVCONTAINER_CLI_PATH/p:IN_WSL:WSL_DISTRO_NAME:$WSLENV"
        CLI=$(wslpath -m "$REMOTE_CONTAINERS_PATH/dev-containers-user-cli/cli.js")
        ELECTRON=$(wslpath -u "$VSCODE_PATH")

        # TODO - piping to `cat` is a temporary workaround for the CRLF->LF conversion
        # that occurs when calling Windows Electron binary from WSL
        IN_WSL=$IN_WSL DEVCONTAINER_CLI_PATH="$DEVCONTAINER_CLI_PATH" ELECTRON_RUN_AS_NODE=1 "$ELECTRON" --ms-enable-electron-run-as-node "$CLI" "$@" | cat
        exit $?
else
        ELECTRON="$VSCODE_PATH"
        CLI=$REMOTE_CONTAINERS_PATH/dev-containers-user-cli/cli.js
        DEVCONTAINER_CLI_PATH="$DEVCONTAINER_CLI_PATH" ELECTRON_RUN_AS_NODE=1 "$ELECTRON" --ms-enable-electron-run-as-node "$CLI" "$@"
        exit $?
fi

Sorry my bad, devcontainer --help works with my temporary fix but devcontainer open  does not hang like before and give some output (ex: "[457 ms] @devcontainers/cli 0.42.0. Node.js v16.17.1. win32 10.0.19045 x64.") but it does not open VsCode

Same problem. It worked and one day it stopped... Give output:
"[782 ms] @devcontainers/cli 0.46.0. Node.js v16.17.1. win32 10.0.19045 x64."
but vscode does not open the project.


Running Format/FormatWrite wipes out all existing marks

Running Format/FormatWrite removes all marks on file.
vim-prettier fixed issue this way: https://github.com/prettier/vim-prettier/pull/300
That would be great addition to formatter.nvim if you could lockmarks somehow.
Not 100% sure if that can be done in this plugin, but it's done in vim-prettier already.
(I also tried to use :lockmarks FormatWrite, but that doesn't work.).

Using nvim_buf_set_text instead of nvim_buf_set_lines should fix this #22

Closing as we'll keep this with #22


Fahrgast -> Fahrauftrag -> two zones

A ride has a origin- and a destination-addresse. Each adress is in one zone, but not neccesarily the same zones.
REFERENCE: This is described in the interface specification for accounting in http://sourceforge.net/projects/itixi/files/interfaces/accounting/iTIXI_Specification_Accounting_Interface.docx.
GAP: the form for the ride-order shall display both zones, not just one. This is also true for repeating orders.

This issue is rejected, we will implement with one zone only. Each Zone has a "priority", the highest prority is the zone displayed.


Fixes filters emitted from table viz

The table viz supports emitting filters event when clicking on dimensions (there's a checkbox control to enable that). Since recent changes it was emitting a filter on the mutated html content of the table cell as opposed to this original value.
@ascott
Fixes https://github.com/airbnb/superset/issues/2327 https://github.com/airbnb/superset/issues/2332

Fixes https://github.com/airbnb/superset/issues/2327

LGTM

Table filter has stopped working for recent released version 0.19.1


Add tests

Codecov Report

Merging #2 (480f3a2) into master (8737054) will increase coverage by 20.85%.
The diff coverage is 100.00%.

@@             Coverage Diff             @@
##           master       #2       +/-   ##
===========================================
+ Coverage   27.45%   48.30%   +20.85%
===========================================
  Files           7        7
  Lines         102      118       +16
===========================================
+ Hits           28       57       +29
+ Misses         74       61       -13

Impacted Files
Coverage Δ

src/logic/holiday.py
100.00% <100.00%> (+52.00%)
:arrow_up:

test/logic/test_holiday_engine.py
100.00% <100.00%> (ø)

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