any technical reason why RNCryptor doesn't allow subclassing?

RNCryptor/RNCryptor.swift doesn't allow subclassing due to the final keyword for all the class definitions.
I'd love to subclass because the app I'm working on requires a somewhat different file format than RNCryptor (in order to be compatible with other architectures which don't get to use RNCryptor) and I'd like my subclass to be able to manipulate the data to & from that cross compiling format.
I'm sure other RNCryptor users have wondered about this as well.  I realize I can update my local copy of RNCryptor to remove all the final keywords and build from that, but is there some technical reason why it was decided to declare all classes as final?  If it's a performance reason, Swift 1.2 fixed things up.  Perhaps it's some other technical rationale?

The classes in RNCryptor are final because they're not designed for subclassing. Allowing arbitrary subclasses is fragile if not documented and tested. If you need to look at my implementation in order to write your subclass, then you're relying on an implementation detail that may change. It's hard to imagine many subclasses that you could safely write purely from the API documentation. (This is one of the great mistakes that happened in OOP, which is one reason there is frequent discussion of making final the default in Swift and a general leaning towards structs.)
As a rule, I strongly recommend composition rather than inheritance. If at all possible, create a type of your own that HAS-A RNCryptor rather than IS-A RNCryptor. I'd be happy to discuss any specific problem you're facing.
A subclass of RNCryptor that modifies the format is also not an RNCryptor. That's just Liskov. A subclass must be substitutable in every place that the superclass is usable. If MYModifiedCryptor cannot accept the RNCryptor format as input, then it it's incorrect to make it a subclass. So any interesting subclass you made would likely have to be more complicated than you desire.
RNCryptor is already designed for composition. You'll note that the "RNCryptor-level" objects forward everything to "V3-level" objects. This is intentional. It is possible to create a new encryptor or decryptor that writes a different format and plug it into the RNCryptor wrapper. That's currently private but could potentially be made more public. (That said, I'm not certain how it would help very much unless you're trying to read both RNCryptor and other formats; in that case, it's built exactly to solve that problem.)
Note that RNCryptor is widely ported and is generally not that hard to implement. Is there a particular platform that would be helpful to have an implementation?
Can you explain the actual problem you're facing and what other format you're trying to write? I'm having trouble picturing what kind of format a subclass would be useful for. Perhaps you just want to use Engine. I intentionally pulled that out to make it easier to write more general-purpose formats.


Generic functions in std.math cannot be overloaded

rumbu reported this on 2018-01-16T19:39:31Z
Transfered from https://issues.dlang.org/show_bug.cgi?id=18244
CC List

Ajieskola
bugzilla (@WalterBright)
hsteoh

Description
There are functions in std.math accepting any parameter type instead of being specialized on numeric types.

This blocks overloading of such functions for other custom numeric types as long as the end user imports std.math.

List of generic functions:

approxEqual
isFinite
isNormal
isSubnormal
NaN
nextafter
signbit
sgn

Special case: "abs" assumes wrongly that the comparison operator is pure @safe nothrow @nogc for any imaginable type which supports T.init < 0;

hsteoh commented on 2018-01-16T22:15:17Z
W.r.t NaN, I agree it would be nice to be able to specify the exact type desired. I don't think the payload has to be a template parameter; it should just be ulong since all unsigned integral types will convert to ulong, and we can just take however many bits to fit into the payload of the requested type.

Again, I think this belongs in a separate issue so that it doesn't get lost amid the overloading issues.

hsteoh commented on 2018-01-16T21:31:04Z
https://github.com/dlang/phobos/pull/6040

Note that currently I'm not touching abs() because the sig constraint is just too complex.

hsteoh commented on 2018-01-16T22:12:07Z
Huh. I'm surprised there's such a thing as fabs in std.math. :-P

I agree that we should just unify everything under abs(). This is not C, where we can't overload abs() for integer vs. float arguments.  Of course, fabs can still be left as an alias to abs() for backward compatibility.

We should probably file a separate bug to track this issue.

github-bugzilla commented on 2018-02-24T11:36:41Z
Commits pushed to master at https://github.com/dlang/phobos

https://github.com/dlang/phobos/commit/a74271be4c93cf95905265597ac9bca426378f55
Revert "Issue 18244: std.math generic functions need to have sig constraints"

https://github.com/dlang/phobos/commit/230292aef0353efa25ba5e298c30cdf5ce6717b5
Merge pull request #6213 from dlang/revert-6040-signbit_sig

Revert "Issue 18244: std.math generic functions need to have sig constraints"
merged-on-behalf-of: Iain Buclaw<EMAIL_ADDRESS>


Clean up php 8.0.

Q
A

Is bugfix?
✔

New feature?
✔

Breaks BC?
❌

[x] Add union types.
[x] Fix phpdoc.
[x] Add use function.
[x] Upload code coverage 100%.
[x] Upload infection 99,99%.
[x] Clean up code.
[x] Add more tests.

👍


docs(readme): fix maintainer guidelines link

Description:
link in readme is broken
Related issue (if exists):

nah thank u for rxjs


fixed bug in PSmask

A bug in PSmask option was fixed:
Previously, extremely large energy or NaN showed up when KY pseudopotential of H atom was used (division by zero or very small number was done depending on pseudopotential data). This was fixed.

Thank you @ayamada224 san, I will check it later.


Add module-info

Adding a module-info would help others projects that are also modularizing.
I would like to know if I should update the other pom files too.
Thanks!

Hi @Thihup,
Thank you for the pull request.
We use pom-central.xml and pom-central17.xml for jar files, not pom.xml.


Replace bootstrap theme with something without shadows

The css shadows are not included in the element dimension and causes issues inside juicy-tile-table with default styles:

Possible replacements:
Cosmo
https://bootswatch.com/cosmo/

Yeti
https://bootswatch.com/yeti/

More themes are available there: https://bootswatch.com/.
cc @warpech, @andwah, @tomalec

Yeti theme has a glow on input focus. Doesn't it cause the scrollbars to appear?

Yes, it has outline for inputs, but this does not cause scroll bars and without the outline it still looks good. So, it can be removed with custom style overwrite.

The Cosmo theme also has those shadows.

Yes, it has outline for inputs, but this does not cause scroll bars and without the outline it still looks good.

How come it does not cause scrollbars? Doesn't box-shadow always trigger scrollbars?

It is not box-shadow, it is outline.

In Yeti, it is definitely box-shadow:

But maybe I am puzzled about this, and actually it is NOT box-shadow that causes scrollbars?

Looks like we mixed up two issues.

The box-shadow property is not included in the box dimension, and being cut of by Shadow DOM container of juicy-tile.
The tiles with display: inline; causes scroll bar.
The margin-bottom property also causes scroll bar, often with 1px in size.

As https://github.com/Juicy/juicy-tile-table/issues/4 is fixed, and juicy-tile-table supports shadows for tiles gently. Do we still need this one?

@tomalec thank you for the fix, I don't think we need to change the theme right now. Closing the issue.


Contract test: Handle responses with no body

Discussed in https://github.com/apideck-libraries/portman/discussions/239

Originally posted by savage-alex January 14, 2022
204 is a popular status code for PUT, POST and DELETE operations and MUST not have a response body says the RFC7231
In portman contract tests it correctly omits any body or schema checks if there is no content in the and API definition for the 2xx response. What it currently does not to is add a check for response body to be empty style test such as
pm.test("response should be empty", function () {
pm.response.to.not.be.withBody;
});
Is this something that would be useful to add to ensure we are not returning bodies when the contact is saying we shall not?

@savage-alex It works very well, Thank you @thim81


Verdaccio Web login plugin

Setup the plugin to allow login via web without Github token
see; https://github.com/n4bb12/verdaccio-github-oauth-ui

Moved to new repo


Correct handling of TM sign

Copied from 59

[reply to @Shreeshrii]
@theraysmith commented
TM is also difficult, as it is in conflict with the needs of fi/fl, which should not appear in the output.

Copied from:  issue https://github.com/tesseract-ocr/tesseract/issues/761
https://groups.google.com/forum/?utm_medium=email&utm_source=footer#!msg/tesseract-ocr/JvEF7f0KU8I/La50m7SzEgAJ
the trademark symbol is still not recognized properly. With the newly generated traineddata the symbol is recognized as TM
Looking at Latin.unicharset I see that the normalized form of these is just the regular numbers or TM.
@theraysmith Does this need to be changed?
™ 0 63,201,209,255,101,273,0,59,104,293 Common 1496 10 1496 TM	# ™ [2122 ]
² 0 3,192,209,255,50,248,0,105,0,293 Common 1090 2 1090 2	# ² [b2 ]
³ 0 0,192,209,255,48,268,0,99,0,293 Common 1091 2 1091 3	# ³ [b3 ]

After thinkiing about this carefully, I decided to undo a change I had made
for the LSTM engine, and better solve tatweel.
The fi/fl ligatures will no longer be included in unicharsets, but will
still be included in the training text, by replacing them with fi/fl pairs
at the same time that tatweel is deleted.
This allows the output to be un-normalized, shaped quotes to be brought
back, and the TM symbol recognized as a single character.
It doesn't help with the sub/superscript problem, and I have another idea
that I want to try that is more important first...
On Fri, Mar 31, 2017 at 2:10 AM, Shreeshrii<EMAIL_ADDRESS>wrote:

Copied from: issue tesseract-ocr/tesseract#761
https://github.com/tesseract-ocr/tesseract/issues/761
https://groups.google.com/forum/?utm_medium=email&utm_
source=footer#!msg/tesseract-ocr/JvEF7f0KU8I/La50m7SzEgAJ
the trademark symbol is still not recognized properly. With the newly
generated traineddata the symbol is recognized as TM
Looking at Latin.unicharset I see that the normalized form of these is
just the regular numbers or TM.
@theraysmith https://github.com/theraysmith Does this need to be
changed?
™ 0 63,201,209,255,101,273,0,59,104,293 Common 1496 10 1496 TM # ™ [2122 ]
² 0 3,192,209,255,50,248,0,105,0,293 Common 1090 2 1090 2 # ² [b2 ]
³ 0 0,192,209,255,48,268,0,99,0,293 Common 1091 2 1091 3 # ³ [b3 ]
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/tesseract-ocr/langdata/issues/63#issuecomment-290660919,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AL056S7lNKS0yOgG9kjZBsmxKT7ziM9Dks5rrMMRgaJpZM4MvZ7k
.

--
Ray.

https://github.com/tesseract-ocr/langdata/issues/59#issuecomment-294256255
theraysmith commented

Thanks for opening the new issues 62, 63. I will continue to think about the best approach. I tried to include TM in the current round of training, but it is too infrequent to have made the cut line. I will have to add it to the desired_characters list.

https://github.com/tesseract-ocr/tesseract/commit/b0ead95d64a366

@theraysmith
tesseract-ocr/tesseract@b0ead95 does not seem to solve this.
Does it also require your newer language models?
I did replace a layer training with fonts FreeSerif and FreeSans till 0.01% error rate. However, it seems to still recognize TM trademark sign as letters TM and not the sign, while testing with same tif which was used for training.
zip file with training text, synthetic training images, generated traineddata and OCR output with --oem1 is attached.
eng.englayer.zip

I notice that the unicharset still has TM as normalized version instead of sign.
Does latin.unicharset need updating?
™ 0 63,201,209,255,101,273,0,59,104,293 Common 112 10 112 TM	# ™ [2122 ]
· 10 64,148,129,255,13,238,5,125,39,293 Common 113 10 113 ·	# · [b7 ]p
℠ 0 130,152,235,249,167,228,3,30,192,234 Common 114 10 114 SM	# ℠ [2120 ]
℗ 0 13,65,229,255,165,244,0,30,169,273 Common 115 10 115 ℗	# ℗ [2117 ]

No there are still one or two commits to go before that will work. I might
get them in today.
On Tue, Jul 25, 2017 at 5:14 AM, Shreeshrii<EMAIL_ADDRESS>wrote:

I notice that the unicharset still has TM as normalized version instead of
sign.
Does latin.unicharset need updating?
™ 0 63,201,209,255,101,273,0,59,104,293 Common 112 10 112 TM	# ™ [2122 ]
· 10 64,148,129,255,13,238,5,125,39,293 Common 113 10 113 ·	# · [b7 ]p
℠ 0 130,152,235,249,167,228,3,30,192,234 Common 114 10 114 SM	# ℠ [2120 ]
℗ 0 13,65,229,255,165,244,0,30,169,273 Common 115 10 115 ℗	# ℗ [2117 ]
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/tesseract-ocr/langdata/issues/63#issuecomment-317718938,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AL056XZjuyi-GE1ZoAK1_71EoZThUYcRks5sRdwYgaJpZM4MvZ7k
.

--
Ray.

Right try it now.
You need commits b0ead95d..0e95e2ca and 1a0f501..3e32be3 (in langdata)
I think they are everything you need.
The new English model will contain TM.
On Tue, Jul 25, 2017 at 8:29 AM, Ray Smith<EMAIL_ADDRESS>wrote:

No there are still one or two commits to go before that will work. I might
get them in today.
On Tue, Jul 25, 2017 at 5:14 AM, Shreeshrii<EMAIL_ADDRESS>wrote:

I notice that the unicharset still has TM as normalized version instead
of sign.
Does latin.unicharset need updating?
™ 0 63,201,209,255,101,273,0,59,104,293 Common 112 10 112 TM	# ™ [2122 ]
· 10 64,148,129,255,13,238,5,125,39,293 Common 113 10 113 ·	# · [b7 ]p
℠ 0 130,152,235,249,167,228,3,30,192,234 Common 114 10 114 SM	# ℠ [2120 ]
℗ 0 13,65,229,255,165,244,0,30,169,273 Common 115 10 115 ℗	# ℗ [2117 ]
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/tesseract-ocr/langdata/issues/63#issuecomment-317718938,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AL056XZjuyi-GE1ZoAK1_71EoZThUYcRks5sRdwYgaJpZM4MvZ7k
.

--
Ray.

--
Ray.

Ray,
I updated langdata and tesseract and built tesseract again.
With the new traineddata, TM is not being recognized at all - it is getting dropped.
with eng.traineddata
The trademark symbol (*), in Unicode U+2122 *~ trade mark sign (HTML ™ — ™),
\texttrademark in LaTeX,[1] [2] is a symbol used to indicate an assertion that the preceding mark

is a trademark. Registered trademarks are indicated using the registered trademark symbol (®),

with new englayer.traineddata
The trademark symbol (), in Unicode U+2122 " trade mark sign (HTML ™ · ™),
\texttrademark in LaTeX,[1] [2] is a symbol used to indicate an assertion that the preceding mark

is a trademark. Registered trademarks are indicated using the registered trademark symbol (®),

I used the old .lstmf files to do training - would that be a problem?

@theraysmith
I trained again after creating new box/tiff and lstmf files using the new code and new langdata.
TM sign is now being recognized correctly.
It is also NOT treating fl and fi as glyphs but as separate letters in words such as film, first, flounder, reflect etc.
Thanks!

Great!
That is the objective with fi and fl ligatures. They now have similar
status as tatweel: used for rendering, but not for output, except of course
that fi and fl produce output characters, but tatweel disappears completely.
On Thu, Jul 27, 2017 at 8:31 PM, Shreeshrii<EMAIL_ADDRESS>wrote:

@theraysmith https://github.com/theraysmith
I trained again after creating new box/tiff and lstmf files using the new
code and new langdata.
TM sign is now being recognized correctly.
It is also NOT treating fl and fi as glyphs but as separate letters
in words such as film, first, flounder, reflect etc.
Thanks!
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/tesseract-ocr/langdata/issues/63#issuecomment-318548187,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AL056c1Ib60ZnotAwdlQhSlJ2uw0gtnbks5sSVYcgaJpZM4MvZ7k
.

--
Ray.


Animated and Variable-Height TopBar

I'm trying to do this effect[http://callmenick.com/_development/resize-header-on-scroll/], using cascated Sticky components, with
 objects, but without success:

        <StickyContainer>
          <Sticky>
            <TopFlag />
            <StickyContainer>

1

              <Sticky>

1

                <StickyContainer>
                  <Sticky>
                    <BreadCrumbs />
                  </Sticky>
                </StickyContainer>
              </Sticky>
            </StickyContainer>
          </Sticky>

            {this.props.children}

        </StickyContainer>

Result:

The idea was to make
 be overlaped by next elements, and the be hidden, making total header height lower.
Any ideas?

Hey @femesq, I'm not sure that I'm understanding. I think you have a sticky top section, which has sticky sections within it?
I see that this issue is quite old, so I'm going to close it out proactively. I'd be happy to reopen it if you're still encountering this issue, just let me know what version you're seeing it on.


Add function preferredColormaps to Colormap module

This PR adds a common place where to store a list of preferred colormaps to propose to the user.
It also update the ColormapDialog to use it. @payno let me know if you prefer that I remove this commit from the PR.
Close #1411

is preferredColormaps could be static ?

Fix issue with Qt4 missing a QComboBox method.

LGTM


enhancement:enhance random stream.

Let events be evenly distributed over one second.You can specify the time interval.For example CREATE RANDOM STREAM rand_stream(i int default rand()%5) SETTINGS eps=1000, interval_time=100; means generating equal amount of data for each interval(eps / (1000 / interval_time)).
fixed the bug when creating random stream with unknown settings the proton-server will core-dump
add random stream smoke test

(Jove Github Bot) assuming it is not done, deferred this ticket to the next sprint.

closed by #103


Add hamlet burger

About the PR

Recipe:

Put Hamlet, bun and slice of chesse in microwave for 5 sec.
Send that burger to the captain.
Feel yourself like a villain.

Media

[x] I have added screenshots/videos to this PR showcasing its changes ingame, or this PR does not require an ingame showcase

Changelog

:cl: lzk

add: Added Hamlet burger

now this is the content im looking for

need ianburger too :trollface:

emo based

is this actually creatable ingame? i would figure hamlet would explode in the microwave before being made into a burger

theres a similar issue eith mice and rat kebab, might need to put cooking at higher priority than gibbing animals

im a really big fan of someone trying to make a burger out of hamlet and he just fucking explodes

yeah i tested every way i could think of, its impossible to make hampter burger he just explodes every time
also shouldve named it hamburger 100%

also nutrition wise its the same as just the bun, so hampter is made of antimatter cheese (and making hampter burger is not kitchen meta!!!)

theres a similar issue with mice and rat kebab, might need to put cooking at higher priority than gibbing animals

you have to butcher mice to get rat meat for rat kebab, you don't actually put the whole mouse into the microwave

Ahh...

you don't realistically microwave a burger to cook it unless you want to give someone food poisoning

ok now remake the entire cooking system to be realistic :trollface:

you oculd get around this by creating hamlet meat, which spawns on butcher of hamlet. Then having burger use hamlet meat.


Use new vmmv.v pseudoinstruction consistently

Closes #620

@kasanovic this is noncontroversial.

@kasanovic this is noncontroversial.


sidebar-menu

Notes for Reviewers
This PR fixes #11460
https://github.com/user-attachments/assets/eb31b068-a43f-4c55-8081-49c05fc6030b
Signed commits

[x] Yes, I signed my commits.

🚀 Preview for commit d72b72c491946acc2770ce04f44dab46b2bbb3ab at: https://66a528a74a43fc50cd0e70bd--meshery-docs-preview.netlify.app

Thanks, @vaishnaviadhau!

@hargunkaur286, you might like to review...

@vaishnaviadhau
Let's discuss this on the website's call today at 5:30 PM IST. Adding this as an agenda item to the meeting minutes.

@vishalvivekm  sir I didnt finf the link pls send me the meeting link


model training got stuck when running the official tutorial example

What happened + What you expected to happen
Hi,
I'm new to nixtla. When I was trying to run the example code in official tutorial on my local machine(Linux, CentOS): https://nixtlaverse.nixtla.io/neuralforecast/examples/getting_started_complete.html, I found it got stuck at nf.fit(df=Y_df) step:
2024-03-21 17:00:29,350	INFO worker.py:1724 -- Started a local Ray instance.
2024-03-21 17:00:29,926	INFO tune.py:220 -- Initializing Ray automatically. For cluster usage or custom Ray initialization, call `ray.init(...)` before `Tuner(...)`.
2024-03-21 17:00:29,927	INFO tune.py:592 -- [output] This will use the new output engine with verbosity 0. To disable the new output and use the legacy output engine, set the environment variable RAY_AIR_NEW_OUTPUT=0. For more information, please see https://github.com/ray-project/ray/issues/36949
╭────────────────────────────────────────────────────────────────────╮
│ Configuration for experiment     _train_tune_2024-03-21_17-00-27   │
├────────────────────────────────────────────────────────────────────┤
│ Search algorithm                 BasicVariantGenerator             │
│ Scheduler                        FIFOScheduler                     │
│ Number of trials                 5                                 │
╰────────────────────────────────────────────────────────────────────╯

View detailed results here: /root/ray_results/_train_tune_2024-03-21_17-00-27
To visualize your results with TensorBoard, run: `tensorboard --logdir /root/ray_results/_train_tune_2024-03-21_17-00-27`
(_train_tune pid=2885517) Seed set to 11
(_train_tune pid=2885517) [rank: 0] Seed set to 11
(_train_tune pid=2885517) Initializing distributed: GLOBAL_RANK: 0, MEMBER: 1/2

The processe I did to set up:

use conda to create a new environment with Python 3.9
run pip install statsforecast s3fs datasetsforecast in the tutorial example.
run  pip install git+https://github.com/Nixtla/neuralforecast.git@main in the tutorial example.
run pip install matplotlib in order to get the 3rd step of the tutorial work.
I change the code in:  nf = NeuralForecast(  models=[      AutoNHITS(h=48, config=config_nhits, loss=MQLoss(), num_samples=5),      AutoLSTM(h=48, config=config_lstm, loss=MQLoss(), num_samples=2),  ],  freq='H' ) with freq='H' to freq=1 since ValueError: Time column contains integers but the specified frequency is not an integer. Please provide a valid integer, e.g. 'freq=1'

I was wondering what could possibly go wrong in the upper steps and why it got stuck at the training process.
Then, I tried the tutorial notebook in Colab. The fit process can be done, though there is an error when evaluation evaluation_df = accuracy(cv_df, [mse, mae, rmse], agg_by=['unique_id']):
ValueError                                Traceback (most recent call last)
[/usr/local/lib/python3.10/dist-packages/triad/collections/schema.py](https://localhost:8080/#) in append(self, obj)
    359             elif isinstance(obj, pd.DataFrame):
--> 360                 self._append_pa_schema(PD_UTILS.to_schema(obj))
    361             elif isinstance(obj, Tuple):  # type: ignore

11 frames
ValueError: pandas like datafame index can't have name

During handling of the above exception, another exception occurred:

SchemaError                               Traceback (most recent call last)
[/usr/local/lib/python3.10/dist-packages/triad/collections/schema.py](https://localhost:8080/#) in append(self, obj)
    370             raise
    371         except Exception as e:
--> 372             raise SchemaError(str(e))
    373
    374     def remove(  # noqa: C901

SchemaError: pandas like datafame index can't have name

Looking forward to your reply.
Versions / Dependencies
OS: Linux CentOS
neuralforecast            1.6.4
python                    3.9.18
ray                       2.9.3
torch                     2.2.1
transformers              4.39.0
pandas                    2.2.1
Reproduction script
Official tutorial example: https://nixtlaverse.nixtla.io/neuralforecast/examples/getting_started_complete.html
Issue Severity
High: It blocks me from completing my task.

Hey @hxuaj, sorry for the troubles.
The first error should be fixed by setting the CUDA_VISIBLE_DEVICES env variable to one of your devices (0 or 1), either through the terminal or in your session with os.environ.
The second error I'm guessing refers to the fact that the dataframe has an index, but we're deprecating the datasetsforecast losses, so that should do something like this instead:
from utilsforecast.evaluation import evaluate
from utilsforecast.losses import mse, mae, rmse

evaluation_df = evaluate(cv_df, [mse, mae, rmse])

Hey @hxuaj, sorry for the troubles.
The first error should be fixed by setting the CUDA_VISIBLE_DEVICES env variable to one of your devices (0 or 1), either through the terminal or in your session with os.environ.
The second error I'm guessing refers to the fact that the dataframe has an index, but we're deprecating the datasetsforecast losses, so that should do something like this instead:
from utilsforecast.evaluation import evaluate
from utilsforecast.losses import mse, mae, rmse

evaluation_df = evaluate(cv_df, [mse, mae, rmse])

Hi,
Thx for the quick reply.
For the first error, my local machine has 2 gpus, seems like a bug with Pytorch lightning: https://github.com/Lightning-AI/pytorch-lightning/issues/4612. However I didn't find a proper solution to this. Just as you suggested, now I can run model fit with only one gpu visible as a workaround.
For the second error, I changed the code to:
from utilsforecast.evaluation import evaluate
from utilsforecast.losses import mse, mae, rmse

cv_df.reset_index(inplace=True)
evaluation_df = evaluate(cv_df, [mse, mae, rmse])

Just add index to df before evaluation. Now it works fine.
Could you update the relevant parts in this official tutorial, since it might be frustrated to encounter such error in the exampls. Thank you again.

Just add index to df before evaluation. Now it works fine.

Just ran into the same issue, your workaround fixed it, thanks @hxuaj
@jmoralez , BTW, the error has a typo - datafame vs dataf_r_ame
(and may as well fix the grammer too: pandas-like dataframe index can't have name)

That's not coming from our libs, feel free to open an issue in the corresponding lib.


Objet in configuration gets reference to Shared location, like: "/CDP_300/Shared/policy_internal" they should not.

Environment
Application Services Version: 3.31.0
BIG-IP Version: 16.1.0
Summary
Some objet in configuration gets reference to Shared location - despite they are not in initial config.
Issue open per comment:
https://github.com/f5devcentral/f5-as3-config-converter/issues/59#issuecomment-952269676
Steps To Reproduce
Steps to reproduce the behavior:

Convert following config:

ltm node /CDP_300_1/application_3/n_nide1_v4 {
    address <IP_ADDRESS>
}
ltm virtual /CDP_300_1/application_3/vs_test_internal_v4_443 {
    destination /CDP_300_1/application_3/<IP_ADDRESS>%300:443
    ip-protocol tcp
    mask <IP_ADDRESS>
    persist {
        /Common/cookie {
            default yes
        }
    }
    policies {
        /CDP_300/policy_internal { }
    }
    pool /CDP_300_1/application_3/p_pool_v4_443
    profiles {
        /CDP_300_1/application_3/clientssl_test.com {
            context clientside
        }
        /CDP_300_1/application_3/http_f5std { }
        /Common/serverssl-insecure-compatible {
            context serverside
        }
        /Common/tcp-lan-optimized { }
    }
    serverssl-use-sni disabled
    source <IP_ADDRESS>/0
    source-address-translation {
        pool /CDP_300_1/application_3/spool_pool_com
        type snat
    }
    translate-address enabled
    translate-port enabled
    vlans {
        /Common/cdp_300_internal
    }
    vlans-enabled
}
ltm pool /CDP_300_1/application_3/p_pool_v4_443 {
    members {
        /CDP_300_1/application_3/n_nide2_v4:443 {
            address <IP_ADDRESS>
        }
        /CDP_300_1/application_3/n_nide1_v4:443 {
            address <IP_ADDRESS>
        }
    }
    monitor /CDP_300_1/application_3/m_monitor_HTTPS
}

ltm profile client-ssl /CDP_300_1/application_3/clientssl_test.com {
    app-service none
    cert /Common/default.crt
    cert-key-chain {
        Intermediate_0 {
            cert /Common/default.crt
            key /Common/default.key
        }
    }
    defaults-from /Common/clientssl
    inherit-ca-certkeychain true
    inherit-certkeychain false
    key /Common/default.key
    passphrase none
}
ltm node /CDP_300_1/application_3/n_nide2_v4 {
    address <IP_ADDRESS>
}
ltm profile http /CDP_300_1/application_3/http_f5std {
    app-service none
    defaults-from /Common/http
    insert-xforwarded-for enabled
    proxy-type reverse
}
ltm profile server-ssl /Common/https_test_ssl_profile {
    app-service none
    cert none
    cipher-group none
    ciphers DEFAULT
    defaults-from /Common/serverssl
    key none
    options { dont-insert-empty-fragments no-dtlsv1.2 }
}
ltm snat-translation /CDP_300_1/application_3/<IP_ADDRESS> {
    address <IP_ADDRESS>
    inherited-traffic-group true
    traffic-group /Common/traffic-group-1
}
ltm snatpool /CDP_300_1/application_3/spool_pool_com {
    members {
        /CDP_300_1/application_3/<IP_ADDRESS>
    }
}
ltm virtual-address /CDP_300_1/application_3/<IP_ADDRESS>%300 {
    address <IP_ADDRESS>
    arp enabled
    icmp-echo enabled
    mask <IP_ADDRESS>
    traffic-group /Common/traffic-group-1
}
ltm monitor https /CDP_300_1/application_3/m_monitor_HTTPS {
    adaptive disabled
    cipherlist DEFAULT
    compatibility enabled
    defaults-from /Common/https
    destination *:*
    interval 25
    ip-dscp 0
    recv "200 OK"
    recv-disable none
    send "GET /Harmony/ HTTP/1.1\r\nHost: test.local.com\r\nUser-Agent: Big-IP Monitor\r\nConnection: Close\r\n"
    ssl-profile /Common/https_test_ssl_profile
    time-until-up 0
    timeout 76
}

Observe the following error message:

 brzosko@WRW-ML-00011994  ~/Downloads/AS3/test_manual  curl -k -u admin:admin -X POST -H "Content-Type: application/json" -H "Expect:" -d<EMAIL_ADDRESS>https://<IP_ADDRESS>/mgmt/shared/appsvcs/declare | jq
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
100  2153  100   193  100  1960    140   1429  0:00:01  0:00:01 --:--:--  1569
{
  "code": 422,
  "errors": [
    "/CDP_300_1/application_3/vs_test_internal_v4_443/policyEndpoint: contains path to non-existent object CDP_300"
  ],
  "declarationFullId": "",
  "message": "declaration is invalid"
}

Expected Behavior
objects reference should state as in initial config.
Actual Behavior
conversion output:
{
"class": "ADC",
"schemaVersion": "3.30.0",
"id": "urn:uuid:1874dcae-276d-4a18-a684-d6d2a264a552",
"label": "Converted Declaration",
"remark": "Auto-generated by AS3 Config Converter",
"CDP_300_1": {
"class": "Tenant",
"application_3": {
"class": "Application",
"template": "generic",
"vs_test_internal_v4_443": {
"layer4": "tcp",
"persistenceMethods": [
"cookie"
],
"pool": "p_pool_v4_443",
"translateServerAddress": true,
"translateServerPort": true,
"class": "Service_HTTPS",
"serverTLS": "clientssl_test.com",
"profileHTTP": {
"use": "/CDP_300_1/application_3/http_f5std"
},
"clientTLS": {
"bigip": "/Common/serverssl-insecure-compatible"
},
"redirect80": false,
"virtualAddresses": [
"<IP_ADDRESS>%300"
],
"snat": {
"use": "/CDP_300_1/application_3/spool_pool_com"
},
"allowVlans": [
{
"bigip": "/Common/cdp_300_internal"
}
],
"policyEndpoint": "/CDP_300/Shared/policy_internal" <---------------------------!!!!!!! why Shared is added??!!!!!
},
"p_pool_v4_443": {
"members": [
{
"addressDiscovery": "static",
"servicePort": 443,
"serverAddresses": [
"<IP_ADDRESS>",
"<IP_ADDRESS>"
],
"shareNodes": true
}
],
"monitors": [
{
"use": "/CDP_300_1/application_3/m_monitor_HTTPS"
}
],
"class": "Pool"
},
"certificate_default": {
"class": "Certificate",
"certificate": {
"bigip": "/Common/default.crt"
},
"privateKey": {
"bigip": "/Common/default.key"
}
},
"clientssl_test.com": {
"certificates": [
{
"certificate": "certificate_default"
}
],
"class": "TLS_Server",
"insertEmptyFragmentsEnabled": true
},
"http_f5std": {
"class": "HTTP_Profile"
},
"spool_pool_com": {
"snatAddresses": [
"<IP_ADDRESS>"
],
"class": "SNAT_Pool"
},
"m_monitor_HTTPS": {
"adaptive": false,
"ciphers": "DEFAULT",
"interval": 25,
"dscp": 0,
"receive": "200 OK",
"send": "GET /Harmony/ HTTP/1.1\r\nHost: test.local.com\r\nUser-Agent: Big-IP Monitor\r\nConnection: Close\r\n",
"clientTLS": {
"use": "/Common/Shared/https_test_ssl_profile"
},
"timeout": 76,
"class": "Monitor",
"monitorType": "https",
"targetPort": 0,
"transparent": false,
"reverse": false
}
}
},
"Common": {
"class": "Tenant",
"Shared": {
"class": "Application",
"template": "shared",
"https_test_ssl_profile": {
"ciphers": "DEFAULT",
"class": "TLS_Client"
}
}
}
}

Cannot reproduce. Do not see policyEndpoint with reference to /Common/Shared
https://github.com/mdditt2000/f5-appsvcs-acc/tree/master/Github/71
Issue can be closed

@lukaszbrzosko please validate that you using ACC 1.16

Will do indeed in my repo it is 1.15 let me confirm if this is good on 1.16.

You are correct there is no more such a statement ("policyEndpoint": "/CDP_300/Shared/policy_internal") in latest release.
In fact there is no entry for policy at all, will it not produce problem ? why this policy statement is gone ?

It looks like ACC isn't aware of what policy is. It seems to be a internal traffic policy created in tenant /CDP_300
policies {
/CDP_300/policy_internal { }
}
Can you provide me more details on the policies policy_internal. Most likely its a default policy.

This policy is defined in bigip.conf file of partition CDP_300 itself:
ltm policy /CDP_300/policy_internal {
controls { forwarding }
requires { http }
rules {
"Test Redirect" {
actions {
0 {
http-reply
redirect
location /Test/
}
}
conditions {
0 {
http-uri
path
values { / }
}
}
ordinal 1
}
}
strategy /Common/first-match
}
It is indeed not defined in the input file for charon, there is just reference to it.
So this is expected that it will skip reference to this policy once it will not find definition for that ?

@lukaszbrzosko please send me the entire bigip.conf. You can email me at<EMAIL_ADDRESS>

@lukaszbrzosko ACC does convert the above policy is specified in the bigip.conf
{
    "$schema": "https://raw.githubusercontent.com/F5Networks/f5-appsvcs-extension/master/schema/latest/as3-schema.json",
    "class": "AS3",
    "declaration": {
        "class": "ADC",
        "schemaVersion": "3.34.0",
        "id": "urn:uuid:1c3872df-9cec-41bd-8346-b9ffb7afd37a",
        "label": "Converted Declaration",
        "remark": "Generated by Automation Config Converter",
        "CDP_300": {
            "class": "Tenant",
            "Shared": {
                "class": "Application",
                "template": "shared",
                "policy_internal": {
                    "rules": [
                        {
                            "name": "Test Redirect",
                            "conditions": [
                                {
                                    "event": "request",
                                    "type": "httpUri",
                                    "path": {
                                        "values": [
                                            "/"
                                        ],
                                        "operand": "equals"
                                    }
                                }
                            ],
                            "actions": [
                                {
                                    "event": "request",
                                    "type": "httpRedirect",
                                    "location": "/Test/"
                                }
                            ]
                        }
                    ],
                    "strategy": "first-match",
                    "class": "Endpoint_Policy"
                }
            }
        }
    }
}

The bigip.conf file i received has no reference to the above policy. If the policy is missing and only imput is
    policies {
        /CDP_300/policy_internal { }
    }

ACC will simply just ignore. Validated using ACC 1.19. Closing this issue


ARROW-6503: [C++] Add an argument of memory pool object to SparseTensorConverter

In this pull-request, I'd like to add an argument of MemoryPool to SparseTensorConverter, and replace SparseTensorImpl's constructors from Tensor with static factory functions.

I guess this failure on Travis CI isn't caused by this change.

@mrkn indeed. It's fixed on git master, you can rebase if you want.

@pitrou Thank you for telling me it.  I've rebased on to the master.

Codecov Report

Merging #5707 into master will increase coverage by 0.55%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master    #5707      +/-   ##
==========================================
+ Coverage   88.97%   89.53%   +0.55%
==========================================
  Files         990      798     -192
  Lines      134876   119376   -15500
  Branches     1501        0    -1501
==========================================
- Hits       120010   106878   -13132
+ Misses      14501    12498    -2003
+ Partials      365        0     -365

Impacted Files
Coverage Δ

cpp/src/arrow/sparse_tensor.h
100% <100%> (ø)
:arrow_up:

cpp/src/arrow/sparse_tensor_test.cc
100% <100%> (ø)
:arrow_up:

cpp/src/arrow/python/numpy_convert.cc
95.02% <100%> (-0.05%)
:arrow_down:

cpp/src/arrow/ipc/read_write_test.cc
99.87% <100%> (ø)
:arrow_up:

cpp/src/arrow/sparse_tensor.cc
98.55% <100%> (-0.03%)
:arrow_down:

cpp/src/arrow/json/converter.cc
90.05% <0%> (-1.76%)
:arrow_down:

cpp/src/arrow/json/chunked_builder.cc
80% <0%> (-1.67%)
:arrow_down:

cpp/src/arrow/util/thread_pool_test.cc
97.66% <0%> (-0.94%)
:arrow_down:

go/arrow/ipc/writer.go

go/arrow/math/uint64_amd64.go

... and 190 more

Continue to review full report at Codecov.

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


Android setup

Prerequisites

[X] I checked the documentation and FAQ without finding a solution
[X] I checked to make sure that this issue has not already been filed
[X] I'm sure that question is related to the library itself and not Bluetooth Low Energy or Classic in general. If that so, please post your question on StackOverflow.
[X] I'm running the latest version

Question
Hi, I have read the docs and seen a related issue specific to Expo but this is running a raw RN project. Is there a need to manually import and add the BlePlxPackage in Android in MainApplication.java? I installed the package and checked that we have all of the necessary config in our AndroidManifest.xml but when running the app on Android we are seeing the error Cannot read property 'createClient' of null from instantiating BleManager.
I have checked your example project which contains no such manual installation setup, but I'm wondering why the module cannot be found unless we're missing some steps.

I tried with the latest RN template and I don't see any problems - you can check all the changes I made to run the scan: https://github.com/dominik-czupryna-withintent/i1202/commit/883bab543dd3dcbf9f12a250cfa6cd2d5073c01e
Maybe something is cached?

My bad, we have a custom setup and needed to manually register the module. Thanks.


Webpack5 supports ES6

Minor typo - ES2015 is ES6 and webpack 5 now supports ES6/ES2015 code.

I think it was correct. Output can be either ES5 or ES2015 (and newer).

Maybe a better way to express this will be : webpack 5 can generate both ES5 and ES6/ES2015 code now. The default configuration will generate ES2015 code. If you need to support older browser (like IE11), you can decrease this to output.ecmaVersion: 5.`

Also the previous paragraph is misleading. It should state "minimum" version and not maximum version, I think.

It should state "minimum" version and not maximum version, I think.

Depends on from which side you look at it.
It generates code with the specified version or lower. -> maximum
The browser need to support the specified version or higher. -> minimum


Check for number fields.

I noticed this while working on a college project. There's probably other field types that need to be checked for, but I'm a little busy right now so I can't track them down.

Thanks for the PR!  Pushed a 1.0.2 release out with your changes.
For what it's worth I recommend yarn, react, and redux for new applications going forward:
https://github.com/facebook/create-react-app
JQuery applications can become harder to maintain over time.  In 2018 much of the field switched over to React:
https://stateofjs.com/


Providing a custom fallback option?

Is it possible to set a different fallback option, besides Growl?
Growl seems problematic, as it's a separate install for most users as I understand it. I'd like to use this as a fallback personally: https://github.com/blainesch/electron-notifications/
Is there any way to do this without forking?

Currently there is no way of doing this without forking. Though you could do your own implementation/abstraction similar to https://github.com/mikaelbr/node-notifier/blob/master/index.js.
This won't be the same as fallback option as included in the individual notifiers (as seen here https://github.com/mikaelbr/node-notifier/blob/master/notifiers/notificationcenter.js#L80-L92) though. But you could expand your implementation to include with the different checks.
Or you could fork and send a PR if you have the time 👍


Built-in html formatter generates a blank html report if scenarios have not finished yet completely

👓 What did you see?

When running tests using built-in html formatter, it generates a blank HTML file until all scenarios finished completely.
✅ What did you expect to see?

HTML report to be generated should not be blank after a scenario has been finished
📦 Which tool/library version are you using?

cucumber 8.0.0
cucumber-html-formatter 19.2.0
ruby 3.1.1
Windows 10
🔬 How could we reproduce it?

Steps to reproduce the behavior:

Have more than one scenario in a test.
Execute cucumber --format HTML --out report.html
Open the html report when test is still running and html file is blank.

📚 Any additional context?

As I have not found it in the issue list, I'm picking up the issue mentioned in comment https://github.com/cucumber/cucumber-ruby/issues/1524#issuecomment-824267664 as it shows up as unacceptable drawback compared with older versions, especially when having a lot of scenarios or long scenarios or even both of this. Additionally, when issues like https://github.com/cucumber/cucumber-ruby/issues/1524 occur, the whole report is affected and not only a part of it.
written by marnen:

Unfortunately, the way that’s done makes the Cucumber HTML report much less useful than it used to be. Back in the day, Cucumber used to use a similar approach to RSpec: it wrote straight HTML as scenarios ran, so that the file would render something useful in the browser before it was completely written. That makes it ideal for monitoring a long run with lots of scenarios.
Current Cucumber HTML output, though, renders nothing until the run is complete (I think because window.CUCUMBER_MESSAGES isn’t well-formed JSON/JS till the final closing bracket is written), which means I need to wait for the entire run to finish before I see any output. Is there an HTML formatter for current Cucumber that doesn’t have this drawback? I haven’t found one, and I really miss the old RSpec-like behavior. (And if this is off-topic, I’ll raise a new issue or ask on the list.)

answered by mattwynne:

In fact, this kind of live monitoring of a test run from a browser should now be easier to do, given that the results are streamed out as little ndjson packets. The problem would be getting that stream from your Cucumber process to your browser. It's a really cool idea, but yes it is off-topic for this thread I think :) If you're interested in pursuing it, let's create a new issue (in https://github.com/cucumber/cucumber) and discuss it there.

This text was originally generated from a template, then edited by hand. You can modify the template here.

Well, just to avoid misunderstanding and to be competely clear on that, I do not expect process monitoring, but partial output when a part (scenario) is finished. This means I expect behaviour which existed in former Cucumber versions.

Just to check, is this in a case where cucumber is writing to the filesystem and you're opening the file in your web browser?

I don't think the actual html-formatter would be able to show any progress. It aims to generate an all-inclusive static report once the execution is done.
Yes, at some point, it kinda shows some progress, but that was only a side effect of how it was built before.
What you are requesting would require a brand new dedicated formatter which would actually be able to process messages stream as they arrive and thus monitor the execution

I think maybe this can be solved without too much effort. Rather than writing a window.CUCUMBER_MESSAGES array at the end of the run, we could write messages as they come in - wrapped in a JavaScript function call:
<script>processMessage({...})</script>
<script>processMessage({...})</script>
<script>processMessage({...})</script>

Every time another line is added, the UI would update. We'd have to make sure it doesn't impact performance when a finished page is loaded, but I don't think that's hard to do either.

Thanks for raising this issue @seleniumghost - I think fixing this would be an improvement. I might take a crack at it one day, if I can find some time.


Add "not matching regex" conditional

Closes https://github.com/casey/just/issues/2416

The Unspecified token kind isn't really intended to be surfaced to users, it's for constructing an compile error, which normally requires a token, when there is none.
See the test I added in the last commit, which fails with something like:
error: Expected '&&', '!=', '!~', '||', '==', '=~', '+', or '/', but found unspecified
 ——▶ justfile:1:12
  │
1 │ x := if '' !! '' { '' } else { '' }
  │            ^

I think the signature of lex_choices should be changed to:
  fn lex_choices(
    &mut self,
    first: char,
    choices: &[(char, TokenKind)],
    otherwise: Option<TokenKind>,
  ) -> CompileResult<'src> {

And if otherwise is None, it generates an UnexpectedCharacter error.

Nice catch! This is an easy fix, there's an UnexpectedEndOfToken error, for just this eventuality. Fixed in #2520.

Good catch, fixed in #2522!


make postgres connection string optional

How was this tested? (Can we add it to Nessie? Do we want to??)

[Not in this PR, of course...]

Codecov Report

Merging #948 (0be1f2b) into master (0f3084a) will decrease coverage by 0.04%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##           master     #948      +/-   ##
==========================================
- Coverage   44.44%   44.39%   -0.05%
==========================================
  Files         143      143
  Lines       11486    11496      +10
==========================================
- Hits         5105     5104       -1
- Misses       5744     5755      +11
  Partials      637      637

Impacted Files
Coverage Δ

config/config.go
33.33% <ø> (-0.46%)
:arrow_down:

db/migration.go
0.00% <0.00%> (ø)

catalog/cataloger_create_entry.go
94.73% <0.00%> (-5.27%)
:arrow_down:

catalog/cataloger_create_repository.go
62.96% <0.00%> (+3.70%)
:arrow_up:

Continue to review full report at Codecov.

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


Debug picker switches away from compound launch config

Select the "Debug VS Code Main and Renderer" compound launch config
Click debug
See it switch to one of the configs, so I have to fix it next time I want to debug

Same issue. Pretty annoying for debugging...

Definitely annoying. +1 for a fix to this issue.

Duplicate of https://github.com/Microsoft/vscode/issues/32318?

@Gama11 thanks, closing as duplicate. This should be fixed in latest insiders


Ability to set a BaseUrl to a Session

[ ] Explain the use-case carefully.
When using Session object, it makes sense to have the option to use a url to prefix all the requests calls, this avoid redundancy and better handling of concatenation between the url and the endpoint

 with requests.session(base_url='http://api.example.com') as s:
     resp = s.get('/v1/users')

[ ] Who needs this?
This was a recurrent issue on the requests package
[ ] What pain does this resolve?
The Session object is used to persist parameters/cookies/headers between different requests, and I find it makes sense to also persist the base_url, as it will simplify the code and the "process" of appending endpoint to url, by normalizing the handling of '/'
[ ] Is this standard?
I am not sure how to answer this, but for example:

Gruzzle a php HTTP Client works like that
req a Golang HTTP Client

We just implemented this. And will be available in the next minor version.
To have an early access, install Niquests via git. pip install git+https://github.com/jawah/niquests.


Config option "join_spill_memory_threshold" (kJoinSpillMemoryThreshold) not working

Upstreaming
https://github.com/facebookincubator/velox/pull/4711


getIronSession doesn't work with edge-runtime middleware

When I use getIronSession (according to your docs) with the stable version of next.js middleware (12.2), which runs on edge runtime. throws this error:
error -<EMAIL_ADDRESS>(9:0) @ new SubtleCrypto
error - The edge runtime does not support Node.js 'crypto' module.

Is there something I missed?
Your help is greatly appreciated; thanks.

Packages:
"next": "^12.2.5",
"iron-session": "^6.2.0",

Most likely you've imported non-edge version inside your src/core/server/iron-session file.

Thank for you help. Yes, I re-exported the custom versions of withSession Route/Ssr in the index file of that directory.
Might it be better to mention this in the readme?

Hello, i had a same problems with<EMAIL_ADDRESS>How do you resolve this issue ? @Maders
Thanks

Hello, i had a same problems with<EMAIL_ADDRESS>How do you resolve this issue ? @Maders

You must not import withSessionSsr or withSessionRoute or any other thing from 'iron-session' in the middleware file.
In my screenshot, I re-export a custom version of withSession function in the index file of my src/core/server/iron-session so the bundler resolves these functions when importing ironOptions

I'm having the same problem.
Is there a way to fix this?


Typing the letters k,m,b will multiply the number respective to 1k,1m,1b

Problem Statement
Using the key binds k, m, and b as a shortcut to multiplying by 1,000, 1,000,000, and 1,000,000,000 could be useful when dealing with larger numbers.
Evidence or User Insights
This is a feature I've seen appear in user interfaces in games as it correlates with a shortcut people often take in texting. It's fairly common to see 1,000 written as 1k.
Proposal
Adding these shortcuts can improve the speed of keyboard only operation. I've attempted using these key binds through the calculator and haven't seen them being used for anything else at the moment.
Goals
User can quickly multiply by 1,000, 1,000,000, and 1,000,000,000 using the key binds: k, m, and b respectively
Non-Goals
Low-Fidelity Concept

This is your friendly Microsoft Issue Bot. I've seen this issue come in and have gone to tell a human about it.

Interesting idea, @rudes. Thanks for filing the suggestion.
One thing for us to consider with this: similar to the comments from #425 which discussed using c to clear the display, we'd run into an issue with b in Programmer mode where b is a valid value for input.

I saw #425 and tested the key binds myself but didn't notice any change when using them, I must not have gone down the right path to make b available. A page inside of the app that listed all key binds and how they work would be incredibly useful.

A page inside of the app that listed all key binds and how they work would be incredibly useful.

#157 talks about that a bit.  I think there might be another issue as well that talks about there being a separate page that just covers all the keyboard shortcuts/accelerators....I'm having trouble at the moment finding where that was mentioned.

We list our keyboard shortcuts publicly here.  I do think the list is slightly out-of-date.

Yeah I don't see b for example. however unless i'm missing something that link is not readily accessible in the application itself.

What would be really useful would be having a form within the app for a user to configure their own key binds instead of only having a default, hidden, set of key binds.

Yeah I don't see b for example. however unless i'm missing something that link is not readily accessible in the application itself.

In this case, b isn't a shortcut.  It's input in the same was as the numbers 1 - 9 are, as well as the operators.    They're not actually shortcuts.

Ah I see it's used for hexadecimal notation. You could disable the feature when in hex notation though, if you needed to insert multiple 0s you could still do so with k or m. Is the key bind code capable of seeing the state of the app and which mode it's in?

'b' is language specific, for example in Spanish it means: 1 000 000 000 000, not 1 000 000 000 which is called "Millardo". Those shortcuts would have to be i18n. (https://en.wikipedia.org/wiki/Long_and_short_scales)
It would be better to use the ISO standard https://en.wikipedia.org/wiki/Orders_of_magnitude_(numbers)

@rudes This is definitely an interesting idea. We had the chance to review this pitch more closely, and while it is certainly not uncommon to represent thousands/millions/billions with k/M/B, we felt that supporting this notation in input might not be clear to all users. Also, as others point out, there are complexities with localization and conflicts with other key bindings that would make this difficult to implement. For those reasons, we have decided not to move forward with this idea, but thanks again for you suggestion and interest in the project! Keep the ideas coming :)


[5.4] Allow dependency injection on seeders run method.

This makes Seeder classes look a lot more like how Command classes work: instead of using an abstract method with no parameters, implement a method that delegates the call to the DI container.
Should be 99.9% backwards compatible. Added a SeedCommandTest to be sure.
Also, used the __invoke magic method but never really invoked any seeder, so that could be changed as well.

Thanks


Dogfooding Fixes

Tests

[x] Tested on mobile
[x] Tested on dektop

This PR should be manually tested by @PauloSousapt, @psoaresbj, @obernardovieira, included on reviewers.

alright. i'm on it 👌


Add option to ignore whitespaces during comparison

I am at a very early stage of learning Morse code and I mess up my spacing all the time. For a "Your Input" of "M MM" I would for now be equally happy with "M M M" or "MM M". However, the comparison does care about whitespace and gives me the following results:

or

Both results are of course technically correct, and I fully understand where they come from. Still, I find the second one confusing (four 'M's in the result when the compared texts each have only three of them).
So instead of practicing to improve my timing, I implemented an option to "Ignore whitespace during comparison" ...

... to get the following result:

73 de OE6WCB

Thanks for your pull request!
I thought about this feature: yes - when I am keying, I always mess up my spacing as well!
One thing that is not yet handled in your pull request ist when the result is saved, the state of the "ignore whitespace" flag is not respected. So I think, the state of the flag needs to be stored as well and used in the saved result list then:

And I think, the new checkbox should be moved down to the result box.
I added both changes and really like this feature! Thanks!

I'm happy that you liked my idea. I'm even happier with what you made of it. :+1:

thanks for your contribution and making the app better!


Exclude non-pickable arguments from checkpointing

Currently, non-pickable objects passed as (extra) argument to a task function cause a failed checkpointing operation that is not made explicit to the user. There should be a warning and said argument simply excluded from calculating the checksums.

Peter

I came across similar issue just now and have one thought: could ruffus uses dill instead the default pickle facility for pickling? dill works for things that are not normally picklable with cPickle


Modernizing the PowerToys icon

Summary of the new feature/enhancement
The current PowerToys icon feels dated. With the legacy and connection with Windows 95 this makes sense, but I believe modernizing it might be better to make it feel more native to Windows 10. In 2020, we'll see lots of inbox application icons modernized (more on Microsoft Design's push to re-design hundreds of OS icons here).
Another issue with the current icon is the fact that it feels outdated and doesn't scale very well to smaller sizes (for e.g. the systemtray)
Proposed technical implementation details
The proposal is to redesign the current PowerToys icon, so it feels part of the Windows 10 family of experiences (e.g. Terminal, other inbox apps)

I can finish all the other sizes of only two icons from this. Tell me which ones you really like :smile:
Or maybe I'll leave this as some kind of reference(?) for the design team at Microsoft to build upon.

I like your first concept earlier and i like 2 & 4.  My feedback here are I wonder how they'll scale for task bar / systray.  I think they are a tad busy.
For colorblind, i think they are fine, it is when you use colors for alerting for instance and mix things like red and green you need to be aware.

1 and 5, hands dow

One and Only

A couple of quick concepts I just made. Top row using more traditional colours, and bottom row uses more bluish colours inspired by some of the other Windows icons.
I wanted to experiment with breaking the colours out of their box, so I came up with three variants. On the left, the colours are coming out through the 'screen', in the middle they simply sit on top of it, and the right keeps it closer to the original design.
What do you think?

A couple of quick concepts I just made. Top row using more traditional colours, and bottom row uses more bluish colours inspired by some of the other Windows icons.
I wanted to experiment with breaking the colours out of their box, so I came up with three variants. On the left, the colours are coming out through the 'screen', in the middle they simply sit on top of it, and the right keeps it closer to the original design.
What do you think?

It's amazing but i's too little for the taskbar icon ...

Here's another shot at my previous ones. Adding a systemtray version as well (both in color as in monochrome)

2 is verry good but you have a problem:

(the "+" is the light part, the "-" is the dark part)

@crutkas May I ask for exactly what the states are?

Here's some more icons I came up with, in a variety of colours for potential different states as mentioned prior.

Here's how they look at 16px:

And at 32px:

(I have these files saved on Figma if they're needed, btw!)

So I decided to keep finalizing these two concepts of my own. Thoughts?
There are like 9 different icons here. It'd be way harder for @crutkas and his team to pick one from, so that led me wondering how would the collaboration process between all these awesome designers go?

Here's some more icons I came up with, in a variety of colours for potential different states as mentioned prior.

The frame 2 is verry good because the blue is the color of windows, and the icon is verry good in the taskbar, this respect the fluent design

@Elttob Can you try a monochrome icon for the taskbar?

@Elttob Can you try a monochrome icon for the taskbar?

How's this?

All contributions are amazing!
I can't wait to see how many great ideas you all with have for the other icons (the once showed in the settings for each PowerToy module). I guess we will open a new issue for those.
Thanks to all of you again for the fantastic work!

@Elttob Can you try a monochrome icon for the taskbar?

How's this?

edit: combined with some of the colours from above (this would be good for indicating status!)

I like that - I think a monochrome-ish icon for the systray seperate from the actual app icon is required to make it all scale well.

Just something different.

I combined the circuit board icon design and the vertical icon I've made a while ago, thought it'd be cool and nostalgic maybe?

A new idea. Battery + slinky = power + toy?
I'm still figuring out how to make this looks less busy and more recognizable. Also noo idea if this qualifies as a slinky and a battery or not lol

Just something different.

That's a good one!
I've decided to expand on your idea a bit more. I made the wind-up key kinda dives into the pixels. I'm also thinking about the animation like, when the key spins it'll flare some colorful sparks.

My final version of this concept. In the system tray, the icon will now be more in line with the other systray icons. Adding a status indicator (as mentioned by @crutkas ) is now also easier. Also shown below is how the new icon integrates with the new Fluent icons.

We also need an icon for the FancyZones editor, idea? https://github.com/microsoft/PowerToys/issues/1395

Honestly, i'm in love with this @niels9001.  More so when we couple it with color in the systray for status.

Based on @niels9001 's refined design, I did some tweaks  to bring ours closer together,, since that's the look that everyone seems to be going for here 😀
I specifically sought to increase the colour contrast of the icon and make it much easier to distinguish / much less blurry and muddy looking. I hand-tweaked the colours to match the other Fluent icons without being too vibrant, did some further pixel optimisation work and brought the monochrome icon style in line with their design.
Here's what I got!

@Elttob I really like this - maybe we could use the calculator gray for the backdrop so it stands out slightly better! The idea of reducing the colors (especially since the orange color fades away between yellow and red). On the other hand, it breaks a bit the rainbow effect. @crutkas Any thoughts on this related to the legacy of the icon?
I think we're at a point that we can do minor refinements on these concepts. @crutkas Do you think we could have some experts review these (like Christina or Mike that were on the Windows Insiders Podcast yesterday)?

@Elttob I really like the evolutions!
Couple of things:

Maybe we could make the backdrop a bit lighter, similar to the calculator-grey so it stands out a bit more? It's indeed a bit 'muddy' or dark.
I like removing one of the colors, since the orange fades away with the red and yellow! (@crutkas can we do this, in terms of the legacy of the icon?)

I think we're at a point that we can do further minor refinements on these concepts. @crutkas could we get someone from the icon design team (like Christina or Mike that were present in the Windows Insiders Podcast yesterday) to chime in? I think we could use some guidance on e.g. the colors and best practices (also for accessibility reasons).

Here is a rough idea for the icon.  Including a simplified 16px tray icon with status colours.
I placed the icon on various surfaces, to show how it stands out, and adapts to Light and Dark themes.

@niels9001 i'm syncing with the icon team tomorrow :)

Here is a refinement on the logo idea I posted earlier

@niels9001 Here's what it looks like with the grey as you suggested; it's pretty nice!

I also experimented with adding some more drop shadows to help make some of the shapes more distinct. What do you think?

@elttob, i love the brighter grey.  Maybe even match the grey from Calc?
wow, that is awesome.
For the color icons, i think top, left, right grey border would need to be maybe a pixel or two bigger.  To match the calc / Movie & tv distance border

Iterated a bit further - using a similar color scheme as the calculator, alarm and photos apps.
Added a small shadow at the bottom. And using the indicator colors for the systray.

@crutkas It's the same grey used in the Calculator icon; it might look different due to the different tile colour it's placed on.
I made some adjustments to my icon to make the padding thicker to match the stroke width of the other icons visually, as well as matching the drop shadow for the icon on the tile a bit better. Do you think this lines up better with the other icons?

My Logo design alongside other Tiles

@niels9001 @Elttob Awesome icons! I think it might be a tad too small on the live tile though? Compared to the other ones that is

@mdtauk really like your icon, especially the gradient cuz it feels nostalgic. the rest looks super sharp and modern.

@mdtauk really like your icon, especially the gradient cuz it feels nostalgic. the rest looks super sharp and modern.

The idea was to represent the Microsoft logo colours, the colour bars from the current logo, at the same time, reducing the amount of elements that make up the logo.

Added the coloured system tray icons to meet the initial requirements. I think they're pretty clean, but might tweak them a bit in the future if I have any new ideas on how to do them.

Added the coloured system tray icons to meet the initial requirements. I think they're pretty clean, but might tweak them a bit in the future if I have any new ideas on how to do them.

@Elttob I may be a bit late in the discussion, but I really liked your previous idea of having the gradient in the system tray. I love the latest iteration though. Some of the previous ones looked a bit like a trackpad icon, which is also why I liked the gradient idea. The gradient would work well with colours, as you showed previously.

We're working with the icon team right now.  May be 0.18, maybe 0.17 for new icon

Why does the PowerToys icon need to show state? Is there a feature in the works or planned that requires this feature? I'm also having a hard time understanding what the icon even represents. Is it supposed to be an old TV with a test pattern? What are all the buttons for?

@crutkas Will we have some PT icons with red and yellow colour or warning and stop sign? We could use such icons in the toast notifications. For example when the update failed or when the 'elevated running' notification is shown.

Why does the PowerToys icon need to show state? Is there a feature in the works or planned that requires this feature? I'm also having a hard time understanding what the icon even represents. Is it supposed to be an old TV with a test pattern? What are all the buttons for? I don't want a gradient in my system tray, that's distracting.

Waiting on a mono line that we then can do status on. Example: detected an elevated app or An app that causes conflict.

@crutkas Not sure what that first part means. Second part seems like a bug (elevated apps). And not sure notifying the user about known bad apps is useful through an icon color change.
Is there a design spec for this?

Is there a design spec for this?

I have written down my thoughts of how we can handle this here: https://github.com/microsoft/PowerToys/issues/1958#issuecomment-613334280

[Opinion]
I think conflicting apps can be detected at install time or simply listed in the README. (It's my understanding power users are the audience here.) Building out all this infrastructure for a handful of buggy apps seems extremely wasteful to me. (FancyZones elevated window issues seem temporary and fixable.)

The new icon is now available in 0.18 https://github.com/microsoft/PowerToys/releases


Error while trying to read builtins dir on MacOS

My LSP logs show the following:
[ERROR][2023-09-10 17:15:49] .../vim/lsp/rpc.lua:734    "rpc"   "/Users/bdavis/.local/share/nvim/mason/bin/openscad-lsp"        "stderr"     '[server] start sucess\n[error] read external builtin file error: Os { code: 2, kind: NotFound, message: "No such file or directory" }\n\n[server] search paths:\n[server] file:///Applications/OpenSCAD.app/Contents/Resources/libraries/\n\n'

This was installed using mason (nvim), which installs the ls using cargo install. Since the builtins file is not included in the cargo package, this essentially breaks the ls.

You can ignore this error, will use the content include in binary. @bendavis78


Rate limit joins to rooms

Sending a couple thousand join requests is bound to fail

Oh wait, we don't do this now. I'm sure we made it so that joins are 6 seconds apart.

don't we do this by now? https://github.com/Half-Shot/matrix-appservice-discord/blob/master/src/matrixroomhandler.ts#L513-L530

I suspect we do and this is an old bug.


Add more post-render lifecycle methods

Description
Since our renderer batches DOM updates in a non-deterministic fashion, there's no easy way for user code to check whether DOM updates are flushed or not. This is especially a requirement in cases where users want to run something that's dependent on state of DOM. Providing helper lifecycle methods that users can use would be great. We already have afterMount on WatchfulRenderElement so newer methods should have similar API:
Progress

[x] Add State.afterMount
[ ] Add State.afterUpdate
[x] Add WatchfulRenderElement.afterMount
[ ] Add WatchfulRenderElement.afterUpdate

Codecov Report

Merging #15 (be77d55) into main (f6f8831) will not change coverage.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##             main      #15   +/-   ##
=======================================
  Coverage   98.33%   98.33%
=======================================
  Files         182      182
  Lines        3847     3847
=======================================
  Hits         3783     3783
  Misses         64       64

Impacted Files
Coverage Δ

packages/rad/lib/src/widgets/stateful_widget.dart
100.00% <100.00%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us.

Codecov Report

Merging #15 (23abc4b) into main (f6f8831) will increase coverage by 0.03%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main      #15      +/-   ##
==========================================
+ Coverage   98.33%   98.36%   +0.03%
==========================================
  Files         182      182
  Lines        3847     3860      +13
==========================================
+ Hits         3783     3797      +14
+ Misses         64       63       -1

Impacted Files
Coverage Δ

.../core/common/abstract/watchful_render_element.dart
100.00% <100.00%> (+6.66%)
:arrow_up:

packages/rad/lib/src/core/renderer/renderer.dart
93.87% <100.00%> (+0.05%)
:arrow_up:

packages/rad/lib/src/widgets/stateful_widget.dart
100.00% <100.00%> (ø)

packages/rad/lib/src/core/run_app.dart
100.00% <0.00%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us.


Track relevant issues from iojs.org/old nodejs.org

There is a small list of issues (mainly enhancements and ideas), which might be relevant for nodejs.org as well:
https://github.com/nodejs/iojs.org/labels/new nodejs.org
https://github.com/nodejs/nodejs.org/labels/new.nodejs.org

I think we're up to date now, closing.

I think we should keep those around:

https://github.com/nodejs/iojs.org/issues/235 (Browser language based redirection for i18n)
https://github.com/nodejs/iojs.org/issues/80 ('latest' symlink for downloads)
https://github.com/nodejs/nodejs.org/issues/144 ('how to verify downloads' guide)
https://github.com/nodejs/nodejs.org/issues/74 (document supported versions + lts)
https://github.com/nodejs/nodejs.org/issues/49 (aka fix old documentation)


Traitor skill no longer exists

If you look in the passive skill tree in game, Conduit is in the place where "The Traitor" exists in PoB.

NVM I didn't understand a jewel could change a passive.


Consensus manager removal

Moved logic from consensus manager to consensus loop
Fix: #1436

I don't see anything wrong at 1st glance, but I have not run anything as usual and this PR needs just running everything more than anything else.

I think its fairly safe to commit this PR as is.

I think the 2 interfaces here are not necessary:
INetworkDifficulty, IGetUnspentTransaction.
INetworkDifficulty is only used there and IGetUnspentTransaction is also used in MempoolManager like so:
public interface IPooledGetUnspentTransaction : IGetUnspentTransaction
{
}

I think the 2 interfaces here are not necessary

Agreed and I think we should get rid of them since they are more like a hack to be honest. In general I don't like the way we break architecture sometimes by defining the interface in the base feature and implementing it somewhere else.

INetworkDifficulty is only used there

It's used in FN controller's GetInfo() (so is the 2nd guy).
I think those things should be improved but they are outside of the scope of this PR. It's more like a design decision of how we deal with dependencies or do we keep RPC in one feature or do the same as we do with API - different features have their controllers.

ah yeah didn't see that.
Ok leave it for now, we can rediscuss that after the current more pressing work.
Are all the tests passing?

Are all the tests passing, apart from the ones that were already failing?

Yes. Also synced the chains- all fine for me.

ok i'll push it.


dlpi_name can be null sometimes.

Hello!
Thanks for the library. I'm using it on Android with conjunction xUnwind.
I've found that android x86 apiLevel=23 (emulator) sometimes call build_id_find_nhdr_callback function with nullptr of info->dlpi_name
If I change
static int
build_id_find_nhdr_callback(struct dl_phdr_info *info, size_t size, void *data_)
{
(...)
    if (data->tag == BY_NAME && strcmp(info->dlpi_name, data->name) != 0)
        return 0;

to
static int
build_id_find_nhdr_callback(struct dl_phdr_info *info, size_t size, void *data_)
{
(...)
    if (data->tag == BY_NAME && (!info->dlpi_name || strcmp(info->dlpi_name, data->name) != 0))
        return 0;

then works fine: buildId is arrived successfully.
On apiLevel=30 is not reproduced.

Interesting, thanks!
Would you like to make a pull request?

I'm not pretend to history of the commit records, so you can do it yourself.
Thanks for the offer.


Improving folder structure and adding react component test files

react component test files are not there, we can add test files to test react components by running npm test
improve folder structure, like keeping every component in a separate folder with their corresponding test files.

Any principals and style guides you want us to follow or it's just a suggestion? @agentraghav

it's a suggestion and yaa there are few rules that must be followed or it will be good for contributors if they are followed like

Every component should be in a separate folder with the corresponding testing file.
It is good to keep the styling file of that component in that folder instead of creating a separate styles folder.
file naming should be in lower case, use a (-) dash or underscore ( _ ) instead of using camel case.

@Mr-spShuvo  I can raise a PR with improved folder structure and including test files, if you find that useful, you can merge that

Thanks for your personal suggestions, but we are not looking for any changes to current directory structures. As there are no hard and fast rules, and everybody prefers different working tree, we just want to stick to the current folders and files structures. May be in future we can change that, but currently our main focus is releasing the MVP asap.

okay


Compatibility with Rails 6 and Zeitwerk loader

This change seems to fix the problem in Rails 6 production env, when zeitwerk tries to eager-load all the code and can't find the constants RailsWarden::Version. I didn't have any problems in development env, for some reason, though.
[...]
/home/florian/.rvm/gems/ruby-2.6.5@default/gems/zeitwerk-2.4.0/lib/zeitwerk/loader/callbacks.rb:17:in `on_file_autoloaded': expected file /home/florian/.rvm/
<EMAIL_ADDRESS>to define constant RailsWarden::Version, but didn't (Zeitwerk::Nam
eError)

@Raven24 any idea if this is backwards compatible with older Rails versions or does this need to be released as a major version?

I think the best way forward with this is to fix the folder / file layout to meet Zeitwerk expectations, I have a branch ready which does this however would like #34 reviewed first

I think the best way forward with this is to fix the folder / file layout to meet Zeitwerk expectations, I have a branch ready which does this however would like #34 reviewed first

That would be great. Unfortunately I don't know if my change would be backwards compatible. I just made the smallest modification I could in this PR, to make it work in the Rails app I am working on.
So I'd absolutely be in favor of dropping this PR for something better already in the pipeline.

That would be great. Unfortunately I don't know if my change would be backwards compatible. I just made the smallest modification I could in this PR, to make it work in the Rails app I am working on.
So I'd absolutely be in favor of dropping this PR for something better already in the pipeline.

@Raven24 @jsmestad I did some testing with a Rails 5.0.7 application and and a Rails 6 application and this PR works fine with both versions so I would say this is good to merge and release a new Zeitwerk compatible version

@Raven24 @jsmestad I did some testing with a Rails 5.0.7 application and and a Rails 6 application and this PR works fine with both versions so I would say this is good to merge and release a new Zeitwerk compatible version

@jmortlock Thanks! I will keep this in master for awhile and make sure it's working for more folks before cutting an official release.

@jmortlock Thanks! I will keep this in master for awhile and make sure it's working for more folks before cutting an official release.


[WIP] Deleted API param description

Motivation and context

Added API documentation for deleted parameter. Added missed params without_tasks, names_only description.
How has this been tested?

Manually
Checklist

[x] I submit my changes into the develop branch
[ ] I have added description of my changes into CHANGELOG file
- [ ] I have updated the documentation accordingly
- [ ] I have added tests to cover my changes
- [ ] I have linked related issues (read github docs)
- [ ] I have increased versions of npm packages if it is necessary (cvat-canvas,
cvat-core, cvat-data and cvat-ui)

License

[x] I submit my code changes under the same MIT License that covers the project.
Feel free to contact the maintainers if that's a concern.
[x] I have updated the license header for each file (see an example below)

# Copyright (C) 2021 Intel Corporation
#
# SPDX-License-Identifier: MIT

Coverage decreased (-0.01%) to 75.246% when pulling 5c82b9605f565b20d83f30608a17a3c939002db8 on dk/label-deleting-doc into e41c301251187844b87336dcba18e9f89613281d on develop.


where to start after crash of kafka connect oracle

I found that after  kafka connect had crashed or stopped , then start it now, it will start new capture from now on. So, some data will miss in kafka topic.  How to capture from the timepoint it crash?

Hi,
if "reset.offset"  configuration property is set to true , every time connector start it began to capture from now on . If it is set to false it began from where it left off. Could you please check it ?
Thanks

thanks .


Bug in translation

I found a bug in translation.... Show "Cler" but should show "Clear"

When you are planing implement multilanguage?

Closing as duplicate of #262


GO:0050754	 positive regulation of fractalkine biosynthetic process merge into GO:0032603 production

add reference PMID:12729461
add synonyms CX3CL1

I had omitted

'GO:0050752 regulation of fractalkine biosynthetic process' > merge into 'GO:0032644 regulation of fractalkine production'
'GO:0050753 negative regulation of fractalkine biosynthetic process' -> merge into 'GO:0032684 negative regulation of fractalkine production'
(no annotations)


Improve link sharing to switch icons, rather than have a popup

Use the approach in https://github.com/ParkingReformNetwork/reform-map/pull/291. It looks more professional.

Closed by https://github.com/ParkingReformNetwork/parking-lot-map/pull/175


Prints attrs and instance vars

Not sure if this is intended, but I just recently updated from an old version and discovered that ap now prints out attr_reader attributes, instance variables and the like.  This results in a lot of extra output, that makes things so verbose it's hard to read.  Just curious if anyone else is experiencing this and if there's a way to reduce the amount of amount from ap.

Yeah, I've had to go back to pp myself for that reason.

Yes, displaying attributes and instance vars of an object is part of v1.0 release. What is the object that you're inspecting?

For me, it's mostly mongoid objects
On Dec 13, 2011, at 7:16 PM, Michael Dvorkin
<EMAIL_ADDRESS>wrote:

Yes, displaying attributes and instance vars of an object is part of v1.0 release. What is the object that you're inspecting?

Reply to this email directly or view it on GitHub:
https://github.com/michaeldv/awesome_print/issues/66#issuecomment-3133241

I have problems with ActiveRecord and ActiveRelation objects.  Especially when you're dealing with an array of even two or three, it becomes really hard to read.  Here's a super simple example where creating a new AR object prints with a ton of extra data:
Employee.create :username => 'john.smith', :name => 'John Smith', :email =><EMAIL_ADDRESS>:aim => 'jsmith', :gchat =><EMAIL_ADDRESS>:twitter => 'jsmith'
#<Employee:0x1118ca340
    @attributes_cache = {
        "created_at" => Wed, 14 Dec 2011 03:00:37 UTC +00:00,
        "updated_at" => Wed, 14 Dec 2011 03:00:37 UTC +00:00
    },
    @destroyed = false,
    @marked_for_destruction = false,
    @new_record = false,
    @previously_changed = {
             "gchat" => [
            [0] nil,
            [1]<EMAIL_ADDRESS>        ],
              "name" => [
            [0] nil,
            [1] "John Smith"
        ],
        "created_at" => [
            [0] nil,
            [1] Wed, 14 Dec 2011 03:00:37 UTC +00:00
        ],
        "updated_at" => [
            [0] nil,
            [1] Wed, 14 Dec 2011 03:00:37 UTC +00:00
        ],
          "username" => [
            [0] nil,
            [1] "john.smith"
        ],
               "aim" => [
            [0] nil,
            [1] "jsmith"
        ],
                "id" => [
            [0] nil,
            [1] 1
        ],
             "email" => [
            [0] nil,
            [1]<EMAIL_ADDRESS>        ],
           "twitter" => [
            [0] nil,
            [1] "jsmith"
        ]
    },
    @readonly = false,
    attr_accessor :attributes = {
             "gchat" =><EMAIL_ADDRESS>              "name" => "John Smith",
        "created_at" => Wed Dec 14 03:00:37 UTC 2011,
            "github" => nil,
        "updated_at" => Wed Dec 14 03:00:37 UTC 2011,
          "username" => "john.smith",
                "id" => 1,
               "aim" => "jsmith",
            "mobile" => nil,
           "twitter" => "jsmith",
             "email" =><EMAIL_ADDRESS>    },
    attr_accessor :validation_context = nil,
    attr_reader :changed_attributes = {},
    attr_reader :errors = {}

I don't want to whine, but I find it a bit verbose as well. I would just like to reiterate that I love ap and I appreciate all your hard work keeping us whiners happy!

Yeah, I have to agree with gurgeous.  I apologize for being a whiner on this.  Not cool, given the hard work, and value added to the community.  My sincere apologies Michael.

My last commit takes care of extra noise for ActiveRecord models. By default only model attributes are shown unless you specify :raw => true. This change should be part of upcoming 1.0.2 release.

@jeffdeville Mongoid instances are going to be nicely formatted too.  Thanks!
On Dec 19, 2011, at 2:07 PM, Michael Dvorkin
<EMAIL_ADDRESS>wrote:

@jeffdeville Mongoid instances are going to be nicely formatted too. Check out https://github.com/michaeldv/awesome_print/commit/19c8e74e853f29f533ba88afa9065dab3d2f0cff commit by @gurgeous :-)

Reply to this email directly or view it on GitHub:
https://github.com/michaeldv/awesome_print/issues/66#issuecomment-3207820

@michaeldv awesome news thank you!  As everyone has said, we all love ap, so thank you for staying on top of it all and dealing with us whiners :)

Just pushed 1.0.2 to Rubygems

Hey, this also causes an infinite loop/recursion if you have cyclical references in objects you're inspecting. I think this feature needs to be rethought or made optional. Has anyone else encountered this?

There's built-in mechanism to detect cyclical references based on object.object_id. For example:
 > data = [ false, 42, %w(forty two) ]
 > data << data  # <-- Nested array.
 > ap data, :multiline => false
 [ false, 42, [ "forty", "two" ], [...] ]

Do you have the code to reproduce the problem?

Ok, I'm not 100% sure it's a cyclic thing, but if I inspect any object from the program I'm working on it works for a few minutes until the process runs out of memory. Unfortunately it's not an open source program that I can share. I'll see if I get the same behaviour with something open source or otherwise reproduce the problem. My simple attempts to reproduce it haven't worked so far.

To give you an idea of how much stuff it's trying to output when I inspect an object, take a look at this gist:
The numbers it prints is how deeply nested the inspect is when it enters the awesome_hash method.
https://gist.github.com/1509116
Note that I only bothered monkey patching the awesome_hash method which is why the nesting levels are not sequential.

Could you please try this and see what happens:
require 'yaml'
y your_object

(basically try 'y' instead of 'ap').

The data's not yaml compatible but maybe you'll get a vague sense of it from this: https://gist.github.com/83621e2e4b2c3b0aef4e though it's still pretty short on information.
Essentially there are a bunch of arrays/sets/hashes of objects that define the object, some of which reference the class that contains it and often other simillar classes as well, for instance to define links or dependencies between the classes.

What about introducing :depth option?

That is possible, but really when you think about it, isn't it a bad idea
to just automatically display all of a class's private instance variables?
I would really hate as a library owner it if that extra transparency caused
users to start to directly manipulate the private state of my objects and
thereby make it impossible for me to refactor without breaking their code.
The more I think about this feature the more convinced I am that it should
be optional -- and default to off.
The problem with specifying depth is that it affects my ability to easily
view other data structures, which is something I often want to do. In fact
the ability to clearly see the contents of deeply nested hashes or arrays
is the whole reason I use awesome print in the first place. To be forced to
set a global depth defeats the purpose.
Thoughts?
Darrick
On Thu, Dec 22, 2011 at 1:51 AM, Michael Dvorkin <
<EMAIL_ADDRESS>
wrote:

What about introducing :depth option?

Reply to this email directly or view it on GitHub:
https://github.com/michaeldv/awesome_print/issues/66#issuecomment-3246111

Well, 'y' from yaml standard Ruby library is also dumping all the instance variables and accessors. It has been this way for years and I haven't heard any requests to change that behavior. awesome_print just adds some better formatting but as far as the content goes it's all the same.

That may be true, but 'y' isn't typically used in the console to display
the output for user consumption so I don't think the comparison is really
apt.
I'm also not saying that this isn't a very useful feature, in fact I can
think of many situations that using ap to see instance variables will be
very handy.
The only thing that I think is a mistake is to make it the default
behaviour when I inspect a value in my console.
In addition, don't you think this behaviour is really broken in a situation
like the following?
[34] pry(main)> class Password; def initialize; @pw = 'abc'; end ; def
inspect; "(Password #{'*' * @pw.length })"; end end
nil
[35] pry(main)> Password.new
#<Password:0x00000ff8
@pw = "abc"

A hallmark of an excellent Ruby library is that it displays useful and
intelligent information in the inspect method. That is a long standing Ruby
convention. Don't you think that this behaviour breaks some very valid
assumptions about how Ruby objects are used?
Regards,
Darrick
On Thu, Dec 22, 2011 at 2:26 AM, Michael Dvorkin <
<EMAIL_ADDRESS>
wrote:

Well, 'y' from yaml standard Ruby library is also dumping all the instance
variables and accessors. It has been this way for years and I haven't heard
any requests to change that behavior. awesome_print just adds some better
formatting but as far as the content goes it's all the same.

Reply to this email directly or view it on GitHub:
https://github.com/michaeldv/awesome_print/issues/66#issuecomment-3246305


Add file input widget

When 'blob' input is used the string is often too long to be pasted into the 'Object' field.
This PR adds a file input widgets which sets the 'objects' field from the file data as well as the 'extension' from the filename

Hi @Jhsmit
Thanks for this contribution. I just checked it out using a pdb file with the content below.


Group Column for a dynamic chart

Hello,
I need to use a Stacked bar chart, but my groups being dynamic values.
I have one service who give me my datapoints, datapoints and groups.
Looking at code, I can see that "addGroup" function only being used at "chartGroup Directive", and, if I use that, the groups will be static values.
I need something like this:
<c3chart bindto-id="chart-timeseries" chart-data="datapoints" chart-columns="datapoints" chart-x="datax" chart-group="dataGroups -> ['group1', 'group2']">
  <chart-axis-x axis-id="x" axis-type="category">
    <chart-axis-x-tick tick-rotate=90 tick-fit="false" tick-multiline="false" />
  </chart-axis-x>
  <chart-events on-click-data="ctrl.redirecionarSlaMensagensRespondidasParaAtendimentos(data)" />
</c3chart>

Its possible to do with actual version of directive?
If not, looking at the code, this feature look like easy to do, only need to add ["chartGroups": "=chartGroups"] at the directive config. What you think about this feature?

I am also having same issue, dynamic groups, stacks can be added or removed dynamically. but chart is not ready for this kind of requirement.

Yes. I tried to to the modification I say in my local code and work fine. If autor prefer, I can fork and submit a pull request with this feature.

Pull request would be nice, I am very busy for already a few weeks, don't
see it change the coming few weeks. Need to find some time to take the
library to the next step.
On Tue, Oct 27, 2015 at 1:35 PM, oelze-cit<EMAIL_ADDRESS>wrote:

Yes. I tried to to the modification I say in my local code and work fine.
If autor prefer, I can fork and submit a pull request with this feature.
—
Reply to this email directly or view it on GitHub
https://github.com/jettro/c3-angular-directive/issues/72#issuecomment-151477864
.

--
Jettro Coenradie
http://www.gridshore.nl

@oelze-cit can you quickly provide fork? I can get help from your work. I need it urgently.

@sajidali Yes, i can work on this today. At the end of day I will open a pull request (After my work time). Can y w8ait at end of day?

@oelze-cit sure :+1:

@sajidali , sorry dude, I'm very busy yesterday. Today I will work on this feature and open a pull request as soon as I can.

I will work on it now. After a few minutes i will finish. (I'm oelze-cit, but this is my personal account)

Waiting, hopefully it will be useful in my own use case.

@sajidali , I'm having some problems with the feature. I add you to contribute to the repository, if can y can help contact me please :+1:

@oelze-cit Added Comments in your fork.
https://github.com/henriqueoelze/c3-angular-directive/commit/faf2f0848e44780cfbefd7ef10c93590db85b206

Is still issue still needed? I like the idea, but I have more dynamic requests. I am thinking about a way to make this more generic.

Hi,
I have a need for a dynamic group.. I want to provide a button for stacked and unstacked..
any progress on that PR?


Remove IdentityServer from angular template

Use default Identity UI with cookies instead

Contributes to https://github.com/dotnet/aspnetcore/issues/47226

Is there no way to know when you are authenticated or not? Isn't it confusing that you log in, come back to the app, and you see no visual change?

Nope. Currently the best we can do with just cookie auth and the default Identity UI is detect the redirect. Later, MapIdentityApi will provide a way to detect this which we will provide samples for. Eventually it would be nice to update these project templates to use it and not use the default Identity UI razor pages at all, but it's not ready yet.

What happens with all the state the app has before we redirect? Do we throw it all out?

Yes.

Why are we removing the authorize guard? That's meant to ensure you are authenticated when you access some areas of the page.

Because we have no way to know if we're authenticated or not.

How do you log out in the template?

You click the "Account" link in the nav bar. And then click "Logout" on the /Identity/Account/Manage page.

Where are we wiring up Antiforgery?

It's not explicit. This is just the default Identity UI razor pages. I can confirm that the __RequestVerificationToken is rendered though.


Add third-party library code examples

Summary
Update and add the code samples currently published on codesandbox.io to version control.
How did you test your changes?
Manual testing.
cd examples/<sample name>
npm install
npm start

Too big. Breaking this up into smaller PRs.


Support mut, remove readonly

Supporting both forms at once is not possible; we can offer a language version flag to support transitions


Bump version to 3.4.0

Thanks for reviewing @kevindew. I made an off-hand comment in Slack sometime ago after checking a few repos, where the only issue was a missing desc. More troublesome repos:

Whitehall (inline classes/methods)
Smart Answers (inline classes/methods)

It'll be in its own dependabot PR, so I'm just going to merge this, and we can always revert if it's proves too troublesome to adhere to. Most of the fixes should be pretty easy.


Add request alias specific to the Laminas namespace

Q
A

Bug fix?
yes

Related tickets
fixes #198

License
MIT

What's in this PR?
Since laminas-diactoros 2.7.0 - it appears that Message Factory discovery fails (When the deprecated MessageFactoryDiscovery class is used)
Pretty sure that this is down to the removal of laminas-zendframework-bridge in that release.
To reproduce
composer require php-http/discovery
composer require laminas/laminas-diactoros ^2.7
composer require php-http/message-factory

And then run a php script along the lines of:
<?php
require 'vendor/autoload.php';

use Http\Discovery\MessageFactoryDiscovery;
$factory = MessageFactoryDiscovery::find();

This should yield a DiscoveryFailed exception.
Apologies, but I'm not sure how to go about writing tests for this scenario…
Checklist

[x] Updated CHANGELOG.md to describe BC breaks / deprecations | new feature | bugfix

https://github.com/php-http/discovery/releases/tag/1.14.1


[Feature Request] Light Sensor

Is it possible to use this LDR Sensor with your Out of the Box Clockwise version?
https://de.aliexpress.com/item/32648002929.html?spm=a2g0o.productlist.main.91.3bb35cbbJCel8r&algo_pvid=7b60ab7a-91fb-4bee-8bce-148f8f87c61c&algo_exp_id=7b60ab7a-91fb-4bee-8bce-148f8f87c61c-45&pdp_npi=3%40dis!EUR!0.40!0.35!!!0.43!!%402145280e16903917914547086d0762!10000001111815755!sea!DE!2103824747&curPageLogUid=P2nasSHnZjtv
Thanks in advance.
Ps.: Best Clock I ever had :)

Hi Fabian, thank you. It is possible but you can simplify it using only the LDR. If you are using a Trinity board, there is placeholder in the board or if you are using  a ESP32 devboard, connect it like in the diagram here -> https://github.com/jnthas/clockwise/wiki/Connecting-the-LDR

I cannot find the resistor value of A9050. What is the recommended value?

The A9050 is an Light Dependent Resistor or LDR


chore: add sovrin buildernet

Summary of Changes
This adds the Sovrin BuilderNet ledger, and also places the Sovrin StagingNet as first in the array since it should take precedence over the testing networks.
Related Issues
N/A
Pull Request Checklist
This is just a reminder about the most common mistakes. Please make sure that you tick all appropriate boxes. But please read our contribution guide at least once; it will save you a few review cycles!
If an item doesn't apply to your pull request, check it anyway to make it apparent that there's nothing to do.

[x] All commits contain a DCO Signed-off-by line (we use the DCO GitHub app to enforce this).
[x] Updated LICENSE-3RD-PARTY.md for any added dependencies or vendored components.
[x] Run prettier: npm run style-format
[x] Updated documentation for changed code and new or modified features.

If you have any questions to any of the points above, just submit and ask! This checklist is here to help you, not to deter you from contributing!
PR template adapted from the Python attrs project.

There's a timeout issue with the tests right now so the last merged PR skips those until we can update them. For now if you update this branch it should pass all checks and be merged.