text:
dolichyl diphosphate biosynthetic process (or merge)

GO term ID and Label

dolichyl diphosphate biosynthetic process

Reason for deprecation

MF in BP (single gene product)

"Replace by" term (ID and label)
If all annotations can safely be moved to that term

I moved the PomBase annotation up to parent.

not sure if we need this but needs to be part of a more systematic review, it isn't causing any immediate problems

@

repo: geneontology/go-ontology


text:
Change label 'motor activity' -> 'cytoskeletal motor activity'

After discussion with @thomaspd
While 'molecular moteor' can include nucleic acid motors, polymerisation motors, rotary motors, etc (see https://en.wikipedia.org/wiki/Molecular_motor), in GO grouping all these classes makes too broad a classification. I will change the label of  'motor activity' to 'cytoskeletal motor activity' (and related terms), and move out 'chromatin extrusion motor activity'.
Thanks, Pascale

Also changed:
id: GO:0140659
-name: motor activity regulator activity
+name: cytoskeletal motor regulator activity
id: GO:0140660
-name: motor activity activator activity
+name: cytoskeletal motor activator activity
id: GO:0140661
-name: motor activity inhibitor activity
+name: cytoskeletal motor inhibitor activity

source: gharchive/issue

license_type: permissive

license_source: github-api


text:
Runtime panic with CloseNotify

The integration test sometimes fails with a runtime panic.
e.g. https://travis-ci.org/gengo/grpc-gateway/jobs/111913217
$ env GLOG_logtostderr=1 go test -v github.com/gengo/grpc-gateway/...
=== RUN   TestIntegration
E0226 02:27:10.407340    5251 a_bit_of_everything.go:67] nested:<name:"hoge" amount:10 > nested:<name:"fuga" amount:20 > float_value:1.5 double_value:2.5 int64_value:4294967296 uint64_value:9223372036854775807 int32_value:-2147483648 fixed64_value:9223372036854775807 fixed32_value:4294967295 bool_value:true string_value:"strprefix/foo" uint32_value:4294967295 sfixed32_value:2147483647 sfixed64_value:-4611686018427387904 sint32_value:2147483647 sint64_value:4611686018427387903
E0226 02:27:10.407661    5251 a_bit_of_everything.go:67] nested:<name:"hoge" amount:10 > nested:<name:"fuga" amount:20 > float_value:1.5 double_value:2.5 int64_value:4294967296 uint64_value:9223372036854775807 int32_value:-2147483648 fixed64_value:9223372036854775807 fixed32_value:4294967295 bool_value:true string_value:"strprefix/bar" uint32_value:4294967295 sfixed32_value:2147483647 sfixed64_value:-4611686018427387904 sint32_value:2147483647 sint64_value:4611686018427387903
E0226 02:27:10.407774    5251 a_bit_of_everything.go:67] nested:<name:"hoge" amount:10 > nested:<name:"fuga" amount:20 > float_value:1.5 double_value:2.5 int64_value:4294967296 uint64_value:9223372036854775807 int32_value:-2147483648 fixed64_value:9223372036854775807 fixed32_value:4294967295 bool_value:true string_value:"strprefix/baz" uint32_value:4294967295 sfixed32_value:2147483647 sfixed64_value:-4611686018427387904 sint32_value:2147483647 sint64_value:4611686018427387903
E0226 02:27:10.407873    5251 a_bit_of_everything.go:67] nested:<name:"hoge" amount:10 > nested:<name:"fuga" amount:20 > float_value:1.5 double_value:2.5 int64_value:4294967296 uint64_value:9223372036854775807 int32_value:-2147483648 fixed64_value:9223372036854775807 fixed32_value:4294967295 bool_value:true string_value:"strprefix/qux" uint32_value:4294967295 sfixed32_value:2147483647 sfixed64_value:-4611686018427387904 sint32_value:2147483647 sint64_value:4611686018427387903
E0226 02:27:10.407969    5251 a_bit_of_everything.go:67] nested:<name:"hoge" amount:10 > nested:<name:"fuga" amount:20 > float_value:1.5 double_value:2.5 int64_value:4294967296 uint64_value:9223372036854775807 int32_value:-2147483648 fixed64_value:9223372036854775807 fixed32_value:4294967295 bool_value:true string_value:"strprefix/quux" uint32_value:4294967295 sfixed32_value:2147483647 sfixed64_value:-4611686018427387904 sint32_value:2147483647 sint64_value:4611686018427387903
panic: net/http: CloseNotify called after ServeHTTP finished
goroutine 89 [running]:
panic(0x87f8e0, 0xc82036c150)
	/home/travis/.gimme/versions/go/src/runtime/panic.go:483 +0x3f3
net/http.(*response).CloseNotify(0xc820360270, 0x0)
	/home/travis/.gimme/versions/go/src/net/http/server.go:1535 +0x9d
github.com/gengo/grpc-gateway/examples/examplepb.RegisterABitOfEverythingServiceHandler.func9.1(0x7fe8ab5313a0, 0xc820360270, 0xc82036c070)
	/home/travis/gopath/src/github.com/gengo/grpc-gateway/examples/examplepb/a_bit_of_everything.pb.gw.go:658 +0x27
created by github.com/gengo/grpc-gateway/examples/examplepb.RegisterABitOfEverythingServiceHandler.func9
	/home/travis/gopath/src/github.com/gengo/grpc-gateway/examples/examplepb/a_bit_of_everything.pb.gw.go:660 +0x132
FAIL	github.com/gengo/grpc-gateway/examples	0.149s
?   	github.com/gengo/grpc-gateway/examples/examplepb	[no test files]
?   	github.com/gengo/grpc-gateway/examples/server	[no test files]
?   	github.com/gengo/grpc-gateway/examples/sub	[no test files]
?   	github.com/gengo/grpc-gateway/protoc-gen-grpc-gateway	[no test files]

I've noticed this in my env.

authors: kazegusuri yugui

repo: gengo/grpc-gateway

license_source: bigquery


text:
Array type issue

Building_classification has this error when running.
`---------------------------------------------------------------------------
IndexError                                Traceback (most recent call last)
 in <cell line: 1>()
----> 1 le2.inverse_transform( rawDf.iloc[:, nCols-2].unique() )
/usr/local/lib/python3.10/dist-packages/sklearn/preprocessing/_label.py in inverse_transform(self, y)
162             raise ValueError("y contains previously unseen labels: %s" % str(diff))
163         y = np.asarray(y)
--> 164         return self.classes_[y]
165
166     def _more_tags(self):
IndexError: arrays used as indices must be of integer (or boolean) type`

this issue happens in "chapter2.ipynb" as well.

repo: geo-smart/Flood-risk-ML

license: MIT


text:
Question

Your Question
I cant install anything in-game, Are geode servers down or somenthing?

Fixed.

repo: geode-sdk/geode


text:
allow for values to be a scope array

I tried to bind an array and it didn't work. This allows for that.
Is there any particular reason it was a one way previously?

Do you mind bumping the version in package.json and bower.json?

Wow, I am just now seeing this. Sure!

source: gharchive/pull-request

authors: DarrellBailey geoffreybauduin

license: mit


text:
Can Redhook (or other project) hook into Rust code?

If something is compiled with -C prefer-dynamic, can, for example, <std::fs::File as std::io::Write>::write be instrumented by LD_PRELOAD to inject failures?

Looks like the answer is yes, if you're willing to dig out the mangled name, and of course if the Rust compiler version is the same (since there's no stable ABI). If I cargo new --bin hello and cargo new p, edit p/Cargo.toml to make it a dylib and depend on libc, and then put the following in p/src/lib.rs:
#![allow(bad_style)]

extern crate libc;

use std::fmt::Arguments;
use std::mem::transmute;
use libc::{c_char, c_void};

#[link(name="dl")]
extern {
    fn dlsym(handle: *const c_void, symbol: *const c_char) -> *const c_void;
}

const RTLD_NEXT: *const c_void = -1isize as *const c_void;

#[no_mangle]
pub unsafe fn _ZN2io5stdio6_print20h47445faa595ef503E6gE(args: Arguments) {
    let ptr: unsafe extern fn(args: Arguments) =
        transmute(dlsym(RTLD_NEXT, b"_ZN2io5stdio6_print20h47445faa595ef503E6gE\0".as_ptr() as *const _));
    ptr(args);
    ptr(args);
}

then I can build both with -C prefer-dynamic as follows:
titan:~/src/rust/t/p geofft$ cargo rustc -- -C prefer-dynamic
   Compiling libc v0.2.9
   Compiling p v0.1.0 (file:///home/geofft/src/rust/t/p)
titan:~/src/rust/t/p geofft$ cd ../hello
titan:~/src/rust/t/hello geofft$ cargo rustc -- -C prefer-dynamic
   Compiling hello v0.1.0 (file:///home/geofft/src/rust/t/hello)
titan:~/src/rust/t/hello geofft$ LD_PRELOAD=../p/target/debug/libp.so target/debug/hello
Hello, world!
Hello, world!

The difference with what redhook does already is that this is a pub unsafe fn, not a pub unsafe extern fn. I don't think there's a good way to programmatically derive the mangled name from the actual one, especially because the crate's name is not part of it and the crate's random ID is. If it's still useful for you with this restriction, I'm happy to add something to the syntax to let you drop the extern from redhook's generated code.

I was just curious.
When Rust's stable ABI land and there be libstd in distros like today's libc, will Redhook extend its scope to also cover hooking into Rust?

authors: geofft vi

repo: geofft/redhook


text:
Demo marker not loading

Ah thanks - forgot about this leaflet bug.
There are workarounds for it:
https://stackoverflow.com/questions/41144319/leaflet-marker-not-found-production-env
Added the workaround to geoman.io - fixed now 👍

authors: codeofsumit joonaspaakko

repo: geoman-io/leaflet-geoman


text:
ProfileIOGroup reports available signals without application connection

This issue is for the new-profile branch.
geopmread lists various PROFILE:: signals as available, but they are not valid; trying to read gives the error:
> ./geopmread PROFILE::REGION_HASH board 0
Error: cannot read signal: <geopm> Runtime error: ApplicationSamplerImp::cpu_region_hash(): cannot read process info before connect().: at geopm/src/ApplicationSampler.cpp:265

These signals should either return NAN when no application is connected, or not be present in the list of available signals.

This is inconsistent with how the MSRIOGroup works.  It might be nice to be able to use geopmread --info for some of these signals, even if they can't be read (e.g. because the msr-safe driver is not present).

This is inconsistent with how the MSRIOGroup works.  It might be nice to be able to use geopmread --info for some of these signals, even if they can't be read (e.g. because the msr-safe driver is not present).

The way this should be handled by each IOGroup should be consistent and documented by 2.0.

This is no longer an issue.  ProfileIOGroup is only loaded when the HPC Runtime singleton accessor PlatformIOProf::platform_io() is used to get access to PlatformIO.

authors: cmcantalupo dianarg

repo: geopm/geopm


text:
expand-platform-support

Add support for KNL, Broadwell, and Skylake servers in the
Platform and PlatformImp classes.

No Skylake support yet.

authors: cmcantalupo sssylvester


text:
Support for OS mailbox programmable signals

This is an IOGroup that may need to be dependent on the MSRIOGroup, depending on how programmable counters will be implemented.  Most of the logic should be able to be shared with programmable perfmon counters.  See #123

This is covered by #1448


text:
pycsw does not decode spaces in the query part of GET requests correctly

Description
Whenever a GET request includes a KVP parameter whose value includes spaces that have been urlencoded to '+', pycsw is not able to correctly unquote them.
According to this stackoverflow post (which in turn references other reliable sources), the recommended way for a client to quote an HTTP GET request's query part is to turn spaces into the '+' character. (Note that in the path part of the URL, the correct quoting would be to turn spaces into '%20').  Apparently, both the '+' and the '%20' percent encoding are supported in the query part, but '+' is recommended. Either way, a robust server should be able to unquote both.
Encoding spaces as '+' is indeed how clients such as python's requests module or httpie work.
However, pycsw does not unquote these '+' characters back to the space character, which means that when using such clients for querying a pycsw instance, requests are not parsed correctly.
The fix for this error is pretty simple, and consists in replacing the usage of the six.moves.urllib.parse.unquote function with six.moves.urllib.parse.unquote_plus. The latter is aware of this problematic space unquoting issue and does the right thing if the request is encoded with '+' or with '%20'.
Environment

operating system:
Python version:
pycsw version:
source/distribution

[x] git clone
[ ] DebianGIS/UbuntuGIS
[ ] PyPI
[ ] zip/tar.gz
[ ] other (please specify):

web server

[x] Apache/mod_wsgi
[ ] CGI
[ ] other (please specify):

Steps to Reproduce

Set up a local pycsw instance and use the test suite in pycsw/tests/suites/cite as sample data and configuration:

# creating a docker container, to prevent messing up our own installation
docker run --rm -ti -p 8000:8000 ubuntu:16.04 /bin/bash
apt-get update
apt-get install -y git libgeos-dev libxml2-dev libxslt1-dev libz-dev python-dev python-pip
git clone https://github.com/geopython/pycsw.git
cd pycsw
pip install -r requirements.txt -r requirements-standalone.txt -r requirements-dev.txt
pip install -e .
paver test
export PYCSW_CONFIG=${PWD}/tests/suites/cite/default.cfg
python pycsw/wsgi.py

# pycsw should be running now

Now, opening another terminal, lets use python's requests module to make a GetRecords request using as CQL filter: AnyText like '%lorem%'. This filter contains spaces, so it will be incorrectly parsed by pycsw:

import requests
from lxml import etree

url = "http://localhost:8000"
constraint = "AnyText like '%lorem%'"
payload = {
    "service": "CSW",
    "version": "2.0.2",
    "request": "GetRecords",
    "resulttype": "results",
    "elementsetname": "brief",
    "typenames": "csw:Record",
    "constraintlanguage": "CQL_TEXT",
    "constraint": constraint
}
response = requests.get(url, params=payload)
response_element = etree.fromstring(response.text.encode("utf-8"))
print(etree.tostring(response_element, pretty_print=True))

Pycsw's response is:
<ows:ExceptionReport xmlns:csw="http://www.opengis.net/cat/csw/2.0.2" xmlns:dc="http:/
/purl.org/dc/elements/1.1/" xmlns:dct="http://purl.org/dc/terms/" xmlns:gmd="http://www.isotc211.org/
2005/gmd" xmlns:gml="http://www.opengis.net/gml" xmlns:ows="http://www.opengis.net/ows" xmlns:xs="htt
p://www.w3.org/2001/XMLSchema" xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" language="en-US"
 version="1.2.0" xsi:schemaLocation="http://www.opengis.net/ows http://schemas.opengis.net/ows/1.0.0/
owsExceptionReport.xsd">
  <ows:Exception exceptionCode="InvalidParameterValue" locator="constraint">
<ows:ExceptionText>Invalid query: (sqlite3.OperationalError) no such column: like [SQL: u"SELECT
count(*) AS count_1 \nFROM (SELECT records.identifier AS records_identifier, records.typename AS rec$rds_typename, records.schema AS records_schema, records.mdsource AS records_mdsource, records.insert$date AS records_insert_date, records.xml AS records_xml, records.anytext AS records_anytext, records$language AS records_language, records.type AS records_type, records.title AS records_title, records.$itle_alternate AS records_title_alternate, records.abstract AS records_abstract, records.keywords AS records_keywords, records.keywordstype AS records_keywordstype, records.parentidentifier AS records_$arentidentifier, records.relation AS records_relation, records.time_begin AS records_time_begin, rec$rds.time_end AS records_time_end, records.topicategory AS records_topicategory, records.resourcelang$age AS records_resourcelanguage, records.creator AS records_creator, records.publisher AS records_pu$lisher, records.contributor AS records_contributor, records.organization AS records_organization, re$ords.securityconstraints AS records_securityconstraints, records.accessconstraints AS records_access$onstraints, records.otherconstraints AS records_otherconstraints, records.date AS records_date, reco$ds.date_revision AS records_date_revision, records.date_creation AS records_date_creation, records.d$te_publication AS records_date_publication, records.date_modified AS records_date_modified, records.$ormat AS records_format, records.source AS records_source, records.crs AS records_crs, records.geode$code AS records_geodescode, records.denominator AS records_denominator, records.distancevalue AS rec$rds_distancevalue, records.distanceuom AS records_distanceuom, records.wkt_geometry AS records_wkt_g$ometry, records.servicetype AS records_servicetype, records.servicetypeversion AS records_servicetypeversion, records.operation AS records_operation, records.couplingtype AS records_couplingtype, record
s.operateson AS records_operateson, records.operatesonidentifier AS records_operatesonidentifier, rec
ords.operatesoname AS records_operatesoname, records.degree AS records_degree, records.classification
 AS records_classification, records.conditionapplyingtoaccessanduse AS records_conditionapplyingtoacc
essanduse, records.lineage AS records_lineage, records.responsiblepartyrole AS records_responsiblepar
tyrole, records.specificationtitle AS records_specificationtitle, records.specificationdate AS record
s_specificationdate, records.specificationdatetype AS records_specificationdatetype, records.links AS
 records_links \nFROM records \nWHERE AnyText+like+'%lorem%') AS anon_1"]</ows:ExceptionText>
  </ows:Exception>
</ows:ExceptionReport>

The error complains about an invalid query.
The contents of the error (which probably should not be displayed back to the client, but that can be handled in another issue) show that the database is being fed a query that includes `WHERE AnyText+like+'%lorem%'. This means that the url unquoting is not being done correctly.
Additional Information
I've fixed this issue locally and am willing to send a PR with the fix, in a short while.

Thank you @ricardogsilva for the report.
The solution seems pretty straightforward.

Thanks for the report / investigation @ricardogsilva. Looking forward to PR.

Fixed in master and backported to 2.0 branch.  Thanks @ricardogsilva!

Hi all,
Even after the last patch we are still experiencing problems with spaces in CQL queries.
Here are the symptoms.
Version 1.10.3:
curl "http://apps.ecmwf.int/csw/?service=CSW&version=2.0.2&request=GetRecords&typenames=csw:Record&elementSetName=full&resultType=results&constraintLanguage=CQL_TEXT&startposition=1&constraint=dc%3Asubject%20like%20%27%25Product%20family%25%27"
-> I get the expected list of products.
git clone master && git pull version
curl "http://85.204.96.249:8000/?service=CSW&version=2.0.2&request=GetRecords&typenames=csw:Record&elementSetName=full&resultType=results&constraintLanguage=CQL_TEXT&startposition=1&constraint=dc%3Asubject%20like%20%27%25Product%20family%25%27"
-> I get the "Invalid Filter syntax" error.
If I replace the space character (%20) in the single-quoted string of the LIKE statement with a wildcard character (%27) the latest version will return a valid response.
I would appreciate any suggestion.
Miha

@oblakeobjet thanks for the info. I've filed a new issue at #483 given it is not related to @ricardogsilva's fixes here.

cc @kalxas
@ricardogsilva note that during our 2.0.3 release today we encountered problems testing against OGC CITE CSW3 (CSW 2 worked) using the online testing engine.
When I tested against CITE in a local environment (see https://bpaste.net/show/4dcff963c88e), all CSW 3 tests passed.  Which allowed us to move ahead with 2.0.3 but makes me wonder whether the online test is sending spaces differently than when testing on the command line.  Needs verification.

authors: kalxas oblakeobjet ricardogsilva tomkralidis

repo: geopython/pycsw


text:
[SITE] start working on site.

Contents of this PR:

Creating an orphan branch gh_site, hosting the maven-site.
The maven-site is useful for the generated documentation. Maven users will want to see the generates goals documentation.
Adds a new profile docs which will generate javadoc documentation using dokka. I would recommend to move the source-plugin to this profile as well, you don't need it in your everyday verify goal as a developer. It is only needed for releases.

Not in the scope of this PR:

convert all of this stuff to jreleaser. 😃
move source-plugin to the docs profile.
Fix docs
Tests. I cannot test the release script easily, as I do not have push access. I might change the distribution URL  for testing, though. Consider this untested unless noted otherwise.

Codecov Report
Base: 68.27% // Head: 68.27% // No change to project coverage :thumbsup:

Coverage data is based on head (210510e) compared to base (5c06812).
Patch has no changes to coverable lines.

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

Additional details and impacted files
@@            Coverage Diff            @@
##             master      #51   +/-   ##
=========================================
  Coverage     68.27%   68.27%
  Complexity       40       40
=========================================
  Files            10       10
  Lines           186      186
  Branches         10       10
=========================================
  Hits            127      127
  Misses           51       51
  Partials          8        8

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

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

This works for me locally after adding  -DskipStaging=true -DskipRemoteStaging=true to the release.sh script. It failed (as expected) on the git push command because I had set origin to Georg's url (instead of using origin for mine and upstream for Georg's).
Can be reviewed now. 😄

authors: bmarwell codecov-commenter


text:
Breakdown Epics into smaller issues

The Epics:

Generalize OrbitAI
Evaluate the ranger library for batch training with Random Forest

Load the issues into the GitHub Projects board so that they can be tracked:
https://github.com/georgeslabreche/opssat-orbitai/projects/2

@georgeslabreche @Evridiki-ase please review.

@TanguySoto this looks OK on my end.

I've broken down the epics further.

authors: TanguySoto georgeslabreche

repo: georgeslabreche/opssat-orbitai


text:
🛑 Dementia uk is down

In 84b50bd, Dementia uk (https://www.dementiauk.org/) was down:

HTTP code: 0
Response time: 0 ms

Resolved: Dementia uk is back up in c589eb6 after 9 minutes.

repo: georgewelly/upriseuptime


text:
Use tempfile::tempdir instead of /tmp (fix Windows tests)

[x] I agree to follow the project's code of conduct.
[x] I added an entry to CHANGES.md if knowledge of this change could be valuable to users.

Creating PR from other branch

repo: georust/gdal


text:
Wrap OGR_L_GetExtent

I'm not sure whether it's all right to expose OGREnvelope directly, any preferences?

Thanks for your contribution :)
I guess it is ok to expose OGREnvelope since it is exposed by gdal-sys anyway.

authors: jdroenner lnicola

repo: georust/rust-gdal


text:
Merge consecutive range requests

We have the offsets of each image tile and can calculate which tiles we need to read for a given partial read so this shouldn't be too difficult.
Ref: https://trac.osgeo.org/gdal/wiki/ConfigOptions#GDAL_HTTP_MERGE_CONSECUTIVE_RANGES

With async we actually benefit from many little requests than fewer large requests, so I'm closing this for now.

repo: geospatial-jeff/async-cog-reader


text:
gstreamer 内存泄漏

rtspsrc location=rtsp://admin:admin123@172.23.240.20:554/cam/realmonitor?channel=1&subtype=0 latency=0 drop-on-latency=true ! rtph264depay ! h264parse ! omxh264dec ! nvvidconv ! video/x-raw, format=(string)NV12  ! appsink emit-signals=true sync=false max-buffers=1 drop=true
你好，我使用的是jetson nano,然后通过上面的gst-url 使用new-sample、pull-sample 来获取摄像头的每一帧 ，随着运行过程中 内存会一直增加，请问有什么解决思路吗？

手头没有测试环境，暂时我能想到的有两点。
1.appsink出来的sample，你在用完之后需要手动调用gst_sample_unref(sample);释放掉，不知道你sink出来的sample是否得到了正确的释放。我的测试程序里通过cv::Mat和std::shared_ptr做了sample下的内存管理，相当于使用std::shared_ptrcv::Mat接管了sample下的buffer内存，这种做法其实是不严谨的，我感觉还是会产生一些细微的内存碎片。正确的做法应该是定义一个class然后封装一个GstSample对象，然后用这个class的对象接管sink出来的GstSample，并在对象析构的时候调用gst_sample_unref(sample);。
2.假如不是由于appsink插件导致的内存泄露，那么我建议你逐个插件调研，看看是不是omxh264dec这个解码器导致的内存泄露，比如说你把appsink改成fakesink，然后用free看memory的使用和释放情况。高通的硬件解码器在早期是会导致内存泄露的，当时解了挺长时间，不过我觉得omxh264dec应该是一个很成熟的插件了，不会存在这种问题。
最后就是Jetson我建议用DeepStream开发，Nvidia论坛上的CE回复问题还挺快的。

嗯 收到 我这边排查下来还有一个怀疑思路：有可能是omxh264dec、nvvidconv 并发太多导致硬件资源不够，导致内存会有buffer或者别的缓存等导致的？ 但是不确定怎么去确定(我们是python3的程序)。因为我们发现把并发讲下来（比如从8到1），基本就没有增长了。

python是否有valgrind这样的内存泄漏的追查工具呢，我不太了解Python，这类边缘盒子设备还是尽量用C++开发吧orz。
1.假如你怀疑硬件资源不够，你那边是否有jetson nx这种比nano性能更强的设备，跑起来看看呢？
2.假如是omxh264enc的问题，解决起来难度会很大，我这边能给的思路就是去看看omxh264dec的各个锁的持有情况，是不是buffer被锁住了一直没释放（qtivdec上是这个原因导致的，但是细节因为是机密所以不能透露更多）。
3.降并发的情况下，你延长测试时间是否还有泄露，可以写个脚本每隔一段时间看一下内存的状态，跑满几个小时 看看。另外假如降并发有效，那么你可以保持原有并发数，但是降低rtsp camera的帧数和分辨率，看看内存消耗的情况。

好的 非常感谢   按着你上面的思路我去试试

authors: gesanqiu lytasky

repo: gesanqiu/gstreamer-example


text:
doc: Correct phoenixd install script instructions in readme

Add command to make the install script executable in the instructions.

thanks!

repo: getAlby/hub


text:
Parsing a comment bit does not return ids

For a comment bit:
[.|pageNumber]
[@id:946897]
5

Parsed JSON:
[
    {
        "bit": {
            "type": "_comment"
        },
        "parser": {
            "version": "1.5.22",
            "bitmarkVersion": "2",
            "textParserVersion": "8.10.3",
            "commentedBitType": "[.pageNumber]"
        },
        "bitmark": "[.|pageNumber]\n[@id:946897]\n5"
    }
]

There is no .id in the parsed bit JSON. The ids are essential when we publish the same book.

@six5536 It would be great as well to add a property for the commented out bit type.
"commentedOutBitType": "pageNumber"

@six5536 It would be great as well to add a property for the commented out bit type. "commentedOutBitType": "pageNumber"

@gaba-getmorebrain this already exists in the parser section.

@gaba-getmorebrain I think a good solution here is to parser the comment bits as normal, but just:

Replace the bit type with "type": "_comment"
Add "commentedBitType": "[.pageNumber]" to the parser section

[.|article]
[@id:12353456]

This is the table caption

<==>
[
  {
    "bit": {
      "type": "_comment",
      "commentedType": "article",
      "id": [
        "12353456"
      ],
      "body": [
        {
          "type": "paragraph",
          "content": [
            {
              "text": "This is the table caption",
              "type": "text"
            }
          ],
          "attrs": {}
        }
      ]
    },
    "parser": {
      "version": "1.5.23",
      "bitmarkVersion": "3",
      "textParserVersion": "8.10.3"
    },
    "bitmark": "[.|article]\n[@id:12353456]\n\nThis is the table caption"
  }
]

Released in https://github.com/getMoreBrain/bitmark-parser-generator/releases/tag/v1.5.23

FYI @stefanszasz

authors: gaba-getmorebrain six5536 stefanszasz

repo: getMoreBrain/bitmark-parser-generator

license: ISC


text:
Mixed case syntax

Other than familiarity, is there a reason BEM settled on the all lowercase syntax that was generally popular for non structured CSS naming conventions?
I'm not sure if there was a particular reason everyone settled on the all lowercase dash separated syntax other than lack of need for a more descriptive syntax, and a desire to match HTML conventions. But CSS is not HTML so is there a reason for them to match? Since BEM is much more descriptive, I like to use mixed case to provide more versatility.
Personally I've settled on BlockName_ElementName-attributeName using PascalCase/ camelCase to invoke a class/variable feel. IMO it's easier to read than block-name__element-name--attribute-name because each name token is blocked together. Also __ and -- look ugly :)
Is there a particular reason BEM settled on all lowercase instead of mixed case? Did I make a grave mistake, or is it purely cosmetic?

My justification for all lowercase is that it ignores the use of capitalisation. Personally simplifies things for me and less worry about having that extra criteria to capitalise my selectors.
I found the underscore symbol __ to mean 'below/descendent of' and double dash -- to mean 'variation of'.

I don't quite get that though. We use significant capitalization in pretty much every other language that has significant casing so why not use it with CSS as well?
HTML ignores tag casing so it makes sense in that language, but CSS has significant casing so why not use it?
I also don't see how it's any more complicated. It's the same number of significant separators. There's a block-element separator, an attribute separator and a word separator either way. Personally mixed case seems simpler to me because camel casing is very standard in other languages and you don't need to understand the difference between - and --. If I had no experience with BEM I would need to be taught the difference between dashes, but with camel casing the significance of the underscores and dashes are more obvious to me.

🚲 🔴 🏠 🚲 🔵 🏠
I think part of the reason was related to tooling. It's pretty simple to write tools that parse text on -- and __.  Having to do mixed case adds more complexity to the tooling.
At this point the ship has sailed on how the naming works. If you want your projects to use camel case with a single _ and - you're free to do that, nothing forces it. However, most tooling will be built around knowing that -- and __ are used.
As for ease of reading, that's fairly subjective in the context of these CSS selectors.
Related to language the mixcase or not doesn't really matter.  CSS is most often used with HTML for web developers, so following some of the standards that HTML use make sense to me.

Are word separators significant in BEM? I was under the impression that foo and foo-bar were just treated as token name blobs? I didn't happen to see any tooling mentioned on the website so maybe there's a constraint I'm missing, but I thought the only really significant syntax tokens were the block/element separator and the (block|element)/attribute separator, and anything in between was pretty insignificant.
I of course haven't thought about the problem as much as you guys so I'm curious if there's some constraint that requires you to parse the characters within the name token blob. It would be nice if the tooling would allow you to choose the separator token.

authors: etler garrypolley sumnermic


text:
Could not find a declaration file for module '@getbrevo/brevo'.

When I require the Node SDK:
const brevo = require('@getbrevo/brevo')

I get this error:
Could not find a declaration file for module '@getbrevo/brevo'. '/project/node_modules/@getbrevo/brevo/src/index.js' implicitly has an 'any' type.
  Try `npm i --save-dev @types/getbrevo__brevo` if it exists or add a new declaration (.d.ts) file containing `declare module '@getbrevo/brevo';`ts(7016)
module "/project/node_modules/@getbrevo/brevo/src/index"
No quick fixes available

@types/getbrevo__brevo obviously doesn't exist — how should this be resolved?

I'm getting this error - error ./node_modules/@getbrevo/brevo/src/index.js
Module not found: Can't resolve 'ApiClient'
Did you mean './ApiClient'?
Requests that should resolve in the current directory need to start with './'.
Requests that start with a name are treated as module requests and resolve within module directories (node_modules, /Users/angelo/Documents/APP_DEV/nextjs/gestyon-stok).
If changing the source code is not an option, there is also a resolve option called 'preferRelative' which tries to resolve these types of requests in the current directory too.

@shubhamUpadhyayInBlue my project is not a TS project. It’s just plain JS.

Hi @getaaron
I think that your IDE might have some settings due to which it is considering it as a TS project.
I found error similar to yours in one of the blog posts. They face the same error.
Otherwise, for plain NodeJS projects I don't think this error makes much sense, I mean it should not occur.

When I require the Node SDK:
const brevo = require('@getbrevo/brevo')

I get this error:
Could not find a declaration file for module '@getbrevo/brevo'. '/project/node_modules/@getbrevo/brevo/src/index.js' implicitly has an 'any' type.
  Try `npm i --save-dev @types/getbrevo__brevo` if it exists or add a new declaration (.d.ts) file containing `declare module '@getbrevo/brevo';`ts(7016)
module "/project/node_modules/@getbrevo/brevo/src/index"
No quick fixes available

@types/getbrevo__brevo obviously doesn't exist — how should this be resolved?

has anything be done to this

repo: getbrevo/brevo-node


text:
Any 'first good issue' I could work on?

@kytwb, are there any 'good first issue' I could work on?
EDIT: added good first issue url

https://github.com/getferdi/ferdi/issues?q=is%3Aissue+is%3Aopen+label%3A"good+first+issue"

https://github.com/getferdi/ferdi/issues/529 may be interesting to implement. You'll need to add a new item to the menubar to allow the usage of shortcuts and then you toggle between services using that.

Thanks, @vantezzen. I will look at #513.
I am closing this issue for now.

authors: Krzyku vantezzen yourcontact

repo: getferdi/ferdi


text:
With Firefox, cannot select any element in yahoo.com

Expected behavior
Taiko can select elements in yahoo.com with firefox.
Actual behavior
Taiko cannot select any element in yahoo.com with firefox.
Steps to reproduce
*: The step 4 is optional.

$ TAIKO_BROWSER_PATH=<path-to-firefox-nightly> taiko
openBrowser()
goto('yahoo.com')
link('News').exists()
Uncaught Cannot read property 'type' of undefined
focus(link('News'))  // takes few seconds...
✘ Error: Cannot read property 'type' of undefined, run .trace for more info.
.trace

"TypeError: Cannot read property 'type' of undefined
" +
  '    at getObjectIdsFromResult (/home/vagrant/.config/nvm/12.18.3/lib/node_modules/taiko/lib/handlers/runtimeHandler.js:30:46)
' +
  '    at Object.findElements (/home/vagrant/.config/nvm/12.18.3/lib/node_modules/taiko/lib/handlers/runtimeHandler.js:141:34)
' +
  '    at runMicrotasks (<anonymous>)
' +
  '    at processTicksAndRejections (internal/process/task_queues.js:97:5)
' +
  '    at async $function (/home/vagrant/.config/nvm/12.18.3/lib/node_modules/taiko/lib/elementSearch.js:206:5)
' +
  '    at async get (/home/vagrant/.config/nvm/12.18.3/lib/node_modules/taiko/lib/elementSearch.js:121:16)
' +
  '    at async Object.elements (/home/vagrant/.config/nvm/12.18.3/lib/node_modules/taiko/lib/elementSearch.js:255:47)
' +
  '    at async /home/vagrant/.config/nvm/12.18.3/lib/node_modules/taiko/lib/elementWrapper/linkWrapper.js:15:19
' +
  '    at async get (/home/vagrant/.config/nvm/12.18.3/lib/node_modules/taiko/lib/elementWrapper/helper.js:62:50)
' +
  '    at async /home/vagrant/.config/nvm/12.18.3/lib/node_modules/taiko/lib/elementSearch.js:249:22'

Versions
Taiko: Version: 1.0.21 (Chromium: 86.0.4229.0) RELEASE
Firefox: Mozilla Firefox 82.0a1

Thanks for reporting, I could see the issue happening with latest firefox nightly. Will try fixing this

I can select elements in yahoo.com with the latest version.
Thank you.

repo: getgauge/taiko


text:
API to support textarea.

Expected behavior
Should be able to access textarea with attributes, label or proximity selectors.
Actual behavior
No support for textarea currently
Versions
taiko : 0.6.0

Fix available in 56c63b9

This feature has been verified and found fixed with the version 0ae7ff5

authors: Apoorva-GA Debashis9012 NivedhaSenthil


text:
Taiko runner does not work when run from a Gauge project

Expected behavior
taiko sample.js must work when run inside a directory that is a Gauge project
Actual behavior
taiko sample.js --observe
Invalid file extension. Only javascript files are accepted.

Steps to reproduce

gauge init js
npm install -g taiko
Launch taiko REPL inside the directory to record and save a simple script .code sample.js
Run taiko sample.js

Versions
node -v
v12.4.0

taiko -v
Version: 1.0.0 (Chromium: 76.0.3803.0) RELEASE

Abe to reproduce this same issue in the latest commit: f1036f4d06845a8f4ecefe667e216befc59756ac.

This issue has been verified and found fixed with the version: babdc48cf1328cb0a3ba0402d377e4752f90b465

I am still facing the same issue right now

@acecyq Are you using the nightly of taiko? Or the released version of it?

Hi there. Got Version: 1.0.0 (Chromium: 76.0.3803.0) RELEASE when i run taiko -v on Terminal. Basically i updated my Node recently to the latest LTS, ran npm i -g npm@latest and npm i -g taiko. Hope this helps.

@acecyq I would suggest you to use the latest nightly that will solve your problem...

@acecyq Make sure  taiko dependency has been updated in package.json. Something like
{
  "name": "gauge-taiko-template",
  "version": "0.0.1",
  "description": "Starter template for writing JavaScript tests for Gauge",
  "scripts": {
    "test": "gauge run specs/"
  },
  "dependencies": {
    "taiko": "https://github.com/getgauge/taiko"
  }
}

And then do a npm install. That should solve the problem.
We will be releasing taiko with this fix soon.

authors: Apoorva-GA Debashis9012 acecyq luciferankon zabil


text:
[add-resource] New Theme

Hello !
I would like to add another theme to the Grav Repository.
Here are the project details: https://github.com/lab-dev-code/grav-theme-lens.
I have followed the same instructions that I did for the Identity theme.
A demo is available here, still using an out of the box Grav Installation.
I hope I didn't forget anything.

your demo page is reporting an issue: http://lab.dev-code.fr/demos/grav-themes/lens

This is strange ... It worked and the only thing I did after that is updating Grav to RC4.
I'll investigate.

Fixed. A simple permissions mistake.

btw i'm traveling at the moment, and will add this as soon as i can.  thanks for your patience :)

Okay ! Thanks :)

ok should be out there! thanks!

authors: lab-dev-code rhukster

repo: getgrav/grav


text:
Support for plink2

Hi Kai,
Hope all is well.
Could I ask if you have plans to provide support for plink2 in SuSiEx_LD.py? We have a large amount of pgens and skipping the --make-bed conversion would save a lot of time/compute. I imagine the LD calculations themselves may be faster too.
Many thanks for your help,
Chris

Hi Chris - I think this is not too difficult to implement.
@yorkklause Maybe you can add a plink2 option when you get a chance?

Hi Tian, Kai,
That would be fantastic! I think this would also benefit other users who work with or are transitioning to plink2 binary files.
Happy to help test if that's of use.
Many thanks,
Chris

Hi Tian, Kai,
That would be fantastic! I think this would also benefit other users who work with or are transitioning to plink2 binary files.
Happy to help test if that's of use.
Many thanks,
Chris

plink does distribute python bindings to pgens via Pgenlib: https://github.com/chrchang/plink-ng/tree/master/2.0/Python

Hi Chris @cqgd ,
I apologize for the delayed response.
Thank you for your suggestions. Just to confirm, you would like all the LD calculations to be performed using plink2 and only in the pgen format, correct?
Upon checking the latest version of plink2, I found that the LD calculation option "--r-unphased" remains unavailable in the Alpha version (5.10 final, 5 Jan). It appears to be only accessible in the development version (2 Mar). This function may still be considered unstable. At this stage, I think we still have to use plink1.9 and bed format to calculate LD.
I will update the script later when plink2 formally releases this function.
@pettyalex Thank you for sharing this information. We will consider incorporating it into our future versions.
My best
Kai Yuan


text:
[Bug] Insomnia hangs when encountering redirect loop

Overview

Insomnia Version: 5.7.4
Operating System:  Windows 10 Pro 1703
Summary:  When a server repeatedly responds with redirects, Insomnia seems to hang forever, probably stuck in an endless loop. The request can be cancelled, though.

How To Reproduce

Perform a GET request to https://ethereal-deposit.glitch.me/redirect-loop, a test URL I set up that redirects to itself.

Other Notes
After a few redirects, Chrome and Firefox react with a "too many redirects" error page. This is the behavior I'd expect from Insomnia as well. Note that a redirect loop could be indirect as well, e.g. A redirecting to B, B to C, C to A.
Fun fact: In this situation, Postman claims it "could not connect" which is a plain lie.

Thanks @bannmann! I just added a fix for this. Going forward, the max redirects will be 20, which is apparently what most browsers use.

authors: bannmann gschier

repo: getinsomnia/insomnia


text:
[Bug] Remove \n from generated code payload

Operating System:  Windows 7

Details
I'm doing a quite simple query

Then generating python code, either http-client or request, here it is
import requests

url = "http://192.168.60.10/api/index.php"

payload = "{\n    \"action\": \"SHOW\",\n    \"object\": \"STPL\"\n}\n"
headers = {
    'content-type': "application/json",
    'centreon-auth-token': "bla bla bla"
    }

response = requests.request("POST", url, data=payload, headers=headers)

print(response.text)

As you can see, the payload is somehow messed up. I've the same issue for all the others language, like go 	payload := strings.NewReader("{\n    \"action\": \"SHOW\",\n    \"object\": \"STPL\"\n}\n") & so on.
And thanks for the awesome client :+1:
Best regards,

Hi @rgarrigue,
I'm not sure I understand the problem. The \n characters represent the newlines in your request body. Is there an alternate way you'd like to see them represented?

I'm sorry to refute your comment, but I don't believe that's correct.
Newlines were part of your original data. The difference is that the Insomnia editor chooses to show them visually (as new rows) instead of literally (as \n). However, the underlying data is the same in both cases.
To demonstrate this, removing the \n characters from your original data would look like this:

Uh, okay. Yeah my bad obviously. It bugged me since json is usually a big one liner, and I've to ask my browser via an extension or my IDE to display this nicely, without the formating being a visible part of the data.
Well, I'm gonna close this one then. Sorry for the disturbance

No worries. If you (or anyone else) needs/wants to send compressed (minimal whitespace) request bodies, it's possible to write a Request Hook Plugin that looks for all application/json requests, parses and compresses the request body, and overwrites the request body with the compressed one.

authors: gschier rgarrigue


text:
[BUG]: customers endpoints "resource_not_found"

Describe the bug
Getting a Customer using this client library does not work, although making a direct request (using Insomnia or axios) to Lago to get the same Customer does work. Lago logs seem to suggest that the something related to path being unmatched?
Lago logs:
lago-api     | I, [2023-09-08T05:03:01.869264 #11]  INFO -- : [17d85fb2-4a78-4747-95d8-59fd18859b1b] {"method":"GET","path":"/customers/EXAMPLEID01","format":"*/*","controller":"ApplicationController","action":"not_found","status":404,"duration":0.65,"view":0.2,"db":0.0,"ddsource":"ruby","params":{"unmatched":"customers/EXAMPLEID01"}}
The error response says "resource_not_found", which is the same as what I would get when sending a request to an invalid endpoint to my instance of Lago.
To Reproduce
Steps to reproduce the behavior:
Call this with this client:
const response = await client.customers.findCustomer("EXAMPLEID01");
Expected behavior
Response of the Customer object if Customer exists, or a "customer_not_found" response.
Support

Version "^0.45.0-beta"

P.S.
Customer creation via this client also does not work, but direct requests work.

Hmm, seems to need the /api/v1 when setting up the custom baseUrl

repo: getlago/lago-javascript-client


text:
Add deprecation notice

It's unclear the reason for forking, adding deprecation notice and archiving.

Thanks @simontabor

repo: getndazn/drone-terraform-plugin


text:
Design tweaks

Starting a list of small changes to improve the over all experience

Add a "+" icon to the add button for repeats
For the add button for repeats, add the repeat label (really the label of the outer group) in the button text. That would be similar to Collect. For example, if the repeat label is "person", the button label would be "Add person". The danger is that of course the repeat label could be poorly selected. In Collect, the repeat name is used as-is and put in quotes (Add "person"?). We could consider doing things like forcing lowercase.

For the add button for repeats, what do you think about also adding the repeat label (really the label of the outer group) in the button text? That would be similar to Collect. For example, if the repeat label is "person", the button label would be "Add person". The danger is that of course the repeat label could be poorly selected. In Collect, the repeat name is used as-is and put in quotes (Add "person"?). We could consider doing things like forcing lowercase.

@lognaturel Yes, I think this would be a super nice improvement and would align nicely with Collect!

Comments by @eyelidlessness about Form Header on #132:

I was curious to see how wrapping looks for forms with long titles. Pretty good! I think we may want to iterate later on...

More consistent alignment of controls (icons in larger views, hamburger menu in smaller). The vertical centering makes those a moving target. I think we'll probably want some design guidance on that.
Maybe slightly reducing text size/weight over a certain character length? (Honestly the weight feels pretty heavy regardless of length, wrapping.)

Normalize vertical spacing, especially after text boxes:

authors: alyblenkin lognaturel sadiqkhoja

repo: getodk/web-forms


text:
publish: getsentry/sentry-cocoa@8.30.1

Requested by: @armcknight
Merge target: (default)
Quick links:

View changes
View check runs

Assign the accepted label to this issue to approve the release.
Leave a comment containing #retract under this issue to retract the release (original issuer only).
Targets

[ ] github
[ ] cocoapods[sentry-cocoapod]
[ ] registry

Targets marked with a checkbox have already been executed.  Administrators can manually tick a checkbox to force craft to skip it.

#retract

After fixing the breakage with https://github.com/getsentry/sentry-cocoa/pull/4151: https://github.com/getsentry/publish/issues/4072

authors: armcknight getsentry-release

repo: getsentry/publish


text:
Rename Angular 1 to AngularJS and Angular 2 to Angular

hi @benvinegar

one of the diffs looks funky because I switched the file names:
angular.rst -> angularjs.rst
angular2.rst -> angular.rst

Looks good, but I think we should clarify that "Angular" is "Angular 2+" somewhere. The naming scheme is newly introduced and I'm not sure everyone will piece it together.

Looks good, but I think we should clarify that "Angular" is "Angular 2+" somewhere. The naming scheme is newly introduced and I'm not sure everyone will piece it together.

I added https://github.com/getsentry/raven-js/pull/928/commits/b43bcd99c876726a9f38acde5dcdffd735bd1707 and https://github.com/getsentry/raven-js/pull/928/commits/476e57588dd2edf8700e2c0059ce77764f71f91d to address this

These docs are pulled into Sentry using some convoluted scheme ... I just need to make sure changing the file paths won't mess something up otherwise lgtm.

authors: benvinegar jessla

repo: getsentry/raven-js


text:
Bump Flutter plugin example

:scroll: Description
#skip-changelog
:bulb: Motivation and Context
e: /home/runner/.pub-cache/hosted/pub.dartlang.org/package_info_plus-1.4.0/android/src/main/kotlin/dev/fluttercommunity/plus/packageinfo/PackageInfoPlugin.kt: (30, 9): Class 'kotlin.Unit' was compiled with an incompatible version of Kotlin. The binary version of its metadata is 1.6.0, expected version is 1.4.2.
Due to https://github.com/fluttercommunity/plus_plugins/pull/670/files
:green_heart: How did you test it?
:pencil: Checklist

[ ] I reviewed submitted code
[ ] I added tests to verify changes
[ ] I updated the docs if needed
[ ] All tests passing
[ ] No breaking changes

:crystal_ball: Next steps

Codecov Report

Merging #762 (31f861e) into main (e5d8519) will increase coverage by 0.02%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main     #762      +/-   ##
==========================================
+ Coverage   90.53%   90.56%   +0.02%
==========================================
  Files         104       96       -8
  Lines        3328     3210     -118
==========================================
- Hits         3013     2907     -106
+ Misses        315      303      -12

Impacted Files
Coverage Δ

dio/lib/src/failed_request_interceptor.dart

dio/lib/src/tracing_client_adapter.dart

dio/lib/src/breadcrumb_client_adapter.dart

logging/lib/src/logging_integration.dart

logging/lib/src/extension.dart

dio/lib/src/sentry_dio_client_adapter.dart

dio/lib/src/sentry_dio_extension.dart

dio/lib/src/sentry_transformer.dart

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 e5d8519...31f861e. Read the comment docs.

authors: codecov-commenter marandaneto

repo: getsentry/sentry-dart


text:
Remove :use_error_logger configuration

Received another report along the same lines as #144, where setting use_error_logger: true did not set up the handler properly.  Adding :error_logger.add_report_handler(Sentry.Logger) into the user's application start method resolved it, so we'll just have to remove this: https://github.com/getsentry/sentry-elixir/blob/master/lib/sentry.ex#L105-L107
and update documentation.

That would be great, I was just bitten by this as well.

@RobinClowers Thank you, it's good to know it's a bigger issue than I initially thought.  This is at the top of my list.

authors: RobinClowers mitchellhenke

repo: getsentry/sentry-elixir


text:
[sentry-vite-debug-id-upload-plugin] Client network socket disconnected before TLS connection was established

Environment
node 16.13.1
What version are you running? Etc.
"@sentry/react": "7.75.1",
"@sentry/vite-plugin": "2.9.0",
"vite": "^4.4.5",
npm
Steps to Reproduce

upload sentry sourcemap after build
this is how my sentryVitePlugin specified in vite.config.ts

export default defineConfig(({ mode }) => {

return {
  // ...
plugins: [
react(),
sentryVitePlugin({
      org: "my-sentry",
       project: "my-sentry-web-project",
       authToken: process.env.SENTRY_AUTH_TOKEN,
       errorHandler: function(err) {
       console.log("sentryVitePlugn err during upload: ", err),
})
],
 // ...
}
...

it worked a week ago, but broken recently.
Expected Result
should be build successfully
Actual Result

What actually happened. Maybe a screenshot/recording? Maybe some logs?

Hi, thanks for reporting this!

Are you on SaaS or self hosted Sentry?
Is this consistently happening?
Can you try setting telemetry: false in the options and see if it is still happening? Thanks!

It's not happening with telemetry: false, however, I'm assume this should be work without set false to telemetry.

Yeah correct. We changed something a few days ago, lemme release that an we'll try again!

@dante01yoon We just released version 2.10.3. Can you try that one out? Thanks!

It works with telemetry flag. Thanks for quick catching up!

Resolved  via 2.10.3 update


text:
Can the rsbuild plugin be supported?

Hi, I'm using rsbuild
but my project uses the sentry monitoring system, I hope you can support rsbuildPlugin!

cc @smeubank
https://www.npmjs.com/package/@rsbuild/core

there is pretty solid adoption, so I think we should start considering it. Maybe when it gets above 100k weekly downloads?

@mengbo-ji for now you can use tools.rspack (https://rsbuild.dev/zh/config/tools/rspack#toolsrspack) and add the sentry webpack plugin: https://www.npmjs.com/package/@sentry/webpack-plugin

@mengbo-ji or @AbhiPrasad, could you provide an example of how to properly configure this as @AbhiPrasad described? I'm having a difficult time figuring that part out.

I haven't tried it myself tbh but I'd assume that you would just have to add the webpack plugin here: https://rspack.dev/guide/migration/webpack#community-plugins
What exactly isn't working?

Can confirm that tools.rspack works just fine:
  tools: {
    rspack: (config, { appendPlugins }) => {
...
          appendPlugins([
            sentryWebpackPlugin({
              authToken: process.env.SENTRY_AUTH_TOKEN,
              org: 'foo',
              project: 'bar'
            })
          ]);
...
      return config;
    }

in my RsBuild config worked without issue.

SHELL Application RsBuild Config:
const EXTRA_KEY = 'ROUTE_TO';

const transport = makeMultiplexedTransport(makeFetchTransport, args => {
  const event = args.getEvent();
  if (
    event &&
    event.extra &&
    EXTRA_KEY in event.extra &&
    Array.isArray(event.extra[EXTRA_KEY])
  ) {
    return event.extra[EXTRA_KEY];
  }
  return [];
});

init({
  dsn: 'https://88c48b1b16d1657bf55218fb9a73107e@o4508374916726784.ingest.us.sentry.io/4508374919479296',
  integrations: [moduleMetadataIntegration()],
  transport,
  beforeSend: event => {
    if (event?.exception?.values?.[0]?.stacktrace?.frames) {
      const { frames } = event.exception.values[0].stacktrace;
      // Find the last frame with module metadata containing a DSN
      const routeTo = frames
        .filter(frame => frame.module_metadata && frame.module_metadata.dsn)
        .map(v => v.module_metadata)
        .slice(-1); // using top frame only - you may want to customize this according to your needs

      if (routeTo.length) {
        // eslint-disable-next-line no-param-reassign
        event.extra = {
          ...event.extra,
          [EXTRA_KEY]: routeTo,
        };
      }
    }

    return event;
  },
});

Remote Application RSBuild Config :
  tools: {
    webpack: {
      plugins: [
        sentryWebpackPlugin({
          moduleMetadata: ({ release }) => ({
            dsn: "REMOTE_DSN",
            release,
          })
        })
      ],
    },
  },

on printing event which is being sent, there are not meta data inside the events when printed from shell application side. Any idea why?

@basitqayoom not sure about your rsbuild setup but have you tried the setup in the comment above (adding the config to tools.rspack)?

Closing for cleanup. See:

I haven't tried it myself tbh but I'd assume that you would just have to add the webpack plugin here: https://rspack.dev/guide/migration/webpack#community-plugins
What exactly isn't working?

If this is still an issue, please re-open.

authors: AbhiPrasad andreiborza basitqayoom chargome eliliam ephill lforst mengbo-ji


text:
Convert the README and CHANGELOG files from reStructuredText to Markdown

Q
A

Branch?
master

Bug fix?
no

New feature?
no

BC breaks?
no

Deprecations?
no

Tests pass?
yes

License
MIT

I converted the README and CHANGELOG files from the reStructuredText format to Markdown, which GitHub renders correctly. I would like to add the Sentry logo as image of the README header, but I would need it in PNG format. The logo on your website is instead an SVG, which GitHub does not accept.

I just realized one downside of converting CHANGELOG is that if we wanted to display it inside of our docs we wouldn't be able to (since the docs are RST).

Unfortunately I was hoping that you did not show the changelog on your docs, because I knew there could have been problems of format. It's up to you to decide what do you, but I strongly suggest to keep Markdown and just link from the docs to be file here on GitHub because probably most developers will always look at the changelog here instead of in the docs. Same goes for the README

@ste93cry thats fair -- im ok with future solutions being a permalink vs embedding

I saw you already merged the PR, so it's late, but what about the question of the logo? It would be cool to have it in the README

@ste93cry sorry missed that -- we could add it: https://sentry.io/branding/

Ops, I missed that page and I found out that linking an SVG inside an img tag on GitHub works. I opened #435 to add the logo

repo: getsentry/sentry-php


text:
feat: Split draft and publish flows

Add intermediate "Save Draft" step before a post can be published.
Fixes GH-56

repo: getsentry/vanguard


text:
PROOF server "running" contextualization

When you click " start proof server" it nearly instantly says "Job Status: RUNNING". but that really means that the process to create a server is running NOT the server itself.  IF the server itself is running then the "Current PROOF server info" should be available.

Another issue I'm running into every time is basically at some point we should be polling to see if the API from the started server is receiving input and responding b/c it's delayed even once your server gets a job on a node.  THEN if a basic API call comes back successful, THEN we should say, "PROOF server is Running".
Just my 2 cents.

Yeah, now my proof server appears to be running but yet I can't get a response on the other tab.

When you're using proof, the user, IMHO expects that if "it's running" then functions on all tabs should work.  However that delay for the difference between the server running and the API actually working so all the tabs work, should be invisible to the user.  Don't tell them it's working if the part of it that they care about is not yet working. ;)

Thanks. This should be fixed on dev branch.

authors: sckott vortexing

repo: getwilds/proof


text:
crash

xcode 12.1, simulator iphone 12,

flutter 1.22.3

fixed. 1db4d670602b3ed3e55bef4df40bbcd97481515e


text:
Update sbt to 1.9.9

About this PR
📦 Updates org.scala-sbt:sbt from 1.6.2 to 1.9.9
📜 GitHub Release Notes - Version Diff
Usage
✅ Please merge!
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

⚙ Adjust future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.scala-sbt", artifactId = "sbt" } ]

Or, add this to slow down future updates of this dependency:
dependencyOverrides = [{
  pullRequests = { frequency = "30 days" },
  dependency = { groupId = "org.scala-sbt", artifactId = "sbt" }
}]

labels: library-update, early-semver-minor, semver-spec-minor, version-scheme:early-semver, commit-count:1

Superseded by #158.

repo: gfc-collective/gfc-id


text:
Setup automatic builds for Windows

AFAIK the most used service is AppVeyor which is free for opensource projects.
(cc @retep998)

This seems handy: https://github.com/Eljay/directx-sys/blob/master/appveyor.yml
Apparently, no DX12 as of Sep 2015

Looks like DX12 will come any moment now - http://help.appveyor.com/discussions/problems/3531-cannot-run-direct12-unit-test

Also see https://github.com/retep998/winapi-rs/blob/master/appveyor.yml

I've added https://ci.appveyor.com/project/kvark/gfx

I've set up the build script and a webhook for gitter. However, I think I got the target config wrong:
"stable-x86_64-pc-windows-msvc" - says it can't be found

You can't specify stable- like that because the stable distributions aren't distributed like that. You have to explicitly reference the stable version, like I do with 1.4.0-x86_64-pc-windows-gnu, or you have to parse the stable channel file to figure out what the current version of stable is.

@retep998 Ugh... ok, we can use 1.7.0 for now.
Also, this is the second error:

CMake Error: CMake was unable to find a build program corresponding to "Unix Makefiles"

How about... we feature gate glfw or limit it to non-Windows builds?

:+1: to gating GLFW

I believe this is working now

authors: fkaa kvark retep998

repo: gfx-rs/gfx


text:
Support product flavor

makeGrayscaleLauncherIcon() is only support main product flavor.
This PR for supporting other product flavor of main.
makeGrayscaleLauncherIcon(file("src/free"), "ic_launcher.png", ["freeDebug"])

これflavorがない場合でも動きますかね？

もちろん〜
makeGrayscaleLauncherIcon(file("src/main"), "ic_launcher.png", ["debug"])

と書けばdebugディレクトリに灰色アイコンができますぞー

naruhodo. LGTM

authors: gfx tomorrowkey

repo: gfx/gradle-android-utils


text:
Bash Autocomplete Depends on bash-completion Library

It uses _get_cword, which is part of the bash-completion library.

I'm not seeing the problem here. If you want to use bash autocomplete shouldn't you have the bash-completion library installed anyhow?

If you want to use bash autocomplete shouldn't you have the bash-completion library installed anyhow?

No, an external library is not necessary. Bash has builtin commands for this.

@sshaw I'd love to see how much code is required to remove the dependency on bash-completion. Care to throw together a PR for us?

authors: josephholsten jschpp sshaw


text:
zfile: fix build and runtime errors

As reported in #1147, the build fails if either zlib.h or lzma.h are unavailable.
Additionally, the feof check in zfile.c is backwards and causes a "truncated file" abort after searching the first zipped file.

FWIW, I addressed this issue with a simple workaround at #1146 : https://github.com/ggreer/the_silver_searcher/pull/1146/commits/bf375de7b1177df7eb08eddc13c33c2e188c81c5 .
Your fix is clearly more complete, however, it does clutter the code quite a bit.
IMHO the best fix would be to refactor zfile.c such that it doesn't need gazilion #ifdefs. Or, alternatively, take the approach in my PR and just tie fopencookie to both lzma and zlib availability (and further remove the remaining #ifdefs - which I didn't do in my PR). There is a fallback anyway, so I think this approach is not too terrible.
Code with so many #ifdefs is not fun to maintain...

Good point; I know the pain of dealing with code that's #ifdef'd to hell.
I can refactor zfile.c to put all of the zlib/lzma specific code into their own functions with a wrapper to check cookie->ctype, which would clean up zfile_read a lot.

I tried adding a layer of abstraction (2 layers of inline functions, with stubs #define'd for unsupported modes), but it might be even worse than littering zfile_read with #ifdefs.
Perhaps your solution to just require both zlib and lzma to support fopencookie is better. (has anyone  benchmarked whether fopencookie even gives a noticeable performance boost?)
The alternate version is on my zfile-fix-redo branch: aswild/the_silver_searcher@24995eb239799e52ae09d47b7520d50a3b8b606c
Regardless of the above, I think we do need some form of my second commit to fix the feof handling, that missing ! totally breaks zfile searching unless the file sizes happen to line up nicely. (aswild/the_silver_searcher@09b0bd8545d08d17ef70893fa15ae644e0bb8b2f)

authors: aswild avih


text:
Update_attributes_analysis_methods

Fixes issue (if applicable) #
Proposed Changes

Updated the attribute slot to the Analysis Method

Checks

[ ] Builds locally

Update addressed in #169

authors: anandhi-iyappan mauerk

repo: ghga-de/ghga-metadata-schema


text:
Feature/config to generic dao base

Title for squash and merge:
add config to Generic DAO base

Description for squash and merge:
It is now part of every DAO interface that config
params are passed via a pydantic BaseSettings class.

Codecov Report

Merging #35 (8a4a6df) into dev (7bc4c9a) will increase coverage by 0.28%.
The diff coverage is 91.66%.

@@            Coverage Diff             @@
##              dev      #35      +/-   ##
==========================================
+ Coverage   80.08%   80.36%   +0.28%
==========================================
  Files          11       12       +1
  Lines         477      489      +12
==========================================
+ Hits          382      393      +11
- Misses         95       96       +1

Impacted Files
Coverage Δ

ghga_service_chassis_lib/utils.py
68.75% <80.00%> (+5.11%)
:arrow_up:

ghga_service_chassis_lib/s3.py
73.45% <100.00%> (+0.16%)
:arrow_up:

ghga_service_chassis_lib/s3_testing.py
100.00% <100.00%> (ø)

Continue to review full report at Codecov.

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

authors: KerstenBreuer codecov-commenter

repo: ghga-de/ghga-service-chassis-lib


text:
Migrate NavigationBrand to function components

Migrate NavigationBrand to function components.

Closed by #28


text:
can we do powerManagement.cpuFreqGovernor onDemand

 powerManagement.cpuFreqGovernor = "ondemand";

It doesn't seem to be working:
$ sudo cpupower frequency-info
analyzing CPU 3:
  no or unknown cpufreq driver is active on this CPU
  CPUs which run at the same hardware frequency: Not Available
  CPUs which need to have their frequency coordinated by software: Not Available
  maximum transition latency:  Cannot determine or is not supported.
Not Available
  available cpufreq governors: Not Available
  Unable to determine current policy
  current CPU frequency: Unable to call hardware
  current CPU frequency:  Unable to call to kernel

$ sudo lshw -c cpu
  *-cpu:0
       description: CPU
       product: cpu
       physical id: 1
       bus info: cpu@0
       capabilities: fp asimd evtstrm aes pmull sha1 sha2 crc32 cpuid
  *-cpu:1
       description: CPU
       product: cpu
       physical id: 2
       bus info: cpu@1
       capabilities: fp asimd evtstrm aes pmull sha1 sha2 crc32 cpuid
  *-cpu:2
       description: CPU
       product: cpu
       physical id: 3
       bus info: cpu@2
       capabilities: fp asimd evtstrm aes pmull sha1 sha2 crc32 cpuid
  *-cpu:3
       description: CPU
       product: cpu
       physical id: 4
       bus info: cpu@3
       capabilities: fp asimd evtstrm aes pmull sha1 sha2 crc32 cpuid

Sep 18 09:25:01 surfer cpupower[677]: Setting cpu: 0
Sep 18 09:25:01 surfer cpupower[677]: Error setting new values. Common errors:
Sep 18 09:25:01 surfer cpupower[677]: - Do you have proper administration rights? (super-user?)
Sep 18 09:25:01 surfer cpupower[677]: - Is the governor you requested available and modprobed?
Sep 18 09:25:01 surfer cpupower[677]: - Trying to set an invalid policy?
Sep 18 09:25:01 surfer cpupower[677]: - Trying to set a specific frequency, but userspace governor is not available,
Sep 18 09:25:01 surfer cpupower[677]:    for example because of hardware which cannot be set to a specific frequency
Sep 18 09:25:01 surfer cpupower[677]:    or because the userspace governor isn't loaded?
Sep 18 09:25:01 surfer systemd[1]: Finished CPU Frequency Setup.

$ sudo modprobe cpufreq_governor
modprobe: FATAL: Module cpufreq_governor not found in directory /run/booted-system/kernel-modules/lib/modules/6.4.11

Oct 24 16:33:20 surfer hm-activate-kghost[726]: Starting Home Manager activation
Oct 24 16:33:20 surfer cpupower[724]: Setting cpu: 0
Oct 24 16:33:20 surfer cpupower[724]: Error setting new values. Common errors:
Oct 24 16:33:20 surfer cpupower[724]: - Do you have proper administration rights? (super-user?)
Oct 24 16:33:20 surfer cpupower[724]: - Is the governor you requested available and modprobed?
Oct 24 16:33:20 surfer cpupower[724]: - Trying to set an invalid policy?
Oct 24 16:33:20 surfer cpupower[724]: - Trying to set a specific frequency, but userspace governor is not available,
Oct 24 16:33:20 surfer cpupower[724]:    for example because of hardware which cannot be set to a specific frequency
Oct 24 16:33:20 surfer cpupower[724]:    or because the userspace governor isn't loaded?


text:
¿En que formato debe estar el certificado digital que utiliza greenter?

Buenos días @giansalex , el certificado digital que adquirí está en formato .p12 y para sunat tengo entendido que tengo que convertirlo a formato .cer pero para greenter ¿ en qué formato debe estar?.
Gracias de antemano!.

Greenter es en formato .pem

en requerimientos, indica el formato del ceritificado, creo que puedes convertirlo a .pfx, y siguiendo estos pasos obtener el .pem (clave privada + publica)

@giansalex  gracias!

repo: giansalex/greenter


text:
add github workflows

This PR:

adds github workflows

Checklist

[x] Update changelog in CHANGELOG.md.
[x] Make sure values.yaml and values.schema.json are valid.

This should be generated by giantswarm/github. LMK if you need help with that.

authors: kopiczko vxav

repo: giantswarm/cluster-cloud-director


text:
Fix typo in CAPA private suite name

What this PR does
Fixed a copy&paste typo.
Checklist

[x] Update changelog in CHANGELOG.md.

Trigger e2e tests

/run cluster-test-suites

The failure are unrelated so I'm going to add the skip label as this is only a string string change.

repo: giantswarm/cluster-test-suites


text:
Use ARN from config repository

Tested on giraffe.

can we release it as well? so I can  push both changes into 20.0.0?

can we release it as well? so I can push both changes into 20.0.0?

I am waiting for approval from biscuit here:
https://github.com/giantswarm/config/pull/232

authors: MarcelMue calvix

repo: giantswarm/kyverno-policies


text:
extending waiting period for AWSClusterUpdateFailed

Larger clusters might take longer to update, depending on the size of
the cluster and the ASG settings of pause time between rolling nodes.
Issue: https://github.com/giantswarm/giantswarm/issues/23291

Going with a ping ✌🏻

repo: giantswarm/prometheus-rules


text:
Install on Mac OS X - libiconv is a problem again

Issue #584 seems to have come alive again on Mac OS X High Sierra 10.13.5. If you use the brew version of the iconv library you get the error message:
Undefined symbols for architecture x86_64:
      "_iconv", referenced from:
          _hs_iconv in libHSbase-4.10.1.0.a(iconv.o)

But if you follow the instructions suggested in the ticket and edit slack.yaml to include:
extra-lib-dirs:
    - /usr/lib
extra-include-dirs:
    - /usr/include

you get the error message:
dyld: Library not loaded: @rpath/libiconv.2.dylib
      Referenced from: /private/var/folders/zd/g08zsw0158v7fnh39l95n3bm0000gn/T/stack78541/glib-0.13.6.0/.stack-work/dist/x86_64-osx/Cabal-2.0.1.0/build/System/Glib/StoreValue_hsc_make
      Reason: Incompatible library version: StoreValue_hsc_make requires version 9.0.0 or later, but libiconv.2.dylib provides version 7.0.0

By switching back and forth between the two options everytime I got the error I did manage to reach the end and build iHaskell. Not sure how you can fix this, but the above is crazy.

I don't see how this is an issue with IHaskell.

authors: AronT-TLV vaibhavsagar

repo: gibiansky/IHaskell


text:
IE9 doesn't work

Tried to get it running with IE9, stops on setCallerURL...somehow the flash isn't initialized properly.

No similar issue reported so far, so assuming environment-specific issue. Closing.

authors: gimite mendrik

repo: gimite/web-socket-js


text:
Fix bugs in expression evaluator

I realized that some operations weren't using wrapping arithmetic.
While fixing these I realized that the reason these weren't causing
assertions was that several tests were using DW_OP_skip and simply
jumping over the remaining tests.  Fixing this in turn revealed a few
more bugs.

Coverage increased (+0.4%) to 91.122% when pulling 6b658f1552ae5b34f191f254a7b04783205fd29e on tromey:wrapping-operators into 9285db308de1529b9adc8259499ffc4d4eb2d001 on gimli-rs:master.

Coverage increased (+0.4%) to 91.151% when pulling 575740a7222f046acbbe49c1c80a70a729d5b1e1 on tromey:wrapping-operators into 9285db308de1529b9adc8259499ffc4d4eb2d001 on gimli-rs:master.

LGTM feel free to merge yourself :)

authors: coveralls fitzgen tromey

repo: gimli-rs/gimli


text:
Controlling the pulse width

Is there a way to set the pulse width. It looks like it's currently about 5us on the nano.

No. The pulse is set to high by the output compare match and set to low in the interrupt.
Changing it configurable, would me

No. The pulse is set to high by the output compare match and set to low in the interrupt.
Changing it configurable, would me

It seems my driver card does not react on a 5us pulse. How would I solve this? Is there not way to make a fixed patch for creating a longer pulse. I can't find in the code where the timer is set to 5 us.

authors: XenuIsWatching gin66 lboclboc


text:
Update ci.yml

Proposed changes
Add Node 14 to unit-test matrix. Node 10 reaches end-of-life on 2021-04-30, need to update the support matrix
Types of changes
What types of changes does your code introduce to the project: Put an x in the boxes that apply

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

Checklist
Put an x in the boxes that apply. You can also fill these out after creating the PR. If you're unsure about any of
them, don't hesitate to ask. We're here to help! This is simply a reminder of what we are going to look for before
merging your code.

[x] I have read the CONTRIBUTING doc
[x] I have signed the CLA
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] I have added necessary documentation (if appropriate)
[ ] Any dependent changes have been merged and published in downstream modules

Further comments
If this is a relatively large or complex change, kick off the discussion by explaining why you chose the solution you
did and what alternatives you considered, etc...

Thank you for your contribution Jason.

repo: ging/fiware-pep-proxy


text:
Fill holes in data not working in 0.9.7

Hi,
I'm trying to use the functionality to fill holes in data in my line chart but it won't work. This is my config:
lineChartOptions:any = {     animation: false,     responsive: true,     fullWidth: true,     chartPadding: {       right: 40     },     lineSmooth: Chartist.Interpolation.cardinal({       fillHoles: true,     }),     low: 0   };
which looks like this:

How to I make this work? I get no errors and everything is working except for this.
On version 0.9.7

Tested with 0.9.8. does not seem to be an issue. http://jsbin.com/linegah/edit?js,output

Please re-open if issue persists

@gionkunz Thank you, will check later.

authors: danielalvenstrand gionkunz

repo: gionkunz/chartist-js


text:
BUG 86 : en tant que CO, quand j'essaye de me rattacher a un carnet, ca tourne et ne fonctionne pas.

https://mattermost.incubateur.net/betagouv/pl/tcti5a8hebbefjr1pqun9unniy

Il existe 4 entrées dans la table orientation_manager qui n'ont pas de compte associé (ce qui est théoriquement impossible). Ce sont tous des orientation_manager du 86.
Je n'ai aucune idée de comment c'est possible, mais pour régler le souci il faudrait supprimer ces 4 orientation_manager qui ne peuvent de toute façon se connecter.

Voici la requête qui permet de diagnostiquer ces comptes :
SELECT orientation_manager.firstname,
       orientation_manager.lastname,
       orientation_manager.email,
       a.id,
       d.label
from orientation_manager
         LEFT JOIN account a on orientation_manager.id = a.orientation_manager_id
INNER JOIN deployment d on orientation_manager.deployment_id = d.id
where a.id is null;

Le code qui pose problème est AddOrientationManagerForm.svelte ligne 27
$: orientationManagerOptions =
		$orientationManagerStore.data?.orientation_manager.map((pro) => ({
			name: pro.account.id,
			label: displayFullName(pro) || pro.email,
		})) ?? [];

pro.account peut être nul. Il faudrait probablement filtrer les OM qui n'ont pas de compte

:tada: This issue has been resolved in version 1.246.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:

authors: HadrienMP chdechamps service-dev-gip-inclusion vjousse

repo: gip-inclusion/carnet-de-bord


text:
Types of Notifications in Android Application

Description
In this I will be explaining about the different types of Notifications that can be implemented in our App. These include :

Simple Notification
Big Text Style Notification
Big Picture Style Notification
Inbox Style Notification
Performing Actions on Notifications

Domain
Android Dev (Java)
Type of Contribution
Documentation
Code of Conduct

[X] I follow Contributing Guidelines & Code of conduct of this project.

@VikasPandey121 Can I work on this Issue?

/assign

repo: girlscript/winter-of-contributing


text:
Mutable and immutable objects in Python

Description 📜
Create README.md for mutable and immutable objects in Python.
Explained the concept with examples and codes.
Fixes #4353

Type of change 📝

[ ] Au#dio (Should be in mp3 format Includes speech clarity, Concise ,Low distortion)
[ ] Vi#deo (Animations, screen-recordings, presentations and regular explanatory films are all possibilties etc)
[x] Documentation (Content Creation in the form of codes or tutorials)
[ ] Other (If you choose other, Please mention changes below)

Domain of Contribution 📊

[ ] Android Dev #(Flutter)
[ ] Android Dev #(Java)
[ ] Android Dev #(Kotlin)
[ ] Backend Dev #(Java)
[ ] Backend Dev #(.NET)
[ ] Backend Dev #(PHP)
[ ] Backend Dev #(Python)
[ ] C/#CPP
[ ] Competitive #Programming
[ ] Cyber #Security
[ ] DS#A
[ ] Data#base
[ ] Datascience with #Python
[ ] Datascience with #R
[ ] Frontend Dev #HTML/CSS/JS
[ ] Frontend Dev #React/Angular/Vue
[ ] Go#lang
[ ] Interview #Prep
[ ] Java_#Domain
[ ] JavaScript_#Domain
[ ] ME#RN
[ ] Machine #Learning
[ ] Open #Source
[x] Python_Domain
[ ] Ru#st
[ ] Statis#tics
[ ] UI/#UX

Checklist ✅

[x] I follow Contributing Guidelines & Code of conduct of this project.
[x] My changes generates no new warnings.
[x] I'm GWOC'21 contributor

Screenshots / Gif (Optional) 📸

@AbcSxyZ
I want my PR to be counted for Hacktoberfest as well. It would be great if you could add the Hacktoberfest label.

@sakshi430 not a maintainer at all, just a random guy who went on your PR :)

Oh.. okay.


text:
HTML video and audio tag

Description 📜
Documentation on HTML  audio and video tag
Fixes #5180

Type of change 📝

[x] Doc#umentation (Content Creation in the form of codes or tutorials)

Domain of Contribution 📊

[x] Frontend Dev #HTML/CSS/JS

Checklist ✅

[x] I follow Contributing Guidelines & Code of conduct of this project.
[x] I have performed a self-review of my own code or work.
[x] I have commented my code, particularly in hard-to-understand areas.
[x] My changes generates no new warnings.
[x] I'm GWOC'21 contributor

@2024-SANDHYA Please review my PR.

But in the issue it is said to make two different files. @2024-SANDHYA

But in the issue it is said to make two different files. @2024-SANDHYA

Merge both in one @yellowberard

@2024-SANDHYA I have merged both the files.

63% Plagiarized

@2024-SANDHYA I have made the changes do I have to open a new PR for this?


text:
gwithin, pwithin, gintersect, notgwithin, notpwithin, notgintersect accept geohash

In gitlab by @sylvaingaudan on Jul 10, 2017, 14:14
Need
The ARLAS API accepts geohash parameters when specifying filters
Feature
If a parameter starts with the hash character (#) then the following string must be considered as a  list of geohash. The items are seperated with a comma. The described geometry is the UNION of the geohash.
Example

pwithin=#ag4,4tp32

means any object having it centroid within ag4 or within 4tp32

In gitlab by @sylvaingaudan on Jul 10, 2017, 14:14
changed the description

In gitlab by @sfalquier on Jul 10, 2017, 14:34
added ~1918367 and removed ~1918908 labels

In gitlab by @sfalquier on Jul 10, 2017, 14:34
added ~1918908 label

In gitlab by @sfalquier on Jul 12, 2017, 15:37
added ~1918908 and removed ~1918367 labels

In gitlab by @sfalquier on Jul 20, 2017, 09:16
changed milestone to %6

In gitlab by @sylvaingaudan on Jul 17, 2017, 15:27
added ~2329926 label

In gitlab by @sfalquier on Jul 20, 2017, 09:17
removed milestone

In gitlab by @sylvaingaudan on Jul 20, 2017, 10:25
removed ~1918908 label

In gitlab by @sylvaingaudan on Jul 21, 2017, 12:13
changed milestone to %6

In gitlab by @sfalquier on Jul 28, 2017, 10:27
added ~2377174 label

In gitlab by @sylvaingaudan on Aug 4, 2017, 09:31
changed milestone to %10

In gitlab by @sylvaingaudan on Aug 10, 2017, 08:29
changed milestone to %11

In gitlab by @sylvaingaudan on Aug 4, 2017, 09:53
removed ~2424198 label

In gitlab by @sfalquier on Aug 25, 2017, 13:46
added ~2424198 label

In gitlab by @sylvaingaudan on Sep 8, 2017, 16:01
changed milestone to %12

In gitlab by @sylvaingaudan on Sep 11, 2017, 11:48
changed milestone to %13

In gitlab by @sfalquier on Sep 15, 2017, 11:00
removed ~2416396 label

In gitlab by @sfalquier on Sep 15, 2017, 11:00
changed milestone to %14

In gitlab by @sfalquier on Sep 15, 2017, 11:08
changed milestone to %15

In gitlab by @sylvaingaudan on Sep 22, 2017, 14:09
changed milestone to %16

In gitlab by @sylvaingaudan on Oct 9, 2017, 10:29
changed milestone to %17

In gitlab by @sfalquier on Oct 13, 2017, 09:26
changed milestone to %18

In gitlab by @sfalquier on Oct 13, 2017, 09:26
removed ~2424198 label

In gitlab by @sfalquier on Oct 20, 2017, 10:34
removed milestone

repo: gisaia/ARLAS-server


text:
Implemented filtering by repository on my tickets page

OK, I think I got it right now. Pulled develop, then merged my ticket-57 changes on top of it.

Aside from the little comments I made we have two larger problems.  One is easily fixable.  The other.... not sure.

You can't search by the repository name.  It's problematic because of punctuation marks, slashes, etc.  Instead we must specify and search for the rid which is a hash of the repository name.  So everywhere you specify a repository name value it must be the rid instead.  RepositoryModel.getRID() will give you this value.

We have a scaling problem.  I have 50+ repos in my test environment.  This generates a lengthy dropdown menu which scrolls offscreen.  And it really won't scale for those with hundreds or thousands of repos so we need a better plan.  The bootstap in-use is really old and we don't have scrollable dropdown menus.  :(  I think we need to list only the repos that are in the infiltered My Tickets resultset.  That means we need to execute two Lucene queries.  Hopefully the performance impact will be minor compared to the entire page-loading cycle.

@James, re-submitted with changes based on your code review. Also changed the repository options to constrain to those relevant to the user's watched tickets.
(Sorry about the changes of tabs to spaces, please ignore white-space differences when you compare.)

Good work.  We are very close now and this will be a nice usability improvement.

@gitblit These changes should be it. If no more review comments, I'll proceed with fixing the mentions bug, err, mentioned above.

Thanks Eddy!  Merged.

A pleasure. I'd really like to look into implementing tickets across groups. Is that on the roadmap for 1.7.0? Otherwise, I can pick another 1.7.0 item.

Tickets are intricately tied to repositories.  It would take some heroic changes (i.e. likely not mergeable) to accommodate  that.
If you really wanted to explore that I would suggest implementing some sort of ticket-container object with it's own persistence mechanism and UI.  Maybe this could be a plugin but I haven't spent more than 30 seconds thinking about this so maybe that is a dumb idea.  ;)

Hmm. I forgot about the integration of tickets within repositories, but your idea of a “repository group ticket service” is worth pursuing. I’ll think about that a bit more this week-end.
From: James Moger [mailto:notifications@github.com]
Sent: 27 May 2015 22:18
To: gitblit/gitblit
Cc: Eddy Young
Subject: Re: [gitblit] Implemented filtering by repository on my tickets page (#259)
Tickets are intricately tied to repositories. It would take some heroic changes (i.e. likely not mergeable) to accommodate that.
If you really wanted to explore that I would suggest implementing some sort of ticket-container object with it's own persistence mechanism and UI. Maybe this could be a plugin but I haven't spent more than 30 seconds thinking about this so maybe that is a dumb idea. ;)
—
Reply to this email directly or view it on GitHub https://github.com/gitblit/gitblit/pull/259#issuecomment-106081914 .  https://github.com/notifications/beacon/AAEGeQ7SfcrFMsr3E7Ox_AOTwonZBlYoks5oNiwFgaJpZM4Eg_wa.gif

authors: gitblit jeyoung

repo: gitblit/gitblit


text:
review: quick glance

Potential Improvs

[x] would it make sense to make splitTransfer an external function as opposed to a public ? I read that it takes less gas and we know this function is not going to be used internally
[ ] should we add modifier / assert ensuring we invoke the ERC20.transferFrom if valueFirst & valueSecond are not 0 ?
[ ] should we emit an event after each successfull transaction is done. I know it would be taken care in the ERC20 contract but still?
[ ] should we come up with better variable names toFirst  & valueFirst ?

Not needed at this point but just popped into my head
would it make sense to iterate on this to make it reusable for bounties

supporting ETH tokens
mix and match of diff ERC20 / ETH tokens.
allowing more than two splits ( for advanced payout ). Maybe keep an upper limit of 5?

cc @owocki / @danlipert

looks liek splitTransfer() requires someone to have previously approve()ed a transfer on the token contract address. is that right?
should we wrap ERC20(tokenAddress).transferFrom() in a require()?
should we check the bbalance before we invoke transfer?

@thelostone-mc I changed the function from public to external - nice suggestion! Regarding the assert ensuring that the values aren't zero I suppose we could do that, but it would be extra gas for each call, and the underlying ERC20 contracts should ensure its a positive value. Events also increase gas, but we can add one if we have a good reason to. Regarding the variable names, let me know what you think would be a better naming convention.
@owocki This will require someone to approve() first, unfortunately. I don't think theres any other way we can do it, as theres no payable type of interface for ERC20 where we could trigger a function call when someone transfers an ERC20 token to this contract. Regarding checking the balance and adding a require, happy to add those if you think its neccessary, but I think its a bit redundant as the ERC20 contract will have a require or SafeMath sub() call protecting the balance ledger.

but I think its a bit redundant as the ERC20 contract will have a require or SafeMath sub() call protecting the balance ledger.

Ahh !  I ran into this when I built the first version of bounties (before standard bounties).  not all ERC20 implementations follow the spec to a t do a require and use safemath :)

I don't think theres any other way we can do it

Theres no way to do the approve() inline right?
I guess the only other way would be to pay the splitter contract the tokens, then allow each of the recipients to debit the splitter contract?  Your way is certainly simpler.

@owocki Unfortunately we can't do the approve() inline since it typically uses msg.sender to determine who is approve()-ing the token spend, and in this case that would be the contract that called it. Also that is horrifying that not all ERC20 tokens check to make sure you don't get a negative balance :O I'll add that check to our contract here

Unfortunately we can't do the approve() inline since it typically uses msg.sender to determine who is approve()-ing the token spend, and in this case that would be the contract that called it.

IIRC there is a way to call a sub contract, but with the parent contracts scope.  i forget the name of the function, but i recall that John Mardlin from Consensys Dilligence had told me about it.

Closing this out as we've got the base setup done which seems to be

authors: danlipert owocki thelostone-mc

repo: gitcoinco/Splitter


text:
🛑 inv.riverside.rocks is down

In 29305f2, inv.riverside.rocks (https://inv.riverside.rocks/) was down:

HTTP code: 0
Response time: 0 ms

Resolved: inv.riverside.rocks is back up in bbb88b8.

repo: gitetsu/invidious-instances-upptime


text:
🛑 invidious.lidarshield.cloud is down

In 2de73e3, invidious.lidarshield.cloud (https://invidious.lidarshield.cloud/) was down:

HTTP code: 0
Response time: 0 ms

Resolved: invidious.lidarshield.cloud is back up in 018e555 after 38 minutes.


text:
🛑 invidious.privacydev.net is down

In 9c171d8, invidious.privacydev.net (https://invidious.privacydev.net/) was down:

HTTP code: 0
Response time: 0 ms

Resolved: invidious.privacydev.net is back up in 05bbd96.


text:
🛑 whoogle.privacydev.net is down

In 84a9ab4, whoogle.privacydev.net (https://whoogle.privacydev.net) was down:

HTTP code: 0
Response time: 0 ms

Resolved: whoogle.privacydev.net is back up in 0a11e67 after 4 minutes.

repo: gitetsu/whoogle-instances-upptime


text:
🛑 whoogle.privacydev.net is down

In c68dbb7, whoogle.privacydev.net (https://whoogle.privacydev.net) was down:

HTTP code: 0
Response time: 0 ms

Resolved: whoogle.privacydev.net is back up in cbf1d7b.


text:
[GHSA-7wh2-wxc7-9ph5] WiX Toolset's .be TEMP folder is vulnerable to DLL redirection attacks that allow the attacker to escalate privileges

Updates

Affected products

Comments
according to https://github.com/wixtoolset/issues/security/advisories/GHSA-7wh2-wxc7-9ph5 3.14.0 has also been patched, so should not be affected by the vulnerability?

Hi there @hathawcs and @robmen! A community member has suggested an improvement to your security advisory. If approved, this change will affect the global advisory listed at github.com/advisories. It will not affect the version listed in your project repository.
This change will be reviewed by our Security Curation Team. If you have thoughts or feedback, please share them in a comment here! If this PR has already been closed, you can start a new community contribution for this advisory

Thanks. I fixed the advisory last night but got the range syntax wrong. I fixed it this morning so the new version should be published now.

authors: github ppv-milestone robmen

repo: github/advisory-database


text:
[GHSA-6pw3-8h9w-32gc] Apache Airflow vulnerable to OS Command Injection via example DAGs

Updates

Affected products
References

Comments
Add patch links related to CVE-2022-40127.

Hi @sunSUNQ, I'm not accepting this contribution because these links appear to be duplicates of https://github.com/apache/airflow/commit/372e699c2d1e11f7087b5340454d0a0a6a56fbf5, which is already in the reference links of the advisory. Thank you for your interest in GHSA-6pw3-8h9w-32gc!

authors: shelbyc sunSUNQ


text:
translate syntax_tips.md into ja-JP

Why:
reusables.search.syntax_tips was not translated into ja-JP.

What's being changed (if available, include any code snippets, screenshots, or gifs):

I translated https://github.com/github/docs/blob/43f4e15159284b61ecb0228cf368690c1ee8be7d/translations/ja-JP/data/reusables/search/syntax_tips.md into ja-JP.
Check off the following:

[ ] I have reviewed my changes in staging (look for the "Automatically generated comment" and click the links in the "Preview" column to view your latest changes).
[x] For content changes, I have completed the self-review checklist.

Writer impact (This section is for GitHub staff members only):

[ ] This pull request impacts the contribution experience

[ ] I have added the 'writer impact' label
[ ] I have added a description and/or a video demo of the changes below (e.g. a "before and after video")

@progfay Thanks for reaching out! We appreciate you taking the time to open these translation PRs. However, we don't accept changes to our translated files as we use an internal automated solution for translations. :yellow_heart:
See https://github.com/github/docs/blob/main/contributing/types-of-contributions.md#earth_asia-translations.
Please take a look at our help wanted section to find open issues you can work on.
I'm going to go ahead and close these translation PRs now. Thank you for your interest and passion for improving the docs! ✨

@steveward
Thanks for your reviewing!

we don't accept changes to our translated files as we use an internal automated solution for translations.

Sorry, I didn't know that.
I will try to contribute in another way!

authors: progfay steveward

repo: github/docs


text:
Update troubleshooting-authentication-to-a-repository.md

Why:
Closes: #31718

What's being changed (if available, include any code snippets, screenshots, or gifs):
replaced "PAT" with a more generic placeholder such as "USERNAME"

Check off the following:

[x] I have reviewed my changes in staging, available via the View deployment link in this PR's timeline (this link will be available after opening the PR).

For content changes, you will also see an automatically generated comment with links directly to pages you've modified. The comment won't appear if your PR only edits files in the data directory.

[x] For content changes, I have completed the self-review checklist.

@SoundaryaKoutharapu Thanks so much for opening a PR with this fix! I'll update the branch and get this merged once tests are passing 🍏


text:
Use consistent language for placeholders in AWS OpenID setup

Why:
These two phrases use a different language than other places. A follow-up to #34454
Closes: N/A

What's being changed (if available, include any code snippets, screenshots, or gifs):
Sentence structure changes from "Add ... here" to "Replace this with ..." to be consistent with other places:

Check off the following:

[x] I have reviewed my changes in staging, available via the View deployment link in this PR's timeline (this link will be available after opening the PR).

For content changes, you will also see an automatically generated comment with links directly to pages you've modified. The comment won't appear if your PR only edits files in the data directory.

[x] For content changes, I have completed the self-review checklist.

@hsnabszhdn Thanks so much for opening a PR with this fix! I'll update the branch and get this merged once tests are passing 🍏

@hsnabszhdn Thank you so much for opening a PR with this fix! ✨ This repo is currently in a code freeze, but we will get this merged as soon as the freeze ends on 9/16 🍏 thanks again!

@hsnabszhdn Thank you so much for opening a PR with this fix! ✨ This repo is currently in a code freeze, but we will get this merged as soon as the freeze ends on 9/17 🍏 thanks again!


text:
Update VisualStudio.gitignore

Ignoring lowercase *.cache as well, C# will generates those.

@MOROZILnic is this a new change to VS? How did you find this out?

Just randomly bumped into a file named *.csproj.GenerateResource.Cache in C# project (VS 2013 Professional). Besides that, there already a lot of names with both capitalisations in VisualStudio.gitignore

Thanks, but wouldn't your example *.csproj.GenerateResource.Cache have been covered by the existing rule *.Cache? Is it ever produced with a lowercase .cache?

Oh, my bad. Here is some examples of lowercase cache
DesignTimeResolveAssemblyReferences.cache
DesignTimeResolveAssemblyReferencesInput.cache
*.csprojResolveAssemblyReference.cache

Ok I was pretty sure it was just a typo, but just checking. Looks good if this is something you're seeing!

authors: MOROZILnic arcresu shiftkey

repo: github/gitignore


text:
Add the entire .idea folder

Reasons for making this change:
This folder is created purely at compile-time and will be re-written every time you compile with a JetBrains IDE. Additionally, all of the files in this directory control user-specific settings that will very often change for every user in the project. Let me know if/why this reasoning is incorrect please.
Links to documentation supporting these rule changes:
https://intellij-support.jetbrains.com/hc/en-us/articles/206544839/comments/200744669

Added JetBrains .gitignore
⇒  git status .idea/
On branch master
Your branch is up to date with 'origin/master'.

Untracked files:
  (use "git add <file>..." to include in what will be committed)

        .idea/project-name.iml
        .idea/dataSources.xml
        .idea/encodings.xml
        .idea/misc.xml
        .idea/modules.xml
        .idea/vcs.xml

nothing added to commit but untracked files present (use "git add" to track)

Closing this out because this outstanding question wasn't answered:

It's not clear to me what's in those files that might be per-user (it's been a while since I've used any JetBrains IDEs to see for myself). Care to elaborate on why you think those can be safely ignored?

I'm still in favour of versioning what can be shared between users of JetBrains IDEs, and ignoring .idea/ goes against the goal of the template.

See #3451 for an optional solution

authors: BrianMSheldon elkshadow5 iamandrewluca shiftkey


text:
Listing Github details for contributors between two tags

One of the steps in WP-CLI's release process is to run ./utils/contrib-list (ref) to produce a list of contributors between two tags (or an existing tag and master). The end result is something like this:

However, the script is broken-ish right now. I've been meaning to fix it, but thought it might make a good contribution to hub instead. Something you'd consider a PR for?

It's an interesting case but I don't think it belogs in hub core right now. A list of people's names is easily available through git log. I guess the complex part of this script would be mapping the email addresses to GitHub account handles?

I guess the complex part of this script would be mapping their email addresses to GitHub account handles?

Right — doing some amount of authenticated Github.com API request to fetch these details.

Here's a simple bash function that converts an email to a GitHub handle:
find_user() {
  curl -fsSL "https://api.github.com/search/users?q=in:email%20$1" | \
    grep '"login":' | cut -d'"' -f4
}

handle="$(find_user "mislav.marohnic@gmail.com")"
echo $handle #=> "mislav"

It will yield blank output if a user couldn't be found. In such cases, the person's full name from git log should be used instead of GitHub handle.

If this isn't going to happen, should we consider this issue closed then?

For this comfortable with PHP, here's what I ended up implementing for WP-CLI: https://github.com/wp-cli/wp-cli/blob/master/utils/contrib-list.php

authors: danielbachhuber mislav sinisterstuf

repo: github/hub


text:
Can't use hub fork command outside of my terminal

I have an issue where if I try to use hub fork command not from my terminal but from another environment I get access denied error.
I am not sure what I should source or do to make the error go away. I checked and I don't even have ~/.config/hub config file.
However this hub fork command works when run from my terminal.
What is it that I should source or do to make this command work and not throw this error:
github.com username: github.com password for  (never stored): Error forking repository: Unauthorized (HTTP 401)
Requires authentication

hub commands such as hub fork don't necessarily need to be run while attached to a terminal, but they do need some environment variables, such as HOME to find your $HOME/.config/hub configuration file where OAuth token is stored. Alternatively you can ensure that GITHUB_TOKEN is set, if you don't want to rely on having $HOME/.config/hub file. When GITHUB_TOKEN is detected, its value is used as the OAuth token and you're never prompted for username/password.

authors: mislav nikitavoloboev