is it important with right farmer_public_key when creating a plot with pool_contract_address?

If I do not set farmer_public_key when creating a plot with pool_contract_address, it will use the first farmer_public_key (fingerprint<PHONE_NUMBER>)in my key list by default. In fact I use the third key (fingerprint<PHONE_NUMBER>)as a farmer joining the pool and it works. Why?

the farmer public key and be any one of the keys that are in the keychain of your farmer machine.


fluentbit: Add support for Kube_Meta_Cache_TTL parameter

Q
A

Bug fix?
no

New feature?
yes

API breaks?
no

Deprecations?
no

Related tickets
/

License
Apache 2.0

What's in this PR?

Adding support for another parameter from https://docs.fluentbit.io/manual/pipeline/filters/kubernetes#configuration-parameters
Why?

Currently it caches the metadata forever, so when changing a pods label it wont be reflected in the log output until fluentbit is restarted.
Additional context

Checklist

[x] Code meets the Developer Guide
[x] User guide and development docs updated (if needed)

To Do

 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.

@ahma fixed, please note I didnt have a chance to actually test this, I copied how the other options are exposed, and hope for the best :)


GSD Request

--- GSD JSON ---
{
  "vendor_name": "Linux",
  "product_name": "Kernel",
  "product_version": "versions from v4.2 to before v4.19.221",
  "vulnerability_type": "unspecified",
  "affected_component": "unspecified",
  "attack_vector": "unspecified",
  "impact": "unspecified",
  "credit": "",
  "references": [
    "https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux.git/commit/?id=ae6d9ce05691bf79694074db7c7da980080548af",
    "https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux.git/commit/?id=f5deab10ced368c807866283f8b79144c4823be8"
  ],
  "extended_references": [
    {
      "type": "commit",
      "value": "ae6d9ce05691bf79694074db7c7da980080548af",
      "note": "introduced"
    },
    {
      "type": "commit",
      "value": "f5deab10ced368c807866283f8b79144c4823be8",
      "note": "fixed"
    }
  ],
  "reporter": "joshbressers",
  "reporter_id": 1692786,
  "notes": "",
  "description": "iio: mma8452: Fix trigger reference couting\n\nThis is an automated ID intended to aid in discovery of potential security vulnerabilities. The actual impact and attack plausibility have not yet been proven.\nThis ID is fixed in Linux Kernel version v4.19.221 by commit f5deab10ced368c807866283f8b79144c4823be8, it was introduced in version v4.2 by commit ae6d9ce05691bf79694074db7c7da980080548af. For more details please see the references link."
}
--- GSD JSON ---

This issue has been assigned GSD-2021-1002431


Update env_logger to 0.7.1

This avoids introducing duplicate versions of env_logger in crates that
depend on both env_logger 0.7 and file-per-thread-logger.

Thanks Josh! I shall set up one bot for dependency updates.

Also just published a new version to crates.io.


Add omega into s03

Currently s03 does not have an omega axis and so Artemis fails to run against it, we should add one in.

turns out it does have one, but it's named Omega instead of OMEGA as expected

@dperl-dls tried to do this but S03 wasn't playing ball. I will investigate

Confirmed using the following test that it is now correct:
import pytest

from artemis.devices.I03Smargon import I03Smargon

@pytest.mark.s03
def test_when_smargon_created_against_s03_then_can_connect():
    smargon = I03Smargon("BL03S", name="smargon")

    smargon.omega.wait_for_connection()

However, with the new PVs added in https://github.com/DiamondLightSource/python-artemis/pull/371 doing smargon.wait_for_connection() will fail. I'll make a new issue for this.


Change from NotFound error to BadRequest - 404 to 400

Current coverage is 97.33% (diff: 100%)

Merging #177 into master will increase coverage by 0.01%

@@             master       #177   diff @@
==========================================
  Files            37         37
  Lines           967        974     +7
  Methods         950        967    +17
  Messages          0          0
  Branches         17          7    -10
==========================================
+ Hits            941        948     +7
  Misses           26         26
  Partials          0          0

Powered by Codecov. Last update 4cda751...2cef87e

closing to enable clean merge


go mods

any chance you can version control the dependencies ?

Yep. I just haven't gotten around to it for a lot of my repositories yet. PRs welcome though.

Fixed.


Freeze Protection

Thank you for this excellent integration, I have waited years to be able to control my MrCool multi-zone unit.  Everything seems to be working perfectly, but I do not see a setting to turn on the Freeze Protection function.  Am I just missing it?  If not, it would be handy to have access to this function as I use this mini split in a remote cabin where the units live in FP mode for a good portion of the winter.

@the-gray-ghost I didn't integrate the FP mode, i don't use it. But i'll check on it and come back to you. :)

Try the last version 1.4.5
add on this commit : a254e736775075c0636e1891674842a959c1b5a2

Wow, thank you for the fast update!  I installed the update and have tested it out.  For me, the update turns on the unit into "heat" mode with a minimum temperature of 50 degrees F.  However, it does not appear to be turning on the actual "Freeze Protection" mode.  This is a separate mode that is turned on/off like "Turbo" in the Cielo app and differs in functionality by allowing a minimum temp as low as 35 degrees before turning the unit on and heating up to 45 degrees before powering off.  This low temp range is not achievable under the standard heating mode.  Sorry, I don't mean to be nit-picky, but since you don't use these features, you may not be aware that there is a functionality difference between them.  With this capability, we could use every feature built into these units, which would be awesome!  I do greatly appreciate you looking into this!

I also wanted to add that Freeze Protection doesn't change the set temperature setting of the heat mode when turning freeze protection on/off, it just "resumes" the previous settings.

@the-gray-ghost when when i test it with my device on this website https://home.cielowigle.com/.
Everythings work fine!

But the app said i must be on heat before activate the FP mode.

Ok, I was able to enter this mode properly.  However, there seems to be lag with this mode reflecting changes.  I'm also getting a lot of "snap back" when I change settings.  Some changes are giving me instant feedback, but others are not.  Unfortunately, my internet at the location is currently going in and out (which isn't helping), so I am having difficulty testing.  I will let you know what I find out.

Can you confirm me my behavior if it's ok? When you activate the FP mode i check if your are in heat mode. If your are not i change it to heat and after activate the FP mode. If you are already in heat mode i just activate the FP mode.

Ok, it appears I can enter FP mode without issue, I just have to manually refresh the Cielo app to see the change.  However, I am unable to exit FP mode through HA.  Turning off the FP switch or changing any settings snaps back to the FP settings after only a second or two.  I have to turn FP mode off in the Cielo app to restore control to HA.
On a side note, would it be possible to get the FP mode "switch" in the thermostat controls as a "preset" like "Turbo" mode is now (this is how the Cielo app presents it) instead of being it's own special switch?  Or, does it need to live as a separate switch for automation or other reasons?

which Cielo thermostat or controller you use?


Remove deprecated NewFixedWatcher

Last user, in knative.dev was the eventing repo.
That is cleaned up here:
https://github.com/knative/eventing/pull/1927
/hold

/lgtm
/approve
@matzew just drop the hold when the other PR merges

/hold cancel


Can several DB be backup to one file? (Not issue)

Hello
First of all, thank you very much for your hard work.
I have been using the tool and its amazing.
I quick questions nevertheless. Is it possible to backup several databases to the same file?
Thanks in advance

As an alternative to backup all database in single file, you can backup all of them to separate files within a single click button.
This concept is demonstrated in the following article:
https://www.codeproject.com/Tips/1260485/Backup-All-MySQL-Databases-in-Csharp-WinForm
A sub-project of MySqlBackup.NET:
https://github.com/MySqlBackupNET/MySqlBackup_All_DB


Volume value percentage text

My suggestion is to show the sound's volume value text next to volume's slider.

It works in mobile?
It can also be shown in the desktop version as text.


Support PAS, CD PAS, and DPAS pubids

We need to add tests for:

PAS in WD stage => WD PAS
PAS in CD stage => CD PAS
PAS in DIS stage => DPAS
PAS in IS stage => PAS
Also, we need to prohibit FDIS stage for PAS type by raising an error.

ISO/DPAS 5643:2021(E)
ISO/PAS 5643:2021
ISO/PRF PAS 5643
ISO/PRF PAS 5643:2021
ISO/DPAS 50010
ISO/PAS 5112:2022
ISO/DPAS 22101-1.5   (this means DIS iteration 5)
ISO/PAS 22101-1:2022
ISO/SAE DPAS 22736:2021
ISO/SAE PAS 22736:2021
ISO/SAE PRF PAS 22736
ISO/CD PAS 50010
ISO/CD PAS 24019
ISO/CD PAS 22399

@ronaldtse Shouldn't "ISO/CD PAS 22399" be rendered as "ISO/PAS/CD 22399"? See requirements in #105

@mico No, the non-IS document type ("PAS") never has a trailing slash ("PAS/CD" is invalid). "ISO/CD PAS 22399" is correct.

The correct patterns are:

"{multiple-publishers} {stage + doctype}"
"ISO/{stage + doctype}"

There is no contradiction with #105.

@mico No, the non-IS document type ("PAS") never has a trailing slash ("PAS/CD" is invalid). "ISO/CD PAS 22399" is correct.

But we have ISO/TR identifiers like "ISO/TR 12134:2010" - should it look something like ""ISO/WD TR 12134:2010" if stage applied?

@mico yes you're correct, "ISO/WD TR 12134:2010" is accurate.

@ronaldtse we don't include "IS" for supplements and "PAS" documents. Is there any other documents for which we exclude "IS" from output?

@mico Only "IS" is omitted from all PubIDs. Supplements will always have the full base PubID. PAS documents are just like TR and TS.


contrail-controller api is not ready: keystone connection down

contrail-controller api is not in ready state during bundle deployment.  Please see reason messages below.  The workaround was to restart configapi_api_1.
contrail-controller/0             waiting      idle       18         <IP_ADDRESS>                                             api is not ready. Reason: (Generic Connection:Keystone[] connection down)
contrail-controller/1*            waiting      idle       19         <IP_ADDRESS>                                             api is not ready. Reason: (Generic Connection:Keystone[Error: An unexpected error prevented the server from fulfilling your request. (HTTP 500) (Request-ID: req-c267e248-6e5a-4599-abf5-657ca7dbd264) at UTC 2019-07-10 23:59:27.985450] connection down)
contrail-controller/2             waiting      executing  20         <IP_ADDRESS>                                             (juju-run) api is not ready. Reason: (Generic Connection:Keystone[Error: An unexpected error prevented the server from fulfilling your request. (HTTP 500) (Request-ID: req-d65ecf57-15fc-44e8-883d-0a25be51ae9b) at UTC 2019-07-11 00:01:52.759035] connection down)

looks like that keystone code is broken again in config-api.

fixed in contrail's code.


Consider a way to allow custom API convention selection

The current implementation does not allow for the API convention selection behavior to be customized. It's tied strictly to the implementation of analyzers with no way to choose an alternate algorithm for setting this. We should revisit this prior to 2.2.0-RC and see if we have had any feedback on if there's a need to customize this. I'd like to avoid building an extensibility behavior that makes it difficult for us to evolve the design of this feature.
Suggested design for this would be to do the discovery as part of the ApiBehaviorApplicationModelProvider and stick this in the ControllerActionDescriptor.Properties

This is done as part of https://github.com/aspnet/Mvc/commit/94a7c839984b7143ba83ed2173b70ff25c401f0e. ApiConventionResult can be set as part on ActionDescriptor.Properties in a convention.


upstream merge template repository

Merge https://github.com/NuschtOS/template 9a7f951

Didn't we want to group them to not have 10 PRs every week?


Consider additional use-cases to conclude a function as "used"

Description
This PR fixes issue 121. It modifies the A0-1-3 query to consider functions as used for the following use-cases.
The address of a function is taken
The operand of an expression in an unevaluated context
Functions marked with [[maybe_unused]]
Explicitly deleted functions e.g. =delete
Use of any overload of a function in an overload set constitute a use of all members of the set. An overload set is a set of functions with the same name that differ in the number, type and/or qualifiers of their parameters.
Change request type

[ ] Release or process automation (GitHub workflows, internal scripts)
[ ] Internal documentation
[ ] External documentation
[x] Query files (.ql, .qll, .qls or unit tests)
[ ] External scripts (analysis report or other code shipped as part of a release)

Rules with added or modified queries

[ ] No rules added
[ ] Queries have been added for the following rules:

rule number here

[x] Queries have been modified for the following rules:

A0-1-3

Release change checklist
A change note (development_handbook.md#change-notes) is required for any pull request which modifies:

The structure or layout of the release artifacts.
The evaluation performance (memory, execution time) of an existing query.
The results of an existing query in any circumstance.

If you are only adding new rule queries, a change note is not required.
Author: Is a change note required?

[x] Yes
[ ] No

🚨🚨🚨
Reviewer: Confirm that format of shared queries (not the .qll file, the
.ql file that imports it) is valid by running them within VS Code.

[ ] Confirmed

Reviewer: Confirm that either a change note is not required or the change note is required and has been added.

[ ] Confirmed

Query development review checklist
For PRs that add new queries or modify existing queries, the following checklist should be completed by both the author and reviewer:
Author

[x] Have all the relevant rule package description files been checked in?
[x] Have you verified that the metadata properties of each new query is set appropriately?
[x] Do all the unit tests contain both "COMPLIANT" and "NON_COMPLIANT" cases?
[x] Are the alert messages properly formatted and consistent with the style guide?
[x] Have you run the queries on OpenPilot and verified that the performance and results are acceptable?As a rule of thumb, predicates specific to the query should take no more than 1 minute, and for simple queries be under 10 seconds. If this is not the case, this should be highlighted and agreed in the code review process.
[x] Does the query have an appropriate level of in-query comments/documentation?
[x] Have you considered/identified possible edge cases?
[x] Does the query not reinvent features in the standard library?
[x] Can the query be simplified further (not golfed!)

Reviewer

[ ] Have all the relevant rule package description files been checked in?
[ ] Have you verified that the metadata properties of each new query is set appropriately?
[ ] Do all the unit tests contain both "COMPLIANT" and "NON_COMPLIANT" cases?
[ ] Are the alert messages properly formatted and consistent with the style guide?
[ ] Have you run the queries on OpenPilot and verified that the performance and results are acceptable?As a rule of thumb, predicates specific to the query should take no more than 1 minute, and for simple queries be under 10 seconds. If this is not the case, this should be highlighted and agreed in the code review process.
[ ] Does the query have an appropriate level of in-query comments/documentation?
[ ] Have you considered/identified possible edge cases?
[ ] Does the query not reinvent features in the standard library?
[ ] Can the query be simplified further (not golfed!)

@lcartey : Kindly help review this PR.

@rak3-sh also needs clang-format.

Applied in https://github.com/github/codeql-coding-standards/pull/347/commits/07a78fe455bb3dac829becad1886801bb29babd0

@lcartey : Thanks a lot for your valuable inputs. I've reworked based on your suggestions. Kindly let me know in case of any further questions.

@lcartey : Kindly refer to commit: a8e929a for the requested changes.

@lcartey : Thanks a lot for your help in fixing the documentation. I've committed the same in 9396451


Error when compiling with -Wall -Werror

I saw this when I enabled warnings as errors, just noting it down so I don't forget.
write all outputs:47:97: error: incompatible integer to pointer conversion passing 'iint_t' (aka 'long long') to parameter of t
ype 'iarray_iint_t' (aka 'iarray_struct *') [-Werror,-Wint-conversion]
                error = psv_output_int (fd, buffer, buffer_end, &buffer_ptr, iarray_iint_index (iarray_iint_index (p0->gonzo_ix
_0, gonzo_0_i), gonzo_0_i));

Program (source : Stream (Sum Error String))
Precomputations:
equine morbillivirus4 =
    Left []
Stream transformers:
gou virus7 =
    source
chickenpox8 =
    source
camelpox5 =
    source
Reductions:
measles0 =
    rfold (
    \smallpox6
    \chickenpox5
    Some Struct [(yellow,"")])
    (Sum_fold# (
     \measles4 Some Struct [(yellow
                            ,"")]) (
     \smallpox6 None) (Left ()))
    camelpox5
Postcomputations:
equine morbillivirus2 =
    equine morbillivirus4
rotavirus5 =
    not# (False)
Returning:
gonzo =
    [[]]

I believe this is fixed by https://github.com/ambiata/icicle/pull/319


Consider adding Redis or similar

Currently, some data structures are just represented as JS variables.
E.g., the automated scheduler stores all of its scheduling history info as JS arrays, objects, maps, etc., and so won't easily scale, won't persist across server crashes/restarts, and isn't stored in a way that would allow future analysis.
The other potential thing that we might want to move to Redis is the oracle mode global flag.

What's the advantage over storing these things in postgres?

Went with Postgres for now.


pcntl extension is missing/not enabled in image

Hi,
thank you for this cool tiny image.
While playing with it, I have discovered that it is impossible to use -j(aka --processes) option.
Staketrace should be self-explanatory
$ docker run -v $PWD:/mnt/src --rm -u "$(id -u):$(id -g)" cloudflare/phan:latest -j=2
Error: Call to undefined function Phan\pcntl_fork() in /opt/phan/src/Phan/ForkPool.php:53
Stack trace:
#0 /opt/phan/src/Phan/Phan.php(160): Phan\ForkPool->__construct(2, Array, Object(Closure), Object(Closure), Object(Closure))
#1 /opt/phan/src/phan.php(42): Phan\Phan::analyzeFileList(Object(Phan\CodeBase), Array)
#2 /opt/phan/phan(2): require_once('/opt/phan/src/p...')
#3 {main}

@akovalyov this should have been fixed by #2 ;) (and my testing confirms this?)

@fruitl00p thank you for feedback. I had an old version of image cached on my machine. docker latest tag was confusing. After I pulled latest latest, everything works. Thanks again!


No module named 'events'

Tebless version: 1.2.3
Python version: 3.9.5
Operating System: Windows 10

Description
I was trying to import the Window widget.
What I Did
For the following code:
from tebless.widgets import Label, Window, Input

I get the following error:
Traceback (most recent call last):
  File "F:\Coding\Projects\sound-recognition\pico_test.py", line 1, in <module>
    from tebless.widgets import Label, Window, Input
  File "C:\ProgramData\Anaconda3\envs\blessed\lib\site-packages\tebless\widgets\__init__.py", line 5, in <module>
    from .window import Window
  File "C:\ProgramData\Anaconda3\envs\blessed\lib\site-packages\tebless\widgets\window.py", line 18, in <module>
    from events import Events
ModuleNotFoundError: No module named 'events'

Am I missing some specific module to make this work on Windows, or was it never supported on Windows?
I tried installing events but then I only got further errors.

pip3 install events fixed it for me.

this lib require events package but i have my own version i added it to the requirements.txt file


Ensure the 'shape' field of LWPOLYLINE is boolean

Unit tests are failing because the 'shape' field of a LWPOLYLINE is
expected to be a boolean, but is an integer.
Commit 1b00a3027dcec02b39170c9086a1dc5447b234ec misuse some operators
(operator '&' has a lower precendence than operator '==='),
causing (curr.value & 1 === 1) to be parsed as (curr.value & (1 === 1)).
This patch add parenthesis to make sure the '1' flag of curr.value is
properly tested.

@orange-vertex This project has unit tests, take advantage of them!
Unit tests are still failing due to the 'rotation' field of TEXT, introduced by 194d528320c1bfb632bf07e32530496ae177fb29 not being present in the unit test expected results.
I'll submit another patch about that.


NoArg Deprecation

Deprecated NoArg as scala.DelayedInit is dropped from Scala 3 already,
Removed NoArg from Scala 3 build.
Updated scaladoc about the scala.DelayedInit being dropped in Scala 3.

@cheeseng please wait with merging, I will come back to this soon.

@cheeseng please wait with merging, I will come back to this soon.

Hi @mkurz , just checking if you have a better idea for this?

@cheeseng Sorry for the delay. Please don't remove NoArg from the Scala 3 build (even though it is doing nothing). I think the deprecation is enough to make people aware.
I want to keep it because in scalatestplus-play I want to offer people a smoother migration instead of having compiler errors where users maybe don't know what to do.
So if someone moves from Scala 2 to 3 (by simply changing the scalaVersion in build.sbt) what they will see in scalatestplus-play is following error (instead of compile error if you would remove NoArg):
[info] MixedFixtureSpec:
[info] org.scalatestplus.play.MixedFixtureSpec *** ABORTED ***
[info]   scala.NotImplementedError was thrown inside "The App function" must, construction cannot continue: "
[info]
[info]   For Scala 3 you need to wrap the body of App in an `override def running() = ...` method:
[info]
[info]   // Old:
[info]   new App() {
[info]
[info]   }
[info]
[info]   // New:
[info]   new App() {
[info]     override def running() = {
[info]
[info]     }
[info]   }
[info]
[info]   " (MixedFixtureSpec.scala:66)

I implemented that hint here: https://github.com/playframework/scalatestplus-play/pull/320/commits/36fdb5eb755a75096675084986f2b95f79789544
Ands wrapped the test methods here: https://github.com/playframework/scalatestplus-play/pull/320/commits/60ddc34462ac730b1b4193bc05961f30623cdf0e
Therefore I am asking you to not remove NoArg in the Scala 3 build but keep it wit the deprecation warning.

IMHO it should be fine to keep it. It was here all the time in the Scala 3 build and no one was bothered by it :wink:

@cheeseng Yes it should be fine to keep NoArg in the Scala 3 build, at least for a good long time. We used to keep deprecated things in the library for at least two years. With Scala 2 to 3, because people will take longer to upgrade, we should probably keep it even longer.

@bvenners @mkurz fyi I have undo the NoArg removal from Scala 3 build.
Thanks!


TypeError: no_grad.init() on import

Trying to import  from segmoe import SegMoEPipeline, throws:
File "C:\Users\xyz\AppData\Roaming\Python\Python310\site-packages\segmoe\main.py", line 89, in <module>
  class SegMoEPipeline:
File "C:\Users\xyz\AppData\Roaming\Python\Python310\site-packages\segmoe\main.py", line 1260, in SegMoEPipeline
  def get_hidden_states(self, model, positive, negative, average: bool = True):
TypeError: no_grad.__init__() takes 1 positional argument but 2 were given

I couldn't update torch because of another program's dependency. Ended up installing segmoe in venv instead, that worked.

I couldn't update torch because of another program's dependency. Ended up installing segmoe in venv instead, that worked.

i wonder which version of torch, transformers and diffusers you are using? i faced the same problem, and my env is: torch==2.0.1, transformers==4.38.1, diffusers==0.26.3


Use grunt in build process

replace Makefile with Gruntfile.js
build is now dist
updated examples and updated CSS to Bootstrap 3.3.2

@katspaugh :) refs #315

Thank you very much!


Go To Definition for Functions

We already collect data of all functions when running the scanner for autocomplete. As it uses regular expressions, we have the exact file position. It should be possible to implement a feature to "Go To Definition" when standing on a function call.
Given this language doesn't use reflection or lambdas this alone should be enough.

Addressed by https://github.com/gosukiwi/atom-agk/pull/10


Col xs={false} should be !important

<Col xs={false} [...] should lead to a style value of display: none!important.
The reason is that if you create a display: flex setting for the column for normal operations, it will override display: none for xs screens. And xs={false} is a hard rule.

I encounter this problem too but display: none !important will leads to other problems. Try yourself by simply adding:
diff --git a/src/components/Col.js b/src/components/Col.js
index 2f06d0a..f2e7f4c 100644
--- a/src/components/Col.js
+++ b/src/components/Col.js
@@ -59,7 +59,7 @@ export default styled(Col)`
           display: block;
         `
         // Hide element
-        : 'display: none;'
+        : 'display: none !important;'
       )
     }`)
   }

And see the last example in demo, the Col will never be visible:

Due to !important weight...
I don't have solution at this time but I am really open to propositions ;)

I suppose the media selector for display: none !important could indicate a max width: 64em. But for larger screens you would need to define a min-width AND a max-width for this case. Just thinking out loud. Not sure if this would not lead to more problems.


fix: bump isort version

isort recently released a bugfix that fixes Poetry installation. We can upgrade our isort version to accommodate this.

@chrisjsewell I'd prefer to merge this following your review. Let me know if you're not going to be able to get to it — my judgement here is that this is a non-invasive, well motivated bugfix that only affects our dev environment, so I'm happy to merge it if you don't have availability. Just let me know!

@agoose77 ah this one got superceded by merging https://github.com/executablebooks/MyST-NB/pull/460 - sorry for the confusion. Generally speaking you should feel free to self-merge clear fixes / maintenance improvements like this as long as they aren't too complex.


Barrage projectile count isn't working

Projectile count added by Dying Sun and +2 Barrage enchant aren't working com dps calculation.

Both are fixed in the next update.


Avoid using userType.unwrap(), prefer UserType.unwrap(userType)

less indirections = cheaper gas

closed in #1567


test: Ignore the format-zero-length for sprintf

As printf("") will trigger a warning of format-zero-length in
build stage, and warning is treated as error by default. This
test case is failed to build. In order to test this, -Wno-format
-zero-length is added for this test case, to make it run.
Signed-off-by: Hu Zhenyu<EMAIL_ADDRESS>

This PR is to fix https://github.com/zephyrproject-rtos/zephyr/issues/47714

For MEC15, the problem is gone when update the cmake to 3.24.rc3
The issue will also be tracked by https://github.com/zephyrproject-rtos/zephyr/issues/47588. So this PR can be closed.


Bug: Incorrect tagging for flowers + stew

What happened?
PL's flowers should be in the small flowers tag, wild flax should be in tall flowers tag, both block and item tags.
Should double check suspicious stew recipes for the flowers as well
Mod Version
Beta 2.1.0
Fabric API Version
No response
Relevant log output
No response
Other mods
No response
Additional Information
No response

i fixed this literal years ago oops


[Feature]: ImageMaid to Ignore Uploads/themes/

Describe the Feature Request
Greetings!
I recently discovered https://github.com/LizardByte/Themerr-plex which I've been really enjoying. One thing I've noticed however is unfortunately the themes that Themerr adds to the movies is also marked as a bloat file by ImageMaid. This leads to /Uploads/themes/ being removed on a 'remove' ImageMaid run; unfortunately 'breaking' the themes.
Hoping it would be possible to have an additional options or flag that would prevent ImageMaid from treating those uploaded themes as bloat files but still removing the /Uploads/posters/ that are indeed bloat.
Attaching a link to a snippit of the logs while running on 'report' that shows those themes being marked as bloat files.
https://pastebin.com/egK802mg

I am the maintainer of Themerr, and to add some, hopefully, helpful information.
We "lock" the themes, just as if the user were to lock a any field of metadata or image. This script should probably ignore anything that is locked as the user intentionally set it that way.


'Auto-Close Reconcile Popup' feature causes closure before adjustment is even entered

Name of Defective Feature
Auto-Close Reconcile Popup
System Settings (Required)
ToolKit Version: 1.1.0
Operating System (Mac/Windows/Etc): Mac
Browser (Chrome/Firefox/Safari): Chrome
Browser Version: version 59.0.3071.115
Other Extensions: 1Password, Buffer, Authy, Grammarly, Trello, Bitly, Pinterest, WhatFont, Honey, Ebates, Privacy Badger, Page Monitor, Protect My Choices, HTTPS Everywhere, Star this page (Favorites), Adplock Plus, Print Friendly & PDF, Justdelete.me, Boomerang, Inbox Pause, iChrome, Application Launcher for Drive
Export of Toolkit Settings (Bottom Left of Settings Page)
[{"key":"AccountsDisplayDensity","value":"1"},{"key":"AccountsEmphasizedOutflows","value":false},{"key":"AccountsStripedRows","value":false},{"key":"AdditionalColumns","value":true},{"key":"AdjustableColumnWidths","value":false},{"key":"AutoCloseReconcile","value":true},{"key":"ChangeEnterBehavior","value":false},{"key":"CheckNumbers","value":false},{"key":"ColourBlindMode","value":false},{"key":"CustomFlagNames","value":false},{"key":"DisplayTargetGoalAmount","value":"0"},{"key":"GoalWarningColor","value":false},{"key":"HideAgeOfMoney","value":false},{"key":"HideReferralBanner","value":true},{"key":"PrintingImprovements","value":false},{"key":"RemovePositiveHighlight","value":false},{"key":"RowHeight","value":"0"},{"key":"RunningBalance","value":false},{"key":"ShowCategoryBalance","value":false},{"key":"SquareNegativeMode","value":false},{"key":"StealingFromFuture","value":false},{"key":"TargetBalanceWarning","value":false},{"key":"accountsClearSelection","value":false},{"key":"activityTransactionLink","value":true},{"key":"betterScrollbars","value":"2"},{"key":"budgetBalanceToZero","value":true},{"key":"budgetProgressBars","value":"0"},{"key":"budgetQuickSwitch","value":false},{"key":"budgetRowsHeight","value":"0"},{"key":"calendarFirstDay","value":"0"},{"key":"categoryActivityPopupWidth","value":"0"},{"key":"checkCreditBalances","value":false},{"key":"collapseExpandBudgetGroups","value":false},{"key":"collapseSideMenu","value":true},{"key":"compactIncomeVsExpense","value":false},{"key":"currentMonthIndicator","value":false},{"key":"daysOfBuffering","value":false},{"key":"daysOfBufferingHistoryLookup","value":"0"},{"key":"editButtonPosition","value":"2"},{"key":"enableRetroCalculator","value":false},{"key":"enlargeCategoriesDropdown","value":true},{"key":"goalIndicator","value":false},{"key":"googleFontsSelector","value":"0"},{"key":"hideAccountBalancesType","value":"0"},{"key":"hideHelp","value":true},{"key":"highlightNegativesNegative","value":false},{"key":"importNotification","value":"0"},{"key":"incomeFromLastMonth","value":"0"},{"key":"l10n","value":"0"},{"key":"largerClickableIcons","value":false},{"key":"monthlyNotesPopupWidth","value":"0"},{"key":"navDisplayDensity","value":"1"},{"key":"options.dark-mode","value":true},{"key":"pacing","value":"0"},{"key":"popupCalculator","value":true},{"key":"privacyMode","value":"0"},{"key":"reconciledTextColor","value":"0"},{"key":"removeZeroCategories","value":false},{"key":"reports","value":true},{"key":"resizeInspector","value":false},{"key":"rightClickToEdit","value":true},{"key":"seamlessBudgetHeader","value":false},{"key":"showIntercom","value":true},{"key":"spareChange","value":true},{"key":"splitKeyboardShortcut","value":true},{"key":"splitTransactionAutoAdjust","value":true},{"key":"swapClearedFlagged","value":false},{"key":"toBeBudgetedWarning","value":false},{"key":"toggleSplits","value":true},{"key":"toggleTransactionFilters","value":"0"},{"key":"warnOnQuickBudget","value":false}]

Steps to Reproduce the Behavior
Please write down the steps necessary to reproduce the behavior you're seeing. Please be as thorough as possible.

Add as many features one by one as I deemed useful.
Click 'Auto-Close Reconcile Popup' to ON
Save Toolkit Settings
Go to YNAB and reload page
Go to an account and click 'Reconcile Account'
Click 'No' to make an adjustment.

Expected Behavior
Able to enter the current amount of the account for the adjustment.
Actual Behavior
Popup disappears before any amount can be entered (remains on screen for roughly 1-2 seconds)

Hey @snales thanks for the report!
This was supposed to be fixed in our last release but it didn't make it in. We'll have this fixed for sure on the next one! Sorry.

Ah! Excellent. I thought that the 1.1.0 was the fix. Thanks for the update!

I just encountered this one also. Should not happen if "No" is clicked; only if "Yes" is clicked.

I'm getting this as well. Do you know if there is there a certain feature I should disable or just disable the whole extension until resolved?

Just turn off the Close Reconcile Notification Popup in Accounts. Worked for me. I will try it again next revision.

This problem had been fixed and has reappeared this week, presumably with the latest YNAB updates.

Closing this issue since YNAB has implemented the feature.


Almost get flag managers working, needs cleanup

This looks like the correct approach - all vtables and most weak functions are in the right order - but the four weak dtors from the subclasses at the end of the file are reversed for some reason. Help appreciated. Otherwise this obviously needs cleanup...

I cannot figure out how the ordering works either from the small bit of playing around i did. It certainly feels better


Adding Callback Support for Tool Execution (Stream Mode)

Adding Callback Support for Tool Execution (Stream Mode)
A new callback option has been implemented in the stream processing component of the AI service, allowing consumers to receive notifications during tool execution. Users can also know which tools are being called in stream mode, just as they would in call mode.

Change

Callback for Tool Invocation Results (Stream Mode)
General checklist

[X] There are no breaking changes
[ ] I have added unit and integration tests for my change
[ ] I have manually run all the unit and integration tests in the module I have added/changed, and they are all green
[ ] I have manually run all the unit and integration tests in the core and main modules, and they are all green

[ ] I have added/updated the documentation
[ ] I have added an example in the examples repo (only for "big" features)
[ ] I have added/updated Spring Boot starter(s) (if applicable)

@sakoojiang thank you, but PRs without tests are declined, so please add tests


ios launcher icons with more padding, optimized

cc @HansMuller @Hixie

closes https://github.com/flutter/flutter/issues/5404

This LGTM but having 3 copies of ~40 files doesn't seem like a good idea

I am starting to get concerned as to the number of binary files in the flutter repo. Every one of these files (even the ones we've deleted) have to be downloaded by everyone who uses flutter.

Can we move them into a google storage bucket and pull them down like we do with other binary artifacts?

@Hixie I've added this as a separate issue.

Would it be ok to land this now and then figure out how to deal with binary
assets? I don't think I'm greatly increasing the number of files? Also, I
ran optipng on everything:)
On Wed, Aug 17, 2016 at 15:49 Dragoș Tiselice<EMAIL_ADDRESS>wrote:

@Hixie https://github.com/Hixie I've added this as a separate issue
https://github.com/flutter/flutter/issues/5471.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/flutter/flutter/pull/5470#issuecomment-240572970, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAAVZ8SQdsEtnuOVCk7jYQz9TpG9__nRks5qg4_wgaJpZM4Jm7xu
.

Would it be ok to land this now and then figure out how to deal with binary assets?

Once you land this, the binary files will be in the history forever.  It's probably better to address this issue before digging ourselves deeper into this hole.

OK, just tell me where to put the new icons. Thanks!

Opened https://github.com/flutter/flutter/issues/5472 to track the intention to pull down binary assets from a source external to the repo.

Talked to @abarth and I can land this one. But we do want a long-term solution for assets that can live outside the repo.


Feature/add tada animation

Description of Change

This is very much a work in progress but it is here to show some progress on the pre-built animations along with a samples section.
https://user-images.githubusercontent.com/17139988/126558891-9d5a300e-7565-4136-b87e-8c10e0b263cf.mov
Bugs Fixed

Fixes #

API Changes

Behavioral Changes

PR Checklist

[ ] Has a linked Issue, and the Issue has been approved
[ ] Has tests (if omitted, state reason in description)
[ ] Has samples (if omitted, state reason in description)
[ ] Rebased on top of main at time of PR
[ ] Changes adhere to coding standard

[ ] Updated documentation

Corrected this mess in #1527


Give this project a purpose, see where this goes

haha awesome! Maybe someone will come along and document how to run this in the README. :wink:

Will continue to add useless features to be documented.

On May 17, 2015, at 11:04 PM, Ryan Seys<EMAIL_ADDRESS>wrote:
Merged #4.
—
Reply to this email directly or view it on GitHub.


Coding Challenge 150 Runway Rainbows

I am trying to do this from a hotspot with barely any connection and having trouble, any help appreciated!


Any chance to succeed without the AS3 code ?

Hi,
congrats for this great work.
My kids have their teachers send them to  a lot of flash player based animations created by other school teachers years ago, that nobody can use any more.
I tried to help in downloading the SWF and converting it thanks to your app. I produce interesting results but not quite what it should look like.
Exemple here : http://www.pccl.fr/physique_chimie_college_lycee/quatrieme/electricite/lois_intensites_noeuds_mesures.htm

Hey! Thanks.
Unfortunately, this is not possible without the as3 code. I also tried decompiling your SWF, and as2 code was found there. It is not compatible with this tool.

You can try this SWF player written on js.
This player able to work with as2.

Thank you very much for your help.


Unable to delete/remove attachment from work item by using VSTS api

Currently, we are developing a tool by using VSTS api (https://docs.microsoft.com/en-us/rest/api/azure/devops/wit/attachments/get?view=azure-devops-rest-5.0) + WorkItemTrackingHttpClient to manage attachments and work items for our customers. But there is no option to delete/remove an attachment from the work item. Is this feature already available, please give me an update or can you enable that feature?
Regards,
Eranga

@erangaapp That's good feedback, thanks for letting us know. I am forwarding this request to have the ability to delete/remove an attachment to work items to the product team and closing this issue, since the issues in this repo are for content feedback. Although I am forwarding this to the product team, it will carry more weight if you also file a feature request here:
https://developercommunity.visualstudio.com/spaces/21/index.html
You can also submit product feedback, such as bug reports or feature requests, by choosing Product feedback at the bottom of any article:

Thank you.

Is there any update/solution for this? I'm in the same boat now.


[1195] Implement save on confirm - personal details form

Context
https://trello.com/c/fkj9L8rN/1195-l-implement-save-on-confirm-personal-details
Changes proposed in this pull request

Pull in the spike code from #534
Fix all the broken tests
Tweaked ConfirmDetailsController to work with existing non-redis sections and the new redis-backed approach

Note: we have to call clear_form_stash in a few places to handle the situation where the user enters changes but doesn't confirm, but instead cancels or clicks the back button.
Guidance to review

Visit the personal details section and make changes. If you confirm, they're permament otherwise if you cancel and click the back button, those changes will be discarded.

 [L] Implement save on confirm - personal details

Visit the personal details section and make changes. If you confirm, they're permanent otherwise if you cancel and click the back button, those changes will be discarded.

This isn't the behaviour I was expecting. For non-drafts, changes shouldn't be permanent until you confirm they're correct from the confirmation page. We shouldn't accept and then reject the changes - we should not accept until they're confirmed.

There's at least two bugs with nationality:
1: the multiple nationalities js doesn't seem to be running:

Other nationality doesn't appear to get submitted
It also seems to accept not having a nationality.

Gender also doesn't appear to get set correctly.

I've submitted a gender and a Nationality, but this is what gets shown

Issues - these values were filled in - they're now missing. If they are missing, we should have had a validation error.


Illuminate\Http\Request and helper request() can have different values

Laravel Version
10.14.1
PHP Version
8.2.3
Database Driver & Version
No response
Description
Good afternoon. It is correct that Illuminate\Http\Request and helper request() may be different.
Simple example:
    $request->request->set('asd', 'test');

    var_dump($request->get('asd')); // return test

    dd(request()->get('asd')); // return null

Shouldn't they be identical?
Steps To Reproduce
$request->request->set('asd', 'test');   dd(request()->get('asd')); // return null

@Samoylenko666 I have tested but its returns "test" for both output.
Laravel Version
10.14.1
PHP Version
8.2.7


.eh_frame based stack walking - v1

List of things we're planning to clean up and merge in main before releasing the next version of the agent.

[x] Reverting back to the C toolchain #869
[x] Unwind table generation + printing #867
[x] Validation and correctness tests/program, related binaries https://github.com/parca-dev/parca-agent/pull/917
[x] Add the BPF unwinder, behind a flag

All of this has been merged now in the main with the final PR #948.


add meta-arm and its dependencies

Development moved to https://github.com/zarhus/meta-zarhus-bsp-rockchip/pull/2. Closing this PR, thanks for contribution.


Default closure based ImageFilter struct

I think many people will like to create filters on the fly with something like that:
let filter = ImageFilter("MyFilter") { image in
    // Do something the image
   return image.someTransformation()
}

It would be really convenient in some cases to avoid creating many files for isolated cases. In this example, the filter would have "MyFilter" as its identifier. (I don't think we can automatically generate a same unique identifier for this on-the-fly filter every time it is created / copied)
This would require to rename the protocol ImageFilter into ImageFilterType.

The same could be applied to (to be announced) CompositeImageFilter type. It could be created with a variadic array of ImageFilterType or a variadic array of Image -> Image closures / functions.
let composite = CompositeImageFilter(filters: SizeToFillFilter(size: size), CircleFilter(), BlackAndWhiteFilter())

CompositeImageFilter protocol would be renamed CompositeImageFilterType.

We can even think of a custom operator to create composite image filters:
let filter = SizeToFillFilter(size: size) => CircleFilter() => MyCustomFilter()

which is clearer and shorted than:
let filter = CompositeImageFilter(filters: SizeToFillFilter(size: size), CircleFilter(), MyCustomFilter())

I think something like DynamicImageFilter or CustomImageFilter that takes an identifier and trailing filter closure in the initializer could definitely be useful. If you want to take a stab at this along with some tests, that would be awesome!
As for a similar type of CompositeImageFilter, I think it would be good to have one that has an initializer that takes an array of filters. I think a variadic initializer is fine as well. How about both? Also, how about a name like DynamicCompositeImageFilter?
Now as for adding operator overloads, I'm hesitant. If you'd like to put that together in the PR, then I'll gladly take a look. No promises though on actually merging that. Just thinking through it off the top of my head it would have to merge two ImageFilter comforming objects into a CompositeImageFilter, then append additional filters into the composite. It would certainly be more clear and only slightly more verbose to use a variadic initializer on the DynamicCompositeImageFilter. My two cents...

@cnoon ok, let's first start with DynamicImageFilter and DynamicCompositeImageFilter. I think we'll have to use them before deciding wether custom operators are useful for creating composite filters.
I'll try to PR in the next few days.

Okay sounds great. Just let me know if you need anything from me!

Completed in #14.


nemo guardrails

detailed visibility into datasets guardrails will have to be instrumented directly since nemo is more of a generic flow execution framework it seems


Add scala solution for 198-House-Robber

File(s) Modified: 198-House-Robber.scala
Language(s) Used: scala
Submission URL: _https://leetcode.com/submissions/detail/767416714/_

Thanks, @ChrisKheng!


Fix simulator builds by adding +nightly

Adding +nightly to simulator builds  seems to fix pd-rs/crank/issues/57

Thanks! 👍
But what if user hasn't rustup, but just nightly toolchain?

I don’t use rustup and adding +nightly causes errors for me.
It would be great if this could do something like check for the presence of rustup or be configured via an environment variable.

Rustup existence can be determined by env var, yes. Example.

Maybe just call crank so cargo +nightly crank? I'm not sure now, but if there cargo executable determined by CARGO env, it could fix this. 🤷🏻♂


Allow topics to have configurable retention

Probably an extra option on the topic declaration.
Not sure what happens to subscriptions that are outside the retention period.

Putting this back in triage because the original urgency is now negated with the encrypted fields


Enhance User Security by Adding Password Strength Validation to Registration Form (#471)

Fixes #471
This PR adds password strength validation to the new user registration form, ensuring that users create a secure password that meets specific strength requirements. The validation requires that passwords contain at least one lowercase letter, one uppercase letter, one number, and one special character, helping to protect user accounts.
Changes

Updated the registration form to include password strength validation.
Implemented validation rules in the frontend and backend to check for:
At least one lowercase letter
At least one uppercase letter
At least one number
At least one special character
Added error messages to inform users if their password does not meet these requirements, guiding them toward stronger passwords.
Ensured cross-browser and cross-device compatibility for consistent behavior.

Testing Instructions

Pull this branch.
Run npm install to install dependencies.
Run npm test to execute the test suite.
Verify that ...

Screenshots (if applicable)
Uploading 09.11.2024_12.15.41_REC.mp4…
Checklist

[x] I have performed a self-review of my code
[x] I have commented my code, particularly in hard-to-understand areas
[x] My changes generate no new warnings
[x] I am working on this issue under GSSOC

@dipexplorer recording?

@Soujanya2004 .
sry for this i am facing is issue every time either video or image not uploaded at time.
i need to waite for an hour to upload this.
check linkedin


refactor(website): convert website to TypeScript

Motivation
Originally from https://github.com/facebook/docusaurus/pull/5233#discussion_r684116642. Dogfood TS support on the official Docusaurus website, while also enforcing type checks for other contributors.
Have you read the Contributing Guidelines on pull requests?
Yep
Test Plan
Build passes; pages like:

https://deploy-preview-5328--docusaurus-2.netlify.app/docs/styling-layout/#styling-your-site-with-infima
https://deploy-preview-5328--docusaurus-2.netlify.app/versions
https://deploy-preview-5328--docusaurus-2.netlify.app/showcase/

are working as intended

LGTM thanks 👍 good catch for the edit URL 😅


[CLEANUP] javadoc: namedcluster saved in embedded metastore

Build Completed
:x: This pull request has errors. They will need to be addressed before it can be accepted. See below for more details. Some links are also available below for further assistance in addressing those issues.
Build Commands
mvn -B -fn -f 'pom.xml' -pl 'engine' -P '!assemblies' -amd clean install && mvn -B -f 'pom.xml' -pl 'engine' -P '!assemblies' -amd site

Cleanup Commands
mvn -B -f 'pom.xml' -pl 'engine' -P '!assemblies' -amd build-helper:remove-project-artifact

Changed files
  engine/src/main/java/org/pentaho/di/trans/steps/named/cluster/NamedClusterEmbedManager.java

Newly Broken Tests:
org.pentaho.di.core.logging.MetricsPainterTest.testDrawTimeScaleLineInfinityLoop:
:red_circle: java.lang.Exception
java.lang.Exception: test timed out after 1000 milliseconds
	at org.pentaho.di.core.logging.MetricsPainterTest.testDrawTimeScaleLineInfinityLoop(MetricsPainterTest.java:65)

Unit test coverage change
These statistics help you identify how your changes have affected the coverage of the following files. If a file is not in this list, then its coverage was not affected by your changes. To get some help interpreting these metrics, please refer to Jacoco's documentation.
org.pentaho.di.core.logging.MetricsPainter

Branch Change: -8.33%:small_red_triangle_down:
Complexity Change: -10.53%:small_red_triangle_down:
Instruction Change: -19.04%:small_red_triangle_down:
Line Change: -17.91%:small_red_triangle_down:
Method Change: -14.29%:small_red_triangle_down:

org.pentaho.di.job.entries.evalfilesmetrics.JobEntryEvalFilesMetrics

Branch Change: -.77%:small_red_triangle_down:
Complexity Change: -1.02%:small_red_triangle_down:
Instruction Change: -.16%:small_red_triangle_down:

org.pentaho.di.job.entries.sql.JobEntrySQL

Instruction Change:  + 3.28%
Line Change:  + 3.87%

org.pentaho.di.job.entries.waitforsql.JobEntryWaitForSQL

Branch Change: -.66%:small_red_triangle_down:
Complexity Change: -.93%:small_red_triangle_down:
Instruction Change: -.13%:small_red_triangle_down:

org.pentaho.di.trans.steps.calculator.CalculatorMetaFunction

Branch Change:  + 8.70%
Complexity Change:  + 3.03%
Instruction Change:  + .75%
Line Change:  + .89%

org.pentaho.di.trans.steps.combinationlookup.CombinationLookupMeta

Instruction Change: -.04%:small_red_triangle_down:

org.pentaho.di.trans.steps.constant.ConstantMeta

Instruction Change:  + .10%

org.pentaho.di.trans.steps.datagrid.DataGridMeta

Branch Change:  + 1.92%
Complexity Change:  + 1.69%
Instruction Change:  + .17%

org.pentaho.di.trans.steps.groupby.GroupByMeta

Branch Change:  + 1.04%
Complexity Change:  + 1.05%

org.pentaho.di.trans.steps.ifnull.IfNullMeta

Branch Change:  + 2.00%
Complexity Change:  + 1.89%
Instruction Change:  + .17%

org.pentaho.di.trans.steps.insertupdate.InsertUpdateMeta

Instruction Change:  + .05%

org.pentaho.di.trans.steps.ldapoutput.LDAPOutputMeta

Instruction Change:  + .06%

org.pentaho.di.trans.steps.memgroupby.MemoryGroupByMeta

Branch Change: -2.25%:small_red_triangle_down:
Complexity Change: -2.60%:small_red_triangle_down:

org.pentaho.di.trans.steps.replacestring.ReplaceStringMeta

Instruction Change:  + .06%

org.pentaho.di.trans.steps.rowgenerator.RowGeneratorMeta

Instruction Change: -.08%:small_red_triangle_down:

org.pentaho.di.trans.steps.setvalueconstant.SetValueConstantMeta

Instruction Change: -.14%:small_red_triangle_down:

org.pentaho.di.trans.steps.sort.SortRowsMeta

Instruction Change: -.07%:small_red_triangle_down:

org.pentaho.di.trans.steps.synchronizeaftermerge.SynchronizeAfterMergeMeta

Instruction Change:  + .04%

org.pentaho.di.trans.steps.tableoutput.TableOutputMeta

Branch Change: -.91%:small_red_triangle_down:
Complexity Change: -.82%:small_red_triangle_down:

org.pentaho.di.trans.steps.textfileinput.TextFileInputMeta

Branch Change: -.53%:small_red_triangle_down:

org.pentaho.di.trans.steps.uniquerows.UniqueRowsMeta

Instruction Change:  + .18%

org.pentaho.di.trans.steps.googleanalytics.GaInputStepMeta

Instruction Change:  + .07%

org.pentaho.di.ui.repository.pur.repositoryexplorer.model.UISecurityRole

Branch Change: -1.79%:small_red_triangle_down:
Complexity Change: -1.54%:small_red_triangle_down:
Instruction Change: -.16%:small_red_triangle_down:
Line Change: -.58%:small_red_triangle_down:

org.pentaho.di.trans.steps.salesforceinsert.SalesforceInsertMeta

Instruction Change: -.15%:small_red_triangle_down:

@pentaho/ackbar @tkafalas

I can't reproduce the test error locally.

AFAIR the test is failing like this for years

The failure happens when the system is on high load, failing any test with timeouts.


Error in reproducing "analyzer_with_table_transformer_notebook.md"

Bug 💥
The second cell of the notebook is crashing when it attempts to load model weights. It appears to be calling a PyTorch method on a model which, in the debugger, I found to have TensorFlow variables. Reproduce in Colab here: https://colab.research.google.com/drive/1QK4N4xwoX3FkEPqjXPdbyC68uGjAo3Wy?usp=sharing
Screenshots 🖼

Environment:
Google Colaboratory

Many thanks!
There is actually a bug when setting the priority env variables USE_TF, USE_TORCH if both Tensorflow and Torch are installed.

 How can I avoid this bug in the short term? I tried setting those env variables as shown here:

The Colab is updated to include that attempted, but unsuccessful fix -
https://colab.research.google.com/drive/1QK4N4xwoX3FkEPqjXPdbyC68uGjAo3Wy#scrollTo=cZ43sflbQ7m7

Ok, thanks for the response. I was able to fix this bug by adding:
os.environ["DD_USE_TORCH"] = "1"
os.environ["DD_USE_TF"]="0"
os.environ["USE_TORCH"] = "1"
os.environ["USE_TF"] = "0"

Before the line where I import deepdoctection (import deepdoctection as dd)

I have provided a fix so that PyTorch is the default library even if Tensorflow is also installed. It should be equivalent to your variable configuration.
You can find the fix on the master branch. It has not been added to a new release yet.

I just pip installed from the master branch and the fix you describe solved my issue. Thanks for the quick fix!

Closing with https://github.com/deepdoctection/deepdoctection/commit/bfa642e302b069ca2aae8cc67cad8e4c0e20016b.


"odo url create" should work without an active connection with the OpenShift cluster

Tangential question: shouln't odo url create work without having an active connection with the OpenShift cluster? 🤔

Yes, it should, there is nothing that we need from the cluster.
Originally posted by @kadel in https://github.com/openshift/odo/issues/1213#issuecomment-501231074

/close
this is already covered in https://github.com/openshift/odo/issues/1588#issuecomment-501206443


Token caching?

The TokenClient works as expected, but do you foresee adding support for caching of that token, or is that outside the scope of the library?
For example, if the token expiration is in 3600 seconds, you could call something like client.GetOrRequestClientCredentialsAsync() and until close to expiration, you would continue to get the same response.AccessToken?

token caching is out of scope. This is a protocol library.


why when I change the answer if it don‘t know, it will return a wrong answer 【Preventing hallucination】

prompt = """Answer the question as truthfully as possible, and if you're unsure of the answer, say "sorry, 我不知啊".
Q: Who won the 2020 Summer Olympics men's high jump?
A:"""
openai.Completion.create(
prompt=prompt,
temperature=0,
max_tokens=300,
model=COMPLETIONS_MODEL
)["choices"][0]["text"].strip(" \n")
it return 'Marius Lindvik of Norway.'  but it expected to answer 'sorry, 我不知啊'

Unfortunately the models will still sometimes make mistakes, even with the "if you're unsure of the answer, do X" instructions.


Textual definition update: GO:0003896   DNA primase activity

Guidelines
See http://wiki.geneontology.org/index.php/Guidelines_for_GO_textual_definitions#Use_genus-differentia_patterns_for_definitions
More details at https://douroucouli.wordpress.com/2019/07/08/ontotip-write-simple-concise-clear-operational-textual-definitions/

GO term ID and label for which you request a definition update
GO:0003896  DNA primase activity

New proposed definition (genus-differentia pattern, plus additional information if needed)

Catalysis of the synthesis of a short RNA or DNA primer on a DNA template, providing a free 3'-OH that can be extended by DNA-directed DNA polymerases. PMID:26184436, PMID:29608762

Reference(s)
PubMed=29608762

Additional information
It turns out that PRIMPOL UniProtKB/Q96LW4 has a primase activity but preferentially uses DNA primers instead of RNA primers.

Also edited the corresponding GO:0006269 BP term DNA replication, synthesis of primer

[ ] Changed label to DNA replication, synthesis of primer
[ ] Fixed definition from "The synthesis of a short RNA polymer, usually 4-15 nucleotides long, using one strand of unwound DNA as a template; the RNA then serves as a primer from which DNA polymerases extend synthesis." [PMID:11395402] to  "The synthesis of a short RNA polymer, usually 4-15 nucleotides long, using one strand of unwound DNA as a template; the RNA then serves as a primer from which DNA polymerases extend synthesis. In some cases the primer can be made of DNA." [PMID:11395402, PMID:29608762]

@ValWood Are you OK with obsoletion of the children
GO:1902295 synthesis of RNA primer involved in cell cycle DNA replication >> 1 pombase TAS
GO:1902330 synthesis of RNA primer involved in bacterial-type DNA replication >> no annotation
GO:1902318 synthesis of RNA primer involved in nuclear cell cycle DNA replication >> no annotation
These are precomposed terms, the proper DNA replication term should be sufficient. What do you think ?
Thanks, Pascale

@lbreuza
Looking at this more closely, in fact the parentage is now incorrect: GO:0003896 DNA primase activity is a
GO:0003899 DNA-directed 5'-3' RNA polymerase activity
So, it seems we may need a new term for this activity. How is this represented in RHEA?
Thanks, Pascale

yes, but here Lionel is asking that we also include DNA primers, so wouldn't that make it a DNA polymerase as well?

There is no Rhea but please have a look there : https://enzyme.expasy.org/EC/<IP_ADDRESS>

I'm a bit confused. I thought  that the definition of primase was the activity of adding RNA molecules using a DNA template, because the primer for repliciation is defined as an RNA sequence.
If it is adding DNA molecules isn't it  a DNA polymerase? so in this case it would have both primase or polymerase activities?
i.e
A primer is a short nucleic acid sequence that provides a starting point for DNA synthesis. In living organisms, primers are short strands of RNA****. A primer must be synthesized by an enzyme called primase, which is a type of RNA polymerase, before DNA replication can occur.
or is this an out of date view?

It seems some primases make a DNA primer under certain conditions, see the paper cited by @lbreuza  PubMed=29608762(and you can find many more in PubMed)

OK, I didn't know that ! I don't think there's a yeast ortholog as far as I can tell.

[ ] Changed label: GO:0006269  DNA replication, synthesis of RNA primer to  DNA replication, synthesis of primer
[ ] GO:0006269 Changed definition: from: "The synthesis of a short RNA polymer, usually 4-15 nucleotides long, using one strand of unwound DNA as a template; the RNA then serves as a primer from which DNA polymerases extend synthesis." [PMID:11395402]
to :  "The synthesis of a short nucleotide polymer, usually a RNA molecule between 4-15 nucleotides long, using
one strand of unwound DNA as a template. This primer is extend by DNA polymerases for DNA synthesis. In certai
n conditions, for example in response to DNA damage, some primases synthesize a DNA primer." [PMID:11395402, PM
ID:38203225, PMID:38492718]
[ ] DNA primase activity: changed definition from "Catalysis of the synthesis of a short RNA primer on a DNA template, providing a free 3'-OH that can be extended by DNA-directed DNA polymerases." [GOC:mah, GOC:mcc, ISBN:0716720094, PMID:26184436] to  "Catalysis of the synthesis of a short RNA primer on a DNA template, providing a free 3'-OH that can be extended by DNA-directed DNA polymerases. In certain conditions, for example in response to DNA damage, some primases synthesize a DNA primer." [GOC:mah, GOC:mcc, PMID:11395402, PMID:26184436, PMID:38203225, PMID:38492718]
[ ] Added EC:<IP_ADDRESS> {source="skos:narrowMatch"}
[ ] Made  EC:<IP_ADDRESS> "skos:narrowMatch"

Thanks, Pascale


WRP-7019: Update package dependencies to fix security vulnerabilities

Checklist

[x] I have read and understand the contribution guide
[ ] A CHANGELOG entry is included
[x] Documentation was added or is not needed
[ ] This is an API breaking change

Issue Resolved / Feature Added
Updated dependencies to the latest minor version.
Resolution
Updated dependencies to the latest minor version.
Additional Considerations
Links
WRP-7019
Comments
Enact-DCO-1.0-Signed-off-by: Taeyoung Hong<EMAIL_ADDRESS>

What was the intention of https://github.com/enactjs/enact/pull/3132/commits/f31130b86c31478a5348fd0e207ba7aa06233128?

What was the intention of f31130b?

The intention is to prevent vulnerability coupling issues(refer to the comment of WRP-3071).
I set the ground rule to use npm install --production.


request READ_MEDIA_IMAGES permission for Android SDK >= 33

The PR requests for READ_MEDIA_IMAGES for Android SDK >= 33 [1] and for SDK 29 to 32, it requests READ_EXTERNAL_STORAGE as mentioned in [2]. For SDKs lower than 29 it assumes the permission is granted (the current plugin behavior).
References:
[1] https://developer.android.com/reference/android/Manifest.permission#READ_MEDIA_AUDIO
[2] https://developer.android.com/reference/android/Manifest.permission#READ_EXTERNAL_STORAGE

Would really love this to be merged as it fixes my issues too.

@chiraganand @rapelt done in 3.0.1


[Bug]: Continuously creating 1000 tables each containing 10,000 fields, the service terminates

Is there an existing issue for the same bug?

[X] I have checked the existing issues.

Branch name
main
Commit ID
5751f09
Other environment information
No response
Actual behavior
When continuously creating 1,000 data tables containing 10,000 fields of different types, the service was terminated. Service terminated after table creation for 683th times.

Expected behavior
No response
Steps to reproduce
Just use for loop to create 1000 tables with 10000 columns with various column types.

Additional information
No response

function test code can be found in python/test/test_table.py  test_various_tables_with_various_columns

It is OK to create so many tables for now. But if we shutdown the server and restarted, infinity will be terminated. Reason:

Read WAL entries from WAL file.
Construct the catalog structure in memory.
Force to a full checkpoint, which will generate a json structure according to the catalog. This part will cost lots of memory, if memory is not enough, the program will be terminated.

Solution 1. During system restart, before trigger full checkpoint, CLEAN UP need to be done. #635
Solution 2. Split catalog to small parts, use a cache such as LRU to manage the catalog.
Solution 3. Using a memory saving catalog engine, such as leveldb/rocksdb.

Now, no errors are encountered during the creation of tables, or during the shutdown or restart of the service.


Overhaul CI/CD Process

Should we move to Github Actions?  Are there benefits in terms of:

Build times?
Simplicity of secret management?
Ease of publishing releases to sonatype and docker via Github Actions>

We just made the transition ourselves with Apache TrafficControl.  The biggest thing we gained from it was the ability of PR authors to make changes to GHA without having to manage a separate trust for trusted contributors.  This also allowed us to leverage some of our more complicated test suites that have docker-compose based sidecars.

We punted on manual releases, it is possible in Travis but if we are going to revisit doing releases via CI it makes sense to take a look at it.  There is a lot of precedent at this point

I am going to move this to Ready and pick it up.  We need fully automated public releases, so we should be on actions, manage secrets in GH, and publicly release here as well.
Will need separate issues to follow suit for the client libraries

I recently merged in a PR for Github Actions for:

PR build (verify + func test)
Merge to master (publish docs)

@nimaeskandary was looking into Docker.  fwiw, we already have a full blown docker build and release under the build directory that we can use.  We can release docker images and then for the time being notary sign them out of band after publishing (at least based on what I have ready).
I would like an initial release process that 1) creates a draft github release with master, auto assigns the next version number and 2) builds and publishes the releases to docker hub.  That would be a good first step until we get the notary thing figured out.

The only remaining thing on this issue now is to create a github workflow that:

Does a semantic version bump
Runs the docker release which is just a shell script /build/docker-release.sh -c -l -p - the arguments are "clean latest push".  Make sure to use the version bump
Once complete, do a github release and create and push a git tag.  I actually think you can just create and push the git tag with the version v0.9.7 for example and have another workflow pick up that event and generate the github release

Bonus points to generate a changelog on the Github release, looking back at commits / issues / prs or what have you since the prior release.  Seems like there are a few ways to do that.


Xcode 7b3 iOS 9b3 breaks VGParallaxHeaderMode.Fill

Hi, updating from Xcode 7b2 to Xcode 7b3 breaks the Fill mode. in Xcode 7b3 (Swift based project) the VGParallaxHeader acts like .Center mode and does not fill the header on pull down. Gaps appear above and below the 'header' with the underlying view showing through.

Sorry for late replay is it still happening

Not anymore. The b4 update resolved it - must have been a temporary bug. Sorry, forgot to close this!

No worries thanks for the info!


PackageCollectionsSigning: avoid peeking into X509_STORE_CTX

This uses the CCryptoBoringSSL_X509_STORE_CTX_get_current_cert
accessor to access the current_cert member.  This is purely a cleanup
change that will become important when swift-crypto is bumped as
BoringSSL has made more of the interfaces opaque.
[One line description of your change]
Motivation:
[Explain here the context, and why you're making that change. What is the problem you're trying to solve.]
Modifications:
[Describe the modifications you've done.]
Result:
[After your change, what will change.]

@swift-ci please test

CC: @yim-lee

@swift-ci please smoke test


WIP: Refactoring of multiple components: MNIST MLBox, SSH and Singularity runners

Summary
Runners

New Docker, Singularity and SSH runners.
Docker runner was refactored to provide the same CLI as other runners.
SSH runner supports Docker and Singularity runners on remote hosts.

Documentation

Docker, Singularity and SSH runners.
MNIST MLBox example.
Hello World MLBox example.

MLBoxes

Hello World was refactored to support all runners.

Issues
This PR implements #49, #61 and #64.

here are some minor comments:

shall we rename various occurrences of mlbox to mlcommons-box
maybe the directory mnist/platform can use plural form instead (mnist/platforms), so that it can be consistent with how existing directories are named (e.g. mnist/tasks)

+1 on using entrypoints to make the runner CLI a standalone implementation

Also, could you check the test failure?
And if possible add basic tests and CI for the runners? (We can file an issue and do this in a follow-up PR too)

I think the PR needs a rebase before merging.

@xyhuang @sub-mod @swiftdiaries @bitfort
Thanks for review! I think I addressed some of them. I also have several questions about others.

if we want to improve the UX a little bit, we may add an entry_points property in the runner's setup scripts, so that user can run the runner by directly calling mlbox_docker_run instead of python mlbox_docker_run. refer to the setuptools doc or this stackoverflow answer

[not fixed] MLCommons-Box runners already provide __main__.py files that define entry points. In this stackoverflow answer they say that setup.py is not required to provide entrypoints in this case. Or do we still need to define entrypoint because that's considered to be a good practice?

shall we rename various occurrences of mlbox to mlcommons-box

[not fixed] I looked at various sources where MLBox is mentioned and wanted to get your inputs before I update files.

How do we refer to MLBox library and MLBox instances in documentation files?

MLCommons-Box for the library?
MLCommons-Box runners for runners in general?
MLCommons-Box RUNNER for concrete runners, such as MLCommons-Box Docker Runner?
MLCommons-Box boxes for packaged workloads in general?
BOX-NAME MLCommons-Box  for concrete instances, such as Hello World MLCommons-Box, MNIST MLCommons-Box, Transformer MLCommons-Box?

Using these full qualified names makes documentation pages look not cool. Assuming the above notation has been introduced, maybe we can just use the library, runners and Docker Runner for the first three items. I am not sure about good short terms for the last two: boxes, MNIST box?
2. Do we need to rename directories for the reference runners?

mlbox_docker_run -> mlcommons_box_docker_run
mlbox_singularity_run -> mlcommons_box_singularity_run
mlbox_ssh_run -> mlcommons_box_ssh_run

maybe the directory mnist/platform can use plural form instead (mnist/platforms), so that it can be consistent with how existing directories are named (e.g. mnist/tasks)

[fixed]. What about run directory?

Move from typer to click

[fixed].

Do we want to version the libraries in the requirements files?

[fixed].

Also, could you check the test failure?

[fixed]. I think it was due to the wrong version of mlspeclib library (last week).

And if possible add basic tests and CI for the runners? (We can file an issue and do this in a follow-up PR too)

[not fixed yet].

I think the PR needs a rebase before merging.

Is this to keep the commit history? I do not know much about it, what about squash merge since I do not think intermediate commits are useful from historical perspective?
Other questions
The setup.py files are empty now for runners. Can I just reuse mlcommons-box's implementation with some minor changes?

Thanks for the changes @sergey-serebryakov !

Or do we still need to define entrypoint because that's considered to be a good practice?

I think we want some conformance across python packages in the repo. So if we decide on one and stick with it throughout, it's good practice for readability and for onboarding developers.
And secondly, I think we want all runners to be independently publishable to PyPi. So each runner is it's own standalone CLI/package.

Using these fully qualified names makes documentation pages look not cool.

I agree. We can codify some notations and denote it at the start of the docs and use it recurrently throughout the repo. I don't really have an opinion on the choices. I think the ones you picked are fine. Regarding the example mlboxes, we can call them boxes in plural and <use-case> box similar to how you proposed can be an option.

Do we need to rename directories for the reference runners?

Yes, I think we can rename them so that whatever the PR team comes up with we can uniformly search-and-replace. If we have divergent names, we might have some potential toil when we have to rename.

Can I just reuse mlcommons-box's implementation with some minor changes?

Yes, we'll automatically ensure commonality across the repo and developers adding new runners will also tend to follow established code practices. I think we want repeatable patterns in the common library and the runners.

The main.py files of all three runners look exactly the same. Two differences - runner and metadata classes. I guess it can be refactored somewhow.

This is probably a minor nit imho. If you feel strongly about it, we can file an issue to better address it in a later sprint.

Hi! I addressed 3 comments:

Renamed various occurrences of mlbox to mlcommons_box. This includes documentation, code comments and directory structure (that resulted in different package names for runners - mlcommons_box_docker_run etc.).
Added implementation for setup.py files including entrypoint specification. This is  basically a copy of the mlcommons_box's setup.py
Added primitive CLI unit tests and CI workflow steps for runners. Same as current mlcommons_box implementation.
Removed existing content from runners' README files, added references to documentation pages.

TODO

Rebase.
I need your advice. Runners depend on mlcommons_box package. This includes, for instance, the metadata parser.

Is it a good idea to have this package dependency (runner -> mlcommons_box)?
What will runners' requirements.txt files contain - something like mlcommons_box==0.1?

Rebase done. Remaining question - runners' dependency on MLCommons-Box library.

Hey @sergey-serebryakov
Sorry for the late review.
Thank you so much for the patience with this PR 💯

need your advice. Runners depend on mlcommons_box package. This includes, for instance, the metadata parser.
Is it a good idea to have this package dependency (runner -> mlcommons_box)?
What will runners' requirements.txt files contain - something like mlcommons_box==0.1?

I think we want that dependency with the way we've structured the common libraries and schema.
Yes, the requirements.txt will have the same versioning as the mlcommons_box. If you're making changes with both the packages, the expectation is to change version numbers on both ends

Merging based on everyone's approval .


Title of link should be highlighted

I think title of a link should be highlighted because in my opinion the title is as important as the URL. Also for consistency automatic link should also be highlighted.

This seems to be a problem with your current theme. 'Solarized' or 'Twilight', for example, are working for me. From the other themes I tried, some work, some don't. In general, the dark and more colorful themes seem to map more style variables than others.
'Brackets Light' and 'Brackets Dark' are not working, unfortunately. I might change the appropriate style variables to some variables which are mapped by the default styles. However, this might have unwanted effects with other styles.
I am pretty unsure about the best way to handle this.

Good question... how can we know which token type are mapped for a theme ?
Maybe we could use a more restrictive subset of token types when we don't know the theme and use a richer subset of token types when the theme is Solarized, Twilight... ?
But I think the highlight should work with the default theme (at least for very common markups like bold, italic or lists)

... variables get the token type cm-variable-2 which is just the default text color.
https://github.com/adobe/brackets/issues/2842#issue-10869951

I can see that we're using variable-2 for list, numbered, unnumbered, is there any other token available ?

variable-2 is also used for the link title. I think we should just change that so that it works well for the default themes. However, I do not see an easy way to display the square brackets in the same color as the link and only hightlight the title separately. I suggest to simply display the whole link including square brackets and title in the same color.
As for the lists, we can just choose a different mapping which looks nice with the default themes.

Sounds good to me :+1:


Support Request:

Describe Your Problem
[SwitchBot] Was 'node-switchbot' found: false
Relevant log output
[SwitchBot] Was 'node-switchbot' found: false
[SwitchBot] Curtain: Bedroom wasn't able to establish BLE Connection, node-switchbot: false

Config for homebridge-switchbot
No response
Screenshots
Can't seem to get my bluetooth working, it was a fresh install
Device & Model
Raspberry Pi Zero W
Node.js Version
v16.15.1
NPM Version
v5.51
Homebridge Version
v1.5.0
Homebridge SwitchBot Plugin Version
v1.14.0
Homebridge Config UI X Plugin Version
No response
Operating System
Raspbian

Did you follow the prerequisites? On node-SwitchBot that this plugin directs too?

Hi @donavanbecker, I followed the instructions, including the noble prerequisites but also got the same issue.
[SwitchBot] [DEBUG] Meter uuid: F6BF2C272FDE-Meter, (95af9565-4d00-417c-855e-a6fb3cd2f90a)
[SwitchBot] [DEBUG] Meter: DE BLE Address: f6:bf:2c:27:2f:de
[SwitchBot] Meter: DE wasn't able to establish BLE Connection, node-switchbot: false

it is your operating system for some reason it is not able to connect with BLE, sorry not much I can do. If you can figure it out and can add something to like a helpful hints I can create a wiki for it.


Invalid WebSocket protocol in LiveClient constructor

What is the current behavior?
When using the v3 Deepgram SDK in a ClouldFlare worker, the LiveClient constructor creates a WebSocket with an invalid protocol when running deepGram.listen.live({ model: 'nova' });. This results in a "WebSocket Constructor: The protocol header token is invalid" error.
[mf:err] SyntaxError: WebSocket Constructor: The protocol header token is invalid.     at new LiveClient      at ListenClient.live      at Object.fetch      at async jsonError
Steps to reproduce

Call deepGram.listen.live({ model: 'nova' }) in ClouldFlare worker with Node.
Observe the "WebSocket Constructor: The protocol header token is invalid" error.

Expected behavior
Not produce error
Please tell us about your environment

Operating System/Version: MacOS Ventura
Language: TypeScript
Browser: Node / ClouldFlare worker

Thanks for letting me know. It seems the cloudflare worker makes the SDK think it's not a browser version of the websocket, and tried to define headers.
I'll look more at this today!

This remains unresolved!

fixed in v3.0.0-beta.4


Adding Date Range as a Column Header Filter in Data Table Grid

Summary
In the data table grid, under header filters, there's a date picker option, but not a date range. We would like your team to add your date range component as a header filter.
Examples
https://mui.com/x/react-data-grid/filtering/header-filters/
https://mui.com/x/react-date-pickers/date-range-picker/
Motivation
In the MUI data table grid, we want users to have the ability to filter a date column by a date range as a column header filter.
Search keywords: date range, header filter

Hey @spencer-rf! I'll migrate this issue to the MUI X repo so the team can take a look.


How to install the brig cli tool properly

Hi,
I'm on OSX, I've followed the various guides and I've built the brig via the make commands. But there are still a few issues:

The brig client isn't available in the terminal (command not found). However I can use it by navigating to the bin folder.
The brig client doesn't seem to recognize the Kubernetes cloud

I can see that I have projects with the helm command, but they are nowhere to be seen with the brig project list command.
I've tried explicitly specifying context and config file but with the same result. Also tried mapping $KUBECONFIG to the ./kube/config file.

Brigade seems nice but so far it's really difficult to get everything running with missing pieces in the documentation.

I've solved the 2nd issue (probably a noob issue, but there may be more of us out there!).
Add this to your users ".profile" file:
#KUBECONFIG
export KUBECONFIG=/Users/richardpergament/.kube/config

.... And you can solve the 1st issue in a similar way. Add this to your .profile:
PATH="/go/src/github.com/Azure/Brigade/bin:${PATH}"
Now the brig command will be linked to the built binary


Some limitations of public preview of Featured Tables can be removed

The updated limitations should be as follows:
Here are limitations for the initial preview.
•	Featured tables in Power BI datasets that use the following capabilities aren't shown in Excel:
o	DirectQuery datasets.
o	Datasets with a live connection.
•	Excel shows only data in columns and calculated columns in the featured table. The following aren't provided in the initial preview:
o	Measures defined on related tables, and implicit measures calculated from relationships.
•	Excel only displays featured tables that are stored in the new Power BI workspaces. Featured tables stored in the classic workspaces aren't shown as data types in Excel. You can upgrade classic workspaces to the new workspaces in Power BI.
•	See Considerations and limitations in the article "Access Power BI featured tables in Excel" for other Excel considerations.

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

ID: 74f8fd61-1d7c-6da2-7279-d88d72668a8a
Version Independent ID: ce00fe65-ac80-be40-1bbd-ac799e6a2c53
Content: Set featured tables in Power BI Desktop (preview) - Power BI
Content Source: powerbi-docs/collaborate-share/service-create-excel-featured-tables.md
Service: powerbi
Sub-service: powerbi-service
GitHub Login: @maggiesMSFT
Microsoft Alias: maggies

Hi, @markvsql -- Thanks for the update! I'll ping you offline about how you can make updates yourself!

@markvsql -- updating now.

Awesome. Thanks so much!
Mark Vaillancourt
(he/him)
Senior Program Manager
Power BI Customer Advisory Team
<EMAIL_ADDRESS>From: Maggie Sparkman<EMAIL_ADDRESS>Sent: Thursday, September 17, 2020 5:26 PM
To: MicrosoftDocs/powerbi-docs<EMAIL_ADDRESS>Cc: Mark Vaillancourt<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [MicrosoftDocs/powerbi-docs] Some limitations of public preview of Featured Tables can be removed (#2504)
@markvsqlhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fmarkvsql&data=02|01|mavail%40microsoft.com|2d9c193d54e54f314bb108d85b58a9cb|72f988bf86f141af91ab2d7cd011db47|1|0|637359784266440249&sdata=OvJKNiOi0V3zdt6kO6vmyG6cHZ%2F2SdGA2ItMIr%2BmBCU%3D&reserved=0 -- updating now.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicrosoftDocs%2Fpowerbi-docs%2Fissues%2F2504%23issuecomment-694532315&data=02|01|mavail%40microsoft.com|2d9c193d54e54f314bb108d85b58a9cb|72f988bf86f141af91ab2d7cd011db47|1|0|637359784266440249&sdata=%2BSdP7AexVqyh4QBSu3FC1kZhwt2%2FUZGU3W79wgwRrT8%3D&reserved=0, or unsubscribehttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAEGVAOXBFUOPFE2OHBLDZG3SGKEHXANCNFSM4RRBA2TA&data=02|01|mavail%40microsoft.com|2d9c193d54e54f314bb108d85b58a9cb|72f988bf86f141af91ab2d7cd011db47|1|0|637359784266450243&sdata=z1Znp2yFmnCjT65baIOe1o1BD%2BmESDdfWZ5EV2%2F%2BFWw%3D&reserved=0.


added workflow.yml

.github/workflows/workflow.yml -- the GitHub Actions file that automates testing

IBM Cloud toolchain: Delivery Pipeline deployed products to dev, including commit 7e6d4a76b5b677caeba31f93cbc71ac441c5773c

IBM Cloud toolchain: Delivery Pipeline deployed products to prod, including commit 7e6d4a76b5b677caeba31f93cbc71ac441c5773c


execute() should return true and non-null _resultSet when there is no match

Hi team, I am using jooq for querying Pinot but running into an exception when there is no match.
Caused by: java.sql.SQLException: Unknown column index : 2

I think the root cause is b/c:
1.PinotStatement/PinotPreparedStatement returns a null when there is empty-row(no match) result;
https://github.com/apache/pinot/blob/a75544f2c644874a88b61cea48b640d9cf05283c/pinot-clients/pinot-jdbc-client/src/main/java/org/apache/pinot/client/PinotStatement.java#L80-L90
line 82 returns a non-null _resultSet. However, since _resultSet.next() return false(rowCount == 0), _resultSet is reset to null and the method returns false.

Then Jooq tries to insert an UPDATE_COUNT row which fails to be parsed with the original cols

I wonder if this has any existing solution or we should update PinotStatement.execute()(and PinotPreparedStatement.execute()) to return a non-null ResultSet with 0 row instead(this looks the correct fix since an empty-row ResultSet should still be valid based on the discussion here).

The Jooq integration was introduced from this issue: https://github.com/apache/pinot/issues/6557

Seems like returning an empty resultSet should be the right approach? Can you file a PR?
cc: @KKcorps

iirc these classes are adhering to JDBC interfaces
if JDBC allows the signature change then we can do it

Folks, I was pointed here from twitter. Maybe, this blog post helps with the delicate JDBC statement execution protocol:
https://blog.jooq.org/how-i-incorrectly-fetched-jdbc-resultsets-again/
It's really not obvious to implement correctly, unfortunately.


what is this error?

opt.completeopt = "menu,menuone,noinsert,fuzzy,popup" some value of this option i implement on neovim main branch. if you are using release version of neovim remove fuzzy and popup


Configure: add missing dependency to fix parallel builds on Windows

The issue was encountered when testing parallel builds of OpenSSL on Windows using jom instead of nmake. The builds persistently failed with the following error message because the generated file "buildinf.h" did not exist yet.
crypto\info.c(15): fatal error C1083:
    cannot open include file: "buildinf.h": No such file or directory

Apparently this error does not occur on Linux because make parallelizes the builds differently such that crypto\cversion.c, which has an explicit dependency on buildinf.h, gets compiled first. Also, the
include dependency was added only recently in commit 096978f0990. (=> no backport to 1.1.1 required)

Ups, the email address seems to be misconfigured on my windows machine. I'll fix that later.

Very good catch!

Thanks for the praise. Actually, jom deserves it. But I learned a bit about the build system that way.
BTW: why isn't this include dependency detected automatically, just as all the others? Is there a special problem because the header file is autogenerated?

BTW: why isn't this include dependency detected automatically, just as all the others? Is there a special problem because the header file is autogenerated?

Yes.  For this to work, we would need to run a makedepend phase first, but we have chosen to have automatic dependency generation through compilation, so to have the initial dependencies complete, we need to make the few dependencies on generated files explicit

Merged to 387bbce45bfe9c0bccbd2c84206ca7719d463740, thanks!


Update go to 1.12.7

Let me know if this is accurate

Looks like validation failed:

!!! shasum verify check failed: checksums differ for /root/.gofish/go-1.12.7-darwin-amd64.gz: expected '9108ce8b3d47eb2edf10bb721e47ec10d0733c3a13a0a0bb86c67ca7178b0699', got '282a8b54ea8339bbe0d34b8bdc1128511d04a6bd875ac8596bc6cac708e287ef'

Closing this for different branch after actually RTFM ... sorry for spam

REF: #238


Broken Test For Yishai

This is the hello world of creating a domain and repo and it doesn't work.
{
  "Resources": {
    "CodeArtifactDomain": {
      "Type": "AWSdevToolsBeta::CodeArtifact::Domain",
      "Properties": {
        "DomainName": "richarddomain"
      }
    },
    "CodeArtifactRepository": {
      "Type": "AWSdevToolsBeta::CodeArtifact::Repository",
      "Properties": {
        "RepositoryName": "richardrepo",
        "DomainName": {
          "Fn::GetAtt": ["CodeArtifactDomain", "DomainName"]
        },
        "PermissionsPolicyDocument": {
          "Version": "2012-10-17",
          "Statement": [
            {
              "Action": "codeartifact:*",
              "Effect": "Allow",
              "Principal": "*",
              "Resource": "*"
            }
          ]
        }
      }
    }
  }
}

CFN throws the following error: Requested attribute DomainName must be a readonly property in schema for AWSdevToolsBeta::CodeArtifact::Domain

dependent on expansion of GetAtt beyond readOnlyProperties:
https://github.com/aws-cloudformation/aws-cloudformation-coverage-roadmap/issues/68

dependent on expansion of GetAtt beyond readOnlyProperties:
https://github.com/aws-cloudformation/aws-cloudformation-coverage-roadmap/issues/68


🐛 Bug Report: Gravatar logo used as profile picture when user has no Gravatar account

Reproduction steps

Create user with an email not associated with a Gravatar account
Login as the user
Refresh the page to load the profile picture (if needed)

Expected behavior
The profile picture should be the user's initials just like it was before the Gravatar integration (#34) .
Actual Behavior
The profile picture shows the Gravatar logo.
I assume that this is because the url of the gravatar picture is not associated to any user so i returns the Gravatar logo.
Maybe add some logic to check if the user exists on gravatar and if not, show the initials.

Thanks for reporting this. In v0.7.1 it should take the initials if the Gravatar avatar doesn't exist.


Match .nvmrc node version to contribution guide

The line in question in CONTRIBUTING.md:
https://github.com/CodingTrain/thecodingtrain.com/blob/055797c74191d7a7c7f2cb3267bd9627b52948d4/CONTRIBUTING.md?plain=1#L52
This also matters because Netlify references this file to decide which version of node the build process will use.

Good catch!


presidio-structured misidentifies email as URL

Presidio-structured incorrectly identifies an email address as a URL within the extracted entities. This can be observed in the following example output:
StructuredAnalysis(entity_mapping={'name': 'PERSON', 'email': 'URL', 'city': 'LOCATION', 'state': 'LOCATION'})
the value in the "email" column<EMAIL_ADDRESS>is mistakenly identified as a URL ("URL") instead of an email address ("EMAIL") during entity extraction.

I've also encountered this issue.
The issue mainly stems from the _find_most_common_entity() method where email addresses in test_structured.csv are being incorrectly identified as URLs, albeit with low confidence. It prioritises the entity with the highest count.
Observed behavior:

Entity Count: {'URL': 6, 'EMAIL_ADDRESS': 3}
Confidence Scores: {'EMAIL_ADDRESS': [1.0, 1.0, 1.0], 'URL': [0.5, 0.5, 0.5, 0.5, 0.5, 0.5]}

The emails are accurately recognised but are outnumbered by the URL identifications due to their higher frequency, despite lower confidence levels.
I would like to suggest two potential improvements:

Adapting _find_most_common_entity() to Consider Confidence Scores:  It might be beneficial to adjust the method to account for the actual confidence scores provided by the recognizer results.
Enhancing the URL Recognizer:  Improving the recognizer's ability to differentiate between URLs and email addresses could help reduce this type of misidentification

I'm keen to contribute to making these improvements and would love to work on refining the logic. Any thoughts or feedback on these suggestions would be greatly appreciated!

Thanks for the feedback! the URL recognizer detects parts of emails as well (e.g. microsoft.com is a url inside john.doe@microsoft.com), which makes it detect more URLs than emails.
I think that a good way forward here would be to allow the user to decide on a strategy for the entity selected. In some cases, we would want the entity with the majority of cases, in others we'd like the one that has the highest confidence, and in others we might want a mix of the two (e.g. most common entity, if confidence > 0.5)
A quick fix could be to update the structured analysis once finalized, in case the column's name is "email" but the detection is actually "URL".
If you're interested in creating a PR, I'd be happy to review it and discuss.


How about a NuGet package?

Hi there,
This would be really handy to have a Nuget pacakge so it can be installed via nuget.
I notice there is Fluent version of the Task Scheduler here on codeplex which has a nuget package but appears to only run in .net 4 and i am stuck currently in 3.5
Just my 2 cents :-)
Originally posted: 2011-03-21T04:15:55

Thanks for the pointer. I hadn't even heard of NuGet. Nice solution. I have uploaded the 1.6 files and they are now available through NuGet.
Originally posted: 2011-03-21T18:14:28


Safari iPhone issue

On iPhone the menu click do not scroll to the begining of the section. Works perfecly on ipad. Yes m'y sections do have différent heights.
Any idea

Do you have a link?

Yes
http://francois-lartigau.com/spip.php
But I Think I just get the point : the scroll down menu is the culpright as it changes the viewport height when down. I'll investigate some hacks monday.   I'll share if I find something

Appears to be working fine for me.

Did you test it in ios safari iphone?
On my side the scroll stops at mid sections...
eCreatures | Solutions agiles
Bâtiment Cre@ticité C
Technopole Izarbel
64210 Bidart
mob.<IP_ADDRESS>.23
tel. 09 <IP_ADDRESS>
fax. 09 <IP_ADDRESS>

Le 13 févr. 2015 à 20:42, Trevor Davis<EMAIL_ADDRESS>a écrit :
Appears to be working fine for me.
—
Reply to this email directly or view it on GitHub.

The top of the browser starts with the top of each section for me.

Thanks for testing but that's definitaly not the case on my device top h3 markup and begining of the text is hidden for most of the sections. Works perfectly on desktops and iPad where the hamburger menu is not visible
eCreatures | Solutions agiles
Bâtiment Cre@ticité C
Technopole Izarbel
64210 Bidart
mob.<IP_ADDRESS>.23
tel. 09 <IP_ADDRESS>
fax. 09 <IP_ADDRESS>

Le 13 févr. 2015 à 21:41, Trevor Davis<EMAIL_ADDRESS>a écrit :
with


Show bundle size badge instead of install size

5MB (install size) vs < 1 kB (bundle size) make a huge difference 📦 😋
<EMAIL_ADDRESS>

Yeah. I noticed that. However, bundle phobia is for browser bundles but defu is for Node.js. Maybe it is more realistic that we show actual "npm install" package size which included README and CHANGELOG. We can have both badges to indicate core is < 1kB :)