lucianmarin.com

What is the URL of the website or web application you want to add?
https://lucianmarin.com/
What is the total size of data transferred while loading your URL in a web browser? (See club rules > point 1 to figure how to measure the data transfer size.)
6.1 KB
Provide a link to a discussion on Reddit or Hacker News where some content from your website received at least 100 upvotes.
https://news.ycombinator.com/item?id=32445450
https://news.ycombinator.com/item?id=28628011

When I load the URL https://lucianmarin.com/ in Firefox, I see a compressed transfer size of 42 kB. The bulk of the transfer size is due to .woff2 files. Here is the metrics report from the metrics script in this project:
$ node src/metrics.js https://lucianmarin.com/
url: https://lucianmarin.com/
htmlSize: 1853
jsonSize: 0
imageSize: 0
base64Size: 0
videoSize: 0
cssSize: 896
jsSize: 0
webfontSize: 38227
otherSize: 0
contentSize: 1853
extraSize: 39123
totalSize: 40976

total size: 40.0 KB
content size: 1.8 KB
content ratio: 5%

I am afraid I will not be adding this website to the club list at this time due to large transfer size. However, if the compressed transfer size becomes less than 10240 bytes in future, please reopen this ticket and I will add this website to the club list.


Allow specifying the caller in set_code function

What does it do?
Adds a new caller parameter to set_code function. This change, allows to manage several behaviors inside each custom implementation of set_code, like applying a filter to control which addresses can deploy a contract via inner calls of type CALL(CREATE).
To give more context, in Tanssi and our corresponding forks (of both Frontier and evm), we have implemented such use case. More specifically we have two filters implemented in Frontier:

CreateOrigin: this filter acts when an account is trying to deploy a contract via CREATE method, which is the direct way of creating contracts. In this case, we don't need the caller to be present in set_code() as we control the CREATE deployments in pallet-evm directly.

CreateInnerOrigin: this filter is applied when an account is trying to deploy a contract via a CALL method, that internally contains a CREATE. In this case, we need the caller to be present in set_code(), as this creation will take place inside the substack, and we can't control that behavior from pallet-evm.

Also, a good aspect of this change, is that it doesn't break any existing behavior nor compatibility inside the evm.
Implementation PRs (Tanssi and forks)
[Tanssi] Contract deployment customization in frontier template: https://github.com/moondance-labs/tanssi/pull/571
[Frontier fork] Customize origin for contract deployment: https://github.com/moondance-labs/frontier/pull/1
[evm fork] Modify set_code to make it fallible: https://github.com/moondance-labs/evm/pull/1

@koushiro @sorpaas what do you think about this? it allows us to filter addresses that are able to deploy contracts with just a very small change which should not have any effect in the rest of the codebase


HDX-10119 Add "scan" functionality for analytical and maintenance operations on all of HDX

Purpose
Version for this PR: 2024.8.2
The aim of this PR is to provide functionality for actions on all datasets in HDX. This uses the package_search endpoint in CKAN directly to download the whole catalogue. The actions supported are survey which counts occurrences of a key, distribution which calculates the distribution of values in a key, list which lists the values of a key for each dataset (like the existing list command)  and delete_key which deletes a selected key. This final action was the original purpose of the update - to remove resources._csrf_token keys, this functionality is limited to only accept the _csrf_token and extras keys as arguments.

HDX-10119 - Detect and remove the "_csrf_token" at resource level.

Major file changes
This PR adds the ckan_utilities.py and test_ckan_utilities.py files which implement the scan functionality. The DEMO.md file is renamed to USERGUIDE.md and updated to include the new command, and reorganised to reflect the more mature status of the project.
Minor file changes
tests/test_hdx_utilities_integration.py appears to have changed a lot but this is just a result of a change in line endings configuration. The new content in this file is the test function test_get_hdx_url_and_key, which tests the new get_hdx_url_and_key function.
Versioning
hdx-cli-toolkit uses the CalVer versioning scheme with format YYYY.MM.Micro i.e. 2022.12.1 which is updated manually in pyproject.toml. The "Micro" component is simply an integer increased by 1 at each version, starting from 0.

[x] Version updated in pyproject.toml and PR description
[x] Update README.md and DEMO.md with any new CLI commands

@turnerm I added a list action to scan which replicates the list command but applies by default to all of HDX. It is more performant than I expected <10s to show the values of 2 keys for all datasets (based on a cached version of the data). I expect it will run in >1s for you ;-)


Hawkular Sink: Support trusting specific certificate

Currently we are running into an issue where Heapster connects to Hawkular Metrics via its service hostname (eg 'hawkular-metrics') but a user provided certificate may not contain 'hawkular-metrics' as part of the certificate's san.
In this situation the connection will be refused.
We already pass the certificate used by Hawkular Metrics to the Hawkular Sink, and we should just trust this certificate for connections to the 'hawkular-metrics' hostname, regardless if the certificate actually contains 'hawkular-metrics'.

Adding that is an option sounds ok, but should not be the default: if users have gone to the trouble of configuring TLS then we should adhere to proper TLS handshake/certificate validation.

It wont be by default. The user would have to explicitly configure the sink to trust very specific certificates.
Looking at the current options, we don't actually pass the server certificate but the CA certificate. As part of this we will need to add a new option to specify the server certificate to trust.

Passing a server certificate seems odd. If you have a CA then you should give the CA certificate. If you are using self-signed server certificates then that also acts as the CA for the server certificate (their one & the same thing) so we don't need to add an option for specifying a server certificate.
Even so, the only way to trust a badly issued certificate (e.g. one that has a hostname that isn't the same as the actual hostname used to connect to the service) is to disable TLS verification completely: it's pretty worthless anyway in this case. That is already supported.
I'm not sure anything needs to be done here.


Support connected services

In GitLab by @EpicWink on Feb 7, 2019, 06:07
Support connected services

Include synchronicity
Include output handling?
Include error handling
Include parameter passing
Include waiting for task token

In GitLab by @EpicWink on May 9, 2019, 08:07
changed the description

In GitLab by @EpicWink on May 24, 2019, 10:36
changed the description


Use custom template

Hello, I wrote a very basic CSV template for trivy and I would like to use it with the trivy GitHub action.

hi @nleconte-csgroup - that's cool, what were you looking to do with it?

Hey @nleconte-csgroup, I ran into this issue on Google while I was trying to do the same thing. I know your post is a year old, but I found the solution in case anyone else finds this thread looking for it.
So it turns out that the Trivy scanner from aquasecurity/trivy-action runs inside a docker instance within your job. It actually mounts your checked out repository from actions/checkout as a volume inside the container that runs the scanner. The mapping looks like this in Trivy's workflow debug log:
-v "/home/runner/work/YOUR/REPOSITORY":"/github/workspace"

This means that if you want to give the scanner a template file, you have to use the container's internal filesystem structure, not the normal runner filesystem structure.

So, for example you had a Trivy remplate file in the root of your repository called template.tpl. Your template config lines in your Trivy action step should look like this:
format: 'template'
template<EMAIL_ADDRESS>
This is certainly not intuitive whatsoever and probably not the best design structure for this config flag, but this is the way to do it. Hope this helps!

@neonspectra Thanks for this, I was able to do it with:
format: template
template<EMAIL_ADDRESS>

@AshwinSarimin would you be so kind to share the markdown.tpl? 🙏


Dropout: make seed and states_num kernel arguments

Every time a dropout descriptor is created (even if it's with a different seed), the initialization kernel is recompiled. This PR turns the number of states to initialize and random seed into runtime parameters. There seems to be no effect on the runtime of the initialization kernel (which I'm assuming is not performance critical), but fewer JIT compilation calls.
Note: I tried setting prng_seed to be an unsigned long long parameter, but that passed a wrong value onto the kernel itself.

@atamazov could you help to take a look?

@JehandadKhan sorry, called clang-format on the code now.

@tbennun  The CL file needs to formatted as well.

@JehandadKhan Done now. I thought I had formatted both.

@tbennun could you resolve the review opinions above? We will try to get this merged.

@junliume I now addressed the two comments, see above and new commit.

@atamazov could you comment on the user's question?

The private_seg_size of 3204 seems to be caused by the precomputed XORWOW matrices, which are placed in OpenCL constant memory and, upon compilation, may be stored in every thread's memory. It might be useful to store one or both matrices in global memory to avoid the private_seg_size pollution.

CC: @zjing14

@junliume I need some clue to the code as I am not confident with dropout. What are the symbolic names of the "the precomputed XORWOW matrices"?
/cc @zjing14

@junliume I need some clue to the code as I am not confident with dropout. What are the symbolic names of the "the precomputed XORWOW matrices"?
/cc @zjing14

@atamazov This refers to the two included files in the Dropout kernel, which initialize large arrays in constant memory:
https://github.com/ROCmSoftwarePlatform/MIOpen/blob/develop/src/kernels/precalc_xorwow_skipahead_sequence_matrices_kernel.h
https://github.com/ROCmSoftwarePlatform/MIOpen/blob/develop/src/kernels/precalc_xorwow_skipahead_matrices_kernel.h

@tbennun @junliume AFAICS the parts of these matrices are copied to the private memory for a purpose, i.e. each workitem modifies its local copy. If we want to put those into global memory, then we need to allocate some workspace (enough to hold data from matrices) and then manually manage it, which doesn't seem easy to me.
But why we may need to save space in private segment? Are we hitting the limit?
/cc @zjing14

@atamazov yes we are hitting a limit on constant memory, with this fix it "gives a reduction in initialization time of a Transformer model from 10:22 minutes to 31 seconds." We should take a look at the OCL code to see if we can work on heap memory instead.

@CAHEK7 Thanks for feedback. Now we at least better understand what to begin with.
/cc @JehandadKhan @junliume


Create minimum function and spec file

Created minimum function, spec file, and required function in index.js

Pull Request Test Coverage Report for Build 86

24 of 24 (100.0%) changed or added relevant lines in 2 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 100.0%

Totals

Change from base Build 84:
0.0%

Covered Lines:
690

Relevant Lines:
690

💛  - Coveralls


fix #693

The content cannot be read from the RequestBody
when CopyRequestBody == false

@akkuman we already set it:
https://github.com/casdoor/casdoor/blob/7b78e6026589bc2691eee0f2d34439b61ef4c4e1/conf/app.conf#L1-L7

but I think it should not display in app.conf, user could modify.
but when it is set to false, the related functionality may become unavailable.

@akkuman I don't think it's a big issue

I still stand by my opinion that this option should not be available to users in configuration files

However, I don't think it is a bug now, I just saw this problem in the QQ group, I searched the code, but I neglected the default configuration, it should not be a bug
I just don't think that configuration files should be used to control whether bugs occur or not, as that would confuse users


Cloning struct with unexported pointer fails

Repro:
package main

import clone "github.com/huandu/go-clone/generic"

type Foo string

type Bar struct {
	foo *Foo
}

func main() {
	foo := Foo("hello")

	test := Bar{
		foo: &foo,
	}

	clone.Clone(test)
}

https://go.dev/play/p/ieZSLWAQR40
Result: panic: reflect.Set: value of type string is not assignable to type main.Foo

Thanks for raising this issue to me. Just fix it.


Git flow support

@thiagotognoli I added git flow support to new 0.3.0 release. Please check out. And read README.

cool. thanks


Missing exported variables on first run

First run with "include docker" showed the following variables not exported from params.pp:
$storage_config
$service_config
$service_overrides_template
Had to add the following to manifests/params.pp for init to work:

$storage_config                    = undef
$service_config                    = undef
$service_overrides_template        = undef

Version 5.1.0

Hi @joeljacobs, could I check what OS you're running? Providing the output from facter would be really handy. The params file is fairly OS sensitive, my guess is you're using an OS I haven't tested the module with.

We're using ubuntu 14.04.
facter.txt

Tried newest master as well, and it still required adding those 3 lines to params.pp on Ubuntu 14.04


Interested in contributing in this project

Hello,
I m trying to use allure report for my flutter integration test project and I came across this project, it seems like something that I need and I am interested in working this project.
Thanks,
C

Hi! Glad to see you here :-)
How can I help you? Did you try it already or it is unclear how to setup it?
Allure Reporter has basic functionality.
It has tests, groups, features, stories, links and etc.
It is lack of metadata, like runner info and documentation
Also I see example report was not generated after last changes. Now it is fixed, you can check it on repository main page

Thanks for the reply. I m looking to do some html reporting for flutter integration test report. Currently we are using the junit xml and the default xml junit plugin from jenkins. But it is difficult to see as the test suites grow, so I want to look something like allure or some html test report, for example xunit viewer in node.


Question - Please advise how to create a dataset and train it

Hello,
I a few thousand face images from around the web.
Can you please explain (or point me to a doc / website) that will describe how I get from face image to the point that is required to run in your app. I assume being the "model".
I am new to this (retired 65yo IT guy of 44 years in coding) and would love to be able to lean more about this.
Thanks in advance.

Hello Krammig,
Basically you need to train the model on your own dataset , the model structure is included in the code , you just need to prepare your dataset and run the model on it.

Thanks for the reply on this.
And by dataset I presume you mean images ?
Cheers

Hey,
is everything working fine now?


Markdown Preview ReactApp

Prerequisites

[X] I am a GSSoC-22 Participant
[X] I checked to make sure that this issue has not already been filed

Description
Real Time .md text file editor
Tech Stack
React

@manisha-27  Please assign this task to me


TestFWCore Subdirectory with ExampleEventDataProducer

Adds a test and example how to use the I/O components in the Framework.

This moved to key4hep/k4-project-template.


Implement preliminary support for bezier patch subdivision surfaces

Related to godotengine/godot-proposals#784
See also the Wikipedia article on Bezier surfaces

GREEN: Original lowpoly quad model ||| YELLOW: Bezier patch control points after preprocessing ||| RED: Final tessellated model ||| BLUE: Catmull-Clark subdivision for comparison ||| WHITE: Quad/Bezier patch borders

Circle test. Note how bezier patch subdivision surfaces (RED) need fewer points than Catmull-Clark subdivision surfaces (BLUE) to create rounder circles.

Bevel test. Support loops work the same way for both surfaces.
Bezier patch vs Catmull-Clark pros:

Bezier patches are simpler to evaluate and their mathematical properties have been more thoroughly researched.
Points on the limit surface can easily be directly evaluated (this is in fact the standard way to evaluate bezier curves/patches), unlike Catmull-Clark which by default must be iteratively refined and shrinks with every iteration
Patches are completely independent of each other, making them friendlier towards a potential tessellation or mesh shader implementation
More flexibility for the preprocessing algorithm to decide where to place control points, without affecting the complexity of bezier patch evaluation during runtime.

Cons:

Bezier patches are not an industry standard subdivision surface. While this PR has made great efforts to ensure that traditional Catmull-Clark modeling rules work as expected (edges are sharpened with support loops, regular polygons result in circles, etc), they will not be able to match the curvature exactly. This becomes less of a problem as the model is subdivided to a higher density.
The added flexibility of allowing the preprocessing algorithm freedom to assign control points doubles as a fantastic footgun.
The preprocessing implementation in this PR only supports quads. Non-quad models will need to be pre-subdivided once in the DCC application to produce quads.

Contents of this PR:

GLTF meshes suffixed with "-experimental-subd" are now treated as subdivision surfaces and automatically converted on load. This was done to simplify this PR as it aims to strictly focus on the backend. Replacing this with a proper interface in the future is highly recommended.
Subdivision surfaces conversion is done via the following two-step process:

Preprocessing: Convert the input quad model into a list of bezier patches at load time. Control points are assigned based on neighboring faces to ensure G1 continuity between patch seams like with Catmull-Clark subdivision surfaces. Most of the complexity lies in this function, although this is in terms of comprehensibility. Performance should not be an issue.
Tessellation: Convert the bezier patch list into a standard triangle model, tessellating faces up to a target polycount (100,000 for now) and adding them as LODs. This is currently also done at load time. In the future, bezier patches can be sent directly to a tessellation or mesh shader for direct evaluation on the GPU, without needing to compute and store the intermediate highpoly triangle model, allowing for per-frame adaptive runtime tessellation and reducing VRAM and rasterizer usage.

Standardizes a quad representation where every six indices form a triangle pair with the following ordering:

This allows the rest of Godot's rendering system to remain blissfully quad-unaware. Note that I haven't checked thoroughly to see if quads won't be clobbered by mesh processing somewhere, but at least with the basic testing I've done so far, quads are preserved correctly from import to render.
Quad-based PrimitiveMeshes now always outputs in this format
The GLTF importer now attempts to detect quads if possible. This may or may not work depending on how the file was exported, but Blender seems to always place triangles in the same polygon next to each other.
Now preserves quads in OBJ files

Are you familiar with this encoding? https://github.com/KhronosGroup/glTF/pull/1620

Are you familiar with this encoding? KhronosGroup/glTF#1620

I went with Ultra Engine's proposal for quads since I think there's value in having a low level GL_QUADS-like primitive.
Ngons should probably be considered a (slightly) higher level feature to be implemented separately. Using meshlets for this purpose may be a promising direction. Unlike FB_ngon_encoding, meshlets can represent concave polygons and holes, although they have a different type of limitation where vertex/triangle counts are capped to 256.

Formal spec https://github.com/UltraEngine/glTF/blob/main/extensions/2.0/Vendor/ULTRA_primitive_quads/README.md @aaronfranke if youre interested

For the record I would like this, but need some review from @clayjohn and others.

Incidentally, I initially wrote this PR against Godot core for convenience sake, and to make it easier to investigate feasibility.
However, it's becoming evident that this mainly affects the importer, so it should be possible to move most of this into an official glTF-focused GDExtension and avoid impacting non-3D users of Godot, if this is desired.
Tessellation shaders will still need to involve core if they're ever implemented, though.

Assuming I'm reading this PR right, there's actually a bit of a use for bezier patches in core, hell quake 3 used them extensively. I was going to implement them myself for my own q3 style map system too.

Let me know when this is ready for merge review.

Hopefully this is adequate for the first pass, since it's all I can find the time for right now. Some functionality (such as bones) is still missing though, so only merge this if it's useful as-is (essentially static meshes only) and it's acceptable for the missing features to be added later.

What do I need to get bones functionality?

quads_to_bezier_patch() and tessellate_bezier_patch() will need to be extended to handle Mesh::ARRAY_BONES. Unlike the other attributes, Mesh::ARRAY_BONES is made up of discrete integer IDs, so they will need to be copied from the nearest quad point/bezier control point (essentially, nearest neighbor interpolation rather than linear interpolation).

I haven’t had any rendering team approvals, but I can look into bones. Since theres a lack of rendering team approvals, the probability of merge is lower.

I salvaged the branch here https://github.com/V-Sekai/godot/tree/quad-bezier-surfaces as I am still interested in this.


lolcat doesn't seem to play nice with '\b' (backspace) or "\33[2K\r" (clear console line)

The '\b' character is often used for backspace and "\33[2K\r" is used to clear the console line on unix machines. Neither of which lolcat seems to play nicely with. It would be awesome if you were able to update lolcat so it does..

Yeah I was hoping to make all my manpages look nice with something like:
$export MANPAGER='bash -c "lolcat -f \"\$@\" | less -R" PAGER'
But I just start seeing all the ^H control codes. Meh.

I might have you sorted.. I ended up making my own version of lolcat which is ~4 times faster than the c (old world's fastest). I haven't release it yet (will in a few days), but it's in-built in to Nift (https://nift.dev). Eg. install Nift then nift lolcat-cc for lolcat-cc command (can even do nift sh then lolcat.on and you will get rainbow colours for output by default :).

@n-ham @tbooth
lolcat tries to deal with control sequences to a degree
but it's tricky and it doesn't always succeed.
This may be improved in the future (PRs always welcome 😬).
In most cases you can help it a little by stripping the
problematic sequences before feeding the input to lolcat.
For man-pages you can use this:
export MANPAGER='bash -c "col -bx | lolcat -f \"\$@\" | less -R" PAGER'

HERE BE DRAGONS. another trick is bash | lolcat (NO WARRANTIES)
that works ok good with mc, but not so much with modplug123, nor ocp


Canvas and CORS

I have stumbled onto a problem in Development only.
I am loading images provided by Webpack in development onto img src tags and then trying to use the images data (getImageData) on a Canvas element. When I do this, I get a complaint that the canvas is tainted by cross origin data.
Does anybody know how I can get around this in development? It all works out fine on a final build, but kind of annoying that I have to produce a build just to test my work!

looks like img.crossOrigin = 'anonymous' works for this actually!


add redux-devtools

This thing allows you to see in real time all the actions that are taking place in our store, all changes, time travel and much more:

Good idea.
@koretskiyav Since it's a new dependency, it needs to be added to LICENSE file
@amiryonatan Can we make sure this has no effect on production builds?

as far as I see the license is identical to that of react

as far as I see the license is identical to that of react

@koretskiyav
It's still have to be mentioned here - https://github.com/wix/repluggable/blob/master/docs/3rd-party-licenses.md

as far as I see the license is identical to that of react

@koretskiyav
It's still have to be mentioned here - https://github.com/wix/repluggable/blob/master/docs/3rd-party-licenses.md

@itsh01 Ah, I got it! Added license to document!


Should docs reflect that Maybe type satisfies Ord spec?

Currently maybe doc is not saying that maybe implements Ord spec, but it has .lte method.

Good catch. Would you like to open a pull request to correct this? If so, please also update the documentation for the Either type.

@davidchambers done
Replaced Setoid by Ord spec.

Fixed in #422.


Unknown command?

I am designing my own smartwatch and wanted to use your Chronos app to connect to it. Everything is going well but I sometimes receive a command 0x8a (0xab,0x00,0x07,0xff,0x8a,0x80,0x00,0x03,0xee,0x00) which seems to be related to weather updates. I have not found this command in any documentation in your repositories. Could you help me out with this command?
Thanks!
PS: I posted this same message by mistake in another issue... Sorry...

It is UV and Air Pressure data used on some WearFit watches, I will add it to the library.
uv = data[6]; // (0x00) -> 0
airPressure = (data[7] * 256) + data[8]; // (0x03, 0xEE) -> 1006hPa

Thank you and great work on your app!


Add Support for ReplyTo

Thanks for contributing this! Adding the ReplyTo feature seems to make much sense to me.
However, in what way is ReplyTo directly related to the addressing with +? I guess, the addressing with plus does work for Cc, Bcc and To? And you want people who respond to reach your address with +? Did you try if directly sending From an address with plus does fail?
Just asking out of curiosity, adding ReplyTo makes sense either way.

I didn't test that to be honest, I just implemented as per the postmark docs.
ReplyTo will override the default which I believe comes from what is set in your postmark sender signature.
If you are using inbound mail from Postmark and you use plus addressing in your outbound then on the inbound postmark will pull out the plus value for you, making it easy to link messages with threads in a system for example.

Just gave it a try, both works, mails are reaching my gmail.

v1.1.11 is live on NuGet now.

OK cool, good to know.
Thanks for the super fast turn around!


Feature request: Add validator for custom hashes

Hello!
Thank you for this great library!
However, I would propose to introduce generic validator for different types of hashes. Right now, we have isMD5() validator, which is great, but there are no validators for other hash types, like sha256.
I would suggest the following API:isHash(str, algorithm).
We could call it like this: isHash('6b86b273ff34fce19d6b804eff5a3f5747ada4eaa22f1d49c01e52ddb7875b4b', 'sha256').
Here's the list of most popular hash functions (algorithms): http://stackoverflow.com/q/16393650/1056679.
The original isMD5() could be deprecated over time in favor of isHash(str, 'md5').
What do you think?
Thanks!

Great idea.
It could be something simple like:
var lengths = {
  md5: 32,
  sha1: 40,
  sha256: 64,
  // etc
}

function isHash(str, algorithm) {
  return validator.isHexadecimal(str) && str.length === lengths[algorithm];
}

Fixed by #711.


Update autoassign gh action and remove deprecated actions

no need to mentions anymore? 🙂

nope since it automatically add reviewers, and also it caused issues in forked repos :)


GenomeComparison does not allow users to compare one ref and one external genomes

The submit button is grayed out in the screenshot:

Currently "Add Up to 8 Genomes" selection is required.
From: "Fangfang Xia"<EMAIL_ADDRESS>To: "PATRIC3/patric3_website"<EMAIL_ADDRESS>Sent: Tuesday, June 16, 2015 5:32:38 PM
Subject: [patric3_website] GenomeComparison does not allow users to compare one ref and one external genomes (#306)
The submit button is grayed out in the screenshot:
—
Reply to this email directly or view it on GitHub .
--
Chunhong Mao, Ph. D.
Senior Project Associate
Virginia Bioinformatics Institute
Virginia Tech (0477)
1015 Life Science Circle
Blacksburg, VA 24061
Tel:<PHONE_NUMBER>
Fax:<PHONE_NUMBER>

Giving to FangFang as he appears to own the backend code; is this a restriction there or on the GUI? If the GUI go ahead and reassign to @maovt since it looks like she wrote the gui.
(It seems to me one should be able to compare an external fasta to a reference genome)

I'll work on this.
Chunhong
From: "olsonanl"<EMAIL_ADDRESS>To: "PATRIC3/patric3_website"<EMAIL_ADDRESS>Cc: "Chunhong Mao"<EMAIL_ADDRESS>Sent: Tuesday, August 4, 2015 3:35:43 PM
Subject: Re: [patric3_website] GenomeComparison does not allow users to compare one ref and one external genomes (#306)
Giving to FangFang as he appears to own the backend code; is this a restriction there or on the GUI? If the GUI go ahead and reassign to @maovt since it looks like she wrote the gui.
(It seems to me one should be able to compare an external fasta to a reference genome)
—
Reply to this email directly or view it on GitHub .
--
Chunhong Mao, Ph. D.
Senior Project Associate
Virginia Bioinformatics Institute
Virginia Tech (0477)
1015 Life Science Circle
Blacksburg, VA 24061
Tel:<PHONE_NUMBER>
Fax:<PHONE_NUMBER>

@maovt Chunhong, any update on this?

Fixed


To do for next release with Gammapy v1.3 support

🚀 The feature, motivation and pitch
Additions/Modifications -

[x] Internal code for estimation of EBL-deabsorbed spectra/model
[ ] Maybe fix #175
[x] Resolve #170
[ ] Check for reverting #182
[x]  Update to astropy =7 or < 8.0
[x] Update lines in code for FoVBackgroundModel
[x] Check the usage of the new FitResult object - Serialization being combined with those of Models
[x] Create a dedicated documentation page to list the distinct features of asgardpy in comparison to the latest stable release of gammapy
[x] The default spectral model's index is set at 0. Fix in the test here
[ ] #187
[x] #188
[x] Support python 3.13 and maybe remove 3.11?
[x] Check the serialisation efforts on each type of Dataset. See example of 3D dataset here and maybe try generalizing or including serialisation as yaml files as another option. Have to modify code here.
[x] Include the workaround for reading Models file created with Gammapy v <1.2 which includes TemplateSpatialModel - See Issue 5549
[ ] #195

Removal -

[x] Additional dataset of PKS 2155-304 (From #174)
[x] Additional dataset of LST1 DL4 dataset (From #184) Not included in Gammapy v1.3

Alternatives
No response
Additional context
No response

Compare with the new update on Gammapy HLI from PR 3852 to update https://asgardpy.readthedocs.io/en/latest/need_for_asgardpy.html

The workaround required to read TemplateSpatialModel objects written with Gammapy <1.2 is not essential to be included in asgardpy, as it only creates such objects with the base Gammapy, and does not read an existing object.


enable find in cell output when output is a data stream

This PR is an initial step to enable find in cell output. Changes here enable find when the cell output is a stream.
As part of this, I refactored the addDecorations/removeDecorations code from text cell component to its abstract parent class 'CellView' so that it can be reused by output component.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

10 of 139 (7.19%)  changed or added relevant lines in 9 files are covered.
4 unchanged lines in 3 files lost coverage.
Overall coverage decreased (-0.05%) to 41.61%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/sql/workbench/contrib/notebook/browser/cellViews/textCell.component.ts
1
2
50.0%

src/sql/workbench/contrib/notebook/browser/cellViews/codeCell.component.ts
1
4
25.0%

src/sql/workbench/contrib/notebook/browser/cellViews/output.component.ts
2
6
33.33%

src/sql/workbench/contrib/notebook/browser/cellViews/outputArea.component.ts
2
7
28.57%

src/sql/workbench/contrib/notebook/browser/find/notebookFindDecorations.ts
0
5
0.0%

src/sql/workbench/contrib/notebook/browser/notebook.component.ts
0
14
0.0%

src/sql/workbench/contrib/notebook/browser/find/notebookFindModel.ts
1
29
3.45%

src/sql/workbench/contrib/notebook/browser/cellViews/interfaces.ts
2
71
2.82%

Files with Coverage Reduction
New Missed Lines
%

src/sql/workbench/contrib/notebook/browser/cellViews/code.component.ts
1
4.11%

src/sql/workbench/contrib/notebook/browser/cellViews/textCell.component.ts
1
5.8%

src/sql/workbench/contrib/notebook/browser/notebook.component.ts
2
2.1%

Totals

Change from base Build<PHONE_NUMBER>:
-0.05%

Covered Lines:
27182

Relevant Lines:
59266

💛  - Coveralls

Hey Maddy, while testing I noticed that when I search for "hello", "print" also gets highlighted in the cell below:

This happens for new untitled notebooks as well as if I add this cell to an existing notebook, "print" always gets highlighted when searching for "hello".

Hey Maddy, while testing I noticed that when I search for "hello", "print" also gets highlighted in the cell below:
This happens for new untitled notebooks as well as if I add this cell to an existing notebook, "print" always gets highlighted when searching for "hello".

I can repro it too, I guess I regressed something during clean up, thanks for catching this, will look into it.

tests? 😄

Tested and find in cell output works overall but I did notice one bug: The highlight for the second html mime output never gets removed. Repro steps:

Run SQL-WorldWideImport.ipynb
Search for "total"
Then search for something else
The original highlight for "time" does not get removed.

Screen.Recording.2021-12-20.at.8.31.39.AM.mov

Thanks Lucy for testing this, I fixed it now. Appreciate your help!


Enable honey pot?

Great plugin! 😎
In Mailchimp form builder output, there's a field that looks something like:
<div style="position: absolute; left: -5000px;" aria-hidden="true">
  <input type="text" name="b_1a45e336541f2uhe6ef351e00_f58763f205" tabindex="-1" value="" />
</div>

Is there a way to make this work through your plugin?
Thanks!!

this seems like a good idea. open for PRs or suggestions!


chore: add prepublishOnly script

This can be tested with npm publish --dry-run.

Both would work. Prepare would also invoke post install and post link.