Plugin is connected to websocket server successfully but cannot receive data

My grafana server is connected with my websocket API successfully (checked with logs file). However, when I try to send a string JSON, the plugin cannot receive and show the data. I tested my server with other clients which are all succeed to receive the JSON string. Is there any thing I need to consider when send JSON to websocket plugin?
Programming Language: Python (websockets v10)
Grafana: v8.4.4

@tungshido could you give more details of our problem?

A sample/example of the exact string JSON that you are sending through your websocket API?

@tungshido could you give more details of your problem?

A sample/example of the exact string JSON that you are sending through your websocket API?

Hi, this is my sample code to try stream data. I would like to send simple Json like this {"result":18}
Server
async def hello(websocket):
    name = await websocket.recv()
    await websocket.send(name)

async def main():
    async with websockets.serve(hello, "localhost", 8765):
        await asyncio.Future()  # run forever

if __name__ == "__main__":
    asyncio.run(main())

Client
async def hello():
    uri = "ws://localhost:8765/v1/ws/data/"
    async with websockets.connect(uri) as websocket:
        await websocket.send(json.dumps({"result":18}))
        greeting = await websocket.recv()
        print(f"<<< {greeting}")

if __name__ == "__main__":
    asyncio.run(hello())

Grafana logs
logger=plugin.golioth-websocket-datasource t=2022-03-31T21:36:27.03+0700 lvl=info msg="RunStream called" Path=.
logger=plugin.golioth-websocket-datasource t=2022-03-31T21:36:27.03+0700 lvl=info msg="Ws Connect" connectingto=ws://localhost:8765/v1/ws/data
logger=plugin.golioth-websocket-datasource t=2022-03-31T21:36:27.03+0700 lvl=info msg="Ws Connect" connectedto=ws://localhost:8765/v1/ws/data

@tungshido Do you mind explaining what you did to make it work? I'm facing this same issue where I do not know how to send a payload (json.dumps) in this plugin as well. Thanks!


distribution variable names in error messages to match manual

Change our error messages to describe variables using the descriptive name and the variable name from the manual, as in "scale argument sigma" or "location argument mu"
We don't want to use "parameter" because it can be confused with the parameters in a Stan program.  And names like "precision" can be confused with numerical precision (thanks to https://github.com/jeffalstott for pointing this out and suggesting the new form in #297;  this issue is more general and calls for a review of all of our distribution functions).

I was just looking at this. Should we just refer to things as their argument names? That would be pretty easy to fix and make the manual match.
For example, for the normal distribution:

y instead of random variable
mu instead of location parameter
sigma instead of scale parameter
?


[feature request] Add option for whitelist(positive search)

it would be nice to specify only what file types you want like -w php,txt,xhtml
thanks!

Hello!,
Would be great, but I don't have the time to make it happen.
But you can do something like
grep -vE "robot.txt$|robots.txt\/$|.gif|.jpg|.jpeg|.css|.tiff|.ico|.png|.svg|.ttf|.woff|.woff2|.eot|.pdf|.js" "gauplus_results.txt" > "gauplus_clean.txt"
change it to whatever you need.

True. I could also pipe through grep or fork the repo and try and make the changes myself. Anyways, i figured i would ask.
Thanks for a great tool and a quick response!


[Bug] Trim the tree after removal

Describe the bug
We currently only trim the tree when adding nodes
Expected behavior
The tree should also be trimmed when removing nodes.

This is now mentioned in https://github.com/mlswg/mls-protocol/issues/459.

And the spec is fixed in https://github.com/mlswg/mls-protocol/pull/461

Fixed in spec and code: Closing.


Edit Functionality Implemented Dynamically from Sdk

IMPORTANT: Where possible all PRs must be linked to a Github issue
Fixes https://github.com/opensrp/fhircore/issues/79
Type
Choose one: (Feature)
Checklist

[x] I have written Unit tests for any new feature(s) and edge cases for bug fixes
[x] I have added any strings visible on UI components to the strings.xml file
[x] I have updated the  CHANGELOG.md file for any notable changes to the codebase
[x] I have run ./gradlew spotlessApply and ./gradlew spotlessCheck to check my code follows the project's style guide
[x] I have built and run the fhircore app to verify my change fixes the issue and/or does not break the app

Pull Request Test Coverage Report for Build #293

0 of 0   changed or added relevant lines in 0 files are covered.
35 unchanged lines in 2 files lost coverage.
Overall coverage decreased (-1.4%) to 89.971%

Files with Coverage Reduction
New Missed Lines
%

src/main/java/org/smartregister/fhircore/activity/RecordVaccineActivity.kt
9
88.75%

src/main/java/org/smartregister/fhircore/activity/core/QuestionnaireActivity.kt
26
64.86%

Totals

Change from base Build #292:
-1.4%

Covered Lines:
1561

Relevant Lines:
1735

💛  - Coveralls

plz check the commit history is starting from Jun2

plz check the commit history is starting from Jun2

good point, something looks off there

plz check the commit history is starting from Jun2

good point, something looks off there

The reason is, this feature keep updating in the sdk, and then tested into fhir core until we successfully completed and merged it in the SDK. that is why the commit history is starting from June 2


Correct missing - in vtgate flag

Signed-off-by: Florent Poinsard<EMAIL_ADDRESS>

ansible is failing


domainLock should work in SVG like in HTML

Expected Behavior
domainLock should work in SVG like in HTML.
Current Behavior
JavaScript in SVG always fails if using domainLock.
Steps to Reproduce (for bugs)
Any JavaScript file included in SVG will fail if domainLock.
Problem apparently is it looks for document.domain, which only is there in HTML, but not in SVG.
What it should do is if document.domain doesn't exist it should look at document.location.hostname, which is there in SVG.
Apparently document.location.hostname is there in both HTML and SVG, maybe just use that.
Wasn't able to tell for sure how that code works in javascript-obfuscator/src/templates/domain-lock-nodes/domain-lock-node/DomainLockNodeTemplate.ts else would have forked and tried to fix.

I have succeeded at rigging my locally installed copy to run in both SVG and HTML with DomainLockNodeTemplate.ts having
if ((!document && !domain) || (!that[document] && !that[document][domain] &&
     !that[document]["location"] && !that[document]["location"]["hostname"])) {
    return;
}

var currentDomain = that[document][domain] || that[document]["location"]["hostname"];

but as I see interesting choices made around that, I let @sanex3339 pick how exactly he wants to roll that in.
As mentioned, maybe just use document.location.hostname instead of document.domain for less conditionals, shorter overall code?

0.17.0 version in out!


Can't adjust the descriptor_key for the rate limiting based on remote_address

Description:
I configured the global rate limit on Istio/Envoy using Envoyfilter.
My rate limit configuration for the ratelimit service looks like this.
configmap
...yaml
domain: mydomain.com
descriptors:
    - key: remote_address
      value: <IP_ADDRESS>
      rate_limit:
        unit: minute
        requests_per_unit: 2

...

And my envoyfilter looks like this.
Spec:
  Config Patches:
    Apply To:  VIRTUAL_HOST
    Match:
      Context:  GATEWAY
      Route Configuration:
        Vhost:
          Name:  mydomain.com:80
          Route:
    Patch:
      Operation:  MERGE
      Value:
        rate_limits:
          Actions:
            remote_address:
.
.
.

When I send a request, the envoy sends a ratelimit request to the ratelimit service, but it looks for a different descriptor key.
The ratelimit service uses mydomain.com.remote_address_<IP_ADDRESS> as a key.
{"@message":"loading descriptor: key=mydomain.com.remote_address_<IP_ADDRESS> ratelimit={requests_per_unit=2, unit=MINUTE, unlimited=false}","@timestamp":"2022-05-27T10:57:46.115910779+09:00","level":"debug"}

While the envoy sends a request with remote_address_<IP_ADDRESS>.
{"@message":"looking up key: remote_address_<IP_ADDRESS>","@timestamp":"2022-05-27T11:00:33.149314244+09:00","level":"debug"}

It looks the RemoteAddress config does not have any way to configure a descriptor key.
https://www.envoyproxy.io/docs/envoy/latest/api-v3/config/route/v3/route_components.proto#config-route-v3-ratelimit-action-remoteaddress
How can I do the rate limiting based on the remote address?
I am using the 1.21.1 version of envoy.
k exec -it istio-ingressgateway-gateway5-ff8bb5df9-wmf52 -c istio-proxy  -- pilot-agent request GET server_info --log_as_json | jq {version}
{
  "version": "15cc87023540f5f81331c11ef9010784f7ef2460/1.21.2-dev/Clean/RELEASE/BoringSSL"
}

I'm not very familiar with the RLS server implementation. Envoy uses a fixed key literally called remote_address for this action. Maybe it's the issue with the mis-matched rate limit domain? It's not virtual host.

@kyessenov Thank you for the answer.
We are supposed to specify the domain in the configmap for reatelimit service configuration.
...yaml
domain: mydomain.com  # mandatory.
descriptors:
    - key: remote_address
      value: <IP_ADDRESS>
      rate_limit:
        unit: minute
        requests_per_unit: 2

...

If you mentioned about the EnvoyFilter, could you let me know where to put such configurations?

closing as this was because of misconfiguration.
The domain is already considered.


Extracted useful subinterfaces of HybridEditorSpec and made SimpleHybridSynchronyzer work without parser

Why did you rename TokenCompletion to EditorCompleter. This class still completes tokens.
P.S. renamings are better called renames.


Missing JAX-RS 2.0 module

While working on CXF support for Swagger, we have noticed that the swagger-jaxrs module is missing support for @MatrixParam and @BeanParam.
While supporting the former seems to be controversial (swagger-api/swagger-spec#69) at the moment - even though part of the JAX-RS 1.1 spec, the latter is instead currently implemented by swagger-jersey2-jaxrs, even if this is actually a standard JAX-RS 2.0 feature rather than Jersey's.
The initial work for adding such support is currently hold in the PR apache/cxf#100.
Is it fine if I provide a PR here to create a new swagger-jaxrs2 module, and to refactor swagger-jersey2-jaxrs to work with it?

There's no support for @MatrixParam because it is not supported by the Swagger spec - see https://github.com/swagger-api/swagger-spec/issues/69.
As for @BeanParam support, that's currently provided only by swagger-jersey2-jaxrs which can be used for CXF as well. As it currently stands, that's the module that supports JAX-RS2. Yes, it pulls in Jersey as well, but that shouldn't affect the application. See https://github.com/swagger-api/swagger-core/issues/1244.

What is wrong with having a swagger-jaxrs20 which would be implementation neutral and a Jersey specific -jersey module that would depend on swagger-jaxrs20 ?
BeanParam is not Jersey specific. swagger-jaxrs is not called swagger-jersey-jaxrs11.
Would you consider splitting  this module into two, a neutral and Jersey specific ?
Cheers, Sergey

I recognize now this issue is kind of duplicate of #1244 by @elakito

Nothing is wrong with having it, we just never got around to changing the structure of the project. That's exactly what's suggested in #1244.


Remove workaround for DoctrineMigrationsBundle

The bundle uses Doctrine\Migrations\Configuration\Connection\ExistingConnection by default. Although the connection is configurable via a separate config file or cli argument, this will not work because the ExistingConnection behaviour prevents overriding.
This issue is currently not resolvable and should be checked again from time to time.
The workaround was introduced in #64.

The problem above is resolved. There's a new issue. To be able to use multiple connections, the bundle requires a default connection. Still not resolvable and should be checked again from time to time.
The new workaround was introduced in #70.

Not going to be fixed. It is what it is.


Input/Output terminology confusing

This library's choice of jargon is that a mailbox's message receiver is "output", and its message supplier is "input".  This makes sense from the perspective of a person interacting with a mailbox, since the message is going out of the person's hand (into the mailbox), or into the person's hand (from the mailbox).  However, if you flip the perspective to the mailbox itself, the reverse terminology makes sense.  Neither approach seems inherently superior to the other, but some people may intuit the wrong meanings of the terms, leading to confusion.
This arbitrariness suggests that these aren't quite the right words to use.  The phrases "output from the mailbox" and "input from the mailbox" essentially mean the same thing, just from different perspectives.  Similarly, "input to the mailbox" and "output to the mailbox" equivalently describe the flow of data.  The key words that distinguish them are the "to" and "from" and their direct objects, not the "input" and "output".

I'm probably not going to change the name of these at this point, given the age of the library
I'm also not sure what a better naming scheme would be

Since we're just bike-shedding...
I think input and output are the right names for this particular job, and I find them intuitive to use.  Usually, I think of the computational event (that is usually anthropomorphized as a visually-challenged person) as central to the narrative drama and the messaging effect (aka the mailbox) as a piece of necessary but ultimately boring plumbing.
So the in- and the out- seem natural directions.  in being a reserved word we need some sort of suffix, and -put comes from the latin 'putare' meaning 'to think', so an input is an effect going in to a think, and an output is an effect going out of a think.  Perfectly named.

@Gabriel439 Regarding what a better naming scheme would be, perhaps Source instead of Input, and Sink instead of Output.  To me this seems less ambiguous, less dependent on whether you're thinking from the perspective of the user of the mailbox or from the perspective of the mailbox itself.  Of course, I understand your point about age.  It's probably too late to make such a change, now.  Should I just close this issue?

Yeah, I'm probably not going to change the name, so I'll go ahead and close the issue


Add safe2 as signatory

Can you fix the : in the yaml file and move the logo to the members dir under images?

Sorry about that Harper - changes made as requested


Older <4.4 Android Compatibility

Notes:

https://github.com/kivy/python-for-android/pull/903

The issue appears to be with loading sqlite3 https://github.com/kivy/python-for-android/issues/925

Based on barriers around target API versions and Play Store requirements, I don't think we're pushing for versions older than this now.


⚠ Minetest Wiki (for comparison against mirror) has degraded performance

In 6e4b9e5, Minetest Wiki (for comparison against mirror) (https://wiki.minetest.net/Main_Page) experienced degraded performance:

HTTP code: 200
Response time: 3637 ms

Resolved: Minetest Wiki (for comparison against mirror) performance has improved in 23f3e9d after 10 minutes.


python3Packages.mechanicalsoup: 1.0.0 -> 1.1.0

Motivation for this change
Update to latest upstream release 1.1.0
Change log: https://github.com/MechanicalSoup/MechanicalSoup/releases/tag/v1.1.0

Rename acc. PEP503
Switch to GitHub as source
Enable tests

Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Added a release notes entry if the change is major or breaking
[x] Fits CONTRIBUTING.md.

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

2 packages built:

python38Packages.mechanicalsoup
python39Packages.mechanicalsoup

Result of nixpkgs-review pr 124978 at 54584a99 run on aarch64-linux 1

2 packages built successfully:

python38Packages.mechanicalsoup
python39Packages.mechanicalsoup


Integrate new error status codes from Provider

Blocked by: https://github.com/oceanprotocol/provider/issues/523
Display user-friendly error messages for C2D failed jobs

The provider issue is merged. We can start working on this one

Waiting for the documentation of the provider errors before merging this

fixed with #1940


Allow user to avoid creating declare_maven

I'm guessing it allows Stripe to customize the callback?
I'm not sure it's useful for most users. Perhaps allow users to tell bazel-deps "create a callback inside 3rdparty/workspace.bzl while you write it?

yes. Many companies have their own nexus or maven cache. I think it is common to want to customize that industrially. That is the mechanism.
We can make it so a default works. PR?

@johnynek can you explain how customizing the callback helps you with your internal artifactory? I think I'm missing something


Proguard editing buttons not visible in Firefox

Version: 1.3.0 but was also broken in 1.2.1
Chrome:

Firefox:

I can click the buttons but they are invisible in Firefox.

They show up if i change to column widths in Firefox.

I'm aware of this issue and have reported it upstream https://github.com/vaadin/vaadin-grid/issues/2152

You can thumb up the upstream issue to indicate you're impacted by the issue. That might up its priority for the folks at vaadin.

Done. I also added a comment: https://github.com/vaadin/vaadin-grid/issues/2152#issuecomment-828251847


an idea to improve the validation of large datasets

Hello Alexey,i want to add a functionality to my version of this tool which will move the passed image (with the right arrow key or the space bar) and it's annotation file (.txt file) to another specified folder because when validating the annotations of large datasets it can take some days to complete and without this you will have to leave the pc running,and if you didn't take note of the last image or the index and something happens like a power failure or an error like BSOD you will lose the progress.Can you point me to the code line where the tool saves the annotation to the text file and moves on to the next image so i can insert few lines of code in there to create this functionality.

@k00r Hi,
What progress do you want to save? When you go to the new image by click (or arrow key), the txt-label file will be saved to the disk. So after BSOD, you just can open your dataset, and list images (hold SPACE-key) until the image will not have labels.
In this part of code the txt-label will be saved: https://github.com/AlexeyAB/Yolo_mark/blob/5082b30c51b13dbef4da1f8b407997a5320da267/main.cpp#L344-L374

no,for example you want to change some annotations on a previously annotated dataset or add a new object class,in these cases there is no way of knowing your progress because all of the image files will have labels.I want to move the checked image and it's text file to another folder after the yolo_mark tool's save operation.

May be you can try to sort txt-labels by date of creation.
Or as you want - you can try to save new labels to the new directory - just change this line: https://github.com/AlexeyAB/Yolo_mark/blob/5082b30c51b13dbef4da1f8b407997a5320da267/main.cpp#L351
To something like this:
std::string const txt_filename_path = images_path + "/new_directory/" + txt_filename;

i want to move the image also,i want something like this;
-right arrow key pressed
-annotations are saved to the text file
-move the text file and the image file to the new folder
-jump to the next image


Improvising README.md

I was thinking🤔 to compress the 📖README.md so that it become easier to read for the users, but I want suggestions on how to do it or should not do it at all.

should I break the main file in multiple file (one for contributing instructions, one for installation instructions etc) and link them in the main readme.md file

orshould I use a drop down menu like this one

or something else....(your suggestion)🤔

Hi @Arsenic-ATG,
The seconds one is ok.

@ming-tsai thanks for suggestion
I have used that method to compress the readme.md, you can see the the changes on my fork to get an rough idea how things will look after changes.
there is still some 🛠formatting left to do before launching a PR.
any suggestions would be appreciated.🙂


check-ttf: add check to determine if legacy version has ttfautohint x-height rounding enabled

As the title suggests. We're receiving several issues over on fonts/google regarding this matter, 644, 528.
Imo, this should be treated as a regression issue and the settings should match the previous version as best as possible

Alright, the x-height rounding values are set in the fpgm table.
I've compared 3 fonts:
auto, x-height rounding=20, x-height rounding=0

I'll see if fontTools.ttLib will allow me to parse this. If so, I'll start writing an FB test.

Implemented in pr #1229,


[BUG REPORT]: NBM QPF - Geoproc - wrong band

Wrong band is being selected for geoproc:

Should be addressed in : https://github.com/USACE/cumulus/issues/296 - it was not

@adamscarberry, I'll work on test coverage around this one. Stay tuned!

thanks @brettpalmberg   looks good
QPF Verification

Airtemp verification


Change value of todayBtn to linked

https://github.com/c0shea/AdminLTE-Template/blob/576ebecb26cceeaa992671f6bb3e6063303f474e/src/AdminLTE Template/Content/js/init.js#L5
https://bootstrap-datepicker.readthedocs.io/en/latest/options.html#todaybtn
True only brings today's date into view while 'linked' also selects today's date.

Resolved in #27


Support list_name in OSM worksheet.

Dealias and group headers for OSM worksheets.
In the OSM worksheet headers resolve all aliases and group them before generating the OSM tags.
closes #109

In the xml__contains strings in your tests, could you slim down the provided array of strings to test just the aspects of the XML document relevant to the OSM features?
An example is in the other tests, like the test_groups.py file, where individual XML changes are tested.

@dorey I have reduced the right side of the test/equality to the minimum.

@urbanslug confirming the OSM tags are present is equally important as well.


Question: Metal support

Platform: iOS
OpenGL was deprecated in iOS 12. Do you have any plans to add metal support? And how hard would it be?

CARTO's mobile-sdk (formerly Nutiteq) uses https://github.com/kakashidinho/metalangle to run their rendering on Metal instead of OpenGL. https://github.com/CartoDB/mobile-sdk/blob/master/NEWS.md


[SPARK-34747][SQL][DOCS] Add virtual operators to the built-in function document

What changes were proposed in this pull request?

This PR fix an issue that virtual operators (||, !=, <>, between and case) are absent from the Spark SQL Built-in functions document.
Why are the changes needed?

The document should explain about all the supported built-in operators.
Does this PR introduce any user-facing change?

No.
How was this patch tested?

Built the document with SKIP_SCALADOC=1 SKIP_RDOC=1 SKIP_PYTHONDOC=1 bundler exec jekyll build and then, confirmed the document.

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/136075/

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/40658/

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/136079/

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/40662/

Build failure on GA is not related to this change.
If there is no more comment or objection, I'll merge this tomorrow.

Merged to master. branch-3.1 and branch-3.0

Oh, you already merged. Thanks.


try

Codecov Report

Merging #13 (7f5990b) into dev2 (6155357) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             dev2      #13   +/-   ##
=======================================
  Coverage   63.97%   63.97%
=======================================
  Files          87       87
  Lines        4847     4847
=======================================
  Hits         3101     3101
  Misses       1746     1746

Flag
Coverage Δ

singa-cpp
63.97% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Continue to review full report at Codecov.

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

Codecov Report

Merging #13 (7f5990b) into dev2 (6155357) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             dev2      #13   +/-   ##
=======================================
  Coverage   63.97%   63.97%
=======================================
  Files          87       87
  Lines        4847     4847
=======================================
  Hits         3101     3101
  Misses       1746     1746

Flag
Coverage Δ

singa-cpp
63.97% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Continue to review full report at Codecov.

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


Restore static CUDA runtime linking

Fixes #2162.  This problem was caused by a kvikio change that requires downstream projects to set CUDA_STATIC_RUNTIME when they need a static runtime.
In the process of investigating this, I discovered there's an existing rapids_cuda_set_runtime that's supposed to be used for specifying the CUDA runtime settings.  I updated the CMakeLists.txt to use it, although it does not by itself fix the issue.  The CUDA_STATIC_RUNTIME variable is required, but we should update to the RAPIDS method for setting CUDA runtime parameters to match how libcudf is doing it in case the implementation details change in the future.

build

build


Release process to be modified for testenv_arm32.properties

testenv_arm32.properties in the aqa-tests branch needs to be updated to point at the right place once the GA tags for that release are in place (they are date stamped so can't be predicted in advance ...)
Originally posted by @sxa in https://github.com/temurin-compliance/temurin-compliance/issues/476#issuecomment-2066860262

@smlambert Hi Shelley! Is the testenv file automatically populated now so that this issue isn't needed?

The scripts are created (re: https://github.com/adoptium/github-release-scripts/issues/151), but not quite hooked into our flow (a couple of bugs still present), so still a manual step.


Feature Request: Access list of translation items from external

Background

I have a fairly large AngularJS + angular-formly + angular-translate front end...
which uses 2-3 non-angular components like tinyMCE
I have custom translations on these third party, non-angular components
...which I must provide with a list of translations while they load

I have two options:

create another, completely separate language management for these components
manage the languages in angular-translate, and then write a few lines of code to pass on the relevant values

I prefer option two, but this requires that I can say something like (pseudo code)
var x = translate.dictionary.Extensions.TinyMce;
foreach(key in x)
   addToTinyMce(key, ...)

this is not possible. As far as i can see the list of translations cannot be retrieved from outside.

@iJungleboy  Well, we can provide something like $translate.getCurrentTranslations() only as the translate service does not know  which translation could be possible. Said this, if you would call getCurrentTranslations() before the loader has (successfully) completed, you would even get an empty list. If you use something like a partial loader, you will get incomplete list. If you use a mixed setup with fallbacks, this actual "list of current translations" is not that easy as it looks in the first view.
I'll mark this as a feature request with call for contribution. A nice idea, but definitely with some work to do.

For anybody else trying to find a temporary workaround, my solution is to keep a comma-separated list of keys for all items I need to forward to TinyMCE, and then loop through these to lookup the translations and then forward them to TinyMCE. Not perfect, but works and keeps all my translations in one place.

+1 would be perfect if you could read and overwrite translations from outside of angular. Right now I'm using a custom loader using translations from an external variable.

How about if it just returns object like $translationTable[traslation_id]

Landed.

Very cool :). I'll test it within the next few weeks.

To anybody who is interested - here's how I used the angular-translate table to translate TinyMCE in 2sxc
    // Initialize the tinymce resources which we translate ourselves
    function initLangResources(editor, language, $translate) {
        var keys = [], mceTranslations = {}, prefix = "Extension.TinyMce.", pLen = prefix.length, primaryLan = "en";

        // find all relevant keys by querying the primary language
        var all = $translate.getTranslationTable(primaryLan);
        for (var key in all)
            if (key.indexOf(prefix) === 0)
                keys.push(key);

        var translations = $translate.instant(keys);

        for (var k = 0; k < keys.length; k++)
            mceTranslations[keys[k].substring(pLen)] = translations[keys[k]];
        tinymce.addI18n(language, mceTranslations);
    }


Restore snapshot wording

Hi
When restoring a snapshot on the english version of the ui the following comes up:

Instead of "I knew" a more appropriate translation would probably be "I acknowledge"

@Opswatch
Thank you for your advice, we will reply to you as soon as possible.

Fixed on 4.0 closing this.


Fix Unexpected Functionality for "Maximum Length of Comment" Field

This PR fixes the issue documented in #6872

Test results after implementing the requested changes:
Scenario 1. "Maximum Length of Comment" set to a value <= -10,000,000,000,000,000,000,000,000,000,000 (32 digits or more).
Here I'm proposing to change the original error message, so instead of "Maximum Length of Comment is not numeric" we can display something like: "Input Value Out of Range. Please enter a numeric value with less than 32 digits".

Scenario 2. "Maximum Length of Comment" set to a value >= -9,999,999,999,999,999,999,999,999,999,999 AND < 1 (up to 31 digits).

Scenario 3. "Maximum Length of Comment" set to a value >= 1 AND <= 250.

Scenario 4. "Maximum Length of Comment" set to a value >= 251 AND <= 9,999,999,999,999,999,999,999,999,999,999 (up to 31 digits).

Scenario 5. "Maximum Length of Comment" set to a value >= 10,000,000,000,000,000,000,000,000,000,000 (32 digits or more). Also here I would like to use the revised version of the error message instead of the original ("Maximum Length of Comment is not numeric").

If you have any questions or comments, please let me know.
Best regards,
Francisco Milan.

Closes #6872


Feature/maali garrawarla

A first attempt at having a garrawarla specific spack configs.

Currently seems fine based on tests by Harrison. The only issue might be an update to the modules path. However, this is likely best handled manually to ensure that modules follow the flat hierarchy that exists on garrawarla.

Going to add comments here and there in the code, where relevant :)


Unable to install

Hello,
Im not able to install the addon. I followed the instructions but Im getting this error everytime:

The image is still building to docker hub. Please try again in an hour or so.

Ohh ok :D Sorry ^^


Add monitoring mode

Apparently much more complex to implement, but also much more useful is to have an ability for the program to monitor the saves folder and backup when changes (and/or on time interval).

Surely, the problem is cross-platform file & directory monitoring is a PITA. Any suggestions of libraries that would deal with that (in a portable way) would be great.

A quick search turned up https://github.com/howeyc/fsnotify, although I haven't used it before.

Two years old... but another aspect: For single-file saves watching for "last change" would probably work, but for save-directory... how would it be able to tell "ok, all has been saved, now is the time to back this up"? Unless "incremental backups"... but whole point of this tools is being simple.

You could look for files in data/save/current being deleted - DF should write them to there first, then copy them to the region folder.


GeoIP Plugin - Offer Expanded MaxMind Data

Moved from https://github.com/elastic/logstash/issues/9202 , original issue by @ubhackn
For really quick and easy security analysis of IP data, it would be helpful if the existing geoip plugin could include the GeoIP Anonymous IP database from MaxMind such that we don't need to purchase and maintain the license with them directly. Having a field like geoip.proxy_name would allow for highlighting of traffic from known proxy/anonymizer IPs (like TOR), which we can then use as an additional data point for threat detection.

from what I understand the GeoIP Anonymous IP database is a commercial offering from Maxmind, so we won't be releasing it with Logstash which is an open source project.


@OneToMany not saving foreign key column in child table

DBFlow Version: 3.0.1
Issue Kind (Bug, Question, Feature): Question/Bug?
Description:
I have two models: Country and Subdivision. Country is composed of a list of Subdivisions.
Models
Country
@Table(database = AppDatabase.class)
@ModelContainer
public class Country extends BaseModel {

    @PrimaryKey(autoincrement = true)
    @NotNull
    long id;

    @Column
    String name;

    private List<Subdivision> subdivisions;

    @OneToMany(
            methods = {OneToMany.Method.ALL},
            variableName = "subdivisions",
            isVariablePrivate = true
    )
    public List<Subdivision> getSubdivisions() {
        if (subdivisions == null || subdivisions.isEmpty()) {
            subdivisions = new Select()
                    .from(Subdivision.class)
                    .where(Subdivision_Table.country_id.eq(id))
                    .queryList();
        }
        return subdivisions;
    }

    public void setSubdivisions(List<Subdivision> subdivisions) {
        this.subdivisions = subdivisions;
    }
}

Subdivision
@Table(database = AppDatabase.class)
public class Subdivision extends BaseModel {

    @PrimaryKey(autoincrement = true)
    @NotNull
    long id;

    @Column
    String name;

    @ForeignKey(tableClass = Country.class,
            onDelete = ForeignKeyAction.CASCADE,
            onUpdate = ForeignKeyAction.CASCADE,
            saveForeignKeyModel = false)
    public ForeignKeyContainer<Country> country;
}

Test Code
When I call .save() on a Country that has Subdivisions, the Subdivision rows are created but the foreign key column (country_id) is not set. Is this a bug or am I doing something wrong?
MainActivity
    Country country = new Country();
    country.setId(1);
    country.setName("United States");

    Subdivision subdivision = new Subdivision();
    subdivision.id = 1;
    subdivision.name = "California";

    ArrayList<Subdivision> subdivisionList = new ArrayList<>();
    subdivisionList.add(subdivision);

    country.setSubdivisions(subdivisionList);

    country.save(); // Saves 1 row in Country and 3 in Subdivision. Subdivision.country_id is null on each row

Also, should the following statement return a Country with the subdivision List set, or do I have to call the getSubdivisions() method every time? I'm confused about the functionality of OneToMany.Method.ALL here.
List<Country> countries = new Select().from(Country.class).queryList();

you need to associate the Country in each subdivison:
 Subdivision subdivision = new Subdivision();
    subdivision.id = 1;
    subdivision.name = "California";

you're not setting the Country object here..ForeignKeyContainers are no longer used in 4.0.0-beta1 +, rather we use stubbedRelationship. So much easier to work with.

Does this mean that if we get an object of Country from backend we have to iterate over its list of Subdivision, add the refrence of parent country to each and then save to db? Is there a better way to do this. For me it becomes more complicated as I receive a list of parent objects, lets say countries in this example, then i have to iterate over each parent object first...


about create_images.py

Hi Swetha,
Thank you very much for sharing this code. Could you please clarify the following below
inpfile = mode + 'data'
outDir = mode + '/image_'

In above 'data' is referring to??

你好，解决了嘛？我也遇到了同样的问题

@a56256324 ..Can please let me know your comment in English translation..thanks

Oh, I'm really sorry, my friend. I mean I want to generate some data of my own, but I also have some difficulties when I run create_images.py. Have you succeeded in debugging?

I am also getting the same error when trying to create my own data from raw files. @a56256324 Did you figure out what the problem is?

I figured out the reason that create_images.py wasn't properly working. It uses the pandas dataframe what was created with do_test_train_split to find all of the raw CT scans. The dataframe that was included in the git repo contains the dataframe that was generated by the repo owner. I don't have some of the scans that are in her dataframe and therefore it can't find the files.
You can get around this issue by deleting all of the HDF5 datasets and pandas dataframes included in src/data and regenerating them from scratch using your own raw datasets. This means downloading the LUNA16 dataset or the LIDC which are massive. I suggest. you download a single scan from the cancer imaging archive and use that to test the prediction model. This also means modifying the create_images.py to preprocess the data with only a single scan. A candidates.csv is also needed for that single scan and the details you can copy from the place you downloaded the scan from.

Thank you very much, my friend.


MinMax not used when computing Surface sampling

In tofu/geom/_comp.py in the function _Ves_get_sampleS four parameters are taken and never used: Min1, Max1, Min2, Max2. They are transformed to two list, see:
https://github.com/ToFuProject/tofu/blob/a5c8cf8d43b8a43b73bee8f6688d6a1b74ea2979/tofu/geom/_comp.py#L287-L288
But, when calling core function for sampling they are never used. Since there is no doc, not sure what are this parameters ? Are they the limits for Phi ?

After discussing with @Didou09 these variables look useless, taking them out in new branch...

Fixed, in devel, by PR #181


Drift chat button covers up menu button on mobile

Describe the bug
The Drift chat button covers up the menu button when viewing the docs on a mobile browser, making it almost impossible to open the menu. This makes the docs virtually inaccessible unless you use the Previous/Next navigation on the bottom.
To Reproduce
Navigate to the docs in a mobile browser.

Expected behavior
The menu button should be accessible to enable easy navigation when viewing the site on a mobile browser. It would be enough to have the option to hide the chat button. Otherwise, one button or the other could be moved to another corner of the screen.
Environment:

OS: Chrome on Android  81.0.4044.138

Fixed :) Thanks @danielrearden !


[Mobile] About us | missing sponsor logos

env: staging
Steps:

Open About us page
Go to the Our supporters sections. Logos are missing:

@galeza I think it is not that they are missing, but that they have not loaded yet. Can you please re test, and maybe wait for a bit? I think we have to find a solution to reduce the size of this page as it is quite heavy with all the pics and logos.

@ppagarawal @galeza
Unable to reproduce the issue on iPhone n andriod devices:

Hi @idjevm
I refresh a page but still nothing.
I have an impression that they are loaded.
I'm using chrome://inspect/#devices and remote debugging:

It might be an issue with this environment, @rabbi08, @sheeba1, @galeza, @hellynyc, @nivetheatha can you all go to https://staging.fightpandemics.work/about-us from your phones please? I think this is only on this environment.

@idjevm The sponsor logos load fine in staging on my androids (OnePlus 7 Pro & Google Pixel), both running OS10.

@idjevm
No it's not environment. When I use my tablet samsung galaxy tab I see all icons.

Looks like this works fine in all devices @galeza, I think it is an issue only from this emulator or the tool you are using to test. Let's wait for @sheeba1, @nivetheatha, @rabbi08  to test out on their devices, and then we should decide whether to explore or just close this issue.

@idjevm
No, I test on a device.
It is a problem with a chrome browser. With 81 icons are not visible. I upgraded to version 83 and it works

True! OMFG great catch! Just tested on my reg browser on my phone and they didn't load :O I will have to check why is that though !

@idjevm I can able to see the logos.Mine is Samsung Galaxy

@idjevm , working fine for me  iphone XR

Working fine on iPhone 10, 11, ZTE Blade Max(Andriod), Mac with Chrome (Version 83.0.4103.116 (Official Build) (64-bit))

@rabbi08
It was seen for Chrome v.81

@galeza is this still an issue with the new version of the about us in prod?

@CamilaArias I'm closing this issue.


add support for arc.json and arc.yaml (instead of just .arc)

Recently we got a little attention on hackernews and I was a bit dismayed that much of the conversation was focused on the .arc format. Rather than defend it as the only option I'd rather present it as the best option. And hopefully the conversation can move towards the more tangible benefits! I've found we barely spend any time in .arc even though it is an important communication vehicle for the intent of an application (vs a manifest of vendor infra and non-portable nomenclature).
I briefly revisited TOML but found the lack of anon/unnamed "tables" makes it unsuitable for how we're approaching config.
Currently .arc is read only once in arc-workflows/src/utill/init; we should refactor it to look first for .arc, then arc.yaml and finally arc.json. If nothing is found it should error out.
The following files will be equivalent:
A proper .arc file:
# commenst oo ehh
@app
testapp

@domain
testapp.com

@aws
region us-west-2
profile personal

@static
staging testapp-bucket
production testapp-buckea-prod

@html
get /
post /login

@css
/index.css

@js
/js/index.js
/js/:mjs

@text
/robots.txt
/humans.txt

@json
get /api/notes
put /api/notes/:noteID
post /api/notes
delete /api/notes/:noteID
patch /api/notes/:noteID

@xml
get /index.rss

@events:
send-welcome-sms

@tables
notes:
  authorID *String
  noteID **String

authors
  authorID *String

@indexes
authors
  phone *String

@scheduled
daily-report rate(1 day)

@slack:
statsbot

An arc.yaml file (which isn't bad really…but does get a bit hairy when you arrive at tables):
---
# commenst oo ehh
app: testapp
description: Example arc-to-json
domain: testapp.com
aws:
  region: us-west-2
  profile: personal
static:
  staging: testapp-bucket
  production: testapp-buckea-prod
html:
- get: "/"
- post: "/login"
css:
- "/index.css"
js:
- "/js/index.js"
- "/js/:mjs"
text:
- "/robots.txt"
- "/humans.txt"
json:
- get: "/api/notes"
- put: "/api/notes/:noteID"
- post: "/api/notes"
- delete: "/api/notes/:noteID"
- patch: "/api/notes/:noteID"
xml:
- get: index.rss
events:
- send-welcome-sms
tables:
- notes:
    authorID: "*String"
    noteID: "**String"
- authors:
    authorID: "*String"
indexes:
- authors:
    phone: "*String"
scheduled:
  daily-report: rate(1 day)
slack:
- statsbot

…and finally the most verbose, rigid and of course lacking comments arc.json:
{
  "app": "testapp",
  "description": "Example arc-to-json",
  "domain": "testapp.com",
  "aws": {
    "region": "us-west-2",
    "profile": "personal"
  },
  "static": {
    "staging": "testapp-bucket",
    "production": "testapp-buckea-prod"
  },
  "html": [
    {
      "get": "/"
    },
    {
      "post": "/login"
    }
  ],
  "css": [
    "/index.css"
  ],
  "js": [
    "/js/index.js",
    "/js/:mjs"
  ],
  "text": [
    "/robots.txt",
    "/humans.txt"
  ],
  "json": [
    {
      "get": "/api/notes"
    },
    {
      "put": "/api/notes/:noteID"
    },
    {
      "post": "/api/notes"
    },
    {
      "delete": "/api/notes/:noteID"
    },
    {
      "patch": "/api/notes/:noteID"
    }
  ],
  "xml": [
    {
      "get": "index.rss"
    }
  ],
  "events": [
    "send-welcome-sms"
  ],
  "tables": [
    {
      "notes": {
        "authorID": "*String",
        "noteID": "**String"
      }
    },
    {
      "authors": {
        "authorID": "*String"
      }
    }
  ],
  "indexes": [
    {
      "authors": {
        "phone": "*String"
      }
    }
  ],
  "scheduled": {
    "daily-report": "rate(1 day)"
  },
  "slack": [
    "statsbot"
  ]
}

side note: h/t to @csantanapr for giving me the same feedback roughly a year ago! finally we get around to it. ;P

I'm just going to put json5 on your radar as it would give you comment support.
I believe it's the format used in VSCode's settings editor.

If it's not a major undertaking, absolutely do it. "Meet people where they are."
This may also open the door for interoperability with SAM, Serverless Framework, etc...

published in<EMAIL_ADDRESS>just need to add docs! here's the tester file I was using
---
app: testapp-ragalkasklsadlksdlklkds
aws:
  region: us-east-1
  profile: personal
events:
- one
- two
html:
- get: "/"
json:
- get: "/api"
text:
- "/hi"
queues:
- test-q


Add a connection chooser menu on the bottom panel

Konga is awesome, but there is one little thing that sets me back a lot of time: having to go into settings to choose the connection.
I use Kong in several independent clusters and it would be nice to have a quick way to alternate connections, let's say, a menu that could open when you click the current connection on the status bar. In the same sense that there are links to the github page and to the github issues page, it would be a nice feature that would immensely improve the usability of Konga.

@PauloGDPeixoto ,
Great idea! Definitely gonna apply it on the upcoming version!
Thanks!

@PauloGDPeixoto ,
check out the new release https://github.com/pantsel/konga/releases/tag/0.6.3


[AutoPR search/resource-manager] Add L-Series SKU values to management plane

Created to sync https://github.com/Azure/azure-rest-api-specs/pull/5386

(message created by the CI based on PR content)

This PR has been merged into https://github.com/Azure/azure-sdk-for-python/pull/4503


https not working

I can open the website using https, but webrtc won't initialise at all (getusermedia).
Switching to the same site using http works.

iPhone 5s. Latest version from appstore

I'm having the same problem

Any HTTPS site?

Please try to add these code in OpenWebRTCViewController.m, it will load https request without any challenge.

(void)webView:(WKWebView *)webView didReceiveAuthenticationChallenge:(NSURLAuthenticationChallenge )challenge completionHandler:(void (^)(NSURLSessionAuthChallengeDisposition, NSURLCredential * _Nullable))completionHandler
{
NSURLCredential cre = [NSURLCredential credentialForTrust:challenge.protectionSpace.serverTrust];
[challenge.sender useCredential:cre forAuthenticationChallenge:challenge];
completionHandler(NSURLSessionAuthChallengePerformDefaultHandling, cre);
}

We would be happy to accept a Pull Request from you if you have the possibility.

@Mamong I added what you suggested in this PR https://github.com/EricssonResearch/openwebrtc-ios-sdk/issues/52 but it does not solve the problem of blocked gUM.

This has now been fixed in OpenWebRTC and I have submitted a new version 0.6.1 to the App Store for review:

If you want it already now you need to build OpenWebRTC yourself.
Thanks @pererikb for the patch!


org.openrewrite.java.cleanup.Cleanup rewrites build.gradle into invalid state

As discussed in Discord two days ago with pway PFA a demo gradle build file that corrupts itself when invoking ./gradlew rewriteRun
build.gradle.demo.txt
This file shows the corruption in two places in the file - the string containing newlines has been broken (the same occurs with escaped double quotes I believe) and the * in the encoding setting is has duplicate text and question marks added.
build.gradle.corrupt.txt

Thanks @mattthebig

This has likely been fixed, can you please try this with the latest release?

Nope not even slightly fixed - still rewrites unwanted files - still breaks build.gradle
I'd focus on just getting the files to work on correct - the groovy parser is probably hard to fix but theres no way it should be being called at all from a java cleanup.


BetterBehaviour doesn't work with UnityEvent

Hi. There was an event system added to Unity 4.6 that I was interested in using called UnityEvents. It was originally just for GUI but was extended to work throughout a game. Here's a script with the event,
using UnityEngine;
using UnityEngine.Events;
using Vexe.Runtime.Types;

public class NewBehaviourScript : MonoBehaviour
{
    public UnityEvent MyEvent;
}

When I attach this script to a Game Object, I get a nice inspector list as expected. However, when I instead inherit from BetterBehaviour, the inspector shows two strange fields, "M Persistent Calls" and "M Type Name". These fields don't seem to work as the inspector does with the MonoBehaviour.
I'd like to use these UnityEvents. Are there any Vexe types that play well with UnityEvent? (I'm aware of a different event system but not one built on UnityEvent.) If not, is there a quick edit I can make to VFW that would make it so the inspector reverts to the default MonoBehaviour style for just classes inheriting from UnityEvent?
Side note: Video of this system in use in this video (skip to minute 9).
Thanks

The problem is Unity and Vfw use different gui layout systems that are not compatible. (GUILayout vs RabbitGUI) - If you look in InlineDrawer, you'll see me emulating Unity drawers for certain components. I'll see if I can find a way to tell Vfw to let a member be drawn by Unity's layout. If I can do that, the field will either appear on top or below of all the other members in the behaviour.

Fixed: https://github.com/vexe/VFW/commit/6d92c0cc37942cfc8d5e2655ba44fc65d5c26b51
Remember, you can always use VFW delegates (aka uAction, uFunc and uDelegate)

That was so fast. Thank you very much :)


Steps Responsiveness

Please always provide the GitHub issue(s) your PR is for, as well as test URLs where your change can be observed (before and after):
Fix #13, maybe Fix #5
Test URLs:

Before: https://main--{repo}--{owner}.hlx.page/
After: https://qsci--contraversumlandingpage--contraversum.hlx.page/

My initial plan was to create a separate PR once the pink background is fixed.


Versioning and naming

This ticket is about renaming projects, maven artifacts, and packages of swagger-core project and modules; a separate ticket will hold needed changes for dependent projects (as swagger-parser), following a similar pattern.
The immediate goal is adopting a naming and versioning policy allowing v2 and v3 libraries to coexist; the update will also ensure a consistent version upgrade mechanism, avoiding (transitive) dependencies issues for as much as possible.
Credits to @eed3si9n (persistent versioning) and @JakeWharton (java-interoperability-policy-for-major-version-updates) for their clear posts addressing this kind of issues.
It includes the following changes:

Rename groupdId of root to io.swagger.core.v3
Rename groupdId of swagger-annotations module to io.swagger.core.v3
Rename groupdId of swagger-models module to io.swagger.core.v3
Rename groupdId of swagger-core module to io.swagger.core.v3
Rename groupdId of swagger-integration module to io.swagger.core.v3
Rename groupdId of swagger-jaxrs2 module to io.swagger.core.v3

Rename root package of swagger-annotations module to io.swagger.v3.oas.annotations
Rename root package of swagger-models module to io.swagger.v3.oas.models
Rename root package of swagger-core module to io.swagger.v3
Rename root package of swagger-integration module to io.swagger.v3.integration
Rename root package of swagger-jaxrs2 module to io.swagger.v3.jaxrs2

Adapt POM, travis and any affected config/metadata

implemented in #2495


Simple clear database functionality

Problem
When I'm running my unit tests, I use a virtual temporary database in order to test my routes. Prisma doesn't provide (to my knowledge) no "clean" way to delete everything inside the database. After each unit test the database tables should be emptied.
Suggested solution
I would really like to see a simple function in order to delete all data from the tables, without giving a constraint error!
const prisma = new PrismaClient();
prisma.clear();

Alternatives
The only way I found to mimic this behavior is by creating a file that contains all the tables, looping through every table and delete all the records one by one. The tables must be looped in order to not get a constraint error. This is just not the way you want to go... but there's just no other way (I think?).
Additional context
It would look something like this using Jest:
afterEach(async () => {
  prisma.clear();
});

afterAll(async () => {
  prisma.$disconnect();
})

Please let me know about any alternatives! I'm dying to know! 😄

Related to: #5230 and #5231

Yeah, closing as duplicate of above noted issues.


Face Recognition

To detect and recognize specified face with name tag with help of opencv.

I would like to work on this issue. please assign it to me(GSSOC'21)


FltCreateSectionForDataScan ObjectAttributes parameter is confusing.

The statement "[in, optional] ObjectAttributes
A pointer to an OBJECT_ATTRIBUTES structure that specifies the object name and other attributes. Use the InitializeObjectAttributes macro to initialize this structure. Because FltCreateSectionForDataScan inserts this object into the process handle table, the caller must specify the OBJ_KERNEL_HANDLE attribute when it calls InitializeObjectAttributes."
I believe the default behavior is that FltCreateSectionForDataScan is to create a user handle unless OBJ_KERNEL_HANDLE is specified in the OBJECT_ATTRIBUTES parameter, as written it sort of implies that it will always be a user handle... which leaves us asking why we must specify a KERNEL_HANDLE if so....  I believe the use of 'object' is also not appropriate since it is the handle that it would insert into the respective table, the section object itself is a kernel object.

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

ID: f8e812a1-9d9a-78fb-297e-099676149507
Version Independent ID: 7765b998-7e93-417b-1672-01208a6a3058
Content: FltCreateSectionForDataScan function (fltkernel.h) - Windows drivers
Content Source: wdk-ddi-src/content/fltkernel/nf-fltkernel-fltcreatesectionfordatascan.md
Product: windows-hardware
Technology: wdk-api-reference
GitHub Login: @lorihollasch
Microsoft Alias: loriwhip

Thanks for filing this issue, @camptatopenmars - great call on needing clarity of user handles (default) and kernel handles. I've updated the ObjectAttributes and SectionHandle parameter descriptions, and added a paragraph in the Remarks.


Refactor and modernize Consignments/Welcome & Inbox empty state

Modernizing these views and move into FlatList

#skip_new_tests
Before

After

Before

After

:rocket: PR was released in v1.19.7 :rocket:


Refactoring / Code Improvements discussion

 Issue by metaprime
Wednesday May 24, 2017 at 18:34 GMT
Originally opened as https://github.com/4pr0n/ripme/issues/557

@Gaboso Let's talk about which changes need to be done, style guidelines, etc. If we're going to make a push to improve code quality, let's also strive for consistency.

@metaprime I've been out for a while, because of other projects, but now be able to follow up with greater frequency.
Exploring the project in general, I found some inconsistencies and points to improve, especially on rippers.
I believe that the initial focus for improvements should be to clean up the code and also fix issues.
To scan the code I used the SonarQube server on my machine, but to use it on the project I think it would be interesting to create an account in the SonarQube cloud.

^ What everything can SonarQube do, assuming there's no need to pay ?

It's free for open source projects.
SonarQube supports analysis of:

Bugs and code vulnerabilities.
Issues.
Coverage of code by unit tests.
Complexity of code.
Code duplication.

The paid version has as a differential, more plugins and support to other languages, besides Java, JavaScript, C, C ++ and C#.
But the ripme uses Java and is open source, so it has free access in the cloud.
Follow the SonarQube cloud link: https://about.sonarcloud.io/

^

SonarQube supports analysis of:
Issues.

What issues ?

While running an analysis, SonarQube raises an issue every time a piece of code breaks a coding rule. The set of coding rules is defined through the quality profile associated with the project.
Each issue has one of five severities:
BLOCKER
Bug with a high probability to impact the behavior of the application in production: memory leak, unclosed JDBC connection, .... The code MUST be immediately fixed.
CRITICAL
Either a bug with a low probability to impact the behavior of the application in production or an issue which represents a security flaw: empty catch block, SQL injection, ... The code MUST be immediately reviewed.
MAJOR
Quality flaw which can highly impact the developer productivity: uncovered piece of code, duplicated blocks, unused parameters, ...
MINOR
Quality flaw which can slightly impact the developer productivity: lines should not be too long, "switch" statements should have at least 3 cases, ...
INFO
Neither a bug nor a quality flaw, just a finding.

Source: https://docs.sonarqube.org/display/SONAR/Issues

^ You dunno anything the docs doesn't tell ?

In fact, I did not say anything, I just forwarded the content that explains how SonarQube handles an issue and its internal ratings. They probably explained better than I did.
Attached is a print of an analysis performed on the ripme:

^ Those are orders of magnitude worse things than my extremely limited Java understanding and knowledge had. I mean, I knew RipMe could use refactoring and improving (and I even have some things in mind) but those things are, like, really bad in comparison.
Just, wow.

Any news on this ? I'm mainly doing code cleanup for now and this would be really useful.


Merge main into release-11.0

Merged main into release-11.0.
#no-changelog

Please investigate https://github.com/firebase/firebase-ios-sdk/actions/runs/9619022396/job/26534197765?pr=13169#step:8:52


typo

Please correct "Service Principle" to be "Service Principal".

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

ID: 4617d07d-6e59-43af-6652-6409298f0957
Version Independent ID: c9681b64-9af4-c627-925a-6ccd8aa59607
Content: Add a service principle to Azure Analysis Services server admin role
Content Source: articles/analysis-services/analysis-services-addservprinc-admins.md
Service: analysis-services
GitHub Login: @Minewiskan
Microsoft Alias: owend

@gilherau Thanks for the feedback! I have assigned the issue to the content author to evaluate and update as appropriate.

@gilherau We will now proceed to close this thread. If there are further questions regarding this matter, please reopen it and we will gladly continue the discussion.


WIP: SpringBoot v2 enabler sample config: Working, but needs trimming…

… and compacting.

So far the submitted changes are not appropriate for a review. Btw, who is pinpan?


Link error in Gigasecond

Hi, I recently discovered Exercism and have become somewhat addicted.  I wasn't able to finish the C++ Gigasecond exercise because it can't find the required Boost library:
make[2]: *** No rule to make target '/usr/lib/libboost_date_time.a', needed by 'gigasecond'.  Stop.
make[1]: *** [CMakeFiles/Makefile2:77: CMakeFiles/gigasecond.dir/all] Error 2
make: *** [Makefile:84: all] Error 2
I am working in the browser editor.
I  skipped Meetup also because it depends on Boost.  Wouldn't it make sense to use  the date and time functions in the standard library instead of Boost?

I'm trying to sort this out in https://github.com/exercism/exercism/issues/6332. I'm not sure what's going on, but the exercise should work in the online editor since #495 was checked in back in December. I've confirmed it works starting from a new account, but maybe you downloaded the gigasecond exercise before last December?
The exercises use Boost because we haven't updated the repo to C++20, where the date time utilities in the stdlib provide a decent interface. And because I didn't want to bother copying over the C tracks solutions here when I ripped Boost out of the rest of the track, since the solutions would be identical.
Considering complaints about Boost are still the largest single source of issue with the track, we should probably try to do that sooner rather than later.
The Boost dependency is 100% of the reason why these exercises are marked as optional.

Thanks for the response.   I think I must have first tried running the tests shortly before #495 went in.  I have subsequently found that if I submit from my own machine, the solution goes through without any problem.


BUG: Specify CMAKE_GENERATOR for Eigen external project build

On Windows, with the Ninja Generator and a non-default Visual Studio
toolset, the Eigen External Project will fail to build, attempting to
use MSBuild, without the generator specified.

Good fix, that's the only caveat of execute_process.
It would be great if CMake refactors all the logic used in ExternalProject_add to handle toolchains and generators into some kind of variable i.e ${CMAKE_TOOLCHAIN_VARIABLES} or similar.
The good thing is that we might only need to worry about the generator in this case, because we are not really compiling anything.


removed ::before html element for the kill-btn

When I hovered on the kill-btn there was a tiny jump on the col-action tag child elements. I identified that it was caused by ::before element. It was also placing the entire button behind itself blocking the view. Removing it seems to fix both issues hence the PR

@outranker thank you.


\node_modules\babel-runtime\node_modules\core-js: Command failed.

Exit code: 1
Command: node -e "try{require('./postinstall')}catch(e){}"
Arguments:
Directory: E:\Test\react\todolist\my-app\node_modules\babel-runtime\node_modules\core-js
Output:

More info please. Please see Bug Report for writing a better bug report

issure +1

有解决方案嘛


migrate (back) to TypeScript

Related Issue
resolves #43
Summary of Changes

Converted all components
Converted all services and established types
Converted all pages

TODOs

[x] Convert all components / remove greeting
[x] Convert all services and define types (Post, Event, Album, Artist, Card)
[x] Convert all pages
[x] TS Linting (via ESLint)
[x] Validate bundle output / performance stays the same
[ ] Sourcemaps / feedback in development - https://github.com/ProjectEvergreen/greenwood/issues/661
[x] more compilerOptions / IDE settings

handle decorators (just needed to make sure VS Code was using project version of TS)
.css extension
no .ts. imports

[x] Investigate build logs (fail build / get them in dev)? - Looks like no way around .ts? - https://github.com/Microsoft/TypeScript/issues/27481 and https://github.com/microsoft/TypeScript/issues/37582@rollup/plugin-typescript TS2307: Cannot find module './events-calendar.css?type=css' or its corresponding type declarations.
@rollup/plugin-typescript TS2417: Class static side 'typeof EventsCalendarComponent' incorrectly extends base class static side 'typeof LitElement'.
Types of property 'properties' are incompatible.
    Type '() => { events: { type: ArrayConstructor; }; currentMonthIndex: { type: NumberConstructor; }; currentYear: { type: NumberConstructor; }; }' is not assignable to type 'PropertyDeclarations'.
  Index signature for type 'string' is missing in type '() => { events: { type: ArrayConstructor; }; currentMonthIndex: { type: NumberConstructor; }; currentYear: { type: NumberConstructor; }; }'.
@rollup/plugin-typescript TS2307: Cannot find module './footer.css?type=css' or its corresponding type declarations.
@rollup/plugin-typescript TS2307: Cannot find module './header.css?type=css' or its corresponding type declarations.
@rollup/plugin-typescript TS2307: Cannot find module './navigation.css?type=css' or its corresponding type declarations.
@rollup/plugin-typescript TS2307: Cannot find module './posts-list.css?type=css' or its corresponding type declarations.
@rollup/plugin-typescript TS2417: Class static side 'typeof PostsListComponent' incorrectly extends base class static side 'typeof LitElement'.
Types of property 'properties' are incompatible.
Type '() => { max: { type: NumberConstructor; }; posts: { type: ArrayConstructor; }; }' is not assignable to type 'PropertyDeclarations'.
  Index signature for type 'string' is missing in type '() => { max: { type: NumberConstructor; }; posts: { type: ArrayConstructor; }; }'.

[ ] Sweep all files for typings, protected, return, TemplateResult, etc

## Related Issue
POC for a TypeScript plugin for Greenwood

Summary of Changes

Rolled own TypeScript plugin (for now) with a "greeting" component example pulled from the lit.dev docs

This will definitely break until https://github.com/ProjectEvergreen/greenwood/pull/649 is merged

TODO

[x] Wait for https://github.com/ProjectEvergreen/greenwood/pull/649 to get merged
[x] Wait for upstream implementation for TypeScript support for Greenwood - https://github.com/ProjectEvergreen/greenwood/issues/647
[x] Reuse Greenwood plugin with WTR
[x] Unit testing support (pending) - https://github.com/modernweb-dev/web/issues/1376#issuecomment-873616337
[x] Storybook testing
[x] Known issue https://github.com/ProjectEvergreen/greenwood/issues/661

Looks like bundle size is pretty much the same, maybe an extra 2KB with TypeScript
before

after

@aholtzman
Are you familiar with / have used TypeScript?  Was just was curious if you had and if this looked good to you and / or wouldn't be a burden on contributing if it was merged.

I do use typescript at work. I can take a look in the next day or so. And I can contribute with it is ts

Thanks for taking a look!  I haven't used it regularly in a while so just wanted to get a second opinion before going all in.  👍


Reconsider or refine dependabot updates

There are currently 5 PRs attempting to update the requirements that are failing because coldfront has pinned requirements.
We should prevent Dependabot from trying to update secondary dependancies like this. This is a general pain point with using Dependabot with pip-tools, so it'd be nice to work out how to configure that.

I am considering using Renovate instead of Dependabot. They have a pip-compile-specific dependency manager
Thoughts @cc-a

Worth a go I think.

Kind of fell at the first hurdle with this. I tried to configure the Renovate GitHub App for this repo and I wasn't able to get it to access the repo. I had to request access, and I thought that since I'm an admin for the repo, I would be able to grant the access, but I was not. Did you see anything @cc-a ?


possible to turn this into a pandoc filter?

Is it possible to turn the preprocessor into a pandoc filter?
There is a filter to produce criticmarkup from docx comments [1]. With the preprocessor I only get HTML output, but it would be nice to do the roundtrip with docx. As a filter one would be compatible with the rest of pandoc option.
[1] https://gist.github.com/noamross/12e67a8d8d1fb71c4669cd1ceb9bbcf9

I'm sure you can make the opposite flow work, but probably not neatly as a single filter. You'll need this or a similar pre-processor to convert the CriticMarkup to inline spans or something else that has AST coverage in Pandoc, then use a filter to massage the AST into whatever the docx Writer component needs it to be.

c.f. https://github.com/jgm/pandoc/issues/1560#issuecomment-862325091 (I know it is also by you but just for future reference)
In short this is out of scope. I agree with @alerque that a better design may be preprocessor+filter s.t. the CriticMarkup is turn into some native AST structure first and have a filter to further process it, that enables someone to write a filter that work with CriticMarkup/pancritic.
There's a way to make the pancritic cli to be "compatible with the rest of pandoc option", i.e. just use some trickery to get arbitrary args and pass it to pandoc. I thought about it but didn't have time to implement it as it is not that important (as it can be used as an explicit pre-processor and pipe it to pandoc.)
The problem with CriticMarkup is that it is supposed to markup the source, which should be orthogonal to the markdown syntax. So it isn't nicely fit into the pandoc AST model. The original CriticMarkup team acknowledge this limitation (such as CriticMarkup breaking markdown syntax boundary) and never find a way to properly fix it. To me, CriticMarkup is more like an abandonware.
It would be nice if we can make CriticMarkup to work with docx. But I use docx very rarely so it is unlikely I would invest time into this. To future self or somebody else: see https://pandoc.org/MANUAL.html#reader-options --track-changes.
Also, according to Pandoc Extras · jgm/pandoc Wiki, there's PanDiff.


Add Field Level Help and Inline Help variants to Input

This implements the "Field level help" and "Inline help" variants for Input.

@interactivellama can you take a look?  Thanks!
Fixes https://github.com/salesforce/design-system-react/issues/1474
Additional description

CONTRIBUTOR checklist (do not remove)
Please complete for every pull request

[ ] First-time contributors should sign the Contributor License Agreement.  If you haven't before, wait a few minutes and a bot will comment on this pull request with instructions.
[ ] npm run lint:fix has been run and linting passes.
[ ] Mocha, Jest (Storyshots), and components/component-docs.json CI tests pass (npm test).
[ ] Tests have been added for new props to prevent regressions in the future. See readme.
[ ] Review the appropriate Storybook stories. Open http://localhost:9001/.
[ ] The Accessibility panel of each Storybook story has 0 violations (aXe). Open http://localhost:9001/.
[ ] Review tests are passing in the browser. Open http://localhost:8001/.
[ ] Review markup conforms to SLDS by looking at DOM snapshot strings.

REVIEWER checklist (do not remove)

[ ] TravisCI tests pass. This includes linting, Mocha, Jest, Storyshots, and components/component-docs.json tests.
[ ] Tests have been added for new props to prevent regressions in the future. See readme.
[ ] Review the appropriate Storybook stories. Open http://localhost:9001/.
[ ] The Accessibility panel of each Storybook story has 0 violations (aXe). Open http://localhost:9001/.
[ ] Review tests are passing in the browser. Open http://localhost:8001/.
[ ] Review markup conforms to SLDS by looking at DOM snapshot strings.
[ ] Add year-first date and commit SHA to last-slds-markup-review in package.json and push.
[ ] Request a review of the deployed Heroku app by the Salesforce UX Accessibility Team.
[ ] Add year-first review date, and commit SHA, last-accessibility-review, to package.json and push.
[ ] While the contributor's branch is checked out, run npm run local-update within locally cloned site repo to confirm the site will function correctly at the next release.

@interactivellama check out the changes based on your comments here:
https://github.com/salesforce/design-system-react/pull/1491/commits/18df0beeb6c56e1857a5bb91b2b7a005d9782c38
thanks!

Looks good! Thanks @garygong


UV-K5 HF 0.5 (Fullband receive version)

Hello! I really liked your idea! I developed a mod board 4732, now the radio can be received using a standard connector with a 280mm antenna. You can watch the video on my channel.
https://youtube.com/shorts/q_5cJCiE34M?si=ByEecNyiQW6IKz49
I would like you to make a couple of corrections to the firmware code. Please disable RSSI polling, it interferes with sound reception tick tick tick, in SSB it shows everything incorrectly. And change the parameter "Property 0x3103. AM_AUTOMATIC_VOLUME_CONTROL_MAX_GAIN" 48 dB
Thank you for your understanding, I hope you will respond. I will post the gerberas and the board diagram after all the tests. I hope the clicking stops interfering. Best regards, UD3SBV

Hi UD3SVB

Please disable RSSI polling, it interferes with sound reception tick tick tick, in SSB it shows everything incorrectly.
And change the parameter "Property 0x3103. AM_AUTOMATIC_VOLUME_CONTROL_MAX_GAIN" 48 dB

Do you have a fork of such modified firmware to test?
Thanks and 73,
Vojtech OK1IAK


Cornstover biorefinery: sub-system TEA

Description
In using the bst.TEA.like function, sub-system TEA inherit all settings of the given TEA, which is fine for most of the parts, but OSBL_units, and labor_cost should be adjusted based on the sub-system, not copied from the given TEA.
This is most evident for the Area100 TEA, which only contains the FeedstockHandling unit and shouldn't have any installation cost associated with it, but

Environment
OS: macOS Catalina 10.15.3
python: v3.7.6
biosteam: v2.12.0
biorefineries: v2.9.0

The "Area" TEA objects are not used for performing techno-economic analysis. They are only used to get installation cost for making plots. Please ignore the "Area" TEA objects, they are just for internal use.

Yes I figured, indeed that doesn't affect calculation, thanks!
A side note, in the model.py script for doing system evaluation, there are:
def cooling_duty_function(tea):
    heat_utilities = sum([i.heat_utilities for i in tea.units if i._N_heat_utilities], ())
    cooling_utilities = [i for i in heat_utilities if i.duty < 0]
    return lambda: sum([i.duty for i in cooling_utilities])

for i, tea in enumerate(areas, 1):
    Area = f'Area {i}00'
    metrics.extend(
        (Metric('Electricity', electricity_rate_function(tea), 'MW', Area),
         Metric('Cooling duty', cooling_duty_function(tea), 'MMkcal/hr', Area),
         Metric('Installation cost', installation_cost_function(tea), '10^6 USD', Area)))

I think the unit for 'Cooling duty' should be MMkJ/hr as opposed to MMkcal/hr?

Thanks!

Ahh, yeah, @scyjth let me know that the cooling duty may be wrong. I think you found the error. I didn't actually use the cooling duty results myself so I didn't notice. Thanks!

😃 Yeah!

Well, turns out in the reply below the cooling_duty should be in kJ/hr (not MMkJ/hr) and installation_cost should be in USD (not 10^6 USD). Thanks @scyjth for pointing this out! (I hope I get it right this time)

Yes I figured, indeed that doesn't affect calculation, thanks!
A side note, in the model.py script for doing system evaluation, there are:
def cooling_duty_function(tea):
    heat_utilities = sum([i.heat_utilities for i in tea.units if i._N_heat_utilities], ())
    cooling_utilities = [i for i in heat_utilities if i.duty < 0]
    return lambda: sum([i.duty for i in cooling_utilities])

for i, tea in enumerate(areas, 1):
    Area = f'Area {i}00'
    metrics.extend(
        (Metric('Electricity', electricity_rate_function(tea), 'MW', Area),
         Metric('Cooling duty', cooling_duty_function(tea), 'MMkcal/hr', Area),
         Metric('Installation cost', installation_cost_function(tea), '10^6 USD', Area)))

I think the unit for 'Cooling duty' should be MMkJ/hr as opposed to MMkcal/hr?

Thanks!


Enable bandwidths for scalars and timeseries

A new OEDatamodel version shall be developed, which allows bandwidths for scalars and timeseries per scenario.
IMO this needs three implementation changes:

value in scalars must be changed to hold bandwidths (allowing fixed, discrete and continuous values)
multiple timeseries regarding same region/tech/etc. are allowed, but must be distinguishable by user
a concrete instance of the scenario using fixed values and fixed timeseries from scenario with bandwiths must be build

Possible implementations can be discussed here.

My suggestion for task one (bandwidths in scalars), is to change type of column value (now float) into list-of-floats and to add column bandwith_type (or different name) of type string containing one of fixed/discrete/continuous.

Regarding task two, we could use tag column for explanations about the timeseries

Task three could be implemented, by adding three new tables scenario_instance, scalar_instances, timeseries_instances (search for better names!) with following functions:

scenario_instance simply contains PK and FK to instantiated scenario
scalar_instances contains columns PK, FK to scenario_instance, FK to scalar (with bandwiths) and value (float) holding a valid value within bandwidth of related scalar entry
timeseries_instances contains PK, FK to scenario_instance and FK to timeseries

From those tables all needed information for one reference scenario can be gathered...

Great initiative!
Regarding 1., I suggest to allow for fixed scalars, lists of scalars and distributions. Distributions can be specified by a finite number of parameters, i.e. a uniform distribution between an upper and lower bound, a Gaussian characterized by mean and variance etc. Together with a sampling method, this is enough information such that concrete scenario instances can be produced.

Hi @jnnr
Regarding 1., @srhbrnds, @henhuy and I discussed your suggestion to "specify distributions by a finite number of parameters" and were wondering what the use-case of this could be? Do I understand you correctly that in essence, it wouldn't change henhuy suggestion other than allowing more values of type string other than fixed/discrete/continuous, for example Gaussian. I suppose in that case somewhere would need to be defined in which order mean, variance, etc. are given.

Hi @chrwm!
The usecase would be the similar to the usecase of that feature in general: Support creation of several concrete scenarios by specifing some higher-level parameters. This can be applied in scenario comparison and sensitivity analysis.
You are right, this is an extension of @henhuy s proposal. In fact, I wonder what "discrete" or "continous" should mean without defining some distribution.
To make it a bit more concrete:

fixed: pass a single value
discrete: pass a list of values
continous: pass parameters of a distribution, e.g.

uniform: lower_bound, upper_bound
gaussian: mean, variance
etc.

The bandwidth_types will be based on NetCDF conventions and extended with custom conventions from the SEDOS project. Your proposals and definitions can be included as the latter.

@chrwm from 2022-07-28 AP1 meeting:

in the case of performance issues due to many timeseries per csv-table consider ANSI-encoding instead of UTF-8
then make sure to address encoding errors script-based


E_LIGHTNING is specified instead of E_THUNDER under the help documentation for conditional effects.

Thank you for all of your great work lyrgard!


Refactor the way request is handled: function to method

Codecov Report

Merging #12 into master will increase coverage by 1.53%.
The diff coverage is 67.25%.

@@            Coverage Diff             @@
##           master      #12      +/-   ##
==========================================
+ Coverage   61.06%   62.59%   +1.53%
==========================================
  Files           2        2
  Lines         113      131      +18
==========================================
+ Hits           69       82      +13
- Misses         37       41       +4
- Partials        7        8       +1

Impacted Files
Coverage Δ

server/server.go
66.95% <67.25%> (ø)

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 a87c52d...d3f3bd1. Read the comment docs.


Issue with parsing pipe symbol inside of the cell

We have faced an issue when the table cell contains text with the pipe symbol |.
According to gherkin documentation, we are trying to escape it \| but prettier removes \ end we are getting an error
Row must have the same cells count as table's header
Code example:
| element                  | should           | value                                                       |
| all select fields list   | have length      | 5                                                           |
| select audit             | contain text     | 79636 \| B - 10/10/2017 \| some social audit description #1 |

Prettier version 3.2.5

Hi,
This may be a known issue, I think that you can read the documentation about escapeBackslashes : there is a difference between the behat parser and the gherkin documentation and the cucumber implementation.
To be honest, I do not remember exactly what you should do in your case, but you may want to set it to true or try to escape your pipe twice : \\|.

@ext-rhorodechnyi

We have faced an issue when the table cell contains text with the pipe symbol |. According to gherkin documentation, we are trying to escape it \| but prettier removes \ end we are getting an error Row must have the same cells count as table's header  Code example:
| element                  | should           | value                                                       |
| all select fields list   | have length      | 5                                                           |
| select audit             | contain text     | 79636 \| B - 10/10/2017 \| some social audit description #1 |

Prettier version 3.2.5

were you able to find a solution to this?
I have a similar issue where escaping multiple pipe symbols in one field:
| firstColumn  | secondColumn | value                    |
| foo          | bar          | a\| b\| c \|             |

is being formatted to:
| firstColumn  | secondColumn  | value               |
| foo          | bar           | a\| b| c|           |

and with escapeBackslashes set to true it just changes to a\\| b| c| .
So after formatting, running it again leaves error inconsistent cell count within the table, because of unescaped pipes.
Escaping with double backslash like this a\\| b\\| c \\|  leave me also with inconsistent cell count error.
How can multiple pipe symbols be escaped then?

Sorry, I did miss the point where several pipes are escaped !
I'm looking at it right now.

The muli-escaped pipes bug has been fixed in 3.1.0.


Users who never got a promotion do not have a rank

If no rank records were created for a user, the user will have no rank,
For these users, add a rank record with their current rank. Probably set the date to the user's enlistment date.

Will be fixed in next release.


[App]: Init function jumbles the order of configuration parameters in dictionary

Current Behavior
Hello
I'm using Hydra and Weights & Biases to configure my application and log all the configuration parameters. Hydra produces nested dictionaries which wandb doesn't properly process, so I use the function _flatten_dict from the link below, to flatten the dictionary before passing it to wandb.
https://github.com/PyTorchLightning/pytorch-lightning/blob/87bd54aedfcd990b935860d020ca89d192e2ba40/pytorch_lightning/utilities/logger.py
I've updated the code above a bit to return a sorted dictionary by key. I then log the parameters as follows:
wandb.init(project=cfg.project_name, config=_flatten_dict(cfg), mode=cfg.logger.mode)
If I print the keys, values in _flatten_dict(cfg) I get what I expect, which is a dictionary sorted by the key. However, once it is logged in wandb the order is completely jumbled and I have parameters all over the place, which makes it very difficult to see what is happening. Furthermore, there is no option to sort by key on the wandb UI.
Expected Behavior
Passing a sorted dictionary to wandb should keep the ordering and present it as such. Also there should be a way to sort keys/values on the actual UI.
Steps To Reproduce

Produce a big dict of configs such as:
mydict = {"data": "mnist", "data/image_size": 32, "model": resnet, "model/output_dim": 256, "logger": "wandb", ...}

log it with:
wandb.init(project="TEST", config=mydict)

See the results on the UI

Screenshots
No response
Environment
OS: Unix/Linux
Browsers:
Version:
Additional Context
No response

Hi, thank you for the response. Unfortunately, on my project they don't even seem to be in reverse alphabetical order - they just seem to be kind of random? Thanks for increasing priority.

Hello,
It looks like this was fixed as now the configs seem to be sorted and categorised, thank you. Here's the link to my project, though it is private:
https://wandb.ai/azinonos/byol_video

i'm wondering if this issue has been solved since i also encounted the same issue here.


Add a signature message to the xrefs proto. [phab:D1162]

Created by zarko at 2016-11-28 21:32:40:
This amends xref.proto to use Signature messages instead
of Printable messages for marked-up pieces of code.
Printables should still be used for prose (like documentation
text). Signatures will replace formats.

zarko wrote at 2016-12-01 01:11:26:

Rename Signature to MarkedSource

fromberger accepted this revision at 2016-12-01 16:10:34

zarko closed via commit d20ca2720f4d68dc07677f1cf06618aecd577ae6 at 2016-12-01 17:26:00


fabtools.system.distrib_codename() is a missing command on debian stock

Using fabtools.system.distrib_codename() doesn't seem to work on Debian wheezy, at least a stock install.

Fatal error: run() received nonzero return code 127 while executing!
Requested: lsb_release --codename --short
Executed: /bin/bash -l -c "lsb_release --codename --short"
=============================== Standard output ===============================
/bin/bash: lsb_release: command not found
Aborting.

Looks like VPS providers often customize their images. I've been surprised more than once by these customizations. (For example, it looks like OVH ships Ubuntu 14.04 VPS's with APT's crontab entry disabled, preventing automatic security updates to work...)


cherry pick radeon note

Hang on.  Better wait on this.  Closing for now to avoid accidental merge.


Replace a bash command with "stat" module to check if python exists

This task won't be marked as "changed" on every run.

Hi, have you tested this with Ubuntu 16?
The only reason I had to use the raw module was that Ansible expects (by default) the python binary as python or /usr/bin/python while there is no such thing on Ubuntu16 (only python3).
Hence why using the raw module and a bash command, which do not require any python.
If this works now in Ubuntu 16 then we can remove the entire section.

Yes, I'm actually only running this on Ubuntu 16.04 because this is a requirement for HDP.
Of course, on a fresh server I first have to install python for ansible to work at all:
# apt install python
But then it's in the expected path:
# cat /etc/lsb-release
DISTRIB_ID=Ubuntu
DISTRIB_RELEASE=16.04
DISTRIB_CODENAME=xenial
DISTRIB_DESCRIPTION="Ubuntu 16.04.5 LTS"

# which python
/usr/bin/python

Yes, of course if you do apt install python before using Ansible it will work :) since it will install Python2.7.
The idea here is that these playbooks should be able to run on freshly built cloud servers, with Ansible being the first time something actually logged on and did stuff.
And the raw module was the way chosen to handle this, since it does not use python. Therefore these playbooks work even if you don't do apt install python because of this raw module.
As per ansible docs, the ansible_python_interpreter needs to be set to /usr/bin/python3 and this is what the above code is trying to do, before actually doing any Ansible work in python.
The only possible improvement I can see here so that it doesn't check the python binary all the time is to put all that section under a block with when: ansible_python_interpreter is defined so that if you ever set this variable in your ansible.cfg or inventory, it will not try to execute those checks.
But by default, even with the above improvement, the behaviour will still be the same since since I cannot set this variable by default as I don't know the environment on which these playbooks will run and these should run on all the supported OS-es, python2 or 3 or not.

OK, thanks for the explanation! Then this pull request doesn't make sense indeed. Plus, I forgot to update "default_python3_path.stdout" check later on.

Yes, because mainly it will break deployments on freshly built ubuntu16+ (without python2 installed by default).
I also don't like using raw, but in this case it has a very specific and useful purpose: to detect which python path to use before executing any Ansible python modules.
On the other hand, this might cause a rare bug when ansible_python_interpreter is already set to something else.
So I've commited https://github.com/hortonworks/ansible-hortonworks/commit/88a4208c8c863e2e774ee5c17d5d2e4cd233bfd3 which should also allow you to skip those checks as long as you set the ansible_python_interpreter in the inventory or via extra-vars:
./install_cluster.sh -e 'ansible_python_interpreter=/usr/bin/python3'


Can't run Spark 2.7.2 (eclipse / maven)

Created new project on Eclipse, spark-core 2.7.2 dependency to pom.xml. Running main with a get route stops with an exception (below). So, the example app won't run. Everything works fine when changing the spark-core version to 2.7.1.
Exception in thread "Thread-0" java.lang.NoClassDefFoundError: org/eclipse/jetty/http/HttpField 	at spark.embeddedserver.jetty.JettyServer.create(JettyServer.java:46) 	at spark.embeddedserver.jetty.EmbeddedJettyServer.ignite(EmbeddedJettyServer.java:100) 	at spark.Service.lambda$init$2(Service.java:549) 	at java.lang.Thread.run(Thread.java:745) Caused by: java.lang.ClassNotFoundException: org.eclipse.jetty.http.HttpField 	at java.net.URLClassLoader.findClass(URLClassLoader.java:381) 	at java.lang.ClassLoader.loadClass(ClassLoader.java:424) 	at sun.misc.Launcher$AppClassLoader.loadClass(Launcher.java:331) 	at java.lang.ClassLoader.loadClass(ClassLoader.java:357) 	... 4 more
Same thing happens when updating the spark-core to 2.7.2 on my other spark projects.

Hmh, can't reproduce this on my MacOS env, so it has to be something screwed within my Win/eclipse/jetty or so. Closing the issue..

Anyone bumping into same issue, I had to delete the local jetty-http and jett-io jars used by spark-core 2.7.2, and force update the maven project. This on Windows only. I suppose the jars were corrupted, wouldn't be the first time.

just update jetty to latest version. make sure no duplicate versions exists.


[3.x][GDNative] Expose String::join() over to GDNative Core API v1.3

Expose method join of the class String to GDNative.
This method was already implemented before and it was exposed to GDScript in the PR #56369.
In this PR it's just exposed over to GDNative Core API v1.3.
This PR is logically related to #55826. The proposal for this specific change was discussed here

Thanks!


Winform application System.EntryPointNotFoundException

I have followed the videos and instructions and yet I receive the following when the dll attempts to initialize:
'Unable to find an entry point named 'FPDF_InitEx' in DLL 'pdfium.dll'.'
This is a .Net 4.62 application running in x86 -- follwing is the simplistic code:
using System;
using System.Collections.Generic;
using System.ComponentModel;
using System.Data;
using System.Drawing;
using System.Linq;
using System.Text;
using System.Threading.Tasks;
using System.Windows.Forms;
using Patagames.Pdf.Net;
namespace SecurChekCompanion
{
public partial class roiViewer : Form
{
public Byte[] pdfByte;
public roiViewer()
{
InitializeComponent();
}
    private void roiViewer_Load(object sender, EventArgs e)
    {
        PdfCommon.Initialize();
        pdfViewer.LoadDocument(pdfByte);
    }

Thank you in advance...

I had the same issue and I managed to solve it.
This is what fixed it for me: https://github.com/pvginkel/PdfiumViewer/issues/154
Additional information :
This library does not have the native PDFium libraries which are required.
Here's the link to download the builds you need. https://github.com/pvginkel/PdfiumBuild
From this link, if you're targeting "Any CPU" make sure you have the 32-bit architecture installed from. If you need 64, download the 64-bit architecture. If you need both, download and install both.
That should solve it not running. That's what solved it for me. I also wasn't aware it was separate, but I guess I should have read it more carefully 😅


fixed Target, so it can be set with rules_file

Should be fixed by #43 - pleas re-open if required.


Ensure MESSAGE.enforce() runs for MESSAGE_MACRO

Closes #591 by ensuring that the MESSAGE.enforce() method, ensuring consistency of internal sets and parameters, is called when a scenario is solved using the MESSAGE_MACRO class.
Note that this demonstrates the workflow described at https://github.com/iiasa/message_ix/pull/603#issuecomment-1123486549:

Author opens a PR with some commits. Status in our tracker is "In progress".
Identifying that their PR ~touches the GAMS implementation~ involves the nightly tests, author pushes a commit uncommenting these lines: https://github.com/iiasa/message_ix/blob/b572ccee45c293a88f1f2acf91a0811d50cf6593/.github/workflows/nightly.yaml#L6-L8 with a message like TEMPORARY Enable nightly workflow for PR.
Author develops their PR, pushing commits, noting and ensuring that the "pytest", "lint" and "nightly" checks pass.
Author requests a review; status is "In review".
Reviewer does the review, possibly requesting changes.
Review and author interact until reviewer is satisfied.
Reviewer approves.
Author or maintainer performs git rebase -i on the branch to drop the "TEMPORARY" commit from step 2.
Merge.

How to review

Read the diff and note that the CI checks all pass.
Run specific scenarios as described at https://github.com/iiasa/message_ix/issues/591#issuecomment-1136024937 and ensure the behaviour remains the same as prior to the merge of #561

PR checklist

[x] Continuous integration checks all ✅
[x] Add or expand tests; coverage checks both ✅
~Add, expand, or update documentation.~ N/A, fix bug to restore intended/documented behaviour
[x] Update release notes.

I've restarted one flaky test (failure is the in the R notebook, so unrelated), but otherwise this is complete. So now I'll request a review, with both the failing and passing runs visible.

LGTM - elegant solution to reduce code duplication, but does introduce the diamond problem. For now let's just note that here. I highly doubt we will ever run into it given we do not touch these classes often. A pythonic resolution to this for future searchers can be found here (if it ever becomes a problem).
Happy to merge as is - should we ignore the tutorial failure?

@gidden —thanks for the review, but the merge was a little premature.
Like the comments/description said, I was trying to demonstrate the simple workflow for running the nightly tests on a PR branch. Now we've skipped from step (7) to (9), missing (8), so changes from the "TEMPORARY" commit now appear on main.
I'll revert and see if I can fix.

Done via #614.


Python KeyError at final segmentation: Manually initialized 'segmentations' in config file

I ran into the following issue while running: run.py -t nuc_it00  (running nucleus segmentation after annotation of gt_cube)
The prompted Error is:
Traceback (most recent call last):
File "/home/rheinnec/miniconda3/envs/cebra-em-env/bin/run.py", line 7, in
exec(compile(f.read(), file, 'exec'))
File "/home/rheinnec/repos/CebraEM/cebra-em/bin/run.py", line 325, in
run(
File "/home/rheinnec/repos/CebraEM/cebra-em/bin/run.py", line 211, in run
if bm not in config_seg['segmentations'].keys():
KeyError: 'segmentations'
I saw that in config/config_nuc_it00.json there is no 'segmentations' entry, which I think the script is looking for.
I think actually this is only needed to check if it is already there and otherwise it will be created. So I included a dummy-segmentation entry in the config file (or just copied and renamed another dictonary entry):
"segmentations": {
"cpuxxxs": 1
}
After that the segmentation started and is currently running.
I am not sure if it is again on my side that there is no entry, but also creating another iteration for nuc (nuc_it01) with init_segmentation.py and linking the ground thruths with link_gt.py did not add an 'segmentations' entry to the config. Maybe i am missing another function call?
Best wishes!
Marco

@mrheinnecker
Hi Marco,
I was wondering if you managed to complete the whole CebraEM workflow. I am encountering some issues. I would be really grateful if you could let me know.
Best wishes,
Bianca

Yes i managed to run it.


Reimplement moulds

Moulds are only disabled because fixing them for multiplayer was more effort than felt worth it to FC, fixing them for multiplayer could mean the return of these.
BTW-Community/BTW-Source#6

Id also actually use them if they stayed in the anvil so I could shift click the resources in without having to make stacks and stacks of moulds. Maybe make them not stack also