[RFC] Document the process for requesting professional development

Proposal:
Document the engineering team's process for requesting training, conference attendance, and learning materials.
Reasoning
We want to have clarity in what types of professional development activities are reimbursed and how to request them. Clarifying this process will hopefully lead to increased requests to pursue opportunities for learning and growing in the engineering team, and a smoother request/approval process for both engineers and their managers.
How is this RFC resolved?
A playbook document outlining the current process for requesting professional development. This playbook should address questions that engineers and managers have about the process.

Please comment with any questions that you have about the current process 🙌

Great idea. Note the existing conferences playbook that could be expanded for this purpose.
I think it would be helpful to clarify:

criteria for reimbursed trainings/conferences (the playbook mentions "...should be related to the individual's role/growth or Artsy's challenges/technologies" but it may not always be clear).
and the exact approvals required.

👍 from me, echoing Joey's recommendation of expanding the conferences playbook.

Big fan of clarifying the process and expanding the playbook! I would find it helpful to have a clearly delineated set of steps, e.g.
You want to go to a conference! Great!

Find out how much the conference costs, including an estimate for any travel
Put this information in form XYZ and send it to your manager
If your manager gives their stamp of approval, book the tickets
Attend conference!
In the same month you booked the tickets, use Concur to submit a reimbursement request. Ask your manager for help if you haven't used Concur before or have questions
Write up your notes from the conference and share with the team in #slack-channel-that-may-or-may-not-exist

kinda thing

@mdole I like point 6 especially – I've started a folder in this repo to collect such conference notes: https://github.com/artsy/README/tree/master/conference-notes

These are great resources, @joeyAghion and @ashfurrow. I'll make sure that the outcome of this RFC expands on existing documentation.
@mdole thanks for getting the ball rolling on a set of steps. I'll add some more details to what you've shared and include that as well.

I've kept this RFC open as a reminder to update the playbook, but since the next steps are clear, I will close it and move to update the playbook.

@iskounen We have a template for resolving an RFC which includes Additional Context and Next Steps. Would you be able to comment again with that template filled out?
https://github.com/artsy/README/blob/master/playbooks/rfcs.md#resolution

Sure thing! I'll update my previous comment to match the template.


Added Support For Zapache monitoring script

This adds the zapache monitoring script to the puppet zabbix module.
The class was added in zapache.pp and the required files are in a directory called "zapache" under files.
Although this isn't fully automated (templates must be added to zabbix manually for now) it should work fine for most purposes.

Thanks! :+1:

@dj-wasabi
Are you sure this was successfully merged?
"Closed with unmerged commits"
It says that on the pull and I don't see the files in the main repo.
~ef

Hi,
Sorry, I pushed the wrong button! :-)


CBL-2912 : Update LiteCore, Fleece Headers, and Export Symbols

Updated LiteCore to the lastest master

Used new Fleece Headers and excluded the expert headers from the installing files

Updated exported symbols

I'm using some of the functions which are now in the expert headers. Since they were not marked experimental or volatile, removing them from the API would be a breaking change. I might be the only one using them, though.

I also use "expert" function FLValue_FromData from fleece/FLExpert.h.

@blaugold @Dushistov Can you share your use case for the FLValue_FromData?

I'm developing a Couchbase Lite SDK for Dart and Flutter and making extensive use of FLEncoder. To read data that was encoded through a FLEncoder I'm using FLValue_FromData,  but I could just as well use FLDoc_FromResultData for that. I have also found FLData_Dump quite helpful for debugging.
Further, I've been using FLSharedKeys in benchmarks for de/encoding between Fleece and Dart objects to get results that are comparable to de/encoding database document data.
I can live without all of that functionallity if I have to.

FLDoc_FromResultData is what is recommended to use instead of FLValue_FromData. I could see the benefit of the FLData_Dump, but FLSharedKeys API is kind of out-of-context for CBL's users in general. I will discuss with the team about your feedback. Anyway, fyi, we are not changing anything in the Lithium branch

but FLSharedKeys API is more an internal API for CBL's users in general.

I can see that, and removing it's not breaking anything essential on my part.

FLDoc_FromResultData is what is recommended to use instead of FLValue_FromData

But FLDoc_FromResultData is completely different function.
FLValue_FromData get input as slice and return FLValue that actually pointer to this slice.
While FLDoc_FromResultData accept allocated slice, and allocate Doc.
So for each de-serialization it requires two extra allocations, may be more.
Plus  FLDoc_FromResultData there is need for FLDoc_GetRoot, plus FLDoc_Release.

@Dushistov your understanding is not wrong in that perspective; using FLDoc_FromResultData has some additional allocation and requires releases.
However, the FLDoc_FromResultData is recommended as it's more safe to use in general as the fleece data will be retained by the FLDoc and as long as the FLDoc is retained, the FLValue pointing to the fleece data will always be valid.

I have created https://issues.couchbase.com/browse/CBL-2921 to see if we want to export some of the API discussed in this PRs.

@Dushistov What are you using FLValue_FromData for? and does FLDoc_FromResultData cover your needs besides additional steps and allocs/releases?

What are you using FLValue_FromData for? and What is this Fleece data that you need to parse? Also does FLDoc_FromResultData cover your needs besides additional steps and allocs/releases?

I used FLValue_FromData for the same reasons as blaugold - deserialization, but instead of Dart I used Rust.
And I can wrap  FLValue_FromData  with function with lifetime annotations,  so Rust compiler during parsing can validate that FLValue and slice has the same lifetime, so there is no any safety concerns in my case.

Hi folks — I'm unclear why you need FLValue_FromData as part of serializing. Loading/parsing documents is already taken care of by CBL, so you shouldn't need to take a data blob and parse it as Fleece; instead you use the result of CBLDocument_GetProperties.

Also @Dushistov, if the library you're talking about is https://github.com/Dushistov/couchbase-lite-rust, then you're already groping inside CBL-C to access private "c4_) APIs (for example) so what's stopping you from also using FLValue_FromData once it becomes private?
To be clear, we obviously can't stop you from using private APIs, but we're not going to support it or make changes to accommodate it.

Hey @snej, as I said, I don't need to use FLValue_FromData and can easily switch to FLDoc_FromResultData.
I don't rely on any of the APIs removed in this PR such that removing them impacts me in a serious way.
Please correct me if I'm wrong, but I don't think they were marked as private or experimental, which is why I was surprised that they were removed without notice. I fully understand the intention behind removing APIs that are meant to be internal from CBL C, though.

@blaugold Sincerely apologize for this breaking change. Previously, fleece headers were not organized to separate the APIs that are meant for internal usages (known as expert APIs). Also we expected that CBL users don't need to deal with Fleece serialization / deserialization when using CBL-C API. If there are anything that you can't do without the expert APIs (in the scope of CBL), we could discuss further.

@blaugold I don't know dart and limitation or inefficiency around wrapping around FLDict or FLMutableDict versus implementing mutable fleece in dart. Does this also mean you need to serialize FLDict to return from C to Dart when calling CBLDocument_Properties(CBLDocument*)? Anyway, can you create a new ticket with what you need and proposal so we will keep other discussion out of this PR?

I have added the expert headers back. Note that the expert headers will be removed in the next major release.
Reference ticket: https://issues.couchbase.com/browse/CBL-2956


Wrong private IP assignment in Kubernetes workload

Description
Deploying Kubernetes using tfcmd with two workers on the same node result in private ips being equal, which should not happen.


data.md, driver.md, evm.md, verification.md: unify java and ocaml semantics some

Do not review yet

@ehildenb I can't just remove the powmod function as it is used by the MODEXP precompiled contract, and the behavior of returning zero when the modulus is zero is necessary there. Do you want me to change ^Word though to refer directly to ^%Int?

@dwightguth yeah I think it makes sense to switch the uses of powmod to _^%Int__ because essentially powmod has become just another name for _^%Int__. It will just add an (admittedly small) reduction step to go through powmod.

Fixed. @ehildenb can you re-review?

Jenkins: test this please


Add repr(transparent) to Reg struct

To guarantee that the register has the proper layout so that the cast of an integer to a *const RegisterBlock is sound.
Note that this is necessary but not sufficient.  The underlying VolatileCell also currently uses the default repr.  There is an open issue and PR to address the concern.

r? @ryankurte
(rust_highfive has picked a reviewer for you, use r? to override)


imdshift: init at 1.0.0

Description of changes
Tool to migrate workloads to IMDSv2
https://github.com/ayushpriya10/IMDShift
Related to #81418

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, as applicable:

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

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

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

[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 251116 run on x86_64-linux 1

2 packages built:

imdshift
imdshift.dist


journald log driver doesn't work as advertised

Description
No log outputs directly to journald and fields like CONTAINER_NAME are populated.  The only way to get logs in journal is via SYSLOG_IDENTIFIER
Steps to reproduce the issue
nerdctl run -d --privileged --log-driver=journald nginx
ac67ef179dd3 docker.io/library/nginx:alpine "/docker-entrypoint.…" 5 minutes ago Up nginx-ac67e
journalctl -a CONTAINER_NAME=nginx-ac67e
-- No entries --
docker logs 61a2f083b42c
^C
root@sbc21n1-mke:~# docker inspect nginx-ac67e
[
{
"Id": "ac67ef179dd34ce3146047728986c5247117d1faab3c7b2efcfbca3e39b6303d",
"Created": "2024-10-02T17:48:04.18807001Z",
"Path": "/docker-entrypoint.sh",
"Args": [
"nginx",
"-g",
"daemon off;"
],
"State": {
"Status": "running",
"Running": true,
"Paused": false,
"Restarting": false,
"Pid": 1704668,
"ExitCode": 0,
"Error": "",
"StartedAt": "2024-10-02T17:48:04.505471873Z",
"FinishedAt": ""
},
"Image": "docker.io/library/nginx:alpine",
"ResolvConfPath": "/var/lib/nerdctl/1935db59/containers/default/ac67ef179dd34ce3146047728986c5247117d1faab3c7b2efcfbca3e39b6303d/resolv.conf",
"HostnamePath": "/var/lib/nerdctl/1935db59/containers/default/ac67ef179dd34ce3146047728986c5247117d1faab3c7b2efcfbca3e39b6303d/hostname",
"LogPath": "",
"Name": "nginx-ac67e",
"RestartCount": 0,
"Driver": "overlayfs",
"Platform": "linux",
"AppArmorProfile": "",
"Mounts": [
{
"Type": "bind",
"Source": "/var/lib/nerdctl/1935db59/containers/default/ac67ef179dd34ce3146047728986c5247117d1faab3c7b2efcfbca3e39b6303d/resolv.conf",
"Destination": "/etc/resolv.conf",
"Mode": "bind,rprivate",
"RW": true,
"Propagation": "rprivate"
},
{
"Type": "bind",
"Source": "/var/lib/nerdctl/1935db59/etchosts/default/ac67ef179dd34ce3146047728986c5247117d1faab3c7b2efcfbca3e39b6303d/hosts",
"Destination": "/etc/hosts",
"Mode": "bind,rprivate",
"RW": true,
"Propagation": "rprivate"
},
{
"Type": "bind",
"Source": "/var/lib/nerdctl/1935db59/containers/default/ac67ef179dd34ce3146047728986c5247117d1faab3c7b2efcfbca3e39b6303d/hostname",
"Destination": "/etc/hostname",
"Mode": "bind,rprivate",
"RW": true,
"Propagation": "rprivate"
}
],
"Config": {
"Hostname": "ac67ef179dd3",
"AttachStdin": false,
"Labels": {
"io.containerd.image.config.stop-signal": "SIGQUIT",
"maintainer": "NGINX Docker Maintainers<EMAIL_ADDRESS>"nerdctl/extraHosts": "null",
"nerdctl/hostname": "ac67ef179dd3",
"nerdctl/ipc": "{"mode":"private"}",
"nerdctl/log-uri": "binary:///usr/local/bin/nerdctl?_NERDCTL_INTERNAL_LOGGING=%2Fvar%2Flib%2Fnerdctl%2F1935db59",
"nerdctl/name": "nginx-ac67e",
"nerdctl/namespace": "default",
"nerdctl/networks": "["bridge"]",
"nerdctl/platform": "linux/amd64",
"nerdctl/state-dir": "/var/lib/nerdctl/1935db59/containers/default/ac67ef179dd34ce3146047728986c5247117d1faab3c7b2efcfbca3e39b6303d"
}
},
"NetworkSettings": {
"Ports": {},
"GlobalIPv6Address": "",
"GlobalIPv6PrefixLen": 0,
"IPAddress": "<IP_ADDRESS>",
"IPPrefixLen": 24,
"MacAddress": "26:1f:36:d0:89:1f",
"Networks": {
"unknown-eth0": {
"IPAddress": "<IP_ADDRESS>",
"IPPrefixLen": 24,
"GlobalIPv6Address": "",
"GlobalIPv6PrefixLen": 0,
"MacAddress": "26:1f:36:d0:89:1f"
}
}
}
}
]
Describe the results you received and expected
journalctl -a CONTAINER_NAME=middle
...
Sep 15 17:41:22 sbc21n1-mke middle[38321]: Executing middle on middle
Sep 15 17:41:22 sbc21n1-mke middle[38321]: + exec /usr/local/registrator/bin/middle -c /usr/local/registrator/cfg/middle.cfg
Sep 17 13:43:51 sbc21n1-mke middle[149996]: + set -e
Sep 17 13:43:51 sbc21n1-mke middle[149996]: + trap stop_middle SIGTERM
Sep 17 13:43:51 sbc21n1-mke middle[149996]: + export LD_LIBRARY_PATH=/usr/local/registrator/lib:/usr/local/lib:
Sep 17 13:43:51 sbc21n1-mke middle[149996]: + LD_LIBRARY_PATH=/usr/local/registrator/lib:/usr/local/lib:
Sep 17 13:43:51 sbc21n1-mke middle[149996]: + umask 2
Sep 17 13:43:51 sbc21n1-mke middle[149996]: + chown xcast:xcast /usr/local/registrator/cfg/middle.cfg
Sep 17 13:43:51 sbc21n1-mke middle[149996]: + chmod 644 /usr/local/registrator/cfg/middle.cfg
Sep 17 13:43:51 sbc21n1-mke middle[149996]: + echo '<IP_ADDRESS> middle'
Sep 17 13:43:51 sbc21n1-mke middle[149996]: + cd /usr/local/registrator/run/middle
Sep 17 13:43:51 sbc21n1-mke middle[149996]: + [[ -f /usr/local/registrator/bin/middle ]]
Sep 17 13:43:51 sbc21n1-mke middle[149996]: + [[ -f /usr/local/registrator/cfg/middle.cfg ]]
Sep 17 13:43:51 sbc21n1-mke middle[149996]: Executing middle on middle
Sep 17 13:43:51 sbc21n1-mke middle[149996]: + echo Executing middle on middle
Sep 17 13:43:51 sbc21n1-mke middle[149996]: + exec /usr/local/registrator/bin/middle -c /usr/local/registrator/cfg/middle.cfg
lines 964-1022/1022 (END)
What version of nerdctl are you using?
Client:
Version:       v2.0.0-rc.2
OS/Arch:       linux/amd64
Git commit:    cfa7e5388f0d8fb4fe982a47c86e8f141c952d72
buildctl:
Version:      v0.15.2
GitCommit:    9e14164a1099d3e41b58fc879cbdd6f2b2edb04e
Server:
containerd:
Version:      v2.0.0-rc.4
GitCommit:    fa5bf66fdff44846dc475c022bf6b47197febffd
runc:
Version:      1.1.14
GitCommit:    v1.1.14-0-g2c9f5602
Are you using a variant of nerdctl? (e.g., Rancher Desktop)
None
Host information
Client:
Namespace:     default
Debug Mode:    false
Server:
Server Version: v2.0.0-rc.4
Storage Driver: overlayfs
Logging Driver: json-file
Cgroup Driver:  : systemd
Cgroup Version: : 2
Plugins:
Log:     fluentd journald json-file syslog
Storage: native overlayfs
Security Options:
apparmor
seccomp
Profile:     builtin
cgroupns
Kernel Version:   6.8.0-45-generic
Operating System: Ubuntu 24.04.1 LTS
OSType:           linux
Architecture:     x86_64
CPUs:             48
Total Memory:     503.3GiB
Name:             sbc21n1-mke.siptalk.com
ID:               5844d823-3362-49bf-b02e-225929245b00
WARNING: bridge-nf-call-iptables is disabled
WARNING: bridge-nf-call-ip6tables is disabled

Hi, @AkihiroSuda
Can I create a PR that also outputs CONTAINER_NAME in the journal log in nerdctl, like docker cli, if necessary?

Yes, thanks, it should just correspond to Docker's behavior

Was this fixed by @haytok with #3667?

@apostasie , Yes, this issue was resolved #3667 .


Passing variable notation in helpers

Hello and thanks for your (wonderful) plugin.
I'm working on a dynamic dialog that will popup using jsrender template and helpers.
I've this (simplified) code
<script type="text/x-jsrender" id="mytmpl">
  <p>{{:~GetLabel('label_key', '{{:my_variable}}')}}</p>
</script>

<script>
$.views.helpers({
    GetLabel: function (key, args) { return GetLabel(key, args, true); }
});
function GetLabel(key, args) {
  var value = null;
  $.ajax({
    /* ajax stuff */
  }).done(function (response) {
    value = response;
  });
  return value;
}
</script>

I've a "Label" controller in my WebAPI that search for "key" and, if match, try to perform a String.Format passing "args"
I try passing to my helper function either {{:my_variable}} and {{:my_variable}} with no luck
What I'm trying to obtain is to call server side label and put print command in that when I call template render
Is it possible in some way?
Thank you in advance
Stefano

If my-variable is a property of your contextual data object, then you can pass it to a helper like this
{{:~GetLabel('label_key', my_variable)}}

You never need to nest {{:...}} inside itself. The ... is where you put a code expression such as myProp + 'x', or ~GetLabel(1, 33, 'somestring', some.object.path).
But GetLabel() needs to return its result synchronously, so your ajax fetch result won't be returned correctly. You need to do the ajax fetch before rendering the template....


Improvement on administrator alerts

ISSUE TYPE

Improvement Request

COMPONENT NAME
Alerts

CLOUDSTACK VERSION
4.16.1 and onwards

CONFIGURATION
Use the default values for the configurations:

capacity.check.period : 300000
alert.purge.delay : 0
alert.purge.interval : 86400
Define notification thresholds for cluster capacity: (for example cluster.storage.capacity.notificationthreshold : 0.75 )

OS / ENVIRONMENT
N/A
SUMMARY
The first time a cluster threshold is exceeded CloudStack notifies the administrator by sending an email and creates a record on the alert table. If the alert.purge.delay value is set to 0 (default value) then this record is never deleted, so the next time the threshold is exceeded the administrator does not get notified. CloudStack queries the database and searches for records on the alert table for the same alert type and cluster, and in case it finds an existing records it does not send the notification email.
Example:
DB:
mysql> select * from alert where type = 3 and cluster_id = 1 and archived = false;
+-------+--------------------------------------+------+------------+--------+----------------+------------------------------------------------------------------------------------------------------------------+------------+---------------------+---------------------+----------+----------+-------------------------+---------------------------------------------------------------------------------------+
| id    | uuid                                 | type | cluster_id | pod_id | data_center_id | subject                                                                                | sent_count | created             | last_sent           | resolved | archived | name                    | content                                                                               |
+-------+--------------------------------------+------+------------+--------+----------------+------------------------------------------------------------------------------------------------------------------+------------+---------------------+---------------------+----------+----------+-------------------------+---------------------------------------------------------------------------------------+
| 28623 | 885be044-4507-4a18-b71a-44af87ada1cf |    3 |          1 |      1 |              1 | System Alert: Remaining unallocated Storage is low in cluster BNZ-CL-001 pod BNZ-POD-01 of availability zone BNZ |          1 | 2021-09-05 02:41:48 | 2021-09-05 02:41:48 | NULL     |        0 | ALERT.STORAGE.ALLOCATED | Unallocated storage space is low, total: 20971496 MB, allocated: 16332262 MB (77.88%) |
+-------+--------------------------------------+------+------------+--------+----------------+------------------------------------------------------------------------------------------------------------------+------------+---------------------+---------------------+----------+----------+-------------------------+---------------------------------------------------------------------------------------+
1 row in set (1.26 sec)

Logs:
2022-08-03 00:01:53,297 DEBUG [c.c.a.AlertManagerImpl] (CapacityChecker:ctx-166624fc) (logid:8e4ca7a5) System Alert: Remaining unallocated Storage is low in cluster BNZ-CL-001 pod BNZ-POD-01 of availability zone BNZ
2022-08-03 00:01:53,298 DEBUG [c.c.a.AlertManagerImpl] (CapacityChecker:ctx-166624fc) (logid:8e4ca7a5) Unallocated storage space is low, total: 62914488 MB, allocated: 50750692 MB (80.67%)
2022-08-03 00:01:53,298 WARN  [c.c.a.AlertManagerImpl] (CapacityChecker:ctx-166624fc) (logid:8e4ca7a5) alertType=[3] dataCenterId=[1] podId=[1] clusterId=[1] message=[System Alert: Remaining unallocated Storage is low in cluster BNZ-CL-001 pod BNZ-POD-01 of availability zone BNZ].

2022-08-03 00:01:54,255 DEBUG [c.c.a.AlertManagerImpl] (CapacityChecker:ctx-166624fc) (logid:8e4ca7a5) Have already sent: 1 emails for alert type '3' -- skipping send email

I would like to add this enhancement feature to this project. Kindly guide me through the process as I am a fresher in this field.

@gitsan13 have you looked at the hackerbook? (https://github.com/shapeblue/hackerbook) When you feel comfortable just submit a PR and we'll go from there.


Split off bevy_image from bevy_render

Objective

bevy_render is gargantuan

Solution

Split off bevy_image

Testing

Ran some examples

Adding this to the 0.15 milestone. I don't want this to wait around because of the merge conflict pain, and merging this just after a release will make point release fixes brutal.

I think I would prefer a more general crate than bevy_image, to be able to move other kinds of assets currently living in bevy_render like Mesh, rather than multiplying small crates like bevy_image, bevy_mesh, ...
maybe a bevy_render_assets?

I'm neutral on the two approaches, but I'd like to decide relatively quickly. I do like that the current split means 2D games don't have to pull in 3D mesh formats, but that can be feature-flagged within this crate.

There was more discussion on Discord, but we're moving forward with the bevy_image + bevy_mesh approach.


Is it possible to add a new example to the data module to use Spring Data Neo4j?

The Spring Data Neo4j module already has a reactive repository, ReactiveNeo4jRepository, but it presents a problem: there is no support for Criteria. A similar concept is used with Example<T> and ExampleMatcher, but it was complicated to add this module in place of MongoDB.
I would be very grateful if you could give me some advice or example.
Thank you @siyual-park
https://neo4j.com/docs/getting-started/current/languages-guides/java/spring-data-neo4j/

I have redefined my own Criterion and Repository interface to include all these changes. We looked inside implementations such as ReactiveRepository, found a direct query request method used by ReactiveRepository, and used a method of directly implementing Repository interface by converting Criteria into a specific query.
I don't use Neo4j, so I don't have any examples of implementation, but I think I'll add a new repository implementation like the r2dbc, mongo, in_memory implementation that exists on https://github.com/siyual-park/spring-webflux-multi-module-boilerplate/tree/master/module/data/src/main/kotlin/io/github/siyual_park/data/repository .

😌 Hello! I was trying to make my own implementation to use Neo4j, taking your code as a base, but I'm having some problems with some tests. I suspect they have to do with concurrency issues. Apparently, some tests fail to find the element in the Neo4j database when concurrent calls are chained to modify an element in the database. I don't know if you have time to look at my code since I don't know the reason why those two tests are failing.
@siyul-park the example tests is in this class and my PR is https://github.com/yacosta738/astrum/pull/21
Also, I opened an issue to the SDN guys to see if they can help me. The issue that describes the problem is: https://github.com/spring-projects/spring-data-neo4j/issues/2717
Any help or clue you can give me will be greatly appreciated. Thank you in advance.


[question] Backwards-incompatible changes

Hi,
I would like to create a rpm package for Fedora/EPEL as a dependency for some other packages, but it will be quite difficult because of a lot of backwards-incompatible changes. It's well documented (thanks for it), but don't you want to deliver more bugfixes/improvements without these incompatibilities? It's very hard and uncomfortable to use newer version when some eg. class is just renamed. It will be nice to have incompatibilities between a major versions (I can deliver new major version to a new version of distribution), not minor.
Thanks.

Hi @jakubjedelsky,
This is possible, yes. But there is a reason I choose not to do it right now.
The prompt_toolkit library is still very young (I think it started about two years ago). In many ways, the design has been changed over time, in order to make it as good as possible. Currently, I consider some of APIs unstable, because new use cases are starting to emerge and we want to be as flexible as possible. Unfortunately, right now, there is no way I will be able to support multiple branches. (Like backporting features or bug fixes to older versions, and doing multiple releases every time.) So, for now, it's best to consider every single release (each one that is still starting with a zero), to be a major upgrade, that is incompatible and cannot be done automatically.
By the way, in the Python world, we prefer to use pip to install packages, rather than rpm or deb packages, and pip is able to correctly install the right dependencies. (So, there's it's not much of an issue.)
However, I seriously intend to move to a version scheme with major/minor version numbers, where minors don't introduce backward-incompatible changes. But currently, I'm first experimenting with a few more projects that depend on prompt_toolkit and which will heavily challenge the design. I expect that a few more backwards-incompatible changes are possible, but hopefully in a few months, we can move to a version 1.0 and consider the API stable.
Jonathan

Hi Jonathan,
thanks for reply. I understand you, but your library is already used by a few projects and these can't be used together right now. I checked projects mentioned in readme and found, that I need 5 different versions of prompt_toolkit to use it (doesn't matter if pip or rpm/deb).
Well, I'll try to prepare newest version and if something will depend on it, one have to create patch. Simple enough :) Hope it will work for some time.
And hope you'll find your cool library mature soon, many thanks for your work!
J.

Speaking as one of the prompt-toolkit maintainers for Debian (where we packaged it because applications that were already packaged started using it), I share this concern.  I started looking at updating to the newest release in Debian and got depressed at the number of incompatibilities I needed to evaluate for impact on other packages.
In previous versions you've maintained the old names while adding the new ones to promote backward compatibility.  It would be great if you could continue to do that until things stabilize.  The prompt-toolkit users aren't keeping up.

Hi @kitterma,
Thanks for sharing this concern! I will take this seriously.
Jonathan


Fix encoding declaration in Python code

As of https://www.python.org/dev/peps/pep-0263/, magic comment must be the first or the second line of the file. Also, copyright symbol in license is UTF-8.
File "/usr/share/jmxtrans/tools/yaml2jmxtrans.py", line 4
SyntaxError: Non-ASCII character '\xc2' in file /usr/share/jmxtrans/tools/yaml2jmxtrans.py on line 4, but no encoding declared; see http://www.python.org/peps/pep-0263.html for details

Looks good, thanks!


The ability for users to structural mapping has been removed

Besides removing the HTML page and view class designed for adding structural mapping, I have also deleted the deletion HTML pages and view class. Let me know if the deletion functionality is still required, or if I removed anything incorrectly in any way.

There's a quite a few changes missing from this David. We also need to remove the model from admin.py, comment the code for the model in models.py remove MappingRule from import statements at the top etc. So I'm going to go ahead and close this PR


RFC: Beginnings of "socially intelligent cron jobs"

Right now, we're capable of scheduling a package invocation for a future time. That means we technically support an agent reaching out to tell someone something, but the reality of working with that is a bit tricky:

What if some other conversation in the interim renders the original message moot? Or socially weird?

ENTER:
The Socially Intelligent CROD Job!
(sorry I'm in a silly mood..)
Here's the rough idea:

You can enqueue message sends (to start; more complex stuff in V2) with different kinds of invalidation flags.

Example invalidation flags: "If another message send is sent with this key" or "if the user messages me back before the send"

Then, we schedule a maybe_send callback for the time when the send was scheduled. But it isn't guaranteed to succeed if some kind of invalidation event occurred.

 What's cookin?

best_friend.send(response)
agent.social_cron("1 day + rand(1 day)", "What's cookin? How's your day?", expire_if_user_responds=True)

this idea has tons of potential.
aside: i wonder which has more resonance with devs and users: "socially-intelligent" or "context-aware" or something less jargon-y like "conditional reminder" or even "if-not-then" (aside-aside: do people prefer to describe in the negative or positive -- if  expired, do  vs. if not , do ).
my sketch expanding upon it:
agent.schedule(
  action=FutureAction(tool="GreetingTool", input=user.name),
  condition=NoUserMessagesCondition(),
  delay="1d",
  recurring=True,
  context=ctx
)

I like this expansion a lot. Also that it's an Action your scheduling rather than a message -- fits better into the rest of the Action/Tool framing.
This is me just piling on ideas, but a few other things that I was thinking could be interesting (though maybe indulgent for v1?) could be:

A bit of randomness
Some kind of time-zone awareness of the user.. combined with an enum of human-style time blocks?

E.g.:
send this tomorrow-morning
send this late at night
send this on the weekend
send this around dinnertime


Web share article

Target Live Date: 2019-05-02

[ ] This has been reviewed and approved by (NAME)
[x] I have run npm test locally and all tests pass.
[x] I have added the appropriate type-something label.
[ ] I've staged the site and manually verified that my content displays correctly.

CC: @petele

Whoops!
There were 5 critical errors that broke the build and prevented it from being automatically deployed.
There were 6 warnings that will prevent this PR from being merged. Please take a look, and either fix, or provide a justification for why they can't be fixed.
ERRORS
src/content/en/updates/2019/05/web-share-files.md#L24 - Don’t use multiple top level headings (24:1)
src/content/en/updates/2019/05/web-share-files.md#L28 - Do not hard code developers.google.com in links.
src/content/en/updates/2019/05/web-share-files.md#L31 - Do not hard code developers.google.com in links.
src/content/en/updates/2019/05/web-share-files.md#L44 - Don’t use multiple top level headings (44:1)
src/content/en/updates/2019/05/web-share-files.md#L94 - Don’t use multiple top level headings (94:1)
WARNINGS
src/content/en/updates/2016/09/navigator-share.md#L139 - Consider adding a "was this page helpful?" widget to your page: https://developers.google.com/web/resources/widgets#helpful
src/content/en/updates/2016/09/navigator-share.md#L139 - Updates post is missing RSS feed widget: '{% include "web/_shared/rss-widget-updates.html" %}'
src/content/en/updates/2016/09/navigator-share.md#L138 - Line must be at most 100 characters
src/content/en/updates/2016/09/navigator-share.md#L139 - Line must be at most 100 characters
src/content/en/updates/2019/05/web-share-files.md#L51 - Common typo found: 'javascript' Should it be 'JavaScript'?
src/content/en/updates/2019/05/web-share-files.md#L70 - Common typo found: 'javascript' Should it be 'JavaScript'?

Whoops!
There were 6 warnings that will prevent this PR from being merged. Please take a look, and either fix, or provide a justification for why they can't be fixed.
WARNINGS
src/content/en/updates/2016/09/navigator-share.md#L139 - Consider adding a "was this page helpful?" widget to your page: https://developers.google.com/web/resources/widgets#helpful
src/content/en/updates/2016/09/navigator-share.md#L139 - Updates post is missing RSS feed widget: '{% include "web/_shared/rss-widget-updates.html" %}'
src/content/en/updates/2016/09/navigator-share.md#L138 - Line must be at most 100 characters
src/content/en/updates/2016/09/navigator-share.md#L139 - Line must be at most 100 characters
src/content/en/updates/2019/05/web-share-files.md#L49 - Common typo found: 'javascript' Should it be 'JavaScript'?
src/content/en/updates/2019/05/web-share-files.md#L68 - Common typo found: 'javascript' Should it be 'JavaScript'?

Whoops!
There were 6 warnings that will prevent this PR from being merged. Please take a look, and either fix, or provide a justification for why they can't be fixed.
WARNINGS
src/content/en/updates/2016/09/navigator-share.md#L139 - Consider adding a "was this page helpful?" widget to your page: https://developers.google.com/web/resources/widgets#helpful
src/content/en/updates/2016/09/navigator-share.md#L139 - Updates post is missing RSS feed widget: '{% include "web/_shared/rss-widget-updates.html" %}'
src/content/en/updates/2016/09/navigator-share.md#L138 - Line must be at most 100 characters
src/content/en/updates/2016/09/navigator-share.md#L139 - Line must be at most 100 characters
src/content/en/updates/2019/05/web-share-files.md#L49 - Common typo found: 'javascript' Should it be 'JavaScript'?
src/content/en/updates/2019/05/web-share-files.md#L68 - Common typo found: 'javascript' Should it be 'JavaScript'?

PTAL

Whoops!
There were 6 warnings that will prevent this PR from being merged. Please take a look, and either fix, or provide a justification for why they can't be fixed.
WARNINGS
src/content/en/updates/2016/09/navigator-share.md#L139 - Consider adding a "was this page helpful?" widget to your page: https://developers.google.com/web/resources/widgets#helpful
src/content/en/updates/2016/09/navigator-share.md#L139 - Updates post is missing RSS feed widget: '{% include "web/_shared/rss-widget-updates.html" %}'
src/content/en/updates/2016/09/navigator-share.md#L138 - Line must be at most 100 characters
src/content/en/updates/2016/09/navigator-share.md#L139 - Line must be at most 100 characters
src/content/en/updates/2019/05/web-share-files.md#L50 - Common typo found: 'javascript' Should it be 'JavaScript'?
src/content/en/updates/2019/05/web-share-files.md#L69 - Common typo found: 'javascript' Should it be 'JavaScript'?


About keccak256 introduction

I noticed that keccak256 was introduced here: https://github.com/openssl/openssl/issues/13033
I'm wondering which version shall I use in order to have that algorithm available on my C++ compilation.
It is on 1.1 or 3.0?
I'm asking this because I'm getting "Digest type not supported in this cryptolib."
I've installed libcrypt-dev 1:4.4.18-4 and libssl-dev/ 1.1.1n-0
Thanks in advance.

Thanks for your reply, there is a approximated date for that release? Can I see the roadmap somewhere?

There is no definitive date for it. Probably next year some time.

The only thing that made it into 3.0 was (KECCAK-KMAC-128/256) which is only useful for KMAC support.

3.1 will be our next release (planned for next week). 3.2 will be coming "later this year".

marking as inactive, will be closed at the end of the 3.4 dev cycle barring further input

3.2 with Keccak256 was released


Update to Threads in Plotter

As discussed in  #109 , this fixes an issue with handling of multiprocesses in Plotter class.

Thanks @dvspirito!


update README.md

Hello,
I appreciate flutter_credit_card package with fancy credit card input UI/UX and its animations.
Just a small changes, but this PR updates README.md which shows how to use various CreditCardWidget's properties to be compatible with latest version of this package.
Thank you.

Thanks for your contribution. This change has been merged in the PR #84.


Trim scheme double slash

Fixes https://github.com/spring-cloud/spring-cloud-kubernetes/issues/486

Let's do this in s-k-k8s


where to install database

hi thanks for cms
but I don't find any installation file that create database tables!
please help me

Hi plz database schema !!!!!!!!!!

great but how can I see what it's about without a database? :))


Fix compilation warning.

Detected during inclusion of this library in Blender when compiling using GPU 11.2.0/Linux.
The compiler suggests to add the parentheses around the '&&', but makes more sense
to add them around the '||' as the message is to inform the developer about the issue.

Thank you for this contribution. Indeed it was a bug.


2-1-1 Menu accessibility

Rework component to include the header in the tabbing when there is a hamburger menu bar.

Also fix this spacing issue that occurs with other languages in the 211 version:

Also fix this spacing issue that occurs with other languages in the 211 version:

But it's so pretty :P


fix ReX

change Rex to ReX.

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


Issue - Linkedin Learning "unknown url type:"

I am using youtube-dl v 2019.1.17
but I am unable to download video from https://www.linkedin.com/learning/
I have a premium Linkedin account, You can obtain a premium account for free for 3 months >>
Steps:

Sign up for the FREE Visual Studio Dev Essentials program that you can find here: https://www.visualstudio.com/en-us/products/visual-studio-dev-essentials-vs.aspx
After you create your account, Click on the “Get Code” under the Linkedin Learning Tile

And I am getting this result when downloading
C:\Users\BM>youtube-dl -u<EMAIL_ADDRESS>-p password "https://www.linkedin.com/learning/c-programming-for-embedded-applications"
[linkedin:learning:course] Downloading login page
[linkedin:learning:course] Logging in
[linkedin:learning:course] Downloading JSON metadata
[download] Downloading playlist: C Programming for Embedded Applications
[linkedin:learning:course] playlist C Programming for Embedded Applications: Collected 28 video ids (downloading 28 of them)
[download] Downloading video 1 of 28
[linkedin:learning] Downloading login page
[linkedin:learning] Logging in
Traceback (most recent call last):
File "c:\users\BM\appdata\local\programs\python\python36-32\lib\runpy.py", line 193, in run_module_as_main
"main", mod_spec)
File "c:\users\BM\appdata\local\programs\python\python36-32\lib\runpy.py", line 85, in run_code
exec(code, run_globals)
File "C:\Users\BM\AppData\Local\Programs\Python\Python36-32\Scripts\youtube-dl.exe_main.py", line 9, in
File "c:\users\BM\appdata\local\programs\python\python36-32\lib\site-packages\youtube_dl_init.py", line 472, in main
real_main(argv)
File "c:\users\BM\appdata\local\programs\python\python36-32\lib\site-packages\youtube_dl_init.py", line 462, in _real_main
retcode = ydl.download(all_urls)
File "c:\users\BM\appdata\local\programs\python\python36-32\lib\site-packages\youtube_dl\YoutubeDL.py", line 2002, in download
url, force_generic_extractor=self.params.get('force_generic_extractor', False))
File "c:\users\BM\appdata\local\programs\python\python36-32\lib\site-packages\youtube_dl\YoutubeDL.py", line 804, in extract_info
return self.process_ie_result(ie_result, download, extra_info)
File "c:\users\BM\appdata\local\programs\python\python36-32\lib\site-packages\youtube_dl\YoutubeDL.py", line 1003, in process_ie_result
extra_info=extra)
File "c:\users\BM\appdata\local\programs\python\python36-32\lib\site-packages\youtube_dl\YoutubeDL.py", line 870, in process_ie_result
extra_info=extra_info, download=False, process=False)
File "c:\users\BM\appdata\local\programs\python\python36-32\lib\site-packages\youtube_dl\YoutubeDL.py", line 793, in extract_info
ie_result = ie.extract(url)
File "c:\users\BM\appdata\local\programs\python\python36-32\lib\site-packages\youtube_dl\extractor\common.py", line 507, in extract
self.initialize()
File "c:\users\BM\appdata\local\programs\python\python36-32\lib\site-packages\youtube_dl\extractor\common.py", line 411, in initialize
self._real_initialize()
File "c:\users\BM\appdata\local\programs\python\python36-32\lib\site-packages\youtube_dl\extractor\linkedin.py", line 62, in _real_initialize
data=urlencode_postdata(data))
File "c:\users\BM\appdata\local\programs\python\python36-32\lib\site-packages\youtube_dl\extractor\common.py", line 772, in _download_webpage
expected_status=expected_status)
File "c:\users\BM\appdata\local\programs\python\python36-32\lib\site-packages\youtube_dl\extractor\common.py", line 638, in _download_webpage_handle
urlh = self._request_webpage(url_or_request, video_id, note, errnote, fatal, data=data, headers=headers, query=query, expected_status=expected_status)
File "c:\users\BM\appdata\local\programs\python\python36-32\lib\site-packages\youtube_dl\extractor\common.py", line 603, in _request_webpage
url_or_request = sanitized_Request(url_or_request, data, headers)
File "c:\users\BM\appdata\local\programs\python\python36-32\lib\site-packages\youtube_dl\utils.py", line 561, in sanitized_Request
return compat_urllib_request.Request(sanitize_url(url), *args, **kwargs)
File "c:\users\BM\appdata\local\programs\python\python36-32\lib\urllib\request.py", line 329, in init
self.full_url = url
File "c:\users\BM\appdata\local\programs\python\python36-32\lib\urllib\request.py", line 355, in full_url
self._parse()
File "c:\users\BM\appdata\local\programs\python\python36-32\lib\urllib\request.py", line 384, in _parse
raise ValueError("unknown url type: %r" % self.full_url)
ValueError: unknown url type: '/checkpoint/lg/login-submit'

I have a similar issuse
`
Arch:/dev/shm $ youtube-dl --verbose 'https://www.linkedin.com/learning/kubernetes-for-java-developers/applying-java-skills-to-kubernetes'
[debug] System config: []
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: ['--verbose', 'https://www.linkedin.com/learning/kubernetes-for-java-developers/applying-java-skills-to-kubernetes']
[debug] Encodings: locale UTF-8, fs utf-8, out UTF-8, pref UTF-8
[debug] youtube-dl version 2019.02.18
[debug] Python version 3.7.2 (CPython) - Linux-4.20.12-arch1-1-ARCH-x86_64-with-arch
[debug] exe versions: ffmpeg 4.1.1, ffprobe 4.1.1, rtmpdump 2.4
[debug] Proxy map: {}
ERROR: An extractor error has occurred. (caused by KeyError('JSESSIONID')); please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; see  https://yt-dl.org/update  on how to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.
Traceback (most recent call last):
File "/usr/lib/python3.7/site-packages/youtube_dl/extractor/common.py", line 508, in extract
ie_result = self._real_extract(url)
File "/usr/lib/python3.7/site-packages/youtube_dl/extractor/linkedin.py", line 95, in _real_extract
course_slug, 'selectedVideo', video_slug, height)['selectedVideo']
File "/usr/lib/python3.7/site-packages/youtube_dl/extractor/linkedin.py", line 34, in _call_api
'Csrf-Token': self._get_cookies(api_url)['JSESSIONID'].value,
KeyError: 'JSESSIONID'
Traceback (most recent call last):
File "/usr/lib/python3.7/site-packages/youtube_dl/extractor/common.py", line 508, in extract
ie_result = self._real_extract(url)
File "/usr/lib/python3.7/site-packages/youtube_dl/extractor/linkedin.py", line 95, in _real_extract
course_slug, 'selectedVideo', video_slug, height)['selectedVideo']
File "/usr/lib/python3.7/site-packages/youtube_dl/extractor/linkedin.py", line 34, in _call_api
'Csrf-Token': self._get_cookies(api_url)['JSESSIONID'].value,
KeyError: 'JSESSIONID'
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "/usr/lib/python3.7/site-packages/youtube_dl/YoutubeDL.py", line 794, in extract_info
ie_result = ie.extract(url)
File "/usr/lib/python3.7/site-packages/youtube_dl/extractor/common.py", line 521, in extract
raise ExtractorError('An extractor error has occurred.', cause=e)
youtube_dl.utils.ExtractorError: An extractor error has occurred. (caused by KeyError('JSESSIONID')); please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; see  https://yt-dl.org/update  on how to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.
`


assemblyId argument to referencesets-search CLI ignored

When we pass the --assemblyId argument to the  referencesets-search  command it is not passed on to the server.

Fixed in #1298


Jadeite/Rowan/3.2.15/SystemUser Rowan Method List Browser does not have breakpoint setting enabled ...

Don't know whether or not this is an inherited Jade feature or the fact that the Rowan Method List Browser is not complete:) ... if it is inherited by Jade, then I assume that this is expected behavior, otherwise I will be patient and wait:)

I think this one is fixed, but just to be sure ...


Java Code Coverage does not work with TFVC

This agent uses TFVC server workspaces which set all source files to ReadOnly. The CodeCoverage tools rely on editing the pom.xml, build.xml, etc. This fails when your projects are checked in to TFVC

It's been resolved.


"__" is considered reserved in PHP

From http://php.net/manual/en/language.oop5.magic.php (and later http://us3.php.net/manual/en/userlandnaming.rules.php)
"PHP reserves all function names starting with __ as magical. It is recommended that you do not use function names with __ in PHP unless you want some documented magic functionality."
This does make me a bit reluctant about using the library. As far as I know there are no existing conflicts, but I worry about the potential down the road.

For what it's worth, there are only 3 places __ gets explicitly referenced:

the __ function declaration
a single refererence in the __ function
the __ class declaration

So protecting yourself against any future conflicts is pretty straightforward - just rename __ to whatever you'd like.
Ideally, I could have just named the library _, but that's an alias for gettext in most PHP installations. So in the end, I had to make a judgement call weighing current issues, potential future issues, and similarity to Underscore.js.
Hopefully this isn't too much of an issue. I'll reopen this ticket if more people reach out and express concern over this.

I think we'll probably just do that for now. Thanks for responding.

Cool. Thanks for letting me know your concerns over the potential naming conflicts. Cheers.


Add handling for incoming RPC requests

Instead of handling every incoming JSON RPC call as either a notification
or a response, the frontend is now also able to properly handle requests.

I've addressed the comment and also learned about the YAGNI principle. Thanks for the review!

Thanks!


OCPBUGS-44243:unit test issue fix on git submodule creation

This is the test PR to fix the bug OCPBUGS-44243 . Unit test case that create git submodule fails permanently. This is a test to see if it fixes.

/test unit

this is a test hence closing


Have the worker startup introspect the platform more

A lot of folks have expressed interest in getting Vespene to work on more than the "official" list of Linux distros or occasionally have installed from very minimal environments.
For instance, an environment might be missing "ssh-add" or it might be out of path.  They also might have forgotten to create the build root.
In these scenarios, the worker really needs a precheck class (workers/pre_check.py) that runs when the daemon starts.  This can use shutil.which to find if certain CLI tools are in path and that the build environment is as expected.
It could also choose to warn if the permissions of the build root were not locked down enough.
reference #66 #67
and
https://talk.vespene.io/t/simplest-project-throws-nonetype-object-has-no-attribute-append-message/46/8

Other things to do (notes):
change default build dir to /opt/vespene_build
don't allow the admin to pick "vespene" as the sudo user
if the sudo user is the same as the current user, this should be an error (security reasons)
if the build dir is /opt/vespene, same
warn if the build dir is /tmp
fatal if the build dir is set to "" (not sure why that happens, but python allows chdir("") without exceptions).

Additional:
make new exception classes in common/ (like VespeneException) that we can explicitly catch and handle differently - we should have something like a CommandException that doesn't log a full traceback but just the command, and so forth.

So FYI for those looking at this - I havent' done this just yet and it's next on the list.
Many of the startup problems were caused by the shell script itself, so it's going to be a lot less possible for the build root to be blank now.
For those who only tried previous versions of the install scripts, this was fixed yesterday.

Concerns are mostly dealt with now from installer upgrades so I think this is closable.


Better install instructions for new Julia users

How about changing

Installing Oceananigans through the built-in Julia package manager is a good idea because Julia will take care of all dependencies and let you update Oceananigans using a single command: ] update.

to something like
"We recommend installing Oceananigans.jl with the built-in Julia package manager, because this installs a stable, tagged release.
Oceananigans.jl can be updated from the package manager by typing
(v1.1) pkg> update Oceananigans

At this time, updating Oceananigans.jl should be done with care, as Oceananigans.jl is under rapid development and breaking changes to the user API occur often.
Do we want to also move the "Developers" section to the end of the README? Maybe expand the scope of this PR slightly while we are looking at it?


Release version 1.8.1

Coverage remained the same at 98.658% when pulling 48f336fbb902ec98fc4cf2f5250acb6f9d5c7e55 on release-1.8.1 into 0c2f274a05154d3a4b5ba40ed6bbd11cc1be65cf on master.


usbip: Add user presence mechanism using signals

This patch adds a new user presence mechanism to the usbip runner that accepts the next user presence request within one second after receiving a SIGUSR1 signal.  This makes it possible to script user presence requests, e. g. for test suites.
Fixes: https://github.com/Nitrokey/nitrokey-3-firmware/issues/321

besides the nitpick wording thing commented this lgtm!

GitHub did not realize that this has been merged. https://github.com/Nitrokey/nitrokey-3-firmware/commit/0e23c75318e2016ac1cfb8345de9279e3ad2eaf9


Expose parameters for losses

Motivation for the feature

The SoftSeg paper recommends an omega of 8 for MS lesion segmentation, but that parameter is hard-coded to 14:
https://ivadomed.org/api_ref.html#ivadomed.losses.AdapWingLoss
Description of the feature

Make it possible to parametrize these values

@jcohenadad Not 100% sure but I think they are actually exposed. For example, I remember trying FocalLoss with a config like so:
    "training_parameters": {
        "batch_size": 4,
        "loss": {
            "name": "FocalLoss",
            "gamma": 0.5
        },

From a quick test, it seems to be parsed correctly:

To be fair, the doc is not really clear on this (from https://ivadomed.org/configuration_file.html#loss):

Metadata for the loss function. Other parameters that could be needed in the Loss function definition: see attributes of the Loss function of interest (e.g. 'gamma': 0.5 for FocalLoss).

Thank you Armand! We'll keep this open to improve the doc, then


Fix flaky tests

Determine the root cause of test flakiness.  The flakiness of the tests is currently the main limiting factor causing PRs not to land.

This is a duplicate of issue #446.


Social media links often blocked by adblocker

Using even the non-aggressive settings in uBlock Origin, a lot of the social media links get display: none;.
The two main triggers I'm seeing are having the ID of "sociallinks" and the title attributes on the links (at least for Twitter).
I was able to resolve this locally by removing the title attribute and changing the ID to something like "socialinfo". I can open a PR for this later.

Thanks for figuring this out. Neither the ID nor title attributes are essential, so you can change/remove them. I will wait for you pull request.


Add a docker-compose demo

I can help you with this... let's first merge the bug from #20 ok?

Thanks @isaqueprofeta, that's great.
I assigned this issue to you.

Great to see that we have the same opinion. I also replied to the comments.

That's well.
Let's do follow your suggestion, and drop the example by pure local run.
And remind only docker-compose example for users that want to try redisbeat.

Can you please review this last commit dropping the local example? The most important part I think that is the README.md with the explanations.

Just a little comment for me.
Others LGTM.


[Feature] TensorDict support for first class dimensions

This PR adds support for first class dimensions to TensorDict.
I've added logic that is able to calculate how dimensions update when indexed or ordered which allows us to index and order both MetaTensor and empty TensorDict instances. I've tried to cover as many edge cases as possible in the tests.

We’re using _something not for file-specific primitives but also for
feauters that are not exposed to the users.
For example _getitem_batch_size is used across files but we may not want
people to rely on it for code outside of TensorDict (+ we don’t necessarily
want to write a proper deep doc about it). Happy to discuss/revise those
conventions though!
On Sat, 5 Nov 2022 at 17:57, Shagun Sodhani @.***>
wrote:

@.**** commented on this pull request.
In tensordict/metatensor.py
https://github.com/pytorch-labs/tensordict/pull/5#discussion_r1010989320
:

@@ -13,8 +13,25 @@
import torch

from tensordict.memmap import MemmapTensor
-from tensordict.utils import DEVICE_TYPING, INDEX_TYPING
-from tensordict.utils import _getitem_batch_size, _get_shape
+from tensordict.utils import (

DEVICE_TYPING,
INDEX_TYPING,
LEVELS_TYPING,
convert_ellipsis_to_idx,
+)
+from tensordict.utils import (
_getitem_batch_size,

Minor comment: maybe these methods should not start with an "_" if they
are to be imported.
—
Reply to this email directly, view it on GitHub
https://github.com/pytorch-labs/tensordict/pull/5#pullrequestreview-1164175588,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AGCY4GSJOY6DN4LA5WOCIULWG2NZTANCNFSM6AAAAAARP5GP2A
.
You are receiving this because you modified the open/close state.Message
ID: @.***>

We’re using _something not for file-specific primitives but also for feauters that are not exposed to the users. For example _getitem_batch_size is used across files but we may not want people to rely on it for code outside of TensorDict (+ we don’t necessarily want to write a proper deep doc about it). Happy to discuss/revise those conventions though!

Makes sense! Thanks for the clarification :)


Mark terminated contracts

This PR builds on the Token Service PR to identify terminated contracts and allow further optimisation of network usage. Once contracts are identified as terminated they are no longer updated or polled.

@JamesSmartCell makes sense but need to fix conflicts


TextSplitter by Character trim leading whitespces

Describe the bug
TextSplitter by character with Trim=False removes leading whitespaces. This is not the case with HuggingFace tokenizer text spltiter.
To Reproduce
from semantic_text_splitter import TextSplitter
splitter = TextSplitter(20, trim=False, overlap=5)
split_text = splitter.chunks("This is a test document. It has two sentences. Maybe three ? To update your OXE type 'please help me' in the command line interface.")
print(split_text)
['This is a test ', 'test document. ', 'It has two sentences', '. Maybe three ? ', 'To update your OXE ', " OXE type 'please ", " help me' in the ", ' the command line ', 'line interface.']

In list element with index 1, the leading overlaping whitespace in front of test in missing. Causing issues then when we want to calculate overlap and merge texts.
Expected behavior
If trim=False, do not remove leading whitespaces.
The behaviour of TextSplitter.from_huggingface_tokenizer() is exactly this. Leading whitespaces are not removed.
Desktop (please complete the following information):

OS: [e.g. iOS]: Linux Docker Ubuntu
Version [e.g. 22]: 0.14.1

Hi @lambda-science this example actually looks correct to me. You have an overlap of 5 characters, and the last 5 characters of the first chunk are test
The reason it works with Hugging Face is that if the tokenizer has a whitespace prefix setting on (which many do) then there are tokens for many words with and without whitespace, so then including the whitespace doesn't increase the number of tokens, but is the same amount.
It isn't being trimmed, it is just that including the whitespace would go over the allowed character count in the overlap.
If you want the behavior you are describing, you would likely need to use the huggingface tokenizer for your usecase,.


git merge --no-ff bug?

Hello, this simple operation
git merge --no-ff bug

ends with an error

This can be worked around by using a fake argument
git merge --no-ff 1 bug

minimal reproducible test case
git checkout -b bug
git commit
git checkout master
git merge --no-ff bug

This is intentional more or less; just like git merge requires an argument, it does as well with the --no-ff flag.
i think the issue is the semantics of "1 argument". I actually mean "one non-flag argument" :)

Oh I see what you mean, yes its likely just parsing it incorrectly. justasec

fixed!

@pcottle epic! Thank you! :)


Implicit auto-complete doesn't seem to work on arrays

@ Array(1,2,3)
res6: scala.Array[Int] = Array(1, 2, 3)
@ res6.
!=             ##             ==             apply          asInstanceOf   clone
eq             equals         getClass       hashCode       isInstanceOf   length
ne             notify         notifyAll      synchronized   toString       update
wait

We should get .reduce and all the other extension-methods as well

@SethTisue said that this works in the new 2.11.8 pressy-powered REPL. We should steal their code

Closing in favor of https://github.com/lihaoyi/Ammonite/issues/275


Use an unstable algorithm for grenad::Sorter when possible

Pull Request
What does this PR do?
Use an unstable algorithm to sort the internal vector used by grenad::Sorter whenever possible to speed up indexing.
In practice, every time the merge function creates a RoaringBitmap, we use an unstable sort. For every other merge function, such as keep_first, keep_last, etc., a stable sort is used.
This is a draft PR for now because:

I'm waiting for the benchmark results
The version of grenad that accepts a SortAlgorithm as an argument to the sorter builder is not merged or released on crates.io yet

Result of the benchmarks:
group                                                                     indexing_grenad-sorter-unstable_25d66dc6    indexing_main_f8697075
-----                                                                     ----------------------------------------    ----------------------
indexing/-geo-delete-facetedNumber-facetedGeo-searchable-                 1.00    41.7±11.50ms        ? ?/sec         1.03     42.8±7.88ms        ? ?/sec
indexing/-movies-delete-facetedString-facetedNumber-searchable-           1.06      9.1±1.56ms        ? ?/sec         1.00      8.6±2.26ms        ? ?/sec
indexing/-movies-delete-facetedString-facetedNumber-searchable-nested-    1.69    23.7±31.95ms        ? ?/sec         1.00     14.1±3.11ms        ? ?/sec
indexing/-songs-delete-facetedString-facetedNumber-searchable-            1.00     42.1±5.07ms        ? ?/sec         1.08     45.6±7.98ms        ? ?/sec
indexing/-wiki-delete-searchable-                                         1.01   263.7±13.44ms        ? ?/sec         1.00   260.8±18.31ms        ? ?/sec
indexing/Indexing geo_point                                               1.00      47.4±0.62s        ? ?/sec         1.01      47.8±0.40s        ? ?/sec
indexing/Indexing movies in three batches                                 1.00      12.9±0.17s        ? ?/sec         1.03      13.2±0.08s        ? ?/sec
indexing/Indexing movies with default settings                            1.00      10.3±0.29s        ? ?/sec         1.06      10.9±0.26s        ? ?/sec
indexing/Indexing nested movies with default settings                     1.00       7.8±0.08s        ? ?/sec         1.04       8.0±0.36s        ? ?/sec
indexing/Indexing nested movies without any facets                        1.00       7.3±0.17s        ? ?/sec         1.02       7.5±0.25s        ? ?/sec
indexing/Indexing songs in three batches with default settings            1.00      48.4±0.81s        ? ?/sec         1.02      49.5±0.97s        ? ?/sec
indexing/Indexing songs with default settings                             1.00      44.1±1.36s        ? ?/sec         1.04      46.0±1.26s        ? ?/sec
indexing/Indexing songs without any facets                                1.00      41.9±0.76s        ? ?/sec         1.03      43.1±0.84s        ? ?/sec
indexing/Indexing songs without faceted numbers                           1.00      43.8±0.89s        ? ?/sec         1.02      44.6±0.61s        ? ?/sec
indexing/Indexing wiki                                                    1.00    869.9±20.14s        ? ?/sec         1.01    877.5±20.43s        ? ?/sec
indexing/Indexing wiki in three batches                                   1.00     935.7±5.74s        ? ?/sec         1.01     942.8±8.63s        ? ?/sec
indexing/Reindexing geo_point                                             1.01      15.8±0.26s        ? ?/sec         1.00      15.7±0.10s        ? ?/sec
indexing/Reindexing movies with default settings                          1.02   261.4±21.94ms        ? ?/sec         1.00   257.0±14.29ms        ? ?/sec
indexing/Reindexing songs with default settings                           1.00       4.1±0.06s        ? ?/sec         1.03       4.3±0.11s        ? ?/sec
indexing/Reindexing wiki                                                  1.00   1495.6±30.18s        ? ?/sec         1.00   1499.5±26.89s        ? ?/sec

As usual, the results looked a bit better on my computer. But it still pretty nice, we get a 6% speed up on the default movies bench :)


Bump dependencies to solidus v1.2

Bump to a brand-new solidus minor :gem:

Thanks @dfranciosi, I'm pro switching to using solidus_version, but do we need to bump the solidus dependency?
I don't know of anything in the 1.2 changelog that would stop us from using 1.1 on this.

@cbrunsdon I agree with you, honestly we don't need to force 1.2 here. Let's close this one, I will open a new one just for a bunch of deprecation warning fixes. :wink:

Awesome, thanks @dfranciosi


  All I see is "Assessment Extensions. See"  no link or other words after "See"
Chrome Version 75.0.3770.142 (Official Build) (64-bit)

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

ID: 45daf2f0-bb22-338f-062b-a9fabcb71a92
Version Independent ID: b6e909fe-15d9-c1cf-c018-d4f6f36ff31f
Content: Download SQL Server Management Studio (SSMS) - SQL Server
Content Source: docs/ssms/download-sql-server-management-studio-ssms.md
Product: sql
Technology: ssms
GitHub Login: @dnethi
Microsoft Alias: dinethi

@JohnValenzuela Thank you for the feedback.
The documentation has been updated.


Some questions about the code

Hi pat,
Good job! This code have a excellent performance in continue space env. But I have some questions about the code.

I can't understand the meaning of logvar_speed, could you explain that why it can make faster updates or give me some paper or blog something like these to understand it?

Why value function need use the 2 batches to update? which means every batch need to be updated 2 time.

Why checking variance before and after update value function can diagnose over-fitting?

I am looking forward your reply. : )

@chase20
Sorry for the slow reply - it has been a very busy week.

The learning rate is typically bounded by the deeper layers in the
network. The derivative of policy loss with respected to weights are
calculated independently. So several weights will tend to constructively
combine and push the policy too far and cause the update to overshoot. I'm
sure you've seen the cartoons where the update jumps across the convexity
and makes the loss worse. So you lower the learning rate to where you get
good improvement in training loss with each gradient update.

Unfortunately the rate the policy NN and Q function NN is much slower than
the rate the variance variables need. These are just learnable variables
with no NN. So by making the variance the sum of "logvar_speed" variables,
this increases the gradient update to variance by a factor of
"logvar_speed". This was a bit of a hack, a cleaner approach would have
been to just multiply the grads of variance variable by log_speed.
I noted the the variance was training much more slowly than I expected. It
was an almost perfect linear ramp, which indicated to me that it was
limited by the learning rate. Anyway, this made a big difference in
performance - maybe single biggest difference of improvements I made.

Just to smooth things out. Sometimes the policy gets worse after an
update, you don't want the Q function to chase this around. So, in some
ways, this is similar to a replay buffer in A3C if you're familiar with
that.

Somewhat related to item 2, with TRPO we are limiting policy change
between training iterations. So, the Q values of the policy shouldn't
change too much. So if the old Q function only explains 50% of the variance
after an update, but the newly fitted Q function explains 95% of the
variance, then you've over-fit the last batch of episodes and your Q
function isn't likely to extrapolate well when calculating next batch of
advantages.

A bit long-winded, but I hope this helps.
Pat
On Tue, Aug 22, 2017 at 11:16 PM, 20chase<EMAIL_ADDRESS>wrote:

Hi pat,
Good job! This code have a excellent performance in continue space env.
But I have some questions about the code.

I can't understand the meaning of logvar_speed, could you explain that
why it can make faster updates or give me some paper or blog something like
these to understand it?
2.
Why value function need use the 2 batches to update? which means every
batch need to be updated 2 time.
3.
Why checking variance before and after update value function can
diagnose over-fitting?
I am looking forward your reply. : )
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/pat-coady/trpo/issues/5, or mute the thread
https://github.com/notifications/unsubscribe-auth/AWdFxIvi2mhrTd1KJbW7mZjIegCuPuVOks5sa7WkgaJpZM4O_f7h
.


swapping pager's adapter bugfix

In Carbon the using of ViewPager and PagerTabStrip is following:
pager.setAdapter(new MyPagerAdapter(myData));
pagerTabStrip.setViewPager(pager);

When the pager's dataset is changed, one needs to call pagerTabStrip.setViewPager(pager); again in order to update the tabStrip. However, there is a bug: the TabBuilder instance of pagerTabStrip is not updated and keeps a reference to the old adapter (see initTabs()), that makes the data inconsistent or throws exceptions. In my case, the number of tabs and their titles can be changed.
There are several ways to fix this, I suppose there is nothing bad in making new instance of TabBuilder, as I propose in this request.
Ideally, the PagerTabStrip should be updated automatically.
Like this project!:thumbsup:

Thanks for your help!
In your pull request the tabBuilder field is being overwitten every time, so you cannot use a custom TabBuilder. The best solution would reuse the set TabBuilder with the new data.

I've opened an issue for tracking this bug, see: #101. I've also pushed a change that should fix your case, but I had no time to recreate it and check yet.

I applied your solution, that's ok, we can close this.


UI: Minor UI improvements

Description
This PR makes minor UI improvements.
While editing zone, use  “Default network domain for Isolated networks” instead of “Network Domain” & use “Default Guest CIDR for Isolated Networks” instead of “guest CIDR”
Before

After

While deploying a VM, selectively change "launch" to "created" on the button when "start instance" is set to OFF.
Before

After

Add space between system vm toggle & account while adding IP range for Public Traffic Type.
Before

After

Fix static NAT IP appearing twice
Before

After

Types of changes

[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] New feature (non-breaking change which adds functionality)
[ ] Bug fix (non-breaking change which fixes an issue)
[x] Enhancement (improves an existing feature and functionality)
[ ] Cleanup (Code refactoring and cleanup, that may add test cases)
[ ] build/CI
[ ] test (unit or integration test code)

Feature/Enhancement Scale or Bug Severity
Feature/Enhancement Scale

[ ] Major
[x] Minor

Bug Severity

[ ] BLOCKER
[ ] Critical
[ ] Major
[x] Minor
[ ] Trivial

Screenshots (if appropriate):
How Has This Been Tested?

How did you try to break this feature and the system with this change?

@blueorangutan ui

UI build: :heavy_check_mark:
Live QA URL: https://qa.cloudstack.cloud/simulator/pr/9307 (QA-JID-389)

@blueorangutan ui

@vishesh92 a Jenkins job has been kicked to build UI QA env. I'll keep you posted as I make progress.

UI build: :heavy_check_mark:
Live QA URL: https://qa.cloudstack.cloud/simulator/pr/9307 (QA-JID-390)

@blueorangutan ui

@vishesh92 a Jenkins job has been kicked to build UI QA env. I'll keep you posted as I make progress.

UI build: :heavy_check_mark:
Live QA URL: https://qa.cloudstack.cloud/simulator/pr/9307 (QA-JID-391)

@blueorangutan ui

@vishesh92 a Jenkins job has been kicked to build UI QA env. I'll keep you posted as I make progress.

UI build: :heavy_check_mark:
Live QA URL: https://qa.cloudstack.cloud/simulator/pr/9307 (QA-JID-392)


If SPF-record contains invalid qualifier, output is always True

If SPF-record contains invalid qualifier, the output is always True: https://www.reddit.com/r/PowerShell/comments/occgr2/powershell_script_for_checking_spf_dkim_and_dmarc/h3twvjg?utm_source=share&utm_medium=web2x&context=3

Fixed in version 1.4


resx files not working after changing default language.

Default DNN9 install with latest 2sxc and 2shine.
When I first used the theme, the imprint and privacy text and links worked.
After changing the default language to pt-PT and disabling the en-US, the text translations no longer work. I added multiple copies of my resx file (one for each possible language) to ensure at least one of them worked, but still nothing.
I added the AA and BB to make sure the links worked, and they do, so it's just the resx part not doing what it's supposed. Perhaps the function that loads the file is looking for some other pattern?
BTW, the default imprint and terms labels are reversed in the ascx.

Message:The following resource file caused an error while reading: C:\Inetpub\vhosts\pensaremsaude.pt\woundcare.app\portals_default\skins\2shine bs5\app_localresources\2shine.resx
Message:The following resource file caused an error while reading: C:\Inetpub\vhosts\pensaremsaude.pt\woundcare.app\portals_default\skins\2shine bs5\app_localresources\2shine.pt-pt.resx

Message:The following resource file caused an error while reading: ... \portals_default\skins\2shine bs5\app_localresources\2shine.pt-pt.resx
Message:The following resource file caused an error while reading:  ... \portals_default\skins\2shine bs5\app_localresources\2shine.resx

Nevermind. The end of the file was trimmed by whatever reason...


⚠ Processors has degraded performance

In 1d5f1c6, Processors ($STATUS_URL) experienced degraded performance:

HTTP code: 200
Response time: 91 ms

Resolved: Processors performance has improved in 56006fe.


save() would empty $meteor.object / $scope.$meteorObject, when called with no field changed

I was going through the official tutorials. In step 6: http://angular-meteor.com/tutorials/angular1/bind-one-object
After implement the code fragment 6.5/6.6,
If I click the save button without any modification, the name/description fields would be emptied. these fields are gone from the object or scope.
If I click the save button with a little modification, it works fine.
But in either case, the success promise supplies Object {action: "updated"}, not the number of successful docs changed.
<EMAIL_ADDRESS>

Hello, I have the same problem. When I return to an earlier version (urigo:angular), it works again...

Same issue here. @jeje87 what version were you on?

yes I can confirm that, looking at it now

thanks !!


[BUG?] requests.exceptions.TooManyRedirects: Exceeded 30 redirects.

Describe the bug
Getting this error after multiple times of using cl.user_medias(page_id, 15) function. I'm tried with different proxies.
To Reproduce
from instagrapi import Client
...
cl = Client()
cl.login(account[0], account[1])
data = cl.user_medias(page_id, 15)

Traceback
Traceback (most recent call last):
  File "/usr/local/lib/python3.8/dist-packages/instagrapi/mixins/public.py", line 132, in _send_public_request
    self.last_public_json = response.json()
  File "/usr/local/lib/python3.8/dist-packages/requests/models.py", line 910, in json
    return complexjson.loads(self.text, **kwargs)
  File "/usr/lib/python3.8/json/__init__.py", line 357, in loads
    return _default_decoder.decode(s)
  File "/usr/lib/python3.8/json/decoder.py", line 337, in decode
    obj, end = self.raw_decode(s, idx=_w(s, 0).end())
  File "/usr/lib/python3.8/json/decoder.py", line 355, in raw_decode
    raise JSONDecodeError("Expecting value", s, err.value) from None
json.decoder.JSONDecodeError: Expecting value: line 1 column 1 (char 0)

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/usr/local/lib/python3.8/dist-packages/instagrapi/mixins/media.py", line 677, in user_medias
    medias = self.user_medias_gql(user_id, amount)  # retry
  File "/usr/local/lib/python3.8/dist-packages/instagrapi/mixins/media.py", line 518, in user_medias_gql
    medias_page, end_cursor = self.user_medias_paginated_gql(
  File "/usr/local/lib/python3.8/dist-packages/instagrapi/mixins/media.py", line 467, in user_medias_paginated_gql
    data = self.public_graphql_request(
  File "/usr/local/lib/python3.8/dist-packages/instagrapi/mixins/public.py", line 210, in public_graphql_request
    body_json = self.public_request(
  File "/usr/local/lib/python3.8/dist-packages/instagrapi/mixins/public.py", line 76, in public_request
    raise e  # Stop retries
  File "/usr/local/lib/python3.8/dist-packages/instagrapi/mixins/public.py", line 74, in public_request
    return self._send_public_request(url, **kwargs)
  File "/usr/local/lib/python3.8/dist-packages/instagrapi/mixins/public.py", line 138, in _send_public_request
    raise ClientLoginRequired(e, response=response)
instagrapi.exceptions.ClientLoginRequired: Expecting value: line 1 column 1 (char 0)

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/usr/local/lib/python3.8/dist-packages/instagrapi/mixins/media.py", line 554, in user_medias_paginated_v1
    items = self.private_request(
  File "/usr/local/lib/python3.8/dist-packages/instagrapi/mixins/private.py", line 447, in private_request
    raise e
  File "/usr/local/lib/python3.8/dist-packages/instagrapi/mixins/private.py", line 434, in private_request
    self._send_private_request(endpoint, **kwargs)
  File "/usr/local/lib/python3.8/dist-packages/instagrapi/mixins/private.py", line 289, in _send_private_request
    response = self.private.get(api_url, params=params)
  File "/usr/local/lib/python3.8/dist-packages/requests/sessions.py", line 555, in get
    return self.request('GET', url, **kwargs)
  File "/usr/local/lib/python3.8/dist-packages/requests/sessions.py", line 542, in request
    resp = self.send(prep, **send_kwargs)
  File "/usr/local/lib/python3.8/dist-packages/requests/sessions.py", line 677, in send
    history = [resp for resp in gen]
  File "/usr/local/lib/python3.8/dist-packages/requests/sessions.py", line 677, in <listcomp>
    history = [resp for resp in gen]
  File "/usr/local/lib/python3.8/dist-packages/requests/sessions.py", line 166, in resolve_redirects
    raise TooManyRedirects('Exceeded {} redirects.'.format(self.max_redirects), response=resp)
requests.exceptions.TooManyRedirects: Exceeded 30 redirects.

Expected behavior
Getting user medias.
Desktop (please complete the following information):

OS: 18.04.5
Python version 3.8.0
instagrapi version 1.16.27

Additional context
I wonder why this error happens. Too many requests? Block by proxy/IP? Or problems with account (temp ban)?

I've got the same error.
2022-07-31 09:09:51,533 INFO [4606178816] <instagrapi_crawler.py, InstagrapiPageCrawler, next_page, 0066>: crystalward2228, try to fetch the next page with<PHONE_NUMBER>, None)
^@2022-07-31 09:10:01,812 ERROR [4606178816] <media.py, instagrapi, my_user_medias_paginated_v1, 0148>: Exceeded 30 redirects.
Traceback (most recent call last):
  File "/Users/shangyang/.local/share/virtualenvs/instacrawl-rjFv3BNe/lib/python3.8/site-packages/instacrawl/crawlerprovider/providers/instagrapi_override/media.py", line 137, in my_user_medias_paginated_v1
    items = self.private_request(
  File "/Users/shangyang/.local/share/virtualenvs/instacrawl-rjFv3BNe/lib/python3.8/site-packages/instagrapi/mixins/private.py", line 447, in private_request
    raise e
  File "/Users/shangyang/.local/share/virtualenvs/instacrawl-rjFv3BNe/lib/python3.8/site-packages/instagrapi/mixins/private.py", line 434, in private_request
    self._send_private_request(endpoint, **kwargs)
  File "/Users/shangyang/.local/share/virtualenvs/instacrawl-rjFv3BNe/lib/python3.8/site-packages/instagrapi/mixins/private.py", line 289, in _send_private_request
    response = self.private.get(api_url, params=params)
  File "/Users/shangyang/.local/share/virtualenvs/instacrawl-rjFv3BNe/lib/python3.8/site-packages/requests/sessions.py", line 600, in get
    return self.request("GET", url, **kwargs)
  File "/Users/shangyang/.local/share/virtualenvs/instacrawl-rjFv3BNe/lib/python3.8/site-packages/requests/sessions.py", line 587, in request
    resp = self.send(prep, **send_kwargs)
  File "/Users/shangyang/.local/share/virtualenvs/instacrawl-rjFv3BNe/lib/python3.8/site-packages/requests/sessions.py", line 723, in send
    history = [resp for resp in gen]
  File "/Users/shangyang/.local/share/virtualenvs/instacrawl-rjFv3BNe/lib/python3.8/site-packages/requests/sessions.py", line 723, in <listcomp>
    history = [resp for resp in gen]
  File "/Users/shangyang/.local/share/virtualenvs/instacrawl-rjFv3BNe/lib/python3.8/site-packages/requests/sessions.py", line 191, in resolve_redirects
    raise TooManyRedirects(
requests.exceptions.TooManyRedirects: Exceeded 30 redirects.

@kostyakoz  I cannot reproduce

but you can try using _v1:
data = cl.user_medias_v1(cl.user_id, 1)

@adw0rd same results with _v1:
Traceback (most recent call last):
  File "/usr/local/lib/python3.8/dist-packages/instagrapi/mixins/media.py", line 554, in user_medias_paginated_v1
    items = self.private_request(
  File "/usr/local/lib/python3.8/dist-packages/instagrapi/mixins/private.py", line 447, in private_request
    raise e
  File "/usr/local/lib/python3.8/dist-packages/instagrapi/mixins/private.py", line 434, in private_request
    self._send_private_request(endpoint, **kwargs)
  File "/usr/local/lib/python3.8/dist-packages/instagrapi/mixins/private.py", line 289, in _send_private_request
    response = self.private.get(api_url, params=params)
  File "/usr/local/lib/python3.8/dist-packages/requests/sessions.py", line 555, in get
    return self.request('GET', url, **kwargs)
  File "/usr/local/lib/python3.8/dist-packages/requests/sessions.py", line 542, in request
    resp = self.send(prep, **send_kwargs)
  File "/usr/local/lib/python3.8/dist-packages/requests/sessions.py", line 677, in send
    history = [resp for resp in gen]
  File "/usr/local/lib/python3.8/dist-packages/requests/sessions.py", line 677, in <listcomp>
    history = [resp for resp in gen]
  File "/usr/local/lib/python3.8/dist-packages/requests/sessions.py", line 166, in resolve_redirects
    raise TooManyRedirects('Exceeded {} redirects.'.format(self.max_redirects), response=resp)
requests.exceptions.TooManyRedirects: Exceeded 30 redirects.

It happens after multiple calls of the method. I wonder what reason cause it, API limits maybe?

@kostyakoz maybe yes, can you try another account/proxy for test?

@adw0rd changing proxy for cached account keep this problem; fresh accounts work for some time (few hours), also I'm using sleep for at least one minute between calls. As I understand when limits reached API should rise respective error?
By the way are limits available somewhere? As I read, 150 calls per acc/day. In my situation no more than 90 calls per acc/day, all accounts SMS verified.

@kostyakoz my accounts process up to ~1000 requests per day, but I have different types of requests

@adw0rd I'm also make cl.user_stories(page_id) requests, but less often. I've checked it now, and these requests don't give this error (with same accounts & proxies), only with cl.user_medias(page_id, 10)

For some reason when this error raise it even can't be handled with except requests.exceptions.TooManyRedirects:. How it can be handled?
try: cl.user_medias_v1(page_id, 10)
except requests.exceptions.TooManyRedirects:

Exceeded 30 redirects.
Traceback (most recent call last):
  File "/usr/local/lib/python3.8/dist-packages/instagrapi/mixins/media.py", line 554, in user_medias_paginated_v1
    items = self.private_request(
  File "/usr/local/lib/python3.8/dist-packages/instagrapi/mixins/private.py", line 447, in private_request
    raise e
  File "/usr/local/lib/python3.8/dist-packages/instagrapi/mixins/private.py", line 434, in private_request
    self._send_private_request(endpoint, **kwargs)
  File "/usr/local/lib/python3.8/dist-packages/instagrapi/mixins/private.py", line 289, in _send_private_request
    response = self.private.get(api_url, params=params)
  File "/usr/local/lib/python3.8/dist-packages/requests/sessions.py", line 555, in get
    return self.request('GET', url, **kwargs)
  File "/usr/local/lib/python3.8/dist-packages/requests/sessions.py", line 542, in request
    resp = self.send(prep, **send_kwargs)
  File "/usr/local/lib/python3.8/dist-packages/requests/sessions.py", line 677, in send
    history = [resp for resp in gen]
  File "/usr/local/lib/python3.8/dist-packages/requests/sessions.py", line 677, in <listcomp>
    history = [resp for resp in gen]
  File "/usr/local/lib/python3.8/dist-packages/requests/sessions.py", line 166, in resolve_redirects
    raise TooManyRedirects('Exceeded {} redirects.'.format(self.max_redirects), response=resp)
requests.exceptions.TooManyRedirects: Exceeded 30 redirects.

I had the same error for the last 2-3 days by now. No "job" finished successfully. I now had time to take a look at it and tried a frehs login (same account, same settings, same proxy - just a fresh login).
After entering the 6-digit 2FA code the script started running again and seems to be working.
Maybe doing a fresh login helps others, too?

@kostyakoz my accounts process up to ~1000 requests per day, but I have different types of requests
IDK the actual instagram limits, i think it's the values floating

Hi, Can you recommend a tutorial on how to implement the challenge function? I'm not using 2 factor auth so I hope it'll be easier to use that function.


rustfmt config: nightly and stable

The config options introduces in https://github.com/interchainio/tendermint-rs/pull/121 are causing some confusion. Maybe we should revert them, or, simply wait until they hit stable and close this issue then. Until then I'll keep this here for context for cointributors: You can ignore the warnings below:

Can we close this?

OK for me.


(BOLT-1175) Support running plans from YAML plans

This adds a plan step which can be used to run another plan.
This also fixes an issue with scope leaking between plan invocations.
Previously, we were running plans in the scope in which they were called,
causing variables in the surrounding scope to be visible to the plan. We now
run plans from the scope in which they're defined, which is global scope. That
causes every plan invocation to have a fresh scope without any stray variables
polluting it.

CLA signed by all contributors.


enable gosec

Signed-off-by: Kim Tsao<EMAIL_ADDRESS>What does this PR do?:

enables gosec in githubs actions, we will not fail if there are scan findings
adds rule to Makefile
clean up other rules in Makefile

Which issue(s) this PR fixes:

https://github.com/devfile/api/issues/937
PR acceptance criteria:
Testing and documentation do not need to be complete in order for this PR to be approved. We just need to ensure tracking issues are opened.

Open new test/doc issues under the devfile/api repo
Check each criteria if:
There is a separate tracking issue. Add the issue link under the criteria
or
test/doc updates are made as part of this PR
If unchecked, explain why it's not needed

[ ] Unit/Functional tests

[ ] QE Integration test

[ ] Documentation

[ ] Client Impact

How to test changes / Special notes to the reviewer:

Codecov Report
Base: 58.23% // Head: 58.23% // No change to project coverage :thumbsup:

Coverage data is based on head (1b036f4) compared to base (4c7c5dc).
Patch has no changes to coverable lines.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #147   +/-   ##
=======================================
  Coverage   58.23%   58.23%
=======================================
  Files          35       35
  Lines        3963     3963
=======================================
  Hits         2308     2308
  Misses       1516     1516
  Partials      139      139

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

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

Handled all Gosec findings by either fixing them or annotating them with nosec.  Some details of the fixes include:

Following the guideline for G304 to clean user provided filepaths in order to avoid path traversal
Updating the permissions to 0644 if the JSON schema is updated and replaced.  This is not a security concern since it's part of a script that's run outside core code to update the API dependency so I restricted the level of permissions based on our current defaults.  This will not eliminate the finding since it expects 0600 but I think 0644 is appropriate for our needs


Set shouldPureComponentUpdate on the prototype

Dunno if it has any real effect on performance, but this lets you avoid having to reassign it per instance.

Doubtful, but seems like a reasonable change.


Improve opportunity section

Description
I want to improve the opportunity section because when I visited opportunity section in mobile .so it doesn't looks good .
Screenshots

I want to work on this under GSSOC2022. Can you assign it to me.

@codingkush Go for it!

Thanks for assigning it to me

Ok i do what you said .....

sir...plz tell me ... is this okay???

Please assign me this issue under hacktoberfest. , if it is still open.

@ampsteric bro i have already fixed this issue this but i haven't submitted because my pratical was going on and final sem exams are coming that's why I didn't submitted PR

@SauravMukherjee44 sorry for delay...i was very busy in pratical and examination so... i wasn't able to submit my PR

I want to contribute to this issue.
Kindly assign me if open