Support for Synology Nas

Since GoGs is an autonomous low resources system, a Nas is a perfect place to put it in so we even have the security of multi disk. We may already deploy it but Synology offers a plugin system, easy to setup. I'm sure it can help the fast growing pace of GOGS and it would also be my best case for use.

Hi @Solido , thanks your feedback!
As I did some googling and found out that Synology Nas is a hardware? So, hmmm... it's more likely a feature request for Go to support it's architecture.

Please take a look at https://github.com/SynoCommunity/spksrc/issues/1522

Oh, wow, thanks @tboerger !

Hmm, I think you should keep track on upstream, there is very little (nothing) we could do?


Coroutine example is doing duplicate request.

Library Version [e.g. v1.0.3]

Hi @kronismax !
Thank you for your issue :)
It is fixed and re-published as version 1.0.3.


qtwayland: declare proper app_id for wrapped executables

Motivation for this change
Since NixOS 21.11 it seems as if QT uses Wayland if possible[1].
However, my pinentry-qt flavor stopped floating because it's now
running in Wayland-mode rather than in XWayland mode where this seems to
be fine.
I wanted to add a rule to my sway(1)-config for that, but realized
that pinentry is missing an app_id to match:
$ swaymsg -t get_tree | jq '.nodes[2].nodes[1].nodes[1].nodes[1].app_id'
""

This is because QWaylandWindow::initWindow() uses the application's
baseName to determine the app id. Unfortunately the baseName
drops all chars of the filename after the first dot[2]. This means that
every wrapped Nix package (i.e. pkgs.foo with $out/bin/.foo-wrapped)
will have an empty-string as baseName because the first character of the
filename is a dot. Since we're using the wrapQtAppsHook quite
excessively, a lot of programs are affected by this.
In order to work around this, I implemented a small patch for
qtwayland that strips away the nixpkgs-specific .(name)-wrapped of
a filename if needed and then sets the app_id to the expected
baseName. This is useful to make e.g. sway-configs with
for_window[3]-expressions from other distros compatible.
With this change, the app_id is set as I'd expect it:
$ swaymsg -t get_tree | jq '.nodes[2].nodes[1].nodes[1].nodes[1].app_id'
"pinentry-qt"

Even though we'll need the patch to get e.g. foo from .foo-wrapped,
I decided to file a bug-report against upstream[4].
[1] https://nixos.org/manual/nixos/stable/release-notes.html#sec-release-21.11
[2] https://doc.qt.io/qt-5/qfileinfo.html#baseName
[3] https://man.archlinux.org/man/sway.5.en
[4] https://bugreports.qt.io/browse/QTBUG-99137
Note: I really don't want to decide this on my own, but since it's technically a bugfix, I'd like to raise the question whether it's suitable for a backport (in case it's sufficiently reviewed & tested first).
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/)
22.05 Release Notes (or backporting 21.11 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
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

@lheckemann moved the comment to the patch, based onto staging :)

We probably want to consider adding a "libQtNixOS" thing, even if it's a header-only library, where we can implement these kind of things once.
This is the implementation in kwin:

https://github.com/NixOS/nixpkgs/blob/17336f60ffafb332cf27c9d8dd6e0aed54a95167/pkgs/desktops/plasma-5/kwin/0001-NixOS-Unwrap-executable-name-for-.desktop-search.patch#L53-L59

I figure it woule be best if we can at least sync-up the implementation. This one has been reviewed by @ttuegel.
Note: I have not comparatively "reviewed" your implementation. I'm not a C/C++/Qt expert, so I don't know which one would be "best". I'm only bringing up my concern over multiple implementations of the same concept starting to spring up left and right in the Qt/Plasma ecosystem

So your suggestion is to effectively use unwrapExecutablePath rather than inlining the implementation as it's done in the current patch?
Seems to be a fine solution for me, should we create a repo for this library in the NixOS/-org, then? Also, would be interested in @ttuegel's opinion on this :)

My suggestion was more about starting to think about that, but yes.
Using a unified implementation would be safer against "drift" if for some reason there was any difference in the actual behaviour of the different implementations.
I don't really know what the proper way forward would be, though.

By the way, assuming this current change fixes things, I don't mind it being merged, and ear-marked for future unification. Don't block improving support for Qt things under wayland on ideal implementation!

My suggestion was more about starting to think about that, but yes.

Fair. As soon as this is merged, I'd open an issue to keep track of this.
While the patch works fine for me, I'm only using a hand full of QT applications (most notably pinentry-qt where I really want a floating in sway), but I'm not sure if this could cause unexpected issues for e.g. KDE/Plasma users (cc @NixOS/qt-kde for a review).

Just FWIW: Wayland-native pinentry-qt used to float in Sway just fine.  It's only stopped working in the last few months or so.

@alyssais are you sure? Mine has only ever floated correctly when using Xwayland.

Yeah, I'm sure.  In fact, it still does, except for the window that asks me my smartcard passphrase (the window asking me to insert the smartcard floats, and the passphrase one used to but doesn't since 85b180f1b014 ("qt: Show hint if passphrase is shown and formatting is enabled")).  I tested this config in a stateless VM:
{ pkgs, ... }:

let
  gpg-agent-conf = pkgs.writeText "gpg-agent.conf" ''
    pinentry-program ${pkgs.pinentry.qt}/bin/pinentry
  '';
in

{
  systemd.tmpfiles.rules = [
    "d /home/qyliss/.gnupg 0700 qyliss users"
    "L+ /home/qyliss/.gnupg/gpg-agent.conf - - - - ${gpg-agent-conf}"
  ];

  environment.etc."sway/config.d/qyliss.conf".source = pkgs.writeText "qyliss.conf" ''
    input * xkb_layout dvorak
    exec foot
  '';

  environment.systemPackages = with pkgs; [ gnupg ];

  programs.sway.enable = true;
  programs.sway.extraPackages = with pkgs; [ foot ];
}

@alyssais I guess I can confirm your observations (passphrase forms don't float, but e.g. the "passphrase isn't safe, select a new one?"-window does - yes I gave the gpg key I generated in a VM for testing purposes a weak passphrase :sweat_smile: ).
And while it's actually quite nice that you seem to have fixed the issue for pinentry-qt (ref https://dev.gnupg.org/D549 for everyone following the thread), I still think there's still an issue with qtwayland on NixOS: I'd like to be able to configure any kind of window in my sway with some sort of matching mechanism, e.g. the app_id and this doesn't work for QT applications very well because $0 is called .foo-wrapped when using wrapQTAppsHook, so I still think that this patch is relevant.
ping @ttuegel any chance to get a review from you? I'm actually using this patch since December on my laptop and it works fine for me :)


Get-EXOMailbox like filter fails when using single quotation marks.

I'm using  ExchangeOnlineManagement version 3.1.0 and wish to list all mailboxes without a CustomAttribute2 attribute. Here is a sample:
Get-EXOMailbox -ResultSize 10 -Properties  CustomAttribute2 -Filter {CustomAttribute2 -like ''} | Select-Object Name, Customattribute2

Unfortunately, it fails with this error:
Get-EXOMailbox : Error while querying REST service. HttpStatusCode=400 ErrorMessage={"error":{"code":"BadRequest","message":"Invalid filter clause","innererror":{"message":"Invalid filter
clause","type":"Microsoft.Exchange.Admin.OData.Core.ODataServiceException"}}}}
At line:1 char:1
+ Get-EXOMailbox -ResultSize 10 -Properties  CustomAttribute2 -Filter { ...
+ ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    + CategoryInfo          : ProtocolError: (:) [Get-EXOMailbox], RestClientException
    + FullyQualifiedErrorId : An error occurred while processing this request.,Microsoft.Exchange.Management.RestApiClient.GetExoMailbox

If you change the line to use double quotation marks like this:
Get-EXOMailbox -ResultSize 10 -Properties  CustomAttribute2 -Filter {CustomAttribute2 -like ""} | Select-Object Name, Customattribute2

The line works fine.  This is against about_Quoting_Rules, and it's a massive pain when you have automatic code refactoring in VSCode, as any time a non-expandable string is spotted using double quotes, those are automatically converted to single quotes.

I have found a workaround:
Get-EXOMailbox -ResultSize 10 -Properties  CustomAttribute2 -Filter {CustomAttribute2  -eq $null} | Select-Object Name, Customattribute2

But I think the double quotes issue should be fixed all the same.

@kieranwalsh, I highly recommend opening a support ticket about this. The issue will (eventually) get in front of the people who can fix it.

@kieranwalsh Hope this comment is helpful for you. Considering the nature of the issue, as advised, please open a service ticket in your tenant and follow with them for the resolution. You can contact Microsoft support via the need help section on the admin center. We proceed to close it as here we are limited to documentation fixes and improvements only.
We will be very happy if you can share the outcome or resolution with us if you see documentation update is required. We will certainly update this documentation with that valuable information. Thanks again, for taking out some time to open the issue. Appreciate and encourage you to do the same in future also.


assertion on control naming

Fix for #242

Hi @joanpablo
Added assestion to prevent issues with control names which contain dots see #242

Thanks for quick response)


Define event location: cdCON 2021

[ ] Reserve virtual platform June 23 - 25 | Emily | Deadline: Oct. 30, 2020 |
[ ] Define event start & end time | Emily | Deadline: Nov. 13, 2020
[ ] Are we doing multiple events? | Jacqueline | Deadline: Nov. 13, 2020

cdcon will be virtual on June 23 & 24, 2021 using the virtual platform Hopin. We will support the Americas timezone and start 7am EST time to also be EU friendly.


FAQ updates from main repo

Copied FAQ from https://github.com/organicmaps/organicmaps/blob/master/data/faq.html Nothing changed.
Added service page /faq/translation_status with all FAQ questions with translations as a table to track statuses.
FAQ categories need translations:

EN
RU
DE
ES
FR
PR
PR-BR
PL
TR
UK
ZH-Hans

App
Общие
?
Aplicación
?
?
?
?
?
Загальні
?

Bookmarks
Метки и Треки
?
?
?
?
?
?
?
Мітки та Маршрути
?

Map Editing
Редактирование
?
?
?
?
?
?
?
Редагування
?

Map
Карта
?
?
?
?
?
?
?
Мапа
?

Voice Directions
Голосовые подсказки
?
Instrucciones de voz
?
?
?
?
?
?
?

I think Roman's PR #115 could be closed.

Should we synchronize organicmaps/data/faq.html file with this repo? Some Python script to copy text from MD to HTML?

Thanks a lot for your work! I love the translation status page! Is it possible in addition to compare FAQ article dates, and mark translations as outdated if they were not changed after the English md file was edited?
Left some comments, it would be great to also sync ru, uk and other languages.

@rtsisyk try https://faq-updates-from-main-repo.organicmaps.pages.dev
Did you see this block?


✨[Feature Request]: Add hover effect to Find A Doctor button on the navbar of home page

Is there an existing issue for this?

[X] I have searched the existing issues

Feature Description
The Find a Doctor button does not have any colour change effect on hovering.I would like to add a colour change hover effect to the button.Please assign me this issue under GSSOC 2024.

Use Case
The feture would improve the design of the website.
Benefits
No response
Add ScreenShots
No response
Priority
High
Record

[X] I have read the Contributing Guidelines
[X] I'm a GSSOC'24 contributor
[X] I have starred the repository

I would like to work on this issue. Kindly assign it to me.


rally-elasticsearch workload

Rally-Elasticsearch workload

Please add the license information in the doc/overview/overview.md

Usually we recommend putting this in internal repo and move it once it passes QoS. Unless you have high confidence


create a page for club03

create a page called
sae-cea.html under src/campus-life/clubs-at-cea/
with headers and footers from
src/contact.html
Make sure you change the title in html page to SAE | CEA

Hello - I'll give this a try. Any other content to put on the page?

@virenb Just header and footer is enough for now. We're structuring the whole website

@abhijithvijayan thanks, went to create this but looks like someone else has completed it already.

@virenb hey, sorry that we didn't close this issue soon after the issue has been solved

[Merged] #178
Closing


Fix ztest

Due to a variable sharing bug, ztest was re-running the same test under the guise of all the different tests it was running.
The first commit in this PR illustrates that with an invalid test that passes.
The second commit fixes ztest which fails the bad test. The third commit removes. the bad test.

Oops, this has uncovered some failing tests that we thought were green! Will fix.


Add ability create custom manifest operation IDs

The generate-persisted-query-manifest CLI now has the ability to generate custom manifest operation IDs. When using a config file format that supports functions (i.e. js and ts extensions), a createCustomId function can be added to customize how the ID should be generated for a given query.
Here is an example that shows how to create a base64 encoded ID of the query when its operation name is TestOperation:
// persisted-query-manifest.config.ts
import { PersistedQueryManifestConfig } from '@apollo/generate-persisted-query-manifest';
import { Buffer } from 'node:buffer'

const config: PersistedQueryManifestConfig = {
  createOperationId(query, { operationName, createDefaultId }) {
    switch (operationName) {
      case 'TestOperation':
        return Buffer.from(query).toString('base64');
      default:
        return createDefaultId();
    }
  }
}

export default config;

The id returned from createOperationId will be validated for uniqueness before the manifest file is written.

The createOperationId function signature is as-follows:
createOperationId(query: string, options: CreateOperationIdOptions) => string

type CreateOperationIdOptions {
  operationName: string;
  type: OperationType;
  createDefaultId: () => string;
}

I don't have a local dataset to test this

I've been testing on the spotify demo for this. But yes, I want to add some tests soon (unfortunately there weren't any yet in this package when I started working on it).


Script to read in Higgs XSWG excel files

Can be extended to produce tables automatically.

One issue is that xlrd doesn't seem to come with the python bundled inside the recommended CMSSW version.
Second, would it be possible to add the BSM tables?

On 30 Jan 2017, at 10:45 , nckw<EMAIL_ADDRESS>wrote:
One issue is that xlrd doesn't seem to come with the python bundled inside the recommended CMSSW version.

I don't think this tool should be bound by CMSSW packages; in particular, it's reproducibility is really not an issue.

Second, would it be possible to add the BSM tables?

Sure.

I've merged this in but can you provide a command line example?
It would be useful to add options to select only certain tabs to pipe the output into separate text files which we can then store in the relevant subdirs.


update packages

:white_check_mark: Build TheIdServer 2.1.1-fix-dependencies1-0002 completed (commit https://github.com/Aguafrommars/TheIdServer/commit/69fdb061da by @)


run tests selection does not work as expected

In https://github.com/RedHat-SP-Security/tests/tree/sopos-rsyslog-tls-shutdown/rsyslog/Regression/bz1880434-gnutls-shutdown:
$ tmt tests ls .
/rsyslog/Regression/bz1880434-gnutls-shutdown
$ tmt run -v discover tests --name .
/var/tmp/tmt/run-016
Found 1 plan.

/changed-tests
summary: check the changed tests functionality
    discover
        how: fmf
        directory: /home/dapospis/test/git/SP_components/tests
        hash: 677439b
        names: /rsyslog/Regression/bz1880434-gnutls-shutdown
        modified-ref: master
        warn: Unable to create a 'rsyslog/Library/basic' symlink for a deep library ([Errno 17] File exists: 'rsyslog/Library/basic' -> '/var/tmp/tmt/run-016/changed-tests/discover/default/libs/tests/basic').
        summary: 2 tests selected
            /rsyslog/Regression/bz1880434-gnutls-shutdown
            /rsyslog/Regression/bz1886400-gnutls-shutdown-relp

This may be related to https://github.com/psss/tmt/issues/752

I played with this a bit and found out that the issue comes from the fact that there's a tmt plan defined which executes changed tests. This causes the unwanted tests to be included.
The solution is to use plans selector to effectively disable all the possibly existing plans like this: tmt run plans --default.


Add management command to check status on individual job task

Instead of having this section of the admin guide, we can just run the get_status method of the given task.
The command can (verbose mode) print the full response from the component's API, for debugging.
https://docs.google.com/document/d/1iMxMEheQw3656Lxi4d9zZ01KR536EnBhqJFLvzNpfrw/edit#heading=h.71dk8yqc550

Running get_status is the same as looking at the job's tasks' statuses in the Django admin, so closing.
Re: the admin guide, reading the get_status code is clear enough.


New urls PDOK, nlmaps may stop working on 2023, August 2

Pdok has new urls, see pull request
https://github.com/geo-frontend/nlmaps/pull/133
and urgent message from pdk (published july 4 2023)
https://www.pdok.nl/-/urgent-oude-url-s-pdok-locatieserver-per-2-augustus-uit-productie?redirect=%2Fnieuws
I assume this will impact this project those using this.
Any change on updates?
Since there is no activity since 2 years on this project.

Looks like the url's of nlmaps are not in the migration list:
https://www.pdok.nl/documents/d/pdok/cloudmigratie-oude-en-nieuwe-url-s-07-06-2023
link found here: https://www.pdok.nl/releases1


Releases 0.22.1

This new PR releases a new version of the library including:

List repository projects: #365
List cards: #364
List milestone and List Labels in a project : #359
Transition to the new 47degrees organization

Among other @scala-steward updates.

Oh, just realized that this new release would include breaking changes due to this PR: https://github.com/47degrees/github4s/pull/367 by @BenFradet

Codecov Report

Merging #372 into master will not change coverage.
The diff coverage is n/a.

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


[CERT-TEST-FAILURE] - TC-BRBINFO-1.1 - Test failed

Feature Area
Other
Test Case
TC-BRBINFO-1.1
Reproduction steps

Bring up the DUT with Bridge-app.
Pair the DUT using chip-tool through "Onnetwork" transport.
Run the command ./chip-tool bridgeddevicebasicinformation read cluster-revision 1 3

Expected Outcome:

[1699964532.740562][8099:8101] CHIP:TOO: Endpoint: 3 Cluster: 0x0000_0039 Attribute 0x0000_FFFD DataVersion:<PHONE_NUMBER>
[1699964532.740706][8099:8101] CHIP:TOO:   ClusterRevision: 3
[1699964532.741053][8099:8101] CHIP:EM: <<< [E:24809i S:16555 M:161513284 (Ack:30709492)] (S) Msg TX to 1:0000000000000001 [9BA3] [UDP:[fe80::e65f:1ff:fe0e:be37%eth0]:5540] --- Type 0000:10 (SecureChannel:StandaloneAck)

Actual Outcome:

[1699964532.740562][8099:8101] CHIP:TOO: Endpoint: 3 Cluster: 0x0000_0039 Attribute 0x0000_FFFD DataVersion:<PHONE_NUMBER>
[1699964532.740706][8099:8101] CHIP:TOO:   ClusterRevision: 2
[1699964532.741053][8099:8101] CHIP:EM: <<< [E:24809i S:16555 M:161513284 (Ack:30709492)] (S) Msg TX to 1:0000000000000001 [9BA3] [UDP:[fe80::e65f:1ff:fe0e:be37%eth0]:5540] --- Type 0000:10 (SecureChannel:StandaloneAck)

Bug prevalence
Whenever I try to read ClusterRevision.
GitHub hash of the SDK that was being used
f09120f41f3d32678454db1915730f6f5730bb39
Platform
raspi
Anything else?
In SPEC, the ClusterRevision value has been updated to 3. As per spec changes, SDK has to update the ClusterRevision value from 2 to 3.
Specification Reference:

We should close this as "do not fix". 1.3 is out the door already, this test is being removed in 1.4 - https://github.com/CHIP-Specifications/chip-test-plans/pull/4104


Default Map implementations take to too much storage.

VM and dart2js implementations of LinkedHashMap seem to use an order of magnitude more storage than JavaScript.
-------- test.js:
function main() {
  var a = [];
  while (true) {
    a.push({'u': 1, 'v': 2, 'w': 3, 'x': 4, 'y': 5});
    if (a.length % 100000 == 0) print(a.length);
  }
}
main();
-------- test.dart:
main() {
  var a = [];
  while (true) {
    a.add({'u': 1, 'v': 2, 'w': 3, 'x': 4, 'y': 5});
    if (a.length % 100000 == 0) print(a.length);
  }
}

The highest numbers printed before an out-of-memory error by running this program on the 32 bit VMs are:
"sdk/bin/dart test.dart"   2200000 (2.2M)

"d8 test.js"              22100000 (22.1M)
"d8 test.dart.js"          1000000 (1M)
"jsshell test.js"         41800000 (41.8M)
"jsshell test.dart.js"     8400000 (8.4M)
From this it looks like Dart LinkedHashMaps take 5x-20x the storage of JavaScript Objects.

This issue is now specifically about dart2js, which uses its own map implementation.


Implement the batch endpoint

This PR will implement the batch endpoint specified in https://github.com/github/git-lfs/pull/285

Just make it #MODERN.           #jgaj

Yes.We merged it from multitransfer,commit at master.Therefore,endpoint still will work if it is from the master..Thanks..       #jgaj


chore: drop beta package descriptor feature, closes HYB-546

Motivation:

Package Descriptor feature is going out of beta and v1 is coming

Modifications:

Remove beta package descriptor deployment call from enterpriseUpload target

Result:

Beta package descriptor: out

I let this PR in draft until commons next update

Depends on https://github.com/gatling/gatling-enterprise-plugin-commons/pull/126 release


DRILL-6327: Update unary operators to handle IterOutcome.EMIT

        Note: Handles for Non-Blocking Unary operators (like Filter/Project/etc) with EMIT Iter.Outcome

+1. Very nicely done.


setup.py and .pth

I find there is no setup.py and the pre_trained model which you offer is not match the network.
I think you trained the model on another script which you not offer ,recently you improved your script, but you didn't retrained .

The current code base adopts the implementation of InplaceSyncBN from https://github.com/mapillary/inplace_abn. The compiling of the InplaceSyncBN operator will be automatically triggered when running the code.

Our former codebase is based on PyTorch version 0.4. However, we have upgraded to pytorch>1.0 for simplicity usage.
From my side, the current provided pre-trained model matches the network configuration. Please clarify what kind of mismatch.

After killing several bugs that produced by my environment.The major bug apper.
(pytorch15) q30004136:~/SCHP_org$ python train.py
Traceback (most recent call last):
File "train.py", line 25, in
import networks
File "/data_ssd3/q30004136/SCHP_org/networks/init.py", line 3, in
from networks.AugmentCE2P import resnet101
File "/data_ssd3/q30004136/SCHP_org/networks/AugmentCE2P.py", line 21, in
from modules import InPlaceABNSync
File "/data_ssd3/q30004136/SCHP_org/modules/init.py", line 1, in
from .bn import ABN, InPlaceABN, InPlaceABNSync
File "/data_ssd3/q30004136/SCHP_org/modules/bn.py", line 10, in
from .functions import *
File "/data_ssd3/q30004136/SCHP_org/modules/functions.py", line 18, in
extra_cuda_cflags=["--expt-extended-lambda"])
File "/data_ssd3/q30004136/anaconda3/envs/pytorch15/lib/python3.7/site-packages/torch/utils/cpp_extension.py", line 921, in load
is_python_module)
File "/data_ssd3/q30004136/anaconda3/envs/pytorch15/lib/python3.7/site-packages/torch/utils/cpp_extension.py", line 1120, in _jit_compile
return _import_module_from_library(name, build_directory, is_python_module)
File "/data_ssd3/q30004136/anaconda3/envs/pytorch15/lib/python3.7/site-packages/torch/utils/cpp_extension.py", line 1441, in _import_module_from_library
file, path, description = imp.find_module(module_name, [path])
File "/data_ssd3/q30004136/anaconda3/envs/pytorch15/lib/python3.7/imp.py", line 296, in find_module
raise ImportError(_ERR_MSG.format(name), name=name)
ImportError: No module named 'inplace_abn'
(pytorch15) q30004136:~/SCHP_org$
my environment is CUDA 10.2 ,ubuntu 18.0.4,python3.7
The error is  there is no inplace_abn model .Do i need download the inplace_abn from  https://github.com/mapillary/inplace_abn
and then put into the project?
when i run this command ：pip install inplace_abn.it says that: Requirement already satisfied: inplace_abn in /data_ssd3/q30004136/anaconda3/envs/pytorch15/lib/python3.7/site-packages/inplace_abn-0.0.0-py3.7-linux-x86_64.egg (0.0.0)
i don't kown how to solve this problem. I need your help please.

You don't need to install inplace_abn by yourself. The current code base already contains the inplace_abn implementation in ./modules. And the compiling of the InplaceSyncBN operator will be automatically triggered when running the code.

I have tried several times ，it always the same error.
File "/data_ssd3/q30004136/SCHP_org/modules/functions.py", line 18, in
extra_cuda_cflags=["--expt-extended-lambda"])
ImportError: No module named 'inplace_abn'
How do i solve this problem？thank you very much！

@yunshangyue71 ImportError: No module named 'inplace_abn' did you solve this？

Can we use some alternatives for that? The inplace_abn has many problems with versions of pytorch and cuda and the environment. For example, replace InPlaceABNSync with nn.BatchNorm2d and nn.ReLU to save functionality and fix the problem.
Did anyone try to do that?

The inplace_abn has many problems with versions of pytorch and cuda and the environment. Can we use some alternatives for that? For example, replace InPlaceABNSync with nn.BatchNorm2d and nn.ReLU to save functionality and fix the problem. Did anyone try to do that?

if you retrain the model I think  nn.BatchNorm2d is and relu are ok。if you use the ckpt to predict, I think  nn.BatchNorm2d is ok. Because InPlaceABNSync is nn.BatchNorm2d used by multi GPUs. perhaps the method is do  nn.BatchNorm2d in each GPU, and return the means and norms， then get the total BN in the master GPU。 so i think replace InPlaceABNSync with nn.BatchNorm2d is ok.
 I think there still a lot of room for improvement.
have you tried a light model trained by Apple or Google.Im not mean there are better but give you more choice.
you can alse follow some works about MPI.


format_handler error when setting non-string as type for OutputField

Thanks for all your hard work on this project.
When trying to set the dspy.OutputField as a list
class SRLSignature(dspy.Signature):
    """
    Generate Semantic Role Labeling (SRL) for a given sentence.
    """
    sentence: str = dspy.InputField()
    predicates: list[str] = dspy.OutputField(
        desc="The list of predicates in the sentence",
        format="answers"
    )

items = df.groupby('sentence').apply(lambda x: {'sentence': x.name, 'predicates': list(set(x['predicate']))}).tolist()
examples = [dspy.Example(**item).with_inputs('sentence') for item in items]

class SRLModule(dspy.Module):
    """
    Generate Semantic Role Labeling (SRL) for a given sentence.
    """
    def __init__(self):
        super().__init__()
        self.runner = dspy.Predict(SRLSignature)

    def forward(self, sentence):
        return self.runner(sentence=sentence)

mod = SRLModule()
teleprompter = BootstrapFewShot(metric = answer_exact_match, max_bootstrapped_demos=8, max_labeled_demos=4, max_rounds=2)
compiled_program = teleprompter.compile(student = mod, teacher = mod, trainset=examples)

returns this error when running the compiler:
AssertionError: Need format_handler for predicates of type <class 'list'>

I scoured the docs and code but couldn't make sense of this error. I know it has something to do with Template V2 put past that I gave up.
So the questions are:

Is it only possible to provide strings as field types?
If we can provide other types, how do we properly set the format_handler? The only option I see is to provide a format argument - but this is not clear what is supposed to go there. The docs mention the format arg only in passing
Do we have to request some form of json output as discussed in #264 instead? I have inquired about what "typed dspy" means in this context in that issue

I think at the very least we need a better error message. I would be happy to help with this - just need to understand whats happening here first :)
By the way, we get the same error when using a non-string type for the dspy.InputField too
Thanks for any guidance

I think you want:
    **format=list**

The easiest thing to do is to format your item as a string before you pass it to the method

But you can pass format=fn where fn is a function that will create a string out of your list.
from dsp import passages2text
is a function that does that

I think you want:
    format=list

OK this was annoyingly simple. Thanks for the quick reply.
However it only works when I change the output field name to "answer":
class SRLSignature(dspy.Signature):
    """
    Generate Semantic Role Labeling (SRL) for a given sentence.
    """
    sentence: str = dspy.InputField()
    # predicates: list[str] = dspy.OutputField(
    #     desc="The list of predicates in the sentence",
    #     format=list
    # )
    answer: list[str] = dspy.OutputField(
        desc="The list of predicates in the sentence",
        format=list
    )

Now the compiler seems to run

But you can pass format=fn where fn is a function that will create a string out of your list.
from dsp import passages2text
is a function that does that

This also has the compiler running, so long as again I set my output to "answer".
Not sure which one is the best - maybe both?
I also know there is a big rewrite going on, so to summarise the things Ive learnt that may be addressed in the rewrite/updated docs:

The format are in a signature field can take the sequence type list. It can also take a function (ie. passages2text)
The above solutions only work if you explicitly set the output field in your signature to answer. Wondering if this works for scenarios where you are asking for multiple OutputFields in the signature?

Let me know if I have misrepresented anything. Otherwise, thanks again both for your help

Closing as I think the TypedSignature seems to be doing the trick for me

Closing as I think the TypedPredictors seems to be doing the trick for me

@stantonius do you mind sharing your solution with TypePredictors? TypePredictors afaik aren't the same as dspy.Output/InputField, so I'm not sure how you are trying to solve this but would love to learn!

Some things I tried that failed:
# 2. Define ICL Module that takes multiple few-shot examples (problem-solution pairs)
class ICLMathModule(dspy.Signature):
    """Use ICL with multiple math problem-solution pairs to generate an answer."""
    # context = dspy.InputField(desc="Math context for ICL")
    examples = dspy.InputField(type=list[str], desc="List of problem-solution pairs for few-shot ICL")
    question = dspy.InputField(type=str, desc="New math question to solve")
    answer = dspy.OutputField(type=str, desc="Answer generated by ICL")
    ```

    just typing directly.

The easiest thing to do is to format your item as a string before you pass it to the method

This seems to work so far:
    def forward(self, contexts: list[str], question: str):
        _contexts: str = '\n'.join(contexts)
        # Step 1: Generate math problem-solution pairs from the list of contexts
        result = self.math_problems(contexts=_contexts)
        generated_qa_pairs = result.question_answer_pairs
       ...


Add observability_services interop test case for config dump + admin

@ericgribkoff
Since people are pointing me to this doc, I assume the test plan people want is an additional interop test case. This test case will test all 3 observability projects that I'm working on:

CSDS
Admin interface
Admin CLI (the name isn't settled yet)

From the test driver point of view, this test will be quite special because it needs to invoke another binary. It probably would cause a little hassle in dependency management to fetch the latest admin CLI in our CI. But the logic is straightforward.
From the client and the backend point of view, we will need to add admin interface to the controlling port. It will require code changes to each language.

Moving to internal document (for now).


Release notes for 24072

Update release notes for Microsoft defender for Endpoint - Linux. Release version 24072 updated.

@microsoft-github-policy-service agree company="Microsoft"


deepClone is declared twice

https://github.com/jakubpawlowicz/clean-css/blob/00288426664db2c90afff6a7090a953277268332/lib/optimizer/level-2/properties/override-properties.js#L10-L11

Good catch, thanks! :+1:


context2d arc issue

I have an issue with exporting a pie chart svg using jspdf 1.5.3.
This issue can be reproduced using the examples/canvg_context2d/bar_graph_with_text_and_lines.html using the following svg:
<svg id="svg" width="815.29" height="481.78" xmlns="http://www.w3.org/2000/svg">
	<g transform="translate(0, 0)">
		<g transform="translate(440.5, 168.5)">
			<g>
				<path fill="#5cbae6" d="M7.841413454940502e-15,-128.06A128.06,128.06 0 1,1 -126.47379768010367,-20.093334725032697L0,0Z" />
			</g>
			<g>
				<path fill="#b6d957" d="M-126.47379768010367,-20.093334725032697A128.06,128.06 0 0,1 -2.3524240364821506e-14,-128.06L0,0Z" />
			</g>
		</g>
	</g>
</svg>

The resulting pdf doesn't have the correct arcs drawn.
This was not reproducible using the 1.4.1 version of jspdf before the context2d refactor.

Oha, have to check this.

Reverting the createArc function to the previous implementation seems to help a bit:
    c2d.internal.createArc = function (radius, startAngle, endAngle, anticlockwise) {
        var EPSILON = 0.00001; // Roughly 1/1000th of a degree, see below
        var twoPI = Math.PI * 2;
        var piOverTwo = Math.PI / 2.0;

        // normalize startAngle, endAngle to [0, 2PI]
        var startAngleN = startAngle;
        if (startAngleN < twoPI || startAngleN > twoPI) {
            startAngleN = startAngleN % twoPI;
        }
        if (startAngleN < 0) {
            startAngleN = twoPI + startAngleN;
        }

        while (startAngle > endAngle) {
            startAngle = startAngle - twoPI;
        }
        var totalAngle = Math.abs(endAngle - startAngle);
        if (totalAngle < twoPI) {
            if (anticlockwise) {
                totalAngle = twoPI - totalAngle;
            }
        }

        // Compute the sequence of arc curves, up to PI/2 at a time.
        var curves = [];
        var sgn = anticlockwise ? -1 : +1;

        var a1 = startAngleN;
        for (; totalAngle > EPSILON;) {
            var remain = sgn * Math.min(totalAngle, piOverTwo);
            var a2 = a1 + remain;
            curves.push(this.createSmallArc(radius, a1, a2));
            totalAngle -= Math.abs(a2 - a1);
            a1 = a2;
        }

        return curves;
    };

Hello Conor,
I will check it later ;). I invested few hours for the arc issue, but I was
having the feeling that my changes are not sufficient enough.
Will see if the changes are fixing this issue.
Conor Smyth<EMAIL_ADDRESS>schrieb am Fr., 12. Apr. 2019, 12:44:

Reverting the createArc function to the previous implementation seems to
help a bit:
c2d.internal.createArc = function (radius, startAngle, endAngle, anticlockwise) {
    var EPSILON = 0.00001; // Roughly 1/1000th of a degree, see below
    var twoPI = Math.PI * 2;
    var piOverTwo = Math.PI / 2.0;

    // normalize startAngle, endAngle to [0, 2PI]
    var startAngleN = startAngle;
    if (startAngleN < twoPI || startAngleN > twoPI) {
        startAngleN = startAngleN % twoPI;
    }
    if (startAngleN < 0) {
        startAngleN = twoPI + startAngleN;
    }

    while (startAngle > endAngle) {
        startAngle = startAngle - twoPI;
    }
    var totalAngle = Math.abs(endAngle - startAngle);
    if (totalAngle < twoPI) {
        if (anticlockwise) {
            totalAngle = twoPI - totalAngle;
        }
    }

    // Compute the sequence of arc curves, up to PI/2 at a time.
    var curves = [];
    var sgn = anticlockwise ? -1 : +1;

    var a1 = startAngleN;
    for (; totalAngle > EPSILON;) {
        var remain = sgn * Math.min(totalAngle, piOverTwo);
        var a2 = a1 + remain;
        curves.push(this.createSmallArc(radius, a1, a2));
        totalAngle -= Math.abs(a2 - a1);
        a1 = a2;
    }

    return curves;
};

—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/MrRio/jsPDF/issues/2314#issuecomment-482528651, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AE0yHKVpk1rPWXKbwCQSAYNxI_LM4Zd_ks5vgGN6gaJpZM4bpAMs
.

Well...

Yes, doesn't fix it but it's an improvement. Seems like there's an issue filling an arc so there's probably a point or two missing from the path that needs to be taken to correctly complete the arc.

Well, it is just doing the Sekante. If you dig further in it. I am digging into it too. Would be cool if we could fix it ;)

Yes I'll be looking into it as well so I'll let you know if I get anywhere!
Just there was work done previously in this area, hasn't helped me figuring it out but it might ring some bells with yourself!
https://github.com/MrRio/jsPDF/commits/master?before=b5cd29b5515eb7473c4a6de2d91292d4c3ece695+840
#1043
#1044

That is tooo old. I loaded 1.4.1 in rawgit and injected your svg with developer tools into the website and try to compare the generated pdf and the steps of context2d to generate the PDF.

I think i fixed it. Have to do more tests.

Done with #2417.
nice.

Great, thanks! I'll check it out on Monday and let you know if I notice anything!

Works with the pie chart given as an example however here's another example where it does not work:
<svg id="svg" width="815.29" height="481.78" xmlns="http://www.w3.org/2000/svg">
	<g transform="translate(14, 57.96)">
		<g>
			<g transform="translate(0, 0)">
				<g transform="translate(350.8409375, 146.52)">
					<g>
						<path fill="#5cbae6" d="M6.818539462420668e-15,-111.35520000000001A111.35520000000001,111.35520000000001 0 1,1 -109.97591000646166,-17.472257589980956L-54.98795500323083,-8.736128794990478A55.677600000000005,55.677600000000005 0 1,0 3.409269731210334e-15,-55.677600000000005Z"></path>
					</g>
					<g>
						<path fill="#b6d957" d="M-109.97591000646166,-17.472257589980956A111.35520000000001,111.35520000000001 0 0,1 -2.0455618387262003e-14,-111.35520000000001L-1.0227809193631001e-14,-55.677600000000005A55.677600000000005,55.677600000000005 0 0,0 -54.98795500323083,-8.736128794990478Z"></path>
					</g>
				</g>
			</g>
		</g>
	</g>
</svg>

Could you atleast supply images of it? I always read my messages and think about possible solutions.

Sorry I should have given a picture,
Here's what it looks like in 1.5.3:

Here's what it looked like in 1.4.1:


Enable OSE repo for nss_wrapper and install nss_wrapper in RHEL7 Dockerfile.

In CentOS Dockerfile nss_wrapper is installed but not in RHEL7 version. So tests are failing.
To install nss_wrapper OSE repository has to be enabled.
@hhorak Please take a look and merge.

LGTM.


intermittent/flaky terraform failures could be somewhat mitigated

What is your feature idea?

i'm seeing a behavior with maybe a 10% or so consistency lately where terraform entrypoints will randomly fail to compile a plugin compile during provisioning. the nice ux provides a retry, and provided there's no real system issue, will be successful on that retry a mere moment later. it would be good to have a single error handler added to the terraform execution. on failure, there should be no negative consequence of a 10s sleep and init/apply again, allowing to raise that error back to the user. this would save the user a click and provide a better picture of stability overall.
Why is it needed?
stability and ux
@jarededwards has already taken a look at what this would take, we can likely get it into one of these next few patches.
Is this missing feature preventing you from using kubefirst?

[X] Yes

Code of Conduct

[X] I agree to follow this project's Code of Conduct

fixed by https://github.com/kubefirst/kubefirst-api/pull/269


node: fix inconsistently failing test

Fix the test for pruning a node's inventory which inconsistently and rarely fails.  It fails due to each test peers' clocks being slightly out of sync in the test.  One peer may have their clock skewed one second before the other.
The test confirms how pruning inventory entries relates to time.  Time therefor needs to be accurate to test against internal edge cases.
Note:
I would prefer to sync the clocks manually, the alternative of using an 'if' and 'elapsed' combination seems a bit much.
i.e.
bob.clock().set(alice.clock().local_time());

One question this prompts for me is whether this is just fixing the test rather than fixing the actual problem in the code? I'm not sure of the answer because I'm not sure I understand the issue :)

One question this prompts for me is whether this is just fixing the test rather than fixing the actual problem in the code? I'm not sure of the answer because I'm not sure I understand the issue :)

The PR comment and commit message needed some tending to.  Is the new description clear?

I think so. I suppose another way of phrasing my question is, "Can this clock skew happen in practice? And is the failure correct in that case?"

The test is for confirming whether internal mechanisms work as expected using the public api.

I think so. I suppose another way of phrasing my question is, "Can this clock skew happen in practice? And is the failure correct in that case?"

Thanks for the review :+1:.  It does feel off using one peer's timestamp for another's message.  I'll try addressing this in this PR.  Upcoming tests may run into the same issue, so it is good to address.

@FintanH I made some significant changes

Nice.

Nice one :) Much better!


Compiler(amd64): raise error on SSE4 unsupported CPUs when needed

Is your feature request related to a problem? Please describe.
We've already used SSE4.x features in floating-point conversions and SIMD implementations. However, some fairly old amd64 CPUs do not have support for these features.
Practically speaking, SSE4.1 features was introduced 15-years ago, and actually, the SIMD proposal has been merged with the baseline being SSE4 on Intel CPUs(https://github.com/WebAssembly/simd/pull/201#issuecomment-600240445). It seems really hard (maybe impossible) to implement without the feature (https://github.com/bytecodealliance/wasmtime/issues/3809, https://github.com/bytecodealliance/wasm-micro-runtime/issues/1022,https://github.com/wasmerio/wasmer/pull/2775, https://github.com/wasmerio/wasmer/pull/2775).
It's nice if we could have support for these CPUs, but given that almost all amd64 CPUs have SSE4, that's not our priority and revisit what we can do.
Describe the solution you'd like
Get the available features by CPUID instruction, and raise a compilation error if the features are necessary while not available in the hardware.
Describe alternatives you've considered
N/A
Additional context
Go has introduced GOAMD64 flag as of 1.18, and we could leverage the flag and check if the flag has >=v2.

ps I would like to put this limitation only on SIMD instructions, so I will identity where SSE4.1 features are used in non-SIMD instructions (one place I can think of is floating conversions.

assigning @evacchi


feat(jotai): add @suspensive/jotai package

Overview
Roadmap

[x] Atom (setAtom)
[x] SetAtom (useSetAtom)
[x] AtomValue (useAtomValue)

Tests

[x] local package test (pack .tgz)
[x] jotai-tanstack-query (query, mutation)

PR Checklist

[x] I did below actions if need

I read the Contributing Guide
I added documents and tests.

Codecov Report
Attention: Patch coverage is 0% with 6 lines in your changes missing coverage. Please review.

Project coverage is 84.40%. Comparing base (421fa2e) to head (baf4dc1).

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #1042      +/-   ##
==========================================
- Coverage   85.34%   84.40%   -0.94%
==========================================
  Files          45       48       +3
  Lines         539      545       +6
  Branches      118      118
==========================================
  Hits          460      460
- Misses         74       80       +6
  Partials        5        5

Components
Coverage Δ

@suspensive/react
97.03% <ø> (ø)

@suspensive/react-query
75.28% <ø> (ø)

@suspensive/react-query-4
0.00% <ø> (ø)

@suspensive/react-query-5
0.00% <ø> (ø)

@suspensive/react-await
100.00% <ø> (ø)

@suspensive/react-image
80.39% <ø> (ø)

@gwansikk I will call you after making new Pull Request into this branch to modify this codes. could you review it later?

https://github.com/toss/suspensive/pull/1045 I left improving Pull Request

@gwansikk Could I merge first? In my opinion, interfaces should be improved and unified. But in zero version @suspensive/jotai could be improved even with BREAKING CHANGE. so It would be better that we release it first
In my opinion, It is better to merge first to create an environment where documentation and feature improvements can be worked on in parallel.

Yes, I agree! It seems better to deploy version 0.0.0 first and then make improvements.

@gwansikk I added @suspensive/jotai in codecov.yml at b10b946


Rule proposal: use 'const enum' instead of 'enum' when possible.

TSLint version: 3.13.0
TypeScript version: 2.0.3
Running TSLint via: CLI

Since const enums are inlined, there is a possible small performance gain when using a const enum instead of an enum. I'd like a rule that would let me know if my enum could be replaced with a const enum to "avoid paying the cost of extra generated code and additional indirection when accessing enum values."

@LMFinney I gave it a shot: https://www.npmjs.com/package/tslint-consistent-codestyle#prefer-const-enum-wip
This is still work in progress, but early feedback is very appreciated.
Please note the current limitations listed over there.
Also this package only works with tslint@>4 and is developed and tested with typescript 2.1.1
@adidahiya I'd like to develop and test rules in my own repository. If they have come to a certain maturity / quality, I would be glad if some of them could be included in tslint core.

OTOH, const enums wreak havoc with Webpack:

https://github.com/s-panferov/awesome-typescript-loader/issues/137
https://github.com/Microsoft/TypeScript/issues/16671
https://github.com/wmonk/create-react-app-typescript/issues/400

If anything, we should have a rule for the other way around: to prefer enums.

Note: per #4534, this issue will be closed in less than a month if no PR is sent to add the rule. If you really need the rule, custom rules are always an option and can be maintained outside this repo!

💀 It's time! 💀
TSLint is being deprecated and no longer accepting pull requests for new rules. See #4534. 😱
If you'd like to see this rule implemented, you have two choices:

Recommended: Use typescript-eslint to lint your TypeScript code with ESLint ✅
Legacy: Write it as a custom TSLint rule 🤷♂

👋 It was a pleasure open sourcing with you!
If you believe this message was posted here in error, please comment so we can re-open the issue!

🤖 Beep boop! 👉  TSLint is deprecated 👈 (#4534)  and you should switch to typescript-eslint! 🤖
🔒 This issue is being locked to prevent further unnecessary discussions. Thank you! 👋


Add advanced parallelized 3 note auto and annoying auto

Annoying is also annoying to work with, and 3 note may not make it in the time allotted (17 seconds).

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

#47  👈
#46
#45
#44
#43
#42
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @BruceMcRooster and the rest of your teammates on  Graphite

Merge activity

Apr 27, 9:21 AM EDT: @BruceMcRooster started a stack merge that includes this pull request via Graphite.

Merging this stack because the season is over, and this is the code we sent to the district competition.


improve IDEMIX code

This is shopping list of things we want to improve about the IDEMIX code, maybe with some help of @FriederikePollmann:

[ ] improve documentation of parameters in the code
[ ] sort out local and global arrays (e.g. why is IDEMIX_V0 a global array?, IDEMIX_tau_d can be made a local array, IDEMIX might just return a "tendency" IDEMIX_tau_d*IDEMIX_E**2, as this is the only thing that's used in ggl90_calc.F
[ ] fix a restart problem (see PR #334), this involves carefully investigating the exchanges.
[ ] use the value of PI defined in PARAMS.h (in functions IDEMIX_gofx2 and IDEMIX_hofx1), maybe conflict with inlining.

I don't know if this is a "real" issue, but I like to mention that (potentially) different value of PI are used in the 2 functions
within ggl90_idemix.F and we may want to always used the value from PARAMS.h

@jm-c good point, I added it to the list. One comment though: if we include PARAMS.h (and SIZE.h), the functions become very long and automatic inlining may not work anymore leading to potential performance issues (on vector architectures). Maybe be best to add PI to the list of arguments instead?

Each of these 2 function are only called once, so it's not a big issue to add "PI" as argument.
Otherwise, I suggest to wait for issue (3) i.e., restart, to be fixed before removing array in common block since we
might need to have few more with bi,bj so that they can be exchanged.

I took an other look at the code and it seems that the restart issue should be easy to fix, just need to apply EXCH to GGL90TKE when useIDEMIX=T and IDEMIX_tau_h > 0. This also seems to be the case where we need to apply EXCH to IDEMIX_E.
@mjlosch I could start a PR with my modified version of ggl90_exchanges.F if you prefer.

@jm-c cool. I tried something similar but didn't succeed yet, glad you found a fix. I have a few other things already here: https://github.com/mjlosch/MITgcm/tree/improve_idemix_code, but they are only comments and some reordering, so it would be easy for me to add them to a PR that you start.

Each of these 2 function are only called once, so it's not a big issue to add "PI" as argument.

This is Pi in IDEMIX_gofx2/hofx1 (first line) and in model/inc/PARAMS.h (second lind)
      PARAMETER( pi = 3.14159265358979323846264338327950588d0 )
      PARAMETER ( PI = 3.14159265358979323844D0   )

any idea, why we use "4" for the last digit instead of "6" (I checked and it should be "6").

Each of these 2 function are only called once, so it's not a big issue to add "PI" as argument.

This is Pi in IDEMIX_gofx2/hofx1 (first line) and in model/inc/PARAMS.h (second lind)
      PARAMETER( pi = 3.14159265358979323846264338327950588d0 )
      PARAMETER ( PI = 3.14159265358979323844D0   )

any idea, why we use "4" for the last digit instead of "6" (I checked and it should be "6").

No idea. But if we were to change this it would need to be in a dedicated PR.

An other minor thing: We could move a bit the place ggl90_idemix is called: right now, it's called between calculation of SQRTTKE(k=1) and calculation of SQRTTKE(k>1) ; would be simpler to move the call earlier (before both) or after both are computed since it's independent of SQRTTKE.

Makes sense. I suggest to move to after the mixing length computation, i.e. just before
https://github.com/jm-c/MITgcm/blob/09b790b44aef499431807d23f10e7ff4c1eb1d0d/pkg/ggl90/ggl90_calc.F#L554
or alternatively right after hFacI/recip_hFacI have been computed (but I like later better):
https://github.com/jm-c/MITgcm/blob/09b790b44aef499431807d23f10e7ff4c1eb1d0d/pkg/ggl90/ggl90_calc.F#L212
As we think about moving code around, would it make sense to move the long and confusing mixingLength code into a separate s/r ggl90_mixing_length? Right now, this long stretch of code makes it very difficult to read the gg90_calc-routine.

@mjlosch thinking of where to call GGL90_IDEMIX, i would prefer just after hFacI calculation: the idemix S/R is stepping forward it's own state var so it seems better to separate this from the rest of ggl90_calc.F. And regarding
mixingLength code, a specific S/R might requires some changes for TAF and OpenAD and would be simpler to avoid this (keep the code inside gg90_calc.F).

@jm-c Do you think we can also tick the first box in the initial issue?

@mjlosch I think yes, you did a good job in improving the doc in the code.


dockerize trt_pose problem.

Hello everyone,
I’m trying to run trt_pose inside of docker container, for now I’m trying to run live_demp.ipynb, and everything works fine except for jetcam.
piece of code:

from jetcam.usb_camera import USBCamera
from jetcam.utils import bgr8_to_jpeg

camera = USBCamera(width=WIDTH, height=HEIGHT, capture_fps=30)
camera.running = True

error:
---------------------------------------------------------------------------
RuntimeError                              Traceback (most recent call last)
/usr/local/lib/python3.6/dist-packages/jetcam-0.0.0-py3.6.egg/jetcam/usb_camera.py in __init__(self, *args, **kwargs)
     23             if not re:
---> 24                 raise RuntimeError('Could not read image from camera.')
     25

RuntimeError: Could not read image from camera.

During handling of the above exception, another exception occurred:

RuntimeError                              Traceback (most recent call last)
<ipython-input-12-3efd176c52b8> in <module>
      3 from jetcam.utils import bgr8_to_jpeg
      4
----> 5 camera = USBCamera(width=WIDTH, height=HEIGHT, capture_fps=30)
      6 # camera = CSICamera(width=WIDTH, height=HEIGHT, capture_fps=30)
      7

/usr/local/lib/python3.6/dist-packages/jetcam-0.0.0-py3.6.egg/jetcam/usb_camera.py in __init__(self, *args, **kwargs)
     26         except:
     27             raise RuntimeError(
---> 28                 'Could not initialize camera.  Please see error trace.')
     29
     30         atexit.register(self.cap.release)

RuntimeError: Could not initialize camera.  Please see error trace.

output of v4l2-ctl -d /dev/video0 --list-formats-ext :
ioctl: VIDIOC_ENUM_FMT
	Index       : 0
	Type        : Video Capture
	Pixel Format: 'YUYV'
	Name        : YUYV 4:2:2
		Size: Discrete 640x480
			Interval: Discrete 0.033s (30.000 fps)
		Size: Discrete 160x120
			Interval: Discrete 0.033s (30.000 fps)
		Size: Discrete 176x144
			Interval: Discrete 0.033s (30.000 fps)
		Size: Discrete 320x240
			Interval: Discrete 0.033s (30.000 fps)
		Size: Discrete 352x288
			Interval: Discrete 0.033s (30.000 fps)
		Size: Discrete 1280x720
			Interval: Discrete 0.100s (10.000 fps)
		Size: Discrete 1920x1080
			Interval: Discrete 0.200s (5.000 fps)

	Index       : 1
	Type        : Video Capture
	Pixel Format: 'MJPG' (compressed)
	Name        : Motion-JPEG
		Size: Discrete 640x480
			Interval: Discrete 0.033s (30.000 fps)
		Size: Discrete 160x120
			Interval: Discrete 0.033s (30.000 fps)
		Size: Discrete 176x144
			Interval: Discrete 0.033s (30.000 fps)
		Size: Discrete 320x240
			Interval: Discrete 0.033s (30.000 fps)
		Size: Discrete 352x288
			Interval: Discrete 0.033s (30.000 fps)
		Size: Discrete 1280x720
			Interval: Discrete 0.033s (30.000 fps)
		Size: Discrete 1920x1080
			Interval: Discrete 0.033s (30.000 fps)

output of gst-launch-1.0 v4l2src device=/dev/video0 ! videoconvert ! xvimagesink:
Setting pipeline to PAUSED ...
ERROR: Pipeline doesn't want to pause.
ERROR: from element /GstPipeline:pipeline0/GstXvImageSink:xvimagesink0: Could not initialise Xv output
Additional debug info:
xvimagesink.c(1773): gst_xv_image_sink_open (): /GstPipeline:pipeline0/GstXvImageSink:xvimagesink0:
Could not open display (null)
Setting pipeline to NULL ...
Freeing pipeline ...

I map the host machine webcam to the docker container, this mean the container recognize the camera, but jetcam don’t for some reason. and I'm able to open jupyter on the host machine.
Thanks in advance,
Hassan

just replaced it with opencv, check: #73


trx.commit, trx.rollback give eslint violations

Hi, it's not really an important issue, knex works fine (thanks ! :) )
but eslint gives this error: "expected an assignment or function call and instead saw an expression."
whenever I use trx.commit; or trx.rollback;  Is there a way to avoid this ?
Thanks in advance,
Rob

I don't think this is an issue for the library authors :) Sounds like you need to either change your coding style, or your .eslintrc settings!
A code example would definitely help clear it up either way.

yeah I know nothing is wrong with knex, I figured I might be doing something wrong
...
.catch(err => {
            console.log('error saving account, rolling back');
            console.error(err);
           ** trx.rollback;**
          })

So, am I using this the wrong way, if so, how should I use it ? should i just do ?

trx.rollbackis function, try calling trx.rollback(). Please reopen the issue is problems continue.

if I call trx.commit(), i get  "Transaction query already complete, run with DEBUG=knex:tx for more info" Even though I'm just calling it once. When I use trx.commit; or trx.rollback; withouth the () it works fine, same goes for rollback. And I am just calling it once, strange...

You're likely returning a promise to the transaction function which then automatically commits or rolls back the transaction, in which case you don't need to use trx.commit or trx.rollback

Yep, works fine without it :o Thanks!
On 4 Feb 2016 11:35 pm, "Tim Griesser"<EMAIL_ADDRESS>wrote:

You're likely returning a promise to the transaction function which then
automatically commits or rolls back the transaction, in which case you
don't need to use trx.commit or trx.rollback
—
Reply to this email directly or view it on GitHub
https://github.com/tgriesser/knex/issues/1166#issuecomment-180082810.


Add support for keyless signatures for cosign

Describe the feature
Cosign offers yet another even more experimental feature: keyless signatures via OIDC. Connaisseur should attempt to support this based on Fulcio code signing certificates.
Optional: Implementation ideas
Not sure.

duplicate of #1493


Executing a "hot" backup for Swarm

First thank you very much for having followed on https://github.com/docker/docker.github.io/issues/1072
We can read:

It is possible to take a backup while the manager is running (a "hot" backup), but this is not recommended and your results will be less predictable when restoring.

Is there any plan to provide a safe way to do an hot backup ? Of course, no one will shut down their managers to do backups right ?
Thank you!

This is not a docs issue. Please file a feature request against the moby/moby repository. Thanks!


Fix Upcoming Matches displaying LIVE everywhere

Apologies for making another PR. The previous PR fixed the unknown message but because of the or with "ago" it would make each live. I might've missed the problem when I was testing myself but works perfect now. Feel free to confirm incase it's wrong.

fix the overall issue with this code change: https://github.com/axsddlr/vlrggapi/commit/26bcee708244001ccfda58a5680ce77dc1c7d686
additionally add feature to iterate through wf-modules to look for live matches

Your commit doesn't address the issue that every match in upcoming is appearing as "LIVE".
 The very bottom of the whole list displays as Live

@Chefski
I have a development branch with this fix. Try testing. Its fine on my end. https://github.com/axsddlr/vlrggapi/commit/b2f1bdc32cb1ac6a8ccf4ac9c5e7495205d6296a
Sorry I misunderstood the error and realized there's two redundant upcoming endpoints. This should fix the live issue and show all upcoming matches.

Oh I see the different branch now haha. I was looking on Github mobile. It's fine yeah! I was actually working on multiple matches appearing in live yesterday and was also gone remove live matches from upcoming haha. Thanks for doing that :)

Oh I see the different branch now haha. I was looking on Github mobile. It's fine yeah! I was actually working on multiple matches appearing in live yesterday and was also gone remove live matches from upcoming haha. Thanks for doing that :)

Havent worked on this project in a long while so had to test the endpoints against matches on the site. All good!


[WIP]move octant plugins to separate folder

To decouple Kubernetes API dependencies with octant,
move octant plugins to separate folder.
In this way, we will not have conflict K8s API dependencies with octant.

@antoninbas  Thanks for the comments. It is just one possible solution to decouple Kubernetes API dependencies with octant. I am just sending this for more opinions and not going to merge it. Yes, there will be some duplicate client code if we goes this way.
I have worked with Quan and found that we can use K8s API 1.17.6 (versions below 1.18.0 will work) and octant v0.10.2 (it can support both antrea-octant-plugin and traceflow-plugin) if we don't want to move octant-plugins to separate folder. And now we prefer to go with this combination.

Have checked in another PR to resolve this.


@import and @ref track dependencies

This PR introduces dependency tracking for @ref and @import directives.
When an @import directive is processed:

Add the current HttpRegion as a dependent to each of the imported file's global script regions.

When a @ref directive is processed:

Add the current HttpRegion as a dependent to the referenced HttpRegion.

When processHttpRegionActions is called:

Recursively go though all dependents that were previously added and reset them.
Resetting means deleting the response property and all variables stored from previous execution (if any).

The dependents for each region is tracked in an array property named refDependencies on ProcessorContext.options. Each item in the array contains a dependents array that refers to the dependent regions that need to be reset, when the region referred to by the entry is re-processed.
Resetting dependents internally keeps track of which region already have been reset when recursively walking the dependencies. That way circular dependencies are ignored in order to avoid stack overflows.

@AnWeber I don't understand why the tests fail. Why would this happen:

TypeError: utils.report is not a function

Thanks for the improvement


Change Title name of a field in a Dialog at runtime

Hey
how can I change a title of a field (eg Product Name to Product Internal Name) at runtime?.
I tried to get the field with jquery, but without succes.
and tried with this.form.field.getGridField() and this.form.field.element.closest(".field"), without success too.

Do you mean on the dialog or the grid? And do you mean the title of the dialog or grid, or a property field on the dialog or grid?

Title of a Grid: On top of your XYZEntityRow.cs class declaration you will see an attribute that says DisplayName("PluralEntities"). Changing the string inside of this DisplayName() attribute will change what is displayed as the title of the grid
Title of a Dialog: Similarly, next to the DisplayName() attribute on top of your entity class declaration you will see an attribute InstanceName("SingularEntity"). The string inside of this InstanceName() attribute will be displayed as the title of a new dialog

For property fields on the grid, decorate the property field whose column name you want changed with a [DisplayName("Product Internal Name")] attribute directly above it:
// ....
public class ProductColumns
{
    [DisplayName("Product Internal Name")]
    public string ProductName { get; set; }

    // ....
}

You can do the same thing for property fields on the XYZEntityForm.cs class as well to change the label for fields on a dialog form.

Hey man !
I mean the title of a Dialog.
These step you told about is changing the title before the runtime. So, the title will be static. To change, I need to set the DisplayName property, build and run again.
I need to change the dialog field title during the runtime, when my user is editing the data.
Why? Here in Brazil we have 2 kind of ID: Personal ID (eg CPF), and Business ID (eg CNPJ).
to identify the name of Personal ID, my field title will show "NAME". But to identify Business ID, my field title must show "RAZAO SOCIAL", that is the name of the business registered in oficial documents.

Dialog has a dialogTitle property. Best place to set it is in updateTitle method of entity dialog.

It's not working..
I tried
this.form.RazaoSocial.getGridField().text('MUDEI O NOME!!!');
and
this.form.RazaoSocial.getGridField().text(function (i, oldText) {
return oldText === 'Nome' ? 'Razão Social' : oldText;
});
but it replace all LINE to the changed text.
one idea that i have is how to get the id field at client side in run time (eg BELECH_Gestao_ClientesDialog10_NomeAbreviado)

What are you doing there. I said dialogTitle, not getGridField etc.

If you mean change title of field,
this.form.RazaoSocial.getGridField().find('.caption').text('asdadasd')

=)
Almost close !!!
So, the command above worked when the caption field is not a required field
this.form.RazaoSocial.getGridField().find('.caption').text('asdadasd')
but, if the caption is a required field, the command above not working well, because he will erase the "*" (signal of required field).
So, in these case to change the title of required field, the command below works good :
var texto = this.form.RazaoSocial.getGridField().find('.caption').prop('outerHTML').split('Razão Social').join('Nome');
this.form.RazaoSocial.getGridField().find('.caption').prop('outerHTML', texto);
Thanks @jsbUSMC and @volkanceylan for your time and patience!

PS.: For some reason, In typescript, String.Replace only replaces first occurrence of matched string.
So, I used the function split(oldtext).join(newtext)

Hi all,
I have made a WIKI article out of this with a generic solution which does not need a hard-coded "old value".
Hope this helps someone.
With kind regards,
John


(GH-1673) Add run_task_with plan function

This adds a new plan function run_task_with that runs a task on a set
of targets with target-specific parameters. It accepts a lambda that
returns a Hash of parameters to pass to the task run on a particular
target.
As an example, a task might expect a parameter with a value that is set
by a target's facts. The run_task_with function would have a lambda
that assigns this parameter with the appropriate fact:
$results = run_task_with('my_task', $targets) |$t| {
    { 'osfamily' => $t.facts['osfamily'] }
}

Similar to the run_task plan function, run_task_with also accepts
an optional description parameter, as well as three metaparameters:

_catch_errors
_noop
_run_as

Closes #1673
!feature

Added run_task_with plan function
(#1673)
The new plan function run_task_with lets you run tasks on a set of
targets with target-specific parameters. It accepts a lambda that
returns a Hash of parameters for a particular target.

CLA signed by all contributors.

This looks good to me but has failing tests.


install target

Please consider adding an install target to cmake.
Cheers, Armin

@achamayou, @davidchisnall do you have thoughts on the correct way to do this with respect to versioning?
I propose we install
libsnmallocshim.so
libsnmallocshim-1mib.so
libsnmallocshim-16mib.so
libsnmallocshim-static.a
libsnmallocshim-static-1mib.a
libsnmallocshim-static-16mib.a

I don't think we need to install the Rust static library, as that is only consumed via the crate.  If we expose, functions beyond the standard malloc interface, then we could expose a header, but currently we only do that via the header only version of the library, i.e. how Verona consumes it.
I don't think there is anything to install for the Open Enclave target as that is consumed and published via Open enclave.
Any thing else I should consider for this?

There is no need of an install target for Open Enclave. I suppose the .so could be versioned as .so.0 just in case?

We now have an install target.


Need a way to send synchronous keyboard events from embedder to Dart and get a response.

Internal: b/140830852
Similar to #28310, we need to be able to synchronously take keyboard events from the platform embedder, pass them to Dart to be handled, and if they are not handled, then pass them back to the platform to give it a chance to handle them.
This is needed so that add2app has a hope of working in environments with keyboards (like desktops), and so that we can capture keys before they get to a soft keyboard, as requested way back in #5986 (and has come up repeatedly).
We need a way to determine whether key events have been handled, synchronously, so that embedders know whether to continue propagating them in the native view hierarchy.
It needs to be synchronous because key events need to be responded to synchronously on host platforms.

Relatedly, there have been general conversations several times in the past about the need for a formal keyboard API in embedding.h, rather than passing text input and key events via a platform channel.

b/140830852

I have a Wayland Embedder client working with the exception of Keyboard.  I send key events, and they're dropped by the engine.  It looks like I need to modify the Engine to register a "flutter/keyevent" handler.  Will this change enable a default channel in the embedder for keyevents?

It looks like I need to modify the Engine to register a "flutter/keyevent" handler.

The listener is in the framework code, and is always present. You should not need to modify the engine.

Will this change enable a default channel in the embedder for keyevents?

No (per above), but it will provide an explicit API in embedder.h that embedders should use instead of the message channel.

I see said the blind man as he picked up the hammer and saw 😁.  Thanks for the pointer, it's all clear now 👍

IIRC, someone were experimenting on merging Platform Runner & UI Runner.
Would it be another option?

This was fixed by the various delay key event changes.  Closing.

This was fixed by the various delay key event changes.  Closing.

This can be moved to "Done" in project "Desktop Features", right?

@NilsUX Yes, done.  Thanks.

I have a Wayland Embedder client working with the exception of Keyboard. I send key events, and they're dropped by the engine. It looks like I need to modify the Engine to register a "flutter/keyevent" handler. Will this change enable a default channel in the embedder for keyevents?

Sorry to bump an old issue, but I'm having the same problem with this comment and I'm not sure where to ask.
I have successfully embed Flutter in my OpenGL app using software renderer. Rendering & mouse/pointer events works fine. However, I'm not able to send or receive any platform messages to/from the flutter engine:

The FlutterProjectArgs.platform_message_callback is never called, and
sending keyboard events via flutter/keyevent has no effect (I implement something similar to the glfw key handler)

Also if I use a native plugin, the MethodChannel never returns.
For the project args, I only filled assets_path, icu_data_path & platform_message_callback, and the rest I leave to the default values.
Are there other prerequisites for communication with flutter using platform messages?
Thanks!

@siavee That has nothing to do with this issue. Most likely you aren't servicing the engine tasks (either by providing a platform task runner, or regularly calling the deprecated __FlutterEngineFlushPendingTasksNow).
If that's not it, you should either file a new issue or use a resource like StackOverlfow or Discord, since this issue was about adding specific new functionality for key events.

@stuartmorgan Thanks for the response! That's probably it, I'll try providing a task runner and I'm very sorry about the noise.


Allow search for IP addresses within range

Issue type: Enhancement
Python version: 2.7.12
NetBox version: 2.1.2
Oddly, I find I cannot search for addresses within a range.
For example: I want to search for all IPAddress objects "192.168.7.x".  Doing a search for <IP_ADDRESS>/24 either in global search or in IP Address search returns no results.
Of course, if there exists a prefix object <IP_ADDRESS>/24 then it's fine: I find the prefix, and then look at the child addresses of that prefix.

There is a "parent prefix" field on the search panel to the right of the IP list. Use this parent filter to search for all IPs within an arbitrary prefix. The given prefix does not need to be defined in NetBox.
/ipam/ip-addresses/?parent=<IP_ADDRESS>/24

OK, that works, thank you.
So a suggestion for global search: if it returns no prefixes or addresses, but the query is in the form of a prefix, then search for all addresses within that prefix.


cannot connect to S3 (Min.io) :  "NoCredentialProviders: no valid providers in chain. Deprecated"

Hi there all,
lakeFS 0.33.1 has been configured (configuration attached below) to connect to a local Min.io process running on http port 9000.

All is installed on prem , no containers;
minio version: RELEASE.2021-03-01T04-20-55Z
lakeFS version: 0.33.1  (https://github.com/treeverse/lakeFS/releases/download/v0.33.1/lakeFS_0.33.1_Linux_x86_64.tar.gz)
os: RHEL 7.6

lakeFS throws the following warning during startup:

_WARNING[2021-04-06T19:08:56+02:00]lakeFS/pkg/logging/logger.go:119 pkg/logging.(*logrusEntryWrapper).Warnf NoCredentialProviders: no valid providers in chain. Deprecated.
        For verbose messaging see aws.Config.CredentialsChainVerboseErrors: failed to get AWS account ID for BI_

log capturing the  process start:
run.log
When trying to create a new repository via the UI the same error is thrown in the log:
ERROR  [2021-04-06T20:09:34+02:00]lakeFS/pkg/block/s3/adapter.go:174 pkg/block/s3.(*Adapter).streamToS3 failed to sign request                        error="NoCredentialProviders: no valid providers in chain. Deprecated.\n\tFor verbose messaging see aws.Config.CredentialsChainVerboseErrors" host="<IP_ADDRESS>:8000" method=POST operation=PutObject path=/api/v1/repositories request_id=0c8faf9f-afe9-4f60-8b90-8ca9c4947ca9 service_name=rest_api user=admin
WARNING[2021-04-06T20:09:34+02:00]lakeFS/pkg/api/controller.go:567 pkg/api.(*Controller).CreateRepositoryHandler.func1 Could not access storage namespace            error="NoCredentialProviders: no valid providers in chain. Deprecated.\n\tFor verbose messaging see aws.Config.CredentialsChainVerboseErrors" service=api_gateway storage_namespace="s3://datalake/"

Here you have the relevant s3 config (see lakefs.yaml  attached);
tried both static and file based credentials properties:
...
s3:
    region: us-east-1
    force_path_style: true
    endpoint: https://localhost:9000
    credentials:
      #access_key_id: admin
      #access_secret_key: Intesa01
      credentials_file: /cvai0/sw/lakefs_0.33.1/aws_credentials

...
lakefs.yaml:
lakefs.txt
aws_credentials file:
aws_credentials.txt
BTW: when trying to connect to Minio with another s3 client (eg s3cmd) all is working smoothless, ie:
s3cmd -c s3cfg ls s3://datalake/
s3cfg:
# Setup endpoint
host_base = localhost:9000
host_bucket = localhost:9000
bucket_location = us-east-1
use_https = False

# Setup access keys
access_key = admin
secret_key = Intesa01

# Enable S3 v4 signature APIs
signature_v2 = False

best regards,
thanks
Mario Ramaglia

Thanks for the detailed report!
I am a bit confused about another part of the s3 stanza:
s3:
    region: us-east-1
    force_path_style: true
    endpoint: https://localhost:9000
    credentials:
      #access_key_id: admin
      #access_secret_key: Intesa01
      credentials_file: /cvai0/sw/lakefs_0.33.1/aws_credentials

The S3 endpoint is https://localhost:9000.  I am not sure how lakeFS would agree to accept a certificate for localhost.  Could you try to connect on loopback over http?

hi there, thanks for your prompt comments;
as per your suggestions, I've tried with both loopback/<IP_ADDRESS> and the IP address (<IP_ADDRESS>) but the error is still there.
lakefs UI:

minio ui:

Both Minio "admin console" (Minio Server logs) and
"admin trace" (Minio server Http API tracing) report nothings
thanks,
best regards.

Also... :warning:  I think you're configuring under credentials, where you should be a level up (YAML whitespace confuses me):
s3:
    region: us-east-1
    force_path_style: true
    endpoint: https://localhost:9000
    credentials_file: /cvai0/sw/lakefs_0.33.1/aws_credentials
    #credentials:
      #access_key_id: admin
      #access_secret_key: Intesa01

Could you try that, please?

credentials_file goes under s3 you're right.
blockstore.s3.credentials_file (string : ) - If specified, will be used as a credentials file
blockstore.s3.credentials.access_key_id (string : ) - If specified, will be used as a static set of credential
blockstore.s3.credentials.access_secret_key (string : ) - If specified, will be used as a static set of credential

fixed and retried but nothing changed.
thanks

many thanks, fixed and the error now is :
ERROR  [2021-04-07T12:13:42+02:00]lakeFS/pkg/block/s3/adapter.go:202 pkg/block/s3.(*Adapter).streamToS3 error making request request                  error="Put \"https://<IP_ADDRESS>:9000/datalake/dummy\": http: server gave HTTP response to HTTPS client" host="<IP_ADDRESS>:8000" method=POST operation=PutObject path=/api/v1/repositories request_id=b1bebea4-d78c-48aa-89ed-269cf409a9f6 service_name=rest_api url="https://<IP_ADDRESS>:9000/datalake/dummy" user=admin
I guess there's no way to make lakefs talks http with Minio, right ?

If your minio server accets HTTP on port 9000, simply changing the endpoint from https://... to http://... should do the trick

whops...I didn't notice I left "https" in the s3 endpoint config. it works like a charm now.
many thanks.


Add custom emojis to deep dives embed (3)

Summary
This PR adds custom emojis to deep dive embeds as well as refactors some of the code
Before:

After:

Type of Change
Please select all that apply:

[ ] Bug Fix
[x] New Feature
[ ] Documentation Update

Included Changes
Changes from PRs (I) and (II)

Rename Type enum to DiveType - just a better name
Move primary and secondary deep dive stages to their own object Mission with type property built-in
Move all emojis generation to get_emoji
Add custom emojis to all objectives, anomalies and warnings
Add tests for that

New Changes (III)

Add IDs of emojis from support server
Rename the DiveType enum to DiveVariant - a more consistent name
Remove all logic from Deep Dive models and all embed generation logic from bot.py and move that to a new class DeepDiveManager
Update invite links to use new permissions sum
Use Imgur url as deep dive embed thumbnail, it's just simpler do do that rather than menage uploading file
Add check if Bosco has permission to use custom emojis

@MoritzHayden any update on that?

@MoritzHayden any update on that?

Sorry for the delay, life got really busy for me. I'll review this soon and we can hopefully have it merged by the new year. Are all the changes tested and working?

Well, I can't test it properly since I don't have a bot in the support server. But from my side, my emojis ID are working and permission warning also works,

According to this Discord blog post, it looks like this feature is no longer supported

According to this Discord blog post, it looks like this feature is no longer supported

Bots were never able to use emojis packs, this has nothing to do with using custom emojis.

Happy Birthday PR #60 🥳


Edit dropdown in discussion doesn't workt

Title.

Closing as the project isn't maintained anymore.


Update scalafmt-core to 3.4.3

Updates org.scalameta:scalafmt-core from 3.4.2 to 3.4.3.
GitHub Release Notes - Version Diff
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.scalameta", artifactId = "scalafmt-core" } ]

labels: library-update, early-semver-patch, semver-spec-patch, commit-count:1

Codecov Report

Merging #176 (6c92060) into master (2b2d934) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #176   +/-   ##
=======================================
  Coverage   80.35%   80.35%
=======================================
  Files           6        6
  Lines         112      112
=======================================
  Hits           90       90
  Misses         22       22

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 2b2d934...6c92060. Read the comment docs.


Missing confirm or reject after request - using unknown chargeTypeOwner

Environment B-002
Using unknown charge type owner "8100000000099" in request REM-033 / create price information - expect an reject, but I didn't get anything at all ( confirm/reject)
Please investigate and make a work around solution until we have full validation of chargeTypeOwner when the market roles are integrated.
AC1)
Receive a reject after using unknown Chargetypeowner
Request:
cim:businessSector.type23</cim:businessSector.type>
<cim:sender_MarketParticipant.mRID codingScheme="A10">8100000000047</cim:sender_MarketParticipant.mRID>
cim:sender_MarketParticipant.marketRole.typeDDM</cim:sender_MarketParticipant.marketRole.type>
<cim:receiver_MarketParticipant.mRID codingScheme="A10">5790001330552</cim:receiver_MarketParticipant.mRID>
cim:receiver_MarketParticipant.marketRole.typeDDZ</cim:receiver_MarketParticipant.marketRole.type>
cim:createdDateTime{{ISO8601Timestamp}}</cim:createdDateTime>
cim:MktActivityRecord
cim:mRIDOpId{{$isoTimestamp}}</cim:mRID>
cim:ChargeGroup
cim:ChargeType
<cim:chargeTypeOwner_MarketParticipant.mRID codingScheme="A10">8100000000099</cim:chargeTypeOwner_MarketParticipant.mRID>
cim:typeD02</cim:type>
cim:mRIDPI8_fee01</cim:mRID>
cim:namePI8 Charge Fee Name</cim:name>
cim:descriptionThe charge description</cim:description>
cim:priceTimeFrame_Period.resolutionP1M</cim:priceTimeFrame_Period.resolution>
cim:effectiveDate2021-11-30T23:00:00Z</cim:effectiveDate>
cim:VATPayerD01</cim:VATPayer>
cim:transparentInvoicingfalse</cim:transparentInvoicing>
cim:taxIndicatorfalse</cim:taxIndicator>

Likely to be solved by #1418

Solved by #1418 (Charge Information) / #1419 (Charge Prices)


Orochi support for Adrenalin 22.7.1 on Windows

Adrenalin 22.7.1 (driver version 31.0.12015.1008) seems to break Orochi as amdhip64.dll no longer exposes hiprtc* symbols. The driver doesn't seem to install hiprtc.dll either.

This is a regression in the driver. A fix is in here which we'll merge soon.
https://github.com/GPUOpen-LibrariesAndSDKs/Orochi/pull/36

Note that this dll may not be compatible with older driver. So we recommend to use it only with 22.7.1.

A fix is implemented and merged. https://github.com/GPUOpen-LibrariesAndSDKs/Orochi/pull/38

Hi,
curiously seems 22.8.1 is still missing hiprtc.dll also..
can use this dll on 22.8.1?
will take many time standard AMD drivers to include hiprtc.dll by default?
thanks..


Make colorpickercontrol refer to default color scheme

This will reduce the code that point to specific color scheme and let ColorSchemeManager be the single point of contact for obtaining color scheme.
Developers can set company-specific default color scheme in common.js.
@williaster @conglei @graceguo-supercat

Codecov Report

Merging #6031 into master will increase coverage by 0.02%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master    #6031      +/-   ##
==========================================
+ Coverage   64.63%   64.65%   +0.02%
==========================================
  Files         449      449
  Lines       23821    23817       -4
  Branches     2639     2639
==========================================
+ Hits        15396    15400       +4
+ Misses       8412     8404       -8
  Partials       13       13

Impacted Files
Coverage Δ

superset/assets/src/modules/colors.js
44% <ø> (-7.73%)
:arrow_down:

...explore/components/controls/ColorPickerControl.jsx
96.15% <100%> (ø)
:arrow_up:

superset/assets/src/modules/colorSchemes/airbnb.js
0% <0%> (-100%)
:arrow_down:

superset/assets/src/modules/colorSchemes/lyft.js
0% <0%> (-100%)
:arrow_down:

superset/connectors/sqla/models.py
81.13% <0%> (+0.75%)
:arrow_up:

superset/db_engine_specs.py
55.7% <0%> (+0.87%)
:arrow_up:

Continue to review full report at Codecov.

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


shall we disable SmartAPI metadata auto-sync on dev server?

For now, just to put some notes here for feedbacks from the team.
How about this:

Only enable automated smartapi sync (as a schedule job) when NODE_ENV=production (therefore only enabled for production server)
Add a npm run smartapi_sync script to manually trigger sync when needed.

Pros:

no local smartapi metadata changes during the development
can still test with the latest local metadata cache by manually triggering the sync
less unnecessary merging conflicts caused by the auto-updated local metadata cache files

Cons:

Works differently comparing to the prod server, will need to remember to test against latest metadata when relevant (but we will always test against the latest metadata during automated tests and tests before deployment)

Your thoughts?

This seems like a very good idea to me.  I can see the "con" of this being the root cause of some mysterious issue sometime in the future, but hopefully clear documentation (including this ticket) will help us solve the mystery quickly.

My requests are:

the prod server url has regular updates from the Registry/trapi-endpoints (like every hour or so?)
the local instances we start up have an option for manually refreshing from the registry/trapi-endpoints

@tokebe let's implement this issue as described on my first comment:

Only enable automated smartapi sync (as a schedule job) when NODE_ENV=production (therefore only enabled for production server)
Add a npm run smartapi_sync script to manually trigger sync when needed.

But also add a third-item:

Modify this issue : https://github.com/biothings/BioThings_Explorer_TRAPI/issues/252, to allow a SMARTAPI_SYNC=true|false ENV variable to manually trigger syncing (override the default behavior of the default behavior when needed)

What do you think?

@newgene For the third term, do you mean using it to manually trigger a single synchronization, and then not continue synchronizing?

@tokebe previously you implemented DISABLE_SMARTAPI_SYNC for disabling syncing when syncing is always on. Now we can control syncing on or off for our server, we might want to change DISABLE_SMARTAPI_SYNC to just SMARTAPI_SYNC, which can be used to override the default behavior, like SMARTAPI_SYNC=true will toggle syncing on regardless, and false value will toggle syncing off. It might just be the naming change of this ENV variable.

@newgene @tokebe Can this issue be closed now?

I believe so.


Posting Tagged Data Payload without protocolVersion fails

Describe the bug
Posting the Minimal Block with Tagged Data Payload example from https://editor.swagger.io/?url=https://raw.githubusercontent.com/iotaledger/tips/stardust-api/tips/TIP-0025/core-rest-api.yaml for /api/core/v2/blocks fails with
{"error":{"code":"400","message":"invalid parameter, error: invalid block, error: protocolVersion invalid: code=400, message=invalid parameter"}}.
Adding "protocolVersion": 2, doesn't return this error anymore.
To Reproduce
Steps to reproduce the behavior:
Send this request
curl -X POST https://api.testnet.shimmer.network/api/core/v2/blocks -H 'Content-Type: application/json' -d '{
  "payload": {
    "type": 5,
    "tag": "0x68656c6c6f20776f726c64",
    "data": "0x5370616d6d696e6720646174612e0a436f756e743a203037323935320a54696d657374616d703a20323032312d30322d31315431303a32333a34392b30313a30300a54697073656c656374696f6e3a203934c2b573"
  }
}'

Expected behavior
It should work (or with the URL in my example return {"error":{"code":"400","message":"invalid parameter, error: no parents given and node PoW is disabled: invalid block: code=400, message=invalid parameter"}} instead)

according to TIP-0024 a block should include protocol version, parents, payload, and nonce.

Yes the final block needs to have the fields, but afaik posting just the payload should also work and the node will add the other fields (when remote PoW is enabled)

TIP-25 Node Core REST API was outdated and wrong


Can't compile WinMSRDriver

I'm trying to compile WinMSRDriver under Windows 10 within the VS 2015 developer command prompt. When it tries to run nmake it errors with makefile(1) : fatal error U1052: file '\makefile.def' not found. I downloaded WDK 10 in the hope this is compatible (as the instructions mention the DDK).
Otherwise, I could compile the VS solution without problems, it's just the driver which is making trouble.

we do not support WDK10. Could you try compiling it with WDK 8.1? WDK8.1 also has an option/utility to convert the makefile/sources into a MSVC project.

I have tried to compile WinMSRDriver under Windows 10 within VS 2015 with 14393 DDK.
I created new VS 2015 KMDF Project, then manually added all h and c files and compiled it. It worked fine and generated msr.sys file.
After that I generated pcm.exe but when I run pcm.exe  I get this error - SystemAccessViolation Exception

SystemAccessViolation issue has been resolved (fix merged)

@anurag-daware could you please create a pull request with your VS 2015 KMDF project? Thank you!

sorry for the delay
Not sure how to create a pull request with my VS 2015 Win MSR Project from this repo (I use perforce extensively and newbie in git).
Here I have uploaded the same with new repo- https://github.com/anurag-daware/MSRWinDriver_VS2015
Let me know how can I help further.

@anurag-daware  could you try to create follow this manual (https://gist.github.com/Chaser324/ce0505fbed06b947d962) : create a fork add just MSR.inf and MSR.vcxproj into WinMSRDriver/Win7 directory and then submit a pull request?
Thank you

@anurag-daware  thanks for your help! I am closing this issue now

MSR.vcxproj is in the master branch now


Add documentation for GITHUB_TOKEN requirement

The documentation does not make it very clear what type of access is required:

fine-grained personal access token ?
personal access token (classic) ?

What type of permissions ?
Thanks

This mainly because it was unclear from the REST API docs what exact permission it needed to execute.
You are correct though. We should mention explicitly what permissions are used when the token is required.

Just so you can see what vagueness I'm working with:

REST API list of endpoints for commits and issues which is used for PR threads also. The pulls endpoints are about PR reviews which are much more complex to implement, so we don't use the pulls endpoints.
We use the following REST API when tokens are needed:

For a PR:

GET /repos/{owner}/{repo}/issues/{issue_number}/comments (needed for fetching existing comments)
DELETE /repos/{owner}/{repo}/issues/comments/{comment_id} (used to delete comments that this action creates)
POST /repos/{owner}/{repo}/issues/{issue_number}/comments (used to create a new thread comment)

For a commit (push event):

GET /repos/{owner}/{repo}/commits/{commit_sha}/comments (needed for fetching existing comments)
DELETE /repos/{owner}/{repo}/comments/{comment_id} (used to delete comments that this action creates)
POST /repos/{owner}/{repo}/commits/{commit_sha}/comments (used to create a new thread comment)

fine-grained personal access token ?

This feature is still in beta and lacks adequate documentation to answer the question. While it might work with this action, we won't be held responsible if it doesn't.

From what I can gather, I think the following permissions are used when a token is required (for a public repo). Note, this is mostly guesses since the REST API doesn't always mention for what the token needs permission.
permissions:
  checks: read|write
  # write for our file-annotations option (I think - probably not)

  pull-requests: read|write
  # write to remove duplicate action comments and post new comments
  # read to get a list of changed files and a list of existing comments

  repository-projects: read|write  # (for push events)
  # write to remove duplicate action comments and post new comments
  # read to get list of changed files and a list of existing comments

See also Assigning permissions to jobs.
Private repo access seems largely undocumented, but we have [reportedly] run into permission problems when getting a list of PR comments and a list of changed files.

I think the only way we can be certain what permissions are needed is by trial and error (we have a test repo for that). But my time is currently limited, and I would very much appreciate user feedback on what permission settings worked and what didn't.

Could also comment, why at least one alternative solution works out of the box (no need for GITHUB_TOKEN setup):

https://github.com/marketplace/actions/clang-tidy-review#inputs

They use the token with implied consent, and it is unclear what they need it for without delving into the source code.
We refuse to confiscate permission without user consent. That's why we won't use a default value for the token. It is better to know when your permission is needed for something, instead of giving us the house keys and trusting we don't burn the house down.

It seems the GIthub REST API docs have been updated again. They now explicitly state what permissions are needed for each REST API endpoint.
Re-opened this as it should be addressed in #198


Param "exhaustive_search" only works as lowercase string ("true", or "false"). Python booleans not working.

Description

Param "exhaustive_search" only works as lowercase string ("true", or "false"). Python booleans not working.
Steps to reproduce

Expected Behavior

Actual Behavior

Metadata
Typsense Version:
OS:

Fixed via https://github.com/typesense/typesense-python/pull/29


The subscriptions page is broken when youtube center is enabled

It squishes the row of videos in the subscriptions page to 1 1/2 rows of videos.

I found this thread because I had the same issue. You can work around it if you switch the layout. On top of the first video's thumbnail you can change from grid to list layout, which does work properly.

You can add https://www.youtube.com/feed/subscriptions to excluded pages as a workaround.

Did you try to disable grid subscriptions from YTC settings?

I can't find that setting.. (Developer Version - Build #538)

i excluded too, waiting a fix :yum:

@ErdeFB This fixed it, thank you.

Since I already had this issue and fixed it with the grid option I would now like to know what is actually the problem? I mean, is that not considered a bug or is it one and that's an actual workaround for now?


Update sap_checklist.en.json

Items related to "Storage" have been deleted.
Please feel free to add more in the future if necessary.

Hey @naokilgarashi, sorry I didnt see this one! Could you please remove the comma in line 1080?
@alastairmorrison @mottach, is the change OK for you? If so could you please approve?
Thanks!
Jose

[like]  Naoki Igarashi reacted to your message:

From: Jose Moreno @.>
Sent: Tuesday, January 30, 2024 9:21:57 AM
To: Azure/review-checklists @.>
Cc: Naoki Igarashi @.>; Author @.>
Subject: Re: [Azure/review-checklists] Update sap_checklist.en.json (PR #626)
Merged #626https://github.com/Azure/review-checklists/pull/626 into main.
—
Reply to this email directly, view it on GitHubhttps://github.com/Azure/review-checklists/pull/626#event-11638688909, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ANX4D5KNPLCNXSE57ZHW4V3YRC3TLAVCNFSM6AAAAABB74R6ASVHI2DSMVQWIX3LMV45UABCJFZXG5LFIV3GK3TUJZXXI2LGNFRWC5DJN5XDWMJRGYZTQNRYHA4TAOI.
You are receiving this because you authored the thread.Message ID: @.***>


Arci steam farm doesn't work

My problem is, the program just write this: This account has nothing to farm

Please review the guidelines for contributing to this repository.

https://github.com/JustArchi/ArchiSteamFarm/blob/master/CONTRIBUTING.md

If my issue is not meeting contributing guidelines specified above, especially if it's a question or technical issue, please close my issue immediately - I won't complain.