Add beta netplay support

This adds support for the new mainline dolphin. Enable by using useBetaNetplay = true in the config.
I know there's a draft PR for Vulkan support. Somehow, the mainline dolphin supports Vulkan OOB.

I don't know if GitHub allows merges to be squashed into a single commit, but I recommend you do that with this one because of that "Merge branch" commit. That was an accident.

Thanks for the PR! I will hopefully have some time later today to take a look! Also, I'm sorry that the GitHub actions aren't setup to run on PRs! I will need to fix that...

Sorry for the delay! There were some formatting errors that the GitHub actions stuff should have highlighted and I wanted to get that working before I continued here.
You should be able to rebase, run nix fmt, commit the properly-formatted code, and force push and we should be good to go!

Should be good now! Sorry for the long wait.

I also added a little blurb to the README, so hopefully that helps too.

Merged! Thanks very much for this!


Can this code be used for VOC Object detection?

The authors have another paper Learning Data Augmentation Strategies for Object Detection with policies specifically searched for object detection.


SAA-170 AM Events Endpoint: Client for (prison api) appointments

New endpoint for events
New client to prison api

Codecov Report
Base: 75.86% // Head: 60.47% // Decreases project coverage by -15.39% :warning:

Coverage data is based on head (35177e0) compared to base (60558ce).
Patch coverage: 39.51% of modified lines in pull request are covered.

Additional details and impacted files
@@              Coverage Diff              @@
##               main      #69       +/-   ##
=============================================
- Coverage     75.86%   60.47%   -15.40%
- Complexity      308      370       +62
=============================================
  Files            70       95       +25
  Lines          1065     1847      +782
  Branches         28       32        +4
=============================================
+ Hits            808     1117      +309
- Misses          240      710      +470
- Partials         17       20        +3

Impacted Files
Coverage Δ

...itiesmanagementapi/client/prisonapi/model/Alert.kt
0.00% <0.00%> (ø)

...itiesmanagementapi/client/prisonapi/model/Alias.kt
0.00% <0.00%> (ø)

...api/client/prisonapi/model/OffenceHistoryDetail.kt
0.00% <0.00%> (ø)

...ntapi/client/prisonapi/model/OffenderIdentifier.kt
0.00% <0.00%> (ø)

...pi/client/prisonapi/model/OffenderSentenceTerms.kt
0.00% <0.00%> (ø)

...mentapi/client/prisonapi/model/PersonalCareNeed.kt
0.00% <0.00%> (ø)

...i/client/prisonapi/model/PhysicalCharacteristic.kt
0.00% <0.00%> (ø)

...nagementapi/client/prisonapi/model/PhysicalMark.kt
0.00% <0.00%> (ø)

...ementapi/client/prisonapi/model/PrivilegeDetail.kt
0.00% <0.00%> (ø)

.../client/prisonapi/model/ScheduledAppointmentDto.kt
0.00% <0.00%> (ø)

... and 17 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

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


Print the pixel format in the window example

Based over #371
Merge only when x11/cocoa implement the function, otherwise the window example will panic.

Thanks @tomaka ! I may need to implement it for X11 myself. But even then, can't rely on it until someone takes care of OSX :/

Why aren't you merging this? I'll have it conflicting with mine

I'm waitin for an answer in #371

Cocoa implementation is now merged, how is X11 status?

I believe support was added for X11 in #375.

Oh I wanted to rebase this, but totally forgot.

@SSheldon while you're at it, do you mind taking a look at https://github.com/servo/rust-core-graphics/pull/35 and https://github.com/servo/rust-cocoa/pull/85? Both are very simple changes :)

@fkaa alas, I'm not a contributor to those repos so I can't do much for you :) But I agree they look good!


Additional: Add validation rules for Chilean RUT

Description

Thank you!

You'd better add test cases in test/additional.

I noticed that there is more than just the Chilean RUT in this pull request.  There is also a new method for a Chilean phone number and a different version of "lettersonly", which can't possibly apply only to one country.  I think you should use this PR for only the RUT validation and put the other two validations in separate PRs.
And as stated earlier, add unit tests.


Dark Mode bugs in /log-in

Describe the bug
The text ist unreadably in dark mode and the Ash logo locks strange.

To Reproduce
Change your environment to dark mode and browse to http://localhost:4000/log-in
Expected behavior
Ideally the page would become a "proper" dark mode page with a dark background. But at least it should be usable. The text has to be readable.

I think we might just need to update the default for the sign in live root class:
  override SignInLive do
    set :root_class, "grid h-screen place-items-center dark:bg-black"
  end

Adding dark:bg-black there. I fixed that in my app by updating my template.


Should it be a stateful set instead of deployment?

If the cronjobber must be a singleton, I think stateful set is probably more appropriate.

@somdoron I'm curious, where did read that it needs to be a singleton?

@obitech honestly, I don't remember... I've seen it somewhere.


Victor VosMottor: Watch incracked.com

Victor VosMottor requests the watch of the watch_keyword incracked\.com. See the MS search here and the Stack Exchange search in text, in URLs, and in code.

incracked\.com has been seen in 1 true positive, 0 false positives, and 0 NAAs.

Approved by Spevacus in Charcoal HQ


Create changhyunh.txt

@changhyunh.goe.go.kr
This is the domain address provided by the school.
Renews every school year and is not available after graduation.

http://changhyun.org/

http://changhyun.org/
The site design is wrong and you have to double-click to connect.
If necessary, I will provide the school address and phone number.

http://changhyun.org/
The site design is wrong and you have to double-click to connect.
Address : 216-1, Worldcup-ro Paldal-gu, Suwon, Gyeonggi-do, Korea(South) 16495
Phone : +82-31-548-5000~1 Fax : +82-31-214-8422

http://changhyun.org/


sw_engine: trimpath regression bug

Looks a bug of the trimpath regression at frog_vr.json

curlprit: https://github.com/thorvg/thorvg/pull/2782/files


switch python to 3.12

I ran a test run with performance results here. Looks like all benchmarks ran and there were no performance differences.

@ursabot help


Extract morphometric index from T1 and DWI using Schefer400 atlas

Hi @faskowit ,
I am working with tht extraction of morphometric analysis using Schefer400-7net atlas.
First, the FreeSurfer reconstruction was performed using your amazing tool. Individual .annot files were generated using the Schafer400-7net .gcs files and then individually parceled to extract surface area, cortical thickness, gray matter volume, Gaussian curvature, and mean curvature.
Second, I'm going to use DWI images to generate fractional anisotropy and mean diffusivity and also want to parcel them to Schefer400-7net to generate corresponding index like the way of T1 images for each subject. I wonder if this is feasible?
Best,
Yunhong

Hello -- are you planning to run tractography and then extract FA and MD along the streamlines?  Otherwise, I do not think your question makes 100% sense to me.  There are a handful of papers that extract DWI metrics in the gray matter, but they use specialized DWI acquisitions: for example https://www.ncbi.nlm.nih.gov/pmc/articles/PMC6706419/.  So overall, the feasibility depends on the data you are using!

Hi @faskowit
Yes, I would like to run tractography and then extract FA and MD along the streamlines.
I'm going to extract morphometric index using in this paper https://linkinghub.elsevier.com/retrieve/pii/S0896627317310929. Different from the paper using DK308 atlas, I want to use Schaefer400-7net.
Best,
Yunhong

ok yes indeed, good luck with that!  I hope you get nice results!

Thank you for the detailed answer! @faskowit


Fix ACL cache retention after policy updates

The ACL cache was not cleaned up during update of policies.
This PR fixes this issue and dumps the old cache when the policy changes.

Codecov Report

Merging #351 into master will not change coverage.
The diff coverage is 100%.

@@           Coverage Diff           @@
##           master     #351   +/-   ##
=======================================
  Coverage   59.55%   59.55%
=======================================
  Files          51       51
  Lines        5813     5813
=======================================
  Hits         3462     3462
  Misses       2085     2085
  Partials      266      266

Impacted Files
Coverage Δ

enforcer/datapath.go
64.77% <100%> (ø)
: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 be78fac...5092f8f. Read the comment docs.


Mobile Menu Bubble

Menu Bubble Android Pie Compatibility
The Menu Bubble is currently incompatible with Android Pie, as selecting the text brings up a magnifier, and letting go brings up the Android selection context menu.
Suggestion
Perhaps when viewed in mobile, the Menu Bubble can display /below/ the highlighted text rather than above so that users can still interact with it.

Duplicate of #37
A bubble menu is a bad solution for mobile. Just use the regular menu bar. I try to add a solution for v1 to add custom tooltips.

You can:
// add this line of code
document.addEventListener("contextmenu", function(e) {
  e.preventDefault();
});

It's useful for me. :3


Day2 part3

Using admin power here. I wan't to see how it deploys to adjust our deployment issues.


Unstable in using Wi-Fi

My environment.

iPhone7(MNCV2J/A) 256GB
iOS 11.4.1
OSC Version 2.4(5)

Problem
When Wi-Fi using, Application is unstable. Logging in is unable and quit suddenly. Without Wi-Fi using, I succeed in logging in.
Upploading data is too slow. But without Wi-Fi, I can upload data smoothly.
When W-Fi using, everything is slow and unstable.

is this still happening?


Final release todos

Mandatory

[ ] pass labels + ticks from recipes to axis
[ ] pass attributes to camera
[x] static compilation
[ ] recipe improvements (https://github.com/JuliaPlots/AbstractPlotting.jl/issues/14)
[ ] make it easier to pass attributes to sub plots
[x] fix legends
[x] improve layouting ( do we already have good algorithms for layouting?)
[x] make sure nothing is in the way of group
[x] fix theming
[x] fix a bunch of small bugs that crept up
[x] use namedtuples for more compact attribute definition syntax
[ ] recipes doc section
[x] hide window for inline backend

Optional

[x] webgl backend
[x] switch out Reactive with Observables.jl or Signals.jl

I'll add to this some more points later!

Previous developments  would be really great!
I do not want to add too much work but I wanted to spot a small thing which could be missing for users coming from matlab. The possibility to have global handles analogous to gcf and gca (current figure and current axis) is often very convenient.
I guess in Makie it would be the possibility to recover easily the current Screen and the current Scene. Perhaps it is already possible :)?

 Makie.global_gl_screen()
AbstractPlotting.current_scene()

Perfect!!

that will come with the webgl backend!

I haven't used Makie yet but I see something like axis[:names][:textcolor] in the tutorial. Now that you focus on 1.0, maybe you should switch to overriding Base.getproperty. I can tell from PyPlot experience that typing those brackets and colons gets really annoying over time. :)

Sure, that sounds like a good idea :)

Outdated


PMO: build.js starts ios emulator when android is selected

Generator puts this in:
if(buildCordova) {
  buildPromise.then(stealCordova.build).then(stealCordova.ios.emulate);
}

Even if you had selected Android.

This is fixed in donejs-cordova 0.1.3


feat: Permission handler

Checklist

Add Permission handler
Add example

https://github.com/surfstudio/surf-flutter-app-template/assets/39735343/c0d5c69d-c691-4cfd-8146-826923ad9817

[ ] Have you added an explanation of what your changes do and why you'd like us to include them?
[ ] Is there an existing issue for this PR?

link issue here (use keywords like fix, close, resolve etc. if necessary)

[ ] Have the files been linted and formatted?
[ ] Have the docs been updated to match the changes in the PR?
[ ] Have the tests been updated to match the changes in the PR?
[x] Attached videos/screenshots demonstrating the fix/feature.
[ ] Have you run the tests locally to confirm they pass?

New Features
What new capabilities does this PR add?
What docs changes are needed to explain this?

Codecov Report
Attention: Patch coverage is 0% with 104 lines in your changes are missing coverage. Please review.

Project coverage is 23.77%. Comparing base (339c48c) to head (2677ded).
Report is 10 commits behind head on main.

Files
Patch %
Lines

.../features/debug/presentation/ui_kit/ui_kit_wm.dart
0.00%
45 Missing :warning:

...tures/debug/presentation/ui_kit/ui_kit_screen.dart
0.00%
16 Missing :warning:

...ta/repositories/permission_handler_repository.dart
0.00%
11 Missing :warning:

lib/l10n/app_localizations_en.g.dart
0.00%
10 Missing :warning:

lib/l10n/app_localizations_ru.g.dart
0.00%
10 Missing :warning:

...ed/domain/entities/failure/permission_failure.dart
0.00%
5 Missing :warning:

lib/features/app/di/app_scope.dart
0.00%
4 Missing :warning:

...atures/debug/presentation/ui_kit/ui_kit_model.dart
0.00%
3 Missing :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #117      +/-   ##
==========================================
- Coverage   26.39%   23.77%   -2.62%
==========================================
  Files          55       57       +2
  Lines         917     1018     +101
==========================================
  Hits          242      242
- Misses        675      776     +101

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

In general: do we need this functionality for all of our projects?

We can use the same package to access the device file system/gallery/camera etc. This tends to be more common and adding the package to the template becomes more useful


Use Currency class from hdx-python-country

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

7 of 7 (100.0%)  changed or added relevant lines in 3 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.03%) to 94.558%

Totals

Change from base Build<PHONE_NUMBER>:
0.03%

Covered Lines:
556

Relevant Lines:
588

💛  - Coveralls


Question about esm_embeddings

when I run this code
python datasets/esm_embedding_preparation.py --protein_ligand_csv data/protein_ligand_example_csv.csv --out_file data/prepared_for_esm.fasta
I met this notice:
encountered unknown AA:  TPO  in the complex  /raid/ligl/data/data/PDBBind_processed/4yo6/4yo6_protein_processed.pdb . Replacing it with a dash - .
encountered unknown AA:  SEP  in the complex  /raid/ligl/data/data/PDBBind_processed/4yo6/4yo6_protein_processed.pdb . Replacing it with a dash - .
  0%|                                        | 39/16379 [00:03<20:34, 13.24it/s]encountered unknown AA:  SEP  in the complex  /raid/ligl/data/data/PDBBind_processed/6esa/6esa_protein_processed.pdb . Replacing it with a dash - .
encountered unknown AA:  SEP  in the complex  /raid/ligl/data/data/PDBBind_processed/6esa/6esa_protein_processed.pdb . Replacing it with a dash - .
encountered unknown AA:  TPO  in the complex  /raid/ligl/data/data/PDBBind_processed/6esa/6esa_protein_processed.pdb . Replacing it with a dash - .
encountered unknown AA:  SEP  in the complex  /raid/ligl/data/data/PDBBind_processed/6esa/6esa_protein_processed.pdb . Replacing it with a dash - .
  0%|▏                                       | 79/16379 [00:07<30:56,  8.78it/s]encountered unknown AA:  SEP  in the complex  /raid/ligl/data/data/PDBBind_processed/3f2a/3f2a_protein_processed.pdb . Replacing it with a dash - .
  1%|▏                                       | 97/16379 [00:09<25:34, 10.61it/s]encountered unknown AA:  PCA  in the complex  /raid/ligl/data/data/PDBBind_processed/5t1k/5t1k_protein_processed.pdb . Replacing it with a dash - .
  1%|▏                                       | 99/16379 [00:09<24:53, 10.90it/s]encountered unknown AA:  SEP  in the complex

When I produce dataset with esm2_3billion_embeddings.pt
I met this notice:
loading complexes 9/17:  52%|████████▎       | 523/1000 [03:39<04:09,  1.91it/s]Skipping 2pcp because of the error:
Encountered valid chain id that was not present in the LM embeddings
loading complexes 9/17:  53%|████████▍       | 526/1000 [03:40<02:35,  3.04it/s]Skipping 4do4 because of the error:
Encountered valid chain id that was not present in the LM embeddings
loading complexes 9/17:  53%|████████▍       | 528/1000 [03:40<02:18,  3.40it/s]Skipping 1rri because of the error:
Encountered valid chain id that was not present in the LM embeddings
loading complexes 9/17:  53%|████████▍       | 530/1000 [03:41<03:11,  2.45it/s]Skipping 4k9g because of the error:
Encountered valid chain id that was not present in the LM embeddings
loading complexes 9/17:  54%|████████▌       | 539/1000 [03:44<02:45,  2.78it/s]Skipping 2qwd because of the error:
Encountered valid chain id that was not present in the LM embeddings
loading complexes 9/17:  54%|████████▋       | 540/1000 [03:45<02:50,  2.70it/s]Skipping 1hqh because of the error:
Encountered valid chain id that was not present in the LM embeddings

Both trainset and testst, it skip many items. Is it normal? How can I  fix this error？

Same problem. Have you found a solution to this issue yet? 😞

Probably a typo of some sort, TPO and SEP are not amino acid codes so they got skipped. Then when the fastq are processed later the discrepancy is found.

Same problem, it says it's skipping for every single complex.

Are you sure that you are using the same .fasta file for both steps? @JuLieAlgebra Would you be able to describe the specific setting and procedure you run for one of the proteins where the issue occurs?

It seems that the error occurs when one tries to retrain the model with own complexes containing and '' (underscore) in the name of the structure. In such a case the key_name = key.split('')[0] assigns the wrong value to the key_name.

SEP is phosphorylated SER and TPO is phosphorylated THR.
I am wondering if anyone can explain how this kind of amino acids are handled by DiffDock? Does DiffDock just skip them? What will happen if SEP is renamed to SER? Will it be handled just as SER?


Hosting with AWS Amplify

Adding AWS Amplify hosting to the docs.

We have moved the Hugo site source to its own repository, so we would appreciate if you could open this PR directly in hugoDocs:  https://github.com/gohugoio/hugoDocs/pulls

In docs you have mentioned, amplify will automatically detect hugo configuration and it will build. But its not happening, it showing error that config file is not there( my config file is in config dir). When i am trying to pass this info through arg flag to hugo command, then its showing hugo command not found. Why dockerfile have fixed version of 0.51, anything can be done to upgrade (not issue to raise here, just saying). Any suggestion to make it work will be helpful.

Reading the conversation above this note probably fit's in on gohugoio/hugoDocs.
See https://github.com/gohugoio/hugoDocs/pull/736.

Thank you for pointing this out. But this has nothing to do with my problem. I can build my site with config.toml kept at root folder. But it is failing when I was using config folder to save my config.toml for production and staging server.


Stop saying errors are not going to be captured in dev

This is useful to have logged in production, but is distracting
everywhere else.

Loving this change 😃.


initial score is larger than score for the whole training process

Dear author:
Thanks for your sharing the code!. I try to replicate your performance using Lits.json as training set. However, it is tough to make score larger than initial score. I do not know what problem it is.  The training process can be seen in the Figure. Meanwhile, I also do some parameter adjustment. For example, decrease reward and alpha. But it still not work.  Could you give me some advice about this problem. Thanks for the help and looking forward to receiving your response.

Hello, I found an error in the operation process, hope to get your help, good luck!
Number of data in combine-train is 2302
Number of data in lpba-val is 39
Traceback (most recent call last):
File "E:\SPAC-Deformable-Registration-main\code\main.py", line 63, in
env = Env(generator, stn, seg_stn, cfg.USE_SEG, device)
File "E:\SPAC-Deformable-Registration-main\code\env.py", line 28, in init
self.reset()
File "E:\SPAC-Deformable-Registration-main\code\env.py", line 43, in reset
items = next(self.generator)
File "E:\SPAC-Deformable-Registration-main\code\data_util\brain.py", line 93, in generator
scheme = self.schemes[subset]
KeyError: 'lits'


Peers: UI  for swarm connect

Here's an idea:  "Add Connection" button

could be a button similar to "Add to IPFS" from  Files screen
opens form that accepts multiaddr, when confirmed executes ipfs swarm connect
open questions:

how to indicate success/failure?

somehow store and "pin" peerid of manually added connection(s) to the top of Peer list?
display error message instead of peerid line if failed?

support "pet names" ? could be first step towards ipfs-friends-like feature

Thanks! I thought we had this written up somewhere, but apprently not. Yes, let's build it, I love it.


Discuss Projects

Share any ideas you have for projects the group(or groups) could make together.

app that will contain resources and project ideas

Create a scheduler for keeping track of upcoming meetings in the React group in Discord.

I don't know how we are going to handle feature requests, but I think it would be a nice idea to not only add the time the meeting will be (countdown + time in GMT+0), but a way to quickly convert it to the local time zone might be neat.


Error：Less than 50% of SNPs have  genetic positions in genetic map file

Hi there
Thanks for your excellent work on the development of this useful tool! I got an error when used it in my family data as follows:
$ ibd.py --bgen chr1.phased --pedigree pedigree.txt --map geneticmap1000G_chr1b37.txt --out chr1 --threads 12 --ld_out
Number of threads: 12
Reading pedigree from pedigree.txt
Found 38 full sibling pairs
No genotyping error probability provided. Will attempt to estimate from parent-offspring pairs.
86 parent-offspring pairs found in chr1.phased.bgen
Estimated mean genotyping error probability: 1e-06
Reading genotypes from chr1.phased.bgen
Inferring IBD for chromosome 0
reading -- time=0:00:11.95, thread 1 of 12, part 19,323 of 19,323
Calculating allele frequencies
38 sibpairs have genotypes
Before filtering on MAF, missingness, and genotyping error, there were 231869 SNPs
After filtering, there are 191618 SNPs
Reading map from geneticmap1000G_chr1b37.txt
Traceback (most recent call last):
File "/home/sherry/anaconda3/bin/ibd.py", line 171, in
main(args)
File "/home/sherry/anaconda3/bin/ibd.py", line 164, in main
snipar.ibd.infer_ibd_chr(sibpairs, error_prob, error_probs_i, outprefix,
File "/home/sherry/anaconda3/lib/python3.9/site-packages/snipar/ibd.py", line 375, in infer_ibd_chr
gts.map = get_map_positions(mapfile, gts)
File "/home/sherry/anaconda3/lib/python3.9/site-packages/snipar/map.py", line 65, in get_map_positions
raise(ValueError('Only '+str(round(100prop_in_map))+'% of SNPs have genetic positions in '+mapfile+'. Need at least '+str(round(100min_map_prop))+'%'))
ValueError: Only 33% of SNPs have genetic positions in geneticmap1000G_chr1b37.txt. Need at least 50%
The genetic map files are from 1000G.V3 reference panel provided by impute2 website(https://mathgen.stats.ox.ac.uk/impute/1000GP_Phase3.html) and I used them to phase and impute my genotype data before mendelian imputation. I don't know why to produce this error because I think that most of the genetic variants I input also exist in 1000G panel. Do you have any suggestions?
Best regards!

PS: I have filtered SNPs with low quality as described in the tutorial.

We have finally resolved it and hereby close the issues

Hi sorry for the slow reply.
Did you discover any bug in the software?
Thanks,
Alex
On Sun, Dec 11, 2022, 7:40 PM sherryfan030 @.***> wrote:

Closed #24 https://github.com/AlexTISYoung/snipar/issues/24 as
completed.
—
Reply to this email directly, view it on GitHub
https://github.com/AlexTISYoung/snipar/issues/24#event-8009837172, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/ABQQS6LJDMZLQAZBRG2NDJDWM2NEHANCNFSM6AAAAAASXYUKBA
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>


[python]: bad script fails wrong test

I do not really know how to word the problem other than:

Follow instructions of challenge to best of ability

Run tests

See hint given does not correspond to issue

Expected Behaviour
When tests do not rely on Camper code "compiling", the tests should still be allowed to pass/run.
OR
When Camper code does not "compile", a different error should appear other than defaulting to the first hint

I'm going to need some more context. Which challenge is this and where is the code?

Duplicate of https://github.com/freeCodeCamp/freeCodeCamp/issues/52411

Thanks @ieahleen 🚀
@ojeytonwilliams As also shown in #52411 , if I have a test assert.match(code, /a = 1/), but a Camper has a syntax/logic error in their Python:
a = 1
b =

The tests will fail, and the first hint text will be given to the Camper. That hint text could be You should have "a = 1" in your code (which it clearly does). So, the hint does not make sense.

Thanks for clarifying, @ieahleen and @ShaunSHamilton. Is the desired behaviour: if the python script raises an exception, show a hint like "something went wrong in your python script, please see the console for more information"?

Is the desired behaviour:

I would say we should have the option to:
a) Allow an invalid script to pass all tests (what if we want to show what happens with errors) - just test the code string
b) Manually check the code exceptions or not

Closed by #53040


Replaces multiple matched tokens error with a log message, removing matching tokens from cache

Replaces multiple matched tokens error with a log message, removing matching tokens from cache.

Codecov Report

Merging #6311 (81c0927) into dev (81d34b4) will decrease coverage by 1.94%.
Report is 234 commits behind head on dev.
The diff coverage is 67.42%.

Flag
Coverage Δ

msal-angular
96.73% <92.78%> (+0.22%)
:arrow_up:

msal-browser
85.55% <ø> (-0.91%)
:arrow_down:

msal-common
83.01% <ø> (-1.53%)
:arrow_down:

msal-core
?

msal-node
79.29% <ø> (-4.10%)
:arrow_down:

msal-node-extensions
68.51% <60.77%> (-7.13%)
:arrow_down:

msal-react
94.24% <ø> (-0.45%)
:arrow_down:

node-token-validation
?

Files Changed
Coverage Δ

...ions/msal-node-extensions/src/utils/Environment.ts
20.51% <0.00%> (-22.08%)
:arrow_down:

lib/msal-angular/src/msal.interceptor.ts
100.00% <ø> (ø)

lib/msal-angular/src/msal.module.ts
100.00% <ø> (ø)

lib/msal-angular/src/msal.navigation.client.ts
93.33% <ø> (+0.47%)
:arrow_up:

lib/msal-angular/src/msal.redirect.component.ts
100.00% <ø> (ø)

lib/msal-angular/src/msal.service.ts
100.00% <ø> (ø)

lib/msal-angular/src/packageMetadata.ts
100.00% <ø> (ø)

...b/msal-browser/src/app/IPublicClientApplication.ts
42.42% <ø> (-1.33%)
:arrow_down:

...ib/msal-browser/src/app/PublicClientApplication.ts
67.44% <ø> (-27.89%)
:arrow_down:

...er/src/broker/nativeBroker/NativeMessageHandler.ts
86.29% <ø> (+0.33%)
:arrow_up:

... and 73 more

... and 159 files with indirect coverage changes


Error when fork Async Backing Enabling Runtime

Hi, we encountered a similar issue: https://github.com/AcalaNetwork/chopsticks/pull/743. However, the fix doesn't help in our case.
    app: "chopsticks"
                     runtime  ERROR: panicked at /home/jaypan/.cargo/git/checkouts/polkadot-sdk-d4b773be765ce22f/538d3a3/substrate/frame/aura/src/lib.rs:416:9:
assertion `left == right` failed: Timestamp slot must match `CurrentSlot`
  left: Slot(287413136)
 right: Slot(287413137)
[07:53:50.693] WARN (block-builder): Failed to apply inherents {} Error: wasm `unreachable` instruction executed
    app: "chopsticks"
file:///home/jaypan/Work/peaq/chopsticks/packages/core/src/blockchain/block-builder.ts:190
            throw new Error('Failed to apply inherents');
                  ^

Error: Failed to apply inherents
    at initNewBlock (file:///home/jaypan/Work/peaq/chopsticks/packages/core/src/blockchain/block-builder.ts:190:19)
    at async buildBlock (file:///home/jaypan/Work/peaq/chopsticks/packages/core/src/blockchain/block-builder.ts:297:44)
    at async #buildBlock (file:///home/jaypan/Work/peaq/chopsticks/packages/core/src/blockchain/txpool.ts:215:47)
    at async #buildBlockIfNeeded (file:///home/jaypan/Work/peaq/chopsticks/packages/core/src/blockchain/txpool.ts:199:13)

Is that related to https://github.com/paritytech/polkadot-sdk/pull/3211?
My command is this
yarn start -e wss://docker-test.peaq.network -w peaq_runtime.compact.compressed.wasm.peaq.v0.0.101

And the WASM code is here
The version of WASM is based on polkadot-sdk 1.7.2

your problem seems to be missing aura runtime api to get slot duration and chopsticks is falling back to 12s block time

Hi, thanks for replying. I'm checking about the aura runtime API (auraApi.slotDuration) and the return is 6000.

In my survey, it looks like this didn't affect after writing, but I don't understand why

@sfffaaa ok I see. Chopsticks can't find aura storage and assuming there's no aura_runtime_api and it fallback to 12s block time

Yep, you are right. I also found the solution and open the MR

I have this PR opened. I will apply a fix there

Cool, that will be great! Thank you very much!


[BUG] - Save in Root

Description
The extension no longer saves the images in ComfyUI's output folder as usual, in this case it saves the images. Now the images all end up in the root folder of the hard disc, without a file folder.

Reproduction steps
No response
Image file
No response

Nevermind, was a bug from a other node, after Updating the Nodes everything works.


Date fields in loop break the questionnaire

SD survey, with last versions of Queen
@AnneHuSKa will qualify the bug soon

Not only date fields but all fields I think : maybe a Lunatic pb (tested with https://inseefr.github.io/Lunatic/storybook-beta)
Issue52.zip

For SD, problem is at:

This sub-sequence is looped upon with this loop definition:

And i also see the strange display of the components:


You may also need the length of secret

in your text you say

Knowing only data, H, and signature, the attacker's goal is to append 'append' to data and generate a valid signature for the new data. And that's easy to do! Let's see how.

but actually, you need the length of secret to calculate the length of padding?

Yes, unfortunately you do. I should mention that in the text.
On Thu, Oct 25, 2018 at 2:51 AM 勇者护手<EMAIL_ADDRESS>wrote:

in your text you say
Knowing only data, H, and signature, the attacker's goal is to append
'append' to data and generate a valid signature for the new data. And
that's easy to do! Let's see how.
but actually, you need the length of secret to calculate the length of
padding?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/iagox86/hash_extender/issues/10, or mute the thread
https://github.com/notifications/unsubscribe-auth/AAgITEyeoOQREnqwp7yhm0L_SqrdBQVsks5uoYmDgaJpZM4X551Z
.


[draft] Make HttpDispatchError an enum

Following #1530, I made HttpDispatchError into an enum to see how it would work.
However, I would also like to discuss this idea:
Flattening HttpDispatchError into RusotoError
That means making futures that currently returns HttpDispatchError return RusotoError instead:
impl Future for HttpClientFuture {
    type Item = HttpResponse;
    type Error = RusotoError;

And flattening current HttpDispatchError variants onto RusotoError:
#[derive(Debug, PartialEq)]
pub enum RusotoError<E> {
    /// A service-specific error occurred.
    Service(E),
    /// Request timed out
    Timeout,
    /// Error from client future
    ClientFutureError (String),
    /// Internal error in deadline implementation.
    DeadlineError (String),
    /// Error from hyper
    HyperError (String),
    /// Io error
    IoError (String),
    /// An error was encountered with AWS credentials.
    Credentials(CredentialsError),
    /// A validation error occurred.  Details from AWS are provided.
    Validation(String),
    /// An error occurred parsing the response payload.
    ParseError(String),
    /// An unknown error occurred.  The raw HTTP response is provided.
    Unknown(BufferedHttpResponse),
}

(I wasn't exactly sure how to describe ClientFuture error etc yet)
Do you find the implemented or aforedescribed ideas ok, or do we need to find another way to fix #1530?

Thanks for the PR!
I'm booked up the next few days. Hopefully someone else can come by and give it a look. 😄 Otherwise it'll be next week for me to review this.


Rename ?foaX to ?argX throughout.

Can we get some more context here? Is this mainly just to remove the implicit "foa" label from within the How/Where learning mechanisms? Are there other places we would need to look at for this?

Context: the naming ?foa# for where part assignments is a holdover from simstudent when all of the where parts came from foas. Now that foas are no longer a hard constraint, I have suggested we used the naming "?sel", "?arg0", "arg1", ... etc to make the different 'where' parts explicitly match our vocabulary sheet. As of my latest commit, I've replaced all uses inside ModularAgent, but have held off on making this replacement in the where learners (didn't want to open that can of worms rn. There shouldn't be any compatibility issues, but we should watch out for the specificToGeneral learner).

Looks like this one is solved? @DannyWeitekamp can we close is?

This is definitely fixed for the most part in smartsheet branch. There are a few mentions still in Chris's where learners but I'm afraid to touch those because there are no test cases.


setting the viewport size is not allowed (limited by fingerprint).

Hi @CheshireCaat  ,
I purchased the license. I'm getting this error while setting the viewport in headful mode.
My problem statement is to set the browser full screen by
await page.setViewport({ width: 1255, height: 800, });
or
args: ['--start-maximized']
both are not working.

Viewport is changed when setting fingerprint. Changing it manually is not allowed.


FX Options WG meeting - July 23rd 2020

Date
July 23rd 2020 - 12pm ET / 5pm BST
Approve past meeting minutes - #181
Untracked attendees

Fullname, Affiliation, (optional) GitHub username
...

Agenda

[ ] Convene & roll call (5mins)
[ ] Averaging Feature: Modelling Review - Vijay (30mins)
[ ] AOB, Q&A & Adjourn (5mins)

Decisions Made

[ ] Decision 1
[ ] Decision 2
[ ] ...

Action Items

[ ] Action 1
[ ] Action 2
[ ] ...

WebEx info

Meeting link: https://finos.webex.com/finos/j.phpMTID=m9989b4e8a8a07ed8677e35cf4bb879cd
Meeting number: 667 055 449
Password: CWfvDt6yA36
Join by video system: Dial<EMAIL_ADDRESS>You can also dial <IP_ADDRESS> and enter your meeting number.
Join by phone: +1-415-655-0003 US Toll | +44-20319-88141 UK Toll | Access code: 667 055 449

Aqui!

present

meeting minutes for 7/16 meeting were approved

present


Add hooks

Would be nice if there was hooks to change the behaviour of an action, like useThrottle.
Yet this would be a breaking change. With useEvent, for example, this would remove BindEvent and BindSyncEvent, as they wouldn't be needed anymore.

Same goes for MiddlewareAction vs useMiddleware.


Revert "Simple SILOptimizer peepholes for packs"

Reverts apple/swift#65119
Likely causes a crash in CSE in the IRGen/run_variadic_generics.sil  test on linux
rdar://108045677

@swift-ci smoke test

Closing because I disable the failing test: https://github.com/apple/swift/pull/65180


Adds the Will parameter on the connect available options

Hi there,
I’ve fixed Bug #7 in the branch named “ issue-7-will-option” in my fork. This is a simple change and I've tested here and everything works fine if you use or not the will option. I update the README too.
Thanks!

Nice work!


Publish docs

Action to publish documentation
A link to the docs in the readme

With readthedocs there is apparently also the possibility that the actionn is triggered on their side whenever the git is updated.
That sounds nice, but also means that the setup needs to be done on readthedocs. Don't know what is better.

I read up a bit on this. It seems like we have have to configure the build on readthedocs if we want to use their service. If we want to use github actions, we have to roll our own thing and deploy to github pages.
I am in favor of using what readthedocs provides because it supports versioning. Like this we'll preserve a documentation history. Reproducing this using github actions would be cumbersome.

I'm ok with whatever works best. I personally am always very annoyed by versioned documentation, as search engines tend to show old versions.
So searching for "clingo api" on duckduckgo gives me the first link:
https://potassco.org/clingo/python-api/5.4/
which is very outdated. Is this better with readthedocs ?

Is this better with readthedocs ?

At least with readthedocs there is an easy way to jump to the latest version.


[Android] How to update new sqlite in StreamingAssets folder ?

I going to ship my product with pre-defined database in Assets/StreamingAssets. However, when I update application with new database (install the update from App store, but not delete and reinstall), the pre-defined sqlite is not update. (it still use old database).
How can I get my data updated? (without delete and reinstall app on phone)
Thank you

I think you can keep your sqlite db in the Resources folder and when you open the App for the first time, you will copy it to Application.persistentDataPath.
Something like that (the file need to be named with .txt extension):
TextAsset sqdbTextAsset = Resources.Load("sqlite_folder_name/your_database.sqdb") as TextAsset; File.WriteAllBytes(dbFilePath, sqdbTextAsset.bytes);


Return the output datasets

Notice

[ x] I acknowledge and agree that, by checking this box and clicking "Submit Pull Request":

I submit this contribution under the Apache 2.0 license and represent that I am entitled to do so on behalf of myself, my employer, or relevant third parties, as applicable.

I certify that (a) this contribution is my original creation and / or (b) to the extent it is not my original creation, I am authorised to submit this contribution on behalf of the original creator(s) or their licensees.

I certify that the use of this contribution as authorised by the Apache 2.0 license does not violate the intellectual property rights of anyone else.

Motivation and Context
KedroContext.run does not return the output datasets returned by runner.run
How has this been tested?
pytest
Checklist

[x ] Read the contributing guidelines
[ ] Opened this PR as a 'Draft Pull Request' if it is work-in-progress
[ ] Updated the documentation to reflect the code changes
[ ] Added new entries to the RELEASE.md file
[ ] Added tests to cover my changes
[ ] Assigned myself to the PR

@921kiyo Okay. To resolve the conflicts and reflect your feedback, I opened a new pull request at https://github.com/quantumblacklabs/kedro/pull/67.


add links to projects

projects.html has the github and web icons for the projects.
projects.yml has been modified with the links of the projects

Resolves #37
Please, go through these steps before you submit a PR.

[x] My Pod Leader knows I'm working on this Pull Request
[x] I've explained what the Pull Request is adding.
[x] I've explained why this is important.

@akshitadixit this resolves issue #37 . Thanks


[SDPA-2976] Fetch alert notifications from site taxonomy

JIRA issue: https://digital-engagement.atlassian.net/browse/SDPA-2976
Changed

Removes seperate call for alerts and gets them from site taxonomy now.
Multiple alerts can be present now.

Screenshots

@tim-yao I've added a router middleware to check when the sitedata was last fetched and refetch it if its been a while (set to 1 minute but we can tweak it). I reckon thats a good compromise. What do you think?


Update ERC-6982: Move to Last Call

New PR on this new repo to move ERC-6982 to Last Call

I tested if fixing the other ERCs passes the checks. I will revert the changes, later.

I reverted it.


fixed search issue with mixing tags and search keywords, updated tests

https://app.clubhouse.io/augur/story/17740/filtering-by-tag-is-treated-as-a-general-search

Coverage increased (+0.009%) to 56.469% when pulling 64ce588303c87e70b22f0b5c90e531eb44785083 on tomhaile/ch17740/filtering-by-tag-is-treated-as-a-general into c8d72391cf09fb91fbd9e09cb08eb008d4867bfa on master.

Coverage increased (+0.009%) to 56.469% when pulling 64ce588303c87e70b22f0b5c90e531eb44785083 on tomhaile/ch17740/filtering-by-tag-is-treated-as-a-general into c8d72391cf09fb91fbd9e09cb08eb008d4867bfa on master.


Commands of plugin 'Obsidian Git' doesn't display in commands menu in Obsidian 0.9.22 which installed from snap

Commands of plugin 'Obsidian Git' doesn't display in commands menu (Ctrl + P) in Obsidian 0.9.22 installed from snap
But this plugin work at the same version Obsidian 0.9.22 from AppImage package
All packages links i get from https://obsidian.md/
May be root problem from Obsidian core, not in a plugin.
Screenshoot Obsidian 0.9.22 installed from snap:

I had a similar issue and it was an issue with GIT, not obsidian. In my case, I had to accept some new TOS in Xcode (Mac). In any case, viewing the developer tools gave me a clear error code to explore. Once I resolved my GIT issue, plugin worked as expected.

I had a similar issue and it was an issue with GIT, not obsidian. In my case, I had to accept some new TOS in Xcode (Mac). In any case, viewing the developer tools gave me a clear error code to explore. Once I resolved my GIT issue, plugin worked as expected.

It's a problem with snap release not a plugin itself. Yesterday I have tested it on my Ubuntu machine and I can confirm that this plugin is not working on snap version. Problem is not in plugin, but in snap packages and their sandbox approach to file access, etc. This plugin works utilizing system 'git' command and if it's not accessible, plugin is not loading.

If you are on Linux I can recommend the flatpak version.(Or of course AppImage)
You should have an error in your console (Ctrl + Shift +i). Snap apps are in a sandbox, why Obsidian doesn't have access to git.


Use CAFile even if client certificate is not specified

Even if client cert auth is not used, it is sometimes necessary to specify the trusted root ca's of the API server

@jbeda PTAL


CsvGuessPlugin enables suggesting for a few rows

Current CsvGuessPlugin could not handle a few rows (i.e. only 2 or 3 lines in a CSV file) and suggest Embulk config appropriately. Because it doesn't use a last line string as sample lines and removes it from the sample lines.
In this PR, the change of guess_plugin.rb enables CsvGuessPlugin to use a last line string to suggest a config. If the sample data ends with newline string suggested, the string actually could be treated as a last line.
This PR also fixes NewlineGuessPlugin and inherit guess method in TextGuessPlugin instead of guess_text method to suggest newline string. TextGuessPlugin#guess_text always appends CRLF to a sample text.

Thank you for reviewing. Fixed on b098936b38de82dbb91dea36f71718645e1aa2ff

git rebase master and git push -f


Is DoRA necessary during training?

I noticed that DoRA is being used in the training code. Is it necessary? If we switch to LoRA, how significant would the impact on the results be?
https://github.com/TIGER-AI-Lab/VLM2Vec/blob/1ecb8612415e1ce7498e34e3077dc7c2874f2bab/src/model.py#L94C17-L94C31

Thank you for your reply!


ACS Commons fails to deploy to AEM as a Cloud Service due to inclusion of /var nodes

Required Information
Originally reported with clear description of symptoms by @stefanseifert  at: https://github.com/Adobe-Consulting-Services/acs-aem-commons/issues/2334

[X] AEM Version, including Service Packs, Cumulative Fix Packs, etc: AEM as a Cloud Service
[X] ACS AEM Commons Version: 4.x.x
[X] Reproducible on Latest? yes

ACS AEM Commons fails to deploy to AEM as a Cloud Service because AEM's replication service user does not support replication (due to permissions of the service user) or nodes under /var, which the ACS Commons ui.content project has.

These nodes should should be establishing using repoinit scripts (targetting the minimal runmodes) as needed. This looks trivial for all nodes EXCEPT /var/acs-commons/reports which contains cq:Page definitions (which are large, and complex in structure).
@klcodanr is there a reason these nodes are in /var? It actually seems strange these are in /var which historically has more transient, runtime content (these appear to be report definitions). Thoughts on if they could be easily moved outside of /var?
Also, can you help me confirm:

[x] /var/acs-commons/httpcache -> Should exist on Author and Publish (@niekraaijmakers)
[ ] /var/acs-commons/mcp -> Should exist only on Author (@badvision)
[x] /var/acs-commons/on-deploy-scripts -> Should exist on Author and Publish (@HitmanInWis)
[ ] /var/acs-commons/reports -> Should exist only on Author (@klcodanr)

The working PR to resolve this is at: https://github.com/Adobe-Consulting-Services/acs-aem-commons/pull/2342

@davidjgonzalez
We should be able to relocate. As a bridge, the listing component could be changed to look in two locations whereas new reports would be created in the new location and either add a feature or a note for AEM Cloud Service users telling them to relocate reports.  There are a couple options I could think of:
1. Put under /content
We could move to /content/acs-commons/reports or similar path.  There are a couple of downsides:

It makes me uncomfortable to put non-publishable content under the /content tree, especially since it's a reporting feature, which would seem bad to accidentally publish
You can't exclude a folder from the Sites view without overlaying /libs because the excluded pattern is set in properties at: /libs/wcm/core/content/sites/jcr:content/views/column/datasource@exclude

2. Put under /conf
We could move to /conf/acs-commons/reports or similar path. I'm not 100% sure on whether this would work since it seems like /var should have been fine as well based on the documentation: https://docs.adobe.com/content/help/en/experience-manager-cloud-service/implementing/developing/aem-project-content-package-structure.html
If it does work, that would seem like a better approach as /conf is not going to generally be enabled for web access.
Definitely open to other options.

Yes /var/acs-commons/on-deploy-scripts should exist on author/publish - many times On-Deploy Scripts do content refactors that we dont want to require a publish event if it can be avoided.

Thanks @klcodanr  - it seems like if we have an idea of where we'd like to "eventually" move all the utility pages from /etc/acs-commons, it would be good to make align to that.
Do the reports cq:Page sub-trees need to be authorable (persistable?) and/or would a customer add/modifiable? If not - could we move them to /apps perhaps? (the filter.xml rule makes it seem like customers might be adding/modifying this area tho)
I too share your concerns of creating /content/acs-commons .. it just doesn't feel right to me, as the SiteAdmin (which surfaces /content) is well, the "Site admin" ... it should list the web sites/properties hosted by AEM.
I want to say @kwin had thoughts on the larger /etc/acs-commons move .. but dont have the GH issue handy.

@davidjgonzalez Unfortunately, due to a "feature" in the product, you could not use TouchUI under /etc:
https://blogs.perficient.com/2017/11/16/no-matter-what-a-anyone-tells-you-theres-no-touchui-under-etc/
I just checked in AEM 6.5 and this seems to still be an issue.

Argh, I misread your initial comment, so yes, the reports need to be authorable, it wouldn't make a lot of sense to move them to /apps. IMO - /conf seems like the best option.

configuring a report is analogous to configuring a template or content fragment model -- I support the idea of /conf.  I don't think there's an API that standardizes how we use that area so proceed with caution.  Might be worth abstracting a model for working with conf, I feel like we're going to be here again.

Keep in mind that the dialog generator is now enabled by default, so it could be as simple as a sling model that explains the configuration stored in /conf and a little wiring to access the dialog to edit those values.

@badvision I think the larger problem w/ /conf is there isn't a simple, OOTB framework for managing the pages themselves (rather than content IN the pages). This was the big benefit of /miscadmin .. it basically cloned the siteadmin capabilities outside the context of Sites... unless i missed something and the Dialog Generator can also generate that page administrator experience.

It cannot -- and you hit the nail on the head.  That was the challenge I ran into switching generic lists to use dialog generator; I had to clone the site admin console and hack at the code until things fit right.  I would like to not repeat that process as it doesn't scale and is a bear to maintain as well I'm sure.

But we need something like /miscadmin, a generic console that adapts to the types we need to manage through it, what toolbars are available, etc.  These were never abstracted well, and it doesn't seem like the product is on track to do that for us.  However, dialog generation scales.  Console generation should also scale as well.  It's just a different type of abstraction that results in synthetic resources.

Like dialog generation, if the console generation is no longer compatible at some future revision, then a good abstraction of what a console should provide will still hold up, we just change the generator to match what the product needs and use version detection or something similar so it uses the right implementation.  Will need to give it some thought.

Agreed 100% @davidjgonzalez and @badvision -- I would also mention that if we were to do this, I would recommend a "clean room" approach to the implementation vs relying on CoralUI. Coral UI has the "fun" tendency to have incompatible changes between versions that break things in "exciting" ways.
https://blogs.perficient.com/2020/04/13/building-aem-admin-consoles-that-will-not-break-with-new-aem-releases/ (sorry for the broken images, I'll bug our website people)

@klcodanr Agreed 100% -- it needs to be abstracted, e.g. a UML-like breakdown of what is needed at the core.  The UI is an implementation of that abstraction and should not inform the core model.  If we do that, it should be possible to keep it relevant regardless of UI changes.  I've never been a fan of the death-by-convention approach used pervasively, but fortunately the open design of the product allows plenty of room for clean abstractions to co-exist with legacy approaches.   So it's not all bad, we have room to grow here.

I don't understand why editing of pages in /conf is not OOTB included? We just copy pasted /sites.html and made it /conf.html so that you get the full page UI editor. It all works, we had to do almost no adjustments at all. This way you can just make new conf pages like you would in /miscadmin

On 24 Jun 2020, at 17:43, Brendan Robert<EMAIL_ADDRESS>wrote:
@klcodanr https://github.com/klcodanr Agreed 100% -- it needs to be abstracted, e.g. a UML-like breakdown of what is needed at the core. The UI is an implementation of that abstraction and should not inform the core model. If we do that, it should be possible to keep it relevant regardless of UI changes. I've never been a fan of the death-by-convention approach used pervasively, but fortunately the open design of the product allows plenty of room for clean abstractions to co-exist with legacy approaches. So it's not all bad, we have room to grow here.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub https://github.com/Adobe-Consulting-Services/acs-aem-commons/issues/2341#issuecomment-648900522, or unsubscribe https://github.com/notifications/unsubscribe-auth/ABF42TQEMTAL65J77B52XYDRYINJHANCNFSM4OGWOJ6A.

like

I thought the same thing when I wanted to add touchui admin interface for generic lists but it is not as genetically defined as I thought it was and needed to be cloned to remove extra junk not needed such as versions, etc.

this problem is not yet fixed in the latest 4.7.2 release.
is there a time plan for this issue? - imho this completely block projects deploying to AEM cloud using ACS AEM Commons.

This needs /var/acs-commons/reporting to be moved out (i guess to /etc/acs-commons/reporting .. i wish we had a better vision for where these should land so we dont have to move them again)
This would also be a breaking change, for folks that have created reports under /var, so it'd have to roll into v5.0.0.
@klcodanr any luck finding time to move the reports?

If we're discussing a v5.0.0, would this be a good time to drop support for 6.3 and possibly even 6.4 for v5.0.0?

@HitmanInWis the plan was to drop 6.3 support in v5.0.0.
The 5.0.0 release is tracked here: https://github.com/Adobe-Consulting-Services/acs-aem-commons/issues/2288
If you have a compelling pitch for dropping 6.4, drop in there! :)

For starters, 6.5's been out for a year...

I know what @badvision says is a bit tongue in cheek, but truthfully we should be on 6.6 now (instead we have AEMaaCS) so really 6.4 is currently 2 versions old.  Furthermore, new features contributed are expected to be 6.5+CS capable.  I'm just not sure the 6.4 support is really compelling.  When does 6.4 go EOL for Adobe support?

Is 6.4 compat preventing us from doing anything we want to do?
Can we list the benefits of dropping 6.4 support? I understand it's old, but people are, for better or worse, still on 6.4.x and i believe Adobe still release SP's for 6.4 (or did in May, https://docs.adobe.com/content/help/en/experience-manager-64/release-notes/sp-release-notes.html)
If we can support 6.4 at no cost to us, then why not? If there is a cost, what is it?

Just committed an approach to moving the reports under /conf in #2365 -- IMO we could also look into removing the report console entirely, though we'd have to consider how to handle existing reports under /var

the problem still exists in ACS AEM Commons 4.8.4.
(is no one using ACS AEM Commons in AEM as a cloud service? it's not even possible to deploy it there.)

@stefanseifert yeh - have to release 5.0.0 to incorporate @klcodanr 's fix ... havent found the time. maybe this weekend?
Just a mental reminder, i think we have to adjust services users to be defined in [ ] as well so they can be used w/ prinicipal based ACLs (i should make an issue for this)
https://github.com/Adobe-Consulting-Services/acs-aem-commons/issues/2288

gentle nudge - problem is still not solved, unable to deploy ACS AEM commons on AEM as a cloud service

Same here - this project says it's compatible but it is not. This compatibility note should be removed until this issue is closed, as compatibility with local AEMaaCS instances is no help if it can't be deployed via CloudManager.

@davidjgonzalez @klcodanr What do you think about a minimal fix for the 4.x versions to create the /var content structure via repoinit? That requires fewer changes and should not cause regressions.
I fear that 5.x will no happen too soon and this blocks usage of ACS AEM Commons with AEMaaCS (despite what is written in https://adobe-consulting-services.github.io/acs-aem-commons/pages/compatibility.html#aem-as-a-cloud-service-feature-incompatibility)

@davidjgonzalez @klcodanr What do you think about a minimal fix for the 4.x versions to create the /var content structure via repoinit? That requires fewer changes and should not cause regressions.
I fear that 5.x will no happen too soon and this blocks usage of ACS AEM Commons with AEMaaCS (despite what is written in https://adobe-consulting-services.github.io/acs-aem-commons/pages/compatibility.html#aem-as-a-cloud-service-feature-incompatibility)

@kwin we could - that would be a breaking change - so 5.x right?
I have the 5.x branch which I think should be the last of it - I think the biggest thing is doing a regression since so many filters changed.
If you think we can do a simple adjustment to 5.x and bump what's prepped for 5.x to 6.x
Agree 5.x is dragging way too long (I just had the time outside work to work on it :()

@kwin we could - that would be a breaking change - so 5.x right?
I have the 5.x branch which I think should be the last of it - I think the biggest thing is doing a regression since so many filters changed.
If you think we can do a simple adjustment to 5.x and bump what's prepped for 5.x to 6.x
Agree 5.x is dragging way too long (I just had the time outside work to work on it :()

that would be a breaking change

IMHO not, my proposal is to leave everything at /var but just create the content with repoinit instead of content package. That is no breaking change and should work with AEM Classic and Cloud.

that would be a breaking change

IMHO not, my proposal is to leave everything at /var but just create the content with repoinit instead of content package. That is no breaking change and should work with AEM Classic and Cloud.

@davidjgonzalez Is this considered a valid workaround: https://helpx.adobe.com/in/experience-manager/kb/cm/cloudmanager-deploy-fails-due-to-sling-distribution-aem.html?
If so we could add write permissions to the acs-aem-commons specific var folders for the 'sling-distribution-importer' to a repoinit section. If you are fine with that approach, I will prepare a PR for 4.x.

@kwin yeh - i think this is a good idea. Would you be able to pull the PR together? If so, i can cut a release.

Just for the record: The default permissions of the system user on the publish are listed in https://github.com/Netcentric/aem-cloud-validator/issues/9#issuecomment-802121462.


MKL: update MKL-DNN version to not compute bias for BWD_F

The original driver for MKL-DNN computed both gradients for weights and bias for BWD_F. This patch disables bias gradient computations by default. The aim of this change is to make it easier to compare MKL and MKL-DNN performance.

Looks OK to me, merged.


3.16.2 build generated invalid file names

E.g. EachListCallback<I>.hx instead of EachListCallback.hx. Which behaves weirdly in zip files so haxelib install fails.

I updated it, it should be better.

That was fast :+1:, but seems like it also removed some files that were OK.

It's because sometimes, I forget to delete all files before regenerating bindings. I did it this time. Deleted files were valid for previous Phaser version but they no more exists.


greif prevention bugged up

so poked my head back on the server a few days ago,and got a tiny head start done already,and i just got on and noticed my home got re-rolled to older position while i was scouting on the first day.

and yea my claim blocks is now negative -6.5k my first plot shows up buggy, and when i plopped down a chest i got the starter bonus of claimed land,which my initial plot already got once.
wtf is going on with greif prevention?

Basically...Its a 1.7.9 plugin, running on 1.7.10 server, running a completely modified .jar, running alongside mods. These things happen.
As far as claims being removed or altered. You probably weren't logged on for long enough and the claim got pruned and deleted.

ok thank you.


[bug 1193332] Remove eq_ and ok_: translations

This commit removes eq_ and ok_ imports as well as their use in the
files under translations directory. It replaces eq_ and ok_ with proper
assert statements.

Awesome!


apply to pyfar

fix tests
remove github-ci, tox, travis and authorfiles
update cookiecutter-pypackage Contributing
add default python version for cicle_ci test doc + test flake8
configure circle_ci like in pyfar
apply templates from pyfar (issure pr)
update .gitignore
add readthedocs conf
apply readme and Contributing from pyfar
apply doc settings
add minimum python version and create supported versions in setup.py and .circleci/conf automaticly
test everthing
all tests and doc tests are working see https://github.com/pyfar/imkar/pull/20
default doc is here https://imkar--20.org.readthedocs.build/en/20/

Thanks for the quick review, pypi_username is by default the gitusername, i think we can leave it like this. And I'm not sure if you reviewd the latest version, since [1] and [4] are already set to you suggestions. Anyway i applied your remaining suggestions.


add customized item template to hide depositor

fix https://github.com/antleaf/nims-mdr-development/issues/614

@anusharanganathan Thank you!


Quantization-aware training with non-parameter layers and BN layers?

According to this paper, we should accurately simulate quantization related effects such as fusion of operations and batch normalization folding during quant-aware training.
PostTrainLInearQuantizer seems to replace layers with no parameter (e.g. torch.cat)  by the quantized version of them (e.g. RangeLinearQuantConcatWrapper).
https://github.com/NervanaSystems/distiller/blob/fac2359bbd075879aa633aa6cfd57981bee1e250/distiller/quantization/range_linear.py#L797-L802
However, in QuantAwareTrainRangeLienarQuantizer, there is no such handling for non-parameter layers.
So, my question is that, when I use layers like concat, eltwise-* in quant-aware training,

should I add such replacements ? and
how can I simulate the batch normalization folding behavior?

Thank you !

Hi @Wanwannodao
Indeed, as it is QuantAwareTrainRangeLinearQuantizer only fake-quantizes the activations of nn.ReLU modules. This essentially means that if we have the following sequence:
relu_0 --> layer_x --> layer_x+1 --> ... --> layer_x+n --> relu_1
Then, in the context of activations quantization, the layers [layer_x, layer_x+1, ..., layer_x+n] are assumed to be fused with "relu_1".
You'd be correct in saying this covers only a specific (albeit a very common) pattern.
If you have cases where you know you need to insert fake-quantization nodes, then at the moment I can offer the following:

Sub-class QuantAwareTrainRangeLinearQuantizer
In your sub-class, add replacement factory entries for the layer type whose output you wish to quantize.
If you need to quantize only specific instances of that layer type, then take care of that using the overrides mechanism.

(You could of course just modify QuantAwareTrainRangeLinearQuantizer directly instead of sub-classing)
Let me know if you need more help with this.
Regarding BN folding - this has been raised before, see #166. Unfortunately we haven't gotten to implementing this yet. I hope we can get to it soon, but can't say at the moment when that would be. If this is something you think you'd like to tackle, let us know.

Hi, thank you for your answering !
That means, for example, the following pattern isn't supported for now, is it ?
relu_0 -- \
             add_x --> relu_2 ...
relu_1 -- /
I mean, there are layer types whose input tensors have to be requantized to match the scaling factors and the zero-points, and output tensor may not require quantization, right ?
In such a case, what should I do ? Defining another fake-quantization wrapper with forward looking like below ?
def forward(self, *input):
    res = self.fake_requant(res)
    res = self.wrapped_module(*input)
    return res

And, yes, I will probably need to implement BN folding.

Regarding the pattern you drew - the code you wrote is a start, indeed you'd need to have fake quant operation before the element-wise add module. But, in the case of element wise addition, the inputs are re-quantized according to the scale of the output. So the fake requant operation will need to get some info from the fake requant op in the output.
All of that requires a way to understand the order of execution of the model. We have the infrastructure for that with the SummaryGraph class, it's just not readily available for the quantizer. This is WIP, I hope to push it sometime next week.
Regarding BN folding - we will take a look at it from our side. If we end up deciding we want help, I'll update here.

Hi.
Ah, that's true... It requires more steps... Thanks for your explanation.
Okay. No problem.

Hi,
BatchNorm folding has been implemented and merged.
This issue seems resolved, and hence I'm closing it. Feel free to reopen if needed.

In this case: conv->conv->conv->relu, we need convolution output scale/zero_point to convert output to  int8/uint8 for accelerating speed, how to solve it ?


Android ActionBar ActionItems tooltips style

Hi,
I noticed the ActionBar's ActionItems have a nice tooltip on Android when long pressing them, based on their text attribute:

I also noticed the automatically generated "Overflow Button" have a "More options" tooltip, whose text is based on the system's language:

Unfortunately, the styles of the tooltips don't match (tight black box vs. large white one).
Is it possible to have a similar, black-background tooltip for custom ActionItems?

Duplicate of https://github.com/NativeScript/NativeScript/issues/6460


[CT-1048] As part of proto integration convert NodeFinished class and run_results to use proto definitions

The proto events will need to be generated outside of our normal class hierarchy.  We need to establish best practices and methods for doing this easily, since we'll need node output for all of our node types. In addition we need proto definitions for RunResults.
As part of this effort we will collect requirements from other stakeholders to use when choosing which information to include and to plan additional events.

@gshank could you edit this ticket to better describe what we talked about

Will identify a list of existing events that would be most useful for updating the progress of execution and parsing, and if there are gaps, create new events. In addition will create logging events that will contain some of the information that is currently retrieved from run_result.json.

A related part of this is identifying ways to generate proto definitions from Python classes, because creating a set of parallel classes where every time we add a new attribute it must be added in two places is suboptimal. This might look like adding proto integer identifiers to attributes, like pure_protobuf.

Split the spike/investigation into a new ticket: https://github.com/dbt-labs/dbt-core/issues/5987
Let's keep this ticket focused on the info from NodeFinished that can support the end-to-end Model Timing MVP with Metadata

@jtcohen6  Question: We have 22 logging events that include node_info right now. But a lot of them are variants such as "PrintModelResultLine" and "PrintModelErrorResultLine".  These events match the stdout lines that we produce in a terminal. But a consumer of the logs is going to have to subscribe to multiple events for every one of the node types in order to capture both success and failure.
I'm wondering if it might not work better to have a single "ModelResult", which contains a success attribute, and can produce either the success or result message.

This has been done as part other tickets.


What did I do wrong?

I was doing the following commands to launch Shimeji. Then I encountered the following:
invertigo_2@penguin:~$ cd linux-shimeji
invertigo_2@penguin:~/linux-shimeji$ ./launch.sh
./launch.sh: line 1: java: command not found

What does it mean that the ./launch.sh command wasn't found by Java? On an Acer Chromebook with Debian 10/Buster. (By the way, invertigo2 is my name on Linux.)

I think you simply do not have Java installed on your pc
If you check, launch.sh simply launch a Java command

after check, simply do:
sudo apt update
sudo apt install default-jdk

How do I check launch.sh? I'm sorry if I don't understand.

I tried to launch /launch.sh , but it says this:
-bash: ./launch.sh: No such file or directory
Also, it hangs when I attempt to download Java. :/

I figured out how to get the ./launch.sh open, what next?


Dashboard Triangle symbol

When the sample application html is saved and opened from local file, the Dashboard triangles (e.g. Row Styling) becomes: â-¶ unexpanded, â-¼ expanded.
Adding the following tag after the head tag fixes it:
<meta http-equiv="content-type" content="text/html; charset=UTF-8">
Saving from the web browser will also add it in (don't need the other style tags it adds: injected-stylesheet-filter-tree-base and injected-stylesheet-finbar-base)

@willsolve Would you want to take the credit in the codebase and submit a PR? Its a one-liner but its yours!

Resolving workstation bash pull requests.  May take a while to figure out.


ref #802: do not handle languages on backend site editing

Q
A

Branch
7.1.x

Bug fix?
yes

New feature?
no

BC breaks?
no

Deprecations?
no

Tests pass?
yes (check issue ticket reproduce steps)

Fixed issues
https://github.com/chameleon-system/chameleon-system/issues/802

License
MIT

For details see linked issue.

Not necessary.


test-branch/execute-job/comment-from-wrong-user-on-pr/6dafa5df-7cc8-4704-8c25-a9700a315288

PR for testing fde7203a39334ea90fee4cca48b429e73c69d956

/canonical/self-hosted-runners/run-workflows fde7203a39334ea90fee4cca48b429e73c69d956


Error: Cannot find module 'tailwindcss/defaultTheme' when deploying to production server

I am running into a serious blocker when I attempt to deploy my Rails 7.0.5 project (using Ruby 3.1.2) to my Ubuntu 20.04 server. My project was setup using the rails --css tailwind option so tailwindcss-rails was preinstalled. Everything works just fine in development mode, but when I deploy using Capistrano I end of getting a bunch of errors beginning with the following one, all related to tailwindcss. I have made sure to configure bundler as prescribed using bundle lock --add-platform <platform-name> with the appropriate info for my platform.
    bundler:install       01 $HOME/.rbenv/bin/rbenv exec bundle install --jobs 4 --quiet     ✔ 01<EMAIL_ADDRESS>51.372s 01:20 deploy:assets:precompile       01 $HOME/.rbenv/bin/rbenv exec bundle exec rake assets:precompile       01       01 Rebuilding...       01 Error: Cannot find module 'tailwindcss/defaultTheme'       01 Require stack:       01 - /var/www/ai_demo/releases/20230526122825/config/tailwind.config.js       01     at Function.Module._resolveFilename (node:internal/modules/cjs/loader:933:15)       01     at Function._resolveFilename (pkg/prelude/bootstrap.js:1955:46)       01     at Function.resolve (node:internal/modules/cjs/helpers:108:19)       01     at _resolve (/snapshot/tailwindcss/node_modules/jiti/dist/jiti.js:1:241025)       01     at jiti (/snapshot/tailwindcss/node_modules/jiti/dist/jiti.js:1:243309)       01     at /var/www/ai_demo/releases/20230526122825/config/tailwind.config.js:1:97       01     at jiti (/snapshot/tailwindcss/node_modules/jiti/dist/jiti.js:1:245784)       01     at /snapshot/tailwindcss/lib/lib/load-config.js:37:30       01     at loadConfig (/snapshot/tailwindcss/lib/lib/load-config.js:39:6)       01     at Object.loadConfig (/snapshot/tailwindcss/lib/cli/build/plugin.js:135:49) {       01   code: 'MODULE_NOT_FOUND',       01   requireStack: [       01     '/var/www/ai_demo/releases/20230526122825/config/tailwind.config.js'       01   ]       01 }       01 rake aborted!       01 Command failed with exit 1: /var/www/ai_demo/shared/bundle/ruby/3.1.0/gems/tailwindcss-rails-2.0.29-x86_64-linux/exe/x86_64-lin…       01 /var/www/ai_demo/shared/bundle/ruby/3.1.0/gems/tailwindcss-rails-2.0.29-x86_64-linux/lib/tasks/build.rake:7:in system'       01 /var/www/ai_demo/shared/bundle/ruby/3.1.0/gems/tailwindcss-rails-2.0.29-x86_64-linux/lib/tasks/build.rake:7:in block    (2 levels…       01 /var/www/ai_demo/shared/bundle/ruby/3.1.0/gems/rake-13.0.6/exe/rake:27:in <top (required)>'       01 /home/deploy/.rbenv/versions/3.1.2/bin/bundle:25:in load'       01 /home/deploy/.rbenv/versions/3.1.2/bin/bundle:25:in <main>'       01 Tasks: TOP => assets:precompile => tailwindcss:build       01 (See full trace by running task with --trace)
Here is my tailwind.config.js
`
const defaultTheme = require('tailwindcss/defaultTheme')
module.exports = {
  content: [
    './public/*.html',
    './app/helpers/**/*.rb',
    './app/javascript/**/*.js',
    './app/views/**/*.{erb,haml,html,slim}',
    "./node_modules/flowbite/**/*.js",
  ],
  theme: {
    extend: {
      fontFamily: {
        sans: ['Inter var', ...defaultTheme.fontFamily.sans],
      },
    },
  },
  plugins: [
    require('@tailwindcss/forms'),
    require('@tailwindcss/aspect-ratio'),
    require('@tailwindcss/typography'),
    require('flowbite/plugin'),
  ]
}

`

I tried commenting out const defaultTheme = require('tailwindcss/defaultTheme') and sans: ['Inter var', ...defaultTheme.fontFamily.sans] and deploying again. I then ran into a problem where @tailwindcss/forms couldn't be located: Error: Cannot find module '@tailwindcss/forms'


NETOBSERV-1398: fix loki url when same namespace

Description
Host like "loki-gateway-http.svc" seem invalid
Dependencies
n/a
Checklist
If you are not familiar with our processes or don't know what to answer in the list below, let us know in a comment: the maintainers will take care of that.

[x] Is this PR backed with a JIRA ticket? If so, make sure it is written as a title prefix (in general, PRs affecting the NetObserv/Network Observability product should be backed with a JIRA ticket - especially if they bring user facing changes).
[ ] Does this PR require product documentation?

[ ] If so, make sure the JIRA epic is labelled with "documentation" and provides a description relevant for doc writers, such as use cases or scenarios. Any required step to activate or configure the feature should be documented there, such as new CRD knobs.

[ ] Does this PR require a product release notes entry?

[ ] If so, fill in "Release Note Text" in the JIRA.

[ ] Is there anything else the QE team should know before testing? E.g: configuration changes, environment setup, etc.

[ ] If so, make sure it is described in the JIRA ticket.

QE requirements (check 1 from the list):

[x] Standard QE validation, with pre-merge tests unless stated otherwise.
[ ] Regression tests only (e.g. refactoring with no user-facing change).
[ ] No QE (e.g. trivial change with high reviewer's confidence, or per agreement with the QE team).

/hold for premerge testing

/ok-to-test

/ok-to-test

/unhold

/label qe-approved

/approve


Improve UI

The new UI should be elegant and concise.

https://github.com/Ark2000/PankuConsole/commit/6ebb95a572d0aec95a91eac676ec1cc132641b30


Output channels duplicated when adding a folder to the workspace

Issue Type: Bug

Open code-insiders on some folder
Observe the dropdown of output channels, they are as expected
code-insiders -a <some other folder>

Output channels for extensions are now duplicated:

VS Code version: Code - Insiders 1.29.0-insider (4b2fac2b6da220f62ff58b2766e4ecc1f561be94, 2018-10-30T16:48:18.436Z)
OS version: Linux x64 4.4.0-137-generic

Actually, this also happens in stable. Is this by design?


Setup way too complex

Isn't it sufficient to only point to the schema.graphql file? Also, can't it make some assumptions about where that file might be?

Thank you for the suggestions. I myself have not worked with a lot of graphql projects, and it seems to me different teams organize their schemas differently. For the projects I'm working on, the schema is too big to be contained in one file, and before making the plugin, I'd seen more projects which followed similar trends of splitting up schema, but their folder structure was different enough, so baking in default seemed more work than necessary. That being said, I appreciate your concern, but would like more discussion around this to see how other developers prefer to organize their projects. I'll keep this issue open for discussion.

I have a fairly large schema at work, which we split into conceptually segregated js files (usually one file for each type and all its resolvers), and use the merge-graphql-schemas package to merge the types and resolvers from the various files.
I think for any large schema it makes sense to modularize somehow, so I appreciate that some config will often be required to tell this extension about it.
However, the fact that one needs this @playlyfe/gql library and then also watchman (which is a PITA trying to set it up on anything else but Macs), in addition to the configuration file, just in order to get the basics of this extension running, does strike me as quite an overhead.
Watchman should certainly not be necessary. Linters and others just spawn a background 'server' process to watch files, completely transparent to the user.

Great discussion so far. The schema.graphql file that I'm mentioning is a compiled asset that combines all types for your GraphQL schema into one file. I agree that any non-trivial GraphQL service should have its source code for all of its types split up into different files.
Here's what I've been using to create the single GraphQL file: https://github.com/relayjs/relay-examples/blob/72e0644c572ef4a5f74d113fc75bc8a27030edd3/todo/scripts/updateSchema.js
It's not very user friendly, but it works.

@matthewoates the plugin automatically combines the graphql schema so you don't have to.
Yes, I take your point about watchman - it really is a PITA. I work on Windows, and facebook's tooling is not at all productive on windows - watchman is still ok, but getting flow running and keeping it running on my machine has been impossible for me. It'd be helpful if an issue can be opened on gql regarding a move to something better than watchman.

@kumarharsh Done (https://github.com/Mayank1791989/gql/issues/62).
Although I have not suggested a particular replacement (not sure myself) at this point, it would at least get the thought on the radar.

The fact that one must install watchman globally to make this plugin work is a major code smell. If the project implements watchman, is there not a way to encapsulate that in the plugin?
same with @playlyfe/gql , is it not possible to make this be a dependency of the plugin?

Agreed @the-simian. As per https://github.com/Mayank1791989/gql/issues/62#issuecomment-330945526, I think the reliance of watchman itself might be removed. I'll keep this topic updated.

Nice! The watchman requirement being removed from gql is one step forwards, but, like @the-simian, I also wonder if the dependency on gql can’t be done as part of this extension rather than requiring the user to add it to their dependencies?

Schema files don't necessarily need to present in the file system it should also be able to be queried from the server. That way we don't need to generate a single gql file when we are stitching schemas using graphql-tools or using apollo.
https://github.com/Quramy/ts-graphql-plugin
They allow a path or a url,
schema": "path-or-url-to-your-schema.json"

It also seems like a simple flow. This will a real booster for apollo based projects which is gaining a lot of traction these days.

I agree with other that the gql dependency should be included inside the plugin. It's weird that I need to install a module in my project for an extension that my coworkers won't use because they are not all on VS Code.
Also, I think the .gqlconfig file's option should be options of the extension (options means it can be defined globally, per workspace, per user, etc.), for the same reason. Again, I do not see why I should create a file in the project for my editor. The project files are for the team.
At least that's my point of view. That's still awesome to have an extension for graphql intellisense.

Watchman also seems to be a blocker for setups on WSL. VSCode is running on Windows but node_modules is installed under WSL. In this case I'm getting "Watchman was not found in PATH" and server closes.
Any workaround for this in the meantime?

You can install watchman too under WSL .

yep, I have it installed under WSL, but there seems to be an issue with VSCode running on Windows while watchman is under WSL.

@kumarharsh are there any blockers left for releasing a new version we can help with?

Any update on this?

Its mostly done - one issue with the automated setup that we explored was to use a node-based watched for people who don't want to use watchman. But during the course of development, we found that there no no watched which provides bullet-proof watching capabilities cross-platform - even VSCode is using three different watchers(!) we just need to sort some stuff around that.

@kumarharsh what is the problem with chokidar? It is used by many popular cross-platform tools, e.g. Gulp

@kumarharsh despite that, many tools work just fine with chokidar. How about using watchman when it is available and falling back to chokidar?

I'll be happy to help with fs-watcher

Hi, what's the status on this?

No updates.

Hi, just an update that I am unable to get this working in Windows due to watchman screwing up my paths.


[SPARK-10367][SQL][WIP] Support Parquet logical type INTERVAL

What changes were proposed in this pull request?
Adds support for writing and reading interval data into parquet files.
In this PR, we encode Catalyst CalendarIntervalType to Parquet's FIXED_LEN_BINARY
annotated by INTERVAL. This is in WIP state for further discussions.
How was this patch tested?
Added new tests in ParquetIOSuite

Test build #66271 has started for PR 15334 at commit 8ac351f.

Test build #66271 has finished for PR 15334 at commit 8ac351f.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/66271/
Test FAILed.

Merged build finished. Test FAILed.

retest this please

Test build #66278 has started for PR 15334 at commit 8ac351f.

Test build #66278 has finished for PR 15334 at commit 8ac351f.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/66278/
Test PASSed.

Test build #66409 has started for PR 15334 at commit 8a6b4a8.

Test build #66409 has finished for PR 15334 at commit 8a6b4a8.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/66409/
Test PASSed.

Merged build finished. Test PASSed.

Test build #66585 has started for PR 15334 at commit 72bc930.

Test build #66585 has finished for PR 15334 at commit 72bc930.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/66585/
Test PASSed.

Saw @viirya submitted a PR for the same issue: https://github.com/apache/spark/pull/7793

@gatorsmile Thanks .. I had absolutely no idea about this PR :-)

Test build #75376 has started for PR 15334 at commit 35ec9f1.

Test build #75376 has finished for PR 15334 at commit 35ec9f1.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/75376/
Test PASSed.

Test build #75519 has started for PR 15334 at commit eb84eaf.

Test build #75519 has finished for PR 15334 at commit eb84eaf.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/75519/
Test PASSed.

retest this please

Test build #77256 has started for PR 15334 at commit eb84eaf.

Test build #77256 has finished for PR 15334 at commit eb84eaf.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/77256/
Test PASSed.

Test build #78542 has started for PR 15334 at commit dd5b862.

Test build #78542 has finished for PR 15334 at commit dd5b862.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/78542/
Test PASSed.

Merged build finished. Test PASSed.

@gatorsmile Hi Sean, i am sorry, i didn't see your ping on this pr. I will get back to you on your question as i need to re-create my env. I do remember one thing though. I do remember looking for writers to test this newly added logical type. But i didn't find any.. and i had to create the data files using parquet apis. I will re-check again and get back.
Since we are coding here to a defined parquet spec, i felt safe to add this PR. We know that there are databases such as postgress, oracle , informix which support interval type. So if we were to make this data available to spark to read, they would have the option to encode their interval data in parquet more naturally and spark would be able to read it.

If we can find the other writer that can generate such a parquet file, we should support it; otherwise, we will hold it until we find such writers. Thank you!

@gatorsmile Hi Sean, i tried apache-drill after looking through their documentation.
SLF4J: Defaulting to no-operation (NOP) logger implementation
SLF4J: See http://www.slf4j.org/codes.html#StaticLoggerBinder for further details.
+-----------+----------------------------+
| Fragment  | Number of records written  |
+-----------+----------------------------+
| 0_0       | 3                          |
+-----------+----------------------------+

Here is the schema of the written parquet file.
message root {
  optional fixed_len_byte_array(12) INTERVALYEAR_col (INTERVAL);
  optional fixed_len_byte_array(12) INTERVALDAY_col (INTERVAL);
  optional fixed_len_byte_array(12) INTERVAL_col (INTERVAL);
}

From presto's documentation, it seems like they also may be able to encode interval data. But i haven't tried.
FYI - i also tried hive. Its not possible to encode interval data in parquet format through hive.

IIUC we have no plan to expose Interval types now, so can we close this for now? cc: @gatorsmile

oh, I noticed the jira ticket has already been closed as later, so can you close this? @dilipbiswal

@maropu Thanks.. closing it.


OSError: [Errno 5] Input/output error

I am using the forked version from rplidar-roboticia @jjehl and run the example python code in my Raspberry Pi 3. I am new to python so I don't understand what cause this not working. The scanner motor is running but no scan datas are being read.

Traceback (most recent call last):
File "/home/pi/example.py", line 10, in
for i, scan in enumerate(lidar.iter_measures()):
File "/usr/local/lib/python3.5/dist-packages/rplidar.py", line 397, in iter_measures
raw = self._read_response(dsize)
File "/usr/local/lib/python3.5/dist-packages/rplidar.py", line 223, in _read_response
while self._serial.inWaiting() < dsize:
File "/usr/lib/python3/dist-packages/serial/serialutil.py", line 572, in inWaiting
return self.in_waiting
File "/usr/lib/python3/dist-packages/serial/serialposix.py", line 456, in in_waiting
s = fcntl.ioctl(self.fd, TIOCINQ, TIOCM_zero_str)
OSError: [Errno 5] Input/output error

Hope someone can help me on this. Thank you and sorry for my bad english.

Your error means that the data are wrong.  Be sure to use a rplidar A2 and check that the raw serial data are received correctly.
Please find here : https://github.com/Roboticia/Roboticia-drive/blob/master/software/roboticia_drive/sensor/lidar_a2.py
an example about how to use the module.

Yes I am using rplidar A2. I ran the code in my laptop using python IDLE and I can get the scan datas. But when I run the code in the Raspbian OS of the Raspberry Pi 3, the errors above occured. So do you know how to receive the raw serial data correctly? Is this happened due to hardware (Raspberry Pi3) or software(pyserial, pushing out the read/write calls faster than the hardware can accept them?, I2C)
Thanks in advance.

I ran on raspberry pi3 without problem. It is a sofware issue with your pi3.
_ Be sure to set the correct usb to collect the data
_ Try to colect serial from another source to check if it works (arduino is a good test)
you can test with the roboticia OS derived from Raspbian because with this setup I am sure it works : http://www.roboticia.com/?page_id=258

Yes I am using the UART adapter from the RPlidar A2. The lidar is plugged with USB on RPi.
I just checked using the lsusb command in the raspbian terminal. The number(name) of usb changed once the lidar motor is started and thus the data cannot be read. I think this is the problem.

@willnil  I meet the same problem when i use a UART TO USB cable receiving the serial data, and guess it is a bug of raspbian system. I wonder have you solve this problem?

@DIYalways hello the problem is caused by the insufficient power supply to the RPLidar. Once I connected the rplidar to a power port instead of the usb port of my laptop, that solved my problem.


Suggestion (not a spot price): include SmartEnergy SmartTimes

Apart from hourly Spot market, there are also other smart tariffs out there. e.g. Smart Times:

You have 3 price windows within a day: off-peak, shoulder, peak
the tarrif changes each month

It would be nice if such semi-smart options can be included in this plugin.

Smart Times is also coming from Smart Energy.
I am not sure if their API does provide this data.

Thanks for the positive uptake of "my" wish.
According to the detailed information here: https://www.smartenergy.at/fileadmin/user_upload/downloads/Kundeninformation_und_Preisblatt_-_smartTIMES.pdf
The price calculation is as follows:
Preis – Off-Peak (aktuell) = Austrian Power Future Base Front Month
Preis – Shoulder (aktuell) = Austrian Power Future Base Front Month + 15 % Aufschlag
Preis – Peak (aktuell) = Austrian Power Future Base Front Month + 40 % Aufschlag
The base price can be found also here:
https://www.eex.com/de/marktdaten/strom/futures (net price)

Actually I think we can just scrape the data from the smart energy site.
I was thinking of providing three separate entities for peak shoulder and off-peak.
Users could manually create a template sensor which, based on the current time, selects the correct sensor.
Although something like this could already be achieved with home assistant native web scrapers.


Added Brain Card Game

PR Description 📜
Please include summary related to the issue you have fixed and describe your PR in brief over here by specifying the issue number on which you were working below
Added #4023

Mark the task you have completed ✅

[x] I follow CONTRIBUTING GUIDELINE & CODE OF CONDUCT of this project.
[x] I have performed a self-review of my own code or work.
[x] I have commented my code, particularly in hard-to-understand areas.
[x] My changes generates no new warnings.
[x] I have followed proper naming convention showed in CONTRIBUTING GUIDELINE
[x] I have added screenshot for website preview in assets/images
[x] I have added entries for my game in main README.md
[x] I have added README.md in my folder
[x] I have specified the respective issue number for which I have requested the new game.

Add your screenshots(Optional) 📸

Thank you soo much for contributing to our repository 💗

Thank you @CodeXsubham ,for creating the PR and contributing to our GameZone 💗
Review team will review the PR and will reach out to you soon! 😇
Make sure that you have marked all the tasks that you are done with ✅.
Thank you for your patience! 😀

@kunjgit Hey PA,
Kindly review it and merge it 🙂

Thank you  @CodeXsubham , for your valuable time and contribution in our GameZone 💗.
It’s our GameZone, so Let’s build this GameZone altogether !!🤝
Hoping to see you soon with another PR again 😇
Wishing you all the best for your journey into Open Source🚀


Ensure we quote ETHTOOL/BONDING_OPTS

Signed-off-by: Tom Doherty<EMAIL_ADDRESS>Description
Quote ETHTOOL_OPTS and BONDING_OPTS so ifcfg-if changes from
BONDING_OPTS=mode=active-backup miimon=100 # invalid
to
BONDING_OPTS="mode=active-backup miimon=100" # valid
Issues Resolved
N/A
Check List

[ ] New functionality includes tests
[ ] All tests pass
[ ] RELEASE_NOTES.md, has been updated if required (not required for bugfixes, required for API changes)
[ ] All commits have been signed-off for the Developer Certificate of Origin. See https://github.com/chef/chef/blob/master/CONTRIBUTING.md#developer-certification-of-origin-dco

The aborted Travis job 18 is unrelated to this change. Merging!


Multiline strings is formated invalid with stringify()

Describe the bug
Hi. Seems like for some specific multiline strings, output of stringify() is formatted invalid.
To Reproduce
Example:
console.log(
  stringify({
    'nginx.ingress.kubernetes.io/configuration-snippet':
      'location ~* ^/sites/example.com/files/(.+) {\n  return 302 https://example.com/sites/example.com/files/$1;\n}\nlocation ~* ^/partner-application/cls/(.+) {\n  return 301 https://example.com/partner-application/cls/$1$is_args$args;\n}\n',
  })
);

Output:
nginx.ingress.kubernetes.io/configuration-snippet: >
  location ~*
  ^/sites/example.com/files/(.+) {
    return 302 https://example.com/sites/example.com/files/$1;
  }

  location ~* ^/partner-application/cls/(.+) {
    return 301 https://example.com/partner-application/cls/$1$is_args$args;
  }

Expected behaviour
Expected output:
nginx.ingress.kubernetes.io/configuration-snippet: >
  location ~* ^/sites/example.com/files/(.+) {
    return 302 https://example.com/sites/example.com/files/$1;
  }
  location ~* ^/partner-application/cls/(.+) {
    return 301 https://example.com/partner-application/cls/$1$is_args$args;
  }

Versions (please complete the following information):
Node:
$ node --version
v16.18.1

Yaml: 2.1.3

It does look like the string folding is a bit buggy there. The value represented by the stringified result is valid and does match the source, but there's no good reason why that first line should be folded like that.

Sorry, I don not notice, there are option related with line height and this fix my problem. Closing :-)
console.log(
  stringify(
    {
      'nginx.ingress.kubernetes.io/configuration-snippet':
        'location ~* ^/sites/aaaaaaa.aa/files/(.+) {\n  return 302 https://process.aaaaaaa.aa/sites/aaaaaaa.aa/files/$1;\n}\nlocation ~* ^/partner-application/cls/(.+) {\n  return 301 https://process.aaaaaaa.aa/partner-application/cls/$1$is_args$args;\n}\n',
    },
    { blockQuote: 'literal', lineWidth: 0 }
  )
);

Output is OK now:
nginx.ingress.kubernetes.io/configuration-snippet: |
  location ~* ^/sites/aaaaaaa.aa/files/(.+) {
    return 302 https://process.aaaaaaa.aa/sites/aaaaaaa.aa/files/$1;
  }
  location ~* ^/partner-application/cls/(.+) {
    return 301 https://process.aaaaaaa.aa/partner-application/cls/$1$is_args$args;
  }

Thank you!

No, this is still a bug that should be fixed. The > style folding of your input is bad, even if it is valid.

True, I see it now. Happy to found new bug :-) blockQuote: literal + lineWidth: 0 solve this for me. Thanks again.


Update one-sided-test.md

🚀 Deployed on https://66677d0d2425d158a8ff6961--cozy-fox-0defec.netlify.app


The first POST request to the akka http connection flow has zero length

even if a non-zero length entity was POSTed.
any consequent requests behave correctly
also if any GET request is made first then it's ok

This looks like a quirk of akka-http but it may be that the connection async handler needs to be initialized programatically.

This has to do with proxies that accidentally upgrade all connections to websockets, the solution on Apache Server is:
{Code}
RewriteEngine on
RewriteCond %{HTTP:Upgrade} =websocket [NC]
RewriteCond %{THE_REQUEST} "^[a-zA-Z]+ /(.) HTTP/\d+(.\d+)?$"
RewriteRule .? ws://<IP_ADDRESS>:8881/%1 [P,NE]
RewriteCond %{HTTP:Upgrade} !=websocket [NC]
RewriteCond %{REQUEST_URI} !^/.well-known/acme-challenge/?
RewriteCond %{THE_REQUEST} "^[a-zA-Z]+ /(.) HTTP/\d+(.\d+)?$"
RewriteRule .? http://<IP_ADDRESS>:8881/%1 [P,NE]
{Code}