text:
Fiend subclass spell prerequisites seem off

While looking at the subclass spells, I noticed that all of the prerequisites for the fiend subclass reference warlock level 1. This doesn't seem correct. I'll be using information from the Roll20 Compendium page on the warlock to show you what I mean.
The table for the fiend spells is the following:

Spell Levels
Spells

1st
Burning Hands, Command

2nd
Blindness/Deafness, Scorching Ray

3rd
Fireball, Stinking Cloud

4th
Fire Shield, Wall of Fire

5th
Flame Strike, Hallow

Also worth pointing out is the compendium's entry on spells known of first level or higher.

You learn a new Warlock spell every time you gain a level from 2 through 9, as well as at level 19. A spell you choose must be of a level no higher than what's shown in the table's Slot Level column for your level. When you reach 6th level, for example, you learn a new Warlock spell, which can be 1st, 2nd, or 3rd Level.

Looking at these two pieces of information, it doesn't seem to make sense making all of the fiend pact's spells prerequisite's level 1. That would mean, for example, that a player could use Flame Strike, a 5th level spell, when they only have 1st level spell slots available.
I propose that the prerequisite levels be changed to match when the warlock will have slots for a given spell.

I think that's a reasonable proposal. Feel free to cut a PR to address it!

There are no prerequisites for those spells. The spell level is an indication, and there shouldn't be any prerequisites because the data don't directly say there is any. Basically, all this feature does is adding the spells to the character's available warlock spell lists.

This proposed change is for the Warlock level that the spell becomes available at.

make it optional

It already is optional, in the sense that the consumer can ignore it if they want to.
However, if they're making a character sheet builder or similar, I think it could be really useful to know the level at which the spells become available to the character (i.e. when the Warlock's spell slot level is high enough), so that the spells are only added to the sheet when they're usable. But again, it's just an optional thing for the consumer to use if they want to do it that way - they don't have to.

A naive implementation would show two prerequisites: the one from the spell and the one from the subclass. And they'd be the same, and the developer would have to check every field and map logic to check that they're indeed equivalent.
There are no prerequisites in the description of the subclass, there shouldn't be any in the corresponding data.

I've thought about this a bit, and I think there's a good argument for there being no prerequisites. I'll repeat the rule for warlocks learning new spells:

You learn a new Warlock spell every time you gain a level from 2 through 9, as well as at level 19. A spell you choose must be of a level no higher than what's shown in the table's Slot Level column for your level. When you reach 6th level, for example, you learn a new Warlock spell, which can be 1st, 2nd, or 3rd Level.

If we take @fergcb's example where someone's making a character sheet builder, they'll likely already have logic for handling which spells are available to learn at a given level. After all, the subclass spells aren't the only spells that will be available to learn: they'll also have to apply that logic to the rest of the warlock's spell list. It would make more sense for the maker of the character sheet builder to use the same logic for both the class and subclass spells than for subclass spells to be special.

Also, if we take the logic of "must be warlock level X", and we extend it to everything (because we should if we are consistent), then they all have this prerequisite and what happens? Features that say "take a spell from another spell list" don't work anymore.

Should I reopen this issue or make a new one?

authors: abias1122 bagelbits fergcb ogregoire


text:
技术咨询：WVP 是否支持为不同的拉流代理设置不同的录制时长

内容描述:
在使用 ZLMediaKit 时，可以通过接口 /index/api/addStreamProxy 添加拉流代理，并可以传入参数 mp4_max_second 来控制每个拉流代理的 MP4 文件切片时长。
然而，在使用 WVP时，似乎在添加拉流代理的过程中没有类似的选项。
翻阅相关问题时，提到可以在 ZLMediaKit 的配置文件中设置 mp4_max_second，但这是一个全局配置，无法满足针对每个拉流代理进行个性化配置的需求。
我想确认：

WVP 是否支持为每个拉流代理单独设置录制时长？
如果目前不支持，是否有计划在未来版本中添加这个功能？或者是否有其他建议的解决方案？

截图
WVP添加拉流代理的时候无法设置mp4_max_second

目前不支持, 感觉意义不大

source: gharchive/issue

license: MIT

license_type: permissive

license_source: github-api


text:
chore: add linting workflow

implement auto linting for PRs

:tada: this issue has been resolved in version 1.0.0 :tada:
release is available on GitHub

source: gharchive/pull-request


text:
Actual full example

Good comp.. but no-where is there an example as to WHAT to actually import..

Hi, this is the same as a svelte file you would import locally.  It is a default export so you can name it whatever you want such as:
<script>
import Foobar from 'svelte-calendar'
</script>
<Foobar />

Updated readme.


text:
Fixed dj, dk, dw, db. Fixed word motion ignoring new line characters.

dj, dk, dw, and db now work as expected.
Fixed word motion ignoring new line characters.
Example:
//   | is the cursor
var a = |5;
var b = 5;

Pressing w before resulted in
//   | is the cursor
var a = 5;
var |b = 5;

Now it correctly moves forward one word.
//   | is the cursor
var a = 5;
|var b = 5;

What is wrong in dj and dk?

Your approach feels exaggeratedly.
I wrote this plugin in 3 days at last week, so its structure has some problems.
I want to think a little more slowly.

Fair enough. It's probably for the best to rethink the structure. I'd recommend looking at https://github.com/atom/vim-mode.
Also for dj, dk. The expected behavior is to delete the line the cursor started at, and the line the cursor ends at. However, before it was deleting from the start position of the cursor to the end position.

dj and dk already are able to delete lines. DeleteAction has is-treat-line option, dj and dk use it.

I converted a large chunk of atoms code into this project, but it would still need a couple days of testing to move in, and I'm worried that it would be too large of a move.
I have finished most of the selection mode, and will put a PR for you guys to look over in the next couple days :).

@DennyScott It's probably for the best. Atom's vim-mode has been around for a long time and they probably have a good implementation of vim features, but it's up to @74th on what's the best way to handle everything.
If you commit your changes to your branch, I could help test it for you.

I know atom/vim-mode is a great product and it is a good idea for increasing vim simulation.
But I think the goal of this plugin is not to contain atom/vim-mode.
Because I does not want to tail atom/vim-mode updates, and to make an integration atom and vscode.
The reason I started is VSCodeVim/Vim supporting any vim functions seems not good.
So I'll make the smallest number of functions to use VSCode without stress.

Would you be opposed to us forking your project and continuing on our own? It seems your desire is to add the keybindings you need/use, but not support the others. I don't think thats a fair assessment.

@benjaminRomano We could always move over support to the https://github.com/VSCodeVim/Vim project, they're a step behind this project in terms of keys, but their goal is to emulate the atom-vim mode.

@DennyScott I'll check it out.

I desire this features are

the largest benefit is corporation with vscode functions and other extensions with use vim keybinding
does not conflict with vscode functions and other extensions
does not support functions vscode supports, you should get to use them ( ex, undo, search, save command. )

So I think this plugin does not have to support keybinding with ctrl key.
They are used by vscode functions and other extensions.
(When I begun to code this, the largest problem is writing texts by an extantion does not corporate with Intellisence. Now, in insert mode it call to show suggesutions, and when it shows suggestions does not hundle keybinds. This problem has not been solved yet.)
My next update is if users add a pare of a vim-style-keybind and a vscode command to setting.json, it calls the command by the keybinding.
If you are not sure of this poricy or want to more improve vim simulation, please folk and release.

authors: 74th DennyScott benjaminRomano


text:
emailly.co

can you add following domains pls?
emailly.co
emailtoshare.com
getmail.pics
mailhq.club
meetmail.me
openmail.lol
usemail.live
usemail.xyz

Thanks for bugreport, we have been watching this provider but due to some changes, it was not working properly. We have those domains in our database now.


text:
fix: Pass the operationId option correctly to @hey-api/openapi-ts

fix #133

npm publish


text:
Add method to describe required Options for a dataset

Currently, when someone is using a dataset for the first time, they might not know all the Options required in their config in order to call that dataset.
In order to help users, we should add a describe method where the user can see what options are required.
@dataset
def add(x: int = Option('X'), y: int = Option('Y'):
    return x + y

add.describe() = {'X', 'Y'}

The difficulty is that required options can change depending on other values in the options dictionary due to overloads. We would need to think through how to deal with overloads in this scenario.

This will be .explain on rewrite/monadic


text:
修复缓存不生效；

es.forgeSearch.havingConditions 查询总是不为 nil, 使用 len == 0 判断；
主键查询缓存不生效，取消 84-85这两句后正常了；

主键查询缓存不生效，取消 84-85这两句后正常了；
这个改了会有问题的，单测可以试试

单测跑了也是触发不了缓存的 😭

主键查询缓存不生效，取消 84-85这两句后正常了；
这个改了会有问题的，单测可以试试

单测跑了也是触发不了缓存的。😭

看下example的主键缓存
八叉树
4932004@qq.com
------------------ 原始邮件 ------------------
发件人: KyleHwang <notifications@github.com>
发送时间: 2020年12月9日 06:22
收件人: 8treenet/gcache <gcache@noreply.github.com>
抄送: Subscribed <subscribed@noreply.github.com>
主题: 回复：[8treenet/gcache] 修复缓存不生效； (#19)
es.forgeSearch.havingConditions 查询总是不为 nil, 使用 len == 0 判断；
主键查询缓存不生效，取消 84-85这两句后正常了；
You can view, comment on, or merge this pull request online at:
  https://github.com/8treenet/gcache/pull/19
Commit Summary
修复缓存不生效；
File Changes
 M     internal/call_query.go     (2)

 M     internal/easy_scope.go     (3)

Patch Links:
https://github.com/8treenet/gcache/pull/19.patch
https://github.com/8treenet/gcache/pull/19.diff
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or unsubscribe.

看下example的主键缓存 八叉树 4932004@qq.com
…

不知道为什么，又没问题了。😳


text:
你这个有bug

生成不按顺序啊，我最近的一篇文章生成的是22，我有43篇文章呢

zhege这个我也写了啊

license: WTFPL


text:
es6-port

Port code to es6 with babel

Can I do it?

Sure, fork this repo, create your own branch and start working. Please use contribution guidelines provided.

@dhruvdutt just to clarify things, are you still working on this issue.

Sorry for the delay.
Most of the things seem alright. I'm still learning ES6. Is there anything to do except transforming normal functions into arrow functions?

It's fine, take your time, we need quality over quantity. Basically you can use arrow functions as a short hand syntax for normal functions, but you need to be careful about the context since arrow functions takes up the parent context. Also you can features like default arguments in ES6 within functions.

Also worth mentioning is that ES6 has a new class syntax that's way nicer. Import and export have changed as well, you can do, for example, export class MyClass instead of modules.export = {}. Read more about it here.

@rafaelklaessen You can take this up if you wish.
I have been really busy lately with academics work.

Okay, I will. I've never worked with React though, so it will take a while.

@rajikaimal I've created a pull request with the ES6 port.

I started a review on PR #17

I'm reviewing too. 😸

So, is my ES6 version going to be used or...?

We'll start improvements based on the work that you've done on this issue.

Okay

Hi I am interested on this. Can I start working on it  ? if it's not completed.

Looking at the code that I wrote back then there's plenty of room for improvement.
It's now kinda-partially ES6

Is it on a particular branch ? Or is it already merged ? I can continue with the things you have done.

It's on the es6 branch.
Do note that there's 41 new commits in master that need to be merged as well (or you can create a new branch and just redo it all)

Okay thanks for the heads up. I will start working on them.

Is the issue still open, as I am willing to do it

Hi, yes you can give a try.

authors: dhruvdutt dilantha111 rafaelklaessen rajikaimal shubhamkakkar


text:
added 'nosync' functionality

By default, 9volt-cfg will sync whatever is in its local configs against etcd - ie. it will remove any "orphaned" configs from etcd. Orphaned configs == any config in etcd that does not have a corresponding local config.
This functionality can be disabled by tacking on a -n || --nosync flag.

I got tired of manually removing entries in etcd - quickly hacked this up.

license: mit

license_source: bigquery


text:
Unify XML documentation and Resharper warning suppression

Fixes 2 small issues :

Use attributes instead of comments to disable R# warnings in SetWindowPosFlags #242
Use a consistent XML documentation text for nested types #241

LGTM

authors: AArnott vbfox

repo: AArnott/pinvoke


text:
ci: release changes

release changes

:tada: This PR is included in version 1.0.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


text:
NFSE Itajai - qual provedor devo utilizar? o arquivo de municipios não tem Itajai - SC

NFSE Itajai - qual provedor devo utilizar? o arquivo de municipios não tem Itajai - SC.
Qual deve ser o provedor escolhido para gerar nfses para Itajai? Eu vi que existe o schemma do provedor Publica, porém não existe  o provedor publica no enum, e nem no arquivo de municipios está incluso Itajai.
Se puderem me indicar o caminho para a configuração para Itajai agradeço.
Att,
Fernando

Quem define o provedor é a prefeitura da sua cidade, se o provedor dela é este publica então não temos ele implementado.
Você precisa implementar ele se quiser usar.

Rafael,
Obrigado pela resposta. Fiquei em duvida pois vi vários threads ref a acbr.net usado Itajai, e na pasta schemas do projeto existe este provedor publica.
Sabe me dizer como o pessoal tem usado para Itajai?

Você pode esta confundindo com o projeto em Delphi lá tem este provedor, ou alguém implementou mais não fez um pull request.

Você pode esta confundindo com o projeto em Delphi lá tem este provedor, ou alguém implementou mais não fez um pull request.

Fechando issue por falta de retorno do usuário.

Fechando issue por falta de retorno do usuário.

authors: FernandoARamos rftd


text:
Code review tools

resolution_fixed   |    by saw562@nci.org.au

At the last meeting it was brought up that it would be good to have peer review of people's rose configurations. After a quick bit of research it looks like there are tools for this that we can install on accessdev and hook into trac and the subversion repo, e.g. http://reviewboard.org

Issue migrated from trac:66 at 2024-01-31 17:29:39 +1100

@scott.wales@bom.gov.au edited the issue description

ibc599 commented

Just noticed that there are some modules for postgresql written by Kevin back in 2012 in the repository. Probably should be removed or updated since we are not using them.

ibc599 commented

Currently there is no postgres or mysql databases on accessdev.

@scott.wales@bom.gov.au changed status from new to accepted

@scott.wales@bom.gov.au commented

Is the NCI module handling databases Chris, or should I use puppetlabs/postgresql?

@scott.wales@bom.gov.au commented

I've put together a module for reviewboard, available at https://github.com/ScottWales/puppet-reviewboard
It requires a database, do we have an existing postgres DB to hook in to?

Scott Wales scott.wales@unimelb.edu.au changed status from accepted to closed

@scott.wales@bom.gov.au commented

Branch saw562/reviewboard adds reviewboard (amongst other minor fixes).
Changes are available for review at http://130.56.244.76/reviewboard/r/4/ (note I've not enabled LDAP auth yet because it's not a https server so you won't be able to log on)

@scott.wales@bom.gov.au commented

Since there's been no objections I'll merge this in today

@scott.wales@bom.gov.au commented

Split DB question to #79

@scott.wales@bom.gov.au changed owner from `` to saw562

@scott.wales@bom.gov.au changed status from closed to reopened

@scott.wales@bom.gov.au commented

Oh yay, more permission problems - https://accessdev.nci.org.au/review. Folders need to be owned by apache for it to work.

Scott Wales scott.wales@unimelb.edu.au committed [61068]

In [changeset:"61068c7a3eebb464476fe193356fc5edef062f9a/puppet"]:
#CommitTicketReference repository="puppet" revision="61068c7a3eebb464476fe193356fc5edef062f9a"
Merge branch 'saw562/reviewboard'

Fixes #66

 * Add reviewboard module
 * Add puppetlabs/postgresql module
 * Remove Kevin's postgresql module
 * Rename accessdev-node to accessdevnode ('-' is not allowed in module
   names)
 * Refactor web development tools into subclass of accessdevnode

@scott.wales@bom.gov.au changed resolution from fixed to ``

Scott Wales scott.wales@unimelb.edu.au changed resolution from `` to fixed

Scott Wales scott.wales@unimelb.edu.au committed [2]

In [changeset:"2ed474ade93d9febc80656842c522e9c7dacb168/puppet"]:
#CommitTicketReference repository="puppet" revision="2ed474ade93d9febc80656842c522e9c7dacb168"
s/var\/httpd/var\/www

Refs #66

Reviewboard commented

saw562 created review request review:4

Reviewboard changed reviews which not transferred by tractive

Scott Wales scott.wales@unimelb.edu.au committed [07]

In [changeset:"07be12a148adefd291781d0efa0070e4be75b405/puppet"]:
#CommitTicketReference repository="puppet" revision="07be12a148adefd291781d0efa0070e4be75b405"
Move reviewboard to /var/httpd/reviewboard

Refs #66

This should fix the permission issues, at the risk of code reviews being
lost on server reset

@scott.wales@bom.gov.au commented

Reviewboard is now running properly, just need to do a bit of plumbing between it and trac & should be good to go.

@scott.wales@bom.gov.au commented

Milestone Future deleted

@scott.wales@bom.gov.au changed status from reopened to closed

@scott.wales@bom.gov.au removed milestone (was Future)

@scott.wales@bom.gov.au changed resolution from `` to fixed

repo: ACCESS-NRI/accessdev-Trac-archive


text:
LNURL Lightning Address Support 'text/identifier' field

text/identifier is an optional metadata field used with a lightning address.
It seems Phoenix errors if there are any metadata types that are not known to Phoenix, it also errors if nothing is set and it's an empty array.
Here's the documentation.
https://github.com/fiatjaf/lnurl-rfc/blob/luds/16.md
Here's the error logs from phoenix:

2021-10-18 11:48:37,760 INFO  f.a.p.lnurl.LNUrl$Util  - retrieving metadata from LNURL=https://bitrefill.me/.well-known/lnurlp/max
2021-10-18 11:48:38,982 WARN  f.a.p.lnurl.LNUrl$Util  - could not process raw meta=["text/identifier","max@bitrefill.me"] at index=0: unhandled metadata type=text/identifier
2021-10-18 11:48:38,991 ERROR f.a.p.lnurl.LNUrl$Util  - could not read raw meta=[["text/identifier","max@bitrefill.me"]]: java.lang.NullPointerException: null
at fr.acinq.phoenix.lnurl.LNUrl$Util.buildLNUrlPayMeta(LNUrl.kt:193)
at fr.acinq.phoenix.lnurl.LNUrl$Util.extractLNUrl(LNUrl.kt:131)
at fr.acinq.phoenix.utils.Wallet.parseLNObject(Wallet.kt:143)
at fr.acinq.phoenix.send.ReadInputViewModel$readInput$1$1.invokeSuspend(ReadInputViewModel.kt:73)
at kotlin.coroutines.jvm.internal.BaseContinuationImpl.resumeWith(ContinuationImpl.kt:33)
at kotlinx.coroutines.DispatchedTask.run(DispatchedTask.kt:106)
at kotlinx.coroutines.scheduling.CoroutineScheduler.runSafely(CoroutineScheduler.kt:571)
at kotlinx.coroutines.scheduling.CoroutineScheduler$Worker.executeTask(CoroutineScheduler.kt:738)
at kotlinx.coroutines.scheduling.CoroutineScheduler$Worker.runWorker(CoroutineScheduler.kt:678)
at kotlinx.coroutines.scheduling.CoroutineScheduler$Worker.run(CoroutineScheduler.kt:665)
2021-10-18 11:48:38,992 INFO  f.a.p.s.ReadInputViewModel  - invalid lightning object:

I've removed the metadata field now so it'll work with us, but I think other services use it, not sure.

There's no issue on Phoenix's side at least in the example you gave: the full metadata provided by the bitrefill.me service was [["text/identifier","max@bitrefill.me"]], which violates the LUD-06 specs (because it does not contain the mandatory "text/plain" metadata). Phoenix will also error if the metadata field is empty, that's due to that same spec constraint.
The log of Phoenix you attached is not very explicit and a bit scary, but it actually means that "text/plain" is missing.
You can add the text/identifier metadata field back and it should work as long as the text/plain field is there. The identifier field will just be ignored (there will be a warning in the logs), but the app won't error. Please reopen this issue if that's not the case.

repo: ACINQ/phoenix


text:
Scope

Does HybridAuth respect scopes? Like the active field on the normal sign in method? I ask this because it seems to be ignoring that. Thus users that are not active (those that were deactivated) are still able to sign in via social account.

You can use custom finder by using finder option in Hybridauth config, which will allow you to modify the query used to find matching user record from db. Check the CakePHP manual for details about using custom finder for authenticators.

@ADmad I did it but if the user can't login because he is not active or blocked, the SQL Log still contains the user find query even the user didn't try to login again as you can see in this image sorry if I'm not clear enough.

@ADmad What do you think about this issue, please?

I don't understand what your issue is. Your screenshot shows the social profiles table being queried not users table. The social profiles table will always be queried but is corresponding users table record is not enabled and your custom finder sets conditions accordingly then user won't be logged in.

@ADmad Thank you

authors: ADmad BassemN urionlinejudge

repo: ADmad/CakePHP-HybridAuth


text:
BlockExecutor: Transaction is not executable

Description
Multiple BP nodes block sync up got transaction execution error. Error message in below:
[2018-10-31 09:48:50.220 - Error - BlockExecutor][12]: Exception while execute block. AElf.Synchronization.BlockExecution.InvalidBlockException: Transaction is not executable.
{ "TransactionId": "c7a070c5c619e915212b8ff388a3b2e36915c42e6a833c9e5d2b533ce4097ef5", "Transaction": { "From": "c6c16dcc28af8546d7aebffad69ad38ad370", "To": "9d8a8f1691509530a852aa6ea
56d6446b845", "RefBlockNumber": "143", "RefBlockPrefix": "/nGobw==", "MethodName": "PublishOutValueAndSignature", "Params": "CgIIKBImCiRjNmMxNmRjYzI4YWY4NTQ2ZDdhZWJmZmFkNjlhZDM4YWQ
zNzAaIgogmPI6ZUL0fSWpKmFq2qbr91uU4bwPS5bquReptVcwdAkiIgogi/PleRiBoiadovNHgG424nM6mPe7lyFMoVU3Z7TEBFYqBgid0rGcEQ==", "Sig": { "R": "5da4IfBYKkV1ReLkqz3oGT2JOcThCO9W98eA4bGLkXU=", "S
": "xaEQXweOTQZ+Rme8KDEKIpk7t9eQUOmjsFzL+p+Ia40=", "P": "BFbsmh2PM/Z2XI/l3v9voQ2DaFdHAdNwuzLHAVb1H+fuAsjntHiPCWQ8c7NszdjGwW3MKK+FRteuv/rWmtOK03A=" }, "type": "DposTransaction" }, "
SignatureSt": "SignatureValid", "RefBlockSt": "RefBlockValid", "Status": "TransactionExecuted", "ExecutedBlockNumber": "148" }
   at AElf.Synchronization.BlockExecution.BlockExecutor.ExecuteBlock(IBlock block) in /home/aelf/github/AElf/AElf.Synchronization/BlockExecution/BlockExecutor.cs:line 80

Repro Steps

Launch 3 BP nodes normally.
Stop one of BP node few minutes randomly.
Restart the stopped BP node.
Observe the BP node sync up process status.

Testing snapshot

#587 fixes it

This seems like a result of mistakes from other places. In most cases I have met, it is because invalid transaction in pool memory.

Issue fixed and not repro'd long time, close it.

authors: EricBlack gldeng shiwk

repo: AElfProject/AElf


text:
AFNetworking 里面存在UIWebView审核会出现警告， ITMS-90809: Deprecated API Usage - App updates that use UIWebView will no longer be accepted as of December 2020. Instead, use WKWebView for improved security and reliability，这个问题怎么解决

ℹ Please fill out this template when filing an issue.
All lines beginning with an ℹ symbol instruct you with what info we expect. You can delete those lines once you've filled in the info.
Per our *CONTRIBUTING guidelines, we use GitHub for
bugs and feature requests, not general support. Other issues should be opened on Stack Overflow with the tag afnetworking.
Please remove this line and everything above it before submitting.

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

What did you do?
ℹ Please replace this with what you did.
What did you expect to happen?
ℹ Please replace this with what you expected to happen.
What happened instead?
ℹ Please replace this with of what happened instead.
AFNetworking Environment
AFNetworking version:
Xcode version:
Swift version:
Platform(s) running AFNetworking:
macOS version running Xcode:
Demo Project
ℹ Please link to or upload a project we can download that reproduces the issue.

Use 4.x instead.

已经知道怎么改了，非常感谢
发自我的iPhone
------------------ 原始邮件 ------------------
发件人: Elf Sundae <notifications@github.com>
发送时间: 2020年6月1日 20:09
收件人: AFNetworking/AFNetworking <AFNetworking@noreply.github.com>
抄送: burningshen <1070192262@qq.com>, Author <author@noreply.github.com>
主题: 回复：[AFNetworking/AFNetworking] AFNetworking 里面存在UIWebView审核会出现警告， ITMS-90809: Deprecated API Usage - App updates that use UIWebView will no longer be accepted as of December 2020. Instead, use WKWebView for improved security and reliability，这个问题怎么解决 (#4579)
Use 4.x instead.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or unsubscribe.

authors: ElfSundae dulifu

repo: AFNetworking/AFNetworking


text:
Initialise homepage project

All instructions are on link:https://binary-studio.notion.site/Git-Homework-3d21a4e2665c4943aa743236ea3e9b93
Create repository on GitHub(public repository with name "homepage", add relevant topics from link (level 1.3).
Set security protocol (level 1.4).
Clone this repository to local computer.

This task was completed.

repo: AHarahulia/homepage


text:
Investigate Webcam Lag

Webcam is laggy for Howell, but not for Anish, Jilin, or James.
Howell will do research on this issue

Howell discovered it was a browser issue on his side

repo: AI-Coach-PT/ExerSights


text:
IndexError: single positional indexer is out-of-bounds

在运行add_labels文件时，大家出现过这个错误么？

您的错误没有截全，可能不太好判断。您可以从最底下往上截图。


text:
import autogen issue

ModuleNotFoundError                       Traceback (most recent call last)
Cell In[1], line 2
1 import os
----> 2 import autogen
3 from textwrap import dedent
4 from finrobot.utils import register_keys_from_json
ModuleNotFoundError: No module named 'autogen'

I have installed thepyautogen but getting this error

ModuleNotFoundError Traceback (most recent call last) Cell In[1], line 2 1 import os ----> 2 import autogen 3 from textwrap import dedent 4 from finrobot.utils import register_keys_from_json
ModuleNotFoundError: No module named 'autogen'

Have you able to resolve this?

authors: DrRoad Vishnuujain


text:
Error Running the HairFastGAN

Received this error:
(venv) G:\HairFastGAN>python main.py
No CUDA runtime is found, using CUDA_HOME='C:\Program Files\NVIDIA GPU Computing Toolkit\CUDA\v11.8'
G:\HairFastGAN\venv\lib\site-packages\torch\utils\cpp_extension.py:358: UserWarning: Error checking compiler version for cl: [WinError 2] The system cannot find the file specified
warnings.warn(f'Error checking compiler version for {compiler}: {error}')
Traceback (most recent call last):
File "G:\HairFastGAN\main.py", line 9, in
from hair_swap import HairFast, get_parser
File "G:\HairFastGAN\hair_swap.py", line 13, in
from models.Alignment import Alignment
File "G:\HairFastGAN\models\Alignment.py", line 8, in
from models.Encoders import RotateModel
File "G:\HairFastGAN\models\Encoders.py", line 9, in
from models.Net import FeatureEncoderMult, IBasicBlock, conv1x1
File "G:\HairFastGAN\models\Net.py", line 9, in
from models.stylegan2.model import Generator
File "G:\HairFastGAN\models\stylegan2\model.py", line 11, in
from models.stylegan2.op import FusedLeakyReLU, fused_leaky_relu, upfirdn2d
File "G:\HairFastGAN\models\stylegan2\op_init_.py", line 1, in
from .fused_act import FusedLeakyReLU, fused_leaky_relu
File "G:\HairFastGAN\models\stylegan2\op\fused_act.py", line 10, in
fused = load(
File "G:\HairFastGAN\venv\lib\site-packages\torch\utils\cpp_extension.py", line 1284, in load
return _jit_compile(
File "G:\HairFastGAN\venv\lib\site-packages\torch\utils\cpp_extension.py", line 1508, in _jit_compile
_write_ninja_file_and_build_library(
File "G:\HairFastGAN\venv\lib\site-packages\torch\utils\cpp_extension.py", line 1610, in _write_ninja_file_and_build_library
_write_ninja_file_to_build_library(
File "G:\HairFastGAN\venv\lib\site-packages\torch\utils\cpp_extension.py", line 2014, in _write_ninja_file_to_build_library
cuda_flags = common_cflags + COMMON_NVCC_FLAGS + _get_cuda_arch_flags()
File "G:\HairFastGAN\venv\lib\site-packages\torch\utils\cpp_extension.py", line 1780, in _get_cuda_arch_flags
arch_list[-1] += '+PTX'
IndexError: list index out of range

You need to try
pip3 install torch=1.13.1 torchvision=0.14.1 --index-url https://download.pytorch.org/whl/cu118

好像还是不行

Try without specifying the version like below if someone gets stuck on this
pip3 install torch torchvision --index-url https://download.pytorch.org/whl/cu118

I didn't check for problems that could occur from the version mismatch (it will update PyTorch to 2.X.X), but at least I was able to run the demo.

Thank you for reporting this issue. I apologize for the inconvenience you're experiencing while trying to run the method.
Based on the error message you provided, it appears that there's a CUDA configuration problem on your Windows system, which is preventing the ninja build process from completing successfully.
To resolve this, I recommend using Windows Subsystem for Linux (WSL). This approach has been tested by our team and should help avoid environment-related issues. Here's a step-by-step guide to get you started:

Install WSL following the instructions at: https://learn.microsoft.com/en-us/windows/wsl/install

Once WSL is set up, install Python 3.10 within your WSL environment.

Use poetry to install all required packages. This will ensure a consistent and isolated environment for the project.

Try running the method again within this WSL environment.

If you encounter any difficulties with this process or if the error persists after following these steps, please don't hesitate to reach out. Provide details about any new errors or issues you face, and I'll be glad to assist you further.

authors: ButyAfter Ja-sonYun bhavesh-hirpara ialhabbal maximkm

repo: AIRI-Institute/HairFastGAN


text:
About data in the summary.csv

Hi,
Your work is really awesome!
I am trying to train models with atomic DFT features like Löwdin atomic charges, but the wavefunc dataset is so large. Currently the summary.csv (https://n-usr-31b1j.s3pd12.sbercloud.ru/b-usr-31b1j-qz9/data/nablaDFT/summary.csv) is not available, so I cannot download and check. I wonder if I could find atomic charges in the summary.csv? Also, may I ask when the update would be ready for download?
Thank you so much!

Hello!
Here is an updated link to the summary file: https://a002dlils-kadurin-nabladft.obs.ru-moscow-1.hc.sbercloud.ru/data/nablaDFT/summary.csv
Unfortunately at this point Löwdin atomic charges are only available from the wave functions files.

Thank you！

authors: EBGU KuzmaKhrabrov

repo: AIRI-Institute/nablaDFT


text:
Container modes for running the DB

The ingestion process has different requirements, needs and contraints compared to the query use in which the database. Once ingestion has finished the constraints that were relevant during ingestion really are no longer relevant.
After ingestion it may be worth rebooting the containers with different configurations and tweaking the database settings, to prioritise ingestion.
The modes

Ingestion mode

has increased share memory
maybe disable indexes
disable foreign keys

Query mode

has less shared memory
activates indexes (if turned off)
activates foreign keys

interesting idea, but not worth my attention right now.


text:
Splice formal constraints into auto-annotation flow

This PR splices in align.schema.constraint constraints into  the align.compiler flow.
Note that we have created pydantic versions of our legacy constraints to complete this exercise. We are not using the new SMT enhanced constraints in this PR yet. In other words, type checking is enforced but geometry checking is not.

Ready for review @kkunal1408 @stevenmburns. This PR already touches 238 files so I would like to get this merged in before I start replacing the legacy constraints with the new SMT-based ones.
A couple of follow up items we may want to address (as a separate issue / PR):

I noticed we have a lot of  layers.json floating around for testing purposes. We should refer to a single layers.json for testing as far as possible.
I have renamed our legacy .const files to be .const.old to remove confusion. I would like to eventually get rid of them altogether. Is there any information we need to port over to .const.json format @kkunal1408 ?

No information is needed. I will check if those constraints are still working and move them to .const.json in a separate branch after discussing there sanity  with designers.

Looks good. Let's get this in.
How are we using these 'gold' directories?

They are not being used at all right now. I believe they were added at some point to support tests/regression/test_regression.py. But the --runregression flag required to enable these tests are never invoked by our CI. @kkunal1408 did you have any plans to turn them back on? We should consider removing tests/gold if not.
I also saw some other testcase_* directories in PlaceRouteHierFlow which don't appear to be used.

repo: ALIGN-analoglayout/ALIGN-public


text:
Initial changes to support substreams

Create substream_constraints.json
Update constraint_set.json to include the new substreams property
Update Examples.md to add MUX example using substreams

Looks good. Suggest review use of capitalisation ("Mux")

Now addressed in commit 96e4c524089583bead764cf8598bfbca2097dbce


text:
NAT scopes --> PR #50

Adds the three proposed scopes (global, Network Device, interface) to PR #50.

Superseded by #61.

repo: AMWA-TV/nmos-network-control


text:
add animation on scroll on home page

i want to add animations on scroll on the home page
please assign this issue to me @ANSHIKA-26

not needed thank you !
we prefer the website simple and clean without any animations

not needed thank you ! we prefer the website simple and clean without any animations

ok no problem 🫡


text:
Loc Automation - Updating ETA of loc job: 316

Loc Automation Updating ETA of loc job: 316

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

jobs/test/316/info.json
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

authors: APEXTest mayast

repo: APEXTest/AirLocTesting


text:
REFACTOR - General Solution APIBrasil

This pull request includes significant updates to the project, focusing on updating the README documentation, modifying the GitHub Actions workflow, and transitioning from Pipfile to Poetry for dependency management.
Documentation updates:

README.md: Completely revamped the documentation, including new installation instructions, updated API usage examples, and new sections for different services like WhatsApp, Vehicles, CNPJ, Correios, GeoLocalization, CPF, and SMS. [1] [2] [3] [4]

Workflow improvements:

.github/workflows/python-publish.yml: Added workflow_dispatch to enable manual triggering of the workflow. Updated steps to use Poetry for dependency management and package publishing. [1] [2]

Dependency management:

Pipfile: Removed the old Pipfile as the project transitions to using Poetry for dependency management.

Other changes:

.github/FUNDING.yml: Updated GitHub Sponsors usernames to include jhowbhz and IvanildoBarauna.
Service.py: Added comments indicating successful methods.

Oi @jhowbhz, esse é o draft da PR.
Todos os testes de integração eu fiz, e para testes eu publiquei a lib no test.pypi caso você também queira testar :)
https://test.pypi.org/project/api-brasil/

repo: APIBrasil/apigratis-sdk-python


text:
The first E3SM SCM results

I'm writing to submit the E3SM SCM results to the COMBLE-MIP Part I. The SCM results can be found in the /output_scm/E3SM. We ran the E3SM v2.1 and the v3 development version for the COMBLE case.

@mengz1993 It looks like all of your filenames contain 'alt_no_ugvg' and some contain 'alt_no_ugvg_sens_nudging' -- hopefully you have some info in the global attribute talking about this (I can't see that here on Github). Can you briefly explain here what these mean? Thanks.

Hi @twjuliano, sorry about the confusion here. alt_no_ugvg means that our E3SM SCM simulations didn't use the pressure gradient forcing via geostrophic wind because our model doesn't have this capability. The file name with "_sens_nudging" means we turned on the nudging on U, V, T, Q in our SCM runs as additional sensitivity experiments. Thanks.

Hi @twjuliano, sorry about the confusion here. alt_no_ugvg means that our E3SM SCM simulations didn't use the pressure gradient forcing via geostrophic wind because our model doesn't have this capability. The file name with "_sens_nudging" means we turned on the nudging on U, V, T, Q in our SCM runs as additional sensitivity experiments. Thanks.

Got it, thanks for clarifying @mengz1993.

repo: ARM-Development/comble-mip


text:
expanding mobile menu and conversion of codebase to SASS.

incredibly lazy and dangerous commit
What this is
IN DEVELOPMENT
This will give us a proper mobile menu. it's a wide change that touches a majority of client.
specific and dangerous places
new-header.scss line 411 is especially dangerous!!

all done and tested! merging now ~


text:
Fix LISA-shell paths

lisa-help only works when called from the LISA root folder. Fix this by
storing LISA_HOME correctly and add a useful error message if someone
tries to initialize the LISA-shell outside the LISA root folder.
Signed-off-by: Morten Rasmussen morten.rasmussen@arm.com

This seems to be fixed in mainline now.

authors: derkling msrasmussen

repo: ARM-software/lisa


text:
Use c_lib std by default

All targets use standard C libraries instead of the small C libraries. This prevents potential issues with certain drivers that may have dependencies on c_lib implementation.
Summary of changes

[] I confirm this contribution is my own and I agree to license it with Apache 2.0.
[] I confirm the moderators may change the PR before merging it in.
For new board enablements only:
[] I confirm the board is Mbed Enabled and passes the Mbed Enabled test set.
[] I confirm the contribution has been tested properly and the tests results for TESTS are attached.
[] I confirm mbed-os.lib and mbed-cloud-client.lib hashes or the content in folders mbed-os and mbed-cloud-client were not modified in order to pass the tests.

Maintainers of this repository update the Mbed OS and Client hashes.
Please report any issues through issues in relevant repositories.

build-only

build-only

build-only

build-only

repo: ARMmbed/mbed-os-example-pelion


text:
Review PSA Crypto compliance test suite results for PSA compliance - CI

TF-M report some non-compliance issues discovered by their test suite.

[ ] Get PSA Crypto compliance suite into CI
[ ] Identify non-compliance issues and create tasks in the compliance epic for each

The relevant version of the test suite is the tip of https://github.com/ARM-software/psa-arch-tests on the crypto1.0 branch.

repo: ARMmbed/mbedtls


text:
Update soon to be expired crl

Description
Update crl.pem, as it will expire on November 25 2019.
Resolves #2357.
Status
READY
Requires Backporting
Yes
As this crl will be expired on the LTS branch, which will still be supported on November 25 2019,  need to backport as well.
Todos

[x] Tests
[ ] Documentation
[ ] Changelog updated
[ ] Backported

@mpg Thank you for your review!
I have just noticed I have missed adding crl.pem as a dependency in the Makefile.
Backported to 2.7 and 2.16 in #2418 and #2419

@hanno-arm Yes. All the other CRLs we have, the soonest year they expire is 2024.
We have other CRLs that expired at 2011, so since we didn't get any errors sine, they are probably not used in our tests, and we require some house cleaning for certificates and crls not used in tests.

Or perhaps the expired CRLs are used in tests that actually need the CRL to be expired?

perhaps.
Anyway, when I run:
faketime -f '+2y' make test , the tests pass. ( 3 years ahead I get failures.
I have noticed that: data_files/test-ca.crt is valid until:
Not After : Feb 12 14:44:00 2021 GMT

and that data_files/server3.crt is valid until:
Not After : Aug  7 09:17:03 2023 GMT

I haven't checked other certificates. Anyway, this should be handled in a different PR, and maintained regularly.

@RonEld I'd prefer those CRTs to be regenerated (and their build-instructions to be rewritten) so that they don't expire in the next 3 years. Now you/we are into the issue and it's easy to fix along the way, while waiting for another report to come up in a year's time or so and triaging and fixing it again will be more work.

In addition to what Hanno wrote, our LTS branches are supported for 3 years, so I think it would make sense for us to always ensure faketime -f '+3y' make test pass when we release a new LTS (and released one very recently, so I think we should have it pass now).

@hanno-arm @mpg I agree that we should have a test that will ensure faketime -f '+3y' make test will pass. However, I disagree that this should be handled in this PR, as it is an entirely different task, as I think we should align all our certificates and crls to be expired on the same time, for simplicity

@RonEld I'm not sure this is an entirely different task - this PR is about making tests pass in 1 year, so just changing the numeric value from 1 to 3 hardly changes the nature of the task IMO. (The larger work you suggest in your email would indeed be a pretty different task.)
However, if you really to handle the two remaining certs necessary to pass in 3 years in this PR, I'll still approve this PR as it stands in the interest of unblocking it, as it already brings a valuable improvement that I don't want to block by requesting more of the same.

I am working on making the tests pass in three years. however htis is more than just changing two certificates

@RonEld Why is that more than just changing the two certificates you mentioned in this earlier message? How much more?
I based my previous comment on the assumption that it would be just changing two certs, so if it's much more that than I could change my mind.

The task for making it poass in three years include:

As you can see in the list I stated in my email, you will see many certificates that expire in three years. ( I listed the main server1.crt and test-ca.crt but all their derivatives as well). THis is not a big issue though.
Changing the Makefile in data_files to generate update certificates. Not a big issue.
Modifying the .data file in the x509parse test suite, to have the correc tinformation. This is not difficult, only takes time.

As mentioned, I am currently working on this, probably be ready by end of day

@hanno-arm @mpg I have updated the certificates and corresponding test data files to make our tests pass 3 years ahead.
I still think this is a patch, as it should be handled in a more maintained way though.
I ran the following tests to verify:
faketime -f '+3y' make test
faketime -f '+3y' tests/ssl-opts.sh
faketime -f '+3y' programs/ssl/ssl_server2 and faketime -f '+3y' programs/ssl/ssl_client2

A few notes:

server2-badsign.crt was generated manually, using a debugger to corrupt the signature, so no Makefile entry done for that.
enco-ca-prstr.pem was generated through mbedtls_cert_write program, since openssl didn't generate the CN in printablestring. Our certificate write writes in UTF8, so manually modified this. Therefore no Makefile entry for that as well.
I am not sure what is the purpose of test-ca_uppercase.crt, as the content of original file was not in uppercase. I generated the file, as original CN.

Please review

Note: I think it's worth fixing #822 as part of this PR

patch applies cleanly to 2.16.0 but only adds another 10 years to the life of the software:
openssl crl -in ./tests/data_files/crl.pem -noout -text
        Last Update: Feb  6 16:34:46 2019 GMT
        Next Update: Feb  6 16:34:46 2029 GMT

cant you make it expire in the year 2999?

@mpg, @hanno-arm - please re-review this PR.

After think a bit some more, #822 should be handled in a different PR,as part of a house cleaning task, removing unused certificates.

@bmwiedemann
Yes, the crls are valid for 10 years, as they were before.Sine the lifespan of our LTS branches are ~3 years, and these are test certificates, I think its reasonable to have these certificates expire in 10 years.
However I am willing to hear other thoughts on this matter.
@mpg @hanno-arm any comments?

Keeping the "needs work" label until conflicts are resolved.

I have rebased to resolve conflixts.
I have also added a commit to update expiration date of new certificates.
@mpg @hanno-arm Do you want me to squash https://github.com/ARMmbed/mbedtls/pull/2417/commits/66d99ae2aa9343b4d5e74f5ba312676595874168 with https://github.com/ARMmbed/mbedtls/pull/2417/commits/23054c8b9480fb94428913d4361aea2ebf9e4b5a?

@mpg @hanno-arm Do you want me to squash 66d99ae with 23054c8?

As you prefer, both are fine with me - though I have a slight preference for squashing if nobody opposes.
Also, github is still showing conflicts.

@mpg yes, the new conflict is from the merging of #1520
I will rebase to fix the merge conflict.

As you prefer, both are fine with me - though I have a slight preference for squashing if nobody opposes.

Personally I think it's better if I squashed the commits, but I think it will make the reviewers an easier life if the squashing will happen after all the changes will be reviewed

@mpg I have resolved the conflicts. Please review.
I have noticed  a typo in the last commit message, but I wouldn't bother, since the last two commits will be squashed to https://github.com/ARMmbed/mbedtls/pull/2417/commits/e87c6855bd7601cf8c7c5063990a8068c42b3fc2

Thanks! Actually, would you be able to find another reviewer? Atm as you now I'm fully dedicated to some other work.

@mpg sure, no problem.
I just thought that since you started this, it would be faster if you finish.
I'll assign other people

@AndrzejKurek Thank you for the review.
As mentioned in https://github.com/ARMmbed/mbedtls/pull/2417#issuecomment-470439682, I have squashed the latest commit to clear history, but you can look in this diff that nothing has changed in the process since your last approval.
I apologize for not clearing the history prior to your review.

@k-stachowiak @AndrzejKurek I amended the commit "Update certificates to expire in 2029 " to fix CI failure.
It was only removing of an extra ; character in certs.c line 203. This caused a failure in ARMCC5:
"certs.c", line 205: Error:  #381-D: extra ";" ignored
  const char mbedtls_test_srv_crt_rsa[]     =
EST_SRV_CRT_RSA_SHA256;
                                                                      ^
certs.c: 0 warnings, 1 error
Makefile:164: recipe for target 'certs.o' failed
make[1]: *** [certs.o] Error 1
make[1]: *** Waiting for unfinished jobs....
make: *** [lib] Error 2
Makefile:18: recipe for target 'lib' failed
^^^^build_armcc: build: ARM Compiler 5, make: command make CC=/usr/local/ARM_Compiler_5.06u3/bin//armcc AR=/usr/local/ARM_Compiler_5.06u3/bin//armar WARNING_CFLAGS=--strict --c99 lib -> 2^^^^

Please review

This PR and its backports have been fully reviewed and approved. Removing the "needs backports" label
cc @Patater

Hi, I'm wondering what is missing here for this to get merged? The 2019-11-25 expiry of the crl is not that far away.

collides with
commit 53756b32282d0b36a6900cf2459250a328ae6214
Author: Hanno Becker <...@arm.com>
Date:   Mon Jun 3 14:14:38 2019 +0100

    Add MD[245] test CRTs to tree

@k-stachowiak @AndrzejKurek I have rebased to resolve conflicts
During rebasing, I found new certificates that need to be updated, and other certificates that fail with faketime -f '+3y' since this PR was raised.
Please review

This PR is fully reviewed, so removing the "needs backports" label
cc @Patater

authors: AndrzejKurek RonEld bmwiedemann hanno-arm mpg


text:
🛑 URL Shortener API is down

In 70c0341, URL Shortener API (https://api-url-zgau.onrender.com/) was down:

HTTP code: 0
Response time: 0 ms

Resolved: URL Shortener API is back up in af738c2 after 10 minutes.

repo: ASJordi/website-activity-status


text:
Create Profile for ...Amanyara Allan

Name
Amanyara Allan
Profile Image

Country
Uganda
Region
West Nile
Organization
The Youth Empowerment Foundation
Languages

English (good) , - kiswahili (fair), madi(good), lugbara (good).

Skill 1
Media & Art
Qualifications for Skill 1
I have a Diploma in Journalism and Mass Communication from  UMCAT School of Journalism and Mass Communication in Kampala Uganda. Experience  of over five years in Media Practices, I worked as Media Analyst in Reel Analytics Uganda Limited a company of Media Monitoring, I also did worked as Presenter and Producer with UBC West Nile and also worked with Luuke Media Ltd as Video and Photo Editor, currently working in his personnel projects of doing Podcasts, Documentaries and Television stories  among others.
Skill 2
Organizational Development
Qualifications for Skill 2
Am currently pursuing a Bachelors Degree in Human Resource Management in Metropolitan International University Kampala Uganda. I also attained Management skills, short Conflict and Resolutions training and Crisis Management.
Skill 3
Web & Software
Qualifications for Skill 3
I have a diploma in Journalism and Mass Communication who specialized in Media Production such as Photography and Videography, Graphics design, Video and Audio Production, Filming and Documentaries etc
Skill 4
Scripting and Documentation
Qualifications for Skill 4
Still Journalism and Mass Communication Background earn me most of this, I do scripting for all the media production activities such as Films, Documentaries, Television and Radio Adverts, Podcasts, and Radio and Television Programs.
No response
describe yourself
Amanyara Allan,a Ugandan by nationality ,34 years old living in adjumani district, currently in pagirinya refugee settlement, doing some voluntary work at The Youth Empowerment Foundation (YEF) pagirinya satellite space.
I am Journalist and Human Resource manager by profession.I love seeing people succeed in doing their things.Working together with others is a priority. # TOGETHER WE STAND,DIVIDED,WE FALL.
E-Mail
amanyaraa@gmail.com
Phone
+256787805016/+256702199041
WhatsApp
+255702199041
Website
No response
Telegram
No response
GitHub
No response
LinkedIn
No response
Twitter
No response
Facebook
Amanyara Allan
Instagram
No response
Mastodon
No response
WikiFab
No response
Other
…
No response

uploaded the data to content branch, will be visible after the next pull request

Edited this profiles description, @romeoplat and the rest cross check.

Hey could someone collect the links that could affirm Amanyara's work please
@chardso @vugawilliam

@timmwille this profile has been created along time go and is even live on the main LEAD website

authors: DutKulang MartinSchott chardso romeoplat

repo: ASKnetCommunity/LEAD


text:
#278 Add plugins to Makefile

Added GitLens, Auto Rename Tag, Go, and GoTestExplorer extensions to install-code-plugins phony

@AUBGTheHUB/nosyncdev any suggestions on what plugins we should add to the default installation? @r-mishev can you check if this is included in the installation scripts and if it's not, please add it to the post installation step.

authors: asynchroza r-mishev


text:
As a careless traveller, I want to set the quantity of items I need to pack

... so that I can pack without missing out anything

Done by #143 and #150


text:
Make supported OSes clear in UG

What this does
This PR makes it clear what Operating Systems are supported in our User Guide, to avoid any confusion.

Fixes #154 .
How to test

Just read the diff.
Notes

I removed Unix from the Mainstream OS definition in DG as well. It's not wrong, just not helpful to our DG, and we don't get to test on the BSDs etc.

Codecov Report

Merging #168 into master will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master     #168   +/-   ##
=========================================
  Coverage     63.75%   63.75%
  Complexity      932      932
=========================================
  Files           155      155
  Lines          3383     3383
  Branches        428      428
=========================================
  Hits           2157     2157
  Misses         1078     1078
  Partials        148      148

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 71fb480...0806be2. Read the comment docs.

authors: JingYenLoh codecov-io


text:
Missing tests for undo/redo functionality

Write missing tests for the new classes added for undo/redo.

All tests that use assertCommandSuccess also test for undo now, as well as some other tests that I wrote, so i'm closing this for now.

authors: seowalex zhiayang


text:
Add default welcome message

(Closes #89)
Previously, CLInkedIn had no welcome message to users who launch the application.

Let's add a welcome message to give them information on starting the application as well as several useful commands to get started
Additionally, let's make the resultDisplay textarea wrap text automatically, so that users will be able to view the messages on all screen sizes

Here's what the welcome message looks like:

Codecov Report
Base: 55.76% // Head: 55.73% // Decreases project coverage by -0.02% :warning:

Coverage data is based on head (2e8c4f5) compared to base (03bdd5d).
Patch coverage: 0.00% of modified lines in pull request are covered.

Additional details and impacted files
@@             Coverage Diff              @@
##             master     #107      +/-   ##
============================================
- Coverage     55.76%   55.73%   -0.03%
  Complexity      496      496
============================================
  Files           102      102
  Lines          2064     2065       +1
  Branches        231      231
============================================
  Hits           1151     1151
- Misses          823      824       +1
  Partials         90       90

Impacted Files
Coverage Δ

...rc/main/java/seedu/clinkedin/ui/ResultDisplay.java
0.00% <0.00%> (ø)

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

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

authors: Josephaedan codecov-commenter


text:
Add test cases to improve code coverage

Added tests in AppParametersTest and the following packages:

seedu.guest/logic
seedu.guest/model
seedu.guest/storage

Codecov Report
Base: 72.18% // Head: 75.00% // Increases project coverage by +2.81% :tada:

Coverage data is based on head (a70058b) compared to base (e3d7a1a).
Patch coverage: 100.00% of modified lines in pull request are covered.

Additional details and impacted files
@@             Coverage Diff              @@
##             master      #78      +/-   ##
============================================
+ Coverage     72.18%   75.00%   +2.81%
- Complexity      392      421      +29
============================================
  Files            69       69
  Lines          1208     1208
  Branches        120      120
============================================
+ Hits            872      906      +34
+ Misses          304      275      -29
+ Partials         32       27       -5

Impacted Files
Coverage Δ

...in/java/seedu/guest/model/util/SampleDataUtil.java
83.33% <100.00%> (+83.33%)
:arrow_up:

src/main/java/seedu/guest/model/guest/Guest.java
94.87% <0.00%> (+2.56%)
:arrow_up:

...ain/java/seedu/guest/commons/core/GuiSettings.java
73.07% <0.00%> (+3.84%)
:arrow_up:

src/main/java/seedu/guest/model/GuestBook.java
95.83% <0.00%> (+4.16%)
:arrow_up:

.../java/seedu/guest/model/guest/UniqueGuestList.java
97.56% <0.00%> (+4.87%)
:arrow_up:

...c/main/java/seedu/guest/model/guest/DateRange.java
94.73% <0.00%> (+5.26%)
:arrow_up:

.../main/java/seedu/guest/storage/StorageManager.java
100.00% <0.00%> (+5.55%)
:arrow_up:

src/main/java/seedu/guest/model/guest/Name.java
90.00% <0.00%> (+10.00%)
:arrow_up:

src/main/java/seedu/guest/model/guest/Email.java
90.00% <0.00%> (+10.00%)
:arrow_up:

src/main/java/seedu/guest/model/guest/Phone.java
90.00% <0.00%> (+10.00%)
:arrow_up:

... and 7 more

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

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

authors: codecov-commenter jehousoh


text:
Edit consultation test cases

Add edit consultation test
Add test for non-existing module and tutorial in editStudentCommandTest
Fix sample data error (Empty string for grade throws an exception) -> change to PENDING...

Closes #237

Codecov Report
Base: 59.47% // Head: 63.03% // Increases project coverage by +3.56% :tada:

Coverage data is based on head (9ba3562) compared to base (2848296).
Patch has no changes to coverable lines.

Additional details and impacted files
@@             Coverage Diff              @@
##             master     #238      +/-   ##
============================================
+ Coverage     59.47%   63.03%   +3.56%
- Complexity      812      856      +44
============================================
  Files           156      156
  Lines          3252     3252
  Branches        420      420
============================================
+ Hits           1934     2050     +116
+ Misses         1134     1022     -112
+ Partials        184      180       -4

Impacted Files
Coverage Δ

.../java/seedu/address/model/util/SampleDataUtil.java
9.09% <ø> (ø)

...main/java/seedu/address/model/student/Student.java
91.66% <0.00%> (+2.77%)
:arrow_up:

...ess/logic/commands/student/EditStudentCommand.java
92.17% <0.00%> (+3.47%)
:arrow_up:

...seedu/address/model/consultation/Consultation.java
75.00% <0.00%> (+6.81%)
:arrow_up:

src/main/java/seedu/address/model/AddressBook.java
69.23% <0.00%> (+7.69%)
:arrow_up:

...c/main/java/seedu/address/model/commons/Venue.java
80.00% <0.00%> (+10.00%)
:arrow_up:

...n/java/seedu/address/model/commons/ModuleCode.java
80.00% <0.00%> (+10.00%)
:arrow_up:

...ss/model/consultation/ConsultationDescription.java
75.00% <0.00%> (+12.50%)
:arrow_up:

...rc/main/java/seedu/address/model/ModelManager.java
80.30% <0.00%> (+14.39%)
:arrow_up:

...ess/model/consultation/UniqueConsultationList.java
56.81% <0.00%> (+15.90%)
:arrow_up:

... and 1 more

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

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

authors: april-anh codecov-commenter


text:
Saving window preferences

https://github.com/AY2223S2-CS2103-F11-3/tp/blob/bd171c0108f75ff1c61c6bca63f618fb758eacda/docs/DeveloperGuide.md?plain=1#L1134-L1139
Don’t think we have such a feature right?

My bad the feature works, just that you have to cleanly exit the app


text:
[PE-D][Tester D] Attendance not marked as intended (Date)

Test command:

student 1B attendance in/02 att/10/12/2012

The result of attendance in the GUI is still indicated as F instead of showing the date of attendance.

Labels: type.FunctionalityBug severity.High
original: Jnjy/ped#2

See Issue https://github.com/AY2223S2-CS2103T-T09-1/tp/issues/174 for feature fix for attendance


text:
[PE-D][Tester B] Needless case sensitivity used

commands like p/ , e/, etc are needlessly case sensitive. Restricts the usability of the application.

Labels: type.FeatureFlaw severity.Low
original: Saezenn/ped#6

Can add a footnote in the UG or something, this doesn't seem very valid to me.

authors: keaganpzh nus-se-script


text:
Add Skeletal PPP CY

Create docs/team/KumChaiYin.md
Closes #31

LGTM

authors: KumChaiYin brendanneojw


text:
Tag UI

UI for tag (different colour for each word)
incorporate it into the enums class, “NR(Colour.Red)” from import.java.awt.Color

UI for tags have been implemented


text:
Update AboutUs page - Kay Hian

Fixes #13
Update AboutUs.md with Kay Hian's info

LGTM!

authors: k-hian wkxcass


text:
home_view options are not working.

strategy:
  type: custom:mushroom-strategy
  options:
    home_view:
      hidden:
        - greeting
views: []

unfortunately this did not work either.
Originally posted by @derekpurdy in https://github.com/AalianKhan/mushroom-strategy/issues/82#issuecomment-1910286580

I got it working:

@derekpurdy I've checked and it all seems fine to me.
Are you sure you're using v2.0.0 and don't have any caching issues?

@derekpurdy I've checked and it all seems fine to me. Are you sure you're using v2.0.0 and don't have any caching issues?

i tried it incognito, and it appears you're right.. it's updating now as I'm making changes.

authors: DigiLive clowgg derekpurdy

repo: AalianKhan/mushroom-strategy


text:
FDroid

Is this on the f droid store?

No and I didn't want to add it for a few reasons mainly

My code is terrible
No one releases FOSS apps as split APKs. Practically this app can only be used for merging closed source apks, I don't think it will make a lot of sense for F-Droid

Does it send a notification if there's an updated version to download from GitHub?

Does it send a notification if there's an updated version to download from GitHub?

authors: AbdurazaaqMohammed PeterKW

repo: AbdurazaaqMohammed/AntiSplit-M


text:
Issue when parsing model

A few BSP files contains models that generates wrong faces when parsed.
This is a small file that can reproduce the issue:
wrong.zip
This is how model *2 is supposed to look:

This is how model *2 geometry look when parsed with Decay-Library. Looks like it is getting indices from other models and also missing a few indices from model *2 itself:

Model *2 is just the orange column

This issue is caused when you itereate thru your edges list in Tree.cpp
You are using a 16-bits index value where some models can have up to 65535 edges
I've implemented a quick BSP parser myself and managed to reproduce the same issue as yours

repo: AbitTheGray/Decay-Library


text:
A cache for Processors

Some applications would benefit from having a cache for Processor instances.  For example, some applications import hundreds of files and wind up requesting a conversion between the same pair of color spaces many times.  This is slow and could waste memory.  As another example, timeline-based applications have strict performance requirements and cannot wait to create a new Processor on each shot.
The OCIO context variable mechanism poses challenges in the development of a useful cache.  A naive solution would be to simply invalidate the entire cache whenever the context changes at all.  However, this would make it worthless for timeline-based applications since it is common for at least some part of the context to change on every shot.  Ideally, OCIO would be able to determine which context variables were used for the creation of a given Processor and use this information to invalidate the cache intelligently.
The generation of the cache key needs to be very fast, to make it suitable for timeline-based applications.
In addition to the original Processor, the optimized Processor, the CPUProcessor, and the GPUProcessor would be cached.
Unlike the existing file cache, which is global, the processor cache would be part of a config instance.
The API would provide a way to turn off the caching (e.g. for debugging purposes).
As discussed at a recent OCIO v2 working group meeting, we have been working on a cache and are creating this issue to collect thoughts and requirements.

This would be great. I'd love to be able to "pre-cache" a processor for a set of set of contexts in applications with multi-shot timelines.
Would the cache persist when toggling it on and off?
Could the caches be serialized? I.e., as a catalog of ctfs for different permutations of context variables? That could be really handy.

@zachlewis , yes, an application should be able to build all of the Processors in a "pre cache" step in order to load the cache.  Requesting any of those Processors a second time should be very fast.
What we have will clear the cache when it is turned off.  We feel that this is an important part of turning it off, so we want to keep it that way.  However someone could add a third mode (beyond off and on) that would not flush the cache but also not add any new ones.
Our implementation of the cache itself will not support serialization.  You may already serialize arbitrary Processors as CTF, so maybe that helps for what you have in mind.  Although building the cache is not something you want to do while playing back a timeline, it's not like it takes a really long time to do either, so it's not clear that a feature for serializing the cache is worth the effort.

@zachlewis, The work is now completed and merged if you see some improvements you can still log another issue.

authors: doug-walker hodoulp zachlewis

repo: AcademySoftwareFoundation/OpenColorIO


text:
New Landscape Entry - xStudio and OpenRV

Project Name
xStudio and OpenRV
Description
tbd
Homepage URL
tbd
Repo URL
tbd
Organization hosting or owning the project
ASWF
Logo
In artwork repo
Category
Imaging and Color - Display and Review

@yarille Once you have the artwork for both of these - can you add in the landscape?

@alatdneg could you provide us an SVG of the xSTUDIO logo?


text:
TAC meeting notes 2020-May-20

TAC meeting notes 2020-May-20
Signed-off-by: Jean-Francois Panisset panisset@gmail.com

Thanks JF!

authors: dheckenberg jfpanisset

repo: AcademySoftwareFoundation/tac


text:
Unit test fixes for changes to audit log processing

Audit log searches are now filtered over the promotion and demotion date ranges
Audit log fixture data actor changed to match report user otherwise all entries are filtered out
Also fixes a formatting issue in config.js

@droidpl - Was curious to see what the unit test changes were.  Think this covers them.

@steve-norman-rft exactly that was the thing. I knew it was related to the phrase as I changed it to get better results, and seems I forgot to change the tests. Shame on me 😄

authors: droidpl steve-norman-rft


text:
fix(versions): update Activiti/activiti-cloud-connectors versions

UpdateBot pushed maven dependency: org.activiti.cloud.common:activiti-cloud-service-common-dependencies to: 7.0.62

/test this
/approve

UpdateBot commands:
updatebot push-version --kind maven org.activiti.cloud.common:activiti-cloud-service-common-dependencies 7.0.62

repo: Activiti/activiti-cloud-connectors


text:
fix(versions): update Activiti/activiti-cloud-dependencies versions into develop

UpdateBot pushed maven dependency: org.activiti.cloud.query:activiti-cloud-query-dependencies to: 7.1.102

UpdateBot commands:
updatebot push-version --kind maven org.activiti.cloud.query:activiti-cloud-query-dependencies 7.1.102

UpdateBot commands:
updatebot push-version --kind maven org.activiti.cloud.rb:activiti-cloud-runtime-bundle-dependencies 7.1.120

UpdateBot commands:
updatebot push-version --kind maven org.activiti.cloud.notifications.graphql:activiti-cloud-notifications-graphql-dependencies 7.1.109

repo: Activiti/activiti-cloud-dependencies


text:
fix(versions): update Activiti/activiti-cloud-full-chart-security versions

UpdateBot pushed helm dependency: activiti-cloud-full-example to: 7.1.143

UpdateBot commands:
updatebot push-version --kind helm activiti-cloud-full-example 7.1.143


text:
fix(versions): update Activiti/activiti-cloud-full-chart-security versions

UpdateBot pushed helm dependency: activiti-cloud-full-example to: 7.1.164

UpdateBot commands:
updatebot push-version --kind helm activiti-cloud-full-example 7.1.164


text:
update org.activiti.dependencies:activiti-dependencies to 7.1.101

UpdateBot pushed maven dependency: org.activiti.dependencies:activiti-dependencies to: 7.1.91

UpdateBot commands:
updatebot push-version --kind maven org.activiti.dependencies:activiti-dependencies 7.1.91 org.activiti.api:activiti-api-dependencies 7.1.50 org.activiti.core.common:activiti-core-common-dependencies 7.1.57 org.activiti.build:activiti-parent 7.1.28

UpdateBot commands:
updatebot push-version --kind maven org.activiti.dependencies:activiti-dependencies 7.1.92 org.activiti.api:activiti-api-dependencies 7.1.50 org.activiti.core.common:activiti-core-common-dependencies 7.1.57 org.activiti.build:activiti-parent 7.1.28

UpdateBot commands:
updatebot push-version --kind maven org.activiti.dependencies:activiti-dependencies 7.1.93 org.activiti.api:activiti-api-dependencies 7.1.50 org.activiti.core.common:activiti-core-common-dependencies 7.1.57 org.activiti.build:activiti-parent 7.1.28

UpdateBot commands:
updatebot push-version --kind maven org.activiti.dependencies:activiti-dependencies 7.1.95 org.activiti.api:activiti-api-dependencies 7.1.50 org.activiti.core.common:activiti-core-common-dependencies 7.1.57 org.activiti.build:activiti-parent 7.1.28

UpdateBot commands:
updatebot push-version --kind maven org.activiti.dependencies:activiti-dependencies 7.1.96 org.activiti.api:activiti-api-dependencies 7.1.50 org.activiti.core.common:activiti-core-common-dependencies 7.1.57 org.activiti.build:activiti-parent 7.1.28

UpdateBot commands:
updatebot push-version --kind maven org.activiti.dependencies:activiti-dependencies 7.1.97 org.activiti.api:activiti-api-dependencies 7.1.50 org.activiti.core.common:activiti-core-common-dependencies 7.1.57 org.activiti.build:activiti-parent 7.1.28

UpdateBot commands:
updatebot push-version --kind maven org.activiti.dependencies:activiti-dependencies 7.1.98 org.activiti.api:activiti-api-dependencies 7.1.50 org.activiti.core.common:activiti-core-common-dependencies 7.1.57 org.activiti.build:activiti-parent 7.1.28

UpdateBot commands:
updatebot push-version --kind maven org.activiti.dependencies:activiti-dependencies 7.1.99 org.activiti.api:activiti-api-dependencies 7.1.50 org.activiti.core.common:activiti-core-common-dependencies 7.1.57 org.activiti.build:activiti-parent 7.1.28

UpdateBot commands:
updatebot push-version --kind maven org.activiti.dependencies:activiti-dependencies 7.1.100 org.activiti.api:activiti-api-dependencies 7.1.50 org.activiti.core.common:activiti-core-common-dependencies 7.1.57 org.activiti.build:activiti-parent 7.1.28

UpdateBot commands:
updatebot push-version --kind maven org.activiti.dependencies:activiti-dependencies 7.1.101 org.activiti.api:activiti-api-dependencies 7.1.50 org.activiti.core.common:activiti-core-common-dependencies 7.1.57 org.activiti.build:activiti-parent 7.1.28

repo: Activiti/activiti-examples


text:
update org.activiti.cloud.dependencies:activiti-cloud-dependencies to 7.1.41

UpdateBot pushed maven dependency: org.activiti.cloud.dependencies:activiti-cloud-dependencies to: 7.1.41

UpdateBot commands:
updatebot push-version --kind maven org.activiti.cloud.dependencies:activiti-cloud-dependencies 7.1.41 --merge false

repo: Activiti/example-cloud-connector


text:
update org.activiti.cloud.dependencies:activiti-cloud-dependencies to 7.1.45

UpdateBot pushed maven dependency: org.activiti.cloud.dependencies:activiti-cloud-dependencies to: 7.1.45

UpdateBot commands:
updatebot push-version --kind maven org.activiti.cloud.dependencies:activiti-cloud-dependencies 7.1.45 --merge false

repo: Activiti/ttc-rb-english-campaign


text:
Incorrect DbWeight for update_job_assignments pallet call method.

Method update_job_assignments contains incorrect weight:
#[pallet::weight(10_000 + T::DbWeight::get().reads_writes(2, 1))]
There are iteration through the updates vector:
The input vec should be used for specifying the weight
So 2 reads and 1 set db per each element for updates Vector.
pub fn update_job_assignments(
    origin: OriginFor<T>,
    updates: Vec<JobAssignmentUpdate<T::AccountId>>,
) -> DispatchResultWithPostInfo {
    let who = ensure_signed(origin)?;
    if !T::JobAssignmentUpdateBarrier::can_update_assigned_jobs(&who, &updates) {
        return Err(Error::<T>::JobAssignmentUpdateNotAllowed)?;
    }
    for update in &updates {
        let job_registration =
            <StoredJobRegistration<T>>::get(&update.job_id.0, &update.job_id.1)
                .ok_or(Error::<T>::JobRegistrationNotFound)?;

        ensure_source_allowed::<T>(&update.assignee, &job_registration)?;

        let mut assignments =
            <StoredJobAssignment<T>>::get(&update.assignee).unwrap_or_default();
        if !assignments.contains(&update.job_id) {
            assignments.push(update.job_id.clone());
        }
        <StoredJobAssignment<T>>::set(&update.assignee, Some(assignments));
    }
    Self::deposit_event(Event::JobAssignmentUpdate(who, updates));
    Ok(().into())
}

There is additional db read inside ensure_source_allowed method:
let attestation =  <StoredAttestation<T>>::get(source).ok_or(Error::<T>::FulfillSourceNotVerified)?;

Thank you for the remark :+1:
There is currently a task going on to benchmark and update the weights of all extrinsic calls.
The update_job_assignments is supposed to be temporary and will be removed in the future.

The benchmarks and the respective call weights were added to acurast pallet.
Closing the issue.

Amazing!

repo: Acurast/acurast-core


text:
Redmine 2.x Update

I coded the following patch to enable redmine_http_auth to be used with Redmine 2.x (at work, with an Apache 2.x + mod_ldap setup). Tested on Redmine 2.1.2 with Rails 3.2.8
The patch itself :
http://www.yomi.darkbsd.org/~darksoul/redmine_http_auth-for-redmine-2x.patch
The patched archive (from master branch) :
http://www.yomi.darkbsd.org/~darksoul/redmine_http_auth-for-redmine-2x.tar.gz
The patch includes :

Routes remade in Rails 3 style :
Style change : Routes.draw |map| ... end -> RedmineApp::Application.routes. draw do match ... end
Application initialization remade in Rails 3 style
Style change : Dispatcher.to_prepare -> RedmineApp::Application.config.after_initialize
Explicit requires replaced with implicit ones
Created a RedmineHttpAuth module to take advantage of implicit module file inclusion
Class structure : the application controller is now made to include RedmineHttpAuth::HTTPAuthPatch instead of directly including HTTPAuthPatch
Include style : protection from double includes

@ghost I get the same error as you:
NameError (undefined local variable or method `httpauthselfregister_url' for #<WelcomeController:0x000000079849f8>):
  app/controllers/application_controller.rb:100:in `user_setup'

Any idea?

@phlegx Having the same issue ... were you able to resolve it ?

@KingBain Some forks has fix the problem.

authors: KingBain darksoul42 phlegx


text:
update to fix deprecation warnings about style bound properties

Fix the deprecation errors.
Originally the htmlSafe was called and fixed this problem, but calling toString on the result reintroduced the problem.

Sorry to hear that the deprecations are still there.
With this solution, the demo app doesn't run. See https://github.com/Addepar/ember-table-addon/pull/15. We'd need to fix that before merging.

This works for me:
styleString.toString().htmlSafe()
No deprecations and it keeps working :)

@HenryVonfire if you submit that as a PR I'll merge it in.

@HenryVonfire I guess I misunderstood your comment - does the demo app work for you after applying this PR?
Since we've merged ember-table-addon back into ember-table, I'm going to close this PR. I'll be happy to keep discussing if anyone opens a new PR in ember-table.

authors: HenryVonfire azirbel jimnanney

repo: Addepar/ember-table-addon


text:
New issues

New change for testing

add code

repo: Addis-Mendaye/github-slideshow


text:
Adldap-Laravel import user filter

Not sure if this is possible:
I have 700+ users that import when I run the adldap:import command. I would like to only import users associated with a specific group or groups.  When using the '-f' filter on the import command How can i see what options would be available to be filtered upon?

Hi @jbowm2,

When using the '-f' filter on the import command How can i see what options would be available to be filtered upon?

What do you mean by "see what options are available to be filtered upon"? Would you perhaps be able to elaborate?

I am not entirely sure on which groups I need to import and would like to see what groups are available. Can I import all with associated group keys?

Unfortunately this isn't possible at the moment. You'll need to look on your AD server to verify the distinguished name of the group and then enter it in the filter option on the command:
php artisan adldap:import --filter="(memberof=CN=YourGroup,OU=Users,DC=YourDomain,DC=com)"

This will import all users that are members of the group: YourGroup.

Thank you @stevebauman
I wasn't sure what capabilities I would have without having direct access to the AD server to verify the distinguished name.
Thanks, again

No problem! You've given me an idea though. I'm implementing the ability to list users to be imported before the import takes place, and also give the ability to cancel the import afterwards :).

Added:
https://github.com/Adldap2/Adldap2-Laravel/commit/834dee922600d6733ee915cbfee0a4397ab584b2

@stevebauman
I'm sure its an error on my end, but I had my IT dept make a new group to corral the users I need to import called "Sales" i added the filter to the adldap:import like you suggested above and added memberof=CN=Sales and the correct DC domain. returns 0 users?

Are you sure you have the correct distinguished name of the group and it isn't contained in any OU or anything?

Since you're referring to your IT Dept, I'm assuming you're in an enterprise environment. They'll most likely be storing that group inside an OU of some sort.

would the OU be Sales?

No, if it's a group, it would be named regularly like any other object. Usually these groups are contained in OU's (Organizational Units) in Active Directory. So it's distinguished name would be:
cn=Sales,ou=User Groups,dc=corp,dc=acme,dc=org

If you have connectivity to your AD server already, you can easily query for this group in your routes file and then visiting your dev site:
use Adldap\Laravel\Facades\Adldap;

Route::get('/', function () {

    $group = Adldap::search()->groups()->find('Sales');

    dd($group->getDn());

});

@stevebauman
That is amazing!!! Users Imported everything looks Correct

Haha glad you've found it useful!
If you encounter any further issues or need any help, definitely create an issue and I'll be able to assist.


text:
Fresh install but cannot login

Laravel Version: 5.4
Adldap2-Laravel Version: 3.0
PHP Version: 5.6

Description:
I follow step from
Quick Start - From Scratch - https://github.com/Adldap2/Adldap2-Laravel/blob/master/docs/quick-start.md
my ldap connection looks ok because i dd() Adldap::auth(); get true
and Adldap::search()->users()->get(); show all the user information
Adldap::auth()->attempt('username','password'); also ok when i test using correct username it return true and false for wrong username
Problem:
when i tried login using form, it return back to login form, the code looks like not create laravel auth from login post

Hi @shahrinrazmi,
This seems like a configuration issue. Are you trying to login users by their username or email address?
Can you also post your config/adldap.php and config/adldap_auth.php file (with sensitive details removed)?

login by username
i using online ldap test server from this
http://www.forumsys.com/tutorials/integration-how-to/ldap/online-ldap-test-server/
i think the details not sensitive, so i post all the code

config/adldap.php
return [
'connections' => [
'default' => [
'auto_connect' => true,
'connection' => Adldap\Connections\Ldap::class,
'schema' => Adldap\Schemas\OpenLDAP::class,
'connection_settings' => [
'account_prefix' => env('ADLDAP_ACCOUNT_PREFIX', 'uid='),
'account_suffix' => env('ADLDAP_ACCOUNT_SUFFIX', ',dc=example,dc=com'),
'domain_controllers' => ['ldap.forumsys.com'],
'port' => env('ADLDAP_PORT', 389),
'timeout' => env('ADLDAP_TIMEOUT', 5),
'base_dn' => env('ADLDAP_BASEDN', 'dc=example,dc=com'),
'admin_account_prefix' => 'cn=',
'admin_account_suffix' => env('ADLDAP_ADMIN_ACCOUNT_SUFFIX', ',dc=example,dc=com'),
'admin_username' => env('ADLDAP_ADMIN_USERNAME', 'read-only-admin'),
'admin_password' => env('ADLDAP_ADMIN_PASSWORD', 'password'),
'follow_referrals' => false,
'use_ssl' => false,
'use_tls' => false,
],
],
],
];

config/adldap_auth.php
return [
'connection' => env('ADLDAP_CONNECTION', 'default'),
'provider' => Adldap\Laravel\Auth\DatabaseUserProvider::class,
'resolver' => Adldap\Laravel\Auth\Resolver::class,
'importer' => Adldap\Laravel\Auth\Importer::class,
'rules' => [
Adldap\Laravel\Validation\Rules\DenyTrashed::class,
],
'scopes' => [
    Adldap\Laravel\Scopes\UpnScope::class,
],

'usernames' => [
    'ldap' => 'uid',
    'eloquent' => 'username',
],

'login_fallback' => env('ADLDAP_LOGIN_FALLBACK', false),
'password_sync' => env('ADLDAP_PASSWORD_SYNC', true),
'windows_auth_attribute' => ['samaccountname' => 'AUTH_USER'],

'sync_attributes' => [
    'email' => 'mail',
    'name' => 'cn',
],

];

I have the same issue. At localhost (Win10 Machine) works fine but on Ubuntu server have this login loop.

i using windows 10, but cannot login. @renanwilliam, do you just follow the tutorial strictly?
or my config missing something?

Using the login form I can't login even in Windows. I can login just in API using Passport but I was necessary implement the methods findForPassport and validateForPassportPasswordGrant in User model.
    public function findForPassport($username)
    {
        $user = $this->where('email', $username)->first();
        if (!$user) {
            $email_parts = explode("@", $username);
            if (count($email_parts) > 0) {
                $login = $email_parts[0];
                $user = $this->where('username', $login)->first();
            }
        }

        return $user;
    }

    public function validateForPassportPasswordGrant($password)
    {
        if (Adldap::auth()->attempt($this->username . '@' . $this->companyid, $password)) {
            return true;
        }

        return false;
    }

Are you using Passport too? It's looking as a conflict for me.

@shahrinrazmi I found what is my problem. In my case I'm using complete e-mail address to login but I was included this method in my LoginController:
public function username()
{
   return 'username';
}

As I not changed anything in resources/views/auth/login.blade.php the form was searching for the username field and returning a required field error. When the method was deleted from LoginController authentication is working in login form and in Passport Routes

authors: renanwilliam shahrinrazmi stevebauman


text:
[Carousel][Tabs] Remove usage of JCR API in ChildrenEditorServlet

The Node#orderBefore JCR API is used in ChildrenEditorServlet:
https://github.com/Adobe-Marketing-Cloud/aem-core-wcm-components/blob/development/bundles/core/src/main/java/com/adobe/cq/wcm/core/components/sandbox/internal/servlets/ChildrenEditorServlet.java#L100
We should get rid of all the JCR API usages in our code.

Sling does not provide an API to order nodes. I created a Sling issue to fix this:
SLING-7975 - Add a Java API for ordering resources
https://issues.apache.org/jira/browse/SLING-7975
Once such an API is provided, we will update the code.

(CQ-4254209)


text:
Correct small spelling mistake.

Purpose of this pull request
This pull request (PR) ...
Affected pages

...

Links to Magento Open Source code

...

Closing due to inactivity. I made the fix in #171

authors: keharper pseudovar

repo: AdobeDocs/commerce-extensibility


text:
Boot JDK guessing is not smart enough

According to our README, it should be enough to type ./makejdk-any-platform.sh jdk15u to build a JDK, especially after having provisioned a machine with out playbooks. But it seldom is:
https://github.com/AdoptOpenJDK/openjdk-build/blob/e9d3ebfc4c912bafa14ba47cad44c59d09f208e3/sbin/common/common.sh#L157-L182
This guesses the bood JDK by using whatever java points to. This is seldom the right choice and causes weird errors that are hard to diagnose for newcomers like
checking for cmd.exe... /cygdrive/c/Windows/system32/cmd.exe
checking build system type... Invalid configuration `Files/Java/jdk8u265-b01': machine `Files/Java/jdk8u265' not recognized
configure: error: /usr/bin/bash /cygdrive/c/Users/Administrator/openjdk-build/workspace/build/src/make/autoconf/build-aux/config.sub Files/Java/jdk8u265-b01 failed
configure exiting with result code 1
Error: No configurations found for /cygdrive/c/Users/Administrator/openjdk-build/workspace/build/src.
Please run 'bash configure' to create a configuration.
/cygdrive/c/Users/Administrator/openjdk-build/workspace/build/src/make/Init.gmk:124: *** Cannot continue.  Stop.

We know in almost every case (except Dragonwell 8) what version the boot JDK should have. So we should test if the boot JDK has the right version and fail the build (overriding that check should still be possible for experiments). Furthermore, we should investigate what else we could do, for example downloading a boot JDK. There is some code that should do that:
https://github.com/AdoptOpenJDK/openjdk-build/blob/e9d3ebfc4c912bafa14ba47cad44c59d09f208e3/build-farm/platform-specific-configurations/windows.sh#L29
Haven't investigated yet why that isn't triggered.

To clarify why it isn't triggered in your situation, the code you reference in platform-specific-configurations is only triggered via our jenkins pipelines which run build-farm/make-adopt-build-farm.sh which ultimately runs makejdk-any-platform.sh, but not when that is run in isolation

authors: aahlenst sxa

repo: AdoptOpenJDK/openjdk-build