redis adapter

Summary
I want to be able to use feathers-sync with redis to distribute events across nodes.
Changes

[x] move lib => src to follow newer repos
[x] rename package to sync
[x] rename repo to feathers-sync
[x] clean up debug messages
[x] changed interface to sync('redis', {}) where first argument is the adapter, the second is options
[x] updates to readme
[x] existing tests pass
[x] new tests

Travis looks like it just needs redis added and the existing tests should still pass.
When I tested mongo vs redis, redis seemed to be significantly faster
Need to clean up the way the pub / subs are created, might have to build an array of clients

This is awesome, thank you! It looks like Travis CI does not have a Redis server running. Adding
services:
  - redis-server

To .travis.yml should fix it. Once the tests are passing I will merge and after that do a little bit of refactoring I've been planning on for a while already (e.g. all our up-to-date plugins have been migrated to ES6).

Cool, are you happy with it the way it is based on the notes I added above? I'm happy to leave those be if you have some bigger refactors you want to throw down.
I'll add the redis-server now.

I'm adding tests for redis - don't merge yet

Nice, the tests passed. I'd say let's merge it, I'll migrate everything to ES6 and then we can iterate to see if there are ways to improve it. Sounds good?

Well once you added the tests I mean.

Yeah I'm just fixing a couple things real quick on test coverage. I'll ping you shortly.

@daffl ok i'm done! merge away.

btw, it should be ready to be npm published to feathers-sync now if you want to release it there

This is so awesome!  Thanks for adding this.

Perfect. I'll update it to ES6 later today. One thing I may add to the refactoring is removing the redis and mubsub dependencies and passing them in your application. Something like:
var sync = require('feathers-sync');
var redis = require('redis');

app.configure(sync(sync.redis(redis)));

That way this module doesn't install dependencies you don't use (we've done a similar thing for the database adapters).

Awe yeeeaaaah! Thanks @slajax!

@daffl I agree with your suggestions. I was thinking exporting them directly would make sense initially because they probably have different options arguments but also it would be an even more ideal scenario to abstract out the dependencies to other repos to save installing mubpub if you use redis so :+1:
Will the current version be published to NPM as a 0.2.0 or do you intend to do these refactors first? My reason for asking is purely selfish, I want to use the current version in my builds up until that refactor is ready.


Fixing typos in .eleventy.js

Thanks @edjw!
I mopped these up (and probably added other typos!) as part of some other development work on this. So I'm closing this PR without merging.
Thanks for the sharp eyes!


Fix server data folder not created & test races

What was changed

Fixes temporal server start-dev not creating the folder for Data/DB
Why?

The issue with temporal server start-dev not creating the data folder was exposed by merging https://github.com/temporalio/tctl-kit/pull/35 : the env feature was creating the folder so the tests were succeeding
Checklist

Closes

How was this tested:

Unit tests

Any docs updates needed?

https://github.com/temporalio/temporal/issues/3708 seems to be causing the nil pointer exception with SQLite driver

flakiness comes from https://github.com/temporalio/temporal/issues/3708
merging


Readthedocs .rst page for scenarios is not displaying module methods

This code at the bottom of the docs page /docs/scenarios.rst should display all the methods for Scenario and ScenarioList:

But they are currently not being rendered:

Methods in docs pages for models, questions and surveys are rendering with the same code:
https://docs.expectedparrot.com/en/latest/language_models.html
https://docs.expectedparrot.com/en/latest/questions.html
https://docs.expectedparrot.com/en/latest/surveys.html
When this problem has occurred before it seemed to be fixed by running make docs.

When I run make docs-view I do see all the content locally.

I do see the content locally:

it's fixed


Let's not show private keys... K thx

Publicly publishing private keys isn't a good idea, is it?

The creds are invalid

Yea, cuz I regenerated them...

what keys


Upgrade Gradle to 5.3 on Travis CI and AppVeyor

Maybe this is the wrong place, but I recently stumbled across gradlew, the Gradle wrapper again and I am wondering now why we do not use this instead of the local Gradle version.
Using the Gradle wrapper would no longer require everyone to install Gradle locally for building Alpha and we would be sure that it always builds with the same Gradle version.
What are the arguments against it?

Using the Gradle wrapper would no longer require everyone to install Gradle locally for building Alpha and we would be sure that it always builds with the same Gradle version.

Correct.

What are the arguments against it?

The downside that I see is that since the Gradle Wrapper is implemented in Java (./gradlew runs java -jar gradle/wrapper/gradle-wrapper.jar), which requires a JAR being checked in to the Git repository. That's something I'd like to avoid.

I lean towards using the Gradle Wrapper, because:

we do not push binary releases often, thus the only way to get a recent version of the Alpha is to build it directly from master.
Gradle is not installed in most standard OS distributions, making it another hurdle for someone just trying Alpha (or for anyone having to switch work environments -- like I did in the last year).

Since I would like to remove as many hurdles as possible, the cost of having a tiny .jar in the repo seem negligible to me.

See #179, which IMO we should deal with after #178 and #176 are merged.

The AppVeyor build fails

I fixed this by also installing Gradle 5.3 on AppVeyor.


[BUG]

1.20.4 server
does not accept special characters in the password $ # @ ! and so on

You should enclose it with quotes in register command:
/register "temp*" "temp*"
/login temp*


refactor($compile): avoid catastrophic backtracking when parsing bindings

This isn't expected to have any actual impact, since AngularJS is only intended to be used in the browser (not the server) and for this RegExp to be exploited by malicious user code the developer would have to have to give the user rights to execute arbitrary JavaScript code anyway.
Fixing as a general good practice and to avoid encouraging use of a similar RegExp in other environments where it might actually matter.

@gkalpak - did you cherry-pick to 1.6.x too?

Yes.
master: c0adcc3a4
v1.6.x: b30a925d5


AWS S3 get_bucket_location fails with Access Denied

Description:

What issue is being seen?  Describe what should be happening instead of the bug, for example: Cartography should not crash, the expected value isn't returned, the data schema is wrong, etc.

S3 get_bucket_location() can fail with a permissions error. Given that as much of cartography should run with only SecurityAudit, we should handle this exception gracefully.
To Reproduce:

Steps to reproduce the behavior.  Provide all data and inputs required to reproduce the issue.

Run the s3 sync on an account that does not have get_bucket_location permissions.
Logs:

If applicable, copy and paste your console log with the failing stack trace.

  File "/{PATH/cartography/intel/aws/__init__.py", line 98, in _sync_multiple_accounts
    _sync_one_account(neo4j_session, boto3_session, account_id, sync_tag, common_job_parameters)
  File "/{PATH/cartography/intel/aws/__init__.py", line 29, in _sync_one_account
    s3.sync(neo4j_session, boto3_session, account_id, sync_tag, common_job_parameters)
  File "/{PATH/cartography/util.py", line 58, in timed
    result = method(*args, **kwargs)
  File "/{PATH/cartography/intel/aws/s3.py", line 361, in sync
    bucket_data = get_s3_bucket_list(boto3_session)
  File "/{PATH/cartography/util.py", line 58, in timed
    result = method(*args, **kwargs)
  File "/{PATH/cartography/intel/aws/s3.py", line 21, in get_s3_bucket_list
    bucket['Region'] = client.get_bucket_location(Bucket=bucket['Name'])['LocationConstraint']
  File "/{PATH/botocore/client.py", line 357, in _api_call
    return self._make_api_call(operation_name, kwargs)
  File "/{PATH/botocore/client.py", line 676, in _make_api_call
    raise error_class(parsed_response, operation_name)
botocore.exceptions.ClientError: An error occurred (AccessDenied) when calling the GetBucketLocation operation: Access Denied

Please complete the following information::

Cartography release version or commit hash [e.g. 0.12.0 or 95e8e11913e2a44a4d4682506d8364a638ceac69]

0f71b3f0246665d5fa065afa2e3dc46c22d6c689
Additional context:

Add any other context about the problem here.

This is an unintended side effect of https://github.com/lyft/cartography/pull/470.

cc: @shabin-slr FYI. I think I have a quick fix but I haven't tested (tied up with other things right now). If you have time and are able to write a fix and test it faster than me that'd be super helpful.

I tested this and it should work  now.

@achantavy, SecurityAudity policy has an Allow rule "s3:GetBucket*". Isn't this necessary to call "s3.get_bucket_location()" ?


Oracle Connection

Hello, I would like to seek help if  someone manage to have oracle ETL connection to wherehows and successfully loaded data to UI. Thanks for the assistance.

manage to crawl data to oracle source. Closing now the issue.


feat(memo): Cache timeout

Proposed changes (including videos or screenshots)

Issue(s)

Further comments

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.1 out of 2 committers have signed the CLA.:white_check_mark: tassoevan:x: guijun13You have signed the CLA already but the status is still pending? Let us recheck it.


Better OpenMP

Two main things:

use schedule(dynamic,1) for FD loops (delivers better performance than schedule(static,1) on all tested xeons)
use schedule(dynamic,chunk_size) for sparse loops, where chunk_size is number_of_iterations / (nthreads * alpha) where alpha is currently set to 3 (found empirically)

Would appreciate some performance testing

@georgebisbas :

Did you try with autotuning off?

yes.
@mloubout, @georgebisbas

branch is green. You've got a few days for your experiments (thanks!). When happy, please poke me and I'll land this

Does seem to improve performance on  my machine (tested mostly 3D tti, rest is cheap :D)

Merged.


Heading stylings translated from # in markdown are too large

Traditionally, we've used one # for the top heading and ## and ### for additional headings on a markdown page. With recent style guide updates, this looks too large on the page. Perhaps we take # down to H2 or maybe even H3 and so on.
This came up in comments in https://github.com/mapzen/documentation/pull/283. Adding here because it's an issue across all docs.

@rfriberg noted that the documentation pages have two <h1> tags on them, one for the text in the top banner and one for the first heading on the page. This can cause some search engine optimization issues.
"An ideal H1 will be specific and relevant to the page. What we could do is make the banner text a different heading tag and make the first (actual) heading on the page an H1 (shrunk down a bit if that's what's desired)."

@rmglennon - I'm trying to assign this to @meghanhade .  Does she need to be added to some user list on this repo?

👋 I'm here to help

see https://precog.mapzen.com/mapzen/documentation/meghan-heading-styling/documentation/overview/ for a preview

The styleguide changes are in this PR: https://github.com/mapzen/styleguide/pull/477

Preview looks good.  My only concern is that we now have the same font size (24px) for both h2 and h3 (except on mobile).

@rfriberg I'll fix that.

@rfriberg @rmglennon I bumped the h3 font-size down and added in the padding-top for the documentation-header as well. These changes are on the meghan-heading-styling branch.

Looks good to me. Thanks!

Thanks @meghanhade. pull request when ready!

@rmglennon @rfriberg Final step--now that this styling has been merged in on the styleguide, I've removed the hard-coded css from the documentation file. If you could take one last look and approve, I'll merge.

fixed over in https://github.com/mapzen/styleguide/pull/477


Icon overlay support

Give ability to assign multiple icons to some targets so that we can add overlay to existing icons.
Purposal:
"fileNames" : {
    ".editorconfig" : ["file", "overlay.config"], // Assign a multiple-layer icon
    ".test." : {"overlays" : "overlay.test"},     // Assign an overlay to existing icon
    ".tests." : {"overlays" : "overlay.test"},
}

To keep the number of issues in our inbox on a manageable level, we're closing issues that have been on the backlog for a long time but haven't gained traction: We look at the number of votes the issue has received and the number of duplicates issues filed.
If you disagree and feel that this issue is crucial: We're happy to listen and to reconsider.
Maybe you want to give it a try yourself and provide the feature through an extension or a PR: We're happy to give advise and get you started.
Thanks for your understanding. Thanks for helping and happy coding!


Scrollbars should not go into header row

In a number of places, including Recent topics and settings tables, we have scrollbars that go into the header row. This is incorrect, as the header row is not scrolled. We should fix this.

if this issue is open, can i work on it ?

@zulipbot claim

This is likely the same issue as #17933; we use the same type of table component in both settings and recent topics. That other issue has some useful discussion about what makes fixing this nontrivial.

Closing as a duplicate of #17933.


info about that Function!

hi, working with AGK-ATOM i noted that it will be good to have information about the fields of the functions im editing.
some functions, you don't remember what data go in what place, so, i go to show in a image from IDeal with blitz3D.
sorry my english!!
regards!!!!!

While this is a nice feature, I think the autocomplete + go to definition should be enough to know the arguments needed for a function.
Also, now that there is a symbol explorer, it tells you which arguments are expected for each function. So I think this is a bit redundant.

hi, yes but, autocomplete works if you creating, but if you go to edit,
there is no info about the function.
regards!
El sáb., 11 jul. 2020 a las 17:14, Federico Ramirez (<
<EMAIL_ADDRESS>escribió:

While this is a nice feature, I think the autocomplete + go to definition
should be enough to know the arguments needed for a function.
Also, now that there is a symbol explorer, it tells you which arguments
are expected for each function. So I think this is a bit redundant.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/gosukiwi/atom-agk/issues/25#issuecomment-657122784,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AMP5FVLBCV3WMGUQYPBYSWDR3DB2PANCNFSM4OS6FKFA
.

Mmm true, but at least the Symbol Explorer shows info on all functions in the current file, you can look it up there. While it's not as convenient, I'd really like to avoid changing the status line.


Revert "Deprecate Legacy driver by removing incremental builds and adding warnings"

Reverts apple/swift#64656

@swift-ci smoke test

@swift-ci test Windows platform


fix: Add support for unresolved fields from mixins

Motivation
Mixins are used very frequently. We should have better code nav for them.
TODO

[ ] Manually test Go to Def and Find References when an undeclared field is present in different files. (Currently, this will create two different SymbolInformation values for the same symbol in different files.)

[ ] Manually test an inheritance hierarchy involving multiple classes. In that situation, it seems like the transitive linkage should be happening through relation traversal. E.g. if C < B < A then C.@f would be defined by B.@f which would be defined by A.@f. In that case, we should check if all references to A.@f show up when we do Find references on C.@f. There is a TODO related to this, which should be updated/removed.

[x] Remove debugging statements

[x] Right now, we emit a SymbolInformation is emitted when a definition is emitted, but it is possible for a field inside a class coming from a mixin to not have any definition inside the class. Consider:
module M
    def set_f_0
        @f = 0
    end
end
class C
    def set_f_1
        @f = 1
    end
end
class D < C
    include M
    def get_f
        @f
    end
end

The code is currently set up so that:

The reference inside a class will emit a symbol for the further ancestor which mentions the class. (However, strictly speaking, we don't require a definition inside that ancestor's body right now... -- but it would be strange for a superclass to access a field that it never sets, so...). In this case, the occurrence for @f in get_f will use C#@f. (← Not sure if this is a good idea though, see next point.)
We emit a reference relationship for symbols that are mentioned (transitively) in mixins when emitting the definition. In this case, there is no definition for @f inside D, so no relationship is emitted. (Additionally, it is not guaranteed that we can modify C#@f's SymbolInformation, since C may be coming from a different index altogether. -- This makes me question if we should be instead using D@#f for the occurrence of @f instead.)

It is unclear to me what exactly we should do. There are a couple of options:

Emit a fake definition for D#@f (what location should be used?), and attach reference relationships to it which connect it to M#@f and C#@f.
Emit 2 occurrences for every @f in D, one for C#@f and another for M#@f.

[x] More tests, including for complex include chains

Test plan

See included automated tests.

Something seems to be off with the ref panel, preventing manual testing. Let's merge this first... I'll file a follow-up issue for manual testing. https://github.com/sourcegraph/sourcegraph/issues/42499


Task System

Currently LÖVR is still pretty much single threaded.  On multi-core machines the framework is only able to take advantage of a tiny fraction of the total power of the machine.  Moving to modern graphics APIs also especially benefits from being able to do rendering work on multiple threads.
The lovr.thread module lets you create threads and communicate between them, but it's pretty difficult to use.  Also, giving people control over threads directly has a lot of pitfalls.  Often times in this setup, people will create too few or too many threads, and getting an optimal setup that minimizes stalls, starvation, context switching, bugs, etc. is difficult or impossible.
Instead of a thread-oriented approach, a higher level task-oriented approach has some advantages.  In this setup, LÖVR would internally create a global pool of worker threads (usually one for every core on the machine), and tasks could be scheduled to run on the threads.  In Lua, a task would just be a function plus argument(s), which would get dumped to bytecode and loaded on whichever threads it runs on.  A scheduling system would ensure that tasks are distributed among the threads.  Some very simple pseudocode:
function lovr.load()
  lovr.thread.schedule(function()
    while true do
      -- some task that runs once per second, like
      -- checking file modification times for reloading

      -- coroutines could let tasks sleep or wait
      -- for something else to finish
      coroutine.yield(1)
    end
  end)
end

function lovr.update(dt)
  -- the update function could be broken up into
  -- lots of different tasks that get run on threads
  -- these tasks could even spawn other tasks

  -- simple fork/join to update objects in parallel chunks of 100
  local handle = lovr.thread.schedule(function(objects)
    for i = 1, #objects, 100 do
      lovr.thread.schedule(updateObjects, { i, i + 99 })
    end
  end)

  -- do some other work while objects are updating

  lovr.thread.wait(handle)
end

The advantages of this are:

Lua code wouldn't need to create and manage threads.  Instead regular functions are scheduled, which is easier to use and understand.
The number of threads is abstracted, you could use a conf.lua flag to switch between single-threaded tasks and saturating all the cores.  People don't have to worry about creating the correct number of threads.
A higher level task system would allow for declaratively defining task dependencies.  You could schedule something and say "by the way, wait for the frustum culling task to finish before you do this", instead of needing to explicitly set up channels or signals.

Restrictions/Challenges

Sharing Data - In C it's pretty easy to give data to another thread since every thread shares an address space.  In Lua it's way harder because every thread has its own Lua state and tables can't be shared without performing deep copies to and from C.  I don't really see any way of working around this sort of thing, other than accepting it and encouraging people to use LÖVR objects to hold their shared stuff.

Somewhat related, because tasks are dumped to bytecode and will run in an arbitrary Lua state, they can't use globals or upvalues.  So they can really only communicate via Channels, side effects, or potentially arguments/returns serialized between the Lua states.

Timing Model? - Still can't really wrap my head around whether you would want to stick exclusively to short-lived tasks that get scheduled/joined every frame, or long-running coroutines that are scheduled once and just yield to wait for the next frame.  Maybe a mixture of both could work.  In general, figuring out when the scheduler actually decides to run coroutines for the next frame is pretty confusing.
Synchronization - Being able to wait for other stuff to finish is important, but some of the details are still fuzzy.  If you pass a big set of tasks to a single schedule invocation, with dependency information, we are able to figure out the DAG and mostly do the waits for you.  But if those are spread out over multiple calls to schedule, or if they need to synchronize with long-running coroutines, things get a lot more difficult.  Maybe raw Signal objects could be exposed that wrap condition variables, but this becomes significantly more difficult to use and possibly dangerous.

There is a job system used internally now, and it is increasingly being used to parallelize internal work, such as compiling GPU pipelines, recording GPU render passes, loading assets asynchronously in the background, and physics updates.
This doesn't solve the original intent of this issue (user-facing task-based multithreading using coroutines).  However, I think the coroutine scheduler is a lot more tricky to design and pull off.  The internal job system lets us realize benefits of threads in the most common cases, so I think we'll stick with that for now.


chore: release helm-cosmo-router 0.8.0 🚀

:robot: I have created a release beep boop
0.8.0 (2024-09-05)
Features

enable using HTTP(S)_PROXY in router  (#1136) (4600fdf)

This PR was generated with Release Please. See documentation.

:robot: Created releases:

<EMAIL_ADDRESS>:sunflower:


mafSurvival plot does not correspond with real time

I am working with maftools survival analysis function (mafSurvival), and the output results I am getting do not correspond with the data I have. I am concerned about how the comparison is made, as for example, if I have under "Hugo_Symbol" altered KRAS variants in my maf file, does it compare survival against the rest of the samples that do not appear with this Hugo_Symbol (KRAS) in the same column, thus corresponding to the wild-type KRAS?
I am running:
maSurvival(maf, genes = 'KRAS', time= 'months', Status = 'event_death', addInfo = TRUE)
The resultant plot is accurate regarding the time for the mutant KRAS, but the time for the wt KRAS on the x axis does not represent the time for the rest of samples with no KRAS alteration.
Many thanks,

Hi,
Yes, it compares those samples where KRAS appears as mutated against those where it doesn't (WT).
Is it possible to provide a reproducible example ?


[feature] Enable listing controls of a specific framework

When I run kubescape list controls I see a list of all of the controls. I will like to list controls of a specific framework.
I suggest Kubescape should add a column with the framework name and maybe even an option for listing controls of a specific framework.
e.g. Adding a column for frameworks:
+------------+---------------------------------------------------------------+-----------------------+-----------------------+
| CONTROL ID |                         CONTROL NAME                          |                  DOCS                 |       FRAMEWORKS        |
+------------+---------------------------------------------------------------+-----------------------+-----------------------+
| C-0001     | Forbidden Container Registries                                | https://hub.armosec.io/docs/c-0001     |      NSA, MITRE             |
+------------+---------------------------------------------------------------+------------------------+----------------------+
| C-0002     | Exec into container                                           | https://hub.armosec.io/docs/c-0002              |      NSA                          |
+------------+---------------------------------------------------------------+------------------------+----------------------+
| C-0004     | Resources memory limit and                                    | https://hub.armosec.io/docs/c-0004     |                                       |
+------------+---------------------------------------------------------------+------------------------+----------------------+

e.g. Listing controls of a specific framework
kubescape list controls --framework NSA

The output table should be the same as the command without the flag.

Hi, I was interested on working on this issue

Hi, I was interested on working on this issue

hey JusteenR are you working this isuue ?


open from jquery mobile popup

mobipick does not open from text input which is located inside jquery mobile popup.

This is beyond the scope of the plugin. Feel free to submit a pull request.

I have the same problem... and i need to open the mobipick datepicker from it...
I'm compelled to change datepicker even if this one is really great :(
Hope to see this functionality sooner or later


[GENERAL]: Issue with the dark mode in FAQ section

Topic

Upon changing to dark mode, the FAQ section doesn't completely changes into dark mode

light mode

dark mode

Details

Fixing this issue would help in maintaing the consistency

@07sumit1002


Keycloak 19 fix realm name

Motivation
Fixed #143
In Keycloak 19 the 'realm' in the metrics contain a UUID instead of the Realm name.
Apparently Keycloak changed the value of event.getRealmId() from a Realm name to a real Realm id.
What
This change will fix this by looking up the Realm by Id via a RealmProvider so the metrics will still contain a Realm name.
Upgraded Keycloak dependency to 19.0.1 to also verify this library is not using any deprecated sources.
Removed the MetricsEventListenerSpi since this seems to be an old way of adding a listener and is not needed anymore for v19 Quarkus Keycloak.
Why
To have readable realm names in the metrics and on the Keycloak dashboard.
How
Realm name is looked up by using a RealmProvider which is accessible from the KeycloakSession when the MetricsEventListenerFactory is created.
Verification Steps

Build the package, include it in a custom Keycloak 19+ image and start the container.
Login into the admin console
Enable metrics-listener for events in the Realm settings.
Logout
Log back into admin console
Run curl -vk https://<keycloak-url>:<port>/auth/realms/master/metrics | grep -v '^#' | grep login and check metrics output.
The metrics should still contain the realm name like

keycloak_logins{realm="admin",provider="keycloak",client_id="account-console",} 1.0

Checklist:

[x] Code has been tested locally by PR requester
[x] Changes have been successfully verified by another team member

Progress

[x] Finished task
[ ] TODO

thanks @diversit
Does that mean that at the moment using Keycloak 19, you would see UUIDs in the metric labels instead of a realm name?

@pb82 Yes, with the current 2.5.3 version on KC 19 the metrics label will contain a UUID for the realm instead of the name. This change fixes this.

@diversit is this backward compatible with older Keycloak versions?

We're currently in the process of migrating our Keycloak instance to 19 (however still using the Wildfly distribution). I'll apply this patchset without the SPI-file removals and see how they work for us. Since support for Wildfly will get dropped in version 20, rather than version 19, maybe it's best to hold off on removing those files for now?

So far, I haven't noticed any problems in our test bed environment with this patch set applied.

Can this be merged?

@diversit yes, I'd like to verify this against KC 21, once we got the compatibility PR merged. Afterwards, this can be merged too.

@diversit I've rebased your changes and merged them in #159
Thanks!


More tests for proper sort order

So changes can be made with more confidence, covered
by more tests.
One suite here was taken from Stanford code, we can't actually
sort everthing they want as wanted at present, commented some out.

Thanks!


update fifthpost

👷 Deploy Preview for johare-cv processing.

Name
Link

🔨 Latest commit
56f1df5f3a733848aa13ae3fbdecaf85d1fdcbc3

🔍 Latest deploy log
https://app.netlify.com/sites/johare-cv/deploys/65db8b3dbacb2f0008169766


Admin: Blueprints & Taxonomies error

Hi there, I think I found a bug with the admin page editing. I created a blueprint which contains Taxonomies. Taxonomy is also in the Options tab, so technically it's on the page twice.
Behaviour:

Go to the custom tab and edit the taxonomy
Remove a tag
Press Save
Saved, but tag still present. If you press save and do nothing, the tag is added twice (to the page file)

Blueprint:
title: Item
'@extends':
    type: default
    context: blueprints://pages

form:
  fields:
    tabs:

      fields:
        blog:
          type: tab
          title: TAB

          fields:
            header.taxonomy:
              type: taxonomy
              label: PLUGIN_ADMIN.TAXONOMY
              multiple: true
              validate:
                type: array

version: Grav v1.3.2 - Admin v1.5.2

Actually this error persists even though the duplicate taxonomy is removed.
It isn't possible to remove taxonomies from the admin panel

Did you try to unset the taxonomy field: https://learn.getgrav.org/forms/blueprints/advanced-features#removing-fields-properties-unset

Hi @rhukster. I removed my blueprint, however the taxonomy was still duplicating the tags and was unable to remove them

@rhukster , @matejkramny , It's happening the same to me , does this have any kind of solution?

same here


Black a Screen In android 9 UI "One UI 1.0"

My app works fine in android 4.2.2 but  in android 9 UI "One UI 1.0"  phone  Samsung galaxy j 4 pluse debug mode get this exception
The GPU does not support this version of OpenGL ES, try an older version
the app when I press in button location its work  but all element, not showing
Flutter (Channel stable, v1.9.1+hotfix.2, on Microsoft Windows [Version 10.0.17763.737], locale en-US)
• Flutter version 1.9.1+hotfix.2 at C:\flutter
• Framework revision 2d2a1ffec9 (4 weeks ago), 2019-09-06 18:39:49 -0700
• Engine revision b863200c37
• Dart version 2.5.0

Hi,
I'm sorry to hear that. I'm pretty sure this is fixed on the beta and master channels if you want to give that a try.
I'm sorry that I can't say when it will be in stable.
related https://github.com/flutter/flutter/issues/40150

how can I know what channel im using and how can switch between them

Please see https://github.com/flutter/flutter/wiki/Flutter-build-release-channels
After changing do a flutter upgrade

Any news on this matter?
I am facing the same issue.
Thanks in advance.

Fixed.
Solution: check the flutter channel you are on and switch to master.. using:
to check your channel:  flutter channel
to switch channel: flutter channel master
after that run this: flutter upgrade


Portuguese model

It doesn't work with the portuguese model, do you have any idea why?

closing this issue as this repo will no longer be actively maintained and be put in read-only mode


⚠ Bofrost.be has degraded performance

In 21f919f, Bofrost.be (https://www.bofrost.be) experienced degraded performance:

HTTP code: 200
Response time: 1545 ms

Resolved: Bofrost.be performance has improved in 240ea36 after 17 minutes.


Add configuration file to allow flexible table name

See PR #3

Wonderful solution, pull-request merged. Thanks a ton!


Run flake8 on tests

PR Checklist

[x] A description of the changes is added to the description of this PR.
[ ] If there is a related issue, make sure it is linked to this PR.
[ ] If you've fixed a bug or added code that should be tested, add tests!
[ ] Documentation in docs is updated

Description of changes
Run flake8 on tests to run the same checks as we run on the code and prevent mistakes. Also updating mypy configuration so that it runs on scripts.

@mkniewallner This MR now has a small merge conflict due to me merging your other PR's.

Codecov Report

Merging #147 (c3b64d1) into main (243ff7a) will decrease coverage by 0.0%.
The diff coverage is 100.0%.

@@           Coverage Diff           @@
##            main    #147     +/-   ##
=======================================
- Coverage   92.4%   92.3%   -0.1%
=======================================
  Files         39      39
  Lines       1354    1342     -12
=======================================
- Hits        1252    1240     -12
  Misses       102     102

Impacted Files
Coverage Δ

tests/test_core.py
100.0% <ø> (ø)

tests/test_notebook_import_extractor.py
100.0% <ø> (ø)

tests/test_pyproject_toml_dependency_getter.py
100.0% <ø> (ø)

tests/test_requirements_txt_dependency_getter.py
100.0% <ø> (ø)

tests/test_result_logger.py
100.0% <ø> (ø)

tests/test_transitive_dependencies_finder.py
100.0% <ø> (ø)

tests/test_cli.py
100.0% <100.0%> (ø)

tests/test_dependency.py
97.1% <100.0%> (ø)

tests/test_import_parser.py
100.0% <100.0%> (ø)

tests/test_module.py
100.0% <100.0%> (ø)

... and 1 more

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

@mkniewallner This MR now has a small merge conflict due to me merging your other PR's.

Conflicts have been resolved.


Take inventory of all BeagleBones and add to wiki:

We need to account for all of the BeagleBones that we have now.
We just received a donation of 4 broken beaglebones from Josh Stevens( #5-#8) and I managed to fix #5, I got the rest to work on  HDMI but can't get any other connectivity out of them.
If you see any beaglebones without a number on them, please add a number and update the wiki.
https://github.com/IEEERobotics/bot/wiki/Inventory

They should all be cataloged by now.


[devicelab] mark uncaught exceptions as flaky

Description
Running pub more often should not cause this test to fail, assuming a problem with the test itself. Example output:
2020-10-23T11:12:42.529120: [        ] Artifact Instance of 'FlutterRunnerSDKArtifacts' is not required, skipping update.
2020-10-23T11:12:42.529228: [        ] Artifact Instance of 'FlutterRunnerDebugSymbols' is not required, skipping update.
2020-10-23T11:12:42.609343: [  +79 ms] Running "flutter pub get" in image_loading...
2020-10-23T11:12:42.612634: [   +3 ms] executing: [/home/flutter/.cocoon/flutter/dev/integration_tests/image_loading/] /home/flutter/.cocoon/flutter/bin/cache/dart-sdk/bin/pub --verbose get --no-precompile
2020-10-23T11:12:42.709773: [  +56 ms] MSG : Resolving dependencies...
2020-10-23T11:12:43.010775: [   +2 ms] MSG : Overriding the upper bound Dart SDK constraint to <=2.11.0-242.0.dev for the following packages:
2020-10-23T11:12:43.011213: [        ]     |
[        ]     | async, boolean_selector, characters, charcode, collection, js, matcher, meta, path, pedantic, pool, source_map_stack_trace, source_maps, source_span, stack_trace, stream_channel, string_scanner, term_glyph, test, test_api, test_core, typed_data, vector_math
[        ]     |
2020-10-23T11:12:43.011537: [        ]     | To disable this you can set the PUB_ALLOW_PRERELEASE_SDK system environment variable to `false`, or you can silence this message by setting it to `quiet`.
2020-10-23T12:13:39.579859: Task execution finished

Task failed with the following reason:
Timeout waiting for task completion: Future not completed

Landing TBR to fix tree


Consul services always return 404?

Hi, there,
I'm using the latest version of spring cloud and consul, http requests from spring-cloud-gateway always return 404, seems cannot find the right path from lb services?
spring cloud version: Finchley.RC1
spring boot version: 2.0.2.RELEASE
main project: https://github.com/kopstill/peach
spring cloud config: https://github.com/kopstill/peach-config

Got it, it's the path problem, use "filters -> SetPath" would solve it.

nice! Solve the problem that has troubled me for 2 days.


MF:Schematics to configure webpack

@skocharian commented on Wed Sep 30 2020
Once MF: Resources & scss  is done we need to automate this with set of  schematics.

Create schematics that can generate Webpack configuration based on the angular JSON
-- New project:  @fundamental-ngx/app-shell:ng-add
-- Existing project: @fundamental-ngx/app-shell:generate schematics
Reorg- existing angular structure to match our ng-mf-submodules

closed as duplicate of #33


Python 2 compatibility tests?

Does anybody want to do some compatibility tests for Python 2? Presently only really checked on Python 3 (3.5). I've added from __future__ import division to most of the files but this may not be enough..

I'm currently working on this. (No guarantees!)
One question, there's a future package in setup.py's dependency list; Is it intentional? i.e. Shall I use this library for python2 support?

Well, I think I used python2 with conda last time, as I needed to tweak setup.py considerably to make it work on a fresh python2.7 virtualenv install.
Anyway, it works now, so if anyone needs python2 support real quick, one can probably use:
pip install git+https://github.com/Scorbie/StainTools.git#egg=staintools
I'm not issuing a pull request right now, as setup.py  became a little bloated to address all the version clashes for jupyter. This takes quite some time, and I'm not sure if I can maintain this when this breaks again. (Let's just hope it doesn't...)
Also, regarding #24, what do you think of

Removing unused dependencies?
:wastebasket: cython future (which is different from __future__ btw)
Moving jupyter to dev dependencies?
Just because they have to install with setup.py develop anyway.


Where can I download the OpenImage V2 dataset?

Dear all,
Thanks for your attention! I am very interested in OpenImage V2 dataset. But, it's difficult to download it one image by one image according the the image link.
Thus, I want to ask that "is there a simple way to download the OpenImage V2 dataset?"
Or, does the OpenImage V4/V5/V6 include the OpenImage V2 ?
Thanks a lot.
@jponttuset @gkrasin @andreasveit @vfgomes

Hi!
The set of images does not change from version to version, so you can download the images as explained in the Open Images V6 website:
https://g.co/dataset/open-images
Best,


Replaced react icons and changed website title

Description
Replaced react icons and changed website title
Type of change

[x] User Interface

Checklist:

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[x] I have npm run format before commiting
[x] I replicated the UI with the Design

Screenshots
1)

2)

I have changed the favicon but its thumbnail was not refreshed.

@harkiratsm Please have a look at this

LGTM 🚀


Fix android plugin functions are not called

some functions are not called properly in android plugin after #39855

@volzhs The methods you added should already be automatically invoked based on the Fragment lifecycle. Is there an actual bug that's caused by not having them?

@m4gr3d yes. I'm having an issue which is onMainActivityResult is not called in android custom plugin.
with this fix, it solved.

@m4gr3d yes. I'm having an issue which is onMainActivityResult is not called in android custom plugin.
with this fix, it solved.

@volzhs Did you call startActivityForResult instead of getActivity().startActivityForResult as mentioned in this SO answer?

@m4gr3d yes. I'm having an issue which is onMainActivityResult is not called in android custom plugin.
with this fix, it solved.

@volzhs Did you call startActivityForResult instead of getActivity().startActivityForResult as mentioned in this SO answer?

@volzhs From the plugin code, it would be getGodot().startActivityForResult instead of getActivity().startActivityForResult.

@m4gr3d  getGodot().startActivityForResult did fix my issue. :+1:
I searched everywhere before this PR but couldn't find about this.
need documentation for it.
closing as this PR does not need.

@volzhs Glad to hear! What were you using before?

@m4gr3d it was getActivity().startActivityForResult


Cache intermediate docker layers on travis

Some builds tests take a long time because they have to build our repo2docker "base" again and again.
Travis can cache files between builds however they explicitly mention docker images as something not to cache. I think the reasoning is that pulling them from docker hub is just as fast/slow. In our case we want to cache the intermediate build layers.
What do people think about adding that and how it relates to Travis' wishes about docker image caching?

Worth a shot

Is this still relevant ?

Would be interesting to experiment with to find out if this speeds up our CI runs.

I'll close this, we've got other issues related to caching and speeding up builds.


chore: make image urls in readme absolute

Currently, the images only render when the readme is displayed on github - they're broken on other sites such as crates.io:

Codecov Report

Merging #61 (770d35e) into main (08bcaa0) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main      #61   +/-   ##
=======================================
  Coverage   99.39%   99.39%
=======================================
  Files           2        2
  Lines         166      166
=======================================
  Hits          165      165
  Misses          1        1

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 08bcaa0...770d35e. Read the comment docs.


Cutflow tables

This has a few commits changing the background files slightly and making the postprocessor easier to read, then changes the cutflow.py functions efficiency(), cut_breakdown(), and print_multi_table(). All three of these were much larger and less efficient than they had to be, and are now much simpler and more straightforward. They all have the same functionality as before.

Just recording here that my fix for the print_multi_table() function involved importing numpy for the transpose() function. I realized it might be important to document that.

This looks very nice @yfv2ev ! Let's take a quick look together in BASH today, then we can probably merge into main


Do not use the lowermost bit when forcing hash to not match known values

We need to make sure that the real hashes do not match one of the special values. That requires wasting of a few bits of the hash, but we have a choice which bits we are sacrificing.
The lowermost bit might not be the best choice as it is plausible that some hashing schemes will produce consecutive/dense hashes and  | 1 will result in a collision in every adjacent pair.
Fixes: https://github.com/VSadov/NonBlocking/issues/21

Perhaps checking for the changed max size is in order too?

Perhaps checking for the changed max size is in order too?

Right, with this change there is no point to have tables larger than 1 << 29. While previously we could use the extra space for collision resolution, it would not work now.
It is probably ok as a tradeoff. 1/2 bln items is still a very large hashtable.

For very large tables idx could be shifted left, but I personally don't think it is worth the effort to implement.

I have played with this a bit to see what happens when the hashtable get close to the limits of its capacity. Turns out allowing table sizes > 1^29 can still useful.
When we need a table that is this large, things are obviously not fast. However, when we can allocate a 1 << 30 table, we can still utilize the extra elements. Reprobing may need quite a few extra hops, but because it is quadratic, it can still utilize the other half of the table.
On the other hand if we can only have 1 << 29 table and really need something bigger, we get into the churning mode when the hashtable is usable, but has more elements than a single table can fit. In such mode the dictionary continuously tries to rehash, hoping that some keys will be dropped and we will fit into the same table size.
This mode is a lot slower than when we might need to reprobe extra few times.
I think the aggressiveness of rehashing could be tuned down a bit when the table is large to better accommodate the "more items than can fit" scenario, but ultimately it will always perform suboptimally.
Allowing 1^30 elements table makes some edge case scenarios still usable, compared to 1^29 limit.
With smaller sizes, the limit does not have much effect either way, so I think there is no reason to make the limit smaller.

Thanks!!


⚠ Bofrost.at has degraded performance

In b27a635, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 2249 ms

Resolved: Bofrost.at performance has improved in 3a14f79 after 6 minutes.


optimize suggestion for resources repeat requests

May be maintain a static pictures list 。If resources has already been requested,do not repeat send a new request to get the image.i find the edge/chrome browser (chromium kernel) does not optimize for repeating the same resource and still sends a request even this resource was been cached. The will send many many request ,even close to ten thousand requests when i load schematics about 30 of them.And i use firefox and safari did not find this issue.they are not send repeated request of image ,that face to same requset
Sent from PPHub

Related issue: #18.


Path?? in Mount - provide instructions, maybe default

Newbie problem.
I installed DA 0.4.0.  Its running green and I see the Disks Window.  I plug in a USB stick.  I see its there but not mounted.  I press Mount, pick options and...... type what in the Path?
Can you provide instructions what should go here.... or even better, autofill a default entry for us non-professionals whom simply want a bit more control over what connects to their Mac.
Thanks,
Kevin

If you leave the path blank, the system will follow it's normal procedure: it will create a directory in /Volumes that matches the name of the volume, plus a numeric counter if needed, then mount the volume with that directory as the mount point. You only have to fill in the path if you would like the mount point to be somewhere else.
I like your suggestions, though, to document in the interface that the field is optional and also what the default action is.

If you leave the path blank, the system will follow it's normal procedure: it will create a directory in /Volumes that matches the name of the volume, plus a numeric counter if needed, then mount the volume with that directory as the mount point. You only have to fill in the path if you would like the mount point to be somewhere else.
I like your suggestions, though, to document in the interface that the field is optional and also what the default action is.

A tooltip was added in 9667487f99184511aa7cee08b8b7c92309e4e79b


[Menu] Cannot select menu item via enter key when rendered as link

Bug report
Current behavior
When selecting a menu item that contains a link, for example using a Link component from Inertia.js's Link component, or react-routers, it just closes the menu and doesn't actually go to it. This works fine if you select it with your mouse just not via the enter key.
Expected behavior
I'd expect it to go to the url selected upon pressing the enter key, as it works fine when using the enter key.
Reproducible example
Link to a StackBlitz (You'll need to open the full preview to see the url change).
Base UI version
v1.0.0-alpha.4
Which browser are you using?
Safari, but not relevant here.
Which OS are you using?
Mac OS, but not relevant here.
Which assistive tech are you using (if applicable)?
N/A
Additional context
The browser shows the url at the bottom but it just closes the menu instead of executing the link.

@mj12albert  This event.preventDefault() for useButton is the cause
https://github.com/mui/base-ui/blob/38f77b6db63bd5a33daac40bc722b9006849092a/packages/react/src/use-button/useButton.ts#L78

There weren't any tests for menu items or useButton rendering link components in the legacy package 🤔


Allow for deployment to local packages other than '$tmp'

Packages starting with '$' or 'T' can be local and not require a transport.
https://answers.sap.com/questions/9025981/a-question-about-creating-packages-as-local-object.html
I would like to change the validation to check for these conditions instead of if the package name is '$TMP'. I will submit a PR if thats alright.

Fixed in #16


Provide more guidance about data modeling prior distributed tables section

@umurc will write a "How to model your data" section. We should put it before the "Distributed Tables" section in the TOC.

This is done.


Require that from_v1 migrations are not from v2.

resolves #578

Codecov Report
Base: 95.08% // Head: 95.08% // Increases project coverage by +0.00% :tada:

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

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #598   +/-   ##
=======================================
  Coverage   95.08%   95.08%
=======================================
  Files          41       41
  Lines        3782     3788    +6
=======================================
+ Hits         3596     3602    +6
  Misses        186      186

Impacted Files
Coverage Δ

contracts/dao-core/src/contract.rs
97.61% <100.00%> (+0.01%)
:arrow_up:

...racts/proposal/dao-proposal-single/src/contract.rs
98.65% <100.00%> (+<0.01%)
:arrow_up:

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

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


allow for multiple names per kind

With this PR multiple kind names can be assigned to a kind implementation.
Kinds will register themselves with all their defined kindnames.
The default credentials map is now filled from the kind implementation via the nodes.SetDefaultCredentials(...) function.
Retrieving the default credentials works via nodes.GetDefaultCredentialsForKind(<AnyOfTheAssignedKindNames>)
I want to also get rid of the remaining:
const (
	NodeKindBridge          = "bridge"
	NodeKindHOST = "host"
	NodeKindOVS  = "ovs-bridge"
	NodeKindSRL  = "srl"
)

however, therefore we need to also work on the clab.config.go since it checks these Kinds. Therefore I had to also come up with a hack in the srl.init() where the kind is overwritten and forced to be nodes.NodeKindSRL.

issue for the kind consts created #845

Implementing #801

crazy week here. will try to review that by the mid next week


How to query across projects?

Hello, I see that API URL has the project name defined in the API call. Is there a way to query across projects for related work items?
I ask because via the browser when I log into ADO board when I go to a certain work item I can see all the links across projects in the browser. Is it possible to handle this use case via API call?

Document Details
⚠ Do not edit this section. It is required for learn.microsoft.com ➟ GitHub issue linking.

ID: e97a5d24-74dc-c495-5a48-0ccad5a61304
Version Independent ID: 6e6e297e-8491-243e-6744-f5e96f5b2f20
Content: Query for linked work items - Azure DevOps
Content Source: docs/report/extend-analytics/work-item-links.md
Service: azure-devops
Sub-service: azure-devops-analytics
GitHub Login: @chcomley
Microsoft Alias: chcomley

@adopowerbi -- thank you for your questions.  Please consider these resources:

Azure DevOps Services
Azure DevOps on Stack Overflow

@chcomley -- Chrystal, please look into this potential documentation issue.

https://learn.microsoft.com/en-us/azure/devops/boards/queries/using-queries?view=azure-devops&tabs=browser#across-projects


Fix rocket names

No more entity.rocket.name showing in OneProbe, /say commands and more. I left the entity.advancedRocketry.rocket.name in, just to be save.

Thanks!


TypeLoadException when storing struct with ArraySegment of own type

Description
Not necessarily instantiating, but storing a struct A with an ArraySegment<A> inside causes a TypeLoadException.
This seems to happen when placing a struct with a generic array of type A inside a struct of type A. This shouldn't be a problem tho, since the array is a 'pointer' and therefore takes up constant space inside the struct regardless of type.
Here is a mostly minimal Sample:
using System;

namespace CompilerBug2Tests {
  class Program {
    struct A {
      public ArraySegment<A> Array;
    }

    struct B {
      public B[] Array;
    }

    struct Segment<T> {
      public T[] Arr;
    }
    struct C {
      public Segment<A> Array;
    }

    struct DSegment {
      public D[] Arr;
    }
    struct D {
      public DSegment Array;
    }

    static void Main(string[] args) {
      // this does not crash
      var b = new B();

      // neither does this
      new A();

      // this however does
      //var a = new A();

      // as well as this
      //var c = new C();

      // this works again
      var d = new D();
    }
  }
}

Configuration

the project targets core 3.1
dotnet --version : 5.0.100
Win10 x64
I Don't know if the bug also occurs on different platforms, I don't have the time to test atm.

@SaculRennorb the simplest repro of your issue looks like as following:
struct P<T> {
}

struct Q {
    public P<Q> Value;
}

class Program
{
    static void Main(string[] args)
    {
        var q = new Q();
    }
}

This is basically a duplicate of dotnet/runtime#6924, and unlikely to be fixed according to https://github.com/dotnet/runtime/issues/6924#issuecomment-547078135. If you think you have a good justification (real life use case) feel free to comment under that discussion.


[SMALLFIX] Trim whitespace for keys values set in site-properties

@riversand9

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/17493/
Test FAILed.

Jenkins, test this please

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/17544/
Test PASSed.

Merged build finished. Test PASSed.


Automatic Landing when Program Ends

I am using arm_and_takeoff(5) function. After that the program ends.
The vehicle mode gets automatically changed to LAND mode.
Is this a expected behavior?
I am using Gazebo for simulation.

Probably not, but you'd have to do a whole lot more information for anyone to help you. Ie how did you install dronekit, what flight stack and flight stack version you are using. What exactly you saw - ie did it take off? What code were you running?
Depending on what you want to do you might add some of the additional telemetry in https://dronekit-python.readthedocs.io/en/latest/guide/taking_off.html so you can see how far the vehicle got and what its state was before it decided to land.


Finish LIMIT for postgreSQL for visitor

For #4271.

Codecov Report

Merging #4283 into master will decrease coverage by 0.07%.
The diff coverage is 0%.

@@             Coverage Diff              @@
##             master    #4283      +/-   ##
============================================
- Coverage     59.27%   59.19%   -0.08%
  Complexity      352      352
============================================
  Files          1018     1018
  Lines         17446    17468      +22
  Branches       3111     3118       +7
============================================
  Hits          10341    10341
- Misses         6450     6472      +22
  Partials        655      655

Impacted Files
Coverage Δ
Complexity Δ

...phere/sql/parser/visitor/impl/MySQLDMLVisitor.java
0% <0%> (ø)
0 <0> (ø)
:arrow_down:

.../sql/parser/visitor/impl/PostgreSQLDMLVisitor.java
0% <0%> (ø)
0 <0> (ø)
:arrow_down:

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 87d6bd6...af4c50a. Read the comment docs.

Pull Request Test Coverage Report for Build 9498

0 of 16 (0.0%)  changed or added relevant lines in 2 files are covered.
3 unchanged lines in 1 file lost coverage.
Overall coverage decreased (-0.05%) to 63.007%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

shardingsphere-sql-parser/shardingsphere-sql-parser-mysql/src/main/java/org/apache/shardingsphere/sql/parser/visitor/impl/MySQLDMLVisitor.java
0
3
0.0%

shardingsphere-sql-parser/shardingsphere-sql-parser-postgresql/src/main/java/org/apache/shardingsphere/sql/parser/visitor/impl/PostgreSQLDMLVisitor.java
0
13
0.0%

Files with Coverage Reduction
New Missed Lines
%

sharding-orchestration/sharding-orchestration-core/src/main/java/org/apache/shardingsphere/orchestration/internal/util/IpUtils.java
3
76.0%

Totals

Change from base Build 839:
-0.05%

Covered Lines:
11006

Relevant Lines:
17468

💛  - Coveralls


ConcurrentModificationException inside getSecondaryResources

Bug Report

What did you do?
Called context.getSecondaryResources(resource) inside ResourceDiscriminator.

What did you expect to see?
Secondary resources.

What did you see instead? Under which circumstances?
Not sure the exact circumstances, but saw:
java.util.ConcurrentModificationException
	at java.base/java.util.HashMap$ValueSpliterator.forEachRemaining(HashMap.java:1792)
	at java.base/java.util.stream.AbstractPipeline.copyInto(AbstractPipeline.java:509)
	at java.base/java.util.stream.AbstractPipeline.wrapAndCopyInto(AbstractPipeline.java:499)
	at java.base/java.util.stream.ReduceOps$ReduceOp.evaluateSequential(ReduceOps.java:921)
	at java.base/java.util.stream.AbstractPipeline.evaluate(AbstractPipeline.java:234)
	at java.base/java.util.stream.ReferencePipeline.collect(ReferencePipeline.java:682)
	at io.javaoperatorsdk.operator.processing.event.EventSources.getEventSources(EventSources.java:184)
	at io.javaoperatorsdk.operator.processing.event.EventSourceManager.getResourceEventSourcesFor(EventSourceManager.java:228)
	at io.javaoperatorsdk.operator.api.reconciler.DefaultContext.getSecondaryResourcesAsStream(DefaultContext.java:50)
	at io.javaoperatorsdk.operator.api.reconciler.DefaultContext.getSecondaryResources(DefaultContext.java:40)

Environment
Kubernetes cluster type:
testing with kind via quarkus dev services.

io.quarkiverse.operatorsdk:quarkus-operator-sdk:6.7.2
io.javaoperatorsdk:operator-framework-core:4.9.2

Eclipse Temurin OpenJDK Runtime Environment 21.0.4+7-LTS

Possible Solution

Additional context

Unfortunately, baring a JDK bug, I don't see anything in the code that would cause this problem since these operations are read-only.

I'm not sure I can replicate it easily. But I think I can easily demonstrate the underlying issue.
    @Test
    void testConcurrency() throws InterruptedException {
        AtomicBoolean done = new AtomicBoolean();
        while (!done.get()) {
            Phaser phaser = new Phaser(2);
            Map<Object, Object> map = new HashMap<>();
            for (int i = 0; i < 10; i++) {
                map.put(i, i);
            }
            Stream<Object> stream = map.values().stream();
            Thread thread1 = new Thread(() -> {
                phaser.arriveAndAwaitAdvance();
                for (int i = 10; i < 20; i++) {
                    map.put(i, i);
                }
            });
            Thread thread2 = new Thread(() -> {
                phaser.arriveAndAwaitAdvance();
                stream.collect(Collectors.toList());
            });
            thread2.setUncaughtExceptionHandler((t, e) -> {
                if (e instanceof ConcurrentModificationException) {
                    e.printStackTrace(System.out);
                    done.set(true);
                }
            });
            thread1.start();
            thread2.start();
            thread1.join();
            thread2.join();
        }
    }

java.util.ConcurrentModificationException
	at java.base/java.util.HashMap$ValueSpliterator.forEachRemaining(HashMap.java:1792)
	at java.base/java.util.stream.AbstractPipeline.copyInto(AbstractPipeline.java:509)
	at java.base/java.util.stream.AbstractPipeline.wrapAndCopyInto(AbstractPipeline.java:499)
	at java.base/java.util.stream.ReduceOps$ReduceOp.evaluateSequential(ReduceOps.java:921)
	at java.base/java.util.stream.AbstractPipeline.evaluate(AbstractPipeline.java:234)
	at java.base/java.util.stream.ReferencePipeline.collect(ReferencePipeline.java:682)
	at com.hedera.solo.operator.test.unit.ConsensusPodPropertiesTest.lambda$testConcurrency$39(ConsensusPodPropertiesTest.java:423)
	at java.base/java.lang.Thread.run(Thread.java:1583)

OK so the issue would happen when your ResourceDiscriminator would be called while other event sources are still being initialized? If you are initializing your event sources manually, you could try to set the event source's priority to EventSourceStartPriority.RESOURCE_STATE_LOADER to see if this addresses the issue (you can see an example of this being used at https://github.com/operator-framework/java-operator-sdk/blob/main/operator-framework/src/test/java/io/javaoperatorsdk/operator/sample/externalstate/ExternalStateReconciler.java#L107).
If you're not setting up sources manually, could you detail how they are set up, please? Maybe we could (should?) surface that priority setting?

This reconciler is "managed", in the sense that dependent resources are added through annotations. No event sources for this reconciler are initialized manually.
This reconciler does however have six dependent resources with activation conditions, including the dependent resource whose ResourceDiscriminator was called when this exception was thrown. I could be wrong, but I suspect this may be related to the activation conditions.
AbstractWorkflowExecutor::registerOrDeregisterEventSourceBasedOnActivation
EventSourceManager::dynamicallyRegisterEventSource
EventSourceManager::registerEventSource
EventSources::add

Yes, event sources are added dynamically with activation condition, this will be the issue

Added fix for both main and v5


[Feature Request] Disable Module in Settings (not planned; macros solve this)

A quick "disable the modules functionality" setting would be nice to have for other modules to temporarily turn on and off when say, summoning tokens and in turn, "hurting" them by changing their HP, causing them to get a blood splatter when they honestly shouldn't.

there's a built in macro for that!  see https://github.com/shemetz/tokenmagic-automatic-wounds#macros-in-tokenmagic-automatic-wounds-compendium-pack
you want "Toggle Auto-Wounds For Token" or "Toggle Auto-Wounds For Actor", to disable it for certain bloodless enemies.

FYI, if you're developing a module that wants to interact like this, you can use:
const MODULE_ID = 'tokenmagic-automatic-wounds'
const FLAG_DISABLE_WOUNDS = 'enabled-for-token'
actor.setFlag(MODULE_ID, FLAG_DISABLE_WOUNDS, false)

Is there a flag stored when this is done so I can re-enable the feature?
wt., 20 gru 2022, 09:45 użytkownik shemetz @.***>
napisał:

Closed #12
https://github.com/shemetz/tokenmagic-automatic-wounds/issues/12 as not
planned.
—
Reply to this email directly, view it on GitHub
https://github.com/shemetz/tokenmagic-automatic-wounds/issues/12#event-8076242288,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AHUOGHELBFITTR3POSWO3O3WOFW3VANCNFSM6AAAAAATAHOUSQ
.
You are receiving this because you authored the thread.Message ID:
<shemetz/tokenmagic-automatic-wounds/issue/12/issue_event/8076242288@
github.com>

That doesn't help with anything because the wounds don't disappear on setting the flag.

In the end I found I can just use TokenMagicAutomaticWounds.toggleDisableWounds, that will suffice.


Handle rotation with in game API

This change will drop old rotation calc based on current position + last position for the in-game rotation provided by Overwolf API.

Is keeping track of lastPosition still useful if we are no longer using it?


Possible inconsistency in FlippedSignature

I may have found an inconsistency in lib.wiring by creating a component with an incoming wishbone bus. It appears to be an issue with how Amaranth creates interfaces for flipped signatures (and not with amaranth-soc's wishbone implementation). Although the result was unexpected, I'm unsure of whether lib.wiring's design actually supports this kind of access. Please let me know if I'm misusing something here.
The original issue can be reproduced like so:
from amaranth import *
from amaranth.lib.wiring import In, Out, Component
from amaranth_soc import wishbone

class BusComponent(Component):
    output: Out(wishbone.bus.Signature(addr_width=32, data_width=32))
    input:  In(wishbone.bus.Signature(addr_width=32, data_width=32))

c = BusComponent()

print(type(c.output))
print(type(c.input))

c.output.addr_width
c.input.addr_width

Result:
<class 'amaranth_soc.wishbone.bus.Interface'>
<class 'amaranth.lib.wiring.Interface'>
Traceback (most recent call last):
[...]
    c.input.addr_width
AttributeError: 'Interface' object has no attribute 'addr_width'

As expected, the c.output attribute is a wishbone.bus.Interface. This is because wishbone.bus.Signature overrides the create method here to return a wishbone-specific interface subclass. On the other hand, the c.input attribute is a simple wiring.Interface, which doesn't forward attribute access to the wishbone implementation. As a result, it isn't possible to access the addr_width attribute on the incoming bus. I found that surprising and unexpected.
Flipped interfaces don't seem to be affected, only interfaces created from flipped signatures:
from amaranth import *
from amaranth.lib import wiring
from amaranth_soc import wishbone

wishbone.bus.Interface(addr_width=32, data_width=32).addr_width
wiring.flipped(wishbone.bus.Interface(addr_width=32, data_width=32)).addr_width

wishbone.bus.Signature(addr_width=32, data_width=32).create().addr_width
wishbone.bus.Signature(addr_width=32, data_width=32).flip().create().addr_width

Result:
Traceback (most recent call last):
    wishbone.bus.Signature(addr_width=32, data_width=32).flip().create().addr_width
AttributeError: 'Interface' object has no attribute 'addr_width'

One possible solution would be to implement FlippedSignature.create to return the flipped interface constructed from the unflipped signature. After implementing it as below, the above reproductions appear to work correctly.
class FlippedSignature:
    # ...
    def create(self, *, path=()):
        return flipped(self.__unflipped.create(path=path))

Is there a better way to achieve the same effect?

I think this is just a bug! We will look into your proposed solution. Thank you for filing it ^^


Contrib shared library not found with TensorFlow

Issue description
Trying to use TensorFlow contribs from Nix ends up looking for libforestproto.so and can't find it.
Steps to reproduce
In a TensorFlow program, do something like
from tensorflow.contrib.framework.python.ops import add_arg_scope
This gives:
from tcn import TemporalConvNet
  File "/anomaly/TCN/tcn.py", line 4, in <module>
from tensorflow.contrib.framework.python.ops import add_arg_scope
  File "/nix/store/6qwz85c73as9qdyr5llzcpj9aja8rrf2-python3-3.6.6-env/lib/python3.6/site-packages/tensorflow/contrib/__init__.py", line 86, in <module>
from tensorflow.contrib import tensor_forest
  File "/nix/store/6qwz85c73as9qdyr5llzcpj9aja8rrf2-python3-3.6.6-env/lib/python3.6/site-packages/tensorflow/contrib/tensor_forest/__init__.py", line 21, in <module>
from tensorflow.contrib.tensor_forest.client import *
  File "/nix/store/6qwz85c73as9qdyr5llzcpj9aja8rrf2-python3-3.6.6-env/lib/python3.6/site-packages/tensorflow/contrib/tensor_forest/client/__init__.py", line 22, in <module>
from tensorflow.contrib.tensor_forest.client import random_forest
  File "/nix/store/6qwz85c73as9qdyr5llzcpj9aja8rrf2-python3-3.6.6-env/lib/python3.6/site-packages/tensorflow/contrib/tensor_forest/client/random_forest.py", line 27, in <module>
from tensorflow.contrib.tensor_forest.python import tensor_forest
  File "/nix/store/6qwz85c73as9qdyr5llzcpj9aja8rrf2-python3-3.6.6-env/lib/python3.6/site-packages/tensorflow/contrib/tensor_forest/python/__init__.py", line 21, in <module>
from tensorflow.contrib.tensor_forest.python import tensor_forest
  File "/nix/store/6qwz85c73as9qdyr5llzcpj9aja8rrf2-python3-3.6.6-env/lib/python3.6/site-packages/tensorflow/contrib/tensor_forest/python/tensor_forest.py", line 29, in <module>
from tensorflow.contrib.tensor_forest.python.ops import data_ops
  File "/nix/store/6qwz85c73as9qdyr5llzcpj9aja8rrf2-python3-3.6.6-env/lib/python3.6/site-packages/tensorflow/contrib/tensor_forest/python/ops/data_ops.py", line 20, in <module>
from tensorflow.contrib.tensor_forest.python.ops import tensor_forest_ops
  File "/nix/store/6qwz85c73as9qdyr5llzcpj9aja8rrf2-python3-3.6.6-env/lib/python3.6/site-packages/tensorflow/contrib/tensor_forest/python/ops/tensor_forest_ops.py", line 28, in <module>
resource_loader.get_path_to_datafile('_tensor_forest_ops.so'))
  File "/nix/store/6qwz85c73as9qdyr5llzcpj9aja8rrf2-python3-3.6.6-env/lib/python3.6/site-packages/tensorflow/contrib/util/loader.py", line 56, in load_op_library
ret = load_library.load_op_library(path)
  File "/nix/store/6qwz85c73as9qdyr5llzcpj9aja8rrf2-python3-3.6.6-env/lib/python3.6/site-packages/tensorflow/python/framework/load_library.py", line 56, in load_op_library
lib_handle = py_tf.TF_LoadLibrary(library_filename)
tensorflow.python.framework.errors_impl.NotFoundError: libforestprotos.so: cannot open shared object file: No such file or directory

What happens is that it tries do dlopen _tensor_forest_ops.so, which has a depenency on libforestprotos.so:
# ldd /nix/store/6qwz85c73as9qdyr5llzcpj9aja8rrf2-python3-3.6.6-env/lib/python3.6/site-packages/tensorflow/contrib/tensor_forest/python/ops/_tensor_forest_ops.so
linux-vdso.so.1 (0x00007ffde179e000)
libforestprotos.so => not found
libtensorflow_framework.so => /nix/store/5bkq6sd9h057dgy0i0zqdv7hw2lq8wvv-python3.6-tensorflow-1.9.0/lib/python3.6/site-packages/tensorflow/libtensorflow_framework.so (0x00007ff64d0ff000)
libm.so.6 => /nix/store/fg4yq8i8wd08xg3fy58l6q73cjy8hjr2-glibc-2.27/lib/libm.so.6 (0x00007ff64cd6a000)
libstdc++.so.6 => /nix/store/zk5zj2307zxaq7dx585yia3dn5k4qlsl-gcc-7.3.0-lib/lib/libstdc++.so.6 (0x00007ff64c9e3000)
libgcc_s.so.1 => /nix/store/zk5zj2307zxaq7dx585yia3dn5k4qlsl-gcc-7.3.0-lib/lib/libgcc_s.so.1 (0x00007ff64c7cb000)
libc.so.6 => /nix/store/fg4yq8i8wd08xg3fy58l6q73cjy8hjr2-glibc-2.27/lib/libc.so.6 (0x00007ff64c417000)
libdl.so.2 => /nix/store/fg4yq8i8wd08xg3fy58l6q73cjy8hjr2-glibc-2.27/lib/libdl.so.2 (0x00007ff64c213000)
libpthread.so.0 => /nix/store/fg4yq8i8wd08xg3fy58l6q73cjy8hjr2-glibc-2.27/lib/libpthread.so.0 (0x00007ff64bff4000)
/nix/store/fg4yq8i8wd08xg3fy58l6q73cjy8hjr2-glibc-2.27/lib64/ld-linux-x86-64.so.2 (0x00007ff64e050000)

Technical details
Please run nix-shell -p nix-info --run "nix-info -m" and paste the
results.

system: "x86_64-linux"
host os: Linux 4.9.93-linuxkit-aufs, Alpine Linux, noversion
multi-user?: yes
sandbox: no
version: nix-env (Nix) 2.1.1
channels(root): "nixpkgs-19.03pre152822.f1737fa6466"
nixpkgs: /nix/var/nix/profiles/per-user/root/channels/nixpkgs

See #46841 and a fix in  #47114.

@mikesperber  #47114 is merged

Many thanks!


Enhancements to variables related to SDC files

~ Rename RCX_SDC_FILE to SIGNOFF_SDC_FILE.
~ Rename BASE_SDC_FILE to IMPLEMENTATION_SDC_FILE.
~ Always use IMPLMEMENTATION_SDC_FILE instead of CURRENT_SDC except during signoff stage.
+ Warn when IMPLEMENTATION_SDC_FILE is not overwritten by the user.
+ Add SDC_IN and IMPLEMENTATION_SDC_FILE_DEFAULT to ignore list in ci Check Variables flow

@donn APU had an increase in the number of antenna violations. I don't see something that this PR does that would affect antenna violations. Perhaps subtle changes in input can affect the output. One solution would be enabling RUN_HEURISITC_DIODE_INSERTION and DIODE_ON_PORTS. What do you think?

+ Warn when IMPLEMENTATION_SDC_FILE is not overwritten by the user.
Why?

+ Warn when IMPLEMENTATION_SDC_FILE is not overwritten by the user.
Why?

It was a request made by @shalan


TF Rule: S3 bucket policy should not have a wildcard principal

Add a new Terraform rule:
An S3 bucket policy should not use a wildcard principal.

This rule is covered in cli/assets/terraform.yml and cli/assets/terraform12.yml with rule S3_BUCKET_POLICY_WILDCARD_PRINCIPAL


Add docker collectd plugin.

Install latest pip and setuptools on every node
Add docker collectd plugin

merged in https://github.com/CiscoCloud/microservices-infrastructure/pull/417


Update command to install chocolatey packages

Changes choco install command to ahdere to chocolatey scripting best practices.
In particular:

use choco upgrade rather choco install
move the package name to be the first parameter
explicitly set choco source

This isn't actually going to do anything. Builds are working again today. Chocolatey must've just been down yesterday.

I tried to do that but it doesn't work in main:

That's an intentional check Ed added, since pushing would simply fail to build (due to the same thing).  They're put in place to mark temporary workarounds that need to be reviewed at a future date.  Ed's really been doing the best job keeping track of the "why" for each of these timebomb blocks, so I normally would consult with him. But...

This isn't actually going to do anything.

...maybe this PR should be closed?  (though we appreciate the time/effort you put into it).

this PR introduces an improvement in the code base

Thanks for pushing back.  I would defer to your expertise on the subject.
@edsantiago since you're already building images, and chocolaty seems to be behaving, would you consider rebasing these changes into your PR?  An alternative is, eat the risk and add the magic [skip-ci] onto this PR and just ram it in - I see no compelling reason to subject @l0rd to the complexities of image-building based on the diff here.

I can't fold this into any of my PRs, and I can't condone force-merging it either. Rearranging arg order is fine, but:

explicit URL is evil. It's like dnf install --from=today's-fedora-repo.org/today's-api-version. The URL and/or API version will change in the future, it is not OK to force that onto future maintainers. Install tools should know where to get their packages from
"upgrade" is not an acceptable term. A normal human being reading that line would see upgrade and waste time investigating which version might already be installed, which one we're asking for, and get confused. (It boils down to whether this script is being written for experts-in-podman or experts-in-chocolatey. I believe we are writing for the former).
finally, yes, any change that's as confusing and hard-to-maintain as this requires a large comment (as you requested).

Since I'm doomed to spend my days rebuilding VMs, I will incorporate two changes into one of my future PRs: rearrange args, and add a link to chocolatey best practices. But I humbly ask you not to incorporate this PR as it stands.

I can't fold this into any of my PRs, and I can't condone force-merging it either. Rearranging arg order is fine, but:
* explicit URL is evil. It's like `dnf install --from=today's-fedora-repo.org/today's-api-version`. The URL and/or API version will change in the future, it is not OK to force that onto future maintainers. Install tools should know where to get their packages from

* "upgrade" is not an acceptable term. A normal human being reading that line would see `upgrade` and waste time investigating which version might already be installed, which one we're asking for, and get confused. (It boils down to whether this script is being written for experts-in-podman or experts-in-chocolatey. I believe we are writing for the former).

* finally, yes, any change that's as confusing and hard-to-maintain as this requires a large comment (as you requested).

I agree that the chocolatey scripting "best practices" are counterintuitive and can be hard to maintain. Ignoring them though, can hurt us even more in the future. As per your request I am closing this PR and will let you decide what to include or not in yours.

Thanks @l0rd for working with us and sharing your expertise.  I'm confident meaningful improvements will come from this.


SCons: Prefer Exit() method over sys.exit()

Sconscript provides it's own Exit() method which is currently an alias for sys.exit() internally, with the only difference that if no exit code is specified, it defaults to 0.
This encourages the usage of SCons-implemented methods like Glob() over glob.glob(), which may overcome limitations of the built-in Python features in the future, or provide additional features not typically seen in plain Python.

Thanks!


Default demo of syncthing doesnot work.

Default demo of syncthing doesnot work. Seems packages in mod dir cannot be searched.
go-callvis -focus upgrade -group pkg,type -limit github.com/syncthing/syncthing -ignore github.com/syncthing/syncthing/lib/logger github.com/syncthing/syncthing/cmd/syncthing
src/github.com/syncthing/syncthing/cmd/syncthing/main.go:45:2:  cannot find package "github.com/pkg/errors" in any of: 	/usr/local/go/src/github.com/pkg/errors (from $GOROOT) 	/Users/dove/viper/src/github.com/pkg/errors (from $GOPATH)`
Locates  errors package
find . -name errors ./pkg/mod/cache/download/github.com/go-errors/errors ./pkg/mod/cache/download/github.com/pkg/errors<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>

Hey, try moving into syncthing repo and run in there: go-callvis [OPTIONS] ./cmd/syncthing


Add linux support

Here's a first stab at adding Linux support into boxen. There's more details in README_LINUX.md, but basically I modded a bunch of the puppet-* modules, as well as our-boxen, so that boxen uses gnome-keyring and linuxbrew instead of osx-keychain and homebrew when running on Linux. Let me know if you want any more details.

Why don't fork this repo and produce a specific linux-based boxen?

Thanks for your work on this, @telamonian. Closing this due to inactivity—please feel free to open a new PR if you decide to rebase and give it another go.


Added support to view connected clients client-side, feature requested by @triwine on discord.

First PR ever, sorry in advance for crippled code :)

I think the best way would be to create a new payload type.

Yeah that can be done too.


fix: make sure ptotoc prebuild doesn't fetch source code with system …

…defined github token

This change is

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

Project coverage is 62.73%. Comparing base (1e291f7) to head (3a56f4a).
Report is 2 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #207      +/-   ##
==========================================
+ Coverage   62.69%   62.73%   +0.03%
==========================================
  Files          26       26
  Lines         965      966       +1
  Branches      965      966       +1
==========================================
+ Hits          605      606       +1
  Misses        334      334
  Partials       26       26

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

crates/mempool_infra/build.rs line 8 at r1 (raw file):

fn main() {
    set_var("PROTOC_PREBUILT_NOT_ADD_GITHUB_TOKEN", "true");
    let (protoc_bin, _) = init("27.0").unwrap();

Do we want to unset it post command?
Suggestion:
    set_var("PROTOC_PREBUILT_NOT_ADD_GITHUB_TOKEN", "true");
    let (protoc_bin, _) = init("27.0").unwrap();
    unset_var("PROTOC_PREBUILT_NOT_ADD_GITHUB_TOKEN);


Add System Python UDF documentation

Added Python UDF documentation under Query Tracker section. Checked locally

LGTM

Why are we putting this in query_tracker chapter and not in the yql one?

Why are we putting this in query_tracker chapter and not in the yql one?

We agreed that I will put it here as the best place available for external people
YQL documentation is special. It's source of truth is combined from ydb repository and internal Yandex repository. YT specific things are not included in ydb repository so the only thing that left for yt-specific YQL things in internal Yandex repository.
Later this part can be moved to a better place. cc @ogorbacheva @blinkov


Add staticcheck static analyzer to Travis

staticcheck is a popular static analyzer. It's a free bug finder that catches some things that pass go vet.

Coverage remained the same at 94.762% when pulling 0f49b43254f710742d2c45a4d3bb93db20bbbec8 on y0ssar1an:staticcheck into aab6caaf3a868d3f1a8cd87b7c34b364306df278 on nats-io:master.

Can't make staticcheck work with Travis :(

Are you sure you need the go_import_path: honnef.co/go/staticcheck/cmd/staticcheck line? Is go get not enough?

Experimented with removing the go_import_path statement and it seems to be working now. Check the branch: https://github.com/nats-io/nats/tree/test_staticcheck

I initially tried without go_import_path and it didn't work. I now have an identical .travis.yml file in #233 to the one in test_staticcheck and it's still not working.


populate an OpenJS grip based on the value

Hi,
I want to populate an OpenJS grip based on the value selected by the used on a drop down list. The calling script is time_data.php which extracts and populate the data. How can I achieve this ?
html PART

            Id
            Name
            Time In
            Time Out
            Status
            Approval
        <?php }?>

time_data.php
true,
    "delete"=>true,
    "editing"=>true,
    "where"=>"time_grpc = 'CARP'",
    "joins"=> array(
               "LEFT JOIN tbl_tmstat ON (tmstat_code = time_stats)"
         ),

    "select" => 'selectFunction'
    ));

    // 'selectFunction'

    function selectFunction($grid) {
        $selects = array();
    //Session_start;
       // category select
        $grid->table = "tbl_tmstat";
        $selects["time_stats"] = $grid->makeSelect("tmstat_code","tmstat_name");

        // active select
        $selects["active"] = array("1"=>"true","0"=>"false");

        // render data
        $grid->render($selects);
    }`enter code here`

See this video for master-detail
https://www.youtube.com/watch?v=U5MGopAoQrM
You just need to add an action to your drop down to do what we're doing in this video for the child table.


webpki-roots/webpki-root-certs: 0.26.3 -> 0.26.4

Since the 0.26.3 release has already been tagged before the webpki-roots-crate existed I think it makes sense to bump the version for the first webpki-roots-crate release and describe it as a no-op update for webpki-roots.
Proposed release notes

New webpki-root-certs crate. This crate is similar to webpki-roots, but for use with other projects that require the full self-signed X.509 certificate for each trusted root. This is unnecessary overhead for webpki and rustls and you should prefer using webpki-roots for these projects.
There are no changes to webpki-roots in this release.

I think the promise to release in lockstep is a little tricky, and I'm not sure it's all that helpful in the end? It's fine as an aspirational thing but I'm not sure it'll always be practical.

we could release webpki-root-certs at 0.26.3 instead?

What I don't love about that is if you clone the repo and check out the v/0.26.3 tag there's no webpki-root-certs. I don't want to delete/recreate the tag so it seemed like a no-op point release for the one crate was the least bad option.

I don't think we need to do it 100% all of the time, but more like the "normal" release flow for this repo should be to release both, with the same version. That means we don't need to bifurcate the release notes or tags.
For this initial release, I would suggest not publishing webpki-roots 0.26.3.
Then the next release would be 0.26.5 for both crates.

For this initial release, I would suggest not publishing webpki-roots 0.26.4.

That sounds OK to me. Should the version in webpki-roots/Cargo.toml remain at 0.26.3 in that case?

Maybe move it forward to version = "0.26.4-not-released"?

Reworked the version change/commit message/release notes :+1:

Created v/0.26.4 tag.
Published webpki-root-certs v0.26.4 at registry crates-io
Created 0.26.4 GitHub release


'busy' should be a property or function

Some clients will implement 'busy' as an explicitly set state; others prefer to evaluate busy state as a property


Without this fix the container restarts after every login over ssh

Yes, but I don't know why?

Not sure couldn't get useful logs out of docker. I think priv sep must trigger a bug in it
Don't think it's a big deal to turn this off.

any chance we can get this one merged and a rebuild done?

sorry, See it laterly.


feat: CLI to update registry credentials

New simple CLI that can update registry credentials
➜  credential-manager update registry-credentials --address=docker.io --username=user --password=pass
Updated credentials
➜ kubectl view-secret -n kube-system   staticcredentialproviderauth
Choosing key: static-image-credentials.json
{"kind":"CredentialProviderResponse","apiVersion":"credentialprovider.kubelet.k8s.io/v1beta1","cacheKeyType":"Image","cacheDuration":"0s","auth":{"docker.io":{"username":"user","password":"pass"}}}

Love the idea! What do you think about set-credentials, get-credentials, delete-credentials instead of update?


The method 'FormBuilderCustomField' isn't defined for the type'

flutter_form_builder does not recognize FormBuilderCustomField after upgrading to ^6.0.0-nullsafety.1
any work around??

FormBuilderCustomField was deprecated in v4 onwards and replaced by FormBuilderField kindly look at the Migration guide for v3 to v4


Refresh SBAL memory before destructing log

Adds free methode to remove log message from SBAL memory. It's recommended to call this method before destructing any instance of zcl_logger.
Closes #130

Thanks


Fix test expectation

Instead of comparing output that contains irrelevant dots, we strip them
before asserting that we have the expected percentage values
Added a method that allows to query the output for a line containing a
given String

🤦♂ Managed to push the version with the intentional mistake to validate the assertion ...


Add support for consuming events from Kafka

Pull Request type

[ ] Bugfix
[x] Feature
[ ] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[ ] WHOSUSING.md
[ ] Other (please describe):

NOTE: Please remember to run ./gradlew spotlessApply to fix any format violations.
Changes in this PR
I have implemented native support for consuming events from Kafka Topics using Conductor events handlers. This works the same as for example the SQS queue implementation but with Kafka.
Issue #

Fixing a handling issue with size retrieval when an EventHandler with a new Topic name is created by this queue and the AdminClient's metadata has not been updated


Update workflows to the ballerina-library

Purpose
Examples
Checklist

[ ] Linked to an issue
[ ] Updated the changelog
[ ] Added tests
[ ] Updated the spec
[ ] Checked native-image compatibility

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


Why always icon "f(x)"?

Whatever file I open, all the icons before the symbols are the same "f(x)".
If they are always the same, I perfer to remove them. Or at least, allow me to hide them.
Even if someday there come more icons, I still don't like the font icon "f(x)". It looks like characters (I can even type them here) instead of graphic, and thus always distracts me when I read the symbol right after it.

@grandsong Sorry for that... It's maybe because your source file can't be supported well by ctags... You can try C or Java source files, there will be some other type symbols
I'm not good at designing, so I just create this f(x) icon by some simple tools... I'm also looking forward some better icons. That would be cool if you could help, thank you!

I gues there can hardly be any graphic icon good enough for abstract concepts like function, variable, etc. So, I suggest you simply remove all icons before symbols.
And as some, including me, request in other issues, it will be great if different types of symbols are shown in different colors (or even more, in font style bold, italic, etc; just let CSS do the job). I believe that  will be enough and thus no icons will be needed anymore.
To play safe, you can keep icons shown by default and allow users to hide them by an option in settings.

My two cents: What about grouping similar symbols (like functions, variables, classes, ...) together and make it a collapsable tree view? Similar to the way Geany does it? I've looked into a lot of editors and IDEs, but I haven't (yet) found one that is as fast and powerful as Geany (yes, this includes Atom).

Also, I personally think that the icons seen in the screenshot are a very nice solution to the above discussed problem: How to represent something as abstract as a method, class or member with a symbol?

+1 for removing or allowing hiding. I think this issue can be labeled as feature request though.
@riker09 : This seems it may be being tracked in issue #68 (sort by types) and kind of in #17 (hierarchy in panel)

@brettz9 Yes, I agree. The issues mentioned cover what I proposed.
This original purpose of this issue was to find some sort of alternative for the "f(x)" icons. My proposal is as following: Use three boxes for classes, use two boxes for functions/methods and one box for variables. This should cover a lot of use cases already and should work rather well for PHP class files.

The current state is definitely not ergonomic. It would be better if the symbols had another color (e.g. darker), but now they are distracting the eye from the actual symbols.

@derwaldgeist Sorry for that... I will try to add config to use different colors instead of icons later, thank you very much!

No problem, no need to apologize. I still appreciate that somebody builds this stuff!
Actually I did not mean to have different colors instead of icons (I meant to have the icons in a darker color than the text), but having different colors for the types would also be awesome. Plus an option to sort the symbols by name or type (and fix this sorting order across files and sessions).


Get-ADUser is not recognized

When I tried using this command, I get:
Get-ADUser : The term 'Get-ADUser' is not recognized as the name of a cmdlet, function, script file, or operable program. Check the spelling of the name, or if a path
was included, verify that the path is correct and try again.
Also, your SearchBase string isn't identified in your example.  I'm not sure what those parts are referring to.  Not very helpful page.

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: f4cb3748-12d0-b494-5fff-71c06e2973d1
Version Independent ID: b013a1e2-5879-5924-9dbd-a8db54885d10
Content: Get-ADUser (activedirectory)
Content Source: docset/winserver2012-ps/activedirectory/Get-ADUser.md
Product: w10
GitHub Login: @brianlic-msft
Microsoft Alias: brianlic

The error message tells you that PowerShell cannot find the Get-ADUser cmdlet. That means you do not have the module loaded.
You can easily add the AD RSAT tools to provide this module. Or use implicit remoting to import the module into your current PowerShell session.
Seachhbase is the OU you want this command to Target. This means the cmdlet just returns the AD uses in the specified OU or the OU and child OUs.
The port 3268 is the port to connect to the Global Catalog.

Hello @michcleary ,
We are closing this issue since your support question seems to be answered. Feel free to re-open this issue if you still experience problems.
Thanks.

@AndreaBarr Recommended closure.

You can check out one of the best solution on the same issue
Here


Check Leap Year C

Hi, can you please assign me this issue?
Thank you.

Hello, can I be assigned this one? I can do this, surely. Thank you!


Lots of updates

Codecov Report
Patch coverage: 85.05% and project coverage change: -1.77% :warning:

Comparison is base (f03dd8e) 83.45% compared to head (9f4cfcc) 81.69%.
Report is 2 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main      #24      +/-   ##
==========================================
- Coverage   83.45%   81.69%   -1.77%
==========================================
  Files           8        8
  Lines         659      683      +24
==========================================
+ Hits          550      558       +8
- Misses        109      125      +16

Flag
Coverage Δ

unittests
81.69% <85.05%> (-1.77%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

Files Changed
Coverage Δ

intake_axds/utils.py
82.94% <75.92%> (-6.64%)
:arrow_down:

intake_axds/axds.py
44.05% <100.00%> (-1.90%)
:arrow_down:

intake_axds/axds_cat.py
91.42% <100.00%> (+1.19%)
:arrow_up:

tests/test_axds_cat.py
100.00% <100.00%> (ø)

tests/test_axds_sensor.py
90.16% <100.00%> (-9.84%)
:arrow_down:

tests/test_utils.py
100.00% <100.00%> (ø)

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


Null-aware operator with cascade doesn't work with newlines?

Following code works:
void main() {
  int? x;
  x?..abs()..ceil()..floor();
}

This code gives an error:
void main() {
  int? x;
  x? // <- Conditions must have a static type of 'bool' - line 3
    ..abs()
    ..ceil()
    ..floor();
}

?.. is parsed as a single token, so there is no support for putting whitespace between those three characters.

Should there be, though? This seems surprising.

What's even weirder is that ! works with newlines as expected:
if (point != null) {
  point!
    ..x = 5
    ..y = 10
    ..z = 15;
}

Well, there's a delicate balance in the choice of tokens. If we choose to consider ?.. as a single token then the lexical analysis will decide that it has encountered that token upon seeing ?.., which implies that it is parsing a conditional cascade. If we were to split all such tokens (thus allowing whitespace to occur in additional locations) then the token parsed at that point would be ?. As a consequence, there would be many more possible results, because <expression> '?' ... can be derived from several different grammar rules. So whenever we split a token we also create additional parsing complexity (and potentially ambiguity). It also reduces the available syntactic design space for the language, because syntactic extensions that were otherwise possible will now create (insurmountable) ambiguities.
The null check '!' is actually a <selector>, and this means that it is parsed in the same way as .myGetter or ?[42]. So we don't parse a!.b as a followed by !.b, it is a followed by ! followed by .b, whereas a?.b is a followed by ?.b.
So it may be convenient and tempting to be able to put whitespace everywhere (ok, except for the mid dle of an identifier ;-), but it doesn't come for free.


tests: hopefully fix jenkins build

Jenkins is currently failing after the banish PR. Looks like an action-at-a-distance test issue.
Use the choice manager in the currently failing test instead of just setting a property on it.

I'm confused, the code you're contributing is also the code I merged is it not??

https://github.com/kolmafia/kolmafia/pull/590/files#diff-59b620c79cdc7958f745ea45853bc0d859f56ad0b0a3456b081444e5769bd4b2R406-R408

Oh I see this is a different test. Let's merge this yes


Update version to 1.0.81-0 on staging

Update version to 1.0.81-0

🚀 Deployed to staging         in version: 1.0.81-0🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅

🚀 Deployed to production         in version: 1.0.81-4🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅


remember the last state

بسم الله الرحمن الرحيم
planning to save the state:

current page
bookmarks

to do so I think I should this repo might be useful alhamdo li Allah
but I will have to test it


Donation Support in HomeBridge UI

The dedication to improving and supporting this plugin is first class. You should add donation capabilities so people can show their appreciation. Even if doing for fun, can’t hurt to allow people to show their appreciation.

Just installed after watching the project for a few weeks. Pretty slick.

Working on adding this.  Thanks for the advice.

Its there.  Fee free to use!!


Polish DefaultSetProperty

Replace anonymous inner class by lambda

Reviewing cheatsheet
Before merging the PR, comments starting with

❌ ❓must be fixed
🤔 💅 should be fixed
💭 may be fixed
🎉 celebrate happy things

@bot-gradle merge

The merge queue build has failed. Click here to see all failures.

Thanks, @abstratt, please look again.

@bot-gradle test ACC

I've triggered the following builds for you. Click here to see all build failures.

AllConfigCacheTestsPullRequestFeedback build

@bot-gradle merge

@bot-gradle cancel

@bot-gradle merge

The merge queue build has started. Click here to see all failures if any.