content store encryption and full text search

Hi afaust,
Does full text search still work if i implement content store encryption?
because i think that solr does not support indexing encrypted files
I made a small POC yesterday and i tested it with a file that i encrypted with AES and i also encrypted the word i wanted to search with the same key, i then tried to find the file by typing the encrypted word in the Alfresco full text search bar but no results.
I then looked in the solr indexing folder of alfresco but i found that my encrypted file was not indexed.
Yours sincerely

It depends on how / where you encrypt the file. If the file BEFORE uploading to Alfresco, you will have have issues dealing with it, e.g. searching for content in it. If you use an encrypting content store within Alfresco which is transparent to indexing, it will not create problems.

ok but even within Alfresco the solr indexing files will stay unsecured (because it's not encrypted) is that right?

Right - they are low level binary files that are heavy in-use and constantly encrypting/decrypting those files would extremely impact the performance of SOLR.

Okay, thank you for your responsiveness but if what i understood is correct:
Alfresco maps contentstore files to solr indexing files using a map (stored at postgresql database) to retreive files and their metadata. right?
if i use full text search, alfresco should show me all the thumbnails of the files that contain my word, so it should match the indexing files to their contentstore files. right?
but how can i ensure this matching if my content store is encrypted within Alfresco?
otherwise in which part of the process is decryption made?

With a lot of flexibility in interpretation, the descriptions might be considered correct. Technically, SOLR will fetch data from Alfresco. There is no direct mapping of any kind except that the ID of a node (document / folder) in Alfresco is included as an indexed field, and that alone allows for an association. Content files are not directly mapped to indexing files as-is - they are extensively processed to split their content into tokens / fragments. It is extremely cumbersome to reconstruct a document's content from the indexing files alone - it would be easier to just retrieve that from the SOLR content cache (Alfresco-specific, compressed storage to avoid too many DB lookups if not necessary for index updates).
If a file is stored using an encrypted content store, SOLR will get the unencrypted content via the Alfresco API used to fetch data for indexing. The encrypted content store is an active component within Alfresco which encrypts files during write to disk and decrypts them during read - all other components deal with the unencrypted content in transit / memory.

thanks Mister Faust,
I understand better now the architecture of Alfresco. Everything is based on the node id and every node (file or folder) can have many properties and can also have relation with another node. Right?
but I still have one last question that occupies my mind: How is the link made between a File Node and its Physical File?
I have checked and found that the File Node ID and  the physical file ID of my uploaded file are not the same.
So when we read a file how do we get it back from Content Store?

In the Alfresco database, there is a relation between the node (id) and a content URL - the content URL essentially is a handle for a content store to resolve to a file. By default, the content URL will be a relative path to the physical file, but it may also be something completely different and it is up to the content store implementation to resolve it.

@AFaust how about closing this issue?
It seems it is not related to any real issue with the addon.

@douglascrp You are right about that - since I have had a couple of issues where follow-up questions came a couple of weeks / months after initial filing, I have gotten into the habbit of not closing them as soon as "I felt" they were properly answered.


Is it really file transfer tool based on webRTC ?

Hi,
I have tested the user-end functionality. It involves a simple process of uploading files and downloading them through direct links. Could you please provide more details? I'm actually searching for a peer-to-peer file sharing platform
Here is my idea
First, I'll upload a file to an online server (FileManager), and it will generate a unique URL for that file. This URL will be shared on any website / social media ETC
When the first user decides to download the file, it will be retrieved directly from the server since only one peer is involved.
As more users come in to download the file, the number of peers increases. For instance, when the second user arrives, the file will have two peers: the server and the first user who downloaded it. This means the second user will download the file from both the server and the first user.
Similarly, when a third user arrives, there will be three peers: the server, the first user, and the second user. This setup grows with more users, resulting in multiple peers for each download.
So if no one is downloading, MY server will always be available as the default peer. This idea is to make shared file peer active for a lifetime

Maybe your need BitTorrent


Albumentations and imgaug versions

When cloning into the tutorial ipynb pip throws an error during dependency installation in the first cell.

ERROR: pip's dependency resolver does not currently take into account all the packages that are installed. This behaviour is the source of the following dependency conflicts.
albumentations 0.1.12 requires imgaug<0.2.7,>=0.2.5, but you have imgaug 0.2.9 which is incompatible.

Maybe specify versions in pip call to resolve?

This issue occurs because of incompatible packages required for the notebook. This message will continue to occur but causes no errors downstream.


Set up CI

It's time we start continuously testing this repository to ensure we don't break anything in the future.

[ ] Implement testing matrix using nox & nox-poetry
[ ] Implement continuous integration using Github Actions
[ ] Integrate with codecov for coverage reporting
[ ] Integrate with deepcode for code improvement suggestions

Why nox and not tox? Why not try and keep it similar to celery?

Why nox and not tox? Why not try and keep it similar to celery?

can github action parse nox files?

can github action parse nox files?

Why nox and not tox? Why not try and keep it similar to celery?

nox provide python file like syntax https://nox.thea.codes/en/stable/tutorial.html :D if this goes well we might consider changing celery too.

Why nox and not tox? Why not try and keep it similar to celery?

nox provide python file like syntax https://nox.thea.codes/en/stable/tutorial.html :D if this goes well we might consider changing celery too.

I don't profess to be an expert. I only setup one package on GH actions about a month or two ago. But it seems similar enough to doing GL or Travis where you are defining the container that runs everything in. So, as long as the environment is setup, I see no reason why it couldn't run nox.

I don't profess to be an expert. I only setup one package on GH actions about a month or two ago. But it seems similar enough to doing GL or Travis where you are defining the container that runs everything in. So, as long as the environment is setup, I see no reason why it couldn't run nox.

nox allows us to programmatically introduce network partitions using blockade or something like seaworthy.
tox-docker is far less flexible.

nox allows us to programmatically introduce network partitions using blockade or something like seaworthy.
tox-docker is far less flexible.

@ahopkins If you think tox is what we should be using in the future, now's the time to make the argument for it.

@ahopkins If you think tox is what we should be using in the future, now's the time to make the argument for it.

I am not familiar with blockade or seaworthy. I have used tox-docker with success.
My comment was based upon two things: (1) celery uses tox, and (2) tox is widely known and adopted in the Python community (not really a reason to shirk it). If the idea is to use this as an experiment before switching celery, then go for it. I have no objection.

I am not familiar with blockade or seaworthy. I have used tox-docker with success.
My comment was based upon two things: (1) celery uses tox, and (2) tox is widely known and adopted in the Python community (not really a reason to shirk it). If the idea is to use this as an experiment before switching celery, then go for it. I have no objection.

@ahopkins Very large projects like salt and urllib3 use nox.
There's no reason not to try it. If it works well, we can replace tox with nox without fear.

@ahopkins Very large projects like salt and urllib3 use nox.
There's no reason not to try it. If it works well, we can replace tox with nox without fear.

just sharing a thing from my little knowledge reading with github action doc so far, even without tox/nox we can run CI/CD in github action using their built-in features as well. [doing that in django-activity-stream & might try in celery projects]

just sharing a thing from my little knowledge reading with github action doc so far, even without tox/nox we can run CI/CD in github action using their built-in features as well. [doing that in django-activity-stream & might try in celery projects]


Brown face NPC in whiterun

Describe the bug
One of the NPCs in whiterun has a brown face bug
To Reproduce
Just find the NPC
Expected behavior
Can it be corrected?
Screenshots
If applicable, add screenshots to help explain your problem. I highly recommend opening the console and clicking on the object that has a bug. If your bug is location based, use the book from this mod: https://www.nexusmods.com/skyrimspecialedition/mods/30907.

Fixed


[CIS v2] ClusterScanReport CRD - List and Detail Page - Needs to add download report

ClusterScanReport CRD Explanation:
Once a ClusterScan is completed, the report from the kube-bench tool is stored in a ClusterScanReport instance.
ClusterScanReport  CR fields:

BenchmarkVersion string json:"benchmarkVersion,omitempty"
LastRunTimestamp string             yaml:"last_run_timestamp" json:"lastRunTimestamp"
ReportJSON string json:"reportJSON"

UI options:

Report CR available after ClusterScan finishes
View detail report can consume the ReportJSON like in 2.4 to show list of tests passed/skipped and nodes. So we should have a UI to consume the report and show result per test:

Download Report
In 2.4 this was a link to download the reportJSON
Now we are planning to put it in the CRD – do we need special way of downloading only the JSON?

[ ] Remove ability to create ClusterScanReport from list page of ClusterScanReport

Moving this all to be incorporated as part of the CusterScan page


Support for reparenting

React native can move nodes without re-creating it when there placed in the same tree, like <View>{node1}{node2}</View>.
But if we have to re-parenting nodes, that will cause re-creating. For example, we want to move node1 into the Modal, node1 will be re-creating.
<View>
  {node1}
  {node2}
  <Modal></Modal>
</View>

The states of node1 will be purge as the old node1. In most cases it's not a big problem, we can maintain states in the root view, and pass it as props to subnodes, but we offload some expensive view or task into native ui component, it's a problem. For example:
<View>
  {fullscreen ? <Modal>{player}</Modal> : (
    <View>
      {player}
      <Text>{video.title}</Text>
    </View>
  )}
</View>

We solved the problem by create a wrapper for the native component.
let uid = 0;
class VideoPage extends React.Component {
  _uid = uid++;
  render() {
    const player = <Player {...this.playerProps} reuseToken={this._uid};
    return (
      <View>
       {fullscreen ? <Modal>{player}</Modal> : player}
      </View>
    )
  }
}

ViewManager.m:
- (UIView)view {
  return [Wrapper new];
}
RCT_EXPORT_METHOD(destroy:(nonnull NSNumber*)reactTag) {
  [self.bridge.uiManager addUIBlock:^(RCTUIManager* manager, NSDictionary* registry) {
    WrappedPlayer* view = registry[reactTag];
    if ([view isKindOfClass:[WrappedPlayer class]]) {
      [view destory];
    }
  }];
}
RCT_EXPORT_PROPERTY(reuseToken, NSNumber *);

PlayerWrapper.m:
static NSDictionary<WrappedPlayer *> * players;
- (void) setReuseToken:(nonnull NSNumber *) token {
  _reuseToken = token;
  _player = players[token];
  if (player == nil) {
    _player = [WrappedPlayer new];
    players[reuseToken] = _player;
    [self initPlayer];
  }
}
- (void)destory {
  if (_player) {
    [players deleteObject:_reuseToken];
    [_player destory];
  }
}

But it would be better to have some help from react native core.

Thanks for opening the issue, however it looks like a feature request. As noted in the Issue template we'd like to use the GitHub issues to track bugs only. Can you implement the feature as a standalone npm module? If not consider sending a pull request or a create an entry on Canny. It has a voting system and if the feature gets upvoted enough it might get implemented by someone from the community. Closing this now, thanks for understanding!


[ZEPPELIN-1743] Use explicit arguments in functions in paragraph.controller.js

What is this PR for?
This is paragraph.controller.js maintenance PR. It refactors most functions in paragraph controller to accept explicit parameters instead of getting data from $scope. It is a first - simple and safe - step in making paragraph controller more maintainable.
What type of PR is it?
Refactoring
Todos

[x] - Call functions explicitly with parameters when makes sense
[x] - Do minor refactoring when makes sense

What is the Jira issue?
https://issues.apache.org/jira/browse/ZEPPELIN-1743
How should this be tested?
Make sure that paragraph works as expected by interacting with it;

Looks great to me, thank you @felizbear for frontend code improvements!
Let's wait for CI, meanwhile, would you mind creating a JIRA issue please?

LGTM!

Looks great to me!
@felizbear In my experience, if the contribution author followup on CI failure - that speeds up the merge time tremendously.
It looks like at least CI is failing on npm run test (which fails on the grunt test? ), could you please double-check\digg into it?
ERROR]
[ERROR] npm ERR! Linux 4.8.7-040807-generic
[ERROR] npm ERR! argv "/home/travis/build/apache/zeppelin/zeppelin-web/node/node" "/home/travis/build/apache/zeppelin/zeppelin-web/node/node_modules/npm/bin/npm-cli.js" "run" "test"
[ERROR] npm ERR! node v6.9.1
[ERROR] npm ERR! npm  v3.10.8
[ERROR] npm ERR! code ELIFECYCLE
[ERROR] npm ERR<EMAIL_ADDRESS>test: `grunt test`
[ERROR] npm ERR! Exit status 3
[ERROR] npm ERR!
[ERROR] npm ERR! Failed at the<EMAIL_ADDRESS>test script 'grunt test'.
[ERROR] npm ERR! Make sure you have the latest version of node.js and npm installed.
[ERROR] npm ERR! If you do, this is most likely a problem with the zeppelin-web package,
[ERROR] npm ERR! not with npm itself.
[ERROR] npm ERR! Tell the author that this fails on your system:
[ERROR] npm ERR!     grunt test
[ERROR] npm ERR! You can get information on how to open an issue for this project with:
[ERROR] npm ERR!     npm bugs zeppelin-web
[ERROR] npm ERR! Or if that isn't available, you can get their info via:
[ERROR] npm ERR!     npm owner ls zeppelin-web
[ERROR] npm ERR! There is likely additional logging output above.
[ERROR]
[ERROR] npm ERR! Please include the following file with any support request:
[ERROR] npm ERR!     /home/travis/build/apache/zeppelin/zeppelin-web/npm-debug.log
[ERROR] Failed to execute goal com.github.eirslett:frontend-maven-plugin:1.3:npm (npm test) on project zeppelin-web: Failed to run task: 'npm run test' failed. (error code 1) -> [Help 1]
[ERROR]
[ERROR] To see the full stack trace of the errors, re-run Maven with the -e switch.
[ERROR] Re-run Maven using the -X switch to enable full debug logging.

this backend test currently fails
- should provide onclick method *** FAILED ***

  The code passed to eventually never returned normally. Attempted 1 times over 516.964883 milliseconds. Last failure message: 0 was not equal to 1. (AbstractAngularElemTest.scala:64)

investingating why

All checks have passed. Let's merge it please

Thank you @felizbear !
Merging to master if there is no further discussion


Do not map  (Return/Enter)

Use CTRL-J and CTRL-K (or CTRL-N and CTRL-P) to change the selection

I have recently started the habit of using <C-j> instead of Enter. It would be nice if it worked in fzf, too.
Can this be configured?

export FZF_DEFAULT_OPTS='--bind ctrl-j:accept'

:)


[RLMArray][List] Implement Move method

To support drag'n'drop reordering in lists we need a move method.
This is needed by sync as a move is semantically very different from a delete and insert. It is also important to avoid triggering cascading deletes when we start supporting that.
- (void)moveObjectFromIndex:(NSUInteger)from toIndex:(NSUInteger)to;

Does list_list already implement the necessary methods?

The LinkView class in Core already has a move() method. I guess it
just have to be exposed at the binding level.


improve auditability of tag history

Add the calling Service-Account to the tag_history table, so that the audits can show who changed a tag

Done


Create dataset loader for Thai Gov v2 Corpus

Dataloader name: thai_gov_v2/thai_gov_v2.py
DataCatalogue: http://seacrowd.github.io/seacrowd-catalogue/card.html?thai_gov_v2

Dataset
thai_gov_v2

Description
The dataset consists of individual news articles, each corresponding to a unique URL at the Thai government website (https://www.thaigov.go.th/). The dataset structure is as follows: a topic header is followed by the content of the news article, which is then succeeded by a blank line and the source URL

Subsets
-

Languages
tha

Tasks
Language Modeling

License
Open Data Commons Public Domain Dedication and License (pddl)

Homepage
https://github.com/PyThaiNLP/thaigov-v2-corpus/tree/master/data

HF URL
-

Paper URL
-

#self-assign


SQL Server CU must be installed to fix broken path

While going through the tutorial i encountered

CREATE FILE encountered operating system error 5(Access is denied.) while attempting to open or create the physical file '

I noticed that the path was missing a backslash. The bug report 384 did contain the same problem. The advice was to update to the latest SQL Server® 2017 for Microsoft® Windows Latest Cumulative Update . This fixed the problem.
Suggestion: add a note to the tutorial to install the latest updates.

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

ID: 6719f08e-3bd7-dc1a-71df-f2ef9fbca9d8
Version Independent ID: 7096fdb3-612e-9e00-bd0b-8ea4886a09ce
Content: Add a model to a Razor Pages app in ASP.NET Core
Content Source: aspnetcore/tutorials/razor-pages/model.md
Product: aspnet-core
Technology: aspnetcore-tutorials
GitHub Login: @Rick-Anderson
Microsoft Alias: riande

Dup of #15680


Reverting a commit that broke signature widget's background color

Probably a way to do it without reverting all the other changes that didn't break anything. The two things I broke were the background color in draw activity and startActivity instead of startActivityForResult


Update in to swift 5

The master has been updated to Swift 5, will do a release soon, sorry for the delay.


if zip isn't on our database, first search for it in google maps api and add it.

Seems like a good one for me.


If user press backspace it continiously add "with region text" in the edit text if "withMask" didnt start with white space.

@nexuspay i will try to fix at this week.


[1.0Xpt] Front-end: Initial set-up

create a repo to front-end with React
Install dependencies.
Add basic redux structure using the ducks pattern.
Add basic routing structure using react-router-dom.
Install the project's base dependencies and linters for code best practices.


Feature request: Alert using a dialog

For my purpose, I added a zenity dialog on timeout. Would be nice to have it inbuilt. Also, thanks for this app.

Agreed. I'm planning to update the code to use the below package, would love for it to be built-in.
https://github.com/deckarep/gosx-notifier

Can you look at https://github.com/ctcpip/notifize. very simple code.
func Display(summary string, body string, isUrgent bool, iconPath string) {

	switch runtime.GOOS {

	case "darwin":

		exec.Command("osascript", "-e", "display notification \""+body+"\" with title \""+summary+"\"").Run()

	case "linux":

		if isUrgent {
			exec.Command("notify-send", "-i", iconPath, summary, body, "-u", "critical").Run()
		} else {
			exec.Command("notify-send", "-i", iconPath, summary, body).Run()
		}

	case "android", "dragonfly", "freebsd", "nacl", "netbsd", "openbsd", "plan9", "solaris", "windows":
		// not implemented

	}

}

This might be fixed by #6 Thanks!


Load a set of predefined labels

This pull request add ability to load a set of predefined labels (Issue #1562). This feature is really useful for user who create same labels on a lot of repositories.
Default file
The default set of predefined labels only contains:
#e11d21 bug
#009800 enhancement
#eb6420 task
#fbca04 suggestion
#000000 won't fix

Customization:
User can customize available label sets by adding files on custom/conf/label directory. The file must respect the following format : <html-color><space><label-name>
Preview:
In the following screenshots, Github and Gitlab presets are not included on this pull request. Their two files are only on my own custom/conf/label directory.

Thanks :)

Seems having conflict..

I fix this today

doesn't seem fixed...?

Its due to some files modified by previously accepted merge requests since yesterday. Fix push.

Hmm, I'm sorry but we have conflict again..

Thanks!


Error when adding a row

Error when adding a row to the table. Data is stored, but application reports an error.

Will be fixed in version 2.5.1

Fixed in 2.5.1


SR-15316: Emit warning when topics section does not have 1+ subheading

Bug/issue #, if applicable: SR-15316
Summary
Provide a description of what your PR addresses, explaining the expected user experience.
Also, provide an overview of your implementation.
A Topics section must have at least one subheading, otherwise, it is omitted. This PR aims to add a checker that will emit a warning any time a Topics section does not have at least one subheading.
Dependencies
Describe any dependencies this PR might have, such as an associated branch in another repository.
No dependencies
Testing
Describe how a reviewer can test the functionality of your PR. Provide test content to test with if
applicable.
Steps:

Provide setup instructions.
Explain in detail how the functionality can be tested.

Checklist
Make sure you check off the following items. If they cannot be completed, provide a reason.

[x] Added tests
[x] Ran the ./bin/test script and it succeeded
[x] Updated documentation if necessary

@d-ronnqvist I ended up renaming the checker to TopicsSectionWIthoutSubheadings, what are your thoughts on that?

I ended up renaming the checker to TopicsSectionWIthoutSubheading; what are your thoughts on that?

That sounds like a perfect name. It clearly described what it is checking and in what context.

@swift-ci please test


Demo: Mouse Tracker demo throws StackOverflowException (not reproducible?)

Bug Report
Issue: StackOverflowException while dragging "Mouse Tracker" demo's graph.
Stack trace:
[External Code]
ScottPlot.FormsPlot.Refresh(lowQuality, skipIfCurrentlyRendering) Line 158
ScottPlot.Demo.WinForms.WinFormsDemos.MouseTracker.formsPlot1_MouseMoved_1(sender, e) Line 40
ScottPlot.FormsPlot.PictureBox1_MouseMove(sender, e) Line 205
[External Code]
...repeated...

Reproducing:

Start WinForms demo project
Click "Mouse Tracker" button
Drag around the graph until the exception is thrown (left button, press it and don't release).

System Details

ScottPlot Version: main@a3b5909 (4.1.49)
Operating System: Windows 10
Application Type: WinForms
.NET Version: .NET 6.0

Thanks for reporting this @AbeniMatteo!
Unfortunately I can't seem to recreate this bug on my system... so it may be extremely challenging to diagnose.
If this is reliably reproducible on your system, could you download this repository and run the demo application from its source code? Hopefully the exception thrown that way will be more informative.
If I don't hear back within several days I'll probably close this issue since I can't recreate it, but I welcome input from anyone else who's experienced this issue. It's possible it's specific to a certain graphics driver's interaction with GDI+, and I've seen some weird bugs improved by updating graphics drivers (e.g., #1600, #1733).

Seems like the stacks never pops while looping:
formsPlot1_MouseMoved_1 > formsPlot1.Refresh() > Application.DoEvents() > back to formsPlot1_MouseMoved_1
I'm already running the demo from its source with VS, commit a3b5909.
This is the stack trace that repeats (with external code this time):
ScottPlot.Demo.WinForms.WinFormsDemos.MouseTracker.formsPlot1_MouseMoved_1(sender, e) Line 40
System.Windows.Forms.Control.OnMouseMove(e)
ScottPlot.FormsPlot.PictureBox1_MouseMove(sender, e) Line 205
System.Windows.Forms.Control.OnMouseMove(e)
System.Windows.Forms.Control.WmMouseMove(m)
System.Windows.Forms.Control.WndProc(m)
System.Windows.Forms.Control.ControlNativeWindow.OnMessage(m)
System.Windows.Forms.Control.ControlNativeWindow.WndProc(m)
System.Windows.Forms.NativeWindow.Callback(hWnd, msg, wparam, lparam)
[Native to Managed Transition]
[Managed to Native Transition]
System.Windows.Forms.Application.ComponentManager.Interop.Mso.IMsoComponentManager.FPushMessageLoop(dwComponentID, uReason, pvLoopData)
System.Windows.Forms.Application.ThreadContext.RunMessageLoopInner(reason, context)
System.Windows.Forms.Application.ThreadContext.RunMessageLoop(reason, context)
System.Windows.Forms.Application.DoEvents()
ScottPlot.FormsPlot.OnBitmapUpdated(sender, e) Line 191
ScottPlot.Control.ControlBackEnd.Render(lowQuality, skipIfCurrentlyRendering) Line 367
ScottPlot.Control.ControlBackEnd.ProcessEvent(uiEvent) Line 600
ScottPlot.Control.ControlBackEnd.MouseMove(input) Line 566
ScottPlot.FormsPlot.PictureBox1_MouseMove(sender, e) Line 205
System.Windows.Forms.Control.OnMouseMove(e)
System.Windows.Forms.Control.WmMouseMove(m)
System.Windows.Forms.Control.WndProc(m)
System.Windows.Forms.Control.ControlNativeWindow.OnMessage(m)
System.Windows.Forms.Control.ControlNativeWindow.WndProc(m)
System.Windows.Forms.NativeWindow.Callback(hWnd, msg, wparam, lparam)
[Native to Managed Transition]
[Managed to Native Transition]
System.Windows.Forms.Application.ComponentManager.Interop.Mso.IMsoComponentManager.FPushMessageLoop(dwComponentID, uReason, pvLoopData)
System.Windows.Forms.Application.ThreadContext.RunMessageLoopInner(reason, context)
System.Windows.Forms.Application.ThreadContext.RunMessageLoop(reason, context)
System.Windows.Forms.Application.DoEvents()
ScottPlot.FormsPlot.Refresh(lowQuality, skipIfCurrentlyRendering) Line 158
ScottPlot.Demo.WinForms.WinFormsDemos.MouseTracker.formsPlot1_MouseMoved_1(sender, e) Line 40

And this is what i do (video ends when the exception is thrown and VS debugger stops the execution):
https://user-images.githubusercontent.com/634710/175782418-7b26ec90-58b8-4462-8695-2ce40229b8bc.mp4

This is a minimal repro, it's not limited to the demo:
var ctrl = new ScottPlot.FormsPlot { Dock = DockStyle.Fill };
var cross = ctrl.Plot.AddCrosshair(0, 0);
ctrl.MouseMove += (sender, e) =>
{
    var (mx, my) = ctrl.GetMouseCoordinates();
    cross.X = mx;
    cross.Y = my;
    ctrl.Refresh();
};

var form = new Form { Width = 500, Height = 500 };
form.Controls.Add(ctrl);

form.ShowDialog();

Thanks for the additional information @AbeniMatteo! This was really helpful.
I was able to recreate this error with your minimal demo (not the mouse tracker demo though)
If your minimal example is modified from:
ctrl.Refresh();

To instead call:
ctrl.Refresh(skipIfCurrentlyRendering: true);

it seems to work on my machine (similar to the mouse tracker demo). Does your machine continue to throw the stack overflow exception with this modification?
Thanks for your patience and help figuring this one out with me!

The stack keeps growing when the message loop can't process messages fast enough because Application.DoEvents() returns only when there are no more messages to handle
No need to drag the graph, you can simply move the mouse over the control
You probably can't reproduce the problem because your system is faster then mine\different mouse dpi settings\different resolution
skipIfCurrentlyRendering: true (also used in the demo) reduces the time spent rendering giving more time to the message loop to consume messages (this is the reason you can't reproduce the problem using the demo)

With this new repro you can directly see the stack depth as a graph:

Tweak Thread.Sleep() to slow the loop (1-2ms should be enough))
I crash at depth ~6500
The graph is updated only when you move the mouse over it

var ys = new double[500];

var ctrl = new ScottPlot.FormsPlot { Dock = DockStyle.Fill };
ctrl.Plot.YAxis.Label("Depth");
ctrl.Plot.XAxis.Label("Time");
ctrl.Plot.XAxis.Ticks(false);
ctrl.Plot.SetAxisLimits(0, ys.Length, 0, 7000);
ctrl.Plot.AddSignal(ys);
var line = ctrl.Plot.AddVerticalLine(0);

var x = 0;
ctrl.MouseMove += (sender, e) =>
{
    ys[x] = new System.Diagnostics.StackTrace().FrameCount;
    line.X = x;
    x = (x + 1) % ys.Length;

    ctrl.Refresh();
    Thread.Sleep(0);
};

var form = new Form { Width = 500, Height = 500 };
form.Controls.Add(ctrl);

form.ShowDialog();

You probably can't reproduce the problem because your system is faster then mine\different mouse dpi settings\different resolution

Makes sense!

ys[x] = new System.Diagnostics.StackTrace().FrameCount;

I love how you're using a plot to graph data about the plotting bug 🤣

The stack keeps growing ... simply move the mouse over the control

Do you have a suggestion about how to fix this issue?
I could modify the shared control back-end code (used by all controls) to skip the render if the stack is too deep:
https://github.com/ScottPlot/ScottPlot/blob/da2e321af86e31ed3a3c7d39ee53675049d30da9/src/ScottPlot4/ScottPlot/Control/Backend.cs#L598-L600
Alternatively I could skip DoEvents() inside the WinForms control if the stack is too deep:
https://github.com/ScottPlot/ScottPlot/blob/da2e321af86e31ed3a3c7d39ee53675049d30da9/src/ScottPlot4/ScottPlot.WinForms/FormsPlot.cs#L191

Alternatively, the skipIfCurrentlyRendering argument could signal to also check for stack depth and skip if too deep.
https://github.com/ScottPlot/ScottPlot/blob/da2e321af86e31ed3a3c7d39ee53675049d30da9/src/ScottPlot4/ScottPlot/Control/Backend.cs#L323-L330

Is this actually a ScottPlot issue? I created a Form with no plot and recreated this issue.
public partial class Form1 : Form
{
    public Form1()
    {
        InitializeComponent();
        this.MouseMove += FormsPlot1_MouseMove;
    }

    private void FormsPlot1_MouseMove(object sender, MouseEventArgs e)
    {
        int depth = new System.Diagnostics.StackTrace().FrameCount;
        Application.DoEvents();
        System.Threading.Thread.Sleep(1);
        label1.Text = $"Stack depth: {depth}";
        progressBar1.Value = depth;
    }
}

I'm starting to think the only solution here is to wrap any call to Application.DoEvents() in some logic that checks how deep the stack is. It looks like there are 2 in the WinForms control:
https://github.com/ScottPlot/ScottPlot/blob/da2e321af86e31ed3a3c7d39ee53675049d30da9/src/ScottPlot4/ScottPlot.WinForms/FormsPlot.cs#L156-L161
https://github.com/ScottPlot/ScottPlot/blob/da2e321af86e31ed3a3c7d39ee53675049d30da9/src/ScottPlot4/ScottPlot.WinForms/FormsPlot.cs#L191

Just a quick comment, i'll read all the comments tomorrow.

Is this actually a ScottPlot issue? I created a Form with no plot and recreated this issue.

Yes, because the user is calling .Refresh() to update the graph and ScottPlot does internally something that crashes the program.
Generally, the problem is callingApplication.DoEvents() while already "doing events".
If .Refresh() is always invoked from the main thread i don't think Application.DoEvents() is necessary, as soon the refresh and the event handler (in this case MouseMove) are done, the control returns to the message loop.

If .Refresh() is always invoked from the main thread I don't think Application.DoEvents()

From what I recall I had to add the DoEvents() to achieve proper click-drag behavior. Without it I think the bitmap wasn't updating while dragging objects or maybe it was sending outdated mouse coordinates.

i'll read all the comments tomorrow

No worries! Take your time, and I really appreciate your input. You could skip the text and glance over #1913 which I think solves this issue adequately (preventing the exception), though there's room for optimization.
I'm going to merge it and issue a release tonight. I'll close this issue because I think the exception issue has been solved, but I'm happy to continue to work on this if you have any suggestions for how to improve from here.
Thanks again for your help!


bug: Error in decoration provider vim_lsp_inlayhint.win: Error executing lua: ...acos-arm64/share/nvim/runtime/lua/vim/lsp/inlay_hint.lua:366: Invalid 'col': out of range

Did you check docs and existing issues?

[X] I have read all the LazyVim docs
[X] I have updated the plugin to the latest version before submitting this issue
[X] I have searched the existing issues of LazyVim
[X] I have searched the existing issues of plugins related to this issue

Neovim version (nvim -v)
v0.10.1
Operating system/version
MacOS 14.5
Describe the bug
I run into this issue fairly frequently when editing bouts of code, often because there are errors in the code I'm editing. Running :e will often temporarily solve the issue but not fix it. I've tried removing all custom plugins and .lua config, but the issue remains. Other times, the issue will occur and updating the mason languages will fix it.
I am happy to do more troubleshooting and disable more things, if needed.
Steps To Reproduce

Open a file, and induce some code that causes an error to be shown in the treesitter.
See above.

Expected Behavior
neovim does not throw an error.
Repro
vim.env.LAZY_STDPATH = ".repro"
load(vim.fn.system("curl -s https://raw.githubusercontent.com/folke/lazy.nvim/main/bootstrap.lua"))()

require("lazy.minit").repro({
  spec = {
    { "LazyVim/LazyVim", import = "lazyvim.plugins" },
    -- add any other plugins here
  },
})

I'm unable to reproduce this with the repro.lua file, so this may be an issue on my end. Maybe close for now, and I'll see if I can get it to happen again?

i'm seeing this same issue with a fresh install of LazyVim. here is the full stacktrace:
Error in decoration provider vim_lsp_inlayhint.win:
Error executing lua: ...AD-f55213c/share/nvim/runtime/lua/vim/lsp/inlay_hint.lua:360: Invalid 'col': out of range
stack traceback:
	[C]: in function 'nvim_buf_set_extmark'
	...AD-f55213c/share/nvim/runtime/lua/vim/lsp/inlay_hint.lua:360: in function <...AD-f55213c/share/nvim/runtime/lua/vim/lsp/inlay_hint.lua:314>
	[C]: in function 'nvim_exec2'
	vim/_editor.lua: in function 'cmd'
	.../share/nvim/lazy/nvim-notify/lua/notify/service/init.lua:63: in function 'push'
	...ocal/share/nvim/lazy/nvim-notify/lua/notify/instance.lua:77: in function <...ocal/share/nvim/lazy/nvim-notify/lua/notify/instance.lua:41>
	...e/nvim/lazy/noice.nvim/lua/noice/view/backend/notify.lua:168: in function '_notify'
	...e/nvim/lazy/noice.nvim/lua/noice/view/backend/notify.lua:198: in function 'show'
	...local/share/nvim/lazy/noice.nvim/lua/noice/view/init.lua:156: in function <...local/share/nvim/lazy/noice.nvim/lua/noice/view/init.lua:154>
	[C]: in function 'xpcall'
	...local/share/nvi

i'm seeing this same issue with a fresh install of LazyVim. here is the full stacktrace:
Error in decoration provider vim_lsp_inlayhint.win:
Error executing lua: ...AD-f55213c/share/nvim/runtime/lua/vim/lsp/inlay_hint.lua:360: Invalid 'col': out of range
stack traceback:
	[C]: in function 'nvim_buf_set_extmark'
	...AD-f55213c/share/nvim/runtime/lua/vim/lsp/inlay_hint.lua:360: in function <...AD-f55213c/share/nvim/runtime/lua/vim/lsp/inlay_hint.lua:314>
	[C]: in function 'nvim_exec2'
	vim/_editor.lua: in function 'cmd'
	.../share/nvim/lazy/nvim-notify/lua/notify/service/init.lua:63: in function 'push'
	...ocal/share/nvim/lazy/nvim-notify/lua/notify/instance.lua:77: in function <...ocal/share/nvim/lazy/nvim-notify/lua/notify/instance.lua:41>
	...e/nvim/lazy/noice.nvim/lua/noice/view/backend/notify.lua:168: in function '_notify'
	...e/nvim/lazy/noice.nvim/lua/noice/view/backend/notify.lua:198: in function 'show'
	...local/share/nvim/lazy/noice.nvim/lua/noice/view/init.lua:156: in function <...local/share/nvim/lazy/noice.nvim/lua/noice/view/init.lua:154>
	[C]: in function 'xpcall'
	...local/share/nvi

+1 for me. The same error

@eiiot seems like others are still having this issue. Can you re-open please?

This seems similar to #3284 that was closed as not LazyVim related, but Neovim trying to render outdated inlay hints. So, from what I read it either has to do with a Neovim upstream bug or some specific LSP not returning valid inlay hints positions.
Probably best to wait on maintainer's input on this as he is more knowledgeable about the issue at hand.

Will reopen given other people are running into this – day 3 of my fresh install and I'm also running into this error:
vtsls: -32603: Request textDocument/inlayHint failed with message: <semantic> TypeScript Server Error (5.3.3)
Debug Failure. Unexpected node.
Node IndexSignature was unexpected.
Error: Debug Failure. Unexpected node.
Node IndexSignature was unexpected.

Seems to be possibly related?

This is the same error as a user mentions here. In the next comment the author of vtsls mentions that that error is fixed in Typescript version 5.4.
There's also an open issue at vtsls repo about the error that you mention as you can see here. It seems to come directly from the LSP server and doesn't seem to have anything to do with LazyVim from what I've read so far.

@dpetka2001 our project is using typescript v5.4.3.

I'm not the one you should be asking about this. Rather the creator of vtsls LSP server. I already linked an open issue in my previous post about the error for inlay hints. It's not solved yet.
Look at this comment by the creator himself. He suggests this is an issue with tsserver itself upstream and also links to the upstream issue https://github.com/microsoft/TypeScript/issues/53984. That issue was closed without referencing any solution.
The last Typescript version it was tested mentioned by the bot that still reproduced the issue was Typescript 5.4.2. Still there's a user comment at the end of that issue that is asking for clarifications if it was fixed or not because the issue was closed without referencing a solution.

For context, up until about a week ago I had been using my own custom non-lazyvim config for years, and I never ran into this issue while using either vtsls or typescript-language-server.
I must have not had inlay hints enabled. I'm going to try disabling.

Just enable :lua vim.lsp.inlay_hints.enable(true) with your old configuration and compare.

I'm able to reproduce the issue with a minimal config. Here are the steps I took:

Run Neovim with nvim -u mini.lua (contents below)
:MasonInstall typescript-language-server
Close and reopen the editor
Open a.ts (contents below)
Press O (S-o) on line 1

mini.lua
local function setup()
  if vim.env.LAZY_STDPATH then
    local root = vim.fn.fnamemodify(vim.env.LAZY_STDPATH, ":p")
    for _, name in ipairs({ "config", "data", "state", "cache" }) do
      vim.env[("XDG_%s_HOME"):format(name:upper())] = root .. "/" .. name
    end
  end

  if vim.env.LAZY_PATH and not vim.uv.fs_stat(vim.env.LAZY_PATH) then
    vim.env.LAZY_PATH = nil
  end

  local lazypath = vim.env.LAZY_PATH or vim.fn.stdpath("data") .. "/lazy/lazy.nvim"
  if not vim.env.LAZY_PATH and not (vim.uv or vim.loop).fs_stat(lazypath) then
    vim.api.nvim_echo({
      {
        "Cloning lazy.nvim\n\n",
        "DiagnosticInfo",
      },
    }, true, {})
    local lazyrepo = "https://github.com/folke/lazy.nvim.git"
    local ok, out = pcall(vim.fn.system, {
      "git",
      "clone",
      "--filter=blob:none",
      lazyrepo,
      lazypath,
    })
    if not ok or vim.v.shell_error ~= 0 then
      vim.api.nvim_echo({
        { "Failed to clone lazy.nvim\n", "ErrorMsg" },
        { vim.trim(out or ""), "WarningMsg" },
        { "\nPress any key to exit...", "MoreMsg" },
      }, true, {})
      vim.fn.getchar()
      os.exit(1)
    end
  end
  vim.opt.rtp:prepend(lazypath)
end

local runtime_dir = vim.env.XDG_RUNTIME_DIR
vim.env.LAZY_STDPATH = vim.fs.joinpath(runtime_dir, "lazy_mini")
setup()

require("lazy.minit").repro({
  spec = {
    { "LazyVim/LazyVim", import = "lazyvim.plugins" },
    -- add any other plugins here
    { import = "lazyvim.plugins.extras.lang.typescript" },
  },
})

a.ts
function f() {
  return "Hello world"
}

Solution
Because of the rename from tsserver to ts_ls, the code that disables typescript-language-server is outdated. So when typescript-language-server is installed by mason.nvim, it runs at the same time as vtsls.
typescript-language-server should be disabled in extras/lang/typescript.lua like so:
{
  "neovim/nvim-lspconfig",
  opts = {
    servers = {
      ts_ls = { enabled = false },
    },
  },
},

When applying it to mini.lua, the error goes away:
--- mini.lua	2024-10-01 19:48:22.183802698 -0500
+++ fix.lua		2024-10-01 19:48:19.521775037 -0500
@@ -48,5 +48,13 @@
     { "LazyVim/LazyVim", import = "lazyvim.plugins" },
     -- add any other plugins here
     { import = "lazyvim.plugins.extras.lang.typescript" },
+    {
+      "neovim/nvim-lspconfig",
+      opts = {
+        servers = {
+          ts_ls = { enabled = false },
+        },
+      },
+    },
   },
 })

typescript-language-server is not installed by default. That's something that a user specifically installs. I thought we were talking about the default Typescript Extra. Minimal repros should be done with only the minimal repro available in the bug issue template and the corresponding Extras enabled. No user configuration should take place.

That's something that a user specifically installs.

Not necessarily. It can be leftover from before the switch to vtsls took place.
In any case, I don't see what relevance that has. Just because typescript-language-server happens to be installed doesn't mean the user deserves to suffer through incessant mysterious error messages.

Anyway, I'm turning off notifications. Not interested in a pedantic back-and-forth.

Just because typescript-language-server happens to be installed doesn't mean the user deserves to suffer through incessant mysterious error messages.

True. If that's really the problem, then the PR will fix it. But the issues i linked from vtsls repo with the same error msg as OP mentioned are 4 months old, way before the renaming of typescript-language-server happened. I hope it's what you say it is, but from what i've read it seems something deeper (of course i might be wrong).

I had this previously, I updated the neovim (I use the master branch of neovim). After the upgrade I didn't have this.

Should be fixed by https://github.com/LazyVim/LazyVim/pull/4406
Either way, this is not a LAzyVim issue, but an issue between tsserver and Neovim's inlay hints

This has been resolved for me 👍

This isn't a language specific or LazyVim related one but given this one has been re-opened for the awareness, I'd like to mention that the exact same error happened to me in Rust. It certainly is something upstream I believe.


WIP Rework hash formats and add JSON format

Closes #70
Test object
{ a: 'cheese', 'a' => 'burger', 1 => 'fries',  1.0 => 'version', count: 33, time: Time.now, nothing: nil, list: [1, 'a', :sym, Time.now, nil], nested_hash: { hi: 'there' } }

Should we also support the JSON format for arrays? I guess the only thing it would do is convert symbols to strings inside the array which doesn't seem particularly helpful but also feels a bit incomplete without it if we're doing that in hashes.


Add BigQuery engine specifications

As contributed by @mxmzdlv on issue #945

Coverage decreased (-0.06%) to 69.321% when pulling 536bb065d5706855a1d946b410174346b6e19343 on mistercrunch:big_query into fb866a937bba48a94dc986789129f7f83cf2aa31 on apache:master.

Coverage decreased (-0.06%) to 69.321% when pulling 536bb065d5706855a1d946b410174346b6e19343 on mistercrunch:big_query into fb866a937bba48a94dc986789129f7f83cf2aa31 on apache:master.

Coverage decreased (-0.06%) to 69.321% when pulling 536bb065d5706855a1d946b410174346b6e19343 on mistercrunch:big_query into fb866a937bba48a94dc986789129f7f83cf2aa31 on apache:master.


Changed "affect" to "effect"

@rcabr : Thanks for your contribution! The author, @minewiskan, has been notified to review your proposed change.

@rcabr - nice catch. thanks!


Add support to use ks image in the kubectl admin of KubeSphere

Currently, you can use the following command to archive that. But I believe that it can be easier.
ks -n kubesphere-controls-system set image deploy kubectl-admin kubectl=surenpi/ks:v0.0.26
For example, we can use do it like: ks config use-ks

Here's an example to use ks in KubeSphere kubectl console page:


No file can be indexed on local filesystem provider.

What version of Music Assistant has the issue?
2.0.0b143
What version of the Home Assistant Integration have you got installed?
2024.4.0
Have you tried everything in the Troubleshooting FAQ and reviewed the Open and Closed Issues and Discussions to resolve this yourself?

[X] Yes

The problem
No file can be indexed on local filesystem provider, logs shows error accessing the files.
How to reproduce
Problems started with 2.0.0b141, I tried to clear cache, but nothing changed.
Since I read that the database need to be reconstructed with 2.0.0b143 I decided to start from scratch and I reinstalled the docker, but problems still persists.
Music Providers
File system provider only.
Player Providers
Airplay player only.
Full log output
musicassistant.log
Additional information
Music Assistant runs inside a docker that have access to a volume that maps to a an Unraid folder.
Everything worked well since 2.0.0b141.
What version of Home Assistant Core are your running
2024.4.0
What type of installation are you running?
Home Assistant Container
On what type of hardware are you running?
Linux

Please roll back via your backup to confirm this is actually a MA problem and not a permission issue at your end

Thank you very much for your response and for your hard work, unfortunately I don't have a backup.
I forgot to mention that using Browse functionality of MA I'm able to list files, but I click on Show info an empty page is shown.
Below a screenshot of a ls performed inside the MA docker on the mapped folder:

Ok please be patient until this can get looked at further

This has nothing to do with MA but is an access rights issue. Make sure that you create the docker container with the correct (elevated!) access rights and that MA has read rights to your media files.

@marcelveldt thank you very much, yes the container wasn't privileged, now it works.
I'm quite sure that before b141 everything was working even without privileged flag, but anyway thank you for your work! 👍


Order by polya tail length

Hi
Great work with trackplot
I  currently plotted some reads which have polyA tail length as attached
is there a way to order these reads by either transcript ie same exon and intron and then by Poly A tail length?

Hi,
Sorry for the late response, trackplot don't have a parameters to control the ordering of poly a tail length. But trackplot draw all the poly a tails according to your input files, please try with an ordered poly a tails inputs.
Best

Thank you for reply @ygidtu
is there away to achieve squashed kind of plots similar to attached . Sometime we have have many thousand of reads how can we squash them to achieve this look

Hi,
Trackplot doesn't include a specific squashed mode. But it can control the height ratio for each individual plot by modifying the height_ratio list starting from line 1158 in trackplot/plot.py. The height_ratio is a list of numbers representing the percentage of the graph height. For example, 1 means plotting with the normal height, and 0.5 means that the corresponding graph has only half of the specified height.
Best
Yiming


FadeInImage.assetNetwork  throws Null check operator used on a null value (Flutter 3.3.0)

  ════════ Exception caught by widgets library ═══════════════════════════════════════════════════════
The following _CastError was thrown building _AnimatedFadeOutFadeIn(duration: 1000ms, dirty, state: _AnimatedFadeOutFadeInState#8beb1(ticker inactive, targetOpacity: null, placeholderOpacity: null)):
Null check operator used on a null value

octor summary (to see all details, run flutter doctor -v):
[✓] Flutter (Channel stable, 3.3.0, on macOS 13.0.1 22A400 darwin-x64, locale zh-Hans-CN)
[✓] Android toolchain - develop for Android devices (Android SDK version 30.0.3)
[✓] Xcode - develop for iOS and macOS (Xcode 14.1)

Hi @devyueyue, you are on an older version of flutter. Kindly upgrade to the latest version of flutter which contains the fix.
This was initially reported in https://github.com/flutter/flutter/issues/111011 and was fixed by https://github.com/flutter/flutter/pull/111035 which was cherry-picked into flutter 3.3.5. See https://github.com/flutter/flutter/wiki/Hotfixes-to-the-Stable-Channel#335-oct-19-2022.
I'll be closing this as this has already been fixed on the latest versions of flutter. If you run into this issue on the latest version of stable (3.3.10), kindly leave a comment below with a minimal reproducible code sample along with the updated output of flutter doctor -v and I will reopen this.
Thank you

Thank you for your reply!
3.3.10 Version, picture display is normal, no problem


autostart

its prolly too much of a pitb but autostart with pc would be nice and/or autorun when it sees you logoff, r.io's app does the latter but idk how they detect it. probably when files get written?

Added in v0.6.0


Enable configuring a custom language server through an environment variable

Changes

Enable starting the extension with a custom language server using an environment variable, which is useful for development scenarios. You can imagine having a make command in the Dafny repository that starts VSCode with the language server from the repository: DAFNY_LANGUAGE_SERVER=./Binaries/DafnyLanguageServer.dll code.
Add a run configuration to launch.json to start the extension with the language server configured in the environment variable CUSTOM_DAFNY_SERVER, for when you're working on ide-vscode changes that require an unreleased Dafny version.

Testing
Manually tested both features

YES, love it!
I'm a bit confused about why we need both environment variables though. Couldn't we have the launch.json pass through the same variable with something like "DAFNY_LANGUAGE_SERVER": "${env:DAFNY_LANGUAGE_SERVER}"?

Don't you need a similar variable for where to find Dafny.dll (for compilation/execution?)
Maybe it should just be DAFNY_DLLS=./Binaries/?

Don't you need a similar variable for where to find Dafny.dll (for compilation/execution?) Maybe it should just be DAFNY_DLLS=./Binaries/?

O, sorry I read this too late. I haven't run into this use-case yet but it sounds plausible. Since this is a development feature I think we're free to change the variable name at any point.


Ranged descriptors

Adds rudimentary support for ranged descriptors. ~We could add more convenience methods, but this requires us to copy / implement a lot of traits such as ForEachKey.~

Loosened the trait bound in Descriptor<Pk> as FromStr. Now Pk can be pretty much any MiniscriptKey.

Fixed the issue. Now we can decode arbitrary Policys and Descriptors.

str::split_once() is available from 1.52.0 so I need to find a workaround.

Fixed and rebased on master.

Same CI failure as in #158 . Cherry-picked the fix.

Superseded by #162. We will use ideas from this PR when we implement Simplicity descriptors over in elements-miniscript. But there is no reason to keep this PR open.


Numbered list new line issue in demo

Hi,
if i visit the demo page that comes with the core of the project (this demo is available also at this host https://www.softaculous.com/demos/GRAV), i see a new line after the second numbered list element.

This is really strange, and depends from the block code that comes after it. If I remove the code block, the new line character disappear.

Any idea?
Thanks

Pretty minor issue, but it's caused by the default ol styling of Spectre.css:
ol {
    list-style: decimal inside;
}

When this is removed, the styling is as expected.  I'm not sure why Spectre.css has this behavior for lists by default, but Quark tries to keep things as Spectre-ish as possible.

I've updated this in REPO for Quark.


Change wording about a number not being positve

Pre-Submission Checklist

[x] Your pull request targets the staging branch of FreeCodeCamp.
[x] Branch starts with either fix/, feature/, or translate/ (e.g. fix/signin-issue)
[x] You have only one commit (if not, squash them into one commit).
[x] All new and existing tests pass the command npm run test-challenges. Use git commit --amend to amend any fixes.

Type of Change

[x] Small bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds new functionality)
[ ] Breaking change (fix or feature that would change existing functionality)
[ ] Add new translation (feature adding new translations)

Checklist:

[x] Tested changes locally.
[ ] Closes currently open issue (replace XXXX with an issue no): Closes #8550

Description
Change the basic-bonfires.json file to reword a what happens when a number is 0, thus not positive.

:sparkles: :sparkles: LGTM. :sparkles: :sparkles:
:tada: Thanks and congrats for your first contribution to Free Code Camp :tada:


storing graphics in dict

addresses #64
moved to store graphics in dict with keys as the graphic class
keys currently need to passed in as a string, need to work on implementation to allow for
gp[0,0].get_graphics()[ImageGraphic]

as opposed to
gp[0,0].get_graphics()["ImageGraphic"]

right now, I have it creating the keys to the dict by
self._graphics[graphic.__class__.__name__] = list()
self._graphics[graphic.__class__.__name__].append(graphic)

but __name__ returns a string, any suggestions on how to get graphic class name not as a string when just __class__ gives fastplotlib.graphics.image.ImageGraphic

yea not sure whether the best approach is to have it as a str or the class type itself, I think using graphic.__class__ should work, you can check if:
graphic.__class__ is ImageGraphic

after this is done modify the __repr__ for PlotArea such that if there are > 5 graphics of any type, show only the first 5 and last 5 because I know there will be someone at the workshop who crashes their jupyter because they ended up calling the repr while having 10,000 graphics in a gridplot :upside_down_face:

Now that #78 brings events to GraphicFeatures we want to make link() work seamlessly for both pygfx events and GraphicFeature events. Something like this should work:
def link(self, event_type: str, target: Any, ...):
  pygfx_events = ["click", ...]
  graphic_feature_events = ["colors", "data", "present", ...]

  if event_type in pygfx_events:
    self.world_object.add_event_handler(...)

  elif event_type in graphic_feature_events:
    feature = getattr(self, event_type)
    feature.add_event_handler(...)

after #88 there is now a class attribute that stores the name of the graphic as a str in lower case, see: https://github.com/kushalkolar/fastplotlib/pull/88/files#diff-a57994ecff1cde4e05c8175265219f0b5200da144ce8733b9eb560c09bcea7ddR9

rebase on mater or just start a new PR, this is pretty straight forward.

I think that since PlotAreas are indexable we don't really need this. We can revisit if it becomes necessary in the future.


Update fmt::Debug to produce new info

Closes  #79
Let me know what to change

Rebase on the current master to fix the current CI error (#84)

Let me know if I missed something

fixed it all


perf: various fixes to improve vector transform performance at high scales

Adds with_row_address to python bindings (for internal use case)
Removes rayon from transform functions (there is already plenty of data-parallelism here in the outer loop)
Use Vec<Vec<_>> instead of HashMap for partition id lookup.

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

Project coverage is 79.41%. Comparing base (a349b6d) to head (ebadf19).
Report is 1 commits behind head on main.

Files
Patch %
Lines

rust/lance/src/index/vector/ivf/builder.rs
0.00%
74 Missing :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #2703      +/-   ##
==========================================
- Coverage   79.49%   79.41%   -0.08%
==========================================
  Files         226      226
  Lines       66655    66558      -97
  Branches    66655    66558      -97
==========================================
- Hits        52986    52859     -127
- Misses      10554    10616      +62
+ Partials     3115     3083      -32

Flag
Coverage Δ

unittests
79.41% <13.95%> (-0.08%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

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

Curious what kind of performance improvement we can expect here

Curious what kind of performance improvement we can expect here

At low / medium scales not much.  Even benching with 1M I didn't see any real change.  The rayon removal doesn't really speedup anything, it just avoids crashes on many-core systems.
At high scales the improvement is significant.  Hashmap performance starts to really fall over as it approaches 1B rows.  The Vec solution is much faster then.


BibTex

Hello,
I was wondering, in regards to BibTex, what to do if we are citing a non-academic article in our proposals (example - a Pew article online)...
Many thanks,
Marie

You would probably want to use the MISC document type. See here for examples: http://tex.stackexchange.com/questions/3587/how-can-i-use-bibtex-to-cite-a-web-page
Apart from a few variations, it is pretty similar to ARTICLE.

Thank you!

Professor,
I am still struggling with this one, despite googling every step of the way anytime there was an issue (pretty much all the time).
The approach cited in the above article is reliant on one of two packages, hyperref or url - I have tried to install both to no avail. It says they're not compatible with the latest version of R.
Or at least, that is what I took this line to mean:
You should also perhaps include an author if you know it. And remember to load a package such as hyperref or url.
So then I tried this approach:
@MISC{doe:website,
AUTHOR = "John Doe",
TITLE = "the website title",
MONTH = feb,
YEAR = 2011,
NOTE = "\url{http://whereever/file.ext}"
But when I try to knit that it lists a whole bunch of errors.
So a few questions....
Is there a way to install packages that initially say they're not compatible with R 3.3.1?
Could you please clarify exactly what the key is in these lines of code? The key is supposed to be a unique identifier so I don't quite understand where the unique part is... I was just using the same approach as with academic journals (that approach I get) but maybe it's not the same approach..?
Many thanks,
Marie

Hey Marie,
if you skip installing any new packages and just try putting the URL in a Field called "Url":
Example:
@Misc{Klausa.2016,
  Title                    = {Torben's Website},

  Author                   = {Torben Klausa},
  Year                     = {2016},

  Owner                    = {torben},
  Timestamp                = {2016.10.26},
  Url                      = {http://www.hertie-school.org}
}

Does that work?
Best,
Torben

Hi @MarieAgosta. Sorry about the confusion. The packages the stack exchange article are TeX packages, not R packages. They actually already come with LaTeX when you download it. So there is no need to install. Those ones should also be loaded by R Markdown when you knit it.
@torbatschow's example should work. I think in the BibTeX entry you have, there is inconsistent use of " or alternatively {} around each element of the entry.

Thank you!
From: Christopher Gandrud <notifications<EMAIL_ADDRESS>Reply-To: HertieDataScience/SyllabusAndLectures <reply<EMAIL_ADDRESS>Date: Thursday, October 27, 2016 at 8:49 AM
To: HertieDataScience/SyllabusAndLectures <SyllabusAndLectures<EMAIL_ADDRESS>Cc: Marie <M.Agosta<EMAIL_ADDRESS>Mention <mention<EMAIL_ADDRESS>Subject: Re: [HertieDataScience/SyllabusAndLectures] BibTex (#112)
Hi @MarieAgostahttps://github.com/MarieAgosta. Sorry about the confusion. The packages the stack exchange article are TeX packages, not R packages. They actually already come with LaTeX when you download it. So there is no need to install. Those ones should also be loaded by R Markdown when you knit it.
@torbatschowhttps://github.com/torbatschow's example should work. I think in the BibTeX entry you have, there is inconsistent use of " or alternatively {} around each element of the entry.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/HertieDataScience/SyllabusAndLectures/issues/112#issuecomment-256560737, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AVA71jJqIsciPbYU2MV0DuLUXIO3SG8Rks5q4EmNgaJpZM4KgD6M.

Much better :)
Thanks Torben! Appreciate it.
From: Torben Klausa <notifications<EMAIL_ADDRESS>Reply-To: HertieDataScience/SyllabusAndLectures <reply<EMAIL_ADDRESS>Date: Wednesday, October 26, 2016 at 8:29 PM
To: HertieDataScience/SyllabusAndLectures <SyllabusAndLectures<EMAIL_ADDRESS>Cc: Marie <M.Agosta<EMAIL_ADDRESS>Author <author<EMAIL_ADDRESS>Subject: Re: [HertieDataScience/SyllabusAndLectures] BibTex (#112)
Hey Marie,
if you skip installing any new packages and just try putting the URL in a Field called "Url":
Example:
@Misc{Klausa.2016,
Title                    = {Torben's Website},
Author                   = {Torben Klausa},
Year                     = {2016},
Owner                    = {torben},
Timestamp                = {2016.10.26},
Url                      = {http://www.hertie-school.org}
}
[Example Reference with URL]<www.torbenklausa.de/files/BibTexURL.png>
Does that work?
Best,
Torben
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/HertieDataScience/SyllabusAndLectures/issues/112#issuecomment-256436689, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AVA71ry0AfVINipukmdHTBPETfMerQvAks5q35wLgaJpZM4KgD6M.


Expand user spec

Issue #155 Expand User model test

This looks good to me. If anyone else wants to review and provide feedback on these tests (my RSpec test writing skills are admittedly not amazing) please go for it. Otherwise this can be merged in.

Again, it may be wise to convert tabs to 2-space indentations to keep consistent with the rest of the project 😄

This is in really good shape! So awesome! :smile:
Ready to merge once the latest comments are taken care of @mzagaja

I jumped the gun and merged it in since @veddster had addressed the comment you left before I thought it was ready. @veddster can you just update your branch and put in another pull request? Thanks for putting in the effort to improve this.

No problem!


Memory leak in styx V 0.14.0

The problem
We are facing a memory leak in Styx V-0.14.0.
Detailed description
After investigation we found that the root cause of this issue is related to the dashboard each time we trigger the dashborad update event the code register new Origins objects to the google guava eventbus and it dose not unregister old objects see
com.hotels.styx.admin.dashboard.DashboardData$Backend constructer
with time this behavior fill the memory with unused objects that not eligible for the GC
Acceptance criteria
When trigger the dashboard update event before registering new origins to the eventbus, old Origins MUST unregistered.

@salahqasem A PR is pending. Feel free to have a look.

Thanks @mikkokar


TwoWay binding calls setter on creation

Failing tests:
[Fact]
public void TwoWay_Binding_Should_Not_Call_Setter_On_Creation()
{
    var target = new Class1();
    var source = new TestTwoWayBindingViewModel() { Value = 1 };

    target.Bind(Class1.DoubleValueProperty, new Binding(nameof(source.Value), BindingMode.TwoWay) { Source = source });

    Assert.False(source.SetterCalled); // SetterCalled is true
}

[Fact]
public void TwoWay_Binding_Should_Not_Call_Setter_On_Creation_Indexer()
{
    var target = new Class1();
    var source = new TestTwoWayBindingViewModel() { [0] = 1 };

    target.Bind(Class1.DoubleValueProperty, new Binding("[0]", BindingMode.TwoWay) { Source = source });

    Assert.False(source.SetterCalled); // SetterCalled is true
}

Both "Class1" and "TestTwoWayBindingViewModel" exist in AvaloniaObjectTests_Binding test class.

It doesn't call setter if property has default value.
Because of that existing TwoWay_Binding_Should_Not_Call_Setter_On_Creation and TwoWay_Binding_Should_Not_Call_Setter_On_Creation_Indexer tests passing.

Can't be reproduced anymore. Added tests in related PR https://github.com/AvaloniaUI/Avalonia/pull/7786


[FABC-865] Fix setting TLS files by cert/key.file

This CR allows operations.tls.{cert, key}.file key to set
operations TLS certificate and key file.
It is set operation TLS files to operations.tls.{cert, key}.file key
in config. However, operations TLS struct accepts
operations.tls.{certfile, keyfile}. As a result, it doesn't not
working even if it was set in operations.tls.{cert, key}.file.
Signed-off-by: Nao Nishijima<EMAIL_ADDRESS>

Thank you for reviewing. I updated the patch.

Looks good. Can you please rebase this on the current master and it's ready to merge.
Thanks!

@naonishijima I requested a rebase to avoid the merge. I'll fix this up, push, and merge on your behalf.
Thanks.


NDA issue

Make the repo private

No need it has a license and previous wehack NDA's haven't done shitOn 23 Sep 2019 23:33, Ubaid Usmani<EMAIL_ADDRESS>wrote:Make the repo private
—You are receiving this because you are subscribed to this thread.Reply to this email directly, view it on GitHub, or mute the thread.

Issue Closed, MIT License negates conflict with NDA


fix: 🐛 JSON launch not working properly (#1089)

JSON launch not working properly after QIDO/WADO switch #1089
Closes: #1089
TEST URL: https://deploy-preview-1093--ohif.netlify.com/pwa/viewer?url=https://s3.eu-central-1.amazonaws.com/ohif-viewer/JSON/PTCTStudy.json

@swederik I don't know how this got broken but as you can see from the changes, it has nothing to do with QIDO+WADO feature...

?url=https://s3.eu-central-1.amazonaws.com/ohif-viewer/JSON/PTCTStudy.json -->

When I first tried to load the JSON URL on my local environment, the only exception I got was coming from the <Viewer> component and was related to the fact that the viewports property (which is required) was absent:
https://github.com/OHIF/Viewers/blob/bef50f929853ee70c519ece87d5914b8fd96414f/platform/viewer/src/connectedComponents/Viewer.js#L64
After changing the render method to use <ConnectedViewer> instead of <Viewer> the exception was gone but still the viewport wouldn't render.
Comparing how <ConnectedViewer> was invoked inside <ViewerRetrieveStudyData> I noticed the studyInstanceUids property was absent so I added that to make sure it was not interfering with anything.
While debugging downstream (inside <ViewerMain>) I realized that the viewport was not being initialized because the fillEmptyViewportPanes method was being called before the component state was properly updated (from componentDidMount method);
So, the fix was:

Replace <Viewer> by <ConnectedViewer>;
Make sure fillEmptyViewportPanes method was only called after the component state was properly updated;

As I said, I don't know how or when this feature got broken but I couldn't find any relationship between this issue and the QIDO+WADO changes...

@emfol, I appreciate the explanation! It helps me wrap my head around what's going on much better.


Add possibility to use continue with "Manual mode"

Currently when in manual mode and clicking continue on some entry in the list this entry is started and the timer is switched back to timer mode.
Instead of this it would make sense to switch the continue also to Manual mode when timer is manual mode. This means that when you are in manual mode and click continue on one of the past entries it opens up the edit view as with the timer link click and it fills in all the data with the data from the continued time entry.
Same should go for the keyboard shortcuts.

Looks to me like this should be handled in the library when the toggl_continue_latest command is called (which it is in the cases described), and not in the ui.

It's not only about continuing the latest entry you can continue any entry in the list. But yes it should be handled in the lib.

so basically the continued entry will pop up in editor. will it be started (running) or not? if not, what will be its start and end time?

It will not be started.
Works as the regular manual mode.
Opens up edit view and fills in details like usual continue. Start ,duration and stop are not filled in. They are same as they are in the regular manual mode created entry.


switch to oxocarbon-lua

dunno if you want to spend your time to build it yourself

Sounds like a fair change for now, will probably fix some install issues as well. I'll eventually add a base16-based colorscheme-generator module but this should do for now. Thanks!

i would love to see a https://github.com/danth/stylix module for neovim or for nix-colors :D


MML-133 - Forms cannot have the same id inside the same message

It will need some refactoring on other tests. Closing this PR for now, will open another once ready again.


fix: upgrade deps

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

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

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
185

Relevant Lines:
196

💛  - Coveralls


Backport to GCC 4.8

ETA?  Would it help to prod you on this if I told you I just recommended this implementation to some at-risk youths who are forced to use GCC 4.8.3?

Hi Zach! I'm planning to work on this next week. Last time I tried this it was having trouble with
the cartesian product generation here: https://github.com/mpark/variant/blob/master/include/mpark/variant.hpp#L574-L575
So I'll have to figure out how to deal with that part.
Appreciate  the recommendation!

Closed by 12b77f3325b81ba56a3c3f0b88e61b57505b9ff0.
The minimum repro of the problems were https://wandbox.org/permlink/ypmovjwyTaXnSnwy.
In short, S<alias<pack..., thing>> drops thing and incorrectly passes alias<pack...> instead.
This caused the true in all implementation to be dropped in bool_sequence<Bs..., true>,
and the J was dropped in index_sequence<Is..., J>. Since the issue was due to the fact that
bool_sequence and index_sequence are aliases, integer_sequence is now used directly.

Thanks for the quick resolution!


Bug with requirePermission

I found a couple bugs regarding the requirePermissions, it looks like craft is using the section->uid now instead of the sectionId for checking the permission.
Also on line 162 of SaveController.php there is an extra colon, the $permissionSuffix variable already contains the colon.
$this->requirePermission('publishPeerEntries:' . $permissionSuffix);
Thanks love the plugin btw!

Thanks for letting me know. I'll take a look at these issues.
I'm glad it's working well for you otherwise. It was a huge help for me on several projects.

Just pushed out 1.0.3 which should fix these couple issues.

Amazingly quick! Just tested and works great, thanks!!


Fix compiler warnig about comparing signed and unsigned integers.

All the stuff it interacts with is unsigned.

Closing, this is included in https://github.com/hyprwm/Hyprland/pull/781


Drawer sticky buttons

Drawer buttons do not stick to bottom if window is too small and there is scroll. Buttons stick to bottom of the initial viewport, but does not stick to bottom if you start scrolling.  You can see this in the example in the docs.
https://hedwig.posten.no/7cc809d17/p/283a39-drawer/b/77e793
I am having a little scroll in my drawer if the window is too small. The drawer buttons will then overlay the content because the buttons are not sticking to bottom, only of the inital viewport (before scroll)

Made a copy of this issue to KP Trello


update java samples

Signed-off-by: wanjunlei<EMAIL_ADDRESS>

@wanjunlei Don't forget to update openfunction.dev:
https://openfunction.dev/docs/getting-started/quickstarts/sync-functions/
https://openfunction.dev/docs/getting-started/quickstarts/async-functions/
https://openfunction.dev/docs/concepts/function_signatures/


1589 - Maximum Sum Obtained of Any Permutation

Description of the Problem
We have an array of integers, nums, and an array of requests where requests[i] = [starti, endi]. The ith request asks for the sum of nums[starti] + nums[starti + 1] + ... + nums[endi - 1] + nums[endi]. Both starti and endi are 0-indexed.
Return the maximum total sum of all requests among all permutations of nums.
Since the answer may be too large, return it modulo 109 + 7.
Link To The LeetCode Problem
LeetCode

I'd like to take this up.

If this issue is currently open, I would like to solve it.
Can you assign this to me?
Thank You!

please assign this to me! Thanks!

This is my first hacktoberfest participation and getting to solve a problem would help me grow in terms of knowledge and help towards open-source. Can you assign this to me?


Advanced post processing effects

Is it possible to apply noise effect and glitch effect to the stage?
It can work like 'Canvas Effect'

PR #373

the ability to apply effects to specific a sprite would be good, i could create realistic frosted glass!


Moved checksumType variables' definition to setting for Ketarin

Unfortunately this change is not quite complete. If you are going to change the ability to pass in your own checksum type, you need to edit the ketarin settings c# to pivot on that value and calculate the proper checksum. Right now it only calculates on sha256.

I've noticed that the ketarin's C# setting only calculates sha256, and now I just want to move the definition of checksum type variable from chocolateyInstall.ps1 to the package's ketarin setting.

Yes, but you can't do that without also fixing up the c# code in the ketarin settings file to allow for it to receive other types.

HTH

Now I have turned to AU so I won't continue this improve.

@Elderry said...
Now I have turned to AU so I won't continue this improve.

Thanks! If you can offer any help in porting to AU, we would appreciate it 👍


CNRichTextView showing nothing

I'm trying to use <CNRichTextView text={text} /> but it doesn't seem to work.
I set text as simple html (

Heading 1

 for example - that is produced by convertToHtmlString() ) but it just shows nothing at all.
If I convertToObject() and set it into CNRichTextEditor it works properly and shows up good.
I'm using a simple rendering without anything else in the screen.
Any idea what I'm doing wrong?
P.S.
It would be very useful to have a doc about CNRichTextView, since I discovered it exploring the src directory.

@koo27 , I tried below sample and it worked fine. Please

Be sure to pass htmlstring to CNRichTextView
and pass a positioning style to  CNRichTextView like style={{flex: 1}}

Simple Usage :
import React, { Component } from 'react';
import { View } from 'react-native';
import { CNRichTextView } from "react-native-cn-richtext-editor";

class App extends Component {
constructor(props) {
   super(props);

    this.state = {
        text: '

Heading 1

'
    };
 }

render() {
   return (
         <View style={{
             flex: 1, padding: 20
         }}>
             <CNRichTextView
              style={{flex: 1}}
              text={this.state.text}
             />
         </View>
     );
 }
}

And we'll add this component to our documentation for sure :)

@imnapo Thank you very much, it works perfectly. I was missing the flex:1 style, my bad.


Issues with current website

#7 - (Emily) Add SHIFT Media's logo on the navigation bar at the top of the page.
#7 - (Emily) Add images onto the image background on the image carousel.
#7 - (Emily) Remove calendar from the image carousel and place below the links but above the footer.

#7 - (Wayne) Include SHIFT logo.
#7 - (Wayne) Only Ofsted's link works, all others are broken.
#7 - (Wayne) Force social media to open as new tabs.
#7 - (Wayne) Add icons/pictures to the image on the carousel.
#7 - (Wayne) Create at bare minimum the hamburger menus for the pages that are stored on the website server.


Bad JS load path when URL missing trailing slash

Remember to prefix assets with a forward slash so that they're not relative. Blank white screen.

some more information please? Where exactly do you spotted this problem?

As this report was not really descriptive and I could not reproduce it, i'm not sure if your problem may be fixed with latest version. Added validation to urlPrefix there...

closed as there is no feedback and i believe its fixed with later releases. Please reopen with more details if not fixed


add fused = true to adam, except pagedAdam

Context
What is the purpose of this PR? Is it to

[x] add a new feature
[ ] fix a bug
[ ] update tests and/or documentation
[ ] other (please add here)

update all optimizers to use fused, since its faster
Test plan

So the only reason fused=True isn't default is because it's relatively new and torch wants to give some break in time?


Make all rule links link to the "Console" tab rather than "Graph"

Clicking on a rule, either the name or the expression, opens the rule
result (or the corresponding expression, repsectively) in the
expression browser. This should by default happen in the console tab,
as, more often than not, displaying it in the graph tab runs into a
timeout.
@grobie we talked about this. I wanted to fix it for ages, and finally it should go into the bug fix release.
IIRC @brian-brazil said he thought this has been fixed long ago. Thus, it's definitely the expected behavior, and what we have now is a bug.

IIRC @brian-brazil said he thought this has been fixed long ago. Thus, it's definitely the expected behavior, and what we have now is a bug.

I believe I was referring to something different, a recent expression browser regression that has been since fixed.
Doing a bit of digging this has been this way since it was added in #301, and I don't recall any discussions around this at any point.
There was something else we flipped in one direction or the other though, but I can't find it now.
I don't think this would classify as a bug one way or the other.

:+1: on the change itself. No strong feelings on whether that should be part of 1.7.2, slight preference to not include this in a bugfix release.

Changed to merge into master.


Mising "CREATE" when generating DDL for Oracle trigger

Description
The generated SQL has no "CREATE" keyword
DBeaver Version
<IP_ADDRESS>208051447
Operating System
Windows 10
Database and driver
Database: Oracle 12c
Steps to reproduce
No response
Additional context
No response

Thank you for bug report.


Mldb 1344 aggregator in transform

Fix transform procedure not working when there are aggregators without a group by.

+1


[Improvement][Task Plugin] Remove Passed Parameter --queue in Seatunnel Task Plugin

Search before asking

[X] I had searched in the issues and found no similar feature requirement.

Description
Remove --queue parameter because it passed.
Are you willing to submit a PR?

[X] Yes I am willing to submit a PR!

Code of Conduct

[X] I agree to follow this project's Code of Conduct

It has been merged into the 3.1.0 branch.


Move {AnyNodeRef, AstNode} to ruff_python_ast crate root.

A quality of life improvement mostly mechanical refactoring.

Current dependencies on/for this PR:

main

PR #7946

PR #8011   👈

This comment was auto-generated by Graphite.

Apparently we use AnyNode (the non-ref version) only in tests

Dang it, I just merged this into a non-main branch, I'm the worst.

(I will fix up. I wanted to merge to avoid rebase conflicts since this changes so many files.)


Fix error when loading fits for pandas dataframe

Fixed error that occurred when reading a fits file and converting it to pandas using tables_io and python in version 3.10

the file format is .hdf5


Project Feedback!

It looks like the following features are not reflected on your GIF walkthrough:

User is displayed the relative timestamp for each tweet ""8m"", ""7h"""

In order for us to count these towards your submission, please record another GIF that captures these features. Once you do, please push your updates and submit your assignment again through the course portal within 48 hours from the posted deadline so that we can regrade it.

Hello,
Does that mean I need to use an extension for this?
On Tue, Oct 5, 2021 at 5:44 AM CodePath Reviewers @.***>
wrote:

It looks like the following features are not reflected on your GIF
walkthrough:

User is displayed the relative timestamp for each tweet ""8m"",
""7h"""

In order for us to count these towards your submission, please record
another GIF that captures these features. Once you do, please push your
updates and submit your assignment again through the course portal
within 48 hours from the posted deadline so that we can regrade it.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/Pht7/SimpleTweet/issues/2, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AOE2SJMJ374JC423LJGDOXTUFLCJHANCNFSM5FLLB36Q
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

--
Peter Tran
Computer Science 2022
Ying Wu College of Computing - New Jersey Institute of Technology
Alpha Sigma Phi - Alpha Rho - Vice President of Finance


New events can not be added!

Server issue
Some of the data is set to required in DB but queries do not include them hence its not adding new events.

New Events and Projects can be added through Desktop. DB is sorted


Relative base API URL

I hope ng-admin will support relative base api url instead of absolute one.
It'll make the deployment much easier

Hello,
You can set relatives URLs with :
app.baseApiUrl('');

When using ng-admin on the page : http://localhost:8000/, requests will be executed on this host, like :
http://localhost:8000/tags?_end=30&_start=0.

My API Endpoint is http://quiz.dev/api/v2
My ng-admin endpoint is http://quiz.dev/admincp
I set app..baseApiUrl('api/v2/')
ng-admin request this api:
http://quiz.dev/api/v2/api/v2/category?_sort=id&page=1&per_page=10
I tried to edit into /api/v2 or /api/v2/ but none of them work.

Same for me.

Hello,
Restangular doesn't allows relative URLs.
You can reuse your host & port with :
app.baseApiUrl(location.protocol + '//' + location.hostname + (location.port ? ':' + location.port : '') + '/')

Thank you Manu !

@manuquentin  thank you

@manuquentin Thank You for this simple solution.

Hello,
does it apply to sails also? because i tried to use http://localhost:1337 but it is giving me the same ?_sort=id&page=1&per_page=10

@Shazam14 posting the same question at several places at once (cf #695) actually decreases your chances of getting an answer. Please don't do that anymore.


Sequence and TemporalStateful should be the same interface

Right now, there are two abstract classes Sequence for something with is an input sequence and TemporalStateful which generates as a result of a computation. This in fact should be the same inerface, such that we can easily stack model parts.

Is the following too audacious?

Keep Sequence and get rid of TemporalStateful.
A recurrent encoder will have a property called state_sequence (or output_sequence or whatever) of type Sequence, which will replace the properties temporal_states and temporal_length.
Make it possible to reference objects' attributes in the config file. Then we can write, for instance:

[encoder1]
input_sequence=<embeddings>

[encoder2]
input_sequence=<encoder1.state_sequence>

[decoder]
input_sequence=<encoder2.state_sequence>

I'd do it the other way - throw away the sequence class and keep *statefuls. First, there is the SpatialStateful interface for 2D state matrices. Second, Sequence is a model part, which is not a very intuitive thing. Third, Sequence is now only used in the embedding-encoder relationship, whereas TemporalStateful is newer and used on more places.
So, for example for embeddings which are done in the EmbeddedSequence class, I would create a class Embedding which would be a TemporalStateful instance of ModelPart, and consume a data series and embedding size.
However, we should find a commonly accepted solution and I am open to have my mind changed..
As for the object attribute references in the config, I'm very much in favor for doing it.

Please also check somewhat related issue #575 (when deciding where to apply dropout).

I created issue for the referencing (#576)

This is done, but I can find the PR.


[bug]: error occurring while installing the components

Describe the bug
npx shadcn@latest add avatar
⠴ Checking registry.
Something went wrong. Please check the error below for more details.
If the problem persists, please open an issue on GitHub.
request to https://ui.shadcn.com/r/index.json failed, reason: self-signed certificate in certificate chain
Affected component/components
all
How to reproduce

Go to terminal
Try to install any component(eg: button, textarea, input ...)

Codesandbox/StackBlitz link
No response
Logs
No response
System Info
windows 11 pro

Before submitting

[X] I've made research efforts and searched the documentation
[X] I've searched for existing issues

I have the same problem too 😭😭😭😭😭

same here

Same here


[Ask] Where was the old ding sound from

Where was the old ding! sound from? I have trouble hearing the new pop sound even with it turned up to max.

hm. maybe we can just add it back as selectable options


Question regarding statistical power with small numbers of samples and mutations

Hello,
Thanks for developing an excellent set of tools for this important field.
This is not a code issue, but more of an operational question, so I apologize for opening an issue, but it seems that your team is responsive here and I thought others might find the discussion useful. I'm not sure what other forum would be appropriate, but feel free to suggest an alternative.
I have a few questions relating to small sample sizes. We are interested in understanding the mutational processes in the context of chemical exposures, using data produced from mutation assays (primarily Duplex Sequencing, currently) to gain mechanistic insights.

Do you have any recommendations about the minimum number of samples and the minimum number of mutations one should use for de novo signature extraction?
We are aware that to extract the COSMIC signatures you are dealing with massive mutational catalogs encompassing thousands of tumors and millions of mutations. Discussing this with Ludmil in the past has led me to understand that it isn't really appropriate to use a tool like SigProfilerExtractor when the number of samples, groups, and mutations are comparatively small.
The datasets we are talking about have on average 3 to 6 replicates per group, and somewhere between hundreds or thousands of mutations per sample.
So for a case like this, is the recommended approach to only use SigProfilerAssignment? If I use the SigProfilerExtractor function to extract a de novo "signature" for one of these smaller datasets, the COSMIC-based decomposition of these signatures is much cleaner (i.e., fewer signatures reported to reconstruct my data) than if I only use SigProfilerAssignment. This seems to suggest that de novo extraction may be preferable, but I'm not certain it is appropriate. Do you have any comment on this?

Comparisons between exposure types: this is another point where I would appreciate advice.
Again, there are a few choices on how to approach this, and they won't give the same answer.
Would you consider it a valid analytical approach to aggregate mutations from biological replicates (i.e., different individuals or different animals), in order to obtain enough mutations for de novo extraction?

Taking into account depth of sequencing for genomic regions
For Duplex Sequencing, we have seen people perform a normalization for the depth of sequencing of each of the 32 trinucleotide sequences to account for variability in the sampling of different genomic regions. Using other signature fitting algorithms, we have attempted to do the same thing; but in the case of SigProfilerExtractor, I believe you expect raw count data. Do you have any opinions on whether I should attempt to perform such a normalization prior to using SigProfilerExtractor?

Thanks again!
Matt

Hi Ludmil,
Thank you so much for taking the time to reply! I appreciate the thought you have given the questions.
Very interesting idea on the chemical exposures being used to somehow "reconstruct" signatures themselves from individual chemical mutational profiles. I will certainly consider that.
Regarding duplex sequencing... we are indeed using the "mutagenesis panel" with which I am sure you're familiar. The trinucleotide frequency was designed in a way to be reasonably representative of the respective mouse or human genome, but yes, that makes sense to normalize the signatures instead of the data, as you have suggested previously (and we have done for the lacZ gene). That gives me more food for thought.
All the best,
Matt


Allow builders to provide a BuilderID during Tekton Pipelines installation

Feature request
Allow builders to provide a BuilderID during Tekton Pipelines installation
For the SLSA provenance, Tekton Chains needs to fetch the builderID. This is currently done via a configMap in Tekton-chains. However, Tekton-pipelines is what is performing the build and builders should be able to provide a builderID that demonstrates how the system was configured to ensure the quality (SLSA L2, L3 etc.) of the build.
We (Tekton Chains WG) think that it might be better for the Tekton service providers to set this field during installation of Tekton pipelines instead of Tekton Chains. The Tekton pipelines controller should in turn attach this field to the taskrun/pipelinerun objects's status.
Suggestions

Make builderID a configMap option. Tekton pipelines controller can read that and attach it to the Status.

cc @lcarva @wlynch @chuangw6 feel free to add more info that I may have skipped.

What will be the default builderID in Tekton Pipelines? It defaults to "https://tekton.dev/chains/v2" in Tekton Chains
It's just a place holder I think. I don't think any default value would make realistic sense since the builder (or the service provider) needs to provide that. We can set it to something like not-provided by default.
Is there a way to generalize this to something that makes sense in Tekton Pipelines by itself? I have concerns about introducing SLSA concepts directly into Tekton Pipelines
Good question. I'm not sure yet. I wonder if including SLSA concepts into Tekton pipelines is a bad thing. Mainly because its Tekton Pipelines that is really doing the build so such information makes sense to be captured there.  But, I'm open to other ideas as well.

@chitrangpatel that "configmap" option would be cluster-wide (in tekton-pipelines namespace) or could be "per project" (aka per namespace or something) ? Or even per run ?

I had imagined that it would be cluster wide originally.

For example, we might have an instance of pipelines that is "consumed" by multiple "builders" (or even "non-builders" like kubeflow), so we may need to have a flexible way to set the builderID 👼🏼 .
Good callout here! I had not considered this use case. I agree. We should be able to distinguish between these.

Good question. I'm not sure yet. I wonder if including SLSA concepts into Tekton pipelines is a bad thing. Mainly because its Tekton Pipelines that is really doing the build so such information makes sense to be captured there. But, I'm open to other ideas as well.

I think @jerop's point is that tekton pipeline itself is opiniated-less, and is not necessarily used as a CI/build tool (aka it can support a lot of "workflows" that have nothing to do with CI and SLSA). So anything that helps implementing SLSA with pipeline is good, but having SLSA specific things in pipeline might be tangeant (it's more of a concept thing to think about but 👼🏼 ).

Fair enough!

@chitrangpatel not that it's definitely not a "no" on that feature btw, we just need to think it with the SLSA lens as well as without it 👼🏼

That's what I thought. Thanks for confirming 😄!


Add a dataloader for Bundle Adjustment

Need to write a dataloader for the BAL dataset in order to test the functionality of SBT.
It is recommended to use TorchData like https://github.com/pypose/pypose/pull/226 to keep implementation short.

I would like to take on this problem.

I would like to take on this problem.

That will be great! @zitongzhan can provide more information.


Cubemap is not saving

Godot_v2.1.5-rc2_x11
By default cubemap is not saved correctly to tres scene.
How to reproduce:
If you add WorldEnviroment to scene (camera as well).
Set background to cubemap. Load cubemap with 6 of defauilt godot's icon.png-s the cubemap shows in editor. But If you try to launch it - there is no cubemap shown.
If you save your project and restart godot editor - you will notice that only 1st of six images in cubemap was saved.
Good thing there is a way to avoid it:
It you need to manualy save cubemap to non-TRES format (like xxx.cbm)

This is an old bug, other issues maybe got closed since there is no more cubemaps on 3, the thing is that cubemaps always try to save the images inside the resource/scene making it too big and corrupting the scene file.
Another workaround is saving the scene with the cubemap as .scn to avoid the size problem which is not present in binary files.

PR #20307 fixes this issue for 3.1. As for 2.1, someone will need to port that fix across probably.

If its fixed in 3.1, then it should be closed, we are no longer provides non-critical updates for 2.x version


Add BSD 2-clause license

This PR adds a license to the project. If you prefer a different license, please disregard and push your own.

Thanks for that.


support for non dask arrays for HyperbandSearchCV

The main changes in this PR are:

change to  the async def _fit function in dask_ml.model_selection._incremental.py to allow Hyperband to work with non dask arrays

fixed default test_size which didn't work with pandas dataframe

@stsievert I implemented the changes we discussed.
sorry it took so long but I have been busy with other projects.
Cheers
Gio

@stsievert do you have another chance to look at this?

Hi @stsievert I agree with you in principle the check should be done on X_train.
And sorry for the comments, it was just for me to remember where I made changes.
Of course I can remove them.

does anyone have any idea of what happened with the tests?
it's like it's stuck
many thanks

I'm not sure, but it does look there's a linting issue (both black and isort fail):
Checking black...
black, version 19.10b0
would reformat /home/vsts/work/1/s/dask_ml/model_selection/_incremental.py
would reformat /home/vsts/work/1/s/tests/model_selection/test_hyperband.py
Oh no! 💥 💔 💥
2 files would be reformatted, 98 files would be left unchanged.
Checking black... DONE
Checking isort...
4.3.21
ERROR: /home/vsts/work/1/s/tests/model_selection/test_hyperband.py Imports are incorrectly sorted.
Checking isort... DONE

Try making those lint changes and pushing; that will (likely) resolve the issue.

I've fixed the linting changes locally (and substantially reduced the computational size of the test), but the test_pandas still hangs. I've played around a bit, but haven't been able to identify the root cause yet.


rofi-bluetooth: init at unstable-2021-03-05

Description of changes

Adds rofi-bluetooth, a script to control Bluetooth connection via rofi.
For more info/demo see the projects GitHub page.
Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[ ] Fits CONTRIBUTING.md.

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


Can we use fixed-data-table with react 0.13?

The package.json requires "react": "0.12.x"

Yes fixed-data-table works great with react 0.13 (internally Facebook uses react master), but you are right we should update the package to allow npm installing with<EMAIL_ADDRESS>We should be able to put out a new release pretty soon with this change.

@TomClarkson I have released 0.1.2 which updates the package.json to support react 0.13.0 as well as 0.12.x. Sorry i didn't get to this early but i couldn't get the package.json config to allow the beta version as a dependency.


Revert the sort order of episodes/seasons

Add options to be able to revert the default sorting strategy (latest first) of the episodes and seasons list (oldest first).

When episodes are displayed in descending order, the list is scrolled to the bottom. It should always be displayed from the top.