feat(outputs): add hostname, tags, and custom fields to TimesacleDB

What type of PR is this?
/kind feature
Any specific area of the project related to this PR?
/area outputs
What this PR does / why we need it:
This PR adds support for hostname, tags, and custom fields for the TimescaleDB output.
Which issue(s) this PR fixes:
Fixes #

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: alika
Once this PR has been reviewed and has the lgtm label, please assign issif for approval. For more information see the Kubernetes Code Review Process.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

I see the output fields are not added, because of the the schema. Do you think it worth to add k8s.pod.name and k8s.ns.name at least? IN prometheus for example, to avoid huge cardinality, we add these fields as labels and the custom fields.

Sure, sounds reasonable. I'll add support for output fields.

I see the output fields are not added, because of the the schema. Do you think it worth to add k8s.pod.name and k8s.ns.name at least? IN prometheus for example, to avoid huge cardinality, we add these fields as labels and the custom fields.

Sure, sounds reasonable. I'll add support for output fields.

All output fields it will be hard, because they are dynamic, but at the custom fields added statically by Falcosidekick BUT it means you need to deal with the addition of new fields after the creation of the index aka, ie, a schema update.
Tell me when it's ready, I'll review this PR at this moment. Thanks.

I see the output fields are not added, because of the the schema. Do you think it worth to add k8s.pod.name and k8s.ns.name at least? IN prometheus for example, to avoid huge cardinality, we add these fields as labels and the custom fields.

Sure, sounds reasonable. I'll add support for output fields.

All output fields it will be hard, because they are dynamic, but at least the custom fields added statically by Falcosidekick BUT it means you need to deal with the addition of new fields after the creation of the index, ie, a schema update.
Tell me when it's ready, I'll review this PR at this moment. Thanks.

After I looked at how newFalcoPayload works I understand how OutputFields is already populated w/Customfields and Templatedfields. I have an implementation right now that only adds to TimescaleDB payload if it is declared in either Customefields or Templatedfields. Just doing a bit more actual testing and I'll have this PR updated soon (<24h).

@Issif ready for review now. I had to touch config because I was getting a panic on templated fields:
panic: assignment to entry in nil map

goroutine 1 [running]:
main.getConfig()
        /Users/alika/projects/ref/falcosidekick/config.go:474 +0x3eeb
main.init.0()
        /Users/alika/projects/ref/falcosidekick/main.go:89 +0xff

Sorry for the delay, I was successively sick, off and at kubecon in last weeks. Have you fixed the issue with the nil map? Can I review the whole PR now? Thanks

Sorry for the delay, I was successively sick, off and at kubecon in last weeks. Have you fixed the issue with the nil map? Can I review the whole PR now? Thanks

No worries. Hope kubecon was fun. I did fix this issue w/the nil map and I'm ready for a review.

LGTM label has been added.  Git tree hash: a3aac9de46e6817dc30db3c9e2b7d2150577bb20


GH-39049: [C++] Use Cast() instead of CastTo() for Dictionary Scalar in test

Rationale for this change
Remove legacy code

What changes are included in this PR?
Replace the legacy scalar CastTo implementation for Dictionary Scalar in test.

Are these changes tested?
Yes. It is passed by existing test cases.

Are there any user-facing changes?
No.

Closes: #39049

@kou Would it be possible for you to review this PR? I anticipate that once this PR and the GLib PR are merged, we will be able to proceed with the task of deprecating CastTo. #39182
Thank you as always.


Updating types package to interval

Coverage remained the same at 51.774% when pulling 34612c4276ecba23ac862cbfb72e30c2bf2c42ea on to-interval into 0895dde70da469d78869b18e669aa3aecec49bb2 on master.

Coverage remained the same at 51.774% when pulling 453a5a79be80b933fe726958e2f9420ab05961e4 on to-interval into 0895dde70da469d78869b18e669aa3aecec49bb2 on master.


Windows cannot detect language

Select English when it's not possible to detect LOCALE in Welcome Screen.

This issue doesn't seem to only affect windows. I think the reason is because locale are more specific than just the two character code.
English, for example can be en-US, en-UK, etc.  PR #181 should resolve.
Changes the default selection so that if a value is returned from getLocale, that value is split on -, and only the first part used.  So on my mac when getLocale() gives en-US it'll still set the default value to en.
$('#locales-list').val(app.getLocale() ? app.getLocale().split('-')[0] : 'en');

Updated PR with better logic after I noticed some of the translations files use the full locale, and others are just a two character code. New logic will support both, and default to english if nothing matches.
// Try to figure out user's language
if (app.getLocale() in i18n.fullLocaleList) {
  $('#locales-list').val(app.getLocale());
} else if (app.getLocale().split('-')[0] in i18n.fullLocaleList) {
  $('#locales-list').val(app.getLocale().split('-')[0]);
} else {
  $('#locales-list').val('en');
}

Changes the default selection so that if a value is returned from getLocale, that value is split on -, and only the first part used. So on my mac when getLocale() gives en-US it'll still set the default value to en.

Nice! I didn't noticed that issue here because my mac is pt-BR, the same available language.


java-generator: unit tests validate Java source syntax with the [javaparser] library

awesome - merging now


How can i update vuetable with custom data ?

I have an ajax function where i get custom data from the server. So i already have some data on the vuetable. I want to manually refresh the table and add the custom data.

@Nisthar If I understand you correctly, you would like to have vuetable using the data you've already retrieved via existing ajax function, right? If so, you cannot, vuetable will automatically load the data from the server via api-url prop. If I misunderstand your point, please explain more.

Yes, But in addition to the automatic ajax function of the vuetable, i want
to manually do ajax and change the data of the vuetable.
Anyway, I accomplished this with another way. I changed the apiUrl prop of
the vuetable and set it to new url which returns the custom data that i
want  (not the original data) and refreshed the table. So the table got
populated with the custom data.
BTW is there any other way like manually setting the "tableData" with
custom data and refreshing the table?
On Sun, Dec 4, 2016 at 11:56 AM, Rati Wannapanop<EMAIL_ADDRESS>wrote:

@Nisthar https://github.com/Nisthar If I understand you correctly, you
would like to have vuetable using the data you've already retrieved via
existing ajax function, right? If so, you cannot, vuetable will
automatically load the data from the server via api-url prop. If I
misunderstand your point, please explain more.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/ratiw/vue-table/issues/132#issuecomment-264687165,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ALcELXTzpGvaD4lSWc5xTtBwzJYUOXOwks5rEl0xgaJpZM4LDS_B
.

@Nisthar There is actually another component called vueitems, but it serves different purpose than vuetable.
With vueitems you have to manually retrieve the data yourself and give it to vueitems to display. It is aimed to be part of a form input (think of invoice items), so many functionalities related to pagination, sorting are removed. However, I haven't put any documentation on the project, but most of the props and events are the same as vuetable. You just need to go through source code to learn about it.

I also think there should be a way to set any data to the table. May be a callback function or event. There are many cases where you dont need to make an api call to display a table with some info.

Yes, absolutely. What about a prop to provide data to the table, and if it's not used, then use the API call (or vice versa - doesn't really matter)?

I managed to reload vuetable for non-api table when data is changed in the background with this.$refs.vuetable.setData(this.data).


Refactoring - GridPropertyCollection

[x] Improve unit test coverage
[x] Decrease complexity for code

Remaining work for Future Enhancements:

add test data for Nexus ICOARS usage
add unit tests for decoarsening if ICOARS properties
refactor decoarsening function(s)


$(".well:nth-child(2)").addClass("animated bounce") doesn't work

Challenge http://www.freecodecamp.com/challenges/waypoint-target-a-specific-child-of-an-element-using-jquery has an issue.
I add $(".well:nth-child(2)").addClass("animated bounce"); code into document.ready in jQuery. Buttons bounce, but the challenge is not considered completed.

I've understood my mistake :-) Closing the issue.

@husamp: Could you tell me how you resolve this  problem?
I tried  $("#left-well:nth-child(2)").addClass("animated bounce"); but it still not work.

am stuck at that same point mind sharing the solution


Firebase keys

Not sure if it's intentional but your firebase keys are public

Hi, LittleFrosty!
Thank you for bringing this to my attention. I appreciate your concern. However, I want to clarify that the Firebase keys are intentionally public as they are needed for the project to function properly. I have created necessary Firebase security rules to ensure the safety and integrity of the data. If you have any further questions or concerns, please don't hesitate to reach out.


EDF+ Annotation Timestamps missing submillisecond accuracy

It was noticed that the onset time for annotations in an EDF+ file might be missing the submillisecond accuracy accociated with the start time of the recording. This results in annotation timestamps being delayed from the true event time.
Expected results
The EDF+ file used here contains patient data and cannot be shared. However, if this bug is confirmed all EDF+ files will be affected.
If the EDF+ file is brought into EDFbrowser it will be noticed that the millisecond aspect of the start time is displayed in the Start field:

For instance, this results in the annotation timestamps being:

Actual results
When loading the data into MNE the annotation timestamps return:

If you take the difference between the MNE timestamp and the EDFbrowser timestamp it is equal to the submillisecond portion of the recording start time.
Additional information
I reached out to Teunis van Beelen, the developer of EDFbrowser, to get an explanation. The explanation provided was as follows:

EDF does not have support for subsecond precision of the starttime but EDF+ does. Because EDF+ needed to be backwards compatible with EDF and EDF doesn't have the possibility to write subsecond precision for the starttime into the header, the subsecond part of the starttime is written outside of the EDF header. The way the subsecond part of the starttime is stored in EDF+ is described here: https://www.edfplus.info/specs/edfplus.html#timekeeping

Teunis van Beelen provided a quote from the EDF specs:

"The startdate/time of a file is specified in the EDF+ header fields 'startdate of recording' and 'starttime of recording'. These fields must indicate the absolute second in which the start of the first data record falls. So, the first TAL in the first data record always starts with +0.X2020, indicating that the first data record starts a fraction, X, of a second after the startdate/time that is specified in the EDF+ header. If X=0, then the .X may be omitted."

In other words, the subsecond part of the starttime is written in the EDF Annotations "signal".

do you know any public file that would allow us to
replicate/investigate this?
otherwise you can try to hack the edf.py file. Relevant code starts here:
https://github.com/mne-tools/mne-python/blob/master/mne/io/edf/edf.py#L231

I have de-identified the file I used:
https://drive.google.com/file/d/1yql0bMGcXXrEZr1At_Cp-eMi6rTKSr_F/view?usp=sharing

can you try this PR https://github.com/mne-tools/mne-python/pull/7875
your explanations helped a lot !

Glad I could contribute. The explanations provided by Teunis van Beelen helped me to understand the issue so credit should be given to him.

@agramfort @cbrnr I tested the response of FieldTrip and I am getting the same behavior. Seems to be missing the submillisecond accuracy:

Will reference this PR on their repo

Biosig also doesn't support this, I've reported this to the maintainer. Seems like edflib is the only app with proper support (but now also MNE).

@cbrnr interestingly though pyEDFlib is a python wrapper for EDFlib and it also shows the incorrect behavior... just tested

@greydongilmore this is because we haven't integrated the new EDFlib version 1.17 yet (see https://github.com/holgern/pyedflib/pull/81)
I hope I find time soon to implement the changes.


ip allocator: rename AllocateIPs method

📑 Description

The AllocateIPs method name is misleading; it receives a slice of IPs as parameter, which leads the user to think that it can allocate the IPs in the slice in a single subnet - i.e. for subnet <IP_ADDRESS>/24, it could allocate IPs <IP_ADDRESS> and <IP_ADDRESS>.
That is not the case: that function allocates a single IP address in each of the subnets the IP allocator for said network manages. For instance, on a dual-stack network, it could handle the following subnets: <IP_ADDRESS>/24 and fd10::/64. The function could then the invoked with ips=<IP_ADDRESS>,fd10::2 to allocate one IP in each of the two subnets managed by the IP allocator.

Fixes #
Additional Information for reviewers

I've renamed the method, and corrected its docstring, in hope of making its intended usage clearer for future users.
✅ Checks

[ ] My code requires changes to the documentation
[ ] if so, I have updated the documentation as required
[ ] My code requires tests
[ ] if so, I have added and/or updated the tests as required
[ ] All the tests have passed in the CI

How to verify it

/assign @martinkennelly


iOS build error 'SDWebImage/SDWebImage.h' file not found

iOS build fails with
'SDWebImage/SDWebImage.h' file not found
at
#import "ScreenGuard.h"
#import <SDWebImage/SDWebImage.h>
Did arch -x86_64 pod install also with update, desintegrate and then pod install again, clean build folder. Could it be something on my side?
"react-native": "^0.70.8",
"react-native-screenguard": "^0.3.0",

platform :ios, '12.4'

it is for beta version only, please revise (README.md)[https://github.com/gbumps/react-native-screenguard#readme] carefully before proceeding.

I have installed stable, not beta. Do you mean I have to install Beta for ios or what to look for in Readme? Or I could somehow exclude SDWebImage using the basic stable version? How? I need only the basic functionality or android only is still OK if I can build on ios.

I still don't understand what your intention of integration with SDWebImage on the stable version. Please describe more detailed.

OK I see now: I did not want to install beta or SDWebImage. I just did
npm install react-native-screenguard --save
When trying to build on ios it raises the error. It seems like the stable version also tries to include SDWebImage.

yes. Just remove the  <SDWebImage/SDWebImage.h> ad rebuild, I'm doing on a fix for the previous one

Thank you!It works!Will try it on live app after some testing!:)

This issue has been resolved at version 0.3.2, please install and rebuild. Thanks!

@gbumps I think the issue is back. I just installed via yarn add react-native-screenguard (and run pod install). After building, I get
(node_modules/react-native-screenguard/ios/ScreenGuard.mm:2:9)
> 2 | #import "SDWebImage/SDWebImage.h"
    |         ^ 'SDWebImage/SDWebImage.h' file not found

Am I missing some installation step?

hi @ncoen97, please go back to the stable version 0.3.6 while i'm reviewing the code once more.
thanks!

Great, thanks for the reply!
Are there any known issues in 0.3.6 I should be aware of? I recall seeing some bugfix commits between the 0.3.6 and 0.4 history 🤔

there are very small update between 0.4.0 and 0.3.6, so don't worry about it.
I have been mismatched with the code between stable version and beta version. All 2 versions will be merged into one on the next release.

Same issue.

Hi @ncoen97 @wongzigii, version 1.0.0 has been fully released, please install and rebuild.
Thanks!


How to return the output as a Map not as String ?

I want to receive the output in Map .ie _result['textOutputOrError'] value must be a map.
but I'm receiving String ?
def method():
"code"
dict['a']="ListA"
dict['b']="ListB"
return  dict
print(method) // result['textOutputOrError'] contains value in string, I want value in a map
// I don't write print(method) output value is null

The sole purpose of this library is to run any python script. can you try out this example code and see if the script runs or not? The problem with the current script is that you need to be sure about the indentation errors which is pretty hard to identify with static scripts.

My script is running perfectly fine.
I just want to recive output in Map not in string

result['textOutputOrError'] is { 'a': [0,1,2] ,'b': [0,1,2],'c': [0,1,2],} as String I want it as map

@vishalideastoimpact the thing is that, this library is for running the python script that you provide as an input and returns the output string. If you want to do the processing on the output code, you need to provide final script which performs all the operations.

can Tell how to do that? if you have any example so

Instead of printing whole dictionary via using print(method), you can just use print(method()['a']), this will just print the value associated with key 'a', whatever processing you want, you can put it in a single python script.

I have multiple Key: value in my map

From: Jay Ashokbhai Dangar @.>
Sent: 30 May 2022 11:14
To: jaydangar/chaquopy_flutter @.>
Cc: Vishal Kajale @.>; Mention @.>
Subject: Re: [jaydangar/chaquopy_flutter] How to return the output as a Map not as String ? (Issue #27)
Instead of printing whole dictionary via using print(method), you can just use print(method()['a']), this will just print the value associated with key 'a', whatever processing you want, you can put it in a single python script.
—
Reply to this email directly, view it on GitHubhttps://github.com/jaydangar/chaquopy_flutter/issues/27#issuecomment-1140723439, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AX2SJ77GJKSZUH2GT5MRXR3VMRIUVANCNFSM5XJI7Y5Q.
You are receiving this because you were mentioned.Message ID: @.***>

I have found Work around.
What I have done is

Replace single Quotes with double Quotes
Use json Decode to convert json String  to Map


some question

Hi, I'm trying to get Cassie's dynamics Model. I followed the steps, but I get the following error message ¿What could I be doing wrong?
错误使用 eval_math (第 46 行)
The evaluation of Mathematica expression failed.
出错 SymExpression (第 91 行)
eval_math([obj.s '=' obj.f ';']);
出错 eval_math_fun (第 53 行)
ret = SymExpression(fstr,varargin{:});
出错 RobotLinks/configureDynamics (第 44 行)
De{i} = eval_math_fun('InertiaMatrixByLink',[links(i),{obj.numState}]);
出错 model_gen (第 36 行)
cassie.configureDynamics('DelayCoriolisSet',false);

I have met the same question, and I solved it. You should change your Mathematica version<13.1
Every time when I run "frost_addpath" the first time, it will fail and say something about symbol "BlockDiagonalMatrix", both defined in frost and system builtin, (I'm not sure, because I don't familiar with Mathematica)see https://reference.wolfram.com/language/ref/BlockDiagonalMatrix.html. So I used version 13.0.1, the problem disappeard.


Index page - Top usernames by number of archives

Create a Leaderboard section that shows a list of top usernames by number of archives

The leaderboard should include links to the profile pages of the top users.


TextEdit/LineEdit: Handle numpad when NumLock is disabled

A small request from the list on #2982:

numpad arrow/home/end/pgup/pgdn should work as usual when numlock is off
-- grigi

What about PageUp, PageDown; Home and End?

Totally forgot those! :P
Should I rebase to merge both commits?

Should I rebase to merge both commits?

As you prefer, IMO it's fine like that two as both commits can work fine independently from one another, but it wouldn't hurt to have them merged either. Your call :)

LineEdit can also support Home and End: https://github.com/neikeq/godot/blob/a825cfd93288b4418f445cb1d8bfb4d4d2857f44/scene/gui/line_edit.cpp#L288
If you do a third commit, then yeah you can merge them ;)

Right... I must have my head in the clouds :)

Looks good.


Update pi_2_dht_read.c

Hi, I've been testing some changes to this code base to resolve some reliability issues with an AM2302 on a Raspberry Pi 2.  I was having to sample 40 times and take the median to get a value that was mostly reliable.  This was much better on a short cable run than a long cable run.  After making the adjustments below I now get reliable values every time (although I'm still sampling 11 times just in case I have not seen an anomaly yet).  The first change may not be necessary, but I've left it in there.
These changes significantly improve the reliability of reading temperature and humidity settings on a Raspberry Pi 2.
Resolved: Incorrect readings caused by premature detection of DHT pull low.
Resolved: Improved reliability on longer cables - this code is now reliable over a 10m cat5e cable @ 3.3V.

Same problem here.
Need i to compile .c file or is the modify enough?
Thank you.

hi this repo is going to be deprecated soon, please try
https://github.com/adafruit/Adafruit_CircuitPython_DHT
which will replace this library


Update pubspec.yaml

This is required to work with Dart SDK version is 2.12.2.
Else the package cant build in Flutter2

Dart and packages were upgraded when moving to current version of the library. Thanks for your help!


Documentation Updates

Dreate docs folder for language-specific documentation.
Include separate README for builtin functions.
Update index page to point to new location of language specs.

Commits:

28f32505509bc558e9b499e24a4a4b8a53120383
c57bb6921852f5a819662232623f1358e2223089
544cc08e7658255a84b0b6b4ad5902a5be2dcaea


#245 -- Change the implementation of getAllUserPreferences()

Addresses #245

new interface = { userID: ID; lastUpdateDate: Date; preferences: UserPreferences }
now this implementation return a Promise<Array<{ userID: ID; lastUpdateDate: Date; preferences: UserPreferences }>>
created tests to arrive if it contains these properties

Closed in favor of #247


add @mvinyard as author

Hey @mvinyard ! I previously missed your contributions to the task since the task got renamed at some point.
@lazappi and I have been making improvements to the dimensionality reduction task over time, and are prepping a new release for this task.
@mvinyard Would you like to be involved in the review of the updated results?

TODO: rerender readme

Already merging this, but feel free to follow up on this @mvinyard

Hi @rcannood, I appreciate the inclusion. I don't have a ton of bandwidth right now to dedicate to open-problems, so I cannot promise any significant future contributions to review.


/apps/legacy-api - HealthCheck & via Actuator

What would you like to be added:
_actuator/probes/... health check. We can follow the ocean-api design towards how we can implement the health-check. Should also import apps/ocean-api files to keep it DRY.
Why is this needed:
To take down legacy-api nodes when they end up in a bad state.
/triage accepted
/assign @eli-lim
/area apps
/priority important-soon

Following this design: https://github.com/DeFiCh/jellyfish/blob/main/apps/ocean-api/src/controllers/ActuatorController.ts

Update: simply importing the files from apps/ocean-api will cause nest to fail some dependency look-ups. Trying to find a way to share the ActuatorModule between the two (possibly more next time) apps

We will need something like jellyfish-api-template. All of our API going to need this implemented, I am copy pasting all those modules in the new rich-list-api at the moment and leaving them as it is and wait for the solution here.

We will need something like jellyfish-api-template. All of our API going to need this implemented, I am copy pasting all those modules in the new rich-list-api at the moment and leaving them as it is and wait for the solution here.

@eli-lim worked on https://github.com/DeFiCh/jellyfish/pull/1078


No command-line interface available

I would be nice to have a command-line interface to this module:
./node_modules/.bin/gjslint [OPTIONS...]

To do that, it would need to add the following in package.json:
  "bin": {
    "gjslint': "./bin/gjslint"
  },

(See jshint's package.json#L22 as an example)
The script bin/gjslint would call gjslint(...) with the given command-line arguments. See jshint's bin/jshint and jshint's src/cli.js as examples.

I'd be a nice feature. Can you make a PR?


Displaying preset value

The editor requires the user to click within the editor for the preset content to appear

Works as expected for me.


Test PR

Purpose
This is a test PR to check banner change.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.Tharika Madurapperuma seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.

Closing this PR as the issue is fixed now.


⬆ spell-check 0.74.2

Requirements for Contributing a Bug Fix

Fill out the template below. Any pull request that does not include enough information to be reviewed in a timely manner may be closed at the maintainers' discretion.
The pull request must only fix an existing bug. To contribute other changes, you must use a different template. You can see all templates at https://github.com/atom/atom/tree/master/.github/PULL_REQUEST_TEMPLATE.
The pull request must update the test suite to demonstrate the changed functionality. For guidance, please see https://flight-manual.atom.io/hacking-atom/sections/writing-specs/.
After you create the pull request, all status checks must be pass before a maintainer reviews your contribution. For more details, please see https://github.com/atom/atom/tree/master/CONTRIBUTING.md#pull-requests.

Identify the Bug
https://github.com/atom/spell-check/issues/210
Description of the Change
Update spell-check to v0.74.2.
Alternate Designs
N/A.
Possible Drawbacks
A bug was introduced in https://github.com/atom/spell-check/pull/271 that breaks something else.
Verification Process
Atom was launched in development mode with the changes applied verifying that the fix worked.

Hey @Arcanemagus, can you also update this line in package.json?  The tarball line gets updated automatically once you run script/bootstrap after changing the package version in the packageDependencies section so you'll never have to touch that one manually.

ในวันที่ พ. 26 ก.ย. 2018 23:51 David Wilson<EMAIL_ADDRESS>เขียนว่า:

Hey @Arcanemagus https://github.com/Arcanemagus, can you also update this
line
https://github.com/atom/atom/blob/f25e61adb530feb4ff4557f0c1797e09ce121df1/package.json#L221
in package.json? The tarball line gets updated automatically once you run
script/bootstrap after changing the package version in the
packageDependencies section so you'll never have to touch that one
manually.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/atom/atom/pull/18107#issuecomment-424789531, or mute
the thread
https://github.com/notifications/unsubscribe-auth/Apj8c5lOdbKYDxclasFglzxtAoWA3L5Jks5ue7CLgaJpZM4W5i3K
.

Updated, ran ./script/bootstrap again just to make sure that didn't change anything else 😉.

Thanks a lot!  Once the CI goes green again I'll merge it ✨


universal-cookie path param does not add cookie if page navigated to via client side nav

What's up
Hey all, we are seeing the cookie not properly added to the browser if you redirect to the path via client side navigation.
Example: cookie params
{
    domain: CONFIG.cookieDomain,
    expires: null,
    path: `/super-secret-page`,
    sameSite: 'lax',
    secure: true,
}

Repro:

Land on /regular-page, set a cookie on click of a button with path: '/'
client side navigate to /super-secret-page
set another cookie on click of a button with path set as above
open cookies in dev tools

Expected:

the new cookie would appear

Actual:

the new cookie isn't there

Questions
Is this behavior by design? If so, would you provide more information around how path is compared?

This is happening to me too. I had to redirect the full page with window.location.href= to make sure the cookie is being saved in the path I need.

This should be fixed in v6, please re-open if it's not the case.


Securing your Python application from hackers

Article checklist
ATTENTION: In your PR - add the ISSUE (topic form) # that this PR corresponds to. You can do this by also adding a comment that states "This fixes #(enter your Idea Content Form #)" to link and close your Idea Content (issue) when the PR is merged.
NOTE: (Please ensure that you have only one open issue + linked pull request at a time. This will ensure that we complete the article in a timely manner from inception to publishing.)
If you have not already please go over our Resources Page for more tips and tricks on how to improve our overall technical writing.
Preliminary Checks - Formatting and Structure

[x] Does your article follow any of the suggested structure formats? - see example formats
[x] Is your article properly formatted in Markdown? - see Github Markdown guide
[x] Have you used the correct folder and file structure? - see Contribution Guidelines
[x] Is your article over 750 words? - Tool: Wordcounter.net
[x] Have you used our preferred heading structure? - H3 (###) and up
[x] Does your article provide enough value and detail about your topic? - Articles should be clear, accurate, and fully explained.
[x] Can your article be understood by beginners? - Assume the audience is smart but has no prior exposure to the common terminology in your article.
[x] Have you included a hero image that is 600x400 pixels and have the copyright to use it? - Tools: Pixlr Image Editor for resizing and Unsplash for Creative Commons images
[x] In your PR - add a comment that states "This fixes #(enter your Idea Content Form #)" to link and close your Idea Content (issue) when the PR is merged.

Grammar & plagiarism checks

[x] Have you spellchecked and grammar checked your article? - Tools: Write&Improve - ProWritingAid -  Grammarly
[x] Please place your article through a 3rd party plagiarism checker? We suggest using Quetext, this tool is free and has a daily limit. We typically accept articles with 10% or less.
[x] Have you checked your article for readability? - Tool: Hemmingway
[x] Have you added sources for quotes and images that aren't yours?

Technical checks

[x] Are your code snippets properly formatted for syntax highlighting - see Syntax guide

Contribution guidelines
For first-time contributors and for more details, see Contributing Guidelines
This fixes #2133

Thank you v much for your patience as we worked through our Repo re-structuring process.
The purpose behind this restructure - was to better allow us to handle more volume and allow students to publish more articles and get more involved with the community (with open-source contribution and suggestions).
With most of the changes coming to completion - we will be sure to get your article published asap.
Thank you again - and we are looking forward to more amazing articles.

plagiarism check done

@ahmadmardeni1 great work - nice article -very easy to follow along.
I deleted the two gifs - I deleted the 1st one to help avoid page loading speed.
I was going to leave the 2nd one - but noticed it linked out - best practice is to include the file in the article like an image.
Just in case it gets deleted from the external servers - the gif would still remain in your article.
Just FYI.
Please go over any minor edit I may have made just to see any small changes.
As we are going to start empowering Peer reviewers to merge (publish) articles themselves -  this will require that articles are 100% error free - including small formatting issues and other things of that nature.
cc @zolomohan
More to be shared in the PR channel soon. 🚀


refactor: automatically extract fields from URL

the function getContextFromUrl should be generalized to extract multiple fields from the url (query params), such as tags

closed via https://github.com/nya1/scatola/pull/13


Typo in 'Publishing with Gradle' section (README.md)

Awesome that this section is here!!!
In the code snippet all occurrences of version are spelt verson
configurations {
    apiElements {
        outgoing {
            capability("${project.group}:${project.name}:${project.verson}") // keep default capability 'org.ow2.asm:asm'
            capability("asm:asm:${project.verson}")                          // add 'asm:asm'
        }
    }
    runtimeElements {
        outgoing {
            capability("${project.group}:${project.name}:${project.verson}") // keep default capability 'org.ow2.asm:asm'
            capability("asm:asm:${project.verson}")                          // add 'asm:asm'
        }
    }
}```

Thanks for pointing this out @cloudshiftchris. Time to get these snippets tested somehow...
Fixed now!


Consider adopting Prettier

We should consider adopting Prettier for formatting JSDoc's source code.

On second thought, I'd prefer not to use Prettier, as much as I like the idea of doing so. The big downside is that using Prettier would make git blame much less useful, because of the one huge reformatting commit that would touch most of JSDoc's source code. I don't think that tradeoff is worth making.


Only load url-quote-magic if it is available.

Partially fixes #3614 by checking to see if url-quote-magic exists before loading it.

+1, works with 2fb3265 @webmaster128

Sorry for dup.
I closed my PR.
https://github.com/robbyrussell/oh-my-zsh/pull/4038


Add support for pyarrow-backed data types in find_column_type method (Generated by Ana - AI SDE)

This patch modifies the find_column_type method to include support for various pyarrow-backed data types, ensuring greater compatibility with pandas.
This patch was generated by Ana - AI SDE, an AI-powered software development assistant.
It's a fix for Issue 282

Thanks for the PR, at this time the SDE contributions don't meet our contributor guidelines. Closing this for as of now.


Flutter 1.17.0.dev.3.2 cherrypicks

engine CPs:

update 1.17 engine to Dart 2.8.0-dev.20.8 · Issue #54897 · flutter/flutter

flutter/flutter CPs:

Git warning of a missing default when upgrading Flutter · Issue #54983 · flutter/flutter
Multiple --dart-define cause crash during android apk build · Issue #54825 · flutter/flutter

will need to roll a new engine with a previous versioning scheme for dart on this one


Agent response format

What does this PR do?
Add response format for agents structured output.

[ ] Using structured output for agents (interior_design app as an example) (#issue) https://github.com/meta-llama/llama-stack-apps/issues/122

Test Plan
E2E test plan with llama-stack-apps interior_design
Please describe:
Test ran:

provide instructions so it can be reproduced.
Start your distro:
llama stack run  llama_stack/templates/fireworks/run.yaml --env FIREWORKS_API_KEY=<API_KEY>

Run api test:
PYTHONPATH=. python examples/interior_design_assistant/api.py localhost 5000 examples/interior_design_assistant/resources/documents/ examples/interior_design_assistant/resources/images/fireplaces
Sources
Results:
https://github.com/meta-llama/llama-stack-client-python/pull/72
Before submitting

[ ] This PR fixes a typo or improves the docs (you can dismiss the other checks if that's the case).
[ ] Ran pre-commit to handle lint / formatting issues.
[x] Read the contributor guideline,
Pull Request section?
[ ] Updated relevant documentation.
[ ] Wrote necessary unit or integration tests.

Thanks! Could you add an integration test for this in https://github.com/meta-llama/llama-stack/blob/main/tests/client-sdk/agents/test_agents.py?


Use derived types to define trajectories

Currently, the x,y,z and i,j,k positions are stored in the 2D arrays trj (doubles) and nrj (integers).
I propose to use derived types, i.e. define a type such as
 type trajectory      integer :: i,j,k,n      real*8  :: x,y,z,tt,t0,subvol      real*4  :: tem, sal, rho      integer :: nk   end type trajectory
and then allocate
  type(trajectory), allocatable, dimension(:) :: trajectories   allocate( trajectories(ntracmax) )
The positions of a trajectory can thus be accessed as
x = trajectories(ntrac)%x y = trajectories(ntrac)%y z = trajectories(ntrac)%z
and similarly for i,j,k etc.
This would make the code more readable, make sure we only allocate the memory we actually need, and make it more flexible if we want to add or remove a property of trajectories, e.g. chlorophyll concentration, pCO2, diabatic heating or something.

This is now included in branch "standard_configs"


Cloud IoT Manager example does not work in py2 or py3

Hi,
I am trying to follow this example:
https://github.com/googleapis/python-iot/blob/main/samples/api-client/manager/manager.py
I am running this in cloud shell:
python manager.py --project_id=MY-PORJECT --cloud_region=us-central1 --service_account_json=foo.json list-registries

 File "manager.py", line 296
    parent = f"projects/{project_id}/locations/{cloud_region}"
 SyntaxError: invalid syntax

and with python3:
Traceback (most recent call last):
  File "manager.py", line 38, in <module>
    from google.cloud import iot_v1
ImportError: cannot import name 'iot_v1' from 'google.cloud' (unknown location)

@codepriestz Please could you take a look?

Friendly ping :)


[PDI-16488] Spoon cannot monitor nor execute jobs and transformations…

… on the Pentaho server or Carte with SSL
@pamval @mbatchelor @bmorrise @pentaho-lmartins

Build Completed
:fire: This pull request has some issues. It would be preferable to fix them in order for it to be just perfect. See below for more details. Some links are also available below for further assistance in addressing those issues.
Build Commands
mvn -B -fn -f 'pom.xml' -pl 'ui' -P '!assemblies' -Dsurefire.runOrder=alphabetical -Daudit -amd clean install

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

Changed files
  ui/src/main/java/org/pentaho/di/ui/cluster/dialog/SlaveServerDialog.java
  ui/src/main/java/org/pentaho/di/ui/core/dialog/ErrorDialog.java
  ui/src/main/java/org/pentaho/di/ui/spoon/SpoonSlave.java

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

:warning: Coverage Changes (click to expand)
org.pentaho.di.trans.steps.salesforceinsert.SalesforceInsertMeta

Instruction Change: -0.1% :small_red_triangle_down:

org.pentaho.di.trans.steps.salesforceupsert.SalesforceUpsertMeta

Instruction Change: -0.1% :small_red_triangle_down:

org.pentaho.di.ui.core.dialog.ErrorDialog

Branch Change: -0.2% :small_red_triangle_down:
Complexity Change: -0.4% :small_red_triangle_down:
Instruction Change: -0.1% :small_red_triangle_down:

Pedro, can you write some test cases to confirm the new behavior of the exception handling.

Build Completed
:fire: This pull request has some issues. It would be preferable to fix them in order for it to be just perfect. See below for more details. Some links are also available below for further assistance in addressing those issues.
Build Commands
mvn -B -fn -f 'pom.xml' -pl 'ui' -P '!assemblies' -Dsurefire.runOrder=alphabetical -Daudit -amd clean install

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

Changed files
  ui/src/main/java/org/pentaho/di/ui/cluster/dialog/SlaveServerDialog.java
  ui/src/main/java/org/pentaho/di/ui/core/dialog/ErrorDialog.java
  ui/src/main/java/org/pentaho/di/ui/spoon/SpoonSlave.java
  ui/src/test/java/org/pentaho/di/ui/core/dialog/ErrorDialogTest.java
  ui/src/test/java/org/pentaho/di/ui/spoon/SpoonSlaveTest.java

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

:warning: Coverage Changes (click to expand)
org.pentaho.di.trans.steps.gpload.GPLoadMeta

Branch Change: -0.9% :small_red_triangle_down:
Complexity Change: -0.8% :small_red_triangle_down:
Instruction Change: -0.2% :small_red_triangle_down:

Build Completed
:fire: This pull request has some issues. It would be preferable to fix them in order for it to be just perfect. See below for more details. Some links are also available below for further assistance in addressing those issues.
Build Commands
mvn -B -fn -f 'pom.xml' -pl 'ui' -P '!assemblies' -Dsurefire.runOrder=alphabetical -Daudit -amd clean install

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

Changed files
  ui/src/main/java/org/pentaho/di/ui/cluster/dialog/SlaveServerDialog.java
  ui/src/main/java/org/pentaho/di/ui/core/dialog/ErrorDialog.java
  ui/src/main/java/org/pentaho/di/ui/spoon/SpoonSlave.java
  ui/src/test/java/org/pentaho/di/ui/core/dialog/ErrorDialogTest.java
  ui/src/test/java/org/pentaho/di/ui/spoon/SpoonSlaveTest.java

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

:warning: Coverage Changes (click to expand)
org.pentaho.di.trans.steps.salesforceinsert.SalesforceInsertMeta

Instruction Change: -0.1% :small_red_triangle_down:

org.pentaho.di.trans.steps.salesforceupsert.SalesforceUpsertMeta

Instruction Change: -0.1% :small_red_triangle_down:

Hi Luis, test cases included. I've also done some refactoring for better testing the new behavior of the exception handling.
PR updated.


feat(tests): add more tests for codegen with multipart body

PR Description
This PR adds tests for requests with a multipart body.
@animator @ashitaprasad Do I add more tests for post requests (e.g when request params and request headers are disabled).
Also, Currently, I've just added the request models. Do I add them to individual codegen tests as well or I should do this in a separate PR after this is merged? Currently, This PR doesn't add any functionality to the actual tests but rather just code that other tests will use.
Related Issues

Related Issue #234

Checklist

[x] I have gone through the contributing guide
[x] I have run the tests (flutter test) and all tests are passing

Added/updated tests?
We encourage you to add relevant test cases.

[x] Yes
[ ] No, and this is why: please replace this line with details on why tests have not been included
[ ] I need help with writing tests

Thanks for adding the request models. I have made some changes to use our APIs. I think these models should cover multipart test cases for now.


LSP position back to offset

I have such a requirement, I need to know what AST node is currently selected by the mouse.
My AST uses offset to store range information.
Because binary search based on offset is easier to write.
The error reporting library (ariadne) I use is also offset based.
So I want an method like the following:
fn lsp_pos_to_offset(&self, lsp_pos: &lsp_types::Position) -> Option<usize>

There's a private function that converts Pos to byte offset: https://github.com/artempyanykh/zeta-note/blob/main/lsp-document/src/lib.rs#L390-L394
I guess it's OK to expose it as a part of TextMap API.

I've move lsp-document into a separate repo. @oovm feel free to open an issue there if this is something you're still interested in.

thanks


[CI] Quick fix

Description
This PR makes several changes:

Always trigger sphinx build by default but deploy to Github page only when pushing to main branch. Since API changes may also break the docs, it would be better to detect the error during the PR.
Update README instructions for reproducing benchmark results in a clean environment.
Add a deepspeed testing script for GPT2. As it requires to reset the model config, it cannot be directly called by bench_single_node.py.
Small fix to print all the changed files in git-black.

Checklist

[x] PR's title starts with a category (e.g. [Bugfix], [Model], [Tutorial], etc)
[x] Changes are complete (i.e. I finished coding on this PR)

Thanks @chhzh123


python executable no longer selectable

Issue description
I upgraded to pipenv version 11.9.0 yesterday.  I'm running on Ubuntu 20.04 with multiple versions of Python installed.
After the upgrade, I can no longer get pipenv to use the specific python path I want it to.
export PIPENV_PYTHON=/usr/bin/python3 doesn't help.
pipenv install --python=/usr/bin/python3 doesn't help.
/usr/bin/python3 /usr/bin/pipenv install doesn't help.
In every case, it reverts to using /usr/bin/python which I do not want it to use.
This was not a problem until I upgraded.
Expected result
I should be able to specify the path to the python executable on my system that I want pipenv to use.
Actual result
Using /usr/bin/python3 (3.8.5) to create virtualenv…
Warning: Your Pipfile requires python_version 3.8, but you are using 2.7.18 (/bin/python).

Steps to replicate
Upgrade to the latest version and watch it break.

Please run $ pipenv --support, and paste the results here. Don't put backticks (`) around it! The output already contains Markdown formatting.
$ pipenv --support | xclip

Error: no such option: --support

Version 11.9 is several years out of date, please install pipenv with pip instead

Ah.  I did install it with pip.  However I still had an old system version running which was getting in the way of the latest version for some reason.  (I'd previously been using a pip installed one as well, and it did not have problems).  Removing the legacy system one seems to have solved this.  Thanks!


ESLINT errors not solvable without refactoring

Please lint the javascript code and make sure travis passes when linting. Example of issues are:

@sanjeevghimire what do you think of https://github.com/IBM/watson-voice-bot/pull/6 ?

#6 fixes the spacing issue but not object referenced issues. if we hook eslint with travis, it will complain.

@sanjeevghimire i created PR #8 too, just poking around with eslint.
Hi @ptrikkur can you look at #6 and #8

@stevemart I merged both #6 and #8 . I took the resulting script.js and linted it. I got one error.
1:5  error  Parsing error: Unexpected token conversationContext
✖ 1 problem (1 error, 0 warnings)

@ptrikkur I have fixed the linting issues. I have also removed few unused functions, if you can review it as well that would be great. #9

i think this is fixed. closing it.


Dynamic Database Memory Usage

Make it so all guild DBs are not in memory all the time, but rather dynamically loaded and unloaded from memory, as to lower memory usage

Done.

Not relevant anymore.


[D2IQ-64191] Cassandra enable Prometheus exporter for the metrics

currently   "cassandra-exporter-jar" is pointing to s3, once everything is fine I will upload same to downloads.mesosphere.com

@viivek46 and @farhan5900 what is the status of this?
/cc @kaiwalyajoshi

@DominikDary I think we need to re-evaluate the approach taken here.
The JMX-Prometheus exporter plugin used here is based on this repository which has status of Beta https://github.com/instaclustr/cassandra-exporter
This is not the same as the plugin used by HDFS which is https://github.com/prometheus/jmx_exporter
and is official list supported by Prometheus.
For long term support, I'd prefer to use jmx_exporter and not have many different versions of metrics exporter plugins for each data-service we support unless there is a very strong reason to break from it.
I'm looking into jmx_exporter now


Loader::lookup/lookup_complete/lookup_with_args return Option

Loader::lookup/lookup_complete/lookup_with_args now return
Option<String> so the user can control what happens when a lookup
returns nothing.
Ref: https://github.com/XAMPPRocky/fluent-templates/issues/32

Thank you for your PR, and congrats on your first contribution! 🎉


Location template for page

Is your feature request related to a problem? Please describe.
Hi, i have 1 request, i need create locale template for difference location, as an example i have en, fe and cn locations user in my app and in home page i need have some difference block, but the whole page is the same, difference 2-3 block in every location. Are there any solutions for this now? v-if with locale($i18n) too hard to maintain we have 20 locale now. Custom routing create for me 1500-1600 routes. Maybe you can give advice something other?

We are approaching the Nuxt 2 EOL date (June 30, 2024) - see this article for more information. I'm closing this issue as it's marked as a Nuxt 2 related enhancement and it's not a critical issue.
That doesn't mean it might not be relevant for Nuxt 3. If it is, please feel free to open a new issue (or just comment, and I can reopen it). 🙏
Thank you for your understanding and for your contribution to Nuxt! 🎉


Set ring color

How do I change the ring color without affecting the text label color? setForegroundColor() will change both the ring color and text label color

This is as intended. Please use a subclass or provide a pull request.


Size attr is not working

In my case, using the box-icon tag and adding attr size, I tried to fill it with 100px but only the container grew, the icon didn't change at all, both using defaults such as xs, lg, md and others, it's the same. I checked the inspect path, it didn't change. basically the icon doesn't change, only the wrapping container gets bigger

Same issue as #373


mapping time Analyzer is getting removed when defining for analyzer on property directly

I have mixed up analyzers as below.
Number - directly with property
Name- with mapping.
I have recognized that Name is not analyzed with my custom analyzers at all. just standard although I am only using standard for Number property.
When I remove the SearchAnalyzer="standard" from number propery, my custom analyzers for Name filed is take again.
Is it by design like that or an issue?
I am using 2.3.1 version for Nest and Elasticsearch.net
 public class Product
  {

    [Number(NumberType.Integer, Name = "id", Index = NonStringIndexOption.No, Store = true)]
    public int ProductId { get; set; }

    [StringLength(255)]
    [String(Index = FieldIndexOption.Analyzed, Store = true)]
    public string Name { get; set; }

    [StringLength(64)]
    [String(Index = FieldIndexOption.Analyzed, SearchAnalyzer="standard", Store = false)]
    public string Number{ get; set; }

}

var mapResult = Client.Map<Index.Product>(c => c.AutoMap().Properties(props => props
  .String(s => s
   .Name(p => p.Name).Fields(fs => fs.String(ss => ss.Name(aa => aa.Name.Suffix("raw")).Analyzer("my_delimiter")))
                  .Analyzer("my_analyzer")
                  .SearchAnalyzer("standard"))

Hey @EmilAlipiev I'm not able to reproduce this, or I'm not understanding the issue entirely.  Here's the JSON that's produced using your snippet above:
{
  "properties": {
    "id": {
      "type": "integer",
      "store": true,
      "index": "no"
    },
    "name": {
      "type": "string",
      "fields": {
        "raw": {
          "type": "string",
          "analyzer": "my_delimiter"
        }
      },
      "analyzer": "my_analyzer",
      "search_analyzer": "standard"
    },
    "number": {
      "type": "string",
      "store": false,
      "index": "analyzed",
      "search_analyzer": "standard"
    }
  }
}

Is this not your expectation?

Closing.  Feel free to re-open @EmilAlipiev if this is still an issue for you.


Set default GPU mem to 16 for all raspberry pi-based devices

Change-type: minor
Signed-off-by: Stevche Radevski<EMAIL_ADDRESS>

Can one of the admins verify this patch?


Replica connections keep growing into the thousands even though connection pool limit is 64

Expected behavior
Connections pool to be capped at 64.
We only have 10 k8s pods. So that should be max 640 connections per pod per redis node.
Actual behavior
We have thousands of connections to each replica node and they keep growing.
This only happened when I switched to using Redisson for reading, instead of Lettuce.
You can see the switch over in the graph:

We are using Redisson for batch writing as well and that is taking much more load than the reads but its connections to the master nodes is correctly staying below 640, currently around 490.
Another graph showing new connections being made all the time.

Steps to reproduce or test case
override suspend fun read(trx: String, timeout: Long?): List<Placeholder> {
    val resolvedTimeout = timeout ?: blockTime // 1000
    val stream = redissonClient.reactive().getStream<String, String>(trx, StringCodec.INSTANCE)

    return stream
        .read(
            StreamReadArgs
                .greaterThan(StreamMessageId(0, 0))
                .timeout(Duration.ofMillis(resolvedTimeout))
        )
        .awaitSingle()
        .map { response ->
            response.value.map { Placeholder(it.key, it.value) }
        }
        .flatten()
}

Redis version
7.0.5
Redisson version
3.19.1
Redisson configuration
val config = Config()
val clusterServersConfig = config.useClusterServers()
clusterServersConfig.nodeAddresses = listOf("rediss://$host:$port")
clusterServersConfig.password = password
clusterServersConfig.isKeepAlive = false
clusterServersConfig.scanInterval = 30000

return Redisson.create(config)

There is no awaitSingle() method in Mono interface

Unable to reproduce with the test below.
        RedisRunner master1 = new RedisRunner().port(6890).randomDir().nosave();
        RedisRunner master2 = new RedisRunner().port(6891).randomDir().nosave();
        RedisRunner master3 = new RedisRunner().port(6892).randomDir().nosave();
        RedisRunner slave1 = new RedisRunner().port(6900).randomDir().nosave();
        RedisRunner slave2 = new RedisRunner().port(6901).randomDir().nosave();
        RedisRunner slave3 = new RedisRunner().port(6902).randomDir().nosave();
        RedisRunner slave4 = new RedisRunner().port(6903).randomDir().nosave();

        ClusterRunner clusterRunner = new ClusterRunner()
                .addNode(master1, slave1, slave4)
                .addNode(master2, slave2)
                .addNode(master3, slave3);
        ClusterProcesses process = clusterRunner.run();

        Thread.sleep(5000);

        Config config = new Config();
        config.useClusterServers()
                .setLoadBalancer(new RandomLoadBalancer())
                .addNodeAddress(process.getNodes().stream().findAny().get().getRedisServerAddressAndPort());
        RedissonClient redisson = Redisson.create(config);

        RStream<String, String> stream = redisson.getStream("test", StringCodec.INSTANCE);

        for (int i = 0; i < 10000; i++) {
            Map<StreamMessageId, Map<String, String>> t = stream.read(
                    StreamReadArgs
                            .greaterThan(new StreamMessageId(0, 0))
                            .timeout(Duration.ofMillis(1000))
            );
        }

Is it a replicated or cluster AWS Elasticache setup?

Cluster AWS Elasticache setup. With 3 shards, each with 2 replicas

The new connections seem correlated with traffic

Can you provide some test case to reproduce it?

I'm not sure I can reproduce it, seems to only be an issue on our production environment, which seems to point to high load being the cause?
Does this connection pattern look correct for this kind of request pattern? It seems correct to me, the current connections always drop back down after a spike. This is in our test environment.

I will ask our database team if they can provide the memory dump

I just redeployed the app, looks like num connections has capped out and no more connections can be made to the replicas. Unable to connect to Redis server. The writing connected to master nodes is working fine.

Does this connection pattern look correct for this kind of request pattern? It seems correct to me, the current connections always drop back down after a spike.

It's an issue. Can you share the logs with trace logging level for org.redisson package?

As for a test case the only difference i can see from my code and your test code is that you are not using Kotlin coroutines. Not sure if that could cause an issue?

It shouldn't.

I got the logs from my local connecting to test env and i added these config settings:
clusterServersConfig.masterConnectionMinimumIdleSize = 1
clusterServersConfig.slaveConnectionMinimumIdleSize = 1
clusterServersConfig.subscriptionConnectionMinimumIdleSize = 1

I then fired a few requests. Do you want me to get logs from test env directly?
redisson.log

"Connection created" in your logs is expected if you set IdleSize = 1 but overall connections shouldn't exceed the limit. Can you provide the logs with "Connection created" amount exceed  the limit?

In production the errors are
exception: { [[-]](https://splunk.prod.egmonitoring.expedia.com/en-US/app/search/search?q=search%20index%3D%22eg-runtime-rcp%22%20kube_namespace%3D%22experience-manager%22%20sourcetype%3D%22kube%3Acontainer%3Aexperience-placeholders-api%22%20region%3D%22us-west-2%22%20%7C%20spath%20level%20%7C%20search%20level%3DERROR%7C%20spath%20logger_name%20%7C%20search%20logger_name%3D%22org.redisson*%22&earliest=-2h%40m&latest=now&display.page.search.mode=smart&dispatch.sample_ratio=1&display.events.timelineEarliestTime=1674219180&display.events.timelineLatestTime=1674219240&sid=1674222428.7363_74A8010F-0152-4D34-A2CF-20634C54649C#)
     exception_class: java.util.concurrent.CompletionException
     exception_message: org.redisson.client.RedisConnectionException: Unable to connect to Redis server: <IP_ADDRESS>/<IP_ADDRESS>:6379
     stacktrace: java.util.concurrent.CompletionException: org.redisson.client.RedisConnectionException: Unable to connect to Redis server: <IP_ADDRESS>/<IP_ADDRESS>:6379
	at java.base/java.util.concurrent.CompletableFuture.encodeThrowable(Unknown Source)
	at java.base/java.util.concurrent.CompletableFuture.completeThrowable(Unknown Source)
	at java.base/java.util.concurrent.CompletableFuture$BiRelay.tryFire(Unknown Source)
	at java.base/java.util.concurrent.CompletableFuture.postComplete(Unknown Source)
	at java.base/java.util.concurrent.CompletableFuture.completeExceptionally(Unknown Source)
	at org.redisson.connection.pool.ConnectionPool.lambda$createConnection$0(ConnectionPool.java:155)
	at java.base/java.util.concurrent.CompletableFuture.uniWhenComplete(Unknown Source)
	at java.base/java.util.concurrent.CompletableFuture$UniWhenComplete.tryFire(Unknown Source)
	at java.base/java.util.concurrent.CompletableFuture.postComplete(Unknown Source)
	at java.base/java.util.concurrent.CompletableFuture.completeExceptionally(Unknown Source)
	at org.redisson.connection.pool.ConnectionPool.promiseFailure(ConnectionPool.java:318)
	at org.redisson.connection.pool.ConnectionPool.lambda$createConnection$6(ConnectionPool.java:277)
	at java.base/java.util.concurrent.CompletableFuture.uniWhenComplete(Unknown Source)
	at java.base/java.util.concurrent.CompletableFuture$UniWhenComplete.tryFire(Unknown Source)
	at java.base/java.util.concurrent.CompletableFuture.postComplete(Unknown Source)
	at java.base/java.util.concurrent.CompletableFuture.completeExceptionally(Unknown Source)
	at org.redisson.client.RedisClient$1$1.run(RedisClient.java:241)
	at io.netty.util.concurrent.PromiseTask.runTask(PromiseTask.java:98)
	at io.netty.util.concurrent.PromiseTask.run(PromiseTask.java:106)
	at io.netty.util.concurrent.AbstractEventExecutor.runTask(AbstractEventExecutor.java:174)
	at io.netty.util.concurrent.AbstractEventExecutor.safeExecute(AbstractEventExecutor.java:167)
	at io.netty.util.concurrent.SingleThreadEventExecutor.runAllTasks(SingleThreadEventExecutor.java:470)
	at io.netty.channel.nio.NioEventLoop.run(NioEventLoop.java:569)
	at io.netty.util.concurrent.SingleThreadEventExecutor$4.run(SingleThreadEventExecutor.java:997)
	at io.netty.util.internal.ThreadExecutorMap$2.run(ThreadExecutorMap.java:74)
	at io.netty.util.concurrent.FastThreadLocalRunnable.run(FastThreadLocalRunnable.java:30)
	at java.base/java.lang.Thread.run(Unknown Source)
Caused by: org.redisson.client.RedisConnectionException: Unable to connect to Redis server: <IP_ADDRESS>/<IP_ADDRESS>:6379
	at org.redisson.connection.pool.ConnectionPool.lambda$createConnection$0(ConnectionPool.java:154)
	... 21 more
Caused by: java.util.concurrent.CompletionException: java.nio.channels.ClosedChannelException
	at java.base/java.util.concurrent.CompletableFuture.encodeRelay(Unknown Source)
	at java.base/java.util.concurrent.CompletableFuture.completeRelay(Unknown Source)
	at java.base/java.util.concurrent.CompletableFuture$UniRelay.tryFire(Unknown Source)
	... 13 more
Caused by: java.nio.channels.ClosedChannelException
	at io.netty.handler.ssl.SslHandler.channelInactive(SslHandler.java:1065)
	at io.netty.channel.AbstractChannelHandlerContext.invokeChannelInactive(AbstractChannelHandlerContext.java:305)
	at io.netty.channel.AbstractChannelHandlerContext.invokeChannelInactive(AbstractChannelHandlerContext.java:281)
	at io.netty.channel.AbstractChannelHandlerContext.fireChannelInactive(AbstractChannelHandlerContext.java:274)
	at io.netty.channel.DefaultChannelPipeline$HeadContext.channelInactive(DefaultChannelPipeline.java:1405)
	at io.netty.channel.AbstractChannelHandlerContext.invokeChannelInactive(AbstractChannelHandlerContext.java:301)
	at io.netty.channel.AbstractChannelHandlerContext.invokeChannelInactive(AbstractChannelHandlerContext.java:281)
	at io.netty.channel.DefaultChannelPipeline.fireChannelInactive(DefaultChannelPipeline.java:901)
	at io.netty.channel.AbstractChannel$AbstractUnsafe$7.run(AbstractChannel.java:813)
	at io.netty.util.concurrent.PromiseTask.runTask(PromiseTask.java:98)
	at io.netty.util.concurrent.PromiseTask.run(PromiseTask.java:106)
	at io.netty.util.concurrent.AbstractEventExecutor.runTask(AbstractEventExecutor.java:174)
	at io.netty.util.concurrent.AbstractEventExecutor.safeExecute(AbstractEventExecutor.java:167)
	at io.netty.util.concurrent.SingleThreadEventExecutor.runAllTasks(SingleThreadEventExecutor.java:470)
	at io.netty.channel.nio.NioEventLoop.run(NioEventLoop.java:566)
	... 4 more

   }
   level: ERROR
   log_type: main
   logger_name: org.redisson.cluster.ClusterConnectionManager
   mdc: { [[+]](https://splunk.prod.egmonitoring.expedia.com/en-US/app/search/search?q=search%20index%3D%22eg-runtime-rcp%22%20kube_namespace%3D%22experience-manager%22%20sourcetype%3D%22kube%3Acontainer%3Aexperience-placeholders-api%22%20region%3D%22us-west-2%22%20%7C%20spath%20level%20%7C%20search%20level%3DERROR%7C%20spath%20logger_name%20%7C%20search%20logger_name%3D%22org.redisson*%22&earliest=-2h%40m&latest=now&display.page.search.mode=smart&dispatch.sample_ratio=1&display.events.timelineEarliestTime=1674219180&display.events.timelineLatestTime=1674219240&sid=1674222428.7363_74A8010F-0152-4D34-A2CF-20634C54649C#)
   }
   message: Can't add slave: rediss://<IP_ADDRESS>:6379
   source_host: experience-placeholders-api-us-west-2-master-template-676d8rpz2
   thread_name: redisson-netty-3-11

Can you try version attached on test environment with 8K+ connections spikes? log level is 'debug' for org.redisson.command package. Look for forceFastReconnectAsync executed for ...  messages
redisson-3.19.2-SNAPSHOT.jar.zip

Test doesn't have 8k connection spikes. Those are the requests coming into the app. The connections in test env peaked around 90 per node. Which is still higher than the default pool limit of 64.
With that info does test env still look like it is an issue?

Can you try version attached on test environment with 8K+ connections spikes? log level is 'debug' for org.redisson.command package. Look for forceFastReconnectAsync executed for ...  messages

So just to be clear I will start the application locally using this new version. The application will connect to the test cluster. I will use gatling to run some load on it.

I haven't been able to get anything to break on test environment, haven't seen any of those logs you asked for.
I think test environment is healthy. I will get our database team to re-start the cluster and report back.

Okay so we have re-started the cluster to wipe lost connections. I've also switched back to Lettuce for reading for now. We still have Redisson setup though and are using it for batch writing.
Now we are getting lots thousands of errors like this
exception_class: reactor.core.Exceptions.ErrorCallbackNotImplemented
     exception_message: org.redisson.client.RedisNodeNotFoundException: Node: NodeSource [slot=14663, addr=rediss://<IP_ADDRESS>:6379, redisClient=null, redirect=MOVED, entry=null] hasn't been discovered yet. Increase value of retryAttempts and/or retryInterval settings.
     stacktrace: reactor.core.Exceptions$ErrorCallbackNotImplemented: org.redisson.client.RedisNodeNotFoundException: Node: NodeSource [slot=14663, addr=rediss://<IP_ADDRESS>:6379, redisClient=null, redirect=MOVED, entry=null] hasn't been discovered yet. Increase value of retryAttempts and/or retryInterval settings.
Caused by: org.redisson.client.RedisNodeNotFoundException: Node: NodeSource [slot=14663, addr=rediss://<IP_ADDRESS>:6379, redisClient=null, redirect=MOVED, entry=null] hasn't been discovered yet. Increase value of retryAttempts and/or retryInterval settings.
	at org.redisson.connection.MasterSlaveConnectionManager.createNodeNotFoundException(MasterSlaveConnectionManager.java:566)
	at org.redisson.connection.MasterSlaveConnectionManager.connectionWriteOp(MasterSlaveConnectionManager.java:515)
	at org.redisson.command.RedisExecutor.getConnection(RedisExecutor.java:643)
	at org.redisson.command.RedisExecutor.execute(RedisExecutor.java:124)
	at org.redisson.command.RedisExecutor$1.run(RedisExecutor.java:298)
	at io.netty.util.HashedWheelTimer$HashedWheelTimeout.run(HashedWheelTimer.java:715)
	at io.netty.util.concurrent.ImmediateExecutor.execute(ImmediateExecutor.java:34)
	at io.netty.util.HashedWheelTimer$HashedWheelTimeout.expire(HashedWheelTimer.java:703)
	at io.netty.util.HashedWheelTimer$HashedWheelBucket.expireTimeouts(HashedWheelTimer.java:790)
	at io.netty.util.HashedWheelTimer$Worker.run(HashedWheelTimer.java:503)
	at io.netty.util.concurrent.FastThreadLocalRunnable.run(FastThreadLocalRunnable.java:30)
	at java.base/java.lang.Thread.run(Unknown Source)

Can you share the logs from application start moment till the appearance of this error?

Try to set dnsMonitoringInterval = -1 in production to resolve this issue. I think it's related to issue then a single AWS hostname resolves to a multiple slave node IPs. Please check that

Unable to reproduce it using AWS Elasticache Cluster with 3 shards and each has 2 replicas.
Below is the test.
        Config c = new Config();
        c.useClusterServers().addNodeAddress(args.length > 0 ? args[0] : "redis://<IP_ADDRESS>:6901");
        RedissonClient redisson = Redisson.create(c);

        ExecutorService e = Executors.newFixedThreadPool(128);
        for (int i = 0; i < 100000; i++) {
            int j = i;
            e.submit(() -> {
                try {
                    RStream<String, String> stream = redisson.getStream("test" + ThreadLocalRandom.current().nextLong(), StringCodec.INSTANCE);
                    Map<StreamMessageId, Map<String, String>> t = stream.read(
                            StreamReadArgs
                                    .greaterThan(new StreamMessageId(0, 0))
                                    .timeout(Duration.ofMillis(ThreadLocalRandom.current().nextInt(10, 2000)))
                    );
                    if (j % 10 == 0) {
                        log.info("index: " + j);
                    }
                } catch (Exception ex) {
                    log.error(ex.getMessage(), ex);
                }
            });
        }
        e.shutdown();
        assert e.awaitTermination(3, TimeUnit.HOURS);

Do you have any update?

Hi, thanks for following up. I'm not sure when this will get investigated again on our side. I've moved off the project for now but will make sure the new team follow up on this. Not a high priority for us as the app is working fine with Redisson for writes and Lettuce for reads, but It makes sense to push for a single library.

@Strydom
Can you share logs from production from application start moment till the first error appearance?

I have had the same issue as described here. I am using the Vertx Redis client so I don't know if my problem is relevant to this situation or not. You can find more info here: https://github.com/vert-x3/vertx-redis-client/issues/365.


Update readme on how to use js-slang

Pull Request Test Coverage Report for Build 1131

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

Totals

Change from base Build 1125:
0.0%

Covered Lines:
922

Relevant Lines:
2994

💛  - Coveralls


GAWB-2472: changing jackson version

ran leo locally to make sure it didn't mess with anything

[ ] Submitter: Include the JIRA issue number in the PR description
[ ] Submitter: Make sure Swagger is updated if API changes
[ ] Submitter: If updating admin endpoints, also update firecloud-admin-cli
[ ] Submitter: Check documentation and code comments. Add explanatory PR comments if helpful.
[ ] Submitter: JIRA ticket checks:

Acceptance criteria exists and is met
Note any changes to implementation from the description
To Demo flag is set
Release Summary is filled out, if applicable
Add notes on how to QA

[ ] Submitter: Update RC_XXX release ticket with any config or environment changes necessary
[ ] Submitter: Update FISMA documentation if changes to:

Authentication
Authorization
Encryption
Audit trails

[ ] Tell the tech lead (TL) that the PR exists if they wants to look at it
[ ] Anoint a lead reviewer (LR). Assign PR to LR

Review cycle:

LR reviews
Rest of team may comment on PR at will
LR assigns to submitter for feedback fixes
Submitter rebases to develop again if necessary
Submitter makes further commits. DO NOT SQUASH
Submitter updates documentation as needed
Submitter reassigns to LR for further feedback

[ ] TL sign off
[ ] LR sign off
[ ] Product Owner sign off
[ ] Assign to submitter to finalize
[ ] Submitter: Verify all tests go green, including CI tests
[ ] Submitter: Squash commits and merge to develop
[ ] Submitter: Delete branch after merge
[ ] Submitter: Test this change works on dev environment after deployment. YOU own getting it fixed if dev isn't working for ANY reason!
[ ] Submitter: Verify swagger UI on dev environment still works after deployment
[ ] Submitter: Inform other teams of any API changes via Slack and/or email
[ ] Submitter: Mark JIRA issue as resolved once this checklist is completed

Does workbench-libs need an update too?
https://github.com/broadinstitute/workbench-libs/blob/develop/project/Dependencies.scala#L7

Coverage remained the same at 72.512% when pulling d882708c95ef2401f5953581b4720139a3dcb1bd on gawb2472 into 92e64b3ed949a475a3d735abebb47e089aa28e6b on develop.

^Seems like a good idea.


Smarter redirect on slice creation

Please note i can't test the druid codepath.
After ea8a7ec1ba26393047f70bcacd335b9e22fc63b8 creating a slice
started redirecting to druid datasource from sqlalchemy tables.
That's quite painful for sqlalchemy tables users.
Instead of hardcoding a choice just query the db, if we don't
have any druid datasource fallback to sqlalchemy tables.
Bonus points we remove hacky javascript and make the message
translatable.

Coverage decreased (-0.4%) to 80.661% when pulling 4a9fbff852ba6cbf77877f539229e62aee804f78 on xrmx:smartercreateslice into dbb9356d7e4c9f9e809fc42d6705542b83de6e79 on airbnb:master.

Coverage decreased (-0.3%) to 80.785% when pulling 430db630661097b7f139616c295e7006eb1e2699 on xrmx:smartercreateslice into dbb9356d7e4c9f9e809fc42d6705542b83de6e79 on airbnb:master.

Coverage decreased (-0.4%) to 80.614% when pulling 2aafb7413d5197b2a8bf32413efff1b16e687f05 on xrmx:smartercreateslice into 8135c240dc178014f03c943ba1b9beab26716325 on airbnb:master.

Herllo, anything i can do to move this forward? thanks

Coverage increased (+0.2%) to 81.393% when pulling 0c68bc73f9f96d7fd1af87b2b92aec9e894aa409 on xrmx:smartercreateslice into ee9141a31a84c7c1114163995edb718f89372a15 on airbnb:master.

Coverage increased (+0.2%) to 81.393% when pulling b253b258751df6a78c30af3363468c5db0789532 on xrmx:smartercreateslice into ee9141a31a84c7c1114163995edb718f89372a15 on airbnb:master.

Coverage increased (+0.2%) to 81.363% when pulling 4761a68e8938d61855af665fe1305b66c9a00e6d on xrmx:smartercreateslice into ee9141a31a84c7c1114163995edb718f89372a15 on airbnb:master.

Sorry about the delay, I have merged a while back...


opt fix several insert into related issues

http_stream TVF should always generate one fragment plan
http_stream TVF plan should not check root as scan node
distinguish group_commit TVF with normal insert statement
index and generate slot should based on type cast base slot
agg_state could cast from nullable to non-nullable
colocated and bucket scan range compute should only on scan node

run buildall

run feut

run buildall


Persistant Volumes are not created

I'm not sure this is related to this specific project but it look like the persistent volumes aren't created dynamically. I've added the following StorageClasses but once I deploy a pod with a persistent volume claim the volume isn't created.
USE_KUBE_UI=true MASTER_CPUS=1 NODE_MEM=4096 MASTER_MEM=2048 NODE_CPUS=1 NODES=5 vagrant up

kind: StorageClass
apiVersion: storage.k8s.io/v1beta1
metadata:
  name: standard
  annotations:
    storageclass.beta.kubernetes.io/is-default-class: "true"
  labels:
    kubernetes.io/cluster-service: "true"
    addonmanager.kubernetes.io/mode: "EnsureExists"
provisioner: kubernetes.io/gce-pd
parameters:
  type: pd-standard
---
kind: StorageClass
apiVersion: storage.k8s.io/v1
metadata:
  name: slow
  annotations:
    storageclass.beta.kubernetes.io/is-default-class: "false"
provisioner: kubernetes.io/gce-pd
parameters:
  type: pd-standard
  fsType: ext4
---
kind: StorageClass
apiVersion: storage.k8s.io/v1beta1
metadata:
  name: fast
  annotations:
    storageclass.beta.kubernetes.io/is-default-class: "false"
provisioner: kubernetes.io/gce-pd
parameters:
  type: pd-ssd
---
apiVersion: v1
kind: PersistentVolumeClaim
metadata:
  name: local-postgres
  namespace: local
spec:
  accessModes:
    - ReadWriteOnce
  resources:
    requests:
      storage: 5Gi

provisioner: kubernetes.io/gce-pd

Did you use this repo to provision a cluster on top of GCP?

No in vagrant locally, I completely overlooked the provisioner line, what should that be for a local env?

Whatever the provisioner you want to use defines.

I don't see why the PV's aren't created dynamically based on the PVC's...

The provisioner you chose is not available.

Maybe a stupid question but is there a way to see which ones are available or how can I install these, duckduckgo/google doesn't seem to be of any help

Here's a list of known provisioners.
For local disk volumes, it's tricky - there's alpha support coming in Kubernetes 1.9. Perhaps, you'll be able to get the NFS provisioner to work.


Packagist release

I'm currently using the NestedSetsQueryBehavior to make the Kartik-V Tree Manager work with MongoDB and it works great!
I however had to copy the classes directly into my codebase because I couldn't find them on packagist. Do you plan on releasing this repository there any time soon?

i dont develop it, i just convert kartik-V's class to support mongodb because i need it!

https://packagist.org/packages/letyii/yii2-mongo-nested-set

Oh great, thanks for pointing me there! Guess I overlooked it earlier...


Add extended validate generator

Hello.
Some probes may include additional data in probe packets like timestamps, TTL, etc. It may be desirable to include additional information into validation input besides src and dst IPs.
This patch adds additional validation generator function which allows to fill all fields of the aes_input array, so that probe could use it to include additional information to sourch of the validation.
Not sure though about the best naming for the new function.

Thanks!


Feature/previous projects

Please briefly describe the changes made in your pull request below
Implemented the entire previous projects from 2023 page
Known Issues

Mobile scaling
next js routing

If your changes made visual updates to the website, please add screenshot(s) below

So this looks pretty good so far. Thank you! For components like the carousel I usually prefer to find a library that does it for me so that I don't have to worry about all the complicated logic of the component and making sure it sizes dynamically. Also do you think you could add the prize categories to the site? So that visitors see which projects won what.
You might want to check out this cool component I found on NPM: https://www.npmjs.com/package/react-multi-carousel

Is there a way to do it without the library that seems more complicated and also not sure how I would use that and still match the figma design.
I am also confused how I would add prize categories where would it go?

The way that image carousels like this one are implemented are usually done by having a parent div with a fixed width and the css property overflow-x: hidden. Inside that div you have a list of all the carousel elements, each element has the same width as the parent div. Then when the user clicks a button, you just set the translation (x-movement) of all the carousel items to be offset left or right by the width * visible_index (i.e. which item is visible). Your approach also works well, if you wanted to get fancy without changing too much of the logic, you could do a fade-out, fade-in type of thing without adding too much logic using requestAnimationFrame().
Another thing that is important the Figma designs are not the end all be all, they are a suggestion for the layout, but ultimately the developer has the final say on the layout and design. If you want you could move the buttons, change their sizes, change colors, etc. I strongly encourage you to change the designs as the design for the buttons on Figma was not supposed to be like that on the site.
Lastly, about the prize categories, I would recommend placing them under the project title in maybe smaller font? Ultimately its up to you as the dev.


Signature matching algorithm does not computer properly  [BUG]

Hello! @yu-jingfeng
Great work on this library!
I saw that your library is based off of:  https://github.com/ProvenanceLabs/image-match (Documentation here)
In your example, when comparing Image #1 & Image #2 (Which are the same as the python example), yours returns 0.4600757541644907
The Python example returns: Returns 0.22095170140933634.
In the documentation of the Python example it says:

Normalized distances of less than 0.40 are very likely matches.

Is this a bug? Can you explain what is going on here please? Is the development complete for a .NET port of this Python library?
Thanks!

Thank you, @jeffward01
Because the AutoEntropyCrop option (default true) uses the image.Mutate(o => o.EntropyCrop()) method to crop 2.jpg but not 1.jpg, so there was a difference in the result.
Now the auto crop has been removed and the image is not cropped by default, please set the SignatureOptions.CropPercentiles option if you need to crop


real time updates of pipelines seem to be broke in fabric8-ui

so you have to keep hitting refresh to see a pipeline move through stages to completion

@pmuir any idea on this one? is it related to updates in how the observables are working?

This was working fine as long the websockets connected as recently as Weds.

seems fine now!


Kernel updates 2024-09-08

Description of changes
https://cdn.kernel.org/pub/linux/kernel/v6.x/ChangeLog-6.10.9
Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
24.11 Release Notes (or backporting 23.11 and 24.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

@ofborg test latestKernel.login


Migrate to Hertz

你当前使用的版本
v1.0.2
描述一下此特性
Hello @1379, sonic looks very cool and I note that there are plans to migrate to Hertz in its Todo, so feel free to contact me if you have any problems migrating to Hertz. Best wishes❤
附加信息
No response

Thanks! If you are interested, welcome to be a contributor!  :) Best wishes❤


the code for choosing hard class-m？

Hi，
May I ask where is the code for choosing hard class-m ?
Thanks

Hello, liu. Your work is insighting and very useful to my work on peptide discovering, will you please update the implementation for the hard-meta batch in the current repository?


请求服务报错，TypeError: 'Request' object does not support item assignment

[2020-08-17 19:50:17 +0800] [6338] [INFO] Starting gunicorn 20.0.4
[2020-08-17 19:50:17 +0800] [6338] [INFO] Listening at: http://<IP_ADDRESS>:8001 (6338)
[2020-08-17 19:50:17 +0800] [6338] [INFO] Using worker: sanic.worker.GunicornWorker
[2020-08-17 19:50:17 +0800] [6349] [INFO] Booting worker with pid: 6349
2020-08-17 19:50:17,380 WARNING aiocache.log(13) | cPickle module not found, using pickle
[2020-08-17 19:50:17 +0800] [6350] [INFO] Booting worker with pid: 6350
2020-08-17 19:50:17,463 WARNING aiocache.log(13) | cPickle module not found, using pickle
2020-08-17 19:50:17,510 INFO root(25) | owllook 启动模式：PRO
2020-08-17 19:50:17,624 INFO root(25) | owllook 启动模式：PRO
2020-08-17 19:50:18,058 INFO root(27) | Starting aiocache
2020-08-17 19:50:18,058 INFO root(38) | Starting redis pool
2020-08-17 19:50:18,137 INFO root(27) | Starting aiocache
2020-08-17 19:50:18,137 INFO root(38) | Starting redis pool
2020-08-17 19:50:28,549 INFO root(55) | user ip is: None
hello world
2020-08-17 19:50:28,550 INFO asyncio_redis(99) | Connecting to redis
2020-08-17 19:50:28,551 INFO asyncio_redis(821) | Redis connection made
2020-08-17 19:50:28,551 INFO asyncio_redis(99) | Connecting to redis
2020-08-17 19:50:28,552 INFO asyncio_redis(821) | Redis connection made
2020-08-17 19:50:28,552 INFO asyncio_redis(99) | Connecting to redis
2020-08-17 19:50:28,553 INFO asyncio_redis(821) | Redis connection made
2020-08-17 19:50:28,553 INFO asyncio_redis(99) | Connecting to redis
2020-08-17 19:50:28,554 INFO asyncio_redis(821) | Redis connection made
2020-08-17 19:50:28,554 INFO asyncio_redis(99) | Connecting to redis
2020-08-17 19:50:28,555 INFO asyncio_redis(821) | Redis connection made
2020-08-17 19:50:28,555 INFO asyncio_redis(99) | Connecting to redis
2020-08-17 19:50:28,556 INFO asyncio_redis(821) | Redis connection made
2020-08-17 19:50:28,556 INFO asyncio_redis(99) | Connecting to redis
2020-08-17 19:50:28,557 INFO asyncio_redis(821) | Redis connection made
2020-08-17 19:50:28,557 INFO asyncio_redis(99) | Connecting to redis
2020-08-17 19:50:28,557 INFO asyncio_redis(821) | Redis connection made
2020-08-17 19:50:28,558 INFO asyncio_redis(99) | Connecting to redis
2020-08-17 19:50:28,558 INFO asyncio_redis(821) | Redis connection made
2020-08-17 19:50:28,559 INFO asyncio_redis(99) | Connecting to redis
2020-08-17 19:50:28,559 INFO asyncio_redis(821) | Redis connection made
[2020-08-17 19:50:28 +0800] [6350] [ERROR] Exception occurred while handling uri: 'http://<IP_ADDRESS>:8001/'
Traceback (most recent call last):
File "/root/.local/share/virtualenvs/owllook-wVMRXxFE/lib/python3.6/site-packages/sanic/app.py", line 907, in handle_request
request, request_name=name
File "/root/.local/share/virtualenvs/owllook-wVMRXxFE/lib/python3.6/site-packages/sanic/app.py", line 1265, in _run_request_middleware
response = await response
File "/soft/owllook/owllook/server.py", line 63, in add_session_to_request
await app.session_interface.open(request)
File "/root/.local/share/virtualenvs/owllook-wVMRXxFE/lib/python3.6/site-packages/sanic_session/redis_session_interface.py", line 70, in open
request['session'] = session_dict
TypeError: 'Request' object does not support item assignment
2020-08-17 19:50:28,560 ERROR sanic.root(140) | Exception occurred while handling uri: 'http://<IP_ADDRESS>:8001/'
Traceback (most recent call last):
File "/root/.local/share/virtualenvs/owllook-wVMRXxFE/lib/python3.6/site-packages/sanic/app.py", line 907, in handle_request
request, request_name=name
File "/root/.local/share/virtualenvs/owllook-wVMRXxFE/lib/python3.6/site-packages/sanic/app.py", line 1265, in _run_request_middleware
response = await response
File "/soft/owllook/owllook/server.py", line 63, in add_session_to_request
await app.session_interface.open(request)
File "/root/.local/share/virtualenvs/owllook-wVMRXxFE/lib/python3.6/site-packages/sanic_session/redis_session_interface.py", line 70, in open
request['session'] = session_dict
TypeError: 'Request' object does not support item assignment
[2020-08-17 19:50:28 +0800] - (sanic.access)[INFO][<IP_ADDRESS>:54176]: GET http://<IP_ADDRESS>:8001/  500 250
2020-08-17 19:50:28,563 INFO sanic.access(494) |
[2020-08-17 19:50:28 +0800] - (sanic.access)[INFO][<IP_ADDRESS>:54178]: GET http://<IP_ADDRESS>:8001/favicon.ico  200 296
2020-08-17 19:50:28,860 INFO sanic.access(494) |

大佬这个是语法错误吗？

File "/root/.local/share/virtualenvs/owllook-wVMRXxFE/lib/python3.6/site-packages/sanic_session/redis_session_interface.py", line 70, in open
request['session'] = session_dict
TypeError: 'Request' object does not support item assignment

大佬这个是语法错误吗？

应该是你sanic版本不对
2020-08-17 20:30 GMT+08:00, banwang811<EMAIL_ADDRESS>
大佬这个是语法错误吗？

--

You are receiving this because you are subscribed to this thread.

Reply to this email directly or view it on GitHub:

https://github.com/howie6879/owllook/issues/95#issuecomment-674852615

感谢！是request库的版本太高导致的。


Update latest changes

Docs Build status updates of commit c00a341:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

ThirdPartyNotices
:white_check_mark:Succeeded

sandbox/demos/github-release-notes.md
:white_check_mark:Succeeded
View

sandbox/demos/notbacon.md
:white_check_mark:Succeeded
View

sandbox/demos/nothotdog.md
:white_check_mark:Succeeded
View

sandbox/demos/octobot.md
:white_check_mark:Succeeded
View

sandbox/functions-recipes/durable-manage-orchestrations.md
:white_check_mark:Succeeded
View

sandbox/functions-recipes/includes/cli-publish-locally.md
:white_check_mark:Succeeded
View

sandbox/functions-recipes/includes/durable-activity-functions-errors.md
:white_check_mark:Succeeded
View

sandbox/functions-recipes/includes/durable-activity-functions-with-retry.md
:white_check_mark:Succeeded
View

sandbox/functions-recipes/includes/durable-http-api.md
:white_check_mark:Succeeded
View

sandbox/functions-recipes/includes/durable-signaling-events.md
:white_check_mark:Succeeded
View

sandbox/functions-recipes/includes/portal-environment-controlling-http-verbs.md
:white_check_mark:Succeeded
View

sandbox/functions-recipes/includes/portal-environment-daily-usage-quota.md
:white_check_mark:Succeeded
View

sandbox/functions-recipes/includes/portal-environment-keys.md
:white_check_mark:Succeeded
View

sandbox/functions-recipes/includes/portal-functions-in-portal.md
:white_check_mark:Succeeded
View

sandbox/functions-recipes/includes/proxies-conditional-responses.md
:white_check_mark:Succeeded
View

sandbox/functions-recipes/includes/proxies-mock-api.md
:white_check_mark:Succeeded
View

sandbox/functions-recipes/includes/proxies-proxies-json.md
:white_check_mark:Succeeded
View

sandbox/functions-recipes/includes/proxies-redirects.md
:white_check_mark:Succeeded
View

sandbox/functions-recipes/includes/proxies-variables.md
:white_check_mark:Succeeded
View

sandbox/functions-recipes/includes/table-storage-input-binding.md
:white_check_mark:Succeeded
View

sandbox/functions-recipes/includes/visual-studio-function-tools.md
:white_check_mark:Succeeded
View

sandbox/gamedev/azure-gaming-samples.md
:white_check_mark:Succeeded
View

sandbox/gamedev/index.yml
:white_check_mark:Succeeded
View

sandbox/gamedev/monogame/azure-monogame.md
:white_check_mark:Succeeded
View

sandbox/gamedev/unity/azure-event-hubs-unity-2017.md
:white_check_mark:Succeeded
View

sandbox/gamedev/unity/azure-event-hubs-unity.md
:white_check_mark:Succeeded
View

sandbox/gamedev/unity/azure-mobile-apps-unity-2017.md
:white_check_mark:Succeeded
View

sandbox/gamedev/unity/azure-mobile-apps-unity.md
:white_check_mark:Succeeded
View

sandbox/gamedev/unity/azure-storage-unity-2017.md
:white_check_mark:Succeeded
View

sandbox/gamedev/unity/azure-storage-unity.md
:white_check_mark:Succeeded
View

sandbox/gamedev/unity/samples/azure-facebook-auth-unity.md
:white_check_mark:Succeeded
View

sandbox/gamedev/unity/samples/azure-mobile-apps-unity-racer.md
:white_check_mark:Succeeded
View

sandbox/gamedev/unity/samples/azure-storage-bird.md
:white_check_mark:Succeeded
View

sandbox/gamedev/unreal/azure-storage-unreal.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


use the download url from build order file in all the build commands

Fixing this may also resolve #92.


Make travis fail based on scripts

Make sure we will fail the scripts.

Tried to understand this PR, but very lost. Explain in person :)


MArkdig produce error "Operation could destabilize the runtime." on remote IIS web server

> PEVerify.exe markdig.dll
returns more than 300 errors similar to this:

[MD]: Error: Token 0x0200012f following ELEMENT_TYPE_CLASS (_VALUETYPE) in signature is a ValueType (Class,respectively). [token:0x1B0000AE]

Because of these errors markdig is unusable on many (Italian) web servers due to the error:
Operation could destabilize the runtime.

Description: An unhandled exception occurred during the execution of the current
web request. Please review the stack trace for more information about the error
and where it originated in the code.

Exception Details: System.Security.VerificationException: Operation could
destabilize the runtime.

Any work around?

Markdig is using unsafe code so it can't be verified.
About this error I can't tell, but PEVerify is barely usable today on most OSS libraries that are using structs...etc
So we can't do anything about this. You should try to find a provider that allows you to host .NET Core ASP.NET, you won't get such errors (e.g Azure)


[x] Suggest an Awesome item.
[ ] Web site maintenance.
[ ] Other.

What's in this PR?
This PR adds Null++ podcast in Arabic to Mutimedia
Checklist

[x] Ensure to check our Contribution Criteria.
[x] The contribution doesn't already exist (you can use search functionality).
[x] The contribution is provided in both Arabic and English.

@MhmoudAlim Thanks for your contribution :raised_hands:
I made a tiny tidy-up, so it's ready to merge.