Remove unneccessary AppsandFeaturesEntries * manifests/g/Geph/Geph/4.8.5/Geph.Geph.installer.yaml

Microsoft Reviewers: Open in CodeFlow

--> Installing WinGet
--> Disabling safety warning when running installers
Tip: you can type 'Update-EnvironmentVariables' to update your environment variables, such as after installing a new software.

--> Configuring Winget
Enabled admin setting 'LocalManifestFiles'.
Enabled admin setting 'LocalArchiveMalwareScanOverride'.

--> Installing the Manifest 4.8.5

Found Geph [Geph.Geph] Version 4.8.5
This application is licensed to you by its owner.
Microsoft is not responsible for, nor does it grant any licenses to, third-party packages.
This package requires the following dependencies:
  - Packages
      Microsoft.EdgeWebView2Runtime
(1/1) Found Microsoft Edge WebView2 Runtime [Microsoft.EdgeWebView2Runtime] Version 131.0.2903.70
This application is licensed to you by its owner.
Microsoft is not responsible for, nor does it grant any licenses to, third-party packages.
Downloading https://msedge.sf.dl.delivery.mp.microsoft.com/filestreamingservice/files/91530f73-1202-471b-86a9-a3b1d9655560/MicrosoftEdgeWebView2RuntimeInstallerX64.exe
  ██████████████████████████████   173 MB /  173 MB
Successfully verified installer hash
Starting package install...
Successfully installed

Downloading https://f001.backblazeb2.com/file/geph4-dl/geph-releases/windows-stable/4.8.5/geph-windows-setup.exe
  ██████████████████████████████  12.9 MB / 12.9 MB
Successfully verified installer hash
Starting package install...
Successfully installed

--> Refreshing environment variables

--> Comparing ARP Entries

DisplayName           DisplayVersion Publisher  ProductCode                                Scope
-----------           -------------- ---------  -----------                                -----
Microsoft Edge Update <IP_ADDRESS>                Microsoft Edge Update                      Machine
Geph version v4.8.5   v4.8.5         Gephyra OÜ {09220679-1AE0-43B6-A263-AAE2CC36B9E3}_is1 Machine

URL:
Invalid URI: The hostname could not be parsed.
(Automated message - build 899)

@wingetbot run

@wingetbot run

@wingetbot run

@wingetbot run

URL:
Invalid URI: The hostname could not be parsed.
(Automated message - build 902)

URL:
Invalid URI: The hostname could not be parsed.
(Automated message - build 903)

URL:
Invalid URI: The hostname could not be parsed.
(Automated message - build 903)

URL:
Invalid URI: The hostname could not be parsed.
(Automated message - build 904)

@wingetbot run


Code clean up and avoid use anonymous class.

Ⅰ. Describe what this PR did

Simplified some code.
Avoid using anonymous list class.

Ⅱ. Does this pull request fix one issue?
No.

Codecov Report

Merging #1050 into develop will decrease coverage by <.01%.
The diff coverage is 42.85%.

@@              Coverage Diff              @@
##             develop    #1050      +/-   ##
=============================================
- Coverage       38.5%   38.49%   -0.01%
- Complexity      1076     1077       +1
=============================================
  Files            220      220
  Lines           8774     8772       -2
  Branches        1108     1108
=============================================
- Hits            3378     3377       -1
+ Misses          4966     4965       -1
  Partials         430      430

Impacted Files
Coverage Δ
Complexity Δ

...o/seata/rm/datasource/sql/struct/TableRecords.java
36.84% <0%> (ø)
8 <0> (ø)
:arrow_down:

...a/io/seata/rm/datasource/sql/struct/TableMeta.java
33.33% <58.33%> (-0.76%)
15 <3> (+1)

...seata/rm/datasource/sql/struct/TableMetaCache.java
49.31% <71.42%> (+0.33%)
9 <0> (ø)
:arrow_down:

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 545111c...f1916fe. Read the comment docs.


Throw error if global mode isn't supported

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

7 of 8 (87.5%)  changed or added relevant lines in 3 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.03%) to 74.634%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

packages/react-widgets/src/models/utils.js
1
2
50.0%

Totals

Change from base Build<PHONE_NUMBER>:
-0.03%

Covered Lines:
1522

Relevant Lines:
1921

💛  - Coveralls


First deployment of vite app failed

Environment information
Vite-react 1-click deploy

Describe the bug
Customer tweeted here: https://x.com/davidefish/status/1829206390080188445. They have the role assigned but they get the following error:

Reproduction steps
Deploy Vite-React sample

👋 I just tested the Vite-React quickstart guide and was able to deploy the backend without any errors. The outlined error can typically occur if the IAM role is missing permissions to deploy the necessary backend resources.
From the discussion on the X thread, it seems that the IAM role is using the correct managed IAM policy so that can be ruled out.
As next steps, I will recommend reviewing the Service Control Policies that could be enforced on the IAM user/role on an account level within the organization. The SCPs are used to guardrail permissions and can lead to such build failures due to missing permissions.

Adding to the comment above: I was able to reproduce the issue

reproduction steps:

Open cloudformation console, select CDKToolkit → resources
Open the file-publishing-role and deleted it
New deploy should throw this error

Marking as bug for better error messaging.As provided in the comment above this can occur due to Service Control Policies enforced on a account level.
Note, additionally observed the following logs
2024-09-24T23:37:37.673Z [INFO]: [33mcurrent credentials could not be used to assume 'arn:aws:iam::<account-id>:role/cdk-hnb659fds-file-publishing-role-<account-id>-us-west-2', but are for the right account. Proceeding anyway.[39m71
2024-09-24T23:37:37.681Z [INFO]: [33mcurrent credentials could not be used to assume 'arn:aws:iam::<account-id>:role/cdk-hnb659fds-file-publishing-role-<account-id>-us-west-2', but are for the right account. Proceeding anyway.[39m
2024-09-24T23:37:37.684Z [INFO]: amplify-d3ljpx5j1ptm55-main-branch-e3fa383e95: start: Building cc01911d0269d4080ea57505dc445dfc315ef7ad85d3d9d4ea1357858bff451d:current_account-current_region
73
2024-09-24T23:37:37.684Z [INFO]: amplify-d3ljpx5j1ptm55-main-branch-e3fa383e95: success: Built cc01911d0269d4080ea57505dc445dfc315ef7ad85d3d9d4ea1357858bff451d:current_account-current_region
74
2024-09-24T23:37:37.684Z [INFO]: amplify-d3ljpx5j1ptm55-main-branch-e3fa383e95: start: Publishing 3665466abf40cc747bff935f1cf2f49edd920e2cd12d45d502d3857488b3ff3e:current_account-current_region
75
2024-09-24T23:37:37.687Z [INFO]: [33mcurrent credentials could not be used to assume 'arn:aws:iam::<account-id>:role/cdk-hnb659fds-file-publishing-role-<account-id>-us-west-2', but are for the right account. Proceeding anyway.[39m
76
2024-09-24T23:37:37.693Z [INFO]: [33mcurrent credentials could not be used to assume 'arn:aws:iam::<account-id>:role/cdk-hnb659fds-file-publishing-role-<account-id>-us-west-2', but are for the right account. Proceeding anyway.[39m
77
2024-09-24T23:37:37.695Z [INFO]: [33mcurrent credentials could not be used to assume 'arn:aws:iam::<account-id>:role/cdk-hnb659fds-file-publishing-role-<account-id>-us-west-2', but are for the right account. Proceeding anyway.[39m
78
2024-09-24T23:37:37.703Z [INFO]: amplify-d3ljpx5j1ptm55-main-branch-e3fa383e95: start: Building a183ddccbd956316c38ef97177b8f088ef0826f62023323f5ae6053d348ccffc:current_account-current_region
79
amplify-d3ljpx5j1ptm55-main-branch-e3fa383e95: success: Built a183ddccbd956316c38ef97177b8f088ef0826f62023323f5ae6053d348ccffc:current_account-current_region
80
2024-09-24T23:37:37.717Z [INFO]: [33mcurrent credentials could not be used to assume 'arn:aws:iam::<account-id>:role/cdk-hnb659fds-file-publishing-role-<account-id>-us-west-2', but are for the right account. Proceeding anyway.[39m

note: similarily error can occur on sandbox deploy if user has a proxy setup such as vpn
adding --debug to npx ampx sandbox provides this information.
to mitigate this adding the relevant pem file to node's cert storage should mitigate the issue. Consult with your IT team before modifying the trust store


WIP: Feature default NC27

@wiktor2200 I am having a strange behavior.. locally it is working good the playbook, but on GA not. Could you try to run  molecule test locally?

Hi @staticdev! The same on my side. I confirm that it works locally, but on GA I'm having errors. I've also noticed, that errors also showing up in other steps (such as linter).
First attempt failed: https://github.com/nextcloud/ansible-collection-nextcloud-admin/actions/runs/5363160101/attempts/1
But then after re-play it started working: https://github.com/nextcloud/ansible-collection-nextcloud-admin/actions/runs/5363160101

@wiktor2200 then, I also suggest to merge and watch. I will start working now on Debian 12 upgrade ;)


performance measurement of command line tools

break proteins from shotgun metagenomics dataset into peptides, and lookup LCA for each individual peptide

$ fasta2prot < patient_TS19_CDS | prot2pept | peptfilter | sort | uniq | unipept pept2lca -e

number of proteins: 363716 (processed in 0m0.949s)
number of peptides: 2784140 (processed in 0m12.430s (was 0m26.967s))
number of peptides after filtering: 1685570 (processed in 0m16.343s (was 0m38.717s))
number of unique peptides after filtering: 1137246 (processed in 0m17.378s (was 0m49.794s))
lookup of LCAs: ??? (could not be determined due to an error in unipept pept2lca)
Original issue by @pdawyndt on Mon Apr 28 2014 at 20:03.
Closed by an unknown user on Sun May 11 2014 at 13:52.

I've changed the batch size to 10, so you should see more gradual output now. (see unipept 0.2.8)
I've also improved the speed of pept2lca (I performed an unnecessary join).
Original comment by ghost on Tue Apr 29 2014 at 22:21.

error
$ time fasta2prot < patient_TS19_CDS | head -n2000 | prot2pept | peptfilter | sort | uniq | unipept pept2lca
Exception IOError: IOError(32, 'Broken pipe') in <_io.TextIOWrapper name='<stdout>' mode='w' encoding='ANSI_X3.4-1968'> ignored
API endpoint gave an error, exiting.

real    0m0.310s
user    0m0.372s
sys     0m0.044s

performance has increased; at least, now I can process the first 1000 DNA reads in no time
$ time fasta2prot < patient_TS19_CDS | head -n1000 | prot2pept | peptfilter | sort | uniq | unipept pept2lca                      | wc -l
API endpoint gave an error, exiting.
0

real    0m0.318s
user    0m0.284s
sys     0m0.060s

but after a while I always get the same error message
API endpoint gave an error, exiting.

and none of the unipept analysis works anymore; is this a server-related issue?
Original comment by @pdawyndt on Wed Apr 30 2014 at 09:47.

could it be a problem with a cache; if following upon the above analysis I start a request that is new (note that I added the -e option), it seems to work again, but not if I try to repeat an analysis (that uses cached results)
$ fasta2prot < patient_TS19_CDS | head -n10 | prot2pept | peptfilter | sort | uniq | unipept pept2lca
API endpoint gave an error, exiting.
$ fasta2prot < patient_TS19_CDS | head -n10 | prot2pept | peptfilter | sort | uniq | unipept pept2lca -e
sequence,taxon_id,taxon_name,taxon_parent_id,taxon_rank
AVADGR,1,root,1,no rank
DLHNEK,1,root,1,no rank
FFALK,1,root,1,no rank
FHAGK,1,root,1,no rank
NDGAR,1,root,1,no rank
NEGLGK,1,root,1,no rank
NLTSGFK,1,root,1,no rank
NQDSGR,1,root,1,no rank
SGYGNR,1,root,1,no rank
SPEGK,1,root,1,no rank
VVELK,1,root,1,no rank
ALFLDTPEPR,816,Bacteroides,815,genus
GEVLGGVGSTGK,816,Bacteroides,815,genus
GTFTELAR,816,Bacteroides,815,genus
LDQSSTYQALQQLQMFMEYFPNSTK,816,Bacteroides,815,genus
LNNQLQVLQK,816,Bacteroides,815,genus
LSVAVADLFKPGDGVTLTPK,816,Bacteroides,815,genus
NGQNLPMPHFGNNFALEAQQVLYAGGALNSSLTLAELGQQMAALDWQK,816,Bacteroides,815,genus
QEAQDMLFALQDK,816,Bacteroides,815,genus
SMLDMEGYWK,816,Bacteroides,815,genus
VVNPVNYYFMDLSAEDYDR,816,Bacteroides,815,genus
AENAFLPR,817,Bacteroides fragilis,816,species
NGEYLPSFLSLDK,817,Bacteroides fragilis,816,species
LVLDHGFGYQTVYAHLR,820,Bacteroides uniformis,816,species
SLNFPEETLFFALTTK,820,Bacteroides uniformis,816,species
VGNVPATLDWLLTDSR,820,Bacteroides uniformis,816,species
YLADLYAR,820,Bacteroides uniformis,816,species
AAAVVLGHTDESVLQEYVADVLVGK,821,Bacteroides vulgatus,816,species
NLLETGNMGR,976,Bacteroidetes,68336,phylum
AEELLGAVENK,1263,Ruminococcus,541000,genus
AVSLAQSHR,1263,Ruminococcus,541000,genus
GTGLTEAANK,1263,Ruminococcus,541000,genus
LTGVVPEEQK,1263,Ruminococcus,541000,genus
VTPVHEFCDK,1263,Ruminococcus,541000,genus
FLLTGYYLDLYK,171549,Bacteroidales,200643,order
LTNEVVAMK,171549,Bacteroidales,200643,order
STGPHLHYEVHVK,171549,Bacteroidales,200643,order
FDSDLFVSK,186802,Clostridiales,186801,order
LLEVLGHK,186802,Clostridiales,186801,order
VDLLSLVK,186802,Clostridiales,186801,order
VMVELTDYGDAR,186802,Clostridiales,186801,order

Original comment by @pdawyndt on Wed Apr 30 2014 at 09:49.

another issue
$ time fasta2prot < patient_TS19_CDS | head -n5000 | prot2pept | peptfilter | sort | uniq | unipept pept2lca -e | wc -l
API endpoint gave an error, exiting.
2694

real    0m0.843s
user    0m0.752s
sys     0m0.032s
$ time fasta2prot < patient_TS19_CDS | head -n10000 | prot2pept | peptfilter | sort | uniq | unipept pept2lca -e | wc -l
Exception IOError: IOError(32, 'Broken pipe') in <_io.TextIOWrapper name='<stdout>' mode='w' encoding='ANSI_X3.4-1968'> ignored

Original comment by @pdawyndt on Wed Apr 30 2014 at 09:51.

source of error:
ArgumentError (comparison of NilClass with 51669 failed):
  app/controllers/api/api_controller.rb:67:in `sort'
  app/controllers/api/api_controller.rb:67:in `lca'

Original comment by @bmesuere on Wed Apr 30 2014 at 10:05.

This should now be fixed. I've pushed unipept 0.2.10 which should also exit more cleanly.
Original comment by ghost on Fri May 02 2014 at 11:47.


Dealing with MJPEG-Streams

I have expanded #AlamofireImage for dealing with #MJPEG-Streams

Heer is the link to my commit. https://github.com/ugoArangino/AlamofireImage/commit/5f2e952ccf242aa038e6bffbb91cc6d0bb9a1ecb
What do you think about this addition. Makes a pull request sense?
Or should I create a separate framework that provides a closure fur the streaming API.

We're busy working through the AF 4 release right now @ugoArangino...I promise I'll dig in here and get you an answer soon. Please hang tight. 🍻

Updated this to the AF 4 changes and submitted a new PR at #185.

Closing this out for #185. Please redirect all further comments to #185.

This is exact something I was looking for a support to MJPEG on tvOS.
Is there any link/example codes available on this?
Thanks!

@rat-moonshine This might help you https://github.com/ugoArangino/MJPEG-Framework

This is exact something I was looking for a support to MJPEG on tvOS.
Is there any link/example codes available on this?
Thanks!

@rat-moonshine This might help you https://github.com/ugoArangino/MJPEG-Framework

This is exact something I was looking for a support to MJPEG on tvOS.
Is there any link/example codes available on this?
Thanks!

Hei thanks @ugoArangino ! Co-incidentally I have found another one https://github.com/WrathChaos/MJPEGStreamLib and this seems to be a more updated one too! Running good on Swift 5!


Fix incorrect log file location

Turns out that the log file was always written to the directory from where the process was launched, not where the sh script actually resides.

But the old behavior was correct! We want logfiles relative to cwd, not the e2etest installation, don't we? That way, wherever the caller is, they get it dropped there.

Made https://github.com/boxine/pentf/pull/251 to revert that change and make all other files use cwd too


Format notebooks with Ruff

Follow-up to #2197. Draft until that PR is merged, then I'll rebase this.
Also a partial fix for #1806 depending if we want to also lint code examples in MDX.

Gonna wait to merge this until after #2139 is merged to avoid further disruption.


Cell width not getting set

The headers render correctly, but the cells are all set to exactly the space they need to there contents.
I am updating from what downloaded as 1.4.3-59 to 2.0b1-8
I am thinking that I am missing a setting or something that I didn't need before.
In my copy, I added after
rule = columnCssRulesL[i];
in the applyColumnWidths function.
rule.style.width = (w - cellWidthDiff) + "px";
That fixes my issue for now, but I think I might be better off figuring out what I need to change... Any one have an idea?

Can you clarify?  Why do you need width set on the cell style?
BACKGROUND:
2.0 switched from generating .c1 / .c2 / ... CSS rules that contained widths to generating .l1 / .l2 / ... and .r1 / .r2 / ... pairs that contained left and right positions respectively.  This lets us easily support colspan by applying different .lX and .rX styles.  For example, a cell spanning for column 2 to column 4 would get ".l2 .r4" styles.

Ok, was something in my CSS. Just redid the entire thing, so I am not sure which item changed it.


I can't access to my tydom

Hello,
Thank you very much for your great effort.
I've been with home Assitant for a one year and the only thing I haven't been able to integrate is Tydom. It's very frustrating.
To integrate tydom in home assistant there are 2 pieces, the mosquitto broker and the tydom2mqtt.
I don't know which one is failing. I'll pass you the configuration of my tydom2mqtt and the log to see if it helps.
In red the altered values but that are good
Thanks a lot

Hi,
Thank you for your support.
I think you got 2 errors.
The 1st one Parsing error TYDOM_ALARM_PIN
Even if you don't have any alarm, please set an arbitrary value (eg 123456)
It's an issue I need to resolve (see #12)
The 2nd one is related to MQTT connection error ... (127.0.0..1, 1883)
Please notice that the addon is not running on the docker host network so the MQTT host is likely to be different from localhost|<IP_ADDRESS>

Hello,
Thank you very much for your quick response.
I have solved the 2 problems thanks to your help and I already have Tydom working on my hassio.
Great work of yours......


Does not work with torch 1.7.1+

The demo code does not work with this version of PyTorch. No output is printed
import torch
from pytorch_memlab import LineProfiler

def inner():
    torch.nn.Linear(100, 100).cuda()

def outer():
    linear = torch.nn.Linear(100, 100).cuda()
    linear2 = torch.nn.Linear(100, 100).cuda()
    inner()

with LineProfiler(outer, inner) as prof:
    outer()
    prof.display()

sorry, I was too fast, it works but the documentation is a bit confusing.
the last line should be prof.print_stats()


Accept build_directory config for theme extensions

WHY are these changes introduced?
Fixes #867

WHAT is this pull request doing?

Accepts a build_directory config in the extension config file. This can be used to have a src/dist directory setup for theme extensions.
How to test your changes?

Move your theme files to a build directory, and configure build_directory in your theme app extension's TOML file. Things should work as usual.
Post-release steps

Need to document this!
Measuring impact
How do we know this change was effective? Please choose one:

[x] n/a - this doesn't need measurement, e.g. a linting rule or a bug-fix
[ ] Existing analytics will cater for this addition
[ ] PR includes analytics changes to measure impact

Checklist

[x] I've considered possible cross-platform impacts (Mac, Linux, Windows)
[x] I've considered possible documentation changes
[x] I've made sure that any changes to dev or deploy have been reflected in the internal flowchart.

My understanding from @karreiro's comment here is that they don't have a strong opinion about this, since it's not strictly theme-related.

Still relevant!

cc @amcaplan @karreiro lets try and get this one fixed

Just checking in if we can get some sort of action solution 👍🏻


Shorten labels to match convential prefixes

Codecov Report

Merging #1894 (e1a66ef) into main (4746eb1) will decrease coverage by 0.06%.
The diff coverage is 85.71%.

@@            Coverage Diff             @@
##             main    #1894      +/-   ##
==========================================
- Coverage   57.37%   57.32%   -0.06%
==========================================
  Files         242      242
  Lines       16223    16226       +3
==========================================
- Hits         9308     9301       -7
- Misses       5960     5969       +9
- Partials      955      956       +1

Impacted Files
Coverage Δ

libs/header/sync/sync.go
64.16% <50.00%> (ø)

libs/header/sync/ranges.go
92.55% <100.00%> (+0.24%)
:arrow_up:

... and 2 files with indirect coverage changes
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


Additional user role(s)

There should be at least 1 restricted user type, who can run protocols but can't edit protocols. Open to naming ideas for these admin/not user roles... "admin" and "operator"?

[ ] Add an administrators policy editor UI

This was completed when we added the casbin library. Examples are included for roles "admin" and "user" (user can run protocols but cannot edit protocols). Arbitrary permissions groups can also be created using this library.


Intellisense doesn't really work

Issue:
When writing code there are no suggestion(intellisense).
For example when starting to write ui... there is not popup to show possible options.

https://github.com/user-attachments/assets/b7a02dcd-6632-46dc-83f8-2479d19b59a5
Example


auto import { base } from "$app/paths"?

Is there a way to automatically import { base } from "$app/paths"?
I can't figure out.

Yes, using mapping:
autoImport({
  mapping: {
    base: `import { base } from "$app/paths"`,
  }
})

However, the base is a very common word, it's also possible to rename it to another word.
autoImport({
  mapping: {
    AppBase: `import { base as AppBase } from "$app/paths"`,
  }
});


Boxplot not reordering plots

Hello,
I have a dataframe:
Groups    Score
A              1
B              2
C              3
That I want to use for regression and boxplots (using gadfly).
I reordered my categorical variables using:
pool!(df2, [:Group])
df2[:Group] = PooledDataArray(df2[:Group], UTF8String["A", "C", "B"])
Now my regression output shows the coefficients in the order that I want.
However, using gadfly:
p1= plot(df2, x="Group", y="Score", Geom.boxplot,
Guide.ylabel("Scores"), # label for y-axis
Guide.title("Scores by Groups"))
The groups are still ordered alphabetically and not according to my reorder.
It would be useful to have Gadfly keep the order that I already instituted for the categorical variable.
Thanks.

Could you provide a stand-alone code snippet that reproduces the problem by any chance?  I was unable to get the above to run and I might be missing something.
Are you pooling your data just to reorder it?  Plot with Geom.boxplot preserves the order in which the categorical variable appears, so you can just reorder the dataframe:
df2 = DataFrame(Group = ["A", "B", "C"], Score = [1,2,3])
p1= plot(df2, x="Group", y="Score", Geom.boxplot)

#This is a terrible way to reorder
df3 = DataFrame(Group=ASCIIString[], Score=Int64[])
for gr in ["A", "C", "B"]
    append!(df3, df2[df2[:Group].==gr,:])
end
p1= plot(df3, x="Group", y="Score", Geom.boxplot)

Alex,
Here's my entire code:
using DataFrames
using GLM
using Distributions
using NumericExtensions
using Gadfly
using Cairo
#load modules
df = readtable("~\ipython-test\MIF-IC6.csv", header=true)
#read entire database
df2 = df[(df[:Group] .== "Control") |(df[:Group] .== "UTI") |(df[:Group] .== "Radiation"), :]
#select 3 groups only
pool!(df2, [:Group])
#need to do this to make sure that julia sees it as categorical
df2[:Group] = PooledDataArray(df2[:Group], UTF8String["Control", "UTI", "Radiation"])
#reorder Group levels
levels(df2[:Group])
#to check they are listed in correct order
lm1 = fit(LinearModel, MIF_pg_ml ~ Group, df2)
#run linear regression; coefficients are planned t-test comparisons
p1= plot(df2, x=PooledDataArray(df2[:Group], UTF8String["Control", "UTI", "Radiation"]), y="MIF_pg_ml", Geom.boxplot,
Guide.ylabel("MIF(pg/ml)"), # label for y-axis
Guide.title("Urine MIF concentrations"))
#to create graph - order is still alphabetical
I am trying to reorder the dataframe (and I can do this quite easily in R). Was told that PooledDataArray was the way to go and it does work for the regression but not for the boxplots. Ideally, I would like to do it just once and not reorder again just for the graphs, however, I'm just learning Julia.
Thanks for the help.
Pedro

It won't respect the order of the pool in a PooledDataArray (which is what R does, so it's a reasonable assumption), but like @amellnik pointed it out, it does respect order the values appear in a vector. You can reorder the rows of the data frame, but it should also work if you pass Scale.x_discrete(levels=UTF8String["Control", "UTI", "Radiation"]) to plot, where levels are in the order you'd like.

That works great!
Thanks for your help.

Sure, no problem.


Update launch.sh to fix goaccess data storage

With the current settings I was only able to see one day's worth of logs in goaccess. This change checks if $LOGROTATE and $GOA are true and if so it kills goaccess before running the log rotate which causes goaccess to save its db files. (goaccess automatically restarts with your current settings)
If $GOA is false and $LOGROTATE is true then the system works as before just running logrotate.

Thanks, but maybe this should be done inside the logrotate conf using prerotate? See: https://linux.die.net/man/8/logrotate and https://github.com/ZoeyVid/NPMplus/blob/develop/rootfs/etc/logrotate

What do you think? I think pretoate would be a safe implementation. And thanks for the note on the compose.yaml, I chaged it now so new instances already have this fixed before this is merged


[Feature Request] ValidateAuthority parameter not exposed causing the library not to be used in ADFS deployment. #289

[Feature Request] ValidateAuthority parameter not exposed causing the library not to be used in ADFS deployment. #289

I will open the PR with all the required changes.


L2CAP ecred test cases failed.

When I test L2CAP cases with PTS. There are 3 cases failed.
L2CAP/ECFC/BV-11-C, L2CAP/ECFC/BV-13-C and L2CAP/ECFC/BV-15-C.
From the test log output, the issue is that when send le_ecred_conn_resp packet to PTS. The PTS needs including valid CID value. But the l2cap of zephyr response with "CID 0x0000".
It looks reasonable, but I want to know is zephyr passed these PTS cases or not.
If zephyr projecct have some CICD job for these test cases?
Thank you.

There is an open PTS CASE for these tests (CASE0072485). PTS does not conform to the latest spec, where 0x0000 is the expected CID when the connection request is rejected.
Also note that L2CAP/ECFC/BV-15-C is category D, so it is not needed for qualification.

Ok, got it.
Now, I find the issue is processing and release a patch about the PTS tool. If the result of the connection is not success, the CID should be 0x0000. So, I think the zephyr code and your comment is correct.
Thank you.

This is an issue with PTS, see  CASE0072485


Assignments: Negative candidates

One could enter negative values for candidates to be elected.

@flanagan You can try this issue if you like.


Allow specifying second argument for RTCPeerConnection constructor

The constructor RTCPeerConnection takes a second argument. This is deprecated (if it was ever in the spec), but AFAICT this is the only way to pass vendor-specific constrains supported by Chrome, such as googDscpand googIPv6.
In particular I am interested in enabling DSCP, but passing googDscp as a constraint in any other way does not seem to have any effect.

Due to this not being part of the spec, and the default SDH covering the basic use cases. I am inclined to push this off to your application. You should create your own SDH copied from ours with this change integrated. Doing something like this is a prime use case for creating your own SDH.

FWIW, I decided to monkey-patch RTCPeerConnection instead.
const realRTCPeerConnection = window.RTCPeerConnection
window.RTCPeerConnection = configuration => new realRTCPeerConnection(configuration, { optional: [{ googDscp: true }] })


Can't loading

Hi, the mod cannot load
I used Fabric - 1.0.0 from modrinth
Minecraft 1.19.4
Fabric Loader 0.14.19
https://mclo.gs/214nmwl

Mods:

architectury-8.1.80-fabric
fabric-api-0.80.0+1.19.4
spritearrows-fabric-1.0.0

Thanks for telling me, it was a typo in the fabric.mod.json file that somehow got in there between creation and publishing of the mod. idk how it happened, but I'll upload a fixed version asap.

Fixed, you can get version 1.0.1 on modrinth

Thanks!


Meri/insert from select

Current coverage is 81.05% (diff: 100%)

Merging #4560 into master will decrease coverage by 0.15%

@@             master      #4560   diff @@
==========================================
  Files          1294       1294
  Lines         44090      44090
  Methods           0          0
  Messages          0          0
  Branches       7011       7011
==========================================
- Hits          35805      35735    -70
- Misses         5590       5651    +61
- Partials       2695       2704     +9

Powered by Codecov. Last update 59586d5...e6e85a5

@seut please review


Provide useful error messages

In a number of cases, you throw errors: https://github.com/aserputov/qck-ssg-final/blob/main/index.js#L111.  This is going to mean that the program crashes and the user is left with a cryptic message and call stack.
Instead, you should log a useful error message.  Useful here means "actionable," and indicates to the user how to fix things so that the program can work next time.  You should also exit the program with an error code (e.g., non-zero error code):
if(err) {
  console.error('Helpful error message here...');
  process.exit(1);
}

Done. Thanks.


[DOCFIX] Add Release Notes 2.7.3

What changes are proposed in this pull request?
add release notes
Why are the changes needed?
bring release notes to docs
Does this PR introduce any user facing changes?
n/a

alluxio-bot, merge this please


MINOR: demo a generic Jackson serde

One way to achieve a generic json serde through Jackson. This implementation isn't super robust, just a POC.
Committer Checklist (excluded from commit message)

[ ] Verify design and implementation
[ ] Verify test coverage and CI build status
[ ] Verify documentation (including upgrade notes)

@vvcephei forgive my ignorance, but what is the purpose of this?

@dguy @viktorsomogyi here's the motivation: Kafka Streams cannot inference types and hence auto-register serdes for all intermediate data due to type erasure in Java. We think this is a minor issue in practice since for production code, most likely the data format is not primitive types, but schema-oriented like JSON / Avro, etc. In this case, a single registered serde class at config time should be sufficient to serde all types of data.
But users have been pointed out, quoted below:
... if you are using a lot of POJO(Plain old java object) in your application, you have the added task of specifying how to serialize them in order to pass them down the pipeline, Avro schema or not. Although understandable, it adds an extra dimension of complexity that I'm not sure is always worth it in business logic heavy applications.

This complexity is illustrated in the PageViewTypedDemo.java example code: comparing with PageViewUntypedDemo.java it requires a serde class for each POJO typed class, and also users have to specify them during the topology via Consumed, Serialized, etc.
The idea here, is to define a parent interface that returns the  right serde. Then, you can set the default serde to it that extracts the right serde dynamically.
As you can see the diff, although the key types are still primitive, hence enforcing users to specify different serdes whenever it changes, the value serde is always JSONSerde. So imagine the key type is not primitive but also JSON typed, we can actually save all the serde specifications but just specify it via config.
@vvcephei a qq for you: this implementation seems be relying on objectMapper#convertValue in JSON, so I'm wondering if it can be generalized for any serde types, like Avro, Protobuf, Thrift etc for POJO classes?

Thanks for the reviews, @dguy , @bbejeck , @viktorsomogyi , and @guozhangwang ,
As Guozhang said, this was really just intended to sketch out one way of doing "generic" serialization with Jackson, as it's cumbersome in practice to have a separate serde for every single type.
I actually wasn't thinking that we'd merge this PR, just use it as a point of discussion.
@guozhangwang I don't know about Thrift or Protobuf, but Avro requires a schema. I'm pretty sure you can't just feed it a random POJO and tell it to (de/)serialize it. I've always seen projects start with a schema and then generate their Java data classes. The data class actually contains the schema built in, and you register all the classes with the (de)serializer, so it can properly serialize objects according to their schema, and detect the right schema to deserialize with on the flip side.
So, if I understand you question right, no, I don't think that there is one approach we can use for all serialization protocols. I suspect that each protocol will have its own corresponding algorithm.
This is the inverse of what I was going for by including the Java class in the json. It's a low-fi version of the official way to accomplish this task in Jackson: https://fasterxml.github.io/jackson-annotations/javadoc/2.4/com/fasterxml/jackson/annotation/JsonTypeInfo.html
To @bbejeck 's point, it may not be acceptable to see the class information in output topics, or for that matter, to require them in input topics. I have one idea about this, but it seems pretty brittle to me:
if the key set is unique for each data class in the application, then the classes can all register their "signature" as the set of json keys they generate. Then the deserializer just has to extract the key set and look up the right class to deserialize to, et voila!
Let me know if you'd rather see this POC'd.

@vvcephei yeah that makes sense. For Avro the GenericRecord type is similar to the untyped example here, while for SpecificRecord one can defines a SpecificAvroSerde<T extends SpecificRecord> implements Serde<T>, and the schema generated classes will have the form of MyClass extends org.apache.avro.specific.SpecificRecordBase implements org.apache.avro.specific.SpecificRecord (i.e. it is much more friendly than POJO) so it should be okay. For other types although I'm not very familiar either my guess would be some tricks like the one for Avro can be applied as well.

I think for JSON it is sufficient to show as an example, rather than going fancy with the signature registration. Just leaving it as a comment is good enough.

Ok, if you actually plan to merge this as an example, I think I'd like to change it to use JsonTypeInfo.
Should I?

Ok, if you actually plan to merge this as an example, I think I'd like to change it to use JsonTypeInfo.
Should I?

Go ahead :)

@guozhangwang @bbejeck ,
I've updated the example to use Jackson annotations instead. Also, it has a more compact output of (e.g.) {"_t":"rc","count":1,"region":"region2"} instead of printing the whole class name.
I also cleaned up the now-unused serdes.

I left the data generator in there because I thought it was kind of nice. WDYT?

@dguy @vvcephei thanks for the context, now it makes much more sense :)

No problem @viktorsomogyi ;
Sorry about that. I was initially just using this as a way to demonstrate something to Guozhang. I didn't expect everyone to take an interest!
But now, it sounds like we are actually thinking about merging this, so I guess it's good in the end.

I see. Yeah, I don't see how we could provide such a generic serde for Jackson. The java classes themselves are not portable, so the fully-qualified name would be a terrible protocol for sharing data across applications. The current "type identifier" is a little more portable, but it would still be a super awkward protocol for data sharing.
Personally, I think something like this is good for quick-and-dirty prototyping, but ultimately, I'd recommend anyone to use something like Avro for production:

json is far more wasteful than binary serialization on the wire and in the broker
we are basically storing the "schema" in every single record via "_t", which is also wasteful. It's much better to store the schema externally and just map it to the topic somehow.


✨ Don't use the policy filename in SARIF results

This uses a default "no-source-file" for results that don't have explicit files to display.
Since we're removing policy files from the GH action, that makes it less confusing for users.

friendly ping to LGTM, as discussed offline


Empty themes listing in preferences after upgrade 1.3 → 1.4

The themes are not listed any more. Any idea?

Hi,
yes it's because of the compatible field on themes, I recently update the themes.
On the file metadata.json, you need to add the field "compatible" and add the version of Bludit.

Solution: All themes must have as @dignajar mentioned the Compatible line example:
 Alexander K @HoundSU 11:05 { "author": "n33co", "email": "", "website": "https://github.com/dignajar/bludit-themes", "version": "1.1", "releaseDate": "2016-01-21", "license": "CCA 3.0", **"compatible": "1.0,1.1,1.1.2,1.3,1.4",** "notes": "This theme is based on Future Imperfect, all credits to the author n33co." }
Without that line no themes shall work.

I made the updates on all themes from themes.bludit.com


Concatenating and deserializing streams written by SnappyOutputStream

I'd like to be able to write data to multiple files using separate SnappyOutputStreams, then concatenate the serialized files and read the combined file using a SnappyInputStream.  Does SnappyOutputStream support this use-case?  If not, is this a prohibitively difficult feature to add?  The snappy framing format linked from the Snappy website explicitly supports this use-case, but I think that SnappyOutputStream uses a different format.

@xerial not sure how feasible or likely it is for this to happen, but it'd help tremendously Spark's performance because we are experimenting with a new shuffle path that uses channel.transferTo to avoid user space copying. However, for that to work, we'd need the underlying format to support concatenation. As far we know, LZF has this property, and Snappy might also have it (but snappy-java implementation doesn't support it).

Currently SnappyOutputStream doesn't support this use case, but SnappyOutputStream's data format is simple:
(header) (compressed size, compressed data)*

So it would be easy to let SnappyInputStream read concatenated chunks like this:
(header) (compressed size, compressed data)* ...   (header) (compressed size, compressed data)*

Just adding header check code before reading a compression block.
I think SnappyFramedInput/OutputStream implementation is not matured (#81) compared to SnappyOutput/InputStreams. So extending SnappyInputStream to support reading concatenated chunks is the easiest approach.

@JoshRosen @rxin
Created snappy-java-1.1.2-SNAPSHOT, which supports reading concatenated data of SnappyOutputStreams:
https://oss.sonatype.org/content/repositories/snapshots/org/xerial/snappy/snappy-java/1.1.2-SNAPSHOT/
Does this extension of SnappyInputStream satisfy your use case?

@xerial I'll test out the snapshot this afternoon, but based on reading the code it looks like this will address our use-case.
Does snappy-java provide a programmatic way to access its version number?  I'd like to be able to detect at runtime whether we're using a snappy-java version that supports reading concatenated data, since I'm worried about scenarios where Spark ships with 1.1.2+ but a dependency conflict forces an earlier version to be used; in these cases, it would be helpful to be able to detect that we're using an older version and fall back to an old code path which doesn't perform the concatenation.

@JoshRosen
Use Snappy.getNativeLibraryVersion() and snappy-java-1.1.2-SNAPSHOT will return "1.1.2" (string).
But due to a packaging failure, which is fixed in b1b8276, the previous versions may return "unknown".
I updated 1.1.2-SNAPSHOT jar with that fix.

The SnappyFramedInputStream also supports reading multiple framed streams concatenated together.
As for @xerial's reference to issue (#81), I do not see any actual issue. There is a difference in the way memory is allocated that significantly impacted a "fake" benchmark (reading/writing a single byte) and has little or no applicability in real use.

@bokken, in our use-case we're always going to be performing fairly large bulk writes, so it sounds like we shouldn't expect to see a performance penalty from switching to the framed stream.  I might actually prefer to take that approach since that won't require a dependency upgrade or introduce the potential for dependency conflicts to break our concatenation.  @rxin, do you remember why you opened #81 or otherwise anticipate problems with switching Spark to use the framed stream?

It might not be a problem anymore with sort-based shuffle, since at any given time we have only one stream.

@bokken @rxin
That is good to know. I'll close #81.
Note that an expected overhead in SnappyFramedOutputStream is the computation of crc32, and there might be a portability problem since SnappyFramedOutputStream relies on sun.misc.Cleaner, which is not available on some platform (e.g., Android). But this would not be a problem unless @rxin's joke on April fool becomes a reality :)
https://databricks.com/blog/2015/04/01/spark-2-rearchitecting-spark-for-mobile.html
Anyway, I'll deploy snappy-java-1.1.2 (-RC1) to Maven central for the ease of your testing.

@xerial, sun.misc.Cleaner is only used via reflections if present. If not present, there is simply no aggressive reclamation of direct byte buffers (i.e. native memory).

@bokken Thanks for the clarification. Then it's safe at any platform.

@JoshRosen
Just deployed snappy-java-1.1.2-RC1, which will be available soon.
Now let me close the ticket. If you find any problem, please open a new ticket.


VoodooPS2Controller version

Hi I noticed that you are using a new VoodooPS2Controller version. However I cannot find it anywhere else on the web. Is this something that you have build it specifically for the Xiaomi Pro.
I have a XiaoMi-Air-6200U
Thanks!

Hi, you can find source code of most kexts in the Credits section of [[README.md]]. New version of VoodooPS2’s source code is at https://github.com/acidanthera/VoodooPS2

thanks @stevezhengshiqi for the super fast reply!!!!!!!


Possible issue in UserHasTeams Trait?

Hi there,
I'm toying around with your package and came across an issue, if one uses the package together with eg. the Entrust role based permission package.
If you add the EntrustUserTrait and the UserHasTeams Trait to your User Model, you'll cause an fatal error with the following message:
"Trait method boot has not been applied, because there are collisions with other trait methods on App\User"
Because both Traits have a boot method to reset the many to many relation on deletion. I'm not sure, if one can omit this by renaming the boot method to:
bootUserHasTeams
which should be called automatically by Laravel and don't call the parent boot method in that one. Also I'm not sure, if this keeps the desired behavior and works as expected?
And yes I could have asked the same question in the Entrust repo, but I thought this one is better maintained :)

Hi,
thanks for reporting this issue!
I fixed it by using bootUserHasTeams as you suggested. :+1:

Hi,
great! Glad to help :+1:
So, it still works as expected?
Came to another possible issue in the same method. I think in L5.* the Trait for the SoftDelete feature is called SoftDeletes and not SoftDeletingTrait like in L4.2, so the method body has to change to:
if ( !method_exists( \Config::get( 'auth.model' ), 'bootSoftDeletes' ) )
Or am I going in the wrong direction?

Yes, you're right. Fixed it :)


Default value for required variables

Just confirming that plugman already handles the default attribute on preference tags before I merge this in.

Here's the JIRA issue which added this support: https://issues.apache.org/jira/browse/CB-9162

Awesome. Thanks for the commit!


Page scheduling, FirstPublishedDate = NULL, Page doesn't  display at top of Highlighted Content

When I setup approval and page scheduling in the SitePages library of the Microsoft Learning Pathways solution provisioned through the provisioning service. The page is published on the scheduled date, but the First Published Date column contains a null value so the page does not display at the top of the Highlighted Content web part. Is this a know problem? Is there a solution? I have bee setting the dates prior approval with powershell, but I am looking for a more automated process. Thanks - Greg

@dcashpeterson - I posted here on the recommendation Premier Support.  While working with Support, scheduling + approvals worked as expected on two other non-SP provisioned sites.

@sympmarc - Thanks. It certainly does walk like a bug and talk like a bug :-)

Closed as answered. If you are still having an issue please create a new issue.


TileManager cleanup + use make_unique

GetInstance did nothing but create new instances

Looks good to me, cheers to c++14!

I agree.


Can't load images.

Hi, I want to run the following command:
CUDA_VISIBLE_DEVICES=0 python -m paddle.distributed.launch --selected_gpus=0 train_with_fleet.py --batch_size=128 --model=ResNet50 --total_images=50000 --data_dir=/data/PaddlePaddle/imagenet/
but I get 'NoneType' object has no attribute 'shape', which means that probably images couldn't be found.
Inside imagenet directory I have train_list.txt and train dir, where in train_list.txt I point to images inside train direcotry. This works just fine for PaddlePaddle/models/ repository when I run resnet50 from there. Here it doesn't work. At first I got error that train.txt doesn't exist, so I created it, just renamed my previous train_list.txt, but it still doesn't work.
Could you clarify what the structure of the directory with dataset should look like??

Please print the first few lines of train.txt.

For example:
train/ILSVRC2012_train_00049311.JPEG 61
train/ILSVRC2012_train_00049312.JPEG 834
train/ILSVRC2012_train_00049313.JPEG 87
train/ILSVRC2012_train_00049314.JPEG 451

so basically
<path> <int: category>
is this how this file should look like?
It works just fine for PaddlePaddle/models as I said, seems to me that you guys use cv2-based reader there as well, but maybe there are some differences.
I wanted to avoid investigating it myself, hence the issue :)

No need for the prefix 'train/', just like the following:
ILSVRC2012_train_00049311.JPEG 61
ILSVRC2012_train_00049312.JPEG 834
ILSVRC2012_train_00049313.JPEG 87
ILSVRC2012_train_00049314.JPEG 451

So now the structure is the following:
imagenet directory contains:
train.txt file and train directory, where train.txt has just img names as you proposed, with no train/ prefix.
Well, it doesn't work, same error as before. Deleting the prefix seems weird to me as previously <path to train.txt> + whatever was in train.txt concatenated resulted in absolute path to the image, now it doesn't because we skipped the train directory.
My question is - why can't there be one unified loader and dataset structure Paddle requires? Even this small change in file name, from train_list.txt to train.txt seems unnecessary to me.

Sadly it doesn't work. I also tried running it with old structure of dataset but with the following changes in the reader_cv2.py:
if mode == 'train':
                    img_path, label = line.split()
                    #img_path = img_path.replace("JPEG", "jpeg")
                    img_path = os.path.join(data_dir, img_path)
                    yield (img_path, int(label))

Next week I'll launch debugger on that,  but for now could you tell me what exactly is your dataset structure? Are you able to run it no problem?

No need for the prefix 'train/', just like the following:
ILSVRC2012_train_00049311.JPEG 61
ILSVRC2012_train_00049312.JPEG 834
ILSVRC2012_train_00049313.JPEG 87
ILSVRC2012_train_00049314.JPEG 451

Sorry for my fault, the prefix 'train/' is needed.
In conclusion, the imagenet directory contains the train.txt file and the train directory. Each line of train.txt contains the path for an image and its corresponding label, like
train/ILSVRC2012_train_00049311.JPEG 61
train/ILSVRC2012_train_00049312.JPEG 834
train/ILSVRC2012_train_00049313.JPEG 87
train/ILSVRC2012_train_00049314.JPEG 451


Fixups localvideo

Significant improvements to local video.  Closes #29 and also speeds up classification loop.
Before

After

Looks awesome! 👍🏼


Add support for UDP_GRO

Motivation:
Support was added for UDP_SEGMENT in #2372 which allows for large UDP datagrams to be written to a socket by letting the kernel or NIC segment the data across multiple datagrams. This reduces traversals across the network stack which can lead to performance improvements. UDP_GRO is the receive-side counterpart allowing the kernel/NIC to aggregate datagrams and reduce network stack traversals.
Modifications:

Add a function in CNIOLinux to check whether UDP_GRO is supported
Add the relevant socket and channel options
Add tests

Result:

UDP_GRO can be enabled where supported and applications may receive large buffers.

These tests are failing for me on Ubuntu 20.04 x86_64 and the CI is not running them, which may be why that was missed:
DatagramChannelTests.testChannelCanReceiveLargeBufferWithGROUsingScalarReads : Test skipped: required false value but got true - UDP_SEGMENT (GSO) is not supported on this platform

This is the error I see when they are run locally:
/home/butta/swift-nio/Tests/NIOPosixTests/DatagramChannelTests.swift:1378: error: DatagramChannelTests.testChannelCanReceiveLargeBufferWithGROUsingScalarReads : XCTAssertEqual failed: ("1000") is not equal to ("10000") -Test Case 'DatagramChannelTests.testChannelCanReceiveLargeBufferWithGROUsingScalarReads' failed (0.007 seconds)

Thanks for this. We’re tracking this on our side as well, as our internal CI platform has also flagged this regression.

Also as a breadcrumb, can you provide the output of uname -a?

Linux Butta-server 5.4.0-132-generic #148-Ubuntu SMP Mon Oct 17 16:02:06 UTC 2022 x86_64 x86_64 x86_64 GNU/Linux

10/10 for the hostname.

We also see failures on 5.4.
FWIW these tests do pass with a newer kernel (5.15).


Repo setup

Codecov Report

:exclamation: No coverage uploaded for pull request base (main@bbee367). Click here to learn what that means.
Patch has no changes to coverable lines.

Additional details and impacted files
@@           Coverage Diff            @@
##             main        #1   +/-   ##
========================================
  Coverage        ?   100.00%
========================================
  Files           ?         1
  Lines           ?        25
  Branches        ?         0
========================================
  Hits            ?        25
  Misses          ?         0
  Partials        ?         0

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Integrate TravisBuddy

TravisBuddy will comment on pull requests in your public repository everytime a test failed in one of them. The comment will include only the part of the build log that applies to your testing framework, so your contirbutors won't have to enter Travis's website and search the long and annoying build log for the reason the tests failed.
Here's an example: https://github.com/bluzi/express/pull/1

Hey @bluzi.  Thanks for the contribution.  Personally, I am not a fan of this change.  This repo already generates tons of emails to many of the >1000 people who follow it.  Adding to those emails just to say tests failed seems like needless noise to me.  Just my 2c.

We could probably set Travis CI to just directly email the commit author or something (or maybe TravisBuddy could do that?) which would cut down the noise. Every comment in the repo currently emails 1,772 people.

That would be better IMO.  @bluzi is that something you can setup with TravisBuddy?

Let me see if I understand you correctly: You want TravisBuddy to send an email to the author of the PR instead of comment on the PR?

Yes, because everyone who watches the repo should not receive an email for every test failure on a PR.

We can setup Travis CI to do this out of the box, though it doesn't contain the snippet of the build log like TravisBuddy does (just an email with the status and a link to the build).

What I also noticed was in yiur example, the comment is really really long, and just a few of those comments will quickly make it hard to scroll to find conversation comments. I'm not sure if that is the comment length to expect in this repository or not, just noticed it in the example. I have to swipe many times on my phone just to get past the comment.

The example comment could use a collapsable section in markdown: https://github.com/dear-github/dear-github/issues/166#issuecomment-236342209

That's awesome @wesleytodd, it'll be used for sure, have no doubt.
Thanks a lot!

Hey guys,
I wanted to let you know that collapsible sections has been added to TravisBuddy's comments.
Here's an example on my fork of this repository: https://github.com/bluzi/express/pull/3
The comment in the example is the longest possible, as the PR breaks the tests in every Node version.
Mostly it'll be even shorter.
Let me know what do you think, so far your comments are very helpful :)

Hey guys, I wanted you to know that TravisBuddy is running for about a week now with expandable sections, so the comments are much shorter now.
Some live examples:
https://github.com/HoeenCoder/Wavelength/pull/107
https://github.com/didi/cube-ui/pull/47
https://github.com/Zarel/Pokemon-Showdown/pull/4235
We're planning on making the comments even shorter in the near future, for example by merging similar logs.

I still think this does not resolve the issue of emailing the entire follower list for random PR failures.  It looks like a decent tool for smaller projects, but not something I would want on the express repo.

@bluzi Only part about it I really could care less about is how it kinda tosses it in your face, "You are a failure," now don't get me wrong I love my boy Travis

@wesleytodd

I still think this does not resolve the issue of emailing the entire follower list for random PR failures. It looks like a decent tool for smaller projects, but not something I would want on the express repo.

Maybe we could work something out, let me dig in GitHub's notification preferences a bit more, I'll get back here if I'll find a solution.
 Could we handle it by using different wording?

Would you mind moving that conversation off this thread, to avoid the exact issue which I am trying to prevent (email spam)?

Like you see its a really huge popup not like a simple message sort of ordeal seems a bit "in-your-face" about it so to say @bluzi but yeah this is an amazing thing for GitHubs from my personal experience and I'm glad Zarel added it onto Pokemon-Showdown so I saw it (which is why Wavelength and a few others that actually update their GitHub's actually have it now)

Hey guys,
Just wanted to keep you posted on some progress we've made in the past few months, the project is more mature and we were focusing mainly on reducing spam, so I figured you might want to reconsider:

First, we now use collapsible sections as @wesleytodd suggested, we've been using them for few months now and it works great
We are now only commenting the parts of the log that are relevant to the failure, meaning only the logs of the test scripts that failed. This reduces a lot of the comment size.
We now support update mode. When update mode is enabled, @TravisBuddy will comment once per pull request, and will update its single comment every time a build is finished. Hopefully this feature will eliminate your concerns about email spam.
We also support custom comment templates, and many other features.

Let me know what do you think.
@wesleytodd @dougwilson

This repo no longer uses Travis CI.


marimo integration with Safe Exam Browser

Description
Hi. For conducting teaching and assessments using marimo, it is becoming critical to have some configuration that would preclude students from cheating during assessment moments.  Launching the Safe Exam Browser software (SEB) in the Moodle platform is a well-known process for implementing that security safeguard.  However, the Moodle platform has severe limitations if we want to use a notebook like marimo. Is it possible to create a configuration that would preclude students from opening a new window when marimo is running during a test/exam? For teaching/grading, this functionality would be a game changer.
Suggested solution
Create a configuration that would preclude students from opening a new window when marimo is running during a test/exam.
Alternative
No response
Additional context
No response

I understand.  This problem will become a big challenge for those who use notebooks (jupyter, marimo, pluto, and the like) in their teaching and assessment moments when many students are involved. Teaching using a notebook and taking the tests on paper does not make sense.  Thank you very much for clarifying this issue.

@VivaldoMendes Did you use Jupyter before marimo? What did you do before for mimicking "safe exam browser"?

@mscolnick  I stoped using Jupyter in 2020, on a regular basis because of no-reacticity. I have been using Pluto quite a lot since then, and since last July I have also been using marimo. Currently, I have two courses running on Pluto notebooks, and I tried to implement some form of integration between Pluto and Safe Exam Browser, but I failed.

Thanks for the explanation. I am going to move this to a discussion since I don't think there is any action/solution on our end at the moment.


changing transformers $value checks to use empty()

Instead of simple null checks. key's can't (shouldn't) be anything that empty() would return true for.
Simply checking for null === $value means that an empty string was being tried as the key during transformation to an entity. Other values such as 0 should not be used as keys in a database either. changing the empty() fixes this and prevents an erroneous form validation error.

That makes sense, thanks.
Merged.


treemap 设置 rich 后整体label不水平垂直居中

Version
5.0.2
Steps to reproduce
1、打开地址：https://echarts.apache.org/examples/en/editor.html?c=treemap-disk
2、输入以下option：
option = {
    series: [{
        type: 'treemap',
        name: '中国',
        leafDepth: 1,
        label: {
            formatter: '{b}',
            rich:{}, // 只要加上这句话，即使rich是空的，定位也会不居中
        },
        data: [{
            name: '河北省',            // First tree
            value: 10,
            children: [{
                name: '邯郸市',       // First leaf of first tree
                value: 4
            }, {
                name: '保定市',       // Second leaf of first tree
                value: 6
            }],
        }, {
            name: '河南省',            // Second tree
            value: 20,
            children: [{
                name: '郑州市',       // Son of first tree
                value: 20,
                children: [{
                    name: 'xxx县',  // Granson of first tree
                    value: 20
                }]
            }]
        }]
    }]
};

3、只要加上了 rich: {}，即使是空的，label定位也有问题。不加的时候是水平垂直居中的，加上后变为lefttop 布局，设置 verticalAlign: "middle"  也不生效
What is expected?
希望“河北省”、“河南省”在块中是居中的
What is actually happening?
实际上跑到左上角了

I'm really need the rich function, it is very important to me, please give some answer~

Met the same issue after upgrade to 5.0.2. The rich with verticalAlign: middle doesn't work.

Get a solution (the verticalAlign is 'top' by default):
        labelLayout: (params) => {           return {             y: params.labelRect.y,             align: 'center',           }         },

@89466598946659
It is a option problem.
Try the following configuration。
{
series: [{
type: 'treemap',
name: '中国',
leafDepth: 1,
label: {
position: ['50%', '50%'], // I think it will help you
formatter: '{b}',
rich:{},
},
data: [{
name: '河北省',            // First tree
value: 10,
children: [{
name: '邯郸市',       // First leaf of first tree
value: 4
}, {
name: '保定市',       // Second leaf of first tree
value: 6
}],
}, {
name: '河南省',            // Second tree
value: 20,
children: [{
name: '郑州市',       // Son of first tree
value: 20,
children: [{
name: 'xxx县',  // Granson of first tree
value: 20
}]
}]
}]
}]
}

Get a solution (the verticalAlign is 'top' by default):
 labelLayout: (params) => { return { y: params.labelRect.y, align: 'center', } },

thanks，but what I want is like this: the label words in the red rect.

@89466598946659
It is a option problem.
Try the following configuration.
{
        series: [{
                type: 'treemap',
                name: '中国',
                leafDepth: 1,
                label: {
                    position: ['50%', '50%'], // I think it will help you
                    formatter: '{b}',
                    rich:{},
                },
                data: [{
                    name: '河北省',            // First tree
                    value: 10,
                    children: [{
                        name: '邯郸市',       // First leaf of first tree
                        value: 4
                    }, {
                        name: '保定市',       // Second leaf of first tree
                        value: 6
                    }],
                }, {
                    name: '河南省',            // Second tree
                    value: 20,
                    children: [{
                        name: '郑州市',       // Son of first tree
                        value: 20,
                        children: [{
                            name: 'xxx县',  // Granson of first tree
                            value: 20
                        }]
                    }]
                }]
            }]
    }

thanks, really, I have two questions for this.
first: The lable is not align center
second: What I want is the red rect. to the yellow rect.

I have the same problem, is there a solution?

您的邮件已收到！O(∩_∩)O！--------Pan

I have the same problem, is there a solution?

No solution. Change another charts.😂

Looks the width and height are set to rectangle width and height to truncate texts in https://github.com/apache/echarts/blob/master/src/chart/treemap/TreemapView.ts#L1004 . But when using rich text it will also affect the layout.

ecomfe/zrender/pull/905

this commit contains two contents:

fix current issue by #ecomfe/zrender/pull/905
fix another problem when setting the width and height of treemap.label, but not work.

and added a test case about them.

i fix it by set  offset: [0, 5], but it not a good solution

您的邮件已收到！O(∩_∩)O！--------Pan

Has the problem been solved？I also can't perfectly center after using rich

您的邮件已收到！O(∩_∩)O！--------Pan

您的邮件已收到！O(∩_∩)O！--------Pan


Master

Cool thanks. I am gonna test these first and then merge them. Thanks for your work Ollie, it looks great.

Just found an error on line 4 which was truncating some dependencies. I have now fixed and am testing again. I will report back when it's done in a few hours.

i tried the lower half of it earlier today - skipped the bitcoin install bit and ran the electrum and btchip part .. and although it ran there was no electrum icon in desktop menu - so it needs work.  Its probably bettter to install with pip as per electrum.org ... also the btchip stuff did not seem to work - i tried loading a ledger wallet with electrum but got errors and reported them to Ollie

oh you got some errors
On Fri, Apr 22, 2016 at 9:12 PM, Open Provenance<EMAIL_ADDRESS>wrote:

i tried the lower half of it earlier today - skipped the bitcoin install
bit and ran the electrum and btchip part .. and although it ran there was
no electrum icon in desktop menu - so it needs work. Its probably bettter
to install with pip as per electrum.org ... also the btchip stuff did not
seem to work - i tried loading a ledger wallet with electrum but got errors
and reported them to Ollie
—
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub
https://github.com/MrChrisJ/fullnode/pull/26#issuecomment-213574180

i didn't comment who did this ?

Not me

sorry yes i did write this lol i thought it was the 23rd of march but its 23rd of april 2016 lol


Reimplement proof verifier

Codecov Report

Merging #24 (7f9e382) into main (607eb47) will decrease coverage by 0.11%.
The diff coverage is 96.45%.

@@            Coverage Diff             @@
##             main      #24      +/-   ##
==========================================
- Coverage   96.89%   96.78%   -0.12%
==========================================
  Files           6        5       -1
  Lines         387      373      -14
==========================================
- Hits          375      361      -14
  Misses         10       10
  Partials        2        2

Impacted Files
Coverage Δ

proof/verify.go
94.62% <94.62%> (ø)

compact/nodes.go
85.71% <100.00%> (+1.09%)
:arrow_up:

compact/range.go
100.00% <100.00%> (ø)

proof/proof.go
97.14% <100.00%> (+0.20%)
: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 607eb47...7f9e382. Read the comment docs.


Use correct class name in estimator tutorial

Summary
The estimator tutorial text still says PauliSumOp even though opflow has been deprecated and the code doesn't actually use it. Updated to match the code.
Details and comments
Fixes #

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

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

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
3414

Relevant Lines:
5082

💛  - Coveralls


fix(client): ensure __runTest is used

fix: load framerunner after learner code
fix: add script id for testing

Checklist:

[x] I have read and followed the contribution guidelines.
[x] I have read and followed the how to open a pull request guide.
[x] My pull request targets the main branch of freeCodeCamp.
[x] I have tested these changes either locally on my machine, or GitPod.

The motivation was to prevent __runTest getting overwritten, but I also like the distinction between test runner and dependencies. I think separating them makes their roles clearer.

I do not see anything wrong with this, but I have been trying to understand the workflow.
I see this change really only affects the backend -style challenges. In which case, there is no user input code - all of the code in the test runner is from the Markdown.
So, assuming the backend challenges managed to work in the browser with Camper-code, Campers still could:
document.getElementById("fcc-test-runner").remove();

Am I misunderstanding this?

I see this change really only affects the backend -style challenges

It affects all challenges that run tests in the browser except JS-only challenges, since they have their own test runner.

Campers still could:
document.getElementById("fcc-test-runner").remove();

Not as easily, since the test runner doesn't exist at the time when the user's code is evaluated.
So, in production the order of execution is

frame-runner
user code
__initTestFrame called
tests use __runTest

in this PR it's

user code
frame-runner
__initTestFrame called
tests use __runTest

I doubt this is insurmountable, but I like this way structuring the code more and it is somewhat harder to mess with the test suite.


Incompatibility with Carpet /player

Please can you fix the incompatibility that happens when using this(and the gunpowder base) with Carpet?

Carpet is known to break many mods and thus not supported. If you give a crash log I can take a look, but otherwise I can't do anything.

From experience it hasn't broken any mods that I've used. LuckPerms broke Carpet.
However, I can't give you a log so I'll close this.
Sorry for a late response.


Template Library

Added filtering by latest version and by cicero version.

Coverage increased (+0.08%) to 91.334% when pulling f006cb10d562bafad97153c20b18aa14723b84f7 on dselman:nologic into 3475dc1c28c0a9ba5fe63d2f7b9ac871ab9e15d2 on accordproject:master.


Add .dockerignore file.

Ignore cargo target build directory when adding files to docker image.

This line in the Dockerfile copies all files from the repo into the docker image:
https://github.com/fedimint/minimint/blob/83e16321ea6328b1691222f768873534ea666d3d/Dockerfile#L2
Some of these files/directories (like .git and target) are unnecessary. They can be several gigabytes in size. .dockerignore solves this problem, see here.


Apply Choco Best Practices

As per suggestion in #15

Love this. Maybe we should have a page that lists best practices. Where did you find the choco best practices?

I don't see any "' vs '" inconsistencies any more, would like to merge this to master.  Objections?

I think we have most of these mentioned in places across the website, but not in one complete list.  Perhaps that is something that we need to add?
They are summarised in the issue that @ferventcoder created here: https://github.com/Microsoft/windows-dev-box-setup-scripts/issues/15 but I will try to get a page created on chocolatey.org that we can also point people at.

@crutkas I am not seeing the inconsistency that you refer to.  Can you point me to it, or comment on the line in question in the PR and I can get it fixed.  Thanks.

Clearly My eyes need checking as I could have swore I saw the quote vs tick flipped.

@crutkas @yodurr are we good to get this merged?

@yodurr thanks for merging!


login popups redirect to the first tab after login

Describe the bug
Whenever I lwant to login to a website and the website has one of the other methods like google, GitHub or twitter for login then the browser creates a new tab for me to login. after successful login the tab closes and redirects me to the first tab in my opened tabs list. Even after successful login from the sources the website still asks me to wait for login. everytime I get sent to the first tab instead of the tab that requested that login.
How to Reproduce
Steps to reproduce the behavior:
Make sure there are multiple tabs opened in the browser already.

Go to any website that has google, github login etc
Click on one if them
In the new tab enter the login id and password or whatever is required and wait.
It will close the tab and send you to the very first tab in your tabs list

Expected behavior
Should work normally and send me to the website tab with login done.
Smartphone (please complete the following information):

DDG App Version: 5.57.1
Device: One Plus 6
OS: Android 10

In this gif you can see the bug I'm talking about. In this case the login was successful but the redirect was done to the first tab. sometimes this causes login not to complete.

Thanks, we have some issues open related to our navigation between tabs and I think this is related to that odd behavior we still need to fix. Like this one https://github.com/duckduckgo/Android/issues/815 (not the same but the root cause could be the culprit).
Thanks for that gif, really love it when people share some visuals here, really helpful. Hope we can fix it soon!

yes, indeed it is one of the navigation issues. Only if I worked on android I'd have tried looking into it but I guess I'd just leave it to you guys to fix it. Also, thanks for the amazing browser, I try to urge people to start using it instead of others. Keep up the good work. 🙌

We introduced a few improvements in our navigation between tabs. However, I've just tested this scenario again, and the described issue is still there.
It seems we only solved the cases when the tab is closed by the user, but not when the tab is closed by the site itself.

https://app.asana.com/0/0/1199514257586721/f

https://app.asana.com/0/0/1199514257586721/f

Fixed by https://github.com/duckduckgo/Android/pull/1053
Thanks @tvalle

Fixed by https://github.com/duckduckgo/Android/pull/1053
Thanks @tvalle


Could not load JSON source, please review the documentation and try again

Hello, please i need your help, I'm using
<iframe src="//api.apiembed.com/?source=http://localhost/test/demo.json&targets=all" frameborder="0" scrolling="no" width="100%" height="300px" seamless></iframe>
but it gives me this error
Could not load JSON source, please review the documentation and try again
my jsonfile is saved locally, I am stilling in localhost, and it's as follows:
`{
"method": "POST",
"url": "http://localhost/test/web/app_dev.php/v1/photos",
"httpVersion": "HTTP/1.1",
"queryString": [
{ "image": "photos/1.jpg", "value": "photos/1.jpg" }
],
"headers": [
{ "name": "Accept", "value": "application/json" },
{ "name": "Content-Type", "value": "application/json" }
],
"postData": {
"mimeType": "application/json",
"image": "{"photos/1.jpg": "photos/1.jpg"}"
}
}
`

you can't use localhost urls with this online service.


Add a simple binary?

I’m not sure if that is possible, but I’d find it useful if I could generate an import map like this:
npx @jspm/generator node_modules/

Did you mean, one importmap for every package in node_modules?
hypothetically considering following an entry point for the generated importmap for each package?
import package from 'package'

Duplicate of https://github.com/jspm/generator/issues/64. There will eventually be a CLI relaunch when the timing is right.


GraphQL POC

I haven't replaced the REST logic for now but have created complementary graphql equivalents.
To test:

Make sure you have wpgraphql installed on your WP instance (https://www.wpgraphql.com/) - happy to support with this if necessary.
Add a .env variable GRAPHQL_URI= (once you've set up WPGraphQL this will be WP_ROOT/graphql
Start Next
Navigate to /postlist

You should a list of all the posts on your WP instance

Click on each of the posts

This should bring you to a properly rendered page for each post.

@all-contributors please add @doingandlearning for code


feature request: enable EBS default encryption at the account > region level

Hello,
It would be nice to have a feature in org-formation that enabled default EBS encryption.  Default encryption is enabled/disabled per region in a given account.

https://aws.amazon.com/blogs/aws/new-opt-in-to-default-encryption-for-new-ebs-volumes/
https://docs.aws.amazon.com/cli/latest/reference/ec2/enable-ebs-encryption-by-default.html

see: https://github.com/OlafConijn/AwsOrganizationFormation/issues/83
there might be a bunch of other account/org/region level settings come to might. will definately put this one top of the list too. :)

hi, we are implementing a library of resource providers and this will be one.
A resource provider can be deployed to your AWS account/region and allows you to extend the CloudFormation types.
This will look like:
AWSTemplateFormatVersion: 2010-09-09
Resources:
  EbsEncryptionDefaults:
    Type: 'Community::Organizations::EbsEncryptionDefaults'
    Properties:
      EnableEbsEncryptionByDefault: true

Using org-formation you can apply these resources to multiple accounts/regions using an organization binding.
The resource providers still need some polishing and a method to easily deploy these types.
The repository can be found at: https://github.com/org-formation/aws-resource-providers.


Command to extract and install client files.

As seen in #15.

It would be neat if this feature could be implemented with Automodpack in mind by extracting both client and server stuff into different directories. That way they can be served to the client and a user would only need a very basic setup to join an Automodpack-enabled server.

@limejet thanks for your input!
I have never used AutoModpack, are there some specific things I have to care about or is it sufficient or is it sufficient if I expose 2 cli flags to defined the 2 paths?

By default automodpack just sends files in mods/ and config/  (configurable in automodpack/automodpack-server.json) as well as everything in automodpack/host-modpack. This allows for sending shaderpacks and things of that nature to the client without polluting the server instance with unused files.
A --client flag should be enough to get a working setup, because it could be done in two steps:

mrpack-install <pack-name> /path/to/minecraft/  to set up the instance
something like mrpack-install <pack-name> --client /path/to/minecraft/automodpack/host-modpack

Maybe with the addition of a flag like --data-only (or some better name) that would prevent files not in the mrpack (server.jar and such) from being downloaded.

Okay this sounds good, I gonna try to implement this in a clean way that is nice to use.


How to make workable in android browser

I used it into my code and it works fine in the Desktop browser, got the latitude and longitude but it is not getting the latitude and longitude when I open the app from my mobile browser. Can you help me with this?
I tried the demo site you provided (https://no23reason.github.io/react-geolocated/) from my mobile browser and it shows me the latitude and longitude but don't know why it's not working for my react app.

Hi, so if I understand correctly, the demo page works in your browser, but your app does not? Can you please share how you use the geolocated functions?

hi , i have same issue when run my app in my desktop ok, but when open it with safari on my iphone isGeolocationEnabled is false but my location service for safari is on?
this is my code :
return (
{props.isGeolocationAvailable?
 support
 :
not support
 }
    {props.isGeolocationEnabled ?
 enable
:
 not enable
}
    {props.coords?
lat:{props.coords.latitude} lng:{props.coords.longitude}
 :
 geting locations
}
    <GoogleMap
        onLoad={loadHandler}
            mapContainerStyle={containerStyle}
            center={{ lat: props.prop.locations.YPOINT, lng: props.prop.locations.XPOINT }}
            zoom={18}
            options={{ gestureHandling: 'greedy', }}
            onClick={e => {
                //    setClickedLatLng(e.latLng.toJSON());
                   setIsOpen(false) }}
        >
</Google>)

export default  geolocated({
    positionOptions: {
        enableHighAccuracy: true,
    },
    userDecisionTimeout: 2000,
})(React.memo(CurrentLocationMap));

Hi, does the demo page work for you? https://no23reason.github.io/react-geolocated/ if not, there might be some problem with iOS permissions on your device. I tried it just now on my iPhone and it works fine.

@no23reason Finally worked. Did nothing, just made the domain of the app SSL certified. Thanks anyway. I am closing that issue.


No implicit conversion of nil into String (TypeError)

Trying to convert some of the Veewee Windows templates to packer, I get the following error. I'm running master of veewee-to-packer rather than the version on gems, but the same occurs in both. Is this likely to be a problem with the template itself (others work successfully)?
$ bundle exec veewee-to-packer templates/windows-2012R2-serverdatacenter-amd64/definition.rb
/opt/rubies/2.1.2/lib/ruby/gems/2.1.0/bundler/gems/veewee-to-packer-e34d64b8f5de/lib/veewee-to-packer.rb:80:in `gsub': no implicit conversion of nil into String (TypeError)
	from /opt/rubies/2.1.2/lib/ruby/gems/2.1.0/bundler/gems/veewee-to-packer-e34d64b8f5de/lib/veewee-to-packer.rb:80:in `block in convert'
	from /opt/rubies/2.1.2/lib/ruby/gems/2.1.0/bundler/gems/veewee-to-packer-e34d64b8f5de/lib/veewee-to-packer.rb:78:in `each'
	from /opt/rubies/2.1.2/lib/ruby/gems/2.1.0/bundler/gems/veewee-to-packer-e34d64b8f5de/lib/veewee-to-packer.rb:78:in `convert'
	from /opt/rubies/2.1.2/lib/ruby/gems/2.1.0/bundler/gems/veewee-to-packer-e34d64b8f5de/bin/veewee-to-packer:43:in `<top (required)>'
	from /opt/rubies/2.1.2/lib/ruby/gems/2.1.0/bin/veewee-to-packer:23:in `load'
	from /opt/rubies/2.1.2/lib/ruby/gems/2.1.0/bin/veewee-to-packer:23:in `<main>'

Fails because it is looking for an undefined :ssh_password in the Windows templates. A workaround would just be to add :ssh_password => "", to your definition.rb file for the specific template.
veewee-to-packer.rb:80 should null check (definition[:ssh_password] || "") or definition[:ssh_password].to_s. I believe that would resolve the issue.


AndroidManifest.xml could not be found.

Hi, I'm trying to run an example (after git cloning ).
Getting following error:
Launching lib/main.dart on Android SDK built for x86 64 in debug mode... AndroidManifest.xml could not be found. Please check /home/dominik/Flutter_projects/flutter-expandable/example/android/AndroidManifest.xml for errors. No application found for TargetPlatform.android_x64. Is your project missing an android/AndroidManifest.xml? Consider running "flutter create ." to create one.
How can I run it?

It tells you right there: flutter create .
You need to run that command because it generates platform-specific files
that aren’t necessary to put into git.
On Tue, Jan 7, 2020 at 3:50 PM Dzunior<EMAIL_ADDRESS>wrote:

Hi, I'm trying to run an example (after git cloning ).
Getting following error:
Launching lib/main.dart on Android SDK built for x86 64 in debug mode...
AndroidManifest.xml could not be found. Please check
/home/dominik/Flutter_projects/flutter-expandable/example/android/AndroidManifest.xml
for errors. No application found for TargetPlatform.android_x64. Is your
project missing an android/AndroidManifest.xml? Consider running "flutter
create ." to create one.
How can I run it?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/aryzhov/flutter-expandable/issues/40?email_source=notifications&email_token=AAEW6EQKVXYHEXRTQIIMXBDQ4UINNA5CNFSM4KEAUUNKYY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4IET7GIA,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAEW6EQXALPKF3OOU5P6EPDQ4UINNANCNFSM4KEAUUNA
.

when i run "flutter create ." i have this erorr
Oops; flutter has exited unexpectedly: "FileSystemException: Cannot create file, path = 'C:\Windows\System32.gitignore'(OS Error: Access is denied.
, errno = 5)".
A crash report has been written to C:\Users\anf\AppData\Local\Temp\flutter_tools.b6652c61\flutter_01.log.
what should i do now?


storage: add runtime support for batch chunk

Add the runtime support for small chunk mergence.
basic usage
Add the --batch-size arg to command to enable chunk mergence for supported conversion types:
nydus-image create --bootstrap ~/bootstrap --blob-dir ~/blobs ~/source --batch-size 0x100000

nydus-image create --type estargz-rafs --bootstrap ~/bootstrap --blob-dir ~/blobs ~/source.tar.gz --batch-size 0x100000

nydus-image create --type targz-rafs --bootstrap ~/bootstrap --blob-dir ~/blobs ~/source.tar.gz --batch-size 0x100000

nydus-image create --type tar-rafs --bootstrap ~/bootstrap --blob-dir ~/blobs ~/source.tar --batch-size 0x100000

nydusify convert --source node:latest --target node:latest-nydus-batch-256K --batch-size 0x40000

benchmarks
Tested on 4C4G VM, pulled from private harbor registry in the same Campus Network. Cleaned containerd images and nydus-snapshotter cache before each run. Each value is averaged over five runs.

repositories
command
tags
prefetch
startup e2e time (s)
%

python
python --version
latest-rafs
✔
2.746
-

python
python --version
latest-rafs (second run)
✔
2.995
+9.08%

python
python --version
latest-rafs-batch-64K
✔
2.202
-19.80%

python
python --version
latest-rafs-batch-128K
✔
2.580
-6.05%

python
python --version
latest-rafs-batch-256K
✔
2.272
-17.27%

python
python --version
latest-rafs-batch-512K
✔
2.157
-21.44%

python
python --version
latest-rafs-batch-1M
✔
1.628
-40.71%

python
python --version
latest-rafs
✖
1.369
-

python
python --version
latest-rafs (second run)
✖
1.174
-14.25%

python
python --version
latest-rafs-batch-64K
✖
1.052
-23.15%

python
python --version
latest-rafs-batch-128K
✖
1.081
-21.05%

python
python --version
latest-rafs-batch-256K
✖
1.075
-21.48%

python
python --version
latest-rafs-batch-512K
✖
1.100
-19.62%

python
python --version
latest-rafs-batch-1M
✖
1.100
-19.60%

python
python --version
alpine3.17-rafs
✔
0.784
-

python
python --version
alpine3.17-rafs (second run)
✔
0.803
+2.42%

python
python --version
alpine3.17-rafs-batch-64K
✔
0.811
+3.41%

python
python --version
alpine3.17-rafs-batch-128K
✔
0.818
+4.30%

python
python --version
alpine3.17-rafs-batch-256K
✔
0.764
-2.58%

python
python --version
alpine3.17-rafs-batch-512K
✔
0.776
-0.96%

python
python --version
alpine3.17-rafs-batch-1M
✔
0.750
-4.33%

python
python --version
alpine3.17-rafs
✖
0.850
-

python
python --version
alpine3.17-rafs (second run)
✖
0.911
+7.17%

python
python --version
alpine3.17-rafs-batch-64K
✖
0.805
-5.22%

python
python --version
alpine3.17-rafs-batch-128K
✖
0.834
-1.86%

python
python --version
alpine3.17-rafs-batch-256K
✖
0.813
-4.35%

python
python --version
alpine3.17-rafs-batch-512K
✖
0.829
-2.47%

python
python --version
alpine3.17-rafs-batch-1M
✖
0.805
-5.21%

node
npm -v
latest-rafs
✔
8.395
-

node
npm -v
latest-rafs (second run)
✔
11.928
+42.08%

node
npm -v
latest-rafs-batch-64K
✔
12.704
+51.32%

node
npm -v
latest-rafs-batch-128K
✔
9.614
+14.51%

node
npm -v
latest-rafs-batch-256K
✔
9.164
+9.15%

node
npm -v
latest-rafs-batch-512K
✔
5.605
-33.24%

node
npm -v
latest-rafs-batch-1M
✔
4.192
-50.07%

node
npm -v
latest-rafs
✖
1.593
-

node
npm -v
latest-rafs (second run)
✖
1.595
+0.11%

node
npm -v
latest-rafs-batch-64K
✖
1.576
-1.07%

node
npm -v
latest-rafs-batch-128K
✖
1.506
-5.48%

node
npm -v
latest-rafs-batch-256K
✖
1.517
-4.78%

node
npm -v
latest-rafs-batch-512K
✖
1.543
-3.14%

node
npm -v
latest-rafs-batch-1M
✖
1.589
-0.22%

node
npm -v
19-alpine-rafs
✔
2.829
-

node
npm -v
19-alpine-rafs (second run)
✔
2.667
-5.72%

node
npm -v
19-alpine-rafs-batch-64K
✔
3.423
+21.01%

node
npm -v
19-alpine-rafs-batch-128K
✔
3.097
+9.50%

node
npm -v
19-alpine-rafs-batch-256K
✔
3.249
+14.85%

node
npm -v
19-alpine-rafs-batch-512K
✔
3.054
+7.95%

node
npm -v
19-alpine-rafs-batch-1M
✔
2.792
-1.32%

node
npm -v
19-alpine-rafs
✖
1.372
-

node
npm -v
19-alpine-rafs (second run)
✖
1.414
+3.06%

node
npm -v
19-alpine-rafs-batch-64K
✖
1.392
+1.46%

node
npm -v
19-alpine-rafs-batch-128K
✖
1.396
+1.73%

node
npm -v
19-alpine-rafs-batch-256K
✖
1.430
+4.19%

node
npm -v
19-alpine-rafs-batch-512K
✖
1.393
+1.50%

node
npm -v
19-alpine-rafs-batch-1M
✖
1.411
+2.86%

This PR is related to https://github.com/dragonflyoss/image-service/pull/1202, https://github.com/dragonflyoss/image-service/issues/884, https://github.com/dragonflyoss/image-service/issues/885, https://github.com/dragonflyoss/Dragonfly2/issues/1858

Good work! This seems to be valuable in most use cases.

Can we also give the perf result for wordpress image?
What's the network bandwidth limitation in the tests? It seems that the startup time is slowed down when prefetch all is enabled.
Can we use 1MB as the default --batch-size value?

The images are pulled from a private harbor registry in the same Campus Network with 100Mbps bandwidth. The benchmark is just a rough estimation for container startup time beacuse: (1) The startup time difference between the same image latest-rafs and latest-rafs (second run) shows that it is volatile; (2) The benchmark does not utilize the containers to run workloads, but only checks the versions of python and npm, which may amplify the shortcomings of prefetching, which could occupy network and I/O resources to preload useless data and slow down on-demand requests; ③ The benchmark config follows the default config settings, and the default setting of 8 prefetching threads may be too many for the 4C4G VM and network bandwidth. Anyway, the benchmark is just a rough estimation for presentation. We should precisely evaluate the performance later. Maybe it should be assigned to @guangyuan.

Wordpress depends on an external database and does not have a short inspection command to exit the container after execution, like python --version. We should precisely evaluate the performance of Wordpress later.

It is said that the best practice of the size of read amplification is 128K in datacenter scenarios. Just from a performance point of view, I guess 256K-512K would be good as default value. With good prefetch list assisted at build time, maybe 1M is the best.

@hangvane
Ok, thanks, we can set the default value of --batch-size to 128k/256k/512k/1MB in nydusify to compare the wordpress image with the master branch.
Please also add some tests to passing the codecov test (>=25%).
cc @jiangliu

@imeoer Could you please provide me with some guidance on adding tests? I'm finding it difficult to follow the existing tests and add similar ones.

@hangvane You may refer to some exists tests in pub(crate) mod tests  section.
@adamqqqplay Is it >=25% test coverage too strict for PR?

@hangvane You may refer to some exists tests in pub(crate) mod tests section. @adamqqqplay Is it >=25% test coverage too strict for PR?

@imeoer I think 25% is a pretty loose value, meaning that only a few unit tests need to be added for new code. If the PR is a bugfix, there is usually no need to add unit tests. In Google's best practice, they require at least 60% coverage.
Of course, if it is really hard to add more unit tests, you can also bypass this check and merge PR.

/retest

/retest

/retest


PHP 5.6 compatibility

forget about this :)

Cool only 7.1 dependancies in the lock file are for dev, not production. So yeah should work with 5.6 fine, that said I now only test for 7.1+ as anything <=7.0 is end of life see https://www.php.net/supported-versions.php

Thanks for replying when even there is no question anymore.
I updated the version to the latest and received an error which comes from a package which requires php7. But it turned out was one of the other libraries I use.
Have a nice day!


d3plus issue chrome and require

Hi guys,
I have an issue when I am using your visualization library in our BI framework.
The issue raises when I am using Chrome, in Firefox it is working fine.
Note I am using require.js (AMD)...
Can you please have a look:

http://stackoverflow.com/questions/43322044/d3plus-chrome-error-d3plus-is-not-defined

Thanks and regards,
Patric

Solved!
Require config section was defined in wrong way!


Delay asset pipeline until rust build finished

My use case includes copying SCSS files through a libraries build.rs file into my application root directory.
Building the styles (SASS) will only work with the output from this build script.
But the asset pipeline always runs in parallel / can not be configured to run after a finished rust build. Errors in the SASS pipeline lead to a retry, also skipping/aborting the rust build, leading to no build.rs being executed an and endless retry-loop.
Could such an optional behavior be integrated into trunk?

A not-so-great workaround would be to add a “pre-build” hook that executes a cargo check to enforce running the build once.

Interesting. Right now, there isn't a real scheduler/dependency tree in the trunk build pipeline. However, there's another use case, where web-workers might have a dependency on each other, and the application being last (in order to inject env-vars with cache busting name).
Your use case looks like one more use case for having dependencies between different build steps.

I have a very similar issue, but in my case the build always runs twice, which is annoying.
My build scripts generates some static assets to be served to the website.
There are two ways that I can think about to solve this:

Be able to configure trunk to wait for cargo build to finish before loading the assets.
Be able to "defer" the loading of certain assets which are dependent on build scripts.

Yea, in general it might be great to have a dependency graph inside of trunk for that. But I agree that having a simple flag of "run pipelines sequentially" might be a good alternative.


Modify column - does not persist Nullable(Clickhouse)

https://gist.github.com/subkanthi/4ce3f454b397f3ee6dbb3f16c0d1d73f

This is not an issue. closing


Update template SharePoint-SingleFarm-FullConfig

Enhancements & bug-fixes - Published in September 18, 2020

Run SPDistributedCacheService as farm account instead of a different service account
Disable IE Enhanced Security Configuration (ESC) on SharePoint VMs
Disable the first run wizard of IE on SharePoint VMs
Set new tabs to open "about:blank" in IE on SharePoint VMs
Move resources to avoid error on ExtendMainWebApp

Enhancements & bug-fixes - Published in October 13, 2020

Set FrontEnd VMs with SharePoint MinRole Front-End on SharePoint versions that support MinRoles configuration
Increase max numberOfAdditionalFrontEnd from 3 to 4
Install Edge Chromium in SharePoint VM and Front-End VMs through Chocolatey
Install Notepad++ in SharePoint VM and Front-End VMs through Chocolatey
Install 7-zip in SharePoint VM and Front-End VMs through Chocolatey
Install Fiddler in SharePoint VM and Front-End VMs through Chocolatey
Install ULS Viewer in SharePoint VM and Front-End VMs through Chocolatey
Install Chrome in Front-End VMs through Chocolatey
Install Everything in Front-End VM through Chocolatey
Define the list of all possible values for the time zone parameter vmsTimeZone
Update WaitToAvoidServersJoiningFarmSimultaneously to ensure it runs only 1 time, and updated the delay from 60 to 90 secs to improve reliability
Use a unique location for custom registry keys
Update parameters passed to ConfigureFE
Update DSC module SharePointDSC from 4.2 to 4.3
Update DSC module NetworkingDsc from 8.0 to 8.1
Update DSC module ActiveDirectoryCSDsc from 4.1 to 5.0
Update DSC module xWebAdministration from 3.1.1 to 3.2

This PR is ready for approval

@leovms this PR is ready for your approval


track database execution sql

pinpoint:2.4.1
agnet:2.4.1
Hello!
I use kingbase database,Can you tell me how to develop  plugin to trace  database execution sql
Thanks

Please refer to the Plugin Development Guide.
https://pinpoint-apm.gitbook.io/pinpoint/documents/plugin-dev-guide


Can't find a way to customize the input fields

I find the datetime-picker element really nice. Unfortunately, I cannot figure out how to style the not native input fields with id #date and #time. There is a mixin for the inner calendar-element but this is not what I am looking for.

Oops, there is a dedicated mixin for picker inputs:
datetime-picker {
    --datetime-picker-input: {
        background-color: yellow;
        border: 1px solid red;
    }
}


Installation with brew

To make installation easy for MacOS users we should support brew.
brew tap etopeter/kubectl-view-utilization
brew install kubectl-view-utilization

Pull #48

Brew support added via separate tap repository.
brew tap etopeter/tap
brew install kubectl-view-utilization


Add Integrations API

I have added integrations API to easier integrate with other libraries. As proof I added: Oby and native non-reactive implementation as a banchmark for other libs.
I also did a bit of restructuring, not sure why but that helped, and now the tests are stable and do not have impact on each other like on current main branch.
EDIT:
Actually, tests are more stable but still the memory leaks to each and every test. We would need to run them on separate threads in the future. I think this might help: https://github.com/tinylibs/tinypool

I like idea, however it seems to me that we can't get to be straight-forward to the implementation because the API is abstracted, shouldn't this impact the benchmark negatively for some of the frameworks?


Catboost serving

Good day. Can I serve a catboost model with kfserving? If yes. Can you point me to an example or explain the process. Thank you

/assign
Will take a look at this.

Thanks. I appreciate it. Can't really test the method right now but it would be useful later on.


Fresh install today did not work

Installed a few hours ago on a colleague's computer, ran the script from readme.md
It was at this commit. (4 commits back)
c34098af20cafdf016f68f3e7f612ed60ca93aa1
After install it refused to load any modules. Never figured out why. Deleted the files, reset hard to a commit from march 2014 which I had on my computer and knew was working, reinstalled and it worked.
This may mean that the current master is broken for fresh installs.

It would not recognise the prompt command (or any) + there seemed to be a little ascii guy with !1 in the bottom corner.

I use the latest on my system; if it were broken, I would see it immediately. Some Chinese users had trouble installing due to the Great Firewall of China.

Allright, we get new hires next week, when I install it on their brand new computers I'll see if I can reproduce consistently

Try to paste the code in a file and execute said file. Make sure that there are no .z* files present in the home directory.
I have avoided adding an installer because I want advanced users to use it to keep the number of issues low.

Thanks will do! And thanks for Prezto!


Optimize integer types

What was wrong?
Integer types in base_types.py were slow.
How was it fixed?
I have optimized the integer code in base_types.py, mostly by endrunning python's classes machinery (no calls to super()) and eliminating redundant checks. This results in a more than 2x speedup in dataset item generation time and knocks about 25% off cache generation on my machine under cpython.
U256 and Uint32 now use mostly shared code.
Cute Animal Picture

Codecov Report

Merging #324 (fff4db0) into master (4aaa4a8) will increase coverage by 6.00%.
The diff coverage is 98.31%.

@@            Coverage Diff             @@
##           master     #324      +/-   ##
==========================================
+ Coverage   88.16%   94.17%   +6.00%
==========================================
  Files          38       38
  Lines        2230     2007     -223
==========================================
- Hits         1966     1890      -76
+ Misses        264      117     -147

Flag
Coverage Δ

unittests
94.17% <98.31%> (+6.00%)
:arrow_up:

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

Impacted Files
Coverage Δ

src/ethereum/base_types.py
88.68% <98.31%> (+19.89%)
: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 4aaa4a8...fff4db0. Read the comment docs.