Would it possible to add decoys to GRCh37 reference?

Hello!
Thanks for the great pipeline!
I found a closed issue about decoys in the reference here https://github.com/bcbio/bcbio-nextgen/issues/1234
It is 2018, but still many projects are using grch37 reference and it is problematic to switch them to hg38.
The current reference sequence in bcbio is from 1kg project and does not have decoys and EBV:
http://www.internationalgenome.org/category/grch37/
Would it be possible to add decoy sequences to GRCh37 reference?
File sizes would not increase much: hg38 with decoy is 3112M, GRCh37 without decoy is 3007M.
Adding decoys would remove some FP rare variants, and increase trust in bcbio variant calls for people doing variant interpretation.
I could put some efforts to make it happen, if needed.
Thanks!
Sergey

Sergey;
Thanks for starting this thread. Is using build 38 a solution for your needs? We can't modify the existing hg19 and GRCh37 installs at this point as that would end up breaking runs using the previous approaches since the sequencing dictionaries would be incompatible. We've held off adding decoys in build 37 to avoid this churn and also because I've seen validations where they seem to help and also seem to hurt so the improvements are not unambiguously better. The alt approach in hg38 universally improves resolution of FPs in all our validations, so we've focused instead of supporting build 38 and making it equivalent to build 37 runs.
If you absolutely need to use build 37 with decoys your best approach would be to add as a custom genome to your install:
https://bcbio-nextgen.readthedocs.io/en/latest/contents/configuration.html#adding-custom-genomes
Thanks again for this discussion.

Hi Brad, thanks for the explanation.
Unfortunately, moving to hg38 is not an option for now for the particular project I have. My indirect and rough (max) estimate is that decoy allows to reduce FDR in SNPs for WES by 0.4%, from 0.6% to 0.2%.
The benefit is not that dramatic to put a lot of effort into decoy. Do you think that the following 2step approach will work:

create custom reference grch37decoy with bwa index and nothing else.
do alignment with custom genome
do variant calling with regular grch37 reference using noalt_calling starting with bam produced with step1. Or bcbio will complain about incompatible references in that case?

Sergey

Sergey;
That approach should work, although I'd also suggest using bam_clean: remove_extracontigs when restarting with the decoy aligned BAM:
https://bcbio-nextgen.readthedocs.io/en/latest/contents/configuration.html#alignment
This will extract only the standard contigs and avoid any potential issues with GATK or Picard derived tools being unhappy with the reference contig differences. Thanks for coming up with a creative solution and hope this works for you.

Sergey;
Apologies about the issues, we certainly don't want to lose reads here. Do you have a sense of what reads are not present from the initial input file? Do you have the same issue if you turn off realignment (realign: false). We're not recommending this step and I wonder if it's doing something different with the decoy sequence. If that doesn't change the outputs, you could also try without recalibration to help determine if we can isolate the step that is dropping reads. Hope this helps with debugging and please let us know what you find.

Hi Brad!
I'm still testing where reads disappear when aligning go grch37d5.
Also found another problem: when running variant calling and annotation against grch37 using bam aligned to grch37d5 as input, I'm getting errors related to the reference name (grch37d5), neither bam_clean: remove_extracontigs, nor bam_clean: picard helps to solve it.
When using bam_clean: remove_extracontigs, error is when running manta:
[2018-10-19T15:14:06.169761Z] [r1a-25.cm.cluster] [204417_1] [WorkflowRunner] [ERROR] [2018-10-19T15:13:58.793153Z] [r1a-25.cm.cluster] [204417_1] [makeLocusGraph_chromId_007_8_0006] std::exception::what: Split alignment segment maps to an unknown chromosome: 'hs37d5'
[2018-10-19T15:14:06.169859Z] [r1a-25.cm.cluster] [204417_1] [WorkflowRunner] [ERROR] [2018-10-19T15:13:58.901687Z] [r1a-25.cm.cluster] [204417_1] [makeLocusGraph_chromId_017_18_0001] 	local_bam_record: E00386:198:HLNVCCCXY:5:2214:27559:64175/1 tid:pos:strand 17:11463289:+ cigar: 5S47M99S templSize: 0 sa: hs37d5,30281334,+,99S43M9S,9,1; mate_tid:pos:strand -1:35254438:-

When using bam_clean: picard, error with gatk PrintReads:
October 22, 2018 9:30:29 PM EDT] org.broadinstitute.hellbender.tools.PrintReads done. Elapsed time: 52.40 minutes.
Runtime.totalMemory()=506986496
htsjdk.samtools.SAMException: Exception when processing alignment for BAM index E00386:198:HLNVCCCXY:5:1101:10003:11013 1/2 151b aligned read.
	at htsjdk.samtools.BAMFileWriter.writeAlignment(BAMFileWriter.java:140)
	at htsjdk.samtools.SAMFileWriterImpl.addAlignment(SAMFileWriterImpl.java:199)
	at htsjdk.samtools.AsyncSAMFileWriter.synchronouslyWrite(AsyncSAMFileWriter.java:36)
	at htsjdk.samtools.AsyncSAMFileWriter.synchronouslyWrite(AsyncSAMFileWriter.java:16)
	at htsjdk.samtools.util.AbstractAsyncWriter$WriterRunnable.run(AbstractAsyncWriter.java:123)
	at java.lang.Thread.run(Thread.java:748)
Caused by: htsjdk.samtools.SAMException: Exception creating BAM index for record E00386:198:HLNVCCCXY:5:1101:10003:11013 1/2 151b aligned read.
	at htsjdk.samtools.BAMIndexer.processAlignment(BAMIndexer.java:119)
	at htsjdk.samtools.BAMFileWriter.writeAlignment(BAMFileWriter.java:137)
	... 5 more
Caused by: htsjdk.samtools.SAMException: Unexpected reference -1 when constructing index for 83 for record E00386:198:HLNVCCCXY:5:1101:10003:11013 1/2 151b aligned read.
	at htsjdk.samtools.BAMIndexer$BAMIndexBuilder.processAlignment(BAMIndexer.java:218)
	at htsjdk.samtools.BAMIndexer.processAlignment(BAMIndexer.java:117)
	... 6 more
' returned non-zero exit status 3

Do you have any suggestions how to debug 2-step variant calling in grch37 with decoy?
Thanks!
Sergey

Sergey;
Sorry about the issues here with cleaning your decoy BAM file. It looks like the issue is that reads get pulled along from the decoy region hs37d5 despite our attempts to remove those with samtools view. These look like secondary and split reads that map to both a standard contig and the decoy. Apologies, it's quite difficult to get these removals correct and it appears as if manta is especially sensitive to detecting this and failing out. Apparently the variant callers that you used were okay since it got to this point, but it's getting all the downstream tool support correct that's the issue.
Practically, the best approach at this point might be to have your decoy as a separate custom genome build rather than trying to remove the extra contigs as part of bcbio. Is that an option?
If you want to remove these additional contigs before, you could try using VariantBam (https://github.com/walaj/VariantBam) as it has mate linking:
variant in.bam -L decoy_contigs.bed -o out.bam -b

It would be useful to know if excluding the problem reads like that and then feeding into bcbio resolves the issue.
Sorry again about all the issue and hope this helps.


Dropdown showing cursor in iphone safari

Please check (select) dropdown it showing cursor of input in iphone safari

It's fixed in my PR : https://github.com/Dogfalo/materialize/pull/2514

Is there a fix for this?

The same phenomenon occurred in the iPhone 6S Chrome 51.0.2704.64.

This also happens in Internet Explorer 11 and earlier. The problem is that materializecss renders an input field to display the selected option's text. The dropdown gets shown when that input field is focused. As far as I know IE renders that focus cursor always above everything else. I guess the best fix would be to not use the input field to display the selected option's text.

Maybe we could focusout displayed input on focusin without hiding dropdown ?

@storm1er That could also be a solution. But that would break the navigation by keys, that part would have to be rewritten too.

Just checked this and it doesn't seem to be happening on iOS safari or chrome

@Dogfalo it does, I just verified the bug on iOS 10.3.2 and iPhone 6S. In addition it is also a problem in internet explorer. Please reopen that issue or refer to a duplicate issue if there is one.

I would suggest to position the input field off screen with position: absolute; left: -9999px; and then render the selected option to a normal div that is styled like an input field. That keeps the select box focusable using navigation by keys and also removes the blinking cursor. In addition all the focus and blur events can stay the same.

https://github.com/Dogfalo/materialize/issues/3790


Superfluous tab stop in expander

@ButchersBoy should be able to close this now that it is merged.


Using NaiveEngine with Unix fork() hangs during prediction.

Hello everyone,
I need to have single thread process running and performing prediction. After using defaut engine type and setting CPU thread environment variables to 1, I always end up with 2 threads. The only way I could achieve achieve single thread execution is using NaiveEngine.
In this particular context I must use fork() for paralelization but with NaiveEngine it simply just hangs during prediction.
Have anyone experience this behaviour or knows how to make it work?
Cheers,
Miguel

@anirudh2290 , would you be able to help?

@mlourencoeb please provide a reproducible script.

I ended up converting the modelo to onnx ans performing inference with opencv where we can have granular update of the number of threads.
Thanks anyway :)


Extended lending schema to include account/position data

To compare diff:
https://raw.githubusercontent.com/messari/subgraphs/master/schema-lending.graphql
https://raw.githubusercontent.com/messari/subgraphs/b7f5412c347eef3928d0d6f9a4b7fcf95c7489b3/schema-lending-extended.graphql
paste into:
https://www.diffchecker.com/
Use Cases

View the current positions of an account (e.g. how much money do I have in Aave?)
View all historical positions of an account (e.g. what does my lending/borrowing behavior look like?)
View all the open positions in a market

Open Questions

Should we calculate USD prices in positions? These can be computed offchain more easily

Aside from missing the latest commits to the regular lending schema it LGTM

Thanks for looking it over!

Seems like there is a demand for even more nuanced data. I think the easier thing to do is to include everything we want, see how the indexing time is, and then do ablation studies to find the middle ground if indexing time is too long

Seems like there is a demand for even more nuanced data. I think the easier thing to do is to include everything we want, see how the indexing time is, and then do ablation studies to find the middle ground if indexing time is too long

It may not be in scope but the next big thing would be flashloans.
I will stare at the extended schema more but likely won't have a ton more of in depth feedback until it's deployed somewhere and I can start to mess with it

No problem. This will be an iterative process

I am thinking about a dedicated use case - "what addresses are underwater"?
To calculate this, we need to know the positions of all its collaterals and its borrowing assets. I am wondering which part in the schema keeps track of its collateral?

All positions with PositionSide being lender will give you the collaterals

@this-username-is-taken Let's say in compound, I:

deposit 10 ETH and get 500 cETH
use 400 cETH as collateral to borrow 1000 USDC

So I have positions of:

10 ETH (lender)
400 cETH (lender) -- collateral
1000 USDC (borrower)

is that correct? 🤔

@this-username-is-taken Let's say in compound, I:

deposit 10 ETH and get 500 cETH
use 400 cETH as collateral to borrow 1000 USDC

So I have positions of:

10 ETH (lender)
400 cETH (lender) -- collateral
1000 USDC (borrower)

is that correct? 🤔

I've updated the docs. Let me know if it answers your question. With regard to your example, why do we care about the cETH?


Not reloading?

I'm running this in a docker container on a project i'm working on.
root@165b57827097:/go/src/github.com/crsmithdev/app-template# rerun github.com/crsmithdev/app-template
2015/05/09 18:07:37 setting up github.com/crsmithdev/app-template []
2015/05/09 18:07:38 [app-template]
starting

However, touching or editing main.go doesn't produce a reload, it only changes when i ctrl-C and rerun again.

I'm experiencing the same issue here.

@crsmithdev did you ever find out the source or a fix?

I am observing the same issue.

I had this problem and left for gulp for a while. I've just been refactoring an app and decided to try it again - this time using Docker for Mac and it works fine.

I'm having this problem now, and it reloads properly the first couple of saves, then it stops responding and has to be restarted. Not really usable in its current state.


Information about the problem with caching the seo_fields key.

I suggest using this description.
#81

Thanks for the note! I think it will be useful to new users of Laravel. :)


[4.0] Editor lightmap softlock upon re-importing an OBJ

Godot version
v4.0.alpha1.official [31a7ddbf8]
System information
Windows 10, Vulkan, 1660Ti
Issue description
In my attempts to attach a material to an OBJ exported from Qodot using the last tab of Import dialog I have managed to get the editor to reliably softlock upon opening the scene,
ERROR: Condition "!lm" is true. Returning: false
   at: lightmap_is_interior (servers/rendering/renderer_rd/renderer_storage_rd.cpp:7493)
ERROR: Condition "!lm" is true. Returning: AABB()
   at: lightmap_get_aabb (servers/rendering/renderer_rd/renderer_storage_rd.cpp:7499)
ERROR: Condition "!lm" is true. Returning: false
   at: lightmap_is_interior (servers/rendering/renderer_rd/renderer_storage_rd.cpp:7493)
ERROR: Condition "!lm" is true. Returning: AABB()
   at: lightmap_get_aabb (servers/rendering/renderer_rd/renderer_storage_rd.cpp:7499)

Steps to reproduce
Open the attached project, open TestScene
Minimal reproduction project
QodotTest4.zip

@YellowAfterlife Can you reproduce this in 3.4.2?

@Calinou I'm not sure if this can be done in 3.4.2 since the advanced import menu (with ability to set materials per-mesh, which is how I achieved this crash) was introduced after that

I'm encountering a similar error myself, using the tbloader plugin and a lightmap node. For me, the error occurs when I bake the lightmap, save and close the scene, and re-open it (the log begins flooding with Condition "!lm" is true errors), and if I close the project entirely and re-open it the editor freezes while opening, still flooding the log.


⚠ GitLab Server has degraded performance

In dbb4bdd, GitLab Server (https://gitlab01.its-telekom.eu) experienced degraded performance:

HTTP code: 200
Response time: 10343 ms

Resolved: GitLab Server performance has improved in a716c23.


Share player cam to multiple views

At the moment in NTC, the players cams are only shared with one "primary view".
That is done because layouts dictate some characteristics of the videos, like the resolution requested.
The "primary view" is basically the last loaded view owned by the game host. When a game host loads a view, it will set itself as the primary view, and instruct all the producers to push their video streams to it, while stopping their previous sharing.
This has caused some issues in the past:

People using multiple views and refreshing can cause (accidental) shifts in where the camera streams go
Folks who hope to do real time streaming in multiple languages in multiple channels are not able to load the full view with player cam (happened at CTWC23 where Pollo restreamed which introduced lag)

Potential approach:

All views owned by the game host get videos
Whenever a new view loads, it gets added to the lost, and its resolution request is sent to the producers. 2 approaches for sending the video streams:

producers generate an independant video stream for each producer based on their requested size
producers identify the largest resolution request, generate a video stream for that, and share the same stream with all producers

If a view disconnects, the producers need to be informed, so the video stream call to that view is dropped

Note that views also have additional characteristics like the number of players they support, and whether the view support 2 concurrent matches ornot. The primary view is still an important concept because it dictates what the admin page shows based on what the primary view supports.
This has also been an issue when folks are loading both a single match view and a 2-concurrent-match view, refreshing one or the other (currently) changes what the admin page shows (i.e. all the match controls...)

In addition to the primary view concept, NTC has not supported player cams to multi-views to save on network bandwidth. This was to guard against cases where game host may have multiple OBS scenes where the browser source stays active, but the scene is not displayed. In such setup, there could be a huge amount of video stream data travelling on the network to feed all these views.
In my own setup, I typically only have one NTC scene and I switch the browser source URL to the view I want when I need. But there is no way to prevent people from loading MANY views, so there's a risk people do not notice.


[SPIKE] Explore usage of custom properties

PR collecting different pieces of work to explore CSS custom properties as part of #4900.

Closing this as the spike has now ended and we've learnt what we set out to learn.


Search for custom accessor for select and select_multiple column

The brand new Ajax Table is nice...
There is a handy searchLogic for custom accessor  too
However, I am using connected model with custom accessor, the $query string only stick with the current model but not the connected entity
is it really impossible to search for custom accessor for select and select_multiple column ??

When you say "[you are] using connected model with custom accessor" what do you mean exactly?

Let me elaborate more on my scenario...
I am searching on "Price" eloquent model, which has relationship with "CruiseShip" & "Fee" model.
It is fine that if the "Fee" model using the attribute appear on the actual MySQL Table's column...
$this->crud->addColumn([
    'label' => "Fee Name",
    'type' => "select",
    'name' => 'fee_name_id',
    'entity' => 'fee_name',
    'attribute' => "name", // This is on the actual MySQL Table
    'model' => "App\Models\Fee",
]);  // This is perfectly fine

However, the column "CruiseShip"(the connected model) using the custom attribute (the customized Accessor which combined two columns) which is not the actual table's column...
$this->crud->addColumn([
    'label' => "Cruise Ship",
    'type' => "select",
    'name' => 'cruise_ship_id',
    'entity' => 'cruise_ship',
    'attribute' => "cruise_ship_name_date", // combined name & date column
    'model' => "App\Models\CruiseShip",
    'searchLogic' => function ($query, $column, $searchTerm) {
        $query->orWhere('name', 'like', '%'.$searchTerm.'%');  // Trying to query on actual "name" column
    }
]);

This will get the SQL query error
"SQLSTATE[42S22]: Column not found: 1054 Unknown column 'name' in 'where clause' (SQL: select count(*) as aggregate from `prices` where (`name` like %m%))"

I understand that the $query is executing the SQL query for 'prices' table, I am wondering how can I query other connected table...

You sound like you want a whereHas or an orWhereHas (if such a thing exists); I think. Whether that $query will allow that or not is a different question.
I think what your saying is that you want to limit the thing in "cruise ships" where the name on the WHAT MODEL (crusise ship model?) is like 'some search time'.

It is because I am searching on "Price" not "CruiseShip" model.
"Price" is belongsTo "CruiseShip" model
the $query is looking to "prices" table only but not other table if i am using searchLogic feature

Is $query a query builder?

I am not very sure on that. from the syntax i believe it is a query builder.
This is what i found from the source code.
https://github.com/Laravel-Backpack/CRUD/blob/0be2bae4a58ce417ddd5f8865b1b6e38ee87879d/src/CrudPanel.php
    // ------------------------------------------------------
    // BASICS - model, route, entity_name, entity_name_plural
    // ------------------------------------------------------

    /**
     * This function binds the CRUD to its corresponding Model (which extends Eloquent).
     * All Create-Read-Update-Delete operations are done using that Eloquent Collection.
     *
     * @param string $model_namespace Full model namespace. Ex: App\Models\Article]
     *
     * @throws \Exception in case the model does not exist
     */
    public function setModel($model_namespace)
    {
        if (! class_exists($model_namespace)) {
            throw new \Exception('This model does not exist.', 404);
        }

        $this->model = new $model_namespace();
        $this->query = $this->model->select('*');
        $this->entry = null;
    }

The searchLogic is defined and called in the Backpack\CRUD\PanelTraits\Search
CrudPanel.php uses this Traits and public $query is defined in CrudPanel.php ...
So, basically the $query tight with $this->model

@lloy0076 you are right! whereHas is the answer!
'searchLogic' => function ($query, $column, $searchTerm) {
    $query->orWhereHas('cruise_ship', function ($q) use ($column, $searchTerm) {
        $q->where('name', 'like', '%'.$searchTerm.'%')
          ->orWhereDate('depart_at', '=', date($searchTerm));
    });
}

I hope you could include this in the readme. Thanks a lot !

@kiddtang - thanks for the confirmation. Just updated the docs with your example. Cheers!


SQS Checksum Validation

Can be disabled with DisableComputeChecksums. This may be modeled now?
internal id: 176558055

See SQS documentation for more details on checksums.


Roadmap for Svelte 5

Change Type
Addition
Proposed Changes
This is actually moreso to start a discussion about Svelte 5 and shadcn-svelte. I know we're still quite a ways out from using runes in production code, and I imagine many libs won't even begin the work to support runes until Svelte 5 has been officially released.
That said, shadcn and by extension this lib are a little different in how users integrate it into their projects than conventional UI libs.
Do you mind sharing your plan for Svelte 5 support? If you've got a roadmap, I'm sure there's a few of us who'd love to help you support the transition.

Fortunately, at this time I have no roadmap. If I did it would have had to change about 10 times since the initial announcement, which is why I haven't been thinking about it and instead have been polishing the underlying libs - bits & melt which will be where most of the heavy lifts for v5 transitions will be.
Once it stabilizes and gets into a release candidate phase I'll start looking into it, but for now, my thoughts are better spent elsewhere!
I will of course update you all once those thoughts do start crossing my mind!

@huntabyte

Whenever you are ready to track progress towards svelte 5 support, can we please re-open this ticket instead of creating a new ticket? This is so that users who have already subscribed to this thread can get notified on updates.

So can we upgrade svelte to version 5 and still use this lib?

@OSZII

So can we upgrade svelte to version 5 and still use this lib?

Since Svelte 5 supports backwards compatibility, you can use the library even if the Svelte version of the package and in your projects are not the same.

and there is a active pullrequest
https://github.com/huntabyte/shadcn-svelte/pull/1182


Support multi-line if-then-else

Currently we can do:
if a==b then action else
    anotheraction

or
if a==b then
    action

It would be good to allow:
if a==b then
    action
else
    anotheraction

(we currently do this by using case a=b of to achieve similar results).

Will need some help from @kazcw

To debug the tests one has to:

Change to debug = 2 in [profile.test] section of the root Cargo.toml to see the local variables.

The handling of if_then and also _else is happening in fn if_body(segments: NonEmptyVec<MatchedSegment>) in built_in.rs. Simple one line if True then 0 else 1 sends in three segments. However trying to:
enso$ git diff lib/rust/parser/debug/tests/parse.rs
diff --git lib/rust/parser/debug/tests/parse.rs lib/rust/parser/debug/tests/parse.rs
index 4a22c36cc5..e6470a4eae 100644
--- lib/rust/parser/debug/tests/parse.rs
+++ lib/rust/parser/debug/tests/parse.rs
@@ -102,6 +102,23 @@ fn else_block() {
                           ((Ident else) (BodyBlock #((Ident False))))))]);
 }

+#[test]
+fn true_else_blocks() {
+    #[rustfmt::skip]
+    let lines = vec![
+        "if True then",
+        "    0",
+        "else",
+        "    1",
+    ];
+    let text = &lines.join("\n");
+    let expected = block![
+        (MultiSegmentApp #(((Ident if) (Ident True))
+                           ((Ident then) (BodyBlock #((Number () "0" ()))))
+                           ((Ident else) (BodyBlock #((Number () "0" ()))))))];
+    test(text, expected);
+}
+

 // === Comments ===

triggers just the if_then macro (and sends just two segments) as macro processing is terminated by resolvers.rs finish_current_line which has no idea there is going to be else somewhere below. else is then treated as a separate identifier invoked with the 1 block:
(BodyBlock #((MultiSegmentApp #(((Ident if) (Ident True)) ((Ident then)
(BodyBlock #((Number () \"0\" ()))))))
(ArgumentBlockApplication (Ident else) #((Number () \"1\" ())))))

I got an idea to disable the if_then macro by
enso$ git diff lib/rust/parser/src/macros/built_in.rs
diff --git lib/rust/parser/src/macros/built_in.rs lib/rust/parser/src/macros/built_in.rs
index 1f789687af..b6eecb4c8c 100644
--- lib/rust/parser/src/macros/built_in.rs
+++ lib/rust/parser/src/macros/built_in.rs
@@ -19,7 +19,7 @@ pub fn all() -> resolver::MacroMap {
 /// Built-in macro definitions that match anywhere in an expression.
 fn expression() -> resolver::SegmentMap<'static> {
     let mut macro_map = resolver::SegmentMap::default();
-    macro_map.register(if_then());
+//    macro_map.register(if_then());
     macro_map.register(if_then_else());
     macro_map.register(group());
     macro_map.register(lambda());

that prevents the first line to be recognized as if_then macro, but of course, it doesn't recognize multi line if_then_else either.

Just adding to that to clarify that if possible it would be best to also be able to 'chain' multiple conditions without introducing additional indentation, e.g.:
if cond1 then
    ...
else if cond2 then
    ...
else if cond3 then
    ...
else
    ...


Responsewriter status in before funcs is broken

This commit https://github.com/urfave/negroni/commit/8b113cd90ea0a4dcd51cac1c8addb5ebc927532d broke the ability to retrieve the request status code inside a middleware
As you can see, status header is set before actually calling callBefore (https://github.com/urfave/negroni/blob/master/response_writer.go#L46-L47) so I was retrieving the status code in my before funcs.
Now with this new validation  https://github.com/urfave/negroni/blob/master/response_writer.go#L62-L69 it's not possible anymore.
@jszwedko

Now with this new validation https://github.com/urfave/negroni/blob/master/response_writer.go#L62-L69 it's not possible anymore.

I know it's possible but I shouldn't have to change my code as this should work as indicated.

Fix in https://github.com/urfave/negroni/pull/130

Closed by #124 -- thanks again for the report!


Added IP2Location LITE database support.

Related to #69 .

Hi @cdown,
May I check what is the progress now?

Hi! I generally have concerns here: there should be one uniform schema and right now there is not one. Previously it's upheld by the contract from Maxmind but now there are two separate contracts, which is not great. The code duplication is also not great.
In general I'd accept a small amount of code growth for a small maintainability burden, but this adds a large one for a code path I won't personally use, so it's realistically unlikely to go in unless that is reduced dramatically and the other issues are also addressed.

Understood. You can close this ticket.


AjaxControlToolkit Version=3.5.40412.0 does not found for an old project

0
I am using Microsoft Visual Studio 2019 and I have the reference for AjaxControlToolkit with warning in the project references.
Web.config: <add assembly="AjaxControlToolkit, Version=3.5.40412.0 ...>
I installed the latest version on AjaxControlToolkit V20.1, but even in the Toolbox is disabled.
I tried to remove the version in Web.config: but I still get the same build error: Error CS0400 The type or namespace name 'AjaxControlToolkit' could not be found in the global namespace (are you missing an assembly reference?)

I found version 4 works


SteelSeries 3 Wireless Light Settings

Hi, I bought the SteelSeries 3 wireless mouse this Christmast. I found it a really good mouse but i have a problem. In my laptop , I only have Debian11 installed as OS. I am an university student so I go to library and for me, is really annoying having mouse lights working on the library. I really want if it's possible for you to implement light control for Steelseries rival 3 wireless . I would really apreciate it.

Hello,
I still have some work to do on this device but I cannot work on it currently as I do not own the device and my colleague that lend it to me is not available...
I close this issue as there is already #146 about the support of this device :)


#19: improve logging of user for c8yclient commands

c8y/client has some special handling of users that might be confusing when using cy.c8yclient.
Behaviour of cy.c8yclient now changed to:

prefer basic auth if passed as previousSubject as with cy.getAuth(...).c8yclient(...). this is similar to using preferBasicAuth option.
debug logging of CookieAuth and BasicAuth being used or available to c8y/client
debug log user for CookieAuth and BasicAuth

cy.login now stores the logged in user as Cypress env C8Y_LOGGED_IN_USER.

@hnaether-sag @CGuether is change 1.) now as expected? Thats addressing the problem reported in your tests when cookie user from cy.login was used instead of the user passed into cy.c8yclient via cy.getAuth.


Failed to boot WebContainer  Booting WebContainer

Describe the bug
Suddenly, I have got a message error like the following:
Failed to boot WebContainer.
Reloading doesn't help. Web container is always down.
Link to the blitz that caused the error
https://stackblitz.com/edit/angular-clean-view-child-d9bmxz?file=package.json
Steps to reproduce

Go to https://stackblitz.com/edit/angular-clean-view-child-d9bmxz?file=package.json

Expected behavior
Web container works.
Parity with Local

[X] I have run the project in my local machine and I could not reproduce the issue.

Screenshots

Platform
Browser name  = Chrome
Full version  = <IP_ADDRESS>
Major version = 109
navigator.appName = Netscape
navigator.userAgent = Mozilla/5.0 (Macintosh; Intel Mac OS X 10_15_7) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/<IP_ADDRESS> Safari/537.36
performance.memory = {
  "totalJSHeapSize": 278920381,
  "usedJSHeapSize": 217495269,
  "jsHeapSizeLimit":<PHONE_NUMBER>
}
Hash = 685d3c93544a5e80be2c9e6906d68c0f4bd854cc

Additional context
No response

Hi @Arthur-Fedotiev. Thanks for reaching out.
It's probably due to the fact that you have to many WebContainer projects open, could that be the case? We have a section in our docs on this https://developer.stackblitz.com/codeflow/working-in-codeflow-ide#out-of-memory-error. Maybe we should print a more decent error message in the devtools pointing to that section.
Hope this helps!

:wave: We're closing this due to inactivity. Please, feel free to open a new one if you're still encountering any errors.


Near create_account reports unclear error when account already exists

near create_account nearproject
Error:  { BadRequestError: Transaction PwxM38gabyPbmtsi3RcOzvsDUwF/Lu9o6sjVbDnl4wQ= on unknown contract failed.
    at sendJson (/usr/local/lib/node_modules/near-shell/node_modules/nearlib/internal/send-json.js:12:15)
    at <anonymous>
    at process._tickCallback (internal/process/next_tick.js:182:7)
  message: 'Transaction PwxM38gabyPbmtsi3RcOzvsDUwF/Lu9o6sjVbDnl4wQ= on unknown contract failed. ' }

This works fine now e.g.
[janelocal]: Runtime error: Account unittest2 already exists
[ 'Runtime error: Account unittest2 already exists' ]


ModalBottomSheet: Incorrect scrolling behavior when partially expanded

Description:
When the ModalBottomSheet is partially expanded, scrolling the list upwards causes the sheet to move along with the list. The expected behavior is that the list should scroll normally within the partially expanded sheet, and the sheet should only be draggable by the header or other static elements, or when the end or beginning of the list is reached (depending on the direction of movement).
Steps to Reproduce:

Initialize a ModalBottomSheet with the following state:val partiallyExpanded = remember {
    SheetDetent(identifier = "part") { containerHeight, sheetHeight ->
        containerHeight * 0.5f
    }
}

val sheetState = rememberModalBottomSheetState(
    initialDetent = Hidden,
    detents = list of(Hidden, partiallyExpanded, FullyExpanded),
)

Set up the initial content and the ModalBottomSheet as follows:Box(modifier = Modifier.fillMaxSize().background(Color.White), contentAlignment = Alignment.Center) {
    Text("Show ModalSheet", modifier = Modifier.padding(20.dp).clickable { sheetState.currentDetent = partiallyExpanded })
}

ModalBottomSheet(state = sheetState) {
    Sheet(modifier = Modifier.statusBarsPadding().displayCutoutPadding().padding(top = 20.dp)) {
        Column(modifier = Modifier.fillMaxSize().background(Color.DarkGray)) {
            Box(modifier = Modifier.fillMaxWidth()) {
                Text("some header", modifier = Modifier.align(Alignment.Center).padding(vertical = 20.dp), color = Color.White)
            }
            LazyColumn(
                modifier = Modifier.fillMaxSize(),
                horizontalAlignment = Alignment.CenterHorizontally
            ) {
                items(100) { index ->
                    Text(text = index.toString(), modifier = Modifier.padding(8.dp), color = Color.White)
                }
            }
        }
    }
}

Expand the ModalBottomSheet to the partially expanded state.
Scroll the list upwards.

Expected Behavior:

The list should scroll normally within the partially expanded sheet.
The sheet should only be draggable by the header or other static elements, or when the end or beginning of the list is reached (depending on the direction of movement).

Actual Behavior:

The sheet moves along with the list when scrolling upwards.

Library version: 1.11.0

https://github.com/user-attachments/assets/b18de698-2b47-4ae3-8d5d-a697941b63b8

Fixed in 1.11.1 https://github.com/composablehorizons/composables-core/releases/tag/1.11.1


How to get Created By Name and Last Updated By Name at querying?

How do I get the CreatedBy user's Name and LastUpdateBy user's Name at querying? Let's say I want to display who created a product and who updated it last, how do I get it? Because the feature for products query is in the Application layer and identity-related entities are in the Infrastructure layer. In short, how do I add created by user's name and last updated by user's name in GetAllProductQuery's response?

First, Do u understand the Onion pattern used?
That is first thing in having flexibility with the clean architecture.
For me i have converted the Onion pattern to my normal way of coding but i understand the Onion first.
Cheers to guys behind the template !

@chrisgate, I do understand the Onion Architecture. That's why I don't want any external dependency like identity in my Domain. Infact, I'm using BlazorHero template on my project. I don't understand what you are trying to convey. I do have some workaround to achieve this. But I wanted to know if there is a better way. How would you suggest that you would achieve this?

@ANMSalman, You can rewrite the following code to replace userid with username
public override async Task<int> SaveChangesAsync(CancellationToken cancellationToken = new CancellationToken())
        {
            foreach (var entry in ChangeTracker.Entries<AuditableEntity>())
            {
                switch (entry.State)
                {
                    case EntityState.Added:
                        entry.Entity.CreatedBy = _currentUserService.UserId;
                        entry.Entity.Created = _dateTime.Now;
                        break;

                    case EntityState.Modified:
                        entry.Entity.LastModifiedBy = _currentUserService.UserId;
                        entry.Entity.LastModified = _dateTime.Now;
                        break;
                }
            }

            var result = await base.SaveChangesAsync(cancellationToken);

            await DispatchEvents();

            return result;
        }


Samplers can't be seeded or pickled

Hello dear pytorchphysics team :)
I am trying to optimize the hyperparameter of some PINN models, which requires me to compute a reliable validation error. Because I want to run the optimization on multiple machine, I need a way to distribute the same validation points to all the workers. I want to use the same validation points as this reduces the variance of the loss value quite a bit. This is a fairly standard approach in HPO reduce the noise.
I didn't find any option to seed a sampler (which would be a cool feature in general), so I tried to create a static sampler to store on disk. Unfortunately, the Sampler objects are neither torch.nn.Module with a state_dict nor can they be pickled and unpickled with pickle.load. The latter results in a TypeError: unhashable type: 'collections.OrderedDict'.
Is there an easy way to create the same static sampler on multiple machines? Any pointer would be appreciated :)

Hey,
Adding seeds to the random sampler is a good suggestion.
Regarding your problem: Maybe the "tp.samplers.DataSampler" can help? You could first create your validation points any way you like (numpy, pytorch, etc.), save them, and then load them for use in the DataSampler, which will return exactly those points on each machine.

Thanks @TomF98, that should work. I didn't know about the DataSampler, so many thanks for the pointer. That should fit my use case.

Thanks @TomF98, that should work. I didn't know about the DataSampler, so many thanks for the pointer. That should fit my use case.


NB

NB

What do you mean with that?

nice
发自我的iPhone
------------------ Original ------------------
From: Daniel<EMAIL_ADDRESS>Date: Tue,Jul 14,2020 3:02 PM
To: vigorcrust/tcpforward<EMAIL_ADDRESS>Cc: guozhong<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [vigorcrust/tcpforward] NB (#1)
What do you mean with that?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or unsubscribe.


subscribed view (useView) does not re-run after doc delete while reducing

Description
Adding and updating docs re-runs a saved view as expected and runs map and reduce (_sum) properly. Deleting docs does not re-run the view as expected and results in incorrect _sum calculation (no update).
Steps to Reproduce
Delete doc from a saved view while reducing.
Expected Behavior
subscribed view should re-run after deleting document related to a subscribed view.
Your Environment

CouchDB version used:
3.1.1
PouchDB version used:
7.2.2
React version used:
17.0.1
Browser name and version:
Firefox / ios / safari
Operating system and version:
Macos

Additional Context

Thanks for reporting this bug!
It was a known one. I should have documented it. It also doesn't update when you update a document in a way, that would remove it from the view.
Up until now useView, when reducing, would only listen to the changes feed filtered by the view.
I did chosen this way, because I didn't know how to make it else fast.
In the next release it will update for every update. But it will only start an update if no update is already running. If that is the case, then it will start another one once the running one did finish.
This is the compromise I have come up. I can't just listen to view updates (for reduce), because

Reduce doesn't have the doc IDs. So I can't check for updates that left the view.
A user could reduce over a large set of docs, so it would probably very memory consuming if I would query with and without reduce, to get the doc IDs.

Not reduced views subscribe to the view-update and the IDs in the result.

:tada: This issue has been resolved in version 1.3.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Add code-syntax with collection name to resolve CSS variables in Figma

[x] Input Helper
[x] UI
[ ] Action

As a general note the next collection should provide a variable for all modes in the previous collection. This is needed to generate the output consistently without making the solution overly complex.
Previous collection: System state
Next collection: User state
The next collection, “User state” should support all modes in the previous collection, “System state”:
User state modes: initial, hover
System state modes: enabled, disabled
-enabled-initial
-enabled-hover
-disabled-initial
-disabled-hovrer
The complex solution would be to check if the system state is disabled and then ignore the user state segment when resolving the variable.
The benefit of the complex solutions is that we don't duplicate the disabled token into both disable/initial and disable/hover.

From an architectural point of view it could be better to separate the enabled/disabled tokens. Then we would not have the System state collection with the enabled and disabled mode. We could have the disabled tokens as separate variables by default all the other variables are enabled.
This complexity occur when we want to switch between the different modes using variable collections. If we could lean on component properties for state/mode like disabled and enabled this would be resolved.

Update: We now have Action, Input, and Interaction State collections with all the different states/modes.


Lack of actual JSON RPC API

Related to #105.

@dulanov Besides questions I listed in the code comments, another one is about bytearray encoding. So we'll use the same approach as Ethereum RPC and encode bytearray as hex strings, right?
Right now there's a single difference for Ethereum RPC encoding and the one SputnikVM accepts -- SputnikVM doesn't require hex strings to be prefixed by 0x. What's the case for emerald? Do you think it's a good idea to require 0x prefixing or make it optional?

@sorpaas yes, generally we should implement something like personal module from geth, but with some additions - https://github.com/ethereum/go-ethereum/wiki/Management-APIs#personal.

About 0x prefix, for emerald it's optional now, you can add it or not - no difference


support trace_beginex and trace_endex

Note: Please adhere to Contributing Guidelines.
Summary
Update this section with information on why change is necessary,
what it exactly does and how, if new feature shows up, provide
references (dependencies, similar problems and solutions), etc.
Impact
Update this section, where applicable, on how change affects users,
build process, hardware, documentation, security, compatibility, etc.
Testing
Update this section with details on how did you verify the change,
what Host was used for build (OS, CPU, compiler, ..), what Target was
used for verification (arch, board:config, ..), etc. Providing build
and runtime logs from before and after change is highly appreciated.

let's ignore macOS ci failure which is fixed by the last master.


tooltip icon and hover not showing on index

laravel/nova v2.10.1
The issue:
Tooltip::make('Roles', function () {     return 'tooltip message'; })     ->onlyOnIndex(),
The tooltip icon is not showing and there's no hover text.

Hey @chorton, I've just installed this package and the same happened to me. I fixed it by running "npm run dev" on the root of the project, have you tried that?

That worked!  Thank you, @gmedeiros


Base the profile-and-properties itest on Java code rather than build output  Det

@jpkrohling could you please review and eventually merge?
c61cff2 is an important bugfix due to which I'd release after this PR is merged.


Completed Task 519 : Storybook >Elements> Side Navigation > Side navigation is not as per the application

Description

Changes done for the enhancement of the side navigation
Added 4 new components for the side navigation
1)Collapse
2)List Expand
3)List Collapse
4)Right Collapse

modified basic side navigation for removed islocked button and side bar is not getting hidden it is showing in default state
Screenshots :

Checklist:

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[x] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes

closing this pr because Deepashri has already done enhancement of the side navigation


Unable to load authentication plugin 'caching_sha2_password'

System information:

MS Windows 10, 10.0.19041 Build 19041
DBeaver <IP_ADDRESS>010041557

Connection specification:

YeahRight Mysql 8.0.21-0ubuntu<IP_ADDRESS>

Describe your question:
Unable to load authentication plugin 'caching_sha2_password'

Hi @rezetta ,
could you provide steps to reproduce?

I have the same issue. How did you resolve it @rezetta ?


Text color

Hi,
I have a light vscode workbench theme.
When I use the vector-replace extension, the text labels of the checkboxes are in the same dark color as the text in the workbench theme (dark text on light background).
But the background color of the vector-replace pane is black (dark text on black background).
So it is not readable.
Thanks

Hi,
This is a bug in stylesheet, so I will fix it soon.
Thanks for nice issue.


Added message component support

-Added support for message components.
-Updated deprecated channelTypes
-Messages owned by the client user are now sent as the client user
-Fixed the bug in LoadMessages function

Thanks for working on this @SjogrenDev, I do have a question though, why are you deciding to store the old message id in the backup now? Additionally what is the purpose of adding the messagesPerChannel property? I would imagine it would be better to just derive the value off of the number of actually saved messages in a channel rather than assume every channel has the maximum which is already provided through the options anyway.

Hi,
I've added the old message id storing because with Discord bots we often work with channel ids and message ids, so the old id might be needed. Additionally, the project currently stores old channel ids, so it's a good practice to do it for messages too.
I've added the messagesPerChannel property because currently there's no way to check how many messages per channel are saved on an existing backup and we need that number for the backup load function.

messagesPerChannel does not appear to be used in the backup load function, do you mean you are using that to pass to the option maxMessagesPerChannel yourself?

messagesPerChannel does not appear to be used in the backup load function, do you mean you are using that to pass to the option maxMessagesPerChannel yourself?

Yeah, currently we cannot know how many messages per channel were saved on a backup, thus why I added it, so we can pass the number or lower in maxMessagesPerChannel in the backup load function


Not sure I understand this.

In your explanation you said the following:
Return a Task object, instead of returning an awaited Task object, when there's no need to process the result of an asynchronous operation. This is more performant due to less context switching being performed.
I would like an example of this as I am not sure what is meant by this.  Thanks.

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

ID: 4a6c9a34-df78-94ca-5e6d-701c743eb658
Version Independent ID: 4c81800f-dc7f-aa92-55c0-d5a90ed8c3c2
Content: Improve Xamarin.Forms App Performance - Xamarin
Content Source: docs/xamarin-forms/deploy-test/performance.md
Product: xamarin
Technology: xamarin-forms
GitHub Login: @davidbritch
Microsoft Alias: dabritch

Hi @joanneraye
Hopefully these videos will help:
https://channel9.msdn.com/Shows/XamarinShow/Best-Practices-Async--Await--The-Xamarin-Show
https://channel9.msdn.com/Shows/XamarinShow/Best-Practices-Advanced-Async--Await--The-Xamarin-Show


Added test cases for sidekick , service volume mount and host routining.

@alena1108 , could you please review the test cases for sidekick and service volume mount feature that covers the primary use cases.

LGTM


⚠ Bofrost.be has degraded performance

In 29cee91, Bofrost.be (https://www.bofrost.be) experienced degraded performance:

HTTP code: 200
Response time: 3483 ms

Resolved: Bofrost.be performance has improved in f26ede5.


Releasing/1.8.4 [wip]

What does this PR do?
TBD
Before submitting

[x] Was this discussed/approved via a GitHub issue? (not for typos and docs)
[x] Did you read the contributor guideline, Pull Request section?
[x] Did you make sure your PR does only one thing, instead of bundling different changes together?
[ ] Did you make sure to update the documentation with your changes? (if necessary)
[ ] Did you write any new necessary tests? (not for typos and docs)
[ ] Did you verify new and existing tests pass locally with your changes?
[ ] Did you list all the breaking changes introduced by this pull request?
[x] Did you update the CHANGELOG? (not for typos, docs, test updates, or minor internal changes/refactors)

PR review
Anyone in the community is welcome to review the PR.
Before you start reviewing, make sure you have read the review guidelines. In short, see the following bullet-list:

[ ] Is this pull request ready for review? (if not, please submit in draft mode)
[ ] Check that all items from Before submitting are resolved
[ ] Make sure the title is self-explanatory and the description concisely explains the PR
[ ] Add labels and milestones (and optionally projects) to the PR so it can be classified

Did you have fun?
Make sure you had fun coding 🙃
cc @borda

trying to resolve the hanging benchmarks in #15879

resolving the failing TOU tests (in fact the tests even won't start) in #15876

also would be good to have #15798

adding automation in #15871 and #15887

seem the failing PL-cpu test are green but the CI/check ends-up red :otter:

@tchaton see like all e2e tests are failing after picking your last commit, mind having a look?


Update vue 2.3.2 -> 2.4.2, vue-router 2.5.2 -> 2.7.0, vue-resource 1.…

Update vue 2.3.2 -> 2.4.2,
vue-router 2.5.2 -> 2.7.0,
vue-resource 1.3.1 -> 1.3.4

Sorry for the slow reply. I've been away and it looks none of the other owners picked it up.
This PR is missing the changes to the Gem version. Checkout this previous PR for an example of how to update this.
Also it would be worth bringing the dependencies up-to-date as well if new versions have been published over the last week.

@rkJun 👍  Good job, updated Gem has been published. Thanks for contributing!


chore(sdf): remove check when cancelling an action

If an action fails to run to completion (i.e. func execution fails, there is a graph conflict, a transient/uncaught database error, etc), it may be left on the graph in a "running" state. Prior to this change we checked the action to see if it's dispatched or running and returned an error as this would imply that our system supports first-class interuptable cancellation of functions (which we do not yet support). This change removes the check and will consequently remove the action from the graph, allowing a user to re-enqueue the action to run again, etc.
Closes BUG-331

bors try

bors r+


Support for Country Code and National Number

closes #6
Example usage:
final parsed = await PhoneNumber.parse("+79991234567");
platformVersion = """
country code: ${parsed['country_code']}
national number: ${parsed['national_number']}
""";

In case of +79991234567 country code is 7 and national number is<PHONE_NUMBER>

@nashfive could you please push an update with this change included? 0.3.1 maybe?

@vanyasem yeah sorry, I will do it. I have very little time right for personal projects, but I will create the release asap


Gotype crashes in terminal with package viper

What version of Go are you using (go version)?

$ go version
go version go1.13 darwin/amd64

Does this issue reproduce with the latest release?
Yes
What operating system and processor architecture are you using (go env)?
go env Output
$ go env
GO111MODULE=""
GOARCH="amd64"
GOBIN=""
GOCACHE="/Users/corentin/Library/Caches/go-build"
GOENV="/Users/corentin/Library/Application Support/go/env"
GOEXE=""
GOFLAGS=""
GOHOSTARCH="amd64"
GOHOSTOS="darwin"
GONOPROXY=""
GONOSUMDB=""
GOOS="darwin"
GOPATH="/Users/corentin/go"
GOPRIVATE=""
GOPROXY="https://proxy.golang.org,direct"
GOROOT="/usr/local/Cellar/go/1.13/libexec"
GOSUMDB="sum.golang.org"
GOTMPDIR=""
GOTOOLDIR="/usr/local/Cellar/go/1.13/libexec/pkg/tool/darwin_amd64"
GCCGO="gccgo"
AR="ar"
CC="clang"
CXX="clang++"
CGO_ENABLED="1"
GOMOD="/Users/corentin/code/my_recipes/backend/go.mod"
CGO_CFLAGS="-g -O2"
CGO_CPPFLAGS=""
CGO_CXXFLAGS="-g -O2"
CGO_FFLAGS="-g -O2"
CGO_LDFLAGS="-g -O2"
PKG_CONFIG="pkg-config"
GOGCCFLAGS="-fPIC -m64 -pthread -fno-caret-diagnostics -Qunused-arguments -fmessage-length=0 -fdebug-prefix-map=/var/folders/sz/z0q535dx2876nknnf0dfk2fh0000gn/T/go-build204703159=/tmp/go-build -gno-record-gcc-switches -fno-common"

What did you do?
I have the following (simple) project, with a main.go:
package main

import (
	"fmt"
	"github.com/spf13/viper"
)
func init() {
        // this is to avoid the "imported but not used" error
	viper.SetConfigFile("./main")
}
func main() {
	fmt.Println("hello")
}

And the following go.mod:
module testissue
go 1.13
require github.com/spf13/viper v1.4.0

I can build and run the binary, but if y run gotype -t -e main.go I get the following error :
main.go:5:2: could not import github.com/spf13/viper (type-checking package "github.com/spf13/viper" failed (/Users/corentin/go/pkg/mod/github.com/spf13/viper@v1.4.0/flags.go:3:8: could not import github.com/spf13/pflag (go/build: importGo github.com/spf13/pflag: exit status 1
error writing go.mod: open<EMAIL_ADDRESS>permission denied

)))
zsh: exit 2     gotype -t -e main.go

This is only the case with viper, if i use another package, such as cobra  by the same team, there is no error.
Even wierder, if I run gotype from the gotype extention of Sublime-Linter, it manages to lint correctly every time with no errors.
What did you expect to see?
No error when running gotype, and no mention of writing a go.mod in an external package.
What did you see instead?
The error mentioned above.

I also opened an issue in the viper repository because I don't know if the error is caused by gotype or viper.

Not sure what's going on here. /cc @bcmills in case this is modules related, and /cc @griesemer in case this is related to gotypes.

This isn't a crash, it's a properly returned error (adjusted title).
I can reproduce this, and the underlying error seems to be:
go/build: importGo github.com/spf13/pflag: exit status 1
error writing go.mod: open<EMAIL_ADDRESS>permission denied

which points at go/build. @bcmills ?

This is directly related to a suspected existing bug in go/build:
https://github.com/golang/go/blob/b649bdc7f3c99c5288c91a1ce148efadd86e19a4/src/go/build/build.go#L1057-L1060
It's nice to have a concrete reproducer, though. 😅


Add Icon to GridButtonItem

HI!
is possibile to implement icons in grid button item widget?
Thanks
Riccardo

Sorry for late reply.
Now added the "child" property, you can add widgets such as icons.


Add "Tags" tab to Settings

This issue covers the following requirements:
[REQ_SET_014] A user should be able to manage account tags in Settings
Mockup

When implementing this requirement, please check out if we need to create a separate task for refactoring account tags (ngrx)


Vector versions of Spleen

I absolutely love this font, it would be great if it had a vector version of the font so I could use it for GTK and Firefox. The 32x64 TTF does look okay on scaling down.

Release tarballs contain the fonts in various formats, including OTF: https://github.com/fcambus/spleen/releases/download/1.9.1/spleen-1.9.1.tar.gz

By Vector version, I meant a scalable font, not a bitmap font.

So do I, the OTF versions are vectorized using bdf2sfd.

they look pixelized when scaling up


KeyError: "attribute 'weight' already exists"

composer train.py mpt-1b.yaml，get the keyerror
GPU：3*RTX4090

This seems to be an issue with torch2 and weight tying which we are working on fixing. You can set use_orig_params=False in your fsdp_config in the meantime, and it should work. Or use torch 1.13 instead.

This should be resolved on main. Please let us know if you are still encountering problems.


docs: add integration-doc.md

First stab at integration document.
Plenty still to be fleshed out, but wanted to get this out to make sure it's useful.

can you try npm run docgen with that folder under ./docs? I am not sure whether solidoc would remove the existing folder first or just do an update.....

can you try npm run docgen with that folder under ./docs? I am not sure whether solidoc would remove the existing folder first or just do an update.....the new file might potentially be deleted by solidoc?

It doesn't delete it 😅
In any case, I was going to move the generated docs to ./docs/contracts or something. Do you agree?

@kevsul @boolafish I am wondering if you guys think that documentation on the user/data flows is part of the integration doc. Let's take for example ERC20 deposits.
1.) User approves the amount of ERC20 tokens that are about to be transferred.
2.) User creates a RLP encoded transaction, that is decoded in ERC20Vault.
3.)safeTransferFrom is called which does the ownership transfer of the tokens
4.) The transaction is included in an empty block and Merkle root is used to create a new deposit block.
5.) The DepositCreated event is triggered which does ...
I think documentation that surfaces interaction between the components contracts, child chain, watcher and client would be really useful.

5.) The DepositCreated event is triggered. The child chain listens for this even and does ..
I think documentation that surfaces interaction between the components contracts, child chain, watcher and client would be really useful.

This is interesting, I'm very supportive of producing this kind of docs. Let us know if you need help with it, when it comes to chch/watcher.
I think, even if this content is written down somewhere, it might be quite dispersed, so there's surely a need for docs that describe this specifically. So far, we only have a general documentation of those interactions in here.
The question here is where do we draw the line between contracts documentation and child chain/watcher documentation.
My suggestion would be to write only general expectations (or better put - intention of how events and entry-points to the contracts should be used), and go into the specifics of "what happens" on the child chain/watcher docs' end.
E.g. contracts would only have:

The DepositCreated event is triggered. This event can be listened to to make the deposited funds spendable on the child chain

vs on the chch/watcher:

DepositCreated is recognised in the child chain after a deposit_finality_margin.
After that the child chain will allow to spend the given UTXO in transactions, provided a correct utxo position (output id) is given along with a correct signature.
DepositCreated is recognised in the watcher after a deposit_finality_margin. After that, the Watcher will treat transactions correctly spending the deposit as valid...

and so forth...

My suggestion would be to write only general expectations (or better put - intention of how events and entry-points to the contracts should be used), and go into the specifics of "what happens" on the child chain/watcher docs' end.
Makes sense also from a security perspective. Then people looking at the security of the system have some context when they look at the actual implementation and look for gaps.
It's ok to have it in different places and I can see why the teams want to maintain their own. My only request is that they should overlay so if someone wants to understand the entire system they have an easier time with it.

they should overlay so if someone wants to understand the entire system they have an easier time with it.

Totally :+1:. I tried to make the example :point_up: overlay.
Enough information to describe the apis, data structures, flows, etc.
For super detailed explanations on the how and why it all works, you can go to the spec.

The question here is where do we draw the line between contracts documentation and child chain/watcher documentation.

Do the child chain/watcher docs already exist @pdobacz ?

@kevsul

Do the child chain/watcher docs already exist @pdobacz ?

There's some but dispersed, incomplete and not on the proper level of detail I guess. Docs like this doc mentioned above or this and comments throughout code.

Gerhard has asked that I review this content from a language and layout perspective. I've created a google doc out of the content at https://github.com/omisego/plasma-contracts/blob/master/plasma_framework/docs/integration-docs/integration-doc.md and will edit offline with suggested comments and then create a PR for the changes. Is that OK with everyone involved?


PR: Migrate the Outline Explorer to use LSP information

Description of Changes
This PR migrates the Outline Explorer to use the information provided by the LSP via textDocument/documentSymbols call.

[ ] Wrote at least one-line docstrings (for any new functions)
[ ] Added unit test(s) covering the changes (if testable)

[x] Included a screenshot or animation (if affecting the UI, see Licecap)

Issue(s) Resolved

Fixes #
Affirmation
By submitting this Pull Request or typing my (user)name below,
I affirm the Developer Certificate of Origin
with respect to all commits and content included in this PR,
and understand I am releasing the same under Spyder's MIT (Expat) license.

I certify the above statement is true and correct: @andfoy

Hello @andfoy! Thanks for opening this PR. We checked the lines you've touched for PEP 8 issues, and found:

In the file spyder/plugins/outlineexplorer/plugin.py:

Line 57:5: E265 block comment should start with '# '
Line 57:80: E501 line too long (80 > 79 characters)

/show binder

My comments from manually testing this:

This PR is showing the imported modules in a file, which takes a lot of space (one per line) and I don't think it's really necessary. Perhaps we should show remove them? How does this work in VSCode?

Multiline imports are shown as nested now.

This PR is only showing the current file on the tree. This is a loss of a functionality because right now we are showing all open files on the Outline:

It takes several seconds (three or more) to display the contents of the tree the first time a file is opened. This is something we really need to fix before merging it.

This PR is only showing the current file on the tree. That's a loss of a functionality because right now we are showing all open files on the Outline:

If you have the option Show all files, then all the files will appear:

How does this work in VSCode?

It seems that variables are displayed, but imports not.

Methods and attributes of classes are displayed as variables and functions:

This is a limitation of the LSP spec, which does not consider attributes nor methods

Thanks @andfoy for your latest updates to this PR! The spinner interaction is working great, but we're still showing a lot of unnecessary imports:

I think this is something that needs to be fixed because it constitutes an important regression with respect to what we have now:

For the record, as requested by @ccordoba12 , some quick test results moving lines that result in Outline Explorer updates, all with default Spyder settings, on mainwindow.py starting from around line 1000 and moving for 100 lines:
Previous result without this PR on 4.x, Outline Explorer hidden: 13 lines/s
Current result with this PR, Outline Explorer hidden: 13 lines/s
Current result with this PR, Outline Explorer shown: 10 lines/s
The Outline Explorer appears to update approximately 1-2x second with perhaps 100-200 ms lag, which would explain most of the slowdown (assuming it was blocking or stealing cycles from GUI updates during most of that time).


RuntimeError : Address already in use at /opt/conda/conda-bld/pytorch_1535491974311/work/torch/lib/THD/process_g roup/General.cpp:17

When I try to run 'main.py', the error occur.
I don't know the reason.

I got same problem when I run this code in multi machine, but it works in single machine.

Please try this branch https://github.com/narumiruna/pytorch-distributed-example/tree/migrate-to-pytorch-1-0-1/mnist, and set the env variable:
GLOO_SOCKET_IFNAME, for example export GLOO_SOCKET_IFNAME=eth0
https://pytorch.org/docs/stable/distributed.html#choosing-the-network-interface-to-use

Thx for reply，but I don't know where to set environment variable is this means os.environ[' GLOO_SOCKET_IFNAME']=eth0 or set in linux?   I am new to linux and distributed computing

Thx for reply，but I don't know where to set environment variable is this means os.environ[' GLOO_SOCKET_IFNAME']=eth0 or set in linux? I am new to linux and distributed computing

Use ifconfig to find your internet interface
If your internet interface is 'eth0', you can use this command to set the environment variable
$ export GLOO_SOCKET_IFNAME=eth0
or
os.environ["GLOO_SOCKET_IFNAME"]="eth0"
in python.

I try your method， however it not work and shows same problem.

Could you give me more information about your environment, e.q. python version, pytorch version (how you installed it? conda, pip, ...), OS version, CUDA/cuDNN version, or more. It can help me to reproduce the problem. Thank you.

The above problem I use Ubuntu 16.04 LTS as rank 0 and another is  ubuntu  18.04 LTS.  Both of environment installed  by pip and  set device with cpu.
Dose the linux version affect the program's performance? i try your program in two identical computer(with same cpu, enviroment and system ubuntu  18.04 LTS ) rank 1 is work and download mnist dataset and rank 0 did nothing.

The above problem I use Ubuntu 16.04 LTS as rank 0 and another is ubuntu 18.04 LTS. Both of environment installed by pip and set device with cpu.
torch==1.0
torchvision==0.2.1(rank0) and 0.2.2(rank1)
Dose the linux version affect the program's performance? i try your program in two identical computer(with same cpu, enviroment and system ubuntu 18.04 LTS ) rank 1 is work and download mnist dataset and rank 0 did nothing.

Try pytorch v1.0.1?

ubuntu 16.04 is 0.4.1 and ubuntu 18.04 are 1.0.1

I fix it seems to be the problem of torch version and systems' version, Thank you for your help!


Set names correctly for all threads

Set name for all threads except brpc, because bprc does not provide a way to set the thread name

run starrocks_be_unittest


Launch Indicator Search

Append a Link or Button at the top of each card display that launches a new browser tab to the 3rd party app search page for the indicator in question. This probably requires a new field in the integrations section in order to define the INTEGRATION_SEARCH_URL.
Where the title of each card becomes ${INTEGRATION} for ${indicator} . link/button to ${INTEGRATION_SEARCH_URL}

@WilliamSalusky was this your request?
How is this different than links? Maybe we could add a default link group that includes these? I feel like this functionality to very close to links and makes more sense there.
Also, there is a problem with certain resource's search capability. For example, Arkime, we need to know the iType of the indicator in order to create the link. If it's an ip, the query would be something like, arkimeservice.org/sessions?expression=ip==<IP_ADDRESS>. If it's domain, the query would be something like arkimeservice.org/sessions?expression=hostname==test.com. But if it were a phone number, it wouldn't apply. Also, services like censys don't have a single search endpoint, you need to know whether you're searching for a host or a certificate. So, again, we need to know the iType of the indicator.
Cards are generic and don't assume a specific iType, so I feel like this functionality makes more sense in a link group. Thoughts?
@awick what do you think?

The intent is to make it as simple as possible to get from a specific card view to the full third party UI web app and not have to navigate the link group section since we are already looking at the card. (card displays are pretty stripped down result sets as compared to what the integration specific full UI external app would display). Virustotal and URLScan are good examples. In most cases the card display is good enough, but in plenty of cases I do want to see all of the additional data enrichment that the full web app presents and is unique to the full app, also not accessible via API results.


cmd/compile/internal/ssa: TestInlineLines fails on Windows

CL 351529 removes a workaround for #43938 on Windows on the regression test for #47880.
Unfortunately, that workaround seems to have been masking another bug on Windows.
https://storage.googleapis.com/go-build-log/033ed565/windows-amd64-2016_fd7dc5ba.log:
--- FAIL: TestInlineLines (0.07s)
    debug_lines_test.go:173: wanted inlines [[3] [4] [5] [4 10] [4 10] [4 11] [5 10] [5 10] [5 11] [4 10 16] [4 11 16] [5 10 16] [5 11 16]] but got []
FAIL
FAIL	cmd/compile/internal/ssa	5.867s

The failing test was added in CL 348970 (CC @dr2chase, @randall77), so this appears to be a test-regression for Go 1.18. (It is not clear to me whether the test regression also represents a regression in functionality.)

That's not an inlining bug, that's a bug in the regular expression for a path separator in the test.
We don't need this test to run on all OSes, so I would just leave it disabled on Windows like it originally was.

Oh, that doesn't seem at all difficult to fix! I'll fold it in to CL 351529.


SubMenu header clickable

Hi,
I have a dynamic submenu for which all the terminal nodes are opening the same list filtered. I used addClickable for this. But I would need also to be able to click on the top of the submenu to show the list unfiltered.
Is it possible?

Surely it is possible, but since this requries new funtionality from the UI component, you will have to create your own component that allows you to hide/show these filtered elements.
You can checkout this Component and this One


fix: db error con missing config

@MystPi If I didn't screwed something doing the PR this includes everything from the store PR and the fix.

Thanks!

🤔 I will check downloading the 2.2.4 branch

So the original error was fixed, but now when you save your settings it gives a similar error.

Did you actually test your settings DB pull request?

Yes, I have Ninetails running right now with my code.
Let me rerun everything and check.

@MystPi
I have another unrelated error
Uncaught (in promise) ReferenceError: createTab is not defined
    at startup.js:13

@MystPi I have another unrelated error
Uncaught (in promise) ReferenceError: createTab is not defined
    at startup.js:13

It didn't say that before your PR... hmm.

Found it!
I'll redo everything and send a new PR, sorry
I have found the problem, was a messed file with local merge. 😞

Found it! I'll redo everything and send a new PR, sorry I have found the problem, was a messed file with local merge. 😞

Great! It's fine. :)


refactor: use SecondsPerYear

Closes https://github.com/celestiaorg/celestia-app/issues/2040

@SurajAnand88 thanks for the contribution! I renamed the PR title to conform to conventional commits.
Ref: https://github.com/celestiaorg/celestia-app#contributing

@evan-forbes I've done the  linting part. Could you please check once is it resolved? Because I'm new to open source.

@SurajAnand88 welcome to open source and we appreciate your contribution! You fixed lint correctly b/c this check now succeeds.

Thanks ❤

Codecov Report

Merging #2043 (d4467a9) into main (bbcab84) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main    #2043   +/-   ##
=======================================
  Coverage   21.54%   21.54%
=======================================
  Files         127      127
  Lines       14293    14293
=======================================
  Hits         3079     3079
  Misses      10922    10922
  Partials      292      292

Impacted Files
Coverage Δ

x/mint/types/constants.go
100.00% <ø> (ø)


[BEAM-3076] support TIMESTAMP in BeamRecordType

seems jenkins is not healthy recently. If this build still fails, let's hold for some time. Sorry for the inconvenience.

retest this please

R: @XuMingmin @xumingming

Thanks for reviewing @XuMingmin. I restored .github/PULL_REQUEST_TEMPLATE.md.

@zangshayang1 I see some tests failed in the jenkins job, can you take a look?
https://builds.apache.org/job/beam_PreCommit_Java_MavenInstall/15409/org.apache.beam$beam-sdks-java-extensions-sql/

SUCCESS
--none--

SUCCESS
--none--

FAILURE
--none--

Hey @XuMingmin, any updates on this PR?


Hot Module Reloading

Rough plan: make ts-node compile to using require-based CommonJS syntax, then leverage the exposed require.cache object to force cache reloads
Once HMR is integrated, finish Sentinel.
Note: Make sure to kill children to avoid creating unclearable trash

HMR added in ef471d87f77fee7082f54d622ea8c7a5492450ca

Debounced command HMR added in 12159f29d23d7f0b1882b6892b4c7d7cb7fbbc20


Add option to include python in environment

Not sure if this should be the default, but there should be two ways to bundle environments:

With a symlinked python and inherited system packages on the target system
With a fully bundled python

Is this implemented?

+1 I think it makes more sense to have the bundled Python by default, and I do think it's more likely to use the packed venv somewhere else instead of in the same machine that has the same Python in the same path.

+1 I think it makes more sense to have the bundled Python by default, and I do think it's more likely to use the packed venv somewhere else instead of in the same machine that has the same Python in the same path.

To add a bit of more context:

I implemented --archive and spark.archives option in other cluster modes, and documented that together with this library in PySpark in order to make this to be more widely used.
See this demo documentation https://hyukjin-spark.readthedocs.io/en/stable/user_guide/python_packaging.html#using-virtualenv which will be in Spark 3.1. RC will be cut out next week.

conda-pack does self-include Python.

I documented that venv-pack also includes Python itself mistakenly. I think I did something wrong when I tested it. If it's not expected to be fixed soon in this repo itself, I will change the documentation to mention this limitation.

To add a bit of more context:

I implemented --archive and spark.archives option in other cluster modes, and documented that together with this library in PySpark in order to make this to be more widely used.
See this demo documentation https://hyukjin-spark.readthedocs.io/en/stable/user_guide/python_packaging.html#using-virtualenv which will be in Spark 3.1. RC will be cut out next week.

conda-pack does self-include Python.

I documented that venv-pack also includes Python itself mistakenly. I think I did something wrong when I tested it. If it's not expected to be fixed soon in this repo itself, I will change the documentation to mention this limitation.


ENOENT error on windows

Hi,
I tried using the drivelist.list function and it returned this error:
Error: spawn \scripts\win32.bat ENOENT
Not really sure is this a problem on my end or something is buggy with the the library.

That is very weird. What Windows version are you using? What happens if you run https://github.com/resin-io-modules/drivelist/blob/master/scripts/win32.bat directly? Can you confirm that there wasn't any NPM issues that could caused the downloaded package to lack certain files? What are the contents of the drivelist directory inside your node_modules?

I am using windows 8.1 64bit.
I just ran the script from a command line and it works fine.
`device: "\\.\PHYSICALDRIVE1"
description: "Generic- Multi-Card USB Device"
size:
raw: "\\.\PHYSICALDRIVE1"
system: False
protected: False
mountpoints: []
device: "\\.\PHYSICALDRIVE0"
description: "WDC WD6400BPVT-75HXZT1"
size:<PHONE_NUMBER>00
raw: "\\.\PHYSICALDRIVE0"
system: True
protected: False
mountpoints:

path: "C:"
path: "D:"
path: "E:"
`

The contents of the drivelist directory: http://prnt.sc/dfrnfd
This is strange... I dont know what could cause this error, since the script itself runs fine on the pc.
I am using your script as a part of a electronJS project, I dont know if that is of any importance though..

That is very weird indeed. This module has been developed for an Electron application (https://github.com/resin-io/etcher) so I don't think that is the issue. Can I run this somewhere? Also, what version of Electron is your application running?

I dont have the app up yet, I have started working on it like a week ago.
But, I do use this template: https://github.com/chentsulin/electron-react-boilerplate.
You can try on this one. The only thing I have different from this one is a few more react dependencies like modals, tree-ui and so on.. Everything else is the same (I am new to electron so I have yet managed to mess around with it)
Electron version from the package.json: "electron": "^1.4.4",

How are you loading/executing drivelist in that boilerplate? Maybe you can share a small patch I can apply to the boilerplate to get a setup similar to yours?

Sorry I take so long to respond, I did have some other matters to finish.
About the code, here is the component I use.
I just execute the drivelist in the constructor just to see what the results are.. Then I ll find an appropriate usage for it.
import React, {Component} from 'react';
import Modal from 'react-modal';
import drivelist from 'drivelist';
import fs from 'fs';

export default class FileDialog extends Component {

  constructor() {
    super();
    this.state = {modalIsOpen: false};
    drivelist.list((error, drives) => {
      if (error) {
        throw error;
      }

      drives.forEach((drive) => {
        console.log(drive);
      });
    });
  }

  openModal = () => {
    this.setState({modalIsOpen: true});
  };

  afterOpenModal = () => {
    // references are now sync'd and can be accessed.
    // this.refs.subtitle.style.color = '#f00';
  };

  closeModal = () => {
    this.setState({modalIsOpen: false});
  };

  render() {
    return (
      <div>
        <button onClick={this.openModal}>Open Modal</button>
        <Modal
          isOpen={this.state.modalIsOpen}
          onAfterOpen={this.afterOpenModal}
          onRequestClose={this.closeModal}
          contentLabel="Example Modal"
        >
        </Modal>
      </div>
    );
  }
};

and I also have the drivelist in the package.json dependencies (added by --save from npm install)
  "dependencies": {
    "drivelist": "^5.0.4",
....

Also experiencing this issue.

Hi @SomeoneWeird ,
Thanks for reaching out. Do you have something I can clone and run to try to reproduce myself?

We have exactly the same issue with running external scripts from within our app. It looks like an issue with path or external dependencies resolving when trying to run this script from electron app. It might be connected to webpack or node configuration but I'm not sure.
We've managed to find a workaround for that issue by copying code and patching the code to look for scripts in our scripts directory but we are still looking for a better solution. We also want to use your etcher-image-write module and can't run it because of the same issue with path resolving.
This is our repo: https://github.com/maciaszczykm/ubikube
If you add and try to import and use drivelist from app then it will fail on finding the script file.

Hi @floreks ,
Thanks a lot for the link to the repo. Let me try it myself, and hopefully and I can reproduce this time.

Hi @floreks ,
OK, so I found the issue. When you build your application, the JavaScript gets concatenated, but any other resources from the NPM package (like drivelist scripts/ directory) is omitted in the final results, thus ENOENT is rightfully thrown.
There are two solutions as far as I can see:

You can modify your build system to not omit these resources. I have never used WebPack, but I guess you can somehow specify certain patterns to be copied to the destination, and use that functionality to bring the scripts directory automatically. Alternatively, you can whitelist certain extensions, like .sh and .bat.

We could inline the source of the platform scripts inside the JavaScript files in drivelist and make the module extract them out to temporary files before executing them, so the whole package would be completely self-contained, however that sounds messy and I'd prefer not to do it unless its really a must.

Its weird that you get errors with etcher-image-write, since that module doesn't execute any internal child process.
Notice that we dynamically require diskpart if we're running on Windows (https://github.com/resin-io-modules/etcher-image-write/blob/master/lib/win32.js#L27), so I have the theory that web pack tries to process every single require() call, but since diskpart is a Windows only dependency, it will not be installed on OS X when doing npm install, and therefore WebPack will not find it in node_modules.
If that's the case, you can do npm install -f, which will force platform specific dependencies to be installed to node_modules.
Let me know how that goes!

Hi @jviotti,

OK, so I found the issue. When you build your application, the JavaScript gets concatenated, but any other resources from the NPM package (like drivelist scripts/ directory) is omitted in the final results, thus ENOENT is rightfully thrown.

Unfortunately we already knew that :), but were unable to find a solution that would force webpack to load raw files (raw-loader did not work correctly). I believe that our etcher-image-write issues are also caused by webpack configuration.
The weird thing is that once we copied the code and scripts directly to scripts dir on our repo then drivelist was able to call them even that they are not picked up by webpack. This path resolving is still very vague to me.

We could inline the source of the platform scripts inside the JavaScript files in drivelist and make the module extract them out to temporary files before executing them, so the whole package would be completely self-contained, however that sounds messy and I'd prefer not to do it unless its really a must.

I wouldn't do that either just to solve some bundler issues. :) With proper configuration everything should work. Webpack is new to us so I believe that we just don't know how to configure it properly.

Notice that we dynamically require diskpart if we're running on Windows (https://github.com/resin-io-modules/etcher-image-write/blob/master/lib/win32.js#L27), so I have the theory that web pack tries to process every single require() call, but since diskpart is a Windows only dependency, it will not be installed on OS X when doing npm install, and therefore WebPack will not find it in node_modules.

This is actually not our issue with image writer. It looks like it might be connected to our transpiler or also webpack config. Code crashes with very mysterious error ctor is not defined in through2 library.
Thanks for your help! We might end up ditching webpack after all.

The weird thing is that once we copied the code and scripts directly to scripts dir on our repo then drivelist was able to call them even that they are not picked up by webpack. This path resolving is still very vague to me.

So drivelist will attempt to load the scripts from ../scripts relatively to the script location, so if you compile your app to, for example, myapp/output/app.min.js, then drivelist will attempt to load them from myapp/scripts.
I'm happy to make drivelist load the scripts relatively to its own code, so in the above case it will look for them at myapp/output. Let me know if this helps and I'll make the necessary changes.

I wouldn't do that either just to solve some bundler issues. :) With proper configuration everything should work. Webpack is new to us so I believe that we just don't know how to configure it properly.

In the case of a desktop application, concatenation and minification is not highly necessary, and I'd assume the impact on runtime speed is minimal (although I bet you're saving a lot of space by doing so). We don't process the JavaScript files in any way in Etcher for instance.

This is actually not our issue with image writer. It looks like it might be connected to our transpiler or also webpack config. Code crashes with very mysterious error ctor is not defined in through2 library.

Ah, really weird. My first thought was that ctor was a native NodeJS C++ addon and thus WebPack was not handling it correctly neither, but looks that that's not the case neither.

Thanks for your help! We might end up ditching webpack after all.

Yeah, that might be the way to go :) In case you're interested, we have our own set of cross-platform build scripts for Etcher to deal with some things we found other solutions lacking, like proper native module compilation, etc. We plan to release these as a separate project at some point, but feel free to take a look at https://github.com/resin-io/etcher/tree/master/scripts/build in the mid-time.

I'll close this issue since its a Webpack specific problem.


DOC: Update README.md with JOSS article info

Coverage remained the same at 95.12% when pulling df779b907752bfe52cf8eb417729e68e23e91d6a on doc/update-readme-joss into 2c999e0a4a3a2cc622770da78a851f63bd3c0a0f on main.


Indirect prompt injection?

This is a very interesting and complete work. Can the threat model established in this work be understood as indirect prompt injection?

The short answer is yes. In our example of automated screening, the outsider threat agent (i.e., candidate) provides data (i.e., resume) which is then retrieved by the service user (i.e., HR) to query the LLMs.
Our threat model also considers "direct" prompt injection where the threat agent is the malicious user. In this case, it is consistent to the threat modeling in works like PLeak.


feat!: CRM list custom objects endpoint should also include labels

Tested locally
This contains a breaking interface change

Is there a staging env I can play-through this on?

Yes -- any of them that has hubspot connected.


Improved ValueError string

This is a small readability fix. When the path to a lookup table is incorrect, the ValueError raised has the string Could not load lookup table /some/incorrect/pathMake sure you've provided the correct path, fixed this to add a whitespace Could not load lookup table /some/incorrect/path Make sure you've provided the correct path
Proposed changes:

small readability fix

Status (please check what you already did):

[ ] added some tests for the functionality
[ ] updated the documentation
[ ] updated the changelog (please check changelog for instructions)
[ ] reformat files using black (please check Readme for instructions)

@erohmensing can you check it now?

Looks great. Thanks!


Add getLoanInfo rpc

What kind of PR is this?:

/kind feature
What this PR does / why we need it:
Partial #488 implementation
Which issue(s) does this PR fixes?:

Fixes #
Additional comments?:
Partial of this rpc execution on defid is actually same as #571

You probably should add another test unit
should getLoanInfo - include collateralValueUSD and loanValueUSD

Some of your tests failed if the docker image is updated to the latest version

Some of your tests failed if the docker image is updated to the latest version

Was already fixed since last week 1cb952d, maybe some cache.

You may need to resolve conflicts. Also for your info,

minColRatio and interest rate have negative correlation
The higher the minColRatio, the lower the interest rate


Surfaces and sensors not showing in coregistration/alignment visualization

Describe the bug
Replace this text with a description of the bug.
Using mne.viz.plot_alignment() (as e.g. in tutorial 20_source_alignment.html only shows landmarks but not surfaces or sensors.
Version 1.0.3 under Linux (Centos 7).
Steps to reproduce
tutorial 20_source_alignment.html
Expected results
Plots as in tutorial https://mne.tools/stable/auto_tutorials/forward/20_source_alignment.html.
Actual results

Additional information
Platform:         Linux-3.10.0-1160.el7.x86_64-x86_64-with-glibc2.17
Python:           3.10.4 | packaged by conda-forge | (main, Mar 24 2022, 17:38:57) [GCC 10.3.0]
Executable:       /imaging/local/software/mne_python/mne1.0.3_0/bin/python
CPU:              x86_64: 20 cores
Memory:           251.7 GB
mne:              1.0.3
numpy:            1.21.6 {blas=NO_ATLAS_INFO, lapack=lapack}
scipy:            1.8.0
matplotlib:       3.5.2 {backend=QtAgg}
sklearn:          1.0.2
numba:            0.55.1
nibabel:          3.2.2
nilearn:          0.9.1
dipy:             1.5.0
cupy:             Not found
pandas:           1.4.2
pyvista:          0.34.0 {OpenGL 3.3 (Core Profile) Mesa 18.3.4 via llvmpipe (LLVM 7.0, 256 bits)}
pyvistaqt:        0.9.0
ipyvtklink:       0.2.2
vtk:              9.1.0
PyQt5:            5.12.3
ipympl:           0.9.1
pooch:            v1.6.0
mne_bids:         0.10
mne_nirs:         0.2.1
mne_features:     0.2
mne_qt_browser:   0.3.1
mne_connectivity: 0.3

I'm getting the same behavior on a documentation build on github action, but not running old versions of MESA. Relevant part of a sys_info:
Platform:                 Linux-6.5.0-1018-azure-x86_64-with-glibc2.35
Python:                   3.9.19 (main, Mar 20 2024, 16:40:02)  [GCC 11.4.0]
Executable:               /opt/hostedtoolcache/Python/3.9.19/x64/bin/python
CPU:                      x86_64
Physical cores:           2
Logical cores:            4
RAM:                      15.6 GB
SWAP:                     4.0 GB

Core dependencies
✔ mne (>=1.7):            1.7.0
✔ numpy (>=1.21):         1.26.4

Optional 'doc' dependencies
✔ matplotlib:             3.8.4 (backend: QtAgg)
✔ PyQt6:                  6.7.0
✔ pyvista:                0.43.7 (OpenGL 4.5 (Core Profile) Mesa 23.2.1-1ubuntu3.1~22.04.2 via llvmpipe (LLVM 15.0.7, 256 bits))
✔ pyvistaqt:              0.11.0
✔ sphinx (!=7.2.*):       7.3.7
✔ sphinx-gallery:         0.16.0

The associated GA action setups a headless display with pyvista/setup-headless-display-action@main.
I'm going to try to change the runner, reporting in case you have an idea to spare me some headaches :exploding_head:

Can you see if any of these help?
https://mne.tools/dev/install/advanced.html#off-screen-rendering-with-mesa
In particular something like
export MNE_3D_OPTION_MULTI_SAMPLES=1
export MNE_3D_OPTION_ANTIALIAS=false

might fix it

Thank you @larsoner, we have a winner LIBGL_ALWAYS_SOFTWARE: true does the trick.
I'm not very familiar with 3D rendering, but should this variable be part of the variables defined by pyvista/setup-headless-display-action@main, namely DISPLAY: :99.0 and PYVISTA_OFF_SCREEN: true? Or is there a downside for some workflows in settings this environment variable on github runners?

Actually MNE_3D_OPTION_MULTI_SAMPLES: 1 is also needed, I probably got baited by some cache on my browser.


Tile 4215 marked as unsure: lots of flagged fibers on P4.

Eddie says "It doesn't have typical defocus behavior, but it isn't really plausible that P4 has only 0.8 throughput."
Anand says "for what is worth, it looks like it s the first half of the fibers which are more affected by that."

@julienguy , mind taking a look at this?  I'm surprised by the low THRUFRAC on P4.  It did not look to me like an obvious defocus case.  The turbulence is a bit odd but affects ~half the focal plane and doesn't look so bad to me.  I'm not sure what's up.

This looks to be one of the cases where it's the first tile after the slew to the SGC, following the upgrade to the fast sequencer.  I think I thought this was fixed but I am not sure I've verified that.

Judging by my past emails, the fix was implemented on 6/12, and we only had one very bad night's observing before the shut down after that.  We clearly still had fast sequencer & long slew issues on 6/11
https://data.desi.lbl.gov/desi/spectro/redux/daily/nightqa/20220611/tileqa-20220611.pdf
We need to check this again after restart.  This is in principle something we could test before the spectrographs come on, but after the focal plane is ready?

I've marked this tile as good and am closing, those we still need to check that the fast sequencer is now working well for long slews.  I think the fix was implemented and involved waiting for the ADC (which is fast), but my memory is hazy.


[Task] update one last pass on content consistency

We need to update with the latest content changes from Figma.
Current behavior

old content is on main

Expected behavior

new content on main

Consistency pass completed with design help.