Add maps/local results detection

It would be nice to see Maps results detected in scrapes.

Hi, yes would be nice. In what kind of data you are interessted for map results?

basics i would look for would be if map results are shown and the content of each one.
Would be nice to extend that out to then visit the maps listings for the search and scrape each one of those. the below are what i am using for gmaps selectors currently in chrome selenium. there are some issues but its a start?
google home More Places link class
mp = ".DLOTif"
# CSS selector for all the listings in left column
al = ".uQ4NLd"
# Ad CSS selector for container
ad = ".Z98Wse"
# Image Container CSS selector
img = ".b9tNq"
# company name in left bar css selector
cn = ".dbg0pd"
# company name in the right hand side
cnr = ".kp-header .kno-ecr-pt span"
# right hand address
addr = ".LrzXr"
# rating value
rtng = ".immersive-container .TLYLSe .rtng"
# amount of google reviews text
ratingtext = ".immersive-container .TLYLSe .fl a"
# missing elements class name
missingel = ".qe9kJc"
# phone number
pho = ".lqhpac div:nth-child(4) span"


Roadmap: Interface

I want to start work on user interface (for consoles, gas tank, pumps, fire alarms, air alarms, gas mixers, etc) but I don't know archicture of project.
So I need consultaion, how much better introduce it, interaction between objects and ui, etc..
E.g. Gas Tank, I already made window for that one, it will show pressure and valve's status and player with it will can toggle valve.
But how introduce it? Where place it? In client/represation? But when player clicks on gas tank, it should create form probably through an interface (I supposed). For example,
void GasTank::AttackBy(IdPtr<Item> item)
{
    IdPtr<Hand> hand = item;
    if (!hand.IsValid())
    {
        return;
    }

    if (state_ == State::BROKEN)
    {
        return;
    }

   eazytouch_interface_->CreateForm("gastank", this);
}

Or something like that.

States of objects changes only when Process() called? If yes, we can update user interface form (e.g. gas tank, pressure's value and valve status, if it is broken then close form) for object at the same time with Object's Process. Else preferred make separate process.

This issue is actually pretty complex by its nature, so it will request some time from me to describe my vision properly. So I will answer in details in couple days probably.
But TLDR: changes everywhere will be needed (server, core, client), the issue is big.

Some notes can be found here #106
What do we want?
We want a way to display an arbitrary complex interface to user and interact with game via this interface. There will be plenty of such interfaces, and it would be nice to have ability to create them in a swift manner.
Some internal details
The game has 3 main parts: representation, core and server. All visible information is passed via frame https://github.com/griefly/griefly/blob/5445dda253aa8a9bac27de23a97c6cc4a18409bc/sources/core_headers/CoreInterface.h#L19 so it should be possible to fully draw interface from frame data (i.e. no chain dependencies from previous frames).
Obvious choice of a data format in frame for UI is JSON, but maybe there are better ways.
Overall:

With some way (e.g. browser) the UI is drawn on user screen.
When it is needed to make some interaction with the world then the UI passes the intention to the client. There should be certain standart way to do it, just like with usual UI and clicks.
The client passes that intention to the server (something like MakeClickMessage, but with some JSON as a parameter).
The server broadcasts such message to all cores (just like with other messages).
Each core handles the message and changes something if it is needed.
When Represent is requested (basically, every tick) the game object which represents the UI generates an appropriate JSON (or something) for client-side.
Goto 1.

Currently I think to pick HTML and JS as base for such UI, because they are easy tools for flexible interfaces. QWebChannel/QWebEngine can be very helpful here. UI should have 1 main entry point - something like stateUpdate(json) which will be called with new data each tick.
There are a lot of other small details, and they can be discovered only during the development. It may request a lot of testing and changes.


Workflow dispatch runner orchestrator

Once you get your multi repository product setup for gh actions, there's further things left like how could you fire a build of "everything"?
Simplest use case is to just fire up x number of workflow dispatches, in a correct order, waiting those workflows to succeed and eventually you should have last built repo. This is relatively simple for plain snapshot builds as you don't need to massage versions in a pom files(in case you're using maven).
For full release build this are more complex as we need to know a build version of a dependency project and that version is then a one which you'd need to massage poms in dependant projects.

Initial impl done.


Update download-mappings.md

If all CVS stuff is gone, is there only one way to access the mappings (http://geneontology.org/external2go/), or did I miss something

@kltm Is this the correct link for ? http://geneontology.org/external2go/ ?
Also, when will this be complete? http://release.geneontology.org/2018-10-01/ontology/external2go/index.html
Or are we not maintaining the other mappings ?
Thanks, Pascale

I believe that the fate of external2go is still under discussion.
tagging @cmungall

Closed due to file changes, please continue the discussion of external2GO on https://github.com/geneontology/go-docs/issues/39


Upgrade to version 1.1.0

Version 1.1.0 - 2023-08-04
Added:

Add changelog.md for tracking version changes
More example problems for warehouse domain

Changed:

Updated documentation (adding more descriptions): Input_Output_Format.md, Prepare_Your_Planner.md
Add upgrade instruction in README.md
Add warnings in logger when number of tasks in problem file is smaller than team size

Fixed:

Fixed issue with example task file for warehouse large only has 1000 tasks (smaller than team size), of which 10000 tasks must be replaced
Fixed issue with edge conflict checking causing integer overflow bug.

@GreatenAnoymous @YueZhang-studyuse @r06921017 @HanZhang39  I created a pull request to merge the newest bug fixes and changes to the main branch. Please play with the newest develop branch and check if you can find any issues.

delete one duplicate document


henchman version is not getting updated

Its for many builds..
henchman version 0.1.194

Need to pull that ldflags trick.
@jlin21 want to take a stab at this?
http://stackoverflow.com/questions/11354518/golang-application-auto-build-versioning

We have to update main.go for every feature push.  We just have to remember to change it

Nvm didn't see you comment @sudharsh

@sudharsh Do we want the min version to be linked with the build number of circle?  If we just want each build to be something different we can do it'll be more straightforward

Let's stick to commit hashes and build numbers. 1.[build].[commit]
On 08-Jan-2016 7:52 pm, "jlin21"<EMAIL_ADDRESS>wrote:

@sudharsh https://github.com/sudharsh Do we want the min version to be
linked with the build number of circle? If we just want each build to be
something different we can do it'll be more straightforward
—
Reply to this email directly or view it on GitHub
https://github.com/apigee/henchman/issues/113#issuecomment-170189265.


images replaced on KMSv2 blog article with SVG vector versions #36654

On publication, https://k8s.io/blog/2022/09/09/kms-v2-improvements/ Mermaid generated online images replaced using figures in the markdown source and used SVG format of images. https://github.com/kubernetes/website/issues/36654

/approve

/retitle Images replaced on KMSv2 blog article with SVG vector versions


Office365 Provider

Hi Dear do you have a new plan to add more providers like the office365 provider
regards.
shady

No. There are no plans for development. Now only support.
Thank you for message.


ImportError: No module named parse

I have Python 2.7 installed and I have this issue related to the name of a module of Python 3:
Reloading plugin /Users/johndoe/Library/Application Support/Sublime Text 2/Packages/GitHubMarkdownPreview-master/GHMarkdownPreview.py
Traceback (most recent call last):
  File "./sublime_plugin.py", line 62, in reload_plugin
  File "./GHMarkdownPreview.py", line 8, in <module>
    import urllib.parse
ImportError: No module named parse

Modifying the import urllib.parse for from urlparse import urlparse seems to fix the previous bug but after running it I get a "error: Error in GitHubMarkdownPreview package:" with no explanation.


Add  support for rosa cluster creation

two questions:

instead of writing the ROSA class and related scripts, couldn't you re-use the cluster creation code from ocm.py?
I understood that we were turning our ops script into robot framework tasks, correct me if I'm wrong (@mwaykole @tarukumar @manosnoam @aloganat). Are you planning to do the same for this scenario?

the minor changes will be taken care in next pr


Error when adding an eCheck payment account

JAXBException occurred : unable to marshal type "tyler.efm.services.schema.common.ErrorType" as an element because it is missing an @XmlRootElement annotation.

Looks like I was mistakenly trying to send back the entire XML error object instead of just the error text (why Java needs there to be an XML root and it can't just serialize the object is beyond me).
Should be fixed now.


Fix typos in policy descriptions

This PR fixes typos in policy descriptions, please see two examples below. Additionally, is there a better way to approach this bugfix/PR? The amount of changes in the commit diff looks enormous...
Before: "The policy wil  set"
After (fixed in my branch): "The policy will set"
Before: "all metrics  enabled."
After (fixed in my branch): "all metrics enabled."

Hi thanks @hjscherer I'll review and either close this PR or make changes to reflect.

@mabesignal any update on this? should we close this PR?


Remove the proposal of the single workload endpoint from KEP#168-2

What type of PR is this?
/kind cleanup
/kind api-change

What this PR does / why we need it:
The single workload endpoint is redundant, as we can achieve the same goals using LocalQueue endpoint. We'd like to narrow the scope of KEP#168-2 and leave it as a potential follow-up feature.
Which issue(s) this PR fixes:

Special notes for your reviewer:
Does this PR introduce a user-facing change?

NONE

/assign @mimowo
/assign @alculquicondor

/ok-to-test

If we don't implement the single workload endpoint, we need to implement similar logic in the kueue kubectl plugin, right?

Correct

I think it might be better to implement almost logic in the server side as much as possible.

Indeed. The PR proposes to just decouple this part of work, so that it can be prioritized independently. So, the idea was, that it would be added when needed, for example for the kueuecli. However, I'm not sure decoupling is a good idea in this case. WDYT?

I would just leave it in the same KEP, possibly highlighting that it's not implemented yet.

Not a good-first-issue actually....
Ideally, we should have a single endpoint that covers both pending and admitted workloads. Not that easy.

Ideally, we should have a single endpoint that covers both pending and admitted workloads. Not that easy.

Hm, but I think it still makes sense to expose the endpoint as in the KEP, just for pending, so that it would return the positions. I think the positions don't  think we have well defined positions for admitted workload, so we would need to re-design the API, which is a separate KEP then, I think.

IMO, looking at the "pending workloads" of a (Local/Cluster)Queue makes a lot of sense, and we should keep these endpoints long term.
For the case of workload, it might make sense to just have an endpoint for "workload", period. Although it's unclear which fields we would need for an admitted Workload that isn't already in the Workload status. My main worry is that a user does a GET for "pending workload X" and it gets 404, just because the workload was admitted.
All these implications need to be explored in a future KEP.

For the case of workload, it might make sense to just have an endpoint for "workload", period. Although it's unclear which fields we would need for an admitted Workload that isn't already in the Workload status. My main worry is that a user does a GET for "pending workload X" and it gets 404, just because the workload was admitted.
All these implications need to be explored in a future KEP.

SGTM, in that case, we just close this PR, and eventually complete this KEP with the new endpoint.
As for the worry about 404, indeed, we could mitigate this scenario by doing the /workload endpoint, but analogously we could get 404 just before the Job completed and the workload is deleted, but we can discuss the implications in the future KEP indeed.

My main worry is that a user does a GET for "pending workload X" and it gets 404, just because the workload was admitted.
All these implications need to be explored in a future KEP.

I agree with this concern. We should have another KEP if we want to have the workloads endpoint.


Expose current selected node into global variable

It would be very helpful to investigate typescript type nodes in dev tools console

Hey @cevek, yup, that's already a hidden feature :)
https://github.com/dsherret/ts-ast-viewer/blob/master/docs/about.md#interacting-with-compiler-objects

@dsherret Oh, nice! Thanks!


[#1673] feat(partition): Support list partition names for Hive catalog

What changes were proposed in this pull request?
Support list partition names for Hive catalog
Why are the changes needed?
Fix: #1673
Does this PR introduce any user-facing change?
no
How was this patch tested?
tests added

Base on #1577


Update popup.html

Added an image to the top.
Changed the "Add Protein" button to give a 3D image.

Thank you @DennisKnie really appreciate your work !


set personal data for clearingtypes other than creditcard

I find it a bit strange that in ShopServer gateway for clearingtypes other than creditcard the only way to set personal data like fistName, lastName, billingCountry is by 'card' => ['firstName' => ... ]
Is there a reason  why this is so focused on card authorizations? Or am I overseeing something?

Omnipay has a long history, starting from when only credit cards needed to be supported, and long before PCI requirements stopped sites handling credit card details directly. It still retains a lot of that legacy structure, and that is what you are seeing.
Ultimately what I would like to see are separate classes for personal details, payment types, shipping methods and so on. We aren't there yet, so personal details go into the mis-named credit card object. You don't need to put credit card details into that object though, just use it for personal details.
Gateways that support other payment types will handle it their own way. Collecting together all of these methods from all the 100+ Omnipay gateways to standardise them is a massive job, but yeah, I think it needs to happen.

Hi Jason, Thanks for the quick response & explanation. Good to know.


Technical blog on implementing full datascience pipeline

Checklist

[X ] You have read the Contribution Guidlines before creating this PR.
[X ] You created a branch from dev branch
[ X] Blog follows Blog Guidelines.
[X ] Checked for all grammatical errors in the blog.
[X ] Checked for any duplicate blog entry.
[X ] You PR is raised on dev branch and not onmaster.

Author Details for Blog Post Entry
id: Rinki Nag
bio: Hi, I'm Rinki, an AI enthusiast and currently 🔭 working with the AIML team. at Fractal analytics. I love experimenting and learning new technologies ⚡
github:eaglewarrior
stackoverflow:
linkedin:https://www.linkedin.com/in/rinki-nag/
medium:
twitter:

@puneetsingh24  I have just realized that I have raised PR from my master branch, I have raised a separate PR #326 from my dev branch to login radius dev branch

Contributor raised a new pull request


Created Pandovan.java

Hey! I created the Pandovan Sequence in Java. Please review the PR. Thanks!

Hey it works well, thanks.


Align carousel indicators at bottom middle

It would be nice to simply set a middle class such as <ol class="carousel-indicators middle"> and have this all set. It's very simple to override the actual indicators:
.carousel-indicators.middle {
	left: 0;
	right: 0;
	top: auto;
	bottom: 15px;
	text-align: center;
}
	.carousel-indicators.middle li {
		float: none;
		display: inline-block;
	}

This is the result:

Just setting the "top" css attribute of the class as shown below should place the indicators where (350px from the top in this case) you want them.
.carousel-indicators {
top: 350px;
}


Fixes #483 to use shorter dependency names in spring-boot-new-project…

… if running in CLI mode

[merge]


WIP: implement F1 HelpContextService

Initial stab at implementing #1938

I haven't yet deleted any of the old F1 help code or tests.
I've copied all existing unit tests. I am not sure how much I can delete. :/
Two tests involving type providers and assembly references are commented out, all others pass.

Fabulous.  Really fabulous!

Super happy to see this!  I think we can leave the old F1 help code for now.  Once we ship RTM of Visual Studio, one of the tasks we'll have is to delete all the dead code in the old language service.  #1679 is tracking that.

Once we ship RTM of Visual Studio, one of the tasks we'll have is to delete all the dead code in the old language service

Just a heads up that the really big challenge will be to bring across the massive amount of testing in VisualFsharp.Tests that sits on top of this code, all of which is valuable.  This is currently exercising the old language service code, but which in turn exercises the FCS API, and is a key source of correctness testing for FCS.  FCS itself doesn't test many editor features because there is so much testing via this route.
We can lose a little bit of the testing along the way.  But there is really important stuff here that represents many man years of investment and crucial corner-case bugs in both FCS, the F# compiler itself and even type providers.

Yup, agreed.  Hmmm.   I may be wrong but I cannot remember F1 working ever, starting from VS 2012.

All old tests have been enabled now. I'm pretty sure the CI errors are because #1967 has not been merged.

Really good work on the tests.  Can you delete the corresponding tests in the old test suite as part of this PR please?  That will avoid someone duplicating the work later. thanks

Done. I didn't delete any of the old code though.

Rebased against master. Still works.

Then it probably doesn't need the path qualifier.

@rojepp
Thanks for taking care of this.
Kevin


remove koa-convert

Coverage remained the same at 86.733% when pulling 7e589f269512a78bd32b8671447af915a27c4571 on koa-convert into 34f36a29e6bef8f7437413ca8276471d4a7efdd8 on master.

Coverage increased (+0.9%) to 87.617% when pulling 35fde9fb0a09c28a6baf9990cd72708ae8868c06 on koa-convert into 34f36a29e6bef8f7437413ca8276471d4a7efdd8 on master.


How to use MaxItems in a multi-field FieldArray/Repeat

@aitboudad  Appreciate all the efforts you have put in into this package and your prompt follow-up
Question
How to use MaxItems in a multi-field FieldArray/Repeat
I have looked at issue #1692 where you have an example for MaxItems in a single field.
I have created the following stackblitz to capture my usage scenario.
https://stackblitz.com/edit/angular-uurguv?file=src%2Fapp%2Fapp.component.ts
Additionally I have one more observation to make.
For some reason in my setup when I add "location" into the model as shown in this stackblitz I don't see the fields appear on my screen unless I click the "Add" button. But it does work in the stackblitz.
Any thoughts on what could be causing this behavior?

There is no diff between single and multi-field array, rely on to.maxItems option:
<button *ngIf="!to.maxItems || to.maxItems > field.fieldGroup.length" (click)="add()">
 {{ to.addText }}
</button>

Any thoughts on what could be causing this behavior?

verify the model input is defined [model]="model" and avoid using push or object assignment which doesn't detect model changes:
// detect change
this.model = { location: [{}] };
this.model = { ...this.model, location: [{}] };

// won't detect change
this.model.location = [{}];
this.model.location.push({});

@aitboudad
Thank you for the prompt response and the recommendation.
Looking back the original example, it looks like I missed the "ngIf....." part within the repeat type. I have tested it and it works fine.
On the second part of the question, I am using the the first option in the "won't detect change"  which explains the behavior in my setup.


Bun integration for disco-(core/node/web) and server

This PR changes the node runtime to Bun.
Bun is fast, heavily maintained, and most importantly for this project, allows to avoid the compilation of disco-core whenever a change is made. Moreover, it provides clear type definitions (@types/bun) and a well made test suite (bun:test)

So, I tried a bit more but bun full support seems too early for disco

doesn't really work for monorepo
it can't replace tsc (the TypeScript compiler) because of missing .d.ts generation

it is needed when publishing a version to NPM, or building a local version

I tried to get all the files of discojs-* and the server to be watched but I was unable to do so

even with path rewriting, it looks into the dist dir which is populated by tsc

Overall, it's a bit disapointing: it's a cool technology but it has to mature a bit more for theses use cases.
I'll close this PR in favor of simply using #617, which is slowler and less fancy but is fonctional.

it indeed does not support monorepo, I'll ask Jarred and the team over on discord for it
tsc can be used separately,the usage of bun does not prevent the use of tsc it think. Anyways, someone made a bun plugin for it: https://github.com/wobsoriano/bun-plugin-dts
I believe the paths to watch are detected during runtime. Files that are imported and functions that are called are added to a watchlist sort of I think. Could you be more specific with the issue you had and what you did exactly? I had some issues with hot reloading and napi. Also, you can try both --hot and --watch, they work slightly differently

tsc can be used separately,the usage of bun does not prevent the use of tsc it think. Anyways, someone made a bun plugin for it: https://github.com/wobsoriano/bun-plugin-dts

ho, I didn't knew about this plugin. at least, it can help consumer of the lib with types.
the issue at core there is that I want to actually publish the packages to NPM so I need to build the code itself as JS. sadly bun really wants to bundle everything so I'm unable to use it for that (no tree shaking for common deps). so it's only useful for running TypeScript, not build it.

I believe the paths to watch are detected during runtime. Files that are imported and functions that are called are added to a watchlist sort of I think. Could you be more specific with the issue you had and what you did exactly? I had some issues with hot reloading and napi. Also, you can try both --hot and --watch, they work slightly differently

I was hoping to improve the server build/run workflow, smth like cd server && npx bun run --watch ./src/run_server.ts that restarts the process when modifying smth in disco-node/src (not dist). even when adding some paths config to tsconfig.json, it doesn't watch theses.
"paths": {
    "@epfml/discojs-core": ["../../discojs/discojs-core/src"],
    "@epfml/discojs-node": ["../../discojs/discojs-node/src"]
}

so I still need to build discojs-{core,node} by hand to restart the process.
I think the main issue here is that I don't really know where bun will be useful for us: is doesn't work as a packager, nor as a auto-builder. the only remaining thing is that I can run files, but that's already taken care of by tsc & node.

after discussion with @peacefulotter, bun doesn't seems to work as well as wanted for our use-cases. anyway, it's a very cool intiative, that will probably work better in some months, let's keep an eye on it, thanks for the work.


Is the Power Apps CLI open source?

It's a nice start but has several limitations that would be nice to fix.

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

ID: 46d11001-a078-60f0-19d9-357f9397c7a7
Version Independent ID: 395e7245-8c1d-d168-ee6e-cc18b79bd789
Content: Install Power Apps CLI - Power Apps
Content Source: powerapps-docs/developer/common-data-service/powerapps-cli.md
Service: powerapps
GitHub Login: @Nkrb
Microsoft Alias: nabuthuk

@onetalltree Power Apps CLI is not an open source tool. You can share your ideas for the features at https://ideas.powerapps.com. Closing this issue.


Better bets handling

[x] - show avatar of user for everyone.

[x] - delete skins 100 in game stats

[x] - show koef/retrieveKoef of the bet

[x]  - show avatar of user for everyone.
[x]  - delete skins 100 in game stats
[x]  - show koef/retrieveKoef of the bet

1


AssertionError when crossProject is used as a ProjectRef in another project

Hello everyone!
I encountered with a problem appeared when crossProject imports in other one as a ProjectRef.
This repo show how to reproduce this error https://github.com/yelbota/scalajs-crossProjectRef-error
Travis build log https://travis-ci.org/yelbota/scalajs-crossProjectRef-error

The Travis URL you point to does not show anything.

I get this error locally:
doeraene@lamppc37:~/projects/scalajs-crossProjectRef-error$ sbt
[info] Loading global plugins from /localhome/doeraene/.sbt/0.13/plugins
[info] Loading project definition from /localhome/doeraene/projects/scalajs-crossProjectRef-error/project
[info] Updating {file:/localhome/doeraene/projects/scalajs-crossProjectRef-error/project/}scalajs-crossprojectref-error-build...
[info] Resolving org.fusesource.jansi#jansi;1.4 ...
[info] Done updating.
[info] Loading project definition from /localhome/doeraene/projects/scalajs-crossProjectRef-error/crossProjectRef/project
[info] Updating {file:/localhome/doeraene/projects/scalajs-crossProjectRef-error/crossProjectRef/project/}crossprojectref-build...
[info] Resolving org.fusesource.jansi#jansi;1.4 ...
[info] Done updating.
java.lang.AssertionError: assertion failed: Directory /localhome/doeraene/projects/scalajs-crossProjectRef-error/.jvm is not contained in build root /localhome/doeraene/projects/scalajs-crossProjectRef-error/crossProjectRef
        at scala.Predef$.assert(Predef.scala:179)
        at sbt.Load$.checkProjectBase(Load.scala:345)
        at sbt.Load$.sbt$Load$$resolve$1(Load.scala:380)
        at sbt.Load$$anonfun$resolveBase$1.apply(Load.scala:383)
        at sbt.Load$$anonfun$resolveBase$1.apply(Load.scala:383)
        at scala.Function1$$anonfun$compose$1.apply(Function1.scala:47)
        at scala.collection.TraversableLike$$anonfun$map$1.apply(TraversableLike.scala:244)
        at scala.collection.TraversableLike$$anonfun$map$1.apply(TraversableLike.scala:244)
        at scala.collection.immutable.List.foreach(List.scala:318)
        at scala.collection.TraversableLike$class.map(TraversableLike.scala:244)
        at scala.collection.AbstractTraversable.map(Traversable.scala:105)
        at sbt.Load$.projects(Load.scala:407)
        at sbt.Load$.loaded(Load.scala:308)
        at sbt.Load$.loadAll(Load.scala:334)
        at sbt.Load$.loadURI(Load.scala:289)
        at sbt.Load$.load(Load.scala:285)
        at sbt.Load$.load(Load.scala:276)
        at sbt.Load$.apply(Load.scala:130)
        at sbt.Load$.defaultLoad(Load.scala:36)
        at sbt.BuiltinCommands$.doLoadProject(Main.scala:481)
        at sbt.BuiltinCommands$$anonfun$loadProjectImpl$2.apply(Main.scala:475)
        at sbt.BuiltinCommands$$anonfun$loadProjectImpl$2.apply(Main.scala:475)
        at sbt.Command$$anonfun$applyEffect$1$$anonfun$apply$2.apply(Command.scala:58)
        at sbt.Command$$anonfun$applyEffect$1$$anonfun$apply$2.apply(Command.scala:58)
        at sbt.Command$$anonfun$applyEffect$2$$anonfun$apply$3.apply(Command.scala:60)
        at sbt.Command$$anonfun$applyEffect$2$$anonfun$apply$3.apply(Command.scala:60)
        at sbt.Command$.process(Command.scala:92)
        at sbt.MainLoop$$anonfun$1$$anonfun$apply$1.apply(MainLoop.scala:98)
        at sbt.MainLoop$$anonfun$1$$anonfun$apply$1.apply(MainLoop.scala:98)
        at sbt.State$$anon$1.process(State.scala:184)
        at sbt.MainLoop$$anonfun$1.apply(MainLoop.scala:98)
        at sbt.MainLoop$$anonfun$1.apply(MainLoop.scala:98)
        at sbt.ErrorHandling$.wideConvert(ErrorHandling.scala:17)
        at sbt.MainLoop$.next(MainLoop.scala:98)
        at sbt.MainLoop$.run(MainLoop.scala:91)
        at sbt.MainLoop$$anonfun$runWithNewLog$1.apply(MainLoop.scala:70)
        at sbt.MainLoop$$anonfun$runWithNewLog$1.apply(MainLoop.scala:65)
        at sbt.Using.apply(Using.scala:24)
        at sbt.MainLoop$.runWithNewLog(MainLoop.scala:65)
        at sbt.MainLoop$.runAndClearLast(MainLoop.scala:48)
        at sbt.MainLoop$.runLoggedLoop(MainLoop.scala:32)
        at sbt.MainLoop$.runLogged(MainLoop.scala:24)
        at sbt.StandardMain$.runManaged(Main.scala:53)
        at sbt.xMain.run(Main.scala:28)
        at xsbt.boot.Launch$$anonfun$run$1.apply(Launch.scala:57)
        at xsbt.boot.Launch$.withContextLoader(Launch.scala:77)
        at xsbt.boot.Launch$.run(Launch.scala:57)
        at xsbt.boot.Launch$$anonfun$explicit$1.apply(Launch.scala:45)
        at xsbt.boot.Launch$.launch(Launch.scala:65)
        at xsbt.boot.Launch$.apply(Launch.scala:16)
        at xsbt.boot.Boot$.runImpl(Boot.scala:32)
        at xsbt.boot.Boot$.main(Boot.scala:21)
        at xsbt.boot.Boot.main(Boot.scala)
[error] java.lang.AssertionError: assertion failed: Directory /localhome/doeraene/projects/scalajs-crossProjectRef-error/.jvm is not contained in build root /localhome/doeraene/projects/scalajs-crossProjectRef-error/crossProjectRef
[error] Use 'last' for the full log.
Project loading failed: (r)etry, (q)uit, (l)ast, or (i)gnore?

Check it now.

I fix a travis. Error logs are matches

Could we please have a minimal example in this ticket itself?

Could we please have a minimal example in this ticket itself?

+ /build.sbt
+ /crossProjectRef/build.sbt

crossProjectRef/build.sbt contains definition of crossProject: crossProjectJS and crossProjectJVM. build.sbt in root contains two ProjectRefs. Thats all. When we try to run a project an error occurs.
Directory scalajs-crossProjectRef-error/.jvm is not contained in build root scalajs-crossProjectRef-error/crossProjectRef

I believe that the error is in the example (and hence the original build). Using this as the subproject file works:
lazy val crossProjectRef = crossProject.crossType(CrossType.Pure).in(file("crossProjectRef")).
  settings(scalaVersion := "2.11.6")

lazy val crossProjectRefJS = crossProjectRef.js

lazy val crossProjectRefJVM = crossProjectRef.jvm

Why? see http://www.scala-sbt.org/0.13.5/docs/Getting-Started/Multi-Project.html#default-root-project
"Any .sbt files in foo, say foo/build.sbt, will be merged with the build definition for the entire build, but scoped to the hello-foo project."
so when the sub project sbt file is merged, the directory stays as is - "."

file("crossProjectRef")

Looks like absolute path. What if a referenced project placed outside of hierarchy? It could be project in other git repository.
See
http://www.scala-sbt.org/0.13.5/docs/Detailed-Topics/Paths.html#constructing-a-file

Relative files should only be used when defining the base directory of a Project, where they will be resolved properly.

val root = Project("root", file("."))

yes -  only be used when defining the base directory of a Project - But lazy val crossProjectRef = crossProject.crossType(CrossType.Pure).in(file(".")) is not a Project - it is something else

is not a Project - it is something else

Semantically it is a project. CrossProject have similar API and used for define "real" projects.

Look at this. ProjectRef can be anywhere
http://www.scala-sbt.org/0.12.4/docs/Dormant/Full-Configuration.html#project-relationships

I'm just saying that it's not a Project (a sealed trait), and can't be thought as such


Error on build production

Error on dependency parsing for ng2-handsontable/bundles/ng2-handsontable.umd.js at file:///Users/xxxx/Documents/ulcalculator/node_modules/ng2-handsontable/bundles/ng2-handsontable.umd.js
Loading dist/tmp/app/pagetable/pagetable.module.js
Loading dist/tmp/app/app.module.js
Loading dist/tmp/app/main.js
Multiple anonymous defines.

Details:
originalErr: Error: Multiple anonymous defines.
Stack:
Error on dependency parsing for ng2-handsontable/bundles/ng2-handsontable.umd.js at file:///Users/xxxx/Documents/ulcalculator/node_modules/ng2-handsontable/bundles/ng2-handsontable.umd.js
Loading dist/tmp/app/pagetable/pagetable.module.js
Loading dist/tmp/app/app.module.js
Loading dist/tmp/app/main.js
Error: Multiple anonymous defines.
Help me please.  :)

Hello,
Can you tell wich version of npm you are running?  Have you had any error when you were downloading the modules?

"@angular/common": "^4.1.0",
"@angular/compiler": "^4.1.0",
"@angular/core": "^4.1.0",
"@angular/flex-layout": "2.0.0-rc.1",
"@angular/forms": "^4.1.0",
"@angular/http": "^4.1.0",
"@angular/material": "^2.0.0-beta.7",
"@angular/platform-browser": "^4.1.0",
"@angular/platform-browser-dynamic": "^4.1.0",
"@angular/compiler-cli": "^4.0.0",
"@angular/platform-server": "^4.1.0",
"ng2-handsontable": "^1.0.3",


Motion create with specific group permissions not possible

Describe the bug
If a participant has the group permissions 'can see agenda, 'Can see internal items and time scheduling of agenda, 'Can manage agenda', 'can see motions', 'Can create motions' and 'Can manage motion metadata' motion creation is not possible. By saving it an error appears.
Error message: Fehler: You are not allowed to perform action motion.create. Forbidden fields: agenda_type, agenda_create
How to Reproduce
Steps to reproduce the behavior:

open a meeting
open participants > groups > grant the delegate group the above mentioned permissions
open participants > create a user as delegate
login as the user > open motions > create a new motion > enter title and text > try to save

Expected behavior
The motion creation with the specific set of permissions should work.
Please check, if agenda_type, agenda_create would have other negative impacts on creating motions.
It could be possible, that there is a relation to the issue #4290

The user should be allowed to create agenda item, if he has the permission for it. The backend doesn't allow it.
As far as I can see, the backend action is now missing the agenda item permission check and a test for it.
-> To backend.


Random note

Please Select if your Request is Either something new or an Enhancement

[ ] Enhancement of an existing Feature.
[x] Request of a new feature.

Please select the area your request applys to. (Multiple selections are Possible)

[ ] Onboard. Initial vault setup and import/export pods
[ ] Create. Note creation, lookup, snippets and templates
[x] Retrieve. Backlinks, references, graph view
[ ] Structure. Refactoring, multi-vault and schemas
[ ] Publish. Sharing your repo with the world
[ ] Misc (Choose this if your not sure)

Is your feature request related to a problem? Please describe
I would like to have a way to open a random note from my vault, similar to how the Obsidian's Random note command works.
Describe the solution you'd like
A Dendron command to open a random note.
Describe alternatives you've considered
vscode-memo extension has a "Open random note" command which seems to work fine on top of a Dendron vault, but using yet another extension for just this one feature is excessive.
Additional context
Ideally, I this command should be configurable to whitelist/blacklist certain hierarchies from which you do or do not want random notes to be picked up.

+1 I think this is something great to have to re-discover knowledge in your notes

Think this is a great idea, especially with the addition of whitelisting/blacklisting


Add a recursive sbt.IO.listFiles

As of SBT 0.13.7, there doesn't appear to be a way to recursively list files in a directory using the provided sbt.IO.listFiles helper functions.  Essentially I want an sbt.IO.listFiles that lists all files within a given directory, and optionally all of its subdirectories.
http://www.scala-sbt.org/0.13.7/api/index.html#sbt.IO$
When needed, I end up doing something like this peppered around several build definitions:
def listFilesRecursively(dir: File): Seq[File] = {
  val list = sbt.IO.listFiles(dir)
  list.filter(_.isFile) ++ list.filter(_.isDirectory).flatMap(listFilesRecursively)
}

It would be great if SBT could fold this into sbt.IO — either by creating a new function named listFilesRecursively, or add an optional recursive:Boolean = false argument to the existing sbt.IO.listFiles functions which one could set to true to make them recursive as needed.

Just in case anyone comes across this, Path.allSubpaths

We should add this to the docs. Can we reopen @eed3si9n and add the "missing docs" label? That way we won't forget.


Ensure that depth write state is updated before transparent pass in OpenGL3 renderer

Fixes: https://github.com/godotengine/godot/issues/75063
The core of the issue comes from here:
https://github.com/godotengine/godot/blob/ab7cb2a95d060a6533e6ff5111c11f71972ab43f/drivers/gles3/rasterizer_scene_gles3.cpp#L1949-L1951
and here:
https://github.com/godotengine/godot/blob/ab7cb2a95d060a6533e6ff5111c11f71972ab43f/drivers/gles3/rasterizer_scene_gles3.cpp#L2138-L2152
The behaviour of DEPTH_DRAW_OPAQUE changes depending on if we are in the transparent pass or in the Opaque pass. Further, the setting doesn't get updated unless another depth draw type is used.
#75063 is fixed just by setting the default depth draw mode to DEPTH_DRAW_ALWAYS before the opaque pass. However, a similar issue would arise if there were opaque objects in the scene. Therefore, the full solution is to ensure that the depth mode defaults to DEPTH_DRAW_DISABLED before the transparent pass so the glMask() state is correctly set regardless of what states were used in the opaque pass.
This PR also removes a check for the depth test state that was totally unnecessary as the state was set immediately after.

Cherry-picked for 4.0.3.


cant return to title screen with this launcher

as the title says you cant return to title screen with this launcher. only KH1 FM tested.
somtimes it strait up crashes, or it eternally is loading. i primarally play with special k, as it that helps with the perodic crashes (specificly the playtime crash of 1 -3 hours of play) in the game. but with or without special k this launcher breaks returning to title screen.
to reproduce

launch KH1 final mix through the simple launcher. note: skip copyright screen dosent matter. it crashes weather thats enabled or disabled
get to the title screen and load any game save. location of the save dosent matter.
use the save point to try and return to the title screen.
the game will be stuck on the spinning heart loading screen.

expected behavior
the game returns you to the title screen with some copyright notices before.
my specs
mobo: MSI PRO B650-P WIFI
CPU: AMD Ryzen 5 8500G
GPU: GIGABYTE GeForce RTX 4070 Ti SUPER WINDFORCE OC 16G (GV-N407TSWF3OC-16GD)
memory: 2x16 teamgroup DDR5-5200
OS: Windows 10 pro 22h2
full specs in screenshot (note: the memory clock in the screenshot is wrong. i have expo enabled and my ram is running at its intended 5200 mhz)

can't know what's the cause personally
if somebody else figures it out let me know

sorry for the delay i was hospitalised.
ok tested on KH1 with the "return or original launcher" and it still gets stuck on loading.
KH re:com works though

i have no clue then

KH2 return to title works.

All "Return To Title" options, apart from KH1, are genuine and will work with this launcher.
KH1 straight up crashes the game when returning to title, and reboots using the launcher, hence it won't work.


Add cookie banner code example to page template

Add the default cookie banner to the customised page template example as discussed in https://github.com/alphagov/govuk-design-system/issues/1475#issuecomment-775053243.
Partially fixes https://github.com/alphagov/govuk-design-system/issues/1475

You can preview this change here:
Built with commit d8708ffd340fbcf3a5007d9323daf1bdbb95a1a1
https://deploy-preview-1495--govuk-design-system-preview.netlify.app


refresh the browser get 404

Hello,
I'm trying to use ng-admin to create admin panel. It's kinda working (except it doesn't show any data from backend -- i'm still trying to figure it out how to use it). My question is, when I refresh the browser on example.com/admin (where ng-admin app being used) which redirected to /dashboard and because I do not have route to /dashboard, it ended up get 404 page that provided by sails js. Is this expected behavior or should I configure some file to make it work ? Thanks

ng-admin redirects to #/dashboard, not to /dashboard (see the demo). There must be something wrong in your angular setup.

Ah thanks! I accidentally added unnecessary routes in module. Removing that fixed the issue.


hotfix: fix tooltip link color

Issues
Fix tooltip link color
Fixes #3742
Changes

Fix color property not applied to link

How to test

Go to a proposal list with vote
Hover the icon, the tooltip text should be same as if it was a text

Self-review checklist

[x] I have performed a full self-review of my changes
[x] I have tested my changes on a preview deployment
[ ] I have tested my changes on a custom domain
[ ] I have run end-to-end tests yarn cypress:test:e2e, and they have passed

Normal text color also seems to be affected

Normal text color also seems to be affected

Can you point me to a proposal where I can see this in live ? I've only tested in a simulated env

https://snapshot.org/#/stgdao.eth/proposal/0x9b511ccf8bd097176255da63725877de9f80caeea8a28256ff50dfad10a3820d
Search for vote from 0x5acfbbF0aA370F232E341BC0B1a40e996c960e07


Download intellij platform sources does nothing

What happened?
I opened the com.intellij.openapi.application.ModalityState class, and I asked to download intellij platform sources in order to see the ModalityState.java sources. I see progress bar saying it's downloading sources, then nothing happen. I still see the "Download intellij platform sources" button, and I don't see the ModalityState.java sources.
Please see uploaded logs and video: Upload id: 2024_08_27_jBSbyVSkvqo9xAgEWYB6Cr (files: 2024-08-27 13-36-35.mkv and 2 more)
Relevant log output or stack trace
No response
Steps to reproduce
try to open ModalityState and download sources
Gradle IntelliJ Plugin version
2.0.1
Gradle version
8.10
Operating System
Windows
Link to build, i.e. failing GitHub Action job
No response

Thanks for reporting.
Which IntelliJ Platform do you target?
What happens if you manually trigger the Gradle project refresh after downloading sources?

I'm targeting IC 2024.2.1.
I can see sources if I trigger the Gradle project refresh after downloading sources. I have to do this in all my projects.

Thanks for confirming. This is a problem that should be somehow addressed within the DevKit plugin inside the IntelliJ IDEA.
Related issue: https://youtrack.jetbrains.com/issue/IJPL-158473/Support-more-cases-of-attaching-IntelliJ-Platform-sources-by-DevKit-plugin


pull request for e2e test at 12/31/2016 17:26:12

:white_check_mark: Validation status: passed
For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Fix the way we target linting at specific path

Resolves #4117
What is being addressed
Linting tags was supposed to target specific paths for each pass, but apparently it didn't or a update caused it to malfunction.
How is this addressed

Update the environment variable name used to send the path to the linter

/test-force-approve

LGTM if it works. 😊

It does :-)

/test-force-approve


teleport: 7.3.2 -> 8.0.6

Motivation for this change
New major version.
Added support for the role tester component written in Rust. Did not manage to package RDP desktop access just yet, as it uses a fork of a crate (outside of crates.io).
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

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

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

[x] Fits CONTRIBUTING.md.

JFYI #150717

JFYI #150717

@ymatsiuk Thanks for the ping, I sort of forgot about this PR. Updated it to 8.0.6, should basically superseed your PR. Hope you don't mind.

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

1 package built:

teleport

@justinas could you please add doCheck = !stdenv.isDarwin section. It builds fine on my Mac, but not tests 🤷

@justinas could you please add doCheck = !stdenv.isDarwin section. It builds fine on my Mac, but not tests shrug

Added it, seems like an okay workaround for now.

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

1 package built:

teleport

This is tested and ready to merge, so I'm randomly adding reviewers and hoping for merge 😉 🙏🏻


[Feature] env::log sent to stdout, making it difficult to parse env::write's output

Feature
Since #1115, the output of env::log is also sent to stdout; to allow it to be read from the host. However, this means it gets mixed with the output written using env::write(), making it difficult to parse the latter. It'd be better if there was a way to distinguish the messages written to the log from the secret values written using env::write().
Motivation
If you write a program that uses env::log (maybe only during development) and env::write (to output secret values), it becomes very difficult to parse the values written using env::write in the host code. Ideally, I'd like to do something similar to the voting machine example, where the value is simply converted using from_slice. However, my stdout contains some logging messages before and after the actual value, and the length of those messages can change.
Implementation
An idea is to send the output to env::log to stderr. Hence, stdout would only contain values the developer wrote using env::write, while stderr would contain 'debugging info'. Alternatively, the log could be a separate output from stdout and stderr.

related to https://github.com/risc0/risc0/issues/1038

I'm going to close this as duplicate of #1038 so let's move the discussion to that issue.
@jswalens we'll be implementing the idea in #1038 shortly. In the meantime could you give feedback the idea proposed in #1038?


About project structure

Thank you for your work! Regarding the project structure, I would like to know the design purposes of the modeling_eagle and ea_model source files? It appears that both describe the structure of the original model and a single decoder layer. Perhaps modeling_eagle is specifically designed for inference using a custom model?

Perhaps modeling_eagle is specifically designed for inference using a custom model?

Yes, modeling_eagle can be used to accelerate any model in the transformers library. To use modeling_eagle, you need to slightly modify the code (pre-allocated KV cache and tree mask, refer to modeling_llama_kv.py and modeling_Mixtral_kv.py for examples).


Refactor press-scorer

This PR refactors the current press class as described in detail in #21.

Most presses are now composed of a scorer-pruner combination, e.g. DefaultPruner(compression_ratio=compression_ratio, scorer=KnormScorer()).
Presses are flexible and don't need to have compression_ratio, see e.g. PerLayerCompressionPruner.
Another example of flexibility is ThinKPress that implements channel compression.

Note: As the Pr is already quite large, I didn't add more refactoring, as suggested in the comment of #21. These helpful changes can be added in a subsequent PR.
Closes #21

A proposal to refactor the refacto and make it more compact would be to have only 2 directories (presses and scorers) instead of 3 (presses, prunner and scorers).
kvpress
├── __init__.py
├── pipeline.py
├── presses
│   ├── __init__.py
│   ├── base_press.py
│   ├── base_pruner_press.py
│   ├── observed_attention_press.py
│   ├── per_layer_pruner_press.py
│   └── think_press.py
└── scorers
    ├── __init__.py
    ├── base_scorer.py
    ├── expected_attention_scorer.py
    ├── ...

Basically, the word "pruner" is replaced by the the word "press" and:

BasePress only have a forward_hook method raising NotImplementedError
BasePrunerPress has a compression_ratio parameter and is the default one used for all scorers
eager_attention_pruner.py is replaced by observed_attention_press.py

At the end of each file _scorer.py, I would define the associated BasePrunerPress within the same file to simplify the workflow to add a new press. This can be done in a more pythonic way using multiple inheritance:
class MyScorer:
    ...

class MyPress(MyScorer, BasePrunerPress):
    pass

I would also remove the inner_press argument in ThinKPress because it's a general press composition operation that could be implemented easily. This can be done in a follow up PR but at least I would remove it from the README.

One of the major impacts of this refacto is that compression_ratio is not mandatory anymore as input of a press. I would make it clearer in the README.
However, we should maintain an easy way to measure the compression_ratio for a given prompt. Please add a test to make sure that any press has an attribute compression_ratio after you run it on a prompt. I would create a PRESS_LIST in the tests so that when you add a new press, you don't have to edit the tests in multiple places.


feat: add an AttibutesStamp to handle serialized message attributes

As stated in the README, it comes very handy when combined with subscription filters (also the use case we currently need).
I did not add the stamp back in the receiver since there is already a GpsReceivedStamp which allows to do:
$stamp = $envelope->last(GpsReceivedStamp::class);
if ($stamp instanceof GpsReceivedStamp) {
    $attributes = $stamp->getMessage()->attributes();

    // do something with attributes
}

@HeahDude Hi, I also need the pubsub filtering feature. Why is it still waiting? Is there any reaction from contributors?

@HeahDude thanks for contribution


Rustup

@bors r+

:pushpin: Commit 716dc0928c70eda002855e680b782d207982bd02 has been approved by RalfJung
It is now in the queue for this repository.

:hourglass: Testing commit 716dc0928c70eda002855e680b782d207982bd02 with merge 8095bbc0a85b4c3dac5919117cfd04c52801ee72...

:sunny: Test successful - checks-actions
Approved by: RalfJung
Pushing 8095bbc0a85b4c3dac5919117cfd04c52801ee72 to master...

:sunny: Test successful - checks-actions
Approved by: RalfJung
Pushing 8095bbc0a85b4c3dac5919117cfd04c52801ee72 to master...


Bug in testme in fitModel

When trying to call testme() in fitModel I get:
Note the data file is there, but testme() is not finding it.
import teUtils
teUtils.fitModel.testme()
Traceback (most recent call last):
File "", line 1, in
teUtils.fitModel.testme()
File "C:\Tellurium-Winpython-3.7\python-3.7.7.amd64\lib\site-packages\teUtils\fitModel.py", line 101, in testme
f = fitModel('testdata.txt')
File "C:\Tellurium-Winpython-3.7\python-3.7.7.amd64\lib\site-packages\teUtils\fitModel.py", line 22, in init
self.timeSeriesData = np.loadtxt(fileName, delimiter=",")
File "C:\Tellurium-Winpython-3.7\python-3.7.7.amd64\lib\site-packages\numpy\lib\npyio.py", line 961, in loadtxt
fh = np.lib._datasource.open(fname, 'rt', encoding=encoding)
File "C:\Tellurium-Winpython-3.7\python-3.7.7.amd64\lib\site-packages\numpy\lib_datasource.py", line 195, in open
return ds.open(path, mode, encoding=encoding, newline=newline)
File "C:\Tellurium-Winpython-3.7\python-3.7.7.amd64\lib\site-packages\numpy\lib_datasource.py", line 535, in open
raise IOError("%s not found." % path)
OSError: testdata.txt not found.

fitModel.py is deprecated. You should use model_fitter.py

Ah yes, see that. code completion was picking up the old fitter.
H
On Sat, Jul 25, 2020 at 12:05 PM Joseph Hellerstein <
<EMAIL_ADDRESS>wrote:

fitModel.py is deprecated. You should use model_fitter.py
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/sys-bio/teUtils/issues/1#issuecomment-663889730, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AAIBSDTJDD7ZE2FFIO7HAPTR5MUEBANCNFSM4PHRT3NQ
.

--
Herbert Sauro, Professor
University of Washington, Bioengineering
206-685-2119, www.sys-bio.org
<EMAIL_ADDRESS>Books: http://books.analogmachine.org/


Custom file query

Thanks for the great work on this plugin!
I had a question about a more flexible way of specifying what files to query with grep. Let's say I have a custom command relevant-files that prints out a list of files/directories (it can change depending on what the CWD is). I'd like fzf-lua to use this command in the call to rg (something like rg --vimgrep {q} -- $(relevant-files)). Is there a way to support this?
I already tried using the cwd/filename options, setting them to -- $(relevant-files) but there is some shell escaping that prevents the command from working. After disabling the escaping, things seem to work but I also had to modify the get_grep_command function to add -- before search_path. Any ideas on how to make this possible?

I noticed there is inconsistency between how cmd behaves in files vs grep what if I change get_grep_ cmd to work as it does in get_files_cmd, I.e. you can pass any command you want by using cmd = … and you’ll be responsible for the shell escaping, will that work for you?

Without changing any existing functionality I added raw_cmd option to both files and grep - can you try sending your entire command to grep({ raw_cmd = ... }) and let me know if that works for you?

I think something like that could work but I do need to inject the search query at some point. The raw_cmd won't have the search query from live_grep in it.
Ideally I want a final query that looks like this:
rg {q} -- $(relevant-files)

Does that make sense?

What happens if you simply send live_grep({ raw_cmd = “rg {q} -- $(relevant-files)” })?
Looking at the code this will send this command (with || true added) to the reload event, the only issue is it might not work due to the initial command being invalid, I didn’t test it yet.
Another option is I can add another raw_filespec variable that would replace filename/cwd without the shell escape and add — to the command.
This is easily solve-able in many ways I’m just trying to find something that would make sense in the future that wouldn’t make me go “what is this and why did I add it” lol

The initial command does not seem to be working :(. There's no visible error but the search is not working. The {q} causes a regex error when I try running the command manually.

The initial command does not seem to be working :(. There's no visible error but the search is not working. The {q} causes a regex error when I try running the command manually.

Ok, I added filespec as an option that would be added with no shell escape, I didn't add the -- so users can have the option to omit that if they wish, to achieve the command you're after use
:lua require('fzf-lua').live_grep({ filespec = '-- $(relevant-files)' })

Let me know if this works for you?

Yes it works! Thank you for the quick turnaround!

Nice :-)


RuntimeError: attn_softmax_inplace_forward_ not implemented on CPU

What's this error?
Is there any prepared jod when I try to run?
'RuntimeError: attn_softmax_inplace_forward_ not implemented on CPU'

Hi KYQiu21,
I haven't solved it yet, and I've stopped working for a little bit.

Check issue #293

If you are using CUDA through a conda install like (conda install nvidia/label/cuda-11.3.1::cuda), you might need to have these env variables set , so that it correctly points to the environment's CUDA directories.
export CUDA_HOME=$CONDA_PREFIX
export CUDA_PATH=$CONDA_PREFIX
export LIBRARY_PATH=$CONDA_PREFIX/lib:$LIBRARY_PATH
export LD_LIBRARY_PATH=$CONDA_PREFIX/lib:$LD_LIBRARY_PATH


How do insert image to table? Please help #456

How do insert image to table? Please help me#456

I wrote a tool that inserts a ContentEdit.Image at the current cursor position, the problem is that it gets deleted whenever I insert it into a table.  Any ideas?


build_deepSpeech2 function bug?

the first three conv layers asked the input to be like [batch, freq_bin, time_len, in_channels]
''' Parameters:

          maxTimeSteps: maximum time steps of input spectrogram power
          inputX: spectrogram power of audios, [batch, freq_bin, time_len, in_channels]
          seqLengths: lengths of samples in a mini-batch
   '''
# 3 2-D convolution layers
    layer1_filter = tf.get_variable('layer1_filter', shape=(41, 11, 1, 32), dtype=tf.float32)
    layer1_stride = [1, 2, 2, 1]
    layer2_filter = tf.get_variable('layer2_filter', shape=(21, 11, 32, 32), dtype=tf.float32)
    layer2_stride = [1, 2, 1, 1]
    layer3_filter = tf.get_variable('layer3_filter', shape=(21, 11, 32, 96), dtype=tf.float32)
    layer3_stride = [1, 2, 1, 1]
    layer1 = tf.nn.conv2d(inputX, layer1_filter, layer1_stride, padding='SAME')
    layer1 = tf.layers.batch_normalization(layer1, training=args.is_training)
    layer1 = tf.contrib.layers.dropout(layer1, keep_prob=args.keep_prob[0], is_training=args.is_training)

    layer2 = tf.nn.conv2d(layer1, layer2_filter, layer2_stride, padding='SAME')
    layer2 = tf.layers.batch_normalization(layer2, training=args.isTraining)
    layer2 = tf.contrib.layers.dropout(layer2, keep_prob=args.keep_prob[1], is_training=args.is_training)

    layer3 = tf.nn.conv2d(layer2, layer3_filter, layer3_stride, padding='SAME')
    layer3 = tf.layers.batch_normalization(layer3, training=args.isTraining)
    layer3 = tf.contrib.layers.dropout(layer3, keep_prob=args.keep_prob[2], is_training=args.is_training)

However, the rnn layers asked the batch to be like [max_time, batch_size ,...]
    # 4 recurrent layers
    # inputs must be [max_time, batch_size ,...]
    layer4_cell = cell_fn(args.num_hidden, activation=args.activation)
    layer4 = tf.nn.dynamic_rnn(layer4_cell, layer3, sequence_length=seqLengths, time_major=True)
    layer4 = tf.layers.batch_normalization(layer4, training=args.isTraining)
    layer4 = tf.contrib.layers.dropout(layer4, keep_prob=args.keep_prob[3], is_training=args.is_training)

And I don't see any transpose being made to make time to be the first axis, so after the conv layers the tensor is organized like [batch, freq_bin, time_len, 96] ,  where time is the third axis ,so is this a bug?

ps: the following code is located in the models/DeepSpeech2.py
And there is more
1.In the deepspeech2 paper, they use min(relu(x),20) as activation function in the convolution layers, however in this project, I don't see any activation in the convolution layers, which is really strange,
2. The num_layer seems completely useless in the DeepSpeech2 model
3. The deepspeech2 paper didn't mention using any dropout after batch_norm, and normally we don't use them together.


[Feature Request] the only reason I would avoid using this is healthcheck awareness.

I would desperately want this technology if it would be aware of the upgrade status of the containers and auto-rollback and notify on failure.
Basically as a user: I would love to have a tool that would attempt to preserve uptime as well as do automatic upgrades.  If the container themselves don't report healthy within a specified amount of time then do a rollback to the previous container image.  If it does report healthy then remove/clean up old images.
I realize this is a fairly complex suggestion but even with docker being docker and the goal of containers being stated as to be as ephemeral as possible (stateless) you don't always achieve the degree of stability you want.

Don't get me wrong I'm currently writing this into several of my docker containers that have no deps or real repercussions for automatic updates.

In your scenario, honestly I don't think ouroboros is the solution, but rather kubernetes.

@circa10a Why go that complex?

this is native to swarm.

@DirtyCajunRice oh interesting, have you got a link to this?  I think it would be an excellent feature to have on smaller single hosts with a bunch of potentially brittle microservices.

just read up on updating an image to a swarm service.

Do you think that they're basically aiming to have docker swarm stacks replace the functionality of docker-compose?

no. they have totally different purposes, but you are trying to do a docker swarm feature in regular docker. So thats my suggestion.

Thanks for the details I will definitely have a look at that feature but unfortunately at the moment upgrading to a swarm from a single node host is a bit out of my scope.  I am aware that you can technically have a single host swarm but all of my docker-compose microservices stacks are focused on docker-compose for portability and publishability purposes.

Also for future people looking at this thread https://docs.docker.com/engine/swarm/swarm-tutorial/rolling-update/ is what I imagine that @DirtyCajunRice is referring to.


CLEANUP: Remove xml module usage in bucketGetAcl api

Fix #269

Use aclUtils.convertToXml function to generate the XML DOM
Remove aclUtils.constructGetACLsJson function because XML
is generated with the grantInfo object and does not need to
be formatted for the XML module
Remove utils.convertToXml because its purpose was to 1) handle
JSON construction with aclUtils.constructGetACLsJson 2) return
the XML DOM using the XML module

@ironman-machine

Hello @bennettbuchanan
Starting end to end procedure using the following payload:
{"REPO_NAME": "S3", "SCALITY_S3_BRANCH": "dev/CLEANUP/remove_xml_module_bucketGetAcl", "DEFAULT_BRANCH": "master"}
Please follow http://ci.ironmann.io/gh/scality/Integration/5843 for CI status.


Swagger UI does not set content-type headers

For security reasons we use "Content-Type-Options: nosniff" header. Swagger UI Middleware does not respond with content type causing the index.html to be display in source - no rendering.
Expected behavior: Files are delivered with correct content-type headers (text/html, application/javascript, ...)

Sorry, didnt use most recent NSwag version.


DOC: adding one-liner summary to landing page

This should close #274

I'll be using this PR for fixing https://github.com/fornax-navo/fornax-demo-notebooks/issues/273, too


Braces in list field cause parts of an item to disappear

Inspired by https://tex.stackexchange.com/q/585520/35864
Consider the following MWE
\documentclass[british]{article}
\usepackage[T1]{fontenc}
\usepackage{babel}
\usepackage{csquotes}

\usepackage[backend=biber, style=authoryear]{biblatex}

\begin{filecontents}{\jobname.bib}
@book{book1,
  title     = {Book 1},
  author    = {A.},
  date      = {2011},
  publisher = {A {and} B, C {and} D, E {and} F},
}
@book{book2,
  title     = {Book 2},
  author    = {A.},
  date      = {2012},
  publisher = {{A and B, C and D, E and F}},
}
@book{book3,
  title     = {Book 3},
  author    = {A.},
  date      = {2013},
  publisher = {{A and B}, {C and D}, {E and F}},
}
@book{compare,
  title     = {Compare},
  author    = {Z},
  date      = {2014},
  publisher = {X and Y and Z},
}
@book{bookc1,
  title     = {Book C 1},
  author    = {Z},
  date      = {2011},
  publisher = {A {and} B and C {and} D and E {and} F},
}
@book{bookc2,
  title     = {Book C 2},
  author    = {Z},
  date      = {2012},
  publisher = {{A and B and C and D and E and F}},
}
@book{bookc3,
  title     = {Book C 3},
  author    = {Z},
  date      = {2013},
  publisher = {{A and B} and {C and D} and {E and F}},
}
\end{filecontents}
\addbibresource{\jobname.bib}

\begin{document}
\nocite{*}

\printbibliography
\end{document}

which produces

A. (2011). Book 1. A and B, C and D, E and F.
— (2012). Book 2. A and B, C and D, E and F.
— (2013). Book 3. A and B.

Somehow the last few parts of the list in book3 went missing.
Indeed the .bbl file shows
% book1
      \list{publisher}{1}{%
        {A {and} B, C {and} D, E {and} F}%
      }
% book2
      \list{publisher}{1}{%
        {A and B, C and D, E and F}%
      }
% book3
      \list{publisher}{1}{%
        {A and B}, {C and D}, {E and F}%
      }

It appears to me that the entry for book3 should be
      \list{publisher}{1}{%
        {{A and B}, {C and D}, {E and F}}%
      }

On closer inspection it looks as though there is some code to strip "outer braces" in list items, cf book2 which has {A and B, C and D, E and F}% and not {{A and B, C and D, E and F}}% as one might have expected. This fails with book3 because there is a { at the very beginning and a } at the end, but those two aren't matching.


Qemu Description can be empty & delete fmt.Print Debug login & GoFmt *

Hi,
3 small modifications :

The qemu description can be empty, it fix the bug that if the descrition is empty, the Config() function crash :

panic: interface conversion: interface {} is nil, not string

There was a fmt.Print that print the Login Post information for debug. I commented it.

I launched GoFmt on all files.

Thanks for this Lib !!

Thank you for improving the lib.


clean up anchors

Better comments
renamed NewAnchorID -> ToAnchorID since this is conversion but not a creation
renamed NewDocRoot -> ToDocumentRoot since this is conversion
renamed NewRandomDocRoot -> RandomDocumentRoot
Unexported anchorCommittedWatcher, anchorConfirmationTask

@mikiquantum this is only anchors. Will give another PR for geth client.

@vedhavyas from the https://github.com/centrifuge/go-centrifuge/issues/337 issue, I thought that was targeting identities as an initial pass. Maybe if that is not accurate want to update the issue to be on the same page?

Updated @mikiquantum

Thanks!


Testing threads causes a race condition

Observed in #493:
When testing threads, a race condition will be detected. The minimal test case is:
code := `
finish_message = nil

thread do
  finish_message = "thread"
end

sleep 1
finish_message
`

v := initTestVM()
v.testEval(t, code, getFilename())

it's not clear if this is an issue of the testing framework, or the interpreter.

Um...I don't think it'd be a test framework issue, should be something wrong in the interpreter.

It seems that the race condition is caused by assigning the string object "thread" to the finish_message pointer's value, while we're reading that pointer's value after evaluating a call frame for checking error objects.
I think the best solution is to change the way we do error checking, I'll try to fix this in this week.

@saveriomiroddi I think I might need your help on this. We sometimes might have race condition on pointer's target read/assignment. In current implementation this might happen when we assign values to the var finish_message at thread A, and checking if the top pointer (which is also var finish_message)'s value is an error at thread B. I'm not familiar with the pointer's thread safety, should we also need to implement something like share_ptr in c++?
BTW if we just put a fmt.Println("123") inside thread's execInstruction function then everything will work fine, I don't know why

Very interesting. I'll review this over the next few days!

Hello! This is a very interesting subject. As a general observation, I think this is the typical reasons why scripting interpreters use the GIL :smile:
Having said that, I had a cursory look around at the general patterns for handling shared access to pointers (which are not thread safe in Go).
I think that the high level behavior of the code would reflect the low-level one (note that I didn't read the source code yet) - in other words, at high level, there is a shared access to a variable, at low level, to a pointer.
If this is the case, there are two options:

use sync.Once, which includes the boilerplate for variables shared access; this works as expected, but is in general discouraged in Go
use thread channels; this is encouraged in Go.

Not knowing Go well, and maybe, considering it as easier solution, I'd use 1.

@saveriomiroddi Actually after thinking for a few days, I think this case is acceptable. Yes, it might have a race condition on pointer's target, but that's what users should expect when writing such code.
And the error is raised because the potential race condition is detected, not because there's something went wrong. So I suggest we should have a different place to put all the thread-related tests, and not running race condition check on them. What do you think?

If you're completely sure that writing this code won't case the executable to panic (which is undesirable, even for broken code) due to pointers undefined behavior, then that's a solution to the problem.

We can start writing the thread-related tests and I guess we'll find out then. But does this mean your mutex lock feature will be blocked?

We can start writing the thread-related tests and I guess we'll find out then. But does this mean your mutex lock feature will be blocked?

I will check it!

I've changed the title and description, since the race condition is not related to the test environment.


Vulnerability roundup 85: mpack-1.6: 1 advisory [7.5]

search, files

[ ] CVE-2011-4919 CVSSv3=7.5 (nixos-20.03, nixos-unstable)

Scanned versions: nixos-20.03: a84b797b28e; nixos-unstable: 22c98819ccd. May contain false positives.

Here is an archived version of the patch: https://web.archive.org/web/20120128080247/http://git.goodpoint.de/?p=mpack.git;a=commitdiff;h=0c87201f64491575350b18d04c62ec142e119d1f#patch1
According to the reporter, upstream is dead: https://openwall.com/lists/oss-security/2011/12/31/1


Question: How does this protect the "PoW Shield" from DDoS?

According to this slide, you can simply flood the PoW Shield to prevent all clients to be validated and proxied to the Protected Server. Is this correct?

Yes, in production you'll need to autoscale the PoW Shield container. However, request computational cost is extremely low on PoW Shield, which leads to you only having to pay for a fractional of the cost when compared with the requests going directly to the protected server. It also has internal rate limiting and firewall functionality to block malicious intent up to a certain degree. Hope that clarifies.


Add new Connect prometheus endpoint

With the release of Connect 2024.02.0, there is a documented Prometheus endpoint that the Connect chart should add in support for. Note currently this (https://docs.posit.co/connect/admin/operational-metrics/#included-metrics) is the list of metrics that does not cover the existing graphite metrics. For this reason it might make the most sense to expose the new metrics in ServiceMonitor alongside the Graphite exporter.
Listing out some changes that may need to be made:

Setting Metrics.PrometheusListen in the default config
Adding a named port (metrics is already in use so will need a different name) in the Connect service to the value of Metrics.PrometheusListen
Add a new entry to ServiceMonitor to pick up the new named port

Closed by https://github.com/rstudio/helm/pull/516


Compilation fails on Windows 10 ("system cannot find the file specified")

When compiling this library on Windows 10, this error occurs:
C:\erlang-oauth>make
process_begin: CreateProcess(NULL, rm -rf ebin/*.beam, ...) failed.
make (e=2): The system cannot find the file specified.
make: *** [clean] Error 2

make is running from: C:\Program Files (x86)\GnuWin32\bin

exactly the same problem ...did you come up with any solution after all ? @IRLeif

The Makefile wasn't meant for windows.
I've removed it in v2, you should be able to build on windows using erl.exe -make

The Makefile wasn't meant for windows.
I've removed it in v2, you should be able to build on windows using erl.exe -make


Include zip file support

Currently the engine supports tar files in the form of *.egg files.
I believe the engine could mimic the behaviour of love2d.
you just zip the file in a folder named  game and put a file named main.wren.
then rename the file mygame.zip to mygame.egg.
mygame.egg
  game/
    main.wren
    other_files....

zip files are easier to work with than tar files, because the main operating systems have native support for it without using the command line or downloading special programs.
here is a simple zip lib. https://github.com/kuba--/zip
:)

I appreciate the suggestion. I'll have to review this in the roadmap. I'd like to get DOME closer to how Love2D does it's single-file distribution (including what they call "Fused Mode").
The choice between TAR and ZIP has certain drawbacks and advantages, so it isn't completely settled, but integrating a single library to handle TAR was much easier at the time.
I don't expect most people to be using the Egg method for distribution, so if I change it, I won't be considering it a breaking change (or DOME could potentially handle both formats).


Docs currently suggest to use VSCode in the bastion node

Hey, and thanks for putting this documentation together! It maybe worth bookmarking the link on SWC's  #computing channel to make it easily discoverable.
In the "Remote development" section of the docs, it is more or less suggested to use VSCode in the bastion node:

Then, when you click on the “Open a Remote Window” button in the bottom left corner of the VS Code window, you will see a list of the SSH hosts you have configured in your ~/.ssh/config file. You can then select the host you want to connect to - e.g. swc-gateway.

But if many users start doing this, the bastion node could run into memory errors due to many memory-hungry VSCode apps being opened at the same time. Could the docs be updated to instead recommend and explain how to set-up VSCode into a compute node? A guide to do exactly this was actually put together by Cristofer Holobetz (to find it, search "cristofer holobetz pdf" on slack). Thanks!

Thanks @pierreglaser, we've also noticed the issue with our remote development guide, and we're discussing changes to that here.
We are considering to update or even remove that section, because it's indeed misleading.
Regarding Cristofer's guide, I have some reservations. It's indeed possible to ssh into a compute node and do remote develpment via VSCode in this way. However, this way of running jobs is not really controlled/limited via SLURM and could lead to consuming all resources in a node. At least that's what I've understood from my discussions with @adamltyson on this topic.
It would be great to come up a remote development solution that doesn't burden the bastion node and also respects SLURM.

Other potential solutions for remote development:

Some version of what you describe here with JupyterLab. I've tried these instructions, and they work. They should also respect SLURM resource allocation because you're are starting the JupyterLab session with srun.
Using a Jupyer notebook/lab app running via OpenOnDemand. @lauraporta in our team has had success with this and it could be the more user-friendly way to go, provided that we work out the kinks and document the workflow.

Thanks for the quick answer!

However, this way of running jobs is not really controlled/limited via SLURM and could lead to consuming all resources in a node. At least that's what I've understood from my discussions with @adamltyson on this topic.

I'm not sure about this: Cristofer's tutorial clearly states that the node in which VSCode will be started should be obtained through SLURM, using srun for instance. Did you have anything else in mind? I think that this part of the docs is very useful and I recommend keeping it, unless there are clear drawbacks (which right now I don't see once this bastion node issue is addressed).

Very possible that I'm wrong, but I don't understand how just by SSH-ing to a node, somehow that workload is therefore monitored by the SLURM job scheduler. It may be possible if you request the entire node, but then I'm not sure if SLURM will be able to kill the job etc.

I think if you start an interactive job via srun, and then ssh into it using the node name, you are indeed using the node you requested, but I don't know if the the constraints on memory, cores etc are respected in that case. For example if you do srun --mem 8G and then ssh into that node, what guarantees that you won't exceed the 8G?
Anyhow, I'll message Cristofer so he can also participate in the discussion. I think he may have asked the scientific computing team about this.

In the case that @niksirbi is right, I've found another possible solution: start a job that runs sshd and connect vscode to it. In this way the resources used by vscode will be effectively controlled by slurm. I didn't test this solution yet.
Also, open ondemand offers code server: vscode accessed via the browser running within a slurm job. I was interested in installing it some time ago.

but I don't understand how just by SSH-ing to a node, somehow that workload is therefore monitored by the SLURM job scheduler

The VSCode app has to be ran in a compute node obtained through slurm. However, to port forward VSCode back to your local machine, you have to start an ssh process from your machine to the compute node. This ssh process won't run anything, it just allows ports to be forwarded, which requires much more complex solutions to be done via slurm.

For example if you do srun --mem 8G and then ssh into that node, what guarantees that you won't exceed the 8G?

As this the slurm app on the SWC cluster is currently configured, yes :-)

You may very well be right @pierreglaser, but I don't sufficiently understand the internals of SLURM, cgroup, VScode's remote ssh plugin and their interactions to be confident about it. We may have to do some tests to confirm and consult Alex and John about it. Assuming we can confirm this, I'm happy to update the VSCode instructions according to @cristofer-holobetz 's guide.

Mmmh. Note that regardless of whether I'm correct, Cristofer's solution is an improvement over what is currently officially suggested (use VSCode on the login node). So not sure why we should delay moving forward with this solution.

I think it's important we only document things we know to be correct. It's unlikely that users will regularly consult this documentation to change their workflows.
@niksirbi for now, shall we just remove this section?

For now I suggest the following:

Remove the "remote development" section for now, to minimise the damage from additional people reading it and applying it as is
Apply the other small fixes suggested in #61 that could help with reducing the workload on the login nodes
Open a new issue for coming up with a proper long-form "Remote development with VSCode" guide, with input from Pierre and Cristofer

I can get this done this week if you agree.

Sounds good to me. Thanks!

I looked more into this issue: it turns out VSCode remote SSH mode does not use SLURM. Cristofer tutorial required you to ask for a compute node via slurm prior to connecting to the said node using VSCode, which led me to think otherwise. But this step is not required as VSCode just starts its own ssh connection.
As the link @lauraporta referenced link shows, this seems to be a well-documented issue on the VSCode side with only partial fixes existing. One option uses sshd within a slurm-allocated compute node (the one Laura mentioned), but the SLURM environment variables are not inherited by the new connections, and require additional hacks to be fully functional, so not ideal.
Another option is to use code-server (a program which serves VSCode in a webapp) in compute nodes, and use VSCode in your local machine's browser. Unlike other alternatives, the steps to get setup are very simple:

install code-server by downloading the binaries (alternatively we could ask IT to set it up globally on all nodes)

export VERSION=4.91.0
mkdir -p ~/.local/lib ~/.local/bin
curl -fL https://github.com/coder/code-server/releases/download/v$VERSION/code-server-$VERSION-linux-amd64.tar.gz \
  | tar -C ~/.local/lib -xz
mv ~/.local/lib/code-server-$VERSION-linux-amd64 ~/.local/lib/code-server-$VERSION
ln -s ~/.local/lib/code-server-$VERSION/bin/code-server ~/.local/bin/code-server
PATH="~/.local/bin:$PATH"

ask slurm for an interactive node via srun (srun --pty /bin/bash -l), find it's host name
start code-server on some port (code-server --bind-addr=localhost:8081)
port forward between the compute node and your machine's: ssh pierreg@<compute-node-hostname> -J hpc-gw1 -N -L 8081:localhost:8081
open localhost:8081 on google chrome (flawless UI for as much as I tried. I tried other browsers prior to this one and the UI was buggy).

The in-browser experience is pretty-much feature-complete since VSCode is ran under the hood (You can install extensions, start a terminal etc).
code-server seems widespread, and the solution is both robust and respects SLURM. WDYT?

Thanks a lot for investigating this @pierreglaser!
I gave it a shot and it indeed seems to work just fine (incl. from Firefox). This is definitely an improvement on the previous guide, so I'll write up some and have it tested by a few more people.
If all seems well we can ask IT to centrally install code-server, which will make the instructions even simpler.

If we're asking IT to install stuff centrally, is it worth just going straight for VSCode via OOD?

Well the two things are complementary. If people would like to use a VSCode app via OOD, the IT would have to anyway centrally install code-server and then link it to OOD.
Pierre's instructions provide a way to use code-server directly, without making it an OOD app. In a way OOD is an abstraction layer that will make this procedure more user-friendly, and can additionally serve as an entry-point to other apps like Jupyter Lab.
So installing code-server (+ the how to guide that comes with it) is a stepping stone towards full OOD functionality, not opposed to it.

Cool, I assumed that the existing VSCode OOD app worked some other way, so it would be duplicating effort for IT.

From what I can find browing online, using code-server seems to be the most popular choice for creating a VSCode app for OOD, see https://discourse.openondemand.org/t/vscode-showcase/2256


Remove background

The textured background makes the site feel a little dated to me. I'm not familiar with the history of the design, but was curious what everyone would think about dropping the background. Thoughts?

/cc @github/choosealicense-com

I want to say it might have been @cobyism's doing?

/cc @jasonlong

Yep, that was my doing and I definitely agree @bkeepers. :thumbsup: from me.

:cool:, thanks @jasonlong.  Anyone else have thoughts?

:shipit:


Call delete API in hover dropdown menu

Summary
When user clicks on delete in the hover menu, that media element is deleted from their media library. If successful, the element is removed visually from the library. If failed, the element stays and an error snackbar appears.
Displays a confirmation dialog informing the user that the deletion is permanent.

To-do
Turn on dropdown flag when Edit UI is merged.

Should this PR remove image / videos within existing pages of the current story. Feels really weird that it doesnt do this.

After deleting image, the more icon doesn't disappear as I would expect.
See this video.
https://youtu.be/R3W8xnspYyM
Steps to replicate.

Hover over image in media library.
Click on more icon.
Click delete.
Delete image.
More icon now appears on different image.

Should this PR remove image / videos within existing pages of the current story. Feels really weird that it doesnt do this.

IMO we shouldn't, that doesn't happen if we deleted the images / videos from the WP media library anyway. @samitron7 what are your thoughts?

After deleting image, the more icon doesn't disappear as I would expect.
See this video.
https://youtu.be/R3W8xnspYyM
Steps to replicate.

Hover over image in media library.
Click on more icon.
Click delete.
Delete image.
More icon now appears on different image.

Ah, that looks like an issue from #1459, I'll fix it in that PR.

Still can't test this in my browser. Am I missing something.

Still can't test this in my browser. Am I missing something.

You need to enable the feature flag mediaDropdownMenu.


Temp 2 Drive?

looking through trying to find out if this is able to be done through start-plotospawns and I'm not seeing the ability to do so.
I plot on my NVME then to free up space I have it compress to an SSD then from there I have it move to my HDD farm drive
this allows me to use my fastest drive to do more plots while my mid speed drive does the compressing and transferring to farm drive
-2 [tmp dir 2]: Define a secondary temporary directory for plot creation. This is where Plotting Phase 3 (Compression) and Phase 4 (Checkpoints) occur. Depending on your OS, -2 might default to either -t or -d. Therefore, if either -t or -d are running low on space, it's recommended to set -2 manually. The -2 dir requires an equal amount of working space as the final size of the plot.

Hei there @KillaDreams. You’re right. Currently Ploto has no params for setting -2 dir. However, I‘ll add it to the backlog using this issue.

thank you for the quick response looking forward to seeing this implemented

Available for testing as deployed with 558d0f3 on testing-t2 (Not pre-tested at all)

this is working fine but it stops after a certain amount of plots ran possibly to "no space on T2Drive" but the way the T2 drive works is it acts like a buffer drive to remove some load from TempDrive before moving to OutDrive.
my setup is as follows
Plot - 2TB NVME
T2 - 512 GB RAID 0 SSDs
Farm - HDDs
I am only able to run 5 plots at a time with 120 min delay due to my CPU restrictions.
The T2Drive should only be checked for "storage size" one time at the beginning of Start-PlotoSpawns since it is only used for compressing from TempDrive to free up space and speed from the main plotting drive.
plot stages 1-2 TempDrive -> Compress files Stage 3 to T2Drive -> move files Stage 4 to OutDrive
JobId           : 91437547-b038-4f2b-92ca-01b934072199
ProcessID       : 3040
OutDrive        : J:
TempDrive       : G:
T2Drive         : H:
ArgumentsList   : plots create -k 32 -b 3390 -r 2 -t G:\ -d J:\ -2 H:
ChiaVersionUsed : 1.1.5
LogPath         : .chia\mainnet\plotter\PlotoSpawnerLog_20_5_6_42_91437547-b038-4f2b-92ca-01b934072199_Tmp-G_Out-J.txt
StartTime       : 5/20/2021 6:42:09 AM

Alerts enabled. In Param and config is there and says send, WhenJobCreated
PlotoManager @ 5/20/2021 6:57:18 AM : Amount of spawned Plots in this iteration: 3
PlotoManager @ 5/20/2021 6:57:18 AM : Overall spawned Plots since start of script: 3

PlotoSpawner @ 5/20/2021 8:49:15 AM : Spawned the following plot Job:
JobId           : 82b0c668-74b6-49c6-9526-4ee387a4e15f
ProcessID       : 17748
OutDrive        : J:
TempDrive       : G:
T2Drive         : None
ArgumentsList   : plots create -k 32 -b 3390 -r 2 -t G:\ -d J:
ChiaVersionUsed : 1.1.5
LogPath         : .chia\mainnet\plotter\PlotoSpawnerLog_20_5_8_49_82b0c668-74b6-49c6-9526-4ee387a4e15f_Tmp-G_Out-J.txt
StartTime       : 5/20/2021 8:49:07 AM

Ok so I hope I can answer these questions properly also forgive me I'm on my phone with the github app and it's very hard to understand the layout if this

Some questions for my understanding

We rather focus on the overall, initial free space of the disk, but from my understanding right now, is also neglibile?

The max overall space in your example is 4 at a time but that would be at the same exact time as the compressing build new files on the T2Drives it doesn't use the full 107 GB size instantly.

At which point would we define not to use that disk, ie when completely full (Because we maybe do 16x jobs in parallel, all pointing to 1-2x t2drives)?

Would these 16 jobs be delayed or all at same time? Then as you calculated you would need 4 512GB size T2Drives to be able to do this
I don't think we would ever tell it not to use T2 unless we disable it in config file.

Check Jobs upon phase of job that uses this disk?

Possibly have it check jobs in stage 3 and stage 4 then calculate
MAX minus stage 3 plus stage 4 is greater than 0 then allow new plot to spawn

And for the fact that one would want to use more than 1x t2 drive, we'd need a way to determine the most suitable, available t2drive.

Using the 16 parallel example as above it could
Rotate through the multiple T2Drives with each new starting plot  so it may look as follows with Drive - Plots
T2D 1 - 1,5,9,13
T2D 2 - 2,6,10,14
T2D 3 - 3,7,11,15
T2D 4 - 4,8,12,16

Again, maybe upon amount of jobs on this disk in given phase?

Do we need to check free space at all? Or does that chia.exe itself and determines what/when it can offload? And if we check free space initially, what for?

Yes. If chia runs out of temp space it will wait 5 mins and try again and as long as a plot is in stage 4 drive space will free up so back to max 4 example if the 5th plot is trying to compress to same T2D then as long another plot is already in stage 4 the space will free up for the 5th to compress once the stage 4 completes the move and deletes the compressed file

Based on that, I adjusted Get-PlotoT2Drives with the following:

To get active Jobs, added a filter that only gets jobs on that drive in p3 and p4. (Jobs not in P3/4 will not be added to calculation)
Rotation trough t2 drives is already there.

Availabele in latest commit 2eb07ab on branch https://github.com/tydeno/Ploto/tree/t2-enhancement. Not pre-tested may break. Will test tonigt along with branch dynamic-stagger

If I may add, having an option to "sync" t2 and outdrives may also be desirable to some setups (like mine).
If t2 and outdrive is the same, chia.exe skips the Phase 5 (or the Copy phase) all together. This can potentially save up 4-7minutes of job time (depending on your outdrive) so that a new job can be (considered to be) started immediately.
I leave the copying from t2 to actual destination to be done by PlotoMover instead.

@PiSCaRGoT I just set t2drive and outdrive the same and then have plotomoves loop from outdrive to final location

Small side-question... Would it be weird to have my smaller 1TB NVMe bootdisk act as temp-2 drive, while my 1.7 TB SSDs do the regular temp-stuff? I see everyone here does it the other way around :)

Small side-question... Would it be weird to have my smaller 1TB NVMe bootdisk act as temp-2 drive, while my 1.7 TB SSDs do the regular temp-stuff? I see everyone here does it the other way around :)

You want your fastest drive to be your tempdrives then to offload space to T2drives to free up space and transfer speed from main drive so you can continue to plot more faster while the T2Drive does the transfer to the outdrive


Explain more - "Camel case: defined" is unclear

https://google.github.io/styleguide/jsguide.html#naming-camel-case-defined
Correct
XmlHttpRequest
newCustomerId
innerStopwatch
supportsIpv6OnIos
YouTubeImporter
checkNonempty/checkNonEmpty

When we are talking about Camel case, why below are starting with Capital letters,
Please explain in detail.

XmlHttpRequest
YouTubeImporter

See item #3 in the list:  "lower camel case" and "upper camel case" are two different conventions each used for different purposes.


Fails to resolve miniconda paths

See https://github.com/microsoft/vscode-python/issues/23922
In the issue I have asked for more details.

Upstream issue was closed


1.12.2 login on 1.13.2 spams console

What is the output url of /viaversion dump?

https://dump.viaversion.com/42c8377f8762fe6f811ed4e9699683244d106f6eadecdcadcd6355214bc88c5c

How/when does this error happen? login?:

Happens on login with 1.12.2 on 1.13.2 server. Doesn't seem to happen when logging in with 1.13.2 on the 1.13.2 server.

Is there an error in the console? Use pastebin.com. Is there a kick message?:

Yes: https://pastebin.com/9g5Hk8re Doesn't seem to cause any harm, just spams console on login.

1.13.2 isn't complete mapping wise.


Make Symfony generator works

[ ] Update .env with correct TRUSTED_HOST
[ ] Update Nginx configuration to redirect / to index.php
[ ] Specify the correct path to run composer commands
[ ] Update installation documentation to have database working (.env update)

This issue has been split over time and some point are already fixed.


Reproducing benchmark results on the SIDER dataset

Thanks for your inspiring work. However, we've got problems when fine-tuning MolCLR on the SIDER dataset.
It's reported in the paper that MolCLR achieves 68.0 in terms of ROC_AUC under Scaffold split on SIDER, but we fail to reproduce the results. Here's what we've tried:

In our implementation, the average ROC_AUC of MolCLR over 27 tasks using default settings is approximately 62.3, which is far from the reported results. I observe that in the original paper of MoleculeNet, models are evaluated using Random split instead of Scaffold split on SIDER. It seems that the comparison between RF (68.4) and MolCLR (68.0) in the benchmark table is unfair.
We conduct experiments with Random split on SIDER. Here's the results.

We've also tried several hyper-parameter combinations by changing dropout, hidden size and activation functions of MLP, which yields similar results.
We hope the authors could kindly offer the experiment settings and hyper-parameters on SIDER that fully reproduces the promising results of MolCLR.

Thanks for your interest in our work. A suggested setting for SIDER:

lr: 0.0005
base ratio: 0.4
dropout: 0.3
batch size: 32
pooling: max

Also, tuning the hyperparameters for each task individually can improve performance. Hope this helps.
Best,
Yuyang

Thanks for your response! We've successfully reproduced the reported results on SIDER.


DBZ-7588 Disable debug prints on tests

closes: https://issues.redhat.com/browse/DBZ-7588

Applied to main (=2.6), thanks @mfvitale


MAPREDUCE-7288. Fix TestLongLong#testRightShift

In the testRightShift method is not implements the right verify operations.
https://issues.apache.org/jira/browse/MAPREDUCE-7288

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
1m  6s
Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s
No case conflicting files found.

+1 :green_heart:
@author
0m  0s
The patch does not contain any @author tags.

+1 :green_heart:
test4tests
0m  0s
The patch appears to include 1 new or modified test files.

_ trunk Compile Tests _

+1 :green_heart:
mvninstall
19m 15s
trunk passed

+1 :green_heart:
compile
0m 28s
trunk passed with JDK Ubuntu-11.0.8+10-post-Ubuntu-0ubuntu118.04.1

+1 :green_heart:
compile
0m 27s
trunk passed with JDK Private Build-1.8.0_252-8u252-b09-1~18.04-b09

+1 :green_heart:
checkstyle
0m 22s
trunk passed

+1 :green_heart:
mvnsite
0m 29s
trunk passed

+1 :green_heart:
shadedclient
14m 36s
branch has no errors when building and testing our client artifacts.

+1 :green_heart:
javadoc
0m 28s
trunk passed with JDK Ubuntu-11.0.8+10-post-Ubuntu-0ubuntu118.04.1

+1 :green_heart:
javadoc
0m 26s
trunk passed with JDK Private Build-1.8.0_252-8u252-b09-1~18.04-b09

+0 :ok:
spotbugs
0m 45s
Used deprecated FindBugs config; considering switching to SpotBugs.

+1 :green_heart:
findbugs
0m 42s
trunk passed

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
0m 25s
the patch passed

+1 :green_heart:
compile
0m 20s
the patch passed with JDK Ubuntu-11.0.8+10-post-Ubuntu-0ubuntu118.04.1

+1 :green_heart:
javac
0m 20s
the patch passed

+1 :green_heart:
compile
0m 20s
the patch passed with JDK Private Build-1.8.0_252-8u252-b09-1~18.04-b09

+1 :green_heart:
javac
0m 20s
the patch passed

+1 :green_heart:
checkstyle
0m 15s
the patch passed

+1 :green_heart:
mvnsite
0m 21s
the patch passed

+1 :green_heart:
whitespace
0m  0s
The patch has no whitespace issues.

+1 :green_heart:
shadedclient
13m 48s
patch has no errors when building and testing our client artifacts.

+1 :green_heart:
javadoc
0m 23s
the patch passed with JDK Ubuntu-11.0.8+10-post-Ubuntu-0ubuntu118.04.1

+1 :green_heart:
javadoc
0m 22s
the patch passed with JDK Private Build-1.8.0_252-8u252-b09-1~18.04-b09

+1 :green_heart:
findbugs
0m 47s
the patch passed

_ Other Tests _

+1 :green_heart:
unit
0m 41s
hadoop-mapreduce-examples in the patch passed.

+1 :green_heart:
asflicense
0m 31s
The patch does not generate ASF License warnings.

58m 50s

Subsystem
Report/Notes

Docker
ClientAPI=1.40 ServerAPI=1.40 base: https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-2183/1/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hadoop/pull/2183

Optional Tests
dupname asflicense compile javac javadoc mvninstall mvnsite unit shadedclient findbugs checkstyle

uname
Linux 9b4ca3cdc6ae 4.15.0-60-generic #67-Ubuntu SMP Thu Aug 22 16:55:30 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
personality/hadoop.sh

git revision
trunk / c40cbc57fa2

Default Java
Private Build-1.8.0_252-8u252-b09-1~18.04-b09

Multi-JDK versions
/usr/lib/jvm/java-11-openjdk-amd64:Ubuntu-11.0.8+10-post-Ubuntu-0ubuntu118.04.1 /usr/lib/jvm/java-8-openjdk-amd64:Private Build-1.8.0_252-8u252-b09-1~18.04-b09

Test Results
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-2183/1/testReport/

Max. process+thread count
420 (vs. ulimit of 5500)

modules
C: hadoop-mapreduce-project/hadoop-mapreduce-examples U: hadoop-mapreduce-project/hadoop-mapreduce-examples

Console output
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-2183/1/console

versions
git=2.17.1 maven=3.6.0 findbugs=4.0.6

Powered by
Apache Yetus 0.13.0-SNAPSHOT https://yetus.apache.org

This message was automatically generated.

Hi @aajisaka , can you help to review?