Test_ServerSingleGraphiteIntegration_NoDatabase is racy

https://circle-artifacts.com/gh/influxdb/influxdb/2384/artifacts/1/tmp/circle-artifacts.AiSZUt5/test_logs_race.txt
[messaging] 2015/05/13 22:35:12 connected to broker: http://:44741/messaging/messages?index=0&streaming=true&topicID=1
--- FAIL: Test_ServerSingleGraphiteIntegration_NoDatabase (6.92s)
	server_integration_test.go:1977: Graphite Connection String: <IP_ADDRESS>:0
	server_integration_test.go:88: Creating cluster of 1 nodes for test graphite integration
	server_integration_test.go:97: Test graphite integration: using tmp directory "/tmp/influxdb-279773726/broker-integration-test" for brokers
	server_integration_test.go:98: Test graphite integration: using tmp directory "/tmp/influxdb-279773726/data-integration-test" for data nodes
	server_integration_test.go:138: http://:49997
	server_integration_test.go:2004: Writing data
	server_integration_test.go:2019: Test "graphite integration" failed, expected: {"results":[{"series":[{"name":"cpu","columns":["time","value"],"values":[["2015-05-13T22:35:05Z",23.456]]}]}]}, got: {"results":[{"series":[{"name":"cpu","columns":["time","value"]}]}]}
--- PASS: Test_ServerOpenTSDBIntegration_BadData (6.89s)
	server_integration_test.go:2157: OpenTSDB Connection String: <IP_ADDRESS>:0
	server_integration_test.go:88: Creating cluster of 1 nodes for test ServerOpenTSDBIntegration_BadData
	server_integration_test.go:97: Test ServerOpenTSDBIntegration_BadData: using tmp directory "/tmp/influxdb-239093855/broker-integration-test" for brokers
	server_integration_test.go:98: Test ServerOpenTSDBIntegration_BadData: using tmp directory "/tmp/influxdb-239093855/data-integration-test" for data nodes
	server_integration_test.go:138: http://:38835
	server_integration_test.go:177: Test: ServerOpenTSDBIntegration_BadData: creating database opentsdb
	server_integration_test.go:184: Creating retention policy raw for database opentsdb
	server_integration_test.go:2172: Writing data
[raft] 2015/05/13 22:35:12 leader[2]: leaderLoop: send heartbeat: success url=http://:51052
2015/05/13 22:35:12 openTSDB TCP listener closed
2015/05/13 22:35:12 error reading from openTSDB connection EOF
--- PASS: Test_ServerOpenTSDBIntegration (7.36s)
	server_integration_test.go:2042: OpenTSDB Connection String: <IP_ADDRESS>:0
	server_integration_test.go:88: Creating cluster of 1 nodes for test ServerOpenTSDBIntegration
	server_integration_test.go:97: Test ServerOpenTSDBIntegration: using tmp directory "/tmp/influxdb-999206117/broker-integration-test" for brokers
	server_integration_test.go:98: Test ServerOpenTSDBIntegration: using tmp directory "/tmp/influxdb-999206117/data-integration-test" for data nodes
	server_integration_test.go:138: http://:44741
	server_integration_test.go:177: Test: ServerOpenTSDBIntegration: creating database opentsdb
	server_integration_test.go:184: Creating retention policy raw for database opentsdb
	server_integration_test.go:2057: Writing data
[messaging] 2015/05/13 22:35:12 connected to broker: http://:35486/messaging/messages?index=0&streaming=true&topicID=1
[server] 2015/05/13 22:35:12 redirect join: http://:59830/data/data_nodes

@corylanou

I'm guessing this is related to every "racy" write condition we are seeing.  I'll wait until that is solved before I dig into this one.

CI under heavy refactoring right now. If this still exists on the other side we will address it then.

For the record, this ticket was not to track an issue with CI, or the tests, it was that the database software itself seemed to have issues. In other words, the test was failing for a reason.
That said, the some of the latest work is going to make this issue moot, so closing it is fine.


Do you know why my bottom buttons ( Cancel / Save ) are black and not white ?

Do you know why my bottom buttons ( Cancel / Save ) are black and not white ?

Check the appearance methods in your app

Indeed the code explicitly sets them to white cancelButton.tintColor = .white so there might be something like UIBarButtonItem.appearance().tintColor = .black somewhere in your code?

No...so its why i dont understand. its really weird :/

Try to debug your view hierarchy. Maybe there is something, that overlaying these buttons.

Thats works but now i dont see cancel / next buttons in the previous VC because they are white....

That most likely on your side. However, u can send a test project, that we can search for a solution or a bug. Also be sure, that u use the last release from master branch.

Hi, this is the right solution, using appearance. Note that Theme.....Color is just a series of structs into my app.
`        let navBarImagePicker = UINavigationBar.appearance(whenContainedInInstancesOf: [YPImagePicker.self])
navBarImagePicker.isTranslucent = Theme.ImagePicker.NavigationBar.isTranslucent
navBarImagePicker.tintColor = Theme.ImagePicker.NavigationBar.tintColor
navBarImagePicker.barTintColor = Theme.ImagePicker.NavigationBar.barColor
navBarImagePicker.titleTextAttributes = [NSAttributedString.Key.foregroundColor: Theme.ImagePicker.NavigationBar.titleColor, NSAttributedString.Key.font : Theme.ImagePicker.NavigationBar.titleFont]
     let barBtnImagePicker = UIBarButtonItem.appearance(whenContainedInInstancesOf: [YPImagePicker.self])
    barBtnImagePicker.setTitleTextAttributes([NSAttributedString.Key.foregroundColor: Theme.ImagePicker.BarButtonItem.tintColor, NSAttributedString.Key.font : Theme.ImagePicker.BarButtonItem.titleFont], for: UIControl.State.normal)
    barBtnImagePicker.setTitleTextAttributes([NSAttributedString.Key.foregroundColor: Theme.ImagePicker.BarButtonItem.tintColor, NSAttributedString.Key.font : Theme.ImagePicker.BarButtonItem.titleFont], for: UIControl.State.highlighted)`

Too old issue. Feel free to open new.


Implement set pricing options and plans by admin

eventyay-common: The system currently does not have a pricing structure for the event, e.g. 2.5% for tickets sold. Discuss the pricing structure with the team and implement it. Provide options for admins to set the pricing.
Also create a page at the slug /pricing with the pricing information.

Hi @mariobehling, is this issue relate to fee per ticket sold of this issue #383
So we will have a setting on global settings to fee for all event?
and about /pricing page, is it FAQ of this issue #383 ?

Hi @mariobehling, is this issue relate to fee per ticket sold of this issue #383 So we will have a setting on global settings to fee for all event? and about /pricing page, is it FAQ of this issue #383 ?

Correct.


Report payload function errors back over MQTT

If a payload function fails, the processing is stopped and there should be a way to inform the user (see conversation in #248).
The extent to which we make these errors available to the end user, e.g. in the dashboard or ttnctl, is to define later, but it all starts with reporting back on MQTT.
For uplink payload function errors, I propose reporting the error to :app-id/devices/:dev-id/up/rejected, and for downlink :app-id/devices/:dev-id/down/rejected. This follows AWS IoT naming, and allows for a companion named accepted.
The error should indicate the type of error (payload function) and the concerning payload function (decoder, converter or validator).
The type of error should be clear, because there are other use cases for this topic. For example, we can use the .../up/rejected also for payload marked as invalid by the validator, and .../down/rejected in cases where we can't find a gateway for downlink in time.

Related to https://github.com/TheThingsNetwork/ttn/issues/159

The question here is if we should put something behind :app-id/devices/:dev-id/up/..., because we're also publishing payload fields here. So accepted and rejected become reserved words for fields.

OK, so after a bit of internal discussion regarding naming, reserved words, broader use as in #159, an application events topic is a better option.
Errors would be reported to :app-id/events/errors, other information may fall also under events. Monitoring can listen to :app-id/events/#. All messages posted in that tree should have the same payload format.

Closed with https://github.com/TheThingsNetwork/ttn/issues/159
Errors are reported in :app-id/devices/:dev-id/events/up/errors or :app-id/devices/:dev-id/events/down/errors


Standardize subscription lengths

Many subscriptions listed under "Subscribe" (https://test-mep.cdh.princeton.edu/admin/accounts/subscribe/) have start and end dates that do not correspond to the duration of the subscription.  After consulting with Josh, I have learned that the logbooks only have start dates, so the end date must always be extrapolated from the duration.  This must be what is happening here--but, in many cases, the end date looks off.  What's going on?

@elspethgreen I think what you must be seeing is the way I'm tallying months right now, i.e. 3 months = 4 * 7.
I don't know the Beach records well enough to say, but she might be thinking like that, or she might be thinking as in 'a month is equal to the same day next month'. It's also possible I have a blooper in the code.
Do you know how precise she was in that metric (i.e. month = 4, 7 day weeks vs month = from mm/dd to mm+1/dd)? It would inform the import script to be sure.

I talked to Josh this afternoon, and while we don't know exactly what her methods were, it seems far more likely that a 1 month of subscription from October 10 would be up on November 10, just for simplicity's sake (also, only February is actually 28 days).  So that's what we would prefer on the import!

Yes, one month is one calendar month: so a six-month subscription that starts on 11/4 ends on 5/4 the next calendar year.  See card attached below as an example. -- J
<EMAIL_ADDRESS>
Joshua Kotin, Associate Professor
Princeton University
press.princeton.edu/titles/11207.htmlhttp://press.princeton.edu/titles/11207.html
On Oct 11, 2017, at 6:11 PM, elspethgreen <notifications<EMAIL_ADDRESS>wrote:
I talked to Josh this afternoon, and while we don't know exactly what her methods were, it seems far more likely that a 1 month of subscription from October 10 would be up on November 10, just for simplicity's sake (also, only February is actually 28 days). So that's what we would prefer on the import!
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.

Adding card image for documentation purposes:

Thanks @rlskoeser.

I'm closing this as a duplicate of tasks in #36 and will link it there as well.


Big updates

Important: improved model for C_r used, in line with previous PRL now. kwarg is set to use the improved model, may be adjusted by changing the kwarg.

Benchmarks work


[HOW-TO] Add new Scaler crops

The only scaler crop that seems to work is 64x64. If I try to change the scaler crop, nothing happens. The only scaler crops that are shown by the metadata is (0,0,64,64). I have a Camera Module 2 IMX219 and libcamera-apps works just fine (at full resolution). However, picamera2 does not allow me to change the scaler crop. Any help/pointers appreciated.

Hi, and thanks for the question. I'm afraid I wasn't quite sure what you were wanting to do, or what you had tried. Might you be able to attach a short self-contained test script (just a few lines, preferably) that shows what is not working as you expect? Thanks!


Nesting Navigators prevents WindowInsets from being updated upon device rotation on iOS

I have found that on iOS, window insets are not being updated correctly after the device is rotated, only if you are using nested navigators.
I have linked a sample project that contains the minimal reproducible code. Please see the App.kt file and change the useNestedNavigators value.
I've tested on both 1.1.0-alpha03, where the issue is present. Upon downgrading to 1.0.0 with no other changes, the issue goes away.
Sample Project: https://github.com/ElliotSknr/WindowInsetsDemo
https://github.com/adrielcafe/voyager/assets/16073141/cad5eeba-af1c-498b-aca5-f2bafd2587d6
https://github.com/adrielcafe/voyager/assets/16073141/301f6326-0c11-4090-a037-30091f5e372a

I've updated the sample project which replicates the issue to use 1.1.0-alpha04. Unfortunately the bump to Compose Multiplatform 1.6.1 in this version does not seem to have fixed this particular bug.


chore update regression-test README

Proposed changes
Issue Number: close #xxx

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

run buildall

run p0

run pipelinex_p0

run buildall

run pipelinex_p0


Provide Apache proxy working example

Lots of people want LDAP backends, and rely on a proxy design.
I wish we had a reference Apache config alongside the nginx example (in nginx.md) so we can get rid of the burden answering people in that setup.

Think this one could be added to the list in https://github.com/docker/distribution/issues/879 ?

+1 this could be a recipe
On Sat, Aug 15, 2015 at 12:50 PM, Sebastiaan van Stijn <
<EMAIL_ADDRESS>wrote:

Think this one could be added to the list in #879
https://github.com/docker/distribution/issues/879 ?
—
Reply to this email directly or view it on GitHub
https://github.com/docker/distribution/issues/855#issuecomment-131438142
.


Test icon is missing in the sidebar

My setup
Version: 1.61.2
Commit: 6cba118ac49a1b88332f312a8f67186f7f3c1643
Date: 2021-10-19T15:03:09.437Z
Electron: 13.5.1
Chrome: 91.0.4472.164
Node.js: 14.16.0
V8: 9.1.269.39-electron.0
OS: Darwin arm64 21.1.0
Case 1.
When "testExplorer.useNativeTesting": true; the Test UI tab is not available at all with only Test Adapter Converter and Test Explorer UI extensions installed

Case 2.
However, when I additionally installed Jest Test Explorer it shows the Test UI tab but can't find any tests and asks to install test extension.

Case 3.
When "testExplorer.useNativeTesting": false; and all three extensions installed everything works except for native Test UI.

Please clarify what's wrong with my setup. Thanks in advance.

Please share an example repo where this happens

Double checked in my other public repo - the same result.
Here is the example repo

This works for me:

Disabled all extensions except tests related - doesn't help.
Can it be node version-related? I have 16.10.0
Do you only have testExplorer.useNativeTesting: true for testExplorer config?
Can you please share your vscode config file?

I think this happens because some Test Adapters are only activated when the test view is selected, but when you use native testing, the test view only becomes visible when some tests have been registered, creating a chicken-and-egg problem.
See also microsoft/vscode#132395

In this case the test extension in the original comment's screenshot (https://github.com/microsoft/vscode-test-adapter-converter/issues/23#issue-1043757587) activates on *, so that should not be an issue here. There's also the "history" icon on the extension which indicates its activation time.

@connor4312 can you please give me a hint in which direction to debug that issue if it intended to work due to my screenshot?

Not really sure what's going on here. But you could also try vscode-jest which is the significantly more widely-supported and adopted extension for Jest.


Add EMAPNIK error code to render errors

Add the EMAPNIK error code to errors thrown by vtile.render() so that it's easier to detect and handle invalid styles.
cc @yhahn

@GretaCB is there a testcase you could add for this?

@yhahn on it :+1:


support for HTTP api gateway

Issue #, if available:
Description of changes:
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Codecov Report

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

@@           Coverage Diff           @@
##           master     #274   +/-   ##
=======================================
  Coverage   74.63%   74.63%
=======================================
  Files          20       20
  Lines         682      682
=======================================
  Hits          509      509
  Misses        128      128
  Partials       45       45

Continue to review full report at Codecov.

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

Renamed the struct to APIGatewayV2Request

Added test case

@bmoffatt  Is the next release version come soon with these changes or there is an schedule date for release?

@shashank-sachan i've released this change today as v1.16.0 - Enjoy!

@shashank-sachan i've released this change today as v1.16.0 - Enjoy!

Thanks @bmoffatt


tests: ❓ implement an IBC handshake using mock consensus engine

stub issue.
see #3588.
once we have the basics of our consensus engine working, we should proceed to provide facilities to test IBC functionality. let's start by writing a test that an IBC handshake works properly between two chains.

this wasn't assigned during spring planning, so i am going to remove it from the current sprint.


Override existing jar instead of deleting it when registering UDF

On some OS systems(eg.windows), it is not possible to delete a file when it is open or in use. The Jar file will be used by the acitive UDFClassLoader and can not be deleted On windows, which will lead to the failure of the CreateFunctionTask.


Added softbuffer

It looks like I made a typo. I'll correct that shortly. Is there any way to build the site locally to make sure that everything is correct in that regard?


EF Core Updates incorrect fields

Hi,
I'm experiencing data corruptions in FK fields. I tracked the problem down to a buggy UPDATE statement.
Indeed EF Core seem to update fields in tables that were not changed, resulting in corruption of certain FK fields.
Steps to reproduce
I can reproduce this bug with a databse schema containing one-to-one, one-to-zero and one-to-many relationships.
The bug occurs when updating related entities independently : If A owns B then update B independently of A and you have a chance of corrupting some FKs in A.
Further technical details
EF Core version: 2.1.1
Database Provider: Microsoft.EntityFrameworkCore.SqlServer
Operating system: Windows
IDE: Visual Studio 2017 15.4
Also using Microsoft.EntityFrameworkCore.Proxies

Adding this line in the code seem to prevent the corruptions :
Context.Entry().State = Microsoft.EntityFrameworkCore.EntityState.Detached;

@Yuri6037 Please post a small, runnable project/solution or complete code listing that demonstrates the behavior you are seeing.

The issue seem to happen only with large data sets so you'd need to seed the database first.
It's also kind of random, there is a chance a line of the database that was linked to an upated line in the other entity would not be affected.
I can't seem to reproduce this issue with reduced data sets. It does happen with more than 100 of lines in dbo.Symbols and more than 30 lines in dbo.SymbolRefs.
I can however link you to the repository it's open source, would that be sufficient ?
https://dev.azure.com/WikiLibs/_git/API?version=GBdevelop
I can also tell you were exactly the issue occurs : it occurs when updating the dbo.SymbolRefs table independently of the dbo.Symbols table the entire DB schema can be found in WikiLibs.Data.
The boggus code can be found in WikiLibs.Symbols.SymbolManager.OptimizeAsync.
PS : Don't forget to check you are on develop branch

@Yuri6037 How are you managing the lifecycle of the context? (I.e. when and how does a context instance get created and when and how does it get disposed.) Also, how are you ensuring that the context instance is not used by multiple threads concurrently?

In Unit Testing (WikiLibs.API.Tests)
The lifecycle is simple a new context is created each time a test is executed.
In Production/Development (WikiLibs.Core)
The entire lifecycle is defined by ASP.NET Core services
           services.AddDbContext<Data.Context>(o => o.UseLazyLoadingProxies()                                                       .UseSqlServer(Configuration.GetConnectionString("Default")));


Need a tool to bisect noisy tests/benchmarks against both framework and engine commits

The tool should

provide clear signals even if the benchmark is very noisy. The noise may make the bisect process longer, but should never make the final signal less confident.
be able to handle the engine build and the --local-engine flag to bisect engine commits in addition to framework commits.
be able to run on the device lab in case a regression is only reproducible there.
be able to retry flaky and failed test runs, or count the flaky/failed rate to bisect when it becomes flaky.
not require any human intervention during the bisection as it may take hours to finish a bisection for very noisy and lengthy benchmarks.

I'm currently working on it in spare time. This should help

https://github.com/flutter/flutter/issues/60210
https://github.com/flutter/flutter/issues/58248
https://github.com/flutter/flutter/issues/50570
https://github.com/flutter/flutter/issues/60559

closing as obsolete.


update GitHub Actions to use miniforge v3 (fixes mambaforge deprecation)

Mambaforge support in Miniconda is being deprecated since mamba is now included directly in miniforge, as described here.  To "encourage users to switch to Miniforge" a set of "brownouts" has been scheduled at increasing frequency, and one of them happened today, causing GitHub Actions workflows to stop working.
This PR updates the GitHub Actions workflows to use miniconda@v3, which fixes the issue.
Note: there are still some failures in the OS/BLAS test matrix workflow for Intel-based MacOS (example).  I'm not sure these are directly related to miniforge/mamba since they appear to affect only certain configurations in the test matrix.  I'll post a separate issue on that one.

coverage: 94.73%. remained the same
when pulling 4c76aa8caf432c01b8d909d694b41738e6a6451d on murrayrm:gha_mambaforge-14Dec2024
into 8b6d041e213574bf0ef9a62314c61f7432d0e3ad on python-control:main.


Added requirements.txt and other minor tweaks

Shell script to install ML models
requirements.txt
latest opencv (<IP_ADDRESS>) fixed the bug so removed workaround instructions

I cleaned them up. Also tested this on Windows and it seems to be working fine


Use cross-spawn in integration tests.

Summary
Uses cross-spawn for integration tests.  This allows some integration tests to be run on Windows.

Test plan
Nothing should change.

Wohoo! This is great.
cc @Daniel15 who is probably excited about this :D


feat: component dependencies resolver

Automatically loads components and injects dependencies.

Note: A proof of concept draft, working in progress

Checklist

[ ] npm test passes
[x] tests and/or benchmarks are included
[ ] documentation is changed or added

Closed in favor of #167


[Docs] Beta TestNet - Service Cheatsheet

Summary
Cheatsheet to add a new onchain Service plus a few other improvements.

Add a service quickstart
Update .gitignore w/ a few config files
Add tl;dr to CLI installation and use account setup`
Add instructions on creating a new Debian user

Issue

#901

Type of change
Select one or more from the following:

[ ] New feature, functionality or library
[ ] Consensus breaking; add the consensus-breaking label if so. See #791 for details
[ ] Bug fix
[ ] Code health or cleanup
[x] Documentation
[ ] Other (specify)

Testing

Testing things manually by following the instructions

@red-0ne @okdas
I ended up splitting Supplier CheatSheet and Service Cheatsheet into two as I started work on #947.
I think we should merge this in first.
Some asks as you review this:

Not the types of tl;drs I'm adding
Note that types of cross-references & pre-requisites I'm adding
Please inspect ☝ and provide your feedback

Our goals are to:

Stay consistent across all our docs
Make it easy for anyone to get running
Account for everyone's short attention spans
Have the right amount of cross-references; too many and too few are both bad

Good, I like how concise it is.

@okdas Ty. Was hoping to use this as an example/baseline for other cheatsheets too.

Good, I like how concise it is.

@red-0ne Appreciate the improvement suggestions!


[Question]:

Question Validation

[X] I have searched both the documentation and discord for an answer.

Question
I am using PandasInstructionParser(df_1) but its throwing effor as df_1 not defined how to over come that?
I am using 2 query pipeline for 2 different data one has dataframe as df that is running fine but this is causing problem
Traceback (most recent call last):
File "/usr/local/lib/python3.10/dist-packages/llama_index/experimental/query_engine/pandas/output_parser.py", line 61, in default_output_processor
output_str = str(safe_eval(module_end_str, global_vars, local_vars))
File "/usr/local/lib/python3.10/dist-packages/llama_index/experimental/exec_utils.py", line 159, in safe_eval
return eval(__source, _get_restricted_globals(__globals), __locals)
File "", line 1, in
NameError: name 'df_1' is not defined

@dosu
how to Check Variable Scope if df_1 is in correct scope?


Improve public API, unsubscribe, channels

Solved:

Event scoping
channel unsubscribe
channels list
auth token only for private/presence channels
Handle pusher:error
Configure as a named process

Ebert has finished reviewing this Pull Request and has found:

1 possible new issue (including those that may have been commented here).

But beware that this branch is 2 commits behind the stepnivlk:master branch, and a review of an up to date branch would produce more accurate results.
You can see more details about this review at https://ebertapp.io/github/stepnivlk/pushex/pulls/2.


Installing Firebase Tools

I am trying to install firebase tools for deploying my site using
npm install -g firebase-tools.
and I am getting a deprecated warning along with this link
https://github.com/request/request/issues/3142
Been stuck since 2 hours

A deprecated warning would not stop the install. The issue is likely elsewhere in your code or dependencies.

Okay I figured it out
On Wed, Apr 22, 2020 at 11:45 PM Mendel Konikov<EMAIL_ADDRESS>wrote:

A deprecated warning would not stop the install. The issue is likely
elsewhere in your code or dependencies.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/request/request/issues/3291#issuecomment-617945031,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AL3ERQQHVDSMAFARELEIP7TRN4X25ANCNFSM4LQ6JPOA
.

@harsh242 hey buddy how did u fix this error ???? i exp this same issue

I had problems with my NPM, I tried reinstalling node but it did not work so I shifted to Yarn- Package Manager and it solved the issue. since then I am using the Yarn Package Manager.

Request has been deprecated, alternative libraries are here.
Preferred library is Axios

thanks buddy ❤
On Wed, 21 Oct 2020, 10:24 am Harsh Badhai<EMAIL_ADDRESS>wrote:

Request has been deprecated, alternative libraries are here
https://github.com/request/request/issues/3143.
Preferred library is Axios
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/request/request/issues/3291#issuecomment-713311373,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AQ63RWZ4BMPQJ62RAUBHO7DSLZWBDANCNFSM4LQ6JPOA
.


Strange sorting by a column values in Chrome

Describe the bug
"Result search" table allows to sort data by certain columns, e.g. "num_gpus", score. The sorting is however different in Chrome and Firefox. Sorting in Chrome (version 87) seems to be incorrect, e.g. "1 2 4 8 1 4 2".
To Reproduce
Steps to reproduce the behavior:

Use Chrome web-browser. Used by me: Version 87.0.4280.88 (Official Build) (64-bit), English, Linux OS.
Go to https://perf.test.fedcloud.eu/result_search?benchmark=deephdc/deep-oc-benchmarks_cnn:synthetic
Scroll the table to see "num_gpus", click to sort.
See the order "1 2 4 8 1 4 2" (all data belong to the same Site but two "Flavours". "8 1 4 2" - belong to the same "Flavour").

Expected behavior
Firefox lists same data as "1 1 2 2 4 4 8", same system.
One may request additional feature to group results by a) Site b) Site + Flavour, e.g. "1 2 4 1 2 4 8"
Screenshots
Chrome:

I seem to have confused the js array sort comparator with C++-like boolean comparators. I'm surprised firefox knew how to handle it, but chrome couldn't. I've tested it and it seems to work properly in chrome now.


updates to handlers for new sync - note - removing filter level query

some changes to handlers for new sync.
Note: removing queryParams.filter, to conform with syncs' default handlers.
ping @wtrocki

Looks ok, but we will need to make changes in demo application in order to integrate this. Going to merge that to custom branch and leave that for now.


Fix AES authentication, and no-pass issue

Re-submitting the following PR with signed commits:
Fix AES authentication and no-pass issue #135

that was fast, thanks!


feat: set cpu/mem requests and limits for prisma and cloudsql

addresses #271
particularly the exceeds its request of 0 as mentioned https://github.com/paritytech/Nomidot/issues/271#issuecomment-608330706
I had previously mistaken the mem/cpu requests fields to be only relate to HPA (which we agreed was unnecessary for our app), which was why that PR was closed.
This PR sets the requests for resources to be as following:
prisma:

mem
cpu

request
1.28GiB (failed at 1278848Ki === 1.219GiBi)
250m default (wasn't the cause of failure)

limit
2.56GiB
500m

cloudsql

mem
cpu

request
16Mi (failed at 9364Ki === 9.145Mi)
250m default (wasn't the cause of failure)

limit
32Mi
500m

see also:

https://kubernetes.io/docs/concepts/configuration/manage-compute-resources-container/
https://medium.com/kubernetes-tutorials/efficient-node-out-of-resource-management-in-kubernetes-67f158da6e59
https://kubernetes.io/docs/tasks/administer-cluster/out-of-resource/

actually you did :) #271 (comment)

Hmm, I think I didn't write this clearly. The pod describe was on the last 50k. I don't think the nodewatcher ever got evicted. But I can't remember exactly tbh.

hmm but the containers that ran out of resources according to that log are in nodewatcher deployment.


Working with extensions now library is in pypi

Currently the library works when you do a git clone but if you've installed it from pypi the directories will be different.
https://github.com/iiif-prezi/iiif-prezi3/blob/0e66bc41438772c75e064c20964ed01aff1f3709/iiif_prezi3/loader.py#L6

We could refactor this in a number of ways:

Take a list of file paths and import them. Difficulty with passing paths is that it needs to be in the working directory not in parent directories with ..
Use a dictionary with the extensions listed (including extensions  in the extensions directory with enabled true/false)
Pass a list of extensions

Have very explicit documentation put extensions in extensions directory and do the following steps....

Mike has fixed this for bundled extensions but need to look at how users can add local extensions.
Mike also going to look at simplifying the extension process so you can pass a class to the extensions function.

Documentation now at: https://github.com/iiif-prezi/iiif-prezi3#extensions


IRGen: Fix runtime crash with default implementation of protocol requirement with a generic class

If the 'self' type is abstract, we still have to consider it as
a source of type metadata, because it might be a class-bound
generic parameter and the class might have generic parameters.
Fixes rdar://problem/36093833.

@swift-ci Please smoke test


Any plan to release the pre-trained models?

Yes it is doable. But the training itself should also be very fast: it can be finished in no more than half a day on one 2080ti/1080ti-like GPU.


Make sure {{FieldDB.FieldDBObject.application.brand}} is getting replaced by brand in help

... it isn't currently in the beta deploy ...

in dativebeta i can see its being replaced but maybe thats because i'm signed in?

I tried updating my pull requests to your master, it looks like there is nothing in the codebase anymore that does new FieldDB.Application so that might explain this issue.

I remember you asked if we could only create a FieldDB.Application when we know its a fielddb app which the user wants to use. Maybe that conditional got lost somewhere, so now there is never an app in FieldDB.FieldDBObject.application
Maybe this line could be where it is created?
https://github.com/jrwdunham/dative/pull/145/files#diff-b7081bb22c0fcbaf994a7243eaa4f98eL131
And we can put a simple object in FieldDB.FieldDBObject.application  which contains only the brand so that the help file renders when you are not using fielddb.


Inaccurate command description for where

Hey @verma-kunal, correct me if I'm wrong
On Journey/Day09.md, the mention
where <command> → to find the location of the executable file of that command

is inaccurate. Linux offers two ways to find the location of a binary:

whereis will locate the binary, source and man page for a command. (see more)
which will show the full path of shell command. (see more)

Hey @anubha-v-ardhan , thanks for opening an issue!
So, in UNIX based terminals (like in mac), we can find the location of the executables using the where command as well (example below)
To make these notes accessible for other systems as well, I'll add in your point
Thanks :)

Keeping this issue as open to remember the change :)

If you are interested, you can make this change as well in that file :)


Provide more helpful error message when name: is missing from pubspec

name: is now a required top-level field in the pubspec.yaml file.
Suggest making a more helpful error message when name: is missing.
Perhaps:
"Missing a required top-level name: key in the [path to file] pubspec"

This issue has been moved to dart-lang/pub#78.


Use faster foreach_set for keyframe interpolation in 2.90+

The faster foreach_set method now works for setting the keyframe interpolation type in Blender 2.90.
This PR uses the faster method if a supported Blender version is detected. In 2.90 and above, this makes the import always as fast as it currently is with the Bezier interpolation type. Performance in 2.8x is unchanged.
Some performance tests on a 57 MB .agr and Blender 2.90:

Version
Bezier interpolation import time
Constant interpolation import time

Current
172,8565 s
482,3850 s

This PR
171,7507 s
172,2544 s

Thank you :)


request: convert times to users local timezones

Checklist

[x] I'm reporting a feature request
[x] I've verified that I'm running youtube-dl version 2019.08.13
[x] I've searched the bugtracker for similar feature requests including closed ones

Description

I'd like youtube-dl to convert upload dates to the users current timezone when substituting for '%(upload_date)s' in output filenames.  Twitch in particular (and presumably many more) return their timestamps in UTC, and youtube-dl does not convert them before substitution which leads to the upload date being inconsistent in different timezones.  In my case this results in evening streams being labelled as the day after, while morning streams are marked labelled properly.  I originally thought this was a bug ( #22173 ), more specific examples of this behavior are available there.  Thank you for considering my request!

This will not work since you don't know the original timezone in general.

I meant the timezone of the downloader, not the recorder

To clarify, I was looking to have the date twitch reports (in UTC) to the downloader's current timezone.


How to serialize/deserialize solver

I am interested to save and restore a solver between sessions. Is it possible? I am using the .NET package. Thanks!

How to do this?

@lperron Would you mind clarifying?
I am reviewing a patch someone forwarded to me, which appears to be adapted almost verbatim from a Run example . AFAIK, this is a good start for Linear Solver serialization.
I'll start with the Linear Solver since the examples seem to be more readily available, but ultimately I am more interested in Constraint Solver since that is much of the work I am doing.
For Constraint Solver, initial analysis leads me to think that some sort of wrapper function around ExportModel and LoadModel would be desirable; i.e. export model to protobuf, save protobuf to file; followed by load protobuf from file, load model from protobuf.
Thanks...

@lperron Couple of questions following up:

Where is FLAGS_forced_mps_format defined? I take it as a std::string, but where? Third party library?

Also, studying the example, how does the model proto actually land in the solver? I see a reference to a .name() call, but what does a "name" have to do with the model? variables? constraints or other rules? etc?

Thanks!

@lperron I have a couple more follow on questions. Speaking about Linear Solver for the moment:

Let's say that model and/or request proto, once parsed, have a time limit set. How is that conveyed to the solver? It seems like this is set as a global, once, and is not part of the model itself. Which strikes me as a bit of a snafu in the making, but I could be wrong. Is there a better way to handle saving/loading the time limit value?

It seems to me that proper support for time limit is not going to be easy. Once it is set, the model and/or the time limit can easily get away from you. A bit deeper introspection, I don't want to turn time limit treatment into a huge rabbit hole for purposes of satisfactory serialization.
Thoughts?

@lperron Approaching an initial commit and merge candidate in order to expose a Linear Solver ReadModelFromFile patch.
ws2_32.lib(WS2_32.dll) : warning LNK4006: __NULL_IMPORT_DESCRIPTOR already defined in psapi.lib(PSAPI.DLL); second definition ignored
ws2_32.lib(WS2_32.dll) : warning LNK4221: This object file does not define any previously undefined public symbols, so it will not be used by any link operation that consumes this library
shlwapi.lib(SHLWAPI.dll) : warning LNK4006: __NULL_IMPORT_DESCRIPTOR already defined in psapi.lib(PSAPI.DLL); second definition ignored
shlwapi.lib(SHLWAPI.dll) : warning LNK4221: This object file does not define any previously undefined public symbols, so it will not be used by any link operation that consumes this library
sulum_interface.obj : warning LNK4221: This object file does not define any previously undefined public symbols, so it will not be used by any link operation that consumes this library
cplex_interface.obj : warning LNK4221: This object file does not define any previously undefined public symbols, so it will not be used by any link operation that consumes this library

There are a couple of linker warnings, probably should be errors. I hadn't noticed them before, but I also was not looking for them. The DLL does get linked it would seem.

@lperron Before I commit, here's the code I've got thus far. Basically adapted from examples/cpp/solver.cc.
bool MPSolver::ReadModelFromFile(const std::string& model_path, MpsReadMode read_mode) {
  MPModelProto model_proto;
  // TODO: TBD: case may be important here, either in the context of this function,
  // or more broadly in HasSuffixString and HasPrefixString ...
  if (HasSuffixString(model_path, ".mps") ||
    HasSuffixString(model_path, ".mps.gz")) {
    // TODO: TBD: is it really necessary to duplicate concerns, and potentially read cycles, due to an overlapping mode?
    glop::LinearProgram linear_program_free;
    glop::LinearProgram linear_program_fixed;
    glop::MPSReader mps_reader;

    auto should_read_free = (read_mode & MPS_FREE_FORM) == MPS_FREE_FORM;
    auto should_read_fixed = (read_mode & MPS_FIXED_FORM) == MPS_FIXED_FORM;

    const auto did_read_free = should_read_free &&
      mps_reader.LoadFileWithMode(model_path, true, &linear_program_free);
    auto did_read_fixed = should_read_fixed &&
      mps_reader.LoadFileWithMode(model_path, false, &linear_program_fixed);

    CHECK(did_read_free || did_read_fixed)
      << "Failed to parse the mps file '" + model_path + "'.";

    if (did_read_free && did_read_fixed) {
      // Fall back on the Free form mode.
      did_read_fixed = false;
    }

    if (did_read_free) {
      LOG(INFO) << "Read file '" + model_path + "' in free format.";
      LinearProgramToMPModelProto(linear_program_free, &model_proto);
    } else if (did_read_fixed) {
      LOG(INFO) << "Read file '" + model_path + "' in fixed format.";
      LinearProgramToMPModelProto(linear_program_fixed, &model_proto);
    }
  } else {
    MPModelRequest request_proto;

    CHECK(ReadFileToProto(model_path, &model_proto))
      << "Failed to read file '" + model_path + "' into model proto.";
    CHECK(ReadFileToProto(model_path, &request_proto))
      << "Failed to read file '" + model_path + "' into request proto.";

      // If the input proto is in binary format, both ReadFileToProto could return true.
    // Instead use the actual number of variables found to test the correct format of
    // the input.
    const auto is_model_proto = model_proto.variable_size() > 0;
    const auto is_request_proto = request_proto.model().variable_size() > 0;
    // If neither is the case then log a fatal error.
    if (!(is_model_proto || is_request_proto)) {
      LOG(FATAL) << "Failed to parse '" << model_path
        << "' as an MPModelProto or an MPModelRequest.";
      return false;
    } else {
      // TODO: TBD: not sure if this completely guards against the BOTH being the case.
      // What happens with is_model_proto, for instance?

      // We have checked whether neither case exists; also rule out both cases.
      CHECK(!(is_model_proto && is_request_proto));

      // TODO: TBD: how good/better/best to relay and/or set the Linear Solver time limit?
      if (is_request_proto) {
        LOG(INFO) << "Read input proto as an MPModelRequest.";

        // TODO: TBD: does the timer need to be re-factored to the model itself?
        if (request_proto.has_solver_time_limit_seconds()) {
          set_time_limit(static_cast<int64>(request_proto.solver_time_limit_seconds() * 1000));
        }

        // Run with the request proto in lieu of the "model proto".
        model_proto.Swap(request_proto.mutable_model());
      }
      else {
        LOG(INFO)
          << "Read input proto as an MPModelProto.";
      }

      std::string error_message;

      // Load the proto into the solver.
      const MPSolverResponseStatus status =
        LoadModelFromProto(model_proto, &error_message);

      CHECK_EQ(MPSOLVER_MODEL_IS_VALID, status)
        << MPSolverResponseStatus_Name(status) << ": " << error_message;

      LOG(INFO)
        << "Solver Dimension: " << NumConstraints() << " x " << NumVariables();
    }
  }

  return true;
}

And which SWIG exposes as:
public bool ReadModelFromFile(string model_name, int read_mode)
{
    // ...
}

But for the linker warning, seems to be okay in the build itself. Haven't verified in a run-time yet.

@lperron Slightly refined. Verified build, but untested.
bool MPSolver::ReadModelFromFile(const std::string& model_path, MpsReadMode read_mode) {
  MPModelProto model_proto;
  // TODO: TBD: case may be important here, either in the context of this function,
  // or more broadly in HasSuffixString and HasPrefixString ...
  if (HasSuffixString(model_path, ".mps") || // TODO: TBD: this is reading from linear program (?): shouldn't this be checking for LP then?
    HasSuffixString(model_path, ".mps.gz")) {
    // TODO: TBD: is it really necessary to duplicate concerns, and potentially read cycles, due to an overlapping mode?
    glop::LinearProgram linear_program_free;
    glop::LinearProgram linear_program_fixed;
    glop::MPSReader mps_reader;

    auto should_read_free = (read_mode & MPS_FREE_FORM) > 0;
    auto should_read_fixed = (read_mode & MPS_FIXED_FORM) > 0;

    const auto did_read_free = should_read_free &&
      mps_reader.LoadFileWithMode(model_path, true, &linear_program_free);
    auto did_read_fixed = should_read_fixed &&
      mps_reader.LoadFileWithMode(model_path, false, &linear_program_fixed);

    CHECK(did_read_free || did_read_fixed)
      << "Failed to parse the mps file '" + model_path + "'.";

    if (did_read_free && did_read_fixed) {
      // Fall back on the Free form mode.
      did_read_fixed = false;
    }

    if (did_read_free) {
      LOG(INFO) << "Read file '" + model_path + "' in free format.";
      LinearProgramToMPModelProto(linear_program_free, &model_proto);
    } else if (did_read_fixed) {
      LOG(INFO) << "Read file '" + model_path + "' in fixed format.";
      LinearProgramToMPModelProto(linear_program_fixed, &model_proto);
    }
  } else { // Read from .lp (?)
    MPModelRequest request_proto;

    // // TODO: TBD: actually, on second thought, maybe these two SHOULD NOT be checked, after all...
    // CHECK(ReadFileToProto(model_path, &model_proto))
    //   << "Failed to read file '" + model_path + "' into model proto.";
    // CHECK(ReadFileToProto(model_path, &request_proto))
    //   << "Failed to read file '" + model_path + "' into request proto.";
    ReadFileToProto(model_path, &model_proto);
    ReadFileToProto(model_path, &request_proto);

    // If the input proto is in binary format, both ReadFileToProto could return true.
    // Instead use the actual number of variables found to test the correct format of
    // the input.
    const auto is_model_proto = model_proto.variable_size() > 0;
    const auto is_request_proto = request_proto.model().variable_size() > 0;
    // If neither is the case then log a fatal error.
    if (!(is_model_proto || is_request_proto)) {
      LOG(FATAL) << "Failed to parse '" << model_path
        << "' input proto as either MPModelProto or MPModelRequest.";
      return false;
    } else {
      // TODO: TBD: not sure if this completely guards against the BOTH being the case.
      // What happens with is_model_proto, for instance?

      // We have checked whether neither case exists; also rule out both cases.
      CHECK(!(is_model_proto && is_request_proto));

      // TODO: TBD: how good/better/best to relay and/or set the Linear Solver time limit?
      if (is_request_proto) {
        LOG(INFO) << "Read input proto as an MPModelRequest.";

        // TODO: TBD: could potentially be refactored if, for instance, solver time limit is swapped into model_proto above...
        if (request_proto.has_solver_time_limit_seconds()) {
          set_time_limit(static_cast<int64>(request_proto.solver_time_limit_seconds() * 1000));
        }

      // Run with the request proto in lieu of the model proto.
        model_proto.Swap(request_proto.mutable_model());
      }
      else {
        LOG(INFO) << "Read input proto as an MPModelProto.";

        // TODO: TBD: what happens, if anything, for solver time limit as part of MPModelProto (?), esp. since not a member...
      }

      std::string error_message;

      // TODO: TBD: here, we aren't concerned with previous solver state? i.e. clearing it out at all?
      //  i.e. likely usage scenario, starting from new solver instance, clean slate...

      // Load the model proto into the solver.
      const MPSolverResponseStatus status =
        LoadModelFromProto(model_proto, &error_message);

      CHECK_EQ(MPSOLVER_MODEL_IS_VALID, status)
        << MPSolverResponseStatus_Name(status) << ": " << error_message;

      LOG(INFO) << "Solver Dimension: " << NumConstraints() << " x " << NumVariables();
    }
  }

  return true;
}

@lperron Couple of assumptions I am making based on what's there:

Makes an appropriate read of either MPS (or gzipped MPS), versus LP format in the else clause.

Making appropriate juggling of either fixed and/or free form input during the MPS read.

The transmission of the solver time limit seems somewhat limited to me. That's a rabbit hole I'm not jumping down just now as the implications seem significantly more vast to me than the issue warrants.

Assumes that the current state of the solver can simply be loaded, and that nothing needs to be cleared from the solver prior to load.

@lperron Which will also require a couple of minor changes in the HasSuffixString and HasPrefixString string utilities.
Can go about this a hundred different ways, probably. I saw one example leveraging the std::basic_string<> character traits for case sensitivity. However, given the mixed platform support, this approach is about as good as any.
#include <algorithm>

Defaulting to case sensitivity for *nix purposes. Otherwise, can specific insensitivity, i.e. for Windows.
inline bool HasSuffixString(const StringPiece& str, const StringPiece& suffix, bool case_insensitive = false) {

  // Just return when case sensitive.
  if (!case_insensitive) {
    return str.ends_with(suffix);
  }

  return str.length() > suffix.length() &&
         std::equal(str.begin() + (str.length() - suffix.length()), str.end(), suffix.begin(),
         [](const char& a, const char& b) { return ::tolower(a) == ::tolower(b); });
}

inline bool HasPrefixString(const StringPiece& str, const StringPiece& prefix, bool case_insensitive = false) {

  // Just return when case sensitive.
  if (!case_insensitive) {
    return str.starts_with(prefix);
  }

  return str.length() > prefix.length() &&
         std::equal(str.begin(), str.begin() + prefix.length(), prefix.begin(),
         [](const char& a, const char& b) { return ::tolower(a) == ::tolower(b); });
}

I'm pretty sure the iterator boundaries are correct, but please double check them.

Hi Michael,
I wonder what is the end goal of this. If you want to store models, or send
models from masters to workers, you can use MPS or proto file.
If you do not need MPS, I would stick to proto files. Furthermore, we have
code to make them work in all languages.
We have implemented a linear solver proto that allows these forms of
communications.
You create a MPSolverRequest -- which does contains time limit -- and then
solve it with MPSolver::SolveWithProto().
And you can store these proto, send them in binary form stored inside a
string.
For CP, we have a similar mechanism with the 'model.proto' file. And we are
working on the same features with our CP on SAT solver.
So bottom line,
I you do not need MPS, stick to proto.
If you need MPS, you do not need linear_solver, just use the solver
binaries.
I hope this helps.
--Laurent
Laurent Perron | Operations Research |<EMAIL_ADDRESS>| (33) 1 42 68 53
00
2017-03-15 20:49 GMT+01:00 Michael Powell<EMAIL_ADDRESS>
@lperron https://github.com/lperron Which will also require a couple of
minor changes in the HasSuffixString and HasPrefixString string utilities.
Can go about this a hundred different ways, probably. I saw one example
leveraging the std::basic_string<> character traits for case sensitivity.
However, given the mixed platform support, this approach is about as good
as any.
#include
Defaulting to case sensitivity for *nix purposes. Otherwise, can
specific insensitivity, i.e. for Windows.
inline bool HasSuffixString(const StringPiece& str, const StringPiece& suffix, bool case_insensitive = false) {
// Just return when case sensitive.
if (!case_insensitive) {
return str.ends_with(suffix);
}
return str.length() > suffix.length() &&
std::equal(str.begin() + (str.length() - suffix.length()), str.end(), suffix.begin(),
[](const char& a, const char& b) { return ::tolower(a) == ::tolower(b); });
}
inline bool HasPrefixString(const StringPiece& str, const StringPiece& prefix, bool case_insensitive = false) {
// Just return when case sensitive.
if (!case_insensitive) {
return str.starts_with(prefix);
}
return str.length() > prefix.length() &&
std::equal(str.begin(), str.begin() + prefix.length(), prefix.begin(),
[](const char& a, const char& b) { return ::tolower(a) == ::tolower(b); });
}
I'm pretty sure the iterator boundaries are correct, but please double
check them.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/google/or-tools/issues/325#issuecomment-286858296,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AKj17f9l-tqhdMZAB9G3DGXpfx-Yq_xqks5rmEDZgaJpZM4MKRmr
.

@lperron Indeed, well, among my goals is to serialize the solver state, including variables, constraints, and anything else, such as already visited solution candidates (i.e. decision builder?), necessary to restore and continue running where it left off. That's one of my goals for the Constraint Solver. Possibly for the Linear Solver, eventually, but I don't have that use case, yet.
SolveWithProto is not (currently?) exposed via the SWIG to Solver. From the C++ perspective, a const pointer is probably the right thing to do. But it isn't being interpreted correctly by the SWIG generated code. Either ExportModel needs to return a CpModel, or perhaps accept an out CpModel proto.
Okay, so taking a look at the SWIG-generated assembly.
Further, from the C# perspective, there is Solver.ExportModel and Solver.LoadModel, correct? I'm not sure how SWIGTYPE_p_operations_research__CpModel (brilliant name, BTW) can be created since the constructors are not visible to outside callers, and it's not like there is a ref or an out parameter, from C# perspective.
Perhaps, notwithstanding some further C++ and/or SWIG work needing to be done, the first thing I want to do is extend SWIGTYPE_p_operations_research__CpModel to something more meaningfully named, like simply class CpModel : SWIGTYPE_p_operations_research__CpModel {}.
Minimally, do the SWIGTYPE_p_operations_research__CpModel constructor(s) need to be exposed?

@lperron So when I try something like this:
public class CpModel : SWIGTYPE_p_operations_research__CpModel { }

[Test]
public void Testing()
{
    var s = new Solver("Test");
    var d = new CpModel();
    var bit = s.MakeIntVar(0, 1);
    bit.SetName("Bit");
    try
    {
        s.ExportModel(d);
    }
    catch (Exception ex)
    {
    }
    var s2 = new Solver("");
    s2.LoadModel(d);
}

I get as far as ExportModel and the whole thing crashes and burns. No exception. Just CRASH.
Without diving further into the ExportModel code itself, it seems to me the fact that the CpModel is, but default basically a proxy in a null state internally (?), that the whole thing dies.
Which is my understanding of the IntPtr.Zero, it's like having a null C++ pointer, handle, or whatever else it is in which it is standing.
protected SWIGTYPE_p_operations_research__CpModel()
{
    swigCPtr = new global::System.Runtime.InteropServices.HandleRef(null, global::System.IntPtr.Zero);
}

But I could be wrong about that.
Then there is the cPtr, futureUse ctor. Is this not fully implemented, then? If so, what can I do to help that along?
internal SWIGTYPE_p_operations_research__CpModel(global::System.IntPtr cPtr, bool futureUse)
{
    swigCPtr = new global::System.Runtime.InteropServices.HandleRef(this, cPtr);
}

@lperron Thoughts, comments, concerns?
Digging in a bit further, I believe there are SWIG ways of exposing a C# out parameter, leaving aside other targets for the time being. I don't care about Java or Python for these purpose, not to mention SWIG mapping is likely very different.
The Solver::ExportModel is overloaded. Leaving aside the overloads, the parameter I might be interested to overload is CpModel* const model_proto.
void Solver::ExportModel(const std::vector<SearchMonitor*>& monitors,
                         CpModel* const model_proto,
                         DecisionBuilder* const db) const;

void Solver::ExportModel(const std::vector<SearchMonitor*>& monitors,
                         CpModel* const model_proto) const;

void Solver::ExportModel(CpModel* const model_proto) const;

Which gets mapped as this in the current release:
  public void ExportModel(SWIGTYPE_p_operations_research__CpModel proto);

  public void ExportModel(SearchMonitorVector monitors, SWIGTYPE_p_operations_research__CpModel proto);

  public void ExportModel(SearchMonitorVector monitors, SWIGTYPE_p_operations_research__CpModel proto, DecisionBuilder db);

Which assumes I can create a new instance of SWIGTYPE_p_operations_research__CpModel, which I cannot in the current release.
For brevity I will use the simplest ExportModel overload. I'm thinking the C++ needs to change as such:
void Solver::ExportModel(CpModel* model_proto) const;

And the SWIG would look something like this, methinks:
%apply CpModel* OUTPUT { CpModel* model_proto };
  void ExportModel(CpModel* model_proto);

Which if I understand the SWIG bits, should yield this:
void ExportModel(out SWIGTYPE_p_operations_research__CpModel proto)
{
    // ...
}

Or like this:
CpModel* Solver::ExportModel() const;

Which I think would just return the instance outright:
SWIGTYPE_p_operations_research__CpModel ExportModel()
{
    // ...
}

That's the export side of things. Of course, what to do for the load/import side of things?
I'm going to poke around a little, but interested in receiving some feedback along these lines in order to get it resolved.

@lperron There's also the matter of how the SWIG treatment of the Protobuf ::google::protobuf::Message happens. It seems to me the *message-iness, so to speak, gets lost in translation. Perhaps it is not important for sake of the SWIG translation, unless we discover that it is advantageous to treat things of type Protobuf Message in a consistent manner. Never minding the fact that a public ctor is not exposed, so somehow at minimum at least internal treatment needs to happen.

@lperron Supposedly this would work, at least according to the Using typemaps documentation, unless there is something masking the "out" keyword...
%include <typemaps.i>

Followed by this when it is important:
%apply CpModel *OUTPUT { CpModel *proto };
Solver::ExportModel(CpModel* proto) const;

It will also be important to reorder the CpModel* proto as the last parameter in the list on account of the C# language out parameter rules. That's crucial for the DecisionBuilder overload.
void ExportModel(const std::vector<SearchMonitor*>& monitors,
                 DecisionBuilder* const db, CpModel* proto) const;

However, I try this mapping application and does not seem to have the desired effect, or as I mentioned the out keyword is somehow otherwise being masked.
The Solver.cs is still reflecting this:
public void ExportModel(SWIGTYPE_p_operations_research__CpModel proto)
{
    operations_research_constraint_solverPINVOKE.Solver_ExportModel__SWIG_0(swigCPtr, SWIGTYPE_p_operations_research__CpModel.getCPtr(proto));
}

Next visit will be to the Makefiles, methinks. Perhaps there is a SWIG option that is forcing the issue one way or another.

@lperron Reviewing the supporting SWIG files. I'm not sure why this is being done, particularly in a "shared" include across potentially differing language capabilities.
%typemap(out) std::string
%{ $result = jenv->NewStringUTF($1.c_str()); %}

But for language specific blocks such as the following, i.e.:
#ifdef SWIGJAVA
// ...
#endif

This could really be re-factored to separate SWIG files to help with through and through clarity.

@lperron Any thoughts as to upgrading the version of SWIG dependency? Currently version swigwin-3.0.10.zip however 3.0.12 is available.

@lperron I noticed I was working with python\constraint_solver.swig. Reverted that and added the parts I figured were important in the csharp\constraint_solver.swig, but still not the desired effect.
At the moment, it's a delicate operation to say the least, wrapping the call in an appropriate invocation:
namespace operations_research {

class CpModel;

%typemap(cstype) const CpModel* "$csclassname"
%typemap(csin,
         pre="  $csclassname temp$csinput = new $csclassname();"
        ) const CpModel* "$csclassname.getCPtr(temp$csinput)"

%typemap(cstype) CpModel* "out $csclassname"
%typemap(csin,
         pre="  $csclassname temp$csinput = new $csclassname();",
         post="  $csinput = new $csclassname(temp$csinput, false);",
         cshin="out $csinput"
        ) CpModel* "$csclassname.getCPtr(temp$csinput)"

} // namespace operations_research

Far from ready right at the moment, but it's a start, I think.

@lperron I'm starting to wonder if such a direct path to exported and re-loaded models is not the right answer. Not a wrong answer, per se, but whether there is a better answer. Sorry if my technical vocabulary and/or grammar isn't conveying my thoughts correctly here. It is not quite second hand yet, so all the SWIG and bouncing between languages is a bit clumsy still.
This is a bit of a stumbling block for me. I see how the model.proto and so on are being gen'ed up. The CpModel message, in this case, lands as a fully featured, sort of boilerplate class, complete with strongly typed message fields. All well and good so far.
Then there is the SWIGTYPE_p_operations_research__CpModel SWIG adapter, which I really do not get at all. There is no mention of anything to do with the underlying CpModel, per se, not even a PInvoke of any variety, which I would half expect. Only a loose System.IntPtr pointer variable, for instance. In the baseline, at any rate, SWIGTYPE_p_operations_research__CpModel stands in to capture the reference to a presumably PInvoke-captured C++ language level pointer, and that's it. There is no other point of origin for the "instance" from the C# or .NET perspective. All well and good, fair enough.
It seems like to me there should be some way of capturing a fresh instance of the protobuf model. Like either return a new instance, like MakeModelProto, something like that, exposed by the Solver, which seems like would do the right thing from a C++ language level and SWIG perspective, or be able to create a new instance of SWIGTYPE_p_operations_research__CpModel outright. But then there is a lot of nastiness of how to inform the C# type of the appropriate PInvokable C++ bits. Perhaps the MakeModelProto would be the simplest answer, and perhaps several other bits supporting the actual saving/loading of the message as necessary.
Then there is the internal FLAGS construct going on: specifically, FLAGS_cp_export_file. This is currently set once, that I know of. Perhaps an export file could be specified, something like:
using (var s = new Solver())
{
    s.Parameters().ExportFile = pathToExportFile;
}

It seems  to me that ExportModel being called with every ProcessConstraints is a bit excessive. I have in mind to run a solver during a Windows Service session, for instance. Perhaps periodic exportation would be desirable, as well as just prior to stopping the service. But not much more frequent than that, I think.
But then, I'm not sure why LoadModel is not exposed. For instance, when the service starts. Load the most recent state, reconnect with variables, etc, and proceed from there.
Hope that makes sense... Thanks again.

@lperron I will close this one as it is fairly broad even for my purposes right at the moment. Will open new ones that are more specific.

If you want to store models, or send models from masters to workers, you can use MPS or proto file. If you do not need MPS, I would stick to proto files. Furthermore, we have code to make them work in all languages.

@lperron Where is the code to obtain a proto from Python? I only see ExportModelAsMpsFormat and ExportModelAsLpFormat (both of which are incomplete serializations) in the SWIG definition, and ExportModelToProto is not wrapped and therefore not available. So from Python the only serializations are incomplete. Furthermore there is no deserialization on the solution side.

You are right, the python interface for the linear solver to import/export
from/to a proto is just broken.
We will try to correct this in the next release.
Thanks
Laurent Perron | Operations Research |<EMAIL_ADDRESS>| (33) 1 42 68 53
00
Le jeu. 29 mars 2018 à 08:47, yingzong<EMAIL_ADDRESS>a écrit :

If you want to store models, or send models from masters to workers, you
can use MPS or proto file. If you do not need MPS, I would stick to proto
files. Furthermore, we have code to make them work in all languages.
@lperron https://github.com/lperron Where is the code to obtain a proto
from Python? I only see ExportModelAsMpsFormat and ExportModelAsLpFormat
(both of which are incomplete serializations) in the SWIG definition, and
ExportModelToProto is not wrapped and therefore not available. So from
Python the only serializations are incomplete. Furthermore there is no
deserialization on the solution side.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/google/or-tools/issues/325#issuecomment-377279339,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AKj17UvkCIc_RBGXLIsmeyOO5yfCyjaTks5tjQH1gaJpZM4MKRmr
.


[Request] Make A /Modtweaker or /Modtweaker Help Command

I would like to Request a /Modtweaker or /Modtweaker Help command, that would list all the Modtweaker Added commands.

Most of these are in /mt help. However, all of the commands are listed in there. Do note that most of them are WIP and in 0.7, a version that is not ready for the public yet.


Request databases to be added to gget

https://omnipathdb.org/

Nomadscores
genename.org
GWAS
GTEx
UCSC in-silico PCR
String
Chembl

https://github.com/pachterlab/gget_examples/issues/2

Some databases (GTEx, String, ChEMBL) can be accessed through gget opentargets added in gget version >= 0.29.0


DrumKit

Hi, have uploaded a great project of a Drum kit. It can be used with all the devices mobile and laptop and tablet too. I hope you merger it.

i have uploaded project hope you will merge it


Just looked at the project to understand what it does. Here are a few comments.

Suggestion to pin #1 and/or enable Github Discussions
This creates a user.js with an syntax errors
https://github.com/trytomakeyouprivate/Arkenfox-softening/blob/926cd50530d049ccdbca424b2bcbb03e4317e026/arkenfox-script.sh#L18
https://github.com/trytomakeyouprivate/Arkenfox-softening/blob/926cd50530d049ccdbca424b2bcbb03e4317e026/arkenfox-script.sh#L51
For what are those extra ""? Note this is a bash script and not a python script
https://github.com/trytomakeyouprivate/Arkenfox-softening/blob/926cd50530d049ccdbca424b2bcbb03e4317e026/arkenfox-script.sh#L31
LibreWolf patch?
https://github.com/trytomakeyouprivate/Arkenfox-softening/blob/926cd50530d049ccdbca424b2bcbb03e4317e026/arkenfox-script.sh#L32

Hi, the script was a total mess and yes It didnt work. I will look at your first issue, the other ones should be solved.
I made lots of things choosable now and I think I found an okay way to do this. It was a learning project for me, I did all that basically the first time.
It is now fundamentally changed and made simpler.

only one script patcher and chooser script, all you do is change the paths there
the chooser script should include all changes that are discussable, so nothing weird happens
many fixes
changes should not break things anymore

Thanks for the fix, forgot it was "//" and not "#". I changed the method to avoid needing "//" as it breaks sed somehow, maybe the symbols can be deactivated.

Yes the librewolf patch doesnt make sense, but I keep it there as Arkenfox doesnt have settings that are unneeded, so if they may add this config, it will directly be removed ;D

https://github.com/trytomakeyouprivate/Arkenfox-softening/blob/f4594a93f0aa4054cc4e871dfa8f59bfc311d175/arkenfox-script.sh#L89-L116

The first printf prints the unit and it's path to stdout but does not create any file
The second printf creates a timer unit in a file named .service staring itself.

It was a learning project for me, I did all that basically the first time.

There's still a lot to learn.

Thanks, fixed it now!
Anything else missing? Maybe bad coding habits? I think it works quite well now, but still if something changes maybe other things break too, for example all these fixed deletion lines.

https://github.com/trytomakeyouprivate/Arkenfox-softening/blob/debf051df1915264e9964ccbecf01c862e8f82ac/arkenfox-script.sh#L30
This will add a syntax error in the user.js

Hi, the code was a complete mess. Should be cleaned up now.


Add tests generated by skeleton

Add tests generated by skeleton

:x: coverage decreased from 100% to 46.72%


Output file plugin not resolving tag name

Describe the bug
When using tag in the path for out_file, the variable does not get replaced with its value.
To Reproduce
   <match **>
      @type file
      format json
      path /containerlogs/test_${tag}.log
      <buffer tag>
        flush_interval 5s
      </buffer>
    </match>

Going by the document https://docs.fluentd.org/configuration/buffer-section#placeholders
I expected path /containerlogs/test_${tag}.log to become path /containerlogs/test_sometag.log
Instead, the file is created with just the text string without replacing the variable.
$ ls /containerlogs/
test_${tag}.log.20210729.b5c845fc991babff3

Expected behavior
$ ls /containerlogs/
test_<sometag>.log.20210729.b5c845fc991babff3

Your Environment
- Fluentd version: image: k8s.gcr.io/fluentd-gcp:1.30
- TD Agent version:
- Operating system: RHEL8.2
- Kernel version: 4.18.0-240.22.1.el8_3.x86_64

Your Configuration
   <match **>
      @type file
      format json
      path /containerlogs/test_${tag}.log
      <buffer tag>
        flush_interval 5s
      </buffer>
    </match>

Your Error Log
2021-07-29 16:56:09 +0000 [warn]: parameter 'flush_interval' in <buffer tag>
  flush_interval 5s
</buffer> is not used.

2021-07-29 16:56:09 +0000 [info]: following tail of /var/log/containers/csi-driver-node-h5t7w_csi-5611cbe0953b680489c67fc1dc025aa133bfaeaa7eec891463e52e474a8cd9d2.log
2021-07-29 16:56:09 +0000 [info]: following tail of /var/log/containers/csi-driver-node-h5t7w_csi-node-driver-registrar-a07890c431bbf51b0168b4e8a03441734f33a5e74aabdd9eba3952abd7d9ad51.log
2021-07-29 16:56:09 +0000 [info]: following tail of /var/log/containers/csi-driver-controller-csi-provisioner-11b4571c6d549054e7db89bcc6ac94da19f9dd1b90919acc18830303be692c0a.log
2021-07-29 16:56:09 +0000 [info]: following tail of /var/log/containers/csi-driver-controller-csi-attacher-4da17a3653ed957c5e46bf02bbccf76cc8fcb595cb35d774fdbe8fe45d91a7d7.log
2021-07-29 16:56:09 +0000 [info]: following tail of /var/log/containers/csi-driver-controller-0_csi-resizer-be02d609663933966492083f7396acadcbc777421464f6e56a61eb894a51d24d.log
2021-07-29 16:56:09 +0000 [info]: following tail of /var/log/containers/csi-driver-controller-0_csi-snapshotter e7dd56d9ca71d23de5afb845765d15c9d610cbf617668a7f464f4164912f0517.log
2021-07-29 16:56:09 +0000 [info]: following tail of /var/log/containers/csi-driver-controller-0-plugin-8069ed10cb71ab7b5111bbf5d79ddbd224360753a439ee51c6d037b2a3ac0826.log

Additional context
No response

Realized that this was happening because the version I was using was v0.12, and the documentation I referred was for v1.0. Updating the fluentd version solved the issue.


Incredible startup times on large files, pipe to stdout

Startup time on large files is incredible
 time wc myCsv.csv
   208725  12180461 663641644 myCsv.csv

real    0m5.915s
user    0m5.862s
sys     0m0.053s

vs.
 time micro myCsv.csv

real    0m3.104s
user    0m1.478s
sys     0m0.431s

both more or less cold.
All around performance for me recently has been off the charts, it's become my go to csv and tsv editor for large files thanks to the rock solid stability and textfilter.  Could not be happier with a tool I use, thank you team!
Pipe to stdout
I use an editor in pipe fairly often.  One liner demonstartion would be:
sometemp=`mktemp`; seq 5 > $sometemp; micro $sometemp; cat $sometemp | figlet

The correct way is something like vipe or this or this.
My actual workflow will generally be something along the lines of:
someCsvToolHere someCommand myfile.csv
exit(1): No, just no.   Just saying.

someOtherTool myFile.csv | editorinpipe `# manual labor` | someCsvToolHere someCommand -

I could imagine it going something like
  micro myFile -2stdout
 someCommand myFile | micro -- -- | someOtherCommand
Again, thank you.  Micro is a gem, very grateful for it.

Awesome I'm happy micro is performing well!
You may be glad to hear that micro actually already supports being in a pipe chain. If stdout is connected to a pipe, when micro quits it will send the data of the open buffer to the pipe.
For example, trye ifconfig | micro | cat.

I'm happy that this already exists, though sad I broke something on my end.  Not surprised though, I've done terrible unspeakable things to this computer.
SVG Demo of my failures
Just tried as you described by installing via the package manager on Termux on android and it worked no issue.  Still no go even when sshing into another machine (the same binary however) or by running from within a docker container using ubuntu latest and the package manager copy of micro.
Potentially window manager related, or some other peculiarity of my setup which needs fixing.  Feel free to close this for now and I'll reopen it if I do some testing and can trace it to something specific with micro.  I am perfectly content as it is now, and
             ██
            ░░
 ██████████  ██  █████  ██████  ██████
░░██░░██░░██░██ ██░░░██░░██░░█ ██░░░░██
 ░██ ░██ ░██░██░██  ░░  ░██ ░ ░██   ░██
 ░██ ░██ ░██░██░██   ██ ░██   ░██   ░██
 ███ ░██ ░██░██░░█████ ░███   ░░██████
░░░  ░░  ░░ ░░  ░░░░░  ░░░     ░░░░░░
                         ██
                        ░██
 ██████  ██████   █████ ░██  ██  ██████
░░██░░█ ██░░░░██ ██░░░██░██ ██  ██░░░░
 ░██ ░ ░██   ░██░██  ░░ ░████  ░░█████
 ░██   ░██   ░██░██   ██░██░██  ░░░░░██
░███   ░░██████ ░░█████ ░██░░██ ██████
░░░     ░░░░░░   ░░░░░  ░░  ░░ ░░░░░░

I recommend installing via the released binaries on Github rather than package managers, because the package manager versions are sometimes out of date. This is a relatively recent feature (new with version 2.0.3).

Awesome!


Profanity filter

Description
This PR contains a custom profanity filter. The profanity_phrases_words directory contains two 2 CSV files, readme.MD, and the script for flagging transcription texts.  Initially, the profanity filter used profanity_filter and profanity_check libraries. However, these filters don't have an exhaustive pull of words. Also, they suffered from the "Scunthorpe effect" where words that contained profanity were flagged --which gives false positives. This function seeks to address the two problems.
Loom video:  https://www.loom.com/share/9236075a79d747a49afedd5b65cada75
Type of change

[ ] Bug fix (non-breaking change which fixes an issue)
[ X] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

Change Status

[X ] Complete, tested, ready to review and merge
[ ] Complete, but not tested (may need new tests)
[ ] Incomplete/work-in-progress, PR is for discussion/feedback

Has This Been Tested

[ X] Yes
[ ] No
[ ] Not necessary

Checklist

[X ] My code follows the style guidelines of this project
[X] I have performed a self-review of my own code
[X] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes
[ ] There are no merge conflicts

Thanks, @cryptobellum for pointing that out to me. I will get it fixed.


Dependency Version Detection

Currently when installing plugins via the GPM dependencies are installed with the most up to date version, or if already installed does not check for a minimum version. For example the OAuth plugin will require that the login plugin is updated to v2.x.x

Yes, this is a known issue and something we intend to add.
it's actually already mentioned in the Grav 1.X section of the 2016 plans: https://getgrav.org/blog/plans-for-2016

also related to #563 so closing as we know about this already :)


Add new Linesource and MovingLinesource

add a new source class that has poin sources arranged in a line

mentioned in commit c84eba4297bfe39dbf0138ceebda0c8236baa264


allow default font of containers to be changed

Containers don't actually have default fonts. Widgets use default_font when created


Data transformation

Thanks for your great work!
I am reading some registers that need a factor and an offset to be applied to the register value. Is there any way to achieve this through modpoll in the configuration?

Hi Daniel, current version only support applying a ratio but not offset.
However, if you use MQTT as interface, I would recommend you to process data by subscribing on raw data topic and publish processed data to new topics. By doing this way, you always keep the first hand raw data and have the flexibility to apply different algorithms or even chain them as a pipeline.

Node-RED is great for this!


MINOR: [R] Fix broken link in R developer setup vignette

The link in the final section of the Developer environment setup docs 404s and this points the link to the installation details vignette. I think this is what was intended but let me know.

Is there a good way to automate checking of links in Rmarkdown? Possibly one could do this in the generated HTML, but maybe checking when parsing maybe more efficient.

Thanks for the look @thisisnic. I made the change to the link text in https://github.com/apache/arrow/pull/14443/commits/97015594f4cab41eb81c8c65cbe23cc36773c8eb. This should be good to merge.


look into the memory leak

As mentioned in this PR #74 , there is a memory leak concern on GPU. check if this is solved and add tests

@nathanrchn


yices: drop symlink hack, avoid ldconfig on linux

Reported by sternenseemann as a failure on darwin:
https://github.com/NixOS/nixpkgs/pull/141541#discussion_r739634263
Instead of emulating symlink creation in .nix file let's create
them in Makefile.build directly in a way that can be upstreamed.

Motivation for this change
Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
21.11 Release Notes (or backporting 21.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

[ ] Fits CONTRIBUTING.md.

Proposed the same fix upstream as https://github.com/SRI-CSL/yices2/pull/389

Thanks for testing @r-burns!


Update CLI

Summary:
Updated CLI
Issues:
https://github.com/rancher/rancher/issues/17540 Chart answers cannot be reset
https://github.com/rancher/rancher/issues/17405 Cannot add answers to chart

lgtm


[PROJECT ADDITION] Autism Identification System

Deep Learning Simplified Repository (Proposing new issue)
:red_circle: Autism Identification System :
:red_circle: Aim : This project aims to address the need for a time-efficient and accessible ASD screening tool using machine learning techniques.

:red_circle::yellow_circle: Points to Note :

The issues will be assigned on a first come first serve basis, 1 Issue == 1 PR.
"Issue Title" and "PR Title should be the same. Include issue number along with it.
Follow Contributing Guidelines & Code of Conduct before start Contributing.

:white_check_mark: To be Mentioned while taking the issue :

Full name : Pavitraa G
GitHub Profile Link : @pavitraag
Email ID<EMAIL_ADDRESS>What is your participant role? GSSOC 24 Contributor

Hi @pavitraag can you share the dataset URL and the models you are planning to implement here?

Models used are:
Logistic Regression
Support Vector Machine
Random Forest Classifier

Models used are:

Logistic Regression
Support Vector Machine
Random Forest Classifier

Data
Data.csv

Machine learning models will not work here, you need to implement deep learning models for this problem statement.
Looking forward to hearing from you.

I would like to add ANN model in it

@abhisheks008 I would like to add ANN model in it

Need to be brief with the proposed approach. At least add 3-4 deep learning arechitectures for this problem statement.


Use Maven Wrapper for building trevas-jupyter

Maven wrapper ensures that developer and CI pipeline use always the same version of the Maven build tool. This is useful to make the build reproducible.
You can find more information here:
https://maven.apache.org/wrapper/
To configure the Maven project with Maven wrapper initialize once for the project
mvn wrapper:wrapper

Commit all files that the command creates.
Afterwards each developer can simply use the mvnw, e.g.
mvnw package
It will check if Maven is installed, if not then it will fetch it and provide the specified version in the wrapper conf file. If it is then available it will run simply maven with the parameter specified (in the example: package)

Good idea.
Could you submit a PR?

Fix 936295c555608cc0241a40cc6cfe9521c3296063


CHANGE:issue#44 Agentless and port-forward by default

issue #44
change the mode to agentless and port-forward default.
if user want to close agentless or port-forward mode, should use
--agentless=false --port-forward=false

Thanks @calmkart !


⚠ Visual-Crew_Builder has degraded performance

In 61eacbf, Visual-Crew_Builder (https://visual-crew-builder.ai) experienced degraded performance:

HTTP code: 200
Response time: 1299 ms

Resolved: Visual-Crew_Builder performance has improved in 24fa0b5 after 14 minutes.


missing staging url

https://github.com/Crossmint/embed/blob/b4be4540dde56d67f3a850128ff2425680ea92ac/src/types/config.ts#L44

Fixed in https://github.com/Crossmint/embed/pull/6


New gun + katana

Can not attack using new gun and the rusted katana

can you stop.


Issue of playing vimeo vidoe with videogular

Hi
I am using angularjs on front end. I have implemented videogular for video cue points. I want to play vimeo video with videogular. I am using "videogular-vimeo" plugin for this. But when i trying to play vimeo video with videogular, it is not playing vimeo videos. Can anybody resolve my problem or videogular itself provide my support?

Please file an issue with the 3rd party videogular-vimeo project.


macOS Sierra: No library after upgrade

Hi, just upgraded to Sierra, and Radiant Player seems to be broken. It can open and show the window frame, but there are no contents, nothing in my library, and I'm unable to search for anything. It's just blank.
This worked yesterday, before the Sierra upgrade.

I'm running Sierra without any issues, so we should be able to get this figured out.  Can you right-click on the blank space, pick Inspect Element, and post back with a screenshot of the console tab?

Oh, and if you're getting the "Can't load library" error, can you sign out from within Radiant and sign back in?  That fixes that particular error.

Oh, thanks, logging out and logging back in worked. I'll try to search the closed bugs too next time; didn't realize I'm 6 months behind on this update 😁

Awesome, glad it's working for you!


Add did-pkh-poly

Adds did-pkh-poly method.
Signed-off-by: Tiago Nascimento<EMAIL_ADDRESS>

Should I update the PR to follow #252?

@theosirian I think that change can be addressed in #252, assuming this PR is merged first

@theosirian commit suggestion and GTG?


LST error fixed

What's changed?
In the provided code, I made the following changes:
Updated defaults Method:
Added spec.typeValidationOptions(TypeValidation.none()) within the defaults method. This change disables type validation during the rewriting process. Here's the updated snippet:
@Override
public void defaults(RecipeSpec spec) {
   // Apply MethodParamPad recipe and disable type validation
   spec.typeValidationOptions(TypeValidation.none());
   spec.recipe(new MethodParamPad());
}

Explanation:
By adding spec.typeValidationOptions(TypeValidation.none()), we ensure that OpenRewrite does not perform type validation checks during the rewriting process. This addresses the issue you mentioned regarding the java.lang.IllegalStateException related to missing or invalid type information when rewriting J.MethodInvocation arguments of type Map.
Impact:
This change should help in running your tests (rewriteRun-tests) without encountering the aforementioned exception. It focuses on allowing the rewrite operations to proceed without validating the types strictly, which appears to be triggering the error in your specific scenario.
Remaining Code:
The rest of the MethodParamPadTest class remains unchanged, including the test methods (addSpacePadding, removeSpacePadding, etc.), which continue to validate various formatting scenarios related to method parameter padding.
By incorporating this change, you should be able to proceed with your rewrite tests smoothly, as it mitigates the issue related to type validation in OpenRewrite 8.28.1.
What's your motivation?

Anything in particular you'd like reviewers to focus on?

Anyone you would like to review specifically?

Have you considered any alternatives or workarounds?

Any additional context
Closes issue #4280
Checklist

[ ] I've added unit tests to cover both positive and negative cases
[ ] I've read and applied the recipe conventions and best practices
[ ] I've used the IntelliJ IDEA auto-formatter on affected files

Hi @Riyazul555 ; Appreciate you looking to help out. I don't quite see how the change you've done here related to the issue logged under

#4280

The change you've done here is listed as a workaround, not as an actual fix. And you've applied that workaround to a test that also passes without this in place. So while I appreciate you looking to help out, I don't think this change is appropriate. Please provide more information if I've somehow missed the relation.
If you'd like to help out in some other way, then we have a label for good first issues, that might help you get more familiar with the project.
I'll close this PR for now; we can reopen if we learn more.


Add threads reaper

In some situations using threaded way of reporting errors when main process exits payloads are not delivered because all threads are killed straightaway.
For example rake tasks:
Rollbar.use_threaded

task :foo do
  begin
    raise 'bar'
  rescue => error
    Rollbar.error(error)
  end
end

Another way is to add a method that would do something like this code, but it looks ugly for me:
begin
  tmp = Rollbar.configuration.use_async
  Rollbar.configuration.use_async = false
  yield
ensure
  Rollbar.configuration.use_async = tmp
end

I'd like to discuss it and when it's ok I'll add some spec and do some refactoring if it's needed

Hey @amenzhinsky, thank you very much for this PR. There is one point I'm concerned about: spawn the reaper always this class is loaded, even it's not used as async handler.
I think we should create this thread only if the Rollbar::Delay::Thread is being used. For this we can spawn the reaper thread if it's not spawned on first .call(). What do you think?
About refactoring, just take a look at https://codeclimate.com/github/rollbar/rollbar-gem/pull/405, since the spawn method is too long. We can split it into 3 methods: spawn_threads_reaper, build_reaper_thread, configure_exit_handler or similar.
We'll need to test this manually and we'll need specs, but this is a great work! Thanks

Hi @jondeandres, I updated the PG, I followed all your advises except  build_reaper_thread, I added ::reaper method that memoizes the reaper thread and call it in spawn_threads_reaper because I need access to it in ::configure_exit_handler it's implicit a little. What do you think?

@jondeandres done, please have a look

@amenzhinsky the PR is great now. I will need to play a little with this and check everything is working as expected.
Feel free to write and fix the specs for this.
Thanks!


provide better wrapper than ArrayIterator for user data

This is related to:
https://github.com/NervanaSystems/neon/issues/249
take /neon/examples/timeseries_lstm.py for example:
in DataIteratorSequence:
    self.X_dev = self.be.iobuf((self.nfeatures, time_steps))
    ....
    self.X = self.X.reshape(Xshape).transpose(1, 0, 2, 3)

these code are related to device (GPU) memory implementation detail, the ordinary Neon user (e.g. me) knows nothing about how the GPU memory is organized, that's why I use a library such as Neon. Why should I concerned about this? especially those magic number in .transpose(1, 0, 2, 3), how do I know if I did the correct thing, or made a mistake?
The current ArrayIterator only can hold data in the memory, my data is:

dynamically genereted
doesn't fit memory

I hope Neon provide better wrapper for user data. This will help many users like me.
Thanks.

Related discussion here:
https://groups.google.com/forum/#!topic/neon-users/hMTRAT4Hf4M
￼==================================================================
<EMAIL_ADDRESS>￼￼Jun 24
￼Hello,
I am developing a network that is trained using reinforcement learning, and I need to input two streams of data into the Model.fprop, which then get merged after a sequence of layers before the output. At each time step, I have an MxM image and an N dimensional state vector. I need to load them separately, so that the image can be run through a sequence of convolutional layers before being merged with the state vector for the fully connected layers. I have checked the mnist_merge.py and mnist_branch.py examples, but they don't address the issue of loading two different data vectors simultaneously. Is there a way to load data like this?
Thank you,
Josh
￼==================================================================
On Friday, June 24, 2016 at 5:04:06 PM UTC-7<EMAIL_ADDRESS>wrote:
Hi Josh,
Thanks for your question. Take a look at our image_caption.py example that demonstrates how to use neon's MergeMultistream container for handling the merging of multiple streams of data.
Hanlin
￼==================================================================
Are you referring to
https://github.com/NervanaSystems/neon/blob/master/neon/data/imagecaption.py#L217
            prob = model.fprop((x[0], y), inference=True).get()[:, :-self.be.bsz].copy()

It feed a tuple to model.fprop. and y is init-ed to:
    y = self.be.zeros(self.dev_X.shape)

But in Neon doc: http://neon.nervanasys.com/docs/latest/generated/neon.models.model.Model.html?highlight=model.fprop#neon.models.model.Model.fprop
it says:
fprop(x, inference=False)[source]
Forward propagates a minibatch x through the model.
Parameters:
x (Tensor) – Input minibatch data
and if you click Tensor, it refers to neon.backends.backend.Tensor
I think there are 2 issues here:

the Neon doc need to be more accurate; it clearly not only accept Tensor, but also some tuple type, (maybe also other types?)
all these self.be.zeros, self.be.iobuf backend device types are difficult for library user to master. I guess Neon choose to do it this way for performance reason. But as library user, I don't want to know such implementation detail. I just want to prepare my data as (MxM matrix, N vector) in this particular example, and pass it to model.fit() or model.fprop(). I think Neon should provide a better API in this case (maybe via s thin wrapper which I don't care as a library user).

Hello,
Supplying a tuple instead of a tensor is used with multiple input/outputs. We will amend the API to reflect this. Thanks for bringing this to our attention.
If your data is being generated dynamically for each minibatch, then I'm afraid the only route is to write a custom dataset object that will load the generated data into the GPU and provision those buffers to the model. We've developed a notebook here to guide users through this process.
I am closing this issue for now. Feel free to reopen if you have issues implementing the above.


Ship Wikimedia UI and its dependency (bootstrap)

I haven't tested it (yet)

PSA: Do not clik on the "Files changed" link above :)   I did and it locked up my computer.

https://www.google.de/webhp?sourceid=chrome-instant&ion=1&espv=2&ie=UTF-8#q=PSA+meaning
In order to prevent LOLs in the future PSA stands for "public service announcement"

This change was addressed in https://github.com/wiki-ai/ores/pull/149


feat: Added filter blogs option

Title and Issue number
Title : Added filter blogs option
Issue No. : #1206
Code Stack : HTML, CSS and JS
Close #<1206
Description
Added
- filter functionality to blog categories
- a reset button
- smooth transition between filter and reset
Video/Screenshots (mandatory)
https://www.awesomescreenshot.com/video/29952216?key=de10ee3a45170d004bec0c814de7cd57
Type of PR

[ ] Bug fix
[x] Feature enhancement
[ ] Documentation update
[ ] Other (specify): _______________

Checklist:

[x] I have performed a self-review of my code.
[x] I have commented my code, particularly in hard-to-understand areas.
[x] I have tested the changes thoroughly before submitting this pull request.
[x] I have provided relevant issue numbers, screenshots, and videos after making the changes.
[x] I have gone through the  contributing.md file before contributing

##Are you contributing under any Open-source programme?

[x] I am contributing under GSSOC'24
[ ] I am contributing under VSOC'24

@apu52 please check. It was a very difficult function to implement keeping the transition smoothness in mind


Reword text instructions for first forum post

Pre-Submission Checklist

[x] Your pull request targets the staging branch of FreeCodeCamp.
[x] Branch starts with either fix/, feature/, or translate/ (e.g. fix/signin-issue)
[x] You have only one commit (if not, squash them into one commit).
[x] All new and existing tests pass the command npm test. Use git commit --amend to amend any fixes.

Type of Change

[x] Small bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds new functionality)
[ ] Breaking change (fix or feature that would change existing functionality)
[ ] Add new translation (feature adding new translations)

Checklist:

[ ] Tested changes locally.
[ ] Closes currently open issue (replace XXXX with an issue no): Closes #XXXX

Description

As originally worded, many new campers seemed to be getting the idea that they should create their own introduction thread instead of replying to the main one linked to in the challenge.  This slight rephrasing will (hopefully) make it more clear that they should rather be replying.

LGTM


Support new treesitter captures

Treesitter was recently updated, which breaks a few colors in vim-nightfly-colors.
Notice of breaking change:
https://github.com/nvim-treesitter/nvim-treesitter/issues/2293#issuecomment-1900679583
More information:
https://www.reddit.com/r/neovim/comments/19aratu/psa_nvimtreesitter_breaking_changes_on_highlight/
Screenshots
As an example, here are some screenshots of ruby code (this file) using nvim-treesitter pinned to 0.9.2 vs unpinned.
nvim-treesitter pinned to version="0.9.2":

nvim-treesitter with no version pin (using current master):

I now support those new capture names, whilst keeping support for the legacy capture names (since I myself still use stable Neovim 0.9.4).
Note, this was quite painful.
Please test it out and let me know if you encounter issues; I would not be surprised if have introduced a regression.
Hopefully this is the last time Tree-sitter highlight groups get renamed, this is now the 2nd time.
Best regards.

There are fewer colors than before, so there is probably leak.
https://github.com/nvim-treesitter/nvim-treesitter/blob/master/CONTRIBUTING.md#highlights

There are fewer colors than before, so there is probably leak.

Can you provide more details; please run the :Inspect command over the regression.

@uga-rosa,
I found and fixed a few errors. Update again; if there are still issues then run :Inspect on top of the regression.
Cheers.

It looks okay. Thanks for responding!


[language] eliminate recursion between pay_from_sender and create_new…

…_account in LibraAccount
This was confusing and unnecessary. Now, create_new_account creates a new account, and filling the account with some initial funds is a separate operation.

Motivation
Working toward accepting transaction metadata in pay_from_sender and ran into this odd code path. Refactoring eliminates a call to pay_from_sender and will make my transaction metadata change easier.
Have you read the Contributing Guidelines on pull requests?
Yes.
Test Plan
E2E tests, should be pure refactoring.

@bors-libra r=tnowacki

:pushpin: Commit 237289f has been approved by tnowacki

:hourglass: Testing commit 237289fd3692223ee07e7c46c0ca587a8618b351 with merge 1216a1374e3718e64573e85b9ded435107a9860b...

:broken_heart: Test failed - checks-circle_commit_workflow

we need to update genesis for terraform too which I don't see in this diff.
This is also a breaking change we want to document given it will break the whitelisted transactions, right?

we need to update genesis for terraform too which I don't see in this diff.
This is also a breaking change we want to document given it will break the whitelisted transactions, right?

Agreed. We do need to upgrade the terraform genesis. (I guess that is why storage_integration_test was failing here.)

I initially did not realize that I would need to change the create_account script and thus break everything...
I wanted to limit the scope of this diff to the refactoring needed to support tx metadata; doesn't seem like a good idea to combine that with breaking changes to scripts. Let me do things a bit differently so I don't need to change the script.

:umbrella: The latest upstream changes (presumably 75c207d3f7a5a6a66f8fabaf10cdcae6ffb85c24) made this pull request unmergeable. Please resolve the merge conflicts.

@bors-libra r+

:pushpin: Commit 5f7ff15 has been approved by sblackshear

:hourglass: Testing commit 5f7ff158304a0c5864c62a4678eb224dd7ba2e08 with merge f14cbad4c403760e8da82c7e134dad2cc4cccf01...

:sunny: Test successful - checks-circle_commit_workflow
Approved by: sblackshear
Pushing f14cbad4c403760e8da82c7e134dad2cc4cccf01 to master...


Return the QR code in a string representation to device engagement

The current implementation of the library returns a QR code as a PNG image to device engagement. I suggest that the library return a string value instead. This change would increase flexibility, allowing users to generate QR images with the desired logos and resolution, or use the string in other ways. It would also make the API more consistent, since the Android library already returns a string for device engagement.

Change included in main branch and tag 0.3.7