UI design Bug in Mentors & Squads page

Bug Report
What is the current behavior?
The drop-down to select the number of rows to be displayed in the Mentors & Squads page is not fully visible but only partially visible which causes the users to double check their selection.
What is the expected behavior?
The drop-down to select the number of rows to be displayed in the Mentors & Squads page must be fully visible.
What steps did you take to get this behavior?
Steps to replicate :

Login to https://operationcode.org
In the Top Navigation Bar, click 'Mentors'
In the Mentors page, the drop-down to choose the number of rows to be displayed is not fully visible but only visible partially.
The same behavior can also be found in the row drop-down present in Squads page.
This causes users to double check on the number of rows they've selected.

By inspecting the dom with dev tools, I found that when I comment out height: 20px on line 18 of login.css, the desired results are displayed. I'll go ahead and fix this up.

I polished up the profile page with some new images and text.
I also added a script to package.json to easily start the backend. Prior to finding this I didn't know there was a script in bin to start a backend. This is important for debugging things locally that require the user to be signed in.
Run npm run backend and in another terminal run npm start to have a working dev environment.
TODO: Add these instructions to the README.


Post-MVP: Set-up embedded data so information is passed  between qualtrics/platform

Currently, we don't connect qualtrics survey to the student.
The easiest way to do so is to pass some identifier (user_id) to qualtrics in the iframe URL. Then, we can do a merge on user_id in post.

Addressed in #105


HDDS-3378. OzoneManager group init failed because of incorrect snapshot directory location.

What changes were proposed in this pull request?
Make ratis.snapshot.dir do not depend on ozone.om.ratis.storage.dir, so that ozone.om.ratis.storage.dir does not have any other directories.
And also if ratis.storage.dir is not defined, the default we fall back to ozone.metadata.dirs, then we will have ratis.storage.dir as ozone.metadata.dirs + "/ratis".
And for older clusters, the directory will be moved to new ratis.snapshot.dir, if snapshot directory exists in ozone.om.ratis.storage.dir.
What is the link to the Apache JIRA
https://issues.apache.org/jira/browse/HDDS-3378
Please replace this section with the link to the Apache JIRA)
How was this patch tested?
Existing tests.

Thank You @hanishakoneru for the offline discussion.
I have posted the PR, as discussed to solve the issue for fresh instllation and also for upgrades.

Thanks Bharat for updating the patch.
We can use the Const OM_RATIS_SNAPSHOT_DIR in OzoneManagerRatisServer#getOMRatisSnapshotDirectory() also.
I am +1 with that fixed. Thanks.

Thank You @hanishakoneru for the review. Fixed it.

Thank You @hanishakoneru for the review.


Cleaned up some parameter parsing & dynamic dependencies

Would be great to get your input @tommyengstrom – in particular I'm not sure about the __main__ stuff you were doing in load_task

Didn't you just start to remove the optparse stuff in #612? It's confusing that there still are traces of it.

Yeah wasn't super consistent... but this one should be merged first. For the optparse PR I want to give it a few days to see if anyone has any issues. I probably have to rebase the optparse PR later.

I can remove the first commit from this PR if you think it makes more sense? The two commits have very little with each other to do

Added even more tests now. Fairly sure this PR works. Definitely simplifies a lot of code with dynamic dependencies.
In case you haven't noticed, I'm trying to refactor various stuff with luigi.interface and luigi.parameter. It's all entangled so I have to start in other places.

Alright LGTM but porbably give @tommyengstrom a day or so to review. :)

Not sure what's up with Travis, but it looks like it passed

Also does dynamic dependencies not run on other workers? When I tried to run it with multiple workers it doesn't look like it's parallelizing the tasks. I remember talking about this but forgot what we concluded – is there a fundamental reason why you can't put a dynamic dependency on another worker?


deprecated code ->setParameters(

PHP Version
8.3.3
Shopware Version
<IP_ADDRESS>.RC.3
Expected behaviour
use Doctrine\Common\Collections\ArrayCollection;
use Doctrine\ORM\Query\Parameter;
$builder->setParameters(new ArrayCollection([
new Parameter('techName', InvoiceRenderer::TYPE),
new Parameter('orderId', Uuid::fromHexToBytes($orderId))
]));
Actual behaviour
$builder->setParameters([
'techName' => InvoiceRenderer::TYPE,
'orderId' => Uuid::fromHexToBytes($orderId),
]);
How to reproduce
in
src/Core/Checkout/Document/Service/ReferenceInvoiceLoader.php and in others files where is ->setParameters(
this deprecated code will result in error in version 4.0
https://www.doctrine-project.org/projects/doctrine-orm/en/3.0/reference/query-builder.html
https://www.doctrine-project.org/projects/doctrine-orm/en/latest/reference/query-builder.html

Hi,
thx for your issue we will put it into our new evaluation process and have a look at it again.

[public] Automated response: This issue is linked to the internal issue https://shopware.atlassian.net/browse/NEXT-37967.
[created from NEXT-37967, comment 489092]

Hey, thanks for raising the issue, but actually we are not affected by that deprecation as Doctrine has to QueryBuilder classes.
One from the ORM project, that is the one you are referring to, and one from the DBAL project, that is the one that we are using.
And in DBAL Query Builder the method signature will stay the same, see https://www.doctrine-project.org/projects/doctrine-dbal/en/4.1/reference/query-builder.html.


Decode GPS data into location name

This PR decodes GPS data (latitude and longitude) into location name, i.e. street, city, country etc.
It works as follows:

table 'photos' gets a new column 'location'
during upload, latitude and longitude is decoded into street, city, country etc. using OpenStreetmap (can be deactivated in settings, default is not enabled)
OpenStreetmap only allows 1 request per second, hence we limit request rate is limited to 1 request per second
Requests are cached for speedup and reduce call to OpenStreetmap (Default: using filesystem as cache, can be deactivated or set to in-memory; the latter only works by invoking decoding through separate command - see below - since memory is cleared after every photo)
Decoded location is shown below map, but can be deactivated for all or just for public access (needs separate PR for frontend)
Location field is added to columns used by search function

Using php artisan lychee:decode_GPS_locations, a decoding of existing GPS data can be triggered. In-memory caching is supported in this case.

Can you explain your choices for the cache system? I don't really understand that part.
Laravel comes already with a cache system: https://laravel.com/docs/7.x/cache and to me it feels you are adding another system on top of it (or am I missing something?).
Also why do you need uploads/cache to be publically available? 🤔

Can you explain your choices for the cache system? I don't really understand that part.
Laravel comes already with a cache system: https://laravel.com/docs/7.x/cache and to me it feels you are adding another system on top of it (or am I missing something?).

Happy to use the provided cache, but I didn't found a way to do it. We have the following requirements:

We need a http adapter to work
It should support caching to reduce impact of 3.
It must support request throttling to limit calls to Openstreetmap to 1 call per second

Please correct me on the following thoughts
re 1. Larval does not bring a native http client, but recommends guzzlehttp/guzzle (https://laravel.com/docs/7.x/http-client). I used php-http/guzzle6-adapter as a http adapter for the afore mentioned.
re 2. the Geodecoder already offers a provider for caching (https://github.com/geocoder-php/cache-provider), but it need an actual caching implementation below. The caching implementation needs to be a PSR-6 compatible cache driver - an this is where I didn't find a solution.
There is madewithlove/illuminate-psr-cache-bridge, but working only with Larval 5. I did not succeed including the larval cache - happy to get pointed into the right direction.
re 3. We now need some throttling, which can be done by using spatie/guzzle-rate-limiter-middleware (used) or hamburgscleanest/guzzle-advanced-throttle.

Also why do you need uploads/cache to be publically available? 🤔

No, it does not need to be public (sorry for the perhaps stupid question - how do I make it non-public?)

@tmp-hallenser I cleaned the cache part. :) It should now directly use the one provided by Laravel. :)
Can you double check I did not break anything?
By the way, why do we need that?
"nyholm/psr7": "^1.2",

@tmp-hallenser I cleaned the cache part. :) It should now directly use the one provided by Laravel. :)
Can you double check I did not break anything?
By the way, why do we need that?
"nyholm/psr7": "^1.2",

Good point - not needed since we're using guzzle.

First of all - many thanks for your push!
Unfortunately, it does not work for me.
I needed to install php-http/message to fix the first issue. Now, I'm stuck with the following error:
   TypeError

  Argument 2 passed to Geocoder\Provider\Cache\ProviderCache::__construct() must implement interface Psr\SimpleCache\CacheInterface, instance of Symfony\Component\Cache\Adapter\Psr16Adapter given, called in /html/Lychee-Laravel/app/Metadata/Geodecoder.php on line 45

  at vendor/geocoder-php/cache-provider/ProviderCache.php:48
    44|      * @param Provider       $realProvider
    45|      * @param CacheInterface $cache
    46|      * @param int            $lifetime
    47|      */
  > 48|     final public function __construct(Provider $realProvider, CacheInterface $cache, int $lifetime = null)
    49|     {
    50|         $this->realProvider = $realProvider;
    51|         $this->cache = $cache;
    52|         $this->lifetime = $lifetime;

  1   app/Metadata/Geodecoder.php:45
      Geocoder\Provider\Cache\ProviderCache::__construct(Object(Geocoder\Provider\Nominatim\Nominatim), Object(Symfony\Component\Cache\Adapter\Psr16Adapter))

  2   app/Console/Commands/decode_GPS_locations.php:69
      App\Metadata\Geodecoder::getGeocoderProvider()

Can you give me a picture for which has data to decode so I can try to fix that ? :)

There are two ways:

Just upload an image (w/o gps data) and insert some random (more or less, e.g. 2.294481 and 48.858370) coordinates into the database, e.g. using phpmyadmin, and call php artisan lychee:decode_GPS_locations afterwards.

Upload images with gps data, e.g. use the ones provided in #451 (they both should contain GPS data); Or use php artisan lychee:sync /folder/to/sync

Thanks for you effort!

@tmp-hallenser and now ? :smiley:

I needed to install php-http/message to fix the first issue. Now, I'm stuck with the following error:

weird.

Codecov Report

Merging #457 into master will decrease coverage by 0.59%.
The diff coverage is 13.72%.

php-http/message

still need it


K8SSAND-1742 ⁃ k8ssandra-operator should define all bound ports in pod spec

What is missing?
The k8ssandra-operator defined in the helm chart binds three TCP ports but only one of those ports (the webhook) is defined in the deployment's pod spec.
Why do we need it?
In some environments any workload providing a validating or mutating webhook must be run on the host network (for example, EKS with an alternative CNI). Without all ports being defined, kubernetes cannot deconflict other host network workloads that attempt to bind the same port, leading to failures.
Environment

K8ssandra Operator version:
1.2.0

**Anything else we need to know?**:

┆Issue is synchronized with this Jira Task by Unito
┆friendlyId: K8SSAND-1742
┆priority: Medium

Oops, I see the helm chart is in k8ssandra/k8ssandra, closing this and opening an issue there instead.

For reference https://github.com/k8ssandra/k8ssandra/issues/1488

I'm reopening this because we need to the changes to the kustomize manifests which live in this repo.

@petercable Would you mind creating similar issues for cass-operator?


Fix phala.sh

@skysummerice the fixes seem to solve the bug and work. Issues are mainly in the IF statements.

@skysummerice pls double check the merge commit for correctness

@skysummerice pls double check the merge commit for correctness

After merging the commits something. The Ubuntu 21.1 support and Kernel additions should still be in there but are removed.


send an Image through Signal Messenger

can you add the function to send a image?
https://api.callmebot.com/signal/send.php?phone=[phone_number]&apikey=[your_apikey]&image=[url_image]
[url_image]: URL of the image to send. The URL has to be urlencoded. You can use this online converter to encode the message. (i.e. %20 for space).
Example:
https://api.callmebot.com/signal/send.php?phone=+49123123123&apikey=567890&image=https://www.callmebot.com/wp-content/uploads/pinguin.jpg

@Dettenrocks Ooooh, that looks cool. I can add it the next weeks

Hi!
The only way I see at the moment is to upload an image via ftp, check that the file was successfully uploaded and send the command.
It is to check, if you can send the image/file from a protected (i.e. with .htaccess) location on the webserver.

Yes, that's a solution.
I'm trying at the last months the libsignal library in a new adapter (iobroker.signal-messenger). I can request a code via SMS or call... but I can not verify with the library :(
Can you program ;) ?
I need help :)
If I can't get this working, then I try an other way in this adapter B)

Sorry, I am not a coder.
But I searched for verificationCode on the project Whisperfish, because I know, that they use libsignal. Maybe this helps?!?

Ooooh, thank you. I take a look there.


Paused deployments are incorrectly identified in Prefect 3.0

Describe the bug a clear and concise description of what the bug is.
The exporter incorrectly identifies paused deployments in Prefect 3.0. The exporter fetches all deployments by issuing the following query against the server:
curl --location 'http://<prefect-server-url:port>/api/deployments/filter' \
--header 'Content-Type: application/json' \
--header 'Accept: application/json' \
--data '{
    "offset": <some-offset>,
    "limit": <some-limit>
}'

It then creates the prefect_info_deployment metric and adds an is_schedule_active label by taking the value directly from the output dictionary in the response - this can be seen here. The Grafana dashboard uses this to filter for paused deployments via the following Prometheus query:
prefect_info_deployment{is_schedule_active="False"} == 1.
In Prefect 3.0, the is_schedule_active key is no longer present in the response, so the label value is always null. Instead, deployments now have a boolean paused key that should be used as a fallback, e.g
str(deployment.get("is_schedule_active", str(deployment.get("paused", "null"))))

Application name
prometheus-prefect-exporter
Application version
1.5.0
What you expected to happen?
To preserve backwards compatibility, the is_schedule_active label should be set to the value of the paused key in the response, instead of the is_schedule_active key, which does not exist anymore.
How to reproduce it?
Run a local Prefect server along with Prometheus, the exporter and Grafana. Create a deployment without a schedule. Import the dashboard. prefect_info_deployment{is_schedule_active="False"} == 1 reports 0 even though there is a paused deployment
More info
No response

Thanks for reaching out @ubiquitousbyte! Appreciate the thorough issue details - are you interested in contributing the fix? If not, no worries - we'll try to get to this within the next 2 weeks.

As much as I would love to get involved in this, I won't be able to do so in the next two weeks. Thank you for picking it up.

With https://github.com/PrefectHQ/prometheus-prefect-exporter/pull/47, I'll close this for now. I'll cut a release for this as well. Reach out if you run into any problems with it 👍🏼

Available in 1.6.0:
$ helm search repo prefect/prometheus-prefect-exporter -l | head -n 2
NAME                                    CHART VERSION           APP VERSION     DESCRIPTION
prefect/prometheus-prefect-exporter     2024.10.17192636        1.6.0           A Helm chart to deploy Prometheus Prefect Exporter


ZSH_AUTOSUGGEST_HIGHLIGHT_STYLE can't work

fg=8 can't work

try fg=black,bold

Going to go ahead and close this because there's basically no description of the problem. Feel free to re-open with more information.


How to represent the space for the circle

I am confused about the space for the circle. In the file circle.py, the notes, in the function _addition_theorem, say that
the input param X and X2 are one-dimensional. However, we should represent the circle with two parameters. So, the input should be two-dimensional.
How to understand this case? Or, we just denote the space for the circle with the following codes:
ProductDiscreteSpectrumSpace(Circle(), Circle(), num_eigen=num)

Check this code https://github.com/Fangwq/Research-Progress/blob/main/20221201/circle_kernel.py. Are these codes right or not? Thank you very much !

Hello @Fangwq. The elements of Circle are supposed to be parameterized by their angle. X should consist of angles X[i] from $0$ to $2 \pi$ that correspond to the $(\cos(X[i]), \sin(X[i]))$ point on the circle $\{ \mathbf{v} \in \mathbb{R}^2 : |\mathbf{v}| = 1 \}$.

Please tell me if this helps.

Thank you for your clarification. It works. The codes I have written are wrong.


Include receipt PDF in "thank you" email

User story
As a financial contributor, I can easily download receipts from the email sent when I make a contribution (the Thank you for your $_ contribution to ____ email). We used to have a link in there that pointed directly to the invoices server, but this has been deprecated because it didn't meet our security standard.
Best solution for this problem
Two possible solutions:

When sending the email from API, fetch the invoice from the invoices server and attach it to the email. If the invoice generation fails, we should still send the email without it.
Create a page to show a single transaction on /{collective}/transactions/${transactionId} that has a download invoice button on it. Link this page form the email.

My personal preference goes to (1) because it's a simpler UX and it doesn't seems more difficult to implement. But implementing both solutions could actually be nice.
Metrics

Number of support requests for people not finding their receipts

We also have to think how this would play with https://github.com/opencollective/opencollective/issues/2864

This would greatly simplify our workflow with OpenCollective and reduce admin overhead so we can sponsor more projects and individuals.
Currently we have to login to OpenCollective every month and manually download the billing receipt PDFs for each individual project and for the right months and then email and important that into our accounting system. Instead of just getting a single overall receipt PDF every month that can be auto imported into our systems.

@sbinlondon thanks for implementing this, will make our lives easier! cc @arirawr

Very nice! Is there anything we as sponsors need to configure to receive the PDFs?

@Piedone Nothing to do on your side, the PDFs should just land with your next "Thank you" emails :slightly_smiling_face: Please note that we can't guarantee that this will happen with the next batch on 1st November as we still need to do some load testing to make sure it will run smoothly for everyone. We're also going to tackle https://github.com/opencollective/opencollective/issues/3626 to make sure your accountant receive them too!


Separate columns in model.summary() to improve readability

This patch makes sure that there is always at least one space between the columns produced by model.summary(). This helps to separate the output shape from the parameter number. For example, this layer looks like it has a surprisingly large number of parameters:
____________________________________________________________________________________________________
Layer (type)                     Output Shape          Param #     Connected to
====================================================================================================
...
____________________________________________________________________________________________________
convolution3d_1 (Convolution3D)  (None, 100, 13, 13, 132800        input_patch[0][0]
____________________________________________________________________________________________________
...

This gets especially confusing if you're looking at a table with many rows. Adding a space between the two columns makes it more readable:
____________________________________________________________________________________________________
Layer (type)                     Output Shape          Param #     Connected to
====================================================================================================
...
____________________________________________________________________________________________________
convolution3d_1 (Convolution3D)  (None, 100, 13, 13, 1 2800        input_patch[0][0]
____________________________________________________________________________________________________
...

LGTM


Update util functions to depend on properties instead of mojo

Almost all app service util functions depend on the whole mojo, which makes the mojo passing around kinds of functions. This kind of dependency is heavy and makes it hard for scaling and reusing.
Update util functions depend on independently properties to reduce the mojo dependencies all around the code and to prepare for decoupling handlers from mojo.

Fixed in PR #372


fix: add potentially missing selinux labels back to nvidia-modeset

Apparently this stops sddm from crashing a lot when logging in with a VRR monitor conntected (gdm untested as everyone affected were on kde)
PRing into testing since i have not gotten proper verification that this fully fixes the issue.
can be tracked here: https://www.answeroverflow.com/m/1285186896137425016

Open to it, I look forward to the day this can be removed

Believe this was confirmed to not work? Let me know if I'm mistaken


How to tell the extension about dependencies that do not come from Maven or Gradle?

This is one part a question and one part, I assume, a feature request.
How can I teach the extension about dependencies that I am going to put on the classpath/modulepath which do not come from a pom.xml or build.gradle definition?
My situation is that I am generating files which have the path I intend to use and supplying those to java/javac via an argfile.
Example:
jresolve --output-file deps<EMAIL_ADDRESS>
deps:
/Users/emccue/.jresolve/cache/https/repo1.maven.org/maven2/com/fasterxml/jackson/core/jackson-annotations/2.15.2/jackson-annotations-2.15.2.jar:/Users/emccue/.jresolve/cache/https/repo1.maven.org/maven2/com/fasterxml/jackson/core/jackson-core/2.15.2/jackson-core-2.15.2.jar:/Users/emccue/.jresolve/cache/https/repo1.maven.org/maven2/com/fasterxml/jackson/core/jackson-databind/2.15.2/jackson-databind-2.15.2.jar

java --class-path @deps src/Main.java

Though I can easily imagine someone else wanting to just use a jar they have on their filesystem or something similar.

You need to add these to the gradle file.

@robaho There is no gradle file

The oracle plugin requires using gradle or maven.

@robaho That's...thats what this issue is about...

Then I would rephrase your question as a feature request to have the extension support non managed projects.

Though I can easily imagine someone else wanting to just use a jar they have on their filesystem or something similar.

This. I'd like to put a project-local lib/ directory on the module path, for example. Which should result in

javac --module-path lib at compile-time and
java --module-path lib run-time respectively.

@Achal1607 Okay, I tried editing a settings.json and the run configurations options. No dice.

FWIW, I would suggest to use absolute paths.

@lahodaj That makes a run configuration pretty darn non-portable. I guess fine if its generated by a tool, but not if i'm doing it manually.
It does work though with an absolute path. Go to definition doesn't and it doesn't work reading args from an argfile, but i guess its a start?
so that works with a module path and a class path

Regarding "Go to definition doesn't" - could you please elaborate? I have some sample source using multi-source launcher. I can navigate (Ctrl-click or F12) across various source files that refer to each other. When I try to use GSon, I can navigate to the GSon class - if there are no sources, then it will "decompile". If there is a source bundle (I use gson-2.10.1.jar as the binary, and next to it, I have the source bundle: gson-2.10.1-sources.jar), then it navigates to the sources from the bundle. (Plus other things work better when the source bundle is available.)

if there are no sources, then it will "decompile". If there is a source bundle (I use gson-2.10.1.jar as the binary, and next to it, I have the source bundle: gson-2.10.1-sources.jar)

I mean it does not do the first behavior. I haven't downloaded a sources jar to check the 2nd behavior but i can in a bit.

If you want to work with the same clay I am, the tool I made to resolve dependencies is here
https://github.com/bowbahdoe/jresolve-cli/releases/download/v2024.05.10/jresolve.jar
You can run it with java -jar jresolve.jar and give it a series of "package urls" like you find on the sonatype site
So example incantations:
$ java -jar jresolve.jar<EMAIL_ADDRESS>/Users/emccue/.jresolve/cache/https/repo1.maven.org/maven2/com/google/code/gson/gson/2.10.1/gson-2.10.1.jar

$ java -jar jresolve.jar --print-tree<EMAIL_ADDRESS>com.google.code.gson/gson 2.10.1

$ java -jar jresolve.jar --output-directory libs<EMAIL_ADDRESS>$ ls libs
gson-2.10.1.jar

$ java -jar jresolve.jar --print-tree<EMAIL_ADDRESS>com.fasterxml.jackson.core/jackson-databind 2.17.1
  . com.fasterxml.jackson.core/jackson-annotations 2.17.1
  . com.fasterxml.jackson.core/jackson-core 2.17.1

$ java -jar jresolve.jar --print-tree<EMAIL_ADDRESS><EMAIL_ADDRESS>com.fasterxml.jackson.core/jackson-databind 2.17.1
  . com.fasterxml.jackson.core/jackson-annotations 2.17.1
  . com.fasterxml.jackson.core/jackson-core 2.17.1

(if package urls in a file)
$ java -jar jresolve.jar --print-tree @libs.txt
<EMAIL_ADDRESS>com.fasterxml.jackson.core/jackson-databind 2.17.1
  . com.fasterxml.jackson.core/jackson-annotations 2.17.1
  . com.fasterxml.jackson.core/jackson-core 2.17.1

You get the idea.
My goal, which overlaps with the "I just downloaded a jar" folks, is to make that "work" here

Hm, I've done something like this:
java -jar jresolve-cli-2024.05.10-uber.jar<EMAIL_ADDRESS>.../.jresolve/cache/https/repo1.maven.org/maven2/com/google/code/gson/gson/2.10.1/gson-2.10.1.jar

In the VS Code, I've specified VM Options as:
--module-path .../.jresolve/cache/https/repo1.maven.org/maven2/com/google/code/gson/gson/2.10.1/gson-2.10.1.jar --add-modules com.google.gson

and have a source like:
package test;

import com.google.gson.Gson;

public class TryGson {

    Gson g = new Gson();

}

Pressing F12 on any instance of Gson takes me to the decompiled source. If I copy the gson-2.10.1-sources.jar to the .../.jresolve/cache/https/repo1.maven.org/maven2/com/google/code/gson/gson/2.10.1/ directory, then F12 jumps to the sources.
Could you please provide reliable reproducible steps? Thanks!

@lahodaj Okay, so it worked with GSON on my end too. My guess is something in the decompiler silently fails with some byte code level 8 < x <= 21. Thats workable.
Now the question is how can tools/users who just download deps directly best inform vscode of those paths. Relative paths and argfiles both not working is a bummer.

I am looking into supporting relative paths(*) and argfiles. It is a bit tricky, but probably doable. Note that there are limits on sharing the configuration - the path separator is different between Windows and Linux/Mac, so sharing the configuration across platforms is not very easy.
(*) there is the obvious question "relative to what". There are many possible answers, but I think I'll try to go with "the enclosing workspace folder", if I can.
(As a side note, I would say the lack of source (or at least javadoc) bundles is likely to have a detrimental ongoing effect on the usability of the setup. But, it is difficult for the extension to download the source bundle in this setup, where there is no project description/metadata, and where all there is is a collection of jars with indeterminable origin.)

As a side note, I would say the lack of source (or at least javadoc) bundles is likely to have a detrimental ongoing effect on the usability of the setup.

Well, for the use cases we are talking about
* People manually downloading jars could manually download sources+javadoc jars too. They probably won't, but they could.

* Tools could be told to dump source + javadoc jars into the same / related directories. You got it working with gson

* If you really wanted to be clever, jars often have a pom.xml in them that you could use to infer where to get that stuff.

So i'd say thats a workable tradeoff.

I have no doubt it is a workable tradeoff. What I am trying to say is that if you are serious about developing using this mode, someone needs to work that out. And, given there is (per my understanding) no "standardized" configuration in tools like jresolve, but rather just free-form command line invocations (mostly analogous to wget), the tool doing the download is probably the only place which can ensure the source bundle is available with sufficient reliability.
(Looking for pom.xml is nice, as long as the only place one wants to support is the one well-known repository. Note this is different for Maven - the extension can read the Maven configuration, as that is declarative and defined, and can do what is needed based on that.)

Merged https://github.com/oracle/javavscode/pull/173. So closing this issue. If you have any other queries or issues please feel free to open an issue.


Refactor GenomeLocParser and GenomeLoc

Currently, in order to create a GenomeLoc, you need to create a GenomeLocParser, which requires either a reference fasta or a sequence dictionary for validation. Sometimes, however, you don't want this level of validation (perhaps you have already checked that the interval is within bounds, making the extra validation wasteful -- this happens in a few places in the new engine). There should be a way to instantiate a GenomeLoc directly, without the need to pass around a GenomeLocParser.

@droazen can you clarify what still needs to be done for this ticket? unassign when clarified.

Closing -- this was addressed via SimpleInterval


Problem with Casting on Sql Server Date Time Field

When I run the following query I get the incorrect result in sqlectron.
SELECT CAST(GETDATE() AS DATE)
Here's an image of the results of the query from sqlectron and ssms:

Looking at this, it seems to fix this will require a larger rewrite of the underlying sqlectron-core library. Right now, the logic for rendering a value is relatively simple:
https://github.com/sqlectron/sqlectron-gui/blob/593b75056bde23f20c18f88575a06a29fd50c7b4/src/renderer/utils/convert.js#L21-L47
The simplicity in the rendering here is a result that the underlying core does not surface meaningful information on what type a given column is and so that information must first be written into the core, which is not clear how much effort that will be.


Implement storage lib of vineyard backend

What do these changes do?
implement vineyard backend storage lib.

[x] vineyard backend
[x] unit test case

this pr should be merged after #1904
Related issue number
#1905

#1904 has been merged, there's some small changes in storage lib API.

I think vineyard storage can be tested in vineyard platform testing only.

Overall LGTM. Just one concern is should be impose another storage (maybe VINEYARD) and use it for vineyard?
Or MEMORY is fine enough here?
I would like to know comments from @qinxuye.

You mean storage level? I think MEMORY is enough.

Could we add vineyard (maybe a fixed version) to the requirements.txt of mars?

As for requirements, it's about necessary libraries, how about add a pip install mars[vineyard] support?

You mean storage level? I think MEMORY is enough.

Then this pull request looks to me.

As for requirements, it's about necessary libraries, how about add a pip install mars[vineyard] support?

Add a vvineyard profile is also ok. I'm wondering if we could put vineyard at the same dependency level with plasma.

You mean storage level? I think MEMORY is enough.

Then this pull request looks to me.

As for requirements, it's about necessary libraries, how about add a pip install mars[vineyard] support?

Add a vvineyard profile is also ok. I'm wondering if we could put vineyard at the same dependency level with plasma.

Yeah, but for now, Mars does not rely on plasma either, but in the near future, multiprocessing will be the default option, till then, we can see if vineyard could be the default library.

@wjsi Could you please help to check why vineyard storage is not covered?

@wjsi Could you please help to check why vineyard storage is not covered?

I'm not very sure, I think it is becase there are three pytest call and the result for the first one is overrided by the later run. Maybe we need to combine like https://github.com/mars-project/mars/blob/master/.github/workflows/run-tests.sh#L16

@wjsi Could you please help to check why vineyard storage is not covered?

I'm not very sure, I think it is becase there are three pytest call and the result for the first one is overrided by the later run. Maybe we need to combine like https://github.com/mars-project/mars/blob/master/.github/workflows/run-tests.sh#L16

Perhaps, is it possible to combine them into one? @acezen

@wjsi Could you please help to check why vineyard storage is not covered?

I'm not very sure, I think it is becase there are three pytest call and the result for the first one is overrided by the later run. Maybe we need to combine like https://github.com/mars-project/mars/blob/master/.github/workflows/run-tests.sh#L16

Perhaps, is it possible to combine them into one? @acezen

ok

Hi, @acezen, @wjsi is on it about combing coverage


Remove (soon) from yo package

Thanks so much for your contributions. :)

Its really my pleasure. Thanks for letting me help :)

@vinkla @GrahamCampbell Thank you very much!


Update NYSE holidays: fix New Year's Day observed

Proposed change

Update NYSE holidays (fix New Year's Day observed and some special holidays).
Closes #1583.
Type of change

[ ] New country/market holidays support (thank you!)
[ ] Supported country/market holidays update (calendar discrepancy fix, localization)
[ ] Existing code/documentation/test/process quality improvement (best practice, cleanup, refactoring, optimization)
[ ] Dependency update (version deprecation/upgrade)
[x] Bugfix (non-breaking change which fixes an issue)
[ ] Breaking change (a code change causing existing functionality to break)
[ ] New feature (new python-holidays functionality in general)

Checklist

[x] I've followed the contributing guidelines
[x] I've run make pre-commit, it didn't generate any changes
[x] I've run make test, all tests passed locally

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

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 100.0%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
10286

Relevant Lines:
10286

💛  - Coveralls


Update the old arm architecture with the recent fix

Summary
arch/arm: align arm vector related code with armv7-a
arch/arm: Add PSR_ prefix to the mode state like armv7-a
Impact
No real function change
Testing
Pass CI

@xiaoxiang781216
Could you rebase the branch to fix the conflicts?

@masayuki2009 done.


Fix for CVE-2021-20293

Note: httpClientTimeoutsWork unit tests failing, WIP

Closing ticket out, please visit MONIT for more information


Carousel one by one?

I need to do this in my carousel.
https://codepen.io/mephysto/pen/ZYVKRY
Any help or directions would be appreciated.

Closing this as this project is no longer being maintained, per the README and the ISSUE_TEMPLATE.


[BACKLOG-35286] Gwt Java 11 compatibility changes

:x: Build failed in 1m 6s
Build command:
mvn clean verify -B -e -Daudit -amd -pl plugins/pur/core

:exclamation: No tests found!
Errors:
Filtered log (click to expand)

script returned exit code 1

:information_source:  This is an automatic message

:white_check_mark: Build finished in 8m 16s
Build command:
mvn clean verify -B -e -Daudit -amd -pl plugins/pur/core

:ok_hand: All tests passed!
Tests run: 280, Failures: 0, Skipped: 0    Test Results

:information_source:  This is an automatic message


expose important metrics via k8s events

Summary
We need to expose important metrics which are used in triaging via k8s events.
Eg: Controller error stating ISB creation failed
We also need to revisit all the events emitted. Need to make sure they are tagged in better object (eg pipeline vs vertex)

we need to see what is missing, we have few added in #856

Related to #972

Working with @mshakira to determine what new events need to be implemented and which currently may need to be adjusted.
 Re-open or create a new one if anything else is needed.


Delete unnecessary old branches

There are a few old branches (https://github.com/git/git-scm.com/branches/all) in this repository. Besides the contributors-page branch, the other seem to be old work that we don't want to keep or that's so outdated that the changes would need to be reviewed completely.
@peff @jnavila @sxlijin what do you think? just a minor cleaning :)

Anything that's been merged should obviously be deleted.
Anything with a closed PR should probably be deleted. GitHub holds onto the tips via refs/pull, so it can always be restored if necessary. A few of the branches in this state could not be deleted because "they have open pull requests". But this can be tricky if the PR is spammy, since it doesn't appear in the PR list, then. For contributors-page, I closed #385 (which I found via git for-each-ref --contains). But I can't seem to find any PR that refers to the github branch. I think I'll just delete it manually via Git.
For anything else, deleting the branch would lose any reference to the work in question. So it may be worth pinging the author before deleting.

There's only 1 branch remaining: git-for-windows, which tries to introduce a new git for windows page
Can you tell us if this work should be kept or deleted @blackant ?

Thanks @pedrorijo91, that can be deleted. (@blackant used to be my username 😺 )


glab: install shell completions

Description of changes
Added glab shell completions.
Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] 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
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
22.05 Release Notes (or backporting 21.11 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
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

There is however an older PR (#166580) that takes into account cross-compilation, but does not use installShellFiles. I think it would be nice to have a PR which does both.

I attempted to combine the two PRs. Please check if it is okay now.

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/prs-ready-for-review/3032/809

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/prs-already-reviewed/2617/487


How to set colors programatically

Hi, I would like to know, what values should I pass to method e.g setCurrentDayBackgroundColor(). I see it should be int, but I can't find what exactly. Thanks in advance

https://developer.android.com/reference/android/graphics/Color.html
You can use for example Color.parseColor("#fff"); // white. It can take a hexidecimal color. Color provides many ways to get color values.

Oh, thanks. I haven't thought about this :D


Dropbox: datastore API is deprecated - find alternative

Datastore API is being deprecated.
Support ends October 2015, endopoints will be removed in April 2016.
rclone uses the datastore API for storing metadata about the files (md5sum and modification date).
Need an alternative...
Ideally store the metadata on the objects, but this doesn't appear to be possible with the new API.

Maybe Dropbox can help.

The Datastore API is somewhat unique in that it, unlike the Core API, deals with non-file data, so we’re working directly with individual developers to help them migrate to an alternative. Existing apps built using the Datastore API will continue to work in the meantime.

Looking through their comments, it looks like they direct developers to contact them directly: https://www.dropbox.com/developers/contact.

@austinginder I think I might take a different approach and stop storing metadata at all for dropbox.  This would downgrade the syncs somewhat so the MD5SUMs wouldn't be checked and every sync would be a --size-only sync.  However it would have the advantage of making dropbox a lot more reliable and faster.
I'll try contacting them anyway and see what happens!

That would be great if possible. I currently only check the size when doing a sync to Dropbox.

I had a chat with Dropbox support and they come up with a couple of new ideas - Parse or Firebase
I think the options are

Store the data in a file manually (eg in one big json file, or lots of little ones)
Downgrade dropbox to a size only backend
Use a 3rd party database to store the metadata eg Parse or Firebase

All of these options could potentially work with other cloud providers (eg Microsoft) which don't provide MD5SUMs or Dates.
I think Option 1) would work, but I know from bitter experience that keeping that file up to date will be very complicated and it will be prone to corruption. Using one metadata file per file would work better, but that has the disadvantage that they would all appear in the dropbox interface.
Option 2) fits in well with rclone's philosophy of small and lightweight.  It would be fairly straight forward to implement and lower the barrier to entry for new cloud storage providers.
Option 3) would work too, but I'm not a big fan of using two different providers to solve the metadata problem.
My preferred choice would be option 2)

Likewise option 2 would be great. I'm using rclone to incrementally backup to my Dropbox Business account. This works out amazingly as my Dropbox account as unlimited revisions. Thanks for the quick response and for doing the research.

Would option 1 work if you saved the json files locally but set rclone to ignore them for syncing (as .dropbox files are now - or is that a function of the API rather than rclone)?

@darren12345 yes that would work but it wouldn't be ideal as there is stuff that can get out of sync (the local .json file and the remote cloud storage)

But my point was that the json files would only be saved locally and not synced at all - referring to your point:

Using one metadata file per file would work better, but that has the disadvantage that they would all appear in the dropbox interface.

The json file can store the same data the datastore currently does (md5 hash etc) to be used for comparison if required. One obvious downside is that the json file(s) would have to be generated on each local instance before syncing could commence.
Or have you considered something like SQLite to store metadata?

Hi Nick. So does this mean that rclone only compares sizes by default when syncing with Dropbox now or is it also checking mtime (not sure if that's possible with the Dropbox API)? Thanks for your work on this.

Yes it only compares sizes (not mtime) which is less than perfect, but for nearly all use cases it is good enough.  I hope that doesn't inconvenience you too much.
I may at some point add a DB or json file or something like that, but I decided not to for the moment.

No, that's peachy. Like you say, it will be good enough in 99% of cases I should think.


DC Milo 0.2438

Release Notes: DC Milo 0.2438

Codecov Report
Attention: Patch coverage is 27.78448% with 1796 lines in your changes missing coverage. Please review.

Project coverage is 70.45%. Comparing base (9a26748) to head (5b63036).
Report is 154 commits behind head on main.

Files with missing lines
Patch %
Lines

acrobat/blocks/nonprofit/nonprofit.js
0.00%
805 Missing :warning:

acrobat/blocks/nonprofit/constants.js
0.00%
427 Missing :warning:

acrobat/blocks/nonprofit/select.js
0.00%
256 Missing :warning:

acrobat/blocks/unity/unity.js
0.00%
113 Missing :warning:

acrobat/blocks/verb-widget/verb-widget.js
41.08%
109 Missing :warning:

acrobat/scripts/reactiveStore.js
0.00%
50 Missing :warning:

acrobat/scripts/alloy/verb-widget.js
3.22%
30 Missing :warning:

acrobat/blocks/rnr/rnr.js
99.14%
4 Missing :warning:

acrobat/scripts/utils.js
90.00%
2 Missing :warning:

Additional details and impacted files
@@             Coverage Diff             @@
##             main     #810       +/-   ##
===========================================
- Coverage   93.81%   70.45%   -23.36%
===========================================
  Files          40       47        +7
  Lines        4944     7109     +2165
===========================================
+ Hits         4638     5009      +371
- Misses        306     2100     +1794

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Make changes to site after feedback is gathered

After content has been given a pass make issues to ensure content change are completed by engineering/content/design/etc

some of this text is repetitive. consider updating:

Adjusted copy under the 'Who We Are' and 'What We Do' bubbles in the blue ribbon for a little variety in language. Let me know of any questions.
Also made one tweak to the copy in the R2D2 'Ongoing Technical Advice' section under 'Our Work' based on direct feedback from Brett L. via Slack.


Add support for the SkSVG canvas backend

We're currently using NGraphics for exporting vector images for the various platforms. We'd really like to be able to output an SVG file and Skia looks promising. We'd give SkiaSharp a try if this feature ever makes it.
https://skia.org/user/api/canvas#sksvg

I think that full SVG support should be added to th SkiaSharp library, including SVG Animations ( https://svgwg.org/specs/animations/ ).
If SkiaSharp is going to be the engine behind Xamarin.Forms (and the whole WPF crossplatform), then SVG support is as necessary as PNG or JPG support.
You cannot treat SVG as a external option, something that you can delegate to a secondary framework made by amateurs.
It is the main vector image format around. 90% of all vector graphic resources (icons, graphic elements, UI skins, animations, backgrounds, etc...) are made in SVG, exported in SVG and distributed in SVG. Artists and designers are used to save their vector images in SVG format. App design tools like Sketch or AdobeXD export the app design storyboards as SVG files, ready to be imported in the app. Popular mobile apps like Viber use SVG SMIL animations extensively for ALL their awesome animated stickers. With the advent of the modern high density display jungle, everybody is dropping bitmap assets for SVG assets.
Intro animations for the "Onboarding" phase of new apps are now often made with SVG SMIL animated clips. The new popular SVG morphing algorithms are now used by many design tools to create animated visual transitions easily between UI states, just by choosing a start and an end SVG image. From those two inbetweens are automatically generated at 60 fps, for a smooth and modern UI.
No vector framework is going to succeed with developers without SVG support, no more than a bitmap imaging framework without PNG or JPG support. Other crossplatform 2D libraries like NGraphics for example already support SVG.
This is very urgent, otherwise Xamarin.Forms is going to be more and more detached from the needs of modern app development.

I don't think this is going to be done at all anytime soon.
Here is my code (it binds all that there is to bind):

Managed Branch: https://github.com/mono/SkiaSharp/tree/xamarin-svg-canvas
Native Branch: https://github.com/mono/skia/tree/xamarin-svg-canvas

I did a check to see what's what and this is what I found:

SkSVGCanvas is a thing, but it is very limited:

basically just paths, shapes, text and images
linear gradients are supported, but no others
effects and filters are not

Unless I get a go-ahead from @bholmes or @migueldeicaza, I will not be merging these changes into the main branch anytime soon. We have to wait for SVG to become something in Skia.

A SKSvgCanvas can be created on the fly, requiring only a bounds and an SKXmlWriter:
// create a writeable stream
// SKFileWStream skStream = ...
// SKDynamicMemoryWStream skStream = ...
System.IO.Stream stream = ...
SkiaSharp.SKManagedWStream skStream = new SKManagedWStream(stream);

// now create the xml writer and the canvas
using (var writer = new SKXmlStreamWriter(skStream))
using (var canvas = SKSvgCanvas.Create(SKRect.Create(0, 0, (int)Bounds.Width, (int)Bounds.Height), writer))
{
    // draw as normal since "canvas" is just a "SKCanvas"
}

// you may have to:
//skStream.Flush();

Awesome job on getting those bindings.

basically just paths, shapes, text and images

That's basically all I need for what we are using NGraphics for. Hopefully we'll get a chance to try it out soon. Thanks!

@zone117x Make sure you use the svg betas as they haven't been merged into master/stable yet

I have run some tests using Version 1.53.2 (SVG Beta 1) and generally works great!
Found one bug when drawing paths with multiple figure.
The following path is not drawn correctly in Svg:
M210,150L330,150L330,210L210,210z M225,165L315,165L315,195L225,195z

Below is the Svg output using SkiaSharp:
<?xml version="1.0" encoding="utf-8" ?>
<svg xmlns="http://www.w3.org/2000/svg" xmlns:xlink="http://www.w3.org/1999/xlink" width="840" height="600">
	<rect fill="rgb(255,255,255)" stroke="none" width="840" height="600"/>
	<path fill="rgb(0,0,0)" fill-opacity="0.3137255" stroke="none" d="M210 150L330 150L330 210L210 210L210 150ZM225 165L315 165L315 195L225 195L225 165Z"/>
	<path fill="none" stroke="rgb(0,0,0)" stroke-width="2" stroke-linecap="round" stroke-miterlimit="4" d="M210 150L330 150L330 210L210 210L210 150ZM225 165L315 165L315 195L225 195L225 165Z"/>
</svg>

In above Svg you can see that there are two paths, one with fill and one with stroke only, but they do not combine as they should.
Code:
var presenter = new ContainerPresenter(); // MY DRAWING  CODE

using (SKFileWStream stream = new SKFileWStream(path))
using (var writer = new SKXmlStreamWriter(stream))
using (var canvas = SKSvgCanvas.Create(SKRect.Create(0, 0, (int)container.Width, (int)container.Height), writer))
{
    presenter.Render(canvas, renderer, container, 0, 0); // MY DRAWING CODE
}

I use same drawing code for creating Pdf's using SkiaSharp and output is correct.
Pdf:

Svg:

I found the issue, the svg canvas backend does not set proper fill-rule, this is fixed version as it should be (the only difference is  fill-rule="evenodd"):
<?xml version="1.0" encoding="utf-8" ?>
<svg xmlns="http://www.w3.org/2000/svg" xmlns:xlink="http://www.w3.org/1999/xlink" width="840" height="600">
    <rect fill="rgb(255,255,255)" stroke="none" width="840" height="600"/>
    <path fill-rule="evenodd" fill="rgb(0,0,0)" fill-opacity="0.3137255" stroke="none" d="M210 150L330 150L330 210L210 210L210 150ZM225 165L315 165L315 195L225 195L225 165Z"/>
    <path fill="none" stroke="rgb(0,0,0)" stroke-width="2" stroke-linecap="round" stroke-miterlimit="4" d="M210 150L330 150L330 210L210 210L210 150ZM225 165L315 165L315 195L225 195L225 165Z"/>
</svg>

This is related to the fact that to use different color for fill and stroke using SKPaint you have to issue two separate draw calls:
private void DrawPathInternal(SKCanvas canvas, SKPaint brush, SKPaint pen, bool isStroked, bool isFilled, SKPath path)
{
    if (isFilled)
    {
        canvas.DrawPath(path, brush);
    }

    if (isStroked)
    {
        canvas.DrawPath(path, pen);
    }
}

The fix should probably be implemented in Skia?

@wieslawsoltes I think @zone117x is correct. If you look here: https://github.com/google/skia/blob/589a39eb81d25fa7af95e7366db17ac7a70a7d03/src/svg/SkSVGDevice.cpp#L348 they only do a few things.
I would suggest opening an issue on http://bugs.skia.org and maybe creating a fiddle (https://fiddle.skia.org/) showing what you are trying to do.
https://bugs.chromium.org/p/skia/issues/detail?id=5596

@mattleibow Opened issue https://bugs.chromium.org/p/skia/issues/detail?id=5712

That is awesome!
Just repo-ing the C# code for "testing" purposes:
using (var stream = new SKFileWStream("test.svg"))
using (var streamWriter = new SKXmlStreamWriter(stream))
using (var canvas = SKSvgCanvas.Create(new SKRect(0, 0, 100, 100), streamWriter))
{
	var p1 = new SKPaint();
	p1.Color = SKColors.Red;
	p1.IsAntialias = true;
	p1.Style = SKPaintStyle.Fill;
	p1.StrokeWidth = 1;

	var p2 = new SKPaint();
	p2.Color = SKColors.Black;
	p2.IsAntialias = true;
	p2.Style = SKPaintStyle.Stroke;
	p2.StrokeWidth = 2;

	var path = new SKPath();
	path.FillType = SKPathFillType.EvenOdd;
	path.MoveTo(10.0f, 10.0f);
	path.LineTo(90.0f, 10.0f);
	path.LineTo(90.0f, 90.0f);
	path.LineTo(10.0f, 90.0f);
	path.Close();
	path.MoveTo(20.0f, 20.0f);
	path.LineTo(80.0f, 20.0f);
	path.LineTo(80.0f, 80.0f);
	path.LineTo(20.0f, 80.0f);
	path.Close();

	canvas.DrawPath(path, p1);
	canvas.DrawPath(path, p2);
}

@mattleibow https://bugs.chromium.org/p/skia/issues/detail?id=5712#c2

Opacity - Not working:
<svg width='50' height='50' viewBox='0 0 50 50'><rect width='50' height='50' fill='#000' opacity='0.5' /></svg>

Fill-Opacty - Working
<svg width='50' height='50' viewBox='0 0 50 50'><rect width='50' height='50' fill='#000' fill-opacity='0.5' /></svg>

More here: https://github.com/luberda-molinet/FFImageLoading/issues/436

@daniel-luberda I have added support for "opacity" in this commit: https://github.com/mono/SkiaSharp/commit/cfe6afbf8aece4a2d4b835b727073cc50a0015da

I have merged the svg branch into the master, but please note that this area (svg canvas) is still very much in beta on the native side. Thus, it is a beta state in SkiaSharp.

@mattleibow Thanks for committing to this and getting it done. Great work!


Exhibit your image processing skills!

Improve the image segmentation algorithm to segment the leaves more efficiently. May work on segmenting leaves from a cluster or detection of more colors in leaf (say, purple, yellow, etc.) other than just green, that report healthy signs.

It involves improving the image processing steps in main.py to segment the leaves from the image more efficiently. Refer to the README.md to know the steps currently been followed to do the same.
If you still have some questions DM me on gitter.

okk

I want a working code of this project plz help me.

I want to work on this issue

I want to work on this issue i have basic knowledge of opencv and image processing

What are your expectations in this course? For clustering, we need a training data but we actually don't have a training data. If we collect leaves images from internet than those images involve background as well. Do you recommend me to collect images from internet as training and test data and then filter background and test with different clustering techniques to check for accuracy. Can you tell me why we are going for hue value?

What are your expectations in this? For clustering, we need a training data but we actually don't have a training data. If we collect leaves images from internet than those images involve background as well. Do you recommend me to collect images from internet as training and test data and then filter background and test with different clustering techniques to check for accuracy. Can you tell me why we are going for hue value?


[SPARK-40879][CONNECT] Support Join UsingColumns in proto

What changes were proposed in this pull request?

I was working on refactoring Connect proto tests from Catalyst DSL to DataFrame API, and identified that Join in Connect does not support UsingColumns. This is a gap between the Connect proto and DataFrame API. This also blocks the refactoring work because without UsingColumns, there is no compatible DataFrame Join API that we can covert existing tests to.
This PR adds the support for Join's UsingColumns.
Why are the changes needed?

Improve API coverage.
Unblock testing refactoring.

Does this PR introduce any user-facing change?

No.
How was this patch tested?

UT

R: @cloud-fan

Can one of the admins verify this patch?

thanks, merging to master!


[1363] Add seeds for access request table

Without this it's hard to test the support app being able to action them
while connected to local development backend app.

 Allow mc-support to grant access requests through mc-be


Linked example page doesn't show any examples

Contrary to the claim, the article linked to in the Remarks section doesn't contain any usage examples for IndentedStringBuilder.

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

ID: 73ab18e3-2d97-eec3-3809-8db14b638bd2
Version Independent ID: 1c2ee81c-4481-3b72-8205-f687ca41560b
Content: IndentedStringBuilder Class (Microsoft.EntityFrameworkCore.Infrastructure)
Content Source: dotnet/xml/Microsoft.EntityFrameworkCore.Infrastructure/IndentedStringBuilder.xml
Product: ef-core-api
GitHub Login: @dotnet-bot
Microsoft Alias: avickers

@InteXX the Remarks section links to the general section in the EF docs that's related to the API, this doesn't guarantee we've written examples specifically for this API.
IndentedStringBuilder is an API to be used by database providers in order to e.g. generate SQL; it's not intended for use by end users. The number of such APIs exposed by EF is huge, and there's no way we can exhaustively document and provide samples for them all (nor would that be useful).

Very well; thank you for your quick reply.


localhost problems

Hey,
I want to overwrite my localhost/ dir but just the localhost/webseiten/ and one dir under cause after my pages coming with content that i dont want to ow.. so how can i fix this, I tried regex at all...?
Regards and thanks
Muuvmuuv

I am not sure i understand what you mean, can you be a bit more specific?

mhm i mean, that i can say that my style will just shown in this " localhost/webseiten/* " for example: "localhost/Webseiten/work"
"localhost/webseiten/"
But not In the dir under "work" - "localhost/webseiten/work/project1" !
My Filetree looks like this:
localhost/webseiten
localhost/webseiten/work
localhost/webseiten/mysite
localhost/webseiten/work/project1

Did anyone has an answer?

@ankit maybe you now know an answer for me. Im still in trouble and really want to use a stil for this space of sites.

Lmao, I wrote that? Omg haha, I do not know what I was trying back then but I had to be drunk. 🤣

hehe :)


CI: test with unoptimized docker image that the deployment works

This PR is for testing use and should be optimized!

Codecov Report

:exclamation: No coverage uploaded for pull request base (HH-220-fix-docker-config@9071caa). Click here to learn what that means.
The diff coverage is n/a.

@@                     Coverage Diff                     @@
##             HH-220-fix-docker-config      #20   +/-   ##
===========================================================
  Coverage                            ?   66.78%
===========================================================
  Files                               ?      120
  Lines                               ?     2827
  Branches                            ?      574
===========================================================
  Hits                                ?     1888
  Misses                              ?      934
  Partials                            ?        5

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Logout on change of password

When a user changes their own password, their session is killed off to force a new login

@carlsixsmith-moj , made changes as suggested, please review.


Login failing issue

I checked that instagram-scraper ver 1.5.32. worked very well on 4.29. 06:00 UTC, but starts to fail logging in when I checked on 4.29. 18:00 UTC. Login method might be changed.

Instagram changed method to authenticate.. Try this in login function:
self.session.get(BASE_URL)
self.session.headers.update({'X-CSRFToken': self.session.cookies.get_dict()['csrftoken']})

@dmoreno1994 I still get the following error when modifying the app.py script
Traceback (most recent call last):
  File "/Users/robin/anaconda3/envs/analytics/bin/instagram-scraper", line 11, in <module>
    sys.exit(main())
  File "/Users/robin/anaconda3/envs/analytics/lib/python3.6/site-packages/instagram_scraper/app.py", line 1136, in main
    scraper.login()
  File "/Users/robin/anaconda3/envs/analytics/lib/python3.6/site-packages/instagram_scraper/app.py", line 215, in login
    self.session.headers.update({'X-CSRFToken': self.session.cookies.get_dict()['csrftoken']})
KeyError: 'csrftoken'


[BUG]

问题描述
JSON反序列化时，当json字符串内容过长时，出现 illegal fieldName input 122, offset 4773, character z, line 9, column 8, fastjson-version 2.0.52。
环境信息
请填写以下信息：

OS信息：  [e.g.：Microsoft Windows [版本 10.0.19045.4780]]
JDK信息： [e.g.：Openjdk 22.0.1]
版本信息：[e.g.：Fastjson2 2.0.52]

重现步骤
如何操作可以重现该问题：
可以复制以下代码测试。我将数据放在多个json在线解析网站测试，结果都是无误的，不存在格式之类的问题
public class Test {
public static void main(String[] args) {
String a = "[{"location":{"x": "-0.768332","y": "0.05704","z": "0.0"},"index": 2},{"location":{"x": "-0.811024","y": "0.008477","z": "0.0"},"index": 4},{"location":{"x": "-0.830571","y": "-0.013758","z": "0.0"},"index": 6},{"location":{"x": "-0.824935","y": "-0.007346","z": "0.0"},"index": 8},{"location":{"x": "-0.795674","y": "0.025938","z": "0.0"},"index": 10},{"location":{"x": "-0.738112","y": "0.091415","z": "0.0"},"index": 12},{"location":{"x": "-0.659925","y": "0.180356","z": "0.0"},"index": 14},{"location":{"x": "-0.554274","y": "0.300534","z": "0.0"},"inde\n" +
"x": 16},{"location":{"x": "-0.369033","y": "0.780241","z": "0.0"},"index": 19},{"location":{"x": "0.144912","y": "0.797848","z": "0.0"},"index": 21},{"location":{"x": "0.233746","y": "1.012275","z": "0.0"},"index": 23},{"location":{"x": "0.086928","y": "0.580593","z": "0.0"},"index": 24},{"location":{"x": "0.035156","y": "0.6286","z": "0.0"},"index": 25},{"location":{"x": "-0.054684","y": "0.681895","z": "0.0"},"index": 26},{"location":{"x": "-0.160876","y": "0.742971","z": "0.0"},"index": 27},{"location":{"x": "-0.281857","y": "0.860878","z": "0.0"},"index": 28},{"location":{"x": "-0.388065",\n" +
""y": "0.991797","z": "0.0"},"index": 29},{"location":{"x": "-0.469992","y": "1.100613","z": "0.0"},"index": 30},{"location":{"x": "-0.532405","y": "1.19655","z": "0.0"},"index": 31},{"location":{"x": "-0.5597","y": "1.255629","z": "0.0"},"index": 32},{"location":{"x": "-0.562715","y": "1.292603","z": "0.0"},"index": 33},{"location":{"x": "-0.576496","y": "1.297537","z": "0.0"},"index": 34},{"location":{"x": "-0.634179","y": "1.288641","z": "0.0"},"index": 35},{"location":{"x": "-0.683457","y": "1.324805","z": "0.0"},"index": 36},{"location":{"x": "-0.694109","y": "1.407581","z": "0.0"},"index"\n" +
": 37},{"location":{"x": "-0.675899","y": "1.535812","z": "0.0"},"index": 38},{"location":{"x": "-0.628826","y": "1.709494","z": "0.0"},"index": 39},{"location":{"x": "-0.552893","y": "1.928629","z": "0.0"},"index": 40},{"location":{"x": "-0.52288","y": "2.274498","z": "0.0"},"index": 41},{"location":{"x": "-0.53556","y": "2.56024","z": "0.0"},"index": 42},{"location":{"x": "-0.503766","y": "2.745405","z": "0.0"},"index": 43},{"location":{"x": "-0.427497","y": "2.829996","z": "0.0"},"index": 44},{"location":{"x": "-0.306759","y": "2.814007","z": "0.0"},"index": 45},{"location":{"x": "-0.111591"\n" +
","y": "2.66642","z": "0.0"},"index": 46},{"location":{"x": "0.142525","y": "2.450066","z": "0.0"},"index": 47},{"location":{"x": "0.5204","y": "2.272087","z": "0.0"},"index": 48},{"location":{"x": "0.829484","y": "2.162884","z": "0.0"},"index": 49},{"location":{"x": "0.967172","y": "2.094109","z": "0.0"},"index": 50},{"location":{"x": "0.904161","y": "2.067463","z": "0.0"},"index": 51},{"location":{"x": "0.656207","y": "2.089619","z": "0.0"},"index": 52},{"location":{"x": "0.236857","y": "2.152184","z": "0.0"},"index": 53},{"location":{"x": "-0.297248","y": "2.217514","z": "0.0"},"index": 54},\n" +
"{"location":{"x": "-0.766695","y": "2.236366","z": "0.0"},"index": 55},{"location":{"x": "-1.140412","y": "2.255207","z": "0.0"},"index": 56},{"location":{"x": "-1.328735","y": "2.28587","z": "0.0"},"index": 57},{"location":{"x": "-1.393793","y": "2.329117","z": "0.0"},"index": 58},{"location":{"x": "-1.335579","y": "2.384964","z": "0.0"},"index": 59},{"location":{"x": "-1.154099","y": "2.453384","z": "0.0"},"index": 60},{"location":{"x": "-1.071793","y": "2.522034","z": "0.0"},"index": 61},{"location":{"x": "-0.956085","y": "2.725189","z": "0.0"},"index": 62},{"location":{"x": "-0.494337","y"\n" +
": "3.161255","z": "0.0"},"index": 63},{"location":{"x": "0.281134","y": "3.855354","z": "0.0"},"index": 64},{"location":{"x": "1.00738","y": "4.531952","z": "0.0"},"index": 65},{"location":{"x": "1.319183","y": "4.759972","z": "0.0"},"index": 66},{"location":{"x": "1.214396","y": "4.55452","z": "0.0"},"index": 67},{"location":{"x": "1.211981","y": "4.475883","z": "0.0"},"index": 68},{"location":{"x": "1.169556","y": "4.318573","z": "0.0"},"index": 69},{"location":{"x": "0.866768","y": "3.817009","z": "0.0"},"index": 70},{"location":{"x": "0.339333","y": "3.01899","z": "0.0"},"index": 71},{"loc\n" +
"ation":{"x": "0.038861","y": "2.521973","z": "0.0"},"index": 72},{"location":{"x": "-0.056871","y": "2.259514","z": "0.0"},"index": 73},{"location":{"x": "-0.050248","y": "2.134033","z": "0.0"},"index": 74},{"location":{"x": "-0.100498","y": "1.950005","z": "0.0"},"index": 75},{"location":{"x": "-0.115635","y": "1.70536","z": "0.0"},"index": 76},{"location":{"x": "-0.063417","y": "1.480621","z": "0.0"},"index": 77},{"location":{"x": "-0.011171","y": "1.235985","z": "0.0"},"index": 78},{"location":{"x": "0.010653","y": "1.038094","z": "0.0"},"index": 79},{"location":{"x": "0.009596","y": "0.920\n" +
"036","z": "0.0"},"index": 80},{"location":{"x": "0.005243","y": "0.922554","z": "0.0"},"index": 81},{"location":{"x": "0.01108","y": "0.954651","z": "0.0"},"index": 82},{"location":{"x": "0.012066","y": "0.935028","z": "0.0"},"index": 83},{"location":{"x": "0.008198","y": "0.863815","z": "0.0"},"index": 84},{"location":{"x": "-0.000525","y": "0.740875","z": "0.0"},"index": 85},{"location":{"x": "-0.016732","y": "0.533966","z": "0.0"},"index": 86},{"location":{"x": "-0.078013","y": "0.528214","z": "0.0"},"index": 88},{"location":{"x": "-0.545872","y": "-0.089172","z": "0.0"},"index": 97},{"loca\n" +
"tion":{"x": "-0.251307","y": "-0.541229","z": "0.0"},"index": 106},{"location":{"x": "0.090087","y": "-0.563965","z": "0.0"},"index": 111},{"location":{"x": "0.07787","y": "-0.523376","z": "0.0"},"index": 115},{"location":{"x": "0.060936","y": "-0.640396","z": "0.0"},"index": 118},{"location":{"x": "0.219096","y": "0.560257","z": "0.0"},"index": 127},{"location":{"x": "0.091289","y": "0.619583","z": "0.0"},"index": 130},{"location":{"x": "0.092523","y": "0.774017","z": "0.0"},"index": 133},{"location":{"x": "0.058727","y": "0.665146","z": "0.0"},"index": 135},{"location":{"x": "0.053468","y": \n" +
""0.843719","z": "0.0"},"index": 137},{"location":{"x": "0.046363","y": "1.051315","z": "0.0"},"index": 139},{"location":{"x": "0.019871","y": "0.615494","z": "0.0"},"index": 140},{"location":{"x": "0.016915","y": "0.692123","z": "0.0"},"index": 141},{"location":{"x": "0.014041","y": "0.764374","z": "0.0"},"index": 142},{"location":{"x": "0.010906","y": "0.841629","z": "0.0"},"index": 143},{"location":{"x": "0.001625","y": "0.940063","z": "0.0"},"index": 144},{"location":{"x": "-0.017946","y": "1.069077","z": "0.0"},"index": 145},{"location":{"x": "-0.039027","y": "1.210983","z": "0.0"},"index"\n" +
": 146},{"location":{"x": "-0.053061","y": "1.320877","z": "0.0"},"index": 147},{"location":{"x": "-0.063165","y": "1.417953","z": "0.0"},"index": 148},{"location":{"x": "-0.069345","y": "1.50235","z": "0.0"},"index": 149},{"location":{"x": "-0.071602","y": "1.573959","z": "0.0"},"index": 150},{"location":{"x": "-0.069133","y": "1.639404","z": "0.0"},"index": 151},{"location":{"x": "-0.062554","y": "1.682266","z": "0.0"},"index": 152},{"location":{"x": "-0.052053","y": "1.712463","z": "0.0"},"index": 153},{"location":{"x": "-0.037623","y": "1.729767","z": "0.0"},"index": 154},{"location":{"x": \n" +
""-0.019269","y": "1.734421","z": "0.0"},"index": 155},{"location":{"x": "0.006757","y": "1.723206","z": "0.0"},"index": 156},{"location":{"x": "0.033942","y": "1.699142","z": "0.0"},"index": 157},{"location":{"x": "0.065054","y": "1.662323","z": "0.0"},"index": 158},{"location":{"x": "0.100094","y": "1.612671","z": "0.0"},"index": 159},{"location":{"x": "0.139055","y": "1.550232","z": "0.0"},"index": 160},{"location":{"x": "0.194776","y": "1.520416","z": "0.0"},"index": 161},{"location":{"x": "0.246774","y": "1.535873","z": "0.0"},"index": 162},{"location":{"x": "0.293115","y": "1.545883","z":\n" +
" "0.0"},"index": 163},{"location":{"x": "0.333815","y": "1.550537","z": "0.0"},"index": 164},{"location":{"x": "0.368876","y": "1.54985","z": "0.0"},"index": 165},{"location":{"x": "0.401687","y": "1.542145","z": "0.0"},"index": 166},{"location":{"x": "0.424045","y": "1.529404","z": "0.0"},"index": 167}]";
System.out.println(JSON.parseArray(a, ActorSkillMontageRootMotionBo.class));
}
}
@Data
public class ActorSkillMontageRootMotionBo {
private int index;

private Location location;

}
/**

坐标
*/
@Data
@AllArgsConstructor
@NoArgsConstructor
public class Location implements Location3D {
private double x;
private double y;
private double z;

}
期待的正确结果
反序列化应该是正常的，当只有10多个元素时，序列化正常，我尝试只序列化报错内容指定的那一段数据是正常的，删除报错得那段数据测试后依旧报错，目前得出得结果发现只要数据过多时必报错。

补充一下，接口Locaion3D的代码片段
public interface Location3D {
double getX();

double getY();

double getZ();

}

我发现是数据内容里面有换行符\r\n导致了这个问题，去掉后没有了


python SyntaxError: invalid syntax when building plugin

I'm running Mac OS 10.13.6
bazel version output:
bazel version
WARNING: failed to raise resource limit 8 to 32768: Invalid argument
Build label: 0.18.1-homebrew
Build target: bazel-out/darwin-opt/bin/src/main/java/com/google/devtools/build/lib/bazel/BazelServer_deploy.jar
Build time: Fri Nov 2 11:16:56 2018<PHONE_NUMBER>)
Build timestamp:<PHONE_NUMBER>
Build timestamp as int:<PHONE_NUMBER>

I ran command: bazel build //ijwb:ijwb_bazel_zip --define=ij_product=intellij-latest
And got output:
WARNING: failed to raise resource limit 8 to 32768: Invalid argument
INFO: Analysed target //ijwb:ijwb_bazel_zip (58 packages loaded).
INFO: Found 1 target...
ERROR: /Users/drichelson/workspace/intellij/ijwb/BUILD:27:1: Executing genrule //ijwb:merged_plugin_xml_common failed (Exit 1)
  File "/private/var/tmp/_bazel_drichelson/216b61342e9110c2ce77e6949d8864b1/sandbox/darwin-sandbox/279/execroot/intellij_with_bazel/bazel-out/host/bin/build_defs/merge_xml.runfiles/intellij_with_bazel/build_defs/merge_xml.py", line 58
    print dom.toxml()
            ^
SyntaxError: invalid syntax
Target //ijwb:ijwb_bazel_zip failed to build
Use --verbose_failures to see the command lines of failed build steps.
INFO: Elapsed time: 35.753s, Critical Path: 30.29s
INFO: 285 processes: 276 darwin-sandbox, 9 worker.
FAILED: Build did NOT complete successfully

According to the mentioned PRs, this issue should be fixed. If there's still an issue when using Python 3, please open another bug report.


Union of same types

https://github.com/microsoft/vscode-languageserver-node/blob/d2def542712faf49c86c9fe3c8f4ce31729cd8f7/server/src/common/semanticTokens.ts#L23
In the above line of code (and some other places), there is a SemanticTokensDeltaPartialResult | SemanticTokensDeltaPartialResult, which makes no sense to me. I guess this ought to be SemanticTokensDeltaPartialResult | SemanticTokensPartialResult?

See https://github.com/microsoft/vscode-languageserver-node/pull/1047


[Extensibility BUG]  codeunit 487 "Job Queue Start Report" - OnRunReportOnAfterCalcShouldModifyNotifyOnSuccess

I requested an event for OnRunReportOnAfterCalcShouldModifyNotifyOnSuccess and specified that the event sould be added to a public (Access = External) Codeunit that we can subscribe to it. See Issue https://github.com/microsoft/ALAppExtensions/issues/16505
Could you please fix this?
codeunit 487 "Job Queue Start Report"
{
    Access = Internal; // this is the problem
    TableNo = "Job Queue Entry";

    ....

/// THIS EVENT has to be in a PUBLIC accesible Codeunit
    [IntegrationEvent(false, false)]
    local procedure OnRunReportOnAfterCalcShouldModifyNotifyOnSuccess(ReportID: Integer; var JobQueueEntry: Record "Job Queue Entry"; var ShouldModifyNotifyOnSuccess: Boolean)
    begin
    end;
}

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.


Nixpkgs bundle feedback

Hi. I was looking to test out this project for doing Neovim configuration because I like the ML family and this is one of the only active ML projects compiling to Lua/LuaJIT (though never touched SML). The first step was getting it to compile on NixOS and with Nixpkgs for myself. However, I ran into a couple of snags that I think make help make the project a bit more robust. I did get the project to build and was thinking of submitting it, but I feel y'all may have some ideas on how to round off the rough edges.
To start, here is my branch and the main file in question to make the derivation: https://gitlab.com/toastal/nixpkgs/-/blob/lunarml/pkgs/development/compilers/lunarml/default.nix
As for the feedback:

I would like to see make install from the root as the installPhase block of mine is a bit of mess as I can't seem to figure out the install ... flags so I went with cp. A Nix derivation prefers the Unix-style file structure with binaries in bin, libraries in lib/$PROJECT_NAME, docs in share/doc/$PROJECT_NAME, etc. This leads to more follow ups...
Can the sml-lib not be hardcoded from the binary? This prevents me from running make test in the checkPhase because I used substituteInPlace to rewrite that directory. Because I put lunarml in bin and sml-lib in lib/$PROJECT_NAME for the final build, compiling this hardcoded path is a problem as make test is ran in the current directory that doesn't have a lib/$PROJECT_NAME/sml-lib to work with.
Is it ideal to fetch the libraries with curl in a Makefile? This is subjective, but the Nix sandbox doesn't allow a lot of IO, like connecting to the network, so I couldn't just cd thirdparty && make install. I'm fetching the files in the derivation and it works, so this is more of an acknowledgement.

Tangential feature requests:

Can the compiler do stdin and stdout compilation (no writing/reading files)? Ultimately I would like to hook this into home-manager's Neovim configuration machinery where they allow a specified Lua compiler for configuration. Currently it support VimL, Lua... but also Fennel and Teal. Skipping the disk makes this a lot easier in many use cases like this.
Tree-shaking unused functions. Compiling hello.sml was fast, but it's over 7000 lines 😅
I'd love to see documentation on how to use your own or other's libraries. I don't quite understand how $SML_LIB is supposed to work.
Some sort of nested set() function (didn't see). Like set' [ "vim"; "g"; "scrolloff" ] 4 for vim.g.scrolloff = 4.

Thanks for the feedback!
As for the feedback:

OK.
OK, I'll make the path customizable via an command-line option or environment variable. Is that sufficient?
The repository contains thirdparty/ml-yacc.tgz and thirdparty/smlnj-lib.tgz. Can't they be used?

As for the feature requests:

OK. I'll add options like --input-stdin / --output-to-stdout.
In fact, LunarML already does dead code elimination. The problem is, DCE and other optimizations like inlining are not very powerful yet.
There are package managers for SML like smackage or smlpkg. We'll need to find out if they are usable with LunarML.
Writing your own set function shouldn't be too hard. Here's one (not tested):

local
fun set' (table, fieldName, [], newValue) = Lua.set (table, Lua.fromString fieldName, newValue)
  | set' (table, fieldName, x :: xs, newValue) = set' (Lua.field (table, fieldName), x, xs, newValue)
in
(* set : string list -> Lua.value -> unit *)
fun set [] newValue = raise Fail "set: empty"
  | set [name] newValue = raise Fail "We don't have Lua.setGlobal yet"
  | set (name :: x :: xs) newValue = set' (Lua.global name, x, xs, newValue)
end
val () = set ["vim", "g", "scrolloff"] (Lua.fromInt 4)

Well, I'll consider adding Lua.setGlobal and Lua.setField...

Thanks for the quick response.

OK, I'll make the path customizable via an command-line option or environment variable. Is that sufficient?

Either of these can be used from the sandbox just fine. It could help users that want to develop on alternative stdlibs?? Unsure.

The repository contains thirdparty/ml-yacc.tgz and thirdparty/smlnj-lib.tgz. Can't they be used?

Ah, I didn't check until after I ran make install so I didn't see it. I could, but usually I'm not a fan of checking binary files into my repo for the long-term as the git pulls can get unreasonable. But I probably should use the tarballs while they are there.

OK. I'll add options like --input-stdin / --output-to-stdout

I'd go for just plain --stdin and --stdout or have these be the defaults and and files need -o and -f or -i. I've seen these patterns most commonly with tools--plus it's a bit redundant to "input in" and "output out".

In fact, LunarML already does dead code elimination.

Ah. The -O flag didn't seem to do much on the hello.sml so I wasn't sure.

Writing your own set function shouldn't be too hard. Here's one (not tested):

Sure, but I think this may come up a lot in folks' code when interacting with Lua code bases like the ones in Neovim I mentioned. I see love.graphics.arc has a lot of nested calls. Awesome (window manager) can too. I have a hunch that it would be common enough task with FFI that it could be stdlib worthy.

I've changed the directory layout and added -B option to specify the library directory (default: ../lib/lunarml).

https://gitlab.com/toastal/nixpkgs/-/blob/lunarml/pkgs/development/compilers/lunarml/default.nix
Much cleaner. I'll close this for now because the Nix bundling is now very concise with little issue. Anything other feedback/suggestions would be better raised as separate issues if any pique the project's interest.


[Question] Crypted configs

i just wanted to know when are you planing to add the crypted config feature ??
i can see that 2 forks have it actualy.

I plan to make a branch of the projects that will support Encrypted configs but you will most likely need to host your own server. I will give indications on how to implement the API to make it work with the branched client.

I might even give a very simple ASP.NET core 3.0 web API project you can spawn on any server to have an easy to access place to host your encrypted Configs.

Thanks !
But maybe a php version would be more easier to use, it's easier to find a free hosting with PH than ASP...

You can buy a linux VPS for 3$ a month and place it there. I don't know PHP, if you want you can write your own PHP webapp following the API specifications I will give.

Ho yes for sure, i'll try to convert it in PHP then.

But any idea about "when" you'll add this feature ?? all sellers and those who want to hide their work are waiting for it.... that's mainly the reason why people are going to the forked mods offering it.


NEXT-9105 - Discount: Wrong payment snippet in checkout

1. Why is this change necessary?
Display the translated notice instead of snippet key
2. What does this change do, exactly?
Display proper text instead of shippet:

3. Describe each step to reproduce the issue or behaviour.
Steps to reproduce:

Setup shopping cart rule with a. payment type condition
Create a discount applied when rule is valid
Add item to shopping cart, select payment method
(copied from original ticket https://github.com/shopwareBoostDay/platform/issues/40)

4. Please link to the relevant issues (if any).
https://github.com/shopwareBoostDay/platform/issues/40
5. Checklist

[ ] I have written tests and verified that they fail without my change
[X] I have squashed any insignificant commits
[ ] I have created a changelog file with all necessary information about my changes
[ ] I have written or adjusted the documentation according to my changes
[ ] This change has comments for package types, values, functions, and non-obvious lines of code
[ ] I have read the contribution requirements and fulfil them.

This might require some refactor as current implementation uses ErrorCollection for displaying all types of messages: Notices, Warnings and Errors. Previously it was used only for Errors but it was extended by this change:
ENT-2303 - display errors by type in storefront  Signed-off-by: Jan Philipp Pietrzyk<EMAIL_ADDRESS>Potentially there is another bug as there are three LEVELs groups in src\Storefront\Resources\views\storefront\component\checkout\cart-alerts.html.twig and
    {% block component_checkout_cart_alerts_warnings %}
        {% for error in page.cart.errors.warnings %}
            {% set snippetName = "error.#{error.messageKey}" %}      <---------- here is the 'error' as well but should be probably 'checkout'


[Backend] Add a route to poll for the sync status on a repository

Whenever a user triggers a sync for a database, the client will need to poll the api to know when the sync of a repository is over.
We need a route that will return the status of a sync for a repository:

Closing since the last sync at value is part of the show action on repositories. The frontend can use this field to poll for a sync update.


Added the missing set namespace shell script + Fix Deploys

The namespace shell script seemed to be missing from my previous PR - so have added it here with some additional fixes.
Have also added a fix for deploying to garnet as it wasn't properly reading the environment variables.

Have added a fix for deploying to garnet - as it wasn't properly reading the world address from env variables.

Can we add some docs for this feature ?

Looks like it does not work in MAC

Still messes up when the input is given second time


adds object creation using only +

Imagine you're stranded on an island, nothing there except a terminal. But oh no, it's only possible to enter the characters ![]()+. You know you can escape the island when you're able to create a function with only these characters in JavaScript ... and you remember, wasn't there an extreme guide on how to create objects? Yes there was. And it saved your life.
Conclusion: This PR incresases you chance of survival when stranded on an island

Thank you, now I'm prepared for what Tom Hanks was not.


Security update needed

https://github.com/TryGhost/Ghost/security/advisories/GHSA-65p7-pjj8-ggmr

@sstruble we do not have ghost installed because we're using gatsby and gatsby-source-ghost to pull data in from ghost.  Because of this I don't think this vulnerability applies to us.  Let me know if there is something that I'm missing!

ok! I get 'critical security update available' every time I log into the Ghost CMS for DSNP.org, but if it doesn't actually affect us, I'll ignore it. Thanks for looking at this.


Syncing blocks halts on NEP-5 token transfer/mint

In tested neo-local instances, neo scan stops syncing blocks when performing a transfer implementation (mintTokens, deploy that sends tokens to the owner, transfer, ... ) on a NEP-5 contract.
This was confirmed by @WyattMufson on a separate occasion

I can reproduce the issue on the latest version of neo-local with this repo. I run deploy and then transfer. It might have to do with the fact that when I transfer neo-scan has it so the owner wallet has a negative balance (I forgot to update totalSupply when I minted it for the testnet - example wallet

After a little bit more investigation it seems that it's the OnTransfer call that is halting neo-scan. Previously my deploy function didn't cause it to stop even though it was transferring tokens. Once I added the line OnTransfer(None, TOKEN_OWNER, TOKEN_INITIAL_AMOUNT) in the deploy function it now stops neo-scan.

Seems the problem comes from the python notification server that returns transfer amount as integer instead of returning it as string for the endpoint http://server/v1/notifications/block/index

Can confirm that the recent release of neo-python fixes the neo-scan issue in neo-local. Opened a PR to update to the newer version https://github.com/CityOfZion/neo-local/pull/67

I will close this issue then


Selectable columns in users table

idk how this would be implemented but it came in Make a Wish

it'd be awesome to be able to select the email address in the users table without going to their detail page

How do you feel about making just the name clickable and the rest not? The teams admin page has the Name column as a link (though the rest of the row is clickable anyway).

Fixed in Nova


SAA-2065: Remove GET appointment set unused endpoint

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 84.30%. Comparing base (9322a41) to head (98d5536).

Additional details and impacted files
@@             Coverage Diff              @@
##               main    #1138      +/-   ##
============================================
- Coverage     84.30%   84.30%   -0.01%
+ Complexity     3724     3722       -2
============================================
  Files           383      383
  Lines         12624    12619       -5
  Branches       1095     1095
============================================
- Hits          10643    10638       -5
  Misses         1478     1478
  Partials        503      503

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Enlarge story thumbnails and improve story list Material layout

I've improved the story layout by having the thumbnails to display to the side of the the story and have the indices integrated into the side as shown in the images below. Let me know your thoughts on the new layout!

Does this pull thumbnails from the actual stories? Or is it just the favicon from the website the story is posted on?

@DeveloperPaul123 This is the favicons. In the app settings it is referred to as thumbnails

This is still a WIP. Happy to have any assistance.
When you turn off thumbnails but keep numbered stories in the settings we need to revert to the old layout with a number in front of the title.

Ok, I see.
I know you didn't' ask for my opinion, but I would say that it would be a better use of space if you were able to pull some sort of image from the article if the image is going to take up that much space in the row. Giving that much space to the favicon doesn't add much information (in my opinion) when scanning a list.
Just my 2 cents. Ultimately it will be up to Simon to merge this.

Hey sorry about not looking into this earlier - been away on various things.
While I think this is cool - my main concern is the resolution and general quality of the thumbnails. This look gives the thumbnails quite a big focus, taking some focus away from the title. However the thunbnails are not very specialized to the content (e.g.. all GitHub is the same),  generally are not too well-designed because of their constraints, and are sometimes not background safe.
For these reasons I'm hesitant to further promote them in the app.
What @DeveloperPaul123 is saying about images from the website (e.g. pretty much all Reddit clients RIP) is something I've been interested in for a long time but never have got around to.


says access denied when writing registry values.

Looks like TrustedInstaller cannot stop the windefend service anymore. It now says that it fails to stop the service and that it cannot write
to TamperProtection, DisableRealTimeMonitoring, etc.

hi, have you tried manually turning off tamper protection and running the program again? some other users ran into the same issue after a windows update

Sorry. I haven't tried that. Its working now.


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
Multiple installed packages found.
Brief description of your issue
WinGet-CLI Insists on Upgrading Microsoft's Powertoys from version 0.79.0 to 0.80.0, even though the version that's installed and running is 0.80.0

[!NOTE]
I'm unsure if this's a problem to be reported in this repo, or winget-cli repo, any guidance will help, thanks in advance 😄

Steps to reproduce

Have Microsoft's Powertoys installed on your system, with version 0.79.0 (This is the version I'm having problem with, maybe older version have the same problem when trying to upgrade to version 0.80.0)
Run CMD.exe or PowerShell Terminal as Admin
run winget upgrade Microsoft.PowerToys
Notice that the Program upgraded successfully, but now there's two packages with Identifier Microsoft.PowerToys listed when running winget list, but with different versions.

Actual behavior
Upgraded Microsoft.PowerToys but there's two Packages of it, with different versions.
Plus, when trying to upgrade using winget upgrade Microsoft.PowerToys or winget upgrade --all, it'll try and upgrade to version 0.80.0, even though it's already upgraded.
Expected behavior
Microsoft's Powertoys upgrades without problems
Environment
Windows Package Manager v1.7.10861
Copyright (c) Microsoft Corporation. All rights reserved.

Windows: Windows.Desktop v10.0.22631.3296
System Architecture: X64
Package: Microsoft.DesktopAppInstaller v1.22.10861.0

Winget Directories
-----------------------------------------------------------------------------------------------------------------------
Logs                               %LOCALAPPDATA%\Packages\Microsoft.DesktopAppInstaller_8wekyb3d8bbwe\LocalState\Diag…
User Settings                      %LOCALAPPDATA%\Packages\Microsoft.DesktopAppInstaller_8wekyb3d8bbwe\LocalState\sett…
Portable Links Directory (User)    %LOCALAPPDATA%\Microsoft\WinGet\Links
Portable Links Directory (Machine) C:\Program Files\WinGet\Links
Portable Package Root (User)       %LOCALAPPDATA%\Microsoft\WinGet\Packages
Portable Package Root              C:\Program Files\WinGet\Packages
Portable Package Root (x86)        C:\Program Files (x86)\WinGet\Packages
Installer Downloads                %USERPROFILE%\Downloads

Links
---------------------------------------------------------------------------
Privacy Statement   https://aka.ms/winget-privacy
License Agreement   https://aka.ms/winget-license
Third Party Notices https://aka.ms/winget-3rdPartyNotice
Homepage            https://aka.ms/winget
Windows Store Terms https://www.microsoft.com/en-us/storedocs/terms-of-sale

Admin Setting                             State
--------------------------------------------------
LocalManifestFiles                        Enabled
BypassCertificatePinningForMicrosoftStore Disabled
InstallerHashOverride                     Disabled
LocalArchiveMalwareScanOverride           Enabled

Screenshots and Logs

Hi @og-mrk,
Thanks for adding a screenshot. This happens sometimes when a package that's not designed for side-by-side install becomes installed twice, through a bug or other unexpected situation. Could that have happened here?

Hi @og-mrk,
Thanks for adding a screenshot. This happens sometimes when a package that's not designed for side-by-side install becomes installed twice, through a bug or other unexpected situation. Could that have happened here?

Hello @stephengillie,
No problem with the screenshots, happy it was useful 😄
And this problem might be related to winget-cli it self, not really to the PowerToys package, as I don't really see why it would fail (Nothing out of the ordinary when comparing the Installer manifest files).
I'll keep this issue open, as I'm unsure the root case of it, further investigation will be required to know why it happens.

It's not impossible that an internal error might cause incorrect output, but usually a duplicate package entry is caused by a duplicate Registry entry. Does the Apps and features Settings page (or the Control Panel's Add and Remove Programs page) have 2 entries for this also, or just one?
Generally, the package manager downloads a file with all of the manifest data, then compares this with the data in the Registry, returning matching results. These pages read the same Registry entries, so they can give another perspective of the data stored there.

It's not impossible that an internal error might cause incorrect output, but usually a duplicate package entry is caused by a duplicate Registry entry. Does the Apps and features Settings page (or the Control Panel's Add and Remove Programs page) have 2 entries for this also, or just one?

Yes there's 2 entries in Apps and Features Settings page, see screenshot below.

If you click on the 3 dots to the right, is it possible to uninstall the older version? (If trying this won't work out, it's okay.)

If you click on the 3 dots to the right, is it possible to uninstall the older version? (If trying this won't work out, it's okay.)

Shows that a specific registry key isn't available, screenshot below.

Trying to uninstall PowerToys in general isn't possible using WinGet, getting Ox800401f5 : unknown error, screenshot below.

Thanks for these. Could it be that the Registry entry is available, but the files it points to aren't?
It seems like a 2nd Registry entry got created somehow. The fix would be removing that entry. Some common ways are:

Uninstall the program through other means, such as the Apps and features Settings page, from the installer, or maybe with uninstaller software.

Then run this again, to remove the other Registry entry.
Then reinstall with the package manager.

Edit the Registry, with the Registry Editor or another tool. (Caution: not recommended, as it's easy to accidentally change the wrong setting, and there might not be an undo or backup.)

Fixing this might be outside of the package manager's current abilities.

Thanks for these. Could it be that the Registry entry is available, but the files it points to aren't?

This is true, there's a Registry Key for 0.79.0 version of Microsoft's PowerToys, but the Files for it isn't available, because while upgrading PowerToys using winget-cli, the newer version (0.80.0) of the package has deleted the older one (0.79.0).

It seems like a 2nd Registry entry got created somehow. The fix would be removing that entry. Some common ways are:

Uninstall the program through other means, such as the Apps and features Settings page, from the installer, or maybe with uninstaller software.

Then run this again, to remove the other Registry entry.
Then reinstall with the package manager.

Edit the Registry, with the Registry Editor or another tool. (Caution: not recommended, as it's easy to accidentally change the wrong setting, and there might not be an undo or backup.)

I've fixed the Registry issue by using a Free and Open Source program called BCUninstaller, used it to get and delete the Registry Key for PowerToys version 0.79.0, link to their GitHub Repo.

Fixing this might be outside of the package manager's current abilities.

I do agree on this, while the root of this problem might be related to PowerToys Package it selfg (Very Unlikely, the package manifest is quite normal IMHO), or WinGet it self, it would make sense that WinGet makes a Registry Backup of the old version, then purges said version, then installs the new version... but I don't have a say on this, due to my own lack of expertise in this field.

Will close this issue, as this problem has been fixed for me using the above solution, thanks for the help/feedback @stephengillie 😄


Should hint about deprecated [dev_dependencies]

https://github.com/mainmatter/cargo-autoinherit/blob/eb922d3ea3d4376fca33fd5ef4a9ffe580b7f20c/src/lib.rs#L135-L138
This panics on [dev_dependencies] but I was confused by the panic until looking at the source code.

I forgot the snake case variant was still accepted.
We need to fix this, for build dependencies as well.


Updated CI Xcode versions

Are you no longer able to take PRs?

@Brett-Best sorry, my bad! Closed this by complete accident, thought I was pruning an old branch.


backport PR 218 to 2024.1 add confirmation to link enabling/disabling

Please backport PR #218 to 2024.1

@italovalcy fyi https://github.com/kytos-ng/topology/issues/221


Do not use 'master' in release name when replication is not enabled

What this PR does / why we need it: Improve the release name when the chart only uses one pod for the database.

/ok-to-test


On Components/Spinners page, link to "text alignment" utility has incorrect path

On the Components/Spinners page, under the "Placement" header, the link to the "text alignment" utility leads to Content/Typography page, not the Utilities/Text page (to the header "Text alignment").
Just thought you should be aware of this.

Newbie here, would it be okay if I give this a try? I believe the correct link should be here.