AssertionError: Negative optical density.

Had a few issues while using this library. They are as follows:

Installation issue(spams error)
Running stain normalization on a batch if 15000 images resulted in the Negative optical density error.

Looking forward to a fix. Thanks.

Lets track the pip install issue here #32.
In terms of the negative optical density issue can you provide more details.

Do you have a stack trace?
Which algorithm were you using (macenko or vahadane)?
Can you provide an example failure image and script to run?
How negative were the OD values?

Vahadane algorithm was used for stain normalization. The stack trace is given below:
/usr/local/lib/python3.6/dist-packages/staintools/stain_normalizer.py:41: RuntimeWarning: invalid value encountered in true_divide
  source_concentrations *= (self.maxC_target / maxC_source)
/usr/local/lib/python3.6/dist-packages/staintools/miscellaneous/miscellaneous_functions.py:26: RuntimeWarning: invalid value encountered in true_divide
  return A / np.linalg.norm(A, axis=1)[:, None]
/usr/local/lib/python3.6/dist-packages/numpy/core/_methods.py:29: RuntimeWarning: invalid value encountered in reduce
  return umr_minimum(a, axis, None, out, keepdims)
---------------------------------------------------------------------------
AssertionError                            Traceback (most recent call last)
<ipython-input-18-636ad9ad5400> in <module>()
      3
      4 for i in x_train[:]:
----> 5   img = stain_normalize(i)
      6   x_train_st.append(img)
      7   #print(count)

<ipython-input-15-66a64e10eb9f> in stain_normalize(img)
     13   # Stain normalize
     14   normalizer = staintools.StainNormalizer(method='vahadane')
---> 15   normalizer.fit(target)
     16   transformed = normalizer.transform(to_transform)
     17   return transformed

/usr/local/lib/python3.6/dist-packages/staintools/stain_normalizer.py in fit(self, target)
     27         self.target_concentrations = get_concentrations(target, self.stain_matrix_target)
     28         self.maxC_target = np.percentile(self.target_concentrations, 99, axis=0).reshape((1, 2))
---> 29         self.stain_matrix_target_RGB = convert_OD_to_RGB(self.stain_matrix_target)  # useful to visualize.
     30
     31     def transform(self, I):

/usr/local/lib/python3.6/dist-packages/staintools/miscellaneous/optical_density_conversion.py in convert_OD_to_RGB(OD)
     25     :return: Image RGB uint8.
     26     """
---> 27     assert OD.min() >= 0, "Negative optical density."
     28     OD = np.maximum(OD, 1e-6)
     29     return (255 * np.exp(-1 * OD)).astype(np.uint8)

AssertionError: Negative optical density.

Th code:
def stain_normalize(img):

  '''Return the image with the stain removed, probably...'''

  # here, both are same image
  target = img
  to_transform = img

  # Standardize brightness (This step is optional but can improve the tissue mask calculation)
  target = staintools.LuminosityStandardizer.standardize(target)
  to_transform = staintools.LuminosityStandardizer.standardize(to_transform)

  # Stain normalize
  normalizer = staintools.StainNormalizer(method='vahadane')
  normalizer.fit(target)
  transformed = normalizer.transform(to_transform)
  return transformed

x_train_st =[] # store normalize images here
count = 0

# x_train is of shape (27588, 200, 200, 3)

for i in x_train[:]:
  img = stain_normalize(i)
  x_train_st.append(img)
  #print(count)
  #count+=1

x_train_st = np.array(x_train_st)
x_train_st.shape

Seems odd. I wonder is there anything atypical about the images for which the method is failing (all white, all black)?

On another note you seem to be fitting and transforming to the same img. I'm not quite sure of the point here. If the library is working correctly you should just get back the original image?

Will check the image(s) were the error occurred. Fitting and transforming to the same image results in this:

Well this image is not really typical of an image where the algorithms will perform well. Thinking just about stain extraction any pixel in an ideal H&E image should have its color composed from a combination of the two basis stains (H & E). The original image above (left) looks to have no/very-little Hematoxylin (would be purplish) and therefore the algorithm will not be able to accurately determine the stains and will therefore probably not do much sensible.
In practice most real images also have some background which will be white/light-gray. To accurately determine the stain colors these pixels should be removed before stain extraction. The current stain-normalizer implementation tries to remove these by filtering on the luminosity (I think currentlyit uses a fixed value of 0.8). So pixels with a luminosity higher than the threshold are considered background and are ignored for stain extraction. This can lead to the final image not looking exactly like the original, but for typical images the difference shouldn't be substantial.
I see you are using the brightness-standardising step first. This is intended for images which are dim and where the background is significantly grey. The idea is to try and make the background close to white in order to help segment the tissue and remove the background. For the image above though it would essentially make the entire image white as the image is very uniform in color and (assuming that really is tissue and not background) that would therefore be an error as when we then try to identify the tissue and extract from the background we will get almost nothing back.
I see there is also what looks like a black segmentation mask around the image. That will also cause problems in determining the stains in the image.

@Peter554  What would you suggest I do?
Can you also give me a requirements.txt file from you end, so that I can make a separate environment using that?

Regarding requirements please consult the setup.py.
Perhaps have a look inside stain_normalizer.py. This is basically a wrapper for a few simple steps e.g. extract stains, get concentrations. If you just do the steps directly you could override the default value for the luminosity threshold (this is an input to stain_extractor). For that image above you would want not to standardize the brightness using my crude method. Have a think about your set of images as a whole and perhaps you may be able to come up with something better? The library source code is not long so maybe just have a look inside and just take what works for you.
On the other hand if you have such a large dataset I wouldn't necessarily advise bothering with stain normalization. If you're doing something like deep learning you may just be better of and save a lot of preprocessing time to apply some on the fly color augmentation #29.

Going back to the original issue though. Even though I wouldn't expect the algorithm to work very well on the image above, I'm still not quite sure why this negative optical density is occurring. If I work it out I'll let you know.

Alright, thank you for your help.


fix: add color swatches to tsx files

add color swatches to tsx files using className
closes #24

Nice :), that should also fix it for jsx files resp. all react-like frameworks which use className.


RFC: Generate babel-plugin-mylib for import rewrites

Taking a cue from react-native-web, we could likely generate a babel-plugin-mylib in the template that rewrites imports to their preshaken path.
This could be super nice in addition to multi-entry support for component libraries

So I'm not sure that this is totally necessary because babel-plugin-transform-imports could be easily configured to do something like this.
I've previously used it for transforming imports to react-bootstrap to individual component imports


Good job. (inb4 fixes next day)


fix for using with bootstrapv3

Sorry, why are these changes necessary?

When using with Twitter Bootstrap, a:hover is set to blue so hovering a button makes the text disappear on the blue button background. This isn't just for Bootstrap, it's pretty common to change link colors on hover.
The margin change, is to remove the link artifact and make the hover effect over the button not empty space.
A demonstration of the problem when hovering over the buttons on a page with Bootstrap:

Ah awesome, good catch, thanks man!


No support for Spark 2.0, write document fails

Hi,
The basic example with read from google sheets and write back fails with a stacktrace when running on Spark 2.0;
val df = sqlContext.read.format("com.github.potix2.spark.google.spreadsheets").
option("serviceAccountId", accountId).
option("credentialPath", credPath).
load(SPREADSHEET_ID+"/"+SPREADSHEET_NAME)
df: org.apache.spark.sql.DataFrame = [TEST : string, TEST2: string]
scala> df.show
+-----+-----+
|TEST |TEST2|
+-----+-----+
|    a    |    b   |
|    c    |    d   |
+-----+-----+
scala> df.write.format("com.github.potix2.spark.google.spreadsheets").
option("serviceAccountId", accountId).
option("credentialPath", credPath).
save(SPREADSHEET_ID+"/"+SPREADSHEET_NAME+"_new")
java.lang.AbstractMethodError: com.github.potix2.spark.google.spreadsheets.DefaultSource.createRelation(Lorg/apache/spark/sql/SQLContext;Lorg/apache/spark/sql/SaveMode;Lscala/collection/immutable/Map;Lorg/apache/spark/sql/Dataset;)Lorg/apache/spark/sql/sources/BaseRelation;
at org.apache.spark.sql.execution.datasources.DataSource.write(DataSource.scala:442)
at org.apache.spark.sql.DataFrameWriter.save(DataFrameWriter.scala:211)
at org.apache.spark.sql.DataFrameWriter.save(DataFrameWriter.scala:194)
... 48 elided

Must be a class version error.. I was running Spark 2.0.2 when spark-google-spreadsheets lib is compiled with Spark-core 1.6.2. Do you plan to release a new build for 2.0 series Spark?

@Henningdev Thank you for reporting. I am planning to support spark 2.0 and there is a development branch locally. I will do my best to be released as soon as possible.

@potix2 Superb! Google Sheets and Spark is a nice combo when analysing and presenting data

Now, version 0.5.0 for Spark 2.0.x is released!


Prometheus not doing remote write to InfluxDB OSS v2.0

Steps to reproduce:
List the minimal actions needed to reproduce the behavior.
Trying to do prometheus remote write in influxdbv2.0

...
remote_write:
- url: http://localhost:8086/api/v1/prom/write?db=prometheus&u=****&p=*****
basic_auth:
password: ********
...
...

Expected behavior:
Prometheus remote write should happen in influxdb2.0 and can see the measurements
Actual behavior:
Describe What actually happened.
No data getting created at influxDB
Environment info:

System info: Run uname -srm and copy the output here
InfluxDB version: Run influxd version and copy the output here
Other relevant environment details: Container runtime, disk info, etc

Config:
Copy any non-default config values here or attach the full config as a gist or file.

Logs:
Include snippet of errors in log.
Performance:
Generate profiles with the following commands for bugs related to performance, locking, out of memory (OOM), etc.
# Commands should be run when the bug is actively happening.
# Note: This command will run for ~30 seconds.
curl -o profiles.tar.gz "http://localhost:8086/debug/pprof/all?cpu=30s"
iostat -xd 1 30 > iostat.txt
# Attach the `profiles.tar.gz` and `iostat.txt` output files.

duplicate: https://github.com/influxdata/influxdb/issues/17958


Add a .gitattributes to use export-ignore

export-ignore can be used to exclude files from the downloaded zip file used by Godot, preventing downloading unnecessary/unrelated files such as the GUT addon
An example .gitattributes file using export-ignore:
/** export-ignore
/addons -export-ignore
/addons/2d_regular_polygons/** -export-ignore

Additional Resources:
Submitting to Godot Asset Library
gitattribute documentation

#34


Wrong result for example?

It may be that I'm not understanding how to use this code correctly, but as I'm interpreting the output on the example included with this package it is not optimal. The scipy implementation finds the correct one.
In [1]: import numpy as np

In [2]: import hungarian

In [3]: from scipy.optimize import linear_sum_assignment

In [4]: inf = 1000

In [5]: a = np.array( [[inf,2,11,10,8,7,6,5],
                  [6,inf,1,8,8,4,6,7],
                  [5,12,inf,11,8,12,3,11],
                  [11,9,10,inf,1,9,8,10],
                  [11,11,9,4,inf,2,10,9],
                  [12,8,5,2,11,inf,11,9],
                  [10,11,12,10,9,12,inf,3],
                  [10,10,10,10,6,3,1,inf]] )

In [6]: sorted(zip(*hungarian.lap(a)))
Out[6]: [(0, 1), (1, 2), (2, 0), (3, 4), (4, 5), (5, 3), (6, 6), (7, 7)]

In [7]: sorted(zip(*linear_sum_assignment(a)))
Out[7]: [(0, 1), (1, 2), (2, 0), (3, 4), (4, 5), (5, 3), (6, 7), (7, 6)]

In [8]: sum(a[hungarian.lap(a)])
Out[8]: 2013

In [9]: sum(a[linear_sum_assignment(a)])
Out[9]: 17

Seems like I'm misunderstanding the output of this algorithm - sum(a[range(8),hungarian.lap(a)[0]]) gives the correct sequence.

Ah okay, thanks for looking into it.
Now that scipy has their own hungarian implementation I should probably just deprecate this package in favor of theirs, so thanks for alerting me to that as well.

Please do not - the scipy one is incredibly slow on large problems.
This package runs in about 36 minutes on my matrix, I have yet to see the scipy one return after a day or running.

Interesting! Alright I'll keep it alive then, and maybe see if I can speed up scipy's version too…

I think scipy adopted https://github.com/ben-eysenbach/munkres which is pure python. I had fixed it up for my own use before I noticed it in scipy, but it was already way too slow.


Requirement Migrations

Summary

Implemented requirement migrations structure
Adjusted types to accommodate migrations for majors
Decorated migration requirements with courses with JSON generator
Wrote documentation on how to add requirement migrations each year

Test Plan

Using the ECE Major, change entry year to post / pre 2021 and notice changes in Core Courses requirements
Adding new migrations for other majors/minors and seeing changes in requirement bar
Check that after adding new migrations, especially Modify migrations, that the order of requirements stays the same

Notes
Documentation in Notion describing how to add requirement migrations: https://www.notion.so/courseplan/Requirement-Migrations-126ffdc4c2274e15a9e40bb8432b791b?pvs=4

Great work Nidhi! I think this is a great step for adding support to changing majors and minors. Documentation of this code looks great, I couldn't find any immediate problem with the code written. Excellent use of maps and filters. In the future have we considered adding support of this feature for changing college requirements? The ILR school changed their college requirements in 2022, it would be useful to be able to adopt similar method to encounter this case in the future.

Looks really good Nidhi! I'm curious about what we want to do with the requirements migrations after everyone that the former requirement applies to graduates. They don't necessarily take up a lot of space, but we might want to think about cleaning them up in our annual update, or at least sorting them by year so we can easily keep track of which migrations exist and are applicable.

Tested according to test plan and works great, well done Nidhi. Not doing a detailed review on account of all the feedback you received earlier.


There are a few things I'd like to know

Hi, I have paid attention to your project and I think it is very good, but I still don't know what the output is. Could you explain it to me roughly? Can it render guassiansplat

Hi! Thank you very much for your interest in my project!
Right now, the available version is an intermediate one, and it is unable to render Gaussian splats yet. However, I'm actively working on it as part of my master's thesis, and I plan to bring it to a full release within the next six to eight months. Stay tuned for updates!
Thanks again for your feedback and for following the project!


Support ArgMin and ArgMax for NVGPU Backend

I added these tests mainly to get a better understanding of how the Argmax op works in higher order tensors.
I am just temporarily disabling these tests for the GPU since I have not completed my implementation. I will enable them once I have a working implementation.
From: Robert Kimball<EMAIL_ADDRESS>Reply-To: NervanaSystems/ngraph<EMAIL_ADDRESS>Date: Thursday, October 4, 2018 at 9:18 AM
To: NervanaSystems/ngraph<EMAIL_ADDRESS>Cc: "Moitra, Ayan"<EMAIL_ADDRESS>Assign<EMAIL_ADDRESS>Subject: Re: [NervanaSystems/ngraph] Support ArgMin and ArgMax for NVGPU Backend (#1737)
@rkimballn1 requested changes on this pull request.

In src/ngraph/runtime/gpu/unit_test.manifesthttps://github.com/NervanaSystems/ngraph/pull/1737#discussion_r222663847:

@@ -34,6 +34,9 @@ max_pool_3d

avg_pool_3d
argmin_trivial
argmax_trivial
+argmax_3D_axis_0
You are disabling these new tests. Do they not pass? Do they pass for all other backends?
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHubhttps://github.com/NervanaSystems/ngraph/pull/1737#pullrequestreview-161604341, or mute the threadhttps://github.com/notifications/unsubscribe-auth/Ao_k5CzlOqNXTGPV7FDHv7mva6Itr96vks5uhgp4gaJpZM4XGQz-.


Can't find lcov.info if it's excluded from the VSCode project

Great plugin – now that I got it working!
I spent the first 10 minutes fighting with it - reloading, reinstalling, changing config settings.
Then I realized the problem might be that the lcov.info is in an excluded path. (I exclude my coverage folder so the hundreds of generated HTML files – one per source file – don't clog up my search results)
{
    "files.exclude": {
        "**/.git": true,
        "**/.svn": true,
        "**/.hg": true,
        "**/CVS": true,
        "**/.DS_Store": true,
        "dist-*/**": true,
        "coverage/**": true
    }
}

I've found a workaround:
        "coverage/{server,shared,client,lcov-report}/**": true

But it would be nice if I didn't have to do this!

Thanks for the issue keith, I have had other coworkers give me similar feedback around excluding files and having the lcov not be detected. I believe this is related to the findFiles api vscode exposes for extension development.
https://code.visualstudio.com/docs/extensionAPI/vscode-api
Find files in the workspace.

sample - findFiles('**∕*.js', '**∕node_modules∕**', 10)
Parameter	Description
include: string
A glob pattern that defines the files to search for.

exclude?: string
A glob pattern that defines files and folders to exclude.

maxResults?: number
An upper-bound for the result.

token?: CancellationToken
A token that can be used to signal cancellation to the underlying search engine.

Returns	Description
Thenable<Uri[]>
A thenable that resolves to an array of resource identifiers.

My implementation: https://github.com/ryanluker/vscode-coverage-gutters/blob/master/src/lcov.ts#L22
I am thinking a work around for this would be to try to do a raw node file system search (slower, but wouldnt use vscode's exclude settings) in the case where the findFiles fails to locate. Although, maybe
this could even be the default option if I find the performance hit isnt too bad... Anyways thanks again for the issue, I will try to get it into the 1.0 release. I hope you continue to enjoy the plugin.

@keithkml the glob solution in #67 will go out in the version 1.0.0 release (hopefully within a week) thanks again for submitting an issue about this 😄

Yay thanks!!! Love the plugin!


Does CRNN support text line images?

I tried to run the demo of a trained model on a number of images with text separated by spaces - but only the first word was predicted. I'm a little confused by this - there are multiple words in the images, and I have checked that a space is also in the character table.

Of course, it's just that the pre-trained model is based on a single word. The training set can use a whole paragraph of text, but there is a theoretical upper limit for the length, and the accuracy will be greatly reduced if the length exceeds a certain length.
There is a more complex system here, although the principles of recognition are similar.


check name of lat/long columns

I recently renamed the latitude/longitute columns in baserow to lat/long which for sure might break some of the existing code;

please check the baserow tables for consistent naming of those columns
update the code base accordingly

changed columns in documents table to lat/long, the rest was already named accordingly

Adjusted code for Neighbourhoods, Documents doesn't seem to be using the lat/long columns in any code (yet)


Make PhpOption optional

Hi there,
I had to update my generated src/gtfs-realtime.php (in https://github.com/Bertware/gtfs-realtime-bindings-php, which depends on your library), and it turns out to have PhpOption objects returned where fields are optional now. Unfortunately, that doesn't go well with my existing code.
Is there any chance to make the use of PhpOption optional when generating the files? I'm happy to create a merge request for that, just wanted to check before.
Cheers

I believe Option (and a bunch of bug fixes) are the only things we really added to the original DrSlump library.    We're pretty opinionated that optionals should not be optional.

Thanks for your response.


Fix duplicate submodules in .gitmodules.

Currently there are duplicate entries for jemalloc and googletest in the .gitmodules file for this repo. While git itself has no issue with this, other implementations of git do.
This PR removes the duplicate entries to allow the repository to work properly with other git implementations while ensuring that it still builds.

Thanks!

No problem! It was an easy fix, and this language looks very interesting.

I'm curious, how did you discover this issue?

other implementations of git do

Which one did you use if it's not secret? I've never heard of anyone using a different git implementation for real before.

The language's caching behaviour is of particular interest to me for my work on Luna, so I was browsing it on the go using Working Copy. Seeing as all the folders under third-party were empty, I filed a bug with the developer and he worked out the issue!
I believe it uses libgit2 under the hood to implement the git functionality.

Nice, thanks for the context! Luna sounds really interesting! https://luna-lang.org/

No problem! Glad you like the look of Luna!


Refactor the damage and resource pipelines

Refactors the damage and resource pipelines to be standardized and to provide a sequence of steps

Adds a new pipelines module, which now has the damage and resource pipelines
The damage pipeline has been broken into sequence of discrete steps
Expands on the BonusesDataModel, DamageBonusesDataModel models to support skills such as Defensive Mastery, Symbol of Weakness.

After this latest iteration I feel more confident on the architecture, so I believe it's worth taking a look and now and giving feedback in order to improve the design.


Show minimap with image overlay

Hi,
I'm trying to show minimap with image and markers,
my problem is that the minimap doesn't show all map as expected...
I see all the map on the screen but in the minimap just part of the map, this is a picture:

I attached my code:
let osm2 = this.mapDataService.drawMap(this.mapData);
let markers2 = this.itemsDrawService.draw(this.items);
let layers = new LayerGroup([...osm2.layers, ...markers2]);
let miniMap = new MiniMap(layers, {toggleDisplay: true});
miniMap.addTo(this.map);
Thanks
Yael

Hi,
I have the same issue, is there a solution?
Thanks
Michael


Changelog is not readable

🐛 Bug Report
It seems you've automated releases and changelog but in the meantime it became unreadable: https://github.com/cloudflare/wrangler/releases
It would be far more useful if someone described major changes like before instead of just writing commit messages most of which are "merge with master"

hey @sheerun ! you've caught us. The release you linked is actually a Release Candidate for 1.4.0, which will go out tomorrow with our usual carefully-curated changelog! Stay tuned...


There is VecInit.tabulate, why not VecInit.fill?

Type of issue: feature request

Impact: API addition (no impact on existing code)

Development Phase: request
Other information

It looks like there is a VecInit.tabulate. But there isn't a VecInit.fill. This seems asymmetrical.
There is VecINit(Seq.fill(...) {...}) but this is more verbose. Why should there be one and not the other?
https://www.chisel-lang.org/api/latest/chisel3/VecInit$.html#tabulate[T<:chisel3.Data](n:Int)(gen:Int=>T):chisel3.Vec[T]
If the current behavior is a bug, please provide the steps to reproduce the problem:
What is the current behavior?
There is only VecInit.tabulate, so you have to do somethign like VecInit.tabulate(num){unused => false.b})
What is the expected behavior?
That one could call VecInit.fill(num){false.b}
Please tell us about your environment:

What is the use case for changing the behavior?
more consistent, intuitive API

Closed by #2059 , thanks @anniej-sifive !


Manual npm publish

Adding steps for manually publishing to npm

we should make these scripts windows compatible?

Would be overkill I think. In future we can have it similar to how our dash works. Once someone manually creates a tag we can run a job. These scripts will then be run on travis.

@himadrisingh @AdityaHegde it would probably be good for us to consider doing this down the line for package.json scripts ~ see #168


Update README.md

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

0 of 0   changed or added relevant lines in 0 files are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage remained the same at 84.796%

Files with Coverage Reduction
New Missed Lines
%

pynapple/core/time_series.py
1
48.48%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
2187

Relevant Lines:
2515

💛  - Coveralls


Subnets broken

Currently we have the VPC set to use CIDR : <IP_ADDRESS>/16 the public subnet uses <IP_ADDRESS>/16 which is meaningless and takes up all the IPs in the VPC. This prevents the creation of multiple subnets in the region to exploit pricing.

[x]  Terminate instances in the Public Subnet and remove network interfaces.
[x]  Move the public subnet to CIDR : <IP_ADDRESS>/24
[x] Create 4 additional subnet, one for each AZ : 10.0.{1,2,3,4}.0/24
[x] Update the scaling groups and launch configs to launch into any of these subnets.

All these issues have been fixed on Turing-2.0


[SPARK-10371] [SQL][follow-up] fix code style

cc @nongli @yhuai

Merged build triggered.

Merged build started.

Test build #45633 has started for PR 9627 at commit 717d709.

Test build #45633 has finished for PR 9627 at commit 717d709.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):\n  * public class JavaLBFGSExample \n  * class LDA @Since(\"1.6.0\") (\n  * class SlidingRDDPartition[T](val idx: Int, val prev: Partition, val tail: Seq[T], val offset: Int)\n  * class SlidingRDD[T: ClassTag](@transient val parent: RDD[T], val windowSize: Int, val step: Int)\n

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/45633/
Test FAILed.

retest this please.

Merged build triggered.

Merged build started.

Test build #45638 has started for PR 9627 at commit 717d709.

Test build #45638 has finished for PR 9627 at commit 717d709.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/45638/
Test FAILed.

retest this please

Merged build triggered.

Merged build started.

Test build #45643 has started for PR 9627 at commit 717d709.

LGTM thanks!

Test build #45643 has finished for PR 9627 at commit 717d709.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/45643/
Test PASSed.

Thanks. Merging to master and branch 1.6.


Improve the color of text in submit button in Waline

To avoid something like this:

:tada: This PR is included in version 2.4.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Sanitise away timeline from json we submit to Mitre

shouldn't need to do this if timeline is in the right format (which is is now)


doc: migrate existing resource from a cluster to karmada

What type of PR is this?
/kind documentation
What this PR does / why we need it:
Best practice of migrate existing resource from a cluster to karmada
Which issue(s) this PR fixes:
https://github.com/karmada-io/karmada/issues/3824
Special notes for your reviewer:
@RainbowMango

My opinion:

This doc should focus on teaching people how to migrate legacy resource from a cluster to Karmada instead of a demo;
The content now seems like a tutorial which should be organized into docs/tutorials.

/cc RainbowMango
I think it is ready to review again~

kindly ping @RainbowMango  quick click

kindly ping @RainbowMango


Implement mlib example word2vec.py

Did the ml version, now I will do the correct version in mllib


WatchOS Standalone Application - Connection issue: HTTP Insecure Requests

Hi,
I am new to TDLib and TDLibKit. I've been digging into these libraries for just a couple weeks so far and everything seems to be quite fine except for one thing:
When I run the application, I create a tdlibclient instance and in the update handler I run the setTdLibParameters function. Once done, I keep getting these warnings for insecure http requests:
[Updates] [debug] Starting handler
[ 3][t 4][1704344174.582313060][Td.cpp:2609][#1][!MultiTd]	Create Td with layer 169, database version 14 and version 52 on 4 threads
[ 3][t 4][1704344174.584079980][Td.cpp:4121][#1][!Td][&td_requests]	Sending update: updateOption {
  name = "version"
  value = optionValueString {
    value = "1.8.23"
  }
}
[ 3][t 4][1704344174.584203004][Td.cpp:4121][#1][!Td][&td_requests]	Sending update: updateOption {
  name = "commit_hash"
  value = optionValueString {
    value = ""
  }
}
[ 3][t 0][1704344174.584208965][Client.cpp:293][&td_requests]	End to wait for updates, returning object 0 0x600000c54f90
[ 3][t 4][1704344174.584319114][Td.cpp:4121][#1][!Td][&td_requests]	Sending update: updateAuthorizationState {
  authorization_state = authorizationStateWaitTdlibParameters {
  }
}
[ 3][t 4][1704344174.584327936][Td.cpp:3001][#1][!Td][&td_requests]	Receive request 1: getOption {
  name = "version"
}
[ 3][t 4][1704344174.585586071][Td.cpp:4138][#1][!Td][&td_requests]	Sending result for request 1: optionValueString {
  value = "1.8.23"
}
[ 3][t 4][1704344174.585790157][Td.cpp:3001][#1][!Td][&td_requests]	Receive request 2: setLogVerbosityLevel {
  new_verbosity_level = 2
}
[cache] [debug] Database path: /Users/user/Library/Developer/CoreSimulator/Devices/C0383F4D-36B2-45BC-9302-07912C0A3063/data/Containers/Data/Application/18BFFB2A-7BC7-4065-B4FA-58F231F19294/Library/Application Support/TWatchGram/cache.sqlite
[cache] [notice] Started StorageService
[file:///Users/user/Library/Developer/CoreSimulator/Devices/C0383F4D-36B2-45BC-9302-07912C0A3063/data/Containers/Data/Application/18BFFB2A-7BC7-4065-B4FA-58F231F19294/Library/Caches/]
Info.plist contained no UIScene configuration dictionary (looking for configuration named "Default Configuration")
[ 2][t 1][1704344174.883208036][TdDb.cpp:428][#1][!RunOnSchedulerWorker]	Set PRAGMA user_version = 14
[ 2][t 4][1704344174.898053169][AuthDataShared.cpp:117][#1][!Td]	DcId{1} [auth_key_id:0][state:Empty][created_at:0][last_used:0]
[ 2][t 4][1704344174.903321266][Session.cpp:271][#1][!SessionProxy:1:main]	Generate new session_id<PHONE_NUMBER>071772451 for auth key 0 for main DC1
[ 2][t 4][1704344174.909845113][Session.cpp:271][#1][!SessionProxy:1:main]	Generate new session_id<PHONE_NUMBER>5837301747 for auth key 0 for main DC1
PDTask <291D928D-25D7-4D82-A7E3-C82F86FD3862>.<1> finished with error [-1022] Error Domain=NSURLErrorDomain Code=-1022 "The resource could not be loaded because the App Transport Security policy requires the use of a secure connection." UserInfo={NSErrorFailingURLStringKey=http://<IP_ADDRESS>:443/api, NSErrorFailingURLKey=http://<IP_ADDRESS>:443/api, _NSURLErrorRelatedURLSessionTaskErrorKey=(
    "LocalDataPDTask <291D928D-25D7-4D82-A7E3-C82F86FD3862>.<1>",
    "LocalDataTask <291D928D-25D7-4D82-A7E3-C82F86FD3862>.<1>"
), _NSURLErrorFailingURLSessionTaskErrorKey=LocalDataPDTask <291D928D-25D7-4D82-A7E3-C82F86FD3862>.<1>, NSLocalizedDescription=The resource could not be loaded because the App Transport Security policy requires the use of a secure connection.}
Task <291D928D-25D7-4D82-A7E3-C82F86FD3862>.<1> finished with error [-1022] Error Domain=NSURLErrorDomain Code=-1022 "The resource could not be loaded because the App Transport Security policy requires the use of a secure connection." UserInfo={NSErrorFailingURLStringKey=http://<IP_ADDRESS>:443/api, NSErrorFailingURLKey=http://<IP_ADDRESS>:443/api, _NSURLErrorRelatedURLSessionTaskErrorKey=(
    "LocalDataTask <291D928D-25D7-4D82-A7E3-C82F86FD3862>.<1>",
    "LocalDataPDTask <291D928D-25D7-4D82-A7E3-C82F86FD3862>.<1>",
    "LocalDataTask <291D928D-25D7-4D82-A7E3-C82F86FD3862>.<1>"
), _NSURLErrorFailingURLSessionTaskErrorKey=LocalDataTask <291D928D-25D7-4D82-A7E3-C82F86FD3862>.<1>, NSLocalizedDescription=The resource could not be loaded because the App Transport Security policy requires the use of a secure connection.}
PDTask <FDEA4F3B-DB33-4215-8B70-76F40FD1FE5F>.<2> finished with error [-1022] Error Domain=NSURLErrorDomain Code=-1022 "The resource could not be loaded because the App Transport Security policy requires the use of a secure connection." UserInfo={NSErrorFailingURLStringKey=http://<IP_ADDRESS>:80/api, NSErrorFailingURLKey=http://<IP_ADDRESS>:80/api, _NSURLErrorRelatedURLSessionTaskErrorKey=(
    "LocalDataPDTask <FDEA4F3B-DB33-4215-8B70-76F40FD1FE5F>.<2>",
    "LocalDataTask <FDEA4F3B-DB33-4215-8B70-76F40FD1FE5F>.<2>"
), _NSURLErrorFailingURLSessionTaskErrorKey=LocalDataPDTask <FDEA4F3B-DB33-4215-8B70-76F40FD1FE5F>.<2>, NSLocalizedDescription=The resource could not be loaded because the App Transport Security policy requires the use of a secure connection.}
Task <FDEA4F3B-DB33-4215-8B70-76F40FD1FE5F>.<2> finished with error [-1022] Error Domain=NSURLErrorDomain Code=-1022 "The resource could not be loaded because the App Transport Security policy requires the use of a secure connection." UserInfo={NSErrorFailingURLStringKey=http://<IP_ADDRESS>:80/api, NSErrorFailingURLKey=http://<IP_ADDRESS>:80/api, _NSURLErrorRelatedURLSessionTaskErrorKey=(
    "LocalDataTask <FDEA4F3B-DB33-4215-8B70-76F40FD1FE5F>.<2>",
    "LocalDataPDTask <FDEA4F3B-DB33-4215-8B70-76F40FD1FE5F>.<2>",
    "LocalDataTask <FDEA4F3B-DB33-4215-8B70-76F40FD1FE5F>.<2>"
), _NSURLErrorFailingURLSessionTaskErrorKey=LocalDataTask <FDEA4F3B-DB33-4215-8B70-76F40FD1FE5F>.<2>, NSLocalizedDescription=The resource could not be loaded because the App Transport Security policy requires the use of a secure connection.}
PDTask <A4DBEFCB-7C16-4366-B1ED-24E3DF7E69BE>.<3> finished with error [-1022] Error Domain=NSURLErrorDomain Code=-1022 "The resource could not be loaded because the App Transport Security policy requires the use of a secure connection." UserInfo={NSErrorFailingURLStringKey=http://<IP_ADDRESS>:5222/api, NSErrorFailingURLKey=http://<IP_ADDRESS>:5222/api, _NSURLErrorRelatedURLSessionTaskErrorKey=(
    "LocalDataPDTask <A4DBEFCB-7C16-4366-B1ED-24E3DF7E69BE>.<3>",
    "LocalDataTask <A4DBEFCB-7C16-4366-B1ED-24E3DF7E69BE>.<3>"
), _NSURLErrorFailingURLSessionTaskErrorKey=LocalDataPDTask <A4DBEFCB-7C16-4366-B1ED-24E3DF7E69BE>.<3>, NSLocalizedDescription=The resource could not be loaded because the App Transport Security policy requires the use of a secure connection.}
Task <A4DBEFCB-7C16-4366-B1ED-24E3DF7E69BE>.<3> finished with error [-1022] Error Domain=NSURLErrorDomain Code=-1022 "The resource could not be loaded because the App Transport Security policy requires the use of a secure connection." UserInfo={NSErrorFailingURLStringKey=http://<IP_ADDRESS>:5222/api, NSErrorFailingURLKey=http://<IP_ADDRESS>:5222/api, _NSURLErrorRelatedURLSessionTaskErrorKey=(
    "LocalDataTask <A4DBEFCB-7C16-4366-B1ED-24E3DF7E69BE>.<3>",
    "LocalDataPDTask <A4DBEFCB-7C16-4366-B1ED-24E3DF7E69BE>.<3>",
    "LocalDataTask <A4DBEFCB-7C16-4366-B1ED-24E3DF7E69BE>.<3>"
), _NSURLErrorFailingURLSessionTaskErrorKey=LocalDataTask <A4DBEFCB-7C16-4366-B1ED-24E3DF7E69BE>.<3>, NSLocalizedDescription=The resource could not be loaded because the App Transport Security policy requires the use of a secure connection.}
PDTask <E3DA30BA-E26F-4F42-8896-6CF5ADDF6ED0>.<4> finished with error [-1022] Error Domain=NSURLErrorDomain Code=-1022 "The resource could not be loaded because the App Transport Security policy requires the use of a secure connection." UserInfo={NSErrorFailingURLStringKey=http://[2001:b28:f23d:f001::a]:443/api, NSErrorFailingURLKey=http://[2001:b28:f23d:f001::a]:443/api, _NSURLErrorRelatedURLSessionTaskErrorKey=(
    "LocalDataPDTask <E3DA30BA-E26F-4F42-8896-6CF5ADDF6ED0>.<4>",
    "LocalDataTask <E3DA30BA-E26F-4F42-8896-6CF5ADDF6ED0>.<4>"
), _NSURLErrorFailingURLSessionTaskErrorKey=LocalDataPDTask <E3DA30BA-E26F-4F42-8896-6CF5ADDF6ED0>.<4>, NSLocalizedDescription=The resource could not be loaded because the App Transport Security policy requires the use of a secure connection.}
Task <E3DA30BA-E26F-4F42-8896-6CF5ADDF6ED0>.<4> finished with error [-1022] Error Domain=NSURLErrorDomain Code=-1022 "The resource could not be loaded because the App Transport Security policy requires the use of a secure connection." UserInfo={NSErrorFailingURLStringKey=http://[2001:b28:f23d:f001::a]:443/api, NSErrorFailingURLKey=http://[2001:b28:f23d:f001::a]:443/api, _NSURLErrorRelatedURLSessionTaskErrorKey=(
    "LocalDataTask <E3DA30BA-E26F-4F42-8896-6CF5ADDF6ED0>.<4>",
    "LocalDataPDTask <E3DA30BA-E26F-4F42-8896-6CF5ADDF6ED0>.<4>",
    "LocalDataTask <E3DA30BA-E26F-4F42-8896-6CF5ADDF6ED0>.<4>"
), _NSURLErrorFailingURLSessionTaskErrorKey=LocalDataTask <E3DA30BA-E26F-4F42-8896-6CF5ADDF6ED0>.<4>, NSLocalizedDescription=The resource could not be loaded because the App Transport Security policy requires the use of a secure connection.}

I noticed that there's these two lines in TDLib's ConnectionCreator.cpp file, @ line 682
#if TD_DARWIN_WATCH_OS
  only_http = true;
#endif

Could this be the reason why on watchOS the requests are using the http protocol, in place of https?
Maybe this is something to ask on the TDLib issues directly?
This is annoying because in order to be able to get the connection to telegram servers to work, I need to set domain exceptions for the url that TDLib uses to make requests.
I will really appreciate any insight on this matter. Thank you in advance.

Telegram doesn't use TLS for encryption and instead uses MTProto protocol to transfer encrypted data. HTTP is used only as a transport for the encrypted data and doesn't impose any security issues for Telegram. As you mentioned in the issue description, you can acknowledge that the requests are secure by adding to NSExceptionDomains all IP addresses from https://github.com/tdlib/td/blob/d963044eb9b8bb075e3f63b8bfd8da735c4c37d9/td/telegram/net/ConnectionCreator.cpp#L1232-L1252

@levlam thank you for your kind and quick reply.
I thought that this was not going to be a security issue since everything is encrypted before transferring data.
What I am more concerned about is the fact that I will be uploading my application to the AppStore and remember that Apple  is quite strict about the security policy for secure loads.
Do you know if using http requests with exception domains will prevent the application from being accepted?

Apple's documentation doesn't mention that whitelisting of specific domains requires special approvement. That's all I know.


Check for dart keywords

Please list issues fixed by this PR here, using format "Fixes #the-issue-number".
Checklist

[x] #957
[ ] New tests are added to ensure new features are working. End-to-end tests are usually in the ./frb_example/pure_dart example, more specifically, rust/src/api.rs and dart/lib/main.dart.
I'm not sure how to create tests for not processing files.
[ ] The code generator is run and the code is formatted (e.g. via just refresh_all).
Well, I got a lot of errors:

Error: Couldn't resolve the package 'build_cli_annotations' in 'package:build_cli_annotations/build_cli_annotations.dart'.
Error: Couldn't resolve the package 'shelf' in 'package:shelf/shelf.dart'.
Error: Couldn't resolve the package 'shelf' in 'package:shelf/shelf_io.dart'.
Error: Couldn't resolve the package 'shelf_static' in 'package:shelf_static/shelf_static.dart'.
Error: Couldn't resolve the package 'path' in 'package:path/path.dart'.
Error: Couldn't resolve the package 'shelf_web_socket' in 'package:shelf_web_socket/shelf_web_socket.dart'.
Error: Couldn't resolve the package 'web_socket_channel' in 'package:web_socket_channel/web_socket_channel.dart'.
Error: Couldn't resolve the package 'puppeteer' in 'package:puppeteer/puppeteer.dart'.
Error: Couldn't resolve the package 'colorize' in 'package:colorize/colorize.dart'.
Error: Couldn't resolve the package 'yaml' in 'package:yaml/yaml.dart'.
error: Recipe `gen-help` failed on line 148 with exit code 254
error: Recipe `refresh_all` failed on line 100 with exit code 254

Apparently, it requires me to install dart stuff also by hand. Would it be a
good idea to add dependencies installation to just file also?

[ ] If this PR adds/changes features, documentations (in the ./book folder) are updated.
I believe it is pretty self-explanatory
[ ] CI is passing.

Remark for PR creator

New contributors will be blocked by GitHub from running CI, but you can use a trick to workaround this, and verify your code using CI by yourself.
If fzyzcjy does not reply for a few days, maybe he just did not see it, so please ping him.

Well, that's a very drafty example of how it could be done.
Good questions are:

Where it's better to keep the keywords?
Is is possible to write tests for such scenario?

Would it be a
good idea to add dependencies installation to just file also?

Definitely. the justfile is quite messy now... so check whether it is already there before changing (or make a overhaul of it)

Where it's better to keep the keywords?

at least in a const :)

Is is possible to write tests for such scenario?

not very sure, may need a bit of digging

@all-contributors please add @anstadnik for code


Create colegiultehnicmedia.txt

Added Colegiul Tehnic Media
Domain: @colegiultehnicmedia.ro
Website: https://colegiultehnicmedia.ro/

@alexvoicu97 First stage of verification passed, official domain is colegiultehnicmedia.ro as far as I verified. This is just an informational note. Pull request is still in the review. I appreciate your patience while waiting for the final decision.

@alexvoicu97 Thank you for your patience while waiting for a review and processing your request. Pull request merged.


Set a member of @allhosts group to this host to allow job submission

Problem: Job submisson out-of-the-box not possible
sgeadmin@3bd66b62587e:~$ echo "/usr/bin/date"|qsub
Unable to run job: warning: sgeadmin's job is not allowed to run in any queue
Your job 1 ("STDIN") has been submitted
Exiting.

The link in this PR is now broken as well. Found another copy here: http://download.jboss.org/jbosstools/updates/requirements/izpack/4.3.5/.


KNX Cover Stop

Hi
I have Hager TXA624C and these have a stop function to stop it is this planned to be added?
And from what i understand it is also used when using position.
Running version: 0.57.3

add your stop group address as move_short_address: '3/0/1'. (Most KNX components don't have a dedicated stop address).


Feature: "add" and "edit" buttons for files

This pull request adds the buttons/links for adding files to and _editing_files of a page when using the plugin with file-related options such as options: files or options: documents.
This is especially useful in connection with kirby-hidebar-field since the ability to add and edit files disappears completely.

Cool, thanks!


How to publish from multiple CI jobs to a single repository?

I have a project which requires artifacts to be published from different machines (some are built on MacOS). To try this plugin, I tried to configure publishing via multiple jobs in a simpler project.
From the README, my understanding is that I should have:

a first job that runs initializeSonatypeStagingRepository
a second job that runs findSonatypeStagingRepository publishToSonatype
a final job that runs findSonatypeStagingRepository cloneSonatypeStagingRepository

The first job does manage to create the repository. The second job does manage to list it, but it still creates a new one. The last job then fails because of duplicate repositories, which sounds normal.
Is it normal that the second job creates a new repository even though one already exists? Did I configure something incorrectly?
The full repository is here. The Nexus configuration is in the root build.gradle.kts, as requested in the documentation. The configuration of maven-publish, signing & co are in gradle/conventions/library/src/main/kotlin/library.gradle.kts, and is applied to the project :core, which I expect to be able to publish to Central.

Does problem in #273 cover your case? If yes, I propose to move the discussion there (where hopefully we will be able to propose some idea/solution/workaround "soon").

I'm not sure this is the same problem. In my case, I do not need different tasks or anything, all the libraries have the same type. This currently works fine, and :closeSonatypeStagingRepository works fine.
The only problem I have is that :publishToSonatype invokes :initializeSonatypeStagingRepository, even if :findSonatypeStagingRepository in the same Gradle invocation (see here, see lines 92 and 108).
I was expecting that, if :findSonatypeStagingRepository is called, :initializeSonatypeStagingRepository would not be called and :publishToSonatype would just use the found repository.

@CLOVIS-AI I've just looked closely at that case and put my finding in #273. If you are not interested in details, the following commands should work for you:
gw initializeSonatypeStagingRepository
gw -x initializeSonatypeStagingRepository findSonatypeStagingRepository publishToSonatype:mylib:publishAllPublicationsToStageRepository
gw findSonatypeStagingRepository cloneSonatypeStagingRepository

Btw, the following error from your CI:

Too many repositories found for stagingProfileId: 2387d0230df2b4, descriptionRegex: \b\Qdev.opensavvy.playground:Playground:0.1.0+central.4\E(\s|$). If some of the repositories are not needed, consider deleting them manually. Here are the repositories matching the regular expression: [ReadStagingRepository(repositoryId=devopensavvy-1037, type=open, transitioning=false, description=dev.opensavvy.playground:Playground:0.1.0+central.4), ReadStagingRepository(repositoryId=devopensavvy-1038, type=open, transitioning=false, description=dev.opensavvy.playground:Playground:0.1.0+central.4)]

might occur even with the aforementioned commands, for example if the pipeline fails and you want to release the same version again using an another pipeline execution. One solution would be adding the pipeline ID (probably CI_PIPELINE_IID would enough) to repositoryDescription used by the tasks.

Thanks a lot, I'll try that (probably tomorrow or the day after) and report back. I'm following the other issue for curiosity's sake :)

It works <3 Thanks a lot. I was originally using maven-publish directly, and I kept having issues with half of the project being in a staging repository, and the other half being in another one. Of course, they wouldn't pass validation checks since the signatures were not in the same repository.


Assist: CMD+ENTER not working

Check for existing issues

[X] Completed

Describe the bug / provide steps to reproduce it
I am using the Sublime Text default keymap with some additional mappings, but I can't get cmd+enter to work on the new assistant panel. The magic button in the top right does work.
[
    {
        "context": "Editor",
        "bindings": {
            "ctrl-[": "editor::MovePageUp",
            "ctrl-]": "editor::MovePageDown",
            "ctrl-d": "editor::GoToDefinition",
            "ctrl-f": "editor::MoveToNextWordEnd",
            "ctrl-b": "editor::MoveToPreviousWordStart"
        }
    }
]

Environment
Zed: v0.92.3 (stable)
OS: macOS 13.1.0
Memory: 32 GiB
Architecture: aarch64

If applicable, add mockups / screenshots to help explain present your vision of the feature

If applicable, attach your ~/Library/Logs/Zed/Zed.log file to this issue.
If you only need the most recent lines, you can run the zed: open log command palette action to see the last 1000.
No response

cmd-> also doesn't work, it's supposed to insert selected text from the last active editor pane

It looks like Atom, Sublime, and Textmate keymaps specifically have a binding "cmd-enter": "editor::NewlineBelow" that is clobbering the new internal assistant command.
As a temp fix, you might just be able to insert a block for the default assisstant commands in your custom keymap file:
{
  "context": "ConversationEditor > Editor",
  "bindings": {
    "cmd-enter": "assistant::Assist",
    "cmd-s": "workspace::Save",
    "cmd->": "assistant::QuoteSelection",
    "shift-enter": "assistant::Split",
    "ctrl-r": "assistant::CycleMessageRole"
  }
}

The real fix will be to adjust the keymap files such that those cmd-enters don't clobber the assistant.

This should be fixed in our next minor stable release (v0.93.1).


[Feature Request] Submitting an application should redirect to the submitted application's detail page

Instead of directing back to the "list" of applications, just go straight to the detail page. This will make it even more simple to use.

fixed in dc8117908cb465eb534075ba372e7a3881274539


set CWD on Command

Consider this scenario:
# script.sh
export ABC=2

# .zshrc
source script.sh

The ABC variable will not be set because the shell executes under a different folder (I think it's /Applications?).
However, if I have source /Users/myusername/script.sh it works.

So this PR basically sets Home as the CWD for the command. It works for me and I couldn't think of any negative aspects of doing this.

oh formatting is broken :/

fixed


feat: add static buffer as an alternative to allocating memory

Draft for static buffer as an alternative to allocating memory. Closes #878.
User can give fn() -> Option<&'static mut [u8]>, which is a factory for static buffers. I personally only need one, but there are problems if we want user to provide &'static mut [u8]. It is not Copy or Clone for obvious reasons, but Config is. It also cannot be wrapped in RefCell to be taken from config by shared reference, because there is a Sync assert.
In my code, this feature could used in 100% safe way:
fn static_buffer_factory() -> Option<&'static mut [u8]> {
    #[repr(align(32))]
    struct Memory([u8; 64 * 1024]);

    // Now the linker will refuse to create the final binary
    // if there is not enough space in the target RAM
    static WASMI_LINEAR_MEMORY: StaticCell<Memory> = StaticCell::new();

    // Will only succeed the first time. Uses atomic CAS operations under the hood
    WASMI_LINEAR_MEMORY
        .try_init_with(|| Memory([0; 64 * 1024]))
        .map(|x| &mut x.0[..])
}

let mut config = Config::default();
let config = config.set_static_buffer_factory(static_buffer_factory);
let engine = Engine::new(config);

As can be seen from godbolt, rustc optimizes out enum access in the hot methods data, data_mut and len, so there is no need for unsafe code and enum is sufficient.

Unresolved questions

Are you okay with that kind of design?
Should we strengthen/weaken the requirements? For example, add an alignment check?

fixes #878

@Ddystopia Thank you a lot for the PR!
I have to make sure (at least locally) that these changes indeed won't have an effect on the performance of Wasmi for inputs that should have a large impact on linear memory accesses.
I am not yet sold on the config addition but it seems to be necessary to allow for non-imported linear memory backed by static buffers. Is this a critical feature to you? Could we maybe separate this out to make this feature a bit smaller and only support imported linear memory for now?

Thank you for the response! I don't actually know, would imported memory be sufficient or not. Is there a way to compile Rust to wasm so that it would only import memory and not export any, do that wasmi would not create them on it's own?

Force-pushed to remove config changes. This is now only available for imported memories.

Force pushed to fix formatting and clippy jobs

I'm not sure what should I do with that clippy job - it would not compile if I remove those 2 imports. They're probably unused under some combination of features. Should I gate those imports?

I think it is a clippy or even Rust compiler bug. Nothing to do here for us at the moment.

Unfortunately, compiler did not manage to optimize that time when I've added a len to the static buffer, so I went with your initial idea to use vec's raw parts. There are also some tests that I tried with miri and they do pass.
Additionally rebased to main.

Okay, I'll implement it tomorrow as a separate commit.


Does anyone check if installation works regularly?

I mean does it hurt to update the requirements.txt with exact versions.. since you wont know when a package is updated and if its compatible with other versions.
Does anyone ever compile code to work with newer versions of Cuda, Python or VS?
Seems like stable diffusion and other tools similar to this finish something then just give up on maintaining.
C:\Program Files\NVIDIA GPU Computing Toolkit\CUDA\v12.1\include\crt/host_config.h(153): fatal error C1189: #error:  -- unsupported Microsoft Visual Studio version! Only the versions between 2017 and 2022 (inclusive) are supported! The nvcc flag '-allow-unsupported-compiler' can be used to override this version check; however, using an unsupported host compiler may cause compilation failure or incorrect run time execution. Use at your own risk.
texture.cu
ninja: build stopped: subcommand failed.
VS Build Tools 2022 17.12.2 / 14.42.34433
VS Professional 2022 14.29.30133
Desktop development with C++  / tried both MSVC v142/143
Cuda 12.1
Python 3.10
Windows 11 / Not using Conda
Nvidia RTX 4080
(venv) PS E:\InstantMesh-main> python run.py configs/instant-mesh-large.yaml examples/bird.jpg --save_video --no_rembg
Seed set to 42
Loading diffusion model ...
Loading pipeline components...:  12%|██████▌                                             | 1/8 [00:00<00:00,  7.82it/s]The config attributes {'dropout': 0.0, 'reverse_transformer_layers_per_block': None} were passed to UNet2DConditionModel, but are not expected and will be ignored. Please verify your config.json configuration file.
Loading pipeline components...: 100%|████████████████████████████████████████████████████| 8/8 [00:01<00:00,  6.42it/s]
Loading custom white-background unet ...
Loading reconstruction model ...
Some weights of ViTModel were not initialized from the model checkpoint at facebook/dino-vitb16 and are newly initialized: ['encoder.layer.7.adaLN_modulation.1.bias', 'encoder.layer.6.adaLN_modulation.1.weight', 'encoder.layer.4.adaLN_modulation.1.bias', 'encoder.layer.0.adaLN_modulation.1.weight', 'encoder.layer.6.adaLN_modulation.1.bias', 'encoder.layer.1.adaLN_modulation.1.bias', 'encoder.layer.10.adaLN_modulation.1.bias', 'encoder.layer.2.adaLN_modulation.1.weight', 'encoder.layer.7.adaLN_modulation.1.weight', 'encoder.layer.9.adaLN_modulation.1.weight', 'encoder.layer.3.adaLN_modulation.1.weight', 'encoder.layer.2.adaLN_modulation.1.bias', 'encoder.layer.8.adaLN_modulation.1.weight', 'encoder.layer.11.adaLN_modulation.1.bias', 'encoder.layer.8.adaLN_modulation.1.bias', 'encoder.layer.5.adaLN_modulation.1.bias', 'encoder.layer.3.adaLN_modulation.1.bias', 'encoder.layer.5.adaLN_modulation.1.weight', 'encoder.layer.9.adaLN_modulation.1.bias', 'encoder.layer.10.adaLN_modulation.1.weight', 'encoder.layer.4.adaLN_modulation.1.weight', 'encoder.layer.1.adaLN_modulation.1.weight', 'encoder.layer.11.adaLN_modulation.1.weight', 'encoder.layer.0.adaLN_modulation.1.bias']
You should probably TRAIN this model on a down-stream task to be able to use it for predictions and inference.
Traceback (most recent call last):
File "E:\InstantMesh-main\venv\lib\site-packages\torch\utils\cpp_extension.py", line 2100, in _run_ninja_build
subprocess.run(
File "C:\Program Files\Python310\lib\subprocess.py", line 524, in run
raise CalledProcessError(retcode, process.args,
subprocess.CalledProcessError: Command '['ninja', '-v']' returned non-zero exit status 1.
The above exception was the direct cause of the following exception:
Traceback (most recent call last):
File "E:\InstantMesh-main\run.py", line 132, in
model.init_flexicubes_geometry(device, fovy=30.0)
File "E:\InstantMesh-main\src\models\lrm_mesh.py", line 79, in init_flexicubes_geometry
renderer = NeuralRender(device, camera_model=camera)
File "E:\InstantMesh-main\src\models\geometry\render\neural_render.py", line 73, in init
self.ctx = dr.RasterizeCudaContext(device=device)
File "E:\InstantMesh-main\venv\lib\site-packages\nvdiffrast\torch\ops.py", line 184, in init
self.cpp_wrapper = _get_plugin().RasterizeCRStateWrapper(cuda_device_idx)
File "E:\InstantMesh-main\venv\lib\site-packages\nvdiffrast\torch\ops.py", line 125, in _get_plugin
torch.utils.cpp_extension.load(name=plugin_name, sources=source_paths, extra_cflags=common_opts+cc_opts, extra_cuda_cflags=common_opts+['-lineinfo'], extra_ldflags=ldflags, with_cuda=True, verbose=False)
File "E:\InstantMesh-main\venv\lib\site-packages\torch\utils\cpp_extension.py", line 1308, in load
return _jit_compile(
File "E:\InstantMesh-main\venv\lib\site-packages\torch\utils\cpp_extension.py", line 1710, in _jit_compile
_write_ninja_file_and_build_library(
File "E:\InstantMesh-main\venv\lib\site-packages\torch\utils\cpp_extension.py", line 1823, in _write_ninja_file_and_build_library
_run_ninja_build(
File "E:\InstantMesh-main\venv\lib\site-packages\torch\utils\cpp_extension.py", line 2116, in _run_ninja_build
raise RuntimeError(message) from e

I have uninstalled Python and moved to 3.10.6 and separately installed StableDiffusion and its working . I am getting a shyt ton of package dependancy conflicts and one that always pops up is huggingface_hub

Check https://github.com/TencentARC/InstantMesh/issues/175#issuecomment-2495631712
If it's a problem in a Dockerfile, which should be stable in theory, it's gonna be even a bigger problem in regular installs...
In the end I did run it via docker, but when generating a mesh it errored out without any message. Not sure where to go on after that :/

I tried on a different computer today, with just the bare minimum of 3.10, visual studio community 2022, build tools, cuda 12.1 etc and that is getting very similar errors to what I'm getting on this PC. I suspect mass package dependency version issues just don't know the right combo..


Implement go to definition for F#

Please provide a succinct description of the issue.
Repro steps
Currently "Go to definition" in F# only works within user-defined symbols, it's hard for users to quickly lookup library functions' usage.
I'm wondering now that the Intellisense in Visual Studio can generate F# lib functions' usage for user, why can't user use "Go to definition" or "Peek definition" to see usage in F# syntax from metadata just like what C# does.
Expected behavior
display usage in F# syntax from metadata just like what C# does.
Actual behavior
can not find the definition.
Known workarounds
vs
Related information
Provide any related information

Operating system Windows 10 17134.228
Branch all branches
.NET Runtime, CoreCLR or Mono Version .NET Core 2.1.400
Editing Tools (e.g. Visual Studio Version) vs 15.8
Links to F# RFCs or entries on https://github.com/fsharp/fslang-suggestions n/a
Links to performance testing scripts n/a
Indications of severity normal

Closing as duplicate of #1939
The main issue isn't in reading metadata, but writing a palatable representation to a temporary window. There is a closed trial PR that gets this mostly there, but it had too many quirks to accept at the time. I'm interested in trying to take it on for VS 2019, though.


Deep NSObject to NSDictionary

Hello, thanks for Motis. I use it in all my apps!
I've got a question though:
I noticed the method
- (NSDictionary*)mts_dictionaryWithValuesForKeys:(NSArray *)keys

doesn't do a 'deep' dictionary representation. What I mean is if you have an NSObject subclass such as Customer, and Customer has an array of Orders (Also an NSObject subclass). This method will return an array of Order instances as opposed to returning an array of Order dictionary representations.
Similarly, if Customer has an Address property (Also an NSObject subclass) this method will return an Address instance as opposed to a dictionary representation of that address instance.
Current output of:
[customer mts_dictionaryWithValuesForKeys:[[[customer class] mts_mapping] allKeys]];

is
{"name":"Curtis","address":"<Address mem_address>","orders":["<Order mem_address>","<Order mem_address>"]}

Desired output:
 {"name":"Curtis","address":{"street":"1111 Awesome Street","city":"You get the idea"},"orders": [{"id":1,"price":"5.00"},{"id":2,"price":"3.00"}]}

Basically, I'm looking to do the standard Motis flow, but backwards.
Is there a way to do this with the current motis version that I'm overlooking? Is it something you guys have though about including if not?

Thanks

Hello @dirtbikerdude91,
Thanks for using Motis! I'm very happy to know when someone is using it!
Regarding the - (NSDictionary*)mts_dictionaryWithValuesForKeys:(NSArray *)keys method, this has been added to Motis as a convenience method, but not to perform a fully object to json mapping (I call it a "Reverse Motis").
Therefore, to implement "Reverse Motis" it is required to specify how to serialize each property of an object to JSON and this is not yet supported and I cannot promise you it will be in the future, I still have to think about how to do it and if I want to invest the time to do it.
Consequently, you will have to use another library to do it or just do it manually. Also you can fork Motis and start working on it ;).
Thanks,


Dismiss custom sql banner on viz page via link in banner

Banner is only dismissed until a route change, page refresh
or sql update brings it back.
TODO: Dismiss persists through route/page changes?
@javisantana what would the best way (within the framework of whats here) to persist the notification remaining hidden across page refreshes and state changes, but not if the user updates the SQL for the layer or applies/updates the filters?

:+1: so far.

No idea what is the way we are doing, I guess with some cookie cc @xavijam

Great. Looks like we may not need to persist this notification staying dismissed, going to merge whats here for now and address that a bit later if necessary.

about this feature: what is the difference between dimiss and clear view?

This just hides the banner without clearing the view

what is the usecase?

@mbforr probably has a better idea of that than I do

That should satisfy that.
On Mon, Mar 14, 2016 at 10:09 AM, jbranigan<EMAIL_ADDRESS>wrote:

This just hides the banner without clearing the view
—
Reply to this email directly or view it on GitHub
https://github.com/cartodb-org/cartodb/pull/18#issuecomment-196323350.

--
Matt Forrest
Account Manager
CartoDB
201 Moore St, Brooklyn, NY, 11206, USA
T:<PHONE_NUMBER> Skype: mbforr
W: www.cartodb.com
Live sessions to learn CartoDB http://cartodb.com/webinars/


chore: replace function to const

replaced function to const in remix default template
Closes: #

[ ] Docs
[ ] Tests

Testing Strategy:

We're ok with function here


Serve assets with webpack

In branch webpack I will rework the Vue parts of the site to be served with webpack. Also the static assets should probably be packed with it.
Current state:

[x] reworked all the components
[x] reworked index.html, task.html and tira_admin.html

vue part is now in a separate file that gets packed and served by webpack
https://github.com/tira-io/tira/tree/webpack/application/src/tira/frontend/src

[ ]  pack css with js

[x] index
[ ] task
[ ] tira_admin

[ ] pack images with css
[x] pour into image
[x] test on test live domain

Test version is live at https://backup.tira.io.
Race conditions work nice all the Vue apps always load.
Bugs:

When an app was opened and then you go to the settings page its still in the DOM (at the bottom)

when the "Edit Organization" button is clicked a DOM node is created right before the end of the body for the modal (id="edit-organization-modal"). Then it is not removed when changing the page due to how Ember transitions work.


update & fix compilation issue by switching to nightly

Associated to discussion: https://github.com/async-graphql/async-graphql/discussions/1458

Feel free to close it as it seems we found a way for tin to compile 😄

Thank you! @Miaxos
However, I would like to use the stable  channel.


Invalid argument @ rb_sysopen when deploygin ovf files on Windows

While PR are not merged, I just wanted to document a problem on Windows and a possible fix.
After generating box in the ovf format, if you try to use them to create new vagrant boxes on a Windows machine, you will encounter the following error:
$ vagrant up
...
An unexpected error occurred when executing the action on the
'centos65-build-1' machine. Please report this as a bug:

Invalid argument @ rb_sysopen - file://C:/Users/mpereira/.vagrant.d/boxes/centos65-build/0/vmware_ovf/centos65-build.ovf

C:/HashiCorp/Vagrant/embedded/lib/ruby/2.2.0/open-uri.rb:36:in `initialize'
C:/HashiCorp/Vagrant/embedded/lib/ruby/2.2.0/open-uri.rb:36:in `open'
C:/HashiCorp/Vagrant/embedded/lib/ruby/2.2.0/open-uri.rb:36:in `open'
C:/Users/mpereira/.vagrant.d/gems/gems/vagrant-rbvmomi-1.8.1/lib/rbvmomi/utils/deploy.rb:118:in `upload_ovf_as_template'
C:/Users/mpereira/.vagrant.d/gems/gems/vagrant-vcenter-0.3.2/lib/vagrant-vcenter/action/inventory_check.rb:82:in `vcenter_upload_box'
C:/Users/mpereira/.vagrant.d/gems/gems/vagrant-vcenter-0.3.2/lib/vagrant-vcenter/action/inventory_check.rb:122:in `vcenter_check_inventory'
C:/Users/mpereira/.vagrant.d/gems/gems/vagrant-vcenter-0.3.2/lib/vagrant-vcenter/action/inventory_check.rb:21:in `call'
C:/HashiCorp/Vagrant/embedded/gems/gems/vagrant-1.8.1/lib/vagrant/action/warden.rb:34:in `call'
C:/HashiCorp/Vagrant/embedded/gems/gems/vagrant-1.8.1/lib/vagrant/action/warden.rb:95:in `block in finalize_action'
C:/HashiCorp/Vagrant/embedded/gems/gems/vagrant-1.8.1/lib/vagrant/action/warden.rb:34:in `call'
C:/HashiCorp/Vagrant/embedded/gems/gems/vagrant-1.8.1/lib/vagrant/action/warden.rb:34:in `call'
C:/HashiCorp/Vagrant/embedded/gems/gems/vagrant-1.8.1/lib/vagrant/action/builtin/handle_box.rb:56:in `call'
C:/HashiCorp/Vagrant/embedded/gems/gems/vagrant-1.8.1/lib/vagrant/action/warden.rb:34:in `call'
C:/HashiCorp/Vagrant/embedded/gems/gems/vagrant-1.8.1/lib/vagrant/action/warden.rb:95:in `block in finalize_action'
C:/HashiCorp/Vagrant/embedded/gems/gems/vagrant-1.8.1/lib/vagrant/action/warden.rb:34:in `call'
C:/HashiCorp/Vagrant/embedded/gems/gems/vagrant-1.8.1/lib/vagrant/action/warden.rb:34:in `call'
C:/HashiCorp/Vagrant/embedded/gems/gems/vagrant-1.8.1/lib/vagrant/action/builder.rb:116:in `call'
C:/HashiCorp/Vagrant/embedded/gems/gems/vagrant-1.8.1/lib/vagrant/action/runner.rb:66:in `block in run'
C:/HashiCorp/Vagrant/embedded/gems/gems/vagrant-1.8.1/lib/vagrant/util/busy.rb:19:in `busy'
C:/HashiCorp/Vagrant/embedded/gems/gems/vagrant-1.8.1/lib/vagrant/action/runner.rb:66:in `run'
C:/HashiCorp/Vagrant/embedded/gems/gems/vagrant-1.8.1/lib/vagrant/action/builtin/call.rb:53:in `call'
C:/HashiCorp/Vagrant/embedded/gems/gems/vagrant-1.8.1/lib/vagrant/action/warden.rb:34:in `call'
C:/Users/mpereira/.vagrant.d/gems/gems/vagrant-vcenter-0.3.2/lib/vagrant-vcenter/action/connect_vcenter.rb:32:in `call'
C:/HashiCorp/Vagrant/embedded/gems/gems/vagrant-1.8.1/lib/vagrant/action/warden.rb:34:in `call'
C:/HashiCorp/Vagrant/embedded/gems/gems/vagrant-1.8.1/lib/vagrant/action/builtin/config_validate.rb:25:in `call'
C:/HashiCorp/Vagrant/embedded/gems/gems/vagrant-1.8.1/lib/vagrant/action/warden.rb:34:in `call'
C:/HashiCorp/Vagrant/embedded/gems/gems/vagrant-1.8.1/lib/vagrant/action/builder.rb:116:in `call'
C:/HashiCorp/Vagrant/embedded/gems/gems/vagrant-1.8.1/lib/vagrant/action/runner.rb:66:in `block in run'
C:/HashiCorp/Vagrant/embedded/gems/gems/vagrant-1.8.1/lib/vagrant/util/busy.rb:19:in `busy'
C:/HashiCorp/Vagrant/embedded/gems/gems/vagrant-1.8.1/lib/vagrant/action/runner.rb:66:in `run'
C:/HashiCorp/Vagrant/embedded/gems/gems/vagrant-1.8.1/lib/vagrant/machine.rb:224:in `action_raw'
C:/HashiCorp/Vagrant/embedded/gems/gems/vagrant-1.8.1/lib/vagrant/machine.rb:199:in `block in action'
C:/HashiCorp/Vagrant/embedded/gems/gems/vagrant-1.8.1/lib/vagrant/environment.rb:561:in `lock'
C:/HashiCorp/Vagrant/embedded/gems/gems/vagrant-1.8.1/lib/vagrant/machine.rb:185:in `call'
C:/HashiCorp/Vagrant/embedded/gems/gems/vagrant-1.8.1/lib/vagrant/machine.rb:185:in `action'
C:/HashiCorp/Vagrant/embedded/gems/gems/vagrant-1.8.1/lib/vagrant/batch_action.rb:82:in `block (2 levels) in run'

This is due to a bug in lib/rbvmomi/utils/deploy.rb which treats windows path names as URI and discards the "host" part which removes the root driver letter from the file path.
I fixed this problem in my fork with this commit: f4f397f762738f22c277c28a417013880f96b1f8
But as it requires the os gem, I'm waiting on the PR #33 to be resolved before creating a proper PR. For the moment, you need to add the os gem yourself with:
$ vagrant plugin install os --plugin-version 0.9.6

CC @frapposelli as the bug was introduced in 69a888f93d9bad9f0a2de295bc9dc9e06aff38f0

This is not a rbvmomi bug but a vagrant-vcenter bug, please file it there.
Rbvmomi version used in vagrant-vcenter is a fork of the main rbvmomi.

@frapposelli do you mean https://github.com/frapposelli/vagrant-rbvmomi? I will create a pull request there tonight.


Fix silly bugs in custom_setings_value_wait

Critical Changes
Changes

Fix the custom_settings_value_wait task to not throw an exception if a custom settings record doesn't yet exist. Instead it'll continue to query the object until the record exists and the value matches
Fix the custom_settings_value_wait task to not throw an exception if the field api name passed as an option doesn't exactly match the case of the field api name in the object.

Issues Closed

Thanks for the review @davisagli. I made a couple of updates per your review.

I pushed a change to simplify error handling, removing a condition that could not occur.
I'll approve this too but I think the existing task (not these changes; thank you, @force2b) needs more work - it has no timeout so can spin a worker until it dies looking for a custom setting, and it does not currently have support for automatic injection. I'll add a WI for those changes.

Thanks @davidmreed. I wrote the original task, so I'll be keeping an eye out for changes to learn how to do a better job building these in the future.

@force2b We changed CumulusCI underneath the task. We just need to update it to keep up with the times! The move to automatic inference for namespace injection was required to make 2GP work.

@davidmreed I added automatic namespace injection and filled in test coverage; could you give this one more look?


Update scala-jsonschema, ... to 0.7.8

Updates

com.github.andyglow:scala-jsonschema
com.github.andyglow:scala-jsonschema-enumeratum

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

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

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

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

Superseded by #19.


Updating an axis from horizontal to vertical breaks tick alignment

JSFiddle: https://jsfiddle.net/u6mLuxpq/
I've noticed that if I take an existing axis that is horizontal (say axisBottom) and then re-call it with axisRight or axisLeft the tick marks all end up on an angle. For right now, my fix is to call select('g.ticks').remove() but I'm assuming that this isn't the ideal solution.

Changing the orientation of an axis is not currently supported. The recommended way to handle orientation changes is to remove the entire axis (the parent G element) and then add a new axis.

@mbostock ok good to know. That's what I'm doing now. I do realize it is an unusual situation and only comes up in a demonstration mode I have. Figured I'd mention it just in case it wasn't known.

If you look closely in the README, it says:

The orientation of an axis is fixed; to change the orientation, remove the old axis and create a new axis.

😉


Package with tsup

This PR moves connect-query to tsup.
Closes https://github.com/bufbuild/connect-query/issues/17.

validated with connect-query 0.2.0-alpha.5 on buf.build's repo

@mckelveygreg @sher @nickadamson @getinnocuous are you able to try out 0.2.0-alpha.5<EMAIL_ADDRESS>I think we should be good across the board now.

@mckelveygreg @sher @nickadamson @getinnocuous are you able to try out 0.2.0-alpha.5<EMAIL_ADDRESS>I think we should be good across the board now.

Thank you, will try it out asap and get back to you.

@dimitropoulos Confirmed, this version works as expected 👍. In our case a) with nextjs, no pre-transpilation is required and b) default transport is picked up implicitly from the TransportProvider.


add numpy function support in Groupby.agg

Everything is OK except the return type infer. @HyukjinKwon Can help me fix it ?

gentle ping @RainFung, are you still working on this?

Sorry, i'm busy recently. I'll work it in this weekend.

Thanks @RainFung!

@RainFung Do you want to continue working on this?
If you don't have time, shall we close this for now and someone else might want to work on this.
Thanks!

I can back to it.

Im back to it.

@HyukjinKwon Sorry, Sir. There are too many problems when i solve this PR. I'm afraid I can't finish it.Can you complete this PR?

Yeah, that's fine. I will take this over when I find some time. Let's leave this closed for now.


npm ERR! Could not resolve dependency: npm ERR! peer<EMAIL_ADDRESS>from<EMAIL_ADDRESS>

🐞 Bug report
Command (mark with an x)

[ x] new
[ ] build
[ ] serve
[ ] test
[ ] e2e
[ ] generate
[ ] add
[ ] update
[ ] lint
[ ] extract-i18n
[ ] run
[ ] config
[ ] help
[ ] version
[ ] doc

| Installing packages (npm)...npm ERR! code ERESOLVE
npm ERR! ERESOLVE unable to resolve dependency tree
npm ERR!
npm ERR! While resolving<EMAIL_ADDRESS>npm ERR! Found<EMAIL_ADDRESS>npm ERR! node_modules/jasmine-core
npm ERR!   dev<EMAIL_ADDRESS>from the root project
npm ERR!
npm ERR! Could not resolve dependency:
npm ERR! peer<EMAIL_ADDRESS>from<EMAIL_ADDRESS>npm ERR! node_modules/karma-jasmine-html-reporter
npm ERR!   dev<EMAIL_ADDRESS>from the root project
npm ERR!
npm ERR! Fix the upstream dependency conflict, or retry
npm ERR! this command with --force, or --legacy-peer-deps
npm ERR! to accept an incorrect (and potentially broken) dependency resolution.
npm ERR!
npm ERR! See C:\Users\ads\AppData\Local\npm-cache\eresolve-report.txt for a full report.

I get the following error when I try to create a new project. I've updated ng cli after reading this issue, but I'm still getting the error.

Package                      Version
@angular-devkit/architect    0.1201.1 (cli-only)
@angular-devkit/core         12.1.1 (cli-only)
@angular-devkit/schematics   12.1.1 (cli-only)
@schematics/angular          12.1.1 (cli-only)

Looks like the issue is caused by a karma-jasmine-html-reporter release, were they dropped support for jasmine-core version 3.7 in a minor version.
Removing the lower bounds of a peerDependency should be considered a breaking changed due to NPM 7 new peer dependencies behaviour.
As a workaround you can do the following;
ng new --skip-install
Open package.json and bump jasmine-core to `~3.8.0`
npm i

Thank you.


Add 'Email' in profile manager when creating account.

Description

The user will also save its email and should be saved in the metadata.

WIP:
Email added in creating account tab and in profile manager but still investigating to get the value of email

WIP:
Email is working fine in the profile manager, now working on adding email in profile page

Work Completed:

Email saved in metadata and added to (connect your wallet, your profile, and TFChain Wallet)
it can be added/edited in your profile page

verified on devnet
version : ac5cd85
adding email and editing it is now available when creating an account and also in my profile page you can edit or add an email.

https://app.testlodge.com/a/26076/projects/40893/suites/234374?expand_section=422331#case_4150327
https://app.testlodge.com/a/26076/projects/40893/suites/234374?expand_section=422331#case_4246385

When validating a mnemonic or creating a new account, all the inputs are disabled except for the email.

Done on #2506

Verified, Devnet f3e199e.

https://github.com/threefoldtech/tfgrid-sdk-ts/issues/2066#issuecomment-2016723920
And I found an issue reported here:https://github.com/threefoldtech/tfgrid-sdk-ts/issues/2555.


mention pip3

It's probably an elementary thing but I couldn't install this application because my $ python was MacOS Python, which is v2.x and this application requires Python v3.x. Now, pip is v2.x Python application; to install using Python v.3.x you must call pip3.

Definitely a good think to mention at least until 2020 ;-)

Thank you!

thanks! Having said that, it took me a while to install this, and this instruction should be properly supplemented with thorough writeup. Basically, mention peeps should install latest Python, if unexperienced, peeps should not bother with vm's, but ensure that the secret path where particular Python installs things in in the $PATH. In my case, it was not in the $PATH and I wasted time blaming Python, pip and multiple flavours of Python existing on the system (all false leads). It was just $PATH issue.

Which version of Python did you use? Jinjalint is tested with Python 3.5 which is kinda old these days (I have 3.6 even on my OpenBSD box).
The transition from Python 2.x to 3.x has been somewhat complicated, usually people have both versions installed on their machines (with funky secret paths). I use Python occasionally for years and I have to admit I still struggle against this sometimes.

3.7.
In the end I installed the downloadable image from Python's website, then using python3.7 -m pip install jinjalint. This installed jinjalint into /Library/Frameworks/Python.framework/Versions/3.7/bin which was not in the $PATH which luckily I knew how to manage/set (and knew that Path was at the first place). For posterity, since I'm in ZSH shell, it was just a matter of updating .zshconfig, there's an export call somewhere in the middle of the file which exports the paths. It's a string where each path is joined with :.


Bump Dependencies #6930436854

Bumps dependencies for product-is. Link : https://github.com/wso2/product-is/actions/runs/6930436854

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.jenkins-is-staging seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


refactor: Add UPX and switch runtime to distroless

Description
This PR helps to reduce image size from 247MB to 52MB
Checklist

[x] Have you tested your changes?
[ ] Does this PR include changes that require our documentation to be updated, if so, have you created a PR on the docs repo to make the necessary changes?

hey @stake-machine welcome!!! would you be able to update the commit message to follow convention semantics? Something like refactor: Add UPX and switch runtime to distroless?

Hey Hope! Sure, done!

@chriswessels is that ok now?

Looks like bullseye upx is pretty old, so I've added additional stage for upx based on alpine. Good you please try it?


Follow MS guidelines for custom exception types.

Reference Issue
No existing recorded issue.
What does this implement/fix? Explain your changes.
If using this SVG library within a .Net framework AppDomain, exceptions must be serializable to cross AppDomain boundaries.
In addition, implement common exception constructors, in line with guidance at https://docs.microsoft.com/en-us/dotnet/standard/exceptions/best-practices-for-exceptions
WARNING: potentially breaking change.
I have also made the exception classes sealed. But this may be a breaking change if any clients have created exceptions derived from these classes. I can revert that if you think it will break compatibility for existing clients.

@paulushub - I've removed sealed and added release notes.

@mrbean-bremen Thank you, the private constructors must be revert back to protected

Thank you, the private constructors must revert back to protected

Ah right, thanks - missed this. Fixed now.

Ah right, thanks - missed this. Fixed now.

You are on fire! you are sure you do not want to also revert the decision on contribution? 😄

you are sure you do not want to also revert the decision on contribution?

Quite sure 😁
Still can do trivial stuff like this, of course...


Failed to update a ServiceWorker

Library Affected:
workbox
TypeError: Failed to update a ServiceWorker: An unknown error occurred when fetching the script
Browser & Platform:
Chrome Mobile 73.0.3683 on Android9
Samsung Internet 9.2.0 on Android 8.1
Issue or Feature Request Description:
Is it possible to ignore network errors on serviceworker updates?
When reporting bugs, please include relevant JavaScript Console logs and links to public URLs at which the issue could be reproduced.

Sorry, I have to close this as there's not enough information to take any action. There are lots of reasons an error might occur when fetching the script (network errors, scripts errors, etc). If you can provide more information or link to a page showing the issue, then we can try to help.


tslint mdx files

Question
Anyone have success using tslint with docz?
Description
I've seen this issue but does anyone have a complete example?
Thanks a lot!
Give more information about your doubt

Twitter: If it's just a quick question you can ping me on Twitter: https://twitter.com/pedronauck

The OP of that issue sounds like he wanted his editor to flag the type errors which is currently a limitation of mdx file. See mdx/Intellisense / Typescript
Also since tslint is slowly going away should look for eslint solutions. Somthing like eslint-plugin-mdx.
I havent used any custom plugins of config options yets for docz so i cant help ya on the modifyBundlerConfig example.
But a quick github search I found this, I would believe to a good base to go off of.


Send an email to existing, active Users that are added to an Organization

Currently, pre-existing and active users (i.e. Users not created via an invitation to an organization) receive no notification that they have been added to an organization. This PR adds the send_notification function to the InvitationBackend to send an email informing these users that they have been added to an organization and providing them with a link they can follow to view this organization.
This PR implements #114.

Thanks! Have a couple of small changes and then this should be ready!

Thanks for the feedback! I'll work on implementing these changes early next week.


Use token from LDAP instead of Token Role

Hi
When I tried to use token that generate from other Authentication type like LDAP and set token to configmap.
I've always face error about token role from line of code:
https://github.com/Boostport/kubernetes-vault/blob/master/cmd/controller/client/vault.go#L261
So my questions is

Is it always need to use token role only for Kubernetes-Vault?
Could we add parameter to skip above role name checking?

Best regards,
Monchai

Hey Monchai,
The reason we expected a role name is because the controller needs a periodic token, so that it can renew the token. Periodic tokens can be created using token roles and some a few auth methods: https://www.vaultproject.io/docs/concepts/tokens.html#periodic-tokens
Does the LDAP auth method support the creation of periodic token? If so, I think it should be possible to modify the code to not check the role name in certain cases.

Hi @F21
Yes I think LDAP authentication support the creation of periodic token.
Below is result when I execute command vault token lookup
Key                  Value
---                  -----
accessor             d5c7c2atAsy1iXdjXCTIAiqq
creation_time<PHONE_NUMBER>
creation_ttl         24h
display_name         ldap-username
entity_id            3273bd33-4a70-0ee7-8134-1dde32f142dd
expire_time          2019-10-21T04:20:57.973333833+07:00
explicit_max_ttl     0s
id                   s.APXKt43sA5nvIzNnDa3fqvdO
issue_time           2019-09-20T10:52:46.899753954+07:00
last_renewal         2019-09-30T08:20:57.973334075+07:00
last_renewal_time<PHONE_NUMBER>
meta                 map[username:username]
num_uses             0
orphan               true
path                 auth/ldap/login/username
policies             [default]
renewable            true
ttl                  499h59m25s
type                 service

Could I modify your code and create PR for this changes.
BR,
Monchai

Hi @monchaio ,
Sure, please go ahead and create a PR. Maybe call the parameter skip_token_role_name_validation. It needs to be documented so that users can understand they should only use this if they really need to.
I was looking at the Vault API, hoping there would be a generalized method to check if a token is periodic, but I think we will need to implement this check for every auth backend. For example, for the token backend, it's part of a token role, for LDAP, it's part of the config, so it would be virtually impossible to implement, given that it's not possible to tell which auth backend created the token.

Hi @F21
Will create PR for this issue. Thanks so much for answer my question
Best regards
Monchai

Hi @F21
Sorry for late reply I just created PR https://github.com/Boostport/kubernetes-vault/pull/157
To add new parameter skipTokenRoleNameValidation.
Could you please kindly review?
Best regards,
Monchai

Fixed in #157


No single container in the whole website

Bug Description
In our website, Header should be in the container. It's just aligned in the center.
Issue
When we zoom out the whole header is just aligned in left.
That's really bad for any website. So Header should be in fix size center aligned container, so if anyone just zoom out then as well our header will also be in container only.
This is our website when zooming out, it's just floated on entire screen

This is the perfect containerized website example. Even if we zoon

It would be great if we can make our whole website in perfect fix size container,
@tobySolutions You can assign this task to me, would be happy to do :)

Wow!! This is an amazing issue @MadhuSaini22. I love the fact that you raised this responsiveness issue. Sure, go ahead, I'd love to see this issue fixed!!!!

How is this going @MadhuSaini22?? :grin:

How is this going @MadhuSaini22?? 😁

Going good Tobi!!
I was busy yesterday, so will make PR now :)


Button readonly issues and adjustments

Closes https://github.com/gisce/webclient/issues/612

:tada: This PR is included in version 2.0.0-alpha.58 :tada:
The release is available on:

npm package (@alpha dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Event Insider wants to add songs

*For showing specific youtube embed links: https://www.w3.org/blog/2008/09/howto-insert-youtube-video/


Closes #30 - Modified load to insert data into Fluo as data is parsed

Refactored webindex script
Created loadurls command to load data from AWS url
Renamed load command to loaddir

+1


Check SSL certificate of asset websites (a.k.a proof-of-authenticity)

Requested by several users, and generally a good idea.
http://blog.coinprism.com/2014/09/10/proof-of-authenticity-of-cryptoassets/

---
Want to back this issue? **[Post a bounty on it!](https://www.bountysource.com/issues/6598617-check-ssl-certificate-of-asset-websites-a-k-a-proof-of-authenticity?utm_campaign=plugin&utm_content=tracker%2F686853&utm_medium=issues&utm_source=github)** We accept bounties via [Bountysource](https://www.bountysource.com/?utm_campaign=plugin&utm_content=tracker%2F686853&utm_medium=issues&utm_source=github).

You could say we already do this: if you specify https://, then HTTPS will be used. The site that displays this output from Counterblock API can validate the certificate on its own. If the site refuses self-signed or "invalid" certificates, then it can not display the contents, or display asset details so that it's clear the certificate didn't validate.
We don't validate certificates, but we could.

Change documentation to notify asset owners that https:// will be validated
Change HTTPS verification to True in: https://github.com/CounterpartyXCP/counterblock/blob/master/counterblock/lib/util.py#L252


No license listed

Under what licensing terms are the source files here available under? I can't find a LICENSE or COPYING file anywhere.

Thanks for pointing it out! vim-minimap is released under a MIT license. I've added the corresponding LICENSE file.


Event Registrations from Events Page

As O'Reilly Organizer, on a desktop, I'd like a clear path toward seeing registrants from the Events page. I need a clear UX for going Events > Clicking My Event > knowing where to look next to get to registrants. (Edit event, save is not intuitive.)
Two thoughts:

Move away from two-column view and keep things simple with an event details page navigated to from event listings every time (hello, again, my old friend ;)
If in grabbing the top-level events listing we know whether a user is organizer / want to check permissions, perhaps on the details sidebar we can show a button to view details that takes you to the details component. Since this widget is shared, I believe, it complicates the widget based on the context of where it is being shown.

Agree @KrisJordan - I think in general, events will need to be redone at some point. I will make another ticket mentioning this and other events changes, then close this!

@KrisJordan update - changed the name, will keep this ticket


Exception when trying to insert my json object inside json file

Description
I'm trying to append my json object to inside a json file.
Reproduction steps
Bug occurs when trying to .insert()
Expected vs. actual results
This is how my json file looks like
{     "Base3": [         {             "Name": "Base3",             "X": 69,             "Y": 999,             "Z": 999         }     ],     "Base4": [         {             "Name": "Base4",             "X": 69,             "Y": 999,             "Z": 999         }     ],     "Base5": [         {             "Name": "Base5",             "X": 69,             "Y": 999,             "Z": 999         }     ] }
What I expected ;
{     "Base3": [         {             "Name": "Base3",             "X": 69,             "Y": 999,             "Z": 999         }     ],     "Base4": [         {             "Name": "Base4",             "X": 69,             "Y": 999,             "Z": 999         }     ],     "Base5": [         {             "Name": "Base5",             "X": 69,             "Y": 999,             "Z": 999         }     ],     "Base6": [         {             "Name": "Base6",             "X": 22,             "Y": 33,             "Z": 11         }     ] }
Minimal code example
using Json = nlohmann::json;
int main()
{

	Json Data;
	std::string Name = "Base6";
	float xx = 22;
	float yy = 33;
	float zz = 11;

	Data[Name]["Name"] = Name;
	Data[Name]["X"] = xx;
	Data[Name]["Y"] = yy;
	Data[Name]["Z"] = zz;

	std::fstream filewrite;
	filewrite.open("coords.json", std::ios::out | std::ios::in);

	Json existingjson = Json::parse(filewrite);
	filewrite.seekg(0, filewrite.beg);
	existingjson.insert(existingjson.begin(), Data);
	filewrite << existingjson;

}

Error messages
Unhandled exception at 0x00007FF8101F565C in JSON.exe: Microsoft C++ exception: nlohmann::json_abi_v3_11_2::detail::type_error at memory location 0x000000000014FC80.

Compiler and operating system
MSVC 14.38 / WINDOWS11
Library version
3.11.2
Validation

[ ] The bug also occurs if the latest version from the develop branch is used.
[ ] I can successfully compile and run the unit tests.

json::insert() only operates on arrays, not objects in the form you used.  (Form (1) at the API documentation.).  Only Form (5) of insert() is for inserting into objects.
You should be using json::update().

The type_error exception should provide more information in the what string. If you print it, it should probably indicate what @nomadwolf0 wrote.

I tried to reproduce the error and it is showing a good exception actually.

@marfixdev Can I close this issue?


feat: add settings-menu

[x] keyboard control requires fine tuning. focus to expanded submenu when hidden to shown settings menu

test URL: https://media-chrome-git-fork-luwes-settings-menu-mux.vercel.app/examples/vanilla/control-elements/media-settings-menu.html

this change adds new settings-menu and settings-menu-item elements and logic to handle submenus with customizable headers, back button, chevron icons, etc...
the case of a settings-menu in another settings-menu is not yet tried or implemented, it's a more rare case I'd like to add later.
the settings-menu-item has a description slot which automatically gets populated with the submenu checked item text.
it's possible for the developer to opt-out by setting a slotted description element.
all icons can be customized with slots, the headers of the menu can be customized with a slot or just the title keeping the back button intact.

If you open a sub-menu and then resize the player the sub menu bleeds out of the player

If you open a sub-menu and then resize the player the sub menu bleeds out of the player

this one should be fixed


Cannot publish page - Integrity error - UNIQUE constraint failed

See Gist for full traceback:
https://gist.github.com/petrklus/8f31dfe0e790b99a47a9

@petrklus
Hey, are you able to consistently reproduce this still?
Please provide some info:
Does the page being published have a parent?
Is this the first time this page is published?
Do you know if anyone else was making changes to this or other pages around the time you published?

This should now be fixed on 3.3
Please check it out and run cms fix-tree


[Bug]: Incompatible with opendk@21

Platform
All
Plugin
connectitvity_plus
Version
connectivity_plus: ^6.1.0
Flutter SDK
3.24
Steps to reproduce
1 2 3
Code Sample
doesn't work

Logs
AILURE: Build failed with an exception.
[        ] * What went wrong:
[        ] Execution failed for task ':connectivity_plus:compileReleaseJavaWithJavac'.
[        ] > Could not resolve all files for configuration ':connectivity_plus:androidJdkImage'.
[        ]    > Failed to transform core-for-system-modules.jar to match attributes {artifactType=_internal_android_jdk_image, org.gradle.libraryelements=jar, org.gradle.usage=java-runtime}.
[        ]       > Execution failed for JdkImageTransform: /Users/Patrick.Lemiuex/Library/Android/sdk/platforms/android-34/core-for-system-modules.jar.
[        ]          > Error while executing process<EMAIL_ADDRESS>with arguments {--module-path
/Users/Patrick.Lemiuex/.gradle/caches/transforms-3/07f95f853439878af0ee0099c399c2fd/transformed/output/temp/jmod --add-modules java.base --output /Users/Patrick.Lemiuex/.gradle/caches/transforms-3/07f95f853439878af0ee0099c399c2fd/transformed/output/jdkImage
--disable-plugin system-modules}
[        ] * Try:
[        ] > Run with --debug option to get more log output.
[        ] > Run with --scan to get full insights.
[        ] * Exception is:
[        ] org.gradle.api.tasks.TaskExecutionException: Execution failed for task ':connectivity_plus:compileReleaseJavaWithJavac'.

Flutter Doctor
[✓] Flutter (Channel stable, 3.24.3, on macOS 15.1.1 24B91 darwin-arm64, locale en-US)
[✓] Android toolchain - develop for Android devices (Android SDK version 34.0.0)
[✓] Xcode - develop for iOS and macOS (Xcode 16.1)
[✓] Chrome - develop for the web
[✓] Android Studio (version 2024.2)
[✓] VS Code (version 1.95.3)
[✓] VS Code (version 1.95.2)
[✓] Connected device (4 available)
    ! Error: Browsing on the local area network for Patrick’s iPhone. Ensure the device is unlocked and attached with a cable or associated with the same local area network as this Mac.
      The device must be opted into Developer Mode to connect wirelessly. (code -27)
[✓] Network resources

Checklist before submitting a bug

[X] I searched issues in this repository and couldn't find such bug/problem
[X] I Google'd a solution and I couldn't find it
[X] I searched on StackOverflow for a solution and I couldn't find it
[X] I read the README.md file of the plugin
[X] I'm using the latest version of the plugin
[X] All dependencies are up to date with flutter pub upgrade
[X] I did a flutter clean
[X] I tried running the example project

This a Flutter issue, not a plus plugins issue.
Upgrade your gradle files as described in #3303


Expandable Cards in Swiper F7 v4

Framework7 version: 4.0.1
https://jsfiddle.net/cp027hua/8/

Describe the bug
The expandable cards do not work properly in a Swiper slide

It can't work in Swiper and other types of content with overflow: hidden


Table reservation Management - Page (tab)

Implement Page with:

UI for choosing tables
options for accepting/cancelling/editing next to it

restricted only for logged-in Manager (not for Client)
use design from: #51

postponed to Sprint 4