text:
Do I need to configure anything in AndroidManifest.xml？

I want to use the openDocumentTree method to authorize the Android/data/somedirectory/ Directory
Then use the getDocumentContent method to read or write sub files in the directory, so I don't need to authorize the files I want to read and write again through the openDocument method.
Because my current situation is that if I edit the file using another app (an app of the file management editing type), the sub file permissions that I authorize using the openDocument method within my application will become invalid, and then I need to call the openDocument method again to let the user select the file to regain read and write permissions.
So I want to make sure that my application won't invalidate my file permissions when other applications edit the file. Do I need to configure something in AndroidManifest.xml? How can I achieve this

So I want to make sure that my application won't invalidate my file permissions when other applications edit the file.

This is the default behavior, once the user grant permission over a Uri, you have permission over all subfiles (recursively), so even if other app edit a file that is in under your granted tree, you still have access to it, also applies for created files.
I need some code and details to know what is exactly happening in your case, can you show how you are using and what is your target Android version?

A side-note: Android/data folder is not permitted to be selected anymore (even if the user wants!) on higher Android versions (12+), so be aware of this limitation (same applies for MANAGE_EXTERNAL_STORAGE permission) see https://www.reddit.com/r/Android/comments/wru35i/clearing_up_confusion_about_how_to_access/.

TL;DR: Try declaring the tree owner of your document at fUri.
final fUri = Uri.parse(
- 'content://com.android.externalstorage.documents/document/primary%3AAndroid%2Fdata%2Fmark.via%2Ffiles%2Flogs%2Fapp.ini'
+ 'content://com.android.externalstorage.documents/tree/primary%3AAndroid%2Fdata%2Fmark.via/document/primary%3AAndroid%2Fdata%2Fmark.via%2Ffiles%2Flogs%2Fapp.ini'
);

What I could understand from your functions is that:

When you manually select the file, your code works because the returned Uri is containing the tree path plus the document uri, as it should be.
When you select the tree, you are trying to read the document path without the tree path, AFAIK this leads to SecurityException.

So here is an example that creates your dUri and fUri correctly and can possible fix your issue:
void main() {
  // Granted tree uri:
  final dUri = Uri(
    scheme: 'content',
    host: 'com.android.externalstorage.documents',
    path: '/tree/primary%3AAndroid%2Fdata%2Fmark.via',
  );

  // content://com.android.externalstorage.documents/tree/primary%3AAndroid%2Fdata%2Fmark.via
  print(dUri);

  // Sub-file uri:
  final fUri = dUri.replace(
    path: dUri.path +
        '/document/primary%3AAndroid%2Fdata%2Fmark.via%2Ffiles%2Flogs%2Fapp.ini',
  );

  // content://com.android.externalstorage.documents/tree/primary%3AAndroid%2Fdata%2Fmark.via/document/primary%3AAndroid%2Fdata%2Fmark.via%2Ffiles%2Flogs%2Fapp.ini
  print(fUri);
}

TD;DR: Follow these steps:

Use the following template:

content://com.android.externalstorage.documents/tree/{encodedTreeUri}/document/{encodedSubfileUri}

Replace encodedTreeUri with the URL encoded version of primary:{path-to-your-desired-storage-tree}, in your case: primary:Android/data/mark.via (The path must not start with a /).
Replace encodedSubfileUri with the URL encoded version of primary:{path-to-your-desired-storage-file}, in your case: primary:Android/data/mark.via/files/logs/app.ini (The path must not start with a /).
The final result is the subfile (aka child file) URI.

Dart implementation:
final Uri dUri = createTreeUri('Android/data/mark.via');
final Uri fUri =
    createDocumentUri(dUri, 'Android/data/mark.via/files/logs/app.ini');

Uri createTreeUri(
  String treePath, {
  String host = 'com.android.externalstorage.documents',
  String scheme = 'content',
  String storage = 'primary',
}) {
  assert(
    !treePath.startsWith('/'),
    'Please, remove `/` from the start of your [treePath].',
  );

  final Uri treeUri = Uri(
    host: host,
    scheme: scheme,
    path: '/tree/${Uri.encodeComponent('$storage:$treePath')}',
  );

  return treeUri;
}

Uri createDocumentUri(
  Uri treeUri,
  String documentPath, {
  String storage = 'primary',
}) {
  assert(
    !documentPath.startsWith('/'),
    'Please, remove `/` from the start of your [documentPath].',
  );

  final Uri documentUri = treeUri.replace(
    path:
        '${treeUri.path}/document/${Uri.encodeComponent('$storage:$documentPath')}',
  );

  return documentUri;
}

I must warn that this is specific for your use-case: generating a pre-defined Uri from a pre-defined path that will be in the primary storage of the device using the default android storage provider. This will not work for all use-cases, like generating a URI for a cloud storage provider or even a custom storage provider. This also may not work for some devices.

Do not re-use closed issues, open a new one.

authors: Rclsss alexrintt

repo: alexrintt/shared-storage


text:
Unnecessary icons for local files?

Should the icons in the local file list be removed?
They will always be the same, dependant on your pdf viewer.

Imo it's good to have icon there. The only thing which I'd change is size of it.

@hitechcrafters Want to create an issue and adjust the size to a bit smaller?

source: gharchive/issue

authors: alexweininger hitechcrafters

repo: alexweininger/recent-pdf

license: MIT

license_type: permissive

license_source: github-api


text:
Installation sur Mac

Comment dois je faire pour installé Jarvis sur mon Mac ?
Merci pour l'aide

Suis arrivé a installer jarvis mais j'arrive pas a configurer le speaker ni le micro.
J'ai une erreur lors de la configuration :
Selection of the speaker device
utils/configure.sh: line 138: aplay: command not found
Indicate the card # to use [0-9]: 1
Indicate the device # to use [0-9]: 1
Updating ~/.asoundrc...
Reloading Alsa...
sudo: /etc/init.d/alsa-utils: command not found
utils/configure.sh: line 134: play: command not found
J'entends pourtant Jarvis me dire Hello puis il quitte l'appli.

Normalement ça marche du premier coup il faut répondre oui que tu as bien entendu le son. Dans le doute il fait remettre les fichiers play_hw et rec_hw a "false" dans le dossier config/

Le 22 mai 2017 à 23:07, Boulogne Frédéric notifications@github.com a écrit :
Suis arrivé a installer jarvis mais j'arrive pas a configurer le speaker ni le micro.
J'ai une erreur lors de la configuration :
Selection of the speaker device
utils/configure.sh: line 138: aplay: command not found
Indicate the card # to use [0-9]: 1
Indicate the device # to use [0-9]: 1
Updating ~/.asoundrc...
Reloading Alsa...
sudo: /etc/init.d/alsa-utils: command not found
utils/configure.sh: line 134: play: command not found
J'entends pourtant Jarvis me dire Hello puis il quitte l'appli.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.

Merci . Je vais faire cela au plus vite :)

Clôture du ticket le temps que tu fasse les tests prévus.
N'hésites pas à nous en faire un retour 😉

authors: alexylem polbru40 wikijm

repo: alexylem/jarvis


text:
Release baresip version 0.5.0

testing and documentation ..

@alfredh, looks like I'll open a couple of pull request that I would prefer to be reviewed before 0.5.0 is cut.

Looks like I am done with testing.  With an exception below everything seems fine on OpenBSD-current.
I experience an issue when using libv4l2: baresip picks YU12 (V4L2_PIX_FMT_YUV420), and my video is distorted (topmost line is just repeated over and over).  I blame libv4l2 which is very buggy here (eg. it hides all native formats of my webcam), but I would appreciate if you could make a test build of baresip which forces YU12 and see whether your image is distorted.

version 0.5.0 is now released:
https://github.com/alfredh/baresip/releases/tag/v0.5.0
Announcement email:
http://lists.creytiv.com/pipermail/re-devel/2016-December/001154.html

authors: alfredh czarkoff

repo: alfredh/baresip


text:
dshow: remove qedit.h

Hello
According to this issue: https://github.com/alfredh/baresip/issues/464 I moved declarations of ISampleGrabberCB and ISampleGrabber into dshow.cpp so it does not require qedit.h file.
Here is MSVS build log.
@nicotyze would you like to compile with mingw please? I have some troubles with compilation both in current state and this pr. Thank you.
Alexey

Hello,
Successfully compiled (mingw) and tested (Win10) from my side as well.
Nicolas

thank you all !
also compile-time tested with mingw-w64 v5.0.4
Alfred

source: gharchive/pull-request

authors: Encamy alfredh nicotyze


text:
feat: add index exists method

The spec and tests for this new method are described here: https://github.com/algolia/algoliasearch-client-specs/pull/26

Here's the doc PR to update once it's done: https://github.com/algolia/doc/pull/3399

repo: algolia/algoliasearch-client-javascript

license: mit

license_source: bigquery


text:
Menu is too big

Actually it's still too big, but now on mobile 😄 / screens smaller than 15"

Maybe we could have more breakpoints than mobile / desktop? cc @JonasBa

Actually it's still too big, but now on mobile 😄 / screens smaller than 15"

We could just hide it on mobile otherwise, usecase of getting the v1 docs should be rare..

flex layout @JonasBa so it can takes the space remaining?

This seems to be closed in https://github.com/algolia/instantsearch.js/pull/2466

authors: Haroenv bobylito iam4x vvo


text:
fix(searchBox): handling pasting event with contextual menu.

fixes #467

@maxiloc I think we figured a more advanced way to handle that in https://github.com/algolia/instant-search-demo/pull/20 mentioned by @vvo in the issue.

Ok cool, I am modifying that

Ok cool, I am modifying that

:+1: awesome

I am not sure there's an "input" magical event in pure DOM

Should be pretty-well supported:
http://caniuse.com/#feat=input-event
But be careful, the order in which input, change, keypress, keyup and keydown are fired is tricky.

@maxiloc If you want to see how we did it in the demo: https://github.com/algolia/instant-search-demo/pull/20/files
@vvo's suggestion works pretty well because it also supports cutting and deleting with the mouse, plus the case where someone might paste text on to the end of an existing query.

So I wonder, @pixelastic can't we just use 'input' event for everything (it would handle keyup, change, paste..).
Then the only missing part would be that on IE9, deleting would not produce a new search, we can fix that separately.

I rebased, the handling of event is a bit different than jquery but it works

I rebased, the handling of event is a bit different than jquery but it works

Did you pushed? :D

So I wonder, @pixelastic can't we just use 'input' event for everything (it would handle keyup, change, paste..). Then the only missing part would be that on IE9, deleting would not produce a new search, we can fix that separately.

Yep, that could be a good idea (separate issue). We should fallback to classical keypress for IE9

Fixed test

added test for input event

This one is ready

I'll do a additional commit and merge, nice work @maxiloc :+1:

@maxiloc some comments from @pixelastic

@vvo I misunderstood the comment of @redox. I am gonna make the change

@pixelastic Rebased. Tests should be better

handled ie compatibility, we just won't have copy paste for ie9 for now
cc @vvo

I think you messed up something with the rebasing :)

weird :smile:

fixed

done

We are good, awesome work because it was a tricky one. If we have 0 bugs following this one then congrats :D

authors: dustincoates maxiloc pixelastic redox vvo


text:
fix(router): set correct ui state when instance is re-started

Summary
This PR fixes the router middleware to set correct initial ui state when instantSearchInstance is re-used.
When does this happen?
In Vue InstantSearch, when ais-instant-search component gets unmounted, it calls dispose().
https://github.com/algolia/vue-instantsearch/blob/2d6879be7a16bbf7fc6465891180304782d492e7/src/util/createInstantSearchComponent.js#L72:L74
Later when the component gets mounted again, it will call instantSearchInstance.start().
Why is it broken?
https://github.com/algolia/instantsearch.js/blob/0e151a9552092807ecbc6993f3f6193fef621f44/src/middlewares/createRouterMiddleware.ts#L60:L63
Inside createRouterMiddleware, we read the url from the router at creation of router middleware, whereas it was supposed to be at subscribe(). Thus, it reads only once, and when it's re-used it will stay outdated.

More info:
When the router middleware is created, it updates instantSearchInstance._initialUiState. At this point, instantSearchInstance.start() hasn't been called yet. When start() is called, it initialize the mainIndex with mainIndex.init(...) and even though corresponding widgets don't exist in the widget tree, it takes initialUiState into account and keep it for a while.
const search = instantsearch({
   indexName: 'hello',
   searchClient,
   initialUiState: {
    hello: {
      query: 'world'
    }
   }
})

search.start()

Even though there is no searchBox widget to hold query inside the uiState, it will hold query for a while.

If searchBox is added, then it will trigger a new search with the query (Before then, the search param doesn't include it.)
Before searchBox is added, if search.getUiState() is called, it will re-evaluate it and query disappears.

This seems inconsistent, considering we normally eliminate states from uiState if there is no corresponding widget (no matter we like the behavior or not.)

Solutions
Fix 1. Use setUiState instead of updating initialUiState.
Instead of manipulating the private instance variable _initialUiState, leveraging the public API setUiState seems solid. However, this unnecessarily triggers uiState change to instantsearch({ onStateChange }) and to middlewares. It also could cause one more search API calls (empty one + another one with the new state.) If we call setUiState from the router middleware when only there is something to set, then it's less problematic.
Another problem with this approach is, if corresponding widgets are not in the widget tree, the result of setUiState will go away, whereas what's written in initialUiState remains regardless.
Fix 2. Call middleware.subscribe() before mainIndex.init()
Then it can get to update _initialUiState before it's passed to mainIndex.init(), but this doesn't seem right. It's better to have middlewares started after we initialize the main index and settle down with states. Also, this change will break the insights middleware and possibly custom middlewares.
Fix 3. Re-create middlewares when restarted 👈
When the instance gets reused and restarted, we'd probably want to re-create the middlewares. The definition of middleware in IS.js is "a function that returns an object." We could re-run the functions to replace the middleware objects. This is a non-breaking change, and possibly will work. I will try it out and will keep this PR updated.

The tests are broken now. Don't review it yet. I'll fix them.

Rerunning e2e twice made it pass... Anyway reading the test case, it seems fine.


text:
Error when querying deleted app from the api

When looking up a deleted app through the API, not returning anything informative.
On python returns: HTTPError: {current round: <round number>}
with curl returns: {current round: <round number>}
for example:
curl "localhost:8980/v2/applications/238940466"

Try curl "localhost:8980/v2/applications/238940466?include-all=true"

@anya2go Did Will's comment help. If so, we'll close.

authors: anya2go winder yaovi-a

repo: algorand/indexer


text:
Add an ability to type or upload signatures

Description:
Currently, users can only add their signature to an invoice by drawing it on canvas. They should also have the option to either type their signature or upload an image of their handwritten signature too.
Acceptance Criteria:

[x] Add a signature field that opens a modal to the signing page.
[ ] Provide a "Draw Signature" option, allowing users to draw their signature on canvas
[ ] Provide a "Type Signature" option, allowing users to input their signature using a digital pen or touchpad.
[ ] Provide an "Upload Signature" option, allowing users to upload an image file of their handwritten signature (e.g., PNG, JPG).
[ ] Display the chosen signature on the document.
[ ] Make sure the user experience is intuitive and user-friendly.
[ ] Test the feature thoroughly to ensure it works consistently across browsers and devices.

Additional Information:
This feature will greatly benefit our users, as it simplifies the process of signing documents digitally and eliminates the need for printing, scanning, and uploading signed documents. It should improve user satisfaction and efficiency.

Example of UI:

✅ Implemented draw functionality within the signature modal:

✅ Implemented type signature functionality within the signature modal type tab:


text:
打包成exe执行的时候报错缺少文件或文件夹'C:\Users\82437\lAppData\Locall\Temp\ MEI32122\lfuasr\version.txt'

python程序 pycharm直接运行没问题，用pyinstaller打包了之后运行exe 报错缺少这个文件。
使用pipeline，   task='speaker-verification',
模型：speech_campplus_sv_zh-cn_16k-common'
funasr版本：1.0.14
modelscope版本：1.13.1

Sorry, I haven't used the tool you mentioned, so I can't provide any guidance.

authors: eugeneYz yfchenlucky


text:
能批量导入图片吗？

我们会有很多图片压缩成zip。服务端需要解压Zip，拿到一张张图片，服务将图片一张张上传到文件中心获得文件ID，根据规则解析图片名称，将文件ID按照要求存库。不知道这个框架能否导入含有图片的ZIP包

同问，我们这边也会有很多的自定义编码格式的文件，需要大量的导入与导出。不止只有图片，还有视频等其他。

这个可以支持，文件读写用的是一个SPI，下周写一个基本的demo出来

authors: ZeganLi clsaa sh442399601

repo: alibaba/AGEIPort


text:
Refine document to make users/developers easy to use GraphAr

Is your feature request related to a problem? Please describe.
A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]
Describe the solution you'd like
A clear and concise description of what you want to happen.
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
Add any other context or screenshots about the feature request here.

solved by pull request #64

repo: alibaba/GraphAr


text:
feat(interactive): Support VarChar as yet another property type

Currently Interactive only using string as vertex property. We also need to support string as edge property.
To support string edge property on csr storage, we need to ensure the maximum length of string, to reserve enough space.
Need more detail design and explanation.

The previous String type can be viewed as a varchar with a default maximum length as a fixed value.

A possible usage of VARCHAR can be
property_type:
  primitive_type: DT_VARCHAR(256)

property_type:
  var_char:
    max_length: 256

repo: alibaba/GraphScope


text:
Improve GetInnerVertex() by querying local fragment only

What do these changes do?
The current implementation of GetInnerVertex() is to query all the fragment, it's not needed as we only need the inner vertex

Codecov Report

Merging #2772 (6f0e497) into main (bf8d7b0) will increase coverage by 26.05%.
The diff coverage is n/a.

Additional details and impacted files

@@             Coverage Diff             @@
##             main    #2772       +/-   ##
===========================================
+ Coverage   41.39%   67.45%   +26.05%
===========================================
  Files          99       99
  Lines       10381    10381
===========================================
+ Hits         4297     7002     +2705
+ Misses       6084     3379     -2705

see 60 files with indirect coverage changes

Continue to review full report in Codecov by Sentry.

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

authors: codecov-commenter songqing


text:
refactor(interactive): Initiate Compiler Service with Yaml Configuration

…yaml

What do these changes do?
as titled.

Related issue number

Fixes

Codecov Report

Merging #3141 (f37a2b8) into main (2ba0922) will increase coverage by 0.00%.
Report is 9 commits behind head on main.
The diff coverage is n/a.

Additional details and impacted files

@@           Coverage Diff           @@
##             main    #3141   +/-   ##
=======================================
  Coverage   42.63%   42.64%
=======================================
  Files         100      100
  Lines       10817    10817
=======================================
+ Hits         4612     4613    +1
+ Misses       6205     6204    -1

see 2 files with indirect coverage changes

Continue to review full report in Codecov by Sentry.

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


text:
feat(interactive): Provide Cypher benchmark tool

What do these changes do?

Related issue number

Fixes #3862

Closing this PR for tmp as additional testing features are still to be integrated into the benchmark tool.


text:
Update DeviceInfo.scala

[BUG]/[FEATURE] DeviceMonitor init failed because of NPE.
What changes were proposed in this pull request?
Why are the changes needed?

If multiple directories are mounted to the same disk,the mounting information of the working directory may not be found .As a result,the npe occurs,causing DeviceMonitor  initialization failure.
What are the items that need reviewer attention?
Related issues.
Related pull requests.
How was this patch tested?
/cc @related-reviewer
/assign @main-reviewer

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

To let the UTs pass, plz also change the DeviceMonitorSuite
  val dfCmd = "df -h"

repo: alibaba/RemoteShuffleService


text:
added the generated_caption_mapper

as the title says, related feature request issue #116

Thanks  @zhijianma 's comments! I have modified the codes accordingly, see this commit.

Thanks for @HYLcool 's comments. I couldn't spare the time to fix and test these comments within the last few days. Please help me to continue developing this OP.

The Huggingface model Salesforce/blip2-opt-2.7b is too large (> 15.5GB). The runner provided by GitHub has only 14GB of disk space. So we decide to test this new OP locally and skip unit tests of it in the GitHub Action. @yxdyc @chenhesen

This OP has passed the local tests. Online unit tests on GitHub for this OP are skipped. Please review the latest commits. Thanks. @zhijianma @chenhesen

authors: HYLcool yxdyc

repo: alibaba/data-juicer


text:
impala和mongo获取驱动类错误

com.alibaba.druid.util.JdbcUtils类，getDriverClassName(String rawUrl)方法，建议新增对impala和mongo的支持

com.alibaba.druid.util.JdbcUtils类，getDriverClassName(String rawUrl)方法，建议新增对impala和mongo的支持

参考 https://github.com/alibaba/druid/issues/3423#issuecomment-526152631

repo: alibaba/druid


text:
有办法在SqlObject中获取对象的位置信息么？

目前知道lexer词法分析器中有位置信息，但在parser语法分析的时候并没有带给对应的sqlobject位置信息。
这块我们想给运维批量执行脚本时，对脚本做静态检查，把例如一些高风险语句delete没有where条件或where条件永真的语句找出来标红。没有位置信息就不太方便了

先确保是单线程的情况下，尝试在nextToken()包了一层try finally，把line的开始和结束记到一个全局静态对象中，然后在SqlObjectImpl的构造函数中用了这个全局静态的值，并记录到新增的成员变量里(不过pos好像不会因为换行改变)

Lexer.scanMutiLineComment() 没有对满足 \n的换行符做  line++ 动作导致行数不准
OracleLexer.scanComment()同理

OracleLexer在处理 \r\n时，少scanchar一次，如果补了行号会导致多一行。（少scan一次，那么nextToken拿到的就是\n导致多一行）

lexer 和 mysql的MySqlLexer同理
但 MySql的lexer有一点特殊，在indexOf('*') 处理后的stringVal直接被截断加入注释了
需要统计stringVal中的\n个数，553行那个for循环中


text:
Dependency Cycle When Using Spring Boot +Dubbo + Spring Data JPA

在使用Spring Boot 1.5.x 版本+Spring Data JPA时，并使用Dubbot（2.5.6/8），如果在Service里注入Repository，就有机会触发Dependency Cycle error。
在上层代码中，如果autowire service的顺序不同，也会触发这个Denpendency Cycle，如：
@Autowired
private OrganizationService organizationService;
@Autowired
private CityDictionaryService cityDictionaryService;
不会报错，但是调换声明顺序就会。
如果删除Dubbo的Dependency，问题解决。
Dependency Cycle和EntityManager有关，而且很诡异，无论是自定义EntityManager还是Spring自动创建的都会报错。
报错信息：
Description:
The dependencies of some of the beans in the application context form a cycle:
testTravelController defined in file [省略TestTravelController.class]
↓
organizationServiceImpl defined in file [省略e\impl\OrganizationServiceImpl.class]
↓
departmentRepository
↓
(inner bean)#1e495414
┌─────┐
|  orderEntityManagerFactory
↑     ↓
|  com.xxx.jtxt.oa.common.service.UserService
↑     ↓
|  userServiceImpl defined in file [省略\impl\UserServiceImpl.class]
↑     ↓
|  userRepositoryImpl
↑     ↓
|  com.xxx.jtxt.oa.common.service.OrganizationService
└─────┘
另：stackOverflow上也有人遇到了类似问题：
https://stackoverflow.com/questions/41608705/the-dependencies-of-some-of-the-beans-in-the-application-context-form-a-cycle

Have you used Constructor Injection?
Maybe this post can help you http://www.baeldung.com/circular-dependencies-in-spring
Do you use dubbo Annotation in your project? I suspect it's a Spring related problem, may have nothing to do with dubbo, but i am not sure yet. Can you provide a runnable demo so we can figure out the problem?

Yes, the project used the constructor injection for test mock purpose.
But, these classes have no dependency circle in logic!
Please note that when Dubbo dependency removed, all the injection works well without any other changes! There are just normal services class with repository injected.
E.g,
public ServiceAImpl{
@Autowired
public ServiceAImpl(UserRepo A)
....
}
public ServiceBImpl{
@Autowired
public ServiceAImpl(DeptRepo A,CityRepo B)
....
}
then the circle error happens...the circle relies on the entityMananger, it's obviously strange.
Please test the Spring boot + dubbo +Spring data jpa, with the One Service class has two repo injected, then you will know what I mean.

repo: alibaba/dubbo


text:
版本1.2.41配置FastJsonpResponseBodyAdvice后jsonp请求返回数据中文乱码

版本1.2.41配置FastJsonpResponseBodyAdvice后，jsonp请求返回数据中文乱码， 如果不是jsonp请求就不会乱码！
`
@Configuration
public class FastJsonConverterConfig extends WebMvcConfigurerAdapter {
@Override
public void configureMessageConverters(List<HttpMessageConverter<?>> converters) {
//创建fastJson的消息转换器
FastJsonHttpMessageConverter fastJsonHttpMessageConverter = new FastJsonHttpMessageConverter();
//创建配置类
FastJsonConfig fastJsonConfig = new FastJsonConfig();
//修改配置返回内容的过滤
fastJsonConfig.setSerializerFeatures(
SerializerFeature.WriteNullNumberAsZero,
SerializerFeature.WriteMapNullValue,
SerializerFeature.WriteNullListAsEmpty,
SerializerFeature.WriteNullStringAsEmpty,
SerializerFeature.DisableCircularReferenceDetect
);
//配置时间转换
SerializeConfig serializeConfig = new SerializeConfig();
String dateFormat = "yyyy-MM-dd HH:mm:ss";
serializeConfig.put(Timestamp.class, new SimpleDateFormatSerializer(dateFormat));
fastJsonConfig.setSerializeConfig(serializeConfig);
//处理中文乱码问题
List fastMediaTypes = new ArrayList<>();
fastMediaTypes.add(MediaType.APPLICATION_JSON_UTF8);
//消息转换器装载中文乱码配置
fastJsonHttpMessageConverter.setSupportedMediaTypes(fastMediaTypes);
//消息转换器装载fastJson配置
fastJsonHttpMessageConverter.setFastJsonConfig(fastJsonConfig);
converters.add(fastJsonHttpMessageConverter);
//调用父类的配置
super.configureMessageConverters(converters);
}
@Bean
public FastJsonpResponseBodyAdvice fastJsonpResponseBodyAdvice() {
    return new FastJsonpResponseBodyAdvice();
}

}`

问题出在 jsonp时输出的不是 APPLICATION_JSON_UTF8，
而是APPLICATION_JAVASCRIPT = new MediaType("application", "javascript");
试试用 FastJsonpHttpMessageConverter4 吧

问题确实是这个问题， 但是FastJsonpHttpMessageConverter4 已经被标记为Deprecated。 而且也是解决不了问题。 就算是使用：@ResponseJSONP 和 JSONPResponseBodyAdvice 还是不可以解决问题。 从原理上来说，还是需要通过指定 Response 的 Content-Type:application/javascript;charset=utf-8。
低版本倒是没有问题。还是希望官方解决。
APPLICATION_JAVASCRIPT = new MediaType("application", "javascript", Charsets.UTF_8);

@leokongwq 这个问题在于jsonp其实是javascript， 并不是json。 所以contentType采用javascript其实是正确的。

@neil4dong contentType 指定为 application/javascript 没有问题，但只是简单的设定为  application/javascript 就会把 MediaType 的 Charset 信息丢掉，这样会导致乱码问题；希望提供一个给 jsonp 格式设置编码的接口，由使用方来设定 Charset 信息；

@kakakeven 最终响应的Charset不取决于JSONPResponseBodyAdvice 的操作， JSONPResponseBodyAdvice 仅仅只是将返回结果做了封装，  实际响应Charset是取决于FastJsonHttpMessageConverter中的Charset配置。
如果有出现错误的例子，请问方便给一个testcase吗

我也遇到了同样的问题，现在除了改成 JavaScript content-type 外还有什么优雅的解决办法吗

@kakakeven 最终响应的Charset不取决于JSONPResponseBodyAdvice 的操作， JSONPResponseBodyAdvice 仅仅只是将返回结果做了封装， 实际响应Charset是取决于FastJsonHttpMessageConverter中的Charset配置。
如果有出现错误的例子，请问方便给一个testcase吗

返回的ContentType只有application/javascript，没有charset，已经在fastjson上设置了DefalutCharset，和SupportMediaType，也在RequestMapping上设置过producer，都没有用，只能手动在HttpServletResponse上设置Content-type

repo: alibaba/fastjson


text:
反序列SMGPSubmitMessage的问题。

引入坐标：

com.chinamobile.cmos
sms-core
2.1.9

测试代码：
    SMGPSubmitMessage smgpSubmitMessage=new SMGPSubmitMessage();
    smgpSubmitMessage.setSequenceNo(1);
    smgpSubmitMessage.setServiceId("hell");
    smgpSubmitMessage.setMsgContent(“hello”);   //注释掉可以正常
    smgpSubmitMessage.setChargeTermId("123555");
    smgpSubmitMessage.setSrcTermId("10086");
    CmppSubmitResponseMessage submitResponseMessage=new CmppSubmitResponseMessage(1);
    submitResponseMessage.setResult(0);
    submitResponseMessage.setMsgId(new MsgId());

    String smsMsg = JSON.toJSONString(smgpSubmitMessage);
    System.out.println(smsMsg);

    smgpSubmitMessage=JSON.parseObject(smsMsg,SMGPSubmitMessage.class);

    System.out.println(smgpSubmitMessage);

===============
把
smgpSubmitMessage.setMsgContent(“hello”);   //注释掉可以正常运行，加上后报错：
Exception in thread "main" com.alibaba.fastjson.JSONException: syntax error, expect {, actual string, pos 249, fieldName msgContent, fastjson-version 1.2.59

因为msgConten入参需要的是SmsMessage对象，而序列化通过get方法拿到的是个String，所以反序列化就报错了。
解决方法:因为第三方类，无法加注解，我们可以通过使用新特性Mixin来给setMsgContent自定义反序列化器，可以看看这个用例 #2793


text:
@JSONField format support Java8 and Joda time

Issue fix up:
https://github.com/alibaba/fastjson/issues/2305
Test case is:
Jdk8TimeFormatTest
Jdk6TimeFormatTest
Java8InstantDeserializeFromMilliTest

Codecov Report

Merging #2490 into master will decrease coverage by 0.2%.
The diff coverage is 48.09%.

@@             Coverage Diff              @@
##             master    #2490      +/-   ##
============================================
- Coverage     81.55%   81.34%   -0.21%
- Complexity     7033     7055      +22
============================================
  Files           143      150       +7
  Lines         26183    26291     +108
  Branches       6262     6278      +16
============================================
+ Hits          21354    21387      +33
- Misses         3034     3087      +53
- Partials       1795     1817      +22

Impacted Files
Coverage Δ
Complexity Δ

...ava/com/alibaba/fastjson/serializer/JodaCodec.java
46.04% <100%> (-0.77%)
34 <1> (-1)

...ava/com/alibaba/fastjson/serializer/DateCodec.java
89.86% <100%> (-1.64%)
37 <0> (-1)

...m/alibaba/fastjson/serializer/FieldSerializer.java
79.51% <100%> (+0.35%)
62 <18> (ø)
:arrow_down:

.../fastjson/serializer/formatter/EmptyFormatter.java
14.28% <14.28%> (ø)
1 <1> (?)

...ain/java/com/alibaba/fastjson/util/ClassUtils.java
38.46% <38.46%> (ø)
3 <3> (?)

.../fastjson/serializer/formatter/TimeFormatters.java
44.73% <44.73%> (ø)
4 <4> (?)

...a/fastjson/serializer/formatter/DateFormatter.java
46.66% <46.66%> (ø)
5 <5> (?)

...a/fastjson/serializer/formatter/JodaFormatter.java
50% <50%> (ø)
4 <4> (?)

...in/java/com/alibaba/fastjson/util/JavaVersion.java
50% <50%> (ø)
2 <2> (?)

...stjson/serializer/formatter/Jdk8TimeFormatter.java
52.63% <52.63%> (ø)
9 <9> (?)

... and 12 more

Continue to review full report at Codecov.

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

authors: codecov-io rmfish


text:
0-0问一下相册组件怎么样了,目前市面没有一个好的,希望咸鱼各位大大们加快开源啊

0-0问一下相册组件怎么样了,目前市面没有一个好的,希望咸鱼各位大大们加快开源啊

0-0焦急等待

0-0焦急等待

已经距离10月前开源的公众号发表过去1个多月了  能问下现在进度吗

cc @kaisa695275735

已经距离10月前开源的公众号发表过去1个多月了 能问下现在进度吗

由于原来的相册组件依赖于闲鱼自定义的flutter engine patch。使用该组件需要给flutter engine  打对应的patch。
现在我们找到新的方法，能够在保证性能的条件下，摆脱对这个patch的依赖。
预计这个逻辑修改将月底完成，届时开始真正走开源流程。

尽管很想回馈社区更多有意义的开源项目，但是闲鱼技术还是一个很小的Team，我们精力有限，所以很多时候会归结为优先级问题。
但我们在急招人，欢迎 对开源技术，对架构设计，对Flutter，对下一代研发体系感兴趣的同学骚扰。
求 Mail('tino.wjf@alibaba-inc.com', 简历) -， -

cc @kaisa695275735 0-0老哥 怎么样了,不是催0-0 就是想知道下进度

cc @kaisa695275735  0-0老哥 怎么样了,不是催0-0 就是想知道下进度

cc @kaisa695275735  日常疑问

cc @kaisa695275735 每日一问0-0

@kaisa695275735  每日一问0-0

@kaisa695275735 redux这个是不是废弃了 pr都没人合 麻烦给反馈下

authors: kaisa695275735 niugengyun zhangruiyu zjuwjf

repo: alibaba/fish-redux


text:
fix: format of printing errs in e2e test

Ⅰ. Describe what this PR did

Content-Type为form-data类型时，通常会在后面跟一个boudary的数据，在header中实际应为（举例）:"ContentType":    "multipart/form-data; boundary=----WebKitFormBoundaryAnydWsQ1ajKuGoCd"，为此body类型补充了一些检查逻辑
CompareRequest&Response中的一些报错输出存在一些bug，比如对于body数据结构的类型断言失败时，不应打印body内容，因为类型未知。

Ⅱ. Does this pull request fix one issue?

Ⅲ. Why don't you add test cases (unit test/integration test)?
Ⅳ. Describe how to verify it
运行所有e2e测试
Ⅴ. Special notes for reviews

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

Comparison is base (a71ecf4) 38.08% compared to head (f8afea6) 38.08%.

Additional details and impacted files

@@           Coverage Diff           @@
##             main     #777   +/-   ##
=======================================
  Coverage   38.08%   38.08%
=======================================
  Files          61       61
  Lines       10323    10323
=======================================
  Hits         3932     3932
  Misses       6091     6091
  Partials      300      300

duplicate with https://github.com/alibaba/higress/pull/778

authors: Uncle-Justice codecov-commenter

repo: alibaba/higress


text:
ngx.timer  is nil ?

version: tengine-2.0.3
access_by_lua'
        local function push_data(premature, uri, args, status)
                    ngx.log(ngx.ERR, "failed to create timer: ", "11")
                end
                local ok, err = ngx.timer.at(0, push_data,
                                             ngx.var.uri, ngx.var.args, ngx.header.status)
                if not ok then
                    ngx.log(ngx.ERR, "failed to create timer: ", err)
                    return
                end
';

when i access the url,  the error.log is
[string "access_by_lua"]:5: attempt to index field 'timer' (a nil value)

ngx.lua in tengine-2.0.3 is old and does not support ngx.timer api.
So which timer api can we use?

@detailyang
There is no api ngx.timer.at in tengine-2.0.3, which means you cannot create an nginx timer with a user callback function.
And there is only one api ngx.timer.at in ngx.timer lib.

i have try tengine2.1.0, and get the error as follow:
2015/07/31 18:09:42 [error] 4841#0: *3 lua entry thread aborted: runtime error: [string "content_by_lua"]:2: attempt to index field 'timer' (a nil value)
stack traceback:
coroutine 0:
	[string "content_by_lua"]:2: in function <[string "content_by_lua"]:1>, client: 127.0.0.1, server: , request: "GET / HTTP/1.1", host: "127.0.0.1:22222"

my conf is
location / {
    content_by_lua '
        local timer = ngx.timer.at;
        ngx.say("test");
    ';
}

my tengine version is
Tengine version: Tengine/2.1.0 (nginx/1.6.2)
built by gcc 4.4.7 20120313 (Red Hat 4.4.7-11) (GCC)
TLS SNI support enabled
configure arguments: --prefix=/opt/tengine --sbin-path=/opt/tengine/sbin/tengine --conf-path=/opt/tengine/conf/tengine.conf --pid-path=/var/run/tengine.pid --with-jemalloc --http-client-body-temp-path=/dev/shm/client_body_temp --http-proxy-temp-path=/dev/shm/proxy_temp --http-fastcgi-temp-path=/dev/shm/fastcgi_temp --http-uwsgi-temp-path=/dev/shm/uwsgi_temp --http-scgi-temp-path=/dev/shm/scgi_temp --with-file-aio --with-http_addition_module=shared --with-http_xslt_module=shared --with-http_image_filter_module=shared --with-http_geoip_module=shared --with-http_sub_module=shared --with-http_flv_module=shared --with-http_slice_module=shared --with-http_mp4_module=shared --with-http_concat_module=shared --with-http_random_index_module=shared --with-http_secure_link_module=shared --with-http_sysguard_module=shared --with-http_charset_filter_module=shared --with-http_userid_filter_module=shared --with-http_footer_filter_module=shared --with-http_trim_filter_module=shared --with-http_access_module=shared --with-http_autoindex_module=shared --with-http_map_module=shared --with-http_split_clients_module=shared --with-http_referer_module=shared --with-http_rewrite_module=shared --with-http_fastcgi_module=shared --with-http_uwsgi_module=shared --with-http_scgi_module=shared --with-http_memcached_module=shared --with-http_limit_conn_module=shared --with-http_limit_req_module=shared --with-http_empty_gif_module=shared --with-http_browser_module=shared --with-http_user_agent_module=shared --with-http_upstream_ip_hash_module=shared --with-http_upstream_least_conn_module=shared --with-http_upstream_session_sticky_module=shared --with-http_lua_module=shared
loaded modules:
    ngx_core_module (static)
    ngx_errlog_module (static)
    ngx_conf_module (static)
    ngx_dso_module (static)
    ngx_syslog_module (static)
    ngx_events_module (static)
    ngx_event_core_module (static)
    ngx_epoll_module (static)
    ngx_procs_module (static)
    ngx_proc_core_module (static)
    ngx_openssl_module (static)
    ngx_regex_module (static)
    ngx_http_module (static)
    ngx_http_core_module (static)
    ngx_http_log_module (static)
    ngx_http_upstream_module (static)
    ngx_http_static_module (static)
    ngx_http_index_module (static)
    ngx_http_access_module (shared, 3.1)
    ngx_http_auth_basic_module (static)
    ngx_http_geo_module (static)
    ngx_http_split_clients_module (shared, 3.1)
    ngx_http_ssl_module (static)
    ngx_http_fastcgi_module (shared, 3.1)
    ngx_http_empty_gif_module (shared, 3.1)
    ngx_http_upstream_ip_hash_module (shared, 3.1)
    ngx_http_proxy_module (static)
    ngx_http_upstream_consistent_hash_module (static)
    ngx_http_upstream_check_module (static)
    ngx_http_reqstat_module (static)
    ngx_http_rewrite_module (shared, 3.1)
    ngx_http_upstream_keepalive_module (static)
    ngx_http_upstream_dynamic_module (static)
    ngx_http_stub_status_module (static)
    ngx_http_write_filter_module (static)
    ngx_http_header_filter_module (static)
    ngx_http_chunked_filter_module (static)
    ngx_http_range_header_filter_module (static)
    ngx_http_gzip_filter_module (static)
    ngx_http_postpone_filter_module (static)
    ngx_http_charset_filter_module (shared, 3.1)
    ngx_http_upstream_session_sticky_module (shared, 3.1)
    ngx_http_ssi_filter_module (static)
    ngx_http_headers_filter_module (static)
    ngx_http_lua_module (shared, 3.1)
    ngx_http_copy_filter_module (static)
    ngx_http_range_body_filter_module (static)
    ngx_http_not_modified_filter_module (static)

@detailyang
Sorry for that I misstated the tengine version before.
Tengine-2.1.0 does not include commit of ngx.lua updating.
You can try the latest tengine git master, which has included ngx.lua-0.9.12.

authors: binlaniua chobits detailyang

repo: alibaba/tengine


text:
2.2.0版本已经基于nginx1.8.1了，但不支持--with-threads选项，这个选项在nginx1.7.11就已经支持了

我们的系统主要是提供大文件下载的，nginx对文件下载可以优化
location /video/ {
    sendfile       on;
    aio            threads;
}

使用aio threads，需要添加--with-threads配置
# ./configure --without-dso --without-http_autoindex_module --without-http_fastcgi_module --without-http_uwsgi_module --without-http_scgi_module --without-http_userid_module --without-http_footer_filter_module --without-http_upstream_dynamic_module --with-file-aio --with-http_sub_module --add-module=../echo-nginx-module-0.60/ --with-threads
执行后提示
./configure: error: invalid option "--with-threads"
linux内核版本为2.6.32-279.el6.x86_64

目前proc模块和--with-threads功能还需要整合，所以暂时没有开放这个option

hi, --with-threads option已经在master最新分支中打开，可以使用了

感谢 @chobits , 好快啊

你先试用下，如果有问题可以反馈。这个功能只同步到官方1.8.1版本，如果后续还有更新，会继续merge到tengine master

压测暂时没发现问题，后续有问题再反馈，谢谢。

authors: WPG chobits


text:
[Feature]: Proofreading RFC8999 zh | 校对 RFC8999 中文译稿

Before request a new feature

[X] I have searched the issues of this repository and believe that this is not a duplicate.

Describe the feature you'd like supported
当前XQUIC项目组开源的译稿 draft-ietf-quic-invariants-13-zh.md 是基于 ietf draft-13版本，与 RFC8999 内容基本一致。
在此基础上进行校对修正，提供正式的 RFC8999 中文译本。
Describe alternatives you've considered
No response
Additional context
No response

•	Packets containing frames besides ACK or CONNECTION_CLOSE frames count toward congestion control limits and are considered to be in flight
•	包含ACK和CONNECTION_CLOSE以外帧的包会被计入到拥塞控制限制，并且被认为是in-flight数据。
这个应该是除了ack和connection_close的帧 而不是包含

authors: Lynne-v RuiqiZhou

repo: alibaba/xquic


text:
Using Tags to identify all the resources

When I did the deployment into production environment, the EC2 instances created with production-a-xdfdsfdfs names and so are the other AWS resources with the similar names.
Is there a way to pass tags attributes to the EB deployment environment??

Honestly I've never heard of tags.  If you can point me to one of the AWS docs, I can tell you if it integrates.

Refer to:
https://github.com/ThoughtWorksStudios/eb_deployer/search?utf8=✓&q=tags
      def create_environment(app_name, env_name, stack_name, cname_prefix, version, tier, tags, settings)
        request = reject_nil({
          :application_name => app_name,
          :tier => environment_tier(tier),
          :cname_prefix => name_prefix,

**          :tags => tags**
        })
        @client.create_environment(request)

Refer to:
https://github.com/ThoughtWorksStudios/eb_deployer/blob/master/lib/eb_deployer.rb
    environment = Environment.new(application, env_name, stack_name, bs) do |env|
      env.resource_stacks = resource_stacks
      env.settings = opts[:option_settings] || opts[:settings] || []
      env.inactive_settings = opts[:inactive_settings] || []
      env.creation_opts = {
        :solution_stack => opts[:solution_stack_name],
        :cname_prefix =>  opts[:cname_prefix],
        :smoke_test => opts[:smoke_test],
        :phoenix_mode => opts[:phoenix_mode],
        :blue_green_terminate_inactive => opts[:blue_green_terminate_inactive] || false,
        :blue_green_terminate_inactive_wait => opts[:blue_green_terminate_inactive_wait] || 600,
        :blue_green_terminate_inactive_sleep => opts[:blue_green_terminate_inactive_sleep] || 15,
**        :tags => opts[:tags],**  # this gem is not passing the tags
        :tier => opts[:tier]
      }

See here:
https://github.com/alienfast/elastic-beanstalk/blob/master/lib/elastic/beanstalk/tasks/eb.rake#L328
options = {
        application: EbConfig.app,
        environment: EbConfig.environment,
        version_label: find_option_app_version,
        solution_stack_name: EbConfig.solution_stack_name,
        option_settings: EbConfig.option_settings,
        inactive_settings: EbConfig.inactive_settings,
        strategy: EbConfig.strategy.to_sym,
        package: package
    }

    options[:package_bucket] = EbConfig.package_bucket unless EbConfig.package_bucket.nil?
    options[:keep_latest] = EbConfig.keep_latest unless EbConfig.keep_latest.nil?
    options[:version_prefix] = EbConfig.version_prefix unless EbConfig.version_prefix.nil?
    options[:tier] = EbConfig.tier unless EbConfig.tier.nil?

If we can do the following line added above:
options[:tags] = EbConfig.tags unless EbConfig.tags.nil?
supports:
package_bucket: 'app-name-124-deploy'
**tags:
  Name: app-name124**

This will help name all the EC2 instances and other resources with name starting from app-name124-a-sdfsdfsd.
That will help identify all the resources ... especially useful when we have multiple apps / environments running under the same AWS login.
Hope the above is clear!

Feel free to send a PR with any of the additional options for eb_deployer, there isn't a reason I can see that we shouldn't support passing through those options.  We've just been adding them as needed.

Closed with #49, thanks @canatella

authors: dashbitla rosskevin

repo: alienfast/elastic-beanstalk


text:
Portfolio site changes

added a project.html page and in this I added information about my past projects.

just added the changes!


text:
Undo/Redo functions are removing 2 objects at same time

Hello I'm using fabric js latest version and creating an editor tool. we create shapes by using mouse down and modify them when mouse is move . when I use undo/redo function it remove 2 objects at same time .
But when I add the shape first by clicking on the button with default width and height and top and left the undo redo function works fine.
i've tried to debug it on mouse down the object is added first with width and height 0 and on mouse move the object is modified with calculations of mouse movement thus 2 objects are maintained in the history instead of one how can i fix that ?
https://jsfiddle.net/durga598/7amunr73/2/
here is my code use it with history js and try to create multiple shapes and then use undo redo it will create the bug

https://github.com/alimozdemir/fabric-history/issues/36 Think this will be helpful.

facing same issue. only last 2 objects are getting deleted.

repo: alimozdemir/fabric-history


text:
Comparing data between two fields rather than using hardcoded values

Discussed in https://github.com/alirezanet/Gridify/discussions/153

Originally posted by Robotxm February 21, 2024
Hello, I have following object definitions:
public record Item(string Name, List<TimeSchedule> Schedules);

public record TimeSchedule(int Start, int End);

Now I want to find all Items who has a schedule that its End time is before Start time.
With native LINQ I can do this:
List<Item> items =
[
    new Item("Item1", [new TimeSchedule(1, 2), new TimeSchedule(2, 3)]),
    new Item("Item2", [new TimeSchedule(1, 4), new TimeSchedule(4, 3)]),
    new Item("Item3", [new TimeSchedule(3, 2), new TimeSchedule(2, 3)]),
];

var results = items.Where(x => x.Schedules.Any(s => s.End < s.Start));

So how can I achieve this using Gridify?

Hi, have you been able work on this or do you know if it is possible? This is something that I would also be interested in and I also tried to make it work with custom maps and operator but it does not seem possible at the moment.

Hi @k94ll13nn3,
Right now, it is not possible, but this is a feature I wanted to implement for v2.15. However, due to my busy schedule and other issues, it has been postponed. So far, we only have the test case :) but in the future, I'll definitely add support for this feature. Keep in mind that contributions are always welcome 😉

repo: alirezanet/Gridify


text:
[EMR-SDK] add Datahub writer

        ## What changes were proposed in this pull request?

        Spark can write data to Datahub sink.

        Here are some options:
        * "batch.size":
          To improve performance, datahub writer will try to pack records in a pack,
          and send them to Datahub in a requset. Note that, "1" is the default value. It means that
          the write will send record to datahub one by one, by default.

        * "retry": How many times the client will try after put records
           to Datahub failed.

        ```scala
         df.write
          .format("datahub")
          .option("access.key.id", "yourAccessKeyId")
          .option(, "yourAccessKeySecret")
          .option(DatahubSourceProvider.OPTION_KEY_ENDPOINT, "datahubEndpoint")
          .option(DatahubSourceProvider.OPTION_KEY_PROJECT, "yourProject")
          .option(DatahubSourceProvider.OPTION_KEY_TOPIC, "yourTopic")
          .option(DatahubSourceProvider.OPTION_KEY_BATCH_SIZE, "100")
          .option(DatahubSourceProvider.OPTION_KEY_RETRIES, "3")
          .mode("append")
          .save()

        ```
        ## How was this patch tested?
        N/A

What changes were proposed in this pull request?
(Please fill in changes proposed in this fix)
How was this patch tested?
(Please explain how this patch was tested. E.g. unit tests, integration tests, manual tests)
(Refer to this test guide)

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

authors: claassistantio kexianda

repo: aliyun/aliyun-emapreduce-sdk


text:
add fun init

Add fun init [options] [location]  command:
Usage: init [options] [location]
Initializes a new fun project
Options:
-o, --output-dir [outputDir]  where to output the initialized app into (default: .)
-n, --name [name]             name of your project to be generated as a folder (default: fun-app)
-i, --no-input [noInput]      disable prompting and accept default values defined template config
-V, --var [vars]              template variable
-h, --help                    output usage information

Examples:
$ fun init
$ fun init helloworld-nodejs8
$ fun init foo/bar
$ fun init gh:foo/bar
$ fun init gl:foo/bar
$ fun init bb:foo/bar
$ fun init git+ssh://git@github.com/foo/bar.git
$ fun init hg+ssh://hg@bitbucket.org/bar/foo
$ fun init git@github.com/foo/bar.git
$ fun init hg@bitbucket.org/bar/foo
$ fun init https://github.com/foo/bar.git
$ fun init /path/foo/bar
$ fun init -n fun-app -V foo=bar /path/foo/bar

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.kevin seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.

一般来说， cmd [opt] [arg] ，opt 放在 arg 前面会不会更合理些？

已经改成 opt 在 arg 的前面了

authors: CLAassistant muxiangqiu

repo: aliyun/fun


text:
Pretraining a PyTorch model of BERT

We want to pretrain BERT on a costume dataset and then use this trained model with the pytorch_pretrained_bert package.
We saw that your repository contains a PyTorch model of pretrained sci-BERT, and wondered which tools did you use to create this?

You can convert a BERT model (trained with the official TensorFlow implementation) with the pytorch-pretrained-BERT library, that can be found here.
Here's a good example of how to use convert a trained model into a PyTorch model:
https://github.com/huggingface/pytorch-pretrained-BERT#bert-1 :)

Thank you for the fast and informative response! :)

@stefan-it  Thanks for your help answering these questions!

authors: kyleclo nadavborenstein stefan-it

repo: allenai/scibert


text:
Cannot recover value when using the Beaker Executor

🐛 Describe the bug
I created a simple example that has a single step that returns 5.  When I run it without a tango.yml the following code recovers the number 5:
>>> from tango import Workspace
>>> workspace = Workspace.from_url("local:///Users/michaels/hack/allenai/tango-test/workspace")
>>> workspace.step_result_for_run("fancy-dingo", "main")
5

However, when I use the following tango.yml I get an exception instead.
executor:
  type: beaker
  beaker_workspace: ai2/michaels-default
  clusters:
    - ai2/general-cirrascale
  docker_image: ghcr.io/allenai/pytorch:1.12.0-cuda11.3-python3.9
  venv_name: base

>>> from tango import Workspace
>>> workspace = Workspace.from_url("local:///Users/michaels/hack/allenai/tango-test/workspace")
>>> workspace.step_result_for_run("calm-squid", "main")
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/Users/michaels/hack/allenai/tango/tango/workspace.py", line 240, in step_result_for_run
    return self.step_cache[step_info]
  File "/Users/michaels/hack/allenai/tango/tango/step_caches/local_step_cache.py", line 115, in __getitem__
    raise KeyError(step)
KeyError: StepInfo(unique_id='Main-4jtQuipCnThKTpamAPe4JAhrViYmmWHK', step_class_name='Main', dependencies=set(), cacheable=True, step_name='main', version=None, start_time=None, end_time=None, error=None, result_location=None, config={'type': 'main'}, platform=PlatformMetadata(operating_system='macOS-10.16-x86_64-i386-64bit', cpu_count=16, user='michaels', host='ip-172-16-21-35.us-west-2.compute.internal'), environment=EnvironmentMetadata(python='3.9.12', executable=PosixPath('/Users/michaels/miniconda3/envs/tango/bin/python'), command='/Users/michaels/miniconda3/envs/tango/bin/tango run steps.jsonnet -i main -w workspace', root=PosixPath('/Users/michaels/hack/allenai/tango-test'), packages=[('absl-py', '1.2.0'), ('ai2-tango', '0.11.0'), ('aiohttp', '3.8.1'), ('aiosignal', '1.2.0'), ('astunparse', '1.6.3'), ('async-timeout', '4.0.2'), ('attrs', '22.1.0'), ('base58', '2.1.1'), ('beaker-py', '1.6.9'), ('boto3', '1.24.48'), ('botocore', '1.27.48'), ('cached-path', '1.1.5'), ('cachetools', '5.2.0'), ('certifi', '2022.6.15'), ('charset-normalizer', '2.1.0'), ('chex', '0.1.4'), ('click', '8.1.3'), ('click-help-colors', '0.9.1'), ('commonmark', '0.9.1'), ('cycler', '0.11.0'), ('datasets', '2.4.0'), ('dill', '0.3.5.1'), ('dm-tree', '0.1.7'), ('docker', '5.0.3'), ('docker-pycreds', '0.4.0'), ('etils', '0.7.1'), ('fairscale', '0.4.6'), ('filelock', '3.7.1'), ('flatbuffers', '1.12'), ('flax', '0.5.0'), ('fonttools', '4.34.4'), ('frozenlist', '1.3.1'), ('fsspec', '2022.7.1'), ('gast', '0.4.0'), ('gitdb', '4.0.9'), ('gitpython', '3.1.27'), ('glob2', '0.7'), ('google-api-core', '2.8.2'), ('google-auth', '2.10.0'), ('google-auth-oauthlib', '0.4.6'), ('google-cloud-core', '2.3.2'), ('google-cloud-storage', '2.5.0'), ('google-crc32c', '1.3.0'), ('google-pasta', '0.2.0'), ('google-resumable-media', '2.3.3'), ('googleapis-common-protos', '1.56.4'), ('grpcio', '1.47.0'), ('h5py', '3.7.0'), ('huggingface-hub', '0.8.1'), ('idna', '3.3'), ('importlib-metadata', '4.12.0'), ('importlib-resources', '5.9.0'), ('jax', '0.3.15'), ('jaxlib', '0.3.15'), ('jmespath', '1.0.1'), ('joblib', '1.1.0'), ('jsonnet', '0.18.0'), ('jsonpickle', '2.1.0'), ('keras', '2.9.0'), ('keras-preprocessing', '1.1.2'), ('kiwisolver', '1.4.4'), ('libclang', '14.0.6'), ('markdown', '3.4.1'), ('markupsafe', '2.1.1'), ('matplotlib', '3.5.2'), ('more-itertools', '8.14.0'), ('msgpack', '1.0.4'), ('multidict', '6.0.2'), ('multiprocess', '0.70.13'), ('numpy', '1.23.1'), ('oauthlib', '3.2.0'), ('opt-einsum', '3.3.0'), ('optax', '0.1.3'), ('packaging', '21.3'), ('pandas', '1.4.3'), ('pathtools', '0.1.2'), ('petname', '2.6'), ('pillow', '9.2.0'), ('pip', '22.1.2'), ('promise', '2.3'), ('protobuf', '3.19.4'), ('psutil', '5.9.1'), ('pyarrow', '9.0.0'), ('pyasn1', '0.4.8'), ('pyasn1-modules', '0.2.8'), ('pydantic', '1.9.1'), ('pydeprecate', '0.3.2'), ('pygments', '2.12.0'), ('pyparsing', '3.0.9'), ('python-dateutil', '2.8.2'), ('pytorch-lightning', '1.6.5'), ('pytz', '2022.1'), ('pyyaml', '6.0'), ('regex', '2022.7.25'), ('requests', '2.28.1'), ('requests-oauthlib', '1.3.1'), ('responses', '0.18.0'), ('rich', '12.5.1'), ('rsa', '4.9'), ('s3transfer', '0.6.0'), ('sacremoses', '0.0.53'), ('scipy', '1.9.0'), ('sentencepiece', '0.1.97'), ('sentry-sdk', '1.9.3'), ('setproctitle', '1.3.1'), ('setuptools', '61.2.0'), ('shortuuid', '1.0.9'), ('six', '1.16.0'), ('smmap', '5.0.0'), ('sqlitedict', '2.0.0'), ('tblib', '1.7.0'), ('tensorboard', '2.9.1'), ('tensorboard-data-server', '0.6.1'), ('tensorboard-plugin-wit', '1.8.1'), ('tensorflow-cpu', '2.9.1'), ('tensorflow-estimator', '2.9.0'), ('tensorflow-io-gcs-filesystem', '0.26.0'), ('termcolor', '1.1.0'), ('tokenizers', '0.12.1'), ('toolz', '0.12.0'), ('torch', '1.12.1'), ('torchmetrics', '0.9.3'), ('tqdm', '4.64.0'), ('transformers', '4.21.1'), ('typing-extensions', '4.3.0'), ('urllib3', '1.26.11'), ('wandb', '0.12.21'), ('websocket-client', '1.3.3'), ('werkzeug', '2.2.2'), ('wheel', '0.37.1'), ('wrapt', '1.14.1'), ('xxhash', '3.0.0'), ('yarl', '1.8.1'), ('zipp', '3.8.1')], git=GitMetadata(commit='0c5e94c5abe7f082be43e479b769855eecd88212', remote='git@github.com:schmmd/tango-test.git'), tango=TangoMetadata(version='0.11.0')))

I also noticed that if I rerun the steps with the BeakerExecutor is will run each time, but if i run it without and then with, it will simply use the cached result.
Versions
latest

I think the root cause here was that I was using a local workspace but the Beaker executor.

It might be helpful to have a better exception like "Could not find any results for step 'main'"

repo: allenai/tango


text:
how to disable POI

Hi Allen Kim,
Thank you for your Directive.
I need to disable POI. How can I do it?
`
var myStyles =[
{
featureType: "poi",
elementType: "labels",
stylers: [
{ visibility: "off" }
]
}
];
var myOptions = {
zoom: 10,
center: homeLatlng,
mapTypeId: google.maps.MapTypeId.ROADMAP,
styles: myStyles
};
`

sorry I forgot about "styles"
 <ng-map styles="{{ myStyles }} "></ng-map>

repo: allenhwkim/angularjs-google-maps


text:
Any plans to support OpenLayers 4?

Was looking to use react-openlayers but my coworkers (who have much more OpenLayers experience than I) were encouraging me to use OL4...
(they said OL4 had some features to better limit the resulting webpack javascript bundle size?)

It's already OL4
https://github.com/allenhwkim/react-openlayers/blob/master/package.json#L47

Oh awesome!!
(just fyi/fwiw I'm new to OpenLayers so maybe I read it differently than others would have but it was the description "A minimal React wrapper of OpenLayer 3+ written in TypeScript" that I misread as "OpenLayers 3" I didn't even notice the "+" part until your message)

authors: allenhwkim darrencruse

repo: allenhwkim/react-openlayers


text:
Use MSVC_RUNTIME_LIBRARY instead of setting compile flags manually

https://cmake.org/cmake/help/latest/prop_tgt/MSVC_RUNTIME_LIBRARY.html
Clang's GNU driver wouldn't recognize /MT flag when cross-compiling for Windows, let CMake handle compile flags internally instead

@illnyang this is not working as intended on Windows / MSVC . Please verify and try again. The produced Windows plugin should be statically linked in Release build and should have no CRT dependencies.
cmake --build . --config Release
Thanks!

I didn't notice that the minimum required version throughout ida-cmake is 3.12, it needs to be bumped to 3.15 for this property to take effect. I think it's reasonable to do so, that version was released back in 2017 and CMAKE_CXX_STANDARD  is set to 17 in all sample addons as well.

sorry, I did not notice you updated the PR. I read quickly and saw you asked for a bump.
Indeed, I tought I did that before. Ok, I pushed to 3.27 ; fair and from last year.

please revert the last change. it is a different topic anyway.

Clang's GNU driver wouldn't recognize /MT flag when cross-compiling for Windows, let CMake handle compile flags internally instead

Can you please tell me how to cross-compile and verify (full steps)?
I can test, like I did before, on Windows. So give me some time to make sure on Windows, after bumping the CMake version, after your changes, we still get static linking.

I use the following CMAKE_TOOLCHAIN_FILE:
find_program(LLD_LINK_PATH NAMES lld-link)
if (${LLD_LINK_PATH} STREQUAL "LLD_LINK_PATH-NOTFOUND")
    message(SEND_ERROR "Unable to find lld-link")
endif ()

find_program(CLANG_C_PATH NAMES clang)
if (${CLANG_C_PATH} STREQUAL "CLANG_C_PATH-NOTFOUND")
    message(SEND_ERROR "Unable to find clang")
endif ()

find_program(CLANG_CXX_PATH NAMES clang++)
if (${CLANG_CXX_PATH} STREQUAL "CLANG_CXX_PATH-NOTFOUND")
    message(SEND_ERROR "Unable to find clang++")
endif ()

set(CMAKE_LINKER "${LLVM_LIB_PATH}" CACHE FILEPATH "")
set(CMAKE_LINKER_TYPE LLD CACHE STRING "")
set(CMAKE_C_COMPILER "${CLANG_C_PATH}" CACHE FILEPATH "")
set(CMAKE_CXX_COMPILER "${CLANG_CXX_PATH}" CACHE FILEPATH "")

I'm not sure whether find_program will find your LLVM installation on Windows, but you can set paths manually if it doesn't.
I verified that it works by opening built DLL in pe-bear:

repo: allthingsida/ida-cmake


text:
Add more digits for timeline

It haven't done before because that looked not good on our example

Now I added workaround for it.
Fixes #815

@just-boris thanks!

authors: baev just-boris

repo: allure-framework/allure-core


text:
[GH-21] Link

Add <Link /> component

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

repo: allyssonsantos/frigobar


text:
Installation instructions for macOS

If you would like to add installation instructions for Mac, they are the same as for linux, but the PATH variables are:
export PATH=path/to/bin:$PATH
export DYLD_LIBRARY_PATH=path/to/lib:$PATH

Yeah, so this is the wrong repo. Sorry. It was meant for elastix itself, not the wrapper.

repo: almarklein/pyelastix


text:
make theme state persistent

as of now, if you toggle theme to dark and reload, theme state is lost and default to light mode. please, make theme state stored in localstorage(maybe) and make theme state persistent.

above PR resolved this issue.


text:
First example does not work

When I ran your first very simple example I got an error
pql.find("a > 1 and b == 'foo' or not c.d == False")
ParseError: Unsupported syntax (NameConstant). options: ['Call', 'Dict', 'List', 'Name', 'Num', 'Str', 'operator_and_right']

I think you need to replace False with null.
pql.find("a > 1 and b == 'foo' or not c.d == null")
Out[24]: {'$or': [{'a': 1}, {'$and': [{'b': '2'}, {'c.d': None}]}]}

I used Python 3.5.
Cheers
H

It seems the library doesn't support python 3. I'll fix it soon.

+1

+1

Not work now :(

you can install pqlpython3

authors: alonho hdzierz maimike sp98 vishalmelmatti

repo: alonho/pql


text:
Update sbt to 1.10.3

About this PR
📦 Updates org.scala-sbt:sbt from 1.9.9 to 1.10.3
📜 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:n:2

Superseded by #592.


text:
Add notice about supported Preact versions in README

#677 (not to be merged) provides a branch with commits for:

manual and automated testing against a branch loading Preact 10 in the example (first commit)
manual and automated testing building the component with Preact 10 (second commit)

This commits adds a notice to the README to clarify which versions of Preact were tested, in a similar fashion to the section for React.
Closes #667

Converting back to draft while I re-check the results.

repo: alphagov/accessible-autocomplete


text:
Remove the organisation data from the details hash

This needs to be removed as the frontend now picks up organisation
from the links hash, not the details hash (https://github.com/alphagov/contacts-frontend/pull/41). This was only kept in here
so that we didn't have to faff around changing too much stuff and
potentially confusing things at deploy-time for #219. This tidies up.

Note: We'll need to run the Republish Contacts Rake task on deploy, so that details no longer have organisations in them.
Trello: https://trello.com/c/HEB3ukuP/263-remove-organisations-from-details-hash-for-contacts, related to https://trello.com/c/lIuKYwFE/235-use-the-full-organisation-name-instead-of-the-abbrevation-to-simplify-the-schemas-and-admin-tool.

Other than the "huh, turns out" comment on the ContactOrganisationPresenter this looks good.  Merging.

repo: alphagov/contacts-admin


text:
[WIP] Select2 tagging UI

Not exactly a progressive enhancement, more a JS dependant frontend.
https://trello.com/c/v256NfML

Superseded by https://github.com/alphagov/content-performance-manager/pull/224.

authors: tijmenb whoojemaflip

repo: alphagov/content-performance-manager


text:
Remove deprecated routes

All frontend apps are now up to date with the latest utils, so these deprecated routes can be removed because they are no longer used by anything.

Has this been tested against the functional tests?

Barring one failure because of not having a valid email key to send an invitation email, yes they have now.

:+1:

authors: TheDoubleK robyoung

repo: alphagov/digitalmarketplace-api


text:
Remove unused set-search-alias script

We now use the update-index-alias.py script to update search index aliases.

@risicle I can't see it being used anyway in Jenkins

repo: alphagov/digitalmarketplace-scripts


text:
Make service link relative

On the dashboard, make the services link relative so that it points to
the correct environment.
Fixes: 94162812

This isn't a fix for the linked card because that is about the admin app. And is itself a duplicate of 94069990 which is currently blocked.
That said, this is a worthwhile change so imma merge it.

authors: quis robyoung


text:
Feature/add simplecov

Hey @pixeltrix,
This is not solving an issue but more of a nice to have.

I added SimpleCov as a development dependency. It's useful to find what parts of the app are least covered (ie. libraries)
Please check it out and let me know if you think it's useful.
Best,
Ernesto

Happy to merge as long as it's a opt-in and not on by default (which it appears to be 👍 )

@pixeltrix Cool, just updated the changes based on your feedback re: .gitignore

@etagwerker thanks! 👍

authors: etagwerker pixeltrix

repo: alphagov/e-petitions


text:
Migrate SpecialRoutePublisher to use v2 of publishing-api

All clients of publishing-api should use v2 of the API.  This library contains one class, SpecialRoutePublisher which talks to the publishing-api (rather than just provide convenience adapter methods to do so).  This change modifies that class to use v2 of the API.
Trello: https://trello.com/c/yaUJdIRH/456-migrate-gds-api-adapters-gdsapi-publishingapi-specialroutepublisher-to-use-v2-of-publishing-api

I've tried running this:
vagrant@development:~/govuk/rummager (master)$ be rake publishing_api:publish_special_routes
Publishing exact route /sitemap.xml, routing to rummager
rake aborted!
undefined method `put_content' for #<GdsApi::PublishingApi:0x007f4583beaa08>

This is because SpecialRoutePublisher still uses the PublishingApi V1 class. Because we've stubbed the put_content and publish calls, the tests pass while they shouldn't.
I would suggest getting rid of the stubs first and test with the assert_X test helpers, so that the tests fail (like https://github.com/alphagov/gds-api-adapters/pull/404#discussion_r47073218).

Have changed in line with Tijmen's comments

Any objection to me picking this up and getting it across the line? We want to update the feedback app to use V2 for its special route.

boffbowsh - yep, please do. I think I've done all that I was going to do anyway - would be interested to see if you want to make any changes.

authors: boffbowsh stephen-richards tijmenb

repo: alphagov/gds-api-adapters


text:
Change Shelter links

https://trello.com/c/zH5i1W3G/263-change-shelter-link-to-not-be-an-anchor-link

Getting Hannah to raise this in the main repo, not from a fork.

repo: alphagov/govuk-coronavirus-find-support


text:
Tests are reliant on the review app

Cause
Frontend Puppeteer tests rely on the review app in order to run correctly. The tests are tightly coupled to the review app's structure and the examples we use in the review app.
Consequences
This causes us a few different issues. It overlaps with the issues described in https://github.com/alphagov/govuk-frontend/issues/2242 which is an issue we experience now.
In the future, this may trip us up more when we want to add tests for configurable components. At the moment, components are already initialised by the time the tests are run. This wouldn't work for components that we want to configure and test.
It also generally doesn't feel ideal to have the tests and the review app so closely coupled. Ideally, we would have more control over the environment the tests run in without having to rely on there being an existing review app page that does what you want. This could be something like having an empty page which we can then add HTML into within the test suite.
When working on this, we need to consider the impact to debugging. Having tests associated with the review app does make it easier for us to debug if a test fails because we can go straight to that page to figure out what's going on. We might lose this if we decouple the tests and review app.
Note: I've split this tech debt card out separately from https://github.com/alphagov/govuk-frontend/issues/2242 because it may be possible to address #2242 without picking this wider piece of work up if we need a short term solution. However, it's possible we may be forced to address this as part of the JavaScript work if we're introducing configurable components.
Impact of debt
Medium
Reason (impact of debt)
No response
Effort to pay down
Medium
Reason (effort to pay down)
No response
Overall rating
Medium
Reason (overall rating)
No response

A couple of thoughts on the topic, so I don't forget them:

wondering how much of the tests running through Puppeteer to run against a little static server rendering a blank boilerplate page (or populating about:blank or a blank page using a data-URL) and using our renderAndInitialise() helper to render examples. That would allow us to not be coupled with the review app anymore.
wondering if the app used for testing (that light one or the review app) could be made to run against the sources rather than our built package during development. Similarly to how you'd run an app for development locally and for production when deployed, we could have quick local tests that skip building and more secure testing on CI that run against the built package.

We used the following PRs to break the govuk-frontend package away from the Review app:

https://github.com/alphagov/govuk-frontend/pull/4329
https://github.com/alphagov/govuk-frontend/pull/4345
https://github.com/alphagov/govuk-frontend/pull/4346
https://github.com/alphagov/govuk-frontend/pull/4359

Now only the Other examples  tests (e.g. Conditional reveals, Translated) need the Review app running

repo: alphagov/govuk-frontend


text:
Removing cronjob after it was replaced in a previous PR

I renamed this cron job in another PR which has been merged.  Although there are caused by having two of the same cronjob. It's best to remove.

MWG

repo: alphagov/govuk-puppet


text:
Tidy up removal of DFID transition app.

Setting ENV vars doesn't work if you've removed
the app because the directory isn't there anymore.
Also removes the procfile worker properly.

merge when green :+1:


text:
Update rubocop-govuk to 4.0.0.pre.1

Trello: https://trello.com/c/pqXbKZiC/215-test-rubocop-govuk-400pre1-prior-to-400-release
This bumps to the 4.0.0 pre release of rubocop-govuk which is part of trialling the new rubocop-govuk release. It seems to have gone into this app without too many problems which is good news.
All changes here are to resolve linter issues, the pin to 4.0.0.pre.1 is intended to be removed once the final release of rubocop-govuk 4.0.0 is out.
This ruby-version for this repo is changed to be the minimum this gem supports (2.6) to reflect both best practice for Ruby gem development and a cop that expects the minimum ruby version to be set and linted against.

LGTM - though I wonder what downsides the last commit has 🤔 we've explained that we usually set a more modern version than the minimum, but not why that's the case.

I think the only reason we set a more modern version in the .ruby-version is because it gets caught up in initiatives to upgrade all GOV.UK versions of Ruby (I think we even have a script to apply it). Aside from that scenario it's a relatively unusual approach for general work as it provides a lot more risks than working with the minimum version does.

Would it be better to set the minimum version in the Gemfile with ruby "~> 2.6.6", and leave the .ruby-version as the recommended 'developer' version of Ruby?

If you have a required_ruby_version and .ruby-version mismatch, rubocop will complain:

Gemspec/RequiredRubyVersion: required_ruby_version (2.6, declared in govuk_publishing_components.gemspec) and TargetRubyVersion (2.7, which may be specified in .rubocop.yml) should be equal.

which seems reasonable given you want the linting rules applied based on the version of Ruby you support. You can, if needed, tell rubocop which Ruby version to lint for in .rubocop.yml but this seems just another thing to maintain and I'm not convinced there is a good argument to do so.
I'm assuming your suggested constraint was meant to be ~> 2.6, as ~> 2.6.6 would actually only allow you to install >= 2.6.6, < 2.7. ~> 2.6 is basically just a pessimistic version of what we have now as it means >= 2.6, < 3 and it seems unnecessary to be pessimistic.
I'd imagine most of the arguments that we want a newer minimum Ruby version for gems should be justified by the gem dropping support for that Ruby version.

Thanks @kevindew , that all sounds good to me 👍

authors: ChrisBAshton kevindew


text:
Adding html field to response from the template preview

What problem does the pull request solve?
Adds the html field to the TemplatePreview object
Checklist

[x] I’ve used the pull request template
[x] I’ve written unit tests for these changes
[x] I’ve update the documentation (in DOCUMENTATION.md)
[x] I’ve bumped the version number (in src/main/resources/application.properties)
and run the update_version.sh script

Can one of the admins verify this patch?

Hi @mpmiller thanks for the contribution. I’ll get someone with a bit more Java knowledge than me to look at it shortly…

@govuk-notify-jenkins test this please

authors: govuk-notify-jenkins mpmiller quis

repo: alphagov/notifications-java-client


text:
[#149929104] Add uaa_lock_user.rb script

What
This will replace the rather cumbersome process for locking (or unlocking)
users that's currently described in the team manual:

https://government-paas-team-manual.readthedocs.io/en/latest/guides/CF_UAA_user_management/#locking-a-user-account

I've mocked just enough of the UAA responses to make it work and verify that
we're sending PATCH requests when the state needs to change.
Per the original script, I've left TARGET and TOKEN as environments, to call
out the fact that they are mandatory and to help avoid you storing tokens in
your shell history.
How to review

Code review
See if the tests make sense and pass
Try it in a dev environment:

Create an account using scripts/create-user.sh
Verify that you can login
Lock the account and verify that you can no longer login
Unlock the account and verify that you can login again

Who can review
Anyone.

Ready to merge but managed software centre failed to upgrade xcode, and now my dev tools are broken so I cant merge the PR

authors: dcarley timmow

repo: alphagov/paas-cf


text:
[#159620811] rds-broker to 0.1.42 (v0.20.0 of paas-rds-broker)

What
This is to fix the bug where RDS instances would have some flags
overwritten during operations like master password resets.
How to review
See:

the bosh-release build
the rds-broker build
the PR

Who can review
Anyone

We've run this through @blairboy362's pipeline without error, and he's checked that resetting the master password on the instance no longer resets the other properties. Should be good to go.


text:
[#122183539] Visibility of important platform metrics

Who
#122183539 - Visibility of important platform metrics
We want to be able to deploy Grafana with dashboard configuration files (JSON files) supplied via the manifest. This pull request configures Grafana to use Graphite as a datasource, updates the pipeline to generate YAML containing the minified JSON, and adds the dashboard JSON file.
How to review
Note: review after pull request 58 on paas-docker-cloudfoundry-tools has been merged and this FIXME commit on this repository has been removed.

Complete the review in pull request 1 on paas-grafana-boshrelease. This must be done first.
Once the pull request above is merged we can point to the new Grafana bosh release. I am happy to do that, but for reference the steps are:

removing this FIXME commit
tagging the merge commit. The tag name tends to follow this convention
updating the manifest so releases.grafana.version is the same as the new tag name
update the pipeline to use the new release via tag filter

Re-deploy using the newly pinned resource.
Log into Grafana using make dev showenv to get the URL.
Check the User Impact dashboard contains the metrics as defined in Pivotal Tracker.
Consider whether the scales and times used are appropriate. Your development environment may not have a lot of metrics, so you can get creative generating some.

Who
Anyone but @saliceti @combor or me

It would be good to pin the Y axes of these graphs to 0.
Otherwise looks good.

@alext
I have:

rebased against master, now that the upgrade to CF v238 has been merged.
Removed both FIXME commits
Tagged the merge commit in the paas-grafana-boshrelease repository
Updated the releases.grafana.version value in the manifest to reflect the new tag.
Set the minimum Y-axis values to 0.

Note: I never needed to update the pipeline, because removing the merge commit made that update for me.
You can now deploy from visibility-of-metrics-122183539-minify again and review.

@alext: I have made the changes in the comments for the diff of c7ea017ad42af4d2ade3e6b3ef7389b3fa6c23f6. The changes were squashed into that commit.

After discussing with @saliceti , I think we should rename Router Xxx per minute to App Xxx per minute. If the app is missing, router returns 404, but that's not an app metric (as app is missing) and is not added into the metric that we display...

@mtekel: The change has been made and squashed into d790681.

There's one more thing - I think it would be good to test that the dashboards don't break our manifest. In some future versions grafana could change format or include weird characters (e.g. for color specification, or some unicode for labels), which when included into the final manifest could break yaml.
In this case we don't need to use any additional fixtures, but can use dashboards directly, as they are present in the repository at the test time...

It also looks like cc-generic doesn't involve all tasks in CC job queue. I would change the metric displayed to total instead, which seems to be more accurate - and less delayed.

@mtekel: test helpers updated and changed cc-generic to total.

Nice. I've tried the tests by creating a bogus dashboard that would not output from script correctly, breaking the tests...

authors: HenryTK alext mtekel


text:
[#157032509] Update BOSH cli to v3.0.1-gds1 (custom build)

What
Set the Subject Key Identifier when generating certificates
See https://github.com/alphagov/paas-bosh-cli/pull/1
How to review
Code review
How to merge
❗ The WIP commit needs to be updated with the final BOSH cli build
Who can review it
Not me.

I've updated the WIP commit now that the bosh-cli PR has been merged and released.

authors: alext bandesz

repo: alphagov/paas-docker-cloudfoundry-tools


text:
Adds design system components and warning text

Context
We don't have a good way to put warnings. The Design System offers us this, among other things which will be useful in future.
Changes proposed in this pull request
Adds GOV.UK Design System components and implements a warning symbol by means of testing on the "Payment flow" page.
Guidance to review
Preview at https://govukpay-hammertime.cloudapps.digital/payment_flow_overview/#payment-flow

Going to reopen this with a better approach

repo: alphagov/pay-tech-docs


text:
BAU Bump node-sass to address npm audit vulnerability report

npm audit fix bumps node sass to address vulnerability report.

Superseded by https://github.com/alphagov/pay-toolbox/pull/385

repo: alphagov/pay-toolbox


text:
Upgrade Nokogiri from 1.6.7.2 to 1.6.8

Fix for CVE-2015-8806 (denial of service or RCE from libxml2 and
libxslt).

👍  merge when green

authors: MatMoore mobaig

repo: alphagov/policy-publisher


text:
Remove links from randomly generated edition

Links were causing this test to fail intermittently. This will fix the links issue.
I think this might well have been already an intermittent test so I've updated the errors so it's clearer what happened and hopefully we can look into fixing it fully if it occurs again.

@danielroseman there doesn't seem to be a particular reason why we're using them there and does seem to add unnecessary baggage to this test that isn't concerned with it.
They have in the past helped us at catching problems where we don't properly support a schema. So might well make sense to remove them from this test and put together an integration test that is focused on random schemas.

The reason to use random content items is that if the schema changes, the tests will automatically be updated. By using hard coded content items / fixtures we run the risk that if you update the schema, you also have to update fixtures across GOV.UK (which was one of the problems with gettin rid of format).
Maybe the links in the content schemas was merged too soon? What kind of problems has it caused?

(For context, I wrote that test in https://github.com/alphagov/publishing-api/pull/523)

@tijmenb it wasn't probably merged too soon by us. We have an error thrown if links is present. Which will be cool once https://github.com/alphagov/publishing-api/pull/749 is merged.
The problem we tend to hit with tests related to randomly generated schemas is it's not particularly easy to determine the problem was from our code not supporting a randomly generated schema, and then a rebuild goes green and it goes unresolved.

Right, makes sense. An integration test that explicitly tests against randomised content items should surface those problems, assuming enough examples are run.

Merging this now so we have less build fails. Card to explore this at: https://trello.com/c/rZmA19cv/429-create-integration-test-for-randomly-generated-editions

authors: kevindew tijmenb

repo: alphagov/publishing-api


text:
Update finder deploy process

Alterations made as a result of running through the deploy process for raib

👍

authors: Rosa-Fox Seraphiana

repo: alphagov/specialist-publisher-rebuild


text:
Add new generic unprocessable entity page

https://trello.com/c/Uzpw1bDA/243-document-how-error-pages-work-on-govuk
Related to: https://github.com/alphagov/govuk-puppet/pull/10346
Extracted from: https://github.com/alphagov/email-alert-frontend/pull/743

@benthorner I don't think we should have this one as a generic error page as it stops us from rendering any non-generic 422 pages, unless I've missed some trick we have?
When we render a form with errors it's appropriate to use the 422 status code however this view would clobber the page.
We have a long, long running problem with this same situation with gone pages where they get clobbered by the page from static which means we have to do the rather grim thing of rendering a 200 response for gone pages with a customised response. Example:  https://www.gov.uk/government/publications/public-inquiries-timeliness-of-case-listings--2

@kevindew yes, that's right. We wanted to add this in favour of the previous "override of an override" approach. The thinking here is that we already return "200" for some unprocessable entity responses, in order to get around Slimmer casting them all to a "500".
While we have a related PR to remove that problem, I think we still need some kind of fallback error page. The problem is: we can't have a fallback and have specific error handling, unless we replicate the error pages in each app (or remove them from Nginx).
I agree it's not semantically correct to return "200"s in order to handle errors more specifically. The positive side is that our apps don't need to be watertight in order to show something vaguely helpful. I guess the downside is bad caching behaviour.
Looking at our logs in more details (see comment), perhaps 422 can be an exception to the rule, and the app must cope with it. This means we'd need to do some work on a few apps, though. Do you think we should adopt a different approach?

Example: https://www.gov.uk/government/publications/public-inquiries-timeliness-of-case-listings--2

Err, this might not be the finest example of a need for this: "duplication error"

@kevindew yes, that's right. We wanted to add this in favour of the previous "override of an override" approach. The thinking here is that we already return "200" for some unprocessable entity responses, in order to get around Slimmer casting them all to a "500".
While we have a related PR to remove that problem, I think we still need some kind of fallback error page. The problem is: we can't have a fallback and have specific error handling, unless we replicate the error pages in each app (or remove them from Nginx).

Yup, that's the real problem. It's frustratingly inconvenient to follow HTTP spec and have the nginx pages. I tried looking into whether you could configure nginx to allow more control but that didn't look simple.

I agree it's not semantically correct to return "200"s in order to handle errors more specifically. The positive side is that our apps don't need to be watertight in order to show something vaguely helpful. I guess the downside is bad caching behaviour.
Looking at our logs in more details (see comment), perhaps 422 can be an exception to the rule, and the app must cope with it. This means we'd need to do some work on a few apps, though. Do you think we should adopt a different approach?

Yup I'm keen for 422 to be something apps support. I worry that once this change settles in it'll be quite hard to get out of (as is the case with gone).

Example: https://www.gov.uk/government/publications/public-inquiries-timeliness-of-case-listings--2

Err, this might not be the finest example of a need for this: "duplication error"

Ha yup, though to me it's even worse that this returns a 200. I don't think there is anything preventing search engines from indexing it 😱 https://www.google.com/search?q=site:www.gov.uk+"The+information+on+this+page+has+been+removed+because+it+was+published+in+error."&sxsrf=ALeKk025PHewu8FVDfQUS4qNvKP26IhULQ:1589302256799&filter=0&biw=1210&bih=754
On a tangent I do wonder what the right way should be to be in a world where you can have fallback error pages and maintain the ability to override them, particularly if we start moving to a world where we do much less nginx configuration (as we already live in with govuk-docker). I guess we need some static error pages to handle cases where routes don't proxy to any app, so they may have to exist in some form. Then app wise we've got a situation where we're configuring individual error pages for each publishing app (that we might want to simplify for re-use) and thus may want to use the same mechanism in frontend apps which would bring it within Rails control for a less rigid approach.

@kevindew so my current plan is to proceed with the work we're doing on https://github.com/alphagov/static/pull/2167, so that we remove a second layer of error handling on GOV.UK. I think this is non-contentious, as it'll be a net improvement in the quality of the error pages seen by the user.
We then have a separate question of whether to revisit this specific issue with 422 response codes, and whether we should remove the fallback. This would mean:

Coping a generic 422 template into not-sure-how-many apps.
Adding support for 422 into Slimmer (or maybe just any code)

For me, the first point is a smell. To try and resolve it, perhaps we can introduce this as functionality of GovukPublishingComponents, with consuming apps doing something like:
# config/routes.rb
mount GovukPublishingComponents::Errors, at: "/"

# config/application.rb
config.exceptions_app = routes

Although this would be beneficial if GOV.UK becomes a more interactive site, it's hard to see the benefits in the medium term. I can add a card for us to try and do this, but we may need to abandon the idea if it starts blowing up, as it's increasingly tangential to our original aim. If we have to abandon it, we could extend the docs we're trying to write, to indicate the future intent.
How does that sound?

Yup, the mounting of an error controller from govuk_publishing_components sounds like a good approach. It'd be nice to have the configuration of Content Publisher easier to drop into apps for error handling and having frontend and backend consistent. Do agree it could blow up in complexity though.
I end up rather conflicted around this as I do appreciate the convenience and ubiquity of the nginx configuration, but I do feel it encourages dubious HTTP patterns, annoying work arounds and surprising effects.
Anyway thanks for carrying on the conversation about this 🙂

authors: benthorner kevindew

repo: alphagov/static


text:
Tweaks for newer Stylelint 14 rules

Hello 👋
Hope these updates are handy
In this PR I've mopped up a few newer Stylelint v14 rule updates for:

Configure stylelint@14.7.0 selector-not-notation
Configure stylelint@14.10.0 annotation-no-unknown

I've also updated to the latest v14 compatible versions of:
stylelint-config-standard@29
stylelint-config-standard-scss@6.1.0

To avoid warnings and to seamlessly run npm link against govuk-frontend I've had to bump package-lock.json to "lockfileVersion": 2 but this maintains compatibility with npm v5 and v6

For context, I've been working on the govuk-frontend Dart Sass update:

https://github.com/alphagov/govuk-frontend/pull/3164

Hey, thanks for this @colinrotherham. Sorry for the lengthy delay, I've just returned from a long period of leave and I'm somewhat the only core maintainer. I've got a lot to catch up on so may be a week or two until I get to this, but thanks so much and I'll get back to you soon.

Thanks for this @colinrotherham. I've found some time to take a look at this, most of the changes look great. I am however a little reluctant to do a release that handles both Stylelint 14 and 15, I feel it'd be better to do one just for Stylelint 15 (I've started pulling that together in: https://github.com/alphagov/stylelint-config-gds/pull/32 with commits from this PR).
The big thing that seems a pain is the removal of stylistic rules for the adoption of prettier, which seems to be a case of working out if GDS are happier with prettier and how we provide migration guidance.
P.S Are you working at GDS? We should, if you don't already have it, get your write access to this repo.

@kevindew Yeah send me a message on Slack, I'm happy to help and/or split this up to assist 😊

Hi @kevindew I've added a CHANGELOG entry and dropped the Stylelint v15 peerDependency
Happy to make further changes, tweak wording etc
Thanks again

authors: colinrotherham kevindew

repo: alphagov/stylelint-config-gds


text:
Add rake tasks for asset audit

Adds rake tasks for auditing the assets:

asset_manager:dump, gives (as a CSV) the URLs of every visible asset in Whitehall and its asset-manager path.
asset_manager:audit_check_status, reads the CSV and prints a CSV of the status code, "location" header, "content-type" header, and "content-length" header.

Trello Card

Thanks @elliotcm. I don't think there's any benefit to checking all the headers as there are bound to be many differences when it comes to version numbers and cache control headers and things like that, I think it's better to only check the headers we're checking now. I've addressed your other comments now.

Thanks @elliotcm, I just had to make a quick change for 2FA access through Signon if you'd be able to take another look.

authors: barrucadu thomasleese

repo: alphagov/whitehall


text:
avoid creating extra closure

callback is bound to component context here: https://github.com/rwjblue/ember-lifeline/blob/v2.0.0/addon/mixins/dom.js#L90

Released with v0.4.11

authors: bekzod offirgolan

repo: alphasights/ember-scrollable


text:
Failed to parse JSON from "https://api.travis-ci.org/jobs/147525158". Error: Unexpected token F

See https://github.com/semantic-release/semantic-release/issues/265. I think this is an issue within travis-after-all. I’m happy to help dig into it, I’d appreciate pointers / ideas of how to debug it

workes again, see https://github.com/semantic-release/semantic-release/issues/265

From https://github.com/semantic-release/semantic-release/issues/265:

Turns out the API call to https://api.travis-ci.org/jobs/147525158 returns Forbidden. Any idea why that might happen?

@gr2m That might have been due to a service outage, or Travis CI may have changed something related to the number of requests you can do to their API? 😕

There was no documented service outage during this time, I double checked. I pinged Travis support about it, let’s see what they have to say :)

I pinged Travis support about it, let’s see what they have to say :)

@gr2m Great, thanks! :)

repo: alrra/travis-after-all


text:
Fix readthedocs build

Fix #1520

RTD came back green 💹🥬🍏♻😂, merging!

repo: alteryx/evalml


text:
Add validation to URL logical type

Closes #1081

The URL validate method has a similar enough structure to the EmailAddress one, that I wonder if we could pull out the logic into a helper util that can be used for any validate method that relies on a regex.

authors: jeff-hernandez tamargrey

repo: alteryx/woodwork


text:
Messenger for desktop newer version. When it will be released?

Windows 7 sp1 ultimate
2.0.9 stable
I want to know when you released the new version of messenger for desktop cause that 2.0.9 stable version is toooooo unstable cause many times when i open the program and write to some of my friend the text bat i write the messages is disappear and the only solution is to RESTART the program.
PLEASE FIX THIS ISSUE.

We are not paid to update this software so fell free to contribute, I personally don't have a lot of time to give to this project anymore.

repo: aluxian/Messenger-for-Desktop


text:
How to get SnowKITTI ？

Hello ,thank you for your great work,
I have seen the simulation model in https://github.com/SysCV/LiDAR_snow_sim. A question is that point cloud in KITTI has 4 channels(x,y,z, intensity)，the simulation model is used for STF which has 5 channels. Could you please tell me how can we change the code to match the KITTI dataset. Hoping for your reply.

Hello,
You can download SnowyKITTI here: https://www.dropbox.com/s/o3r654cdzfl405d/snowyKITTI.zip?dl=0
The fifth channel is for laser elevation angle. Since KITTI does not include that, you have to compute those yourself e.g. project the point cloud and use the row indices.
If you plan to use our work, please cite us.
Citation is coming soon, accepted by IEEE Robotics and Automation Letters.
Best regards,
Alvari

authors: alvariseppanen dinghuixiang-github