Assets Missing

One of the required assets, "Race Tracks" is no longer available on the Unity Assets Store. Due to this Unity is unable to build the project.

Thank you very much for your interest in the Cybersickness Evaluation Testbed VR (CET-VR) repository and for bringing this issue to our attention. I apologize for the inconvenience you encountered with the unavailable "Race Tracks" asset.
Unfortunately, the Unity EULA does not allow us to publish Asset Store packages, even if they are free, as part of our open-source project. To address this issue in the long term, we are planning to replicate the asset with custom content, which will allow us to include it in the repository.
In the meantime, you could try to reach out to the original asset creator, Abdul Rahim (which e-mail address is still available through WaybackMachine: https://web.archive.org/web/20210730115143/https://assetstore.unity.com/publishers/23148, and ask if he can send you the asset package directly outside of the Asset Store.
Please note that a similar issue might arise with the Viking Village asset, as Unity recently replaced the previous version used in CET-VR with a new URP version. This could cause some issues because Unity may have changed the structure of the components, and the rest of the testbed is built for the Built-In Render Pipeline (whereas the new Viking Village is URP-only). Please let me know if you encounter these issues.


Using mocks on methods with refinements fails

In a situation when a method comes from an included module and then this same method is refined, invoking expect().to receive fails as if the method is not present at all.
After spending some time in debugger I realised that the expectance itself passes, but the restore_original_visibility fails, because no method was found to change visibility for
Reproduction self-executing snippet. Note that if I move inner method directly into Parent instead of include Inc the example passes without failures
#!/usr/bin/env ruby

module Inc
  def inner
    'inc'
  end
end

class Parent
  include Inc
end

module Ref
  refine Parent do
    def inner
      super + ' refined'
    end
  end
end

class Child < Parent
  using Ref

  def outer
    inner
  end
end

require 'rspec/autorun'

describe Child do
  describe '#outer' do
    it 'calls inner' do
      child = Child.new
      expect(child).to receive(:inner)
      child.outer
    end
  end
end

@dreyks your example works for me on RSpec master on both 2.4 and 2.2. However, I will note that refinements do not work with any_instance. As per @JonRowe's PR.

If the repro doesn’t fail anymore, is it time to close this?

hmm... I've just rerun the example on both ruby 2.4.1 and 2.2.6 with rspec@master (3.8.pre) and it fails with
NameError:
       undefined method `inner' for class `#<Class:#<Child:0x007ffd1785b670>>'


release: update opae release branch targets

Update the following opae dependencies for opae-sdk release/2.0.1-1 so that said dependencies point to a snapshot release branch/tag, and not to continue tracking master.

opae-libs

Subtree has been updated to handle opae-test and opae-sim

opae-legacy

This commit should not be pulled back into master, as the master branch should naturally track the latest HEAD of each opae dependency. Rather, this commit will be the tag target for release 2.0.1-1.

Closing - the git history diff between the two branches seems off (the version change should already be in release/2.0.1-1).
Will open new PR shortly.


Add custom index type prefix to error messages

Related issues

Fixes #171.
Description

Overloaded ParserUtil::parseIndex to take in an additional indexType argument that will indicate the type of index that was parsed should a ParseException be thrown.
// The following will throw a ParseException with the following message:
// "Module index is not a non-zero unsigned integer."
parseIndex("0", "Module");

// The following will throw a ParseException with the following message:
// "Index is not a non-zero unsigned integer."
parseIndex("0");

// The following will throw a ParseException with the following message:
// "Index is not a non-zero unsigned integer."
parseIndex("0", "");

How to test

Start LinkyTime and enter the following command: edit 2 m/.
You should now see the following:

Other information

Codecov Report

Merging #204 (a12724a) into master (ea8bf7a) will increase coverage by 0.10%.
The diff coverage is 100.00%.

@@             Coverage Diff              @@
##             master     #204      +/-   ##
============================================
+ Coverage     71.61%   71.72%   +0.10%
- Complexity      560      564       +4
============================================
  Files            94       94
  Lines          1878     1885       +7
  Branches        202      203       +1
============================================
+ Hits           1345     1352       +7
  Misses          447      447
  Partials         86       86

Impacted Files
Coverage Δ

...in/java/seedu/address/logic/parser/ParserUtil.java
98.11% <100.00%> (+0.28%)
:arrow_up:

...logic/parser/meeting/EditMeetingCommandParser.java
94.87% <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 ea8bf7a...a12724a. Read the comment docs.

LGTM. Do you think we should extend this to AddMeetingCommandParser as well?

I don't mind, but would that count as a "feature"? We were able to fix this as it was classified as a bug.


Improve context.createFile() API

There isn't really a need to have both id and path parameters. Every plugin produces only one file, so id could be inferred. If a plugin needs to reference another plugin's output file, it should be able to import ID (output path) from that plugin. This change does not consider what would happen if a plugin creates multiple files as there isn't such use case at the moment.

Related https://github.com/hey-api/openapi-ts/issues/1359


[$1000] Fix dev warnings after RN update

Problem
With the RN update, there is a few warnings in dev which show up but they dont influence the app behaviour so it was not a blocker.
Why
However, we need to fix them in order to ensure developers can see if somethignn is actually broken/ new warnings introduced
Solution
Fix the warnings, more context here https://github.com/Expensify/App/pull/15130#issuecomment-1497776229
cc @aimane-chnaif
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~013bb4e758444068c8
Upwork Job ID:<PHONE_NUMBER>635106816
Last Price Increase: 2023-04-25

Triggered auto assignment to @strepanier03 (Bug), see https://stackoverflow.com/c/expensify/questions/14418 for more details.

Bug0 Triage Checklist (Main S/O)

[ ] This "bug" occurs on a supported platform (ensure Platforms in OP are ✅)
[ ] This bug is not a duplicate report (check E/App issues and #expensify-bugs)

If it is, comment with a link to the original report, close the issue and add any novel details to the original issue instead

[ ] This bug is reproducible using the reproduction steps in the OP. S/O

If the reproduction steps are clear and you're unable to reproduce the bug, check with the reporter and QA first, then close the issue.
If the reproduction steps aren't clear and you determine the correct steps, please update the OP.

[ ] This issue is filled out as thoroughly and clearly as possible

Pay special attention to the title, results, platforms where the bug occurs, and if the bug happens on staging/production.

[ ] I have reviewed and subscribed to the linked Slack conversation to ensure Slack/Github stay in sync

@mountiny - Does this need the regular BZ process? I don't want to mess up whatever is already moving here but I'm not sure what steps I should take next. Thank you!

@strepanier03 haha no all good, @aimane-chnaif will handle these and once we are all done, we will just need to issue a payment

@aimane-chnaif I don't see many errors in the LogBox ignore right now. Do you see any change these would go away with some React-native-web update? If that is the case then adding them to the LogBox ignore list makes sense but if its permanent then I would prefer fixing this with platform specific files.

@aimane-chnaif that would be ideal, would you be able to do that?

👍 Sounds great @mountiny - I'll follow along and handle payment when the time comes.

Making this weekly before we get more clarity on the RNW question

Any update, have you go ta chance to create an issue in the repo @aimane-chnaif ?

@strepanier03 @mountiny @aimane-chnaif this issue was created 2 weeks ago. Are we close to a solution? Let's make sure we're treating this as a top priority. Don't hesitate to create a thread in #expensify-open-source to align faster in real time. Thanks!

@aimane-chnaif how is this looking, I will make this Daily and make it External but you are assigned as a contributor. Let me know if you would rather let this being worked on by someone else and be a C+, not sure whats your workload right now.

Job added to Upwork: https://www.upwork.com/jobs/~013bb4e758444068c8

Current assignee @strepanier03 is eligible for the External assigner, not assigning anyone new.

Current assignee @aimane-chnaif is eligible for the External assigner, not assigning anyone new.

Current assignee @mountiny is eligible for the External assigner, not assigning anyone new.

@aimane-chnaif @mountiny if we are looking for a contributor to work on the warnings. I would be happy to volunteer for the task.
I have already worked on the RN migration and some of the warnings in the discussion i have pointed out myself as part of testing.

I will wait for Aimane, previously we agree they would work on it so they have first dibs, but if they want to off load this, thats completely fine by me.

@aimane-chnaif were there no other RN warnings? I thought there was more than just one. Or are they all coming from the RN and RNW?

Upwork job price has been updated to $250

@aimane-chnaif ok, thanks, did you manage to start the issue in RNW already?
To speed up things, I propose to create issue in RNW and RN or wherever we need to and if you want to also create a PR if its going to be simple. Add the warning to the LogBox ignore you have mentioned until this is fixed and once fixed we can remove it.
Once we create the issues upstream we can close this and @aimane-chnaif you could then bump this issue once fixed upstream so we can remove the warnings from logBox ignore after we update the RNW version which includes the fix

@abdulrahuman5196 what do you think?

📣 @abdulrahuman5196 You have been assigned to this job by @mountiny!
Please apply to this job in Upwork and leave a comment on the Github issue letting us know when we can expect a PR to be ready for review 🧑💻
Keep in mind: Code of Conduct | Contributing 📖

Assigned you, lets first create the RNW issue baed on their standards and then please make a PR to add the warnings to the LogBox ignore.

@mountiny I created the PR and a issue at RNW as well
PR - https://github.com/Expensify/App/pull/17908
RNW issue - https://github.com/necolas/react-native-web/issues/2514
Response from RNW

Those warnings are coming from React Native. If you're using the latest version of RN, you can pass userSelect styles to Text. I don't think cursor is supported, and you can use Platform.select to pass specific props or styles to the element only for certain platforms.

https://github.com/necolas/react-native-web/issues/2514#issuecomment-1520709980
The recommendation from RNW is to use platform specific code on this.

If required we can think of created platform specific code for this case.
Anyways I created a PR to add the warnings to logbox ignore logs list - https://github.com/Expensify/App/pull/17908
Note: Logbox ignoreLogs will only hide the yellow warnings on UI but would still print the warnings in console.

@abdulrahuman5196 by latest he meant 0.72?

Do you think the userSelect would work for our usecase?

I think maybe we are using WebkitUserSelect duplicate for safari specifically.

Given that RNW won't handle this issue and RN became more strict on style props, LogBox doesn't seem like permanent solution. Maybe better to define platform specific files for those styles and update all usages throughout the app.

Yeah. agree on long term.

Alright lets do that then, I will increase the job reward to 500 to update this everywhere.

Upwork job price has been updated to $500

I am proposing to create a separate files for cursor and userselect similar to what we use for wordBreak - https://github.com/Expensify/App/tree/acb4cc7d4a5fd78aa49b7b90f419b387e757a774/src/styles/utilities/wordBreak
There are other examples in the same parent folder.
And we can change usages of cusor and userSelect to use the new styles throughout the app. Let me know if you want any different way?
@mountiny
And on side note: IMO, with the new development, we should consider this a normal bug and revert to the original compensation this bug was created with. Since we would have to do code change, refractor and testing in different devices to make sure it doesn't break anything, compared to previous requirement of just updating logbox. This is just my opinion, but i will adhere to any decision team might think is best for this issue.

I am going to be OoO from April 26-May 3. I plan to leave this GH assigned to me during my out-of-office as it likely won't need my action before I return.
If something changes and action is needed from the BugZero team, please reach out in #expensify-open-source to request help from another team member.
Thanks, chat with you when I get back!

And we can change usages of cusor and userSelect to use the new styles throughout the app. Let me know if you want any different way?

Can you detail what difference this would be please?
And I am happy to make this normal bug if we are doing bigger refactoring

📣 @aimane-chnaif You have been assigned to this job by @mountiny!
Please apply to this job in Upwork and leave a comment on the Github issue letting us know when we can expect a PR to be ready for review 🧑💻
Keep in mind: Code of Conduct | Contributing 📖

Upwork job price has been updated to $1000

And we can change usages of cusor and userSelect to use the new styles throughout the app. Let me know if you want any different way?

Can you detail what difference this would be please?

@mountiny  Sure. Let me reference it with a existing implementation in current codebase - whiteSpace style

whiteSpace style is defined in platform specific file
https://github.com/Expensify/App/blob/5a0c876fabbccf8279eb5483c92997f565633267/src/styles/utilities/whiteSpace/index.js#L1-L11

https://github.com/Expensify/App/blob/5a0c876fabbccf8279eb5483c92997f565633267/src/styles/utilities/whiteSpace/index.native.js#L1-L4
Here native platform file doesn't have any style inside the declaration like empty 'noWrap' declaration. So the property itself will not be set in native.

This is added to the main styles.js file so that components can access it as 'styles.noWrap'
https://github.com/Expensify/App/blob/5a0c876fabbccf8279eb5483c92997f565633267/src/styles/styles.js#L157

And for compound styles with multiple properties present in StyleUtils.js and Styles.js we have used spread operator to use the whitespace property
https://github.com/Expensify/App/blob/5a0c876fabbccf8279eb5483c92997f565633267/src/styles/styles.js#L324-L330

We will do similar implementation for 'cursor' and 'WebkitUserSelect' and refractor accordingly in styles classes. And In most cases we won't be required to update directly in a UI component class but we can double confirm during implementation and testing.

Sounds great, thanks for taking time to explain that 🙇

@aimane-chnaif  I have completed the PR with the latest requirement - https://github.com/Expensify/App/pull/17908
Kindly have a look into it

@aimane-chnaif  And we just have to add video for android and ios right? Since this issue will only happen on that device?

add other platforms (web, desktop) as well with console log window as it's not a big deal

Sure will do. But it won't have any difference though.
Just a helping note on review - Example page where the yellow warnings used to show up.
cursor - Reimbursement page
WebkitUserSelect - chat page with any already made reaction.

Reviewing label has been removed, please complete the "BugZero Checklist".

The solution for this issue has been :rocket: deployed to production :rocket: in version 1.3.7-3 and is now subject to a 7-day regression period :calendar:. Here is the list of pull requests that resolve this issue:

https://github.com/Expensify/App/pull/17908

If no regressions arise, payment will be issued on 2023-05-05. :confetti_ball:
After the hold period is over and BZ checklist items are completed, please complete any of the applicable payments for this issue, and check them off once done.

[ ] External issue reporter
[ ] Contributor that fixed the issue
[ ] Contributor+ that helped on the issue and/or PR

As a reminder, here are the bonuses/penalties that should be applied for any External issue:

Merged PR within 3 business days of assignment - 50% bonus
Merged PR more than 9 business days after assignment - 50% penalty

BugZero Checklist: The PR fixing this issue has been merged! The following checklist (instructions) will need to be completed before the issue can be closed:

[ ] [@aimane-chnaif] The PR that introduced the bug has been identified. Link to the PR:
[ ] [@aimane-chnaif] The offending PR has been commented on, pointing out the bug it caused and why, so the author and reviewers can learn from the mistake. Link to comment:
[ ] [@aimane-chnaif] A discussion in #expensify-bugs has been started about whether any other steps should be taken (e.g. updating the PR review checklist) in order to catch this type of bug sooner. Link to discussion:
[ ] [@aimane-chnaif] Determine if we should create a regression test for this bug.
[ ] [@aimane-chnaif] If we decide to create a regression test for the bug, please propose the regression test steps to ensure the same bug will not reach production again.
[ ] [@strepanier03] Link the GH issue for creating/updating the regression test once above steps have been agreed upon:

I think no action is required here, just dv warning after RN update

@aimane-chnaif and @abdulrahuman5196 - I have hired you both in Upwork for this job, please accept when you can and I'll pay out tomorrow when this comes off hold.

Accepted the invite @strepanier03 .

@aimane-chnaif and @abdulrahuman5196 - I've completed the checklist and paid you both via Upwork.
I'm going to close this now, thank you both for your hard work and @mountiny for helping to keep this moving forward.


update Go module and docker image to latest (1.17)

Is your feature request related to a problem? Please describe.
Our scanners are picking up some high and critical vulnerabilities in Go 1.15
Critical
CVE-2022-23806
CVE-2021-38297
High
CVE-2021-39293
CVE-2022-23772
CVE-2022-23773
CVE-2021-29923
Describe the solution you'd like
update Go module and docker image to latest (1.17)
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
Add any other context or screenshots about the feature request here.
┆Issue is synchronized with this Jira Task by Unito
┆friendlyId: K8SSAND-1268
┆priority: Medium

Thank you sir

@jsanda i see you are a contributor to reaper-operator as well and did the last upgrade to Go 1.15, can that be updated as well or has reaper-operator been migrated as well?
Please let me know

See https://github.com/k8ssandra/k8ssandra-operator/issues/310#issuecomment-1048973982.

@jsanda Thank you for the prompt responses.


Implement mention_text, contact-bot, and improve typing configuration.

This PR includes feature implementations, codebase enhancements (type annotations), and a code example.

Fixes #178 by implementing mention_text
Configure mypy to ignore missing imports for untyped modules instead of using type: ignore directives
Enforce type hints for functions by adding the disallow_untyped_defs option
Add contact-bot from the typescript project, which can be used in the documentation

Possible follow-ups in later PRs:

Integrate unit tests in the project
Finish up docs/references/contact.md
Possibly add utilities like isort and Black for codebase formatting
The version bumping hook is quite annonying, as it would repeatedly bump versions even if it has done it in the previous commit.

 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.

Wechaty Contributors Nomination
Link to https://github.com/wechaty/PMC/issues/16


New version: Kronecker v0.4.2

Registering package: Kronecker
Repository: https://github.com/MichielStock/Kronecker.jl
Created by: @MichielStock
Version: v0.4.2
Commit: f93f7cedebf61b62af04278cfba571925a0d06da
Reviewed by: @MichielStock
Reference: https://github.com/MichielStock/Kronecker.jl/commit/f93f7cedebf61b62af04278cfba571925a0d06da#commitcomment-36316835

wait, will fix them


Use sphinx-design to improve home page layout

Now that the Hugo site is history, we should improve the appearance of the Training home page using sphinx-design to emphasize the current and most important trainings.

Hey , I wanted to contribute in this , I have read the 'Contributing to Plone Training' and signed the agreement , can you please explain the issue to me.
Thanks

@Ajeett01 actually this issue is obsolete, since we archived trainings from 2022 and older. Closing.

@Ajeett01 actually this issue is obsolete, since we archived trainings from 2022 and older. Closing.

Ok , can you please tell any issue I can work on.

No. That's your job. See Item 1 in https://6.docs.plone.org/contributing/first-time.html#work-with-github-issues


KRI Report generation

Overview

Get site and country report running.
Update examples
Finalize various features.

Fixes #1646 #1637 #1592 #1457
Fixes
Test Notes/Sample Code
Known issues:

Need to look at Report_* functions to align on dfSite/dfGroups usage.
SiteSelect overall dropdown is not populating with sites
StudyID is not being passed through to report (currently looking for it in dfStudy)
Study Overview table column labels are off (see below)

Ran the following to generate a report for KRI0001:
devtools::load_all()
# Load raw data and workflows
lRaw <- list(
    dfSUBJ = clindata::rawplus_dm,
    dfAE = clindata::rawplus_ae,
    dfPD = clindata::ctms_protdev,
    dfLB = clindata::rawplus_lb,
    dfSTUDCOMP = clindata::rawplus_studcomp,
    dfSDRGCOMP = clindata::rawplus_sdrgcomp %>% dplyr::filter(.data$phase == 'Blinded Study Drug Completion'),
    dfDATACHG = clindata::edc_data_points,
    dfDATAENT = clindata::edc_data_pages,
    dfQUERY = clindata::edc_queries,
    dfENROLL = clindata::rawplus_enroll
)

wf_mapping <- MakeWorkflowList(strNames = "mapping")
wf_kri <- MakeWorkflowList(strNames="kri0001")
wf_reporting <- MakeWorkflowList(strNames = "reporting")

# Generate Mapped Data
lMapped <- RunWorkflows(lWorkflow = wf_mapping, lData = lRaw)

# Generate Analysis Results Data
lAnalysis <- RunWorkflows(lWorkflow = wf_kri, lData = lMapped)

# Generate Reporting Data
lReporting_Input <- list(
    ctms_site = clindata::ctms_site,
    ctms_study = clindata::ctms_study,
    dfEnrolled =lMapped$dfEnrolled,
    lWorkflows = wf_kri,
    lAnalysis = list(kri0001 = lAnalysis),
    dSnapshotDate = Sys.Date(),
    strStudyID = "ABC-123"
)

lReporting <- RunWorkflows(lWorkflow = wf_reporting, lData = lReporting_Input)

# Convience Mappings
dfGroups <- lReporting$dfGroups
dfMetrics <- lReporting$dfMetrics
dfSummary <- lReporting$dfSummary
dfBounds <- lReporting$dfBounds

# Create dfSites and dfStudy pending rbm-viz update to use dfGroups
dfSite <- dfSites <- dfGroups %>%
  filter(GroupLevel == "Site") %>%
  pivot_wider(names_from=Param, values_from=Value) %>%
  rename(
    SiteID = GroupID,
    status = Status,
    enrolled_participants = ParticipantCount
  )

dfStudy <- dfStudies <- dfGroups %>% filter(GroupLevel == "Study") %>% pivot_wider(names_from=Param, values_from=Value)

# Lazy longitudinal data
dfSummary_long <- bind_rows(
  dfSummary %>% mutate(SnapshotDate = Sys.Date()),
  dfSummary %>% mutate(SnapshotDate = Sys.Date() - 1),
  dfSummary %>% mutate(SnapshotDate = Sys.Date() - 14)
)

#Create Charts for all metrics
options(vsc.viewer = FALSE)
metrics<- unique(dfMetrics$MetricID)
charts <- metrics %>% map(~Visualize_Metric(
  dfSummary = dfSummary_long,
  dfBounds = dfBounds,
  dfGroups = dfGroups,
  dfMetrics = dfMetrics,
  strMetricID = .x
)
) %>% setNames(metrics)

Report_KRI(lCharts = charts,
 dfSummary = dfSummary,
dfStudy = dfStudy,
dfSite = dfSite,
dfMetrics = dfMetrics,
strOutpath = "~/test.html")

Notes:

To do

[ ] dfSummary -> dfResults in widget params
[ ] Update dfGroupLevel to use unique(dfMetrics$GroupLevel) by default in Widget_GroupOverview

Site-level report is working for a single snapshot via inst/examples/snapshot.R. Remaining to do's below. Will get this ready to review/merge Monday, and we can file any needed issues.
To do

[x] https://github.com/Gilead-BioStats/gsm/issues/1646
[ ] https://github.com/Gilead-BioStats/gsm/issues/1637
[x] https://github.com/Gilead-BioStats/gsm/issues/1592
[x] https://github.com/Gilead-BioStats/gsm/issues/1457
[ ] dfSummary -> dfResults in widget params
[ ] Update dfGroupLevel to use unique(dfMetrics$GroupLevel) by default in Widget_GroupOverview
[ ] Fully remove DT requirement?
[ ] Update/streamline/de-duplicate all inst/example
[ ] Make test scripts/debug country-level and longitudinal reports
[ ] Merge dev in to branch
[ ] Merge in #1570
[ ] Fix (or maybe remove/refactor) global site filter
[ ] Update examples for all widgets and report to use new sample data sets from #1629 .

@jwildfire I'm taking these three tasks from the list above this morning:

dfSummary -> dfResults in widget params
Update dfGroupLevel to use unique(dfMetrics$GroupLevel) by default in Widget_GroupOverview
Update examples for all widgets and report to use new sample data sets from https://github.com/Gilead-BioStats/gsm/pull/1629 .

@jonthegeek and @jwildfire it looks like sampleResults and sampleSummary have integer GroupID values, while they should be characters. this pops up when testing/making examples for Report_MetricTable(). going to fix this, unless I missed something at the end of last week. let me know!

@jonthegeek and @jwildfire it looks like sampleResults and sampleSummary have integer GroupID values, while they should be characters. this pops up when testing/making examples for Report_MetricTable(). going to fix this, unless I missed something at the end of last week. let me know!

Agree it makes more sense for them to be character.

Sounds good, @lauramaxwell. I'm planning on working on example updates, including country-level and longitudinal reports before scrum. I'll work in a separate branch to avoid conflicts.

They were auto-set to integer because the ones in the csv happen to all be integers. Definitely hard-code them to character in the construction script (and probably double-check all columns in all the sample data for similar issues)!

(and probably double-check all columns in all the sample data for similar issues)!

yep! already did this in the script for the GroupIDs, but found that the dates are also stripped of their date class, so i'm going to go in and edit that in the construction script as well.

Updated widgets to get inst/examples/snapshot.R running again and filed issues to clean up remaining issues/


Correct symbol for kibibyte

The symbol for kibi is "Ki" with a capital k. Since the International
System of Units restricts the use of SI prefixes strictly to powers of
10, the description of that line is updated to not mention SI in the
context.

ACK.  Thanks!
For anyone else who is confused as I am, k is the generic SI prefix for kilo but they decided to use K for kibi 'cause reasons, or something:

(For consistency with the other prefixes for binary multiples, the symbol Ki is used for 2^10 rather than ki.)

source

I grepped the repository and could not find a single instance of kiB beyond the one in STYLE.md.

Thanks @Xekyo. TIL!

For anyone else who is confused as I am, k is the generic SI prefix for kilo but they decided to use K for kibi 'cause reasons, or something:

I think the general pattern for metric prefixes was supposed to be majuscules for prefixes denominating greater powers than 1000^0 (k, M, G, T, P, E, Z, Y) and minuscules for prefixes denominating fractions (m, μ, n, p, f, a, z, y). From what I gather, kilo got the symbol "k" because the "K" was already the symbol for Kelvin, one of the seven base units. That collision doesn't exist in that form for kibi (Ki), so they could stay consistent. :)


Discrepancy between the numbers shown in the dashboard categories for certain collections

The numbers for Review, Working, Unscrapeable do not total that in All.

Missing sources:

http://www.accesshollywood.com/#spider
http://www.americaru.com/
http://axs.com/
http://www.bizjournals.com
http://blogs.wsj.com
http://money.cnn.com/
http://www.contactomagazine.com
http://www.c-span.org
http://foxbusiness.com/
http://www.hollywoodreporter.com
http://hosted.ap.org
http://wire.kapitall.com
http://www.meridianul.com/
http://univision.com/
http://www.weeklystandard.com

@titinfoundation -- can you provide a few example collections for this?

Hi @dsjen I have more examples of other collections where the same thing is happening.

https://sources.mediacloud.org/#/collections/34412372/manage-source-list

https://sources.mediacloud.org/#/collections/34412126/manage-source-list

https://sources.mediacloud.org/#/collections/34412409/manage-source-list

oh I see. I was just wandering what happened with the one that are in neither of the categories. Do I assume there working?

Thanks! Yes, please do let us know when you have an example of a missing source from a collection! It'll help us in figuring out what's up.

@dsjen what I mean with this is that sources that appear here: https://sources.mediacloud.org/#/collections/34412372
don't match the detailed segmentation in here: https://sources.mediacloud.org/#/collections/34412372/manage-source-list
13 missing sources seem to be missing from the Working segment in the Review Sources segment, maybe it doesn't matter and I should assume they are there and working but I just saw the difference and found it odd.  I cant' tell you which ones exactly are missing form the Working segment because everytime I try to download a csv I get hit with:

Ok, thanks! I'm not positive, but in my diagraming, I agree that the numbers don't seem to be matching up.

I'm almost positive that this will be fixed when https://github.com/mediacloud/backend/issues/726 is complete and am closing this issue (we can re-open if it's not though).


Support custom metrics in the timeout middleware

This is a Feature Proposal
Description

Need a way to also track the occurrence of timeouts as a metric. Ideally a way to plug in datadog or newrelic clients to track custom metrics

cc @onlyplo

As I mentioned on the other thread, the simplest solution I can think of is to let you pass in a callback function, but let's carry on the discussed on the other thread.

Yo, the initial work i have done here: https://github.com/getndazn/dazn-lambda-powertools/commit/d4f873b9e705ceb5f415b57be1fa2c4fe37a2e8c already starts to extend the middleware with a custom callback function.
Will probably need to change the naming and some logic based on our discussions to split off custom logging and custom metrics into separate issues.
Happy for me to submit a PR for this?

Sure, go for it!

@onlyplo any update on this?


[Bug]:  Footnote disappearing when hovering

Give a brief about the bug ✍
When we hover over the footer of the main page, the Foot note "2024 Made by ❤" is disappearing.
What is the expected behavior? 🤔
It should always be visible.
Provide step by step information reproduce the bug 📄
https://github.com/user-attachments/assets/c35bd2df-3cce-4e0b-97ad-42ee3e50d434
Select program in which you are contributing
GSSoC24
Code of Conduct

[X] I follow CONTRIBUTING GUIDELINE of this project.

@ayush-t02 please kindly have a look.


Uncouple from suit

Work is in progress to uncouple suit-dashboard from suit. A new name should then be used, something like django-admin-dashboard to keep things simple? Or something more personal?
Of course the display is way better with Suit but not everybody want to use it. The main issue is that we can't extend the admin index.html since admin or simple admin app in settings' apps will always result in "you don't have any permissions to edit" message.
So we have to implement a special html base that display again the list of applications.

Displaying the list of applications is left to the user. Uncoupling done (handle breadcrumbs for Suit and classic styles, add context variable suit)


Issue with SAKT model

I noticed a weird way of invoking the TranformerLayers in SAKT model as per the below code snippet, you are only getting the output of the last layer applied on the seq_data input, you should either combine the pool or add y as input per each call!
 y = seq_data         for block in self.attn_blocks:             y = block(mask=1, query=q_data, key=seq_data, values=seq_data)

Thanks for pointing out this issue! It is indeed a mistake and has been fixed in the latest commit. Meanwhile, considering that the original paper only uses a single transformer layer, our previous implementation just works the same. So the experimental results are still reliable.


RFC: change how DraftEntity stores internal

Summary
This issue is a Request For Comments (RFC) to propose a change to the way the DraftEntity stores its internal data structure.
In order to better support concurrent updates to Draft editor instances, we propose to modify DraftEntity to use concurrency-safe keys.
How DraftEntity works
DraftEntity, when creating a new entity uses incremental numbers as the key (source) which can be problematic if the map is updated concurrently in two different locations.
Proposed changes

We propose to change how DraftEntity generates keys for new entities. Instead of incremental integers, we create a unique ID, e.g. using uuid(), and use that as the ID.
We also propose to expose the getAll and loadWith methods to get the map of entities to export them to other editor instances and initialize an editor instance with an existing map.

I see that this has already been committed to 'master', but I can't build it. See #2385.

Hello, it seems that DraftEntity has methods which add new entities to the global Map, but there is no way to remove (clean up) any entities which are no longer needed.
For our project, this becomes a memory leak for us, as we end up creating a lot of entities, but when the contents are no longer needed, we can't clean them up, meaning that the memory is still allocated.
Could you please provide a way to delete an entity from DraftEntity given the entity key?
Thanks,
Geoff Peters
Vancouver Canada


Crash when setting codec in createProducer

Hello,
I'm trying to set the video codec when creating the producer by passing a json in the codec parameter but it crashes systematically. It works perfectly well when the parameter is nil.
Any idea ? I try to understand the crash and it looks like some memory corruption that leads to a crash later in the matchCodecs method. Maybe something to do with concurrency ?
For info, this is the kind of json I'm using : "{
"kind" : "video",
"rtcpFeedback" : [
{
"type" : "nack",
"parameter" : ""
},
{
"type" : "nack",
"parameter" : "pli"
},
{
"type" : "ccm",
"parameter" : "fir"
},
{
"type" : "goog-remb",
"parameter" : ""
},
{
"type" : "transport-cc",
"parameter" : ""
}
],
"parameters" : {
"packetization-mode" : 1,
"level-asymmetry-allowed" : 1,
"profile-level-id" : "42e01f"
},
"clockRate" : 90000,
"preferredPayloadType" : 103,
"mimeType" : "video/H264"
}"

Fixed in version 0.4.1

Thanks , it works !! I'm able to select vp8 or h264 now. Unfortunately simulcast  still don't work  in H264 for me. Have you see it working on your side ?

Unfortunately simulcast still don't work in H264 for me. Have you see it working on your side ?
Yes, simulcast works for me on h264. But now I have similar problem. In case I select VP8 (a non-default) over h264 (default, listed on the first place in router capabilities) it works only without simulcast. If I create additional video layers, other clients do not receive any data and see no video. As I use only one layer, outgoing video works with VP8. But now I don't have time to dive deeper into this problem.

Thanks for your answer, I will make some test to see if changing the default change something.


Migrate issues to MongoEngine/marshmallow-mongoengine

Since the repo was transferred to MongoEngine/marshmallow-mongoengine, no "issues" tracker was created there.
I suppose issues should be transferred and this tracker should be closed.

Any news on this? Is this repo (or more exactly its forked, over at https://github.com/MongoEngine/marshmallow-mongoengine), still supported? If not: is there any recommended alternative?

is there any recommended alternative?

We use and co-maintain umongo, by the author of marshmallow-mongoengine, as an alternative to mongoengine + marshmallow-mongoengine. Not much activity over there either but it works. I didn't have the time to port it to marshmallow 3 yet.
If I was starting from scratch, I'd be tempted to look at pymodm and try to see if it is reasonable to build a marshmallow-pymodm library. It should be similar to marshmallow-mongoengine.


Workspace: Remove vscode/desktop from image

The big thing blocking this merge is going to be making sure suid-python correctly picks up the dependencies for various challenges (e.g. unicorn/asteval/flask). Probably testing against building-a-web-server and embroyasm will be sufficient to make sure I've set it up correctly.


application: Remove unused JavaScript file.

main.js stopped being used with commit 7514f134 ('Drop "Main document"
support'), so we can remove it and get rid of an unlicensed file.

lgtm


[Hub Generated] Review request for Microsoft.CostManagement to add version stable/2020-06-01

This is a PR generated at OpenAPI Hub. You can view your work branch via this link.
Contribution checklist:

[ ] I commit to follow the Breaking Change Policy of “no breaking changes
[ ] I have reviewed the documentation for the workflow.
[ ] Validation tools were run on swagger spec(s) and errors have all been fixed in this PR. How to fix?

If any further question about AME onboarding or validation tools, please view the FAQ.
ARM API Review Checklist

[ ] Ensure to check this box if one of the following scenarios meet updates in the PR, so that label “WaitForARMFeedback” will be added automatically to involve ARM API Review. Failure to comply may result in delays for manifest application. Note this does not apply to data plane APIs, all “removals” and “adding a new property” no more require ARM API review.

Adding new API(s)
Adding a new API version
Adding a new service

[ ] If you are blocked on ARM review and want to get the PR merged with urgency, please get the ARM oncall for reviews (RP Manifest Approvers team under Azure Resource Manager service) from IcM and reach out to them.

Breaking Change Review Checklist
If there are following updates in the PR, ensure to request an approval from API Review Board as defined in the Breaking Change Policy.

[ ] Removing API(s) in stable version
[ ] Removing properties in stable version
[ ] Removing API version(s) in stable version
[ ] Updating API in stable version with Breaking Change Validation errors
[ ] Updating API(s) in preview over 1 year

Please follow the link to find more details on PR review process.

/azp run automation - sdk

/azp run automation - sdk

/azp run automation - sdk

/azp run automation - sdk

Please go through the contribution checklist added as part of the review and mark them completed. This is a requirement before review can be continued.

Looks like the change is to split the file and no new API introduced/change. Removing ARMFeedback required tag.

/azp run automation - sdk

/azp run automation - sdk

/azp run automation - sdk


Can't access Administration Console page with keycloak behind Kong

NOTE: GitHub issues are reserved for bug reports only.
For anything else, please join the conversation
in Kong Nation https://discuss.konghq.com/c/kubernetes.

Summary
I have Keycloak behind Kong Ingress Controller.
I 'm able to see keycloak welcome page at my {url}/auth/. However, when I click at Administration Console I am redirected to {url}:8000/auth/admin/master/console/
Further information:
In the first moment I thought that the problem was with keycloak, but @Igor-lkm tested keycloak behind express-gateway and it's worked ok. So, looks like the problem is with kong.
@Igor-lkm made others test cases: here
Kubernetes version

kubectl:Client Version: version.Info{Major:"1", Minor:"14", GitVersion:"v1.14.1", GitCommit:"b7394102d6ef778017f2ca4046abbaa23b88c290", GitTreeState:"clean", BuildDate:"2019-04-08T17:11:31Z", GoVersion:"go1.12.1", Compiler:"gc", Platform:"linux/amd64"}
Server Version: version.Info{Major:"1", Minor:"11", GitVersion:"v1.11.9", GitCommit:"16236ce91790d4c75b79f6ce96841db1c843e7d2", GitTreeState:"clean", BuildDate:"2019-03-25T06:30:48Z", GoVersion:"go1.10.8", Compiler:"gc", Platform:"linux/amd64"}

Environment

Cloud provider or hardware configuration: AWS
Install tools: kubectl, helm
Others:

What happened
I have Keycloak behind Kong Ingress Controller.
I 'm able to see keycloak welcome page at my {url}/auth/. However, when I click at Administration Console I am redirected to {url}:8000/auth/admin/master/console/
Expected behvaior
When I click at Administration Console I should be redirect to {url}/auth/admin/master/console/
Steps To Reproduce

Create a kubernetes cluster with kops in AWS
Installing Kong Ingress Controller executing the following command:

helm install --name kong-develop --namespace develop stable/kong \
--set ingressController.enabled=true

Create an AWS Classic Load Balancer following the instructions bellow:

3.1 Basic Configuration

Load Balancer name: KongDevelopLB
Create LB Inside: Choose your cluster VPC

3.2 Listeners

Load Balancer Protocol
Load Balancer Port
Instance Protocol
Instance Port

TCP
80
TCP
kong-proxy NodePort service

TCP
443
TCP
kong-proxy TLS NodePort service

TCP
8444
TCP
kong-admin NodePort service

To help you detect the correct ports, take a look at the example image bellow:

In the example cluster shown in the image above:

Load Balancer port 80 should point to instance port 32518;
Load Balancer port 443 should point to instance port 31021;
Load Balancer port 8444 should point to instance port 31725.

3.3 Select Subnets

Select Subnets: Select the cluster available subnets

3.4 Security Group

Assign a security group: Create a new security group
Security group name: KongDevelopLB-SecurityGroup

Type
Protocol
Port Range
Source

Custom TCP Rule
TCP
80
Anywhere

Custom TCP Rule
TCP
443
Anywhere

Custom TCP Rule
TCP
8444
Anywhere

3.5 Configure Health Check
Health Check should point to kong-admin NodePort service. To help you find the correct port take a look at the example image bellow.
Based on the image bellow health check should point to 31725 port.

3.6 Add EC2 Instances
Add only the node instances.
3.8 Bind Load Balancer and Cluster
Edit the node cluster Inbound security group adding the following rules:

Type
Protocol
Port Range
Source
Description

Custom TCP Rule
TCP
kong-admin NodePort service
Custom - the KongDevelopLB security group id
kongDevelopLB

Custom TCP Rule
TCP
kong-proxy NodePort service
Custom - the KongDevelopLB security group id
kongDevelopLB

Custom TCP Rule
TCP
kong-proxy TLS NodePort service
Custom - the KongDevelopLB security group id
kongDevelopLB

Example:

Based on the image above:

kong-admin NodePort service is 31725.
kong-proxy NodePort service is 32518.
kong-proxy TLS NodePort service is 31021.

3.9 Create an alias to the Load Balancer
In Route53 service, create an alias record set inside inside cluster hosted zone to point to kong Load Balancer.
Example:
If your cluster DNS name is kubernetes.mydomain.com
you could create an alias record set like this: develop.kubernetes.mydomain.com

Installing keycloak behind Kong Ingress

4.1 Create a values.yaml file with the content bellow:
init:
 image:
  pullPolicy: Always

keycloak:
 username: admin
 password: admin
 readinessProbe:
  timeoutSeconds: 60
 service:
  type: NodePort
 persistence:
  deployPostgres: true
  dbVendor: postgres
 ingress:
  enabled: true
  hosts:
   - develop.kubernetes.mydomain.com
 extraEnv: |
   - name: PROXY_ADDRESS_FORWARDING
     value: "true"

4.2 Install keycloak using the values.yaml files craeted in step 4.1
Execute the code bellow:
helm install --name keycloak-develop --namespace develop codecentric/keycloak --values values.yaml

4.3 Access keycloak
Open the browser and access the following url: http://develop.kubernetes.mydomain.com/ and you gonna be redirected to http://develop.kubernetes.mydomain.com/auth/
4.4 Access the Administration Console
Click at the Administration Console link and you will be redirected to https://develop.kubernetes.mydomain.com:8000/auth/admin/master/console/

I had the same problem. But I did not use helm to deploy keycloak.
I had to set these variables in keycloak manifest to make the redirect work.
- name: PROXY_ADDRESS_FORWARDING
value: "true"
- name: KEYCLOAK_HOSTNAME
value: keycloak.example.com #your hostname
- name: KEYCLOAK_HTTP_PORT
value: "80"
- name: KEYCLOAK_HTTPS_PORT
value: "443"
hope it helps.

@jhonyscamacho Could you try the steps recommended by @gpguerra and close the issue if i fixes the problem for you?

I had the same problem. But I did not use helm to deploy keycloak.
I had to set these variables in keycloak manifest to make the redirect work.

name: PROXY_ADDRESS_FORWARDING
value: "true"
name: KEYCLOAK_HOSTNAME
value: keycloak.example.com #your hostname
name: KEYCLOAK_HTTP_PORT
value: "80"
name: KEYCLOAK_HTTPS_PORT
value: "443"

hope it helps.

@gpguerra thanks for the help!
I follow the steps and it didn't work. The problem persists.
I configure the variables with keycloak helm via keycloak.extraEnv parameter and with deployment file, using the keycloak docker image directly
In both cases the problem is the same: the estrange redirect to port 8000 when I click at the Administration Console link.
I'm still try to solve the problem. Any help is welcome.

Definitely it is a Ingress problem.
I created a keycloak deployment (using service LoadBalancer) based in the yaml file below and it worked, keycloak Admin Console was accessible and the 8000 redirect was gone.

keycloak:
 image:
  pullPolicy: Always
 username: admin
 password: admin
 readinessProbe:
  timeoutSeconds: 60
 service:
  type: LoadBalancer
 persistence:
  deployPostgres: true
  dbVendor: postgres
 extraEnv: |
  - name: KEYCLOAK_HTTP_PORT
    value: "80"
  - name: KEYCLOAK_HTTPS_PORT
    value: "443"

I'm still getting an strange HTTPS error but it's a work in progress.
I have no idea about what is causing the ingress problem/bu. Any help is welcome.

I spent some time to replicate this issue and found the following:
The ELB is configured with TCP protocol on both sides, meaning ELB terminates the TCP connection from client. Kong receives the connection and then injects X-Forwarded-Port header. Since Kong is listening on port 8000, it sets the header value to 8000.
Keycloak respects the header value and constructs the redirect URL accordingly.
So, there is nothing technically wrong here with Kong or Keycloak, in fact they are doing what exactly should be doing:
To fix the problem, you have two options:

Configure keycloak with explicit ports via env vars as you have done
Change Kong listeners to listen on 80 and 443 for HTTP and HTTPS respectively and then the injected X-Forwarded-Port will be correct

I spent some time to replicate this issue and found the following:
The ELB is configured with TCP protocol on both sides, meaning ELB terminates the TCP connection from client. Kong receives the connection and then injects X-Forwarded-Port header. Since Kong is listening on port 8000, it sets the header value to 8000.
Keycloak respects the header value and constructs the redirect URL accordingly.
So, there is nothing technically wrong here with Kong or Keycloak, in fact they are doing what exactly should be doing:
To fix the problem, you have two options:

Configure keycloak with explicit ports via env vars as you have done
Change Kong listeners to listen on 80 and 443 for HTTP and HTTPS respectively and then the injected X-Forwarded-Port will be correct

@hbagdi thanks for the help!
Could you please share both kong Ingress and keycloak configuration yaml files?

The first option, you have figured out the solution yourself. Add the following to the values.yaml for keycloak helm chart:
 extraEnv: |
  - name: KEYCLOAK_HTTP_PORT
    value: "80"
  - name: KEYCLOAK_HTTPS_PORT
    value: "443"

For the second, use the following values for the helm chart:
proxy:
  # HTTP plain-text traffic
  http:
    enabled: true
    servicePort: 80
    containerPort: 80
  tls:
    enabled: true
    servicePort: 443
    containerPort: 443
ingressController:
  enabled: true

Please close this issue if your issue is resolved, thanks!

The first option, you have figured out the solution yourself. Add the following to the values.yaml for keycloak helm chart:
 extraEnv: |
  - name: KEYCLOAK_HTTP_PORT
    value: "80"
  - name: KEYCLOAK_HTTPS_PORT
    value: "443"

For the second, use the following values for the helm chart:
proxy:
  # HTTP plain-text traffic
  http:
    enabled: true
    servicePort: 80
    containerPort: 80
  tls:
    enabled: true
    servicePort: 443
    containerPort: 443
ingressController:
  enabled: true

Please close this issue if your issue is resolved, thanks!

Thanks!
My kong Ingress configuration was missing the enabled and service parameters.
I will test it and give feedback asap!

The first option, you have figured out the solution yourself. Add the following to the values.yaml for keycloak helm chart:
 extraEnv: |
  - name: KEYCLOAK_HTTP_PORT
    value: "80"
  - name: KEYCLOAK_HTTPS_PORT
    value: "443"

For the second, use the following values for the helm chart:
proxy:
  # HTTP plain-text traffic
  http:
    enabled: true
    servicePort: 80
    containerPort: 80
  tls:
    enabled: true
    servicePort: 443
    containerPort: 443
ingressController:
  enabled: true

Please close this issue if your issue is resolved, thanks!

@hbagdi I tried the second option and it didn't work.
When I try to change Kong listeners to listen on 80 and 443 for HTTP and HTTPS respectively kong pod status get CrashLoopBackOff as shown in the image bellow:

For the second option to work, you will need to add cap_net_bind_service permission to the Kong's executable: https://github.com/Kong/docker-kong/blob/master/alpine/docker-entrypoint.sh#L31
This is intentionally left as is to not give Kong the ability to bind to privileged ports.

For the second option to work, you will need to add cap_net_bind_service permission to the Kong's executable: https://github.com/Kong/docker-kong/blob/master/alpine/docker-entrypoint.sh#L31
This is intentionally left as is to not give Kong the ability to bind to privileged ports.

Is there a way to configure it using helm chart?

You will have to create a new Docker image using Kong's docker image to give Kong that permission. There is nothing that can be done with Helm for this (and whatever is needed is already present in the Helm chart).
The first solution should suffice to solve the problem you're facing.

I will test and provide feedback

The first option, you have figured out the solution yourself. Add the following to the values.yaml for keycloak helm chart:
 extraEnv: |
  - name: KEYCLOAK_HTTP_PORT
    value: "80"
  - name: KEYCLOAK_HTTPS_PORT
    value: "443"

For the second, use the following values for the helm chart:
proxy:
  # HTTP plain-text traffic
  http:
    enabled: true
    servicePort: 80
    containerPort: 80
  tls:
    enabled: true
    servicePort: 443
    containerPort: 443
ingressController:
  enabled: true

Please close this issue if your issue is resolved, thanks!

@hbagdi I tried the second option and it didn't work.
When I try to change Kong listeners to listen on 80 and 443 for HTTP and HTTPS respectively kong pod status get CrashLoopBackOff as shown in the image bellow:

Although, kong ingress listen to default ports works correctly.

@hbagdi as you said, the first solution worked.
Thanks for the help!
@gpguerra thank you!

For the second option to work, you will need to add cap_net_bind_service permission to the Kong's executable: https://github.com/Kong/docker-kong/blob/master/alpine/docker-entrypoint.sh#L31
This is intentionally left as is to not give Kong the ability to bind to privileged ports.

Hello, i am having the exact same issue with my keycloak behing the kong ingress controler.
The first solution that add ExtraEnv varaible is not working wiht me and i am trying the second one, but i am not able to manualy change the proxy https redirection, how can i add cap_net_bind_service permission to the Kong's executable , as you said before ?

@jhonyscamacho Having the same issue also ^ but I binded the port on 80 with keycloak already and still having the error.

@jhonyscamacho is not working either 🤔

In fact I just had a second look and seems env variable KEYCLOAK_HOSTNAME is required. So this config would work:
  extraEnv: |
    - name: PROXY_ADDRESS_FORWARDING
      value: "true"
    - name: KEYCLOAK_HTTP_PORT
      value: "80"
    - name: KEYCLOAK_HTTPS_PORT
      value: "443"
    - name: KEYCLOAK_HOSTNAME
      value: example.com

And this one would not:
  extraEnv: |
    - name: KEYCLOAK_HTTP_PORT
      value: "80"
    - name: KEYCLOAK_HTTPS_PORT
      value: "443"

😲 🎉


Update irfanview.json

persist "Toolbars"

Files inside this directory will never get updated, because scoop doesn't support merging the new directory with the current one, yet.
The same goes for https://github.com/lukesampson/scoop-extras/issues/1766

@r15ch13 That’s true for current users, but new users will benefit, yes?

This will be the case on the next update for every user

@r15ch13 so you’re advising rejecting this PR as ineffective?

@rasa yes


client: RPC engine

Pull Request Test Coverage Report for Build 438935220

139 of 174 (79.89%)  changed or added relevant lines in 8 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.4%) to 35.489%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

telegram/rpc.go
2
4
50.0%

telegram/rpc/ack.go
25
27
92.59%

telegram/rpc/sender.go
0
7
0.0%

telegram/rpc/rpc.go
93
117
79.49%

Totals

Change from base Build 438797003:
0.4%

Covered Lines:
2486

Relevant Lines:
7005

💛  - Coveralls


Using task list page is outdated

Verified using: Visual Studio Code version 1.63.0
Referring to this page:  https://docs.microsoft.com/en-us/visualstudio/ide/using-the-task-list?view=vs-2022
Current GUI has changed for task list, it is not available under View Menu but under the Activity bar
Issue 01
Title:  The Task List window
Paragraph: "When Task List is open, it appears at the bottom of the application window.
To open Task List, select View > Task List, or from the keyboard press Ctrl+,T. "
Description:
Currently on this Studio Code version there is no View > Task list option anymore but available in the Activity Bar
keyboard shortcut does not work or not valid anymore  (https://docs.microsoft.com/en-us/visualstudio/ide/default-keyboard-shortcuts-in-visual-studio?view=vs-2022)
Issue 02
Title:  Custom tokens
Paragraph: "To create a custom token:
On the Tools menu, choose Options.
Open the Environment folder and then choose Task List. "
Description: Tool menu is not available hence could not find how to create a custom token
I don't believe that this is the only document page that should be updated - please update with the correct interface steps
Thank you

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

ID: 93b78b04-7dd5-453e-de60-8cdd16594a99
Version Independent ID: 27ddd786-e4c5-2b2e-e7c8-e103745dd656
Content: Task List, Environment, Options Dialog Box - Visual Studio (Windows)
Content Source: docs/ide/reference/task-list-environment-options-dialog-box.md
Product: visual-studio-windows
Technology: vs-ide-general
GitHub Login: @TerryGLee
Microsoft Alias: tglee

@angelica-serafini-cko The problem you describe is for Visual Studio Code (which includes a text editor), but the content you visited is for Visual Studio (which includes a fully integrated development environment). In other words, these are two separate products in the Visual Studio family.
However, here is a Visual Studio Code page that might help you with the problem you describe: https://code.visualstudio.com/Docs/editor/tasks.
We'll close this issue now; thanks for reaching out and we hope that the link helps you with the issue that you described.


Adds Google Optimize snippet

This is to enable Google Optimize.

Thanks! Is it possible to deploy it as well?

Thanks!


New component with multiple types of child components

I'm playing around with halogen's new components and got stuck where I want to install different types of child components into the same parent component.
Basically, I have three state and input types for the children, but the ParentComponent and InstalledComponent type aliases only allow me to specify one type of children state and input, respectively.
Is there a pattern for these kinds of situations that you had in mind?

This is what #166 is attempting to address :)

Ah I missed that PR :) Also thanks for the example in there!

I'm hoping to have it wrapped up and merged soon, the PR as it is does work but there are a few things I want to take another look at regarding the types variables used for peek and the like, as even though I wrote it I don't fully understand the implications of the composition setup when peeking is also desired :wink:

Ok, I'm curious to see how it will turn out! Really looking forward to migrating our project to the new components version.


Remove trailing whitespace in all index.html.md.erb files

Context
There are a lot of trailing whitespaces in index.html.md.erb files we use to render our docs. This isn't good practice even though it's not actually important as far as Middleman's rendering is concerned, because it may be awkward for some people navigating through the docs in future when making edits using Git.
Changes proposed in this pull request
Rids all docs files of whitespace.
Guidance to review
I've tested and built this locally and can't see any changes as far as the user is concerned. If I've missed something we can revert this commit.

I'm glad I remembered how to rebase 🤦♂


Hardcode Node API version (to version 8 at this time)

The Node API version specifies which Node API functions are available for use, and which node.js releases will be able to run the addon. Right now, our CMakeLists.txt file sets it to whatever the last-supported version in the currently-installed node.js is, which is a great way to accidentally change our minimum requirements in the future, without even needing whatever new functions are added.

I'll take care of this in my fix-windows-build branch


Lab Submission

Don't commit .DS_Store files! Also, you shouldn't be committing binary files --> instead commit the original source files.


Removed short content from blog cards

What is the change?
removed the short content visible on blog cards in home(/), category(/category), dashboard(/dashboard) and author(/author/:authorId) UI.
Correspondingly added posts partial as we are displaying the posts(blogcards) in same way in the above mentioned routes.
Related issue?
close: #447
Old pr: #462
How was it tested?
Tested the functioning and responsiveness thoroughly in localhost.
Checklist:
Before you create this PR, confirm all the requirements listed below by checking the checkboxes [x]:

[x] Have you followed the Contribution Guidelines while contributing.
[x] Have you checked there aren't other open Pull Requests for the same update/change?
[] Have you made corresponding changes to the documentation?
[x] Your submission doesn't break any existing feature.
[x] Have you tested the code before submission?

Screenshots or Video:

@A7abhilash resolve conflicts carefully, I think it is ready to merge

@A7abhilash resolve conflicts carefully, I think it is ready to merge

Yeah yeah🙌
I dont think there are conflicts... The master branch has the div of displaying posts but i had now kept it in partials right(you can also check this once)... So i think u can go on to merge

@A7abhilash resolve conflicts carefully, I think it is ready to merge

Yeah yeah🙌
I dont think there are conflicts... The master branch has the div of displaying posts but i had now kept it in partials right(you can also check this once)... So i think u can go on to merge

I know it's just a partial added for the post but the merge button doesn't appear untill you resolve them..

@A7abhilash resolve conflicts carefully, I think it is ready to merge

Yeah yeah🙌
I dont think there are conflicts... The master branch has the div of displaying posts but i had now kept it in partials right(you can also check this once)... So i think u can go on to merge

I know it's just a partial added for the post but the merge button doesn't appear untill you resolve them..

Umm... Then shall i make a new pr by resolving the conflict and then later in next pr ill add the partials on home ui also?

@A7abhilash resolve conflicts carefully, I think it is ready to merge

Yeah yeah🙌
I dont think there are conflicts... The master branch has the div of displaying posts but i had now kept it in partials right(you can also check this once)... So i think u can go on to merge

I know it's just a partial added for the post but the merge button doesn't appear untill you resolve them..

Umm... Then shall i make a new pr by resolving the conflict and then later in next pr ill add the partials on home ui also?

No, resolving doesn't require to make a new PR. Just click on resolve conflicts button and you can resolve them on github

resolved them

/gssoc/

/level2/


Support for concat 'demuxer' instead of concat filter?

Hi folks,
Thank you for creating ffmpeg-python! I'm running a very simple concat operator using ffmpeg-python like so:
def concatVideos(args):
files = []
streams =[]
for fileName in os.listdir(args.directorySrc):
    files.append(fileName)
    fileName = os.path.join(args.directorySrc, fileName)
for f in files:
    streams.append(ffmpeg.input(fileName))

dest = '/home/user/full.mp4'
ffmpeg.concat(*streams).output(dest).run()

And its running super slow, here is the output i see from my script:
[Parsed_concat_0 @ 0x560af8a22460] Buffer queue overflow, dropping.=6348.7kbits/s speed=0.324x
 Last message repeated 17 times
[Parsed_concat_0 @ 0x560af8a22460] Buffer queue overflow, dropping.=5929.5kbits/s speed=0.328x
 Last message repeated 29 times
[Parsed_concat_0 @ 0x560af8a22460] Buffer queue overflow, dropping.=6091.4kbits/s speed=0.335x

When I run the same concatenation from command line:
 $ cat /tmp/filelist
 file '/mnt/smb/src/18.10.27_121205.mp4'
 file '/mnt/smb/src/18.10.27_123438.mp4'
 file '/mnt/smb/src/18.10.27_131923.mp4'
 $ ffmpeg -f concat -safe 0 -i /tmp/filelist  -c copy full.mp4

I get a much faster operation:
frame=10218 fps=2243 q=-1.0 Lsize=  522251kB time=00:02:50.43 bitrate=25101.8kbits/s speed=37.4x

Turns out the command line version is using the concat 'demuxer' which is significantly faster, this is documented here:
https://trac.ffmpeg.org/wiki/Concatenate
Curious if ffmpeg-python has any support for this type of speedy operation :)

format='concat', safe=0)
^
SyntaxError: invalid syntax
Why does this fail?

Hey,
having a related issue that I thought I'd add here. I'm trying to concat two streams with
`
vid1 = ffmpeg.input(v1filename)
vid2 = ffmpeg.input(v2filename)
result = ffmpeg.concat(vid1,vid2)
out = ffmpeg.output(result, 'final.mkv',c="copy")
This only gives meStreamcopy requested for output stream 0:0, which is fed from a complex filtergraph. Filtering and streamcopy cannot be used together.`
Is use of the concat demuxer (instead of the filter) not possible with the ffmpeg.concat function, supplying the streams directly instead of a text file?

richardKafati wrote:

format='concat', safe=0)
^
SyntaxError: invalid syntax

Works for me with Python 3.12.1 and ffmpeg-python 0.2.

krateng wrote:

Is use of the concat demuxer (instead of the filter) not possible with the ffmpeg.concat function, supplying the streams directly instead of a text file?

As of  release 0.2 ffmpeg-python simply doesn't provide any syntactic sugar around creating file lists for the concat demuxer. The concat() function is just able to construct concat filter arguments.


[Feature Request] Allow an actor to exist in both the Bestiary and NPC Gallery simultaneously

There are some actors where I'd like to be able to reveal both combat statistics in the Bestiary as well as track their roleplay details as an NPC in the gallery. Is this a possibility?

It currently works that way. If you add an NPC, then it's combat statistics can be viewed. (short clip below)
https://github.com/user-attachments/assets/1b80557d-23e5-478d-a5f7-962b523c5606
In this case I didn't set up any stats or stuff on this one. But NPCs have everything creatures have, and a little extra. 🤖 👌


[Bug Report] - Still serving portions of the old site

Describe the bug.
Somehow github is still serving portions of the old site. For instance...
https://uptane.github.io/deployment-considerations/index.html
Describe the steps to reproduce the behavior.
No response
Expected behavior.
No response
Screenshots.

Operating environment
No response
What browsers are you seeing the problem on?
Chrome
Additional context
No response

This might be something related to cache or something like that we might need to look into the deployment workflow

also @hexsecs any update on this one : https://github.com/uptane/uptane.github.io/issues/59 , I think setting up domain might will help but I am not sure as uptane.github.io still be live

I think I might have IDed the root cause. The docs repository and uptane standard repository were actually serving up webpages. I turned off the github pages part in the two other repos.

Now I get a 404 not found when I click on https://uptane.github.io/deployment-considerations/index.html. I think this issue might be solved.


feat: add synchronous api

There are cases where synchronous apis are the only way to go, therefore
one might choose the slower api over the async one.
For example when running in a constructor of a class.
Closes #4

@h2non Would you mind to review and release this?

LGTM!
Can you add the public sync API documentation in the README as well?

@h2non Sure, added it.

The new release is now available on npm.

Awesome. Thank you :rocket:


Superembed Extension

Still I got no solution for streaming 😐 tried various ways but none worked.

In Comick needs fix in detail section for fetching chapter's and watch section.
Docs: https://api.comick.fun/docs/static/index.html
Refer: https://github.com/manga-download/hakuneko/blob/master/src/web/mjs/connectors/ComicK.mjs#L23

With this changes it's showing chapters https://pastebin.com/ra3BtXTt but not all chapters are being shown also some chapters returning null title.

wait a minute i'll try

wait a minute i'll try

Hmm and also check if something can be done with Superembed or i should remove it

Maybe item.chap should be used?

This can support multiple languages https://pastebin.com/kvTQ1STA

Ok not all chapters showing i fixed let me update the code but still need fixes in watch section

Now remain watch section for comick and detail and watch section for Superembed

https://api.comick.app/chapter/y50kP?tachiyomi=true
Source images

Okk i fixed the watch section please check Superembed once as I am not able to fix that extension, should I remove it?

Okk i fixed the watch section please check Superembed once as I am not able to fix that extension, should I remove it?

Wait

https://pastebin.com/2jxYTQ0L

Need to update watch section so it can support playback by both movie and series
Request movie by TMDB id: https://multiembed.mov/directstream.php?video_id=447365&tmdb=1
Request episode ( series ) by TMDB id: https://multiembed.mov/directstream.php?video_id=114472&tmdb=1&s=1&e=2

Also when I tested your updated code it's giving me status code 400 error

Merge it I have removed that extension.


epicon: memory(mem_buff) related problem

problem report and remedy is committed to my repo, https://github.com/jg1uaa/IOT-House/commit/3e3245801ea384922624b57666565180060d060c (in IPv6 branch)

Thank you for reporting the bug. I'm sorry, but I can't scrutinize the content because I'm not the ability at the time of development. So if there is no verification problem, please update.


Adding previously missing services package.

Adding previously untagged package for the Microsoft.Resource 2016-02-01.
This PR should not be merged before https://github.com/Azure/azure-rest-api-specs/pull/1901

Updating the profile definition of 2017-03-09 as per Azure Stack Team's guidance.


Issue with client.v2.tweet Function Not Returning Response and Freezing Server

I'm encountering an issue with the client.v2.tweet function from the Twitter API library within supabase cloud functions. The problem is that when I use this function to post a tweet, it doesn't seem to return a response as expected, and it leads to the server becoming unresponsive or freezing. but it does post the tweet to my twitter account.
code
const client = userClient.readWrite;
try{
  const postTweet = await client.v2.tweet({text:tweetText});
  // console.log(postTweet )
  return new Response(
    JSON.stringify({
      message: postTweet,
    }),
    { headers: { "Content-Type": "application/json" }, status: 200, },
  );
}catch(e){
  const error = {
    message: e,
  };

  return new Response(
    JSON.stringify(error),
    { headers: { "Content-Type": "application/json" },status:300 },
  );
}

Please let me know if there are any suggestions or potential solutions to resolve this issue. Your assistance is greatly appreciated.

I am getting following error on my free account
 data: {
        title: 'Unsupported Authentication',
        detail: 'Authenticating with OAuth 2.0 Application-Only is forbidden for this endpoint.  Supported authentication types are [OAuth 1.0a User Context, OAuth 2.0 User Context].',
        type: 'https://api.twitter.com/2/problems/unsupported-authentication',
        status: 403
      }

Do you have free or paid account?


clean up duplicate package imports

What this PR does / why we need it:
clean up duplicate package imports
Special notes for your reviewer:
Does this PR introduce a user-facing change?

NONE

/cc @windsonsea

Hi @SSmallMonster , these code changes need your help to review.

Codecov Report

Merging #717 (ba4a64e) into main (504e9f4) will not change coverage.
The diff coverage is n/a.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@           Coverage Diff           @@
##             main     #717   +/-   ##
=======================================
  Coverage   30.59%   30.59%
=======================================
  Files          25       25
  Lines        1556     1556
=======================================
  Hits          476      476
  Misses       1015     1015
  Partials       65       65

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

LGTM


feat: add basic structures for graphql support

Purpose
Fixes #1440
How does the solution address the problem

Changes made

As this is change, which should not affect existing code (which means it should not break anything) I would say lets deploy it and fix bugs, which come from testing

/deploy

PR NET pr-1974 Successfully deployed!explorerhttps://explorer-pr-1974.prnet.taraxa.io  RPChttps://rpc-pr-1974.prnet.taraxa.io Boot NodesBoot Node <IP_ADDRESS>Boot Node <IP_ADDRESS>Boot Node 3<IP_ADDRESS> RPC NodesRPC Node <IP_ADDRESS>RPC Node <IP_ADDRESS>RPC Node 3<IP_ADDRESS>


Fix: Create Base Query consistent return value

This change switches how we handle queryObserver updates. This change will update the resource by first making a refetch call and then track the value updates using the resource states introduced in Solid v1.5^. These updates will then update the main state store in a computation which will enable us to make data consistent between the resource and the state store

Closing this request with https://github.com/ardeora/query/pull/23 as duplicate


[Feature]: Support to use markdown backend for Markdown-like filetypes

Enhancement Request: Support to use markdown backend for Markdown-like filetypes
I use vimwiki with the markdown syntax. The filetype reported by the buffer is vimwiki instead of markdown, which trips up this check in markdown.lua:
M.is_supported = function(bufnr)
  if not vim.tbl_contains(util.get_filetypes(bufnr), "markdown") then
    return false, "Filetype is not markdown"
  end
  return true, nil
end

A simple proposal may be to add a configuration for what constitutes as markdown, perhaps something like config.markdown.filetypes that can default to markdown or be a list.
Is this worth submitting a PR @stevearc ?

Had a closer look at get_filetypes. Looks like I can set the check can rely on . separated filetypes - e.g. vimwiki.markdown
I was able to hook to autocommand to get this to work, e.g:  autocmd FileType vimwiki setlocal filetype=vimwiki.markdown


Fix bug in add command

As mentioned in #123.

Codecov Report

Merging #124 (5478355) into master (8ed83dc) will increase coverage by 0.04%.
The diff coverage is 100.00%.

@@             Coverage Diff              @@
##             master     #124      +/-   ##
============================================
+ Coverage     58.80%   58.84%   +0.04%
  Complexity      523      523
============================================
  Files            98       98
  Lines          2051     2053       +2
  Branches        252      252
============================================
+ Hits           1206     1208       +2
  Misses          753      753
  Partials         92       92

Impacted Files
Coverage Δ

.../java/seedu/address/logic/commands/AddCommand.java
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 8ed83dc...5478355. Read the comment docs.


Fix bug with alt+shift on linux

Thank you guys for franz.
This PR fixes https://github.com/meetfranz/franz/issues/1088
Description
There is an issue on XFCE, when you setting a focus to input field, then press Alt + Shift, the Alt button opens application menu and blocks the input. Btw Shift + Alt works correcty. This bug reproduces on Slack, Google Play Music Desktop Player. And NOT reproduces on, for example, Atom.
I'm not so good in JS, and this fix I was took from https://github.com/ramboxapp/community-edition/pull/432. Any notes and chages of course welcome.
Motivation and Context
It's a very popular bug on electron apps:
https://github.com/ramboxapp/community-edition/issues/120
https://github.com/zulip/zulip-electron/issues/528
https://github.com/RocketChat/Rocket.Chat.Electron/issues/50
https://github.com/vector-im/riot-web/issues/4345
How Has This Been Tested?
Just runned it on XFCE 4.12
Screenshots (if appropriate):
Types of changes

[x] Bug fix (non-breaking change which fixes an issue)

Checklist:

[x] My code follows the code style of this project (run $ yarn lint).

Guys, will it be reviewed and merged?

I don't have a VM with XFCE at the moment but this fix would lead to menu items to be displayed like this on Windows and Mac:

We would need to fork react-electron-titlebar to add this before this PR can get merged. I'll add this to my todo list.


Background Mode is running for only few(probably 5 mins) minutes in Android 8 and Android 9

The App is running for almost 5 minutes at the background. After that, it stops working. If I resume then it starts again.
I have tried several solutions. I have also used the plugin https://github.com/DavidBriglio/cordova-plugin-foreground-service and added
cordova.plugins.foregroundService.start('GPS Running', 'Background Service', 'myicon', 3, 10);
cordova.plugins.backgroundMode.disableBatteryOptimizations();
cordova.plugins.backgroundMode.disableWebViewOptimizations();
But it did not work. I also tried with Cordova power management. This is my GitHub repo https://github.com/ahsanhabib23/testBackgroundIonic3 where the issue has been reproduced.
I am using the fork version. Most added the library by running this command
cordova plugin add https://github.com/katzer/cordova-plugin-background-mode --nofetch
I know there is an existing open issue. But still posting because comments from there did not help.
Ionic:

   ionic (Ionic CLI)  : 4.12.0 (/usr/local/lib/node_modules/ionic)
   Ionic Framework    : ionic-angular 3.9.5
   @ionic/app-scripts : 3.2.2

Cordova:

   cordova (Cordova CLI) : 8.1.2<EMAIL_ADDRESS>   Cordova Platforms     : android 7.1.4
   Cordova Plugins       : cordova-plugin-ionic-keyboard 2.1.3, cordova-plugin-ionic-webview 4.1.1, (and 6 other plugins)

System:

   Android SDK Tools : 26.1.1 (/Users/bs236/Library/Android/sdk/)
   ios-deploy        : 1.9.2
   NodeJS            : v10.15.3 (/usr/local/bin/node)
   npm               : 6.9.0
   OS                : macOS High Sierra
   Xcode             : Xcode 10.1 Build version 10B61

I founded the solution at https://github.com/katzer/cordova-plugin-background-mode/issues/430
disableBatteryOptimizations

I founded the solution at #430
disableBatteryOptimizations

not working

I have the same problem !

Solução crie uma service com uma notificação passos!!!!!
index.js >> receivedEvent, onDeviceReady

antes cordova.plugins.backgroundMode.setDefaults({
title: "title",
text: "text",
icon: 'icon', // this will look for icon.png in platforms/android/res/drawable|mipmap
color: "6E10A0", // hex format like 'F14F4D'
resume: false,
silent: false,
hidden: false,
bigText: false,
})

       >>> aqui    cordova.plugins.backgroundMode.enable();
      >>> antes   cordova.plugins.backgroundMode.onactivate = function () {
                    cordova.plugins.backgroundMode.disableWebViewOptimizations();
                };
                cordova.plugins.backgroundMode.ondeactivate = function () {
                    alert('teste')
                };
    >>>> aqui      cordova.plugins.backgroundMode.disableBatteryOptimizations();

inicie o service com uma notificação
pegue parte do codigo plugin powemanger e inclua em seu service ele vai trabalha para servir cordova.plugins.backgroundMode para manter ele sempre ativo! abraços!!!!! testado Android 8 e Android 9 rodando aqui 24hs por dias! vlw

Solução crie uma service com uma notificação passos!!!!!
index.js >> receivedEvent, onDeviceReady

antes cordova.plugins.backgroundMode.setDefaults({
title: "title",
text: "text",
icon: 'icon', // this will look for icon.png in platforms/android/res/drawable|mipmap
color: "6E10A0", // hex format like 'F14F4D'
resume: false,
silent: false,
hidden: false,
bigText: false,
})

       >>> aqui    cordova.plugins.backgroundMode.enable();
      >>> antes   cordova.plugins.backgroundMode.onactivate = function () {
                    cordova.plugins.backgroundMode.disableWebViewOptimizations();
                };
                cordova.plugins.backgroundMode.ondeactivate = function () {
                    alert('teste')
                };
    >>>> aqui      cordova.plugins.backgroundMode.disableBatteryOptimizations();

plugin.mxl
add permission
inicie o service com uma notificação
pegue parte do codigo plugin powemanger e inclua em seu service ele vai trabalha para servir cordova.plugins.backgroundMode para manter ele sempre ativo! abraços!!!!! testado Android 8 e Android 9 rodando aqui 24hs por dias! vlw

Haa ótima ideia :)
@ahsanhabib23

Eu tenho o mesmo problema !

#461

For me not working :( !!

if i set targetSdkVersion="26" or targetSdkVersion="27" this plugins works, but the Google not will accepted API smaller than 28  !! and with targetSdkVersion="28" not working :( !

Do you still have the issue if you use this plugin?
https://bitbucket.org/TheBosZ/cordova-plugin-run-in-background

@TDola , I cant even get the plugin from that url.

@utan in this url, have a description of instalation

@TDola , Yes I indeed ran the installation procedure , it failed with the following:
`Failed to fetch plugin https://bitbucket.org/TheBosZ/cordova-plugin-run-in-background via registry.
Probably this is either a connection problem, or plugin spec is incorrect.
Check your connection and plugin name/version/URL.
Error: npm: Command failed with exit code 1 Error output:
npm ERR! path git
npm ERR! code ENOENT
npm ERR! errno ENOENT
npm ERR! syscall spawn git
npm ERR! enoent Error while executing:
npm ERR! enoent undefined ls-remote -h -t https://bitbucket.org/TheBosZ/cordova-plugin-run-in-background.git
npm ERR! enoent
npm ERR! enoent
npm ERR! enoent spawn git ENOENT
npm ERR! enoent This is related to npm not being able to find a file.
npm ERR! enoent
npm ERR! A complete log of this run can be found in:
npm ERR!     /home/utan-dev/.npm/_logs/2019-08-20T13_23_10_949Z-debug.log
`

@utan do you tryed to create a simple installation and testing this plugin? i haved this error, but, when i create a new app with same version, works !

Ok, the plugin got installed..
But It won't stay connected once go into background mode..
I have cordova @9.. any other way to make this work..?

@utan cordova 8.1.2<EMAIL_ADDRESS>

@Flavioamorim , so with this cordova 8.1.2 works?

Works fine in cordova 9, be sure to set the disable battery optimizations, otherwise Android Pie shuts down anything that runs too long.

Hi @TDola  I have that! do you mean disableWebVIewOptimization:method ?
``js
cordova.plugins.backgroundMode.enable();
                    cordova.plugins.backgroundMode.on('activate', function() {
                        cordova.plugins.backgroundMode.disableWebViewOptimizations();
                    });

``

yes that should be all that's needed.  If its not working as it should, post a new issue here https://bitbucket.org/TheBosZ/cordova-plugin-run-in-background

@TDola , I will, I had it wrapped in a timeout of 500 mils .. will test without it.. and see.

You can fix that with cordova-plugin-background-geolocation, if that's an option for you.

Same issue here, working for 5min on a Xiaomi device with Android One.
I also tried to add a setInterval which attempts to do a cordova.plugins['backgroundMode'].moveToForeground() but without success...

Same issue here, the app stops after 5 minutes. Tested every sugestion above..

@andreddosantos did you try https://bitbucket.org/TheBosZ/cordova-plugin-run-in-background ?

Yes while in background i used to call a sqliteplugin to persist some coordinates and now after five minute the calls to the sqlite just pile in the callstack

I have the same problem, after 5 minutes the application stops, on android 8 and 9.
Anyone already have any solution?

Not sure, my own usage is designed to give up after 15 minutes anyways as to not kill the users battery.
But in theory I am calling configure a lot.  I use the notification text as a progress indicator.  So the frequent updates, once per 20 seconds roughly, might be tricking Android into keeping it alive?

Not sure, my own usage is designed to give up after 15 minutes anyways as to not kill the users battery.
But in theory I am calling configure a lot. I use the notification text as a progress indicator. So the frequent updates, once per 20 seconds roughly, might be tricking Android into keeping it alive?
My usage is basically this:
On app pause event -> Check a queue, if anything important is in the queue, activate background mode
As queue runs, it updates background mode with config. The queue is essentially a setTimeout that recursively calls itself and burns down a list of tasks in a sqllite database, updating as it goes, and uploading a file. Every successful loop causes the config call.
If the queue runs out of important tasks, even if there is still more tasks in it, it tells background mode to shut down. On some phones the app keeps going anyways, on others it stops immediately.
In the case of iOS, I use background fetch instead. It's the same basic principal, except it shuts itself down after 25 seconds as to not use too much time on iOS and risk the OS killing the process.
Background mode is broken in iOS and won't be accepted to the app store anymore

Did you found a Solution?

Not sure, my own usage is designed to give up after 15 minutes anyways as to not kill the users battery.
But in theory I am calling configure a lot. I use the notification text as a progress indicator. So the frequent updates, once per 20 seconds roughly, might be tricking Android into keeping it alive?
My usage is basically this:
On app pause event -> Check a queue, if anything important is in the queue, activate background mode
As queue runs, it updates background mode with config. The queue is essentially a setTimeout that recursively calls itself and burns down a list of tasks in a sqllite database, updating as it goes, and uploading a file. Every successful loop causes the config call.
If the queue runs out of important tasks, even if there is still more tasks in it, it tells background mode to shut down. On some phones the app keeps going anyways, on others it stops immediately.
In the case of iOS, I use background fetch instead. It's the same basic principal, except it shuts itself down after 25 seconds as to not use too much time on iOS and risk the OS killing the process.
Background mode is broken in iOS and won't be accepted to the app store anymore

Did you found a Solution?

To it only working for 5 minutes?
Far as I can tell, as long as there is activity to the notification, it keeps working.  Try calling configure in a setTimeout loop to update the notifications periodically and see if it still stops running after 5 minutes.

@megadrazz
Your solution worked well for me. But all of a sudden it does not work. Additionally, whenever I go to the background, I console.log some data inside cordova.plugins.backgroundMode.onactivate and cordova.plugins.backgroundMode.ondeactivate. So whenever I go back and forth, nothing  is printing

same problem here. 5 min and no more XHR calls from the background. still getting geo location updates but the call to the server is blocked by the system.
any solution ?

do you request android.permission.ACCESS_BACKGROUND_LOCATION in your MainActivity ?

If you can, you might want to reconsider tracking in the background.
https://developer.android.com/about/versions/11/privacy/location
https://developer.android.com/training/location/permissions#request-background-location
As noted above, google is cracking down on it.  Your app must now either request foreground, or background location access.

You can call the wakeUp method in a given interval to keep the app awake.
const timer = 4*60*1000; // four minutes
 this.backgroundMode.enable();
 this.backgroundMode.on('activate')
    .subscribe(()=>{
       setInterval(()=>{
            this.backgroundMode.wakeUp();
            this.backgroundMode.moveToForeground();
          },timer);
    });

@katzer


Renew package to use with node 20+

Hello everyone,
We have made efforts to revitalise this package to migrate the backend part of our plugin from Go to TypeScript.
This change was necessary because we had a lot of complex logic, 90% of which was duplicated on both the frontend and backend. Maintaining this logic in two different languages is quite timeconsuming.
What has been done:

Updated the main proto file to ensure compatibility with the current version of the Go SDK.
Migrated the build process to use Webpack (webpack-be.config.js).
Updated all outdated dependencies prevented project from starting and some build scripts.
Verified that the project runs successfully on Node.js 20+
Added build support using pkg for all systems/architectures.

Issues that are still unresolved:

There is an issue with incorrect conversion to DataFrame for string data types. I am working on a fix for this.
The build process needs to be run from the project where the plugin is deployed. We need to improve the scripts so that the build can be initiated with a single command.

It would be great if you could take a look and share any feeback, sorry for any stupid errors, I'm quite new to working with grafana SDKs, Thank you!

Hi @lunaticusgreen,
I appreciate the effort you've gone to in order to reach this point with the PR. But I also wish you had reached out to us first...  unfortunately, we're not going to be able to offer assistance here.
As per the readme, the typescript SDK is unofficial and not supported - we're not in a position where we can provide feedback on this PR. We focus on maintaining a single SDK, and despite the ability to use other languages and directly interact with the plugin protocol via gRPC - we generally strongly prefer the use of Go and the SDK as it is simpler both for development and review.
I know this isn't the outcome you were hoping for, and I certainly understand the additional complexity of a typescript frontend and go backend - which would be lessened through use of the node backend.
You mention that 90% of the logic is duplicated between the frontend and backend - could you provide a few examples of this and where it is causing you particular pain with maintenance of your plugin? Perhaps there are alternatives we could explore which would at least eliminate the need to have this in both.
If you want to continue this conversation via email, please reach out to<EMAIL_ADDRESS>


[Feature Request] Publish the Glodium on Modrinth

Hi, me again with Modrinth stuff. I saw that ExtendedAE requires Glodium to work and it is not yet on Modrinth, unlike ExtendedAE. So could you please publish it on Modrinth too?

uploaded now


Some icons look bad / strange on Mac

Anyone on a mac able to provide screenshots at different browser zoom levels?
I noticed this when someone was screensharing and using a Mac, but did not grab screenshots.


Deprecate Image ID

Problem Statement
Docker's Image ID is rarely used to identify an image. Users often use only the repository name, the name and tag or the name with the digest.
Describe the Proposal
Remove the instantiation of the DockerImage class using the image ID and just use repotag
Steps to Implement Proposal

The ABS Image needs to use repotag
The get_image_option function can be removed as we will just take one string matching either the image:tag pattern or the image@digest pattern.

Fixed with commit abf6e91f55d92ee93c8229246f518cccb12d1324


Add contact info to the README

From @marcingrzejszczak on June 27, 2016 20:59
Copied from original issue: Codearte/accurest#291

Gitter badge added


fix(debugging): Removing all the debugging statements

:tada: This PR is included in version 4.17.6 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Potential net6 App Insights Memory Leak

List of NuGet packages and version that you are using:
<PackageReference Include="Microsoft.ApplicationInsights" Version="2.19.0" />
<PackageReference Include="Microsoft.ApplicationInsights.AspNetCore" Version="2.19.0" />
Runtime version (e.g. net461, net48, netcoreapp2.1, netcoreapp3.1, etc. You can find this information from the *.csproj file):
net6
Hosting environment (e.g. Azure Web App, App Service on Linux, Windows, Ubuntu, etc.):
Azure Web App

Describe the bug
Created a net6 project hosted in Azure Web App to host a reasonably simple REST API. After running for about a day noticed extremely high memory usage akin to a memory leak application insights described in #1678
Was able to capture memory dump and find 26.5k long running net tasks seemingly coming from Microsoft.AI.PerfCounterCollector

Mitigated by using versions
<PackageReference Include="Microsoft.ApplicationInsights" Version="2.11.0" />
<PackageReference Include="Microsoft.ApplicationInsights.AspNetCore" Version="2.8.2" />

Hi @BPHartel Can you share a minimal repro? This would allow us to collect a dump to investigate.

I am also seeing this issue and it may prevent us from using App Insights

Was able to capture memory dump and find 26.5k long running net tasks seemingly coming from Microsoft.AI.PerfCounterCollector

26564 is the thread ID, not the number of threads.