Translated CONTRIBUTION-GUIDELINES.md into Hindi

translate CODE_OF_CONDUCT.md to hindi
Closes [3rd of 5] #30 issue
Issue Addressed
Closes [3rd of 5] #30 issue
Proposed Changes and Benefits
Added Hindi translation of CONTRIBUTION-GUIDELINES.md
Checklist

[x] I have thoroughly read and understand The Contribution Guidelines.
[x] The title of this PR follows the keyword: brief description of change format, using one of the following keywords:

Feature - adds new or amends existing user-facing behavior.
Chore - changes that have no user-facing value, refactors, dependency bumps, etc.
Fix - bug fixes.
Docs - fixing typos and adding new content.

[x] I have listed my proposed changes and their benefits.
[x] I have linked the corresponding issue in the Issues section.

Hello @XanderRubio please accept my PR.
And kindly add hacktoberfest label to the same.

Hi @abhinaba-dash!
Thank you for taking the time to contribute to Before I Die Code! I am currently traveling, and once I get situated in my next location I will review your pull request and if all is good merge right away. Have a nice day!
Xander

Hello @XanderRubio please accept my PR. And kindly add hacktoberfest label to the same.

What did you mean the label to be hacktoberfest? For this issue or pull request?

This pull request adds to issue #30 and is missing only one more task to be completed.

Congratulations, @abhinaba-dash, for your pull request being merged!  Then, you can become a member of the Before I Die organizationadditionally. Have a great day!


SwiftDialog Broken?

Hi,
the Swift Dialog Onboarding stopped working about a week ago.
I detected, that the URL https://catlab.blob.core.windows.net/swiftdialog/catlab.json
in the 1-installSwiftDialog.zsh isn't working anymore.
Could that be the issue? Maybe depending with the new Swift Dialog 2.5.0 Version released last month?
Best regards

Have a look here and follow this method instead. That one is using json hosted on my Blob storage, which I’ve since closed.
shell-intune-samples/macOS/Config/Swift Dialog (PKG) at master · microsoft/shell-intune-samples (github.com)https://github.com/microsoft/shell-intune-samples/tree/master/macOS/Config/Swift Dialog (PKG)
From: TH2307 @.>
Date: Monday, 8 July 2024 at 15:29
To: microsoft/shell-intune-samples @.>
Cc: Subscribed @.***>
Subject: [microsoft/shell-intune-samples] SwiftDialog Broken? (Issue #145)
Hi,
the Swift Dialog Onboarding stopped working about a week ago.
I detected, that the URL https://catlab.blob.core.windows.net/swiftdialog/catlab.json
in the 1-installSwiftDialog.zsh isn't working anymore.
Could that be the issue? Maybe depending with the new Swift Dialog 2.5.0 Version released last month?
Best regards
—
Reply to this email directly, view it on GitHubhttps://github.com/microsoft/shell-intune-samples/issues/145 or unsubscribehttps://github.com/notifications/unsubscribe-auth/AGOC2JLIKUIXL6ODBLJF6NTZLKO4HBFKMF2HI4TJMJ2XIZLTSOBKK5TBNR2WLJDUOJ2WLJDOMFWWLO3UNBZGKYLEL5YGC4TUNFRWS4DBNZ2F6YLDORUXM2LUPGBKK5TBNR2WLJLJONZXKZNENZQW2ZNLORUHEZLBMRPXI6LQMWBKK5TBNR2WLJDUOJ2WLJDOMFWWLLTXMF2GG2C7MFRXI2LWNF2HTLDTOVRGUZLDORPXI6LQMWSUS43TOVS2M5DPOBUWG44SQKSHI6LQMWVHEZLQN5ZWS5DPOJ42K5TBNR2WLKJSG43DKMJZHA3DBAVEOR4XAZNFNFZXG5LFUV3GC3DVMWVDEMZZGU4DAOJWG442O5DSNFTWOZLSUZRXEZLBORSQ.
You are receiving this email because you are subscribed to this thread.
Triage notifications on the go with GitHub Mobile for iOShttps://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Androidhttps://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

Hi there,
Thanks for the quick reply.
Unfortunately we were unable to get the PKG version to work.
Script, files and icons have been modified. Package payload content has been adjusted accordingly.
Manual terminal execution gets error ERROR: Cannot find swiftDialog binary at /Library/Application Support/Dialog/Dialog.app/Contents/MacOS/Dialog
Any ideas? Even if we build the package without modifying the content source from the zip, we get the same error.
Best regards

Hi there,
Thanks for the quick reply.
Unfortunately we were unable to get the PKG version to work.
Script, files and icons have been modified. Package payload content has been adjusted accordingly.
Manual terminal execution gets error ERROR: Cannot find swiftDialog binary at /Library/Application Support/Dialog/Dialog.app/Contents/MacOS/Dialog
Any ideas? Even if we build the package without modifying the content source from the zip, we get the same error.
Best regards

Or could you provide an example PKG File for Validation?
Then we could rule out the error on our side.

I’ve not heard of anyone else having problems with the published version on GitHub. Have you tried not editing it and just building the PKG? You’ll need to grant the packages app disk access.
From: TH2307 @.>
Date: Tuesday, 9 July 2024 at 10:29
To: microsoft/shell-intune-samples @.>
Cc: Subscribed @.>, Comment @.>
Subject: Re: [microsoft/shell-intune-samples] SwiftDialog Broken? (Issue #145)
Or could you provide an example PKG File for Validation?
Then we could rule out the error on our side.
—
Reply to this email directly, view it on GitHubhttps://github.com/microsoft/shell-intune-samples/issues/145#issuecomment-2217150720 or unsubscribehttps://github.com/notifications/unsubscribe-auth/AGOC2JOIFMMWO4RAVXUMVPTZLOUO7BFKMF2HI4TJMJ2XIZLTSOBKK5TBNR2WLJDUOJ2WLJDOMFWWLO3UNBZGKYLEL5YGC4TUNFRWS4DBNZ2F6YLDORUXM2LUPGBKK5TBNR2WLJDUOJ2WLJDOMFWWLLTXMF2GG2C7MFRXI2LWNF2HTAVFOZQWY5LFUVUXG43VMWSG4YLNMWVXI2DSMVQWIX3UPFYGLLDTOVRGUZLDORPXI6LQMWWES43TOVSUG33NNVSW45FGORXXA2LDOOJIFJDUPFYGLKTSMVYG643JORXXE6NFOZQWY5LFVEZDONRVGE4TQNRQQKSHI6LQMWSWS43TOVS2K5TBNR2WLKRSGM4TKOBQHE3DOONHORZGSZ3HMVZKMY3SMVQXIZI.
You are receiving this email because you commented on the thread.
Triage notifications on the go with GitHub Mobile for iOShttps://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Androidhttps://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

Packages app has full disk access.
Building the Package not editing the content didnt work either. Same error message.

I’m not sure what you’re doing wrong. I just downloaded the ZIP, extracted, opened the Swift Dialog Onboarding project and hit CMD+B and it built the package.
Can you be super precise about what you are doing, step by step.
[image.png]
From: TH2307 @.>
Date: Wednesday, 10 July 2024 at 09:09
To: microsoft/shell-intune-samples @.>
Cc: Comment @.>, Subscribed @.>
Subject: Re: [microsoft/shell-intune-samples] SwiftDialog Broken? (Issue #145)
Packages app has full disk access.
Building the Package not editing the content didnt work either. Same error message.
—
Reply to this email directly, view it on GitHubhttps://github.com/microsoft/shell-intune-samples/issues/145#issuecomment-2219835376 or unsubscribehttps://github.com/notifications/unsubscribe-auth/AGOC2JN5QVLIH6EY4HL4HXTZLTT5BBFKMF2HI4TJMJ2XIZLTSOBKK5TBNR2WLJDUOJ2WLJDOMFWWLO3UNBZGKYLEL5YGC4TUNFRWS4DBNZ2F6YLDORUXM2LUPGBKK5TBNR2WLJDUOJ2WLJDOMFWWLLTXMF2GG2C7MFRXI2LWNF2HTAVFOZQWY5LFUVUXG43VMWSG4YLNMWVXI2DSMVQWIX3UPFYGLLDTOVRGUZLDORPXI6LQMWWES43TOVSUG33NNVSW45FGORXXA2LDOOJIFJDUPFYGLKTSMVYG643JORXXE6NFOZQWY5LFVEZDONRVGE4TQNRQQKSHI6LQMWSWS43TOVS2K5TBNR2WLKRSGM4TKOBQHE3DOONHORZGSZ3HMVZKMY3SMVQXIZI.
You are receiving this email because you commented on the thread.
Triage notifications on the go with GitHub Mobile for iOShttps://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Androidhttps://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.


Quick fix to implement interface does not work if interface contains certain external references

Working in js debug in the latest Insiders (TS 4.2.3):
x.ts
export interface IFoo {
  bar(): void;
}

y.ts
import { IFoo } from './x';

export class Foo implements IFoo {

}

Trigger a quick fix on Foo
You have the option to implement the interface
Add import { ExtensionContext } from 'vscode' to x.ts and change the interface to bar(x: ExtensionContext): void
There is no longer an option to quick fix

Works fine for me in a simple project with just those files and @types/vscode

Any extra steps?


Add support for HTTP MERGE request generation

For Merge requests
corresponding changes have been in FioriDAST
I will email you the PR for FioriDAST
test sample for this implementation

Please also create a new version on merge to master.

Codecov Report

Merging #91 (c702449) into master (62b4c17) will decrease coverage by 1.73%.
The diff coverage is 27.77%.

@@            Coverage Diff             @@
##           master      #91      +/-   ##
==========================================
- Coverage   60.76%   59.02%   -1.74%
==========================================
  Files          18       18
  Lines        3517     3397     -120
==========================================
- Hits         2137     2005     -132
- Misses       1380     1392      +12

Impacted Files
Coverage Δ

odfuzz/fuzzer.py
31.17% <20.00%> (-2.39%)
:arrow_down:

odfuzz/entities.py
73.10% <66.66%> (-0.93%)
:arrow_down:

odfuzz/mutators.py
51.72% <0.00%> (-5.67%)
:arrow_down:

odfuzz/output.py
61.90% <0.00%> (-3.32%)
:arrow_down:

odfuzz/databases.py
71.42% <0.00%> (-3.29%)
:arrow_down:

odfuzz/config.py
89.13% <0.00%> (-2.10%)
:arrow_down:

odfuzz/encoders.py
90.90% <0.00%> (-1.40%)
:arrow_down:

odfuzz/restrictions.py
71.66% <0.00%> (-0.92%)
:arrow_down:

Continue to review full report at Codecov.

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

Small notes, generally OK. Don't forget to update changelog.md with the new feature as well pls.


tabs: way to have same height in all elements

Currently the height of the Toggle element varies depending on the active tab. This makes sense as the default behavior, not to make any assumptions.
Could it have a prop to enable a stable height across all tabs though? It'd have to be the tallest one I suppose.

[ ] Port to other engines if implemented 🙂

p.s. should we create an iterative/web team for tagging on these? @rogermparent @julieg18

Could it have a prop to enable a stable height across all tabs though? It'd have to be the tallest one I suppose.

Makes sense, it would make tab switching be less jerky. We should be able to implement this! But what if we looked into making the content of a tab behave like a details component? Have the content of the tab expand smoothly instead? Then we could just apply that to all tabs without needing any props 🤔

what if we looked into making the content of a tab behave like a details component? Have the content of the tab expand smoothly

Such an animation would definitely improve the default behavior. I may still want this feature on top but it would be more of a nice-to-have at that point 🙂  Up to you.

We do! iterative/websites

Oh lol k

I may still want this feature on top but it would be more of a nice-to-have at that point 🙂

Well... I just tried using tabs [at the end of a doc](I may still want this feature on top but it would be more of a nice-to-have at that point 🙂) and the side menus move a lot with the height. I don't see how an animation would help there -- it'd still be quite distracting. Equal/max height may be the best way to go after all @julieg18 ?

Well... I just tried using tabs [at the end of a doc](I may still want this feature on top but it would be more of a nice-to-have at that point 🙂) and the side menus move a lot with the tab changes (if you scroll down enough to see the page footer). I don't see how an animation would help there -- it'd still be quite distracting. Equal/max height may be the best way to go after all @julieg18 ?

Sounds good! We'll put our focus on adding a prop for equal tab height.

I have tried setting the height of the tab to maximum content height, which introduces the possibility of having a large gap.
Here is the demo video:
https://user-images.githubusercontent.com/20840228/150820948-b9dde230-d08c-409d-a63b-5ad871eaf7c5.mov
So, I think we need to introduce some maximum default height which of course will introduce a scrollbar. Will that be okay?
Here is the demo video:
https://user-images.githubusercontent.com/20840228/150822440-1ce69473-5bf8-484e-8c00-a89e7b1e7222.mov
Also, we may also have props in case we want height more than the maximum default height.
cc @jorgeorpinel  @julieg18

@yathomasi could we explore other docs/products with tabs, share some links, etc - how do they do this?
as we discussed today during the call, I would also to better understand how big of a problem it is.
If this is about configuring - Windows vs Linux vs Mac - IMO it becomes less of a problem since you switch once, and don't pay attention to the other tabs.
If tabs are being used instead of sections, and there is an expectation for users to read all of them to get the comprehensive meaning of the page, that's where it becomes a problem. But it's not clear that this use case is the right one for tabs in the first place
cc @jorgeorpinel - what's your take?

introduces the possibility of having a large gap

Yeah I was thinking equal max height would have to be enabled via a prop.

we may also have props in case we want height more than the maximum default height

OK, this could be an alternative solution, which prop would that be? Do you mean something like <tab height="30m"> with some hardcoded dimension? Thanks @yathomasi

how big of a problem it is

@shcheklein not a big problem unless we use tabs near the end of a page. Per https://github.com/iterative/dvc.org/issues/2939 we're trying to avoid very creative uses of tabs: mainly focus on language or format switching (users would only switch once mostly), at first at least.

I have opened a pull request in cml.dev as cml.dev seems to use the toggle more. I ended up providing a simple implementation that uses height as a prop on the toggle component.

Cool, thanks! That works. Please link this issue to the PR that ports that feature over. Thanks @yathomasi !

Looks like this is resolved by https://github.com/iterative/dvc.org/pull/3257. Thanks again @yathomasi


Keep only organization_guid and space_guid in manifest tags

Pull Request Test Coverage Report for Build 2885

1 of 1 (100.0%)  changed or added relevant line in 1 file are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage decreased (-0.008%) to 90.058%

Files with Coverage Reduction
New Missed Lines
%

common/utils/index.js
1
89.51%

Totals

Change from base Build 2884:
-0.008%

Covered Lines:
9103

Relevant Lines:
9772

💛  - Coveralls


Change relationship declaration to contributor model to match author model

Why was this change made? 🤔
Another attempt to fix https://github.com/sul-dlss/happy-heron/issues/2240 which causes https://app.honeybadger.io/projects/77112/faults/85307350 .... basically a contributor model being saved before the associated work version is saved, throwing a null violation constraint.
This change just makes the relationship declaration on both associated models match.  inverse_of should be inferred for authors as it is for contributors.
How was this change tested? 🤨
Existing tests.

@peetucket I don't understand how defining this relationship explicitly would impact the order in which things are saved in. It already is able to derive this relationship implicitly and none of that logic governs save order.

Interesting -- rubocop doesn't like the missing inverse_of for the Author declaration.  https://www.rubydoc.info/gems/rubocop/0.61.1/RuboCop/Cop/Rails/InverseOf  Apparently using some options prevents Rails from inferring it automatically.  I guess we leave it in.


Move the open package to full dependency

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 89.56%. Comparing base (73f53a0) to head (0c0698f).

Additional details and impacted files
@@           Coverage Diff           @@
##             main       #3   +/-   ##
=======================================
  Coverage   89.56%   89.56%
=======================================
  Files           4        4
  Lines         297      297
  Branches       23       23
=======================================
  Hits          266      266
  Misses         30       30
  Partials        1        1

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


Add files via upload

This is solving FizzBuzz using a repeat while loop.
I haven't seen a solution like this so thought I would add it.
I appreciate your time!

Neat! Thanks :)


recipe for target 'redcarpet.so' failed

$ gem install redcarpet -v '3.3.3'

Building native extensions.  This could take a while...
ERROR:  Error installing redcarpet:
	ERROR: Failed to build gem native extension.

    current directory: /usr/local/rvm/gems/ruby-head/gems/redcarpet-3.3.3/ext/redcarpet
/usr/local/rvm/rubies/ruby-head/bin/ruby -r ./siteconf20151023-19873-1ccj2bc.rb extconf.rb
creating Makefile

current directory: /usr/local/rvm/gems/ruby-head/gems/redcarpet-3.3.3/ext/redcarpet
make "DESTDIR=" clean

current directory: /usr/local/rvm/gems/ruby-head/gems/redcarpet-3.3.3/ext/redcarpet
make "DESTDIR="
compiling houdini_html_e.c
compiling rc_markdown.c
rc_markdown.c: In function ‘rb_redcarpet_md_render’:
rc_markdown.c:132:2: warning: ISO C90 forbids mixed declarations and code [-Wdeclaration-after-statement]
  struct rb_redcarpet_rndr *renderer;
  ^
compiling html.c
compiling markdown.c
markdown.c: In function ‘char_link’:
markdown.c:1047:3: warning: ISO C90 forbids mixed declarations and code [-Wdeclaration-after-statement]
   struct buf id = { 0, 0, 0, 0 };
   ^
markdown.c:1087:3: warning: ISO C90 forbids mixed declarations and code [-Wdeclaration-after-statement]
   size_t nb_p = 0;
   ^
markdown.c: In function ‘sd_markdown_render’:
markdown.c:2820:2: warning: ISO C90 forbids mixed declarations and code [-Wdeclaration-after-statement]
  int footnotes_enabled  = md->ext_flags & MKDEXT_FOOTNOTES;
  ^
markdown.c:2838:43: warning: passing argument 1 of ‘is_codefence’ discards ‘const’ qualifier from pointer target type
   if (codefences_enabled && (is_codefence(document + beg, doc_size - beg, NULL) != 0))
                                           ^
markdown.c:1391:1: note: expected ‘uint8_t *’ but argument is of type ‘const uint8_t *’
 is_codefence(uint8_t *data, size_t size, struct buf *syntax)
 ^
markdown.c: At top level:
markdown.c:94:15: warning: ‘char_underline’ declared ‘static’ but never defined [-Wunused-function]
 static size_t char_underline(struct buf *ob, struct sd_markdown *rndr, uint8_t *data, size_t offset, size_t size);
               ^
markdown.c:95:15: warning: ‘char_highlight’ declared ‘static’ but never defined [-Wunused-function]
 static size_t char_highlight(struct buf *ob, struct sd_markdown *rndr, uint8_t *data, size_t offset, size_t size);
               ^
compiling html_smartypants.c
html_smartypants.c: In function ‘squote_len’:
html_smartypants.c:104:2: warning: initialization discards ‘const’ qualifier from pointer target type
  static char* single_quote_list[] = { "'", "'", "'", "'", NULL };
  ^
html_smartypants.c:104:2: warning: initialization discards ‘const’ qualifier from pointer target type
html_smartypants.c:104:2: warning: initialization discards ‘const’ qualifier from pointer target type
html_smartypants.c:104:2: warning: initialization discards ‘const’ qualifier from pointer target type
html_smartypants.c: In function ‘smartypants_squote’:
html_smartypants.c:149:30: warning: comparison between signed and unsigned integer expressions [-Wsign-compare]
    uint8_t next_char = (size > 1+next_squote_len) ? text[1+next_squote_len] : 0;
                              ^
html_smartypants.c: In function ‘smartypants_cb__amp’:
html_smartypants.c:250:2: warning: ISO C90 forbids mixed declarations and code [-Wdeclaration-after-statement]
  int len = squote_len(text, size);
  ^
compiling houdini_href_e.c
compiling stack.c
compiling buffer.c
buffer.c: In function ‘bufcstr’:
buffer.c:110:45: warning: passing argument 1 of ‘bufgrow’ discards ‘const’ qualifier from pointer target type
  if (buf->size + 1 <= buf->asize || bufgrow(buf, buf->size + 1) == BUF_OK) {
                                             ^
buffer.c:59:1: note: expected ‘struct buf *’ but argument is of type ‘const struct buf *’
 bufgrow(struct buf *buf, size_t neosz)
 ^
compiling autolink.c
compiling rc_render.c
rc_render.c: In function ‘rndr_link_attributes’:
rc_render.c:287:28: warning: variable ‘rndr’ set but not used [-Wunused-but-set-variable]
  struct rb_redcarpet_rndr *rndr;
                            ^
linking shared-object redcarpet.so
/usr/bin/ld: cannot find -lgmp
collect2: error: ld returned 1 exit status
Makefile:254: recipe for target 'redcarpet.so' failed
make: *** [redcarpet.so] Error 1

make failed, exit code 2

Gem files will remain installed in /usr/local/rvm/gems/ruby-head/gems/redcarpet-3.3.3 for inspection.
Results logged to /usr/local/rvm/gems/ruby-head/extensions/x86_64-linux/2.3.0/redcarpet-3.3.3/gem_make.out

fixed after:
aptitude install libgmp-dev

Let's continue the discussion in #523 ; thanks for reporting anyway! :-)


Image conversion on upload

Author Name: Christian Hüning (Christian Hüning)
Original Redmine Issue: 3132, https://dev.opennebula.org/issues/3132
Original Date: 2014-08-06

It would be great if you could select to automatically convert an image when uploading it to the datastore.
We have our users creating images with Virtual Box and tell them to use the qcow format. However we would like to use qcow2 instead. Though it's easily possible to make this conversion, it would be very, very convenient if it could be done automatically in the upload dialog.
We would also contribute some work into this feature to make it happen!
Cheers,
Christian

Original Redmine Comment
Author Name: Carlo Daffara (Carlo Daffara)
Original Date: 2014-08-06T13:54:44Z

I would certainly second that. It can be implemented though qemu-img, currently supports raw, qcow2, vhd, vmdk and several other formats very reliably; as an added requirement it would be quite small as it does not need the entire qemu-kvm hypervisor to work.

Original Redmine Comment
Author Name: Ruben S. Montero (@rsmontero)
Original Date: 2014-08-06T17:30:27Z

Hi
I'm moving this to backlog for possible next release, note that some Datastore drivers are doing this right now, converting from qcow2 to raw for Ceph, to vmfs for VMware and so on... Meanwhile you can easily insert the qemu-img command in the cp script (remotes/datastore/fs). You may take other ds as a reference e.g. ceph/ds does the opposite.
Cheers
Ruben

Original Redmine Comment
Author Name: Christian Hüning (Christian Hüning)
Original Date: 2014-08-06T18:36:29Z

Hi,
thanks! Will try the cp script tomorrow.
Cheers,
Christian

Original Redmine Comment
Author Name: Christian Hüning (Christian Hüning)
Original Date: 2014-08-10T12:26:57Z

Well, i figured that it would be a too static behaviour if every image type would get converted into qcow2. I'd really like to select and control this from Sunstone. So i guess I'll wait for an upcoming release.

Ideally, this would be implemented on an image basis like oneimage create -d 1 --name raw_to_qcow2 --convert qcow2 --path http://service.ci/images/alpine-testing.raw basically extending the one.image.allocate call. It would also require Sunstone/Fireedge support.


feat: fix the e2e test case of the server-info plugin for APISIX v2.13

fix the e2e test case of the server-info plugin for APISIX v2.13
#952
Type of change:

[x] Bugfix
[ ] New feature provided
[ ] Improve performance
[ ] Backport patches

What this PR does / why we need it:

Pre-submission checklist:

[x] Did you explain what problem does this PR solve? Or what new features have been added?
[ ] Have you added corresponding test cases?
[ ] Have you modified the corresponding document?
[ ] Is this PR backward compatible? If it is not backward compatible, please discuss on the mailing list first

Codecov Report

Merging #959 (d43ec24) into master (22cfb5e) will not change coverage.
The diff coverage is n/a.

:exclamation: Current head d43ec24 differs from pull request most recent head e20bee0. Consider uploading reports for the commit e20bee0 to get more accurate results

@@           Coverage Diff           @@
##           master     #959   +/-   ##
=======================================
  Coverage   31.75%   31.75%
=======================================
  Files          72       72
  Lines        7935     7935
=======================================
  Hits         2520     2520
  Misses       5139     5139
  Partials      276      276

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 22cfb5e...e20bee0. Read the comment docs.

please check the CI jobs error

re-run CI jobs.


http://stratux.me/ is 404

http://stratux.me/ is 404
Config issue? And why not https?

Chris has closed his website, what are you looking for? A lot of information can be found on our Europe Edition fork: https://github.com/b3nn0/stratux.

No, it wasn't closed... Sean (Crew Dog) manages it now. The registrar was changed and there was an issue with transitioning the hosting. It should be back before too long.

DNS records updated with the new registrar. Shows stratux.me domain as being verified on GitHub. Still getting a 404 error. Anyone good with gh-pages? I looked through the FAQ and couldn't figure out why we are still getting the error.

Snapshot from June 25, 2022 available from the Wayback machine in the meantime http://web.archive.org/web/20220625024754/http://stratux.me/

DNS records updated with the new registrar. Shows stratux.me domain as being verified on GitHub. Still getting a 404 error. Anyone good with gh-pages? I looked through the FAQ and couldn't figure out why we are still getting the error.

I am good with static websites and have done a few things in that arena (GH Pages). I am happy to help.

I noticed that the archived page is http://    I have seen issues like this
before, where pages won't render if not https://   As Microsoxx owns
github, I would not put it past them to fail to direct pages based on
"security".
historically, we operated on git hub this way:
https://stackoverflow.com/questions/40659168/difference-between-gitgithub-com-and-https-github-com
See resolving conflicts with mixed content here:
https://docs.github.com/en/pages/getting-started-with-github-pages/securing-your-github-pages-site-with-https
*The page owner should take a look at the "enforce https" check box and see
it is checked.  If so, unchecking it may cure the issue. *
It may be that you can no longer uncheck said box. I can't load the page
either. My antivirus flags the site as insecure before even though landing
on the 404 page!
*Also see *Resolving problems with mixed content
I am thinking that the issue may be here:
Enforcing HTTPS for your GitHub Pages site

On GitHub, navigate to your site's repository.
2.
Under your repository name, click

Settings. [image: Repository settings button]
In the "Code and automation" section of the sidebar, click

Pages*.

Under "GitHub Pages," select Enforce HTTPS. [image: Enforce HTTPS
checkbox]
Troubleshooting certificate provisioning ("Certificate not yet created"
error)
When you set or change your custom domain in the Pages settings, an
automatic DNS check begins. This check determines if your DNS settings are
configured to allow GitHub to obtain a certificate automatically. If the
check is successful, GitHub queues a job to request a TLS certificate
from Let's
Encrypt https://letsencrypt.org/. On receiving a valid certificate,
GitHub automatically uploads it to the servers that handle TLS termination
for Pages. When this process completes successfully, a check mark is
displayed beside your custom domain name.
The process may take some time. If the process has not completed several
minutes after you clicked Save, try clicking Remove next to your custom
domain name. Retype the domain name and click Save again. This will
cancel and restart the provisioning process.
On Thu, Dec 15, 2022 at 12:54 PM mnordhaus @.***> wrote:

DNS records updated with the new registrar. Shows stratux.me domain as
being verified on GitHub. Still getting a 404 error. Anyone good with
gh-pages? I looked through the FAQ and couldn't figure out why we are still
getting the error.
I am good with static websites and have done a few things in that arena
(GH Pages). I am happy to help.
—
Reply to this email directly, view it on GitHub
https://github.com/cyoung/stratux/issues/858#issuecomment-1353488756,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AHAT66DM6PU74IH6LWMWKBLWNNLNPANCNFSM6AAAAAAQCGA5NE
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>


Honors college does not want to receive committee approved emails

Consider preventing their email from being CC'd on the email that goes out when the committee approves.  This would add logic that we are trying to avoid, so I'd be fine with denying this feature request.
Service Now ticket: https://pennstate.service-now.com/now/nav/ui/classic/params/target/incident.do%3Fsys_id%3D5df524f187d9cad0e0624c8e0ebb35b1%26sysparm_stack%3Dincident_list.do%3Fsysparm_query%3Dactive%3Dtrue

Let's not do this.  We are trying to avoid excess logic for small things like this.


Creating version support (URIs won't change)

@proycon I am merging because there is no major change to review, it's just addressing issue https://github.com/SoftwareUnderstanding/software-iodata/issues (for this repo)


How to use Observables for emitting events?

Hello,
I have a question about the usage of rxgo.
When using rxjs I usually use Subjects and .asObservable to get a simple event pattern. However in rxgo there is no Subject (yet?) and from the tutorials I could not find any way to pass data at runtime.
In the tutorials all observers are generated from existing arrays or functions which provide data, however Usually I would do something like this:
type MyStruct struct {
    s subject.Subject
}

func NewMyStruct() MyStruct {
    m := MyStruct{}
    m.s = subject.New()
    return s
}

func (m *MyStruct) doSomeActionWithNewValue(value interface{}) {
    m.s.Next(value)
}

func(m *MyStruct) getObservable() observable.Observable {
    return m.s.AsObservable()
}

func main() {
    myStruct := NewMyStruct()
    myStruct.getObservable().Subscribe(func(value interface{}) {
        // do work here
    }
    myStruct.doSomeActionWithNewValue("hello world")
}

Is it just because subjects are not yet implemented or is it intended to do something like this in a totally different way in go?
I already tried using a channel as base for the observable and using closures to provide a provider function for the observable, but both resulted in go telling me, that my application deadlocked, when waiting for the subscriptions.

Using iterable
input := make(chan interface{})
iter, _ := iterable.New(input)
source := From(iter)
source.Subscribe(handlers.NextFunc(func(value interface{}) {
	// do work here
}))
input <- "hello world"

Or you can impl your own Iterator
https://github.com/ReactiveX/RxGo/blob/master/iterator.go

Subject is simply an Observer and Observable, and I don't see a good use to implement this at this point. As per @raininfall you can implement an Iterator for your case or wrap a <-chan interface{} with Observable:

input := make(chan interface{})
go func() {
        input <- "hello world"
        close(input)
}()

<-Observable(input).Subscribe(myHandler)

Closing and merging with issue #54


[INLONG-6771][Manager] Add login failure limit

Prepare a Pull Request
(Change the title refer to the following example)

Title Example: [INLONG-XYZ][Component] Title of the pull request

(The following XYZ should be replaced by the actual GitHub Issue number)

Fixes #6771

Motivation
if the user login fails, we should limit the login times to prevent violent retries.
here we can limit it to 10 fails times and wait for 3 minutes.
Modifications
loginService
Verifying this change

[x] This change is already covered by existing tests, such as:
AnnoControllerTest

Documentation

Does this pull request introduce a new feature? (yes / no)
If yes, how is the feature documented? (not applicable / docs / JavaDocs / not documented)
If a feature is not applicable for documentation, explain why?
If a feature is not documented yet in this PR, please create a follow-up issue for adding the documentation

@fuweng11 PTAL, thanks.


Right parenthesis in a filename breaks CSS output

STOP! If your issue is with URI rewriting, read the wiki first:
http://code.google.com/p/minify/wiki/CommonProblems#URIs_are_re-
written_incorrectly_in_CSS_output
Please remove this message from the issue.

Minify version:
PHP version: 2.1.3

What steps will reproduce the problem?
1. Use brackets in a filename in a CSS declaration e.g.
div.MultiBoxLoading
{
	background: url('/images/multibox/ajax-loader(2).gif') no-repeat center;
}

Expected output:
Minified CSS

Actual output:
Broken minified CSS

Did any unit tests FAIL? (Please do not post the full list)
Not sure what this means, sorry. I have only just started using minifier

Please provide any additional information below. If this has to do with URI
rewriting, please include your DOCUMENT_ROOT, the

Original issue reported on code.google.com by<EMAIL_ADDRESS>on 19 Feb 2010 at 3:47

You're right that this is valid CSS content (it's a bug in our URL matching),
but
URLs should always have "(", ")", and several other chars encoded as they were
declared "unsafe": http://tools.ietf.org/html/rfc1738#page-3

Hence, I'm leaning on "WontFix"-ing this unless the fix would be simple. This
line
captures the url(...) bit:
http://code.google.com/p/minify/source/browse/tags/release_2.1.3/min/lib/Minify/
CSS/UriRewriter.php#79
But passes the inside bit (possibly containing quotes) here:
http://code.google.com/p/minify/source/browse/tags/release_2.1.3/min/lib/Minify/
CSS/UriRewriter.php#143

Original comment by<EMAIL_ADDRESS>on 20 Feb 2010 at 7:39

Changed title: Right parenthesis in a filename breaks CSS output
Changed state: Accepted
Added labels: Priority-Low
Removed labels: Priority-Medium

FYI: Oops RFC1738 doesn't say parenthesis are unsafe in URLs, in fact they're
"special".

Original comment by<EMAIL_ADDRESS>on 10 May 2010 at 7:48


Feature request: adding severity in ModSecurityIntervention

Hi, I think would be very useful adding in ModSecurityIntervention a severity field populated by disruptives evaluate function.
In this way a connector can use the intervention.log based the severity of the log.
For example https://github.com/SpiderLabs/ModSecurity-nginx/issues/274

Hello @FedericoHeichou ,
Could you describe more fully what you're hoping to achieve here. Perhaps a specific example that outlines the tangible benefit?

Hello @FedericoHeichou ,
Could you describe more fully what you're hoping to achieve here. Perhaps a specific example that outlines the tangible benefit?

Hello @martinhsv
For example in the nginx connector Modsecurity-nginx there are these lines of code
    log = intervention.log;
    if (intervention.log == NULL) {
        log = "(no log message was specified)";
    }

    ngx_log_error(NGX_LOG_ERR, (ngx_log_t *)r->connection->log, 0, "%s", log);

As you can see it prints everything with the severity level NGX_LOG_ERROR. If I set in a rule that a transaction has a severity of DEBUG the connector should know the severity, so it can use it to handle own right log level. With the code committed the Modsecurity-nginx code could became
    log = intervention.log;
    if (intervention.log == NULL) {
        log = "(no log message was specified)";
    }

    ngx_log_error(intervention.severity, (ngx_log_t *)r->connection->log, 0, "%s", log);

(I don't know if they uses the same level so we should check if the levels are the same but it is just an example)

HI @FedericoHeichou ,
Ignore the source code for now. What I was really prompting for was a description of what the benefit is, or what the user-visible effect is.
Suppose you have two rules, something like:
SecRule ... "... id:1001, severity:CRITICAL"
SecRule ... "... id:1002, severity:WARNING"

And the nginx config includes:
error_log /var/log/nginx/error.log error;

With your suggestion here, how would rules 1001 and 1002 be treated differently? What would look different as far as the admin is concerned? And how is this different from current functionality?

HI @FedericoHeichou ,
Ignore the source code for now. What I was really prompting for was a description of what the benefit is, or what the user-visible effect is.
Suppose you have two rules, something like:
SecRule ... "... id:1001, severity:CRITICAL"
SecRule ... "... id:1002, severity:WARNING"

And the nginx config includes:
error_log /var/log/nginx/error.log error;

With your suggestion here, how would rules 1001 and 1002 be treated differently? What would look different as far as the admin is concerned? And how is this different from current functionality?

With your example it now prints both 1001 and 1002 even though you set the logger level to "error" and the 1002 is "warning" and the nginx.conf asking to add to the error.log file only >= errors.
Adding the severity in the "intervention" it would be possible to modify a connector to print the intervention in the right logging level, in this way only 1001 would be added in the log file with the error log you specified.
As a sysadm I added the code I committed to my running modsecurity (and modified modsecurity-nginx too) because it was creating too much logs because every severity was threaten like a "error"

I see. I can think of at least a couple of reasons to be hesitant about an approach like that.

Although both Apache HTTP Server and nginx use comparable log level structures, we cannot (and should not) assume that that will always be the case. Creating unnecessary dependencies and linkages makes ModSecurity less flexible for the future. For example, I believe IIS already does not have the same syslog-inspired log levels.

I don't think we should assume that it's even desirable that there be a direct mapping between the 'severity' action of a ModSecurity rule and the severity of any output in error.log. They are intended to reference different things. One is the relative severity of a ModSecurity rule, while the other is the relative severity of an effect on the web server. To make an analogy. if a web server might result in output to syslog in some cases, I wouldn't assume that the syslog level must be the same as a related message in the web server's error.log. And must a 'CRITICAL' rule really generate a 'crit' level log line in the error.log rather than 'error'?

[-- or maybe it's just 2b)] Tying the two severities together makes it impossible for a rule to generate different error.log severities under circumstances separate from rule itself. For example, if a rule has a severity of ERROR, we may want that rule to result in an 'error' level message in error.log when SecRuleEngine On but not when SecRuleEngine DetectionOnly

I see. I can think of at least a couple of reasons to be hesitant about an approach like that.

Although both Apache HTTP Server and nginx use comparable log level structures, we cannot (and should not) assume that that will always be the case. Creating unnecessary dependencies and linkages makes ModSecurity less flexible for the future. For example, I believe IIS already does not have the same syslog-inspired log levels.
I don't think we should assume that it's even desirable that there be a direct mapping between the 'severity' action of a ModSecurity rule and the severity of any output in error.log. They are intended to reference different things. One is the relative severity of a ModSecurity rule, while the other is the relative severity of an effect on the web server. To make an analogy. if a web server might result in output to syslog in some cases, I wouldn't assume that the syslog level must be the same as a related message in the web server's error.log. And must a 'CRITICAL' rule really generate a 'crit' level log line in the error.log rather than 'error'?
[-- or maybe it's just 2b)] Tying the two severities together makes it impossible for a rule to generate different error.log severities under circumstances separate from the rule itself. For example, if a rule has a severity of ERROR, we may want that rule to result in an 'error' level message in error.log when SecRuleEngine On but not when SecRuleEngine DetectionOnly

I think it should be the connector's task to map the "standard" (maybe the facto) logging level to its logging level
You are be right but I could ask you: do you really want a DEBUG/INFO severity rule being logged as ERROR in a connector? Modsecurity has the settings to prevent the execution of the callback (setting it to warn will never execute the logging callback with info), still the severity is not provided in the response, so a connector can not know how to act. A better example of usage can just be a connector can provide a setting of "modsecurity-max-logging-level" and for example if you set to WARN it will send to logger everything as ERROR but only with level >= WARN. If a connector doesn't provide this setting or a user doesn't use it it will print everything as the current implementation. I think this would be a better usage of this intervention->severity


Convert Expo project to Original RN App

I did yarn run eject to eject but it gave me this warning
Warning! We found at least one file where your project imports the Expo SDK

I know I have some modules which use Expo API like this -
await Expo.Font.loadAsync({
      Roboto: require('native-base/Fonts/Roboto.ttf'),
      Roboto_medium: require('native-base/Fonts/Roboto_medium.ttf'),
    });

Now I want to convert this to React Native Components so that I can eject without any errors so that I get
index.android.js & index.ios.js in the root folder like we do while using react-native init example

After 4 hours, I got the answer..
Had to install exp using npm i -g exp & then used exp detach to detach the project
Also I added some fields in app.json for it to work as follows
{
  "expo": {
    "name": "Project",
    "slug": "project",
    "sdkVersion": "18.0.0",
    "privacy": "public",
    "android": {
      "package": "com.example.project"
    }
 }
}

Docs are given here

IS your detach sucess how you able build apk after detach?
Can we run it using react-native run-android?

IS your detach sucess how you able build apk after detach?

Yep detach was successful. Follow this for building apk after detach.

Can we run it using react-native run-android?

Not sure what you're saying. But if I remember, npm start will do the job.

Docs link is not working ...
404
This page could not be found.

Try again


chore: update year on the chainsafe cli notice

Motivation
It's 2023 already
Description
Update year on the chainsafe cli notice

Why there has to be a year there BTW?

Why there has to be a year there BTW?

Not sure TBH... is this a requirement that was put in there from sometime back in the early days @wemeetagain ? Would prefer for it to be automated, but not a huge deal.

Good idea!

:tada: This PR is included in v1.9.0 :tada:


Rename isolated() modifier as standalone()

Motivation
It is not as intuitive as it could be; standalone() is a better name. isolated() is meant to declare independence of the view, such that it does not need the view’s cache providing capabilities. A better word for this, however, is standalone().

Renamed in 55263d37e78f4776c984f3525b529705be68fcd6.


Publish for Scala 2.12.17, please?

2.12.17 JARs are on Maven Central, as per https://contributors.scala-lang.org/t/scala-2-12-17-release-planning/5805

Released for versions 4.5.11-4.5.13 : https://repo1.maven.org/maven2/org/scalameta/semanticdb-scalac-core_2.12.17/
Though 4.5.12 is still pending.


File passes jshint - manual-trigger.js

Added semicolon and changed == to === statement so file will pass jshint.

Infinite Scroll v3 is out!. It's an all new re-write, so take a look at how to upgrade from v2.  With the brand new release, this contribution is no longer applicable.
Thank you for your interest and effort in Infinite Scroll. Sorry I had to pass on this one. 🌈🐻


Python formatter doesn't follow the setting

Does this issue occur when all extensions are disabled?: Yes

VS Code Version: 1.65.2
OS Version: win10

Steps to Reproduce:

mysetting
"settings": {
"autoSnippet.snippets": [
{
"language": "python",
"snippet": "python-future-features"
}
],
"editor.fontSize": 13,
// "files.insertFinalNewline": true,
// "files.trimFinalNewlines": true,
// "files.trimTrailingWhitespace": true,
"python.linting.flake8Enabled": true,
"python.linting.enabled": true,
"python.linting.flake8Args": [
"--max-line-length=79"
],
"python.formatting.provider": "yapf",
"python.linting.flake8Path": "d:\program\Anaconda3\envs\batcave3\Scripts\flake8.exe",
"python.formatting.yapfPath": "d:\program\Anaconda3\envs\batcave3\Scripts\yapf.exe",
"python.linting.flake8CategorySeverity.W": "Information",
}

When I press alt+shift+F, the vscode shows "Formatting with autopep8", not yapf.

Also, the linter flake8 doesn't show the hint message -- it seems like not working.

I can use flake8 and yapf in my terminal, and the setting of ‘fontSize’ is working.

Please start by filing an issue with the Python extension https://github.com/microsoft/vscode-python


Update tekton pruner logic

We need to adjust our tekton pruner logic so it will be deleting pods and pvcs separately so we will be able to clean these resources independently.

The "pruner" resource of the "edp-tekton" Helm chart has been updated. Now it allows to separately clean pvcs and pods, thus allowing to preserve the Tekton pipeline run history for the desired amount of time without keeping unused pvcs.
This functionality is implemented by two parameters:
tekton.pruner.recentMinutes_pods (formerly recentMinutes)
tekton.pruner.recentMinutes_pvcs


Blog post: new Rice document published in 2023

@anermina can you please help write a blog post about this new Rice document and explain its changes? Thanks!
Originally posted by @ronaldtse in https://github.com/metanorma/mn-samples-iso/issues/89#issuecomment-2016759474

Also incorporate content from:

https://github.com/metanorma/mn-samples-iso/issues/97
https://github.com/metanorma/mn-samples-iso/issues/95
https://github.com/metanorma/mn-samples-iso/issues/96


Support Orchestration restart

Trying to manage an Eternal Orchestration is difficult due to:

Must provide instance id at startup and check if already running.
If orchestration was somehow terminated, it is impossible to restart.

I believe there is a TODO item in the CreateInstance stored procedure which would address this, but it appears the database schema itself blocks this since it would cause a duplicate key. Perhaps I'm misunderstanding, shouldn't ExecutionId be used as part of the unique identifier? In other words, a combination of TaskHub, InstanceId, and ExecutionId signify a unique orchestration, with ContinueAsNew starting a new Orchestration with the same InstanceId and different ExecutionId.
This ticket requests a solution for restarting orchestrations using the same InstanceId.

Support for this is now released: https://github.com/microsoft/durabletask-mssql/releases/tag/v1.0.0-rc

Wicked fast, thank you!


Compatibility/fakeplayer issue - Broken elevators -SpongeForge&GriefPrevention

Since the latest update (7.27) a compatibility issue has occurred where when the RFTools elevator is being used the server will get hung up and time out all players. Once this occurs then the server is stuck in a frozen state.
This issue occurs as soon as the elevator platform is set in motion.
Issue does not occur on previous version (7.26)
When this occurs the last message the server console spits out is this:
[22:31:49] [Server thread/WARN] [Sponge]: No entity type is registered for class net.minecraftforge.common.util.FakePlayer
I'm wondering if an error was made in recent changes to: https://github.com/McJty/RFTools/blob/19f938d39f9b1e5abf3200f47e7961f63d112092/src/main/java/mcjty/rftools/blocks/builder/BuilderTileEntity.java
I'm no coder so I wouldn't know, but that's my only guess as the issue only started to occur on the latest version.
I also ran some debug checks with GriefPrevention for SpongeForge but found no apparent issues on their side.

I tried working through the problem with Blood over on the GriefPrevention issue tracker and he seems to have traced it back to an issue within RFTools:
https://github.com/MinecraftPortCentral/GriefPrevention/issues/587#issuecomment-362841939

Ah, given that information, I was able to reproduce this without Sponge, by using a block protector to protect the platform block immediately in front of the elevator. I'll upload a fix shortly.

@josephcsible Please pass the player interacting with ScreenTileEntity instead of a fakeplayer so GriefPrevention can protect properly.

But the elevator shouldn't consider different blocks to be part of its platform when you call it with redstone versus when you call it with a screen.

The player triggering the move should always be passed in Forge's 'BlockEvent.BreakEvent'. Only use fakeplayer when you have no player. In this case you should always have a player.

It's the elevator that's doing the breaking though, not the player. When a player starts up an RFTools Builder, it's the builder that counts as breaking all of the blocks. Are there any other modded blocks that work this way?

It doesn't matter what block is doing the actual breaking. What matters is what "caused" the block to break. In this case the player that interacted with the ScreenTileEntity caused the elevator to move. And yes plenty of other mods work this way. The ones that do not are doing it wrong.

@bloodmc I disagree, but I'll ask @McJty to have the final say since it is his mod.
@rob5251 This will be fixed in the next release.

Thanks. 👍

@bloodmc When elevator/builder are activated using a screen I can in principle know the player. However all machines in RFTools can also be operated using redstone logic and machines that procude redstone. In such situations I don't have a player and a fake player is required.


Errors not forwarded in hooks

When an error occurs in the onRequest hook, the error is not shown in the console and no request is send.
I would expect an error that occurs in the hook to still be shown in the console.
The following will create a fetch object that makes no requests, but prints no error.
$fetch.create({
  async onRequest(ctx): Promise<void> {
    throw "debug error"
  }
})

Let's track via https://github.com/unjs/ofetch/issues/396


Change build config

Changed webpack config to apply optimizations only for the production build
Changed development sourcemap from source-map to cheap-module-eval-source-map
Changed dependencies

Go ts!

@isitinme sounds like a feature request :) Actually might be a good idea for a separate branch or documentation section.


Clockwork: Create x Valkyrien Skies crash issue

Is it a mod compatibility issue?
Yes
If it's a mod compatibility issue, please provide the link(s) to the incompatible mod(s)
https://www.curseforge.com/minecraft/mc-mods/create-clockwork/comments
Minecraft version
1.20.1
Describe the issue
Just crashes on start up.
latest.log

also another issue is Eureka! Ships! for Valkyrien Skies when you make a ship it completely disappears been testing it in 1.18, you get stuck there completely and can't move and you have to kill yourself to get out of this block state

Ikr?!
On Thu, May 30, 2024, 1:35 PM Miner34dev @.***> wrote:

This is weird. The log says something about a missing data fixer. Weird,
if i remember correctly this generally happens when you disable the DFU.
Anyways i will try on my pc to see if it is a third party mod or not. Would
be great to have this to work with clockwork (to have stuff like cameras).
—
Reply to this email directly, view it on GitHub
https://github.com/iPortalTeam/ImmersivePortalsMod/issues/1551#issuecomment-2140410331,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AMQXCJDZ76HCUD7HCZ6G2O3ZE5PPTAVCNFSM6AAAAABG3QVV26VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCNBQGQYTAMZTGE
.
You are receiving this because you authored the thread.Message ID:
@.***>

Can confirm.
[Render thread/ERROR]: No data fixer registered for valkyrienskies:ship_mounting_entity [Render thread/ERROR]: No data fixer registered for valkyrienskies:vs_physics_entity

@Miner34dev  This is not the issue; that happens even without immersive portals. VS2 changes too much rendering jargon for it to be compatible with immersive portals without actual compat work being done, which right now is not a priority.

Isn't there an option to not render portals, but just have the classic purple portal texture + instant warps? Could Immersive Portals possibly work with it in that mode sooner without a massive rewrite?
I love the portal rendering and would be sad to see it go, but at this point I'm mostly attached and reluctant to remove it in favor of VS2 for the functionality of my servers hub area connecting everyone together.

@ficarra1002 Vanilla allows reducing nether portal animation delay via game rule.

I meant for overworld <-> overworld portals. And it preloads chunks even if it's not rendering through doesn't it, immersive portals with the rendering disabled would still be much more instant than vanilla


fix(learn): clear instructions for basic JavaScript

Checklist:

[X] I have read freeCodeCamp's contribution guidelines.
[X] My pull request has a descriptive title (not a vague title like Update index.md)
[X] My pull request targets the master branch of freeCodeCamp.
[X] All the files I changed are in the same world language, for example: only English changes, or only Chinese changes, etc.

Closes #39515

Hi @elcordova  Thank you for your PR, but an earlier PR (#39587) has already been created with the same changes.  We are closing this PR as a duplicate.
We look forward to future contributions from you.
Thanks again! 😊

If you have any questions, feel free to reach out through Gitter or by commenting below. 💬


Enter PIN screen gets mounted twice on app open

Description
When the app is first opened (after onboarding has been completed) the Enter PIN screen appears to be mounted twice. See below:
https://github.com/bcgov/bc-wallet-mobile/assets/32586431/43361685-cb8f-42b5-a607-92e2b7a7b921

Works as intended. Tried on iOS and Android v 1.0.16 build 1636.


Add support for phi-3 models

What does this PR do?
This PR addresses https://github.com/huggingface/optimum/issues/1826 and adds phi-3 support to the normalized config.

Fixes # (issue)
Before submitting

[ ] This PR fixes a typo or improves the docs (you can dismiss the other checks if that's the case).
[ ] Did you make sure to update the documentation with your changes?
[ ] Did you write any new necessary tests?

Who can review?
@fxmarty, @echarlaix, @JingyaHuang, @michaelbenayoun

Thanks @Whadup, this should have been tackled by #1841.


ESM Problems with Jest@26 and Node.js 14.x

Hi, I'm using jest to test my project.
Trying to test a file where I import uuid results with an error:
 ● Test suite failed to run

    SyntaxError: The requested module 'uuid' does not provide an export named 'v1'

          at async Promise.all (index 0)
      at jasmine2 (node_modules/jest-jasmine2/build/index.js:228:5)

I saw @dps910 comment and I guess it's something similar.
I'm using UUID@8, node@14 and Jest@26 (and it's freshly-supported esm modules implementation).
Jest command:
node --experimental-vm-modules .\node_modules\jest\bin\jest.js
The file that I'm testing imports uuid like you suggested:
import { v1 as uuidv1 } from 'uuid';
uuidv1();

It fails only when I'm testing though.
Originally posted by @Tzahile in https://github.com/uuidjs/uuid/issues/245#issuecomment-628650062

Looks like "exports" is not supported yet.
https://github.com/facebook/jest/issues/9430#issuecomment-623941616

@Tzahile as pointed out by @TrySound, ESM support in jest is not complete yet. I think you will have to wait for ESM support in jest to be finished before you can use it with this library.

FWIW, tracking issue: https://github.com/facebook/jest/issues/9771

My workaround for this was to manually map the uuid module to it's commonJS export:
// jest.config.js
moduleNameMapper: {
    // Force module uuid to resolve with the CJS entry point, because JEst does not support package.json.exports. See https://github.com/uuidjs/uuid/issues/451
    "uuid": require.resolve('uuid'),
},

I can confirm #616 fixes this issue in my project using Jest 28 and jest-environment-jsdom. Thanks for working on it.

So that seems like an awesome workaround, but it doesn't work if you're using a jest.config.mjs instead as you can't require things there.  Any thoughts on what the syntax might be in that case?

https://nodejs.org/api/module.html#modulecreaterequirefilename

My workaround for this was to manually map the uuid module to it's commonJS export:
// jest.config.js
moduleNameMapper: {
    // Force module uuid to resolve with the CJS entry point, because Jest does not support package.json.exports. See https://github.com/uuidjs/uuid/issues/451
    "uuid": require.resolve('uuid'),
},

EDIT: This might not work depending on your projects dependency structure. See microsoft/accessibility-insights-web#5421 (comment) for a much better explanation.

This works and is more scoped than overriding exports conditions, but has the downside that it is essentially a silent yarn resolution; it forces every dependency chain through uuid to use whatever version of uuid happens to be hoisted, even if some chains want older versions.

Thanks so much. This solved my problem which I was suffering since 15 hours. Altough I have jest 28 and did everything on StackOverflow nothing helped except this.

I have released<EMAIL_ADDRESS>that should restore compatibility with Jest. Please try it out and let me know if it fixes the Jest interoperability (to be tested with jest@29.0.0-alpha.1).

Hi @ctavan, for my project we still need the workaround for jest tests to pass after updating to 9.0.0 with jest 29:
# jest.config.js
module.exports {
  moduleNameMapper: {
    // Workaround for Jest not having ESM support yet
    // See: https://github.com/uuidjs/uuid/issues/451
    uuid: require.resolve('uuid'),

Turns out mock service worker (@mswjs) has its own implementation of uuidv4 in a file called uuid.ts.
This caused an unexpected side effect when implementing the suggested moduleNameMapper fix. It caused the following error in the @mswjs package:
TypeError: uuid_1.uuidv4 is not a function

To resolve this, simply use a regex with an imperative start and end like so:
moduleNameMapper: {
  "^uuid$": require.resolve('uuid'),
}

✅ This ensures only the actual uuid package is affected.

Note that moduleNameMapper is not needed for Jest 29 and uuid v9. Anyone still having issues on latest should open a new issue with a reproduction.

@SimenB Yep, that's true and great that the root cause is fixed but until dependency packages are updated it's a waiting game. I'm using next-auth and some @aws-sdk packages which are still reliant on v8 unfortunately.

Hoping this can help everyone solve this problem

Jest Version : "@types/jest": "^29.4.0",
UUID Version : "uuid": "^9.0.0"

Even without the jest.config.js this will work because if you have multiple configs inside your project it will generate an error saying : Multiple Configuration Found problem
So in your __tests__ folder and App-test.tsx if you are using typescript
import 'react-native';
import React from 'react';
import App from '../src/App';
import {v4 as uuidv4} from 'uuid';

// Note: test renderer must be required after react-native.
import renderer from 'react-test-renderer';

it('renders correctly', () => {
  renderer.create(<App />);
});

jest.mock('uuid', () => {
  return {
    v4: jest.fn(() => 1)
  }
})

@pl12133 i love you buddy

I'm still seeing this issue.  uuid 9.0.0 and jest 29.4.0/29.5.0, and node 16.19.1
It looks like jest is not picking up the right version of the uuid module, opting for the one in esm-browser folder
 FAIL  services/user/test/api.test.js
  ● Test suite failed to run

    Jest encountered an unexpected token

    Jest failed to parse a file. This happens e.g. when your code or its dependencies use non-standard JavaScript syntax, or when Jest is not configured to support such syntax.

    Out of the box Jest supports Babel, which will be used to transform your files into valid JS based on your Babel configuration.

    By default "node_modules" folder is ignored by transformers.

    Here's what you can do:
     • If you are trying to use ECMAScript Modules, see https://jestjs.io/docs/ecmascript-modules for how to enable it.
     • If you are trying to use TypeScript, see https://jestjs.io/docs/getting-started#using-typescript
     • To have some of your "node_modules" files transformed, you can specify a custom "transformIgnorePatterns" in your config.
     • If you need a custom transformation specify a "transform" option in your config.
     • If you simply want to mock your non-JS modules (e.g. binary assets) you can stub them out with the "moduleNameMapper" config option.

    You'll find more details and examples of these config options in the docs:
    https://jestjs.io/docs/configuration
    For information about custom transformations, see:
    https://jestjs.io/docs/code-transformation

    Details:

    /Users/jim/development/domo/foyer/node_modules/uuid/dist/esm-browser/index.js:1
    ({"Object.<anonymous>":function(module,exports,require,__dirname,__filename,jest){export { default as v1 } from './v1.js';
                                                                                ^^^^^^

    SyntaxError: Unexpected token 'export'

      40 |
      41 |     setupModelTest() {
    > 42 |         const { MongoMemoryServer } = require('mongodb-memory-server');
         |                                       ^
      43 |
      44 |         // const { MongoMemoryServer } = require('mongodb-memory-server');             // use this if mongo not on host
      45 |         // const { MongoMemoryServer } = require('mongodb-memory-server-core');

      at Runtime.createScriptFromCode (node_modules/jest-cli/node_modules/jest-runtime/build/index.js:1495:14)
      at Object.<anonymous> (node_modules/mongodb-memory-server-core/src/util/lockfile.ts:7:1)
      at Object.<anonymous> (node_modules/mongodb-memory-server-core/src/util/MongoBinary.ts:9:1)
      at Object.<anonymous> (node_modules/mongodb-memory-server-core/src/util/MongoInstance.ts:3:1)
      at Object.<anonymous> (node_modules/mongodb-memory-server-core/src/MongoMemoryServer.ts:15:1)
      at Object.<anonymous> (node_modules/mongodb-memory-server-core/src/index.ts:2:1)
      at Object.<anonymous> (node_modules/mongodb-memory-server/index.js:5:20)
      at Object.require [as setupModelTest] (test/test-utils.js:42:39)
      at Object.setupModelTest (services/user/test/api.test.js:6:6)

Im still seeing it as well.
Node 16,   "uuid": "^9.0.0", "jest": "^29.5.0", "jest-environment-jsdom": "^29.5.0",

GCS uses uuid v8: https://www.runpkg.com/<EMAIL_ADDRESS>


NoneType' object has no attribute 'group

Hi i've got this problem today
Trying to login...
Logged in successfully...
Seems like this video isn't available...

Logged in successfully...
Traceback (most recent call last):
File "anime_dl_main_.py", line 16, in
File "anime_dl_main_.py", line 73, in Main
File "anime_dl\Anime_dl.py", line 30, in init
File "anime_dl\sites\crunchyroll.py", line 42, in init
File "anime_dl\sites\crunchyroll.py", line 65, in single_episode
File "anime_dl\sites\crunchyroll.py", line 425, in m3u8_finder
AttributeError: 'NoneType' object has no attribute 'group'
[15980] Failed to execute script main

Any Workaround ? Still not working

What are you trying to download exactly?
Please pastebin the actual command you are typing in please.
Please remove your email and password before doing so!

What are you trying to download exactly?
Please paste the actual command you are typing in please.
Please remove your email and password before doing so!

@echo off
set /p lien= Enter link :
echo %lien%
pause
anime-dl.exe -i "%lien%" -u "user" -p "password"  -r "720" -v
pause
I precise i always use that for yeras now and i don't work anymore since  6days. I think it's more a backend change frorm Crunchyroll who cause the problem.

I'm using it and I'm not having a problem atm.
What anime are you trying to download?

I'm using it and I'm not having a problem atm.
What anime are you trying to download?

Nothing precize, i try multiple episode on Crunchyroll and it doing the same error

I just did
"./main.py -i https://www.crunchyroll.com/welcome-to-demon-school-iruma-kun -u me -p pass -r 480 -o /home/anime/myshow/
It downloaded my videos fine.  Just curious is all.

I just did
"./main.py -i https://www.crunchyroll.com/welcome-to-demon-school-iruma-kun -u me -p pass -r 480 -o /home/anime/myshow/
It downloaded my videos fine.  Just curious is all.

I tried it work only with no premium episodes. BUT my account is okay and subscribed ( I can read premium video on the website)

Wait a second.. I just realized you're using windows.. My apologies. I have not a clue what your error could be then.. Let me do some looking because I use Linux to do my dirty work.

I'm pretty sure the anime_dl.exe is out dated...

I'm pretty sure the anime_dl.exe is out dated...

Mine is dating from January 2019, I try with the last one on the projet and is not even able to login so ...

It's the anime_dl.exe. Give me a minute to compile a new one and I'll post it here for you.

I've compiled it multiple times with the newest stuff and it's just not working.. I think I know what the issue is so I'll see if I can fix it but below is the issue I'm getting.
Traceback (most recent call last):   File "anime_dl\__main__.py", line 22, in <module>     class Main():   File "anime_dl\__main__.py", line 82, in Main     logger=logger, episode_range=args.range, output=args.output)   File "anime_dl\Anime_dl.py", line 30, in __init__     skipper=skipper, logger=logger, episode_range=episode_range, output=output)   File "anime_dl\sites\crunchyroll.py", line 36, in __init__   File "anime_dl\common\browser_instance.py", line 76, in login_crunchyroll   File "anime_dl\common\browser_instance.py", line 92, in login_check TypeError: a bytes-like object is required, not 'str' [26508] Failed to execute script __main__

Fixed-2020-01-01.zip
Here is a temp fix, it worked for me on my Windows Machine. Let me know if it works for you!
@Thedarklife74

Fixed-2020-01-01.zip
Here is a temp fix, it worked for me on my Windows Machine. Let me know if it works for you!
@Thedarklife74

Same problem.
Trying to login...
Let me check again...
Unable to Log you in. Check credentials again.
Failed Login!!!
Traceback (most recent call last):
File "anime_dl_main_.py", line 22, in
class Main():
File "anime_dl_main_.py", line 82, in Main
logger=logger, episode_range=args.range, output=args.output)
File "anime_dl\Anime_dl.py", line 30, in init
skipper=skipper, logger=logger, episode_range=episode_range, output=output)
File "anime_dl\sites\crunchyroll.py", line 49, in init
NameError: name 'exit' is not defined
[10672] Failed to execute script main

@Xonshiz Any ideas ?

I tried wit a complete fresh install with no .exe and use only python, here are the logs
Trying to login...
Traceback (most recent call last):
File "main.py", line 22, in
class Main():
File "main.py", line 80, in Main
AnimeDL(url=args.input, username=args.username, password=args.password,
File "anime_dl\Anime_dl.py", line 28, in init
sites.crunchyroll.Crunchyroll(
File "\anime_dl\sites\crunchyroll.py", line 33, in init
login_response, cookies, token = anime_dl.common.browser_instance.login_crunchyroll(url=url,
File "\anime_dl\common\browser_instance.py", line 76, in login_crunchyroll
login_check_response, login_cookies = login_check(html_source=login_post.text.encode('utf-8'), cookies=login_post.cookies)
File "\anime_dl\common\browser_instance.py", line 92, in login_check
if "href="/logout"" in html_source:
TypeError: a bytes-like object is required, not 'str'
And here the comand :
@echo off
set /p lien= Enter link :
pause
python main.py -i "%lien%" -u "user" -p "pass" -r "720" -v
pause

What version of Python are you running, I can't remember if I was having issues with Python3 or Python2 not working with the current code base.

What version of Python are you running, I can't remember if I was having issues with Python3 or Python2 not working with the current code base.

Python 3.8.0

I just got home and checked quickly and an older copy of the anime-dl code I have only runs correctly using Python 2.7.17.
I don't have all the extras installed for Python3 like cfscrape and all the other pre-requiresites.  I'm pretty sure this Linux box was rebuilt with only the working things I needed for a web-server and anime download server.
It might be a pain, but I would recommend trying Python 2.7.17.  If that works, then awesome, otherwise I can't remember if 3.6 -> 3.7 > 3.8 have any differences that would affect the anime-dl Python code.

I have Python3.7 and I'm not having any problems at the moment with the current code I gave above. :(

I have Python3.7 and I'm not having any problems at the moment with the current code I gave above. :(

Ok I uninstall Python 3.8 and install python 3.7
Official release of the scripti : Don't work (missing modules)
Release by @EpicUnknown  : Work perfectly
Very weird problem because i have 2 computer where I've didn't change anything and the script stopping working one day so .....    quite weird

@EpicUnknown  I donwload the update who @Xonshiz down and the exe you gave me doesn't work anymore ...
Trying to login...
Let me check again...
Unable to Log you in. Check credentials again.
Failed Login!!!
Traceback (most recent call last):
File "anime_dl_main_.py", line 22, in
class Main():
File "anime_dl_main_.py", line 82, in Main
logger=logger, episode_range=args.range, output=args.output)
File "anime_dl\Anime_dl.py", line 30, in init
skipper=skipper, logger=logger, episode_range=episode_range, output=output)
File "anime_dl\sites\crunchyroll.py", line 49, in init
NameError: name 'exit' is not defined
[14076] Failed to execute script main

I still have the followinf problem with all last updates :
Trying to login...
Logged in successfully...
Traceback (most recent call last):
  File "anime_dl\__main__.py", line 16, in <module>
  File "anime_dl\__main__.py", line 73, in Main
  File "anime_dl\Anime_dl.py", line 30, in __init__
  File "anime_dl\sites\crunchyroll.py", line 42, in __init__
  File "anime_dl\sites\crunchyroll.py", line 65, in single_episode
  File "anime_dl\sites\crunchyroll.py", line 425, in m3u8_finder
AttributeError: 'NoneType' object has no attribute 'group'
[2560] Failed to execute script __main__

I'll check it out tonight but I didn't have any problems with my latest build and I even upgrading to Python 3.8. I'll reply again when I take a look.

Trying to login...
Logged in successfully...
Traceback (most recent call last):
  File "anime_dl\__main__.py", line 22, in <module>
    class Main():
  File "anime_dl\__main__.py", line 82, in Main
    logger=logger, episode_range=args.range, output=args.output)
  File "anime_dl\Anime_dl.py", line 30, in __init__
    skipper=skipper, logger=logger, episode_range=episode_range, output=output)
  File "anime_dl\sites\crunchyroll.py", line 44, in __init__
    self.whole_show(url=url, cookie=cookies, token=token, language=language, resolution=resolution, skipper=skipper, episode_range=episode_range, output=output)
  File "anime_dl\sites\crunchyroll.py", line 127, in whole_show
    dub_list, ep_sub_list = self.episode_list_extractor(page_source=page_source, url=url)
  File "anime_dl\sites\crunchyroll.py", line 185, in episode_list_extractor
    if "(Dub)" in u' '.join(title_value).encode('utf-8').strip():
TypeError: a bytes-like object is required, not 'str'
[7088] Failed to execute script __main__

@EpicUnknown Did you manage to find out something about this error??
Trying to login...
Logged in successfully...
Traceback (most recent call last):
  File "anime_dl\__main__.py", line 22, in <module>
    class Main():
  File "anime_dl\__main__.py", line 82, in Main
    logger=logger, episode_range=args.range, output=args.output)
  File "anime_dl\Anime_dl.py", line 30, in __init__
    skipper=skipper, logger=logger, episode_range=episode_range, output=output)
  File "anime_dl\sites\crunchyroll.py", line 44, in __init__
    self.whole_show(url=url, cookie=cookies, token=token, language=language, resolution=resolution, skipper=skipper, episode_range=episode_range, output=output)
  File "anime_dl\sites\crunchyroll.py", line 127, in whole_show
    dub_list, ep_sub_list = self.episode_list_extractor(page_source=page_source, url=url)
  File "anime_dl\sites\crunchyroll.py", line 185, in episode_list_extractor
    if "(Dub)" in u' '.join(title_value).encode('utf-8').strip():
TypeError: a bytes-like object is required, not 'str'
[7088] Failed to execute script __main__

I sure did! It's a very simple fix and I apologize for not posting about it earlier.
So the script SHOULD check if "title_value" is a byte like object or a regular string.
Because it is needed to be a byte like object for the anime you are downloading, simply make "(Dub)" now b'Dub' and yes that b needs to be in front of the single quotes, not inside.
Happy coding and let me know if it fixes it for you! :)


CTRL_N in insert mode works like...

From VimGolf : Real Vim ninjas count every keystroke - do you?
Start file
organize(cupboard, 3, 2);
prioritize(bureau, 8, 7);
realize(bannister, 4, 4);
moralize(railing, 3,9);

End file
cupboard.organize(3, 2);
bureau.prioritize(8, 7);
bannister.realize(4, 4);
railing.moralize(3,9);

Discussion
Smylers2 user question: Anybody able to explain why neither qai<C-N><C-N>.<Esc>3wdW<CR>q3@aZZ nor qai<C-N>.<Esc>3wdW<CR>q3@aZZ work? I'd expect  at the start of a line to insert the first word on the line (and so need  for the second one), so was surprised on the first line a single  inserting the second word — the further surprised to find on subsequent lines it does insert the first one. So qai.3wdWq3@aZZ does work, but uses up 4 keystrokes inserting then deleting the black line at the start.
udioica user: That's a bug that's only triggered when using  on top of a word at the first byte of a file. I've run into that bug many times before.

Can you provide a readable reproducible example that shows your problem? Sorry, pasting some discussion here does not make much sense.

Thanks for very speedy answer, to resume :
Changing the text from:
organize(cupboard, 3, 2);
prioritize(bureau, 8, 7);
realize(bannister, 4, 4);
moralize(railing, 3,9);

to
cupboard.organize(3, 2);
bureau.prioritize(8, 7);
bannister.realize(4, 4);
railing.moralize(3,9);

with one of macros :
qai<C-N><C-N>.<Esc>3wdW<CR>q3@aZZ
or
qai<C-N>.<Esc>3wdW<CR>q3@aZZ
doesn't work

After this commit https://github.com/zuloloxi/vim/commit/55b5755799f096597fff20e00036680afde5ff6c , the macro:
qai<C-N>.<Esc>3wdW<CR>q3@aZZ

works !

Hi, I could reproduce the issue and created a fix

Thanks a lot for the nice work ! Would be nice to use only CTRL-N -> and to start completion with next word on line (not the current cursor word) .
Best regards !

Fixed by v8.0.0669

Thanks, all the best !


Comparator result negation fix

https://github.com/Azure/azure-sdk-for-java/issues/9109

Can one of the admins verify this patch?

@moderakh How do I re-run the ci?

@moderakh All checks have passed! Thank you!

@moderakh All checks have passed! Thank you!

Looks good, I will get this in now.


Typo Fix for docker-compose up.

Proposed changes
Changes "docker compose up" to "docker-compse up" which is the correct command in documentation.

@EarthSchlange docker compose is indeed a command. See https://docs.docker.com/compose/cli-command/ and https://docs.docker.com/engine/reference/commandline/compose/ for details on the new docker compose command.


Replace PSM with CLB and Update to GHC 9.6.5; cardano-api 8.38, cardano-wallet v2024-03-27

Summary
The main purpose of this PR is to add CLB. A WIP replacement for PSM. In order to do so, we also updated cardano-api and cardano-wallet to newer versions, prompting a significant overhaul. The associated versions of the dependencies should allow conway support.
There's definitely some changes to be made pending review. Such as version bumps etc.
Type of Change
Please mark the relevant option(s) for your pull request:

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[x] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Code refactoring (improving code quality without changing its behavior)
[ ] Documentation update (adding or updating documentation related to the project)

Checklist
Please ensure that your pull request meets the following criteria:

[x] I have read the Contributing Guide
[x] My code follows the project's coding style and best practices
[x] My code is appropriately commented and includes relevant documentation
[x] I have added tests to cover my changes
[x] All new and existing tests pass
[x] I have updated the documentation, if necessary

Testing
Please describe the tests you have added or modified, and provide any additional context or instructions needed to run the tests.
This PR was originally supposed to come with an extra test suite. However, we are having some issues with plutus tx compilation (classic) so I had to temporarily remove it (see last commit: 476b2f4)
Additional Information

@TotallyNotChase

This PR was originally supposed to come with an extra test suite. However, we are having some issues with plutus tx compilation (classic) so I had to temporarily remove it (see last commit: 476b2f4)

Could it be due to missing target-version plutus-tx plugin option? Could you try importing common plutus-ghc-options in your test suite?

@TotallyNotChase

This PR was originally supposed to come with an extra test suite. However, we are having some issues with plutus tx compilation (classic) so I had to temporarily remove it (see last commit: 476b2f4)

Could it be due to missing target-version plutus-tx plugin option? Could you try importing common plutus-ghc-options in your test suite?

Doesn't seem to fix it unfortunately. Here's the actual error:
suite/OnChain/BetRef.hs:41:1: error: [GHC-83865]
    • Couldn't match type: forall fun2.
                           PlutusTx.Lift.Class.RTCompile
                             plutus-core-<IP_ADDRESS>:PlutusCore.Default.Universe.DefaultUni
                             fun2
                             (plutus-core-<IP_ADDRESS>:plutus-ir:PlutusIR.Core.Type.Term
                                PlutusCore.Name.TyName
                                PlutusCore.Name.Name
                                plutus-core-<IP_ADDRESS>:PlutusCore.Default.Universe.DefaultUni
                                fun2
                                ())
                     with: plutus-core-<IP_ADDRESS>:plutus-ir:PlutusIR.Compiler.Definitions.DefT
                             template-haskell-<IP_ADDRESS>:Language.Haskell.TH.Syntax.Name
                             plutus-core-<IP_ADDRESS>:PlutusCore.Default.Universe.DefaultUni
                             fun1
                             ()
                             PlutusCore.Quote.Quote
                             (plutus-core-<IP_ADDRESS>:plutus-ir:PlutusIR.Core.Type.Term
                                PlutusCore.Name.TyName
                                PlutusCore.Name.Name
                                plutus-core-<IP_ADDRESS>:PlutusCore.Default.Universe.DefaultUni
                                fun1
                                ())
      Expected: plutus-tx-<IP_ADDRESS>:PlutusTx.Lift.TH.CompileTerm
                -> plutus-core-<IP_ADDRESS>:plutus-ir:PlutusIR.Compiler.Definitions.DefT
                     template-haskell-<IP_ADDRESS>:Language.Haskell.TH.Syntax.Name
                     plutus-core-<IP_ADDRESS>:PlutusCore.Default.Universe.DefaultUni
                     fun1
                     ()
                     PlutusCore.Quote.Quote
                     (plutus-core-<IP_ADDRESS>:plutus-ir:PlutusIR.Core.Type.Term
                        PlutusCore.Name.TyName
                        PlutusCore.Name.Name
                        plutus-core-<IP_ADDRESS>:PlutusCore.Default.Universe.DefaultUni
                        fun1
                        ())
        Actual: plutus-tx-<IP_ADDRESS>:PlutusTx.Lift.TH.CompileTerm
                -> forall fun.
                   PlutusTx.Lift.Class.RTCompile
                     plutus-core-<IP_ADDRESS>:PlutusCore.Default.Universe.DefaultUni
                     fun
                     (plutus-core-<IP_ADDRESS>:plutus-ir:PlutusIR.Core.Type.Term
                        PlutusCore.Name.TyName
                        PlutusCore.Name.Name
                        plutus-core-<IP_ADDRESS>:PlutusCore.Default.Universe.DefaultUni
                        fun
                        ())
    • In the first argument of ‘(Data.Functor.<$>)’, namely
        ‘plutus-tx-<IP_ADDRESS>:PlutusTx.Lift.TH.unCompileTerm’
      In the first argument of ‘Data.Traversable.sequence’, namely
        ‘(plutus-tx-<IP_ADDRESS>:PlutusTx.Lift.TH.unCompileTerm
            Data.Functor.<$> liftExprs'_a7NH)’
      In a stmt of a 'do' block:
        lifts_a7NL :: [plutus-core-<IP_ADDRESS>:plutus-ir:PlutusIR.Core.Type.Term PlutusCore.Name.TyName PlutusCore.Name.Name plutus-core-<IP_ADDRESS>:PlutusCore.Default.Universe.DefaultUni fun ()] <- Data.Traversable.sequence
                                                                                (plutus-tx-<IP_ADDRESS>:PlutusTx.Lift.TH.unCompileTerm
                                                                                Data.Functor.<$>
                                                                                liftExprs'_a7NH)
    • Relevant bindings include
        constr_a7NK :: plutus-core-<IP_ADDRESS>:plutus-ir:PlutusIR.Core.Type.Term
                         PlutusCore.Name.TyName
                         PlutusCore.Name.Name
                         plutus-core-<IP_ADDRESS>:PlutusCore.Default.Universe.DefaultUni
                         fun1
                         ()
          (bound at suite/OnChain/BetRef.hs:41:1)
        constrs_a7NJ :: [plutus-core-<IP_ADDRESS>:plutus-ir:PlutusIR.Core.Type.Term
                           PlutusCore.Name.TyName
                           PlutusCore.Name.Name
                           plutus-core-<IP_ADDRESS>:PlutusCore.Default.Universe.DefaultUni
                           fun1
                           ()]
          (bound at suite/OnChain/BetRef.hs:41:1)
        maybeConstructors_a7NI :: Maybe
                                    [plutus-core-<IP_ADDRESS>:plutus-ir:PlutusIR.Core.Type.Term
                                       PlutusCore.Name.TyName
                                       PlutusCore.Name.Name
                                       plutus-core-<IP_ADDRESS>:PlutusCore.Default.Universe.DefaultUni
                                       fun1
                                       ()]
          (bound at suite/OnChain/BetRef.hs:41:1)
   |
41 | PlutusTx.makeLift ''BetRefParams


Move binderhub_config.py to /etc/binderhub/config/binderhub_config.py

As per this comment: https://github.com/jupyterhub/binderhub/blob/master/helm-chart/images/binderhub/Dockerfile#L42-L43
Now requires the user to mount the binderhub_config.py to /etc/binderhub/config/binderhub_config.py, making things more flexible especially on Kubernetes.
I have two open questions though:

should we leave a "default" binderhub_config.py file "hardcoded" in the image on the new path, which will then be overridden? To enable users without the configfile at hand to run it outside of Kubernetes.
should we change the CMD in the Dockerfile (like done here) or rather change the command + args in the Kubernetes manifest?

should we leave a "default" binderhub_config.py file "hardcoded" in the image on the new path, which will then be overridden? To enable users without the configfile at hand to run it outside of Kubernetes.

I'd say we should aim for a hard failure, and let users mount it with docker or similarly if they run it locally. I think a hard failure is better than confusion when there is a small difference.

should we change the CMD in the Dockerfile (like done here) or rather change the command + args in the Kubernetes manifest?

I think both.


Add flags to the heron executor.

Add executor flags to variables so they no longer have to have an order. Also, this makes it easier to debug and reason about the values.
This cleans up the previous pull request https://github.com/twitter/heron/pull/240

@objmagic @huijunw - any comments?

our ci is dead. @objmagic is trying to fix it. we plan to test this pr after we fix the ci.

our ci pass

sorry i broke this branch when resolving the conflict... will try to restore it.


You shouldn't use deprecated bluez utilities

Hi,
why are you using deprecated bluez utilities https://git.kernel.org/pub/scm/bluetooth/bluez.git/commit/?id=b1eb2c4cd057624312e0412f6c4be000f7fc3617?
I suggest to use dbus api directly instead of hcitool and btmon/btsnoop instead of hcidump.
This is a problem if you use home-assistant on Arch Linux

Yes, we are aware of the situation. Initially, the component code was written that way, and as users (non-programmers) who picked up the idea of the component, our first task was to understand the principle, debug it, and bring it to a stable usable state. More recently, it was a crude non-working gist. After the release of 0.4.0 with support for all (as far as I know) Xiaomi BLE sensors, I just planned to tackle the issue of obtaining data in a normal way.


Kotlin fixes

Codecov Report
Base: 77.11% // Head: 77.10% // Decreases project coverage by -0.01% :warning:

Coverage data is based on head (5ca3c66) compared to base (eacc0df).
Patch has no changes to coverable lines.

Additional details and impacted files
@@                  Coverage Diff                  @@
##             feature/plugins     #228      +/-   ##
=====================================================
- Coverage              77.11%   77.10%   -0.02%
+ Complexity              8050     8048       -2
=====================================================
  Files                    232      232
  Lines                  25874    25874
  Branches                6668     6668
=====================================================
- Hits                   19954    19951       -3
- Misses                  3926     3927       +1
- Partials                1994     1996       +2

Impacted Files
Coverage Δ

...decompiler/deobfuscator/ExceptionDeobfuscator.java
83.47% <0.00%> (-0.43%)
:arrow_down:

...decompiler/modules/decompiler/stats/Statement.java
80.93% <0.00%> (-0.41%)
:arrow_down:

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.


lv_obj_get_screen endless loop

The following code triggers an endless loop in lv_obj_get_screen:
lv_obj_t *scr_page = lv_page_create(lv_scr_act(),NULL);

lv_obj_t * cont_systeme[8];
lv_obj_t * label_systeme[8];

for(int i=0; i<8; i++){
	cont_systeme[i] = lv_cont_create(scr_page,NULL);
	label_systeme[i] = lv_label_create(cont_systeme[i],NULL);
}

lv_obj_clean(scr_page);

cont_systeme[0] = lv_cont_create(scr_page,NULL); //triggers the bug

The problem is that the child and parent in lv_page_signal have the same address. lv_obj_get_ext_attr returns the address of the parent and lv_obj_set_parent now has the address of the parent for tmp and ext->scrl. This triggers the loop in lv_obj_get_screen because par is always != NULL.
I'm using the lvgl master from github.

Hi,
Thanks for the clear and simple code example! :+1:
I've tried it but found that the problem is something different. To clean a page you should use lv_page_clean(scr_page); It will clean the children only from the scrollable part of the page.
lv_obj_clean() deletes the scrollable part of the page too but its an essential part of the page and if it's not there LittlevGL crashes.

Thanks for the fast feedback! It works now with lv_page_clean(scr_page); Thanks!


Data race on cursor

Looks like there is a potential data race scenario that could result in the sink writing incorrect cursor to the database.
Hard to recreate but race detecter sees it:
go run -race . inject -e mainnet.eth.streamingfast.io:443 "badger3://$(pwd)/badger_data.db" ../../examples/wasm-query-service/substreams.yaml

==================
WARNING: DATA RACE
Read at 0x00c003bcf600 by goroutine 412:
  github.com/streamingfast/substreams-sink-kv/db.cursorToBytes()
      /Users/shkvo/github/Pinax/StreamingFast/substreams-sink-kv/db/cursor.go:38 +0xbe
  github.com/streamingfast/substreams-sink-kv/db.(*DB).WriteCursor()
      /Users/shkvo/github/Pinax/StreamingFast/substreams-sink-kv/db/cursor.go:30 +0x64
  github.com/streamingfast/substreams-sink-kv/sinker.(*KVSinker).Stop()
      /Users/shkvo/github/Pinax/StreamingFast/substreams-sink-kv/sinker/sinker.go:117 +0x11d
  github.com/streamingfast/substreams-sink-kv/sinker.New.func1()
      /Users/shkvo/github/Pinax/StreamingFast/substreams-sink-kv/sinker/sinker.go:74 +0xa0
  github.com/streamingfast/shutter.(*Shutter).Shutdown()
      /Users/shkvo/go/pkg/mod/github.com/streamingfast/shutter@v1.5.0/shutter.go:111 +0x174
  main.injectRunE.func1()
      /Users/shkvo/github/Pinax/StreamingFast/substreams-sink-kv/cmd/substreams-sink-kv/inject.go:142 +0x87
  github.com/streamingfast/shutter.(*Shutter).Shutdown()
      /Users/shkvo/go/pkg/mod/github.com/streamingfast/shutter@v1.5.0/shutter.go:111 +0x174
  main.injectRunE.func5()
      /Users/shkvo/github/Pinax/StreamingFast/substreams-sink-kv/cmd/substreams-sink-kv/inject.go:180 +0x3d

Previous write at 0x00c003bcf600 by goroutine 88:
  github.com/streamingfast/substreams-sink.NewCursor()
      /Users/shkvo/go/pkg/mod/github.com/streamingfast/substreams-sink@v0.0.0-20230403140748-db3b0d195289/types.go:18 +0x1065
  github.com/streamingfast/substreams-sink.(*Sinker).doRequest()
      /Users/shkvo/go/pkg/mod/github.com/streamingfast/substreams-sink@v0.0.0-20230403140748-db3b0d195289/sinker.go:268 +0xff8
  github.com/streamingfast/substreams-sink.(*Sinker).run()
      /Users/shkvo/go/pkg/mod/github.com/streamingfast/substreams-sink@v0.0.0-20230403140748-db3b0d195289/sinker.go:164 +0xbc4
  github.com/streamingfast/substreams-sink.(*Sinker).Start()
      /Users/shkvo/go/pkg/mod/github.com/streamingfast/substreams-sink@v0.0.0-20230403140748-db3b0d195289/sinker.go:114 +0x266
  github.com/streamingfast/substreams-sink-kv/sinker.(*KVSinker).Run()
      /Users/shkvo/github/Pinax/StreamingFast/substreams-sink-kv/sinker/sinker.go:147 +0x6aa
  github.com/streamingfast/substreams-sink-kv/sinker.(*KVSinker).Start()
      /Users/shkvo/github/Pinax/StreamingFast/substreams-sink-kv/sinker/sinker.go:109 +0x5c4
  main.injectRunE.func2()
      /Users/shkvo/github/Pinax/StreamingFast/substreams-sink-kv/cmd/substreams-sink-kv/inject.go:146 +0x64

This seems to happen at shutdown of the whole stack, so I'll take a closer look.
The code around shutdown has been heavily refactored in v2 bump, I'll run again with race detection to see if it stills report the problem.


README can use some updating

As an "Action Hero" (user of lib actionpack)
I want to know more about "What Actions are for" and what Result objects offer
So I can more immediately know how to leverage the lib

• demonstrate how to .perform an Action
• outline how to produce and what to do with a Result


fetch_all_next_msgs not exposed to WASM bindings

Description
Expose fetch_all_next_msgs to WASM bindings
Motivation
To use such a function from Javascript
Requirements
Expose the functionality following a similar approach than fetch_next_msgs
Open questions (optional)
Are you planning to do it yourself in a pull request?
No

@arnauorriols totally agree that fetch_all_msgs must be used with care (only when you know number of messages is low) but given the amount of time it can take to retrieve all messages it would be more sensible for an application to retrieve them progressively so that the user starts seeing something as soon as possible, augmenting the overall responsiveness


Update Guice 4.1.0 -> 5.0.1

SpongeAPI | Sponge
This allows us to be somewhat future-proofed, but does mean implementations may have to use a fork of Guice in order to remain compatible with our Guava version
There are some drawbacks on the implementation side of this -- see that PR for a rundown of the issues.
This fixes #2305

We depend on Guava 21.0 here too, so I think its best to just specify the Guice fork here too.

While it would be strictly correct to specify the fork here, it would cause a few issues:

Cross-platform plugins that use upstream guice would have duplicate classes in their classpaths, since there's nothing that tells gradle that ca.stellardrift.guice-backport:guice and com.google.inject:guice provide the same API.
We add API in our fork (by reverting a commit that removed deprecated API) since no suitable replacement exists yet. Currently, we can drop that revert if upstream does eventually add the necessary API. If the fork were visible in SpongeAPI, we would have to maintain that change until a breaking API revision.

Because we explicitly exclude Guava from the Guice dependency, the newer version shouldn't pollute plugin classpaths in most cases, so upstream seems to be the least bad choice for API imo.
On Mon, Mar 15, 2021, at 10:14, Jamie Mansfield wrote:

We depend on Guava 21.0 here too, so I think its best to just specify the Guice fork here too.

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/SpongePowered/SpongeAPI/pull/2311#issuecomment-799592719, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAEZSZHQF7GMODLFRD7CXPTTDY55XANCNFSM4ZBQKQNA.

Yes -- it's not at all an ideal solution, but we don't really have any good options. We were forced to upgrade because guice 4.1.0 shaded an ancient version of ASM that was incompatible with any Java version newer than Java 8, and needed workarounds to allow its outdated method of class generation to work. Guice 5 is the earliest version that fixes the class generation issues, but also removes the API we need on the implementation side.
The issues with third-party implementations has been brought up on the implementation PR -- hoping to hear from @Cybermaxke on how suitable this route will be for Lantern.