Orchard 1.10 / 1.9.3 Media 'ImageEditor': Image not shown

couple weeks ago I opened #6285 on a 'similar' issue. Sebastien recommended to use 1.10 / 1.9.3. from github.
I tried  with both Github versions. On both I see the ImageEditor button
BUT: when clicking, the Image does NOT appear ; there is only the little 'cross' shown!
I loaded some images via Media->Import; they appear correct when clicking the appropriate folder! The 'Edit' function works correctly!
PS: I did NOT load any additional Modules etc. and I realize that 'Media' and 'MediaPicker' is labled 'depreciated': I explicitely disabled them!

Thanks for your time and hints, ed

Can you look at the network trace on chrome to see what is the url it can't load?

Below info is from Edge (I certainly could test with chrome).  Thanks for your time ... ed
After Media --> Pictures(my folder) the HTML source:

After clicking the image: HTML source

The Edge F12 --> Network trace:
Name	Protocol	Method	Result	Content type	Received	Time	Initiator
http://localhost:63053/Admin/Orchard.ImageEditor/Index/19?returnUrl=http%3A%2F%2Flocalhost%3A63053%2FAdmin%2FOrchard.MediaLibrary%3FfolderPath%3Dpictures

Does the /Media/Default/pictures/flugzeug.JPG image exist ?

Or even better, are you able to provide some repro steps from a vanilla download of 1.9.3/1.10, meaning, clone to a new folder and stat from a clean setup ?

After Media --> Pictures(my folder) I see my image. When clicking the image, on the right side (PROPERTIES) I see my Image, the ' Edit' AND the 'Image Editor' button etc. So the image is there.
In fact I started with a fresh copy 1.9.3 AND 1.10. Well, to confirm I do it again on a freshly downloaded 1.9.3 from GitHub. Then I only 'Import' one picture in one newly created media folder! ... I let you know.

my test sequence on VS2015:

freshly downloaded 1.9.3 from GitHub
enabled the necessary modules
created one Media folder (pictures) and imported 4 images
the 'Image Editor' shows my image; I can eg resize -> all ok!
Now I want to create my Web version:
using 'publish' in VS2015 --> runs without any errors!
in WebMatrix it loads ok
however, the 'Image Editor' doesn't work --> I still get the little cross
PS: on my tests before I always had some user modules loaded like Bing Maps, facebook, Vandelay Industries etc. Sorry, that might have messed up the tests.
When checking the Media folders I can see the images in: Media --> pictures, Media Storage and in the profile!
your time hints are appreciated, ed

Works fine for me in 1.9.3 and 1.10. I am using Orchard.Web.zip not source.

Did you upgrade infoset's and migrate images with Orchard Upgrade?

for some other reason I downloaded a new 1.10 from GitHub, interesting:
On VS2015
importing couple pictures --> 'Image Editor': OK
after publishing as Orchard.Web (WebMartrix) -- > loading some pictures --> 'Image Editor' : won't work
donaldboulton: I dont't think I need to care about 'infoset', do I?  I'm doing this with a fresh 1.10 !

@edikaufmann what error(s), if any, do you get?  Anything in the developer console?

In VS2015:
remember here 'Image Editor' works; kind of!
e.g. when clicking filters and set 'brightness from 0 (default) to eg 1 --> the picture is totally greyed out!
e.g. 'preset' doesn't show anything!
but
resize, rotate works
In WebMatrix:
I do Not get any errors (not in console nor App_data Log)  --> i.e. only the little cross!.
PS: to me the HTML code looks alike! !the 1.10 download I did on Feb 24 !

Is the Base URL of the site settings the same as the actual URL in the browser?

Sipke, thanks a million! My fault; due to a 'hack' on the browser (not related to orchard !) under very special circumstances I changed the browser URL !
I apologize to everybody who spent time on this, ed


DropdownMenu.Root open property is not reactive and won't close the menu

Describe the bug
If I bind open, and set it to false it does not close the menu. It should close the menu, but currently it does nothing.
Reproduction
Set up a menu, bind to open, set the variable to false. Observe menu does not close.
Logs
No response
System Info
.

Severity
annoyance

This was invalid, it works.


Custom Close Lid functionality when external displays are connected

Description of the new feature / enhancement
Allow users to customize their laptop's close lid behavior through the awake module based on whether or not they are connected to an external display.
Perhaps even charging status dependent 🤷.
Scenario when this would be used?
Many people these days use their laptops with secondary displays, often when i am using multiple screens, i will want to close my laptops lid, if i want to game, and dont want distractions, or clear up some desk space.
I am hesitant about changing the power close lid setting, because i might forget about it, and my laptop might overheat in my bag, as it very often does for  lot of Windows users
Supporting information
No response

Thank you for the suggestion! We'll investigate and set prioritization following our general stability efforts.

Closing this as wont-fix because state management to this level is out-of-scope for Awake. My goal with the utility is to make sure that it works reasonably well for most cases while more personalized scenarios (lid behavior customization is one of them) remains something that the user can set up through their power plan. The last thing I want to do is re-implement the Windows power management features, but in Awake 😊


YQL-17502 Date32 from/to string conversion

@amikish you broke some tests. Please fix them or revert changes.


Use conversations instead of channels due to deprecation

channels.x operations have been deprecated in favour of conversations.x and do not work at all since June 10th, 2020 (https://api.slack.com/methods/channels.list)

Oops - I only just saw this. We've just merged #220 which does the same thing - I really appreciate you taking the time to put this together though 🙇


vSwitch and SecurityGroup query refactor

With the work going into #11, we wanted to tidy up the querying methods to be similar to other providers as well as prepare the API to be moved to openshift/api.
This pull request addresses the feedback given in #11 and I will work on a corresponding PR to the installer.

@JoelSpeed I have some questions about testing this. Due to our renaming in the go.mod to module github.com/openshift/... the installer repository does not like the replace directive as we are renaming. I can get around this but I have to refactor code to make the libraries match which backtracks a bit of the work in this PR. If you have a few minutes to spare tomorrow that would be great.

@JoelSpeed @elmiko This PR addresses the feedback left for #11. Please review and let me know if there is anything you would like me to fix.
I was able to test this code and the machinesets successfully created the nodes.
NAME                                 STATUS   ROLES    AGE     VERSION
test-6m9rh-master-0                  Ready	master   137m    v1.22.1+f773b8b
test-6m9rh-master-1                  Ready	master   136m    v1.22.1+f773b8b
test-6m9rh-master-2                  Ready	master   138m    v1.22.1+f773b8b
test-6m9rh-worker-us-east-1a-v2wn5   Ready	worker   28m     v1.22.1+f773b8b
test-6m9rh-worker-us-east-1b-8dlqd   Ready    worker   81s     v1.22.1+f773b8b
test-6m9rh-worker-us-east-1b-xwzhd   Ready    worker   3m40s   v1.22.1+f773b8b
NAME                                 PHASE     TYPE            REGION	   ZONE         AGE
test-6m9rh-master-0                  Running   ecs.g6.xlarge   us-east-1   us-east-1b   139m
test-6m9rh-master-1                  Running   ecs.g6.xlarge   us-east-1   us-east-1a   139m
test-6m9rh-master-2                  Running   ecs.g6.xlarge   us-east-1   us-east-1b   139m
test-6m9rh-worker-us-east-1a-v2wn5   Running   ecs.g6.large    us-east-1   us-east-1a   32m
test-6m9rh-worker-us-east-1b-8dlqd   Running   ecs.g6.large    us-east-1   us-east-1b   6m
test-6m9rh-worker-us-east-1b-xwzhd   Running   ecs.g6.large    us-east-1   us-east-1b   7m40s

@elmiko @JoelSpeed I have updated this PR from your comments. I have also created the corresponding installer PR here https://github.com/openshift/installer/pull/5381.
I have created 3 clusters using this code and all have been successful creates.  This also included removing dependencies and getting updates to cluster-api-provider-{ovirt,openstack} and updates to machine-api-operator-openstack to remove older mao dependencies.
Please TAL


Privacy setting copy

#6212
Need help to review whether I am doing this in right way or not.

Thanks @Skchoudhary!
I don't think splitting the logic between views/page_privacy.py and views/pages/copy.py is quite right here - having a copy_privacy method in views/page_privacy.py would imply that there's a URL that users can visit in the admin specifically for copying privacy settings.
In fact, I don't think you need to change view-level code at all - if a developer was to call page.copy(...) from their own code (for example, a maintenance script), we'd still want privacy settings to be copied, even though that's not going through the admin views at all. So, all of the new code should be within the Page.copy method in wagtail/core/models.py.

okay @gasman, will move the logic to wagtail/core/models.py.

Hello @gasman
Can you please review the commit.

Hello @gasman
I have done the requested changes, but bit confused about the test cases and have come up with these two test cases.
    def test_copy_page_copies_restriction_when_parent_page_have_no_restriction(self):
        # Test case where Page we are copy from have restriction and parent have no restriction
        ...

    def test_copy_page_doesnt_copies_restriction_when_parent_page_have_restriction(self):
        # Test case where Page we are copy from have restriction and parent have restriction
        ...

On looking some other test cases code, found a way to create Page object with SimplePage, but in this we cannot add restriction, can you point to some code where from I can take reference.

@Skchoudhary Thanks for the updates!
Adding a view restriction inside test code can be done the same way as in any other code, by creating a PageViewRestriction object. There's an example here: https://github.com/wagtail/wagtail/blob/b0fa9c30d77972d90e0961fa6b42d4447e211e21/wagtail/core/tests/test_page_queryset.py#L363-L369

Hello @gasman
Test cases added for the changed code, please review the changes. Will rebase the branch with the master after that :)

@Skchoudhary been a while, are you able to rebase this on main. I know that you were waiting on some additional feedback, sorry that never happened.
There is another PR that aims to make page copying better and it reminded me of this PR.

Hello @lb- I will rebase the code onto the main branch by the end of this week.

Epic - thanks @Skchoudhary

Hello @lb- PR is ready for the review and I have tested the changes on the squash too.

Thanks. I've labelled as needs review again. I'll try to get to it if @gasman doesn't soon.


debugger and spork

I hope I'm not bothering you a 10000th time but I really can't find a solution.
I use spork with Guard + Rspec and the debugger doesn't work as expected:

I added require 'spork/ext/ruby-debug' just after the require 'spork'

it properly stops on debugger breakpoints

but I can't access irb, it spits: Command is available only in local mode.

Do you, please, know how to get around this?
Thanks,
Ben

Turn off guard interactive mode. See https://github.com/guard/guard/issues/142


Cartesian speed

How do can I change Cartesian speed for Franka Emika with this script? I tried changing "max_cartesian_speed" parameter, but it does not change anything. Can you help?

Hello @albic98,
That sounds like a bug. I think this might be because the functionality is simply not implemented for execute_via_moveit=False. You could try with execute_via_moveit=True and see if it changes anything.
Alternatively, you could try taking a look at moveit2.py and see how @max_cartesian_speed.setter needs to be modified to fix this. PRs are welcome. :slightly_smiling_face:

I tried switching execute_via_moveit to True, but it keeps moving in joint path, not cartesian. Also I tried everything with the setter, but still no solution.

Addressed by https://github.com/AndrejOrsula/pymoveit2/pull/53


Change chapter tag check to include alternative names

For example "Chapter Markers" instead of just "uberdrive mediamarkers".

Closing cause we fixed it


get_nets -of_objects returns a list of valid nets and a NULL

I had to catch the NULL in the list of nets to avoid a script error.
Is this behaviour of get_nets intended or a bug?
This is the script part that I added to flow.tcl to get a robust clock net routing:
Global routing
create_ndr -name mult -spacing { *2 } -width {met1:met4 *3}
set nets [get_nets -of_objects [get_fanout -from [get_full_name [sta::get_clocks *]]] -quiet]
foreach net $nets {
if {![string equal $net NULL]} {
assign_ndr -ndr mult -net [get_full_name $net]
}
}

Please provide a test case

Uploading run_all_aes_sky130hd_base_2022-01-23_21-53.tar.gz…
get_nets command is part of run_all_aes_sky130hd_base_2022-01-23_21-53/scripts/pre_global_route.tcl
without the comments the layout is generated with the custom clock tree routing rule

run_all_aes_sky130hd_base_2022-01-23_21-53.tar.gz
get_nets command is part of run_all_aes_sky130hd_base_2022-01-23_21-53/scripts/pre_global_route.tcl
without the comments the layout is generated with the custom clock tree routing rule


Command Palette score increases with selection

Sublime will increase the score of an item for specific characters when selected.
ie. if I type li.js and I select components/list/index.jsx and it is several items down in the list, the next time I type li.js it will have higher scoring as a relevant result.

Thanks for the suggestion :v:  Seems like you're referring to the fuzzy-finder?  If so, can you re-open in https://github.com/atom/fuzzy-finder if there's not an existing issue for it so we have it in the right place?  If you can also use the template as much as possible, that would be super helpful for us.


DS-3130: Fix inconsistent logging of EventServiceImpl

Minor fixes for
https://jira.duraspace.org/browse/DS-3130
Fix inconsistent logging. Remove incorrect comments. Change constructor method to private.

+1 by inspection.  This all looks sensible, and the old code was a bit random.

Merging this as an obvious fix to some random logging/error throwing.


Cleanup of error prone and warnings

FWIW I dislike this change, but if we want value out of errorprone and compiler warnings, we need to reduce the noise on every build.

The error prone checks should be made to error if you're fixing them. Warnings are generally useless. It should either always error and require suppression with a comment or just be disabled.

@swankjesse @JakeWharton Thanks for the discussion.  Agreed on the errorprone becoming errors, I'll work towards this module by module.


"Dead pixels" in some generated gradients

I haven't yet tried to debug the gradient code, but there is a "dead pixel" effect in some of the examples.
I see the same effect in both Chrome and Firefox (with the dead pixels in exactly the same places), so it isn't a browser issue.  I suspect it is some sort of rounding error.  You would need to compare with the Cairo rendering of the same gradients to determine if the problem is inherent in the rendering code or if it was introduced by converting to JavaScript.
Sample 2:

Sample 3:

Also possibly related: the left edge of each image contains a row of off-color pixels. That may need to be a separate issue.


Post #17

Also add signature image to all posts.
post_17.docx

Markdown conversion enclosed:
post_17.md

Markdown conversion enclosed:
post_17.md


refactor(Migrations): split computed column into separate operations

[x] The code builds and tests pass (verified by our automated build checks)
[x] Commit messages follow this format
[ ] Tests for the changes have been added (for bug fixes / features)
[x] Code meets the expectations our engineering guidelines.

See #6898
cc @bricelam

Thanks again for updating this!


Where should the citation information live?

@SMLPfaendler noted that some client onboarding material uses an out-of-date link for how to cite systems/software in research. The citing information now lives in the FAQs section of the support page.
Sieglinde asks if we can set up a redirect from the old link (quantum-computing.com/docs/manage/systems) to the new location.
It is not easy to directly link to an item within the FAQ (we can create a custom link but it requires the person who wishes to use the link to look into the code to find that link), so the question is, should the citation info move somewhere more discoverable before re-routing the broken link?
cc: @Eric-Arellano @javabster @beckykd

We're also trying to figure out where to put the contribution guidelines etc.  I wonder if this ought to live nearby wherever we decide to put that stuff.

On Friday, January 26, 2024, abbycross @.***> wrote:

@SMLPfaendler noted that some client onboarding material uses an
out-of-date link for how to cite systems/software in research. The citing
information now lives in the FAQs section of the support page.
Sieglinde asks if we can set up a redirect from the old link (
quantum-computing.com/docs/manage/systems) to the new location.
It is not easy to directly link to an item within the FAQ (we can create
a custom link but it requires the person who wishes to use the link to look
into the code to find that link), so the question is, should the citation
info move somewhere more discoverable before re-routing the broken link?
cc: @Eric-Arellano @javabster @beckykd
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you were mentioned.Message ID:
@.***>


versions of the libraries

The requirements file does not include the versions of the libraries.
I have 3060Ti and having some problem with creating the .so files.
can someone write the versions of the libraries?

Could you provide and check the nvidia-driver version, the cuda version, and the cuda version that pytorch is compiled on (torch.version.cuda).

It seems that your environment contains multiple cuda version. Please clean it up and it is better to keep version number aligned in nvcc / torch.cuda / nvidia-driver.


Every time i type command in server it types this

Ignoring exception in on_command_error
Traceback (most recent call last):
File "site-packages\discord\client.py", line 333, in _run_event
File "c-realV2.py", line 318, in on_command_error
File "c-realV2.py", line 276, in log
AttributeError: 'NoneType' object has no attribute 'id'

This is a well known bug for us right now.
But as for now you can don't really need the servers command to connect to a server. Use connect <server id> to connect.

its now fixed


Suggestion: adding station as index to multi-station retrievals

When getting observations from multiple stations:
from ndbc_api import NdbcApi

api = NdbcApi()

# get all standard meterological measurements for stations tplm2 and apam2
stdmet_df = api.get_data(
    station_ids=['tplm2', 'apam2'],
    mode='stdmet',
    start_time='2021-01-01',
    end_time='2022-01-01',
)

stdmet_df

The index of the returned df is only the timestamp, resulting in a non-unique index (because timestamps can be duplicated between different stations). Perhaps it may be useful to make station_id an index as well.
In addition to better leveraging pandas's index-based functions, this would also make it easier to flip between pandas and xarray (since df.to_xarray() uses those indices), for example.
(it's easy enough to do 'manually' through stdmet_df.reset_index().set_index(['station_id','timestamp']), but it may be a neater way of showing this information)
This is related to JOSS review https://github.com/openjournals/joss-reviews/issues/7406.

As for #51 , the suggestions are incorporated in #52 and the release in #53. I am scoping an appropriate change which allows for station metadata in the returned xarray Datasets.  Thanks again for the suggestions!


uses too much ram and cpu

pls fix

what's your platform, on my osx it looks work well

what about ram?

So embarrassing it is. It's normal to consume much RAM due to Electron.

This is super awkward, Electron use much RAM is normal

ok, so can you use some other platform that isn't a resource hog? my 2005 thinkpad is struggling to run this

Can you just buy anew laptop?

umm macbooks are very expensive and i can't run libreboot on them

@MegaApuTurkUltra You can consider other wechat projects using search of Github. I remember a wechat client on Github have 8000+ stars in another repo.


pytest failed

Hello,
I was trying to install pysoftk as per the instructions provided in you git and when I did pytest, I got this error.
ERROR pysoftk_dir/pysoftk/docs/source/tutorials/scripts/htp_test.py - AttributeError: 'NoneType' object has no attribute 'Setup'.
Could you please let me know the cause for it.
Thanks

Hi fairycod,
Thanks for your interest in PySoftK. If you want to run the tests, the correct folder is not located in the docs. There is a folder called test in the same path where the folder docs and pysoftk is placed. This is the correct one to run the tests. Please be awaeto install xtb first, otherwise the calculations of the htp functionality are going to fail. I hope this helps.


Carousel image - BOX 🗳

The woman king high quality -> https://weliveentertainment.com/wp-content/uploads/2022/09/the-woman-king-banner-2.jpg


[SPARK-15968][SQL] Nonempty partitioned metastore tables are not cached

(Please note this is a revision of PR #13686, which has been closed in favor of this PR.)
This PR addresses SPARK-15968.
What changes were proposed in this pull request?
The getCached method of HiveMetastoreCatalog computes pathsInMetastore from the metastore relation's catalog table. This only returns the table base path, which is incomplete/inaccurate for a nonempty partitioned table. As a result, cached lookups on nonempty partitioned tables always miss.
Rather than get pathsInMetastore from
metastoreRelation.catalogTable.storage.locationUri.toSeq

I modified the getCached method to take a pathsInMetastore argument. Calls to this method pass in the paths computed from calls to the Hive metastore. This is how getCached was implemented in Spark 1.5:
https://github.com/apache/spark/blob/e0c3212a9b42e3e704b070da4ac25b68c584427f/sql/hive/src/main/scala/org/apache/spark/sql/hive/HiveMetastoreCatalog.scala#L444.
I also added a call in InsertIntoHiveTable.scala to invalidate the table from the SQL session catalog.
How was this patch tested?
I've added a new unit test to parquetSuites.scala:
SPARK-15968: nonempty partitioned metastore Parquet table lookup should use cached relation

Note that the only difference between this new test and the one above it in the file is that the new test populates its partitioned table with a single value, while the existing test leaves the table empty. This reveals a subtle, unexpected hole in test coverage present before this patch.
Note I also modified a different but related unit test in parquetSuites.scala:
SPARK-15248: explicitly added partitions should be readable

This unit test asserts that Spark SQL should return data from a table partition which has been placed there outside a metastore query immediately after it is added. I changed the test so that, instead of adding the data as a parquet file saved in the partition's location, the data is added through a SQL INSERT query. I made this change because I could find no way to efficiently support partitioned table caching without failing that test.
In addition to my primary motivation, I can offer a few reasons I believe this is an acceptable weakening of that test. First, it still validates a fix for SPARK-15248, the issue for which it was written. Second, the assertion made is stronger than that required for non-partitioned tables. If you write data to the storage location of a non-partitioned metastore table without using a proper SQL DML query, a subsequent call to show that data will not return it. I believe this is an intentional limitation put in place to make table caching feasible, but I'm only speculating.
Building a large HadoopFsRelation requires stat-ing all of its data files. In our environment, where we have tables with 10's of thousands of partitions, the difference between using a cached relation versus a new one is a matter of seconds versus minutes. Caching partitioned table metadata vastly improves the usability of Spark SQL for these cases.
Thanks.

Can one of the admins verify this patch?

@hvanhovell I'm mentioning you here because you commented on my previous PR for this Jira issue. In response to your original question, yes, I have added a unit test for this patch.

Test build #3124 has started for PR 13818 at commit 8a058c6.

Test build #3124 has finished for PR 13818 at commit 8a058c6.

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

cc @cloud-fan / @liancheng

seems a reasonable change to me, thanks for working on it?

You are very welcome. Thank you for taking time to review it! 😃

LGTM, cc @liancheng

ok to test

test this please

Test build #61594 has started for PR 13818 at commit c2ba4af.

Test build #61594 has finished for PR 13818 at commit c2ba4af.

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/61594/
Test PASSed.

LGTM except for minor styling issues. Thanks!

Test build #61730 has started for PR 13818 at commit 91ef950.

I believe I've addressed @liancheng's style issues in my new unit test, along with the same in the two tests from which it was copy-pasta'd (boy scout rule).

Test build #61730 has finished for PR 13818 at commit 91ef950.

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/61730/
Test PASSed.

thanks, merging to master!

Shall we also have this in branch-2.0? This seems to be a pretty serious bug. cc @rxin.

I have a few questions.

Is it a regression from 1.6? Looks like not?
Is it a correctness issue or a performance issue? Seems it is a performance issue?
If it is a performance issue. What is the impact? For a hive parquet/orc table, after we convert them to Spark's native code path, there is no partitioning discovery. So, I guess the performance is mainly coming from querying metastore? If so, what will be the perf difference after spark.sql.hive.metastorePartitionPruning (only querying needed partition info from Hive metastore) is enabled?

My feeling is that if it is a perf issue and it is not a regression from 1.6, merging to master should be good enough.

FYI this breaks Scala 2.10:
[info] Compiling 254 Scala sources and 5 Java sources to /home/jenkins/workspace/spark-master-compile-sbt-scala-2.10/mllib/target/scala-2.10/classes...
[error] /home/jenkins/workspace/spark-master-compile-sbt-scala-2.10/sql/hive/src/main/scala/org/apache/spark/sql/hive/execution/InsertIntoHiveTable.scala:301: value invalidateTable is not a member of org.apache.spark.sql.catalyst.catalog.SessionCatalog
[error]     sqlContext.sessionState.catalog.invalidateTable(table.catalogTable.identifier)
[error]                                     ^
[error] one error found
[warn] Multiple main classes detected.  Run 'show discoveredMainClasses' to see the list
[info] Packaging /home/jenkins/workspace/spark-master-compile-sbt-scala-2.10/mllib/target/scala-2.10/spark-mllib_2.10-2.0.0-SNAPSHOT.jar ...

I have a few questions.
Is it a regression from 1.6? Looks like not?

I don't know about 1.6. I know it's a regression from 1.5.

Is it a correctness issue or a performance issue? Seems it is a performance issue?

It is a performance issue.

If it is a performance issue. What is the impact? For a hive parquet/orc table, after we convert them to Spark's native code path, there is no partitioning discovery. So, I guess the performance is mainly coming from querying metastore? If so, what will be the perf difference after spark.sql.hive.metastorePartitionPruning (only querying needed partition info from Hive metastore) is enabled?

The problem this PR addresses occurs in the analysis phase of query planning. The property spark.sql.hive.metastorePartitionPruning only comes into play in HiveTableScanExec, which is part of physical planning. (And I don't believe it's used to read Parquet tables.) Therefore, that property has no bearing on this problem.
Regarding the impact, I'll quote from the last paragraph of the PR description:

Building a large HadoopFsRelation requires stat-ing all of its data files. In our environment, where we have tables with 10's of thousands of partitions, the difference between using a cached relation versus a new one is a matter of seconds versus minutes. Caching partitioned table metadata vastly improves the usability of Spark SQL for these cases.

My feeling is that if it is a perf issue and it is not a regression from 1.6, merging to master should be good enough.

For some (like us), I'd say this extends beyond a performance issue into a usability issue. We can't use Spark 2.0 as-is if it takes us several minutes to build a query plan.

@zsxwing I was able to do following without error:
git clone git@github.com:apache/spark.git spark-master
cd spark-master
./dev/change-scala-version.sh 2.10
./build/mvn -Pyarn -Phadoop-2.4 -Dscala-2.10 -DskipTests clean package

@mallman never mind. https://github.com/apache/spark/commit/5b7a1770ac9cf36a1e92b31d10fe6eeee92fef17 fixed the issue.


Issue debugging JobSet

Possibly related: #141
I created an operator that uses jobset to put together a slurm cluster, and I should have 4 replicated jobs, all indexed, and all of size 1 except for the workers. I'm having trouble just debugging what is going on - the controller reports everything is created (minus the weird log message linked above) and I can get/describe the jobset, but I don't see any pods or events. Here is the only thing I can see, and I'm not sure how to investigate or debug without more information:
apiVersion: v1
items:
- apiVersion: jobset.x-k8s.io/v1alpha1
  kind: JobSet
  metadata:
    creationTimestamp: "2023-06-06T18:44:05Z"
    generation: 1
    name: slurm-sample
    namespace: slurm-operator
    ownerReferences:
    - apiVersion: flux-framework.org/v1alpha1
      blockOwnerDeletion: true
      controller: true
      kind: Slurm
      name: slurm-sample
      uid: 2fb95f68-47a8-417c-983e-250e45113384
    resourceVersion: "3396"
    uid: 3f472f6d-bf23-4f35-a965-2c134f226816
  spec:
    replicatedJobs:
    - name: slurm-sample-server
      network:
        enableDNSHostnames: false
      replicas: 1
      template:
        metadata:
          name: slurm-sample-server
          namespace: slurm-operator
        spec:
          activeDeadlineSeconds: 31500000
          backoffLimit: 100
          completionMode: Indexed
          completions: 1
          parallelism: 1
          template:
            metadata:
              labels:
                app.kubernetes.io/name: slurm-sample
                cluster-name: slurm-sample
                namespace: slurm-operator
              name: slurm-sample
              namespace: slurm-operator
            spec:
              containers:
              - command:
                - /bin/bash
                - /slurm_operator/start-server.sh
                - echo hello world
                image: ghcr.io/converged-computing/slurm
                imagePullPolicy: IfNotPresent
                name: server-node
                resources: {}
                stdin: true
                tty: true
                volumeMounts:
                - mountPath: /slurm_operator/
                  name: slurm-sample-entrypoint
                  readOnly: true
                - mountPath: /etc/slurm/
                  name: slurm-sample-config
                  readOnly: true
              restartPolicy: OnFailure
              subdomain: slurm-service
              volumes:
              - configMap:
                  items:
                  - key: start-server
                    mode: 511
                    path: start-server.sh
                  - key: start-worker
                    mode: 511
                    path: start-worker.sh
                  - key: start-daemon
                    mode: 511
                    path: start-daemon.sh
                  name: slurm-sample-entrypoint
                name: slurm-sample-entrypoint
              - configMap:
                  items:
                  - key: slurm.conf
                    mode: 600
                    path: slurm.conf
                  - key: slurmdbd.conf
                    mode: 600
                    path: slurmdbd.conf
                  name: slurm-sample-config
                name: slurm-sample-config
    - name: slurm-sample-database
      network:
        enableDNSHostnames: true
      replicas: 1
      template:
        metadata:
          name: slurm-sample-database
          namespace: slurm-operator
        spec:
          activeDeadlineSeconds: 31500000
          backoffLimit: 100
          completionMode: Indexed
          completions: 1
          parallelism: 1
          template:
            metadata:
              labels:
                app.kubernetes.io/name: slurm-sample
                cluster-name: slurm-sample
                namespace: slurm-operator
              name: slurm-sample
              namespace: slurm-operator
            spec:
              containers:
              - env:
                - name: MYSQL_RANDOM_ROOT_PASSWORD
                  value: "yes"
                - name: MYSQL_DATABASE
                - name: MYSQL_USER
                - name: MYSQL_PASSWORD
                imagePullPolicy: IfNotPresent
                name: slurm-sample-database
                resources: {}
                stdin: true
                tty: true
              restartPolicy: OnFailure
              subdomain: slurm-service
    - name: slurm-sample-daemon
      network:
        enableDNSHostnames: false
      replicas: 1
      template:
        metadata:
          name: slurm-sample-daemon
          namespace: slurm-operator
        spec:
          activeDeadlineSeconds: 31500000
          backoffLimit: 100
          completionMode: Indexed
          completions: 1
          parallelism: 1
          template:
            metadata:
              labels:
                app.kubernetes.io/name: slurm-sample
                cluster-name: slurm-sample
                namespace: slurm-operator
              name: slurm-sample
              namespace: slurm-operator
            spec:
              containers:
              - command:
                - /bin/bash
                - /slurm_operator/start-daemon.sh
                - echo hello world
                image: ghcr.io/converged-computing/slurm
                imagePullPolicy: IfNotPresent
                name: daemon-node
                resources: {}
                stdin: true
                tty: true
                volumeMounts:
                - mountPath: /slurm_operator/
                  name: slurm-sample-entrypoint
                  readOnly: true
                - mountPath: /etc/slurm/
                  name: slurm-sample-config
                  readOnly: true
              restartPolicy: OnFailure
              subdomain: slurm-service
              volumes:
              - configMap:
                  items:
                  - key: start-server
                    mode: 511
                    path: start-server.sh
                  - key: start-worker
                    mode: 511
                    path: start-worker.sh
                  - key: start-daemon
                    mode: 511
                    path: start-daemon.sh
                  name: slurm-sample-entrypoint
                name: slurm-sample-entrypoint
              - configMap:
                  items:
                  - key: slurm.conf
                    mode: 600
                    path: slurm.conf
                  - key: slurmdbd.conf
                    mode: 600
                    path: slurmdbd.conf
                  name: slurm-sample-config
                name: slurm-sample-config
    - name: slurm-sample-worker
      network:
        enableDNSHostnames: false
      replicas: 1
      template:
        metadata:
          name: slurm-sample-worker
          namespace: slurm-operator
        spec:
          activeDeadlineSeconds: 31500000
          backoffLimit: 100
          completionMode: Indexed
          completions: 1
          parallelism: 1
          template:
            metadata:
              labels:
                app.kubernetes.io/name: slurm-sample
                cluster-name: slurm-sample
                namespace: slurm-operator
              name: slurm-sample
              namespace: slurm-operator
            spec:
              containers:
              - command:
                - /bin/bash
                - /slurm_operator/start-worker.sh
                - echo hello world
                image: ghcr.io/converged-computing/slurm
                imagePullPolicy: IfNotPresent
                name: worker-node
                resources: {}
                stdin: true
                tty: true
                volumeMounts:
                - mountPath: /slurm_operator/
                  name: slurm-sample-entrypoint
                  readOnly: true
                - mountPath: /etc/slurm/
                  name: slurm-sample-config
                  readOnly: true
              restartPolicy: OnFailure
              subdomain: slurm-service
              volumes:
              - configMap:
                  items:
                  - key: start-server
                    mode: 511
                    path: start-server.sh
                  - key: start-worker
                    mode: 511
                    path: start-worker.sh
                  - key: start-daemon
                    mode: 511
                    path: start-daemon.sh
                  name: slurm-sample-entrypoint
                name: slurm-sample-entrypoint
              - configMap:
                  items:
                  - key: slurm.conf
                    mode: 600
                    path: slurm.conf
                  - key: slurmdbd.conf
                    mode: 600
                    path: slurmdbd.conf
                  name: slurm-sample-config
                name: slurm-sample-config
    suspend: false
kind: List
metadata:

Hopefully this is something transient but I'm totally out of ideas! I'll keep trying to tweak things like removing some of the replicated jobs in case one is somehow silently wonky. If it doesn't work I might refactor back to using an indexed job with some kind of side services, but that's not ideal.

Got it working - I used the wrong format for an octal number and the error was higher in the logs than I originally looked. For future folks that don't see anything appear, scroll ALL the way up in the jobset controller logs! The error is there... :laughing:

This is great! It would be great if we an add this to JobSet documentation under tasks: https://github.com/kubernetes-sigs/jobset/tree/main/docs/tasks; the task would be: Setup a Slurm for example, what do you think?

Or maybe a link to projects that use jobset? There are several other config maps and templates that the (now existing!) slurm operator use so it would be easier to point to the project then dump out the yaml files: https://github.com/converged-computing/slurm-operator/.
I also commented on another issue, but for this case I had a lot of trouble with the hostnames turning out too long, and had to make them arbitrarily short. I know we need to scope it to the namespace / name / service, etc., but I wonder if there still could be a shorthand option (and trusting the user to not break that?)


Suggestion: Open your project to contributions

Why
So I (or anyone that likes the project and want to contribute to it) can make PRs on it 😛

At the moment project structure is not ready to be contributed on. It misses contributing guidelines and templates for issues and PR. But you're welcome to fork the repository and make a PR if you want to contribute to the project.


@JuliaRegistrator register()

@JuliaRegistrator register()

Registration pull request created: JuliaRegistries/General/60618
After the above pull request is merged, it is recommended that a tag is created on this repository for the registered package version.
This will be done automatically if the Julia TagBot GitHub Action is installed, or can be done manually through the github interface, or via:
git tag -a v0.1.0 -m "<description of version>" 4129e20c229577c3c290a14fba4e745b479335c8
git push origin v0.1.0

@JuliaRegistrator register()

Registration pull request updated: JuliaRegistries/General/60618
After the above pull request is merged, it is recommended that a tag is created on this repository for the registered package version.
This will be done automatically if the Julia TagBot GitHub Action is installed, or can be done manually through the github interface, or via:
git tag -a v0.1.0 -m "<description of version>" dac2d9af0ab403b6372a43f52a99fddfb12fe199
git push origin v0.1.0

Registration pull request created: JuliaRegistries/General/60929
After the above pull request is merged, it is recommended that a tag is created on this repository for the registered package version.
This will be done automatically if the Julia TagBot GitHub Action is installed, or can be done manually through the github interface, or via:
git tag -a v0.2.0 -m "<description of version>" 2f5e3b8d8464b75b6653432baf74ca4dcce5dd25
git push origin v0.2.0

@JuliaRegistrator register()

Error while trying to register: Changing package repo URL not allowed, please submit a pull request with the URL change to the target registry and retry.

@JuliaRegistrator register(branch="ai4energy/OptControl.jl.git/main")

@JuliaRegistrator register()

Registration pull request created: JuliaRegistries/General/61710
After the above pull request is merged, it is recommended that a tag is created on this repository for the registered package version.
This will be done automatically if the Julia TagBot GitHub Action is installed, or can be done manually through the github interface, or via:
git tag -a v1.0.0 -m "<description of version>" 9e5a4b0871bffe82240cb9a7c0c04ef3c99c8bf9
git push origin v1.0.0

@JuliaRegistrator register()

Registration pull request created: JuliaRegistries/General/74438
After the above pull request is merged, it is recommended that a tag is created on this repository for the registered package version.
This will be done automatically if the Julia TagBot GitHub Action is installed, or can be done manually through the github interface, or via:
git tag -a v1.0.1 -m "<description of version>" af06a391868449af59be975ca3206e1bac2a972a
git push origin v1.0.1


Enable Disable?

I could do another pull request.. but it would be great that DateRangePickerView.isEnabled = false would actually disable the control.

Pushed e0c4062f10abfb50a334df650597d347d5c3936e.


[BUG]CRSError: During execution of task ExportToTiff: The EPSG code is unknown. PROJ: proj_create_from_database: C:\Program Files\PostgreSQL\14\share\contrib\postgis-3.2\proj\proj.db contains DATABASE.LAYOUT.VERSION.MINOR = 0 whereas a number >= 2 is expected. It comes from another PROJ installation.

Hello,
I witnessed with one problem when I tried to implement the example notebooks which is extra_example-L1C.ipynb when I put my AOI and started to implement the line of the code below
time_interval = ['2021-05-01','2021-09-30']
idx = 0
bbox = bbox_splitter.bbox_list[idx]
info = bbox_splitter.info_list[idx]
tiff_name = f'val_count_sh_eopatch_{idx}_row-{info["index_x"]}col-{info["index_y"]}.tiff'
patch_name = f'eopatch{idx}_row-{info["index_x"]}_col-{info["index_y"]}'
results = workflow.execute({input_task:{'bbox':bbox, 'time_interval':time_interval},
export_val_sh:{'filename':str(data/'data'/'valid_count-L1C'/tiff_name)},
save:{'eopatch_folder':patch_name}
})
I get an error which is

Based on the error I'd say you have two installations of PROJ that are not playing nicely with one another.
I suggest you have a look if that is the case, remove (possibly both) and reinstall. The best approach on Win OS is to install pyproj with conda.

Closing due to inactivity.


[FCL-309] Add the ability for documents to store abstract identifiers in MarkLogic properties

Summary of changes

[x] New IdentifierSchema and Identifier abstract classes which define interfaces for how we think about what any given type of identifier looks like (IdentifierSchema), and how we might be able to interact with specific instances of that identifier (Identifier)
[x] New NeutralCitationSchema and NeutralCitation concretions of the above abstract classes, defining NCN-specific implementations of methods where necessary.
[ ] New methods for a Document which allow CRUD operations on its identifiers, using the above classes to turn identifier concepts into serialisations which can be stashed in the MarkLogic properties for a document.

Checklist

[ ] I have created/updated method docstrings (if necessary)
[ ] I have considered if this is a breaking change
[ ] I have updated the changelog (if necessary)

Jira
FCL-309

I'm a touch concerned about the properties being used because they might be stale if overwritten, but that's a risk anyway at the moment.
Do we want more general properties parsing and editing? (Probably not)


fixing safety inference and safety adapter for new API spec. Pinned t…

…he llama_models version to 0.0.24 as the latest version 0.0.35 has the model descriptor name changed. I was getting the missing package error during runtime as well, hence added the dependency to requirements.txt
This was tested with local instance of llama-stack in container as well as without container

Everything looks great other than the change in requirements.txt

@ashwinb requirements.txt is fixed and has no changes anymore.


TagBot trigger issue

This issue is used to trigger TagBot; feel free to unsubscribe.
If you haven't already, you should update your TagBot.yml to include issue comment triggers.
Please see this post on Discourse for instructions and more details.
If you'd like for me to do this for you, comment TagBot fix on this issue.
I'll open a PR within a few hours, please be patient!

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/65301

Triggering TagBot for merged registry pull request
This extra notification is being sent because I expected a tag to exist by now, but it doesn't.
You may want to check your TagBot configuration to ensure that it's running, and if it is, check the logs to make sure that there are no errors.

Triggering TagBot for merged registry pull request
This extra notification is being sent because I expected a tag to exist by now, but it doesn't.
You may want to check your TagBot configuration to ensure that it's running, and if it is, check the logs to make sure that there are no errors.

Triggering TagBot for merged registry pull request
This extra notification is being sent because I expected a tag to exist by now, but it doesn't.
You may want to check your TagBot configuration to ensure that it's running, and if it is, check the logs to make sure that there are no errors.

Triggering TagBot for merged registry pull request
This extra notification is being sent because I expected a tag to exist by now, but it doesn't.
You may want to check your TagBot configuration to ensure that it's running, and if it is, check the logs to make sure that there are no errors.

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/67726

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/69596

Triggering TagBot for merged registry pull request
This extra notification is being sent because I expected a tag to exist by now, but it doesn't.
You may want to check your TagBot configuration to ensure that it's running, and if it is, check the logs to make sure that there are no errors.

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/72054

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/88031

Triggering TagBot for merged registry pull request
This extra notification is being sent because I expected a tag to exist by now, but it doesn't.
You may want to check your TagBot configuration to ensure that it's running, and if it is, check the logs to make sure that there are no errors.

Triggering TagBot for merged registry pull request
This extra notification is being sent because I expected a tag to exist by now, but it doesn't.
You may want to check your TagBot configuration to ensure that it's running, and if it is, check the logs to make sure that there are no errors.

Triggering TagBot for merged registry pull request
This extra notification is being sent because I expected a tag to exist by now, but it doesn't.
You may want to check your TagBot configuration to ensure that it's running, and if it is, check the logs to make sure that there are no errors.


UI improve

hey @Kritika30032002
I made some changes to your ui if you like it then keep it if not then tell me i have made some changes by keeping whatever ui you had previously
Thank you for considering...!!✌

not good


[ISSUE] Unsupported Version of Elasticsearch [8.7.1] used in Helm Chart

Describe the issue:
According to the documentation, all Camunda Components require Elasticsearch 8.8.x+.
https://docs.camunda.io/docs/reference/supported-environments/
Actual behavior:
The Helm Chart uses Elasticsearch 8.7.1
Expected behavior:
Helm Chart uses a supported Elasticsearch version
How to reproduce:
helm install camunda camunda/camunda-platform
kubectl describe sts camunda-elasticsearch-master
Logs:
  Containers:
   elasticsearch:
    Image:       docker.io/bitnami/elasticsearch:8.7.1
    Ports:       9200/TCP, 9300/TCP
    Host Ports:  0/TCP, 0/TCP
    Limits:
      cpu:     2
      memory:  2Gi
    Requests:
      cpu:      1
      memory:   2Gi

Environment:
Please note: Without the following info, it's hard to resolve the issue and probably it will be closed.

Platform: GCP
Helm CLI version: version.BuildInfo{Version:"v3.10.1", GitCommit:"9f88ccb6aee40b9a0535fcc7efea6055e1ef72c9", GitTreeState:"clean", GoVersion:"go1.18.7"}
Chart version: 8.3.1
Values file:  default

Thanks @ManuelDittmar for reporting that.
It has been fixed in the latest version of C8 Helm chart 8.3.2 (it happened because we pin the ES version in Renovate bot config, and we will find a way to connect Renovate with the apps to get the latest supported ES version)


Block solver for multi-constraint joints

Many joint types (and contacts with two contact points!) require more than one constraint (up to three in 2D as there are three degrees of freedom, but possibly more with inequality constraints I think?), which should ideally be solved simultaneously for improved convergence and stability.
XPBD note: Joints require fewer constraints now, so this is less impactful, but should still be nice for double contacts. In particular, this should fix the issue with things sometimes getting stuck on walls due to friction

Closing all issues, moving planning to my notes repo for more freely writing down ideas


Dependent Dropdowns

I have two select2 dropdowns and I want to change options the second dropdown depending on first dropdown.
For example,
    {
      data: 'country_id',
      editor: 'select2',
      renderer: customDropdownRenderer,
      select2Options: {
              data: {!! $production_units !!} ,
              dropdownAutoWidth: true,
      }
    },
    {
      data: 'state_id',
      editor: 'select2',
      renderer: customDropdownRenderer,
      select2Options: {
              data: [],
              dropdownAutoWidth: true,
              width: 'resolve'
      }
    },

Depending on country_id, I want to change select2 options of state_id. I know how to make this work with just select2, but I am not able to figure out how to make it work with handsontable.
I have change select2Options in afterChange, but how to do that?
afterChange: function (change, source) {
    if(change)
    {

      if(change[0][1] == 'country_id')
      {
        $.get("/api/states?country="+change[0][3], function(data){

             //What should be done here?

        });
      }

    }

  },

I have posted a question at
http://stackoverflow.com/questions/31691214/handsontable-select2-dependent-dropdowns

Answer from Stackoverflow:
You retrieve all the states for that country. Make sure the states list is in the same format that select2 expects.
Then loop through the columns list and determine the dependent column for the current column that has been modified.
Get the cell properties:
var cellProperties = instance.getCellMeta(rowIndex, dependentColumnIndex);
Update the select2 source:
cellProperties['select2Options'].data = [states list];
I might have done this slightly differently, but this is probably better than the approach I had thought of.

Hi,
Following the same example, this solution is not working.
Let's say I chose "Australia" in first row, it shows states in Australia in the states column of the corresponding row.
In another row below it, If i chose "India", then the problem starts, The states data of first row in which we selected "Australia" is also changes to states list of "India".
In short, select2Options are changing for all rows.


MissingMethodException

Hi, I'm using .NET Core 7.0.15 and Microsoft.EntityframeworkCore.SqlServer 7.0.15 and I'm getting this error when starting my ASP.NET Core application. Do I need to upgrade to .NET 8.0?
MissingMethodException:
Method not found: 'Microsoft.EntityFrameworkCore.Migrations.Operations.Builders.OperationBuilder`1<Microsoft.EntityFrameworkCore.Migrations.Operations.CreateIndexOperation> Microsoft.EntityFrameworkCore.Migrations.MigrationBuilder.CreateIndex(System.String, System.String, System.String[], System.String, Boolean, System.String)'.

Stott.Security.Optimizely.Migrations.InitialDbSetup.Up(MigrationBuilder migrationBuilder)

Hello @kennethskantz,
I had a similar issue previously with a breaking change in EntityFrameworkCore between .NET 6 and .NET 8 which was handled and resolved here : #179
I'm referencing 6.0.6 of Microsoft.EntityframeworkCore.SqlServer assuming most people are using .NET 6 until Optimizely confirms full support for .NET 8 (Which I believe is not promised until CMS 13).  I have noted that Microsoft.EntityframeworkCore.SqlServer 7.20.0 is .NET 6.0 compatible.  So I'll look at updating that dependency and see what happens.
I manage this package in my free time, so if you can give me a couple of days to look into this, that would be great.

Hello @kennethskantz
I've managed to reproduce the issue you raised and it's due to a breaking change in Microsoft.EntityframeworkCore.SqlServer v7.x.x
I have created a hotfix package that updates the dependencies as follows:

EPiServer.CMS.UI.Core : >= 12.31.0 & < 13.0.0
Microsoft.EntityFrameworkCore.SqlServer (.NET 6.0) : >= 7.0.0 & < 8.0.0
Microsoft.EntityFrameworkCore.SqlServer (.NET 8.0) : >= 8.0.1 & < 9.0.0

I've installed this into my test solutions for .NET 6.0 and .NET 8.0 and executed them both with existing databases and fresh databases and they appear to work.
I've published this to nuget.org as a 2.7.1-beta release because obviously I can't test in your specific scenario.  But if you can install this and let me know how that works for you, that would be great.

Hi and thanks for your quick response @GeekInTheNorth!
Unfortunately we're currently on EPiServer.CMS.UI.Core 12.27.0 och not ready to update to the required 12.31.0, since that requires updating both EPiServer.CMS.Core and EPiServer.Framework, which would require regression testing of the site.
We'll have to postpone using this package until we are ready to update the core CMS packages.

Hi @kennethskantz,
I can roll back the dependency for EPiServer.CMS.UI.Core to 12.27.0 if that helps.  I've not merged the hotfix branch in and that's also why I've made it a -beta release.

Yes please do, if possible, that would be very helpful.

@kennethskantz I have uploaded a 2.7.2-beta release onto nuget.org now.  The only difference between this and 2.7.0 is the following dependency ranges:
EPiServer.CMS.UI.Core : >= 12.27.0 & < 13.0.0
Microsoft.EntityFrameworkCore.SqlServer (.NET 6.0) : >= 7.0.0 & < 8.0.0
Microsoft.EntityFrameworkCore.SqlServer (.NET 8.0) : >= 8.0.1 & < 9.0.0

I don't want to bother you too much, but I must be doing something wrong.
The nuget-installation went fine, but I get an error on startup.
In csproj:
<PackageReference Include="Stott.Security.Optimizely" Version="2.7.2-beta" />
On startup (restarted PC, cleaned bin- and obj-folders):
ArgumentException: Unable to find a module by assembly 'Stott.Security.Optimizely, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=null' (Parameter 'moduleAssembly')
EPiServer.Shell.Paths.ToResource(Assembly moduleAssembly, string moduleRelativeResourcePath)

ArgumentException: Unable to find a module by assembly 'Stott.Security.Optimizely, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=null' (Parameter 'moduleAssembly')
EPiServer.Shell.Paths.ToResource(Assembly moduleAssembly, string moduleRelativeResourcePath)
EPiServer.Shell.ViewComposition.IFrameComponentAttribute.CreateComponentDefinition(Type attributedType)
EPiServer.Shell.ViewComposition.AttributedComponentProvider+<>c.<GetComponentDefinitions>b__5_1(<>f__AnonymousType15<Type, ComponentAttribute> cat)
System.Linq.Enumerable+SelectEnumerableIterator<TSource, TResult>.ToList()
System.Linq.Enumerable.ToList<TSource>(IEnumerable<TSource> source)
EPiServer.Shell.ViewComposition.AttributedComponentProvider.GetComponentDefinitions()
EPiServer.Shell.ViewComposition.ComponentManager.ListAll()

The dll in my bin-folder:

@kennethskantz
Can you check that the nuget package has installed this file in your web project:
\modules\_protected\Stott.Security.Optimizely\module.config
It should look like this:
<?xml version="1.0" encoding="utf-8" ?>
<module loadFromBin="true" clientResourceRelativePath="" viewEngine="Razor" authorizationPolicy="Stott:SecurityOptimizely:Policy" moduleJsonSerializerType="None" prefferedUiJsonSerializerType="Net">
  <assemblies>
    <add assembly="Stott.Security.Optimizely" />
  </assemblies>

  <clientModule>
    <moduleDependencies>
      <add dependency="CMS" />
    </moduleDependencies>
  </clientModule>
</module>

I'm also wondering if the -beta is interfering, but I didn't observe such an issue when testing both my .NET 6 and 8 solutions.

I'm a bit embarassed, the error was caused because this line was missing:
services.AddStottSecurity()
Thanks for your help, it seems to be working perfectly now.

@kennethskantz I'm glad the issue was resolved for you and that it is working. I think I will merge this into a new major release, just because there is a breaking change with the dependencies.  Thank you for reaching out so we can solve this issue together.

Just upgraded to 2.7.0, and getting this exception:
System.ArgumentException: Unable to find a module by assembly 'Stott.Security.Optimizely, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=null' (Parameter 'moduleAssembly')
   at EPiServer.Shell.Paths.ToResource(Assembly moduleAssembly, String moduleRelativeResourcePath)
   at EPiServer.Shell.ViewComposition.IFrameComponentAttribute.CreateComponentDefinition(Type attributedType)
   at EPiServer.Shell.ViewComposition.AttributedComponentProvider.<>c.<GetComponentDefinitions>b__5_1(<>f__AnonymousType15`2 cat)
   at System.Linq.Enumerable.SelectEnumerableIterator`2.ToList()
   at EPiServer.Shell.ViewComposition.AttributedComponentProvider.GetComponentDefinitions()
   at EPiServer.Shell.ViewComposition.ComponentManager.ListAll()
   at EPiServer.Shell.ViewComposition.DefaultViewManager..ctor(IEnumerable`1 viewProviders, IEnumerable`1 viewTransformers, IComponentManager componentManager)
   at System.RuntimeMethodHandle.InvokeMethod(Object target, Void** arguments, Signature sig, Boolean isConstructor)
   at System.Reflection.MethodBaseInvoker.InvokeDirectByRefWithFewArgs(Object obj, Span`1 copyOfArgs, BindingFlags invokeAttr)
   at System.Reflection.MethodBaseInvoker.InvokeWithFewArgs(Object obj, BindingFlags invokeAttr, Binder binder, Object[] parameters, CultureInfo culture)
   at System.Reflection.RuntimeConstructorInfo.Invoke(BindingFlags invokeAttr, Binder binder, Object[] parameters, CultureInfo culture)
   at Microsoft.Extensions.DependencyInjection.ServiceLookup.CallSiteRuntimeResolver.VisitConstructor(ConstructorCallSite constructorCallSite, RuntimeResolverContext context)
   at Microsoft.Extensions.DependencyInjection.ServiceLookup.CallSiteVisitor`2.VisitCallSiteMain(ServiceCallSite callSite, TArgument argument)
   at Microsoft.Extensions.DependencyInjection.ServiceLookup.CallSiteRuntimeResolver.VisitRootCache(ServiceCallSite callSite, RuntimeResolverContext context)
   at Microsoft.Extensions.DependencyInjection.ServiceLookup.CallSiteVisitor`2.VisitCallSite(ServiceCallSite callSite, TArgument argument)
   at Microsoft.Extensions.DependencyInjection.ServiceLookup.CallSiteRuntimeResolver.Resolve(ServiceCallSite callSite, ServiceProviderEngineScope scope)
   at Microsoft.Extensions.DependencyInjection.ServiceProvider.CreateServiceAccessor(ServiceIdentifier serviceIdentifier)
   at System.Collections.Concurrent.ConcurrentDictionary`2.GetOrAdd(TKey key, Func`2 valueFactory)
   at Microsoft.Extensions.DependencyInjection.ServiceProvider.GetService(ServiceIdentifier serviceIdentifier, ServiceProviderEngineScope serviceProviderEngineScope)
   at Microsoft.Extensions.DependencyInjection.ServiceLookup.ServiceProviderEngineScope.GetService(Type serviceType)
   at Microsoft.Extensions.DependencyInjection.ServiceProviderServiceExtensions.GetRequiredService(IServiceProvider provider, Type serviceType)
   at Microsoft.Extensions.DependencyInjection.ServiceProviderServiceExtensions.GetRequiredService[T](IServiceProvider provider)
   at EPiServer.Shell.Modules.ModuleInitializer.RegisterServerRoutes(ShellModule shellModule, IEndpointRouteBuilder routeBuilder)
   at EPiServer.Shell.Modules.ModuleInitializer.RegisterModules(IEnumerable`1 modules, IEndpointRouteBuilder routeBuilder)
   at EPiServer.Shell.Routing.ShellEndpointRoutingExtension.InitializeModules(IEndpointRouteBuilder routeBuilder, IEnumerable`1 modules)
   at EPiServer.Shell.Routing.ShellEndpointRoutingExtension.MapEndpoints(IEndpointRouteBuilder endpointRouteBuilder)
   at EPiServer.Web.Routing.EndpointRouteBuilderExtensions.MapEPiServerExtensionEndpoints(IEndpointRouteBuilder defaultEndpointRouteBuilder)
   at EPiServer.Web.Routing.ContentEndpointRouteBuilderExtensions.MapContent(IEndpointRouteBuilder defaultEndpointRouteBuilder)
   at AlloyTours.Startup.<>c.<Configure>b__8_0(IEndpointRouteBuilder endpoints) in D:\Dev\AlloyTours\AlloyTours\Startup.cs:line 339
   at Microsoft.AspNetCore.Builder.EndpointRoutingApplicationBuilderExtensions.UseEndpoints(IApplicationBuilder builder, Action`1 configure)
   at AlloyTours.Startup.Configure(IApplicationBuilder app, IWebHostEnvironment env) in D:\Dev\AlloyTours\AlloyTours\Startup.cs:line 337

I do have services.AddStottSecurity(); in my startup, but I do not have a \modules\_protected\Stott.Security.Optimizely\ directory.
Not sure this will do the trick:
<contentFiles>
  <files include="module.config" buildAction="None" />
</contentFiles>

@evest the file is only added to the protected modules folder on build and should be included already within the NuGet package within the specified folder.
What happens if you do a full clean and rebuild of your solution?
Can you tell me more about your solution? Are you using Visual Studio 2022 or another IDE? Is this happening in a CI/CD pipeline and could you tell me more?

Sorry for the late reply; I'm using Rider, and I tried cleaning and rebuilding.
Didn't work. I copied the file manually, and that helped. I see the targets file in the package, but maybe Rider is more strict?
Since the files under modules are committed, it works in the DevOps pipeline.
I see other packages are constructed a bit differently, in the build directory, they have one or more framework folders with the targets file in, and under ContentFiles, there are any/any/ folders. Not sure how that makes a difference though, since the script points to the folders anyway.

Hello @evest
I've been using Visual Studio and Visual Studio Code for developing this AddOn as I can use those for free.  I'll have to install JetBrains Rider and see if I can reproduce and hopefully solve the issue.
I can tell you that the NuGet package is working in GitHub Actions, Visual Studio and ADO as expected with the modules folder added to the .gitignore, so this is potentially a rider specific issue as you say.

Hello @evest
I've downloaded a trial version of Rider and I can reproduce the issue with the module.config not being installed properly using 2.7.0.
Now I built a 2.7.2-beta release for @kennethskantz and when I use that version and build the project, the module.config file is copied correctly.  Now the only difference here was the version range change I made to the package:
Previous
  <ItemGroup>
    <PackageReference Include="EPiServer.CMS.UI.Core" Version="12.23.0" />
  </ItemGroup>

  <ItemGroup Condition="'$(TargetFramework)' == 'net6.0'">
    <PackageReference Include="Microsoft.EntityFrameworkCore.SqlServer" Version="6.0.6" />
  </ItemGroup>

  <ItemGroup Condition="'$(TargetFramework)' == 'net8.0'">
    <PackageReference Include="Microsoft.EntityFrameworkCore.SqlServer" Version="8.0.1" />
  </ItemGroup>

New
  <ItemGroup>
    <PackageReference Include="EPiServer.CMS.UI.Core" Version="[12.27.0,13.0.0)" />
  </ItemGroup>

  <ItemGroup Condition="'$(TargetFramework)' == 'net6.0'">
    <PackageReference Include="Microsoft.EntityFrameworkCore.SqlServer" Version="[7.0.0,8.0.0)" />
  </ItemGroup>

  <ItemGroup Condition="'$(TargetFramework)' == 'net8.0'">
    <PackageReference Include="Microsoft.EntityFrameworkCore.SqlServer" Version="[8.0.1,9.0.0)" />
  </ItemGroup>

I'm not sure why that would make a difference to rider handling the package differently and correctly installing the file on build.

Hello @evest & @kennethskantz
I have now released a version 2.8 which contains the fixes listed here.  You can read more here : #244


Additional include for MicrosoftOffice template

Please add ~$*.dot* for temporary Word template files to the gitignore template MicrosoftOffice.

Please submit template changes to https://github.com/dvcs/gitignore


Implement other community profile files in the template

For those files not migrated in #2, make sure they're shipped with the template

Things like CONTRIBUTING.md should be housed in template, while issue templates and such can reside in blakeNaccarato/.github.

We will implement the pull request template in blakeNaccarato/.github, and do contributing guidelines here.

Well, this one is finally complete. I've seen projects tend to move towards a philosophy of "Issues are for bugs and actionable work items only," which is enabled by Discussion forms and other recent GitHub features. Waiting until these features arrived made for less churn in this config, which we'll keep for awhile and see how it works before making any significant changes.
Will need to document the expected Discussion and Issue Labels in repository setup, though, but the labels have been chosen so that a vanilla repo with default labels will still work.


Add visual tests

Fixes #3

This change is

Review status: 0 of 13 files reviewed at latest revision, 1 unresolved discussion.

test/visual/test.js, line 4 at r1 (raw file):

  var demos = [
    {name: 'index', snippetCount: 2}

I would add a description to each test so as dir structure makes more sense than index-1 index-2 ...

Comments from Reviewable

test/visual/test.js, line 4 at r1 (raw file):
Previously, manolo (Manuel Carrasco Moñino) wrote…
I would add a description to each test so as dir structure makes more sense than index-1 index-2 ...

Good idea. Done.

Comments from Reviewable

something is wrong with paper-input/default/edge.png. The 1st field styling looks like focused and there's something like a cursor in the 3rd input

Review status: 0 of 13 files reviewed at latest revision, 1 unresolved discussion, some commit checks failed.

Comments from Reviewable

That's how the demo appears if you open it with Edge

Comments from Reviewable

Weird :) But not in scope of this PR

Review status: 0 of 13 files reviewed at latest revision, 1 unresolved discussion, some commit checks failed.

Comments from Reviewable

build is failing, something missing or you need to update screenshots?

Review status: 0 of 13 files reviewed at latest revision, all discussions resolved, some commit checks failed.

Comments from Reviewable

From my experience, it's better to have wait time 5s to have more stable results. I added the commit and now tests are passing, @tomivirkki please pull the updates.

Review status: 0 of 13 files reviewed at latest revision, all discussions resolved.

Comments from Reviewable

I don't think it was the wait time that caused the test failures but the blinking caret. Your commit includes a screenshot with the caret hidden while the previous reference image had it visible. Need to look into why Edge focuses the demo input automatically.

Comments from Reviewable

Yes, that is the problem, in vaadin-text-field happened the same, finally the only working solution was either configure a no size font (https://github.com/vaadin/vaadin-text-field/blob/master/test/visual/default.html#L22)  (or not focus the input but change the styles setting the focused attribute manually (https://github.com/vaadin/vaadin-text-field/blob/master/test/visual/styling.html#L90). I played with other options like a layer covering the cursor, etc, but does not work in IE.

Review status: 0 of 13 files reviewed at latest revision, all discussions resolved.

Comments from Reviewable

That input should not be focused so there should not be any cursor caret. Also, first input looks like focused but it should not be. Those are issues related to Edge, not to this PR.

Review status: 0 of 13 files reviewed at latest revision, all discussions resolved.

Comments from Reviewable


Summary of Grouper changes for script_run

Summary data should be made available after script_run is completed. This should include the number of adds, drops, and total changes.It should also include the EDS and Grouper queries.
I recommend using a dict type. This allows for easy transfer to a pandas dataframe for a final summary.
This can be implemented in release/v0.14.0

This has been tested and the results are what we expect. Clean summary
09:46:27 -     INFO: SUMMARY DATA
|                  |   num_EDS |   num_Grouper |   adds |   drops |   total |
|:-----------------|----------:|--------------:|-------:|--------:|--------:|
| fine_arts        |       266 |           266 |      0 |       0 |       0 |
| performing_arts  |       452 |           452 |      0 |       0 |       0 |
| architecture     |       175 |           175 |      0 |       0 |       0 |
| arts_design      |        37 |            37 |      0 |       0 |       0 |
| business_econ    |       432 |           432 |      0 |       0 |       0 |
| management       |       260 |           260 |      0 |       0 |       0 |
| economics        |       108 |           108 |      0 |       0 |       0 |
| education        |      1397 |          1396 |      1 |       0 |       1 |
| english          |       296 |           296 |      0 |       0 |       0 |
| lang_culture     |       392 |           392 |      0 |       0 |       0 |
| humanities       |       252 |           252 |      0 |       0 |       0 |
| law              |       387 |           387 |      0 |       0 |       0 |
| nursing          |       322 |           322 |      0 |       0 |       0 |
| med_health       |      2006 |          2006 |      0 |       0 |       0 |
| clinical         |       846 |           846 |      0 |       0 |       0 |
| ped_reprod       |       203 |           203 |      0 |       0 |       0 |
| neurology        |       112 |           112 |      0 |       0 |       0 |
| oncology         |       279 |           279 |      0 |       0 |       0 |
| pharmacology     |       722 |           723 |      0 |       1 |       1 |
| physiology       |       137 |           137 |      0 |       0 |       0 |
| public_health    |       471 |           471 |      0 |       0 |       0 |
| astro            |       609 |           609 |      0 |       0 |       0 |
| cognitive_sci    |       347 |           347 |      0 |       0 |       0 |
| life_sci         |      1733 |          1733 |      0 |       0 |       0 |
| sci_math         |      1868 |          1866 |      2 |       0 |       2 |
| earth_sci        |      1019 |          1016 |      3 |       0 |       3 |
| physics          |       200 |           200 |      0 |       0 |       0 |
| lpl              |       338 |           338 |      0 |       0 |       0 |
| agriculture      |       134 |           134 |      0 |       0 |       0 |
| anthropology     |       211 |           211 |      0 |       0 |       0 |
| social_sci       |       875 |           875 |      0 |       0 |       0 |
| cultural_studies |       168 |           168 |      0 |       0 |       0 |
| history          |        80 |            80 |      0 |       0 |       0 |
| journalism       |       251 |           251 |      0 |       0 |       0 |
| engineering      |      1302 |          1301 |      1 |       0 |       1 |
| technology       |       109 |           109 |      0 |       0 |       0 |


Map breaks when a query does not work

Describe the problem
When a router is down and the query fails to retrieve data, the map crashes.
The issue #40 seems to address the same problem.
To Reproduce
To replicate this behavior is as easy as to add a link connected to a "broken" query.
Expected behavior
When a router or a link is unavailable, the map should remain functioning with the links with broken queries painted black.
Screenshots

**VERSIONS! **

Browser chrome and firefox
Grafana Version 9.3.1
Plugin Version 0.3.5
Influxdb v2

If this is easy to replicate for you, would you be able to send the DataFrame JSON for when this error occurs. At the top of the panel where you click to edit, you can instead click Inspect > Panel JSON and then at the top of the dropdown choose "Dataframe JSON" and just paste it here / attach it if it's too large.

Hi @knightss27 ,
See one part of this export.
I think the problem is when Fields and Values are equal to Null.
[
{
"schema": {
"refId": "CMA1-MXDF2",
"meta": {
"executedQueryString": "from(bucket: "telemetry")\r\n  |> range(start: 2023-03-15T23:40:17.012Z, stop: 2023-03-16T00:10:17.012Z)\r\n  |> filter(fn: (r) => r["_measurement"] == "/interfaces/")\r\n  |> filter(fn: (r) => r["_field"] == "/interfaces/interface/subinterfaces/subinterface/state/counters/out-octets")\r\n  |> filter(fn: (r) => r["device"] == "cma1")\r\n  |> filter(fn: (r) => r["interface_name"] == "et-0/1/9")\r\n  |> filter(fn: (r) => r["subinterface_index"] == "3002")\r\n  |> map(fn: (r) => ({r with _value: r._value * 8}))\r\n  |> derivative(nonNegative: true, unit: 1s)\r\n  |> set(key: "_field", value: "cma1-mxdf2")"
},
"fields": []
},
"data": {
"values": []
}
},

This issue appears to still be present in v0.3.6:
DIA-Weathermap-Error.txt
DIA-Weathermap.json.zip
Error output and dashboard json attached.

Are you able to replicate this easily? And if so write out how to do so?

Are you able to replicate this easily? And if so write out how to do so?

Just create a query with a non-existing interface to have a return equals to empty [].

With InfluxDB datasource? Or can you replicate it with a different datasource?

Only with InfluxDB datasource? Or can you replicate it with a different datasource?

I use only InfluxDB.

I just updated my plugin to version 0.4.2 and the issue is still occurring.
The problem is that when a router goes down, due to a connectivity problem, the queries to my database (InfluxDB) get no data in response, and then crashes the plugin.
There should be a condition like that:
if there is no data returned to a query, displays n/a in the link. This already happens when we hide a query.

I think I actually fixed it finally. Sneaky little error (actually tested it with an InfluxDB instance to find this). Will submit the next release to Grafana early this week.

@ladvocat Have you had the time to update? Does this appear fixed to you?

@knightss27 Awesome! It worked! Thanks a lot!


fix issue #88 by fixing the path to bundle.js in index.html,

and remove the generated file bundle.js from source control for cleanliness sake

Codecov Report

Merging #90 into master will not impact coverage.

@@           Coverage Diff           @@
##           master      #90   +/-   ##
=======================================
  Coverage   77.23%   77.23%
=======================================
  Files         112      112
  Lines        2719     2719
  Branches      278      278
=======================================
  Hits         2100     2100
  Misses        413      413
  Partials      206      206

Continue to review full report at Codecov.

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

Changes look good!


Suggestion: single batch action after all effects are resolved

Currently batch(effects) is dispatching the resulting actions once all effects are resolved.
Proposal:
In addition to sending invidual actions when effects are resolved, also send one single action after all effects are resolved.
Motivation:
Imagine you have collection (list) of items you want to refresh all at once, but they don't have API to do it in one request so you need to invoke API request for all of them one by one. You would like to start spinner before actual request are initiated and stop spinner after all effects are resolved.
Currently this is very hard achievable as there is no action to be added indicating all effects were resolved.
Sample code:
ACTIONS
export function refreshAllDeliveries() {
  return {
    type: ActionTypes.DELIVERY_REFRESH_ALL
  };
}

// fetches single delivery
export function fetchDelivery(trackingNr, pinCode, color) {
  return api.findDelivery(trackingNr, pinCode)
  .then((response) => ({
    type: ActionTypes.DELIVERY_FETCH_SUCCESS,
    payload: {
      trackingNr,
      pinCode,
      ...response.data,
      lastUpdated: Date.now(),
    }
  }))
  .catch((response) => ({
    type: ActionTypes.DELIVERY_FETCH_FAILURE,
    payload: response
  }));
}

REDUCER
...
case ActionTypes.DELIVERY_REFRESH_ALL:
  // create batch from deliveries to handle situations where API only allows retrieval of items one by one
  const batch = values(state.deliveries).reduce(
    (deliveriesToRefresh, delivery) => {
      if (['UNSPECIFIED_TIME_WINDOWS', 'SCHEDULED', 'ASSIGNED'].indexOf(delivery.state) > -1) {
        const promise = Effects.promise(
          fetchDelivery,
          delivery.trackingNr,
          delivery.pinCode,
          delivery.color
        )
        deliveriesToRefresh.push(promise);
      }
      return deliveriesToRefresh;
    }, []
  );
  return loop(
    state,
    Effects.batch(batch)
  );
...

React component where we call refreshAllDeliveries and have spinner
class DeliveriesPage extends Component {
  render() {
    const isFetching = this.props.isFetching;

  render() {
    const isFetching = this.props.isFetching; //passed react-redux mapStateToProps

    return (
      <div>
        <Refresh
          isRefreshing={isFetching}
          onClick={this.props.refreshAllDeliveries}
        />
      </div>
  }
}

why not just add an Effects.constant at the end of the array passed to batch?

@bdwain that is actually great advice. Will try that.


Feature Request: Comments from Mastodon

Hello! First off, I love this theme and am very grateful that you've shared it.
I've seen other people add Mastodon-powered comments to their blogs (any blogs, not necessarily Astro) and I think it's a great idea. I would try to do it myself, as I've already forked and customised the project a bit, but I am Terrible At Design and don't really know what I'm doing.
This person seems to have made it work: https://sam.pikesley.org/blog/2023/03/06/mastodon-powered-comments-in-astro/
From your comments on the search feature request, I understand that you might not want to include a Mastodon-specific comments component, but I'm curious to hear your thoughts either way.
Edit: this person has gone further and combined Mastodon comments and webmentions! https://kld.dev/adding-webmentions

Hi @kdmcguire, thanks, and appreciate the suggestion. Atm I'm holding off on adding stuff in except simple fixes due to astro about to hit v3.0 so I'm hoping nothing is going to break when updated. Once that's done, I will definitely circle back round and take a look at this.

Hi @kdmcguire, I've created a working branch for integrating webmentions to the theme.
I'm hoping to merge it with main soon but it's quite hard to test due to lack of actual webmentions the site has. If you, or anyone else reading this, get a chance to test this out and give some feedback it would really help. I've created a blog post to accompany this which should hopefully help in setting it up.

Ah, amazing! How can I help test it? If I posted on Mastodon with a link to that post, would it appear in the 'Webmentions for this post' list?

This seems to work perfectly!
https://deploy-preview-198--kieran-mcguire.netlify.app/posts/monday-ritual/

Brilliant, thanks for letting me know.
Ah yeah apologies, that was me trying to be clever and thinking it would be easier to set everything in that file, site.config.ts. Unfortunately, astro doesn't like people doing that and blows up, I just forgot to change the comment back.


Change getOutgoingUrls() return type from Set to List

I think it does not make sense to store outgoing links as Set instead of List, see ParseData.getOutgoingUrls(), cause it breaks the order, whereasFrontier.scheduleAll(List<WebURL> urls) has List in method declaration.
In my current situation order matters.

Avoiding duplicate URLs -> Set


[Bug]: Unable to authorize Gsheets DS on selecting "Selected GSheets"

Is there an existing issue for this?

[X] I have searched the existing issues

Description
When I try to authorize Gsheets using the Selected GSheets option, It does not work. It defaults to All GSheets, and then when we retry on edit, it gives an error.
Steps To Reproduce
https://github.com/appsmithorg/appsmith/assets/101863839/1ac74327-b9a9-4c0f-a165-a14d5bcd0f3f
Public Sample App
No response
Environment
Release
Issue video log
No response
Version
Cloud

@sneha122 - A small step that is mandatory here, is create a new gsheets DS, select the permission as Selected Google Sheets, and then rename the GSheets DS to some name, and then click Save and Authorize.

After multiple discussions with @sneha122 - since we are unable to reproduce this consistently, reducing the priority to high.

Bumping this issue up to prod because it should have made it's way there by now if it was not fixed

marking as medium effort since reproducing the issue itself can be complicated


⚠ Bofrost.es has degraded performance

In b3007a7, Bofrost.es (https://www.bofrost.es) experienced degraded performance:

HTTP code: 200
Response time: 1691 ms

Resolved: Bofrost.es performance has improved in 4543965 after 13 minutes.


fix: support native esm test files

This change is

I've tried this locally on a project with the latest mocha and this was enough for it to work with native module tests.


v3 USelectMenu webCrypto.randomUUID is not a function

Environment

Operating System: Windows_NT
Node Version:     v20.9.0
Nuxt Version:     3.14.1592
CLI Version:      3.16.0
Nitro Version:    2.10.4
Package Manager<EMAIL_ADDRESS>Builder:          -
User Config:      default
Runtime Modules<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>Build Modules:    -

Is this bug related to Nuxt or Vue?
Nuxt
Version
3.0.0-alpha.10
Reproduction

Description
I am getting this error when using USelect or USelectMenu when accessing in development mode from a url other than localhost when using the npm run dev -- --host command may be because I am not using http

Additional context
No response
Logs

I have run my application with the command
npm run dev -- --https --host
https://developer.mozilla.org/en-US/docs/Web/API/Crypto/randomUUID
Secure context: This feature is available only in secure contexts (HTTPS), in some or all supporting browsers.


Initial value does not get set on remote typeahead when multiple remote typeaheads are displayed

The setting of a initial value of the remote typeaheads do not work when multiple remote typeaheads are displayed on one page.
I provided an example branch with the name (feat/remote_typeahead_initialization_bug) in this repository.
On initial run of the application sometimes it works. But when you do a reload of the page it does not work anymore.
Hint:
I did a small analysis and could be that this function is the reason for the issue:
 updateInputValue(value) {
    if (this.updateInputValueTimer) {
      clearTimeout(this.updateInputValueTimer);
    }
    this.updateInputValueTimer = setTimeout(() => {
      if (!lodash_isnil(this.inputElement)) {
        this.inputElement.value = value;
        this.inputValue = value;
      }
    }, 0);
  }

since it uses timeoutes maybe it is a timing issue.

Will investigate this one, roughly mid of next week

@nobilo & @m4rc0z could you retest that with the newest version?

if i have a look at the created branch version with the usage of the new design system version 10.24.9 the values are still not initialized.

@m4rc0z can u test this version<EMAIL_ADDRESS>


Test against multiple GHC and resolvers

(This might mean changing from CircleCI to Travis CI.)

the nice thing is hackage apparently does it on package upload, for a bunch of GHC versions:
https://matrix.hackage.haskell.org/package/slack-web
and of course stackage is doing it also when the package is published, for their nightly setup.
but yeah on package upload to hackage is too late to be doing it. Won't have time to work on that myself though.