Geometric primitives can't be cimported

I'd like to be able to do the following:
from raysect.primitive cimport Mesh, Cylinder, Cone, Intersect, Subtract, Union
so that I can instantiate many of these primitives in a loop without having to go through the python layer every time. However, I can't find an __init__.pxd where these primitives are defined. They're all cdef class so I should in theory be able to cminport them if an __init__.pxd was provided. Is there a reason this hasn't already been done?

This is definitely an Alex issue. Will let him comment here before I try to do anything.

I've added the missing pxd files, they will be available in the next release.
I've also given some thought to adding the new_* methods we discussed in person. I'm not going to add them at this stage as it will require significant refactoring of the initialisation of each primitive. It would be best to defer this until the next major release as it might result in some minor changes to the initialisation routines in python. I'm not comfortable making the change in a minor release.


Rayleigh, Normal, and Uniform distributions are not supported in the SDK

As a user I would like to be able to use the Uniform, Normal, and Rayleigh distributions in Factors.
This is needed in the Mixture factor in the ICRA Tutorial 2 example.

Which packages contains these factors? RoME, IIF, something else?

Uniform, Normal, Rayleigh, AMP.ManifoldKernelDensity etc are Distributions which for which serialization is mostly held in IIF.
Factors are Pose2Pose2, PriorPose2, etc.

IIF.Mixture currently has inverse API than what it should have, so I'm toying with the idea of fixing that before we hack at the Data model Sam mentioned.

JuliaRobotics/Caesar.jl#808


BLD: should we use tempita for cython templating

see here
might replace some of the manual code paths in generate_code.py

@gfyoung can you comment on your experience

+1 from me - it's currently used in scipy and numpy and I in fact put it to good use in one of my PR's against numpy in fact.  Also, since tempita comes with Cython (Cython.Tempita), if it's possible to use it in order to cut down on manual code, certainly!

Quite interesting. I'm just adding new cython templates to support more sparse dtypes, and it can be much simpler using it.

@gfyoung can u post a link to the PR where u r using this?

Link to PR here.  Key file to examine is randint_helpers.pxi.in and the templating code here
Useful discussion points regarding Tempita in my PR start here

thanks @gfyoung
note that I have found fused types pretty useful as well. here is an example of a numeric type which is used in window.pyx & some of the algos.pyx. These are pretty easy to define and work with; and somewhat remove the need for templating.

@jreback : yes, I remember you mentioning that in that same PR <a href=https://github.com/numpy/numpy/pull/6938#discussion_r64143615">here.  I guess not all maintainers are a fan of it though.

right they are in newer cython (though our min 0.19.1 certainly has it)

well different repository, different rules :smile: - fused_types may not be as applicable in this case though given that there are some things in the template to generate_code.py that aren't fuse-type-able FLOABW.

@gfyoung yeah, though if I could use a fused type I think we should. Just easier to read/maintain. They seem to act very much like 'real' types, with a tiny exception. You need to this to avoid warnings when compiling.
cdef numeric val

if numeric in cython.floating and val == val:
     # do something if we are not-NaN

I agree that fused types are nice, but in my experience they end up falling short. Tempita looks pretty nice, better than our ad-hoc current system.


meteor-counter-cache incompatible with Meteor 1.1?

Hello!
First of all I want to thank you for creating & sharing this really useful package with Meteor community.
After upgrading to Meteor version 1.1 I got this exception:
W20150401-13:07:15.059(3)? (STDERR) The package mrt:collection-hooks at 0.6.7 is incompatible with Meteor 0.9.0 or later.
W20150401-13:07:15.060(3)? (STDERR) If a new, compatible version of this package exists, running 'meteor update' should cause you to update.
W20150401-13:07:15.715(3)? (STDERR)
W20150401-13:07:15.715(3)? (STDERR) /home/hakki/.meteor/packages/meteor-tool/.<IP_ADDRESS>wce6u++os.linux.x86_64+web.browser+web.cordova/mt-os.linux.x86_64/dev_bundle/server-lib/node_modules/fibers/future.js:245
W20150401-13:07:15.715(3)? (STDERR) 						throw(ex);
W20150401-13:07:15.715(3)? (STDERR) 						      ^
W20150401-13:07:15.721(3)? (STDERR) TypeError: Cannot call method 'insert' of undefined
W20150401-13:07:15.721(3)? (STDERR)     at [object Object].Mongo.Collection.maintainCountOf (packages/dburles:counter-cache/counter-cache.js:54:1)
W20150401-13:07:15.721(3)? (STDERR)     at app/server/startup/init/counter.js:3:10

Any ideas what is broken?
EDIT: I got it working by replacing<EMAIL_ADDRESS>with<EMAIL_ADDRESS>

Forked version: https://github.com/hazio/meteor-counter-cache

Hey @hazio feel free to send us a PR.


Why not provide a window size limit property in AppWindow?

Describe the bug
Why not provide a window size limit property in AppWindow?
为什么不在 AppWindow 提供窗口大小限制属性？
Since WinUI 3 is widely used in desktop applications, in some scenarios, we need to control the maximum and minimum value of the application window to meet special needs. So hopefully the Windows App SDK team or WinUI team will consider adding the MinWidth MinHeight MaxWidth MaxHeight property to AppWindow.
由于 WinUI 3 目前广泛应用于桌面应用，在一些场景中，我们需要控制应用窗口的最大最小值以满足特殊的需求，所以希望 Windows App SDK 团队或 WinUI 团队能考虑在 AppWindow 中添加上 MinWidth MinHeight MaxWidth MaxHeight 属性。
Steps to reproduce the bug
See describe
Expected behavior
Provide MinWidth MinHeight MaxWidth MaxHeight in AppWindow
在 AppWindow 中提供 MinWidth MinHeight MaxWidth MaxHeight 属性
Screenshots
None
NuGet package version
WinUI 3 - Windows App SDK 1.6.0: 1.6.240829007
Windows version
Windows 11 (22H2): Build 22621
Additional context
None

Related to #2731.
Also see microsoft/WindowsAppSDK#4710.

Related to #2731.
Also see microsoft/WindowsAppSDK#4710.

Thank you very much


Support copyparams to uninitialized parameters

link_a.copyparams(link_b) skips uninitialized parameters of link_a, even if the corresponding parameters of link_b are already initialized. With this PR, these parameters are also copied to link_a.

LGTM


Feature/keeper script

Simple script designed to be run with cron/cloud funcs

For now, high-level feedback: IME the biggest danger/pain – the only thing that's really an issue if it doesn't work – is status/error reporting. What worked well for me is Telegram Bot API, it's super simple and never failed on me. I'd do this in addition to whatever GCF are offering for monitoring.
def alert(msg: str, alert=False):
    # https://api.telegram.org/bot[BOT_API_KEY]/sendMessage?chat_id=[MY_CHANNEL_NAME]&text=[MY_MESSAGE_TEXT]
    if alert:
        chat_id = alert_chat_id
    else:
        chat_id = status_chat_id
    params = {
        'chat_id': chat_id,
        'text': msg,
    }
    logger.info(msg)
    res = requests.get("https://api.telegram.org/bot" + BOT_API_KEY + "/sendMessage", params=params)
    logger.debug(res.text)
    res.raise_for_status()

I'd go as far as this
except Exception as e:
        # https://stackoverflow.com/a/54083435/2997134
        full_traceback = "".join(traceback.format_exception(e, value=e, tb=e.__traceback__))
        alert("Exiting, encountered fatal error: " + str(e) + ". \n" + full_traceback, True)

Isn't there some functionality on g funcs to send a notification if stderr returns something? Like cron with MAILTO(i think)?

Don't know 🤷

Also, isn't there some functionality on g funcs to send a notification if stderr returns something? Like cron with MAILTO(i think)?

GCP suggests using this https://github.com/GoogleCloudPlatform/python-docs-samples/blob/HEAD/functions/helloworld/main.py#L190-L204
But that only stores the error in their error reports, so maybe combination of this and telegram BOT?

@DaveVodrazka Looks like the setup will be:
Setup bude Cloud Scheduler -> Cloud Functions -> Cloud Logging and from there: https://cloud.google.com/monitoring/support/notification-options#mobile-app
(Alongside Telegram)


Command: refactor to remove configuration global variable

Fixes #2017
Signed-off-by: Brandon Forster<EMAIL_ADDRESS>

Codecov Report

Merging #2118 into master will increase coverage by 0.05%.
The diff coverage is 0%.

@@            Coverage Diff             @@
##           master    #2118      +/-   ##
==========================================
+ Coverage   41.76%   41.82%   +0.05%
==========================================
  Files         184      183       -1
  Lines       12151    12134      -17
==========================================
  Hits         5075     5075
+ Misses       6779     6762      -17
  Partials      297      297

Impacted Files
Coverage Δ

internal/core/command/router.go
0% <0%> (ø)
:arrow_up:

internal/core/command/restDevice.go
0% <0%> (ø)
:arrow_up:

cmd/core-command/main.go
0% <0%> (ø)
:arrow_up:

internal/core/command/init.go
0% <0%> (ø)
:arrow_up:

internal/core/command/device.go
31.3% <0%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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


How does cargo-auditable construct the information that it includes in the binary?

Hi!
Your README makes the following claim:
"""
Know the exact crate versions used to build your Rust executable. Audit binaries for known bugs or security vulnerabilities in production, at scale, with zero bookkeeping.
"""
So, this should include all dependencies used (as opposed to only those statically linked, which would be a subset of those used). An example would be a procedural macro, which only generates code, but is not linked into an executable.
My question is, does this actually include this information, and if so, how does it gather the information?
Also,  given the answer to the first question, how probable is it that a particular package that was used would be omitted from the cargo-auditable generated information in the executable?

It uses cargo metadata: https://github.com/rust-secure-code/cargo-auditable/blob/da85607fb1a09435d77288ccf05a92b2e8ec3f71/cargo-auditable/src/collect_audit_data.rs#L18 cargo metadata has an issue where it sometimes returns more dependencies than actually get compiled in when you use the v2 resolver (I believe it is hard coded to the v1 resolver), but afaik it will never omit any dependencies.


Tuesday 04-07-2023 roll call

Roll Call!
Leave a comment with these two things:

A summary of your study using only emojis
Something you'd like to share, anything goes! (within respect)

emoji cheat sheet

Good morning everyone 🌺

Present 🙇♂

Present

Hola amigos

Good morning

Hi

Good morning

Good Morning

good morning

good morning

good morning

Good morning

👋🏼


Change the x icon for closing the compose box

The x icon we currently use for closing the compose box is inconsistent with other x icons, and does not look good next to the ^, as discussed in this thread. We should replace it with the Font Awesome (v4.7) icon we use elsewhere, i.e. fa-remove.

@zulipbot claim

@zulipbot claim

@zulipbot claim

@zulipbot claim

@zulipbot claim

HI @alya Please review this PR #20519. I have made the required changes to the 'x' icon.
Here are the screenshots

@Arch0125 there is in-progress work on this issue in the linked PR (#20531). Please:

Review the linked PR and comment if you have any improvements to suggest, or
If you feel that a different approach is needed, clearly explain why your PR has a different solution from #20531.

@alya The PR #20531 uses fa-remove icon which still lacks flexibility of changing the thickness as compared to fa-times icon which can even be made of hairline thickness.
The major difference between my PR and #20531 is that I've created a seperate class for the 'x' icon instead of using class="fa fa-remove". Which will allow future contributions to use the same attributes for other icons as well. As mine uses unicode(&times!) to show the icon as it was in the original codebase.
Although both of the PRs are great. I think my PR provides more flexibility for future contributions

@zulipbot claim

@zulipbot abandon

Hello. I would like to work on this issue if it is still open. Can I get some guidance?

@zulipbot claim

@zulipbot claim

@asah have you fixed this issue?

@zulipbot abandon

merged from main, retested, still looks good:

cc @alya

and expanded:

@kunalcodes007 https://chat.zulip.org/#narrow/stream/9-issues/topic/first.20issue/near/1350935

Hey! was this issue totally fixed or may I work on it?
@zulipbot claim

Invite accepted.
@zulipbot claim

@zulipbot claim


Dark mode support for GHES

Summary
After upgrading to GHES 3.2, dark mode is generally available with a seamless dark mode experience across GitHub.
Intended Outcome
Whether as a personal preference or as a way to reduce eye strain in low light conditions, dark mode gives developers more choice and flexibility in how and when they use GitHub.
How will it work?
From the appearance settings page, a user is able to choose between default to system (matching OS preferences), light mode or dark mode for their preferred GitHub theme setting.

<EMAIL_ADDRESS>


Update testing.yml

:warning: Please install the  to ensure uploads and comments are reliably processed by Codecov.
Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Please upload report for BASE (main@bd72719). Learn more about missing BASE report.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@           Coverage Diff           @@
##             main      #12   +/-   ##
=======================================
  Coverage        ?   93.82%
=======================================
  Files           ?        5
  Lines           ?     1053
  Branches        ?        0
=======================================
  Hits            ?      988
  Misses          ?       65
  Partials        ?        0

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


Differences between umap representations for sc.pp.pca(adata, use_highly_variable=True), and HVG subsetted data

To elaborate a bit on my comment on pull request #284 that sc.pp.pca(adata, use_highly_variable=True) does not reproduce the same umap embedding as subsetting the genes.
I have done the following:
disp_filter = sc.pp.filter_genes_dispersion(adata.X, flavor='cell_ranger', n_top_genes=4000, log=False)
adata_hvg = adata.copy()
adata_hvg = adata_hvg[:, disp_filter['gene_subset']]
sc.pp.filter_genes_dispersion(adata, flavor='cell_ranger', n_top_genes=4000, log=False, subset=False)

sc.pp.pca(adata, n_comps=50, use_highly_variable=True, svd_solver='arpack')
sc.pp.neighbors(adata)
sc.tl.umap(adata)

sc.pp.pca(adata_hvg, n_comps=50, svd_solver='arpack')
sc.pp.neighbors(adata_hvg)
sc.tl.umap(adata_hvg)

sc.pl.umap(adata, color='n_counts', use_raw=False)
sc.pl.umap(adata_hvg, color='n_counts', use_raw=False)

The umap output is:

There is clearly a difference between the two cases also in the tsne, draw_graph and diffmap visualisations. Only pca seems to be consistent judging by eye.

@VolkerBergen any idea why this might be?

So I've been sitting offline with @VolkerBergen trying to get to the bottom of this. It seems that the precision of adata.X is reduced after subsetting. This is the case when using either of:
adata_hvg = adata_hvg[:, disp_filter['gene_subset']]
adata_hvg._inplace_subset_var(disp_filter['gene_subset'])

Either way adata[:,disp_filter['gene_subset']].X gives a higher precision than adata_hvg.X

Are you manually setting the precision of adata.X to 'float64'? I can't reproduce the problem with the default 'float32'. I get exactly the same result for the last digit.
If I set the precision of adata.X to 'float64', I can reproduce the problem. It's due to the call of .copy which used the default precision 'float32' and not the one of the actual object... 🙈
It's fixed here: https://github.com/theislab/anndata/commit/ff509e32c985f5fc96a99fe6762149ac89a373ce

I have not set the precision to float64 manually anywhere. It may however be the case that ComBat batch correction automatically uses float64. I will test and let you know.

So it seems ComBat outputs np.float64 😐. I assume with the anndata fix that should be fine now though? I will update and rerun...

Yes, with the most recent AnnData (0.6.13), there won't be any unexpected changes of precision after the object has been created.


Write a blog post for Ratify v1.2.0

To promote Ratify v1.2.0 and gather feedback for the release candidate, we will need to write a blog post to introduce what's new in this release and highlights. Here is a sample blog post https://ratify.dev/blog/ratify-v1.0.0-is-available.

I can help with this. :)

Thanks for your interest @duffney . Just assigned to you. You will be owning this issue so feel free to draft an article in a PR when you are available. I will review it then.


Migrate Account to component

Fixes #782

PR Checklist

[x] Tests
[ ] Tried the feature on a public network
[ ] Documentation

I added a preset with tests, maybe it is too much for just one PR. Let me know if I should better split it. Documenting the preset in the doc-site API corresponding section is still missing.

Moved the preset changes to #803

LGTM! I just left a question

We also have to update the API from our "Embeddable Implementations" discussion. Maybe it's best to just slip that in here?

I updated the Account API sections. Let me know your thoughts. When we achieve some consensus around the naming of the section, I will update them all (other existing API pages) before merging the PR.
cc @martriay @andrew-fleming

I did also update the component names to add the Component suffix.

Very nice! My question was meant just for the account API, but doing them all also works :)
I'm thinking it might be better, in the API, to change EMBEDDABLE CAMELCASE SUPPORT to EMBEDDABLE CAMELCASE IMPLEMENTATIONS to provide more consistency with the naming convention. Wdyt?
Also, we'll have to update the code block under the component title in the API here e.g.
Account
use openzeppelin::account::Account;

I'm thinking it might be better, in the API, to change EMBEDDABLE CAMELCASE SUPPORT to EMBEDDABLE CAMELCASE IMPLEMENTATIONS to provide more consistency with the naming convention. Wdyt?

I think I lean more towards EMBEDDABLE IMPLEMENTATIONS (CAMELCASE), just because I think it looks better visually, lol, but no strong defense, let me know which one you prefer and I will update accordingly.

I think I lean more towards EMBEDDABLE IMPLEMENTATIONS (CAMELCASE), just because I think it looks better visually, lol, but no strong defense, let me know which one you prefer and I will update accordingly.

Sounds good to me!


Vary log level on several request depending on some context status

Hi, I have an issue with logging levels.
We need to vary log level for specific log entries, based on some conditions, leaving configuration as is for all other log entries.
Our first idea was inherit from NLog.Logger, aiming to override IsXXEnabled properties, but those properties are not virtual.
We've managed to solve the issue using Filters, but we needed to remove the MinLevel setting on the Logger config, like:
NLog.config
<logger name="*" writeTo="async">
   <filters>
      <when Condition="MY_CONDITION" action="Log" />
      <when Condition="level < LogLevel.Info" action="Ignore" />
   </filters>
</logger>

So we have two filter "Check" on every log entry, and most of the times MY_CONDITION wil be false.
It's working, but our concern is we think that an override for IsXXEnabled would have a higher performance than our current solution.
What do you think?
Do you recommend to change Logger properties to virtual so we can override it?
Is there a simpler way to achieve this?
Thanks...

This is not planned for now


feat(web): add public talks editor

:tada: This PR is included in version 2.24.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Issue running recon.FBP with 2D data

Tried to run the sandstone notebook by importing FBP from the recon module instead of from astra, and running FBP failed with no useful error message.
@gfardell and I stepped through and found it was an issue with the filtering in the 2D case. It was specifically using:
https://github.com/TomographicImaging/CIL/blob/b70febf37b215dd6672ddc38b05effdecfb6a086/src/Core/FBP_filtering.cpp#L81

The issue is the data is still uint16 when it's passed to the recon class, and the recon class doesn't check for that and just assumes it's float32.

Add a check on datatype to the recon class, or try to convert the data with:
np.asarray(dtype=np.float32, order='C')
This returns the original array if it already satisfies the request, or reorder/cast if able to, or raise and error if it can't do it.

Change the sandstone demo and convert the data to float32 before this point.

AcquisitionData and ImageData should check the array and geometry dtype are consistent.

I had a similar problem with filtering. Ensuring the datatype compatibility (according to the suggestion above) helped me! :)


Handle incoming IRC messages in the ordered event queue

This removes the need to add or remove puppets from the member list list prematurely which should fix ghosting.

Closed with #54


don't change focus to the quicklist

By default setqflist focuses the quicklist, which is annoying.
This restores the cursor position after calling setqflist.

Hi @docwhat,
Thanks for submitting this PR, I on the other hand like the auto focus on the quicklist and see it as a good feature, a lot of the times i want to go straight to the place where things are broken so that i can fix it.
I also agree that other people may prefer the behaviour your described above, I wonder if we should make this configurable. What are your thoughts around it ?
thank you a lot for submitting this by the way, just want to make sure we can find a solution that can work well for everyone

Configurable sounds good.

If you can add configuration for this and keep the normal defaults I will then be able to merge this PR. To add a new configuration you need to:

edit ./plugin/prettier.vim and a a new line like:

let g:prettier#quickfix_auto_focus = get(g:, 'prettier#quickfix_auto_focus', 1)

edit ./autoload/prettier.vim and wrap your changes on a conditional check based on that variable being defined

Update README.md and ./doc/prettier.txt with this new configuration

Onc ethe above is done I will manually check on all supported vim versions and if all is good will merge it!
Once again thank you for the PR !

I’m confused. I did that last night. Do you want me to change the name of the variable?

Hi @docwhat,
Sorry I probably commented on a cached page, it was not updated when I saw!
This is awesome! Will test this out and merge it as soon as I get home! Thank you for your contributions !!

winnr is harmless. Only wincmd needs to be checked. `


sudoers entries should be prefixed with a digest spec

Allowing password-less execution of /usr/local/bin/vde_vmnet as root is a vulnerability when the user has non-sudo write access to /usr/local/bin (which is typically the case when using homebrew), because they could simply replace vde_vmnet with any other command or script and then execute that under root.
This can be mitigated by including a checksum of the executable in the sudo rule, e.g.
$ sha256sum /usr/local/bin/vde_vmnet
cabb4c8bac4a2923a1feb21f597ae6c8145de25e44f408b75ec254da6ffa09ce  /usr/local/bin/vde_vmnet

should lead to a rule such as (untested):
%staff ALL=(root:root) NOPASSWD:NOSETENV: sha256:cabb4c8bac4a2923a1feb21f597ae6c8145de25e44f408b75ec254da6ffa09ce /usr/local/bin/vde_vmnet --vmnet-gateway=<IP_ADDRESS> /var/run/vde.ctl

Most compact supported checksum seems to be sha224 in base64 format:
$ openssl dgst -binary -sha224 /usr/local/bin/vde_vmnet | openssl base64
ZcbnYBmyNCQZGuamPvTkKBDoj4RXqZw/OyPHoA==

leading to
%staff ALL=(root:root) NOPASSWD:NOSETENV: sha224:ZcbnYBmyNCQZGuamPvTkKBDoj4RXqZw/OyPHoA== /usr/local/bin/vde_vmnet --vmnet-gateway=<IP_ADDRESS> /var/run/vde.ctl

Thanks, opened https://github.com/lima-vm/vde_vmnet/pull/20


internal: Move editing cursors into one texture

This should simplify adding new cursors or creating variations

Great idea and thanks! However, on the sysgpu branch I've been working on, I recently merged a change that uses pixies own file setup for its art assets as well. I apologize for the duplicate work, I should have done something to make known that active development is currently a little split with the change to sysgpu. Its also a bit complex because sysgpu fixes a long-standing bug but currently mach-core is disabling sysgpu due to zig not being able to handle optional dependencies, so it could be in this state of limbo for a bit. See here.

Ah cool. I noticed some chatter on that in the magenta bug thread but didn't realize the full scope. In that case sounds like you have this covered already, no worries.

Closing now, I just merged the sysgpu branch which brings this update to Pixi, you can now edit Pixi's artwork directly using Pixi itself. (open the pixi/assets/src folder)


Gene annotations track test broken

Commit: a426e08
Steps to reproduce

Run the gene annotations track tests or check this travis log error https://travis-ci.org/github/higlass/higlass/builds/661547809#L1163

Observed behavior

Script error. thrown for the "changes the color of the minus strand" test.

Expected behavior

Test succeeds / passes

Note that the test does pass sometimes so it probably involves a race condition or timing issue

Just FYI, I may have fixed that in https://github.com/higlass/higlass/pull/874/
Currently working on another test failure.

Oh great!

I don't think it is fixed. Just got it again. As Mark said, it fails only sometimes.


primaryField not being in the SELECT to read back the record after an INSERT

I sent this via a private email, but thought it would be better to document here:
While creating a record I noticed that the WHERE condition in the forceReloadAfterCreateCallback() was not being added.
It turns out that the primary key (coming from Postgres on the ‘RETURNING id’ suffix), was being correctly read into the primaryField but is not setting the field.IsBlank to false.
To fix just need to:
in: callback_create.go
was:
120            if err := scope.SQLDB().QueryRow(scope.SQL, scope.SQLVars...).Scan(primaryField.Field.Addr().Interface()); scope.Err(err) == nil {
121                scope.db.RowsAffected = 1
122            }
needs to be:
120            if err := scope.SQLDB().QueryRow(scope.SQL, scope.SQLVars...).Scan(primaryField.Field.Addr().Interface()); scope.Err(err) == nil {
121                scope.db.RowsAffected = 1
122                primaryField.IsBlank = false
123            }
The bug causes the table where the insert is happening to be fully read/scanned on each INSERT which is less than optimal.

diff --git a/callback_create.go b/callback_create.go
index 14b8204..2308e1f 100644
--- a/callback_create.go
+++ b/callback_create.go
@@ -119,6 +119,7 @@ func createCallback(scope *Scope) {
                } else {
                        if err := scope.SQLDB().QueryRow(scope.SQL, scope.SQLVars...).Scan(primaryField.Field.Addr().Interface()); scope.Err(err) == nil {
                                scope.db.RowsAffected = 1
+                               primaryField.IsBlank = false
                        }
                }
        }

Thank you for your report, and solution, just fixed that. https://github.com/jinzhu/gorm/commit/3425c1d38d18375cc3e27d0c3a8c0e7907eb1b3e


utils: Shorten query in getPathInfo()

Shorter query after https://github.com/serlo/api.serlo.org/pull/136 got
merged

@inyono You can merge this PR after you have published the current master into production.


Add BlazorWebApp to samples

Related to  #79

I spent sometime to resolve this issue but I have many thing to do that's why I late to submit a PR, but I'm glad you did it
Hope to review your PR ASAP

Could you please add a functional test while I'm reviewing this?

@LunaWanderer1 any updates on this?

Hey, haven't had much time to look more into the testing but I noticed the IStringLocalizer doesn't work since it uses FilePaths so that may require more changes

We need to revise this again, so we can take an action quickly

@LunaWanderer1 can we revise this again, sorry for not merging this PR, but PR without testing might breaks anything else, hope you understand my concern


shrink-osd fails due to missing FSIDs

Bug Report
What happened:
I attempted to shrink a number of OSDs by using the shink-osd.yml playbook. However, the set_fact osd_hosts task fails as it attempts to extract a value for the non-existent key osd_fsid from the output of ceph osd find. Although my cluster was previously created with the lvm strategy, and my OSDs have FSIDs as confirmed by running ceph-volume lvm list on the host, the output of ceph osd find does not include a FSID. As a result, the playbook fails with a templating error;
TASK [set_fact osd_hosts] *******************************************************************************************************
Friday 22 February 2019  23:36:40 +0000 (0:00:00.074)       0:00:06.379 *******
ok: [localhost] => (item={'_ansible_parsed': True, 'stderr_lines': [], '_ansible_item_result': True, u'end': u'2019-02-22 23:36:40.459355', '_ansible_no_log': False, '_ansible_delegated_vars': {'ansible_delegated_host': u'allmight.fc.kj', 'ansible_host': u'allmight.fc.kj'}, u'cmd': [u'ceph', u'--cluster', u'ceph', u'osd', u'find', u'0'], u'rc': 0, u'stdout': u'{\n    "osd": 0,\n    "ip": "<IP_ADDRESS>:6803/1951",\n    "crush_location": {\n        "datacenter": "fc",\n        "host": "allmight",\n        "room": "office",\n        "root": "default"\n    }\n}', 'item': u'0', u'delta': u'0:00:00.314916', '_ansible_item_label': u'0', u'stderr': u'', u'changed': True, u'invocation': {u'module_args': {u'warn': True, u'executable': None, u'_uses_shell': False, u'_raw_params': u' ceph --cluster ceph osd find 0', u'removes': None, u'argv': None, u'creates': None, u'chdir': None, u'stdin': None}}, 'stdout_lines': [u'{', u'    "osd": 0,', u'    "ip": "<IP_ADDRESS>:6803/1951",', u'    "crush_location": {', u'        "datacenter": "fc",', u'        "host": "allmight",', u'        "room": "office",', u'        "root": "default"', u'    }', u'}'], u'start': u'2019-02-22 23:36:40.144439', '_ansible_ignore_errors': None, 'failed': False})
fatal: [localhost]: FAILED! => {"msg": "Unexpected templating type error occurred on ({{ osd_hosts | default([]) + [ (item.stdout | from_json).crush_location.host, (item.stdout | from_json).osd_fsid ] }}): coercing to Unicode: need string or buffer, list found"}

Output of ceph osd find when run on the host - osd_fsid key is missing;
ceph --cluster ceph osd find 0
{
    "osd": 0,
    "ip": "<IP_ADDRESS>:6803/1951",
    "crush_location": {
        "datacenter": "fc",
        "host": "allmight",
        "room": "office",
        "root": "default"
    }
}

What you expected to happen:
The playbook to execute and remove the OSDs specified.
How to reproduce it (minimal and precise):

Check out the v3.2.7 tag.
Copy the shrink-osd.yml playbook to the main directory.
Execute the playbook against a Ceph Mimic cluster. Observe the playbook failing as the osd_fsid key is not present in the output of ceph osd find run by a previous task.

This issue looks to have been introduced by the backport of #3515 to v3.2 via #3530. Using the the shink-osd.yml playbook in the v3.2.5 tag appears to work without issue - the playbook executes without any errors and lsblk lists no LVM partitions on the disks previously used for the Ceph OSDs.
Environment:

OS (e.g. from /etc/os-release): Ubuntu 18.04.2 LTS
Kernel (e.g. uname -a): 4.15.0-45-generic #48-Ubuntu SMP Tue Jan 29 16:28:13 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux
Docker version if applicable (e.g. docker version): N/A
Ansible version (e.g. ansible-playbook --version): 2.6.11
ceph-ansible version (e.g. git head or tag or stable branch): tag 3.2.7
Ceph version (e.g. ceph -v): ceph version 13.2.4 (b10be4d44915a4d78a8e06aa31919e74927b142e) mimic (stable)

@leseb @guits should we revert be59e0b on 3.2 branch since ceph osd find command doesn't return the osd_fsid attribute for releases before nautilus ?

@dsavineau that's might be the right thing to do. @noahdesu could you please work on a workaround and patch stable-3.2? Thanks.

@dsavineau @leseb the change on the ceph side was merged into mimic and luminous a bit after the 13.2.4 release was cut. I'll find out when a point release is going to be cut which should solve this issue after a monitor upgrade. Otherwise, maybe we can handle this out of band from the official playbook?

I'm seeing that the releases are being cut now, so they should be arriving very soon

I've retried this today against ceph-ansible tag v3.2.10 and did not encounter the same issue. However, this is down to my Ceph cluster now running 13.2.5 instead of 13.2.4 as previously, and hence ceph --cluster ceph osd find <id> now returns the OSD's FSID;
ceph --cluster ceph osd find 0
{
    "osd": 0,
    "ip": "<IP_ADDRESS>:6802/2255",
    "osd_fsid": "5d35d0a9-c620-4701-9609-b4a2f8d7135f",
    "crush_location": {
        "datacenter": "fc",
        "host": "allmight",
        "room": "study",
        "root": "default"
    }
}

Although the issue still persists for Ceph <13.2.5 and <12.2.11, I can confirm that the @noahdesu's backport of the FSID dump to 13.2.5 works fine with the shink-osd.yml v3.2.10 playbook.

@KingJ thanks for the report and confirmation of the fix in 13.2.5!

Would you consider this to close the ticket?

I'm happy to close the ticket as the issue is resolved by a Ceph upgrade. Thank you for the explanation and upstream fix!


Creating new project interpreter does not work on windows

Attempting to add a new project interpreter it goes through the motions then gives the error "Directory C:\Users\XXX\AppData\Local\pypoetry\Cache\virtualenvs\sample-poetry-app-FpOpy9hG-py3.8
\bin\python not found".
The environment does get created but the path to python instead is:
C:\Users\XXX\AppData\Local\pypoetry\Cache\virtualenvs\sample-poetry-app-FpOpy9hG-py3.8\Scripts\python.exe
Python bases attempted: 3.8.3 and 3.7.3
poetry version: 1.0.6
OS: Windows 10
PyCharm: 2020.1.2 Profession Edition

Upon inspection it appears the file com.koxudaxi.poetry.poetry has the method:
fun getPythonExecutable(homePath: String): String =
        PythonSdkUtil.getPythonExecutable(homePath) ?: FileUtil.join(homePath, "bin", "python")

This perhaps needs to check if it exists and if it does not use:
FileUtil.join(homePath, "Scripts", "python")
instead. I am unsure if it needs the exe as I think it will pick it up without it.

@akinnear
Thank you for post this issue.
Interesting.
I tested the plugin on Windows 10 and python3.8. It works fine.
I needed to test it for a new version.
Also, This version is released and we can download complied binary.
But, the version is not in the market place yet. because we must be waiting for it's approved for about two business days.
Sorry, I'm not a windows expert. I will do research about python executable on windows tonight.


Modified tests according to the chances

Not ready for merging yet:
src/test sorted.
checkstyle violation not resolved. (WIP)

Ready for merging.


Add relativePath to always get parent from repo and silence warning

we might need to add this elsewhere, too, if this works as expected


Consolidate in one bar w/ both Yes and No's

The current approach can be confusing because it shows two different views of the same data on the page. This can be fixed by consolidating all votes into one bar, similar to how it works on tezos agora (see here for example:  https://www.tezosagora.org/period/38) or on Decred Politeia (https://proposals.decred.org/proposals/020b8b0)


Connecting Semesters to Profile

Overview
Resolves #357
Changed the profile page to include the semesters and removed the majors and minors option.
What Changed
Changed the profile component to remove the majors and minors text fields. Also changed the handle submit to include starting and ending semesters. Changed the user trpc updateUserProfile to allow these changes.
Other Notes
Had a few problems with non-null assertions. Removed the ones giving in error in useEffect but kept them in for the handleSubmit. Don't know if using non-null assertions is good practice or not, I can change them if needed.

Oh and also make sure your code is formatted, just noticed the workflow failed. You can run npx  prettier -w . from the project root or install the VSCode extension (if you use that).


Install option defaults to yes on empty input

As per issue #102 I've added a default option for the package install question prompted when running elm-package install

Alright, just merged this in. Thanks!


Update resizedata! with RaggedMatrix

Fixed resizedata! so that resized RaggedMatrices have at least as many rows (.m field) as you ask to have (if you provide an integer number of rows), and preserve the existing datasize (if you provide a Colon).

This isn't quite right: you specify the number of columns, where K corresponds to the maximum number of rows.  I think in the for loop it should be
            for j = B.datasize[2]+1:n-1
                cs = colstop(B.op,j)
                B.data.cols[j+1] = B.data.cols[j] + cs
                K = max(K,cs)
            end

to correctly calculate the maximum colstop.

Also, a CachedOperator is meant to always represent the underlying operator, so the number of rows in the ragged matrix is not really used.  This is why the resizedata!(A,n::Integer,m::Integer) seems wrong.
What's your use case?

Ah, cool. In any case, what I was trying to do seems to break some other things too!
My use case is viewing entries of the operator below the RaggedMatrix's ragged edge: these are zero of course, but attempting to access them (e.g. in showing ragged-below operators) gives you an error.

OK, maybe we want this (sorry the code is inconsistent between n x m and n x m dimensions):
function resizedata!{T<:Number}(B::CachedOperator{T,RaggedMatrix{T}},n::Integer,m::Integer)
    resizedata!(B,:,m)
    B.data.m = max(B.data.m,n)   # make sure we have at least n rows
end

Yes, looks like that works!

Can you update the pull request (just push to the branch) and add a unit test, please?

OK should be good to go I hope

Have you checked that the tests pass? Travis is throwing an error, though I'm not sure what the culprit is.  (It might be some other change but development is not seeing the same error)

PS some of the changes I've made today might have broken the tests.  Just in case this confuses you why the tests fail if you merge back development.

OK, cool. So I've made the above changes (and also fixed a couple of bugs in how resizedata!(B,:,m) calculates B.data.m), and added a unit test in Operators.jl.

Thanks a lot John!  I'll merge this as soon as my changes stabilize.  (I made the mistake of adding …operatortest routines that run a bunch of tests on a specified operator, which of course found numerous bugs.)


fixed case in url

Pressing "Save & Test" in etherpad plugin config caused a redirect to a non-existent route. This PR should fix it.

https://github.com/humhub/humhub-modules-notes/commit/32f0be4874a7a8501171f1f76b1c71d2629a7997


Symlink from /etc/ to /usr/etc/ breaks install to-filesystem

I've installed an RPM that includes a symlink from a path under /etc to a path under /usr/etc. In the container image:
# ls -l /etc/ | grep podman
lrwxrwxrwx 1 root root       21 Nov 12  2023 podman_hpc -> ../usr/etc/podman_hpc

# bootc --version
bootc 1.1.2

Attempting to run bootc install to-filesystem with this image (Containerfile) fails. Using https://github.com/osbuild/bootc-image-builder, the error is:
# podman run --rm -it --privileged --pull=newer --security-opt label=type:unconfined_t -v ./output:/output -v /var/lib/containers/storage:/var/lib/containers/storage quay.io/centos-bootc/bootc-image-builder:latest --local --rootfs ext4 localhost/broken:4
...

This comes from ostree-ext/src/container/store.rs and ostree-ext/src/tar/write.rs AFAICT.
Building from an image without the link (Containerfile) succeeds.
This looks a bit similar to https://github.com/containers/bootc/issues/637 and https://github.com/containers/bootc/issues/856, but with a symlink rather than a hard link.

I've installed an RPM that includes a symlink from a path under /etc to a path under /usr/etc.

It's not about the symlink, it's about having both /etc and /usr/etc at all, it's not supported....
OK well, surprisingly right now it does work but this was not intentional and I'm inclined to make it a hard error.
What are you putting in /usr/etc?
(The semantics right now apparently are that we end up effectively silently "merging" the two but again, not something I want to try to support)

OSTree:ERROR:src/ostree/ot-main.c:324:ostree_run: assertion failed: (success || error)

Hmm fun, that's a different bug where we apparently aren't setting an error correctly.

Completely unrelated to this bug, but fwiw rpm directly supports fetching URLs so the reproducer boils down to
FROM quay.io/fedora/fedora-bootc:42
RUN rpm -ivh --nodeps https://github.com/NERSC/podman-hpc/releases/download/v1.1.0/podman-hpc-1.1.0-1.x86_64.rpm

(And if you do want to fetch URLs, curl is part of the base image, I don't know why people turn to wget; but if you do do it that way you should also remember to rm the RPM afterwards so it's not pointlessly taking up space)

And sorry so I now understand I (or someone) needs to go talk to the https://github.com/NERSC/podman-hpc folks about this, it's not something you specifically are doing.

Universal blue also used to use both /usr/etc and /etc for two reasons: (1) using /usr/etc when making a package allows overriding the /etc files of another package due to how rpm-ostree works, (2) confusion and user error
We found that out when building rechunk and quirked it
I think its done enough where it might be warranted to quirk it and throw a warning instead.

(1) using /usr/etc when making a package allows overriding the /etc files of another package due to how rpm-ostree works

It was really never intended for that. Do you have references to where this hack was used so we can track them? When doing container layering in particular it's totally fine to overwrite files owned by packages, especially for /etc it's just expected. If this is correct then I think the fix here is to just stop using RPM for those files.

Here are some of the PRs that fixed that hack when we switched to rechunk
https://github.com/ublue-os/ucore/pull/183
https://github.com/ublue-os/bluefin/pull/1580
https://github.com/ublue-os/config/pull/310
Currently, the only place it is used is the policy.json file due to (1)
https://github.com/ublue-os/config/pull/311

@rainest FYI, https://github.com/NERSC/podman-hpc/releases/tag/v1.1.1 includes a fix on their side for this. Looks like that got rolled up into the current latest in https://github.com/NERSC/podman-hpc/releases/tag/v1.1.3

Ours was just (2) for lack of knowledge about the separate flag for config location when prefixing the Python install 👍
Dunno if there's still reason to keep it open for the (1) case, or to just fix the error propagation.


sw-emulator does not properly set fuse ready bit in status

The sw-emulator does not properly set GENERIC_AND_FUSE_REG_CPTRA_FLOW_STATUS_READY_FOR_FUSES_MASK in CALIPTRA_TOP_REG_GENERIC_AND_FUSE_REG_CPTRA_FLOW_STATUS. This was confirmed to be an emulator issue by enabling verilator and polling the same bit, which happens immediately.
This test was done using the libcaliptra hwmodel example (currently in PR), which defaults to the sw-emulator. Rebuilding the model and binding with --features verilator and rebuilding the example (and enabling the currently disabled initial poll of the fuse bit) we poll the bit and immediately pass the check. In the software simulator it fails.
Of note, the following line appears on the verilated output:
4,100 ready_for_fuses is high
This line is never seen in the sw-simulator, I do not know if it should be.

In the init_fuses method of the HW Model implementation we are not checking for ready for fuses, and that`s why this issue was probably never exposed


Intermediate commit

resolves #

Description

Checklist

[x] I have signed the CLA
[ ] I have run this code in development and it appears to resolve the stated issue
[ ] This PR includes tests, or tests are not required/relevant for this PR
[ ] I have updated the CHANGELOG.md and added information about my change to the "dbt-snowflake next" section.

Accidentally opened this against the main repository instead of my fork 🤦  Please ignore!


Table order section will allow modification of order and splitting of table.

Added category buttons and menu item buttons. Need to add some additional buttons to go back to the categories list and need some more data to really test functionality. Goes hand in hand with https://github.com/jayzamazing/Restaurant-POS/issues/30


pathway_annotation issue

Thank you for the nice package!
However, when I did pathway_annotation(), data loss occurred.
When I subset about 242 KO  data into 80 lines and divided them into 3 pieces of data, then ran the pathway_annotation().
My data is in the link below.
mydata
f1 <- ko_daa_sub_method_results_df[1:80, ] # 121
f2 <- ko_daa_sub_method_results_df[81:161, ] # 121
f3 <- ko_daa_sub_method_results_df[162:242, ] # 121

d1 <-pathway_annotation(pathway = "KO",daa_results_df = f1, ko_to_kegg = TRUE) # 53 × 11
d2 <-pathway_annotation(pathway = "KO",daa_results_df = f2, ko_to_kegg = TRUE) # 58 × 11
d3 <-pathway_annotation(pathway = "KO",daa_results_df = f3, ko_to_kegg = TRUE) # 50 × 11
annotation <- rbind(d1, d2, d3)
dim(annotation) # 161

The result left only 161 rows out of 242 KO data.
output is here result
I thought it was a problem with keggGet(), so I took the contents inside the pathway_annotation()and ran it separately.

ko_id <- ko_daa_sub_method_results_df$feature # 242
pathway_table <-  data.frame(row.names = ko_id)

KeggGet_results <- list()
for ( i in ko_id) {
  KeggGet_results[[i]] <- tryCatch(keggGet(i), error=function(e) NULL) # https://www.biostars.org/p/366463/
}

ko_result_id <- KeggGet_results %>% names # 236

for (i in ko_result_id){
        pathway_table[i, "Level1"]         <- strsplit( KeggGet_results[[i]][[1]]$CLASS, "; ")[[1]][1]
        pathway_table[i, "Level2"]         <- strsplit( KeggGet_results[[i]][[1]]$CLASS, "; ")[[1]][2]
        pathway_table[i, "Level3"]         <-  KeggGet_results[[i]][[1]]$PATHWAY_MAP
        if ( !is.null(KeggGet_results[[i]][[1]]$DESCRIPTION)) {
          pathway_table[i, "pathway_description"] <-  KeggGet_results[[i]][[1]]$DESCRIPTION
          }
}

The following message appeared.
_

Error in [<-.data.frame(*tmp*, i, "pathway_description", value = c("Type I diabetes mellitus is a disease that results from autoimmune destruction of the insulin-producing beta-cells. Certain beta-cell proteins act as autoantigens after being processed by antigen-presenting cell (APC), such as macrophages and dendritic cells, and presented in a complex with MHC-II molecules on the surface of the APC. Then immunogenic signals from APC activate CD4+ T cells, predominantly of the Th1 subset. Antigen-activated Th1 cells produce IL-2 and IFNgamma. They activate macrophages and cytotoxic CD8+ T cells, and these effector cells may kill islet beta-cells by one or both of two types of mechanisms: (1) direct interactions of antigen-specific cytotoxic T cells with a beta-cell autoantigen-MHC-I complex on the beta-cell, and (2) non-specific inflammatory mediators, such as free radicals/oxidants and cytokines (IL-1, TNFalpha, TNFbeta, IFNgamma).", :
replacement has 2 rows, data has 1

_
The error did not appear because I excluded the DESCRIPTION column.
for (i in ko_result_id){
        pathway_table[i, "Level1"]         <- strsplit( KeggGet_results[[i]][[1]]$CLASS, "; ")[[1]][1]
        pathway_table[i, "Level2"]         <- strsplit( KeggGet_results[[i]][[1]]$CLASS, "; ")[[1]][2]
        pathway_table[i, "Level3"]         <-  KeggGet_results[[i]][[1]]$PATHWAY_MAP
        # if ( !is.null(KeggGet_results[[i]][[1]]$DESCRIPTION)) {
        #   pathway_table[i, "pathway_description"] <-  KeggGet_results[[i]][[1]]$DESCRIPTION
        #   }
}

Is there any way to get rid of these loss of data in ggpicrust as well?
I will be grateful if you answer.

Dear KitHubb,
Thank you for reaching out to me. I confirm that using the split function to separate large tables and annotations is a correct approach. As for the error you encountered in the [<-.data.frame()] function, it seems that the replacement has two rows, but the data has only one. To solve this problem, you can try excluding the "ko04910" pathway from the table, as shown in the code in the appendix.
Regarding the data loss issue you faced while using the ggpicrust package, it is possible that the root cause lies in the limitations of the kegg database API. In order to address this issue, I suggest that you divide the table into smaller sub-tables, set up a loop, and take breaks between annotations to avoid overwhelming the API. Please refer to the code in the appendix for an example of how to implement this approach.
Jordan.R.zip
I hope this information is helpful to you. Please let me know if you have any further questions or concerns.
Best regards,
Chen YANG

Another possible reason for the discrepancy between the input and output rows could be related to API restrictions. To address this issue, I have implemented a code that only annotates pathways with a p.adjust value less than 0.05 for the daa_results_df. As a result, the number of input and output rows might differ.

In addition to the previous information, I wanted to let you know that I have updated the package to the latest version 1.6.1. This new version should have resolved the issue of Error in [<-.data.frame(tmp, i, "pathway_description". Please let me know if you encounter any further issues or have any additional questions.

I'm glad to hear that my R package could be helpful to you. As a developer,
my goal is to create practical and friendly tools for more people to easily
conduct data analysis and visualization.
Please feel free to provide valuable feedback and suggestions during use. I
will listen carefully and work hard to improve and make this package more
handy and powerful. If you encounter any problems in actual use, please
also let me know. I will fix bugs as soon as possible or update the
documentation.
Finally, I wish you can find more interesting topics and make more
eye-opening discoveries in your future work and study! The stage of
progress is full of surprises and excitement. I believe that as long as you
keep working hard and stay curious, you will continue to achieve success.
Thank you again for using and feedback! I will continue to improve and work
hard to live up to the support and trust of every user.
KitHubb @.***> 于2023年4月18日周二 10:00写道：

I am using the attached R script. No results yet, but thanks for the quick
feedback!
I will try to use the updated package in the future.
Thank you for making a package that is exactly what I wanted! :D 👍👍
—
Reply to this email directly, view it on GitHub
https://github.com/cafferychen777/ggpicrust2/issues/17#issuecomment-1512328363,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ATZEQTSJOIYQKHJTOREFKR3XBXYUZANCNFSM6AAAAAAXAZOMGU
.
You are receiving this because you commented.Message ID:
@.***>


.github/workflows: do not persist credentials

A PR created by a GitHub action cannot start new GH actions. Such situation happened with https://github.com/prometheus-operator/kube-prometheus/pull/985.
To work around this limitation actions/checkout@v2 needs to be configured to stop persisting credentials. (source: https://github.community/t/push-from-action-even-with-pat-does-not-trigger-action/17622/5).
I am not sure if this needs to be configured only on the version updater workflow (I assume so) or should it be done in all workflows. Let's try first with only a small change and if this won't work I'll create a follow-up PR.

lgtm


Last slide visually shifted in HTML export

Really love marp for vscode - Thank you.
I noticed that the last slide on the generated HTML export is always slightly shifted vertically in the browser.  It looks lke the "Marp Core's Browser" JS is getting put inside the last slide's SVG.  If the JS was moved outside the last SVG (perhaps just before the closing body tag), then the preview appears correct without any visual position shifting.
tested on:
macOS 10.14 - Chrome latest, Safari 13.0.2, Firefox 70.0.1
Example MD and HTML attached:
marp-shift.zip

Thanks for your catching of details.
JS injection to the last SVG slide has implemented in recent Marp Core. This behavior is for dealing with Marp CLI's bug reported by power-user (marp-team/marp-cli#168), but it seems to get small visual regression as you see (VS Code preview is disabled this injection so would make clear the difference).
I'm sure moving <script> to the outside of SVG would work fine, but we have to improve architectures of Marp ecosystem to get it.

Fixed in v0.11.2.


Update 'Currently Updated' section in the readme

This update has been generated by the new report.py script.

Preview available on https://github.com/endoflife-date/release-data/blob/update-readme/README.md.


test_mio.py::test_recarray failure due to dtype handling changes in numpy

This test failure started to show up in the last 3 days in the prerelease CI, presumably due to a change in NumPy main regarding dtype infra (EDIT: see full log here):
numpy.core._exceptions._UFuncNoLoopError: ufunc 'equal' did not contain a loop with signature matching types (<class 'numpy.dtype[bytes_]'>, <class 'numpy.dtype[str_]'>) -> <class 'numpy.dtype[bool_]'>
The above exception was the direct cause of the following exception:
../testenv/lib/python3.11/site-packages/scipy/io/matlab/tests/test_mio.py:786: in test_recarray
    savemat(stream, {'arr': arr})
        arr        = array([( 0.5, b'python'), (99. , b'not perl')],
      dtype=[('f1', '<f8'), ('f2', 'S10')])
        dt         = [('f1', 'f8'), ('f2', 'S10')]
        stream     = <_io.BytesIO object at 0x7f7dd9f58590>
../testenv/lib/python3.11/site-packages/scipy/io/matlab/_mio.py:298: in savemat
    MW.put_variables(mdict)
        MW         = <scipy.io.matlab._mio5.MatFile5Writer object at 0x7f7dcc15f590>
        appendmat  = True
        do_compression = False
        file_name  = <_io.BytesIO object at 0x7f7dd9f58590>
        file_stream = <_io.BytesIO object at 0x7f7dd9f58590>
        format     = '5'
        long_field_names = False
        mdict      = {'arr': array([( 0.5, b'python'), (99. , b'not perl')],
      dtype=[('f1', '<f8'), ('f2', 'S10')])}
        oned_as    = 'row'
../testenv/lib/python3.11/site-packages/scipy/io/matlab/_mio5.py:892: in put_variables
    self._matrix_writer.write_top(var, name.encode('latin1'), is_global)
        is_global  = False
        mdict      = {'arr': array([( 0.5, b'python'), (99. , b'not perl')],
      dtype=[('f1', '<f8'), ('f2', 'S10')])}
        name       = 'arr'
        self       = <scipy.io.matlab._mio5.MatFile5Writer object at 0x7f7dcc15f590>
        var        = array([( 0.5, b'python'), (99. , b'not perl')],
      dtype=[('f1', '<f8'), ('f2', 'S10')])
        write_header = True
../testenv/lib/python3.11/site-packages/scipy/io/matlab/_mio5.py:633: in write_top
    self.write(arr)
        arr        = array([( 0.5, b'python'), (99. , b'not perl')],
      dtype=[('f1', '<f8'), ('f2', 'S10')])
        is_global  = False
        name       = b'arr'
        self       = <scipy.io.matlab._mio5.VarWriter5 object at 0x7f7dcc15f650>
../testenv/lib/python3.11/site-packages/scipy/io/matlab/_mio5.py:662: in write
    self.write_struct(narr)
        arr        = array([( 0.5, b'python'), (99. , b'not perl')],
      dtype=[('f1', '<f8'), ('f2', 'S10')])
        mat_tag_pos = 128
        narr       = array([( 0.5, b'python'), (99. , b'not perl')],
      dtype=[('f1', '<f8'), ('f2', 'S10')])
        self       = <scipy.io.matlab._mio5.VarWriter5 object at 0x7f7dcc15f650>
../testenv/lib/python3.11/site-packages/scipy/io/matlab/_mio5.py:781: in write_struct
    self._write_items(arr)
        arr        = array([( 0.5, b'python'), (99. , b'not perl')],
      dtype=[('f1', '<f8'), ('f2', 'S10')])
        self       = <scipy.io.matlab._mio5.VarWriter5 object at 0x7f7dcc15f650>
../testenv/lib/python3.11/site-packages/scipy/io/matlab/_mio5.py:798: in _write_items
    self.write(el[f])
        A          = array([( 0.5, b'python'), (99. , b'not perl')],
      dtype=[('f1', '<f8'), ('f2', 'S10')])
        arr        = array([( 0.5, b'python'), (99. , b'not perl')],
      dtype=[('f1', '<f8'), ('f2', 'S10')])
        el         = (0.5, b'python')
        f          = 'f2'
        fieldnames = ['f1', 'f2']
        length     = 3
        max_length = 32
        self       = <scipy.io.matlab._mio5.VarWriter5 object at 0x7f7dcc15f650>
../testenv/lib/python3.11/site-packages/scipy/io/matlab/_mio5.py:670: in write
    self.write_char(narr, codec)
        arr        = b'python'
        codec      = 'UTF8'
        mat_tag_pos = 264
        narr       = array(b'python', dtype='|S6')
        self       = <scipy.io.matlab._mio5.VarWriter5 object at 0x7f7dcc15f650>
../testenv/lib/python3.11/site-packages/scipy/io/matlab/_mio5.py:720: in write_char
    arr = arr_to_chars(arr)
        arr        = array(b'python', dtype='|S6')
        codec      = 'UTF8'
        self       = <scipy.io.matlab._mio5.VarWriter5 object at 0x7f7dcc15f650>
../testenv/lib/python3.11/site-packages/scipy/io/matlab/_miobase.py:424: in arr_to_chars
    empties = [arr == '']
E   FutureWarning: elementwise comparison failed; returning scalar instead, but in the future will perform elementwise comparison
        arr        = array([[b'p', b'y', b't', b'h', b'o', b'n']], dtype='|S1')
        dims       = [1, 6]
=========================== short test summary info ============================
FAILED ../testenv/lib/python3.11/site-packages/scipy/io/matlab/tests/test_mio.py::test_recarray

The FutureWarning is clear, but let's run it by @seberg anyway to see if this was intended. And if so, is it going to turn into an error in 2 releases, or only in a major numpy release?

Yeah, this was the string function change.  It basically expanded the existing FutureWarning to string comparisons (as in comparing unicode and byte-strings).
Since the code seems arguably broken, I would just roll with it?  The code should read arr == b"" using a byte-string for comparison, because you cannot compare unicode and byte-strings!
It is a bit of a short-notice, because numba asked for backporting this last minute (unfortunately, I am not sure it is actually enough for them to be useful).

Ah, this FutureWarning has been around forever on the non-string paths unfortunately.  So to be 100% sure that this only happens after 2 releases, we should possibly add a test somewhere (or rely on SciPy to be that "test").

I opened #16393 already to mitigate it in our CI

The weird thing is that it feels like this actually fixes the logic?  Since before:
In [4]: np.array(["asdf", "fdsa", ""], dtype="S") == ""
Out[4]: False

would never reach the path that replaces empty fields with a space?

I also found the output weird when looking at it since the new code produces an array as I would expect (form a quick glance around)
np.array(["asdf", "fdsa", ""], dtype="S") == np.array([""], dtype="S")
Out[5]: array([False, False,  True])


Accept strings in "keyword" type

closes https://github.com/koleksiuk/dry-elastic_model/issues/10

Please rebase to latest origin/master to get the build passing


fix: improving pivoting logic for when a catalogsource gets updated

Another thing I realize: we have this CatalogSource custom resource sample, but when I was originally bootstrapping this controller I was using the magic catalog CLI as it's easier to quickly check whether you're pivoting logic works as intended without having to deal with rebuilding the FBC index image, and waiting for the next registry poll.

@timflannagan

I think we can drop the the status fields, and only have a status field for conditions
So these fields are currently used to determine if the version we've installed with a BundleInstance is different than the most recently reconciled package with the highest semver in the catalog. Should we instead move to just always update the BundleInstance, even if the information is the same? The effect is probably the same but with a few wasted cycles.

We can update the r.ensureBundleInstance method to simply wrap the controllerutil.CreateOrRecreate function to ensure that resource matches our desired specification
That sounds doable, let me pencil around with this locally and see what I can do.

Another thing I realize: we have this CatalogSource custom resource sample, but when I was originally bootstrapping this controller I was using the magic catalog CLI as it's easier to quickly check whether you're pivoting logic works as intended without having to deal with rebuilding the FBC index image, and waiting for the next registry poll.

I'll have to look into that as I haven't used it before, myself. Shouldn't be too bad and I'll take a look at its documentation.

Should we instead move to just always update the BundleInstance, even if the information is the same

I think we'd have to always perform this work of calculating what's the most recent registry bundle from that platform-operator's package unless I'm missing something obvious. Whether or not we update that BundleInstance is another story as that CreateOrUpdate logic is responsible for checking whether the current and desired client.Object(s) are semantically equal.

Gotcha, this is mainly just my lack knowledge about that function then. I'll mess around with it locally and come up with some new code.

@timflannagan Alright I got this all setup as you said. I think we should do a follow-up where we handle status-management, what do you think?


nuxt.js v2.4.0 && elment-ui v2.8.0+ HTMLElement is not defined

nuxt.js v2.4.0 && elment-ui v2.8.0+
I got a error

ReferenceError
HTMLElement is not defined

node_modules/element-ui/lib/element-ui.common.js  open in editor
props: {
src: String,
fit: String,
lazy: Boolean,
scrollContainer: [String, HTMLElement],
alt: String
},
data: function data() {
return {
Js
@36822:31

when i use l element-ui v 2.8.0-  is ok，

Well, nice that you have this bot working. But it is actually happening to me as well.

Well, nice that you have this bot working. But it is actually happening to me as well.

#15261


minidjvu: mark as vulnerable

Closes https://github.com/NixOS/nixpkgs/issues/90896

Things done

Built on platform(s)

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

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

Todo: Drop this package, upstream is dead (c.f. https://github.com/NixOS/nixpkgs/issues/90896#issuecomment-2409083776).


Validation of s3 objects deletion

This PR adds some tests to validate issue localstack/localstack#5558.

Hi @pinzon , can you please rebase the PR and resolve the conflicts, so we can get this merged? Thanks!

Hi @pinzon , sorry for the ping one more time ;) - the tests on master have now been fixed, can you please rebase the branch one more time, to get the builds green? Thx!

@whummer of course, no problem!


Parenthesis sometimes missing

Hi,
after the latest update, I sometimes get an error like this:
37  nlp_eq2.. sqr((2) * (x3) * (x3) + (-1.328171015325041) * (x3) + -2.076874228151837e-10) + sqr((+(-2.076874228151837e-10
****                                                                  $445
**** 445  More than one operator in a row. You need to use parenthesis
****        if you want to use such statements. For example
****            x := x + - not y;  ->  x := x + (- (not y));
****            sum(i$not x, ..)   ->  sum(i$(not x),..)

I assume that there is a small bug when writing from the Julia expression to the GAMS representation.
Double values maybe should always have parenthesis?

Oh, thanks for reporting! Will fix it right away.

This should be fixed with v0.5.1. Sorry for this inconvenience! Please check if this is fixed for you and then close this issue if so - as I haven't checked your particular model.

I appriciate the fast reponse!
I updated to v0.5.1 with GAMS 46.2.0, but still have the issue.
I found a simple problem that reproduces the issue, maybe you can check it out.
My current status:
Pkg.status(["JuMP","MathOptInterface","GAMS"])
Status `~/.julia/environments/v1.10/Project.toml`
  [1ca51c6a] GAMS v0.5.1
  [4076af6c] JuMP v1.20.0
  [b8f27783] MathOptInterface v1.27.1

the test problem:
 using JuMP
 using GAMS
 model2 =  Model()
 set_optimizer(model2, optimizer_with_attributes(GAMS.Optimizer, "Solver" => "Baron"))
 set_optimizer_attribute(model2, "SysDir", "/opt/gams/gams46.2_linux_x64_64_sfx")
 @variable(model2, -1.0 <= x <= 1.0)
 @variable(model2, must_have_eq, Bin, start = 0)
 @constraint(model2, con, 0 <= x^3 +  (1.0 - must_have_eq) * 100.0)
 optimize!(model2)

results for me in
24  eq1.. 0 - (power(x1, 3) + -100 * x2 + 100) =L= 0;
****                            $445
**** 445  More than one operator in a row. You need to use parenthesis
****        if you want to use such statements.

Thanks, I can reproduce the issue and will add the model to the tests. Sorry for this.

Fixed with v0.5.2.

This fixed the issue for me, glad it was helpful to fix the issue and thanks again for the quick respones.


working svg example for fontello.com?

I just tired to upload lots of different variants of the same quite simple SVG shape , generated with 4 different applications (including Inkscape). All consist of the required single 'compound' path. I tested no fill, black fill (r0g0b0), almost everything I could imagine. All files render correctly in every browser and open in all applications.
But fontello.com tells me either 'If image looks not as expected please convert to compound path manually.' or it only renders only a placeholder box.
Cound anyone please provide me with a working svg file, so I can transform mines to the subset, fontello understands. Thank you.

https://github.com/fontello see any font repo there, src/svg folder.
Also this should help https://github.com/fontello/fontello/wiki/How-to-use-custom-images. Remove all fills and make sure contours have proper directions.

Thank you. That helped.
It seems that we have to ...

keep SVG 1.0 standard from scratch (couldn't find out how to tell inkscape so save 1.0 if the original source was 1.1)
be cautious with too high geometric precision (e.g. 1:100 @ 2048px base grid)
force applications to generate a path attribute even if a polygon (without any curves) would be sufficient to descripe the shape

The only thing that really does matter - convert image to compound path. Everything else is not significant.
If you provide image with single compound path, fontello will do the rest well.

convert image to compound path

It will fail on a polygon. When converted to a path (e.g. using any of inkscapes path functions), it will work.
While this will do the job for me, I only wonder if describing even straight lines as beziers is overkill and needless computation for any font renderer.

... if describing even straight lines as beziers ...

Ok, this seems only to be the case in Inkscape, which treats even straight path segments as (potential) curves. Other applications don't so I assume fontell doesn't as well.
Thanks again for your help. Fontello is a great thing.

I think if you don't create huge number of segments, you will not see difference between beziers & lines in modern renderers.
Some editors can have optimizers to simplify contours, but i'm not familiar with details. In general - if you draw manually - don't waste your time for optimizations :)


Instagram and Foursquare don't work with ProxyPass

Instagram can be cheated by setting the callback URI to localhost:3000 and then manually copying out the token from the response, but Foursquare don't like it. This is a problem because I run Apache as my web server and am proxy passing my api.foo.com requests to localhost:3000.

Looks like Foursquare is doing something weird. I was able to cheat it into giving me an access token, but I had to put the callback URI as http://http://api.foo.com


Running a Shiny app also opens the help tool window

The correct behavior is that only the internal browser shows up (if it is configured that way) instead of popping both the internal browser and the help browser.

https://github.com/Microsoft/RTVS/commit/27e2df12c3e58a8c4f2fe236e4d9c2278357563a


NVIDIA opt-out

Add the ability to completely op-out of setup up NVIDIA support, whether the hardware is present or not, by setting snap config.
By setting nvidia-support.disabled=true the daemon config file will have the nvidia runtime config remove, the CDI config for nvidia will be removed too, and the nvidia-container-toolkit service will be skipped.
This also improves the nvidia hardware detection, by checking for the presence of a PCI device with the relevant ID.
Fixes a bug effecting first install of this snap on a system.

@xnox While I had some time I too a look at adding opt-out via snap config.  IMO this make this safer for eventual stable release.
I also took the chance to improve the NV hardware detection [ in the way you suggested ], and luckily found and fixed a bug with first install.
PTAL when you get a chance.

It seems like disabling service is not notified to the snap via configure hook trigger. Sadness.

@xnox Thanks for the review :+1:
Unless you have any better suggestion right now, I vote to merge this for now as is, as it fixes a bug too.
I'd be happy to re-visit the "disable mechanism" topic in the future if one of us or anyone else has an idea to improve it.

@xnox Thanks for the review +1
Unless you have any better suggestion right now, I vote to merge this for now as is, as it fixes a bug too.
I'd be happy to re-visit the "disable mechanism" topic in the future if one of us or anyone else has an idea to improve it.

Hi @xnox  - apologies, I'm sure you're busy, but just wondering if saw my message above ?
Are you happy to merge as-is for now, especially as it fixes a bug, or would you prefer some changes first ?
Thanks.


hashNamespace.ts is dependent on the window object and thus on client-side JS

We should implement another way of encrypting to support both environment while keeping the library lightweight without installing too much bloatware. The goal of the chat is to be easily embeddable so we don't want it to clutter our user's stack 😃 !

I closed this because I'm a dumbass as insofar as we're working with WEB3, the project is tied to client-side JavaScript.
My bad 😅! window is here to stay...


(SOFTWARE-3752) List recipients of release announcement emails

Actually, could we change those addresses into links to the listserv interface so they could subscribe?


Add script for capturing DHLog files in Zip file

Summary of the pull request
References and relevant issues
This is a temporary fix for #2952
Detailed description of the pull request / Additional comments
A simple script for capturing the user DHLog files and zipping them up for easy submissions with their PRs.
Validation steps performed
PR checklist

[ ] Closes #xxx
[ ] Tests added/passed
[ ] Documentation updated

Closing in favor of #3127


introduce sidecar feature gate

What this PR does / why we need it:
It introduces a sidecar feature gate in order to give cluster administrator some control on executed code within pod's environment
Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes #
Special notes for your reviewer:

[x] add functional and unit tests
[x] update user guide

Release note:

introduce sidecar feature gate

https://github.com/kubevirt/user-guide/pull/263 updated user guide

Causes: [
                    {
                        Type: "FieldValueInvalid",
                        Message: "sidecar feature gate is not enabled in kubevirt-config, invalid entry metadata.annotations.hooks.kubevirt.io/hookSidecars",
                        Field: "metadata.annotations",
                    },
                ],

The code structure looks great, but there's something wrong with the functional tests.

@davidvossel thanks for the review, tests failed because I forgot to update testing manifest. Let's wait for CI and see how it goes now.

/lgtm

/test all

/test pull-kubevirt-e2e-k8s-1.11.0

/test pull-kubevirt-e2e-k8s-genie-1.11.1
/test pull-kubevirt-e2e-os-3.11.0
/test pull-kubevirt-e2e-os-3.11.0-multus
/test pull-kubevirt-e2e-windows2016

let's wait for the test lanes which did not run yet on the latest commit, but I guess this good to be merged
/lgtm
/approve

@slintes thanks for the review, I got the appropriate polarion id so I had to update the PR.
@jerry7z please check if it's ok.

@slintes all comments have been addressed, let's wait for CI and after that, I think we can merge it.

/lgtm


Issue in SCIM 2.0 REST API ULRs in User Roles

Is your suggestion related to a missing or misleading document? Please describe.
In Use SCIM 2.0 REST API section of Add User Roles documentation, the request URL is given as https://{IS_IP}:{IS_PORT}/wso2/scim2/Groups and sample request URL is given as https://localhost:9443/wso2/scim2/Groups.

The given URL is not working. The Working URL is https://{IS_IP}:{IS_PORT}/scim2/Groups.
In 5.11 or above versions of Identity Server, Roles end point should be used for add user role operation instead of Groups end point.

Describe the improvement
The URL should be updated to use correct URL with Roles end point instead of Groups end point.

Hi All,
In addition to the above, the statement "In SCIM 2.0, creating a role is the same as creating a group." is also misleading.
Thanks & Regards.

This issue is being closed due to extended inactivity. Please feel free to reopen it if further attention is needed. Thank you for helping us keep the issue list relevant and focused!


Error - BigQuery Create Partition Table

Hi! this is great tutorial for getting started to learning data pipeline using Airflow and BigQuery, however, I had a problem here when I run the third task I got error message like so :
ERROR - BigQuery job failed. Final error was: {'reason': 'invalid', 'message': 'Partitioning specification must be provided in order to create partitioned table'}.
and after I googling I got some advice from stackoverflow, thus, "I need to create the partitioned destination table first" with the following instruction:
Option 1 using CLI bq mk --time_partitioning_type=DAY mydataset.temps
Or, option 2 using query like this (as bq docs) :
CREATE TABLE
  mydataset.newtable (transaction_id INT64,
    transaction_date DATE)
PARTITION BY
  transaction_date
OPTIONS
  ( partition_expiration_days=3,
    description="a table partitioned by transaction_date" )

Or, do you have an advice to solve this problem? thanks!

Hi @imamdigmi,
I just re-ran the bigquery_github_trends dag, and all the tasks ran through without any errors. Can you check if you use the correct Airflow version?
Maybe trying this command to rebuild the image for the bigquery tutorial:
docker-compose -f docker-compose-gcloud.yml up --build
Let me know if you still have the error.

Hi @tuanavu thanks for your fast response, actually I use the latest stable version of Airflow (1.10.2) and I use Airflow on my local machine with Anaconda, which means that without docker container, and I wrote the DAG and other files by myself, I can make sure there is no problem with my DAG or my Airflow, I confirmed it in the following way:
python $DAGS_FOLDER/bigquery_github_trends.py

there is no error, and I test every single tasks with the following command:
$ airflow test bigquery_github_trends bq_check_githubarchive_day 2018-12-02
$ airflow test bigquery_github_trends bq_check_hackernews_full 2018-12-02

eventually, I stuck on testing the third tasks with the above errors, but, I follow this instruction from , which is I should make empty table on BigQuery with the following command on Google Console Shell:
$ bq mk --time_partitioning_type=DAY my-project:github_trends.github_daily_metrics
$ bq mk --time_partitioning_type=DAY my-project:github_trends.github_agg
$ bq mk --time_partitioning_type=DAY my-project:github_trends.hackernews_agg
$ bq mk --time_partitioning_type=DAY my-project:github_trends.hackernews_github_agg

After that, I re-ran the third test with:
airflow test bigquery_github_trends bq_write_to_github_daily_metrics 2018-12-02

Hi @imamdigmi,
I understand your frustration. That is problem with open source project, and why I have to use docker for version control. Because different Airflow version may have a different version of the Operators that makes the tasks fail.
The Airflow version used in this tutorial is 1.10.1. I will take a look and see if I can reproduce your errors in the new Airflow version 1.10.2.

Thank you very much @tuanavu for your willingness to help me

Hi @tuanavu I just re-ran the DAGs using Airflow version 1.10.1, it's successfully create partition table automatically, bu unfortunately, the data still not stored in destination table (empty)

Hi @imamdigmi,
I believe the reason your query result is not stored in the destination table is because of this setting partition_expiration_days=3. This means all partitions older than 3 days should expire and be deleted. So when you try to run a test on 2018-12-02, which is older than 3 days, the data expired immediately after inserted to the table.
Try to delete and recreate the partitioned table without the partition_expiration_days. Or pick a date in the date range of your partition_expiration_days, and you should see the output in the destination table.

SOLVED! thanks @tuanavu ! but I still wondering, why in Airflow 1.10.2 I have to create table first, while in 1.10.1 version this automatically created when Airflow execute a task.

UPDATE SOLUTION:
You can use parameter time_partitioning={'type': 'DAY'}, to avoiding this error


does it support m1 chips?

Don't know since I never tried it on a M1 machine.
But it should, since it's a simple shell script.


MarkovChain: state_values

Add a method to set state_values afterwards
Once mc is created by mc = MarkovChain(p), then the state_values field has type UnitRange. Currently, it is not allowed to change it to, say, an Array of state values. (This is relevant in particular for DPSolveResult.mc.)

Allow state_values to be AbstractArray?
When state values are 2-dimensional for example, the state space may be represented by a Matrix, but it is not accepted, because of TV<:AbstractVector where state_values::TV.

In response to the first point, I would prefer to have the MarkovChain type be immutable. Instead of changing the state values later on, I would encourage users to simply construct a new MarkovChain instance using the original transition matrix and supplying new state values.
I'd be open to allowing the state_values to be an AbstractArray, with the number of elements in the first dimension equaling the number of rows and columns of the transition matrix. We'd have to think carefully (and probably prototype) to make sure that things like simulation are still seamless/natural if we can't make the assumption of AbstractVector anymore.  An alternative to using Matrix{T} you could use Vector{NTuple{N,T}}  where N is the number of columns the Matrix would have. There are tradeoffs there too...


Will not install Babel

I am trying to install babel for React, an I keep getting an NPM error

log error:
><EMAIL_ADDRESS>postinstall /tmp/apm-install-dir-2020325-66114-9l6560.9j8h/node_modules/language-babel/node_modules/core-js
> node -e "try{require('./postinstall')}catch(e){}"

npm WARN deprecated<EMAIL_ADDRESS>core-js@<3 is no longer maintained and not recommended for usage due to the number of issues. Please, upgrade your dependencies to the actual version of core-js@3.
npm WARN lifecycle The node binary used for scripts is /home/pcon/.asdf/shims/node but npm is using /usr/bin/node itself. Use the `--scripts-prepend-node-path` option to include the path for the node binary npm was executed with.
unknown command: node. Perhaps you have to reshim?
npm WARN enoent ENOENT: no such file or directory, open '/tmp/apm-install-dir-2020325-66114-9l6560.9j8h/package.json'
npm WARN apm-install-dir-2020325-66114-9l6560.9j8h No description
npm WARN apm-install-dir-2020325-66114-9l6560.9j8h No repository field.
npm WARN apm-install-dir-2020325-66114-9l6560.9j8h No README data
npm WARN apm-install-dir-2020325-66114-9l6560.9j8h No license field.

npm ERR! Maximum call stack size exceeded

npm ERR!


is not supported

GitHub markdown supports it and styling it in round-rect. It means keyboard key.
Example of usage :
Press Ctrl+F10 to Compile.
Press Ctrl+F10 to Compile.

This has been fixed in a PR yet to be merged. Just in case you want, I've merged that specific patch in my fork hosted on http://rohitt.me/markdown-editor

Closing this. Please refer to #77.


'EntityFrameworkCore' does not exist in the namespace 'Microsoft'

In VS2019, I created TodoContext.cs as described above, but I get the error:
https://snipboard.io/38bGEn.jpg
1>Models\TodoContext.cs(1,17,1,36): error CS0234: The type or namespace name 'EntityFrameworkCore' does not exist in the namespace 'Microsoft' (are you missing an assembly reference?)

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

ID: 583ae3e7-5915-f22b-e54c-6c100cf2a7da
Version Independent ID: 95f2549e-29f9-e31a-b4d0-432b971643ff
Content: Tutorial: Create a web API with ASP.NET Core
Content Source: aspnetcore/tutorials/first-web-api.md
Product: aspnet-core
Technology: aspnetcore-tutorials
GitHub Login: @Rick-Anderson
Microsoft Alias: riande

When registering TodoContext in Startup.cs, I get the error:
Startup.cs(31,21,31,40): error CS1061: 'DbContextOptionsBuilder' does not contain a definition for 'UseInMemoryDatabase' and no accessible extension method 'UseInMemoryDatabase' accepting a first argument of type 'DbContextOptionsBuilder' could be found (are you missing a using directive or an assembly reference?)

Why is the statement
services.AddMvc().SetCompatibilityVersion(CompatibilityVersion.Version_2_2);
need in ConfigureServices?

The API template has in the ConfigureServices method the following statement:
services.AddControllers();
Should this be removed?

Can you create an issue from the doc page? That will connect the doc to the issue. Go to the bottom of the page and select the Content feedback issue.


Turning off 'Render labels' setting does not persist

Describe the bug
When turning off the 'Render labels' setting in the Settings page, it stays selected and the labels are still shown.
To Reproduce
Steps to reproduce the behavior:

Go to 'Todoist' in 'Settings'
Click on the 'Render labels' toggle to turn it off
Close 'Settings' and see that any labels are still shown in the task list
Go back to 'Todoist' in 'Settings' and see that the 'Render labels' toggle is still on

Expected behavior
The 'Render labels' toggle stays off and labels are not shown
Desktop (please complete the following information):

Plugin Version [e.g. 1.3.0]
Obsidian Version [e.g. 0.8.15]

I made a copy-paste error in #42 🤦♂
Will be a quick and easy one to fix!

Fixed in v1.3.1


License

Hello!
Thanks for your projects, much appreciated. Could you please indicate a license, even if it's a proprietary one?
Thank you!

Hi,
done. Just finished the direct integration between MISP and McAfee Active Response.
Repo will follow soon.
All the best,


scvi.dataset import issue

Hi,
I tried to use virtual environment but met with the same issue (Python 3.7.6). There is something noteworthy though, that is it can't automatically install llvmlite 0.30.0, numba 0.45.0, and hyperopt 0.1.2. I downloaded these whl packages and manually install them. Can this be what is causing the problem?

Hi @sshen82, did you see my other comment in the previous issue? I believe there was a bug in the code snippet you sent.

Yes, I modified that, but it's still not working. Thanks by the way.

I downloaded these whl packages and manually install them.

Can you specify what you mean exactly here?

What I mean is that when I tried to use "pip install scvi", it shows:
ERROR: Could not find a version that satisfies the requirement llvmlite==0.30.0 (from scvi) (from versions: 0.31.0)
After installing llvmlite 0.30.0 version (I download it from https://pypi.org/project/llvmlite/0.30.0/#files), it shows:
ERROR: Could not find a version that satisfies the requirement numba==0.45.0 (from scvi) (from versions: 0.48.0)
then I install numba using the same procedure as above, it shows error for hyperopt. After installing it, when I finally tried "pip install scvi" again, it works, but I just can't use scvi.dataset.

yeah so this really appears to be your python setup.
Here's what it should look like when installing.
05]Mar20 13:09@Adams-MacBook-Pro:scVI/$ mkvirtualenv scvi_pip_test
Using base prefix '/usr/local/Cellar/python/3.7.3/Frameworks/Python.framework/Versions/3.7'
New python executable in /Users/adamgayoso/.virtualenvs/scvi_pip_test/bin/python3.7
Also creating executable in /Users/adamgayoso/.virtualenvs/scvi_pip_test/bin/python
Installing setuptools, pip, wheel...
done.
virtualenvwrapper.user_scripts creating /Users/adamgayoso/.virtualenvs/scvi_pip_test/bin/predeactivate
virtualenvwrapper.user_scripts creating /Users/adamgayoso/.virtualenvs/scvi_pip_test/bin/postdeactivate
virtualenvwrapper.user_scripts creating /Users/adamgayoso/.virtualenvs/scvi_pip_test/bin/preactivate
virtualenvwrapper.user_scripts creating /Users/adamgayoso/.virtualenvs/scvi_pip_test/bin/postactivate
virtualenvwrapper.user_scripts creating /Users/adamgayoso/.virtualenvs/scvi_pip_test/bin/get_env_details
(scvi_pip_test) 05]Mar20 13:09@Adams-MacBook-Pro:scVI/$ pip3 install scvi
Collecting scvi
  Using cached scvi-0.6.1-py2.py3-none-any.whl (150 kB)
Collecting xlrd>=1.2.0
  Using cached xlrd-1.2.0-py2.py3-none-any.whl (103 kB)
Collecting statsmodels
  Using cached statsmodels-0.11.1-cp37-cp37m-macosx_10_13_x86_64.whl (8.4 MB)
Collecting numba==0.45.0
  Using cached numba-0.45.0-cp37-cp37m-macosx_10_9_x86_64.whl (1.9 MB)
Collecting pandas>=0.24.2
  Downloading pandas-1.0.3-cp37-cp37m-macosx_10_9_x86_64.whl (10.0 MB)
     |████████████████████████████████| 10.0 MB 5.2 MB/s
Collecting hyperopt==0.1.2
  Using cached hyperopt-0.1.2-py3-none-any.whl (115 kB)
Collecting torch>=1.0.1
  Using cached torch-1.4.0-cp37-none-macosx_10_9_x86_64.whl (81.1 MB)
Collecting tqdm>=4.31.1
  Using cached tqdm-4.43.0-py2.py3-none-any.whl (59 kB)
Processing /Users/adamgayoso/Library/Caches/pip/wheels/f9/a4/90/5a98ad83419732b0fba533b81a2a52ba3dbe230a936ca4cdc9/loompy-3.0.6-cp37-none-any.whl
Collecting h5py>=2.9.0
  Using cached h5py-2.10.0-cp37-cp37m-macosx_10_6_intel.whl (3.0 MB)
Collecting anndata>=0.6.22rc1
  Using cached anndata-0.7.1-py3-none-any.whl (97 kB)
Collecting scikit-learn>=0.20.3
  Downloading scikit_learn-0.22.2.post1-cp37-cp37m-macosx_10_9_x86_64.whl (7.1 MB)
     |████████████████████████████████| 7.1 MB 49.6 MB/s
Collecting numpy>=1.16.2
  Downloading numpy-1.18.2-cp37-cp37m-macosx_10_9_x86_64.whl (15.1 MB)
     |████████████████████████████████| 15.1 MB 37.6 MB/s
Collecting llvmlite==0.30.0
  Using cached llvmlite-0.30.0-cp37-cp37m-macosx_10_9_x86_64.whl (15.9 MB)
Collecting matplotlib>=3.0.3
  Downloading matplotlib-3.2.1-cp37-cp37m-macosx_10_9_x86_64.whl (12.4 MB)
     |████████████████████████████████| 12.4 MB 45.0 MB/s
Collecting scipy>=1.0
  Using cached scipy-1.4.1-cp37-cp37m-macosx_10_6_intel.whl (28.4 MB)
Collecting patsy>=0.5
  Using cached patsy-0.5.1-py2.py3-none-any.whl (231 kB)
Collecting python-dateutil>=2.6.1
  Using cached python_dateutil-2.8.1-py2.py3-none-any.whl (227 kB)
Collecting pytz>=2017.2
  Using cached pytz-2019.3-py2.py3-none-any.whl (509 kB)
Processing /Users/adamgayoso/Library/Caches/pip/wheels/8b/99/a0/81daf51dcd359a9377b110a8a886b3895921802d2fc1b2397e/future-0.18.2-cp37-none-any.whl
Collecting networkx
  Using cached networkx-2.4-py3-none-any.whl (1.6 MB)
Collecting six
  Using cached six-1.14.0-py2.py3-none-any.whl (10 kB)
Collecting pymongo
  Using cached pymongo-3.10.1-cp37-cp37m-macosx_10_9_x86_64.whl (350 kB)
Processing /Users/adamgayoso/Library/Caches/pip/wheels/30/ac/83/64d5f9293aeaec63f9539142fc629a41af064cae1b3d8d94aa/numpy_groupies-0+unknown-cp37-none-any.whl
Collecting click
  Using cached click-7.1.1-py2.py3-none-any.whl (82 kB)
Requirement already satisfied: setuptools in /Users/adamgayoso/.virtualenvs/scvi_pip_test/lib/python3.7/site-packages (from loompy>=2.0.16->scvi) (46.0.0)
Collecting importlib-metadata>=0.7; python_version < "3.8"
  Using cached importlib_metadata-1.5.0-py2.py3-none-any.whl (30 kB)
Collecting natsort
  Using cached natsort-7.0.1-py3-none-any.whl (33 kB)
Collecting packaging
  Downloading packaging-20.3-py2.py3-none-any.whl (37 kB)
Collecting joblib>=0.11
  Using cached joblib-0.14.1-py2.py3-none-any.whl (294 kB)
Collecting pyparsing!=2.0.4,!=2.1.2,!=2.1.6,>=2.0.1
  Using cached pyparsing-2.4.6-py2.py3-none-any.whl (67 kB)
Collecting cycler>=0.10
  Using cached cycler-0.10.0-py2.py3-none-any.whl (6.5 kB)
Collecting kiwisolver>=1.0.1
  Using cached kiwisolver-1.1.0-cp37-cp37m-macosx_10_6_intel.macosx_10_9_intel.macosx_10_9_x86_64.macosx_10_10_intel.macosx_10_10_x86_64.whl (113 kB)
Collecting decorator>=4.3.0
  Downloading decorator-4.4.2-py2.py3-none-any.whl (9.2 kB)
Collecting zipp>=0.5
  Downloading zipp-3.1.0-py3-none-any.whl (4.9 kB)
Installing collected packages: xlrd, six, python-dateutil, pytz, numpy, pandas, scipy, patsy, statsmodels, llvmlite, numba, future, decorator, networkx, pymongo, tqdm, hyperopt, torch, h5py, numpy-groupies, click, loompy, zipp, importlib-metadata, natsort, pyparsing, packaging, anndata, joblib, scikit-learn, cycler, kiwisolver, matplotlib, scvi
Successfully installed anndata-0.7.1 click-7.1.1 cycler-0.10.0 decorator-4.4.2 future-0.18.2 h5py-2.10.0 hyperopt-0.1.2 importlib-metadata-1.5.0 joblib-0.14.1 kiwisolver-1.1.0 llvmlite-0.30.0 loompy-3.0.6 matplotlib-3.2.1 natsort-7.0.1 networkx-2.4 numba-0.45.0 numpy-1.18.2 numpy-groupies-0+unknown packaging-20.3 pandas-1.0.3 patsy-0.5.1 pymongo-3.10.1 pyparsing-2.4.6 python-dateutil-2.8.1 pytz-2019.3 scikit-learn-0.22.2.post1 scipy-1.4.1 scvi-0.6.1 six-1.14.0 statsmodels-0.11.1 torch-1.4.0 tqdm-4.43.0 xlrd-1.2.0 zipp-3.1.0

how did you install python?

I use miniconda3 downloaded from here. https://docs.conda.io/en/latest/miniconda.html

I recommend using a conda environment and installing using
conda install scvi -c bioconda -c conda-forge

Hmm, this maybe actually be a naive question, because I find that after typing python in terminal, and use "exec(open("/Users/shensiqi/Desktop/scvi.py").read())", it works, but if I type python /Users/shensiqi/Desktop/scvi.py, it doesn't work. Do you know why?

perhaps you have path issues with python? I would follow the conda install instructions for making sure when you type python it's actually pointing to the conda installed version.

I am really new to python, so will typing "python" and "python /.../a.py" use different python version?

It should not, though it's entirely possible that you have some path issues.
I recommend ensuring you have conda properly installed https://conda.io/projects/conda/en/latest/user-guide/install/index.html.
Unfortunately I can't provide any further guidance on this issue. Personally I use pyenv and pyenv-virtualenv instead of conda.

Thank you for your help!


Total transfer progress is not working for downloadFromRemote task

Hi! I can't see a transfer progress for downloadFromRemote task. I've tried using --info=progress2 and --info=progress1 as described here https://github.com/Adambl4/mirakle/blob/development/docs/TIPS_AND_TRICKS.md#show-total-transfer-progress but I can't see any difference in the output. By the way I can see "sent" and "received" for uploadToRemote task.

Here is how it looks for me if I add to rsync args. "--info=progress2"
    388,004,750  95%   10.78MB/s    0:00:34 (xfr#16358, to-chk=0/23994)

Check your rsync --version on local machine and update if it is too old. Mine is 3.2.4.

My rsync version is 3.2.3, not too old

The problem was in -q option. Removing it fixed the problem