add 0 in knex version in example package.json

Coverage remained the same at 95.05% when pulling 608c577ed5874560291d065b03e3b5ec77b53adb on hanhsu:example-pkg-json-fix into d7318ea356f868269286a14df5c1378605ebae3e on Vincit:master.


Crash in demo project

Hi there,
I've tried the demo project under the solution but something went wrong.
I loaded two books and clicked the menu, the compiler always  broke in this line
await Parchment.LoadContentAsync(e.AddedItems.FirstOrDefault() as NavPoint);, which is located in file TableOfContents.xaml.cs , line 69, with the exception of System.IO.FileNotFoundException
Having no idea what's going on, please check it out.
Any help are appreciated!

I should be able to take a look at it this evening. Could you link to the books you're trying? ePub is an incredibly versatile format, and I'm guessing there's something different about the specific books you're trying.

Ebook.zip
Here are two books I've tried.
Thanks for your help!

@totoroyyb Thanks for bringing this up! Apparently it's "valid" for an eBook to reference a file that does not exist. I've updated Papyrus to handle that. It looks like there's still an issue where the last line of a chapter is shown on the next chapter's page, but I'm not sure if that's specific to your eBooks or if it's just a general issue, and it's not really related to this issue anyway, so I'm closing this and I'll take a look at the last line duplication separately.

Thanks for checking out, glad to here it's fixed


ERC20 Token Subgraph and Analytics APP

Check in ERC20 token subgraph and Analytics APP for review. Thanks!

Can you use prettier to format the code?

You should put the erc721 subgraph in a separate PR

PR

Done. Create a new PR at https://github.com/messari/subgraphs/pull/689/commits

Awesome job on this!


Deprecate the PagerDuty plugin

Hey, I just made a Pull Request!
According to the discussion in https://github.com/backstage/backstage/issues/19347 we're deprecating the PagerDuty plugin created by Backstage. We are redirecting users instead to https://www.npmjs.com/package/@pagerduty/backstage-plugin.
:heavy_check_mark: Checklist

[x] A changeset describing the change and affected packages. (more info)
[x] Added or updated documentation
[ ] Tests for new functionality and regression tests for bug fixes
[ ] Screenshots attached (for UI changes)
[x] All your commits have a Signed-off-by line in the message. (more info)

Looks like all checks have passed. Can you double-check if I'm doing everything properly @Rugvip @camilaibs? Thanks!

@camilaibs I have added the deprecated annotation in the PagerDuty Entity and the cards being exported. I don't know if you want this to be placed in other locations but I believe these are the components used by developers.


Install fails using Conda anaconda-nb-extensions channels

OS: OSX
Py2.7
`➜  ~  conda install -c anaconda-nb-extensions nbpresent
Fetching package metadata: ......
Solving package specifications: ................................................
Package plan for installation in environment /Users/scollis/anaconda:
The following NEW packages will be INSTALLED:
nbpresent: 1.0.0-py27_0

Proceed ([y]/n)? y
Linking packages ...
copying /Users/scollis/anaconda/lib/python2.7/site-packages/nbpresent-1.0.0-py2.7.egg/nbpresent/static/nbpresent -> /Users/scollis/anaconda/share/jupyter/nbextensions/nbpresent
Traceback (most recent call last):
File "/Users/scollis/anaconda/lib/python2.7/runpy.py", line 162, in _run_module_as_main
"main", fname, loader, pkg_name)
File "/Users/scollis/anaconda/lib/python2.7/runpy.py", line 72, in _run_code
exec code in run_globals
File "/Users/scollis/anaconda/lib/python2.7/site-packages/nbpresent-1.0.0-py2.7.egg/nbpresent/install.py", line 112, in
install(**parser.parse_args().dict)
File "/Users/scollis/anaconda/lib/python2.7/site-packages/nbpresent-1.0.0-py2.7.egg/nbpresent/install.py", line 39, in install
install_nbextension(directory, **kwargs)
File "/Users/scollis/anaconda/lib/python2.7/site-packages/notebook/nbextensions.py", line 220, in install_nbextension
_maybe_copy(src, full_dest, verbose)
File "/Users/scollis/anaconda/lib/python2.7/site-packages/notebook/nbextensions.py", line 53, in _maybe_copy
shutil.copy2(src, dest)
File "/Users/scollis/anaconda/lib/python2.7/shutil.py", line 130, in copy2
copyfile(src, dst)
File "/Users/scollis/anaconda/lib/python2.7/shutil.py", line 82, in copyfile
with open(src, 'rb') as fsrc:
IOError: [Errno 2] No such file or directory: u'/Users/scollis/anaconda/lib/python2.7/site-packages/nbpresent-1.0.0-py2.7.egg/nbpresent/static/nbpresent'
Error: Error: post-link failed for: nbpresent-1.0.0-py27_0                                                                                |   0%
➜`
Conda env:
`  ~  conda update conda
Fetching package metadata: ....
All requested packages already installed.
packages in environment at /Users/scollis/anaconda:

conda                     3.19.3                   py27_0  `

Note: works when I use a 3.5 env

I was never able to reproduce this particular error (though I found lots of conda issues, or rather, issues with how I was using conda!). However, since I did so much work across all the platforms, I'd like to close this one, and re-open with a new issue.
Thanks again @scollis for the report!


Target Resource should allow setting an RG

for az monitor alert create the --target params should allow you to set a resource in another RG without having to drop to using the ID (which would turn this into a 2-line command for ANY interactive use)

This would break the convention we use everywhere in the CLI that we don't expose dedicated RG arguments for resources other than the main resource. Otherwise you get the Xplat fun times of "foo-rg", "foo-subscription"... etc etc.


Allow starting in regtest mode

Resolves #2313
Next I'd make the oracle configurable. Thatway we can have an offline-dev setup :)
Tested using https://github.com/vulpemventures/nigiri

bors r+

bors r+


Update documentation links

This PR updates some links in the documentation to have full path and correct line number

Codecov Report

Merging #231 (975767e) into master (79fa2b4) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #231   +/-   ##
=======================================
  Coverage   57.07%   57.07%
=======================================
  Files          37       37
  Lines        4007     4007
=======================================
  Hits         2287     2287
  Misses       1596     1596
  Partials      124      124

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


hack(gl-search): fix bash subshell function export

This PR fixes hack/gl-search.sh for osx users.
The issue was that osx bash does not load exported functions into sub shells automatically.
This commit puts the functions in a separate file, which is then sourced on demand.
@fwilhe Thanks for reporting!

Thanks for looking into it, I can confirm that this resolves the issue I had 👍

@fwilhe Thanks! Once this PR gets approved I will merge it


ci: remove lint step with ref to pull_request in release workflow (#219)

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

Please upload report for BASE (release/4.1@74124fb). Learn more about missing BASE report.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@              Coverage Diff               @@
##             release/4.1     #220   +/-   ##
==============================================
  Coverage               ?   29.47%
==============================================
  Files                  ?        1
  Lines                  ?      699
  Branches               ?        0
==============================================
  Hits                   ?      206
  Misses                 ?      488
  Partials               ?        5

Flag
Coverage Δ

integrationtests
29.47% <ø> (?)

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

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


Add counterexample-collecting model-checker listener

Closes #1632
Add a counterexample-collecting model-checker listener.
The plan is to hook this up to SeqModelChecker to obtain counterexamples for #1588.

Codecov Report

Merging #1635 (c5dfdea) into unstable (fd3332a) will increase coverage by 0.01%.
The diff coverage is 75.00%.

@@             Coverage Diff              @@
##           unstable    #1635      +/-   ##
============================================
+ Coverage     74.95%   74.97%   +0.01%
============================================
  Files           357      358       +1
  Lines         11609    11611       +2
  Branches        562      541      -21
============================================
+ Hits           8702     8705       +3
+ Misses         2907     2906       -1

Impacted Files
Coverage Δ

...ache/tla/bmcmt/passes/BoundedCheckerPassImpl.scala
0.00% <0.00%> (ø)

...apalache/tla/bmcmt/search/ModelCheckerParams.scala
77.77% <ø> (ø)

...ollectCounterexamplesSeqModelCheckerListener.scala
100.00% <100.00%> (ø)

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 fd3332a...c5dfdea. Read the comment docs.


improve compute resource console output

Addresses #85

I'm actually going to merge this now and release to pypi because this was accidentally polling the server every 5 seconds! Not sure how that happened, it was supposed to be every 10 minutes. :face_with_spiral_eyes:


Update Coached Conv Pref DatasetCard

Update Coached Conversation Preferance DatasetCard

Really cool!
Can you add some task tags for dialogue-modeling (under sequence-modeling) and parsing (under structured-prediction)?

Really cool!
Can you add some task tags for dialogue-modeling (under sequence-modeling) and parsing (under structured-prediction)?


Dispatcher doesn't know if resuming or starting fresh

Currently, Dispatcher.dispatch is invoked on every onResume of the Activity.
If the onResume does not follow an onDestroy and the previous view is still in place (e.g. after opening the app switcher and returning), a naive Dispatcher will recreate the view, losing its state.
Something like this happens e.g. in flow-sample-basic:

Open app
Type some characters
Open app switcher and close again
KeyChanger has created a new view and added it on top of the old one, both views are visible. The new view lost the text in the EditText.

 (Click to enlarge)
This does not occur after an orientation change, as the view is destroyed before dispatch is called.
Should dispatch always be called after onResume? Is it the responsibility of the Dispatcher (or KeyChanger when using KeyDispatcher) to detect whether an action is required or if the traversal can be ignored?

It's strange, but you are right. Even if you're on the second view (second key), then if you put the app to background and bring it back, a key change is executed:
MainKeyChanger﹕ Change Key: [null] - [com.zhuinden.flow_alpha_project.OtherKey@52a0dba4]

I'm not sure this is expected behavior.

previously when we had full control of dispatching I always omitted calls to FlowConfig.onPause() and FlowConfig.onResume()
now since everything is done implicitly through Fragments, I don't have that chance anymore
can you please consider bringing back all the explicit lifecycle method calls?

Should InternalLifecycleIntegration onResume() method look like this?
@Override public void onResume() {
    super.onResume();
    if (!dispatcherSet) {
      flow.setDispatcher(dispatcher,true);
      dispatcherSet = true;
    }
  }

When I minimalize application and onResume is called, KeyManager is incrementing unnecessary Key object counter. This breaks services lifecycle.

Could someone confirm that the fix just merged from https://github.com/square/flow/pull/190 covers this as well?

This strikes me as a dup of #171, where I just argued that this behavior is by design.

Yes, but if you put the app in background and bring it foreground, you can't short-circuit based on previous/new key because previous key is empty because it is a bootstrap traversal

It's not about the previous key in the transiiton. It's about the key
embedded in the view that is currently showing.
Something like if (traversal.destination.top().equals(Flow.getKey(containerFrame.getChildAt(0)) return;
On Wed, Apr 12, 2017, 11:51 PM Gabor Varadi<EMAIL_ADDRESS>wrote:

Yes, but if you put the app in background and bring it foreground, you
can't short-circuit based on previous/new key because previous key is empty
because it is a bootstrap traversal
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/square/flow/issues/173#issuecomment-293806316, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ABzBHeodN3oqOtSx5erbp5oNmrrrz5pjks5rvcXrgaJpZM4HpV10
.

Ew.

Seems like pretty traditional UI de-bouncing to me. Obviously that phrase
wouldn't appear literally.
On Thu, Apr 13, 2017, 8:39 AM Gabor Varadi<EMAIL_ADDRESS>wrote:

Ew.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/square/flow/issues/173#issuecomment-293932430, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ABzBHXhqOdkc_qUI2Csmcg1xF77lLoSoks5rvkHIgaJpZM4HpV10
.

Controller: "Hey UI, show this."
UI: "Already am, thanks."

Thinking about this some more, I don't see what else is possible. Flow cannot, should not make assumptions about what state your dispatcher is in. The problem here is in thinking of the call to dispatch as being an event. It isn't, it's a notification of the state that Flow expects you to be rendering.
Keeping this issue open to address in Alpha 3, but as a documentation task.

FYI for others who also ran into this issue, don't forget to call TraversalCallback#onTraversalCompleted() when you ignore a key change otherwise the back-history will not get updated. We ran into really strange problems where further key changes were just getting ignored because of this.
So the final code would look somewhat like this:
Object existingKey = Flow.getKey(frame.getChildAt(0));
Object incomingKey = incomingState.getKey();

if (outgoingState == null && existingKey.equals(incomingKey) && direction == Direction.REPLACE) {
  // https://github.com/square/flow/issues/173
  callback.onTraversalCompleted();
  return;
}


NY move to 16+

[{"key":"new_york","type":"vaccination","value":{"content":{"phases":[{"id":"current","qualifications":[{"question":"c19.eligibility.question/age.16_up","moreInfoText":"c19.eligibility.moreinfo/age.are_you_age_30.ny.current","moreInfoUrl":"https://www1.nyc.gov/site/doh/covid/covid-19-vaccine-eligibility.page"}]}]}}},{"key":"new_york","type":"strings","value":{"content":{"c19.eligibility.moreinfo/age.are_you_age_30.ny.current":{"en-us":"Note: People who are 16 and 17 years old are only eligible for the Pfizer vaccine.\n\nThe COVID-19 vaccine is currently available to all New York State residents 16 and older. People who live outside of the state but work in New York are also eligible to get vaccinated if they belong to one of the groups at the link. These groups that are eligible based on employment include both paid and unpaid workers.\n\n"}}}}]


Enhancement: Generate multiple resume from single repository

Trying to generate multiple resumes through a single repository where a user who wishes to generate their resume can view it by adding their name as a path.
For example - https://demo-resume.netlify.app/abc returns the resume of the person whose name is abc.
What type of changes to be done in order to obtain the expected result?

this would require a rewrite to change the structure of the input files used.
currently, the page that actually generates the index.html output is the introduction.md. there is data from multiple other sources, like the data and entries directories.
off the top of my head, all of that data would have to be included in a user-specific folder that generates individual pages based on a slug.
at the moment, that is beyond the scope of this project though.

Sorry I didn't get the notification of your comment.
So, basically I'm trying to develop this as my school project where my classmates can upload their resume in a single repo, which would be very much helpful for our placements.
I was able to split the project into individual pages by adding the introduction.md in a separate folder with the entries in it. The problem is all the entries data in the project gets displayed instead of specific individuals entries.
It would be very much helpful if you can guide me through the process.

@Harish2997 sorry this is out of scope for the project - you're free to fork the repo and make your own multi-user version.


Dependency error on github.com/kr/text

I have been publishing changes to my hugo site via CodeShip and it is now failing when running go get -u -v github.com/spf13/hugo It seems to be failing when loading github.com/kr/text which doesn't appear to have a master branch. Is this because that repo had a master branch and now it doesn't, or is this specific to the way the dependency is defined in hugo? Any thoughts for a workaround or what the solution for this would be?  Built fine. The missing master branch isn't something new, so that must have been working. I suggest you check your Go build chain. Closing this as this isn't related to Hugo, per se. Use the discussion forum for further questions.


OpenTofu support

Is your feature request related to a problem? Please describe.
I don't have terraform installed. Instead I use OpenTofu. However, when running Kuzco without terraform installed, the following error occurs:
Error: error extracting provider schema: error running terraform init: exec: "terraform": executable file not found in $PATH

Describe the solution you'd like
Kuzco works fine with just OpenTofu installed
Describe alternatives you've considered

alias terraform=tofu does not work

Thank you for this issue! I'm adding it to the roadmap and will add support this week for this request

OpenTofu configurations are now supported! Enjoy!


bug: Can't open party overview on old pf2e worlds on module versions greater than 2.6

See https://discord.com/channels/170995199584108546/697845559435853865/934429580780007444
Copied image of error one user gets below:

On my forge world I don't get an error, but party overview won't open until I reverted to version 2.6
So, I'm guessing it's related to the new show skills feature for pf2e.
Familiars, loot actors and player-owned npcs need special handling in
https://github.com/mclemente/party-overview/blob/8b97d0473c26e5c93c77d57502710da82cfb8a13/module/SystemProvider.js#L595
To get at familiar skills, follow data.master.id and use their skills. (Or, maybe, just don't show skills for familiars).
Loot actors can safely be shown as having no skills.
Not sure what the issue with player-owned npcs is. The one in question has no skills, so maybe it's that?

Should be fixed now.

Works fine, thanks!


[$1000] Assign task- 404 error in the console appears when create a task

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Action Performed:

Go to URL https://staging.new.expensify.com/
Login
Go to any #announce
Create a task

Expected Result:
No errors in console
Actual Result:
404 error in the console appears
Workaround:
unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[ ] Android / native
[ ] Android / Chrome
[ ] iOS / native
[ ] iOS / Safari
[X] MacOS / Chrome / Safari
[ ] MacOS / Desktop

Version Number: v1.3.15-4
Reproducible in staging?: Y
Reproducible in production?: Y
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation

https://github.com/Expensify/App/assets/44479856/5fb0d74a-b670-4f99-ae2b-4233ac98669d
Expensify/Expensify Issue URL:
Issue reported by: Applause - Internal Team
Slack conversation:
View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~012e1b763b4e7b70a1
Upwork Job ID:<PHONE_NUMBER>624666112
Last Price Increase: 2023-05-19

Yep, reproduced!

@thienlnam I think this should be over to you! I'll label it Internal to get the ball rolling, and will also then assign you too

If I'm wrong, let me know!

The error is with AuthenticatePusher, not sure why that is showing up. The EditTask command should now be on production and so that error shouldn't be happening anymore

I think we can have this external for someone to investigate

Proposal
Please re-state the problem that we are trying to solve in this issue.
Assign task- 404 error in the console appears when create a task
What is the root cause of that problem?
404 error is come from this API to authorize Pusher when our App need to subscribe to a socket channel.

I suspect the API to create Task (+ report) is not finished yet, then we navigate to taskReport in this line
The ReportScreen will load and it triggers this line to subscribe to private socket channels of the report
https://github.com/Expensify/App/blob/737862677811acc2b05c68209a56ecd58bee59f3/src/pages/home/report/ReportActionsView.js#L230-L233
=> Can cause 404 error because the create Task API is not finished yet (report is not created yet)

Screenshots when navigate to a newly chatReport and a  newly taskReport

newly taskReport

newly chatReport

What changes do you think we should make in order to solve the problem?
To solve this problem we need to ensure the taskReport is created successfully before subscribe to its private channels
More details:
We need to follow this condition:
https://github.com/Expensify/App/blob/737862677811acc2b05c68209a56ecd58bee59f3/src/pages/home/report/ReportActionsView.js#L225-L229
That means when build optimisticTaskReport data, we need to add pendingFields.createChat with  CONST.RED_BRICK_ROAD_PENDING_ACTION.ADD (and isOptimisticReport: true so it won't trigger OpenReport API when it's not ready).
                pendingFields: {
                    createChat: CONST.RED_BRICK_ROAD_PENDING_ACTION.ADD,
                },
                isOptimisticReport: true,

And reset those fields in successData.
We used this trick in some other places:

IOU chat
ExpenseChats
Normal report

Result:

Unassigning @laurenreidexpensify as I'm already assigned as BugZero

Asked in Slack

Awesome - looks great thanks!

PR will ready in a couple hour

@aimane-chnaif @thienlnam  PR is ready to review

Checking that in our internal Slack channel

look fine now. Thanks for supporting

PR reviewed and on Staging

Argh, Upwork isn't letting me hire on the existing job. I'll have to create new ones - tomorrow!

Looks like this was merged in less than 3 business days, so adding a 50% urgency bonus for C and C+

@aimane-chnaif contract sent via Upwork
@phuchoang23 Can you reply here with your Upwork profile URL please?

I will shortly send the contract, yes!

@phuchoang23 Paid!

Alright, just the BugZero checklist here

@conorpendergrast Thanks 👍

@aimane-chnaif Bump on the BugZero checklist here please!

Got it. Sounds like we're all done. Closing, thanks all!


createTerminal does not set the code point of the windows process to utf8, even though it expects the terminal to write utf8

Does this issue occur when all extensions are disabled?: No (but I did disable all the other ones)

VS Code Version: 1.83.1
OS Version:  Windows 11

Steps to Reproduce:

make a batch file (saved as utf8). (please note that I just made it a batch file for easier reproduction, in reality it's a full terminal).

echo off

echo "Code page before: "
chcp

echo "Failing unicode: 🚀"

echo "Now chaching codepage to UTF8"
chcp 65001
echo "Working unicode: 🚀"
timeout 25

add a command that executes the following code in an extension:

vscode.window.createTerminal({
    name: "Test",
    shellPath: "c:\\users\\davy\\test.bat"
});

see if fail to properly print utf8 stuff:

As I understand it:

VS Code & xterm on Windows seem to fall back to expecting utf8 (it even sets up LANG: en_US.UTF-8)
windows processes by default are run in the codepage of the user configuration, which can be CP437 or CP1252.
a process that is writing to it's output stream should respect the code page it's started with.
therefore it starts outputting bytes in it's code page.
once we manually switch to the expected codepage by vs code, suddenly utf8 bytes are passed along as utf8 bytes, and the terminal is happy to display emojis.

The actual bigger case is a java program that uses jline to make a REPL. As java picks up the code page from the windows kernel call, it sets up System.out with CP437  encoding.  Now stuff gets messy.
My proposal would be at the very least to have an option for createTerminal to change the code page (instead of only setting en_US.UTF-8). And I'm not sure, but doing this by default might also change some of the encoding issues I see around emoji's in windows terminals.
[edit]
I've been doing some further reading, these discussions might be related:

https://github.com/microsoft/terminal/issues/1802
https://github.com/microsoft/terminal/issues/15678

This issue seems to originate from an upstream component and is not something VSCode can fix by making a change to our code. As such, I will go ahead and close it. Please try to reproduce in Windows Terminal and report to microsoft/terminal with repro steps.
If you cannot reproduce in Windows Terminal, chances are it's fixed in a later version of Windows. You can read more about how we consume conpty here. Additionally, soon we plan on shipping conpty v2 with VS Code which should bring our behavior in line with Windows Terminal. Thanks for understanding.
/label upstream terminal-conpty windows

Hi @Tyriar we actually ran into this for a custom terminal we created in VS Code (so just starting process and hooking it directly to the input and output streams)
I only used basic conhost/cmd in this example as a small test case. I linked terminal issues to show they had a similar problem


[OTHER] state doesn't properly works

What would you like to share or ask?
Hi,
Seems like there is an issue using State variable.  When I am using Taipy state variable the UI, the variable get updated but the UI get stack.
The UI looks like its loading for infint time and doesn't response anymore (seems like its 'bussy').
What am I doing wrong? am I not using correctly the State variable? I need help to fix it - else I am giving up =[
UI.py:
project_number   = ''

main_page = """ ...
<|card
Project Number
 <|{project_number}|selector|width="180"|lov={project_numbers}|label=project numbers|dropdown|>
         <|Apply|button|on_action=update_master|>

|>
|>
"""

utlity.py:
def update_master(state):
...
project_num = state.project_number
...

notify(state, "success", "[INFO] Master file has been updated")
return

But the function get stack when the notify comes and than the UI is not responding anymore.
Code of Conduct

[X] I have checked the existing issues.
[ ] I am willing to work on this issue (optional)

Could you a code from start to finish in a single file with your issue?
You can look into our Tutorials to understand how Taipy works.

Tal, we have exchanged for months on Discord and by email to help you with your support requests. If you need more help with this specific issue, please get in touch with me by email.


Plugin no longer works?

Have used this plugin for a long time, in the past few weeks it has stopped working completely.

are you on transparent mode? likely duplicate of  #19 - I have a PR to fix it but need @robrichardson13 to review and approve #20

are you on transparent mode? that's a known issue, established in #19. I have a PR #20 to fix it, but I need @robrichardson13's approval to merge in, then will need RL team to review the subsequent plugin-hub PR before it can be fixed.
in the meantime, using "hidden" mode should work.

if you're on transparent mode, I'll consider this a duplicate of #19 and close it. if you're referring to a different issue, feel free to reopen this thread.
fix will go out with version 1.5 once I make the plugin-hub PR and runelite maintainers review it

Mine is not working on either setting currently, have uninstalled and reinstalled as well. I have not changed any logic things because I have no idea how lol. Hopefully all is fixed with 1.5

What's happening with it then? Is it not filtering anything at all? Do you have the plugin enabled? lol
curious what your config is like, but "not working at all" is not something I've been able to reproduce
(screenshots or w/e would be helpful I suppose)

It then still alchs it. I do not have instant inventory plugin, but I do have a lot of plugins. Again this worked great for a long time before

Given that I can't reproduce your issue, there's probably another plugin that it's conflicting with. This is definitely different from the issue I just fixed. What I was seeing is that objects would be filtered, but they wouldn't prevent alching. In your case, though, it's not even visually filtering the objects at all.
I'd suggest the following steps:

Create a plugin profile with your current configuration (second category in the plugin tab, between plugin list and plugin hub)
Create a new profile for testing so you can easily revert to your preferred configuration
Disable every other plugin you can and see if it fixes it

That should help you confirm that it is a plugin conflict of some kind. Then you'll just have to go through the process of turning things on and off until you figure out what the issue is. Because at this point, it really seems like it has to be a plugin conflict. But we should determine that as concretely as we can.

THANK YOU! Remaining Casts Plugin

Checking 'show on menu tooltip' is what seems to break it
Thank you!

Fascinating - I'll try to take a look at that plugin when I get a chance, can update the README in the meantime to make sure other folks don't hit the same issue. Glad you figured it out! :)

Thanks! Ive been having this issue since OP posted this as well. Gave up on it till I decided to revisit the thread to see if there was a solution- this did the trick!


Use image instead of title text

I have a requirement where we need to show a logo instead of the title on the first page of the client site.
Is there a way to do this?

Already asked and answered


Feature request: image/vnd.microsoft.icon support

[x] I've read, understood, and done my best to follow the *CONTRIBUTING guidelines.

What did you do?
Pulled favicon images from various sources.
What did you expect to happen?
All of them loaded correctly
What happened instead?
Only icons that didn't follow the IANA standard loaded
Desctription
As this stackoverflow post indicates, there's not really a good standard for favicon mime types. AlamofireImage however only supports 1 of the 2 options that's regularly used, making a lot of favicons fail to load.
The main purpose of this issue is to see if there's any use proposing a pull request that adds support for the image/vnd.microsoft.icon type.

Given that the "standard" isn't actually a standard (Microsoft didn't register it), and it's easy to add such a type to the accepted types, as outlined in our docs, we won't be adding support for this type.


Make OTLP receiver enabled by default

This simplifies compatibility with OTEL/OTLP by not requiring to set the option to enable OTLP receiver. The option is still supported and allows OTLP receiver to be disabled when needed.

dont ask me, its update all posted


Test cases

This definitely is a fascinating project. The docker ceo apparently said that if WASM existed there would have been no need for docker. He was referring to sand boxing.
Did you hit many bugs when attempting compilation using tinygo. I think they did a release recently or it might still be on a branch . Tinygo is efficient enough that you could run on mobile or have a good chance.
I have a interesting idea. Your sage accounting software uses a bucket file system dB and a git based vcs to presumerable allow sync of the underlying files . Would be interesting test case if these two work running on top of the wasm indexdb / FS. It’s a decent test case I feel.
Would badger dB in theory work ?

Thanks for checking it out @gedw99! 😄

Thanks for your thoughts @gedw99 🙂 Since I haven't heard back in a bit, I'll close for now. Feel free to re-open of course!

https://github.com/genjidb/genji.js


Rename networkID property in TransactionMeta to chainId

TransactionMeta.networkID should be renamed to chainId.

I believe this has been resolved by #349
The networkID property remains, but it now actually refers to the network ID. Both IDs were kept around to maintain backwards compatibility (at least temporarily).


feat: hotm changes

You should add descriptions to these to indicate your progress 👍

You should add descriptions to these to indicate your progress 👍

Sorry saw your comment too late, im done 🧌


Add universal links URL

What
Adds click.em.kickstarter.com as a URL that we support for universal links.
Why
Some of our newsletter emails use links from click.em.kickstarter.com.
How
Added to Debug and Release entitlements.

 Bug: System Email links deep-link to app while Newsletter links go to mobile web


Rapid test integration - documentation

From comment of @raceface2nd: https://github.com/corona-warn-app/cwa-quick-test-frontend/issues/51#issue-854736431
What is missing

Are there any documents (sketches, descriptions, charts or whatever) you can provide that we might be able to prepare the ability on our technical side?
Why should it be included

To be able to provide this additional benefit feature to our company customers I am very interested on getting information on how the technical process is planned to deliver the test results to the CWA.
Can you also estimate when this feature will be available? We have some festivals and concerts which will be held somehow in the next months hopefully as demonstrator events in conjunction with quick testing. The event organizers would love to use the CWA as trusted instance instead of using different apps for ticketing, test results and pooled contact tracing. The first event which will take place is the "Bundesgartenschau" which is held every 2 years and which open doors in about 2 weeks. We would like to support the organizers of the Bundesgartenschau in making the event as save as possible by using CWA together with quick tests.
Where should it be included

Internal Tracking ID: EXPOSUREAPP-6382

See also https://github.com/corona-warn-app/cwa-documentation/issues/595.

Closing this issue as the related internal ticket is set to obsolete.
Developer comment:

The technical information how to integrate with the CWA is provided by TSI. TSI provides information to partners who want to integrate their RAT solutions


add list, run cli flags to integration tests.

fixes: #338

Lint check fails.

running cargo clippy --workspace --exclude integration-test -- --deny warnings on my system doesn't give any warnings.

That's because CI is running with Rust nightly. You are going to see those errors when you do:
cargo +nightly clippy --workspace --exclude integration-test -- --deny warnings

nothing:
(rust:nightly) ↪ cargo +nightly clippy --workspace --exclude integration-test -- --deny warnings
   Compiling aya-bpf-cty v0.2.1 (/home/abhi/code/aya/bpf/aya-bpf-cty)
   Compiling aya-bpf-bindings v0.1.0 (/home/abhi/code/aya/bpf/aya-bpf-bindings)
   Compiling aya-bpf v0.1.0 (/home/abhi/code/aya/bpf/aya-bpf)
   Compiling aya-bpf-macros v0.1.0 (/home/abhi/code/aya/aya-bpf-macros)
   Compiling aya v0.11.0 (/home/abhi/code/aya/aya)
    Checking aya-tool v0.1.0 (/home/abhi/code/aya/aya-tool)
    Checking xtask v0.1.0 (/home/abhi/code/aya/xtask)
    Checking integration-test-macros v0.1.0 (/home/abhi/code/aya/test/integration-test-macros)
    Checking integration-ebpf v0.1.0 (/home/abhi/code/aya/test/integration-ebpf)
    Checking aya-log-common v0.1.12-dev.0 (/home/abhi/code/aya/aya-log-common)
    Checking aya-log v0.1.12-dev.0 (/home/abhi/code/aya/aya-log)
    Checking aya-log-parser v0.1.11-dev.0 (/home/abhi/code/aya/aya-log-parser)
    Checking aya-log-ebpf-macros v0.1.0 (/home/abhi/code/aya/aya-log-ebpf-macros)
    Checking aya-log-ebpf v0.1.0 (/home/abhi/code/aya/bpf/aya-log-ebpf)
    Finished dev [unoptimized + debuginfo] target(s) in 6.31s
abhi at hyperion in ~/c/aya on integration-tests-cli-options [✓]
(rust:nightly) ↪ rustc --version
rustc 1.66.0-nightly (f83e0266c 2022-10-03)

Try rustup update, there is a newer version of nightly available (you have f83e0266c 2022-10-03, the newest one is 3cdf1ab25 2022-10-07).

ok, i get those warnings now. should i fix them all in the same PR?

Maybe do a separate one, since those warnings are not related to your changes and they appear on the current main branch too.

I merged the lint PR. You can rebase this one and I will merge it once CI passes.


It's set include_images=True, but there is no picture

That's my code：
`from trafilatura import fetch_url, extract
url = 'https://shumeipai.nxez.com/2020/06/11/stanford-pupper-assembly-tutorial.html'
downloaded = fetch_url(url)
result = extract(downloaded, output_format='markdown', favor_recall=True, include_images=True, include_links=True)`

I can indeed reproduce the bug. Images are not my priority, the corresponding code mostly consists of a series of contributions and it's not perfect. Let's see if someone can improve on this.我确实可以重现这个错误。图片不是我的重点，相应的代码主要由一系列贡献组成，而且并不完美。让我们看看是否有人能对此加以改进。

Thank you very much, I found that most of the sites can't get pictures in the process, and this is just one of the cases

That's my code： `from trafilatura import fetch_url, extract
url = 'https://shumeipai.nxez.com/2020/06/11/stanford-pupper-assembly-tutorial.html' downloaded = fetch_url(url) result = extract(downloaded, output_format='markdown', favor_recall=True, include_images=True, include_links=True)`

Try it:
`from trafilatura import fetch_url, extract
url = 'https://shumeipai.nxez.com/2020/06/11/stanford-pupper-assembly-tutorial.html'
downloaded = fetch_url(url)
img_src_regex = r'<img[^>]+src="([^"]+)"[^>]*>'
def replace_img_tags(match):
src = match.group(1)
return f'111222333000-{src}-000333222111'
downloaded = re.sub(img_src_regex, replace_img_tags, downloaded)
result = extract(downloaded, output_format='markdown', favor_recall=True, include_images=True, include_links=True)
result = re.sub("111222333000-","<img src="", result)
result = re.sub("-000333222111","">", result)
print(result)`

For further reference: see also https://github.com/adbar/trafilatura/issues/662.


Unhandled exception in script

Every time I try to run this program it just shows this. Everything was working fine until yesterday.

Looks like your bot is having trouble connecting to the internet maybe?

It is trying to get the latest version of the bot from this url:
https://api.github.com/repos/iholston/lol-bot/releases/latest
and it looks like it should be able to get it if you have no internet issues


v0.2.0

@JuliaRegistrator register

@JuliaRegistrator register

Error while trying to register: "Tag with name v0.2.0 already exists and points to a different commit"


API calls for AWS is not being mocked even though moto is being used

I have referred #1926, #1793 and #2118 before opening this issue.

I am getting the error for ecr:
botocore.exceptions.ClientError: An error occurred (UnrecognizedClientException) when calling the DescribeRepositories operation: The security token included in the request is invalid.

and, I am also getting:
/home/x/proj/lib/python3.6/site-packages/requests/__init__.py:80: RequestsDependencyWarning: urllib3 (1.23) or chardet (3.0.4) doesn't match a supported version!
  RequestsDependencyWarning)
Performing system checks...

I am getting this error only when I add moto to requirements.txt
If I run pip install moto after running pip install -r requirements.txt, I am not getting both of the errors.
boto3==1.9.88
botocore==1.12.0
moto==1.3.8
It is accessing aws apis are when I disconnected, I got this:
Exception Value: Could not connect to the endpoint URL: "https://ecr.us-east-1.amazonaws.com/"

Code:
@if_non_prod_aws_mocker(mock_ecr)
@if_non_prod_aws_mocker(mock_sts)
def get_aws_credentials_for_submission(challenge, participant_team):
    repository, created = get_or_create_ecr_repository(
        ecr_repository_name, aws_keys
    )
    # getting error on line above
    #other codes

get_or_create_ecr_repository():
def get_or_create_ecr_repository(name, aws_keys):
    repository, created = None, False
    client = get_boto3_client("ecr", aws_keys)
    try:
        response = client.describe_repositories(
            registryId=aws_keys.get("AWS_ACCOUNT_ID"), repositoryNames=[name]  # Error is pointing at this line
        )
        repository = response["repositories"][0]
    except ClientError as e:
        if e.response["Error"]["Code"] == "RepositoryNotFoundException" or\
           e.response["Error"]["Code"] == "400":
            response = client.create_repository(repositoryName=name)
            repository = response["repository"]
            created = True
        else:
            logger.exception(e)
    return (repository, created)

@if_non_prod_aws_mocker:
def if_non_prod_aws_mocker(mocker):
    def decorator(func):
        if not (settings.DEBUG or settings.TEST):
            return func
        return mocker(func)
    return decorator

Updated botocore and moto using:
botocore==1.12.85
git+https://github.com/spulec/moto.git@df493ea18de22b4533073eee3e296686019911c2 # installing via commit till moto>1.3.8


make caffe converter tool available without caffe installation

according to https://github.com/kitofans/caffe-theano-conversion, from #597

Hi. thanks for the PR. Could you please update the README as well?

great PR, thanks!
beside naiyan's comment, could you please acknowledge that we used caffe's .proto in tools/cafe_converter/readme.md. also do we still need google protobuf? if yes, we probably remove the auto generated pb.py file and provide a makefile to generate it instead. since the file varies for different protobuf version.

readme updated.
the proto file was removed and readme shows how pb.py was generated.

LGTM, thanks

@dongguosheng my apologies that i didn't check it clearly.
my question is, does a user needs to install protobuf to the codes. in other words, does the xxx.pb.py requires protobuf to run. If not, then it is fine. But if yes, I suggest to

only keeps the .proto file while delete the .pb.py file, since the latter varies for different versions of google protobuf
add a makefile which will compile the .pb.py using the protobuf
add a readme to tell the users how to install the google protobuf

Also add an option to turn it off so we don't have to depend on protobuf

@mli i got your idea :)
protobuf isn't needed to use this tool, i mean the protoc  bin and protobuf.so.xxx aren't needed.
The google protobuf python package is definely needed.
xxx.pb.py left to make this tool easier to use. even without google protobuf python package, pip intall is simple, maybe i should add google protobuf python package's version in readme?

update in #662

Thanks for the contribution! Have added you to https://github.com/dmlc/mxnet/blob/master/CONTRIBUTORS.md Please feel free to update more information about you and contribute more of your wisdom to mxnet!


[Feature]:  Support for ARM windows

Description of the new feature / enhancement
Compile NVM to support ARM, as latest Windows CoPilot+ PC's are running on ARM
Scenario when this would be used?
All Windows Copilot+ PC's
Supporting information
No response

Duplicate #502.
There is an open PR for this, but our code locked signing certificate still hasn't been resolved.


Implement Binary Search in Python

Issue Description

Need the python implementation of Binary search algorithm.
Corresponding post on website
Permitted languages - Python

Guidelines

Your code will be live on a tutorial website, so code readability and understandability will be given paramount importance.
Please follow coding conventions listed in CONVENTIONS.md
The directory structure of the project is pretty obvious, but take a moment to go through STRUCTURE.md if you are confused where to place your code.
Please add a few sample input output test case files too. Minimum 1. The more the merrier!

@VamsiSangam I will work on this.

Sure @ranesr go ahead!


Find highlights both camelcase and non-camelcase for the same spelled word

I did a find on "objectStore".
It highlighted both "objectStore" and "objectstore".
I verified it on another word. Same behaviour.
Is this the desired behaviour?
Lew

Toggle on "Match Case":


defaultValue not set on upsert

defaultValue is not set when a doc is inserted via upsert API.
e.g.
Books.attachSchema(new SimpleSchema({
  author: {
    type: String
  },
  title: {
    type: String,
    defaultValue: 'a',
    optional: true
  },
  price: {
    type: Number
  }
}));

Books.upsert({author: 'foo'}, {$set: {author: 'fooo', price: 1}});
// => no title is set.

+1

+1

+1

Please don't comment +1 unless you can add to the discussion. There is a reaction button for that.

Closing old issues. Most likely this has been fixed at some point in SimpleSchema package.


ContainerRemove returns containerNotFoundError on 404

ContainerRemove returns containerNotFoundError on 404, just like ContainerInspect

Please sign your commits following these rules:
https://github.com/docker/docker/blob/master/CONTRIBUTING.md#sign-your-work
The easiest way to do this is to amend the last commit:
$ git clone -b "master" git@github.com:hanyong/engine-api.git somewhere
$ cd somewhere
$ git commit --amend -s --no-edit
$ git push -f

Ammending updates the existing PR. You DO NOT need to open a new one.

not sure what's up, but I can't seem to find the branch for this PR. I don't even see a docker repo under the "hanyong" github id.  Clearly my git/github skills are lacking  :-)

never mind - I see I'm in the wrong repo  :-)

Before this change I see:
$ docker rm 123123123
Error response from daemon: No such container: 123123123

with this change I now see:
$ docker rm 123123123
Error: No such container: 123123123

However, when I look at other cmds, like stop, I see:
$ docker stop 123123123
Error response from daemon: No such container: 123123123

And some other cmds, like start, follow an even different pattern:
$ docker start 123123123
Error response from daemon: No such container: 123123123
Error: failed to start containers: 123123123

I don't care much about which pattern we follow, but before we change this one I'd prefer if we picked one and applied it to all of the cmds so there's some consistency.

we use engine-api for docker container management, we want to distinguish container not found and other errors. we found ContainerInspect did this, it's useful to add this feature to other functions.

I agree with @duglin. We should be consistent in all remote calls.

@hanyong Are you up for changing the other calls to return the correct error on 404 as well?

Thanks for this PR.
We have moved the code from this repo back into the main docker/docker repository; if you still wish to make this change you will need to reopen this PR on the main repo.
You can find the code for the client and types here:
https://github.com/docker/docker/tree/master/api/types
https://github.com/docker/docker/tree/master/client
Sorry for the inconvenience but making this change helps people better contribute and not have the vendor mess going forward when changing the types or client for Docker.
Sorry about that and feel free to ask if you have any questions.


Use recursive lookup for nested group search in AD

Till now we have been using the Microsoft recommended AD query for looking up nested group memberships. That query turned out to be very slow and times out on large databases on the AD server side. We even found some links where others using this query have also found it to be slow.
So we are now going to use our own logic to get a user's parent groups recursively. According to some answers online, that is faster than MS AD query.
Openldap already had this code, so this PR uses the same code for AD. It puts the code in common package, so both OpenLdap and AD can call the common methods
Issue: https://github.com/rancher/rancher/issues/15950

Do we have a validation test for checking nested groups functionality for AD?

LGTM

@prachidamle yes we do have validation tests for nested lookup for AD and openldap


Could not update user profile

Describe the bug
I try to update profile on : https://localhost:44398/user-profile/4f534928-1503-4972-99fa-5f4e3d7eeedd
To Reproduce
Steps to reproduce the behavior:

https://localhost:44398/user-profile/4f534928-1503-4972-99fa-5f4e3d7eeedd
Update data
Click save changes

Expected behavior
update profile
Screenshots

Desktop (please complete the following information):
Google Chrome

Hi! This view is yet not completed. The idea is to just display profile information of other users as well. Something like a public profile page, maybe. I have not yet changed the textfields to readonly. To update the profile, one has to go to the Accounts page. The Save Changes that you see here are for Admin Only (To toggle the status of Non-Admin users). Will make it more clear in the next commit. Hold on

Do check out the code and close this issue if necessary.


(Stock Center) Tasks - 7/19/2023

7/19/2023

Order Flow
priority:

Put "order flow" inside shopping cart
- make sure all views in order flow have tests
one more wireframe for a view [complete by tomorrow]
- before cart,  previous view
- route on the edges of the graph
- remove unnecessary colors
- light background for stateful view/components (anything that uses jotai)
- each view has tests
working on submit view
url route for shipping/payment/submit view
for each view, have a wireframe
have the state represented somewhere in diagram

/ -> HomePage
:provider/callback -> OauthCallback
plasmids -> PlasmidDetailsContainer
:load/auth -> AuthLoader
logout -> Logout
mydsc -> MyDscPage
information/:name -> InfoPageContainer
information/:edit -> EditInfoPage
addpage -> AddPage
strains/:id -> StrainDetailsContainer
phenotypes/:name PhenotypesWrapper
plasmids/:id -> PlasmidDetailsContainer
contact -> ContactPage
cart -> CartHandler
submitted/:orderId  ->

-> PageNotFound

StrainDetailsContainer

[ ] DetailsHeader

[ ] DetailsHeaderCopyIcon

[ ] StrainDetailsCard

[ ] GenesDisplay
[ ] GenotypesDisplay
[ ] PublicationDisplay
[ ] StrainDetailsCardHeader

[ ] Availability

[ ] AvailabilityDisplay

[ ] AddToCartDialog

[ ] DialogTitleDisplay
[ ] AddToCartDialogContent
[ ] AddToCartDialogActions

[ ] DetailsListItem
[ ] PhenotypeList

[ ] PhenotypeListHeader

[ ] PublicationDisplay
[ ] GenesDisplay

flowchart TD

X([Strains])
A[Catalog Wrapper]
B1[StrainCatalogContainer]
C[StrainDetailsContainer]

X -->|"/strains\nstockType={strain}"| A
A -->|"stockType === 'strain'"| B1
B1 -->|strains/:id|C

classDef state fill:#ab3017
class A,B1,B2 state


Correctly handle interpreter changes for Interactive and Notebook

For #8019

[x] Pull request represents a single change (i.e. not fixing disparate/unrelated things in a single PR)
[x] Title summarizes what is changing
[x] Has a news entry file (remember to thank yourself!)
[x] Appropriate comments and documentation strings in the code
[ ] Has sufficient logging.
[ ] Has telemetry for enhancements.
[x] Unit tests & system/integration tests are added/updated
[ ] Test plan is updated as appropriate
[ ] package-lock.json has been regenerated by running npm install (if dependencies have changed)
[ ] The wiki is updated with any design decisions/details.

Codecov Report

Merging #8385 into master will increase coverage by 0.66%.
The diff coverage is 41.66%.

@@            Coverage Diff             @@
##           master    #8385      +/-   ##
==========================================
+ Coverage    58.7%   59.37%   +0.66%
==========================================
  Files         509      509
  Lines       22923    23299     +376
  Branches     3754     3763       +9
==========================================
+ Hits        13457    13833     +376
+ Misses       8570     8567       -3
- Partials      896      899       +3

Impacted Files
Coverage Δ

src/client/datascience/types.ts
100% <ø> (ø)
:arrow_up:

...science/jupyter/liveshare/guestJupyterExecution.ts
22.91% <0%> (ø)
:arrow_up:

.../datascience/interactive-common/interactiveBase.ts
24.16% <0%> (-0.05%)
:arrow_down:

...ascience/jupyter/liveshare/hostJupyterExecution.ts
45.45% <100%> (ø)
:arrow_up:

...lient/datascience/jupyter/liveshare/serverCache.ts
50% <50%> (-0.91%)
:arrow_down:

...t/datascience/jupyter/jupyterDataRateLimitError.ts
100% <0%> (ø)
:arrow_up:

src/client/formatters/serviceRegistry.ts
100% <0%> (ø)
:arrow_up:

...lient/datascience/jupyter/jupyterSelfCertsError.ts
100% <0%> (ø)
:arrow_up:

src/client/debugger/extension/serviceRegistry.ts
100% <0%> (ø)
:arrow_up:

...ience/jupyter/jupyterDebuggerRemoteNotSupported.ts
100% <0%> (ø)
:arrow_up:

... and 33 more

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 a139d63...ffeb017. Read the comment docs.


Remove mistakenly repeated require statement in director's api.rb file

The PR is described in the title ^^

The error in CI does not seem to be connected with this PR.


Actionbar message not working correctly

It seems like some messages are not displaying correctly. I have changed the action bar message, but it shows the default message, as you can see in the screenshot. Here is my config:
# Harbor - Sleep mechanics enhancement plugin
# An open-source project by TechToolbox (@nkomarn)
# https://github.com/nkomarn/Harbor

# Important note regarding Essentials:
# Please make sure you negate the permission "essentials.sleepingignored" to
# prevent issues with Harbor and server operators when running Essentials.

night-skip:
  enabled: true # Skip the night if a percentage of the players in a world are sleeping
  percentage: 40 # Percentage of players required to be sleeping to skip the night (0 - 100)
  time-rate: 50 # The amount of ticks added to the current time every tick when skipping the night
  daytime-ticks: 1200 # The time in ticks that Harbor considers day
  instant-skip: false # Instantly skip the night when the percentage threshold is reached instead of showing the full animation
  proportional-acceleration: false # Increase the night skipping speed based on the amount of sleeping players
  clear-rain: true # Clear rain (if it's raining) when the night is skipped
  clear-thunder: true # Clear thunder (if it's thundering) when the night is skipped
  reset-phantom-statistic: false # Treats everyone online as if they have slept in the last 3 days after the night is skipped (check out /gamerule doInsomnia on 1.15+)

exclusions:
  ignored-permission: true # Exclude players with the permission "harbor.ignored" from the sleeping count
  exclude-adventure: true # Exclude players in adventure mode from the sleeping count
  exclude-creative: true # Exclude players in creative mode from the sleeping count
  exclude-spectator: true # Exclude players in spectator mode from the sleeping count
  exclude-vanished: true # Exclude vanished players from the sleeping count

# Detect AFK players and automatically remove them from the required sleeping count
# Essentials API is used for AFK detection when available- otherwise a fallback system is used
afk-detection:
  enabled: true
  timeout: 3 # Time in minutes until a player is considered AFK

# Blacklist for worlds- Harbor will ignore these worlds
blacklisted-worlds:
  - "Lobby"
whitelist-mode: false # Will treat the above list as a whitelist instead of a blacklist

messages:
  chat:
    enabled: false
    message-cooldown: 5 # The amount of time in seconds before a player can send another bed enter or leave message
    player-sleeping: "&e[player] está durmiendo ([sleeping]/[needed])"
    player-left-bed: "&e[player] dejó de dormir ([sleeping]/[needed])"
    night-skipping:
      - "&eAcelerando la noche..."
    night-skipped:
      - "&aSe ha saltado la noche"
  actionbar:
    enabled: false
    players-sleeping: "&e[sleeping] de [needed] jugadores están durmiendo"
    night-skipping: "&eTodo el mundo está durmiendo"
  bossbar:
    enabled: true
    players-sleeping:
      message: "&f&l[sleeping] de [needed] jugadores están durmiendo"
      color: BLUE
    night-skipping:
      message: "&f&lTodo el mundo está durmiendo"
      color: GREEN
  miscellaneous:
    chat-prefix: "&8&l[&6&lNoche&8&l]&r "
    unrecognized-command: "Comando desconocido"

# Spooky internal controls
version: 1.6.3
interval: 1
metrics: true
debug: false

I am using Purpur 1.17 build 1204. I have not tested this in previous versions.

The actionbar you're seeing is actually coming from vanilla and was an added feature of 1.17.
I'll see how I can go about overriding this.

In case it is useful, Purpur implemeted a few days ago a setting to modify this message


emails not being delievered

Issue Summary
Emails are not going since today morning. Please review the code below:
$url = 'https://api.sendgrid.com/';
$user = '';
$pass = '';
$message='test message';

$subject='test subject';
$toemail= '';
$fromemail='';
$fromname'='';
    $json_string = array( 'unique_args' => array( 'subject' => $subject ));

$params = array(
    'api_user'  => $user,
    'api_key'   => $pass,
    'to'        => $toemail,
    'subject'   => $subject,
    'html'      => $message,
    'from'      => $fromemail,
    'fromname'  => $fromname,
	'x-smtpapi' => json_encode($json_string),
  );

echo '<pre>';
print_r($params);
echo '</pre>';

$request =  $url.'api/mail.send.json';
//$request =  $url.'mail.send.json';

// Generate curl request
$session = curl_init($request);
// Tell curl to use HTTP POST
curl_setopt ($session, CURLOPT_POST, true);
// Tell curl that this is the body of the POST
curl_setopt ($session, CURLOPT_POSTFIELDS, $params);
// Tell curl not to return headers, but do return the response
curl_setopt($session, CURLOPT_HEADER, false);
curl_setopt($session, CURLOPT_RETURNTRANSFER, true);

// obtain response
$response = curl_exec($session);
curl_close($session);

please help through it as it is very urgent expecting for the best help from you as soon as possible.
Steps to Reproduce

This is the first step
This is the second step
Further steps, etc.

Any other information you want to share that is relevant to the issue being reported. Especially, why do you consider this to be a bug? What do you expect to happen instead?
Technical details:

sendgrid-php Version: master (latest commit: [commit number])
PHP Version: 5.x

Hello @ruchisharma123,
This is not code from this library, could you please reach out to https://support.sendgrid.com? They will be happy to help.
Thanks!


Assorted Fixes

These are the small tweaks from the paper branch, many of which are dependencies of future PR's.

Should this have been merged to paper, too?

FYI, I'm facing some issues after this PR. Looks like in some cases, Vagrant provider is not booted successfully, as I see logs like
INFO [Sep 28 02:36:36.624] Failed to retrieve IP address for 04351510-4404-4616-be87-715290fd8cba. error=exit status 1
ERROR [Sep 28 02:36:38.175] Failed to execute vagrant command: VM must be created before running this command. Run `vagrant up` first.

(logs from #536)
I'll take a peek later (likely tomorrow evening) but wanted to make note.

@tejasmanohar I didn't see that when testing, but our Vagrant backend has been pretty flaky in general (we use Amazon exclusively basically).

If you want to fix it though, that'd be awesome.  It'd definitely be useful, it's just been on the back burner while we tackle other priorities.


Dynamically grab port number

Before submitting

[X] Was this discussed/agreed via a Github issue? (no need for typos and docs improvements) - No, but discussed w/ Aniket on Slack
[X] Did you read the contributor guideline, Pull Request section?
[ ] Did you make sure to update the docs?
[ ] Did you write any new necessary tests?

⚠ How does this PR impact the user? ⚠
Now, running python client.py still works if user specifies port other than 8000 when calling server.run(). User doesn't need to know or have to change the port number generated in the client.py file.
What does this PR do?
No issue created, but can do so for posterity's sake.
PR review
Anyone in the community is free to review the PR once the tests have passed.
If we didn't discuss your PR in GitHub issues there's a high chance it will not be merged.
Did you have fun?
Yes

@aniketmaurya Incorporated your suggestions. Let me know how this looks!
Here's testing just the test_e2e.py file with all tests passing: https://www.loom.com/share/cd95ae0703f84ccea57c4f8eecb8d209?sid=5dc7161c-682a-4367-8f9c-f32f262cff6d
And here is running pytest (all tests):

It seems that this PR removed the functionality that previously skipped updating the client file if it already existed. Currently, if a client file is present, instead of just updating the port, the entire file is being replaced.
Is this an intended change or a potential bug?

great catch @bhimrazy!! it's a bug that missed somehow here.


hosts = { ... } doesn't work if /etc/hosts is not writable

This should be added to the docs womewhere:
To make hosts (hostctl) work in NixOS this setting is required.
It changes /etc/hosts to be writable for root.
environment.etc.hosts.mode = "0644";

Yeah, we should emit that error, could you make a PR?

I had to delete the .devenv folder for this to work for some reason. But it works after:

make /etc/hosts writeable
delete projects .devenv folder
devenv up
devenv shell ping myhostname


Reuse hash of message instead of recomputing it

This reuses the id of ssb-validates append instead of recomputing it. On my machine this saves 5 seconds out of 157 seconds for benchmark 11 in bench-ssb.

There was one more, so the improvement is now a bit bigger. I would like some eyes on the second one, as I'm not 100% its good as the function seems to buffer. The tests parses.

I think it's not safe to take the ID from state.feeds[feed.id].id it's possible that multiple messages are appended to ssb-validate's queue before feed is called - that is a supported and tested feature of ssb-validate. A better fix might be to change ssb-validate to use {key, value, timestamp} format in it's output queue.

 This would mean messages get a completely wrong key!

Right, I removed the last commit as that was not good.


In flutter doctor -v, when JRE is too out-of-date to run sdkmanager, print a helpful error message

Closes https://github.com/flutter/flutter/issues/138132. See this issue for more information.
Pre-launch Checklist

[x] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[x] I read the Tree Hygiene wiki page, which explains my responsibilities.
[x] I read and followed the Flutter Style Guide, including Features we expect every widget to implement.
[x] I signed the CLA.
[x] I listed at least one issue that this PR fixes in the description above.
[x] I updated/added relevant documentation (doc comments with ///).
[x] I added new tests to check the change I am making, or this PR is test-exempt.
[x] All existing and new tests are passing.

If you need help, consider asking for advice on the #hackers-new channel on Discord.

LGTM

Thanks! I appreciate the good level of scrutiny you put into this review

LGTM

Thanks! I appreciate the good level of scrutiny you put into this review

that's a charitable way to put it :)


Sphinx c++ roles for functions aren't being parsed

After documentation is built, ReST c++ roles are left as plaintext (i.e. :role:)

In particular, :param param_name:, :throws: and :returns: roles are left as plaintext

fixed: need to use doxygen roles for inline docstrings


Unable to find element on MacOS desktop

Hi Ken, thanks for developing this tool. It's simple and straightforward and easy to get started.
I'm facing a little problem at the moment. I've installed and setup the following below. But somehow it's not able to find element on desktop screen, the output always returning False. I was trying to get the Calendar app icon.
But it works on Chrome browser and able to find visual elements like logos or buttons.
Did I miss something else?
Install and setup
Install RPA package from pip
Installed JDK 11 from Amazon
Allowed Java and Terminal in accessibility (in Security and Privacy control panel)
r.Init() first time setup successful
Code run
import rpa as r
r.init(visual_automation = True, chrome_browser = False)
r.exist('cal.PNG') # if true = found
The log output below didn't seem to throw any error.
LIVE MODE - type done to quit
[RPA][STARTED]
[RPA][0] - listening for inputs
[RPA][1] - exist_result = exist('cal.PNG').toString()
[RPA][1] - listening for inputs
[RPA][2] - dump exist_result to rpa_python.txt
[RPA][2] - listening for inputs
I'm on Mac Catalina 10.15.4 (early 2015)
display default scale at 1280 x 800

After some time on google, suspect that Retina display could be the cause of problem. So I went to connect my MacBook to an external display and set it at 1920 x 1080. It works!
I also had a Windows setup but it didn't give me this problem. Will use this workaround for the time being :)

Hi @natngmk, sorry I was away the past few months - https://github.com/tebelorg/RPA-Python/issues/144
Yes, your suspicion is correct. I think I saw an open issue regarding Retina display. Will see if I can get more details from there to put this info in docs. From what I remember, the gist of the issue is on Retina display, when a screenshot is taken, it is actually a different resolution and size from the actual screen. So, using that image as an input will lead to it not being found on the screen. Because the image is quite different in size from the actual screen.


Indexed PNG support

the code could be a mess, tried to implement asap

Where is the png.palette coming from ? I don't see the definition of this property in the PngFile.
You forgot to push the types.v file ?

You forgot to push the types.v file ?
Wow, my code for reading PLTE is gone 0_0
Maybe I forgot to commit it? Strange
I fixed it by writing new code, should works

Test files:
if you process input.png you should get output.png-like file
MD5: 5f7a1b28c42b9501bf71989127c9b934
Image from DROD(C) by Caravel Games


Build: Fix trim warnings when using dotnet format

Description

Fixes https://github.com/MudBlazor/MudBlazor/issues/8410 in preparation for a PR dotnet format check
How Has This Been Tested?

Run the test suite
Types of changes

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)

Checklist:

[x] The PR is submitted to the correct branch (dev).
[x] My code follows the code style of this project.
[ ] I've added relevant tests.

LGTM, to as i said I probably name the private method AddCore like MS does, to not confuse with the public methods.

I hope this is what you meant I couldn't see a way to reuse AddCore anywhere so maybe I misunderstood you comment in issue.

I hope this is what you meant I couldn't see a way to reuse AddCore anywhere so maybe I misunderstood you comment in issue.

Yes, thanks.


Cache invalidation on local dependencies when using docker

Hello.
I am having difficulties using uv in docker. My project structure has two python libraries, one is an API server, and the other is an internal library.
The problem I am encountering is that after building, when running the image, the uv run reinstalls my internal libraries, with the following message: Requirement installed, but not fresh: .... This is problematic, because as per docker best practices, it is good to ensure that the install actually happens on the build stage, and not in the actual running phase of my application.
When debugging the issue further, I found out that it happens because the ctime of my pyproject.toml of the internal library changes between the docker build and the actual run.
I finally validated that symptom using the following dockerfile:
FROM ubuntu:22.04

RUN touch a && stat a > /root/stat-a
RUN sleep 5

And running: docker build --no-cache . -t playground && docker run playground sh -c 'cat /root/stat-a && stat a'
The output that I received was:
  File: a
  Size: 0               Blocks: 0          IO Block: 4096   regular empty file
Device: e3h/227d        Inode: 1336246     Links: 1
Access: (0644/-rw-r--r--)  Uid: (    0/    root)   Gid: (    0/    root)
Access: 2024-11-18 10:04:52.462498005 +0000
Modify: 2024-11-18 10:04:52.462498005 +0000
Change: 2024-11-18 10:04:52.462498005 +0000
 Birth: 2024-11-18 10:04:52.462498005 +0000
---
  File: a
  Size: 0               Blocks: 0          IO Block: 4096   regular empty file
Device: e3h/227d        Inode: 225316      Links: 1
Access: (0644/-rw-r--r--)  Uid: (    0/    root)   Gid: (    0/    root)
Access: 2024-11-18 10:04:52.000000000 +0000
Modify: 2024-11-18 10:04:52.000000000 +0000
Change: 2024-11-18 10:04:57.680498008 +0000
 Birth: 2024-11-18 10:04:57.680498008 +0000

As seen, two things are changing between the build and run - both the inode number, and the Change & Birth times.
This seems like an odd behavior of docker, but as I dived deeper it seems inherent to the layering system of docker.
I think that the solution to that issue should come from uv's side, as the usage for similar type of projects in docker should be very common.
A possible approach that would solve both the cache freshness problem and would work with docker's odd behavior would be using checksums for local dependencies freshness check.
Some additional information:

uv version: v0.5.1
Platform: MacOS with Docker Desktop

Thanks in advance!
Yannay

Hey,
So I worked a bit to reproduce it, created this fork of the example from the official documentation of uv
Added one commit that changes the command to use uv run instead of modifying the ENV (which is better anyways in my opinion).
When running the build and then run:
$ docker build -t uv-example .
$ docker run uv-example
Uninstalled 1 package in 0.68ms
Installed 2 packages in 10ms
...

As seen in the last two rows, it uninstalled and reinstalled the package of the example, as the symptoms that I encountered.
Thanks for responding so fast!

Thanks! Interestingly, with that example I see:
❯ docker run uv-example
Installed 1 package in 6ms
Bytecode compiled 1143 files in 49ms
INFO     Using path src/uv_docker_example
INFO     Resolved absolute path /app/src/uv_docker_example

Am I misunderstanding the output, or does it look like a platform difference?

Yeah it does seem different, I am running this on MacOS with docker desktop.
Though in your output it still shows that it installed a package, is it the expected behavior? I would expect it to not install any package as we run uv sync in the dockerfile.

This is very odd then.
I am running on M2 with macOS Sequoia 15.1 and Docker Desktop v4.35.1 with docker engine v27.3.1

Are ya'll definitely testing the same uv version? The image could be cached if you're just using that example repository.

So in my case at least, the installed package in Installed 1 package is Ruff, due to the use of --no-dev when syncing the Dockerfile. (uv run will sync dev by default.)

Have you considered using uv run --frozen in your CMD instead?

(I'm also on Docker version 27.3.1. M3, Sonoma 14.5.)

Still the same when running with --frozen flag.
The uv version from inside the container is 0.5.3.
Could it be related to the VM options in docker?
I am using Apple Virtualization framework with Rosetta enabled and VirtioFS for file sharing implementation.

(I'm also on Docker version 27.3.1. M3, Sonoma 14.5.)

Hey @charliermarsh, is there any more information I can share that will help you reproduce it locally?

Same issue on my side - following this example leads to Uninstalled 1 package [..] Installed 1 package. Using --verbose on uv run surfaces "Requirement installed, but not fresh", with the requirement being the project itself.


🚨 Potential Cross-Site Request Forgery (CSRF)

👋 Hello, @negative0, @greenkeeper[bot], @jackw - a potential high severity Cross-Site Request Forgery (CSRF) vulnerability in your repository has been disclosed to us.
Next Steps
1⃣  Visit https://huntr.dev/bounties/1-other-rclone/rclone-webui-react for more advisory information.
2⃣  Sign-up to validate or speak to the researcher for more assistance.
3⃣ Propose a patch or outsource it to our community - whoever fixes it gets paid.

Confused or need more help?

Join us on our Discord and a member of our team will be happy to help! 🤗

Speak to a member of our team: @JamieSlome

This issue was automatically generated by huntr.dev - a bug bounty board for securing open source code.

Hi @jackw, @negative0, can you please update the report status to valid at https://huntr.dev/bounties/1-other-rclone/rclone-webui-react/

Hi,
Sorry for the late reply.
I am not able to validate using your HTML script, it keeps on asking for username and password, what options are you using while running rclone?

Hi @negative0,
I launched rclone using this command:
rclone rcd --rc-web-gui
Revisiting the issue, the problem seems to be the use of address http://<IP_ADDRESS> instead of http://localhost in the POC. So please try the following POC to alter the bandwitch to 1M  through CSRF.
// changeBandwidth.html
<form method="POST" action="http://localhost:5572/core/bwlimit?rate=1M">
  <input type="submit" value="CSRF" />
  <script>
    document.forms[0].submit();
  </script>
</form>


Chain prelude evaluation so that primitives can depend on each other

This PR allows evaluating a second prelude (like the one in inferno-ml) in an environment that already has the first prelude (the core/builtin one), so that the second prelude can use ops and primitives defined in the first one.
It creates a new Prelude type (that subsumes the type alias ModuleMap) and refactors existing code to use it.

Looks like a nice improvement!


Scrolling enabled in browser as soon as enableBodyScroll is called once, regardless of number of locks

Hey! I'm having a little trouble understanding how enabling scrolling is intended to work in this library. I see that elements are being tracked in the locks array, but this is only used for scrolling on mobile; in the browser scrolling enables as soon as I call enableBodyScroll, I would expect scrolling to be enabled once the last "lock" is cleared. 🤔

An astute observation! Can anybody provide some insight?

They stopped the repairs. I had to do it myself, in the same way, with a new version of typeScript. And fix these problems for everyone to use.
npm i body-scroll-lock-upgrade

repair log，Refer to the releases page.


spec: the identity of instantiated types is not well defined

What version of Go are you using (go version)?

$ go version
go version go1.19.3 linux/amd64

Does this issue reproduce with the latest release?
Yes
What did you do?

package main

type G[T any] struct {}

func main() {
	var x, y, z any = G[int]{}, G[int]{}, G[bool]{}
	println(x == y) // true
	println(x == z) // false
}

The result might be not unexpected (some unexpected for me, as I think x == z should also result in true), but the current spec is not very clear on this:

Instantiating a type results in a new non-generic named type;

People may think the types of x and y are different by this description.

There's also Type identity

Two instantiated types are identical if their defined types and all type arguments are identical.

Didn't notice that line.
Thanks. Closed.


(SampleApp:23011): GStreamer-CRITICAL **: 13:03:05.644: gst_object_unref: assertion 'object != NULL' failed

Hardware:
Device: Raspberry Pi 4 (4 GB)
Software:
OS: Raspbian GNU/Linux 10 (buster)
Distributor ID:	Raspbian
Description:	Raspbian GNU/Linux 10 (buster)
Release:	10
Codename:	buster
Issue: SampleApp does not start.
Step 1: Obtained config.json from the Developer Console
Step 2: Downloaded setup.sh, genConfig.sh & pi.sh
Step 3: Successfully installed using
$ sudo apt update
$ sudo apt upgrade
$ sudo ./setup.sh config.json
Step 4: Launch SampleApp
sudo bash startsample.sh
Error Log:
pi@raspberrypi:~/alexa$ sudo bash startsample.sh
configFile /home/pi/alexa/build/Integration/AlexaClientSDKConfig.json
Running app with log level: DEBUG9
2020-06-26 12:08:29.255 [  1] I AlexaClientSdkInit:initialize:sdkversion=1.20.0
2020-06-26 12:08:29.259 [  1] 0 ConfigurationNode:initializeSuccess
2020-06-26 12:08:29.265 [  1] 1 InMemoryEqualizerConfiguration:validateConfiguration::Validating Equalizer configuration
2020-06-26 12:08:29.265 [  1] 1 InMemoryEqualizerConfiguration:validateConfiguration::Validating default Equalizer state
2020-06-26 12:08:29.265 [  1] 1 InMemoryEqualizerConfiguration:validateConfiguration::Validating Equalizer configuration
2020-06-26 12:08:29.265 [  1] 1 InMemoryEqualizerConfiguration:validateConfiguration::Validating default Equalizer state
2020-06-26 12:08:29.267 [  1] 9 MediaPlayer:createCalled:name=SpeakMediaPlayer
2020-06-26 12:08:29.320 [  1] E MediaPlayer:setupPipelineFailed:name=SpeakMediaPlayer,reason=createAudioSinkElementFailed,audioSinkElement=alsasink
2020-06-26 12:08:29.320 [  1] E MediaPlayer:initPlayerFailed:name=SpeakMediaPlayer,reason=setupPipelineFailed
2020-06-26 12:08:29.320 [  1] 9 MediaPlayer:~MediaPlayer:name=SpeakMediaPlayer
(SampleApp:23044): GStreamer-CRITICAL **: 13:08:29.321: gst_object_unref: assertion 'object != NULL' failed
2020-06-26 12:08:29.321 [  1] 9 MediaPlayer:resetPipeline:name=SpeakMediaPlayer
2020-06-26 12:08:29.321 [  1] E RequiresShutdown:~RequiresShutdownFailed:reason=notShutdown,name=SpeakMediaPlayer
2020-06-26 12:08:29.321 [  1] C SampleApplication:Failed to create application media interfaces for speech!
2020-06-26 12:08:29.322 [  1] C SampleApplication:Failed to initialize SampleApplication
Failed to create SampleApplication!


Fix broken link & update old MSDN links to Docs.

@kmgallahan : Thanks for your contribution! The author, @dragon119, has been notified to review your proposed change.

@Jak-MS : Thanks for your contribution! The author, @dragon119, has been notified to review your proposed change.

@Jak-MS : Thanks for your contribution! The author, @dragon119, has been notified to review your proposed change.

confirmed correction


Tests to ensure standard devise has greater priority than tokens

This PR adds some tests to make sure devise_token_auth doesn't conflict with devise in some edge cases (namely when token information is provided alongside a standard devise session).

@colavitam - going to go ahead and take your word on this, as I don't have any experience with direct warden authentication outside of devise / devise_token_auth and several users are having issues ever since that support went in. Thanks for taking a dive on this!


Improvement on Azure Object Storage Docs and/or azure object config validation

Hi folks. I just spent a few hours trying to thanos query to connect to azure storage. The example in the docs includes a lot fieldname with no examples of the values.
type: AZURE
config:
  storage_account: ""
  storage_account_key: ""
  container: ""
  endpoint: ""

So my first crack was to put https://<storageaccount>.blob.core.windows.net as the value of endpoint. This results in query just crashing out with no error after the output level=debug ts=2021-11-14T08:40:41.878900465Z caller=azure.go:165 msg="creating new Azure bucket connection" component=store. All i see apart from that is exit code 2.
Anyway after rummaging around azure.go i arrived at https://github.com/thanos-io/thanos/blob/d72bff0de8a420c824743a4cedf7879419afe6cd/pkg/objstore/azure/helpers.go#L89 and I now realise that endpoint should have a value like blob.core.windows.net.
Describe the solution you'd like
Please make the use of the "endpoint" field clear in the docs. Some example values (not "") wouldn't be a bad idea too.
Also, the silent mystery crash could use some error output, or alternatively when validating the conf, validate that endpoint doesn't contain https? Maybe that is just [a-z0-9.]+
Additional context
I'm a complete noob with this stack

Hi, cheers for making this issue and addressing the difficulty.
I agree on the config part, this has actually been a somewhat long standing issue to make those more useful. To give you a bit more context; we auto generate these. However that does not provide the defaults nor for example sane values (:
For the not("") ; https://github.com/thanos-io/thanos/blob/d72bff0de8a420c824743a4cedf7879419afe6cd/pkg/objstore/azure/azure.go#L133
So, I think it should be;

Improve usage documentation for Azure Storage
Improve handling of certain inputs in the storage config. To be precise: https:// makes it crash without any useful information.

Let me know if I missed something and thanks again!

That sounds like everything. Cheers 🍻

Facing the same issue. Anyway to provide a full config using MSI with example values in this issue?


Team Aries Code Review

-- This is one of the best code I've seen since the start of my code review, if not the best!
-- Great interface in the end.
-- Congratulation to you kblycaglayan!!!

Hello Team,
Thanks for the feedback.
With all my respect...


Netflix error 1044 android 9.0 motorola g6

Bug report
Your Environment

Netflix add-on version: <1.4.0>
Operating system version/name: <Android 9.0>
Device model:  <Moto G6 - XT1925-3>

Used Operating system:

[x] Android
[ ] iOS
[ ] Linux
[ ] OSX
[ ] Raspberry-Pi
[ ] Windows

Describe the bug
<I enter the extension, log in to my account.  When I insert a video, this error appears (this title is not available for viewing online. Try another one. 1044)>

Expected behavior

Actual behavior

(Watch the video normally)
Steps to reproduce the behavior

I open the kodi
I'll go on netflix
I choose any series
As soon as I watch, the error appears

Possible fix

in a future update
Debug log

The debug log can be found from this link:
(https://drive.google.com/folderview?id=1hd_0iq-vQiLthdvOpxLdMzdyX-KEld-4)
Additional context or screenshots (if appropriate)
Link Screenshots :
(https://drive.google.com/folderview?id=1hd_0iq-vQiLthdvOpxLdMzdyX-KEld-4)
Installation

[ ] I'm using other Netflix Repo
[ ] I'm using a different source

Other information

Screenshots

(https://drive.google.com/folderview?id=1hd_0iq-vQiLthdvOpxLdMzdyX-KEld-4)

In expert settings select Reset ESN
then try again
and get the log

I did what you said, but I was not successful, the same error appears after
cleaning esn
Link new log:
https://drive.google.com/folderview?id=1i7hhDoIXwxt9cKMoUGlKxZtDyT7iT-TI

If you use VPN/Proxies then disable it
if you do not use it, then i do not know what can be the problem with your nf account,
you can try also test to change language of the profile used and see what happen

Netflix suggest to do Logout: https://help.netflix.com/it/node/109329
try also this

I'm not using vpn or proxy, I tried to change the language and leave and
nothing worked, the funny thing is that on my moto g1 android 5.1 phone it
worked

but can this problem be resolved in a future update?  because I don't know what to do.

i have no idea what to do because this is the first time that a user have this problem
and this warning message come directly from netflix not the addon
can you attach here a screenshot of the esn from official netflix app settings,
with censuring last few characters?

is it dangerous to send netflix esn?

no because all model device as your use the same esn
you have only to censure the last numbers part

Screenshot esn

https://drive.google.com/folderview?id=10fHqEylGIxCutjtGKUVTp8nCHPtbDMu9

We can try to use this test version and follow this instructions steps:

Delete previous nf addon (with also user settings when prompt)
Install this:
plugin.video.netflix_1.4.0_20200606_t1.zip
Do the login
Now open add-on Expert settings and enable Force Widevine security level L3
Press Ok to save the setting changes
Now try to play a video

give a log

Another error appeared, but different
Screenshot + New Log
https://drive.google.com/folderview?id=10ok41_2KixYhohrfYgZLo5f4OcQ7aTUg

Write this in expert settings Manual ESN field:
NFANDROID1-PRV-P-L3-MOTORMOTO=G=6=-4445-
let know

the same error appears, I don't know what to do.

and my security level for widevine is L1, but I don't understand why netflix puts it as L3

i do not understand because raise error, i need time to recheck all the code

i have tested to my phone but works,
so with your phone the partial esn seem not accepted
can you try insert also these esn?
NFCDCH-01-GDQQZVC3XCJT8MQDF313WW2Y03RT3W
NFCDCH-AP-GDQQZVC3XCJT8MQDF313WW2Y03RT3W
probably doesn't work, but better try

When I put both esn, the kodi closes, so I open it, select the episode I want to watch and the error 1044 appears again

i am sorry but i do not have other solutions
you can not use it with this device..

So, is this a problem with my device?

Netflix add-on is an unofficial Netflix platform based on website
and it is compatible with the most of devices, but not all devices.
can be hundred of unknown reason or unknown limitation or restrictions imposed by netflix website
to some device types or also some wrong source code implementation of the addon,
some understandable, but many others impossible to understand,
because netflix does not release any source code for a correct implementation of his services.
so i think no one can give you an answer on what and where it doesn't work

I found out that this error 1044 appears when I watch netflix on my moto g6's browser, does that also have anything to do with kodi?

this thing no help to find a solution

ok sorry, i don't understand these things, could you close this topic?

is there a way to change the kodi user agent?  because error 1044 is
changed when I do this in a browser.

because error 1044 is changed when I do this in a browser.

what means "is changed"? more details
what browser name?
i need to understand better what you want to do
because changing user agent involve many other things

I used the Windows 10 user agent with chrome and gave the following error,
different only when testing it on my phone:

I have a theory that this has something to do with the user agent

you can not mix windows 10 with android do not have same widevine architecture

yes, ok, but can this have something to do with it?  because i want to
watch netflix on kodi, because on the netflix app you don't choose the
quality.  seriously am i the only one with this problem?

in this new update, I enabled it to force the L3 widevine and an error
other than 1044 appeared, now only this one appears:

I put your esn and gave this error, and when I put my esn the error from
the previous message appears.

still not solved the error?  put the security level L3 and this message
appears, is there still no solution?  please just want to watch netflix on
kodi .....

i do not know what the problem in your device model, but i have a device with same error
and the problem is that is not full google/netflix certified
You can try to check these things:
-Try install from PlayStore: SafetyNet Test app and run test, have to result in to: all test PASSED
-In the PlayStore, open the menu and choose "Settings", scroll down and look what is written on "Play Protect certification", should be written "device certified".
And if you have done one of these things:
-If you have unlocked your bootloader you can not use netflix
-If you have installed custom rom you can not use netflix
-If you have rooted your smartphone you can not use netflix

play protect certification: "Device not certified", safetynet test: passed,
root: never did
What do I do now ?

Nothing... use a certified device

I saw a tutorial on youtube on how to authorize it back, if I do it will it
work?

I left my device certified and, even then, the error appears, I really
don't understand, I never root and my device is certified.

hi, a new error appeared, and error 1044 changed, do you want me to send a log for analysis?

if you want to attach it

I am constantly updating the logs according to the version of the addon, I
will always leave this link activated if you want to see the logs.
Logs:
https://drive.google.com/folderview?id=17silivVeUGtjAK2w11Pg43LHRhsHF1xl

MSLError: Request blacklisted by key exchange service

this error is explained in wiki
https://github.com/CastagnaIT/plugin.video.netflix/wiki/FAQ-(Errors)#request-blacklisted-by-key-exchange-service
therefore your device model is unusable

Hello! I got this same error using LibreElec 9.8 withi Kodi Matrix (alpha release) I think it is related with the device. I installed LibreElec in an Orange PI PC 2, I shared WIFI connection through USB from a Moto G6 Play. Could it be related with the device also?

but were you able to solve the problem?  I didn't quite understand what you said, I have little knowledge about this type of subject, if you could explain it better, I am researching to understand.

hi i have the same error 1044 and I change the ESN manual with this NFCDCH-01-GDQQZVC3XCJT8MQDF313WW2Y03RT3W
and it work fine to me
i use an raspberry pi4 4gb ram with libreelec 9.2.5

@mac1501 I can confirm it works 🎉🎉🎉🎉🎉 thank you very much!  I found this but I don't know if it works for your Android device NFANDROID1-PVR-T-L3-XIAMMIBOX3, NFANDROID1-PRV-B-L3–FORMUFORMULER=Z8-4445, NFANDROID2-PRV-T-L3-HISILICONZ7=PLUS

@edersonmax21 i have read your email, i receive a lot of emails every day, so if possible ask in the appropriate Issue thread instead send a email
for unicode error, seem that you have entered illegal characters in the ESN field
that cause addon crash, do a ESN Reset from expert setting or reinstall addon
people who force the use of ESN of other devices they do it at their own risk
therefore take responsibility for possible repercussions from netflix
i always recommend to avoid this practice


Fix new clippy warnings

Some now clippy warnings about needless_borrow are fixed here.
bors r+

bors r+


Add & Update Icons

added:

AdAway
DHL
Enpass
Feedly
Figma
Geometric Weather
GrubHub
Loop Habit Tracker
Notion
Obsidian
SeriesGuide
Sparkasse
Thenx

updated:

Mega
Infinity for Reddit

The updated icons weren't up to the specs in the contribution guide, i took the liberty to make more fitting versions.

Hello, and thanks for your interest in contributing! What would be your thoughts on using rounded stroke caps for thenx.svg? Since the icons in Lawnicons generally use such stroke caps, I’d say this would make thenx.svg match the remaining icons better.

Hi @leifniem , could you increase the icon size to fit the 160x160px content area (apart from thenx as I've already updated it)? There's some icons here that would be great additions!

Sorry this has been sitting idle for so long, i kinda lost track of this PR.
Sorry for getting my metrics wrong on the other icons, seems i misaligned my guides, i'll see if i can fix those, apparently i converted my outlines and saved the file 💀

No worries. Happy to see that the icons won't be abandoned and left unfixed. Thanks for trying to fix them :)


Update accuracy.py to solve RuntimeError: view size is not compatible with input tensor's size and stride (at least one dimension spans across two contiguous subspaces). Use .reshape(...) instead

Had this error while training "RuntimeError: view size is not compatible with input tensor's size and stride (at least one dimension spans across two contiguous subspaces). Use .reshape(...) instead" and solved by changing view(-1) to reshape(-1) in accuracy.py
Some people got same error in here:
https://github.com/open-mmlab/mmdetection/issues/3047

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

Thanks for the fix! cc @ZwwWayne

Hi!
@UlkuTuncerKucuktas
First of all, we want to express our gratitude for your significant PR in the OpenMMLab project. Your contribution is highly appreciated, and we are grateful for your efforts in helping improve this open-source project during your personal time. We believe that many developers will benefit from your PR.
We would also like to invite you to join our Special Interest Group (SIG) private channel on Discord, where you can share your experiences, ideas, and build connections with like-minded peers. To join the SIG channel, simply message moderator— OpenMMLab on Discord or briefly share your open-source contributions in the #introductions channel and we will assist you. Look forward to seeing you there! Join us ：https://discord.gg/UjgXkPWNqA
If you have WeChat account，welcome to join our community on WeChat. You can add our assistant ：openmmlabwx. Please add "mmsig + Github ID" as a remark when adding friends：）
Thank you again for your contribution❤


Change event not fired on Thumbnail clic

LO,
I've found that no event is fired when clicking on a thumbnail to navigate !
this.fireChangeEvent(); missing in the select function ?
Vincent.

Agree!


[BUG] Cast error on GET request with body params

Information

Version: "5.44.17

I have this really, really weird error with a get request. I have this code here:
  @Get('/:form')
  get(@PathParams('form') form: string) {
    return 'hi';
  }

And whenever I make the request, I get this error: Cast error. Expression value is not a number.. Other GET requests with path params work fine, and very weirdly this doesn't error:
  @Get('/:form/a')
  get(@PathParams('form') form: string) {
    return 'hi';
  }

I've been struggling with this for hours now so if anyone can help that would be amazing.

Hello @robertwestbury
Unfortunately, I haven't reproduced the bug. Maybe an access to your repo can help me (or a small repo with the reproducible error).
See you
Romain

I figured it out - it's because there was another route above it that had the same "route" without the wildcard (e.g. /hello and /:name can both be matched). Maybe load the functions in the order they're defined in the class like nestjs does?

Nevermind - it's not.

I have not been able to produce a reproducible example, but I would be happy to show you my code or hop into a call if that would be helpful. I have been stuck on this for days now and I'm getting pretty frustrated. @Romakita

@robertwestbury Yes. You can send me an invitation to your repository and review the code. You can add also a comment to figure out the code where the problem is ;)
A call can be planed next week if it's better for you, but an access to your repo will be enough to understand your  problem and propose a fix if it's necessary :)
Romain

I will not be able to share the code, but if we can call that would be perfect

I hope I can help: Look at this example that works perfectly for me.
import {Controller, Get, PathParams, ReturnType} from '@tsed/common'
import { UserRepository } from '@src/Repository/UserRepository ';
import { NotFound } from 'ts-httpexceptions'
import {UserDTO} from "@src/DTO/UserDTO"

@Controller('/users')
export class UserController {

    constructor(private userepository: UserRepository ) {
    }

    @Get('/:id(\\d+)')
    @ReturnType({type: UserDTO})
    async check (@PathParams('id') id: string): Promise<UserDTO> {
        const user = await this.userRepository.findOne(id);
        if (!user) {
            throw new NotFound('Not user found')
        }

        return user;
    }
    @Get('/hello$')
    hello (): string {
        return 'Hi! TS.ED';
    }
    @Get('/:id/form')
     form (@PathParams('id') id: string): string {
       return id
    }

}

https://expressjs.com/en/guide/routing.html
https://tsed.io/docs/controllers.html#routes-order

@robertwestbury When are you available?


WIP: Speed up events fetching

This PR is POC of an improve in EventFetcher that can significantly reduce Beamer transaction latency.
What's new:

After EventMonitor did the sync, subscribe for newHeads in beamer contracts in EventMonitor
Just after subscription was established, call self._web3.eth.get_block("latest") to catch the events that could had been created during creating subscription

TODO:

[ ] Catch IO errors during ws communication
[ ] Fallback to self._web3.eth.get_block("latest")
[ ] Compare block datatypes returned by self._web3.eth.get_block("latest") and eth_subscribe (for instance web3.py returns block number as int, while eth_subscribe returns as hex str  block_data["number"] = int(block_data["number"], 16)    # "0x9" -> 9)
[ ] Test coverage (in the test run a websocket server in a thread and make agent to communicate to it).

The intention was to minimise the number of changes in existing codebase and to only plugin into def fetch() and speed it up.

This PR is missing a test for the added functionality. We should test and make sure that the WebSockets connection is working as expected. I am not yet sure how the test should look like -- any suggestions?

True. Added 4th item:
[ ] Test coverage (in the test run a websocket server in a thread and make agent to communicate to it).

Let's archive this for now.


[Feature request] Create widget(s)

Hi @Oliv4945,
I was wondering if you plan to create some widget to start voice recognition without starting the app.
For example, Domo Widget let user start voice recognition and get voice+text result outside of an app:

Good idea @wikijm, I might do that in the future. It is added to long term todo list
By the way the option "Start app with voice recognition" is very close as you have only one icon to tap, but at the end you have the application opened.


The download link on the site becomes not working

The download link on the site has started to links to 404 error page:
http://www.wavesurfer.fm/build/wavesurfer.min.js

Added a PR to fix this @katspaugh

Thanks, merged!


Feature request: asm highlight is needed

Description
[Description of the feature]
A highlight of asm is urgely needed!!!

Code highlight is not done by theme. Is is done by Jekyll, with Rouge. Try type nasm instead of asm as a workaround.

Code highlight is not done by theme. Is is done by Jekyll, with Rouge. Try type nasm instead of asm as a workaround.

Yes, you are right. That works. Thank you.


Issue with trusted process

Receiving this : This process is not trusted! Input event monitoring will not be possible until it is added to accessibility clients.

Yes, I added a note about this to the README