Returning correct Response Code HTTP 429 when taskQueue reached maxSize

Fixes https://github.com/apache/druid/issues/15380
Description
Currently when we submit a task to druid and number of currently active tasks has already reached (druid.indexer.queue.maxSize) then 500 ISE is thrown as per shown in the screenshot in  https://github.com/apache/druid/issues/15380.
This fix will return HTTP 429 Too Many Requests(with proper error message) instead of 500 ISE, when we submit a task and queueSize has reached.
Tested and attached the screenshot

Release note
Returning correct Response Code HTTP 429 when taskQueue reached maxSize
Key changed/added classes in this PR

TooManyTasksException
TaskQueue
OverlordResource
SeekableStreamSupervisor
TaskQueueTest
KinesisSupervisorTest
TaskLockConfigTest
SeekableStreamSupervisorStateTest

This PR has:

[ ] been self-reviewed.

[ ] using the concurrency checklist (Remove this item if the PR doesn't have any relation to concurrency.)

[ ] added documentation for new or modified features or behaviors.
[ ] a release note entry in the PR description.
[ ] added Javadocs for most classes and all non-trivial methods. Linked related entities via Javadoc links.
[ ] added or updated version, license, or notice information in licenses.yaml
[ ] added comments explaining the "why" and the intent of the code wherever would not be obvious for an unfamiliar reader.
[ ] added unit tests or modified existing tests to cover new code paths, ensuring the threshold for code coverage is met.
[ ] added integration tests.
[ ] been tested in a test Druid cluster.

Thank you @oo007 for your first contribution.

Thank you for all the support @abhishekagarwal87 . Much appreciated :-)


nin operator uses invalid rsql syntax

Made an assumption that it was "=nin=", but in fact it is "=out=".

Fixed by https://github.com/RutledgePaulV/q-builders/commit/7429ca745da14c8e3c591130d0674766cc608d5f


Remove compass-director-gateway-integration from control-plane-integration pipeline

Description

We need to remove compass-director-gateway-integration from control-plane-integration pipeline.
Reasons

This job tests only the Compass components which are not a part of this repo anymore.

Looks it's already removed


Incorrect shebang node --harmony

When I try to run learn-generator I get the following error message:
/usr/bin/env: node --harmony: Aucun fichier ou dossier de ce type

On some systems, only one argument is allowed to be specified on the shebang directive (see Wikipedia).
I mentioned this in issue joyent/node#8338.

Oh la la, need to think about that.

I got the same error:
/usr/bin/env: node --harmony: No such file or directory

Should be fixed in https://github.com/isRuslan/learn-generators/commit/6fd7a5631c86e459b31d0416a55f7de03eba177d as 1.2.4 version.
@noblica @julien-f Can you update the package and check this out?

Hope this solved.

Unfortunatly it does not work because learn-generators.js is not in the current working directory.
I am going to make a PR to solve this issue.

@julien-f Which version of learn-generators do you use?
It was fixed in #16

It was fixed for global install but does not work for local install.
I think my solution is better, I am about to make my PR.


Add experimental new api at tiny-atom/future

action packs
where each pack is of shape { state, actions }
all gets/splits inside actions are scoped to that pack
actions are bound to split and can be called as functions
exposes a new method atom.fuse(actions) for extending the action packs

This is built purely on top of the tiny-atom/core with no modifications.
It demos a very decent default approach to working with tiny-atom. Whilst
keeping tiny-atom/core available for any alternative strategies.
Most of all, this preserves the combined effect/reducer approach of
tiny-atom, it preserves the custom merge strategy and is compatible
with all of the existing batteries log/devtool/react/preact.
Finally, it's backwards compatible, you should be able to utilise this in
an existing project gradually if no custom evolve was used.

See examples/future-example for a full TODO app demo.
But here's the gist of the idea:
module.exports.todo = {
  state: {
    items: [
      'learn tiny-atom',
      'use tiny-atom',
      'star tiny-atom'
    ],
    input: ''
  },
  actions: {
    add: (get, split, item) => {
      const items = get().items.concat([item])
      split({ items, input: '' })
      split.analytics.track({ type: 'added' })
    },
    remove: (get, split, index) => {
      const items = get().items.filter((item, i) => i !== index)
      split({ items })
      split.analytics.track({ type: 'removed' })
    }
  }
}

module.exports.analytics = {
  state: {
    events: []
  },
  actions: {
    track: (get, split, event) => {
      const events = get().events.concat([event])
      split({ events })
    }
  }
}

splits can be used as functions, e.g. split.analytics.track
each action pack is namespaced to it's slice of the state by default
it's possible to escape that via get by get('analytics') or get({ root: true })
it's possible to extend the actions dynamically with atom.fuse({ todo: { state, actions } }

I'm contemplating whether we should make this the main api:
const createAtom = require('tiny-atom')

And move the existing lower level api into:
const createAtom = require('tiny-atom/core')

You can try this with yarn add tiny-atom@beta

Im really liking the split() and split.onboarding() way of calling actions on another part of the state. Why aren’t we doing that for the get as well to keep it consistent, get()/get.analytics() rather than get()/get(‘analytics’)
Sent with GitHawk

Also not a fan of potentially effectively having 2 “cores”, anyone who stumbles onto this and wants to quickly use it has to effectively choose a core before even writing any code
Sent with GitHawk

@KwanMan great suggestion! It's now:
get() // scoped to the action pack
get.todo()
get.hints()

Which is symmetric with:
split({ }) // scoped update
split.todo.add()
split.hints.hide()

I've renamed this from future to orb.
Tiny atom will remain what it is, which is a tiny flexible open state management solution at:
require('tiny-atom')

And if you'd benefit from the namespaced locally scoped get/split composed from multiple action packs, good for your typical UI app - you can also just use the provided tiny-atom wrapper - orb:
require('tiny-atom/orb')

In other words, your plain tiny-atom would be:
const createAtom = require('tiny-atom')

const atom = createAtom(initialState, evolve, options)

But your typical usage might end up being:
const createAtom = require('tiny-atom/orb')
const merge = require('tiny-atom/deep-merge')
const debug = require('tiny-atom/log')

const packs = {
  entities: { state, actions },
  feeds: { state, actions },
  picks: { state, actions }
}
const atom = createAtom(packs, { merge, debug })

This latter form - the orb - is basically a good recommended way to use tiny-atom, it's an opinionated approach - but it's no longer flexible. And therefore the core library require('tiny-atom'), which is flexible, is still the main package entry. It's all shipped in the same package, because tiny-atom is batteries included - you install it, and it will cater all your state management needs (structured orb, open core, logger, devtools, preact, react, merge helper, maybe more in the future).
And as a bonus - orb is created in such a way that you can switch to it from the basic version of tiny-atom in a backwards compatible way.

Renamed to tiny-atom/fusion. Since it's about fusing multiple action packs together. Reads better than orb.

Global - same as it is today.
This whole legacy mode can be turned off with options.strict flag. I think
ill turn strict mode on by default.
On Fri, 9 Mar 2018 at 13:35, Tommy Kwan<EMAIL_ADDRESS>wrote:

@KwanMan commented on this pull request.
In src/orb.js
https://github.com/QubitProducts/tiny-atom/pull/32#discussion_r173450345
:

const cache = {}
const actions = {}
const initialState = createInitialState(state, initialActions, options)
const evolve = createEvolve(actions, cache, options)
const atom = createAtomCore(initialState, evolve, options)
atom.fuse = fuse(atom, actions, cache, options)
atom.fuse(initialActions)
return atom
+}

+function createInitialState (state, actions, options) {

if (options.merge) {
return Object.keys(actions).reduce((state, namespace) => {

 const pack = actions[namespace]

 return typeof pack === 'function' ? state : options.merge(state, { [namespace]: pack.state })

so you mean i can call split.existingAction1()? if so, which part of the
state will that action be working on?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/QubitProducts/tiny-atom/pull/32#discussion_r173450345,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AATzWMy4MX924tUcw7MJrJuHR320v4Ckks5tcoU8gaJpZM4Sil2D
.

Will merge for history via #33. Closing this one.


Automate PPS import

So departments don't have to keep doing this. Weekly? Some setting to control?

I'd say nay. But possible if we do the crazy on/off boarding email

We have a department requesting it be automated so they don't have to assign someone to go and hit PPS update every week.

They have a dept where people are joining them every week?
Really, they should use this for the initial load, then the on boarding process should include adding the person. As any roll can add a person, it doesn't need the DA to do it.

Yeah! Crazy right. Department that has almost 800 people in it. People arrive and leave all the time.

Would need to be opt-in.  Should update existing people, add new people, and not remove missing people.

I'm going to punt on this too until we have a request for this feature but a user (or two)


Change the maintainer of apache_exporter.

Hi Prometheus developers!
I'm not very effective as Apache exporter maintainer since I don't actually use it and don't have enough time to do it properly. Luckily, Pedro (Lusitaniae@) is ready to take over it. Please consider it and, if have no other plans for this exporter, update the documentation. Thank you!

Hi Prometheus people,
Hope to be helpful here!

Note that as the new repo is marked as forked that github code search won't work on it.

Brian, thanks for the note, I didn't think about it.
I added a link to it from the old repo. Pedro, did you consider to re-create repository and re-upload code?

Best would be to transfer ownership of the repository, Github will then
also create a HTTP redirect from the old address.
On Thu, Oct 5, 2017 at 12:34 PM neezgee<EMAIL_ADDRESS>wrote:

Brian, thanks for the note, I didn't think about it.
I added a link to it from the old repo. Pedro, did you consider to
re-create repository and re-upload code?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/prometheus/docs/pull/846#issuecomment-334426611, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAANaCZHWdLaG9fhtpHQrosvDeA2Cg6Iks5spLCbgaJpZM4Pu1RG
.

Pedro, can you rename your repo? github doesn't allow to transfer repository when the repository with the same name exist on your account.

I renamed it, should be good now
On Thu, Oct 5, 2017 at 1:26 PM, neezgee<EMAIL_ADDRESS>wrote:

Pedro, can you rename your repo? github doesn't allow to transfer
repository when the repository with the same name exist on your account.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/prometheus/docs/pull/846#issuecomment-334437103, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AADj1facq6VXlBC9rv5W5EOa4saycg-dks5spLzxgaJpZM4Pu1RG
.

Now it says that "Lusitaniae already has a repository in the neezgee/apache_exporter network". Probably you have to delete it completely.
 If you have other not merged changes, I can merge them too.

I didn't add anything else for now.
Can you please try again Neezgee, I removed the repo.
On Thu, Oct 5, 2017 at 1:46 PM, neezgee<EMAIL_ADDRESS>wrote:

Now it says that "Lusitaniae already has a repository in the
neezgee/apache_exporter network". Probably you have to delete it completely.
 If you have other not merged changes, I
can merge them too.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/prometheus/docs/pull/846#issuecomment-334440930, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AADj1T17ZjwpKPirewQ81EWhADpXi2Dyks5spMGjgaJpZM4Pu1RG
.

All good now,
https://github.com/Lusitaniae/apache_exporter
thanks everyone @neezgee @brian-brazil @grobie


Shadows change in a swippable listview after roll to 606f56221bcb222946033cf24017015a0348b8ce

You can see in this video that the shadows are cut off at the top of the swippable element until after I grab it and try to swipe it:
https://drive.google.com/open?id=0B8MlhDnVmzU6Q1ZHZWIzcFlVaGs

This should be resolved by https://github.com/flutter/flutter/commit/d3022132700bf6e258a3b8dd38f6a0e7cbecaf4d which @tvolkert is working to get released into Google.
FYI @jason-simmons

@tvolkert has the shadow disable (https://github.com/flutter/flutter/commit/d3022132700bf6e258a3b8dd38f6a0e7cbecaf4d) been released to Google yet?

If it has, this can be closed.


Added Bubble Sort Implementation in C++

Added Bubble Sort Implementation in C++
Name - Abhishek Kumar Jha ( diabl0-NEMESIS )

Hi @HarshwardhanPatil07, I Hope you are doing Great!
I have raised my First PR can you please validate this and merge it?
Regards,
Abhishek

@diabl0-NEMESIS you need to do more commit at same time small codes not accepted

@HarshwardhanPatil07
"you need to do more commit at same time small codes not accepted"
Can you please elaborate so that I can do exactly the same, I want to close all 4 PR's needed for hacktober today itself so please tell me what do I need to add?

write codes which are complex and helpful for github community @diabl0-NEMESIS


doc: Update README to reflect latest status update of components.

We should also update README to reflect latest status update of components.

Let me take this.

I believe this should have been done. Please let me know if you need more information in our README.


Get row ID from newly inserted rows?

Is there any way to do this effectively without using SELECT MAX(id)?

You can use last_insert_rowid() to look this up.
http://www.sqlite.org/lang_corefunc.html#last_insert_rowid
http://stackoverflow.com/questions/13648158/android-how-to-use-select-last-insert-rowid


[Refactor] Rename 'allow_async_write_back' to 'enable_async_write_back'

What type of PR is this：

[ ] BugFix
[ ] Feature
[x] Enhancement
[ ] Refactor
[ ] UT
[ ] Doc
[ ] Tool

Which issues of this PR fixes ：

Fixes #
Problem Summary(Required) ：

Checklist:

[ ] I have added test cases for my bug fix or my new feature
[ ] This pr will affect users' behaviors
[ ] This pr needs user documentation (for new or modified features or behaviors)

[ ] I have added documentation for my new feature or new function

Bugfix cherry-pick branch check:

[x] I have checked the version labels which the pr will be auto backported to target branch

[x] 3.0
[ ] 2.5
[ ] 2.4
[ ] 2.3
[ ] 2.2

run starrocks_admit_test

run starrocks_admit_test

[FE PR Coverage Check]
:disappointed: fail : 9 / 17 (52.94%)
file detail

path
covered_line
new_line
coverage
not_covered_line_detail

:large_blue_circle:
com/starrocks/lake/LakeMaterializedView.java
0
2
00.00%
[134, 135]

:large_blue_circle:
com/starrocks/common/proc/PartitionsProcDir.java
0
1
00.00%
[410]

:large_blue_circle:
com/starrocks/lake/backup/LakeRestoreJob.java
0
1
00.00%
[311]

:large_blue_circle:
com/starrocks/server/GlobalStateMgr.java
0
1
00.00%
[2335]

:large_blue_circle:
com/starrocks/service/InformationSchemaDataSource.java
0
2
00.00%
[196, 197]

:large_blue_circle:
com/starrocks/sql/ast/SingleRangePartitionDesc.java
3
4
75.00%
[173]

:large_blue_circle:
com/starrocks/lake/LakeTable.java
2
2
100.00%
[]

:large_blue_circle:
com/starrocks/server/LocalMetastore.java
4
4
100.00%
[]

@Mergifyio backport branch-3.0

run starrocks_admit_test


HTML

#HTML
(User Story)
As a boot camp student,
I want a Prework Study Guide website,
so that I can review all the information I learned throughout Prework.
(Acceptance Criteria)
GIVEN a Prework Study Guide website
WHEN I visit the website in my browser
THEN I see four boxes titled HTML, CSS, Git, and JavaScript with associated notes listed
WHEN I view the study guide
THEN I see a dark blue header and footer, and four boxes with a shadow
WHEN I open the console in my browser
THEN I can see the four topics I learned listed along with a suggestion on what I should study first
WHEN I visit the website using the URL
THEN I can access my website from any browser

Done :)


about FormRequest bail

Laravel Version
10.48
PHP Version
8.2
Database Driver & Version
No response
Description
当password字段的验证，使用 Password::min(8)->mixedCase()->numbers() 的时候，即使增加了 bail 属性，验证信息的提示为，密码 至少为 8 个字符。 (and 1 more error)，如何让这个 and 1 more error 消失呢，我已经在FormRequest的类中增加了 protected $stopOnFirstFailure = true; 这个属性
Steps To Reproduce
 public function rules(): array
    {
        return [
            'username' => [
                'required',
                'bail',
                new MobileRule(),
            ],
            'password' => [
                'required',
                'bail',
                config('user.STRONG_PASSWORD') ? Password::min(8)->mixedCase()->numbers() : Password::min(6),
            ],
        ];
    }

Hi there,
Thanks for reporting the problem you are encountering, but it looks like this is a question which may be better suited for a support channel. We only use this issue tracker for reporting bugs with the library itself. If you have a question on how to use functionality provided by this repository you can try one of the following channels:

Laracasts Forums
Laravel.io Forums
StackOverflow
Discord
Larachat
IRC

However, this issue will not be locked and everyone is still free to discuss solutions to your problem!
Thanks.


"All cloud filtered by OpenShift" overview titles

Updates for the Infrastructure > "All cloud filtered by OpenShift" overview.

The card title should be "All cloud filtered by OpenShift cost"
The chart title should be "All cloud filtered by OpenShift cumulative cost comparison ($USD)"

https://github.com/project-koku/koku-ui/issues/1472

Codecov Report

Merging #1473 into master will not change coverage by %.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #1473   +/-   ##
=======================================
  Coverage   76.31%   76.31%
=======================================
  Files         198      198
  Lines        3555     3555
  Branches      672      672
=======================================
  Hits         2713     2713
  Misses        756      756
  Partials       86       86

Continue to review full report at Codecov.

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


add method for checking whether messaging is supported

This is useful if you want to change the display based on whether or
not the device supports messaging without actually trying to send a
message.

Thanks for this - on first glance without running it up locally it looks fine.
I will get round to checking it properly and merging in the request as soon as I can

checked this out now and all working fine. Nice work thanks :)
Will merge in - I may edit the README a bit after the merge to change ordering of some if the info now we have more than one method supported by the API.

Thanks!


Keep traitSet of logicalValues

Description
https://github.com/apache/druid/pull/11058 added a new rule to handle empty tables, but Druid still uses BindableConvention for certain queries that read empty tables. The queries used in unit tests that are changed in this PR are such queries. The bug is in the traitSet of DruidQueryRel being set to a single trait of Convention.NONE when a DruidQueryRel is created from a LogicalValues. However, LogicalValues sometimes have extra traits set such as RelCollation. This makes VolcanoPlanner think the DruidQueryRel is not related to the LogicalValues even though DruidQueryRel is created from it.
There is another method that changes traitSet when creating DruidQueryRel from LogicalTableScan which doesn't seem necessary as well. I will do some testing to make sure that it's not necessary and make another PR to fix it.

This PR has:

[x] been self-reviewed.

[ ] using the concurrency checklist (Remove this item if the PR doesn't have any relation to concurrency.)

[ ] added documentation for new or modified features or behaviors.
[ ] added Javadocs for most classes and all non-trivial methods. Linked related entities via Javadoc links.
[ ] added or updated version, license, or notice information in licenses.yaml
[x] added comments explaining the "why" and the intent of the code wherever would not be obvious for an unfamiliar reader.
[x] added unit tests or modified existing tests to cover new code paths, ensuring the threshold for code coverage is met.
[ ] added integration tests.
[ ] been tested in a test Druid cluster.

@clintropolis thanks for the review 👍


[FIX,TOPI] Default to inlining fused operations for conv NCHWc int8

Inlining fused operations used to be the default and performs better on x86. I accidentally had turned it off.
@masahi

can you try another job?


No possibility for raw tag displaying

Current behaviour:
It is not possible to pass content: "<number><regex>", to the popover, as they will be translated as html tags and won't be displayed.
Expected: Display raw tags.
P.S. Maybe I've lost some skills to google ? Thx

Hi @defusioner, this is by design (https://github.com/Akryum/v-tooltip#other-options). Try using the following piece of code: { content: '<number><regex>', html: false } to insert the content as innerText instead of innerHTML. That will avoid that the text is being inserted as HTML.

@ivansieder Thank you! I was trying to pass html: false to the initializer when I register the component to Vue. I should have done with defaultHtml: false

This is really bad as a default value. Values should by default escape HTML, this is a security risk.
Frameworks have by default behaviour to escape HTML, and you have to explicitly turn it off if you want to display possible unsafe content.

This is really bad as a default value. Values should by default escape HTML, this is a security risk.
Frameworks have by default behaviour to escape HTML, and you have to explicitly turn it off if you want to display possible unsafe content.


[Event Request] Table 36 "Sales Header" - Function OnInitRecord - OnInitRecordOnBeforeAssigmentPostDescription

Could You please, add the following event to the Table 36 "Sales Header"
Object: Table 36 "Sales Header"
Function: OnInitRecord
Publisher Event: OnInitRecordOnBeforeAssigmentPostDescription
Reason: To implement custom business logic and manage the field "Posting Description" externaly
Where is the event placed (see image attached)?:

Insert next 3 lines
IsHandled := false;
OnInitRecordOnBeforeAssigmentPostDescription(Rec, IsHandled);
if not IsHandled then
Above the line
"Posting Description" := Format("Document Type") + ' ' + "No.";
Event Code:
[IntegrationEvent(false, false)]
local procedure OnInitRecordOnBeforeAssigmentPostDescription(var SalesHeader: Record "Sales Header", var IsHandled: Boolean)
begin
end;
Thanks in Advance

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. We will update this issue with information about availability.


Reduce property acquisition and repeated call methods

Reduce property acquisition and repeated call methods to improve performance

Method
Mean
Error
StdDev
Gen 0
Gen 1
Gen 2
Allocated

Descendants
47.73 ms
0.342 ms
0.320 ms
166.6667
166.6667
166.6667
16 MB

OldDescendants
50.80 ms
0.789 ms
0.699 ms
200.0000
200.0000
200.0000
16 MB

Benchmark code: https://github.com/dotnet-campus/Open-XML-SDK/blob/3f7608a23ba3420d382050c2ca5c7f295a5d381b/test/DocumentFormat.OpenXml.Benchmarks/OpenXmlElementDescendantsTests.cs

Also, can you include the benchmark code?

@twsouthwick  Thank you. According to you, I have changed it.

Just a couple suggestions. Also, can you update the readme with this perf change?

@twsouthwick Do you means the CHANGELOG.md file?

Yup the changelog.md

@twsouthwick Could you help me add the document and English is not my native language

Sure no worries. Thanks for the pr


Ensure we are scanning Python packages correctly with uv

https://github.com/astral-sh/uv
Ensure that the library paths are the same as what they are with pip. The package manager setting in the config would still be pip, but the extension should find the dependencies and their version (when applicable)

Verified.
uv venv
source .venv/bin/activate
uv pip install -r requirements.txt
mv requirements.txt requirements.txt.orig

In VSCode, ran Python: Select Interpreter and chose the workspace .venv, then scanned for requirements. The package versions for installed packages were detected.


Possible fix for issue #20

This works for me, but I can't say I'm an experienced coder, and it might conflict with something you've written (even if it's just one line).


Bump charts and enable worker service auth

Enabled authorization except session-manager. Working fine with my env, but sorry if I miss something.


Update files go-swagger thinks need changes

Summary
When branching off the latest changes in main/master, if I run
rm .server_generate.stamp .swagger_build.stamp && make server_generate

I get 486 files changed. Based on spot checking a few, they seem to be whitespace differences. Interestingly, they seem to be undoing the whitespace differences that #9191 made.
I checked my go-swagger version and it is the latest version:

I saw the thread Drew started about the differences between go-swagger from nix vs brew and I wonder if that's what this is. I'm using nix for my dependencies.
 Ideally we wouldn't have to generate the files and then have them fixed by go fmt though.
This PR has both commits, one just after running the gen command, and one after running the files though go fmt. The net result being the 32 file changes being shown in the PR.

Hadn't meant to open this one 🤦🏼 , #9319 is the one to look at.

Warnings

:warning:
Please add the JIRA issue key to the PR title (e.g. MB-123)

  Generated by :no_entry_sign: dangerJS against 791de81d9de8b20d574bda8d84302567a18753f5


csv file has "=" at the beginning and "" at the end of each field

Date,FirstName
"=""2020-03-17T19:27:47.289771Z""","=""Terry"""
"=""2020-03-17T15:31:52.9888394Z""","=""Terry"""

I came across this also while creating CSV import for Xero and discovered that they made the change without notifying of breaking changes.
Commit Making Changes
There should be a prop for determining if the csv values should be escaped or not

I'm getting this same issue.  Has this been resolved yet?

@jny986 thanks a lot for your PR. It is merged now, I'm updating this component so it will be deployed in the next version. Sorry @DedicatedManagers for the late response.


Line must be greater than or equal to 1, got -1

I tried running the repo, but got
Line must be greater than or equal to 1, got -1

TypeError: Line must be greater than or equal to 1, got -1
    at BasicSourceMapConsumer.SourceMapConsumer_findMapping [as _findMapping] (/Users/.../sveltekit-example/node_modules/vite/dist/node/chunks/dep-e0fe87f8.js:65405:13)
    at BasicSourceMapConsumer.SourceMapConsumer_originalPositionFor [as originalPositionFor] (/Users/.../sveltekit-example/node_modules/vite/dist/node/chunks/dep-e0fe87f8.js:65474:22)
    at /Users/.../sveltekit-example/node_modules/vite/dist/node/chunks/dep-e0fe87f8.js:66413:34
    at String.replace (<anonymous>)
    at /Users/.../sveltekit-example/node_modules/vite/dist/node/chunks/dep-e0fe87f8.js:66403:21
    at Array.map (<anonymous>)
    at ssrRewriteStacktrace (/Users/.../sveltekit-example/node_modules/vite/dist/node/chunks/dep-e0fe87f8.js:66402:10)
    at instantiateModule (/Users/.../sveltekit-example/node_modules/vite/dist/node/chunks/dep-e0fe87f8.js:66547:28)

I am using yarn, if that makes a difference

I think there are still issues with the Firebase 9 SDK - if you pin it to the 9.0.0 version it works.

That worked, thanks!

I had this same issue, which brought me here. I later found Ben McCann's comment in the Firebase SDK queue. Adding the suggested config to svelte.config.js seems to solve the problems with firebase/auth. PR seems hardly worth the trouble but I think this is the workaround for now.
diff --git a/svelte.config.js b/svelte.config.js
index da8701d..29f7dff 100644
--- a/svelte.config.js
+++ b/svelte.config.js
@@ -19,6 +19,9 @@ const config = {
                target: '#svelte',

                vite: {
+      ssr: {
+        external: ['whatwg-url', 'node-fetch']
+      },
                        build: {
                                sourcemap: true,
                        },


Switch to time-warp-nt/master

prosti @gromakovsky

Timeouts are back :(

Timeouts are back :(

Haddock causes them, notice that travis-ci/push doesn't time out, but travis-ci/pr does — the difference between them is whether --haddock is used.

Maybe we can separate haddock as we did it for hlint? Does haddock need anything to be compiled?

As far as I know, Haddock needs the entire project (and deps) to be built with --haddock.

I merged it because currently all travis builds fail because of time limit because of haddock. Everything else is correct.


Feature: modify the TF 2.0 upgrade script to convert .ipynb files.

Currently, the TensorFlow 2.0 upgrade script only supports Python (.py) files, not Jupyter notebooks (.ipynb). This can make converting documentation frustrating and time consuming.
The purpose of this issue is to modify the upgrade script to accept a Jupyter notebook, convert it from TensorFlow 1.x to TF 2.0, and to export out a replacement notebook.
For more information, contact @VikramTiwari, @JerryKurata, and @lc0.

Friendly reminder that @lc0 is amazing and http://tf2up.ml is something you should check out. :slightly_smiling_face:
Closing!

I didn't get it, what to do exactly?


ignore StorageTest [FINERACT-828]

FINERACT-828

@awasum @xurror merge? Unless you're motivated to fix StorageTest... I'm giving up on it! :smile:

I just don't get the thing with the storageTest. Seems to be some inconsistences only funny part is these inconsistences does not happen all the time.

/rebase

I'm merging this


Update Kotlin 1.9.20

https://github.com/JetBrains/kotlin/releases/tag/v1.9.20
https://github.com/google/ksp/releases/tag/1.9.20-1.0.14
https://developer.android.com/jetpack/androidx/releases/compose-compiler#1.5.4

🤯 Wow, 41m 19s for the build step, what's happening...

Enabling Gradle Scans would be very helpful to diagnose these kind of build perf issues:

#413

Fixed in #1117


TS: Type instantiation is excessively deep and possibly infinite.

Error message:
/Users/hadihallak/cra-ts/src/App.tsx
TypeScript error in /Users/hadihallak/cra-ts/src/App.tsx(13,7):
Type instantiation is excessively deep and possibly infinite.  TS2589

    11 |   return (
    12 |

  > 13 |       <Box> Hello world </Box>
       |       ^
    14 |

    15 |   )
    16 | }

Relevant packages:
    "@stitches/styled": "^8.0.2",
    "react": "^16.13.1",
    "react-dom": "^16.13.1",
    "react-scripts": "3.4.1",
    "typescript": "~3.7.2"

Can be reproduced with this code in a CRA TS app:
import React from 'react'

import { createStyled } from '@stitches/styled'

export const { styled, css } = createStyled({})

export const Box = styled.div({})

function App() {
  return (

      <Box> Hello world </Box>

  )
}

export default App

@christianalfoni is this to do with the version of TS?

@hadihallak I can't replicate this, using Typescript v3.9.7 https://codesandbox.io/s/fervent-villani-ehrvy?file=/src/App.tsx

@hadihallak whats the deal with this one?

sorry, this must've gotten burred in all of the notifications. this error used to show up in CRA TS apps so will give it a test today and if it works, will close this one.

Just tested this and it's no longer an issue


WalletConnect sign request prompts with garbled plaintext

When prompted to sign, the data displayed in the actionsheet looks garbled. Shall we display it like "0x796492d582dc3b93f13e9147f395702d10c71c47f5ce3e4a999ef370fa65441e" instead?
This is against the site https://zora.co. The prompt to sign happens after connecting via WalletConnect.
This is from another, but similar signing request from the same site, so the shape of the data is the same, but the data wouldn't match the screenshot:
{
   "id" :<PHONE_NUMBER>753426,
   "jsonrpc" : "2.0",
   "method" : "eth_sign",
   "params" : [
      "0xba829c050941f1f50217a665205ccab4d227b851",
      "0x796492d582dc3b93f13e9147f395702d10c71c47f5ce3e4a999ef370fa65441e"
   ]
}

Nice idea, display the raw hex if it doesn't appear to be readable text. You'd need to check for something like Character.isAlphabetic(ch) || Character.isDigit(ch) || Character.isIdeographic(ch) but also allow for emojis, which are renderable in text. It shouldn't be hard to create a heuristic to detect renderable unicode characters.

@JamesSmartCell, @seabornlee or @justindg would you help re-assign among yourself and close this?


automated test for labor model

looks like the labor model is out of date.
Must not be an automated test for it.

fixed in https://github.com/econ-ark/HARK/pull/729


Module name collides with k6/execution

The name of this module collides with exec from k6/execution.
Attempted workaround (does not work)
import exec from 'k6/execution';
import { exec as execer } from 'k6/x/exec';
console.log(execer.command("pwd"))

Results in
TypeError: Cannot read property 'command' of undefined or null

Hi @sarasensible,
There is no collision - both export are a default ones and you can do a default import and when doing that you can name them w/e.
import exec from 'k6/execution'andimport something from 'k6/execution'will just bind the same thing toexecorsomething`. In both cases the module has no way of knowing or caring how the default export it provided is imported.
So in your case you can do import execer from 'k6/x/exec' or if you want you can also do import { default as execer } from 'k6/x/exec'.
If you only need the command import { command } from 'k6/x/exec' and then use it directly as command instead of execer.command.
I will be closing this issue, but if you have more question you might try the community forum

Yes both options you gave work for me.  Thanks!


Cydia sources refresh causing freeze and device restart

IOS 12.1.2 IPhone X
While refreshing sources in Cydia, the device completely locks up and reboots into an unjailbroken state.

I have the same issue. On iPhone X 12.1.2. Installed jailbreak around 3:30PM. Issue occurred around 8:50PM. Opened Cydia (as I've done many times today). Sources were refreshing (but taking longer than usual). Took about 2-3 minutes before screen went completely black. Swiping nor buttons did anything. Waited about another 2-3 minutes and phone rebooted normally.

@pwn20wndstuff @sbingner Alos seen this issue discussed on discord a few times.

confirmed，iphone x，12.1.1 with b30

Same thing on 12.1.2 iPhone X with b36

This seems to be very rare, but I have personally experienced this before.

@Chasewhip8 @Samgisaninja I experienced this twice and I'm even on iOS 11.3.1(iPhone 6s). This has also happened in the Settings app so it seems like it is substrate related.

iOS 12.1.1 iPhone X, phone freezes then reboots, this happens random times when opening Cydia and sources refresh, or when adding or removing a source. This happened with every release, even now on the last beta happens too. With or without tweaks.
Everytime with this Crash log:
Date: 2019-03-01 10:38:00 +0000
Bundle id: com.apple.springboard
Exception type: NSUnknownKeyException
Reason: [<SBDashBoardView 0x143ada320> valueForUndefinedKey:]: this class is not key value coding-compliant for the key _proudLockIconView.
Parent symbol: Foundation - 0x00000001d0e44440  + 268

This happened to me today, iPhone 7+ 11.3.1 unc0ver 3.0.0 b39

Try enabling Increase Memory Limit(Beta) it does cause battery drain but some folks on Reddit says it helps
This has happen to me once but just force restart

It happened also with increase memory limit. Now since b40 I didn't experience anymore freezes of this type. I'll continue testing and will post back if it happens again.

This is not the place for this argument


Fix Python S3 API

I mistakenly thought that the accesskey would be empty if user didn't provide anything. In reality, it was something like AWS :

I mistakenly thought that the accesskey would be empty if user didn't provide anything. In reality, it was something like AWS :

Unit test added.

Unit test added.

alluxio-bot, merge this please

alluxio-bot, merge this please


Missing support for jsonb columns

GeneratedSchemaValidations cannot handle jsonb columns in Postgres. The migration succeeds but shows an error message at the end:

NoMethodError: undefined method `jsonb' for #<GeneratedSchemaValidations::Table:0x00007f2b099fc1b8 @table_name="ahoy_events", @column_names=["visit_id", "user_id", "name"], @possible_belongs_to_not_null_columns=[], @unique_indexes=[], @validations=[]>
Did you mean?  json
Tasks: TOP => db:schema:dump
(See full trace by running task with --trace)

fixed by b673373bfd654d33e4f164fc3126bda0a97ceb21
new version: 0.2.3


Tools needed to build Kodein-DB

Listing everything that needs to be done before releasing Kodein-DB.

crc32c
ios-cmake
leveldb
leveldb-kodein
snappy

Questions?

Do we need to build each of those tools?
Do we need to build those tools for each platform?
For each tool what do we need to run? Thus what compilation tools do we need?

Gradle is configured to build everything for you.
Don't forget to git submodule update --init when you checkout the repo !
What you need on the build system is:

Java 11+
A javah binary (usualy installed with Java 8) with its path defined in gradle.properties: javah=/usr/bin/javah
CLang
CMake

Note that gradle will first build crc32c & snappy, then leveldb using these two libs, then Kodein-DB.
Note also that each libs has to be recompiled for each targetted platform, so it can take a while.


improving definitions for monoid, comonad, applicative, and ADT

Addressing some of the feed back in #220
I looked at "Lift" and while I can see that our definition is more general than "lambda lifting" I kinda prefer that the idea be generalized to all values not just functions. That I think make it easier to teach the idea of computation contexts in the way that Scott W does with the elevated worlds metaphor

@hemanth :shipit: ?


fix a file descriptor leak

urllib2.urlopen returns a file-like object that should be closed. Not
doing so would leave sockets in the TIME_WAIT status. These sockets
may take some time before being cleaned up by the kernel.

You could have simplified this to some degree by using the "with" statement, which is available in python 2.6+.

I almost posted the same thing, but we have to support python 2.4 for our collectd, might as well support it here too.

Oh, I thought we only needed to support back to python 2.5.


fixed header for mobile

Hi @Mide001, are you on Quizard's Slack workspace?

Also, @Mide001, ideally, you should make changes in a specific branch and ensure your PR fixes one thing at a time.
You're currently handling the Netlify badge and page responsiveness in one PR. These are two separate issues.
Also, please comment on the issue you want to work on first and get approval before contributing.
This is so you can know if someone else is already working on the issue and understand what is required for the issue to be correctly merged.
I'm closing this PR right now. Thank you so much for your enthusiasm.


Cypress: Policy Member UI

User Story
As a Developer, I will use this work when I want to automate acceptance testing.
Definition of Done
The Policy Membership UI scenarios from the Acceptance Testing Script are covered by automated Cypress tests.

[ ] Removing a member to a policy.
[ ] Adding a policy member.

this work has been deprioritized.


ES5 request

New request DSL, supports ES2 and ES5, replaces legacy Chewy::Query DSL

I've tried this (maybe i'm wrong) but it doesn't seem to work well with the new bool query on ES 5. as described here:
https://www.elastic.co/guide/en/elasticsearch/reference/5.0/query-dsl-filtered-query.html
Which results in a parsing error:
Elasticsearch::Transport::Transport::Errors::BadRequest: [400] {"error":{"root_cause":[{"type":"parsing_exception","reason":"no [query] registered for [filtered]","line":1,"col":22}],"type":"parsing_exception","reason":"no [query] registered for [filtered]","line":1,"col":22},"status":400}

It still uses the old one, was to configured for the new one, unfortunately, will do it soon

There are actually other problems, like it doesn't support pagination or result objects yet

Whatever, merging this shit.

@pyromaniac will you bump up the version anytime soon?
I really need your code. tnx!

Yeah, will do my best. Doing latest improvements at the moment


⚠ Fortnite has degraded performance

In 05ba534, Fortnite (https://fortnite.com) experienced degraded performance:

HTTP code: 200
Response time: 1611 ms

Resolved: Fortnite performance has improved in 463dde4.


docs(v2): add a community plugin

Add a community plugin of @easyops-cn/docusaurus-search-local.

Motivation
This plugin is originally forked from cmfcmf/docusaurus-search-local, but later fully rewritten to reach our goal: up to date, maintainable, robustness, pretty look, and mostly language of Chinese supported.
We have already using it in our production, hope it will help others.
Have you read the Contributing Guidelines on pull requests?
Yes.
Test Plan
N/A.
Related PRs
No.

@weareoutman thanks!


Plant seeds screen

🗃 Github Issue Or Explanation for this PR. (What is it supposed to do and Why is needed)
#509
✅ Checklist

[x] Github issue details are up to date for people to QA.
[x] I have tested all my changes.

🕵♂ Notes for Code Reviewer
First advance, added:

Interactor folders
Screen
Route

👯♀ Paired with
"nobody"

screen shots?


Fix crash if non-ASCII character precedes $ math

Previously, if a sequence like “$x$” occurs (where the quotes are fancy quotes), then there would be a StringIndexError. This fixes the problem by calling prevind instead of subtracting 1.
Since prevind with the default n=1 can't return a negative number, I've also removed the else branch here since it's only setting 0 to 0.

The second commit fixes a related issue, but during the prerendering KaTeX step. I split up the prerendering test so that I could test it locally, since I haven't been able to figure out node/highlight.js yet.

The Travis 1.0 failure is real, but preexisting (as redirecting stderr/stdout to a IOBuffer is a 1.1 feature). I guess Travis wasn't testing this codepath because it's missing highlight.js; this change made the failure visible by removing that check for this branch.
Maybe upgrading the minimum Julia version to 1.0 is reasonable?


Source '/usr/share/tomcat6' or destination '/var/log/cef' are invalid directories.

2015-12-11 17:13:47,714 ERROR [org.ozoneplatform.auditing.quartz.jobs.AbstractFileMovementJob] (quartzScheduler_Worker-4) Source '/usr/share/tomcat6' or destination '/var/log/cef' are invalid directories.
Where is this configured at?

Within the Configuration Page of the Administration App.  Go to the Auditing section and there will be a number of relevant settings as described below:
"Common Event Format (CEF) Auditing" - This is on by default; turn it off if you do not need audit logs that record all modifications to persisted data
"Object Access Auditing" - This is off by default; turn it on to log all ACCESSES of persisted data to the CEF logs.  This produces very verbose output
"Relocate CEF Logs" - This flag, which is on by default, activates a job which runs periodically to export CEF logs to a separate directory on the system.  This is used in conjunction with other tools which watch for logs deposited in a particular directory.  If you are not using such a system this should be turned off - this should fix the error that you are seeing.
"CEF Log Source Location"
"CEF Log Destination Location" - The directories which the relocation job uses for the logs.  If you want the job to run, but just need it to use different directories, then these are the parameters to adjust.


Feature Request: different destinations based upon log_level

I'm looking at updating my standard logging capability.  I like the looks of the semantic logger.  I also like the feature in Yell that allows me to specify different log destinations on a per log-level basis.  For example I could define a destination for all trace log entries to go to a file named trace.logm  all info and 'debug' log entries to go to STDOUT, and all error and above log entries to go to a centralized log service.
Does anyone else think that would be a handy addition to SemanticLogger?

Use the filter attribute when adding the appender to filter out any log_level you don't want sent to that appender.
Filter Documentation: https://rocketjob.github.io/semantic_logger/filtering.html
All the fields that can filtered on: https://rocketjob.github.io/semantic_logger/log_struct.html

I will review that capability.  Thank you for pointing it out.  To be clear I'm looking to spin up one logger instance logger where logger.info goes to STDOUT and logger.debug goes to STDERR and everything else goes to a file.

Filtering supports the above use-case.

For the next guy/gal ... filtering is easy.  semantic_logger is now my favorite logger libraru!
# level_index #=> 0     1     2    3    4     5
level_names = %w[ trace debug info warn error fatal ]

level_names.each_with_index do |level_name, level_index|
  SemanticLogger.add_appender(
    file_name:  "#{level_name}.log",
    filter:     Proc.new { |log| level_index == log.level_index }
  )
end


If so, how do I generate an API key?

no, this library is for interacting with models hosted within the dreamstudio product.


Skip writing the HTTP actions to config file and log warning when REST Methods are configured for tables/views

Why make this change?

Closes #1571 #1570

At the moment, CLI writes out all the 5 HTTP methods to the config file when adding an entity backed by table/view.

However, for tables/views, the Methods property does not play any role and can be skipped writing to the config file.

What is this change?

EntityRestOptions: Methods property is made nullable. References where it was initialized with an empty array is updated to null.  Appropriate null checks are added.
EntityRestOptionsConverter: Serialization logic is updated to write to the config file only when the Methods within REST element is not empty
RuntimeConfigValidator.ValidatePresenceOfExtraRestProperties(): Helper method to log a warning message when Methods property is added manually to an entity backed by table/view.
HTTP Actions are removed from the snapshot files.
dab-config.* files - Methods property is removed from the reference config files

How was this tested?

[x] Integration Tests
[x] Unit Tests
[x] Manual Tests

Sample Request(s)

CLI: REST actions are not written to the config file
dab add books --source books --permissions "anonymous:*"

Config Json created:
"entities": {
    "books": {
  "source": {
    "object": "books",
    "type": "table"
  },
  "graphql": {
    "enabled": true,
    "type": {
      "singular": "books",
      "plural": "books"
    }
  },
  "rest": {
    "enabled": true
  },
  "permissions": [
    {
      "role": "anonymous",
      "actions": [
        {
          "action": "*"
        }
      ]
    }
  ]
}
  }

When the config file is hand-edited to add methods property to a table/view, engine logs a warning message during start-up

Config:
"Book": {
      "source": {
        "object": "books",
        "type": "table"
      },
      "rest": {
        "enabled": true,
        "methods": [
          "get",
          "post"
        ]
      },
      "graphql": {
        "enabled": true,
        "type": {
          "singular": "book",
          "plural": "books"
        }
      },

Warning Logged:

Some questions and suggestions around the same. Looks good so far!


wrong coment line in sw.js

In the file
https://ampbyexample.com/sw.js
line 110:     //console.log('ignore? ' + path + ' ' + url + ' -> ' + url.test(path));
shows error in the console it should be with a space after //
// console.log('ignore? ' + path + ' ' + url + ' -> ' + url.test(path));

What do you mean by shows error in the console?
On Mon, Nov 13, 2017 at 8:14 PM Lovntola<EMAIL_ADDRESS>wrote:

In the file
https://ampbyexample.com/sw.js
line 110: //console.log('ignore? ' + path + ' ' + url + ' -> ' +
url.test(path));
shows error in the console it should be with a space after //
// console.log('ignore? ' + path + ' ' + url + ' -> ' + url.test(path));
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/ampproject/amp-by-example/issues/1009, or mute the
thread
https://github.com/notifications/unsubscribe-auth/AAXOOP7yUYjK7MqFSNM0GoZEJ3ojIC4Qks5s2KMLgaJpZM4QcWGa
.

and:
url.test is not a function the log points to this line
it was fine after change the coment line
but I can't reproduce it now.
Sorry may be a fault on my site.
Please close this ticket if you don't see this error.
Sorry for waste your time.

np. thanks for looking into this.


Consolidate doTest methods

Updates VimTestCase.kt to consolidate the various doTest functions so everything eventually goes through one doTest function. @JvmOverloads are added for Java tests, and default arguments means it is no longer necessary to specify the default modes of COMMAND/NONE at all call sites (not removed in existing tests). Also fixes a minor bug in one of the deleted doTest overloads that would pass keys to neovim twice (was fortunately never called).
This is groundwork for extending doTest to support testing wrapped and folded text without having to use offsets.

Yay, thank you for cleanup :D


Time series add SST

https://github.com/aqualinkorg/aqualink-app/issues/493
I wasn't able to test the implementation on the sst-backfill script.

Same as spotter, let's make sure we do that more regularly. The goal is to have "almost live" data in our DB


 Last update e0c92fb...99a9781

LGTM


Massimport of users (WIP)

@emanuelschuetze @normanjaeckel Whats your opinion.
There is one trap: The username is generated on creation. So you can import user 'max' twice, because 'max' gets created, and for the second 'max' the username will be 'max 2'.
Now, all users are first validated, so there is no correction of the username, and then bulk_created.
So the drawback is, that we cannot import unique users twice (unique by username) on the same import.
If you want to have two 'max', you have to import him twice with two clicks on 'import'.
The 'importTrackId' is used for checking, which users gets imported. If 4 of 5 get imported, the client wants to know which users are imported. You can pass an importTrackId to every user and get an array of imported ids in the response. Helpful for the client.

I tested a bit. Importing 1000 users via csv in ca. 20sec! Wow! :-) Only one request. Also for importing via text box. Looks good.
Duplicated users (with same username) is an rare case. Then I would stop importing the second one and print out which users are not imported. Reimport of these failed users would be nice (but not really required).
Let me know when I should more retest.

This sould already work with csv. Just one of the duplicates should get a green sign.
Just the button 'remove all imported' is missing.

@emanuelschuetze Please test especially duplicates.

The progressbar is some kind of a problem: Because we have a single request, we just get a single response. So I do not see any possibilities to transfer the status.
May with the notify system, the server could send status updates; but this is not fully implemeted yet.
I've tried to import 5000 users --> No problem. Takes about 30 sec. No Redis, Postgresql, just basic OS. Can you reproduce this?

Tested: 5000 users with one group --> 503.
But if I wait a bit, the server execute all group assigning.
To speed up this group assigning, we need to have an explicit m2m table between users and groups. (In Django's PermissionMixin just a 'normal' m2m relation is established.) If we have this model (e.g. UserGroupRelation) we can just do a bulk_create for all these relations. For this we have to do:

Make a wrapper class around Django's PermissionMixin
Make a migration to copy all existing relations to the new model
(Maybe we have to change group/user code in the views, because of this new model)

@normanjaeckel @ostcar Your opinions?

Hm. I am not sure whether we should spend much time on this old permission modeling. @ostcar has always been against the Django permission system and we can think about removing it completly and building an easier replacment ...

@ostcar ping

I have nothing to say here. I still think, that the current permission system is not good, but I don't have the time to change it.

@FinnStutzenstein Do you need more help/information?

@emanuelschuetze I'll work on next week

@normanjaeckel Please look at the new commit.

The migrationpath is broken; if we use this, we need to copy much data (how to do this?)
I don't like this way of changing the group relation ship.....

Closed in favor of new PR.


fix: disable l2 to l1 events because of eip4844

Hello @montekki, I'm trying to run this project locally but without success. Would you care to explain why you don't listen for L2-L1 events anymore after EIP-4844?

In addition, do you know if a hyperchain that is not EIP-4844 upgraded would have an issue running the latest version of this project?

@therealdannzor we have only listened for these events in the informational purposes, this table was never used in any actian finalization logic. It must not affect your local setup running.


ScanStream: replace custom emissions with Mono::expand

https://github.com/lettuce-io/lettuce-core/issues/1458

Great work, thank you. It's great to see this simplification that removes more code than it adds while becoming more robust.

Thank you for your contribution. That's merged, polished, and backported now.


[Event Request] Table 85 Acc. Schedule Line - LookupDimFilter

Please, add new Event to procedure "LookupDimFilter" in table 85 Acc. Schedule Line
OnBeforeLookUpDimFilter(DimNo,Text,AccSchedName,IsHandled,Result);
[External]
    PROCEDURE LookUpDimFilter@57(DimNo@1000 : Integer;VAR Text@1001 : Text) : Boolean;
    VAR
      DimVal@1002 : Record 349;
      DimValList@1003 : Page 560;
      IsHandled@5562062 : Boolean;
      Result@5562063 : Boolean;
    BEGIN
      GetAccSchedSetup;

      // > new
      IsHandled := FALSE;
      OnBeforeLookUpDimFilter(DimNo,Text,AccSchedName,IsHandled,Result);
      IF IsHandled THEN
        EXIT(Result);
      // <

      CASE DimNo OF
        1:
          DimVal.SETRANGE("Dimension Code",AnalysisView."Dimension 1 Code");
        2:
          DimVal.SETRANGE("Dimension Code",AnalysisView."Dimension 2 Code");
        3:
          DimVal.SETRANGE("Dimension Code",AnalysisView."Dimension 3 Code");
        4:
          DimVal.SETRANGE("Dimension Code",AnalysisView."Dimension 4 Code");
      END;
...

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.


[Bug] Flickering while scrolling

Version
5.4.3
Link to Minimal Reproduction
https://codepen.io/pen?&prefill_data_id=4f4c4159-b422-4f29-9c35-37d61cfcc552
Steps to Reproduce
Issue is reproduceible with official Examples.
Open for example the Large Area Chart
Zoom in with mouse wheel or change dataZoom-> end property for example to 0.1:
  dataZoom: [     {       type: 'inside',       start: 0,       end: 0.1     },
Move the diagram with the mouse alternately to the left and right.
Current Behavior
If you move the dialog box slowly, the border segments suddenly disappear / appear.
This is already strange and not as expected.
If you move the chart back and forth quickly, you will get a very unpleasant flickering.
Expected Behavior
The edge segments should not suddenly appear/disappear, but should always be drawn up to the edge of the diagram.
Environment
- OS: Windows
- Browser: Chrome and Firefox

Any additional comments?
No response

You can see the effect on the GIF. However, you can't see it as a flicker because the GIF fps is too low.

try animation:false in series.  📌 please close issue if problem solved.

it has to do eith the filterMode of the dataZoom component. If a datapoint is moved out of the grid it will be treated like it doesnt exist, thus the white edge border (size depending on how far the next datapoint is to the edge border). Set filterMode: 'none' to disable this behaviour.

try animation:false in series. 📌 please close issue if problem solved.

series: [
    {
      animation: false
    }

That doesn't help. Flickering is still there.

it has to do with the filterMode of the dataZoom component. If a datapoint is moved out of the grid it will be treated like it doesnt exist, thus the white edge border (size depending on how far the next datapoint is to the grid border). Set filterMode: 'none' to disable this behaviour.

If I set filterMode to none, this is not the behavior I want. It is then no longer filtered at all.
It makes scrolling the graph very sluggish.
What I would expect is that the filter should take place,
but only the data that does not affect the visible area should be filtered / clipped.
Currently the filtering affects the first and last visible value in the graph. And that feels broken.

Currently the filtering affects the first and last visible value in the graph. And that feels broken.

filtering doesnt affect any visable values and the observed behaviour is not a bug. Its a little unfortunate in the case of an line graph with area.
What you want seems to be a new feature that allows to set the filterMode separately for the x- and y-axis. Since you want filtering along the y-axis but not on the x-axis.

Thank you for you feedback.
I'm not really familiar with ECharts yet.
I only discovered this great library 2 days ago.
What I need is that the chart is always drawn to the end.
The two red areas should also always be painted over.
One more value should always be taken into account.
This would then certainly draw out of the graph area, which would require the use of clipping.
I have no idea whether clipping is always used in ECharts or whether it is deliberately avoided for performance reasons.

To clarify once again, ECharts currently behaves as follows:

Instead, I would expect the behavior on the sidelines to be something like this:

The flickering on both sides comes from redrawing the fill areas. And the redrawing is due to filterMode being 'filter'.
Change the code to
  yAxis: { max:175, min:0 },
  dataZoom: [{ type: 'inside', filterMode:'none', start: 0,  end: 0.1 },

and you'll see no flickering, but of course the yAxis range will not cover all values.
@MatthiasMert's idea of a new filter by axis is a good one. Then we would have dataZoom: [{ xFilterMode:'none',  yFilterMode:'filter' }] and obtain the effect you want for the entire X axis.

I don't think we understand each other.
I don't think "redrawing" is the reason. If you scroll slowly, you can see the effect much more clearly:

Here the area is either filled or not, depending on the current scroll position.
Perhaps you see/describe the effect as "redrawing". For me, "redrawing" is when something is painted over.
The actual chart line is also drawn at the edge or it is not, depending on the scroll position.
In addition, you can see at the bottom of the X-axis that hourly data appears and disappears again when scrolling, which leads to additional flickering on the X-achsis when scrolling quickly.
I think that the X-axis should only move the displayed values and not increase or decrease the level of detail when scrolling.
This is probably because the two lateral values are sometimes cut off and sometimes not.
This means that you have constantly changing conditions when scrolling the visible area.
For this reason, the chart line also "wobbles" slightly, as the data basis is constantly changing for the interpolation algorithm.
filterMode:'none' has a massive impact on performance.
Moving the chart becomes very sluggish and not nearly as smooth.
I assume that (without filtering) all data is iterated over, even though most of it is not visible.
When switching off the filtering, it seems to me that the FPS deteriorates by a factor of 2 (or more).
If you have a lot of data outside the visible area, then it is also understandable,
that the FPS goes down if you don't filter it out.
It is therefore legitimate and sensible to filter out data that is irrelevant to the visible area.
If I had to use filterMode:'node' to prevent flickering, then unfortunately ECharts is not suitable for my purposes for performance reasons. That would be a great pity.


ci: Refactor CI with resusable, custom GitHub Action

Type
enhancement

Description

Introduced a reusable GitHub Action (invoke-tox) to streamline the setup of test environments across multiple workflows.
The new action supports specifying Python version, deciding whether to install Graphviz, running specific Tox environments, and enabling Codacy code coverage with a token.
Refactored the tests.yml workflow file to utilize the new GitHub Action, significantly simplifying the workflow syntax and making it more maintainable.
Enhanced CI workflows by enabling Graphviz installation and Codacy code coverage directly through action inputs, removing the need for these steps in each job.

Changes walkthrough
Relevant filesEnhancement

action.ymlAdd New GitHub Action for Test Environment Setup

.github/actions/invoke-tox/action.yml
Introduced a new GitHub Action for setting up a test environment.  Allows configuration of Python version, Graphviz installation, and Tox environment.  Supports optional Codacy token input for code coverage.  Simplifies workflow files by encapsulating common setup steps.

+55/-0

tests.ymlRefactor CI Workflows to Use Custom GitHub Action

.github/workflows/tests.yml
Refactored workflow to use the new GitHub Action for test setup.  Simplified test job configurations by removing redundant steps.  Added Graphviz installation and Codacy token support directly in the action inputs.  Made the workflow more readable and maintainable.

+41/-43

✨ PR-Agent usage:
Comment /help on the PR to get a list of all available PR-Agent tools and their descriptions

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

Project coverage is 94.26%. Comparing base (532b54f) to head (bf9321a).

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #252   +/-   ##
=======================================
  Coverage   94.26%   94.26%
=======================================
  Files          12       12
  Lines        1116     1116
=======================================
  Hits         1052     1052
  Misses         64       64

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Decorators don't work with class methods

The documentation states that global permissions can be implemented as either @staticmethod or @classmethod on the model. The decorators (e.g. @unauthenticated_users) assume the first argument of the global permissions methods is a Request. This is true if the methods are static methods. However, when implemented as class methods, the first argument is the class itself.
# Good
@staticmethod
@authenticated_users
def has_read_permission(request):
    return True

# Bad
@classmethod
@authenticated_users  # This will fail because it expects the first argument to be a Request.
def has_read_permission(cls, request):
    return True

Either the decorators should be updated, or the documentation should be modified to note this issue and/or remove the mention of @classmethod.

You're right, the decorators don't work with classmethods. However, you can still use classmethods for global permissions, you just can't also use the decorators on top of that. That being said, you it would be nice to be able to do both. I will look into a solution.

Hi all, just assessing whether this is a good fit for my uses (looks good) and saw this issue.
This library might come in handy, as it provides a workaround to the inherent Python 2 classmethod decorator issue:
https://wrapt.readthedocs.io/en/latest/
as discussed here:
https://hynek.me/articles/decorators/
HTH.

Sorry if I don't understand deeply the problem, it is not possible to fix this problem?
Looking at the decorators I see that the is_object_permission value is used to determine if the request it is on args[1] instead of args[0]
There any problem if it gets generalized to ?
from rest_framework.request import Request
...
request = args[0] if isinstance(args[0], Request) else args[1]

Probably we can even use duck-typing to identify the Request instead of isinstance


Can we use Winston for logging?

In #182 @hrueger wrote that we can use winston for logging, and I think that it's a good idea.
PROS of using winston:

Log file rotation
Support for multiple "transports": not only console or files, but also Syslog, Windows Event Viewer or Amazon Cloudwatch

I think we can use two loggers: one, with "file" and "console" as transports, for logging, and one, with a file transport, for Tally data.
Tomorrow I'll try to open a PR, so we can decide if using winston is a good idea or not.

Sounds great to me.

I like that too. I'm working on sentry error reporting as well. It will give us better insight into how everything is running and where we need to put out time in terms of bug fixing even when someone does not manually submit a bug report.


explicit begin/end for managing connection

currently the first push/val will take a longer time because of the first connection.
it would be better to warm up the connection on begin() instead and tear everything down with end().
As pointed by @yhua537 in #375 that would allow sketches to trade additional latency for more memory available.

Note that this will be affected by #353

@proppy , Any idea when this could be implemented? The original idea is almost 3 years ago.


Change defaults on bulk import from unlimited.

Is your feature request related to a problem? Please describe.
Bulk imports that continually add files to a tablet that already has a lot of files will eventually cause problems.
Describe the solution you'd like
#5104 adds limits that can prevent this situation from happening, but those default to unlimited.  There is also an existing property table.bulk.max.tablets that was introduced in 2.1.0 that defaults to unlimited.    After #5104 is merged could change those defaults to a limit and also change the existing property  table.bulk.max.tablets to also have a default limit.  Could set the limit of all these properties to 100 to start with.  When a user encounters the limit they can adjust as needed.  This will make the default behavior of the system more sustainable.
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.

https://github.com/apache/accumulo/issues/5117#issuecomment-2518542540  this is an example of one problem that could happen w/ too many files in tablet.  Most Accumulo code assumes that all tablet metadata can be read into memory.  When a tablet has 25K files and many threads reading that tablets metadata into memory it can cause a lot of memory pressure.


[Doc Improvement][SBS hello bot with JavaScript][2689558]

Acrolinx Scorecards
We currently enforce a minimum score of 80.
Click the scorecard links for each article to review the Acrolinx feedback on grammar, spelling, punctuation, writing style, and terminology:

Article
Score
Issues
Scorecard
Processed

msteams-platform/sbs-gs-bot.yml
91
37
link
:white_check_mark:

More info about Acrolinx
Use the Acrolinx extension, or sidebar, in Visual Studio Code to check spelling, grammar, style, tone, clarity, and key terminology when you're creating or updating content. For more information, see Use the Visual Studio Code extension to run Acrolinx locally.

Learn Build status updates of commit 9af3c76:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

msteams-platform/sbs-gs-bot.yml
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the learn.microsoft.com contributor guidesPost your question in the Learn support channel

Acrolinx Scorecards
We currently enforce a minimum score of 80.
Click the scorecard links for each article to review the Acrolinx feedback on grammar, spelling, punctuation, writing style, and terminology:

Article
Score
Issues
Scorecard
Processed

msteams-platform/sbs-gs-bot.yml
91
37
link
:white_check_mark:

More info about Acrolinx
Use the Acrolinx extension, or sidebar, in Visual Studio Code to check spelling, grammar, style, tone, clarity, and key terminology when you're creating or updating content. For more information, see Use the Visual Studio Code extension to run Acrolinx locally.

Learn Build status updates of commit 034f699:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

msteams-platform/sbs-gs-bot.yml
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the learn.microsoft.com contributor guidesPost your question in the Learn support channel

Acrolinx Scorecards
We currently enforce a minimum score of 80.
Click the scorecard links for each article to review the Acrolinx feedback on grammar, spelling, punctuation, writing style, and terminology:

Article
Score
Issues
Scorecard
Processed

msteams-platform/sbs-gs-bot.yml
91
37
link
:white_check_mark:

More info about Acrolinx
Use the Acrolinx extension, or sidebar, in Visual Studio Code to check spelling, grammar, style, tone, clarity, and key terminology when you're creating or updating content. For more information, see Use the Visual Studio Code extension to run Acrolinx locally.


ench: improve logging

Related to: #187

Signed-off-by: Amal Thundiyil<EMAIL_ADDRESS>

Nice~
/LGTM
/cc @renyunkang


Rapid test field is set by mechanism

Description
Noticed a bug whereby is_rapid_test was constantly being overwritten by the wrong value, despite the mechanism being set correctly as 'lateral_flow'. https://www.notion.so/joinzoe/Updating-existing-test-to-lateral-flow-sets-it-as-PCR-instead-5020adb4f1944f929060dc6dda58b54f
This PR fixes it in that it shouldn't be overwritten by the DTO creation since this field is no longer shown to a user logging a v2 test.
TBD what we want to do if they update a v1 test (hence why I've just commented out the code for now). Let's see what comes out of wider group QA to see what users expect.
On what platform have you tested the change?

[ ] Android - Emulator
[ ] Android - Physical device
[ ] iOS - Emulator
[x] iOS - Physical device

Screenshots
Checklist (for submission)

[ ] Analytics/tracking events have been added (if relevant)
[ ] Added new screen(s) to src/features/screens.ts and new Redux slice(s) to mocks/mockedInitialState.ts (if relevant)

Checklist (for reviewers)

[ ] Checked against Figma designs (if relevant)
[ ] Checked that data has been successfully saved to the backend (if relevant)

Out of scope and potential follow-up

Can you add a comment or just delete the line of code completely.

@selvach @aikewoody After a break and more testing, I actually decided it was better to keep it in, but move it above the creation of the mechanism DTO (that overwrites is_rapid_test field if needed for PCR / lateral flow tests). This has the benefit of allowing for the update of this field for "other" or "antibody" tests which doesn't assume anything for is_rapid_test.


Stuck at subdomain scanning

This is my first time using the tool.
I just get stuck at subdomain scanning, I tried to leave it for about 3 hours and it was still stuck at the same phase, even I can't cancelling it by hitting ctrl + c, it won't cancel until I close my vps window

Any ideas where is the problem?
Thanks in advance.

I think there's some issue with your amass try running your garud using and check where you get stuck and then let me know will fix it then.
Command:
$ bash -x /usr/bin/garud -d target.com

this is the result:

my vps specs:
2 GB RAM, 1 vCPU, 60 GB SSD
Node.js
I don't think that it has an env, it's amazon lightsail instance

Yes it requires higher specs to work faster but the specs you mentioned here is okay. Actually the issue is with the amass try commenting out the amass scans I'm pasting the line number and code below.
Run garud like this (use screen to let it run in background)
$ env bash
$ garud -d target.com

Line 146
# amass enum -passive -d $DM -o .tmp/amass.list &> /dev/null

Line 153
# timeout 50m amass enum -active -brute -w ~/wordlists/subdomains.txt -d $DM -o .tmp/amassact.list &> /dev/null


Emails not being sent

Hi,
We are self hosting on render.com, using sendgrid.com, everything is working great other than we receive no emails and sendgrid.com shows no activity.  We turned on AUTH_LOG_LEVEL=error but there are no errors, and when using passwordless login via the client we get an 'OK' response back from the auth server.  Any ideas, or a way of turning on more verbose logging?
Cheers and Happy Friday!

These are the logs I get back https://gist.github.com/cj/4512c7c1546f4b65084c3afd26525c29

Hello,
Sorry for the late reply
You can try AUTH_LOG_LEVEL=debug

Hello @cj, are you still facing this issue?

No news, closing. Will reopen if required.
Thanks


Pull request of kata/enumeration/start branch

Pull request of kata/enumeration/start branch

Pull request of kata/enumeration/start branch


Boone, IL

Closes: https://github.com/openaddresses/openaddresses/issues/5057

@justinelliotmeyers can you try running the layer through a ST_MakeValid or something similiar?
GDAL is barfing on it
2020-05-29 16:28:41,807    ERROR: GDAL gave Failure 1: IllegalArgumentException: Invalid number of points in LinearRing found 3 - must be 0 or >= 4

Closing as stale


Style FormSectionDescriptor ?

Thanks for such a robust class. I'm using a row's configuration to set style attributes of my form - like so:
row.configuration[FormRowDescriptor.Configuration.CellConfiguration] = ...
Is there a way to style the header labels for each FormSectionDescriptor ?


implement events

https://github.com/GoogleChrome/puppeteer/blob/v1.6.2/docs/api.md#class-page
page.on('close') and so on.

So...
There is no (simple) way to have a non-blocking events like in Puppeteer:
const page = await browser.newPage()

page.on('console', msg => console.log(msg.text()));

await page.evaluate(async () => { console.log('hi') })
await browser.close()

because Marionette uses a classic asynchronous transport model: sends message with an ID and then asynchronously gets a response using the same ID.
Options I see:

Something like blocking getLog in Nightwatch with an initial global hook:

const page = await browser.newPage()

await page.evaluate(async () => { console.log('hi') })

console.log(await page.getLog())

await browser.close()

Introduce a second layer of real-time communications between page and Foxr using an injected WebSocket. Not sure if Puppeteer has been using the same or there is a direct way to do that in Chrome Debugging Protocol.


operator hive-operator (1.2.4638-56d6d64)

operator hive-operator (1.2.4638-56d6d64)

/pipeline restart operator-release-pipeline


Imperfect amounts

Handle imprecise amounts like "a few"

@halogenandtoast this looks great!! Handling it inside Ingreedy really simplifies things! 🎈
Just 2 things:

Seem to be a typo in https://github.com/cookpad/ingreedy/pull/2/files#diff-264d8e39f4e911fccd13c03abdb2a5baR4 (wondering why the specs would pass. I just added to Travis so we can see on GitHub too)
I just pulled https://github.com/cookpad/ingreedy/pull/3 into master, so maybe better to target master with this one too?

Nice catch on the typo, I failed at rebasing against the or branch correctly

@balvig fixed and based on master ♻

Awesome! 👍
(CI failed on jruby, I guess we don't really care about that though, can remove from travis.yml?)
Usually what we then do is add a spec in the main project to test it works in the context of the app, thanks a lot! 🎉

@balvig what do you mean by main project, are you referring to global-web?

This looks great!

@balvig updated for case insensitivity ♻

I'm going to assume since my other PR had a thumbs up that this is good to merge.

nice!


Broken specs after installing

Ruby 2.2.5
Rspec, factory girl, capybara
After installing test-prof 37 broken specs appeared:
412 examples, 37 failures, 7 pending
After uninstalling test-prof:
412 examples, 0 failures, 7 pending
The most common error - deadlock
ActiveRecord::StatementInvalid:
        PG::TRDeadlockDetected: ERROR:  deadlock detected at character 13
        DETAIL:  Process 11977 waits for RowExclusiveLock on relation 5270960 of database 5270706; blocked by process 30236.
        Process 30236 waits for ShareRowExclusiveLock on relation 5270996 of database 5270706; blocked by process 11977.
        HINT:  See server log for query details.

and also it is not working at all. No reports at rails 4.1, rspec 3,4

What profiler do you use? How do run your tests? Need more info.

for example:
FPROF=flamegraph rspec spec/models/

i don't see any reports, just Rspec report, don't see test-prof default folder ('tmp/test-prof') - it is not existed. Maybe I missed something important?

in my first issue with broken specs after installing test-prof - i used simple
rspec spec/*

without any test-prof env variables

FPROF=1 rspec
did not produced reports, too

Do you see "FactoryProf enabled ..." in your console?

in my first issue with broken specs after installing test-prof - i used simple
rspec spec/*
without any test-prof env variables

Hm, that strange; test-prof is no-op if you do not require something either in your code or through env variables.
Do you use Spring? Have you tried to reload it?

Do you see "FactoryProf enabled ..." in your console?
No, i don't
Do you use Spring? Have you tried to reload it?
No, spring only in develoment environment.

Also i figured out broken specs issue, sorry, my local bug.

Do you see "FactoryProf enabled ..." in your console?

No, i don't

Have you tried other profilers? For example, TAG_PROF=type rspec .... Is there any output?
Maybe, for some reason test-prof is not loaded. Try to add require "test-prof" in the beginning of your spec_helper.rb.

great, after direct requireing in working.
but
FPROF=1 rspec spec

caused such an error
[TEST PROF INFO] FactoryProf enabled (simple mode)
[TEST PROF ERROR]

FactoryProf generates deadlocks and don't wont to work. Wtih TAG_PROF=type rspec working great. So it seems to be a problem with FactoryProf. Anyway, thanks

[TEST PROF ERROR]

There is already an issue #42. It's fixed in master and will be released soon.

FactoryProf generates deadlocks and don't wont to work.

I doubt FactoryProf can cause deadlocks; to do so it should perform DB queries and it does not.
Are there other log messages from TEST PROF?


MINOR: checkstyle version upgrade: 8.20 -->> 8.36.2

details:

checkstyle: 8.20  -->> 8.36.2  https://checkstyle.org/releasenotes.html#Release_8.36.2
spotbugs  : 4.2.2 -->> 4.2.3 https://github.com/spotbugs/spotbugs/blob/4.2.3/CHANGELOG.md

Rationale:

Checktyle 8.20 is two years old; also, both Gradle 6.8 and 7.0 are using recent Checkstyle version (8.37)
Checkstyle 8.36.2 is geared towards recent Java versions (see 8.36 release notes: https://checkstyle.org/releasenotes.html#Release_8.36)
Checkstyle version 8.42 should be skipped (lots of false positives, see here: https://github.com/checkstyle/checkstyle/issues/9957)
more recent Checkstyle versions (i.e. 8.37 and above) are imposing more strict indentation rules and hence we can opt to:

relax Checkstyle indentation rules OR
comply with these new rules and change affected classes (note: total of 50 violations in 18 classes is recorded when compiled with Checkstyle 8.41.1)

@ijuma Please review this.
Also, let me know what do you prefer: to merge this commit or wait for a full upgrade to a latest checkstyle version ? And in case that you opt for the latter: which strategy do you prefer (i.e. are we going to relax checkstyle rules or comply with them) ?

Alas, spotbugs checks fail... https://ci-builds.apache.org/blue/organizations/jenkins/Kafka%2Fkafka-pr/detail/PR-10656/1/pipeline
Let me check again on mu end... converting to a draft.

My bad... I was rushing to scrap PR so I just didn't check spotbugs related gradle tasks on my end... I was counting on a fact that this is just patch, but it turns our that spotbugs version 4.2.3 is not stable. So, it is just checkstyle upgrade, after all.
@ijuma feel free to review now.

@dejan2609, can you share she'll command that checkstyle team can execute to run only checkstyle validation, we will make sure no false positives happen.

@dejan2609, can you share she'll command that checkstyle team can execute to run only checkstyle validation, we will make sure no false positives happen.
Please share link to checkstyle config.

Sure @romani ! I will provide you complete POC/manual on your repository/ related ticket asap.

If you can enforce, no checkstyle violation in CI, and make it possible to change checkstyle version to any version from CLI arguments, checkstyle team can add Kafka project to regression suite and you will never have false positives from CheckStyle any more, as we will validate Kafka execution on each commit in checkstyle.

I am only a (modest) apache-kafka contributor, but I do know that @ijuma can answer you about this.

When you finish build config update to define checkstyle version from CLI, please share with us command.
We will do
git clone
Git checkout ...
Your build command

Sure @romani ! Hopefully I will have something for you in a day or two.

If you can enforce, no checkstyle violation in CI, and make it possible to change checkstyle version to any version from CLI arguments, checkstyle team can add Kafka project to regression suite and you will never have false positives from CheckStyle any more, as we will validate Kafka execution on each commit in checkstyle.

@romani sure, we can do that. @dejan2609 Do you want to contribute the change to allow the checkstyle version to be specified via a parameter? We have a few examples here https://github.com/apache/kafka#common-build-options

@ijuma please review #10698


new term request: establishment of protein localization to cajal body

Hi,
New term request please.
Term: establishment of protein localization to cajal body
Definition:  The directed movement of a protein to the cajal body in the nucleus.
child of: GO:0044744 protein targeting to nucleus
or better suggestions please.
GOC: BHF
GOC: BHF_telomere
GOC: nc
PMID: 25467444.
I want to annotate the TRiC components (apart from CCT1 and CCT2) to this requested term. Figure S1F. Also author states: "Analyzing the TCAB1 channel in a similar manner, we found that TRiC subunits were also required for TCAB1 localization to Cajal bodies (Figure S1F), demonstrating that multiple telomerase components were perturbed due to TRiC depletion".
Thanks
Nancy

Hi Nancy - do you know if there are protein targeting signals involved?  I'm trying to figure out how to best place your requested term and name it.
Possible names (and parentage):
(1) establishment of protein localization to cajal body (is_a establishment of protein localization to organelle)
(2) protein localization to cajal body (is_a protein localization to nucleus)
(3) protein targeting to cajal body   (is_a protein targeting to nucleus)
Parent definitions are here:
protein targeting to nucleus: The process of directing proteins towards the nucleus, usually using signals contained within the protein.
protein localization to nucleus: A process in which a protein is transported to, or maintained in, a location within the nucleus.
establishment of localization to organelle: The directed movement of a protein to a specific location on or in an organelle. Encompasses establishment of localization in the membrane or lumen of a membrane-bounded organelle.
From this issue information alone, without looking at the paper, I would recommend (2).  Feedback welcome.

Hi Tanya,
Thanks for looking into this and clarifiying the differences between the different possible parents.
From this, I agree, 2 seems the best option.
Thanks Tanya.
Nancy

Added using TG.  The parentage was even better:
[Term]
id: GO:1904867
name: protein localization to Cajal body
namespace: biological_process
def: "A process in which a protein is transported to, or maintained in, a location within a Cajal body." [GO_REF:0000087, GOC:BHF, GOC:BHF_telomere, GOC:nc, GOC:TermGenie, PMID:25467444]
synonym: "protein localisation in Cajal body" EXACT [GOC:TermGenie]
synonym: "protein localisation to Cajal body" EXACT [GOC:TermGenie]
synonym: "protein localization in Cajal body" EXACT [GOC:TermGenie]
is_a: GO:1903405 {is_inferred="true"} ! protein localization to nuclear body
intersection_of: GO:0008104 ! protein localization
intersection_of: has_target_end_location GO:0015030 ! Cajal body
relationship: has_target_end_location GO:0015030 {is_inferred="true"} ! Cajal body
created_by: tb
creation_date: 2015-12-17T21:48:18Z


Add safe browsing extension

As privacy is important factor for an user, SquareX is one the best in market for provides features like anonymous browsing and  disposable browser feature.

please assign me this issue

@Sulagna-Dutta-Roy I'd like to work on this issue, if it hasn't been solved yet.
Thank you!

please assign me this issue

Are you from GSSOC 2024? @S-i-y-a-g

Yes @Sulagna-Dutta-Roy

Closing this as no PR received


The official docker image is not accessible

I could not access it with docker pull ghcr.io/eole-nlp/eole:0.0.1-ubuntu22.04-cuda12.1.

Thanks for reporting! Should be accessible now.
For reference, these image can always be rebuilt locally via the provided scripts and Dockerfiles in the docker directory.

Thanks you ! I had managed to build it locally indeed.


Use tmuxp to create a single launch script

Right now we need to manually attach to each docker container to launch things.
Let's add a single script using tmuxp that lets people launch everything at once.
Ideally the script can load in a .yaml config file that specifies:

omniverse environment
number of agents to spawn

@jfkeller perhaps we can point the isaac sim launch to a specific checkpoint for the USD file

Closed for #111


A11y: Include (and where possible, enforce) best practices for embedded videos

Luckily someone has already done the heavy lifting for us:
[https://www.200ok.nl/tips/youtube-embed/|https://www.200ok.nl/tips/youtube-embed/|smart-link]
[https://www.200ok.nl/tips/vimeo-embed/|https://www.200ok.nl/tips/vimeo-embed/|smart-link]
We need to cross reference what settings are available in these platforms with the ones we must include.
{panel:bgColor=#fffae6}
We should always assume the content creator has editing rights/access to the video they are embedding
{panel}

[public]

FixVersion Change: <IP_ADDRESS>

This issue will be / was solved in Shopware version <IP_ADDRESS>.
[created from NEXT-39088, comment 521690]


use gcc for code quality ci

should resolve #712 by using the same compiler for the build as was used for the static libs

closing in favour of #720 - gcc is much slower than clang for code quality builds, so instead migrating to clang for all parts of linux ci/cd


ERROR https://ops.cdn.reyzar.net/api/1.1/deliveryservices/hostname/edge.baidu.cdn.reyzar.net/sslkeys.json returned HTTP 400!

I add the HTTPS delivery services to Ops and "Queue Updates". The ATS(EDGE) execute /opt/ort/traffic_ops_ort.pl by crontab. But the "remap.config" dost not the HTTPS record(The HTTP record is ok before add the HTTPS delivery services)
I execute the traffic_ops_ort.pl script that show result below:
ERROR Lines for /opt/trafficserver/etc/trafficserver/ssl_multicert.config from Traffic Ops do not match file on disk. ERROR Config file ssl_multicert.config line only in TrOps:	ssl_cert_name=edge_baidu_cdn_reyzar_net_cert.cer ssl_key_name=edge.baidu.cdn.reyzar.net.key ERROR Config file ssl_multicert.config line only in TrOps:	ssl_cert_name=edge_taobao_cdn_reyzar_net_cert.cer ssl_key_name=edge.taobao.cdn.reyzar.net.key ERROR Creating backup of file on disk for ssl_multicert.config. ERROR Creating backup of file in TrOps for ssl_multicert.config. ERROR https://ops.cdn.reyzar.net/api/1.1/deliveryservices/hostname/edge.baidu.cdn.reyzar.net/sslkeys.json returned HTTP 400! ERROR result for https://ops.cdn.reyzar.net/api/1.1/deliveryservices/hostname/edge.baidu.cdn.reyzar.net/sslkeys.json is: ...{"alerts":[{"level":"error","text":"Error  - A record for ssl key -latest could not be found.  Response was: No RIAK servers are set to ONLINE in the database.  Please verify RIAK servers are online and reachable from Traffic Ops."}]}...

Problem has been solved! My "Regular expressions for this delivery service" for HTTPS delivery services was wrong.
before:   .baidu.
after:    ..baidu..
Now, the remap.config have the HTTPS record and I will test HTTPS origin server for my Traffic Control!

Good news!


Dynamic Step Generation

Hi,
I've got a wizard, which ought to determine the steps (and the number of steps) dynamically, depending on a combo box, which the user fills in the first step.
What I have tried:

Add the first step to the wizard.
In the first steps onAdvance method (where I know the value of the combo box), I add more steps, as required.

However, that isn't really satisfying, because the "Finish" button, rather than the "Next" button is enabled when the first step is active. Do you have any suggestions on how to achieve my target?
Thanks,
Jochen

Hi,
maybe as a workaround you could use the get methods to change the captions of buttons to better match your situation?
getNextButton()
getBackButton()
getFinishButton()
getCancelButton()


Update summer

Commits explain what is what. Fixed some bugs and changed the design. Remember to include Green Bow's icon, Colorless Breath's icon and to swap the Morgans portraits.

I can't seem to get Custom units icon working... It stays the same if you change movement type and the same happens on the site. The unit gets properly modified though... Do you have any idea on what might be causing this?

Custom unit icon as in their avatar?
I think the avatar was always a silhouette for custom units in previous versions. I may not be understanding your question correctly though.

Changing these ones doesn't change the respective icons, both on this pull request and on the site. (The calc updates fine though)

Ahh ok, Ill have to look into this. Custom units have a bunch of edge casey
code that I've not changed much since the legacy version.
On Sun, Jul 8, 2018, 12:05 PM Lorenzooone<EMAIL_ADDRESS>wrote:

[image: cattura]
https://user-images.githubusercontent.com/7537905/42422171-b3ee18ce-82e1-11e8-8590-387bfe5ecd8b.PNG
Changing these ones doesn't change the respective icons, both on this pull
request and on the site. (The calc updates fine though)
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/winsomniak/kagero-calc/pull/109#issuecomment-403301596,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AK3mrU4uKC48qkQDHvneJm-eNq2_8baBks5uEjvRgaJpZM4VGdGD
.


Drop code old CLI remoting support

Jenkins 2.54 was released in 2017, which is long EOL. Current versions have even dropped support for the old remoting.

Fixed tests so they now only fail in the same way as master. Merging since there's been no code changes outside of tests since the approval.


Error on run IOS demo-ng

When I run:
tns run ios
On /nativescript-masked-text-field/demo-ng
I get this error:
Unable to install dependencies. Make sure your package.json is valid and all dependencies are correct. Error is: Command npm failed with exit code 254

On package.json, change the following line
"nativescript-masked-text-field": "file:../bin/dist",
By
"nativescript-masked-text-field": "^1.0.0",
And it worked!


Spoticraft does not work with feather client. Please do not use feather client as it steals mods.

          Spoticraft does not work with feather client. Please do not use feather client as it steals mods.

Originally posted by @mineblock11 in https://github.com/mineblock11/Spoticraft/issues/13#issuecomment-1631253939
Why not just fix the mod so that it can work w feather?

Because feather steals mods. Please do not open another issue or you will be blocked.


Update tests

Adds Tests Explorer for VSCode, converts Jasmine tests to Mocha (because all my repos should use the same testing library, it's easier to maintain), and replaces Puppeteer by Playwright (to test against more rendering engines).
However for some reason, Webkit in Github Actions is failing at the moment, whereas the exact same tests run fine on Windows 10, so I'm looking into this.

I'm going to disable Webkit tests for now (Chromium + Firefox is already an improvement over Puppeteer anyway) because it appears the Ubuntu image is missing a library Webkit needs, so this need further research:
/home/runner/.cache/ms-playwright/webkit-1269/minibrowser-wpe/MiniBrowser: error while loading shared libraries: libEGL.so.1: cannot open shared object file: No such file or directory

I'll make just one quick test to see if installing libegl1 manually in the Action would solve the issue. However, even if it were to solve that one error, I suspect it probably misses several libraries. Still, it's worth a try.

Well, it solves libEGL.so, but now it misses libopus.so. Therefore yes, there are several dependencies missing.
However I just noticed, while searching for their Dockerfile (to find the list of dependencies they use), that Microsoft has an official Github Action, perhaps this would install the dependencies ?
https://github.com/microsoft/playwright/blob/master/docs/ci.md#github-actions
steps:
  - uses: microsoft/playwright-github-action@v1
  - name: Run your tests
    run: npm test

Ha, the action worked to fix the dependencies issue :tada:
Therefore I'm going to re-enable the other tests.


feat: add the docs for 'kcl.mod'.

1. Does this PR affect any open issues?(Y/N) and add issue references (e.g. "fix #123", "re #123".):

[ ] N
[x] Y

fix issue https://github.com/kcl-lang/kpm/issues/132

2. What is the scope of this PR (e.g. component or file name):

README-zh.md
README.md
docs/kcl_mod-zh.md
docs/kcl_mod.md
3. Provide a description of the PR(e.g. more details, effects, motivations or doc link):

[ ] Affects user behaviors
[ ] Contains syntax changes
[ ] Contains variable changes
[ ] Contains experimental features
[ ] Performance regression: Consumes more CPU
[ ] Performance regression: Consumes more Memory
[x] Other

4. Are there any breaking changes?(Y/N) and describe the breaking changes(e.g. more details, motivations or doc link):

[x] N
[ ] Y

5. Are there test cases for these changes?(Y/N) select and add more details, references or doc links:

[ ] Unit test
[ ] Integration test
[ ] Benchmark (add benchmark stats below)
[ ] Manual test (add detailed scripts or steps below)
[x] Other

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
21 unchanged lines in 2 files lost coverage.
Overall coverage remained the same at 39.835%

Files with Coverage Reduction
New Missed Lines
%

pkg/cmd/cmd_run.go
9
0%

pkg/cmd/cmd_pull.go
12
0%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
1011

Relevant Lines:
2538

💛  - Coveralls


chore(main): release 1.0.2

:robot: I have created a release beep boop
1.0.2 (2024-06-10)
Bug Fixes

goreleaser: bump to v2 (9e89bbf)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/lederniermetre/talosctx/releases/tag/v1.0.2 :sunflower:


SRA and Premade HISAT index

PR checklist

[ ] This comment contains a description of changes (with reason).
[ ] If you've fixed a bug or added code that should be tested, add tests!
[ ] If you've added a new tool - have you followed the pipeline conventions in the contribution docs
[ ] Make sure your code lints (nf-core lint).
[ ] Ensure the test suite passes (nextflow run . -profile test,docker --outdir <OUTDIR>).
[ ] Check for unexpected warnings in debug mode (nextflow run . -profile debug,test,docker --outdir <OUTDIR>).
[ ] Usage Documentation in docs/usage.md is updated.
[ ] Output Documentation in docs/output.md is updated.
[ ] CHANGELOG.md is updated.
[ ] README.md is updated (including new tool citations and authors/contributors).

I don't understand how you are determining if a sample is paired end for SRA input.  What does the csv sheet look like in that case?  will there always be one SRA ID or 2?  if you use prefetch/fasterqdump will it give you 2 files from one SRA ID ?


The use of JSR310Module in the JacksonSerializer should be replaced with the JavaTimeModule

com.fasterxml.jackson.datatype.jsr310.JSR310Module is deprecated since jackson version 2.5. Axon 3.0.5 uses jackson 2.8. The Jackson documentation suggests using com.fasterxml.jackson.datatype.jsr310.JavaTimeModule instead.

Fixed in PR #367