text:
single line comment will cause other line turns to grey

As title
Before:

After:

Sample Code below

Sample Code:
variable A B C : Set

data ℕ : Set where
  zero : ℕ
  suc  : ℕ → ℕ

{-# BUILTIN NATURAL ℕ #-}

infix 2 _≡_
infix 3 _+_
infix 4 _*_
infix 5 _^_
infix 6 _!

_+_ : ℕ → ℕ → ℕ
zero + y = y
suc x + y = suc (x + y)
{-# BUILTIN NATPLUS _+_ #-}

_*_ : ℕ → ℕ → ℕ
zero * y = zero
suc x * y = y + (x * y)
{-# BUILTIN NATTIMES _*_ #-}

_^_ : ℕ → ℕ → ℕ
x ^ zero = 1
x ^ suc y = x * (x ^ y)

_! : ℕ → ℕ
zero ! = 1
suc x ! = (x + 1) * (x !)

data _≡_ {A : Set} : A → A → Set where
  refl : {x : A} → x ≡ x
{-# BUILTIN EQUALITY _≡_  #-}

cong : (f : A → B) {x y : A} → x ≡ y → f x ≡ f y
cong f refl = refl

sym : {x y : A} → x ≡ y → y ≡ x
sym refl = refl

trans : {x y z : A} → x ≡ y → y ≡ z → x ≡ z
trans refl refl = refl

begin_ : {x y : A} → x ≡ y → x ≡ y
begin eq = eq

_≡⟨_⟩_ : (x {y z} : A) → x ≡ y → y ≡ z → x ≡ z
x ≡⟨ refl ⟩ y≡z = y≡z

_∎ : (x : A) → x ≡ x
x ∎ = refl

infix  1 begin_
infixr 2 _≡⟨_⟩_
infix  3 _∎

data _∣_ : ℕ → ℕ → Set where
  divideBy : {a b : ℕ} → (q : ℕ) → ( a ≡ (b * q) ) → b ∣ a

5|10 : 5 ∣ 10
5|10 = divideBy 2 refl

+-zero : (a : ℕ) → a + zero ≡ a
+-zero zero = refl
+-zero (suc a) = cong suc (+-zero a)

+-suc : (a b : ℕ) → (a + suc b) ≡ (suc (a + b))
+-suc zero b = refl
+-suc (suc a) b = cong suc (+-suc a b)

+-assoc : (a b c : ℕ) → (a + (b + c)) ≡ ((a + b) + c)
+-assoc zero b c = refl
+-assoc (suc a) b c = cong suc (+-assoc a b c)

-- test

Sorry for the late response.
But I'm having trouble reproducing this issue.

Am i missing something?

Problem solved after I turned the EOL from CRLF to LF.

repo: banacorn/agda-mode-vscode


text:
图片选择界面 导航栏 添加其他按钮

咨询前必看
请先回答下列三个问题，否则不允处理，谢谢配合。
1、我最新的Demo是否有这个bug？【如果Demo没问题，请升级新版】
答：没有
2、你用的是什么版本？升级到最新版后是否正常？
答：正常
3、是否有改动过我库内部的代码？【如有，请说明改动点】
答：没有
图片选择页面的导航栏如何实现 图片中的效果

我直接设置 self.imagePickerVc.navigationItem.rightBarButtonItems = xxx
不生效

库里面会在viewDidLoad的时候设置navigationItem.rightBarButtonItem，你可以setTimeout一下试试，或者hook TZPhotoPickerController 的viewWillAppear

如有其它需要库里开口子配合的地方，可以提个PR

source: gharchive/issue

authors: banchichen shichuanyang

repo: banchichen/TZImagePickerController

license: MIT

license_type: permissive

license_source: github-api


text:
Gulp, JSHint, and JSCS

This pull request includes Gulp and enforces Bandwidth's style guidelines for JavaScript using JSHint and JSCS. This PR will resolve https://github.com/bandwidthcom/node-bandwidth/issues/18
I changed a few of the style and hint rules as detailed here:

Removed disallowQuotedKeysInObjects

This is required by this specific project for setting some field names that are not valid JavaScript variable names, such as certain HTTP headers

Changed maximum argument count to 5
Removed rule to disallow ++

Because the code coverage threshold is set to 100% Gulp will fail the build until we increase test coverage.
For code review, the first commit in this PR contains all the important changes, including the gulpfile and style rules. The second commit simply updates all the code files to follow those rules.

This PR has been open for a week so I am going to merge it.

source: gharchive/pull-request

repo: bandwidthcom/node-bandwidth

license: mit

license_source: bigquery


text:
Appwrite: bundle endpoint

Summary

Experiement for /bundle endpoint using appwrite with webpack Node runtime
Requirements

Webpack config file
/packages folder for the un-bundled files
Prior-merge: should create new /bundle folder

Reference
N/A

Update

Current webpack implementation is running on local but not on production, webpack not found


text:
Merge from upstream

Summary by RedRover
Release Notes

New Feature: Introduced a new GitHub Actions workflow for generating standards.
Enhancement: Updated the README and custom prompt with information about RedRover.
Refactor: Modified various parameters, functions, and options in action.yml, src/commenter.ts, src/main.ts, and src/options.ts.
Chore: Updated dependencies in package.json and red_rover/requirements.txt.
New Feature: Added functionality to identify and summarize design patterns in red_rover/generate_standards.py and src/bot.ts.
New Feature: Added support for 'gpt-3.5-turbo-16k' model in src/limits.ts.
Refactor: Improved rate limit handling in src/octokit.ts.
Refactor: Changed the ASK_BOT variable value in src/review-comment.ts.

🎉 Here's to the code that we've woven, 🐾
With changes as swift as a rover. 🚀
New features abound, improvements all round, 🌟
Like a faithful dog, always rolling over. 🐶
So let's celebrate this PR, with a hearty "Hurrah!" 🥳
For our codebase now shines, like a bright, guiding star. ⭐

:dog: RedRover
High-Level Summary:
This pull request introduces significant changes to the repository, including the addition of new workflows, modifications to existing ones, updates to dependencies, and enhancements to the functionality of the bot. The changes aim to improve the bot's performance, introduce new features like less verbose reviews, and update documentation.

Files
Summary

.github/workflows/redrover_generate_standards.yml, .github/workflows/redrover_review.yml
Addition and modification of GitHub Actions workflows, introducing new workflows and changing repository behavior.

README.md, red_rover/custom_prompt.txt
Updates to README with information about RedRover and modifications to red_rover/custom_prompt.txt describing the purpose of @redrover.

action.yml, src/commenter.ts, src/main.ts, src/options.ts
Changes include updates to workflow name and description, input parameters, system message, default values for parameters, file filtering patterns, comment tags, greetings, release notes, and addition of new functions. New options lessVerboseReview and githubConcurrencyLimit are added.

package.json, red_rover/requirements.txt
Updates to several dependencies in both JavaScript and Python environments, with changes to version numbers.

red_rover/generate_standards.py, src/bot.ts
Addition of functionality to identify and summarize design patterns in a GitHub repository and an update to the system message.

src/limits.ts
Support for a new model 'gpt-3.5-turbo-16k' with specific token limits is added.

src/octokit.ts
Changes to the onRateLimit and onSecondaryRateLimit functions in the throttle configuration, introducing additional logic for retrying requests.

src/review-comment.ts
The value of the ASK_BOT variable is changed from '@openai' to '@redrover'.

All changes should be reviewed due to their significant impact on the logic, functionality, or documentation of the code.

Chat with 🐶 RedRover Bot (@redrover)

Reply on review comments left by this bot to ask follow-up questions. A review comment is a comment on a diff or a file.
Invite the bot into a review comment chain by tagging @redrover in a reply.

Code suggestions

The bot may make code suggestions, but please review them carefully before committing since the line number ranges may be misaligned.
You can edit the comment made by the bot and manually tweak the suggestion if it is slightly off.

Ignoring further reviews

Type @redrover: ignore anywhere in the PR description to ignore further reviews from the bot.

Files ignored due to filter (1)
Ignored files

dist/index.js

Files not summarized due to errors (3)
Failed to summarize

package-lock.json (diff tokens exceeds limit)
src/prompts.ts (diff tokens exceeds limit)
src/review.ts (diff tokens exceeds limit)

In the recent run, only the files that changed from the base of the PR and between 09e11fa0f9fa894d9ebaa1a4003a2c64b908452c and 1fbcfc9078254048f6261ee7db699da450c78ffa commits were reviewed.

authors: BankrateBot gwilkes-rv

repo: bankrate/red-rover


text:
provide support for the prometheus operator ServiceMonitor CR

Is your feature request related to a problem? Please describe.
the CoreOS Prometheus Operator is here, offically support it by providing a ServiceMonitor resource in vault-operator
Describe the solution you'd like
a flag to choose between the scrape annotations for prometheus vs a service monitor.
Additional context
You will need two variants, vault 1.0 and vault 1.1.. an example ServiceMonitor resource below:
---
apiVersion: monitoring.coreos.com/v1
kind: ServiceMonitor
metadata:
  labels:
    app: vault
    release: prometheus-operator
  name: prometheus-operator-vault
  namespace: secrets
spec:
  endpoints:
   # this scrapes the statsd exporter in 1.0
    - interval: 30s
      port: prometheus
  # this scrapes the native endpoint in 1.1
    - bearerTokenFile: /etc/prometheus/secrets/vault-prometheus-endpoint-token/vault.token
      interval: 30s
      params:
        format: ['prometheus']
      path: /v1/sys/metrics
      port: api-port
      scheme: https
      tlsConfig:
        caFile: /etc/prometheus/secrets/vault-tls/ca.crt
        certFile: /etc/prometheus/secrets/vault-tls/server.crt
        keyFile: /etc/prometheus/secrets/vault-tls/server.key
        insecureSkipVerify: true
  selector:
    matchLabels:
      app: vault
      vault_cr: vault

This is already supported by the Helm chart - see https://github.com/banzaicloud/banzai-charts/blob/master/vault/templates/prometheus-monitorservice.yaml, will add it to the operator as well.

Fixed by #386

Some notes:
The ServiceMonitor for a Vault CR will be installed only if serviceMonitorEnabled: true is given in the resource manifest, see: https://github.com/banzaicloud/bank-vaults/blob/master/operator/deploy/cr-prometheus.yaml#L49
To make this working together with Prometheus the coreos/prometheus-operator has to be installed which registers the ServiceMonitor CRD (that is why it is not enabled by default, otherwise the vault-operator would fail reconciling the Vault CR).
The linked example Vault CR above configures a very limited Vault policy called metrics which is capable of only HTTP GET-ing the /v1/sys/metrics. The provide a Vault token with this policy for Prometheus is currently the job of the user (there are various tools for that, like the vault-agent), so a token has to be provided to Prometheus to be able to authenticate against this endpoint, how to do that properly/conveniently in an automated way is a matter of discussion.

authors: bonifaido jurgenweber matyix

repo: banzaicloud/bank-vaults


text:
Followup Update jmx exporter to version 0.14 and enable cacheing

Q
A

Bug fix?
no

New feature?
yes

API breaks?
no

Deprecations?
no

Related tickets
fixes #471

License
Apache 2.0

What's in this PR?

This PR is a followup of updating jmx exporter to version 0.14. It allows cacheing for all the rules.

[x] Implementation tested
[x] Error handling code meets the guideline
[x] Logging code meets the guideline
[x] User guide and development docs updated (if needed)

+1 !

authors: amuraru baluchicken

repo: banzaicloud/kafka-operator


text:
修复当开启实体类常量配置时，常量名称优先采用用户自定义的属性名称

该Pull Request关联的Issue
https://github.com/baomidou/generator/issues/164
修改描述
修复当开启实体类常量配置时，常量名称优先采用用户自定义的属性名称
测试用例
运行com.baomidou.mybatisplus.generator.samples.EntityConstantAutoGeneratorTest#main
修复效果的截屏

@q920447939 感谢PR，目前暂不考虑，自己优先自定义模板处理

repo: baomidou/generator


text:
使用动态数据源@DS时，如何通过databaseid调用对应ds的select SQL

当前使用版本(必须填写清楚，否则不予处理)
该问题是怎么引起的？(最新版上已修复的会直接close掉)
重现步骤
报错信息

去那边提

不好意思，刚刚问题还没写完，应该去哪提呢？ @miemieYaho

这还要我来给你? https://github.com/baomidou/dynamic-datasource-spring-boot-starter

收到，感谢回复。本人小白，请谅解~
------------------ 原始邮件 ------------------
发件人: "miemieYaho"notifications@github.com;
发送时间: 2019年5月13日(星期一) 晚上9:02
收件人: "baomidou/mybatis-plus"mybatis-plus@noreply.github.com;
抄送: "～"164558184@qq.com;"Author"author@noreply.github.com;
主题: Re: [baomidou/mybatis-plus] 使用动态数据源@DS时，如何通过databaseid调用对应ds的select SQL (#1132)
这还要我来给你? https://github.com/baomidou/dynamic-datasource-spring-boot-starter
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.

authors: birdle-joe miemieYaho

repo: baomidou/mybatis-plus


text:
MP升级到3.4.3后无法自动引入相关依赖

当前使用版本(必填,否则不予处理)
spring-boot 2.7.16
mybatis-plus-boot-starter 3.4.3
该问题是如何引起的？(确定最新版也有问题再提!!!)
升级前MP版本为3.4.2，maven依赖如图：

升级后MP版本为3.4.3，maven依赖如图：

相关的依赖，比如mybatis-plus、spring-boot-starter-jdbc都无法自动引入
重现步骤(如果有就写完整)
MP升级到最新版本3.5.3.2，还是没有相关依赖

报错信息
升级后，由于缺少的mybatis-plus包，事务注解@Transactional找不到

https://mvnrepository.com/artifact/com.baomidou/mybatis-plus-boot-starter/3.5.3.2

authors: OPbjO miemieYaho


text:
apply添加报错

当前使用版本(3.5.5)
该问题是如何引起的？(确定最新版也有问题再提!!!)
使用apply导致
重现步骤(如果有就写完整)
service代码
@Override
public TableDataInfo selectPageWorkerProjectUserList(SysUserBo user, PageQuery pageQuery) {
Page page = baseMapper.selectPageWorkerProjectUserList(pageQuery.build(), this.buildTaskUserQueryWrapper(user));
return TableDataInfo.build(page);
}
private Wrapper<SysUserBo> buildTaskUserQueryWrapper(SysUserBo user) {
    Map<String, Object> params = user.getParams();
    QueryWrapper<SysUserBo> wrapper = Wrappers.query();
    wrapper.eq("u.del_flag", UserConstants.USER_NORMAL)
        .like(StringUtils.isNotBlank(user.getNickName()), "u.nick_name", user.getNickName())
        .eq(StringUtils.isNotBlank(user.getUserType()), "u.user_type", user.getUserType())
        .like(StringUtils.isNotBlank(user.getPhonenumber()), "u.phonenumber", user.getPhonenumber())
        .between(params.get("beginTime") != null && params.get("endTime") != null,
            "u.create_time", params.get("beginTime"), params.get("endTime"));
    if(user.getProjectIds()!=null){
        wrapper.eq("sup.project_id", user.getProjectIds()[0]);
    }
    if(user.getWorkerIds()!=null) {
        wrapper.apply(user.getWorkerIds() != null, "FIND_IN_SET({0}, worker_ids)", user.getWorkerIds()[0]);
    }
    return wrapper;
}

xml
<select id="selectPageWorkerProjectUserList" resultMap="SysUserResult">
    SELECT u.user_id, u.dept_id, u.nick_name, u.user_name, u.email, u.avatar, u.phonenumber, u.sex,
           u.status, u.del_flag, u.login_ip, u.login_date, u.create_by, u.create_time, u.remark,
           u.id_card AS idCard, u.address, u.birthday, sup.project_id AS currentProjectId, sup.project_name AS currentProjectName, worker_ids, worker_names
    FROM sys_user u LEFT JOIN slm_user_project sup ON sup.user_id = u.user_id AND sup.is_main=1
                    LEFT JOIN (SELECT user_id, GROUP_CONCAT(worker_name) AS worker_names, GROUP_CONCAT(worker_id) AS worker_ids FROM slm_user_worker GROUP BY user_id) worker ON worker.user_id=u.user_id
        ${ew.getCustomSqlSegment}
</select>

当只有 user.getWorkerIds()和非条件user.getProjectIds()条件不满足，sql变成了
SELECT COUNT(*) FROM sys_user u where u.del_flag=0 and FIND_IN_SET(123, worker_ids)
当只有 user.getProjectIds()条件满足和其他条件任意组合，sql是正常的
或者没有任务条件触发，sql也是正常的
报错信息

?

这个图可以看出来加了FIND_IN_SET出问题了

什么问题?

我已经解决掉了，谢谢哈

authors: Conquery miemieYaho


text:
Transition hook for Mobile Unit

Mobile Unit could be extended with a transition hook that stretched over the period before and after DOM movement to control the transition.
A generator would be useful for yielding to the point of movement and resuming thereafter. Here follows pseudo-code example tweening the element between its initial & eventual positions:
const fixed = {position: 'fixed'}

m(Unit, {
  *transition({dom, move}){
    const initial  = dom.getBoundingClientRect()

    yield move
    // dom is now in its new position

    const eventual = dom.getBoundingClientRect()

    dom.animate([
      {…fixed, initial },
      {…fixed, eventual},
    ], options)
  }
},
  m(Video)
)

🤔

Jotted down some tentative thoughts for an idiom of ‘transition layout’ components in the Mithril chatroom.
With regard to ‘document transitions’, and in the context of a Mithril SPA, we’d have all route endpoints resolve to the same reference, whose view would persist across the entire application lifecycle. This ‘transition layout’ would instantiate the Mobile controller for all Units that desire to persist between different pages.
In the case of the example application demonstrated in Googles recent page transitions presentation, the prime example is a video component: there is a video whose UI view model we want to persist between different pages. On a very basic level, we want the user to be visually aware of that particular video which occupies the major part of the layout on page 2, being the same video they presumably clicked on from several in a list displayed on page 1. The page layout will change drastically, and the videos position within it, but according to the users mental model it is the same video. Mobile Unit allows us to structure our view model such that this identity persists; we can keep all component state & the video element itself and move them from one structure to another, even as the old is structure is destroyed and the new created — the *transition API allows us to hook into the moment of movement & describe the transition as we see fit to represent it adequately to the user.

The ‘transition layout’ acts as a conceptual amalgamation of concerns here: in a simple 2 page application it would not be too difficult to conceive of a single component containing the logical expressions which determine whether the same identifiable Unit ought to be rendered as part of structure 1 (the list) or structure 2 (the single-video page).
This cuts across Mithrils canonical route idiom whereby different pages are defined with different endpoints before any common view expressions can be described. Nonetheless, we can invert and later revert the order of control such that routing concerns are forked within the Mobile expression, and the controlled Unit reference is thereby able to be passed in to externally-defined, route-specific components: the author can then stick with the more familiar idiom of distinct view expressions for distinct pages, while still keeping the persistent references necessary to keep state & DOM live & transitionable despite their distinct positions being described in different execution contexts …

repo: barneycarroll/mithril-machine-tools


text:
Not able to specify a lexer

If I try to specify a lexer when sending the request, I get:
Traceback (most recent call last):
  File "application.py", line 30, in <module>
  File "requests/models.py", line 939, in raise_for_status
requests.exceptions.HTTPError: 400 Client Error: Bad Request for url: https://dpaste.de/api/
[29409] Failed to execute script application

Also, it seems that the list of available lexers from the official API page is gone.

@adder46 I don't see what you're doing or how you call the API with your traceback above. It's a traceback of your application, I can only see dpaste returns a HTTP 400 Bad Request.
The documention you linked is deprecated, you find the latest docs on https://docs.elephant.house/dpaste/api.html.

@bartTC This is the source code in question, and I'm trying to do something like:
cat spam.sh | dpaste --lexer bash

Hmm, alright. Just one more thing -- I'm not quite sure why does it work when I pass no lexer value (pass it None)? Does passing None assume that lexer value (arguments["LANGUAGE"]) is python? Thanks.

What means it 'doesn't work'? Whats the response body content? If you pass an invalid lexer name it should tell you that, and give you a list of options.

It seems that we don't understand each other.
I'm just saying that it is possible to pass only the expires (note that arguments["LANGUAGE"] is None here):
$ cat ~/.bashrc | dpaste --expires onetime
<Response [200]>
https://dpaste.de/sPaM

...whereas it's not possible to only pass lexer.
$ cat ~/.bashrc | dpaste --lexer bash
<Response [400]>
Invalid expire choice "bash" given. Valid values are: 3600, 86400, onetime, 604800

It seems that if you want to specify the lexer, you must pass both expires and lexer:
$ cat ~/.bashrc | dpaste --expires onetime --lexer bash
<Response [200]>
https://dpaste.de/eggS

If only expires had a default value, like lexer has.

repo: bartTC/dpaste


text:
NativeType File reject mode 'ug=rwx,o=r'

'ug=rwx,o=r' is a valid mode for file resource that is reject by puppetresources with the error:
Invalid mode size at

What does that even  mean ? It makes sense for chmod because there already is a file, but what is it doing when a file is to be created ? Handle it like it was 000 ?

Is it not the same as '0774' ?

Argh I did read ug+rwx ... In that case it is well defined.

Actually it doesn't mean 0774 because you shouldn't touch the suid bits ...

It's ?774

How does that even work ? https://github.com/puppetlabs/puppet/blob/master/lib/puppet/provider/file/posix.rb#L119-L135

I just pushed a quick fix. Is that ok for your use case ?

I will test it tomorrow. Thanks.
This issue is not really personal this time ;-) I quickly change the ug+rwx to 774. But as it might hurt adoption I believe the fix is welcomed.

Work nicely except that it silently fails (set to 0) when using nonsensical input such as ug=rwx,o=z or zg=rwx

An untested quick fix is on its way !

;-)
I have just noticed that puppet-lint actually forces a fix (by default) to use a 4 digit value:
FIXED: mode should be represented as a 4 digit octal value or symbolic mode on line 35

I have done some more testing. Looks fine. Thanks.

authors: PierreR bartavelle

repo: bartavelle/language-puppet


text:
Clarity box readability issue

As a Grammarly pro user i use their clarity beta feature quite a lot, I love the extension but it makes their suggestions unreadable.

Hello!
Sorry for the late response. I'll fix that.
Please let me know if there are more things to fix.
Thank you!

You're absolutely fine, there was another bug with a similar problem. I'll get back to you later today with details.

Similar issue with variety suggestions highlight making it very difficult to read the word while writing.

Try it now I fixed it.

Perfect! You've made it so I actually use Grammarly to write in, thank you! I did find one more thing.

The done box of things like sentence variety. Sorry if this is nit-picky, I just seem to find more boxes the more I write.

No worries, because you help me making this extension better!

Can you please show me more details on how I could get that box?

Good question XD I can't seem to get it to pop back up. If I manage to see it again I'll make a note and get back to you. For now, don't worry about it XD

Okay, I've managed to get it to pop pretty consistently with conciseness alerts. WHen its trying to get you to cut out non-content words.
If you want a quick sentence to test it with:
After the fall of their mighty empire, these once-proud Romans were ultimately held responsible for the crimes of all of their kin and forced into the chains of servitude for generations as retribution.
and then just delete "of all of their kin" when the alert pops up and it should make the bright white box pop up.

Roger that!

Just fixed that! 😄

XD im not quite sure what happened but all the boxes are back to being white

Damn, got it. I can see that it's not my fault because of the box-shadow is white as well. 😁

Weeiirrd, XD I wish this stuff was easier to tell what broke it

@wiredin1 @AayanTDE Try it now guys, I believe I fixed it. 😊

omg i love you

authors: AayanTDE bartzalewski wiredin1

repo: bartzalewski/grammarly-dark


text:
Any debug options?

Once the proxy receive a particular configuration like:
 docker exec kamal-proxy kamal-proxy deploy app-web --target ... --host ...

I would like to know the proxy's current full configuration.
How can we recheck what's actually set, especially running more services?

docker exec kamal-proxy kamal-proxy list will show you what's currently enabled, though it doesn't show you all the settings and it's not been exposed by the kamal gem yet.

Thank you. I think exposing it would be useful, but closing the issue.

authors: djmb strzibny

repo: basecamp/kamal-proxy


text:
Loading production website gets the "Pow is installed" message

We use pow for development locally and have intermittently been getting an error where people on the same wifi network are having trouble accessing our production website. So going to http://example.dev works fine in hitting the local instance, but going to http://www.example.com gets the "Pow is installed" message, or gets a 404. I've literally never seen this error in 5 years of using pow, but it's been happening with an increasing frequency in the last three months. When I tail the pow logs, I see requests for "www.example.com"
Wed Mar 23 2016 14:25:16 GMT-0700 (PDT)] INFO [192.168.0.18] GET www.example.com /favicon.ico
[Wed Mar 23 2016 14:25:18 GMT-0700 (PDT)] INFO [192.168.0.18] GET www.example.com /
[Wed Mar 23 2016 14:25:18 GMT-0700 (PDT)] INFO [192.168.0.18] GET www.example.com /favicon.ico
When it works for local dev, I get this:
[Wed Mar 23 2016 14:40:01 GMT-0700 (PDT)] INFO [127.0.0.1] GET example.dev /
[Wed Mar 23 2016 14:40:01 GMT-0700 (PDT)] INFO [127.0.0.1] GET example.dev /welcome
The rest of the team is getting frustrated since they rely on the website to get anything done, but I really don't want to move away from Pow. Any ideas (or other debug information I can provide)?

We have had the same thing.  We used to have everyone (especially the non devs) install pow but now we're uninstalling pow from everyone's machine.  With our VPN config or switching wifi, someone will get in a situation where facebook.com hits pow locally and they don't know what's going on.
We just have then turn off wifi and turn it back on (releasing dhcp will do the same).  I assume it's dns cache.  I don't have anything other than these useless anecdotes to add unfortunately.

authors: squarism waynn

repo: basecamp/pow


text:
🛑 web is down

In dcc986a, web (https://sitedoctors-in.stackstaging.com/wp) was down:

HTTP code: 403
Response time: 653 ms

Resolved: web is back up in db6051f after 14 minutes.

repo: basharatreyaz/uptimebot


text:
Lights won't dim

I have Adaptive Lighting installed and working, but the lights won't dim.  They stay at whatever brightness is set as the max brightness in AL.  I'm using a dozen KAUF wifi bulbs that I haven't had any issues with outside of this.  I have tried various combinations of "separate turn on commands" and toggling the change brightness and change color switches to no avail.

Same problem here.
I manually set the sunset at 18:00:00 and the brightness at 1%,
But it only dim down to ca 46%.
:-(

I can confirm this issue.
Despite using 1%  the lights are dimmed to 20% instead

Closing due to the broad topic and lack of information in OP. Please post your configuration, and debug logs following the instructions in the readme if you'd like help from others diagnosing your issues.

repo: basnijholt/adaptive-lighting


text:
Forms in v8 - seem to not honor col-x, block vs inline

Hi - thank you for what must have been an enormous amount of work on v8. I'm trying to upgrade.
In the previous docs, it mentioned using col-n to specify width of form fields, but in v8, this no longer seems to work.
<input type="email" name="emailAddress" class="col-4 email-octopus-email-address field rounded-left x-group-item" placeholder="Your Email Address">

Changing field to input works to get some formatting on the field, but, the col-4 no longer works, & the field just spreads out across the page.
In addition, even though I am loading the v7 optional ui-groups css, the x-group-item does not seem to actually group the components.
Is this expected? Do I need to set width simply using, say, inline styles. Also, should the x-group-item still work if I have those styles loaded, e.g:
/* Basscss UI Utility Groups */

.x-group-item { margin-left: -var(--border-width) }
.x-group-item:first-of-type { margin-left: 0 }

.y-group-item { margin-top: -var(--border-width) }
.y-group-item:first-of-type { margin-top: 0 }

.x-group-item:focus,
.y-group-item:focus {
  position: relative;
  z-index: 1;
}

Please advise, thanks!

Another thing I noticed about forms is, the default basscss-forms in vs has them styled as block elements. If I use my v7 markup as is, with v8, it leads to:
Label
[input text field]
[submit]

... but what I want is to have them inlined:
Label [input] [submit]

This worked by default before, but something changed. I tried a number of things like adding the utility classes inline, inline-block and so on, on the various parts of the form and its surrounding divs, but I cannot seem to get it to stop being in block mode. Appreciate any advice here. This is where I'm working on it:

		Footer links, Social icons, Copyright etc

				<form method="post" action="https://emailoctopus.com/lists/11111111-2222-3333-4444-5555555/members/external-add" class="email-octopus-form clearfix inline-block m1">
					Get Email Updates:
					<input type="email" name="emailAddress" class="email-octopus-email-address input bg-teal bg-lighten-1 rounded-left x-group-item" placeholder="Your Email Address">
					<button type="submit" class="btn btn-primary bg-lighten-2 rounded-right x-group-item">Subscribe</button>

						Protecting against bot signups - leave this field blank
						<input type="text" name="hp64dbf442-b268-11e5-b4ea-0a4287b2e8c5" autocomplete="off">

				</form>

Ok, I feel really dull. I aded that col-4 back after restarting my npm start, and it works again. No idea why that would have not been taking hold, but, adding the col-4 back where it was, definitely controls the width of the input text field.

I'm guessing that might've been a source order issue. Let me know if you're still having problems.

Thanks @jxnblk. The x-group-item definitely does not seem to work like it used to, though. Previously it kind of neatly pulled the form elements together, but now they stay separate. I am ignoring it for now, but it would be nice if this would work.

authors: RickCogley jxnblk

repo: basscss/addons


text:
[Security] Bump django from 1.8.18

https://github.com/PatMyron/cloudping/pull/1

Ah, that version of Django requires python3.
"Django 1.11 is the last version to support Python 2.7."
https://travis-ci.org/bastionhost/cloudping/builds/461413925
https://docs.djangoproject.com/en/2.1/faq/install/#what-python-version-can-i-use-with-django

Travis missing Python 3.7:
https://github.com/travis-ci/travis-ci/issues/9815

This project need to be upgraded to django 1.11+ or django2.0+ later ;)

authors: PatMyron iamsk

repo: bastionhost/cloudping


text:
Width and position of dropdown menu

Hello,
is there any way how to set position of opened dropdown? I need to center it along with the ng-select element.
Another thing - i want to set width of menu do different size (base on content), width: 100% set with CSS works fine. Is it right way to to this?
thank you
p.

The dropdown should always have the same width as, and be aligned with the ng-select. So why do you need to center it along with the ng-select element?
You can change the width by changing the CSS width of the ng-select element, that's the right way to do that. The dropdown should then have the same width.

I need different apperance (i'm converting contemporary design based on jquery select2 to ngx), where dropdown has different width than ng-select element (i did this by CSS), but centering couldn't be done the same way - you position it to the left (maybe this default postioning could be changed to center - it will work fine for both same and different widths).
The only option now, is to find raw DOM elements (as angular elements in ng-select components are private and i cannot access them directly in component (ViewChild(SelectComponent)) and resize it after ng-selects positioning.
Do you want this feature or its unwanted?
thank you
ps.

authors: basvandenberg sommcz

repo: basvandenberg/ng-select

license: isc


text:
try ambassador

https://github.com/datawire/ambassador

istio

istio

repo: batazor/shortlink


text:
Create new release for swagger-ui  v2.1.4

https://github.com/swagger-api/swagger-ui/releases/tag/v2.1.4
Fixes many issues including better support for references. Which may fix
#10

They're up to https://github.com/swagger-api/swagger-ui/releases/tag/v2.2.5 now. I'll try and spend a little time to see how hard it'd be to get it upgraded.

authors: SofianO drewish


text:
🛑 SAT - Portal CFDI is down

In b14ee34, SAT - Portal CFDI (https://portalcfdi.facturaelectronica.sat.gob.mx) was down:

HTTP code: 503
Response time: 217 ms

Resolved: SAT - Portal CFDI is back up in efa6909 after 10 minutes.

repo: batnieluyo/sat-monitor


text:
Site audio broken in Google Chrome / Edge

The audio file is being redirected to an https: URL; determine how to work around that.

Setting up a new Azure endpoint to use HTTPS; by tonight's 8pm rebuild, those changes should be effective.

This works once again. (forgot to close this issue; the referenced commit fixed the issue)


text:
Need recipe for JUnit @Suite.SuiteClasses with java_test_suite

Consider the following test suite:
@RunWith(Suite.class)
@Suite.SuiteClasses({
  SomeTest.class,
  AnotherTest.class,
})

public class MyTestSuite {
    // how to support this?
}

All three types are defined in the same Java package, same directory.
With test_suffixes = ["Test.java", "TestSuite.java"] the type MyTestSuite cannot be compiled by Bazel because it has neither access to SomeTest nor AnotherTest.

Maybe the recommendation should be: don't use this.
I think that's ok. Test execution is easier with Bazel and java_test_suite because one could run all or select them on the command line.
This might be just a doc issue.

If you use the java_junit5_test and java_test_suite rules from bazel_config/rules_jvm this is correctly supported. Please read the documentation about how to configure your repo to use the rules correctly in your repo. Please ping any issues you find in that repo rather than here.

The documentation for java_test_suite is pretty terse.
Specifically this is a tricky point for users (with the example above):

With test_suffixes = ["Test.java", "TestSuite.java"] the type MyTestSuite cannot be compiled

You cannot support both with a single java_test_suite macro call.

I never got the Junit5 test suite to compile / run correctly as our use cases never covered it. The closest we got was if you used the Junit @Tag on the tests to define which classes / method you wanted to run with which suite. You could then set the include_tags and exclude_tags on the java_test_suite to select the specific tests.
This would cause a problem if you wanted the same test class in two java_test_suites (presumably with different tags). We solved that by writing a wrapper around the define_test to de-duplicate the names as needed.
It shouldn't be hard to make the JUnit 5 runner support that. It just doesn't at this point.

authors: guw tjoneslo


text:
Fix races on temporary file copies

When two instances of Bazelisk are running concurrently and downloading Bazel, the current code admits a race. This patch fixes the issue by using randomly named files for temporary paths.
For example:
{"error":"could not download Bazel: failed to download bazel: failed to move /root/.bazel_binaries/stripe-bazel/downloads/sha256/09bac5c11165a6ab0b7a90d6937c8bc3fcdcda662ffe0590e87ac2ae0e5e8978/bin/bazel.tmp to /root/.bazel_binaries/stripe-bazel/downloads/sha256/09bac5c11165a6ab0b7a90d6937c8bc3fcdcda662ffe0590e87ac2ae0e5e8978/bin/bazel: rename /root/.bazel_binaries/stripe-bazel/downloads/sha256/09bac5c11165a6ab0b7a90d6937c8bc3fcdcda662ffe0590e87ac2ae0e5e8978/bin/bazel.tmp /root/.bazel_binaries/stripe-bazel/downloads/sha256/09bac5c11165a6ab0b7a90d6937c8bc3fcdcda662ffe0590e87ac2ae0e5e8978/bin/bazel: no such file or directory"}

@fweikert 👋 anything I can help with to get this PR merged?

Thank you!

Hi, when should we expect a release with this change?

Hi, when should we expect a release with this change?

https://github.com/bazelbuild/bazelisk/releases/tag/v1.22.1

Thanks!

repo: bazelbuild/bazelisk


text:
Issues with sysroot cpp compiler and compiler version check.

We recently implemented the crosstool sysroot option which causes the cpp compiler to be pulled down as an external dependency (deb package).  The problem we are running into is that to reference this compiler binary in our cc_wrapper, we have to point to a binary unpacked in our external execroot dir. like so:
CPU_COMPILER = 'external/com_ubuntu_sysroot/usr/bin/clang-3.8'

Our project builds fine except that now the bazel plugin fails to synchronize at this step:
https://github.com/bazelbuild/intellij/blob/d5e4e5073473a4a66d869d40743a08f8846063d5/cpp/src/com/google/idea/blaze/cpp/CompilerVersionCheckerImpl.java#L34
I believe the reason is that it calls the compiler binary with --version with the CWD to be the Workspace root. This is fine in most cases because usually the compiler is referenced with an absolute path (/usr/bin/clang...) but in this case, we are referencing a compiler expanded from a bazel dependency.  I believe the solution is to change this version checker to use the execrootpath rather than workspace root here:
https://github.com/bazelbuild/intellij/blob/d5e4e5073473a4a66d869d40743a08f8846063d5/cpp/src/com/google/idea/blaze/cpp/BlazeConfigurationToolchainResolver.java#L182
See the failure from the idea.log below:

2018-02-13 15:17:21,031 [349719473]   WARN - cpp.CompilerVersionCheckerImpl - Error getting compiler version: "Traceback (most recent call last):
  File "/home/thomas/.cache/bazel/_bazel_thomas/925d2bb91b60f375fce428dfba68c0e1/execroot/xxx/external/xxx/tools/toolchain/cc_wrapper", line 264, in <module>
    sys.exit(main())
  File "/home/thomas/.cache/bazel/_bazel_thomas/925d2bb91b60f375fce428dfba68c0e1/execroot/xxx/external/xxx/tools/toolchain/cc_wrapper", line 242, in main
    return subprocess.call([CPU_COMPILER, "--version"])
  File "/usr/lib/python2.7/subprocess.py", line 523, in call
    return Popen(*popenargs, **kwargs).wait()
  File "/usr/lib/python2.7/subprocess.py", line 711, in __init__
    errread, errwrite)
  File "/usr/lib/python2.7/subprocess.py", line 1343, in _execute_child
    raise child_exception
OSError: [Errno 2] No such file or directory
"

Do you happen to have a public example to test changes against? Thanks!

No, unfortunately not, although I suspect you could reproduce the issue by replacing this binary with a "/bin/pwd" you would see that the working directory is different for bazel compiles vs the version check in the plugin.
https://github.com/tensorflow/tensorflow/blob/b2a0f1c45b2283910548ebd88ee5aaf4b6fc6077/third_party/toolchains/gpus/crosstool/CROSSTOOL#L278

I've committed a fix, which should show up in the next release branch.
CLion also runs the compiler to do some feature detection and dump the predefined macros, so I also changed the following wrapper to switch to the execution root first: https://github.com/bazelbuild/intellij/blob/d5e4e5073473a4a66d869d40743a08f8846063d5/cpp/src/com/google/idea/blaze/cpp/BlazeConfigurationToolchainResolver.java#L296
Thanks!

Changes synced (https://github.com/bazelbuild/intellij/blob/7801ae72aa44d4057f4ff88f9774efc1c81d791e/cpp/src/com/google/idea/blaze/cpp/CompilerVersionCheckerImpl.java#L33)

authors: jvoung tadeegan

repo: bazelbuild/intellij


text:
Add unit tests for TarFile class (for #668)

The unit tests mentioned in #668 .
This PR only covers unit tests for the case which #668 fixed, and a simple test for normalize_path interface. It's far from enough but a good start.

I need Windows environment to reproduce the test failure on Windows platform. I will fix it tomorrow.

I am unavailable for the next week or two, so I can't look deeply until March.
That aside, this seems too complex at first. All you needed for a test case was new test data that could exploit the old problem. Changing the test infrastructure at all seems far too complex.

In this PR, I did:

Move assertTarFileContent() function to helper.py which was in tar_write_test.py, because it could be used in both tar_write_test and my new unit test.
Add two simple tests for build_tar.TarFile, which is test_normalize_path and test_add_tree. The latter is to exploit the problem in #668.
Add new test data for the unit test test_add_tree, and append the information of the new file to glob_for_texts_manifest.golden, which is necessary. The new test data is for the case there is a folder to add and can improve the test coverage for test_add_tree.

To make the test simpler, what I can do is:

Remove the test for test_normalize_path.
Remove the new test data.

I'm glad to make those changes later. However, the PR may still look complex after that.

I have removed the test for normalize_path.
However, it seems when the test was running in Windows environment, all files under testdata were copied to runfiles.Create().Rlocation("rules_pkg/tests/testdata/") although I just declared data = [ "//tests:testdata/hello.txt" ] in the BUILD file. See error logs: https://buildkite.com/bazel/rules-pkg/builds/2248 .
I don't know how to fix that, so I still need a subdirectory and test files in this directory. The directory is named build_tar now.
Any ideas for removing the build_tar subdirectory would be appreciated.

I'm OOO this week. Will look next week.
On Wed, Feb 22, 2023, 2:35 AM Qingyu Sui @.***> wrote:

I have removed the test for normalize_path.
However, it seems when the test was running in Windows environment, all
files under testdata were copied to
runfiles.Create().Rlocation("rules_pkg/tests/testdata/") although I just
declared data = [ "//tests:testdata/hello.txt" ] in the BUILD file. See
error logs: https://buildkite.com/bazel/rules-pkg/builds/2248 .
I don't know how to fix that, so I still need a subdirectory and test
files in this directory. The directory is named build_tar now.
Any ideas for removing the build_tar subdirectory would be appreciated.
—
Reply to this email directly, view it on GitHub
https://github.com/bazelbuild/rules_pkg/pull/675#issuecomment-1439507826,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAXHHHFR5UU4GQGS2BUUNHLWYWXRRANCNFSM6AAAAAAU7H2V3U
.
You are receiving this because your review was requested.Message ID:
@.***>

I think this is not an effective test. It sets up the bad condition within the test harness instead of creating the problem situation with normal inputs such as pkg_files targets.
I am presuming you were motivated to #668 because you had a real situation where it occurred. Can you produce a scaled down version of that input, and the test is to simply built the tar file from that.

@aiuto Unfortunately we could never reproduce the issue with pkg_files-like targets.
It's because, in build_tar.py, there is a file_attributes() function which always generates a non-empty mode. See this link.
In our case, we made some modifications in this rules_pkg repo instead of just calling the targets. We called add_tree() from our code without calling main() and add_manifest_entry() -- that's the only way to reproduce this issue, as I did in the test.
However, I think we still need to fix this issue. That's because we may call add_tree() elsewhere someday in the future, and we should never assume that piece of code can generate a non-empty mode as file_attributes() did. After all, the default value of mode is None in the declaration of add_tree().

authors: aiuto sqybi


text:
The address of zlib is invalid

the address of zlib: https://zlib.net/zlib-1.2.11.tar.gz  is invalid, it can be modified to http://github.com/madler/zlib/archive/v1.2.11.tar.gz, also the value of sha256 should be changed.

my fault


text:
Use consistent way of expressing external deps

Make all the deps use the bind targets.
My motivation for this is that we use:
https://github.com/johnynek/bazel-deps
to specify our maven dependencies and deal with updating the transitive versions at stripe. We'd rather pull all the jars just from maven than from the sdk. With the recent changes, that is now possible since we don't use the scala repl script any longer.

@ittaiz what do you think?

Is there any way we can have the bulk of the scala_repositories code in its own bzl files?
It might help cut down on how long scala.bzl is getting. My critique is that scala_repositories is nominally about setting up a WORKSPACE whereas the other functions here are called in a BUILD file.

@sdtwigg I think that is fine, but I might add a function to redirect to it so we don't break people who bump up their versions.
Would you like to see that in this PR?

Redirect function definitely makes sense.
Up to you on this PR or new PR.

Yes.
On Wed, Apr 19, 2017 at 20:20 Ittai Zeidman notifications@github.com
wrote:

@ittaiz commented on this pull request.
In scala/scala.bzl
https://github.com/bazelbuild/rules_scala/pull/178#discussion_r112373446
:

@@ -755,29 +755,26 @@ def scala_repositories():
)

native.maven_jar(

name = "scalac_rules_guava",

was this just unused?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/bazelbuild/rules_scala/pull/178#pullrequestreview-33682295,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAEJdo-jLohcDe3Q85smlC9YpcGkEW8qks5rxvkRgaJpZM4NCB_e
.

@johnynek  LGTM. I agree with @sdtwigg about breaking out scala_repositories and it's definitely up to you if it's here or in a new PR

authors: ittaiz johnynek sdtwigg


text:
[Purge] : How to purge specific tenant ?

Hello,
I want to know if i edit « connection.py » to edit tag « Netbox-synced » and « NetBox-Orphanned ».
Do you think when I purge one tenant it’s will no have impact on the other tenant ?!
I have more than 20 tenant and I don’t wanna purge all. It’s possible ?
Thank you.

I don't really know what you want to do here.
It is not possible to purge specific tenants.
What is your scenario? Maybe we can come up with a different solution.

repo: bb-Ricardo/netbox-sync


text:
Have redundant return properly handle empty when block

This cop was causing NoMethodError: undefined method 'type' for nil:NilClass if the code contained an empty when block. This fixes the issue by ignoring empty when blocks as they won't contain a return.

👍

repo: bbatsov/rubocop


text:
Allow predicates for Style/SafeNavigation

# It seems to looks like it returns boolean.
# In fact it returns one of [nil, true, false].
user&.admin?

# It clearly returns nil
user && user.admin?

Replace this text with a summary of the changes in your PR.
The more detailed you are, the better.

Before submitting the PR make sure the following are checked:

[x] Wrote good commit messages.
[ ] Commit message starts with [Fix #issue-number] (if the related issue exists).
[x] Used the same coding conventions as the rest of the project.
[x] Feature branch is up-to-date with master (if not - rebase it).
[x] Squashed related commits together.
[x] Added tests.
[x] Added an entry to the Changelog if the new code introduces user-observable changes. See changelog entry format.
[x] All tests(rake spec) are passing.
[x] The new code doesn't generate RuboCop offenses that are checked by rake internal_investigation.
[x] The PR relates to only one subject with a clear title
and description in grammatically correct, complete sentences.
[x] Updated cop documentation with rake generate_cops_documentation (required only when you've added a new cop or changed the configuration/documentation of an existing cop).

I disagree with the rationale behind this proposal:
# It seems to looks like it returns boolean.
# In fact it returns one of [nil, true, false].
user&.admin?

# It clearly returns nil
user && user.admin?

Both those snippets clearly return nil or a boolean. I see no advantage in the adding a special case for the latter and just introducing some artificial difference when there is none.

I think it is too shorthand and people will overlook &.
But I can understand that this proposal is not accepted.
Thanks for review.

authors: bbatsov onk


text:
CLEANUP - Util.rb

This PR shaves 100 lines out of util.rb.  Each commit contains a description of why a method removal works. This was inspired by @Drenmi 's comment -
https://github.com/bbatsov/rubocop/pull/5066#discussion_r153192001 , and a convo about the need to start shaving down on how many things our God-like Cop class is responsible for -
https://github.com/bbatsov/rubocop/pull/5323#issuecomment-354133681 .
Every commit should have a green CI run. So if there is a method we'd like to keep in Util, I can drop that commit.

[x] Wrote good commit messages.
[x] Feature branch is up-to-date with master (if not - rebase it).
[x] Squashed related commits together.
[x] The PR relates to only one subject with a clear title
and description in grammatically correct, complete sentences.
[x] Run rake default or rake parallel. It executes all tests and RuboCop for itself, and generates the documentation.

Apart from my single small remark, the rest of the changes seem solid.

Ops. My bad.
On Sat, Dec 30, 2017 at 12:43 Garett Arrowood notifications@github.com
wrote:

@garettarrowood commented on this pull request.
In lib/rubocop/ast/node.rb
https://github.com/bbatsov/rubocop/pull/5354#discussion_r159120646:

@@ -453,6 +453,10 @@ def binary_operation?
source_range.begin_pos != loc.selector.begin_pos
end

 def parenthesized_call?

Check out the description:
Util#parenthesized_call? is only invoked in the 2 files seen in the diff
below. While the method did have an argument named 'send', it is used with
other types of nodes as well. However, in all places #parenthesized_call?
is used, there are guard clauses that guarantee it is only passed a node
and not anything else. Since the method operates solely on Node, it is
Node's and not a Cop's responsible.
I moved it to SendNode originally, but the test suite calls it on other
types of nodes. Right here
https://github.com/bbatsov/rubocop/pull/5354/commits/3bcfcf558a35c2305de6b47260aabd1beeab87bc#diff-2b41a1cdba20584c980b55084250637cR75
specifically.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/bbatsov/rubocop/pull/5354#discussion_r159120646, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAGVyr65PUqD9aJcJTNFFDTkW--KeR5Hks5tFhPhgaJpZM4RPXEc
.

--
Best Regards,
Bozhidar Batsov
http://www.batsov.com

The cop rename introduced a merge conflict.

Will rebase now :).

Only failure is the Method/Length bug currently on master.  #5364 will resolve it.

authors: bbatsov garettarrowood


text:
Question: How setting method or examples for bbci online with customized function ?

Dear bbci researcher.
My name is Ahn Min-Hee, I am member on mindbrain lab on korea university, in seoul, south korea.
We research & development BMI that is constructed 32 channel EEG of brainproduct co. & customized
robot. We tried implementation for BMI code using bbci_public/online.
First, we search & find a bbci online demo: demo_bbcionline_matlab_feedback.m
Second, we modified its code for our requirement: data acquisition from brain vision recorder,
using our own feature/classification method, and process by our own feedback receiver.
Then, we are hold this work, because we don't know coding or parameter setting methods that
connect with our feature/classification function & our feedback receiver to bbci struct...
We want the calculation using our feature/classification function(and FFT) during every 4 seconds.
(of course, realtime data is received from brain vision recorder. It is successes with above demo)
And receive its result on our feedback receiver.
What shall we does?
Would you send us with some more hint or examples?
Thank you very much.
Best,
tigoum

Dear Ahn Min-Hee,
sorry for the late response. Our toolbox programming day had to be postponed.

"We want the calculation using our feature/classification function(and FFT) during every 4 seconds.
[...] And receive its result on our feedback receiver."
-> A classifier output is generated each time a new data package is received. If you want features and classifier output calculated every 4 seconds, you should set
bbci.source.min_blocklength= 4*1000;

"we don't know coding or parameter setting methods that connect with our feature/classification function & our feedback receiver to bbci struct..."
-> I am not sure what the question is. What kind of feedback would you like to use? (If you generate classifier outputs only every 4s, the continuous cursor control from the demo_bbcionline_matlab_feedback is not appropriate.
-> For testing, you could leave the bbci.feedback empty and just let the output of the classifier be printed on the screen (is bbci.feedback is empty, logging to the screen is done by default).

authors: BenjaminBlankertz tigoum


text:
Can not parse email with blank email address headers

For some incoming emails, it could be with header Return-Path: <>. This is possible for auto-replies emails.
This library always throws MimeMessageParseException when parsing those emails.
related code

How do you propose I check if the value is empty (it's actually an invalid value according to the specs).

I just implemented a simple empty check which checks for null, "" and "<>". Did this for Return-Path as well as Return-Receipt-To and Disposition-Notification-To.
Will be in release 6.0.0.

Released in 6.0.0-rc1!

authors: bbottema pyq

repo: bbottema/simple-java-mail


text:
Return result from run

"Closes" #84. The API could probably be improved and I'm not even sure that's the good approach to let Cucumber::run() return a Result. But it does so!

See https://github.com/bbqsrc/cucumber-rust/issues/84#issuecomment-721998855

authors: bbqsrc nbigaouette

repo: bbqsrc/cucumber-rust


text:
Interpret assembly syntax

WIP
Closes #46

Thread 0
Coords: (0, 0)
Instruction: 0.8:

Thread 0
Coords: (1, 0)
Instruction: TSP 2.0

Thread 0
Coords: (1, 2)
Instruction: PSI #2

Thread 0
Coords: (0, 1)
Instruction: 1.1:       NOP
1.2:    PSI #1
1.3:    PTI
1.4:    TKL
2.0:    PSI #2
2.2:    PSI #2
2.3:    PTI
2.5:    TKL

Something is wrong here!

Input to trilangle -Aawsdf
0.8:	TSP 2.0
1.1:	NOP
1.2:	PSI #1
1.3:	PTI
1.4:	TKL
2.0:	PSI #2
2.2:	PSI #2
2.3:	PTI
2.5:	TKL

Thread 0
Coords: (0, 0)
Instruction:

Thread 0
Coords: (1, 0)
Instruction:

Thread 0
Coords: (1, 2)
Instruction:

Thread 0
Coords: (0, 1)
Instruction:

That's even more wrong


text:
dockerfile tweaks

I've removed a bunch of stuff that seemend superflous and tried to streamline the build.
For maximum perfomance on automated builds, I think we should move to a bcbio base image with all dependencies installed and base the automated build on this image.
That way, the auto-build could just pip install the latest bcbio codebase into the image and start running tests.
Downside here is that the base image would have to be built and pushed manually.

Is the dockerfile in this repo still used then? If not, I think it should be removed to avoid confusion.
M

Thanks, I think we aren't using the Dockerfile here. I'll remove it.

authors: matthdsm roryk

repo: bcbio/bcbio-nextgen


text:
Fossil Resource Potential Classification (FRPC) overlay Colour choice

[ ] Existing colours for High, Medium, Low, Very Low for FRPC BR (bedrock) layers need to stand out more (?increase opacity)
[ ] Dark purple for High is good, but could Medium differ more, and then use more of the "sand" colours for the Low and Very Low?

We will need to instruct users to have FRPC  overlay above Bedrock Geology - turning on both causes FRPC layer colours to be more dull
Primary need is for users to see the outlines of geological units that have high/medium/low potential for impact to fossils. Users also need to generate a map with the geological bedrock names.

FRPC BR has been adjusted from an opacity of 0.4 to 0.6, making it more opaque and visually impacting. Perhaps this scheme resolves the clarity of the medium layer and improves the lower colours? I believe the 'sand' is just the background map and not representative of a bedrock layer as they are all shades of purple and blue. Please let me know if the updates help with differentiating the layers visually.

Yes, looks much better with this change - to my eye, high is now a more solid purple and lower colour are improved

authors: LeannePyle rajpersram

repo: bcgov/BCHeritage


text:
Document upload for dependants with PD

As an applicant,
I want to upload proof of permanent disability for my dependants age 12 and older,
So that they can be included in the assessment process according to SABC policy.
Additional Context

Dependents over 22 (or over 18 if not in postsecondary) with a PD can be included in the family size.
Students with a PD dependant age 12 and older are also eligible for the CSG-FTDEP and/or CSG-PTDEP.
Applicant must provide proof that the PD has been approved for federal or provincial disability assistance.

Acceptance Criteria

[ ]If the student answers "Yes" to "Declared on taxes due to a disability" when adding a dependent, they will be prompted to upload supporting documents.

duplicate

authors: RobinDirks abschwenker

repo: bcgov/SIMS


text:
#1953 - Request an Offering Change - DB Migrations

Created application_offering_change_requests table and application_offering_change_request_status_types enum type

Great start, please take a look at the comments. The other migrations for student assessments will be done as part of the other ticket?

yes @andrewsignori-aot

Great start @ann-aot . Few minor comments.

authors: ann-aot dheepak-aot


text:
Consumer Metrics

There is interest for the API Consumer to be able to see how many calls their credentials (or Application?) are making, and for the Access Manager to see what usage a Consumer is doing.
Using the new Gateway Services metrics UX as inspiration, and the existing data we already get in the Portal, the idea would be to create a component that can be used for both API Consumer and Access Manager, although will depend on what UX is designed for each.

[ ] UX design
[ ] Implementation

One metric that we could investigate for feasibility is a security related one, which would provide some information about what Device/IP their credential is being used from.  This would let the Developer know if the credential is trying to be used from an unknown Device/IP.

The API Provider could use the API usage information to make decisions about revoking consumers that have been inactive for a period of time.  Or as a way of following up with clients if they are not seeing traffic, to ensure they are not having any issues - proactive support.

repo: bcgov/api-services-portal


text:
UI/UX for Application Sharing

As a developer integrating a BC Government API, I want to be able to share an application that I have configured in the API Services Portal with other users, so that others can continue to administer the application in my absence.
Acceptance Criteria

Prototype changes to "Edit application" modal that afford the owner of an application the ability to add or remove one or more additional application owners
Prototype updates to the "My Applications" page so that when the details for an application are shown (disclosure caret is expanded) the owners of the application are displayed next to the Authorized API access and description of the application

Out of scope

User roles (i.e. all owners have the same permissions)

@ikethecoder bump! questions from nathalia...

im meeting with Devs today to discuss whats technically possible from the backend before heading with the mockups.

@ikethecoder bump again -- @nirajCITZ nathalia is going to reach out to you just in case you can help unblock her

I spoke @nirajCITZ and he will run some tests to give advice on what would be the best way to share app ownership. If we can support sharing with more than 1 user at the time, max 2, or once at a time.

Checked keycloak and postgrace db to check how application is maped with the user but it requires some more investigation to verify whether it's feasible to share application with another user

@ikethecoder @Jonesy need you to see what is being called!

@ikethecoder yes chris mentioned that we wont have different roles. Everyone would be "owner" and all have the same rights.
Im not sure what the concept "team" implies and how it would simplify the administration or if its needed.

im reviewing with Aidan some ideas he has around the way we share permission in and org level. We are taking a step back and Aidan is helping me understand the current APS structure and an example of how users interact with the portal on different stages. the outcome will be something similar to a "mental map"

Met with Aidan to understand the API creation process, now im editing the "map". Will be shared when finished. We are having continuation for the App. sharing conversation on Thursday

I met with Aidan today to get feedback on the 'application sharing' designs. As a result of this meeting, I will be doing some modifications to the designs considering:

Ownership will be shared as an invite (an email with a one time use link) that will have certain time (for now it was discussed 2 days) for the recipient to login the APS portal in order to accept it.
We cant make sure the recipient received the email, we can only confirm it was sent
The only validation we can make it's if the value has or not an email format (standard email validation)
We wont show "[application name] updated" success message as we do when the user changes the application name or description. This is because when the invite its sent it doesn't mean they now are owners. they need to accept by logging in. The success message will be more specific (something more like "invites sent"). For now the generic success message for other changes will remain as is.
'Invites' will have 3 states: waiting, accepted, expired. Thinking about adding a re send button.
Nathalia to consider separating the invite sharing from the 'edit application modal'. Probably it will make the interaction easier. Needs to define if sharing ownership is considering an edition to the application, or if its separate. If its separate, then a 3rd option will be added to the "more action" button (the 2 dots)

I met with Aidan, Virgina, James and Josh for the last round of feedback. These were the agreements:

Emails need to be provided in order to send the invitation links. For expired or waiting for response status, we will show the email of the person. When users accept the invitation, we will show the email if we have it. if not we show only the display name.
We will allow deleting an item with the first clic on the delete button. we will add a undo banner similar to gmail or outlook to allow users recover from error without adding more clics to show a modal confirmation
i will create the flow for the recipient who wants to accept the invite

This will continued on sprint 85

@ikethecoder  how the recipient's flow looks like?
I imagine something like: the person receives an email that has a login button/link which take them to the portal. Right after they login i guess there would be a message saying you are now co-owner of application 123. If the user goes to my applications they will see the application there.
I guess they will see exactly the same as the user who invited them, when clicking on "manage application". They will see the list of owners, the option to resend, remove and add more owners.
Can you confirm if this is right? or how this flow would be? I hope I can map that flow this week before I leave :o

Yes, that is correct.  The comment "This will also require an effort to keep track of the invite activity in the gateway activity page. E.g. "Jane Lucas invited jhon@gmail.com to share ownership"" - this is not quite true though as the My Applications does not have anything to do with a Namespace/Gateway, so there is no Activity.  I think we had said that we would display the invitation email on the Manage Ownership dialog so that they are tied together (?)

@ikethecoder when you say "that is correct" you refer to the recipient's flow i mentioned in the comment below right?
About the track in the activity. I dont fully remember but i think we mentioned something about having the track in the activity, but if the application is not related to a gateway then it shouldn't be there. So i will remove this point form the developer notes i left in the figma file.
Yes we said we will keep always the email used for the invite to avoid confusion. First it was said we would take whatever info the invited person used to login (which could be a username or a different email) an accept the invitation. But then i recommended to always keep visible (in the manage ownership modal) the same email used for the invite, EVEN if the person logged in with different email or with username.  Are we on the same page here?

Re point 1 - yes
Re point 3 - will the manage ownership show both the original email and the accepted user's info?  Thinking that over time, updated user info will come from the identity provider (BCSC, IDIR, Github, etc) so that should be visible somewhere.. the original email will be irrelevant over time.

After our collaborative discussion earlier today, we have decided to revert to the original plan mentioned on November 16th: "For expired or waiting for response status, we will show the email of the person. When users accept the invitation, we will show the information we can get when the user clicks on the link and log in, which could be a different email from the invite or a display name. This is because users can receive the invite to a specific email but then accept the invite and loging in with different credentials. If we can get the email the user used to login/accept the invite we will show it. If the user doesnt login with an email then we will show the display name."
During today's discussion, we explored two possible solutions to prevent users from getting confused in case the email to which the invite was sent differs from the information when the invite is accepted:
Add an option somewhere on the 'edit owner' list element to view the original email to which the invite was sent and when.
Introduce an activity page for each application (similar to the one in namespaces) where users can review all activities related to the application sharing, such as who was invited, when, by whom, deletions, etc. While this could be considered a sophisticated feature, it may be highly desired by users, as Aidan mentioned.
Implementing an additional info button for mouse-over to display the email and date when the invite was sent may clutter the element, considering we already have an icon (to indicate the state), a resend button, and a delete button. To maintain a clean interface, my recommendation, for now, is getting the provider identity, and once implemented and in use we can gather feedback from real users to assess:

Whether it is confusing to see the invite identity changing or not.
Whether an activity page would be a desired feature.

Designs are finished. Find the screens and the development notes in the figma file, in the 'ready for development page'. This ticket can be closed or assigned to a developer for implementation :)

is this thing on?

Yes sir!
From: Russell Vinegar @.>
Sent: Thursday, February 22, 2024 1:40 PM
To: bcgov/api-services-portal @.>
Cc: Yeung, Ovey CITZ:EX @.>; Comment @.>
Subject: Re: [bcgov/api-services-portal] UI/UX for Application Sharing (Issue #878)
[EXTERNAL] This email came from an external source. Only open attachments or links that you are expecting from a known sender.
is this thing on?
—
Reply to this email directly, view it on GitHubhttps://github.com/bcgov/api-services-portal/issues/878#issuecomment-1960362429, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ASFIAGZOF343BGUXIVULONTYU63J3AVCNFSM6AAAAAA3SY7HS6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSNRQGM3DENBSHE.
You are receiving this because you commented.Message ID: @.@.>>

RocketChat request from Willem Mulder on May 15 2024 - emerging requirement for SAWSx and DriveBC
Aidan Cope @aidan.cope
Owner
11:28 AM
Hi Willem Mulder, at the moment the only option is for the person replacing you to go through creating an application and requesting new credentials again. But we have completed the UX work on this change ("Application Sharing") and the development work is in our backlog (see https://github.com/bcgov/api-services-portal/issues/127). cc'ing our product owner Virginia Vella for prioritization review.
Willem Mulder
10:54 AM
I am working on the new DriveBC site and as part of that we are pulling in weather data from SAWSx through the gateway. I used my IDIR to login to the API Services portal and created an application + requested access with my account.
My question is, what happens if I were to move to another role and would no longer be managing that API connection? Is there a way for me to share my application with others? Should we be using a service account to set up this connection? What is the best practice?

authors: chrsamp ikethecoder nathaliamorales nirajCITZ odyeung rustyjux veep-22


text:
Callumhaslam map

Updates to MAP credential layer 3 complete.
Please review! Thanks, Bree

please merge to the main branch when complete.

repo: bcgov/bc-vcpedia


text:
Receiving a proof request and offer in succession

In some cases, issuers might want to verify their holders identity before sending them their credentials via VCs. The process currently is connect->request-> home->offer notification.
This ticket is to explore streamlining this flow.
It might be helpful to look into Action Menu Protocol

This is partially addressed by using goal codes. When using aries.vc.issue, the any presentation request received before the issuance will be presented, and the flow ends with the credential acceptance. Might be improved in the future with more granular defined flows.

authors: cvarjao knguyenBC

repo: bcgov/bc-wallet-mobile


text:
BHBC-1007

Overview
Links to jira tickets

BHBC-1007

This PR contains the following changes

added table wldtaxonomic_units

This PR contains the following types of changes

[x] New feature (change which adds functionality)
[ ] Enhancement (improvements to existing functionality)
[ ] Bug fix (change which fixes an issue)
[ ] Misc cleanup / Refactoring / Documentation
[ ] Version change

Checklist
A list of items that are good to consider when making any changes.
Note: this list is not exhaustive, and not all items are always applicable.
General

[x] I have performed a self-review of my own code

Code

[x] New files/classes/functions have appropriately descriptive names and comment blocks to describe their use/behaviour
[x] I have avoided duplicating code when possible, moving re-usable pieces into functions
[x] I have avoided hard-coding values where possible and moved any re-usable constants to a constants file
[x] My code is as flat as possible (avoids deeply nested if/else blocks, promise chains, etc)
[x] My code changes account for null/undefined values and handle errors appropriately
[x] My code uses types/interfaces to help describe values/parameters/etc, help ensure type safety, and improve readability

Style

[x] My code follows the established style conventions
[ ] My code uses native material-ui components/icons/conventions when possible

Documentation

[x] I have commented my code sufficiently, such that an unfamiliar developer could understand my code
[ ] I have added/updated README's and related documentation, as needed

Tests

[ ] I have added/updated unit tests for any code I've added/updated
[ ] I have added/updated the Postman requests/tests to account for any API endpoints I've added/updated

Linting/Formatting

[ ] I have run the linter and fixed any issues, as needed
See the lint commands in package.json
[ ] I have run the formatter and fixed any issues, as needed
See the format commands in package.json

SonarCloud

[ ] I have addressed all SonarCloud Bugs, Vulnerabilities, Security Hotspots, and Code Smells

How Has This Been Tested?
Please describe the tests that you ran to verify your changes.
Screenshots
Please add any relevant UI screenshots, if applicable.

just some merge conflicts to deal with first

authors: cgarrettjones sdevalapurkar

repo: bcgov/biohubbc


text:
As a CIF coordinator, I want to see individual milestones in the TaskList, so that I can see each milestone's individual status and navigate to each easily

Description:
The TaskList should show an overall status for the Milstones (if a milestone is late, it would show late. If the milestones are on track it would show on track).
Each milstone should also be listed in the TaskList individually with a status and a link that will direct the user to the milestones page and directly to that milestone using an anchor.
Wireframe: https://www.figma.com/file/PzOY8RyQnVXnjZJdSv2AqQ/Wireframing_Figma?node-id=1209%3A2663
Acceptance Criteria:
Given I am viewing the TaskList
When there are multiple milestones
Then I can see all milestones with links in the TaskList
Given I am viewing the TaskList
When a milestone is late
Then I can see an overall status of late
Given I am viewing the TaskList
When milestones are on track
Then I can see an overall status of on track
Given I am viewing the TaskList
When I click on a milestone link
Then I am taken directly to that milestone in the milestone page
Development Checklist:

[ ] An overall status for milestones shows whether they are on track or late
[ ] All existing milestones are shown with links in the TaskList
[ ] Clicking a milestone in the TaskList takes the user directly to that milesone
[ ] Meets the DOD

Definition of Ready (Note: If any of these points are not applicable, mark N/A)

[x] User story is included
[ ] User role and type are identified
[x] Acceptance criteria are included
[ ] Wireframes are included (if required)
[ ] Design / Solution is accepted by Product Owner
[ ] Dependencies are identified (technical, business, regulatory/policy)
[ ] Story has been estimated (under 13 pts)

·Definition of Done (Note: If any of these points are not applicable, mark N/A)

[ ] Acceptance criteria are tested by the CI pipeline
[ ] UI meets accessibility requirements
[ ] Configuration changes are documented, documentation and designs are updated
[ ] Passes code peer-review
[ ] Passes QA of Acceptance Criteria with verification in Dev and Test
[ ] Ticket is ready to be merged to main branch
[ ] Can be demoed in Sprint Review
[ ] Bugs or future work cards are identified and created
[ ] Reviewed and approved by Product Owner

Notes:

Hey team! Please add your planning poker estimate with ZenHub @BCerki @dleard @gurjmatharu @matthieu-foucault @mikevespi @Sepehr-Sobhani

@suhafa  does it mean we have one milestone by default? Or does the link become "Milestone reports" in the tasklist, when there are no milestones available?

We decided earlier this week that we don't want default miletstones.

@pbastia @dleard The design has one (empty) milestone by default. Is there a reason why we decided to not have default milestones?

At project creation time, milestones are probably not going to be set right away, so having one by default means attempting to submit the project will be blocked on validation errors until it is removed. Some projects may never even have milestones to enter (projects that do not get approved for funding). Quarterly and Annual reports are not pre-populated with a default report.

Maybe instead we can have a tasklist item that just says "Milestone reports" to mirror what we have for the other reports, and that link would be replaced by the first milestone when it is created?

Good point @dleard didn't realize that constraint. In usability testing sessions, the CIF team clicked on the "Milestone report" many times, and other task list headings to go to that particular page. I wonder if we can keep it clickable even when it has sub-items, especially in the case of forms (quarterly and annual reports too) since all forms are in one page any way (not in the case of Project Overview perhaps where it doesn't have a page of its own, and only has sub-task list items as of now, which could again change in the future)

@suhafa the behaviour of those headers is to expand when needed, to avoid having a large task list when most items have been left untouched. If we changed that behaviour, we'd lose the functionality

authors: dleard pbastia suhafa

repo: bcgov/cas-cif


text:
Convert Mural users from Members to Visitors

Describe the issue
We were using a trial version of Mural (after moving from Miro & Draw.io) & that trial is expiring on Oct 25.
Active members need to be removed & re-invited as "visitors".  These do not need licenses.
Which Sprint Priority is this issue related to?
The 'Milestone' should clearly list Sprint Priorities - which one is this issue related to?
Additional context
Add any other context, attachments or screenshots
Definition of done
Remove members

take over ownership of any member workspaces
Re-invite as visitors

Will pull all 'members' except myself & Stu.
Will wait until we get licensing before inviting team back as visitors.


text:
Mat/Pat - Update the instructions for allowance dates

The instructions for parental leave allowance dates are from when we had start and end dates. We should change it to start date and number of weeks

AC

[ ] Change the points highlighted int he screenshot as follows

If you intend to claim parental leave EI for your entire parental leave, you can enter the start date and number of weeks of your parental leave in the fields below
If both you and your partner intend to claim parental leave EI, i.e. share parental EI, you can enter the start and the number of weeks that you intend to claim parental EI. You can learn more about sharing parental leave EI in the "Parental leave  Leave allowance information" tab

ready for QA

Looks good

looks good

Live in PROD - looks good!

authors: Stella-Archer ayushdamani fazil-ey jerry-ey

repo: bcgov/digital-journeys


text:
TOOL: Forms Version Control and Backup

User Story
As a forms designer, I should be able to download forms from all 3 environments and have a secure place to store all the downloaded forms so that the developed forms have a backup in the program area repo.
Tasks

[x] As a forms designer, the ability to download forms from formsflow

[ ] To finalize the filing structure in the program area repo

[ ] Transfer all forms to the Digital Journeys Program Area Repo

Acceptance Criteria
The forms designer has the ability to download forms from formsflow and have a secure place to back up and store these forms.

All forms from TEST & DEV have been downloaded and saved temporarily in Telework TEAMS channel.

Looked at this ticket on August 10.
Current state is a workaround to back-up current versions. We would like to have a long term version. Not ideal and it is only for front-end.
Simen noted this/some of this work might be included in the road map. Might be worth to reach out for formsflow team.
Iman noted the latest version of FF has added versioning "form mapper" to map. Doesn't map the form.
Our team's requirements include having a timestamped copy and also which version fits with Kamunda.
We will know more after the next upgrade and then we can asses next steps

The team is manually saving copies of versions to the Teams folder.
AOT has this on their roadmap for later this year
CHEFS has this in their version already

authors: Abuchana HasanSifat MeghanStothers


text:
TOOL: "Metrics" dashboard inside the tool

All tickets related to the "Metrics" Dashboard will go in this epic

Dashboard prototype: https://digital-journeys-dev.apps.silver.devops.gov.bc.ca/insights
Metrics: https://digital-journeys.apps.silver.devops.gov.bc.ca/metrics
Digital Journeys
forms-flow-web

#1212 covers this. dont need this epic

authors: HasanSifat Stella-Archer fazil-ey


text:
Mat/Pat - Form validation for Maternity leave screens + additional logic

Birthing Parent Selection

"Are you a birthing parent?" is mandatory

Maternity leave info and Leave length info

no field validation or logic

Start Date

mandatory
must be a valid date (submitter logic)

if before form submission max 9 months in the future from current date, cannot be a past date
if approved:
- if before leave start, max 9 months in the future from current date or at max 1 month in the past
- if after leave start, max 1 month in the past from current date. Cannot be a future date

Number of weeks

mandatory
must be a number
must not be more than 17 weeks
after start date and number of weeks is input, show the info box with start & end dates calculated

Allowance Estimation

main logic in #911
Add info popup that shows instructions on how to calculate basic pay (updated April 12)

Instructions to calculate gross basic pay
In your paystub, ADD all of the line items present in your pay stub from the following list. It is okay if you do not have all of the line items listed below in your pay stub, just add the ones that are present.

Regular Hours Paid Earnings
Salary Protection
Classification Adjustment
Temporary Market Adjustment (AMA Market Adjustment)

Add info popup that shows screenshot of EI estimation screen to help people find where the estimate is Maternity Leave & Parental Leave EI Estimate Popup
~- Add tooltip for additional earnings to be decided~ Not needed, updated april 12

Allowance selection

mandatory

Keep an eye out for timezones @Andrew-Vargas-bcgov
Also refer to the original mat/pat form & logic - maybe things can be reused

need infopop screenshot to add.

After leave starts, can be a date at max 1 month in the past from current date
if editing a submitted form, date can be in the past
Finished researching this. These can be implemented after summary screen created.

On the leave length screen, the dates are calculated even before the value is input. Also, the initial design was to have it as part of an info box. Also info box should appear after the start date is input.

Tested and verified!
The start and end dates appear before the user inputs maternity leave value, like Fazil mentioned. Once the maternity leave field has an input, the 'End date' gets updated.

@Andrew-Vargas-bcgov  @chriscaldwell-psa
Issues found

For consistency lets change the error message for mat leave start date to "You must select a date in the future"

updated validation messages.
Changed allowance selection to mandatory.
added fields for chris to display dates in info boxes.

1 - 4 complete
5. working with Iman on this one. The feature for adding tooltips does not seem to work for number fields.
6. complete

Iman to look into this one

FYI the error message is incorrect. It should use the name of the question "Are you a birthing parent?" instead of the content below.

@fazil-ey This validates in the review section now, so you can skip over it initially and only verify at the end. Whatever is checked on this page or the review page is written to both fields. Is this okay?

Tested:

Birthing section - mandatory ✅

Private Zenhub Image

Maternity leave info and Leave length info - contains no logic ✅

Start date - acceptance criteria met ✅

Private Zenhub Image

Number of weeks - validated, info box appears with calculations ✅

Private Zenhub Image
[Private Zenhub Image]
(https://api.zenhub.com/attachedFiles/eyJfcmFpbHMiOnsibWVzc2FnZSI6IkJBaHBBanc4IiwiZXhwIjpudWxsLCJwdXIiOiJibG9iX2lkIn19--0bb75670e908692bbacb36a2e96a99fbc6153279/image.png)

Allowance estimation - info box for calculating salary and EI estimation does not appear right ❔

Private Zenhub Image

Allowance section - mandatory ✅

Private Zenhub Image

Tested and works as expected. Except Allowance calculator info pop which is split into #1304

authors: Andrew-Vargas-bcgov MeghanStothers ayushdamani fazil-ey


text:
EHPR-93 fix optional field validation error

Refer to EHPR-93

Let's link up to set up prettier on your machine to fix formatting errors as you develop

authors: arranfw sunghwan-park-fw


text:
Formio Custom Validation/Conditional disabled

Formio formbuilder won't work with CSP 'unsafe-eval' disabled, this seems to be a known issue: https://github.com/formio/formio.js/issues/2849
I've currently disabled evals in fornio by setting: Formio.Utils.Evaluator.noeval = true;
but this will cause some of the features to not work (Custom Validation and Custom Conditions), We might need to investigate with other teams if there's a better solution

@jadmsaadaot + @TayaLee8  - hello it's been almost a year since we created this bug. can you please see if this card is still relevant? please reply with a comment on this thread as to what you find

@carlygreenehill not relevant we have no custom validation or custom conditions needs and we do not expect admins to write code in formio code (which is what is needed to do custom conditions and custom validations). I will close it

repo: bcgov/met-public


text:
Nert 536 - Map Popups working with features on view and edit pages

Extended Popup component to hide information that is redundant
Adding new mask information

Just a small observation I see in the MapContainer catch clauses with
console.log('Error getting thumbnail');
I think that should be:
console.error('Error getting thumbnail');

authors: oscar-ip popkinj

repo: bcgov/nert-restoration-tracker


text:
VAULT_TOKEN missing from Dependabot secrets

VAULT_TOKEN is missing from Dependabot secrets, which is causing a break in the pipeline for anything Dependabot generates.
https://github.com/bcgov/nr-forest-client/settings/secrets/dependabot

Token was included as part of the workflow

authors: DerekRoberts paulushcgcj

repo: bcgov/nr-forest-client


text:
feat(fe): Highlight text on Auto Complete options

Thanks for the PR!
Deployments, as required, will be available below:
Any successful deployments (not always required) will be available here
Please create PRs in draft mode.  Mark as ready to enable:

Analysis Workflow

After merge, new images are deployed in:

Merge Workflow

We need to prevent issues when user enters characters like: ( . / and others. In some cases they lead to "SyntaxError: Invalid regular expression". I suggest to install "lodash" and use its escapeRegExp (or if prefered install only that single function with package "lodash.escaperegexp").

We need to prevent issues when user enters characters like: ( . / and others. In some cases they lead to "SyntaxError: Invalid regular expression". I suggest to install "lodash" and use its escapeRegExp (or if prefered install only that single function with package "lodash.escaperegexp").

I did this with a different approach to avoid installing external libraries.


text:
Intake form - advanced mapping component

Description
PADS-160
Advanced mapping features added to Map component
Migration added to add geo_json column to submission table
Model and types modified to include geoJSON
SubmissionIntakeForm updated with advanced map
externalApiService updated with api calls to obtain parcel IDs on polygon draw and nearest occupant on pin drop

Types of changes

Checklist

[x] I have read the CONTRIBUTING doc
[x] I have checked that unit tests pass locally with my changes
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] I have added necessary documentation (if appropriate)

Further comments

Pin drop - We shouldn't allow them to have multiple pin drops. We should allow them to move the location of the pin drop (example: drag and drop the pin to a new location OR just clicking elsewhere will dismiss the initial pin drop and appear at the new location).
We shouldn't allow them to pin drop and draw a polygon on the same map. It should be one or the other.  Thank you!

Pin drop - We shouldn't allow them to have multiple pin drops. We should allow them to move the location of the pin drop (example: drag and drop the pin to a new location OR just clicking elsewhere will dismiss the initial pin drop and appear at the new location).
We shouldn't allow them to pin drop and draw a polygon on the same map. It should be one or the other. Thank you!

done!


text:
Feat/332 replace orchard input with a combobox

Description
Completes #332
Main changes:
Integrate front-end with the new get orchards by vegCode endpoint, and display the options to users in comboboxes, which replaced the old fetch orchard by id flow.
Other changes:

rename OrchardModal to OrchardWarnModal and other variables/functions
stop showing warn modal when adding an orchard
stop displaying the add/delete buttons in form review for orchard step
refine the logic for determining the fetch status of parent-tree-genetic-quality queries
add default behaviour to React Query so we don't have to specify each query to not fetch on window refocus and remount.

  "queries":  {
      "refetchOnMount": false,
      "refetchOnWindowFocus": false,
      "retry": 3
  }

Type of change

[x] New feature (non-breaking change which adds functionality)

How Has This Been Tested?
local test
Checklist

[x] I have read the CONTRIBUTING doc
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes
[x] Any dependent changes have already been accepted and merged

Thanks for the PR!
Any successful deployments (not always required) will be available below.
Backend
Frontend
Oracle-API
Once merged, code will be promoted and handed off to following workflow run.
Main Merge Workflow

Thanks for the PR!
Any successful deployments (not always required) will be available below.
Backend
Frontend
Oracle-API
Once merged, code will be promoted and handed off to following workflow run.
Main Merge Workflow

@mgaseta has approved the PR so I'm bypassing the requirement of needing a code owner's approval

repo: bcgov/nr-spar


text:
ORV2-1379 Update Presentation of PermitType Column in Applications Tables

Description

Provide more consistency between the 3 Applications tables
Display the permit type acronym in the PermitType column (aka. TROS)
Upon hover of the table cell, display a tooltip of the Permit Type's long version (aka. Term Oversize, Term Overweight)
Updated Permit Search Results table to also include the Tooltip for Permit Type

Fixes # (issue)
Type of change
Please delete options that are not relevant.

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

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

[ ] Test A
[ ] Test B

Checklist

[ ] I have read the CONTRIBUTING doc
[ ] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes
[ ] Any dependent changes have already been accepted and merged

Further comments

Thanks for the PR!
Deployments, as required, will be available below:

Frontend
Vehicles
Dops
TPS-Migration

Please create PRs in draft mode.  Mark as ready to enable:

Analysis Workflow

After merge, new images are promoted to:

Merge Workflow

Thanks for the PR!
Deployments, as required, will be available below:

Frontend
Vehicles
Dops
TPS-Migration

Please create PRs in draft mode.  Mark as ready to enable:

Analysis Workflow

After merge, new images are promoted to:

Merge Workflow

Is this change needed for the Permit Search component as well?

good question! We saw the desired acronym in the permit search results during our meeting, but to be consistent I'll follow up with them on the hovering of the permit type name.

Is this change needed for the Permit Search component as well?

good question! We saw the desired acronym in the permit search results during our meeting, but to be consistent I'll follow up with them on the hovering of the permit type name.

Yup, for consistency I've added it to the Permit Search results as well. Good catch.

repo: bcgov/onroutebc


text:
Aligns autocomplete results between V1 and V2 front-ends

Fixes #85
Signed-off-by: Akiff Manji akiff.manji@gmail.com

It's just for development. It's usually pointed to a local instance of Aries-VCR but changing it allows us to test directly against the different API deployments.

repo: bcgov/orgbook-bc-client-vue


text:
Adds revoked parameter to autocomplete

Resolves duplicate named results from displaying in autocomplete. Inactive credentials were filtered out but revoked credentials were not. This aligns with v2 autocomplete functionality.

@amanji, The branch needs to be updated before it can be merged.

authors: WadeBarnes amanji

repo: bcgov/orgbook-on-client


text:
New numeric_spaces filter on Form Validation

Description
The numeric_spaces rule returns true if it gets numbers and spaces.
Use case scenarios
A post code has 1 space in it, ex. 19 012 which causes the numeric filter to fail. In that case the numeric_spaces filter would return true.

There are many rules that would be similar to this one (e.g. somebody may use dashes instead of spaces) and we simply can't add countless rules for every specific use case. You can use regexp_match for that or create your custom rules.

authors: meletisf narfbg

repo: bcit-ci/CodeIgniter


text:
Should define something easy searchable alias for nyc

I'm recently struggling to use this library.
When I search about nyc in Google, of course it reach something related to New york city. That is not problem.
However, even I searched in "nyc code coverage" or "nyc node", most of pages are the pages related to Hackathon or conferences.
Download count of this library is currently growing, so I guess many people will struggle with this problem when many people use this library. Therefore, I suggest to define something alias for nyc.
For example, if you name code name for nyc versions like Android versions, users will write articles with these name, and articles related to nyc will be easier to be found in Google.

I found the easiest way to learn more about a particular package is to use http://npm.im/{PACKAGE_NAME}, which redirects to the npm page for that package. That page usually has a README and links to issue trackers etc.

@LimeStreem I've struggled with this as well, I like the name nyc but it does make the library hard to search for. any thoughts?
Perhaps change the README.md to:

nyc - for coverage

the name nyc, and this project, seems to be getting some traction ... I think we can stick with the short hard to search name at this point.

authors: LimeStreem bcoe novemberborn

repo: bcoe/nyc


text:
How can i check NavigationPane is open in PaneDisplayMode.compact ?

I want to use multiple crossAxisCount in GridView like below. How can i check ?
GridView.builder(
      shrinkWrap: true,
      itemCount: 10,
      gridDelegate: SliverGridDelegateWithFixedCrossAxisCount(
        crossAxisCount: isOpen ? 3 : 4,
      ),
      itemBuilder: (context, index) {
        return itemUI(index);
      },
)

Hi! I checked the API. Using a GlobalKey<NavigationViewState> one can know whether the minimal pane is open, but that won't help you with the compact pane. As of now there seems to be no way of getting this.
Maybe you could change the crossAxisCount based on the space you have available by wrapping your widget with a LayoutBuilder?


text:
console.log causes crashes when used to log some events variables

Describe the bug (Required)
when i try console.log(p.permsName) it just crashed.
To Reproduce (Required)
Steps to reproduce the behavior:
use try statements to prevent crashing maybe...
Expected behavior
should log the content out
Error logs
Exited with code 0xc0000005
Environment

OS: Windows
node.js Version => v18.12.1.
npm Version=> 8.19.2
git Version => I don't use git

Additional context
console.log() doesn't crash normally when using normal text content.

Too less info.

authors: SiddharthDeve karikera

repo: bdsx/bdsx


text:
dark mode

Operating system: manjaro linux
This is:

[ ] a bug
[ x] general feedback
i have low vision

i would love to see a dark theme option inside beaker. i currently use firefox and have it tweaked so its respecting my dark theme and together with a darkreader plugin this works 99% of the time.
beaker is electron app and defaults to a standard GTK or QT white theming?. because its sandboxed frm the os ?
would be nice to see a GTK/QT theme chooser.

After the next major release (0.9) I plan to get this on the roadmap

Duplicate of https://github.com/beakerbrowser/beaker/issues/1035

authors: Ornataweaver iBilley pfrazee

repo: beakerbrowser/beaker


text:
Cannot access context menu in "Create New Hyperdrive" popup

Description
Right clicking on text in the "Create New Hyperdrive" popup does not open a context menu.
To Reproduce
Steps to reproduce the behavior:

Open menu.
Click "+ New Hyperdrive..."
Right click on text field.

Expected behavior
A context menu should drop down allowing copy/paste and spell check correction. Spell check does work in the popup, but the corrections are inaccessible.

OS: Windows 10
Version: 1.0.0-prerelease

Good call, we should fix this

Finally got this one fixed

authors: metanomial pfrazee


text:
Disk usage counts vary btwn OSX and Linux, causing issues with DatArchive quotas

We need to unify the byte-measurements between the platforms.

I changed the logic for this in the beaker-0.8 branch. Should be more consistent now.


text:
search or filter by location

Really appreciate you guys aggregating all these companies.  It would be great if it was possible to search companies by country either through the current text search or via filter options.  Currently the text search doesn't return results based on company.

This has been implemented in https://github.com/beam-community/elixir-companies/pull/627 so I'm closing this issue
Thanks for the request @bradyfontenot :)

nice! I really appreciate it. Thanks.

authors: bradyfontenot tajchumber

repo: beam-community/elixir-companies


text:
Extracting weights from Model Checkpoint & using in TF/Keras

Can anyone tell me how can I extract the weight values from the model_checkpoint.pth.tar and use those in Tensorflow, as pre-trained weights?

I have the same question

authors: debadridtt mihirkhandekar

repo: bearpaw/pytorch-classification


text:
Standardizing CSS Preprocessing for resources

It has come to be that some themes have started to use css preprocessors, notably @TheBITLINK's adapta rewrite is now using Stylus and @SpoopySaitama's new theme is using Less.
My proposal is we decide on a single preprocessor for resources hence making it easier for new projects to be made and code kept in sync (instead of running ported out-of-date code)

I know, this issue was opened in 2016, however...
The only issue with having one set pre-processor is that people may prefer using (for example) Less instead of the set pre-processor which could be Sass.

It's safe to say that Sass should be the single preprocessor, a vast majority uses Sass (source), second closest being LESS which can easily be converted into Sass as the syntaxes are very similar.

Yeah, just read on that. Sass should be it.

Has been almost a year :thinking:

authors: LewisTehMinerz TriggerRimfire zet4

repo: beautiful-discord-community/resources