in script mitmproxy_logging_script.py: No module named 'logging.handlers'

Problem Description
I am using this script for mitmproxy to log mitmproxy traffic: https://github.com/kubistika/mitmproxy-logging-script
Upon launching mitmproxy, I get this error:
mitmdump -q -s mitmproxy_logging_script.py
[05:36:34.003] in script mitmproxy_logging_script.py: No module named 'logging.handlers'

Steps to reproduce the behavior:
Use the unmodified script and config. I'm running as root currently for testing.
System Information
Mitmproxy: 9.0.1 binary
Python:    3.11.0
OpenSSL:   OpenSSL 3.0.7 1 Nov 2022
Platform:  Linux-5.10.0-20-amd64-x86_64-with-glibc2.31

As I understand it, mitmproxy uses its own virtual env, but the logging module should be installed by default.

We can't provide support for third-party scripts, sorry. At a first glance, it looks like this script is using some modules that don't ship with our precompiled binaries. In this case you want to install mitmproxy from PyPI.

Ah. Makes sense. Thanks!

@mhils when running mitmdump, how can i get the output of my addon to be the only output. for example, I want to suppress all the client connect message, all the server connects, etc. I only want what my script is logging to be shown. I tried the -q option, but it suppressed everything.


Deploy problem

Hi all, I have installed theme and in locally if I launch
hugo server

It works all fine.
Now I would like to deploy my site with github following the hugo documentation.
this is my deploy script:
#!/bin/bash

echo -e "\033[0;32mDeploying updates to GitHub...\033[0m"

# Build the project.
hugo -t anubis # if using a theme, replace with `hugo -t hugo-nuo`

# Go To Public folder
cd public
# Add changes to git.
git add .

# Commit changes.
msg="rebuilding site `date`"
if [ $# -eq 1 ]
  then msg="$1"
fi
git commit -m "$msg"

# Push source and build repos.
git push origin master

# Come Back up to the Project Root
cd ..

homepage works fine but pages videos and post don't have css style.
Do you know why?
this is the URL:
My site
Thanks

I have solved adding custom CSS, but I think that is not a good solution.

Hi @AlessandroMinoccheri
Please fill baseUrl param in site config — without it, absolute links to css and other things become relative.

ok, thanks!


Item "isSelected" for "" does not exist in WhiteOctoberAdminBundle:default:list.html.twig at line 93

Hello,
I have this exception when I'm testing the Bundle, I can't do anything about it so I'm submiting this bug.
I have followed the documentation, here is my code:
<?php
namespace Acme\DemoBundle\Admin;
use WhiteOctober\AdminBundle\DataManager\Doctrine\ORM\Admin\DoctrineORMAdmin;

class ArticleAdmin extends DoctrineORMAdmin
{
protected function configure()
{
    $this
      ->setName('Articles')
      ->setDataClass('Acme\DemoBundle\Entity\Article')
      ->addActions(array(
          'doctrine.orm.crud',
      ))
      ->addFields(array(
          'title',
          'body',
      ))
  ;
}
}

My config:
white_october_admin:
    data_managers:
        doctrine:
            orm:    true
            odm:    false
        mandango:   false
    admins:
        - { class: Acme\DemoBundle\Admin\ArticleAdmin }

And a simple annotation mapped entity :
class Article
{
/**
 * @var integer $id
 *
 * @ORM\Column(name="id", type="integer")
 * @ORM\Id
 * @ORM\GeneratedValue(strategy="AUTO")
 */
private $id;

/**
 * @var string $title
 *
 * @ORM\Column(name="title", type="string", length=255)
 */
private $title;

/**
 * @var text $body
 *
 * @ORM\Column(name="body", type="text")
 */
private $body;

As you can see it's very simple.
I'm running a Symfony SE RC4 and everything else is okay.
As the issue appears to be in Twig, is quite difficult to catch inside the stack trace. Can anyone reproduce this? Thank you.

I've just experienced this here (so far I've only personally used the Mandango admin, but now I've experienced it with the Doctrine ORM). I'll take a look...


upgrade to bootstrap4 using bslib

Currently, this www website, especially the index.html, uses the html output of Rmarkdown with bootstrap 3 with then the bootstrap 4 (that added cards functionality) hard coded into the page by dept-11.com and Ben.
Ideally, we could have an Rmarkdown website still generating the home page and others using the _site.yml for generating nav menus and Bootstrap 4 now made available via bslib.
I recommend building up to the current site from a plain Rmd website rather than pare down from existing. Let's start with a new repo.

Ok, starting with new repo www_bs4

Per https://github.com/marineenergy/www_bs4/commit/40e7fea4c482f5062830d8ac5a986e5bc2550033 added nav bar, body, footer and images for homepage:

...

Turned on Github Pages, so now it's visible here:
https://marineenergy.github.io/www_bs4

Needed to manually modify site_libs/bootstrap-4.5.3/bootstrap.min.css/[bootstrap.min.css](https://github.com/marineenergy/www_bs4/blob/047c1f3c841ca2389c6458e56ed759ba340c1bc3/site_libs/bootstrap-4.5.3/bootstrap.min.css) so it imports relative to current page:
@import url(./libs/styles-dept11-bs4.css)

not absolute file reference to my machine (must be a bug in bslib?):
@import url(/Users/bbest/github/marineenergy/www_bs4/libs/styles-dept11-bs4.css)

to avoid error in JS Console at https://marineenergy.github.io/www_bs4
GET https://marineenergy.github.io/Users/bbest/github/marineenergy/www_bs4/libs/styles-dept11-bs4.css net::ERR_ABORTED 404

Also interesting and worthwhile backgrounder on SASS/CSS with R Shiny/Rmd:

Pedro Silva | Styling Shiny with CSS & SASS and Speeding Up Shiny Apps | RStudio - YouTube

Doh! Having trouble installing bslib on server....

> install.packages("bslib")
Installing package into ‘/usr/local/lib/R/site-library’ (as ‘lib’ is unspecified) Warning in install.packages :   package ‘bslib’ is not available (for R version 4.0.2)

> remotes::install_github("rstudio/bslib")
Error in loadNamespace(i, c(lib.loc, .libPaths()), versionCheck = vI[[i]]) :
  namespace ‘htmltools’ 0.5.0 is being loaded, but >= 0.5.1 is required
Calls: <Anonymous> ... withCallingHandlers -> loadNamespace -> namespaceImport -> loadNamespace
Execution halted
ERROR: lazy loading failed for package ‘sass’
* removing ‘/usr/local/lib/R/site-library/sass’
* restoring previous ‘/usr/local/lib/R/site-library/sass’
Error: Failed to install 'bslib' from GitHub:
  Failed to install 'sass' from GitHub:
  (converted from warning) installation of package ‘/tmp/Rtmp5m2v2z/filee6f4639b70f/sass_<IP_ADDRESS>01.tar.gz’ had non-zero exit status

Integrated bslib locally, but having trouble installing on server: https://github.com/marineenergy/www/issues/23#issuecomment-813669543
Started new app splash under branch splash_dev
Splash page using bslib and Bootstrap v5. See latest @ https://github.com/marineenergy/apps/commit/0b255d49a70b0c9fd0f46af687e534f8c51d24e9#commitcomment-57202933

Hey @geocoug,
Ok, I got bslib installed on the server with the following sequence of dependent source R package installs:
remotes::install_github("r-lib/fastmap")
remotes::install_github("rstudio/htmltools")
remotes::install_github("rstudio/sass")
remotes::install_github("rstudio/bslib")

Done per 60fceb8fc924791e13e537450f22dffb882a0635. Follow #24 for additional updates.


Failure when running "npm run serve"

Bug report

What is the current behavior?
Executing (as according to the readme)
npm run serve

will give an error.
If the current behavior is a bug, please provide the steps to reproduce.
Use netlifycms and clone the repo locally and run the command.

What is the expected behavior?
Either change the documentation to refer to
npm run start

or change the scripts in package.json.

Other relevant information:

Node.js version: 12.16.1
NPM/Yarn version: 6.13.4
Operating System: Windows 10
Additional tools:

We appreciate the report and suggestion @Acmion! 💖


Record not recognized as a data type

The Record.make_c_struct throws error in some special cases in the new releases of numba:
import numba
import numpy as np
from numba.types import Record, NestedArray, float64

print(numba.__version__)
print(np.__version__)

cell_shape_poly = np.array([[0,0],[1,1], [1,1],[1,0], [1,1]])

args_dtype = Record.make_c_struct([('cell_shape_poly', NestedArray(dtype=float64, shape=cell_shape_poly.shape)),])
print(args_dtype)
args=np.array((cell_shape_poly,), dtype=args_dtype)

The error:
0.55.2
1.22.4
Record(cell_shape_poly[type=nestedarray(float64, (5, 2));offset=0;alignment=8];72;True)
Traceback (most recent call last):
  File "D:\Projects\Python\tessellation\TorchSpatial\test.py", line 12, in <module>
    args=np.array((cell_shape_poly,), dtype=args_dtype)
TypeError: Cannot interpret 'Record(cell_shape_poly[type=nestedarray(float64, (5, 2));offset=0;alignment=8];72;True)' as a data type

Surprisingly the following code works correctly:
import numba
import numpy as np
from numba.types import Record, NestedArray, float64

print(numba.__version__)
print(np.__version__)

cell_shape_poly = np.array([[0,0],[1,1], [1,1],[1,0]])

args_dtype = Record.make_c_struct([('cell_shape_poly', NestedArray(dtype=float64, shape=cell_shape_poly.shape)),])
print(args_dtype)
args=np.array((cell_shape_poly,), dtype=args_dtype)

0.55.2
1.22.4
Record(cell_shape_poly[type=nestedarray(float64, (4, 2));offset=0;alignment=8];72;True)

All of the above codes work perfectly on numba 0.51.2 and numpy 1.21.6, however numba 0.51.2 is not compatible with python 3.10.4 that I'm currently using.

Thanks for the bug report. I can confirm it.

git bisect is pointing to https://github.com/numba/numba/commit/71118545391ad74f6e3fb90a3607610a85615262 as the first bad commit.


WIP: Second attempt at logging refactor

How much more work is it to just consume the keyvalue.T type and directly write data in logfmt at this point? We could make the package internal to the diagnostic service so its clear its not general purpose.

Don't think it would be too much work, but I could be off. Currently the keyvalue.Ts are both strings. Would have to do a bit of work to make them a little more general.

@desa If the types of keyvalue.T are strings doesn't that work well for logging, since everything will be encoded as text to be able to log it?

@nathanielc I think I've addressed all of the issues from the review so far.

@nathanielc this should be in a ready for another review.

@nathanielc I think it's ready to merge on your signal.


embed pdf error

(ReadBinary->FlatMap->FlatMap pid=17023, ip=<IP_ADDRESS>) OSError: When reading information for key '2205.13708v1.HiJoNLP_at_SemEval_2022_Task_2_Detecting_Idiomaticity_of_Multiword_Expressions_using_Multilingual_Pretrained_Language_Models.pdf' in bucket 'ray-llm-batch-inference': AWS Error [code 15]: No response body. [repeated 157x across cluster]
(ReadBinary->FlatMap->FlatMap pid=27798, ip=<IP_ADDRESS>) invalid pdf header: b'<?xml'
(ReadBinary->FlatMap->FlatMap pid=27798, ip=<IP_ADDRESS>) EOF marker not found

Thanks, solved by downloading pdfs to my bucket.


Update topic_groups definitions: add content_ids, remove description

Context
When a content item is tagged to curated Mainstream browse or Specialist topic and this content item’s slug changes the link disappears from the topic page. This is because the curated lists/groups are based on base paths.
The “contents” slugs don’t get updated as a part of dependency resolution in Publishing API.
We want to base it on content_ids which are less subjected to change (if at all).
Changes proposed

Mark content_ids as not-deprecated.
The content_ids property was added to enable support of service_manual_groups as it was decided that the should use the existing topic groups definition according to https://github.com/alphagov/govuk-content-schemas/pull/203. The schema was then split into "topic_groups" and “service_manual_topic_groups”
 in https://github.com/alphagov/govuk-content-schemas/commit/7bad6a241d3b3e178b66df9e81fd9cc8edc54e95 and the properties marked deprecated for later removal.

Update the description in line with the recently unified topic terminology

Remove deprecated description property from "topic_groups"

Notes

There will be a follow-up PR to add content_ids to the required parameters
We will remove contents property after the migration

Related PRs

https://github.com/alphagov/collections-publisher/pull/1749
https://github.com/alphagov/collections/pull/3056

⚠ This repo is Continuously Deployed: make sure you follow the guidance ⚠
Follow these steps if you are doing a Rails upgrade.

This looks good, should we be marking the contents field as deprecated? As I guess we don't want both of contents and content_ids?
I feel like we could use a description field on content_ids that says what its purpose is too. Something like "the content_id of content that is in the topic group". A thing that is unclear to me too is whether we care about the ordering of content_id so it'd be good to specify that in the description.

Will add the description! I thought we may need a short period of having both contents and content_ids before we change what's used for rendering in Collections.

Yup I expect we do too, be just good to mark that contents is the old way and to be removed.


Crash with Open Blocks Builder Guide

fml-client-latest.log
latest.log
crash-2024-09-22_09.22.17-client.txt

Crash when i right click the builder guide when an item in offhand


Changing cell type from code to mardown does not work

It looks as though it works, save the nb and open it and there's no markdown cell.

Pulling into in progress, as this is a regression in master and needs to be fixed as part of updating cell text as user types into editor... (we had work arounds to ensure things worked when changing cell types, that scenario cannot be tested/nor worked on without this fix)

@greazer /cc adding to current backlog as this is broken in master

@DonJayamanne there a specific repro here? Bug text is not super clear
I was assuming that it was:

Create cell (type something in)
Change cell to markdown
Save nb
Close nb
Cell is either missing or not markdown when reopening nb

But this scenario works or me in our old released code.

in our old released code.

Only on master after we merged custom editor work.
Fixed since then, as part of updating mod we l as you type

Got it. Validated then since the case looks good in master.


doc: guide to getting started with new SkyhookDM and dataset client along with documentation

Create a guide to allow people to get started with installing the new arrow CLS based SkyhookDM cluster and start experimenting with it. Also, make documentation about the architecture, all the new APIs, internal details, etc.

Fixed by #86


Task 5

[X] Create a Python notebook, connect the necessary dependencies.
[X] Connect solutions from previous works.
[X] Generate a data set.

[X] Select some graphs from CFPQ set. Do not forget to justify why you chose these graphs.
[X] Using the function from the first homework, find out the labels of the edges of graphs and, based on this information, formulate at least four different queries for each graph. It is better to use the most common labels. Requirements for requests:

Queries to all graphs must follow some common pattern. For example, if there are graphs g1 and g2 with different sets of labels, then it is expected that requests to them will look, for example, like this:

g1:

(l1 | l2)* l3
(l3 | l4)+ l1*
l1 l2 l3 (l4|l1)*

g2:

(m1 | m3)* m2
(m1 | m3)+ m2*
m1 m2 m3 (m3|m1)*

Queries should use all common constructions of regular expressions (closure, concatenation, alternative). That is, at least one request to each graph must contain each of these constructions.

[X] To generate sets of starting vertices, use this function. Do not forget that the query calculation time strongly depends on how exactly the starting set is arranged.

[X] Formulate the stages of the experiment. What do I need to do to answer these questions? Why?
[X] Conduct the necessary experiments, measurements.
[X] To formalize the results of experiments.
[X] To analyze the results.
[X] To answer the questions posed.
[X] To argue the answers (using the experimental results obtained).
[X] Don't forget to publish your notebook to the repository.

Все actions на момент коммита двухнедельной давности Update rpq.ipynb (
commit 50374b1cbd753da11eccf0452025abc6e1666f53) проходили. Сегодня случайно запушил в эту ветку решение шестой задачи, пришлось делать reset. Убрал. Возникла проблема ruff, о которой можно прочитать здесь: https://github.com/python-lsp/python-lsp-ruff/issues/49. Ничего критичного. Поправил.  Все еще жду ревью/апрув 👉👈


One test for the C ext does not pass on Travis

ping @jpauli

@jpauli do you run PHP with ZTS enabled ? It is enabled on Travis.


Stats Service

NOTE:
Added
attendanceCount: {
type: Number,
default: 0,
},
field to EventSchema + EventValidator
Stats Service:
GET /stats/check-in
Get the number of people checked in (staff only)
-works by getting number of attendees with 1+ events
example response:
{count: 10}
GET /stats/merch-item/:PRICE
Get the number of people eligible for merch item (staff only)
-works by getting number of people with points >= PRICE
example response:
{count: 10}
GET /stats/priority-attendee
Get the number of priority attendees (staff only)
-works by getting all attendees with priority-expiration field in the future
example response:
{count: 10}
GET /stats/attendance/:N
Get the attendance of the past n events (staff only)
-sorted in most recent first
example response:
{attendanceCounts: [2, 4, 2, 1, 10]}
GET /stats/dietary-restrictions
-gets a breakdown of all dietary-restrictions
example response:
{dietaryRestrictions: {"None": 230, "Milk" : 10}}

{"none":1,"dietary_restrictions":0,"allergies":0,"both":2}

Changes:
changed dietary restrictions to string[] in BOTH registration and attendees
added allergies : string[] to BOTH registration and attendees
added checkin : boolean[5] to attendees
changed GET /stats/dietary-restrictions to return data in the format:
{
"none": 140,
"dietary_restrictions": 5,
"allergies": 10,
"both": 23
}

{
"none": 1,
"dietary_restrictions": 0,
"allergies": 0,
"both": 2,
"allergy_counts": {
"EGGS": 2,
"MILK": 1
},
"dietary_restriction_counts": {
"HALAL": 2,
"VEGETARIAN": 2
}
}

Other / notes for alex : set up infra for testing (jest)
Also: default.env files


split tildes; resort

[X] Updated Unreleased in CHANGELOG.md to reflect the changes in code or data.

@kylebgorman I think it might be better to split while scraping: tilde for 'or' is a common convention, and it turns up in a couple of the other files:
    106 per_phonemic.tsv
     30 dsb_phonetic.tsv
      3 tuk_phonemic.tsv
      3 eng_uk_phonetic.tsv
      2 nds_phonemic.tsv
      2 cop_phonemic.tsv
      1 swe_phonetic.tsv
      1 mon_cyrl_phonetic.tsv
      1 kaz_phonetic.tsv
      1 ice_phonemic.tsv
      1 eng_us_phonemic.tsv
      1 eng_uk_phonemic.tsv
      1 dut_phonetic.tsv

I'm not familiar with Persian, but I am somewhat familiar with Lower Sorbian (dsb), where they're all splits. The Icelandic seems to come from "(when derived from Erlendur as pronounced /ˈɛtl(ɛ~ɪ)ntʏr/", and I'm not sure how to fix that in Wiktionary.
Python is not my go-to language, but I imagine it would look something like:
        for word, pron in config.extract_word_pron(title, request, config):
            # Pronunciation processing is done in NFD-space;
            # we convert back to NFC aftewards.
            normalized_pron = unicodedata.normalize("NFC", pron)
            # 'cast' is required 'normalize' doesn't return a 'Pron'
            if not '~' in normalized_pron:
                yield word, cast(Pron, normalized_pron)
            else:
                for partial in normalized_pron.split('~'):
                    yield word, cast(Pron, partial.strip())

You raise a good point. I would probably prefer to just flag and reject
though, I think? They're clearly not to Wiktionary style since they're used
so sparsely, and all the pronunciation templates support multiple entries.
In the past (though I don't see any right now) we saw tildes separates
partial pronunciations, and I think decoding the extension of an implicit
regular expression here is a bridge too far.
@lfashby @jacksonlee any thoughts?
On Sun, Oct 25, 2020 at 6:28 PM Jim Regan<EMAIL_ADDRESS>wrote:

@kylebgorman https://github.com/kylebgorman I think it might be better
to split while scraping: tilde for 'or' is a common convention, and it
turns up in a couple of the other files:
106 per_phonemic.tsv

 30 dsb_phonetic.tsv

  3 tuk_phonemic.tsv

  3 eng_uk_phonetic.tsv

  2 nds_phonemic.tsv

  2 cop_phonemic.tsv

  1 swe_phonetic.tsv

  1 mon_cyrl_phonetic.tsv

  1 kaz_phonetic.tsv

  1 ice_phonemic.tsv

  1 eng_us_phonemic.tsv

  1 eng_uk_phonemic.tsv

  1 dut_phonetic.tsv

I'm not familiar with Persian, but I am somewhat familiar with Lower
Sorbian (dsb), where they're all splits. The Icelandic
https://en.wiktionary.org/wiki/Elli#Icelandic seems to come from "(when
derived from Erlendur as pronounced /ˈɛtl(ɛ~ɪ)ntʏr/", and I'm not sure how
to fix that in Wiktionary.
Python is not my go-to language, but I imagine it would look something
like:
    for word, pron in config.extract_word_pron(title, request, config):

        # Pronunciation processing is done in NFD-space;

        # we convert back to NFC aftewards.

        normalized_pron = unicodedata.normalize("NFC", pron)

        # 'cast' is required 'normalize' doesn't return a 'Pron'

        if not '~' in normalized_pron:

            yield word, cast(Pron, normalized_pron)

        else:

            for partial in normalized_pron.split('~'):

                yield word, cast(Pron, partial.strip())

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/kylebgorman/wikipron/pull/250#issuecomment-716222825,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AABG4OLS3LSPVARVOKBS3D3SMSRABANCNFSM4S6HTSAQ
.

Flag and reject would certainly be the most cautious solution. The most consistent solution would be to modify _skip_pron by adding:
if “~” in pron:
	return True

so it matches our previous treatment of - https://github.com/kylebgorman/wikipron/blob/551c97f555df6920ed8ac3798e5d957c9a44095e/wikipron/extract/core.py#L16-L18 but that doesn’t seem smart. I’m not sure why we don’t flag prons containing -.

The tilde has been a known issue, but I'm having trouble recalling why we didn't decide to do something about it within the WikiPron package here. Given the apparently small number of occurrences, I agree that @lfashby's suggestion of adding if "~" in pron: to check and reject pronunciations seems reasonable.


As a community member, I can help triage questions by typing ^support or^support @user which would give a community member a helpful block of text on how to raise an issue.

TBH this could be done with MEE6 or similar but just some sort of way of directing community members to the support issue queue

Would that command return a link to the support repo?

Desired Outcome:
Those in the Discord channel OR on YouTube will actually get their issue addressed if they use the support channel.
On Discord, we should write a bit more than write an issue, we should assume that the user actually hasn't opened an issue or even a GitHub account.
Maybe there's some guide we can link to that already exists in the eddie YT library or just a 1 min explainer video on issue queue inside here.
I would welcome some better copy than this below:
Discord
<stemount> ^support @newMemberWhoShouldRaiseAnIssue
<EddieBot> hey @newMemberWhoShouldRaiseAnIssue, for speedy help, please submit your video ideas, support questions and requests to be added to the GitHub Community here https://github.com/EddieJaoudeCommunity/support/issues

This is the quickest way to get support as things get lost in Discord

**YouTube**

Two scenarios:

 !support
 Raise a question, support issue or video idea here! https://github.com/EddieJaoudeCommunity/support/issues

 !joincommunity
 Submit a request to add yourself to the EddieJaoudeCommunity over at https://github.com/EddieJaoudeCommunity/support/issues

Good points 👍 - we need to lower the barrier to entry and to reduce us repeating ourselves

I think we have a few options, we could do some of them or all of them

write something in the README
add a command to give more information
short video on creating an issue on GitHub

New issue #198 was opened by @eddiejaoude to fix my linting suggestions however as far as i'm concerned...


egress: add cli verifier and rename traffic match

Description:

Adds a verifier for egress http/https/tcp
destinations.

Renames the traffic match name to be derived
from the port and app protocol. This helps
to map a filter_chain_match to the actual
protocol specfied in an egress policy and not
the kind of filter used within the filter_chain.
This is leveraged by the verifier tool.

Updates CLI args for clarity.

Adds http and https egress verification tests.

Part of #4634

Testing done:
pod to pod
$ osm verify connectivity --from-pod curl/curl-7bb5845476-zwxbt --to-pod httpbin/httpbin-69dc7d545c-n7pjb --to-service httpbin
---------------------------------------------
[+] Context: Verify if pod "curl/curl-7bb5845476-zwxbt" can access pod "httpbin/httpbin-69dc7d545c-n7pjb" for service "httpbin/httpbin"
Status: Success

---------------------------------------------

Egress based on HTTPS demo
$ osm verify connectivity --from-pod curl/curl-7bb5845476-zwxbt --to-ext-port 443 --to-ext-host httpbin.org --app-protocol https
---------------------------------------------
[+] Context: Verify if pod "curl/curl-7bb5845476-zwxbt" can access external service on port 443
Status: Success

---------------------------------------------

Egress based on HTTP demo
$ osm verify connectivity --from-pod curl/curl-7bb5845476-zwxbt --to-ext-port 80 --to-ext-host httpbin.org --app-protocol http
---------------------------------------------
[+] Context: Verify if pod "curl/curl-7bb5845476-zwxbt" can access external service on port 80
Status: Success

---------------------------------------------

Egress when using an invalid port/protocol
$ osm verify connectivity --from-pod curl/curl-7bb5845476-zwxbt --to-ext-port 50 --to-ext-host httpbin.org --app-protocol https
---------------------------------------------
[+] Context: Verify if pod "curl/curl-7bb5845476-zwxbt" can access external service on port 50
Status: Failure
Reason: A verification step failed
Suggestion: Please follow the suggestions listed in the failed steps below to resolve the issue

[++] Context: Verify if namespace "curl" is monitored
Status: Success

[++] Context: Verify Envoy sidecar on pod "curl/curl-7bb5845476-zwxbt"
Status: Success

[++] Context: Verify Envoy config for traffic:
	source pod: curl/curl-7bb5845476-zwxbt
	external host: httpbin.org
	external port: 50
	destination protocol: https
Status: Failure

[+++] Context: Verify Envoy config on source
Status: Failure
Reason: Did not find matching filter chain: filter chain not found for for port=50, protocol=https

---------------------------------------------

Affected area:

Functional Area

CLI Tool
[X]

Control Plane
[X]

Egress
[X]

Please answer the following questions with yes/no.

Does this change contain code from or inspired by another project? no

Did you notify the maintainers and provide attribution?

Is this a breaking change? no

Has documentation corresponding to this change been updated in the osm-docs repo (if applicable)? n/a

Codecov Report

Merging #4724 (436f8f4) into main (55bdb17) will decrease coverage by 0.04%.
The diff coverage is 59.49%.

@@            Coverage Diff             @@
##             main    #4724      +/-   ##
==========================================
- Coverage   69.60%   69.55%   -0.05%
==========================================
  Files         218      220       +2
  Lines       15584    15708     +124
==========================================
+ Hits        10847    10926      +79
- Misses       4687     4732      +45
  Partials       50       50

Flag
Coverage Δ

unittests
69.55% <59.49%> (-0.05%)
:arrow_down:

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

Impacted Files
Coverage Δ

pkg/cli/verifier/connectivity_pod_to_pod.go
53.65% <ø> (ø)

pkg/trafficpolicy/egress_types.go
0.00% <0.00%> (ø)

cmd/cli/verify_connectivity.go
26.31% <20.00%> (-10.53%)
:arrow_down:

pkg/cli/verifier/connectivity_egress.go
57.57% <57.57%> (ø)

pkg/cli/verifier/envoy_config.go
71.87% <80.55%> (+2.90%)
:arrow_up:

pkg/envoy/lds/egress.go
80.55% <85.71%> (-0.27%)
:arrow_down:

pkg/catalog/egress.go
86.60% <100.00%> (+0.19%)
:arrow_up:

pkg/messaging/workqueue.go
89.28% <0.00%> (-10.72%)
:arrow_down:

... and 1 more

Continue to review full report at Codecov.

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


Add Queue interface and BrowserQueue implementation

This PR adds the Queue interface that both the browser queue and the node queue (based on postgres) will implement. (99.99% of this diff in is the lock file which are mostly deletions--so don't be intimidated to review). BTW I'm using pnpm version 8.15.4. it looks like it wanted to do some serious cleanup....

BTW I'm using pnpm version 8.15.4.

Should we make this the version specified for Volta?

BTW I'm using pnpm version 8.15.4.

Should we make this the version specified for Volta?

I have no strong feelings on this


HDFS-14894. Add balancer parameter to balance top used nodes.

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Comment

0
reexec
40
Docker mode activated.

_ Prechecks _

+1
dupname
0
No case conflicting files found.

+1
@author
0
The patch does not contain any @author tags.

+1
test4tests
0
The patch appears to include 1 new or modified test files.

_ trunk Compile Tests _

+1
mvninstall
1100
trunk passed

+1
compile
65
trunk passed

+1
checkstyle
51
trunk passed

+1
mvnsite
70
trunk passed

+1
shadedclient
837
branch has no errors when building and testing our client artifacts.

+1
javadoc
85
trunk passed

0
spotbugs
170
Used deprecated FindBugs config; considering switching to SpotBugs.

+1
findbugs
169
trunk passed

_ Patch Compile Tests _

+1
mvninstall
65
the patch passed

+1
compile
59
the patch passed

+1
javac
59
the patch passed

-0
checkstyle
46
hadoop-hdfs-project/hadoop-hdfs: The patch generated 9 new + 232 unchanged - 0 fixed = 241 total (was 232)

+1
mvnsite
63
the patch passed

+1
whitespace
0
The patch has no whitespace issues.

+1
shadedclient
764
patch has no errors when building and testing our client artifacts.

+1
javadoc
75
the patch passed

+1
findbugs
172
the patch passed

_ Other Tests _

-1
unit
5251
hadoop-hdfs in the patch failed.

+1
asflicense
43
The patch does not generate ASF License warnings.

9025

Reason
Tests

Failed junit tests
hadoop.hdfs.server.namenode.snapshot.TestRenameWithSnapshots

hadoop.hdfs.server.balancer.TestBalancer

hadoop.hdfs.tools.TestDFSZKFailoverController

hadoop.hdfs.TestSafeModeWithStripedFileWithRandomECPolicy

hadoop.hdfs.server.namenode.TestStartup

Subsystem
Report/Notes

Docker
Client=19.03.3 Server=19.03.3 base: https://builds.apache.org/job/hadoop-multibranch/job/PR-1636/1/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hadoop/pull/1636

Optional Tests
dupname asflicense compile javac javadoc mvninstall mvnsite unit shadedclient findbugs checkstyle

uname
Linux 0ca7dcd3b1f9 4.15.0-60-generic #67-Ubuntu SMP Thu Aug 22 16:55:30 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
personality/hadoop.sh

git revision
trunk / eeb58a0

Default Java
1.8.0_222

checkstyle
https://builds.apache.org/job/hadoop-multibranch/job/PR-1636/1/artifact/out/diff-checkstyle-hadoop-hdfs-project_hadoop-hdfs.txt

unit
https://builds.apache.org/job/hadoop-multibranch/job/PR-1636/1/artifact/out/patch-unit-hadoop-hdfs-project_hadoop-hdfs.txt

Test Results
https://builds.apache.org/job/hadoop-multibranch/job/PR-1636/1/testReport/

Max. process+thread count
4433 (vs. ulimit of 5500)

modules
C: hadoop-hdfs-project/hadoop-hdfs U: hadoop-hdfs-project/hadoop-hdfs

Console output
https://builds.apache.org/job/hadoop-multibranch/job/PR-1636/1/console

versions
git=2.7.4 maven=3.3.9 findbugs=3.1.0-RC1

Powered by
Apache Yetus 0.10.0 http://yetus.apache.org

This message was automatically generated.

Fixing UT

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Comment

0
reexec
35
Docker mode activated.

_ Prechecks _

+1
dupname
0
No case conflicting files found.

+1
@author
0
The patch does not contain any @author tags.

+1
test4tests
0
The patch appears to include 1 new or modified test files.

_ trunk Compile Tests _

+1
mvninstall
1190
trunk passed

+1
compile
62
trunk passed

+1
checkstyle
48
trunk passed

+1
mvnsite
68
trunk passed

+1
shadedclient
906
branch has no errors when building and testing our client artifacts.

+1
javadoc
76
trunk passed

0
spotbugs
176
Used deprecated FindBugs config; considering switching to SpotBugs.

+1
findbugs
174
trunk passed

_ Patch Compile Tests _

+1
mvninstall
65
the patch passed

+1
compile
57
the patch passed

+1
javac
57
the patch passed

-0
checkstyle
43
hadoop-hdfs-project/hadoop-hdfs: The patch generated 9 new + 232 unchanged - 0 fixed = 241 total (was 232)

+1
mvnsite
66
the patch passed

+1
whitespace
0
The patch has no whitespace issues.

+1
shadedclient
838
patch has no errors when building and testing our client artifacts.

+1
javadoc
91
the patch passed

+1
findbugs
201
the patch passed

_ Other Tests _

-1
unit
5006
hadoop-hdfs in the patch failed.

+1
asflicense
35
The patch does not generate ASF License warnings.

9030

Reason
Tests

Failed junit tests
hadoop.hdfs.server.namenode.snapshot.TestRenameWithSnapshots

Subsystem
Report/Notes

Docker
Client=19.03.2 Server=19.03.2 base: https://builds.apache.org/job/hadoop-multibranch/job/PR-1636/2/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hadoop/pull/1636

Optional Tests
dupname asflicense compile javac javadoc mvninstall mvnsite unit shadedclient findbugs checkstyle

uname
Linux 696b92a7796b 4.15.0-60-generic #67-Ubuntu SMP Thu Aug 22 16:55:30 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
personality/hadoop.sh

git revision
trunk / 9c72bf4

Default Java
1.8.0_222

checkstyle
https://builds.apache.org/job/hadoop-multibranch/job/PR-1636/2/artifact/out/diff-checkstyle-hadoop-hdfs-project_hadoop-hdfs.txt

unit
https://builds.apache.org/job/hadoop-multibranch/job/PR-1636/2/artifact/out/patch-unit-hadoop-hdfs-project_hadoop-hdfs.txt

Test Results
https://builds.apache.org/job/hadoop-multibranch/job/PR-1636/2/testReport/

Max. process+thread count
3473 (vs. ulimit of 5500)

modules
C: hadoop-hdfs-project/hadoop-hdfs U: hadoop-hdfs-project/hadoop-hdfs

Console output
https://builds.apache.org/job/hadoop-multibranch/job/PR-1636/2/console

versions
git=2.7.4 maven=3.3.9 findbugs=3.1.0-RC1

Powered by
Apache Yetus 0.10.0 http://yetus.apache.org

This message was automatically generated.

@fapifta @jojochuang Just check back on this : ) Could you please take a look
https://issues.apache.org/jira/browse/HDFS-14894


why does swipe not work in the maui application in windows

why does swipe not work in the maui application in windows

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

ID: 0249dadd-67f1-3bf4-a386-6b7b3429612a
Version Independent ID: 0249dadd-67f1-3bf4-a386-6b7b3429612a
Content: SwipeView - .NET MAUI
Content Source: docs/user-interface/controls/swipeview.md
Product: dotnet-mobile
Technology: dotnet-maui
GitHub Login: @davidbritch
Microsoft Alias: dabritch

Afraid it looks like there are issues with that control: https://github.com/dotnet/maui/issues?q=is%3Aissue+is%3Aopen+SwipeView


[Issue]: Invoices not marked as PAID in Mirakl

Describe the Issue
HWConnector picks up the payment file from Mirakl and connects with HyperWallet to transfer the funds to the Seller.
The funds are transferred successfully to the Seller, however the associated invoices with the payment file are not marked as PAID in Mirakl, giving an error as below;
c.p.i.l.AbstractNotificationListener : Notification [wbh-ee68ddd7-58c9-40dd-86e4-b1056a51c049] could not be processed. Reason: [[400] The amount is incorrect for the accounting document with number '230201', actual: '767.36', expected: '737.41']
This issues seems to be similar to the once previously reported by someone else also -
https://github.com/paypal/mirakl-hyperwallet-connector/issues/102
CC: @javierpoloe2y , @max6001
Environment
Live/Production
Version
4.14.0
Expected Behavior
The HWConnector should mark the invoices associated with the Payment file as PAID.
Actual Behavior
The HWConnector does not mark the manual and automatic invoice as PAID.
Steps to Reproduce

There are multiple invoices for a single Seller (automatic invoices for commissions & order payments) and manual credit invoices.
The manual credit invoice -<PHONE_NUMBER>95 - $29.95
The Automatic Invoice -<PHONE_NUMBER>01 - $737.41
Mirakl generates a single payment file combining all the invoices (manual as well as automatic) totaling - $767.36
HWConnector picks up the payment file and sends payment instructions to HyperWallet.

Pre-conditions
No response
Relevant log output
.p.s.AcceptedPaymentNotificationStrategy : Creating payment confirmation request for invoice ID 230201 and amount 767.36
c.p.i.l.AbstractNotificationListener : Notification [wbh-ee68ddd7-58c9-40dd-86e4-b1056a51c049] could not be processed. Reason: [[400] The amount is incorrect for the accounting document with number '230201', actual: '767.36', expected: '737.41']

Hi @pradeep-agnihotri
Thanks for reaching out.
That issue was already reported through https://github.com/paypal/mirakl-hyperwallet-connector/issues/102.
I'm afraid the outcome will the same we've provided back then.

Hello @max6001
Mirakl does provide the invoices that are pending to be marked as PAID, see below IV01 API.
/api/invoices?payment_status=PENDING&start_date=2023-02-08&type=ALL
I am not sure if Mirakl needs to be do anything here; the HWC would need to fetch the invoice via IV01 and then accordingly mark them as PAID using IV07.

@pradeep-agnihotri the HMC is only taking care of automatic billing invoices.
It ignores the manual credit and manual invoices as those money movement are already included into the automatic invoices generated.
Your issue is related to the fact the HMC is successfully pulling the automatic invoice from Mirakl.
It is sending that invoice for payment to Hyperwallet.
Hyperwallet does process that payment and notify the HMC via webhook.
The HMC is then trying to update the status of the payment in Mirakl but we are facing an error because Mirakl is expecting another amount than the one send to and via Hyperwallet.
The different options proposed to Mirakl are detailed in the other ticket.

@pradeep-agnihotri The HMC is using IV01 but ignoring manual credit note. Mirakl includes the amount of the manual credit notes into the next automatic invoice. This means that if the connector was processing manual credit notes, the amount of the credit notes will be payed two times, one in the credit note and one in the next invoice.


Amend macro future proofing RFC

Proposed changes:

Add Semicolon to FOLLOW(ty).
Modify algorithm to allow repetitions after NTs.

(rendered)

It seems like the RFC is incomplete: FIRST needs to be spelled out.

@cmr, I would like to clarify something:
The RFC does not mention handling of TtDelimited token trees.  Are they supposed to be implicitly flattened into the sequence of constituent tokens?  If so, shouldn't the last line of this section say "(Note that open and close delimiters are valid following any NT.)" ?

Okay, updated my PR.

@vadimcn thanks. I'll try to read it and give it some thought. In the meantime, though, @pnkfelix is going to shepherd this PR.

@vadimcn Yes, your understanding is correct.

Could this help solve #773?

Looks like some rebasings have disappeared some of the comments I previously made. Will re-review, but @pnkfelix's and @nikomatsakis's comments are on-point.

Did not read yet, but +1 for allowing repetition after a metavariable.  The inability to do
$x:tt $($xs:tt)*

is a serious limitation with fancy recursive macros.

@cmr, @nikomatsakis: ping

ping @pnkfelix

While this is still open, could we also add | and if to FOLLOW(pat)? (cc rust-lang/rust#20843)

Okay, so I finally got a chance to go through this in some detail.
(I had not really read this RFC carefully enough in the past, so I got caught up in relatively minor issues in its presentation that are not an issue with this PR in particular; hopefully I will have a chance in the near future, while this is fresh in my mind, to try to put up a PR with some revisions to the text to help clarify it.)

I think it would be good to add some examples of the CHECK algorithm's behavior on various inputs.  I know the original RFC did not have this, but I think it should have, and there's no reason we cannot correct that now.  I will help with this (i.e. I'll put up some text somewhere for @vadimcn to incorporate.)

I think there is a bug in this additions being made here, in a corner-case for the FIRST function.  The standard definition for FIRST returns a set made up of terminals plus (potentially) a special "epsilon" marker signifying that the input sequence may rewrite to the empty string.  That can arise for us here (despite effort in this definition to try to avoid it), and needs to be handled.  I'm about to go have dinner so I cannot put in a formal example, but the kind of scenario I am worried about is something like $($($(i:expr)),* $($(j:expr)),*) something.  I will write up more later.

While this is still open, could we also add | and if to FOLLOW(pat)? (cc rust-lang/rust#20843)

Also in for parsing for loop syntax (for $p:pat in iter).

cc #1209

@vadimcn have you given any thought to the feedback I wrote above?
(I know I promised to write up more later, so I cannot claim that I am not without blame w.r.t. the radio silence here.)
Anyway, let me know if you think you'll be able to address the issues above.  If you won't have the time, I can try to take on this RFC myself (i.e. try to figure out what are the edits that I want, and put my my own PR with them...)

@pnkfelix: sorry this slipped through the cracks of my attention.  I'll take another look tonight.

@pnkfelix: indeed, NT matching an empty string seems to be a problem.   I've updated the PR to address this.

What's the status here?

So I managed to spend ages working on the more invasive macro future-proofing amendment #1384 but I failed to incorporate the suggestions from this RFC in that effort.  Sorry about that, I don't know how I overlooked it.

Okay, #1384 handles repetitions in the manner that I think this PR was asking for:
macro_rules! foo { ($a:expr $( , $b:expr ),*) => {} }

compiles fine on Nightly today (but is rejected by Stable right now, of course).
So, given that we now handle sequence repetitions, and that Semi is in FOLLOW(ty), I think #1384 has subsumed this proposed amendment.
Thank you for your patience, @vadimcn


How do I set up matterbridge and get it running

Environment

Version: not installed idk how

Operating system: win 10

Please describe the expected behavior.
Please describe the actual behavior.

Any steps to reproduce the behavior?
Please add your configuration file

I am new, i figured out just recently how to use IRC kind of, but that's not the point, how does one go about using this, all i ever find is install it, but if i just install it idk how/were to host it, can someone please explain how i can use this to bridge my channel IRC from IRC.ircnet.com and my discord server. Any help is appreciated, thanks.

I wrote a blog post a while back on this topic. 216920

Also see https://github.com/matterbridge/wiki/wiki/How-to-create-your-config


Additional patches for Ubuntu 20

Move Ubuntu 20 patches for external packages to a different directory
Add patches for boringssl and fastboot
Rebase cmake with the latest version for Ubuntu 20
New cmake patches to allow compilation on chroot
Add opencv patch that was omitted by mistake before
Add script to build all debians on chroot without using APT, it works for amd64 too.

Had to:
install: apt install gcc-arm-linux-gnueabihf g++-arm-linux-gnueabihf
change toolchain file in platform condition to: if [ -e "toolchain/gcc/${MANIFEST}" ]; then

The changes I referenced are both to be applied to submodules/platform, so I'll put it in a separate PR


validate SSID prompt

This will check that the non-empty SSID is entered before prompting further.
Fixes: https://github.com/spark/particle-cli/issues/62
Error message is: >> please enter a valid SSID
Signed-off-by: Kenneth Lim<EMAIL_ADDRESS>

:+1: thanks!


Update concepts-lake-database.md

In the format, it does not include delta. We can create delta table as well in Lake Database as primarily it is stored in parquet only.

@bhadreshshiyal : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@gsaurer
Can you review this PR?
IMPORTANT: When this content is ready to merge, you must add #sign-off in a comment or the approval may get overlooked.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

#sign-off

@Jak-MS Please merge this PR.


Add docs for routing mode

Direct routing was added as part of #3182. This adds the API documentation to providers as well as other missing documentation.

/lgtm


Duplicate functions on windows

msbuild src\shogun\libshogun.vcxproj /m > build.txt
https://gist.github.com/qcrist/ab8b323802ea2420cdb2f1446a4b5beb
Visual Studio 2017 x64
Followed https://github.com/shogun-toolbox/shogun/blob/develop/appveyor.yml as close as I could, except with VS2017 instead
         T:\shogun\build\src\shogun\base\class_list.cpp(2608): error C2084: function 'shogun::CSGObject *__new_CSpecificityMeasure(shogun::EPrimitiveType)' already has a body [T:\shogun\build\src\shogun\libshogun.vcxproj]          T:\shogun\build\src\shogun\base\class_list.cpp(2608): fatal error C1003: error count exceeds 100; stopping compilation [T:\shogun\build\src\shogun\libshogun.vcxproj]     1557 Warning(s)     102 Error(s) Time Elapsed 00:13:09.24

caused by cmake-gui ^


feat: Create a new suite for releasepipelines tests

Description
Created a suite for the Release Pipelines e2e-tests in folder tests/release, any change will be in this tests/release/pipelines  will not trigger openshift CI  PR created for this.
RHTAPREL-373
Type of change

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

How Has This Been Tested?
Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration
Checklist:

[ ] I have performed a self-review of my code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] I have added meaningful description with JIRA/GitHub issue key(if applicable), for example HASSuiteDescribe("STONE-123456789 devfile source")
[ ] I have updated labels (if needed)

/test redhat-appstudio-e2e

/test redhat-appstudio-e2e

/lgtm

/lgtm


create_log_entries_from_revisions crashes if revisions do not have a specific page class

We've got some page types that were removed from the codebase, but the revisions still exist in the DB. Upgrading to 2.10 and running the create_log_entries_from_revisions command produces a hard crash.
Stack trace
  File "/lib/python3.7/site-packages/wagtail/core/management/commands/create_log_entries_from_revisions.py", line 26, in handle
    current_revision_as_page = revision.as_page_object()
  File "/lib/python3.7/site-packages/wagtail/core/models.py", line 1977, in as_page_object
    return self.page.specific.with_content_json(self.content_json)
  File "/lib/python3.7/site-packages/wagtail/core/models.py", line 1830, in with_content_json
    obj = self.specific_class.from_json(content_json)
AttributeError: 'NoneType' object has no attribute 'from_json'

Workaround
[revision.delete() for revision in PageRevision.objects.all() if not revision.page.specific_class]

Thanks for the report @markfinger! Fixed in #6322.


Refactor promiseQueue implementation

Fix unhandled rejections when function passed into enqueue method throws an error.
Refactor internal implementation so it create only one promise when someone enqueues new executor. Also it does not build entire promise chain after calling enqueue but rather chains next promise right before the next task starts.
Tests, of course.

Fixes #1044

This change is

PR is added to the queue for testing as 1. task. (925e479)

Test 112800850 (925e479) build started.

Please see comment but otherwise

Reviewed 2 of 2 files at r1.
Review status: all files reviewed at latest revision, 1 unresolved discussion.

thali/NextGeneration/promiseQueue.js, line 133 at r1 (raw file):

  var task = this._tasks.shift();
  if (task) {
    this._activePromise = task.promise.then(this._run, this._run);

Why do we need _activePromise? We always handle both outcomes from the then so we shouldn't get an unhandled error exception or anything like that, right? Just curious.

Comments from Reviewable

Test (Success) 112800850 build is completed (925e479)
See https://github.com/ThaliTester/TestResults/tree/112800850925e479_Refactor_promiseQueue_implementation_chapko/ for the logs

Test<PHONE_NUMBER>25e479(925e479) has failed
See https://github.com/ThaliTester/TestResults/tree/112800850925e479_Refactor_promiseQueue_implementation_chapko/ for the fail logs

Comments from Reviewable

Reviewed 2 of 2 files at r1.
Review status: all files reviewed at latest revision, 1 unresolved discussion.

Comments from Reviewable

PR is added to the queue for testing as 1. task. (f460826)

Test 112800850 (f460826) build started.

Review status: 1 of 2 files reviewed at latest revision, 1 unresolved discussion.

thali/NextGeneration/promiseQueue.js, line 133 at r1 (raw file):
Previously, yaronyg (Yaron Y Goland) wrote…
Why do we need _activePromise? We always handle both outcomes from the then so we shouldn't get an unhandled error exception or anything like that, right? Just curious.

Actually we don't need it. I just didn't notice that I don't use it anymore. So it can be safely removed.

Comments from Reviewable

Test (Success) 112800850 build is completed (f460826)
See https://github.com/ThaliTester/TestResults/tree/112800850f460826_Refactor_promiseQueue_implementation_chapko/ for the logs

Test 112800850f460826(f460826) has failed
See https://github.com/ThaliTester/TestResults/tree/112800850f460826_Refactor_promiseQueue_implementation_chapko/ for the fail logs

Reviewed 1 of 1 files at r2.
Review status: all files reviewed at latest revision, all discussions resolved.

Comments from Reviewable


[Question] Object Detection running with UMat and/or OpenCL target noticeably slower

Hey everyone
I have a question regarding the Transparent API / Preferable Target and hope someone can help me understand.
My Object Detection program takes a lot longer to process images when using
net.setPreferableTarget(cv2.dnn.DNN_TARGET_OPENCL_FP16)
or
image = cv2.imread(filePath, cv2.COLOR_BGR2RGB)  uMat = cv2.UMat(image)
I've created 4 benchmark programs running sequentually, processing the same 10 .jpg files.
My Baseline is a standard openCV object detection programm, not using the setPreferableTarget or UMat class for images.
The second one sets the setPreferableTarget to cv2.dnn.DNN_TARGET_OPENCL_FP16
The third converts images into UMat objects
The fourth sets the setPreferableTarget to cv2.dnn.DNN_TARGET_OPENCL_FP16 and converts images into UMat objects.
I always measured the full processing time starting before I read the image, ending after drawing the labels (excluding writing the output image or detection log) as well as the model inference time with
t, _ = net.getPerfProfile()  infTime = (t /  cv2.getTickFrequency())
The collected output is as follows:
Benchmark One Full Processing Time: 2.78063s
Benchmark One Model Inference Time: 1.030843s
Benchmark Two Full Processing Time: 3.2567s
Benchmark Two Model Inference Time: 1.12314s
Benchmark Three Full Processing Time: 12.76886s
Benchmark Three Model Inference Time: 10.83879s
Benchmark Four Full Processing Time: 13.43161047s
Benchmark Four Model Inference Time: 11.27375169s
Is there such a large gap between CPU and GPU execution because of the data transferrel between the processing units? Am I missing something crucial?
If this big gap difference can be explained by the data transfer, is there a possibility to "bundle" my workload to reduce the amount of transferrals?
I can provide the full code for these benchmark Programs if they should be helpful.
Thanks in advance!

There are several reasons for why an execution via VC4CL could be slow, e.g.

The kernel is not very optimized
The kernel is very memory-bound (which is rather slow on the VideoCore IV GPU, esp. read/write-memory)
The measurement includes the kernel compilation time, which can take a while
...

As to bundling the workload, I have no clue about OpenCV, but I think as the actual OpenCL client, it would have to be done there...


Change Survivor Interaction Map Tiles to Google Maps

Description
The map at https://crisiscleanup.org/s/[token] is not currently Google Maps. To help survivors quickly identify their location, change the map Tiles to Google Maps.

Confirmed.


Support for fixed_string named getter.

Fixed.
@jeanga_fr thank you for noticing

Using this latest version, I get an MSVC internal compiler error.
Reported it here: https://developercommunity.visualstudio.com/content/problem/973790/fatal-error-c1001-internal-compiler-error-compiler.html


[BEAM-10673] Add public methods to RetryConfiguration.Builder in DynamoDBIO (aws2)

io.aws2.dynamodb.DynamoDBIO.RetryConfiguration.Builder did not have a public build method. Also, validation was dropped in the conversion to the AutoValue builder pattern. (Related PR: #8987)
This PR makes relevant methods public, and implemented the validation in the build method in accordance to the AutoValue guidelines.
Documentation was updated accordingly.

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Choose reviewer(s) and mention them in a comment (R: @username).
[x] Format the pull request title like [BEAM-XXX] Fixes bug in ApproximateQuantiles, where you replace BEAM-XXX with the appropriate JIRA issue, if applicable. This will automatically link the pull request to the issue.
[ ] Update CHANGES.md with noteworthy changes.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

See the Contributor Guide for more tips on how to make review process smoother.

GitHub Actions Tests Status (on master branch)

See CI.md for more information about GitHub Actions CI.

Merged now manually,  closing and resolving issue.


[Feedback] Nice concept! Couple QOL ideas.

It's funny that you just published this today and I was looking for something close to this. I like where you're going with this! I wanted to offer a couple quality of life ideas.

I think it would be really great to have the extension automatically generate class (model) level PHPDocs with @property  attributes so each of the columns can be validated with intellisense. This shouldn't be too difficult to work into your extension since you're already handling columns and their casting types. This is something I manually do now but it is tedious.

In addition to columns in PHPDocs, is relations. This is more advanced but could be helpful for developers to streamline access to their models . @property-read is a good PHPDoc choice for these.

Perhaps expose ignored fillable columns to configuration as an array of patterns.

The configuration options are available in the new version now.

a couple of things as well,

is this the correct description ? https://github.com/sandaru1/laravel-eloquent-model-attributes/blob/062152e7c8cc79172de3b37fb3971dd6b9f59439/package.json#L109

get the plural table name from the current file class name (get it with either php-parser or file symbols) if nothing is added to the input box.

PHPDoc is option is added. You can use the command : "Insert PHPDoc for Eloquent Model Attributes".


Archive garden-windows-ci repo

Recently we had some issues with the windows-rootfs pipeline failing because of its references to both garden-windows-ci and the WG CI repo. The conflict arose because we have some opsfiles that have the same name, but different functionality. We were able to fix this by removing the bad opsfile, but it would be better if we had one source of truth for our CI pipelines.
Since garden-windows-ci is no longer referenced by our pipelines and the relevant files are now in the WG CI repo, we should archive garden-windows-ci.

This is for my working group and I approve. I can't remember if I need to wait for full TOC approval.

Usually, I take this into the TOC meeting but if we get another approval by @rkoster, @stephanme or @ChrisMcGowan we don't need to wait.

You should remove the project from https://github.com/cloudfoundry/community/blob/main/toc/working-groups/app-runtime-platform.md (repositories of area Garden Containers) as well.

@stephanme thanks for the good catch. The request has been implemented and we have enough approvals I will merge to sped up things.


Building on Linux fails

Looks like this does not build for a linux target as of Go 1.5.1
/src/src/vendor/github.com/davecheney/xattr/xattr_linux.go:18: undefined: getxattr
/src/src/vendor/github.com/davecheney/xattr/xattr_linux.go:24: undefined: getxattr
/src/src/vendor/github.com/davecheney/xattr/xattr_linux.go:35: undefined: listxattr
/src/src/vendor/github.com/davecheney/xattr/xattr_linux.go:41: undefined: listxattr
/src/src/vendor/github.com/davecheney/xattr/xattr_linux.go:51: undefined: setxattr
/src/src/vendor/github.com/davecheney/xattr/xattr_linux.go:60: undefined: removexattr

There does not seem to be an equivalent of https://github.com/davecheney/xattr/blob/master/syscall_darwin.go for linux?

Thanks for your error report. I cannot reproduce the issue, the package builds cleanly on linux. Can you please post the output of
  go build -x github.com/davecheney/xattr

https://github.com/davecheney/xattr/blob/master/xattr_linux.go exists

$:/tmp# go build -x github.com/davecheney/xattr
WORK=/tmp/go-build251114186
mkdir -p $WORK/github.com/davecheney/xattr/_obj/
mkdir -p $WORK/github.com/davecheney/
cd /opt/gopath/src/github.com/davecheney/xattr
/opt/go/pkg/tool/linux_amd64/6g -o $WORK/github.com/davecheney/xattr.a -trimpath $WORK -p github.com/davecheney/xattr -complete -D _/opt/gopath/src/github.com/davecheney/xattr -I $WORK -pack ./xattr.go ./xattr_linux.go
# github.com/davecheney/xattr
/opt/gopath/src/github.com/davecheney/xattr/xattr_linux.go:18: undefined: getxattr
/opt/gopath/src/github.com/davecheney/xattr/xattr_linux.go:24: undefined: getxattr
/opt/gopath/src/github.com/davecheney/xattr/xattr_linux.go:35: undefined: listxattr
/opt/gopath/src/github.com/davecheney/xattr/xattr_linux.go:41: undefined: listxattr
/opt/gopath/src/github.com/davecheney/xattr/xattr_linux.go:51: undefined: setxattr
/opt/gopath/src/github.com/davecheney/xattr/xattr_linux.go:60: undefined: removexattr

Looking on GitHub, while https://github.com/davecheney/xattr/blob/master/xattr_linux.go exists, it calls getxattr  et.al. which for OS-X is defined in https://github.com/davecheney/xattr/blob/master/syscall_darwin.go as I linked above but does not include a syscall_linux.go (I assume?) in my checkout.

I'm not sure what is going on. Which platform are you compiling on ?

Hang in, your build is calling 6g. 6h was removed in go 1.5. Are you sure you installed go 1.5 correctly? Are you sure you do not have GOROOT set?

Fair point, there may be an issue with the image I am using - it is a current ubuntu docker image with a Go 1.5.1 install that I need to look at.
However, I can reproduce using a fairly stock install. The version of Go from the package is pretty old in the script below but it produces the same error.
docker run -ti ubuntu
$ apt-get update && apt-get install -y golang git
$ cd /tmp
$ mkdir test
$ cd test
$ export GOPATH=$PWD
$ go get github.com/davecheney/xattr
# github.com/davecheney/xattr
src/github.com/davecheney/xattr/xattr_linux.go:18: undefined: getxattr
src/github.com/davecheney/xattr/xattr_linux.go:24: undefined: getxattr
src/github.com/davecheney/xattr/xattr_linux.go:35: undefined: listxattr
src/github.com/davecheney/xattr/xattr_linux.go:41: undefined: listxattr
src/github.com/davecheney/xattr/xattr_linux.go:51: undefined: setxattr
src/github.com/davecheney/xattr/xattr_linux.go:60: undefined: removexattr

If you are using a u until container, it may be that go is installed via apt which will be ahead of any custom install in $PATH.
You did not mention which version of Ubuntu you were using my but unless it is the latest, and not upset released, you will not receive go 1.5.1

On 7 Oct 2015, at 22:01, Rahul Powar<EMAIL_ADDRESS>wrote:
Fair point, there may be an issue with the image I am using - it is a current ubuntu docker image with a Go 1.5.1 install that I need to look at.
However, I can reproduce using a fairly stock install. The version of Go from the package is pretty old in the script below but it produces the same error.
docker run -ti ubuntu
$ apt-get update && apt-get install -y golang git
$ cd /tmp
$ mkdir test
$ cd test
$ export GOPATH=$PWD
$ go get github.com/davecheney/xattr
github.com/davecheney/xattr
src/github.com/davecheney/xattr/xattr_linux.go:18: undefined: getxattr
src/github.com/davecheney/xattr/xattr_linux.go:24: undefined: getxattr
src/github.com/davecheney/xattr/xattr_linux.go:35: undefined: listxattr
src/github.com/davecheney/xattr/xattr_linux.go:41: undefined: listxattr
src/github.com/davecheney/xattr/xattr_linux.go:51: undefined: setxattr
src/github.com/davecheney/xattr/xattr_linux.go:60: undefined: removexattr
—
Reply to this email directly or view it on GitHub.

The original issue was with a custom image based on 15.04. Not likely a path issue as in this image, Go was not installed via apt but irrespective:
The script I linked above using the stock ubuntu  should work, correct? You can try it if you have a docker host. Ends up using 14.04

Scratch my previous output log. The host I was on had an old cache of the OS image running a Go 1.4.x, hence that output.
$:/tmp# go get -x github.com/davecheney/xattr
cd .
git clone https://github.com/davecheney/xattr /opt/gopath/src/github.com/davecheney/xattr
cd /opt/gopath/src/github.com/davecheney/xattr
git show-ref
cd /opt/gopath/src/github.com/davecheney/xattr
git checkout master
WORK=/tmp/go-build797213813
mkdir -p $WORK/github.com/davecheney/xattr/_obj/
mkdir -p $WORK/github.com/davecheney/
cd /opt/gopath/src/github.com/davecheney/xattr
/opt/go/pkg/tool/linux_amd64/compile -o $WORK/github.com/davecheney/xattr.a -trimpath $WORK -p github.com/davecheney/xattr -complete -buildid e471c92019b3634cd83e09fc31a8dc831d21f41f -D _/opt/gopath/src/github.com/davecheney/xattr -I $WORK -pack ./xattr.go ./xattr_linux.go
# github.com/davecheney/xattr
/opt/gopath/src/github.com/davecheney/xattr/xattr_linux.go:18: undefined: getxattr
/opt/gopath/src/github.com/davecheney/xattr/xattr_linux.go:24: undefined: getxattr
/opt/gopath/src/github.com/davecheney/xattr/xattr_linux.go:35: undefined: listxattr
/opt/gopath/src/github.com/davecheney/xattr/xattr_linux.go:41: undefined: listxattr
/opt/gopath/src/github.com/davecheney/xattr/xattr_linux.go:51: undefined: setxattr
/opt/gopath/src/github.com/davecheney/xattr/xattr_linux.go:60: undefined: removexattr

Hello,
I'm sorry this was my mistake, there were local commits from two years that I had not pushed.
Given that I last worked on this package back in 2013, i'd regard it with deep suspicion.
If you need extended attr support, I recommend adding to to the golang.org/x/sys repository.


Unify trait impl for reference wrapper types

This changes makes sure that no implementation is forgotten and deduplicates some code.

Looks good to me, thanks! Feel free to merge.

I added better implementations for impl<T> X for Pin<T>. I couldn't get it to work in the macro, so I re-introduced some repetition. :|

It's fine. Still better than before. :)


NHANES accelerometer data analysis

This is for the webinar on 9/2/2020.
Here is the working draft of my google slides for this presentation.

Manuscripts

Troiano, Richard P., et al. "Physical activity in the United States measured by accelerometer." Medicine and science in sports and exercise 40.1 (2008): 181.
Mooer, Hyatt E., et al. "Potential corner case cautions regarding publicly available implementations of the National Cancer Institute’s nonwear/wear classification algorithm for accelerometer data." PLOS ONE 13.12 (2018): 1-11.

NHANES Datasets

NHANES Questionnaires, Datasets, and Related Documentation (CDC website)
NHANES 2003-2004 links (CDC link)
NHANES 2003-2004 physical activity monitor data dictionary ((PAWRAW_C.htm))

Implementations of NCI's nonwear algorithm
National Cancer Institute's SAS code

Website: 	https://epi.grants.cancer.gov/nhanes_pam/create.html
Macro (/day):	https://epi.grants.cancer.gov/nhanes_pam/create.pam_perday.sas

R Accelerometry project

Website: 	https://cran.r-project.org/web/packages/accelerometry/index.html
Doc (.pdf): 	https://cran.r-project.org/web/packages/accelerometry/accelerometry.pdf

ActiLife

White paper:	What is the difference between the wear time validation algorithms

FYI, there's a processed data package for NHANES 0304/0506 PA data, developed by Andrew Leroux.
https://oslerinhealth.org/package/rnhanesdata

Closing this issue now. It is cited in https://github.com/wadpac/oss-dev-webinar-series-pb-field, so people should be able to find this issue.


✨ Added "startsWith" check to the match helper

Added "startsWith" string comparison to the match helper.
I think this simple check will unlock and simplify a lot of use cases for theme developers:
Group logic when dealing with similar custom settings:
{{!-- Suppose layout could be: "List full", "List narrow", "Grid 2 cols", "Grid 3 cols", "Grid masonry" --}}
{{#match @custom.layout "startsWith" "Grid"}}
    has-grid
{{/match}}

Check if a URL is external:
{{^match (url absolute="true") "startsWith" @site.url }}
    is-external
{{/match}}

Simulate nested tags:
{{#match slug "startsWith" "hash-parent-tag-" }}
    {{> template-parent }}
{{/match}}

Help implementing nested navigation:
{{#match label "startsWith" "--" }}
    is-child-nav-item
{{/match}}

Add a badge to a navigation item:
{{#match label "startsWith" "[NEW]" }}
    <span>{{t "NEW"}}</span>
{{/match}}

Please include a description of your change & check your PR against this list, thanks!

[x] There's a clear use-case for this code change, explained below
[x] Commit message has a short title & references relevant issues
[x] The build will pass (run yarn test:all and yarn lint)

We appreciate your contribution!

🤖 Generated by Copilot at 082c4f9
This pull request adds support for the startsWith operator to the match template helper, which allows matching data values against string prefixes. It also adds unit tests for the new operator in the match.test.js file.

@ErisDS Any chance we can get this to upstream?


Surround if block with curly brackets.

.. because it drives me nuts =P

Whoa. That passed lint??


[3656] allow overriding nonce factory

The original implementation create separate instances of nonce factory for each service. That would mean a possibility of nonce collision.
I guess I’d be happy with a single instance of nonce factory for all services.

is this not fixed now?
in the current code I see
public class BTCMarketsBaseService extends BaseExchangeService implements BaseService {

  ...
  protected final SynchronizedValueFactory<Long> nonceFactory;

  public BTCMarketsBaseService(Exchange exchange) {
    super(exchange);
    ...
    this.nonceFactory = exchange.getNonceFactory();

I'm closing this assuming it got fixed


Produces confusing error on empty input

Parsing an empty input yields:

Object parsing failed, but no additional info was provided

I'd argue that this should just return an empty Vec<Node>, but if you insist on an error it should be more descriptive.
Test:
#[test]
fn test_empty_input() -> Result<()> {
    let tokens = quote! {};
    let nodes = parse2(tokens)?;

    assert!(nodes.is_empty());

    Ok(())
}

Yes, i agree with you. It should return empty Vec instead of returning error.


Add ability to process journal loads using nlmtas

we allow for locating and updating journals via nmlta in addition to issns. some other updates:
change in logic for processing updates in the loader - now we control a little more atomically. this involves a read for each existing journal object.  before, it appeared that issns were not being set on the "found" journal object, so that updates were always done for PMC Method A journals, even if not needed.
added a hacky toJournal method for PCMSource, so that we could pass a regular Journal object to the java client's  createResource() method. without this hack, new PMCSource journals would not get created, as the client would not recognize them as PassEntities.
added some extra counting of different results outcomes for the loader

I ran the Medline loader, then analyzed the results.  Got:
12362 online issns
24063 print issns
86 bad issns
3606 records with nonexistant issns
2504 pmc A journals
Ignore the "bad issns" for the moment, those are ISSNs without a PubType prefix, and I didn't run the PMC data yet.  The interesting part is 3606 records without ISSNs.  I looked at one at random:
JrId: 27664
JournalTitle: Quarterly review of surgery, obstetrics and gynecology
MedAbbr: Q Rev Surg Obstet Gynecol
ISSN (Print): 0096-543X
ISSN (Online):
IsoAbbr: Q Rev Surg Obstet Gynecol
NlmId: 60020130R

Sure enough, the corresponding Fedora object had no ISSN.  So somehow the print ISSN didn't make it in.  Need to check to see what ES has to say to confirm.


Add failing test case to illustrate #98

Add a test case to better explain the issue in #98

It should fail, right?

Yes, this test fails on purpose to demonstrate the problem.

if the following line is added to fieldset filter the test will pass
     'sites'  => [Site::LINK_POSTS, Site::ATTRIBUTE_NAME],

So basically it's about filter logic

I've updated the test


Rename/array type

Renames numba_dppy.dppy_array_type to numba_dppy.core.types.array_type. Also renames DPPYArray and DPPYArrayModel to just Array and ArrayModel.

@PokhodenkoSA let me know if these changes impact your work. I modeled how numba's directories are organized. We will slowly get similar structure for our repo too.

@PokhodenkoSA let me know if these changes impact your work.

I think you already modified my code as it was in master in dpnp_ndarray folder.


[FEATURE REQUEST] Allow Realtime Database connection without authentication

This is more of an anti-feature request, I suppose.
I've just begun porting one my games, https://juanitogan.itch.io/sendit-soccer, from another engine to Godot.  The first task, however, is to connect to my Realtime Database like I currently am doing in JavaScript.  That is, without auth.
The only purpose of the DB is to help initiate P2P connections, thus, no auth needed or wanted.  It has other protections against possible abuse.  And, well, I'm still in prototype mode anyhow.
I had thought this tool would allow the same type of connection judging by Activation  steps 3 and 4 (no mention of auth there)... but, no go.  I thought this was a bug at first but the Realtime Database wiki page (and the code) makes it clear auth is needed first.
Thank you.

@juanitogan good question. Some of those pages need to be updated, so thank you for bringing it to our attention.
One way around it is to have the app use anonymous authentication in the background. It should fix the issue of needed auth but not forcing a user to log in.
Let me check with the team to see if there is a different way.
Thanks,
Chuck

We can support that, but I don't know if we should.  I've seen it happen, blew out someone's database immediately and cost them thousands. I recommend anonymous authentication like suggested. If you really think it's necessary to support, I can make a PR for it, but I really would rather not.

In my opinion, feature decisions are up to the service provider and the dev, not the middleman tool.  The risks you mention simply don't apply to my use case.

They absolutely do apply to your use case. For the time being, I'm not going to implement this because of potential liability concerns. Once I've found an appropriate liability disclaimer that I can add to the repo, I'll add the ability for you to do this, but for now, I will not permit this type of functionality in my codebase, not until I can be sure I cannot be held even partially responsible.

Alright. I've talked to someone and I believe the license will cover us. We'll implement this, even though it can only be a bad thing. I don't know how long it'll take. This likely will be a boolean export on the main Firebase scene that can be checked, something like, "Intentionally leave yourself vulnerable to attack?" Hm, too wordy. We'll figure something out, though it will take a little bit of time to remove all headers and query params.

Hi @juanitogan ,
I really want to add a little sidenote to my collaborators' answers. Please read it carefully.
Let's just put aside the fact that phrases like "this thing does not apply my use case" or "I don't need this" won't be for sure the reasons why we will add new features, mostly because the project is open source and any customization can be done with very little effort (depending on the personal feature of course), especially with stuff regarding avoding security.
The only reason we are not likely to support this feature (even though we will in the end, but with tons of disclaimers anyway), it's that we really care about security of users who use our plugin, and not just from a "responsability" perspective (I agree with you on the middleman tool), but mostly because bulding this from scratch as a community we have understood how many critical issues could be avoided following the secure path Firebase itself suggests, expecially for unexperienced users.
With that said, we will work on this anyway, since from our perspective we are clear and it's just an implementation.
Now, I don't know what "other protections" you are actually using (the one and only protection should be Firebase rules themselves, and authentication), for sure they are working since you are really confindent and I belive they do, but please give more attention to security anyway.
The project you shared (your game on itch) just exposes everything (and even more) required to give you some bad times with Firebase. Of course I won't share anything here, but it's not even something that requires any kind of hacking abilities. This is the typical case were some basic security like Authentication could save a couple of nights. I won't go in further details for obvious reasons , instead would like to contact you privately on this one but don't know how.
I'm sure you won't miss this next time, switching from Phaser to Godot.

Why would you need a license or disclaimers to cover services from Google, and use cases they have already found valid and support with their own tools?  Disposable data is what it is.  You may choose to build a fence around it, but Google and I have other priorities with keeping barriers low... for whatever reasons us clueless people have for not protecting our trash heaps better.
But, okay, as long as the group here is willing, I thank you for not making me fork this to provide a more complete tool.
If I knew I was in for such lectures and desires to police my priorities, I would have skipped this abuse and just forked it silently.

We need the license/disclaimer to cover us from your usage of it. When you are hacked and lose thousands of dollars, we need to be ensured you can't sue us for providing the tool that you shot yourself in the foot with. It's clear you don't really understand how any of this works, so yeah, we'll lecture you because we, the folks who know a lot better than you, about the mistakes you're making, because we don't want you to put yourself into that situation.

"we, the folks who know a lot better than you"  :rofl:
And, yet, the concept of disposable data has clearly eluded you.
You have revealed your true self.  Enjoy your totally awesome and superior selves.  I've got work to do.  I disabled the auth in this code hours ago.

But, okay, as long as the group here is willing, I thank you for not making me fork this to provide a more complete tool.
If I knew I was in for such lectures and desires to police my priorities, I would have skipped this abuse and just forked it silently.

Please note that this type of response is uncalled for and falls under our code of conduct. Our team tries hard to take every request and make it work, but there are just things we can't do.
https://github.com/GodotNuts/GodotFirebase/blob/main/CODE_OF_CONDUCT.md


proxylite: Add HTTPS backend support

This changes the internal route syntax.

@nickbp @gabrielhartmann  Ready for review.

Need to rebase after this documentation PR goes through: https://github.com/mesosphere/dcos-commons/pull/600

@mesosphere-ci Retest this please.

retest this please

@mohitsoni @nickbp @gabrielhartmann can someone merge?


is_incremental() macro returns false

Is this a new bug in dbt-core?

[X] I believe this is a new bug in dbt-core
[X] I have searched the existing issues, and I could not find an existing issue for this bug

Current Behavior
is_incrmental() macro returns False

Expected Behavior
is_incremental() should return true
Steps To Reproduce

Relevant log output
No response
Environment
- OS: dbt-cloud
- Python:
- dbt: latest version

Which database adapter are you using with dbt?
bigquery
Additional Context
No response

Hey @jschintz-nytimes 👋
I saw you just closed this, so I'm assuming that not all of the following held true:

The model must already exist in the database
The destination table already exists in the database
The full-refresh flag is not passed
The running model is configured with materialized='incremental'


Queue for incoming activities

For now, Fedify has the queue for outgoing activities, but there's no queue for the opposite.  For better reliability, incoming activities also should be queued.

One thing I'm concerned about is that if the inbox listener is deferred, it can't provide a RequestContext, only a Context. 🤔

I can see it making sense to have a retry queue for this!


Electron Forge v5 -- Built mac app won't run. V6 works

Preflight Checklist

[x] I have read the contribution documentation for this project.
[x] I agree to follow the code of conduct that this project follows, as appropriate.
[x] I have searched the issue tracker for a bug that matches the one I want to file, without success.

Issue Details

Electron Forge Version:
5.2.4
Electron Version:

That command didn't work but version is 8.2.0
Tejass-MacBook-Pro:my-new-app tejasmanohar$ npm ls electron
<EMAIL_ADDRESS>/Users/tejasmanohar/dev/src/github.com/carrytravel/my-new-app
└─┬<EMAIL_ADDRESS>  └──<EMAIL_ADDRESS>
Error --
Tejass-MacBook-Pro:my-new-app tejasmanohar$ node_modules/.bin/electron --version
(electron) 'setName function' is deprecated and will be removed. Please use 'name property' instead.
App threw an error during load
Error: Cannot find module '/Users/tejasmanohar/dev/src/github.com/carrytravel/my-new-app/--version'
Require stack:
- /Users/tejasmanohar/dev/src/github.com/carrytravel/my-new-app/node_modules/electron-prebuilt-compile/lib/es6-init.js
- /Users/tejasmanohar/dev/src/github.com/carrytravel/my-new-app/node_modules/electron/dist/Electron.app/Contents/Resources/default_app.asar/main.js

Operating System:
MacOS 10.14.6
Last Known Working Electron Forge version::
Apps built with Electron Forge v6 seem to work but Electron Forge v5 doesn't
I am moving down to Electron Forge v5 after using v6 templates and having trouble with VSCode debugger with v6

Expected Behavior
I'd expect the app to start successfully when I build it with npm run package. App was bootstrapped via npx electron-forge init my-new-app.
Actual Behavior
Tejass-MacBook-Pro:my-new-app tejasmanohar$ ./out/my-new-app-darwin-x64/my-new-app.app/Contents/MacOS/my-new-app
Trace/BPT trap: 5

Crash log -- https://gist.github.com/tejasmanohar/01f64bd9b612ee0c20d387ae71a72241
To Reproduce
This is the default bootstrapped app with Electron Forge v5

We're not fixing any bugs in Electron Forge 5, use 6 instead.


item category is mapped incorrectly

I found that item category is not populated in GA4 purchase when using this template.

This template assume that EE category it's separate by / character. This is the line of code that split the category string:
const category = i.category ? i.category.split('/') : [];
I've found many cases where the separator it's a > character. It could be easily fixed by using a regex that check for both characters.
@sahava do you think is possibile to make this change?
Thanks for your precious work!

Hi,
Per the Enhanced Ecommerce specification, category separator is / – that's the only way to make it populate the Category Level 1, Category Level 2, etc. dimensions.


Blank document opens when opening pdf from firefox

As the title says. This app will not view pdfs opened from the web.

What happens specifically? I need some details or logs... Thanks!

I can add some info to this. Versions of PDF Viewer Plus up to 2.4 work fine with Firefox (as long as you set the storage permission for the app; if you don't, you get the blank screen as shown above). Since v2.5 it stopped working with Firefox, even if you've set the storage permission correctly.
I've only found Firefox to have this problem though, opening PDFs from other apps work fine.
These are the commits from v2.4 to 2.5, so it looks like this was probably "broken" in #4?
A workaround for now is to go back to version 2.4 (installed from the releases page).

I've made a patch for this issues, but to fix it I made the app download the pdf file.
From what I read here it could be possible to make firefox download the file before triggering an intent to call PdfViewer ? If that is possible I guess that should be better, if it is not possible I think my solution could be acceptable.
I used a AsyncTask as mentionned here https://github.com/barteksc/AndroidPdfViewer/issues/671#issuecomment-417570679
The thing is that PdfViewer will need to request INTERNET permission.
I can't do a push request right now, but when I will be able I will ;)

This doesn't explain why it used to work, though? Did you have the internet permission before and pull it?

I don't really understand how it could have work bedore. PdfViewer don't have the possibility to download a PDF file from an URI, and if it was working that would have mean that Firefox was downloading the PDF file before triggering an intent to launch it in PdfViewer.
The only way I see patching this is making PdfViewer able to download a pdf file, I currently did a patch that was working fine with HTTP request, but I just didn't know how to make it work for HTTPS (of course making it work with HTTPS would mean making it safe by verifying TLS certificate from the system AC store).
I will still work on it, if anyone want to help, I would be please to have any help for the HTTPS support.
I don't know if giving internet and write external storage permission to PdfViewer is something that is wanted by the community (and main developer) if it can solve this issue ?

that would have mean that Firefox was downloading the PDF file before triggering an intent to launch it in PdfViewer.

I believe that is the case, yes


Compare two graphs side by side

Title

Oof. Searched for "graph comparison", not "model comparison" :P


Typo in location for asia-southeast1-c

Seems like a little copy pasta got in this "location" string.

Hey @nathanstilwell, ah yes. Good spot! Thank you!


ParaKubeVirtualization API

Is your feature request related to a problem? Please describe:
I have a use-case where an API that allows a KubeVirt VM to discover details about itself could be quite useful. In my case, the KubeVirt VMs can talk directly to the KubeVirt API for things like device hotplug. For that to happen, the guest needs to know its VMI name and the k8s namespace that it's in.
I noticed KubeVirt adds this information as the cloud-init instance-id (where it's in the form vmi-name.namespace -- sudo cloud-init query v1). However, that requires my guest OS to include and support cloud-init. So for guests that can't or won't support cloud-init, I'm not sure there's any programmatic way for the guest to discover this information about itself (without adding some additional automation somewhere in the cluster or the guest or both).
Describe the solution you'd like:
It would be cool if there was a "ParaKubeVirtualization" API, for example my KubeVirt VM could hit like a whoami endpoint, which could return the VMI yaml or a more reduced set of information including the VMI name and namespaces.

@connorkuehl I'm not sure if a generic API is a good idea. What I mean is that we might not want to expose this kind of information to the VM.
Would be acceptable for you to generate a configmap together with the VM specification and use this? This could be something built on top of KubeVirt.
Another way could be that the qemu-guest-agent generate a file with the needed information. However, this approach has the same drawback as cloud-init that you need to install an additional tool inside the VM.

@connorkuehl @alicefr We do support a downwardAPI volume, you can have a look at an example test https://github.com/kubevirt/kubevirt/blob/main/tests/virtiofs/config.go#L232-L265

Thanks! That looks fascinating but I don't think I can depend on every guest OS having a kernel with virtioFS support.

@connorkuehl It works with disk as well.


Allow PermissionChecker to be a plugin

Is your feature request related to a problem? Please describe.
No
Describe the solution you'd like
In some use cases, we need to provide our own permission checker instead of relying on the one Alluxio provided.
So we want a change to design a PermissionCheckerFactory. By default, it will continue to use the DeafultPermissionChecker. When there is a configuration and a plugin is loaded, then override it with the provided PermissionChecker.
Describe alternatives you've considered
The default Permission checker doesn't work.
Urgency
Without this feature, some deployment cannot be done
Additional context
n/a

@beinan

Thanks @shangxinli for this great suggestion. That aligns with our engineer development principles, to let securities and other users likely customize area pluggable. Will keep this in mind when developing future products or major product upgrades.


New PR to test SignCLA at 2019-08-14 21:59:00.86262336 -0700 PDT m=+3.336692886

Body of PR

corp-pass-fork-staging: test succeeded


nixos/ups: Fix upsmon.settings.RUN_AS_USER default

Description of changes
Without being mkDefault, changing this setting is cumbersome for users. There is no reason to attempt to push users away from changing this setting.
Fixes #318582
Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
24.11 Release Notes (or backporting 23.11 and 24.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

@yodaldevoid: Do you have a working config with non-root user?

@yodaldevoid: Do you have a working config with non-root user?

Yes, you can use lib.mkForce to force the setting. e.g.
power.ups.upsmon.settings.RUN_AS_USER = lib.mkForce config.users.users.nutmonitor.name;

@yodaldevoid: Do you have a working config with non-root user?

Yes, on stable you can use lib.mkForce to force the setting. e.g.
power.ups.upsmon.settings.RUN_AS_USER = lib.mkForce config.users.users.nutmonitor.name;

So there's no permission errors resulting from that? I assumed it was running as root for a reason (permissions). If it's that easy to run as non-root, why don't we create that "nutmonitor" user in NixOS by default and have everyone use that? What's the use case for users to be able to run NUT as arbitrary user?

So there's no permission errors resulting from that?
Nope, no issues. It even has access to a root-only password file at startup.

I assumed it was running as root for a reason (permissions). If it's that easy to run as non-root, why don't we create that "nutmonitor" user in NixOS by default and have everyone use that? What's the use case for users to be able to run NUT as arbitrary user?
I can only make conjectures at this point, but I figure it was the easier thing at the time. If a user was generated and used, I know I would want a way to configure that (at least to turn it off to replace with my custom configured user). At that point, with how easy it is to generate users, why no just have the user do it themself.

If a user was generated and used, I know I would want a way to configure that (at least to turn it off to replace with my custom configured user). At that point, with how easy it is to generate users, why no just have the user do it themself.

That doesn't really explain a use case except for "because we can". What exactly do you want to customize? User name? Additional groups? (That can be done by extending/overriding a base user provided by NixOS.)
After reading https://networkupstools.org/docs/man/upsmon.conf.html (the RUN_AS_USER part), I think this patch is going in the wrong direction.
The way I think it should work is that NixOS generates a user (e.g. "nutmon" as suggested by upstream) and ensures the config file is readable for that user, so that reloading works. (This is actually how it works today, except that the generated user is "root".) Having to use lib.mkForce to change the user from under NixOS is a good thing; it implies a bit of "I know what I'm doing and accept the risks". Because this patch makes it easier to break the ownership of the config file.

I realized that we can turn this around; we can make RUN_AS_USER easily configurable, but then the permissions on the config files must automatically follow. I'm not a fan of making them easily diverge.


Os { code: 1, kind: PermissionDenied, message: "Operation not permitted" }

I have read the issues here, change the surge-ping = "0.7.3" to surge-ping = "0.7.4", but I got another error.
error: failed to select a version for the requirement `surge-ping = "^0.7.4"`
candidate versions found which didn't match: 0.7.3, 0.7.2, 0.7.1, ...

Can you tell me more about your os? In addition, due to some problems, version 0.7.4 has been deprecated, and now version 0.8.0 has been released, you can try it.

Hi, I still get error message IOError(Os { code: 1, kind: PermissionDenied, message: "Operation not permitted" }) when I using the 0.8.0, my OS is popos 22.04 with kernel 6.2.6 .

you can use the setcap command to give your program the CAP_NET_RAW capability
sudo setcap cap_net_raw+ep /path/to/your_bin


Create top_k_frequent_elements.cpp

Top K Frequent Elements in C++
#164

Please review the changes @rudrakshi99

Update the list in the heap folder.

Updated the list. Please check @rudrakshi99 @tarun26091999

@rudrakshi99 @tarun26091999 Please check


bootstrap os 0.9

first version of our  boostrap os

use existing OS
let a lot still be local (just to win time)
we need tools to do this automatically & the output needs to be a tgz which can be expanded on USB stick or any VM
we need support for arm based devices (pi) & 64bit machines

for install need something like

https://github.com/Jumpscale/jumpscale_core8/blob/master/lib/JumpScale/tools/cuisine_/CuisineInstaller.py#L646

all needs to get into

https://github.com/g8os/builder/tree/master

starting points, examples, ...

some specs
https://github.com/g8os/home/blob/master/specs/sandboxing.md

@zaibon @muhamadazmy how far are we here
documentation how to build
example scripts in the repo?

@muhamadazmy make sure we have KVM tools, libvirt, docker tools, openvswitch, mc, ...

sure

Please add documentation and close